Add Items & Inventory functionality

Note: I will start working on this soon. I was focusing on some AzSpeech problems that had occurred in the last few days.

Current status:
Last changes

Created some base classes to test the inventory fuctionality via data-driven tools: Data Tables + Data Assets;
Adjust existing Data Assets to receive correct IDs;
Added a testing blueprint to test the implemented functions;

How is it going?

Items add/remove are working well;
Didn't find a way to work with only UDataAssets, so i've created a FTableRowBase to handle the ID and the respective UDataAsset: The user can search for a UInventoryItemData by giving the Item ID;
There's some helper functions inside the UElementusInventoryFunctions class:
a) GetItemInfoByData: Will search for a FElementusItemInfo inside the InArr by the given UInventoryItemData and return a boolean value referred to the search succession. The passed parameter ItemIndex is a int reference that will be updated with the item index if the function find a valid data;
b) GetItemInfoById: Will search for a FElementusItemInfo inside the InArr by the given Item ID (InID) and return a boolean value referred to the search succession. The passed parameter ItemIndex is a int reference that will be updated with the item index if the function find a valid data;
c) GetUniqueElementusItemById: Will search for a UInventoryItemData inside the passed InDataTable (FElementusItemRowData) using the SearchString as the target RowName.
d) SearchElementusItem: Will search for a Array of items inside the InDataTable (FElementusItemRowData) based on the given parameters.

Note:

AElementusInventoryPackage is not yet implemented.
I don't recommend trying to use this update as it is still in the initial work-in-progress state.

Current Status:
Last Changes

Set data-driven objects to use DataAsset;
Adjust 'search and get' functions to use the AssetManager;
Adjust and improve existing and new DataAssets;
Adjust folders organization;
Remove some unused features;
Create new DataAssets for missing abilities;
Set IDs for existing and new DataAssets;

How is it going?

The item management now is based on the AssetManager: The items are UPrimaryDataAssets objects that can be loaded/unloaded at runtime using the Asset Manager.
The search is asynchronous and based on FPrimaryAssetIDs.

Current Status:
Last Changes

Adjust inventory related functions, actors and components;
Started working with the Inventory Package;
Add debugging functions to test how is it going;

How is it going?

There's some testing assets inside the plugins folder. The last one created was a Package Actor to test Add/Remove items from inventory and package and the behavior of "trade" between player and package.

Current Status:

Last Changes

Add widget to test inventory trade with a package;
Add delegate to track inventory updates;
Adjust folders;
Add new helper functions;

How is it going?

The package testing actor will open a widget when overlapped to test inventory trade between the packaged items and the player inventory.

To use the widget: Just touch the package with the character and double-click a item in the list.
Note: not working with multiplayer, it's currently a work in progress.

Current Status:

Last Changes

Add multiplayer support

How is it going?

Now it's working on multiplayer simulations,

Current Status:
Last Changes

Set the inventory system to use FPrimaryDataIds instead custom structures and adapted the entire code to follow this way;
The functions used are responsible for asynchronously loading and unloading the item data at runtime;

How is it going?

Now the system uses FPrimaryDataIds and the assets will be unloaded after every call of the created functions.
The system will load -> store -> unload -> return the results;

Current Status

Last Changes

Added a PostEditChangePropertyfunction to check the data that was passed into the item stack
Set a filter to log the item stack validation because the CDO is being verified too
Added a message dialog to warn the user about the wrong item data
Added two new functions: CanReceiveItem and CanGiveItem (currently unused)

How is it going?

When the user try to add a item to a item stack that derives from a class different than UInventoryItemData, the system will check all items and remove them from the item stack. Also will open a message dialog and print the informations in the log.
About the filter: For some reason, the PostEditChangeProperty was being called 3 times on every change, so i added this filter to only call if the change came from the default asset or from a world placed actor (IsSelectedInEditor() if WITH_EDITOR is true).
The CanReceiveItem will check if the inventory owner can handle the new items (will check the weight) and and CanGiveItem will verify if the inventory owner have the passed item with the given quantity.

The last commits made some changes to this repository:

Updated the EOS SDK
Remove the external dependency plugins from 'Plugins' folder
Set the external dependency plugins as submodules and created repositories for each one

Upcoming feature
Editor Module

Last Changes
Started creating a list to show all created items in one unique windows. Will create some options to sort, filter, open items and a option to create new items directly from this window.


fix setting indices 16bit or 32 bit based on number of vertices

Thanks for the pull request @joseph-kaile!

:x: Missing CLA.

Please send in a Contributor License Agreement (CLA) and comment back here to let us know to check this!
Maintainers, here's how to check for a submitted CLA.

:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

Reviewers, don't forget to make sure that:

[ ] Cesium for Unreal Samples works.

Good catch @joseph-kaile!
I think it might be wise to disallow std::numeric_limits::max() as well, i.e. switch to 32bit indices when the number of vertices is greater than or equal to that value. I'm mostly basing this on this note in the glTF spec:

The maximum values trigger primitive restart in some graphics APIs and would require client implementations to rebuild the index buffer.

Also, please update CHANGES.md.


correct way to remove BREAD?

Laravel Version: 5.6.22
Voyager Version: 1.1.1
PHP Version:
Database Driver & Version:

Description:
Correct way to remove BREAD. Direct in database?
Steps To Reproduce:

Have you tried the Delete BREAD button on the database list (/admin/database)?

Hi,
I have done this deletei by the manager and the error is shown.
ErrorException (E_ERROR)
SQLSTATE[42S02]: Base table or view not found: 1146 Table 'voy.pages' doesn't exist (SQL: select count(*) as aggregate from pages) (View: C:\www\voy\vendor\tcg\voyager\resources\views\dimmers.blade.php) (View: C:\www\voy\vendor\tcg\voyager\resources\views\dimmers.blade.php)

You have remove the PagesDimmer from your config at voyager.dashboard.widgets.

Perfect men.
Tks,


[ONNX] Reshape op

This PR adds the new allowzero parameter described in onnx opset 14 for reshape: https://github.com/onnx/onnx/blob/main/docs/Operators.md#Reshape
allowzero set to true means that a zero in the shape tensor will be treated as a true zero value (aka an empty tensor), as opposed to it signifying to copy the value at the equivalent position in the data tensor.
It's been a journey. Thanks @jwfromm for co-venturing :ship: :world_map:
Thoughts? @MargaretQian @anwang2009 @sfvaroglu

@AndrewZhaoLuo mind taking a look?

I'll take a look tomorrow. This looks potentially like one that will require attention

Ill look closer tomorrow, for now some initial stuff

Thanks for making this change @CircleSpin! The PR is now merged.


Error in web_1 container

Anyone can help me? I'm beginner in docker.
web_1               | GET http://es:9200/ [status:N/A request:0.004s]
web_1               | Traceback (most recent call last):
web_1               |   File "/usr/local/lib/python3.7/site-packages/urllib3/connection.py", line 171, in _new_conn
web_1               |     (self._dns_host, self.port), self.timeout, **extra_kw)
web_1               |   File "/usr/local/lib/python3.7/site-packages/urllib3/util/connection.py", line 79, in create_connection
web_1               |     raise err
web_1               |   File "/usr/local/lib/python3.7/site-packages/urllib3/util/connection.py", line 69, in create_connection
web_1               |     sock.connect(sa)
web_1               |   File "/usr/local/lib/python3.7/site-packages/gevent/_socket3.py", line 335, in connect
web_1               |     raise error(err, strerror(err))
web_1               | ConnectionRefusedError: [Errno 111] Connection refused
web_1               |
web_1               | During handling of the above exception, another exception occurred:
web_1               |
web_1               | Traceback (most recent call last):
web_1               |   File "/usr/local/lib/python3.7/site-packages/urllib3/connectionpool.py", line 600, in urlopen
web_1               |     chunked=chunked)
web_1               |   File "/usr/local/lib/python3.7/site-packages/urllib3/connectionpool.py", line 354, in _make_request
web_1               |     conn.request(method, url, **httplib_request_kw)
web_1               |   File "/usr/local/lib/python3.7/http/client.py", line 1229, in request
web_1               |     self._send_request(method, url, body, headers, encode_chunked)
web_1               |   File "/usr/local/lib/python3.7/http/client.py", line 1275, in _send_request
web_1               |     self.endheaders(body, encode_chunked=encode_chunked)
web_1               |   File "/usr/local/lib/python3.7/http/client.py", line 1224, in endheaders
web_1               |     self._send_output(message_body, encode_chunked=encode_chunked)
web_1               |   File "/usr/local/lib/python3.7/http/client.py", line 1016, in _send_output
web_1               |     self.send(msg)
web_1               |   File "/usr/local/lib/python3.7/http/client.py", line 956, in send
web_1               |     self.connect()
web_1               |   File "/usr/local/lib/python3.7/site-packages/urllib3/connection.py", line 196, in connect
web_1               |     conn = self._new_conn()
web_1               |   File "/usr/local/lib/python3.7/site-packages/urllib3/connection.py", line 180, in _new_conn
web_1               |     self, "Failed to establish a new connection: %s" % e)
web_1               | urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPConnection object at 0x7f1ba5a61898>: Failed to establish a new connection: [Errno 111] Connection refused

Can you confirm if your ElasticSearch container started up correctly? This error might indicate that ES did not start up due to insufficient map count. See issue #221 and PR #224 for a fix. That involves adding the line
vm.max_map_count=262144

to your /etc/sysctl.conf file and then rebooting your machine.

Please join us on Slack in the #cc-gsoc channel if you are having problems getting started. I'm going to close this issue since it's not a bug.


Can not analyze smart contract due to Souffle

I have configured successfully for all the prerequisite, when I entered
python securify

I can get:
usage: securify contract.sol [opts]
securify: error: the following arguments are required: contract

But when I tried to analyze a simple smart contract with reentrancy vulnerability, I got the error:
Environment variable LD_LIBRARY_PATH not set. Setting it up...
pragma directive defines a prior version to 0.6.7. Changing pragma version to 0.6.7....
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "securify/__main__.py", line 286, in <module>
    main()
  File "securify/__main__.py", line 272, in main
    severity_exc=args.exclude_severity)
  File "securify/__main__.py", line 15, in get_list_of_patterns
    pattern_classes = discover_patterns()
  File "/Users/ningyu/securify2/venv/lib/python3.7/site-packages/securify-0.0.1-py3.7.egg/securify/analyses/analysis.py", line 123, in discover_patterns
    patterns.update(c.get())
  File "/Users/ningyu/securify2/venv/lib/python3.7/site-packages/securify-0.0.1-py3.7.egg/securify/analyses/patterns/static/static_analysis_patterns.py", line 16, in get
    patterns = cls.__list_static_patterns()
  File "/Users/ningyu/securify2/venv/lib/python3.7/site-packages/securify-0.0.1-py3.7.egg/securify/analyses/patterns/static/static_analysis_patterns.py", line 23, in __list_static_patterns
    patterns = static_analysis.discover_patterns()
  File "/Users/ningyu/securify2/venv/lib/python3.7/site-packages/securify-0.0.1-py3.7.egg/securify/staticanalysis/static_analysis.py", line 69, in discover_patterns
    souffle_kwargs=kw_args_souffle)
  File "/Users/ningyu/securify2/venv/lib/python3.7/site-packages/securify-0.0.1-py3.7.egg/securify/staticanalysis/souffle/souffle.py", line 62, in run_souffle
    **(souffle_kwargs or {}))
  File "/Users/ningyu/securify2/venv/lib/python3.7/site-packages/securify-0.0.1-py3.7.egg/securify/staticanalysis/souffle/wrapper.py", line 119, in souffle_wrapper
    stderr_data=codecs.decode(stderrdata),
