How to use?

I used catkin_make to build the package successfully, but it failed to launch. Where was I wrong?
pi@raspberrypi:~ $ roslaunch obstacle_cluster_detection /home/pi/catkin_ws/src/obstacle_cluster_detection/launch/ClusterCounter.launch
RLException: [/home/pi/catkin_ws/src/obstacle_cluster_detection/launch/ClusterCounter.launch] is neither a launch file in package [obstacle_cluster_detection] nor is [obstacle_cluster_detection] a launch file name
The traceback for the exception was written to the log file
pi@raspberrypi:~ $ roslaunch cluster_counter /home/pi/catkin_ws/src/obstacle_cluster_detection/launch/ClusterCounter.launch
RLException: [/home/pi/catkin_ws/src/obstacle_cluster_detection/launch/ClusterCounter.launch] is neither a launch file in package [cluster_counter] nor is [cluster_counter] a launch file name
The traceback for the exception was written to the log file

Sorry, the launch file does not need to be an absolute path.


Setting Characteristic to 0 on Character Sheet doesn't do anything

To revert to 0 or the Species baseline you currently have to clear the value without entering anything.
If you replace the current value with 0 it does nothing, and instead reverts to the value previously entered.

Will be in alpha 6


Add missing history package

- Summary
With a fresh clone of master npm run build gives me the error:
ERROR in ./routing/history.js
Module not found: Error: Cannot resolve module 'history' in /Users/Joseph/Workspace/netlify-cms/src/routing
 @ ./routing/history.js 8:15-33

- Test plan
Before: builds with error.
After: no error when building.
- Description for the changelog
Added missing history package for createHashHistory in history.js.

This was with npm install, with yarn it works fine first time.

No, you're right - we're explicitly importing from the history package, it should be a dep. Good catch.

Changed to ^2.1.2

Travis failed due to an authentication issue, might want to retry?

Yeah we're aware of that, working on it now. I'll rerun the test build when it's resolved.

LGTM. Thanks!


Add the option to run queries in the current database

Create a new page in which the user can run queries on the currently used database.

Hello Sorin, Do you foresee usages currently not covered by extension page Admin.Query?


