python311Packages.google-cloud-datacatalog: 3.15.2 -> 3.16.0

Changelog: https://github.com/googleapis/python-datacatalog/blob/v3.16.0/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 258748 run on x86_64-linux 1

8 packages marked as broken and skipped:

python310Packages.dbt-bigquery
python310Packages.dbt-bigquery.dist
python310Packages.google-cloud-bigquery
python310Packages.google-cloud-bigquery.dist
python311Packages.dbt-bigquery
python311Packages.dbt-bigquery.dist
python311Packages.google-cloud-bigquery
python311Packages.google-cloud-bigquery.dist

4 packages built:

python310Packages.google-cloud-datacatalog
python310Packages.google-cloud-datacatalog.dist
python311Packages.google-cloud-datacatalog
python311Packages.google-cloud-datacatalog.dist


build: docker

I feel this doesn't belong in aux-ts folder.

I feel this doesn't belong in aux-ts folder.

where should it live instead? i put it here because we're containerizing aux-ts/graphql

I feel this doesn't belong in aux-ts folder.

where should it live instead? i put it here because we're containerizing aux-ts/graphql

should be fairly straightforward to use something here like here https://github.com/aux-exchange/aux-exchange/tree/main/aptos/testing
(However, I feel graphql should not be in aux-ts, since it's actually not part of the api).

I feel this doesn't belong in aux-ts folder.

where should it live instead? i put it here because we're containerizing aux-ts/graphql

should be fairly straightforward to use something here like here https://github.com/aux-exchange/aux-exchange/tree/main/aptos/testing
(However, I feel graphql should not be in aux-ts, since it's actually not part of the api).

i think putting these files in a separate directory right now is a bit confusing though, because Dockerfile typically colocates with the source directory
i do agree graphql should be separated. a while back it was separate, but then we combined them. maybe we can create an issue to separate out graphql and figure out the proper npm packaging from there?


Implement 'create named-graph dataset' command

Allow user to create a named graph within an existing dataset resource

Below is a sample structure of a single json-ld document with one node in the default graph (@id:_2342) along with two named graphs defined in the default graph (@id:myNamedGraph1, @id:myNamedGraph2).  The command should allow for inserting a new named graph into the default graph or another named graph. The addressing to specify the parent target should probably be a hierarchy (like insert node   //<my named graph 2> where  will be inserted within <my named graph 2> .
Some rules that I am aware of according to JSON-LD spec:

names of named graph must be unique with respect to the entire json-ld document. This means once a name is used in the document it can no longer be used again even if there is no naming collision at a different level of the hierarchy.
A named graph is such when it is a json object that has two attributes (only) which are @graph and @id where graph has an array value and @id contains an IRI (see spec for definition).

`{
"@context": {
"Person": "http://xmlns.com/foaf/0.1/Person",
"name": "http://xmlns.com/foaf/0.1/name",
"knows": "http://xmlns.com/foaf/0.1/knows"
},
"@graph": [
{
"@id": "myNamedGraph1",
"@graph": [
{
      "@id": "http://person.A",
      "@type": "Person",
      "knows": "http://person.B",
      "name": "A"
    }
  ]
},
{
  "@id": "myNamedGraph2",
  "@graph": [
    {
      "@id": "http://person.B",
      "@type": "Person",
      "knows": "http://person.A",
      "name": "B"
    }
  ]
},
{
   "@id":"_:2342",
   "name":"my standalone node"
}

]
}`

looks like the following syntax is implemented:
dogg3rz create namedgraph --dataset ds3 mygraph
however when I used the --parent option like
dogg3rz create namedgraph --dataset NAME --parent mygraph mysubgraph
I get no errors but a malformed json-ld document as show here:
{
  "@context": {},
  "@graph": [
    {
      "@graph": [
        {
          "@graph": [],
          "@id": "mysubgraph"
        }
      ],
      "@id": "mygraph",
      "__dgrz_mtimes__": {
        "2e5de37e8af76828fd9667700b4c54c02a5694c807ceebd10f23b08a8bc5744d":<PHONE_NUMBER>844281000
      }
    }
  ],
  "__dgrz_mtimes__": {
    "25dfd29c09617dcc9852281c030e5b3037a338a4712a42a21c907f259c6412a0":<PHONE_NUMBER>530809000,
    "9dcf97a184f32623d11a73124ceb99a5709b083721e878a16d78f596718ba7b2":<PHONE_NUMBER>530809000,
    "9e25d6f9a056fe59ab9abb4265350e8f9b903cac597bd960955bb2395b07bab9":<PHONE_NUMBER>923135000
  }
}


Allow deduplication of eslint-scope

Now that the eslint-scope worm has been resolved, can babel-eslint be updated to depend on ^3.7.3 rather than 3.7.1, so that the package can be deduped again?
(For instance, webpack depends on ^3.7.1, so a project containing both webpack and babel-eslint will have both 3.7.1 and 3.7.3 installed.)

Thank you for the issue. Now that @babel/eslint-parser has been released, we are making this repository read-only. If this is a change you would still like to advocate for, please reopen this in the babel/babel monorepo.


feat: add some plugins

Fixes: #1379
Changes:
Add some plugins
Remainning problems:
Some plugins  added missing pictures.
Screenshots of the change:

@uniqie could u add default plugin img? Also could u improve the components? I know you have great technical skills. 😇
You r too highly of me. 🐶

@uniqie Okay, could u help us create an issue to track this?


Unclear error message if winrt/Windows.Foundation.h isn't included before awaiting an IAsyncOperation

When building the following code
#include <iostream>
#include <winrt/Windows.System.h>

winrt::fire_and_forget OpenUri(const winrt::Windows::Foundation::Uri &uri)
{
	bool opened;
	try
	{
		opened = co_await winrt::Windows::System::Launcher::LaunchUriAsync(uri);
	}
	catch (const winrt::hresult_error& err)
	{
		std::wcout << std::wstring_view(err.message());
	}

	// do something if opened is false
}

The error error C2338: Not an awaitable type is given, which is confusing because LaunchUriAsync returns an IAsyncOperation<bool>, which is indeed an awaitable type. This happens because the awaitable support machinery is in <winrt/Windows.Foundation.h>, which was never included. The solution is to include that header.
cppwinrt should try giving a better error in these cases (for example, <winrt/Windows.Foundation.h> needs to be included to await IAsyncOperation).

I'm not sure how cppwinrt as a library could do any better since the error comes from the compiler. Do you have a suggestion?

Actually, the error comes from cppwinrt:
https://github.com/microsoft/cppwinrt/blob/464e59198c2046125d8ee5a8cc6ff1739097345f/strings/base_coroutine_threadpool.h#L239
I'm thinking we could forward-declare IAsyncAction, IAsyncOperation<T> and friends in this file, then if the case that contains this assert is hit, use an if constexpr to detect that it's an IAsyncAction/IAsyncOperation<T>/etc. and adjust the error message accordingly.

Ah, forgot about that... get_awaiter is some serious hackery to enable suspension point hooking, which is an extremely brittle feature requested by the OS team and, as far as I'm aware, to date unused. Gor tells me that the standard library will eventually make it redundant. I would be inclined to remove it in future but certainly not build on it.  If you can find a simple way to make that work I'm not apposed to it but there's just a lot of technical debt piled up here that really needs to be paid down first.


Changed aditya's bio

@leewujung Take a look and merge if you like it!

Looks good! Merging now.

Actually, I just noticed that with your github handle change, the github icon link becomes broken in your profile. Could you send in another PR for that?


[Suggestion] It would be better to change the setting "Azure Resource Groups: Group By" as a drop-down list to select how azure resources are grouped

OS: All
Build Version: 20230509.1
Regression: Not a regression
Repro Steps:

Open the Settings.
Check the Azure Resource Groups: Group By.

Actual:
This setting needs to input how azure resources are grouped.
Suggestion:
It would be better to change the setting "Azure Resource Groups: Group By" as a drop-down list to select how azure resources are grouped.

This is by design due to how we are support ARM tags


Reset section > ul width on mobile

It looks like the fix for #91 introduced a selector section > ul which is more specific than @media(...) { ul {} }. This means that lists on mobile don't get their width reset to take up the full screen.
In particular, checkout this line. You could probably just change this to

section > ul {...}, or
ul, section > ul {...} if you wanted to be defensive about things.

(Though I wonder: the current behavior might mean that the same issue from #91 still applies to lists on mobile. If this hunch is right, then you almost definitely want option 1).
Steps to reproduce

This item is long enough that it would have to wrap when displayed on a mobile screen, because phone screens are small.

Load that on a mobile phone, or a desktop browser with a screen narrow enough to trigger the CSS media query breakpoint.

Observe that the list item takes up about half the screen, instead of the full mobile width.

Sorry for the long wait on this. You were absolutely right. Good catch, and I appreciate the well-written ticket. 💯


Enforce strict mode

Fix all warnings; no impact on code logic.
That is my last PR.

Thank you very much!


Fix #464 - Add emit event to blur and focus listener

The code has been tested here https://codesandbox.io/s/xjo8m6j75p.

@nmfzone Thank you very much for the contribution


Matchzoo ImportError: cannot import name 'Loss' from 'keras.losses'

When following DSSM quick 60s start, and importing "import matchzoo as mz",
get the following error:

Traceback (most recent call last):
File "", line 1, in
File "/Users/user/git_repo/MatchZoo/matchzoo/init.py", line 28, in
from . import losses
File "/Users/user/git_repo/MatchZoo/matchzoo/losses/init.py", line 1, in
from .rank_cross_entropy_loss import RankCrossEntropyLoss
File "/Users/user/git_repo/MatchZoo/matchzoo/losses/rank_cross_entropy_loss.py", line 6, in
from keras.losses import Loss
ImportError: cannot import name 'Loss' from 'keras.losses' (//anaconda3/envs/matchzoo/lib/python3.7/site-packages/keras/losses.py)

Probably, due to Keras version (mine is 2.2.4) and I can't install 2.3.0, since It's now available
for MacOS yet . My MacOS version is 10.14.5.
Is it possible to install previous version of MatchZoo so that it uses older keras?

The current version of matchzoo is not compatible with your keras version. You may use matchzoo version 2.1 instead.


optional handling for unhandled options

I have a case where I want to store unhandled options in a std::vector<std::string> and use later.
I'm thinking adding a std::vector<std::string> parameter to the parse method with a default value of nullptr would suffice.  If parameter was not nullptr the unhandled options would be stored in the vector reference.  If nullptr, it would follow the existing logic.
Would you accept a PR for this?

already supported with result.unmatched()


fix: old CSS cache is still used after refreshing the page. #1087

From this issue #1087
In the development environment, whenever the page is refreshed, the old .css file is cached by the browser.
This will cause the original file to be modified and the hot update is successful, but the old style may appear when the page is refreshed, causing confusion to users
so in order to fix it, I change the catch control of the. CSS file.
Hope it helps.

It will result in all CSS files not being cached...

It will result in all CSS files not being cached...

During development, I think that non-caching .css file is in line with expectations.


Testing on Vagrant

Hello, I'm very interested on this role. I was using puppet for Oracle provisioning but now I'm more involved on Ansible.
I couldn't make it work for me.
Could you provide your Oracle linux box? I think there's something wrong with mine, I'm using this box: https://atlas.hashicorp.com/terrywang/boxes/oraclelinux-6-x86_64
Error:
TASK [oracle-database : file] **************************************************
failed: [test01] => (item=/home/oracle/oracle_inventory) => {"failed": true, "item":          "/home/oracle/oracle_inventory", "module_stderr": "", "module_stdout": "sudo: a password is required\r\n", "msg": "MODULE FAILURE", "parsed": false}
failed: [test01] => (item=/home/oracle/product) => {"failed": true, "item": "/home/oracle/product",    "module_stderr": "", "module_stdout": "sudo: a password is required\r\n", "msg": "MODULE FAILURE", "parsed": false}
failed: [test01] => (item=/home/oracle/product/oracle_home) => {"failed": true, "item":    "/home/oracle/product/oracle_home", "module_stderr": "", "module_stdout": "sudo: a password is   required\r\n", "msg": "MODULE FAILURE", "parsed": false}

Vagrantfile
config.vm.define "test01" do |test01|
    test01.vm.box = "terrywang/oraclelinux-6-x86_64"

    test01.vm.provider "virtualbox" do |vb|
      vb.cpus = 1
      vb.memory = "1500"
    end

    test01.vm.provision "ansible" do |ansible|
      ansible.playbook = "main.yml"
      ansible.galaxy_role_file = "roles.yml"
      ansible.sudo = true
     ansible.sudo_user = "root"
   end
 end

I think you can use "ask_sudo_pass" in vagrant configuration if you know root's password. I have put my base box as public: https://atlas.hashicorp.com/jeqo/boxes/ol7-ansible
If you want to check what it contains: https://github.com/jeqo/packer-linux
Scripts and kickstart file have all packages installed. I'm using Oracle Linux 7.2
Let me know how it goes.
Cheers!

Check if this image works for you: https://atlas.hashicorp.com/jeqo/boxes/oracle-database-11g-se-ol7
it has already installed Oracle Database 11g (<IP_ADDRESS>) Standard Edition in OL7, and you can use configureinstance and configurelistener tags if you want to create a new instance.

I will close this. Let me know if it's resolved to reopen.


Add ability to normalize INDEL variants

Similar to #1709, but for variants.

+1

This algorithm looks reasonable
http://genome.sph.umich.edu/wiki/Variant_Normalization#Implementation
The comparisons are a bit old though, is this still the current best practice?

@heuermh that algorithm is more or less what is implemented for the CIGAR normalization on reads. My preference is not that we add a new algorithm, but rather that we refactor the current read normalization algorithm so that it can also take variants.
To be more specific WRT the changes needed, for reads, normalization is bounded since we can't shift the INDEL before the start of the read alignment. I mean, I do suppose we could, but that would be unusual since we would've expected the read to either be soft-clipped during alignment or aligned without an INDEL. For variants, we don't have a bound that is known a priori from the variant itself. E.g., consider a deletion in a STR. Normalization should move that deletion all the way to the start of the STR, but we don't know how long the STR is from the variant that we are normalizing.

Closing as WontFix, Cannoli provides multiple methods for normalizing variants (bcftoolsNorm, vtNormalize)


Unsure how to use callbacks

Hi, I was hoping for some examples of how to do callbacks. My code:
var stmt = db.prepare('INSERT or IGNORE INTO entities VALUES (?, ?)');
                        try {
                            stmt.run("audiobook", name);
                        } catch (err) {
                            console.error(err);
                        }

    var mname = db.prepare('SELECT * FROM activities WHERE type="audiobook"');
    var rows;
    try {
        rows = mname.get();
    } catch (err) {
        console.error(err);
    }

better-sqlite3 is a synchronous library, so there are no callbacks. Everything about your code above is perfectly valid as-is.


LICENSE

Hi,
Thank you for your code. Could you add a license so people can use it? E.g. MIT or GPLv3.
Thanks,
Jeff

MIT License was added. Thanks for reminding me!

Thank you!


It does not work

Describe the bug
Just doing thins
        dd(User::all());

and it does not work, i get all the useless info
What version does this affect?

Laravel Version: latest Laravel 10
Package Version: 1.1.0

+1, I just tried it because the idea seems pretty neat, however, it doesn't do anything at all. It seems like there was an update in Laravel that practically killed this package.

Should be fixed in the next version.


Session is saved twice

Not sure I understand what this code is originally meant to do but in \Dingo\Api\Http\Middleware\Request::terminate this is done:
foreach ($middlewares as $middleware) {
            list($name, $parameters) = $this->parseMiddleware($middleware);

            $instance = $this->app->make($name);

            if (method_exists($instance, 'terminate')) {
                $instance->terminate($request, $response);
            }
        }

This causes all middlewares to be terminated a second time, it doesn't matter for most but it causes the StartSession middleware to save the session twice, also aging flash data twice. Basically flash messages don't work anymore since they're aged twice before anything can happen.

Introduced in 05cbc2918e118ad71a0f44dfd754552c7388c27e

Laravel or Lumen?

Laravel 5.2

Cheers, I'll take a look, definitely shouldn't terminating them twice.
I'd like to point out that an API shouldn't really save any state between requests either.

Are you doing anything else? When I hit an endpoint with it enabled I'm only seeing it terminated once. I threw a \Log::info('session terminated') in the StartSession.php middleware, it's only logged once per request.
I've also tried with internal requests. Seems to be working fine.
So any code you can provide to help replicate this would be great.

Thanks for looking into this. I'm encountering the exact same problem and was able to reproduce like this:
Did a new laravel install
Added Dingo with
composer require dingo/api:1.0.x@dev
Added provider to app.php
(...)
Illuminate\Validation\ValidationServiceProvider::class,
Illuminate\View\ViewServiceProvider::class,
Dingo\Api\Provider\LaravelServiceProvider::class,

/*
 * Application Service Providers...
 */
App\Providers\AppServiceProvider::class,
App\Providers\AuthServiceProvider::class,
(...)

Used these settings in .env:
API_STANDARDS_TREE=vnd
API_SUBTYPE=testinstall
API_PREFIX=api
API_VERSION=v1
API_NAME=testinstall
API_STRICT=false

Set up my routes like this:
Route::group(['middleware' => ['web']], function () {

    Route::get('/notify',function(){
        Session::flash('msg','Hey, You have a message to read');
        return Redirect::to('/result');
    });

    Route::get('/result', function () {
        dd(Session::all());
    });

});

When I call /notify, it redirects to /results which shows the following information (empty flash):
array:3 [▼
  "_token" => "yTp0B8ws7A9pXRo7bM3NjNbJnqn6BbUtqNwCyo5r"
  "flash" => array:2 [▼
    "new" => []
    "old" => []
  ]
  "_previous" => array:1 [▼
    "url" => "http://localhost:8000/notify"
  ]
]

This is the result page when i disable (comment out) the Dingo api in app.php:
array:4 [▼
  "_token" => "yTp0B8ws7A9pXRo7bM3NjNbJnqn6BbUtqNwCyo5r"
  "flash" => array:2 [▼
    "new" => []
    "old" => array:1 [▼
      0 => "msg"
    ]
  ]
  "_previous" => array:1 [▼
    "url" => "http://localhost:8000/notify"
  ]
  "msg" => "Hey, You have a message to read"
]

I agree API's should not save state, but in my case I'm building a Laravel app with a website and API (for mobile app purposes), where the website will be consuming the API. I wish to use flash messaging on the website.

Yeah this was an oversight on my behalf, thanks for the code samples. Dingo wasn't checking if the request was an API request before terminating the middleware, so it was occurring twice for non-API routes.
Pushed a patch so that only API requests are terminated by Dingo, since they aren't terminated by Laravel/Lumen.
Let me know if things look better now.

All tests are fixed for me as well.

Don't have time to check if it fixes it for me on my end but what @jasonlewis describes sounds like what I had (ie. it was happening on web requests, not API ones) so it should be good as well


feature-request: add a keyboard shortcut for print menu

From<EMAIL_ADDRESS>on 2011-03-10T09:33:40Z
Using shortcuts can make the work with Spyder much more efficient.
Learning these is not easy.
I would like to have a possibility to print the table of the shortcuts contained in the prefernces settings dialogue.
This would serve as quick reference.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=578

From ccordoba12 on 2011-03-22T06:40:26Z
Labels: -Type-Defect Type-Enhancement

From pierre.raybaut on 2011-05-16T01:29:01Z
Labels: Cat-Settings

From pierre.raybaut on 2011-05-18T00:31:24Z
Status: ContributorNeeded

From<EMAIL_ADDRESS>on 2011-11-10T05:52:56Z
added Ctrl+P as shortcut for File->Print option.
File attached
Attachment: editor.py

From<EMAIL_ADDRESS>on 2011-12-18T15:52:36Z
How do I propose incorporate the changes?

From<EMAIL_ADDRESS>on 2011-12-19T09:43:40Z

Create a patch, so that your changes are visible from the tracker
without downloading and comparing editor.py

Comment #5 on issue #578 by<EMAIL_ADDRESS>feature-request: add a print
option for keyboard shortcuts https://code.google.com/p/**spyderlib/issues/detail?id=578http://code.google.com/p/spyderlib/issues/detail?id=578 How do I propose incorporate the changes?

From<EMAIL_ADDRESS>on 2011-12-22T07:24:21Z
I created this patch using 'diff -Naur spyderlib/plugins/editor.py ../spyder-2.1.6-original/spyderlib/plugins/editor.py > print_shortcut.patch ' against the current stable.
The patch is attached. Is this enough/okay?
Attachment: print_shortcut.patch

From<EMAIL_ADDRESS>on 2011-12-22T08:01:24Z
No. The change is opposite. You need 'diff oldfile newfile". The easiest ways to do this is https://code.google.com/p/spyderlib/wiki/NoteForContributors#Running_from_source and use hg diff.

From<EMAIL_ADDRESS>on 2011-12-22T08:27:54Z
Thanks. Attached is the output of hg diff > print_shortcut.patch
Attachment: print_shortcut.patch

From ccordoba12 on 2011-12-22T15:04:00Z
Hi fheday, thanks for your contribution. We will surely push your patch to the main repo because it adds a missing functionality. However, it doesn't really solve the issue as reported by the first person, i.e. Tim.
He wants a way to print the whole table of Spyder shortcuts, the one you find in Tools > Preferences > Keyboard shortcuts
Thanks for taking the time to improve Spyder and for your effort,
Carlos

From ccordoba12 on 2011-12-22T16:34:53Z
I uploaded the patch. It works great! See revision 7fae533eef16

From<EMAIL_ADDRESS>on 2012-02-28T23:03:30Z
This feature request was hijacked, so I changed the title to avoid confusing and time waste.
Tim, could you please open a new issue for your enhancement?
Summary: feature-request: add a keyboard shortcut for print menu
Status: Fixed
Owner: ---


is_nan vs isnan

In the STL it's std::isnan and it's also xsimd::isinf.
So is there a reason for xsimd::is_nan or would it make sense to rename it to xsimd::isnan to adhere to STL and unify the xsimd interface?

Good catch! Indeed that's mistake, it should be renamed into  xsimd::isnan


Bug: release workiflow is invalid

Is there any existing issue for this?

[X] I have searched the existing issues

Description
After running release workflow its showing invalid
Screenshots
No response
Any additional information?
No response
What browser are you seeing the problem on?
No response

This is now fixed


adjust to role replacing author-comment

Cf. https://github.com/AmerMathSoc/texml/issues/208

I've asked for clarification upstream. If we're lucky, this is really just a single tag name change, in which case only
https://github.com/AmerMathSoc/texml-to-html/blob/34823437f16f44bcf0b4305f9e8773e00f90a237/lib/helpers/extractContribGroups.js#L78-L80
needs adjustment.

As per F2F (see upstream issue), we can go ahead with a simple replacement.


Make Final Approval Date Required in the GCI

Currently, choosing the Final Approval Date in the GCI is optional. We need to make this required.
@ErinRiggs @courtneythaxton

We already had this ticket, see here: https://github.com/ClinGen/gene-and-variant-curation-tools/issues/34 #34


fix checkov warnings regarding untrusted input

Codecov Report

Merging #207 (82243f3) into master (6b9eada) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 82243f3 differs from pull request most recent head 512efec. Consider uploading reports for the commit 512efec to get more accurate results

@@           Coverage Diff           @@
##           master     #207   +/-   ##
=======================================
  Coverage   91.33%   91.33%
=======================================
  Files         267      267
  Lines        6463     6463
  Branches     1288     1288
=======================================
  Hits         5903     5903
  Misses        306      306
  Partials      254      254

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Package for Linux, Mac and maybe Windows

See #30 for rationale.

I think that we can close this now that we have Docker images build along with each new Python package. Thanks @mkjmdski !


Security vulnerabilities in PyYAML version 5.1: CVE-2020-1747 and CVE-2019-20477

Hi!
Would it be possible to update the PyYAML version in order to fix CVE-2020-1747 and CVE-2019-20477 ?
Thank you!

Oh dear, this should've been closed forever and a day ago. We've been running PyYAML 5.3.1 for quite awhile.


Instagram getSelfUserDetails bad request 400 eventhough session is valid

I think this issue is related to PR.
What do you think about?
#285

@kkak10 and @jz709u Using that PR's repo for my Podfile corrected this for me.  Hope we can get this merged in soon.  Not sure exactly when or how this broke.  I swear when I first setup InstagramKit's current version in my app 10 months ago everything worked fine.


Possible V1.4.1

This is a collection of PRs that should be considered if we decided to prepare a V1.4.1 release of HPX:
(please keep this list sorted)

#4357: Adding missing #includes to fix gcc V10 linker problems
#4353: Skip MPI_Finalize if MPI_Init is not called from HPX
#4343: Give a hard error if IO counters are enabled on non-Linux systems
#4337: Installing pdb files on Windows
#4335: Adding capability to buildsystem to use an installed version of HPX
#4322: Undefine VT1 and VT2 after boost includes
#4315: Forcing exported symbols from composable_guard to be linked into core library
#4310: Remove environment handling from exception.cpp

Please add more as we go or if you think certain patches should go into a point release

This can be closed now as V1.4.1 was released


implement rand and srand

The stdlib library doesn't have a random number generation feature yet. I want to add this feature to the library.

that would be great! assigned !

This is already done, isn't it?


add more plotting programs (back) into Pmag GUI

core_depthplot replacement iodp_funcs.make_plot
ani_depthplot
aniso_magic
??

It looks like we're going in a different direction and not trying to add more functionality to the GUIs, so I am closing this issue.


Crash on startup, graphical errors

Version: 1.20.1
Loader: Forge

Crash log: https://mclo.gs/yeGwfs4
Game crashes on startup, after game loads it takes you to what should be the crash screen but is the above image. Proceeding past this screen gives the same text graphical bug as seen below:

Attempting to load a world from here leads to the same crash. Uninstalling only this mod fixes the error.

Oh dear! I do not immediately know what could be causing something like this. I have never had any issues like this in all my time bouncing between versions and loading my mod up with friends. I'll take a more in depth look at the log for details in a day or so.

Oop, looks like I don't! I installed through AT Launcher, it usually adds in the dependencies automatically. Might be that that isn't listed in a way Curseforge can detect? That seems to have fixed everything though, thank you very much!

It's listed as a dependency, ATLauncher will let you install mods without dependencies if you go quick.


feat(linter/tree-shaking): support options

As the new added test cases show, the usage is relatively strict and not very convenient to use.
It's will become more useful if we have some way to interaction with imported function/object.

#3504  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @mysteryven and the rest of your teammates on  Graphite


python3Packages.dbutils: 2.0 -> 2.0.1

Motivation for this change
Update to latest upstream release 2.0.1
Change log: https://webwareforpython.github.io/DBUtils/changelog.html
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 129015 at 745db4ce run on x86_64-linux 1

2 packages built successfully:

python38Packages.dbutils
python39Packages.dbutils


let a chance for SIGPIPE for cgi scripts to terminate before using SIGTERM or even SIGKILL

Hello,
When client aborts a request on a CGI script (mod_cgi) with h2 protocol activated, it seems that the script process is killed so that shutdown function are never called. As far as I know Apache will send a SIGPIPE with HTTP/1.1 to let scripts detect end of stream then gracefully finish
exemple script to try (yes, this is "php cli as cgi") :

test.cgi

#!/usr/bin/php
<?php
error_log('script started');
$done=false;
register_shutdown_function('abort');

echo "Content-type: text/html\n\n";
echo "\n\n";
flush();

for($i = 1; $i <= 60; $i++)
{
	sleep(1);
	echo "$i
\n";
	error_log("$i");
}
$done = true;

function abort()
{
	global $done;
	if(!$done) error_log("aborted");
}
error_log('script ended');
?>

without H2 protocol, when I cancel request before 60s I get "aborted" in log
with H2 protocol, when I cancel before 60s I don't get "aborted" in log

Hmm, testing with mod_cgion OS X I do not see an aborted in my logs, neither http/1.1 nor http/2. Both behave the same and stop logging the number a second or two later.
Do you have any more data to reproduce this? Also, I do not see the mentioned kill() invocation in the code, but I am not expert in these modules...

maybe that osx version of PHP does not support register_function_shutdown() in cli mode?
It is PHP 5.6.14 and it must not be an interactive shell (php -a)
Since the request is aborted I can't print out on the screen the content of the shutdown function excepted via error_log
will try to get some debug from mod_cgi

Which version of mod_http2 are you using, btw.?

1.2.5

no debug from mod_cgi
the only configuration manipulation between apache restarts is commenting/un-commenting Protocols directive
for cgi config, I have only
AddHandler cgi-script .cgi
Options +ExecCGI
not using suexec
chmod 755 test.cgi
chown apache:apache test.cgi

to be sure of OS X capabilities, can you reduce to 10 instead of 60 and add
echo "shutdown called\n";
inside function abort() ?
then you should see 'shutdown called' when not aborting connection? same should happen when calling script directly from shell (and let it go to the end)

Not sure what you are seeing. When I start the script:
#!/usr/bin/env php
<?php
error_log('script started');
$done=false;

function mewasaborted()
{
    global $done;
    if(!$done) error_log("aborted");
    error_log("shutdown");
}

register_shutdown_function('mewasaborted');

echo "Content-type: text/html\n\n";
echo "\n\n";
flush();

for($i = 1; $i <= 10; $i++)
{
    sleep(1);
    echo "$i
\n";
    error_log("$i");
}
$done = true;

error_log('script ended');
?>

from the command line, I get as output:
script started
Content-type: text/html

1

1
2

2
3

3
4

4
5

5
6

6
7

7
8

8
9

9
10

10
script ended
shutdown

If I hit ^C while running, I do not get any message, because no one reads stderr of that process anymore.
Same with mod_cgi  in Apache. On timeout or connection close, the cgi just aborts the process and does not read stderr any longer. So, one does not see anything in the logs.
Sorry, but I am unable to reproduce and unable to come up with an explanation of what you seem to notice.

I can now see why you can't reproduce : to log after connection abort from shutdown function, your php.ini (the one for cli version) has to be configured to log in a file and not print errors otherwise you won't see the "aborted" message...
From that, you will have to see the result in the log file, and you should notice the abort when in http/1.1 and not when using http/2

You mean I spent some hours reproducing an error with your script that you yourself never tested?

Of course I tested it, and it currently can show the problem using the first version of the script at the top of this issue as described, I just supposed too much about your default php configuration : you are using a "test/dev configuration" while I'm using a "prod configuration", the noticeable difference between the two php.ini is where error_log is writing out... that's why you could not reproduce on your OS X computer, but if you adjust your php.ini, you will start too see the problem as described
so just set in your php.ini :
display_errors = Off
log_errors = On
error_log=/var/log/apache2/php.log (or another place writeable for php)
at this point, you will notice that abort is printed in php.log only for http/1.1 case

Ok, leaning php a bit. My script now looks like this, independent of any global ini settings:
#!/usr/bin/env php
<?php
ini_set('date.timezone', 'MET');
ini_set('error_log', '/tmp/php.log');
error_log('script started');
$done=false;

function mewasaborted()
{
    global $done;
    if(!$done) error_log("aborted");
    error_log("shutdown");
}

register_shutdown_function('mewasaborted');

echo "Content-type: text/html\n\n";
echo "\n\n";
flush();

for($i = 1; $i <= 10; $i++)
{
    sleep(1);
    echo "$i
\n";
    error_log("$i");
}
$done = true;

error_log('script ended');
?>

I start the server and run:
m -f /tmp/php.log; /opt/apache-trunk/bin/curl -k https://test.example.org:12346/abort_test.cgi; cat /tmp/php.log

What I see on OS X:
1

2

^C
[11-Feb-2016 14:45:49 MET] script started
[11-Feb-2016 14:45:50 MET] 1
[11-Feb-2016 14:45:51 MET] 2

letting it run:
1

2

3

4

5

6

7

8

9

10

[11-Feb-2016 14:45:56 MET] script started
[11-Feb-2016 14:45:57 MET] 1
[11-Feb-2016 14:45:58 MET] 2
[11-Feb-2016 14:45:59 MET] 3
[11-Feb-2016 14:46:00 MET] 4
[11-Feb-2016 14:46:01 MET] 5
[11-Feb-2016 14:46:02 MET] 6
[11-Feb-2016 14:46:03 MET] 7
[11-Feb-2016 14:46:04 MET] 8
[11-Feb-2016 14:46:05 MET] 9
[11-Feb-2016 14:46:06 MET] 10
[11-Feb-2016 14:46:06 MET] script ended
[11-Feb-2016 14:46:06 MET] shutdown

On my Ubuntu machine, I see in 1 out of 10 cases where I interrupt the curl something like:
[11-Feb-2016 14:51:26 MET] script started
[11-Feb-2016 14:51:26 MET] aborted
[11-Feb-2016 14:51:26 MET] shutdown
[11-Feb-2016 14:51:27 MET] 1

So, this whole behaviour - so far having nothing to do with HTTP/2, is highly irregular. It seems that php sometimes invokes this function when the output goes, sometimes not. Given the fact, that mod_cgi never seems to send any signal, this seem to have nothing to do with HTTP/2 per se.

OK, so it seems that php will call shutdown if it sees end of stream sometimes before getting killed but not always, this seems to be highly irregular and may even depend from many local things (server speed, net speed, os, ...). btw I think that the way http2 is working will increase this irregularity (let's say it is by design), so I will note that relying on clean shutdown on abort while using cgi is very unreliable
Thanks you for your patience

You're welcome!


[BUG]Something Network error

Context
In order to help us troubleshoot issues with this project, you must provide the following details:

ElasticDump version: 6.16.0
Elasticsearch version : From 6.4.2(ES in localnetwork) To 7.1.1(AWS ES behind proxy)
Node.js version :v8.16.2

Describe the bug
[root@elasticdump ~]# elasticdump \
--input=http://172.XX.XXX.XXX:9200/index
--output=https://XXXXXX.es.amazonaws.com/index
Wed, 27 Nov 2019 04:17:18 GMT | starting dump
Wed, 27 Nov 2019 04:17:18 GMT | got 100 objects from source elasticsearch (offset: 0)
Wed, 27 Nov 2019 04:17:38 GMT | Error Emitted => tunneling socket could not be established, cause=connect ECONNREFUSED <IP_ADDRESS>:80
Wed, 27 Nov 2019 04:17:38 GMT | Error Emitted => tunneling socket could not be established, cause=connect ECONNREFUSED <IP_ADDRESS>:80
Wed, 27 Nov 2019 04:17:38 GMT | Total Writes: 0
Wed, 27 Nov 2019 04:17:38 GMT | dump ended with error (get phase) => Error: tunneling socket could not be established, cause=connect ECONNREFUSED <IP_ADDRESS>:80
Current behavior
I want to migrate index of ES in local to AWS ES. But It can not work well as shown above.
AWS ES needs proxy to resolve domain name but local ES does not need to.
So I set proxy and no proxy as shown bellow.
export http_proxy="http://ID:PW@IP:PORT"
export https_proxy="http://ID:PW@IP:PORT"
export no_proxy="<IP_ADDRESS>,localhost,IP for ES"
export NO_PROXY="$no_proxy"
I can get right responses from both ES local and AWS by curl command like this.
{
"name" : "node3",
"cluster_name" : "es-cluster",
"cluster_uuid" : "n4GrebxNRSeCT8gUxMjG_g",
"version" : {
"number" : "6.4.2",
"build_flavor" : "default",
"build_type" : "tar",
"build_hash" : "04711c2",
"build_date" : "2018-09-26T13:34:09.098244Z",
"build_snapshot" : false,
"lucene_version" : "7.4.0",
"minimum_wire_compatibility_version" : "5.6.0",
"minimum_index_compatibility_version" : "5.0.0"
},
"tagline" : "You Know, for Search"
}
I could migrate local to AWS the other day.

The error says it cannot connect to a socket on port 80. Not sure what the error is...

It might be that the domain name of AWS ES can not be resolved.
So please tell me how to set proxy for elasticdump.

This isn't an ES dump issue.
process.env.http_proxy = 'http://proxy.example.com:';


Remove unused import

Removed a single unused import.
There also exists an unused import at pyaction/template/{{action_slug}}/main.py but that file only contains the import so I let it be.

Thank you @sammyhori! The change that you made in the io.py file should be changed. I've imported Dict but used the built-in dict. Since PyAction supports Python 3.8 all the way up to 3.12, we better use Dict other than the built-in dict.
The pyaction/template/{{action_slug}}/main.py file, it's a template file that even pre-commit ignores its parent directory while checking. Consider it as a blueprint.

Thank you @sammyhori! The change that you made in the io.py file should be changed. I've imported Dict but used the built-in dict. Since PyAction supports Python 3.8 all the way up to 3.12, we better use Dict other than the built-in dict.

Fixed in 94bd49e

The pyaction/template/{{action_slug}}/main.py file, it's a template file that even pre-commit ignores its parent directory while checking. Consider it as a blueprint.

👍  All good, I thought it was probably something like that.

The pre-commit hook automatically changes Dict to dict, thus undoing my changes.

It's because of the from __future__ import annotations import line in the io.py file. The pyupgrade hook will consider it and change typing.Dict to dict to make the source file be supported on all Python environments. Removing it (the import line) wouldn't make a difference.
Things to do..

Remove the from __future__ import annotations line from the beginning of the io.py file.
Convert dict to Dict.
Squash all your commits into one if you can, please.

Hopefully, it's gonna be alright this time. :))

@sammyhori, It's because of the from __future__ import annotations import line in the io.py file. The pyupgrade hook will consider it and change typing.Dict to dict to make the source file be supported on all Python environments. Removing it (the import line) wouldn't make a difference.
Things to do..

Remove the from __future__ import annotations line from the beginning of the io.py file.
Convert dict to Dict.
Squash all your commits into one if you can, please.

Hopefully, it's gonna be alright this time. :))

All fixed this time, thanks for the advice :)

Thanks, mate! Hope to see more from you! :D


Polymer tests are failing or flaky after mutation-observer microtask change.

 Issue by floitschG
Originally opened as dart-lang/sdk#20075

Failing on Safari:
polymer/test/js_interop_test
Flaky on Safari and IE:
polymer/test/two_way_bind_test

 Comment by floitschG

js_interop_test is actually flaky.

 Comment by sigmundch

Removed Priority-Unassigned label.
Added Priority-Medium, PolymerMilestone-Next labels.

this test no longer exists


Feature/m1/client/create todos

Describe your changes

Adjust header

Make the logo bigger
Add animation
Change the nav component form button to link

Create todos page

Issue
#18
Note
You can't see the tasks in the screen since the seed data doesn't have tasks for today and coming week,
so if you want to see it, please add tasks using createTask api in postman.
URL
http://localhost:3000/
Screenshot (if you made the UI)
https://github.com/HAK2024/fairy-share/assets/132407687/4ee82672-6af2-4d0e-84a9-8ca06179173a
https://github.com/HAK2024/fairy-share/assets/132407687/03582729-f096-4bcf-ad4f-2b7632e58a08
If the user has no tasks

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings and no errors

LGTM!
You can merge this branch now but I think I found a bug in server side. When you make tomorrow's task, you don't get it as coming this week task. Tomorrow's task should include in coming this week. Am I correct?

I fixed the bug! It happens becuase I was using the same variable for weeksTasks as todayTasks and when I mutate currentDate in todayTasks, it affected weekTasks. So I fixed it by seperating the variables.
https://github.com/HAK2024/fairy-share/pull/71/commits/fd4e4702ff26f70fa78929ef75368bbdcef047f9


flutter_gallery_sksl_warmup__transition_perf 90th_percentile_frame_build_time_millis needs a baseline...

@liyuqian I think this may be one of yours?

Same with all of many other flutter_gallery_sksl_warmup_ios32__transition_perf:

Just baselined. Thanks for the reminder!


No CSS Styles

When using webpack, no css styles are applied to the calendar

+1

Styles are kind of up to you there's reference styles in the bootstrap_theme.less. I may include some more generic styling in the future but it's currently not planned.


[Feature request] Allow adding PR reviewers via publish:github:pull-request action

We'd like to be able to add reviewers to PRs created by Backstage via the publish:github:pull-request action by setting them as parameters of the action.
Feature Suggestion
We'd like to have two extra parameters on the publish:github:pull-request action, reviewers and teamReviewers, which would allow us to add reviewers much more easily to Backstage-created PRs just by adding them to the action definition:
    - id: pullrequest
      name: Create pull request
      action: publish:github:pull-request
      input:
        repoUrl: ${{ parameters.repoUrl }}
        branchName: ${{ parameters.branchName }}
        title: New Pull Request
        description: A newly created PR on GitHub
        reviewers:
          - user1
        teamReviewers:
          - team1

Possible Implementation
Currently, we have put together this code to perform this task:
const octokitIntegration = await new OctokitProvider(scmIntegrations).getOctokit(REPOSITORY_URL);
const prUrlSegments = pullRequestUrl.split("/");
await octokitIntegration.client.rest.pulls.requestReviewers({
    owner: octokitIntegration.owner,
    repo: octokitIntegration.repo,
    pull_number: parseInt(prUrlSegments[prUrlSegments.length-1]),
    reviewers: [...],
    team_reviewers: [...]
});

where pullRequestUrl is the remoteUrl of the PR as returned by the publish:github:pull-request action.
Context
We are using the publish:github:pull-request action in our scaffolding process to propagate the creation of the new project to other repositories (mainly IaC-related) by sending PRs.
We need to add a team as a reviewer of these PRs so that those people get notified when Backstage sends a PR.

Can this be reopened? The feature request is still valid.

The feature request is still relevant.

Do you want to contribute this @Fs00?

My colleague @fabuloso and I are working on it, we'll open a PR next week.


Generated code contains "unchecked" warnings.

We enable -Werror and -Xlint:unchecked on all our java Android java projects.
When upgrading to 1.0.0, our builds started failing in compile, due to the generated code for ResponseFieldMarshaller using this construct where similarPrograms is of type List<ProgramPageQuery.SimilarProgram>
writer.writeList($responseFields[1], similarPrograms, new ResponseWriter.ListWriter() {
    @Override
    public void write(List items, ResponseWriter.ListItemWriter listItemWriter) {
        for (Object item : items) {
            listItemWriter.writeObject(((SimilarProgram) item).marshaller());
        }
    }
});

This causes the compiler to emit a warning about an unchecked method invocation:

[unchecked] unchecked method invocation: method writeList in interface ResponseWriter is applied to given types
required: ResponseField,List,ListWriter
found: ResponseField,List,
where T is a type-variable:
T extends Object declared in method writeList(ResponseField,List,ListWriter)

and our build fails due to -Werror.
This is caused by ListWriter now being generic, and expecting a specific element type.
Expected behaviour here is that the generated code should instead be
writer.writeList($responseFields[1], similarPrograms, new ResponseWriter.ListWriter<ProgramPageQuery.SimilarProgram>() {
    @Override
    public void write(List<ProgramPageQuery.SimilarProgram> items, ResponseWriter.ListItemWriter listItemWriter) {
        for (ProgramPageQuery.SimilarProgram item : items) {
            listItemWriter.writeObject(item.marshaller());
        }
    }
});

If this isn't possible due to type erasure, union types or something else outside of your control, the emitted code should instead suppress the "unchecked" warning.

Have you tried this configuration in your build gradle:
apollo {
    ...
    suppressRawTypesWarning = "true"
}

@sav007 Yes, we turned on suppressRawTypesWarning, but it did not suppress the unchecked warnings as far as we can tell.

@sav007 I just wanted to confirm that setting suppressRawTypesWarning to true does not silence the [unchecked] warnings. Turning suppressRawTypesWarning off generates a bunch of [rawtypes] warnings, in addition to the "unchecked method invocation" / "unchecked conversion" warnings that I've mentioned above.

@sav007 Thank you!


publish stats for a handler

This could be that each handler just writes its data to its own endpoint, or we make some shared system? I am open to options.
Mainly we want to make it so that we can get the timings for signalfx's post. If easily we'd want to publish:

ssl connection time
time to first byte
total request time
metrics in flush

We currently publish the total request time and sent it to the same handler.

We should also report the number of metrics sent and the number of metrics dropped

After with cbc3107 we publish send & dropped from all handlers.


[do not merge] [help needed] New Flow introduction tutorial

Pull request information

Status: In development
Kind of changes: New tutorial
Related PR or issue: Following meeting discussion

Description

This is a new tutorial 0 that is to be read before the others. It explains the core idea of Flow (e.g. the difference between scenario and environment, but also a lot of other things) for newcomers, and doesn't contain any code. It can also be used for people that have been using Flow for some time but wish to dwelve more into it, as it explains how our codebase is organized in more details.
I have written section 1 and part of section 2, and I could use some help to write section 2 as I haven't touched to every part of the code (mainly benchmarks, controllers, renderer, multiagent_envs). LMK if you want to help, you can just give me a draft and I will write something clean.

This is a great idea!
A few comments. I'm not sure I understand the goals of Section 1. Perhaps filling out the section on
    Section 1 does XXX.
    Section 2 does YYY.
    Section 3 does ZZZ.

would help.
What is the purpose of this tutorial vs http://flow.readthedocs.io/?
Is this targeted towards users of Flow or developers/contributors of Flow? If the latter, perhaps it should not be the very first tutorial though.

@cathywu Section 1 is for newcomers, to explain the very basics of Flow. Sections 2 and 3 are more to be read after all the other tutorials, when one wants to start creating its own code and contributing. Maybe I'll split it in two tutorials. Maybe it's also not the best idea to be so specific in section 2, I'm open to suggestions!
I'll fill out the sections summaries once all the sections are written, it will be easier.
Concerning readthedocs, personally I've never used it except for the installation instructions. I think people mostly use the Github tutorials? Or at least they are complementary imo. As for the code documentation on readthedocs, it doesn't look up to date at all (core.kernel, envs, visualizers... are all empty).
Let me know what you think.

Ah got it, sounds great. Splitting also makes sense. It may be good to
discuss as a group what is the relative role of the tutorials vs the
readthedocs. I was just referring to our readthedocs recently for the
multiagent examples.
Best wishes,
Cathy


Feature request: Week numbers

First - great widget! It is perfect for my needs.
It would be even better if it could optionally show the current week numbers. It could maybe fit to the right of the current day and date. Note that there are a couple of different numbering schemes (https://en.m.wikipedia.org/wiki/Week_number#Week_numbering)

I should also have added a thank you(!)  for making this widget.

Merging this with more general #216


minor typo

@charleswj : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@charleswj Thank you for the typo fix!


fix: blueprintConfig dropped when restoring a snapshot

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Fixes a bug causing the Blueprint Configuration on studios and showStyles to be lost when restoring a system snapshot.

What is the current behavior? (You can also link to an open issue here)
When restoring a system snapshot, Blueprint Configuration on the studios and showStyles is dropped entirely instead of being restored from the snapshot. The cause is using migrateConfigToBlueprintConfigOnObject, which assumes that blueprintConfig is missing, when restoring snapshots.

What is the new behavior (if this is a feature change)?
Blueprint Configuration is restored correctly. migrateConfigToBlueprintConfigOnObject handles both cases - when blueprintConfig is present and when it's missing.

Other information:

Status

[ ] Code documentation for the relevant parts in the code have been added/updated by the PR author
[x] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

Im not keen on the change here, what happens if the doc somehow has both blueprintConfig and config defined? It will end up with a weird mash of the two, in a not very predictable manner.
I think it would be better to either ignore config if blueprintConfig is already defined, or if config is defined then replace blueprintConfig. I dont have strong feelings on which would be correct

I agree, and I think if both existed, it's better to ignore config because it seems more likely that it would be a leftover and blueprintConfig will have newer data.


New command: spo roledefinition add

Usage
m365 spo roledefinition add [options]
Description
Should add a new roledefinition to web
Options

Option
Description

-u, --webUrl <webUrl>
URL of the site to which role should be added

-n, --name <name>
role definition name

-d, --description <description>
role definition description

Additional Info
I was thinking that now since we added first command to list the role definitions from web we need other as well (get, add and delete) 🤔.
In this issue I would like to consider adding a add command to create new role definition.
We may use this POST request for this
/_api/Web/RoleDefinitions
specifying this JSON  in the body if the request
{
    "BasePermissions":{ // 👉TBH I am still not sure why do we need this node.. need to investigate this a bit ;)
    "High":"176",
    "Low":"138612801"
    },
    "Description":"${description}",
    "Name":"${name}"
}

the expected result should be a new role definition added

Great suggestion. BasePermissions describe the permissions associated with the role definition. It's a bit-flag composed out of the different permissions, so when creating a new definition, we'd need to find a way for users to specify the permissions the role should contain and then translate them to a flag.

as for the flags concept we may put this issue on hold and when those will be added together with !3270 we may got back to this one. What do you think?

Makes sense to avoid duplicate work. Let's do that

waiting for #3359 to be finished and merged and then I may start working on this

As the related issue has been successfully merged for some time, let's remove the on hold label here.


Permission to use Factor logo in presentation

I'm going to be doing a presentation about mal (https://github.com/kanaka/mal) at midwest.io. Factor is one of the implementations and I would like to use a Factor logo in the presentation as part of a list of mal implementations. I found this directory of Factor logos/icons: https://github.com/slavapestov/factor/tree/master/misc/icons. It might be useful if the license.txt file in that directory had some more detail on what permissions are granted in terms of using the logo. Can I get permission to use a Factor logo in my presentation (and the subsequent published video/materials from the conference)?

Yes that would be okay. We are updating the icon for the soon to be released 0.98 (maybe one or two weeks away).  If you want the newer icon let us know.
I'm not sure if the intention would be to have a different license for the icons than the code but we are pretty permissive with BSD license.

@mrjbq7 Thanks! The presentation is Nov 9. If you have something before then and you remember to update this bug before then (because I'm sure I'll forget to check), I would be happy to switch it to the new one.

Would you be interested in a pull request for some of the factor code in mal?

@mrjbq7 Certainly! I'm always interested in contributions that improve the implementations, especially if they make it more idiomatic. FYI, the original factor implementation was done by @jordanlewis


Make fishing nibble sound effect emanate from the player, client-side

When fishing from a distance, it's often difficult or impossible to tell by ear if something is ready to be reeled in.  It makes sense from a player perspective to have more feedback in this already half-baked game mechanic.

Because of some weirdness this would cause in multiplayer I don't think it would work super well to do it as stated. However, I can certainly increase the volume of the fish bobber. Honestly fishing as a whole could do with a pass anyway.

Done for 1.3.9


OLS4 search feedback

Searching for myocardial infarction in OLS3 vs OLS4 striking differences in the quality of results:
OLS3 - first 7 hits are a lexical match.
OLS4 - the lexical match to query isn't in the first two pages.
Also the query autocomplete is less intuitive:
OLS3  - first autocomplete for myocardial infa... has 2 precise and three minor variations on the input term
In OLS4 autocomplete is 'family history of myocardial...'
Suggests the search ranking is performing differently. As a user of the GUI I want precision vs. my search term before recall in all search cases - I'll scroll to get to less precise results.

Thanks Helen - we are aware of this (issue #99 though you provided much more detail than I did). Basically we haven't implemented the list of boost values for the different search fields properly, but should be fixed soon.

Thanks for the update.

This seems resolved now. Searching for "myocardial infarction":

Searching for "myocardial infa":


Unexpected behaviour of Keyboard.keysDown

In the KeysDown example: http://elm-lang.org/edit/examples/Reactive/KeysDown.elm
On a Mac, in Chrome, when I press Cmd+Right Arrow the signal continues reporting the key code for Right Arrow after lifting the key. (I first noticed this after accidentally holding down Cmd while moving right in Mario, but KeysDown is a smaller example that reproduces this.)
I'm not sure if this is a bug or some unintuitive consequence of the signals machinery, but in any case there should be a documented way of getting the expected result, i.e. when Right Arrow key is up it is reported as up.
I have reproduced this in 0.15 as well.

This seems to be a Mac-specific issue with keyboard event handling: http://bitspushedaround.com/on-a-few-things-you-may-not-know-about-the-hellish-command-key-and-javascript-events/
Maybe there's a way to implement a workaround in elm core, but I can't think of one right now.


[SPARK-37283][SQL][FOLLOWUP] Avoid trying to store a table which contains timestamp_ntz types in Hive compatible format

What changes were proposed in this pull request?

This is PR is to avoid trying to store a table which contains timestamp_ntz types in Hive compatible format.
In the current master, Spark tries to store such a table in Hive compatible format first, but it doesn't support timestamp_ntz.
As a result warning is logged like as follows though it's finally stored in Spark specific format.
CREATE TABLE tbl1(a TIMESTAMP_NTZ) USING Parquet
...
21/11/22 21:57:18 WARN HiveExternalCatalog: Could not persist `default`.`tbl1` in a Hive compatible way. Persisting it into Hive metastore in Spark SQL specific format.
org.apache.hadoop.hive.ql.metadata.HiveException: java.lang.IllegalArgumentException: Error: type expected at the position 0 of 'timestamp_ntz' but 'timestamp_ntz' is found.
        at org.apache.hadoop.hive.ql.metadata.Hive.createTable(Hive.java:869)
        at org.apache.hadoop.hive.ql.metadata.Hive.createTable(Hive.java:874)
        at org.apache.spark.sql.hive.client.Shim_v0_12.createTable(HiveShim.scala:614)
        at org.apache.spark.sql.hive.client.HiveClientImpl.$anonfun$createTable$1(HiveClientImpl.scala:554)
        at scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.java:23)
        at org.apache.spark.sql.hive.client.HiveClientImpl.$anonfun$withHiveState$1(HiveClientImpl.scala:304)
        at org.apache.spark.sql.hive.client.HiveClientImpl.liftedTree1$1(HiveClientImpl.scala:235)
        at org.apache.spark.sql.hive.client.HiveClientImpl.retryLocked(HiveClientImpl.scala:234)
        at org.apache.spark.sql.hive.client.HiveClientImpl.withHiveState(HiveClientImpl.scala:284)
        at org.apache.spark.sql.hive.client.HiveClientImpl.createTable(HiveClientImpl.scala:552)
        at org.apache.spark.sql.hive.HiveExternalCatalog.saveTableIntoHive(HiveExternalCatalog.scala:506)
        at org.apache.spark.sql.hive.HiveExternalCatalog.createDataSourceTable(HiveExternalCatalog.scala:404)
        at org.apache.spark.sql.hive.HiveExternalCatalog.$anonfun$createTable$1(HiveExternalCatalog.scala:273)
...

Why are the changes needed?

To fix the confusing behavior.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Modified the test added in #34551

cc @beliefer

Test build #145512 has started for PR 34683 at commit 89f8857.

cc: @cloud-fan too.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49984/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49984/

Test build #145512 has finished for PR 34683 at commit 89f8857.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145512/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/49984/

cc @beliefer

Thank you for ping me.

I think the decision is: table schema with NTZ is not hive-compatible. It's the safest decision as it won't have any regressions.
This PR LGTM

@cloud-fan Thank you for re-visiting this PR!
I'll run GA again just in case.

thanks, merging to master!


Share timeout code between package installation and compilation

Bonus: clarify, mostly via renaming, the usages of the various Params

Test failure is new after my (local) rebase, so I'm guessing I merged something incorrectly.


Output machine ids in juju_machine

Context
Currently in order to get machine ids for application placement we have to do something like the following:
resource "juju_machine" "some_machine" {
  ...
}

resource "juju_application" "some_app" {
  placement = join(",", [split(":", juju_machine.some_machine.id)[1]])
}

Issue
For most replicated applications, each unit will be assigned a dedicated machine with identical configuration.
This makes it good candidate for Terraform's count meta-arg.
However we cannot directly reference machine id in a count as it's (known after apply), having to do something like the following:
join(",", [for machine in juju_machine.some_machine : split(":", machine.id)[1]])

Request
It would be great to shift the logic of looping over the machines from the module user, to the module itself, to reduce complexity
Potential Solution?
Please correct my understanding if I'm wrong, but I believe it should be possible to use outputs to get the Machine IDs after deploy and force a depends_on for juju_application resources that specify it.
I would imagine an output might look something similar to:
resource "juju_application" "some_app" {
  depends_on = [juju_machine.some_machine]

  placement = juju_machine.some_machine.outputs.machine_ids
}

Hi @marcoppenheimer, from what I know, you should already be able to do the following:
resource "juju_application" "some_app" {
  # depends_on = [juju_machine.some_machine]

  placement = join(",", juju_machine.some_machine[*].machine_id)
}

depends_on is not needed since you rely on computed attribute, terraform should have the right deployment graph


Cross compile CI

Run the cross-compile script as part of CI to prevent mistakes like this from happening again.
@nats-io/core

Do we know if this insulates us properly from breaking windows builds?

@derekcollison I verified by making a change that broke Windows build and Travis failed: https://travis-ci.org/nats-io/gnatsd/jobs/240406869


find: dSYM_directory: No such file or directory

Description
My entire project was working fine until I updated Firebase to 10.27.0 & MacOS to 14.5 (I'm using Xcode 15.4).
Now when trying to build the app now I get the following
Unable to read Google Service plist at path
find: dSYM_directory: No such file or directory

I can still Build and Run for iOS and create an Archive for AppStoreConnect, but the Build fails for MacOS and VisionOS.
Reproducing the issue
I have gone through the following steps.
Step 1)

Install Firebase via SPM
Download google-service.plist from Firebase.
Copy to project folder, check target
Add to Target > Build Settings > Copy Bundle Resources

Errors:
Unable to read Google Service plist at path /Users/approot/Shared/GoogleService-Info.plist

find: dSYM_directory: No such file or directory

Sandbox: cp(38870) deny(1) file-read-data /Users/approot/Shared/GoogleService-Info.plist

Step 2) Ensure File Permissions:
Verify that the GoogleService-Info.plist file has the correct permissions.
chmod 644 /Users/approot/Shared/GoogleService-Info.plist

Step 3) Create a new "Run Script Phase" to copy the file
bash
Copy code
SRC="${SRCROOT}/Shared/GoogleService-Info.plist"
DEST="${BUILT_PRODUCTS_DIR}/${EXECUTABLE_NAME}.app/GoogleService-Info.plist"

echo "Source path: $SRC"
echo "Destination path: $DEST"

if [ -f "$SRC" ]; then
    echo "Copying $SRC to $DEST"
    cp "$SRC" "$DEST"
    if [ $? -ne 0 ]; then
        echo "Error copying file."
        exit 1
    fi
else
    echo "File not found: $SRC"
    exit 1
fi

Step 4) Check File System Access:
Ensure that Xcode has full disk access. This can sometimes resolve sandbox-related issues.
Go to System Preferences > Security & Privacy > Privacy tab.
Select Full Disk Access and make sure Xcode is added to the list of applications with full disk access.

Step 5) Check Derived Data:
Delete the derived data folder.
Example Run Script Phase with Debugging
Here’s an enhanced version of the script to include more detailed debugging information:
bash
Copy code
#!/bin/bash

SRC="${SRCROOT}/Shared/GoogleService-Info.plist"
DEST="${BUILT_PRODUCTS_DIR}/${EXECUTABLE_NAME}.app/GoogleService-Info.plist"

echo "Source path: $SRC"
echo "Destination path: $DEST"

if [ -f "$SRC" ]; then
    echo "File exists. Copying $SRC to $DEST"
    cp "$SRC" "$DEST"
    if [ $? -ne 0 ]; then
        echo "Error copying file."
        exit 1
    fi
else
    echo "File not found: $SRC"
    exit 1
fi

Step 6) Restart Xcode

Re-indexed Project
Closed Xcode
deleted derived data
reopened the project to allow
Reinstalled packaages
Reinstall Xcode
Restart MacOS
Jump out a window

I've repeated these steps 250+ times, the file is in the project, it has the target checked, I've added to bundle resources, it literally worked before updating macOS. I even have the previous iOS build uploaded to AppStoreConnect, we just needed latest version and upload MacOS app.
Firebase SDK Version
10.27.0
Xcode Version
15.4
Installation Method
Swift Package Manager
Firebase Product(s)
Analytics, App Check, App Distribution, Authentication, Crashlytics, Database, DynamicLinks, Firestore, Functions, In-App Messaging, Installations, ML Model Downloader, Messaging, Performance, Remote Config, Storage, VertexAI, All, Infrastructure
Targeted Platforms
macCatalyst, visionOS
Relevant Log Output
Sandbox: cp(38870) deny(1) file-read-data /Users/approot/Shared/GoogleService-Info.plist

If using Swift Package Manager, the project's Package.resolved

Expand Package.resolved snippet

Replace this line with the contents of your Package.resolved.

If using CocoaPods, the project's Podfile.lock

Expand Podfile.lock snippet

Replace this line with the contents of your Podfile.lock!

What is the reason for copying the GoogleService-Info.plist file?  The general practice is to add it to the relevant Xcode targets statically.


Implement Helm deployment for OTel collector

Summary

Create a script for Helm deployment.
Call the script from a Github workflow.

Implemented with #13.


compile error

CMake Error in velox/common/encode/CMakeLists.txt:
Imported target "Folly::folly" includes non-existent path
"/Library/Developer/CommandLineTools/SDKs/MacOSX12.sdk/usr/include"

in its INTERFACE_INCLUDE_DIRECTORIES.  Possible reasons include:

The path was deleted, renamed, or moved to another location.

An install or uninstall procedure did not complete successfully.

The installation package was faulty and references files it does not
provide.

the directory of /Library/Developer/CommandLineTools/SDKs/MacOSX12.sdk does not exist, there is a directory of /Library/Developer/CommandLineTools/SDKs/MacOSX11.1.sdk.
why Folly::folly includes /Library/Developer/CommandLineTools/SDKs/MacOSX12.sdk/usr/include, or how upgrade the MacOSX11.1.sdk to the new version of MacOSX12.sdk

CC: @kgpai @raulcd @Yuhta @pedroerp

Hi @natashaxin , Can you post the full cmake output so we can better debug ? Thanks.

Duplicate of #2924


Blank page opens when trying to run Ketcher in IE11

Affected Version: Remote, Standalone
Browser: IE 11
Steps to reproduce:

Launch Ketcher in IE11

Expected result: Ketcher is launched successfully.
Actual result: A blank page opens instead of a program.

Issue closed as fixed.


fix: use absolute socket path in configureTimezones. this fixes #169

The change fixes #169. Tested locally. To reproduce the original issue just try to spin up a local mysql service using nixpkgs-old.legacyPackages.mysql57 package while mysqld.default-time-zone or importTimeZones is set.

If you prefer merging the test cases into an existing one, just let me know.

I don’t mind adding a newer instance of mysql, if that’s required. I think for testing mysql80 it makes sense to add a newer mysql m3, but we could merge the m4 into m2 since m2 is doesn’t test anything apart from starting mariadb on a different port.

I've improved the tzdata tests cases to actually check whether the  time_zone_name table is correctly populated or not. By default this table does not contain any row.

If you prefer merging the test cases into an existing one, just let me know.

I don’t mind adding a newer instance of mysql, if that’s required. I think for testing mysql80 it makes sense to add a newer mysql m3, but we could merge the m4 into m2 since m2 is doesn’t test anything apart from starting mariadb on a different port.

Sure! I'll merge m4 and m2 into one to simplify the code a bit.

Thanks! Everything looks good at a glance. Let me do a quick test on my machine and we are good to merge.
Meanwhile, could you do one last thing. Can you squash your commits into 2 commits? one fixing the importTimezones and another one for adding tests? We are using commit history to generate the changelogs in the future release, so this will be very helpful.

Thanks! Everything looks good at a glance. Let me do a quick test on my machine and then we are good to merge.
Meanwhile, could you do one last thing. Can you squash your commits into 2 commits? one fixing the importTimezones and another one for adding tests? We are using commit history to generate the changelogs in the future releases, so this will be very helpful.

I just wanted to ask! 👍

Awesome! Thanks for the contribution. Merging it


Figure 9 image is too tiny.

Figure 9 image is too tiny. Plz upload image with good quality.

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

ID: 219cb131-c2ad-51b2-0bb8-678c1b1b308b
Version Independent ID: 7726d8df-f346-37cc-306e-8a0c71ca44c3
Content: Hyper-V Network Virtualization Technical Details in Windows Server 2016
Content Source: WindowsServerDocs/networking/sdn/technologies/hyper-v-network-virtualization/hyperv-network-virtualization-technical-details-windows-server.md
Service: virtual-network
GitHub Login: @jamesmci
Microsoft Alias: jamesmci

@Keeper306 -- I've found the image you mentioned and, you are correct, the image is hard to read.  I am redoing the image and it should be added soon.  👍


Added support for backups/restores using S3.

This is related to Issue #29 and takes into account the implementation details mentioned there.
Usage
Export
curl -XPOST 'localhost:9200/test_index/_export/s3?target=/tmp/test_index.tar.gz&s3path=/path/in/bucket/test_index.tar.gz&s3bucket=bucketname&accesskey=XXXX&secretkey=XXXX'

Import
curl -XPOST 'localhost:9200/test_index/_import/s3?target=/tmp/test_index.tar.gz&s3path=/path/in/bucket/test_index.tar.gz&s3bucket=bucketname&accesskey=XXXX&secretkey=XXXX'

Details

If the aws credentials are not provided in the request then the DefaultAWSCredentialsProviderChain is used. (this effectively means that the credentials are automatically delivered through the Amazon EC2 metadata service when knapsack is running on an EC2 instance)

On a successful upload the locally exported archive gets deleted.

plugin.xml adds all the required dependencies but this could be refactored to have multiple assemblies

would love to see this merged =)