securify.staticanalysis.souffle.exceptions.SouffleError: Error during souffle execution:

cannot open file /Users/ningyu/securify2/venv/lib/python3.7/site-packages/securify-0.0.1-py3.7.egg/securify/staticanalysis/souffle_analysis/analysis.dl

I have  tried souffle from 1.6.2 to 1.7.2, none of them can settle this thing down. Could you please help me with this problem, thank you!

Unfortunately, I haven't been able to reproduce your error. It seems like securify is looking for the datalog file in the wrong place. Could you provide more information on the installation process you followed?
What message do you get when you execute the following commands in the securify2 directory?
pip install -r requirements.txt
pip install -e .
securify <solidity-source>.sol

I followed your instructions. The first and second lines execute successfully. However, when I try to run the third line, I got this error:
Environment variable LD_LIBRARY_PATH not set. Setting it up...
pragma directive defines a prior version to 0.6.7. Changing pragma version to 0.6.7....
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "securify/__main__.py", line 286, in <module>
    main()
  File "securify/__main__.py", line 272, in main
    severity_exc=args.exclude_severity)
  File "securify/__main__.py", line 15, in get_list_of_patterns
    pattern_classes = discover_patterns()
  File "/Users/ningyu/securify2/securify2-github/securify/analyses/analysis.py", line 123, in discover_patterns
    patterns.update(c.get())
  File "/Users/ningyu/securify2/securify2-github/securify/analyses/patterns/static/static_analysis_patterns.py", line 16, in get
    patterns = cls.__list_static_patterns()
  File "/Users/ningyu/securify2/securify2-github/securify/analyses/patterns/static/static_analysis_patterns.py", line 23, in __list_static_patterns
    patterns = static_analysis.discover_patterns()
  File "/Users/ningyu/securify2/securify2-github/securify/staticanalysis/static_analysis.py", line 69, in discover_patterns
    souffle_kwargs=kw_args_souffle)
  File "/Users/ningyu/securify2/securify2-github/securify/staticanalysis/souffle/souffle.py", line 62, in run_souffle
    **(souffle_kwargs or {}))
  File "/Users/ningyu/securify2/securify2-github/securify/staticanalysis/souffle/wrapper.py", line 119, in souffle_wrapper
    stderr_data=codecs.decode(stderrdata),
securify.staticanalysis.souffle.exceptions.SouffleError: Error during souffle execution:

This installation of Souffle does not support concurrent jobs.
libc++abi.dylib: terminating with uncaught exception of type std::invalid_argument: stoi: no conversion

It seems like I have another problem with Souffle. Moreover, when I type souffle in command line, I got this version information:
----------------------------------------------------------------------------
Version: 1.7.0
----------------------------------------------------------------------------
Copyright (c) 2016-18 The Souffle Developers.
Copyright (c) 2013-16 Oracle and/or its affiliates.
All rights reserved.

It seems like the problem is related to the souffle binary. Have you tried souffle 1.6.2 instead after following my instructions?

I have reinstalled the souffle 1.6.2, I still got an error:
Environment variable LD_LIBRARY_PATH not set. Setting it up...
pragma directive defines a prior version to 0.6.7. Changing pragma version to 0.6.7....
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "securify/__main__.py", line 286, in <module>
    main()
  File "securify/__main__.py", line 272, in main
    severity_exc=args.exclude_severity)
  File "securify/__main__.py", line 15, in get_list_of_patterns
    pattern_classes = discover_patterns()
  File "/Users/ningyu/securify2/securify2-github/securify/analyses/analysis.py", line 123, in discover_patterns
    patterns.update(c.get())
  File "/Users/ningyu/securify2/securify2-github/securify/analyses/patterns/static/static_analysis_patterns.py", line 16, in get
    patterns = cls.__list_static_patterns()
  File "/Users/ningyu/securify2/securify2-github/securify/analyses/patterns/static/static_analysis_patterns.py", line 23, in __list_static_patterns
    patterns = static_analysis.discover_patterns()
  File "/Users/ningyu/securify2/securify2-github/securify/staticanalysis/static_analysis.py", line 69, in discover_patterns
    souffle_kwargs=kw_args_souffle)
  File "/Users/ningyu/securify2/securify2-github/securify/staticanalysis/souffle/souffle.py", line 62, in run_souffle
    **(souffle_kwargs or {}))
  File "/Users/ningyu/securify2/securify2-github/securify/staticanalysis/souffle/wrapper.py", line 119, in souffle_wrapper
    stderr_data=codecs.decode(stderrdata),
securify.staticanalysis.souffle.exceptions.SouffleError: Error during souffle execution:

Warning: OpenMP is not enabled
libc++abi.dylib: terminating with uncaught exception of type std::invalid_argument: stoi: no conversion

I dont know if it is related to MacOS, maybe I should compile it on linux? Could you please tell me what is your testing operating system and its version? Maybe I should install a virtual machine

It is probably related to MacOS as you mentioned. I would recommend you to try to compile it in a virtual machine with ubuntu 18.04 or use Docker instead.


Incorrect matching of string

Software versions

OS: e.g. Mac OSX 10.13.6
Library version:
Node Version: node --version 11.6.0

Expected behaviour
Expected api response set from pact-jvm
private DslPart getResponseBody()  {
     return new PactDslJsonBody()
                     .stringType("content");
}

Below is updated in the pact-broker
provider will respond with:

{
  "status": 200,
  "headers": {
    "Content-Type": "application/json; charset=UTF-8"
  },
  "body": {
    "content": "string",
    "contentType": "application/pdf"
   }
}

Below is the provider script in pact-node:
const { Verifier } = require('@pact-foundation/pact');
var package = require('../../package.json');
const { app } = require('../../app.js');

app.listen(8081, () => {
    console.log("itg-pdf-generator service listening on http://localhost:8081")
})

describe("Pact provider verification", () => {
    it("Validates the response", () => {
        let opts = {
            provider: "provider",
            logLevel: "DEBUG",
            providerBaseUrl: "http://localhost:8081",
            stateHandlers: {
                "Return response": () => {
                    return Promise.resolve(`Return response`);
                },
            },
            // Fetch pacts from broker
            pactBrokerUrl: "https://pactbroker-snapshot.kops-ci.company.tools",
            // Fetch from broker with given tags
            tags: ["master"],
            publishVerificationResult: true,
            providerVersion: package.version,
        }
        // Validates the pact & uploads the verification results to pact broker
        return new Verifier(opts).verifyProvider().then(output => {
            console.log(output);
        }).catch((error) => {
            console.log('error:', JSON.stringify(error.message))
        })
    })
})

The matching on the response for node content in the expected response should pass any string.
Actual behaviour
The word string is compared with the actual response.
Steps to reproduce
Provide a repository, gist or reproducible code snippet so that we can test the problem.
Consider forking the project and modifying the E2E test
Relevant log files
Please ensure you set logging to DEBUG and attach any relevant log files here (or link from a gist).

Thanks. This probably belongs in pact-js, not here.
Also, please attach your pact file and any relevant logs.
It looks like the matcher is not being serialised into the pact file, as it seems to be matching verbatim.


Gremlin Client NPM Version Clarification

I'm looking to implement a Node.js server that can make queries against a separate graph database (AWS Neptune).  Your repo at https://www.npmjs.com/package/gremlin seemed perfect for my needs when I looked at it on Friday, but Monday morning it was updated to v3.3.2 and points to a completely different repo that has a different interface.
I'm going to try using v2.7 for now, but is there a new recommended way to form a Gremlin client in Node.js?  I want to learn about best practices, but I also appreciate the ReadMe you laid out that isn't present on the v3 npm site. 😅
Thank you for your time,
-- David

PS - oh, and I Ifound an example in their knowledgebase that also uses, this repo.   It looks like yesterday AWS Neptune folks also recommend v2.7 for the time being while they adjust their docs.

This library has been superseded by the official Apache TinkerPop JavaScript/Node.js client which was just published as v3.3.2 on npmjs.org (this matches Apache TinkerPop version). I may be pushing fixes to the v2.x branches here but no further development will be done for this library on this repository.
We decided to publish the official JS Gremlin language variant under the same package name gremlin on npmjs, to avoid fragmenting the ecosystem and adding confusion for newcomers. Since it's a new major version, hopefully it will only cause issue for users relying on the latest version. Doing npm i gremlin@v2 should work for people interested in the older, non official version. I'm happy AWS Neptune already updated their doc! Actually, I wasn't aware that this lib was in the official documentation. I didn't check but I think Microsoft CosmosDB may have to slightly update their doc too.
If you wish to follow the development of the official JavaScript Gremlin language variant, see: https://github.com/apache/tinkerpop/tree/46b743d/gremlin-javascript
I was really happy to start this PoC almost 4 years ago. I learnt a lot! I think this mono-repository will be dedicated to other JS related graph tools, most likely an in-memory graph database, a JS graph traversal machine, and other useful tools which I think would bring a lot of value to the JS/web ecosystem.


Netbox Inventory Custom Field

Since a new fresh installation of Netbox (Version 4.0.9)  Netbox Inventory is not available at "Object types" and "Related object type" when i want Create a Custom Field. Other Plugins like Netbox Attachments are available.

It seems to happen on any netbox 4.0.x version on a fresh install and for now it seems to be an issue related to netbox-attachments plugin or the order in which the plugins are specified in netbox config. All plugins listed after netbox_attachments have their models missing.
So for example with this config:
PLUGINS = [
    'netbox_attachments',
    'netbox_inventory',
    'netbox_dns',
]

I cannot select any object types from netbox_inventory or netbox_dns when creating custom field.
If I move netbox_attachments to last place:
PLUGINS = [
    'netbox_inventory',
    'netbox_dns',
    'netbox_attachments',
]

Then I can select object types from all three plugins.
I'm not sure what is going on. Looking at netbox_inventory and netbox_attachments code, I don't see any obvious reason for this issue. This issue might take a while to resolve.
For now I suggest you try to move  netbox_attachments to last place in your PLUGINS setting and let me know if it works.

