run on haybaler output?

Hi @poer-sophia
It would make sense in my mind to run the taxonomy on haybaler output (it runs well enough on the current .fa.fai input).
What do you think ?
Going further, it would make sense to just integrate it as a method/function in haybaler itself.
cheers,
Colin

Yes of course that makes sense. So you think it works good enough and the few species it does not work with are ok for the moment? Because I tried a few things and to get no double TaxIDs to then check the species which havent workes yet but nothing worked. I am still in the process of trying but if you think it works well enough i can first concetrate on implementing it into haybaler.

Yes, it's actually very helpful. Most of those that don't work are viruses and fungi. We can remove and or ignore them. For the most part we are interested in good bacterial identification, fungi are less prevalent. I changed the script so it printed out all non-working spp/genera
and have an issue named:
metagenome crap after 2021_02 and 2020_05 refs


Having crazy hard time trying to set the value of an input when changing the value of another one

Just want to set the value of an input text to empty when clicking on a radio button and giving up after a few hours trying unsuccessfully... Any idea?

@bstenm, what is the main difficulty? Have you checked selecting form values topic of the documentation?

Yes I have and I understand how to access form values but I just don't see how to set an input value programmatically? Am I missing something obvious? Do you have a working example?
Cheers!!

input.onChange(newValue)?

Please check this option as well

hey yes thanks a lot mate, that's what I was trying and the option that made more sense, but for some reason I couldn;t make it work. I;ll try again it's probably sth silly I can't see because that really should work.
Cheers!!


Python packages: Add cmdline flags for macos, change crate names to fix build

I am just fumbling around with Rust so that this seems to work is as much dumb luck as anything that means I know what I'm doing, but I was able to run the test program from the README with these changes on MacOS. There may be a better way.
The crate changes to src/python.rs I think I required no matter what - without them, running
cargo build --lib --release --features "python-library"
fails on Linux (Ubuntu 18.04 VM) and MacOS
The addition to .cargo/config comes straight from the manual at https://pyo3.rs/v0.12.1/
I did not try anything with maturin so for all I know this blows all the pypi & wheel stuff up completely.
I'm on the nightly toolchain
(reason) epaulson:~/development/reasonable $ rustc --version
rustc 1.49.0-nightly (b1af43bc6 2020-10-10)
(reason) epaulson:~/development/reasonable $ rustup toolchain list
stable-x86_64-apple-darwin (default)
nightly-x86_64-apple-darwin (override)

Reasonable (but not the python library) does build without these changes - if I just do a 'make build' I get an executable at the end in the target/release directory.

@epaulson I think I got Actions working for builds; when you get a chance, can you try doing pip install reasonable==0.1.24  and see if that works for you on your Mac?


Storage Cache .NET SDK 2024-01-01, API version 2023-11-01

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.StorageCache

LGTM. Wait for adding back the attributes.

LGTM. Wait for adding back the attributes.

After discussion, we decided to remove those attributes.


Added a aesy to find option to enable rehash.

I created a new pull request to add a easy to find and relative well commented option to enable rehash.

This should be in lib/completion.zsh. Uncommenting that line would trigger conflicts when updating Oh My Zsh. Instead, you can add that as another option in templates/zshrc.zsh-template.


