Tasks not filtered by name prefix

A GET request to /tasks?name_prefix=foo&view=MINIMAL should return only tasks whose identifiers start with foo. However, it appears that all tasks are returned, not filtered at all.
To reproduce

POST one or more tasks to a given proTES instance
Store URL to proTES TES API in variable TES (e.g., https://my.tes.org/ga4gh/tes/v1)
Run the following commandcurl -X GET "${TES}/tasks?name_prefix=foo&page_size=10&view=MINIMAL" -H "accept: application/json"

Expected behavior
Should only list tasks whose IDs start with foo (probably none at all).
Actual behavior
Lists all tasks, including those whose identifiers do not start with foo.

A GET request to /tasks?name_prefix=foo&view=MINIMAL should return only tasks whose identifiers start with foo. However, it appears that all tasks are returned, not filtered at all.
To reproduce
* `POST` one or more tasks to a given proTES instance

* Store URL to proTES TES API in variable `TES` (e.g., `https://my.tes.org/ga4gh/tes/v1`)

* Run the following command
  ```shell
  curl -X GET "${TES}/tasks?name_prefix=foo&page_size=10&view=MINIMAL" -H "accept: application/json"
  ```

Expected behavior
Should only list tasks whose IDs start with foo (probably none at all).
Actual behavior
Lists all tasks, including those whose identifiers do not start with foo.

Hi @uniqueg ,
I would love to solve this issue.
The brute force approach to solving this would be iterating through all the tasks and just append the tasks which start with say foo (or anything else) to a list which we wll finally return.

Thanks @SohamRatnaparkhi, I have assigned you.
As stated in the issue, the endpoint is GET /tasks. The controller name for that you can find in the OpenAPI specification. I am not looking it up for you, so that you will have a chance to explore how OpenAPI specs are structured ;-)
Note that there are two general approaches you could follow here:

Only fetch tasks matching the prefix from the database. Here you would need to find out how you can define and pass such a filter when doing a database call. Such a solution is preferred as it will be much more performant, especially when we will have thousands of tasks. However, I am not 100% sure it's possible with MongoDB, but https://www.mongodb.com/docs/manual/reference/operator/query/regex/ looks promising.
Filter matching tasks after the database call. This is the general approach you describe. If you do not get anywhere with approach (1), make sure that you don't just use a plain for loop. Instead make use of map() or use a list comprehension. In fact, you should probably try both and see which approach is more performant. If you can think of any other solutions, include them in your benchmark.

Thanks @SohamRatnaparkhi, I have assigned you.
As stated in the issue, the endpoint is GET /tasks. The controller name for that you can find in the OpenAPI specification. I am not looking it up for you, so that you will have a chance to explore how OpenAPI specs are structured ;-)
Note that there are two general approaches you could follow here:
1. **Only fetch tasks matching the prefix from the database.** Here you would need to find out how you can define and pass such a filter when doing a database call. Such a solution is preferred as it will be much more performant, especially when we will have thousands of tasks. However, I am not 100% sure it's possible with MongoDB, but https://www.mongodb.com/docs/manual/reference/operator/query/regex/ looks promising.

2. **Filter matching tasks _after_ the database call.** This is the general approach you describe. If you do not get anywhere with approach (1), make sure that you don't just use a plain `for` loop. Instead make use of `map()` or use a list comprehension. In fact, you should probably try both and see which approach is more performant. If you can think of any other solutions, include them in your benchmark.

Yep. This works as well!
I will try to come up with a solution soon and will draft a PR.


Don't remove trailing / on manual scan requests

Hello
Have a unique request, occasionally I would like to use the /triggers/manual web page to refresh an entire root folder across multiple apps. When I do this, it doesn't work because it says root library cannot be determined. I think this is because autoscan appears to automatically remove the trailing / on submitted requests. Respectfully requesting that this auto removal be removed so that root folders can be submitted for scan requests. I tested this by putting on /path/to/root/folder/* and auto scan no longer threw the error, however, it submits it as litteral, so it obviously didn't scan the root folder.
Thanks in advance for your consideration.
Ed

Same issue, I even tried submitting the curl with a double slash (%2F%2F) at the end and it removed both.

We currently have this trailing slash removal implemented to avoid accidental scans of entire libraries (which can take considerable amounts of time). Autoscan is designed to optimise scans around film/show/season specific directories and such a full library scan would do away with these optimisations. I recommend to either manually trigger a full library scan inside the Plex UI or, similarly to how Autoscan is implemented, manually call the /refresh Plex API endpoint.


dynamically change laser module on hwserver

Finally no more hard-coded module name. Dynamically requests from hwserver when host is supplied.

There is still generally a known bug with dynamically reloading settings while in the callback for settings. I added more context in the error message to mitigate the effects of this bug, but not in the scope of this pull request to fix it.


Complex objects in open type code samples in documentation do not work

In the documentation, the generated code doesn't correctly specify the open type property values when that property is of a complex type. An example is the Planner APIs:
Update plannertask - Microsoft Graph v1.0 | Microsoft Docs
Create plannerTask - Microsoft Graph v1.0 | Microsoft Docs
In these request open type objects require specific property value types, however the code sample has serialized string representation of the data, instead of the object of proper type.
This part:
Assignments = new PlannerAssignments
{
  AdditionalData = new Dictionary<string, object>()
  {
      {
         "fbab97d0-4932-4511-b675-204639209557", "{\"@odata.type\":\"#microsoft.graph.plannerAssignment\",\"orderHint\":\"N9917 U2883!\"}"
      }
  }
},

Should look like this:
Assignments = new PlannerAssignments
{
   AdditionalData = new Dictionary<string, object>()
   {
       {
          "fbab97d0-4932-4511-b675-204639209557", new PlannerAssignment()
          {
               OrderHint:"N9917 U2883!\"
          }
       }
   }
},

I suspect the sample is is created based on the HTTP example, in which case the presence of Odata.type attribute can be used to identity the actual type to be used.
AB#7858

This is fixed with SDK v5 snippets


Added environment definable plugin directories using semi-colon ';' delim

Helps plugin developers organize their development/testing environment how they'd like it set up. Provides power for hosted platforms as well to install as NPM but put plugin directory somewhere else.

Please hold off on merging this, I'd like to abstract that into get_plugin_paths() and add some test coverage. I'll submit a PR against @martin1yness branch and see if he approves.

@martin1yness , I have opened a PR against your master branch. Please review it and if you're happy, merge it and test the changes. It should work exactly as this one, mainly just moving it into an exported function with test coverage. Since this PR is based off your master branch, I'm not sure if merging my PR will update this one. I think not. But either way, post here afterwards and if it hasn't, I'll open a new PR that brings in my tests.


Implement get list of authors API

Implement the get list of authors API as per the specification defined in https://github.com/chinthakarukshan/lbms/issues/19

API implementation completed.


Make css output easier to debug

The generated css will now include source comments which show where in
your sass files each css block comes from.
Also tidied up the options block:

The style option is not supported by grunt-sass (outputStyle is the  correct option)
sourceMap was misspelt and wasn’t generating the source maps
The options have been arranged in alphabetical order to make it easier
to read

Example source comments:
/* line 18, govuk_modules/govuk_frontend_toolkit/stylesheets/_grid_layout.scss */
#content {
  max-width: 960px;
  margin: 0 15px;
}

@media (min-width: 641px) {
  /* line 18, govuk_modules/govuk_frontend_toolkit/stylesheets/_grid_layout.scss */
  #content {
    margin: 0 30px;
  }
}

Nice!
Works for me, here's a couple of screenshots.
on master (without this change):

on this branch:

I've rebased on top of master, happy for you to merge when the tests are complete @alextea.


Generated declaration file for constructor nullable parameter property is non-nullable

TypeScript Version:  2.2.2
Code
// typescript code
export class Foo {
  constructor(public bar?: string) {
  }
}

// generated declaration
export declare class Foo {
    bar: string;
    constructor(bar?: string);
}

Expected behavior:
Foo.bar should be typed as string | undefined
Actual behavior:
Foo.bar is typed as string
Our declaration is no longer compatible with the source that generated it.

Also affects 2.4.0.

Fix is up at #16497


uppercase fiware-service

Hey guys,
I was just checking if i could use your tool to access an orion instance, but unfortunately i'm failing already in the beginning.
It seems that you do not support fiware-services with uppercase letters (seems to be intented as you test that uppercase fiware services are producing an error):
https://github.com/lets-fiware/ngsi-go/blob/1a07b10911df686410566f299b885551fc2313fc/internal/ngsilib/validation_test.go#L27
Is there any reason for this?
The problem is, the orion instance i want to connect to uses fiware services like Foo instead of foo and if i try to connect with a lowercase fiware service, it does not work.
Does work:
➜ ngsi broker add \
  --host myinstance \
  --brokerHost http://localhost:1026 \
  --ngsiType v2 \
  --service foo \
  --path /iot

Does not work
➜ ngsi broker add \
  --host myinstance \
  --brokerHost http://localhost:1026 \
  --ngsiType v2 \
  --service Foo \
  --path /iot

With this error: brokersAdd007 error FIWARE Service: Foo

Orion interprets the tenant name (fiware-service name) in lowercase. To use a coherent tenant name, NGSI Go allows only lowercase letters in tenant names.
Please have a look at MULTI TENANCY section in Orion documentation.
https://fiware-orion.readthedocs.io/en/master/user/multitenancy/index.html

Orion Context Broker interprets the tenant name in lowercase, thus, although you can use tenants such as in updateContext "MyService" it is not advisable, as the notifications related with that tenant will be sent with "myservice" and, in that sense, it is not coherent the tenant you used in updateContext compared with the one that Orion sends in notifyContextRequest.

Orion interprets the tenant name (fiware-service name) in lowercase. To use a coherent tenant name, NGSI Go allows only lowercase letters in tenant names.
Please have a look at MULTI TENANCY section in Orion documentation.
https://fiware-orion.readthedocs.io/en/master/user/multitenancy/index.html

Orion Context Broker interprets the tenant name in lowercase, thus, although you can use tenants such as in updateContext "MyService" it is not advisable, as the notifications related with that tenant will be sent with "myservice" and, in that sense, it is not coherent the tenant you used in updateContext compared with the one that Orion sends in notifyContextRequest.

Thanks for the clarification!

Thanks for the clarification!


appropriate value for mfo when not defined?

Hard to know where is the appropriate forum for my simple question. When defining a quantity like mfo (mass transport in the ocean) where a value is normally required but perhaps is not resolved in the model (e.g. the english channel in my case), what value should be used? I suppose I am meaning missing or 0? I have noticed that ACCESS-CM2 used 0 however a missing value would be more informative and useful in terms of generating stats.?
Thanks in advance