It seems to happen on any netbox 4.0.x version on a fresh install and for now it seems to be an issue related to netbox-attachments plugin or the order in which the plugins are specified in netbox config. All plugins listed after netbox_attachments have their models missing.
So for example with this config:
PLUGINS = [
    'netbox_attachments',
    'netbox_inventory',
    'netbox_dns',
]

I cannot select any object types from netbox_inventory or netbox_dns when creating custom field.
If I move netbox_attachments to last place:
PLUGINS = [
    'netbox_inventory',
    'netbox_dns',
    'netbox_attachments',
]

Then I can select object types from all three plugins.
I'm not sure what is going on. Looking at netbox_inventory and netbox_attachments code, I don't see any obvious reason for this issue. This issue might take a while to resolve.
For now I suggest you try to move netbox_attachments to last place in your PLUGINS setting and let me know if it works.

Thank you so much. That did the trick.  Maybe we should contact the developer of the attachments plugin.


PiCamera out of resources

My camera will not work no matter what and I require help please. I have a V2.1 camera RPI 3B+ and I just reinstalled the os fresh. First thing I did is connect camera, check for uptdates and then wrote this script:
from picamera import PiCamera
camera=PiCamera()
PyCameraMMALError:
Failed to enable connection: Out of resources
This is where I get the out of resources error. I made sure camera is enable in configuration, GPU memory if 512MB so that is def not the problem HELP pls I am trying to fix this issue for 8 hours
No I am not using the camera in ANY other process. This is a fresh OS install I did not touch anything but plug the camera in and check for firmware+software updates, increase gpu memory and then reboot.

Does the camera work with raspistill from the command line? That's usually a good place to start debugging things. If you can get it working with raspistill we can be certain there's no hardware issue at play here.


fix_typo of map_structure doc

Can one of the admins verify this patch?

I signed it!

@tensorflow-jenkins test this please

Yes! My first contribution to Tensorflow. I should do this more often.

Congrats! By all means, contributions are welcome :)


[BUG] - Error while Launching CropImage

class file has wrong version 55.0, should be 52.0
Please remove or make sure it appears in the correct subdirectory of the classpath

Please check previous solved issues, small search always help

Hey welcome 👋 to the library and thanks for the Issue.
Can you please follow the template we have for issues? 🙏
There are many points on the template and will help me to help you.
I will close 🔴  this issue so you can reopen following the template,
Is important to remember that my work here is volunteer and to save time and have better answer help me that people use the templates.
Thanks!! 🎉


Feedback - There is no up-to-date learn documentation

Feedback received - There is no up-to-date learn documentation
Help validate if there are any updates to the docs.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 41c93c1e-c426-148c-a545-f8bd5b3763ff
Version Independent ID: afcc83c4-12c0-bab6-1cec-61ac463b4f60
Content: Add Teams tab to SharePoint - Teams
Content Source: msteams-platform/tabs/how-to/tabs-in-sharepoint.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: surbhigupta

Hello @v-preethah - Thanks for raising this anonymous feedback.
We will look into it and let you know the updates.

Raised the documentation bug for this issue.

As per the discussion with doc PM closing the issue.


Show Purchase button properly on artwork screen when there is a partner offer

This PR resolves EMI-1844
Description
Purchase button was not properly displayed on artwork screen in some cases when the artwork had a partner offer. This PR updates the logic and expands the test cases to cover (hopefully) all major scenarios.
PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

Show Purchase button properly on artwork screen when there is a partner offer - starsirius

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Cross-platform user-facing changes (Show Purchase button properly on artwork screen when there is a partner offer - starsirius)

  Generated by :no_entry_sign: dangerJS against bfe080b13f96df7e8d290c0b314c0150872b7ea0

@MrSltun Since we are approaching code freeze tomorrow and I'm in timezone that's behind, I assigned this PR to you. Before I sign on tomorrow, please feel free to push new changes or merge! 🙏


Kubernetes 1.11

https://github.com/kubernetes/sig-release/blob/master/releases/release-1.11/release-1.11.md

See https://github.com/kontena/pharos-cluster/issues/448#issuecomment-399885443 for kubeadm kubelet --resolv-conf= changes.

Some relevant points from the release notes: https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG-1.11.md

phase/addon resource files applied using Pharos::Kube::Stack are now case-sensitive... with the direct YAML -> JSON conversion that we're doing, this might mean that all of our YAML files are now broken?
kubeadm config is now v1alpha2 with major changes:

.noTaintMaster => .nodeRegistration.taints
.CloudProvider was removed...

XXX: how to set --cloud-provider=external?

.NodeName => .NodeRegistration.Name
.CRISocket => .NodeRegistration.CRISocket

