"Cookies" was not authenticated. Failure message: "Unprotect ticket failed"

Describe the bug
from time to time, I'm getting the message below in my logs, and unauthorised status, but for some requests.
"Cookies" was not authenticated. Failure message: "Unprotect ticket failed"
Code
I'm registering the DataProtection in this way
services
    .AddDataProtection()
    .SetApplicationName(Environment.ApplicationName)
    .PersistKeysToRedis(
        redisConn,
        $"DataProtection-Keys-{Environment.ApplicationName}-{Environment.EnvironmentName}");

Expected behavior
All the requests should not have the unprotect ticket failed issue
Additional context

<PackageReference Include="Microsoft.AspNetCore.Authentication.OpenIdConnect" Version="2.1.2" />
 <PackageReference Include="Microsoft.AspNetCore.DataProtection.Redis" Version="0.4.1" />
 <PackageReference Include="Microsoft.AspNetCore.Mvc.Versioning" Version="2.3.0" />
 <PackageReference Include="Microsoft.AspNetCore.Rewrite" Version="2.1.1" />
 <PackageReference Include="Microsoft.AspNetCore.Server.Kestrel.Https" Version="2.1.3" />
 <PackageReference Include="Microsoft.AspNetCore.SpaServices.Extensions" Version="2.1.1" />
 <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="2.1.1" />

Has your redis cache recycled or otherwise emptied?

Hi @blowdart
Yes, but during the night. Our Redis container was recreated and there was no volume for it. I guess the application will recreate the keys on redis in any additional request.

Besides that, in the screenshot above, you can see two requests, one successful call to the /configuration route and another one with the failure 401, which was in the same moment.
Is there a way to handle those failures when the Redis cache is recycled?

Make the cache persistent would be one way to go, but yea, if your cache empties and all the keys go, then this is rather expected


ed25519 SSH-Keys not working

Gitea version (or commit ref): 1.1.3
Git version: not relevant
Operating system: CentOS 6.8
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

Description
I can't add ed25519 SSH-Keys. Error is:
SSHKeyGenParsePublicKey: fail to parse public key: exec(4:SSHKeyGenParsePublicKey) failed: exit status 1 stdout: /tmp/gitea_keytest143073527 is not a public key file. stderr: line 1 too long

Not a bug? There is an option in app.ini to set ssh.minimum_key_sizes also for ED25519. If ED25519 is not supported it's a confusing option in app.ini

Hard to say, it has some support for it in code but have to check if this is bug or not fully supported

Pretty sure this is because OpenSSH on CentOS 6 is too old to support ED25519 keys.

So this is not a gitea's problem.


chore: Added Envy Proxy

Related Issue
Closes #1088

Description

Proxy server tool for managing and routing network traffic with enhanced security features
Screenshots

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

@Karmpatel960 is it free


Basic z-index support

If z-index is available, use that instead of the z-index derived from the plane
type.
Comes after #3.

Related: https://gitlab.freedesktop.org/wayland/weston/issues/275


Use persistent volumes for data storage

It seems like persistent volumes would be great for cassandra (It is even mentioned by name in the docs on mesos!):
http://mesos.apache.org/documentation/latest/persistent-volume/
Is there a plan to add support for persistent volumes?
What is required to make it work?

This repository is deprecated.  Please use the latest supported framework here:  https://github.com/mesosphere/dcos-cassandra-service.  It fully supports persistent volumes from Mesos.


Error when tables are empty

Do you sample data to be used with all 3 tables ? I found that the got error when electricity_data & pvdata are empty.

How to contribute to your code ?

I solved this problem by applying patch written by<EMAIL_ADDRESS>The page still can be accessed over this url :-
https://web.archive.org/web/20171118151750/http://infrontpower.com/solarpi/howto-en.txt


Release/3.1.0

3.1.0
New

Add ability to check that Pip and system packages have been patched. [Ben Dalling]

Other

Build(deps): bump paambaati/codeclimate-action from 8.0.0 to 9.0.0. [dependabot[bot]]
Bumps paambaati/codeclimate-action from 8.0.0 to 9.0.0.

Release notes
Changelog
Commits

updated-dependencies:

dependency-name: paambaati/codeclimate-action
dependency-type: direct:production
update-type: version-update:semver-major
...

Build(deps): bump pypa/gh-action-pypi-publish from 1.4.2 to 1.9.0. [dependabot[bot]]
Bumps pypa/gh-action-pypi-publish from 1.4.2 to 1.9.0.

Release notes
Commits

updated-dependencies:

dependency-name: pypa/gh-action-pypi-publish
dependency-type: direct:production
update-type: version-update:semver-minor
...

Build(deps): bump urllib3 from 2.2.1 to 2.2.2. [dependabot[bot]]
Bumps urllib3 from 2.2.1 to 2.2.2.

Release notes
Changelog
Commits

updated-dependencies:

dependency-name: urllib3
dependency-type: direct:production
...

Mistakenly raised a release request to the develop branch.


Tabs show home instead of webpage

Overview:
When switching tabs, sometimes the home screen rather than the expected webpage is displayed.
This seems to be a lifecycle management bug that happens when the activity is recreated. It is common for android to kill activities not in use to conserve resources and then recreate them again when needed. We should handle this elegantly.
To reproduce:

Turn on the "Don't keep activities" in developer settings
Open a few webpages in separate tabs and switch between them

Expected behavior:

Tab webpage loads when switching tabs

Actual behavior:

The home screen rather than the tab webpage loads

Internal reference task:
https://app.asana.com/0/414730916066338/1125173205413879

Thanks @donfuxx for submitting a PR, the fix for this will be included in the next release. 🎉


Add env variables for VSCode's dbt Power User

This pull request adds the necessary environment variables for dbt Power User to work outside of Docker. The variables are added to the terminal configuration for macOS and Windows. Additionally, the dbt profiles and project directories are specified, along with the paths for the DuckDB database files.
(My apologies to the reviewer for the horrible conditional attach path; I don't think there's a way to make it more elegant.)

Tested and wotrking even on Windows!


Update ValidationMessages_zh_CN.properties

fix DecimalMax and DecimalMin in ZH_cn Properties

Can one of the admins add this person to the trusted builders? (reply with: "add to whitelist" or "ok to test")

Thanks! I created a JIRA for it and adjusted the commit message.
Will wait for CI and merge!


Math Game

PR Description 📜
I have added a new game based on simple mathematics.
Fixes #1653

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you  @vishalkumarmahto , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

Hey @vishalkumarmahto,
You can't work on a closed issue 👀!
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗


As a user, I want to see available demographic and library information (birth/death dates, nationality, membership dates, link to VIAF if available) for an individual library member, so that I can find out more about the person.

testing notes
background
we'll design test this whole page later when more parts of it (the address map, membership timeline, etc.) have been designed and implemented. for now, if you notice anything flagrantly wrong with the design, feel free to add a comment.
instructions

visit the members list page and pick a member.
you should see the member's membership dates, birth/death date (if available), VIAF (if available), and nationality information (if available).
try repeating with some members with more information and verify that everything looks right. you can use ctrlf (or ⌘cmdf on mac) to search for a member's name on the list page since it's so long.

This is looking great!


Error: Publish function returned an array of non-Cursors

I'm trying to get joins in meteor mongo based on relation id.
When I used your package we are getting data and printed in console as well.
But I'm implementing it in publish method and getting below error.
Exception from sub usersWithAccount id ESzrf5p2AgHJttckF Error: Publish function returned an array of non-Cursors
So, my question is should I write differently in publish method?
Sharing sample code with you for better understanding.
Meteor.publish('usersWithAccount', function() {
var self = this;
var data =  Users.aggregate([{
$lookup: {"localField": "accountId","from": "account", "foreignField": "_id","as":"accountinfo"}
}],
Meteor.bindEnvironment(
function(err, result) {
// Add each of the results to the subscription.
_.each(result, function(e) {
self.added("usersWithAccount", e._id, e);
});
self.ready();
},
function(error) {
Meteor._debug( "Error doing aggregation: " + error);
}
));
return data;
});

same


Add user access page

Fixes
https://issues.redhat.com/browse/HAC-4819
https://issues.redhat.com/browse/HAC-4520
https://issues.redhat.com/browse/HAC-4819

Description

Add user access page
Add user access nav item
Make workspace switcher work with access page

Type of change

[x] Feature
[ ] Bugfix
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Screen shots / Gifs for design review

https://github.com/openshift/hac-dev/assets/20013884/5785fbb7-7bc5-400f-80bf-597bf41ac51b

How to test or reproduce?
Click on user access nav item.

Browser conformance:

[ ] Chrome
[x] Firefox
[ ] Safari
[ ] Edge

/cc @pankajshivpuje

/cc @karthikjeeyar @rohitkrai03

@rottencandy Can you also update your commit messages and PR title based on conventional commit message?

Tried it locally. Everything seems to be working well.


Search working in documents but not in the project Home.

Hello,
I've created a build of ReadTheDocs and have pretty much everything working.
Strangely I seem to be able to search in the document itself but a search in the doc management pages does not return any results.
I am seeing a search related  warning when I build (included below).
Do I need to create an index in Elastic for the combined index?  I am using commonmark… Would that cause a problem with the combined index?
Is this a known issue or have I configured something incorrectly?   It seems really strange that search would work in the document itself but not in the main site.
Please assist.
Thanks
Chris
[27/Apr/2016 15:18:18] ERROR [celery.worker.job:282] Task readthedocs.projects.tasks.update_search[e44ad4f9-074f-4931-b6fe-16829d29ff39] raised unexpected: RequestError(400, u'illegal_argument_exception', {u'status': 400, u'error': {u'root_cause': [{u'reason': u"Can't specify parent if no parent field has been configured", u'type': u'illegal_argument_exception'}], u'type': u'illegal_argument_exception', u'reason': u"Can't specify parent if no parent field has been configured"}})
Traceback (most recent call last):
File "/venv/local/lib/python2.7/site-packages/celery/app/trace.py", line 240, in trace_task
R = retval = fun(_args, *_kwargs)
File "/venv/local/lib/python2.7/site-packages/celery/app/trace.py", line 438, in protected_call
return self.run(_args, *_kwargs)
File "/app/readthedocs/projects/tasks.py", line 561, in update_search
delete=delete_non_commit_files,
File "/app/readthedocs/restapi/utils.py", line 144, in index_search_request
page_obj.bulk_index(index_list, parent=project.slug)
File "/app/readthedocs/search/indexes.py", line 140, in bulk_index
bulk_index(self.es, docs, chunk_size=chunk_size)
File "/venv/local/lib/python2.7/site-packages/elasticsearch/helpers/init.py", line 182, in bulk
for ok, item in streaming_bulk(client, actions, **kwargs):
File "/venv/local/lib/python2.7/site-packages/elasticsearch/helpers/init.py", line 124, in streaming_bulk
raise e
RequestError: TransportError(400, u'illegal_argument_exception')

Yeah, you need elastic search. Our docs go through the required setup there.

Thanks.. I think that there must be something that I am not understanding re the setup instructions.
I've got icu working but it looks like I also have to execute some python.  I'll try to figure out how to do that next.
Thanks
Chris

