Update aerospace extension

Description
Adds a new command to change apps of the current workspace.
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

I understand. My point is about this PR not adding any dependencies yet
updating the lock file. We can leave this for the maintainer if you prefer.
On Sat, Oct 12, 2024 at 12:15 Alejandro Cámara @.***>
wrote:

@.**** commented on this pull request.
On extensions/aerospace/package-lock.json
https://github.com/raycast/extensions/pull/14886#discussion_r1797640081:
In Raycast extension guidelines: “ Please use npm for installing
dependencies and include package-lock.json in your pull request.”
—
Reply to this email directly, view it on GitHub
https://github.com/raycast/extensions/pull/14886#discussion_r1797640081,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACVYGHCI3OWA6GZ2DH2LBLZ3DSB3AVCNFSM6AAAAABPZL7WCKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNRTHAZTONJYHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Honestly, I’m not knowledgeable enough. So whatever is the best thing to do.
Alejandro Cámara López
Interaction Designer
alejandrocamara.info
On Sat, 12 Oct 2024 at 11:19, Ammar Alakkad @.***>
wrote:

I understand. My point is about this PR not adding any dependencies yet
updating the lock file. We can leave this for the maintainer if you
prefer.
On Sat, Oct 12, 2024 at 12:15 Alejandro Cámara @.***>
wrote:

@.**** commented on this pull request.
On extensions/aerospace/package-lock.json
https://github.com/raycast/extensions/pull/14886#discussion_r1797640081:

In Raycast extension guidelines: “ Please use npm for installing
dependencies and include package-lock.json in your pull request.”
—
Reply to this email directly, view it on GitHub
https://github.com/raycast/extensions/pull/14886#discussion_r1797640081,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AACVYGHCI3OWA6GZ2DH2LBLZ3DSB3AVCNFSM6AAAAABPZL7WCKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNRTHAZTONJYHA>

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/raycast/extensions/pull/14886#issuecomment-2408472129,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA5U36NOTAIDRSV7CF4N7DZ3DSSTAVCNFSM6AAAAABPZL7WCKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBYGQ3TEMJSHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

I don't have control over this PR, but

Hey @nikitabobko,
I was afraid you would comment and remind me that I still been to change
that logic.
Family and other things happened so I couldn't do it. I am planning to do
it soon though hopefully in two weeks 😬
I am glad for you to give me a proper way and apologise again for my
tardiness
On Sat, 12 Oct 2024, 12:43 Nikita Bobko, @.***> wrote:

@.**** commented on this pull request.
In extensions/aerospace/assets/scripts/list-windows.sh
https://github.com/raycast/extensions/pull/14886#discussion_r1797686836:

@@ -0,0 +1,15 @@
+#!/usr/bin/env bash

+# References:
+# - https://nikitabobko.github.io/AeroSpace/commands#list-windows
+# - https://www.alfredapp.com/help/workflows/inputs/script-filter/json/
+
+result=$(aerospace list-windows --workspace visible --format "%{app-name},%{window-title},%{window-id},%{app-pid},%{workspace},%{app-bundle-id}")
I don't have control over this PR, and the final say is after people who
support the raycast plugin.
But as a maintainer of AeroSpace, the hacking around unstructured format
makes me sad. Sure, you can use it for your own needs, but when you publish
it for other people, I think it's better to clean it up. I'd wait for
nikitabobko/AeroSpace#577
https://github.com/nikitabobko/AeroSpace/issues/577 before merging this
PR
I want to take this opportunity to remind that the plugin currently parses
aerospace.toml config itself (which is fragile), while the proper API was
provided nikitabobko/AeroSpace#215 (comment)
https://github.com/nikitabobko/AeroSpace/issues/215#issuecomment-2119209902
(and still waits to be used)
This PR increases the technical debt of the raycast plugin
—
Reply to this email directly, view it on GitHub
https://github.com/raycast/extensions/pull/14886#pullrequestreview-2363897481,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB2LFHD6WC2FZMWJ5NKYZX3Z3EDOXAVCNFSM6AAAAABPZL7WCKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNRTHA4TONBYGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Closed in favor of updated version with Aerospace 0.15.0.

So is this switchApps functionality coming soon then?  I was just working on adding it myself - would be great if it's already in the works!

So is this switchApps functionality coming soon then? I was just working on adding it myself - would be great if it's already in the works!

Yes, there is already a new PR with this functionality and aerospace latest —json improvement.


Test filtering/searching on reddit

Someone please test the filtering/searching in the reddit frames please.

Sounds good!

As an update, it looks like it works for :search with a small issue (the next button still shows). But :s does not work. I will submit a PR later to fix this issue!


Record taxon rank & map via export

Taxon names in specimen table in database are of mixed rank, which makes importing to Specify difficult

[x] Recaption "higher taxon" to "family" in autosuggest box (including corresponding table column)
~- [ ] Create taxonrank table for holding taxon rank names for each collection mapping rankid+collectionid to rank name~
[ ] Redo predefined data inserts to include rankname by pulling corresponding data
[x] Change specimen table to accomdate taxonrankname
[ ] Change date entry to save values to column taxonrankname

item 2 - entomology completed

I have been thinking about this a bit...
So the goal here is to have a taxon name table for each collection: NHMD Vascular Plants, NHMD Entomology, and NHMD Aarhus, each containing a new column: Rank-name.
I propose that a switch like statement is implemented adding the rank name to all entries in the taxonomy from Specify. There should be a SQL script file for each collection. This reduces complexity and single point of failures within the SQL script.
THe CASE, WHEN THEN statement will be similar to this:
CASE
	WHEN lower(a."Rank") = 'class' THEN '60'
WHEN lower(a."Rank") = 'order_' THEN '100'
WHEN lower(a."Rank") = 'family' THEN '140'
WHEN lower(a."Rank") = 'genus' THEN '180'
WHEN lower(a."Rank") = 'species' THEN '220'
WHEN lower(a."Rank") = 'subsp.' THEN '230'
WHEN lower(a."Rank") = 'var.' THEN '240'
WHEN lower(a."Rank") = 'subvariety' THEN '250'
WHEN lower(a."Rank") LIKE 'form%' THEN '260'
ELSE NULL END AS "rank", ...
FROM taxon a

We will end up having three scripts and the only thing to be switched in/out is the database for Aarhus.

As agreed during catchup, instead of CASE ... WHEN statement, create taxonrank table linking rankid with rankname that then can be JOINed

Since the record numbers did not quite add up after the pre-Easter effort, I suggest a more modular approach to creating the Editions files.
I propose that this query is run to create a base table for the NHMD Vascular plants, NHMD Entomology, and the NHMA extraction.
SELECT t.* ,
CASE
when t.rankID = 20 then 'subkingdom'
WHEN t.rankID = 30 THEN 'division'
WHEN t.rankid = 40 THEN 'subdivision'
WHEN t.rankid = 50 THEN 'superclass'
WHEN t.rankid = 60 THEN 'class'
WHEN t.rankid = 70 THEN 'subclass'
WHEN t.rankid = 80 THEN 'infraclass'
WHEN t.rankid = 90 THEN 'superorder'
WHEN t.rankid = 100 THEN 'order'
WHEN t.rankid = 110 THEN 'suborder'
WHEN t.rankid = 120 THEN 'infraorder'
WHEN t.rankid = 130 THEN 'superfamily'
WHEN t.rankid = 140 THEN 'family'
WHEN t.rankid = 180 THEN 'genus'
WHEN t.rankid = 220 THEN 'species'
WHEN t.rankid = 230 THEN 'subspecies'
WHEN t.rankid = 240 THEN 'variety'
WHEN t.rankid = 250 THEN 'subvariety'
WHEN t.rankid = 260 THEN 'forma'
ELSE 'unknown'
END AS 'rankname'
FROM taxon t
WHERE t.TaxonTreeDefID IN (2, 5, 13);

Workflow:

Run the SQL for the base table
Export this table result to local DB
Run specific SQL file (to be created by J. Legind) for each Collection
Move the result into the relevant 'editions' directory

Instead of using the big CASE statement I have resolved to having a small taxon-rank table on which to join.


Update _config.yml

Update _config.yml to remove traces of "AB3"

Codecov Report

Merging #42 (4254fcb) into master (2398f2c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #42   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Continue to review full report at Codecov.

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


ODBC query errors

I do not kown how to do.  Thanks for your help.
julia> using ODBC
julia> conn = ODBC.connect("DSN1",usr="wangfeng",pwd="wangfeng")
ODBC Connection Object
Connection Data Source: DSN1
DSN1 Connection Number: 7
Connection pointer: Ptr{Void} @0x000000000bb47b10
Statement pointer: Ptr{Void} @0x000000000bb47e90
Contains resultset? No
julia> query(connection::Connection=conn,"SHOW TABLES;"; file=:DataFrame,delim='
\t')
ERROR: syntax: keyword argument is not a symbol: (:: connection Connection)

@Maple-Wang does the above comment fix your issues? Can this issue be closed?


Add upgrade note for policy controller

Description
This PR adds a note to the K8s upgrade page about the need to upgrade calico-node before updating the policy controller.
Todos

[X] Release note

Release Note

If using the policy controller and upgrading from v2.5.x Calico or earlier it is necessary to upgrade calico-node before upgrading the policy controller.

LGTM.

Scratch that, we're going to look into how we can avoid this requirement.


Fixes #26 #33 #32

This PR fixes multiple issues with setting up postgresql with CentOS6 AND it doesn't not break anything for CentOS7.
Pretty please take a look at this PR and testing support it provides.
just install test-kitchen with
gem install test-kitchen

and run
kitchen setup all -p -c 4

this will provision 4 vms:

2 with 9.3
and 2 with 9.4

Thx
not familiar with kitchen testing running vagrant here but i will give it a try

This actually breaks CentOS 7 + Postgres 9.4 support--see: https://github.com/Open-Future-Belgium/PostgreSQL/issues/19
@Trikke76 it is annoying when I take the time to submit a fix, which is accepted, only to come back a few months later and hit the exact same issue because another fix was merged on top that has a commit doing nothing but reverting my fix. Please don't blindly accept pull requests without at least paying attention to the prior commit history.
Probably a better solution than hardcoding the default socket is making it a role parameter with a default value that changes based on the underlying OS.

apparently, user/db creation should be included in tests/test.yml to mitigate breaking changes 'cause nobody was testing user/db creation at this point.
And also I should point out that there's was no testing  whatsoever until then.

@kostyrevaa +1 for adding tests!

sorry been busy and unable to update the roll
I will try to fix things somewhere next week

issue has been fixed and will now look if rhel/centos 7 is used with pgsql 9.4 and point to /var/run/postgresql/
else /tmp will be used


Language doesn't change when i reset recaptcha

resetCaptcha(lang) {
    this.currentLanguage = lang;
    console.log(this.captcha);
    this.captcha.reset();
}

<re-captcha (captchaResponse)="handleCorrectCaptcha($event)" site_key="{{ captcha_key }}" language="{{ currentLanguage }}"

This captcha log good language but it's not refresh.
Thks ( sorry for my english :) )