apiserver --endpoint-reconciler-type default is now lease, and the old master-count reconciler with --apiserver-count is deprecated
kubeadm /etc/systemd/system/kubelet.service.d/*.conf dropins are different
kubeadm will replace the old Kube DNS with the new CoreDNS by default
kube-proxy's IPVS feature gate is now true by default, but the default mode still seems to be iptables?

    	[WARNING ExternalEtcdVersion]: this version of kubeadm only supports external etcd version >= 3.2.17. Current version: 3.1.12

For the 1.10 -> 1.11 upgrade, kubeadm has a new kubeadm upgrade node config command: https://kubernetes.io/docs/tasks/administer-cluster/kubeadm/kubeadm-upgrade-1-11/#upgrade-kubelet-on-each-node
Need to check out what it actually does, and integrate it into the MigrateWorker phase...

I wonder if kube-proxy is supposed to default to ipvs now? The https://kubernetes.io/docs/reference/command-line-tools-reference/kube-proxy/ --proxy-mode docs may just be outdated...
    	[WARNING RequiredIPVSKernelModulesAvailable]: the IPVS proxier will not be used, because the following required kernel modules are not loaded: [ip_vs_wrr ip_vs_sh ip_vs ip_vs_rr] or no builtin kernel ipvs support: map[ip_vs:{} ip_vs_rr:{} ip_vs_wrr:{} ip_vs_sh:{} nf_conntrack_ipv4:{}]
    you can solve this problem with following methods:
     1. Run 'modprobe -- ' to load missing kernel modules;
    2. Provide the missing builtin kernel ipvs support


Update README.md

Scan submitted to Checkmarx

Checkmarx scan completed
Full Scan Details
Violation Summary

Severity
Count

High
1

Details

Lines
Severity
Category
File
Link

12
High
Command_Injection
src/main/java/com/scalesec/vulnado/CowController.java
Checkmarx


The name of the application is too long to assign an IP

Describe the version
version about:

spiderpool v0.4.0

Describe the bug
The name of the application is too long to assign an IP
How To Reproduce

To construct a deploy with an extra-long name that does not exceed the k8s limit of 63 characters, yaml as follows:

apiVersion: apps/v1
kind: Deployment
metadata:
  name: test-app-xxxxxxxxxxxx-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
  namespace: ns4813-562811032
spec:
  selector:
    matchLabels:
      app: test-app-xxxxxxxxxxxx-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
  template:
    metadata:
      annotations:
        ipam.spidernet.io/ippool-reclaim: "false"
        ipam.spidernet.io/subnet: '{"ipv4":["v4-ss-4814-630082592"],"ipv6":["v6-ss-4814-675208718"]}'

Create a deploy as described above
Failed to create, get the error log as follows

{"level":"ERROR","ts":"2023-05-08T02:55:35.139Z","logger":"SpiderSubnet-Application-Controllers","caller":"applicationcontroller/app_controller.go:579","msg":"encountered app controller syncHandler error 'failed to create or scale IPPool: failed to get auto-created IPPoolList with matchLabels map[ipam.spidernet.io/interface:eth0 ipam.spidernet.io/ip-version:IPv4 ipam.spidernet.io/owner-application:apps_v1_Deployment_ns4813-562811032_test-app-xxxxxxxxxxxx-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx ipam.spidernet.io/owner-spider-subnet:v4-ss-4814-630082592], error: unable to parse requirement: values[0][ipam.spidernet.io/owner-application]: Invalid value: \"apps_v1_Deployment_ns4813-562811032_test-app-xxxxxxxxxxxx-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx\": must be no more than 63 characters; failed to get auto-created IPPoolList with matchLabels map[ipam.spidernet.io/interface:eth0 ipam.spidernet.io/ip-version:IPv6 ipam.spidernet.io/owner-application:apps_v1_Deployment_ns4813-562811032_test-app-xxxxxxxxxxxx-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx ipam.spidernet.io/owner-spider-subnet:v6-ss-4814-675208718], error: unable to parse requirement: values[0][ipam.spidernet.io/owner-application]: Invalid value: \"apps_v1_Deployment_ns4813-562811032_test-app-xxxxxxxxxxxx-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx\": must be no more than 63 characters', requeue it after '5s'","OPERATION":"process_app_items","Application":"Deployment/ns4813-562811032/test-app-xxxxxxxxxxxx-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"}

Additional context
~# kubectl describe po -n ns4813-562811032   test-app-xxxxxxxxxxxx-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx-dhf4vm
Normal   Scheduled               12m                  default-scheduler  Successfully assigned ns4813-562811032/test-app-xxxxxxxxxxxx-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx-dhf4vm to spider-worker
  Warning  FailedCreatePodSandBox  11m                  kubelet            Failed to create pod sandbox: rpc error: code = Unknown desc = failed to setup network for sandbox "92985d986346bf1536c473fa13f1926924a276802aa4fd47e1e127b6dadf4417": plugin type="multus" name="multus-cni-network" failed (add): [ns4813-562811032/test-app-xxxxxxxxxxxx-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx-dhf4vm:macvlan-cni-default]: error adding container to network "macvlan-cni-default": spiderpool IP allocation error: [POST /ipam/ip][500] postIpamIpFailure  failed to allocate IP addresses in standard mode: failed to get IPPool candidates from Subnet: no matching auto-created IPPool candidate with matchLables 'map[ipam.spidernet.io/interface:eth0 ipam.spidernet.io/ip-version:IPv4 ipam.spidernet.io/owner-application:apps_v1_Deployment_ns4813-562811032_test-app-xxxxxxxxxxxx-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx ipam.spidernet.io/owner-application-uid:444e3b76-cce1-4462-aafe-f55761a14af1 ipam.spidernet.io/owner-spider-subnet:v4-ss-4814-630082592]'; no matching auto-created IPPool candidate with matchLables 'map[ipam.spidernet.io/interface:eth0 ipam.spidernet.io/ip-version:IPv6 ipam.spidernet.io/owner-application:apps_v1_Deployment_ns4813-562811032_test-app-xxxxxxxxxxxx-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx ipam.spidernet.io/owner-application-uid:444e3b76-cce1-4462-aafe-f55761a14af1 ipam.spidernet.io/owner-spider-subnet:v6-ss-4814-675208718]'

#1749


Add all circle ci tests to testgrid

Results are up at https://k8s-testgrid.appspot.com/istio#circleci-e2e-pilot but contains only the runs from this particular PR. Once it is merged all the test results will show up.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
Assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

.circleci/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@chxchx: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
b6d5e0a6b826e3778898a4a9a01d2e8eb4e289fe
link
/test istio-presubmit

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

@chxchx: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
33b8739f482398f2b3c892863d5f7bc4b2bdb7ec
link
/test e2e-bookInfo-envoyv2-v1alpha3

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

@chxchx: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
2e7f3b28c611be60f5cd932b102fd9616582369a
link
/test e2e-bookInfo-envoyv2-v1alpha3

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

/assign @costinm @sebastienvas

@chxchx: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
408c1889a737c6d4cd8212f66c94fb2144ccb240
link
/test e2e-bookInfo-envoyv2-v1alpha3

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

@chxchx: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
a4f2cb6300e288e02436ea716a1af75dcf8e6957
link
/test e2e-bookInfo-envoyv2-v1alpha3

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

/lgtm

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: xiaolanz
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
Assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

.circleci/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hklai, xiaolanz
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

.circleci/OWNERS [hklai]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Merging this to get better circleci test visibility on testgrid.


Reuploaded: updates for Visualize SDSS Spectra w Bokeh

@rnikutta, if you could please recover the file sdss_viz_bokeh.py, I think we ought to delay this PR until that file is present.

@weaverba137 and @jacquesalice I have the file! Can you pull it or copy by hand from rnikutta branch?
https://github.com/noaodatalab/notebooks-latest/blob/rnikutta/04_HowTos/FileService/sdss_viz_bokeh.py
My recollection is that it does not yet work with the notebook b/c of the global variables it relies on.

I've added the module.  I'll see if I can debug it quickly.

Many thanks!

Note that this branch seems to have been created from somewhere other than the master branch, because the master branch imports sdss_viz_bokeh, but this notebook does not.  I'm basically going to have to gut-replace the notebook on this branch, as well as debug the module.

At some point I needed to "bless" the master branch to be able to release to auto-NB testing. I suppose that's where I forgot to git add the py file, apologies. It should be only required to import sdss_viz_bokeh, and of course fix how the NB interacts with it.

This is now ready for testing.

Thanks for this one @jacquesalice and @weaverba137 !
Runs fine except two things:

ac.whoAmI() needs to be commented out, and have the standard "If you need to log in to Data Lab ..." before the cell. And ac needs to be actually imported in the import cell.

The last cell ("Navigating through targets") just spins forever, and nothing happens

import time
sp.ispec = 0
for i in range(sp.nspec):
    sp.next()
    time.sleep(0.5)

Also, if this is to be an interactive cell, we need to comment it out, and add a sentence about uncommenting, so that the NB can run in auto-testing mode.

Regarding #2: the code should be working fine, but it is not  obvious. You have to run that cell then scroll back up to the interactive plot to see it change. The documentation should describe that better.

I propose then that we have the "viewport" (can be a new one) immediately above this cell, so that it's immediately obvious. OK?

@rnikutta , it appears that you let #64 pass without making any specific requirements on having ac.whoAmI() commented out.  Therefore this notebook just copies what was done in #64.  I'll change it as requested.  You may want to think about a way to automatically check the boilerplate code in these notebooks, beyond just making sure the notebooks don't fail.

Sorry I misspoke. ac.whoAmI() is fine, just needs to imported (it wasn't). When I wrote this I had ac.login() in mind. Apologies.

OK, new version to test.

Thanks, looks good. Only thing: the loop over spectra takes 2.9s per spectrum (plotting is slow I guess), and sp.nspec is 1000. So that's 48 minutes, way too long for the auto-testing. Would it suffice to loop instead over the first 20 spectra or so, for demonstration purposes? Thanks.

Loop shortened.

Perfect, thank you. Merging.


Updated: webcatalog 19.3.4

$notices:
- Updated `repo`

dcfa41e2-9bdf-40e4-84b5-2c54b3da7a9c

Newer version of webcatalog (20.1.1) is now available in master. 86810109ea1fc55b261b58a658eb3be4689ce4e6


Revert "Revert "Use clang.arc.attachedcall for emission of objc_retainAutoreleasedReturnValue"

Also add an optnone test

Please test with following pull request:
https://github.com/apple/llvm-project/pull/7456
@swift-ci test


Adding Landr-created directories and files to repo

Change-type: patch

@balena-ci I self-certify!


x/tools/go/vcs: refresh from cmd/go

https://github.com/kardianos/govendor/issues/265 is due to golang.org/x/tools/go/vcs/http.go using http.DefaultClient.get(privateBitbucketUrl) without an Authorization header.
bitbucketVCS should extract the login and password from ~/.netrc

@rsc, opinions on this?

Reading .netrc seems a bit hacky.

How is that hacky if the user who created the .netrc file (%HOME%\_netrc in Windows) is running a process which needs to access one of those domains?   ...as long as you don't post the contents to Twitter or something ;-)
Another (not even mutually exclusive) option would be to parse ~/.gitconfig:
[url<EMAIL_ADDRESS>        insteadOf = https://bitbucket.org/
[url<EMAIL_ADDRESS>        insteadOf = https://github.com/

It's hacky because it's not the go command's job to recreate the functionality of other tools. We're not reading .gitconfig and we're not reading .netrc, and that's good.
That said, I think the go command itself handles bitbucketVCS just fine - for an auth failure it falls back to invoking git/hg directly, which is the perfect way to respect .netrc without parsing .netrc - and the x/tools/go/vcs copy is just out of date. Perhaps it is enough for someone to refresh the x/tools copy.

... So govendor sync is calling the wrong X/tools function?

No, I think x/tools/go/vcs (which is a copy of code internal to the go command) is out of date and needs to be updated.

Perhaps the go command should vendor this library so that there is one version of this code to maintain?

There will always be two copies to maintain. The go command is the primary source here.

This just needs to be refreshed from the cmd/go master.

I think go get should respect .netrc file at least while fetching meta-data.
My company's code repos (derived  from Gitlab) that only authorized user access any URL.

This package has been deprecated in #57051 and won't receive further development, so closing this as not planned.


ENH: Use platform-specific EOL character in log file

By default text files have \r\n ending on Windows and \n on Linux/Mac. The old implementation had \n end-of-line character hardcoded for all platforms.
On Windows, when trying to open a file (or copy-paste contents) that use \n as end-of-line character in a simple editor, such as Notepad, all the lines appear in one single line. This is very confusing for average users.
The fix uses the correct platform-specific end-of-line characters (\n or \r\n) instead of a hardcoded \n character.

+1
On Mon, Jan 5, 2015 at 4:42 PM, Andras Lasso<EMAIL_ADDRESS>wrote:

By default text files have \r\n ending on Windows and \n on Linux/Mac. The
old implementation had \n end-of-line character hardcoded for all platforms.
On Windows, when trying to open a file (or copy-paste contents) that use
\n as end-of-line character in a simple editor, such as Notepad, all the
lines appear in one single line. This is very confusing for average users.
The fix uses the correct platform-specific end-of-line characters (\n or
\r\n) instead of a hardcoded \n character.
You can merge this Pull Request by running
git pull https://github.com/lassoan/CTK file-logger-platform-specific-eol
Or view, comment on, or merge it at:
https://github.com/commontk/CTK/pull/526
Commit Summary

ENH: Use platform-specific EOL character in log file

File Changes

M Libs/Core/ctkFileLogger.cpp
https://github.com/commontk/CTK/pull/526/files#diff-0 (2)

Patch Links:

https://github.com/commontk/CTK/pull/526.patch
https://github.com/commontk/CTK/pull/526.diff

—
Reply to this email directly or view it on GitHub
https://github.com/commontk/CTK/pull/526.
The information in this e-mail is intended only for the person to whom it
is
addressed. If you believe this e-mail was sent to you in error and the
e-mail
contains patient information, please contact the Partners Compliance
HelpLine at
http://www.partners.org/complianceline . If the e-mail was sent to you in
error
but does not contain patient information, please contact the sender and
properly
dispose of the e-mail.


agent: Kill only init process if specified this way

In case we were running CRI-O testing inside Semaphore, because the
system is kind of slow, the redis container process takes some time
to really start. Because we were issuing a kill signal right after
the start returned (does not mean the container's process finished
to start), and at the same time we were trying to kill all processes
related to the init process, this was hanging. Indeed, we should not
try to kill all processes until the init one is properly running.
This patch relies on AllProcesses flag provided in the "killcontainer"
payload, instead of hardcoding this value to "true". Indeed, the CRI-O
tests explicitely ask for a kill without the --all falg enabled. This
simple change allows our agent to get CRI-O testing passing properly
on Semaphore CI.
Fixes #61

lgtm

@amshinde, @mcastelino please take a look.

The change looks fine, but please can you add a brief comment in the code explaining why AllProcesses is being used.
Also, there are a couple of typos in the commit (falg => flag, explicitely => explicitly).

The patch looks fine to me, but I do wonder if we are still glossing over a race/bug condition - presumably somebody can still issue a start immediately followed by a kill-all and it could hang up. So, is there a deeper rooted issue here still that should still be fixed some time in the future?

@grahamwhaley yes I agree, and that's why I am planning to start a discussion with CRI-O/runc folks.

@jodh-intel I am fixing the typo, but I don't think we need any comment to explain why we use AllProcesses. I mean, it is obvious that we are passing the flag that we got from our runtime, and that's the same flag expected by libcontainer. What comment would you expect ?

@sboeuf - question: why wasn't it obvious to use it in the previous version of the code then? :)

LGTM


add responsive images as an experimental flag page

Description (required)
Creates a new experimental flag page for responsive images based on what was previously added to configuration-reference
Notably:

ONLY covers the content from experimental.resoponsiveImages and not the image.experimental* features. (Is this how we want to do this? Leave those in regular configuration reference?)

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

en/reference/experimental-flags/responsive-images.mdx
Localization added, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

Here's what your new page will look like @ascorbic ! Feel free to make this look like what you want before the feature merge tomorrow, and even after the fact we can quickly adjust if and as necessary!
PS You can now see what I mean about the additional config options. I'm open to moving them here, or leaving them in regular config reference. What do you think?
PPS I know there is still content in the public/types file that we'll remove at some point. I'm more concerned about getting this page the way you want it first!


Upgrading Moesif SDK Version

Purpose

https://github.com/wso2-enterprise/choreo/issues/25069

Codecov Report

Merging #109 (7ebfe1d) into main (5c003fb) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff            @@
##               main     #109   +/-   ##
=========================================
  Coverage     40.45%   40.45%
  Complexity      164      164
=========================================
  Files            54       54
  Lines          1352     1352
  Branches        125      125
=========================================
  Hits            547      547
  Misses          747      747
  Partials         58       58

Flag
Coverage Δ

unit_tests
40.45% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


version cannot be both static and dynamic

The Insiders version defines version both as a static field (version = "0.2.1") and a dynamic field (dynamic = ["version"] and [tool.hatch.version] path = "src/canonical_imports/__init__.py"), which is forbidden by the relevant PEP.
Therefore, trying to build a source distribution fails:
% pyproject-build
* Creating venv isolated environment...
* Installing packages in isolated environment... (hatchling)
* Getting build dependencies for sdist...
* Building sdist...
Traceback (most recent call last):
  File "/home/pawamoy/.local/pipx/venvs/build/lib/python3.11/site-packages/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
    main()
  File "/home/pawamoy/.local/pipx/venvs/build/lib/python3.11/site-packages/pyproject_hooks/_in_process/_in_process.py", line 335, in main
    json_out['return_val'] = hook(**hook_input['kwargs'])
                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/pawamoy/.local/pipx/venvs/build/lib/python3.11/site-packages/pyproject_hooks/_in_process/_in_process.py", line 304, in build_sdist
    return backend.build_sdist(sdist_directory, config_settings)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/tmp/build-env-g0wj7zqz/lib/python3.11/site-packages/hatchling/build.py", line 34, in build_sdist
    return os.path.basename(next(builder.build(directory=sdist_directory, versions=['standard'])))
                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/tmp/build-env-g0wj7zqz/lib/python3.11/site-packages/hatchling/builders/plugin/interface.py", line 90, in build
    self.metadata.validate_fields()
  File "/tmp/build-env-g0wj7zqz/lib/python3.11/site-packages/hatchling/metadata/core.py", line 243, in validate_fields
    _ = self.version
        ^^^^^^^^^^^^
  File "/tmp/build-env-g0wj7zqz/lib/python3.11/site-packages/hatchling/metadata/core.py", line 128, in version
    self._version = self._get_version()
                    ^^^^^^^^^^^^^^^^^^^
  File "/tmp/build-env-g0wj7zqz/lib/python3.11/site-packages/hatchling/metadata/core.py", line 224, in _get_version
    version = core_metadata.version
              ^^^^^^^^^^^^^^^^^^^^^
  File "/tmp/build-env-g0wj7zqz/lib/python3.11/site-packages/hatchling/metadata/core.py", line 435, in version
    raise ValueError(message)
ValueError: Metadata field `version` cannot be both statically defined and listed in field `project.dynamic`

ERROR Backend subprocess exited when trying to invoke build_sdist

A solution would be to simply remove the version static field, since the version is fetched from __init__.py dynamically.

I removed the dynamic version in 0.3.0


"Toggle pin" doesn't work in macro[BUG]

Describe the bug
I can believe it's something i'm doing, but I can't get the following macro to work:

The daily note opens fine.  However, the toggle never happens.  I tried with no delay and various delays up to 5 seconds.  The results don't change
To Reproduce
Steps to reproduce the behavior:

Create the macro as shown above
Use the command palette to execute the macro from QuickAdd

Expected behavior
I would expect that the daily note opens and then the pin is toggled.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS] : Windows 11
Browser [e.g. chrome, safari] Chrome
Version [e.g. 22] latest

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Obsidian: 1.1.16
QuickAdd 0.11.6
I believe these are both the latest versions.
Add any other context about the problem here.

Trying to help here ....
Maybe this issue is because the note is open but is not active focused before apply the PIN command.
When you execute this command from a previous note opened did you have this issue?

I understand what you are saying about whether the note has focus.  What I tried today was using the macro (which still includes the toggle pin) to open the note.  Doesn't pin the note.  Without doing anything else, I immediately did a ctrl-p (control palette) and selected the toggle pin.  That did pin the note.

Hmm... I'm using the exact same setup, but not getting any errors.

After 5 seconds, the note is pinned.
Does any other command get run after the 5 seconds, if you try inserting one? E.g. splitting panes or something, as that's very visible.

And today, it's working.  I have noticed that sometimes QuickAdd doesn't recognize changes in configuration until I restart Obsidian (at least that's my impression).  I played with it this morning some more and I think I've convinced myself that the delay (wait) is necessary and 500ms is sufficient.  Regardless, I agree that it is working.  I will close based on that.

Great to hear.
I'm working through the list of issues so any saving issues should be fixed in time. I've already fixed some of them, so looking forward to a more consistent experience soon.


Issues with SVG images in resources pages

Some of the SVG images have issues such as intersecting or invisible path (specifically mail-education.svg). This is likely to not render ok in every situation. mail-education.svg does not render fully on inkscape for example.
Besides, the academic cap icon is a pure US / UK cultural specificity. I think that people would still recognize it everywhere in Europe - but not necessarily as it is embedded like this with other content (the drawer).

Is this still an issue that you face @SylvainCorlay

Since this ticket has been inactive for five years, I'm going to close as stale. If you'd like to continue the discussion, feel free to chime in.


Create SkipLink.test.tsx

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.76%. Comparing base (9308769) to head (96a09e9).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #347      +/-   ##
==========================================
+ Coverage   97.11%   98.76%   +1.64%
==========================================
  Files          25       25
  Lines         243      243
  Branches       71       69       -2
==========================================
+ Hits          236      240       +4
+ Misses          7        3       -4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


HMR support for import() with non-constant request (implicit context)

Feature request

What is the expected behavior?
module.hot.accept(...) where the first argument is the same as the first argument given to an import() should be able to work. Currently, as when the path is not static, it is actually compiled to a trampoline module (ContextModule#getLazySource, or webpackAsyncContext by default), and as there is no way to delegate HMR to parents, it is not possible to accept hot reloads from modules with dynamic paths.
It is technically possible to rewrite it in terms of require.context by using the 4th argument, but that loses features like the magic comments and has semantics closer to require.ensure instead of import().
What is motivation or use case for adding/changing the behavior?
My use case: hot reloading of translations in development. The specific translation files loaded depend not only on the language setting, but also on the specific page's requirements (namespaces in i18next terminology), and the latter is a freeform string that corresponds to a filename.
For example, on a backend plugin that loads english:
if (module.hot) {
  module.hot.accept(`./text/en/${namespace}.yaml`, () => {
    services.backendConnector.reload('en', namespace)
  })
}
return (await import(`./text/en/${namespace}.yaml`)).default

The accepted module id is a correct module id for the reloaded module, but there is a context module in between that prevents the hot reload from being accepted.
How should this be implemented in your opinion?
I'm... not sure. I'm not familiar with HMR internals. It'd probably require that context modules get some kind of special treatment by the HMR API, or require that a general purpose API be added to module.hot that could then be called by context modules to say that they are "not part of" the hot reloading chain of the given modules.
This could probably be made slightly easier by converting the context modules to ES modules.
Are you willing to work on this yourself?
Probably not, at least not now.

Could this be solved by making all immediate children of the generated context also children of the generated context's parent?
My simple scenario:
// File: @/lang/index.js

async function fetchMessages(locale) {
  const module = await import(/*
    webpackChunkName: "lang/[request]",
    webpackExclude: /index/
  */ `@/lang/${locale}`)
  return module.default
}

