Check if image already exists?

Not sure if this is possible but I'm trying to figure out the right way to check if a file has already been uploaded. I think would be a great feature to add.

Hi @cyb3rs3ntry so Cloudflare Images does not really offer this kind of features: you upload an image and you get back a unique ID. It's not like in S3 or the like where if you use the same file name it would override the image or throw back an error at you depending on the config.
This check should be handled in your code, before and after the upload. For instance: saving the original filename in a model/table so you can compare if the name of the file was already pushed and to what ID it corresponds. If it's about the image itself, maybe you can try to generate a hash from the image (Using this: https://docs.python.org/3/library/hashlib.html  maybe?) and same idea, save the hash in a model/table for later usage.
Let me know if it helps.

I think it's an option we can add! I'll make a PR

Hi @cyb3rs3ntry, sorry for the late reply I am currently on leave, are you planning to do a PR against this project ?
If so, my point was that this kind of check should be handled in your code (e.g.: not in the library) as it is pretty custom (what type of check, what type of storage, logic, and so on).
Let me know,

I am closing this issue for lack of activity - feel free to reopen it/answer if you need.


ctx.SpawnWorkflow() ignores HATCHET_CLIENT_NAMESPACE

Steps that spawn child workflows should prepend the namespace to the workflow name

Hey @mrprkr, thanks for the report -- this has been fixed in the latest Python and Typescript SDK versions and a fix is on its way for the Go SDK today.


Dont status update

ogger: pyIndego.indego_async_client
Source: custom_components/indego/init.py:511
First occurred: 0:48:23 (161 occurrences)
Last logged: 14:17:18
404, message='', url='https://api.indego.iot.bosch-si.com/api/v1/alms/90........./predictive/lastcutting: Failed GET to Indego, won't retry
HA version: 0.110.3 , indego HACS ver: 3.2

I have asked you for a debug log. You keep sending me fragments of the log, and soe cut out parts of the log.  PM me on Discord when you send me the whole file.


build test in macOS m2 chip structure

how to build the test?
I want to contributed to tvm , but I am using arm64 macOS m2 computer.
I follow the https://tvm.apache.org/docs/tutorial/install.html to install test.
while ./tests/scripts/task_cpp_unittest.sh this file, run failed
the error log is in there.
error.txt

follow the setup, set the llvm path, it may help


Fix SMT interaction in Booster

do not proceed if the ground truth is not SAT:

trigger retry on Unknown
fail hard on Unsat

post-process the validity check result to eliminate Unknowns where possible
remove duplicate log message

No substantial difference in KEVM performance:

Test
georgy-booster-smt-changes time
master-5fdd57a75 time
(georgy-booster-smt-changes/master-5fdd57a75) time

mcd/flopper-dent-guy-same-pass-rough-spec.k
826.04
861.3
0.9590618831998142

mcd/functional-spec.k
589.87
568.68
1.0372617289160864

mcd/vow-fess-fail-rough-spec.k
124.51
114.3
1.089326334208224

TOTAL
1540.4199999999998
1544.28
0.9975004532856735


Extension causes high cpu load

Issue Type: Performance
Extension Name: vscode-icons
Extension Version: 10.1.1
OS Version: Darwin x64 19.4.0
VSCode version: 1.44.2

:warning: Make sure to attach this file from your home-directory:
:warning:/Users/jack.yan/vscode-icons-team.vscode-icons-unresponsive.cpuprofile.txt
Find more details here: https://github.com/microsoft/vscode/wiki/Explain-extension-causes-high-cpu-load

vscode-icons-team.vscode-icons-unresponsive.cpuprofile.txt

Same here.


🥅 - Configure Frontend to display errors via templates

User Story
As developer I expect proper error pages to be returned for application errors and exceptions rather than default debug pages so that users receive specific error pages and detailed debug information is not openly displayed.
Value / Purpose
By providing proper error pages we provide a more intuitive experience to users and keep detailed debug information from being displayed.
Useful Contacts
Mitch Dawson Tom Webber
User Types
No response
Hypothesis
By receiving specific error pages, users can identify much more quickly potential issues in the way they are using the application rather than being bombarded with detailed debug information which has little relevance.
Proposal
No response
Additional Information
No response
Definition of Done
Example - [ ] Documentation has been written / updated

[ ] README has been updated
[ ] User docs have been updated
[ ] Another team member has reviewed
[ ] Tests are green

https://design-system.service.gov.uk/patterns/page-not-found-pages/
Gov uk have a page not found template with content
https://docs.djangoproject.com/en/5.0/ref/views/
Django allow very easy adding of error page templates


add OneBlink Connector

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: tikeyike sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Yet Another Protocol (YAP)

Implementation and Testing of the Yet Another Protocol (YAP) protocol.
Note:
The following files are shown as complete replacements; however these have not been modified. This is due a problem in the Line Encoding, which has since been resolved. Sorry for the inconvinience.

.github/workflows/main.yml
LICENSE
RadonAlcer.code-workspace
doc/doxygen/Doxyfile
lib/LineFollower/ReadyState.cpp
lib/Service/Logging.cpp
lib/Service/Logging.h
webots/LineFollower_track.wbt

General question: How do we differ between YAP and LOG information?

General question: How do we differ between YAP and LOG information?

In the simulation this has been solved by routing the print() methods to the console, and the write() method to the SocketServer; which may not be ideal in the architecture, but allows for nice debugging.  In the target this is not possible, as both print and write are routed to the Serial interface.
Ideally the LOG information would be sent on its own YAP Data channel, meaning that the Logging Module should be able to write to the YAP Server somehow (only the App knows the YAP Server Instance, so it is not an ideal solution). And maybe only the INFO level Logs, not the DEBUG logs.
Other solution is turning Logs completely off on the target, and any information to be transmitted must be available for the App to read and re-route.

In the target this is not possible, as both print and write are routed to the Serial interface.

Thats why I meant that it should be application depended. Have a look:

The definition of Serial will be part of the corresponding application and therefore you can decide there whats the goal.


Fix tsx support

The current tsx support is totally non-functional, with this change many things work. As far as I can tell it supports everything typescript supports.
It might not be the ideal future direction of the plugin, but at least some things work for now so maybe it's worth considering this as a pragmatic change for the time being?

@theHamsta Thanks so much for helping me out so quickly and merging this quickly. One thing I just realised, I should have updated the table in the README to add the same thumbs ups as typescript.

This should actually be done by a CI job in merge to master.. I'll try to figure out what's wrong there


compile for use in web? update docs around web usage?

Right now I can use this package in expo and get web support, but if I try and use it outside expo (storybook in my case) I ran into issues. I eventually found I need to add the following to Webpack to get things both compiled and working correctly:
  config.resolve.extensions = [
    '.web.ts',
    '.web.tsx',
    '.web.js',
    '.web.jsx',
    '.ts',
    '.tsx',
    '.js',
    '.json',
  ]

  config.module.rules.push({
    test: /\.jsx?$/,
    loader: 'babel-loader',

    include: [
      path.resolve(
        __dirname,
        '../node_modules/react-native-gesture-handler',
      ),
    ],
    options: {
      babelrc: false,
      plugins: ['react-native-web'],
      presets: ['module:metro-react-native-babel-preset'],
    },
  })

I wonder if there is a way to make the above not needed? if not, maybe the above needs to be added to a section in the docs around web usage?

web-packaging-for-existing-react-native-apps

Packaging web apps is subtly different to packaging React Native apps and is complicated by the need to tree-shake and code-split non-trivial apps.

@expo/webpack-config is intended only for running react-native-web apps while considering all of the complex inner-workings of the library.


Fix for Anki 23.10+

Also there is a bug about BeautifulSoup, it would be great if it can be fixed.
Debug info:
Anki 23.10.1 (fac9e0ee) Python 3.9.15 Qt 6.5.3 PyQt 6.5.3
Platform: macOS-14.1.1-arm64-arm-64bit
Flags: frz=True ao=True sv=3
Add-ons, last update check: 2023-11-23 19:40:08
Add-ons possibly involved: AnkiOCR
/Users/XXX/Library/Application Support/Anki2/addons21/450181164/api.py:45: MarkupResemblesLocatorWarning: The input looks more like a filename than markup. You may want to open this file and pass the filehandle into Beautiful Soup.
soup = BeautifulSoup(self.field_text, "html.parser")
/Users/XXX/Library/Application Support/Anki2/addons21/450181164/api.py:74: MarkupResemblesLocatorWarning: The input looks more like a filename than markup. You may want to open this file and pass the filehandle into Beautiful Soup.
soup = BeautifulSoup(self.field_text, "html.parser")

I think there are some modules missing too ( PyQt5). I have raised an Issue for it too #45.
is this repo still be being maintained?

Hi Rijul,
Currently its more or less un-maintained due to me no longer having the
time - I may get some time over the holiday period to fix the issues, as I
would like to get this working again! If you are interested in helping out
I'd be more than happy to give you maintainer rights on the repo, let me
know.
On Mon, 18 Dec 2023 at 04:47, Rijul Sharma @.***> wrote:

I think there are some modules missing too ( PyQt5). I have raised an
Issue for it too #45 https://github.com/cfculhane/AnkiOCR/issues/45.
is this repo still be being maintained?
—
Reply to this email directly, view it on GitHub
https://github.com/cfculhane/AnkiOCR/pull/44#issuecomment-1859234656,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJRYIU36O5NX4SBKIOCC4DTYJ4V27AVCNFSM6AAAAAA7X3KH2WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJZGIZTINRVGY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thank you for taking out some precious time out
I like python and anki
Was thinking of updating your repo
If you would approve the pull request,
I'll send one with the new code.


Support environment variables in sg_*.yml files to make them passwordless ITT-1677

closed in favor of https://github.com/floragunncom/search-guard/pull/676


NIFI-10082 Update GCP SDK

Summary
NiFi 10082
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Nifi-10082 issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Built accordingly and it worked. Accidentally cancelled workflow after it completed in my branch.
Build

[x] Build completed using mvn clean install -P contrib-check

[x] JDK 8
[x] JDK 11
[X] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

@exceptionfactory apologies to have this run a 2nd time. Wasnt sure why this failed the 1st time when it passed on my fork. The only change that was made in the fork was this change.

@UcanInfosec, here is one site that describes how to squash multiple commits using Git:
https://gitbetter.substack.com/p/how-to-squash-git-commits
Creating a new branch from your current fork also works.


Attempt to index a nil value. snip_expand

I don't know, how to reproduce it. It occurs periodically, when I use rust and kotlin LSP. Maybe, it was the same with other languages. When this happens, cmp stops working. My config: https://github.com/WinstonMDP/dotfiles/blob/main/.config/nvim/init.vim

I think this may be the same issue as #1084, could you update and check if it persists?

It seems like it works. Thanks


"lcc project initialize": validate project name

Project name should not contain any of these characters [ ] / \

fixed in license-generator


Release azure-cli-documentdb 0.1.4

Reviewers, please verify the following for this PR:

[ ] The PR title has format Release <component-name> <version>.
[ ] setup.py has been modified with the same version as in the PR title.
[ ] HISTORY.rst has been modified with appropriate release notes.
[ ] If releasing azure-cli or azure-cli-core, __version__ defined in __init__.py should also be modified to match.

Started releasing azure-cli-documentdb 0.1.4...

Release of 'azure-cli-documentdb' with version '0.1.4' successful. View at https://pypi.python.org/pypi/azure-cli-documentdb/0.1.4.

GitHub release created. https://github.com/Azure/azure-cli/releases.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3080   +/-   ##
=======================================
  Coverage   70.72%   70.72%
=======================================
  Files         357      357
  Lines       22906    22906
  Branches     3526     3526
=======================================
  Hits        16201    16201
+ Misses       5610     5607    -3
- Partials     1095     1098    +3

Impacted Files
Coverage Δ

src/azure-cli-core/azure/cli/core/util.py
66.42% <0%> (ø)
:arrow_up:

...-cli-role/azure/cli/command_modules/role/custom.py
35.74% <0%> (ø)
:arrow_up:

...dback/azure/cli/command_modules/feedback/custom.py
34.69% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Improved test coverage for executor service mapper classes

+1 from my end, especially since I do work on some of the same classes I'd like to merge before sending PR :)


Minor nav bar tweaks

Replaced general-badge w/ F22 badge
Removed badge drop shadow
Updated logo wording to include "at"
Removed Node Buds from nav temporarily due to unknown status

Don't want to update for F22 just yet actually. Gonna redo this for simple tweaks with current stuff


Feature: data model refactor

Feature Details

Example Code

Possible Implementation

Additional Comments

Closing in favor of more specific issues


Enable lazy loading to improve website performance

Is your feature request related to a problem? Please describe.
Lazy loading is a technique that can be used to improve the performance of a website by delaying the loading of images and other resources until they are needed. This can help to reduce the initial load time of a page, which can improve the user experience and make the website more responsive.
Describe the solution you'd like
The benefits of lazy loading include:
Reduced initial load time
Improved user experience
More responsive website
Increased SEO ranking
Describe alternatives you've considered
No response
Additional context
Assign this issue to me under GSSOC'23

go ahead! @shraddha761 🚀

@J0SAL I am going to implement lazy loading in home page

@shraddha761  Do you mean all the components, including Graphs? Charts and forms?

Sry for the delay.
You can see the graph after implementing lazy loading and it optimize the website a lot

@shraddha761 can you also show the score before the changes were made ?

@Abhijit25Mishra Yahh sure
before

@Abhijit25Mishra Please review this pr

@J0SAL i am not able to view the deployment on vercel. it is one of the checks on github. can you verify this data ? cause i dont think so there should be such a big improvement. having said that i might very well be wrong.

@Abhijit25Mishra is this link not working on your end, https://jarvis-g9xfg1fjb-j0sal.vercel.app/ ?

sorry my bad i was trying to view an older deplyment.

there is about 150ms improvement. @J0SAL

i dont know why but the first paint is always slower in her fork. I think that lazy loading is causing somethings to get slowed down the first time they are loaded.
so what i mean is a new user when they open the page for the first time, it will take about 1-3 sec more. but every other time they load that page it will be about 150ms better. @J0SAL

I feel the only thing it is lacking is the initial load, maybe we can turn it into a feature by providing a good, animated loading screen. (in a diff issue), what do you think @Abhijit25Mishra?

