Comparison between TAP-Net and PIPs

Hi,
You only mention that PIPs is a concurrent work to TAP-Vid/Net.
Are you planning to show any comparisons between these 2 works?

We are working on it. Thanks for your patience.

We are working on it. Thanks for your patience.

Hi, I hope this message finds you well. I was wondering if any progress has been made on this matter. Thank you!

The numbers have been added to the most recent version of our arxiv paper

The numbers have been added to the most recent version of our arxiv paper

Thank you very much for the update! Will you release the evaluation code of PIPs on Kinetics, Kubric, DAVIS, and RGB-Stacking? I am really looking forward to it, thank you!

I've got PIPS results that are close to the ones
reported in the updated arxiv paper using this
somewhat refactored fork of PIPs.
My numbers are in the last two rows:

However, my RAFT numbers do not match the
reported ones, except on the DAVIS subset.
If interested, here are some entry points:

Batched PIPS forward pass for chained trajectory prediction
Evaluation for loop
Documentation on running the evaluation
PIPs environment setup

fork

Really appreciate it! Thanks!

I've got PIPS results that are close to the ones reported in the updated arxiv paper using this somewhat refactored fork of PIPs.
My numbers are in the last two rows:
However, my RAFT numbers do not match the reported ones, except on the DAVIS subset.
If interested, here are some entry points:

Batched PIPS forward pass for chained trajectory prediction
Evaluation for loop
Documentation on running the evaluation
PIPs environment setup

fork
Really appreciate it! Thanks!


Performance of bb8 with tokio-postgres worse than r2d2 with postgres

I wanted to go full async with the application I'm currently building and tried both bb8 and l337 with tokio-postgres just to find that both performed worse than r2d2 with postgres:
https://bitbucket.org/bikeshedder/actix_web_async_postgres/src/master/
Am I doing anything wrong or is this maybe an issue with tokio-postgres?

One useful statistic to know there would be how many postgres connections are actually used in each case.  I suspect the async libraries are using far more connections than the blocking r2d2.

As you can see in the code I use the same pool size for all three implementations:
const POOL_MIN_SIZE: u16 = 4;`
const POOL_MAX_SIZE: u16 = 16;`

They were not extracted into constants before. I just changed that to make it more obvious that the pools are configured exactly the same.
I also made sure to run htop with a filter and could see 48 (3x16) DB connections to PostgreSQL which are all evently utilized, depending on the test currently running.

@bikeshedder Isn't it bacause r2d2+postgres uses threadpool implicitly when calls actix_web::web::block()?

On my machine, with a release build of your test binary, I get
Running 2m test @ http://localhost:8000/v1.0/event_list_l337
  4 threads and 128 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     4.12ms  567.15us  29.99ms   78.09%
    Req/Sec     7.79k   164.50     8.97k    73.35%
  Latency Distribution
     50%    4.04ms
     75%    4.37ms
     90%    4.77ms
     99%    5.98ms
  3720626 requests in 2.00m, 1.49GB read
Requests/sec:  30994.28
Transfer/sec:     12.71MB
Running 2m test @ http://localhost:8000/v1.0/event_list_r2d2
  4 threads and 128 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     4.65ms    2.41ms  50.40ms   88.78%
    Req/Sec     7.23k   316.71    27.38k    98.71%
  Latency Distribution
     50%    3.69ms
     75%    4.42ms
     90%    7.49ms
     99%   14.98ms
  3453682 requests in 2.00m, 1.38GB read
Requests/sec:  28756.74
Transfer/sec:     11.79MB
Running 2m test @ http://localhost:8000/v1.0/event_list_bb8
  4 threads and 128 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     4.96ms  322.86us  32.02ms   82.08%
    Req/Sec     6.48k    92.65     7.41k    92.40%
  Latency Distribution
     50%    4.92ms
     75%    5.09ms
     90%    5.28ms
     99%    5.81ms
  3096725 requests in 2.00m, 1.24GB read
Requests/sec:  25802.00
Transfer/sec:     10.58MB

which is more along the lines of what I would expect.

I just updated r2d2 to the latest RC which also uses a newer version of (tokio-)postgres which resulted in a very measurable performance drop for the r2d2 implementation. It might all be related to this: https://github.com/sfackler/rust-postgres/issues/469

It really seams to be an issue of the rust-postgres crate: https://github.com/sfackler/rust-postgres/issues/469#issuecomment-517764742


I already have Team Administrator and Project Administrator roles but still can't edit Taskboard Styles

[Enter feedback here]
I already have Team Administrator and Project Administrator roles but still can't edit Taskboard Styles. I can edit everything in my project except this.
I get this error: "You do not have sufficient permissions to configure cards for this team. You must either be a team administrator or a project administrator."

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

ID: e56e5bfa-e2ea-c309-8487-3ba3236ee6ee
Version Independent ID: 981fdeed-0f98-135c-effb-2446cff74ea9
Content: Add, remove team administrator - Azure DevOps
Content Source: docs/organizations/settings/add-team-administrator.md
Product: devops
Technology: devops-settings
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@shunoraga - Thanks for your question. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the documentation. Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow


Access denied for user mattermost

hello i have installed the mattermost todo list plugin as described.
my system:
mysql in docker
mattermost local
postgres in docker
mattermost uses postgres, mysql I installed only for the plugin additionally
mysql user : "todouser"
i can display the help in the mattermost via "/todo", but as soon as i want to create a todo i only get a feedback:
Command with a trigger of 'todo' returned response 500 Internal Server Error
the mysql log contains the following:
Access denied for user<EMAIL_ADDRESS>(using password: YES)

You should supply your own MySQL parameters as environment variables to the process running Mattermoster.
Currently, the system in trying to connect to your database using default values:

Parameter
Environment variable
Default value

host
MM_TODO_DBHOST
localhost

port
MM_TODO_DBPORT
3306

user
MM_TODO_DBUSER
mattermoster

password
MM_TODO_DBPASS
mattermoster

database
MM_TODO_DBNAME
mattermoster


Added Contribution Guidelines.

This pull request adds contribution guidelines to the project. The guidelines provide instructions for contributing to the project, creating issues or pull requests. Please review and let me know if any changes are needed.

merged. thank you


New package: BayesianNonparametricStatistics v0.1.0

Registering package: BayesianNonparametricStatistics
Repository: https://github.com/Jan-van-Waaij/BayesianNonparametricStatistics.jl
Created by: @Jan-van-Waaij
Version: v0.1.0
Commit: 0a51df004d6d7bfddc615d5243e8282bb1d622c7
Git reference: HEAD

@Jan-van-Waaij please see the message above.  Once you fix the automerge issues you can register again the new revision without changing the version number and this pull request will be automatically updated.
[noblock]


update() function to update package and TagUI files without internet [done]

python test.py --proxy "https://proxy:8080"
[TAGUI][INFO] - setting up TagUI for use in your Python environment
[TAGUI][INFO] - downloading TagUI (~200MB) and unzipping to below folder...
[TAGUI][INFO] - /Users/bpan1
[TAGUI][ERROR] - failed downloading from https://github.com/tebelorg/Tump/releases/download/v1.0.0/TagUI_macOS.zip...
[Errno socket error] [Errno 8] nodename nor servname provided, or not known
[TAGUI][ERROR] - use init() before using url()
[TAGUI][ERROR] - use init() before using type()
[TAGUI][ERROR] - use init() before using read()

I don't think python command has a --proxy switch. Initial googling seems to suggest that proxy setting is configured from operating system, and any internet traffic will be done through that.
Can you tell me more about your use case why are you using proxy? Is it because of firewall blocking on the company computer?
If yes, you can try using the pack() function on your personal laptop to generate a tagui_python.zip and tagui.py file. Which you can copy to your computer laptop to use with the usual import tagui as t. Details on pack() here - https://github.com/tebelorg/TagUI-Python#core-functions
There might be other dependencies though, which still requires the internet to install. I'm collecting inputs on such environments at this issue - https://github.com/tebelorg/TagUI-Python/issues/36 use in air-gap environments with network restriction and no internet)

Adding on, inherently this tool does not have an option to connect to the internet by proxy. As the design for the tool is to emulate user's actions on the UI. So things that can be configured or done from the user interface on desktop, apps, webpage or command line can be done.
But things that are programmatically done and not doable by user are not in scope. That's also why I did not port over the API step from the original TagUI project that can be used to consume REST web services, but recommend users to use Python requests library to do that, when they call t.api().

I don't think python command has a --proxy switch. Initial googling seems to suggest that proxy setting is configured from operating system, and any internet traffic will be done through that.
Can you tell me more about your use case why are you using proxy? Is it because of firewall blocking on the company computer?
If yes, you can try using the pack() function on your personal laptop to generate a tagui_python.zip and tagui.py file. Which you can copy to your computer laptop to use with the usual import tagui as t. Details on pack() here - https://github.com/tebelorg/TagUI-Python#core-functions
There might be other dependencies though, which still requires the internet to install. I'm collecting inputs on such environments at this issue - #36 use in air-gap environments with network restriction and no internet)

Yes, our company needs to use a proxy to access the Internet. This exception occurred during the initialization phase. I disabled the code blocks and completed the installation manually, but I still want to use a proxy to install or update them.
`# if not download (tagui_zip_url, home_directory + '/' + tagui_zip_file):
             # error message is shown by download (), no need for message here
return False`
# if not download (tagui_delta_url, tagui_delta_file): return False
For test cases, there is no need for a network proxy

I see.. If your company network can access GitHub through the proxy, you'll be able to get the updated TagUI files when you run the updated TagUI for Python tagui.py.
Although user can manually copy the files listed in _tagui_delta() to replace the files on their computer, I think it is too much work to update version.
Let me think over what is a good way to reduce user friction. Maybe I can use similar idea in pack() to create a zip file of the updated files, so that user can copy the small zip to their company laptop easily. (using pack() works too now, but the file to copy is over 150 MB).
Feel free to add on and let me know what is the best way to copy files over to your computer laptop or any other constraints in your computer network. I think copy or email zip file directly will be the easiest way, but may be there are other interesting ways you use that is easier.

Draft of a new function update(), it downloads the latest tagui.py for the package and the updated TagUI files, zip them up, convert the zip binary data into base64, generate an update.py containing code to update on target computer and embedded zipped data of the updated files in base64.
When python update.py is run on the target computer, the embedded zip data will be extracted and the respective files updated to the existing installation in the air-gapped environment without internet. Outstanding items before a new release can be published with this new function -

move updated package file tagui.py to correct package folder from import tagui as t
remove python-generated tagui.pyc and pycache/tagui.cpython-37.pyc
to add code on getting the correct updated version number

