[MacOS Sequioa 15.0.1 M1 Pro] Build fails when building fontconfig

Both
CC=$(brew --prefix llvm)/bin/clang CXX=$(brew --prefix llvm)/bin/clang++ ./Meta/ladybird.sh run ladybird

and
./Meta/ladybird.sh run ladybird

fail with the same error:
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:127 (message):
    Command failed: /opt/homebrew/bin/ninja install -v
    Working Directory: /Users/nikola/src/ladybird/Toolchain/Tarballs/vcpkg/buildtrees/fontconfig/arm64-osx-rel
    Error code: 1
    See logs for more information:
      /Users/nikola/src/ladybird/Toolchain/Tarballs/vcpkg/buildtrees/fontconfig/package-arm64-osx-rel-out.log

Call Stack (most recent call first):
  /Users/nikola/src/ladybird/Build/ladybird/vcpkg_installed/arm64-osx/share/vcpkg-tool-meson/vcpkg_install_meson.cmake:33 (vcpkg_execute_required_process)
  buildtrees/versioning_/versions/fontconfig/de73339d579963e68fed652ed2e206b307a0c03f/portfile.cmake:40 (vcpkg_install_meson)
  scripts/ports.cmake:192 (include)

error: building fontconfig:arm64-osx failed with: BUILD_FAILED
See https://learn.microsoft.com/vcpkg/troubleshoot/build-failures?WT.mc_id=vcpkg_inproduct_cli for more information.
Elapsed time to handle fontconfig:arm64-osx: 7 s
Please ensure you're using the latest port files with `git pull` and `vcpkg update`.
Then check for known issues at:
  https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+fontconfig
You can submit a new issue at:
  https://github.com/microsoft/vcpkg/issues/new?title=[fontconfig]+Build+error+on+arm64-osx&body=Copy+issue+body+from+%2FUsers%2Fnikola%2Fsrc%2Fladybird%2FBuild%2Fladybird%2Fvcpkg_installed%2Fvcpkg%2Fissue_body.md

-- Running vcpkg install - failed
CMake Error at Toolchain/Tarballs/vcpkg/scripts/buildsystems/vcpkg.cmake:904 (message):
  vcpkg install failed.  See logs for more information:
  /Users/nikola/src/ladybird/Build/ladybird/vcpkg-manifest-install.log
Call Stack (most recent call first):
  /opt/homebrew/Cellar/cmake/3.30.5/share/cmake/Modules/CMakeDetermineSystem.cmake:146 (include)
  CMakeLists.txt:15 (project)

CMake Error: CMake was unable to find a build program corresponding to "Ninja".  CMAKE_MAKE_PROGRAM is not set.  You probably need to select a different build tool.
-- Configuring incomplete, errors occurred!

The log from: /Users/nikola/src/ladybird/Toolchain/Tarballs/vcpkg/buildtrees/fontconfig/package-arm64-osx-rel-out.log is attached as log.txt. It complains about Undefined symbols for architecture arm64 for:

fc-list/fc-list
fc-cache/fc-cache
fc-cat/fc-cat
fc-pattern/fc-pattern
fc-scan/fc-scan
fc-validate/fc-validate
fc-query/fc-query
fc-match/fc-match
fc-conflist/fc-conflist

I'm not sure if I should create an issue in vcpkg repo instead?
log.txt

The build log you posted is about libpng. There was some issue with libpng a while ago.  We clone vcpkg into Toolchain/Tarballs/vcpkg from Toolchain/BuildVcpkg.sh

Ladybird commit: d6a8fc00c3435bd9b4e61edeb364dec5c1bf65ea
vcpkg commit: 2960d7d80e8d09c84ae8abf15c12196c2ca7d39a

Doesn't fail for me.

I got desperate so I nuked my whole system. Reinstalled and set up everything from scratch.
And now it works ¯\(ツ)/¯.
The only thing I didn't do is install llvm.


Adds Tags to Cloudfront Distribution

Allows Tags to be added to cloudfront distribution as outlined here:
http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-cloudfront-distribution.html#cfn-cloudfront-distribution-tags

Thanks!


Fixes issue#1; migration to fiber-v2

This pull request migrates goth_fiber to v2 version of go-fiber

README.md updated
Updated the session.Store vs session.Session renaming
Fixed handler signature ( now returns error )
Fixed example snippet
Major - updated import path to github.com/shareed2k/goth_fiber/v2

Signed-off-by: Ankur Srivastava

@Shareed2k I think this can be reviewed now.

can you just update goth to latest version


Specto Fork issues

Help please..I uninstalled Specto and tried to reinstall it because I wasn't able to choose from "in theaters"or look up certain movies.and now it says Add on is incompatible or has been marked broken in repository. Dependencies not met. I tried to reinstall and am unable to. Any help is much appreciated.

try to install URLResolver and specto Themespak
or
install older version and it will update to the latest version

I can no longer use the download function in Specto.  Can you help?

that never worked correctly for me

It's been working fine, but since I did a Kodi reinstall, and reinstalled Specto, I can't get it to work.  That is on the TV with an Android box.  I loaded Kodi into my Windows laptop and everything works fine, including Specto Fork and the downloading.   So until I get over this hump, I take the laptop into the TV room and plug it into the TV with an HDMI cable and watch the shows I've downloaded in Specto on the laptop.
I think a big part of the problem is not being able to bring up the "C" prompt on the Android box, and select a target for the downloads.

In the Specto 2016.08.14,  set the download folder is mandatory


Issues for luks-setup

Hi Jia,
I have tested latest luks-setup with tpm2 support and found some issues:

enroll_token doesn't work

I got the following error when running "luks-setup -V -f -e -d /dev/sdb1 -n luks_sdb1"
[?] Enrolling a new token for the unsealed passphrase ...
Device /dev/sdb1 is not a valid LUKS2 device.
[!] Failed to enroll a new token for the unsealed passphrase
[!] Unable to enroll a new token on the creation for the LUKS volume "luks_sdb1" ...
The root cause is "cryptsetup token import" only supports LUKS2, while the luks type is set to LUKS1 by default in create_luks_volume():
local cmd="cryptsetup --type luks --cipher aes-xts-plain64 --hash sha256
--use-random --key-file "$PASSPHRASE" luksFormat "$luks_dev"
So it's better to add an option to specify LUSK1/LUKS2.

enroll_recovery doesn't work

When I changed luks type to LUKS2 manually, I got another issue on enroll_recovery():
[?] Enrolling a new token for the unsealed passphrase ...
[!] Enrolled the new token for the unsealed passphrase
[!] Created the LUKS volume "luks_sdb1" on the backing device "/dev/sdb1"
[?] Enrolling keyslot and token for recovery ...
[?] Enrolling a new keyslot ...
[?] Retrieving the passphrase ...
[!] Unrecongnized token type "luks-setup-prompt-recovery"
[?] Enrolling a new token for the prompted passphrase for recovery ...
Failed to import token from file.
[!] Failed to enroll a new token for the prompted passphrase for recovery
[!] Unable to enroll a new token for recovery
This is because the token type is set to "luks-setup-prompt-recovery" and it isn't handled in retrieve_passphrase().

Run "luks-setup" without arguments should print usage, but now it doesn't:
$ luks-setup
/usr/sbin/luks-setup: line 605: yum: command not found
[!] Failed to install the package "procps-ng"

Thanks,
Yi

4.The  "--no-tpm" option doesn't work
root@genericx86-64:~# luks-setup -V -f -t -d /dev/sdb1 -n luks_sdb1
Skip the detection of TPM 2.0 device
Prepare to prompt for the passphrase
Temporary directory created at "/tmp/luks-setup.p0jXa9"
The backing device "/dev/sdb1" is not a LUKS volume
[?] Creating the LUKS volume "luks_sdb1" ...
[?] Retrieving the passphrase ...
[!] Unrecongnized token type "luks-setup-prompt"
[?] Mapping the LUKS volume "luks_sdb1" with the token "luks-setup-prompt" ...
[?] Retrieving the passphrase ...
[!] Unrecongnized token type "luks-setup-prompt"
The LUKS volume "luks_sdb1" backing on /dev/sdb1 is created
[?] Cleaning up ...
[!] Exiting

@yizhao1 Thanks for report these issues. All issues except the first one are fixed.
Currently there is no plan to support LUKS version 1, so instead of supporting it, luks-setup will check the default created LUKS version before token enrollment to avoid a token enrollment failure.


Missing YAML docs

Deployer version: 7.0-rc4
Deployment OS: Mac OS

I'm using an deploy.yaml with runLocally to prebuild my assets:
tasks:
  deploy:assets:
    - runLocally: 'yarn install && yarn build'
    - upload:
        src: web/assets/
        dest: '{{release_path}}/web/assets'

This is the verbose output:
task deploy:assets
[hostname]<EMAIL_ADDRESS>/vendor/deployer/deployer/bin/dep worker --port 50806 --task deploy:assets --host hostname --decorated -vvv
[hostname] rsync '-azP' '-e' 'ssh '\''-A'\'' '\''-o'\'' '\''ControlMaster=auto'\'' '\''-o'\'' '\''ControlPersist=60'\'' '\''-o'\'' '\''ControlPath=/Users/dennis/.ssh/deploy@hostname'\''' 'web/assets/' 'deploy@hostname:/web/craft-cms/releases/19/web/assets'
[hostname] sending incremental file list
[hostname] rsync: [sender] change_dir "/web/assets" failed: No such file or directory (2)
[hostname] rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1350) [sender=3.2.3]
[hostname]  error  in Importer.php on line 181:
[hostname] exit code 23 (Unknown error)

The uploads fails because web/assets wasn't generated by the runLocally build.

Found the issue:
-    - runLocally: 'yarn install && yarn build'
+    - run_locally: 'yarn install && yarn build'

Add docs to prevent misuse of YAML (runLocally instead of run_locally).


A question

Hi Alan, I heard about you from a Brototype student I know.  So I was curious and went exploring your github when I came across this repo.
Just one question: Your description mentions that all data is stored locally by the browser. But the primary goal of such a tool is collaboration between team members. If the source of truth of data (task status, description etc) is session or some other local storage, how is this possible?
Please don't take this as a criticism and its not. I have respect for your hardwork and passion. I looked at some other of your repos as well

Hello Dheeraj,
I feel like this really isn't the place for non-development related queries. You can always message me at my email or telegram which I've listed rather than here but anyways,
Would love to know whoever referred to me as if I were a legend 😂 I think that's a little too heavy of a term to live upto.
This was an experimental side project that I've built a really long time ago, even before my Brototype days. Yes it does not have real time collaboration because that was really not what I was building. If you notice, the whole thing does not depend on any frameworks or libraries and is fully vanilla Javascript. From the drag and drop, to the multi board system everything was built from scratch and that caused a lot of headache for me but it was also a valuable experience which taught me a lot.
The data is locally stored, I believe I haven't mentioned anything about real time collaboration for this project. I do not think a kanban board app necessarily needs team collaboration in my opinion as I do use them for personal task management.
I do plan to revisit this concept of building a pure vanilla app and try to make this again with live collaboration sometime in the near future, but preferably not within this same repository as it has sort of become unmaintainable with all the technical debt. I feel like I've learnt a lot since then which would be beneficial to something like this.
Again, thanks for stopping by! Very rarely anyone leaves a comment or a query on what I do, feel free to ping me at my mail or telegram if you wish to know more..
Regards,
Alan.