@jsh2134 have you tried https://github.com/elastic/elasticsearch-cloud-aws with S3 for snapshot/restore? The official solution looks much better and should be recommended over this one here - as far as I can tell, because I do not use S3.


[REF-3192] rx.foreach over computed var list fails

Describe the bug
When I try to render list that is produced by computed state var, I encounter Server Error "TypeError: Cannot read properties of undefined (reading 'map')"
To Reproduce

Define computed var that returns a list in the State.
Use that state var in rx.foreach

import reflex as rx

class ExampleState(rx.State):

    @rx.var
    def items(self) -> list[str]:
        return list(str(i) for i in range(10))

def index():
    return rx.vstack(
        rx.foreach(
            ExampleState.items,
            lambda item: rx.text(item)
        )
    )

app = rx.App()
app.add_page(index)

Expected behavior
the rx.foreach will render 10 text elements
Screenshots

Specifics (please complete the following information):

Python Version: 3.12.2
Reflex Version: 0.5.5

Additional context
Very similar to the old issue https://github.com/reflex-dev/reflex/issues/1886 and the same hack works:
import reflex as rx

class ExampleState(rx.State):

    @rx.var
    def items(self) -> list[str]:
        return list(str(i) for i in range(10))

def index():
    return rx.vstack(
        rx.cond(
            ExampleState.items,
            rx.foreach(
                ExampleState.items,
                lambda item: rx.text(item)
            )
        )
    )