There is the same issue reported here #3803 with more up to date information and also there is a PR in progress for adding the documentation #3881
So closing this in favor of the other one


Get multilevel analysis working in CI

This is a preparatory step for #28.
We decided to abandon rpy2 as it did not want to play nicely with conda, and it was hard to know which version of R was being used.
Instead, a new R project has been added, to manage the R environment with renv, and the python code now uses the subprocess module to call the R script.

It might be worth seeing whether removing the rpy2 dependency fixes #34.


Error handling when there are no Timeline child configurations

Currently the code expects child configurations and if not configure has a bad SOQL exception. More gracefully handling this with a user message to correct would be better.

Fixed in https://github.com/anandbn/timeline-lwc/commit/d2555ff09ffece6c140583f84338eb8ad9185e63


@generate_interface bug : cannot use args...

To implement both simple and parameterized assembly at the same time I tried to do
@generate_interface function AtomsCalculators.potential_energy(sys, calc::SitePotential, args...; kwards...)  ...

this fails with
LoadError: Could not determine calculators type
in expression starting at ...

Here missing a function does this mean you tried to have f(x) = g(x) style statement?
The reason for this not working is that it would complicate the macro implementation and was left out from the implementation. It could be implemented, but it is not easy...

I did use the function style. It has nothing to with that.

Ah sorry I missread completely.
The macro checks that you do not have additional args.... This is a basic feature to inform you that your implenetation is not accordingly the interface.
When the lux model update lands you cannot use the macro implement this at all.
If you still want to have this in. Then you need to implement all by hand. But I advise against it.

No, I prefer we wait for the lux interface. It just means we will also wait porting the parameterized potentials until that is finally merged.

I'm happy to close this then.

If you want to use additional args... then implement a call without them and then additional one (without macro) that has args... that is the safest way.

I would prefer to not maintain 10 copies of the same code.


Handle the two different funding rates in the taker UI (at the moment still global, has to be long/short specific when opening an order)

closed by #1582


Can't Set Interpreter with Remote SSH

Can't set the Python interpreter path when using remote SSH.
VS Code 1.45.1
ms-python.python 2020.5
I see #79, and #5612, and #5724 all appear to be about the same thing. But my impression from them is that this should be resolved.
Is this fixed? Is there a work-around somewhere in those issues that I'm missing?
Thanks!

Alright, so most importantly I get the impression this should work.
(Or at least that those other issues were indeed resolved.)
And thanks for pointing me to the Python-Extension output log. It looks like it goes trying a handful of places looking for Python, including the path I have set in settings.json / python.pythonPath:
> conda --version
> pyenv root
> python3.7 ~/.vscode-server/extensions/ms-python.python-2020.5.86806/pythonFiles/pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> python3.6 ~/.vscode-server/extensions/ms-python.python-2020.5.86806/pythonFiles/pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> python3 ~/.vscode-server/extensions/ms-python.python-2020.5.86806/pythonFiles/pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> python2 ~/.vscode-server/extensions/ms-python.python-2020.5.86806/pythonFiles/pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> python ~/.vscode-server/extensions/ms-python.python-2020.5.86806/pythonFiles/pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> ${MY_CONFIGURED_PATH}/python3 ~/.vscode-server/extensions/ms-python.python-2020.5.86806/pythonFiles/pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> conda info --json

Where the second to last is the one I want.
Attempting to run that command stand-alone generates errors looking for libpython:
python3: error while loading shared libraries: libpython3.7m.so.1.0: cannot open shared object file: No such file or directory

The installation is on a shared Linux server, not in a Conda or Virtualenv. It's configured through a number of environment variables, including adding libpython via $LD_LIBRARY_PATH.
After seeing this failure, I added a python.envFile setting which includes the necessary variables. It does not appear to help. Is this envFile only loaded after Python starts up? And if so, is there another way to set "pre-launch" environment vars?

I also tried adding these as env settings to the launch.json configuration.
No luck there either.

Attempting to run that command stand-alone generates errors looking for libpython:

If pasting the commands in terminal doesn't work, the issue is not related to the extension. Can you share a screenshot?
Also please share your settings.json and env file contents. I am assuming ${MY_CONFIGURED_PATH}/python3 is the one you attempted to select. Try using the full path to python3 where ${MY_CONFIGURED_PATH} is already resolved, and see if that works.

Sorry I could have been more clear here.

Command-line attempts to run > ${MY_CONFIGURED_PATH}/python3 ~/.vscode-server/.../pyvsc-run-isolated.py without a proper set of environment-variables fail to find libpython.
With a correct set of env-vars - particularly LD_LIBRARY_PATH - running the same command prints out the sys.executable path. It appears this is what the extension is looking for while configuring the interpreter.

> ${MY_CONFIGURED_PATH}/python3 ~/.vscode-server/extensions/ms-python.python-2020.5.86806/pythonFiles/pyvsc-run-isolated.py -c "import sys;print(sys.executable)"

${MY_CONFIGURED_PATH}/python3

In short, command-line usage works with the right env-vars, and fails without them.

When invoking via the extension, i.e. by hitting Run -> Start Debugging, the extension pops this up:

You need to select a Python interpreter before you start debugging. Tip: click on "Select Python Interpreter" in the status bar.

This is despite having:

(a) python.pythonPath configured to MY_CONFIGURED_PATH/python3 in settings.json (set  via the same GUI pop-up)
(b) a python.envFile configured with the necessary variables, and
(c) env also configured with the necessary variables in launch.json

My impression is that the pyvsc-run-isolated.py / sys.executable check may not set these variables before running, causing python to fail.

Try using the full path to python3 where ${MY_CONFIGURED_PATH} is already resolved, and see if that works.

It's set to the full path; I'm just using the "MY_CONFIGURED_PATH" notation here in the issue.
Thanks again.

Can you set all these environment variables in the system? I think it'll be picked up if you do that. Meanwhile, I will double check if the shell commands like
${MY_CONFIGURED_PATH}/python3 ~/.vscode-server/extensions/ms-python.python-2020.5.86806/pythonFiles/pyvsc-run-isolated.py -c "import sys;print(sys.executable)"

pick the system environment variables.

It's set to the full path; I'm just using the "MY_CONFIGURED_PATH" notation here in the issue.

The notation makes it seem like you're using an environment variable MY_CONFIGURED_PATH
I also suspect you may not be using python.envFile setting correctly, it's meant to point to a .env file, which has the environment variables. I would need you paste those for me, and also the directory structure of your project.

I guess I regret using the env-style notation here in the issue to describe the path. Seems it has caused some confusion. How about we instead refer to it as /my/configured/path/python3 . This more accurately reflects what's in settings.json (a path literal).
Settings:
{
    "python.pythonPath": "/my/configured/path/python3",
    "python.envFile": "${workspaceFolder}/.env"
}

Again I'll note: just launching my/configured/path/python3 requires some of the variables in ${workspaceFolder}/.env.
So perhaps some more detailed questions:

Is envFile loaded before or after the Python interpreter starts?
Does envFile support variable expansion, e.g. VAR2 = ${VAR1}:/other/
Related, does envFile support multiple assignments, e.g.:

PATH = /one/path
PATH = ${PATH}:/another/path

Sorry I'm not going to be able to share the contents of the .env or project structure.

Is envFile loaded before or after the Python interpreter starts?

The contents from envfile are loaded before any commands you see in the output channel are executed

Does envFile support variable expansion, e.g. VAR2 = ${VAR1}:/other/

Yes it does. But you have to refer system variables and variables defined in the env file differently, as per the docs.

Related, does envFile support multiple assignments, e.g.:

Yes, it does.
You can check if the environment variables are set correctly in .env file, by debugging a python file which prints the environment variables. Let me know if you need help with this.

Thanks very much for clarifying and digging into this.
It looks like the trouble was from bash-style default-value syntax, e.g.
LD_LIBRARY_PATH=/some/path:/another/path:${LD_LIBRARY_PATH:-}

Where the problem is the :- at the end. Removing all of these gets the extension/debugger up and running.
It's not perfectly clear whether this should work. My own trial of the python-dotenv package supports this; perhaps other dotenv parsers do not.
Up to you whether it's an issue to dig into (i.e. you expect this to work). If not, I can close.

I am not sure if we resolve these ourselves or use some package, but I don't think we can prioritize this rn, so please go ahead and close this. Glad you got it working :)

Actually have to walk back my purported cure here.
The env-vars required to start the Python interpreter (particularly LD_LIBRARY_PATH) need to be set when the VS-Code server is started.
Just having them in .env doesn't appear to cut it.


using relative path of pub key and fixed adobe domain to fetch pub key

Description

Related Issue

https://jira.corp.adobe.com/browse/CI-5221

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Also, let's only merge once we can test end-to-end with the new domain.

Also, let's only merge once we can test end-to-end with the new domain.

Yes


Firefox devtools path fixes

In Firefox 44 the paths to internal devtools assets changed. GA Debugger uses several devtools modules and stylesheets to ensure the UI looks and behaves identically to the other built-in tools.
This fix introduces a set of helpers for mapping the new asset urls (defined in FF44) to the old format, allowing GA debugger to work in all current versions of Firefox.
Unfortunately, the theme-switching.js file in tool.xul could not be loaded dynamically using a helper so both the new and old urls have been included with <script> elements. Firefox will try and load both these assets even though only one can ever exist — triggering a browser console warning.
Once pre-44 versions of Firefox have phased out the helpers should be removed and extension install.rdf updated to set FF44 as a minimum requirement.
This fix bumps GA Debugger to 2.1.1, but only Firefox users will get this version.
This PR fixes issue #28

Uploaded to AMO. Waiting for review prior to merge.


feat: split and run grpc_server, ota_core and otaclient main as separated processes

Introduction

[!NOTE]
This PR is the core PR and the final piece of the whole otaclient re-architecture project.

This PR introduces a re-architecture of otaclient, now at runtime each core otaclient components run as standalone processes and utilize IPC for handling OTA request:

main process(by main module): the daemon process that first sets up the queue and shared_memory for each components to work together, and then brings up each component processes one by one. During otaclient running life-cycle, it will monitor each components status(active or exited) and handle signals(SIGTERM, SIGINT) accordingly.
ota_core process (by ota_core module): the process running the core implementation of OTA. It handles the OTA request and actually do the OTA, while reports its status via shared_memory all the time.
grpc_server process (by otaclient.grpc.api_v2 package): the prcoess running OTA Service API grpc server, it handles the OTA API requests, and translates it into otaclient internal form and dispatches to ota_core process via IPC.
ota_proxy process (by ota_proxy package): the process running ota_proxy server, will only be brought up when there is active OTA within the cluster.

A simple otaclient internal IPC interface is implemented using simple types defined in otaclient._types with queue between API grpc server and ota_core. grpc_server dispatches OTA requests down to ota_core(with op_queue) and get the response to the request from ota_core(with ack_queue).
A cros-process status report mechanism is implemented based on sharing hmac-protected(with one-time preshared key) pickled object with shared_memory. ota_core uses this mechanism to write its latest status into the shm, and grpc_server process read this shm to track the ota_core's internal status, and translate it into the OTA grpc API format.
Docs
For the detailed design documentation, see https://tier4.atlassian.net/l/cp/vxCQ0qp0.
Tests

