auth 403

https://android.clients.google.com/auth

I'm also getting a 403 error when attempting to request an authentication token... the response body contains a captcha 😞
Error=CaptchaRequired CaptchaToken={TOKEN} CaptchaUrl=https://accounts.google.com/Captcha?ctoken={TOKEN}&v=2

Hi, can you try opening that link in the browser and solving the captcha? Maybe that will "unlock" your account.

The link leads me to the captcha image but it doesn't provide a submission form or anything for submitting the solution.
I'm trying to figure out how to submit the solution to see if that helps. https://developers.google.com/recaptcha/docs/verify looked promising but I'm getting this response when sending the token as secret and captcha as response.
{ "success": false, "error-codes": [ "invalid-input-response", "invalid-input-secret" ] }
response

I'm probably using the wrong endpoint or sending a bad request.. struggling to find anything much when searching for https://accounts.google.com/Captcha?ctoken

I have analyzed a similar issue in another project which uses the Android Accounts API (Android App Aurora Store). My results so far are that Google seems to decide whether the require a Captcha by analyzing TLS protocol details. More specifically, I was able to make it work by forcing specific ciphers and disabling ALPN.
I have not been successful with solving the Captcha and submitting it anywhere. My solution was to avoid triggering the Captcha.
I am detailing my results here: https://gitlab.com/AuroraOSS/AuroraStore/issues/217#note_249390026
See also NoMore201/googleplay-api#98

Hi @hoffie, thanks for reporting your findings about this issue.

This should be solved by bb5f59ab3be52f53ea0b15bf2b5ebf8de7b0925a.


Should resolve #332

Did you determine if the documentation needs updating to include installing fake global tools, or does paket magically handle that?


Fixing bug in bigmler whizzml when using --username and --api-key

@osroca is waitig for this version to come out. I add you here just to notify you when merged.

thanks @mmerce but no rush with it, as I said, I have a workaround.


Add star in archived repo results in an error

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

[X] Yes https://try.gitea.io/AragurDEV/Archived_Repo

Description
See my repo on try.gitea.io.
Try to star it. This results in the following error
template: repo/header:31:51: executing "repo/header" at <$.UnitTypeCode>: invalid value; expected models.UnitType

This same thing happens when you click on watch in that repo, which narrows it down to most likely being the code here:
https://github.com/go-gitea/gitea/blob/master/routers/repo/repo.go#L288-L316

The issue presented is partially caused by #5980
Now that the 404 page is loading a repo header if possible, this was failing because unit types aren't loaded for the action URL and the archived status is returning a 404 on actions.
I will send a PR to fix this, however I think there may be more work to be done after that.

In my opinion, it may be confusing to the end user why they get a 404 that displays the repo header, however they were unable to (un)watch/star the repo.
We could disable the buttons similarly to how the fork button is disabled if the user cannot fork, but...

Shouldn't a user be able to, at the very least, unstar/watch an archived repo?
Should all watchers and/or stargazers be removed when a repo is archived automatically? (I understand having a starred archived repo since you can still clone/download, but watching would be pretty moot)

In addition, should we display an error to the end user explaining that archived repos cannot be (un)watched/starred?

I think it should be possible to (un)star a archived repo. As of watching it's not that useful but I think enabling it would be better considering usibility

For comparison, GitHub allows you to still fully interact with starring/watching an archived repo.


Check path doesn't contain illegal characters

Adds a regex check before adding the project to make sure the path contains no spaces or special characters other than _ and - as a substitute for spaces.
fixes #875

We should convert this to a bool so True/False instead different types can be used

We use None for ifs all over MSS. I am not sure why it should be different here but okay.

ok, then we do that later when we want to refactor that, agreed.


Translation doesn't work when setting menues

Hi,
I created menues in  2 languages (fr, en) in external link.
The updated is ok, but when i launch frontend it doesn't work, or sometimes it's working for some menues but for all. No error, the translation doesn't work.
Using camaleon_cms 2.0.5 from git://github.com/owen2345/camaleon-cms.git (at master)
Thank you

It works well for me:

Video: http://screencast.com/t/LWWMt77m

Hello,
Thanks for all,
Question : How to change current language ?
2016-01-07 11:18 GMT+01:00 Owen Peredo Diaz<EMAIL_ADDRESS>
It works well for me:
[image: screen shot 2016-01-07 at 6 15 16 am]
https://cloud.githubusercontent.com/assets/1148415/12168070/6afea06c-b506-11e5-8d0a-5d44732357d8.png
Video: http://screencast.com/t/LWWMt77m
—
Reply to this email directly or view it on GitHub
https://github.com/owen2345/camaleon-cms/issues/271#issuecomment-169620081
.

--
Léo MANY

Hello,
I have a big problem for deploying on heroku from my local website
The problem happens when i use my own theme. I want to use my own theme.
How can I solved this?
2016-01-10 12:59 GMT+01:00 Léo Many<EMAIL_ADDRESS>
Hello,
Thanks for all,
Question : How to change current language ?
2016-01-07 11:18 GMT+01:00 Owen Peredo Diaz<EMAIL_ADDRESS>
It works well for me:
[image: screen shot 2016-01-07 at 6 15 16 am]
https://cloud.githubusercontent.com/assets/1148415/12168070/6afea06c-b506-11e5-8d0a-5d44732357d8.png
Video: http://screencast.com/t/LWWMt77m
—
Reply to this email directly or view it on GitHub
https://github.com/owen2345/camaleon-cms/issues/271#issuecomment-169620081
.

--
Léo MANY

--
Léo MANY

Hello,
Its OK, due to version of git which have securty case
/Users/many/.heroku/client/lib/heroku/jsplugin.rb:88: warning: Insecure
world writable dir /usr/local in PATH, mode 040777
Your version of git is 1.9.3. Which has serious security vulnerabilities.
More information here:
https://blog.heroku.com/archives/2014/12/23/update_your_git_clients_on_windows_and_os_x
https://blog.heroku.com/archives/2014/12/23/update_your_git_clients_on_windows_and_os_x
Thks
2016-01-11 2:19 GMT+01:00 Léo Many<EMAIL_ADDRESS>
Hello,
I have a big problem for deploying on heroku from my local website
The problem happens when i use my own theme. I want to use my own theme.
How can I solved this?
2016-01-10 12:59 GMT+01:00 Léo Many<EMAIL_ADDRESS>
Hello,
Thanks for all,
Question : How to change current language ?
2016-01-07 11:18 GMT+01:00 Owen Peredo Diaz<EMAIL_ADDRESS>
It works well for me:
[image: screen shot 2016-01-07 at 6 15 16 am]
https://cloud.githubusercontent.com/assets/1148415/12168070/6afea06c-b506-11e5-8d0a-5d44732357d8.png
Video: http://screencast.com/t/LWWMt77m
—
Reply to this email directly or view it on GitHub
https://github.com/owen2345/camaleon-cms/issues/271#issuecomment-169620081
.

--
Léo MANY

--
Léo MANY

--
Léo MANY

great!
please let me know if you have more problems in heroku.
regards!

Hello,
Please let me know how to use other template then post_type.html.erb
template to render a post_type.
For example for page, i can use a template template_something.html.erb
rather than page.html.erb
Thanks
2016-01-07 11:18 GMT+01:00 Owen Peredo Diaz<EMAIL_ADDRESS>
It works well for me:
[image: screen shot 2016-01-07 at 6 15 16 am]
https://cloud.githubusercontent.com/assets/1148415/12168070/6afea06c-b506-11e5-8d0a-5d44732357d8.png
Video: http://screencast.com/t/LWWMt77m
—
Reply to this email directly or view it on GitHub
https://github.com/owen2345/camaleon-cms/issues/271#issuecomment-169620081
.

--
Léo MANY

Hello,
It's OK for me.
Thanks
2016-02-12 12:03 GMT+01:00 Léo Many<EMAIL_ADDRESS>
Hello,
Please let me know how to use other template then post_type.html.erb
template to render a post_type.
For example for page, i can use a template template_something.html.erb
rather than page.html.erb
Thanks
2016-01-07 11:18 GMT+01:00 Owen Peredo Diaz<EMAIL_ADDRESS>
It works well for me:
[image: screen shot 2016-01-07 at 6 15 16 am]
https://cloud.githubusercontent.com/assets/1148415/12168070/6afea06c-b506-11e5-8d0a-5d44732357d8.png
Video: http://screencast.com/t/LWWMt77m
—
Reply to this email directly or view it on GitHub
https://github.com/owen2345/camaleon-cms/issues/271#issuecomment-169620081
.

--
Léo MANY

--
Léo MANY


Do not offer a default option when prompting for a user choice

I suggest not offering a default option when offering choices to the end user. This will avoid cases when typing too fast ends up with Enter is not being preceeded by the desired number. It also shortens the output and makes less stuff to read.
There are currently at least two cases where a default option is offered.
Which version of vagrant would you like to uninstall?
 1) None [Default - Press Enter]
 2) 1.6.5
 3) All versions

Also:
Do you want to run the script?
 NOTE: If you choose not to run the script, the installation will
 fail.
 Skip is an advanced option and most likely will never be wanted.

 1) yes
 2) no [Default - Press Enter]
 3) skip

For more details: https://groups.google.com/forum/#!topic/chocolatey/UVncL7PxXRg

Moved to choco repo


Update API wheel version

Move to 0.5.0 of the API wheel
Include missing objects from the documentation: ENS_GEOM, ENS_SCENE

@mariostieriansys This one will fail for now (until the api commit goes in).


Can't unfollow a table which got (soft) deleted

Affected module
UI
Describe the bug
I had a couple of tables which where ingested unpurposely because of a badly chosen table filter. So I adopted the filter accordingly and followed one of those tables I expected to get removed after the ingestion run to validate afterwards that it got removed. Today I was able to return to that table and verify the soft delete had happended. But when I try to unfollow the table now, I get the message "table instance for f2beb67e-5eec-420e-8925-131ada99d5e3 not found" which is easily understandable but leaves the user with an zombie entry now.
Expected behavior
As any UI user, I am able to unfollow also (soft) deleted entities, so they do not appear in my "Following" List anymore.
Version:

OS: Ubuntu 22.04
OpenMetadata version: 1.1

Additional context
I'll workaround this by deleting the entry in the database, so this is really a minor bug.

