EquityOrder has .order not .place_order

Hi!
Thanks for providing this API! Just wanted to let you know of a bug:
It says: order_response = equity_order.place_order('AAPL', 'buy', 1, 'limit', 150)
And 'equity_order' is an object of 'EquityOrder' which doesn't have a 'place_order'  method, it only has an 'order' method.
So it should be: 'order_response = equity_order.order('AAPL', 'buy', 1, 'limit', 150)`
Cheers!

Thanks for pointing this out! You're absolutely correct. There were actually a bunch of mistakes that needed cleaning up as well, so I really appreciate the heads up. I need to be more careful in the future about letting ChatGPT write README files for me


what param is for default time...?

one cal time widget set to "13:00" default and the other set to "10:00"
what param is for default time...?  cal/time - same time appear cal with time pickers. you do Not press to appear time picker.

so how date today bu default time 10:00 first and 13:00 second????


LAU2/nuts areas duplicate others and are overwriting them

Think the solution is to remove these from the list of areas


Downloaded images are broken

What steps will reproduce the problem?
I searched a tag from sankukaku in the first tab, get all the images from that page, and head to the download tab to download them. The images are downloaded but are broken or corrupted.
What is the expected behavior? What do you get instead?
I wanted to download images from sankukaku. Instead I get broken files. FML
How often does this problem occur? On which sources?
It's the first time I used this program after transferring from Danbooru Downloader because it wouldn't let me download images over 1000. The source is from Sankukaku
What version of the program are you using? On what operating system?
Latest program I guess? Windows 10
Please provide any additional information below
Also, when I open the images, I get "Expired link - please reload the site". Please help me guys. This is so frustrating

Update: I was able to download from Gelbooru. Sankukaku downloads remain broken however