[x] otaclient can cleanly exit on SIGTERM(by kill) and SIGINT(by ctrl-c) without dangling.

[x] at startup
[x] during OTA

[x] OTA e2e test with VM passed.
[ ] pytest

https://tier4.atlassian.net/browse/RT4-12276

Start to implement test from this point.

Need to add some tests for the main process.


Confusing/Inconsistent error message with circular type definition

TypeScript Version:  2.9.1

Search Terms:
flatMap is:open
Code
type PlainObjectItem = string | number | boolean | null

interface PlainObject {
	[key: string]: PlainObjectItem | PlainObjectItem[] | PlainObject
}

const hoge: PlainObject = {
	key1: 'hoge',
	key2: {
		key21: 'aaa',
		key22: false,
		key23: {
			key231: [1,2,3,4,5],
			date: new Date()
		}
	}
}

Expected behavior:
Error message is something like 'new Date()' is not assignable to type 'PlainObjectItem | PlainObjectItem[] | PlainObject'.
Actual behavior:
Property 'flatMap' is missing in type '{ key21: string; key22: false; key23: { key231: number[]; date: Date; }; }'.
Playground Link:
http://www.typescriptlang.org/play/#src=type PlainObjectItem %3D string | number | boolean | null

interface PlainObject {
	[key%3A string]%3A PlainObjectItem | PlainObjectItem[] | PlainObject
}

const hoge%3A PlainObject %3D {
	key1%3A 'hoge'%2C
	key2%3A {
		key21%3A 'aaa'%2C
		key22%3A false%2C
		key23%3A {
			key231%3A [1%2C2%2C3%2C4%2C5]%2C
			date%3A new Date()
		}
	}
}
(This example says Property 'length' is missing...)
Related Issues:

accidentally submitted. writing right now

If this is working as intended, feel free to close, but I'm not smart enough to understand that message.

I think the error message is confusing.
Adding the Date type at PlainObjectItem solve the error:
type PlainObjectItem = string | number | boolean | null | Date

interface PlainObject {
	[key: string]: PlainObjectItem | PlainObjectItem[] | PlainObject
}

const hoge: PlainObject = {
	key1: 'hoge',
	key2: {
		key21: 'aaa',
		key22: false,
		key23: {
			key231: [1,2,3,4,5],
			date: new Date()
		}
	}
}

Playground link

Just in case, the point of this issue is not solving the error, but the confusing error message itself.

@DanielRosenwasser As part of the error message improvement push, we should track the general category of choosing the "best" constituent of a union when reporting non-assignability.
Currently we throw out things that are disproven by a discriminant property, but we could probably do better by attempting to throw out candidate targets that seem unfixably incompatible:

The target has a numeric index signature or a call signature and the source doesn't
The target is a primitive and the source isn't
The target is an object and the source isn't

IOW the "ideal" error message always refers to a missing property or a property with the wrong type, or relates two primitives.
But probably everything I said up there is disproved by a different example where this generates manifestly worse error messages.

What the... even after our improvements, the current message is
Type '{ key21: string; key22: false; key23: { key231: number[]; date: Date; }; }' is not assignable to type 'string | number | boolean | PlainObject | PlainObjectItem[]'.
  Type '{ key21: string; key22: false; key23: { key231: number[]; date: Date; }; }' is not assignable to type 'string'.ts(2322)

Well, uh, I've got it to be less dumb but longer.
error TS2322: Type '{ key21: string; key22: false; key23: { key231: number[]; date: Date; }; }' is not assignable to type 'string | number | boolean | PlainObject | PlainObjectItem[] | null'.
  Type '{ key21: string; key22: false; key23: { key231: number[]; date: Date; }; }' is not assignable to type 'PlainObject'.
    Property 'key23' is incompatible with index signature.
      Type '{ key231: number[]; date: Date; }' is not assignable to type 'string | number | boolean | PlainObject | PlainObjectItem[] | null'.
        Type '{ key231: number[]; date: Date; }' is not assignable to type 'PlainObject'.
          Property 'date' is incompatible with index signature.
            Type 'Date' is not assignable to type 'string | number | boolean | PlainObject | PlainObjectItem[] | null'.
              Type 'Date' is not assignable to type 'PlainObject'.
                Index signature is missing in type 'Date'.


Provide the validation enhancements for all Jobs without queue-name as alternative of manageJobsWithoutQueueName

What would you like to be added:
I would like to provide the validation feature for all Jobs without queue-name (requiredJobsWithQueueName), which rejects the Jobs without queue-name.
However, the validations will be implemented as VAPs, and we never implement the custom validation mechanism as webhook and other APIs.
If the cluster administrator enables the requiredJobsWithQueueName in the Kueue Configuration API, the kueue-controller-manager will create the VAPs for all enabled Jobs.
Why is this needed:
The manageJobsWithoutQueueName automatically suspends all Jobs without queue-name to avoid the quota violation.
But the feature does not have good UX since it's challenging for batch users to find the reason why their Jobs are not runnable.
So, I would like to provide a way that they can easily find the reason when they submit Jobs.
Completion requirements:
This enhancement requires the following artifacts:

[x] Design doc
[ ] API change
[ ] Docs update

The artifacts should be linked in subsequent comments.

This is under discussion. So, any can not take this one.


Optimize a3p container for low-end computers

The a3p container has become a standard tool for local development. Our getting-started dapps examples utilize this container for building and testing dapps locally. However, a3p is very resource-intensive, consuming significant memory and CPU, which can overwhelm most computers.

[ ] Reduce the memory required to run a3p.
[ ] Reduce CPU consumption.
[ ] Reduce the image size.

Our goal is to minimize the resource requirements for this container so that any new developer, regardless of their resources, can run it on their computer without needing to invest in a high-end laptop.
The issue was originally raised by new contractors, doing DevRel and engineering work, who do not have high-end computers.

... If you want to limit memory usage, we need to switch to a "host binary approach" for dapp development ...

one way to do it is in the wish list:

https://github.com/Agoric/agoric-sdk/issues/8033


Consider updating CONTRIBUTING.MD for PR issue association rule

See also https://github.com/tikv/tikv/issues/11016 & https://github.com/tikv/community/issues/149

/type question


select the least loaded agent worker for job dispatch

I think load balancing needs a bit of work.. but I don't think we'd want to load on the least loaded worker.
this workload is similar to Egress, where we'd ideally want to concentrate work on the top N% of the most loaded (but still available) workers.

one problem here is that agents by default are only updating their availability every 10s so a burst of jobs could overload the selected worker pretty quickly.
https://github.com/livekit/agents/blob/main/livekit-agents/livekit/agents/worker.py#L41
If efficient packing is the goal, maybe decreasing the worker update interval is a better approach for now?

one problem here is that agents by default are only updating their availability every 10s so a burst of jobs could overload the selected worker pretty quickly. https://github.com/livekit/agents/blob/main/livekit-agents/livekit/agents/worker.py#L41
If efficient packing is the goal, maybe decreasing the worker update interval is a better approach for now?

you are right that we should probably not make efficient packing a goal. with that in mind I think your approach is better. but still could be prone to overload if a few jobs come in during that 10s.  I know @paulwe is working on a better scheme for this, but for now, wdyt about

on the agent-side, when load metrics change, update immediately
on the server-side, distribute probabilistically, inversely proportion to load

that works for me, i'll refactor this and add a pr on the agent-side

closing this out as there is some other refactoring work happening here

added a few tests here. @paulwe for these unit tests, i short-circuited some of the simulated worker abilities to make sure they returned more quickly.


Document how to use Json<T> with the query macros

Bug Description
The Json support for postgresql only works through the usage of the Value type. The Json type when used as show in the documentation doesn't work : https://docs.rs/sqlx/latest/sqlx/types/struct.Json.html
I've checked the source and I have the feeling that the Json module is not covered by tests, it may have been broken in one refactoring.
Minimal Reproduction
I've made a minimal repo that repro the compiler error.
https://github.com/ElysaSrc/repro-sqlx-json-bug
Info

SQLx version: 0.7.4
SQLx features enabled: ["runtime-tokio","tls-rustls","postgres","chrono","migrate","uuid","json"]
Database server and version: Postgres 14
Operating system: Nixos unstable - Linux 6.7.9
rustc --version: rustc 1.77.0 (aedd173a2 2024-03-17)

Not a bug. The query macros map JSON columns to serde_json::Value by default. You have to tell the macro to use the Json adapter using the type override syntax: https://docs.rs/sqlx/latest/sqlx/macro.query.html#force-a-differentcustom-type
See this example (uses query!() but the concept is the same): https://github.com/launchbadge/sqlx/blob/main/examples/postgres/json/src/main.rs#L80
Note also that query_as!() does not currently use FromRow as stated in the docs: https://docs.rs/sqlx/latest/sqlx/macro.query_as.html
Leaving this open as a documentation issue.

Hey there, would be open to take this on. @ElysaSrc, where would you have wished to find the hint regarding the need to use the Json adapter?

I just ran into this myself. It would be nice if the docs for the Json<> type mentioned that you need to do the type override thing, with an example. From the examples on that page, I assumed it would work with the macros, and was surprised when it didn't.

@dyc3 I took a shot at writing down what you recommended. Is it helpful?

Yes! That's exactly what I was looking for.

#3447 would have been what I was looking for too! Thanks, sorry for the delay to respond.

Thanks for the Feedback! I will fix the doctests when I get home today so that the PR is mergeable.


Logs from platform not showing platform name

Describe The Bug:
The logs being written from the platform class appear to be missing the indicating they're coming from the platform name.
To Reproduce:
Start up the plugin, and review the logs.
Expected behavior:
All logs coming from the platform when a custom name is used should begin with: [Homebridge Automower Platform - Developer 1]
Logs:
[11/19/2022, 10:02:24 AM] [Developer 1] Initializing Homebridge Automower Platform platform...
[11/19/2022, 10:02:24 AM] [Developer 1] Configuring 'Dobby' from the accessory cache.

Plugin Config:
{
    "name": "Developer 1",
    "lang": "en",
    "appKey": "REDACTED",
    "authentication_mode": "client_credentials",
    "application_secret": "REDACTED",
    "platform": "Homebridge Automower Platform"
}

Environment:

Plugin Version: v1.3.1
Homebridge Version: v1.5.1
Node.js Version: v16.15.1
Operating System: Windows

Instead of changing how the platform logged, I opted to just have the plug-in match what Homebridge itself was doing since the plug-in logger was supposed to be an imitation.


Getting mail for everylogin from different location, is this security issue?

We noticed a new login, piyushkv1 We noticed a login from a device you don't usually use.

Samsung SM-G930F · Instagram · Lucknow, India | Samsung SM-G930F · Instagram · Lucknow, India | Samsung SM-G930F · Instagram · Lucknow, India
I don't own this device. Is this security issue?

I tried to use and got a message like this from my location. "Samsung SM-G930F · Instagram · São Paulo, SP, Brazil"