if (process.env.NODE_ENV !== "production" && module.hot) {
  module.hot.accept("@/lang/en-US", () => { /* ... */ })
}

The HMR service doesn't see this callback because the file "en-US.js" is not a direct descendant of "index.js" as there is the generated async context between them.


About the problems in the update process after I added python

This issue has been moved from a ticket on Developer Community.

I waited for a long time and found that it did not move. Then I wanted to close it. Then I pressed Pause and found that it could not be closed

Original Comments
Feedback Bot on 10/25/2022, 09:32 PM:
(private comment, text removed)

Original Solutions
(no solutions)

broken image:

Doesn't seem like a PTVS issue.

Assigned internal bug to Setup


Create a generated facet for places missing coordinates (not mappable)

So, for each coordinates dimension there should now be a dimension called "[Name of coordinates dimension]: Exists". This dimension will have value true if it exists and false if not. It can be used in the facet filter or in the timeline as a grouping. You can try this on the sample data set and you will see that even there there are many missing coordinates (all because of missing places of death).

Let me know how this is looking and if we need updates or changes?

Looks like this may not be working for the Voltaire data. I'll keep this issue open for the moment.

Ok, I think we're in business now.


POST request hangs if body-parser is configured

If the body-parser module is configured with express as app.use(bodyParser.urlencoded({ extended: true })); then any POST requests handled by express-stormpath hangs.
How to reproduce

Create a new server using this gist.
Start the server.
Navigate to /register.
Sign up for a new account.

Actual
Request hangs indefinitely.
Expected
Page is redirected to /login?status=verified. If you have email verification turned on, then it should redirect to /login?status=unverified.

I've reported the same problem here (as a comment): #194


MRG: TDT delay

Taking over #280.
@rkmaddox can you see if this still does what you want?

Coverage decreased (-1.7%) to 88.981% when pulling 1e16b1d89e9aa7d577b99e4cdd1429be3e826656 on Eric89GXL:tdt-delay into c2b1f65b9deb43b43a3e14ae404126c26dae6eae on LABSN:master.

Travis failure is the one fixed in #281 so can safely be ignored. Ready for review/merge from my end.

Coverage decreased (-2.2%) to 89.083% when pulling 60ffbd539cae867e4ef79a7689328e73d97ca600 on Eric89GXL:tdt-delay into abd14356641b486961a5cb76740a208c46484b90 on LABSN:master.