app = rx.App()
app.add_page(index)

Also, everything works when the state var isn't computed (however it forces us to have some event handler that constatly updates the var):
import reflex as rx

class ExampleState(rx.State):
    items: list[str] = ['a', 'b', 'c']

def index():
    return rx.vstack(
        rx.foreach(
            ExampleState.items,
            lambda item: rx.text(item)
        )
    )

app = rx.App()
app.add_page(index)

REF-3192

Tried to reproduce it on latest main, but your code snippet is working for me.

@SlonSky I believe this is because somewhere your ExampleState.items is being set to None rather than an empty list. Then in Javascript, it's unable to call the map function leading to this error.


Implement more joins and caps

Round join and cap (possible to render in a single draw call per line, instead of two), bevel joins, etc. See https://wwwtyro.net/2019/11/18/instanced-lines.html

Round cap can possibly be implemented without any additional vertices using this suggestion by TimD on the GP discord:

btw, do you render extra cap meshes like in the article you linked? in my impl i just extend the billboard a bit in both directions and use the pixel shader to discard/alpha blend pixels to create a rounded end point

Miter joins were disabled when polyline was updated to 0.6. This will be done as needed.

Has there been any work on this? Does polyline currently support any caps? Because any 90 degree angles currently look pretty ugly :(


scores populate, but not by highest score

adding to route to populate all scores upon quiz completion
scores are available but are not in order

all good now


New Feature: Define order of columns based on optional parameter "var_order"

Description
The dataset used to generate the Event Tree may have columns out of order. Rather than manually swapping them around in a spreadsheet editor, the user could simply add a list ordering of the columns.
Branch
To work on this, please commit to the "event_tree_feature_var_order" branch.
Once complete, please let @g-walley know with a pull request, and he will merge it into the dev branch.
Modification required
In the EventTree class in event.py, add an optional parameter called "var_order".
Parameter will take the form of a list of column names: e.g: ["col1", "col3", "col2"]
Look at this post on stack overflow: https://stackoverflow.com/questions/13148429/how-to-change-the-order-of-dataframe-columns
The response from freddygv looks like a good solution.

@Kaasiak Thanks very much for fixing this issue. I will close this now.


Attempt to fix broken logo images

Currently the logos at the top of the published notebooks are not appearing.
I think the reason is that jupyter-book / myst-nb are not handling the relative path corrections for the html img tags because they are wrapped in an html table.
Here I try to replace the html table with a markdown table instead to see if that fixes the problem.
The way jupyter-book handles html image tags is a bit confusing. We'll see in the PR preview if this helps.

So it turns out that removing the <img> tags from the table fixed the problem of the missing images -- but it doesn't really look as nice this way.
It would be better to have more fine-grained control over image placement, but I don't know how to do that in a way that plays well with Jupyter Book.


network seed derivation using sha256 is vulnerable to length extension attack

I'm not a crypto expert (and haven't reviewed the code too deeply) but it seems that the network seed derivation function uses a raw sha256 hash on concatenated strings so it's vulnerable to a length extension attack (https://en.wikipedia.org/wiki/Length_extension_attack) allowing an attacker which gains access to the network seed to create valid network seeds with higher revisions.
For example let's say the seed for revision 1 was compromised (attacker knows hash(master||constant||1)), they can now produce hash(master||constant||1234), so even if the legitimate owner of the key wants to revoke it by incrementing the version the attacker can create a new seed with the new revision. I think the legitimate owner will require a new master seed in this case.
For key derivation it's more appropriate to use KDF (or HKDF, HMAC-SHA256) or alternatively a hashing algorithm that is not vulnerable (SHA3) or (less ideally) concatenate in a more thoughtful order.