With this new function, besides using proxy, users can simply copy over 1 file update.py to the restricted network computer to update to the latest version of TagUI for Python.
def update():
    """function to update package and TagUI files on an air-gapped computer without internet"""

    print('[TAGUI][INFO] - downloading latest tagui.py and TagUI stable cutting edge version ...')

    # first download updated files to tagui_update folder and zip them to tagui_update.zip
    if not os.path.isdir('tagui_update'): os.mkdir('tagui_update')
    if not os.path.isdir('tagui_update/tagui.sikuli'): os.mkdir('tagui_update/tagui.sikuli')

    tagui_python_url = 'https://raw.githubusercontent.com/tebelorg/TagUI-Python/master/tagui.py'
    if not download(tagui_python_url, 'tagui_update' + '/' + 'tagui.py'): return False

    delta_list = ['tagui', 'tagui.cmd', 'end_processes', 'end_processes.cmd',
                    'tagui_header.js', 'tagui_parse.php', 'tagui.sikuli/tagui.py']

    for delta_file in delta_list:
        tagui_delta_url = 'https://raw.githubusercontent.com/tebelorg/Tump/master/TagUI-Python/' + delta_file
        tagui_delta_file = 'tagui_update' + '/' + delta_file
        if not download(tagui_delta_url, tagui_delta_file): return False

    import shutil
    shutil.make_archive('tagui_update', 'zip', 'tagui_update')

    # next define string variables of update.py header and footer to be used in next section
    update_py_header = \
"""import tagui as t
import platform
import base64
import shutil
import os

tagui_update_zip = \\
"""

    update_py_footer = \
"""

# create update.zip from base64 data embedded in update.py
update_zip_file = open('update.zip','wb')
update_zip_file.write(base64.b64decode(tagui_update_zip))
update_zip_file.close()

# unzip update.zip to tagui folder in user home directory
if platform.system() == 'Windows':
    base_directory = os.environ['APPDATA'] + '/tagui'
else:
    base_directory = os.path.expanduser('~') + '/.tagui'
t.unzip('update.zip', base_directory + '/src')
if os.path.isfile('update.zip'): os.remove('update.zip')

# make sure execute permission is there for .tagui/src/tagui and end_processes
if platform.system() in ['Linux', 'Darwin']:
    os.system('chmod -R 755 ' + base_directory + '/src/tagui > /dev/null 2>&1')
    os.system('chmod -R 755 ' + base_directory + '/src/end_processes > /dev/null 2>&1')

# to move updated package file tagui.py to correct package folder from import tagui as t
# shutil.move(base_directory + '/src/tagui.py', os.path.dirname(t.__file__) + '/tagui.py')

# to add code to remove python-generated tagui.pyc and __pycache__/tagui.cpython-37.pyc

# to add code on getting the correct updated version number for use in next section

# create marker file to skip syncing delta files next time for current release
delta_done_file = t._py23_open(base_directory + '/' + 'tagui_python_' + t.__version__, 'w')
delta_done_file.write(t._py23_write('TagUI installation files used by TagUI for Python'))
delta_done_file.close()
"""

    # finally create update.py containing python code and zipped data of update in base64
    try:
        import base64
        dump(update_py_header, 'update.py')
        update_zip_file = open('tagui_update.zip','rb')
        zip_base64_data  = (base64.b64encode(update_zip_file.read())).decode('utf-8')
        update_zip_file.close()
        write('"""' + zip_base64_data + '"""', 'update.py')
        write(update_py_footer, 'update.py')

        # remove temporary folder and downloaded files, show result and usage message
        if os.path.isdir('tagui_update'): shutil.rmtree('tagui_update')
        if os.path.isfile('tagui_update.zip'): os.remove('tagui_update.zip')
        print('[TAGUI][INFO] - done, copy update.py to your target computer and run')
        print('[TAGUI][INFO] - python update.py to update package to latest version')
        return True

    except Exception as e:
        print('[TAGUI][ERROR] - ' + str(e))
        return False

Dummy comment in order to update readme with direct link here, before publishing v1.20

Thanks for @kensoh update, I think this would be a good solution without network access, but if you can support proxy, which can help users who use the proxy to access the network, it will make this process easier
import tagui as t t.set_proxy('https://proxy.wal-mart.com:8080') t.update()
e.g. https://www.nltk.org/data.html

You can also update through a proxy in the following ways:
$ http_proxy="http://proxy.com:8080" $ export http_proxy $ sudo python test.py

Thanks @bobpan for your feedback, and sharing on proxy!
I'll make a new issue referencing this solution, so other users can add on issues with proxy to work for a better solution like a t.proxy() function to set the proxy to use directly in script.

I'm from China. Due to reasons that are well known, I can't access the github network effectively, so I used the manual download and replacement method of the file in tagui_delta (). After the operation is completed, the 'rpa_python' + version file needs to be created to check whether the operation is completed.

@4linuxfun what you said is correct and works.
The pack() and update() functions depends on a user computer being able to access automated downloads from github.com domain, where the TagUI files are hosted. If the domain can't be accessed by the package, your hack above will work to update package and do initial installation. For initial installation, may also need to hack setup() function.
Is it that in China, for a normal user home laptop without company IT policy restriction, pack() and update() cannot work when it tries to download from github.com domain? I created pack() and update() functions assuming a user can use it from home laptop. Then copy the generated file over to work laptop with restricted internet access.

I use the Proxifier tool proxy and can download the tagui package, but I cannot download the package under https://raw.githubusercontent.com/tebelorg/Tump/master/TagUI-Python/

I see, maybe the way GitHub serves the URL (perhaps redirection) does not work with this proxy tool. Is there any web hosting sites that are free and accessible to China's normal internet?
If there is, you can consider downloading manually the required files, then host the files there. After that fork a version of this project and change all the https://raw.githubusercontent.com and https://github.com URLs to your hosted location. And perhaps publish a new package rpa_cn on pypi.org so users in China can use this package instead of the rpa package.
This tool is very stable and almost no significant change already past 6 months. So there will be no new releases that require you to keep updating your fork or rpa_cn Python package.


Interceptor doesn't work for count query

Current Behavior 😯
After adding soft deletion to entity and fetching graph gql query with total I see the following queries:
2022-12-27T23:23:26.912+0200    INFO    dialect/dialect.go:79   driver.Query: query=SELECT COUNT(DISTINCT "users"."id") FROM "users" args=[]
2022-12-27T23:23:26.914+0200    INFO    dialect/dialect.go:79   driver.Query: query=SELECT DISTINCT "users"."id", "users"."created_at", "users"."updated_at", "users"."deleted_at", "users"."email", "users"."password", "users"."first_name", "users"."last_name", "users"."roles", "users"."status", "users"."phone", "users"."language", "users"."telegram_id", "users"."last_login_at" FROM "users" WHERE "users"."id" > $1 AND "users"."deleted_at" IS NULL ORDER BY "users"."id" ASC LIMIT 4 args=....

The first query without "users"."deleted_at" IS NULL
The second query with "users"."deleted_at" IS NULL

Expected Behavior 🤔
Count query should be with intercepted predicate

Closed with #3194.
Thanks for reporting this, @taraktikos.


Allow setting a (pseudo)4-part title

This title: "Information technology -- Security techniques -- Encryption algorithms -- Part 5: Identity-based ciphers" (https://www.iso.org/standard/59948.html)
Is a 3 part title with the "middle" part split into two.
:title-intro-en: Information technology
:title-main-en: Security techniques -- Encryption algorithms
:title-part-en: Part 5: Identity-based ciphers

This generates this title:
Information technology — Security techniques -- Encryption algorithms -- Part 5: Identity-based ciphers

The middle "--" is used instead of the long dash. I propose we use "--" in source to represent long dashes, and at the end when generating HTML / Word we use "—". The official title as you see on ISO, uses "--".

The conversion of -- to — should be automatic in Asciidoctor. Will check.

Right, it isn't automatic because doc headers are not subject to Asciidoctor's substitutions. Will postprocess in code.

All normal special character replacements that happen in Asciidoctor now apply to title components from the document header.


[REEF-506] Make Driver call DriverRestartActiveContextHandler instead of have the Evaluator decide when to call it

This addressed the issue by

Call restart handlers by checking the restart status based on DriverRestartManager instead of using the protobuf message from the recovered evaluator.

JIRA:
REEF-506

@afchung I've done a first pass.

@markusweimer Updated. Please have another look over. Thanks!

@markusweimer By the way, do you think we should remove the protobuf properties? Or should we leave them there for now?

By the way, do you think we should remove the protobuf properties? Or should we leave them there for now?

Leave them for compatibility, but file a JIRA to remind us to delete them after 0.13 is released.

@markusweimer Updated. Please have another look over. Thanks!

@afchung Thanks, I am doing another pass.

This looks good, I will test (and merge).


3050003: eosio_assert_message_exception "eosio_assert_message assertion failure"

I receive this error when trying to vote for 30 BP's

sorry doppler


Capture Attribute not supported in 4.2

This node gives errors becouse of the new 4.2 update:

Hi @Dangry98, Blender 4.2 is not supported by NodeToPython yet. The capture attribute node should be supported in the Blender-4.2 branch, but this is experimental and may not be fully stable yet. I'm expecting to release these changes in the coming weeks

Thanks! Really appreciate the work you are doing! The add-on is invaluable! Hope you consider adding it to the Blender Extension site as well :)

Glad to hear it's helpful! I'm planning on rolling it out to the extensions platform with the NTP 3.2 update, which will include support for creating the new manifest files, as well

Resolved with https://github.com/BrendanParmer/NodeToPython/pull/117


html_tools/fix: Let etree decide file type and encoding.

closes #1044 #1546

@dgtlmoon Did you change the build test? or does Mine(removeed container build etc in my fork repo.) affect yours?

i change nothing

this needs a test that fails before the fix and passes after

sorry to be annoying, but please, I do not wish to include someone elses copyright HTML in our project.
please remove tests/non_UTF_8_XPath_extraction_HTML.bin
then do something like
    with open("test-datastore/endpoint-content.txt", "wb") as f:
        f.write(b'<html> some binary \b not UTF-8 example</html>')

<title>Clash Royale (@ClashRoyale) | nitter</title>

@dgtlmoon No worries. The reason I said it before is because I'm not familiar with HTML and CSS. Thanks for the how-tos.

Yes, you can use XPath, and It won't break text encoding.

Also, this kind of change may alert false positive changes for one time. (which means literally nothing changed in the information for humans, but bytes changed) It's because encoding is changed.


Docs example doesnt work