after using this?


Feature/us315 fap link postcode search

Amend Href for FaP

:+1:

@Pheonnexx The beta image appears to have been added twice, was there a reason for this?

Second image in templates has been removed.


integrating test with new backend endpoints

[x] support fetch product status by order id
[x] test multiple vendor backends, eg. HGC, DCC and etc.
[x] test privacy TXs

* Preparing request to https://xxx.qlcchain.online/
* Current time is 2021-02-04T02:26:51.738Z
* Using libcurl/7.69.1 OpenSSL/1.1.1g zlib/1.2.11 brotli/1.0.7 libidn2/2.1.1 libssh2/1.9.0 nghttp2/1.41.0
* Using default HTTP version
* Disable timeout
* Enable automatic URL encoding
* Enable SSL validation
* Enable cookie sending with jar of 1 cookie
* Found bundle for host xxx.qlcchain.online: 0x7fe3365eeea0 [can multiplex]
* Re-using existing connection! (#1) with host xxx.qlcchain.online
* Using Stream ID: 3 (easy handle 0x7fe33d911c00)

> POST / HTTP/2
> Host: xxx.qlcchain.online
> user-agent: insomnia/2020.5.2
> cookie: __cfduid=db736e43dad0dc6dc6c6fb2148899e5ea1610593572
> content-type: application/json
> accept: */*
> content-length: 1420

| {
| 	"jsonrpc": "2.0",
| 	"id": 3,
| 	"method": "DoDSettlement_getCreateOrderBlock",
| 	"params": [
| 		{
| 			"buyer": {
| 				"address": "qlc_141nimruqodrj6h94g51dz64kgdz1ugp7jdc1p4ub4a5jqqjtzi1yjd1gmhe",
| 				"name": "Buyer"
| 			},
| 			"seller": {
| 				"address": "qlc_37wwtnrewtpieogxfb3mmpxr5ws76cbk43o4eu9ouopucijdt4ahe9xxgxah",
| 				"name": "HGC"
| 			},
| 			"connections": [
| 				{
| 					"itemId": "PRODUCT-dd26d7bb86c44373b24dec846a689522-2",
| 					"productOfferingId": "dd26d7bb86c44373b24dec846a689522",
| 					"quoteId": "2a6f290483c843d58cd15560c5696bb8",
| 					"quoteItemId": "2",
| 					"buyerProductId": "PRD00FXUZY1GCVE",
| 					"connectionName": "Custom Project Name 03",
| 					"srcCompanyName": "srcCompanyName",
| 					"srcRegion": "srcRegion",
| 					"srcCity": "srcCity",
| 					"srcDataCenter": "srcDataCenter",
| 					"srcPort": "srcPort",
| 					"dstCompanyName": "dstCompanyName",
| 					"dstRegion": "dstRegion",
| 					"dstCity": "dstCity",
| 					"dstDataCenter": "dstDataCenter",
| 					"dstPort": "dstPort",
| 					"paymentType": "invoice",
| 					"billingType": "DOD",
| 					"bandwidth": "10 Mbps",
| 					"bandwidthNum": "10",
| 					"billingUnit": null,
| 					"startTime":<PHONE_NUMBER>,
| 					"endTime":<PHONE_NUMBER>,
| 					"serviceClass": "bronze",
| 					"price": 254,
| 					"currency": "USD"
| 				}
| 			],
| 			"privateFrom": "ariknq5LszK+z2fCkNJ1ULSgh6cf0xG1S3vUPV55Zzg=",
| 			"privateFor": [
| 				"4bUt2v3VOLRZI5uO2OyBvOfi16eJVQoKT++s1JunCTo="
| 			]
| 		}
| 	]
| }

* We are completely uploaded and fine

< HTTP/2 200
< date: Thu, 04 Feb 2021 02:26:52 GMT
< content-type: application/json
< content-length: 1107
< vary: Origin
< cf-cache-status: DYNAMIC
< cf-request-id: 080c76358d0000e26c4c338000000001
< expect-ct: max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"
< report-to: {"max_age":604800,"endpoints":[{"url":"https:\/\/a.nel.cloudflare.com\/report?s=EJoAop2stOB4gL011xL3jFGUxt8Qwp%2Bnd6ww6pjXbMJtLL0h7tpPUiAFOn95r7l%2BH36MHKwgxvv6bs1J5fMhYJN9I2i5U%2BPSUxiteTutzgjUDOaow%2Fq4mw%3D%3D"}],"group":"cf-nel"}
< nel: {"max_age":604800,"report_to":"cf-nel"}
< server: cloudflare
< cf-ray: 61c0f3027f3fe26c-SIN

* Received 1074 B chunk
* Received 33 B chunk
* Connection #1 to host xxx.qlcchain.online left intact


[WIP] [BugFix] Fix memory leak in LocalExchange (#20885) (#20888)

local exchange will leak _name and _memory_manager because of without define virtual in destructor
(cherry picked from commit 5a7837c0f36a983d7db1d6fafb0c620d3983384d)

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


How to support go?

I use gore to repl, i set this, but didn't work.
local iron = require('iron')

iron.core.add_repl_definitions {
  go = {
    mycustom = {
      command = {"gore"}
    }
  }
}

iron.core.set_config {
  preferred = {
    go = "gore",
    python = "ipython",
    typescript = "ts-node",
    javascript = "node"
  }
}

open a go file ,then run repl , nothing happed.

Hi, thanks for the issue.
You almost got it right, on the preferred configuration, you have to add the name of the definition you just added above, in your case mycustom. That's what's going to be used to retrieve the repl configuration.
Best regards,
Henry

@hkupty  so what should i do? i don't write lua . thanks for your reply.

No worries. If you change your definitions to have the key named gore, it should work:
iron.core.add_repl_definitions {
  go = {
    gore = {
      command = {"gore"}
    }
  }
}

@hkupty  Fine, i got it work, But the repl window is very strange, the repl window is very large, it should be half the size,It seems that providing an option to set the position and size of the window is a good way.

@hkupty  Hi,any update for window?

Sorry I missed this issue. I don't mean to make things stalled, but life has been very busy and I'm unfortunately unable to give the proper attention to the project.
Iron exposes it's configuration through a lua variable that can be set like this:
iron.core.set_config {
    repl_open_cmd = "topleft vertical 50 split"
}

The default value is   repl_open_cmd = "topleft vertical 100 split".
Once again, I'm really sorry I lost track of this issue.
Best regards,
Henry

Don't worry, I've solved it recently, everything works fine

Glad to know :)


Properly quote unix launch scripts

This is a classic bash bug. Without quotes, any spaces in the directory that kaleido is installed into will cause the commands to split into multiple arguments. This change fixes #78, a long-standing (3 years) and annoying issue for people using per-project venvs with spaces in the project name.

Don't worry!
El vie, 26 de jul de 2024, 4:14 p. m., Silico_Biomancer <
@.***> escribió:

@.**** commented on this pull request.
In repos/mac_scripts/build_kaleido
https://github.com/plotly/Kaleido/pull/199#discussion_r1693559387:

if [ ! -f "out/Kaleido_mac_$KALEIDO_ARCH/kaleido" ]
then
echo "Error: Kaleido executable was not built";

exit 1

Indeed it is, terribly sorry. Quotes should be the only changes!
—
Reply to this email directly, view it on GitHub
https://github.com/plotly/Kaleido/pull/199#discussion_r1693559387, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHHLRFIMXI6EDANB6WJTXW3ZOKU35AVCNFSM6AAAAABLNP4XTGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDEMBSGY4DCOJYGM
.
You are receiving this because you commented.Message ID:
@.***>

Fixed that now. I also see my editor automatically did some formatting and created errant newlines, apologies for that extra messiness in the diff.


SQLITE ERROR on build a184473

Got past that error but now I get this when I build against a184473
`{ Error: SQLITE_ERROR: unrecognized token: "{"
at Error (native) errno: 1, code: 'SQLITE_ERROR' }
Gekko encountered an error and can't continue
Error:
DB error at get
`

same (with sqlite 3.1.3 or 3.1.4:
2016-07-09 10:22:21 (INFO):
2016-07-09 10:22:21 (INFO):     Setting up:
2016-07-09 10:22:21 (INFO):              Trading Advisor
2016-07-09 10:22:21 (INFO):              Calculate trading advice
2016-07-09 10:22:21 (INFO):
{ Error: SQLITE_ERROR: unrecognized token: "{"
at Error (native) errno: 1, code: 'SQLITE_ERROR' }
Gekko encountered an error and can't continue
Error:
DB error at get
Meta debug info:
Gekko version: v0.3.1
Nodejs version: v6.3.0

I just deleted the history folder and it works.

Could you send me the database file of the exchange you are trying to use? Also what OS are you on?

I'm running on Ubuntu 14, inside a node docker container.  This is the dockerfile configuration:
`FROM node:6
RUN mkdir -p /usr/src/gekko
WORKDIR /usr/src/gekko
#Download & setup Gekko trading bot
RUN git clone -b develop https://github.com/askmike/gekko.git .;
git reset --hard a1844738d86ea44797e993bef8e0c2a49e428463;
npm install
#Enable redis plugin
RUN npm install<EMAIL_ADDRESS>CMD [ "node", "gekko" ]`

Hmm.. I'm unable to reproduce that error at the moment.  I think I'm actually going to change the docker container so that you can run inside the container more easily without the gekko process running at the same time.  You can leave this closed for now.  Thanks.

https://www.dropbox.com/s/gonqii1ypz74edt/bitfinex_0.1.db?dl=0

All is working after renaming/deleting history folder after changing from nodejs5 to nodejs6 and git pull

Very weird, I am not able to reproduce. Could it be caused by not running npm install after upgrading nodejs (and thus running an binary build for an old verison of nodejs against node6)?

@Niller2005 could you share me your config as well, together with an output of npm list? As well as when was the time you last ran npm install (before or after upgrading nodejs)? I am closing for now, as I am unable to reproduce.

Config: http://pastebin.com/WjUSkHn8
npm list: http://pastebin.com/eKu4FDPr
npm install: after upgrading

I am having this issue as well.  I am currently on commit aa169f7851a2ab668882b8f2be7d1f8caedfadbc.
It happens for me whenever I run Gekko, stop it (Ctrl-C), then re-run it.
Config
Log (Console Output)
npm list
I ran npm install after pulling to this commit.
If I delete my history folder and try running, it works fine again (until stopping and re-running again).

@eauel are you also on ubuntu?

@askmike I am running ArchLinux on a Raspberry Pi (armv6h)
uname -r
4.4.14-1-ARCH

That's probably similar to me in that I use docker and often stop the
container without thinking about properly stopping the app.  Does node
listen to HUP requests?
On Sunday, July 10, 2016, eauel<EMAIL_ADDRESS>wrote:

@askmike https://github.com/askmike I am running ArchLinux on a
Raspberry Pi (armv6h)
uname -r
4.4.14-1-ARCH
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/askmike/gekko/issues/375#issuecomment-231610719, or mute
the thread
https://github.com/notifications/unsubscribe/ACU0bQWUekASat0XIEBBYMk9RufjMfFBks5qUV4DgaJpZM4JIh4L
.

I figured it out! I did not properly update a codepath which is seldom walked after changing the way historical data from the sqlite database was passed around. I tested a lot using the backtesting mode, however I didn't properly test this..

Just updated and tried running it again, and so far it's working well; thanks!

Same.
On Sun, Jul 10, 2016 at 8:56 PM, eauel<EMAIL_ADDRESS>wrote:

Just updated and tried running it again, and so far it's working well;
thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/askmike/gekko/issues/375#issuecomment-231621762, or mute
the thread
https://github.com/notifications/unsubscribe/ACU0bRhvhQwIgDXgMJwIDAKmMtmM9kuAks5qUZSygaJpZM4JIh4L
.

Updated and worked.
Is it possible to reimport the old history base wich i have renamed to made it work as say before, to the new actual base?

Awesome!

Is it possible to reimport the old history base wich i have renamed to made it work as say before, to the new actual base?

You can rename it back and Gekko will start using it (there was a bug in the query code, the data itself is not corrupted).
As for combining all data: Gekko has no way to merge sqlite databases. But if you really want to (so you can backtest etc), let me know and I'll write a quick script that copies all data from one into another.

I think it can be not usefull, because it was a 'long' time between the moment i've backed it and started with an empty history folder. So it will be missing almost 1 day of datas.


PowerShell DSC 5.1 report filter API doesn't work

I setup the WMF 5.1 and DSC report server 5.1, I tried to do the filter to return the specified number of DSC report entry data using the below similar request URL, howerver, it doesn't work.
Step1 $count filter worked well by invoking the below URL http://pullserver.example.com:8080/PSDSCPullServer.svc/Nodes(AgentId='2EF12B0F-E2C2-11E6-9A0F-9A320D9CC830')/Reports/$count and it return 1141.
Step2. I used the below URL by adding filter $top=2, howerver, the DSC report server still returned all of 1141 records rather than 2 records. http://pullserver.example.com:8080/PSDSCPullServer.svc/Nodes(AgentId='2EF12B0F-E2C2-11E6-9A0F-9A320D9CC830')/Reports?$top=2
Is there guy who can help me? Thanks in advanced.

$top has not been implemented on the server side, unfortunately. You can, however, filter the results in PS once the results are returned.
Have you looked at using Azure Automation DSC? It provides very robust reporting functionality.
Since this is not an issue with DscResources, please post questions like this on UserVoice: https://windowsserver.uservoice.com/forums/301869-powershell/category/148047-desired-state-configuration-dsc
MarkG

Much appreciated for your quick reply. We can't access to internet due to our limited network environment.


error with transcribing (with the default tiny model)

update 7/23
This is potentially due to file path incorrectly redirected. I am trying to test it both locally and on server again now. Will close this if succeed.
Even though whisper image can be pulled on local machine, and an input.mmif can be successfully generated, the output.mmif has an error. The locally tested video file is cpb-aacip-122a5fd2dcc.mp4. from child

Just for the future, could you not upload screenshots, but instead copy and pates actual terminal text, so that other people can copy and paste from the report for testing? Thanks!

I think b9a423a04e9f3bf9c89bef48343f2ab0473b6d41 should solved this problem. @Brendayy can you try again with v3, and see if the problem persists?

update 7/24
After trying v3 (b9a423a), ffmpeg error disappeared, but now with new error locating the video file after mapping the local directory to container. For example, I first run
podman run --name whisper_test -v /home/yangyangchen/sample_video:/data -p 5000:5000 --rm ghcr.io/clamsproject/app-whisper-wrapper:latest

assuming videos locate at /home/yangyangchen/sample_video on child.
after server is up, running:
clams source audio:/data/cpb-aacip-122a5fd2dcc.mp4 > cpb-aacip-122a5fd2dcc_input.mmif
curl -H "Accept: application/json" -X POST<EMAIL_ADDRESS>-s http://localhost:5000 > cpb-aacip-122a5fd2dcc.mmif

The error message is shown as:

If the file is there in the directory for sure, it's probably file permission issue with rootless podman. Try :z or :Z suffix with the volume argument, e.g., podman run ... -v /home/..../:/data:z ...

Closing as stale (and probably the error is caused from some ill-set container env).


[BUG] OpenImageIO/strutil.h macro pollution

Describe the bug
OpenImageIO/strutil.h includes OpenImageIO/detail/farmhash.h, which defines macros for many common names, including fmix. This pollutes the namespace, breaking client code (in this case, Gaffer).
To Reproduce
Compiling this code :
#include "OpenImageIO/strutil.h"

namespace Test
{
    uint64_t fmix( uint64_t k );
}

Results in an error along these lines :
test.cpp:5:10: error: ‘farmhash’ has not been declared
    5 | uint64_t fmix( uint64_t k );

Expected behavior
Compilation succeeds. OIIO's public headers only define macros with an OIIO prefix.
Evidence
https://github.com/OpenImageIO/oiio/blob/master/src/include/OpenImageIO/detail/farmhash.h#L1142
Platform information:

OIIO branch/version: Version <IP_ADDRESS>.
OS: CentOS 7.
C++ compiler: GCC 9.3.1
Any non-default build flags when you build OIIO:

Now I've tracked this down, I can workaround using #undef fmix after the OIIO includes, so this is not especially urgent.

Fix in #3002

Thanks Larry! That was quick!

Sorry about that. It used to be an internal header (we didn't write it, it came from the farmhash project), but recently we made it public so that the ustring hash could be inline and constexpr for the sake of running identically on GPU, and I guess we didn't realize how much that header pollutes with preprocessor symbols.
I'm still not super happy with the #define and #undef approach, I would prefer either fully named FARMHASH_blah or for them to all be private namespace inline functions or constants. But because this file comes from another project, I'm hesitant to diverge and end up where it's hard to diff or to upgrade the contents with new versions of farmhash.


When running the App how do you switch between shaders?

When running the App how do you switch between geometry shaders like the Bubble, Chemical, and Ripple shaders in the ( ios-scenekit-shadertoy/SceneShadertoy/Shaders/Geometry/ ) Geometry folder?

Sorry for the late reply. There is no way at the moment to switch between the shaders, unfortunately. However, I will add it to the todo list.
Thanks!

It’a been a while since I wrote an answer, please reopen the issue if you have any questions or comments.
Again, thank you for your input, I appreciate your interest and your question will be used to further improve the project.


Unable to find github link in footer.

In footer section unable to find github link  because of chat with us button , it should be in right side instead of left

Thank you @ayushjha119 for pointing it out. We're planning to change the footer with something new, any design ideas?

Can not shift to right, overlaps the "^" button which scrolls to top (mobile).

Are you interested in building a better footer? If yes, join us on discord

Hey I just change i css propery and it is looking good in large medium and small screen i attaching the screenshot , if you allow me i can raise a pull request

Adding extra space to bottom makes it look empty, we'll just fix this with a new design


add --package-version option for installing packages

This might be a question for @jose, but I don't like exposing this "package version" to the users.  They almost never care.  They may want to install a specific version of the upstream software.

@mgummelt which one is "package verions"? E.g. 0.9.0 for marathon. Do we expose the Universe package version (marathon 0 or 1) anywhere in the CLI? To me that is the only one that is consistent. For example Spark has an application version (E.g. 1.4.0) and a CLI version (E.g. 0.1.0). cc @ConnorDoyle

@jsancio: This is exposing the package version, e.g. "0", or "1", which I don't like doing.  dpkg, for example, doesn't have this notion of package versions.  Just normal software versions with package-specific suffixes

@mgummelt Yeah. The only difference is that for us a universe package can contain multiple software. dpkg gets around this by using dependencies and virtual packages. We will probably implement both so maybe you are correct and we should use the software version in package.json.

@mgummelt this pr shows the software version not package version. Ie for marathon it would be 0.8.0 not 0 or 1.

+1

Oh.  Why do you call it "--pkg-version"?

words are hard. --software-version?

Looking at the code, it seems this is using the package version

tests?

@mgummelt updated

Left a couple more comments.  Sorry for the drive-by review @tamarrow, some of this is mess that I left behind originally so I have opinions.

@mgummelt @ConnorDoyle updated

@mgummelt updated

The commit message needs to be more user friendly as jose specified this morning

@tamarrow: ping

sorry, working on a support issue.

@mgummelt updated

@mgummelt updated

@mgummelt updated

LGTM when tests pass

Wohoo! :fireworks:


optimize _bigbird_block_rand_mask function

From the pytorch profiler's report, we found function _bigbird_block_rand_mask has great negative impacts on performance in hf_bigbird model. This function is used to generate adjacency list of random attention. The frequent usages of np.random.permutation lead to big overhead. Thus, we propose to generate the permutation tables at the beginning of the model and use np.random.randint to generate random indexes in the tables.
We test hf_bigbird in pytorch/benchmark on a NVIDIA A100 GPU.

For train mode, the speedup of the optimization is about 1.09X according to pytorch/benchmark's output.  The TFLOPS is increase from 7.83 to 8.54.
For eval mode, the speedup of the optimization is about 1.30X.

@xuzhao9 Shall we submit it to upstream repo?


generate automatic API docs w/ jsdoc

This adds an npm run docs command to generate API docs using jsdoc, it creates a ./jsdoc directory for this.

[x] add npm command to generate jsdoc
[ ] set up CircleCI to push to GH docs

I've just manually updated the gh-pages branch to match fde5434c36940d196ada19928f7f6a9cd1282c89, and brought this PR up-to-date. It just needs the Circle CI deploy bit done.
Circle has docs for this: https://circleci.com/blog/deploying-documentation-to-github-pages-with-continuous-integration/

We could probably use GH actions here instead of CircleCI:
https://github.com/marketplace?type=actions&query=github+pages
I'm not as familiar with GH actions, will take a look. It might be easier for folks that don't have Circle access to manage.

@jonathanmayer This branch adds building and auto-deployment of JSDoc, with some minor cleanup/modernization of the CircleCI config.

OK I've moved the jsdocs into their own subdir on the gh-pages, as discussed: https://mozilla-rally.github.io/web-science/


German Translation of Nicolas Gallagher's article

Please add this link to your official website in section Translations (http://nicolasgallagher.com/about-normalize-css/):
https://www.saskialund.de/normalize-css-was-ist-das/
It contains the german translation (Deutsche Übersetzung) of Nicolas Gallagher's article: http://nicolasgallagher.com/about-normalize-css/
Thank you!
P.S.: Tried to reach out via Twitter several times, however I didn't receive any reply. My translation is  online since June 2015 ;)
See tweet on translation release: https://twitter.com/SaskiaLund/status/609730925551874049

Thanks @s-a-s-k-i-a. You’ll want to reach out to @necolas directly if you want your article linked to his blog. I’ve pinged him here if you two want to discuss this further.


when you try to upload while offline, error message is wrong

@Chrischuck I don't think @abhasinat has a designed a state for this, so how about if the user taps the upload button in the toolbar or on an obs and we know they're offline, show an alert like:
Title: Internet Connection Required
Body: Please try again when you are connected to the internet.


[EMB-228] Restore loading indication and fix edge case

Purpose
We lost the loading indication on the dashboard filter, and also there was an edge case where if you had projects and filtered all of them out, when you cleared the filter, it would show the "You should make a project" state rather than a loading state.

Summary of Changes

Add back in proper tracking of "loading" state
Ensure that we don't show the "you have no projects" if the filter task is running

Side Effects / Testing Notes
Should be much better now.
Ticket
https://openscience.atlassian.net/browse/EMB-228
Reviewer Checklist

[ ] meets requirements
[ ] easy to understand
[ ] DRY
[ ] ~testable and includes test(s)~
[ ] ~changes described in CHANGELOG.md~ Functionality is already included in existing changelog

Coverage decreased (-0.02%) to 48.217% when pulling 05b533e0935490e4b2f6d386143f64ee269830b2 on brianjgeiger:feature/dashboard-empty-filter-clear into 6db3019cc52e9a1fe51f3cf344110aac302d1ad8 on CenterForOpenScience:develop.


Minor inconsistencies adjustments

Removed two lines with wrong definitions and replaced other two by one

Fixed in forked subtree here 9aa12b0ea8b3136fe0217d7e648cd7feab8aedb2


Fixed some pedantic compiler warnings

Fixed some pedantic compiler warnings, mostly shadowing and implicit conversions.

Friendly ping? The warnings spam is annoying.

Sorry about the delay, I must have missed the original notifications because this is the first I've seen these.


fix(TextArea): row auto behavior

Summary
Type

Bug

Summarise concisely:
What is expected?
<TextArea /> and <TextAreaField /> : new prop autoExpandMax
We have 3 cases :

rows = x : the input will have a fixed height, equals to the height of x rows
maxRows = x: the input size will fit the content. When there are x+1 rows, it will stop to grow (upper bound in number of rows)
maxRows = x and rows = y : the input will have a min-height of y (lower bound) and max-height of x (upper bound)

I think padding bottom should be increase:

I think padding bottom should be increase:


Feature/new data model

Handle new data model with classes based on the json-schema
Removed new from specimen table and removed some unnesecary fields
Added publishing of the Digital Media Events from the digital specimen processing service
Removed deprecated old-digital-specimen from controller (will no longer be backwards compatible as it is not used)

https://naturalis.atlassian.net/browse/DD-641
https://naturalis.atlassian.net/browse/DD-645

Two things:

I think I made the unit coverage and covered all major scenarios. However, Sonar want me to cover all contditions in the MIDS service which is not really do-able and doesn't add much.
It also shows a Code smell in the MIDS service but I don't understand why. Inclined to make a false-positive out of this but maybe I am missing something


Delete documentation

How do you actually delete an object on Google Cloud Storage? I don't see any documentation for it.

@hamedhemmati Done


server: direct send raft message to grpc client (in most cases).

LGTM
PTAL @BusyJay

If addresses is stored as rwlock in sender, mio loop can be removed.

Great work 👍

PTAL @BusyJay

This change can reduce performance reduction.

any update? @disksing

please fix the conflict and merge to master, not grpc branch.

PTAL @BusyJay @siddontang

LGTM


columnAtPoint()

Basically I am trying to find out what column I clicked.
I tried adding a mouse listener to the TableHeader, and use JTable.columnAtPoint() to get the index of the column, but columnAtPoint() does not work seem to work properly with the groups - as far as I can see it returns the lowest grouping correctly, as it appears to be only using the x part of the Point (since it is not an overridden method).
I have added the following which is working:
JBroTable.java
	public String getIdentifierAtPoint(Point p) {
		int x = p.x;
		// Below inherited from {@link JTable#columnAtPoint(p)}
		if (!getComponentOrientation().isLeftToRight()) {
			x = getWidth() - x - 1;
		}
		return ((JBroTableColumnModel) getColumnModel()).getIdentifierAtPoint(new Point(x, p.y));
	}

JBroTableColumnModel.java
	public int getColumnIndexAtY(int y) {
		final int MAGIC_HEIGHT_NUMBER = 25;
		y += 1; // Results seem off by 1
		int correctLevel = 0;
		for (int i = 0; i < columns.size() && y > 0; i++) {
			y -= MAGIC_HEIGHT_NUMBER;
			if (y > 0)
				correctLevel++;
		}
		return correctLevel;
	}

	public String getIdentifierAtPoint(Point p) {
		TableColumnModel delegate = getDelegate(getColumnIndexAtY(p.y));
		if (delegate == null)
			return null;
		return (String) delegate.getColumn(delegate.getColumnIndexAtX(p.x)).getIdentifier();
	}

The obvious problem being that MAGIC_HEIGHT_NUMBER is just based off of my own use case. How can we get the row height properly so this returns correctly?

Hi. Thank you for the interest to the project!
One of the lib methods was named incorrectly: getRowAtPoint. I've just committed a renamed version:
https://github.com/Qualtagh/JBroTable/commit/b90862e115653ca2a089c215fb0c01a4adf419f3
Now it's rowAtPoint for consistency with columnAtPoint.
So, you can use the following methods: header.columnAtPoint, header.rowAtPoint, columnModel.getColumnAtAbsolutePosition. See sample snippet:
    final JBroTableHeader header = table.getTableHeader();
    header.addMouseListener( new MouseAdapter() {
      @Override
      public void mousePressed(MouseEvent e) {
        Point p = e.getPoint();
        int column = header.columnAtPoint( p );
        int row = header.rowAtPoint( p );
        System.out.println( column + ", " + row );
        System.out.println( header.getColumnModel().getColumnAtAbsolutePosition( column, row ) );
        System.out.println( header.getUI().getColumnAtPoint( p ) );
      }
    });

Also, header.getUI().getColumnAtPoint( p ) combines all these three and returns JBroTableColumn instance positioned at a given point. You can call getIdentifier() on this instance to get ID.
MAGIC_HEIGHT_NUMBER from your code corresponds to header.getRowHeight( level ). Height of each row is variable and can be customized, so a method call is needed.

Thanks for the quick reply, that makes sense. getUI().getColumnAtPoint() was what I was looking for, I was not aware of getUI().
I know this isn't the best place but since this is such a quiet issues board I don't want to spam a bunch of issues so I'll just send here. If you'd prefer me add a few issues I could do that. FWIW I was trying to make an "excel document preview" where you could have multiple headers, rearrange data, add columns, etc. At the end I would extract the layout and names that they have selected and output to a file in the same way. I was hoping for an addColumn(int idx) to add inbetween existing columns. Also I have the following issue:
I'm unsure if this is a problem with JTable or the implementation, but it seems if a Drag Column operation is interrupted in some way (such as by JDialog or PopupMenu) and the header doesn't get MouseReleased, things start getting wonky.
For example take this code:
		JPopupMenu popupMenu = new JPopupMenu();
		JMenuItem insertLeft = new JMenuItem("Insert Left");
		insertLeft.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				JOptionPane.showInputDialog(null,"test","test",0,null,null,"test");
			}
		});
		popupMenu.add(insertLeft);
		table.getTableHeader().setComponentPopupMenu(popupMenu);

Now if you try to right click to bring up the menu, but accidentally drag a little first instead of a perfect click, the column header hangs in its position. Once this happens sometimes other issues happen (columns show wrong data until clicked, other outdated info).


luci-mod-network: #6962-Cipher option is not required for WPA3 Enterp…

…rise security mode

[ ] This PR is not from my main or master branch :poop:, but a separate branch :white_check_mark:
[ ] Each commit has a valid :black_nib: Signed-off-by<EMAIL_ADDRESS>row (via git commit --signoff)
[ ] Each commit and PR title has a valid :memo: <package name>: title first line subject for packages
[ ] Incremented :up: any PKG_VERSION in the Makefile
[ ] Tested on: (architecture, openwrt version, browser) :white_check_mark:
[ ] ( Preferred ) Mention: @ the original code author for feedback
[ ] ( Preferred ) Screenshot or mp4 of changes:
[ ] ( Optional ) Closes: e.g. openwrt/luci#issue-number
[ ] ( Optional ) Depends on: e.g. openwrt/packages#pr-number in sister repo
[ ] Description: (describe the changes proposed in this PR)

Apparently an older duplicate of #6964 that has comment from jow.
Closing.

I confirm no TKIP for WPA3:

WPA3 Personal -> Simultaneous Authentication of Equals (SAE) with forward secrecy (FS/PFS) -> GCMP - AES
WPA3 Enterprise -> 802.1X/EAP -> GCMP - AES

It must be solved.
Ticket:

https://github.com/openwrt/luci/issues/6962


Sort and Faceting Bug

This is the issue:
var uri,  // a valid uri
    sortObj; // a valid sort object

ERMrest.resolve(uri).then(function (ref) {
    var facetColumns = ref.facetColumns;
    var sortedRef = ref.sort(sortObj);
   console.log(sortedRef.facetColumns[0].reference === sortedRef); // returns false!
});

This is happening because of the _referenceCopy function logic which is doing a shallow-copy. We should change this function to instead of shallow copying the reference, creates a new reference from scratch.

@RFSH have this been fixed?

No, it still exists. The implications of it for chaise is that if you sort based on a column and then change the facet, the sort criteria will be removed.


[qa] Geometry shouldn't be allowed to be None if not mobile

Fixes #10

@nemesisdesign @atb00ker I'm not sure if i did it right or not please have a look and lemme know.
Thanks

@nemesisdesign Please have a look now and lemme know.

Done! @nemesisdesign  Please have a look :)


How to enable syntax highlighting?

Hello
really silly question:
I installed the extension, but nothing happened. The .sas programs are not recognised, the syntax highlighting doesnt work, no snippets, no autocompletion. I restarted vscode, no effect. What am I doing wrong?
Version: 1.67.1
Commit: da15b6fd3ef856477bf6f4fb29ba1b7af717770d
Date: 2022-05-06T12:37:03.389Z
Electron: 17.4.1
Chromium: 98.0.4758.141
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19044

Do you see "SAS" as the language mode on the status bar at bottom right?
Is there any error in the "Log (Extension Host)" output panel?

Same issue:

[2022-06-20 09:22:59.797] [exthost] [info] ExtensionService#_doActivateExtension SAS.sas-lsp, startup: false, activationEvent: 'onLanguage:sas'

And "SAS" is the detected language mode.

I had to use the sas color theme instead of the one I'm currently using.
@meztez SAS color theme is required to see correct syntax highlighting. Other color themes don't have SAS tokens configured.
Autocomplete should work no matter what theme you choose.

thanks for your replies. I changed the language mode manually to SAS, and changed the theme to SAS theme to make the syntax highlighting work. Frankly, the mandatory switch to SAS theme is a killer, especially since other languages' syntaxes are no longer highlighted when using the SAS theme! Is there no way to configure SAS tokens for other themes, as is done for any other language extension? thanks!!

v0.0.6 published. Now SAS themes provide default colors for non-SAS languages. SAS syntax have basic type colors in non-SAS themes.


Add "book package" export and import

It would be nice be able to export individual books into files that someone else can load into their Lute. It'd include all the words/definitions, the audio, the bookmarks, everything. I think a feature like that would really help form a Lute community. I know that I'd love to share my texts with other Czech learners.
This would be something like an "anki package" zip file.  Work involved:

define export, import format
export file versioning, so that importers can deal with old formats
export book and audio only, no terms?
export book and terms?
handle term images
overwrite existing terms on import, or ignore?
book with same title allowed or not?

I was about to create an issue regarding this. These user-generated book "bundles" could indeed be very helpful. I think the inclusion of terms in such bundles would be of trivial benefit, since the central focus is the convenience of sharing books as learning resources. It is analogous to including review statuses of cards in Anki packages – barely useful.


Develop

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: Hazel0928:x: gengleileiYou have signed the CLA already but the status is still pending? Let us recheck it.

LGTM


port commands in init happens before subscribe()

Invoking an outgoing port on init will only work if there's a delay first.
@Janiczek also mentioned that any Cmd on init is not executed on ellie, though when I tried that locally everything worked as expected.
Main.elm
port module Main exposing (main)

import Browser
import Html exposing (Html)
import Process
import Task
import Time

main : Program () Int Msg
main =
    Browser.embed
        { view = \_ -> Html.text "Check console."
        , update = update
        , subscriptions = \_ -> Sub.none
        , init =
            always
                ( 1, echo "Roger Dodger" )

        -- The Following will successfully `echo`
        -- always
        --     ( 1
        --     , Task.perform (always Echo)
        --         (Process.sleep 500
        --             |> Task.andThen (always Time.now)
        --         )
        --     )
        }

type Msg
    = Echo

update msg model =
    case msg of
        Echo ->
            ( model
            , echo "Delayed Roger Dodger"
            )

port echo : String -> Cmd msg

index.html
<html>

<head>
    <script src="elm.js"></script>
</head>

<body>
    <main></main>
    <script>
        var app = Elm.Main.embed(document.querySelector('main'));

        app.ports.echo.subscribe(function (message) {
            console.log(message);
        });

    </script>
</body>

</html>

elm.json
{
    "type": "application",
    "source-directories": [
        "."
    ],
    "elm-version": "0.19.0",
    "dependencies": {
        "elm-lang/browser": "1.0.0",
        "elm-lang/core": "6.0.0",
        "elm-lang/html": "3.0.0",
        "elm-lang/json": "1.0.0",
        "elm-lang/time": "1.0.0"
    },
    "test-dependencies": {},
    "do-not-edit-this-by-hand": {
        "transitive-dependencies": {
            "elm-lang/url": "1.0.0"
        }
    }
}

I got the SSCCE down to this:
port module Main exposing (main)

import Browser
import Html

main : Program () () ()
main =
  Browser.embed
    { init = \_ -> ( (), echo "Roger Dodger" )
    , update = \_ _ -> ( (), Cmd.none )
    , subscriptions = \_ -> Sub.none
    , view = \_ -> Html.text "Check console."
    }

port echo : String -> Cmd msg

The fix is really cool!


fix checks on largest comparison value for upsert ttl and allow to add segments even out of ttl

This PR refines some checks on upsert TTL:

_largestSeenComparisonValue can be negative so check it against special value Double.MIN_VALUE instead of 0;
bump up _largestSeenComparisonValue when addSegment for metadataTTL > 0 too;
allow to adding segments even outside TTL. This may be needed when one uploads segments to upsert table directly, but they are out of TTL. If they are skipped, then all their docs are treated as valid. The table data is not right unless docs in those uploaded segments are all valid.

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

Project coverage is 0.00%. Comparing base (59551e4) to head (4f787c6).
Report is 1094 commits behind head on master.

:exclamation:  There is a different number of reports uploaded between BASE (59551e4) and HEAD (4f787c6). Click for more details.
HEAD has 48 uploads less than BASE

Flag
BASE (59551e4)
HEAD (4f787c6)

integration
7
2

integration2
3
2

temurin
12
2

java-21
7
2

skip-bytebuffers-true
3
1

skip-bytebuffers-false
7
1

unittests
5
0

unittests1
2
0

java-11
5
0

unittests2
3
0

integration1
2
0

custom-integration1
2
0

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #14094       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436        3     -2433
  Lines        133233        6   -133227
  Branches      20636        0    -20636
=============================================
- Hits          82274        0    -82274
+ Misses        44911        6    -44905
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <ø> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <ø> (ø)

java-11
?

java-21
0.00% <ø> (-61.63%)
:arrow_down:

skip-bytebuffers-false
0.00% <ø> (-61.75%)
:arrow_down:

skip-bytebuffers-true
0.00% <ø> (-27.73%)
:arrow_down:

temurin
0.00% <ø> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

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

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


[vm/aot] gen_snapshot does not support concatenated dills

Since 07b6ec3425443449c52793253bf74c6b1f06a474, we support concatenating dill files.
https://github.com/dart-lang/sdk/blob/ad334d85e196244cee0d2043eaf4649466ceef4e/runtime/vm/kernel_binary.cc#L151-L152
The VM supports loading concatenated dill files in JIT. The main from the first dill is run, and the libraries from the other dills are available in the VM.
In AOT, support for this appears to not have been implemented. Calling gen_snapshot with a concatenated dill file (both dills compiled with --aot) fails:
../../runtime/vm/kernel_loader.cc: 225: error: Trying to load a concatenated dill file at a time where that is not allowed
dart::Assert::Fail(const dart::Assert * this, const char * format) (/usr/local/google/home/dacoharkes/dart-sdk/sdk/runtime/platform/assert.cc:55)
dart::kernel::KernelLoader::KernelLoader(dart::kernel::KernelLoader * this, dart::kernel::Program * program, dart::kernel::UriToSourceTable * uri_to_source_table) (/usr/local/google/home/dacoharkes/dart-sdk/sdk/runtime/vm/kernel_loader.cc:223)
dart::BootstrapFromKernel(dart::Thread * thread, const uint8_t * kernel_buffer, intptr_t kernel_buffer_size) (/usr/local/google/home/dacoharkes/dart-sdk/sdk/runtime/vm/bootstrap.cc:112)
dart::Bootstrap::DoBootstrapping(const uint8_t * kernel_buffer, intptr_t kernel_buffer_size) (/usr/local/google/home/dacoharkes/dart-sdk/sdk/runtime/vm/bootstrap.cc:177)
dart::Object::Init(dart::IsolateGroup * isolate_group, const uint8_t * kernel_buffer, intptr_t kernel_buffer_size) (/usr/local/google/home/dacoharkes/dart-sdk/sdk/runtime/vm/object.cc:2454)
dart::Dart::InitIsolateFromSnapshot(dart::Thread * T, dart::Isolate * I, const uint8_t * snapshot_data, const uint8_t * snapshot_instructions, const uint8_t * kernel_buffer, intptr_t kernel_buffer_size) (/usr/local/google/home/dacoharkes/dart-sdk/sdk/runtime/vm/dart.cc:851)
dart::Dart::InitializeIsolate(const uint8_t * snapshot_data, const uint8_t * snapshot_instructions, const uint8_t * kernel_buffer, intptr_t kernel_buffer_size, dart::IsolateGroup * source_isolate_group, void * isolate_data) (/usr/local/google/home/dacoharkes/dart-sdk/sdk/runtime/vm/dart.cc:1036)
dart::CreateIsolate(dart::IsolateGroup * group, bool is_new_group, const char * name, void * isolate_data, char ** error) (/usr/local/google/home/dacoharkes/dart-sdk/sdk/runtime/vm/dart_api_impl.cc:1300)
Dart_CreateIsolateGroupFromKernel(const char * script_uri, const char * name, const uint8_t * kernel_buffer, intptr_t kernel_buffer_size, Dart_IsolateFlags * flags, void * isolate_group_data, void * isolate_data, char ** error) (/usr/local/google/home/dacoharkes/dart-sdk/sdk/runtime/vm/dart_api_impl.cc:1430)
dart::bin::CreateIsolateAndSnapshot() (/usr/local/google/home/dacoharkes/dart-sdk/sdk/runtime/bin/gen_snapshot.cc:756)
dart::bin::main(int argc, char ** argv) (/usr/local/google/home/dacoharkes/dart-sdk/sdk/runtime/bin/gen_snapshot.cc:940)
libc.so.6!__libc_start_main(int (*)(int, char **, char **) main, int argc, char ** argv, int (*)(int, char **, char **) init, void (*)(void) fini, void (*)(void) rtld_fini, void * stack_end) (libc-start.c:332)
_start (Unknown Source:0)

gen_snapshot in AOT mode takes a self-contained dill file which was generated with closed world assumptions. Simply concatenating such dill files is not correct, as code in one dill file does not account for the code in another. So we should probably avoid supporting this case in AOT.
@dcharkes Why do you need to AOT-compile concatenated dill files?

@mkustermann suggested using dill concatenation instead of using --source extra_source.dart with gen_kernel. I was exploring this. Context: internal doc.
In this context it might work, because the second dill just has meta data, no executable code. But I can see how in general this wouldn't work. (We should probably close this issue as wont-fix.)

Could you clarify why metadata should be put into a separate dill file? Is there a reason why we cannot add metadata when generating a dill file? The metadata can cover all platforms and gen_snapshot can pick particular variant depending on its configuration.

We use the single program in a dill file to bootstrap with in gen_kernel.
To make concatenated dill files work, we need to specify that the first (or last) program is used for bootstrapping.
working POC
Side note: We process the programs in a dill file in reverse order in JIT currently:

Load kernel files in reverse order to accommodate topological order that pub build_runner produces.
This fixes a bug with generation of coverage report in presence of mixin declaration/application coming from different dill files.

https://dart-review.googlesource.com/c/sdk/+/87605
Which might not be required anymore, since circular was supported aftewards.

By indexing all sources in a concatenated dill file and then using that
index to find the sources and line starts we can fix missing information
(leading to crashes when collecting coverage) in for instance circular
instances.

https://dart-review.googlesource.com/c/sdk/+/98006

We landed special casing to avoid concatenated AOT dills in the kernel_front_end:
https://github.com/dart-lang/sdk/blob/59a1e9afebecc76e18b5ae4d0abb2863bb4a2e71/pkg/vm/lib/kernel_front_end.dart#L328-L333
https://github.com/dart-lang/sdk/blob/59a1e9afebecc76e18b5ae4d0abb2863bb4a2e71/pkg/vm/lib/kernel_front_end.dart#L335-L338
We're also going to have to land special casing aot in pkg:frontend_server as well, as Flutter depends on that. The frontend server also writes out its own dill files. (Hopefully that's the last place that manually writes out dill files.) This is another reason we should consider supporting concatenated dills in AOT. So that we don't have to push this logic AOT vs JIT to all frontend wrappers.

Reviving this issue. We'll need to support this due to the native assets mapping only being known after kernel compilation when we introduce link hooks that are run after tree shaking. See https://github.com/dart-lang/sdk/issues/55377.
As @alexmarkov noted above, one can't concatenate AOT-compiled Dart code. So the only thing supported in concatenation is adding "metadata" (such as native assets mapping which is an annotation on a predefined library name).


acquireTokenSilent is not working inside Chrome Extension

I am developing a Google Chrome Extension using Angular and I am using azure adb2c as security provider.
I have set up everything regarding the login procedure. After logging in I want to make Http calls to our backend, for this I am using the .acquireTokenSilent method before each request and append the token to the http's header.
Unfortunately this is currently not working at all. Every http call I make throws errors. The thing is that everything works fine if I start the extension as a web application on localhost (using ng serve).

I've read your explanation about the application working in an iFrame to aquire access token and calling .acquireTokenSilent  at that moment is forbidden. https://github.com/AzureAD/microsoft-authentication-library-for-js/issues/1194#issuecomment-572750166
I suspect that MSAL thinks that the Chrome extension is somehow an iFrame-ish thing and that's why I receive the same error. What do you advise me in this scenario?
Thank you!

@directcuteo This error only happens when your app is loaded in iframe, the url contains a hash response from AAD, and the apps tries to invoke acquireTokenSilent (usually this error is thrown when the page used for the redirect URI calls acquireTokenSilent on page load). Can you please provide your configuration for MSAL?
And in the future, please follow one of the pre-defined issue templates, thanks!

Here is the configuration I have. I am not using the MsalGuard (or any guard) for the extension because I the first screen is the login gateway. I tried after login to have a screen where I don't make any http request but the application was still throwing errors when I navigated afterwards to some page which makes http calls.
import { Configuration } from 'msal';
import { MsalAngularConfiguration } from '@azure/msal-angular';
import { environment } from 'src/environments/environment';

// this checks if the app is running on IE
export const appRunsInIE =
  window.navigator.userAgent.indexOf('MSIE ') > -1 ||
  window.navigator.userAgent.indexOf('Trident/') > -1;
/***********************************************************
 * STEP 1 - B2C Policies and User Flows
 ***********************************************************/
export const b2cPolicies = {
  names: {
    signUpSignIn: 'B2C_1_SignUpSignInStandard',
    resetPassword: 'B2C_1_PasswordReset',
  },
  authorities: {
    signUpSignIn: {
      authority:
        'https://testApp.b2clogin.com/testApp.onmicrosoft.com/B2C_1_SignUpSignInStandard',
    },
    resetPassword: {
      authority:
        'https://testApp.b2clogin.com/testApp.onmicrosoft.com/B2C_1_PasswordReset',
    },
  },
};

/***********************************************************
 * STEP 2 - Web API Scopes & URLs (Not Relevant For This Post)
 ***********************************************************/
export const apiConfig: { b2cScopes: string[]; webApi: string } = {
  b2cScopes: ['https://testApp.onmicrosoft.com/api/api-permissions'],
  webApi: 'https://testAppenv1be.azurewebsites.net/api',
};

/***********************************************************
 * STEP 3 - Authentication Configurations
 ***********************************************************/
/***********************************************************
 * Config object to be passed to MSAL on creation.
 * For a full list of msal.js configuration parameters,
 * visit https://azuread.github.io/microsoft-authentication-library-for-js/docs/msal/modules/_configuration_.html
 * *********************************************************/
export const msalConfig: Configuration = {
  auth: {
    clientId: '3xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx8',
    authority: b2cPolicies.authorities.signUpSignIn.authority,
    redirectUri: environment.postLoginRedirectUri, //this will be 'http://localhost:4200/' on dev and  'https://testAppenv01ui.azurewebsites.net' on production
    postLogoutRedirectUri: environment.postLogoutRedirectUri, //this will be 'http://localhost:4200/' on dev and  'https://testAppenv01ui.azurewebsites.net' on production
    navigateToLoginRequestUrl: false,
    validateAuthority: false,
  },
  cache: {
    cacheLocation: 'localStorage',
    // Set this to "true" to save cache in cookies
    // to address trusted zones limitations in IE
    storeAuthStateInCookie: appRunsInIE,
  },
};

/***********************************************************
 * STEP 4 - Scopes Required For Login
 ***********************************************************/
/***********************************************************
 * Scopes you enter here will be consented once you authenticate. For a full list of available authentication parameters,
 * visit https://azuread.github.io/microsoft-authentication-library-for-js/docs/msal/modules/_authenticationparameters_.html
 ***********************************************************/
export const loginRequest: { scopes: string[] } = {
  scopes: ['openid', 'profile'],
};

/***********************************************************
 * STEP 5 - Scopes which will be used for the access token
 *          request for your web API
 ***********************************************************/

export const tokenRequest: { scopes: string[] } = {
  scopes: apiConfig.b2cScopes,
};

/***********************************************************
 * STEP 6 - MSAL Angular Configurations
 *          Define protected API URLs and required scopes
 ***********************************************************/
export const protectedResourceMap: [string, string[]][] = [
  [apiConfig.webApi, apiConfig.b2cScopes],
];

/***********************************************************
 * STEP 7 - MSAL Angular specific configurations
 *
 ***********************************************************/
export const msalAngularConfig: MsalAngularConfiguration = {
  popUp: false,
  consentScopes: [...loginRequest.scopes, ...tokenRequest.scopes],

  // API calls to these coordinates will NOT activate MSALGuard
  unprotectedResources: [],

  // API calls to these coordinates will activate MSALGuard
  protectedResourceMap,
  extraQueryParameters: {},
};

@directcuteo Try creating a blank html page in your app, and using that as the redirect URI in your app when it is running in a Chrome extension. Note, when running in a Chrome extension, it is likely that silent token requests that require a network request will always return an InteractionRequired error, so be sure to have error handling that call acquireTokenPopup and retries the request.


Cow: Blacklist vcubesoftsolutions.com

Cow requests the blacklist of the website vcubesoftsolutions\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

vcubesoftsolutions\.com has been seen in 5 true positives, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


Music Player

Refer this repo I made : https://github.com/vedant-jain03/Play-on-Music

@ridsuteri Please assign me if valid

@vedant-jain03 a music player already exists ! so i can't assign you to add a new player :(
but if you want ,you can improve the existing one


Can I use Storyboard or Interface Builder to create the cells?

Can I use Storyboard or Interface Builder to create the cells?
Or do I have to create them manually by hand code?
Thanks.

Hi All,
I'm able to create subclass from GMGridViewCell with IB but problem I'm facing is accessing that class in main class where I'm using GMGridView. Its not working. I don't know what the issue is. Below is the code what I'm trying to do in main class.
LandingPageCustomCell *cell = (GMGridViewCell *)[gridView dequeueReusableCell];

LandingPageCustomCell is a class which is derived from GMGridView. But its not working.
Please help me if anyone has any idea.


Fix log filtering in bosh-director

Thanks! This is on develop now as 2440b7689abe48715572d3ea8a267d5edb877249
-Nader and Jonathan


Edit fields are not resizable

Altough the value field is pretending to be resizable, it isn't. It's a pain to edit larger text files in there.

the edit dialog can be resized by the drag the right-bottom corner of the text area (not the dialog).

Ah, indeed. I didn't notice this, because the cursor doesn't change at that point.

You are right, a resize cursor on the dialog corner would be nice. It seems that the modal dialog component of bootstrap (i am using it to implement the UI of zkui) does not support resizing. zkui realy need some improments..


Clearpath Microstrain IMU doesn't set proper frame ID

The current configurations [1] don't properly set the IMU header to the frame that the integration sets as imu_N_link and instead generically publishes as imu_link which is the internal IMU link. The orientation of CPR integrations don't put those frames as colinear so orientation vectors aren't correct. The proper frame ID for the configurations need to be set
[1] https://github.com/clearpathrobotics/clearpath_robot/blob/main/clearpath_sensors/config/microstrain_imu.yaml#L33

Is this fixed now?

I fixed it manually on my robot


Put the "Discard Changes" button on the left

Hi
On the "Source Control" Window, Put the "Discard Changes" button on the left of the file name or folder name.
In the current situation, when you click to Open File or Stage Changes, it is easy to click on Discard Changes, resulting in misoperation
            Thanks

Thank you very much for your feedback. Placing buttons both before and after the folder/file name in a list is not possible given the current design system. Having said that, we can certainly look to see whether we can reorder the buttons that appear after the file/folder name.
//cc @misolori, @daviddossett


Support undo in the leave-behind demo

Allow the user to undo a dismiss by tapping UNDO in the corresponding snack-bar.
This seems like a better UI idiom than having the user click a button that appears in the Dismissable's background.
Also:

Renamed Dismissable onResized to onResize, since it's not called after the resize is complete, but during the resize animation.

lgtm modulo the O(N) walk on undo.


Improve nodeset error message

OSPRH-6593

recheck

I had to change things to solve the conflict, @rabi could you please take another look?


Bugfix/allisonbhavsar/1031146707 navbar overlap fix

Administrative Info
Monday Board ID: 853642866
Make sure your branch name conforms to: <feature/staging/hotfix/...>/[username]/[Monday Item ID]-[3-4 word description separated by dashes]. Otherwise, please rename your branch and create a new PR.
Changes
What changes did you make?

Added safe area support to navbar to improve display on newer iPhones

Testing
How did you confirm your changes worked?

Tested on an iPhone X to make sure the home indicator doesn't obscure the navbar
Tested on iPhone 8 to check that navbar still renders properly on normal screens

Confirmation of Change

LGTM, tested with my phone and it looks fine


⚠ ClamAV has degraded performance

In 2cee6d1, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 100 ms

Resolved: ClamAV performance has improved in 5fb3ce1 after 17 minutes.


Some details about the implementations

Hi! Thanks for your extraordinary work! I have a question about the training. The paper mentioned that "We introduce a Lightning T2I branch alongside the regular diffusion branch." But in the method section, all the loss calculation are conducted in the lighting T2I branch. I wonder to know, in the training, is it enough to only need a lighting model (SDXL Lighting) instead of using the original base model (SDXL)? I mean the original base model does not need any trainable parameters. In this way, where does the "alongside" reflect?

In Stage 1, we trained on the non-accelerated model (sdxl-base) using traditional diffusion loss. For Stage 2 and 3, although in the first implementation all losses were calculated on the lightning branch, we believe that it is also okay to place the diffusion loss on the non-accelerated model, and it will also help with compatibility.

Thanks for your quick responses!

By the way? How many timestpes do you used for flux-dev PuLID training (when calculating the ID loss) ?

I am training pulid like non-acclerated model for diffusion loss and acclerated model for id loss, and a weird problem arises: the generated image includes many faces of the same person. how can i resolve it? thx

Hi @garychan22, could you let me know your training dataset? Do you have an OS implementation of your training pipe?


External auth now works in General ring, kindly remove it as a know issue from this page.

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

ID: 4c405ed3-f304-ff58-d32d-108fc725bb6a
Version Independent ID: 4c405ed3-f304-ff58-d32d-108fc725bb6a
Content: Introducing the new Microsoft Teams client - Teams
Content Source: msteams-platform/resources/teams-updates.md
Service: msteams
GitHub Login: @v-ypalikila
Microsoft Alias: lajanuar

@parthiiita -Thanks for reporting your issue.
We will check this at our end and update the document accordingly.