[Tree] left position should respect root in another table

If my category has a root in another table, the Nested Strategy should start the left position for each root separately.
Test case you can see in NestedTreeRootAssociationSubTest
Food <- stored in another table
| - Fruits should has [1, 4]
    | - Apples should has [2, 3]
Sports <- stored in another table
| - Summery should has [1, 6]
    | - Basketball should has [2, 3]
    | - Football should has [4, 5]

Current behaviour is
Food <- stored in another table
| - Fruits has [1, 4]
    | - Apples has [2, 3]
Sports <- stored in another table
| - Summery has [5, 10]
    | - Basketball has [6, 7]
    | - Football has [8, 9]

Any movements here?)

Hi! This was auto-closed because I renamed/removed master. I've marked this as Still Relevant so I know there is some validity to this addition still. If you're able, please create a new PR targeting the main branch. Thank you 😄


tools: add option to set config file to check style

To make it possible to be used by other projects.
Signed-off-by: Bruno Dilly<EMAIL_ADDRESS>

:+1:

:+1:

merged, thanks


add "audacity project" ('.aud') format

would be helpful if there was a sample dataset
branch already in progress, raising this issue to actually track it

should add "audacity text" format before trying to deal with the xml of .aud files

Closing as discussed in #96


Fix various md files

Code of Conduct does not have a contact name.

Also need to remove line 1 from the license file

Also need to update the refs/README.md file

README titles should be centered.


FileNotFoundError: [Errno 2] No such file or directory: 'data/nuscenes/v1.0-trainval/attribute.json'

I followed the installation instructions in the README, but I encountered some issues during the dataset installation step.
I downloaded the dataset from the nuScenes website, but I wasn't sure which files I needed to download. I made an educated guess and downloaded the following:

nuimages-v1.0-all-samples.tgz
nuimages-v1.0-all-sweeps-cam-back-left.tgz
nuimages-v1.0-all-sweeps-cam-back-right.tgz
nuimages-v1.0-all-sweeps-cam-back.tgz
nuimages-v1.0-all-sweeps-cam-front-left.tgz
nuimages-v1.0-all-sweeps-cam-front-right.tgz
nuimages-v1.0-all-sweeps-cam-front.tgz
nuplan-maps-v1.0.zip
nuScenes-lidarseg-all-v1.0.tar

I extracted these datasets to the specified directory.
However, when I tried to run either the train or eval.py scripts, I encountered the following issue:

Namespace(gpus=8, py_config='config/tpv04_occupancy.py', resume_from='', work_dir='out/tpv_occupancy')
tcp://<IP_ADDRESS>:20506
tcp://<IP_ADDRESS>:20506
tcp://<IP_ADDRESS>:20506
tcp://<IP_ADDRESS>:20506
tcp://<IP_ADDRESS>:20506
tcp://<IP_ADDRESS>:20506
tcp://<IP_ADDRESS>:20506
tcp://<IP_ADDRESS>:20506

..................