Snobbery is indeed the path to success. Keep at it. I wish you best, young man.

Hello again,
Just to keep things clear: I sincerely did not mean any offense. I genuinely wish to understand.
I believe I responded professionally and appropriately to your query. I really do not understand why you would leave such a remark.
You asked something about my project and I really love talking to people who are curious about what I do. Perhaps I wrote a little too much? I apologize for being passionate about what I do.
Respectfully,
Alan.


identifier for i18n and tests

Tasks
@Bushuo wrote in #613

Going forward, this all seems pretty overkill.
Maybe the refactoring/maintenance burden is not that high and we really just use some simple ids (with data-testid).

Yes, I agree, it is overkill. Let us keep it as simple as possible, data-testid (and directly using Text in certain situations) is the more straight-forward way.
What about using the same guidelines for both i18next identifiers and data-testid?
Background: There are already some inconsistencies in the  i18next identifiers:

[ ]  do error_ prefixes, other as postfixes  in i18n/en/common.json (I think we should go for prefix)
[ ]  sometimes camcase, e.g.sliderTooltip) in i18n/en/layerSettings.json (I think it should be slider_tooltip)
[ ] "title" and "button" sometimes prefix, sometimes postfix, sometimes hierarchy e.g. i18n/en/maps.json (I think it should be hierarchy or prefix)
[ ] sometimes failed and not error: i18n/en/nextcloudIntegration.json load_image_failed
[ ] sometimes button, sometimes btn i18n/en/seeds.json (I think we should use full words, i.e. button)
[ ] sometimes uppercase i18n/en/undoRedo.json (CREATE_PLANT -> create_plant)

@Bushuo @4ydan what do you think?
Use case
translation
Related Pull request
#613

We both just had a call on discord and we agreed on using testid's. As already mentioned by bushuo interpolation is a big problem and won't allow us to use the jsons consistently. I am also not a fan of mixing stuff, even though it would be really handy to use the jsons and search everything by text. Worst case, when someone breaks a test, because of changing an error message or button text, he/she can easily go and modify the according test object. But this case should be very rare as the texts should be final and reviewed before we write tests for them.

Decisions are either in consent (everyone obviously agrees or doesn't care) or by written decisions (especially for more important stuff where it is important to get the decision right).
Anyway, please read the text above carefully. My idea actually is to use testids (data-testid), and describes a naming convention for them.

But this case should be very rare as the texts should be final and reviewed before we write tests for them.

No, the text is far away from being final. It was written and reviewed by computer scientists with little detailed feedback from real users yet. Thus texts (other than "Submit") are very likely to be changed.

Decisions are either in consent & please read the text above carefully

I agree, when two people agree on something, it doesn't mean that everyone agreed on it. I just wanted to point out, that we two are not fans of using json parsing, as long as interpolation is in our way.
From my point of view, the internationalization should not have an effect on the tests. When I perform tests, I want to purely focus on the functionality. I have no experience with i18n. This is all new to me and I don't understand the problems that you currently have with it.

What about using the same guidelines for both i18next identifiers and data-testid?

Does a guideline for i18next exist? I couldn't find one. Or does this mean as in write them both now so they are the same.

No, the text is far away from being final.

The texts I am referring to are info/error popups and user interactable elements. These should be final if they can't be, we should not use texts.

My idea actually is to use testids (data-testid)

What exactly is the difference between testid and data-testid?

Does a guideline for i18next exist? I couldn't find one. Or does this mean as in write them both now so they are the same.

There is some documentation about i18n (very well hidden) in ./doc/contrib/frontend.md but there is only little guidance about the identifiers.

The texts I am referring to are info/error popups and user interactable elements. These should be final if they can't be, we should not use texts.

I don't know what you mean here? If you talk about testing: Yes, for testing data-testid is more stable than searching for text. It is a bit more effort, though.

What exactly is the difference between testid and data-testid?

I used it: General concept vs. specific attribute to use. But better we only use data-testid from now on as it seems to be clear that we want to use it.

I don't know what you mean here?

Lets define action/user interactable elements
Action: text inputs, checkboxes, radio buttons, select options, mouse clicks, type characters, keys and shortcuts as well as upload files and focus elements.
For me it is important to find the action elements. If its by text, the text better be final, if not, I will use the data-testid. But in general, the action elements text should be final, as most of the time it's some crud operation.
Everything else, that is not an action element, I'm currently not interested in testing for simplicity reasons. Can do it, once we have covered the basic stuff.
Also, using data-testid for all actions, would be my preferred way, since I'm not sure if all actions have a text, so why not use data-testid for all, what is the benefit of mixing it?.

This is what Playwright suggests.
Roles

To make tests resilient, we recommend prioritizing user-facing attributes and explicit contracts such as page.getByRole()

Labels

Most form controls usually have dedicated labels that could be conveniently used to interact with the form. In this case, you can locate the control by its associated label using page.getByLabel().

Text

We recommend using text locators to find non interactive elements like div, span, p, etc. For interactive elements like button, a, input, etc. use role locators.

Test id's

Testing by test ids is the most resilient way of testing as even if your text or role of the attribute changes the test will still pass. QA's and developers should define explicit test ids and query them with page.getByTestId(). However testing by test ids is not user facing. If the role or text value is important to you then consider using user facing locators such as role and text locators.

Here are the suggested locators from playwright for each action:

text inputs: label
checkboxes: label
radio buttons: label
select options: label
mouse clicks: role or text
type characters:  locator
keys and shortcuts: text or role <- this is the only place where I would use text
upload files and focus elements: label

Yes, good inputs. I think you can simply write the discussed guidelines and I will comment there.

Created a first draft.


pip install error on python 3.9 conda environment

pip install error on python 3.9 conda environment and it reports error as pasted below:
Getting requirements to build wheel ... error
error: subprocess-exited-with-error
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> [33 lines of output]
Traceback (most recent call last):
File "/home/wuy/software/envs/metagpt/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in
main()
File "/home/wuy/software/envs/metagpt/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
json_out['return_val'] = hook(**hook_input['kwargs'])
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/wuy/software/envs/metagpt/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 112, in get_requires_for_build_wheel
backend = _build_backend()
^^^^^^^^^^^^^^^^
File "/home/wuy/software/envs/metagpt/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 77, in _build_backend
obj = import_module(mod_path)
^^^^^^^^^^^^^^^^^^^^^^^
File "/home/wuy/software/envs/metagpt/lib/python3.12/importlib/init.py", line 90, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 1387, in _gcd_import
File "", line 1360, in _find_and_load
File "", line 1310, in _find_and_load_unlocked
File "", line 488, in _call_with_frames_removed
File "", line 1387, in _gcd_import
File "", line 1360, in _find_and_load
File "", line 1331, in _find_and_load_unlocked
File "", line 935, in _load_unlocked
File "", line 995, in exec_module
File "", line 488, in _call_with_frames_removed
File "/tmp/pip-build-env-pqdx5bfu/overlay/lib/python3.12/site-packages/setuptools/init.py", line 16, in
import setuptools.version
File "/tmp/pip-build-env-pqdx5bfu/overlay/lib/python3.12/site-packages/setuptools/version.py", line 1, in
import pkg_resources
File "/tmp/pip-build-env-pqdx5bfu/overlay/lib/python3.12/site-packages/pkg_resources/init.py", line 2172, in
register_finder(pkgutil.ImpImporter, find_on_path)
^^^^^^^^^^^^^^^^^^^
AttributeError: module 'pkgutil' has no attribute 'ImpImporter'. Did you mean: 'zipimporter'?
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.
note: This error originates from a subprocess, and is likely not a problem with pip.
(metagpt) wuy@WEST-Algorithm-Server3:~/AI$ conda deactivate

You are not on python 3.9, but python 3.12:
File "/home/wuy/software/envs/metagpt/lib/python3.12/site-packages/...


Fix CodeMirror paths in sw cache list

I messed up on a few paths, this fixes things.

r? @gideonthomas or @Pomax

r+


[#5507] SingleThreadEventExecutor should reject call invoke*() from w…

…ithin the EventLoop.
Motivation:
ExecutorService.invoke*(...) methods may block by API definition. This can lead to deadlocks if called from inside the EventLoop in SingleThreadEventExecutor as it only has one Thread that does all the work.
Modifications:
Throw a RejectedExectionException if someone tries to call SingleThreadEventExecutor.invoke*(...) while in the EventLoop.
Result:
No more deadlock possible.

@Scottmitch @nmittler @trustin PTAL

LGTM

lgtm

Cherry-picked into 4.1 (ce95c50) and 4.0 (438b29f)


Added Decentralized Graffiti Drawing Tool Demo link

As requested by GreyWizard, he wanted to include the matching video of his 'Decentralized Graffiti Drawing' tool.
Section: Community - Tools
Item: Decentralized Graffiti Drawing Tool
Action: Add/Edit/Delete

By submitting this pull request, I promise I've read the contribution guidelines.

Since we already have a link to the tool, why not include the video in the same line?

Currently, things are grouped on a specific category (guides, video, ...) There is already an item covering the same topic, but it's situated as a guide because it's a written article, so I thought I'd put this under the video section.
If you still think this should be referenced in the same line (Community - Guides), let me know, otherwise I propose we put this under the 'Community - Videos' section.

Currently, things are grouped on a specific category (guides, video, ...) There is already an item covering the same topic, but it's situated as a guide because it's a written article, so I thought I'd put this under the video section.
If you still think this should be referenced in the same line (Community - Guides), let me know, otherwise I propose we put this under the 'Community - Videos' section.

The issue I see with this approach is a) you end up with duplicated content, and b) you end up with content that refers to the same "thing" spread across different parts of the page. From a user perspective, I think it makes more sense if they see e.g. a tool and then right next to it a video related to the tool, as opposed to having to see where on the page there may or may not be a related video/article/whatever. In general, is the issue with grouping items by medium instead of by content.
@0xpenguin0x what do you think?