hmm sounds great, i have done this before in a project. (https://enlace-umber.vercel.app/) we actually had similar reasons, as the site was too heavy to load we gave it like 4 sec loading screen animation. used anime.js, you can find the code here -> https://github.com/iiitkottayam/Enlace/tree/main/Enlace-main

@shraddha761 have a look once. if you have any trouble let me know.
also i would suggest you to look into other options, like gsap(greensock) and 3js, these two are also great but will probably require a lot of effort. have a look into this if you want to.

@J0SAL if we plan to go ahead with this make sure to increse the level from 2 to 3.

@Abhijit25Mishra thank you, I was also thinking for the same but don't know how to implement I'll try my best and take help from you

@J0SAL Please merge this pr so that I can work on another one

@shraddha761 you can update the same code, instead of a new PR I'll make the current one into level3, sound good? @Abhijit25Mishra @shraddha761 ?


The examples do not work as described.

The documentation is out of date, wrong, or incomplete. Cannot make the simplest action work as described in this document. This is true of practically all the Unity and MRTK documentation. Go ahead as usual and close the ticket, not enough information is provided. I cannot explain more. Get a new computer, install all the software recommended. Follow the instructions, when you get a working example, send it to me. Don't check any box or make any adjustment that is not described in the documentation.

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

ID: 5336e04e-dfe1-2c66-c216-67432cc317b0
Version Independent ID: 2793cf1a-6a2b-bfb3-26a4-30fdb0d1a199
Content: MRTK 101 - How to use Mixed Reality Toolkit Unity for Basic Interactions (HoloLens 2, HoloLens, Windows Mixed Reality, Open VR) - Mixed Reality
Content Source: mixed-reality-docs/mrtk-101.md
Product: mixed-reality
Technology: mr-dev
GitHub Login: @cre8ivepark
Microsoft Alias: dongpark

Hi @laultman, we're currently going through a complete Unity + MRTK documentation overhaul. As to closing the tickets, I left the main one open so our team could keep a channel of communication open with you and closed the rest of them because they were duplicate or adjacent issues. We appreciate your patience as we update our documentation.


divide by 100 after multiplying by 110 to get a 10% increase

The https://github.com/LIT-Protocol/js-sdk/pull/567 PR seems to have a bug in it that multiplies the gas limit by 110 instead of increasing it by 10%.  This is fixed by adding a division by 100 after the multiplication, which results in a 10% increase
Edit: this PR also adds a fix to the epoch delay, to make it work with the tests.  It works fine on live networks, but in the tests, when we try to use epoch 2, the delay hasn't passed yet, so we try to use epoch 1.  but epoch 1 is the first epoch and has no peers in it.  so the signing process fails in the tests.  the nodes go "i don't know about that peer".  so i put in a fix to only apply the delay if epochNumber >= 3.

Great catch. Thanks!!!


Adaptive card fails to render with specific strings

Copied bug from: https://github.com/OfficeDev/TeamsFx/issues/9826
Details from original post:
Describe the bug
Sometimes Teams would show me an error "Error encountered while rendering this message" without any further detail.
To Reproduce
One of the text fields in my adaptive card is supposed to output the following string (actually, a SQL query):
var sql = "SELECT COUNT(DISTINCT VisitorId) AS UniqueVisitors FROM sitedata WHERE eventdate>='2023-08-01' AND eventdate<'2023-08-08' AND Site =='foo.bar.com' AND Url RLIKE '^http?s://foo.bar.com/([^/]*)/answers/?'"

To render the card I'm using this code:
var cardContent = new AdaptiveCardTemplate(cardTemplate).Expand(
			new
			{
				Query = sql
			}
		);

		var replyActivity = MessageFactory.Attachment
		(
			new Attachment
			{
				ContentType = "application/vnd.microsoft.card.adaptive",
				Content = JsonConvert.DeserializeObject(cardContent),
			}
		);

		await turnContext.SendActivityAsync(replyActivity);

The card's relevant portion is this:
{
      "type": "TextBlock",
      "text": "${query}",
      "wrap": true,
      "fontType": "Monospace",
      "color": "Dark",
      "horizontalAlignment": "Left",
      "separator": true
    }

Steps to reproduce the behavior:
Send the adaptive card and Teams will show that it fails to render the card. There is no exception.
Expected behavior
Card is rendered fine without error.
VS Code Extension Information (please complete the following information):
Teams web or client, both Mac and Windows
Additional context
I tried to figure out what exactly it is that breaks the rendering but the results are non-conclusive.
By trial and error I started removing bits from the SQL string and found that it will still fail with this reduced version of the original string:
"SELECT COUNT(DISTINCT VisitorId) AS UniqueVisitors FROM sitedata WHERE eventdate>='2023-08-01' AND"

However, if I change it further and only remove the last  AND, it renders just fine:
"SELECT COUNT(DISTINCT VisitorId) AS UniqueVisitors FROM sitedata WHERE eventdate>='2023-08-01'"

As a workaround I wrapped the assignment into WebUtility.HtmlEncode(sql)and that fixes the issue. This will however break potential formatting such as Markdown.

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

ID: b743e43d-95cc-3d1e-b142-31a16369f97e
Version Independent ID: b743e43d-95cc-3d1e-b142-31a16369f97e
Content: Teams Toolkit Overview - Teams
Content Source: msteams-platform/toolkit/teams-toolkit-fundamentals.md
Service: msteams
GitHub Login: @zyxiaoyuer
Microsoft Alias: zhany

@Krumelur - Thanks for reporting your issue.
We will check this at our end and will get back to you.

@Krumelur - We are able to repro that issue, we have raised bug for the same. We will inform you once we get any update.
Microsoft Teams Version <IP_ADDRESS>65 (64-bit).


Add TextEncoder and TextDecoder to the module 'util' (Fixes #6326)

Please do not review for now, I noticed I broke other exports for 'util'.

OK, should be fixed

Looks like the CLA was never signed, feel free to reopen if this changes.


Complete documentation page

Complete documentation page according to the issue #8, noting that it is the desktop version. All settings for mobile version will later be made throughout the site.

Merged :) 👍
Thanks @jjaderg


adjustment to seq.py to allow for choice of output to prores

could we make an adjustment to the choice for output so that we can also output directly to pr res, without having to go through the specific folder path for seq2prores?
cheers

Sure, I can add that no problem. Any other codecs you'd like added, maybe h264?

yes, - that would be super thanks, - with watermark and bitc as default for security. if you can give a choice for not, - that would be great. but not an obvious choice, -= something hidden in the help command. which i can update for you if you wish
cheers
R

Cool, so the bitc/watermark would only be for h264? I'll start with clean prores as that's a quick fix. The bitc/watermark for h264 will need a little more thinking as I'd like to somehow harness the bitc.py microservice in order to cut down on the work.

maybe leave that so. the mezz is fine for the moment and we can always bitc from that. i'd be a bit concerned being able to go straight from tiff to h264 clean files for security reasons.
cheers

Prores and command line arguments are added here in this commit. Run update.py to get the latest version.
https://github.com/kieranjol/IFIscripts/commit/a8612998c7e8e1f118ae390e9df3fb53120136b1
The command line argument for prores is -p.
There are also some other changes in there too but that's the main one.

P.S I'm not closing this until we add bitc/h264 support


Fix link to CAS documentation

This is a fairly insignificant change, but Jasig has been renamed to Apereo, and the official CAS site has been moved to GitHub.
There is also https://www.apereo.org/projects/cas, but that doesn't give much info and the "Project Homepage" link on there points to the GitHub page in this PR.

:+1: Thanks


inotify-rs.sublime-project should .gitignored

Not everyone uses Sublime text!

I disagree. Not everyone uses Sublime Text, but I am. Having the file makes things easier for me.
Obviously we wouldn't want to have project files for 20 different IDEs, but I wouldn't mind adding additional project files if that makes things easier for regular contributors.


Fix packaging

Change packaging from com.cba to au.com.cba.

@JeremyMarshall @rowandavies is omnia-test still actively used? Approximately how many repos currently depend on it?


Duplicate image if "Done" button appear at same time

I have selected 4 different image to upload to server, but when done button displayed at the same time,image will duplicated, quantity for images are correct (total 4 images), but 2 of it is duplicate. Means only 3 different photos are uploaded, 1 photo is duplicate. Any handling for upload photo one by one to prevent this?

I think two files are having same file name, you need to rename at Server.
One is overwriting other
On Monday, January 18, 2016, howardhee<EMAIL_ADDRESS>wrote:

I have selected 4 different image to upload to server, but when done
button displayed at the same time,image will duplicated, quantity for
images are correct (total 4 images), but 2 of it is duplicate Means only 3
different photos are uploaded, 1 photo is duplicate Any handling for upload
photo one by one to prevent this?
[image: untitled1]
https://camo.githubusercontent.com/fae49893d31b9f2a37956ec8d64ccab9076ca81f/68747470733a2f2f636c6f756467697468756275736572636f6e74656e74636f6d2f6173736574732f343136363638382f31323338393533332f35626331626461632d626531342d313165352d383032342d353030316463653363333936706e67
—
Reply to this email directly or view it on GitHub
https://github.com/hayageek/jquery-upload-file/issues/92.

Ya, I checked in database, file name are same. But if you see the upload list from screenshot. The file name and size is different. Any solution for upload file one by one automatically?

If you rename the file, and returns the new Filename in the response , it
should work find
On Tuesday, January 19, 2016, howardhee<EMAIL_ADDRESS>wrote:

Ya, I checked in database, file name are same. But if you see the upload
list from screenshot. The file name and size is different. Any solution for
upload file one by one automatically?
—
Reply to this email directly or view it on GitHub
https://github.com/hayageek/jquery-upload-file/issues/92#issuecomment-172724074
.

I am using date("YmdHis") as file name, so if completed same time, file name will duplicated. I follow your tutorial from link http://hayageek.com/jquery-multiple-file-upload/ . I tried use  code below to loop the selected file, but it always get single file condition instead of multiple files condition even i selected multiple.
if(!is_array($_FILES["myfile"]['name'])) //single file
{
$fileName = $_FILES["myfile"]["name"];
move_uploaded_file($_FILES["myfile"]["tmp_name"],$output_dir. $_FILES["myfile"]["name"]);
$ret[$fileName]= $output_dir.$fileName;
}
else
{
$fileCount = count($_FILES["myfile"]['name']);
for($i=0; $i < $fileCount; $i++)
{
$fileName = $_FILES["myfile"]["name"][$i];
$ret[$fileName]= $output_dir.$fileName;
move_uploaded_file($_FILES["myfile"]["tmp_name"][$i],$output_dir.$fileName );
}
	}

issue is solved ?


BUG: FAQ section

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
I wanted to add a drop down which when clicked will show the detailed solution of that question. @arghadipmanna101 Kindly assign me this task under GSSOC'24.

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Can you assign me this issue @arghadipmanna101

@arghadipmanna101 Kindly add the gssoc and level label.


Readme should have contents

Describe the feature you'd like to have
The Readme file currently does not have any content.
It should have a description of the project and how to use it.
Who is the end user and what is the use case where this feature will be valuable?
developers and github users navigating to the project's page.
How will we know we have a good solution? (acceptance criteria)
Add a list of criteria that should be met for this feature to be useful
The Readme:

is not empty
describes what the project offers
describes how to use the operator.

Additional context

@Madhu-1 , @nb-ohad , @subhamkrai - FYI

@obnoxxx sure, we will add this soon with more structuring format

@Madhu-1  wrote:

@obnoxxx sure, we will add this soon with more structuring format

Thanks. but my main point was not lack of formatting, but lack of content: The Readme is empty.

@Madhu-1 wrote:

@obnoxxx sure, we will add this soon with more structuring format

Thanks. but my main point was not lack of formatting, but lack of content: The Readme is empty.

Yes both are missing for now, will let it sooner


[PARQUET-1004] Building fails on windows with PARQUET_ZLIB_VENDORED is turned on

Is this issue not addressed by the ZLIB_MSVC_STATIC_LIB_SUFFIX CMake option?

@wesm Thanks.  I didn't see that option. It  would be great if that is added to the docs

@MaxRis could you add a note to the README/build instructions about this?

@wesm , sure, here it's PR


Maintain a list of all Streams in the database.

Summary of changes
Currently we are only maintaining a list of unsubscribed/unsubscribed streams.
Hence, we need to get all streams in AsyncGetEvents.processRegister() and update the list in case of generate of a new stream.

[x] Code is formatted.

[x] Run the lint tests with ./gradlew lintDebug and make sure BUILD is SUCCESSFUL

[x] If new feature is implemeted then it is compatible with Night theme too.

[x] Commit messages are well-written.

Automated message from Dropbox CLA bot
@Sam1301, it looks like you've already signed the Dropbox CLA. Thanks!

Tried to unsubscribe a stream and got this error!
02-14 13:48:14.704 23537-23696/com.zulip.android.dev E/Error: oops
                                                              java.lang.ClassCastException: com.google.gson.internal.LinkedTreeMap cannot be cast to com.zulip.android.models.Stream
                                                                  at com.zulip.android.networking.AsyncGetEvents.processSubsciptions(AsyncGetEvents.java:445)
                                                                  at com.zulip.android.networking.AsyncGetEvents.processEvents(AsyncGetEvents.java:334)
                                                                  at com.zulip.android.networking.AsyncGetEvents.run(AsyncGetEvents.java:284)

@kunall17 updated the branch :)
changes: made SubscriptionWrapper generic