Thanks for responding and fixing so quickly!
Out of curiosity (not really a crypto expert) why go with sha256d (which has its own, if minor, issues) rather than hmac-sha256 like bip32 (well, that uses hmac-sha512, but similar)

@asssaf Agreed that hmac-sha256 from the get-go would have possibly been a better choice.  Not sure why it wasn't used, myself -- that part of the wallet design predates me.
In the meantime, sha256d did the trick re: the LE attack issue here, while also being a very simple fix.  Chur!


Add support for Colour and MasteringMetadata subelements.

This is in support of BMO# 1539686.
Still need to add tests and think about how the x/y coords are exposed via nestegg_video_params, but the base functionality is now implemented to allow for early experimentation.

Note to self: for primary_[rgb]chromacity[xy], white_point_chromaticity_[xy], and luminance_{max,min} document that NaN means "not present".  The other new fields all have defaults supplied by the Matroska spec.

r? @SingingTree please


apollo subscription

Have you played with graphql-subscriptions?
https://dev-blog.apollodata.com/graphql-subscriptions-in-apollo-client-9a2457f015fb#.pq8cy0x93

I have not played with it yet, but it looks interesting. I am wondering if it would be possible to do this automatically by capturing the events from feathers hooks and sending them over a websocket. The default feathers has a very similar behavior with socket.io, but you do not have to configure it at all
https://docs.feathersjs.com/real-time/socket-io.html