No longer valid; this was included/fixed in the pull request that fixed/changed the subcategories/sections. (https://github.com/isidorosp/awesome-rocketpool/tree/VGR-GIT-Categorize-Subcategories-By-Content-Instead-Of-Medium)


Request : Publish a binary for ubuntu

@conor10
For us folks, behind enterprise firewalls, building from base is kind of troublesome. We run into issues with the proxy during 'make setup' stage. mostly because of certificate checks and other internet connectivity related stuff.
If you can publish an ubuntu binary for crux, we can download the binary into our development environments and start experimenting.

Does the version of Ubuntu it’s built against matter?

It shouldn't. as its an executable binary
Our docker containers are running on Ubuntu 16.04.

Hi @ashfaq-shaik, Linux and OS X release builds are now available https://github.com/blk-io/crux/releases/tag/v1.0.0

Thanks a lot. @conor10


Class method with this typing cannot access private or protected members

TypeScript Version:  2.2.0-dev.20170116
Code
class Foo {
    constructor(
        protected bar: string | null,
        protected baz: number
    ) {}

    doThing(this: Foo & { bar: string }) {
        return this.bar.length > 0 ? this.baz : -1;
    }
}

Expected behavior:
Compiles without error.
Actual behavior:

Property 'baz' does not exist on type 'Foo & { bar: string; }`.

Notably, though, even if baz is public, attempting to use doThing here even from within another method fails:
class Foo {
    constructor(
        protected bar: string | null,
        public baz: number
    ) {}

    doThingIfYouCan() {
        if (this.bar != null) {
            return this.doThing(); // error here
        }
    }

    doThing(this: Foo & { bar: string }) {
        return this.bar.length > 0 ? this.baz : -1;
    }
}

Here the error is

The 'this' context of type 'this' is not assignable to method's 'this' of type 'Foo & { bar: string; }'.
[...]
Property 'bar' is protected in 'Foo' but public in type '{ bar: string; }'.

This is a very similar problem, where the this typing on doThing requires that bar is public even though both canDoThing and doThing should have access to it.
My use-case here is that I have a method that is currently only being called in non-null cases, and only makes sense to call in non-null cases, so I'm currently throwing in a null case (that should never happen). But I would like to enforce that at compile time.

This issue had the same root cause as #11454 and is now fixed in TS 3.9+ (see linked issue).


Spellcheck API docs (Unity Fund Hackathon)

This is part of https://github.com/Qiskit/documentation/issues/818. We still have to decide in that ticket what we long-term with spellchecking the API docs in qiskit/qiskit. In the meantime, it is valuable to fix all current spelling violations.
To do this:

Clone https://github.com/Qiskit/documentation
Edit the cspell config at https://github.com/Qiskit/documentation/blob/b7737fb4f8e7c0483a8e0af0ed62c59c828720c9/cspell/cSpell.json#L51 to not ignore /api/qiskit/dev, but we still want to ignore the other api/qiskit folders like /api/qiskit/0.*/*.mdx and /api/qiskit/*.mdx (current docs)
Run cspell with the instructions at https://github.com/Qiskit/documentation#spellcheck
Fix the spelling violations in a PR in this repository. It's plausible there will be legit words/phrases that are not issues. Those should instead be added to the dictionaries in qiskit/documentation as valid words (see the section "Fixing false positives")

It's best to keep these changes in one PR, rather than opening up a bunch of tiny PRs.

I would like to work on this issue. Kindly please assign this issue to me. Thanks

hi @Raghav-Bell , we are setting some issues (like this one) for coming events. Let's hold it for now :)

Hi, @Eric-Arellano @1ucian0 !
I'd like to work on this issue as a part of unitaryhack.

sure thing @CRTejaswi ! Because the nature of the event, go ahead and PR it. The first/best PR can get the bounty (or could be shared if they complement).

sure thing @CRTejaswi ! Because the nature of the event, go ahead and PR it. The first/best PR can get the bounty (or could be shared if they complement).

Are EN-UK words to be treated as spelling mistakes (hence corrected)?
eg. organised, behaviour (vs organized, behavior in EN-US)

Are EN-UK words to be treated as spelling mistakes (hence corrected)?

Yes, please. We use EN-US for consistency.

Thanks, @CRTejaswi!
Any other Unitary Fund Hackathon people interested in this issue, please wait to get started on this ticket until @CRTejaswi has had some more time to finish. We want to respect how much time they've already put into this ticket and avoid duplicate work.

I apologise for the delay. I've been occupied with some work, so, couldn't finish up. Will do shortly.

All good, @CRTejaswi. By the way, in case it wasn't already clear, the fixes will need to happen in this repo qiskit/qiskit, not qiskit/documentation. That's because the docs are generated from qiskit/qiskit, so it's the "source of truth".

just few remainders to everyone:

this is is closed by PRing Qiskit/qiskit, not Qiskit/documentation :)
more than one PR can be submitted and even accepted (in case they complement each other)
the bounty will be decided based on when it was submitted, how much hand holding required, and how complete it is. It can even be split amount submitters.

@1ucian0 ok then, here is the PR on Qiskit/qiskit https://github.com/Qiskit/qiskit/pull/12501

hello @1ucian0 I guess the owner has to do some checks before the PR to be merged, but I guess my job is done. Let me know to consider this bounty is achieved, thanks!

You need to address the comments and conflicts in the PR. The bounty will be assigned to you after that. The issue will be crossed out from the website once this issue is closed (because the PR was merged)

hi @1ucian0 I resolved the conflict, I think there is no other comment for me, let me know is something else is missing


ABI.solidityPack skips following types/values when type=bytesX