There is currently no support in the ReCaptcha Library to change the language through javascript. The language identifier is passed in through the URL param and needs script reloading to refresh the language.
We could probably expose the captcha service which is currently set to be private but manipulating DOM this way seems to hacky.
@achimha What are your thoughts on this?

We need the ability to change the language of recaptcha too.
The next condition should be changed to check the variability of language.
https://yadi.sk/i/A1NK_Gxi3LdLn2
But the best way to achieve this goal is to define
@Input language: string
in ReCaptchaComponent to allow data binding to him.
Could you provide this solution in the closest time?

@User3290 That's not the problem. We already have the language input that works. Just not the way OP wants.

I have troubles with the language , is shown in spain and i need on english,  how i can change the language??

This is not currently supported in ReCaptcha library. Changing language means re-rendering the component and that is something that can be handled in application code easily.

Anyone can share some code on how they achieved this?

Check the constructor of the class you can send the language

@Filip1986 There is a way you can do it, atleast it makes logical sense. The idea is to re-render/re-initiate the component everytime you need to change language. @patroclo404 Changing the bound language param will not work. The captcha needs to be reinitialized to make the language change work.

@xmaestro: What would be the best way to re-render/re-initiate re-captcha component? I added language attibute and *ngIf="flag" to re-captcha and then I changed language and flag to false and then to true. Re-captcha was reloaded, but language was'nt changed. What could be possibly wrong?


Exception calling "GetBicepResources" with "3" argument(s): "Unable to expand resources because the source file

Github Actions workflows fail to expand and use bicep template with error:
Failed to expand bicep source '...\main.bicep'. Exception calling "GetBicepResources" with "3" argument(s): "Unable to expand resources because the source file '...\main.bicep' was not valid. An error occurred evaluating expression '[length(parameters('resourcegroupNames'))]' line 66. The parameter named 'resourcegroupNames' was not set or a defaultValue was defined."
Steps to reproduce the issue:

Expected behaviour

No error, expand the bicep file.

Module in use and version:

Module: PSRule.Rules.Azure
Version: 1.18.1

Captured output from $PSVersionTable:
Name                           Value
----                           -----
PSVersion                      7.2.[6]
PSEdition                      Core
GitCommitId                    [7].2.6
OS                             Linux 5.15.0-[10]17-azure #20~20.04.1-Ubuntu SMP …
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Additional context

Github Actions workflows fail to expand and use bicep template with error: Failed to expand bicep source '...\main.bicep'. Exception calling "GetBicepResources" with "3" argument(s): "Unable to expand resources because the source file '...\main.bicep' was not valid. An error occurred evaluating expression '[length(parameters('resourcegroupNames'))]' line 66. The parameter named 'resourcegroupNames' was not set or a defaultValue was defined."
Steps to reproduce the issue:
Expected behaviour
No error, expand the bicep file.

Module in use and version:

Module: PSRule.Rules.Azure
Version: 1.18.1

Captured output from $PSVersionTable:
Name                           Value
----                           -----
PSVersion                      7.2.[6]
PSEdition                      Core
GitCommitId                    [7].2.6
OS                             Linux 5.15.0-[10]17-azure #20~20.04.1-Ubuntu SMP …
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Additional context

main.bicep:
param resourcegroupNames array
main.parma.bicep.json:
"resourcegroupNames": {
  "value": [
    {
      "name": "acr",
      "environmentType": "prod"
    },
    {
      "name": "kv",
      "environmentType": "prod"
    }
  ]
}

@oxbo-andre Thanks for reporting the issue.
Expansion requires a parameter file or other bicep deployment referencing the module to provide required parameters.
If you have a parameter file as the post from @aavdberg may suggest then try this:

Double check that the Bicep module main.bicep is excluded. You can do this by setting the Input.PathIgnore option. This is because expansion occurs from the parameter file or other bicep file, so the referenced module needs to be excluded.

Details here: https://azure.github.io/PSRule.Rules.Azure/using-bicep/#configuringpathexclusions

Alternatively, you may consider setting default values (via AZURE_PARAMETER_DEFAULTS) for testing, but that is mostly is designed around common parameters provided by a deployment pipeline. In your case this doesn't seem applicable.

See: https://azure.github.io/PSRule.Rules.Azure/setup/configuring-expansion/#requiredparameterdefaults

Things to note, PSRule will automatically find the .bicep file from the parameter file only if:

Metadata link - An association between parameter file to bicep file via metadata in the parameter file. See https://azure.github.io/PSRule.Rules.Azure/using-templates/#linkingtemplates
Naming convention: This currently doesn't work for Bicep files. That is a bug I found while double checking your scenario. Currently the naming convention is not configurable it has to be <deployment>.parameters.json.

You can also enforce the metadata link be setting the AZURE_PARAMETER_FILE_METADATA_LINK option, to ensure parameter files are not left unaccounted for.
https://azure.github.io/PSRule.Rules.Azure/setup/configuring-expansion/#requiretemplatemetadatalink

I hope that helps. If not, then let me know and we might need to debug it further by collecting come additional information.

Hi @BernieWhite i opened the issue with my working account and commented with my own account.
Linking the parameter file to the template file is not mentioned in the documentatie for Bicep.
my main bicep file is: main.bicep
and my parameter file is main.params.bicep.json.

@aavdberg Thanks for the feedback.
Here the links:

https://azure.github.io/PSRule.Rules.Azure/using-bicep/#usingparameterfiles
https://azure.github.io/PSRule.Rules.Azure/using-bicep/#configuringpathexclusions

But based on your feedback we still have work to do to make it clearer :) thanks.

This is my ps.rule.yaml file:
include:
module:
- PSRule.Rules.Azure
output:
culture:
- "en-US"
input:
pathIgnore:
# Ignore other files in the repository.
- ".vscode/"
- ".github/"
- ".md"
- ".yaml"
- "bicepconfig.json"
- "*.azcli"
Exclude modules but not tests.
#- 'bicep/modules//*.bicep'
#- '!bicep/modules//*.tests.bicep'
execution:
notProcessedWarning: false
configuration:
Enable automatic expansion of Azure parameter files.
AZURE_PARAMETER_FILE_EXPANSION: true
Enable automatic expansion of Azure Bicep source files.
AZURE_BICEP_FILE_EXPANSION: true
Configures the number of seconds to wait for build Bicep files.
AZURE_BICEP_FILE_EXPANSION_TIMEOUT: 20

@aavdberg Nice! Looks mostly good just uncomment:   #- 'bicep/modules/**/*.bicep' and update the path where your Bicep files are located if they are not in the bicep/modules/ directory within your repository.
To metadata link your parameter file it might look something like this if the file main.bicep is in the same directory as the parameter file.
{
    "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentParameters.json#",
    "contentVersion": "<IP_ADDRESS>",
    "metadata": {
        "template": "./main.bicep"
    },
    "parameters": {
        "resourcegroupNames": {
            "value": [
              {
                "name": "acr",
                "environmentType": "prod"
              },
              {
                "name": "kv",
                "environmentType": "prod"
              }
            ]
          }
    }
}

The warning in the parameter file schema is not strictly correct. metadata can be defined almost any property without impact.

With modules you mean that i use in my main.bicep file? The modules are in an Azure Container Registry.

Looks better now 🚀

`
include:
module:
- PSRule.Rules.Azure
output:
culture:
- "en-US"
input:
pathIgnore:
# Ignore other files in the repository.
- ".vscode/"
- ".github/"
- ".md"
- ".yaml"
- "bicepconfig.json"
- "*.azcli"
Exclude modules but not tests.
- 'bicep/modules/**/*.bicep'
- '!bicep/modules/**/*.tests.bicep'
execution:
notProcessedWarning: false
configuration:
Enable automatic expansion of Azure parameter files.
AZURE_PARAMETER_FILE_EXPANSION: true
Enable automatic expansion of Azure Bicep source files.
AZURE_BICEP_FILE_EXPANSION: false
Configures the number of seconds to wait for build Bicep files.
AZURE_BICEP_FILE_EXPANSION_TIMEOUT: 20
`

@aavdberg You can use Bicep code with PSRule in the following ways:

Bicep deployment (i.e. deploy.bicep) with no required parameters -> Bicep module (i.e. main.bicep) stored in Azure Container Registry or via a local relative path.
JSON parameter file -> Bicep module (i.e. main.bicep) stored in Azure Container Registry or via a local relative path.

Also you can also: Bicep deployment/ parameter file -> Bicep module -> Bicep module ....
If you are using the PSRule integration with GitHub Actions or Azure Pipelines then see the following to make sure that the context PSRule runs under has an authenticated connection to be able to complete the restore.
https://azure.github.io/PSRule.Rules.Azure/using-bicep/#restoringmodulesfromaprivateregistry

Looks better now 🚀

@aavdberg Nice!

Thanks @BernieWhite
jobs:
  validate:
    name: Analyze templates
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v3

      # Display PowerShell version
      - name: PowerShell version
        run: |
         $PSVersionTable
        shell: pwsh

      # Login to Azure for access to Azure Container Registry
      - name: OIDC Login to Azure Public Cloud with AzPowershell (enableAzPSSession true)
        uses: azure/login@v1
        with:
          client-id: ${{ secrets.AZURE_CLIENT_ID }}
          tenant-id: ${{ secrets.AZURE_TENANT_ID }}
          subscription-id: ${{ secrets.AZURE_IDENTITY_SUBSCRIPTION_ID }}
          enable-AzPSSession: true

      # Analyze Azure resources using PSRule for Azure
      - name: Analyze Azure template files
        uses<EMAIL_ADDRESS>        with:
          modules: "PSRule.Rules.Azure"

@aavdberg Are you fine for me to convert this to a discussion if these is no longer a bug so that this can help others?
I'll log a separate issue for the bicep naming convention issue.


Improvements for "XMLPort Export Generic CSV"