RE3 Cutscene don't have any dialogue

Version
3.0.4 (f015691)
Seed
R730-R2LC-XXHJ0HCH28F001BNYJ803246CC00RZZZ80Z
Scenario
RE 1 (Chris)
Player Character
Leon
What happened?
Me again
I tried to see if there was any crash with the other Carlos' event
When Carlos (William) appear he had no dialog, so the scene stop there, like waiting to finish his dialog but sincehe had none... nothing happen there
I can skip it anyway, so is not a softlock
Debug save there
Debug Save.zip
btw, i hadn't find any other crashes related with the Granade Launcher in this scene

I can confirm this on my end. Looking at the log I see:

210:255 (0x3044) [CARLOS OLIVEIRA 2] becomes [NIKOLAI ZINOVIEV (william)]
  cutscene #0 contains william, leon
211:255 (0x74A8) [CARLOS OLIVEIRA 2] becomes [NIKOLAI ZINOVIEV (william)]
211:255 (0x74EC) [CARLOS OLIVEIRA 2] becomes [NIKOLAI ZINOVIEV (william)]
    DATA_A/VOICE/M211A010.WAV [jill.re3] becomes pl0/voice/stage3/v034.sap [leon]
    DATA_A/VOICE/M211A020.WAV [jill.re3] becomes pl0/voice/stage1/v014.sap [leon] (0.70-1.65)
    DATA_A/VOICE/M211A030.WAV [carlos] becomes UCbirkin7.ogg [william] (0.00-2.56)
    DATA_A/VOICE/M211A040.WAV [carlos] becomes ev7306.ogg [william] (0.00-2.17)
    DATA_A/VOICE/M211A050.WAV [carlos] becomes ORCBirkin11.ogg [william] (0.00-3.18)