fix(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/cos/ibm (source)
module
patch
7.1.3 -> 7.1.4

terraform-ibm-modules/iam-account-settings/ibm (source)
module
minor
2.5.0 -> 2.6.0

terraform-ibm-modules/observability-instances/ibm (source)
module
minor
2.10.3 -> 2.11.0

Release Notes

terraform-ibm-modules/terraform-ibm-cos (terraform-ibm-modules/cos/ibm)
v7.1.4
Compare Source
Bug Fixes

removed input variables sysdig_crn and activity_tracker_crn from the fscloud submodule as they were unused. If you wish to set a bucket up with metrics or activity tracking, use the optional activity_tracking and metrics_monitoring options in the bucket_configs input variable, as demonstrated in the fscloud example (#490) (00eeab4)

terraform-ibm-modules/terraform-ibm-iam-account-settings (terraform-ibm-modules/iam-account-settings/ibm)
v2.6.0
Compare Source
Features

added new variables fs_validated and user_list_visibility and defaulted them to true since this is required for fscloud. A third variable api_endpoint has also been exposed, allowing the consumer to specify the accounts api endpoint to use. By default it will use the public endpoint accounts.cloud.ibm.com.NOTE: As these features are not available in the IBM terraform provider, the module now requires the Mastercard restapi provider. (#233) (4ded33d)

terraform-ibm-modules/terraform-ibm-observability-instances (terraform-ibm-modules/observability-instances/ibm)
v2.11.0
Compare Source
Features

added the ability to create an auth policy which allows the atracker service object writer access to the COS target. This will only be created if passing a value for the cos_targets input variable, and if service_to_service_enabled is true and skip_atracker_cos_iam_auth_policy is false inside the cos_targets list object. (#433) (653b662)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

As part of upgrade from COS module 7.1, there was an input variable name change in 7.2, had to change this variable name for the cos module call in this source. Should have no further impact.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

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

GitHub release
v1.3.2

Your semantic-release bot :package::rocket:


pvfactors is not using pvlib's convention for array azimuth angles

In order to have a tracker torque tube aligned in the South-North direction

pvfactors requires an array azimuth angle of 90 deg
pvlib requires an array azimuth angle of 0 deg

It would make things easier to align pvfactors to pvlib's convention.

Fixed in #22


Adding a tactile paving as a closed line will make iD render it as an area.

If you add a tactile paving in iD, and choose line type, it will let you draw a line.  If you draw that line as a closed way, however, iD renders it as an area.

So to fix this I changed the tactile_paving preset from matching tactile_paving=* to tactile_paving=yes (because * is ignored for the areaKeys blacklist). But that resulted in #6791, which led to us removing the preset altogether.


Readdocs branch

Ensuring that user's have documentation for model and solution continuously updated via ReadTheDocs

Too many tests failing


operators [O] [CI] strimzi-kafka-operator (0.20.0)

Signed-off-by: J0zi<EMAIL_ADDRESS>Image path fix

/hold cancel

/merge possible

/merge possible


Missing VC++ files required for powershell_exec feature

Description
Windows Server 2016 does not ship with a Visual C++ redistributable by default, leaving it to the package provider to ship a redistributable or the binaries in the base (bin) folder for compiled C++ code (e.g. Chef.PowerShell.Wrapper) to work
Chef Version
14.8
Platform Version
Windows Server 2016 (and assumed 2019)
Replication Case
cookbooks/mycookbook/default.rb
outcome = powershell_exec("Get-WindowsFeature | where { $_.Installed -eq $true }")
if outcome.error?
  outcome.errors.each do |e|
    puts error e
  end
else
  puts "Result: #{outcome.result}"
end

---
driver:
  name: azurerm
  subscription_id: '43706c4e-YOUR-SUBS-CRIP-e64e6bf63ac4'
  location: 'West Europe'
  machine_size: 'Standard_D2_V2'

provisioner:
  name: chef_zero
  retry_on_exit_code:
    - 20
    - 35
  max_retries: 10
  wait_for_retry: 180
  client_rb:
    exit_status: :enabled

platforms:
  - name: windows2016
    driver:
      image_urn: MicrosoftWindowsServer:WindowsServer:2016-Datacenter:latest
    transport:
      name: winrm
      elevated: true

suites:
  - name: default
    run_list:
      - recipe[mycookbook::default]
    attributes:

Client Output
C:\projects\repro\chefwrappertest> kitchen converge
-----> Starting Kitchen (v1.23.2)
-----> Converging <default-windows2016>...
       Preparing files for transfer
       Preparing dna.json
       Preparing cookbooks from project directory
       Removing non-cookbook files before transfer
       Preparing validation.pem
       Preparing client.rb
-----> Installing Chef Omnibus (install only if missing)
       Downloading package from https://packages.chef.io/files/stable/chef/14.8.12/windows/2016/chef-client-14.8.12-1-x64.msi

       resolving cookbooks for run list: ["mycookbook::default"]
       Synchronizing Cookbooks:
         - mycookbook (0.1.0)
       Installing Cookbook Gems:
       Compiling Cookbooks...

       ================================================================================
       Recipe Compile Error in C:/Users/azure/AppData/Local/Temp/kitchen/cache/cookbooks/mycookbook/recipes/default.rb
       ================================================================================

       LoadError
       ---------
       Could not open library 'Chef.PowerShell.Wrapper.dll': The specified module could not be found.

Workaround/fix: copying the files msvcp140.dll and vcruntime140.dll to the c:\opscode\chef\embedded\bin folder resolves the issue

This has been merged to both branches


Generate migrations from resource definition diffs

Eventually, I'll need to generate migrations from changes to the resource model. Resource models can be represented by JSON, so this is a two step process:

Have both old and new resource model
Diff them
Interpret diff, creating migration file from it
Eventually, applying the migration

Some tools that may help with this:
https://github.com/chbrown/rfc6902

This branch begins this work.
Some requirements:

meta needs to be prefixed in the DB to differentiate it from attributes
I will need a deeper understanding of all of the valid postgres types to do a reverse mapping from column description to attribute.type in the Resource Model
I'll need to introduce a schema to be able to get all of the tables. All of the tables are necessary for many-to-many relationships
I'll need to fork pg-metadata to return more info


Save out detector artifacts on detector training

It's a lot of diffs, 1. because I moved some things around.

export_predictions_and_labeled_video method moved from scripts.py to predictions.py.
get_context_img_paths moved to io.py
regenerated api docs.

Because of the tests. I checked in some input files, and the cropzoom expected output files. This way we can visually see the cropping. If we make any intentional changes, all we have to do is copy the test output to the expected_model_output directory.

Addressed all comments! We are off git-lfs, onto figshare.


feat: add max_concurrent_solves configuration

Why
Pixi can possibly use a big amount of memory during the solve. This can be limited by only spawning a limited amount of solve jobs. While we search for a better/automated solution we want to let the user escape the issue by forcing the amount of concurrent jobs.
The related issue is: https://github.com/prefix-dev/pixi/issues/2458
What this PR adds
As a user you can now define the max concurrent solves in two ways
CLI
pixi install -j 3
pixi install --solve-jobs 10
pixi install --max-concurrent-solves 1

configuration
pixi config set max-concurrent-solves 2

config.toml
max-concurrent-solves = 2

TODO:
After initial approval of design I'll add the following:

[ ]: Add documentation
[ ]: Add basic cli and configuration test to the integration tests

Questions for reviewer

What flags should we use? -j is famous in other tools (make) but its not exact same implementation wise, max-concurrent-solves is a mouth full, solve-jobs is limited.

@wolfv merged in https://github.com/prefix-dev/pixi/pull/2388

Thanks! I closed the other one :)


Extension activation fails on Linux

Environment data

VS Code version: 1.56.2
Jupyter Extension version: 2021.6.832593372
Python Extension version (available under the Extensions sidebar): 2021.5.842923320
OS (Windows | Mac | Linux distro) and version: linux
Python and/or Anaconda version: 3.8.5
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Jupyter server running: either

Expected behaviour
Clicking on a Python file should open the file in VSCode
Actual behaviour
File fails to open; instead I get an "Extension activation failed" message related to the Jupyter extension:

Logs

I just replicated this issue on a Mac as well, so not restricted to Linux.

What's the value of the workbench.editorAssociations setting in your settings.json file?
If you set it to the following, this issue should go away:
"workbench.editorAssociations": [
   {
        "viewType": "jupyter-notebook",
        "filenamePattern": "*.ipynb"
   }
]

Yeah, this appears to be associated with contamination of my settings with insider build settings. Removing that fixed the issue.


Implement Set mentor availability endpoint (Admin)

Description:
This issue involves implementing an API to get all mentors. The endpoint should allow Admin to make a PUT request to /api/admin/mentor/{{mentorId}}/availability.
Example mark mentor as available
{
    "mentor_id": 1,
    "created_at": "2023-06-30T10:00:00Z",
    "updated_at": "2023-06-30T14:30:00Z",
    "state": "APPROVED",
    "category": "Technology",
    "application": {
        "country": "",
        "areasOfExpertise": "",
        "expectationsFromMentees": "",
        "mentoringPhilosophy": "",
        "commitmentToProgram": false,
        "previousExperienceAsMentor": false,
        "reasonForBeingMentor": "",
        "cvLink": ""
    },
    "availability": true,
    "profile": {
        "created_at": "2023-06-29T09:15:00Z",
        "updated_at": "2023-06-30T16:45:00Z",
        "primary_email"<EMAIL_ADDRESS>        "contact_email"<EMAIL_ADDRESS>        "first_name": "John",
        "last_name": "Doe",
        "image_url": "https://example.com/mentor_profile_image.jpg",
        "linkedin_url": "https://www.linkedin.com/in/johndoe",
        "type": "DEFAULT",
        "uuid": "12345678-1234-5678-1234-567812345678"
    }
}

Example mark mentor as not available
{
    "mentor_id": 1,
    "created_at": "2023-06-30T10:00:00Z",
    "updated_at": "2023-06-30T14:30:00Z",
    "state": "APPROVED",
    "category": "Technology",
    "application": {
        "country": "",
        "areasOfExpertise": "",
        "expectationsFromMentees": "",
        "mentoringPhilosophy": "",
        "commitmentToProgram": false,
        "previousExperienceAsMentor": false,
        "reasonForBeingMentor": "",
        "cvLink": ""
    },
    "availability": false,
    "profile": {
        "created_at": "2023-06-29T09:15:00Z",
        "updated_at": "2023-06-30T16:45:00Z",
        "primary_email"<EMAIL_ADDRESS>        "contact_email"<EMAIL_ADDRESS>        "first_name": "John",
        "last_name": "Doe",
        "image_url": "https://example.com/mentor_profile_image.jpg",
        "linkedin_url": "https://www.linkedin.com/in/johndoe",
        "type": "DEFAULT",
        "uuid": "12345678-1234-5678-1234-567812345678"
    }
}

Tasks:

Create a controller for the /controllers/mentor endpoint in the backend (create the route mentor if not created).
Parse and validate the request body to ensure it matches the expected format.
Implement appropriate error handling and response status codes for different scenarios (e.g., validation errors, database errors).
Write unit tests to validate the functionality and correctness of the endpoint.

API documentation: https://documenter.getpostman.com/view/27421496/2s93m1a4ac#8744a3ee-970f-489a-853d-8b23fdee8de3
ER diagram: https://drive.google.com/file/d/11KMgdNu2mSAm0Ner8UsSPQpZJS8QNqYc/view
Acceptance Criteria:

The profile API endpoint /controllers/mentor is implemented via a PUT request.
The request body is properly parsed and validated for the required format.
Appropriate error handling is implemented, providing meaningful error messages and correct response status codes.
Unit tests are written to validate the functionality and correctness of the endpoint.

Additional Information:
No
Related Dependencies or References:
No

@anjula-sack I'd like to work on this issue


Add visually hidden text between section and page in search result section info

What
Adds a visually hidden span containing ', in page' before the app-site-search__separator span and adjusts styling of the separator to account for the extra markup spacing.
Why
This is an offshoot from https://github.com/alphagov/govuk-design-system/issues/4006. Whilst we've resolved the relationship between search result and section info, testing for that solution outlined the same problem between search info where the search result was a heading within a page and so the search info was '[section] > [page]' eg: The searh info for result 'Responsive spacing' is 'Styles > Spacing'.
This solution replicates in part the solution between result and ifo where a comma is added, with extra visually hidden content for context.

Testing spreadsheet
Notes
the space between the section and the comma is a pain. This is a result of the spans that make up search item markup being moved onto their own lines. There's not a lot we can do about it and I personally don't think it's a huge issue, even though it's bothersome.
The magic number added here is similarly annoying.
I'm interested in opinions on if the visually hidden text is necessary. I think some extra context might be useful to screen readers, but just a comma is as effective in separating the section and page.

I generally like this approach.
But I'm not sure if the order of the words is correct.
Right now it says:

Responsive spacing, Styles, in page Spacing

But the structure is really 'styles > spacing > responsive spacing' (from big to small), or 'responsive spacing > spacing > styles' (from small to big), or to say it in words alone 'Responsive spacing in page Spacing in section Styles'.
Not sure how to put that neatly into one string that is not too verbose. I'm also aware the order in which it makes sense with words is a different order visually.
I guess it's like that because one part sort of works like breadcrumbs while the other part sort of works like a heading. They do two different things.

Cross posting my response to a similar comment on the previous PR for this work https://github.com/alphagov/govuk-design-system/pull/4063#issuecomment-2320661542

Based on some rough plans we have to review the entire design of the site search, I'm gonna close this for now as it's unlikely this willbe taken forward and if I do address the above comments, it's likely to be undermined by near-future changes


[DO NOT MERGE] Auto PR to update dependencies of mixer

This PR will be merged automatically once checks are successful.

This change is

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
a6228bf2dd953bfa424d9e0e3c601a55e9ae190b
link
/test mixer-e2e-smoketest

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

/retest


Build enhancements

Here are the build enhancements I made, feel free to add them to your npx scripts, or do things differently if you see a better way.

Closed. We used these in the npx script


Added ros interface to blackboard implementation

@daniel86 I think I now incorporated every point of your comments.
I rewrote a substitution to string function, that works with the rosprolog_commandline.
Additionally I added the tf code. The code compiles now, but I still have to test it.

Something weird happened with the diff in this PR for PrologReasoner.h. I simply added an additional version of the eval function.

@daniel86 I incorporated all your comments and splitted my ROSInterface file so that the ResultHandler is in it's own file (for later use with a REST interface). I now named the ResultHandler IterativeQAResultHandler.


Write sails core on es6 / babel

This will attract more developers and adds more features, native support for iojs and transpiler js from babel for nodejs

I agree but this is not urgent. Our top first priority is to release Sails 1.0.0. We'll write Sails on ES6 as soon as we are more stable on our side. Thanks for the feedback, we appreciate.

So around 15 months passed...

+1


Deprecate --no-rebuild

This option was introduced very early in Gradle's life when up-to-date checking was still rather expensive. This is no longer the case. Removing this option removes a bit of complexity and avoids confusion on the user side as to when they might want to use it.

It would be interesting to know if we really optimized enough to let this option go away. Imagine you have a multi-project build with 300 subprojects but only touched 1 that you want to build. Do we know if up-to-date checking is fast enough in those scenarios. A performance test that compares both scenarios would be important to have IMHO.
Can you explain why the command line option is confusing to users?

Imagine you have a multi-project build with 300 subprojects but only touched 1 that you want to build.

We have performance tests with 500 subprojects and doing a single line change takes just ~1s more than calling help on it.

Can you explain why the command line option is confusing to users?

Because it doesn't even work in all cases and we have no intention to fix it.

I've added some comments to this commit. We shouldn't create deprecation warnings in the parameter parser. We should create them when the start parameters are validated later. Otherwise we won't get the warning when:

running in embedded mode
setting options through gradle.properties
settings options programatically

Can you please open a new issue to refactor this logic?

@oehme Sure, I think we could open an issue like "refine deprecation warning handling" to handle all these stuff.


Unit test errors when using \LOCK_EX with \file_put_contents.

It seems that PHP-VCR is causing an issue when you try to acquire an exclusive lock on a file with file_put_contents. I have been able to narrow it down to this line: \file_put_contents( $pFileName, $pContent, \LOCK_EX );
I'm using Windows 7, 64-bit with:
PHP 5.6.0RC3 (cli) (built: Jul 30 2014 11:15:07)
Copyright (c) 1997-2014 The PHP Group
Zend Engine v2.6.0-dev, Copyright (c) 1998-2014 Zend Technologies
    with Xdebug v2.2.4, Copyright (c) 2002-2014, by Derick Rethans

If I do NOT remove the LOCK_EX parameter, I get the following error if PHP-VCR is turned on in my  bootstrap file:
There was 1 error:

1) Kshabazz\Tests\Test\VcrCurlTest::test_saveFile_function
flock(): Illegal operation argument

E:\Khalifah\Projects\test\vendor\php-vcr\php-vcr\src\VCR\Util\StreamProcessor.php:461
E:\Khalifah\Projects\test\src\functions.php:22
E:\Khalifah\Projects\test\tests\VcrCurlTest.php:37
C:\Users\Khalifah\vendor\phpunit\phpunit\src\TextUI\Command.php:186
C:\Users\Khalifah\vendor\phpunit\phpunit\src\TextUI\Command.php:138

FAILURES!
Tests: 2, Assertions: 1, Errors: 1.

I have a function defined as follows:
function saveFile( $pFileName, $pContent )
{
    $directory = \dirname( $pFileName );
    if ( !is_dir($directory) )
    {
        $madeDir = \mkdir( $directory, 0755, TRUE );
        if ( $madeDir === FALSE )
        {
            throw new \Exception( "mkdir: Unable make directory '{$directory}'." );
        }
    }
    // Save data to a file.
    $fileSaved = \file_put_contents( $pFileName, $pContent, \LOCK_EX );
    if ( $fileSaved === FALSE )
    {
        throw new \Exception( "file_put_contents: Unable to save file '{$pFileName}'." );
    }

    return $fileSaved;
}

I test it with this code:
    /**
     * Test saveFile.
     */
    public function test_saveFile_function()
    {
        // todo: change to temp space.
        $tempDir = $this->fixtures . '/bunker';
        $file = $tempDir . '/save-test-file.txt';
        $saved = \Kshabazz\Test\saveFile( $file, 'test' );
        $this->assertEquals( 4, $saved, 'Unable to save file.' );
        $this->assertFileExists( $file, 'Unexpected data loaded from empty JSON test file.' );
        \unlink( $file );
        \rmdir( $tempDir );
    }

I'm having this problem with PHP 7.0.12.


[Doc] Document exporting PVCs to Images

Describe the solution you'd like
The POST endpoint to action on a specific PVC is not documented. Example, exporting a pvc to a new image:
/v1/harvester/persistentvolumeclaims/drew/packer-build?action=export

This looks like the right endpoint, maybe just need to update the available query parameters: https://docs.harvesterhci.io/v1.3/api/replace-namespaced-persistent-volume-claim

Here is where the action options are added to the server path https://github.com/harvester/harvester/blob/ba586eaf0a4eb5fe58b79c444eadc4ce18f9024b/pkg/api/volume/schema.go#L33

and struct shapes: https://github.com/harvester/harvester/blob/ba586eaf0a4eb5fe58b79c444eadc4ce18f9024b/pkg/api/volume/types.go#L3-L7


Allow uses to enter rotation of a waypoint in degrees

Hello,
I want to create an auto with a waypoint that has theta of 90 degrees, it is a bit annoying to enter in rads...

Rotations may be even better (e.g., 0.25 rotations), since that seems like what you're trying to describe.

I would also like degrees. Its how humans think of angles. I'd be fine with it being a setting like how you can use imperial when setting wheel diameter / mass / bumper width / etc.
If you lined up against the side of the subwoofer, you were at +/- 60 degrees. That makes much more sense to me to enter into the text field than 1.0472 radians or .1666 rotations

Degrees are just as arbitrary as rotations.

maybe, but as a human you think in degrees, not rotations and not radians

but as a human you think in degrees

I think in turns/rotations, not degrees.

but in rotations you have mostly the same problem with rads...

Not really. The mistake people make with radians is multiplying the π basis into the number. They're much easier to manage if the π is kept separate (e.g., 90 degrees = 0.5π radians). Using τ=2π as a basis is even more convenient because it operations in rotations (e.g., 90 degrees = 0.25τ = 0.25 rotation).

I can't put pi / 6 radians or 1 / 6 rotations in the text box. My kids have to use a calculator / google to do the conversion so we can put that a floating point number in there.
Or, you make the units on the field configurable, and everyone can enter values that feels most intuitive for them.

There's https://github.com/SleipnirGroup/Choreo/pull/262, which hasn't been finished.

Superseded by #96


ImproveAdd the configuration of whether to synchronize the default value

Proposed changes
At present, when the mysql database sync to doris , there is a default value in the field, but the actual data is null, the data cannot maintain consistency.
Like: when mysql schema has a default value and the insertion is null, the data synchronized to doris uses the default value.
CREATE TABLE `test_sink` (
  `id` int NOT NULL,
  `name` varchar(50) DEFAULT 'test_default',
  `age` int DEFAULT '10000',
  PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_0900_ai_ci

insert into test_sink (id,name,age) values(6, null, 666);
insert into test_sink (id,name,age) values(5, 'zhaoliu', null);

Configure  --ignore-default-value, the default value will not be synchronized, otherwise it will be synchronized.
Problem Summary:
Describe the overview of changes.
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

In actual testing, the parameter of  ignore-default-value is not effective. Regardless of whether this parameter is present or not, the default values of MySQL fields will not be synchronized. Doris's table creation statements and data synchronized from MySQL to Doris do not have default values.


Proposal: Change document to source

Capturing a discussion with @shivchander:
I was writing up a test case for the lmdk cli to test knowledge workflow, but the way that I laid out my qna.yaml is as follows:
test_knowledge_valid = b"""created_by: test-bot
seed_examples:
- question: What is Operator Framework?
  answer: 'The Operator Framework is a set of Kubernetes components and developer tools,
  that aid in Operator development and central management on a multi-tenant cluster.'
- question: What is an Operator?
  answer: 'The goal of an Operator is to put operational knowledge into software.
  Previously this knowledge only resided in the minds of administrators,
  various combinations of shell scripts or automation software like Ansible.
  It was outside of your Kubernetes cluster and hard to integrate.
  With Operators, CoreOS changed that. Operators implement and automate
  common Day-1 (installation, configuration, etc.) and Day-2 (re-configuration,
  update, backup, failover, restore, etc.) activities in a piece of software running
  inside your Kubernetes cluster, by integrating natively with Kubernetes concepts and APIs.
  We call this a Kubernetes-native application.
  With Operators you can stop treating an application as a collection of primitives like Pods,
  Deployments, Services or ConfigMaps, but instead as a single object that only exposes the knobs
  that make sense for the application.'
- question: What is Operator Lifecycle Manager?
  answer: 'OLM is a component of the Operator Framework,
  an open source toolkit to manage Kubernetes native applications,
  called Operators, in an effective, automated, and scalable way.
  OLM extends Kubernetes to provide a declarative way to install,
  manage, and upgrade Operators and their dependencies in a cluster.
task_description: to teach a large language model about the Operator Framework
document:
  repo: https://github.com/anik120/knowledge-doc-test
  commit: bf78d868f544e55d8e1d99f68d9105fc3b8751bd
  patterns:
  - operator-framework*.md

Essentially, the seed_example question/answers I have there are from the overarching project websites https://operatorframework.io/, https://olm.operatorframework.io/ and https://sdk.operatorframework.io/, and the documents I have in https://github.com/anik120/knowledge-doc-test are README.mds from the components' GitHub repositories. In other words, the seed_example question/answers do not actually come from the documents hosted in document.repo.
The way I laid things out, the seed_examples are "product pitch/summary description" and document.repo contains all the docs I want the model to learn about.
Shiv tells me that that's the wrong way of thinking about it, and the verb document should be source in reality, and seed_examples are examples of questions/answers that can be answered by the model once it's been trained on the docs hosted in docs.repo.
Eureka moment: Even after learning* how the taxonomy interacts with the model, I was thinking about the structure of my qna.yaml, the wrong way. It's likely that other users will also confuse the taxonomy/model interactions and lay out the qna.yaml files the wrong way, leading to PR submissions that'll likely not improve model quality.
*only a little while ago, ie fresh info being processed by brain still
Proposed fix: Change document to source
cc: @xukai92 @abhi1092 @aldopareja

@anik120 I think it is beyond when this change could be made. Perhaps we can close this issue?

This is no longer relevant due to the V3 knowledge yaml change.


docs: add best practices for patching and tagging

Took an initial try at this. Open to feedback to improve the examples and wording
Closes #476

fyi @johnsonshi


Solution: Make Debug Logging Configurable

See also #331
With this new configuration_option debug_logging: false (defaults to true) you can mute the Quantum debug-messages (if you want to use the debug-level of Logger for your own stuff).

Pull Request Test Coverage Report for Build 770

22 of 22 (100.0%) changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.585%

Totals

Change from base Build 769:
0.0%

Covered Lines:
248

Relevant Lines:
265

💛  - Coveralls

@DerTim1 Would you add one tests to cover if the code actually works? (I don't think that it is not working but would like to avoid regressions.)

@maennchen Thank you!
I fixed the mistake with the spec and also added one test for capture or not capture the debug logging.

@maennchen Could you please release this?

I second @maennchen 's comment. Is the debug logging configuration feature that keeps us from using Quantum from hex.pm. Is there an ETA for publishing a new release containing this feature? Thank you.


Errors when building for Win10/64 and VC2022

Hello and thank you for your nice project. However, I'm not able to successfully build it. I get 1 success and 1 failed with the following errors...
Error	C2079	'Star::_starOcclusion' uses undefined class 'std::array<GLuint,2>'	SolarSystem	C:\SolarSystem\src\Solar_System\Star.h	60
Error	C3646	'_nearestPlanetIndex': unknown override specifier	SolarSystem	C:\SolarSystem\src\Application.h	39
Error	C2589	'(': illegal token on right side of '::'	SolarSystem	C:\SolarSystem\src\Application.cpp	255
Severity	Code	Description	Project	File	Line	Suppression State
Error	C2065	'ssize_t': undeclared identifier	SolarSystem	C:\SolarSystem\src\Application.cpp	1093
and so on!!!
I've downloaded, ran cmake, loaded the solution, set to release, set all the includes for GL/SDL etc. build. Am I doing it wrong?

@Alienizering
Hi, thanks for the review and sorry for the long wait! To be honest I haven't tested build of my project with Visual Studio compiler, from the very beginning everything was built with mingw and GCC. I can rebuild everything with MSVC, just let me know if it's still relevant to you.

Issue is closed due to long inactivity.


Handle new CBOR metadata format

Solidity is now storing the compiler version along with the swarm hash at the end of the contract (see here). We need to make sure that zOS properly handles this format when a new solidity compiler version is released.
We can also try extracting the compiler version from the hash to automate the zos verify process.
Related to https://github.com/zeppelinos/zos/issues/821.

Fixed in #934


[youtube] fix sig parameter

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[ ] I am the original author of this code and I am willing to release it under Unlicense
[x] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Some YouTube videos do not need signature decoding, just to pass the value in the encoded_url_map.
However, for these videos, a 403 error occurs.
Probably the parameter name is changed.
So I modified "signature =" to "sig =", after which no 403 error occurred.
examples:
youtube-dl -f mp4 --get-url https://youtu.be/otWeDrOWq08
https://r6---sn-3u-u5xz.googlevideo.com/videoplayback?expire=1557509600&ei=f2HVXIvQNqiEs8IP47uPyA8&ip=<IP_ADDRESS>&id=o-AOu-YU0eazP1v5R7Pd4sGPJNS5kk66gQqweXbKVwUiWf&itag=22&source=youtube&requiressl=yes&mm=31,29&mn=sn-3u-u5xz,sn-3u-bh26&ms=au,rdu&mv=m&pl=21&initcwndbps=1446250&mime=video/mp4&ratebypass=yes&dur=217.454&lmt=1550276808297835&mt=1557487825&fvip=7&beids=9466587&c=WEB&txp=5535432&sparams=expire,ei,ip,id,itag,source,requiressl,mime,ratebypass,dur,lmt&lsparams=mm,mn,ms,mv,pl,initcwndbps&lsig=AHylml4wRgIhAPGUIul58uf1bdqR6xf7zrW5iaG5hxR6iT3WJX8qcK_aAiEAq40gwiKEPZwMBqf63JWHBZLOnK2kA-dw3X5SPITagEw=&signature=ALgxI2wwRQIgdit6rriegkMXSG6TflJaxQ7v5EwN5HZ4zR0pGLrdyU8CIQDZ2sMUjvEtyh13KHlJlEhg1OSJAYgx7czXMCJf8VZyLw==
https://r6---sn-3u-u5xz.googlevideo.com/videoplayback?expire=1557509600&ei=f2HVXIvQNqiEs8IP47uPyA8&ip=<IP_ADDRESS>&id=o-AOu-YU0eazP1v5R7Pd4sGPJNS5kk66gQqweXbKVwUiWf&itag=22&source=youtube&requiressl=yes&mm=31,29&mn=sn-3u-u5xz,sn-3u-bh26&ms=au,rdu&mv=m&pl=21&initcwndbps=1446250&mime=video/mp4&ratebypass=yes&dur=217.454&lmt=1550276808297835&mt=1557487825&fvip=7&beids=9466587&c=WEB&txp=5535432&sparams=expire,ei,ip,id,itag,source,requiressl,mime,ratebypass,dur,lmt&lsparams=mm,mn,ms,mv,pl,initcwndbps&lsig=AHylml4wRgIhAPGUIul58uf1bdqR6xf7zrW5iaG5hxR6iT3WJX8qcK_aAiEAq40gwiKEPZwMBqf63JWHBZLOnK2kA-dw3X5SPITagEw=&sig=ALgxI2wwRQIgdit6rriegkMXSG6TflJaxQ7v5EwN5HZ4zR0pGLrdyU8CIQDZ2sMUjvEtyh13KHlJlEhg1OSJAYgx7czXMCJf8VZyLw==
youtube-dl -f mp4 --get-url https://youtu.be/lfUU-C4Rm84
https://r2---sn-3u-u5xe.googlevideo.com/videoplayback?expire=1557513186&ei=gW_VXJnMNtL0qQG53a3gCQ&ip=<IP_ADDRESS>&id=o-ANtzWdQFNXJyI9Ud_y3om56DHUZhpr-J-ha7x4Edc9zg&itag=22&source=youtube&requiressl=yes&mm=31%2C29&mn=sn-3u-u5xe%2Csn-3u-bh2s&ms=au%2Crdu&mv=m&pl=21&initcwndbps=1280000&mime=video%2Fmp4&ratebypass=yes&dur=242.741&lmt=1515316575883630&mt=1557491504&fvip=1&c=WEB&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cmime%2Cratebypass%2Cdur%2Clmt&lsparams=mm%2Cmn%2Cms%2Cmv%2Cpl%2Cinitcwndbps&lsig=AHylml4wRAIgKW0m9n_HaF2aW06Dj_o8hGUmHTAQ7OtIdB34vDQdBfECICk1T3KjJ_j8RFe1uIo0DlfGHXCWSU8f4A4I1d5pQJnF&signature=ALgxI2wwRQIgdojz8eRWZLdcfGTSOD5HAO3uWepvb22gCNe8vb55s-ICIQDtoPaxtwwM3MEREOag2OmnMwAcc5QfqK6wSXz5A7rB1Q==
https://r2---sn-3u-u5xe.googlevideo.com/videoplayback?expire=1557513186&ei=gW_VXJnMNtL0qQG53a3gCQ&ip=<IP_ADDRESS>&id=o-ANtzWdQFNXJyI9Ud_y3om56DHUZhpr-J-ha7x4Edc9zg&itag=22&source=youtube&requiressl=yes&mm=31%2C29&mn=sn-3u-u5xe%2Csn-3u-bh2s&ms=au%2Crdu&mv=m&pl=21&initcwndbps=1280000&mime=video%2Fmp4&ratebypass=yes&dur=242.741&lmt=1515316575883630&mt=1557491504&fvip=1&c=WEB&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cmime%2Cratebypass%2Cdur%2Clmt&lsparams=mm%2Cmn%2Cms%2Cmv%2Cpl%2Cinitcwndbps&lsig=AHylml4wRAIgKW0m9n_HaF2aW06Dj_o8hGUmHTAQ7OtIdB34vDQdBfECICk1T3KjJ_j8RFe1uIo0DlfGHXCWSU8f4A4I1d5pQJnF&sig=ALgxI2wwRQIgdojz8eRWZLdcfGTSOD5HAO3uWepvb22gCNe8vb55s-ICIQDtoPaxtwwM3MEREOag2OmnMwAcc5QfqK6wSXz5A7rB1Q==

These both videos use encrypted signatures.

It does not seem to always use encrypted signatures. 😭
Even if it is the same video ID, it applies randomly.


Cats 1.0.0-MF changes

To achieve this change traverseU was changed to traverse.
Updated to scala 2.11.11 so partial unification was available.

Whether this should PR to master and how you want to handle this PR and subsequent release, I leave up to further discussion.

Hello!
May i ask if there's any chance to get this merge in the nearest future? Unfortunately this is the only piece left that blocks us from moving towards Cats 1.0 MF.

Hi all, sorry for the delay. Went on holiday just around the time this came in.
This looks good to me, so I'm going to merge this into master now and look to put out some kind of 1.0 type release soon.

@phillills Awesome news - thanks for this - and also happy to hear about a potential 1.0 release :+1:

I've just opened #137 to discuss what I'm proposing for releasing this.

This is now released as 1.0.0-M1. Thanks again for the contribution.


Selected date displays last date

The selected date is always showing the last date. For your reference, following is the code used
CalendarAppBar(                               onDateChanged: (value) => print(value),                               firstDate: DateTime.now()                                   .subtract(const Duration(days: 7)),                               events: [DateTime.now()],                               lastDate:                                   DateTime.now().add(const Duration(days: 7)),                               selectedDate: DateTime.now().add(const Duration(days: 2)),                               backButton: false,                               accent: Colors.cyan,                               padding: 5,                               fullCalendar: false,                             ),

the same problem here :/

how to fix it,i have same problem

You can change a to b and b to a in line 189

You can change a to b and b to a in line 189

This only beings the app bar at the first date. Is there any workarounds for this?

You can change a to b and b to a in line 189

This only sets the app bar at the first date. Is there any workarounds for this?

you can use this https://pub.dev/packages/calendar_agenda ,There is no problem with this when I have been tested.


MISO_SASHIMI step fails when using scratch directory

Description of the bug
When using the process$scratch variable in the configuration file, NFCORE_RNASPLICE:RNASPLICE:VISUALISE_MISO:MISO_SASHIMI fails by looking in the scratch directory instead of where the pickle files actually are. Here, process$scratch is set to $TMPDIR, which gets expanded to /local/xxxxx/20738687/cluster_tmp.
However, then MISO fails with:
Exception: Error: no filename /local/xxxxx/20738687/cluster_tmp/nxf.R4ElqFeq2j/index/chrX/ENSG00000004961.pickle

The pickle files are however actually in the output directory, not the scratch directory:
(base) -bash-4.2$ find test2 -name *.pickle | head
test2/misopy/index/chrX/ENSG00000130962.pickle
test2/misopy/index/chrX/ENSG00000126952.pickle
test2/misopy/index/chrX/ENSG00000242013.pickle
test2/misopy/index/chrX/ENSG00000196632.pickle
test2/misopy/index/chrX/ENSG00000130830.pickle
test2/misopy/index/chrX/ENSG00000184033.pickle
test2/misopy/index/chrX/ENSG00000101892.pickle
test2/misopy/index/chrX/ENSG00000133149.pickle
test2/misopy/index/chrX/ENSG00000165194.pickle
test2/misopy/index/chrX/ENSG00000198689.pickle

Full error:
-[nf-core/rnasplice] Pipeline completed with errors-
ERROR ~ Error executing process > 'NFCORE_RNASPLICE:RNASPLICE:VISUALISE_MISO:MISO_SASHIMI (1)'

Caused by:
  Process `NFCORE_RNASPLICE:RNASPLICE:VISUALISE_MISO:MISO_SASHIMI (1)` terminated with an error exit status (1)

Command executed:

  sashimi_plot --plot-event ENSG00000004961 index miso_settings.txt --output-dir sashimi

  cat <<-END_VERSIONS > versions.yml
  "NFCORE_RNASPLICE:RNASPLICE:VISUALISE_MISO:MISO_SASHIMI":
      python: $(python --version | sed "s/Python //g")
      misopy: $(python -c "import pkg_resources; print(pkg_resources.get_distribution('misopy').version)")
  END_VERSIONS

Command exit status:
  1

Command output:
  (empty)

Command error:
  WARNING: DEPRECATED USAGE: Forwarding SINGULARITYENV_TMPDIR as environment variable will not be supported in the future, use APPTAINERENV_TMPDIR instead
  WARNING: DEPRECATED USAGE: Forwarding SINGULARITYENV_NXF_DEBUG as environment variable will not be supported in the future, use APPTAINERENV_NXF_DEBUG instead
  /usr/local/lib/python2.7/site-packages/matplotlib/cbook/deprecation.py:107: MatplotlibDeprecationWarning: The mpl_toolkits.axes_grid module was deprecated in version 2.1. Use mpl_toolkits.axes_grid1 and mpl_toolkits.axisartist provies the same functionality instead.
    warnings.warn(message, mplDeprecation, stacklevel=1)
  Traceback (most recent call last):
    File "/usr/local/bin/sashimi_plot", line 11, in <module>
      sys.exit(main())
    File "/usr/local/lib/python2.7/site-packages/misopy/sashimi_plot/sashimi_plot.py", line 276, in main
      plot_label=plot_label)
    File "/usr/local/lib/python2.7/site-packages/misopy/sashimi_plot/sashimi_plot.py", line 153, in plot_event
      plot_label=plot_label)
    File "/usr/local/lib/python2.7/site-packages/misopy/sashimi_plot/plot_utils/plot_gene.py", line 675, in plot_density_from_file
      parseGene(pickle_filename, event)
    File "/usr/local/lib/python2.7/site-packages/misopy/parse_gene.py", line 11, in parseGene
      raise Exception, "Error: no filename %s" %(pickle_filename)
  Exception: Error: no filename /local/xxxxx/20738687/cluster_tmp/nxf.R4ElqFeq2j/index/chrX/ENSG00000004961.pickle

Command used and terminal output
nextflow run nf-core/rnasplice -profile test -c dkfz.config --outdir test2 -r dev

Relevant files
dkfz.config
params {
    config_profile_description = 'Deutsches Krebsforschungszentrum (DKFZ) HPC cluster profile provided by nf-core/configs'
    config_profile_contact     = 'Kübra Narcı<EMAIL_ADDRESS>    config_profile_name        = 'DKFZ cluster'

    max_cpus   = 30
    max_memory = '250.GB'
    max_time   = '48.h'
}

singularity {
  enabled = true
  autoMounts = true
}

process {
  executor = 'lsf'
  clusterOptions = '-L /bin/bash'
  scratch = '$TMPDIR'
}

executor {
  name = 'lsf'
  perTaskReserve = false
  perJobMemLimit = true
  queueSize = 10
  submitRateLimit = '3 sec'
}

System information

Nextflow version: 23.04.1 build 5866
Hardware: HPC (LSF)
Executor: 'lsf'
Container engine: Singularity
OS: CentOS Linux 7
Version of nf-core/rnasplice: dev

I did a bit more digging -- running it with NXF_DEBUG=1 reveals that two different temporary directories from two different LSF jobs are being used. Specifically, the error is
Exception: Error: no filename /local/xxxxx/20769714/nxf.98A1VB7d5I/index/chrX/ENSG00000005302.pickle

but the job is being executed in /local/xxxxx/20769804/nxf.YYBkdvf6cO:
/bin/bash -c cd /local/xxxxx/20769804/nxf.YYBkdvf6cO; eval export PYTHONNOUSERSITE="1" export R_PROFILE_USER="/.Rprofile" export R_ENVIRON_USER="/.Renviron" export JULIA_DEPOT_PATH="/usr/local/share/julia" export PATH="$PATH:/omics/groups/xxxxx/internal/xxxxx/.nextflow/assets/nf-core/rnasplice/bin"; /bin/bash /omics/odcf/analysis/xxxxx/xxxxx/nextflow/work/33/439d4c44670b38d3bd71734475398f/.command.run nxf_trace

I think the specific problem arises because miso somehow hard-encodes the path in the index. Specifically, if I look for the LSF job ID in the index_gff working directory, I see that
(base) -bash-4.2$ grep -R 20769714 *
Binary file index/genes_to_filenames.shelve.dat matches

so, I suspect that index_gff is hard-encoding the absolute path to the filename. It might be necessary to explicitly specify scratch false for this step. I will try this now.

In my hands, setting scratch false in the module seems to make it work (see here). However, is it better to do this in the module, or in conf/modules.config?


[feature request] Web worker loader

A web worker loader is required such that worker code is bundled with the main bundle, and not requested as a separate file by the browser.
some equivalent of https://github.com/webpack-contrib/worker-loader

I was about to put in this same request ^ nice

I'd be happy to submit a PR to implement this, but I need some initial guidance

@venil7 we are thinking bigger here. Feel free to poke on gitter

@venil7  we are cooking something big that will cover this use case and introduce pure awesomeness in many other places, hang on with us :)

Any update on this issue?

@aretecode ?

Is there progress on web worker support?

I don't think there is an update. You think you can try implementing it @thomas-jeepe

@nchanged Unlikely.

@nchanged @devmondo remind me again to link what I did on this

is there any way to currently use web workers with fuse-box (under both dev and distribution builds)? I tried different things like multiple producers etc. but I keep getting an undefined globals error (I think because the FuseBox api tries to access that)

@devmondo @nchanged for v3 roadmap?

Yep, we can make a sep. plugin for that in v3


About flutter tools

For first run flutter build apk , it will download some compile tools(gradle or some binary file), Where can I find documentation for these things？

@lycstar
I am not sure if you'll get exact details as why it downloads the tools for the time, but I'll try to give you an idea. Hope it helps:
flutter build apk need to gather all arch types (arm, arm86 for example) before it could build the apk and hence when you run it for the first time, it'll download the required tools as below:
Downloading android-arm-profile/darwin-x64 tools...                912ms
Downloading android-arm-release/darwin-x64 tools...                555ms
Downloading android-arm64-profile/darwin-x64 tools...              684ms
Downloading android-arm64-release/darwin-x64 tools...              571ms
Downloading android-x64-profile/darwin-x64 tools...                587ms
Downloading android-x64-release/darwin-x64 tools...                545ms

Next time when you run same command, since the tools are already downloaded, it won't install it again.
You can find information related to how building an apk (although not in exact details, but it should give you an idea) here: https://flutter.dev/docs/deployment/android#build-an-apk

@lycstar
I am not sure if you'll get exact details as why it downloads the tools for the time, but I'll try to give you an idea. Hope it helps:
flutter build apk need to gather all arch types (arm, arm86 for example) before it could build the apk and hence when you run it for the first time, it'll download the required tools as below:
Downloading android-arm-profile/darwin-x64 tools...                912ms
Downloading android-arm-release/darwin-x64 tools...                555ms
Downloading android-arm64-profile/darwin-x64 tools...              684ms
Downloading android-arm64-release/darwin-x64 tools...              571ms
Downloading android-x64-profile/darwin-x64 tools...                587ms
Downloading android-x64-release/darwin-x64 tools...                545ms

Next time when you run same command, since the tools are already downloaded, it won't install it again.
You can find information related to how building an apk (although not in exact details, but it should give you an idea) here: https://flutter.dev/docs/deployment/android#build-an-apk

@darshankawar Thank you for your reply , So Where can I find the x64 tool's source code, because i want to compile to get these tool for linux arm.

@darshankawar Thank you for your reply , So Where can I find the x64 tool's source code, because i want to compile to get these tool for linux arm.

@lycstar
Check out this https://github.com/flutter/flutter/tree/master/packages/flutter_tools/lib/src/linux

@lycstar
Check out this https://github.com/flutter/flutter/tree/master/packages/flutter_tools/lib/src/linux


By default don't import 3rd-party Python modules, only stdlib ones.  …

…Fixes #854.

This change is

This will break a backwards compat change (where we let you configure the transport with the DSN), but as long as we mentioned that in the CHANGES I'm totally ok with it.
We also should simply remove the comments and just kill the lines (we're using version control after all).

So I fixed the tests and cleaned up the code.  One issue that I had was that I have to access Client._registry, and we shouldn't be required to access attributes marked private.  So I propose to rename, Client._registry to Client.registry, what do you think?

Thanks!

Please check if the latest commits address your comments.

Looks good we just need to update the tornado tests to remove the tornado+ prefix I think?

Hm, some Tornado test failing on Python 2.6, but I don't know why :(

2.6 is very much alive, we can't kill it yet.

It's all very fine to say "we must support Python2.6", but you have to realise it is incredibly hard, today, to do anything with Python2.6.  After much insistence, I managed to get a docker image with python2.6 and pip, but even then I didn't manage to run the tests.  The latest error is:
Collecting billiard<3.6.0,>=<IP_ADDRESS> (from celery>=2.5->raven)
  Downloading billiard-<IP_ADDRESS>.tar.gz (157kB)
    100% |################################| 163kB 5.7MB/s
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-build-QHA8oe/billiard/setup.py", line 90, in <module>
        raise ValueError('Versions of Python before 2.7 are not supported')
    ValueError: Versions of Python before 2.7 are not supported

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-QHA8oe/billiard/
make: *** [bootstrap] Error 1
root@45ac8d430fc8:/code#

Please, please, reconsider Python 2.6, it is dragging raven-python down!
Maybe rebrand raven-python into a new major release and state in the release notes that whoever needs Python 2.6 support should continue using the old raven version?
Oh, drat, I see now you had a major release recently :(
There must be something we can do, this is ridiculous.  How are we supposed to fix tests if can't even run them?


How to use nova_get_settings helper in other config files

Hi,
It is possible to use nova_get_settings() outside nova? i.e. in other config files?
Thanks,
Mihai

Hi!
It cannot be done inside config files because the settings are stored in MySQL and requires querying the database.
The database connection info is inside the config files, so you would essentially be trying to query the database with no config.
The best you could do is modify the config inside AppServiceProvider boot() or register() methods.


Fix #1417: Show actual error, not 'multiple calls to done()'

This PR seems to enforce what I'd imagine as correct behavior, but want to hear from @mochajs/mocha

Before PR
$ ./bin/mocha test/integration/fixtures/regression/issue-1417.js

  ․․․․

  0 passing (14ms)
  4 failing

  1)  fails exactly once when a global error is thrown synchronously and done errors:
     Error: test error
      at null._onTimeout (test/integration/fixtures/regression/issue-1417.js:10:10)

  2)  fails exactly once when a global error is thrown synchronously and done errors:
     Error: test error
      at null._onTimeout (test/integration/fixtures/regression/issue-1417.js:10:10)

  3)  fails exactly once when a global error is thrown synchronously and done completes:
     Error: done() called multiple times
      at Test.Runnable (lib/runnable.js:54:17)
      at new Test (lib/test.js:22:12)
      at context.it.context.specify (lib/interfaces/bdd.js:85:18)
      at Object.<anonymous> (test/integration/fixtures/regression/issue-1417.js:16:1)
      at Array.forEach (native)
      at Object.<anonymous> (bin/_mocha:403:18)
      at node.js:868:3

  4)  fails exactly once when a global error is thrown synchronously and done completes:
     Error: done() called multiple times
      at Test.Runnable (lib/runnable.js:54:17)
      at new Test (lib/test.js:22:12)
      at context.it.context.specify (lib/interfaces/bdd.js:85:18)
      at Object.<anonymous> (test/integration/fixtures/regression/issue-1417.js:16:1)
      at Array.forEach (native)
      at Object.<anonymous> (bin/_mocha:403:18)
      at node.js:868:3

After PR
$ ./bin/mocha test/integration/fixtures/regression/issue-1417.js

  ․․

  0 passing (12ms)
  2 failing

  1)  fails exactly once when a global error is thrown synchronously and done errors:
     Error: sync error
      at Context.<anonymous> (test/integration/fixtures/regression/issue-1417.js:13:9)
      at callFnAsync (lib/runnable.js:340:8)
      at Test.Runnable.run (lib/runnable.js:290:7)
      at Runner.runTest (lib/runner.js:422:10)
      at lib/runner.js:529:12
      at next (lib/runner.js:342:14)
      at lib/runner.js:352:7
      at next (lib/runner.js:284:14)
      at Immediate._onImmediate (lib/runner.js:320:5)

  2)  fails exactly once when a global error is thrown synchronously and done completes:
     Error: sync error
      at Context.<anonymous> (test/integration/fixtures/regression/issue-1417.js:21:9)
      at callFnAsync (lib/runnable.js:340:8)
      at Test.Runnable.run (lib/runnable.js:290:7)
      at Runner.runTest (lib/runner.js:422:10)
      at lib/runner.js:529:12
      at next (lib/runner.js:342:14)
      at lib/runner.js:352:7
      at next (lib/runner.js:284:14)
      at Immediate._onImmediate (lib/runner.js:320:5)

It technically masks the fact that done() is invoked after an error is thrown, but I think that's fine. At that point, the test has already failed.

:+1: in general this is a good idea, but @frankleonrose can you add a test which uses promises?  because a Promise won't throw, emitted won't be set to true, so is it possible we could still get the misleading error?

@boneskull I don't think there's a way to cause this error with promises. The tests written require both failing on the sync path and calling done() on the async path to trigger the "double call" warning. Promises don't have a second way in to the termination code so they can't fail this way.

makes sense.

@frankleonrose thanks! merged


Examples of best way to convert multi-band geotiff to suitable RGB image for tree detection? Any particular parameter settings, etc?

Do you have examples of how to convert multi-spectral satellite imagery to a suitable RGB format for processing with DeepForest? I have been using gdal to convert from multi-band tif (Planet imagery, Sentinel-2 imagery) to JPG. Is this a suitable way to do this? Do you recommend any specific creation parameters?
I'm having trouble getting good results so far. Granted, I am continuing to try different DeepForest parameters to accommodate the coarser resolution of my input imagery. But, I would like to make sure from the start I am using a proper input format of the data.
Thanks.

Hi @eriklindquist - We haven't converted multi-spectral to RGB for this purpose, but I think fundamentally there are two approaches that make sense:

Just convert to RGB (or as close as you can get), which is likely what gdal is doing (but I don't know for sure). This will make trees appear in the same color space the model is built on which should make direct application and transfer learning easier.
Alternatively, you could look for band combinations that give you the clearest distinction between trees and background. This would require more retraining though (and potentially kind of deep in the network) so personally I'd lean towards (1) at the moment.

That said, assuming that you're currently converting to something close to RGB, I suspect that (as you mention) the issue really is more about resolution than that conversion.

Yep...sorry. Was trying some things to see if I could contribute but taking longer than I thought. Will close.


Add support for String Expressions in DataPrepper Expression

Is your feature request related to a problem? Please describe.
Add support to allow expressions returning String type. Also add string operations, like string concatenation using "+" as operator
Describe the solution you'd like
Suggested grammar change for supporting String Expressions
stringExpression
    : stringExpression PLUS stringExpression
    | function
    | JsonPointer
    | String

This allows for following expressions
 getMetadata("key") + "suffix"

/key + "suffix"

Describe alternatives you've considered (Optional)
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

This issue will be addressed in two phase. Phase 1 will add support for expressions returning strings from string literals or json pointers or functions. Second phase will address concatenation operation on strings.

@kkondaka , Is this issue complete?


ES6 class support

Implements #157
Classes are converted to string as:
[class NameOfTheClass]
And instances as:
[instance NameOfTheClass]

I'm sorry, but this feature is beyond the scope of this project. Looks like a very opinionated way of string-representing things. And String(new NameOfTheClass()) should be whatever (new NameOfTheClass()).toString() returns, not [instance NameOfTheClass]

Ok then.


Issue with react v15 - uncontrollable/controllable components

When rendering a custom component, the component gets rendered without a value. When you start typing, it gets a value, thus changing to a controllable input. I think the source of the problem is with the getValue() function. Which might need to be changed to:
getValue: function () {     return this.state._value || "";   }
Not sure if this alone will fix it though. Haven't had this issue in v0.17.
This is the error message I receive:
warning.js:44 Warning: TextField is changing a uncontrolled input of type email to be controlled. Input elements should not switch from uncontrolled to controlled (or vice versa). Decide between using a controlled or uncontrolled input element for the lifetime of the component. More info: https://fb.me/react-controlled-components

Same issue

Also see: https://github.com/erikras/redux-form/issues/735

I think something like @zimt28's defensive fix is the way to go; changing getValue to return an empty string wouldn't be useful for components that don't expect a "texty" type of value.
If you're passing the getValue() value directly to your form control element, you could set a defaultProp of '' (empty string) for value in your component?

+1

Just wanted to point out that it's this.getValue() instead of props.getValue()

@uzarubin The HOC will attach the methods to props, the mixin to this.

+1
Why is this not part of formsy-react's default return for getValue()?

When using your component you should provide a value attribute to suppress the React warning about "elements should not switch from uncontrolled to controlled ". For example:
<MyInput value="" name="password" title="Password" type="password" required />


Add pronto-blacklist runner to readme

pronto-blacklist is useful for blocking additions of blacklisted strings. Such as classes that are being deprecated.

Thanks for your contribution!


Increase build timeout of coverage builds by 5 min

This trend shows that coverage builds for core tend to take close to 20min and we regularly run into timeouts: https://ci.centos.org/job/devtools-almighty-core-coverage/buildTimeTrend . That's why I want to increase the build timeout by 5 minutes.
@sbose78 has already written an email about this: https://www.redhat.com/archives/almighty-public/2017-January/msg00036.html

I'm still facing failures @kbsingh @kwk .
https://ci.centos.org/job/devtools-almighty-core-coverage/193/console

@sbose78 non of the past failing tests lasted longer than 20 min so I guess my change to 25 timeout wasn't deployed when I started the old build. I've started another one just moments ago: https://github.com/almighty/almighty-core/pull/655#issuecomment-275072810 . Let's see if that works.

@sbose78 Problem is fixed and coverage build finished in 21 minutes https://ci.centos.org/job/devtools-almighty-core-coverage/195/

Thank you so much for your help @kwk !


Publish-NavContainerApp -useDevEndpoint

Using the new -useDevEndpoint parameter for Publish-NavContainerApp, I get the following error:
Publish-NavContainerApp : Exception calling ".ctor" with "2" argument(s): "Could not find file
'C:\Windows\ArcherPoint_CKSPackaging_<IP_ADDRESS>.app'."
Script to reproduce:
Update-Module navcontainerhelper
$containerName = 'CKS'
$navdockerimage = 'mcr.microsoft.com/businesscentral/onprem:14.0.29537.0-na-ltsc2019'
$companyName = "CKS Packaging"
$navApp = "..\ArcherPoint_CKSPackaging_<IP_ADDRESS>.app"
Remove-NavContainer $containername
docker pull $navdockerimage
New-NavContainer   -accept_eula:$true
-containername $containerName   -imageName $navdockerimage
-auth Windows   -includeCSide
-doNotExportObjectsToText   -enableSymbolLoading
-usessl:$false   -updateHosts
-assignPremiumPlan   -shortcuts Desktop
-licensefile "..\Licenses\bc.flf"   -additionalParameters @('--volume c:\share:c:\share') Publish-NavContainerApp
-useDevEndpoint   -containerName $containerName
-appFile $navApp   -skipVerification
-sync Add   -install
-tenant default

I think the error tells you what is wrong.
You should specify a full path to your app - I don't expect that you put it in C;\Windows
Relative paths like ..\ should never be used
In PowerShell - the relative path is relative to the current directory - not to the directory of the script.
Use $psscriptroot if you need relative to the script (Join-path $psscriptroot "..\myapp.app") is fine.

I would agree with this assessment, except it works if I remove the -useDevEndpoint parameter. I made no other changes to the script, and left my path as relative.

That might be - the script is following two different paths to publish the app, and the error clearly states that it tries to find it at c:\windows\app.app - which indicates that it resolves .. wrongly.


java.util.NoSuchElementException when using enqueueTime

Some tasks are failing when using enqueueTime:
17/07/05 19:34:14 WARN TaskSetManager: Lost task 10.0 in stage 0.0 (TID 10, <IP_ADDRESS>): java.util.NoSuchElementException
        at scala.collection.LinearSeqOptimized$class.last(LinearSeqOptimized.scala:148)
        at scala.collection.immutable.List.last(List.scala:84)
        at org.apache.spark.eventhubscommon.rdd.EventHubsRDD.compute(EventHubsRDD.scala:119)

Using driver version 2.1.1

would you please provide the log entries around this exception?

There isn't much beyond trying to read from the various partitions. No other logging is present.

the issue is tracked in https://github.com/hdinsight/spark-eventhubs/issues/103 (the description may not be absolutely accurate, as we did test with a "non-exist" timestamp before which turns out to be working)
We will resolve it before the next release

would you also provide the value you pass to this parameter?

and I still want to see the log around the exception, specifically, I need to check the line printed by https://github.com/hdinsight/spark-eventhubs/blob/master/core/src/main/scala/org/apache/spark/eventhubscommon/rdd/EventHubsRDD.scala#L106

The number was<PHONE_NUMBER>, but there was no output from that line in the logs that I could see.

I didn't see it is possible to execute https://github.com/hdinsight/spark-eventhubs/blob/master/core/src/main/scala/org/apache/spark/eventhubscommon/rdd/EventHubsRDD.scala#L119 without printing any log

cannot reproduce


Add Winget installation instructions

I have just added nefcon to Winget:

https://github.com/microsoft/winget-pkgs/pull/95152

I will set up the Winget Releaser workflow in a separate PR.

Superseded by #4


region-syncer: full synchronization of new member

What problem does this PR solve?
let new member sync with the leader.
What is changed and how it works?
do full synchronization with the leader.
Check List
Tests

Unit test
Integration test

PTAL


Adjust klarna test

Closes #416

What I did

How to test

Checklist

[x] Make sure your changes are tested (stories and/or unit, integration, or end-to-end tests).
[x] Make sure to add/update documentation regarding your changes.
[x] You are NOT deprecating/removing a feature.

:tada: This PR is included in version 4.5.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Cell/table related bugs

Grades do not seem to getting saved
enter grade, then refresh
For EC, if you reenter the same score, the + does not come up.
when popover in last few rows, need to scroll down to see it.
warning icon appearing in items originating from other tools - possible to treat entire cell as clickable button for popover?
“Delete” option appearing for item originating from other tool
Cell dropdown tight with comments bubble

All fixed in PR #182.
Re:  "warning icon appearing in items originating from other tools" -- we need to retain the icon as all popovers are now on the icon. So I've replaced the warning icon with a locked icon and styled it so it doesn't stand out too much.

These are all looking good. The only one that still seems to be occurring is the first bullet ("Grades do not seem to getting saved enter grade, then refresh"). The GB still seems to be referring to a specific state (e.g., "?132"), which won't display saved grades when the page reloads. A refresh of the tool itself fixes it.

Thanks @kyleblythe - I conveniently missed that one!  I've created a new issue for that bug: #192.

Closing as only remaining bug has been pulled into its own issue #192.


Update slash-commands.md

Summary

Ticket Link

Hello @Sayanta66,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.


chore(docs): update Care Kit documentation

chore(docs): update Care Kit documentation

Hi @pmarchini ,
I'm having trouble with this PR. Can I ask you for support?
Thanks

Thank you @pmarchini,
I'll try what you suggested.

Actually, @pmarchini, inside the service repository there's not link with the duplication runtime_suite/runtime_suite.
All the links are in the form https://docs.mia-platform.eu/docs/runtime_suite/therapy-and-monitoring-manager/overview and, if I understand it correctly, the docs aggregator should be the one in charge of replacing the first part of the link.


ContainsKey refactoring is repetitive, slow, and presumptuous

Scott Hanselman highlighted the ContainsKey refactoring. There are three issues with this refactoring:

The result isn't DRY. The key is repeated in the ContainsKey call and the call to the dictionary indexer. This opens the door to bugs if one instance of the key is changed but not the other. Additionally, the cost of evaluating the key is doubled, which may be significant if not a simple literal.

Two dictionary lookup are required. Instead, the refactoring should generate TryGetValue.

The refactored code assumes that the proper response to a missing key is to do nothing. Somethings this is correct, but often not. Just as generated code for methods includes throw new NotImplementedException(), this refactoring should include code to throw an exception, which the user can replace with the appropriate handling logic or remove as appropriate.

HI Edward,
thank you for your suggestions. It makes sense to improve the output.

You're right.
/fixed - it's now using TryGetValue.

Great! The switch to TryGetValue takes care of my first two concerns. What a about generating code to handle when TryGetValue returns false? This situation is similar to the code generating for implementing methods of an interface. The code generator knows the methods must be there, but doesn't know what to put in them. I think leaving such flow paths empty creates a pitfall. Just like in the case of generating methods, generating code to throw an exception makes the new path more obvious and ensures that if there is a bug, it leads to a fast fail at runtime, rather than something subtle that takes a long time to track down.

It's arguable - I think it's a bit too much for that refactoring. Generating too much code is often an harassment for the users.
However depending on the knowledge level of the users it's a good idea to to generate an exception for the other path - or a bad one. It's hard to decide. I'll try to use that for a while to look for use cases for one solution or the other - sometimes these things need some time to evolve.

I agree it's subtle, and not obvious what is the best trade-off. I just noticed an existing practice counter-example to including code for an exception. If you type override and the name of a method, when VS inserts the method, it includes the call to the base class's method, but no exception; the method is generally blank. I think the assumption is that since it's just one method and you're right there, you'll likely right the code. This differs from implementing an interface, where several methods are generated at once.
In my experience, there will be an else case over half the time, so it would be nice to generate a placeholder for that, even though you can't predict what will be in it. Here's an idea I'll throw out. Generate this:
if (a.TryGetValue(b))
    Foo();
else
    ;

The last semicolon conveniently causes a compiler warning, which helps with avoiding forgetting, and it's code that will generally be needed anyway.


Failed to install

VSCode Version: 1.22.2
OS Version: Windows 10 Pro

Steps to Reproduce:

Update available popup shows up
Click Install Update
Click Update Now once it downloads
Update looking good until suddenly

Does this issue occur when all extensions are disabled?: code is not a recognizable command any longer.
May 16 02:52:59.299 INFO Starting: C:\Program Files\Microsoft VS Code\Code.exe, false
May 16 02:52:59.305 INFO Checking for running Code.exe processes... (attempt 1)
May 16 02:52:59.308 INFO Code.exe is not running
May 16 02:52:59.309 INFO Checking for possible conflicting running processes... (attempt 1)
May 16 02:52:59.316 INFO Inno Updater v0.4.3
May 16 02:52:59.317 INFO Starting update, silent = false
May 16 02:52:59.367 INFO do_update: "C:\\Program Files\\Microsoft VS Code\\Code.exe", _
May 16 02:52:59.369 INFO move_update: "C:\\Program Files\\Microsoft VS Code\\unins000.dat", _
May 16 02:52:59.384 INFO Delete: "api-ms-win-core-console-l1-1-0.dll" (attempt 1)
May 16 02:52:59.385 INFO Delete: "api-ms-win-core-datetime-l1-1-0.dll" (attempt 1)
May 16 02:52:59.386 INFO Delete: "api-ms-win-core-debug-l1-1-0.dll" (attempt 1)
May 16 02:52:59.388 INFO Delete: "api-ms-win-core-errorhandling-l1-1-0.dll" (attempt 1)
May 16 02:52:59.389 INFO Delete: "api-ms-win-core-file-l1-1-0.dll" (attempt 1)
May 16 02:52:59.390 INFO Delete: "api-ms-win-core-file-l1-2-0.dll" (attempt 1)
May 16 02:52:59.391 INFO Delete: "api-ms-win-core-file-l2-1-0.dll" (attempt 1)
May 16 02:52:59.392 INFO Delete: "api-ms-win-core-handle-l1-1-0.dll" (attempt 1)
May 16 02:52:59.393 INFO Delete: "api-ms-win-core-heap-l1-1-0.dll" (attempt 1)
May 16 02:52:59.395 INFO Delete: "api-ms-win-core-interlocked-l1-1-0.dll" (attempt 1)
May 16 02:52:59.396 INFO Delete: "api-ms-win-core-libraryloader-l1-1-0.dll" (attempt 1)
May 16 02:52:59.397 INFO Delete: "api-ms-win-core-localization-l1-2-0.dll" (attempt 1)
May 16 02:52:59.399 INFO Delete: "api-ms-win-core-memory-l1-1-0.dll" (attempt 1)
May 16 02:52:59.400 INFO Delete: "api-ms-win-core-namedpipe-l1-1-0.dll" (attempt 1)
May 16 02:52:59.401 INFO Delete: "api-ms-win-core-processenvironment-l1-1-0.dll" (attempt 1)
May 16 02:52:59.403 INFO Delete: "api-ms-win-core-processthreads-l1-1-0.dll" (attempt 1)
May 16 02:52:59.404 INFO Delete: "api-ms-win-core-processthreads-l1-1-1.dll" (attempt 1)
May 16 02:52:59.406 INFO Delete: "api-ms-win-core-profile-l1-1-0.dll" (attempt 1)
May 16 02:52:59.407 INFO Delete: "api-ms-win-core-rtlsupport-l1-1-0.dll" (attempt 1)
May 16 02:52:59.408 INFO Delete: "api-ms-win-core-string-l1-1-0.dll" (attempt 1)
May 16 02:52:59.409 INFO Delete: "api-ms-win-core-synch-l1-1-0.dll" (attempt 1)
May 16 02:52:59.410 INFO Delete: "api-ms-win-core-synch-l1-2-0.dll" (attempt 1)
May 16 02:52:59.411 INFO Delete: "api-ms-win-core-sysinfo-l1-1-0.dll" (attempt 1)
May 16 02:52:59.413 INFO Delete: "api-ms-win-core-timezone-l1-1-0.dll" (attempt 1)
May 16 02:52:59.414 INFO Delete: "api-ms-win-core-util-l1-1-0.dll" (attempt 1)
May 16 02:52:59.415 INFO Delete: "api-ms-win-crt-conio-l1-1-0.dll" (attempt 1)
May 16 02:52:59.417 INFO Delete: "api-ms-win-crt-convert-l1-1-0.dll" (attempt 1)
May 16 02:52:59.418 INFO Delete: "api-ms-win-crt-environment-l1-1-0.dll" (attempt 1)
May 16 02:52:59.420 INFO Delete: "api-ms-win-crt-filesystem-l1-1-0.dll" (attempt 1)
May 16 02:52:59.421 INFO Delete: "api-ms-win-crt-heap-l1-1-0.dll" (attempt 1)
May 16 02:52:59.422 INFO Delete: "api-ms-win-crt-locale-l1-1-0.dll" (attempt 1)
May 16 02:52:59.423 INFO Delete: "api-ms-win-crt-math-l1-1-0.dll" (attempt 1)
May 16 02:52:59.424 INFO Delete: "api-ms-win-crt-multibyte-l1-1-0.dll" (attempt 1)
May 16 02:52:59.426 INFO Delete: "api-ms-win-crt-private-l1-1-0.dll" (attempt 1)
May 16 02:52:59.427 INFO Delete: "api-ms-win-crt-process-l1-1-0.dll" (attempt 1)
May 16 02:52:59.428 INFO Delete: "api-ms-win-crt-runtime-l1-1-0.dll" (attempt 1)
May 16 02:52:59.430 INFO Delete: "api-ms-win-crt-stdio-l1-1-0.dll" (attempt 1)
May 16 02:52:59.431 INFO Delete: "api-ms-win-crt-string-l1-1-0.dll" (attempt 1)
May 16 02:52:59.432 INFO Delete: "api-ms-win-crt-time-l1-1-0.dll" (attempt 1)
May 16 02:52:59.434 INFO Delete: "api-ms-win-crt-utility-l1-1-0.dll" (attempt 1)
May 16 02:52:59.435 INFO Delete: "bin" (attempt 1)
May 16 02:52:59.439 INFO Delete: "blink_image_resources_200_percent.pak" (attempt 1)
May 16 02:52:59.440 INFO Delete: "Code.exe" (attempt 1)
May 16 02:52:59.492 INFO Delete: "Code.exe" (attempt 2)
May 16 02:52:59.694 INFO Delete: "Code.exe" (attempt 3)
May 16 02:53:00.148 INFO Delete: "Code.exe" (attempt 4)
May 16 02:53:00.953 INFO Delete: "Code.exe" (attempt 5)
May 16 02:53:02.210 INFO Delete: "Code.exe" (attempt 6)
May 16 02:53:04.020 INFO Delete: "Code.exe" (attempt 7)
May 16 02:53:06.482 INFO Delete: "Code.exe" (attempt 8)
May 16 02:53:09.699 INFO Delete: "Code.exe" (attempt 9)
May 16 02:53:13.769 INFO Delete: "Code.exe" (attempt 10)
May 16 02:53:18.794 INFO Delete: "Code.exe" (attempt 11)
May 16 02:53:18.796 ERRO Access is denied. (os error 5)

Code reopened by itself after clicking OK.

Same error and log file for me updating to 1.23.1 on Windows 10. Looks like a duplicate of at least #49620 and possibly others. I checked process explorer and Code.exe is not running. I also downloaded and installed the update, but unlike @maddes it installed fine, Code.exe was no longer locked.

/duplicate #47841


Update LLVM to e899641df2391179e8ec29ca14c53b09ae7ce85c

@philass For the Stablehlo commit... we want to pull in the exact commit which maps to their LLVM upgrade. So in this case it would be 115941b8b0fc3cd289f741d6734a0040361ae725 if you are using LLVM commit e899641df2391179e8ec29ca14c53b09ae7ce85c. As seen from this PR in the community: https://github.com/openxla/stablehlo/commit/115941b8b0fc3cd289f741d6734a0040361ae725/

@hamptonm1

For the Stablehlo I would try to map the LLVM commit as closely to the Stablehlo commit to avoid incurring more errors

That is what I did https://github.com/openxla/stablehlo/pull/2035

@philass The Stablehlo commit would be [115941b](https://github.com/openxla/stablehlo/commit/115941b8b0fc3cd289f741d6734a0040361ae725) based on what you sent me not what you currently have checked out in this PR. That is what I was implying.

Thank you for working on this update, it would be really good to have. Could you use any help with anything?

@brnorris03 sorry I didn't realize some people were blocked by this.
I'm continuing work on this today.
Right now I'm blocked on two things

A single StableHLO lit test that is failing when run onnx Linux (works on my Mac and the Mac CI pipeline)
Windows build of StableHLO

I'll dig more into the two issues and keep the community posted if I get blocked

Hey @brnorris03,
I'm down to just the argmax issue.
I'm struggling to reproduce this issue locally. I've tried a few different variations of ArgMax.cpp that pass locally and in the mac CI pipeline.
Are you familiar with the StableHLO backend? If you have any spare cycles and could take a look at that test it would be greatly appreciated! If not no worries

Are you familiar with the StableHLO backend? If you have any spare cycles and could take a look at that test it would be greatly appreciated! If not no worries

I'm somewhat familiar, happy to take a look!

@philass please try this argmax.patch for the argmax problem.

I just changed the argmax patch slightly to use ranked tensors since that will be needed in the next upstream update.

@brnorris03 Thanks so much! It seemed to do the trick.
I'll be happy to merge this in, and then review your follow up PR

Jenkins Linux amd64 Build #14404 [push] Update LLVM to e899641df... started at 22:53

Jenkins Linux ppc64le Build #13429 [push] Update LLVM to e899641df... started at 00:03

Jenkins Linux s390x Build #14434 [push] Update LLVM to e899641df... started at 23:54

Jenkins Linux amd64 Build #14404 [push] Update LLVM to e899641df... passed after 2 hr 9 min

Jenkins Linux s390x Build #14434 [push] Update LLVM to e899641df... passed after 2 hr 27 min

Jenkins Linux ppc64le Build #13429 [push] Update LLVM to e899641df... passed after 3 hr 10 min


[WFCORE-893] Add a default io subsystem to profiles using a legacy re…

…moting config
This also tweaks the model test validation code to defer validation of boot ops until after the ModelController's initModel method gets a chance to run. That method is where tests can hook in and register resource definitions that allow the validation to complete.

Linux Build 2432 is now running using a merge of 9fb94e3ea02243f9184676dffb89accff4428a88

Linux Build 2432 outcome was SUCCESS using a merge of 9fb94e3ea02243f9184676dffb89accff4428a88
Summary: Tests passed: 3402, ignored: 64 Build time: 00:36:55

WildFly pull requests Build 1859 outcome was SUCCESS using a merge of 9fb94e3ea02243f9184676dffb89accff4428a88
Summary: Running Build time: 00:00:02

Linux Build 2456 is now running using a merge of c86b05e61affb65bedeba2aa3ba40659cbe661a7

Core - Full Integration Build 2 is now running using a merge of c86b05e61affb65bedeba2aa3ba40659cbe661a7

Core - Full Integration Build 2 outcome was FAILURE using a merge of c86b05e61affb65bedeba2aa3ba40659cbe661a7
Summary: Exit code 1 Build time: 00:02:09

Linux Build 2456 outcome was SUCCESS using a merge of c86b05e61affb65bedeba2aa3ba40659cbe661a7
Summary: Tests passed: 3402, ignored: 64 Build time: 00:35:44

WildFly pull requests Build 1879 outcome was FAILURE using a merge of c86b05e61affb65bedeba2aa3ba40659cbe661a7
Summary: Snapshot dependencies failed: 2 (new) Build time: 00:00:07

Linux Build 2467 is now running using a merge of 416b8917cd8c44854866c65e5515b5b6230c7bc1

Core - Full Integration - Windows Build 15 is now running using a merge of 416b8917cd8c44854866c65e5515b5b6230c7bc1

Linux Build 2467 outcome was SUCCESS using a merge of 416b8917cd8c44854866c65e5515b5b6230c7bc1
Summary: Tests passed: 3402, ignored: 64 Build time: 00:42:37

Core - Full Integration - Windows Build 15 outcome was SUCCESS using a merge of 416b8917cd8c44854866c65e5515b5b6230c7bc1
Summary: Tests passed: 3055, ignored: 358 Build time: 00:52:18

WildFly pull requests Build 1889 outcome was SUCCESS using a merge of 416b8917cd8c44854866c65e5515b5b6230c7bc1
Summary: Running Build time: 00:00:02

@bstansberry I am closing this since I believe it is contained in https://github.com/wildfly/wildfly-core/pull/991


How to build from source

It seems that "make local-build" is the way to compile from source? Is there a reason the docs don't mention this? Can I create a PR and update the docs?

You should use ‘make prod-package’

Actually ‘make prod-build’ should work too, depending on what you’re trying to do.
Indeed we should document it.

Tracking as https://www.pivotaltracker.com/story/show/154919877

we need to extend Makefile docs. @ksallberg for now I can only advise going through Makefile targets to find relevant one. We will prioritize landing the better documentation

@sennui I am willing to improve the Makefile docs at least for the local-build, prod-build etc, and create a PR. That is kind of why I opened this ticket :) Would you appreciate that or should I contribute in some other way?


About implicit prompt refiner or negative prompt in playground.

Thank you so much for your contributions to the Text2Video open-source community! I used the same short prompt with the Mochi model through both the CLI demo and the playground, but I noticed a slight difference in the video quality. Could you let me know if the playground includes an implicit prompt refiner or any negative prompts?
Here are my inputs and outputs:
prompt: a fantasy landscape
from cli:
https://github.com/user-attachments/assets/cc86e04a-42ff-48ab-bea6-8ee802cee6f9
from playground:
https://github.com/user-attachments/assets/52282e79-ed04-48cb-8b25-cd00069441ab
prompt: alley
from cli:
https://github.com/user-attachments/assets/9565bfe5-c891-439d-b291-2bd22617049d
from playground:
https://github.com/user-attachments/assets/24807861-8354-46c7-9804-8902891f1a52
I wonder about the methods you use to enhance video quality. Looking forward to your response!

I noticed that the currently open-source model outputs a video with resolution of 480x848, while the videos generated in the playground have a resolution of 960x1696. Does this mean that the quality difference I've noticed is due to different models, or is it simply because I didn't use a longer prompt or a negative prompt?

They are definitely using a different prompt from what the user says (probably sending it to a LLM to enhance it) and then upscaling the generated video from 480p to whatever they give you. What model are you using fp8, fp16?


Hard to Search for "Entries"

As a user, I want to search through the list of "Entries" for a specific type of form (A specific IN, a specific staff member, a specific proponent, etc.
The way to search right now is very unintuitive:

This needs to be changed to make finding the entry the user wants simple and intuitive.

Please add these filters in the test environment as well.

Dinesh reached out to Formsflow.ai team - we may see updates to the "filtering" UI coming soon.


Verify current release without requiring checkout of current release tag

Expected Behavior
master works for verifying current release
Current Behavior
new contract identifiers (ie StableTokenEUR) cause verification to fail on master

So the question is what the solution should look like. The point of this step is to assert that the network is in the state we expect it to, i.e. working. However, what is working mean and what should be the source of truth? I.e. if a new contract gets added, should the list in source code or the registry be the source of truth?
I think the source code should be considered the source of truth as it is the one that is audited. Prior releases could have gone wrong and not registered a contract in which case the environment should be considered at least possibly faulty.
Hence, the tooling/developers should explicitly acknowledge to a user that a new contract was added, and thus will not be reflected. I.e. the list in the tooling should really have a notion of when it was added.

@alecps and I ran into the same issue during the verify-release step of deploying release 2
cc @nambrot

Alright, I have to reopen this issue again, because the Fix of #6850 and #6899 did not actually work after you consider https://github.com/celo-org/celo-monorepo/pull/6780. Specifically we have two competing desires:

For the purposes of verifying in verify-deployed we need some source of truth of which contracts need to be checked. The CeloContractName enum in registry-utils was thought of the best.
Migrations and tests may be updated to refer to newly added contracts in the CeloContractName

It is clear that we can't just pick one or the other, and we need a more fundamental change to the tooling to reflect this change.

I'm inclined to even give this a P1


[zh-*] Optimize string starts_with, ends_with

Optimize the string to make starts_with and ends_with symmetrical.

Thank you


Incident Report: CarbonPlan outage 2021-07-17

Summary
The cluster was out of commission due to the master and core nodes
(containing k8s control plane + hub components) died, and new
replacements couldn't automatically be brought up due to CPU quota
limits on AWS. Making the core nodes even bigger, and manually
reducing the size of the dask worker instance group (497 was
requested, 320 had been provisioned) brought everything back.
Timeline (if relevant)
All times in IST
2021-07-17 2:38 PM
In the course of writing up a previous incident report,
it was observed that the cluster was having another outage - the k8s API
and the hub were both unreachable. Investigation is started.
2:41 PM
Looking at the instances page
on the AWS console in the carbonplan account showed two master nodes,
even though our k8s API wasn't able to reach them. Upon finding their
public IP and sshing in, it was found that one of the master nodes was
maxed out on CPU, due to dask gateway using up a lot of CPU. The gateway
pods themselves didn't have a CPU limit, so had taken down the entire
node.
2:46 PM
The problematic node is rebooted via ssh, and comes back on.
The kubernetes API is reachable sporadically, but the hub is not.
2:52 PM
The core node size is increased again, to m5.2xlarge. However,
EC2 doesn't bring up the new nodes, since we were hitting up our
CPU quotas. Looking at the Activity tab
in the AWS console for the autoscaling group for the master
shows the following error message:

Launching a new EC2 instance. Status Reason: You have requested more vCPU
capacity than your current vCPU limit of 1362 allows for the instance bucket
that the specified instance type belongs to. Please visit
http://aws.amazon.com/contact-us/ec2-request to request an adjustment to this
limit. Launching EC2 instance failed.

The dask-r5-2xlarge instancegroup had been asked to scale up to 500,
and already had about 320 nodes. This had exhausted the CPU quota, causing
problems with bringing up the new master nodes.
3:00 PM
The dask-r5-2xlarge autoscalinggroup is manually set to 0 nodes, and
AWS starts bringing these nodes down. This allows the master to come up,
and start bringing everything back.
3:11 PM
Everything is functional again.
What went wrong

The dask-gateway pods had no CPU limit, so they were able to use
enough CPU to take down the entire cluster.
The hub control plane and the k8s API control plane share nodes,
so problems with the hub control plane can take down the whole
cluster.
We don't have full awareness of quotas on AWS, and how they
might affect workflows.

Where we got lucky

The problem occured on a saturday, and was only luckily discovered
at that time.

Action items
Process improvements

Look at AWS quotas for carbonplan, and request increases where
needed.
Add 'look at quotas' as part of process when setting up a
hub, to evaluate along with users' expected needs

Documentation improvements

Document how to ssh into a node in a kops cluster
Document how to identify the appropriate autoscaling group for
an instancegroup so you can look at its activity report.
Document how to perform manual nodegroup scaling operations
for each instancegroup via the AWS console.

Technical improvements

Explicitly set limits and requests on all hub control plane
pods. https://github.com/2i2c-org/pilot-hubs/pull/525
Setup Grafana / Prometheus for the cluster so we can diagnose
issues better. https://github.com/2i2c-org/pilot-hubs/pull/533
Increase the master node redundancy of the cluster even more.
https://github.com/2i2c-org/pilot-hubs/pull/525 https://github.com/2i2c-org/pilot-hubs/pull/532

Hi folks! At this time, the hub seems to have gone down again.

Thanks again for the write-up, @yuvipanda.
Planning to close this one soon unless you want to keep it open for some reason.
Btw, for historical context, @jhamman's above ping was acknowledged and worked out in Slack.

Hey all - I've moved this to the Activity board so that we don't lose track of it, also added a little checklist to make sure we follow-up on creating issues before closing.
I'm not sure what over the documentation/process steps above need their own issues...@yuvipanda could you advise there?

I'm not sure what over the documentation/process steps above need their own issues...@yuvipanda could you advise there?

Ping @yuvipanda :wink:
IMHO, all of those docs points belong to a new AWS debugging section, maybe???

OK I've added tracking issues for the process + documentation items that @yuvipanda had in the top comment, so going to close this


clause_tokenize returns an empty list.

clause_tokenize should accept a list of strings (words) and return a list of list of strings (clauses). At the moment, it always returns an empty list.
Description

As stated above, clause_tokenize returns an empty list.  I followed the example in the doc, but it did not produce the expected results. The issue still persisted after I downloaded lst20-cls.
from pythainlp.corpus import download
download('lst20-cls')

Corpus: lst20-cls
- Downloading: lst20-cls 0.2
100%|███████████████████████████████████████████████████████████████████| 3738912/3738912 [00:00<00:00, 19519586.71it/s]
True

Expected results

[['ฉัน', 'นอน'], ['และ', 'คุณ', 'เล่น', 'มือถือ'], ['ส่วน', 'น้อง', 'เขียน', 'โปรแกรม']]

Current results

[]
Steps to reproduce

from pythainlp.tokenize import clause_tokenize
clause_tokenize(["ฉัน","นอน","และ","คุณ","เล่น","มือถือ","ส่วน","น้อง","เขียน","โปรแกรม"])

See the returned result.

[]

Your environment

PyThaiNLP version: 2.3.1
Python version: 3.8.10
Operating system and version (distro, 32/64-bit): Ubuntu (64-bit)
More info (Docker, VM, etc.): WSL2

Thank you for your report. We were release PyThaiNLP 2.3.2 for fix this issue.


Fix for incorrect function name in documentation example

[Describe your pull request here. Please read the text below the line, and make sure you follow the checklist.]
Function name in json/docs/mkdocs/docs/features/arbitrary_types.md should be "from_json(...)" instead of "to_json(...)"

Pull request checklist
Read the Contribution Guidelines for detailed information.

[ ]  Changes are described in the pull request, or an existing issue is referenced.
[ ]  The test suite compiles and runs without error.
[ ]  Code coverage is 100%. Test cases can be added by editing the test suite.
[ ]  The source code is amalgamated; that is, after making changes to the sources in the include/nlohmann directory, run make amalgamate to create the single-header files single_include/nlohmann/json.hpp and single_include/nlohmann/json_fwd.hpp. The whole process is described here.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Specifically, I am aware of compilation problems with Microsoft Visual Studio (there even is an issue label for this kind of bug). I understand that even in 2016, complete C++11 support isn't there yet. But please also understand that I do not want to drop features or uglify the code just to make Microsoft's sub-standard compiler happy. The past has shown that there are ways to express the functionality such that the code compiles with the most recent MSVC - unfortunately, this is not the main objective of the project.
Please refrain from proposing changes that would break JSON conformance. If you propose a conformant extension of JSON to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

Thanks!