Hey together,
hope this is the right place to post this issue here ("For collaboration on system and base applications as well as event requests, use ALAppExtensions.") (https://github.com/Microsoft/AL).  Otherwise, can you tell me what would be the right way?
Goal
The above-mentioned XMLPort is good for using it in combination with the Data Exchange Framework. We simply want to export a csv file that is defined by the Data Exchange Definition.
Problem

The XMLPort does not consider the set up "Column Separator" by the Data Exchange Definition. 2. Moreover, it does consider the GetValue() method from the Table Data Exchange Field.

Just compare the XMLPort to the XMLPort "Data Exch. Import - CSV" that is the pendant to it.
At the moment I created a copy of this XMLPort, but would like to have it in the Base Application itself.
Greetings
Andreas

Yes, please.
We also need to be able to change/override the FieldDelimiter, initially just via hard-coding all exports should suffice for the current client, but ideally by adding this to the Data Exch Def and getting it from there. I will be opening a new request for that event.

Hi!
This rather old case just came flying to my desk, after having been touring around for more than a year :-O The problem is, that this isn't really an extensibility request/defect, but rather a small feature request (normally to be logged on aka.ms/bcideas). It hence doesn't really qualify to be processed as a GitHub issue. However, @AndreasGloeckner, you mention you have a working version of that XMLPort already, but would like it to be part of the Base Application. Would you like to contribute your code to the BaseApp? We don't really have any process for that right now, but maybe we could work something out ad-hoc for this case. Code contribution is part of this repository, and in that case, I could see what we can do :-)

Hi Jesper,
I would really like to contribute but I don't have the object anymore. I think it was more or less a copy of the "import generic csv".
Sorry, for that.

No worries :-) I'll see what I can do!

Received confirmation that this should get logged as an idea on aka.ms/bcideas. Reason again being, that this is not an extensibility request or related to extensibility, but that it appears to be a request for a tiny feature. If you would provide the object, I would happily put it into the BaseApp, but we cannot process this request as a GitHub issue (prioritization and all, you know...). Closing this issue and hoping to see this reappear as an idea :-)


 Tree is getting formed only once.
My Config
$scope.treeConfig = {     multiple: false,     animation: true,     core : {         error : function(error) {         },         check_callback : true,     worker : true,     types : {     default : {         icon : 'fa fa-folder'     },     subsystem : {         icon : 'fa fa-wpforms'     },     cloud : {         icon : 'glyphicon glyphicon-cloud'     },     version : 1     },     plugins : ['types']     } };

Issue is resolved by putting worker as false


[horreum-api] Cannot create extractor with the accessor if it previously have been deleted

io.hyperhoil.tools:horreum-api:0.3-SNAPSHOT

Go to Schema->Schema extractors
Create extractor with accessor "my_extractor" and any valid json path. Press save
Remove newly created extractor by pressing Delete and press Save
Create extractor with accessor "my_extractor" and any valid json path. Press save
Expected: new accessor is created with name "my_extractor"
Actual: "Warning alert:Failed to save the schema
There is an existing extractor with accessor 'newExtractor'. This extractor was deleted but it might be still in use. Please use a different accessor."

That's intended behaviour. Remove all usages of the deleted extractor and wait 10 minutes; the deleted extractor should be garbage-collected.
I can see that with a completely deleted extractor (not just deprecated) you have few options to find the usages; that would deserve a UI fix. Though, executors are probably going to be replaced with Labels (#122)

Outdated, closing.


Flexget doesn't read Trakt auth in dev enviroment with virtualenv github install

Expected behaviour:
Use the trakt key and fetch data from trakt
Actual behaviour:
Trakt lists as authed, and lists a token but when running a task flexget thinks it isn't authed.
Log:
C:\Users\living\Documents\GitHub\Flexget>scripts\flexget.exe trakt list

┌────────────┬────────────┬────────────┐
│ Account    │ Created    │ Expires    │
├────────────┼────────────┼────────────┤
│ yolohuset │ 2016-08-28 │ 2016-11-26 │
└────────────┴────────────┴────────────┘

C:\Users\living\Documents\GitHub\Flexget>scripts\flexget.exe -c config\config.yml execute --tasks *movies-discover
2016-08-28 11:29 VERBOSE  task_queue                    There are 1 tasks to execute. Shutdown will commence when they have completed.
2016-08-28 11:29 WARNING  discover      download-movies-discover Input movie_list did not return anything
2016-08-28 11:29 VERBOSE  discover      download-movies-discover Discovering 0 titles ...
2016-08-28 11:29 VERBOSE  details       download-movies-discover Task didn't produce any entries.
2016-08-28 11:29 VERBOSE  details       download-movies-discover Summary - Accepted: 0 (Rejected: 0 Undecided: 0 Failed: 0)
2016-08-28 11:29 CRITICAL plugin        download-movies-discover Account yolohuset has not been authorized. See `flexget trakt auth -h` on how to.
2016-08-28 11:29 WARNING  task          download-movies-discover Aborting task (plugin: list_remove)

Steps to reproduce:

Step 1: Install github version of trakt
Step 2: Auth trakt
Step 3: Run any config with action reliant on trakt connection

Additional information:
The same identical config file succeeds on my pip install of flexget 2.3.11

Flexget Version: 2.3.12.dev
Python Version: 2.7.10
Installation method: Github
OS and version: windows 10
Link to crash log:

Works for me. You sure scripts\flexget.exe -c config\config.yml trakt list also lists your auth? Your auth is tied to a local sqlite file and it is placed in the same directory as your config.yml

Ok, no it doesn't. I'm guessing the roblem is since I didn't secify that config file location, which isn't documented in the trakt auth information and should be clarified.
Solved this by secifying the -c config tag when running the auth command.
Suggested fix: Proper documentation should be added regarding custom config files.

It's not really something specific to trakt cli. Proper cli documentation is on our todo.


I can't install from a sub-directory

mops add https://github.com/vporton/join-proxy-rust/motoko#49c2e31431deb5455c33160a2ddbf9b1e292ef08

adds not just the sub-directory motoko but the entire join-proxy-rust package, what is wrong.

Installing sub-directory from GitHub is not supported

Installing sub-directory from GitHub is not supported

This feature request is asking exactly to support it.


Allow setting visually hidden text for change links in summary lists

Typically, the ‘Change’ link in a summary list is composed of 2 parts: the visible link (usually labelled ‘Change’), and a visually hidden part, which helps tell change links apart when a screen reader announces them.
Currently, the action param for a row in GovukComponent::SummaryList is pretty dumb; it’ll take anything. However, given this link pattern is so intrinsic to this component, it’d be much more helpful if this component enabled setting these values.
How about, in addition to taking any value as it does currently:
<%= component.slot(
    :row,
    key: 'Name',
    value: 'Aardvark',
    action: link_to('Change', '#aardvark')
) %>

you can supply an object with the following values instead:
<%= component.slot(
    :row,
    key: 'Name',
    value: 'Aardvark',
    action: {
        href: "#aardvark",             # Required
        text: "Change",                # Required, defaults to ‘Change’ if not provided
        visually_hidden_text: "name",  # Required, but maybe defaults to humanised `key` value?
        classes: "",                   # Optional
        html_attributes: {},           # Optional
    }
) %>

Something like that.

Excellent suggestion, totally on board with it.


Jupyter Lab Won't launch due to Javascript error

Description

Upon opening JupyterLab I get an error message that says the following...
"A JavaScript error occurred in the main process
Uncaught Exception:
SyntaxError: Unexpected token '
Reproduce

Open JupyterLab
See Error

Expected behavior
I expected JupyterLab to load like it did yesterday
Context

Operating System and version:  --> Windows 11
JupyterLab-Desktop version:  4.0.2-1
Installer:  .exe

Troubleshoot Output

Paste the output from running `jupyter troubleshoot` from the terminal inside of JupyterLab Desktop here.
You may want to sanitize the paths in the output.

Browser Output (required for interface issues after loading)

Paste the output from the DevTools JavaScript console here, if applicable.

Logs (required for issues with loading, server and startup)

Paste the logs from the `main.log` file here (optionally also from `renderer.log`).
There are no logs from today to post

can you please provide logs? you check the doc here.

closing since no response for long


Setting pyright extraPaths does not work

Hi,
i followed your example and tried to set the extraPaths variable [1]. Unfortunately pyright seams to not pick up this path aka completions does not work. But if i set it via pyproject.toml [2] the completion does work.
[1]
local lsp = require('lsp-zero')

lsp.use('pyright', {
  settings = {
    python = {
      analysis = {
        extraPaths = {'/path/to/my/dependencies/src'},
      }
    }
  }
})

[2]
❯ cat pyproject.toml
[tool.pyright]
extraPaths = ["/path/to/my/dependencies/src", ]

There was problem with the setup in lsp-zero. Hopefully the update can fix your issue.

I updated but the error persists. is there a way to get debug messages? Also is it possible to set the extraPaths on runtime?

I've added a new function to inspect the settings of a language server.
Try this command in a python file after the server is attached to a buffer.
:lua require('lsp-zero.check').inspect_settings('pyright')

To check the logs use the command :LspLog


Updating links, and removing useless, empty links

@lfryc looks good ?

Testing

:+1: works & seems fine, merging...

Landed as https://github.com/aerogear/aerogear-unifiedpush-server/commit/069d95b3ab59d0d5ba1d54a1c498f0f53d94dd7b


chore(main): release iso-base 1.0.0

:robot: I have created a release beep boop
1.0.0 (2023-06-22)
⚠ BREAKING CHANGES

base64 no longer matches node (padding)

Features

add, msg, sig, rpc (f574e7b)
base64 no longer matches node (padding) (179ab3d)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/hugomrdias/iso-repo/releases/tag/iso-base-v1.0.0 :sunflower:


Ability to restart a tunnel from the sidebar

Feature description
Right now we can only "disconnect" the tunnel from the sidebar.
The ability to restart it will prevent to disconnect it and then searching the tunnel again to start it...

Now available inside the latest beta release : https://testflight.apple.com/join/QSrBK59z


Massive lag in VAB after loading ship with nerv

KSP.log.zip
Steps to reproduce:

Install KSP <IP_ADDRESS>17 (LinuxPlayer) en-us
Install CKAN v1.27.2
Use CKAN to install PersistentThrust v<IP_ADDRESS>
Launch KSP and start a new sandbox save
Enter VAB
Create a new ship with mk1 command pod, mk1 liquid fuel fuselage, and a nerv
Save the ship
Leave VAB
Enter VAB (which autoloads the saved ship) -> Experience extreme lag while scrolling and clicking around

Workaround:
Remove the nerv engine by dragging to the left.  The lag goes away.  Adding the nerv again and repeating 7, 8 and 9 result in the lag returning.
Discussion:
The attached log contains multiple "ArgumentNullException: Value cannot be null" exceptions with PersistentThrust in the stack.  These exceptions start when the ship is loaded (step 9), and end when I remove the nerv engine (the workaround).  The first exception follows.
[EXC 14:28:58.680] ArgumentNullException: Value cannot be null.
Parameter name: key
    System.Collections.Generic.Dictionary`2[TKey,TValue].FindEntry (TKey key) (at <ad04dee02e7e4a85a1299c7ee81c79f6>:0)
    System.Collections.Generic.Dictionary`2[TKey,TValue].TryGetValue (TKey key, TValue& value) (at <ad04dee02e7e4a85a1299c7ee81c79f6>:0)
    EffectList.Event (System.String eventName, System.Single power, System.Int32 transformIdx) (at <7b275c7511b34ed2b695641a5b2263f8>:0)
    Part.Effect (System.String effectName, System.Single effectPower, System.Int32 transformIdx) (at <7b275c7511b34ed2b695641a5b2263f8>:0)
    PersistentThrust.PersistentEngine.ApplyEffect (System.String effect, System.Single ratio) (at <dc55571d52ca4cd5ac3eebaaba4d4e6a>:0)
    PersistentThrust.PersistentEngine.UpdateFX () (at <dc55571d52ca4cd5ac3eebaaba4d4e6a>:0)
    PersistentThrust.PersistentEngine.Update () (at <dc55571d52ca4cd5ac3eebaaba4d4e6a>:0)
    UnityEngine.DebugLogHandler:LogException(Exception, Object)
    ModuleManager.UnityLogHandle.InterceptLogHandler:LogException(Exception, Object)
    UnityEngine.Debug:CallOverridenDebugHandler(Exception, Object)

I could not reproduce the problem but I think I have identified the problem in the ApplyEffect  which didn't correctly verify the effect string

This is fixed in v1.7.2.  Thanks!


Update alert word UI to match the style of bots.

Resolves #1075.

Automated message from Dropbox CLA bot
@phansen01, it looks like you've already signed the Dropbox CLA. Thanks!

Added casper tests for adding/deleting a word alert.

Cool, tests look pretty reasonable.  I'll give the actual code a review soon.  Thanks @phansen01!

Awesome, thanks for working on this @phansen01!  This looks pretty good; I posted one issue above.  Additionally, it looks like this page was already somewhat confused about whether the feature is called "alert words" or "word alerts", and it seems you've increased the fraction of the code that thinks this is called "word alerts".  I think we should settle on one name for the feature, and use it consistently...

Yeah I think after staring at it for a while my brain became numb to the
distinction. I think alert words sounds a little better?
On Thu, Jun 23, 2016, 5:36 PM Tim Abbott<EMAIL_ADDRESS>wrote:

Awesome, thanks for working on this @phansen01
https://github.com/phansen01! This looks pretty good; I posted one
issue above. Additionally, it looks like this page was already somewhat
confused about whether the feature is called "alert words" or "word
alerts", and it seems you've increased the fraction of the code that thinks
this is called "word alerts". I think we should settle on one name for the
feature, and use it consistently...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/pull/1097#issuecomment-228206505, or mute
the thread
https://github.com/notifications/unsubscribe/AL7-qfKP-B8jK4voZTqB3F3P3Uv7O1-pks5qOwqAgaJpZM4I8J9e
.

Yeah I surveyed with a few people, and people seem to agree that "Alert Words" is more intuitive, so let's stick with that (Both in the UI and in the code).
git grep -i word.alert may be helpful in checking whether you've found them all.
Let me know when you have a new version ready, @phansen01!

Okay, looks like I've got them all. One thing I noticed just recently was that if (for some reason) a user enters an alert word that consists only of numbers, the UI for adding and deleting works fine, but the input is interpreted as an int, and the api for adding an alert word returns an error.
Should we just treat integer inputs as strings?

Yes, we should treat integer inputs as strings.

I'll make that change and update shortly, then.

Let me know if there's anything else I need to do here 🙂

Merged, after a couple small tweaks to the look:
diff --git a/static/templates/alert_word_settings_item.handlebars b/static/templates/alert_word_settings_item.handlebars
index 978dbc5..1748b7b 100644
--- a/static/templates/alert_word_settings_item.handlebars
+++ b/static/templates/alert_word_settings_item.handlebars
@@ -12,7 +12,7 @@
                                 maxlength=100 placeholder="{{t 'Alert Word' }}" value="" />

                         <button class="btn btn-primary add-alert-word" id="create_alert_word_button" type="button">
-                            {{t 'Create Alert Word'}}
+                            {{t 'Add Alert Word'}}
                         </button>

diff --git a/static/templates/settings_tab.handlebars b/static/templates/settings_tab.handlebars
index 0a54bac..770e03a 100644
--- a/static/templates/settings_tab.handlebars
+++ b/static/templates/settings_tab.handlebars
@@ -358,7 +358,6 @@

     {{/tr}}

-

There are a few follow-up things that this has suggested to me:

I think it might look cleaner if the table rows weren't so wide and/or we had headings like we do on the /#administration emoji table.  But this is a problem a lot of our settings things have; we should probably think about a solution here in general across the similar UI elements.
While playing with it for comparison, I noticed that our realm_emoji form on /#administration doesn't properly handle errors (opened https://github.com/zulip/zulip/issues/1116 for this)
our settings/administration related CSS now has a lot of sections like this, where there's a long list of features that are tagged as having a particular styling:

#settings .add-new-bot-box,
#alert_words_list .add-new-alert-word-box,
#alert_words_list .alert-word-information-box,
#emoji-settings .add-new-emoji-box,
#admin-default-streams-list .add-new-default-stream-box{
    background: #e3e3e3;
    padding: 10px;
    margin-left: 38px;
    margin-right: 38px;
    margin-bottom: 15px;
    box-shadow: 0px 0px 2px rgba(0,0,0,0.5);
    position: relative;
    font-size: 14px;
}