It seems like the game can't find the audio file to play.

I think I have fixed this now, hopefully also a duplicate of #278


Change username button layout #783

Description
What's new?

Implement #783
Move appbar content to a separated component
Remove unused component SurveyPrompt

What kind of change does this PR introduce?

[x] Feature


Cannot use CUDNN backend! Please help!

When i try to use cudnn i get this error on ubuntu 16.04, nvidia driver 378.13, Cuda 8.0 (full newest release), Cudnn 6 (same problem with 5.1) using torch-cl and installed all cutorch cunn cudnn with laurocks. Torch works with my gpu i just cant get CUDNN to work. Please help! thanks! (and im using 1070 gpu)
/home/josh/torch-cl/install/share/lua/5.1/cudnn/init.lua:98: attempt to call field 'hasFastHalfInstructions' (a nil value)
stack traceback:
/home/josh/torch-cl/install/share/lua/5.1/cudnn/init.lua:98: in function 'fasterHalfMathTypeForCurrentDevice'
/home/josh/torch-cl/install/share/lua/5.1/cudnn/init.lua:112: in function 'configureMath'
/home/josh/torch-cl/install/share/lua/5.1/cudnn/init.lua:131: in main chunk
[C]: in function 'require'
stdin:1: in main chunk
[C]: at 0x00405e90