@markcollier that variable does exist, see http://clipc-services.ceda.ac.uk/dreq/u/baa60bf2-e5dd-11e5-8482-ac72891c3257.html and PCMDI/cmip6-cmor-tables/Tables/CMIP6_Omon.json.
If a model doesn't resolve this field, then I would use the missing value identifier which is more accurate than 0. In the case that a user wanted to generate a multi-model mean, a NaN will be a far better value to assess than 0 if the transport is not resolved.

i'll send a message to the ACCESS folk in response to their oversight...


Make validation optional

Hi! Great job on this so far!
Would it be possible to add an isRequired field or something like that? I have a form where I do not require a phone number but it is there if the user wants to put one in. Maybe there is already a way to accomplish this but I couldn't figure it out.

Hi, @zakton5 thanks for your feedback. I am going to add new property called ignore Blank.  If set to true this would not validate the field.


soc: nrf: basic power management for nrf53 and nrf91

Only System OFF mode is supported. Based on nrf52 power management, but nrf53 and nrf91 use nrf_regulators HAL instead of nrf_power.

@masz-nordic If it is true that the nrf_regulator is only available for the application core then the current changes need another modification. We need to move the select HAS_SYS_POWER_STATE_DEEP_SLEEP_1 from Kconfig.soc to the Application core Kconfig seeing as the following code in power.c includes the regulator for both cores at the moment:
#ifdef CONFIG_HAS_SYS_POWER_STATE_DEEP_SLEEP_1
#include <hal/nrf_regulators.h>
#endif

@MarcianoPreciado

We need to move the select HAS_SYS_POWER_STATE_DEEP_SLEEP_1 from Kconfig.soc to the Application core Kconfig

That is a part of this PR, see changes in nRF53 Kconfig.series and Kconfig.soc.

It appears that the nrf5340 supports a "Core Force-off mode" in which the app core can force the network core off. This is likely the scope of another commit.

Correct, this is just basic power management, similar to nRF52. Controlling the network core from application core (see also https://github.com/zephyrproject-rtos/zephyr/blob/master/boards/arm/nrf5340_dk_nrf5340/nrf5340_cpunet_reset.c) should be done in a separate PR.

@pabigot @ioannisg Can you review?

Any update on this PR? @masz-nordic did you see the latest feedback from @pabigot? @ioannisg have you had a look at this PR?

Overall, I think this needs to be a common file for nrf SoCs, as @pabigot suggested


⚠ Blutopia has degraded performance

In 778e166, Blutopia ($BLU) experienced degraded performance:

HTTP code: 200
Response time: 13216 ms

Resolved: Blutopia performance has improved in 2a52709 after 20 minutes.


PostgreSQL tests and table bloat due to updates

Investigate what is the table bloat because there are many writes to the World/world table. Autovacuum will run around 200 000 000 writes to the table with the default settings Autovacuum is not enabled. Maybe this will impact some of the projects?
Additional information materials:

https://www.joyent.com/blog/manta-postmortem-7-27-2015 - Joyent had a problem with the vacuum operation on the tables
https://blog.sentry.io/2015/07/23/transaction-id-wraparound-in-postgres.html via the above Joyent post

More information:
https://wiki.postgresql.org/wiki/Show_database_bloat
https://github.com/ioguix/pgsql-bloat-estimation/blob/master/table/table_bloat.sql - query for tables
https://github.com/ioguix/pgsql-bloat-estimation/blob/master/btree/btree_bloat.sql - query for index
the above two are referred from the PostgreSQL Wiki
http://blog.ioguix.net/tag/bloat/
The initial test on Vagrant environment shows some small table bloat, but the table is very small - under 1 MiB. But this is after only under 1 million updates. So maybe it will be different with more updates. I'll adjust the config to use 10 minutes for this test case with servlet-postgres-raw and dropwizard-postgres.

After two runs with 10 minute updates for servlet-postgres-raw and dropwizard-postgres the bloat seems to stay consistent - the whole table is around 1MiB.
Also note that there is world and World tables which should be kept in mind if doing similar comparisons.


Fix crash on identifier name

Was crash on time variable using in subprograms as property.

Download:
Environment: QT=5.12, ARCH=32, CONFIGURATION=release, QTIFW=C:\Qt\Tools\QtInstallerFramework\3.1


test include / exclude doesn't allow limiting scope

Problem
There seems to be no way to run only tests of a specific tag.
As an example: Run only tests that include the tag get
This leads to a situation where -i has no effect.
Steps

target/debug/tremor test api tremor-cli/tests -i get -> runs all tests
target/debug/tremor test api tremor-cli/tests -e all -i get -> runs no tests

Possible Solution(s)
I can think of two possible solutions:

if -i is provided run only included tests not all tests
if -e and -i is included run (all - -e) + -i  (all without but not the excluded but always the included)

adding to 0.9.2 as it is a prereq for #563


Attachment Improvements (Second Phase)

[x] Replace default form file tags with a drag & drop attachment panel (see https://dropzonejs.com)

Use the uploader to upload the document to a new table (hesk_attachments_temp). This table will hold the saved name, unique ID, and date uploaded.
The AJAX call returns the ID of the temp document. This ID is added as a hidden field to the ticket that is sent when "Submit" is pressed.
On Submit:

The attachments are pulled from the temp table based on ID, and are moved to the permanent (hesk_attachments) table.
If there are temp attachments that are over 24 hours old, they are "expired" and removed from the file system to save disk space.

[x] Provide a way to see how far the file has been uploaded

[x] If possible, check the file size before even starting the upload to see if it exceeds the max allowed size.

Each form will have a unique "file upload ID" to keep track of which uploads are for the ticket being submitted.

Relevant UserVoice: https://mods-for-hesk.uservoice.com/forums/254758-general/suggestions/7787049-upload-bar-progress-admin-and-client-side, https://mods-for-hesk.uservoice.com/forums/254758-general/suggestions/7563459-advanced-upload-upload-attachment-s-with-drag-and
Look into how to add a property to the File object so I can append the upload ID to the file, in case a user deletes a file.

Done. Will be released in 2.6.0.


feat: add bumpoverride option

Pull Request
Description
I use conventional commits for changelog generation mainly and have a need to manually set the bump level from time to time (custom CalVer based versioning scheme). Other tools like standard-version of npm also allow for override through a --releaseAs=patch|minor|major parameter.
This PR simply adds an optional --bumpoverride=patch|minor|major argument. If it's not set then automatic version bumping is used as before, when set the rec_bump is overriden.
~/forks/version-bumper   master  docker run -v $PWD:/usr/share/repo jveessen/version-bumper:latest
2.1.0 2.1.1.dev
~/forks/version-bumper   master ±  docker run -v $PWD:/usr/share/repo jveessen/version-bumper:latest --bumpoverride minor
2.1.0 2.1.1.dev
~/forks/version-bumper   master ±  docker run -v $PWD:/usr/share/repo jveessen/version-bumper:latest --bumpoverride patch
2.0.5 2.0.6.dev
~/forks/version-bumper   master ±  docker run -v $PWD:/usr/share/repo jveessen/version-bumper:latest --bumpoverride major
3.0.0 3.0.1.dev

Checklist

[ ] I have followed this repository's contributing guidelines.
[x] I have used the conventional commits specification for my commit messages.
[x] I have signed my commits.
[x] I will adhere to the project's code of conduct.

Additional information
The contributing guidelines link seems to be broken so I couldn't check them.

Anything else?

@MisterTimn
Thank you very much for your contribution,
I'll merge this now and release a new version right asap.
About these failing CI workflows, you have 3 commits in this PR,
one of them isn't following the conventional commits specifications,
another one is not signed off, and another is a good commit.
As this is gonna be squashed before merging and we have one good commit, we can ignore these:

One more point I have,
for future PRs you might prefer to create a branch on your fork before you open a PR.
When you PR from the default branch:
1- CLI users will have to replace references if they want to pitch in on your project, including maintainers that might want to help.
2- As soon as this gets merged - your fork is gonna get out of sync, and you might need some manual commands to get it right. (or recreate the fork).
Thank you very much - merging now!

@MisterTimn Thank you very much for your contribution, I'll merge this now and release a new version right asap.
About these failing CI workflows, you have 3 commits in this PR, one of them isn't following the conventional commits specifications, another one is not signed off, and another is a good commit.
As this is gonna be squashed before merging and we have one good commit, we can ignore these:

One more point I have, for future PRs you might prefer to create a branch on your fork before you open a PR. When you PR from the default branch: 1- CLI users will have to replace references if they want to pitch in on your project, including maintainers that might want to help. 2- As soon as this gets merged - your fork is gonna get out of sync, and you might need some manual commands to get it right. (or recreate the fork).
Thank you very much - merging now!

Thanks for the info and future tips :)

@all-contributors please add @MisterTimn for code

@all-contributors please add @MisterTimn for code

@all-contributors please add @MisterTimn for code

@all-contributors please add @MisterTimn for code

Sorry for that - all-contributors was giving me a hard time.
I ended up using their CLI tool to do it manually: https://github.com/TomerFi/version-bumper/commit/379d420958d97f5f33aa0c3f3e12e75722b86772

@MisterTimn
The new version 2.1.0 release includes your contribution.
Thank you!


[Extension Feature Request] Play Spotify albums

Extension – Spotify Player

Author: @mattisssa

Describe the feature and the current behavior/state.
Currently, the Spotify Player extension is great to search for individual tracks, but that's not the way a lot of people consume music: we play whole albums as well! I would like the search to work for albums. Currently when you select a track you can Cmd-K and it has the Show Album option, but that just opens Spotify in the web. It would be great to have a Play Album option.
Who will benefit with this feature?
Everyone who likes to listen to music through albums!
Any Other info.
Thanks for the extension!

Any chance of this being revisited/reopened? I think this would be a great feature. Additionally, is it possible for the "Show Album" option to open in the Spotify desktop application if installed?

Hey! Oh yeah, I missed that the stale bot closed it.
Actually, it was impossible with the current authorisation flow, but a spoiler alert: I have done it in the beta version and internal version of Raycast where it’s possible and many more. So, set the reminder to around 20-22 of April to get that 😉


On the scroll navbar having the transparent background!!!

Describe the solution you'd like
On the scroll navbar's text is merging into the content of the site and i would like fix these feature as making the background for the navbar for scroll can you assign me this task?

Can you please describe the issue with some screenshot? Please,use the latest commit from main branch.

Try updating the theme to latest version. Feel free to re-open if the issue still persist.


return promises

oh @domenic, god of promises, i beseech favor upon the practitioners of generators

+1

@domenic what's your opinion on this now? major bump and drop node 0.10 support?

@jonathanong sure that'd work for me. But, not invested enough in knox to actually do the work, so, PR welcome :)

may the @learnboost overlords add me as a collaborator?

Just make a PR like a normal person dude ;). I still have time to review, to be clear.