The example on the "deno" page of the repo<EMAIL_ADDRESS>doesnt work. The correct version of this is on the deno blog: https://deno.com/blog/build-a-graphql-server-with-deno. The problem is that the addr parameter in the new Server() doesnt work anymore and instead just port: 3000 or any other port should be passed.
Pls fix soon as this confuses A LOT of people.
Greetings, Alex

will fix tomorrow, sorry for inconvenience


bug: menu break in NextJs production build

What version of daisyUI are you using?
^3.0.2
Describe your issue
Description :
When running locally menu link item has the following CSS rules applied :

But when once you build and serve it the above CSS rules are not added to menu link item
Reproduction steps :

Open https://stackblitz.com/edit/daisyui-nextjs-lzc8vy : This is minimal reproduction repo forked from daisyUI nextjs template and I have just copied menu with icons & badge

Terminate process in terminal and build the app -> npm run build

Serve the exported build -> npm run start

You will the menu item are broken vertical instead of being horizontal :

Notes :
This also breaks in Next version ^13.1.5 here is the repo link to test it out : https://github.com/scaffold-eth/scaffold-eth-2
Some references discussion :
1 . https://github.com/saadeghi/daisyui/discussions/2272
2 . https://github.com/saadeghi/daisyui/discussions/2183
What browsers are you seeing the problem on?
All browsers
Reproduction URL
https://stackblitz.com/edit/daisyui-nextjs-lzc8vy

Update Next.js to the latest version.
It was a CSS processing bug in Next.js but it's fixed now.

@saadeghi I have a similar issue in a create-react-app production build:
Menu in dev:

Menu in prod:

What can I do?
Thanks!

@wuarmin can you update all the dependencies (especially the ones about build tool or CSS processing)?
Because it's not an issue from daisyUI, it was a bug from CSS build tool which must be fixed by the latest version.
If it didn't help, please share a reproduction repo.

@saadeghi Thanks! An upgrade helped. I don't know which package caused the error.


Add OOB invitation_msg_id to connection record

This should allow the Aries agent test harness back channel to associate the invitation with the connection.
@nodlesh

Codecov Report

Merging #867 (2717fcd) into master (b9c59da) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #867   +/-   ##
=======================================
  Coverage   99.06%   99.06%
=======================================
  Files         293      293
  Lines       15981    15983    +2
=======================================
+ Hits        15832    15834    +2
  Misses        149      149


add a read-only-mode server setting

Signed-off-by: David Huffman<EMAIL_ADDRESS>

hold off on this one, want to do something client side


Objects with RoleACL result in (Object not Found) when updating.

