Optimum OpenVINO module

Transfer of https://github.com/dkurt/optimum-openvino to OpenVINO org.
Ticket: 79644
Dependencies:

Required (pip)

transformers (pip)
openvino-dev (pip)
nncf (pip)

Test dependencies

tensorflow-cpu==2.7.0 (pip)
datasets (pip)
soundfile (pip)
librosa (pip)
seqeval (pip)
torchaudio (pip)
torch==1.9.1 (pip)
libsndfile1 (apt-get, Ubuntu)

Thank, @helena-intel! Resolved all comments. POT integration we can make after this PR merge.

Let me merge it as release branch for 2022.1 has been already created


[LSMC] Add chirurgical mask & doctor's coats

Is there an existing issue for this?

[X] I have searched the existing issues

In which category does your improvement fall?
LSMC
Explain the current situation that you do not think is right.
Currently there's no chirurgical mask nor doctor's coats, which could be used to improve immersion and roleplay possibilities.
How is the improvement you propose?
The addition of 3 clothes items:

The (chirurgical) mask: can be used as a separate item, that can protect the wearer from small diseases for an amount of time. After this, it will be converted as an used chirurgical mask that won't protect anymore. Could be recycled by Bluebird, could be produced by PAWL (from papers)
The white lab coat: used for the doctor's that are working inside the LSMC, should be used outside the hospital and will help immersion with the stethoscope. The coat is compatible with the current winter outfit and would replace the jacket.
The chirurgical coat: used when operating a patient, will help immersion, even more with the chirurgical mask (might not be an item for this outfit)

Pull request could be done easily if this issue is accepted.

Ola ! Why not! Are the clothes you propose native to GTA or are they custom clothes?

both custom clothes


RIP

https://creators.vrchat.com/releases/#changes-in-372-beta2
cough time to look to the alternative way cough

might mark the end of this project rip...

This should pretty much only affect scripting I believe, Oct 3rd dev blog speaks in more depth about the protections themselves and upstream explicitly mentions that it's difficult to protect the bundle itself as a whole.
And in all honesty, I think this is probably for the best because it takes a big nuanced argument off the table that AGB can enable abuse per script injection.
They say they did add a CRC32 check for beta2 though, which means I'll need to cook up something for that.

I haven't verified this but my friend told me how it works is they calculate a world bundle hash on the server and that they compare it to the local cached bundle hash thats computed on your PC before the world loads.


Failed to access code serve-web with token on browser

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:
版本: 1.85.2
提交: 8b3775030ed1a69b13e4f4c628c612102e30a681
OS Version:  Fedora 39 Sway
Browser: Firefox and Edge

Steps to Reproduce:

execute: code serve-web --host <IP_ADDRESS> --port 3333 --accept-server-license-terms
Access http://<IP_ADDRESS>:3333?tkn=b6f06646-38dc-4cdb-a39b-44bbd791d350 in browser
Error:

If I launch code with --without-connection-token, vscode was launch normally on the browser.

Please see the notes https://github.com/microsoft/vscode-docs/blob/main/remote-release-notes/v1_82.md#serve-web-cli-command

Note that in order to access the server on non-localhost origins, you will need to serve the UI over TLS using a reverse proxy server such as Caddy.


Event Submission API

This is sweet. I didn't see any stuff for submitting events via the event API. Did I miss it? If not, would you be open to a pull request that adds it? They actually support doing it from ruby and it generally Just Works™ if you use the same user agent and remote ip.
Sorry for opening an issue but I didn't see discussions turned on. Feel free to close this either way.

hey! Thanks a lot for giving this gem a try.
The initial objective was to consume their reporting API, but I'd be more than open to expand the goal. Please feel welcome to add new functionality :)
Cheers!


Fix UX for incompatible plugins

Fixes #811
Fixes #815
Makes it easier for people to understand why those are disabled
Sorts them to bottom and adds a message

Has this been released in the latest version? I am still seeing a bunch of these with a fresh install.

Hey @megaroeny , can you check the Kap version you are running on? Did you download the fresh install from the website? Currently the website is pointing to the previous version of Kap. We are working on fixing it. In the meantime you can get the latest release here: https://github.com/wulkano/kap/releases/tag/v3.2.0

Ah okay yes I did! Thank you! No worries then. 🙂


More specific types for meta.type

Use case
Use case will be for handling formatting of the data value from a response, when meta is present on an app.
Describe the solution you'd like
https://github.com/ClickHouse/clickhouse-js/blob/main/packages/client-common/src/clickhouse_types.ts#L8
This line ^ could be something more like the following
/** @see https://clickhouse.com/docs/en/sql-reference/data-types */
const DATA_TYPES = [
    'UInt8', 'UInt16', 'UInt32', 'UInt64', 'UInt128', 'UInt256', 'Int8', 'Int16', 'Int32', 'Int64', 'Int128', 'Int256',
    'Float32', 'Float64',
    'Boolean',
    'String', 'FixedString',
    'Date', 'Date32',
    'DateTime', 'DateTime64',
    'JSON',
    'UUID',
    'Enum', 'LowCardinality',
    'Array',
    'Map',
    'SimpleAggregateFunction', 'AggregateFunction',
    'Nested',
    'Tuple',
    'Nullable',
    'IPv4', 'IPv6',
    'Point', 'Ring', 'Polygon', 'MultiPolygon',
    'Expression', 'Set', 'Nothing', 'Interval'
] as const;
type DataType = typeof DATA_TYPES[number];

// ...
    meta?: Array<{
        name: string;
        type: DataType;
    }>;