I think it could be really good to convert these into a nice little "Zulip settings" CSS library (maybe in its own file.  Opened this as https://github.com/zulip/zulip/issues/1117.
Thanks for doing this cleanup work @phansen01!  It is very much appreciated :)

no problem!
I agree that separating some of these often-reused styles would make things a lot clearer than having to dig for them.


Add "next step" button with functionality

Currently, we have these action buttons:

start
stop
pause
resume

I received feedback from several engineers, that it might be beneficial to add a "Next step" button.
If the user presses the next step button, the code should be executed until the next step and then paused.
Therefore, they do not miss any particular step due to the speed of the animation.

hey, I would like to work on this issue. Please assign it to me.

Hey @Sapna127 thank you, it is assigned to you 👍

Hey @Sapna127, since there have been no updates for over a month, I will put this issue to unassigned again.
Please let me know if you still want to work on this one.

I'm closing the issue since there are no updates for a long time


Implement semver tags in CI/CD workflows

misc: refactor actions

Remove unused -dev workflows.
Rename files for better readability (respectfully)
Changes workflows so that: - Each PR will take the highest tag from git and increment a patch level, a runner id, and a -dev after it, so that no image can be the same. - To publish a new artifact, a new ~tag has to be pushed~ release must be created.

Closes #68

@fopina anything against these changes? I know it involves a lot changes that are hard to test but I tried summarizing the ideas in the PR description.

I think the initial workflows were a bit of a mess (sorry), I don't have anything against trying to improve them 👌

All right. Merging this one, replicating across the other projects (where it makes sense) over the course of the next weeks. Cheers both.


update manifests/deploy/openshift/helm-operator/metering.yaml to bypa…

…ss error check
having empty spec: results in validation error and the metering CRD was not applied.
error: error validating "/home/pruan/workspace/operator-metering/manifests/deploy/openshift/helm-operator/metering.yaml": error validating data: unknown object type "nil" in Metering.spec; if you choose to ignore these errors, turn validation off with --validate=false

superseded by #615


Website improvements

Add diagram to website instead of logo. etc.

Just a suggestion, rearrange concepts with getting started and change the position of CRUD config
{
  ...,
  "generation": {
    "create": true,
    "update": true,
    "findAll": true,
    "find": true,
    "delete": false,
    "subCreate": false,
    "subUpdate": false,
    "subDelete": false,
    "disableGen": false,
  }
}

To usage section. I just think it is part of setup and should be with the setup section or as close to setup as possible.

@MStokluska yes going to redesign the getting started workflow.

@MStokluska feel free to contribute to the documentation


HOSTEDCP-910: Enable pprof in hosted etcd

What this PR does / why we need it:
Enables pprof in hosted etcd, similar to self-managed or not-hypershift openshift deployments
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #HOSTEDCP-910
Checklist

[ ] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

what's the impact on resource consumption from having this enabled ootb?

what's the impact on resource consumption from having this enabled ootb?

Enabling Pprof does not have any performance impact as long as any of the pprof endpoints are not requested. Some small performance impact is expected when requesting for example a CPU profile for a given time range. But as we're not targeting any kind of continuous profiling on HCP's etcd, the performance impact will be null.
This option is already enabled in classic ROSA, ARO and self-managed deployments, and as far as I can tell it's not impacting the application performance.

@enxebre need a review here

/retest

/approve

/retest-required


chore: reduce max circuit size in bb binary

resolves #1941
Checklist:
Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge.

[ ] If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag.
[ ] I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code.
[ ] Every change is related to the PR description.
[ ] I have linked this pull request to relevant issues (if any exist).

This is included in a change on a branch that Charlie is working on, I cherry-picked this so we can get it in faster


Update existing Helidon lab on OCI

Environment Details

Helidon Version: 2.x
Helidon SE
JDK version: 11
OS: na
Docker version (if applicable):

Problem Description
Lab on OCI is from Helidon 1.x and needs to be updated.
Steps to reproduce
Existing Lab (in prod): https://docs.oracle.com/en/solutions/develop-microservice-java-app/develop-application1.html#GUID-618F068E-3BF2-48B2-ABD2-78BE2A65F00A
Updated Lab to review (in UAT): https://docs-uat.us.oracle.com/en/solutions/develop-microservice-java-app/develop-application1.html#GUID-5B75A27F-D0B1-4CB3-BBCE-D67AABFAFACC

This will not have a PR since the files are outside of Git. I've asked @dalexandrov to review the content with me using the UAT link above.

Create an Oracle Review site to discuss updates https://review.us.oracle.com/review2/Review.html#reviewId=449987;scope=document;status=open,fixed

Just one minor fix, looks good now! Thank you!!!

Here is the build in UAT with all requested changes. https://docs-uat.us.oracle.com/en/solutions/develop-microservice-java-app/develop-application1.html#GUID-618F068E-3BF2-48B2-ABD2-78BE2A65F00A

So long as the Lab steps work as intended, I'm happy. Go ahead and publish if you haven't already.


Addition: Support using a TOML "dotfile" file to configure the tool's options

Hello @Kannav02
It would be beneficial to add support for configuring dialectMorph using .toml configuration files.
This way, the users will be able to specify their preferred options all in one place (.toml file stored in their home directory) instead of having to pass them as command-line arguments.
The contents of the config file can be parsed using toml library, and then merged with the provided command-line options (the passed arguments will take precedence overriding the default config options).
I would love to work on this issue!

Merged #25 into the main branch, thank you for the help!


sha1 hash error on Modrinth (November 5th 2023)

Same issue as the previous ones, Sha 1 hash error.

The issue still happens on Modrinth.

Posted remake v2 of modpack


Demo: Merge into this repo