2023-04-21 00:11:26,596 - mmseg - INFO - Config:
2023-04-21 00:13:02,052 - mmseg - INFO - initialize FPN with init_cfg {'type': 'Xavier', 'layer': 'Conv2d', 'distribution': 'uniform'}
2023-04-21 00:13:03,511 - mmseg - INFO - Number of params: 62465552
done ddp model
Loading NuScenes tables for version v1.0-trainval...
Traceback (most recent call last):
File "train.py", line 401, in
torch.multiprocessing.spawn(main, args=(args,), nprocs=args.gpus)
File "/root/miniconda3/envs/cat_TPV/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 230, in spawn
return start_processes(fn, args, nprocs, join, daemon, start_method='spawn')
File "/root/miniconda3/envs/cat_TPV/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 188, in start_processes
while not context.join():
File "/root/miniconda3/envs/cat_TPV/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 150, in join
raise ProcessRaisedException(msg, error_index, failed_process.pid)
torch.multiprocessing.spawn.ProcessRaisedException:
Process 6 terminated with the following error:
Traceback (most recent call last):
File "/root/miniconda3/envs/cat_TPV/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 59, in _wrap
fn(i, *args)
File "/root/data01/zzy/TPVFormer/train.py", line 99, in main
data_builder.build(
File "/root/data01/zzy/TPVFormer/builder/data_builder.py", line 20, in build
nusc = NuScenes(version=version, dataroot=data_path, verbose=True)
File "/root/miniconda3/envs/cat_TPV/lib/python3.8/site-packages/nuscenes/nuscenes.py", line 70, in init
self.attribute = self.load_table('attribute')
File "/root/miniconda3/envs/cat_TPV/lib/python3.8/site-packages/nuscenes/nuscenes.py", line 136, in load_table
with open(osp.join(self.table_root, '{}.json'.format(table_name))) as f:
FileNotFoundError: [Errno 2] No such file or directory: 'data/nuscenes/v1.0-trainval/attribute.json'

Since I am not entirely certain if I downloaded the correct dataset, I would like to ask if what I did was correct.

ok,I sloved it! Thanks anyway.

I also have the same problem, may I ask why?


First-class conditional patterns do not evaluate the conditional expression

First-class conditional patterns do not evaluate the conditional expression.
Consider the following code:
load "io".
load "util".

let POS_INT = pattern with (x:%integer) %if x > 0.
let NEG_INT = pattern with (x:%integer) %if x < 0.

function fact
    with 0 do
        return 1
    orwith n:*NEG_INT do
        throw Error("factorial is not defined for "+n).
    orwith n:*POS_INT do
        return n * fact (n-1).
    end

println ("The factorial of 3 is: " + fact (3)).
assert (fact(3) == 6).

This code will always throw an error for any integer value, besides 0. This is because the first-class patterns which have been declared, NEG_INT and POS_INT, will only have their patterns evaluated, and not their accompanying conditional expressions, when attempting to unify the functions clauses with the actual arguments.
Additionally, after examining the symbol table, it appears that these first-class patterns were stored into the symbol table without any conditional expressions associated with them.
NEG_INT : ('quote', ('named-pattern', ('id', 'x'), ('typematch', 'integer')))
POS_INT : ('quote', ('named-pattern', ('id', 'x'), ('typematch', 'integer')))

The actual issue was identified in the grammar. The last grammar rework had left the conditional expression with a higher level of precedence then the quote expression. This meant that when encountering a first-class-conditional pattern, the interpreter would first try to evaluate the conditional before attempting to store the associated pattern(sans conditional).
The solution here was to undo this change. An analysis of the interpreters performance along with a its testing suites revealed no negative complications from this switch.
A quote expression now has a higher level of precedence then a conditional expression. This means that in an Asteroid AST representation, a conditional expression node is always below a quote expression node. This allows a conditional expression to be interpreted as a quote, or first-class, expression.
All first-class-conditional pattern expressions now have their conditional expression stored with them in the global symbol table.
The above factorial example now executes correctly regardless of the order of the 3 clauses.
Snip from former EBNF:
pattern
: exp
exp
: conditional
conditional
: quote_exp
(
(CMATCH exp) |   // CMATCH == '%'IF
(IF exp ELSE exp)
)?
quote_exp
: QUOTE head_tail
| PATTERN WITH? head_tail
| head_tail
head_tail
: compound ('|' exp)?
Snip from current EBNF:
pattern
: exp
exp
: quote_exp
quote_exp
: QUOTE head_tail
| PATTERN WITH? head_tail
| head_tail
head_tail
: conditional ('|' exp)?
conditional
: compound
(
(CMATCH exp) |   // CMATCH == '%'IF
(IF exp ELSE exp)
)?


Why metrics elasticsearch_cluster_health_up and elasticsearch_node_stats_up don't have the cluster_name and node_name info?

Hi @Gareer,
they can't. The metric indicates if the API call succeeded. With no response from the API it's hard to tell the cluster_name etc. Exporting the labels only in the success case creates metrics with different label sets which is a no-go in prometheus (and even panics iirc)

OK, got it, thanks!


[W7.7][W11-B3] K.N.S. Address Book

Use of code snippets is good but should be minimal since code gets outdated very quickly.
Try using UML diagrams to supplement your implementation details.
Cons: N/A If there are no disadvantages then there shouldn't be any consideration among choices.


Anvil crashes with: Failed to get storage

Component
Anvil
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (fc534d9 2022-05-23T00:04:01.500526763Z)
What command(s) is the bug in?
anvil --fork-url
Operating System
Linux
Describe the bug
Hi foundry team, I'm from the OpenSea team. We're currently in the process of trying to migrate from hardhat in our e2e tests to anvil and are running into some issues. It appears that anvil crashes during our testing with a pretty obscure error.
Full trace:
project-polygon-mainnet-1  | thread 'fork-backend-thread' panicked at 'Failed to get storage for 0x8f3c…a063 at<PHONE_NUMBER>5618327566453423202919073693767156451540437757185495221258080913552
project-polygon-mainnet-1  | error sending request for url (https://polygon-mainnet.g.alchemy.com/v2/XXXXXXXXXXX): connection error: connection reset', /home/runner/work/foundry/foundry/evm/src/executor/fork/backend.rs:302:33
project-polygon-mainnet-1  | note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
project-polygon-mainnet-1  | ./scripts/run_polygon.sh: line 5:    10 Aborted                 (core dumped) anvil --host <IP_ADDRESS> --fork-url="https://polygon-mainnet.g.alchemy.com/v2/XXXXXXXXXXXXXXXXX" --fork-block-number=28798565 --port=8545

Any idea what's going on here?
Would also be great if we could have a direct contact for assistance in this migration if that would be possible. Thanks!

error sending request for url (https://polygon-mainnet.g.alchemy.com/v2/XXXXXXXXXXX): connection error: connection reset'

looks like you hit a connection reset error.
we currently panic/crash in fork mode if we can't connect because that's the safe way.
since alchemy updated how they count their units, we've been Hit with a lot of 429 rate limit errors, we're about to integrate a rate aware provider that automatically does retries.

Are you also aware of any read timeouts? Have been getting a decent amount of errors regarding that. Don't think it's completely related to the above as was seeing read timeout logs at times where there weren't any crashes

wdym by "read timeouts"?

Our client that hits our RPC node (anvil), for example getting the latest block, just times out, due to anvil not sending us a response back. This only happens later on in our test cycle. Sorry I don't have more concrete numbers.

there was an issue that could lead to situation like that, which was fixed in https://github.com/foundry-rs/foundry/pull/1689 68acdf52de5a5a951419ad788ad4e9ba508c6011
the version you referenced predates that fix, should be included in nightly though

Seem to still be hitting that issue, but would love to try again when the rate aware provider you mentioned before gets implemented. Any idea on when that would be?

@sunwrobert the rate limiting should be present in the latest version, can you give it another try?

The rate limiting seems to have fixed it. I do still have other issues but will open a separate issue for those. Thanks!


divide xtime from frame

xdag_frame_t is basically the number of main block thus it isn't the same as xtime, it is called frame in the xdag whitepaper.

Thank you for your efforts!


Put Jupyter/Binder context back in 1.2, substantially trim details

This largely overlaps with #65, which I like better.
I may pull in some of this, but let's merge that one first.


Migrate to sbt-typelevel

This one's been sitting for ages, anyone wanna merge? 🥺

It looks like @softinio has been busy lately.  We could put out a call for an additional maintainer.


The Orphanage does not affect all deeds with Control >= 2

Card
The Orphanage
Describe the bug
The Orphanage does not properly reduce control or increase production on all deeds with control >= 2
To Reproduce
Steps to reproduce the behavior:

Purchase The Orphanage
Have some deeds with 2 or more control points on the board
Use The Orphanage's ability
See error

Screenshots

Additional context
I haven't tested all the deeds, but the Undertaker and The Mayor's Office both kept their full control after The Orphanage was used. 1st Baptist Church had it's control reduced to 1 but there is no indication on any deed that their production was increased at all. When I finished the day and went through the upkeep phase, neither player received any extra production.

This was a tricky one. Problem with Orphanage is that it affects deeds with 2 or more CP, and effect is that it gives penalty -1 CP. That resulted in a loop effect applications for deeds with 2 CP. First, effect was applied and CP was changed to 1, but then the effect condition was no longer valid so it was unapplied and CP was changed back to 2. But then, the condition was again valid ... I guess you see the pattern. In the end, the loop was broken (not sure why), and it could end with the effect applied or not for control (that is why some deeds were affected and some not), but it never reapplied for production.
Solution: When checking for control, do not take the Orphanage effect into account.


Atom not displaying in ubuntu Xfce desktop environment

Description
When launching atom under Ubuntu Xfce desktop environment, atom window never shows up, and no errors are displayed.  Tried version 1.7.4 .deb package first, then tried cloning 1.8.0-beta4 from the repo and building. Both versions had the exact same problem.  After running the atom command it displays the app load time and then just sits there and the window never comes up.
I am using ubuntu server with the Xfce desktop manager over an x2go connection.
I have also tested it over tightvnc and with KDE4 (with both x2go and tightvnc) with the exact same results.
Steps to Reproduce

Install latest .deb package or build from source
Run atom command
Atom editor window never displays

Expected behavior: Atom window shows up.
Actual behavior: Atom window never shows up.
Versions
Atom: 1.7.4 and 1.8.0-beta4
Electron: 0.36.8
Chrome: <IP_ADDRESS>.110
Node: 5.1.1
Ubuntu: server 14.04.4 LTS
Xfce: 4.10

I've got the same problem with Atom 1.8.0 on Ubuntu 16.04 via x2go. Same behavior as in the original post.
Versions
Atom      : 1.8.0
Electron : 0.36.8
Chrome : 47.0.2526.110
Node     : 5.1.1
Ubuntu  : server 16.04 LTS
Xfce       : 4.12

@cskeen I'm also having the same problem with Visual Studio Code on the same machine, but Atom and Code are the only two programs I'm seeming to have the issue with.
Would you mind trying to run Visual Studio Code and see if you have the same problem?
Just grab the .deb package on their site and install it with dpkg.

VSCode has the same problem.
This might be another example of an upstream Electron issue relating to NX. electron/electron#2256

@cskeen Seems plausible. What's odd to me is that it doesn't throw errors of any sort that I could see, unless there is a log somewhere that I am unaware of.

To give some clarification of the behavior (and the problem...):
When you try starting Atom via x2go, it will start Atom on display :0, regardless of the DISPLAY environment variable or anything else. This is a problem for x2go users because it defaults to a new session on :50.
This behavior can be replicated with the following steps:
1: Find a Linux machine you have local access to.
2: Install x2goserver and x2goclient. Start x2goserver.
3: Use x2goclient to connect to localhost via x2go.
4: Try to launch Atom from the x2go session.
5: The Atom window will open on the local session, not the x2go session.

I was experiencing the same issue in Gnome 3, but noticed that I was able to open atom as root or as a guest user. So I started deleting stuff in my home directory... Hopefully, this may narrow down the problem:
$ atom # window did not open
$ ls -lah
$ rm  .v8flags.<IP_ADDRESS>.misund.json
$ rm .Xauthority
$ rm .xsession-errors*
$ rm -r .gconf/
$ atom # window opened successfully

Closing in favor of #4360 - feel free to subscribe there for updates.

why is this issue been closed? Is the bug fixed?
i have the same problem, but cant find answer.

This comment from #4360, helps workaround this problem:
https://github.com/atom/atom/issues/4360#issuecomment-205122828

@mickeyil according to the steps comment on #4360 ,  still not work.  I tried this method on another computer and worked.  so maybe not the same problem, I have no idea now.

still buggy. atom window loading, but not displaying anything. just black pixels.
Ubuntu 16.04, XFCE 4.12
in 16.04 unity it works perfect.


[CAMEL-13123] Add test stopping endpoint

Thanks, also the source file needs to have the ASF license header. Just look and copy from one of the other java source files.

ping  @tveskrna


SSL issues after upgrade to 1.1.0.beta10

First, I accidentally downgraded from 1.1.0.beta0 to 1.0.0 (I hit the Upgrade button instead of selecting a version manually). After that, I upgraded to 1.1.0.beta10 . EVrything went fine, but after hitting back and reloading, the site would keep on loading forever. I tried connecting to the IP instead of the domain, where I was greeted with the following:

Restarting the droplet and trying a different browser did not resolve things. I vaguely remember facing this problem before, but forgot how it was resolved. Maybe the error is on my end?

no its a weird issue with incomplete typescript compilation in beta.10, shouldnt be an SSL issue. are you able to upgrade to beta.11 (just released) with ssh @iron3oxide ?

 npm install -g @saltcorn/cli@next as saltcorn user

Yep it worked, thanks!


report a bug

The Preset Editor will keep flashing when Dalamud option "Use AXIS as default font" is enabled and it won't be easy to edit anything under such circumstance.

Can you post a screenshot of, or the values for, your Dalamud style editor settings (if you’re not on the default style)?  Also, what is your global font scale setting?  I’ve not been able to reproduce any flashing or resizing issues so far.

Well my Dalamud is (maybe) specially modified for CN client so the settings can hardly help but here are the screenshots

it just keeps switching like this

And I noticed that it won't work when I change Dalamud's language from Chinese to English
This is the setting that cause the problem

Also, the window will automatically expand like this until it finally starts flashing

I use the AXIS font with no issue, so I don’t know why that’s making a difference.  The expanding window to the side issue was fixed quite some time ago, so I’m surprised you’re seeing it.  It looks like in your case it’s probably that the map view button width is being calculated incorrectly for some reason, but if that were bugged, it should happen to everyone, not just you.
Regardless, if you want to post what your style settings are for like cell padding, item padding, window padding, etc., I’ll double check with those (although it looks like you’re using the default Dalamud style?).

Ok, actually there is a layout bug that could potentially affect this in languages other than English, but I still don’t understand how you’re seeing this happen with the plugin in English.

I have never changed the Dalamud style except for the global font scale (set to 16pt). I check that this problem can appear when I
use Dalamud Classic or Dalamud Standard. Now I think maybe this is because I'm using CN client, both the game and Dalamud.

Ok, I’m actually able to reproduce this now.  It appears to be a text wrapping bug at certain non-standard global scales.  I’ll see what I can do to fix this.  Sorry for all the back and forth, and thank you for being patient.

This should be fixed in <IP_ADDRESS>.


tp.py: Add type annotations

This PR adds type annotations to tp.py so that IDEs like Visual Studio Code can provide better information to developers.

@Jcw87 fix the conflicts and we'll pull this in

@Pheenoh rebased


[Feature suggestion] Automatically add new files in albums to a .zip and make it downloadable

I'd like to let users download all files in an album. Now I could upload a zip of all the files that are in the album, however, I'd have to update that whenever I add files. I propose that all albums have a zip file that is hidden from the album view, but downloadable through a button. Of course, this would be optional and could be disabled in the config.
I'm not sure if anyone else is interested in this feature, but I might be able to make pull request for it if there's any interest.

This is a pretty solid idea, I like it a lot. I've been in the process of rewriting lolisafe since yesterday and will make a new dev branch in the following hours if you want to take a look at it to try and figure a nice way of doing it!

Sure, sounds good

Just a heads up, I'll probably start working on this tomorrow, so if there's no dev branch pushed by then I'll go from master.

Closing this since it's been implemented


Register-PSRepository in non-interactive mode?

Team,
I'm using Register-PSRepository for automation on internal build system. On PS 5.1. the cmdlet prompts and I get the following message(from the build logs):

Exception calling "ShouldContinue" with "2" argument(s): "Windows PowerShell is in NonInteractive mode. Read and Prompt functionality is not available."

The cmdlet doesn't have -Force/-Confirm parameters and it doesn't respect the $ConfirmPreference variable. Is there a workaround for this?
Thanks,
Nedko

@bmanikm That command doesn't seem to download nuget.exe. In my environment, I have been running that command, but I don't get the error about non-interactive mode until I get to the publish-module step, which fails because it's trying to prompt about downloading nuget.exe.

@godefroi Correct, Install-PackageProvider only installs NuGet provider, and it doesn't download nuget.exe. NuGet.exe is required only during Publish-Module or Publish-Script operations.
Please take a look at
https://github.com/PowerShell/PowerShellGet/blob/development/tools/build.psm1#L72 to understand the logic for bootstrapping NuGet.exe in automation scripts.

This is extremely unsatisfying.  @bmanikm's suggestion results in downloading NuGet in every iteration/build/deployment!
We end up needing something like this in our deployment scripts?
if(!(Get-PackageProvider NuGet)) {
   Install-PackageProvider Nuget -ForceBootstrap -Force
}

It's a bit frustrating that I need to write so many lines for that, and then I have the same problem when installing modules.

In automation scripts, please use the below simple command to bootstrap the NuGet provider. This bootstrap the NuGet provider only if it is not available on the local machine.
Get-PackageProvider NuGet -ForceBootstrap

In non-interactive mode this step requires additional privileges, at least account under which is running this commandlet has to have SeAssignPrimaryTokenPrivileges and SeServiceLogonRight. I've just extended this script by adding SeAssignPrimaryTokenPrivilege options. Currently Register-PSRepository works fine for me from our Chef recipes.

This should be reopened @bmanikm - I have similar use case with Docker (installing pswh on docker image behind proxies) and it would be very welcomed

Here is my Recipe how to upgrade PowerShellGet under the docker
    Write-Host
    Write-Host "Updating PowerShellGet ..." -ForegroundColor Green
    # Original way to install PowerShellGet does not work under docker or without user confirms
    #Install-Module PowerShellGet -Repository PSGallery -Force -Confirm -AllowClobber -Scope CurrentUser

    # Here is the alternative install script
    $nugetPackageProvider = $null
    $nugetPackageProvider = Get-PackageProvider NuGet -ErrorAction SilentlyContinue

    if (!$nugetPackageProvider) {
        $nugetPackageProvider = Install-PackageProvider Nuget -ForceBootstrap -Force
        Write-Host "Installed NuGet package provider version $($nugetPackageProvider.Version)"
    }

    Write-Host "Installing PowerShellGet module..."
    $userModulesPath = "$([Environment]::GetFolderPath("MyDocuments"))/WindowsPowerShell/Modules"
    New-Item -ItemType Directory -Path $userModulesPath -Force | Out-Null
    Save-Module PowerShellGet -Path $userModulesPath
    Remove-Module PowerShellGet, PackageManagement -Force

    Import-Module PowerShellGet
    $psGetModule = Get-Module PowerShellGet
    $installedVersion = $psGetModule.Version

I was having a similar issue when trying to run a PowerShell script in my release pipeline that needed access to a module in our private Azure DevOps Artifacts feed, so I was attempting to do:
Register-PSRepository -Name MyRepoName -SourceLocation 'https://pkgs.dev.azure.com/[MyOrganization]/_packaging/[MyFeed]/nuget/v2' -InstallationPolicy Trusted

Install-Package MyPrivatePowerShellModule -InstallUpdate -AllowClobber

However, when the script runs on the MS hosted agent, it ends up in an infinite wait loop outputting:

"C:\windows\System32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command ". 'D:\a_temp\527651b0-116e-46aa-a599-141b4dfc9899.ps1'"
Registering PSRepository
[Minimal] [CredentialProvider]DeviceFlow: https://pkgs.dev.azure.com/[MyOrganization]/_packaging/[MyFeed]/nuget/v2
[CredentialProvider]To sign in, use a web browser to open the page https://microsoft.com/devicelogin and enter the code EB64QSSD6 to authenticate.##[section]Starting: PowerShell Script

I came across this feedback post though and was able to get it working with the following code (ensuring $PersonalAccessToken contains a PAT with permissions to read from the provider):
$pat = ConvertTo-SecureString '$PersonalAccessToken' -AsPlainText -Force
$credential = New-Object System.Management.Automation.PSCredential<EMAIL_ADDRESS>$pat

Install-PackageProvider NuGet -Scope CurrentUser -Force > $null

if ($null -eq (Get-PSRepository -Name 'MyRepoName' -ErrorAction SilentlyContinue))
{ Register-PSRepository -Name 'MyRepoName' -SourceLocation 'https://pkgs.dev.azure.com/[MyOrganization]/_packaging/[MyFeed]/nuget/v2' -InstallationPolicy Trusted -Credential $credential }

Install-Module -Name 'MyPrivatePowerShellModule' -Repository 'MyRepoName' -Force -AllowClobber -Credential $credential
Import-Module -Name 'MyPrivatePowerShellModule' -Force

The trick was to first do Install-PackageProvider NuGet -Scope CurrentUser -Force, and then also to use Personal Access Token credentials. I hope this helps anybody else who stumbles across this issue.


embedded-keycloak uses wrong database

keycloak tries to find itself a database to use by invoking e.g. getent hosts mysql
See keycloak-containers/server/tools/docker-entrypoint.sh#L120
As it is on the default bridge-network, in my case it finds something and uses it.
Currently I haven't found a way to easily inject the possible environment variables described on keycloak docker for either telling it, which DB_VENDOR to use, or which DB_ADDR, DB_PORT, …
Locally I cloned this project, made a quickfix by adding this line:
        withEnv("DB_VENDOR", "h2");
at KeycloakContainer.java#L35 to force the embedded h2 database, and use my local embedded-keycloak then.
But it would be nice to have those variables accessible via bootstrap.properties.
From keycloak docker:

You can specify the DB vendor directly with the DB_VENDOR environment variable.
Supported values are:
h2 for the embedded H2 database,
postgres for the Postgres database,
mysql for the MySql database.
mariadb for the MariaDB database.
oracle for the Oracle database.
mssql for the Microsoft SQL Server database.

If DB_VENDOR value is not specified the image will try to detect the DB vendor based…

DB_ADDR: Specify hostname of the database (optional).
DB_PORT: Specify port of the database (optional, default is DB vendor default port)
DB_DATABASE: Specify name of the database to use (optional, default is keycloak).
DB_SCHEMA: Specify name of the schema to use for DB that support schemas (optional, default is public on Postgres).
DB_USER: Specify user to use to authenticate to the database (optional, default is ``).
DB_USER_FILE: Specify user to authenticate to the database via file input (alternative to DB_USER).
DB_PASSWORD: Specify user's password to use to authenticate to the database (optional, default is ``).
DB_PASSWORD_FILE: Specify user's password to use to authenticate to the database via file input (alternative to DB_PASSWORD).

guess it should be very easy PR, since you already have cloned repo. Just bypass value DB_VENDOR from https://github.com/testcontainers/testcontainers-spring-boot/blob/develop/embedded-keycloak/src/main/java/com/playtika/test/keycloak/KeycloakProperties.java
summoning original author @holgerstolzenberg

Looks good to me.


Natalie breaks when using IBAnimatable

I'm not even sure if a fix is possible since it's caused by a third party dependency. But I get build errors when Natalie generates code for a view controller that is subclassed from AnimatableViewController of IBAnimatable

what kind of errors? can you make an example project?

Here is a blank project with IBAnimatable and Natalie. See if you get the error:
IBAnimatableNatalieError.zip

/Users/cupojoe/Code_Repos/NatalieError/IBAnimatableNatalieError/IBAnimatableNatalieError/Storyboards.swift:239:9: Property 'storyboardIdentifier' must be declared public because it matches a requirement in public protocol 'IdentifiableProtocol'

one more question. do you use master branch, or last release? I should do the release with the latest fixes

We are using the release of IBAnimatable (Cocoa Pods) and the master of Natalie, built from source.

the code is
output += "extension IdentifiableProtocol where Self: \(customClass) {\n"
if viewController.customModule != nil {
       output += "    var storyboardIdentifier: String? { return \"\(storyboardIdentifier)\" }\n"
} else {
       output += "    public var storyboardIdentifier: String? { return \"\(storyboardIdentifier)\" }\n"
}

customModule is not nil (because of IBAnimatable module is used)
I do not remember why there is this two case

duplicate of new one with more information #124


avahi_service_browser_new (when starting kernel)

Hello,
I am running Spyder with Python 3.6 on ubuntu. I want to connect to a tobii eyetracker via the tobii_research SDK. When a run:
import tobii_research
tobii_research.find_all_eyetrackers
I get:
An error occurred while starting the kernel
python3.6: browser.c:581: avahi_service_browser_new: Assertion 'client' failed.
(I expect an output with a list of the connected tobii eye tracker)
I have reset tried to: set Spyder to factory defaults, started a new console, restart the computer, check all versions and libraries - can someone help?

I have also tried this and get the same error:
user@00000000:~/Desktop > python3
Python 3.6.9 (default, Oct  8 2020, 12:12:24)
[GCC 8.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
import tobii_research
python3: browser.c:581: avahi_service_browser_new: Assertion `client' failed.
Aborted (core dumped)

Hi @florianteichmann,
How did you installed the package tobii_research ?

Hi @steff456,
with pip install (I also checked with pip list if its the most recent version)

I'm unable to reproduce this issue, can you please post here a screenshot with the error?

@florianteichmann, given that when you run import tobii_research in a Python interpreter outside Spyder, it makes the interpreter crash, this is not a problem related to Spyder but to that package.
Therefore, there's nothing we can do to help you here, sorry. Instead, please try to report it to the maintainers of that package to see if they can help you with it.


NWCHEM: Adrian dipole/homo/lumo changes

Description
fixes logic with parsing homo and lumo energies from nwchem output
Changelog description
Fixes the logic when parsing for homo and lumo energies
Status

[ ] Code base linted
[ ] Ready to go

Can we add tests for these?

I also added a test for homo and lumo energies

What's the hold up on merging this? Did we just forget?

Looks good, slipped through the cracks.


REVOKE: FORCE

#582
FORCE is only used in connection with system_privilege EXECUTE, which is not a valid system_privilege.

Please provide a parser crash log and/or example SQL

I assume that this is adopted from Oracle SQL.
According to Kline, SQL in a Nutshell (3rd edition) on page 341:
FORCE
Required to revoke EXECUTE object privileges on user-defined type objects with
type or table dependencies.

So the change would be:
opt_with_revoke_option -> '$empty'                                                              : ''.
opt_with_revoke_option -> CASCADE CONSTRAINS                                                    : 'cascade constrains'.

opt_with_revoke_option -> FORCE                                                                 : 'force'.


fix: display input next to the button on mobile

Summary
Inputs on the Home page were breaking into two lines on mobile.

Please include SS in PR description when changes are visual, since we don't have much visibility about the app interface yet, it would be useful to understand how the change will impact the application.

Please include SS in PR description when changes are visual, since we don't have much visibility about the app interface yet, it would be useful to understand how the change will impact the final product.

My bad, these are the only inputs we have in the app and I'm just heavily relying on the deployment preview feature jajaja, sorry!


Add flipud and fliplr numpy functions.

Looks good! Thanks for the PR!
I think it's fine to not support None axes yet. We can improve the implementation if someone needs it.
(It looks like flip also supports tuples of axes, but let's save that for a future PR.)


docs: Document the Instruction enum

In addition to adding doc comments I also removed the unused GetIndex instruction and made the two argument variants into struct variants to avoid ambiguity.

@bjz Fixed your other comments. Squashing the commits before merging if everything looks good.

👍


Availability on conda forge?

Dear Maintainers,
Could you please also release this package to conda forge? Thank you very much in advance

Ciao @ilyanoskov ! Sorry for the late reply. I am not an expert at all about conda-forge (i.e. how to make a recipe etc) and especially how to maintain it. Do you have any recommendation? I know greyskull but just in name.


Add Installation Instructions

This PR adds instructions for installation of ROS2, epuck_ros2 and usage in general.

:eyes:


When rollout new controller version, it should not trigger a rolling update

What happened:
In the rolling update triggering logic, we should store the template somewhere else, and only compare the fields with the original one, and check if these fields are updated, then decides to trigger a rolling update. This will mitigate the issue when we added a API field, it will not trigger rolling update when rolling out new controller version
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Kubernetes version (use kubectl version):
LWS version (use git describe --tags --dirty --always):
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

/assign

For the short term, lets patch the current release with a special fix for the new field we added, the NetworkConfig.
Basically if the NetworkConfig is set to the default values, set it to nil in the controller when calculating the hash, this should avoid changing the hash value when the new controller is rolled out.

yep, this could be a short term fix

For the short term fix, don't we already exclude NetworkConfig if it is nil https://github.com/kubernetes-sigs/lws/blob/main/pkg/utils/utils.go#L44? @ahg-g

The issue is that the NetworkConfig will not be nil since it will be defaulted, which means the hash for an existing deployment will be different from the one stored as an annotation and generated previously by the older operator version.

I thought it isn't defaulted when upgrading the LWS controller https://github.com/kubernetes-sigs/lws/pull/197#discussion_r1720806787, and that's why we added the nil check?

It is getting defaulted: https://github.com/kubernetes-sigs/lws/blob/0b8c0948dfb83f1cac26b488e9f85060d9553e10/pkg/webhooks/leaderworkerset_webhook.go#L75
But can you test that on a real cluster?

I tested upgrading from 0.2.0 to 0.4.1 and the upgrade did not trigger an upgrade of the statefulSets (so the template hash was not updated)

Check if the NetworkConfig get defaulted, if not, try to tigger an update on LWS that doesn't trigger a rolling update to trigger calling the defaulting function.

Upgraded from 0.2.0 to 0.4.1. NetworkConfig does not get defaulted. Updated LWS object by removing a metadata label to trigger defaulting option, NetworkConfig gets defaulted and rollingUpdate is triggered.

/assign @Edwinhr716

This will be addressed via https://github.com/kubernetes-sigs/lws/issues/239


Not sure how to get a basic test working

I'm fairly new to Ruby and Rails so no doubt I've done something wrong, however I can't work out how to use the login helper.
I've set things up per as the readme, however I did also have to add the following require to test_helper.rb to avoid uninitialized constant ReviseAuth::Test (NameError):
require 'C:\Ruby33-x64\lib\ruby\gems\3.3.0\gems\revise_auth-0.6.0\lib\revise_auth\test\helpers.rb'

Here's a test to just show the index for a controller (which requires authentication):
class GamesControllerTest < ActionDispatch::IntegrationTest
  setup do
    @user = users(:bob)
    login(@user, password: "password")
    @game = games(:one)
  end

  test "should get index" do
    get games_url
    assert_response :success
  end

Running that test by itself it fails and I don't know why:
❯ bin/rails test test/controllers/games_controller_test.rb:10
Running 7 tests in a single process (parallelization threshold is 50)
Run options: --seed 34325

# Running:

F

Failure:
GamesControllerTest#test_should_get_index [test/controllers/games_controller_test.rb:13]:
Expected response to be a <2XX: success>, but was a <403: Forbidden>

bin/rails test test/controllers/games_controller_test.rb:10

Finished in 1.262554s, 0.7920 runs/s, 0.7920 assertions/s.
1 runs, 1 assertions, 1 failures, 0 errors, 0 skips

The user bob exists in the fixtures (users.yml):
bob:
  email<EMAIL_ADDRESS>  password_digest: <%= BCrypt::Password.create("password") %>

Any clues would be appreciated!

Instead of the custom require you can open ActionDispatch::IntegrationTest class and include the test helpers.
# test_helper.rb
class ActionDispatch::IntegrationTest
  include ReviseAuth::Test::Helpers
end

Not sure why you're getting a 403 error.

Thanks @gathuku but I already had the test helpers included, here's my test_helper.rb:
ENV["RAILS_ENV"] ||= "test"
require_relative "../config/environment"
require "rails/test_help"
require 'C:\Ruby33-x64\lib\ruby\gems\3.3.0\gems\revise_auth-0.6.0\lib\revise_auth\test\helpers.rb'

# needed for password_digest in users.yml fixtures
require "bcrypt"

module ActiveSupport
  class TestCase
    # Run tests in parallel with specified workers
    parallelize(workers: :number_of_processors, with: :threads)

    # Setup all fixtures in test/fixtures/*.yml for all tests in alphabetical order.
    fixtures :all

    # Add more helper methods to be used by all tests here...
  end
end

# see https://github.com/excid3/revise_auth?tab=readme-ov-file#test-helpers
class ActionDispatch::IntegrationTest
  include ReviseAuth::Test::Helpers
end

This was the default test_helper.rb with the only parts added being the revise test helpers class and require, and the require for bycrypt (for the password_digest fields of the user fixtures).
I'm wondering if it's anything to do with being on Windows?
Am I correct in understanding that tests by default don't autoload modules like the app does? In which case this is what makes the explicit require for the revise test helpers necessary?

@Youdaman I think rails autoload all gems in your Gemfile. It's supposed to work without the explicit require.
Are you raising forbidden error anywhere in your GamesController? or how does the code looks like.

My guess is that I haven't started with tests until now and I've been working on this project for a while now, so there's likely some change I've made to the config or another file that is breaking things.
I just created a fresh project, added revise_auth, added the helpers to the test and ran bin/rails test with no problems. I'll start my project again but this time run tests from the very beginning so that whenever I make a change I can run the tests to see what breaks things.
Apologies for the waste of time, closing this!

@gathuku Update -- it turns out that I was using revise 0.5... and when I updated to 0.7 things worked... so that helper class wasn't found because it mustn't have been in 0.5! Tests run now! 😄

I stripped down the users fixture in case the confirmed/email fields were breaking things but still no luck:
one:
  email<EMAIL_ADDRESS>  password_digest: <%= BCrypt::Password.create("password") %>

two:
  email<EMAIL_ADDRESS>  password_digest: <%= BCrypt::Password.create("password") %>

@yorickvandervis not entirely sure by instead of using password_digest try password in fixtures and supply a plain text.

@yorickvandervis added this https://github.com/gathuku/revise_auth_test/commit/ff04b4dcda5cddca663afd4cca6c5e525cd7c01d and seems to work fine.
I think you have other part of your application logic you need to check

Thanks @gathuku I think you're right about it being another part of my application logic interfering because the test fails with:
❯ bin/rails test test/controllers/test_controller_test.rb
Running 2 tests in a single process (parallelization threshold is 50)
Run options: --seed 25006

# Running:

F

Failure:
TestControllerTest#test_index:_success [test/controllers/test_controller_test.rb:9]:
Expected response to be a <200: ok>, but was a <403: Forbidden>.
Expected: 200
  Actual: 403

bin/rails test test/controllers/test_controller_test.rb:4

F

Failure:
TestControllerTest#test_index:_failure [test/controllers/test_controller_test.rb:15]:
Expected response to be a <3XX: redirect>, but was a <403: Forbidden>

bin/rails test test/controllers/test_controller_test.rb:12

Finished in 1.177078s, 1.6991 runs/s, 1.6991 assertions/s.
2 runs, 2 assertions, 2 failures, 0 errors, 0 skips

Leave it with me and I'll work it out, even if I have to start a fresh project. Thanks again for your assistance!

@gathuku hey Moses, I found the issue, you were right it was caused by something else. Turns out I'd set the config.hosts and not included "www.example.com" so the tests wouldn't work when trying to hit that domain. I found out by running the debugger in the tests and checking what the request variable was for your successful example versus my failing tests, and noticed yours had various things like auth cookies whereas mine just had HTML and the page said "Blocked hosts: www.example.com" which gave me the clue :) Thanks again for trying to help me!


Fix WrapWithOverflow handling for TextEndOfParagraph

What does the pull request do?

What is the current behavior?

What is the updated/expected behavior with this PR?

How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues

You can test this PR using the following package version. 11.0.999-cibuild0029306-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0029336-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0029371-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]


Brand Recognition - Data Collection.

From Known Issues

While Parsigs aims to identify and structure dosage instructions effectively, it's worth noting that the identification of brand names as part of the definition may not be as complete as the extraction of frequency, dosage, and period information (as this data is often not explicitly provided in the Sig, e.g., "Take 1 tablet every day").

I would like to have in the data folder a list of all/most/as big as you can make and find it to be of name-brand-medication.
preferably a table or a list of drug name: drug company: generic name:

@itay-goldraich in case we will have all of the drug names, It wont be easy to infer which drug is in the Sig
how did you thought using this here?

It will have 2 main uses.

The autocorrect feature isn't familiar with brand name medications and might make a mistake in confusioning a name with a word.
Farmacist usually get a brand name drug from the doctor, but they can (and often do) change the drug to a generic or other drug with the same affect.
This data can help with a future feature for such common action.


General: Fix active students charts with limited scope

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[ ] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[ ] I chose a title conforming to the naming conventions for pull requests.

Server

[x] I followed the coding and design guidelines.
[x] I implemented the changes with a good performance and prevented too many database calls.

Motivation and Context

This PR is concerning Issue #5101: Currently, the scope limitation of the active students charts does not work as intended. If a course has a start date and the difference between the course start date and the current end date of the chart (either the official end date of the course or the current date) was not exactly a multiple of a week, the chart did not show the correct information.
Furthermore, the server test regarding the scope limitation server code is currently flaky.
Description

I adapted the server code. Now, the difference between start and end dates is computed based on normalized timestamps.
Additionally, I modified the server test so that it passes again.
Steps for Testing

Prerequisites:

1 Instructor
1 Course with student activity

Log in to Artemis
Edit the course
Set a course start date that is  e.g. 20 days ago.
Safe the course and check whether the chart on course detail shows the correct information
Check that the chart in the course management tile shows correct information as well
Edit the course again. Choose the start date so that there is a difference of > 4 weeks between it and the current date. Still, it should not be the current time exactly X weeks ago
Check the charts again that they still work
Edit the course again. Set a course end date that lies in the past. Play around with the start date so that the time difference is not exactly a multiple of a week. Make sure that both charts still work and show correct data.
Also try a combination that leads to a course lifetime > 1 year and verify that the charts handle this correctly.

Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[x] Test 1
[x] Test 2

Test Coverage

Class/File
Branch
Line

Screenshots

The scope limitation should work again:

In step 8 I was able to produce an internal server error since there is no start and end date validation of the course:  Therefore the active student chart wasn't able to handle those invalid dates :D (but this is not in the scope of your PR)
And there is one codacy issue regarding a println - i guess you can remove it.

I agree that this is not in scope of the PR. Please write an issue for the date validation (and maybe mark it as good first issue). And thanks for the heads-up, obviously I missed removing one of my debug statements. 😅

Tested on ts2, works fine. One comment: When I try to view the chart on a small window, it is cut off on the left hand side.

Thanks for reporting. I'm currently working on enhancing the mobile appearance of various charts, including this one. :)

Tested in ts3, it works very well! The chart shows data from one day before the start of the course until two days after today. I only wanted to comment in case this was not intended. Otherwise, very good!

We fetch the data weekly. This means it could be the case that we cover days before the official start date. For example, if the start date is a Wednesday, also data from Monday and Tuesday in this week are included in the weekly presentation. The same holds for the end date: if it is e.g. a Friday, also data from Saturday and Sunday is included. As the scope of the x axis labels is weekly (either calendar weeks in course detail or 'x weeks ago' in course management), this is an acceptable calculation imo. I hope this explanation makes the situation a bit clearer. If there are any further questions, do not hesitate to ask them. :)

Perfect, thank you! Then everything else looks good!


Dart-Sass isn't receiving options set in plugin

It appears that dart-sass is not receiving options set in the plugin for postcss-sass. In this example, I am passing the 'compressed' style option, but the output remains expanded.
// postcss.config.js
...

  plugins: [
    require('autoprefixer')(),
    require('@csstools/postcss-sass')({
      style: 'compressed' // not working
    })

...

Setup a simple test case
https://github.com/frankyonnetti/postcss-sass-options
Thanks!

@frankyonnetti I was running into a similar issue where passing options to dart-sass via this postcss-sass plugin was not working. I looked at the source code and it looks like this plugin is in fact passing the options to dart-sass but the options but the options are not passed in the correct format that dart-sass expects. This plugin is calling sass https://github.com/csstools/postcss-sass/blob/main/dist/index.cjs#L176 render function https://sass-lang.com/documentation/js-api/functions/render/ and the render function expects these options https://sass-lang.com/documentation/js-api/interfaces/legacyfileoptions/ . These are the supported options that the render function uses.
So for your use case to use the compressed style, you would pass the option outputStyle instead of style
// postcss.config.js
plugins: [
    require('autoprefixer')(),
    require('@csstools/postcss-sass')({
      outputStyle: 'compressed' // this works
    })

For my case, I wanted to use dart-sass's --quiet option to silence warnings about deprecated https://sass-lang.com/documentation/cli/dart-sass/#quiet but the option was not working. I had to create a Logger object to do this https://sass-lang.com/documentation/js-api/interfaces/legacyfileoptions/#logger, which was more complicated than it's worth.

Hey @rxnlabs thank you! I didn't even think to use legacy options.


Author PingAccess Profiles via AS3 schema

Add AS3 schema to author PingAccess Profiles
Describe the solution you'd like
Add AS3 schema to create a PingAccess Profile.
It has a small number of inputs:

Properties file is a relatively small text file which contents look something like this:
agent.engine.configuration.scheme=http
agent.engine.configuration.host=<IP_ADDRESS>
agent.engine.configuration.port=9009
agent.engine.configuration.username=F5TestAgent
agent.ssl.protocols=TLSv1.1, TLSv1.2
agent.ssl.ciphers=TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256,TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256,TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256,TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA,TLS_ECDHE_ECDSA_WITH_AES_128_CBC_SHA256,TLS_ECDHE_ECDSA_WITH_AES_128_CBC_SHA,TLS_DHE_RSA_WITH_AES_128_GCM_SHA256,TLS_DHE_RSA_WITH_AES_128_CBC_SHA,TLS_RSA_WITH_AES_128_GCM_SHA256,TLS_RSA_WITH_AES_128_CBC_SHA256,TLS_RSA_WITH_AES_128_CBC_SHA,TLS_ECDH_RSA_WITH_AES_128_GCM_SHA256,TLS_ECDH_RSA_WITH_AES_128_CBC_SHA256,TLS_ECDH_RSA_WITH_AES_128_CBC_SHA,TLS_ECDH_ECDSA_WITH_AES_128_GCM_SHA256,TLS_ECDH_ECDSA_WITH_AES_128_CBC_SHA256,TLS_ECDH_ECDSA_WITH_AES_128_CBC_SHA,TLS_EMPTY_RENEGOTIATION_INFO_SCSV
agent.engine.configuration.shared.secret=secret-here
agent.engine.configuration.bootstrap.truststore=some-base64-content-here
agent.engine.configuration.maxConnections=10
agent.engine.configuration.timeout=30000
agent.engine.configuration.connectTimeout=30000
agent.cache.missInitialTimeout=5
agent.cache.broker.publisherPort=3031
agent.cache.broker.subscriberPort=3032
agent.cache.maxTokens=0
agent.engine.configuration.failover.hosts=
agent.engine.configuration.failover.failedRetryTimeout=60000
agent.engine.configuration.failover.maxRetries=2

The pool and SSL profile elements already have AS3 schema to create them.

I opened #860 to add the ability for a service_http(s) object to reference a pingaccess profile already created in /Common using a Big-IP pointer.
This request is to be able to author the pingaccess profile in AS3 schema AND reference it from service_http(s) class with the normal use pointer.

Created AUTOTOOL-4464. Added to sprint 43-1 scheduled for end of September

@mdditt2000 Thanks!  Possible to get a resolution sooner?  We require this functionality for a critical onboarding.  As we are committed to AS3 several years now, lack of AS3 support for pingaccess profiles means we cannot integrate with our platform.

Can i switch out https://github.com/F5Networks/f5-appsvcs-extension/issues/732 and pickup https://github.com/F5Networks/f5-appsvcs-extension/issues/862. Please advise ASAP as the work on 862 is planned for this week.

If we could switch out #732 with #860 would be awesome! Getting #860 would un-block us.

@megamattzilla done! Will provide a update tomorrow after this evening standup. I will push to possibly get a image this week or early next week for customer.

@megamattzilla @ghalevy PR will be merged tomorrow. Build will be be available on Monday for Matt

This has been resolved in AS3 3.53.0 version , please reach out to myself or Mark for a fix, thank you.

Is the AS3 3.53.0 version available publicly?
Can we keep this issue open until the AS3 release is available?

@ghalevy @megamattzilla  please can you provide some guidance
What methods works best?
Base64 encoded:
"Sample_Ping_Access_Properties": {
    "class": "Access_Profile_Ping_Access_Properties",
    "properties": {         "base64": "i4dmi4"     }
}

File Path reference:
"Sample_Ping_Access_Properties": {
    "class": "Access_Profile_Ping_Access_Properties",
    "properties": {         "file": "/var/config/rest/downloads/properties.json"     }
}

Uri:
"Sample_Ping_Access_Properties": {
    "class": "Access_Profile_Ping_Access_Properties",
    "properties": {         "url": https://example.com/v1/properties     }
}

@ghalevy based on our AS3 work I have provided Mark the below preferences:
Primary:  in-line (base64)
Secondary: URL reference
Tertiary: file path reference

Hi @megamattzilla ,
We will implement and support base64 encoded properties data for now.

Closed Resolved in AS3-53
Release Notes | https://github.com/F5Networks/f5-appsvcs-extension/releases/tag/v3.53.0


Update Program.cs

In the current code, the Length of the sourceBlob is not set - and this causes the code to crash. You must first call FetchAttributes() so I have added this in.
With this and my previous pull request, this code should be working 'out of the box' again...
Thanks for providing it

@aicaffre, the BlobCopy function does call FetchAttributes(). Are you saying calling it twice fixes the crash?

Hi Julia,
Yes that is what I experienced. If you don’t call it the length is -1, which later causes a crash. I looked into this and saw none of the attributes at all are set on the source blob. This is why fetch attributes seemed to sort out the problem.
If the ‘real’ bug is in BlobCopy not calling fetchattributes as expected then by all means fix that instead.
In either case, at least the code works once again 😀
Get Outlook for iOShttps://aka.ms/o0ukef

From: Julia Kornich<EMAIL_ADDRESS>Sent: Monday, November 27, 2017 8:00 pm
Subject: Re: [Azure-Samples/media-services-dotnet-copy-blob-into-asset] Update Program.cs (#4)
To: Azure-Samples/media-services-dotnet-copy-blob-into-asset<EMAIL_ADDRESS>Cc: Aidan Caffrey<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>@aicaffrehttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Faicaffre&data=02|01|aidancaffrey%40hotmail.com|47933aeeb31e4b3d52bf08d535d184ab|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636474096343921701&sdata=PF3GLH0omO6Skk38DlYV6ChNgI8dj7%2BWM4msFyCFIsU%3D&reserved=0, the BlobCopy function does call FetchAttributes(). Are you saying calling it twice fixes the crash?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure-Samples%2Fmedia-services-dotnet-copy-blob-into-asset%2Fpull%2F4%23issuecomment-347308426&data=02|01|aidancaffrey%40hotmail.com|47933aeeb31e4b3d52bf08d535d184ab|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636474096343921701&sdata=iph9t%2BQkOBrZfYPQ8QAh0bcSIdiQpe9weTs36kpcAI0%3D&reserved=0, or mute the threadhttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FASfLaNAgFNZrgb3q1DHqJMPvFmsUScOuks5s6xSlgaJpZM4Qp4Y8&data=02|01|aidancaffrey%40hotmail.com|47933aeeb31e4b3d52bf08d535d184ab|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636474096343921701&sdata=6L32mEuqkEvWyu3QRNs4uG3Q3VGJcyyh5Dcr2RbICyI%3D&reserved=0.

@aicaffre, thank you!


Dapr/ch9 observability

Chapter 9 - Observability

This is the Dapr for .NET Devs book chapter 9 about Observability.
@nishanil and @IEvangelist, please review. Thanks!

A few suggestions, tag me when you update this and I'll review it again.

@IEvangelist Great suggestions! Applied them all.


Build error on Windows due to mismatched API version

Does this issue affect the google-cloud-cpp project?
Yes. I think so.
What component of google-cloud-cpp is this related to?
Several, at least Pubsub
Describe the bug
Building the library (2.3.0) on Windows fails with the following message:
C:\devel\google_cloud\google-cloud-cpp\google\cloud\pubsub\internal\streaming_subscription_batch_source.cc(340,47): error C2039: 'exactly_once_delivery_enabled': is not a member of 'google::pubsub::v1::StreamingPullResponse_SubscriptionProperties'

When I check the current version of this file here I see the required symbol is present. When I check the downloaded version of this file in the build dir it appears to be a previous version where this symbol is indeed missing.
To Reproduce
Build on Windows. Sorry, I can't go into more detail as the build process is not quite easy but I have successfully built many times before and I'm certain the build is as okay as it gets. I have also deleted the cached CMake files and everything to make sure the build is clean.
Expected behavior
Build works
Operating system:
Windows
What compiler and version are you using?
MSVC 14.2
What version of google-cloud-cpp are you using?
2.3.0
Additional context
I've been seeing issues with this for a while now. The last couple of releases I have built complained about various missing protobuf files. I normally took them from the github and put them into the downloaded dirs, which sort of fixed the issue but I already suspected some version mismatch causing this.

Are you perchance rebuilding on a previously used build?  The googleapis SHAs are defined thus:
https://github.com/googleapis/google-cloud-cpp/blob/3942b25db6d47c8d421aa827ff54d28fd5229eb9/cmake/GoogleapisConfig.cmake#L19-L26
That means they are cached in the CMake configuration step.  Rebuilding will not download the new version.  Removing the CMake cache ($BINARY_DIR/CMakeCache.txt) should clear the problem.  Let me see if I can think of a solution to automate this.

Ahh, ok. I see. That solved it. Thanks for the hint.
As for a solution, why do you CACHE those variables in the first place? They could differ after each build, the user can't sensibly edit them and apparently the source code is ground truth here. Wouldn't it solve the issue if you just remove the CACHE STRING ... keywords?

As for a solution, why do you CACHE those variables in the first place?

A misguided attempt at making this something that end-users could override.

Wouldn't it solve the issue if you just remove the CACHE STRING ... keywords?

Yes. I went a step further and implemented a way for end-users to override the source of the tarball.  That can be useful if your build system / environments prohibits downloads.


Missing init-container-tag, or wrongly renamed definition?

Looking at release-0.21/deployments/gpu_plugin/base/intel-gpu-plugin.yaml for 0.21, I see both the regular, and the init-container was bumped to 0.21 - but there is no 0.21-tagged container on docker hub.
Is this a missing container, or a wrongly renamed tag?
For the record, it seems like everything works by editing the manifest with 0.20.0 for the initContainer.

but there is no 0.21-tagged container on docker hub.

0.21.0 tags are now published.

Yeah, works.


Ads not playing

Unchecking "remove ads" in the options doesn't seems to make a difference, be it the normal version or the developer one.
Browser name: Chrome
Browser version: 41.0.2272.118
Browser engine: WebKit
OS: Windows 8.1
YouTube Center version: Developer Version - Build #497
Addon type: Userscript
Debug log: Upload

Duplicate of #1165.


[mempool] Implement RPC for Mempool

Motivation
Built upon a new test framework, RPC now works.  DirectSend has to be used for the older multi node tests due to the overcomplications of running multiple nodes at once with RPC.  I will replace all the old tests in the future with single node tests to have greater control over what's happening.
This should be 100% backwards compatible.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
See existing tests.  Also, E2E tests / cluster test
Related PRs
https://github.com/diem/diem/pull/9638
https://github.com/diem/diem/pull/9616

/canary

:broken_heart: Test Failed - ci-test

/canary

:broken_heart: Test Failed - ci-test

/canary

:broken_heart: Test Failed - ci-test

/canary

/canary

:broken_heart: Test Failed - ci-test

/canary

:broken_heart: Test Failed - ci-test

/canary

:broken_heart: Test Failed - ci-test

/canary

:broken_heart: Test Failed - ci-test

/canary

:broken_heart: Test Failed - ci-test

I haven't looked at it closely, but by using RPC instead of DirectSend, we're more vulnerable to malicious nodes (forced blocking) compared to asynchronous ack. we might need to move the blocking part out from the main loop.

Yeah, I'll have to double check and make sure the RPC wait is in a separate task from the main loop.

/canary

:broken_heart: Test Failed - ci-test


expose more job variables via yaml files

Expose more variables to yaml file, to allow an user to have more control over job management.

LGTM, any comments @brndnmtthws ?

These fields were intentionally dropped. At the very least, it should ignore the disabled field because we don't want to re-enable jobs which were disabled due to failures (that defeats the purpose of the disable-after-N-failures feature).
The other fields were found to be more harm than good when modified, and thus were dropped. I'd be willing to include them back if necessary.

@brndnmtthws that makes sense in regards to disabled, I can remove that field.  The main fields that I want to update are epsilon and retries.  What are your thoughts on the other fields?  I don't see any harm keeping the async field as well.

Yeah, it's reasonable. These things may be dropped in future releases (particularly the async mode), just FYI.


[Storage][Blob] blobBody in download resposne could be confused with Azure Storage blob

Consider renaming to a better name

bodyAsBlob sounds better. suggested here.

contentAsBlob for the record is the final winner.

Please close the issue if merged.


Tests updated with variable names for servers - to consider

Hey Constantine,
Just a note that I recently discovered Context is sort of isolated (I suppose I should read up formally on Pester) and some variables do not persist unless $script: is used.
I littered Copy-DbaDatabase with some forced static names, my apologies. I simply couldn't get it working and it was strange. Ultimately, after the detach/attach, I had to use
$db1 = (Connect-DbaSqlServer -SqlInstance localhost).Databases['detachattach']
$db2 = (Connect-DbaSqlServer -SqlInstance localhost\sql2016).Databases['detachattach']

Instead of Get-DbaDatabase for consistent results ¯_(ツ)_/¯

No apologies needed! It sounds like I should dive into the pester docs a bit deeper and dispel some illusions I have about its control flow.
Thanks for catching this one!


image: Callback returns wrong value for secondary parameter

FLTK: Secondary is true for primary button and vice versa
Cocoa: Secondate button does not seem to trigger callback at all

Fixed now in 0.3.2.


Update Cyclone dds configuration

This PR brings in corrections to how to configure cyclonedds.
Courtesy of @thijsmie

@thijsmie I'm sorry that this got neglected. Do you have any interest to revive it? If not, I can try to piece it back together on top of the current main.

No worries, I was just cleaning up repositories, I was not really intending to close this. I am no longer affiliated with the CycloneDDS project so maybe it is better if someone else handles it.

Thanks; wishing you the best.


parsing quotation

Trimming double and single quots from the words and set the expantions it exists.

--suppressions=
{
	leak readline
	Memcheck:Leak
	...
	fun:readline
}
{
	leak add_history
	Memcheck:Leak
	...
	fun:add_history
}

Done


Creating generic Producer throws argument null exception

Description
I need to dynamically create generic producers for my development, but ProducerBuilder throws ArgumentNullException on Build(). Is it doable or am I implementing it wrong?
How to reproduce
My application framework is netstandard 2.1, nuget version 2.3.0
I have created a simple console app to reproduce it (in this case it is net8):
using Confluent.Kafka;

var config = new ProducerConfig
{
    BootstrapServers = "localhost:9093",
};

var producer = new MessageProducer<SampleMessage>(config);

producer.Produce(new SampleMessage{Value = "Hello World!" }, "sample-message-worker");

Console.ReadLine();

class MessageProducer<TMessage> where TMessage : class
{
    private readonly IProducer<Null, TMessage> _producer;
    public MessageProducer(ProducerConfig pconfig)
    {

        _producer = new ProducerBuilder<Null,TMessage>(pconfig).Build();
    }

    public void Produce(TMessage message, string topic)
    {
        var msg= new Message<Null, TMessage>() { Value = message};
        _producer.Produce(topic, msg);
    }

}

class SampleMessage
{
    public string Value { get; set; }
}

Checklist
Please provide the following information:

[x] A complete (i.e. we can run it), minimal program demonstrating the problem. No need to supply a project file.
[x] Confluent.Kafka nuget version.
[ ] Apache Kafka version.
[ ] Client configuration.
[ ] Operating system.
[ ] Provide logs (with "debug" : "..." as necessary in configuration).
[ ] Provide broker log excerpts.
[ ] Critical issue.

Can you provide more details on the exception with the stacktrace? You can refer our examples https://github.com/confluentinc/confluent-kafka-dotnet/blob/a67bd6c06b7eef4293e6476d9ff6f3e93f0e4cd9/examples/JsonSerialization/Program.cs#L141 to how to use ProducerBuilder correctly.

@Pablovide I see that erroendue to Serializer..
For the Custom model you need to register the SetSerializer method with the Custom Serializer class derived from ISerializer interface.
That should work for you if still you have the issue.


pytorch definition of the model

Is there a pytorch definition and pytorch model weights of the architecture used for the white board leaderboard?
We would like to try our attack on your challenge but unfortunately our code is written in PyTorch.
If there is any other way, please let me know?

We don't really have an equivalent PyTorch implementation (or plans to release one).
That being said, we recently released an adversarial training library in PyTorch: https://github.com/MadryLab/robustness . It has pretrained CIFAR models for L2 and Linf adversaries. This is not really a robustness challenge, but is probably still useful for comparing a new attack to PGD.


Cannot get reference to UIViewController registered for storyboard injection

I'm using Swinject 1.1.1.  I really want to be able to inject a reference to my UIViewController which has been registered for storyboard injection.  I currently have a SpriteKit app where class GameViewController: UIViewController is injected into Main.storyboard via registerForStoryboard().   Creating this view controller causes creation of a bunch of SKScene's one of which needs a reference to the original GameViewController in order to be able to call presentViewController to show settings.  I tried to resolve the GameViewController in the initComplete (to avoid a circular reference) but it comes back as nil.  Am I missing something?  Can this be made to work and if not, what's the underlying issue?

I guess you need to call instantiateViewControllerWithIdentifier or instantiateInitialViewController to get dependencies injected via the storyboard. I should update the document to describe the usage explicitly.
If you show me essence of your code, I might be able to give more accurate answer.
(I think you already know SwinjectStoryboard in this repo is for Swinject v2. V1.x includes SwinjectStoryboad by itself. )

OK, I finally got some time to dig into this.  It appears the problem is my lack of understanding of what is actually going on with storyboard injection.  It appears from SwinjectStoryboard.swift:90 that the UIVIewController is never actually saved in the container.  The only place that you can get the controller is in the initComplete call.  Is that correct?

Yes, UIViewControllers are not stored in a container but just created and returned by calling factory closures, unless .Container scope is specified.
The place you can access the view controllers is, as you told, initCompleted which is the closure specified with registerForStoryboard method in the storyboard injection.

That makes sense as you would generally want to avoid an unnecessary strong reference to the UIVIewController by holding on to it in the Container.  Thanks!


Support Squirrel flag JSON output for SUFeedURL - Sparkle

Describe the solution you'd like
Support the Squirrel flag for the Sparkle update feed. This was supported on HockeyApp and it is required for Electron AutoUpdater, which does not support XML output and requires JSON. See example: https://rink.hockeyapp.net/api/2/apps/bc02347d258a40e2af46cc8de3566bab.json?squirrel=true
Describe alternatives you've considered
It won't be possible for us to complete the transition to AppCenter from HockeyApp until support for Squirrel JSON format is added. Please do not force a transition before this is supported as it will cut off our customer's ability to update their apps.

Hi @rishimodha,
Thanks for bringing up this issue to our notice.
This was of low priority when we started sparkle feed support in AppCenter.
Now, we have added the support of Squirrel JSON format.
Here is the corresponding AppCenter sparkle feed url for your example.

https://api.appcenter.ms/v0.1/public/sparkle/apps/<app_secret>?squirrel=true
This returns the latest release in JSON format and if there is no release then returns empty object.

https://api.appcenter.ms/v0.1/public/sparkle/apps/<app_secret>
This as usual returns the latest 5 releases in xml format.

Please let us know your experience on this and get back to us if you have any other concerns/requirements.

@Ajaykn21 thanks for adding this.
If possible, can you add handling to redirect the HockeyApp URL that contains the squirrel tag to the corresponding AppCenter URL with the squirrel tag?

Yeah @rishimodha, we are working on this already and will update over here once it is ready.

@Ajaykn21 - Any update on this?

@rishimodha , we are flooded with other priority requests. We have started working on this and will update you once done.

@rishimodha, we have released this to Production.
And we redirect HockeyApp Sparkle feed to AppCenter Sparkle feed for the apps that are cutovered.
Although, it is recommended to update your apps SUFeedUrl to AppCenter Sparkle feed url.
Can you please check and let us know.

@Ajaykn21 Yes, this working great. Thank you!

@rishimodha can we close this issue, as this is already fixed ?


Apc Extension failed  in windows

Apc Extension failed: Error: ENOENT: no such file or directory, rename 'c:\Users\rc7ga\AppData\Local\Programs\Microsoft VS Code Insiders\resources\app\out\bootstrap-amd.js' -> 'c:\Users\rc7ga\AppData\Local\Programs\Microsoft VS Code Insiders\resources\app\out\bootstrap-amd.js.apc.extension.backup'

The same
VS Code Insider latest version
APC extension latest version - v0.3.9

Looks like file has been renamed in newer version. There is no longer bootstrap-amd.js in the resources directory.
Apc Extension failed: Error: ENOENT: no such file or directory, rename 'c:\Users\USER\AppData\Local\Programs\Microsoft VS Code Insiders\resources\app\out\bootstrap-amd.js' -> 'c:\Users\USER\AppData\Local\Programs\Microsoft VS Code Insiders\resources\app\out\bootstrap-amd.js.apc.extension.backup'

i think this would help

in src/patch.ts
but havent test yet(not familay with devlop vs code extension)

Got the same issue here. Using VS Code Insiders 1.92.

Apc Extension failed: Error: ENOENT: no such file or directory, rename 'c:\Program Files\Microsoft VS Code Insiders\resources\app\out\bootstrap-amd.js' -> 'c:\Program Files\Microsoft VS Code Insiders\resources\app\out\bootstrap-amd.js.apc.extension.backup'

i think this would help  in src/patch.ts but havent test yet(not familay with devlop vs code extension)

Where this file located? Was it on VS Code installation folder?

i think this would help  in src/patch.ts but havent test yet(not familay with devlop vs code extension)

Where this file located? Was it on VS Code installation folder?

i just clone this repo, modify the source code and build from it, but failed to install on my vscode insiders

Ah, I see. Seem like Microsoft changed the way to do that for APC, I suppose.
Sent from my iPhone 6+ lol

On 13 Jul 2024, at 18.37, Caffeine19 @.***> wrote:
i think this would help  in src/patch.ts but havent test yet(not familay with devlop vs code extension)
Where this file located? Was it on VS Code installation folder?
i just clone this repo, modify the source code and build from it, but failed to install on my vscode insiders
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.

It is not specific to windows, also reported here https://github.com/drcika/apc-extension/issues/198
fwiw — the file does exist on release/1.91 https://github.com/microsoft/vscode/blob/release/1.91/src/bootstrap-amd.js but I can't see any tags or branches with 1.92.0-insiders, the file also exists in the latest commit on main

What is the possible resolution for this file name change? I tried manually changing the name, but it resulted in more errors, which was expected, since it was a wild attempt.

What is the possible resolution for this file name change? I tried manually changing the name, but it resulted in more errors, which was expected, since it was a wild attempt.

My understanding is that this extension is broken completely until they issue an update that works with the Insiders build. aka, don't use Insiders or wait for an update.

Still the same issue with 1.92.0-insider and Apc Customize UI++ from 2024-07-29.

MS pushed this update to mac as well... anyone on a mac that updates VS will get this error and Apc wont work.

got the same issue with Mac, I'm using vscode installed via brew


Pretrained models

Hey @chengtan9907, are there any pretrained models that you could share?
It'd help kickstart learning on newer datasets, as well as act as a stand in set of weights to build pipelines.

Hi, @decent-engineer-decent-datascientist, thanks for your interest in SimVPv2. Since video prediction methods on some low-resolution datasets need tuning of network structures (e.g., SimVP might use different layers in different datasets), it might be not useful to load a pre-trained model for a new dataset. Therefore, we haven't provided trained models in SimVP in the past.
Currently, we are working on this project to add more benchmark results and video prediction datasets. We plan to release trained models and logs in the future in regard to the convenience of the community. Please watch our latest update. And we will inform you when we release the models.

I will close this issue if there is no more question. Please feel free to open a new issue when you have new questions.


why need c_scr?

Hi~
I  am reading the shader code now. And I have already been clear about most of them.
However , in attribure_vs.glsl, I am confused about c_scr, which is computed as follows:
Out.c_scr = vec2((p_scr.xy + 1.0) * viewport.zw * 0.5);

I know p_scr is the point c in screen space,  so what does c_scr mean? And what is the difference between them?

OK， I think I have figured it out.
Because p_scr is in the range [-1,1], and c_scr  use pixel unit.


iOS 10: NSISO8601DateFormatter

iOS 10 Foundation includes NSISO8601DateFormatter which can be used for date parsing

cleaning up old issues - myself, i'll not be investing in this implementation as i support Gloss in maintenance & suggest generally to use Codable


[Enhancement]: develop a better chinese similarity algorithm

What would you like to be added?
When I asked multiple questions, it always return same answers only because they mention same keywords.
Like:
I asked "what is coffee(咖啡是什么)",
it returns. " Coffee is a drink make of coffee beans " ... bla bla.
Correct answer.
But When I asked it "How to make Americano coffee(美式咖啡怎么制作的)", it answered me with same answer like above question.
Expected a better NLP.
Why is this needed?
Because it's totally different question.
Anything else?
No response

If you want to support bilingual cache, you need to use an embedding model that supports bilingual.

The next version, we will support the bilingual embedding model, which is uform, link: https://github.com/unum-cloud/uform

the latest version has support, reference: https://gptcache.readthedocs.io/en/latest/references/embedding.html#module-gptcache.embedding.uform


Weather Dashboard

User Story
AS A traveler
I WANT to see the weather outlook for multiple cities
SO THAT I can plan a trip accordingly
Acceptance Criteria
IVEN a weather dashboard with form inputs
WHEN I search for a city
THEN I am presented with current and future conditions for that city and that city is added to the search history
WHEN I view current weather conditions for that city
THEN I am presented with the city name, the date, an icon representation of weather conditions, the temperature, the humidity, and the wind speed
WHEN I view future weather conditions for that city
THEN I am presented with a 5-day forecast that displays the date, an icon representation of weather conditions, the temperature, the wind speed, and the humidity
WHEN I click on a city in the search history
THEN I am again presented with current and future conditions for that city

Done


vscode: do not show "Rename Chat" in command pallette

This command only works when directly clicked on a chat item in the history. So we use the same enablement condition as when it is shown there.
Test Plan: tested renaming chat still works and that it doesn't appear in command pallette

@sqs I know you want this feature axed. I will do that in the next PR. Just want to fix the bug first in case others decide we actually want to keep this feature.


standard user unable to use cloud credentials in a node template

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

As an admin create a standard user - user1
login as user1
create a cloud credential and a node template
DO nodetemplate fails with error Unauthorized: Unable to authenticate you and aws node template fails to get created with Error MissingParameter: The request must contain the parameter AWSAccessKeyId
Rancher logs:

2021/02/13 15:54:42 [INFO] Failed to proxy: unauthorized u-x3nanjhpzu to cattle-global-data/cc-nb4fm:
2021/02/13 15:54:42 [INFO] Failed to proxy: unauthorized u-x3nanjhpzu to cattle-global-data/cc-nb4fm:

Expected Result:
User should be able to create a nodetemplate
Other details that may be helpful:
This does NOT happen on 2.5-head latest
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): master-head - commit id: 98fe72a4
Installation option (single install/HA): single install

Caused by https://github.com/rancher/rancher/pull/31226

Has this been tested with 2.5.8 and is it still an issue?

@sowmyav27 - Can we re-test this on the latest 2.5-head and we can confirm that it's still happening?

@SheilaghM and @Jono-SUSE-Rancher : This issue is not seen on v2.5.8 but only on the master-head 2.6.
Re-opening this ticket as this is still seen on the master-head ebad1df


TestDescriptor should expose getDisplayName and getClassDisplayName methods

Expected Behavior
The TestDescriptor interface should expose the getDisplayName() and getClassDisplayName() methods which are currently only available on TestDescriptorInternal. This way they can be used in before and after callbacks:
test {
    afterTest { desc, result
        println "$desc.displayName result is $result.resultType"
    }
}

Current Behavior
getDisplayName() and getClassDisplayName() are not available on TestDescriptor
Context
Part of: #10983
Follow up of: #11026

Wouldn't getClassDisplayName be ecosystem-specific? Some ecosystems don't have the concept of class.

I agree with you that exposing getClassDisplayName is problematic. Since DefaultTestClassDescriptor already returns the classDisplayName as displayName I'm not even sure we really need it exposed. The only use case would be if I have a descriptor that represents a method, I could ask for the display name of the containing class. But this could also be done by using TestDescriptor.getParent().

I tend to agree with Cédric. While the test class/method assumption is currently omnipresent in the codebase, not exposing getClassDisplayName() would make it easier to change that in the future.

I changed the name of the issue to refelct this.


Remove compile and load time dependency on Quicklisp

Dynamically probe for :quicklisp package and call its symbols on
demand. References to symbols in that package are made through
uiop:symbol-call and uiop:find-symbol*. The rule doesn't apply
to '.ros' files.

Made the recommended changes. Loaded using ASDF only and it seems to work.


Fix: checksums

Description:
Fixes checksums
Checklist

[X] Documented (Code comments, README, etc.)
[X] Tested (unit, integration, etc.)

The changes look good, but I think we're waiting on making this a sprint task before we merge this.


Multi-Site "Crawling discovered URLs"

I have multi-site WP installed. The parent site is returning error message "Crawling discovered URLs" while the other 4 linked child sites copy just fine. How do you troubleshoot this.

Hi @azbizdoc I've recently put this guide together and addresses the same error (Crawling detected is first phase, Crawling discovered is second, so hints at hitting execution time limits):
https://staticword.press/t/failed-during-crawling-initial-file-list-please-consult-the-help-tab-for-where-to-get-assistance/59
I'll close this issue as likely not code-related, but please let me know how you get on via the forum


Renaming the file paths to avoid issues on Windows

Since Windows does not allow colons in file names, I think we need to rename these files in Git to remove or replace the colons.
Initially, I intended to fix it and create a Pull Request. However, these files cannot be displayed on my local Windows OS.
@Ash-2k3 Could you please take a look and help me resolve these issues? Thanks a lot!
(/cc @U8NWXD for information)
Specifically, the following files:

Tutorial:-Learn-How-to-Write-and-Test-a-Non-Trivial-Beam-Job.md
Tutorial:-Learn-to-figure-out-the-reproduction-steps-for-server-error.md

Suggestions:

Rename the files using the git mv command:

git mv "Tutorial:-Learn-How-to-Write-and-Test-a-Non-Trivial-Beam-Job.md" "Tutorial-Learn-How-to-Write-and-Test-a-Non-Trivial-Beam-Job.md"

git mv "Tutorial:-Learn-to-figure-out-the-reproduction-steps-for-server-error.md" "Tutorial-Learn-to-figure-out-the-reproduction-steps-for-server-error.md"

Update the content in the _Sidebar.md file without colon:

[[Tutorial - Learn-to-figure-out-the-reproduction-steps-for-server-error|Tutorial-Learn-to-figure-out-the-reproduction-steps-for-server-error]]
[[Tutorial - Learn to write and test a non-trivial beam job|Tutorial-Learn-How-to-Write-and-Test-a-Non-Trivial-Beam-Job]]

Hi @miro789, thanks for reporting this issue. I will open a PR for this within an hour.


Error "message.recipients.to must be of type Array."

New installation of Strapi. When sending an Email using this I get the following error:
I know the email service works as I tested it in a separate node project using Azure only.
error: Couldn't send test email: Error "message.recipients.to must be of type Array." occurred in serializing the payload - undefined..
Error: Couldn't send test email: Error "message.recipients.to must be of type Array." occurred in serializing the payload - undefined..

The package.json
    "@strapi/plugin-i18n": "4.10.5",
    "@strapi/plugin-users-permissions": "4.10.5",
    "@strapi/strapi": "4.10.5",
    "better-sqlite3": "8.0.1",
    "strapi-provider-email-azure": "^1.0.1"

Fixed this by changing how the data is formatted in recipents
   recipients: {             to: [               {                 address: options.to               }             ],             cc: options.cc,             bcc: options.bcc,           },

Hey @Freyb can my pull request be merged please?

Hey @Komsomol ,
Thanks for reaching out.
I don't think the solution you've provided is the correct one. As I wrote in the docs of the provider, options.to, options.cc and options.bcc can take up more complex forms than a simple email address.
Nevertheless, thank you for pointing out that the Strapi test email service only gives a single string to the assigned provider.
I've updated the provider to handle this case while keeping the former functionality.

Thanks, @Freyb!


Update eslint-config-standard.

We need to discuss this more because eslint-config-standard new versions have some breaking changes. #34

@vilaboim Can you elaborate?  I'm not very familiar with this project.

@joeheyming we realized that new versions of eslint-config-standard have some breaking changes.
So I believe we will merge your PR and make some changes before release an update.
cc @marlonmleite

Merge. Merge nooooow!!! : )

It has been several months since this commit.  I would probably resubmit this commit with a newer version of eslint-config-standard, just to keep up with the changes


Jerky motion using MoveIt Servo

I am experiencing a problem related to the acceleration profile generated by MoveIt Servo trajectory planner on my UR5e. I’m trying this example of pose tracking. For a bin-picking scenario.
The documentation specifies that MoveIt Servo can enforce only joint position and velocity limits. However, I am experiencing a rough acceleration profile at the start of the motion as shown in the following picture:

At the starting point, the robot joints make an annoying noise, which I want to avoid.  During the deceleration phase this noise is not present and the deceleration profile seems much smoother.
I have seen that the time parametrization in MoveIt2 uses the TOTG algorithm to define the velocity and acceleration profiles, which enforce up to the joint acceleration limit. However, it seems that the acceleration limit is not read in MoveIt Servo. I assume that TOTG is used any time you plan using OMPL and Pilz planner, but I haven’t found this information written somewhere. And I would like to understand if changing to Pilz for instance may be a solution to my problem as it enforce at least the acceleration limit.
 From the documentation I have seen that this additional plugin is specified in the ompl_planning.yaml, therefore I believe it can be used only when planning with OMPL. Is this true?
As my goal is to remove this unwanted sound when I move my robot, do you think that implementing a vanilla Pilz planner enforcing the acceleration limits is enough? Do you have examples I can take to do so? I am using ros2 Humble.
Is there the possibility of implementing a minimum jerk time parametrization?
Thanks.

Your best bet is probably to change this line:
https://github.com/moveit/moveit2/blob/f9b5a11a9fbc5e76ec2cc81fbbbb2ad2b70ffc83/moveit_ros/moveit_servo/config/panda_simulated_config_pose_tracking.yaml#L31
To:
online_signal_smoothing::RuckigFilterPlugin
Which is not in Humble, only later distros. Sorry. I guess you could make a PR to port that change back to Humble.

See https://github.com/moveit/moveit2/blob/main/moveit_core/online_signal_smoothing/src/ruckig_filter.cpp

Hi @AndyZe! Thank you for your answer. I tested the Pilz planner, which significantly reduces the jerk at the starting point, enforcing at least the acceleration limit. The resulting motion is visibly smoother and the acceleration peaks are limited as shown in the following figures:
Pilz Planner

Servo Planner

The graph shows the end effector position and the velocity and acceleration value computed through the central difference derivative, I will have a look directly at the joint trajectory, but it seems that this solution works well for my use case.

Happy that you got it working!


cob_command_tools: 0.6.33-1 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository cob_command_tools to 0.6.33-1:

upstream repository: https://github.com/ipa320/cob_command_tools.git
release repository: https://github.com/ipa320/cob_command_tools-release.git
distro file: noetic/distribution.yaml
bloom version: 0.11.2
previous version for package: 0.6.32-1

cob_command_gui

No changes

cob_command_tools

No changes

cob_dashboard

No changes

cob_helper_tools

No changes

cob_interactive_teleop

No changes

cob_monitoring
* Merge pull request #335 <https://github.com/ipa320/cob_command_tools/issues/335> from benmaidel/fix/em_topic_based_em_stop_mon
  fixed em topic based emergency_stop_monitor
* fix melodic compatibility
* fixed em topic based emergency_stop_monitor
* Merge pull request #334 <https://github.com/ipa320/cob_command_tools/issues/334> from HannesBachter/feature/topic_emergency_stop_monitor
  set light based on emergency stop topic
* log em-stop only after issuing
* set light based on emergency stop topic
* Merge pull request #333 <https://github.com/ipa320/cob_command_tools/issues/333> from benmaidel/feature/battery_monitor_params
  extend battery_monitor with additional rosparams
* fix default values
* extend battery_monitor with additional rosparams
* Contributors: Benjamin Maidel, Felix Messmer, HannesBachter

cob_script_server
* Merge pull request #331 <https://github.com/ipa320/cob_command_tools/issues/331> from benmaidel/feature/allow_bigger_rel_rotation
  allow max relative rotation of pi*2+0.1
* allow max relative rotation of pi*2+0.1
* Contributors: Felix Messmer, mailto:bnm@mrl-2

cob_teleop

No changes

generic_throttle

No changes

scenario_test_tools

No changes

service_tools

No changes

Holding for Noetic sync

This pull request has been mentioned on ROS Discourse. There might be relevant details there:
https://discourse.ros.org/t/preparing-for-noetic-sync-2023-11-09/34547/2


Call for comments: bind prototypes to a registry

Currently, Prototype manages internally its components. I'd like to pass a registry to the constructor instead and therefore to use the registry as its (let me say) underlying storage.
The prototype will create an entity to which to bind the components assigned to it and will destroy the entity in the destructor. All its member functions will have two overloads: one that uses the underlying registry and one that copies the prototype over to a different registry. Something like this:
// creates an entity in the underlying registry and assigns it all the components of the prototype
const auto e0 = prototype();
// creates an entity in another registry (not the underlying one) and assigns it all the components of the prototype (by cross-copying them between the two registries)
const auto e1 = prototype(registry);

This is by far easier to maintain and make much sense for me.
To avoid collisions with the actual types of the components, the prototype will wrap those types in custom ones. As an example:
template<typename Component>
struct Wrapper {
    Component component;
};

This way, prototypes won't interfere in any case with systems that are interested in Component, even when prototypes share the registry with the rest of the application.
Unless someone comes with a good reason not to do it, the change will be part of the next release.

@skypjack So prototype will have inside instance of registry?

@ArnCarveris A reference to a registry, yep - Registry<Entity> &

@skypjack What prototype will hold inside?

@ArnCarveris Prototype will have two data members: a reference to a registry and an entity identifier, the one of the entity to which the prototype attaches its components through the registry.

Should probably store a pointer to the registry rather than a reference so that prototypes can be moved.

@Kerndog73 A reference to a registry should not prevent you from moving a prototype. I didn't get your comment but, well, it makes sense in terms: a prototype should be moveable, I agree. :-)

@skypjack Why Prototype needs entity identifier? This change will break my use cases mentioned in #56?

Sorry, which post? #56 is pretty long... :-(

@ArnCarveris Sorry, which post? #56 is pretty long, unfortunately...

@skypjack right after this

@ArnCarveris I don't get how a prototype that internally uses an entity breaks your use case. Components used by the prototype don't interfere with the ones of the application (it wraps them in a dedicated type). Moreover, it is allowed to use two registries: one to store the prototypes and one for the rest of the applications. Prototypes that use a registry can create entities on a second registry. How does this interfere with the use case mentioned in the other issue exactly?

@skypjack If wont break, when aren't any problem with this.

Just a nitpick: I'm pretty sure the member functions create, assign, accommodate and operator() can be marked const.

@Kerndog73 Yep, it's right. Wouldn't it be a bit misleading anyway? Even if const, they modify the underlying registry by creating a new entity and some components. Not sure if this is something I want to allow with a const prototype. What about?

The prototype does modify the registry but the prototype doesn't "own" the registry. The prototype only refers to the registry. The registry isn't apart of the prototype's "state".
When I pass a const entt::DefaultPrototype & to a function, I expect the function to create entities.
When I pass a entt::DefaultPrototype & to a function, I expect the function to fill up the prototype with components.

@Kerndog73 It makes sense.

I just noticed that Prototype doesn’t have a move constructor. So when you try to move it, you get an assert in registry.destroy.
A “moved from” prototype could store a null entity (just like a “moved from” std::unique_ptr stores a nullptr) so that registry.destroy succeeds (delete nullptr succeeds) but all other operations on the prototype result in asserts in the registry.

Good point. It's a sort of enhancement of the prototype class.
Could you open a dedicated issue for this? It makes sense.


Need Information

is sdp library support different drawable based on screen configuration?

sdp is for size units it doesn't solve for serving different drawables for different resolutions.


[Feature]: Add context menu option to open html file with Edge DevTools

This PR adds a new context menu item when right clicking HTML files in the VSCode file explorer:

Clicking this context menu item will launch a headless version of Edge targeting the page and attach the DevTools.
The end config looks like this:

Note that the user does not need a supported launch.json to run this workflow.

@mliao95 what do you think about leaving the DevTools closed by default (possibly with some rewording of the context menu entry to avoid implying they'll open automatically)?
Given we have the button in the screencast to toggle the tools, I think starting users off with the cleaner, screencast-only entrypoint will reduce how much content we're popping in front of them all at once while still letting them get to the DevTools themselves if needed.


audiounit: Fix race on frames_played.

ddbc367 removed the holding of stm->mutex from the render callback and switched some fields to std::atomic, but frames_played was missed.
Also includes some TSAN fixes for test_sanity.

r? @padenot


How to strictify ZodType

Using z.ZodType, typescript tells me that I'm missing the name key from the objSchema, correctly.
type Obj = {
  id: string
  name: string
}

const objSchema: z.ZodType<Obj> = z.object({
  id: z.string()
}).strict()

But, this example:
type Obj = {
  id: string
  name: string
}

const objSchema: z.ZodType<Obj> = z.object({
  id: z.string(),
  name: z.string(),
  extraParam: z.string() // unwanted extra param
}).strict()

Doesn't warn me that I have an extra key in my schema definition. How can I achieve the same level of strictness from the generic type of the strict() method?

I believe this is a byproduct of TS's structural typing. For example, this code exhibits the same behavior:
class Example<T extends Record<string, string>> {
  shape: T;
  private constructor(shape: T) {
    this.shape = shape;
  }
  static create<U extends Record<string, string>>(shape: U) {
    return new Example(shape);
  }
}
const ex: Example<{foo: string}> = Example.create({foo: "", bar: ""});


i18n: spanish translation

Currently the project has no spanish translation so here is some help for that
I think all the work I've done on that is good. I hope you enjoy this contribution!

Thanks for making this!