Line 453 in /lib/index.js is:
return utils.setLengthRight(value, size)
and should be (if I'm right):
ret.push(utils.setLengthRight(value, size))
Thanks very much

I submitted a fix with new test cases in PR #40.


[fuzzydir] ignore protocol content and add excluded_dir options to ignore specified paths

The successor to #2. I think it's time to implement it.
Close https://github.com/sibwaf/mpv-scripts/issues/9

About #9: let's not jump the gun just yet, we don't have enough information about what's wrong here. Based on the name of the issue it looks like the -Directory parameter for some reason doesn't work, though in theory it should with PowerShell 3+. I do have a suspicion that we don't quote the path properly (it's kinda complicated with PowerShell), but that needs further research.
I should also probably hide those horrendous error messages so people don't get so scared, I guess. It's completely fine in the cases it fails to scan some paths (happens, for example, if a directory doesn't exist) - this particular path just gets ignored and everything continues working as expected. Thus, we don't have a reason to add a crutch for completely skipping all protocols. If we can't scan paths for the opened file - well, just silently fail, it's not a problem.

As for the path exclusion - yeah, it's a good feature to have, but configuring it like this rubs me the wrong way. I think it would be better to add the exclusions to mpv's config directly, something like this (notice the exclamation sign; also - not the final syntax):
sub-file-paths=**;!audio;!Z:\**

This way all the configuration you care about is conveniently kept in one place with consistent syntax. The two properties that are configured in the script directly at the moment are, to be honest, just constants which no one should ever touch without a good reason. Thus, I don't really consider those as configuration.
This is also connected to #4, which is still not completely implemented as there is no way to enforce scanning some directories - I'm thinking about something similar, but I don't really have any good ideas for the syntax for now.

Tested this PR https://raw.githubusercontent.com/sibwaf/mpv-scripts/788dcc5fa15901045aacaafb7c4776924cb90349/fuzzydir.lua
No error message

About #9: let's not jump the gun just yet, we don't have enough information about what's wrong here. Based on the name of the issue it looks like the -Directory parameter for some reason doesn't work, though in theory it should with PowerShell 3+. I do have a suspicion that we don't quote the path properly (it's kinda complicated with PowerShell), but that needs further research.

The syntax of the powershell command is correct. The error was reported only because the network protocol is not a directory, which is in line with expectations.

In fact, as mentioned in the commit, the script has never worked for the protocol content because utils.readdir(path, "dirs") .
This is a bug fix, not a functional break.

As for the path exclusion - yeah, it's a good feature to have, but configuring it like this rubs me the wrong way. I think it would be better to add the exclusions to mpv's config directly, something like this (notice the exclamation sign; also - not the final syntax):
sub-file-paths=**;!audio;!Z:\**

This way all the configuration you care about is conveniently kept in one place with consistent syntax. The two properties that are configured in the script directly at the moment are, to be honest, just constants which no one should ever touch without a good reason. Thus, I don't really consider those as configuration.

I don't quite agree with this because I don't like scripts intruding too much into configuration files of mpv.conf. This approach is too hacker.
It is much cleaner to implement it in the script's configuration file of script-opts/fuzzydir.conf, which is also the recommended usage of mpv

The syntax of the powershell command is correct. The error was reported only because the network protocol is not a directory, which is in line with expectations.

Well, can't say that it's what I expected by the error message "unknown parameter -Directory". If the path doesn't exist it should be "PathNotFound" or something, not an "invalid parameter" for an unrelated parameter.
Whatever, not really important.

This is a bug fix, not a functional break.

 What if some of the protocols work? Or will start to work with the next mpv update? We'll have to make exceptions for the ones that work, and then the script will break on older mpv versions.
The bug is not functional, it's purely visual. You can try adding testtesttest/** to your sub-file-paths and opening something. You will get an error as the script tries to scan a directory that doesn't exist, but the subtitles will still be properly loaded as the error is ignored.
Then you can add this to the script after capture_stdout = true:
capture_stderr = true,

The error will disappear from mpv's log, even though we did nothing to "fix" it. We just hide the output which wasn't supposed to be shown in the first place (and on Linux it's not there by the way, mpv probably forks the process and doesn't connect the stdout/stderr to the main one; though that's just a guess, a bit too lazy to check the source code for that).

I don't like scripts intruding too much into configuration files of mpv.conf

The script already uses *-paths properties in mpv.conf and no one is complaining about this. We're not going to intrude on any other properties and just add more functionality to the same ones. And you certainly need to have inclusions before your eyes for reference when configuring the exclusions.
I also tried the options-way when I was implementing #4. In my opinion, this wasn't convenient to use - thus I skipped on the "force-scan" functionality and then got this idea a bit later. It just doesn't make sense to me to have interconnected configurations being not even in different properties, but in different files.
I can make a vote for this if I didn't manage to convince you, though I'm not sure we'll get enough votes for any conclusive results.

This is a bug fix, not a functional break.

 What if some of the protocols work? Or will start to work with the next mpv update? We'll have to make exceptions for the ones that work, and then the script will break on older mpv versions.

As long as the mechanism of utils.readdir(path, "dirs") does not change, no agreement content will take effect. MPV developers don't seem willing to change this either: https://github.com/mpv-player/mpv/issues/11438

The bug is not functional, it's purely visual. You can try adding testtesttest/** to your sub-file-paths and opening something. You will get an error as the script tries to scan a directory that doesn't exist, but the subtitles will still be properly loaded as the error is ignored.
Then you can add this to the script after capture_stdout = true:
capture_stderr = true,

The error will disappear from mpv's log, even though we did nothing to "fix" it. We just hide the output which wasn't supposed to be shown in the first place (and on Linux it's not there by the way, mpv probably forks the process and doesn't connect the stdout/stderr to the main one; though that's just a guess, a bit too lazy to check the source code for that).

Yes, this can suppress log output and silently fail. This is a better method.

I don't like scripts intruding too much into configuration files of mpv.conf

The script already uses *-paths properties in mpv.conf and no one is complaining about this. We're not going to intrude on any other properties and just add more functionality to the same ones. And you certainly need to have inclusions before your eyes for reference when configuring the exclusions.
I also tried the options-way when I was implementing #4. In my opinion, this wasn't convenient to use - thus I skipped on the "force-scan" functionality and then got this idea a bit later. It just doesn't make sense to me to have interconnected configurations being not even in different properties, but in different files.
I can make a vote for this if I didn't manage to convince you, though I'm not sure we'll get enough votes for any conclusive results.

We cannot agree on the implementation method, and I am not willing to further invade mpv.conf because I will only use it in the script configuration file anyway.


Add SMTP settings to attributes

Description
Add SMTP settings to attributes

This repository is great because it works well with Chef 11 :)


when connecting to QA maker through the emulator a trace stack error occurs

Version
What version of the Emulator are you using.
mac 4.5.2
Describe the bug
When connecting to the qnamaker through the emulator an error is emitted stating the follow: (even thought it is authenticating correctly)
Unable to find a QnA Maker service with Knowledge Base ID xxxxxxx-xxxxxx-44ea-94bf-xxxxxxx. Please add a QnA Maker service to your bot.
To Reproduce
Steps to reproduce the behavior:
create qna
in the settings file place the qna id and endpoint key
run the emulator
the qna works
the trace stack fails
Expected behavior
the tracestack should work
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
[bug]

@CoHealer can you have one of the support engineers look into this, thanks!

I'll check it out.

Unable to repro this right out of the box, so I need more details: what language are you using and where are you seeing that error (chat window, console, sidebar in the emulator, other program, etc)?

@jwiley84 this is a node sdk version of the famework 4.5.1 and it is in the output of the trace in the emulator logs

Hi @xtianus79 - I would like to help but I have a few questions

Are you saying QnA Maker is working correctly and generating answers despite the error message saying it can't find the QnA Maker service?
Do you see any errors in your IDE while debugging your code or do the errors only show up in Emulator?
Do you see any errors in your logs or your channels blade when using other channels or is this Emulator-specific?
Please post screenshots so we can see more clearly what the issue is

@xtianus79 - Are you still working on this?

@v-kydela yes.

yes

only in the emulator but I have to triple check that but for now it is soley the emulator trace that is the issue.

Emulator specific

I will get screen shots but this issue is the same issue as reported previously by another author.

Also, this may be an issue but the setup to the qna goes through the dispatcher.  If that helps.

@xtianus79 - Do you still need help?

Closing inactive. You can reopen if you still need assistance.

I'm having the same issue; if I can pick this up and try and answer the questions that were previously left unanswered:

Are you saying QnA Maker is working correctly and generating answers despite the error message saying it can't find the QnA Maker service?

QnAMaker returns a response as expected despite the "Unable to find a QnA Maker service with Knowledge Base ID [ID]. Please add a QnA Maker service to your bot." message.

[14:34:39]Emulator listening on http://localhost:51917
[14:34:39]ngrok listening on https://6b1809e8.ngrok.io
[14:34:39]ngrok traffic inspector:http://<IP_ADDRESS>:4040
[14:34:39]Will bypass ngrok for local addresses
[14:34:45]<- messageapplication/vnd.microsoft.card.adaptive
[14:34:45]POST200conversations.replyToActivity
[14:34:45]POST200directline.conversationUpdate
[14:34:45]POST201directline.startConversation
[14:34:55]-> messagepayment systems overview
[14:34:56]<- traceLuis Trace
[14:34:56]POST200conversations.replyToActivity
[14:34:56]<- typing
[14:34:56]POST200conversations.replyToActivity
[14:34:58]<- traceLuis Trace
[14:34:58]POST200conversations.replyToActivity
[14:34:58]<- typing
[14:34:58]POST200conversations.replyToActivity
[14:34:59]<- traceQnAMaker Trace // This trace shows the Error
[14:34:59]POST200conversations.replyToActivity
[14:34:59]<- message**Payment Systems Overview** There are two paymen... // This trace shows the QnA Json.
[14:34:59]POST200conversations.replyToActivity
[14:34:59]<- message
[14:34:59]POST200conversations.replyToActivity
[14:34:59]POST200directline.postActivity

Do you see any errors in your IDE while debugging your code or do the errors only show up in Emulator?

No exceptions are generated and no errors are logged in the output. If you didn't see the emulator trace, you would have no way of knowing something went wrong.

Do you see any errors in your logs or your channels blade when using other channels or is this Emulator-specific?

I have no other channels hooked up; using the test in web chat from the azure bot resource does not reveal any errors.

Please post screenshots so we can see more clearly what the issue is

I made this post on StackOverflow. They were kind enough to link me here.

Other (possibly) related issue:
When executing the Update_cognitive_models script with -RemoteToLocal flag, I get the following errors:
-----------------------------------------------------------
 NOTICE:
 This tool has been deprecated.
 All functionality was ported over to the new BF CLI.
 To learn more visit https://aka.ms/NewBFCLI
-----------------------------------------------------------

Sorry, error parsing file as QnA JSON: C:\src\TenaskaVirtualAssistant\TenaskaVirtualAssistant\TenaskaVirtualAssistant\Deployment\Scripts\..\Resources\QnA\en\Chitchat.qna
Stopping further processing.
> Adding Chitchat kb to dispatch model ...
> Updating local Faq.lu file ...

-----------------------------------------------------------
 NOTICE:
 This tool has been deprecated.
 All functionality was ported over to the new BF CLI.
 To learn more visit https://aka.ms/NewBFCLI
-----------------------------------------------------------

Sorry, error parsing file as QnA JSON: C:\src\TenaskaVirtualAssistant\TenaskaVirtualAssistant\TenaskaVirtualAssistant\Deployment\Scripts\..\Resources\QnA\en\Faq.qna
Stopping further processing.
> Adding Faq kb to dispatch model ...
> Updating dispatch model ...
> Running LuisGen for Dispatch app...
> Done.

I know the BF cli is available, and I have it installed, but I'm not super familiar with it and I'm not sure what command(s) I would need to use to imitate what the update_cognitive_models script already does.
I've done the following to try and track down the issue:
Removed the Chitchat.lu, Chitchat.qna, Faq.lu, and Faq.qna files from the .\Deployment\Resources\QnA\en\ folder and let the Update_Cognitive_Models -RemoteToLocal script do its thing. I still have the same error. (I'm not sure if my remote to local syntax is correct there, I set it to true in the script parameters).
Param(    [switch] $RemoteToLocal = $true, ...

I don't know if I'm providing the right info to be helpful; let me know if you want to see anything else. That stack overflow post I made has some more info in it; I can duplicate it here if need be.

@CoHealer
@v-kydela
@stevengum
(Sorry, not sure if this post is still watched by anyone or not)

Apologies @roythompsonuno , this fell off radar after closing amidst an SDK version release here. I'll reopen & @v-kydela will assist you when he is back online this week.

@roythompsonuno - We do prefer that closed issues remain closed, so the thing to do in these situations is open a new issue and link to the closed one. I will try to answer your question anyway.
My colleague linked you to this closed issue: https://github.com/microsoft/BotFramework-Emulator/issues/1586

Currently, the LUIS & QnA inspectors (spawned when clicking a LUIS or QnA trace) still require a .bot file with a corresponding service entry in order to work. They cannot be used with a bot that's connected solely via endpoint.

My suspicion is that the error message is caused by the QnA Maker service not being present in the bot file. If you have no intention of using a bot file or adding your QnA Maker service to it, it is safe to ignore the error message. I will now close this issue again.


New

Making a pull request to review the steps I added into index.md, which obviously didn't work. Looks like I can only make a pull request from Github and not via Gitbash?


New recipe for Daphnia

Current coverage is 100.00%

Merging #58 into master will not affect coverage as of 7d9953b

@@            master     #58   diff @@
======================================
  Files           17      17
  Stmts         1917    1918     +1
  Branches       193     192     -1
  Methods          0       0
======================================
+ Hit           1917    1918     +1
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 7d9953b

Powered by Codecov. Updated on successful CI builds.


improvement(GithubIssues.svelte): Pagination, filtering and ordering support

This change improves the way issues are presented in the issue view -
adding pagination for issue lists over 10 items, filtering bar to filter
issues by name, repo and number and ordering support - currently adding
Title, repo and Date (with ascending/descending orders)
Fixes #231

https://github.com/scylladb/argus/assets/7761415/4fc858da-84a2-40d9-8365-a65a05e306fa

Can we filter also by issue status?
(E.g showing only the opened ones).
Also, I need the combination of filters, e.g filter only open issues of Scylla or Scylla-enterprise.


Read configuration file specified in a command line argument

Signed-off-by: Mitsutaka Takeda<EMAIL_ADDRESS>

Description of the Change
Currently a config file path passed in the command line argument is ignored. This commit read the config file path specified in the argument.

Benefits
It behaves as described in cli help.

Possible Drawbacks
None.

Usage Examples or Tests [optional]
As described in cli help, pass the path to config file.

Alternate Designs [optional]

@takeda-so thank you for the first Iroha 2.0 contribution from the community.
Unfortunately @eadventurous made these changes in #639 yesterday and we didn't saw your PR (it has no iroha2 label our team looking at).
Please accept our apologies. Next time you can request a review directly from me or @eadventurous and put iroha2 label on your PR.

@takeda-so to prevent such a cases in the future we made #642 which will automatically assign me or @eadventurous to all PR into iroha2* related branches.

Thank you for the message. No problem 😄


Readonly api

We should refactor most (if not all) stryker-api interfaces to be readonly. This has no effect on the resulting JavaScript. Instead, it has the effect that our TypeScript code interacting with our API will not have undesired side effects.

Hello! I'm looking for a first open source project to contribute to- may I give this issue a shot?

Hi bicep, awesome to have you onboard the Stryker train. I think in hindsight that this issue might not be that beginner friendly. You might wanna take a look at others with this label. Issue #336 might be the simplest

Is there a quick guide regarding all the different source files and what they do?

Awesome that you want to help! We don't have a description of all files right now.
I would like to point out that Hacktoberfest will start in a few days. It may be fun for you to sign up :) More info: https://hacktoberfest.digitalocean.com

@bicep As @simondel already explained, we don't have a clear description right now. You can take a look at our contribution guide to see how to run stryker locally.
You can start by taking a look at the main runMutationTest method in the Stryker class. If you have any questions: don't hesitate to ask via gitter, email or the issue at hand. We're happy to help.

I've renamed this issue as the options object is the only thing that should become readonly.
Now that I have taken a closer look, i think this we should remove the Config class all together. It doesn't add any value. The implementation of the set(options: Partial<StrykerOptions>) method is the only thing in there and it doesn't belong in the stryker-api. In fact, it isn't serializable using the surrial module.
I'll update the main issue with a todo list.

@nicojs Do we also want to update the RunResult and RunOptions and such?

Yes, they can be readonly as well

I'm closing this issue. It's still a nice-to-have, but we don't want to invest time in this a.t.m.


Form on login page is a bit broken

Closes #122

Removed focus trap on login form: it's bugged and isn't good for a11y anyway; we don't want legally blind people (or linux fullmaster keyboard only pros) to get stuck here
Added dependency to the vee validate nuxt plugin (+ resolved conflicts)

I'm wondering if we shouldn't focus the username input on the page loading?

I'm wondering if we shouldn't focus the username input on the page loading?

we could, but seeing other websites, this isn't really a norm


Remove pypi deploy via Travis.

Doing so requires configuration that is not compatible with our workflow. Comments
stored in TE-1045.

@jzoldak

:+1: :rocket:


Overriding http return stats on exception

I notice that you can update the returned status on success, but can you do the same thing on error? We typically use 422s instead of 400s for malformed requests.
Thanks!

Hi,
it is not possible yet. But I think this would be a good new feature. I will add the possibility to modify status code when ValidationError occurs using flask app config value FLASK_PYDANTIC_VALIDATION_ERROR_STATUS_CODE.
Does this solve your problem?
Jirka

Hey thanks for responding so quickly to this! I was thinking maybe something in the decorator itself? A config value like this could also work though

Fine, it's released in version 0.0.7. Feel free to reopen or create another issue if you wished to use more adjustable way (such as decorator parameter etc.)

Okay I'll see if this works for now. Thank you!


PyPDF in Windows_10

PyPDF2 is deprecated.
You must install this dependency:
pip install pypdf
And then, change this  in code-source:
from pypdf import PdfReader, PdfWriter

You are right. During development I saw that warning and installed the other library. Will fix.

@yerkobits , I tried to use the new version but it won't support heavier encryption. The one I've used with PyPDF2 should be working eventough is deprecated.
Have it compiled and worked ok for you with PyPDF2?

PyPDF2 doesn't work for me.
(Windows 10, Python 3.1)
pip show pypdf
Name: pypdf
Version: 3.9.1
Summary: A pure-python PDF library capable of splitting, merging, cropping, and transforming PDF files
Home-page:
Author:
Author-email: Mathieu Fenniak<EMAIL_ADDRESS>License:
Location: c:\python310\lib\site-packages
Requires:
Required-by:

Have it compiled and worked ok for you with PyPDF2?

I'll put in a comment in the README.MD file about this edge-case :)
Thanks for testing!

I've migrated to the latest version of pypdf as you suggested. It generated a warning message due to AES encryption with some files, that was annoying, so I suppressed it.


has_many :through with :source broken (undefined method `klass' for nil:NilClass)

Hi there,
I tried using squeel today and it broke my has_many :through associations when I manually specify a :source. e.g.
has_many :memberships, :class_name => "Community::Membership", :dependent => :destroy
has_many :members, :through => :memberships, :source => :user

I don't have time to look into what's going on atm, but thought I'd post my stack trace, in case it's of interest to you.
Thanks,
iHiD
NoMethodError: undefined method `klass' for nil:NilClass
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/rails-7ee15e8a0f84/activesupport/lib/active_support/whiny_nil.rb:48:in `method_missing'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/rails-7ee15e8a0f84/activerecord/lib/active_record/reflection.rb:391:in `block in source_reflection'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/rails-7ee15e8a0f84/activerecord/lib/active_record/reflection.rb:391:in `collect'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/rails-7ee15e8a0f84/activerecord/lib/active_record/reflection.rb:391:in `source_reflection'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/rails-7ee15e8a0f84/activerecord/lib/active_record/reflection.rb:538:in `derive_class_name'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/rails-7ee15e8a0f84/activerecord/lib/active_record/reflection.rb:128:in `class_name'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/squeel-67edfcb84837/lib/squeel/adapters/active_record/relation.rb:59:in `block in infer_association_for_relation_merge'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/squeel-67edfcb84837/lib/squeel/adapters/active_record/relation.rb:59:in `each'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/squeel-67edfcb84837/lib/squeel/adapters/active_record/relation.rb:59:in `detect'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/squeel-67edfcb84837/lib/squeel/adapters/active_record/relation.rb:59:in `infer_association_for_relation_merge'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/squeel-67edfcb84837/lib/squeel/adapters/active_record/relation.rb:46:in `merge'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/rails-7ee15e8a0f84/activerecord/lib/active_record/associations/through_association.rb:17:in `block in target_scope'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/rails-7ee15e8a0f84/activerecord/lib/active_record/associations/through_association.rb:16:in `each'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/rails-7ee15e8a0f84/activerecord/lib/active_record/associations/through_association.rb:16:in `target_scope'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/rails-7ee15e8a0f84/activerecord/lib/active_record/associations/association.rb:88:in `scoped'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/rails-7ee15e8a0f84/activerecord/lib/active_record/associations/collection_proxy.rb:47:in `scoped'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/rails-7ee15e8a0f84/activerecord/lib/active_record/associations/collection_proxy.rb:102:in `method_missing'
	from (irb):1
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/rails-7ee15e8a0f84/railties/lib/rails/commands/console.rb:45:in `start'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/rails-7ee15e8a0f84/railties/lib/rails/commands/console.rb:8:in `start'
	from /Users/iHiD/.rvm/gems/ruby-1.9.2-p180/bundler/gems/rails-7ee15e8a0f84/railties/lib/rails/commands.rb:40:in `<top (required)>'

Hi,
I am also getting error like "undefined method `klass' for nil:NilClass"
in
= nested_form_for(@contractor, :url => contractors_path, :html => {:class => 'form-stacked'}, :method => :post) do |f|

@thiyagarajan this doesn't look related to Squeel. More likely that @contractor is nil in your code.

Thanks, we got resolved. Its due to problem with indentation.


Refs #35982 -- Give the test suite a run without decimal formating.

Per this comment cc @timgraham
I would expect some tests to fail on SQLite which could be a lead on how to write a regression test for #18895 without relying on mocking

No failure on SQLite which means that https://github.com/django/django/pull/18895 can only be tested through mocking.


rm paper

remove some paper related code and add explaination to the root readme (choosing paper as the tag). @SolalPirelli you should tag it yourself as AFAIK I can't push tag to other repos.

rm experiments/, used to generate the graphs in the paper
rm nf/bpf/ & tool/bpf/, need root access
rm nf/dns_aaa_records/, unfinished
rm nf/iptables*/, not working w/ recent iptables heuristic changes

Thanks! I tagged.


Rename project to AmazingPrint

Closes #10

@paddor do you have a moment to give this a review? Thanks!


Sankey diagrams

Adds a sankey diagram timeline view.

Do all the files under /d3-sankey-timeline/.yarn/cache/ need to be in the repo or could they be ignored?

Do all the files under /d3-sankey-timeline/.yarn/cache/ need to be in the repo or could they be ignored?

They don't have to be included in the repo, but the philosophy behind including them is that they ensure everyone who clones the project is working with exactly the same dependencies, and they eliminate the need for running npm install when setting up the project. Here is an explanation of the philosophy: https://yarnpkg.com/features/pnp. I'll leave it up to you if you want to include them or not.


Introduction to the WebDAV Protocol

Part of #148.
To do:

[x] Document XML extensibility

FYI @yarikoptic.  You may be interested in this.


swiftc prints ast dump to stderr instead of stdout when more than one source file is provided

Description
I am trying to generate the ast for all of the files in a project using a single swiftc command with the desired options (including -dump-ast). The AST dump is sent to the stderr instead of the stdout as is the case when only one source file is provided.
Reproduction
File main.swift:
func main() {
    let a = Something(name: "Doe")
    print(a)
}

File Something.swift:
struct Something {
    var name: String

    var description: String {
        return "I am something called \(self.name)"
    }
}

Compilation
swiftc -dump-ast *.swift > full.ast 2> compiler.err

Expected behavior
The AST dump is stored in full.ast (redirected from stdout) and compiler.err contains compiler warning and errors (empty in this case).
Instead AST dump is stored in compiled.err and full.ast
If just one file is compiled (swiftc -dump-ast Something.swift > full.ast 2> compiler.err) then the result is as expected.
Environment
% swiftc -version swift-driver version: 1.115 Apple Swift version 6.0 (swiftlang-<IP_ADDRESS>.10 clang-16<IP_ADDRESS>) Target: arm64-apple-macosx15.0
Additional information
No response

I was looking into this a bit and while I have not fully root-caused it, I think one thing that is off is this snippet:
https://github.com/swiftlang/swift/blob/4be399003e70c0cfc22c1aa40b0dd6de26d9a42d/lib/AST/ASTDumper.cpp#L1888-L1891
The newline should go to the output stream OS and not llvm::errs. I am trying to look more into this.

I spent some more time into this and to me, this does not seem to be an issue with swift frontend itself, but possibly has to do with how the shell is redirecting the output.
When you run swiftc -dump-ast *.swift > full.ast 2> compiler.err, this is not a singular command but instead, gets broken up into two commands. You can see that by running below:
% swiftc -dump-ast *.swift > full.ast 2> compiler.err -###
swift-frontend -frontend -dump-ast -primary-file main.swift test.swift ... -o -
swift-frontend -frontend -dump-ast main.swift -primary-file test.swift ... -o -

If you run this commands individually one after the other, they will write the output to stdout (or redirect to the file that stdout points to). If you do a single file, there is just one swift-frontend command that gets run, and so the output redirection works correctly.
% swiftc -dump-ast main.swift -###
swift-frontend -frontend -dump-ast -primary-file main.swift ... -o -

Which is why I don't think there is an issue with swift frontend (except the minor \n being printed to stderr).
Tagging @hborla @xedin  (owner of lib/AST) for viz.


[Docs]: Terraform general updates

type: update
status: pending
location: kubefirst/terraform.md
issues
Terraform & Atlantis: 'kubefirst/terraform.md'

video is enormous here
info on webhooks
info on security measures
info on audit log
info on state object versioning
atlantis in realtime with gitlab
atlantis.yaml
entrypoint walkthroughs

Page: base

Page: users

Page: github

Page: gitlab

Page: vault

The ones that were still relevant were moved into individual issues as I was moving notes from mkdocs.yml from @johndietz into issues.


Roll engine 5aff31194809..d0319631bfb8 (21 commits)

https://github.com/flutter/engine/compare/5aff31194809...d0319631bfb8
git log 5aff31194809..d0319631bfb8 --first-parent --oneline
2020-03-07<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from 95geB... to hW33F... (flutter/engine#17006)
2020-03-07<EMAIL_ADDRESS>Roll src/third_party/skia 24a8e9e170f7..cf573d844da6 (4 commits) (flutter/engine#17004)
2020-03-07<EMAIL_ADDRESS>Started deleting .DS_Store files so licenses can run on mac os x. (flutter/engine#16998)
2020-03-07<EMAIL_ADDRESS>doxygen tooling updates and doxygen for FlutterCodecs.h (flutter/engine#16947)
2020-03-06<EMAIL_ADDRESS>[web] Engine integration test (flutter/engine#16930)
2020-03-06<EMAIL_ADDRESS>Roll src/third_party/skia 6e58290ba639..24a8e9e170f7 (5 commits) (flutter/engine#16996)
2020-03-06<EMAIL_ADDRESS>documented fluttertexture.h (flutter/engine#16950)
2020-03-06<EMAIL_ADDRESS>fushia licenses fix (flutter/engine#16992)
2020-03-06<EMAIL_ADDRESS>Roll src/third_party/skia beaaf4700f50..6e58290ba639 (9 commits) (flutter/engine#16990)
2020-03-06<EMAIL_ADDRESS>remove 10s timeouts from tests (flutter/engine#16988)
2020-03-06<EMAIL_ADDRESS>Roll src/third_party/skia 54de2fa48d85..beaaf4700f50 (3 commits) (flutter/engine#16987)
2020-03-06<EMAIL_ADDRESS>Roll fuchsia/sdk/core/linux-amd64 from cTw2C... to K1wwe... (flutter/engine#16984)
2020-03-06<EMAIL_ADDRESS>Roll src/third_party/skia 74055566bd14..54de2fa48d85 (3 commits) (flutter/engine#16983)
2020-03-06<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from J6ct_... to 95geB... (flutter/engine#16982)
2020-03-06<EMAIL_ADDRESS>[web] Fixes IE11 crash due to missing canvas ellipse support and font polyfill failure (flutter/engine#16965)
2020-03-06<EMAIL_ADDRESS>Fix issue viewdidload call while init FlutterViewController (flutter/engine#16672)
2020-03-06<EMAIL_ADDRESS>[fuchsia] fix broken flows when under high load (flutter/engine#16834)
2020-03-06<EMAIL_ADDRESS>Roll src/third_party/skia 93a2a6b8badb..74055566bd14 (2 commits) (flutter/engine#16981)
2020-03-06<EMAIL_ADDRESS>Roll src/third_party/skia 012f8497802e..93a2a6b8badb (4 commits) (flutter/engine#16974)
2020-03-06<EMAIL_ADDRESS>fix shadows and mask filter blurs (flutter/engine#16963)
2020-03-06<EMAIL_ADDRESS>Roll fuchsia/sdk/core/linux-amd64 from cXgMr... to cTw2C... (flutter/engine#16970)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Gold has detected one or more untriaged digests on patchset 21.
Please triage them at https://flutter-gold.skia.org/search?issue=52164.

Commit queue failed; closing this roll.


predictor not support 'cpu'

The predictor does not support 'cpu', the current version will throw an exception:
https://github.com/microsoft/FOST/blob/main/fostool/pipeline.py#L281)

In current version we are not support and not recommend use CPU to build model due to the NN model set, we will left this issue open and add cpu support in later version.

@chlxydl Thanks for the info. It seems strange that cpu in supported in training but not in inference...


new installation using cmake need more instructions/hints when using local installation of gsl. Simple setting of env variable GSL_ROOT_DIR does't  seem to work.

new installation using cmake need more instructions/hints when using local installation of gsl. Simple setting of env variable GSL_ROOT_DIR does't seem to work

our tech support helped to get gsl installed where cmake expect to find it so I got CCL installed and all check_ccl  passed: RESULTS: 94 tests (94 ok, 0 failed, 0 skipped) ran in 236511 ms
But it will be useful to have a few lines in the installation to indicate how cmake can use another specified directory for gsl.

This is outdated with the new v2 installation, so I'm closing this issue.


HPA fails after upgrade

Noticing a consistent pattern of E2E failures to use horizontal pod autoscaler after a cluster upgrade. Symptom is broad, doesn't seem to be isolated to particular k8s version upgrade paths (i.e., broken from 1.8 to 1.9 and from 1.11. to 1.12)

fixed for most cases


Unable to disable HTTP2 in Blazor Server

The proposed solution to my previous problem (#15123 ) was to disable HTTP2. To do so, I'm using the following Program.cs code:
using System;
using System.Collections.Generic;
using System.IO;
using System.Linq;
using System.Net;
using System.Threading.Tasks;
using Microsoft.AspNetCore;
using Microsoft.AspNetCore.Hosting;
using Microsoft.Extensions.Configuration;
using Microsoft.Extensions.Hosting;
using Microsoft.Extensions.Logging;

namespace Sel.Automate.WebAppServer
{
    public static class Program
    {
        public static void Main(string[] args)
        {
            CreateHostBuilder(args).Build().Run();
        }

        public static IHostBuilder CreateHostBuilder(string[] args) =>
            Host.CreateDefaultBuilder(args)
                .ConfigureWebHostDefaults(webBuilder =>
                {
                    webBuilder.UseStartup<Startup>();
                });

        public static IWebHostBuilder CreateWebHostBuilder(string[] args) =>
            WebHost.CreateDefaultBuilder(args)
                .ConfigureKestrel((context, options) =>
                {
                    // Disable HTTP/2
                    // If it's only your development machines that have this problem, you can wrap
                    // this in `if (context.HostingEnvironment.IsDevelopment())` or some other condition
                    // that is only true in development.
                        options.Listen(IPAddress.Loopback, 5001, listenOptions =>
                        {
                            listenOptions.Protocols = Microsoft.AspNetCore.Server.Kestrel.Core.HttpProtocols.Http1;
                        });
                });
    }
}

Both FF/GC are still not able to load the Blazor Server example due to ERR_HTTP2_INADEQUATE_TRANSPORT_SECURITY. Any thoughts? Thanks!
OS: Windows 10.0.17763 Build 17763

/cc @halter73 \ @Tratcher

@ajillepalli that code doesn't even enable HTTPS, so there's no way to get ERR_HTTP2_INADEQUATE_TRANSPORT_SECURITY. I suspect you did not rebuild and deploy after that change.

Wait, why do you have both CreateHostBuilder and CreateWebHostBuilder? You should combine those.

@Tratcher Thanks for your suggestion. I tried combining both but the compiler isn't able to find some method names (ConfigureKestrel, Listen, Protocols) in the given context.
And yes, I did rebuild after adding in the code to disable HTTP2. Edge is able to load the Web App but not FF/GC.

Did you try the example I gave?

Or here's an easier option, update your appsettings.json like this:
{
  "Logging": {
    "LogLevel": {
      "Default": "Warning",
      "Microsoft.Hosting.Lifetime": "Information"
    }
  },
  "AllowedHosts": "*",
  "Kestrel": {
    "EndpointDefaults": {
      "Protocols": "Http1"
    }
  }
}

Trying out https://github.com/aspnet/AspNetCore/issues/15268#issuecomment-545210364 gives me name not found in the current context errors for ConfigureKestrel, Listen, and Protocols.
Editing the appsettings.json did the trick. The web app now loads in both FF and GC. Thanks a lot.

Closing this issue as there's no more action for us to take here.


Implement forgot password template

forgot-password.html located in EventTrakka-Backend/app/email-templates is the template for the email sent to the user to reset their password. Following the designs from the designers, implement this template. all styling should be done in the html file

I'll like to work on this
Kindly assign me

Assigned!


DTSC Blog post

Content review needed:
https://docs.google.com/document/d/1qyG3hQAw89tjtm2FFOTqVA7AZxmydw8U9Qg8psUZtsA/edit?usp=sharing

Shared with Hayley for content review 6/5/23

Status as of 6/5/23:
from Hayley Figeroid
1:45 PM

Thank you! As far as content, looks good. But we need to wait to publish: Sergio was having Tara look into the appropriateness of us using a P card to buy the Facebook ads (vs. traditional procurement). He's on vacation this week. I've reached out to Tara to see if she knows where they landed on that, but the short version is we may need to wait for Sergio to be back next Monday

Blog post published!


Where to put RouterAutoscroll.animationDuration = (myValue); ?

Hello. Where does this go? In routes.js? Or where?
If in routes.js, can you provide an example?
Thank you for creating this plugin!!

I think you can put it anywhere on the client js with:
RouterAutoscroll.animationDuration = 100;
If you put in somewher like "both" you would use:
if(Meteor.isClient){
  RouterAutoscroll.animationDuration = 100;
}

@deborah-ufw Looks like Paul answered this question for us, I'll close this issue for now.


Local echo settings are marked experimental

Are all these settings still "experimental"? This seems like a mature feature

This is a good point. I believe the experimental wording is used to indicate the functionality has some quirks. We will see if @Tyriar thinks it's fine to remove it.

Yep let's do it, I think we just didn't remember to remove


Lag if drag a seekbar

Lag in equalizer view animation when drag a seekbar in the layout.
My seekbar is sincronized change value of volume stream audio

Hi,
Can you make a short video/gif to show the issue?

I'm also getting this issue. Only on a Pixel XL. I've used a Samsung note 3 and I cannot reproduce the issue on this phone. Weird !

it could be hard for me to see what's going on. Are you able to make a short video about the issue?


Try CI

Codecov Report
Patch coverage: 27.49% and project coverage change: +16.37 :tada:

Comparison is base (f3163b4) 4.14% compared to head (b5734de) 20.52%.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1209       +/-   ##
===========================================
+ Coverage    4.14%   20.52%   +16.37%
===========================================
  Files          24       21        -3
  Lines        7140     5487     -1653
===========================================
+ Hits          296     1126      +830
+ Misses       6844     4361     -2483

Impacted Files
Coverage Δ

nanomq/bridge.c
0.00% <0.00%> (ø)

nanomq/cmd_proc.c
0.00% <0.00%> (-12.50%)
:arrow_down:

nanomq/conf_api.c
0.00% <0.00%> (ø)

nanomq/rest_api.c
0.00% <0.00%> (ø)

nanomq/web_server.c
0.00% <0.00%> (ø)

nanomq/webhook_inproc.c
0.00% <0.00%> (ø)

nanomq/apps/broker.c
20.69% <7.89%> (+2.71%)
:arrow_up:

nanomq/pub_handler.c
49.85% <25.64%> (+48.94%)
:arrow_up:

nanomq/sub_handler.c
61.37% <45.00%> (+16.54%)
:arrow_up:

nanomq_cli/dds2mqtt/mqtt_client.c
64.27% <62.50%> (+64.27%)
:arrow_up:

... and 5 more

... and 9 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Nodes get deselected onMouseUp

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Reproduction link

Steps to reproduce
See link to codesandbox

Use built in behavior 'click-select'
Select a node or multiple nodes
Start mouse click on some other HTML element
Release mouse click on the G6 <canvas /> board

Expected - Nodes still selected
Actual - Nodes de-selected

Environment
Info

g6
4.3.2

System
Windows

Browser
Chrome, Firefox, Edge

Basically a random click that started anywhere but got released on the board will de-select the selected nodes. In the application we're building this provides to be an obstacle as on top of the the G6 canvas we have other React components (topbar, sidebar, etc), but clicks originating there should not affect the state of the graph
In the documentation I did not find precise description how the "de-selection" works or what is the outcome in this case.
shouldBegin(e) gets triggered only on selecting nodes, not de-selecting, so does not help me here

solved by v4.8.0


Do not use maven.build.timestamp for OSGi version of sbt project

Normally the usage of maven.build.timestamp seems to result in a
correct OSGi version but as the ticket shows, it also can result in an
invalid version. The problem is fixed by hardcoding the format of the
generated timestamp to yyyyMMddHHmm. Furthermore we also append the
short git hash to the version. The git hash is not required but may make
it easier to find out against which sources the plugin is built.
Fix #1002667
Review by @dragos

The new version number looks like this:
Bundle-Version: 0.13.8.local-2_11-201603141650-ddf46d0


Introduce Violation trait

For every available rule registry.rs currently defines:

A CheckCode variant to identify the rule.
A CheckKind variant to represent violations of the rule.
A mapping from the CheckCode variant to a placeholder CheckKind instance.
A mapping from the CheckKind to a string description.
A mapping from the CheckKind to a boolean indicating if autofix is available.
A mapping from the CheckKind to a string describing the autofix if available.

Since registry.rs defines all of this for every single rule and ruff has hundreds of rules, this results in the lines specific to a particular rule to be hundreds of lines apart, making the code cumbersome to read and edit.
This commit introduces a new Violation trait so that the rule-specific details of the above steps 2.-6. can be defined next to the rule implementation. The idea is that once all CheckCode/CheckKind variants have been converted to this new approach then the code in registry.rs could simply be generated via a declarative macro, e.g:
define_rule_mapping!(
    E501 => pycodestyle::LineTooLong,
    ...
);

(where pycodestyle::LineTooLong would be a struct that implements the new Violation trait).
The define_rule_mapping! macro would then take care of generating the CheckCode and CheckKind enums, as well as all of the implementations for the previously mentioned mappings, by simply calling the methods of the new Violation trait.
There is another nice benefit from this approach: We want to introduce more thorough documentation for rules and we want the documentation of a rule to be defined close by the implementation (so that it's easier to check if they match and that they're less likely to become out of sync). Since these new Violation structs can be defined close by the implementation of a rule we can also use them as an anchor point for the documentation of a rule by simply adding the documentation in the form of a Rust doc comment to the struct.

I think that this new structure would also simplify contributions since you would no longer need a script (scripts/add_check.py ) to generate the necessary boilerplate but could just copy some example code from CONTRIBUTING.md to some plugin file and add a single line to registry.rs.
And the proposed define_rule_mapping! macro would also address #419.
Since the conversion to the new structure would require many changes ... this would probably best be automated via a single-use Python script.

\cc @squiddy

I think something like this based on traits and macros makes sense and is very likely where we should go (so as to get rid of the current approach in registry.rs, which is very unwieldy).

Ok, great then I'll go ahead with attempting to write a Python script to convert everything in registry.rs to the new structure.

Alright ... I have implemented this in 9 commits ... the commit message of the 3rd commit contains the Python code I used to generate the new structs & trait implementations.
I think this is ready for review ... once reviewed somebody will have to perform these steps again since the main branch has now advanced a couple commits since I started the process. Redoing all of this is quite a hassle and only makes sense when this PR gets merged right after and all other PRs are put on hold in the meantime.

Sidenote: I didn't include the summary function in the Violation trait since I don't think having it makes sense ... I think the message should be a short error message and more thorough documentation IMO belongs in the docstring of the struct (which could then also be shown by e.g. --explain or a dedicated web app, see also #1467).

Great, I'll try to review this soon!

(As for the merging: let's just aim to do a final rebase + force push to this branch once it's in a finished state, then merge without squashing, then I'm happy to be responsible for updating outstanding PRs. Does that sound good?)

Yes that sounds great :)

I promise not to squash :)

@charliermarsh I'm starting the rebase.

Let me know when you want to merge this, then I can rebase the PR for a final time :)

Cool, let's just sort out the final semantics around Violation vs. Diagnostic vs. Rule, then hopefully we can get this merged some time this weekend (or whenever you're available, no pressure).

(I also want to do a quick sanity check to ensure there are no major perf changes prior to merging. I don't expect any, but it's a large change and it's worth validating.)

No noticeable changes, as expected.

I have time right now ... should I do the rebase now?

Go for it. I’m busy for a bit but will merge this next, then rebase any open PRs.

Having the violations defined in one struct like this is so much cleaner. It's a huge improvement.


Split up entities table

We want to break up the one big entities table into an entities table per subgraph. To that end, we'll create a new Postgres schema for each subgraph, and put the entities and entity_history table into that scheme. It should be possible to do that with minimal issues with Diesel, as we can select the subgraph-specific schema by setting Postgres' search_path before querying/modifying one of htese tables.

I love the schema idea, because it's so unobtrusive. I have one question about the first task: Do we even need a mapping table if we use the subgraph deployment ID as the schema name (or suffix). E.g. deployment_${id} or subgraph_${id}?

Another comment: I've marked #795 as a duplicate of this issue, but I think it needs to be incorporated into the plan. Splitting up existing tables with a migration is one thing, but the tables will also have to be created for new subgraphs going forward.

I love the schema idea, because it's so unobtrusive. I have one question about the first task: Do we even need a mapping table if we use the subgraph deployment ID as the schema name (or suffix). E.g. deployment_${id} or subgraph_${id}?

There's a few reasons why I want a separate table for that:

I'd want to use that table to version the schema for each subgraph, so that we do not need to migrate all subgraphs at once (I would stick a version column, and probably a status column to track status of migration for each subgraph into that table). To actually migrate into a split entities table, I am thinking of the following process (i.e., migrating from subgraph schema version 0 to 1):

initial migration only pulls out subgraphs subgraph
over time, we stop ingestion for individual subgraphs, and migrate their data out of the entities table (not quite clear what would trigger that, but I am thinking we have the ingestor for each subgraph check whether it needs migrating, and kick off the migration based on that only fewer than N otehr subgraphs are currently migrating)
when all subgraphs have been migrated, we can drop the entities table; not clear what should happen to unassigned subgraphs, as they would not be migrated with the scheme above

I don't ever want to have to type stuff like select count(*) from qma2wsszn3gsx2hdds47vx3vzudgdblebfjehuvp3j3zmv.entities in psql :wink:

Another question: We also need to set search_path before writing entities, correct, performing reversions etc., correct?

Yes, that's correct; though I am actually not that excited about the whole search_path thing since it will not show up in logs, and therefore make it much harder to follow which queries we are running. I am leaning towards using the dynamic table diesel crate, so that logs will actually show the fully qualified table name, like select count(*) from sgd0.entities where entity='Subgraph'

This work is done as of PR #950 and graph-node release v0.13.0


[WIP] Fix linebreak reconciliation on inlines

@zurfyx is this PR still a thing?

Closing this PR due to staleness! If there are new updates, please reopen the PR.


test: add intl default locale

With this change the testsuite is green. No matter which default locale is set on the local machine.

Thanks Chris!


Allow user to specify namespace in KV

This PR allows users to specify namespace when creating KV store as top level namespace to improve isolation between services.
This PR did not use the namespace feature in etcd because we need to upgrade to go 1.8 in order to use that, when we are ready to upgrade Go we can easily change the impl to use etcd namespace
@xichen2020 @jeromefroe  @dgromov

Coverage decreased (-0.3%) to 88.925% when pulling 5ff6cd02e1f269e304053069682e31fdb44e5afa on chao-namespace into 2eb541bb4b0883e27c78f7594f0eda09f8bf9665 on master.

Coverage decreased (-0.3%) to 88.93% when pulling 9fcf502cb0d136e18511496ecbb66f22c08e1f12 on chao-namespace into 2eb541bb4b0883e27c78f7594f0eda09f8bf9665 on master.

Coverage decreased (-0.3%) to 88.881% when pulling 7bdfdb9164cb3050561f4199f59d78cb30d078fd on chao-namespace into 2eb541bb4b0883e27c78f7594f0eda09f8bf9665 on master.


PHP friendlyTime method is not working in Production environment

Describe the bug
when we get the time from database its treated as string so this line cause the problem
$time = (is_string($time) === true && is_int($time) === false) ? strtotime($time) : $time;

https://github.com/zestframework/Zest_Framework/blob/master/src/Common/Formats.php#L88
when i comment this line the method works fine
Requirements

3.0.0-dev
7..3
Hostinger shared hosting
none

@Maikuolan  could you help to solve this issue?

If $time can be an integer or a string, the corresponding annotation @param (int) should probably be @param (int|string) too, because @param (int) implies that the function should only accept integers.
Do we need to check whether the input a string or an integer? I.e., is it possible that this information, when retrieved from the database, could be an integer or a string (i.e., two different possibilities)? Or, should this information in the database be consistent (i.e., always the same type of data)? I ask this, because there is a possibility of refactoring these methods, and maybe changing the logic a little.
If it's always an integer, the method becomes redundant anyway, and probably could be safely removed, I think. If it's always a string, strtotime() could be called directly without worrying about checking the type of data first. But, if it's sometimes an integer and sometimes a string, the check will still be necessary. Generally though, a database design that enforces consistent data types would be easier to maintain in the long term, I think.

@Maikuolan
Do we need to check whether the input is a string or an integer?
yes because there is two posiblity like int or string
is it possible that this information, when retrieved from the database, could be an integer or a string (i.e., two different possibilities)?
yes
It may be string in case if user/developer do like this
$time = "10 September 2000";

means to say English textual datetime.
But in case of int the time is actually timestamp so it will be int always, and timestamp will store in int type but i dont know on returning it will be treated as string
i tried this issue fix when we use case (int) operator when calling this method like
https://github.com/zestframework/Zest_Web/commit/87d19a0ba13f1a13edd4fc99de1910210bd4f13b
and thanks for your suggestion
https://github.com/zestframework/Zest_Framework/commit/b6e969a2477cc7717dc79f5ee38f3d591e0f6523
if there is better knowledge sire you can send pull request or suggest here
Thanks

I have some ideas. I'll test these ideas, and then send a pull request shortly.

I've created a pull request just now (#132). What it does (as an alternative to is_string), is use preg_match to check whether any non-numeric characters exist in the variable. If non-numeric characters exist, it calls strtotime (i.e., it treats it as a string). If non-numeric characters don't exist, it calls nothing (i.e., it treats it as an integer). Seems to work for me during local testing.

@Maikuolan  yes, its works for me to, also now we dont need to use explicit type casting
Thanks you so much.


static code analysis

Configure static code analysis
https://docs.platformio.org/en/latest/advanced/static-code-analysis/index.html

We will use the inbuilt platformIO tool to periodically analyse memory management. Most of the checks are right now in the form of compiler flags


[FEATURE]: Better handling of rending in pie chart when it has too many data points

What is the bug?
When we have a query like: "source=opensearch_dashboards_sample_data_flights | stats avg(FlightDelayMin), sum(FlightDelayMin) by DestCountry, DestCityName" we get a large amount of data. When we select pie chart all of the data is being overlapped as in screenshot.
What is the expected behavior?
The view of the visualization should be clear and readable.
Do you have any screenshots?

@mengweieric - can we close this one as we discussed in one of the calls that it can be because of the screen resolution?


Force Algolia search to redirect to window.origin URL

Why did you do it?
While publishing a PR, I've noticed the autocomplete search at the header doesn't work on Vercel
Apps or the Local Environment.
What happens is that Algolia results are always returning production
URLs, and thereat any search in Local or Review Apps end up redirecting the user to reactjs.org.
This is not the expected behavior while testing, since you can accidentally leave the Vercel App without noticing, introducing all
kinds of confusion.
What changed?
With this small change, we are always enforcing the origin URL to be
used as the host for the redirect.
This will work both for production and any other environment.
Also, in the process of running lint, I received this warning:
6:7  warning  'path' is assigned a value but never used. Allowed unused vars must match /^_/  no-unused-vars

So I've also removed this unused path variable from one of the gatsby-node.js files.
Reproduction of the Issue:
https://share.getcloudapp.com/RBu9NyZb

This is great, this has been a problem for a long time!


Fix text_unicode.c example crashing

The issue causing the examples/text/text_unicode.c to crash on PLATFORM_WEB was the Line Feed (LF) char \x0A on some entries of messages[] (L71, L77, L78, L79, L80, L81, L119, L120) not being escaped when being passed to SetClipboardText() (L195). All chars on those messages resolve to a singular unicode char, however, the Line Feed (LF) resolves to \n (with the \ plus n) which causes the unescaped error on web.
This can be solved by escaping the \x0A to \\n before passing it to SetClipboardText() if on PLATFORM_WEB (R195-R202).
@raysan5 Could you please check if that MemFree() call is correct? I saw the warning on the header about TextReplace() (L1417) but I'm not sure if I did that right.
Also (unrelated) small fix to float textOffsetY (R352).
Tested these proposed changes on Chromium (version 115.0.5790.170 64-bit) and Firefox (version 115.1.0esr 64-bit) both running on Linux (Linux Mint 21.1 64-bit).
Fixes https://github.com/raysan5/raylib/issues/3249.
Edit 1: added line marks.
Edit 2: forgot to add the test environment.

Follow up issue
There appears to be some other problem on SetClipboardText() for web. Passing an unescaped char to it shouldn't cause the entire program/game to crash on web. Looking at its code (L2150) I expected it just not to finish, but not crashing.
I assembled a minimal reproduction code to test the issue:
#include "raylib.h"

int main(void) {
    InitWindow(800, 450, "test");
    SetTargetFPS(60);
    while (!WindowShouldClose()) {

        // Works
        if (IsKeyPressed(KEY_ONE))   SetClipboardText("1\\n2");

        // Crashes
        if (IsKeyPressed(KEY_TWO))   SetClipboardText("1\n2");

        BeginDrawing();
        ClearBackground(RAYWHITE);
        EndDrawing();
    }
    CloseWindow();
    return 0;
}

Note: Regardless, the text_unicode.c fix on the PR was still necessary because text_unicode.c should still pass a valid string to SetClipboardText() so it can complete successfully.

@ubkp The issue is actually in SetClipboardText(), not properly supporting UTF-8 string data as it should, or at least not on PLATFORM_WEB. I think the easiest solution for this example is just removing that line. It's not needed and I don't know why it is called.
SetClipboardText() on web could be threated as another separate issue.

@raysan5 Adjusted the fix as instructed (L190, L195).
I'll open a new issue about the SetClipboard().

@ubkp thanks for the review!


Allow value_template to replace state in glance cards

I would like to override the state value displayed on glance cards.
Instead of detected/clear, the default for class: motion sensors, I would like to use a template to instead show the number of minutes since the last motion was detected. So instead of "clear", it would say "5 mins ago" if motion was detected 5 mins ago.

I would like something like this:
cards:
  - type: glance
    title: Motion Sensors
    entities:
      - entity: binary_sensor.kitchen_motion
        name: Top Floor
        # Instead of just clear/detected, show last activation time:
        value_template: {{ states.binary_sensor.kitchen_motion.last_changed | timeago }}

So use a template sensor instead?!

I think that templating is the wrong solution here. Allowing to render last changed instead of state could be an option. Like we allow a similar thing soon as second line for entity row

@c727 balloob asked him to open this. Templates are the wrong request, but maybe we could add a config option for last-changed and use ha-relative time like you did with the new row elements?

I was asked to open this issue in the discord chat.
From what I know of the design principles behind lovelace, it seemed wrong to bloat the system with more and more templated copies of existing sensors just for display purposes.
🤷♂

I've adjusted the title and re-opened the issue. Let's forget templates were ever mentioned 😉

would rather have entities display attributes instead of doing value_templating

type: attribute-label
entity: states.sun.sun.attributes.elevation


2.16 only: add missing some error checks in ECP and bignum

This is a backport of https://github.com/ARMmbed/mbedtls/pull/3512 of sorts, fixing some bugs that I discovered while working on the optimizations.
Needs backport to 2.7 where applicable.

The CI only fails in (1) compat.sh in Travis which was fixed in the meantime by #3590 (2) armcc license issue, which is independent from this PR.

I'm going to add a changelog entry, make the (trivial, I think) 2.7 backport, and rework https://github.com/ARMmbed/mbedtls/pull/3512 to be standalone.


semicolon is optional - unless computed

Closes https://github.com/facebook/flow/issues/1698 and https://github.com/facebook/flow/issues/1682

Mac Build fail can be ignored I guess?

Can you add some tests to src/parser/test/hardcoded_tests.js You can run them from the src/parser directory via make test-ocaml. See #1681 and #1703 for examples of adding these tests.

Added 2 Parser tests.

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.


It doesn't start training

Thank you for your great effort!
Yolo doesn't start training
I get the following message:
Using TensorFlow backend.
Generating anchor boxes for training images and annotation...
/home/h/yolo_v3/yolov3-tf2/enviroment/yolito/lib/python3.7/site-packages/imageai/Detection/Custom/gen_anchors.py:15: RuntimeWarning: divide by zero encountered in double_scalars
similarity = wh/(c_wc_h)
Theses folders are created (cache, json, logs, models)  but it doesn't seem to start training after the message below appears, @OlafenwaMoses  should I wait more, is it something related to my tensorflow version?
tf_version -> 1.13
keras -> 2.3.0
(I've also tried with other versions  (tf 2.0 , tf 1.5) )

It may have started, but it can some time before it gives you any output.
Check that ImageAI is in its last version
Are you using a GPU?

Thank you @rola93
This was the problem : RuntimeWarning: divide by zero encountered in double_scalars
similarity = wh/(c_wc_h)
It was due to the way I was generating the VOC files, even thouhgt neither the width nor the height were 0 , the values weren't not correct, I fixed that , and now i'm training hoping to get good results
closing this issue.


Invalid date format

The date format in the dob field returned by this API is unsupported by Safari, in which it says "Invalid Date" after passing it to the JavaScript Date constructor. Currently, it's a string formatted like "1983-07-14 07:29:45". In my opinion, this API should leave the date formatting up to the consumer, and since this API currently returns the date as a string, it inherently comes with its own formatting.
I propose that to mitigate this, the dob field should be either a UNIX timestamp, or potentially a UNIX timestamp with millisecond resolution. UNIX timestamps are a common, well-known, and widely supported method of representing a specific moment in time. However, JavaScript's Date constructor takes a UNIX timestamp with millisecond resolution (instead of the common second resolution), so a decision would have to be made as to if JavaScript consumers must multiply the result by 1000 before passing it to the Date constructor, or if consumers that support normal UNIX timestamps must divide by 1000 before constructing their respective representation of a moment in time.
Regardless, I think that this field should be changed to either a standard UNIX timestamp or one with millisecond resolution so that it's easier for consumers to parse and easier for consumers to format to their users' respective preferences and locale

The date format follows (or attempted to follow) ISO 8601 standards. It looks like there is a T missing in the date format that causes Safari to fail to parse the string. This'll be fixed in the next update.
In regards to the formatting, randomuser.me used to use the UNIX Epoch time, but this was changed to the current format in the current version, 1.1. The issue that sparked this change can be found here, but basically it was impossible for anyone to be born before 1970. To fix this, the format was changed to ISO 8601 since it would allow for dates pre 1970 and be an unopinionated format.


not able to create batch process ACI container

team, while trying to build the ACI container image for batch process update to cosmos db i receive the below error. Tried to use the latest sift ("sift": "13.1.10",) version as well, but no luck. Any help is much appreciated.
npm ERR! sift not accessible from mongoose
npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2020-07-12T12_49_27_888Z-debug.log

I made an update to the package-lock.json and it started working for me. Try pulling latest and trying again

@chzbrgr71 shall we consider this a non-issue and close?

Yes


Updating Rule: HTML smuggling with atob and high entropy

Updating to add a new atob detection.
This will need some FP testing.

We might need to add every permutation of the "atob" string in unicode escape sequences... thoughts?

Not sure how tight you want this to be specifically, but in addition to document write and atop. a.download and download wouldn't be terrible additions to this rule.. Thoughts?
Example
  type.inbound
  and any(attachments, .size <= 20000 and
      (
          .file_extension in~ ("html", "htm") or
          .file_extension in~ $file_extensions_common_archives or
          .file_type == "html"
      )
      and any(beta.binexplode(.),
          .scan.entropy.entropy >= 4.5
          and (
              all(["download", "document", "write", "atob"], . in ..scan.javascript.identifiers)
              // usage: document['write'](atob)
              or any(.scan.strings.strings, strings.ilike(., "*document*write*atob*"))
              or any(.scan.strings.strings, strings.ilike(., "*document*write*atob*"))
              // usage: some_var = atob();
              or any(.scan.strings.strings, strings.ilike(., "*=*atob*;"))
              or any(.scan.strings.strings, strings.ilike(., "*a.download*=*"))
              // usage: obfuscating "atob"
              or any(.scan.javascript.identifiers, strings.ilike(., '*ato\u0062*'))
          )
      )
  )

Sounds like a good idea!

Replaced the all function with a count of the document, write, atob == 3 to avoid any issues with vacuous truth.
Rule identified 6 known samples, and matched on the delivr2 samples in the reference that we were previously missing!
Let's ship it! 🔥