Duplicate of #1057. Let's move the discussion there.
But please note that Sankaku is blocking downloaders (disabled API, lots of limitations, etc.), so Grabber is not really working with it for now (see issue #997).


Invalid cross-device link when shipping npm specified modules to browser

Bundling in Fresh with npm specifiers has this issue when /tmp and home are on two different filesystems:
      detail: Error: Invalid cross-device link (os error 18): link '/var/home/ewired/.cache/deno/npm/registry.npmjs.org/preact/10.15.1/LICENSE' -> '/tmp/68aa0db9/LICENSE'
    at async Object.link (ext:deno_fs/30_fs.js:401:3)
    at async linkRecursive (https://deno.land/x/esbuild_deno_loader@0.8.1/src/loader_native.ts:165:5)
    at async linkRecursive (https://deno.land/x/esbuild_deno_loader@0.8.1/src/loader_native.ts:162:7)
    at async NativeLoader.#nodeModulesDirForPackageInner (https://deno.land/x/esbuild_deno_loader@0.8.1/src/loader_native.ts:126:5)
    at async NativeLoader.nodeModulesDirForPackage (https://deno.land/x/esbuild_deno_loader@0.8.1/src/loader_native.ts:80:17)
    at async onResolve (https://deno.land/x/esbuild_deno_loader@0.8.1/src/plugin_deno_loader.ts:261:28)
    at async requestCallbacks.on-resolve (https://deno.land/x/esbuild@v0.17.19/mod.js:1336:22)
    at async handleRequest (https://deno.land/x/esbuild@v0.17.19/mod.js:699:13) {
        code: "EXDEV"

Ran into this as well and spotted 2 issues
Hard link
Fails as it doesn't make sense when pointing on separate filesystems.
If keeping the hardlink strategy, I see only 2 solutions :

Making a custom temporary folder directly inside DENO_DIR.
Falling back with a separate copy-based strategy

Rename
Fails because Deno.rename just runs the underlying syscall.
std/fs has move but it actually doesn't handle separate file systems. This has been noticed recently and there's a PR on Deno.rename but Ryan goes in the direction of including the case in std, which makes sense.
Anyhow, solving the rename won't solve the hard link strategy !

I actually followed up with solution 1. :point_right: https://github.com/lucacasonato/esbuild_deno_loader/pull/79
@ewired you may solve your issue by temporarily importing from my fork as well

Use at your own risk :upside_down_face:
import { denoPlugins } from "https://raw.githubusercontent.com/ngasull/esbuild_deno_loader/0144b46ceca6324c55b6246018000f81e09ec631/mod.ts";


Wrong result with multiple CPUs

Environment
Tracarbon version: v0.7.2
Environment:

Source: pypi
OS: Linux (NixOS)
Other:

Bug
What happened: The server I tested on has multiple physical CPUs. The resulting energy usage is completely wrong.
What you expected to happen: The energy usage is correct.
How to reproduce it: Run the Tracarbon on a server with multiple physical CPUs
More details:
The problem seems to be that multipe CPUs means that there are multiple RAPL directories. This is handled mostly correctly, but there is the issue that there are multiple dram or core  sensors, which is a problem when storing previous results in a dictionary in the RAPL implementation. There is no distinction between core and dram or CPU 1 or CPU 2.
This then causes problems with the calculation of differences, since the old CPU 2 value could be used for the new CPU 1 value or vice versa.

Thank you for reporting this issue @patricgruber!
Could you please attach an example of the arborescence with files and some examples?
As you can imagine, accessing multiple hardware configurations is challenging.
So, we might need test cases to ensure we cover and provide the correct energy usage.
Thanks in advance 🙏


fix libstdc not found without modifying podfile (eas)

Hi,
Is it possible to fix the libstdc problem when you don't have access to the Podfile?
I'm using expo eas build and I can't modify the podfile without ejecting.
Is it possible to fork the project and modify the podspec or something?
I think eas build service uses xcode 12.5
Thanks..

@RobertSasak
Thanks for the quick reply,
Ok so.. I'm very novice when it comes to this so I will probably not be able to solve it..
If I were to use the OCR that comes with react-native-camera, would it make a big difference?
I mean, if for example I would like to retrieve license number from US plates, I know the format so I should be good to go? I guess what I'm asking is.. Do I lose alot by not using this package?


Add @generated to IGC generated code

The sonar scan is currently showing very high stats on duplicated code
See https://sonar.odpi.org/component_measures?id=org.odpi.egeria%3Aegeria&metric=duplicated_lines_density
Originally the subject area was a big factor (removing this from scans reduced dup counts to 5% or so). However now the generated IGC code is a large factor.
We can then set sonar to skip duplicate scanning on all lines where @generated is used?
This is far from perfect, and if it is awkward I will stick to exclusion based on selected file paths - it just seems @generated is probably a more generic approach to cater for other scanning tools as it's effectively standard java
cc: @cmgrote
See also https://github.com/odpi/ci-management/issues/34

Very happy to make this change — any example you can point me to so I can be sure I get the syntax right? =)

https://docs.oracle.com/javase/8/docs/api/javax/annotation/Generated.html
it seems to still be in Java 11 (which we should compile with)
https://docs.oracle.com/en/java/javase/11/docs/api/java.compiler/javax/annotation/processing/Generated.html
though there appear to have been some changes done around standard annotations after java 8 -- so hopefully things won't break (not tested myself yet)

I'm testing this out on my own (in my own Sonar instance), but it doesn't seem that @Generated is automatically causing duplication checks to be ignored.  Is there something somewhere I need to configure to ensure that @Generated is used to ignore duplicated lines?
(Also as a heads-up, it also seems to have increased the processing time of the Sonar report by 10x -- hopefully avoidable by whatever extra configuration I need for the above point?  Otherwise I'd suggest we're probably back to just ignoring paths -- eg. anything whose code package has generated in it...)

... in fact, this documentation seems to imply that for duplicate code checks, the only way to ignore them is to exclude the files themselves (via a file pattern):
https://docs.sonarqube.org/latest/project-administration/narrowing-the-focus/#header-3
(?)

Indeed - by default that is the case. In my personal nexus scan I do just that.
However since @generated is fairly standard java, the sonar docs give a tip on defining a pattern that (a little crudely) removes @generated tagged files from the scan
I explain the thought in odpi/ci-management#34

I've tried @javax\.annotation\.Generated (as documented), which had no effect; then @Generated assuming it is only doing a regex match, and still no effect...
The generated files are still all listed as having heavy duplication.
I think it's still worthwhile having the annotation to indicate that the classes are generated, so I'll put in a PR with the change, but it won't have any impact on the duplicate line check in Sonar it seems.

your change would only take effect when the sonar config was changed on the build machine. It has been done now, and the reports are looking hugely better :-) (see comment on issue for more info)

So...  we can close this issue now? 😉

yes...


Release public version

nistbeacon is approaching a nice, complete version for the public. It's time to get serious and to start planning the v1.0.0 release.

Depends on #17

Closing for now, as this now depends on #23


fix: watcher hanging

Looks like @tg339 and @phiSgr were right
This seems to fix our hanging tasks issue

Still the explanation is not quite satisfactory. It's only blocking one thread, why can't the other threads run the other async functions?

May as well revert my revert commit.


False positive for lombok with dependency check 3.1.0

False positive on library org.projectlombok:lombok:1.16.20 - reported as cpe:/a:spice_project:spice:1.16.20
http://repo1.maven.org/maven2/org/projectlombok/lombok/1.16.20/
It seems spice is detected because of lombok's description
Spice up your java: Automatic Resource Management, automatic generation of getters, setters, equals, hashCode and toString, and more!
in the maven pom.

Still same for 3.1.1.

I didn't get a chance to go through the FP for the 3.1.1 release - a major issue came about due to changes external to ODC that required 3.1.1 to be released prior to me being able to go through the FP.


Prune NaN values from metrics to send to DataDog

Before this fix, NaN values received from containers were not removed
from the metrics which were to be sent to DataDog. When they were
encoded to JSON before being sent, the JSON encoder barfed and the
entire tranche of metrics was not sent.
Now, NaNs are selectively pruned and valid metrics are sent.
Fixes DCOS-20683

@crackcode123 ready for review :)


Scrolling / menu issues with WKWebView

https://www.youtube.com/watch?v=K_vBD6IJdpE
List of issues visible in this video, with time references:
0:05 - While the menu is open, the content view can be scrolled, then once the scroll has completed (0:11), it locks down and can no longer be scrolled.
0:12 - Some odd behavior with swipe gesture. The menu doesn't initially follow the swipe (presumably because the user was swiping up, first). It then, however, recognizes the swipe after the user has released the screen, and snaps closed.
0:20 - The content view can be scrolled, again. Once scrolled, it stops being scrollable (as it should be from the start (0:22))
WKWebView is definitely an improvement for this behavior, since prior to upgrading, the content view was always scrollable while the menu was open, and now it's just sometimes scrollable, but still not quite right.
Thoughts? Thanks.

@lincolnthree did you find a solution for this problem? I have exactly the same issue here.

@sinedied I think it essentially disables threaded scroll, thus putting scroll onto the main thread.


docs: logical concepts update

Signed-off-by: Lee Calcote<EMAIL_ADDRESS>Notes for Reviewers
This PR is a docs chore
Signed commits

[ ] Yes, I signed my commits.

🚀 Preview for commit ddb327a6195285fc20fe78c33197ab8aa96fbd2f at: https://659387fbf7667f380f1db797--meshery-docs-preview.netlify.app

🚀 Preview for commit f73f84fc6499c96d9365e8afd879e6e819c65b82 at: https://6593c8d4b4d60c62d02e61ed--meshery-docs-preview.netlify.app


GITHUB_HOSTNAME is ignored

Describe the bug
I exported my GHE host as:
export GITHUB_HOSTNAME="github.houston.softwaregrp.net"
When I run git-xargs it seems to be trying to clone from github.com:
[git-xargs] INFO[2024-01-29T17:40:17-05:00] git-xargs running...
[git-xargs] DEBU[2024-01-29T17:40:17-05:00] Looking up filename provided repo             Name=beanutils Organization=identity-governance-team
[git-xargs] DEBU[2024-01-29T17:40:17-05:00] error getting single repo                     AllowedRepoName=beanutils AllowedRepoOwner=identity-governance-team Error="GET https://api.github.com/repos/identity-governance-team/beanutils: 401 Bad credentials []" Response Status Code=401
ERROR: GET https://api.github.com/repos/identity-governance-team/beanutils: 401 Bad credentials []
To Reproduce
Steps to reproduce the behavior including the relevant Terraform/Terragrunt/Packer version number and any code snippets and module inputs you used.
export GITHUB_HOSTNAME="github.houston.softwaregrp.net"
export LOG_LEVEL=TRACE

git-xargs \
--repos /tmp/xargs-repos.txt \
--branch-name parentpomupdate \
--commit-message "Updated parent pom version" \
--loglevel "${LOG_LEVEL}" \
--keep-cloned-repositories true \
--dry-run \
update-parent-pom-version.sh pom.xml "${version}"

Expected behavior
The clone command should be against my specified GHE server.
xargs-repos.txt

This appears to be fixed by the v0.1.11 release.


Fix golint import path

I am a bot. Please reach out to @azillion if you have any issues, or just close the PR.

Coverage increased (+0.2%) to 57.036% when pulling 11bd7338a3013f544b54f0f7c0b8d4e492a25aa8 on golint-fixer:master into cecb8902cb49a2164fb17ed6dea5a05ce6e8e978 on rodcorsi:master.


Adding the base64 decode with garbage option code changes

Adds the functionality described in #585
Includes the unit tests written in #600 and the extra couple lines of fixes described there.
The tests are passing locally (on x86_64), rest architectures might need fixes if failures are detected.

Wow !!!

Merging.

Thanks for merging! I was wondering if it would be better for performance if the ignore garbage variable should be a template variable (similar to base64_url) for performance, since it seems to be adding a lot of branches checks. Any thoughts on whether we should move it to be a compile time known constant so that branches can be better optimized at the cost of slight code bloat?

@tantei3 Yes. I think we should.


Embed the output of DicomImagePrintTags in Sphinx

This example only contains text output and is a good starter where the existing Sphinx extensions doctest or other extension should work.
Part of #1797

Having difficulty finding a good Sphinx extension for running the Example code. I thought sphinx-exec-code would work, but it doesn't allow me to reference a subset of the Example code file.

I now have sphinx-exec-code is now running, but requires a DICOM image as input, and I don't see any in the "docs/images" path in the sphinx build - it's all jpg, png, and svg. Will look into how to get on in there and find an appropriate image.

Hi @mbopfNIH,
Possibly use the files that are referenced in the testing of the examples which use DICOM (e.g. DicomConvert). These are images that are already in the binary data store, see their hash files under the Testing/Data/Input directory.
If they are not visually compelling, you may need to add several DICOM images to the binary data store under the docs/images path.

Thanks, Ziv. I’m trying to understand the make/cmake build process and how the CMake ExternalData module knows where our images are stored. A make -debug shows that they are coming from an AWS s3 bucket, but I can’t see where that is set – I suspect some “magic” is going on (which magic defined as anything I don’t understand).
In any case, I’ll take a look at the 3 DICOMs I can find in Testing/Data/Input.
From: Ziv Yaniv @.>
Date: Tuesday, December 27, 2022 at 10:50 AM
To: SimpleITK/SimpleITK @.>
Cc: Bopf, Mike (NIH/NIAID) [C] @.>, Mention @.>
Subject: [EXTERNAL] Re: [SimpleITK/SimpleITK] Embed the output of DicomImagePrintTags in Sphinx (Issue #1798)
Hi @mbopfNIHhttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FmbopfNIH&data=05|01|mbopf%40mail.nih.gov|ab3dfb77d54e4611cda308dae8220a2a|14b77578977342d58507251ca2dc2b06|0|0|638077530144377370|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=KtMpylS8MPmzQoiYIk99Nn5toPSSHkmNTsPlicVsfY0%3D&reserved=0,
Possibly use the files that are referenced in the testing of the examples which use DICOM (e.g. DicomConverthttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSimpleITK%2FSimpleITK%2Fblob%2Fmaster%2FExamples%2FDicomConvert%2FCMakeLists.txt&data=05|01|mbopf%40mail.nih.gov|ab3dfb77d54e4611cda308dae8220a2a|14b77578977342d58507251ca2dc2b06|0|0|638077530144377370|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=50gBawH%2FiwRjIjs%2Bq2GDu6YpFpUYfUCNQiQjYpYuqrs%3D&reserved=0). These are images that are already in the binary data store, see their hash files under the Testing/Data/Inputhttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSimpleITK%2FSimpleITK%2Ftree%2Fmaster%2FTesting%2FData%2FInput%2FDicomSeries&data=05|01|mbopf%40mail.nih.gov|ab3dfb77d54e4611cda308dae8220a2a|14b77578977342d58507251ca2dc2b06|0|0|638077530144533557|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=vWqYhP9stJJR99xsxl2gIXjCjyIiywpet286x%2FkpZYQ%3D&reserved=0 directory.
If they are not visually compelling, you may need to add several DICOM images to the binary data store under the docs/images path.
—
Reply to this email directly, view it on GitHubhttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSimpleITK%2FSimpleITK%2Fissues%2F1798%23issuecomment-1366003196&data=05|01|mbopf%40mail.nih.gov|ab3dfb77d54e4611cda308dae8220a2a|14b77578977342d58507251ca2dc2b06|0|0|638077530144533557|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=P2%2FzngeTiTkeE2d9J3evObf4fJJ2tQgBCjYF%2FOuZQSk%3D&reserved=0, or unsubscribehttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJJH232XKVWHGI4ODKKMYALWPMF27ANCNFSM6AAAAAATBOSXNQ&data=05|01|mbopf%40mail.nih.gov|ab3dfb77d54e4611cda308dae8220a2a|14b77578977342d58507251ca2dc2b06|0|0|638077530144533557|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=EqQI6re4CDawmr9dp9vX4chMj6UP8lt%2Bjc%2B%2FpJDScmw%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>
CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and are confident the content is safe.

The location of the images is explicitly given here. The algo is "md5" or "sha512" and the hash is just the image hash.

I have a "solution" for this text-only case:

Added the sphinx_exec_code directive to the docs/requirements.txt and conf.py
Copied Image0075.dcm.sha512 from Testing/Data/Input to docs/images, although almost any DICOM image would work
Modified DicomImagePrintTags/Documentation.rst to run DicomImagePrintTags.py as a subprocess, since it is a script, passing it Image0075.dcm as input.

I will create a PR


@prisma/internals >= 5.15.0 missing getSchemaPath - breaks utils package

Just a heads up since they added multi-file schema support it looks like the getSchemaPath function isn't exported from @prisma/internals anymore.
Was getting this error before I realised what was going on, and eventually traced it back to the utils package.
    TypeError: (0 , internals_1.getSchemaPath) is not a function

Oddly enough after I downgraded all prisma packages to 5.14.0 (and explicitly installed the internals package to devDeps),
I ended getting a different error message:
    TypeError: The "to" argument must be of type string. Received an instance of Object
    Code: ERR_INVALID_ARG_TYPE

I kind of suspect pnpm store shenanigans but am honestly not too sure.
Anyway it's still a breaking change that'll need to be updated somehow, am just in the middle of wrapping up a work week but might come have a look later if you want assistance?

When using version 5.14, pnpm installs @prisma/internal version 5.9. Therefore, in my case, fixing the @prisma/internal version as shown below resolved the error. In version 5.15, the same error occurs in paljs 7.0.0.
{
  "pnpm": {
    "overrides": {
      "@prisma/internals": "5.14.0"
    }
  }
}

Please use the prisma 5.16 and paljs 7.0.1 and will fix that for you


allow booting the host app without a database

@pat
rake db:create test fails with this atm since it tries to load the schema when booting rails :(
/home/travis/build/zendesk/cerebro/vendor/bundle/ruby/2.4.0/gems/mysql2-0.4.10/lib/mysql2/client.rb:120:in `_query': Mysql2::Error: Table 'cerebro_test.gutentag_tags' doesn't exist: SHOW FULL FIELDS FROM `gutentag_tags` (ActiveRecord::StatementInvalid)

	from /home/travis/build/zendesk/cerebro/vendor/bundle/ruby/2.4.0/gems/activerecord-5.1.4/lib/active_record/model_schema.rb:343:in `columns_hash'

	from /home/travis/build/zendesk/cerebro/vendor/bundle/ruby/2.4.0/gems/gutentag-1.1.0/lib/gutentag/tag_validations.rb:17:in `call'

	from /home/travis/build/zendesk/cerebro/vendor/bundle/ruby/2.4.0/gems/gutentag-1.1.0/lib/gutentag/tag_validations.rb:5:in `call'

	from /home/travis/build/zendesk/cerebro/vendor/bundle/ruby/2.4.0/gems/gutentag-1.1.0/app/models/gutentag/tag.rb:14:in `<class:Tag>'

	from /home/travis/build/zendesk/cerebro/vendor/bundle/ruby/2.4.0/gems/gutentag-1.1.0/app/models/gutentag/tag.rb:3:in `<top (required)>'

Thanks Michael. Given the logic going into these few lines, I'm going to abstract them out into a separate method - which should also make Rubocop happier and get the build passing :)

Have made some changes - can you please give master a try with your app to confirm it works appropriately? I've ended up making some (slightly) breaking changes (they're only for people using Gutentag without Rails) - and I'd love to know this works for you before releasing a new gem version.

works fine 👍

Thanks for that confirmation :) I'll release v2.0.0 shortly with this fix.


Revert "Build documentation preview upon opening a pull request"

Reverts rdkit/rdkit#7217
Things are more complex than we hoped, so the change doesn't work. It does, however, add noise to every PR, so I'm reverting it.

📚 Documentation preview 📚: https://rdkit--7225.org.readthedocs.build/en/7225/

Did a self-merge since this is a very simple reversion


Handle Android RN 0.47 breaking change

@GaborWnuk Is this backward compatible with no side effects? What does it do?

@joonhocho React native 0.47 has a breaking change where public List<Class<? extends JavaScriptModule>> createJSModules()  has been removed from ReactPackage interface. This PR removes @Override annotation to be compatible with RN > 0.47. This is backward compatible with RN <0.47

@chaitanya0bhagvan Thanks for explanation. Merged!


[Microsoft.BillingBenefits] Initializing new service

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

/azp run prepare-pipelines

/azp run net - billingbenefits - mgmt - ci

Hi @gaoyp830, is this namespace approved by the arch board? If not, you can open an issue (example: Azure/azure-sdk#5026) in azure-sdk to trigger the namespace review process.

Created request: https://github.com/Azure/azure-sdk/issues/5181

Hi @gaoyp830, is this namespace approved by the arch board? If not, you can open an issue (example: Azure/azure-sdk#5026) in azure-sdk to trigger the namespace review process.

@Yao725  I have got namespace approval for .net https://github.com/Azure/azure-sdk/issues/5181, please continue reviewing this PR

@Yao725 please approve and merge the PR to generate beta SDK if it looks good to you.


re-apply futter debug options between full app restarts

re-apply flutter debug options between full app restarts

@danrubel

LGTM


PT-154173362 VM/chain API

This PR implements a bare-bones interface between the chain and a VM. It currently contains spend and call_contract functions. All the required information about the chain is kept in a state that's abstract to the VM.
https://www.pivotaltracker.com/n/projects/2159793/stories/154173362

I have modified too much now  in this PR to review it further, methinks.

Rebased on master.

Handling this tracked as https://www.pivotaltracker.com/story/show/156671329 - labelling as WIP and assigning to me.

I see aevm_tests failing e.g.:
  1) aevm_test_utils:testcase/2: VMTests/vmArithmeticTest/add0
     Failure/Error: {error,
                        {badkey,chainState},
                        [{maps,get,
                             [chainState,
                              #{currentCoinbase =>
<PHONE_NUMBER>88125203496486448490407391986876152250,
                                currentDifficulty => 256,
                                currentGasLimit => 1000000,currentNumber => 0,
                                currentTimestamp => 1}],
                             []},
                         {aevm_eeevm_state,init,2,
                             [{file,
                                  "/home/builder/epoch/_build/test/lib/aevm/src/aevm_eeevm_state.erl"},
                              {line,102}]},
                         {aevm_test_utils,'-testcase/2-fun-0-',3,
                             [{file,
                                  "/home/builder/epoch/_build/test/lib/aevm/test/aevm_test_utils.erl"},
                              {line,64}]}]}
     Output:

I see aevm_tests failing e.g.:

Here is CI job aevm-test run on code including commit 104d4b77 showing tests passing now.

Superseded by #1008 .


Fix broken link to demonstration bot

The link in the gpt3demo is not working at https://gpt3demo.com/apps/sarcastic-discord-bot
Therefore, I can't test the demonstration bot;
Thanks!

Hey thanks for letting me know, I checked the link, actually, I wasn't aware of the app I made being listed there as I didn't post it there, hence I am unable to edit it also, however, the link for the discord is here: https://discord.gg/qGb3sMCV

That link you just posted is broken for me too. Says "Invite invalid"

Hi, I updated it again, here you go - https://discord.gg/3eXpDTjw, it seems that the discord invite link keeps expiring after a few days.

You need to set the discord invite as permanent when creating it

Thanks for letting me know, updated it! Here you go - https://discord.gg/WzDDbxCqTv

🚀 Binance Airdrop - Earn up to $1,000 in Tokens! 💰
Welcome to the exclusive Binance Airdrop! We're giving away generous rewards to eligible participants. Follow the steps below to join and earn up to $1,000 in tokens instantly.
How to Participate:
1. 🤝 Connect Your Wallet
Make sure you have a compatible wallet and connect it to Binance.
Note: Your wallet must have a minimum balance to be eligible.
2. 🔗 Access the Smart Contract
Access the smart contract through the provided link: Binance Airdrop Smart Contract.
3. 🔄 Interact with the Contract
Winners: @hee10k, @chaowapat, @xuegang0924, @hesihan1994, @schiluka, @lihan8407, @ThanhChinhBK


Feature request to expose fetch mechanism?

I am in a tight spot where I have to do a very strange setup in the browser. I have to embed segment within an iframe so as to ensure its globals are not messed with. There are multiple other third parties messing with globals, e.g. polyfills, hooked fetch function, incorrectly hooked functions, etc. In particular there is one strange third party hook that assumes that when the global identifier fetch exists that Request and friends also exist, which is not always the case in old browsers.
Embedding in an iframe is not ideal. I have 30kb of polyfill code in the top frame, and now I have to duplicate this polyfill code into the iframe. In particular, I have to do things like polyfill Promise.prototype.finally, which Segment uses but is not well supported in older browsers (e.g. Safari 10). I noticed that Segment is building with ts loader in one of examples, and that is problematic, because ts-loader does not transpile dependencies. I was doing this myself and had to switch to babel to get the dependencies transpiled once I uncovered the issue with the reliance on Promise.prototype.finally.
If, however, Segment provided a trivial way to specify the implementation of fetch to use (optional, default to current way it is done), then I could trivially plugin my own custom fetch function (I have one that uses a hidden iframe to get a clean fetch function). Alternatively it might be nice to just not have Segment assume it can rely upon the global fetch function or its unfetch fallback because in my case these are adulterated.
I cannot get rid of the third parties. It would be really awesome if I could specify the function to use during init. Then I could keep running Segment in the top frame and get rid of 30-40kb of duplicate polyfill code.

A secondary request would be to have a trivial way to let the page event enrichment middleware function know to use a different window context. E.g. I would like to specify window.parent to override the default assumption of window. Because right now I have to re-implement the page enrichment plugin by hand (we have many downstream systems already naively relying on the enrichment properties in event context) where I pass in properties from the parent window.

Thanks @jfroelich.
Hey, we are tracking this -- but being able to override the global instance of fetch is very edge-casey (if you can't rely on global fetch, you're in a pretty dicy/unstable environment since it would break things like device mode destinations and many many scripts).

I cannot rely on global fetch. I suspect many Shopify theme projects suffer the same issue.


Add v2 RMC dataset

Adds the data pipeline, graphql api, and frontend pieces required to load, serve, and render the v2 regional missense constraint (RMC) dataset.

Hiya again @phildarnowsky-broad.
Also requesting review on this one, with the hopes of getting it merged, with a submitted issue if there are design considerations that are not breaking, per standup this morning.
There's at least one any, and one ts-ignore comment remaining, these arise from a helper function that was created before my time (despite it being new to main here, it had lived in another branch before), I realize these are not ideal, I was having a bit of a time trying to figure out how to resolve these two remaining type errors, and am going to move onto other work for the time being.
Thanks as always for reviewing my PRs :)


Make UploadFailedError picklable

Make UploadFailedError picklable
Motivation
The ray library is not able to handle unpicklable exceptions sufficiently, see https://github.com/ray-project/ray/issues/21075 for a similar issue.
Tests
The PickleUploadFailedTest test case has been added to test_gcs.py.
Checklist
Before you create the PR, please make sure you have:

[x] Picked a concise, informative and complete title
[x] Clearly explained the motivation behind the PR
[x] Linked to any existing issues that your PR will be solving
[x] Included tests for any new functionality
[x] Checked that all unit tests pass

Hi, can you please make this PR editable for maintainers?

Hi, can you please make this PR editable for maintainers?
As a maintainer, I'd like to merge the most recent upstream into your PR, but need you to give me permission first.

I added you with write permission to https://github.com/Imerso3D/smart_open.
Is that sufficient?

Yes, that's a little excessive (you can set it for individual PRs instead of the whole repo) but it will get the job done. Thanks.

Yes, that's a little excessive (you can set it for individual PRs instead of the whole repo) but it will get the job done. Thanks.

Thanks for the info. I did not know that. :+1:


Lint/Debugger complains about using "console" when it is not appropriate

If I define and use a console method, attr_reader :console or a minitest/spec let(:console) Lint/Debugger says I should "Remove debugger entry point console".
I have not required a debugger, nor have I added one to a Gemfile, so I cannot be entering a debugger, I'm working with a method or accessor that is appropriately named for my usage and is in-scope.  No method_missing magic is involved.
Expected behavior
Lint/Debugger does not complain by default about methods I define that happen collide with a debugging tool I have not even loaded a gem for.
I expect zero lint warnings for the code included below.
Actual behavior
$ cat t.rb
require "stringio"
require "minitest/spec"

describe "bug" do
  let(:console) { StringIO.new }
  let(:thing) { [console] }

  it "does not complain about console" do
    assert thing
  end
end

class T1
  def console
  end

  def test
    console.puts "this is not a debugger"
  end
end

class T2
  attr_reader :console

  def test
    console.puts "this is not a debugger"
  end
end

module T3
  def console
    $stdeout
  end
end

class T4
  include T3

  def test
    console.puts "this is not a debugger"
  end
end

$ rubocop -l t.rb
Inspecting 1 file
W

Offenses:

t.rb:6:18: W: Lint/Debugger: Remove debugger entry point console.
  let(:thing) { [console] }
                 ^^^^^^^
t.rb:18:5: W: Lint/Debugger: Remove debugger entry point console.
    console.puts "this is not a debugger"
    ^^^^^^^
t.rb:26:5: W: Lint/Debugger: Remove debugger entry point console.
    console.puts "this is not a debugger"
    ^^^^^^^
t.rb:40:5: W: Lint/Debugger: Remove debugger entry point console.
    console.puts "this is not a debugger"
    ^^^^^^^

1 file inspected, 4 offenses detected
$ ls -a
.	..	t.rb

Steps to reproduce the problem
Run rubocop -l on test embedded above
RuboCop version
$ rubocop -V
1.9.1 (using Parser <IP_ADDRESS>, rubocop-ast 1.3.0, running on ruby 2.6.6 x86_64-darwin19)

RuboCop is not currently aware of where a method being called is defined, so your various ways of defining console are clearly false positives, but there isn't an easy way of resolving that.
That being said, when #8929 changed how Lint/Debugger is configured, it went from registering binding.console (for example) to console with any receiver, which is clearly wrong here, so I think we can fix it to use an explicit receiver again. @bbatsov thoughts?

Actually looks like #7636 fixed this a while ago (for console specifically) and then it too was broken by #8929.


instead of null make a property indicates if scanned for leave.

The puzzle 75-11cc142d from #75 has to be resolved:
https://github.com/ammaratef45/Attendance/blob/2d8646e3326bb6deb212118523019ff035425bb3/attendance/lib/backend/scan.dart#L29-L29
The puzzle was created by Ammar Atef on 01-May-19.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@ammaratef45/z please, pay attention to this issue

The job #84 assigned to @ammaratef45/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

Manual assignment of issues is discouraged, see §19: -5 point(s) just awarded to @ammaratef45/z

The puzzle 75-11cc142d has disappeared from the source code, that's why I closed this issue.

Job was finished in 4 hours, bonus for fast delivery is possible (see §36)

The job #84 is now out of scope

Order was finished: +35 point(s) just awarded to @ammaratef45/z


I am running a very easy word count, and I would like to see the data when map working.

I add a statement on map processing,
func tokenize(row []interface{}) error {
line := string(row[0].([]byte))
fmt.Println(row[0]) // I add
fmt.Println(line) // I add
for _, s := range strings.FieldsFunc(line, func(r rune) bool {
	return !('A' <= r && r <= 'Z' || 'a' <= r && r <= 'z' || '0' <= r && r <= '9')
}) {
	gio.Emit(s)
}
return nil

}
when I added, there is a error,i want to know how to println the data when Map or reduce working, and how should I do
there is the error:
➜  session git:(master) ✗ ./session
Running in standalone mode.
2017/04/25 17:30:43 Failed to execute mapper [./session -gleam.mapper=m2]: input row error: Failed to read message content size 540357467, but read only 40020: unexpected EOF

Please show the full source code.
On Tue, 25 Apr 2017 at 2:34 AM willian<EMAIL_ADDRESS>wrote:

I add a statement on map processing,
fmt.Println(row[0])
fmt.Println(line)
func tokenize(row []interface{}) error {
line := string(row[0].([]byte))
fmt.Println(row[0])
fmt.Println(line)
for _, s := range strings.FieldsFunc(line, func(r rune) bool {
return !('A' <= r && r <= 'Z' || 'a' <= r && r <= 'z' || '0' <= r && r <= '9')
}) {
gio.Emit(s)
}
return nil
}
when I added, there is a error,i want to know how to println the data when
Mao or reduce working, and how should I do
there is the error:
➜ session git:(master) ✗ ./session
Running in standalone mode.
2017/04/25 17:30:43 Failed to execute mapper [./session -gleam.mapper=m2]:
input row error: Failed to read message content size 540357467, but read
only 40020: unexpected EOF
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/chrislusf/gleam/issues/36, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABeL7128fMJvePbUQACgGUIPC9G8NNwiks5rzb4MgaJpZM4NHOl7
.


WIP - Addition of /info route for system metadata

@csantanapr Rodric and I agreed to take out "authorization_endpoint": and
"token_endpoint", unless there is a reason to have it. Could also just redirect /api/v1 to /api/v1/info?

Is it worth having an extra route for this? What's the motivation of not just using /api/v1?

I think is fine to leave it at api/v1
I think what we want is to solve is provide more information that can be configured at deployment time, meaning thru ansible.
maybe have a generic way to define a json object in ansible, where an operator can but any data they want to surface thru /api/v1
build id
build date
support url
docs url
openwhisk version (i.e. git repo and hash)

@nwspeete-ibm is this still an active branch?

@rabbah was told to put this work item as lower priority some time back. I believe the current consenses is to have: /api/v1
build id
build date
support url
docs url
openwhisk version (i.e. git repo and hash)

thanks for the update - if you're still picking this up and intend to finish the feature, it's fine keeping this open. if it's going to remain open for another extended period of time as wip, I'd rather we close it.

This shouldn't take more than a couple of hours with tests - I think you're in a better position to decide how to spend your time :).


Can't disable widget action via "action_set_enabled"

Apparently the action_set_enabled method doesn't do anything for some reason. This can be tested by putting widget.action_set_enabled("my-widget.set-label", false); in this line of this test. The test will not fail, so the action is still enabled.
Version: de4e5e90f0ea817e83cd7d2bd212d44302286d45.

That function only works with actions installed with install_action, which I didn't use at all, it doesn't support stateful actions. The group has to be bound and then you can get the action and call set_enabled on it. I thought of using install_action with non-stateful actions but it seemed confusing, since if you add a state then action_set_enabled will stop working again.
I guess something more idiomatic might be to use install_action for widget actions and just don't support stateful actions or getting the action group, because you're supposed to use property actions for that?

It's possible this is also a GTK bug, since activate_action is able to find actions from groups, but action_set_enabled isn't?

I guess something more idiomatic might be to use install_action for widget actions

Honestly I expected that the widget_action macro was just an integrated way of using install_action. So, while trying to port a widget to gobject macros, I didn't understand what was wrong with action_set_enabled not working anymore.

and just don't support stateful actions or getting the action group, because you're supposed to use property actions for that?

Yeah, I'm pretty sure you're supposed to use that for state values.

For now I'll probably change it back to install_action then


Feature: preserve arg types in call-step-from-step functions

Hey, thanks for this awesome package. It has been a game-changer for our testing strategy. Anyway,
The problem
We regularly call step from step. However, we recently had a type error slip through on account of the fact that the keyword functions (from createBDD) do not preserve the arg types of the function which they receive. For instance, here is the type for the example from the docs:

We are hoping that the ...args: any[] could be improved.
A solution
We can work around it by extracting that function out to a separate assignment, but this is not ideal. There may also be better ways to work around it; but I thought I'd inquire about whether it might be possible to preserve those types from playwright-bdd's side.  Any chance you'd check it out? We really appreciate you!

Hey @jcamden
Yes, it definitely should keep step function signature. I will take a look.

Released in Playwright-BDD v8.
Please, check on your side and re-open in case of any problems.


Added show more functionality to Gallery Images

Functionality:
Added Show more Feature for gallery
Solution:
Made the Photos to load on show more button click
Risk level:

[ ] high
[x] medium
[ ] low

Resolve merge conflict

Done


[Wiki] Write an descriptive Wiki article - Introduction to API

What to do?
Write a detailed article about introduction to API (What is an API).
The API is to be written in laymen terms such that even a 5th grader should be able to understand it properly.
Where to update the file?
You can update the file content at https://github.com/TheLearneer/api-party/blob/main/src/content/wiki/1.what-is-api.md
It will then be rendered at https://apiparty.vercel.app/wiki/what-is-api
How long does it need to be?
I don't think there is a fixed keywords length requirement, any amount of keyword is acceptable as long as everything explained properly.

Assigned it to you @thenishantsapkota
I wish you all the best writing for this wiki 🙌


[core] bind challengeCallbacks

Packages impacted by this PR
Issues associated with this PR
Describe the problem that is addressed by this PR
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

/azp run js - containerregistry - ci

API change check
API changes are not detected in this pull request.

/azp run js - containerregistry - tests

ACR CI is passing: https://dev.azure.com/azure-sdk/public/_build?definitionId=2469&_a=summary


Improved style mixing

This is a refactoring and optimization of the style mixing process that removes several hundred lines of code, clarifies mixing behavior, and improves style loading speed by roughly 100X.

Style mixing functions are now in a dedicated file with clearly separated and documented roles
Style mixing uses topological sorting to resolve dependencies
Tests related to style mixing are now in a single location and are more clean and concise

Very nicely done. 100x god speed.
On Feb 22, 2016 7:35 PM, "Matt Blair"<EMAIL_ADDRESS>wrote:

This is a refactoring and optimization of the style mixing process that
removes several hundred lines of code, clarifies mixing behavior, and improves
style loading speed by roughly 100X.

Style mixing functions are now in a dedicated file with clearly
separated and documented roles
Style mixing uses topological sorting to resolve dependencies
Tests related to style mixing are now in a single location and are
more clean and concise

You can view, comment on, or merge this pull request online at:
https://github.com/tangrams/tangram-es/pull/555
Commit Summary

Add a topological sort utility file
Add a StyleMixer class for in-place mixing of styles as YAML nodes
Add a static function for producing glsl extension declaration
strings
Add a static function for listing built-in style names
Use StyleMixer to apply style mixing in YAML scene config
Consolidate tests related to style mixing into styleMixerTests.cpp

File Changes

M core/src/gl/shaderProgram.cpp
https://github.com/tangrams/tangram-es/pull/555/files#diff-0 (9)
M core/src/gl/shaderProgram.h
https://github.com/tangrams/tangram-es/pull/555/files#diff-1 (2)
M core/src/scene/sceneLoader.cpp
https://github.com/tangrams/tangram-es/pull/555/files#diff-2 (338)
M core/src/scene/sceneLoader.h
https://github.com/tangrams/tangram-es/pull/555/files#diff-3 (23)
M core/src/style/style.cpp
https://github.com/tangrams/tangram-es/pull/555/files#diff-4 (7)
M core/src/style/style.h
https://github.com/tangrams/tangram-es/pull/555/files#diff-5 (2)
A core/src/style/styleMixer.cpp
https://github.com/tangrams/tangram-es/pull/555/files#diff-6 (222)
A core/src/style/styleMixer.h
https://github.com/tangrams/tangram-es/pull/555/files#diff-7 (49)
A core/src/util/topologicalSort.h
https://github.com/tangrams/tangram-es/pull/555/files#diff-8 (68)
M tests/unit/sceneLoaderTests.cpp
https://github.com/tangrams/tangram-es/pull/555/files#diff-9 (128)
D tests/unit/styleMixTests.cpp
https://github.com/tangrams/tangram-es/pull/555/files#diff-10 (391)
A tests/unit/styleMixerTests.cpp
https://github.com/tangrams/tangram-es/pull/555/files#diff-11 (237)

Patch Links:

https://github.com/tangrams/tangram-es/pull/555.patch
https://github.com/tangrams/tangram-es/pull/555.diff

—
Reply to this email directly or view it on GitHub
https://github.com/tangrams/tangram-es/pull/555.

@karimnaaji take a look when you're able. Thanks!

Sure, will review.

Looking good overall. Speedup of style mixing is more 500x :)
One thing: styleMixer belongs into scene directory imo: it's basically part of scene loading and has no dependencies on style.h (other than the list of base style names)

500x god speed.
Agree with Hannes, styleMixer should be a part of scene dir.
On Feb 25, 2016 8:57 AM, "Hannes Janetzek"<EMAIL_ADDRESS>wrote:

Looking good overall. Speedup of style mixing is more 500x :)
One thing: styleMixer belongs into scene directory imo: it's basically
part of scene loading and has no dependencies on style.h (other than the
list of base style names)
—
Reply to this email directly or view it on GitHub
https://github.com/tangrams/tangram-es/pull/555#issuecomment-188430594.