@kunall17 made the requested change.

Can face bug on clicking the @ mention
The message just has the @aaron

02-19 14:46:13.556 28621-28621/com.zulip.android.dev E/AndroidRuntime: FATAL EXCEPTION: main
                                                                       Process: com.zulip.android.dev, PID: 28621
                                                                       java.lang.NullPointerException: Attempt to read from field 'int com.zulip.android.models.Person.id' on a null object reference
                                                                           at com.zulip.android.models.Message.recipientList(Message.java:213)
                                                                           at com.zulip.android.filters.NarrowFilterPM.<init>(NarrowFilterPM.java:38)
                                                                           at com.zulip.android.util.ProfileSpan.onClick(ProfileSpan.java:42)
                                                                           at android.text.method.LinkMovementMethod.onTouchEvent(LinkMovementMethod.java:217)
                                                                           at android.widget.TextView.onTouchEvent(TextView.java:8328)
                                                                           at android.view.View.dispatchTouchEvent(View.java:9294)
                                                                           at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2553)
                                                                           at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2254)
                                                                           at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2553)
                                                                           at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2254)
                                                                           at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2553)
                                                                           at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2254)
                                                                           at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2553)
                                                                           at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2254)
                                                                           at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2553)
                                                                           at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2254)
                                                                           at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2553)
                                                                           at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2254)
                                                                           at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2553)
                                                                           at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2254)
                                                                           at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2553)
                                                                           at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2254)
                                                                           at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2553)
                                                                           at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2254)
                                                                           at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2553)
                                                                           at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2254)
                                                                           at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2553)
                                                                           at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2254)
                                                                           at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2553)
                                                                           at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2254)
                                                                           at com.android.internal.policy.PhoneWindow$DecorView.superDispatchTouchEvent(PhoneWindow.java:2403)
                                                                           at com.android.internal.policy.PhoneWindow.superDispatchTouchEvent(PhoneWindow.java:1737)
                                                                           at android.app.Activity.dispatchTouchEvent(Activity.java:2765)
                                                                           at android.support.v7.view.WindowCallbackWrapper.dispatchTouchEvent(WindowCallbackWrapper.java:63)
                                                                           at android.support.v7.view.WindowCallbackWrapper.dispatchTouchEvent(WindowCallbackWrapper.java:63)
                                                                           at com.android.internal.policy.PhoneWindow$DecorView.dispatchTouchEvent(PhoneWindow.java:2364)
                                                                           at android.view.View.dispatchPointerEvent(View.java:9514)
                                                                           at android.view.ViewRootImpl$ViewPostImeInputStage.processPointerEvent(ViewRootImpl.java:4230)
                                                                           at android.view.ViewRootImpl$ViewPostImeInputStage.onProcess(ViewRootImpl.java:4096)
                                                                           at android.view.ViewRootImpl$InputStage.deliver(ViewRootImpl.java:3642)

@kunall17 added a commit to fix the bug. I took significant time to reproduce it locally in the dev env as it didn't occur too often for me.

Unrelated to this branch  but we should change this https://github.com/zulip/zulip-android/blob/master/app/src/main/java/com/zulip/android/models/Person.java#L112 toLowerCase() here it might be a possibility someone email is saved as<EMAIL_ADDRESS>so it would create false results, as its creating for<EMAIL_ADDRESS>

Good catch! done :)

Gave it a last final check, editing works, new streams delete/add works, clicking on the @ mentions works!
Awesome work, thanks @Sam1301  👍


jetbrains.webstorm: 2019.1.2 -> 2019.1.3

Motivation for this change
updated with update.pl, have not tested the package
@c0bw3b can you take a look?
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

@GrahamcOfBorg build jetbrains.webstorm


echo cmd wrapped in quotes and added she bang;

echo cmd wrapped in quotes and added she bang;

Thanks for the pull request! It's a good idea to explicitly add she bang + can't hurt to wrap the commands in quotes. Merged.

Thanks! We can also add a logging for both flask + gunicorn!


improve grammar in articles: QPS rate limits, Clustering point data and Consumption model

Per prior arrangement, if this PR needs approval, or if the commits need to be pulled into the private repo, only @eriklindeman should be notified. Thanks.

Increased Acrolinx score from 84 to 100 in Azure Maps QPS rate limits.
Increased Acrolinx score from 88 to 98 in Clustering point data in the Web SDK.
Increased Acrolinx score from 89 to 100 in Consumption model.

#sign-off


[hotfix][docs] Translate building.md to building.zh.md

Signed-off-by: 313773543<EMAIL_ADDRESS>
What is the purpose of the change
This pull request just translate the building.md to Chinese. The original one had been tranlated few lines. I just finished the translation.
Brief change log
docs/flinkDev/building.zh.md
Verifying this change
Please see the final file.
Does this pull request potentially affect one of the following parts:
no
Documentation

Does this pull request introduce a new feature? ( no)

@flinkbot approve all

When will this pull request will be acceptted, or should I do some thing?

Please refer to https://github.com/sparanoid/chinese-copywriting-guidelines/blob/master/README.zh-CN.md and typesetting the content.

Ok, I will do typesetting quickly. Thank you.

@TisonKun I have do the typesetting according the materials you offered. Please check. Thank you.

I have done all the modification accordingly. Thank you very much.

@TisonKun @wuchong Please help to review again. Thank you.

Thanks for your translation @wbgentleman

Thanks for your translation @wbgentleman

@TisonKun Thank you very much. Learned lots of thing from you. Thank you again.


convert current write lock into read lock

flock should convert current exclusive lock into a shared lock.

https://man7.org/linux/man-pages/man2/flock.2.html

A process may hold only one type of lock (shared or exclusive) on
a file.  Subsequent flock() calls on an already locked file will
convert an existing lock to the new lock mode.


[Supplier] Supplier unbonding period governance param

Summary
This PR adds

Session number supplier_unbonding_period governance param as a follow-up to #666.
Enforces supplier_unbonding_period resutling blocks to be greater than the cumulated proof window close blocks.
Enforces global sharedParams validation when updating individual params (as some values depend on each other)

Example Supplier unstaking/unbonding timeline
Supplier unstaking (UnbondingPeriod = 3 sessioons, NumBlocksPerSession = 10, ProofWindowClose = 15 blocks)
timeline
    section Session 1
 [1,10]
        Blocks 1 - 10 : Supplier1 is active : Supplier2 is active
    section Session 2
 [11,20]
        Block 11 : Supplier1 unstakes
 Still active
        Block 18 : Supplier2 unstakes
 Still active
        Block 20: DO Deactivate Supplier1 : DO Deactivate Supplier2
    section Session 3
[21, 30]
        Block 21: Supplier1 IS NOT included in sessions : Supplier2 IS NOT included in sessions
    section Session 4
[31, 40]
        Block 35: ProofWindowClosed
    section Session 5
[41, 50]
        Block 50: DO UNBOND Supplier1 : DO UNBOND Supplier2

Issue

#489

Type of change
Select one or more:

[x] New feature, functionality or library
[ ] Bug fix
[ ] Code health or cleanup
[ ] Documentation
[ ] Other (specify)

Testing
Documentation changes (only if making doc changes)

[ ] make docusaurus_start; only needed if you make doc changes

Local Testing (only if making code changes)

[x] Unit Tests: make go_develop_and_test
[x] LocalNet E2E Tests: make test_e2e
See quickstart guide for instructions

PR Testing (only if making code changes)

[ ] DevNet E2E Tests: Add the devnet-test-e2e label to the PR.

THIS IS VERY EXPENSIVE, so only do it after all the reviews are complete.
Optionally run make trigger_ci if you want to re-trigger tests without any code changes
If tests fail, try re-running failed tests only using the GitHub UI as shown here

Sanity Checklist