Would be great to have a Promise returned when methods like putFile, getFile, etc are called without callbacks.

+1


Issue creating Windows Update Deployment

Bicep version
0.4.451
Describe the bug
Trying to create a Windows Update Deployment and hitting a warning with includedUpdateClassifications
To Reproduce
Create a softwareUpdateConfigurations resource and add the below classifications:

Critical
Security
UpdateRollup
Updates

  resource windowsUpdates 'softwareUpdateConfigurations@2019-06-01' = {
    name: 'windowsUpdates'
    properties: {
      scheduleInfo: {
          advancedSchedule: {
            monthlyOccurrences: [
              {
                day: 'Saturday'
                occurrence: 3
              }
            ]
          }
        description: 'Windows OS Updates'
        frequency: 'Month'
        interval: 1
        isEnabled: true
        startTime: startTime
      }
      updateConfiguration: {
        duration: 'PT2H'
        operatingSystem: 'Windows'
        targets: {
          azureQueries: [
            {
              scope: [
                subscription().id
              ]
              tagSettings: {
                filterOperator: 'All'
                tags: {
                  MaintenanceWindow: [
                    'ThirdSaturdayOfMonth'
                  ]
                }
              }
            }
          ]
        }
        windows: {
          includedUpdateClassifications: 'Critical'
          rebootSetting: 'IfRequired'
        }
      }
    }
  }

Additional context
When adding more than 1 update classification, I'm getting the below error:

I don't find the documentation on this very clear, and I don't see anything in the REST API specs either on how to do this correctly
Also, if I'd like this deployment to run at 10pm on the 3rd Saturday of each month (west europe region), how would I set the time for that? I'm not sure how to configure the timeZone
Any help appreciated, thanks

This isn't very obvious, but when I hover over the includedUpdateClassifications property, I see this description:

Update classification included in the software update configuration. A comma separated string with required values

Have you tried specifying multiple settings in a comma-separated format - e.g.:
...
        windows: {
          includedUpdateClassifications: 'Critical,Security,UpdateRollup,Updates'
          rebootSetting: 'IfRequired'
        }
...

Disclaimer - I don't work for the Automation team, so I don't know the ins-and-outs of the service, but from the example the team has published alongside their API specification, it looks like an IANA time zone name:
https://github.com/Azure/azure-rest-api-specs/blob/1b0ed8edd58bb7c9ade9a27430759527bd4eec8e/specification/automation/resource-manager/Microsoft.Automation/stable/2019-06-01/examples/softwareUpdateConfiguration/createSoftwareUpdateConfiguration.json#L73

Thanks for coming back to me, I'll give the update classifications a try. I didn't think of that!
Regarding the timeZone - if I set it to Europe/Amsterdam i get this error:
    "status": "Failed",
    "error": {
        "code": "DeploymentFailed",
        "message": "At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.",
        "details": [
            {
                "code": "BadRequest",
                "message": "{\r\n  \"code\": \"BadRequest\",\r\n  \"message\": \"Argument scheduleAllData with value Orchestrator.Schedules.DataAccess.Models.ScheduleAllData is not valid. Error message: The start time of the schedule must be at least 5 minutes after the time you create the schedule. \"\r\n}"
            }
        ]
    }
}

Without `timeZone` it works fine, but then I can't set up deployments for other regions. I guess I'll have to raise an issue with them somewhere as I'm out of ideas on what to try

@vrdmr - since you're tagged as an automation code owner. Do you know what could be going wrong here? Do you have any documentation to share?

Yep so the below works:
        windows: {
          includedUpdateClassifications: 'Critical,Security,UpdateRollup,Updates'
          rebootSetting: 'IfRequired'
        }

Thanks for your help! 👍


Delauny outputs from HiQGA

Would like to implement a Delaunay triangulation from point clouds, and visualise this within ParaView as a VTK.

Delaunay, not best interpolator for skewed aspect ratios. VTK Structured grid found to be better. Added support with 08ce98a


Fallback to tomcat 8 if artifact type is null

Short Description
Fallback to tomcat 8 if artifact type is null