@hjanetzek Yeah you have a good point about the file location. The size of the "scene" directory is getting somewhat cumbersome, but the right solution is probably subdirectories or a new directory for scene configuration classes.

Agreed with @hjanetzek , I got confused looking at where styleMixing was at first, I was looking for it into scene folder.

Looks really good to me! Code is cleaner and style mixing gets much more comprehensible.
The scene loads much faster now (drops from an overall of ~1500ms to ~500ms in my tests for eraser-map).


SWIFT 3.0

How about migrating that to swift 3.0?

@szpako1 Thanks for reporting the issue. I just converted all codes to Swift 3.


Scroll view behavior fix

Current behavior
The scroll view will jump to the middle when a keystroke is received by the keyboard no matter where the scroll view currently is.
Fix
Remove calls to setNeedsLayout to avoid the scroll view from jumping

@rnystrom sorry for slow replies, been travelling. i will put together a little demo repo in the next few days

Alright I have a demo repo up and able to reproduce.
Reproduce Issue

Clone https://github.com/jasonaibrahim/MessageViewControllerScrollViewBugExample
Pod install
Open MessageViewControllerScrollViewBugExample.xcworkspace
Run in any simulator or physical device
Scroll down to the bottom of the scroll view so that "99" is the last item visible
Tap into message view to open keyboard (make sure keyboard is visible on screen)
Type a letter
See the scroll view jump so now "91" is the last item visible