A very easy workaround for the user is to just DELETE the entity permanently by hitting "DELETE" in the tables menu in the UI. So I guess although this workflow is a little bit "unhappy" its perfectly resolveable, so I`ll close this again.


Update External Repo to Single Directory

With the addition of multiple visualizations in the morpheus-visualizations repo, we now can have multiple examples that need the submodule.
To fix this, the submodule has been moved into external/morpheus-visualizations and symlinks created back to the examples folder.

@gpucibot merge


arguments appear in both path and body

I meet some auto-generated code as the below.
body = {}
    body['tags'] = tags  # dictionary
    body.setdefault('sku', {})['name'] = sku_name  # str
    body.setdefault('sku', {})['tier'] = sku_tier  # str
    return client.create(resource_group_name=resource_group, workspace_collection_name=name, tags=tags, name=sku_name, tier=sku_tier, body=body)

BTW. It's important to generate python sdk and cli code at once. I met inconsistency between the generated CLI code and the python SDK.

this is fixed


Bugfix/library empty key (#159)

move variable to shared because it's used in both library and dashboard

use shared variable

test for when key is empty

remove check for game not being empty string

0.6.2

Pull Request Test Coverage Report for Build 696

6 of 6 (100.0%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 98.945%

Totals

Change from base Build 689:
0.04%

Covered Lines:
2098

Relevant Lines:
2109

💛  - Coveralls


AlexNet Implementation & understanding width_mult

Dear,
It is really a nice repo with clean structure and style, really appreciate the effort you guys put into it, yet I got a few questions would like to ask.

When comparing with original alexnet, I found there are slight differences in pySOT implementation, specifically regarding the conv stride, batchnorm instead of lrn-norm, no pooling in layer5 and using ReLU instead of P-ReLU. Is there any consideration for making those decisions?

Besides, could you please help figure out how the AlexNetLegacy different from AlexNet?

I actually have a hard time understanding the meaning of 'width_mult' in backbone network construction, could you please explain what it is for? It seems it is used for manipulating channel num but when and why should we do that? Is it a hyperparameter searching option? or it is done after a careful decision?

Thanks a lot.

The modified AlexNet is similar to SiamFC, which removes some strides(or pooling layer) to make it suitable for tracking (small stride to keep spatial resolution ). And we just simply follow them and removed group in conv2 and conv4. We believe BatchNorm is better than LRN-norm, however, we didn't do experiments on it. You can test it if you think it will be better.

width_mult is for doubling channel for AlexNet. It's used in SiamRPN (vot2018 entry). More parameters may leads to better performances.

Thank you very much for the info. It's really helpful, regards.

The modified AlexNet is similar to SiamFC, which removes some strides(or pooling layer) to make it suitable for tracking (small stride to keep spatial resolution ). And we just simply follow them and removed group in conv2 and conv4. We believe BatchNorm is better than LRN-norm, however, we didn't do experiments on it. You can test it if you think it will be better.
width_mult is for doubling channel for AlexNet. It's used in SiamRPN (vot2018 entry). More parameters may leads to better performances.

Hi. Can you tell me how to train the Alexnet backbone after these modifications on Imagenet? I need to retrain this backbone on my custom dataset so I can't use the provided alexnet-bn in this repo.
Thanks.


Bump perf-dash to 2.23

This is to include https://github.com/kubernetes/perf-tests/pull/1172

/aasign @ingvagabund

/lgtm

Kicked off reloading perfdash (it will take some time).

@mm4tt the perfdash seems to be down

I think it's still reloading :/

It's back up but it looks like it did not pick the latest changes. I can still see extention_duration_point prefixed metrics. The prefix is supposed to be gone.

I messed it up, reloading again, should be good now

All good now, thanks!!!


Adding physics not working from project explorer

Adding physics not working
When adding physics in the browser version of nunustudio, I can't get physics to get applied from the project explorer.
It works sometimes but 99% of the time adding physics like this doesn't work. http://sc.com.ly/show/b21871ec-5f95-4aad-aa61-75170b18e28a
Using Firefox here but same on chrome. Also the issue does not appear on the downloaded windows version.
Version

Platform

[ ] Web Version
[ ] Windows
[ ] Linux

Hello
Sorry for the late response.
I will try to replicate this problem to figure asolution.
Thanks


You can just hide the cursor with an escape code

https://github.com/SeungheonOh/PixelOnTerminal/blob/9ab64ca0affd8105dbb7162b99bf75b07fac70dd/main.go#L74
    fmt.Print("\033[?25l")

but be sure to show it again on shutdown
    fmt.Print("\033[?25h")

Implemented in #3


Figure out why the stable structure canisters are getting too big...

https://github.com/demergent-labs/kybra/pull/233


[PE-D][Tester D] No error message when data file is corrupted

Data is reset without any notification or warning to the user

Labels: type.FeatureFlaw severity.Medium
original: ginloy/ped#5

already addressed in UG (section 4.15 editing the data file)


[api-minor] Add support for LinkAnnotations with relative URLs, by making it possible to set the baseUrl for those in PDFLinkService (bug 766086)

Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=766086 (and replaces PR #6830).

Refactor LinkAnnotation slightly to add data.url/data.dest at the end
This should simplify reviewing: https://github.com/Snuffleupagus/pdf.js/commit/9a1410252b6597feff762b0a649dea2b1166d40c?w=1

Various improvements for GoToR actions

Add support for the NewWindow property.
Ensure that destinations are applied to the remote document, instead of the current one.
Handle the F entry being a standard string, instead of a dictionary.

Add support for LinkAnnotations with relative URLs, by making it possible to set the baseUrl for those in PDFLinkService (bug 766086)

Some PDF generators unfortunately create files with relative URLs, which thus prevents some inter-PDF links from working in PDF.js.
The reason that PDFLinkService.baseUrl wasn't re-used, is that this would only work in the Firefox addon/built-in version (and not in e.g. the Chrome addon).
Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=766086.
Please note: Given that this patch could be considered somewhat risky, from a security perspective, I'll understand if we ultimately end up rejecting it. But in that case, I think that we should probably close the referenced bug as WONTFIX.

@timvandermeij I wasn't entirely happy with the previous PR (#6830), and as you pointed out on IRC the way that the baseUrl was handled wasn't that great.
So, I've come up the current solution instead, where the API can return a relativeUrl which is then handled in the display layer instead. Does this approach seem better to you?

Yes, this approach looks better to me. I would like to know from @brendandahl or @yurydelendik if the third commit is acceptable from a security point of view. If that is the case, I can review this.


How to add support for additional lsp server.

Thanks for great work!
How to extends support for other language servers such as cmake, ltex.
I have added the configuration for ltex in navigator setup but it seems to be that code-actions and other things are not working.
Nvim lspconfig ltex
Thanks!

My Setup:
OS             Manjaro Linux
Kernel        5.14.18-1-MANJARO
Neovim     0.6

You can check this PR on how to add a new lsp:
https://github.com/ray-x/navigator.lua/pull/107
But it can be simply put the lsp name in the list here:
https://github.com/ray-x/navigator.lua/blob/9aec552ac16206f57cd25aa3b0dfaed74bb0105f/lua/navigator/lspclient/clients.lua#L275-L281
e.g. add cmake and ltex to end of the array.  When navigator startup, it will load default setup for you.
Also I just submit some code to allow you add servers. Please check readme
https://github.com/ray-x/navigator.lua#add-your-own-servers
Would be great you can run some tests and create a PR.  Thanks!

Thank you!

Would be great you can run some tests and create a PR. Thanks!

Sure, I will.


Set error file extension to .txt

Each experiment can define own file types for its compile errors, failed runs and successful runs.
The commit_reports still used .yaml for all of them, but now only uses .yaml for successful runs and .txt otherwise.

Fixed typing and stayed with the custom file ending one has to provide for call of a report generation.
This way other error reports can be used. If no extension is provided the default file extension provided in the report is used as file extension, regardless of the status of the run that created the report.


Refactor dedication routes

refactored dedication routes to be independent of hospital/donor. dedications are now accessed by just dedication_path instead of hospital_dedication_path or donor_dedication_path, etc.

LGTM


Move awslambda HttpResponseStream/streamifyResponse to the aws-sdk or its own package?

The fact that streamifyResponse and HttpResponseStream are only available embedded in the runtime makes it difficult to write simple, coherent unit tests for streaming lambda handlers. There's been at least one good attempt to create a wrapper to support easy testing, but it's not completely interface compatible (lacking, for example, the HttpResponseStream.from(underlyingStream, prelude) method) and it seems wasteful to rewrite/repackage code that's already written just to unbundle it from the rest of the NodeJS runtime.

Agreed. Having hard time getting any typings and proper tests. At the same time, it seems like it's still under development and we are experiencing changes in behaviour multiple times a week for a past month. Any more details, examples, etc. would be appreciated.


[MM-18007] macOS app reopens closed window on Cmd+Tab

Before submitting, please confirm you've

[x] read and understood our Contributing Guidelines
[x] completed Mattermost Contributor Agreement
[x] executed npm run lint:js for proper code formatting

Please provide the following information:
Summary

On macOS, closed window reopens on Cmd+Tab
Issue link

Fixes https://github.com/mattermost/desktop/issues/1068
Test Cases
Unfortunately, my test case is skipped until Chromedriver on Mac issue is addressed: https://bugs.chromium.org/p/chromedriver/issues/detail?id=3023#c2
Maybe I'm mistaken though, please advise if there's an alternative.
Possibly able to run this kind of test using TestCafe.
Additional Notes
For changelog message, do I need conventional commit format? Such as FEAT:, FIX:?

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

@deanwhillier Gentle reminder to review the PR.

Thanks @hanzei! There's actually a question on the issue for this PR that needs answering first. @esethna, @andrewbrown00, can you guys please take a look here when you have a minute: https://github.com/mattermost/desktop/issues/1068#issuecomment-559601915?

Thanks @hanzei! There's actually a question on the issue for this PR that needs answering first. @esethna, @andrewbrown00, can you guys please take a look here when you have a minute: #1068

Added a comment to https://github.com/mattermost/desktop/issues/1068

@andrewbrown00 can you help text this to make sure it matches the behavior you defined here? https://github.com/mattermost/desktop/issues/1068#issuecomment-564117499

@hunterlester or @deanwhillier where do I access the build to test this? I checked in Checks but I don't see the store_artifacts link to download the zips.

@andrewbrown00 it's in Artifacts tab: https://circleci.com/gh/mattermost/desktop/4405#artifacts/containers/0

Reassigning @adamjclarkson for PM review

FWIW - when trying to install this build on Mac OS Catalina 10.15.2 I kept getting error message "The Application "Mattermost" can't be opened" after trying to open the unzipped app.

Just an ok button and no other options. Weird.
Tried checking my security and permissions preferences, no luck. Tried chmod'ing file, no luck. Eventually I found a message board where someone figured out that the Mac Archive Utility caused the problem. They recommended Unarchive utility instead, which I downloaded and installed. And it worked.


scanner stays silently on in background

if my app calls scanner.startScanning() in componentDidMount without ever calling scanner.pauseScanning() or scanner.stopScanning() will the scanner stay on silently and still detect  barcodes even after the view switches to a different component?

As soon the scanner is unmounted, it will be destroyed.


Investigate which dcp 1 projects are re-export-able

The metadata documents for some of the dcp 1 datasets don't have the dcpVersion field set in the database which leads to errors when these datasets have to be re-exported.
The corresponding error log:
ingest-exporter-59cdf8bb59-b97lv :   2021-06-11 13:10:41,267  - exporter.terra.terra_listener - ERROR in terra_listener.py:88 _experiment_message_handler(): Failed to export experiment message with body: {"exportJobId":"60bdfbe520647123c5489704","documentId":"5d1e015e88fa640008b0010b","documentUuid":"8f4d6182-43c5-4dbc-9e71-2b99f1929c2d","callbackLink":"/processes/5d1e015e88fa640008b0010b","documentType":"process","envelopeId":"5d1e015c88fa640008b0009d","envelopeUuid":"7b4cd093-bfa5-477e-9c95-69bafc1cb6bf","projectId":"5d1e015d88fa640008b0009f","projectUuid":"f86f1ab4-1fbb-4510-ae35-3ffd752d4dfc","index":1,"total":19}
ingest-exporter-59cdf8bb59-b97lv :   2021-06-11 13:10:41,267  - exporter.terra.terra_listener - ERROR in terra_listener.py:89 _experiment_message_handler(): strptime() argument 1 must be str, not None
ingest-exporter-59cdf8bb59-b97lv :  Traceback (most recent call last):
ingest-exporter-59cdf8bb59-b97lv :    File "/app/exporter/metadata.py", line 59, in from_dict
ingest-exporter-59cdf8bb59-b97lv :      return MetadataResource(metadata_type, metadata_json, uuid, dcp_version, provenance, full_resource=data)
ingest-exporter-59cdf8bb59-b97lv :    File "/app/exporter/metadata.py", line 38, in __init__
ingest-exporter-59cdf8bb59-b97lv :      self.dcp_version = utils.to_dcp_version(dcp_version)
ingest-exporter-59cdf8bb59-b97lv :    File "/app/exporter/utils.py", line 11, in to_dcp_version
ingest-exporter-59cdf8bb59-b97lv :      date = parse_date_string(date_str)
ingest-exporter-59cdf8bb59-b97lv :    File "/app/exporter/utils.py", line 18, in parse_date_string
ingest-exporter-59cdf8bb59-b97lv :      return datetime.strptime(date_str, date_format)
ingest-exporter-59cdf8bb59-b97lv :  TypeError: strptime() argument 1 must be str, not None
ingest-exporter-59cdf8bb59-b97lv :
ingest-exporter-59cdf8bb59-b97lv :  During handling of the above exception, another exception occurred:
ingest-exporter-59cdf8bb59-b97lv :
ingest-exporter-59cdf8bb59-b97lv :  Traceback (most recent call last):
ingest-exporter-59cdf8bb59-b97lv :    File "/app/exporter/terra/terra_listener.py", line 76, in _experiment_message_handler
ingest-exporter-59cdf8bb59-b97lv :      self.terra_exporter.export(exp.process_uuid, exp.submission_uuid, exp.job_id)
ingest-exporter-59cdf8bb59-b97lv :    File "/app/exporter/terra/terra_exporter.py", line 28, in export
ingest-exporter-59cdf8bb59-b97lv :      process = self.get_process(process_uuid)
ingest-exporter-59cdf8bb59-b97lv :    File "/app/exporter/terra/terra_exporter.py", line 68, in get_process
ingest-exporter-59cdf8bb59-b97lv :      return MetadataResource.from_dict(self.ingest_client.get_entity_by_uuid('processes', process_uuid))
ingest-exporter-59cdf8bb59-b97lv :    File "/app/exporter/metadata.py", line 61, in from_dict
ingest-exporter-59cdf8bb59-b97lv :      raise MetadataParseException(e)
ingest-exporter-59cdf8bb59-b97lv :  exporter.metadata.MetadataParseException: strptime() argument 1 must be str, not None

i.  find out the dcp 1 datasets that don't have this field populated
ii. set this field for those datasets, in the biomaterial, process, file, and protocol collections in the db
for reference, these mongo queries can be used for updating the dcpVersion field
db.protocol.updateMany({"submissionEnvelope" : DBRef("submissionEnvelope", ObjectId("<submission envelope id>")), "project": DBRef("project", ObjectId("<project id>"))}, {$set: {"dcpVersion": new Date("<date>")}})

db.biomaterial.updateMany({"submissionEnvelope" : DBRef("submissionEnvelope", ObjectId("<submission envelope id>")), "project": DBRef("project", ObjectId("<project id>"))}, {$set: {"dcpVersion": new Date("<date>")}})

db.file.updateMany({"submissionEnvelope" : DBRef("submissionEnvelope", ObjectId("<submission envelope id>")), "project": DBRef("project", ObjectId("<project id>"))}, {$set: {"dcpVersion": new Date("<date>")}})

db.process.updateMany({"submissionEnvelope" : DBRef("submissionEnvelope", ObjectId("<submission envelope id>")), "project": DBRef("project", ObjectId("<project id>"))}, {$set: {"dcpVersion": new Date("<date>")}})

for reference, this is the list of dcp1 datasets: https://data.humancellatlas.org/explore/projects?catalog=dcp1&filter=[{"facetName":"biologicalSex","terms":["female"]}]

@yusra-haider is this still relevant?


Issues running stack overflow benchmarks

I'm having a couple issues trying to run the stack overflow benchmarks.

Timeouts. Most of the benchmarks seem to time out before synthesizing a consistent regex. I've tried making the timeout larger (up to a whole minute) but the issue persists. Is this expected behavior? If so, what timeout is appropriate to ensure the benchmarks will complete? Or is it possible to run regel without timeouts?

Java errors. In running exp.py there seem to be some benchmarks that fail to complete due to java errors. For example, benchmark 35 throws a "StringIndexOutOfBoundsException" and benchmark 70 throws a "NullPointerException". Any idea what causes this? I've attached my full terminal output below for reference.
terminal_output_2.txt

Running interactively. When running interactively it often appears that the top k synthesized regexes are all the same or include redundancies. This makes it difficult to input guiding examples. It seems equivalent synthesized regexes are not being eliminated?

Hello,

Are you running in Sketch Completion mode? Noted that for each benchmark we run the top-25 sketches and not all sketches are guaranteed to be able to synthesize a consistent regex. This means that there almost always exist some sketches in a benchmark that are going to timeout (in any "realistic" timeout). So, it is necessary to run Regel with timeouts. Among all the StackOverflow benchmarks, by setting the timeout to be 600 secs per benchmark, there are 11 benchmarks that all sketches times out. For some of the benchmarks, it requires more than a minute to find the first consistent regex but overall it should be finished within a couple of seconds.

I think I have just finished this issue. Let me know if there are any other benchmarks I didn't fix.

Yes, the equivalent synthesized regexes are not being eliminated. It's a nice feature to add and I will add in the future.

Thanks for the update. I reran the benchmarks in sketch completion mode and am still getting an error for benchmarks 35, 60, 16, 28, 17, 26, 110, 91, 30, 106, 55, 70. The update fixed benchmarks 93, 7, 98, 63, 113 (which were all throwing an "example type incorrect" exception). I've attached my terminal output for reference.
terminal_output_rerun.txt

Thanks for providing the outputs! I think all these benchmark reading errors are fixed now.

After the update I'm still getting an error for 28, 60, and 110. It looks like the ground truth for 28 has an extra "concat" and changing it to concat(<A>,concat(<B>,concat(or(<1>,or(<2>,or(<3>,or(<4>,or(<5>,<6>))))),concat(<_>,concat(<num>,concat(<_>,concat(optional(<num1-9>),concat(<.>,concat(<e>,concat(<x>,<e>)))))))))) seems to work. Benchmark 60 doesn't have a ground truth (right now it's just one of the example strings). I'm not sure what's wrong with 110. There seems to be a problem parsing the regex startwith(<{>).

I fixed the benchmark 28, 60. Benchmark 110 is caused by the current regex grammar not able to recognize the terminal "<{>" so I updated the grammar to support this symbol. Thanks!

Thanks. It looks like this fixes all the issues with the so benchmarks!


Avoid lock contention in output event rendering

This is an experiment to see if this strategy improves performance.
This change is not required to enable any functionality.
CI: https://builds.gradle.org/project.html?projectId=Gradle&tab=projectOverview&branch_Gradle=ldaley%2Fasync-output-event-renderer

Closed without merge, see https://github.com/gradle/gradle/pull/4661#issuecomment-372019343


Exception raised in Job APIConnectionError(Connection error.)

Hello, I use local data, including reference and response data, use Azure, and then use ragas to obtain accuracy indicators, but errors are reported:
Evaluating:   0%|          | 1/301 [00:04<21:46,  4.36s/it]Exception raised in Job[7]: APIConnectionError(Connection error.)
I checked the ragas and here and couldn't find an answer to my question.
Your Question
2 questions,
One is, I want to know what this error is and how to solve it. In the following source code, I have tried to set all the places where llm can be set to my configuration, but the problem is still reported, and there is no other prompt. (The code is changed from the example of the official website.)
The other is that if you want to get the value of Semantic Similarity, you should use answer_similarity or Semantic Similarity. I think both of them work, but I don't know which one is right and the difference between them.
Code Examples
`
os.environ['AZURE_OPENAI_API_KEY'] = 'xxxxxxx'.
os.environ['HTTP_PROXY'] = 'http://xxxxx'
os.environ['HTTPS_PROXY'] = 'http://xxxxx'
azure_config = {
"base_url": "https://yaotalai.openai.azure.com/",  # your endpoint
"model_deployment": "gpt-35-turbo-16k",  # your model deployment name
"model_name": "gpt35",  # your model name
"embedding_deployment": "text-embedding-ada-002",  # your embedding deployment name
"embedding_name": "embedding",  # your embedding name
}
evaluator_llm = LangchainLLMWrapper(AzureChatOpenAI(
openai_api_version="2023-05-15",
azure_endpoint=azure_config["base_url"],
azure_deployment=azure_config["model_deployment"],
model=azure_config["model_name"],
validate_base_url=False,
))
init the embeddings for answer_relevancy, answer_correctness and answer_similarity
evaluator_embeddings = LangchainEmbeddingsWrapper(AzureOpenAIEmbeddings(
openai_api_version="2023-05-15",
azure_endpoint=azure_config["base_url"],
azure_deployment=azure_config["embedding_deployment"],
model=azure_config["embedding_name"],
))
file_path = 'ts.xlsx'
data = pd.read_excel(file_path)
selected_columns = data[['reference', 'response']]
dataset = Dataset.from_pandas(selected_columns)
Define the metrics you want to evaluate
answer_similarity.llm = evaluator_llm
answer_similarity.embeddings = evaluator_embeddings
metrics = [
# answer_similarity,
FactualCorrectness(llm=evaluator_llm),
SemanticSimilarity(llm=evaluator_llm, embeddings=evaluator_embeddings),
]
Evaluate the dataset using the selected metrics
results = evaluate(
dataset = dataset,
llm = evaluator_llm,
embeddings = evaluator_embeddings,
metrics = metrics)
print(results)
`

Sorry, the above code format is incorrect, the correct source code is here.
os.environ['AZURE_OPENAI_API_KEY'] = 'xxxxxxx'.
os.environ['HTTP_PROXY'] = 'http://xxxxx/'
os.environ['HTTPS_PROXY'] = 'http://xxxxx/'

azure_config = {
"base_url": "https://yaotalai.openai.azure.com/", # your endpoint
"model_deployment": "gpt-35-turbo-16k", # your model deployment name
"model_name": "gpt35", # your model name
"embedding_deployment": "text-embedding-ada-002", # your embedding deployment name
"embedding_name": "embedding", # your embedding name
}

evaluator_llm = LangchainLLMWrapper(AzureChatOpenAI(
openai_api_version="2023-05-15",
azure_endpoint=azure_config["base_url"],
azure_deployment=azure_config["model_deployment"],
model=azure_config["model_name"],
validate_base_url=False,
))

evaluator_embeddings = LangchainEmbeddingsWrapper(AzureOpenAIEmbeddings(
openai_api_version="2023-05-15",
azure_endpoint=azure_config["base_url"],
azure_deployment=azure_config["embedding_deployment"],
model=azure_config["embedding_name"],
))

file_path = 'ts.xlsx'
data = pd.read_excel(file_path)
selected_columns = data[['reference', 'response']]
dataset = Dataset.from_pandas(selected_columns)

answer_similarity.llm = evaluator_llm
answer_similarity.embeddings = evaluator_embeddings
metrics = [
answer_similarity,
FactualCorrectness(llm=evaluator_llm),
SemanticSimilarity(llm=evaluator_llm, embeddings=evaluator_embeddings),
]

results = evaluate(
dataset = dataset,
llm = evaluator_llm,
embeddings = evaluator_embeddings,
metrics = metrics)
print(results)


[c++/python] Version conflict with pyarrow

The following code errors out in a JupyterHub notebook environment but not in my sandbox. The difference is the pyarrow version.
Code:
#!/usr/bin/env python

import tiledbsoma as soma, tiledbsoma.io

import scanpy as sc
import anndata as ad
import shutil, os

uri = "repro"
if os.path.exists(uri):
    shutil.rmtree(uri)

pbmc3k = sc.datasets.pbmc3k()

tiledbsoma.logging.info()
exp = soma.Experiment(uri)
soma.io.from_anndata(exp, anndata = pbmc3k,measurement_name='pbmc')

print(next(exp.obs.read()))

Output in the notebook environment:
Wrote   repro/obs
Wrote   repro/ms/pbmc/var
Writing repro/ms/pbmc/X/data
Wrote   repro/ms/pbmc/X/data
Wrote   repro
Traceback (most recent call last):
  File "/home/jovyan/chad.py", line 19, in <module>
    print(next(exp.obs.read()))
  File "/opt/conda/lib/python3.9/site-packages/tiledbsoma/util_iter.py", line 22, in __next__
    arrow_table = self.sr.read_next()
  File "pyarrow/array.pxi", line 1331, in pyarrow.lib.Array._import_from_c
TypeError: an integer is required

Notebook environment: pyarrow.__version__ is 6.0.2
Sandbox environment: pyarrow.__version__ is 9.0.0
Workaround for the notebook: pip install pyarrow==9.0.0

Superseded by #714


Fonts dont work properly on incognito mode

i dont know why

please someone help

hey man, i got chu

thx bro


Huggingface Hub

Hi, would you be interested in adding mindspore to Huggingface Hub (Hugging Face Hub)? The Hub offers free hosting, and it would make your work more accessible and visible to the rest of the ML community. I setup a example mindspore-ai organization where models, datasets, and spaces(web demos) can be added similar to the mindspore-ai org on github.
mindspore-ai: https://huggingface.co/mindspore-ai
Example from other organizations:
Keras: https://huggingface.co/keras-io
Microsoft: https://huggingface.co/microsoft
Facebook: https://huggingface.co/facebook

Here is how to integrate a library in the Hub: https://huggingface.co/docs/hub/adding-a-library
Integrating the Hub with your library provides many benefits, including:

Free model hosting for you and your users.
Built-in file versioning - even for huge files - made possible by Git-LFS.
All public models are powered by the Inference API.
In-browser widgets allow users to interact with your hosted models directly.

Please let us know if you would be interested and if you have any questions, we can also help with the technical implementation.

Hi, Thanks for your invitation, we would be very glad to add MindSpore to Huggingface Hub! I will read the doc you mentioned and try to integrate it in the Hub. Besides MindSpore, we also develop a high-level AI toolkit based on MindSpore, which we call it TinyMS: https://github.com/tinyms-ai. If you're interested in this project, we can also add it to Huggingface Hub.

@huxiaoman7 Great, can you please let me know your username of Huggingface so I can add you to the Mindspore-ai org or you can click the request to join this org button on the page https://huggingface.co/mindspore-ai. Then you can add models, dataset and spaces to the org in a similar way to github. Let me know if you have any questions

@AK391 Yes, Sure! My username is Charlotte77, and I also send the request, Thanks! :)

@huxiaoman7 thanks added, to start maybe we could add the quick start image classification example as a gradio space? similar to these keras examples https://huggingface.co/spaces/keras-io/vit-small-ds

@AK391 I've already create a pull request to add mindspore library to Huggingface Hub:https://github.com/huggingface/huggingface_hub/pull/684,  and waiting for approve. If it merge into main branch then you can add a quick start example.

@huxiaoman7 thank you, also adding guides here on adding spaces, models, and datasets to the mindspore-ai org on huggingface
How to add a Space: https://huggingface.co/blog/gradio-spaces
how to add models: https://huggingface.co/docs/hub/adding-a-model
uploading a dataset: https://huggingface.co/docs/datasets/upload_dataset.html
let me know if you have any questions

@huxiaoman7 great, I added the model for LeNet here https://huggingface.co/mindspore-ai/LeNet
for a good model card example see https://huggingface.co/keras-io/semantic-segmentation
regarding the PR, the only feedback currently is to add more models so users can click on the mindspore tag to link to the relevant models, thanks :)

@AK391 Thanks for adding models!! Today I'll add more models based on mindspore on huggingface:)

As I mentioned in the other PR, it would be great to have a couple more models before adding the official tag. Please also add a metadata tag by

Creating a README.md file
Adding some content such as this

---
tags:
- mindspore
---

# Content
...

You can read more about model cards in https://huggingface.co/docs/hub/model-repos#what-are-model-cards-and-why-are-they-useful
cc @julien-c

@osansevieroThanks for kindly reminding me :), I've add a metadata tag as your instructions, you can see it in this model:https://huggingface.co/mindspore-ai/LeNet

This is great @AK391! :rocket:

@huxiaoman7, one question remaining is how to load these pretrained models. Do you have a code snippet showing how to load a model locally? I can then modify it to use a model from the Hub :smile:

@huxiaoman7 also note i get this error when using pip install mindspore==1.6.0
ERROR: Could not find a version that satisfies the requirement mindspore==1.6.0 (from versions: 1.2.0, 1.2.1, 1.3.0, 1.5.1)
https://pypi.org/project/mindspore/1.6.0/#history

@huxiaoman7 mindspore Lenet is trending on the hub

🤗

https://mindspore.cn/tutorials/en/r1.8/beginner/save_load.html   You can use load_checkpoint and load_param_into_net to load ckpt to your model.  @osanseviero

pip install https://ms-release.obs.cn-north-4.myhuaweicloud.com/1.8.1/MindSpore/ascend/aarch64/mindspore_ascend-1.8.1-cp37-cp37m-linux_aarch64.whl --trusted-host ms-release.obs.cn-north-4.myhuaweicloud.com
@AK391


Question regarding Handlebars.Net use cases

Does Handlebars.Net only work in regards to HTML or can I use it as a general purpose template engine? Specifically, I want to use it for code generation. Is this possible?

Handlebars is a text template engine, with some consideration and optimization for HTML use cases. Some people have used it for code generation, and since code is text that is certainly possible. But I wouldn’t use it for code generation - I would look for a tool more purpose built for that.


httpclient/jackson/http - Cross build for Scala 2.13

Problem
httpclient, jackson, and http modules don't build for 2.13
Solution
Cross build these modules.
Notes on changes
The commit for the jackson module brings along some changes necessary for convert Scala 2.12 usages of Seq to Scala 2.13 compliant usages, as detailed here. There are performance considerations for these changes, as detailed in https://github.com/twitter/finagle/issues/818.
For now, the most important aspect here is ensuring that these changes don't regress 2.12 performance. For jackson, I've run the JsonBenchmark against 2.12.11 with and without the changes I've made here. There doesn't seem to be an impact in performance with the changes. The results are here: https://gist.github.com/chrisbenincasa/ff654689fb79ca6443e3b99900a3a824

The commit for http contains some of the Seq changes, but not on high volume code paths, from what I can tell.
I had to remove the mocking from RequestUtilsTest.scala, because Mockito was throwing errors when mocking HeaderMap due to some Map methods being made final in 2.13.
I also introduced 2.13+ and 2.13- directory support for the build in this commit.
Result
httpclient, jackson, and http modules are built for 2.13.

@chrisbenincasa is mocking still a concern with the update to Mockito 3.x and mockito-scala (via util-mock)?

@cacoco yeah that test is using the mixin you mentioned (and new versions of Mockito from what I can tell) but I don't think that changes anything w/r/t mocking HeaderMap here.
Here's what Mockito gives me:
[info]   Cause: org.mockito.exceptions.base.MockitoException: Mockito cannot mock this class: class com.twitter.finagle.http.HeaderMap.
[info]
[info] Mockito can only mock non-private & non-final classes.
[info] If you're not sure why you're getting this error, please report to the mailing list.
[info]

and further down:
[info]   Cause: java.lang.VerifyError: class com.twitter.finagle.http.HeaderMap$MockitoMock$686332027 overrides final method $plus$eq.(Ljava/lang/Object;)Lscala/collection/mutable/Growable;
[info]   at java.lang.ClassLoader.defineClass1(Native Method)

So I think this is solely related to 2.13 marking methods like += to be final.

@chrisbenincasa hmm, is the issue with the ExceptionMapperCollection: https://github.com/twitter/finatra/pull/554/files#diff-a4e23f17bf4ae13f9449d92df45f730e404fd770690af415421ce722ca75fdbd? We may be able to rewrite this. I'd like to push to not have to have customized 2.13 directories if we don't have to, not if we can rewrite the code in such a way to avoid the issue.

Sorry for the delay in response.
@cacoco I don't think the Mockito issue is related to ExceptionMapperCollection - where do you see that?
In terms of version-specific directories... I'm not sure there's a way around that if we wish to have ExceptionMapperCollection extend Scala's collection types. Scala 2.13 marks Traversable as deprecated and Iterable is at the top trait of the collection hierarchy. Perhaps we could change the existing implementation to extend Iterable instead, but it's unclear to me whether that's a breaking change or not.

Sorry for the delay in response.
@cacoco I don't think the Mockito issue is related to ExceptionMapperCollection - where do you see that?
In terms of version-specific directories... I'm not sure there's a way around that if we wish to have ExceptionMapperCollection extend Scala's collection types. Scala 2.13 marks Traversable as deprecated and Iterable is at the top trait of the collection hierarchy. Perhaps we could change the existing implementation to extend Iterable instead, but it's unclear to me whether that's a breaking change or not.

I think our preference would be to rewrite the ExceptionMapperCollection. I don't think it'd be a breaking change and we need to move to Iterable anyway here. I asked  about ExceptionMapperCollection because it was one file that got moved to the specific 2.13 directory.

For the Mockito issue:
 Mockito can only mock non-private & non-final classes.

I think that's different and not necessarily 2.13 related. Did you see that with JDK11?

@yufangong I've removed the forked directories in favor of just reimplementing ExceptionMapperCollection to conform to Iterable, instead of Traversable, per @cacoco's suggestion. This seems to work.
@cacoco I've tried with jdk8 and jdk11. It passes consistently with 2.12.11 and fails consistently on 2.13.1 for me. I get the same output on jdk11:
[info] - throw BadRequestException when missing host header for pathUrl *** FAILED ***
[info]   org.mockito.exceptions.base.MockitoException: Mockito cannot mock this class: class com.twitter.finagle.http.HeaderMap.
[info]
[info] Mockito can only mock non-private & non-final classes.
[info] If you're not sure why you're getting this error, please report to the mailing list.
[info]
[info]
[info] Java               : 11
[info] JVM vendor name    : Oracle Corporation
[info] JVM vendor version : 11.0.8+11
[info] JVM name           : OpenJDK 64-Bit Server VM
[info] JVM version        : 11.0.8+11
[info] JVM info           : mixed mode
[info] OS name            : Mac OS X
[info] OS version         : 10.15.7
[info]
[info]

with the cause:
[info]   Cause: java.lang.VerifyError: class com.twitter.finagle.http.HeaderMap$MockitoMock$1691613127 overrides final method com.twitter.finagle.http.HeaderMap.$plus$eq(Ljava/lang/Object;)Lscala/collection/mutable/Growable;

I believe this is distinctly Scala 2.13 related due to the += method changing to final from 2.12 -> 2.13. This is seen in Finagle as well, as they had to fork the HeaderMap into version specific interfaces in order to conform to that change: https://github.com/twitter/finagle/blob/develop/finagle-base-http/src/main/scala-2.12-/com/twitter/finagle/http/headers/HeaderMapVersionSpecific.scala#L8

@chrisbenincasa I see, ok. It is what it is then. Thanks for the work here.

@chrisbenincasa just a heads up, we are working on some canary and perf testing internally, the merge can be delayed a little bit. Thanks!

@yufangong Thanks for the heads up and no problem! There seems to be a newly failing test in jackson on 2.13 that I was looking into briefly as well. Seems to be related to the ScalaObjectMapper.
This is the failure, if anybody has any inklings:
[info] ScalaObjectMapperTest:
JSON DIFF FAILED!
Received:
 [
  "month: null",
  "weekday: null"
]

Expected:
 [
  "month: key not found: feb",
  "weekday: key not found: sat"
]

                   *
Received: ["month: null","weekday: null"]
Expected: ["month: key not found: feb","weekday: key not found: sat"]
[info] - Scala enums *** FAILED ***
[info]   Json diff failed:
[info]                      *
[info]   Received: ["month: null","weekday: null"]
[info]   Expected: ["month: key not found: feb","weekday: key not found: sat"] (Option.scala:242)

@yufangong Thanks for the heads up and no problem! There seems to be a newly failing test in jackson on 2.13 that I was looking into briefly as well. Seems to be related to the ScalaObjectMapper.
This is the failure, if anybody has any inklings:
[info] ScalaObjectMapperTest:
JSON DIFF FAILED!
Received:
 [
  "month: null",
  "weekday: null"
]

Expected:
 [
  "month: key not found: feb",
  "weekday: key not found: sat"
]

                   *
Received: ["month: null","weekday: null"]
Expected: ["month: key not found: feb","weekday: key not found: sat"]
[info] - Scala enums *** FAILED ***
[info]   Json diff failed:
[info]                      *
[info]   Received: ["month: null","weekday: null"]
[info]   Expected: ["month: key not found: feb","weekday: key not found: sat"] (Option.scala:242)

I believe this is because of the migrating of MapLike: https://www.scala-lang.org/api/2.12.8/scala/collection/mutable/MapLike.html#default(key:K):V

  /** Defines the default value computation for the map,
   *  returned when a key is not found
   *  The method implemented here throws an exception,
   *  but it might be overridden in subclasses.
   *
   *  @param key the given key value for which a binding is missing.
   *  @throws NoSuchElementException
   */
  def default(key: K): V =
    throw new NoSuchElementException("key not found: " + key)

The NoSuchElementException is caught in the case class deserializer without this message in 2.13, this should only have message mismatch, I will alter the message to make the test pass for both 2.12 and 2.13. ty for reporting!


_apns_prepare => TypeError: init() got multiple values for argument 'custom'

Version: 1.6.0
Traceback (most recent call last):
  File "/opt/project/src/api/views.py", line 203, in send_push_notification
    push.send_message(msg)  # ToDo: i18n
  File "/usr/lib/python3.6/site-packages/push_notifications/models.py", line 167, in send_message
    **kwargs
  File "/usr/lib/python3.6/site-packages/push_notifications/apns.py", line 113, in apns_send_message
    certfile=certfile, **kwargs
  File "/usr/lib/python3.6/site-packages/push_notifications/apns.py", line 90, in _apns_send
    data = _apns_prepare(registration_id, alert, **kwargs)
  File "/usr/lib/python3.6/site-packages/push_notifications/apns.py", line 60, in _apns_prepare
    url_args, custom=extra, thread_id=thread_id)
TypeError: __init__() got multiple values for argument 'custom'

Just upgrade to 1.6.1


FIX PS360 Windows

Resolve conflict: value "h0.4" was set for 2 keys

Do you own the device this map matches ? Curious if you can provide a better device name.


default/other models request?

where i can change the default model used in the gimp plug-in?

Unfortunately, it's not possible to select the used model. As this plugin is no longer actively developed, I close this issue.


Markdown field images are not added at the cursor position

When adding an image to a markdown field, it should be added where the text cursor is (for example, between two specific paragraphs). Right now, images are always added at the beginning of the content, before everything else.
This can be easily reproduced via the online Static CMS demo.

This will be fixed in the next release (targeted for later today).


Delete libguestfs cache on install/upgrade

Because libguestfs caches the kernel used to run it, subsequent invocations of transient with -copy-in-before or -copy-out-after will not use any newer kernel we try to provide. To avoid this, we should remove the libguestfs cache during installation of transient.
To avoid breaking any non-transient usage of libguestfs, we should also specify a cache dir for libguestfs under the transient XDG home. This way, we can be sure it is ok to remove the directory without affecting other applications.

This is no longer relevant, as libguestfs isn't used by transient.


Multi-target debugging bugs

vscode 1.53.2
Workspace:
.vscode/launch.json:
{
	"version": "0.2.0",
	"configurations": [
		{
			"name": "one",
			"program": "${workspaceFolder}/one.js",
			"request": "launch",
			"type": "pwa-node"
		},
		{
			"name": "two",
			"program": "${workspaceFolder}/two.js",
			"request": "launch",
			"type": "pwa-node"
		}
	],
	"compounds": [
		{
			"name": "three",
			"configurations": ["one", "two"]
		}
	]
}

one.js:
debugger;

two.js:
setInterval(() => {});

Repro:

Launch three (which is a compound of one and two)
Debugger hits breakpoint in one
Switch from one to two either using the debug console dropdown or by clicking it in the sidebar

note that the debug console dropdown immediately reverts back to 'one' (*1)

Switch back to one using debug toolbar dropdown

note the absence of breakpoint view (*2)

Type something in debug console

it doesn't evaluate (*2)

Terminate both sessions
Launch two
Switch to one using debug console dropdown
Try to switch back to two

(you can't) (*3)

Bugs:

Debug console dropdown displays an incorrect item inconsistent with what's actually being debugged/displayed in debug console.

Can be observed after step 3 in the repro.
Also randomly occurs in a bunch of other scenarios (e.g. when starting/terminating sessions).

Debug toolbar dropdown is broken (?).

Can be observed after steps 4-5 in the repro.
Breakpoint view isn't active after switching to a paused session.
Expressions don't evaluate in debug console.
Doesn't even trigger bug 1
Overall inconsistent behavior with the sidebar and debug console dropdown.

Switching to an inactive session causes debug console dropdown to break.

Repro is pretty self explanatory.

@a-ejs thanks for this bug with nice repro steps. I just tried with latest VS Code insiders and I could only reproduce:
"note that the debug console dropdown immediately reverts back to 'one' (*1)"
All the other points work fine - I pushed some fixes in this area so this might explain it.
I will not investigate into *1 and hopefully fix that too, once I push that I would ask you to try it out with VS Code insiders starting from Friday to let me know how it goes. Thanks!

@isidorn
I tried on Insiders yesterday and couldn't reproduce bug 2, so that must've been fixed.
After the update, bug 1 no longer occurs when switching between sessions manually, but I can still repro it when stopping sessions:

launch three
terminate one (view switches to two)
terminate two (dropdown now shows one but console still shows output of two)

Bug 3 still exists for me:

launch three
stop debugging
launch two alone
switch to output of one
can't switch back to two

PS. Tested on windows

My fix is not yet on Insiders, that is why I said starting from Friday. Today is Thursday (at least in my timezone)
So can you please try again tomorrow?

aye, my bad

It should be out now.

Seems to be fixed on the latest insiders.
However bug 3 is still there (with these steps:)

launch three
stop debugging
launch two alone
switch to output of one
can't switch back to two

Thanks. I pushed a fix.

Tried on latest insiders and could reproduce with slightly different steps:

Launch three
Terminate both sessions
Launch two
Launch one
Terminate two
Switch to two in debug console
Can't switch to one

@a-ejs thanks, i have pushed a fix for this.
Sorry for the multiple fixes. And thanks for trying this out


nEw proejct

Added new project.
Deployment link:https://ankitbhujeja.github.io/GuessTheNumber/

Make sure to Follow @shubham7668 and add star in this repository

Add in this repo

I didn't said anything about authenticity of your work
I said push your code correctly in this repository
See what i supposed to review?


fix: remove React generated code and rename generated React dirs

Description

The name generated for React build in src folder is different for wes and tes package.

IMO, the ideal behaviour would be that dir name be the same as that of dir name that contains to code not the name of the component for it to be cconsistent.

build script generates react directories inside src, we need to clean up this on every build run otherwise it conflicts with subsequent builds.
Possible solution(s)

Additional context

@SalihuDickson Please look into cleanup as well.

@JaeAeich, if you mean deleting the react dir before every build, that is already being done.

@SalihuDickson no that is not being done. But seems like @git-anurag-hub is working on it in #281

@JaeAeich, yes it is. But it seems the ts compiler is being run before that. That's probably what is causing issues. I'll take a look at why that is happening.

I guess we can also add to this, for the sake of tracking; Add clean up functions to the build process in the case of failure or user interruption  during the build process.


Change action logs dropdown filter to popup / add check log row button to transaction and charging stations

 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.Ben Romdhane seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Not linked to an issue!

Hi @mahdibenromdhane,
as always, if you have any questions regarding the feedback, come back to me :)

[x]  Table log action is missing in session in progress table

[ ]  Sorting of Actions in dialog does not work

In this case we only need the log shortcut button for the error list to debug.

Hi @mahdibenromdhane,
as always, if you have any questions regarding the feedback, come back to me :)

[ ]   Table log action is missing in session in progress table

[x]   Sorting of Actions in dialog does not work

In this case we only need the log shortcut button for the error list to debug.

@mahdibenromdhane
Ah okay. I thought it should be in every list, because it is in history list and in error list. But this is fine then.

Also Linked to This Issue

Seems that the merge with master-qa had issues with cars.
Could you check and make it work?
Thanks!

Done :)

Done :)

Thanks!


Can this lib play AES 128 encrypted M3U8 generated by FFMPEG  ?

Which encryption are you using?

Which encryption are you using?

ffmpeg default AES 128 bit

Do you need Authorization flow?
A sample stream will be cool to test

Do you need Authorization flow?
A sample stream will be cool to test
Are you targeting ios or android or both?

Yeah I'm targeting both platforms and what i need is to pass the decryption key which is hard coded in my program to the video element.

AES 128 works both for Android / iOS.
you can test with this stream:
https://www.radiantmediaplayer.com/media/rmp-segment/bbb-abr-aes/playlist.m3u8

chunk manifests come with encryption and key is provided by URL to relative path
#EXT-X-KEY:METHOD=AES-128,URI="file.key",IV=0x9256b016abc7ad3deaef1f093be826d3

chunk manifests come with encryption and key is provided by URL to relative path
#EXT-X-KEY:METHOD=AES-128,URI="file.key",IV=0x9256b016abc7ad3deaef1f093be826d3

does this URI scheme work ? #EXT-X-KEY:METHOD=AES-128,URI="data:text/plain;base64,XXXXXX",

Please, how can i test?

but using a local file doens't work.

the .m3u8 file and the key file has to be in the same directory.

Thanks, its works

@hnrqsss Hello, Can you help me with the issue... By providing a working code of how you did it...? Please do if you can, would be really helpfull.

I am able to play m3u8 with .ts files locally... But would like to mention that it can only be played when i used the exoplayer...Hope it helps...

how to play this?


Updated Gradebook Delete Command

Added necessary test cases
Added validation for delete command

Coverage increased (+1.5%) to 67.911% when pulling 123e51b0c54911707cb85d0c0768584be1a00ace on tristyxxnana:GradebookDeleteCommand into 8ad1d76a2252f3e561ee1b4ba3eb460646900fc0 on CS2113-AY1819S1-T16-1:master.


Self-Signed SSL Certificate

@oznu : This may be outside the scope of the config-ui-x plugin, but I would greatly appreciate a walkthrough for creating / binding a self-signed certificate to the port we configure for config-ui-x.
Creating the Self-Signed Certificate (link) isn't hard, nor is the binding statement (link); however, when I navigate to the port forwarded through my router, I'm unable to achieve a secure connection.
I'm sure this is a simple user education issue; however, after reading this article (link), I thought it would be really nice to be able to securely access my HomeBridge instance remotely.
If you've any tips, or even a rough walkthrough here, I'll commit to writing up a well formatted and fleshed out help document for others to follow.

Was doing more reading on SSL with nodeJS servers and it looks like I was WAY off (I’m mostly familiar with Windows/IIS).
Regardless, I’d still love this feature if it’s something you’d consider.

... and it would be perfect if you add support of lets encrypt for true https ...

Using a reverse proxy to achieve SSL is the method I recommend. Something like nginx is much more efficient when it comes to serving ssl traffic.
I have been thinking of writing up a wiki article explaining how to do this, but there are already many guides available on the internet explaining how to set up let's encrypt using certbot and nginx.
Nginx runs on Windows too!
If you're running this in the oznu/homebridge docker image then I would recommend using this docker image, which I use, to add let's encrypt ssl support: https://github.com/linuxserver/docker-letsencrypt.

Added example nginx vhost to wiki: https://github.com/oznu/homebridge-config-ui-x/wiki/Enabling-SSL-with-Nginx


Add a context menu to the playlist view

Should allow displaying song details dlg (#95), removing songs from library / from disk, "do not like", and possibly others.

Menu introduced in #126.

Play
Remove from library
Delete from disk
Rate (+ individual rating)
Properties


feat: Attribute to disable validation underline on an input with ngModel validation and custom css property

Feature Request
Ionic version:
[x] 5.x
Describe the Feature Request
In Ionic Angular, when you add a <ion-input> with the attribute [(ngModel)] it enables the validation underline. It causes the bottom border color of the input to change from primary (blue)  to success (green).
I would like an attribute on <ion-input> like disableValidation to get rid of the bottom border color changes and stays primary, I just want to use ngModel but the validation color doesn't make sense in my use case.
Maybe it would be nice to also have a custom css property to custom the color on a specified input (instead of changing the primary, success, etc color variables). So it would be possible to set color to transparent

Thanks for the issue. I am going to close this as this is already possible in Ionic Framework. In order to get this validation underline, your input needs to be inside of an ion-item. You can use the --highlight-height variable to disable the underline. You can also use the other highlight variables to change the border color.
Please see the docs for more information: https://ionicframework.com/docs/api/item#input-highlight


js-shopping-cart Graal VM integration on Docker

When build the js-shopping-cartdocker image (see https://github.com/cloudstateio/cloudstate#building-the-native-image) for the Graal VM integration there is the issue when running npm run dockerbuild:
/opt/node-support/proto: warning: directory does not exist.
I found out the issue and i will a PR soon for this. The issue is because the npm task prepare for the node-support is not executed when building the docker image.

The PR is here - https://github.com/cloudstateio/cloudstate/pull/198

This PR (https://github.com/cloudstateio/cloudstate/pull/198) is merged and this issue can be closed.


Private key storage

What is this PR doing?
How can these changes be manually tested?
Does this PR resolve or contribute to any issues?
Checklist

[ ] I have manually tested these changes
[ ] Post a link to the PR in the group chat

Guidelines

If your PR is not ready, mark it as a draft
The resolve conversation button is for reviewers, not authors

(But add a 'done' comment or similar)

Only shared as demo. Not needed.


Use posix-style flags by default

This converts all our non-POSIX-compliant flags to POSIX-compliant ones. The original forms are preserved as aliases, so all our scripts etc. should still work.
This also fixes a flag where the default argument was part of the name 🤦♂
Checklist:

[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them:

Can you also update the flags we use in helm?

Someone else will need to pick that up, I'm working on other things. This was just easy to automate.
The current flags will still work, so I propose doing it as separate PRs anyway.

Change -help and -version?
Also, another task similar to the infra task mentioned above would be to update flags in the tests ( command_line_test.ml, for example)

!approved-for-mainnet


fix: When the target has no mkdir but has unistd.h

Hi,
I was trying to compile it to a target that didn't have mkdir available but had unistd.h, so I checked for it and wrote an alternative using mkdtemp. The C++17 version changes the string in place.
I hope this alternative implementation might be useful for someone else.
Best,
Iuri

:closed_book: MEV SW OD Build #27839 result: FAILURE

Manifest Name
Build Status
Jenkins Link
Artifacts
Unsuccessful Stages
Description

mev-ts-trunk
FAILURE
59604
 Artifacts
IPU SDK Build

mev-trunk
FAILURE
56680
 Artifacts
Simics_C0_ACC

mev-release
NOT AFFECTED

mev-ts-release
NOT AFFECTED

mev-ts-release-1.6.1
NOT AFFECTED

mev-ts-release-bitbake
NOT AFFECTED

mev-ts-release-external
NOT AFFECTED

mev-ts-trunk-bitbake
NOT AFFECTED

mev-ts-trunk-bitbake-c1
NOT AFFECTED

mev-ts-trunk-external
NOT AFFECTED

mev-upstream-idpf
NOT AFFECTED

:closed_book: MEV SW OD Build #27894 result: FAILURE

Manifest Name
Build Status
Jenkins Link
Artifacts
Unsuccessful Stages
Description

mev-trunk
FAILURE
56702
 Artifacts
IPU_IMC_6_6_4

mev-ts-trunk
SUCCESS
59633
 Artifacts

mev-release
NOT AFFECTED

mev-ts-release
NOT AFFECTED

mev-ts-release-1.6.1
NOT AFFECTED

mev-ts-release-bitbake
NOT AFFECTED

mev-ts-release-external
NOT AFFECTED

mev-ts-trunk-bitbake
NOT AFFECTED

mev-ts-trunk-bitbake-c1
NOT AFFECTED

mev-ts-trunk-external
NOT AFFECTED

mev-upstream-idpf
NOT AFFECTED

mev-ts-trunk BAT tests: success - build 59633 - it is required if PR have BAT MEV status check as mandatory. If failed, please review Tasks in provided link to Berta. For questions please contact with<EMAIL_ADDRESS>


Academic account of project for publication

Still seeking a venue for this publication (our Digital Classicist Berlin seminar proposal having bee declined), but we should think about what sort(s) of paper(s) we want to write.

A technical paper for IT/tech audience?
A Digital Humanities paper targeted at DHQ/DCO or similar?
A non-technical paper targeted at epigraphists/classicists?

@ajenhl @polinayordanova any thoughts?

I think 1 and 2 should be merged - there isn't enough interesting technical stuff for a pure IT/tech audience, but some of the pieces might be of interest to the DH community. (though more in what they do/enable than in how they do it. Which, now that I say it, sounds like 2 and 3 merged. Partly this is informed by the earlier (there was on last year, I think?) publication(s) on Kiln, and the big changes in EFES are interesting from a quality of life for epigraphers perspective more than any particular new technology (though the i18n transformer is pretty cool, it's been around in Cocoon for over a decade, for example).

An academic paper is currently being completed by Bodard, Norrish, Stoyanova and Yordanova. Will report here/close ticket when publication is in sight.

Now published at:
Gabriel Bodard & Polina Yordanova. 2020. “Publication, Testing and Visualization with EFES: A tool for all stages of the EpiDoc editing process.” Studia Digitalia Universitatis Babeș-Bolyai 65.1, pp. 17–35. Available: https://doi.org/10.24193/subbdigitalia.2020.1.02


Path completions do not work with nested modules

From https://github.com/Microsoft/vscode/issues/24946
TypeScript Version:  2.6.0-dev.20171011
In a new project:

npm init -y
npm i xstream

Open a new js file:

import x from 'xstream/extra/'

Try triggering suggestions after the extra/

Expected behavior:
Show imports for files in the xstream module's extra directory
Actual behavior:
Generic module names are suggested

It seems that folders are still correctly suggested (asking for completion after writing import x from 'xstream/ will suggest the extra/ folder, with the folder icon too), but asking for completion after the extra/ shows generic names.


feature request: open API

It would be wonderful with an API to integrate with other system(s). It would also be wonderful if the API was easily browsable like https://en.wikipedia.org/wiki/OpenAPI_Specification
My use case is this:
I frequent a community where we cook and eat together. We want to do monthly clearing based on the number of times we each participated and the individual expenses we had for cooking. Tracking of participation is done in other system(s). If spliit had an API we could build automation and save man-hours and human errors transferring the information.
I know of at least 6 communities in my region that share my situation.
This feature may come with some strings attached like splitting front and backend, adding an authentication layer to avoid robots, ...
Thank you guys for a beautiful and simple app

This would probably be better implemented as a separate application with REST endpoints that directly manipulate the database. Self-hosting people could then install this alongside the app, pointing to the same database
I wouldn't necessarily be comfortable with this being available for the public website either.

It would be really useful for a mobile App

How complicated is the database schema to have a separate API? Won't it make more sense for the web service to use the API or for the code to be at least shared? So it won't cause issues between them?

This would probably be better implemented as a separate application with REST endpoints that directly manipulate the database. Self-hosting people could then install this alongside the app, pointing to the same database
I wouldn't necessarily be comfortable with this being available for the public website either.

Wouldn't every change regarding functions interacting with the database need to be implemented twice? Once for the Spliit app itself and once for the REST application? And what would happen if these different implementations don't align?

I think the requirements for the API have to be collected first.
For a mobile app a way of "sync" would be helpful/good but increases complexity quite a lot.
That is a properly a different API use case as @jensfriisnielsen has in mind.
Maybe you could write what kind of endpoints you need first and then one could try to figure out the rest.

I also want to use an API, not only to create an app (I need offline support to add pictures etc) but also to extract expenses done during travel trips and present them in the format asked by my organization.
In term of API, one may want to look at:
https://ihatemoney.readthedocs.io/en/latest/api.html
That has a clear and simple API. Once we have one, we can maybe integrate it to moneybuster that already supports two apps (ihatemoney and nextcloud cospend)

Started writing something along the lines of (still need to fill in all the data fields and functions, need a "get user" function for example):
#!/usr/bin/env python3
import requests
from dataclasses import dataclass
import json

@dataclass
class Spliit():
    group_id: str

    def get_group(self):
        params_input = {"0":{"json":{"groupId": self.group_id }},"1":{"json":{"groupId": self.group_id}}}
        params = {
            'batch': '1',
            "input": json.dumps(params_input)
        }

        response = requests.get('https://spliit.app/api/trpc/groups.get,groups.getDetails', params=params)
        return response.content.decode()

    def add_expense(self, amount: int = 1300):
        params = {
            'batch': '1',
        }

        json_data = {
            '0': {
                'json': {
                    'groupId': self.group_id,
                    'expenseFormValues': {
                        'expenseDate': '2024-11-14T22:26:58.244Z',
                        'title': 'aaa2',
                        'category': 0,
                        'amount': amount,
                        'paidBy': '1pv6Y2kYY-7Y2ch1sSrUL',
                        'paidFor': [
                            {
                                'participant': '1pv6Y2kYY-7Y2ch1sSrUL',
                                'shares': 100,
                            },
                            {
                                'participant': 'HdA1uEaY_vwPQ3V_e_phQ',
                                'shares': 100,
                            },
                            {
                                'participant': 'rI3LmjoengeQHZWVCR8ax',
                                'shares': 100,
                            },
                        ],
                        'splitMode': 'EVENLY',
                        'saveDefaultSplittingOptions': False,
                        'isReimbursement': False,
                        'documents': [],
                        'notes': '',
                    },
                    'participantId': 'None',
                },
                'meta': {
                    'values': {
                        'expenseFormValues.expenseDate': [
                            'Date',
                        ],
                    },
                },
            },
        }

        response = requests.post('https://spliit.app/api/trpc/groups.expenses.create', params=params, json=json_data)
        return response.condent.decode()

if __name__ == "__main__":
    a = Spliit(group_id = "3w4pzrjzIyu3xipruh_XR")
    print(a.get_group())

I would love to have an API that allows me to insert new spendings. That way I could automate pays I made with my iPhone using Apples Shortcuts.
It would remove the need of copy pasting values from one screen to another.


Adds support for is_write_index to alias actions

Looking good. Thanks for contributing. Will be part of 6.2.15.


docs(recipes): Introduce Recipes section of the docs

Resolves #54
Introduce "Recipes" section on the docs. The goal of this section is to have a curated list of questions and answers that are (more or less) common and could benefit people working with fp-ts et al.
I think PR is a good starting point for Recipes. A few ideas to iterate on this:

Use a docs-ts like way of type checking and generating the recipes instead.
Improve the content of docs/recipes/index.md to explain how people can collaborate.

Build is green on CircleCI but still Pending here. https://travis-ci.org/github/gcanti/fp-ts-contrib/builds/665945727


Document Parameter.modifier

The problem
Parameter.modifier was undocumented. See #4294.
Solution provided
This PR documents Paramete.modifier.

Looks good, but I would prefer that the text of the link not be type smiley, here. it's a nickname, and isn't mentioned until several paragraphs in on the linked page.

What's a better term? I could link to the term in the glossary.

If you add type smiley in the glossary, link to that and then have a glossary entry link to the original page that’s fine

I linked straight to the term in the glossary. In the glossary, there was already a link to the page this was linking to originally.


Require certain statements to be revealed in a proof

When it comes to verifiable credentials, there are core statements in the credential, that the relying party MUST be able to see in a given proof in order for them to decide whether the presentation is valid, i.e expiry, revocation or issuance date.
With this, there is two major options,

The relying party must explicitly request this information in a proof request and the proof must be able to prove these statements or prove they were not included in the credential i.e the issuer did not include expiry information.
The issuer designates in the signature they generate, the set of statments that must be revealed from the signature, for the relying party to have all information required to decide whether the presentation is valid for them, e.g the issuer would air mark that the expiry, revocation and issuance date fields must be revealed in the credential, in order for the signature to validate.

When working on a similar idea but using redactable signature schemes i.e. not using ZKP,  I went with option 2 for a few reasons.

The issuer should be able to specify some non-redactable parts that are required for integrity of the assertion- e.g. year to date information to be valid needs an as-of date value.
I decided approach 2 is required for more domain specific statements that may not have standards around them or need to add optional/extended data. The issuer is going to need to be able to set definitively that it's invalid without some contextual data that may not be well known to the verifier.
#2 suffers in the hands of an overly prescriptive issuer but #1 suffers in all but the most strongly defined, well-known, non-deviating credential.

In my case since I was using an accumulator and witnesses for each  statement in the credential there was a witnessed header element that contained a list of required statements for the credential as a whole to be valid and the validation process was responsible to ensure those were in the credential and each had a valid witness in addition to any other statements selectively disclosed. The concept is the same the implementation is clearly different.

@manicprogrammer this is interesting, I have reached the same conclusion, I think we setup a dangerous system if we design for option 1 as the solution, this would mean the verifier or relying party has to explicitly request information that arguable if not revealed effects the integrity of the revealed statements. To make this concrete, I should not as a holder or presenter of a verifiable credential, be able to derive a proof on a verifiable credential that chooses to omit the expiry date because the credential is expired. I agree with your observations around approach 2 however I think it is the only viable approach as the issuer of the credential is the only person with the full context around the information they signed and hence should be able to prevent their assertion being taken out of context.
The solution we are working on now leads to an attribute in the resulting proof that indicates the indicies of the statements in the normalized order that must be revealed. This statement it self is also signed by the issuer and any derived proof must always feature it (even if it is empty).
Since this repository is a reference implementation repository and we now have a draft specification at the CCG, it would be great to continue the conversation here

If you don't mind I am going to open an issue on the CCG side on the use of the indices themselves. This is a larger architectural and intent discussion on the leakage these provide that if I am grokking what I am studying really undercuts the zero knowledge component.


Make Contact Us Card responsive

Currently at some point, content of card come out of it. Fix it and attach screenshots near each breakpoint (where you use media query).
Here is one example at 902 x 523

Another example

sorry but it seems that its already solved . i noticed that the icons are no longer visible, is that something you did to solve this issue?

I don't think I solved it, I will get back to you after checking in some hours.

okay , notice me when you can :)

You are right @evirusky, it has been resolved already here


PPD-299: Fixing npm security audit threshold

What does this pull request do?
Corrects the threshold used to flag the nightly npm security audit as a pass or failure
What is the intent behind these changes?
To make security vulnerabilities more visible

Which one of #442 #443 should be the real one? 😁


Add meshes to renderer

Removed shader for now -- since we only draw a non-tiled, non-transformed image today.


Update erlang package

This PR updates the version of erlang to <IP_ADDRESS>

@cf-london Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

Closing this PR and we'll open another one.


[SPARK-19791] [ML] Add doc and example for fpgrowth

What changes were proposed in this pull request?
Add a new section for fpm
Add Example for FPGrowth in scala and Java
How was this patch tested?
local doc generation.

Test build #73719 has started for PR 17130 at commit fdce240.

Test build #73719 has finished for PR 17130 at commit fdce240.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class JavaFPGrowthExample

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73719/
Test FAILed.

Test build #73790 has started for PR 17130 at commit ca12877.

ping @jkbradley since we're changing the FPGrowth transform.
Sean made a great suggestion to simplify transform code.

Test build #73790 has finished for PR 17130 at commit ca12877.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73790/
Test PASSed.

Test build #74383 has started for PR 17130 at commit 9ce0093.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74383/
Test FAILed.

The updated transform looks good; thanks for pinging!

Test build #3599 has started for PR 17130 at commit 9ce0093.

Test build #3599 has finished for PR 17130 at commit 9ce0093.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #74565 has started for PR 17130 at commit fa4c734.

Test build #74565 has finished for PR 17130 at commit fa4c734.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74565/
Test PASSed.

Test build #74636 has started for PR 17130 at commit de1bfc8.

Test build #74636 has finished for PR 17130 at commit de1bfc8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74636/
Test PASSed.

Refined some comments and minor things. This should be ready for review. Thanks.

Please hold on merging this until https://github.com/apache/spark/pull/17321 is resolved.

Test build #74994 has started for PR 17130 at commit 9fef280.

Test build #74994 has finished for PR 17130 at commit 9fef280.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74994/
Test PASSed.

I'll be happy to help get this merged now that the column renaming is done

Test build #75112 has started for PR 17130 at commit 9e908d0.

Noting here: Please check out the "Issue this PR brought up" here: https://github.com/apache/spark/pull/17218
It may affect this PR.  Thanks!

Test build #75112 has finished for PR 17130 at commit 9e908d0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75112/
Test PASSed.

Merged build finished. Test PASSed.

Test build #75273 has started for PR 17130 at commit c957ba5.

Updated with Python example.

Test build #75273 has finished for PR 17130 at commit c957ba5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75273/
Test PASSed.

Test build #75409 has started for PR 17130 at commit e9b090a.

Test build #75409 has finished for PR 17130 at commit e9b090a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75409/
Test PASSed.

Test build #75714 has started for PR 17130 at commit 0fb5a87.

Test build #75714 has finished for PR 17130 at commit 0fb5a87.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75714/
Test PASSed.

Merged build finished. Test PASSed.

right, how are we on this? let's get this ready soon and merge?
could you also add reference to the R example which is merged yesterday.

Test build #75947 has started for PR 17130 at commit 2b1efb3.

Test build #75947 has finished for PR 17130 at commit 2b1efb3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75947/
Test PASSed.

Test build #76287 has started for PR 17130 at commit ea3b973.

@felixcheung, reverted the code change of transform as requested. Please check the update. Thanks.

LGTM, thanks for adding this!  @felixcheung OK with merging?

Test build #76287 has finished for PR 17130 at commit ea3b973.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76287/
Test PASSed.

merged to master/2.2


BrowserInfo.TimezoneOffset type

Describe the bug
Hello,
The adyen documentation is not really clear on the browserInfo.timeZoneOffset type on /payments API.
API explorer show that it should be an integer
https://docs.adyen.com/api-explorer/Checkout/71/post/payments
and
https://docs.adyen.com/online-payments/3d-secure/api-reference/
is showing it as a string.
On the other end the golang SDK (https://github.com/Adyen/adyen-go-api-library) is expecting a Integer but from time to time the Adyen Component is returning that information as a string creating complexity and inaccurate information.
Expected behavior
Adyen component should probably always return that information as an integer
BrowserInfo
    "browserInfo": {
        "acceptHeader": "*/*",
        "colorDepth": 24,
        "language": "en-US",
        "javaEnabled": false,
        "screenHeight": 1080,
        "screenWidth": 1920,
        "userAgent": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/<IP_ADDRESS>",
        "timeZoneOffset": "-60"
    },

Hey @mderasse ,
Sorry for the delay getting back to you here.
It is weird that you are getting different types for the browserInfo.timeZoneOffset property.  We just call the built-in browser defined method of the Date object, getTimezoneOffset. This returns a number. We don't do any further manipulation of that field.
I tested through BrowserStack different Edge versions (122, 124, 125, 126) and they are all returning as Integer.
I am not sure what can be causing this function to return a string type in your case. Do you have more information about which browsers/versions are returning this as 'string', besides the Edge ones that I mentioned?

Hi @ribeiroguilherme
I was able to list a few user agent based on the payload the adyen component return.
 We tested in different browsers as I mentioned above, and we always get the Integer value. As pointed out above, we do not manipulate/cast the returned value at any moment. I suggest in this case that you add your own casting to Integer if you are facing this problem.
If you are able to reproduce in any browser and you can show us that the value is not being returned as Integer, we can re-evaluate this and perhaps add a strict cast on our side.
Cheers


Getting Error 400: "Changes are sent for review automatically. The query parameter changesNotSentForReview must not be set."

Hi there!
In advance, I want to thank you for this plugin!
It is really a time saver!
I've ran into an error when tried to execute it on my published app:
Traceback (most recent call last):   File "/tmp/bitrise/step_src/rollout_update.py", line 75, in <module>     main()   File "/tmp/bitrise/step_src/rollout_update.py", line 65, in main     commit_request = service.edits().commit(editId=edit_id, packageName=PACKAGE_NAME, changesNotSentForReview=True).execute()   File "/root/.local/share/virtualenvs/src/lib/python3.8/site-packages/googleapiclient/_helpers.py", line 130, in positional_wrapper     return wrapped(*args, **kwargs)   File "/root/.local/share/virtualenvs/src/lib/python3.8/site-packages/googleapiclient/http.py", line 938, in execute     raise HttpError(resp, content, uri=self.uri) googleapiclient.errors.HttpError: <HttpError 400 when requesting https://androidpublisher.googleapis.com/androidpublisher/v3/applications/com.my.app/edits/12345678:commit?changesNotSentForReview=true&alt=json returned "Changes are sent for review automatically. The query parameter changesNotSentForReview must not be set.". Details: "Changes are sent for review automatically. The query parameter changesNotSentForReview must not be set.">
I've forked this project, and changing line 65 to this, worked fine for me:
commit_request = service.edits().commit(editId=edit_id, packageName=PACKAGE_NAME, changesNotSentForReview=False).execute()
I'm not sure if this should be the default, but I'll open a pull request with this change. Please, let me know if this makes sense.
Thanks in advance!

Try upgrading from v0.0.1 to v0.02 of this step


Updated laminas-form to v3

This PR updates laminas-form from 2.17 to 3.0. As laminas-form v3 introduced typed parameters and return types pretty much everywhere, the signatures of some classes had to be changed in order to make tests pass.
A little caveat was the FormButton helper which allowed passing either an element or an array specification. The new method signature public function render(\Laminas\Form\ElementInterface $oElement, ?string $sButtonContent = null): string coming from laminas-form v3 won't allow that, therefore I created a new method renderSpec() and added support for the old style rendering of either elements or array specifications in the __invoke() method.
This PR should be considered a BC break.

@neilime Did you already have a chance to look into this?

I see two options :

Merge this MR and tag it 4.0.0, then merge bootstrap-5 and tag it 5.0.0 so we'll have:

3.x: Form v2 Bootstrap 4
4.x: Form v3 Bootstrap 4
5.x: Form v3 Bootstrap 5

Merge bootstrap-5 and tag it 4.0.0, then merge this MR and tag it 5.0.0 so we'll have:

3.x: Form v2 Bootstrap 4
4.x: Form v2 Bootstrap 5
5.x: Form v3 Bootstrap 5

What's the opinion of twbs-helper-module's contributors and  users ?
(@driehle @rarog as you've contributed to bootstrap 5)
Related to #165

@neilime Personally, I'd much prefer option 1. Upgrading laminas-form from v2 to v3 is rather straight-forward, while upgrading Boostrap from v4 to v5 involves quite some other UI-related word. Therefore, for my projects I have postponed the Boostrap upgrade, while I would be happy to upgrade laminas-form. Hence, the variant Form v3 / Boostrap v5 is the most relevant for me currently.
Do you think it would be possible to make the Boostrap version configurable somehow? If so, support for Bootstrap v5 could be implemented in a 4.1 release. Maybe something like this:
return [
    'twbshelper' => [
        'bootstrapVersion' => TwbsHelper\Module::BOOTSTRAP5,
    ],
];

Do you think it would be possible to make the Boostrap version configurable somehow? If so, support for Bootstrap v5 could be implemented in a 4.1 release.

I think that it will be hard to maintain the both version of bootstrap side by side

I'm for variant i, which would make more accent on forms support.

@driehle can you rebase your MR please

Personally, I'd much prefer option 1. Upgrading laminas-form from v2 to v3 is rather straight-forward, while upgrading Boostrap from v4 to v5 involves quite some other UI-related word.

I prefer this option too

Rebased, btw shouldn't tests/.phpunit.result.cache be deleted and put on .gitignore?

@driehle  Sorry for the late answer.
Beofre going to  form v3 then to bootstrap 5,  I have made some clean and improvments on test, ci and documentation.
Please can you fix the codesniffer issues (you can run composer ci on your side to be sure everything is ok)

@driehle

However, why are you arguing against treatPhpDocTypesAsCertain: false, especially in light of the discussion in phpstan/phpstan#1926?

Quite simple: if I don't have the problem by explicitly specifying the type, then the rest is of no interest and certainly not some relaxation of some type checking rules.

So to sum up, we keep the last commit I've done.
Set the phpstan @treatPhpDocTypesAsCertain: false@ property
Track and fix potential remaining issues
Merge, tag explain the breaking changes in the changelog
Et voilà !
Are you ok with that battle plan?

@neilime Is there something missing for a 4.0 release to be used with laminas-form 3.0? 😃

Done: https://github.com/neilime/twbs-helper-module/releases/tag/v4.0.0

Just updated the first application to laminas-form v3 and twbs-helper-module v4 - works like a charm! :rocket:

Just updated the first application to laminas-form v3 and twbs-helper-module v4 - works like a charm! rocket

Prepare yourself for bootstrap 5 !


File dialog not opening 3.12.x when used inside a modal

So, I recently upgraded the version from 3.4.0 to 3.12.x.
After doing this, the file dialog stopped opening when used inside a modal(tried with react-modal and react-modal-dialog so its not a modal issue) but works when not used inside a modal. I couldn't find the cause but just letting you guys know.
In the meantime, I have downgraded to 3.4.0 and it works completely fine. Let me know if you need more details!

Yes, a reproducible repository on GitHub would be a great start. A PR with a fix would be even better.

Closing of inactivity