The ArtifactType should be correctly set for all invocations now (see #246).


Add release-crates bash script

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
One part of the release process that is time-consuming is running the individual cargo publish commands.
The cargo publish process has improved recently and the command will not complete until the crate has been propagated to the crates.io index so this makes it more compelling to automate this process.
Describe the solution you'd like
Write a script that runs the following commands. The script should check that the command is not being run from a git repository (we need to run from the downloaded tarball).
The documentation at dev/release/README.md should also be updated.
(cd datafusion/common && cargo publish)
(cd datafusion/expr && cargo publish)
(cd datafusion/sql && cargo publish)
(cd datafusion/jit && cargo publish)
(cd datafusion/row && cargo publish)
(cd datafusion/physical-expr && cargo publish)
(cd datafusion/optimizer && cargo publish)
(cd datafusion/core && cargo publish)
(cd datafusion/proto && cargo publish)
(cd datafusion-cli && cargo publish --no-verify)

Describe alternatives you've considered
Keep doing this manually
Additional context
None

On it


Stop countdown timer for new recorder UI

When Pre-Start is in use and a countdown is running, pressing Stop button doesn't really stop the countdown, and the capture still starts after the countdown ends.
This commit should fix #344
Unfortunately I didn't have enough time to go over all modes, but at least for me, New Recorder UI works fine after this change.

Thank you, I'll also apply the same code to the old recorder.


Update Button to use startIconName and endIconName

Description
In an effort to improve our RTL language support we should use start and end instead of right and left for position props.
Technical Details

Update Button prop iconPositionRight to  use startIconName  and endIconName

Acceptance Criteria

All instances of iconPositionRight have been removed in favour of  startIconName  and endIconName
All documentation has been updated

References
https://github.com/MetaMask/metamask-extension/issues/17613
https://github.com/MetaMask/metamask-mobile/issues/5685

@georgewrmarshall
https://www.figma.com/file/HKpPKij9V3TpsyMV1TpV7C/DS-Components?node-id=9970%3A48395&t=ORd6ip1MYrSXEmXE-4


Update storage-account-overview.md

It should be encrypted, not encryption

@iamvighnesh : Thanks for your contribution! The author, @tamram, has been notified to review your proposed change.

@iamvighnesh Great catch! Thank you for submitting this change.


Both .js files just hold "not compatible with v9" alert code & manifest not pointing to a valid download.

Subject says it all really.
There are also some issues with the module.json throwing up deprecation errors, I'll do up a PR for that if I get some time this week, but they should be quite straightforward to fix.

thanks checking into it

fixed in v0.10.1.


Tracking issue for splitting big transaction

Is your feature request related to a problem?
 In the current data flow link (fuller --> sorter --> mounter --> sink), the sorter is equivalent to an infinite reservoir, and its output transaction events can be formalized as
pre_resolved_ts, rawkv_1, rawkv_2,rawkv_3 ...... rawkv_n, resolved_ts
The following problems may occur in the case where n takes a large value.

OOM problem: Sink flushes data downstream only when it receives resolved_ts, so multiple change events will be piled up in memory; meanwhile, since mounter decodes rawkv events as RowChangedEvent, it will further increase memory consumption.
Sink latency problem: In the current implementation, MysqlSink reorganizes the above change events into one transaction based on the StartTs of RowChangedEvent; for a single transaction, MysqlSink uses only one worker to write downstream in order to ensure the ACID characteristics of single-table transactions, which limits the throughput of the sink module in large transaction scenarios.

Describe the feature you'd like
Solve OOM and latency problems by splitting transactions and writing to downstream in multiple batches.
Related Issues

[x] https://github.com/pingcap/tiflow/pull/5203
[x] #1683
[x] #5280
[x] #5398
[x] #5453
[x] Make mq sink support split transactions
[ ] Make redo log compatible with the transaction splitting mechanism
[x] Add integration tests to cover big transaction scenario
[x] add split-txn switcher

/label affects-6.1


Bump React peer dependency to v16

I've been using react-twitter-auth with react@16 for a month now, without any issues.

Coverage remained the same at 16.667% when pulling 33a3ec949531361110beb38a6256e72071b4dc0b on feimosi:patch-1 into 091d860a6a6db29d349d291518a75e76a00624c8 on GenFirst:master.

Coverage remained the same at 16.667% when pulling 33a3ec949531361110beb38a6256e72071b4dc0b on feimosi:patch-1 into 091d860a6a6db29d349d291518a75e76a00624c8 on GenFirst:master.

Hi @feimosi,
Thank you for you contribution. I will do some testing and merge it.

Any updates?

I am currently on vacation. During December and January I had too much obligations, so I haven't finished this. Sorry :(
I have some concerns about backword compatibility. Will check that when I get back from vacation.

If I accept your change then this package will stop working with projects that have react 15.5.x. But will work with 16.6.3. I looked at npm documentation. It seems that both case would work if we would rewrite this:
  "peerDependencies": {
    "react": ">=15.5.0",
    "react-dom": ">=15.5.0",
    "prop-types": ">=15.5.0"
  },

What do you think about that?

My bad, you're totally right. I forgot about backwards compatibility. Actually you can use a range to make sure it won't break with React v17:
 "peerDependencies": {
    "react": "> 15.5.0 < 17",
    "react-dom": "> 15.5.0 < 17",
    "prop-types": "> 15.5.0 < 17"
  },

Tested at https://semver.npmjs.com

Ok, cool. I am totally fine with this changes. Can you push that changes so I could merge them into master?

Done, you can squash these two commits while merging.


Created INSTALL.md file

Moving the TPM 2.0 installation instructions to an INSTALL file and pulling them out of the README.

Closing to fix a git screw-up.


List of assets and owners

Probably not wise to make public for security reasons, but we should have a list of Typelevel assets and who manages them, so we know who to ask and what to transfer as people come and go.  A doc shared among the steering committee would be a good first pass.

I created a Google doc, private for security.  @typelevel/steering and emeritus can ping me for access.


Losing text after footnote

With the following paragraph, the text beginning "Graphics are..." does not appear in the HTML.
	<para>Papers
			typically start on an odd-numbered, right-hand page: this could just follow from
			right-hand pages being the ‘front’ of a leaf, or it could be a by-product of
			single-article reprints of papers needing to start on a right-hand page.<footnote>
				<para>Magazines and newspapers start articles as a two-page spread when it suits them, but in my limited experience, papers in journals and in conference proceedings do not start with a two-page spread.</para>
			</footnote> Graphics are typically floated to the top
			(and sometimes also to the bottom) of the page. In a two-column paper, graphics can be
			either one column wide or span both columns. Large graphics might instead be grouped at
			the back of the paper, after the references. The combination of a two-column layout plus
			floating figures generally takes fewer pages than if all text, headings, and figures
			span the width of the page.</para>

Generates:

Papers
			typically start on an odd-numbered, right-hand page: this could just follow from
			right-hand pages being the ‘front’ of a leaf, or it could be a by-product of
			single-article reprints of papers needing to start on a right-hand page.Magazines and newspapers start articles as a two-page spread when it suits them, but in my limited experience, papers in journals and in conference proceedings do not start with a two-page spread.

Can you provide a sample document that demonstrates this? On its own, that paragraph seems to format just fine as HTML:

and PDF:

The document is https://github.com/tgraham-antenna/MUK-proc/blob/css-proc/proceedings-css.xml, which XIncludes https://github.com/tgraham-antenna/MUK-proc/blob/css-proc/papers-2021/graham/paper.xml
If you want to regress and run Ant rather than Gradle, the command that I use with the css-proc branch is:
ant -emacs -f build-xslTNG.xml gui.single

This expects the xslTNG clone, with gradlew zipStage having been run, in a sibling directory to MUK-proc.

Interesting. A quick test from those sources also doesn't reproduce the problem. I can't imagine how it's related to XInclude, and yet...still investigating.

I've got the css-proc branch, but there seem to be files missing:
ant -emacs -f build-xslTNG.xml gui.single
Buildfile: /Volumes/Projects/docbook/MUK-proc/build-xslTNG.xml

html.prep:

single.basename:

html.single.uptodate:

html.single:

css.prep:

BUILD FAILED
/Volumes/Projects/docbook/MUK-proc/build-xslTNG.xml:151: /Volumes/Projects/docbook/MUK-proc/xsl/img does not exist.

Total time: 0 seconds

If I just create xsl/img to make that error go away, there's an error attempting to copy the dejavu fonts.

Right. This is (another) bug in unwrap-paragraphs. I have to go out for a bit, but I'll try to fix this today.

There would have been a --recurse-submodules to clone xsl that I had forgotten about because I always started from an existing MUK-proc clone and just added/pulled the css-proc branch.  The instructions in README.md for working with the muk2019 branch should apply to the css-proc branch.


ruTorrent: Enable 10gbps speed throttles

This commit increases the max torrent client speed to 10Gbps, while the ruTorrent throttle plugin is active. This is a follow up to the libtorrent throttle patch.

This isn't working properly yet. I'll reopen it if something changes.


release of v0.1.0-rc.0

Includes:

Compatibility with Material-UI v1
The option of not having AppBar


Fix Archive Number

Fixes #64.

Typing a number in the archive field does not remove the validation
message. This patch removes the pattern property as it is not needed.
This will allows the field to validate properly, removing the
validation message when an integer is entered. Letters cannot be typed
into the input field.

Signed-off-by: Will Johnson<EMAIL_ADDRESS>

Codecov Report

Merging #68 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #68   +/-   ##
=======================================
  Coverage   95.01%   95.01%
=======================================
  Files         301      301
  Lines        3971     3971
  Branches      489      489
=======================================
  Hits         3773     3773
  Misses        177      177
  Partials       21       21

Impacted Files
Coverage Δ

source/iml/hsm/add-copytool-modal.js
100% <ø> (ø)
:arrow_up:

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 5703ea2...a3df8fc. Read the comment docs.


contains()

Member functions contains(const Key&) and contains(const Key&, size_t hash) were added to standard associative containers in C++20.

https://wg21.link/p0458r2
https://wg21.link/p0920r2
https://en.cppreference.com/w/cpp/container/unordered_map/contains

Adding these functions will improve compatibility with the latest standard features.

Hi,
Yes, it's effectively a good idea to add them. I made the changes.


Installing .deb when built on Debian 10 appears broken

Hi all,
I've been successful in compiling and packaging into a .deb, however when attempting to install it, I get the error
dpkg: error processing archive dlep-2.1.amd64.deb (--install):
 parsing file '/var/lib/dpkg/tmp.ci/control' near line 6:
 missing 'Package' field
Errors were encountered while processing:
 dlep-2.1.amd64.deb

This only happens when building on a Debian 10 64-bit system. Building and packing from an Ubuntu 18.04 64-bit machine, all works just fine. I've attempted playing with the CMakeLIsts.txt and build/CPackConfig.make files, but haven't had any luck.
Viewing _CPack_Packages/Linux/DEB/dlep-2.1.amd64/control has a Package field set, granted it's not near line 6.
Both systems are up to date with all requirements installed.

I took a look this morning. I can reproduce this issue. This is either something with how Debian 10's dpkg tools are handling this or how cpack in Debian cmake 3.13.4 is creating the package. Ultimately, when the package description is being read into a variable, there is a stray newline being added and that is malforming the control file.
I'd like to look at this more closely when I have more time, but here is a workaround for now.
In the top level CMakeLists.txt, after the line:
FILE(READ "${CMAKE_CURRENT_SOURCE_DIR}/package/packagedesc.txt" PACKAGEDESC)
add the line:
STRING(REGEX REPLACE "\n$" "" PACKAGEDESC ${PACKAGEDESC})
and rerun make package
The resulting deb should now be installable.

You may need to re-run your cmake command before make package as well.

Much appreciated, thank you!


UI issues with new container details page

Rancher Version: master 8/4
Steps to Reproduce:

[x] Can we make font type and size match inside each tab and also across the tabs?

[x] Health Check - Health Check: and Port: should be separated by a line

[x] All pages  - Can the "None" field result be light grey and all lowercase "none".  So it matches the n/a type of field result.

[ ] Command - the User: results is grey?

[x] Scheduling - Change text to simply say "No scheduling rules"

[x] Scheduling - change format of how rules are displayed so it looks nicer

https://github.com/rancher/ui/pull/809

Version - master 8/9
Verified fixed


Use new io for dataset saving

This PR uses the new helper function gli.io.save_graph() and save_task_node_classification() to save CORA dataset. These new functions will be encouraged to use rather than saving manually.
Description

cora.ipynb: Replace the old saving method by the aforementioned new functions.
*.json: Generated automatically.

Related Issue

First step of #437. We will update other datasets as well in the near future.
Motivation and Context

We would like to provide a more clean and user-friendly interface for dataset contribution.
How Has This Been Tested?

Tested locally shown in the cora.ipynb.

The test is unsuccessful due to the server is down temporarily. Will merge the pr after server is up.


Remove holes from some authority boundaries

South Cambridgeshire, Derbyshire, Leicestershire, Nottinghamshire, and Staffordshire have holes in the polygon. For the purposes of the route snapper, that's confusing, because routes can't pass through an invisible boundary in the middle. Fix by removing those holes.
Regenerating route snapper files in those areas now.

Would be good to still show the county boundaries, holes and all, but to allow the routing to go through the city centres. Same concept applies to the previously used buffers: a x km (e.g. 5km) 'routing buffer' outside each authority could encourage inter-region projects.

Where do we want to still show the holes -- on the choose area page, in the scheme sketch, or both? If we show a hole in the scheme sketch, it may be confusing why routing works inside it. Also we've never done that due to how we draw the faded area, and I'm not offhand sure how to start doing it without hunting for some more turf boolean ops packages.
On the choose area page it could be mildly helpful. Maybe we should just sort the areas, so that the smaller ones show up first.
What's the purpose of communicating that a particular boundary has a hole in the middle?

Because that's the authority boundaries. Maybe worth asking users their preferences. However, if they can see the boundaries in another way, e.g. with toggle of boundaries via atip browse mode, that solves the issue. Not a blocker for me, just be aware that there may be some downside to this, although pretty small and the upside is likely bigger. Probably should have phrased as a question: no objection from me if users are OK with it.
My question about routing x distance beyond the boundary probably belongs as a question issue..


Missing XML documentation for .NET classes and members

There is no XML documentation for .NET classes and members. It is needed to switch between VS and web browser just to read purpose of something in XMLUnit.

They are inside the binary release archives (only in the Debug folder as I've just realized). I naively assumed XML docs would go to Symbolsource, seems I was wrong.

I'll look into adding the xml docs to the nuget packages and making them available inside the release folder for the binaries as well. Will cut a new XMLUnit.NET release once I'm done.

Thanks it will be very useful.

May take a few days until I find some cycles, though.

I've just released 2.3.1 the binary distribution and the nuget package should now contain the xml docs.


关于在fairmot框架中简易使用现有模型的问题

目前基本所有的fairmot框架的检测网络都采用DCN结构，对于一些其他普遍性的检测网络，如何修改能够直接在fairmot上运行呢，比如最基础的resnet网络，我苦于无法将改进的网络结构应用在此框架之下。

Hope you don't mind my reply in english:
DCN in FairMOT is usually used to replace the regular Conv2D layer during upsampling of backbone features. So, if you don't want to use DCN, you can just replace any DCN layer back with Conv2D. For example, the resnet_dcn here imports DCN but does not use it in the upsampling layer. This might be a good reference for you.
For my implementation of FairMOT, DCN is not used in the YOLOX detector at all. So you can modify the featurefusor class in the reID head to avoid DCN and see how it performs. Hope this helps.

谢谢,我就是想把resnet+attention添加到fairmot中,但是无法实现正确的生成模型,在你举例的网络resnet_dcn 260line 最后一个forward如果不使用x=self.deconv_layers(x),怎样才能返回合适的x,来生成model.(有些冒昧,谢谢你)

Are you referring to how to get the upscaling to the correct heatmap size? The main layer that upsamples the features is the ConvTranspose2D layer with stride 2. It will basically 2x the image size.


Link to  Core Assistant template gets 404

The bad link is right under the heading "Core Assistant Bot template".

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ae86dcdb-fc63-630a-e45f-a90a41a9e328
Version Independent ID: 60722ca6-74a1-8a73-4246-ffb352bb4814
Content: Templates available in Bot Framework Composer
Content Source: composer-docs/concept-templates.md
Service: bot-composer
GitHub Login: @emgrol
Microsoft Alias: v-eolshefski

Thanks Bruce, a link fix should go live shortly.

Thanks for letting us know, the fix should go live shortly.


app implicit is null in GuiceOneAppPerTest

Hi Scalatest team,
I seem to have the issue that the app implicit is null when using the GuiceOneAppPerTest trait for testing. When I use GuiceOneAppPerSuite it is fine. I discovered this when investigating why my dependency injection in the test did not work as expected: Link
Scala version 2.12.3, play 2.6.3 and scalatestplus-play 3.1.2.
To reproduce:

Create a new play project: sbt new playframework/play-scala-seed.g8
Add a simple println into the test which uses GuiceOneAppPerTest: println("App object", app)

Expected result: app is defined
Actual result: app is null
If you could please investigate that is is really an issue or wrong use from my end or the g8 template. Thank you very much for this great framework!

Hi @christianri, you mean something like this?
class MyTestSpec extends PlaySpec with GuiceOneAppPerTest with Injecting {
  println(s"The app = $app")

  ...
}

Yes, like this (or as I wrote in the ticket above: println("App object", app))
I understand that you initialize app once in the actual test, where also the controller is set. But shouldn't it be set already at this stage with the first one?

Perhaps you could explain your use case a bit more?

My use case is that the play application also opens a java.net.ServerSocket.
It seems that the application is never shut down to a point where the socket is closed via the standard shutdown mechanism and as such I currently rely on a call to close the socket between tests, otherwise I get an exception that this port is already in use.

@christianri thanks for the additional information.

My use case is that my play application also opens a java.net.ServerSocket.

How is the ServerSocket being opened? Do you have a Guice module to open it? If so, are you using play.api.inject.ApplicationLifecycle to register a stop hook? This would be the correct way to do it:
class MyServerSocket @Inject()(applicationLifecycle: ApplicationLifecycle) {
  val serverSocket = ??? // open the server socket somehow
  applicationLifecycle.addStopHook { () =>
    serverSocket.close()
    Future.successful(true)
  }
}

That way, Play will handle the ServerSocket life cycle together with your application.

Yes, I have it like this, the stop hook is registered. In addition, the controller that opens the socket is also started with the App via the task module ( bind[Controller].toSelf.eagerly() ).
I tested a bit further with adding a println into the stop hook code.
Once I run the test with GuiceOneAppPerSuite the stop hook is only called once, at the end of all the tests as expected.
When I run with GuiceOneAppPerTest, the stop hook is called after the test, yet the new app cannot bind to the port as it is still in use. On the surface this seems like a race condition, where the tests that execute quickly after one another don't wait for the Future of the stop hook - or the Play framework closes the app without waiting for it. Could this be the case?


Added support for strided max & average pooling

This pull request adds support for strided max & average pooling, with a numpy solution. It's not optimal, however I don't know if its possible to do with just reshape ops, so had to resort to it. Only uses the numpy solution if stride is specifically specified.
Alternatively max & avg pooling ops could be first class ops, with CPU & GPU implementations with stride support.

So tinygrad proper can't import from an example!
And does this work on GPU?

Yes, I don't know what I was thinking :D Obviously the "core" should be separated. I added couple fixes so it runs on GPU aswell, that being said, it obviously won't run any faster on GPU, since the pooling is done on CPU by numpy...
I recall you saying that tinygrad would in the future run on custom accelerated hardware, is this still the case? Will tinygrad be used in production? Because if so, strided max & avg pooling should probably be implemented as standalone ops, to allow for custom accelerators to be added.

Why did the names change?

@geohot So I thought about backwards compatibility when making max & avg pooling into first class ops, and there are currently 3 options: (1) either we keep the max_pool2d function in the tensor class, and from there call the actual CPU/GPU specific functions, or (2) name the classes Max_Pool2d and Avg_Pool2d, to have backwards compatibility, without needing to have a separate function for it in the Tensor class, and (3) name it MaxPool2d & AvgPool2d, following the same naming convention as other first class ops, but resulting in breaking changes for tinygrad.
Let me know which solution you think is the best, and I will include it in the next commit.

@geohot I finally got done implementing everything, and as the tests show, it is about 150 lines above the line limit. Even if I changed the pool2d to also do the average pooling backwards pass, it still would be more than 90 lines above the line limit. What do you think? (Also I don't know why GPU tests fail, since they pass on my computer)
A couple things:

I run a software studio (Skosh), and I was checking out your website, and thought that we could pimp out your website for free if you are interested, only thing I would ask for would be a "Made by" link at the bottom, and some goodwill. Same offer goes for Cherry Computer, a revolutionary company needs to have a website that reflects the quality of the company!
On a more "Open Source Software"-note, I love the idea of Cherry Computer, and would like to contribute as much as possible. I had a couple ideas such as parallel computing to run on multiple GPUs (or Cherry chips), or a PyTorch compatibility layer with functions etc. from PyTorch implemented in Tinygrad, making Tinygrad more PyTorch "compatible". If any of these seem interesting, or if you'd rather have me work on the TODO list instead, just let me know!

Hmm, there has to be a cleaner way to do this.

Hmm, there has to be a cleaner way to do this. Can't merge now, it's over the line limit.

I agree, and tried countless implementations, all of which failed when testing with combinations of asymmetrical kernel sizes and strides where the sliding window would overflow "outside" the input data, I can try to reduce the amount of lines, but I can't think of another way that supports asymmetric strides, which are essential for a deep learning library (needed for YOLO models)

So is there a way to separate the stride from the pool?
What if you do a conv first to get the stride, then do the pool?

Seems like an interesting approach, will try it out!

Closing, there's a new simpler one.


Can subscribe to 16 topics max

I am running this library on an Arduino Mega.
I am subscribing to more than 16 topics.
mqtt.subscribe() returns true and the broker also logs a successful subscription.
At runtime only the most recently subscribed 16 topics' callbacks get called. The earlier ones are never called when a message is published on their topic.
Is there a parameter to increase this number or another way around this limitation?
Many thanks already.

I found that editing src/MQTTPubSubClient/util/ArxContainer/ArxContainer.h and changing
`
#ifndef ARX_VECTOR_DEFAULT_SIZE
#define ARX_VECTOR_DEFAULT_SIZE 32
#endif  // ARX_VECTOR_DEFAULT_SIZE
#ifndef ARX_DEQUE_DEFAULT_SIZE
#define ARX_DEQUE_DEFAULT_SIZE 32
#endif  // ARX_DEQUE_DEFAULT_SIZE
#ifndef ARX_MAP_DEFAULT_SIZE
#define ARX_MAP_DEFAULT_SIZE 32
#endif  // ARX_MAP_DEFAULT_SIZE
`
increases this limit. It also does not seem to increase memory a significant amount.

@sven1729 Sorry for the late reply. I've added the configuration to change the callback size. Please use it like:
MQTTPubSub::PubSubClient<128, 32> mqtt;

Please let me know (and if reopen this issue)  if you still have a problem.


Optimize usage of riverpod in sudoku

I feel like there maybe have a better way to combine sudokuNotifier with counterNotifier.

sudokuNotifier own gameStatus, when gameStatus is failed or success, it should notify counterNotifier stop timer get the duration.
counterNotifier own the timer and durations, when receive notification from sudokuNotifier, it should stop timer.


Support private read S3 storage

Describe the solution you'd like
Reference implementation of https://github.com/outline/outline.
add an api, to redirect to s3's url which with access token

Type of feature
Integrations
Additional context
No response

Duplicated with https://github.com/usememos/memos/issues/1191


Fix openstack e2e tests image selection

What this PR does / why we need it:
Named image selection in e2e tests for openstack gave us a lot of headache as images are quite often changed at our provider side. This will ensure that we always get an image to use by default without naming it explicitly.
NONE

/retest

/retest

/retest

/retest

/hold cancel

/lgtm

/retest


Error when using with NextJS App

Hi
I am trying to create a NextJS app using Elysia and did the following steps:

bunx create-next-app
bun add elysia @elysiajs/eden
Create a server.ts file under the src directory and add a default elysia server
Replace page.tsx with the following

"use client";

import { edenTreaty } from "@elysiajs/eden";
import { useEffect, useState } from "react";
import type { App } from "../server";

const app = edenTreaty<App>("http://localhost:3000");

export default function Home() {
  const [data, setData] = useState(null);
  useEffect(() => {
    app.api.test.get().then((x: any) => setData(x.data));
  }, []);
  return
test
;
}

When hovering over the type of the app I get back the return type of any.

Is this because the file is a tsx file?
Thanks for any help!

Hey, It's related to this discord thread: https://discord.com/channels/1044804142461362206/1149835565768900758
it's happening because bun create next-app includes "moduleResolution": "blunder" by default on .tsconfig, and this is causing some typing errors in some libraries, you can fix it by changing "blunder" to "Node" or by importing the full path to edenTreaty
import { edenTreaty } from "@elysiajs/eden/treaty"

That was it! Thanks a lot!


How select multiple dates?

Hello,
I need to set multiple dates as previous selected. Please tell me what I need to do?
Thanks,

Make sure that the calendar is in multi-selection mode
For each day you want to select, call calendar.setDateSelected(myDay, true). It accepts Dates, Calendars, or CalendarDays

I have a list of dates for month, i want to highlight that all dates in calendar,i don't want to use multiple selection, just want to highlight some dates, how to do that?


Enhancement: HuggingFace model format detection and model download

Goal-State/What/Result
A clear and concise description of what the enhancement is and the target goal-state.
Milestones:

M1 <>
M2 <>
...

Why/Purpose
Clear and concise answer to why this should be done now.
By When
A target date for done-done completion of the entire enhancement.
Issue/Spec written and reviewed:
M1:
M2:
...
Done-Done:
Done-Done

[ ] First Principles Driven
[ ] The Algorithm
[ ] PM/Design Review
[ ] UX Review
[ ] Threat Model / Security Review
[ ] Tests
[ ] Documentation
[ ] Samples/Quickstarts/Tutorials
[ ] Release Notes

The Algorithm

[ ] Every requirement questioned?
[ ] Delete (Scope) any part you can.
[ ] Simplify.
[ ] Break down into smaller iterations/milestones.
[ ] Opportunities for automation.

Spec
Basic specification of the enhancement.
How/MAP
A clear and concise plan of how this should be implemented.
Implementation (issues)

QA

@Jeadie to close this if not needed

We currently have support for users to specify which model within a HF repo to use, and support for multiple model formats. This is no longer a priority


feat: add migration api which allows impersonation of users

Description

Related Issue

Fixes <issue_link>

Motivation and Context

How Has This Been Tested?

test environment:
test case 1:
test case 2:
...

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

@DeepDiver1975 @kobergj has this PR been finished in https://github.com/owncloud/ocis/pull/9755 already?

Yes sir!


install: don't render role / rolebinding when agent disabled

I missed excluding these roles when the agent is disabled.
Fixes: 0dbde96d
Fixes: #23819

Test failure looks like a flake; the apiserver went away during a test run; Given that Cilium was successfully installed, this can't have broken it.

/test

The only missing tests aren't relevant (ci-verifier, ci-l4lb). Marking as ready to merge.

My mistake, missing @tommyp1ckles review :-)

marking ready-to-merge, no need to run ci-l4lb or ci-verifier. Those don't even use the helm chart.


[Compile error framework-res.apk] Resource entry Theme.Holo already has bag item textEditSuggestionItemLayout.

Information

Apktool Version (apktool -version) -  2.5.0
Operating System (Mac, Linux, Windows) - Windows 10 Pro
APK From? (Playstore, ROM, Other) - AOSP Android 9.0 for ASUS_X00LD

Stacktrace/Logcat
Compiling APK file... (Press ESC to cancel)
/c "java -Xmx1024m -jar "C:\Program Files (x86)\APK Easy Tool\Apktool\apktool_2.5.0.jar" b -f -o "C:\Users\Administrator\Documents\APK Easy Tool\2-Recompiled APKs\framework-res.apk" "C:\Users\Administrator\Documents\APK Easy Tool\1-Decompiled APKs\framework-res" "
I: Using Apktool 2.5.0
W: Could not find sources
I: Building resources...
W: C:\Users\Administrator\Documents\APK Easy Tool\1-Decompiled APKs\framework-res\res\values\styles.xml:1184: error: Resource entry Theme.Holo already has bag item textEditSuggestionItemLayout.
W: C:\Users\Administrator\Documents\APK Easy Tool\1-Decompiled APKs\framework-res\res\values\styles.xml:1183: Originally defined here.
W:
W: C:\Users\Administrator\Documents\APK Easy Tool\1-Decompiled APKs\framework-res\res\values\styles.xml:1260: error: Resource entry Theme.Holo already has bag item preferenceFrameLayoutStyle.
W: C:\Users\Administrator\Documents\APK Easy Tool\1-Decompiled APKs\framework-res\res\values\styles.xml:1259: Originally defined here.
W:
W: C:\Users\Administrator\Documents\APK Easy Tool\1-Decompiled APKs\framework-res\res\values\styles.xml:1269: error: Resource entry Theme.Holo already has bag item textEditSuggestionContainerLayout.
W: C:\Users\Administrator\Documents\APK Easy Tool\1-Decompiled APKs\framework-res\res\values\styles.xml:1268: Originally defined here.
W:
W: C:\Users\Administrator\Documents\APK Easy Tool\1-Decompiled APKs\framework-res\res\values\styles.xml:1271: error: Resource entry Theme.Holo already has bag item textEditSuggestionHighlightStyle.
W: C:\Users\Administrator\Documents\APK Easy Tool\1-Decompiled APKs\framework-res\res\values\styles.xml:1270: Originally defined here.
W:
brut.androlib.AndrolibException: brut.common.BrutException: could not exec (exit code = 1): [C:\Users\ADMINI~1\AppData\Local\Temp\brut_util_Jar_58334102865100678537102944826996642900.tmp, p, --forced-package-id, 1, --min-sdk-version, 28, --target-sdk-version, 28, --version-code, 28, --version-name, 9, --no-version-vectors, -F, C:\Users\ADMINI~1\AppData\Local\Temp\APKTOOL5357856494746311717.tmp, -x, -e, C:\Users\ADMINI~1\AppData\Local\Temp\APKTOOL9110140203189688992.tmp, -0, arsc, -S, C:\Users\Administrator\Documents\APK Easy Tool\1-Decompiled APKs\framework-res\res, -M, C:\Users\Administrator\Documents\APK Easy Tool\1-Decompiled APKs\framework-res\AndroidManifest.xml]
Compile successful.
Please read the FAQ and troubleshooting
https://forum.xda-developers.com/t/tool-windows-apk-easy-tool-v1-57-12-june-2020.3333960/#post-65775601
Include stacktrace here

Steps to Reproduce
I used Easy APK Tool, basically what I am trying to do is changing the minimum screen brightness in the framework-res.apk. The values are in  framework-res\res\values\integers.xml. Decompiling has no problem, but when I tried to recompile the APK it shows up this error. I've tried to decompile and recompile the apk without doing any changes at all, but this error shows up.  I've also tried to use previous version of APK tool, and by using the CMD method but it's still showing up this error. I believe that this is a bug from the original file itself because I did not modify the file 'styles.xml'. From the issue #2290 posted by @Magendanz, it is fixed by deleting the doubled entries in the file, but I have no idea which one is it. How do I fix it?
APK
If this APK can be freely shared, please upload/attach a link to it.
https://mega.nz/file/ZoMmGBza#q4uFbotDT_N2V66fXfVthqN6I9Ha-k0RYd6xvRieBAU
Questions to ask before submission

Have you tried apktool d, apktool b without changing anything? No,
If you are trying to install a modified apk, did you resign it? No.
Are you using the latest apktool version? Yes.

@icypeach
replace the original aapt's in apktool with these to work with framework-res.apk:
prebuilt.zip
# java -jar apktool_2.5.0.jar b framework-res --use-aapt2
I: Using Apktool 2.5.0
W: Could not find sources
I: Checking whether resources has changed...
I: Building resources...
I: Building apk file...
I: Copying unknown files/dir...
I: Built apk...

@icypeach
replace the original aapt's in apktool with these to work with framework-res.apk:
[redacted]
# java -jar apktool_2.5.0.jar b framework-res --use-aapt2
I: Using Apktool 2.5.0
W: Could not find sources
I: Checking whether resources has changed...
I: Building resources...
I: Building apk file...
I: Copying unknown files/dir...
I: Built apk...

I've removed the link. I don't trust sharing prebuilt binaries. Please put a pull request forward to change aapt binaries if you want to put forward a change.

@iBotPeaches
I cannot check your aapt's, they do not work on my old computer, I informed you about this earlier;
excluding ssse3 & sse4.1 optimization for zlib would solve the problem, but slow down performance on newer processors;
external/zlib is not included in platform_frameworks_base, but is included in aapt's, so I compiled aapt's for myself separately, with the zlib patch;
showed the build command and the tool it used, i.e. compiled by me aapt, so that
icypeach can check that the command with the --use-aapt2 flag really works (whether it works on yours - I don't know):
I apologize for the thoughtless advice to replace aapt @icypeach

@pashamcr That's very kind of you trying to help, but nevermind I've figured myself out on how to fix it.
I believe that everyone on stock AOSP 9.0 will face this problem when trying to compile the APK, as this bug is from the original file from Google itself. So here's the fix:

Open the file styles.xml in \framework-res\res\values using any text editor (Windows Notepad will do)

Use the find option to find these entries:
@layout/text_edit_suggestion_item
@style/Widget.Holo.PreferenceFrameLayout
@layout/text_edit_suggestion_container
@style/TextAppearance.Holo.SuggestionHighlight

Each of these entries will have a duplicate just below it, and just delete them

Save the file and try to compile the APK

But another problem comes after I replaced the original framework-res.apk, which is my phone stucked on black screen after reboot (It passes the bootanimation). Basically now it's on bootloop(and I don't really mind cause I've made a backup). I guess the systemui failed to launch. I've done readings on XDA and Reddit before posting this comment, and people have been saying that the modified APK should be signed. I've signed the APK and done the zipalign successfully. Where did I went wrong? And is it appropriate to ask this here, or should I close this issue and create another?

I cannot check your aapt's, they do not work on my old computer, I informed you about this earlier;

Could you put forward a patch? Its hard for me to track things that aren't a PR waiting. Just too much output.

But another problem comes after I replaced the original framework-res.apk, which is my phone stucked on black screen after reboot (It passes the bootanimation). Basically now it's on bootloop(and I don't really mind cause I've made a backup). I guess the systemui failed to launch. I've done readings on XDA and Reddit before posting this comment, and people have been saying that the modified APK should be signed. I've signed the APK and done the zipalign successfully. Where did I went wrong? And is it appropriate to ask this here, or should I close this issue and create another?

Generally the logcat will point at what went wrong, but it won't be easy to understand exactly.

I'm going to dig into why this is duping, since this is part of the resources.arsc. It won't be as easy to dump out, but seeing this:
        <item<EMAIL_ADDRESS>        <item name="textEditSuggestionItemLayout">@layout/text_edit_suggestion_item</item>
        <item name="textEditSuggestionItemLayout">@layout/text_edit_suggestion_item</item>
        <item name="actionModeSelectAllDrawable">@drawable/ic_menu_selectall_holo_dark</item>

A clear duplicate is just strange.

But another problem comes after I replaced the original framework-res.apk, which is my phone stucked on black screen after reboot (It passes the bootanimation). Basically now it's on bootloop(and I don't really mind cause I've made a backup). I guess the systemui failed to launch. I've done readings on XDA and Reddit before posting this comment, and people have been saying that the modified APK should be signed. I've signed the APK and done the zipalign successfully. Where did I went wrong? And is it appropriate to ask this here, or should I close this issue and create another?

Generally the logcat will point at what went wrong, but it won't be easy to understand exactly.

I'm going to dig into why this is duping, since this is part of the resources.arsc. It won't be as easy to dump out, but seeing this:
        <item<EMAIL_ADDRESS>        <item name="textEditSuggestionItemLayout">@layout/text_edit_suggestion_item</item>
        <item name="textEditSuggestionItemLayout">@layout/text_edit_suggestion_item</item>
        <item name="actionModeSelectAllDrawable">@drawable/ic_menu_selectall_holo_dark</item>

A clear duplicate is just strange.

i had the same issue is there no othe way to recompile it?

Resolved with: https://github.com/iBotPeaches/Apktool/commit/86340503acd4f81d8b3decc1f60f35b30faa8d72

Resolved with: 8634050

not solved i have still the same error


Less and SCSS filters in pure PHP

Hello,
what do you think about using Less and SCSS compilers written purly in PHP?
It would make the code less dependent on installing NodeJS or Ruby.
Less: https://github.com/leafo/lessphp
Scss: https://github.com/leafo/scssphp
If you agree I can send a PR.

e.g. this works:
use Leafo\ScssPhp\Compiler;
use MiniAsset\Filter\AssetFilter;

/**
 * Pre-processing filter that adds support for SCSS files.
 */
class ScssFilter extends AssetFilter
{

    protected $_settings = [
        'ext' => '.scss',
    ];

    /**
     * Runs SCSS compiler against any files that match the configured extension.
     *
     * @param string $filename The name of the input file.
     * @param string $input The content of the file.
     * @return string
     */
    public function input($filename, $input)
    {
        if (substr($filename, strlen($this->_settings['ext']) * -1) !== $this->_settings['ext']) {
            return $input;
        }

        $scss = new Compiler();
        return $scss->compile($input);
    }
}

There are already filters for both.

ScssPhp and Lessphp.

Hello, sorry, I've missed this changed. Anyway Scss library https://github.com/leafo/scssphp is updated and now uses namespaces as you can see also from my example. The compiler class is new \Leafo\ScssPhp\Compiler() instead of former new scss().


DaVinci Resolve Recipes

Hey @davidschoefberger, would you consider reverting commit bf00b53b0eb8a825214226da0d1dfa523b4a6af6? We were using the DaVinci Resolve recipes and found them to be working well enough for our use case. Thank you for having written them in the first place.
If you're not interested in maintaining the recipes I could migrate them over to my autopkg repo. Thanks.

Hi @nstrauss,
sure - just reverted it! How do you use them? Since in the current state they did not work in the past.

Ha. Long as the registration info is there it's been pulling the latest version. Coming into this from someone else having set up the recipe. Been working as long as I can tell. Thanks for reverting. If it breaks in the future I can submit a PR.


Main andrea2

👷 Deploy Preview for gladia processing.

Name
Link

🔨 Latest commit
859511d76abaed6962ccaa01db537b311846d1be

🔍 Latest deploy log
https://app.netlify.com/sites/gladia/deploys/642b44592579fb00077b4071


Adds file contents preview to the File Size Inspector debug screens

Task/Issue URL: https://app.asana.com/0/414235014887631/1205515529991083/f
Tech Design URL:
CC:
Description:
Allows the contents of .db, .sqlite, .sqlite3 and .plist file types to be previewed in app

Steps to test this PR:

Open a few tabs & visit a few websites
Navigate to Settings > Debug > File Size Inspector and tap into some of the .db, .sqlite, .sqlite3 and .plist file types and confirm the contents can be viewed e.g. in Application Container > Library > Caches

Copy Testing:

[ ] Use of correct apostrophes in new copy, ie ’ rather than '

Orientation Testing:

[ ] Portrait
[ ] Landscape

Device Testing:

[ ] iPhone SE (1st Gen)
[ ] iPhone 8
[ ] iPhone X
[ ] iPhone 14 Pro
[ ] iPad

OS Testing:

[ ] iOS 14
[ ] iOS 15
[ ] iOS 16

Theme Testing:

[ ] Light theme
[ ] Dark theme

Internal references:
Software Engineering Expectations
Technical Design Template

This is a great thing to have, nice!
I did encounter one issue:

Thanks @THISISDINOSAUR!
It seems like something is off with that one particular db - it's either not SQLite or else it must be encrypted. I tried viewing it with DB Browser for SQLite (which can read every other database file in the app container) and get this error when trying to access it 🤷♀


Documentation entry for SSH and persistPaths are swapped

What happened?
Online documentation for ssh and persistPaths are swapped

Local Environment:

DevSpace Version: [use devspace --version]
Operating System: windows | linux | mac
ARCH of the OS: AMD64 | ARM64 | i386
Kubernetes Cluster:
Cloud Provider: google | aws | azure | other
Kubernetes Version: [use kubectl version]

Anything else we need to know?

/kind bug

@dhruvbaldawa thanks for reporting this issue! You are correct, we need to change that

@dhruvbaldawa should be fixed now


Configures version updates for GitHub Actions

Context
Actions are often updated with bug fixes and new features to make automated processes more reliable, faster, and safer. We should enable Dependabot version updates for GitHub Actions so Dependabot will help ensure that references to actions in a repository's workflow.yml file are kept up to date.
Ensure dependabot is enabled consistently across BAT services based on the existing find implementation here
Changes proposed in this pull request
Addition of github-actions as a package-ecosystem to monitor with Dependabot
Guidance to review
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 Enable dependabot for Github actions


Adding general response certainty based on median log probability

Please manually test before approving.

Otherwise LTGM and very nice :clap:


Fix Policy update and create bug when description is None.

Policy create and update routines contain a code block which tests for the
presence of function parameters to determine json payload. As currently
implemented both the "rules" and "datacenters" fields are determined by
whether the "description" parameter is defined.

@snesbittsea thank you 👍


Parse_decl2 too many arguments error

This line fails because the current parse_decl2 has one less argument now:
https://github.com/fireeye/flare-ida/blob/master/python/flare/apply_callee_type.py#L67
Taking out "blah" still throws an error, and I'm not sure why.

Thanks for reporting the error. I think IDAPython's SWIG usage may have changed the behavior of calling parse_decl2 since this was originally written. I think I'm now using parse_decl2 correctly. Please try the latest in master.

@jhsmith Well, it didn't crash which is good.  But it didn't change my call at all:

What are the manual steps to apply the same type of typing this script does?  I'd like to see if I can do it manually to see if it is even possible.  I can't find, or I'm not aware of, anything in the GUI that will let you do the same thing.

I should note - those commends in there are ones I wrote.

Did IDA pop up an error box about the type declaration being invalid? What function prototype did you try to apply at the address? Can you paste it here in a message?

No popup errors.  This is what I defined it as:
int LdrLoadDll (
  PWCHAR               PathToFile,
  ULONG                Flags,
  PUNICODE_STRING      ModuleFileName,
  PHANDLE             ModuleHandle );

Are there manual steps I can try to see if it is even possible to add this?  I can't find similar functionality through the GUI.

It seems to work for me. I tried it out on a random 4-argument function in an idb. See here pictures attached. Try deleting your comments on the lines with the pushes, and make sure the cursor is on the line with the call, and try running it again.
"Are there manual steps I can try to see if it is even possible to add this?" -> that's why I wrote this plugin, because as far as I can tell you can't do this via the GUI, only from scripting/plugins.

Spot on!  Getting rid of the comments was the solution!

I thought maybe I was missing a GUI command somewhere.  I'm glad to know I'm not the only one that couldn't figure it out.  Very nice plugin!  Very useful!  Thank you!


selecting attributes

Laravel Version: #.#
Adldap2-Laravel Version: #.#
PHP Version: #.#

Description:
I would like to get users in an array of this form
[ 0 => ['cn' => '...',"email" => ''.." ] , 1 => ['cn' => '...',"email" => ''.." ] ,...]
didn't find out how to make it work or how to manipulate selected attributes.
Steps To Reproduce:

Hi @celiakessassi, here's detailed documentation on all model methods, as well as retrieving attributes:
https://github.com/Adldap2/Adldap2/blob/master/docs/models/model.md#getting-attributes
For you question, since results are returned in Laravel collections, you can do this pretty easily:
$users = Adldap::search()->users()->get()->map(function ($user) {
    return $user->getAttributes();
});;

thx @stevebauman this what worked for me
$users = Adldap::search()->users()->get()->map(function ($user) {
return ['uid'=>$user->uid[0],'email'=>$user->email[0],'cn'=>$user->cn[0]];
})->toArray();
don't know why but seems that getAttribute don't change anything


make it available as a commonjs module

I have a similar patch I'm working with.  Instead of looking for exports at the end, I wrap the var joli statement in an a create function that returns joli.
The benefit of doing it this way, is you can use joli for two separate connections/databases.
The downside is, this forces Joli to be a CommonJS module.
Any thoughts on how to have it flexable (checking for exports, so it isn't required to be used as a CommonJS module) yet keeping the var joli in a closure to allow for separate joli instances?
e.g., my code is:
exports.create = function() {

    // START of original joli.js
    var joli = {
    ...
    }
    return joli;
};

I think both ideas can join like so:
if (exports) {
    exports.create = function() {
        return joli;
    };
}

Unfortunately, that doesn't allow you to have two separate joli instances, since they're created off the same var.

Maybe:
var joliCreator = function() {
    var joli = { ... };
    return joli;
};
var joli = joliCreator();
if (exports) {
    exports.joli = joliCreator;
}

Then, you can get a new Joli instance by calling require('joli').joli(), and yet joli is still a global var if you're not using CommonJS.

Oh right.
Well u take some parts out from that:
var joli = function() {
    this.each = function() {
        ...
    };
    this.extend = function() {
        ...
    };
    return this;
};
if (exports) {
    exports.joli = joli;
}

Its about the same thing, just a different approach.

Correct. Maybe just one minor change then:
var joliCreator = function() {
    var joli = { ... };
    return joli;
};
var joli = joliCreator();
if (exports) {
    exports.joli = joliCreator();
}

Cool, that works great.

Hi guys,
I have just pushed a variation of your common work :
var joli = {
    // all the previous stuff
};

if (typeof exports === 'object' && exports) {
    exports = function() {
        var api = {
            connect: function(database) {

                if (database) {
                    joli.connection = new joli.Connection(database);
                }

                return joli;
            }
        };

        return api;
    }();
}

This allows to either load the library the way we did before:
Titanium.include('joli.js');
joli.connection = new joli.Connection('your_database_name');

or the new way:
var joli = require('path/to/joli').connect('your_database_name');

Thanks both for the contribution!

Thanks Xavier.  The one change that isn't captured in your update is the ability to use Joli on two separate databases, because var joli = {} is initialized at the root of the module, instead of in a function closure: function() { var joli = ...; return joli}.
The parsing of CommonJS modules may be cached, so the root-level var joli is only ever initialzied once.  With your change, the second call to .connect() will overwrite the settings of previous calls.
For example:
var joli1 = require('path/to/joli').connect('db1');
var joli2 = require('path/to/joli').connect('db2'); // joli1 now points to db2


attribute named "created" doesn't work on model

If model instance does have attribute named created (that come for example from server) this attribute does not work properly.
instance.attr('created') - actually gets the value, but it does not apper in view.
instance.created - is function (relates to 'created' event I believe)
Not sure if it shoud work by design. Ofcource there should be a list of reserved words by can, but I haven't seen such.
As workaround I currently "convert" it to createdAt on the client using define plugin.

Attr, change, created, updated, destroyed, init, setup.
Any event triggered or method already on a model is off limits.
Sent from my iPhone

On Mar 18, 2015, at 1:58 AM, Alex<EMAIL_ADDRESS>wrote:
If model instance does have attribute created (that come from server) this attribute does not work.
model.attr('created') - actually gets the value, but it does not apper in view.
model.created - is function.
Not sure if it shoud work by design. Ofcource there should be a list of reserved words by can, but I haven't seen such.
—
Reply to this email directly or view it on GitHub.

I thought that would just mean you can't access the property directly like model.created. I would expect it to show up in a view if it is available via .attr: http://jsfiddle.net/uc1m3L4j/2/
Is that just a precedence problem that (Mu)Stache is trying to run the function right away instead of looking it up via .attr first?

Yes it would be ok if it worked in views.

A "created" attr can't work in views.  The "created" model event would conflict with the "created" property change events.

Is that just a precedence problem that (Mu)Stache is trying to run the function right away instead of looking it up via .attr first?

No, it's a different problem.

Closing because this doesn't really have a solution.  "created" is a reserved word in a can.Model.

I would you create and update events instead created, updated


Fix docs component details

What kind of changes does this PR include?

Something else!

Description

Fixes two small details in our docs components that have been bugging me forever.

Removes the not-content class from the “About Astro” block on the homepage so that the link is the correct colour.

Before
After

Adds just enough margin to file-tree entries so that when two neighbouring entries are highlighted, they don’t touch.

Before
After

size-limit report 📦

Path
Size

/index.html
5.13 KB (0%)

/_astro/*.js
19.27 KB (0%)

/_astro/*.css
9.72 KB (0%)


Feature requests

I had some ideas of feature requests which could greatly improve the usefulness of your tool.

Enable GPU mode (seems to only run on CPU as of now)
Enable batch mode (seems to assume that the input is a single image)
Add augmentation option, similar as done in StainTools

I would be more than happy to help or assist in any way.

I believe I have added GPU support now. See my fork for my modifications.
I can make a new pull request after my current one has been merged, where we can attempt to address these ideas.
Regarding GPU compute: I did not see any real improvement. Actually for the image size I was testing, I found a degradation in runtime. I believe the benefit of using a GPU would be greater if there were more operations that could be run in parallel.
If a batch of images were used, each image could be processed in parallel using the GPU. This would also be very useful for training CNNs, where one could normalize/augment the patches on-the-fly.
I could also try to improve batch runtime using multiprocessing, but in this scenario the usage of GPU would be less relevant.

Is the code working correctly? I want to use it in my code.

@Surayuth : I have only added GPU support, as of now. If you wish to use that, simply install by:
pip install git+https://github.com/andreped/torchstain

But in the current state, there was no benefit of using the GPU. We need to modify the code to support batches first.
However, I would like to add these modification to the main repository, but I am waiting for the owners/developer to respond.
I am currently working on another project, where I wish to build upon torchstain (or any fast and accurate color normalization approach) to make a simple CLI. For my CLI see here. Currently, you can perform normalization in parallel using multiprocessing. I see some improvement in runtime.
Simply install and use the CLI by:
pip install git+https://github.com/andreped/fast-stain-normalization.git
faststainnorm --ref full-path-to-reference-image --img path-to-images-to-convert --out path-to-store-output

@andreped Thanks for a quick reply and your improvement. I want to use this method in my project too but I also need the whole process to be differentiable which I don't know if it will work in the current state (I will look into the code later if I have time). I'm glad that you are working on this topic too as I really need this normalization method in my work.

Why do you need it to be differentiable? I am assuming you are training neural networks. Do you wish to use this as part of the layer when training?

Yes. I would like to use this to extract a saliency map instead of using existing CNN (i.e., UNet, ...) because I don't have many images for training my model.

I am not sure I follow. How to you wish to use a stain normalization method to produce a saliency map? And are you training a patch-wise model using an ordinary neural network with pixels as input or are you training a patch-wise CNN classifier?
Because for both I believe there are better ways to produce "saliency maps", which I believe you want to produce some type of "unsupervised" segmentation? Or perhaps to explain which regions the network used to perform a task?

Thanks for pointing it out. I just want to extract the stained area unsupervised but I don't have enough images to train the model. The stain variation is too much that the model trained to extract the stained area is not robust. So, I try to find a simple solution to do the job here. Any suggestion is appreciated here. Thanks!

Do you mean that you just wish to segment the image by saying which pixels in the image contains either haematoxylin (H), eosin (E), or background/glass?
Or do you wish to segment any tissue? Such that the final result is a binary segmentation, 1 for tissue and 0 for glass/background?

I wish to segment the concentration of the H region.  The fact that I want this process to be differentiable is because I want to use this H region in the loss function of my networks.

OK, but this method should work fine. Just remember to pick a representative and suitable reference image. That seems to be key for these methods to work well. Depending on your images, you might need to change the luminence threshold as well, or do some preprocessing on your images.
But still, you have some images, which you wish to use to perform some task. What stops you from not just generating the H image in preprocessing and passing that with the original image (if of interest) to the network during training, and then pass the H image to the loss?
There is no need for the Macenko method itself to be differentiable. That method is completely static. Even if you where to add this as a layer at the start of you neural network, this operation does not need to be differentiable. The weights will be updated in the NN and will not be backpropagation further (through the Macenko-stuff), but there is really no need for that either, since there are no weights to update there.

Thanks for the idea. I understand that the method is static. The thing that I worry about at first is because I want to use this method in the loss function. And if I remember correctly, calling backward() will distribute the gradient along with the computational graph. Some computations, in the past, don't allow the gradient to flow to them, i.e., https://github.com/pytorch/pytorch/issues/27036.

What are you trying to do? Are you using the HE image as input to some neural network, which does something and then you are trying to use the H channel in the loss?
You would still backpropagate from the output through the neural network, until you hit the Macenko-layer. So the network would still update.
And yes, there are likely some operations in the Macenko implementation which is non-differentiable. Were you trying to use Macenko inside the loss function, perhaps? Then I can understand your concern, because that will not work!
The trick is to do this outside the loss computation, just provide the already computed H image the the loss computation. It really depends how you have implemented the whole ting.

What are you trying to do? Are you using the HE image as input to some neural network, which does something and then you are trying to use the H channel in the loss?

Yes, the network will output the H channel (result 1) and I want to compare it with the H channel segmented from Macenko method (result 2).

And yes, there are likely some operations in the Macenko implementation which is non-differentiable. Were you trying to use Macenko inside the loss function, perhaps? Then I can understand your concern, because that will not work!

I see. I will look into the code tmr to see what I can do with it then. Thanks for this helpful discussion today. It helps alot.

I have a lot of experience with this from TensorFlow, which really is annoying at times... This is perfectly possible to do without having the Macenko algorithm differentiable. Because extraction of the H image can actually happen outside the graph (at least outside the graph of interest).
No worries! Happy to help :) Let me know how it went! If you'd like, I could help you set this up, if you are willing to share the (private) repository with me.

Thanks! I would love to work with you. And I agree with you that it's possible. Just need some modifications to the code. I will share the repository with you soon. Thanks again!

Thanks for the very interesting discussion.
I agree that having GPU support would be very nice, and that it should be done with a batch-based approach.
@andreped feel free to draft a PR

Btw, very nice work with https://github.com/andreped/fast-stain-normalization @andreped

@carloalbertobarbano I can make an attempt on the batch-based approach next week.

Sounds good

@carloalbertobarbano shall we make an attempt at a new release where we wrap up the TF support thingy and make it stable and working? We could probably setup some workflows for doing all that using github actions. I could make an attempt, if you'd like?

Yes absolutely, feel free to sketch it up. I have been very busy lately.

I remember that there were issues with installing both TensorFlow and PyTorch, and hence, a good idea could be to install torchstain with a specific backend (custom option in pip). Not sure how you would do that, but I could make an attempt.
I will let you know when I have a PR ready for you to test.

Yes that should be avoided. As you already said (I think) this should be handled using the setup options, for example pip install torchstain[tf] or something like that. If no option is provided I'd fall back to pytorch


"My sources" sidebar and page should use siglum

As noted in #438 the listed sources in the My Sources sidebar should use siglum and not rism_siglum.
On the /my-sources page, the table should also use siglum, and not the full shelfmark (to avoid the screenshot below):

mentioned in #438 - evidently the sidebar still doesn't use the siglum. Reopening.


this could be a good resource!

https://github.com/jhudsl/intro_to_R_class/issues/41

did we add https://rafalab.github.io/dsbook/r-basics.html yet?

PR  #157 includes this


feature: add user flag implement in pouch create and exec

add user implement like format uid[user]:gid[group]
support command pouch exec -u and pouch create -u
Signed-off-by: Ace-Tang<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
add user implement like format uid[user]:gid[group]
support command pouch exec -u and pouch create -u
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

@Ace-Tang Could you also add more verifications in test TestRunWithUser to verify this PR works.

@Letty5411 , yes , I can add more case in TestStartWithUser

Codecov Report

Merging #808 into master will decrease coverage by 0.07%.
The diff coverage is 7.61%.

@@            Coverage Diff             @@
##           master     #808      +/-   ##
==========================================
- Coverage    13.7%   13.63%   -0.08%
==========================================
  Files         108      109       +1
  Lines        6492     6586      +94
==========================================
+ Hits          890      898       +8
- Misses       5543     5628      +85
- Partials       59       60       +1

Impacted Files
Coverage Δ

daemon/mgr/container_types.go
15.38% <ø> (ø)
:arrow_up:

cli/exec.go
0% <0%> (ø)
:arrow_up:

daemon/mgr/container.go
2.82% <0%> (-0.05%)
:arrow_down:

daemon/mgr/spec_process.go
0% <0%> (ø)
:arrow_up:

pkg/user/user.go
9.75% <9.75%> (ø)

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 16fb071...b550932. Read the comment docs.

LGTM


128 MB increments is ambiguous

[Enter feedback here]
128 MB increments is ambiguous, does it mean MiB (134,217,728) or MB (128000000), the proper unit for computer memory is MiB.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 84d52296-198d-7c8a-db99-e8cb336e355f
Version Independent ID: 77670399-109f-3233-0466-c13d84542039
Content: Supported Ubuntu virtual machines on Hyper-V
Content Source: WindowsServerDocs/virtualization/hyper-v/Supported-Ubuntu-virtual-machines-on-Hyper-V.md
Product: windows-server
Technology: hyper-v
GitHub Login: @BenjaminArmstrong
Microsoft Alias: benarm

Microsft uses 1MB = 2^20 = 1048576, or in this case 2^27, 134217728 Bytes.

#assign:xelu86

Hey Zixradoom,
Thanks again for contacting Microsoft. We value your feedback.
The Windows OS displays storage capacity and file sizes using the decimal prefix KB, MB, GB, etc. but still calculates this as KiB, MiB, etc.
As this is a non-documentation issue, we'll be closing this thread.
Thank you and have a most pleasant day.
#please-close


Is this supported with .net 6 or 7?

Im wondering if this still works and if it works with the newer versions of .net? has anyone tried it? is there any known issues?

Hi, it still works, six months ago I created an application based on it and WPF.  You can check it below:
https://github.com/su1cidal/DotaVinci/releases
The main problem for me is that it doesn't support events like: killing Roshan, first blood, etc., or I didn't find it😔

Yeah I'm trying it it out atm. Someone added updates to the repository recently with new stuff. But its not included in the package. So I just made my own reference to the project for now.  There is a Roshan-state but it seems like its not available during a real game. Some stuff are only available as spectator.

I know this was closed. But version 2.0.1 was upgraded to .NET 8.0.


imageSettings in character files do not currently work (providing fix PR)

Describe the bug
Image settings in character files do not currently work.
To Reproduce
To reproduce just try setting character file settings like

and then generating an image. You will always get the default settings because they are not being passed from the character file.
Expected behavior
Expected behavior would be for these image settings to be passed on when generating an image.
Screenshots

Additional context
I'm providing a fix right now. PR incoming.

fixed in #1371


Corelation should be Correlation

https://github.com/huggingface/datasets/blob/0e87e1d053220e8ecddfa679bcd89a4c7bc5af62/metrics/matthews_correlation/matthews_correlation.py#L66

Hi @colbym-MM, thanks for reporting. We are fixing it.