Merge the working demo into the repo for demo/testing purposes.
A bit heavy, since it's a Cordova app and the lib itself is like 2 files... May be better to include the HTML/JS files and instructions to "cordova create" a demo.

Commit 9832cba8a785d074560be5a47e5422f4e4107002 adds the demo app.
Issue #4 exists for polishing up thi demo, which does work but which is not awesome.


CORE-3510: Prevent Kryo Serializing Avro Classes

Reject Kryo serialization of avro generated classes by registering a
default serializer and add new non Avro generated FlowStackItem class.

Add unit test for FlowStackImpl.
Standardize 'FlowStack' interface methods so all behave similarly
when the stack is empty (most of them were returning null while
another one was throwing an exception).
Add 'toMutableMap' extension function to 'KeyValuePairList' class.
Create non Avro version of 'FlowStackItem' and centralize handling
logic regarding wich version to use (Avro or non Avro) within the
'FlowStackImpl' class.
Remove references to Avro generated 'FlowStackItem' class and use
the new non Avro 'FlowStackItem' instead.
Implement and register 'AvroRecordSerializer' as the default
serialiser for any class that implements the SpecificRecord inteface,
preventing the kryo serialisation of avro generated classes.

Apologies for the force push, had to rebase against the original branch due to some conflicts.

I've spoken to Dan about this offline just to ensure my understand is correct, but I will leave a comment here too for the record. I think this PR reflects the solution outlined in the ticket, but I wonder whether the ticket could have done with a little more refinement. We have a general premise where things outside the stack are held as Avro types, largely to aid serialization. Now we see this is getting in the way of, ironically, serialization. I think then we can't use the argument that Avro works for us because of serialization and implement special cases where we find it doesn't just to keep the first rule afloat.
Actually though if we look at the specific problem here, the IO request handlers need some information about the FlowStackItem. Right now that is limited to session ids (a list of strings). We don't need to expose non-Avro FlowStackItems to the entire system just to get those in the IO request messages, we can instead change what those messages contain. In the future we might load them with more information from the flow stack item, but this is localised to that one place at the point of use. With the dual serialization solution we are constantly having to keep both data structures in sync, whether the consumer of the information needs that extra stuff or not.

@simon-johnson-r3 : thanks for pointing out the flaw in the approach, it certainly makes sense, I'll go ahead and make the changes 👍 .

I take the blame for it since I told @jujoramos to do it that way 😢

@lankydan : no blame to take, was a good learning exercise 🍻 .


Circle CI: Run in Ubuntu and macOS environments, in parallel

This PR takes the current Circle CI configuration for the macOS test run and creates a copy for Ubuntu, makes all the necessary changes (some commands have to be replaced, a dependency that is always present on macOS has to be manually installed, the caches need better names) and then adds a "workflow" that makes both test, macOS and Ubuntu, run at the same time.

          1 Warning

:warning:
Pull Request is Work in Progress

  Generated by :no_entry_sign: Danger

Success on first try:

👍

Although Github UI still shows this:

Bug or did I miss something?

Found something, seems to be a limitation of Github/Circle integration:
https://circleci.com/docs/2.0/workflows/#workflows-waiting-for-status-in-github
We can change that when this PR is merged.

Wow 🚀

PR is now ready for review.

Looks like CI is green, but the overall status is missing or something?

This looks awesome, and I'm really excited to have Linux CI running also. Since this is a very Circle heavy PR, I'd love to get @DanToml's input on this before we merge. I know she's on vacation right now, so if it's cool from your side, let's wait for her to come back and get her input?

Absolutely, looking forward to it - this was my first contact with Circle CI.
Further work that benefits from Ubuntu or cross platform testing can just use this branch as base instead of master, so no harm done.

Oh, missed your second comment with the screenshot: Yes, that what I talked about in https://github.com/fastlane/fastlane/pull/11242#issuecomment-352146780 - you (or some other admin) would have to change a setting here in Github for this to be correct.

Yeah, seems like it's something we should wait for until after the merge

Yes, as we know how to solve it (and CircleCI is aware and has this explicit instructions) I don't think this will become a problem to solve, we just shouldn't forget to do it when we merged this.

Thanks for your help here @DanToml ❤

Update:

Waiting for review and merge of #11302
Dependency stuff that should be solved via a Docker container was "outsourced" to #11302

Besides that I hope I covered all comments.

#11302 is merged, I will incorporate this here now.

Rebased, and changes from #11304 adapted for the second half of the file. Circle CI now correctly collects the test reports of rspec and display them:

Your build ran 4541 tests in RSpec with 0 failures
Slowest test: fastlane_core.spec.build_watcher_spec FastlaneCore::BuildWatcher.wait_for_build_processing_to_be_complete returns specified build when strict watch is enabled (took 10.00 seconds).

I think this is now "good" again and all relevant, outstanding feedback was addressed. Can I get another look at it please?
(As said before, I think the dependency and Docker stuff will be handled in a follow up issue: #11302)

@KrauseFx On merge, this will also require changing the required status' for merging, as the old ci/circleci named status doesn't get reported in workflows (because we get a status per job)

Oh, I actually can't merge this before @KrauseFx changes that setting as this is one of the expected checks ;)
See here for instructions: https://circleci.com/docs/2.0/workflows/#workflows-waiting-for-status-in-github


Reset password link not working for the second time

Hello,
I sent a POST request to user_auth/password with email and redirect_url as parameters.
I get an email with a link to reset my password.
If I click on that link it redirects to the url which I sent in the above request and shows me my custom page to change password.
However, if I click on that reset password link in my email, again - it gives "Route not found error".
I have't changed my password. I just visit or click on the reset password link and it doesn't work for the second time.
What I observed is that the reset_password_token is getting changed to raw value instead of encrypted value.
I am using 0.1.37 version of the gem

I experienced the same issue too.

Any of you guys found a solution? I'm experiencing the same issue...

same problem with version 0.1.40

This is because token auth creates whole new session and deletes token which was used for it. This is major bug which would involve a lot of work. This is strictly binded with automatic login before even setting up a new password.

it's not a bug, it's a feature by design so every token is used only once and cannot get hijacked.
my problem was that I had overwritten user as_json method, making the json object to big to used by cookie

@rachuxx is right, the gem is designed not to use the reset password token to validate the password change, but to use to generate a new session where you are certainly logged in and, among other things, you can change your password. You can even navigate the site with this session.

Sounds like the current implementation is good for security so....closing for now.

Sounds like the current implementation is good for security so....closing for now.

@zachfeldman No, it is not.
There is a case that some mail clients visit the links in the mail content before user do. In this case the user always see the Routing error.
This situation is not good for security. Security issue must be solved with reset_password_period_valid?.

@BenjaminKim sorry to hear that you're so upset with the current implementation. This is a volunteer run project, so might I suggest that you submit a pull request if you feel so passionately?


Converts alpha diversity to serial

Per discussion with @EmbrietteH, the alpha diversity call is converted to a serial, rather than parallel alpha diversity. This removes the issue with parallel alpha diversity potentially failing silently.

:+1:


Feature request diagnostics blacklist

There are some 3rd party libraries in my project, that I want to be indexed, but don't want to see diagnostic messages

https://github.com/cquery-project/cquery/blob/master/src/config.h#L147
I am sad some of my notes in https://github.com/cquery-project/cquery/wiki/Initialization-options has been deleted.

So this option should be added to vs code extension? Should I open an issue there?

Yep


[Feature] Add dynamic OpenGL version support

Before submitting a new issue, please verify and check:

[x] The issue is specific to Raylib-cs and not raylib
[x] I checked there is no similar issue already reported
[x] My code has no errors or misuse of Raylib-cs

Issue description
I would love to see this feature in Raylib-cs, then we would have ANGLE support and it would possible to run a new or older version of OpenGL.
public static void SetOpenGLVersion(GlVersion version) {
      // Impliment it here
      // I would suggest doing a variable in the build that can be changed with code, so we can switch the Raylib.dll with the right OpenGL version.
}

Environment
OpenGL

Would require modifying raylib as currently opengl versions are selected with compile flags. Though we are considering custom builds in #118 so this could maybe be looked at with that.

Would require modifying raylib as currently opengl versions are selected with compile flags. Though we are considering custom builds in #118 so this could maybe be looked at with that.

so is that already planned?

Nothing specifically planned at the moment.

So would that possible easy?
I mean we could just switch the dll or not?

Some changes might only involve changing flags. However selecting the opengl version at runtime means modifying raylib to support this which means making a fork and keeping it in sync with raylib.

well, doing just 5 .dll files would work fine too or? every with a diffrent GL version.

but does they get casted then to int?

That would not work because there are 6 different opengl configs for raylib. With 5 platform folders in runtimes that would be 30 different libraries greatly increasing the size of the nuget package and complexity of library loading. At that point it is better for users to build a custom version of raylib instead.

That would not work because there are 6 different opengl configs for raylib. With 5 platform folders in runtimes that would be 30 different libraries greatly increasing the size of the nuget package and complexity of library loading. At that point it is better for users to build a custom version of raylib instead.

oh well

That would not work because there are 6 different opengl configs for raylib. With 5 platform folders in runtimes that would be 30 different libraries greatly increasing the size of the nuget package and complexity of library loading. At that point it is better for users to build a custom version of raylib instead.

Is it hard todo? Because i has no idea how to do that 😅

So... How you want ro implement it?

@MrScautHD Have a look at the SQLite, SQLitePCL, & SQLitePCLRaw Nuget Packages

sad :/

so it is not comming with the Build system?

The local build is to try and make it easier to customize raylib but by default won't include the change to set it at runtime as it is requires a lot of changes to rlgl. The local build should allow changing it at compile time though.

No, Just the example above was the first idea it should just possible to set it before running it

But fine the pull reuquest is still open so it is coming :)
Btw when next nuget update 😅


Relative paths are not correct

Hi,
I have tried out the tool but after requiring AmericanBritishSpellings.php in my code I'm getting fatal errors that AmericanBritishSpellings_Words.php failed to open.
On closer inspection it looks as if the relative paths are incorrect for AmericanBritishSpellings_Words.php and AmericanBritishWords_[LETTER].php.
Everything works fine if I replace the relative path ../classes/Language with __DIR__
Many thanks in advance!
Ross

Hi, Ross!
Thanks for the heads up, I'll take a closer look at this tonight.  Relative paths are indeed just that, relative, so it might be something that requires a code change or just improved documentation.
This project is actually an isolation of the relevant code for the GG-CMS, you can see the AmericanBritishSpellings***.php files located here for it: https://github.com/HoldOffHunger/GreenGluonCMS/tree/master/classes/Language .  It's possible that you're not using a .../classes/... dir, but, maybe that should be addressed in documentation or code.
Thanks again, I'll know more when I look closer tonight.