[#94] Updated GitHub Actions Workflow

[Closes #94]

Updated to Xcode 13.4 on macOS 12
Updated to Ubuntu 22.04
Updated actions/checkout version
Removed redundant newlines
Other updates

swift is not yet available in Ubuntu 22.04 (actions/virtual-environments#5490). Reverting to 20.04.


Colors

Feature request:
I would like to propose to store the colors in colors.py not as a tuple, but as an np.array.
Reasoning:
I'm planning to add the entire Uni Tü color palette. If they're stored as np.arrays, that makes it easier to interpolate between colors, in particular to create lighter versions of existing colors.

Actually it would be nice to just have them as static np.arrays, not as functions, too. Saves a lot of typing.

I think it would be great if we could also define a default color palette.
We could also use the default of matplotlib or seaborn, but I would prefer a custom one. A custom color palette would make our plots look much more recognizable. I guess @philipphennig you would have a preference for main, second, third, ... colors?

 It works well for plots with up to three lines. But I realise that it may not be everyone's cup of tea. We could keep the default color palette as the default and offer the unitue palette as a replacement.
For what it's worth, I'm not a fan of seaborn. It's too heavy-handed for my taste.

thanks for the great inputs!  But I guess constants are more practical anyway -- let's see how far we get with that!
The original reasoning behind using tuples was that this way, the source is dependency-free. However, if we depend on matplotlib anyway (which has a numpy dependency, too), there is no real reason to avoid numpy arrays. So yes, we can change them into arrays.
Regarding the color palette: I think it is a great idea to add color palettes. How about not adding a default color palette, but a range of palettes so a user can choose freely without us imposing our tastes? I have some color cycles lying around (tue, probnum, tufte, etc.) which I can turn into rcParams inputs.
I suppose then it would make sense to branch out the color module:
color/
rgb/    # for RGB constants
cycles/  # for color cycles
from where on one can expand.

See #5.

Ok, great!
I think static constants are fine if they are within the namespace of the package. Numpy has np.pi, so  it can't be that bad :)

#13 introduces color constants.


The https (using rustls or openssl) tls handshake is not 100% successful at high concurrency.

Describe the bug
When utilizing the Rust HTTPS implementation with either rustls or openssl, the TLS handshake fails intermittently under high concurrency. Specifically, when subjected to 300 concurrent threads sending requests, some requests fail instead of all being successfully responded to.
To Reproduce
Steps to reproduce the behavior:

Use the example provided for rustls or the openssl example.
Spawn 300 threads to continuously send requests to the server.
Observe that some of the requests fail instead of completing the TLS handshake.

Expected behavior
The expected behavior is that the server should handle all requests successfully, even under high concurrency.
Screenshots
2024-01-16T13:13:08.523390Z ERROR salvo_core::server: accept connection failed error=Custom { kind: Other, error: "unexpected EOF" }
2024-01-16T13:13:08.524920Z ERROR salvo_core::server: accept connection failed error=Custom { kind: Other, error: "unexpected EOF" }
2024-01-16T13:13:08.526439Z ERROR salvo_core::server: accept connection failed error=Custom { kind: Other, error: "unexpected EOF" }
2024-01-16T13:13:08.527195Z ERROR salvo_core::server: accept connection failed error=Custom { kind: Other, error: "unexpected EOF" }

Desktop (please complete the following information):

OS: mac pro m1
Browser none
Version 0.63.1

Additional context

I just wanted to say I'm impressed by @chrislearn's turnaround time. I was also affected by this bug recently but didn't have time to investigate and create proper report, and @chrislearn fixed and released a new salvo version within hours of it being reported. And it doesn't look like it was a trivial change looking at the commit. Great job!


Help setting custom wallpaper.

I've tried to follow the instructions for setting a custom wallpaper for a theme, but I can't get it to work. Could the method be re-explained please?

You need to use the full path to the asset you want to use (eg C:\Users\... or /Users/....), then go to the settings, search for Doki` and put the full path in correct setting. If it was successful, the plugin will prompt you to reload.  Make sure your wallpaper is transparent otherwise your terminal will NOT work.
Here is a video
https://user-images.githubusercontent.com/15972415/202740915-adf973ed-432b-472e-ab3f-cd15479aaa2c.mp4
That help?

You need to use the full path to the asset you want to use (eg C:\Users\... or /Users/....), then go to the settings, search for Doki` and put the full path in correct setting. If it was successful, the plugin will prompt you to reload. Make sure your wallpaper is transparent otherwise your terminal will NOT work.
Here is a video
Kapture.2022-11-18.at.09.24.34.mp4
That help?

Unfortunately not. The plugin gave me the prompt to reload, but the wallpaper didn't change. The image is transparent and i'm giving it the right path.
I think I saw something about doki.wallpaper.enable on your main page. Might it have something to do with this?

I think I saw something about doki.wallpaper.enable on your main page. Might it have something to do with this?

That's for the Background. The Background is the image that shows up when all the tabs are closed. The Wallpaper is the one that shows up behind the code.

The background for me is the Zero Two that is colored, where as the wallpaper is the one that's outlined & green.
Make sure you are setting the correct setting.

I think it's working now. How did you make the explorer bar (left side) transparent, so that you can see the wallpaper behind it?

How did you make the explorer bar (left side) transparent, so that you can see the wallpaper behind it?

From what I remember, I made some things in the left side bar transparent so you could see the background image drawn on a panel behind it
https://github.com/doki-theme/doki-theme-vscode/pull/54

How would I do that?

I suppose I don't understand.
What are you asking for?
The current version of the plugin & VS code supports a background on all the parts. There isn't any gaps in coverage that I can see.

VSCode Version
Version: 1.73.1
Commit:<PHONE_NUMBER>f055b99068d3688932416f2346dd3b
Date: 2022-11-09T02:22:48.959Z
Electron: 19.0.17
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 22.1.0
Sandboxed: No

I think you understand what I mean.

As you can see here, the Explorer tab isn't transparent. It is hiding some image behind it.
As you've said, current version of the plugin & VS code supports a background on all the parts, but it doesn't look like it's applied here.

I think you understand what I mean.

As you can see, the explorer tab isn't transparent, therefore hiding some of the wallpaper behind it. Do I need to enable this feature in the VSCode setings, or extention settings?

There is background & wallpaper.

background is what you probably have set. It's what goes behind your editor. So when you have your tabs closed it shows up.
wallpaper is what you want to set. It shows up in-front with the code & explorer. You'll want to use an image that is transparent so you can read text & stuff.

Ahh, so in this screenshot, you had it set as wallpaper. I thought it was the other way round. Thanks.


Operators to the right of DAN have undefined results

Right now in Carbide,
vec2->xy + 1

Is the same as
vec2[1], vec2[2] + 1

I don't know that this is the best.
It might be a better idea to have the operator translate to
vec2[1] + 1, vec2[2] + 1

I took a crack at doing this a few minutes ago and I completely screwed everything up and had to revert. I'll try it again soon, but it's kind of important I think, as it saves a fair bit of typing.

Depends on issue #46

That issue is resolved, I'm now writing a test suite for DAN now that the other operators are relatively tested (and fixed as a result!)

Fixed! That was a heck of a lot easier than I was expecting, and the fix should fix a couple other operators with regards to the equals sign being an expression boundary.


User Profile - Wire Post Tab + API

API needed to use target guid.
Implement filling of content in tab for a user. Add link to tab profile navigation. Add view more button.

Wire posts already had a functioning GetByUser api, so much easier than expected.


Minor improvements to integration tests

Raises the global timeout to 20s
See https://github.com/line/armeria/pull/2615
Decouples assertions from reporting order
Allows libraries to backdate spans and report in any order
Fixes some out-of-date links notices when doing above

cc @ikhoon


QR code encoded but not printing with 340 characters.

Thank for very useful library.
I am writing mobile POS app in nativescript.
and i use your encoder. QR code encoded but not printing with 340 characters ?
do you have any solution for long text  #qrcode?

The most likely answer is that your printer does not support qrcodes with that many characters. While there is not a size limit in the specification, individual printers may impose a limit of their own. I for example have a printer that refuses to print anything larger than 255 characters.

To @NielsLeenheer's point, the 255 limit is surpassed here: https://github.com/NielsLeenheer/EscPosEncoder/blob/55f5513deb7b89401d6e24b9fc2498f6f77b48ab/src/esc-pos-encoder.js#L428
The fourth parameter should allow values over 255.  We have near identical logic here: https://github.com/qzind/tray/wiki/2.0-Raw-Printing#escpos-qr-code
If the hardware can't go over 255, you can render the QR code as an image using 3rd party libraries and send the image data to the ESCPOS printer instead.


Add missing expect import

Changed

Added missing expect imports

Codecov Report

Merging #177 (9dc1775) into master (2113486) will increase coverage by 3.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #177      +/-   ##
==========================================
+ Coverage   95.77%   98.88%   +3.11%
==========================================
  Files         103       22      -81
  Lines        1586      359    -1227
  Branches      219       55     -164
==========================================
- Hits         1519      355    -1164
+ Misses         64        4      -60
+ Partials        3        0       -3

Flag
Coverage Δ

cuaktask
99.17% <ø> (ø)

iocuak
?

porygon
98.30% <ø> (ø)

porygon-typeorm
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...duleTask/utils/domain/isContainerModuleTaskKind.ts

.../containerModule/utils/api/isContainerModuleApi.ts

packages/iocuak/src/utils/stringifyServiceId.ts

...ing/models/api/bindingScopeToBindingScopeApiMap.ts

.../domain/ContainerSingletonServiceImplementation.ts

...rm/src/crud/models/domain/CrudTypeOrmModuleType.ts

...s/iocuak/src/binding/models/api/BindingScopeApi.ts

...cuaktask/ContainerInstanceServiceImplementation.ts

...erModuleTask/modules/ContainerModuleTaskBuilder.ts

...eTask/models/domain/ContainerModuleTaskKindType.ts

... and 71 more

Continue to review full report at Codecov.

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


Get data from private discussions

Also handle "previously"

To get data from private discussions, the user needs to create a PAT with the Discussions param set to Read-onyl


Attempt to divide by zero error when using wait-rsjob

Hello,
I was trying to do some tests with your rsjobs module and received an error saying I'm trying to divide by zero when I try using wait-rsjob
$Test = 42
$AnotherTest = 7
$String = 'SomeString'
$ProcName = 'powershell_ise'
$ScriptBlock = {
Param($y,$z)
[pscustomobject] @{
    Test = $y
    Proc = (Get-Process -Name $Using:ProcName)
    String = $Using:String
    AnotherTest = ($z+$_)
    PipedObject = $_
}
}

1..5|Start-RSJob $ScriptBlock -ArgumentList $test, $anothertest
Wait-RSJob
6..10|Start-RSJob $ScriptBlock -ArgumentList $test, $anothertest
Get-RSJob | Receive-RSJob

At C:\Program Files\WindowsPowerShell\Modules\PoshRSJob\Scripts\Wait-RSJob.ps1:143 char:21

...             Write-Progress -Activity "RSJobs Tracker" -Status ("Remai ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [], RuntimeException
FullyQualifiedErrorId : RuntimeException

Or am I doing something wrong?

Weird. I will have to check and see what is going on. It shouldn't be using Write-Progress at all if not using -ShowProgress so it seems like a bug to me.

Hey, sorry the output was with a write-progress in the command. The issue I was talking about was the divide by zero.

I couldn't re-produce this in Windows 7, but appears to be an issue with Windows Server 2012 R2. I'll dig some more to see what the issue is and get it fixed.

Ok, the bug should be fixed now. Had to change a couple of lines of code to avoid that error from popping up.

Ok, it took me a while to realize what was going on but you aren't actually piping anything into Wait-RSjob or specifying a Name, ID or InstanceID parameter. This is why there are DivideByZero errors.


Config file

Is it possible to change the 3 seconds wait to 0 seconds? And could we make it so the trade request only last 5 seconds? It would be great to add a configuration file for these things!

Added.


Surah Info close button not working

fixes #458
Thank you @ATouhou

Deployed to: http://staging.quran.com:32816


Add support for multiple static paths with optional static root

closes #3510
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Proposed changes
Permits single or multiple static source paths and allows each of those to be served from a different root
By adding the httpStaticRoot we can avoid the need for changing httpAdminRoot when httpStatic is enabled.
Examples...
==========
Single path, default root
Settings File
    httpStatic: 'c:\\temp',
    //httpStaticRoot: '/',

Log Output
22 Apr 17:05:34 - [info] HTTP Static    : c:\temp > /

Multiple paths, alternative base root
Settings File
    httpStatic: [{path:'c:\\temp'}, {path:'c:\\tmp'}],
    httpStaticRoot: '/static_stuff/',

Log Output
22 Apr 17:05:34 - [info] HTTP Static    : c:\temp > /static_stuff/
22 Apr 17:05:34 - [info] HTTP Static    : c:\tmp > /static_stuff/

Multiple paths with default root and individual root
Settings File
    httpStatic: [{path:'c:\\temp', root: "/stuff/"}, {path:'c:\\tmp', root: "/other/"}],
    //httpStaticRoot: '/',

Log Output
22 Apr 17:05:34 - [info] HTTP Static    : c:\temp > /stuff/
22 Apr 17:05:34 - [info] HTTP Static    : c:\tmp > /other/

Multiple paths with base root and individual root
Settings File
    httpStatic: [{path:'c:\\temp', root: "/stuff/"}, {path:'c:\\tmp', root: "/other/"}],
    httpStaticRoot: '/static/',

Log Output
22 Apr 17:05:34 - [info] HTTP Static    : c:\temp > /static/stuff/
22 Apr 17:05:34 - [info] HTTP Static    : c:\tmp > /static/other/

Checklist

[x] I have read the contribution guidelines
[x] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage decreased (-0.1%) to 68.655% when pulling 22000f10dfd389b5d906198fa35a3fc4114e45e9 on Steve-Mcl:httpstatic-improvements into d802ce1484ece91981f1afcc7a67323acc09ed14 on node-red:dev.


Support for Swift wrapper

This fixes #99, which is necessary for Swift.
The xcframework built for reactive-native can be shared with the Swift wrapper. But, it requires a different archive format. So I added a workflow step to archive it using zip with a different folder name AskarFramework.
The Swift wrapper in different repo could download the zipped xcframework for its dependencies.

Maybe we should just publish the xcframework as a zip and the Android libraries as a separate .tar.gz?

Maybe we should just publish the xcframework as a zip and the Android libraries as a separate .tar.gz?

Sounds good. @blu3beri What do you think?

I think I need to test uniffi first because uniffi will generate a different header file. I'll close this now and re-open it later if uniffi fails.

Maybe we should just publish the xcframework as a zip and the Android libraries as a separate .tar.gz?

We need the ios and android build to be published in one tar/zip so they can be dynamically downloaded when the react native package is installed from npm. We can't download multiple binaries, or we'd have to implement that logic ourselves


[2.0] beforeRouteEnter's next(vm => { }) callback not always works

This bug this about different path with the same component.
index.vue
<template>
  <div>
    <router-link :to="{ path: '/' }">root</router-link>
    <router-link :to="{ path: '/a' }">a</router-link>
    <router-link :to="{ path: '/b' }">b</router-link>
  </div>
</template>

  beforeRouteEnter: (to, from, next) => {
    console.log(111)
    next(vm => {
      console.log(222,  vm) // <- this not always be printed
    });
  }

main.js
import Index from './index.vue'

Vue.use(VueRouter)

var routes = [
    { path: '/', component: Index },
    { path: '/a', component: Index },
    { path: '/b', component: Index },
]

var router = new VueRouter({
    routes: routes
})

Then access / and just switch between a and b, "222" never be printed.
I already tested it on a new empty project generated by vue-cli. I'll upload that if you need.

Tested on jsfiddle, https://jsfiddle.net/fnlCtrl/a2v5rrcp/
Seems to be a bug.

Same here.
const router = new VueRouter({
    mode: 'history',
    routes: [
        {path: '*', beforeEnter (to, from, next) {
            alert('beforeEnter'); // Appears

            next((vm) => {
                alert("next"); // Doesnt appears
            });
        }}
    ]
});

@fy0 Why would you use the same component for different routes?

@posva Just like the todo-list demo, different items and url, same page.

BTW, using a key attribute with the route path is a workaround for your problem (:key="$route.path" on the router-view element) https://jsfiddle.net/pyqchnrd/

@posva Cool. But It doesn't matter the way it be implemented, bug is bug.

@posva Yeah, thx for contributors' great works.

I'm still seeing this issue using Vue 2.1.0 and Vue Router 2.1.1. I'm investigating whether it's something to do with my setup.

Seems to still have a problem, anyone with me?

I am also having this issue. adding a key to the router-view doesn't seem to work for me.

@dehypnosis @SnooHD
It's a long time ago, now use beforeRouteUpdate to load new data.
And don't forget to call next(), otherwise afterEach won't be emitted.

Add a key seems work for me

Same for me with <router-view :key="$route.path"/>`

Anyone else still experiencing this issue on<EMAIL_ADDRESS>

Yes, I still get the same problem with<EMAIL_ADDRESS>Any ideas rather than forcing a key in the router-view ?

yes, suggestions in comments don't work for me

There's an open issue for this at #2255


No pan on invalid zoom

Fix for the panning issue found during investigating #109.

I can see how some might argue that the center should be respected even if the zoom value cannot be reached, but I think the behavior in this change makes sense (it is certainly better when using the mouse wheel).  I've checked a few examples.  Assuming tests pass, please merge.


Linux .desktop file using absolute icon path

VSCode Version: 1.44.1
OS Version: Ubuntu 19.10 x64

The code.desktop file installed to the system uses an absolute path to the icon, although the icon is saved to a standard system location for icons (/usr/share/pixmaps). This is non-standard — .desktop files should use only the icon filename (without the extension!) and allow the system to search its icon files — and it breaks compatibility with icon themes.
The fix is simple: change the Icon: field to be the following:
Icon=com.visualstudio.code

After making that change, Visual Studio Code uses its default icon unless explicitly overridden by an icon theme. (I've made this change myself on every update to code for the past few months.)
Steps to Reproduce:

Install Visual Studio Code via system package (e.g. DEB).
Load custom icon theme.
Note the icon has not changed. (Then see /usr/share/applications/code.desktop)

Does this issue occur when all extensions are disabled?: Yes

@Tyriar Do you know stuff?

I do, it was changed to absolute to fix another bug
https://github.com/microsoft/vscode/commit/7816110c15b728afb4429994a1261c0da46e2fa7
https://github.com/microsoft/vscode/issues/71942

So... @CodeMouse92 what issue do you actually get? Why have you been changing the icon every month?

@joaomoreno the icon would need to change each month as the installer replaces the file with customizations each update. The question is, what's the right way considering https://github.com/microsoft/vscode/issues/71942 was petitioning the absolute way as they weren't seeing anything.

@joaomoreno I use an icon theme, and the .desktop file won't switch like every other app because of the absolute path.

I'm really not keen on playing ping pong on this. So, unless we find out a different fix for #71942, we shouldn't revert. It's better for people to see an icon than not to be able to theme it, IMO.

@joaomoreno I'd propose leaving this open for now. I'll see if I can find anything about #71942.

@joaomoreno Aha!!! I found a likely cause of the original problem (although I cannot comment on that task): if the .desktop file has the wrong permissions, the icon will not be visible! Has anyone actually checked that the .desktop file was being marked executable on all systems?
SOURCE

@CodeMouse92 wait, so absolute paths work when the file has wrong permissions but not just the file name? 😕

@Tyriar It's possible. If there was something wrong with using a relative path to /usr/share/pixmaps in  .desktop file, then all those users would be unable to see most icons. But they can see 'em all...just not ours. Since the icon file is present in the usual directory, and the .desktop conventions are functioning on everything else, the problem must be our .desktop file's permissions.
Remember that .desktop is merely a convention, and different DEs, programs, widgets, and even versions may handle it differently. They all assume it will follow the convention, so if it doesn't, the results will be unpredictable — in a word, undefined behavior.

@Tyriar @joaomoreno YES! That's indeed it!! I just checked on my system, and the .desktop files are not marked as executable!
That means, to resolve both this and #71942, you must:

Reverse the original workaround; go back to using Icon=com.visualstudio.code in the code.desktop file.

After placing code.desktop and code-url-handler.desktop in /usr/share/applications, they must be marked executable via chmod +x code*.desktop

I would create the PR myself, but I haven't yet figured out my way around the repository.

Yup, definetly agree on this one

Here's a build based on my PR, could some people please verify this works fine? https://az764295.vo.msecnd.net/insider/1a14317adc345847b872b1a922384ee55ba220aa/code-insiders_1.45.0-1587820403_amd64.deb
If so, I can merge in for the May release (it's a little late to go in for the April release).

@Tyriar I can confirm that the insider build picks up the icon from my icons theme and if I delete the icon from there it will use the default shipped one.
Tested on Debian SID.

And when the icon is gone from my icon theme:

I did not need to modify any files for this. In the base build I need to change the code.desktop file = url-handler file to pick up my icon theme instead by just using the name com.visualstudio.code in order to get it displayed. The insider build arleady did this by default and hence it works just fine.

@CodeMouse92 if you could verify too that would be great, should be able to merge this in around 1-2 weeks.

@Tyriar Confirmed that it works! It shows the default, and when I add a custom icon for it to my icon theme, it picks up on the new icon.

Great, I pinged the other issue as well to tell them about the change https://github.com/microsoft/vscode/issues/71942#issuecomment-619374048, I'll merge it in when master is opened for the May release.

Thanks @CodeMouse92 for your analysis!

Looks like a good author-verified situation.


rshell not connecting to pico

Hi,
Super noob question incoming...
I'm running MicroPython version 1.19.1 on a Raspberry Pi Pico board.
When I try to run rshell, I connect to my own PC instead of the Pico board. When I use ls, the list is of the folders in my home directory.
I have previously used ampy to copy across file pico_helloworld.py which you can see in the rshell startup.
Is there something else I have to do in order to connect to the Pico?
rose@xxxxxxx:~$ rshell --port /dev/ttyACM0
Using buffer-size of 32
Connecting to /dev/ttyACM0 (buffer-size 32)...
Trying to connect to REPL  connected
Retrieving sysname ... rp2
Testing if ubinascii.unhexlify exists ... Y
Retrieving root directories ... /pico_helloworld.py/
Setting time ... Aug 18, 2022 11:42:16
Evaluating board_name ... pyboard
Retrieving time epoch ... Jan 01, 1970
Welcome to rshell. Use Control-D (or the exit command) to exit rshell.
/home/rose> ls
Desktop/
Documents/
Downloads/
Music/
Pictures/
Public/
Templates/
Videos/
cmake-3.20.2/
esp/
openocd/
scripts/
snap/
/home/rose> ls -l /flash
Cannot access '/flash': No such file or directory

Hey I solved my own problem....
I needed to copy / list /pyboard and not /flash
For example
/home/rose> ls /pyboard
pico_helloworld.py

Leaving this here in case anyone else is as dumb as me haha


Players who are not dead are marked dead

I'm having a problem where players who are not dead, are marked dead and can't talk during emergency calls because of this. It happened to me at least once when I was imposter myself and second time for another player. Is there a way I can give more details about this? I had to stop using the bot because of this.

Yeah, if you can provide screenshots of the capture.exe logs of it happens again, in conjunction with the logs.txt that is generated by the discord. exe, that would help a ton :)

Here's my logs.txt
logs.txt
The first time it happened to me at around 19:39 (player name Sussikki, discord id<PHONE_NUMBER>21796868)
The second time it happened to my friend at around 20:31 (player name Obama, discord id<PHONE_NUMBER>48499200)
Both of those times are after is happened as I don't really know how to read the log. I took the times from discord messages that the falsely dead players posted in discord after noticing it.
For now I can't get anything from capture.exe as we already stopped playing for the day, I'll see if my friends are willing to test it out more (they started hating on the bot after we had this issue, which is very unfortunate).

have you tried the latest release (2.0.5)? let us know if you see this bug again. thanks!

The bot is working absolutely fine now, played a few games yesterday with the latest updates and had no issues at all. Great work!


Nodist env latest fails in Git bash on Windows

Nodist version: 0.7.2
Bash version: GNU bash, version 4.3.42(2)-release (x86_64-pc-msys) // (comes with git)
Operating system: Windows 10 x64
$ nodist env latest
Could not fetch help info.
You will have to look at the README.
Sorry.

$ nodist env 5.7.0
Could not fetch help info.
You will have to look at the README.
Sorry.

This command works fine in the regular command prompt.

Thank you for reporting this! This feature is not yet implemented. However, bash support is expected to land in v0.8 -- see https://github.com/marcelklehr/nodist/issues/14

I see! I'm not aware of any internals, so I was surprised and confused that this doesn't work. I thought it's just a console application that is being executed.

Manipulating env vars is only possible with shell scripts, thus a bash script is needed that implements nodist env ;)


botocore is a hidden dependency when using luigi.contrib.sge

I am not using luigi.contrib.s3 but nevertheless it tries to get imported:
Traceback (most recent call last):
  File "run.py", line 7, in <module>
    from luigi.contrib.sge import SGEJobTask
  File "/home1/depalati/src/luigi/luigi/contrib/sge.py", line 104, in <module>
    from luigi.contrib.hadoop import create_packages_archive
  File "/home1/depalati/src/luigi/luigi/contrib/hadoop.py", line 54, in <module>
    import luigi.contrib.s3
  File "/home1/depalati/src/luigi/luigi/contrib/s3.py", line 35, in <module>
    import botocore
ModuleNotFoundError: No module named 'botocore'

create_packages_archive looks useful and generic enough that it should live somewhere other than luigi.contrib.hadoop.

One common pattern is to have some top-level utils module that collects things like create_package_archive that don't necessarily fit anywhere else. The danger with this approach is that this can make utils a catchall for just about anything.

There is currently a luigi/util.py so it seems reasonable to have a luigi/contrib/util.py that we carefully allow code into.


fix(prefer-readonly-type): mark rule as deprecated

Fixes: #51
Fixes: #150
Fixes: #151
Fixes: #153
Deprecated in favor of prefer-readonly-type-alias and @typescript-eslint/prefer-readonly-parameter-types.

@jonaskello Do you have any thoughts on this?

Closed in favor of #259


vslidedown behavior when vl<vlmax

For vslidedown, section 17.3.2 says:
vslidedown behavior for source elements for element i in slide
        0 <= i+OFFSET < VLMAX   Read vs2[i+OFFSET]
        VLMAX <= i+OFFSET       Read as 0

When VL<VLMAX, this has the effect of copying tail elements from vs2 into vd, which does not seem right, and also conflicts with the defined behavior of tail elements in section 5.4:
The tail elements during a vector instruction’s execution are the elements past the current
vector length setting. The tail elements do not raise exceptions, and do not update any
destination vector register group.

I would instead have expected the specification to be:
vslidedown behavior for source elements for element i in slide
        0 <= i+OFFSET < VL   Read vs2[i+OFFSET]
        VL <= i+OFFSET       Read as 0

Can you please confirm that the specified behavior is correct? If it is correct, can you state the best way to perform a slidedown that guarantees that all elements that would be sourced from otherwise inaccessible tail elements are zeroed?

I see that this is a duplicate of #261, but that issue was created at a time when tail elements were zeroed, whereas now they are left untouched and likely to be garbage.

The current definition is deliberate (which is not to say your suggestion is off-base). It’s slightly simpler to implement as-is, and it seems to me to work fine in practice. Bear in mind that tail elements aren’t inherently garbage; you can contrive scenarios where shifting down from elements >= vl is actually the desired behavior. And you can often emulate your alternative formulation with low overhead by setting vl to ~0 and zeroing the source vector register before entering the strip-mine loop.
Note also that the current formulation is consistent with vrgather. In that case, vl indicates how many elements are gathered, and it’s quite deliberate that it’s possible to gather elements whose indices lie between vl and VLMAX.
The Section 5.4 rule doesn’t seem contradictory to me, since it doesn’t claim tail elements are never read, just that they aren’t written and don’t raise exceptions. Perhaps the language could be clarified a bit.

Ok, understood. I think I would recommend two clarifications to the specification:

In section 5.4, specifically state instructions that are exceptions to the tail element rule, and for which the final and do not update any destination vector register group statement does not apply; and:
In section 17.3.2, perhaps add a code fragment that shows the best way to perform a slidedown that guarantees that all elements that would be sourced from otherwise inaccessible tail elements are zeroed.

I wonder whether a new kind of vmv.v.v that zeroes the tail would be a useful thing to have?

Reading the spec it seems vslidedown still only processes VL elements so it doesn't touch the destination tail elements and therefore is not an exception to 5.4, which is why Andrew brought up vrgather which also gathers VL elements.

Colin, I think your comment would be correct if it was the case that section 5.4 was intended to apply only to destination elements but this is not stated, and I suspect that it was not the case when it was written. I cannot see that it would do any harm to be more specific here.
I think vrgather should be considered a separate case from vslidedown. For vrgather, the application has complete control over the source elements: if it wants to pick elements from the tail region for some reason, then that is entirely fine. I can't immediately think of a use for this myself, but there is no reason to prevent it. The case for vslidedown is different. Here, the application has no choice: except in the degenerate case of a zero slide, the result will be affected by tail elements in the source (if unmasked).
Suppose that a library routine wants to slide down a vector value by a large or variable amount and fill values sourced from beyond the current vl with zero (or something else).  When sliding up, it looks easy: because destination elements before OFFSET are unchanged, the application can splat the required default over the entire destination and then use vslideup. What is the best way to do something equivalent when sliding down? Perhaps this could be added as an example in 17.3.2.

As Andrew suggested that you can prepare a vector register with all elements being 0 to start with. And when you want to do the slidedown operation, all tail elements are 0 already.
Assuming v8 will be used later for slidedown operation,
vsetvli t0, x0, e32     ; set VL to MAXVL
vxor.vv v8, v8, v8      ; zero all elements
vsetvli t0, a0, e32     ; a0 has the true VL for the algorithm

If this is not possible for your algorithm and the tail elements have values that you want to use later, then
Assuming v8 is the vector register with non-zero tail elements, and v9 will have the slidedown value of v8 with 0 on v8’s shifted down tail elements. a0 has the slidedown OFFSET.
csrr t0, vl             ; done once outside the loop, t0=VL
vxor.vv v7, v7, v7      ; done once outside the loop
…
vslidedown.vx  v9, v8, a0   ; shift down tail elements
sub t1, t0, a0              ; calculate the start offset for zeroing
vslideup.vx  v9, v7, t1     ; zeroing shifted down tail elements

Here is another example using mask registers for LMUL=1. Assuming v8 is the vector register with non-zero tail elements, and v9 will have the slidedown value of v8 with 0 on v8’s shifted down tail elements. a0 has the slidedown OFFSET.
csrr t1, vl            ; t1=VL
setvli t0, x0, e32     ; set VL to MAXVL
vmset.m v1             ; set all MAXVL mask to 1
setvli t0, t1, e32     ; set original VL
vmclr.m v1             ; clear VL mask to 0, all tail element mask remains as 1
…
vslidedown.vx  v9, v8, a0   ; shift down tail elements
vlidedown.vx  v0, v1, a0    ; shift down mask tail elements
vxor.vv v9, v9, v9, v0.t    ; zeroing all shifted down v8 tail elements

The last zeroing instruction may be done once outside the loop if no cross-element computations, such as reduction/slide, are performed on the slided-down vector values.

I prefer to change VLMAX to vl.

I think vrgather should be considered a separate case from vslidedown. For vrgather, the application has complete control over the source elements: if it wants to pick elements from the tail region for some reason, then that is entirely fine. I can't immediately think of a use for this myself, but there is no reason to prevent it. The case for vslidedown is different. Here, the application has no choice: except in the degenerate case of a zero slide, the result will be affected by tail elements in the source (if unmasked).

Totally agree.
I don't know whether we can find a scenario that vslidedown uses tail part. But based on current intrinsic design, it only allowes software programmers use [0, vl) elements.
In addition, software programmers cannot specify register usage.
That means for @chuanhua 's case

As Andrew suggested that you can prepare a vector register with all elements being 0 to start with. And when you want to do the slidedown operation, all tail elements are 0 already.
Assuming v8 will be used later for slidedown operation,
vsetvli t0, x0, e32     ; set VL to MAXVL
vxor.vv v8, v8, v8      ; zero all elements
vsetvli t0, a0, e32     ; a0 has the true VL for the algorithm

is not true for current intrinsic. See the following code.
vsetvlmax_e32m8();
vint32m8_t input = vzero_i32m8();
vsetvl_e32m8(true_vl);
input = vlw_v_i32m8(data);
vint32m8_t result = vslidedown_vx_i8m1(input, 1);

The result[vl - 1] is unknown, because compilers do not guarantee input use the same vector register as previous input.

I believe the remaining issue here is in intrinsic specification rather than in the ISA?

I made a couple of documentation change suggestions (March 6, above) which would help clarify things, I think.

I'm adding some clarification text to close the spec issue.
I believe there still might be some intrinsic issue on how to access the functionality, but that is probably best addressed in intrinsics forum, so I'm going to close here with the edit.


Any support for http proxies?

Mistakenly created this one.
This is the one containing descriptions: https://github.com/mezantrop/ts-warp/issues/2


Fix typo in README

It should be Canada (not Cananda!) 🇨🇦

🍪 that's a funny typo

🐈⬛ My cat missed the typo too!

hey, I think I can fix this typo!

@abbycabs does GitHub have spellcheck? ;)


Fix Paned.GetHandleWindow(): should return gdk.Window, not gtk.Window

See https://docs.gtk.org/gtk3/method.Paned.get_handle_window.html
and see discussion in https://github.com/gotk3/gotk3/pull/913

@andre-hub: the issue lies with an outdated test: gtk\gtk_test.go:873:17


Visting the login URL when logged in still displays the login button

We correctly bounce people through to /releases when we log in.
Also, if we hit the / page - we bounce people to the correct logged in /releases.
However - if we are logged in - and visit /login - it still shows a Login button. I think instead it should either bounce - or display other buttons (Login grayed out? - and some text about how to logout?)

I think bounce is probably correct here - can we see what other prominent web pages do in this situation?


Add dask stack tests

[x] Tests added
[x] Passes pre-commit run --all-files

I think we could consider gradually moving to this fixture model of tests, where we pass da and it's paramaterized across dask / numpy (and anything we choose to add in the future). And it's possible to select the backing array type like selecting any other fixture type.
The general proposal probably needs some socializing and refinement before we start applying it more broadly. I'd be keen to pursue it if others agree — it could bring some more robustness to our testing suite while also making it simpler.

The general proposal probably needs some socializing and refinement before we start applying it more broadly. I'd be keen to pursue it if others agree — it could bring some more robustness to our testing suite while also making it simpler.

👍🏽


KAFKA-8179: Part 7, cooperative rebalancing in Streams

Some key points:

allow PARTITIONS_REVOKED to transition to itself (callback is no-op if revokedPartitions is empty)
tasks will remain available for IQ during a rebalance (but not during restore)
in COOPERATIVE, infer active tasks from the Subscription's ownedPartitions  and make sure no one previously owned a partition/task before assigning to new consumer
give active tasks back to previous consumer within a client, unless a) violates load balance or b) we know a second rebalance will be triggered anyway
continue restoring and processing both active and standby tasks during a rebalance, but do not try to commit (can only process records already fetched for active tasks, pending KAFKA-7312 )

Jenkins seems to be having some issues today but all tests pass locally, kicked off system tests here: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/3031/

@bbejeck @vvcephei

Still debugging the streams_eos_test:StreamsEosTest but all others pass: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/3042/
Jenkins failures cleaned up, retest this please

Doing another run of system tests but initial run passed (exception for already flaky test_broker_type_bounce) : http://confluent-kafka-branch-builder-system-test-results.s3-us-west-2.amazonaws.com/2019-10-03--001.1570083170--ableegoldman--KIP-429-streams-pt2--e92caf1/report.html

Another (this time completely) green run of the system tests: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/3142/

Test failures come from connect and this one:
SmokeTestDriverIntegrationTest.shouldWorkWithRebalance

Not sure if it is consistent, need to re-run.

The new jenkins failures are due to timed out after 270 min (2.12 and 2.11) and connect integration test (2.13). And there's no overlapping test failures in consecutive runs.
Will merge to trunk now.

Also cherry-picked to 2.4


[GSSOC'23]: adding algorithms related to maths, in algorithm learner C++

Describe the Feature you want to end
adding algorithms related to maths, in algorithm learner C++
Expected behavior

Count Primes : Sieve of Eratosthenes
find GCD of two numbers: Euclidean Algorithm
Factorial Algorithm
Fibonacci Sequence Algorithm

@MohitGupta121 @TusharKesarwani sir kindly assign me this issue


Fix 'get property on non-object' notice.

It is possible for ReflectionFunction::getClosureScopeCLass()
to return NULL, at which point there is no name property to get.
Since Closure::bind() does not require the third property this
leaves it out if there is no scope class for the closure.

@rdlowrey any idea when this could be merged (I noticed after there's another similar pull request, although I think mine is a bit cleaner).

The other PR was already merged

OK, I didn't realize, as I was actually referring to this one (which is still open).  Apparently there were 3.
https://github.com/rdlowrey/Auryn/pull/87
Closing this one.


Add OnBefore/After- Lookup, Drilldown, NewRecord event support for page extensions

In case the image above doesn't load here's the code snippet:
pageextension 60089 "TRG General Journal" extends "General Journal"
{
layout
{
modify(CurrentJnlBatchName)
{
trigger OnLookup();
begin
OnSetBatchReversalDateEvent(Rec,ReversalDate);
end;
            trigger OnValidate();
            begin
                OnSetBatchReversalDateEvent(Rec,ReversalDate);
            end;
        }
        modify(Amount)
        {
            trigger OnValidate();
            begin
                SAVERECORD();
            end;
        }
  }

}
Error Message: OnLookup is not a valid trigger/ OnValidate is not a valid trigger.
Is there a way to add code/functions/publishers to the page triggers of the native fields?

Currently you can only add code before or after the OnValidate trigger by using OnAfterValidate and OnBeforeValidate triggers:
modify("Cause Of Absence Filter")
{
            trigger OnAfterValidate()
            begin
            end;
}

You can always see the available triggers by pressing Ctrl + Space (IntelliSense) after the trigger keyword.

Will access to those field triggers be added at some point though? I need to put code on Lookup, Drilldown, OnNewRecord and other native page and page triggers that aren't available now.

@Anna-Gudimova OnNewRecordEvent does exist, like here on Purchase Prices page:
OnNewRecordEvent(VAR Rec : Record "Purchase Price"; BelowxRec : Boolean; VAR xRec : Record "Purchase Price")
But OnlookUp publishers doesn't exist yet ...

Possibility of handle OnLookup event is a need. There are tons of customization here (opening custom lookup pages, applying filters and so on).

hi Stanislaw, what's the status on any native OnValidate/OnBeforeValidate/OnAfterValidate events for pages and tables?

any update on the issue?

pageextensions now have triggers for OnAfterValidate(), OnAssistEdit(), OnBeforeValidate(), OnDrillDown(), and the much-discussed OnLookup().
Is there anything else still needed here? If so, what is it? The discussion is long and not that clear.

pageextensions now have triggers for OnAfterValidate(), OnAssistEdit(), OnBeforeValidate(), OnDrillDown(), and the much-discussed OnLookup().
Is there anything else still needed here? If so, what is it? The discussion is long and not that clear.

Let's close this issue as most of the triggers have been added. For any new triggers, please go to BCIdeas


Method 'next_event' of 'grpc._cython.cygrpc.SegregatedCall' cost 6s on macOS

What version of gRPC and what language are you using?
$ pip show grpcio

Name: grpcio
Version: 1.23.0
Summary: HTTP/2-based RPC framework
Home-page: https://grpc.io
Author: The gRPC Authors
Author-email<EMAIL_ADDRESS>License: Apache License 2.0
Location: /Users/apemost/miniconda3/envs/py37/lib/python3.7/site-packages
Requires: six
Required-by: grpcio-tools

What operating system (Linux, Windows,...) and version?
macOS
What runtime / compiler are you using (e.g. python version or version of gcc)
$ python --version

Python 3.7.3

$ gcc --version

Configured with: --prefix=/Library/Developer/CommandLineTools/usr --with-gxx-include-dir=/Library/Developer/CommandLineTools/SDKs/MacOSX10.14.sdk/usr/include/c++/4.2.1
Apple LLVM version 10.0.1 (clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin18.6.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

What did you do?
If possible, provide a recipe for reproducing the error. Try being specific and include code snippets if helpful.
What did you expect to see?
What did you see instead?
11 function calls in 6.048 seconds

   Ordered by: standard name

   ncalls  tottime  percall  cumtime  percall filename:lineno(function)
        1    0.000    0.000    0.000    0.000 cProfile.py:50(create_stats)
        1    0.000    0.000    0.000    0.000 threading.py:240(__enter__)
        1    0.000    0.000    0.000    0.000 threading.py:243(__exit__)
        1    0.000    0.000    0.000    0.000 threading.py:335(notify)
        1    0.000    0.000    0.000    0.000 threading.py:358(notify_all)
        1    0.000    0.000    0.000    0.000 {built-in method builtins.len}
        1    0.000    0.000    0.000    0.000 {method '__enter__' of '_thread.RLock' objects}
        1    0.000    0.000    0.000    0.000 {method '__exit__' of '_thread.RLock' objects}
        1    0.000    0.000    0.000    0.000 {method '_is_owned' of '_thread.RLock' objects}
        1    0.000    0.000    0.000    0.000 {method 'disable' of '_lsprof.Profiler' objects}
        1    6.048    6.048    6.048    6.048 {method 'next_event' of 'grpc._cython.cygrpc.SegregatedCall' objects}

Anything else we should know about your project / environment?
None.

Yes. The client is currently synchronous and the function you're referring to is a blocking call. Any time RPCs spend transiting the network will be represented by time spent in this call. A non-blocking API leveraging asyncio is being actively worked on, which will allow your application's coroutines to monopolize the GIL while RPCs are in-flight.
Do you have any more specific concerns?


Allow metadata in json or yaml, allow python latest scripts

Purpose
As the commit title states, this PR updates the workflows for this repo to match github.com/rtrox/containers (sorry @onedr0p! I know I promised this a while ago, life and work both got hectic >.<)
Key Changes:

Metadata can be defined in either JSON or YAML (metadata.json OR metadata.yaml). CUE works with both.
Fetching the latest version of a package can be done via either a bash script or a python script.
Images are built and pushed with docker-build-push-action, and have generated metadata.
CUE is run at PR time to verify metadata
Tags can be applied by semantic versioning rules, allowing 1, 1.2 and 1.2.0 to all be defined on a package with version 1.2.0 applied.

Note that in all of these cases, I've deliberately not implemented any of these changes on existing apps yet -- I wanted to do that in separate PRs, I figured one monolithic PR might be tough to review.
Testing
Tested Workflows on my fork -- specifically the PR Validation workflow and the Merge workflow. I haven't yet tested the scheduled run or manual release workflows, but the differences are minimal. Once the scheduled release workflow runs, I'll update here if there's any issues.
Possible Changes Needed
If it's not already set on your bot (I'm pretty sure it is, as I think you use the same bot for renovate, but just in case), the bot will need permissions to create and merge PRs, so that it can automatically regenerate the README.
Current Issues
As of right now, transmission doesn't return it's current version in the render-readme script. I tested this a bit, and it looks like the return from the registry does not contain tags for this package for some reason. I'm continuing to look into it to see if there is another better way to get the tags on the latest container, will update here if I can sort this out.

Working PR & Merge Workflows:

I just tested the scheduled-release as well, and it was working fine :-D I cancelled it so I don't create a trillion packages that I'll have to delete later >.<
That means I can assume the manual release should work too, they're roughly identical

Let's ship it, quick examination it looks fine from my phone 😂

I've been pretty disconnected from things over the past month due to a family trip to the Philippines so I understand.  I'm still sick and tired after the flight home, hopefully I'll be getting better soon.

Manual release looks broken... https://github.com/onedr0p/containers/actions/runs/6441000903
Are we going with appsToBuild or imagesToBuild?

N/m I think I fixed that part.

There looks to be other problems as well, not sure when I get time to hack at fixing them.


WIP @someone: Fixes CI

Fixes #500.

Becoming more a general-purpose performance PR. I'm OK with this.
Really glad when we move to a FRP lib that uses the compiler to warn us we forgot .takeUntil(viewWillDisappearSignal()).

So basically, there are problems with fulfillmentNavigatonController being retained everywhere. I've been setting a breakpoint in the nav controller's deinit function to make sure it's called under different scenarios. Here's the state of this PR right now:
Broken
Bid -> Bid Button -> Paddle/Pin -> Bid -> [Cancel]
Bid -> Bid Button -> Paddle/Pin -> Bid -> Place Another bid
Fixed
Bid -> Cancel
Bid -> Bid Button -> Cancel
Bid -> Bid Button -> Artsy Login
Bid -> Bid Button -> Unknown Phone Number
So now it's just sign-in/bidding that's causing the problem. Will continue.

Yesterday, Andy Matuschack said something to the effect of "it's bananas that you can write a reference cycle in Swift" and I think he was right.

@ashfurrow It’s kind of inherent with ARC style systems. You have to know what you’re doing, except you don’t need to do all the actual work. You can get a lot better with a garbage collector, but even then it’s still possible. (Did you know the Ruby stdlib comes with a WeakRef helper class?)
On whether or not it should be possible I agree, the ideal language is one where you don’t need to.

The cycle fixes look good to me, but you should probably let someone that’s used to Swift+RAC take a look as well.

Agreed on all counts, thanks for the feedback! Andy's talk was kind of "what might swift have looked like if it didn't need ObjC interop?" A neat thought experiment.

The remaining issue is really the following cycle, which affects anything that touches loading (registration, bidding).

Fulfillment controller -> Loading VC -> Loading VM -> Fulfillment controller

The fulfillment controller is used primarily to access the logged-in provider. But the provider doesn't exist until the user is logged in. Quite the pickle.
The tests will need to be updated, too.

Phew, ok.

Tests passing locally. I solved the issue mentioned above using unowned references, and also spotted a reference cycle in didSet for xAccessToken, so any time the user was logged in, there was a leak.
Hopefully Travis will pass. If not, well, there's a Trello card for that, too.

Yup, same problem as before.
@orta this is good to merge now, passes locally. I'll deal with the CI issues separately so we can get this merged.

Looks cool to me

No substantially changes, since CI is not reliable and I got the :+1: I'm going to merge.


Hubble relay doesn't connect to hubble-peers when prometheus is enabled

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
With httpV2 prometheus metrics enabled, Hubble relay fails to connect to peer with the following issue
level=warning msg="Failed to create peer client for peers synchronization; will try again after the timeout has expired" error="context deadline exceeded" subsys=hubble-relay target="hubble-peer.kube-system.svc.cluster.local:80"
With the following values, it works and the connection happens:
cluster:
  name: belt

hubble:
  enabled: true
  metrics:
    enabled:
      - dns:query;ignoreAAAA
      - drop
      - tcp
      - flow
      - icmp
      - http
      - policy:sourceContext=app|workload-name|pod|reserved-identity;destinationContext=app|workload-name|pod|dns|reserved-identity;labelsContext=source_namespace,destination_namespace
      - flow:sourceContext=workload-name|reserved-identity;destinationContext=workload-name|reserved-identity
      #- httpV2:exemplars=true;labelsContext=source_ip,source_namespace,source_workload,destination_ip,destination_namespace,destination_workload,traffic_direction;sourceContext=workload-name|reserved-identity;destinationContext=workload-name|reserved-identity
    enableOpenMetrics: true
    serviceMonitor:
      enabled: true

  tls:
    enabled: false

  relay:
    enabled: true
    prometheus:
      enabled: true
      port: 9966
      serviceMonitor:
        enabled: true

prometheus:
  enabled: true
  serviceMonitor:
    enabled: true

operator:
  enabled: true
  prometheus:
    enabled: true
    serviceMonitor:
      enabled: true

I get this logs on hubble relay:
level=info msg=Connecting address="<IP_ADDRESS>:4244" hubble-tls=false peer=belt/tycho subsys=hubble-relay
level=info msg=Connecting address="<IP_ADDRESS>:4244" hubble-tls=false peer=belt/pallas subsys=hubble-relay
level=info msg=Connecting address="<IP_ADDRESS>:4244" hubble-tls=false peer=belt/ceres subsys=hubble-relay
level=info msg=Connected address="<IP_ADDRESS>:4244" hubble-tls=false peer=belt/ceres subsys=hubble-relay
level=info msg=Connected address="<IP_ADDRESS>:4244" hubble-tls=false peer=belt/pallas subsys=hubble-relay
level=info msg=Connected address="<IP_ADDRESS>:4244" hubble-tls=false peer=belt/tycho subsys=hubble-relay

With this values, hubble relay doesn't connect with peers:
cluster:
  name: belt

hubble:
  enabled: true
  metrics:
    enabled:
      - dns:query;ignoreAAAA
      - drop
      - tcp
      - flow
      - icmp
      - http
      - policy:sourceContext=app|workload-name|pod|reserved-identity;destinationContext=app|workload-name|pod|dns|reserved-identity;labelsContext=source_namespace,destination_namespace
      - flow:sourceContext=workload-name|reserved-identity;destinationContext=workload-name|reserved-identity
      - httpV2:exemplars=true;labelsContext=source_ip,source_namespace,source_workload,destination_ip,destination_namespace,destination_workload,traffic_direction;sourceContext=workload-name|reserved-identity;destinationContext=workload-name|reserved-identity
    enableOpenMetrics: true
    serviceMonitor:
      enabled: true

  tls:
    enabled: false

  relay:
    enabled: true
    prometheus:
      enabled: true
      port: 9966
      serviceMonitor:
        enabled: true

prometheus:
  enabled: true
  serviceMonitor:
    enabled: true

operator:
  enabled: true
  prometheus:
    enabled: true
    serviceMonitor:
      enabled: true

No firewall is enabled on the nodes:
$ ufw status
Status: inactive

I have checked everything is ok following this issue : https://github.com/cilium/cilium/issues/20130#issuecomment-1437045395

Multiple reinstallation has been done either with Helm, the CLI or kubectl apply (via ArgoCD which I use for deploying everything) and each time
No TLS is used because I use insecure (I tried with TLS too and doesn't work either)
cluster domain is cluster.local

When I look if port 4244 (used by hubble-peers) is binded on the node, I get a result when prometheus is false:
# lsof -i -P -n | grep 4244
kube-apis    1485            root  299u  IPv6 125119759      0t0  TCP <IP_ADDRESS>:6443-><IP_ADDRESS>:42446 (ESTABLISHED)
cilium-ag 1037115            root   55u  IPv6 158182034      0t0  TCP *:4244 (LISTEN)
cilium-ag 1037115            root   79u  IPv6 158187002      0t0  TCP <IP_ADDRESS>:4244-><IP_ADDRESS>:33020 (ESTABLISHED)

When prometheus is true, there is nothing listening on :4244.
If I look in the chart, I see that this part are added to cilium-agent Daemonset when enabling prometheus metrics:
{{- if .Values.prometheus.enabled .Values.hubble.metrics.enabled}}
ports:
- name: peer-service
        containerPort: {{.Values.hubble.peerService.targetPort}}
        hostPort: {{.Values.hubble.peerService.targetPort}}
        protocol: TCP
{{- if.Values.prometheus.enabled}}
- name: prometheus
        containerPort: {{.Values.prometheus.port}}
        hostPort: {{.Values.prometheus.port}}
        protocol: TCP

I understand that cilium-agent runs with hostNetwork: true to bind its container port to the host port automatically so I don't get why it's explicitly added with hostPot in the ports section.
I'll gladly accept any help because it forces me to run Hubble without any of the useful features as L7 observability or policy verdict to achieve zero trust :/
Cilium Version
1.14.0 with kube-proxy
Kernel Version
Linux 5.15.0-76-generic
Kubernetes Version
1.27.3 on Baremetal installed with kubeadm
Sysdump
No response
Relevant log output
No response
Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Ok somehow, the issue was with the metrics enabled because with this it works:
   - dns
   - drop
   - tcp
   - icmp
   - "policy:sourceContext=app|workload-name|pod|reserved-identity;destinationContext=app|workload-name|pod|dns|reserved-identity;labelsContext=source_namespace,destination_namespace"
   - "flow:sourceContext=workload-name|reserved-identity;destinationContext=workload-name|reserved-identity"
   - "kafka:labelsContext=source_namespace,source_workload,destination_namespace,destination_workload,traffic_direction;sourceContext=workload-name|reserved-identity;destinationContext=workload-name|reserved-identity"
   - "httpV2:sourceContext=workload-name|pod-name|reserved-identity;destinationContext=workload-name|pod-name|reserved-identity;labelsContext=source_namespace,destination_namespace,traffic_direction"

So maybe enabling both httpV2 and http was the route cause


Set engine env from common config

vendor commond PR https://github.com/containers/common/pull/198 to set container_engine_env
related issue  https://github.com/containers/common/issues/31
Signed-off-by: Qi Wang<EMAIL_ADDRESS>

@rhatdan @mheon  PTAL

LGTM
@mheon @baude @giuseppe @vrothberg @TomSweeneyRedHat @ashley-cui @jwhonce PTAL

@rhatdan @mheon PTAL

/approve
/lgtm


set slice capacity with 3-index syntax

fixes https://github.com/edgedb/edgedb-go/issues/60

This change is based on https://github.com/edgedb/edgedb-go/pull/72. WIP until upstream is merged.

This change is based on https://github.com/edgedb/edgedb-go/pull/72. WIP until upstream is merged.


feat(docs): add Circle CI badge to README

Please verify the following:

[ ] yarn ci:test jest tests pass with new tests, if relevant
[x] README.md has been updated with your changes, if relevant

Describe your PR
Add Circle CI badge to README :

감사합니다!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


About the warm-up phase

I am interested in the warm-up phase mentioned in your paper. But I do not find that in the code.
When I ran the training code, I tried to add a 500,000-step warm-up phase, in which I set lambda = 1e-5 and lr = 5e-5.
In the warm-up phase, I kept the lr constant (and I don't know if it is right).
Unfortunately, I can't get the same result as the model you provided.
So, could you please provide a detailed explanation of the settings for the warm-up phase?

lr=5e-5 should be fine. Or can you try to use smaller lambda, like 1e-6? Optionally try to warm-up the model for more steps? like 700,000?


Send ICECandidate using ToJSON()

Send ICECandidate using ToJSON() rather than String(), same as in the grpc server.
ToJSON() returns sdpMLineIndex  along with the candidate string, where as String() only returns the candidate string.

thanks!

No worries


Resizing while file tree is on the right

I cannot resize ftp-remote's distant tree view if i activate "Show On Right Side" option in tree-view package ;)

The next version will no longer have this option. Only the "native" Dock can be used, and the resize will work there. I close this issue.


watchCollection  on "dotted" properties

Hi,
We experience an issue when using directive scope field containing a "dot" in the name, I mean the field is named for instance "dom.obj", i.e. obj is not a "dom" field. the actual property name contain a "."
BTW if I remove the "." in the name and at the watchCollection first parameter, (expression) it works
Is this scenario not supported?
Thanks

Hi Lucas,
That works (we missed the 'this'), thanks!
@frfancha you are right... but the reason is that these are property from a schema generated on server side ... and acts like a key to property collection member.
We do not use explicitly "dotted properties"
Thanks

Hi Lucas,
That works (we missed the 'this'), thanks!
@frfancha you are right... but the reason is that these are property from a schema generated on server side ... and acts like a key to property collection member.
We do not use explicitly "dotted properties"
Thanks


Support adding a prefix to each class name in the output.

Clear and concise description of the problem
Use Case
In my main project, I am using UnoCSS. I also dynamically import components from another project that also uses UnoCSS. However, when importing components from the other project, the CSS loading order sometimes changes, affecting the styles in the main project.
To avoid conflicts, I would like to add a prefix to the class names in the CSS output of the secondary project. For example, if I use the m-0 syntax in the secondary project, the final build output should be something like prefix-m-0 with the corresponding CSS:
.prefix-m-0 {
  margin: 0;
}

This way, the styles from both projects won't clash.
Suggested solution
Support adding a prefix to each class name in the output.
Alternative
No response
Additional context

Attempted Solution
As shown in the screenshot, I tried writing a transformer to achieve this. The red parts represent the main module, and the blue parts represent the submodule. You can see that the class names in the blue parts have been prefixed with x-.
Unfortunately, this transformer does not handle every type of class name correctly. I am not very familiar with transformers, so I am hoping the official team can implement this feature. I believe this would be very helpful in resolving style priority conflicts between open-source components using UnoCSS and the developer's main project using UnoCSS.

The issue I encountered is that I used regular expressions to match class names. However, in my project, I use clsx to dynamically select different classes. Therefore, I am unsure of a good solution to accurately match these and then add the prefix.
Validations

[X] Read the Contributing Guidelines.
[X] Read the README.md of using the package.
[X] Already used the Interactive Docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Did you try the compile class transformer? https://unocss.dev/transformers/compile-class

Yes, I found this transformer. It works well, but I need to add :uno: to all class names. Therefore, I prefer to find a solution that doesn't require me to review all the existing code again. Perhaps adding a default active configuration to the compile-class transformer would also be a good option.😝

I think prefix option is enough.
Playground

Or you can use this hook to change outputs. docs postprocess


Update tar dependency to 0.4.35

Pulls in a fix which should avoid 0 mtime files from showing up.
Closes #9512

r? @ehuss
(rust-highfive has picked a reviewer for you, use r? to override)

❤
@bors r+

:pushpin: Commit a02b6e5bfc56fb59d36ac865cd4b3ee4c3b5b54a has been approved by ehuss

:hourglass: Testing commit a02b6e5bfc56fb59d36ac865cd4b3ee4c3b5b54a with merge cc75485f72a6aaf871d4368b4b8eb01046f3af28...

:sunny: Test successful - checks-actions
Approved by: ehuss
Pushing cc75485f72a6aaf871d4368b4b8eb01046f3af28 to master...


Issue 751 Add an example for a simple reverse proxy to the README

Add an example for a simple reverse proxy to the README.
Fixes #751

Fantastic! Thank you so much! :heart_eyes:

Not at all.  Thanks for writing and maintaining the module.  4 lines of code for a working Nginx reverse proxy is great value. :)


net: implement multicast network interface API on NetBSD

See https://github.com/golang/go/blob/master/src/net/interface_netbsd.go.

I am learning go - lot of expertise on C and doing router software. Would love to contribute.

@amaitra, feel free to send changes!

I have never worked on this kind of project before - so some FAQs or guidance would be appreciated.


Filters are bugged / Cannot change anything.

It's hard for me to add a screenshot of the bug, but as the title says: I cannot change anything in the app.
Whatever I change, settings, filters, channels etc. it's getting reverted to the previous last functional state. I'm also not sure if the filters work because I still see a lot of WTS, GDKP Posts even if they are blacklisted. Do you have any idea how I can reset the add-on?

After you make your changes are you clicking the "X" button or hitting the "Close" button after making changes?
As far as seeing WTS, GDKP etc..., did you add those words to the "blacklist words" box and did you then also have "Custom" checked under search patterns (along with whatever language)

can confirm this is a problem in sod

can confirm this is a problem in sod.
To Reproduce:

Click any setting checkbox (e.g. Lock minimap button position)
Close config or click away to another config screen
Open config. See that Lock minimap button position is unchecked.

I've asked other players if they have this issue. Some do. Some do not.

After you make your changes are you clicking the "X" button or hitting the "Close" button after making changes?

Yes, interestingly, pressing the close Button instead of X resolved the
problem for me.
On Sun, Feb 4, 2024 at 7:46 PM Vysci @.***> wrote:

can confirm this is a problem in sod.
To Reproduce:

Click any setting checkbox (e.g. Lock minimap button position)
Close config or click away to another config screen
Open config. See that Lock minimap button position is unchecked.

I've asked other players if they have this issue. Some do. Some do not.
After you make your changes are you clicking the "X" button or hitting the
"Close" button after making changes?
—
Reply to this email directly, view it on GitHub
https://github.com/Vysci/LFG-Bulletin-Board/issues/199#issuecomment-1925878556,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPN5VA2LAYTC4XWRL3JLUDYR7JPXAVCNFSM6AAAAABCXC2TV6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRVHA3TQNJVGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

oh, it does for me too. Pressing "Close" saves the settings. I never click the X or Close. I have always pressed ESC when done with the options window.

I made a bug report for this. I'll see if I can fix it.

Using #200 to track this issue instead.


Collsion in response and request headers

If response or request have header with same name, response.headers.valuesOf('Header name') return last header

Can you give me an example on this?

Its more i wrote wrong description,
For example you have in response or request headers like
HTTP/1.1 200 OK
Cache-Control: no-cache, private
Content-Type: application/json
Set-Cookie: Access-Token=value
Set-Cookie: Refresh-Token=value
Vary: Accept-Encoding
Transfer-Encoding: chunked

When i try get access to header you get last one
response.headers.valuesOf('Set-Cookie')

RFC 2616 says it is possible to send multiple values in separate header lines only when it is possible to also combine the values by comma in one header line. Currently it is not supported in Kulala.
Some implementation always return an array, some will combine these to a comma separated list (which I think is the only solution to have backward compatibility)

In jetbrains it's looks like
 response.headers.valuesOf('Set-Cookie').forEach(function (value) {/*... */}

In jetbrains it's looks like
 response.headers.valuesOf('Set-Cookie').forEach(function (value) {/*... */}

Will try in jetbrains and see if we can implement it in kulala.

What is the result when there is only one occurrence? A string (so the caller has to check the result type) or an array with one element (that would be a breaking change in scripts).
we need to decide what we do with the Request Variables: vscode-rest-client returns the first one.
As RFC defines all implementations must accept both forms maybe we return the comma list at least in Request Variables.
if you test against JetBrains please also check if they deliver an array of given as list in one header.

I don't use JetBrains, but my college use like this
JetBrains doc https://www.jetbrains.com/help/idea/http-response-reference.html#headers-reference
 response.headers.valuesOf('Set-Cookie').forEach(function (value) {
        const parts = value.split('=');
        const settings = parts[1].split(';');

        if (parts[0] === 'First_Variant') {
/* ... */
        }

        if (parts[0] === 'Second_Varoant') {
/* .. */
        }
    });


[QA] Several collaborators appear in the subfolders

Tested with

ocis-1.0.0-rc1 via https://github.com/owncloud-docker/compose-playground/blob/master/examples/hetzner-deploy/make_ocis_eos_compose_test.sh
firefox and Chrome in Mac

Steps to reproduce

user A shares a file with user B
2 . user B accepts the share and file appears
User B creates a folder in home path
Browse into the folder
Create a subfolder

Expected behavior
Collaborators shown should be only the own user
Current behavior
It appears 2 collaborators, user A and user B

yes, this is by design in Phoenix.
you see the file owner also as a collaborator
the purpose is so that anyone can see who else has access to the resource, including self

ok, so according to the report it shows the wrong collaborators on unrelated folders, likely a bug with EOS.

@ishank011 didn't you find something similar?

Yes, whenever a share is created with a user, all resources of the recipient show the share creator as the owner. And it's not in EOS, it's seen with owncloud file system as well https://github.com/owncloud/phoenix/issues/3895

but according to the bug report, it even shows the share creator on non-shares resources, as if the OCS share query was returning wrongly filtered results

Yeah exactly. For all of the recipient's resources, it shows the share creator as the owner. I'm guessing the bug is that it is supposed to show that only for the shared resource, but it displays the same for all resources. So the OCS queries are probably correct, but phoenix uses the wrong set of results.

Yeah exactly. For all of the recipient's resources, it shows the share creator as the owner. I'm guessing the bug is that it is supposed to show that only for the shared resource, but it displays the same for all resources. So the OCS queries are probably correct, but phoenix uses the wrong set of results.

Yes, that is exactly the behavior I wanted to explain in the open post

but it works fine when using OC 10 as a backend, so I doubt it's about filtering on the JS side.
needs further research


Documentation now available (questions here)

Hi!
Documentation is now available!
https://rlpyt.readthedocs.io
Feel free to post in this issue for minor clarifications / comments, or start a new issue if it's something bigger.
Hope this helps!!

Thanks @astooke
Just a quick comment, the Gym and Environments data doesn't seem to be available on the website. For example I see documentation here:
https://github.com/astooke/rlpyt/blob/master/rlpyt/envs/atari/atari_env.py
but the corresponding docs page seems to be missing:
https://rlpyt.readthedocs.io/en/latest/pages/env.html#
Not the biggest deal as it is easy to understand the open source code in this particular case, but it may help for the website to have the complete documentation.

Thanks @astooke, is there an example for a simple gym environment like cart pole or mountain car? I found examples of atari and mujoco, but I think it is better to provide an example with simple environments like classic continuous control in gym

@ekorudiawan Agree, very eager to see that, even if only one example is good.

Hi Alex,
First thank you for adding all the documentation! It's tremendously helpful. I had one question about rlpyt.agents.pg.categorical.CategoricalPgAgent.
In the CategoricalPgAgent documentation, you write "[CategoricalPgAgent] has a different interface to the model (model here outputs discrete probabilities rather than means and log_stds." But in the source code, it looks like the model should output a tuple of (pi, value), where pi is a vector of discrete action probabilities and value is a vector of values associated with each.
Which is correct? Thanks!

Just a quick comment, the Gym and Environments data doesn't seem to be available on the website.

@DanielTakeshi Good catch, thanks!  Fixed now (had to mock import atari_py and maybe gym).

@bpiv400 Oops sorry for the confusion, I've just updated the wording in the documentation to clarify that the model should output (pi, value), like in the code.

is there an example for a simple gym environment like cart pole or mountain car?

@ekorudiawan Good idea, no such example exists yet...adding it to the to-do list to make one.  Let me know if you beat me to it ;)

Hi,
Thanks for the documentation. After reading through I'm still confused on the dynamic of batch_T / batch_size /  replay_ratio.
In specific, I'm trying to recreate the training loop here:
the idea is there is train_step / batch_size / gradient_step, and the pseudocode would be:
for step in training_step:
env.step(); add tuple to replay buffer
if step % train_step == 0:
for i in range(gradient_step):
batch = buffer.sample() of size batch_size
update model with batch
Is it possible to replicate this behavior or come close to it in rlpyt? For now I am trying to approximate train_step=100, batch_size =128, gradient_step=100 with batch_T=100, batch_size=128, replay_ratio=1.28.

@bycn OK yes let's see... yes if you use batch_T=100, then the environments will do 100 time-steps in between training, so that's like train_step=100, if you use batch_B=1, which is the number of parallel environments (you could use any batch_T * batch_B = 100).  But your replay ratio needs to be 128, because it's data_consumed / data_generated, which in your case is gradient_step * batch_size / train_step.  Does that make sense?

Could you please add an example of how to use the algorithms? It would be really helpful — perhaps show PPO with a custom Gym environment and custom network architectures.

Hi,
I am experimenting with examples/example_5.py, trying to reproduce results of the original Mnih et al. Nature paper from 2015.
I have started with the following parameters (not touching the source for now): python example_5.py --cuda_idx=0 --n_parallel=20 --game="space_invaders"
Where can I find the trained model? I see lines like dqn_space_invaders_0 itr #181199 saving snapshot... in the log, but where are these snapshots being saved to?

@jetsnguns
Look at data/local and you should see example_5 as a sub-directory. Snapshots are hopefully saved there but only if you have set them to be saved. You may want to add the "snapshot_mode" argument to the logger. Please see my comment here: https://github.com/astooke/rlpyt/issues/66
Just a quick note that the results from the Mnih et al paper were done with a serial sampling code, and examples/exampe_5.py uses the GPU sampler. Thus, this will introduce some slightly different behavior. If you are looking to exactly reproduce Mnih et al, I recommend examples/example_1.py but you'll also need to slightly adjust the exploration schedule, and then to make the images 84x84 instead of 104x80, and many other minor changes. It's a bit cumbersome. That's why papers really need open source code. :)

@DanielTakeshi,
Thanks for a prompt reply!
Yes, I have looked at that folder, but there are only debug.log  params.json  progress.csv files present. I've also looked at the comments you mentioned and will proceed to try it out.
Perhaps it will make it less confusing to remove the messages like:
2020-05-16 05:40:54.588538  | dqn_space_invaders_0 itr #181199 saving snapshot...
2020-05-16 05:40:54.588735  | dqn_space_invaders_0 itr #181199 saved

that I see in the log when nothing is actually saved.

Hi,
I'm experimenting with a custom env on rlpyt. I've the intention to use different data for training and testing (env shows novel states on testing/evaluation vs training).
I have been using example_1 as stepping stone, so far so good, but I'm not sure how to achieve this last bit: testing.
After running runner.train() (as in example_1, inside the logger), I think I should use runner.evaluate_agent() inside a loop to evaluate the agent several times (or maybe use eval_max_steps=NumEvals as SerialSampler argument?).
But I'm a bit (more) lost on how to "send" the test signal to my environment from here, as only the SerialSampler 'knows' where the environtment class is, and I cannot find a way to use it to send some arguments to the env in this phase.
In short (I don't know if i'm being clear) I need to know how:

to test/eval my agent
to send a "test" argument to my custom env on testing phase

Thank you!

hi @jetsnguns , thanks for your question, and you're right, the agent parameters are not being saved by default.  as @DanielTakeshi ponited out, you can add the snapshot_mode argument to the logger_context as:
with logger_context(log_dir, run_ID, name, config, snapshot_mode="last"):

you should see a params.pkl (different from params.json) in that same folder.
to see the snapshot modes available in the logger:
https://github.com/astooke/rlpyt/blob/668290d1ca94e9d193388a599d4f719bc3a23fba/rlpyt/utils/logging/logger.py#L332

@LecJackS Hi sorry for the unclear title, I meant for this thread to be for question about the documentation...let me clarify that in the title...mind copying your question over to a new issue and I'll reply there?


Added button to enable/disable background

Added button to enable/disable background

Can you describe the purpose and show the results with some snapshots?
FYI there is already a variant that disable background rotation

My edit adds a button that completely removes the background making it completely blank for who don't like the backgorund image...
If you don't want it, feel free to reject the idea...

I actually didn't know of the NoAnimatio variant, so I'll close the PR

Maybe this option should be better advertised but I'm not sure how


master to 2.0.0 release candidate

Coverage increased (+0.6%) to 82.501% when pulling 066bfd2a4b290a8f9d60c769626b8cce0c7fa6e0 on jmi-2toM into 43854725d2088c548573a495f4aa735d83f59a11 on 2.0.0-RC.


Serializing things as http headers

From @deissnerk, do we need something like https://github.com/cloudevents/spec/blob/main/cloudevents/bindings/http-protocol-binding.md#3132-http-header-values in the xReg spec?

I think we can close this now since we have this section in the spec: https://github.com/xregistry/spec/blob/main/core/spec.md#http-header-values
@deissnerk let me know if I need to reopen it.


Enhancement: Constrain available Media Use terms to relevant addable media types

Currently, if I go to Content > Media and click the Add Media button, I first have to choose a media type (Audio, Document, Extracted Text, File, FITS Technical metadata, Image, Remote video, or Video).
If I select Audio, I am taken to an "Add Audio" form.
The list of check-able Media Use terms for my audio file include all the Islandora Media Use taxonomy terms, even though most of them are relatively nonsensical for an audio file.
What even happens if I indicate that my Audio file is Media Use: FITS File? Why can I say an audio file should be used as a Thumbnail Image? Presumably there's a reason for making Media Use value selections, and being able to set them so wrong is concerning.
Having to choose from the entire list increases cognitive load and opportunity for user error.
A possible solution?

Add an "Applies to Media Type" (or some other name) field in the Islandora Media Use taxonomy. This should be a multi-valued field. Its allowable terms are the "Add Media" types.
The form at media/add/* pages use a view of the Islandora Media Use taxonomy limited to "Applies to Media Type" = * to generate the list of Media Use terms in the form

Appropriate mappings (initial/tentative)
Top level: Islandora Media Use terms
Indented level: Addable media types
Note: On the media/add page, some clarification of the difference between Document and File would be useful. I am proceeding assuming that Document expresses something about being a human readable formatted document (which would include .pdf and .docx and .odf formats among others), while File is more about plain-text files primarily for system use.

Audit Trail

file
not defined in https://pcdm.org/2015/05/12/use but described in taxonomy term as "Audit trail generated by Fedora 3"

Collection Policy

file
not defined in https://pcdm.org/2015/05/12/use but described in taxonomy term as "Islandora 7.x Collection Policy File"

Dublin Core File

file

Extracted Text

extracted_text

FITS File

fits_technical_metadata

Intermediate File

audio, document (?), image, video

MODS File

file

Original File

audio, document, file (?), image, video

Preservation Master File

audio, document, file (?), image, video

RELS-EXT File

file
not defined in https://pcdm.org/2015/05/12/use but described in taxonomy term as "A RELS-EXT file from an Islandora 7.x instance"

Service File

audio, document (?), file (?), image, video

Thumbnail Image

image

Transcript

document, file
Unclear what the function of this is, from the taxonomy description: "A textual representation of the Object appropriate for presenting to users, such as subtitles or transcript of a video. Can be used as a substitute or complement to other files for accessibility purposes" -- if this should be usable by system for subtitles or video/audio-to-transcript text sync, probably only file is appropriate. If this should be a transcript that itself (on its own) is appropriate for presenting to users, then both document and file seem appropriate

I suppose we could add a form alter hook that filters the available options based on a Type→term mapping (the inverse of the one in the OP for efficiency's sake) stored in config. The hook would be relatively simple, the config form would be a bit more difficult but still do-able.
OR we can add a field to the taxonomy allowing you to select which types it applies to and use a view filter on the form, that may be simpler (and purely config!)...
As for the types in question marks, I would say yes to each of them, but also add 'file' to Intermediate File. We have edited Tiffs that are Intermediate Files which Drupal consider to be the File type.

I may not have the terminology right, but I'm pretty sure you can add a filter to an entity reference form field (i.e. one populated with a taxonomy term).
In poking at another problem, I'd created a Role Type taxonomy with a Role Category field (creator, contributor, publisher, etc) for each term.
I was able to create a Creator field on my content nodes and a Creator form field populated from the Role Type taxonomy, but limited only to terms with Role Category = creator.
Since I actually made that work elsewhere, that's why it popped to my mind as a solution here.
I have no clue whether it's the BEST solution, and I also wondered if the "show the term options as checkboxes in the form" instead of "populate a dropdown in the form" might not work, but that seemed like it would be weirdly inconsistent behavior and I hoped it wouldn't be a problem.

I think multi-file media may alleviate this.  It doesn't use Media Use terms, and instead you would use analagous field names.  E.g. an Original File is not tagged as such, instead it lives on the media as field_original_file.
Right now it's totally do-able, but we have zero documentation on it.  It's still new.

@dannylamb Is that the same "multi-file media" referenced here: "Multi-file Media configurations also attach Media to a parent node, but allow for that node to be represened by multiple "Original File"s. In this model, a Media contains the original file as well as any derivative files created from it (thumbnail, service file, etc.)."
If so, do you think that alleviates this because (guessing) whatever files are added to a resource node as Media using that model are assumed to be original files, and then the system tracks the use/types of derivative files created?
Though it sounds like using multi-file media would be an option and this issue would still remain when uploading Media to content nodes not using multi-file media model. (?)

Yes it is the same.  And yes, it "solves" the problem by entirely sidestepping it, which doesn't help folks who aren't using the multi-file media approach. So maybe it wasn't that helpful of a suggestion.  We'll still need to find a way to lock things down how they work right now.

Here I have added a File media type (.txt file) with Media Use value = Thumbnail Image. No warnings or complaints!

Of course I get no thumbnail:

I try again and actually do it right. It totally lets me attach two media with Media Use value = Thumbnail Image to the same node. No warnings or complaints!
Doesn't fix the actual display as long as the bad media file is there:

Deleting the media/file of the File media type makes my fixed thumbnail show up.

But why would you DO THAT?
Because you can. So someone will. On accident or out of sheer confusion. And they shouldn't be able to, because a system should be smart enough to know it can't use a .txt file as a thumbnail or a .mp4 as a FITS File or whatever other weird quietly not-working possibilities are currently available.


Non-VR camera

We need either a screenspace camera or a perspective camera.
Even if a user is doing only VR, then a non-headtracking camera is used many times for overlay effects and static layers/cards you display for the user.
If you have some good workarounds for this using the VR camera, then that works too.

re-enabled the perspective camera


Add documentation for configuring Stackdriver Logging agents

This PR introduces guide for configuring Stackdriver fluentd instances in the cluster.
/cc @igorpeshansky

This change is

LGTM from me. I'd like to see also @igorpeshansky's review.

@Bekt Deploying section is missing because I was doing two PRs in parallel: this and https://github.com/kubernetes/kubernetes.github.io/pull/2922 Now that the latter is merged, link should be working.

Also does it make sense to link to/from cluster/addons/fluentd-gcp somewhere in this page?

Good question. I deliberately didn't mention it and made a local copy of the DaemonSet and the ConfigMap to avoid problems with the k8s versions, i.e. if something changes in master, docs should not be affected. Does it make sense?

@igorpeshansky Will you have time to take at look at it?
@devin-donnelly Could you please take a look at the text?

@igorpeshansky @devin-donnelly Friendly ping
It would be really great to be able to show this documentation to the users.

@igorpeshansky @devin-donnelly Friendly ping

@igorpeshansky PTAL
@chenopis Sorry, I've just realized that the problem from https://github.com/kubernetes/kubernetes.github.io/pull/3451 was introduced because I separated two PRs and this one wasn't merged before release branch was merged to master. Thanks for keeping an eye on the repo!

@crassirostris No prob

@crassirostris Since devin-donnelly has been busy, I just did the docs review.

@chenopis Thanks! I addressed your comments


Fix link

Signed-off-by: Sameer Raheja<EMAIL_ADDRESS>Fix doc link in tests-spark310+/README.md

build

build


Missing autoload.php

Hi
I downloaded the master branch .I  tried to create a new profile using CustomerProfiles/create-customer-profile.php  . But I get the warning message  "failed to open stream: No such file or directory
require(vendor/autoload.php)"  .There is no autoload.php or vendor directory .

Hi ,
"composer update"  fixed the issue.


Fixes edwko/OuteTTS#4

This fix is to guarantee that force-align always receives mono audio to process.

Thanks for the PR and good catch! I've implemented the fix with a different approach, using waveform = waveform.mean(dim=0, keepdim=True) to handle the mono conversion directly within the align function for better consistency.