FYI doc changes can be seen [here] (https://41-11614571-gh.circle-artifacts.com/0//home/ubuntu/expyfun/doc/_build/html/generated/expyfun.ExperimentController.html)

Coverage increased (+0.02%) to 91.286% when pulling efa7b75606af5abb1b7fff2dad336f8579aaf8f7 on Eric89GXL:tdt-delay into 5a013f888d94235810950c887ed34be7cd26913e on LABSN:master.

LGTM, though would like to run tests on MARLO (and also NINTOK if I can get on the schedule) before merging if that's OK.

That would be good

Any time for this one coming up @drammock?

thanks for the reminder @Eric89GXL ... I've put it on my calendar for tomorrow.

I just ran nosetests on this branch on MARLO, with both TDT and EyeLink connected: FAILED (SKIP=2, errors=4, failures=1).  Then I ran the tests on current master and got the same results.  It seems we've been getting a bit lax about testing on windows with the full complement of hardware attached.  Here are the failures:
ERROR: Test resampling
ImportError: mne-python is required to use the resample function

ERROR: Test EC with TDT
test_experiment_controller.py, line 286
_experiment_controller.py, line 602
_experiment_controller.py, line 1410
RuntimeError: Previous audio must be stopped before playing

ERROR: Test EC window_size=None and button press capture
test_experiment_controller.py, line 362
ValueError: trigger_controller can only be "tdt" if tdt is used for audio

ERROR: Test logging to file (Pyglet)
test_logging.py, line 24
ValueError: trigger_controller can only be "tdt" if tdt is used for audio

FAIL: Test setting background color
test_experiment_controller.py line, 417, AssertionError
assert_true(np.logical_or(white_mask, red_mask).all())

I'll work on these now, then PR into your PR.

I've fixed most of these errors locally on MARLO now.  But something very strange is going on with ec._playing and ec.identify_trial.  If I print the values of ec._playing and ec._trial_progress before and after this line then I get (False, 'stopped') before the line, and (True, 'identified') after the line.  Seems like either:

stopping the circuit is not changing the value in self.rpcox.GetTagVal('playing') to 0, or
stamping triggers is somehow wrongly changing that value to 1

@Eric89GXL any ideas / suggestions?  I'll try looking at the circuit later today, but would appreciate any assistance if you have time.

I can look into it on Monday

@Eric89GXL I've got the tests passing now, but please have a look at the gist I linked from Eric89GXL/expyfun#10.  It's weird behavior that we should try to track down the cause of.

Coverage decreased (-2.2%) to 89.11% when pulling 8c30d5a0b0ff5937a74b8fa236971d3488265215 on Eric89GXL:tdt-delay into 5a013f888d94235810950c887ed34be7cd26913e on LABSN:master.

Coverage decreased (-2.2%) to 89.11% when pulling 72321de3b8b3cb0d7a844d3029c54106232e54a3 on Eric89GXL:tdt-delay into 5a013f888d94235810950c887ed34be7cd26913e on LABSN:master.

Coverage decreased (-2.2%) to 89.11% when pulling 0afb5ea1c01e5a65de5f9f3a6d4cb31ed0c1ba7a on Eric89GXL:tdt-delay into 5a013f888d94235810950c887ed34be7cd26913e on LABSN:master.

@drammock unfortunately I cannot recreate this issue on my Windows install with an RM1. Can you try to replicate it on your computer's Windows VM using the RM1?
Are you all still running WinPython on the experimental computer, or something else? I can try to recreate the environment more precisely here.

Hmm.  I also cannot replicate on RM1/virtualmachine.  Will try testing again on MARLO today or monday.

If it persists there and you can't make sense of it, maybe Tuesday morning
I can remotely debug

Coverage increased (+0.02%) to 91.283% when pulling 5146c3d55951b2668f8d68d834936cde09a0420d on Eric89GXL:tdt-delay into 5a013f888d94235810950c887ed34be7cd26913e on LABSN:master.

@drammock ready for review/merge from my end

tests pass on marlo (w/ RP2) and on moctezuma...  can't test on moctezuma windows VM because nosetests is failing to find package site and I've borked my msys/gitbash install in trying to upgrade things...  Will test with nintok later today.

ping @drammock if you have time...

@Eric89GXL sorry, finally got a chance to test on NINTOK today.  I tried the auto-stop gist script, keypress_demo, sync_test, buffer_test, and simple_experiment.  Everything seems to work as expected, except simple_experiment, which failed because:

it explicitly sets response_device='tdt' in the ExperimentController instantiation line, instead of reading from the machine config file (the optical button-box presents as a keyboard to NINTOK), and

apparently ec.wait_for_presses returns a tuple even when timestamp=False, I think that was part of @ilogue's changes related to press vs release.  This breaks simple_experiment here.

Since there's now a merge conflict on this branch, do you mind making those two fixes to simple_experiment (and to the docstring of ec.wait_for_presses) while you're rebasing?

Yeah that's fine. Sounds like I borked things a bit on rebase anyway...

it explicitly sets response_device='tdt' in the ExperimentController

Actually, I don't see that in simple_experiment.py in master or this PR?

oh...  that may be my bad.  I used a copy of simple_experiment that was already on NINTOK in a previous expyfun_testing folder.

It was probably left over from the time we were trying to get button presses to go from optical to TDT to NINTOK, rather than directly from optical to NINTOK via USB.

by the way, what is the preferred method for testing expyfun changes on NINTOK?  It doesn't seem to have GitBash installed, so I can't run the usual test suite.  (in this case, it was the autostop gist script that really mattered, but in future it would be good to be able to run nosetests there).

Install git bash :)

Coverage increased (+0.04%) to 91.085% when pulling 854eb8149dcecc63ff6f4adc4343923a3e3e1860 on Eric89GXL:tdt-delay into 674d1c9532222859dc6853bca2e88c95435eee1d on LABSN:master.

@drammock comments addressed, feel free to give it another try when you have time

Coverage decreased (-0.7%) to 90.31% when pulling 55f1ac69d091fb19891bbc30850dba9f316e56ac on Eric89GXL:tdt-delay into 674d1c9532222859dc6853bca2e88c95435eee1d on LABSN:master.

Oof yeah, there were some bugs with how presses were handled that are hard to unit test (in the case of no presses, we weren't returning a consistent value). It should be fixed now.

Argh actually tests fail now. I need to actually look into what that previous PR was supposed to change in terms of outputs vs what it wasn't.

Okay so it turns out that #276 broke returns on a number of key-related functions and #266 made them work again. But this PR restores them to their former glory, and should make the outputs match what the docstrings say is actually returned for get_presses, wait_for_presses, and wait_one_press. @drammock can you confirm this is all true? Sorry it's going to be a bit of a PITA to confirm, but trust me it was harder to figure it out in the first place :)

Coverage increased (+0.3%) to 91.319% when pulling 9b9bf06d53a1cb95a94242af14443e987514cb53 on Eric89GXL:tdt-delay into 674d1c9532222859dc6853bca2e88c95435eee1d on LABSN:master.

CircleCI is angry but https://github.com/sphinx-gallery/sphinx-gallery/pull/166 should fix it

Coverage increased (+0.3%) to 91.319% when pulling ae91b87b6391ae40b6025f0c0a16e2af414e67d7 on Eric89GXL:tdt-delay into 674d1c9532222859dc6853bca2e88c95435eee1d on LABSN:master.

Coverage increased (+0.3%) to 91.319% when pulling ddcdc1e1146d0678ed1238314a81971c9650633c on Eric89GXL:tdt-delay into 674d1c9532222859dc6853bca2e88c95435eee1d on LABSN:master.

CircleCI is finally happy

plot_keypress_demo.py:
Traceback (most recent call last):
  File "plot_keypress_demo.py", line 56, in <module>
    ''.format(key, round(time, 4)) for key, time, _ in pressed]
ValueError: need more than 2 values to unpack

plot_keyrelease_demo.py
Traceback (most recent call last):
  File "plot_keyrelease_demo.py", line 61, in <module>
    ''.format(k, r, round(t, 4)) for k, t, r in events]
ValueError: need more than 2 values to unpack

Thanks @drammock should be fixed now

Coverage increased (+0.3%) to 91.319% when pulling 7ee8794744fc2f2e733397f94925783bea7f178b on Eric89GXL:tdt-delay into 674d1c9532222859dc6853bca2e88c95435eee1d on LABSN:master.

Coverage increased (+0.3%) to 91.319% when pulling 7ee8794744fc2f2e733397f94925783bea7f178b on Eric89GXL:tdt-delay into 674d1c9532222859dc6853bca2e88c95435eee1d on LABSN:master.

Thanks for the reviews @drammock


[Event Request] Table 39 - Purchase Line - OnBeforeValidateJobLineType

Dear ALAppExtensions team,
through this issue we would like to request adding event publisher OnBeforeValidateJobLineType in Table 39 Purchase Line with the following signature:
[IntegrationEvent(false, false)]
local procedure OnBeforeValidateJobLineType(var PurchaseLine: Record "Purchase Line"; var IsHandled: Boolean)
begin
end;

which would be called as follows in the OnValidate trigger of field 1002- Job Line Type:
trigger OnValidate()
// >>>> New >>>>>>>>>>>>>>>>
var
    IsHandled: Boolean;
// <<<< New <<<<<<<<<<<<<<<<
begin
    // >>>> New >>>>>>>>>>>>>>>>
    OnBeforeValidateJobLineType(Rec, IsHandled);
    if IsHandled then
        exit;
    // <<<< New <<<<<<<<<<<<<<<<

    TestField("Receipt No.", '');
    if "Document Type" = "Document Type"::Order then
        TestField("Quantity Received", 0);
    if "Job Planning Line No." <> 0 then
        Error(Text048, FieldCaption("Job Line Type"), FieldCaption("Job Planning Line No."));
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 47054.


Security Headers required for assets

Creating this task to track the changes documented in Add COEP and COEP header PR

[x] All assets loaded from a different origin must have the crossorigin attribute set. So it would be  (Achieved in PR)
[x] Set the correct CORS headers (ACAO, aka Access-Control-Allow-Origin) for content.illumidesk.com. We can set that to a wildcard * (this is already being set from S3, need to validate whether the same header is forwarded by CloudFront)
[x] All paths of content.illumidesk.com should have Cross-Origin-Resource-Policy header
[x] The paths under https://content.illumidesk.com/npm/starboard-notebook (e.g<EMAIL_ADDRESS>should additionally have the following headers set: Cross-Origin-Embedder-Policy, Cross-Origin-Opener-Policy

First Point has been resolved in Guido's PR
Points 2, 3 and 4 have been resolved using Cloudfront functions: https://console.aws.amazon.com/cloudfront/v3/home?region=us-east-1#/functions
I haven't checked in Cloudfront functions on Github, if needed I can commit their code in formgradernext or lms repos


Question: how to recover lat/lon?

HI,
Super useful tutorial.
I am struggling with something. If I do:
y = ds.tas.data    - I recover "tas" in a np array
ds["tas"].sel(lat= lat_ref, lon=lon_ref, method='nearest').plot() - I can plot tas for the lat/lon nearest to some reference values I set.
However, in the above expression I am in need to recover the nearest lat and lon xarray  (they appear in the title of the plot) and save them to a variable. I do not know how to do this ...
Thanks for any help...

Super useful tutorial.

@Silviameteoro, Thank you!
Regarding the issue you described above, I'd recommend breaking the last line into two lines:
# Step1: Sub-select data
ds_subset = ds["tas"].sel(lat= lat_ref, lon=lon_ref, method='nearest')

# Optional: If you want to, you should be able to save this subset to a netCDF file via
ds_subset.to_netcdf("my_subsetted_data.nc")

# Step 2: Plot the data
ds_subset.plot()

I hope this helps!


visualization.HTML output is empty

I installed Mirge3.0 using conda and running using the following line obtained from run.log:

/home/ubuntu/anaconda3/envs/mirge3/bin/miRge3.0 -s 1.fastq.gz,2.fastq.gz,3.fastq.gz -lib /home/ubuntu/mirge3_lib -on human -db mirbase -o mirge_out/ -gff -nmir -trf -tcf -cpu 12 -ai -a illumina -g TTAGGC...TGGAATTCTCGGGTGCCAAGGAACTCCAGT

The file mirge3_visualization.html is created, but all of the tabs are empty. The file index_data.js is created with all of the relevant data for plotting. I tried plotting the readlength chart manually in R, but the categories (read length) and data (counts/frequency) do not have equal lengths and cannot be plotted. Furthermore, annotation.report.html is generated and the summary table appears with no issues.
I'm wondering if this is an issue on my end (too few samples/low quality samples) or something else.

Hi @TomRivas,
Yes, this is the adapter issue. You should just use -a illumina and skip -g part. (Giving -g as this TTAGGC...TGGAATTCTCGGGTGCCAAGGAACTCCAGT specifies trim adapter at both ends). If you observer let-7a in your sequences, you can see there is no 5' adapter sequence and has only 3' adapter sequence, which in this specific case is illumina adapter (TGAGGTAGTAGGTTGTATAGTTTGGAATTCTCGGGTGCCAAGGAACTCCA).
let-7a sequence:  TGAGGTAGTAGGTTGTATAGTT
illumina adapter:  TGGAATTCTCGGGTGCCAAGGAACTCCA
This command will yield more unique miRNAs and less unmapped reads:
miRge3.0 -s 1.fastq.gz,2.fastq.gz,3.fastq.gz -lib /home/ubuntu/mirge3_lib -on human -db mirbase -o mirge_out/ -gff -nmir -trf -tcf -cpu 12 -ai -a illumina
This should also fix the visualization html files issue. Let me know if this helps.
Thank you,
Arun.

Hi @arunhpatil, thanks so much for your help! I've tried your suggested changes to the mirge3 command and still have an empty report. The number of unique miRNAs is unchanged and the number of reads corresponding to other RNAs are negligibly different. Furthermore, the lengths of the data to generate the Read Length Distribution plot in index_data.js are still 34 and 35, see example below.
    title: {
        text: 'Sample.1: Read Length Distribution'
    },
    chart: {
        marginRight: 80,
        zoomType: 'xy'
    },
    credits: {
        enabled: false
    },
    xAxis: {
        categories: [16.0, 17.0, 18.0, 19.0, 20.0, 21.0, 22.0, 23.0, 24.0, 25.0, 26.0, 27.0, 28.0, 29.0, 30.0, 31.0, 32.0, 33.0, 34.0, 35.0, 36.0, 37.0, 38.0, 39.0, 40.0, 41.0, 42.0, 43.0, 44.0, 45.0, 46.0, 47.0, 48.0, 49.0, 50.0]
    },
    yAxis: {
        allowDecimals: false,
        title: {
            text: 'Frequency'
        }
    },
    series: [{
        name: 'Read length',
        pointWidth: 10,
        type: 'column',
        colorByPoint: false,
        data: [58487, 65243, 76967, 87066, 74050, 73397, 81951, 92195, 80926, 64658, 56068, 49967, 43080, 39553, 42164, 33331, 24669, 22769, 16921, 15081, 15774, 11566, 9473, 9677, 5603, 3537, 3005, 1286, 659, 484, 398, 424, 0, 82331],
    }]
});```


q: ( how to disallow undefined commands/args )

my question is...
( Is there a way to disallow undefined commands/args? )
	app := &cli.App{
		Name:    "mycli",
		Usage:   "an awesome cli",
		Commands: []*cli.Command{
			{
				Name:  "serve",
				Usage: "start http server",
				Flags: []cli.Flag{
					&cli.StringFlag{
						Name:    "port",
						Aliases: []string{"p"},
						Usage:   "HTTP listen on `PORT`",
						Value:   "8181",
					},
				},
				Action: func(c *cli.Context) error {
					serve(c.String("port"))
					return nil
				},
			},
		},
	}

now if i run:
./mycli server bla bla the command will work using the default value : 8181
Is there a way to not allow adding undefined args?
Or I should run some check of the args inside cli.ActionFunc?
thank you

This library does not handle arg validation, so I would recommend using either Command.Before or Command.Action.
	app := &cli.App{
		Name:    "mycli",
		Usage:   "an awesome cli",
		Commands: []*cli.Command{
			{
				Name:  "serve",
				Usage: "start http server",
				Flags: []cli.Flag{
					&cli.StringFlag{
						Name:    "port",
						Aliases: []string{"p"},
						Usage:   "HTTP listen on `PORT`",
						Value:   "8181",
					},
				},
				Before: func(c *cli.Context) error {
					if c.Args().Present() {
						return errors.New("some helpful error message")
					}
					return nil
				},
				Action: func(c *cli.Context) error {
					serve(c.String("port"))
					return nil
				},
			},
		},
	}

Got it! Thank you very much!


Publish global CLI app to npm

This should also make it easier to install ( #23 ).
TODO:

[ ] running CLI will check the current working directory for files

[ ] gitlab-pages-config.js config file

[ ] Add YAML support for gitlab-pages-config.yml
[ ] If does not exist, walk user through creating one. Answer the following questions:

[ ] Server URL
[ ] Deploy branch
[ ] Temporary pages directory
[ ] Public Pages directory
[ ] Page to SSH public and private keys

[ ] How to create SSH key pair

[ ] publish to npm as gitlab-pages

Use https://www.npmjs.com/package/inquirer for asking questions when generating config file.


Update build.sh

/sbin/init is symlinked to /lib/systemd/systemd
Calling /sbin/init --version on Ubuntu fails, but /lib/systemd/systemd --version succeeds
Hence, changing build code to /lib/systemd/systemd

how can i disable ipv6 in the script so that it can only take ipv4 thanks

any new update for this script ? already install but wont open netflix
what should i do ?


Respnsive for all device

Describe the bug
I can make it responsive for all devices
To Reproduce
Steps to reproduce the behavior:

Go to 'css file'
Click on 'add media query'
Scroll down to 'terminal'
See error

Expected behavior
it should be responsive for all devices. I will make use of inspect for that
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [ windows ]
Browser [chrome]
Version [120.0.6099.130 ]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
please assign this issue to me under SWOC

Hi @itsmeabhishek03 @subh05sus  ! I'm interested in fixing this issue.
Please assign this issue to me.

Hey @prem-prasad1710 I am not admin of this project I am a participant like you that's why I have raised this issue you can find something else. Thank you


BUG: Module not added to package; not importable

ranger currently cannot be used from a pip install because the ranger module was not added to the package. The package is entirely empty, resulting in the following error:
$ pip install git+https://github.com/lessw2020/Ranger-Deep-Learning-Optimizer.git@73811db2eb55e1e3e3b736177cafaebe4807d669
[...]
Installing collected packages: ranger
Successfully installed ranger-0.0.1
$ python
>>> import ranger
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'ranger'
>>> from ranger import Ranger
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'ranger'
>>> import ranger.ranger
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'ranger'

This is resolved by using setuptools.findpackages to add the ranger module to the package.
I also added the README.md contents as long_description in setup.py, and incremented the version number to 0.1.dev0. Using dev or dev0 as the patch number indicates that the version is unstable, i.e. there is a one-to-many mapping from the version number, 0.1.dev0, to the state of the codebase in the repository.
I can split this out into multiple PRs if you'd prefer.

Awesome, thanks a bunch @scottclowe - much appreciated.
I haven't been using pip as I just upload ranger.py directly but I'll start using on new servers to try and keep pip up to date.
Thanks again!

Thanks for the quick response and merging this in.
It would be great if you could release ranger on pip. I realise that @mpariente has put it up there, as they mention in #25. But it isn't their package so they aren't keeping it up to date (i.e. it doesn't have the new Gradient Centralization feature). I'm sure if you ask them, @mpariente will let you take over the pytorch-ranger package identifier on PyPI.

Hi to both of you
Obviously, I'd let @lessw2020 the pytorch-ranger if he wants to upload the package to PyPI, without any problem.

Hi @scottclowe and @mpariente  - thanks to both of you.
I'm happy to upload to the pypi - if it's possible for both @mpariente and I to have access to it that's probably ideal.
For now @mpariente could you upload the latest ranger so that people can use the gc (gradient centralization) addition?  Or if it's faster give me access and I can upload/update later today?
Thanks much!

Hey @lessw2020, can you create an account on pypi please? Or give me your username if you already have one, lessw2020 didn't work.
I'll add you as the owner of the project, it'll be easier

Hi @mpariente -thanks for the update.  I've made a new account on pypi, same user name - lessw2020

I added you as the owner, sorry for the delay !
The new version needs to be above 0.1.1.
torch_optimizer depends on pytorch_ranger now, could we add typing in your repo (as done in my fork) so that pytorch_ranger still meets the requirements of torch_optimizer?
Have a nice day !

Hey @lessw2020 !
Any updates on you releasing new versions of the package?


OpenAI Compatible API

Hello,
I would like to deploy Llama 7B and serve it using Open AI compatible REST API endpoints. Are there any examples of it?
I am aware if projects such as LiteLLM and LocalAI but not sure if/how to use them with Modal.
Thanks

Hey @amir1m!
You can try wrapping a vLLM server in OpenAI-compatible mode with an asgiproxy, which is similar to what we do in our ComfyUI and Streamlit examples. There's also a working, if a bit heavy on boilerplate, version in this PR (#542).
If you have any further questions, feel free to post them in the Modal Slack!


script to plot CER from training logfile

as suggested in https://github.com/tesseract-ocr/tesstrain/issues/200#issuecomment-719579821

CER Data was extracted as follows:
grep 'Eval Char' /home/ubuntu/tess5training-iast/LAYER.log | sed -e 's/^.*[0-9]At iteration //' | \sed -e 's/,.* Eval Char error rate=/\t/'  | sed -e 's/, Word.*$//' | sed -e 's/^/\t\t/'> plot-eval.txt
grep 'best model' /home/ubuntu/tess5training-iast/LAYER.log |  sed  -e 's/^.*\///' |  sed  -e 's/\.checkpoint.*$//' | sed  -e 's/_/\t/g' | sed -e 's/\(.*\)\t\(.*\)/\1/' > plot-best.txt
grep 'At iteration' /home/ubuntu/tess5training-iast/LAYER.log |  sed -e '/^Sub/d' |  sed -e '/^Update/d' | sed  -e 's/At iteration \([0-9]*\).*char train=/\t\t\1\t\t/' |  sed  -e 's/%, word.*$//'   > plot-iteration.txt
sed 'N;s/\nAt iteration 0, stage 0, /At iteration 0, stage 0, /;P;D' /home/ubuntu/tess5training-iast/CHECKeval.test.log | grep 'Eval Char' | sed -e 's/.checkpoint.*Eval Char error rate=/\t\t\t/' | sed -e 's/, Word.*$//' | sed  -e 's/\(^.*\)_\(.*\)_\(.*\)\t/\1\t\t\2\t\t\t/g' > plot-validation.txt
cat plot-header.txt plot-validation.txt  plot-best.txt plot-eval.txt plot-iteration.txt > plot_cer.csv
python plot_cer.py

plot_cer.csv.txt

@kba I have added the info about how to run this in README.
 |tee -a LAYER.log does not capture the output from lstmtraining. I use nohup for capturing all output.
LAYER training is NOT currently supported by Makefile. I had used logfile from an independently run `replace top layer training'.
I used the sample set provided in this repo and ran training from scratch as well as with START-MODEL. Those plots look a bit different. See below.