Thanks for your quick response, I appreciate you looking into this.
Just a thought:
Are you perhaps setting the include path from the webserver? The path in my case is relative to the file itself (AmericanBritishSpellings.php) so it's attempting to load the file using the following path:
/[PROJECT_ROOT]/libraries/convert-british-to-american-spellings/classes/Language/../classes/Language/AmericanBritishSpellings_Words.php
The code is stored in a directory specifically for 3rd-party libraries and is being required using an absolute path.
Many thanks!
Ross

Hey, again, Ross:
I spent some time thinking about this.  TLDR: Your idea of using __DIR__ . was a pretty good suggestion, and I've implemented it into this project.
The DR part: The project that I isolated this from, well, it's imperfect.  I have the dir setup as so...

/apache-hosted-domain-name.com/
/classes/Languages/mybritishamericancode.php
/classes/Languages/etc

So, when a user hit the .htaccess and redirecting index.php of apache-hosted-domain-name.com, to include the british-american code, I needed to step up one directory, hence ../, and then into the right directory, so, /classes/Languages/..., therefore require('../classes/Languages/...*...), etc..
But your method is perfect when it comes to making this project 100%-working-out-of-the-box, especially given how our code samples have worked.
I just pushed improvements to use __DIR__ in the two PHP files, AmericanBritishSpellings.php and AmericanBritishSpellings_Words.php.  I tested things relatively thoroughly, let me know if I missed anything, thanks!
Will give this ticket some time before closing it, if all is good.

Hi,
Thanks again for spending time on this. I'm still having trouble with the filepaths and I'm a bit confused what the differences are between our setups.
Judging from the documentation, __DIR__ is the absolute path of the current file's directory (in this example the AmericanBritishSpellings.php file). This is the way it's working at my end and this is the error:
Failed opening required '/apache-hosted-domain-name.com/web/libraries/convert-british-to-american-spellings/classes/Language/classes/Language/AmericanBritishSpellings_Words.php'
From the new commit it looks as if __DIR__ is the absolute path of the "project root" directory.
In my project I'm using Drupal so I created some test code to check the framework wasn't doing anything I wasn't aware of.
I created the following file structure:
/sites/apache-hosted-domain-name.com/convert-british-to-american-spellings/.. /sites/apache-hosted-domain-name.com/index.php
This is the contents of index.php:
`<?php
require_once DIR . '/convert-british-to-american-spellings/classes/Language/AmericanBritishSpellings.php';
$string = 'travelling';
$spellings = new AmericanBritishSpellings([]);
echo 'Spelling testOriginal' . $string . 'US spelling';
echo $spellings->SwapBritishSpellingsForAmericanSpellings(['text' => $string]) . '';`
Unfornately I still couldn't get the code to work, I have created a fork and this commit fixes the problems I'm facing:
https://github.com/nameless-ross/convert-british-to-american-spellings/commit/cb84b316db16eb9fc84f80213de9587e115bb4c1
As I said, I'm a bit confused and not sure what I'm doing differently. If the code is working fine then I'll happily continue to use the fork.
Many thanks!
Ross

Oh, I looked at your forked project.  I suspect maybe you moved the code in '/classes/Language/*' to just the base directory '/', and that's why your require(__DIR__ . 'AmericanBritish.php) is working fine.
Hrm, your code without those extra directories is a bit cleaner -- I'll probably fix up this project, too, later tonight, to have the same dir structuring.
Also (NOTE TO SELF), I should probably add some installation instructions to README.MD once the above is all said and done.
Thanks again for testing, nameless-ross.  That's usually above and beyond the call of duty, so I appreciate it!

Hi, again, Ross,
Sorry for the delay, been busy, etc..  (have added 2,000 to 3,000 new words to the codebase per newest release)
I've moved all of the code into the lib dir, like an ordinary db.  Hopefully, this is a bit better in being clearer about its operation.  Let me know if you think this is enough, thanks.
https://github.com/HoldOffHunger/convert-british-to-american-spellings/tree/master/lib

Hi there,
Sorry about my late response! The new structure is an improvement however, I still have problems getting this to work out of the box.
The issue for me are the paths specified on these lines:
https://github.com/HoldOffHunger/convert-british-to-american-spellings/blob/master/lib/AmericanBritishSpellings.php#L19
https://github.com/HoldOffHunger/convert-british-to-american-spellings/blob/master/lib/AmericanBritishSpellings_Words.php#L59
In order to fix the issue I have to change the paths to a path relative to the file itself and not the project structure. The above lines in my fork have been changed to:
// AmericanBritishSpellings.php
require(__DIR__ . '/AmericanBritishSpellings_Words.php');
// AmericanBritishSpellings_Words.php
$word_directory = __DIR__ . '/Words/AmericanBritish/';

Would you mind testing those realtive paths at your end?
Many thanks,
Ross

Ah, you are right, sorry about that.
https://github.com/HoldOffHunger/convert-british-to-american-spellings/blob/master/lib/AmericanBritishSpellings.php
https://github.com/HoldOffHunger/convert-british-to-american-spellings/blob/master/lib/AmericanBritishSpellings_Words.php
Lines have been updated:
require(__DIR__ . '/AmericanBritishSpellings_Words.php');

$word_directory = DIR . '/Words/AmericanBritish/';
Thanks for being patient with this!

Hi,
Sorry, once last change needed and then I think we are there!
It needs to be require_once() rather than require() on both of those lines. If the code is used more than once it attempts to redeclare those classes and I get the error:
Got error 'PHP message: PHP Fatal error:  Cannot declare class AmericanBritishWords_A, because the name is already in use
Many thanks!
Ross

Hey hey,
Ah, good point!  Updates have been added, for both the requires in AmericanBritishSpellings.php and AmercianBritishSpellings_Words.php.
Let me know if all is good.

Hi,
Updates work great, I've created a pull request with a couple of fixes for new bugs introduced.
Many thanks,
Ross

Perfect, I love features!  Thanks for your patience and contributions!


Added "fall" as a season & normalized "week of month" patterns

Added "fall" as a season & normalized "week of month" patterns

added fall as a season and added post-processing method for season homonyms (fall/spring)
normalized "week of month" patterns into date ranges (e.g., first week of May, last two weeks of June, etc.)

FYI, @alicekwak.
[info] *** 5 TESTS FAILED ***
[error] Failed tests:
[error] 	org.clulab.numeric.TestNumericEntityRecognition
[error] 	org.clulab.numeric.TestEvalTimeNorm
[error] 	org.clulab.utils.TestDependencyUtils
[error] 	org.clulab.odin.TestMention

[info] - should recognize date ranges from seasons *** FAILED ***
[info]   "[O]" was not equal to "[B-DATE-RANGE]" (TestNumericEntityRecognition.scala:621)
[info]   Analysis:
[info]   "[O]" -> "[B-DATE-RANGE]"

[info] - should recognize date ranges with seasons *** FAILED ***
[info]   "[O]" was not equal to "[B-DATE-RANGE]" (TestNumericEntityRecognition.scala:621)
[info]   Analysis:
[info]   "[O]" -> "[B-DATE-RANGE]"

[info] - should not degrade in performance *** FAILED ***
[info]   0.8487485647201538 was not greater than or equal to 0.85 (TestEvalTimeNorm.scala:15)

[info] - should produce one head using findHeads *** FAILED ***
[info]   Some(0) was not equal to None (TestDependencyUtils.scala:75)
[info]   Analysis:
[info]   Some(x: 0 -> )

[info] - should get None when there are no roots *** FAILED ***
[info]   Some(0) was not equal to None (TestMention.scala:59)

FYI, @alicekwak.
[info] *** 5 TESTS FAILED ***
[error] Failed tests:
[error] 	org.clulab.numeric.TestNumericEntityRecognition
[error] 	org.clulab.numeric.TestEvalTimeNorm
[error] 	org.clulab.utils.TestDependencyUtils
[error] 	org.clulab.odin.TestMention

[info] - should recognize date ranges from seasons *** FAILED ***
[info]   "[O]" was not equal to "[B-DATE-RANGE]" (TestNumericEntityRecognition.scala:621)
[info]   Analysis:
[info]   "[O]" -> "[B-DATE-RANGE]"

[info] - should recognize date ranges with seasons *** FAILED ***
[info]   "[O]" was not equal to "[B-DATE-RANGE]" (TestNumericEntityRecognition.scala:621)
[info]   Analysis:
[info]   "[O]" -> "[B-DATE-RANGE]"

[info] - should not degrade in performance *** FAILED ***
[info]   0.8487485647201538 was not greater than or equal to 0.85 (TestEvalTimeNorm.scala:15)

[info] - should produce one head using findHeads *** FAILED ***
[info]   Some(0) was not equal to None (TestDependencyUtils.scala:75)
[info]   Analysis:
[info]   Some(x: 0 -> )

[info] - should get None when there are no roots *** FAILED ***
[info]   Some(0) was not equal to None (TestMention.scala:59)

Thanks! I'll fix the broken tests before I make this into a real PR.

Hi @kwalcock and @MihaiSurdeanu, I did some debugging and made some more tests pass. However, I'm still failing on three tests (TestParallel, TestDependencyUtils, TestMention) and I'm not sure why they are failing. Below is the console output that I'm seeing. If you have any idea why the tests are failing, could you let me know? Thank you!

[info] *** 1 TEST FAILED ***
[error] Failed tests:
[error] 	org.clulab.processors.TestParallel
[info] Run completed in 16 minutes, 36 seconds.
[info] Total number of tests run: 19
[info] Suites: completed 4, aborted 0
[info] Tests: succeeded 19, failed 0, canceled 0, ignored 0, pending 0
[info] All tests passed.
[info] Run completed in 16 minutes, 36 seconds.
[info] Total number of tests run: 548
[info] Suites: completed 79, aborted 0
[info] Tests: succeeded 546, failed 2, canceled 0, ignored 1, pending 0
[info] *** 2 TESTS FAILED ***
[error] Failed tests:
[error] 	org.clulab.utils.TestDependencyUtils
[error] 	org.clulab.odin.TestMention
[error] (main / Test / test) sbt.TestsFailedException: Tests unsuccessful
[error] (corenlp / Test / test) sbt.TestsFailedException: Tests unsuccessful
[error] Total time: 1045 s (17:25), completed Aug 20, 2023, 6:28:33 AM

FWIW I'm looking at it.

Hi @kwalcock and @MihaiSurdeanu, I believe this PR is now ready to be reviewed and (hopefully) merged now. Would you be able to have a look at this and let me know if there's any issue left? Thank you!

Thanks, @alicekwak!


Add Turkish localisation

Thanks, looking good!
Will release a new version right now.

Or I would - looks like JitPack is currently down, gonna see if I can work around that.

There we go, new version is up on Modrinth and CurseForge is awaiting approval.
Thanks again!