[x] I have tested my changes using the available tooling
[x] I have commented my code
[x] I have performed a self-review of my own code; both comments & source code
[ ] I create and reference any new tickets, if applicable
[ ] I have left TODOs throughout the codebase, if applicable

Summary by CodeRabbit

New Features

Introduced a SupplierUnbondingPeriod parameter, defining the waiting duration for suppliers after unstaking before they can withdraw tokens.
Added a new SupplierUnbondingPeriodSessions parameter to enhance the unbonding process functionality.
Updated operational parameters for session management and claim windows.

Bug Fixes

Enhanced parameter validation logic to ensure correct handling of the new SupplierUnbondingPeriod and SupplierUnbondingPeriodSessions.

Tests

Improved existing test functions for clarity and modularity, enhancing code maintainability.
Renamed functions for better semantic clarity in the test suite.

@red-0ne Reminder to update the mileston on PRs & Issues please


Parse build-config in interactive mode too

Currently each editor mode has to implement it's own build-config parser, and that seems silly and error prone.

It would be easier to implement if we went with the following design:

We add a command line flag like so: --build_config "options: ...; variables:...; other-files: ... "; this argument is only allowed when the --in flag (for interactive mode) is also provided.
The compiler parses this string as it would the contents of a "(--build-config ... --)" block.
It's the job of each editor to detect if the file begins with a build config block; if so, it should pass the contents of that block among the arguments of the fstar process it starts.
If the user edits the build-config while in the interactive mode, the editor has to kill the fstar process and start a new one with the new build-config.

@aseemr @cpitclaudel What do you think?

This sounds good to me. However I wonder how much additional work would it be for the interactive mode to parse the build config directly (without the editor sending the --build_config argument).
-- The interactive mode maintains a flag. The flag is set if either (a) the interactive mode has already processed a build config, or (b) the interactive mode has type checked some code fragment. The flag is false to begin with.
-- If the flag is set, there's no need to look for a build config.
-- If the flag is not set, see if the string sent by the editor starts with a build config format, and if so, parse it (using the existing code in parseit.fs).
-- Batch mode tc the files mentioned in build config and update the envs. Set the flag.
-- If the flag is not set and the string sent by editor does not start with build config, set the flag, and proceed to type check the sent fragment.
I guess with fstar --in, one can specify files on the command line also. That's fine, and they can be processed as is done currently. What do you think ?

@nikswamy Your solution seems fine to me, although @aseemr's suggestion seems a bit more robust. Indeed, if we pass the build config as an argument, we may run into restrictions on the length of argument lists (8191 on Windows; that's large, but I'm sure some large project that auto-generates build configs will run into the issue at one point or other).
Editors will still have to recognize build config sections, as they'll want to restart entirely if the build config is changed; that seems fine by me.

I agree the @Aseemr's proposal is more robust. It requires more significant reworking of the top level fstar.fs, however. Aseem, do you want to implement it?
Sent from my Windows Phone