Describe the alternatives you've considered
This type could be an enum or string literal. I'd have no real preference.
Additional context
What would be nice for me as a consumer of @clickhouse/client is to be able to treat the library as a comprehensive source of truth for what data types I should expect to be able to format. This way, I can format them accordingly in my apps (especially if new data types are added later on or renamed, so I'd see the type change).

Array, Tuple, Map, DateTime, etc, are a bit tricky in this case cause:
➜  ~ curl  --data-binary \
     "SELECT tuple(32, 'foo') AS x FORMAT JSON;" \
     http://localhost:8123
{
	"meta":
	[
		{
			"name": "x",
			"type": "Tuple(UInt8, String)"
		}
	],

	"data":
	[
		{
			"x": [32,"foo"]
		}
	],

	"rows": 1,

	"statistics":
	{
		"elapsed": 0.000961292,
		"rows_read": 1,
		"bytes_read": 1
	}
}

So, the proposed literal won't match the actual type in certain cases.
Another approach is to provide a helper of sorts, possibly utilizing the code from the RowBinary branch (the branch is currently on hold, but the column types parser was ready, just missing some tests). It's the exact same scenario here — we need to parse the appropriate types (preferably with nested types for correctness).
export function parseMetaColumnTypes(
  meta: Array<{ name: string, type: string> }>
): ParsedColumnType

CC @mshustov - I'm curious to hear your opinion as well.

Another approach is to provide a helper of sorts, possibly utilizing the code from the RowBinary branch

@slvrtrn I'm not sure the RowBinary parser would work with the type definition. @tylerlaws0n would @slvrtrn's proposal solve your problem?

I'm not sure the RowBinary parser would work with the type definition.

Just to clarify: the idea was that the meta.type TS hint would stay as is, i.e., just string, cause we can't type every variant, such as Tuple(String, String) etc.
However, we can provide a helper function that will produce some (probably useful) data structures out of the meta type in runtime.
For example:
console.log('### Simple types')
console.dir(parseColumnType('UInt32'))

console.log('### Enums')
console.dir(parseColumnType("Enum8('a' = 1, 'b' = 2)"), {
  depth: 1000,
})

console.log('### Nested types')
console.dir(parseColumnType('Map(String, Array(Tuple(UInt8, String)))'), {
  depth: 1000,
})

### Simple types
{ type: 'Simple', columnType: 'UInt32', sourceType: 'UInt32' }
### Enums
{
  type: 'Enum',
  values: Map(2) { 1 => 'a', 2 => 'b' },
  intSize: 8,
  sourceType: "Enum8('a' = 1, 'b' = 2)"
}
### Nested types
{
  type: 'Map',
  key: { type: 'Simple', columnType: 'String', sourceType: 'String' },
  value: {
    type: 'Array',
    value: {
      type: 'Tuple',
      elements: [
        { type: 'Simple', columnType: 'UInt8', sourceType: 'UInt8' },
        { type: 'Simple', columnType: 'String', sourceType: 'String' }
      ],
      sourceType: 'Tuple(UInt8, String)'
    },
    dimensions: 1,
    sourceType: 'Array(Tuple(UInt8, String))'
  },
  sourceType: 'Map(String, Array(Tuple(UInt8, String)))'
}

... @tylerlaws0n would @slvrtrn's proposal solve your problem?

Yes! This would work and be very helpful.

Just to clarify: the idea was that the meta.type TS hint would stay as is, i.e., just string, cause we can't type every variant, such as Tuple(String, String) etc.

These could actually be typed out via template string literals
In the case of Tuple it could be something like this:
type TupleMember = 'String' | 'Number'; // This is just for example and would clearly need many more members
type Tuple = `Tuple(${TupleMember}, ${TupleMember})`;

I think the helper proposed by @slvrtrn is actually more helpful and easier to implement/maintain than templating out the strings for every possibility. However, I think it is worth sharing this pattern still in case there is ever value to be had from generating the strings of all possible meta types.

If a middle ground is desirable; you could type out just the outer part of the type and use string inside:
type Tuple = `Tuple(${string}, ${string})`;

const aTuple: Tuple = 'Tuple(Foo, Bar)';

yeah, that looks good, thanks!


Confluence Auth using Personal Access Token not supported

Problem
My org uses Confluence Enterprise, and I followed the instructions in their docs to create a Personal Access Token (here - https://confluence.atlassian.com/enterprise/using-personal-access-tokens-1026032365.html#UsingPersonalAccessTokens-UsingPATs). When I test it out using CURL as they suggest, it works fine, but it doesn't work with this library.
Cause
Digging into it I see that the reason is because it is supposed to be used in a header like this (in Python requests) {'Authorization': 'Bearer <yourToken>'}.
As a header
Per the docs, I added these extra headers to my conf.py
confluence_publish_headers = {'Authorization': 'Bearer <yourToken>'}

https://github.com/sphinx-contrib/confluencebuilder/blob/e3f227e69bd83551546048a5212c0d50d4dc2e5c/sphinxcontrib/confluencebuilder/rest.py#L72-L74
That works - if you skip the auth. I tried it like this:
import requests
base_url = 'REDACTED'
url = base_url + '/rest/api/content/123456789012'
session = requests.Session()
session.headers.update({
    'User-Agent': 'Sphinx Confluence Builder',
    'X-Atlassian-Token': 'no-check',
})
token = 'REDACTED'
confluence_publish_headers = {'Authorization': f'Bearer {token}'}
session.headers.update(confluence_publish_headers)
print(session.get(url).text)

The auth
BUT - the problem is, you MUST provide the auth, either as a username and password, or as a custom class. Passing an empty dict works, but this library will discard it because it evaluates to False
https://github.com/sphinx-contrib/confluencebuilder/blob/e3f227e69bd83551546048a5212c0d50d4dc2e5c/sphinxcontrib/confluencebuilder/rest.py#L95-L101
Summary
So, in summary, I cannot use the token in this library as it is intended to be used by Confluence.

... you MUST provide the auth ...

I am a bit confused by this statement. Why is there a requirement for the auth attribute to be set? Or are you saying there is a situation where you want to tailor the auth entry but it does not work when needing to use PATs as well?
From the recent discussion in #611, yes, the manipulation of the confluence_publish_headers configuration (or even using confluence_request_session_override) is required to take advantage of PATs. Ideally, we will have a more graceful option to configure this in the next release. But I assume you may be talking about a different issue here?

Note that there may be a chance to have a v1.8 made this upcoming weekend (tentative). If your interaction with a Confluence instance when using PATs can be resolved from the changes outlined in the related issue, great; however, if you are observing another issue or something else being overlooked, please let me know. This could allow additional corrections to be applied before the next release; otherwise it may be awhile before a following release is made.

Thanks for all your work and comments, I appreciate it. Regarding

I am a bit confused by this statement. Why is there a requirement for the auth attribute to be set? Or are you saying there is a situation where you want to tailor the auth entry but it does not work when needing to use PATs as well?

What I meant was that in this code,
https://github.com/sphinx-contrib/confluencebuilder/blob/e3f227e69bd83551546048a5212c0d50d4dc2e5c/sphinxcontrib/confluencebuilder/rest.py#L95-L101
It requires either confluence_server_user to be set (which we don't use when using a PAT), or a custom confluence_server_auth which also won't work.

Ah ~ ok. I will assume this is a duplicate. Apologies though ~ the "proper" use of PATs were not really intuitive with the existing stable release (v1.7.1). Hopefully, with the recently added configuration and documentation changes available next release (v1.8), configuring to use PATs will much more explicit on how to use.
We can leave this issue opened (if desired), until the release is made. And if there are other concerns related to PATs which have been overlooked, please feel free to mention so.

With the recently made v1.8 release (available on PyPI), confluence_publish_token should now provide a graceful way to configure for PATs. Marking as closed.


referrals are being added to database without creating user

Solution proposal: Need to add transactions

issue is fixed in main


Remove gRPC create operations

What this PR does:
Remove dedicated CreateXxx operations from the gRPC API. They are somewhat redundant, since CREATE statements can be constructed with the query builder, and passed to the generic ExecuteQuery.
Keeping our API lean is beneficial:

less code to maintain
less divergence from the v1 codebase

easier merges (the last one was particularly bad, specifically because of these two operations)
extracting a separate "internal" API will be easier if the code is less entangled

Side note: once SchemaCache becomes the primary way to access schema metadata, we could also remove describe operations.
Which issue(s) this PR fixes:
N/A
Checklist

[x] Changes manually tested
[x] Automated Tests added/updated

Assuming tests in RestApiv2Test still pass should be good. I think removal of these methods added earlier in v2 has been discussed and agreed upon.


Double option support for columns that already have a serde annotation

We replace the deserialize_with option with our own, as we need to distinguish between the null and not set cases. But I think we should do something a little bit smarter here. Ideally the only thing we want to identify is the not set case, and then from that point onward deserialization should happen the same as the user specified.
Also we currently don't support it when you do not derive deserialization, if you have your own implementation of deserialize you are out of luck, I wonder if we could do something here as well.

I think what we will have to do here is not allow custom Deserialize or Serialize implementations on your structs, instead, the macro will generate the Deserialize and Serialize derives for you. We could then always capture and modify any #[serde] attributes on fields.
I'm still wondering whether we should allow the full #[serde] syntax, or if we should provide our own syntax that makes sure that things like field renames work across both the form and json requests. Feels better to not allow any #[serde] annotations at all and let serde be an implementation detail.


Update for Chalk platform

Everything should now look a lot better when run on Chalk!

Mostly comments about updates from custom-defined methods and macros to built-in macros/APIs, and a few places where grect_inset might work better than doing math for generated grects

Comments on the handling of the menu cell height across platforms/SDK apply to all uses, though I did not explicitly point them all out. If you're looking for some inspiration, you may want to ask Heiko for additional ideas/guidance.

If he has time =P

Thanks again. He will likely not have any time.
I want to know how best to label the values for GEdgeInsets. Do we prefer this:
const int main_text_y_inset = 30;
pin_window->main_text = text_layer_create(grect_inset(bounds, GEdgeInsets(main_text_y_margin, 0, 0, 0)));

Or this:
#define MAIN_TEXT_Y_INSET 30

pin_window->main_text = text_layer_create(grect_inset(bounds, GEdgeInsets(MAIN_TEXT_Y_INSET, 0, 0, 0)));

If only used in one spot, our FW coding style guide says: const type lower_case = value;
In case of GEdgeInsets with just one value it might be more expressive to do
const GEdgeInsets main_text_inset = {.top = 30};
grect_inset(bounds, main_text_inset);

Thanks @HBehrens!

@cjwilliams How's this as an alternative to the awkward get_cell_height_callback situation?
static int16_t get_cell_height_callback(struct MenuLayer *menu_layer, MenuIndex *cell_index, void *context) {
  return PBL_IF_ROUND_ELSE(menu_layer_is_index_selected(menu_layer, cell_index) ?
                             MENU_CELL_ROUND_FOCUSED_SHORT_CELL_HEIGHT :
                             MENU_CELL_ROUND_UNFOCUSED_TALL_CELL_HEIGHT,
                           MENU_CELL_BASIC_CELL_HEIGHT);
}

Another alternative that is slightly more readable is this:
static int16_t get_cell_height_callback(struct MenuLayer *menu_layer, MenuIndex *cell_index, void *context) {
#if defined(PBL_ROUND)
  return menu_layer_is_index_selected(menu_layer, cell_index) ?
           MENU_CELL_ROUND_FOCUSED_SHORT_CELL_HEIGHT : MENU_CELL_ROUND_UNFOCUSED_TALL_CELL_HEIGHT;
#else
  return MENU_CELL_BASIC_CELL_HEIGHT;
#endif
}

+1


Web-Client sample doesn't work (404 on everything)

Hello, i've deploy the web-client project with Glassfish but no path that is in the project works (/ai, /client-web, /webhook) and give me 404 error.
What have I to do?

I've fix that


CTRL-S to save text also pops open a Save As dialog box

Would it be possible to make the save hotkey configurable?

I noticed ALT-S works without the dialog box, closing.

This is the second time this has been reported. As I reread this I gather that your complaint is that the dialog box opens, not that ALT-S is an inappropriate key for saving edits. It would be good to know if there are user-interface guidelines for your platform. Do you know? We can summarize here:

Mac: Command-S
Windows: Alt-S ??
Linux: Alt-S ??

Yes, I was on Ubuntu 10.10 when this happened. CTRL-S saves and opens a "Save as" dialog, while ALT-S just saves it.
I agree, the above breakdown for each OS would be useful for new users.

I believe that CTRL-S was not processed by federated wiki. Rather CTRL-S was processed by your browser to save the browser page. It only appeared that federated wiki was processing CTRL-S because the textarea box lost focus when the browser save dialog came up. Saving your text on loss of focus is a feature of federated wiki.
I've revised the online documentation to make it clear that Linux users should use ALT-S.
http://fed.wiki.org/view/add-text/view/command-s


Sanitize for all item in array

Hi
I want to define filter which given array and sanitize all item in array ('*'    =>  'trim|escape|capitalize').
Please guide me
Thanks,

Latest version I believe should have this enhancement.


Add shaven <=> html cli

Use https://www.npmjs.com/package/parse5

This may save you some time: https://www.npmjs.com/search?q=jsonml

Actually It's already implemented: https://github.com/adius/html2shaven

Oh, actually I think this issues was about providing a cli


Cannot import big slashing protection

Internal server error. Error: Maximum call stack size exceeded
Running with --keymanager.bodyLimit<PHONE_NUMBER>0
28M file

Thanks for reporting @caevv, I will look into this

@caevv I am not able to reproduce the issue, even with a bigger slashing protection file it seems to work fine for me. Would it be possible for you to provide some logs of your validator client? The error log should contain the stack trace which would be really helpful to further analyse this issue.
You can observe the logs by running ./ethd logs -f validator or you could just search for the specific log by running ./ethd logs validator | grep "Maximum call stack size exceeded".

Sorry for the delay, here it is:
eth-docker-validator-1  | May-03 19:45:18.833[]                 info: Verified connected beacon node and validator have the same genesisValidatorRoot
eth-docker-validator-1  | May-03 19:45:18.835[]                 info: doppelganger protection enabled detectionEpochs=1
eth-docker-validator-1  | May-03 19:45:18.872[]                 info: Started REST API server address=http://<IP_ADDRESS>:7500
eth-docker-validator-1  | May-03 19:45:18.872[]                 warn: REST API server is exposed, ensure untrusted traffic cannot reach this API
eth-docker-validator-1  | May-03 19:45:18.872[]                 info: REST api server keymanager bearer access token located at:
eth-docker-validator-1  |
eth-docker-validator-1  | /var/lib/lodestar/validators/validator-db/api-token.txt
eth-docker-validator-1  |
eth-docker-validator-1  | May-03 19:57:53.054[]                error: Req req-1 importKeystores error  Maximum call stack size exceeded
eth-docker-validator-1  | RangeError: Maximum call stack size exceeded
eth-docker-validator-1  |     at minEpoch (file:///usr/app/packages/validator/src/slashingProtection/utils.ts:31:35)
eth-docker-validator-1  |     at SlashingProtectionAttestationService.importAttestations (file:///usr/app/packages/validator/src/slashingProtection/attestation/index.ts:148:28)
eth-docker-validator-1  |     at processTicksAndRejections (node:internal/process/task_queues:95:5)
eth-docker-validator-1  |     at SlashingProtection.importInterchange (file:///usr/app/packages/validator/src/slashingProtection/index.ts:64:7)
eth-docker-validator-1  |     at KeymanagerApi.importKeystores (file:///usr/app/packages/cli/src/cmds/validator/keymanager/impl.ts:123:7)
eth-docker-validator-1  |     at Object.handler (file:///usr/app/packages/api/src/utils/server/genericJsonServer.ts:44:23)

❯ ls -la ~/eth-docker/.eth/validator_keys/
total 28544
drwxr-xr-x 2 a a       75 May  3 19:44 .
drwxr-xr-x 4 a a       58 Apr 19 21:17 ..
-rw-r--r-- 1 a a      786 May  3 19:44 keystore-0.json
-rwxr-xr-- 1 a a 29223063 May  3 19:44 slashing_protection.json

Thanks @caevv for providing the logs, that helped a lot, a fix is on the way!

@caevv The issue should be fixed in chainsafe/lodestar:next release and will be included in v1.9.0 which will be released in ~2-3 weeks
Using eth-docker you can test if it works by updating LS_DOCKER_TAG in your .env file
LS_DOCKER_TAG=next


ci: test windows orb fix

What does this PR do?
don't merge it.  It's QA for https://github.com/forcedotcom/npm-release-management-orb/pull/92
What issues does this PR fix or reference?

closed (primary PR merged)


Where can I find a pre-training model for r50_dcn_fcos3d,did you use the original r50 for tiny?

Where can I find a pre-training model for r50_dcn_fcos3d?
Did you use the original r50 for tiny, or pre-trained in fcos3d?

May I ask where the pre training for the Tiny version is located


Update "st2 apikey load" to update an existing entry if one already exists

This pull request fixes a bug / non-consistent behavior reported in #3748.
It updates st2 apikey load command so it doesn't throw an updates an existing entry if input file contains id attribute for an item and item already exists on the server.
This way behavior is also consistent with st2 key load command.
See https://gist.github.com/Kami/bc7567c6b2b8a62f7680f64921cbf7fc for example output.
Resolves #3748

It's also worth noting that API operations exposed for /apikeys and /keys API endpoints are not the same.
datastore exposes get, set, delete while api key one supports get, create, update, delete so the actual CLI code is not 1:1 mapping and we can't simply just call update as we can do in datastore case.

Merged.
Will work on st2tests tests separately since there is really no other way to unit test this.


Update FARM Submodule for EOY 2023

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
Continues #1114
What are the significant changes in functionality due to this change request?
Update FARM Submodule for EOY 2023

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[ ] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

@dylanjm @wangcj05 Happy new year Gents.
I would like to update the FARM submodule to reflect the changes we made at the end of 2023.
This version of FARM works with the recently updated HERON (via #2235).
Could you activate the automatic checks for this PR?
Thanks,
Haoyu

Job Test qsubs sawtooth on bb0cc26 : invalidated by @joshua-cogliati-inl
Failed with: DirectoryNotACondaEnvironmentError: The target directory exists, but it is not a conda environment.

@wanghy-anl Some tests in FARM and HERON  are failed on Windows machine.

@wanghy-anl Some tests in FARM and HERON are failed on Windows machine.

Congjian, could you share the Windows test error message? Everything works on my Windows 10 machine...
BTW, I saw some tests failed on mac. Could you also share the associated error message?
Thanks.

@wanghy-anl Sorry for the late response, I was on travel. Please check the attache file for the plugin tests on Windows machine.
07_Test_Plugins.txt

@wanghy-anl For Mac machine test, the failures are not related to FARM.

@wanghy-anl Sorry for the late response, I was on travel. Please check the attache file for the plugin tests on Windows machine. 07_Test_Plugins.txt

Congjian, Looking at its wall time, it seems to be an time out error. To further investigate, could you share the following file?
\raven\plugins\FARM\tests\heron_validator\FARM_MIMO_selfLearning_fmuUpdate\Sweep_Runs_o\sweep\1\out~inner
Thanks, Haoyu

@wanghy-anl Sorry for the late response, I was on travel. Please check the attache file for the plugin tests on Windows machine. 07_Test_Plugins.txt

Congjian, Looking at its wall time, it seems to be an time out error. To further investigate, could you share the following file? \raven\plugins\FARM\tests\heron_validator\FARM_MIMO_selfLearning_fmuUpdate\Sweep_Runs_o\sweep\1\out~inner Thanks, Haoyu

In this case, you may need to change the "max_time=1200" in your "tests" file, or you need to find a way to accelerate the test.

In addition, there are also two failed HERON tests, are these related to your changes?

@wanghy-anl Sorry for the late response, I was on travel. Please check the attache file for the plugin tests on Windows machine. 07_Test_Plugins.txt

Congjian, Looking at its wall time, it seems to be an time out error. To further investigate, could you share the following file? \raven\plugins\FARM\tests\heron_validator\FARM_MIMO_selfLearning_fmuUpdate\Sweep_Runs_o\sweep\1\out~inner Thanks, Haoyu

In this case, you may need to change the "max_time=1200" in your "tests" file, or you need to find a way to accelerate the test.

I can change the max_time entry, but I am wondering why these tests took almost twice time to finish on the "Mingw" test machine. -- For instance, the HERONvalidator_FARM_MIMO_selfLearning_noUpdate_WindowsOnly took 468.82 secs on a single 2.7GHz core on my machine, but it took 881.16 secs on the "Mingw" test machine.
I saw a line /opt/build_root/raven/: ./run_tests -j 4 -l 4 --plugins in the text file you sent. What does the argument "-l 4" stands for? I have never used this argument before.

In addition, there are also two failed HERON tests, are these related to your changes?

I don't think so. All the HERON tests can pass on my windows machine. I am using the current submodule version of HERON, commit bec6cb76 on Dec 18.

@wangcj05 Congjian, Let me know if this new commit passed all the tests. Thanks. Haoyu

@joshua-cogliati-inl Thank you Joshua for merging. Best, Haoyu


x/wasm hooks

For a couple of months I've been thinking about a cosmwasm system where there can be namespaces: ability to upload codes to a namespace where they can interact each other but also open permissions to other namespaces you can think of like linux namespaces and a firewall.
First I thought just playing around with a fork but then I think that this could also be achieved if another module had access to hooks like
BeforeInstantiate()
BeforeExecute()
BeforeStore()
BeforeMigrate()

then anx/namespace module can just subscribe to hooks and apply whatever logic.
This is just an idea but not sure if others would find this interesting our useful

Thanks for bringing this topic up. I am happy to see new ideas and use cases rising.
We could replace the keeper instance with an interface in msg_server so that you can play around without the need for a fork for now.

Thanks for bringing this topic up. I am happy to see new ideas and use cases rising. We could replace the keeper instance with an interface in msg_server so that you can play around without the need for a fork for now.

Are intra contract calls routed through msg_server? If so that would help  me with exactly what I want

Are intra contract calls routed through msg_server?

yes, you can find all the message handling in https://github.com/CosmWasm/wasmd/blob/v0.44.0/x/wasm/keeper/handler_plugin.go#L54
The wasm keeper has private functions to not mess with the authorization system but we could provide some adapter here to help you out


in plugin.xml, fix the id attribute for evothings/cordova-ble (change it from com.evothings.ble to cordova-plugin-ble)

Many thanks for this contribution, sorry it took so long  before merging!


Rename module and repo

As described in #130

purrrfect LGTM
ping @hoodiehq/maintainers for 2nd review :eyes:

Rad @finnp! LGTM! :+1:


feature: Handling of ASPA

I have added a trust anchor with ASPA objects to my routinator instance. Afterwards I realised that there is no (optional) support for ASPA in routinator at the moment.
Based on that I have two feature requests:
[ ] Track unsupported, but upcoming object types in another state than type="other", state="invalid" in metrics (e.g. type="aspa", state="unsupported".
As a user this allows me to monitor for invalid objects in my repository, even when routinator does not support them yet.
[ ] Add ASPA support for object validation.
As a CA developer this would help me ensure that make sure that our implementation is what is expected by routinator.
[ ] Draft 8210bis ASPA support in RTR
There is a major gap in that there is no specification for the JSON, but for me the first two steps already have value - because right now I need to remove an alert for invalid objects.
# HELP routinator_ta_objects_total VRPs per trust anchor
# TYPE routinator_ta_objects_total gauge
routinator_ta_objects_total{name="localhost", type="manifest", state="valid"} 34113
routinator_ta_objects_total{name="localhost", type="manifest", state="invalid"} 0
routinator_ta_objects_total{name="localhost", type="manifest", state="premature"} 0
routinator_ta_objects_total{name="localhost", type="manifest", state="stale"} 0
routinator_ta_objects_total{name="localhost", type="manifest", state="missing"} 556
routinator_ta_objects_total{name="localhost", type="crl", state="valid"} 34113
routinator_ta_objects_total{name="localhost", type="crl", state="invalid"} 0
routinator_ta_objects_total{name="localhost", type="crl", state="stale"} 0
routinator_ta_objects_total{name="localhost", type="crl", state="stray"} 0
routinator_ta_objects_total{name="localhost", type="ca_cert", state="valid"} 34668
routinator_ta_objects_total{name="localhost", type="router_cert", state="valid"} 0
routinator_ta_objects_total{name="localhost", type="cert", state="invalid"} 0
routinator_ta_objects_total{name="localhost", type="roa", state="valid"} 23817
routinator_ta_objects_total{name="localhost", type="roa", state="invalid"} 0
routinator_ta_objects_total{name="localhost", type="gbr", state="valid"} 0
routinator_ta_objects_total{name="localhost", type="gbr", state="invalid"} 0
routinator_ta_objects_total{name="localhost", type="other", state="invalid"} 60

We have postponed ASPA support primarily because there was still discussions about changes to the ASN.1 specification in SIDROPS. Having Routinator versions with two different ASPA implementations seemed a cause for trouble down the line.
With a new version of the draft now published with the same specification, I assume that those discussions have concluded and we can now indeed implement ASPA in the next version of Routinator.

I do understand not implementing ASPA at the moment because there still is a chance of the profile changing.
Handling it differently in metrics would be a good improvement from my point of view though. Right now I probably should remove type="other" from the alerts we have.
routinator_repository_objects_total{group="pilot", instance="testbed-1.rpki.ripe.net:routinator_pilot", job="validator_testbed_routinator_pilot", state="invalid", support="office-hours", type="other", uri="https://localcert.ripe.net/rrdp/notification.xml"}

For other readers, of course you can filter out this alert, and accept the risk that you miss other unknown files that appear in your repo:
routinator_repository_objects_total{uri="https://my.repo.example.org/notification.xml", state!="valid"} > 0
unless
routinator_repository_objects_total{uri="https://my.repo.example.org/notification.xml", type="other", state="invalid"}

One thing we could do is drop "other" altogether and instead add separate metrics for each file extension.

That would work, but could cause issues (metric explosion) when someone adds a lot of random extensions on a manifest.
Using other for stuff not on the iana registry feels safe to me

Logging about ASPA objects is causing a significant amount of log volume for us. Could you consider adjusting this warning?

Looks like the ASPA profile draft made it to working group last call, so I feel reasonably confident to implement initial ASPA support.

Looks like the ASPA profile draft made it to working group last call, so I feel reasonably confident to implement initial ASPA support.

That would be perfect!


CardActions have different layout from CardContent

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
CardActions is same as on demo.
Current Behavior
CardActions padding and Buttons spacing is different from demo
Demo: https://material-ui-next.com/demos/cards/#cards
Actual rendered: https://codesandbox.io/s/810l94xoml
Steps to Reproduce (for bugs)
Provided above.
Your Environment

Tech
Version

Material-UI
beta.42

React
16.3

browser
Chrome 65.0.3325.181

OS
OSX

@RusinovAnton Where is the difference? This looks the same:
codesandbox

docs

My bad. I've should been more specific from start.
I've just filled buttons with background to make difference more visible.
Demo:

Codesandbox:

It's responsive: https://l4480rk2r7.codesandbox.io/

@RusinovAnton Thanks, it's definitely wrong. What do you think of applying the same fix
https://github.com/mui-org/material-ui/blob/fc6a7fa41119083f1f46c64c80896a2f016d579e/packages/material-ui/src/Dialog/DialogActions.js#L6?

We have the same problem with the ExpansionPanelAction: https://github.com/mui-org/material-ui/blob/fc6a7fa41119083f1f46c64c80896a2f016d579e/packages/material-ui/src/ExpansionPanel/ExpansionPanelActions.js#L24


More functions in BasicMathPackage like min(x, y), max(x, y), ...

It would be nice if the BasicMathPackage implements more basic functions, like some functions processing implements by default.
from processings docs:

[x] abs() Calculates the absolute value (magnitude) of a number
[x] ceil() Calculates the closest int value that is greater than or equal to the value of the parameter
[ ] constrain() Constrains a value to not exceed a maximum and minimum value
[x] exp() Returns Euler's number e (2.71828...) raised to the power of the value parameter
[x] floor() Calculates the closest int value that is less than or equal to the value of the parameter
[ ] lerp() Calculates a number between two numbers at a specific increment
[x] log() Calculates the natural logarithm (the base-e logarithm) of a number
[ ] map() Re-maps a number from one range to another
[ ] max() Determines the largest value in a sequence of numbers
[ ] min() Determines the smallest value in a sequence of numbers
[ ] norm() Normalizes a number from another range into a value between 0 and 1
[ ] pow() Facilitates exponential expressions
[x] round() Calculates the integer closest to the value parameter
[ ] sq() Squares a number (multiplies a number by itself)
[x] sqrt() Calculates the square root of a number

other functions from the docs: (but I'm not quit sure if they would be really necessary and maybe they should go into a separate package for vectors)

[ ] dist() Calculates the distance between two points
[ ] mag() Calculates the magnitude (or length) of a vector

(Sorry if I missed a already implemented function from the packages and I'm in love with this crate, so easy to use)

I think these functions would better fit in the BasicMathPackage because of the simplicity of the functions and the comprehensive area of application.
e.g.:
pixel manipulation from a float value with the range 0.0 - 1.0 to an 0.0 - 255.0 int range -> map(1.0, 0.0, 1.0, 0.0, 255.0) [1]

[1] maybe a better implementation of map can cast between float and integer so that you can do this: map(1.0, 0.0, 1.0, 0, 255)) an store the result in rust as u8

It probably won't be Basic...  if we put it in the standard library, we probably would need to create something like MoreMathPackage etc.
Actually I see many of your proposed functions already in rhai-sci so perhaps it is not a good idea to duplicate.
And the remaining ones are really not very standard and have specific uses in specific cases.

But nevertheless a official and tested distribution of the missing (or more, or less, ...) in rhai-sci would be better than none.

But nevertheless a official and tested distribution of the missing (or more, or less, ...) in rhai-sci would be better than none.

And then maybe it would also be a great idea to mention rhai-sci in the docs page for Numeric Functions.

There are more things to consider. For example, the max function takes two numbers.
Normally, in the standard library, numbers can interoperate, and there are INT, FLOAT, Decimal.
So we're looking at nine combinations and so nine versions of the function.  Seven if excluding float+decimal.
This falls into the gray area where we need to ponder whethrr to add max and min. Because they can easily be replaced by an if expression.

Alright, come to think of it, I agree that max and min probably should belong to the standard library, considering that operators like > and < are defined for numbers.
I'll add them.  Like all the other operators, floating-point can also inter-operate with integers for max and min.


OCAP 2 Integration

Since we now have the OCAP thing, we no longer need the GRAD replay stuff.  I would not merge this into main until OCAP is completely setup, but if you really want to, don't make a new release until it is completely setup.

Should resolve #29, although this PR lacks the custom game events functionality.  Ready to merge pending review.


Derive more anomalies, cluster, filter, bring to experts

Make "fakes and scores" gone through Kostya, so we can derive the number of outliers in the datasets.
Merge master to clustering or vice versa.
Derive new anomalies for all the fields.
Cluster them.
Filter somehow.
Present to Maria.

obsolete, totally


Add apply_chat_template to Tokenizer

This PR adds apply_chat_template to Outline's Tokenizer. This replaces https://github.com/outlines-dev/outlines/pull/1019
Usage:

Unlike Huggingface's tokenizers, we can directly pass the text prompt to tokenizer.apply_chat_template. This will first convert the prompt into conversation format with a predefined system message, then applies the chat template to the result.
answer = generator(
    tokenizer.apply_chat_template(prompt)
)

To use a different system message, either (1) modify tokenizer.SYSTEM_MESSAGE["content"] or (2) build the conversation yourself then pass it to the apply_chat_template here:
conversation = [
    {
        "role": "system",
        "content": "You are a helpful assistant. Please do not say swear words.",
    },
    {
        "role": "user",
        "content": [insert prompt here],
    },
]

answer = generator(
    tokenizer.apply_chat_template(conversation)
)

Interface changes:
Default behavior is kept as is.

@rlouf @lapp0 imo adding it to the tokenizer instead of the model makes more sense.

in chatml format

I think these changes should be applied to the new prompts library.

Closing in favor of outlines-dev/prompts


update flow to 0.89

Since #1765 is blocked on a greenkeeper issue I decided to upgrade flow manually.

@edemaine I ran into this issue with a different project and I was able to fix it by forcing jest to run in single-threaded mode since the circleci instances we're running on are single core instances.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1808   +/-   ##
=======================================
  Coverage   94.58%   94.58%
=======================================
  Files          80       80
  Lines        4655     4655
  Branches      809      809
=======================================
  Hits         4403     4403
  Misses        227      227
  Partials       25       25

Flag
Coverage Δ

#screenshotter
88.83% <ø> (ø)
:arrow_up:

#test
86.61% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


MNT: Update Ubuntu, FreeSurfer, AFNI and Convert3D

Changes proposed in this pull request

Update Ubuntu to 22.04 LTS
Update FreeSurfer to 7.3.2
Update FSL to <IP_ADDRESS>, download only needed packages from conda
Update AFNI installation to work on 22.04
Update ANTs to 2.4.3
Update Convert3D to 1.3.0, from conda-forge

Removing ANTs bump for now. AFAIK there are no outstanding issues we've been waiting on a new ANTs for.

ICA-AROMA is the only thing stopping us from upgrading FSL: #2936.

@mgxd I would appreciate a review on this one today, if possible. It would be good to do all local testing on updated dependencies.

have you tested on any non-CI datasets?

I have tested on one. It will be good to do some more, but the changes relative to 7.2 were not massive.

I think having a FreeSurfer include file, rather than exclude,

The majority of the change was just running sort on the file. I think an include would be more brittle. Here, a name change means we are over-inclusive, while an include file would reverse that.


dont cache if model::$isCachable == false

When calling model::all(), the $isCachable property on the model wasn't checked, resulting in model:all() results to be cached, even when $isCachable was set to false.
This PR attempts to fix that :)

Coverage remained the same at 99.683% when pulling 56c2e757955fb87c3123c05eba14faacf3e1f24f on hxnk:respect-isCachable into 9b8fdb1a53ddaad8854ea0df9775777113c7a399 on GeneaLabs:master.

Hi @hxnk, thank you for submitting this PR! :) I left a comment explaining why this code won't work at this point. Please let me know if you have additional ideas. :)

Hi @mikebronner, what do you think of this solution? It's a bit hacky, but it works :)

disableCache() temporarily disables cache in the config and returns a QueryOrModelCaller object, that is able to direct chained calls to the right objects (all() to Builder, rest to Model). This allows using a query scope and still chain the all() call, which would normally not be possible since a Builder object would be returned
in the QueryOrModelCaller object cache setting in the config is set back to its original state, so that subsequent queries aren't affected

Hi @hxnk, I initially thought that might work as well, but the realized that any global method of disabling cache would cause unexpected behavior for other possible simultaneous request. To avoid uninterrupted side-effects, we will have to find a solution that works on the model instance.

Hi @hxnk  -- I was able to come up with a solution! :) Thanks for keeping the discussion going around this challenge and prompting me to think about different ways of achieving this!


I'd like to be able to save a default sort order for when you search or are in a Browse Category

We've updated the form for performance ensembles for a different feature. @tpendragon will follow up with a link to that work which may guide implementation of this.

related to #697

@hackartisan and @tpendragon . perhaps an easier way forward, that would empower users to create order themselves on pages via a widget (the item row widget, specifically).  If we can constrain proportions of the thumbnail to help standardize the size, I have a feeling that may be an easier and more sustainable route.  We could also workshop the size of the thumbnail, thereby also being able to similarly address and close #858.
There is also the option of getting the item embed widget to function properly. But I would love your thoughts on this.


Add round icon

Round icon added for Google Pixel devices.

The build probably fails because targetSdkVersion is too low (version 25 needed).
https://github.com/k9mail/k-9/blob/master/k9mail/build.gradle#L58

Depends on #2108

Closing this PR as it's not something we'll get to very soon. I created issue #2252 to keep track of this, though.

Thanks.


[G&M] Replaced Selection PropertyGrid

I am not sure if this is related, but when I am creating a Group and I click on an element, I end up with this view:

As you can see I get property list on top as a separate widget and the properties to select for Group query creation is not reachable (I scrolled to the very bottom).

I am not sure if this is related, but when I am creating a Group and I click on an element, I end up with this view:
As you can see I get property list on top as a separate widget and the properties to select for Group query creation is not reachable (I scrolled to the very bottom).

The top widget property grid is unrelated. I'm not sure how to reproduce this. What iTwinJs version are you running?

I am not sure if this is related, but when I am creating a Group and I click on an element, I end up with this view:
As you can see I get property list on top as a separate widget and the properties to select for Group query creation is not reachable (I scrolled to the very bottom).

The top widget property grid is unrelated. I'm not sure how to reproduce this. What iTwinJs version are you running?

It's a fresh app created with npx create-react-app your-app-name --template @itwin/web-viewer --scripts-version @bentley/react-scripts . The app was created with iTwinJs 3.6.2.

The top widget property grid is unrelated. I'm not sure how to reproduce this. What iTwinJs version are you running?

It's a fresh app created with npx create-react-app your-app-name --template @itwin/web-viewer --scripts-version @bentley/react-scripts . The app was created with iTwinJs 3.6.2.

I think I've figured it out. The issue is that the container's size is just too small, causing the PropertyGrid to disappear when its height reaches 0. The Group By container's height stays the same, but its children (the buttons) can't shrink anymore, so they overextend and make the entire GroupAction container overflow. The best solution would be for the contents of the PropertyGrid to cause the GroupAction container to overflow.
But the current implementation of the PropertyGrid makes this tricky. It needs fixed width and height properties and also has internal overflow handling. This makes it difficult for the content inside to overflow anything outside of its container due to its fixed height and width. So, the solution was to keep observing the size changes of the container of the PropertyGrid but this time opted for a 'fullscreen' effect within the widget, providing the necessary space for the PropertyGrid.

The top widget property grid is unrelated. I'm not sure how to reproduce this. What iTwinJs version are you running?

It's a fresh app created with npx create-react-app your-app-name --template @itwin/web-viewer --scripts-version @bentley/react-scripts . The app was created with iTwinJs 3.6.2.

I think I've figured it out. The issue is that the container's size is just too small, causing the PropertyGrid to disappear when its height reaches 0. The Group By container's height stays the same, but its children (the buttons) can't shrink anymore, so they overextend and make the entire GroupAction container overflow. The best solution would be for the contents of the PropertyGrid to cause the GroupAction container to overflow.
But the current implementation of the PropertyGrid makes this tricky. It needs fixed width and height properties and also has internal overflow handling. This makes it difficult for the content inside to overflow anything outside of its container due to its fixed height and width. So, the solution was to keep observing the size changes of the container of the PropertyGrid but this time opted for a 'fullscreen' effect within the widget, providing the necessary space for the PropertyGrid.

I don't think the solution of introducing a button to open properties list is good... It makes the group creation more complex and requires user to click on additional button. Why would I want to open properties when I want to create a group?
Also, the padding of the footer buttons looks off when you click on the "Open Properties" button. It's too small. And some long properties overflow outside of the property table:

Does it happen on both the PropertyGrids? (The included one as well)

Does not happen anymore. Must have been something with my local build.

Looks like the UI changes quite a bit. From this:

To this:

Notice the difference in the scope of vertical scrollbar.
I'd say this needs to be looked at by QA and UX before we can merge this PR.
See https://github.com/iTwin/viewer-components-react/pull/406 which addresses the same issue as this PR but with less drastic changes (UI stays the same).


torchmetrics.Precision error

hello,
I'm trying to test the model based on the ReadMe file you posted, but I'm getting the following error.
"precision": torchmetrics.Precision(num_classes=2, average=None).to(device),
TypeError: new() missing 1 required positional argument: 'task'
I've been try to modify
"torchmetrics.Precision(num_classes=2, average=None).to(device)"  to "torchmetrics.Precision(task = "binary",num_classes=2, average=None).to(device)"
but it doesn't work.

Hello,
the problem seems to be that you haven't installed the specified virtual
environment,
in which torchmetrics 0.9.3 is installed.
The current version of torchmetrics has different input arguments.
Am Di., 21. Nov. 2023 um 09:43 Uhr schrieb Jieun Kim <
@.***>:

hello,
I'm trying to test the model based on the ReadMe file you posted, but I'm
getting the following error.
"precision": torchmetrics.Precision(num_classes=2,
average=None).to(device),
TypeError: new() missing 1 required positional argument: 'task'
I've been try to modify
"torchmetrics.Precision(num_classes=2, average=None).to(device)" to
"torchmetrics.Precision(task = "binary",num_classes=2,
average=None).to(device)"
but it doesn't work.
—
Reply to this email directly, view it on GitHub
https://github.com/ttanida/rgrg/issues/20, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI7Q3AOPX2URHHYLXJV6QVDYFRSUHAVCNFSM6AAAAAA7UFQ5U2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGAYDGNZUHAYTANQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


support max-line-length

Follow up from https://github.com/marko-js/marko-migrate/issues/10
It would be useful to have the max-line-length style setting exposed for easier marko migrations from v2 to v3.

If you are interested in working on this, the current value for "max-line-length" is currently hard coded: https://github.com/marko-js/marko-prettyprint/blob/cd248406373bbf50c5298b81b96803f815e5e699/src/printHtmlElement.js#L12


support scaled_dot_product_attention for shifted window attention

Changes

support torch.nn.functional.scaled_dot_product_attention for shifted_window_attention in swin.
make the mask generation process as function

https://github.com/pytorch/vision/issues/8182
Compatibility Test
    import torch
    import torchvision
    batch_size = 2
    height = 224
    width = 224
    img = torch.rand([batch_size, 3, height, width])
    model = torchvision.models.swin_t()
    ouput = model(img)

    model = torchvision.models.swin_t()
    ouput = model(img)

    model = torchvision.models.swin_v2_t()
    ouput = model(img)

Unit Test

got the identical result with current implementation, when enable_mem_efficient=False

    import torch
    from torchvision.models.swin_transformer import shifted_window_attention

    batch_size = 2
    channels = 96
    height = 128
    width = 128
    window_size = [7, 7]
    num_heads = 3
    shift_size = [0, 0]

    input = torch.rand([batch_size, height, width, channels])
    qkv_weight = torch.rand([3 * channels, channels])
    proj_weight = torch.rand([channels, channels])
    relative_position_bias = torch.rand(
        [1, num_heads, window_size[0] * window_size[1], window_size[0] * window_size[1]]
    )

    old_attn = shifted_window_attention(
        input=input,
        qkv_weight=qkv_weight,
        proj_weight=proj_weight,
        relative_position_bias=relative_position_bias,
        window_size=window_size,
        num_heads=num_heads,
        shift_size=shift_size,
        attention_dropout=0.0,
        dropout=0.0,
        qkv_bias=None,
        proj_bias=None,
        logit_scale=None,
        training=True,
        use_efficient_attention=False,
    )

    new_attn = shifted_window_attention(
        input=input,
        qkv_weight=qkv_weight,
        proj_weight=proj_weight,
        relative_position_bias=relative_position_bias,
        window_size=window_size,
        num_heads=num_heads,
        shift_size=shift_size,
        attention_dropout=0.0,
        dropout=0.0,
        qkv_bias=None,
        proj_bias=None,
        logit_scale=None,
        training=True,
        use_efficient_attention=True,
    )

    diff_attn = torch.abs(old_attn - new_attn)
    print(torch.sum(diff_attn))

attn_mask of scaled_dot_product_attention in torch doesn't support tensor type yet.


Split into local and production database

We want to be able to run this project locally, therefor we need a split between the local and remote databases.
My suggestion is to:

Add the repro.local database by default
Add the possibility to use the env variable to set another name for it.
Set this in CI.

closed by https://github.com/prefix-dev/reproducible-builds/pull/100


ci: Add Binder build trigger

This PR does a few things:

First, it moves Binder related files to a top level directory binder/ as if this directory exists Binder will only look in it for configuration files and ignore all others. A postBuild config file is then used to install all of the Python dependencies in various requirements.txt files that are needed by the notebooks.

Add GitHub Actions workflow and shell script to trigger Binder builds of the repo on the GKE cluster of the Binder Federation (most common one to get requests from North America) upon successful merge of a PR into master. Look at pyhf's workflows for examples of this in action.

The Binder badge is made to use the ?urlpath=lab/tree/book URL appendix to have Binder open Jupyter Lab in the book/ directory upon launch (as there is no obvious default notebook to open up into). Example on my fork:

Tonight (2020-08-18) myst-parser (which gets picked up in jupytext[myst])  released v0.12.0 which is incompatible with myst-nb (I think) resulting in build errors. By explicitly adding myst-parser~=1.11.1 to book/requirements.txt this restricts the allowed range of myst-parser to v1.11.X, allowing for builds to succeed.

Finally, .DS_store files are binaries that MacOS makes for searching and shouldn't be in Git repos, so they're both removed (no need to keep them) and a .gitignore file is made with their pattern in in.

Suggested squash and merge commit log
* Use Binder postBuild configuration file to install Python dependencies
   - c.f. https://mybinder.readthedocs.io/en/latest/config_files.html#postbuild-run-code-after-installing-the-environment
* Add GHA workflow to trigger Binder build on PR merge
* Restrict myst-parser to v1.11.X range to give compatible release for myst-nb
* Update Binder badge to launch into Jupyter Lab
* Remove and ignore .DS_store

@cranmer This is ready for review.

Wow, thanks a lot! I thought I had removed the .DS_store files, thanks for catching that.

FYI: I got this with binder

FYI: I got this with binder

ah. okay. Things should still work fine, but those extensions probably won't work out of the box (though you don't need them). All the notebooks still run fine though, yes?
I can submit another PR later that properly does the jupyterlab build after install to get them from complaining.


Lack of Restaurant Addition Option, Food Menu Complexity, and Responsibility Clarification in Enatega Multivendor

Issue: Lack of Restaurant Addition Option, Food Menu Complexity, and Responsibility Clarification

Description:
There is no option of adding new restaurant from demo , but I assume there will be restaurant adding option, my question is who will add the foods, menu etc ? Will there be any option for restaurant manager ? Can food menu option be little easier to add ? Now its too complex and ambiguous, It can be Menu/categories and products/items only.
Impact:
1**. Lack of Functionality:** The absence of a restaurant addition option affects users who expect to add their own restaurants to the platform.
2. User Confusion: Users are uncertain about how to add and manage food menus, which leads to a poor user experience.
3. Complex Menu Management: The current food menu management system is overly complex and needs simplification for user convenience.
Steps to Reproduce:

Access the website.
Look for the option to add a new restaurant. Observe that this option is not available.
Attempt to add or manage food menus for an existing restaurant.
Notice that the process is complex and unclear.

Expected Resolution:

Implement a clear and user-friendly process for adding new restaurants, with options for restaurant owners or managers.
Simplify the food menu management process by focusing on menu categories and individual food items/products. Ensure that the steps are straightforward and easy to follow.
Provide guidance or documentation regarding the responsibility for adding food items to menus, clarifying whether it's the restaurant owners, managers, or platform administrators.

I believe there may be some confusion regarding the process of adding a restaurant and food items within it. Here are the detailed steps:
Only the vendor has the authority to add a restaurant. To begin, we need to create a test vendor.

Click on the 'Restaurant' button located next to the vendor's profile.

Now, let's create a new test restaurant within this vendor's account.

When you click on 'Add New Restaurant,' make sure to provide the following restaurant information:
Restaurant username and password
Restaurant name
Delivery time
Address
Minimum order amount
Sales tax
Upload an image for the restaurant

Once you have filled out all the necessary information, click 'Save' to successfully create the new restaurant.

By clicking on the newly created restaurant, you'll open the restaurant panel.

Inside this panel, you can add food items to the restaurant, set categories, define the location, and more.

For a visual guide and complete workflow, please refer to the following demo video: Demo Video Link


undefined method `production?' for #UIApplication:0x10d15b7f0

According to the docs, there is an app.production? method which is an alias of release?. I added some code to my App Delegate which runs some code that is intended for production/app store release, however I am seeing this error:
2015-06-02 15:49:22.878 app[98811:33490681] app_delegate.rb:7:in `on_load:': undefined method `production?' for #<UIApplication:0x10d15b7f0> (NoMethodError)
	from delegate_module.rb:16:in `application:didFinishLaunchingWithOptions:'
2015-06-02 15:49:22.890 app[98811:33490681] *** Terminating app due to uncaught exception 'NoMethodError', reason: 'app_delegate.rb:7:in `on_load:': undefined method `production?' for #<UIApplication:0x10d15b7f0> (NoMethodError)
	from delegate_module.rb:16:in `application:didFinishLaunchingWithOptions:'
'

Here is my code:
class AppDelegate < PM::Delegate
  status_bar true, animation: :fade

  def on_load(app, options)
    if app.production?
      $API_ROOT = 'https://production.example.com/api'
    elsif app.staging? # TODO: make this work since there is no iOS staging environment
      $API_ROOT = 'https://staging.example.com/api'
    elsif app.development?
      $API_ROOT = 'http://localhost:3000/api
    end
# ...

Am I doing something wrong?

app probably isn't in the PM:Deleate object. It's a shortcut to rmq.app or find.app. So you can use them. We should add it in RedPotion to PM::Delegate.

Ah, yes, that solved the problem. I don't feel like this type of code belongs here anyway. Where do you typically define your environment config variables?

I cant speak for everyone else, but the way I handle it is something like this....

a yaml file for each env (production, staging, development, integration, travis, etc...)
when compiling an ENV value to tell which ENV ENV=staging rake
my rakefile reads the config.yml, and writes a new class for me app/settings.rb that has all my settings
I can refer to them like Settings.api_host inside the app

this works pretty well for me, its all custom in my app though at the moment and uses the rake-hooks gem, I previously used motion-config-vars but it would mess with the ENV, so app.production and app.development didnt work in the rake file, and it exposed all the ENV values in Info.plist


Device chooser navigation bar become transparent

Hi,
When I compiled the codelabs code on my iPhone 8 Plus (ios 15.0.2), the navigation bar on deviceChooser become transparent. Is it supposed to displayed like that or not?

Is it possible to change/update the navigation bar background color? There is no info mention about the navigation bar ui styles modification on https://developers.google.com/cast/docs/ios_sender/customize_ui section.
Appreciate if anyone can advice on my problem. Im using Xcode 13.0.
Thank you in advance ^_^

Thank you for reporting.  This is a known issue.  Our iOS Sender SDK release 4.7.0 has a fix for this.  Please keep an eye out for the release.  (It should release later today.)

Closing issue as the newest release should provide a fix. Please reopen if issue persists.


update product shots and update copy

can you remove 'and sportsbook' from here and replace image with the market page image below. If you need an smaller size export for it let me know

Remove all the screenshots that are currently in there and add these

Can you replace this image with the new market page


refactor(types): remove privateChannels field from Ready

BREAKING: since Tuesday, 6 April 2021, private_channels field has been undocumented by @night. Reference commit: https://github.com/discord/discord-api-docs/commit/f36156dbb641f5c4d4f4593f345bfd6e27fdee08

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.keenkairos seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Question: Latest Roslyn .vsix

Is it available anywhere?
Couldn't find it in the CI artifacts.

The next preview will contain the F#/Roslyn integration.


Extract common part from 'if'

You perform an advance() in if and else.
https://github.com/compose-generator/ccom/blob/78b923bdfe4eb370a237f34a4c2b3b59cdf43c1e/compiler/cpp/src/lexer/lexer.cpp#L235-L240

E.g. using Java, one could write:
 if (reader.lookAhead() == '\\') {
      reader.advance(); // one extra advance
  }
  value.append(reader.lookAhead());
  reader.advance(); // for next while loop iteration


[Ars Magica 5] Sheet tabs are rendered as plain radio buttons

Hi @Medieve and everyone,
I think there is a problem with the tabs selector on the Ars Magica 5 sheet: it is rendered as 5 plain radio buttons instead of the nice headers visible on the forum. I tried with firefox, on both windows and unix.
Unfortunately, I don't now html/css and I have no idea how to fix this. If anyone has pointers, I can try integrating a fix in #8094 . This is minor as it doesn't break the sheet, but the sheet is otherwise stunning and it would be nice to restore those beautiful headers.
Best !

This is just the state of the sheet in Firefox, I honestly have no idea why Firefox won't render the headers.

Sometime around a year ago Firefox changed how something worked, resulting it that specific way many older sheets use of trying to show tabs not working any longer. This isn't the only sheet that's affected.
The current example on CSS Wizardry works on firefox, but changes the radio inputs to action buttons.
Ambition & Avarice and Stargate-RPG sheet are two examples that use the radio-input + span method that actually works with firefox, and should be somewhat close to how the Ars Magica 5 sheet have implemented it.

This is just the state of the sheet in Firefox, I honestly have no idea why Firefox won't render the headers.

Sometime around a year ago Firefox changed how something worked, resulting it that specific way many older sheets use of trying to show tabs not working any longer. This isn't the only sheet that's affected.
The current example on CSS Wizardry works on firefox, but changes the radio inputs to action buttons.
Ambition & Avarice and Stargate-RPG sheet are two examples that use the radio-input + span method that actually works with firefox, and should be somewhat close to how the Ars Magica 5 sheet have implemented it.

Looks like I completely missed that section in CSS Wizardey (oops!). Thank you so much, I'll use the Stargate/Ambition & Avarice sheet to adapt tabs to firefox.

Looks like I completely missed that section in CSS Wizardey (oops!). Thank you so much, I'll use the Stargate/Ambition & Avarice sheet to adapt tabs to firefox.

I ported the code to the recommended style on the wiki, as it seems much simpler and possibly more compatible with other browsers (e.g. Safari). My progress is on a new branch that I will merge into the PR once ready.
There are still bugs:

The tab button are confined on only half the sheet width, the rolling button using the other half. I would like the tab button to use whatever wpace they need to be on a single row.
The critical button still has a dice icon before the text. The roll and botch do not, and I copied their code, so I am at a lost

I'm completely new to CSS, so I could use a hand on those matters if someone has the time to lok into the HTML/CSS:

html: https://github.com/Riernar/roll20-character-sheets/blob/d538bd057455e1a85587e5eab58ae8fc589666fc/Ars_Magica_5th/template.html#L50
css: https://github.com/Riernar/roll20-character-sheets/blob/d538bd057455e1a85587e5eab58ae8fc589666fc/Ars_Magica_5th/Ars_Magica_5th.css#L93

I ported the code to the recommended style on the wiki, as it seems much simpler and possibly more compatible with other browsers (e.g. Safari). My progress is on a new branch that I will merge into the PR once ready.
There are still bugs:

The tab button are confined on only half the sheet width, the rolling button using the other half. I would like the tab button to use whatever wpace they need to be on a single row.
The critical button still has a dice icon before the text. The roll and botch do not, and I copied their code, so I am at a lost

I'm completely new to CSS, so I could use a hand on those matters if someone has the time to lok into the HTML/CSS:

html: https://github.com/Riernar/roll20-character-sheets/blob/d538bd057455e1a85587e5eab58ae8fc589666fc/Ars_Magica_5th/template.html#L50
css: https://github.com/Riernar/roll20-character-sheets/blob/d538bd057455e1a85587e5eab58ae8fc589666fc/Ars_Magica_5th/Ars_Magica_5th.css#L93

https://wiki.roll20.net/CSS_Wizardry#Button_Styling

https://wiki.roll20.net/CSS_Wizardry#Button_Styling

Oh thanks, I forgot to add the critical button to the CSS selector doing that !
I got most things working now, just got to figure out how to select the tab currently opened in CSS to change de button background.

Oh thanks, I forgot to add the critical button to the CSS selector doing that !
I got most things working now, just got to figure out how to select the tab currently opened in CSS to change de button background.

Layout is working, merged in #8094

Layout is working, merged in #8094


sqlite3 generator does not handle table constraints

Here is my test schema
CREATE TABLE user (
user_id		TEXT NON NULL PRIMARY KEY,
email		TEXT,
first_name 	TEXT,
last_name	TEXT,
headline	TEXT,
profile_url	TEXT,
location	TEXT,
username	TEXT,
password	 TEXT
);
CREATE TABLE id_provider
(
provider_id	TEXT NOT NULL,
user_id		TEXT NOT NULL REFERENCES user( user_id ),
network		TEXT,
PRIMARY KEY( provider_id, user_id )

);
If I remove the PRIMARY KEY table constraint it generates fine. With it the following error is generated
error: sql: Scan error on column index 5: sql/driver: couldn't convert 2 into type bool
Would it be possible to support composite primary keys? I need them for the project I'm working on and would like to use xo to save me some time. If not is there some sort of work around where I can still generate most of code using xo.
Regards and thanks for making this work public.

This can definitely be fixed, but I'm not likely to be able to fix this today. The issue is that I wasn't aware that SQLite supported this for primary keys. I need to rewrite quite a bit of the way this is queried for SQLite. Unfortunately, SQLite works uses PRAGMA statements for introspection, which is completely different from all the other supported databases (all of the other databases use tables/views for SQL introspection). It's not too difficult to fix, but requires some time investment.

BTW -- you have a spelling mistake in your schema: user_id TEXT NON NULL PRIMARY KEY, should be user_id TEXT NOT NULL PRIMARY KEY,

Thanks for the quick response! And pointing out the error!
I'll keep checking back once you have sometime to take a look.
Thanks again.

Please see #58.


Improve bug report template

Improve bug report template to make it more suitable for Ghidra.
Added an attachments section to prompt for file attachments, and added Java version instead of browser (which is likely to be irrelevant).

Good suggestions. Thanks!


Add classifiers to setup.py

Hi!
I've added classifiers to setup.py. Third time's the charm, I hope.
Thank you for all the help and for pipdeptree.

Thanks. I'll push a new release to PyPI shortly.


Support standard ONNX quantized operators?

Hi, I've been looking for an end2end quantization deployment solution, so far I've tried the onnxrutime + TVM stack, but onnxruntime only supports naive PTQ methods. Your work looks really promising, however, I wonder can MQBench export the quantized model in the form of standard ONNX quantized ops like QLinearConv, QuantizeLinear, DequantizeLinear, etc?
See: apache/tvm#8838

We consider deploying quantized model on TVM important, deploying on TVM with QuantDequant node will be supported very soon. Actually we have already done this in PTQ scheme for experiments.

@Tracin
I noticed that the v0.0.3 version has ONNX QNN Ops support, however, I cannot export an ONNX QNN model when running the example code in test_quantize_onnxqnn in test_backend.py:
No Op registered for LearnablePerTensorAffine with domain_version of 11

It seems that ONNX does not support the custom fake quantize Op LearnablePerTensorAffine in the quantized model. I also noticed that there's a function deploy_qparams_tvm which uses the ONNXQNNPass, in which fake quantize Ops are replaced with standard ONNX QNN Ops, but this function is not called during convert_deploy. Is there any documentations explaining how to use these utilities to export to a standard ONNX QNN model?

ONNXQNNPass is registered in scheme ONNX_QNN, you can prepare using backend=ONNX_QNN, then it will be called.


WIP: Archive index design

WIP: Archive index design
Background
For the purposes of this discussion, an IPLD Store is a Mapping of CID to (binary) data[^1].
The industry standard for Mappings of this scale is to use a key-value database.
Databases may be embedded (co-resident) or client-server (maybe co-resident).
[^1]: Breaking this assumption would unlock using graph databases.
I think we should look into that at some point.
Motivate and problem statement
Archival nodes need an IPLD Store for the entire history of a chain.
For mainnet, at time of writing this is O(50TiB), with O(50G) CIDs.
A key selling point for Forest is viability as an archival node on commodity hardware.
This is currently implemented using many .forest.car.zst files, but lookup performance is relatively poor, so we should consider other solutions for beta users.
Evaluation criteria

Disk usage
Incremental updates

Unlike e.g snapshot users, archive users aren't going to want to rebuild their database every night.

Convenience

Import/export tooling
Compatability with e.g snapshot files

Lookup latency
Implementation complexity

Existing IPLD Stores on Filecoin nodes
Every Filecoin node needs an IPLD Store.
Forest engineers have experience with the following IPLD stores.
They are all embedded, presumably because:

embedded is simpler/respoects the node boundary.
embedded suits this scale of database.

Rocksdb
erDiagram

rocksdb-ipldstore {
  u8[] cid PK
  u8[] data
}

ParityDB
erDiagram

paritydb-ipldstore {
  u8[] cid PK
  u8[] data
}

.forest.car.zst files
erDiagram

cid one -- one or more inline-index: hash
inline-index {
  u64 hash
  u64 offset
}
inline-index one -- one file: lookup
file {
  u64 offset PK
  u8[] data
}

Candidates
MySQL
Datatypes cheat sheet:

MySQL
Rust (unsigned)

TINYINT
u8

SMALLINT
u16

MEDIUMINT
u24[^2]

INT
u32

BIGINT
u64

[^2]: not a core primitive type
Aside: storing CIDs
Which scales better?
Here is a faithful translation of CidGeneric<64>, the default cid::Cid
erDiagram

faithful-cid64 {
  ENUM version PK
  BIGINT cid_codec PK
  BIGINT hash_code PK
  VARBINARY(64) hash_digest PK
}

Most hashes look like this
Keep a table for each variant
erDiagram

base32-cidv1-raw-blake2b-256 {
  BINARY(32) digest PK
}

(or somewhere in between)
consideration: what would require code changes?
Custom Binary Search
Custom Hash Table
SQLite3
Evaluation
Evaluation at mainnet scale

Solution
Disk usage
Latency
Inc
Convenience
Complexity

Readers may also be interested in https://github.com/ChainSafe/forest/pull/3839


Add os name, major, minor release

Dependent on this Schema PR discussion:
https://github.com/RedHatInsights/inventory-schemas/pull/27

one comment from me but looks good otherwise

@SteveHNH just letting you know this is ready for your team's eyes and merge hopefully, thanks!

@CodeHeeler can you fix up the conflicts here and then I'll take another look. I think this should be okay as long as it matches the schema

@SteveHNH Done, thanks!

This did pass the relevant checks. The failing test is related to Bonfire


Consider using css modules to provide style

react-toolbox uses css modules to allow theme overrides and css-classname compilation.
I provides a great level of control.

Thanks for raising the suggestion. I've already considered this some time ago though (see here for example) and decided against it. The current approach is as flexible as possible for users who prefer static CSS, inline styles, or CSS modules.
RV aims to avoid all presentational styles anyway, focusing only on functional styles. These don't really lend themselves to needing to be customized in the same way as a UI component library.
Cheers. :smile:


Error when running deno task cli address

System: Ubuntu 22.04 LTS virtual machine
Running a fully synced cardano-node. Kupo and ogmios running as a service locally.
deno task cli init  - worked and created seed.txt
deno task cli address - yields this error:
Task cli deno run --allow-all miner/main.ts "address" error: Top-level await promise never resolved await new Command() ^     at <anonymous> (file://fortuna/miner/main.ts:401:1)
The .env file in fortuna root directory is configured as such (default values for kupo and ogmios):
KUPO_URL="wss://<IP_ADDRESS>" OGMIOS_URL="wss://<IP_ADDRESS>"

OGMIOS_URL needs to be prefixed with "http://" and KUPO_URL with "ws://" unless you are actually using a secured connection

Thank you for replying.
.env file has been changed to:
KUPO_URL="ws://<IP_ADDRESS>:1442" OGMIOS_URL="http://<IP_ADDRESS>:1337"
Now getting this error when running the same command:
error: Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'costModels')           Object.keys(result.costModels).forEach((v) => {                              ^     at WebSocket.client.addEventListener.once (https://deno.land/x/lucid@0.10.1/src/provider/kupmios.ts:45:30)     at innerInvokeEventListeners (ext:deno_web/02_event.js:750:7)     at invokeEventListeners (ext:deno_web/02_event.js:797:5)     at dispatch (ext:deno_web/02_event.js:654:9)     at WebSocket.[[[eventLoop]]] (ext:deno_websocket/01_websocket.js:434:11)     at eventLoopTick (ext:core/01_core.js:183:11)
cardano-node, kupo and ogmios are all running as services in the background.

I can telnet into kupo and ogmios. Cardano-node connection gets closed after a few seconds, but its a fully functional and synced relay with incoming and outgoing connections.
Kupo is fully synced, with current slotNo matching that of cardano-node query tip.
Kupo is running with those flags --since origin --match addr1q9vv07dvhf45w9cx8pcjrseky0azmqg0uh0kkjc3kun3p45pueq3e6tezsa5wzugtmkz4x08xu6pleccdk9g9t9fjfwq0z744r
The address comes from the seed generated by deno task cli init, which I then restored on Eternl wallet. Unsure as to whether that is affecting anything.

For Kupo you will also need to add tuna's validator address as an additional match, like --match addr1wynelppvx0hdjp2tnc78pnt28veznqjecf9h3wy4edqajxsg7hwsc.
Or simply use --match '*/*' - I'm personally running it this way and have been able to mint blocks successfully.

I'm still struggling with this and stuck at the same step. Do you run ogmios and kupo as a service as well? Would it be possible to know your configurations and see if there's something I missed or did wrong?


Adding gamma calculation

Could you please add hacktoberfest label?

Thanks for the contributions @PedroCisnerosSantana! Added the Hacktoberfest label and will merge the PR!!!


Add verify-spelling.sh for English words.

What type of PR is this?
/kind feature
What this PR does / why we need it:
Add verify-spelling.sh to check english typos...
Ignore .golangci.yaml,
./.golangci.yaml:22:6: "importas" is a misspelling of "imports"
./.golangci.yaml:50:2: "importas" is a misspelling of "imports"

because importas is the package name.
golinters importas typos
Which issue(s) this PR fixes:
Fixes #545
Special notes for your reviewer:
Does this PR introduce a user-facing change?
None

So fast for review...😀😀

vendor dir is not exclude..
i will test it againt.
wait a moment for rereviw.

Good suggestion.
Update later.

/retest


eyeD3 version

this project is amazing! I have tried the speaker diarization feature and it really works! Super Cool! However it is also worthwhile to specify the version of eyeD3 being used.. While I was installing this package,  there was an error on eyeD3 module being used. The package will not work with the latest eyeD3 of versions 0.7.x but with versions 0.6.x.
Hope this project will be added in PyPi. The project is promising!

How do I install the old 0.6.x version instead of 0.7.x?

Go back through the commits, find the version you want to load, and then clone that. Does that make sense?

An update: please use the latest library version, and install eyed3 using pip (see wiki and README for instructions).

Traceback (most recent call last):
  File "pyAudioAnalysis/audioAnalysis.py", line 426, in <module>
    dirMp3toWavWrapper(args.input, args.rate, args.channels)
  File "pyAudioAnalysis/audioAnalysis.py", line 30, in dirMp3toWavWrapper
    audioBasicIO.convertDirMP3ToWav(directory, samplerate, channels, useMp3TagsAsNames)
  File "/Users/dofine/Devel/audioAna/pyAudioAnalysis/audioBasicIO.py", line 17, in convertDirMP3ToWav
    tag = eyed3.Tag()
AttributeError: 'module' object has no attribute 'Tag'

eyed3 API change?

I randomly installed eyed3==0.7.11 (pip install eyed3==0.7.11) and it works just fine for me.


Added support for tvOS

Added tvOS to the podspec, plus an example project with similar scenarios as the iOS example. Things seem to be working well, but there are two exceptions:

FLAnimatedImage does not yet support tvOS. I opened a pull request to add it, but in the meantime, you can run the tvOS example by adding pod "FLAnimatedImage", :git => "https://github.com/ioveracker/FLAnimatedImage", :branch => "tvos" to the Podfile in Example-tvOS, above the line for PINRemoteImage.
WebP doesn't seem to work--could be a limitation of the platform, or libwebp may need to be updated.

@ioveracker awesome! I'll try and take a look at this as soon as I can!

FYI, I'm releasing 2.0 very soon, but will make sure to release a 2.1 when we get this in :)

@ioveracker I'm sorry this hasn't made it in yet, embarrassingly, I need to get our team on a newer version of cocoa pods which supports tvOS before I can merge. I'm going to work on that soon! 2.1 will be the tvOS release :)

No worries! The FLAnimatedImage pull request is still pending as well.

@ioveracker Just finished upgrading the team to CocoaPods so I'm swinging back around to this. I downloaded your branch to give it a try, but I'm not able to get the example to run in Example-tvOS. I get an import error on PINRemoteImage every time.
I'm excited to have a Swift example and tvOS support landed :) Do you have any ideas what could be happening? I pod updated in the directory and that didn't work.

Hm, I just did a clean clone, updated the Podfile to use my FLAnimatedImage repo (since that pull request is still pending over at FLAnimatedImage), ran pod install, then opened the workspace and it built and started running on the tvOS simulator. I'm running Xcode 7.2.1 now, and was on whatever the latest was when I started this. :)
Since it's using the local files for PINRemoteImage, maybe there's some crossover between work you've been doing on the master branch? I don't know if that's possible, because it should be building it from source, and when switched to the tvos branch, none of the changes from the master branch should be in the source.

Crossover with the master branch is unlikely--I just pulled the latest master changes into my repo and even merged it into my local tvos branch and didn't encounter any issues. :confused:

Ok, I'll give it another try with a clean clone!

Coming back to this after a month, there are a couple things that I think could be changed (besides the fact that you can't get it to run! :sweat_smile:) :

The WebP example being the first tab isn't exactly ideal, since that's the only example that doesn't actually work on tvOS. :laughing: Perhaps it would be better to move this example to the last tab.
Since the FLAnimatedImage pull request is still pending, would you like to configure a way to build for tvOS without FLAnimatedImage so non-GIF support can be released separately, while FLAnimatedImage figures out their side of things? Or just wait for that to be merged before this gets merged?
I noticed that the Pods directory was committed for the other examples. Would you like the Pods directory to be committed here, too? Will require deciding how to proceed with the FLAnimatedImage side of things first. :grin:

@ioveracker Cool! Maybe it will just work with the updates :P

Yeah, that sounds good.
I think we should configure support for without. Instead of including the PINRemoteImage pod spec, use PINRemoteImage/iOS? Make one for tvOS specifically?
Yes, please commit the Pods directory, it makes it more reliable for using pod try

Thanks for you work on this!

Sounds good! I'll take a stab at these later this evening.

Okay, I've taken care of the above three items.

I removed the GIF and WebP examples. It'll be easy to add them again if/when supported.
I added a tvOS subspec. It's identical to the iOS one, but I thought it would be nice to start with a separate subspec in case they diverge.
Pods directory committed.

I haven't been able to figure out why the Travis build is failing. Is it really being terminated because the log file is exceeding 4 MB just during the normal build, or is something else going on? I see a similar result in #308.1.

@ioveracker ugh, yeah, travis has been a pain to get working correctly lately. Mind removing the verbose flag in the pod lint step in the .travis.yml file?

Actually, I just removed it, so if you could just rebase, I'd appreciate it.

No problem! Done.

It works!

:tada:

Thanks again, this is awesome!

Sure thing! Thanks for all your hard work on PINRemoteImage, it's really great.


Moving country attribute to class

Right now, the country attribute appears to be defined in the __init__ method for each class. I suggest moving it to a class attribute. This will not create any backwards incompatibility that I can see, and will allow the following filtering to work without having to instantiate a couple of hundred classes:
list_of_unique_countries = [c for country in holidays.list_supported_countries() if (c := getattr(holidays, country)).country == c.__name__]

instead of
list_of_unique_countries = [c for country in holidays.list_supported_countries() if (c := getattr(holidays, country))().country == c.__name__]

On a related note, it may be of value to replace assignments like
class GB(UnitedKingdom):

With the simpler
GB = UnitedKingdom

Again, it does not appear that this will cause incompatibilities anywere, and will make it easier to filter unique countries. Combining would simplify things like
class IsleOfMan(UnitedKingdom):
    def __init__(self, **kwargs):
        self.country = "Isle of Man"
        HolidayBase.__init__(self, **kwargs)

To the less error-prone
class IsleOfMan(UnitedKingdom):
    country = "Isle of Man"

Hi,
although I don't feel this as a huge improvement (it could have been if we could, for instance, provide a default value for it, which in fact is not the case), it is semantically correct and less prone to implementation errors, so I decided to implement it all the same.
It is now released in beta, thank you! 👍

Awesome, thanks!

By the way, here is the question that led me to learn about this package and indirectly to raise this issue: https://stackoverflow.com/q/70237552/2988730. Again, thanks for the update and for the great package


Strict checking of package installations.

Description of the new feature/enhancement
Add a --strict option to winget install which enforces certain additional checks to ensure applications installed correctly.

Proposed technical implementation details (optional)

Okay, before I start, this won't really make much sense until https://github.com/microsoft/winget-cli/issues/1073 is implemented. Also, I know what I'm going to describe (if anyone even likes it besides me) will most likely need a spec, which I can draft.
#1073 is designed to allow package maintainers to enumerate every ARP entry a installer makes, so that winget has a better time matching. This should mean instead of the guesswork (which is pretty good, but not 100%) winget does now to try to match packages to the ARP table, it can know exactly what is supposed to be there.
I propose that a new option, --strict, be added so that if a package installs and the correct ARP entries are not made (so either less than enumerated in the manifest, or more, or just different/incorrect ones) that winget should say the install failed, perhaps print the differences (--verbose-logs?) and exit with a non-zero code.
This would help a ton with confidence in winget and allow it to start managing packages more than just lighting the firework and running away, which is the current functionality (well, it tells you if the installer said it failed, but the installers often lie). And of course, this functionality could be extended to do other checks based on assertions we can make by Installer Type. From a user perspective, I have more confidence, and from a package maintainer perspective this allows much better automation for package moderation (run the package with --strict. if the metadata didn't match, that's an error that's now super easy to push to a CI step instead of having to write code ourselves to read the ARP table. We shouldn't have to do that, winget already knows what it can and cannot match ;D)
There's a couple drawbacks to this, namely, that MSIX packages don't use the ARP table (but they have an equivalent), and portable applications don't either (but last I heard, the planned implementation of installing portable apps was to create an ARP entry for them. if that's still the case than it could work). It would of course be optional, as packages do all kinds of nasty things sometimes that we just have to go along with. (Chrome changes it's ARP entry on first run, for example).
Please give me feedback and tell me if this sounds good/useful. I don't think it would be too bad to implement after #1073, but maybe there's some grossness I'm not aware of.

Also, there's not a lot of prior art for this. The only one I know of off the top of my head is SCCM's detection methods, but it doesn't rely on the ARP table for exe installers, instead opting to look for created files (which is more brittle, IMO).


Add some keycodes that needs to set shift to TRUE

A few symbols are missed in function send_keys() because they need to set shift to TRUE, but not in the keycodes mapping.
Testing Done:
send_keys("`~!@#$%^&*()_+{}|[]\\:\";'<>?,./")

Could you sign the CLA?

Have signed CLA, but not sure why it is still pending state.

the CLA robot checks the email for the local git account, which created the PR. Please make sure this email is the same the CLA has been signed for

the CLA robot checks the email for the local git account, which created the PR. Please make sure this email is the same the CLA has been signed for

It is not same, how do I can resign CLA or I have to close this one and request new one?

You can create another PR which has an email you can use to sign the CLA or rebase the commit

thanks :)


Enable Helper to be monitored.

It is necessary to monitor Helper before loading Controller.

Probably, it is completed before starting monitoring.
class Railtie < ::Rails::Railtie
    config.after_initialize do
      # .... If eager_load enabled, Helper loading has been completed!
    end
end

So, should use config.before_initialize instead of config.after_initialize


Why the CPU inference cost gpu memory?

I test the cpu inference by cascade_rcnn model， but it cost the gpu memory 1500M。  (by the way,when i used gpu inference,it cost the gpu memory about 2500M)

I modified like this. now it doesn't cost gpu memory


click must be at least 7.0

because we use hidden options.
This commit fixes https://github.com/raiden-network/raiden-contracts/issues/784

Thank you!


Perfection kills

Consider a plugin for a library named playground.
Library namespace is var PLAYGROUND = { }
Plugin is represented by a constructor
PLAYGROUND.SoundOnDemand = function() { }
reasonable permutations
playground.SoundOnDemand.js
playground.soundOnDemand.js
playground.soundondemand.js
playground.sound-on-demand.js

What would you pick - and WHY?

I'm accustomed to playground.soundOnDemand.js because that's Impact JS's convention, but the AirBnB JS Style Guide makes a good point:

If your file exports a single class, your filename should be exactly the name of the class.

// file contents
class CheckBox {
  // ...
}
module.exports = CheckBox;

// in some other file
// bad
var CheckBox = require('./checkBox');

// bad
var CheckBox = require('./check_box');

// good
var CheckBox = require('./CheckBox');

If you want to follow that convention, it seems that playground.SoundOnDemand.js would be the way to go.

Thanks @nknauth - I am going with AirBnB style guide - kinda...
PLAYGROUND.whateverYouPutThere
dictates:
playground.whateverYouPutThere.js


feat: add FindOrgCodeClaim extension + add IOrganisationWhiteList

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


Hold/unhold not updating sessionDescriptionHandler direction

Hello,
Using the events 'reinviteAccepted' and 'directionChanged' to try and trigger if hold / unhold was successful but the sessionDescriptionHandler.direction never gets updated with the 200 response which contains a=recvonly
Tested with version 0.11.3. Hold from the audio point of view is working correctly.
SIP/2.0 200 OK
Via: SIP/2.0/WSS k41ld7nu8s8e.invalid;rport=53419;received=xxx.xx.xxx.xxx;branch=z9hG4bK2987212
Call-ID: mi5gs3oqbclsp6ep19l1
From: "xxx" <sip:xxx@xxx>;tag=s87khiqgpg
To: <sip:xxx@xxx>;tag=90144b3a-2cd8-45d5-b1db-f94e22ab2012
CSeq: 7151 INVITE
Contact: <sip:xxx.xx.xxx.xx:8089;transport=ws>
Allow: OPTIONS, SUBSCRIBE, NOTIFY, PUBLISH, INVITE, ACK, BYE, CANCEL, UPDATE, PRACK, REGISTER, MESSAGE, REFER
Supported: 100rel, timer, replaces, norefersub
Server: Asterisk PBX 15.5.0
Content-Type: application/sdp
Content-Length:   947

v=0
o=-<PHONE_NUMBER> 5 IN IP4 xxx.xx.xxx.xx
s=Asterisk
c=IN IP4 <IP_ADDRESS>
t=0 0
a=msid-semantic:WMS *
a=group:BUNDLE audio
m=audio 10780 UDP/TLS/RTP/SAVPF 0 8 126
a=connection:existing
a=setup:active
a=fingerprint:xxx
a=ice-ufrag:30ea03f24624c1f034e934db46ee672b
a=ice-pwd:21f91f9731f740db061cb0ad04a2ca2f
a=candidate:H534966c7 1 UDP<PHONE_NUMBER> 2a04:e9c1:0:b::4 10780 typ host
a=candidate:H5bbe08d7 1 UDP<PHONE_NUMBER> fe80::21d:d8ff:feb7:1db1 10780 typ host
a=candidate:Hb95ecc44 1 UDP<PHONE_NUMBER> <IP_ADDRESS> 10780 typ host
a=rtpmap:0 PCMU/8000
a=rtpmap:8 PCMA/8000
a=rtpmap:126 telephone-event/8000
a=fmtp:126 0-16
a=ptime:20
a=maxptime:150
a=rtcp-mux
a=ssrc:2016684846 cname:349f2f0f-bbde-4782-9dda-e73f56cf4779
a=msid:a4d54a58-9d70-4bf1-a6f3-556958fbf0c7 3d8d322e-3282-434d-a857-d4c86b4325e0
a=mid:audio
a=recvonly

Also noticed that 'local_hold' on the session does not get reset on a fail, so cannot use this properly to determine if the session is on hold.
Let me know if you need more info.
Thanks!

What is the direction after getting the 200?

Sorry for delay in reply; the direction is sendonly
Which when reflecting I believe is correct on the reinviteAccepted because the server is saying it is only sending, not the client.
Happy to close this as not an issue if you agree.
Thanks

That was my thought. The direction should reflect the direction of the client, not what was received in the last message from the other endpoint. Thanks.


Expose the CURLOPT_HTTP09_ALLOWED libcurl option

This PR addresses my feature request in #436. It adds the ability to set the CURLOPT_HTTP09_ALLOWED option. Unfortunately there are still use cases for this option.
I took my best guess as to how to properly implement it. Please let me know if there's anything that needs to be changed, or even if exposing this option is acceptable.

Thanks! The implementation looks correct to me, just some minor suggestions.

@sagebind I've updated the method names as you suggested. Thanks for the quick feedback!


update readme

update readme
add architecture image
modify agent.properties item's comment

I add development-guide.md and user-manual.md to simplify readme