CoreOS support for Socket Plane

Is CoreOS currently supported with Socketplane?

Partially yes. With https://github.com/socketplane/socketplane/pull/145 , we added the support for composable dependencies such as docker-ovs. Hence the OVS user-space components are not required on the OS. There are a few issues in the socketplane.sh script that needs addressing.
Once those are resolved, the solution should work in any minimalistic OS.


Initial Athena Documentation

to: @mime-frame @ryandeivert
cc: @airbnb/streamalert-maintainers
size: medium
resolves #231
Background
With the latest Athena deployments, it's time to provide some guidance on how to use the currently implemented features.  The documentation in this PR walks through initial setup, deployment, and basic usage of Athena.
Changes

Athena documentation and images
S3 events module usage
Small script to make writing and testing docs quicker
Addition of the log_level option in the generated Athena Lambda configuration

Coverage remained the same at 95.063% when pulling b089ed240444b52d57bc3196224c98374a630aec on jacknaglieri-athena-docs into 0c3de17f75724b65c19bd29dc666ad6c164f62d4 on master.

Minor nits, looks great @jacknagz !

Coverage remained the same at 95.063% when pulling 266121c5f707c1696fe85d78b67e0a45989db892 on jacknaglieri-athena-docs into 0c3de17f75724b65c19bd29dc666ad6c164f62d4 on master.


Disable Email Editing on User Preferences Page?

CAS uses the email field to store the CCID.
Users can edit their email at:
https://wiki.creativecommons.org/wiki/Special:Preferences
Presumably if the user changes this they can no longer log in. Should we disable this or leave it as a power user feature?

disable it

Roger roger.

i think disableemail might be okay.

That stops sending email to other users. I'll find the template and edit it out.

Okay

Done. The magic is -
$wgHiddenPrefs[] = 'emailaddress';


Tidy up requirements

Please confirm that this PR has done the following:

[x] Tests Added (N/A)
[ ] Decide how to clean up multiple requirements files
[ ] Decide how to document
[ ] Documentation Added
[ ] Description in RELEASE_NOTES.md Added

Adding to RELEASE_NOTES.md (remove section after adding to RELEASE_NOTES.md)
Please add a single line in the release notes similar to the following:
- (#XX)[http://link-to-pr.com] Added feature which does something

Description of PR
Tidies up requirements to ensure it is clear what is required to use pyam, what is required for the CI etc.

@gidden @danielhuppmann at the moment it looks like there's requirements in multiple different places (ci/, requirements.txt, requirements.py). Is it worth trying to put these all in one place? If so, do you have a preference of where?

Coverage remained the same at 84.942% when pulling dce0b0ed66ef9474e82341067f35c0cca5656377 on znicholls:update-ci into c1c0017653fc8a17570a82e77c78767f7d573ccc on IAMconsortium:master.

No strong preference, as long as it is documented. Thanks for helping us
tidy up! =)
On Mon, Apr 8, 2019 at 4:04 AM Coveralls<EMAIL_ADDRESS>wrote:

[image: Coverage Status] https://coveralls.io/builds/22657218
Coverage remained the same at 84.942% when pulling dce0b0e
https://github.com/IAMconsortium/pyam/commit/dce0b0ed66ef9474e82341067f35c0cca5656377
on znicholls:update-ci into c1c0017
https://github.com/IAMconsortium/pyam/commit/c1c0017653fc8a17570a82e77c78767f7d573ccc
on IAMconsortium:master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IAMconsortium/pyam/pull/214#issuecomment-480658308,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABVAEUMKj2AIWYd-yVSQ5jU44vTWLWHkks5veqO9gaJpZM4chJHv
.

But yes, to address your other statement, I believe it is worth consolidating all of this in one spot. The one thing we may need to have a copy of is a specific environment to replicate unit tests on CI (e.g., where to update the matplotlib requirement to get a certain freetype library in).

Ok cool. I think I'll be able to consolidate everything in one spot. The one complication is how to test the different install options (conda + pip or pip only). At the moment we're only testing conda + pip, which I am quite happy with, but if we continue down that route I'd make the docs a touch clearer so people are aware that pip only installs might not play nice with some plotting routines.
If you're happy, the steps I'd take would be:

consolidate requirements in one spot, maintaining the status quo of having tests only run under a conda install then pip install type setup
update docs with installation instructions and warnings about e.g. installing cartopy with pip (has never worked for anyone I've tried it with...)

Yes, sounds good. cartopy and other geospatial libraries have hard requirements on gdal and proj4 which are quite finicky c-level libraries that require either complicated installs or install via conda-forge. So a note in the docs would be quite appreciated..

I think the best I can get it is this (do we have dev docs you want notes like this to go in?):

all requirements go in setup.py, with the geospatial libraries being specified as an optional extra
in ./ci/, you have the environment files which specify how conda should install things for the tests

there is still the awkward thing here where you have to uninstall matplotlib with conda and then reinstall with pip to get the right freetype, I don't know how to fix that and put it all in one place
the problem is that you have to change the matplotlib version in .travis.yml, not in the ci environment files but there it is (one option would be to update all the plots to remove this pip uninstall reinstall business)

you can use the environment files in ./ci/ to make sure that the requirements specified in the feedstock are up to date

I don't see how to make the feedstock automatically keep track with the environment files in ./ci/, maybe it's impossible and this step always has to be done by hand (given it's a 3 minute job, probably not an issue)

Hey @znicholls, just tried make virtual-environment but got
$ make virtual-environment
make: Nothing to be done for 'virtual-environment'.

This was after trying
$ make test
/home/gidden/miniconda3/envs/pyam/bin/pytest tests --mpl --cov -rfsxEX --cov-report term-missing
ImportError while loading conftest '/home/gidden/work/iiasa/pyam/tests/conftest.py'.
tests/conftest.py:2: in <module>
    import matplotlib
E   ModuleNotFoundError: No module named 'matplotlib'

I made a fresh conda env, so assumed I should then use the environment make target. Is there something else I should be doing to check this on my machine?

Nope.. I'll have to try tomorrow with a fresh venv...
Can you post/email me the output of pip list and conda list with the venv
active?
On Tue, 9 Apr 2019 at 5:26 pm, Matthew Gidden<EMAIL_ADDRESS>wrote:

Hey @znicholls https://github.com/znicholls, just tried make
virtual-environment but got
$ make virtual-environment
make: Nothing to be done for 'virtual-environment'.
This was after trying
$ make test
/home/gidden/miniconda3/envs/pyam/bin/pytest tests --mpl --cov -rfsxEX --cov-report term-missing
ImportError while loading conftest '/home/gidden/work/iiasa/pyam/tests/conftest.py'.
tests/conftest.py:2: in
import matplotlib
E   ModuleNotFoundError: No module named 'matplotlib'
I made a fresh conda env, so assumed I should then use the environment
make target. Is there something else I should be doing to check this on my
machine?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IAMconsortium/pyam/pull/214#issuecomment-481135120,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWh-m-FXUymErCeSvZCoouRE6agCxVmjks5vfECugaJpZM4chJHv
.

Fix (for me at least) is inline. If you have time, can you hop on slack real quick? If not no problem.

Ok @znicholls, everything works on my machine with that line removed. I also just added a make install target so we get the "normal" workflow. With your permission, I will commit those changes and merge this.

closing in favor of #216, thanks again for your effort here @znicholls. CI and test infrastructure is even better now!!


python312Packages.aiohttp-remotes: 1.2.0 -> 1.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.aiohttp-remotes is: Set of useful tools for aiohttp.web server
meta.homepage for python312Packages.aiohttp-remotes is: https://github.com/wikibusiness/aiohttp-remotes
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.3.0 with grep in /nix/store/cbb87x53p5ifjrn1qyjzga8c3pnw8dl0-python3.12-aiohttp-remotes-1.3.0
found 1.3.0 in filename of file in /nix/store/cbb87x53p5ifjrn1qyjzga8c3pnw8dl0-python3.12-aiohttp-remotes-1.3.0

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

apache-airflow
python311Packages.aiohttp-remotes
python312Packages.aiohttp-remotes

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/cbb87x53p5ifjrn1qyjzga8c3pnw8dl0-python3.12-aiohttp-remotes-1.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.aiohttp-remotes https://github.com/r-ryantm/nixpkgs/archive/317be9365e804e580f0f64a7b55d1c4ec13a5d20.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/317be9365e804e580f0f64a7b55d1c4ec13a5d20#python312Packages.aiohttp-remotes

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/cbb87x53p5ifjrn1qyjzga8c3pnw8dl0-python3.12-aiohttp-remotes-1.3.0
ls -la /nix/store/cbb87x53p5ifjrn1qyjzga8c3pnw8dl0-python3.12-aiohttp-remotes-1.3.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 4 packages built:

python311Packages.aiohttp-remotes
python311Packages.aiohttp-remotes.dist
python312Packages.aiohttp-remotes
python312Packages.aiohttp-remotes.dist

Maintainer pings
cc @alyssais for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 353248

aarch64-darwin

:fast_forward: 2 packages marked as broken and skipped:

apache-airflow
apache-airflow.dist

:white_check_mark: 4 packages built:

python311Packages.aiohttp-remotes
python311Packages.aiohttp-remotes.dist
python312Packages.aiohttp-remotes
python312Packages.aiohttp-remotes.dist


chore(Collective): Remove deprecated countryISO from GraphQL

Was deprecated in https://github.com/opencollective/opencollective-api/pull/1811, frontend adapted in https://github.com/opencollective/opencollective-frontend/pull/1563.

Coverage decreased (-0.1%) to 64.685% when pulling 72ed27a9794a22c2d949b4900760b117e0b61630 on chore/remove-deprecated-countryISO into cedd887267d5771a0db76771309928a91f1f20b5 on master.


where it sets default implementation of IDistributedCache

As we know in https://docs.microsoft.com/en-us/aspnet/core/fundamentals/app-state?tabs=aspnetcore2x.
We need to Adds a default in-memory implementation of IDistributedCache like below.
public void ConfigureServices(IServiceCollection services)
{
    services.AddMvc();

    // Adds a default in-memory implementation of IDistributedCache.
    services.AddDistributedMemoryCache();

    services.AddSession(options =>
    {
        options.Cookie.Name = ".AdventureWorks.Session";
        options.IdleTimeout = TimeSpan.FromSeconds(10);
    });
}

But I found that services.AddDistributedMemoryCache() is not necessary，session can work without that.
public void ConfigureServices(IServiceCollection services)
{
   services.AddMvc();
   services.AddSession();
}

As we know session is depend on IDistributedCache. So, where it sets default in-memory implementation of IDistributedCache? I have found Microsoft.AspNetCore.Session Microsoft.AspNetCore.Caching Microsoft.AspNetCore.Mvc, but not result.
Any one help?

https://github.com/aspnet/Mvc/blob/068c91dbecc4e77165962743d37617344d6c8990/src/Microsoft.AspNetCore.Mvc.TagHelpers/DependencyInjection/TagHelperExtensions.cs#L34

It is called by services.AddMvc(); @Tratcher ,  Thanks very much  !!!


Add Multi-detector Feeder class

This enables multiple detectors to feed into single tracker, e.g. for a multi-sensor tracking scenario.

If we are feeding detections in from different sensors, they are likely to have different measurement covariances. Is there a way to connect the detections from a particular sensor to a particular Updater? This would also be necessary to handle the case where the detections/measurements have different components e.g A measurement from one sensor has Range, Bearing, Elevation while another sensor has Range, Bearing, Elevation, and Doppler. Each measurement would need it's own Updater, or we would need an Updater that can handle both measurements and likely just call the correct updater.
Not sure if that this is something that needs to be addressed within this pull request, but maybe an issue that needs to be addressed elsewhere?

If we are feeding detections in from different sensors, they are likely to have different measurement covariances. Is there a way to connect the detections from a particular sensor to a particular Updater? This would also be necessary to handle the case where the detections/measurements have different components e.g A measurement from one sensor has Range, Bearing, Elevation while another sensor has Range, Bearing, Elevation, and Doppler. Each measurement would need it's own Updater, or we would need an Updater that can handle both measurements and likely just call the correct updater.
Not sure if that this is something that needs to be addressed within this pull request, but maybe an issue that needs to be addressed elsewhere?

So this is currently addressed by attaching a measurement model to each detection. The Updater will then use this model, falling back to the one provided to the Updater if detection doesn't have it's own one. This feature is also required for example in a radar/bearing sensor, where it's position is changing, as each detection will have a measurement model with a different translations offset.

If we are feeding detections in from different sensors, they are likely to have different measurement covariances. Is there a way to connect the detections from a particular sensor to a particular Updater? This would also be necessary to handle the case where the detections/measurements have different components e.g A measurement from one sensor has Range, Bearing, Elevation while another sensor has Range, Bearing, Elevation, and Doppler. Each measurement would need it's own Updater, or we would need an Updater that can handle both measurements and likely just call the correct updater.
Not sure if that this is something that needs to be addressed within this pull request, but maybe an issue that needs to be addressed elsewhere?

So this is currently addressed by attaching a measurement model to each detection. The Updater will then use this model, falling back to the one provided to the Updater if detection doesn't have it's own one. This feature is also required for example in a radar/bearing sensor, where it's position is changing, as each detection will have a measurement model with a different translations offset.

Ok - that makes sense, I just hadn't seen that before or knew we could do that.


Found a Mind Games leftover

A "Psy_Dialogue" was found in the flies, not sure if this is needed for example of a dialogue.

I Think Its From The Vs Psy Mod Beacuse I Think The Engine Is That Mod

I Dont Know If Its Needed

It is a leftover yeah, but that's because i didn't made a Dad Dialogue portrait yet


Need a sortable table of existing approaches.

Hello everyone,
The current version is not sortable. It is quite difficult to find recent papers in a chronological order.
Can you give me any advice?
Best regards,
Thank you

Thanks a lot for your suggestions. We have created a website to make it easier to search and sort the machine unlearning literature. https://awesome-machine-unlearning.github.io/


Create documentation section in the site.

I'm almost done with creating the react package, and for that I need to write some documentation as well.
With that, maybe it is nice to centralize the documentation.
I got a bit inspired by the ChakraUI website, I think they have an great, simple and nice looking documentation site. I noticed they use their own UI + next.js, the same as we do. And they use enhanced markdown to generate documentation pages. (.mdx files), see their [next.config.js](https://github.com/chakra-ui/chakra-ui/blob/develop/website/next.config.js).
So maybe we can use that as well.

What exactly should be in the documentation?

@JanTrichter I created the first one https://github.com/lucide-icons/lucide/issues/378


desc field not written to M4B

Publisher summary is supposed to be written to desc field in m4b files according to Seanap guide and this readme. But I've noticed only comments is written to m4b using beets-audible. Using Seanaps mp3tag scripts both comments and desc are written to m4b. Audiobookshelf does not seem to import the comments field from m4b.
ffprobe outputs
mp3tag'd example: note both description and comment
  Metadata:
    major_brand     : isom
    minor_version   : 512
    compatible_brands: isomiso2mp41
    title           : Leviathan Wakes (Unabridged)
    artist          : James S. A. Corey
    composer        : Jefferson Mays
    album           : Leviathan Wakes
    genre           : Science Fiction & Fantasy/Science Fiction
    date            : 2011
    encoder         : Lavf56.40.101
    series-part     : 1
    RELEASETIME     : 2014
    ~length         : 19:11:27
    ASIN            : B00P9XDQFY
    SERIES          : The Expanse
    tmp_Genre2      : Science Fiction
    tmp_Genre1      : Science Fiction & Fantasy
    Subtitle        : Book 1 of the Expanse
    WWWAUDIOFILE    : https://www.audible.com/pd/Leviathan-Wakes-Audiobook/B00P9XDQFY?ipRedirectOverride=true&overrideBaseCountry
    media_type      : 2
    album_artist    : James S. A. Corey
    grouping        : The Expanse, Book #1
    copyright       : 2011 James S. A. Corey
    sort_album      : The Expanse 1 - Leviathan Wakes
    description     : Humanity has colonized the planets - interstellar travel is still beyond our reach, but the solar system has become a dense network of colonies. But there are tensions - the mineral-rich outer planets resent their dependence on Earth and Mars and the poli
    comment         : Humanity has colonized the planets - interstellar travel is still beyond our reach, but the solar system has become a dense network of colonies. But there are tensions - the mineral-rich outer planets resent their dependence on Earth and Mars and the poli

beets-audible
  Metadata:
    major_brand     : isom
    minor_version   : 512
    compatible_brands: isomiso2mp41
    title           : Leviathan Wakes (Unabridged)
    artist          : James S. A. Corey
    composer        : Jefferson Mays
    album           : Leviathan Wakes
    genre           : Science Fiction & Fantasy/Fantasy
    track           : 1/1
    disc            : 1/1
    date            : 2011-06-02
    compilation     : 0
    encoder         :
    SERIES-PART     : 1
    LANGUAGE        : English
    MusicBrainz Album Type: audiobook
    ORIGINAL YEAR   : 2011-06-02
    ASIN            : B073H9PF2D
    SERIES          : The Expanse
    LABEL           : Hachette Audio
    publisher       : Hachette Audio
    media_type      : 2
    sort_album_artist:
    sort_artist     :
    sort_composer   :
    album_artist    : James S. A. Corey
    grouping        : The Expanse, Book #1
    sort_album      : The Expanse 1 - Leviathan Wakes
    comment         : From a *New York Times* best-selling and Hugo award-winning author comes a modern masterwork of science fiction, introducing a captain, his crew, and a detective as they unravel a horrifying solar system wide conspiracy that begins with a single missing g
                    :
                    : Humanity has colonized the solar system - Mars, the Moon, the Asteroid Belt and beyond - but the stars are still out of our reach.
                    :
                    : Jim Holden is XO of an ice miner making runs from the rings of Saturn to the mining stations of the Belt. When he and his crew stumble upon a derelict ship, the Scopuli, they find themselves in possession of a secret they never wanted. A secret that someo
                    :
                    : Detective Miller is looking for a girl. One girl in a system of billions, but her parents have money and money talks. When the trail leads him to the Scopuli and rebel sympathizer Holden, he realizes that this girl may be the key to everything.
                    :
                    : Holden and Miller must thread the needle between the Earth government, the Outer Planet revolutionaries, and secretive corporations - and the odds are against them. But out in the Belt, the rules are different, and one small ship can change the fate of th
                    :
                    : **The Expanse**
                    :
                    : * *Leviathan Wakes*
                    : * *Caliban's War*
                    : * *Abaddon's Gate*
                    : * *Cibola Burn*
                    : * *Nemesis Games*
                    : * *Babylon's Ashes*
                    : * *Persepolis Rising*
                    : * *Tiamat's Wrath*
                    : * *Leviathan Falls*
                    : * *Memory's Legion*
                    :
                    : **The Expanse Short Fiction**
                    :
                    : * *Drive*
                    : * *The Butcher of Anderson Station*
                    : * *Gods of Risk*
                    : * *The Churn*
                    : * *The Vital Abyss*
                    : * *Strange Dogs*
                    : * *Auberon*
                    : * *The Sins of Our Fathers*

Thanks for spotting this.
I've written it as a tag called desc, let me know if this works. I wasn't sure if it should be desc or ----:com.apple.iTunes:desc

Audiobookshelf picks up the new desc field and displays it in app.
Thanks!


Multi-threaded execution throws an exception (using GPU).

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
binary
TensorFlow version
2.19.0-dev20241018
Custom code
Yes
OS platform and distribution
Ubuntu 24.04
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
Multi-threaded execution throws an exception (using GPU).
Standalone code to reproduce the issue
import concurrent

import numpy as np
import tensorflow as tf
print(tf.__version__)

executor = concurrent.futures.ThreadPoolExecutor()

def sum(x, axis):
    return tf.reduce_sum(x, axis=axis)

futures = []

for i in range(1000):
    futures.clear()
    for _ in range(4):
        x = tf.convert_to_tensor(np.random.rand(100, 100))
        futures.append(executor.submit(sum, x, 1))
        x = tf.convert_to_tensor(np.random.rand(100))
        futures.append(executor.submit(sum, x, 0))
    concurrent.futures.wait(
        futures, return_when=concurrent.futures.ALL_COMPLETED
    )
    [future.result() for future in futures]

Relevant log output
W tensorflow/core/framework/op_kernel.cc:1840] OP_REQUIRES failed at reduction_ops_common.h:147 : INVALID_ARGUMENT: Invalid reduction dimension (1 for input with 1 dimension(s)
I tensorflow/core/framework/local_rendezvous.cc:405] Local rendezvous is aborting with status: INVALID_ARGUMENT: Invalid reduction dimension (1 for input with 1 dimension(s)

Potentially related: https://github.com/tensorflow/tensorflow/issues/72388

@nicolaspi,
The PR which was raised for the issue has been merged and also tried to execute the mentioned code on both tensorflow v2.17 and tf-nightly, observed on tf-nightly the code was executed. Kindly find the gist of it here and confirm whether the issue was resolved? Thank you!

@tilakrayal You need to !pip install tf-nightly[and-cuda] (it fails using the GPU).

@nicolaspi,
I tried to execute with the latest tensorflow v2.18(GPU), and the code was executed without fail/error as 2.17. Kindly find the gist of it here. Thank you!

@tilakrayal I am still getting the error (with 2.18). Please note that the test is non-deterministic and there is a small probability of success.