From: Clément<EMAIL_ADDRESS>Sent: 9/8/2015 17:31
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Subject: Re: [FStar] Parse build-config in interactive mode too (#346)
@nikswamy Your solution seems fine to me, although @aseemr's suggestion seems a bit more robust. Indeed, if we pass the build config as an argument, we may run into restrictions on the length of argument lists (8191 on Windows; that's large, but I'm sure some large project that auto-generates build configs will run into the issue at one point or other).
Editors will still have to recognize build config sections, as they'll want to restart entirely if the build config is changed; that seems fine by me.

Reply to this email directly or view it on GitHub:
https://github.com/FStarLang/FStar/issues/346#issuecomment-138739131

Pushed a commit for this. The interactive mode now parses the build config as I outlined above. Also pushed a change in fstar-interactive, removing code that used to parse build config (https://github.com/FStarLang/fstar-interactive/commit/f3151ed9b8dd2aa58e1d009a90824887c83d823a).

Awesome! Thanks @aseemr for implementing this so quickly. I'm running into one small issue though:
$ pwd
/build/fstar/examples/algorithms
$ ../../bin/fstar.exe --in
(*--build-config
  options:--z3timeout 20;
  variables:LIB=../../lib;
  other-files:$LIB/classical.fst $LIB/ext.fst $LIB/set.fsi $LIB/set.fst $LIB/heap.fst
              $LIB/stperm.fst $LIB/seq.fsi $LIB/seq.fst $LIB/seqproperties.fst $LIB/arr.fst
              qs_seq.fst qsort_arr.fst
--*)
#end
qsort_arr.fst(180,0-188,46): Refusing to check more than one module at a time incrementally
fail

Thanks for the nice implementation of this feature, Aseem.
The issue with parsing empty fragments should be fixed now.
Please let me know if glitches remain.
Closing it.

Thanks, looks great. One caveat is that F* will let you pop the build-config, but then all sort of funny things will happen. I've made the Emacs mode actively kill F* if the build config is retracted.


Upgrade cert-manager

Upgrade cert-manager to 0.11.  Also pointing documentation to specific version, in case "latest" changes before the manifest upgrades in the marketplace.

Thanks for this, will be live on the site very shortly!


Printer "pExpr_List" does not exist

You guys did a really cool tool!
But I got some errors when I try to convert Doctrine 1.2.5 on PHP 7
https://github.com/doctrine/doctrine1
Please help!
first error:
Could not convert file "source/Doctrine/Connection/UnitOfWork.php" cause : Printer "pExpr_List" does not exist /root/php-to-zephir/vendor/fezfez/php-to-zephir/src/PhpToZephir/Converter/Dispatcher.php 117
fallowing the seconed error:
Fatal error: Uncaught TypeError: Argument 1 passed to PhpParser\Node\Stmt\If_::construct() must be an instance of PhpParser\Node\Expr, null given, called in /root/php-to-zephir/vendor/fezfez/php-to-zephir/src/PhpToZephir/Converter/Printer/Stmt/SwitchPrinter.php on line 108 and defined in /root/php-to-zephir/vendor/nikic/php-parser/lib/PhpParser/Node/Stmt/If.php:28
Stack trace:
#0 /root/php-to-zephir/vendor/fezfez/php-to-zephir/src/PhpToZephir/Converter/Printer/Stmt/SwitchPrinter.php(108): PhpParser\Node\Stmt\If->construct(NULL, Array)
#1 /root/php-to-zephir/vendor/fezfez/php-to-zephir/src/PhpToZephir/Converter/Printer/Stmt/SwitchPrinter.php(36): PhpToZephir\Converter\Printer\Stmt\SwitchPrinter->convertSwitchToIfelse(Object(PhpParser\Node\Stmt\Switch))
#2 [internal function]: PhpToZephir\Converter\Printer\Stmt\SwitchPrinter->convert(Object(PhpParser\Node\Stmt\Switch))
#3 /root/php-to-zephir/vendor/fezfez/php-to-zephir/src/PhpToZephir/Converter/Dispatcher.php(92): call_user_func_array(Array, Array)
#4 /root/php-to-z in /root/php-to-zephir/vendor/nikic/php-parser/lib/PhpParser/Node/Stmt/If_.php on line 28

Hi !
Thanks for the report, i'll take that.

I've fixed the "Printer "pExpr_List" does not exist" in master but i cant reproduce the fatal. Can you identify the class who cause the fatal and paste the code here ?
Thanks !

As i understand, this is a "switch" who provoke the fatal

You can add  "--debug" to identify where the fatal appear

Thanks for your fast reply!
After I add  "--debug" and run again, I got these errors:
[PhpToZephir\Converter\Manipulator\ArrayManipulator::findComplexArrayDimFetch 40 Non supported funccall in array]  line "61"    class "Doctrine_Hydrator_ScalarDriver"

[pExpr_List]                          line "242"   class "Doctrine\Doctrine_Connection_UnitOfWork"
Could not convert file "source/Doctrine/Connection/UnitOfWork.php" cause : Printer "pExpr_List" does not exist /root/php-to-zephir/vendor/fezfez/php-to-zephir/src/PhpToZephir/Converter/Dispatcher.php 117

Fatal error: Uncaught TypeError: Argument 1 passed to PhpParser\Node\Stmt\If_::construct() must be an instance of PhpParser\Node\Expr, null given, called in /root/php-to-zephir/vendor/fezfez/php-to-zephir/src/PhpToZephir/Converter/Printer/Stmt/SwitchPrinter.php on line 108 and defined in /root/php-to-zephir/vendor/nikic/php-parser/lib/PhpParser/Node/Stmt/If.php:28
Stack trace:
#0 /root/php-to-zephir/vendor/fezfez/php-to-zephir/src/PhpToZephir/Converter/Printer/Stmt/SwitchPrinter.php(108): PhpParser\Node\Stmt\If->construct(NULL, Array)
#1 /root/php-to-zephir/vendor/fezfez/php-to-zephir/src/PhpToZephir/Converter/Printer/Stmt/SwitchPrinter.php(36): PhpToZephir\Converter\Printer\Stmt\SwitchPrinter->convertSwitchToIfelse(Object(PhpParser\Node\Stmt\Switch))
#2 [internal function]: PhpToZephir\Converter\Printer\Stmt\SwitchPrinter->convert(Object(PhpParser\Node\Stmt\Switch))
#3 /root/php-to-zephir/vendor/fezfez/php-to-zephir/src/PhpToZephir/Converter/Dispatcher.php(92): call_user_func_array(Array, Array)
#4 /root/php-to-z in /root/php-to-zephir/vendor/nikic/php-parser/lib/PhpParser/Node/Stmt/If_.php on line 28

Can you retry with dev-master ?

I retry with dev-maste, and pExpr_List error gone!
Thank you very much!
But I still got this error:
[PhpToZephir\Converter\Manipulator\ArrayManipulator::findComplexArrayDimFetch 40 Non supported funccall in array] line "61" class "Doctrine_Hydrator_ScalarDriver"
What should I do to bypass it?

I open a new issue on func call in array see #23

@fezfez I'm still getting this error. I don't see a Printer class for pExpr_List in the src/PhpToZephir/Converter/Printer directory of the project.
I've checked on master & 0.2.1. What is a pExpr_List referring to in PHP? I might be able to crank out the Printer class if I know.


Evaluate GMM at a point

This supports evaluating a GMM gmm at a point pt with the syntax
gmm(pt).

LGTM!


Warning: count(): Parameter must be an array or an object that implements Countable in C:\xampp\htdocs\google\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php on line 67

Hello,
I am trying to use the Google API client to make a dashboard with relevant information. However, when I follow the steps on the hello analytics tutorial, and try to connect, I get the following error:
Warning: count(): Parameter must be an array or an object that implements Countable in C:\xampp\htdocs\google\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php on line 67
How would I go about solving this?
Thanks.

This warning has been cropping up when an older version of Guzzle is used with PHP 7.2. If upgrading Guzzle doesn't work please include a full stack trace as well as version of PHP and Guzzle.

Thanks. How do you upgrade Guzzle? Because if you follow the steps specified by Guzzle itself, the installation fails because google/auth requires certain versions.

what is a full stack trace?
my php version is 7.2.2
when I update guzzle via "composer update guzzlehttp/guzzle" it says
"Updating dependencies (including require-dev)
Nothing to install or update
Generating autoload files"

So.. I am trying to update Guzzle to 6.0.3, but this doesn't work, since it says guzzlehttp/guzzle 6.3.0 requires guzzlehttp/psr7^1.4
but if I try to install psr7[1.4*] it won't let me, since it says google/auth v0.8 requires guzzlehttp/psr7 1.2.* but these conflict with your requirements or minimum-stability.
So is there any work-around?

One work around is to copy your composer.json to a new directory and run composer install and then to use that vendor directory to replace your old one. Another is to use a packaged release provided by us at https://github.com/google/google-api-php-client/releases

Thank you. This indeed solved the problems.

This solution didn't work form me...

The problem is in PHP 7.2 the parameter for count() can't be NULL. The warning gets displayed when $this->handles equals NULL. Just replace line 67 in CurlFactory.php with the following:
if (($this->handles ? count($this->handles) : 0) >= $this->maxHandles) {

thanks a lot! @vinnoangel

@jdpedrie Hi John, I see that you've made some related changes recently, can we re-open this please for tracking purposes? Asking folks to manually change this project's dependencies isn't really a solution :)

Hi @beverloo
Could you give me more information on what you'd like me to track? I'd be happy to reopen the issue if there's a need, but beyond potentially requiring guzzle 6.3+, I'm not sure if there's any action I can take.
The correct solution would be to upgrade to guzzle 6.3 or above. I'm not sure why anyone would be pinning to a version of guzzle 6 below that, but I think the only way this error would occur would be if for some reason you were stuck with an older version of guzzle for a reason unrelated to this library.
Perhaps we can update the minimum version for 6.x to 6.3.0. I'll need to discuss the implications of us doing that internally, but there's certainly nothing preventing you from requiring a higher version of guzzle within the 6.x series. The downloadable archives provide (for 2.4.0) guzzle 6.3.3.

Guzzle 6.3 is indeed the requirement for PHP 7.4 compatibility, so updating the minimum version certainly seems like the correct solution to me as well. I'm not familiar with the impact of that on the rest of the project to assess its feasibility.
I can manually install a different version, but my project depends on google/apiclient, which, in its default configuration, throws warnings. That seems undesirable.
It's a side project for me, but assuming you mean Google with internal discussions, feel free to loop me in if I could provide further data.

Thanks for the reply! 🙂
The default configuration would install the latest release in the 6.x tree. You’d need to explicitly require guzzle at a lower 6.x version somewhere else in the dependency tree to encounter the error. That’s why I’m unsure whether there’s any action needed on our end.
That said, we want to try and remove as many potential headaches as possible, given other constraints on time and backwards compatibility. So I will raise the possibility of bumping the minimum version of guzzle and see whether it’s doable in the immediate future.

The problem is in PHP 7.2 the parameter for count() can't be NULL. The warning gets displayed when $this->handles equals NULL. Just replace line 67 in CurlFactory.php with the following:
if (count( (array) $this->handles) >= $this->maxHandles) {

Thank you. solved the problems.

Hi @beverloo
Could you give me more information on what you'd like me to track? I'd be happy to reopen the issue if there's a need, but beyond potentially requiring guzzle 6.3+, I'm not sure if there's any action I can take.
The correct solution would be to upgrade to guzzle 6.3 or above. I'm not sure why anyone would be pinning to a version of guzzle 6 below that, but I think the only way this error would occur would be if for some reason you were stuck with an older version of guzzle for a reason unrelated to this library.
Perhaps we can update the minimum version for 6.x to 6.3.0. I'll need to discuss the implications of us doing that internally, but there's certainly nothing preventing you from requiring a higher version of guzzle within the 6.x series. The downloadable archives provide (for 2.4.0) guzzle 6.3.3.

Same problem with:

PHP 7.3
google/apiclient 2.7.0
guzzlehttp/guzzle 7.01 (automatically installed)

The quickstart sample codes don't set any dimensions, that's why.
Updated sample for https://developers.google.com/analytics/devguides/reporting/core/v4/quickstart/service-php:
<?php

// Load the Google API PHP Client Library.
require_once __DIR__ . '/vendor/autoload.php';

$analytics = initializeAnalytics();
$response = getReport($analytics);
printResults($response);

/**
 * Initializes an Analytics Reporting API V4 service object.
 *
 * @return An authorized Analytics Reporting API V4 service object.
 */
function initializeAnalytics()
{

  // Use the developers console and download your service account
  // credentials in JSON format. Place them in this directory or
  // change the key file location if necessary.
  $KEY_FILE_LOCATION = __DIR__ . '/service-account-credentials.json';

  // Create and configure a new client object.
  $client = new Google_Client();
  $client->setApplicationName("Hello Analytics Reporting");
  $client->setAuthConfig($KEY_FILE_LOCATION);
  $client->setScopes(['https://www.googleapis.com/auth/analytics.readonly']);
  $analytics = new Google_Service_AnalyticsReporting($client);

  return $analytics;
}

/**
 * Queries the Analytics Reporting API V4.
 *
 * @param service An authorized Analytics Reporting API V4 service object.
 * @return The Analytics Reporting API V4 response.
 */
function getReport($analytics) {

  // Replace with your view ID, for example XXXX.
  $VIEW_ID = "<REPLACE_WITH_VIEW_ID>";

  // Create the DateRange object.
  $dateRange = new Google_Service_AnalyticsReporting_DateRange();
  $dateRange->setStartDate("7daysAgo");
  $dateRange->setEndDate("today");

  // Create the Metrics object.
  $sessions = new Google_Service_AnalyticsReporting_Metric();
  $sessions->setExpression("ga:sessions");
  $sessions->setAlias("sessions");

  //Create the Dimensions object.
  $browser = new Google_Service_AnalyticsReporting_Dimension();
  $browser->setName("ga:browser");

  // Create the ReportRequest object.
  $request = new Google_Service_AnalyticsReporting_ReportRequest();
  $request->setViewId($VIEW_ID);
  $request->setDateRanges($dateRange);
  $request->setDimensions(array($browser));
  $request->setMetrics(array($sessions));

  $body = new Google_Service_AnalyticsReporting_GetReportsRequest();
  $body->setReportRequests( array( $request) );
  return $analytics->reports->batchGet( $body );
}

/**
 * Parses and prints the Analytics Reporting API V4 response.
 *
 * @param An Analytics Reporting API V4 response.
 */
function printResults($reports) {
  for ( $reportIndex = 0; $reportIndex < count( $reports ); $reportIndex++ ) {
    $report = $reports[ $reportIndex ];
    $header = $report->getColumnHeader();
    $dimensionHeaders = $header->getDimensions();
    $metricHeaders = $header->getMetricHeader()->getMetricHeaderEntries();
    $rows = $report->getData()->getRows();

    for ( $rowIndex = 0; $rowIndex < count($rows); $rowIndex++) {
      $row = $rows[ $rowIndex ];
      $dimensions = $row->getDimensions();
      $metrics = $row->getMetrics();
      for ($i = 0; $i < count($dimensionHeaders) && $i < count($dimensions); $i++) {
        print($dimensionHeaders[$i] . ": " . $dimensions[$i] . "\n");
      }

      for ($j = 0; $j < count($metrics); $j++) {
        $values = $metrics[$j]->getValues();
        for ($k = 0; $k < count($values); $k++) {
          $entry = $metricHeaders[$k];
          print($entry->getName() . ": " . $values[$k] . "\n");
        }
      }
    }
  }
}

Just include the following code first and then your PHP code.
error_reporting(0);
This will not show any errors (no error reportings in the page/content) in PHP. Even you can limit error reporting by changing the
number of error reporting function. The format is error_reporting(int);.

I got the same error:
PHP Fatal error: Uncaught TypeError: count(): Argument #1 ($value) must be of type Countable|array, null given in D:\xampp\htdocs\google-drive-api\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php:67 Stack trace: #0 D:\xampp\htdocs\google-drive-api\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php(67): count() #1 D:\xampp\htdocs\google-drive-api\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php(141): GuzzleHttp\Handler\CurlFactory->release() #2 D:\xampp\htdocs\google-drive-api\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php(103): GuzzleHttp\Handler\CurlFactory::finishError() #3 D:\xampp\htdocs\google-drive-api\vendor\guzzlehttp\guzzle\src\Handler\CurlHandler.php(43): GuzzleHttp\Handler\CurlFactory::finish() #4 D:\xampp\htdocs\google-drive-api\vendor\guzzlehttp\guzzle\src\Handler\Proxy.php(28): GuzzleHttp\Handler\CurlHandler->__invoke() #5 D:\xampp\htdocs\google-drive-api\vendor\guzzlehttp\guzzle\src\Handler\Proxy.php(51): GuzzleHttp\Handler\Proxy::GuzzleHttp\Handler{closure}() #6 D:\xampp\htdocs\google-drive-api\vendor\guzzlehttp\guzzle\src\PrepareBodyMiddleware.php(72): GuzzleHttp\Handler\Proxy::GuzzleHttp\Handler{closure}() #7 D:\xampp\htdocs\google-drive-api\vendor\guzzlehttp\guzzle\src\Middleware.php(30): GuzzleHttp\PrepareBodyMiddleware->__invoke() #8 D:\xampp\htdocs\google-drive-api\vendor\guzzlehttp\guzzle\src\RedirectMiddleware.php(68): GuzzleHttp\Middleware::GuzzleHttp{closure}() #9 D:\xampp\htdocs\google-drive-api\vendor\guzzlehttp\guzzle\src\Middleware.php(57): GuzzleHttp\RedirectMiddleware->__invoke() #10 D:\xampp\htdocs\google-drive-api\vendor\guzzlehttp\guzzle\src\HandlerStack.php(67): GuzzleHttp\Middleware::GuzzleHttp{closure}() #11 D:\xampp\htdocs\google-drive-api\vendor\guzzlehttp\guzzle\src\Client.php(268): GuzzleHttp\HandlerStack->__invoke() #12 D:\xampp\htdocs\google-drive-api\vendor\guzzlehttp\guzzle\src\Client.php(96): GuzzleHttp\Client->transfer() #13 D:\xampp\htdocs\google-drive-api\vendor\guzzlehttp\guzzle\src\Client.php(104): GuzzleHttp\Client->sendAsync() #14 D:\xampp\htdocs\google-drive-api\vendor\google\auth\src\HttpHandler\Guzzle6HttpHandler.php(34): GuzzleHttp\Client->send() #15 D:\xampp\htdocs\google-drive-api\vendor\google\auth\src\OAuth2.php(492): Google\Auth\HttpHandle#16 D:\xampp\htdocs\google-drive-api\vendor\google\apiclient\src\Google\Client.php(184): Google\Auth\OAuth2->fetchAuthToken() #17 D:\xampp\htdocs\google-drive-api\google-drive.php(52): Google_Client->fetchAccessTokenWithAuthCode() #18 D:\xampp\htdocs\google-drive-api\google-drive.php(70): getClient() #19 {main} thrown in D:\xampp\htdocs\google-drive-api\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php on line 67
I'm using php-8.0.9
Can someone help me?

SOLUTION:
Open the file indicated by the error and go to the line indicated:

\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php:67

(line 67 in CurlFactory.php)
Code before:

   if (count($this->handles) >= $this->maxHandles) {

Code after:

   if ($this->handles && count($this->handles) >= $this->maxHandles) {

Very easy.  Works now!

@Jamesbansal (and anyone else who finds this): do what is described up above (or just update your code):
https://github.com/googleapis/google-api-php-client/issues/1392#issuecomment-451431434

One work around is to copy your composer.json to a new directory and run composer install and then to use that vendor directory to replace your old one. Another is to use a packaged release provided by us at https://github.com/google/google-api-php-client/releases

Thanks man, it is working well.

The problem is in PHP 7.2 the parameter for count() can't be NULL. The warning gets displayed when $this->handles equals NULL. Just replace line 67 in CurlFactory.php with the following:
if (($this->handles ? count($this->handles) : 0) >= $this->maxHandles) {

this helped... thanks a lot


Integration with C# Dev Kit

Missing integration with c# Dev Kit.
It would be great to have the option to access the nugget from the solution explorer.

+1
Thanks for this beautiful extension. It will be very useful to integrate with Dev Kit

+1 Thanks for this extension.

Hello, I will try and look into this as I have some time now.

Missing integration with c# Dev Kit. It would be great to have the option to access the nugget from the solution explorer.

Can't seem to find any obvious way to integrate with DevKit.
If you have any suggestion or know of other extensions that do integrate this way please let me know.

Closing for now. If anyone has any idea if/how this integration would be possible please let me know.

@adrian-pavel now sure how helpfull will that be, but here's answer from ChatGPT
https://chatgpt.com/share/67282592-c0fc-800c-99eb-7bd7d844d151
That integration would be very in place
And thank's a lot for the extension


Circle around userpin gigantic on iOS

Hey,
Again great component and thanks for updating it this frequently! I have got a qeustion right now I have iOS code running and well but when I move to the user the circle (blue one) on iOS is gigantic. About 500+ radius around it. Is there any way I can reduce the size?
This is my move to user code:
   page.ShowMessage("Bezig met locatie ophalen");
              var locator = CrossGeolocator.Current;
              locator.DesiredAccuracy = 50;
              var location = await locator.GetPositionAsync(timeoutMilliseconds: 1000);
              _userPosition = new Position(location.Latitude, location.Longitude);
              map.MoveToRegion(MapSpan.FromCenterAndRadius(_userPosition, Distance.FromMeters(100)).WithZoom(1));

Thanks and cheers in advance!
Luuk

I think it could be related to GPS signal, could you try it with better signal / emulator?

Map.IsShowingUser is a Google Maps SDK for iOS's stuff.

myLocationEnabled | GMSMapView Class Reference

We could not control the blue circle.


[Bug]: Boost fees are not the same on Bitkit and on mempool

Describe the bug
I did a transaction https://mempool.space/tx/05b69d8af651208d14b682f75b77c4bc8e9a2a4b40f47adbd291340b6a07ff8a
and I boosted it https://mempool.space/tx/05b69d8af651208d14b682f75b77c4bc8e9a2a4b40f47adbd291340b6a07ff8a
The fees indicated on mempool are 1494sats but on Bitkit it's 5146sats
Reproduce

Go to '...'
Click on '....'
Scroll down to '....'
See error

Screenshots / Recording

Operating system
GrapheneOS
Bitkit version
118
Log output
No response

It is by design to show the sum of all fees for the boost activity, so I think it is ok. In this case it is the fee of these two transactions, 1494+3652=5146 :

https://mempool.space/tx/05b69d8af651208d14b682f75b77c4bc8e9a2a4b40f47adbd291340b6a07ff8a
https://mempool.space/tx/94290dc7baf3536ebc8d75d8509c2709f6486c04671f7baa9cca6215d03f82ac

It is by design to show the sum of all fees for the boost activity, so I think it is ok. In this case it is the fee of these two transactions, 1494+3652=5146 :

https://mempool.space/tx/05b69d8af651208d14b682f75b77c4bc8e9a2a4b40f47adbd291340b6a07ff8a
https://mempool.space/tx/94290dc7baf3536ebc8d75d8509c2709f6486c04671f7baa9cca6215d03f82ac

I think that assessment is correct. Current implementation looks good to me.


Suggestion:  "vumeter_tcp"

Hi,
I like to have the possibility to get VU meter values (in stereo) via tcp or mqtt.  Maybe a module "vumeter_tcp" that push out a JSON string on a 100 ms timer with the VU values.
Thanks!

Alfred, that would be great! Is it also possible to get levels for each audio channel, ex if the audio stream is in stereo?
Thanks!

Hi. This is something that would be very helpful for us in radio broadcast scenarios as well.  Relay via MQTT sound great, if we could set time intervall via Config, even better.
/ Ola Palm

So, to summarise:
You want to receive an event with the value of the average audio level (in -dB0V)
for the last 100 milliseconds (or configurable) for the following channels:

Recording direction
Playback direction
For stereo, a channel index (0=left, 1=right)

Hi,
Alfred, yes that's exactly what I like to see, In tcp and MQTT.
Thanks!

@alfredh , Ken2050 & Ola-Palm
I definitely agree, this sounds just perfect.
Recording/playback direction & stereo indication via MQTT, great! If we could set time interval via config, even better!! Anyone got a suggestion how to solve it?
Thanks!

Hi. I got a working example where the already calculated values from the module vumeter is pushed over mqtt as well as printed to screen.
It doesn’t show left/right values, only Tx/Rx.

hi @Ola-Palm
if you have some working code, would you like to submit a PR with your patch?
perhaps we can review it and test it.
Alfred

hi,
I got a nice patch from @Ola-Palm which I made some improvements to:
https://gist.github.com/alfredh/5ad83d7a3fe4cead3f7ac80a9e1435a9
could you please all test this patch and see if works as expected?
(must be applied on top of master HEAD)
run baresip like this:
./baresip -mvumeter.so -v

you should see the periodic VU events, something like this:
menu: [<EMAIL_ADDRESS>call=7837c7434ed834ee ] event: VU_TX_REPORT (-59.16)
menu: [<EMAIL_ADDRESS>call=7837c7434ed834ee ] event: VU_RX_REPORT (-42.64)
menu: [<EMAIL_ADDRESS>call=7837c7434ed834ee ] event: VU_TX_REPORT (-70.67)
menu: [<EMAIL_ADDRESS>call=7837c7434ed834ee ] event: VU_RX_REPORT (-40.94)
menu: [<EMAIL_ADDRESS>call=7837c7434ed834ee ] event: VU_TX_REPORT (-82.19)

the value is in -dB0V, where -127.0 is silence and 0.0 is max volume.

hi,
here is the PR:
https://github.com/alfredh/baresip/pull/450
@Ken2050 and @djhenley do you have some time to test this patch ?
Alfred

@alfredh  & @Ola-Palm. This is great, thanks! We've actually planned to test the patch here tomorrow. I'll get back with feedback here during the day.

thanks for testing. you should see both TX and RX values. are you getting
any audio packets? perhaps post the baresip output and enable rtp_stats config.
a PR is welcome, if you want to add support for configurable interval
in the vumeter module. please add the name of the module as prefix, e.g.:
vumeter_interval  100 # ms


Bring bg-atlasapi closer to the MRI world

We now have a number of atlases with MRI references, but in general there seems to be no overlap between Atlas API users (mostly using it for histology) and MRI researchers. It would be great to bring these together somehow.

Superceeded by https://github.com/brainglobe/bg-atlasgen/issues/85


MissingMetadataException reflection block hurts logging

MissingMetadataException and friends are reflection blocked, which means loggers like the CoreFX XUnit logger that print the exception type print strings like "$BlockedFromReflection_5_6f32c4e1 : This operation cannot be carried out because metadata for the following object was removed for performance reasons". It would be nice if those exceptions (and perhaps any others that users could log) had readable string names.

The type names for everything (including blocked types) are available when developer experience mode is turned on - maybe turning on developer experience should be the fix for this?

Here's a complete string from a developer experience log:
$BlockedFromReflection_3_19f5285d : This object cannot be invoked because it was metadata-enabled for browsing only: 'System.Data.SqlTypes.SqlCompareOptions.value__' For more information, please visit  http://go.microsoft.com/fwlink/?LinkID=616867]]

Oh, interesting. Pretty sure these used to be EETypeRva:0xCAFEBABE(MissingMetadataException): blah. Granted, not much better, but at least is had the exception type name in it.
Would you be okay if the exception type name is only present when developer experience is enabled, or are you asking for a separate channel?

Type.ToString() returns  EETypeRva:0xCAFEBABE(MissingMetadataException): blah.
Type.Name returns $BlockedFromReflection_3_19f5285d
Same with Type.FullName.
Given that this is a logger function, I wish it would have used Type.ToString()

@MichalStrehovsky I don't have a very strong opinion on developer experience for this, but we do enable browse metadata for all (non-reflection blocked) exceptions since these kind of loggers are very common.

So if I whitelist it, reflection will always be enabled on it? We want to make sure there's no possibility of triggering a tragic loop of missing metadata forcing logger to call Type.Name on MissingMetdataException which triggers missing metadata...

That policy isn't a guarantee -- we're free to change our minds. You can add rd.xml to guarantee it though.

Hmm, looks like that "policy" is implemented by rd.xml...
        <!-- Reflecting over exceptions is common in logging libraries. -->
        <Type Name="Exception">
          <Subtypes Browse="All" />
        </Type>

Right, but that's an implementation detail. If you add rd.xml that says something like this, the behavior is better guarded:
<!-- Do not remove to avoid breaking missing metadata logging -->
<Type Name="MissingMetadataException" Browse="Public" />

Changeset #1660982


feat(dotnet): add native AOT example

Continuing with my enthusiasm for traditionally jitted languages moving to native compilation for better integration with containers
This a new feature added in .NET 7: https://learn.microsoft.com/en-us/dotnet/core/deploying/native-aot/

Similarly to GraalVM (#17), symbolication does not currently work:

Debug info are not stripped by default on Linux: https://learn.microsoft.com/en-us/dotnet/core/deploying/native-aot/#native-debug-information
The feature is all new and the documentation around it appears light and I do not know my around the .NET documentation well enough. I am unable to confirm if the frame/stack pointer is preserved and if there is a build flag for it. DWARF unwinding does not work either.

The docs page does say

Limited diagnostic support (debugging and profiling).

To me this looks like there are just no debuginfos in this binary to symbolize. What does nm <binary> output?
At first sight I would say either framepointers are there or dwarf unwinding is working, as the memory addresses look roughly like they could be correct and have a stack depth higher than one. (not an explanation for Maxim or the rest of the maintainers, but anyone who may be reading who's unfamiliar: if the unwinding is broken in some way we typically see a very very high memory address and maximum 1-2 stack depth)

nm output: https://gist.github.com/maxbrunet/207417f6fe17f037e8c684e6876b16fe
For anyone who wants to reproduce:
make -C dotnet build-aot
docker create --name dotnet-aot-example parca-demo:dotnet-aot
docker cp dotnet-aot-example:/bin/demo .
docker rm dotnet-aot-example
nm demo


Chase the pizza edit

Fixes #1275.
Because there was no context for this block, it was placed into the default on start block. Added context so that the containing block is rendered correctly.

Thanks @alex-kulcsar!


[bug] Generic workflow fails on repositories that don't use Go

Describe the bug
If I understand correctly the generic workflow is meant to generate provenance for any language as long as the encoded hash digest of the built artifact is provided as input. However, when I try it on a repository that doesn't use Go (e.g., uses Python) it fails because it tries to download the Go module dependencies.
https://github.com/slsa-framework/slsa-github-generator/blob/668a33920550fa6983e6c850f69da89d28b057eb/.github/workflows/generator_generic_slsa3.yml#L71-L88
Why does the workflow need to download Go modules of the target repository? Please let me know if I'm missing something.
To Reproduce
Run the generic workflow on a repo that doen't use Go.
Screenshots

The code you linked is to build the builder, not the project.
Here's an e2e test that succeeds https://github.com/slsa-framework/example-package/blob/main/.github/workflows/e2e.generic-bazel.push.main.default.slsa3.yml#L27-L66
Maybe you're using a trigger we have not tested yet. Can you link to a workflow you use to call the generic workflow?

Here is the example workflow that fails.

The code you linked is to build the builder, not the project.

Note that my example repo is resolved here at the checkout step.

Hi again @behnazh-w! Thanks for the bug report and for the links. Those were super helpful for figuring out what happened.
This is caused by the fact that we don't support the pull_request trigger for Github Actions. See the docs for the generic workflow. Supporting it is kind of difficult for permissions related issues. Relevant GitHub issues are #358 and #131.
What happened in this case is that our detect-workflow action returned empty strings for the repository and ref which then got fed into the checkout action and it checked out your repository (since the repo on the github context is the default).
@laurentsimon We should probably do something to make this a bit more intuitive. Maybe error out early on pull_request triggers (though we still need to support e2e tests).
@behnazh-w Note also that you want to include the asset name in the digest you send to the workflow so you don't really want to use awk here. I believe the workflow will error out if the name isn't present.

Thanks. I had missed it in the docs that pull_request is not supported. It works fine on the push trigger :+1:

Thanks. I had missed it in the docs that pull_request is not supported. It works fine on the push trigger 👍

Great to hear. I'll go ahead and close this but feel free to reopen or create a new issue if you find something else!