Hello,
I've been creating and saving an object in Cloud.
The User logged in on iOS and is in the ADMIN_ROLE.
Parse-Server v2.1.1 is running locally with mongo db.
The Cloud Code :
        ... var roleACL = new Parse.ACL();
        roleACL.setPublicReadAccess(false);
        roleACL.setPublicWriteAccess(false);

        // Read & Write by Admins
        roleACL.setRoleWriteAccess('ADMIN_ROLE', true);
        roleACL.setRoleReadAccess('ADMIN_ROLE', true);
        object.setACL(roleACL);

        return object.save();  //## Here the object saves successfully
        }).then(function(object) {
        //Do some modifications
        return object.save(); //##  the object fails to save using save()
        return object.save({},{useMasterKey: true}); //##  the object saves successfully
        return object.save({},{sessionToken: request.user.getSessionToken()});//##  the object fails to save
        ...

Shouldn't the save();   without master key be enough in this situation to modify the object since the User is in the Read/Write Role assigned ? or should the master key be always passed ?
this is the Object in the mongo db
{     "_id": "NZ3NGhdbBV",     "_rperm": [         "role:ADMIN_ROLE",     ],     "_wperm": [         "role:ADMIN_ROLE",     ], ...

Also i tried the following:
        ... var roleACL = new Parse.ACL();
        roleACL.setPublicReadAccess(false);
        roleACL.setPublicWriteAccess(false);

        // Read & Write by The User
        roleACL.setWriteAccess(currentUser, true);
        roleACL.setReadAccess(currentUser, true);

        return object.save();  //## Here the object saves successfully
        }).then(function(object) {
        //Do some modifications
        return object.save(); //##  the object fails to save using save()
        return object.save({},{useMasterKey: true}); //##  the object saves successfully

Please try with 2.1.2 which includes #374.
That said, you would need to pass the sessionToken or useMasterKey, since the 'current user' does not exist in node and would not be sent along with the request.

My mistake, i was not aware of 2.1.2
I updated to 2.1.2 and the problem was solved using your suggestion.
Thank you.


[WIP] Add Puppeteer.ConnectAsync + tests

closes #77
One is failing and not sure why yet!
ShouldBeAbleToReconnectToADisconnectedBrowser fails as it doesn't seem to be discovering the same targets?
Will debug further!

This is for v1.0 @cdroulers.
All issues for v0.4 are taken, You can go and see something for v0.5 https://github.com/kblok/puppeteer-sharp/projects/6

I'm trying out browserless.io, so I needed the connect method! I'll check out the v0.5.
https://github.com/kblok/puppeteer-sharp/issues/55 seems to point to a non-existing line anymore and I can't figure out what it pointed to before since I'm not very familiar with all the code yet!

@cdroulers yeah I targeted to master in some of them (doh). I didn't go through v0.5 yet to check those out. I should!
So if you do need this feature, go for it!
I'm just trying to keep everything in order while new people come here to code :)

I'm super interested in our progress on https://www.browserless.io/ BTW!

Ok, I fixed the basic problem that the using around the original page would close it on .Dispose.
But now, the test hangs indefinitely at https://github.com/kblok/puppeteer-sharp/blob/master/lib/PuppeteerSharp/Page.cs#L249 and I have no idea why!
I've tried commenting out the line with Runtime.enable and then it doesn't hang, but the FrameDump method doesn't return all frames.
I'm guessing the Runtime.enable somehow makes it work, but I don't know why it hangs or where to start looking. Any pointers? Otherwise I'll keep digging in the JAvaScript code!

I think I broke the AppVeyor Build! :(

ShouldBeAbleToReconnectToADisconnectedBrowser is getting stuck

Just pushed code review fixes. Will keep trying to figure out why the test hangs (I skipped it in my commit so AppVeyor wouldn't break!
@kblok For the test getting stuck, I described my issue previously, I'm not sure where to start looking!

Well, your PR turned green :)

Also, would you mind if I added some DebuggerDisplay attributes to classes to make debugging easier?
e.g. for Page:
`[DebuggerDisplay("Page {Url}")]

@cdroulers go for it, but let's do this little by little so PRs don't get too big.

GitHub is telling me there's still requested changes asked by @kblok, but I fixed those and apparently, there are conflicts but if I merge upstream/master, everything is up to date! Any ideas?

Looking good @cdroulers, I just made a comment regarding a test. and this is also conflicted now.

@cdroulers Let's put the same test Puppeteer has, I'll take a look tonight. We should always honor Puppeteer tests.
On the merge side, notice that your upstream is akinoxsolutions it should be kblok.

 :P
I know we want to always do it! Hopefully you see something I don't! If you
have pointers for where to debug after the connection sends Runtime.enable,
I'd love to keep digging!
On 16 April 2018 at 10:34, Darío Kondratiuk<EMAIL_ADDRESS>wrote:

@cdroulers https://github.com/cdroulers Let's put the same test
Puppeteer has, I'll take a look tonight. We should always honor
Puppeteer tests.
On the merge side, notice that your upstream is akinoxsolutions it should
be kblok.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kblok/puppeteer-sharp/pull/137#issuecomment-381620474,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArQvoB_LQLMRoKb1HrD9MbId9P1gdALks5tpKwJgaJpZM4TTwBg
.

@kblok I cannot figure out what's wrong. Once it sends Runtime.enable, it seems to just not get any reponses back and hang there forever!

This is what I've done so far:

Pulled from master again
Restored old tests
Add a fix in frames

You can resume from there. I don't want to get distracted by this features, because it's for v1.0. You can keep working on this though.

@cdroulers I can buy that, but your PR should build in the CI, and I should get @Meir017 approval.

Could you take a look at this guy @Meir017?

Merged @cdroulers!
I'm really interested in your progress on https://www.browserless.io/ please send me an email when you have something to show.


Data Validation for Account Verification

Most recent commit: 3cdcd5762008f263abb8464b33264d06536834da
There should be data validation for the verification code field on the Verification Page. The field already only permits numbers, but the data should be validated as a 6-digit number.

@malakivandas hello, I'd love to help with this issue.
I was thinking you can use regex patterns to define the conditions.


develop

fix: minor things
feat: add objectives section
feat: add Wave component
feat: add Objectives section and Objective component
fix: minor things
feat: add ButtonScrollToTop component
feat: add NavBar Floating when scroll > 100vh
feat: add Project section, Project component and Loading component
feat: add Project page and improve projects api
refactor: menu top in mobile device
refactor: side menu in mobile
refactor: side menu close button in mobile
refactor: main menu in tablet/desktop layout
fix: navbar z-index
fix: export default in page Project
fix: pages structure
feat: add license in package.json
feat: add input search
fix: navbar animation
fix: navbar icon alignment
fix: hero icons z-index
fix: add content visibility in sections
feat: add project section button
fix: set antialiasing property in body and img tags
fix: identation in project file
refactor: project component to card component
feat: add post api
fix: post api renamed
fix: loading component size
feat: add Post section
refact: project section
feat: add ContentFul Render component
feat: add posts page
feat: add info fields in slug pages
feat: add tag in card component
feat: add share button component
fix: shadow in wave component
fix: slug pages

contains build error


Go: support the Go vulnerability database/service

Is there an existing issue for this?

[X] I have searched the existing issues

Feature description
The Go team maintains a (new) vulnerability database and service, as explained in https://go.dev/blog/vuln. Contrary to many other SAST tools which purely look at dependencies, the vulnerability DB and the govulncheck utility understand if the vulnerable code path is every (transitively) called by your program (for any modules the Go proxy is aware of).
This makes the alerts much more actionable and purposeful by not simply triggering on whether a vulnerability exists in a version, but if the vulnerability is reachable from your code.
It would be great if Dependabot checked with this database.

This repo is about the code that creates PR's to bump versions--whether from cron or from security alerts--but not about the security alerts themselves. Versus this issue is about the alerts themselves, not the resulting PRs.
So I'm afraid I'll have to close this.
I suggest starting a discussion over in https://github.com/community/community/discussions/categories/code-security as that's probably the best way to reach the team that handles alerts. Feel free to drop a breadcrumb link on this issue to the discussion.
I do know that team is already aware of the go vulnerability work, as I was one of the folks who internally notified them when it first shipped, although I haven't been involved further.


[FEATURE] ES2023 Support

Ensures valid build outputs also for projects using latest ES2023 syntax.
Hashbangs e.g. '#!/usr/bin/env node' are allowed in source files but won't be included in bundles (e.g. Component-preload.js) by UI5 Tooling.
JIRA: CPOUI5FOUNDATION-850

coverage: 94.621% (+0.005%) from 94.616%
when pulling 508c3019a0b50b9fc92799399616348f108aadb2 on es2023-support-v3
into 19ee6a29b79c45bbe0125918886bcd1870255332 on v3.

coverage: 94.621% (+0.005%) from 94.616%
when pulling 30301b24f27beb220c73e0bcc255f1ec806ab819 on es2023-support-v3
into 19ee6a29b79c45bbe0125918886bcd1870255332 on v3.


Make a Product Backlog

The Game must have a Backlog so we know all the functionality that is needed to make tasks.

This is the first backlog made on 20/12/2021, the whole backlog can be found in our Teams


Document design for transport::VSomeIpUTransport

This should cover anything extra being done to support the SOME/IP transport. Are additional threads required? Or a thread pool? Any locking needed? Will this implementation be doing anything special based on message type (e.g. modifying messages, operating in specific modes based on message type, etc.)? If so, how will that be done?

As discussed,
This task is not a single person task and need a breakdown. I will work on this issue so for tracking purpose you can assign this task to me.

I shall be contributing to this task as well


mirror autokit.apk

Hey, started fiddling with some carplay stuff, just trying to get this code up and running, the autokit apk (from carlinkkit) seems to download agonizingly slowly. Do you think mirroring it (either in this repo, or elsewhere) could be a decent idea?

Heya, not sure whether mirroring it elsewhere would be legal. 4x4shop.ca mirrors the last decent version of it, though

Heya, not sure whether mirroring it elsewhere would be legal. 4x4shop.ca mirrors the last decent version of it, though

ah true. it finally downloaded for me! got most things working, too. I've started working on a port to Go for better performance, inspired a lot by this code and your decompiled apk gists. Hoping I can get audio all sorted, as that's the biggest issue with a Sony XAV-AX7000 head unit I have - it crashes fairly often when the audio bitrates change and leave me without carplay for a while.

ah true. it finally downloaded for me! got most things working, too. I've started working on a port to Go for better performance, inspired a lot by this code and your decompiled apk gists. Hoping I can get audio all sorted, as that's the biggest issue with a Sony XAV-AX7000 head unit I have - it crashes fairly often when the audio bitrates change and leave me without carplay for a while.

ah nice! Yeah the audio stuff on this project could definitely benefit from some extra work. At the moment i'm using a separate mpv thread to play it back.  I've been meaning to get around to it- Would definitely benefit from directly writing to something like libalsa so that you can accomodate changes in bitrate for things like phone calls and siri

ah nice! Yeah the audio stuff on this project could definitely benefit from some extra work. At the moment i'm using a separate mpv thread to play it back.  I've been meaning to get around to it- Would definitely benefit from directly writing to something like libalsa so that you can accomodate changes in bitrate for things like phone calls and siri


Spinner When Missing Addons for Stock Notification

tbd

Refers to #5123

same issue with https://github.com/SAP/cloud-commerce-spartacus-storefront/issues/5572

as stock notification is cms based, you would add the cms (or remove it) based on if you want this feature, so closing as not a bug, as designed.


Trouble building a dream project with dune

Hi, I saw Dune on Lobste.rs and the design looks real good!
I tried to run in my machine with no success.
My project consists of:

dune

(executable
   (name main)
   (libraries dream))

dune-project

(lang dune 2.0)

main.ml

let () =
  Dream.run (fun _ ->
    Dream.respond "Good morning, world!")

When I do dune build, I get this error:
Entering directory '/Users/rodrigo/dream'
File "dream/src/vendor/h2/lib/settings.ml", line 284, characters 7-28:
284 |        Angstrom.parse_string
             ^^^^^^^^^^^^^^^^^^^^^
Error: This function has type 'a Angstrom.t -> string -> ('a, string) result
       It is applied to too many arguments; maybe you forgot a `;'.

my opam is 2.0.8, ocaml is 4.10.2, running on windows
also, opam install dream does not work for me:
opam install dream
[ERROR] No package named dream found.

Can anyone help me? Thanks in advance.

Hi! This is probably an issue with some incompatible versions of dependencies.
I assume you cloned Dream. Go to the clone's root, and run
opam install --deps-only .
That should install a consistent set of deps.

also, opam install dream does not work for me:

Yes, Dream is unreleased :) I only made the repo public, but haven't announced or published the actual package yet :) As you can see in the README,

TODO opam release, esy instructions, quick-start script.

I should probably clarify this message slightly, and add more accurate instructions for people that want to try it this early!

Added this to the README:

Dream is not yet announced or released! Before release, Dream will get a quick-start script, and some other help for getting started quickly. However, if you want to try Dream now, you can do:
git clone https://github.com/aantron/dream.git --recursive
cd dream
opam install .

After that, go to one of the examples, such as 1-hello, and try building it:
cd example/1-hello
dune exec --root . ./hello.exe

You should be able to copy the example out to a completely separate directory, if you didn't use a local _opam switch scoped to Dream's clone directory. If you did, you can pin Dream to your clone in a different opam switch with
opam pin add dream /path/to/your/clone

If that doesn't fix it, could you say what you did to install Dream and/or its deps, and show the version of package angstrom with opam list --installed angstrom?

Hi aantron, thanks, I will try this as soon as I finish work. Really excited to test this :)

Great :) and thanks for the gist!


Use deserialize_str for ISO 8601

Follow-up to https://github.com/time-rs/time/pull/469 for recently added ISO 8601.

The diff is fine, but can you change the commit message? It's using deserialize_str and is switching away from deserialize_any.

Oof, copy-paste error.

It happens! Thanks.

Thanks!


Stuck on "Loading content"

Hey Joey - nice work on the app! Much appreciated. I wonder if I'm doing something wrong, though. I've loaded the app, and it sees all of the playlists on my Plex server with no problem. I've got a strong LTE connection and other network related things work well. But every playlist I try gets stuck on "Loading content". How long should it take? I've tried a playlist with 200 songs, and twenty minutes later it has still not completed. Is there something I should be doing besides waiting longer? Thanks!

Just jumping in here - I've seen the same behavior since yesterday when my grandson was poking around and tried to load a different playlist. When trying to restore the one I wanted to listen to, I was stuck on "Loading content" also. I have not tried clearing local cache or local data yet, but if it is still exhibiting this behavior I'll report back.

No progress here. I've cleared out the cache and the data, removed and reinstalled the app, and rebooted the infotainment system with no effect. Except one time it actually stopped and said "something bad happened. Please retry". It's on the latest version (11/14 as far as I can tell). Is there something else I should be trying?

that's pretty much all that can be tried. I'll prioritize this issue as it seems to be a connectivity related bug, although I probably won't have time to look into it for at least a couple weeks. chances are when I improve the overall connection logic, it will self-resolve, or I'll have a better idea as to what's failing.

Just to update, I've done everything I can think of: clear cache, clear data, even uninstall and then reinstall. I just get never ending "loading content" now, and don't even see the playlists show up. Something must be either persistently cached locally that survives an app removal/reinstall (unlikely), or something on the Plex server side may think that device is still active. Hmm, maybe I'll look at my Plex instance and see if I can "kill" the device from there.

i'm not sure if the loading has been more frequent lately, but I've definitely seen it although I've always gotten the app into a working state by navigating back to the main page and selecting a different playlist. sometimes selecting the same playlist didn't resolve it, but selecting a different playlist more consistently gets it into a good state.

OK, no worries. I can run the full Plex app through carplay if needed, but I'd prefer the AAOS native app. No rush. Thanks!

I tried removing Plex AAOS from the Authorized Devices page on my Plex instance and then reinstalled from the car, still no joy. I'll see if it "heals" after an infotainment reset on the car next chance I get, but I may just need to wait for the next release. Thanks for working on this, your efforts are much appreciated!

some info posted on #1 - loading is improved a bit with version 0.4.4, although still expected to fail if the app is opened without an internet connection. in those scenarios, if stuck on loading, the solution for now is to just open another media app and return back, although I'll get to adding a retry mechanism when internet is unavailable - usually the case for the first ~30 seconds after car startup.

Hey Joey – I gave it another try. It looks like a playlist size issue. I can get “short” playlists (less than 100 songs) to load fine. I have one at 233 songs, and it hangs forever.  Thanks, Greg From: Joey BerkovitzSent: Wednesday, December 28, 2022 2:43 PMTo: joeyberkovitz/plex-aaosCc: gregcagle; AuthorSubject: Re: [joeyberkovitz/plex-aaos] Stuck on "Loading content" (Issue #10) some info posted on #1 - loading is improved a bit with version 0.4.4, although still expected to fail if the app is opened without an internet connection. in those scenarios, if stuck on loading, the solution for now is to just open another media app and return back, although I'll get to adding a retry mechanism when internet is unavailable - usually the case for the first ~30 seconds after car startup.—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
understood, I'll have to try with a larger playlist. the largest one I tried is ~120 songs and it seems to work fine on the latest version. sorry if I wasn't clear, but the improvements are in version 0.4.4 which was just submitted to Google for review, likely to be released over the next couple days, maybe next week.

Cool, I’ll try it when I see it. I ran some more tests, a 158 song playlist loads immediately, where a 263 song playlist never did load after five minutes. Maybe it’s a 256 song limit (?). Just a thought. Thanks, Greg From: Joey BerkovitzSent: Wednesday, December 28, 2022 3:47 PMTo: joeyberkovitz/plex-aaosCc: gregcagle; AuthorSubject: Re: [joeyberkovitz/plex-aaos] Stuck on "Loading content" (Issue #10) understood, I'll have to try with a larger playlist. the largest one I tried is ~120 songs and it seems to work fine on the latest version. sorry if I wasn't clear, but the improvements are in version 0.4.4 which was just submitted to Google for review, likely to be released over the next couple days, maybe next week.—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
definitely possible that the API changes with larger playlists or there's some resource limit somewhere. I'll have to put together a large playlist test and see what's going on

I'll give it a shot next time I drive. BTW, I can load smaller playlists but the ones that wouldn't load were quite large (thousands of tracks).

got it, thousands of tracks may be stretching things even if there was no breakage in general. I have to see if the media player (ExoPlayer) supports some sort of dynamic playlist loading because I think that even trying to load a large playlist like that would just generally be problematic and slow

Understood and do not spend extra effort on it unless you really want to. I can live with a 250ish track limit. Thanks! From: Joey BerkovitzSent: Wednesday, December 28, 2022 8:29 PMTo: joeyberkovitz/plex-aaosCc: gregcagle; AuthorSubject: Re: [joeyberkovitz/plex-aaos] Stuck on "Loading content" (Issue #10) got it, thousands of tracks may be stretching things even if there was no breakage in general. I have to see if the media player (ExoPlayer) supports some sort of dynamic playlist loading because I think that even trying to load a large playlist like that would just generally be problematic and slow—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
Hi Joey - I'm still not able to load large playlists even though I was able to with the first public release. Just curious if you're still working on this?

Yes, although I haven't had much time for this project lately, so when I get to it, I can look into it.

Thanks, no hurry!

Sounds good, I've been bogged down rebuilding my Plex server anyway :).

From: Mick Smothers @.>
Sent: Tuesday, March 28, 2023 6:23:27 PM
To: joeyberkovitz/plex-aaos @.>
Cc: gregcagle @.>; Author @.>
Subject: Re: [joeyberkovitz/plex-aaos] Stuck on "Loading content" (Issue #10)
Thanks, no hurry!
—
Reply to this email directly, view it on GitHubhttps://github.com/joeyberkovitz/plex-aaos/issues/10#issuecomment-1487815736, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOR34L2NCSLDNFAW3MHS46TW6OFI7ANCNFSM6AAAAAASILC4FQ.
You are receiving this because you authored the thread.Message ID: @.***>

Version 0.4.5 will should be available in beta over the next few days (pending review from Google). If you have time, please join the beta, once it's available, here (https://play.google.com/apps/testing/us.berkovitz.plexaaos) and let me know if the new functionality works well.
I tested it on the full library playlist, in my case a few thousand items, and it worked. If all is good, I'll push to production in 1-2 weeks.
If you'd like to try it earlier, feel free to send over your email address, either here or to<EMAIL_ADDRESS>and I can add you to the internal testing group which gets updates immediately.

Will do – thanks! From: Joey BerkovitzSent: Thursday, April 13, 2023 11:04 AMTo: joeyberkovitz/plex-aaosCc: gregcagle; AuthorSubject: Re: [joeyberkovitz/plex-aaos] Stuck on "Loading content" (Issue #10) Version 0.4.5 will should be available in beta over the next few days (pending review from Google). If you have time, please join the beta, once it's available, here (https://play.google.com/apps/testing/us.berkovitz.plexaaos) and let me know if the new functionality works well.I tested it on the full library playlist, in my case a few thousand items, and it worked. If all is good, I'll push to production in 1-2 weeks.If you'd like to try it earlier, feel free to send over your email address, either here or to<EMAIL_ADDRESS>and I can add you to the internal testing group which gets updates immediately.—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
I'm going to push this change to production over the next few days. Please let me know if I can close this issue or if there are any remaining concerns related to this problem

Ship it 😊. I was able to load several different 3000+ song playlists with no problem. Good work. From: Joey BerkovitzSent: Tuesday, April 25, 2023 6:13 PMTo: joeyberkovitz/plex-aaosCc: gregcagle; AuthorSubject: Re: [joeyberkovitz/plex-aaos] Stuck on "Loading content" (Issue #10) I'm going to push this change to production over the next few days. Please let me know if I can close this issue or if there are any remaining concerns related to this problem—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
Great, closing this issue as fixed by #14
Rollout to production is pending approval by Google and should occur over the next few days. Feel free to open a new issue if any other problems arise

@joeyberkovitz - I'll test it also with my massive playlist once I get the update. My small playlists still load, but the big ones were of course broken. It will be an additional datapoint. Thanks for this work!


How do I show a compass(bearing) on top of the map?

VERSIONS

Xamarin.Forms.GoogleMaps - x.x.x
Xamarin.Forms - x.x.x

PLATFORMS

[ ] Android
[x] iOS
[ ] UWP

ACTUAL BEHAVIOR

ACTUAL SCREENSHOTS/STACKTRACE

// paste stacktrace

EXPECTED BEHAVIOR

HOW TO REPRODUCE

// code for reproduce

You can use Map.UiSettings.CompassEnabled = true .

Thanks, one more question. How can I rotate the map automatically to the direction the phone is facing?


ACCESS-ESM1.5 journal paper

The DOI points to this url
https://www.publish.csiro.au/ES/ES19035
which says it is licensed under a CC BY-BC-ND license
https://creativecommons.org/licenses/by-nc-nd/4.0/
which says

You are free to:
Share — copy and redistribute the material in any medium or format
The licensor cannot revoke these freedoms as long as you follow the license terms.
Under the following terms:
Attribution — You must give appropriate credit , provide a link to the license, and indicate if changes were made . You may do so in any reasonable manner, but not in any way that suggests the licensor endorses you or your use.
NonCommercial — You may not use the material for commercial purposes .
NoDerivatives — If you remix, transform, or build upon the material, you may not distribute the modified material.
No additional restrictions — You may not apply legal terms or technological measures that legally restrict others from doing anything the license permits.

So if we want to have a PDF copy I think we should start a README.md that gives a brief description of what it is, where it came from and link to the license terms.


[DRAFT] Ported as conceptual article from .NET dev blog: Regex enhancements with .NET 7

Summary
Ported an existing .NET dev blog from @stephentoub into the .NET docs, as a new conceptual article introducing Regex enhancements for .NET 7:

Rephrased wording from passive to active.
Replaced phrases such as "I" or "we", with "you" where appropriate.
Fixed markdown warnings and applied markdown linter fixes where possible.
Renamed the image files and ran them through image compression and resizing.
Added to TOC with the corresponding title.

Fixes #30289

Thanks, @IEvangelist.  Though FWIW I find this out-of-place.  It's one thing to have conceptual docs about GeneratedRegex, but this post was about things that are new in .NET 7; it's not limited to just the source generator, there's a lot of implementation detail in here, and there's a lot more to regex than just what's discussed in this post, since it's written about the delta from .NET 6 to .NET 7.

Thanks, @IEvangelist. Though FWIW I find this out-of-place. It's one thing to have conceptual docs about GeneratedRegex, but this post was about things that are new in .NET 7; it's not limited to just the source generator, there's a lot of implementation detail in here, and there's a lot more to regex than just what's discussed in this post, since it's written about the delta from .NET 6 to .NET 7.

Thanks, @stephentoub. This is still a draft, with lots of plans for stripping it down - I haven't gotten that far yet.  There is a ton of great info in this post, and I want the conceptual parts to remain after I shake away the implementation details. The current title was an initial thought, but this is all in flux right now, but will most likely be hyperfocused on the source generation bits. Give me a bit to work on this more, and I'll tag you for review when it's ready. Is that fair?

Yup :-)

Thanks for the feedback @stephentoub, I've addressed it in https://github.com/dotnet/docs/pull/31758.


Integrated terminal outputs yellow stdout

Version: 1.35 (snap package)
OS: Ubuntu 16.04
Issue:
Once i run the integrated terminal with zsh and oh-my-zsh, the stdout in the terminal is yellow instead of white, see the attached picture

How to reproduce:
I don't know :/

/duplicate https://github.com/microsoft/vscode/issues/75021


Support asynchronous error handling

This PR allows the error handler to be asynchronous. This can be useful for handlers that alter the error, middleware that needs to run before returning the error and makes testing asynchronous error middleware easier.

Would it be possible to add a description to this PR?

Would it be possible to add a description to this PR?

My bad, forgot to add it when making the PR.

It's not returning anything here, right?

It isn't. You're right, awaiting that function shouldn't be necessary. It will just be running in the background after returning, right?
For unit testing you could probably just do your assertions after a sleep of a few milliseconds if you need to assert whatever onError did.

It isn't. You're right, awaiting that function shouldn't be necessary.

It is, since you can modify the error in the handler, and then if it's not awaited, the error is passed on without the modifications.


[iOS][add2app] Documentation for specifying --dart-define with cocoa-pods method

Steps to Reproduce

Run Script Phase -
export FLUTTER_BUILD_MODE=debug
export DART_DEFINES=VARIANT=staging
set -e
set -u
source "${SRCROOT}/../tap_flutter/.ios/Flutter/flutter_export_environment.sh"
export VERBOSE_SCRIPT_LOGGING=1 && "$FLUTTER_ROOT"/packages/flutter_tools/bin/xcode_backend.sh build

Adding export DART_DEFINES=VARIANT=staging, yields Command PhaseScriptExecution failed with a nonzero exit code
Resultant command
/Users/kd/fvm/versions/stable/bin/flutter --verbose assemble --no-version-check --output=/Users/kd/Library/Developer/Xcode/DerivedData/App3-awnkpawzuxuafqgzhvrgwzglbgaz/Build/Products/Debug-iphonesimulator/ -dTargetPlatform=ios -dTargetFile=lib/main.dart -dBuildMode=debug -dIosArchs=arm64 -dSdkRoot=/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator15.2.sdk -dSplitDebugInfo= -dTreeShakeIcons=false -dTrackWidgetCreation=false -dDartObfuscation=false -dEnableBitcode= --ExtraGenSnapshotOptions= --DartDefines=VARIANT=staging --ExtraFrontEndOptions= -dCodesignIdentity=- debug_ios_bundle_flutter_assets
Issue varified on flutter version 2.8.1 and 2.10.4(stable)
Logs

Showing Recent Messages
♦ /Users/kd/fvm/versions/stable/bin/flutter --verbose assemble --no-version-check --output=/Users/kd/Library/Developer/Xcode/DerivedData/App3-awnkpawzuxuafqgzhvrgwzglbgaz/Build/Products/Debug-iphonesimulator/ -dTargetPlatform=ios -dTargetFile=lib/main.dart -dBuildMode=debug -dIosArchs=arm64 -dSdkRoot=/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator15.2.sdk -dSplitDebugInfo= -dTreeShakeIcons=false -dTrackWidgetCreation=false -dDartObfuscation=false -dEnableBitcode= --ExtraGenSnapshotOptions= --DartDefines=VARIANT=staging --ExtraFrontEndOptions= -dCodesignIdentity=- debug_ios_bundle_flutter_assets

[  +10 ms] Error parsing assemble command: your generated configuration may be out of date. Try re-running 'flutter build ios' or the appropriate build command.
[   +2 ms]
           #0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
           #1      AssembleCommand.runCommand (package:flutter_tools/src/commands/assemble.dart:297:7)
           #2      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1290:12)
           <asynchronous suspension>
           #3      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1140:27)
           <asynchronous suspension>
           #4      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #5      CommandRunner.runCommand (package:args/command_runner.dart:209:13)
           <asynchronous suspension>
           #6      FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:288:9)
           <asynchronous suspension>
           #7      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #8      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:236:5)
           <asynchronous suspension>
           #9      run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
           <asynchronous suspension>
           #10     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #11     main (package:flutter_tools/executable.dart:94:3)
           <asynchronous suspension>

Failed to package /Users/kd/Desktop/data/proj/flutter/tm/tap/tap_flutter.

As per the error log, it seems I am providing the --dart-define params in the wrong format.

@kdcyberdude
Thanks for the report. Can you provide the documentation link that you are following if any, to run below commands as you mentioned ?
export FLUTTER_BUILD_MODE=debug
export DART_DEFINES=VARIANT=staging
set -e
set -u
source "${SRCROOT}/../tap_flutter/.ios/Flutter/flutter_export_environment.sh"
export VERBOSE_SCRIPT_LOGGING=1 && "$FLUTTER_ROOT"/packages/flutter_tools/bin/xcode_backend.sh build

I am following https://docs.flutter.dev/development/add-to-app/ios/project-setup#option-a---embed-with-cocoapods-and-the-flutter-sdk with slight changes. Since it does not work with complex workflows or in other words there is no documentation of how add2app should be integrated with complex apps which have different build environments.
As per the documentation -
install_all_flutter_pods(flutter_application_path) get's called. which internally calls three functions.
Check the file for more information - File.join(flutter_application_path, '.ios', 'Flutter', 'podhelper.rb')
def install_all_flutter_pods(flutter_application_path = nil)
     flutter_application_path ||= File.join('..', '..')
     install_flutter_engine_pod
     install_flutter_plugin_pods(flutter_application_path)
     install_flutter_application_pod(flutter_application_path)
end

Where install_flutter_application_pod function implementation is-
def install_flutter_application_pod(flutter_application_path)
  current_directory_pathname = Pathname.new File.expand_path('..', __FILE__)
  app_framework_dir = File.expand_path('App.framework', current_directory_pathname.to_path)
  app_framework_dylib = File.join(app_framework_dir, 'App')
  if !File.exist?(app_framework_dylib)
    # Fake an App.framework to have something to link against if the xcode backend script has not run yet.
    # CocoaPods will not embed the framework on pod install (before any build phases can run) if the dylib does not exist.
    # Create a dummy dylib.
    FileUtils.mkdir_p(app_framework_dir)
    `echo "static const int Moo = 88;" | xcrun clang -x c -dynamiclib -o "#{app_framework_dylib}" -`
  end

  # Keep pod and script phase paths relative so they can be checked into source control.
  # Process will be run from project directory.

  # defined_in_file is set by CocoaPods and is a Pathname to the Podfile.
  project_directory_pathname = defined_in_file.dirname
  relative = current_directory_pathname.relative_path_from project_directory_pathname
  pod 'tap_flutter', :path => relative.to_s, :inhibit_warnings => true

  flutter_export_environment_path = File.join('${SRCROOT}', relative, 'flutter_export_environment.sh');
  script_phase :name => 'Run Flutter Build tap_flutter Script',
    :script => "set -e\nset -u\nsource \"#{flutter_export_environment_path}\"\nexport VERBOSE_SCRIPT_LOGGING=1 && \"$FLUTTER_ROOT\"/packages/flutter_tools/bin/xcode_backend.sh build",
    :execution_position => :before_compile
end

I have to override the above implementation to update the script_phase to configure the different schemes already defined in iOS.
Internally looking at the files xcode_backend.sh and then xcode_backend.dart I am able to understand that we can configure many things just by exporting a environment variable.

Note these "environment variables" are actually your Xcode build settings.  So instead of adding this to your build phase script, which will be overwritten every time you pod install
export DART_DEFINES=VARIANT=staging

you would instead create a custom user-defined build setting in your host app Xcode target.

If you run your module once with --dart-define and any other options, the env variables will be exported in flutter_export_environment.sh which is sourced before xcode_backend.sh build builds.
  flutter_export_environment_path = File.join('${SRCROOT}', relative, 'flutter_export_environment.sh');
  script_phase :name => 'Run Flutter Build tap_flutter Script',
    :script => "set -e\nset -u\nsource \"#{flutter_export_environment_path}...

Thanks @jmagman, I wasn't aware that these "environment variables" are from Xcode build settings. Instead, I create a custom function in a ruby script which I've used to avoid getting it overwritten by pod install. My bad!!
Still, after specifying the DART_DEFINES environment variable in build settings I am facing the same error as described above. Is VARIANT=staging in the correct format for the DART_DEFINES variable?
I am assuming the DART_DEFINES environment variable is the same as adding the --dart-define=VARIANT=staging while building APK, app bundle, AAR, ios-framework, etc!!

Yeah, the DART_DEFINES value gets decoded. We need to pass the base64 encoded value to DART_DEFINES which can be gotten with this command, echo -n 'VARIANT=staging' | openssl base64 on Linux or macOS system. Thanks, @jmagman.
I am sorry I did not get this statement, You should run with the flags you want in your module and copy them to your host app. Are you talking about building ios-framework with appropriate flags and copying them to the host app?
Although we can close this issue now, I have one suggestion if there could be documentation or any article on how to configure build flags after embedding Flutter using the cocoapods approach that would be really great for Flutter devs or Native devs except iOS, who shifted to Flutter. Because it seems like these "environment variables" are actually an Xcode build settings is a fundamental thing for iOS devs. Documentation should also include the possible "environment variables" available and their respective value because I have to look into the file xcode_backend.dart file to see the possible environment variables.

This should stay open as a request for documentation.

I am sorry I did not get this statement, You should run with the flags you want in your module and copy them to your host app. Are you talking about building ios-framework with appropriate flags and copying them to the host app?

If you build your module once with --dart-define and any other options, the env variables will be exported in flutter_export_environment.sh which is sourced before xcode_backend.sh build builds.

Meaning, in your module run flutter build ios or flutter run with the flags you want.  Then check .ios/Flutter/flutter_export_environment.sh for the values it generated.

Duplicate of https://github.com/flutter/flutter/issues/88092


jira issue create not interactive by default

Describe the bug
On Windows PowerShell:
.\jira.exe issues create
✗ Params `--summary` and `--type` is mandatory when using a non-interactive mode

I was expecting to see an interactive CLI when not using any parameters as indicated by the README.
Please provide following details

JiraCLI Version:

(Version="1.5.1", GitCommit="0e0f82d52ef5a775d2cc662fbe38d9732b9b5c59", CommitDate="2024-01-20T10:48:06+00:00", GoVersion="go1.21.5", Compiler="gc", Platform="windows/amd64")

Are you using Jira cloud or on-premise jira server?

SERVER INFO
-----------

Version:         1001.0.0-SNAPSHOT
Build Number:    100251
Deployment Type: Cloud
Default Locale:  en_US

What operating system are you using? Also mention version.

Windows 10, Version 10.0.19045 Build 19045

What terminal are you using? Also mention version.

    Windows Terminal

To Reproduce
Steps to reproduce the behavior:

Download release
Open powershell in bin folder
Execute .\jira.exe issue create

Expected behavior
An interactive CLI

@JosXa This was fixed by me with #706. Most likely, it will be included in the next release :)

Thank you for the work @beatbrot  :)


All Instruction files in HTML Styled Format

Its just helpful for my students because local machine cannot open instruction in md format

This would be better as a build script. Right now the current build script builds the files into docx format.
The Word documents are available on the releases page. I have opened an issue so we can all discuss this and track the discussion. I believe we can end up with a new build script that also output HTML files.
For the short term, your students can view the Markdown preview pages:
Ex: https://github.com/MicrosoftLearning/20532-DevelopingMicrosoftAzureSolutions/blob/c-release/Instructions/Labs/dotnet/Mod01/20532C_LAB_01.md

I see @pushkar72 's point, but I suggest PDF would be the best format. Windows 8 on has a PDF reader, so there would be no need for an additional download.
The amazing pandoc uility (http://pandoc.org) can convert from Markdown to PDF, so you could use that in the build script.

The great thing about the build stripy is that we can add a switch to output as PDF, HTML or DOCX (default). The reason DOCX was used is mostly due to how MOC is distributed today.

From: Concrete Gannet<EMAIL_ADDRESS>Sent: Tuesday, September 27, 2016 3:25:44 AM
To: MicrosoftLearning/20532-DevelopingMicrosoftAzureSolutions
Cc: Sidney Andrews; State change
Subject: Re: [MicrosoftLearning/20532-DevelopingMicrosoftAzureSolutions] All Instruction files in HTML Styled Format (#130)
I see @pushkar72https://github.com/pushkar72 's point, but I suggest PDF would be the best format. Windows 8 on has a PDF reader, so there would be no need for an additional download.
The amazing pandoc uility (http://pandoc.org) can convert from Markdown to PDF, so you could use that in the build script.

You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftLearning/20532-DevelopingMicrosoftAzureSolutions/pull/130#issuecomment-249786808, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AE1SiagETKbjSMAlkVMb-qEhia7xnibfks5quMT4gaJpZM4KHSNy.


Strike Modal: There is no attribute showCloseButton

We need to add a new attribute showCloseButton on strike_modal component, which would default to true so users can pass false if they want to hide it

Expect this feature in our next release


R Scan fails when there is no renv/activate.R

The Scan button appears when there's no renv.lock file. If you have a renv/activate.R file in the project, the Scan button creates a renv.lock file for you.
If there is no renv/activate.R file, the Scan button will fail with:

An error has occurred at homeView::_onScanForRPackageRequirements, Code=ERR_BAD_RESPONSE, Msg=Request failed with status code 500, URL=http://localhost:61514/api/post/packages/r/scan

Logs:

time=2024-05-29T14:03:37.105-04:00 level=DEBUG msg="Running command" cmd="C:\Program Files (x86)\R\R-4.4.0\bin\R.exe" args="-s -e renv::snapshot(lockfile="c:\Users\kgartland\publisher-client\bundles\mpg-shinyApp\renv.lock")"
time=2024-05-29T14:03:37.298-04:00 level=ERROR msg="Error running command" command="C:\Program Files (x86)\R\R-4.4.0\bin\R.exe" error="exit status 1"
Error in file(filename, "r", encoding = encoding) :
cannot open the connection
Calls: source -> file
In addition: Warning message:
In file(filename, "r", encoding = encoding) :
cannot open file 'renv/activate.R': No such file or directory
Execution halted
time=2024-05-29T14:03:37.298-04:00 level=DEBUG msg=renv::snapshot() out="" err="Error in file(filename, "r", encoding = encoding) : \r\n  cannot open the connection\r\nCalls: source -> file\r\nIn addition: Warning message:\r\nIn file(filename, "r", encoding = encoding) :\r\n  cannot open file 'renv/activate.R': No such file or directory\r\nExecution halted\r\n"
time=2024-05-29T14:03:37.298-04:00 level=ERROR msg="Internal Server Error" method=POST url=/api/packages/r/scan error="exit status 1"
time=2024-05-29T14:03:37.298-04:00 level=INFO msg="Access Log" method=POST url=/api/packages/r/scan elapsed_ms=326 status=500 req_size=24 resp_size=21 client_addr=<IP_ADDRESS>:61559 req.saveName=renv.lock

If the Scan function requires a renv/activate.R file, can we hide the Scan button and ask that they run renv::init() or  renv::snapshot()?
Additionally, f you do have an renv.lock file but no renv/activate.R, and click the eye icon to rewrite the file, it will fail in the same way.

We currently do not have a scan operation for R packages so we can close this up.


GH-41884: [Python] Fix RecordBatchReader.cast to support casting to equal schema for all types

Rationale for this change
The cast from date32 to date32 is not included in the table of cast functions.
What changes are included in this PR?
Ensure we don't error if the types are equal in the initialization of the CastingRecordBatchReader
Are these changes tested?
Yes
Are there any user-facing changes?
No

GitHub Issue: #41884

The current fix is probably more of a stop-gap, but already wanted to test this on the Python side. There are multiple ways to fix this though, and I am not entirely sure if we have some general guidelines about this.
The core issue is that GetDate32Cast() does not explicitly create a zero-copy cast kernel for "date32" to "date32". For most other types, such kernel is included for various reasons (eg for time32 it is included because it potentially needs to cast to a different unit).
So should we:

Ensure that for every type we always include a cast kernel to itself in the cast table? (do we want to rely on the assumption this is always present?)
Handle this one level up, i.e. in this case handle that in CanCast by checking type equality explicitly apart from a kernel being available? (Cast itself already handles this case)
Handle this where CanCast/Cast is being used (like in this PR, although I assume it should be one of the two options above)

Actually doing the cast (i.e. not just checking if it is castable with CanCast) does already work with the example of "date32", but that is because for input and output type being equal, we never get to getting the actual cast function (GetCastFunction) and just return the input as is.
So if we follow that logic, I should maybe update CanCast? Although on the other hand, people might rely on the fact that if CanCast is true, they can get a cast function with GetCastFunction (which in this case would then fail, if I don't actually add the "date32" to "date32" zero copy kernel to the table ..).
cc @pitrou @felipecrv

Boolean is another example where the "bool" -> "bool" cast wouldn't be added through some parametrization, and there we explicitly add it:
https://github.com/apache/arrow/blob/baf4089eaa84b2a5c2cb437f3a249f83577dbc29/cpp/src/arrow/compute/kernels/scalar_cast_boolean.cc#L50-L53
so probably the easiest is to do that for date32 and date64 as well.

But, testing this for many types (with from pyarrow.tests.test_types import get_many_types), it seems we have quite some types that don't implement a kernel to cast to themselves.
In addition to date64 and date32, this also gives an error (i.e. CanCast returns False) for null, halffloat, string_view/binary_view/list_view/large_list_view and run_end_encoded.
I suppose that for some of those we just don't yet have implemented any casts (like the new "view" types), but again a plain Cast will work for those types because we first check for type equality.


Update change-address-contact-and-more.md

Updated the character limit as a note

@JohanFreelancer9 Please copy review

issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/8916

@JohanFreelancer9

@cmcatee-MSFT Please merge this PR. Thanks

@cmcatee-MSFT Gentle reminder.

Thank you @cmcatee-MSFT  This has been updated.

@cmcatee-MSFT Please review and merge. Thanks

@msbemba Please implement the suggested change. Thanks

@cmcatee-MSFT I have updated the document as per your suggestion.

Learn Build status updates of commit f296a17:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/admin/manage/change-address-contact-and-more.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 83f3c80:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/admin/manage/change-address-contact-and-more.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


semantic-ui: namespace templates

Closes #165.

Don't the paths need to be updated in the config.py? Or would is it now working due to the fallback?
From what I recall in documentation at migration time:

Assets need to be semantic-ui prefixed (folder)
Templates don't (Maybe because of the fall back?)

When templates are not prefixed, you basically rely on having APP_THEME not being set at all. Since in invenio-app-rdm though we're setting APP_THEME = ['semantic-ui'] explicitly, I think it should be fine.
This also goes well with the practice of being always explicit, and allows extending specific templates in an overlay:
https://github.com/zenodo/zenodo-rdm/blob/e1b4b37a6deaac1a0d2746a3f628603834143c0c/templates/zenodo/invenio_app_rdm/frontpage.html#L1

It might be better to use the config vars, in case the user sets them it will actually make customization useful otherwise they'll need to overwrite the full template.

Agree and applied the suggestions.
There's a bigger topic here to discuss, since now there are three ways to override templates:

Use config variables in render_template/extends/include.

This requires creating a new template in your instance and then changing also the respective config variable. It has the benefit of being very explicit (i.e. no magic names)

Add your instance theme, like APP_THEME = ['zenodo', 'semantic-ui'], and then create the correct template filename

This requires a bit of "mental gymnastics" until you understand the templating lookup logic... But it can be very powerful in the sense that you can easily override any template you want with little effort, no matter if there are config variables defined in the original template to assist you.

In your instance's templates folder (the one that Flask/Jinja looks inside for the lookup), you can create a template with the same path/filename as the one you want to override
This is a bit "magic" overriding, since you rely on the fact that there is a special folder that gets taken into account before everything else

After discussing IRL, using config vars in the frontpage.html is not possible since THEME_FRONTPAGE_TEMPLATE = 'invenio_app_rdm/frontpage.html' and then there is a loop. This goes a long with the comment from @slint above. Maybe invenio-theme should have some sort of "BASE" templates.


Avoid recompilations in user-defined transformations of loop state

As one example validate forwards metrics to the outer loop state and thus after it runs the loop step needs to be recompiled. This is possible in other handlers as well if user's want to forward user-defined state for use later. This is something we should try to avoid if at all possible

Why does it need to be recompiled? If the shape of the metrics is the same, there is no recompilation. Or do you mean something like this happens:

Metrics starts empty on first loop
Metrics are filled at the end of the first loop with new values
Metrics starts second loop with new values

If that's the case, recompilation happens twice... which we may want to avoid?
Some options for fixes are:

Initialize metrics to bogus values
Or separate the metrics internal state from their reported state

This has been mostly resolved, there will be recompilations in certain cases, but I added a note to the docs about those!


[Bug] different results when conducting the attribute and ifthenelse statement

Hello, I’m studying TVM and its low-level operators recently. I tried to construct some test files but I was puzzled by the results of some of the files. When I built different statements, different execution results have occurred.
The reason that TVM throws the exception is that TVM checks the uint’s value and finds the uint expression is negative. But I still have two questions:

Why using different Attribute type key can lead to different execution results?（see Primfunc with body1 and body2）

Why using IfThenElse statement and using different statements as body can lead to different execution results?

The attribute statement with pragma_auto_unroll_max_step will execute normally (see PrimFunc with body4). But the LetStmt statement throws the exception(see PrimFunc with body6). I don't know what causes this difference in execution and whether TVM checks the branch’s expression if the branch is not executed. I have noticed that this problem maybe occurs in the specific version.
I’d be appreciated if anyone can help me.
Actual behavior
The PrimFunc with body1,body3 and body6 will throw the exception and the PrimFunc with body2,body4 and body5 will execute normally. The error messages are:

Environment
TVM version 0.10.dev0, git commit ee319d9d23c80091da9c4fb764b1e6d49d462714
Steps to reproduce
import tvm
from tvm import tir

var1=tir.Var('v1','int32')
var2=tir.Var('v2','uint32')
a=tir.IntImm('uint32',73829502)
b=tir.IntImm('uint32',1726448906)
uint=tir.Sub(a,b)
itevars=tir.IterVar([-16, 0], var1, 4, 'vthread.z')
body1=tir.AttrStmt(itevars,'pragma_auto_unroll_max_step',uint,tir.Evaluate(1.6))
body2=tir.AttrStmt(itevars,'thread_extent',uint,tir.Evaluate(1.6))
body3=tir.LetStmt(var2,uint,tir.Evaluate(2.1))
body4 = tir.IfThenElse(tir.LT(3,1),body1,tir.Evaluate(0))
body5=  tir.IfThenElse(tir.LT(3,1),body2,tir.Evaluate(0))
body6= tir.IfThenElse(tir.LT(3,1),body3,tir.Evaluate(0))
func1=tir.PrimFunc([],body=body1)
func2=tir.PrimFunc([],body=body2)
func3=tir.PrimFunc([],body=body3)
func4=tir.PrimFunc([],body=body4)
func5=tir.PrimFunc([],body=body5)
func6=tir.PrimFunc([],body=body6)
tvm.build(func4)

Triage

tir:op

Hi @blufissure. I cannot reproduce this crash in the latest TVM (i.e., 0.13.dev0). I guess this bug has been fixed.

Hi @blufissure. I cannot reproduce this crash in the latest TVM (i.e., 0.13.dev0). I guess this bug has been fixed.

In the latest version, TVM will not check the negative unsigned integer, so this problem will not occur in the latest version.

Here is another example
import tvm
from tvm import tir
a=tir.IntImm('uint32',73829502)
zero1=tir.IntImm('uint32',0)
div1=tir.Div(a,zero1)
v1=tir.Var('v1','int32')
itevars=tir.IterVar([-16, 0], v1, 4, 'vthread.z')
body1 = tir.AttrStmt(itevars,'rolling_buffer_scope',div1,tir.Evaluate(1.6))
body2 = tir.IfThenElse(tir.LT(3,1),body1,tir.Evaluate(0))
func=tir.PrimFunc([],body=body2)
tvm.build(func)


[FLINK-26004][runtime] Introduce ForwardForConsecutiveHashPartitioner

What is the purpose of the change
Introduce a new partitioner ForwardForConsecutiveHashPartitioner , see FLINK-26004 for details.
Verifying this change
UT ForwardForConsecutiveHashPartitioner Test
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

The CI is green except for the e2e test which is due to known environment issue.
Merging.


Add additional compute optimized VMs.

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

Changes approved.
#sign-off

#sign-off


Feature/referential integrity

Added referential integrity to the entire database and changed all integer primary keys and foreign keys to bigInteger.
See #29

This PR excludes referential integrity for all the audit fields:  created_by, updated_by, deleted_by.
I feel that this PR is a big step in the right direction for the application and careful consideration should be made concerning the audit fields.  A true auditing solution may be a better approach than having these audit fields exist in so many tables.  A good example of their unnecessity is vh_role_permissions because the auditing data just isn't necessary for this administrative data, IMHO.
It is my hope that, having provided a .skipper file, that you will take the time to learn how to create the foreign key relationships for the auditing fields if that is what you decide to do.
Your new ERD can be viewed here:  https://github.com/webreinvent/vaahcms/blob/8e5af6b184118aafa6c69531376debf4d990f9a1/erd.pdf

Per https://github.com/webreinvent/vaahcms/issues/29#issuecomment-890315033 this migrate and migrate:reset commands both work as expected.

@TomHAnderson will review it soon. Thanks

@TomHAnderson vaahcms.skipper please remove this file, I don't think that's required.

An explanation of how I created this migration file is in order.
I used the ORM and Laravel-specific tool Skipper to reverse engineer your database into an ERD.  I then added all the relationships between the tables and arranged them into an organized Entity Relationship Diagram.
I then exported the diagram into a PDF and I exported the diagram into a Laravel migration.  All of the code and comments in the migration were generated by Skipper.
You may be thinking that I'm trying to tie your project to this incredible tool.  Nothing I've done has created such a connection.  However, I ask that you take the due diligence to explore what Skipper can bring to this database project.  Please start by reviewing what an ERD is by reading all of the preview lectures here:  https://erd.design/p/designing-entity-relationship-diagrams-in-laravel
Next, download a trial version of Skipper and open the vaahcms.skipper file included with this PR and explore what Skipper will do for your project.
It's your choice to reject Skipper and request the vaahcms.skipper file be removed from this PR but if that is done then the erd.pdf file must also be removed because it is just a snapshot of the actual ERD (which is vaahcms.skipper).
In closing, I'm asking you to consider using Skipper for your ORM (and database) management in Laravel.  It is a unique application and the only database tool I'm aware of which can export to migrations (and models).

@TomHAnderson Most of your suggestions are accepted and released with minor version at:
v1.5.0
We may release a patch for lower versions later.
Unfortunately, we have decided not to use skipper18 at the moment, instead, we choose https://www.dbml.org/home/ to create visual & sharable database documentation. Feel free to review:

Database Table Structure
Database Relationship

Thank you so much for your suggestion. Do let us know if you have more suggestions.
Feel free join us at: Slack and see roadmap at: Trello


SQL command not properly ended

When i try execute this two string at same time i get error.
COMMENT ON COLUMN XXX.TABLES.SELECT_SQL IS 'ххх';
COMMENT ON COLUMN XXX.TABLES.SELECT_SQL_FORMAT IS 'ххх';

DBCException: SQL Error [933] [42000]: ORA-00933: SQL command not properly ended
java.sql.SQLSyntaxErrorException: SQLSyntaxErrorException: ORA-00933: SQL command not properly ended

What do you mean "at the same time"? These are two separate queries, they should be executed separately. You can execute them with al+x or ctrl+alt+x - then dbeaver will parse them and execute one by one.

I select 2 lines and press ctrl-enter

alt+x work for me, i thought ctrl-enter must work too because lines separate by ;


Revert License to MIT

On the CurseForge page and in your README you have this project listed as being licensed under the MIT license. But your LICENSE file currently says it is CC0 1.0 Universal.
I think what happened is that you accidentally took the LICENSE file from the fabric example mod when porting to 1.17.
This pull request reverts that change

Closing because it was fixed by the maintainer himself.


Allow to send a list of solvers

Discussed in https://github.com/cvxpy/cvxpy/discussions/1522
This feature has been suggested independently by multiple users.

Originally posted by erelsgl November  2, 2021
In some problems, some solvers may be better than others, but they might also fail with a SolverError.  I suggest allow, as the "solver" parameter, to send a list of solvers rather than just one. The function will try them in the order they are given, and return the first solution for which no SolverError is raised.

I see one complication with this feature. How would we parse solver options? There aren't consistent names for things like the maximum number of iterations used by a solver or solver termination criteria.

completed with PR above :)


toPythonModule(toPythonApplication) could add(remove) python version prefixes in the store path

toPythonApplication seems to imply this:
https://github.com/NixOS/nixpkgs/blob/5f0bd4d70f18b3499005eb61a34acb98c9b05756/pkgs/top-level/python-packages.nix#L90-L93
But it doesn't work as it sets passthru.name:
nix-repl> (import <nixpkgs> {}).sqlmap
«derivation /nix/store/6h6gd086bz4z85knyamnlkh3p1qvrvxy-python3.7-sqlmap-1.4.5.drv»

nix-repl> (import <nixpkgs> {}).sqlmap.name
"sqlmap-1.4.5"

The benefit of the current approach is that it may save some rebuilds in some cases. In case of toPythonModule many packages may need to be rebuilt because of enablePython = true settings. And, presumably, builds that go into toPythonApplication are not very heavy ones.

Yes, adding it to passthru.name was on purpose to avoid the rebuild.
Was this a question, or do you think something should be changed?

Yes, adding it to passthru.name was on purpose to avoid the rebuild.

Then we at very least need to correct the comment: passthru.name does not change the store path.

Was this a question, or do you think something should be changed?

I'm not entirely sure if this needs to be changed, but it seems like it only avoids rebuilds in a minority of cases. Here is a crude way to check for this:
cat pkgs/top-level/python-packages.nix | awk -v ORS= '/=.*toPythonModule/ { print "(with import ./. {}; " $1 " == python3Packages." $1 ")\0"; }' | xargs -0 -n1 nix eval

The way I see it, we could commit either to having python version prefixes clearly visible in the store paths, or we could commit to reducing rebuilds. Right now we only do a little bit of the latter.


When using video colab gets killed

Hey @prajwalkr
Thanks for sharing the codebase and models for such a wonderful project! Much appreciated
I have successfully been able to replicate Use Case-2 (using image with lip sync) on both CPU & Google Colab.
However for Use Case 1 (using video with LipSync) Both my CPU and Google Colab quit the process since it consumes a lot of RAM. I have a short text about 75 Characters long. I have an input video of  30 seconds at 60 FPS. How do I end up using it here?

You are trying to load 1800 frames (maybe even HD?) at once. You might need to modify the code to load on the fly and generate or split ur audio and video into several chunks and process them independently.

@prajwalkr Yes they are HD 🤣
Could you please point me in the right direction with the piece of code? Like what and how to change etc?
That would be really helpful

The existing code loads all the video frames, performs face detection on all of them, and yields batches of faces and audio segments. Instead of changing the code I would suggest you to write another script of your own that splits your big audio and video file into one-two second chunks and syncs them and generates one-two second videos and combines them later again. You can do this with FFMPEG and OpenCV.

Got it thanks!
Also when trying the Use Case 1 is it possible to sync audio with the new video?

Yes, the use case #1 is to perform lip-syncing on a video to match any new audio.

Sorry for not being clear! I mean once the resultant video is generated does it also have the new audio file embedded in  it?

Yes. It does. It is saved as result_voice.avi. The one without an audio overlay is saved as result.avi.

Great, thanks!
So I reduced the frames to 30 and lower the resolution so the video file is now around ~5MB's and it seems to work. However, the output video is rotated by 90 degrees and so the lips are not aligned properly (my guess is it gets inputted as a rotated video). Any help on that?

I do not think the LipGAN code rotates the video. Please inspect the video in OpenCV separately and check.

Yes figured that out, seems to be a problem with mobile videos.
I tried to test it on a video I have - attached zip file. You can observe sometimes the video tearing
result.zip
The results you showcased were amazing!
I was wondering if there are some tips on what type of videos (resolutions, fps, category etc) are best with the model?

You can adjust the (lower) padding such that the face crop fully covers the face, including the chin area.
Refer this line for more info.

Hey @prajwalkr!
Is it possible to highlight the face crop area at each frame so that I can debug it more successfully?
Also does the face crop need to cover the full face or just the mouth area?

This line contains the coordinates of the face. If you want you can draw a rectangle using OpenCV.

@Rudrabha  @prajwalkr
Hey guys,
Thanks for the prompt replies - Much appreciated.
I have attached an example that seems to be way out of sync.
To provide more context -

I have used Google's TTS service

The audio file stats seem to be different from what you have trained like sampling_rate etc

Do you have some suggestions on how to fix this? Basically the speaker doesn't seem to pause at right moments (pauses are 1-2 seconds) and lipsync of words doesn't seem to be proper.

result.zip

@prajwalkr Thanks for the info regarding "in the wild" with pauses videos.
Just a small note regarding audio files, the audio_hparams.py doesn't match my intended audio file. Should I change the audio file to match the given parameters or just leave it as it is.

It does not matter, as the code resamples all audio files to the same sampling rate etc.


[Audit] Number input

[ ] make hover area of the carets bigger (32px x 20px)

[ ] focus state should not change icon color:

[ ] double check tokens:

[ ] make sure background color should be $ui-01. This should fix light mobile variant's button color:
Current:

Should be:

@shixiedesign it looks like we are still using the old type mixins for this component. Which type tokens does this component need to use?

focus state should not change icon color

@shixiedesign originally the focus state did not change icon color, and after further investigation it looks like a recent PR we accepted introduced this change https://github.com/IBM/carbon-components/pull/1781. I'm not sure how we are handling theming, but it looks like there is a color token conflict here

originally the focus state did not change icon color, and after further investigation it looks like a recent PR we accepted introduced this change #1781.

Just looked at this PR with Anna, she confirmed that she didn't catch that change so it's an error. We still wanna keep the icon Gray100 on Focus. We don't have it in the system anywhere else to change icon colors. @emyarod still checking on your other comments. Will get back to you shortly!

could you show me how this would look with the error state styles also applied?

Saw your draft PR preview link. It does look pretty uncomfortable. I'm bringing it back to design team to see if anyone has a work around. I'm thinking of increasing focus border on the carets to 2px. Will send spec when confirmed. Meanwhile I think what we have in draft PR is okay!

it looks like the button background color was set to $ui-01 and the input field background color was $field-02

Right I made a mistake with the token assignment! Sorry! Hopefully below image & table helps (pls note the changes are in teal color) :

---
button background
input field background

Default desktop
NA
$field-01

Light desktop
NA
$field-02

Default mobile variant
$ui-01
$field-01

Light mobile variant
$ui-02
$field-02

@shixiedesign I added those changes and set the PR as ready for review https://github.com/IBM/carbon-components/pull/1967


Allow changing dehydrated BASEDIR (to be placed in a volume for example) and other improvements

See commit messages and the updated README and examples for more information.
I have not tested any of the examples but I have tested the meta-entrypoint and dehydrated config in my own container that's close to the standalone example and it works.

Hey @jjakob, Thank you very much for taking the time to compose this PR!
At the moment I have too much on my plate to look into this. But I will, eventually. Sorry for the delay.

Fixes #2