[BUG] [chip-tool] cannot build for armhf (32bit ARM)

Reproduction steps
I tried cross build for armhf on Ubuntu 22.04 x86_64.
As a prerequisite, I added ports.ubuntu.com to /etc/apt/sources.list.d/ports_armhf.list.
$ sudo dpkg --add-architecture armhf
$ sudo apt update
$ sudo apt install crossbuild-essential-armhf
$ sudo apt install libssl-dev:armhf
$ sudo apt install libglib2.0-dev:armhf
$ cd ${MATTER_SDK}
$ source scripts/activate.sh
$ cd examples/chip-tool
$ gn gen out/debug --args='target_os="linux" target_cpu="arm"'
$ ninja -C out/debug

LOG
chip-tool-armhf-build.log

/usr/lib/x86_64-linux-gnu/glib-2.0/include/glibconfig.h

I set target_cpu="arm", but ninja accessed x86_64-linux-gnu directory.
and it maybe root cause is 64bit variable.
Can I resolve it?
Bug prevalence
always
GitHub hash of the SDK that was being used
commit 4088a77f557e8571a39338fad51a1d8eb0131d79 (HEAD, tag: v<IP_ADDRESS>)
Platform
other
Platform Version(s)
No response
Anything else?
Host OS : Ubuntu 22.04

resolve it
$ export PKG_CONFIG_PATH="/usr/lib/arm-linux-gnueabihf/pkgconfig"

Right, the glib path comes from pkg-config, and that will do what you tell it to do....

@mmt-10 , can you show the full content of ports_armhf.list and sources.list files? Thank you very much.

ubuntu@ubuntu-VM:~$ sudo apt install libssl-dev:armhf
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
libssl1.1:armhf : Depends: libc6:armhf (>= 2.28) but it is not going to be installed
E: Unable to correct problems, you have held broken packages.
ubuntu@ubuntu-VM:~$
ubuntu@ubuntu-VM:~$ sudo apt install libssl1.1:armhf
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
libssl1.1:armhf : Depends: libc6:armhf (>= 2.28) but it is not going to be installed
E: Unable to correct problems, you have held broken packages.
ubuntu@ubuntu-VM:~$
ubuntu@ubuntu-VM:~$ sudo apt install libc6:armhf
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
apt : Depends: adduser but it is not going to be installed
   Depends: gpgv or

            gpgv2 but it is not going to be installed or

            gpgv1

   Depends: libapt-pkg6.0 (>= 2.4.6) but it is not going to be installed

   Depends: libc6 (>= 2.34) but it is not installable

   Depends: libgcc-s1 (>= 3.3.1) but it is not going to be installed

   Depends: libgnutls30 (>= 3.7.0) but it is not going to be installed

   Depends: libseccomp2 (>= 2.4.2) but it is not going to be installed

   Depends: libstdc++6 (>= 11) but it is not going to be installed

   Depends: libsystemd0 but it is not going to be installed

   Recommends: ca-certificates but it is not going to be installed

base-files : PreDepends: awk
          Depends: libc6 (>= 2.34) but it is not installable

base-passwd : Depends: libc6 (>= 2.34) but it is not installable
           Depends: libdebconfclient0 (>= 0.145) but it is not going to be installed

bash : PreDepends: libc6 (>= 2.34) but it is not installable
    PreDepends: libtinfo6 (>= 6) but it is not going to be installed

bsdutils : PreDepends: libc6 (>= 2.34) but it is not installable
        PreDepends: libsystemd0 but it is not going to be installed

        Recommends: bsdextrautils

coreutils : PreDepends: libacl1 (>= 2.2.23) but it is not going to be installed
         PreDepends: libattr1 (>= 1:2.4.44) but it is not going to be installed

         PreDepends: libc6 (>= 2.34) but it is not installable

         PreDepends: libgmp10 (>= 2:6.2.1+dfsg) but it is not going to be installed

         PreDepends: libselinux1 (>= 3.1~) but it is not going to be installed

dash : PreDepends: libc6 (>= 2.34) but it is not installable
debianutils : PreDepends: libc6 (>= 2.34) but it is not installable
diffutils : PreDepends: libc6 (>= 2.34) but it is not installable
dpkg : PreDepends: libbz2-1.0 but it is not going to be installed
    PreDepends: libc6 (>= 2.34) but it is not installable

    PreDepends: liblzma5 (>= 5.2.2) but it is not going to be installed

    PreDepends: libselinux1 (>= 3.1~) but it is not going to be installed

    PreDepends: libzstd1 (>= 1.4.0) but it is not going to be installed

    PreDepends: zlib1g (>= 1:1.1.4) but it is not going to be installed

e2fsprogs : PreDepends: libblkid1 (>= 2.36) but it is not going to be installed
         PreDepends: libc6 (>= 2.34) but it is not installable

         PreDepends: libcom-err2 (>= 1.43.9) but it is not going to be installed

         PreDepends: libext2fs2 (= 1.46.5-2ubuntu1.1) but it is not going to be installed

         PreDepends: libss2 (>= 1.38) but it is not going to be installed

         PreDepends: libuuid1 (>= 2.16) but it is not going to be installed

         Depends: logsave

         Recommends: e2fsprogs-l10n but it is not going to be installed

findutils : PreDepends: libc6 (>= 2.34) but it is not installable
         PreDepends: libselinux1 (>= 3.1~) but it is not going to be installed

grep : PreDepends: libc6 (>= 2.34) but it is not installable
    PreDepends: libpcre3 but it is not going to be installed

gzip : PreDepends: libc6 (>= 2.34) but it is not installable
hostname : PreDepends: libc6 (>= 2.34) but it is not installable
init : PreDepends: systemd-sysv
libc-bin : Depends: libc6 (> 2.35) but it is not installable
        Depends: libc6 (< 2.36) but it is not installable

libc6:armhf : Depends: libgcc-s1:armhf but it is not going to be installed
           Depends: libcrypt1:armhf (>= 1:4.4.10-10ubuntu4) but it is not going to be installed

           Recommends: libidn2-0:armhf (>= 2.0.5~) but it is not going to be installed

libcrypt1 : Depends: libc6 (>= 2.25) but it is not installable
login : PreDepends: libaudit1 (>= 1:2.2.1) but it is not going to be installed
     PreDepends: libc6 (>= 2.34) but it is not installable

     PreDepends: libpam0g (>= <IP_ADDRESS>) but it is not going to be installed

     PreDepends: libpam-runtime but it is not going to be installed

     PreDepends: libpam-modules but it is not going to be installed

ncurses-bin : PreDepends: libc6 (>= 2.34) but it is not installable
           PreDepends: libtinfo6 (>= 6.3) but it is not going to be installed

perl-base : PreDepends: libc6 (>= 2.35) but it is not installable
sed : PreDepends: libacl1 (>= 2.2.23) but it is not going to be installed
   PreDepends: libc6 (>= 2.34) but it is not installable

   PreDepends: libselinux1 (>= 3.1~) but it is not going to be installed

shim-signed : Depends: grub-efi-amd64-signed but it is not going to be installed or
                    grub-efi-arm64-signed but it is not installable

           Depends: grub2-common (>= 2.04-1ubuntu24)

           Depends: mokutil (>= 0.3.0+1538710437.fb6250f-0ubuntu2) but it is not going to be installed

           Depends: sbsigntool

           Recommends: secureboot-db but it is not going to be installed

sysvinit-utils : Depends: libc6 (>= 2.34) but it is not installable
tar : PreDepends: libacl1 (>= 2.2.23) but it is not going to be installed
   PreDepends: libc6 (>= 2.34) but it is not installable

   PreDepends: libselinux1 (>= 3.1~) but it is not going to be installed

util-linux : PreDepends: libaudit1 (>= 1:2.2.1) but it is not going to be installed
          PreDepends: libblkid1 (>= 2.37.2) but it is not going to be installed

          PreDepends: libc6 (>= 2.34) but it is not installable

          PreDepends: libcap-ng0 (>= 0.7.9) but it is not going to be installed

          PreDepends: libmount1 (>= 2.37.2) but it is not going to be installed

          PreDepends: libpam0g (>= <IP_ADDRESS>) but it is not going to be installed

          PreDepends: libselinux1 (>= 3.1~) but it is not going to be installed

          PreDepends: libsmartcols1 (>= 2.34) but it is not going to be installed

          PreDepends: libsystemd0 but it is not going to be installed

          PreDepends: libtinfo6 (>= 6) but it is not going to be installed

          PreDepends: libudev1 (>= 183) but it is not going to be installed

          PreDepends: libuuid1 (>= 2.16) but it is not going to be installed

          PreDepends: zlib1g (>= 1:1.1.4) but it is not going to be installed

E: Error, pkgProblemResolver::Resolve generated breaks, this may be caused by held packages.
ubuntu@ubuntu-VM:~$

I'm sorry, I had vacation.
Please refer to below quote.
ports_armhf.list is here.
However, I gave up on cross-build.
because, I cannot resolve the library(glibc3.0 and etc) dependencies for the target board.
If you can resolve the library dependencies, please let me know.

deb [arch=armhf] http://ports.ubuntu.com/ubuntu-ports jammy main restricted universe multiverse
deb-src [arch=armhf] http://ports.ubuntu.com/ubuntu-ports jammy main restricted universe multiverse
deb [arch=armhf] http://ports.ubuntu.com/ubuntu-ports jammy-updates main restricted universe multiverse
deb-src [arch=armhf] http://ports.ubuntu.com/ubuntu-ports jammy-updates main restricted universe multiverse
deb [arch=armhf] http://ports.ubuntu.com/ubuntu-ports jammy-backports main restricted universe multiverse
deb-src [arch=armhf] http://ports.ubuntu.com/ubuntu-ports jammy-backports main restricted universe multiverse
deb [arch=armhf] http://ports.ubuntu.com/ubuntu-ports jammy-security main restricted universe multiverse
deb-src [arch=armhf] http://ports.ubuntu.com/ubuntu-ports jammy-security main restricted universe multiverse


Create issue template for Release process

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1473
Originally created by @bencodezen on 2019-12-16 16:48:23

View 5 previous comments from the original issue on GitLab