Update cutorch:
https://github.com/soumith/cudnn.torch/issues/279

Or try this solution: https://github.com/soumith/cudnn.torch/issues/383
It seems that you have to specify the right branch of cudnn.torch now?


Docs doesn't show props descriptions

Can you link to the doc that contains these props?

@hramos I think this one is from VirtualizedList https://facebook.github.io/react-native/docs/virtualizedlist.html

^ yes, VirtualizedList


Fixes #268 In Python 2.7 True and False show up in the Autos window. …

…This fix excludes them from showing up.

Hi @karthiknadig, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Karthik Nadig). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

:+1:


NetworkInterface Detachment Failure

Describe the bug
The NetworkInterface detachment process is not functioning as expected. After attempting to detach a NetworkInterface, it is successfully removed from the store but remains present in the DomainXML.
To Reproduce

Create a machine with nic.
Detach the nic.
Verify domain machine DomainXML. nic is not getting removed.

Expected behavior
The NetworkInterface should be successfully unplugged from both the store and the DomainXML.
Screenshots
2024-02-05T06:44:55Z	ERROR	machine-reconciler	failed to reconcile machine	{"machineID": "0c058844-7cb9-42a6-b988-d6573f870551", "error": "[network interfaces] attach / detach error(s): [[network interface primary] error detaching: internal error: unable to execute QEMU command 'device_del': Device ua-networkinterface-primary is already in the process of unplug]"}
github.com/ironcore-dev/libvirt-provider/pkg/controllers.(*MachineReconciler).processNextWorkItem
	/home/sosahu/my-workspace/libvirt-provider/UT/store/libvirt-provider/pkg/controllers/machine_controller.go:300
github.com/ironcore-dev/libvirt-provider/pkg/controllers.(*MachineReconciler).Start.func7
	/home/sosahu/my-workspace/libvirt-provider/UT/store/libvirt-provider/pkg/controllers/machine_controller.go:200

@Rohit-0505 we will have probably wrong logic for manage status of nics. We have similar issue for volume: https://github.com/ironcore-dev/libvirt-provider/pull/144
You can inspire by my PR


refactor: use protobuf extension schema defined in proto-graphql/proto-graphql

Pull Request Test Coverage Report for Build 500889661

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.558%

Totals

Change from base Build 497767278:
0.0%

Covered Lines:
734

Relevant Lines:
755

💛  - Coveralls

Pull Request Test Coverage Report for Build 500889661

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.558%

Totals

Change from base Build 497767278:
0.0%

Covered Lines:
734

Relevant Lines:
755

💛  - Coveralls


[BUG] - Invalid expected columns in load unit testing for timestamp

Given the load schema
# metadata/load/starbake/_config.sl.yml
---
version: 1
load:
  name: "starbake"
  metadata:
    directory: "{{incoming_path}}/starbake"