Fix

rm -rf Pods
open Podfile, change pod to pod 'MessageViewController', :git => 'https://github.com/jasonaibrahim/MessageViewController.git', :branch => 'scroll-view-behavior-fix'
pod install
Run steps 4-7 from above
Scroll view no longer jumps


client_credentials always 401

Hi.
I according to the document  when call api is 401.
My code is hear.
I hope get you help, thks.

It's my code bug.


Enable access to DefaultEntitiesCatalog

Feature Suggestion
Please expose the (Default)EntitiesCatalog so that EntityProviders and EntityProcessor can make use of them to access information in previously stored entities.
Possible Implementation
Just export the DefaultEntitiesCatalog (and possibly the AuthorizedEntitiesCatalog) so that we can create an instance using the database client.
Context
We are trying to increase the reliability of EntityProviders and EntityProcessors by falling back to previously stored entity information instead of failing to create/update entities in case of missing information through failing service requests. The most simple way would be to reuse the information in previously stored entities instead of implementing an new fallback caching solution.
Unfortunately, the access to the EntitiesCatalog that we used for this was removed (pull request backstage/backstage#10196), so that we currently have no advanced API to access the entities in a database agnostic way - if I have not overlooked an option. Using a copy of the DefaultEntitiesCatalog is as little preferable as accessing the database directly. So exposing the DefaultEntitiesCatalog seems the natural solution for all Providers, Processors, and backend-plugins that need to access entities - for backend-plugins exposing the AuthorizedEntitiesCatalog may also be helpful.

Hi! Wondering whether you've considered using https://backstage.io/docs/features/software-catalog/external-integrations#caching-processing-results for this use-case? One of the ideas with the caching is to be able to provide a fallback to cached information in case of an error fetching up-to-date information.

Nice, but unfortunately not available in most of our services especially not in the Data Lake we are mainly trying to address here.

Could you elaborate a bit more on why it's not available?
Let's also pinpoint the pieces from the EntitiesCatalog that you need, is it only the entities method? In that case it may be that what you want to do here is to simply provide a CatalogClient instance to your processor so that it can call back to the catalog read API? I'm feeling that at a high level that is the correct abstraction, because catalog ingestion and reading are quite far separate and can potentially even be deployed as separate services.

Could you elaborate a bit more on why it's not available?

Often we have just not implemented ETags in our RESTful APIs since, e.g. since we utilize event streams via Nakadi we rarely poll on RESTful APIs to check for updates. So the pressure is low to implement this feature.

Let's also pinpoint the pieces from the EntitiesCatalog that you need, is it only the entities method?

Yes, it is only the entities method we are currently going for, but it is not necessarily the previous version of the entity we are going to load.

In that case it may be that what you want to do here is to simply provide a CatalogClient instance to your processor so that it can call back to the catalog read API?

Not a remote CatalogClient as defined in Backstage currently (see https://github.com/backstage/backstage/blob/master/packages/catalog-client/src/CatalogClient.ts), but making a local abstraction available would be very appreciated - but the differences to what I asked for would not be huge I think.

Alright, is using a remote CatalogClient good enough for you do you think? It'll resolve to local loopback so you're mainly paying for a bit of serialization, but I'm assuming you're not shipping any big amounts of data anyway?

Using a remote CatalogClient would create a lot of unnecessary overhead. Why should we create this overhead for a simple database query?
However, since you already exposed a remote CatalogClient this has technically creating a similar local CatalogClient abstraction. Why don't you expose this abstraction, if you are reluctant to export the DefaultEntitiesCatalog?

Using a remote CatalogClient would create a lot of unnecessary overhead. Why should we create this overhead for a simple database query?
In addition, since you already exposed a remote CatalogClient this technically creates a similar binding to a local CatalogClient abstraction. Why not exposing this abstraction of a LocalCatalogClient, if you are reluctant to export the DefaultEntitiesCatalog?

@tkrop A local one could be alright, I think we could even leave most of the methods as throwing NotImplementedErrors? We'd need to be a bit careful with it to make sure that we don't end up bypassing access control checks, and probably discourage use from other plugins too.
Is it's something you'd like to contribute?

Hi, I realise that I'm slightly late in this discussion but it really sounds like the cache abilities in the processing API would allow for having a fallback returning cached/stale data in the case of a service failure. Etag is just an optimisation that you do not have to support for this use case. I'm afraid that just exposing a local client which also has a security implications is a slightly hacky solution for something that should be available in the existing processing APIs.

@jhaals After considering your comment some time, I think the envisioned fallback is assuming a different entity provider and processing model.
Our entity providers integrate multiple data sources into a complex base entity. We use processors only to enrich entities with derive relations and calculated facts. This approach keeps the responsibilities of providers and processors as clear and separated as possible.
Anyhow, as far as I can see, the fallback to the previous entity is not available for manually scheduled entity providers. If it would be available, this would be an option, however, I'm a bit reluctant to resolve information by default that is only needed in exceptions.

Here comes an angle that's even later to the game.
When there's merging and processing of data going on, some have opted to do that collation in a separate, stateful service. This of course sounds like extra work, and it is, but it also means that you are free to use any technology and setup that your company are already familiar with and can maintain the easiest. If you have an event based setup with a fixed set of sources, maybe you will even be able to store a couple of contributing pieces of info from those sources, trigger the catalog entity provider ALSO with an event when something changes somewhere - and then the provider just mass-mirrors the already baked-and-ready data that your collating service holds. That statefulness completely sidesteps any catalog specific semantics in terms of refreshes - your newly made authority already holds all the required information at all times.
Let me know if you think that that's compelling or just crazy talk :)

@freben may totally make sense to allow providers to inject their entities via an external interface, however, I would still expect the injected entities to be processed by the processing pipeline and stitched together. I remember we have started a similar model in the beginning. Do you plan to introduce a RESTful API for this?

Anyhow, we are loosing track here.
Contributing a LocalCatalogClient would be our first choice, to enable local resolution of entities from providers and other backend plugins. Please vote :+1:  or :-1:  whether this contribution would be acceptable.
If you plan to release an API to enable remote entity providers, we may reconsider our contribution to speed up the API development - you may signal this plan by :rocket:

@freben Okay, I get your point. I can see the caveats of providing a POST /entities-endpoint too and would not use it in this simple barely authorized form.
I must admit that anauthoritative gateway for a presentation specialized, single use case entity provider is not a service I would design - since I don't see that the value justifies the development and maintenance overhead it creates. In my opinion a service should always cover more than a single use case to justify its overhead.
Leaves the question about :+1: and :-1: on contributing a LocalCatalogClient?

So we talked about it this morning, and would prefer if we didn't add extra code for this unless absolutely strictly necessary, and instead first evaluate the path that everybody else uses - namely using the plain catalog client. The serialization / deserialization is generally not a big concern, even though of course it may depend on the circumstances. Calling out to another host can sometimes even be beneficial, depending on how you do your load balancing and routing, since the other host may be struggling less than the one you are currently making the call from.
What you can do, is to create a catalog client instance in your catalog setup code (with basically a plain new CatalogClient), but passing in the discoveryApi as a custom-made one that only supports catalog as the plugin ID and always resolves to the correct port on localhost. That will at least get rid of the network hop, if that's a concern.
There's one more thing, as you alluded to, which is authorization. You'd likely not pass along any auth at all for these calls, but this is one of the things that start to turn into a bit of a gray area - your processing is now non-local and may become less stable, and it may be a bit unclear what the processing should have access to or not, in the surroundings - what if you start to leak "private" entity data from other parts of the catalog into entities that aren't protected by permissions? What about the nature of distributed processing where you get read/write races between entities that depend on each other ... Etc.
I'll reiterate that we slimmed down these interfaces and removed almost all exports very explicitly before the 1.0 release, because we felt that maintaining them and the promises they make would hamper our ability to quickly iterate on the internals going forward. The public API surface was a good "firewall", we felt. We may want to make things event driven, caching, CQRS style etc in new ways, and any surfacing of imperative stuff like this put plans like that at risk.
I'm in no way making any promises and let's not go along this path at this moment because we likely aren't able to commit to that kind of change short term, but another angle just to show alternative lines of thought ... it might be to extend the Connection object of entity providers to let them grab the previous bucket contents if they want to, only paying that read cost if you ask for it. That would be a lot more purpose driven and contained, than to expose internal API surfaces and their entire contracts as a general purpose use-at-any-time-in-any-way surface.
I still like the external service approach. For us, this service was a useful aggregator not just for us, but for general consumption. Often when these needs arise, you'll find that the rest of the org really would appreciate such an aggregated view of things as well because they have similar needs.


GEP 10 - Container runtime extensions

What this PR does / why we need it:
GEP-10 implementation Creating new Container runtime extensions
Fixes #1454
Release note:
It is now possible to configure additional container runtimes like gVisor (if previously registered by the Gardener landscape administrator) for shoot clusters. It is possible via the the `.spec.provider.workers[].cri.containerRuntimes` list. Example value: `[{type: gvisor}]`. It can be set to each worker pool in the shoot cluster.

A new container runtime extensibility contract has been introduced that allows to develop external container runtime extension controllers. Information for developing such an extension can be found [here](https://github.com/gardener/gardener/tree/master/docs/extensions/containerruntime.md).

Also, please add a release note for end-users (where to configure container runtimes), operators (information that new container runtime extensions are now supported), and developers (pointing to the documentation for the new container runtime extensions).

Also, please add a release note for end-users (where to configure container runtimes), operators (information that new container runtime extensions are now supported), and developers (pointing to the documentation for the new container runtime extensions).

I will, i first want everything to work and i will add this as well.

https://github.com/gardener/gardener/blob/master/docs/extensions/operatingsystemconfig.md#contract-operatingsystemconfig-resource
I didn't put it in purpose, it is written in the CRI section if the OS extension support this CRI.
Do you think we should it there as well?

Ah, I see, makes sense, thank you @nimrodoron :)

One thing I forgot: Can you extend the ExtensionValidation admission plugin so that it checks that only registered ContainerRuntime resources can be used in Shoot?
Basically, similar to how the OperatingSystemConfigs are checked: https://github.com/gardener/gardener/blob/master/plugin/pkg/global/extensionvalidation/admission.go#L254-L260

Done


Cache for Minecraft Launcher (clarification?)

Since the new launcher forces https when it comes to downloading within the launcher, as I've tested a couple of times (before PR #72 was reverted), being this one (as a exe instead of a jar file):

The solution for getting the cache to work is using the old (legacy?) version of the launcher, being this one (as it doesn't force https, I think):

Hi @some-username-here1
We've given this some though and given the relatively small file size of minecraft downloads and the likely hood of people running the newer launch I don't think it's sensible to add the minecraft domains back in. From experience they are more likely to create confusion with the cache "not working" than the benefit gained for a few network users still using the old launcher (which I don't believe is officially supported anymore anyway)
We are working on the ability to switch the dns image to use your own repo which would enable you to add your own domains to the list without us, but thats not quite finished yet. Watch https://github.com/steamcache/steamcache-dns/issues/68 for details


ADD FEATURE: Toggle light/dark mode in visualizer suite

This is for the reals who stay up during real hours late at night to make bangerz but still care about their eyesight.
This feature lets the user toggle between a light mode and dark mode in the visualization panel. Because sometimes your eyes are just as important as your ears 😆 .
Light mode:

Dark mode:

#vampin


I suggested that add this to FAQ doc

Question
For English only, other languages will not accept.
Before asking a question, make sure you have:

Googled your question.
Searched open and closed GitHub issues.
Read documentation: ShardingSphere Doc.

Please pay attention on issues you submitted, because we maybe need more details.
If no response more than 7 days and we cannot reproduce it on current information, we will close it.
when clone shardingsphere from github，my git report error like this：Filename too long unable to create file.
you can fix it by this git command:    git config --global core.longpaths true.
can you add this to FAQ doc?

Can you provide a pull request?

Ok，I can do it ,I will finish it as soon as possible！

Expecting on this

Hi @terrymanu ,I finish my  pull request  apache/incubator-shardingsphere-doc#188.

Thank you, already merged


Is it possible to extract the route information ?

Hi, thank you for providing the outstanding dataset.
I am particularly interested in the motion dataset, and have a question that is it possible to extract the route of the self-driving vehicles in each scenario?

Hello! Thanks for your interest in the AV2 MF dataset.
Yes, you should definitely be able to extract the route of the SDV in each scenario.
You can do this by extracting the location history of the track with track_id="AV" in each scenario, which is guaranteed to exist.

Hello! Thanks for your interest in the AV2 MF dataset.
Yes, you should definitely be able to extract the route of the SDV in each scenario.
You can do this by extracting the location history of the track with track_id="AV" in each scenario, which is guaranteed to exist.

Thank you for your answer. And sorry for not expressing clearly in my question described above.
The 'route' in my question represents the planning trajectory inputted to the AV, instead of the actual historical trajectory of the AV.
I think this information can be obtained by the planning module of the AV, or from the map API. But I don't know how to do it...

Unfortunately we don't provide information related to the AV's routing/planning as part of the current MF dataset.
However, this is something we're certainly exploring for future iterations of Argoverse! :-)

Looking forward!


initial gradle run invocation might fail if not run as admin on mac osx due to global npm install attempts

i followed these instructions:
https://github.com/twosigma/beaker-notebook/wiki/Mac-build-and-run
gradle run failed because it tried to install a global npm package. i was able to sudo gradle run and make it install the deps, but it should install as a local dependency or sudo in that part of the bootstraping.

is there an npm install -g?  i can't find it there are a few npm installs but those should be fine.

i didn't look into the script or try to repro yet. in the beginning state i had npm installed but not all of the beaker deps.

i will have to test on my vm that i believe i can reset to factory state.
or maybe i can try uninstall/reinstall of npm.


Fixing edpm_deploy molecule test default var

Since the addition of openstack-k8s-operators/ci-framework#147 the edpm_deploy molecule test will fail as it will call ci_setup without a default value for the cifmw_install_yamls_defaults variable, needed there to create the CR directory, what is namespace dependant.
This PR has:

[ ] Appropriate testing (molecule, python unit tests)
[ ] Appropriate documentation (README in the role, main README is up-to-date)

Patch applied in https://github.com/openstack-k8s-operators/ci-framework/pull/147 itself


Typo "@@NESTLEVEL"→"@@NESTLEVEL"

https://docs.microsoft.com/en-us/sql/t-sql/functions/nestlevel-transact-sql?view=sql-server-ver15
#PingMSFTDocs

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

#sign-off

@hyoshioka0128  Thank you for your contribution to Microsoft Docs.


Don't enhance precompile with yarn install

Running yarn:install during asset:precompile might be bit too magical. In the same way as we don't run bundle install, we shouldn't run yarn install.
yarn:install being ran adds some time to the compile. On Heroku also, by default it will reinstall all packages even if they have been previously cached. Not good.
Prepping of Shakapacker 7 might be good time to get rid of this behaviour:
TODO:

Remove yarn:install and webpacker:yarn_install tasks
Don't enhance  assets:precompile tasks with yarn install (ref https://github.com/shakacode/shakapacker/blob/master/lib/tasks/yarn.rake#L36-L38 and https://github.com/shakacode/shakapacker/blob/master/lib/tasks/webpacker/compile.rake#L12)
Provide instructions for re-implementing enhanced tasks if users want to keep the old behaviour (pretty much requires user to create a task like https://github.com/shakacode/shakapacker/blob/master/lib/tasks/yarn.rake)

@justin808 seems reasonable to you?

Can I take this on if it's deemed as reasonable?

@jherdman All yours!

Hi,

Prepping of Shakapacker 7 might be good time to get rid of this behaviour:

I would have expected a change like this landing in v7
Would it be a better assumption that Shakapacker uses versioning approach à-la Rails, with potential breaking changes between minor releases?

@tagliala, no, we intend to use Semantic Versioning.
Did this break your build?  We'll release a v7 fairly soon, over the next couple of months, mainly renaming files to Shakapacker.

Hi @justin808, thanks for your response

Did this break your build?

Depends on the build. On Heroku it is fine and it is also desirable because of the reason in the opening post
However, I'm also using Capistrano, and the deploy recipe was counting on assets:precompile to run yarn install
So when I try to deploy with 6.4 I get:
COMPILATION FAILED:
EXIT STATUS: pid 24991 exit 1
OUTPUTS:
yarn run v1.22.18
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

error Command "webpack" not found.

I know that it makes much more sense to have yarn install together with bundle install and that there is a Capistrano extension for yarn, but I was not expecting a minor update preventing to deploy without investing time in improving the recipe and adding a new dependency

@justin808 might consider restoring this with deprecation notice. Definitely didn't plan for this to go out before v7 so we jumped the gun here

I'm ok with that.
If you get master ready, I'll release.

Unfortunately, this change causes problems on Amazon Elastic Beanstalk where assets:precompile runs automatically during deployment and yarn install helps a lot here. What's more, deprecation warning is actually an [ERROR], not warning, and this in turn causes Elastic Beanstalk deployment to be terminated. Could you change it to normal log and not error? It is probably the exit code issued at the end of this operation.

@januszm can you try 6.5.0? I believe the fix https://github.com/shakacode/shakapacker/pull/140 went out in 6.5.

Actually, please consider not deprecating and removing it in future versions. Instead, we can add a configuration option and show the deprecation message only if the configuration is not set. For example: yarn_install: true/false , if true, it behaves as usual, if false it does not yarn install.

On Heroku also, by default it will reinstall all packages even if they have been previously cached. Not good.

Looks like this change is mostly wanted by Heroku users, but at the same time it is unwanted by Amazon Elastic Beanstalk users. It's not for me to judge which platform is more important but I expect a lot of people/teams having troubles because of this change and we can simply make it configurable instead deprecating.

@justin808  I'm on 6.5 and it still breaks builds on AWS

I'm sorry about the confusion regarding the log message guys, obviously the Warning cannot cause exit 1 or 2. Amazon EB somehow merges this log message with a general failure report for the build and it all looks like below:
[ERROR] An error occurred during execution of command [self-startup] - [rake tasks]. Stop running the command. Error: running rake task assets:precompile failed with error command bundle exec rake assets:precompile failed with error Command /bin/su webapp -c bundle exec rake assets:precompile failed with error exit status 1. Stderr:Shakapacker - Automatic installation of yarn packages is deprecated
Automatic installation of yarn packages when assets are precompiled is deprecated and will be removed in Shakapacker v7.

the root cause was somewhere else, AWS just "promoted" the Shakapacker deprecation message to be the root cause of the problem.

I just created PR #236 to allow an ENV SHAKAPACKER_YARN_INSTALL == false to skip the yarn install with assets:precompile.


A Metric Extractor detected a telemetry item with SamplingPercentage < 100

List of NuGet packages and version that you are using:
Microsoft.ApplicationInsights.AspNetCore 2.21.0 (also confirmed with latest 2.22.0-beta3)

Runtime version (e.g. net461, net48, netcoreapp2.1, netcoreapp3.1, etc. You can find this information from the *.csproj file):
net6.0 (.NET 6.0.19)

Hosting environment (e.g. Azure Web App, App Service on Linux, Windows, Ubuntu, etc.):
Issue occurs both in our deployed Ubuntu environments as well as locally under Windows

Describe the bug
When configuring fixed-rate sampling for ASP.NET Core applications we see the following trace log appear multiple times in Application Insights:
"AI: A Metric Extractor detected a telemetry item with SamplingPercentage < 100. Metrics Extractors should be used before Sampling Processors or any other Telemetry Processors that might filter out Telemetry Items. Otherwise, extracted metrics may be incorrect."
To Reproduce
We are calling the following from ConfigureServices in startup.cs:
    public static IServiceCollection AddApplicationInsights(this IServiceCollection services)
    {
        services.Configure<TelemetryConfiguration>(telemetryConfiguration =>
        {
            var builder = telemetryConfiguration.DefaultTelemetrySink.TelemetryProcessorChainBuilder;
            builder.UseSampling(1, excludedTypes: "Event;Exception;Trace");
        });

        var options = new ApplicationInsightsServiceOptions
        {
            ConnectionString = "conection string",
            EnableAdaptiveSampling = false
        };

        services.AddApplicationInsightsTelemetry(options);

        return services;
    }

If we change the order, and call Configure() after calling AddApplicationInsightsTelemetry() then the fixed rate sampling of 1% that we specified does not take effect, we see 100% of requests logged.

same here. Any updates?

The internal log is doing the right thing - the MetricExtractor is expected to extract metrics pre-sampling. So, if it is added after the sampling processor, it'll give inaccurate results, so its doing the right thing by warning!

Can you please explain how to implement this correctly then?
As mentioned above:

If we change the order, and call Configure() after calling AddApplicationInsightsTelemetry() then the fixed rate sampling of 1% that we specified does not take effect, we see 100% of requests logged.

Unfortunately, I do not have the bandwidth to investigate and provide a correct way. I simply removed the "bug" tag, as this is not a bug. If the doc is missing something or showing incorrect thing (likely to be the case, it seems), I agree the docs needs to be fixed. (perhaps better is to open issue against the doc.)

Please open a separate issue for that. The original issue posted is not a bug. But if sampling does not work as expected, then it is a bug, but I'd suggest to open it separately.

Closing as this is not a bug and working as expecting. The sampling-not-working is resolved in the linked issue.


Improve Terminal

There are some commands such as color and ping the could be added. There are more for sure, these are just some of top of my mind.
I could implement color for now and send a PR.
What do you think?

merged #671


svase: command not found

When I try to run anything within svase I get this error.  I've tried svase test out.sv test/assign/assign.sv and got svase: command not found. Then I tried to run make run-tests I get make: *** No rule to make target 'run-tests'.  Stop. Any help would be greatly appreciated.

Hello, you have to include the path where the program is installed (./build/svase). So, you have to run this command:
./build/svase test out.sv test/assign/assign.sv
Or, you include it in the system path by modifying the file bashrc if you use Ubuntu:
nano ~/.bashrc
export PATH="$PATH:/path/to/svase/build"
source ~/.bashrc

Thats correct, there is also an example in scripts/svase.env.
We should probably make this more clear in the Readme.md.


Fixed issue with connection not retrying

👍

Coverage remained the same at 100.0% when pulling 6614b63a3e1597baed9f87aa134fe76f3780ee42 on bbedwell:connection-fix into 6c05ae8d42698ccc68f2e4d542b43d70c7955cbe on CompassPHS:master.


Login limitations

Just a question:
How would you set login limitations. For example only users from a special group should be able to login.
Without changing the base_dn...

Hmm this doesn't exist yet, but this is definitely something I want to implement. I'll work on this now and see what I can do.

How goes a solution Steve? I need something like this for the project i'm working on. Been playing around with some ways of doing it, at least for what I need in my project and came up with some dirty hack job like this https://gist.github.com/shaynejellesma/d6a9a2a4771e6ff79c04 I am sure you could come up with something better, but just throwing a bit of inspiration your way. Thanks for the package :smiley:

Hi @shaynejellesma, that's a good implementation, and I think it'd be nice to set the groups inside the adldap_auth.php configuration file. I'll see what I can do tonight (current working on v6 of Adldap).

@shaynejellesma and @stevebauman, is it possible to search though the groups recursively? To allow for group nesting.

@clonerworks, by default all searches on LDAP through Adldap are recursive.

HI guys.........@stevebauman i looking for some thinks like  @shaynejellesma said and try to implement but i dont know where put this:
$allowedGroups = ['Domain Admins', 'App Login Users'];
Auth::attempt('username' => 'user', 'password' => 'password', 'allowedGroups' => $allowedGroups);
??
I started study laravel now........ Appreciate the help!

Groups are only example. Another would be a specific filter, for example only users with an email address. I think there should be a way to set a ldap filter.

Right, but how filter by ldap group? Only a especific group can login?
Thanks the reply!

There's always the option of overriding the newAdldapUserQuery() method to include a custom filter.
https://github.com/Adldap2/Adldap2-Laravel/blob/v1.4/src/Traits/ImportsUsers.php#L208
Though this shouldn't be hard to implement. I'll take a look right now.

Added in the most recent commit. Will be creating a new release shortly.
https://github.com/Adldap2/Adldap2-Laravel/commit/7e27250e89cecc82797a2460f19ae1fb21dd333e

Didn't know that exists: Config::get('adldap_auth.limitation_filter'); I think that's exactly what I was asking for :+1:
Thans @stevebauman for your hard work!

You'll need to re-publish the configuration file to see the new option :)
No problem at all!

Awesome! ...Thanks  @stevebauman !!

Test and work great! ....just a think......if a type a valid user but with wrong password,this message happens:
ErrorException in Ldap.php line 501:
ldap_bind(): Unable to bind to server: Invalid credentials
A miss something?

 Disable it and you won't have that error message come up (inside your .env file).
Or if you want to stay in debug mode, you can catch the exception:
try {
    if (Auth::attempt($credentials)) {
        // User passed authentication.
    }
} catch (\ErrorException $e) {
     // Catch LDAP Bind Errors
}

Thanks and sorry about the basic questions.........

No problem at all! No need to apologize.

I'm trying to get more information on the login limitation filter however, your link https://github.com/Adldap2/Adldap2-Laravel#login-limitation-filter no longer appears to work. It just takes me to the repository home page.
I'm trying to limit login to a specific group but it doesn't seem to be working. I've added the following to the file config/adldap_auth.php but it seems to ignore it.
'limitation_filter' => '(memberof=CN=FoneFactory Report Users,DC=internal,DC=[mydomain],DC=com)',
I'm using Laravel 5.5 and adldap2/adldap2-laravel v3.0.5, adldap2/adldap2 v7.0.15.

Hi @rnrstar, the limitation filter was removed in favor of scopes and rules.
This is why my link no longer works and you don't see this option in the published configuration.
You will need to create a scope that limits the user query to your filter like so:
namespace App\Scopes;

use Adldap\Query\Builder;
use Adldap\Laravel\Scopes\ScopeInterface;

class MemberOfScope implements ScopeInterface
{
    public function apply(Builder $query)
    {
        $group = 'CN=FoneFactory Report Users,DC=internal,DC=[mydomain],DC=com';

        $query->whereMemberOf($group);
    }
}

Remember to add this scope into your config/adldap_auth.php file when you're done.
For more details please visit this doc:
https://github.com/Adldap2/Adldap2-Laravel/blob/master/docs/auth/scopes.md#scopes

Thanks Steve. Got it working using the rules.

Great, glad you're up and running!


Improve chain listener

[ ] Support network disconnections
[x] Support reconfigurations
[x] Do not call synchronize_from_validators
[x] Move synchronization logic to linera-core
[x] Skip redundant notifications before synchronization

Probably out of scope for now:

[ ] Resist DoS from rogue validators

Support network disconnections

Part of this could be to retry inside the stream returned by GrpcClient::subscribe, depending on the status code, and possibly with some delays (exponential backoff or something). The downside is that the returned stream could now silently skip notifications that were sent while it was reconnecting.
The other status codes would probably terminate the stream.
An alternative is to make the NotificationStream type not Notification but Result<Notification, Status>, with our own Status enum, and handle the different cases in the chain listener.

Closed, and turned the last point into its own issue.


[LEGACY DATASOURCE EDIT]Java script validation is missing when user enters a 3rd party Default endpoint

A clear and concise description of what the bug is.
How to reproduce the bug

Authenticate to the application as a valid user. From "Data" tab click on the "Datasets" link.
For any available data set click on "Edit" button.
Click on the "USE LEGACY DATASOURCE EDITOR" button.
Fill up the form. Use the JavaScript payload "javascript:alert('NetSPI')" as the value of the "Default Endpoint" field and submit the form.
Navigate back to the datasets page. Click on the dataset name.
Observe the JavaScript payload getting executed in the browser.

Expected results
Validation is needed when user can add a malicious javascript inside the default endpoint.
Actual results
Javascript is being executed without any errors.
Screenshots

Now, Navigate back to the datasets page. Click on the dataset name and observe the JavaScript payload getting executed in the browser.
If applicable, add screenshots to help explain your problem.
Environment
(please complete the following information):

browser type and version: chrome 94.0.4606.61
superset version: 1.3
python version: 3.7
node.js version: v16.5.0
any feature flags active:

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[X] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[X] I have reproduced the issue with at least the latest released version of superset.
[X] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

@dharmic6 Unfortunately, we will not be able to fix any issues in the legacy datasource edit form. is it a specific reason that you are still using it, instead of the new modal?

@junlincc we are not using the old modal. This surfaced in our security scans so if we can provide a sunset date, will be fine to ignore. Thanks!

Setting this feature flag to False and noting the deprecation was part of 2.0 I believe. We cannot remove the flag and functionality without it constituting a breaking change, meaning it won't happen until Superset 3.0 when such breaking changes can be introduced. The date for this is not set, but I think we can safely close this as won't-fix. If anyone DOES want to open up a PR around this, we can certainly review it nonetheless.


State check options

Thanks for this plugin. Works perfect and lets me run BASH script, prtty much making HomeKit actions unlimited.
Question... Can I set the on/off result of a state check by reading the contents of a file?
For example if I put a 0 or a 1 in the file?
I have tried cat /someFile, and that file contains 'exit 1' but of course the catting itself was successful so thats what generates the 0.

If the file contains "exit 0", you can run the file with "bash filename". If it contains 0/1, you can try something like "cat file | xargs exit"

Thanks for the fast response!
This works perfect:
Command to check status:
bash /pathToSomeFile.sh
Then for the contents of the file:

"exit 0" tells HomeKit the accessory is on
"exit 1" tells HomeKit the accessory is off


Use the next stable version of asciidoctor.js

This version (1.5.7) includes a fix for warnings during webpack builds
See https://github.com/asciidoctor/asciidoctor.js/issues/508

Thanks @mindreeper2420


fixes equality operator used in platform test

changed the equality operator from equality to strict equality (from == to ===) when testing Platform.OS

Deploy preview for react-native ready!
Built with commit ec55bc181a419e1d4cbd591cf8729c9f1b3e2ab1
https://deploy-preview-2266--react-native.netlify.app


suckerberg and notifications

First of all: good job.
I like suckberberg very much but there's one problem to me.
Notifications aren't shown as proper as in redbasic because the group list is hiding it in most cases?

Hey, thanks for catching this. Turns out the notifications widget has been missing from the Suckerberg PDL files, which are used for page layouts. For now, notifications are on the left-hand side of pages. That should make it more convenient for seeing notifications on mobile devices.
Fixed via 7e8a7d4 !

Very nice but isn't it better placed above menu or underneath forums?

... and it is not pinned so new notifications are invisible whilst scrolling. Any way to change this?


Need switch to not use eventfd on target_os = "linux"

For an SGX enclave (using gramine), I can't use the eventfd mechanism. Would be nice, if I could fallback to the pipe mechanism.
Currently I am using https://github.com/haraldh/mio/commit/5852388314d46f0b231366dec683ede148a76d4e
But an additional feature might be too much maintenance burden.
Would this help? RUSTFLAGS='--cfg no_eventfd' ... so we can do #[cfg(all(not(no_eventfd), any(target_os = "linux", target_os = "android")))]

Oh yeah, RUSTFLAGS='--cfg no_eventfd' works great... see https://github.com/tokio-rs/mio/compare/master...haraldh:mio:no_eventfd_cfg

Is it a documented issue that eventfd can't be used inside of an SGX enclave, but epoll and pipes can?

@notgull at least for gramine:
https://gramine.readthedocs.io/en/stable/manifest-syntax.html#allowing-eventfd


Text in Markdown chunks disappears

Issue Type: Bug
Open a Markdown chunk. Type something in it. Click in another chunk. The text in the Markdown chunk disappears. I've found I have to "run" the chunk with shift+enter or ctrl+enter for it to stay.
Extension version: 2020.1.58038
VS Code version: Code 1.41.1 (26076a4de974ead31f97692a0d32f90d735645c0, 2019-12-18T14:58:56.166Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz (16 x 2400)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.73GB (16.05GB free)

Process Argv

Screen Reader
no

VM
0%

Validated. Working now in the insider's build.

Reopening to add tests. Found it was broken and we didn't have tests to prevent this.

Will add tests in existing PR, no point opening this.


Chart is not filling div width

Hi, I have a stacked bar chart with one horizontal bar. How can I get the chart to fill the whole width of the div? This is the code I have:
var data = {
	series: [
		[5],
		[10],
		[3],
		[0],
		[2],
		[7],
		[6]
	]
};

var options = {
	stackBars: true,
	horizontalBars: true,
	axisX: {
		showGrid: false,
		showLabel: false,
		offset: 0
	},
	axisY: {
		showGrid: false,
		showLabel: false,
		offset: 0
	},
	chartPadding: 0,
}

var c = new Chartist.Bar('.ct-chart', data, options).on('draw', function(data) {
	if(data.type === 'bar') {
		data.element.attr({
			style: 'stroke-width: 31px'
		});
	}
});

I answered something similar on StackOverflow. It might help.
https://stackoverflow.com/questions/40697145/chartist-js-bar-chart-fill-axis

Hello @rotaercz 👋
Chartist has recently been updated to version 1.0 with some new exciting changes like ESM and TypeScript support. The migration guide can be found right here.
Please let me know if the question is still relevant to you so we can further look into it!


Select multiple images for adding new cations

I really would like to be able to select more than one image from the main window, and then hit "apply changes to selected imageS" when I have many with the same caption. I am currently going through large datasets and selecting each image to edit is not fun. Yes Im using this to create captions. NOT just to edit

Yes… I totally agree with you about that, it is really redundant and annoying to select images each by each.
But this extension is working on gradio app, and it is an simple tool to create simple GUI for using ML models. Yes, actually, not designed for managing files like this extension.
To manage images like the explorer.exe or modern cloud apps (like google drive), creating GUI from scratch with javascript  (like openOutpaint extension does) is needed.

Sorry, forget about previous comment.
I will implement the feature.

Thanks for trying, but I dont think you understood. Now you changed it to all displayed, but I still cant select the specific images I want. Say I have 1000 images. They dont have any tags yet. I want to select 20, type a caption and apply that caption to those 20 photos

Thanks for trying, but I dont think you understood. Now you changed it to all displayed, but I still cant select the specific images I want. Say I have 1000 images. They dont have any tags yet. I want to select 20, type a caption and apply that caption to those 20 photos. then I move on to the next batch.
Having to select each photo to become displayed, and first THEN I can apply a caption to them all is even more time consuming. The "filter by selected" tab, adds an extra unnecessary step that can be avoided. Just let us select any photo in the main window, and do with them what we want. MULTI SELECT.
If that is not possible I suggest another option:
You already ALMOST have it.
The "Filter by selection" tab, has a button "add selection" You can use the ENTER button on it.
This allows for quick arrow keys selection for each image. So thats good. But why do the extra step of adding each image to selection before I can edit their tag.
If you allow "apply changes to selected image" to be triggered by ENTER key. I could use arrow keys to select each image very fast and skip the filter by selection tab entirely.
That one feature would save me 80% time "arrow key, enter, arrow key enter, arrow key, enter" done

I totally agree with you, it is redundant and annoying to select each by each.
But this extension is working on gradio and it is simple tool for making simple GUI for ML. In short, it is not designed to manage files like this extension.
To make GUI for managing files as explorer.exe or modern cloud apps (like google drive) does, it is needed to make GUI with javascript from scratch (like openOutpaint extension).
And, unfortunately, I don't have any experience or knowledge on making GUI working on web browser.
I reopen this issue as a reminder, but please note that there is no plan to actively add this feature.

I totally agree with you. It is redundant and annoying to select each by each.
But this extension is working on gradio, which is a simple tool to make simple GUI for ML. In other words, it is not designed to manage files as this extension do.
To manage files as explorer.exe or modern cloud apps do, it is needed to implement GUI with javascript from scratch (like openOutpaint extension).
However, unfortunately, I have no experience or knowledge to make GUI working on web browser.
I reopen this issue as a reminder, but please note that I have no plan to actively add the feature you expected.

That one feature would save me 80% time "arrow key, enter, arrow key enter, arrow key, enter" done

This is an operation in need, required, must be.
gradio gallery has NO way to get which image is selected NOR do multiple selection, and, currently, this extension knows which image is selected by interpreting HTML shown on the web browser.

If you allow "apply changes to selected image" to be triggered by ENTER key. I could use arrow keys to select each image very fast and skip the filter by selection tab entirely.

This could be good for solving this issue.
But for future, I will try to implement as custom keybord shortcut feature.

avesome, that would be greatly appreciated. I just spent 5 hours editing captions. and unfortunately there is no better way to manually create captions to my knowledge. Your extension is quite good.

It seems like the most natural workflow, to select multiple images from the preview and apply certain tags to them all at once.
Going one by one is a Sisyphean work even with a small number of images.

I ran into this too.  For example, if you have an image set of a character with two outfits, and you want to tag each outfit correctly for training.  You want it to say "Add the tag?" for each image and just click "Yes" "No" "Yes" "Yes" "No".
This is the best way I've found.  This was discussed above, but it's bured in conversation, here are steps in case it helps anyone else:

Select "Filter by selection"
Click the first image in the grid
If the image should have the tag, press Enter to add it to the list.
Press Right to move to the next image.
Repeat until you reach the end

When you're at the end, click "Apply selection filter".  Now you can use Batch Edit Captions to add the tag.
Takes a little getting used to, but it's not too bad.  It has some quirks (glitchy keyboard focus, etc.), but they're Gradio and AUTO111 problems, not this extension's.


Get datas from a database

Hi!
I'm struggling to get datas injected dynamically into my basic bar chart.. I'm always facing this error: Uncaught (in promise) TypeError: Cannot read property 'x' of undefined the trying to fill the series.
Is there any example on how format the incoming datas from an ajax request? I'm trying to fetch some json response but nothing comes into my graph.
Here are my datas from the json response:

And here is my code that export the data to my json response:

I'm using Laravel and VueJS.
Thanks in advance for your help!

Hi @bee-interactive
Can you put your data into a text-format for me to copy-paste easily and re-produce the issue on my side?
I would like to see the final datas you generate to see if it in the correct format.

Hi @junedchhipa !
Sure!
Here's the code:
My VueJS Component:
export default {
    components: {
        'apexcharts': VueApexCharts,
    },
    data () {
        return {
            dates: [],
            count: [],
            dateRangeOptions: {
                startDate: format(subDays(new Date(), 7), 'YYYY-MM-DD'),
                endDate: format(new Date(), 'YYYY-MM-DD'),
                format: 'MM/DD/YYYY',
            },
            series: [{
                name: 'sales',
                data: [this.count]
            }]
        }
    },
    methods: {
        fetchData() {
            return axios.post(`/get-sales/${this.range[0]}/${this.range[1]}`).then((response) => {
                this.dates = response.data.dates.dates
                this.count = response.data.dates.count
            })
        }
    }
}

My controller:
public function getSales(Request $request, $start = null, $end = null)
{
        $datas = [];

        $dates = new \DatePeriod(
            new \DateTime($start),
            new \DateInterval('P1D'),
            new \DateTime($end)
        );

        foreach($dates as $date) {
            $datas['days'][] = [
                'day' => $date->format("Y-m-d"),
            ];

            $datas['count'][] = [
                // Here I just fetch all my orders that was made on that given day and get the count of the rows found
                'count' => $this->getSalesByDay($date->format("d")),
            ];
        }

        $datas['dates'] = [
            'dates' => collect(array_values($datas['days']))->implode('day', ', '),
            'count' => collect(array_values($datas['count']))->implode('count', ', '),
        ];

        return json_encode($datas);
}

Hope this will help!

I think, it's more of a Vue issue.
I suggest, you move your series property from data to computed as it depends on your count prop.
computed: {
  series: function() {
      return [{
           name: 'sales',
           data: [this.count]
       }]
  }
}

If you can create a working demo with fake data on codesandbox vue-template, it would be helpful.
I suspect, that the series prop isn't getting the new data, so the re-render isn't happening. Try updating the series prop directly by
this.series = [{
  name: 'sales',
  data: response.data.dates.count
}]

and see if re-rendering happens

@junedchhipa It worked like a charm with the computed data! Thank you so much!

can anyone help me with apexchat?
i want to get data from API and push it into bar chart.. on a stateless component.
i need example
thanks