8tracks

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2<IP_ADDRESS>. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
United Kingdom
Description
I found 8tracks on the list of sites supported by yt-dlp, and I was curious to try it out/download an old playlist I liked, but it seems that 8tracks is no longer supported/broken. While I'm not sure if many people still use this website, it would be nice if this could be resolved or the site could be removed from the list of supported sites. Thank you so much!
Verbose log
yt-dlp -vU "https://8tracks.com/juicedelishblr/b-r-e-a-t-h-e"
[debug] Command-line config: ['-vU', 'https://8tracks.com/juicedelishblr/b-r-e-a-t-h-e']
[debug] User config "/Users/user/.config/yt-dlp/config": ['--no-mtime', '-o', '~/Downloads/%(title)s.%(ext)s', '-S', 'ext:mp4:m4a']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, err utf-8, pref UTF-8
[debug] yt-dlp version 2<IP_ADDRESS> [e491d06d3]
[debug] Python version 3.8.12 (CPython 64bit) - macOS-10.16-x86_64-i386-64bit
[debug] exe versions: ffmpeg 5.0 (setts), ffprobe 5.0
[debug] Optional libraries: brotli, Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
Latest version: 2<IP_ADDRESS>, Current version: 2<IP_ADDRESS>
yt-dlp is up to date (2<IP_ADDRESS>)
[debug] [8tracks] Extracting URL: https://8tracks.com/juicedelishblr/b-r-e-a-t-h-e
[8tracks] b-r-e-a-t-h-e: Downloading webpage
ERROR: [8tracks] b-r-e-a-t-h-e: Unable to download webpage: HTTP Error 500: Internal Server Error (caused by <HTTPError 500: 'Internal Server Error'>); please report this issue on  https://github.com/yt-dlp/yt-dlp , filling out the "Broken site" issue template properly. Confirm you are on the latest version using  yt-dlp -U
  File "/Users/user/opt/miniconda3/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 617, in extract
    ie_result = self._real_extract(url)
  File "/Users/user/opt/miniconda3/lib/python3.8/site-packages/yt_dlp/extractor/eighttracks.py", line 107, in _real_extract
    webpage = self._download_webpage(url, playlist_id)
  File "/Users/user/opt/miniconda3/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 900, in _download_webpage
    res = self._download_webpage_handle(
  File "/Users/user/opt/miniconda3/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 768, in _download_webpage_handle
    urlh = self._request_webpage(url_or_request, video_id, note, errnote, fatal, data=data, headers=headers, query=query, expected_status=expected_status)
  File "/Users/user/opt/miniconda3/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 753, in _request_webpage
    raise ExtractorError(errmsg, cause=err)

  File "/Users/user/opt/miniconda3/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 735, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/Users/user/opt/miniconda3/lib/python3.8/site-packages/yt_dlp/YoutubeDL.py", line 3591, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/Users/user/opt/miniconda3/lib/python3.8/urllib/request.py", line 531, in open
    response = meth(req, response)
  File "/Users/user/opt/miniconda3/lib/python3.8/urllib/request.py", line 640, in http_response
    response = self.parent.error(
  File "/Users/user/opt/miniconda3/lib/python3.8/urllib/request.py", line 569, in error
    return self._call_chain(*args)
  File "/Users/user/opt/miniconda3/lib/python3.8/urllib/request.py", line 502, in _call_chain
    result = func(*args)
  File "/Users/user/opt/miniconda3/lib/python3.8/urllib/request.py", line 649, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 500: Internal Server Error

[X] I've checked that all provided URLs are alive and playable in a browser

You sure? For me, that URL gives HTTP 500 error in browser too

just double checked, the site might be buggy then. sorry for the bother! fingers crossed the site sorts itself out soon :/ it's weird bc i could play music on it but when i refreshed it just went blank? very strange honestly. thanks again for responding!


[1/8] Playlist system - Features added in v1.0

Playlist system in 1.0
This feature implements playlist system (something like youtube's). It will be the key feature of our bot.
Added commands

These will be actually not commands but subcommands of group /playlist
/playlist view <name> [user] - view your or user's playlist(s)
/playlist create <name> [copy-queue] - create a playlist with a given name. The copy-queue argument copies the queue to your playlist if passed
/playlist remove <element> - remove element from playlist
/playlist add <track> - add track to the queue (uses same autocompletion as /play command)
/playlist delete <name> - delete playlist with given name
/playlist play <name> [replace] - Append the queue with playlist's tracks. If replace is true it will replace all elements in the queue with playlist's tracks

Context menus
This update will not focus only on commands! It will implement context menus so you can right-click on a user and select what you want to execute

View user's playlist(s)
Copy user's playlist

Got it in 0.9.0. Planning the grand update 1.0.0 soon! Stay tuned, it will be awesome


Existing Oslo and HTML snippet

For an existing Oslo tenant, it's not clear what the process is to include the sample html snippet.  Is the suggestion to:
Clone the wch-site-application and the sample-html-snippet into a single directory, build and deploy over the existing Oslo application that comes with Standard?

Hey @scrumpmeister The process for installing the sample is:

Open the root folder of the cloned wch-site-application repository in a command terminal.
Run wchtools init to make sure that the tooling is configured with the correct details.
Run npm run install-layouts-from-git https://github.com/ibm-wch/sample-html-snippet.git and follow the prompts. This will deploy the new html-snippet content artifacts to your tenant and import the source code into your wch-site-application project.
Run npm run build to rebuild the SPA with the updated code.
Run npm run deploy to deploy the updated code to your tenant.

After these steps are completed, the html-snippet will be ready to be used throughout the site. Please let me know how these instructions can be better communicated in the readme.


Chore: update old endpoints

Description

A small cleanup for the transfer process controller (there were some things that I forgot to remove during the refactoring)
Removing some deprecated endpoints from the contract definitions controller
Adding a method to get a transfer process state

How to test it

Approach

Open Questions and Pre-Merge TODOs

Learning

Super!


Fix conditional multi return bug in the Wasmi executor

This bug was caused by the execution of Instruction::ReturnNezMany (multi-value).
When the condition was false and the return therefore not taken, the instruction pointer was not properly updated which caused invalid behavior.


Can't find end of central directory (jszip)

Describe the bug
jszip: can‘t find end of central directory: is this a zip file?
Document
java use freemarker, ajax callback a blob, just like this:

i just renderAsync this blob, error.
maybe freemarker created blob is diffrent?
Expected behavior
i hope it working
Desktop (please complete the following information):

OS: windows
Browser chrome
Version 22

At this point library works only with *.docx files, and doesn't support word *.xml files


More cleanup

Allow file extensions to be passed to newUntitled.
Implement dropdown icon over "New File" button.
Fix upload button handling.
Scroll should affect file list only (fixes #16).
Make open() take a path and remove refresh.
Split into two source files.
Show dialog for all model errors.

:+1:


Genji deals too much damage during ultimate

Created by: MaxwellJung
Because genji's global damage is increased, his ultimate damage is also increased.

mentioned in merge request ow1_emulator_backup!174

mentioned in merge request ow1_emulator_backup!144

mentioned in issue ow1_emulator_backup#710

moved from ow1_emulator_backup#711


Update projects 2021 05 13

An update of the projects based on the following issues:

Fixes https://github.com/KhronosGroup/glTF-Project-Explorer/issues/86 : "New Project: Facebook Spark AR Studio"
Might fix https://github.com/KhronosGroup/glTF-Project-Explorer/issues/93 : "New Project: Microsoft FlightSim"

For the Flight Simulator-related projects:
Maybe @elpie89 wants to see whether the wording is appropriate. I didn't dive too deeply into the technicalities of these projects. E.g. there does not seem to be a GitHub repo for the "Blender2MSFS" one (yet?).
@pjcozzi You suggested adding "FlightSim runtime itself" in issue 93. I can see the reasoning. It may be one of the largest and most broadly visible applications of glTF until now. But it's a slippery slope: When the use of glTF becomes broader (in applications and games), we'd have to consider where to draw the line, and how to deal with requests from developers to ~"add their project, because it also uses glTF". I can add it, if it is explicitly requested, but we should keep that in mind...

@javagl looks OK to me, folks are welcome to open a new PR if they have suggestions.


system_monitor: parse file path from file_pointer

fixes #299

@weirdwiz when can we merge this PR?


Gradle Migration Script Failure and Rewrite Issues in Spring Boot Application Upgrading from OptaPlanner to Timefold Solver

Describe the bug
When attempting to migrate a Spring Boot application from OptaPlanner to Timefold Solver using the provided Gradle script, I encountered a failure during the execution process. The goal was to automate the upgrade via the Gradle script specified by Timefold's documentation, aiming for a seamless transition to leverage Timefold Solver's capabilities within my Spring Boot project.
Expected behavior
I anticipated a smooth migration process facilitated by the Gradle script, which would automatically update project dependencies, adjust import statements, and apply necessary configurations for integrating Timefold Solver with my Spring Boot application.
Actual behavior
The migration attempt was unsuccessful, resulting in a method signature error concerning the classpath in the dependencies block of the upgrade-timefold.gradle script. Additionally, the process encountered issues with the rewrite functionality, which was supposed to apply automatic code modifications for the migration but failed to execute as expected.
To Reproduce
Follow Timefold Solver's migration guide for obtaining the upgrade-timefold.gradle script.
In the Spring Boot project directory, execute: gradle -Dorg.gradle.jvmargs=-Xmx2G --init-script upgrade-timefold.gradle rewriteRun -DtimefoldSolverVersion=1.8.0
Encounter the error related to classpath usage and issues with rewrite functionality.
Environment
Timefold Solver Version or Git ref: 1.8.0
Output of java --version:
openjdk 17.0.2 2022-01-18
OpenJDK Runtime Environment (build 17.0.2+8-86)
OpenJDK 64-Bit Server VM (build 17.0.2+8-86, mixed mode, sharing)
Windows 11

Thanks for reporting, @NaruRidan. Can you please provide the version of Gradle you used?

Sure, Gradle 8.6.

Could you tell me how to adjust the encoding settings so that Gradle correctly reads the files?
Thank you. I appreciate your help.

I don't use either Gradle or Windows, but it seems that StackOverflow knows.

It seems the issue on the user side is resolved.
Do we want to add the file encoding property to our gradle command? If that's possible.
Or just close this issue?

I've added the extra argument for encoding to our migration docs. Closing this issue.


#658 #752

If you made a contract change, make sure to complete the checklist below before merging it in master.
Refer to our documentation for more details about contract security best practices.
Contract change checklist:

[ ] Code reviewed by 2 reviewers. See template and example.
[ ] Unit tests pass
[ ] Slither tests pass with no warning
[ ] Echidna tests pass if PR includes changes to OUSD contract (not automated, run manually on local)

Closing since there is another PR with more recent code: #782


[WFLY-13640] Reactive Messaging 1.0 with Kafka Support

https://issues.redhat.com/browse/WFLY-13640
wildfly/wildfly-proposals#320
wildfly/wildfly-capabilities#48
wildfly/boms#98
Old version of PR is at https://github.com/wildfly/wildfly/pull/13783. This was pretty thoroughly reviewed. Apart from upgrades, the code changes are minimal

Unrelated failure CertificateRolesLoginModuleTestCase.testDatabaseCertLoginModule

Seems r


0.13.0-rc is tagged as latest on npm

>  npm dist-tag ls react-native
latest: 0.13.0-rc

Not sure if this was intended, but it means people will get 0.13.0-rc when running npm install react-native.
npm publish defaults to the latest tag, even for -rc builds :( Probably meant to npm publish --tag 0.13.0-rc?

Thanks for the note. It's now updated:
$ npm dist-tag ls react-native
latest: 0.12.0

:+1:

Same for 0.14.0-rc
npm dist-tag ls react-native
latest: 0.14.0-rc

@cosmith thanks! should be fixed now (points at 0.13.2)


Supercharger charges - showing even when free

Describe the bug
If I click "Free Supercharging" within the settings screen, the charge row within the charging_processes table always has a value assigned to it after charging with a Tesla Supercharger.
The code that derived the cost looks like this,  I assume the fast_charger_type is pulled in over the API, however if I look in the charging processes table both fast_charger_brand and fast_charger_type are always saved as
cost =
case {stats, charging_process} do
{%{fast_charger_type: "Tesla" <> _},
%CP{car: %Car{settings: %CarSettings{free_supercharging: true}}}} ->
0.0
Not sure why this is happening, perhaps it is not getting passed through from the API.  I am looking to add a provider to the geofences table, so if there is no workaround for this it could revert to checking against the geofence.
Expected behaviour
The charge should be saved with a zero cost if charging at a Tesla Supercharger and "Free Supercharging" is set to true in the settings screen.
How to reproduce it (as minimally and precisely as possible):

Relevant entries from the logs

paste logs here

Screenshots

Data

Environment

TeslaMate version: 1.22.0
Type of installation: Docker or Manual: Docker
OS TeslaMate is installed on: Synology NAS
User OS & Browser: Mac OS, Safari
Others:

What values are stored for the fast_charger_type in the charges table for the affected charging sessions?

Hey Adrian. Every single row in the charging table has  set against fast_charger_brand and fast_charger_type.

Hmm interesting. Usually, all supercharges are clearly identifiable via these fields. The values come directly from the API. Unless there is another way to detect the charges, there is not much we can do 🤔

How about I add a Tesla Supercharger bool to the geofence table?  I have all the UK superchargers already in there.  It could be used as a fallback in case there is nothing passed back from the API.  I was thinking of adding a "Network" field to the table anyway to allow people to write dashboards that group charging costs by the charging network.
QQ- Does the cost and currency get passed back by the API too?  I struggled to find the piece of code that handled the API call.  Any pointers would be great.

Alternatively, tags could be added to geofences. This would potentially enable other use cases as well. It would be nice if Tesla could provide accurate information via the API though 😄

QQ- Does the cost and currency get passed back by the API too? I struggled to find the piece of code that handled the API call. Any pointers would be great.

Unfortunately not. All available charging-related information is documented here: https://tesla-api.timdorr.com/vehicle/state/chargestate

I just checked out tags in Postgres. I get the concept, it basically creates an array of tags you specify, and then returns them during a SQL query.  I am new to Erlang + Ecto.sql, so dealing with and populating the array from the settings web page will be challenging.  If you are willing to help and give direction I will give it a go, otherwise I will revert to a table column as that will be simpler to alter the existing code.


chore(main): release 1.4.0

:robot: I have created a release beep boop
1.4.0 (2024-10-18)
Features

add-get-values (9158c05)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/psioz-org/go-util/releases/tag/v1.4.0 :sunflower:


Add seaweedfs

[ ] https://github.com/seaweedfs/seaweedfs
[ ] https://github.com/seaweedfs/seaweedfs-csi-driver
[ ] https://github.com/seaweedfs/seaweedfs-operator

See #5233


api/schema: drop merkle_proofs table

This table is no longer used since #40

fwiw I have a backup of the table.


Issue525 paramscheck

Fixes #525: params_check is now skipped for params objects that are not used in the GGIR part it is used in. This avoids unnecessary warning messages.

Checklist before merging:

[ ] Existing tests still work (check by running the test suite, e.g. from RStudio).
[ ] Added tests (if you added functionality) or fixed existing test (if you fixed a bug).
[ ] Updated or expanded the documentation.
[ ] Updated release notes in inst/NEWS.Rd with a user-readable summary. Please, include references to relevant issues or PR discussions.
[ ] Added your name to the contributors lists in the DESCRIPTION and CITATION.cff files.

Codecov Report

Merging #527 (93ac926) into master (6e54839) will increase coverage by 0.02%.
The diff coverage is 92.30%.

@@            Coverage Diff             @@
##           master     #527      +/-   ##
==========================================
+ Coverage   73.26%   73.29%   +0.02%
==========================================
  Files          93       93
  Lines       11223    11243      +20
==========================================
+ Hits         8222     8240      +18
- Misses       3001     3003       +2

Impacted Files
Coverage Δ

R/g.report.part2.R
58.50% <50.00%> (-0.12%)
:arrow_down:

R/extract_params.R
92.59% <87.50%> (-0.33%)
:arrow_down:

R/check_params.R
76.10% <100.00%> (ø)

R/g.part1.R
62.15% <100.00%> (+0.23%)
:arrow_up:

R/g.part2.R
58.88% <100.00%> (+0.22%)
:arrow_up:

R/g.part3.R
62.60% <100.00%> (+0.93%)
:arrow_up:

R/g.part4.R
73.57% <100.00%> (+0.08%)
:arrow_up:

R/g.part5.R
67.32% <100.00%> (+0.09%)
:arrow_up:

Continue to review full report at Codecov.

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


floyd cycle detection

Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Yes
Description
(Write your answer here.)
Floyd’s Cycle Detection Algorithm in Java
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)
Issue: #5917

Rep.it link: https://replit.com/join/hvowvvlb-sapna2001


feat: decentraland-gatsby, decentraland-dapps update and decentraland-ui with UserMenu Update.

BREAKING CHANGE: rename functions in UserMenu props in decentraland-ui

Just a heads up but it looks like the version of the package-lock was updated in this PR (I'm saying because of the +10k line changes). I dunno if that is intended.


Add Google.Iam.V1 API.

This is just the gRPC and proto generated code, for use
in other APIs such as Google.Pubsub.V1.

LGTM

@jskeet , could you clarify this part:

for use in other APIs such as Google.Pubsub.V1.

My understanding is that some Cloud APIs expose their ACLs via IAM features. But how does that work in practice?

Does this mean that the PubSub NuGet package would take a dependency on the IAM one?
Will the PubSub package export its own IAM types (and lead to naming conflicts) with other IAM-powered APIs?

The former - there'll be methods on PublisherClient (for example) which are IAM operations proxying to the gRPC method definitions, but still over the same gRPC channel as the other client calls.


Remove password creation from insecure mode

Fixes #2343

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/f9d3b5442cff87e0696660a3c4e2a52c116f7845/

@awoods187 or @vivekmenezes, since @mberhault is on vacation, can either of you review this docs change?

TFTR, @vivekmenezes.


fix: Restrict :has, :not, and others to selector arguments

(fixes #45)
This isn't the elegant solution I was hoping for, but it's the smallest change that I could make work. Since :has, :not, :is, and :where are the only pseudo-classes that take selector arguments, we can prevent the conflict by disallowing plain values inside of those four altogether.
I bet there's a simpler solution, too, but I'm submitting this just in case.

thank you! sorry for being so behind on reviewing this. just rebased and merged :slightly_smiling_face:


Failed to Verify Contract

What is wrong?
Following the simple dao blog post I installed fe via brew (fe -V => fe 0.26.0) and tried to run the command to verify the contract:
fe verify 0xf0adbb9ed4135d1509ad039505bada942d18755f $ETH_RPC_URL

Whereby I ran this in the fe_contracts directory of the https://github.com/cburgdorf/simple_dao/ repo on tag SimpleDAOBountyDeployment (also doesn't work on master). My RPC seems perfect functional with me able to retrieve both contract code (tested via cast code <addr>) and the current block number. I was using the public https://eth.llamarpc.com endpoint for reference.
Full error from running the command:
Failed to verify contract.

error decoding response body: missing field `result` at line 1 column 143

Hi Philippe, thanks for jumping on this and taking the time to report the issue. The reponse of the node that you are using seems to not have a result field. I'll take a closer look but in the meantime, try out this public node instead: https://eth.public-rpc.com

Ok, I see what the problem is, the node seems to be picky about the request not specifying the Content-Type header. It should be easy to fix.

I created a fix that will be in the next release: https://github.com/ethereum/fe/pull/952
Thanks again for reporting, please use another JSON-RPC provider such as https://eth.public-rpc.com/ in the meantime.


add use strict

Block-scoped declarations were throwing an error without strict mode enabled. This commit implements "use strict"; in

lib/index.js
rules/author.js
rules/date.js
rules/description.js
rules/image.js
rules/index.js
rules/publisher.js
rules/title.js
rules/url.js

CLOSES: #19

Thanks @Chris-James !
Sorry for the trouble on this. I've just published 1.0.6 which just uses Babel to transpile before publishing to npm so that this kind of thing won't be a problem any. Thanks for taking the time to pull request!


Update config for new connection

@SLdragon,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Stop using v1.json file for model definitions and grab from API directly

Current we are using the models in the V1.json file, this is easy but the naming does not match the resource objects directly. It would be better to grab the models from the API resource directly.

On hold until API sandbox access is restored.

Resolved


bug: Tabs using grid-span: 9999

What version of daisyUI are you using?
4.10.2
Which browsers are you seeing the problem on?
All browsers
Reproduction URL
https://play.tailwindcss.com/ajpvJUchNb
Describe your issue
This is not really a bug report rather a suggestion. I was using tabs and I wanted to turn on the grid visualization of the browser, as soon as I did that, my browser started lagging, and I have a fairly powerful PC. I found out that the root of the problem was the tab content being grid-column-end: span 9999.
I tried using grid to fix this issue but I couldnt (without manually adding the number of tabs you have), so I chose a flex-based layout, which seems to work as well as the original one but without the need of creating thousands of columns.
I took some code from the docs and only added a few  css lines to fix it as you can see here

  <input type="radio" name="my_tabs_2" role="tab" class="tab" aria-label="Tab 1" />

Tab content 1

  <input type="radio" name="my_tabs_2" role="tab" class="tab" aria-label="Tab 2" checked />

Tab content 2

  <input type="radio" name="my_tabs_2" role="tab" class="tab" aria-label="Tab 3" />

Tab content 3

That was the code, I just added these lines
.tabs {
  display: flex;
  flex-wrap: wrap;
}
.tab {
  order: 0;
}
.tab-content {
  order: 1;
  width: 100%;
}

By doing so, the tabs are gonna shrink by default, since they're flex-children, and the tab content wants to take 100% of the parent, so I added the flex wrap to make it go on another line, and added the order to place .tab-content as the last element

Hello, I have the same problem. The proposal is good.
I'd rather wait and see. It occurs on Safari (Mac and iOS) and only with the presence of the avatar component in the tab-content.
Here is my page: https://edge-components.jrmc.dev/avatar


ST3 jumps to correct linebut scrolls off the scree

For the most part Ctags seems to work with ST3, but I find quite often even though it jumps to the correct line that line is scrolled several pages up, so yo n eed to scroll up to find the selected line

This bug affects me too. It is really infuriating.


Colin API: Get business accesscode or password

For now, we won't be doing this.
We'll be generating deeplinks with access embedded as the default.

Deemd as not needed.
The Deep link will cover this need


fix(search): fix algolia search ignore ctrl + F in search input

Pre-flight checklist

[x] I have read the Contributing Guidelines on pull requests.
[x] If this is a code change: I have written unit tests and/or added dogfooding pages to fully verify the new behavior.
[x] If this is a new API or substantial change: the PR has an accompanying issue (closes #0000) and the maintainers have approved on my working plan.

Motivation
This fixes a UX bug. Ideally it gets fixed upstream I believe, for now I add the downstream hack.
Test Plan
See the testplan in https://github.com/facebook/docusaurus/issues/10333#issuecomment-2245764216.
Test links

Deploy preview: https://deploy-preview-10342--docusaurus-2.netlify.app/
Related issues/PRs
Fixes https://github.com/facebook/docusaurus/issues/10333

@mxschmitt I tested your fix in my local environment, and it seems to resolve the issue that I reported in #10333.
Thank you for creating this PR to fix the issue!🙏

thanks 👍


Operator[] / emplace / insert_or_assign on existing key may cause iter invalidation

Version of Boost
1.82
Steps necessary to reproduce the problem
https://godbolt.org/z/MeKzYbqsv
Crashes, because operator [] called with existing key as an argument causes reallocation and iterator invalidation ignoring the fact the key hasn't really been inserted. According to docs, iterators are not affected if the key isn't inserted.
All relevant compiler information
LLVM Clang 16.0.0
Originally found on MacOS Ventura 13.2.1
Notes
Relevant documentation for operator[] (reference) declares that iterator invalidation happens only on successful value insertion. However, as it can be found in source code for boost 1.82 (older versions too, probably), operator[] works via calling emplace(key, nullptr).
The real "problem" lies in the boost::json::object::emplace function. It is said in the docs it works the same way as [] - invalidation only upon insertion. Unfortunately, it's not stated that emplace always (ref to boost 1.82 source) reserves memory for at least size() + 1 elements before checking should the element be inserted or not - so iterators could be invalidated (if you are unlucky) even on unsuccessful insertion.
Suggested change
Haven't prepared the MR though, but I suppose it would be enough to move lines with memory reservation in boost::json::object::emplace and boost::json::object::insert_or_assign below checking for existing with (maybe) some extra checks (as detail::find_in_object asserts for non-zero capacity, for example).

This is technically a documentation issue, but I'm sure we should be able to make these functions provide the basic guarantee.

I tried my best to offer the strong guarantee everywhere, it would be disappointing to not achieve that

actually found this to be a duplicate of https://github.com/boostorg/json/issues/763, so you may close this one :)


Ultra slow on Windows 7

Hey,
I just tried PhantomJS with Ghostdriver for a very simple acceptance test. Just filling out 2 inputs on a website takes about 40 seconds (the website is empty besides those two inputs).
Some people told me it just takes time, but it only takes 4 seconds with Selenium. Do you have an idea what the problem is here? I'd still like to use PhantomJS.
Thanks!

Thank you for submitting this ticket!
This issue tracker primarily serves as the tool to report and track bugs, defects, and development tasks.
For general question, code review request, or other related discussion, please post it to either:

the official PhantomJS users mailing-list
Stack Overflow (with phantomjs tag)

If you have not read it yet, please check also a few tips on Effective Q&A.


Allow adding fees on top

Currently, when you donate $100, stripe charges you $100 and fees come out of that. Depending on how much host takes, end collective gets around ~$87.
We want to let users pay more fees on top of their donation. User clicks on Donate $100. Dialog shows:
$100
+ $13 fees
= $113
Confirm
Implementation proposal

Backend: Add a boolean field on the collective feesOnTop
Frontend: Create a new dialog that shows the final amount with fees

If flipped (initially only for the French movement with a French host, so no currency issues), we show the dialog above.
When they click confirm, we pass $113 to Stripe and show the Stripe dialog. Two dialogs aren't ideal but it would be the easiest way to do it. Otherwise, we'd need to rebuild a credit card donation form.

Open questions

Should we do one dialog or two?
Find out exactly how much stripe will charge for a european credit card. We can also assume a flat 3% rate and eat the difference of 20 cents or so.

Stripe fees:
1.4% + 0.25E for european cards
2.9% + 0.25E non european cards
P.
piamancini.com
t: @piamancini
s: piamancini
OpenCollective https://opencollective.com
On Thu, Apr 7, 2016 at 10:27 AM, asood123<EMAIL_ADDRESS>wrote:

Currently, when you donate $100, stripe charges you $100 and fees come out
of that. Depending on how much host takes, end collective gets around ~$87.
We want to let users pay more fees on top of their donation. User clicks
on Donate $100. Dialog shows:
$100

$13 in fees (stripe, platform and host) = $113 [Confirm]

Implementation proposal

Backend: Add a boolean field on the collective feesOnTop

Frontend: Create a new dialog that shows the final amount with fees

If flipped (initially only for the French movement with a French
host, so no currency issues), we show the dialog above.
When they click confirm, we pass $113 to Stripe and show the Stripe
dialog. Two dialogs aren't ideal but it would be the easiest way to do it.
Otherwise, we'd need to rebuild a credit card donation form.

Open questions

Should we do one dialog or two?
Find out exactly how much stripe will charge for a european credit
card. We can also assume a flat 3% rate and eat the difference of 20 cents
or so.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/OpenCollective/OpenCollective/issues/15

We won't know which card it is because this needs to be shown before the card is entered. We can pick the smaller one and eat the difference initially?

ahhh
donation range let's say is 5-20EURO son it's
EC: 0.32 - 0.53
NonEC: 0.39 - 0.83
Let's say in 500 7.57E and 14.75
I think it's ok to do it at the beginning but we should stay on top of it
because in large donations the difference is double.
P.
piamancini.com
t: @piamancini
s: piamancini
OpenCollective https://opencollective.com
On Thu, Apr 7, 2016 at 12:12 PM, asood123<EMAIL_ADDRESS>wrote:

We won't know which card it is because this needs to be shown before the
card is entered. We can pick the smaller one and eat the difference
initially?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/OpenCollective/OpenCollective/issues/15#issuecomment-207053948

PayPal is using
feesPayer = 'SENDER' or feesPayer='RECEIVER'
I would suggest using the same terminology but instead of SENDER and RECEIVER, we could use HOST, BACKER, COLLECTIVE.
For the frontend, and as an MVP, I would simply change the description in the Stripe modal to say "$20 + $3 fees"

I implemented a simple version of this: https://github.com/OpenCollective/website/commit/639cd15aff84852e5db43b73023b533c4b8e0cac. It runs into some issues.
I laid out two methods and their consequences in this spreadsheet: https://docs.google.com/spreadsheets/d/1EthmgOrYGQwZ78YTz3Yma7KHbpQeUuTD2BRcBNPGFXY/edit#gid=0. The commit above is Method 1 in the spreadsheet.
Method 2 is definitely better to implement - it has a huge overhead for every donation, especially smaller ones.

My biggest concern is not implementation - it's the perception. It sucks if we are adding 20% or 30% in fees. One option is not to include host fees, especially for organizations like Women Who Code. Added a few variations at the bottom of the spreadsheet to show what that would look like.

I talked to @cuiki this morning about this. He should come back today with a first draft design.
The idea is the following:
You click on "Donate $20 / month" then you have a new screen (a modal on desktop) that acts as the receipt like in the restaurant. It would basically show the breakdown. E.g:
Your donation: $20
WWCode Austin: $16.1
WWCode HQ: $2
OpenCollective fees: $1

Credit card processing fees: $0.90

Total: $20.90

[Pay by Credit Card ] [ Pay With PayPal ]

For #NuitDebout, it would look something like:
Your donation: €10 / month
#NuitDebout global: <--- €2.50 --->
#NuitDebout / media commission: <--- €2 --->
#NuitDebout / infrastructure: <--- €2  --->
#NuitDebout / security: <--- €2 --->
OpenCollective foundation: <--- €1 --->

Credit card processing fees: €0.50

Total: €10.50

[Pay by Credit Card ] [ Pay With PayPal ]

hey, this is the first approach to the donation "recipe

", i still have to work on the wording of the modal, feel free to give any kind of feedback 😬

I really like it.
Few comments:

Stripe shouldn't be an option (and we should label it "credit card processing fee")
not sure what are the boxes on the left. In that scenario people won't have the option to edit it. So we should also remove the input type text on the right.
I love the sliders!
I'd do a version with stripe and OC fees at the bottom and then have a line with "Total" (for the case when fees are added on top and not part of the donation)

im on it

This is great! 👍
Just also do a version with $100 per month and with two buttons at the bottom:
[ Pay by Credit Card ] [ Pay with PayPal ]
And ship it! :-)

@xdamman Could you tell me which, if there is an endpoint or something that exists that will tell me the fee amount, comission and the collectives where the donation will be distributed?

We are still discussing that with @asood123. For now you can hard code it.
const userEditable = true; // show slider if true
const connectedCollectives = [{slug, name, mission, percentage}, {}, {}];
const processingFees = 0.30 + 0.029 * amount;
const feesOnTop = true; // add the processing fees + OC fees on top of amount, otherwise, subtract those fees from amount

where percentage would be the percentage from the initial amount that should be attributed to that collective. The balance should go to the current collective.

Got it.
I also wanted to ask why you want two buttons saying: Pay with Paypal and Pay with Creditcard.
Wont the user be redirected to the paypal site and still have to choose between credit card and their paypal email?

It depends if the host has set up their paypal account in such a way that they can also accept credit card payments. It's not the default. Regardless, I think it's good to have those two options.
I'd just put Credit Card first, then PayPal.
Also, can you add the disclaimer text below those buttons:
By clicking above, you are pledging to give Women Who Code 501(c)((3) USD $1.00 per month for WWCode Austin. You can cancel anytime.

since that exact text is displayed one step before on the flow, i think it will be redundant, should we still add it?

We should remove that text from that step, it should only be just without clicking on Pay by credit card or Pay with Paypal.

@xdamman The Credit Card Processing Fee can only be displayed AFTER choosing either paypal or creditcard, which by design as of now, it is not possible. @erickrico is working on a redesign to allow you to select the type (paypal or creditcard) before you donate.
What do you think?

Mmmm... good catch. You are right. Then maybe better to find a way to select the payment method and depending on that showing the payment processing fees.
And by default, it should be Credit Card (and note that for most collectives there will only be one payment method)

Maybe add the processing fee just below the donation method? So that there is a direct link between the method you choose and the cost associated to it.

Any update on this?
I think that donation method should be at the bottom. It feels weird in terms of hierarchy of information to have it separated from the donate button. The bottom should be about the actual payment, including the method and related processing fees.

ok, Ascari is already working on it, let me make a new version, with more contextual information

hey, what do you think of this approach, all editable fees go to the top, we create a "sub-total" they pick the donation method, and afterwards you get the total amount the donation, either if we add the credit card processing fee on top of the actual donation or as a part of it

Love it!

this was done a while back.


ParseAssemblyInfo breaks on slightly complex GlobalAssemblyInfo.cs file

What You Are Seeing?
I get this error when running the task:
An error occurred when executing task 'Calculate Version'.
System.InvalidOperationException: Sequence contains more than one element
   at System.Linq.Enumerable.SingleOrDefault[TSource](IEnumerable`1 source)
   at Cake.Common.Solution.Project.Properties.AssemblyInfoParser.Parse(FilePath assemblyInfoPath)
   at Submission#0.ParseAssemblyInfo(FilePath assemblyInfoPath)
   at Submission#0.<<Initialize>>b__0_4(ICakeContext ctx)
   at Cake.Core.CakeTaskBuilderExtensions.<>c__DisplayClass20_0.<Does>b__0(ICakeContext x)
   at Cake.Core.CakeTask.<Execute>d__43.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Cake.Core.DefaultExecutionStrategy.<ExecuteAsync>d__4.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Cake.Core.CakeEngine.<ExecuteTaskAsync>d__31.MoveNext()

My GlobalAssemblyInfo.cs file looks like this:
using System.Reflection;
using System.Runtime.InteropServices;

[assembly: AssemblyCulture("")]
[assembly: AssemblyCompany("XXXXXX")]
[assembly: AssemblyCopyright("Copyright © 2008-2018 by XXXXXX")]
[assembly: AssemblyTrademark("All rights reserved.")]
[assembly: ComVisible(false)]
[assembly: AssemblyProduct("XXXXXX")]

#if(DEBUG)

[assembly: AssemblyConfiguration("Debug")]
#else
[assembly: AssemblyConfiguration("Release")]
#endif

#if (DatabaseVersion)
[assembly: Custom.Namespace.Utils.DatabaseVersion(2040002)]
#endif

[assembly: AssemblyVersion("<IP_ADDRESS>")]
[assembly: AssemblyFileVersion("2.4.52731.0")]
[assembly: AssemblyInformationalVersion("XXXXX version 2.4")]

What is Expected?
I really expected the method to ignore pieces it does not understand. E.g. our custom [assembly: DatabaseVersion()]. But clearly something breaks the method.
For completeness this is the code of our custom attribute:
    [AttributeUsage(AttributeTargets.Assembly)]
    public class DatabaseVersionAttribute  : Attribute
    {
        public DatabaseVersionAttribute(int databaseVersion)
        {
            DatabaseVersion = databaseVersion;
        }

        public int DatabaseVersion { get; set; }
    }

What version of Cake are you using?
0.30.
Are you running on a 32 or 64 bit system?
64 bit.
What environment are you running on?  Windows? Linux? Mac?
Windows 10.
Are you running on a CI Server?  If so, which one?
Not yet, still early development so it is not on the CI server yet.
How Did You Get This To Happen? (Steps to Reproduce)
My task looks like this:
Task("Calculate Version")
    .Does(ctx =>
{
    TeamCity.WriteStatus("Calculating version information.");

    if(!HasEnvironmentVariable("BUILD_VCS_NUMBER"))
    {
        var info = GetSvnDirectoryInfo(parameters.SourcePath).FirstOrDefault();
        var revision = (info != null) ? (int)info.Revision : 0;
        System.Environment.SetEnvironmentVariable("BUILD_VCS_NUMBER", revision.ToString());
    }

    var revisionNo = int.TryParse(EnvironmentVariable("BUILD_VCS_NUMBER"), out var result) ? result : 0;
    var fileVersion = $"{parameters.ProductVersion}.{revisionNo}.{parameters.PatchNumber}";
    var assmVersion = $"{parameters.ProductVersion}.0.0";

    TeamCity.WriteStatus("Updating GlobalAssemblyInfo");

    var assemblyInfoFile = sourcePath.CombineWithFilePath(new FilePath("Common/GlobalAssemblyInfo.cs")).MakeAbsolute(ctx.Environment);

    var infoFile = ParseAssemblyInfo(assemblyInfoFile);

    TeamCity.WriteStatus("Setting TeamCity build number.");
    TeamCity.SetBuildNumber($"{{build.number}} {fileVersion}");

    Information($"Assembly Version: {assmVersion} - File Version: {fileVersion} - Revision {EnvironmentVariable("BUILD_VCS_NUMBER")}.");
})
.OnError(ex => Error(ex.ToString()));

Output Log
GIST LINK

@Tdue21 Alas, the assembly info parser is really, really stupid (by design) and does not understand custom attributes nor pre-processor directives such as #if, #else or #endif.

@Tdue21 A solution would be to put these in a separate file which is not parsed.

@patriksvensson  Well, that requires adding another file to hundreds of projects. Not really a good solution 😄
I get why it might be intentionally stupid, but that also means, that it will is close to useless for many LOB applications.
Anyway, I ported an earlier powershell solution to the same problem, so I am not going to use the Assembly Info methods at all.

@Tdue21 Great. I'm going to close this issue. Feel free to reopen it if you want to.

@Insire True. But it is as much the preprocessor directives that broke it 😄


chore: remove label and assignee from issue template

Description
This PR removes the label and assignee event from the issue template so that the issue falls into the primary triage queue.
Related Issues
Replace this paragraph with a list of issues related to this PR from the issue database. Indicate, which of these issues are resolved or fixed by this PR.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] The analyzer (melos run analyze) does not report any problems on my PR.
[ ] All unit tests pass (melos run test:unit:all doesn't fail).
[x] I read and followed the Flutter Style Guide.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

@lesnitsky, is there anything I need to do to get the code_health check to pass?


[🐞] Qwik sends server version of components to browser (instead of browser version)

Which component is affected?
Qwik Rollup / Vite plugin
Describe the bug
I have a npm package that exports Qwik components. I want to have server, browser and edge versions of each. I use the exports key in package.json to dictate which one should be used.
When importing this package in a Qwik webapp, it will always send the server version of a component to the browser instead of the browser version.
Reproduction
https://github.com/samijaber/qwik-monorepo-playground/tree/qwik-server-import
Steps to reproduce

pnpm i
cd app
pnpm dev

Look at the console logs, and the network tab: qwik-monorepo-playground/lib/lib/node/index.qwik.mjs gets sent to the browser instead of qwik-monorepo-playground/lib/lib/browser/index.qwik.mjs

System Info
System:
    OS: macOS 12.1
    CPU: (10) arm64 Apple M1 Max
    Memory: 394.16 MB / 32.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 21.1.0 - ~/.asdf/installs/nodejs/21.1.0/bin/node
    Yarn: 1.22.19 - ~/.asdf/installs/nodejs/21.1.0/bin/yarn
    npm: 10.2.0 - ~/.asdf/plugins/nodejs/shims/npm
    pnpm: 8.10.2 - ~/.asdf/installs/nodejs/21.1.0/bin/pnpm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 119.0.6045.123
    Safari: 15.2
  npmPackages:
    @builder.io/qwik: ^1.2.17 => 1.2.17
    undici: ^5.26.0 => 5.27.2
    vite: ^4.4.11 => 4.5.0

Additional Information
No response

Qwik always sends what is in the  package.json qwik property. In your case
"qwik": "./lib/browser/index.qwik.mjs",

If you want to control sending different code to client vs server use
import {isServer} from<EMAIL_ADDRESS>
export MyFn = isBrowser ? fnA : fnB;

Keep in mind that isBrowser is dynamic until there is full build at which point it should tree shake the other branch.

Qwik always sends what is in the package.json qwik property. In your case
"qwik": "./lib/browser/index.qwik.mjs",

But as per my github issue, Qwik is actually sending /lib/node/index.qwik.mjs. It does not respect the qwik property.

Difference between dev mode vs build mode.


Nginx is spelled incorrecly

In the screenshot of step 7 there is a machine named "myNgenix" where is hould be "myNginx"

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

ID: abf8bb33-e4cf-3dc5-5510-1111b8f1bdbf
Version Independent ID: 3148d97d-2ff1-30f5-0487-dbfe93888a3f
Content: Deploy nginx to a Linux VM - Azure Pipelines
Content Source: docs/pipelines/apps/cd/deploy-linuxvm-deploygroups.md
Product: devops
GitHub Login: @alexhomer1
Microsoft Alias: ahomer

Thank you for your feedback. I will update the topic to include your recommendations.


Error: No such command 'verify-bioimageio-manifest'.

Is this function replaced by another one or should I just remove it from the checker?
pip install git+https://github.com/bioimage-io/spec-bioimage-io.git
python -m bioimageio.spec verify-bioimageio-manifest --auto-convert manifest.bioimage.io.yaml

This is replaced by bioimageio validate:
bioimageio validate manifest.bioimage.yaml --update-format

(we should update the readme to document these commands)

I have updated the readme.


2.0.0 Error: Path must be a string. Received undefined

I consistently receive this error in every project that I've upgraded to 2.0.0:
ERROR in ./~/css-loader!./~/stylus-loader!src/styles.styl
Module build failed: TypeError: src/styles.styl:87:1
   83|     .dropdown
   84|         position fixed
   85|         left -9999px
   86|         z-index 500
   87|
-------^

Path must be a string. Received undefined

  at assertPath (path.js:7:11)
  at extname (path.js:1429:5)
  at new SourceMapper (./node_modules/stylus/lib/visitor/sourcemapper.js:41:7)
  at Renderer.render (./node_modules/stylus/lib/renderer.js:94:9)
  at ./node_modules/stylus-loader/index.js:149:12
  at tryCatchReject (./node_modules/when/lib/makePromise.js:840:30)
  at runContinuation1 (./node_modules/when/lib/makePromise.js:799:4)
  at Fulfilled.when (./node_modules/when/lib/makePromise.js:590:4)
  at Pending.run (./node_modules/when/lib/makePromise.js:481:13)
  at Scheduler._drain (./node_modules/when/lib/Scheduler.js:62:19)
  at Scheduler.drain (./node_modules/when/lib/Scheduler.js:27:9)
  at _combinedTickCallback (internal/process/next_tick.js:67:7)
  at process._tickCallback (internal/process/next_tick.js:98:9)

Doesn't matter what file. Happens on first stylus file it tries to load and aborts.
Webpack.config.js:
module: {
        loaders: [
            {test: /\.styl$/, loader: 'style-loader!css-loader!stylus-loader'},
        ]
}

+1

Turning source maps off fixes this, it's waiting on #104 to be merged...
Did you guys die? @mzgoddard @shama

@nfour Sometimes I wonder that myself.

I'm getting this for new node versions:
   189| @import './print/print'
   190|
   191| /* Components */
   192| @import './components/**/*'
   193|
--------^

Path must be a string. Received undefined

Doesn't happen for node v5.1.0, but for 6.2.0+

https://github.com/stylus/stylus/blob/dev/History.md#0545--2016-04-28
update stylus to newest version

sorted)

my solution
npm install<EMAIL_ADDRESS>--save-dev
npm install<EMAIL_ADDRESS>--save-dev
npm install<EMAIL_ADDRESS>--save-dev

(node v7.2.0)
it works  😀

Thank you LinZap! We have a team of 5 devs and our projects stalled while onboarding new members because they had a newer version of node than the rest of us (since they had just recently set up their dev environment, and our environments are nearly 2 years old now). Anyway, your solution got us back up and running. Thanks!!!


Add support for custom type inference through the plugin options

Fix #29

Will aim to include as part of the next release if that works :)


Add support to Client Credentials Flow (OAuth2)

issue : https://github.com/tiangolo/fastapi/issues/774

All for this! I do see you didn't provide any tests though, is that on the planning?

Will add test yep, sorry!

Ready to go!

Hey @JarroVGIT, just wondering, is there a chance to get this merged? I'm willing to pick this up if there's any additional work needed to be done before merging.

@kokolem I am not the maintainer of this project; I just do some reviews now and then and help people out with issues. @tiangolo is the one that determines what is merged and not, but as you can see there is quite a backlog he is going through. Eventually he'll get to this, I'm sure :)

Would it be appropriate to add some info about using this flow on the advanced/security/oauth2-scopes/ docs page?

@tiangolo is there any plans to implement client credentials flow?

Also looking for a good client credentials flow. Is this still going?

Also here hoping for progress on the Client Credentials flow @tiangolo


Add support for child object subselects in queries

@jamessimone I think I´ve got a way to handle this feature, but just curious - do you guys have a big need for this feature in your implementation?

@jongpie  so far we haven't had the need.  One big use case I can think of would be once (if?) we move more onto the service side, since sub-selects for Case in particular has proven useful in the past.  That being said, at the moment, there is no practical difference between sub-selects and two queries.

@jamessimone same for me, I haven't had a need to use them yet (especially considering I hate the syntax), but I think I've found a good way to handle including them in the parent object's repo queries.
One difference I found is that the sub-selects don't appear to count towards the limit of 100 queries per transaction - so querying for case, case comments and case tasks seem to count as 1 query (I was always under the impression that they counted as separate queries - 1 for the parent object, 1 for the child object). This guy claims that they count towards a different, hidden limit of 300 sub-selects https://developer.salesforce.com/forums/?id=906F00000008wqLIAQ
With that in mind, I might end start to use them - either way, I think it´d be nice to have in the framework :+1:

Just adding another link with some limits info for future reference - https://developer.salesforce.com/docs/atlas.en-us.salesforce_app_limits_cheatsheet.meta/salesforce_app_limits_cheatsheet/salesforce_app_limits_platform_soslsoql.htm


Install Kind via 'go install'

Postsubmits and periodics are failing with a message to use 'go
install', so trying that. Also upgrading to lastest Kind version at the
same time.
Tested: none, no real way to test without submitting, plus all the tests
are currently broken anyway so can't make things worse.

Hi, may I get a quick lgtm on this please?
/assign @erikgb
/assign @rjbez17

/lgtm

/lgtm

Postscript: postsubmits are running correctly after this change.


Use factoryboy for random object generation in tests.

What's the problem?
https://github.com/ethereum/trinity/blob/3a09276d2f21adfc5b233272e1b1cc2ad1a0c11e/tests/p2p/helpers.py#L17-L25
https://github.com/ethereum/trinity/blob/3a09276d2f21adfc5b233272e1b1cc2ad1a0c11e/tests/p2p/test_discovery_v5.py#L22
https://github.com/ethereum/trinity/blob/3a09276d2f21adfc5b233272e1b1cc2ad1a0c11e/tests/p2p/test_kademlia.py#L257-L267
We have multiple implementations for generating random nodes and addresses.  Here's a new one coming in this
PR: https://github.com/ethereum/trinity/pull/270/files/6af1a64abff48e5e56f7da134c5503653734da2b#diff-7e73dc6326965d5487d2171f23c63beaR59
How can it be fixed.
We could fix this by just ensuring that we always import the one from tests/p2p/helpers.py but this is problematic because

I don't think it's a great pattern to be importing across test modules
If we extract the p2p module from the trinity codebase then those imports will break on the trinity side.
Having to import this utility makes writing tests more cumbersome and boilerplate heavy.

We can fix part of this by moving the utilities into p2p.tools.helpers.  This uses our convention of moving non-core utilities into a namespace that can be reliably imported, even by 3rd party libraries.  But this still means that you have to import the random_node function in your tests and it doesn't actually give much control over any details such as setting the address but randomly generating the public key.
We can fix this by expanding the random_node function signature to allow overriding the various fields as optional function arguments.  However, at this point, we've re-created factoryboy
So, I propose we establish a convention of using library.tools.factories as a standard namespace that houses factoryboy based factories for the various objects we want to generate during test runs.
As for reducing the overhead of having to manually import these in each test file...
Option 1 - fix import overhead with a fixture
Create a base fixture in conftest.py that looks roughly like this.
# tests/conftest.py
from p2p.tools.factories import (
    AddressFactory,
    NodeFactory,
)

class MetaFactory:
    AddressFactory = AddressFactory
    NodeFactory = NodeFactory

@pytest.fixture
def factories():
    return MetaFactory()

Then in our tests we can do:
# tests/p2p/test_thing.py
def test_thing(factories):
    node = factories.NodeFactory()  # generates a random `kademlia.Node`

Option 2 - auto inject access to the factories in tests

This could be difficult to get right due to not wanting to add factoryboy as a core dependency but maybe we can get around that with good error reporting.

We could expose the fixture by default using the pytest plugin API
https://docs.pytest.org/en/latest/writing_plugins.html#making-your-plugin-installable-by-others
# p2p/tools/factories/meta.py
from .kademlia import (
    AddressFactory,
    NodeFactory,
)

class MetaFactory:
    AddressFactory = AddressFactory
    NodeFactory = NodeFactory

# p2p/tools/factories/plugin.py

@pytest.fixture
def p2p_factories():
    try:
        import factoryboy
    except ImportError:
        raise Exception("A friendly error message that lets the user know they need to install factory boy")
    from .meta import MetaFactory
    return MetaFactory()

# setup.py

setup(
    ...
    entry_points={
        "pytest11": ["devp2p = p2p.tools.fixtures.plugin"]},
    },
    ...
)

With this model, the p2p_factories fixture would be available by default, even when installed as a 3rd party library.

cc @hwwhww as I think many of the bcc helper functions could be cleaned up using this pattern, and similarly I looking at sections like this:
https://github.com/ethereum/trinity/blob/3a09276d2f21adfc5b233272e1b1cc2ad1a0c11e/tests/core/p2p-proto/test_block_bodies_request_object.py#L25-L73
We could do major cleanup, probably removing most if not all of those helper functions.

Yeah, this sounds wonderful. I'm too much of a PEP20 fan and I've had to debug ruby too often to like option 2 but option 1 is great :+1:


V3.7?

Hi Tom - I'm interested in your plugin, as mapping product categories to multiple parent categories is exactly what I'm needing.   Do you have an upgraded version of this plugin for nop 3.7?

I believe that newer versions of nopCommerce now include the ability to add products to multiple categories, as is described here.


An error occurs when the route is lazy loaded

This error occurs when the route uses lazy loading.

Hi, I was just looking into the same. And it seems like you are able to solve this using the --deploy-url argument during application building.
e.g.:
ng build --deploy-url /appX/dist/appX/

In this scenario the project is monorepo, and at root you have the index.html with single-spa application, and at /appX you have the application you mention, and append dist/appX/ to for the default build path.
If you were to host these files statically on a webserver or CDN you'd provide the url to the root of the build files.

Hi, I was just looking into the same. And it seems like you are able to solve this using the --deploy-url argument during application building.
e.g.:
ng build --deploy-url /appX/dist/appX/

In this scenario the project is monorepo, and at root you have the index.html with single-spa application, and at /appX you have the application you mention, and append dist/appX/ to for the default build path.
If you were to host these files statically on a webserver or CDN you'd provide the url to the root of the build files.
EDIT:
You can also add this to angular.json, in the build step, you can under options add:
"deployUrl": "/appX/dist/appX/",

Thank you. But img access still failed, I did not find the correct settings.

img access still failed, I did not find the correct settings.

@supersongpiao are you using Angular assets for those images?
If so, there is an open issue about Angular assets in #59. Currently there is some discussion/debate/uncertainty about the best way to approach them within single-spa-angular.
Also, thanks @kbjerke for jumping in and helping out with this one

Closing this due to inactivity. Feel free to reopen with further comments and questions.


feat: put the termination hints before the where

but I am a bit stuck/confused in src/lake/Lake/DSL/DeclUtil.lean

I’m stuck with
| `(structDeclSig| $id:ident { $[$fs $[,]?]* } $[$wds?]?) => do
  let fields ← fs.mapM expandDeclField
  let defn ← `({ name := $(quote id.getId), $fields,* })
  `(command|$[$doc?]? @[$attrs,*] abbrev $(fixName id name?) : $ty := $defn $[$wds?]?)

in lake/Lake/DSL/DeclUtil.lean. It says
application type mismatch
  wds?.raw
argument
  wds?
has type
  TSyntax `Lean.Parser.Term.whereDecls : Type
but is expected to have type
  TSyntax `Lean.Parser.Termination.terminationBy : Type

but I would have expected the parser just skips over the omitted Termination.suffix in
def declValSimple    := leading_parser
  " :=" >> ppHardLineUnlessUngrouped >> termParser >> Termination.suffix >> optional Term.whereDecls

because
def suffix := leading_parser
  optional terminationBy >> optional decreasingBy

Where am I thinking wrong? Or is this a boostrapping-related problem?

Ah, this seems to work. Is there a nicer way?
  let foo : Option (TSyntax ``Termination.terminationBy) := none
  let bar : Option (TSyntax ``Termination.decreasingBy) := none
  `(command|$[$doc?]? @[$attrs,*] abbrev $(fixName id name?) : $ty := $defn $[$foo]? $[$bar]? $[$wds?]?)

Ah, I think I got it. Parser type annotations help!
| `(structDeclSig| $id:ident { $[$fs $[,]?]* } $[$wds?:whereDecls]?) => do
  let fields ← fs.mapM expandDeclField
  let defn ← `({ name := $(quote id.getId), $fields,* })
  `(command|$[$doc?]? @[$attrs,*] abbrev $(fixName id name?) : $ty := $defn $[$wds?:whereDecls]?)
|


Add support for MS SQL

Coverage decreased (-1.1%) to 90.813% when pulling a081d3e7af09b938d508dcf8b4e7afb4aa01eb6a on mssql into 39fec01657f39b0bbb8f64b697081057e0208eff on v1-stable.

Coverage decreased (-1.1%) to 90.813% when pulling b0624ab2164545e3d2c0b37a627c23d94027b2a8 on mssql into 39fec01657f39b0bbb8f64b697081057e0208eff on v1-stable.

Coverage decreased (-0.8%) to 91.166% when pulling 1b383c94d5345167e327ae53308ccbc55c0ae7d1 on mssql into 39fec01657f39b0bbb8f64b697081057e0208eff on v1-stable.

Coverage decreased (-0.8%) to 91.166% when pulling 09384b459498b89222626ab1bb24ba2d31c88bb5 on mssql into 39fec01657f39b0bbb8f64b697081057e0208eff on v1-stable.

Coverage decreased (-0.8%) to 91.166% when pulling 2723399ea6f3ce3f6e5d277992201d6393627c37 on mssql into 39fec01657f39b0bbb8f64b697081057e0208eff on v1-stable.

Coverage decreased (-0.8%) to 91.166% when pulling 2723399ea6f3ce3f6e5d277992201d6393627c37 on mssql into 39fec01657f39b0bbb8f64b697081057e0208eff on v1-stable.

Coverage decreased (-0.8%) to 91.166% when pulling 2723399ea6f3ce3f6e5d277992201d6393627c37 on mssql into 39fec01657f39b0bbb8f64b697081057e0208eff on v1-stable.

Coverage decreased (-0.8%) to 91.166% when pulling 0a38463fc9c26b309726b4822168de0ec2b505e3 on mssql into 39fec01657f39b0bbb8f64b697081057e0208eff on v1-stable.

Coverage decreased (-0.8%) to 91.166% when pulling d0b27205b5cbf2d635a3d2761543dbc41bfdd504 on mssql into 39fec01657f39b0bbb8f64b697081057e0208eff on v1-stable.

Coverage decreased (-0.8%) to 91.166% when pulling 7584b0d023e77fb1edec9c611e689c57efbf160d on mssql into 39fec01657f39b0bbb8f64b697081057e0208eff on v1-stable.

Coverage decreased (-1.1%) to 90.813% when pulling aed88517e138699530a2446da4e925a666ad2fff on mssql into 39fec01657f39b0bbb8f64b697081057e0208eff on v1-stable.

Coverage decreased (-0.8%) to 91.166% when pulling 70325dab376f3e39c548aa6fb1dc53af36ee8390 on mssql into 39fec01657f39b0bbb8f64b697081057e0208eff on v1-stable.

Coverage decreased (-0.9%) to 91.087% when pulling fb598bc9bc201652dc499212151b0ff6ecf571a9 on mssql into 39fec01657f39b0bbb8f64b697081057e0208eff on v1-stable.


fix: columns header title attribute use wrong attr

https://github.com/jspreadsheet/jexcel/commit/0a3c6a82ac7ba092106559393804d146751bae28#comments

It has been merged. Thanks


Don't silently swallow invalid candidate exception

Until #14 is fixed, aioice doesn't work with Chrome until STUN candidates come through, because it doesn't get a valid IP from the initial set of candidates. However, thanks to _early_checks, it does work in ICE-CONTROLLING mode, as long as request_received is called before _connect. In practice, this means the cli.py offer copy/paste example works, but none of the others do unless you add asyncio.sleep(1) before pc.setRemoteDescription.
To make it clear that the candidates are being rejected, this PR logs out instead of silently swallowing an invalid candidate.
As a secondary thing, I can't tell whether the _early_checks behaviour is correct. I don't know whether ICE should work with no candidates (Chrome doesn't even try to connect if there are no channels on a connection), but it seems to me that once you've called _connect, candidates could go into _check_list rather than _early_checks. There's no guarantee a peer will send any more candidates after the initial answer, is there?

Minor correction, it's not about HTTPS but whether you wait for all STUN candidates before answering. I haven't yet worked out why cli.py answer doesn't work.

Minor correction, it's not about HTTPS but whether you wait for all STUN candidates before answering. I haven't yet worked out why cli.py answer doesn't work.

Thanks for your PR!


TsLint PreCommit hook failing with<EMAIL_ADDRESS>

> overcommit --version
overcommit 0.42.0
> tslint --version
5.9.1

Expected result
with tslint semicolon rule enabled overcommit would stop a commit missing a semi colon from proceeding
Actual result
it lets you commit no problem
Debugging
I've tracked it down and it appears that it's related to the default flag option on the TsLint rule --t=prose.  As far back as I checked that flag should apparently NOT work, it's either -t or --format.  But as of<EMAIL_ADDRESS>--t=prose appears to have the same effect as either -t prose or --format prose.  With the latest version<EMAIL_ADDRESS>you get a good exit status but an error is shown:
error: unknown option '--t=prose'
I can't find anything on tslint saying there were changes, I'm guessing that the downstream dependency on commander updated in the interim and how commander parsed args changed.
The workaround for now is to remove the flags, as prose is the default formatter:
  TsLint:
    enabled: true
    flags: []

The weird thing, and why I didn't just do a PR, is that for whatever reason, while -t prose and --format prose both work with<EMAIL_ADDRESS>from the cli, changing the flag argument to either of those still fails with the error that it's an unknown option.  Hopefully someone with more knowledge than I knows the reason and in the meantime, this workaround is out there.

Hey @garrinmf, thanks for the report.
Tagging @jhuiting as they added support for tslint in #421. Happy to accept a pull request fixing this. Thanks!

This was fixed in #594.


canal start sync binlog err: ERROR 1236 (HY000): Could not find first log file name in binary log index file

MySQL master + slave, run go-mysql-elasticsearch in slave, with errors:
INFO[0000] start sync binlog at binlog file (mysql-bin.000059, 49693955)
ERRO[0000] close sync with err: ERROR 1236 (HY000): Could not find first log file name in binary log index file
ERRO[0000] canal start sync binlog err: ERROR 1236 (HY000): Could not find first log file name in binary log index file
ERRO[0000] start canal err ERROR 1236 (HY000): Could not find first log file name in binary log index file
mysql> SHOW SLAVE STATUS\G;
*************************** 1. row ***************************
Slave_IO_State: Waiting for master to send event
Master_Host: <IP_ADDRESS>
Master_User: backup
Master_Port: 3306
Connect_Retry: 60
Master_Log_File: mysql-bin.000059
Read_Master_Log_Pos: 49693955
Relay_Log_File: mysql-relay-bin.000174
Relay_Log_Pos: 49694168
Relay_Master_Log_File: mysql-bin.000059
Slave_IO_Running: Yes
Slave_SQL_Running: Yes
Replicate_Do_DB:
Replicate_Ignore_DB:
Replicate_Do_Table:
Replicate_Ignore_Table:
Replicate_Wild_Do_Table:
Replicate_Wild_Ignore_Table:
Last_Errno: 0
Last_Error:
Skip_Counter: 0
Exec_Master_Log_Pos: 49693955
Relay_Log_Space: 49694422
Until_Condition: None
Until_Log_File:
Until_Log_Pos: 0
Master_SSL_Allowed: No
Master_SSL_CA_File:
Master_SSL_CA_Path:
Master_SSL_Cert:
Master_SSL_Cipher:
Master_SSL_Key:
Seconds_Behind_Master: 0
Master_SSL_Verify_Server_Cert: No
Last_IO_Errno: 0
Last_IO_Error:
Last_SQL_Errno: 0
Last_SQL_Error:
Replicate_Ignore_Server_Ids:
Master_Server_Id: 1
Master_UUID: e654de07-8e39-11e7-9bb3-00163e00702d
Master_Info_File: /data/mysql/master.info
SQL_Delay: 0
SQL_Remaining_Delay: NULL
Slave_SQL_Running_State: Slave has read all relay log; waiting for more updates
Master_Retry_Count: 86400
Master_Bind:
Last_IO_Error_Timestamp:
Last_SQL_Error_Timestamp:
Master_SSL_Crl:
Master_SSL_Crlpath:
Retrieved_Gtid_Set:
Executed_Gtid_Set:
Auto_Position: 0
Replicate_Rewrite_DB:
Channel_Name:
Master_TLS_Version:
1 row in set (0.00 sec)
ERROR:
No query specified
how to fix?
Thanks

start sync binlog at binlog file (mysql-bin.000059, 49693955)
Could not find first log file name in binary log index file

seem your binlog has been truncated. you can remove the master.info and do a full sync.


Devcontainer for CodeSpaces

Created a devcontainer for codespaces to enable development with correct Java version.
#207

Hi, I'll clarify the issue.

This is the error I get when I run the code in Gitpod

Same with Codespaces

I am really new with the devcontainer configuration for codespaces and development with Java in general. I just committed the boilerplate code that GitHub generated for me for Java.
I saw we have to populate the postgres credentials in the .env file, so inserted an configuration for that too.

I saw we have to populate the postgres credentials in the .env file, so inserted an configuration for that too.

the .env and the database are required to run the application, but not to fix a bug or implement a new feature.
So I'm not sure having the postgres container is needed.

Ideally, you write tests, unit-tests and integration tests to validate your changes. That could apply to any development language, JavaScript included 😉
If you want, you can use the docker-compose already present in the repository to start the application. Please, see https://github.com/jenkins-infra/plugin-health-scoring/blob/main/docs/CONTRIBUTING.adoc#how-to-run-the-application-locally

Ideally, you write tests, unit-tests and integration tests to validate your changes. That could apply to any development language, JavaScript included wink

That makes sense. Thanks!
I can't use Docker right now as I am limited by my system resources, so I have everything installed locally.
I could really use some help right now in understanding why I am getting this error even when I have my .env populated

java.lang.RuntimeException: Driver org.postgresql.Driver claims to not accept jdbcUrl, jdbc:postgresql://${POSTGRES_HOST}:${POSTGRES_PORT}/${POSTGRES_DB}
I am running this code mvn spring-boot:run -Dskip.yarn -Dspring.profiles.active=dev

The .env file is used with Docker Compose. But as you are running the application with the Maven CLI, you need to export them in your terminal. Please see https://github.com/jenkins-infra/plugin-health-scoring/blob/main/docs/CONTRIBUTING.adoc#running-the-application-from-cli

The .env file is used with Docker Compose. But as you are running the application with the Maven CLI, you need to export them in your terminal. Please see https://github.com/jenkins-infra/plugin-health-scoring/blob/main/docs/CONTRIBUTING.adoc#running-the-application-from-cli

Thank you!

Short summary of what I have achieved in the latest commit.

I have removed PostgreSQL database as @alecharp pointed out was not required which I found out myself too after having a better grasp of the project.
I have configured a docker-outside-of-docker in devcontainer.json. It's definition as I found it is Re-use the host docker socket, adding the Docker CLI to a container. Feature invokes a script to enable using a forwarded Docker socket within a container to run Docker command.
Successfully tested by running the project on VS Code for Web. Attaching screenshot below

Point to Note: I have to manually forward the port 8080 after only which I can access URL
Some points I need help in understanding

I have used docker-outside-of-docker  instead of docker-in-docker just because the first option was letting me access the port, but I don't have the clarity why.
I ran the project in VSCode for Web. But sometimes (for some reasons not every time) when I try to run it on IntelliJ Idea through Jetbrains Gateway, I get a SSH error. I think this might be related to this post https://code.visualstudio.com/remote/advancedcontainers/develop-remote-host. I need some help in understanding how to go forward to resolve this SSH issue because as far as I understand it would vary from user to user

I'm puzzled. The PostgreSQL database should be required to run the application. I thought this was about being able to contribute to the project from VSCode with some kind of pre-filled environment.
How is this different from #209? Are the two pull requests doing the same thing but on different platform?

I'm puzzled. The PostgreSQL database should be required to run the application. I thought this was about being able to contribute to the project from VSCode with some kind of pre-filled environment.

I think I am not able to convey my message. On opening a github codespace from the web, an environment is generated with the configuration I have committed. This environment allows to run the docker commands which generates a PostgreSQL on its own. I removed PostgreSQL with the thought that it might be unnecessary if we have the option of doing the same thing using Docker.
Should I add it again?

How is this different from #209? Are the two pull requests doing the same thing but on different platform?
#209 is about development using Gitpod. #208 is for development with Github Codespaces

I have an environment in Github codespace prefilled with

Java 17
NodeJS
Yarn
Docker

Is this still a draft @MondalSayantan ?

Hi @alecharp,
This PR is still in draft as I was focused on my other two PRs.
Also, after some research and talking to people, I have come to know that not many people know and use Github codespaces as their development environment. Most people prefer Gitpod(#209). I was wondering if my work on this PR would be helpful to the community.
I know I might be asking this question very late, but I did not realize this when I opened this PR. It would be extremely helpful for me if you could provide your wisdom on this.

Rewrote the complete devcontainer.json file to tackle the basic issues first, i.e, installation of correct Java and Maven version.
Verified it by running mvn verify locally.

I'm closing this pull request because I don't see the need for it. The fact that no one asked to get it merged over the past year illustrate my point.
@MondalSayantan Thank you for the time you dedicated to the project, but this doesn't seem to be to the benefit of the project.


Properly select "Release" build for Travis

Ninja doesn't support multiple build configurations. The option --config Release has no effect when cmake --build is called. Essentially, all testing was performed for "Debug" configuration, which is the default. Maybe it doesn't really matter, but it's misleading. The proper way to select build configuration here is through CMAKE_BUILD_TYPE.

Coverage decreased (-0.5%) to 99.549% when pulling fca58d08848eaf182b1bb7775d747ff07aca98bf on karzhenkov:fix-travis-release into b9d76516a2f4e8102ecac5fd8ba0c8c97ce3c382 on nlohmann:develop.

The coverage test requires the use of the "Debug" configuration. From the GCC documentation:

if you want to prove that every single line in your program was executed, you should not compile with optimization at the same time

@karzhenkov Can you please update to the latest develop branch? I changed a lot of issues in the CI and moved some checks from Travis to GitHub Actions. Maybe the changes of this PR may need to be applied in cmake/ci.cmake.

I think there is nothing here that we can apply to cmake\ci. cmake.

Thanks!


[Schedule] Add notes dropdown UI not intuitive

Expected Behavior: Dropdown bar changes upon changing the startup
Actual Behavior: Dropdown bar doesn't change upon changing the startup
Steps to reproduce:

Go to schedule
Click yellow button to add note on the right hand top corner
Click the dropdown
Note that the only thing that changes is the title. I thought at first nothing changed, because the dropdown bar did not reflect that.

Screencast attached
cast.webm.zip

Not clear to me what you expecting here.
It should change only the startup name.
Please ping me over hangout.

Wasn't able to ping you but this is what I meant but hope this helps to clarify


[Feature] RightToLeft prop missing

Hello, the library input-mask-ios and input-mask-android has created a prop to be able to type from right to left. Do you have any prediction of when it will be implemented here?

https://github.com/RedMadRobot/input-mask-ios/wiki/2.2-Right-to-Left-&-Digit-Grouping
https://github.com/RedMadRobot/input-mask-ios/wiki/2.-Text-Field-Listener

Yes please, is it possible to type a number RTL (like normal monetary inputs) without this feature?

Yes please, is it possible to type a number RTL (like normal monetary inputs) without this feature?

You are welcome to submit a PR for this feature.

Now supported, check out https://github.com/react-native-text-input-mask/react-native-text-input-mask/releases/tag/v3.1.1


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2379

Questions ?
Please refer to the Backport tool documentation


Capture stack traces from remote errors

When an error occurs in a call to eval(), invoke(), xqueryEval(), or a resource service, expose the stack trace from the server in the Node.js MarkLogicError object.
As @jmakeig said in https://github.com/marklogic/node-client-api/issues/297:

We should get the stack trace from the remote error too, serialize it over the wire, and stash it in the MarkLogicError instance. As it is, it's difficult to figure out where errors happen on the server in custom code, like eval and invoke (and presumably resource extensions).

This will require C++ work on the server. From @ehennum:

These endpoints are implemented entirely in C++.  Most of the implementation is shared with the XDBC server.
The C++ code that would have to change is that endpoint code -- not the implementation of the builtin xdmp:eval() and xdmp:invoke() functions exposed to SJS and XQY.
In other words, the C++ builtin functions happily throw an error with a call stack but the endpoint invocation of the same code doesn't support a way to return that callstack as the HTTP response.

QConsole is able to get stack information by wrapping executed code in try/catch statements. This may or may not be relevant. See: Apps/qconsole/endpoints/evaljs.sjs

@jmakeig, right, no work has been done to support this yet. Changed label back to "fix."

Test the conditions.


Add the beginnings of RPC fuzz testing

I noticed while developing pachctl misc grpc that some methods crash pachd when sent empty messages.  So, I decided to write a test that calls every method with an empty request.
This found a lot of bugs.
I added protoc-gen-validate to perform pre-RPC validation (in an interceptor), and used that to fix the bugs.
This pretends to do a unary RPC, but works with streaming RPCs as well.  There are definitely states where the second message being an empty can cause a crash, so we'll start looking at that next.
It's implemented in terms of the testing.F fuzzing API, but fuzzing still isn't super reliable.
This work is part of CORE-1955, but doesn't fix that bug yet.

Overall this is great! We've seen a few weird empty request issues on the Python client side, so definitely a welcome addition.


Create CODE_OF_CONDUCT.md

Issue
Currently a code of conduct is missing.
Fix
Adding Contributor Covenant.
Closes https://github.com/ProgramEquity/amplify/issues/409

Thanks so much! If you add the text: Closes https://github.com/ProgramEquity/amplify/issues/409 to your PR description, that issue will automatically be closed when you merge!

Thanks so much! If you add the text: Closes https://github.com/ProgramEquity/amplify/issues/409 to your PR description, that issue will automatically be closed when you merge!

Congratulations on getting your second PR merged! ✨

Congratulations on getting your second PR merged! ✨

Thank you! 😄


Taken pictures deleted after app update

Bug (?)
Version Number of Plugin: 2.6.2
Device Tested On: Various (iOS)
Expected Behavior
Taking pictures, storing their path in the model will reference them for further use.
Actual Behavior
Taking pictures, storing their path in the model references them for further use... until you update the app!
When i update the App via direct deployment OR apples testflight, everything stays as it should.

Database survives
Database-content survives
Settings stay the same
But the taken pictures seem to get deleted

What am i missing?

Yeah, actually that bug was not directly related to the Media Plugin. It could basically be removed, its nonsense ;)
Von meinem iPhone gesendet

Am 18.05.2017 um 01:03 schrieb James Montemagno<EMAIL_ADDRESS>Did you close this for a reason?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks :)


[Epic] XS security artefacts support

This issue is an epic linking all the backlog items related to end to end management of the XS security artefacts from HANA XS classic engine 1.x versions.

#31
#41
#69
#80
#81
#42
#82

This issue is closed as the project is no longer maintained


Use setup_requires to ensure cython and numpy are present

Addressing issue #256
Summary of changes:
I'm using setup_requires to indicate the dependencies on "setup" time, with install_requires to indicate the dependencies on "run" time.
I'm also ensuring the setup_requires is evaluated before making any reference to them.
As numpy version I've used the declared dependency for Pandas 0.17.0

Current coverage is 87.25% (diff: 100%)

Merging #260 into master will increase coverage by 0.06%

@@             master       #260   diff @@
==========================================
  Files             5          5
  Lines           414        416     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            361        363     +2
  Misses           53         53
  Partials          0          0

Powered by Codecov. Last update 6d93b2f...9c661cc

Need to make sure this doesn't conflict with conda environments -- setuptools has a way of wanting to build NumPy or Cython from source when it's already installed in your site-packages

A problem with using setup_requires this way is that, if they're missing, setuptools will install a temporary copy of cython and numpy for each setup.py invocation.
One solution is to cargo-cult this function around: https://github.com/pyca/cryptography/blob/8e66ca6813016d9fc6f57d5f1e50530fc39f78ae/setup.py#L114 -- the cryptography project runs into a similar chicken-egg problem with cffi.
Mercifully, this will be resolved in pip 10 by PEP 518 (pyproject.toml).
pip PEP 518 tracking bug: https://github.com/pypa/pip/issues/3691

-- although that doesn't model a complete solution for feather since cryptography manages to avoid actually importing cffi directly, relying on cffi's setuptools extension hooks instead; packaging is hard.

Yes, I know :) Thanks for looking into this.

Ping on this? I just tried to install feather-format and it failed. Not a great first-run experience for someone new to the format.
It also means we can't use a requirements.txt, as pip doesn't know the dependency and therefore can't install numpy and Cython first.

Closing as Cython is no longer required to create the feather Python package (which is just a convenience interface to pyarrow.feather)


Bug Fix Spl Flags

default flag is now SplFileObject::READ_AHEAD|SplFileObject::SKIP_EMPTY
AbstractCsv::__toString is also bug fixed

This PR fixes two bugs related to SplFileObject flags used in the library

The default flags are now SplFileObject::READ_AHEAD|SplFileObject::SKIP_EMPTY. So that using foreach will return the expected data even when multiple lines are presents in CSV fields
the __toString method needs to normalize the flag usage to output correclty the file. Flags were badly applied which led to missing rows in the output


Bidi  log.entryAdded event implementation

BIDI log.entryAdded event implementation

squashed and rebased commits to solve merge conflicts.


Ref/bug/dragndrop more robust #861mfzrb9

Summary

Replaced Alpoha with on/off,
Moved logic out of handlers,
Using events inside handlers to handle drag and drop
Only use onHover for showing/hiding images

Task linked: CU-861mfzrb9 Fix: DragAndDropHandler line 31 contains a temporary fix. Make it more robust


Scaffold-DbContext : Exception calling "Start" with "1" argument(s): "The system cannot find the file specified

I keep getting this error while trying to scaffold from an existing mdb file, I have checked and confirmed that ACEOLEDB 12 in installed also I target .NET Framework 4.7.2 and package are EntityFrameworkCore.Jet 2.2.0  and Microsoft.EntityFrameworkCore.Tools 2.2.4
Scaffold-DbContext : Exception calling "Start" with "1" argument(s): "The system cannot find the file specified"
At line:1 char:1
+ Scaffold-DbContext -Connection "Provider=Microsoft.ACE.OLEDB.12.0;Dat ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : NotSpecified: (:) [Scaffold-DbContext], MethodInvocationException
    + FullyQualifiedErrorId : Win32Exception,Scaffold-DbContext
Is there anything I am missing or doing wrong?
Also I tried the same setup targeting .NET Core 2.2 but get a different error
'The application to execute does not exist: 'C:\Users\olash\source\repos\Foltoy.Legacy.RecentBanker\packages\Microsoft.EntityFrameworkCore.Tools.3.1.1\tools\netcoreapp2.0\any\ef.dll'

Please post us the commands you tried to execute.

via PMC
Scaffold-DbContext -Connection "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=C:\RecentBanker_MFB\RDBMS\CDB2.MDB;Jet OLEDB:Database Password=mysecretpassword;" -Provider EntityFrameworkCore.Jet -OutputDir Models -verbose

Do you get the same error when running it via the command line as:
dotnet ef dbcontext scaffold "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=C:\RecentBanker_MFB\RDBMS\CDB2.MDB;Jet OLEDB:Database Password=mysecretpassword;" EntityFrameworkCore.Jet -o Models -verbose
How does the full verbose output look like?

Yet to try running it through the command line .
Although, I setup a fresh project and set it to compile as x86, and is given another error of "file cannot be opened by older application" so I tried using Microsoft.ACE.OLEDB.4.0, but that also is given an error The 'Microsoft.ACE.OLEDB.4.0' provider is not registered on the local machine.
Its obvious the mdb file version is old, and I can seems to find ACOLEDB 4.0 driver online.
SO far, I have spent to much time getting it to work, so I am trying a different path of using ADO in .NET Standard Library and port it into .NET Core and build from there
Thanks

The older 4.0 driver is Microsoft.Jet.OLEDB.4.0 (not ACE).


[Coverity CID :188168] Concurrent data access violations in /lib/posix/pthread.c

Static code scan issues seen in File: /lib/posix/pthread.c
Category: Concurrent data access violations
Function: pthread_create
Component: Other
CID: 188168
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996

Fixed by #10106


[WIP Zeppelin-945] Interpreter authorization

What is this PR for?
This PR is for interpreter authorization.
What type of PR is it?
Improvement
Todos

[ ] - only show the lock button if admin login.

What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-945
How should this be tested?

login as admin
then you can see the lock button on the right top as following image.

you can give users permission as check on the checkbox you want.
run paragraph job as user and see results if job has run or not properly.

Screenshots (if appropriate)

Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? yes

@astroshim Thanks for the contribution.
Regarding UI, why don't we use the same UI with notebook authorization?
That can give the same user experience for all the authorization inside of Zeppelin.
Than it'll look something like

What do you think?

@Leemoonsoo Thank you for your review and I think your opinion makes sense.
I'll change the UI.

@Leemoonsoo I would say, since the use case for this is different than that of notebook, and idea for this is to map access to users with interpreters.
While reading the title/description of this PR this is what I had in my mind

Also think about the case where user is getting auth using LDAP/AD where there are 100s of user, so this will require pagination. So, we need to have a group/profile for users that can have preset for grouping interpreters together.

And once this is set, IMO, then user who don't have permission to a particular interpreter, should not see that interpreter in the notebook page.

@prabhjyotsingh Thank you for share your great idea.
This PR is for interpreter authorization but your idea seems include user management.
I think it should be separated and I agree with @Leemoonsoo's idea of giving the same user experience for the authorization inside of Zeppelin.
and I also agree with @prabhjyotsingh's idea of the difference with notebook's because interpreter authorization is n:n. (it's hard to show with notebook's way.)
so What do you guys think about following way?

when you want to grant permission of interpreters to all users

when you want to grant permission of interpreters to specific user

I'll do this at the another PR.


[wip] First Draft for module reorg and Travis fixes.

@pinpong , this is just an initial draft for the three next steps from #32 . I'd appreciate your review once you have some time. No actual code should have been changed, only reorganizing the modules. Once it's reviewed and merge, I'll send my PR for the Kerl.java and Converter

Sorry for the delay, I've been a bit busy for the last couple of weeks but I'll rebase and update this PR soon.

This PR has been rebased and updated to fix the merge conflicts.

Looks sane to me! Could you just bump the bouncycastle version?

Version bumped


terraform should always follow Plan  then Apply from plan file

The operator doesn't look like it saves the plan file from the plan run. It should always produce a plan output and add that to the execution.
Then, if auto-apply is on, the plan file should be executed, but the plan should be appended.
Approvals should also only run against the plan.

this is the behavior.


chore: replace options: Record<string, any> with options: { access_token?: string } while using profile apis

With better type definition, we should be able to detect error in earlier stage of developing

I think we can define the { access_token?: string } as a option type


ACR image pull with User Assigned Managed Identity fails

This issue is a:

[x] bug report -> please search issues before submitting
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
Similar to issue #905. We are working with Terraform in an enterprise landing zone conforming to Cloud Adoption Framework. We've created a Container Registry in a production subscription and are attempting to pull images from it in acceptance, test and development subscriptions using a managed identity. We have followed documentation as far as we can see. However, since it's a landing zone with 2 hubs and firewalls (prd and acc/tst/dev are separate), the examples and related issues we have found are not applicable.
This is the error we are encountering:
│ Error: creating Container App (Subscription: "A"
│ Resource Group Name: "rg-***-acc-workload"
│ Container App Name: "ca-***-api"): performing CreateOrUpdate: unexpected status 400 (400 Bad Request) with error: InvalidParameterValueInContainerTemplate: The following field(s) are either invalid or missing. Field 'template.containers.***-api.image' is invalid with details: 'Invalid value: "cr***.azurecr.io/***/***-api:initial": unable to pull image using Managed identity /subscriptions/P/resourceGroups/rg-***-prd-workload/providers/Microsoft.ManagedIdentity/userAssignedIdentities/id-prd-acr for registry cr***.azurecr.io';.

Even going across subscriptions within the same hub results in this error. When creating a dedicated Container Registry in the same subscription, this issue does not occur.
Steps to reproduce
Too many to describe. Landing zone architecture with 2 hubs and multiple subscriptions is necessary.
Expected behavior
Successful pulling of image from container registry.
Actual behavior
Non-descriptive error message: 'unable to pull image using Managed identity'.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Using Terraform with AzureRM provider.

@jvalteren does your ACR allow ARM audience tokens? https://learn.microsoft.com/en-us/azure/container-apps/managed-identity-image-pull?tabs=azure-cli&pivots=azure-portal#prerequisites

Hi @vturecek, thanks for your response. The issue has since been resolved. It had to do with the networking between the Production and Acceptance/Test/Development hubs not being correctly configured, so the Acceptance ACA's couldn't reach the ACR in Production. The setup was the first fully Cloud-native Azure setup that was implemented (previous were hybrid), which caused the team to overlook some crucial network configuration between the 2 hubs.
For completeness, the target ACR in Production shows that ARM audience is enabled.
We can now successfully pull images in Acceptance ACA's from the Production ACR.


Removed dot and colon from keywords

This fixes https://github.com/bakpakin/fennel.vim/issues/9

I feel like I'm still missing something, but I'm not exactly sure what yet. I'll keep taking a look to see if I actually missed anything.


added page type in deletable items

What are the relevant tickets?
Closes https://github.com/mitodl/hq/issues/5358
Description (What does it do?)
This PR is to allow users to delete pages only if the content is not being referenced in other contents.
How can this be tested?

Make sure you have permission to delete content (Your email is added to the allowed list on PostHog)
Go to any page and check if the Delete menu is there.
refer to the instructions here for populating referencing content fields.
When the content is being referenced elsewhere, delete is prevented and it shows a warning message like in the screenshot below

The functionality works as expected. 👍🏻
But can you please add the error message prior to clicking Delete button on Dialog.

@ibrahimjaved12. To show the error message before the delete dialogue, I would need to check if the content has any reference. I'd have to create another API that would be called by the Delete button from the vertical menu. That's an API per deletable content type. With this PR, we have three deletable content types.
I would suggest we log this change request in a separate ticket. @pdpinch can triage it.

I agree with Umar. But is this behind a flag so we can merge but control the roll out?

I agree with Umar. But is this behind a flag so we can merge but control the roll out?

This is an addition to the existing delete feature, which is already behind a feature flag. Should we add a separate flag to check the references before delete?


[SMALLFIX] Removed redundant initializer in AlluxioLoginModule

Removed a redundant initializer in the AlluxioLoginModule class.

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/Alluxio-Pull-Request-Builder/9739/
Test PASSed.

LGTM, thanks!

LGTM


Add case when a has no element at index to operator-safe-dereference.md

@mumian
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@dmealo - thank you for your contribution.
#sign-off