# metadata/load/starbake/orders.sl.yml
---
version: 1
table:
  name: "orders"
  pattern: "order.*_(?<mode>F|D).json"
  attributes:
  - name: "customer_id"
    type: "long"
    array: false
  - name: "order_id"
    type: "long"
    array: false
  - name: "status"
    type: "string"
    array: false
  - name: "timestamp"
    type: "iso_date_time"
    array: false
  metadata:
    format: "JSON"
    array: true
    withHeader: true
    writeStrategy:
      types:
        OVERWRITE: 'group("mode") == "F"'
        UPSERT_BY_KEY: 'group("mode") == "D"'
      key:
      - "order_id"

and the given input data to load:
// metadata/tests/load/starbake/orders/init/orders_F.json and _expected.json
[
	{
		"order_id" : 1,
		"customer_id" : 6,
		"timestamp" : "2024-02-05T21:19:15.454Z",
		"status" : "Cancelled"
	},
	{
		"order_id" : 2,
		"customer_id" : 23,
		"timestamp" : "2024-01-02T10:44:37.590Z",
		"status" : "Pending"
	},
	{
		"order_id" : 3,
		"customer_id" : 20,
		"timestamp" : "2024-02-10T22:10:30.685Z",
		"status" : "Delivered"
	}
]

We have as test output:
Missing Records

customer_id,order_id,status,timestamp
20,3,Delivered,2024-02-10 23:10:30.685
23,2,Pending,2024-01-02 11:44:37.59
6,1,Cancelled,2024-02-05 22:19:15.454

Unexpected Records

order_id,customer_id,timestamp,status
1,6,2024-02-05T21:19:15.454Z,Cancelled
2,23,2024-01-02T10:44:37.590Z,Pending
3,20,2024-02-10T22:10:30.685Z,Delivered

We can see that input schema is:

and expected schema is :

I think, comparing string with timestamp generate this output result and might be related to how expected data is loaded into duckdb.

Done
Please update first your metadata/types/default.sl.yml file fro src/main/resources/types/default.sl.yml


No completion builder present after a '('

Input : key.
let f: (val

expected:
let f: (val:

actual:
let f: (ValidityState:

Divides #6079.

If I understand the issue correctly then it was already fixed, cc @jakebailey


Add code for day 11: river orientation

This PR contains code to create a draft figure showing river orientation distributions for day 11: circular theme. Thanks for your initial feedback, @ceenell! I'll iterate on this draft a bit and commit any changes here.

Looking good! Going to merge this and will send you one back in a little bit


Fishy error while installing s3 on bancadati

Traceback (most recent call last):
  File "/opt/code/github/threefoldtech/0-templates/templates/s3/s3.py", line 639, in _install_namespace
    raise NamespaceDeployError(task.eco.message, node)
s3.NamespaceDeployError: (NamespaceDeployError(...), '3 services found')

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "src/gevent/greenlet.py", line 716, in gevent._greenlet.Greenlet.run
  File "/opt/code/github/threefoldtech/0-templates/templates/s3/s3.py", line 643, in _install_namespace
    raise NamespaceDeployError(str(err), node)
s3.NamespaceDeployError: (NamespaceDeployError(...), "(NamespaceDeployError(...), '3 services found')")
2018-11-27T10:25:10Z <Greenlet "Greenlet-0" at 0x7f29b2642648: <bound method S3._install_namespace of <s3.S3 object at 0x7f29d6629da0>>(node={'farmer_id': 'bancadati.threefold.grid.farm01', ', name='03856c5f', disk_type='hdd', size=2, password='txk0tfin7d7ybcww4syu34azd1cst4ie')> failed with NamespaceDeployError

It seems that there were multiple node_port_manager services on most of bancadati nodes. I ran a script to clean it up and @zaibon is looking into the cause of this in zrobot.

fixed in https://github.com/threefoldtech/0-templates/commit/d326da18d1316c43637be18d94092d2e72a380c3


[BUG] glyphicons-halflings font not loadinf

Version information

Laravel: v5.8.15
Voyager: v1.2.3

Description
When I added a description to a field I get
GET http://voice.test/fonts/bootstrap/glyphicons-halflings-regular.woff2 net::ERR_ABORTED 404 (Not Found)
GET http://voice.test/fonts/bootstrap/glyphicons-halflings-regular.woff net::ERR_ABORTED 404 (Not Found)
with the CSS loading from the 'http://voice.test/admin/assets?path=css%2Fapp.css' path
@font-face {     font-family: Glyphicons Halflings;     src: url(../fonts/bootstrap/glyphicons-halflings-regular.eot);     src: url(../fonts/bootstrap/glyphicons-halflings-regular.eot?#iefix) format("embedded-opentype"),         url(../fonts/bootstrap/glyphicons-halflings-regular.woff2) format("woff2"),         url(../fonts/bootstrap/glyphicons-halflings-regular.woff) format("woff"),         url(../fonts/bootstrap/glyphicons-halflings-regular.ttf) format("truetype"),         url(../fonts/bootstrap/glyphicons-halflings-regular.svg#glyphicons_halflingsregular) format("svg") }
Steps To Reproduce
Steps to reproduce the behavior:

Add a description to a field
got to that section and click 'Add New'
See description icon not loading

Expected behavior
The CSS path correctly linked to load the font files and description icon showing correctly

As a temporary solution you can copy fonts folder from /vendor/tcg/voyager/publishable/assets/.
For a definitive solution I think url should be changed in app.css with assets?path=fonts/bootstrap/etc..

@MrCrayon thanks.

I have a similar issue after upgrading to Voyager 1.2 and Laravel 5.6.

I dont't if could be the same or not.

@xXcoronaXx it's the same


Existing methods migrating to Platforms

These are some Methods that would fall into Platforms under the new schema (they are hardware devices or architectures):

[ ] Rigetti Aspen M-1 https://metriq.info/Method/60
[ ] Rigetti Aspen-11 https://metriq.info/Method/59
[ ] 20-Mode Universal Quantum Photonic Processor https://metriq.info/Method/75
[ ] Kerr parametric oscillator https://metriq.info/Method/124
[ ] Neutral-atom quantum processor https://metriq.info/Method/52
[ ] Rigetti Aspen-4 https://metriq.info/Method/29
[ ] Silicon quantum dot processor https://metriq.info/Method/51
[ ] SpinQ Triangulum https://metriq.info/Method/47
[ ] Summit Supercomputer https://metriq.info/Method/67
[ ] Trapped-ion https://metriq.info/Method/41
[ ] Trapped-ion QCCD (quantum charge-coupled device) https://metriq.info/Method/27
[ ] Zuchongzhi https://metriq.info/Method/42
[ ] Ibmq https://metriq.info/Method/97 (Note: This method has 28 child methods with most ibmq devices listed underneath)
[ ] ibmq-almaden https://metriq.info/Method/123
[ ] ibmq-valencia https://metriq.info/Method/121
[ ] Oxford Quantum Circuits - Lucy https://metriq.info/Method/91

This issue can be closed once the migration is complete. It seems to me from https://metriq.info/Platforms that it is almost done!

@WrathfulSpatula just a reminder that this issue is useful to help guide the users, as it cleans up the rationale behind Platforms/Methods content. Maybe @vprusso can help? Here I began to list the hardware devices for platform migration from methods.

@nathanshammah I'll work on this immediately, while I debug minor issues with the platforms feature release.

@nathanshammah There are still cases where the "method" and "platform" are both rightly equivalent to the "platform," specifically for QV and  CLOPS in particular. However, in other benchmark case (like MAXCUT) I replaced the "method" with the appropriate algorithm (QAOA) and keyed to the platform, as well.

I reopen this as the approved definitions for Methods and Platforms revise the choice in the taxonomy. E.g., the methods listed here: https://metriq.info/Submission/125 now fall under Platforms. Same applies to some quantum volume submissions with methods for simulated hardware chips.

I think we addressed that, back in May. (Closing.)


Improve performance of .drop_duplicates()

This is an 'epic'.
Child issues:

[x] #5554
[x] #5613

Let's close this one since all tasks done here?


Make logrus.Fields an implementable interface

I think it would be better if log.WithFields accept as input an interface so everyone can implement such interface if needed.
For example by now if I have a simple structure and i want to use it with the WithFields method I must create a wrapper that returns a logrus.Fields struct or write the attributes manually.
Sample Options struct:
type Options struct {
    Paginate bool
    Count    uint
    MaxPages uint
}

// Wrapper function can became the implementation of the interface
func (o *Options) ToFields() (fields log.Fields) {
    fields = make(log.Fields)
    fields["paginate"] = o.Paginate
    fields["count"] = o.Count
    fields["maxpages"] = o.MaxPages
    return
}

Given that struct we need to do either:
// With the wrapper
log.WithFields(options.ToFields()).Info( "Test" )

OR
// Manual entries
log.WithFields(log.Fields{
    "paginate": options.Paginate,
    "count": options.Count,
    "maxpages": options.MaxPages,
}).Info( "Test" )

But if we have the interface we can simply write:
log.WithFields(options).Info( "Test" )

This is a great idea and solves a bunch of problems people have asked for in the past at the right level of abstraction.

@Sirupsen Do you mind if I take a shot at tackling this?

@ryanfaerman sure!

I'm also interested in this feature, how can we do to don't break the existing function calls?

@alecha  we can add implementation of Fields interface to logrus.Fields and change signature. It would not break compatibility, I think.

That's the approach I'm taking. Creating an interface and then having Fields be a default implementation of said interface.

On Sep 8, 2015, at 5:55 AM, Anton Tyurin<EMAIL_ADDRESS>wrote:
@alecha we can add implementation of Fields interface to logrus.Fields and change signature. It would not break compatibility, I think.
—
Reply to this email directly or view it on GitHub.

I've created a simple interface in my PR, let me know if this is a good approach ;)

Any updates on this?

This issue is 5 years old. No plan for logrus 2 yet. Closing.


cp-helm-charts for production

Hi Team,
Can we use cp-helm-charts for production use, If not what are the changes to make it for production use from below link.
https://github.com/confluentinc/cp-helm-charts
I am planning to use below services from confluent community images.
kafka-connect
kafka
ksql-server
schema-registry
zookeeper
Thanks,
Bharath

You can "use" these charts fine. They are not "supported" and probably won't have frequent releases, is what mostly is meant by "not for production use".
Individually, each container can be deployed into a production environment, but may require a fair amount of configuration

@OneCricketeer Lack of support is fine, but can the components listed in the question be used perpetually without a license?


GPG Signing keys fail to validate again

The Dec. 13, 2019 problem:
GPG Signing keys fail to validate #86931
has returned. This concerns the VS Code repository signature at:
https://packages.microsoft.com/yumrepos/vscode/repodata/
On my machine (openSUSE 15.1), the commands:
sudo zypper clean --repo code
sudo zypper ref -f code
fail with the error messages:
Signature verification failed for file 'repomd.xml' from repository 'Visual Studio Code'.
. . .
Repository 'Visual Studio Code' is invalid.
[code|https://packages.microsoft.com/yumrepos/vscode] Valid metadata not found at specified URL
This problem also occurred on Oct. 11, 2019 (82347 and 82317). Is there a good way to create a long-term solution for this GPG signing issue? Thanks!

Thanks for fixing the GPG key. A few minutes ago I noticed that the date/time stamp for file "repomd.xml.asc" at:
https://packages.microsoft.com/yumrepos/vscode/repodata/
had changed from "13-Jan-2020 06:13" (older than the other files) to "14-Jan-2020 21:01" (one minute newer than the other files). Then the VS Code update from the repository succeeded.