vm-test-run-misc fails, blocking nixos-unstable(-small)

Describe the bug
https://hydra.nixos.org/build/131866531
nixos/tests/misc.nix

$ nix-build -A nixos.tests.misc.x86_64-linux nixos/release-small.nix
...
(0.03 seconds)
(0.08 seconds)
whether kernel.poweroff_cmd is set
machine: must succeed: [ -x "$(cat /proc/sys/kernel/poweroff_cmd)" ]
(0.01 seconds)
(0.01 seconds)
whether the blkio controller is properly enabled
machine: must succeed: [ -e /sys/fs/cgroup/blkio/blkio.reset_stats ]
machine: output:
Test "whether the blkio controller is properly enabled" failed with error: "command `[ -e /sys/fs/cgroup/blkio/blkio.reset_stats ]` failed (exit code 1)"
error:
Traceback (most recent call last):
  File "/nix/store/pjh4wascfzzsdsihbv819qkpk4fh5nrv-nixos-test-driver/bin/.nixos-test-driver-wrapped", line 897, in run_tests
    exec(tests, globals())
  File "<string>", line 1, in <module>
  File "<string>", line 44, in <module>
  File "/nix/store/pjh4wascfzzsdsihbv819qkpk4fh5nrv-nixos-test-driver/bin/.nixos-test-driver-wrapped", line 422, in succeed
    raise Exception(
Exception: command `[ -e /sys/fs/cgroup/blkio/blkio.reset_stats ]` failed (exit code 1)
cleaning up
killing machine (pid 6)
(0.00 seconds)
builder for '/nix/store/q0vgahcsaya1yjhn1khqj41difgfia9l-vm-test-run-misc.drv' failed with exit code 1

To Reproduce
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute: `nixos.tests.misc.x86_64-linux`
# a list of nixos modules affected by the problem
module:

cc @flokli is this related to the cgroupsv2 PR?

Looks like it.
Before:
whether the blkio controller is properly enabled
machine: must succeed: [ -e /sys/fs/cgroup/blkio/blkio.reset_stats ]
machine: output:
Test "whether the blkio controller is properly enabled" failed with error: "command `[ -e /sys/fs/cgroup/blkio/blkio.reset_stats ]` failed (exit code 1)"
error:
Traceback (most recent call last):
  File "/nix/store/pjh4wascfzzsdsihbv819qkpk4fh5nrv-nixos-test-driver/bin/.nixos-test-driver-wrapped", line 897, in run_tests
    exec(tests, globals())
  File "<string>", line 1, in <module>
  File "<string>", line 44, in <module>
  File "/nix/store/pjh4wascfzzsdsihbv819qkpk4fh5nrv-nixos-test-driver/bin/.nixos-test-driver-wrapped", line 422, in succeed
    raise Exception(
Exception: command `[ -e /sys/fs/cgroup/blkio/blkio.reset_stats ]` failed (exit code 1)
cleaning up
killing machine (pid 9)
(0.00 seconds)

After changing default for systemd.unifiedCgroupHierarchy to false, the test succeeds.
Which, well, totally makes sense :P

Proposed patch, but I'd like someone closer to the space to make a judgement:
diff --git a/nixos/tests/misc.nix b/nixos/tests/misc.nix
index ae150553273..40661cdca0a 100644
--- a/nixos/tests/misc.nix
+++ b/nixos/tests/misc.nix
@@ -88,8 +88,8 @@ import ./make-test-python.nix ({ pkgs, ...} : rec {
       with subtest("whether kernel.poweroff_cmd is set"):
           machine.succeed('[ -x "$(cat /proc/sys/kernel/poweroff_cmd)" ]')

-      with subtest("whether the blkio controller is properly enabled"):
-          machine.succeed("[ -e /sys/fs/cgroup/blkio/blkio.reset_stats ]")
+      with subtest("whether the io cgroupv2 controller is properly enabled"):
+          machine.succeed("grep -q '\\bio\\b' /sys/fs/cgroup/cgroup.controllers")

       with subtest("whether we have a reboot record in wtmp"):
           machine.shutdown

yeah, that location looks different with unified cgroups, looking in /sys/fs/cgroup/cgroup.controllers looks probably right.
I didn't check nixos/tests/misc.nix apparently checks for the presences of the blkio controller :shrug:
Thanks for catching that! Could you file a PR?


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_9ek6iewe42/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_9ek6iewe42/clone/recipe/meta.yaml:22: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_9ek6iewe42/clone/recipe/meta.yaml:32: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_9ek6iewe42/clone/recipe/meta.yaml:38: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_9ek6iewe42/clone/recipe/meta.yaml:38: missing_dev_url: The recipe is missing a dev_url
WARNING: /tmp/abs_9ek6iewe42/clone/recipe/meta.yaml:38: missing_description: The recipe is missing a description
ERROR: /tmp/abs_9ek6iewe42/clone/recipe/meta.yaml:38: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found


Triggers are skipped when there are a large number of events

The problem(s)
Events (objects) are returned in descending order of id, which has two consequences:

In most cases this will mean the most recent event/created object results in the first trigger/action. I think for most cases users will want to start with the oldest item.
If my page_size is set to 25, and 30 items are created since my endpoint is last polled, as things stand:

the final 25 are forwarded on to Zapier
The latest object-id is updated
The first 5 objects don't trigger any actions and are lost forever

A proposed solution

Order the payload returned to Zapier in ascending order
A problem with this approach is that if you already have a large number of objects, you obviously don't want to have to iterate through all of them. Therefore:
If no latest-object-id exists, return just the last object and update the latest object id for that scope. (This will work well, since each time a uses a trigger for the first time Zapier will prompt them to try a 'test' poll. Doing this test poll, will update the object-id, and set them up ready to receive subsequent events.)

The "reverse chronological order" is actually part of the Zapier trigger spec:

To make a standard trigger, use an API endpoint that lists new items in an array sorted in reverse chronological order. These are typically the most common API endpoints to read data from a platform. If your API lists items in a different order by default but allows for sorting, include an order or sorting field in your API call.

https://platform.zapier.com/docs/triggers


Set range of Lidar to 50m

Lidar range is in centimeters. Therefore 2m is not very helpful.

This change is

I also updated the documentation.
https://github.com/carla-simulator/carla/pull/1523


Fixed #23273 -- Avoided creating django_migrations table when there are no migrations to apply.

ticket-23273
Scenario
In a multi-db setup, 'TEST': {'MIGRATE': False} is used to skip migrations on a connection. Even so, the test runner will run the migrate command to sync the db (see ticket-32012). At that point, the migration executor will attempt to create a django_migrations table even if there are no migrations to apply.
This PR just short-circuits if there are no migrations to apply.
Similar to #8082.

The additional test looks very good, thanks for finding the analogous existing test. At first I thought it was maybe too bad we couldn't just insert this one new assertion into the existing test, but I guess it's good to have separate tests in this instance -- maybe in the future we won't need to call migrate at all with an empty plan to accomplish app syncing, and this new test can still stand 👍🏻

@jacobtylerwalls Thanks for updates :+1:


Not possible to update unknown words using bulk import (without making them known)

Description
It's currently not possible to update unknown words without unknown words becoming known words
To Reproduce
Create some unknown words as a csv
upload the words using bulk import (create new terms, set new terms to unknown)
observe you now have hover translations for these words and they remain unseen
upload the words again using bulk import
observe these unknown words now all have a status of 1

I was playing around with trying to bulk import an initial dictionary of terms for my target language.
there was some trial and error as I improved the formatting or made mistakes
But i couldn't update unknown words because in the code, uploading an existing word, if its status is 0 changes it to 1.
This was quite frustrating, particularly given i was importing hundreds of thousands of terms.
I think - if it's a bulk import, then the status should not change when updating unless the status is included in the csv.

Thank you, I agree it shouldn’t change it. Sorry about that!  Should be a quick fix.

Thanks @s-cork , I changed this in develop, it makes more sense and the code is clearer too.  Will be in the next launch, apologies for the hassle.  Cheers!

Launched in 3.5.1.


Project missing LICENSE FILE

I'm not sure if I follow. It is right here:
https://github.com/google/flax/blob/prerelease/LICENSE

It wasn't present in the root directory of the project. Hence opened this issue.


x/sys/unix/linux: update Linux to v6.6

The Linux kernel v6.6 released today, 30 October 2023.
https://www.kernel.org
We should be able to update the sources in x/sys by changing the unix/linux/Dockerfile and running GOOS=linux ./mkall.sh. This should be done for all architectures.

Well there is no way I can generate the files for all the architectures, so I'll leave this open until someone decides to contribute all the files in a single CL.

We usually don't require issues for these types of updates in x/sys. It's fine to just open a CL.


[iOS][Adaptive Cards)] [Flow Bot Adaptive Card with button isn't getting displayed in Teams for iOS]

Copied bug from: https://github.com/microsoft/AdaptiveCards/issues/3948
Details from original post:
Platform
What platform is your issue or question related to? (Delete other platforms).

iOS

Author or host
Author, sending cards from Flow to Teams using the "post your own adaptive card as the Flow bot to a channel" step in my flow.
Version of SDK
Tried formatting my message in adaptive card version 1.0 and 1.2, but that didn't make any difference.
Details
Please have a look at the attached screenshots. The pictures are of the same message sent by the Flow bot. As you can see, on desktop, the message with the "approve" button looks like you would expect. On iOS, the message isn't displayed at all. You just see the Flow bot icon and the timestamp.

Please look into this, as we have people working on iPads who need to interact with these messages. ---
Document Details
⚠ Do not edit this section.It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3fb4394d-25b1-8752-3996-fe3a2934732d
Version Independent ID: 6d391381-89d0-117c-27cc-00043bd26a36
Content: Cards reference - Teams
Content Source: msteams-platform/task-modules-and-cards/cards/cards-reference.md
Product: msteams
GitHub Login: @surbhigupta12
Microsoft Alias: surbhigupta12

@menbren - We are looking into this issue.We will update you.

@menbren - We tried your adaptive card json in iOS.We get the message on card and also approve button is clickable and going to the mentioned url.

Could you please try to update your teams version and check again.

Thanks for checking. I’m happy to believe that in the 20 months that have passed since I reported this issue it has been fixed already  😄

We are closing this issue for now, Please feel free to reopen it, if required.


Create new.py

What the program do ?

In what programming language it is written?

This pull request do not contains files in specific folders. Please fix it and reopen


[Bug]: sidebar frontmatter control not working

Describe the bug
The sidebar option in frontmatter does not seem to be having any effect. Should be able to set it to hide or never and have the appearance change.
Steps to Reproduce

Create a .md page.
In the frontmatter, include sidebar: hide
Check if the sidebar is hidden

Logs
No response
System Info
No response
Severity
annoyance
Additional Information, or Workarounds
Sidebar visibility can be set on a project-wide basis using a custom +layout.svelte file as described here: https://docs.evidence.dev/reference/themes-and-layouts/

This issue presented itself when I upgraded from v38 to v39. Only seems to be an issue on the home page (index.md). On any other pages setting sidebar: hide in Frontmatter is working as expected and the side bar is collapsable. On the homepage, however, side is static and always present despite Frontmatter setting.


Should we use a different collaborative note-taking tool than Etherpads?

This just came up in the all-hands call.
Etherpads have been the go to solution for collaborative notetaking, but there reasons we might want to have some other solution.

Formatting. Instead of saying "This is heading 1" you can only say "This is bold and underlined". This does not represent semantics very well.
It's centralized. I mean, c'mon. Seriously?

Let's discuss our options!

@dignifiedquire mentioned ipfsbin. Obviously it wins at point 2, but it also doesn't give us strong formatting. Also it does not allow etherpad-like live collaborative editing.
@flyingzumwalt said he used Confluence before, which let's you choose between a WYSIWYG and some markup language. This one has good formatting but is also centralized and can not do etherpad-like collaborative editing.
I mentioned HackMD, which gives you Markdown formatting (no WYSIWYG, though it gives you a live view of what you type) and has etherpad-like formatting.
Finally I guess the solution we all want runs on IPFS. Since etherpad-like systems afaik usually use CRDTs, maybe this is another use case for OrbitDB? Does our solution need to be live collaborative?

I think the feature of etherpad is really nice, that you can (if you want) collaboratively work on them together and at least see the changes live together.
I think our current solution is not that bad actually, to write the notes during the meeting in etherpad over at mozilla.org and the paste it into Github so we can keep it in SCM. Is there a reason this doesn't work anymore?
Formatting issues are more about how you write the notes or how you format them before pasting them into SCM, rather than about the platform itself.

Gonna suggest https://cryptpad.fr

HackMD looks to  me to have everything we like on etherpads, but with a nicer UI

I just want to clarify that the discussion here is not about where to publish notes. It's about what tool to use to take the notes collaboratively in real time while meetings are happening.
We Publish notes for all meetings here: https://github.com/ipfs/pm/tree/master/meeting-notes
Currently we write those notes in etherpad and then the notetaker is responsible for posting those notes to github as a markdown file in a Pull Request.

what tool to use to take the notes

Before we discuss what tool to use, can we agree that what tool we're using now is actually not good-enough? Formatting is a thing no tool will automagically will fix for us.

HackMD looks great!
Problems with etherpad that I've found painful

Doesn't render well on mobile devices -- and almost impossible to edit from mobile
Needs a real-time HTML preview -- this would encourage better formatting and make the notes/agenda easier to read during meetings

HackMD handles both of these really well.

Thanks for all the feedback! TBH I'm not yet too familiar with the pm workflow, just started joining the calls if I can a few weeks ago and had no idea notes were published on Github and the pads are volatile.
I think HackMD is actually suited much better for our use case because you get a Markdown preview that will look quite similar in Github.
Anyway, I'd love to see both the source code versioning and the project management as apps on IPFS, both for us and other projects. Decentralizing the tools we use to build our software seems like a good idea to me. But that's an entirely different story :)

@flyingzumwalt indeed, it does look very good. The mobile point and the live html preview are good points (etherpads non-mobile UI have bitten me before in the past) and after trying it out lightly, I would be happy to try out to use Hackmd instead etherpad for future calls.

Hey @keks! Awesome issue, thanks a ton.
Markdown editing

Formatting. Instead of saying "This is heading 1" you can only say "This is bold and underlined". This does not represent semantics very well.

I don't understand this. We use Markdown in the text - the only issue is that there isn't a preview. It looks like Cryptpad allows a ton of other options through it's WISYWIG editor. However, these won't port over to GitHub, which is where we generally post our notes. They can be seen using GitHub markdown, both in the issues and through the repo markdown viewer. We use Markdown for historical reasons - we used to put all of the notes in issues - which are now gone, as we have a different workflow. However, having a situation-agnostic syntax is useful, and if the WISYWIG editor doesn't port to markdown, I'd be wary of using it. It doesn't look like it does - do you see an option for that?
Centralization
100% agreed it is centralized. However, it doesn't matter all that much for us, I think - we use a few different etherpad instances, and we don't keep the content around longer. We put the pads on GitHub if we're going to save them. What I really think we should do is to host our own instance (see this issue https://github.com/ipfs/pm/issues/230), or to rewrite an etherpad that uses Orbit-DB on the backend.
Good enough
I think that Etherpad is good enough, for now. @flyingzumwalt raises really good points about mobile, and about having a markdown rendering. I'd like to use Cryptpad, if it allowed you to use Markdown and not a WISYWIG editor, but again, unless I'm misreading it, that's not an option. We could look for an etherpad-like preview tool?
We've also had issues with poor internet and etherpads locking up. Another reason to get it on IPFS.
Hack.md
So, having written all that -- This is beautiful. Can we try this out on the next IPFS call? :)

Hey @RichardLitt, thanks for the feedback.
Like I said before, I wasn't aware that the notes are posted on Github after the call. Obviously Markdown does formatting the way I like it, but I like already having the formatting during the call so it's easier for non-notetakers to follow the notes as they are written. I often feel lost when looking at etherpads, but maybe that's just me.
Now that I know that they are being posted to Github after the calls, the centralization idea is kind of void, I agree. I guess it's just a general approach: let's transition our workflow to decentralized apps to show that they are practical. Also using them will make us make them practical. I know this is not a short-term transition though.
re cryptpad: I really like the idea, but it kind of defeats the purpose if you post the keys to decrypt the pad publicly (which we would do). Also I'm not sure an open call like this one needs to be stored encrypted.

Can we try [HackMD] out on the next IPFS call?

yes please :)

I often feel lost when looking at etherpads, but maybe that's just me.

It's not just you. I think one of the reasons for this is that we tend to split between the notes and the agenda, but that may be a separate topic.

I guess it's just a general approach: let's transition our workflow to decentralized apps to show that they are practical. Also using them will make us make them practical. I know this is not a short-term transition though.

I like this very much.
Ok. Let's try HackMD this next week.

Ok. Let's try HackMD this next week.

I would like to suggest https://beta.cryptpad.fr/ Which is made by some cool people and an encrypted version of a collaborative editor.

Yeah the authors are great people and I love the tool. I've been using it a few times in the last months.
However, I stand by my points that (a) ipfs notes are public and don't require encryption and (b) even if you encrypt everything, this doesn't give you anything if you post the key at untrusted places (like a github issue since that is basically public).
While it doesn't support Markdown preview, there is Markdown syntax highlighting, adding some visual structure.
That being said, I'm open to both editors. It's just that I don't feel like encrypted pads bring much value in this context.

@dignifiedquire cryptpad seems to be a wysiwyg rather than markdown editor with preview though. Not sure if that's helpful since we paste the text into Github issues, but maybe it's just me who can't find the edit mode with markdown in cryptpad.

@VictorBjelkholm  I was able to create Markdown one under /code/ version:
https://beta.cryptpad.fr/code/#/1/edit/JaZJYrbdat9PM5uuefrAEw/LRrI68Xe3oLvAyWVtQSW1We2
There is no preview tho.

@haadcode HackMD can be found here. I've prepared this issue text:

Hey there! We just started hackmd at ipfs, after having a discussion about moving on from etherpads here. A few of us were wondering if it might be possible to use IPFS as a backend for HackMD, making it possible to run on a decentralized, distributed internet.

Would you be interested in working with us to make this possible? We would love to be able to use HackMD along with IPFS.

We could also go check out their gitter channel. What do you think? Should I post this?

@RichardLitt that sounds good to me! We could also just ping the folks under https://github.com/orgs/hackmdio/people and see if they can leave some input here :)

I think that HackMD worked great. Shall we use it for the next sprint, again?
If people say yes, I think I can go through and update our docs to use HackMD instead of etherpads going forward.

Yes to using it for the next sprint - this is a great tool.
Thanks for the recommendation @keks!!!

Since we agreed to use a different tool, let's close this off :) Thanks everyone!

I am so glad we use hackpads now. so much nicer.


sqlite example incorrect?

This is from README.md file:
staging:
  dialect: "sqlite3"
  url: "./staging.sqlite"

Running it, I get:
Unknown dialect !

however in connection_details.go there is this:
func (cd *ConnectionDetails) Finalize() error {
	if cd.URL != "" {
		u, err := url.Parse(cd.URL)
		if err != nil {
			return errors.Wrapf(err, "couldn't parse %s", cd.URL)
		}
		cd.Dialect = u.Scheme

which basically overwrites the dialect: from file with u.Scheme, resulting in
		cd.Dialect = ""

and later:
Unknown dialect !

(note the empty dialect name)
It seems to me that the one way would be to use:
staging:
  url: sqlite://staging.sqlite

Alas... then we have this line:
https://github.com/markbates/pop/blob/master/sqlite.go#L28
Which results in file called ?_busy_timeout=5000 (yep, with a question mark) being created..
and staging.sqlite is nowhere to be found.
Thanks!

Finally, after some digging, the example, probably should be:
staging:
  dialect: sqlite3
  database: ./staging.sqlite

Added a Pull request to fix the docs #14


synthesizing issue with Vivado 2018.2

Hi,
I'm trying to get Spatial working with the latest aws-fpga. The F1 instance has the latest AMI 1.50 and Vivado 2018.2 installed.
I dug into the Makefile in the gen folder of Spatial and found out that Spatial uses cl_dram_dma example as a template (glue code) for Spatial generated CL.
So first, I copied cl_dram_dma from the old aws-fpga branch that Spatial is based on and tried it within the new aws-fpga framework and the new Vivdado 2018.2. It works. So I think the design code and scripts you borrowed from that example should still be good with the new aws-fpga and Vivado.
Then, I ran Spatial as before, but synthesizing stopped after a few minutes. Here's the error message I found in 19*.vivado.log file (after disabling the encrption in the encrypt.tcl:
INFO: [Synth 8-6157] synthesizing module 'cl_dma_pcis_slv' [/home/centos/src/project_data/aws-fpga/hdk/cl/examples/Hui_HelloSpatial_hw/build/src_post_encryption/cl_dma_pcis_slv.sv:16]         Parameter SCRB_MAX_ADDR bound to: 64'b0000000000000000000000000000001111111111111111111111111111111111         Parameter SCRB_BURST_LEN_MINUS1 bound to: 15 - type: integer         Parameter NO_SCRB_INST bound to: 0 - type: integer         Parameter NUM_CFG_STGS_CL_DDR_ATG bound to: 4 - type: integer         Parameter NUM_CFG_STGS_SH_DDR_ATG bound to: 4 - type: integer ERROR: [Synth 8-2916] unconnected interface port 'cl_axi_mstr_bus' [/home/centos/src/project_data/aws-fpga/hdk/cl/examples/Hui_HelloSpatial_hw/build/src_post_encryption/cl_dram_dma.sv:1004] INFO: [Synth 8-5534] Detected attribute (* dont_touch = "32'b00000000000000000000000000000000" *) [/home/centos/src/project_data/aws-fpga/hdk/cl/examples/Hui_HelloSpatial_hw/build/src_post_encryption/cl_dma_pcis_slv.sv:81] INFO: [Synth 8-5534] Detected attribute (* dont_touch = "32'b00000000000000000000000000000000" *) [/home/centos/src/project_data/aws-fpga/hdk/cl/examples/Hui_HelloSpatial_hw/build/src_post_encryption/cl_dma_pcis_slv.sv:82] INFO: [Synth 8-5534] Detected attribute (* dont_touch = "32'b00000000000000000000000000000000" *) [/home/centos/src/project_data/aws-fpga/hdk/cl/examples/Hui_HelloSpatial_hw/build/src_post_encryption/cl_dma_pcis_slv.sv:83] ERROR: [Synth 8-6156] failed synthesizing module 'cl_dma_pcis_slv' [/home/centos/src/project_data/aws-fpga/hdk/cl/examples/Hui_HelloSpatial_hw/build/src_post_encryption/cl_dma_pcis_slv.sv:16] ERROR: [Synth 8-6156] failed synthesizing module 'cl_dram_dma' [/home/centos/src/project_data/aws-fpga/hdk/cl/examples/Hui_HelloSpatial_hw/build/src_post_encryption/cl_dram_dma.sv:18]
I'm not familiar with either v/vhd/sv files (that's reason I want to take advantage of Spatial at the very start), so I do need some help here, thanks!

This branch of Spatial supports aws-fpga 1.4.7 and AMI 1.5.0 (Vivado 2018.2):
https://github.com/stanford-ppl/spatial/tree/stefan
For more details, see here:
https://github.com/stanford-ppl/spatial/blob/stefan/temp_f1.md


Fix tutorial rendering.

Fix bug in tutorial rendering logic. Fixes #2189
The issue was after the full page was rendered by markdown, I had HTML in string format, and I needed the first element of each step. I was using regex to parse the HTML. I am now using the contentdiv passed in and temporarily setting the HTML of each step, and extracting the first elements HTML. Using the DOM to do the parsing.

Needs to be cherrypicked into v0

The browser has a DOMParser right? Does your trick influence the HTML layout?
Get Outlook for iOShttps://aka.ms/o0ukef

From: Sam El-Husseini<EMAIL_ADDRESS>Sent: Friday, June 9, 2017 8:32:07 AM
To: Microsoft/pxt
Cc: Peli de Halleux; Review requested
Subject: Re: [Microsoft/pxt] Fix tutorial rendering. (#2257)
@microsoftsam commented on this pull request.

In pxtrunner/runner.tshttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fpull%2F2257%23discussion_r121155824&data=02|01|jhalleux%40microsoft.com|cf4311e516434c82df5808d4af4cb166|72f988bf86f141af91ab2d7cd011db47|1|0|636326191313965991&sdata=6vYUwFZKw8WNdk9yuUqNzviqxbftu4yDDlXt%2B8IdO6E%3D&reserved=0:

@@ -596,9 +596,11 @@ ${files["main.ts"]}
// Split the steps
let stepcontent = content.innerHTML.split(/<h3.*/h3>/gi);
for (let i = 0; i < stepcontent.length - 1; i++) {

                       stepInfo[i].headerContent = stepcontent[i + 1].split(/(<.*?>.*<\/.*?>)/i)[1];

                       content.innerHTML = stepcontent[i + 1];

There is most definitely a library, but I don't see a need to add another dependency.
I have figured out a way to use the content element (ie: the passed in DOM) to do the parsing.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fpull%2F2257%23discussion_r121155824&data=02|01|jhalleux%40microsoft.com|cf4311e516434c82df5808d4af4cb166|72f988bf86f141af91ab2d7cd011db47|1|0|636326191313965991&sdata=6vYUwFZKw8WNdk9yuUqNzviqxbftu4yDDlXt%2B8IdO6E%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4KbthQzdPEbj9rh83KCKsU4CvLaahks5sCWV3gaJpZM4N1DuT&data=02|01|jhalleux%40microsoft.com|cf4311e516434c82df5808d4af4cb166|72f988bf86f141af91ab2d7cd011db47|1|0|636326191313965991&sdata=%2BEmYi%2FCYMFrbQPWceyi%2FVLwxP%2FPYKkcjIfuTCy2ZA%2FQ%3D&reserved=0.

Nope my trick doesn't influence the HTML layout, as in the case of the tutorial project the content div is never returned, instead the content is returned in messages. The tutorial iframe is 1px by 1px iframe.
Yes I'm using the browser's DOM parser with this.

This needs to be tested on firefox, etc. but I think the reason this trick works is because the content div is already added to the tutorial iframe's DOM. Otherwise you get weird rendering issues, because the font sizes and such aren't passed in.

Sounds good then.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Sam El-Husseini<EMAIL_ADDRESS>Sent: Friday, June 9, 2017 8:44:22 AM
To: Microsoft/pxt
Cc: Peli de Halleux; Review requested
Subject: Re: [Microsoft/pxt] Fix tutorial rendering. (#2257)
Nope my trick doesn't influence the HTML layout, as in the case of the tutorial project the content div is never returned, instead the content is returned in messages. The tutorial iframe is 1px by 1px iframe.
Yes I'm using the browser's DOM parser with this.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fpull%2F2257%23issuecomment-307424828&data=02|01|jhalleux%40microsoft.com|7a0f92a66bb24645c3e008d4af4e67b5|72f988bf86f141af91ab2d7cd011db47|1|0|636326198660025673&sdata=KBCjNQT5XyP8IusnyiI7N8IdeXya%2F5gJynrkT1578r4%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4KZBWuN88-IpqHeatCNdA0OS2ePE8ks5sCWhWgaJpZM4N1DuT&data=02|01|jhalleux%40microsoft.com|7a0f92a66bb24645c3e008d4af4e67b5|72f988bf86f141af91ab2d7cd011db47|1|0|636326198660025673&sdata=X7lkjRu3hFEPI086D3QVupfUehl4KviArnpndXXu%2F5s%3D&reserved=0.

Oh font size computation. Makes sense now. Yes it has to be in the DOM.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Sam El-Husseini<EMAIL_ADDRESS>Sent: Friday, June 9, 2017 8:46:48 AM
To: Microsoft/pxt
Cc: Peli de Halleux; Review requested
Subject: Re: [Microsoft/pxt] Fix tutorial rendering. (#2257)
This needs to be tested on firefox, etc. but I think the reason this trick works is because the content div is already added to the tutorial iframe's DOM. Otherwise you get weird rendering issues, because the font sizes and such aren't passed in.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fpull%2F2257%23issuecomment-307425488&data=02|01|jhalleux%40microsoft.com|d760258f0bda4d85f98f08d4af4ebebb|72f988bf86f141af91ab2d7cd011db47|1|0|636326200118041115&sdata=zc9kvFVnj%2F9jdWr3NWMNFvuTgUwp4Sv5wqIDrEnKnF0%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4KaFCiyZNsk2aMgV4Xg_T7tImjrFCks5sCWjogaJpZM4N1DuT&data=02|01|jhalleux%40microsoft.com|d760258f0bda4d85f98f08d4af4ebebb|72f988bf86f141af91ab2d7cd011db47|1|0|636326200118041115&sdata=BMm1fFL30HpcjrVass98PpUlwT4rmQ2wc8MyoX4Yh8o%3D&reserved=0.

Fails in FF, back to the drawing board.

Fixed in FF.

Need to test on IE / Edge and Windows.


DartPad is loading b0rked

It is coming up in Chrome is broken. Amusingly, it is fine in the same Chrome install, but under a different profile.

can't replicate now


How can enable udev sync successfully in docker?

I have posted the issue on SO, unfortunately, there is no any answer. So I repost here, hope can get answer.
I have downloaded and install the static-linked docker 1.6.1 from this site, and run it on RHEL 7.1:
[root@localhost bin]# ./docker -d
WARN[0000] Udev sync is not supported. This will lead to unexpected behavior, data loss and errors
INFO[0000] +job init_networkdriver()
INFO[0000] +job serveapi(unix:///var/run/docker.sock)
INFO[0000] Listening for HTTP on unix (/var/run/docker.sock)
INFO[0000] -job init_networkdriver() = OK (0)
INFO[0000] Loading containers: start.

INFO[0000] Loading containers: done.
INFO[0000] docker daemon: 1.6.1 97cd073; execdriver: native-0.2; graphdriver: devicemapper
INFO[0000] +job acceptconnections()
INFO[0000] -job acceptconnections() = OK (0)
INFO[0000] Daemon has completed initialization

I can see there is a warning: "Udev sync is not supported. This will lead to unexpected behavior, data loss and errors", and after checking the docker source code, I find the warning log is from deviceset.go:
func (devices *DeviceSet) initDevmapper(doInit bool) error {
	......

	// https://github.com/docker/docker/issues/4036
	if supported := devicemapper.UdevSetSyncSupport(true); !supported {
		log.Warnf("Udev sync is not supported. This will lead to unexpected behavior, data loss and errors")
	}
	log.Debugf("devicemapper: udev sync support: %v", devicemapper.UdevSyncSupported())

	......
}

The devicemapper.UdevSetSyncSupport is like this:
// UdevSyncSupported returns whether device-mapper is able to sync with udev
//
// This is essential otherwise race conditions can arise where both udev and
// device-mapper attempt to create and destroy devices.
func UdevSyncSupported() bool {
	return DmUdevGetSyncSupport() != 0
}

// UdevSetSyncSupport allows setting whether the udev sync should be enabled.
// The return bool indicates the state of whether the sync is enabled.
func UdevSetSyncSupport(enable bool) bool {
	if enable {
		DmUdevSetSyncSupport(1)
	} else {
		DmUdevSetSyncSupport(0)
	}
	return UdevSyncSupported()
}

I can see the reason is enabling udev sync failed. How can enable udev sync successfully?

Hi!
If you are reporting a new issue, PLEASE make sure that does not have any duplicates already open.
We would like to take this time to remind you of the information we need to debug the problem you are seeing. This is an automated response so if this ticket is not about a bug, do not fret.
If you fail to provide this information within 7 days, we will close this because we cannot debug your issue. We can reopen whenever the information is provided.
Thank you.
Please see:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues
Description of problem:

`docker version`:

`docker info`:

`uname -a`:

Environment details (AWS, VirtualBox, physical, etc.):

How reproducible:

Steps to Reproduce:
1.
2.
3.

Actual Results:

Expected Results:

Additional info:

#ENEEDMOREINFO

From poking around, this looks to be enabled via a compile-time option on the lvm2 package. Did you custom build the lvm2 package or is it the official one from the redhat repos?

To enable udev sync, you have to use dynamic-linked docker binary

@phemmer:
@coolljt0725 :
Thanks very much for your comments firstly!
After checking the disassembly code of dm_udev_set_sync_support:
(gdb) disassemble dm_udev_set_sync_support
Dump of assembler code for function dm_udev_set_sync_support:
=> 0x0000000000a3e4e0 <+0>:     repz retq
End of assembler dump.

It is a empty function and does nothing, not mention set sync support. Does this mean this static-built docker binary is nu-use?

This is why we should use a dynamic-linked docker binary to enable udev sync

@coolljt0725 ： It seems that the static-linked binary is no-use. Why does it exist?

@NanXiao Because most people don't use devicemapper and don't need udev sync.
Closing because this is a non-issue.

@cpuguy83 what storage backend would you recommend for RHEL / CentOS 6.X or 7?

@NanXiao docker 1.6.0 is available from epel-testing

@cpuguy83

@NanXiao Because most people don't use devicemapper and don't need udev sync.
Closing because this is a non-issue.

Well, this is disappointing way to answer to say the least.

devicemapper is one of the supported backends
there is no "default always works" supported backend
anyone that I know that don't use Docker for "playing around" but have actual businesses to run, struggle to put their finger on a "this is the stable one" backend to use: we just try shit and stop using it when it falls apart

In my specific case, I ended up here after using BTRFS for months and recently started having Kernel Panic trouble. So I have to look for alternatives for Docker backend storage. I'm thinking to turn to Direct DataMapper with LVM (as this RedHat article suggests it as a good option), and after googling the same error reported by @NanXiao above, I hit a brickwall.
I have grown accustomed to the general lack of "production responsibility" of the whole Docker development team (many interesting conversations with the Boot2Docker folks), that nowdays I don't even bother to report an issue anymore - I just rely on my trustworthy colleagues inside my own company.
Next time someone raises an issue about a SUPPORTED feature of Docker, as much as most people don't use it, try to be more helpful.

@detro My response was terse because there is a whole thread here explaining the issue... though I do see that it is a shitty response.
The reality of the issue is that no one really understood the cause of the issue with devicemapper until after the 1.6.0 release.
The issue of course being that with a statically linked binary, udev sync is impossible, and as such can cause corruption issues.
This was difficult for RedHat (who maintains the devicemapper driver) to pinpoint because they use a dynamically linked binary (which they provide in their repos).
Just after the 1.7.0 release docker started providing rpms and debs with dynamically linked binaries from the main install script @ get.docker.com (and apt repos to match).
With these binaries udev sync is supported and devicemapper should work fine.
Also, I totally agree, the graphdriver situation is not fantastic even today. BTRFS is not really ready for prime-time, overlay is too new (and also has lots of known issues), aufs is basically unsupported (as a project), devicemapper is slow.
This is part of the reason I created #13777, which would allow 3rd parties to write drivers for their proprietary back-ends as well as experiment more rapidly with new tech.
In any case, if you'd like to discuss feel free to ping me on IRC any time.

@cpuguy83 I appreciate your answer. If there is a thread with more info and with detailed answers, you should always link it. As much as there is a search facility in GitHub now, it's always possible to miss topics. Plus I'd be probably right to assume that most people land on issues from a google search; probably their working hours; on a rush to get things done.
Unless it's a Docker contributor, it's unlikely that such person has time or knowledge to know how deep and complex the issue with backends in docker is.
Anyhow, your answer gives me some next step to take: given BTRFS and OverlayFS are both not ready for prime time, and AUFS is unsupported, I'll have to 1) move from Docker 1.5 2) try again direct devicemapper over LVM. Performance wise shouldn't be as bad as devicemapper loopback.


Gallery not visible

[Hi, when I insert the gallery and connect data source, the gallery became invisible When in app mode. Please advise:) thanks # ]

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

ID: 6ae46e7b-18ee-f3a0-0dfd-209e0d093ea5
Version Independent ID: 6ae46e7b-18ee-f3a0-0dfd-209e0d093ea5
Content: ReadNFC function - Power Platform
Content Source: power-platform/power-fx/reference/function-readnfc.md
Service: power-platform
Sub-service: power-fx
GitHub Login: @mikepenaMS
Microsoft Alias: mikepena

Thanks for your comment, @arielcyl. This issue has been assigned to the doc author for review.

Thanks for contacting us. Unfortunately, we're unable to handle Support requests for the product here. To contact Support, see here: https://powerapps.microsoft.com/support/
Thanks,
Mints


xbox 360 emulation example...and help me :)

Starting file: https://github.com/Sentdex/pygta5/blob/master/vjoy.py
So with test(), you can successfully rotate, and the world is great.
Now, I am just trying to wrangle this.
My #1 interest with this is for use when walking around as a ped, looking left/right/up/down, but it would also be great to be able to use in a car to control throttle, braking, and left/right turning.
If anyone has some insight on how to properly control these, I am all ears. Some helper functions or something would be epic, or just some examples of actually controlling and resetting joys would be great.
Instructions for installing all the required stuff at the top of the vjoy.py script.

got a handle on it. Main issue was that things range from 0 to 32768, with 16,393 being the middle point, and then I had to figure out all the unique mappings. Got it all set now though. This opens MANY new doors :)


Use ByteBuffer with Binary propagation instead of byte[]

I'm wondering what are the reasons to not use ByteBuffer with the Binary injection/extraction.
I'm thinking using ByteBuffer could be a nice thing to use (you can even wrap a byte[] with it), with all its range/limit capabilities (and potential re-use).

I think using ByteBuffer everywhere is ok, the only downside is that it forces a new allocation every time you need to serialize, probably the JIT will optimize this.

Can we please hold on to this, the main user of this API is gRPC which uses byte[], so extra unnecessary allocation after this change

@bogdandrutu Sure


Circular import in external_tools

When trying to
from SBART.utils.paths_tools.Load_RVoutputs import find_RVoutputs
we get:
  File "/data/work/amiguel/env/lib/python3.8/site-packages/SBART/utils/paths_tools/Load_RVoutputs.py", line 3, in <module>
    from SBART.data_objects.RV_outputs import RV_holder
  File "/data/work/amiguel/env/lib/python3.8/site-packages/SBART/data_objects/__init__.py", line 7, in <module>
    from .DataClass import DataClass
  File "/data/work/amiguel/env/lib/python3.8/site-packages/SBART/data_objects/DataClass.py", line 18, in <module>
    from SBART.utils.paths_tools.Load_RVoutputs import find_RVoutputs
ImportError: cannot import name 'find_RVoutputs' from partially initialized module 'SBART.utils.paths_tools.Load_RVoutputs' (most likely due to a circular import) (/data/work/amiguel/env/lib/python3.8/site-packages/SBART/utils/paths_tools/Load_RVoutputs.py)

Temporary solution:
import the Dataclass right before this:
from SBART.data_objects import DataClass
from SBART.utils.paths_tools.Load_RVoutputs import find_RVoutputs

The find_RVoutputs function is simply a wrapper for the RV_holder.load_from_disk
Actions:

Change all occurrences of this usage
Move the find_RVoutputs to the external_tools interface

Fixed in v0.1.6 (at issue-closing time only up on dev branch


Add possibility to not copy files when using share files and cli

My actions before raising this issue

[x] Read/searched the docs
[x] Searched #2377 #204

Expected Behaviour
PR #2377 will allow to not copy files inside CVAT when creating a task using share files.
It should be possible to do it through cli too.
Current Behaviour
Impossible to use the cli to not copy files inside CVAT when creating a task using share files.
Possible Solution
Implement the use of cli to not copy files inside CVAT when creating a task using share files.
Context
I use the CLI to automatically add a large number of video tasks to CVAT.
Currently, all this data is copied in CVAT, even if it is freely available in the CVAT share folder (that is a Minio bucket on the same server); hence using double space "for nothing".
Not copying the data would allow to avoid wasting space and speed the upload of tasks on CVAT.
I mentioned my situation here and @Marishka17 asked me to create an issue concerning it.

When using the "Connected file share" on the create task screen (or by CLI) and using a tar file, the data is copied to server. Checking or unchecking "Copy data into CVAT" just changes the location of the data - in raw directly or in folders under it.

https://github.com/openvinotoolkit/cvat/issues/3763 was opened.

It looks like the issue was resolved. I will close it.


how to create with serial primary key

I'm trying to do the most simple create mutation.
mutation ($user:UserInput!) {
  createUser(input: {user:$user}) {
    user{
      id
      name
    }
  }
}

variables:
{"user": {"id": 1, "name": "John" }}

How can I omit the id field?
{"user": {"name": "John" }}

schema:
create table mydb.user (
  id              serial not null primary key,
  name            text not null
);

@jmparsons If you've marked id as primary key and it has a default then you shouldn't need to specify id; if you're still struggling please provide your database schema (or at least the user table) and we'll try to help 👍

@benjie Hi I've updated my question with the schema. It's just two fields id and name.

@jmparsons You should be able to just omit the id... is it that you've set the variables to client instead of user?
If you look in GraphiQL you should see something like this:

The id: Int has no exclamation mark so is not required.

@benjie If I omit I get: "duplicate key value violates unique constraint \"user_pkey\""

@benjie I got it to work, but I had to wipe my data. I have a data seed and the increment tries to use a key that's already be applied giving me "duplicate key value violates unique constraint \"user_pkey\"".
I guess is there a way to update the postgres cursor or something along those lines for seeded data?

That's entirely a PostgreSQL concern; you'd have the same issue on the command line with psql
(But since you asked: you're looking for the ALTER SEQUENCE command)
To keep things manageable I'm going to close this issue as I think it's solved; but if not or you require further help please re-open it.

@benjie You're the best!


test(mempool): refactor MempoolState to have Partial and Full states

This change is

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.92%. Comparing base (ab3a562) to head (6b70786).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #507   +/-   ##
=======================================
  Coverage   82.92%   82.92%
=======================================
  Files          37       37
  Lines        1786     1786
  Branches     1786     1786
=======================================
  Hits         1481     1481
  Misses        228      228
  Partials       77       77

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Build fails on Ubuntu 20.04

Trying to build as
cmake -DCMAKE_PREFIX_PATH=/usr/lib/llvm-10/include .
make

errors as
/home/per/Work/cpp2d/CPP2D/CPP2DPPHandling.h:30:7: error: ‘void CPP2DPPHandling::InclusionDirective(clang::SourceLocation, const clang::Token&, llvm::StringRef, bool, clang::CharSourceRange, const clang::FileEntry*, llvm::StringRef, llvm::StringRef, const clang::Module*)’ marked ‘override’, but does not override
...

on Ubuntu 20.04 with libclang-10-dev installed.
Is this because the Clang API has changed from version 6 to version 10?

I am having this issue too.


important task SQL injection in all model bread

Version information

Laravel: v7.0
Voyager: v1.4
PHP: 7.4
Database:MySQL 8.0

Description

there is SQL injection in all model bread
Steps To Reproduce
Steps to reproduce the behavior:
1- write link /admin/users
2- add parameters like that (/admin/countries?filter=contains&key=%5C&s=1)
3- it display SQLSTATE[42S22]: Column not found: 1054 Unknown column '' in 'where clause' (SQL: select * from countries where \ LIKE %1% order by created_at desc)
Additional context
Add any other context about the problem here.

You can do a lot to make Voyager fail, but as long as you can not manipulate the query to return data you are not allowed to see I don't consider this as a bug or security concern.

it's dangerous to pass the value directly to query without parameters

I like Voyage so much we here to improve not to make fail

Appreciate any input but value is not passed directly to query, is passed to Laravel query builder that takes care of sanitization.
You can join Voyager Slack group and if you have any proof that anything can be injected please contact us there.

ok I understood but can you make at least validation that makes sure if there any problem or any value or column wrong just return null it better than return error
thank you for your great work.


jupyter_sphinx.execute doesn't use the correct environment

running make html from the repo/docs doesn't result in jupyter_sphinx.execute using the code from repo.
This is a problem in Adaptive, whereas a bad workaround we install master in the CI.

One easy and correct way to handle this is to set PYTHONPATH from the sphinx's conf.py.

One easy and correct way to handle this is to set PYTHONPATH from the sphinx's conf.py.

This is not enough, unfortunately. jupyter-execute launches a separate ipython kernel: no user code is executed in the Python interpreter that is running sphinx. The way I have got around this so far is to have a boilerplate.py file that I include with
.. jupyter-execute:: boilerplate.py
    :hide-code:

but this is kind of unsatisfactory because then the downloadable scripts have the contents of boilerplate.py in them. Perhaps we should allow some common code to be executed in a cell prior to any of the cells defined in a given document, and have this cell stripped before writing any output.

The jupyter kernels inherit the environment from the parent process. This is used with jupyter-sphinx for example here


Using DD function raises a SIGSEGV

Using dd on a LibXML::Document raises a SIGSEGV
Follow up of:
https://github.com/rakudo/rakudo/issues/5052
Copying jonathanstowe analysis:

Trimming it somewhat to:
use LibXML;
use LibXML::Document;

sub MAIN(){
    my LibXML::Document $doc .=  parse: :string('<doc />');
    dd $doc;

}

Gives a back trace:
Thread 1 "rakudo-m" received signal SIGSEGV, Segmentation fault.
0x00007ffff788ec2f in get_attribute () from //usr/local/lib/libmoar.so
Missing separate debuginfos, use: dnf debuginfo-install glibc-2.35-15.fc36.x86_64 libxml2-devel-2.9.14-1.fc36.x86_64 xz-libs-5.2.5-9.fc36.x86_64 zlib-1.2.11-32.fc36.x86_64
(gdb) bt full
#0  0x00007ffff788ec2f in get_attribute () from //usr/local/lib/libmoar.so
No symbol table info available.
#1  0x00007ffff7814494 in MVM_interp_run () from //usr/local/lib/libmoar.so
No symbol table info available.
#2  0x00000000004015ef in main ()
No symbol table info available.
It would appear that the segfault is from trying to obtain the value of an attribute of the LibXML::Document object (it also will happen with say $doc.raku,) which would suggest that some otherwise unused attribute of the CStruct of LibXML::Document is defined with the incorrect size.
So it's likely that this is a problem with LibXML rather than rakudo.

Works for me on Rakudo blead 2022.07-3-g9bc1beed7.

As an extra check. both the following C and Raku code agree on an overall size of 176 for xmlDoc:
#include <libxml/tree.h>

int main(void) {
    printf("%ld\n", sizeof(xmlDoc));  # 176
}

use LibXML;
use LibXML::Document;
use NativeCall;

sub MAIN(){
    my LibXML::Document $doc .=  parse: :string('<doc />');
    say nativesizeof($doc.raw); # 176
    dd $doc;
}

the raku method has been aliased in LibXML: 0.9..10 to  work around [this problem in Spreadsheet::XLXS.  The above example now completed with:
"<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n<doc/>\n"


Fix all invalid escape sequences discovered running tests

Fixes warning of the form:
DeprecationWarning: invalid escape sequence \...

See:
https://docs.python.org/3/whatsnew/3.6.html#deprecated-python-behavior

Codecov Report

Merging #1221 into develop will decrease coverage by 0.04%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #1221      +/-   ##
===========================================
- Coverage    98.05%   98.01%   -0.05%
===========================================
  Files           45       45
  Lines         7306     7306
===========================================
- Hits          7164     7161       -3
- Misses         142      145       +3

Impacted Files
Coverage Δ

botocore/compat.py
91.85% <ø> (ø)
:arrow_up:

botocore/utils.py
95.96% <100%> (ø)
:arrow_up:

botocore/docs/sharedexample.py
98.58% <100%> (ø)
:arrow_up:

botocore/handlers.py
96.32% <100%> (ø)
:arrow_up:

botocore/credentials.py
97.73% <0%> (-0.57%)
:arrow_down:

Continue to review full report at Codecov.

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

Looks good, thanks!


Fix incorrect use of S3Fs's API for file deletion

In case of job failure, the file on the S3 bucket is currently removed with s3fs.remove(...) instead of s3fs.rm(...), which raises an error.
closes #1

Thanks!


NFS volume mount getting failed when we tried to start container with exit code 32

Hi ,
I an new to create docker volume using mount option, i am trying to create a service using mount option by using docker-volume-netshare with NFS driver,Please find the following command i tried and its output.
docker create --mount type=volume,volume-opt=o=addr=IPADDRESS,volume-opt=device=:/opt/dockerShare,volume-opt=create=true,volume-driver=nfs,volume-opt=type=nfs,source=vol_nfs1,target=/mount --name mynfs centos:centos7 ping docker.com
-->when i start docker container i observed following error
docker start ba2f7d8dab8a
Error response from daemon: error while mounting volume '': VolumeDriver.Mount: exit status 32
Error: failed to start containers: ba2f7d8dab8a
-->Below is from the docker-volume-netshare logs
./docker-volume-netshare nfs --verbose
INFO[0651] Mounting NFS volume vol_nfs1: on /var/lib/docker-volumes/netshare/nfs/vol_nfs1
DEBU[0651] Mounting with NFSv4 - src: vol_nfs1:, dest: /var/lib/docker-volumes/netshare/nfs/vol_nfs1
DEBU[0651] exec: mount -v -t nfs4 vol_nfs1: /var/lib/docker-volumes/netshare/nfs/vol_nfs1
2017/07/23 23:07:14 mount.nfs4: Failed to resolve server vol_nfs1: Name or service not known
-->if i use below command i  can start container but after few seconds container  getting exited
docker create --mount type=volume,volume-opt=share=<IP_ADDRESS>:/opt/dockerShare/,volume-opt=create=true,volume-driver=nfs,volume-opt=type=nfs,source=vol_nfs1,target=/mount --name mynfs centos:centos7 ping docker.com
-->
DEBU[0042] Entering Create: name: share, options map[create:true share:IPAddress:/opt/dockerShare/ type:nfs]
DEBU[0042] Create volume -> name: share, map[create:true share:IPAddress:/opt/dockerShare type:nfs]
DEBU[0056] Host path for share (share) is at /var/lib/docker-volumes/netshare/nfs/share
DEBU[0062] Entering Mount: {share 6264558d7fe5fe2fb987d43c88a3411772377030d7c47a1aec673449bee4b17e}
INFO[0062] Mounting NFS volume IPAddress:/opt/macaw-shared/platform/ on /var/lib/docker-volumes/netshare/nfs/share
DEBU[0062] Mounting with NFSv4 - src: IPAddress:opt/dockerShareopt/dockerShare, dest: /var/lib/docker-volumes/netshare/nfs/share
DEBU[0062] exec: mount -v -t nfs4 IPAddress:/opt/dockerShare /var/lib/docker-volumes/netshare/nfs/share
INFO[0062] Mount: Share and Create options enabled - using share as sub-dir mount
Can any one help me why its getting failed for first case ? as per my knowledge its trying to mount volume name instead of IP Address.

Does your NFS server support NFS 4.1?
I confirmed "mount" command recognize "-t nfs4" as "vers=4.1" at CentOS 7.3 with nfs-utils-1.3.0-0.48.
If you want to confirm that, try below commands.
mount -t nfs4 -v NFS-SERVER MOUNT-PATH
mount -t nfs -v NFS-SERVER MOUNT-PATH
mount -t nfs -o vers=4.0 -v NFS-SERVER MOUNT-PATH
Maybe only last command will be success because "mount" automatically specify "vers=4.1".
My work around is DKV_NETSHARE_OPTS="nfs -v 3 -o vers=4.0".
It is strange, but good for this situation.


Allow running whatDependsOn without version number

I'd really like to be able to run whatDependsOn without needing the version number.
E.g. right now I'm trying to track down a problem that I know is caused by having both log4j-over-slf4j.jar AND slf4j-log4j12.jar on the classpath, but I don't know what versions are being transitively pulled in. I tried running dependencyTree, but it took half an hour and died with an out of heap space error. Even if I did know the version numbers of the jars it would be more convenient to have the version number be optional

👍

👍

It would be great to have this, I would even pay for such a feature :)

@benmccann this could be interesting https://github.com/evolution-gaming/no-log4j-test

Guys, just in case, look at this stuff: https://github.com/pshirshov/izumi-r2/blob/develop/sbt/sbt-izumi-deps/src/main/scala/com/github/pshirshov/izumi/sbt/deps/IzumiBuildInfoPlugin.scala#L121
This way you may introspect the whole project from a single task execution, so you wouldn't need to execute same task in all the submodules.

This feature will be in 0.9.1.


[Discussion] Use intervention/imagecache to create derived images

Laravel Version: 5.3.28
Voyager Version: 0.10.7
PHP Version: 7.0.14

Curent implementation:
Currently, there is a means to create derived images on save, using a JSON structure to configure the desired image derivatives. This is described in the documentation and implemented in the Controller class.This works well for images that are managed through a BREAD interface, e.g. as part of posts or pages.
Problematic use case:
It cannot be used for images that are imported from an external source, e.g. product images. For these cases however, thumbnails can be created on the fly, using the capability of imagecache to access derived images through a specifically formatted URL /imagecache/<format>/<image-path>.
In my website, there are now two competing mechanisms to arrive at derived images:

On save for BREAD-managed images
On access, for images acquired through other sources

Proposed solution:
I wonder if it would make sense to replace all of Voyager's derive-image-on-save mechanics with an integration of Imagic Imagecache. Specifically, we would use their URL-based manipulation. This seems to be the more general and flexible solution:

General, because I can create derivative images for both kinds of images.
Flexible, because I can add new derived image formats as my site requires them, without re-saving all images.

Best of all, the Voyager code to create derived images on save could be removed.
What do you think?

Forgive me as I'm not very familiar with this library/functionality.
Does this generate the images on-the-fly (as in, on each request)? Or does it cache images somehow?  If it generates it on each request, this could cause issues (though that would easily be fixed with a simple site-wide caching layer, CDN or otherwise).  Overall, I actually like the idea.  Just a few details to work out.

Forgive me as I'm not very familiar with this library/functionality.
Does this generate the images on-the-fly (as in, on each request)? Or does it cache images somehow?  If it generates it on each request, this could cause issues (though that would easily be fixed with a simple site-wide caching layer, CDN or otherwise).  Overall, I actually like the idea.  Just a few details to work out.

Just to be clear, what you want to achieve is a way to create a post with a image, and then use the URL based manipulation for this image.
Am I correct? Then I can look into it.

Closing due to age.  We're going to be completely refactoring all image upload/manipulation/etc functionality soon anyway.


Update go-plugin-helpers with recent submits to fix mount/unmount ID and to add status for Get()

PRs #54 and #56 of go-plugin-helpers are submitted and need to be pulled into our repo.

Closed via #566


ConnectionService retries on error when it should not

Bug Report
Steps to Reproduce:
Using the ConnectionService, accept a connection request using connectionService.acceptConnection(userId), for a user with a connection that has StatusEnum ACCEPTED.
Expected Result:
To throw the following error once:
An unknown error occurred while trying to connect to https://[...]:443/pod. Please check below for more information: {"code":500,"message":"request to userConnections/[...] failed with status 422, message:Status change requested is not consistent with current status."}
Actual Result:
Instead, the RetryWithRecovery kicks in and the connection service keeps trying to accept, each time throwing the same error
Environment:
symphony-bdk 2.2.1

Thanks @giso-cmo for this report 👍
Your issue seems to be related to https://github.com/finos/symphony-bdk-java/pull/509, where we decided to retry on 500 errors.
However, from the log trace you shared above, it looks like the Symphony Backend is returning 500 because of a 422 received from an internal component. Which is the actual issue.
I'll sync internally to make sure this issue resolved in the next release, and keep you in touch through this ticket.
Thibault.

Update: this scenario now correctly returns 400 instead of 500 so the tooling also now responds correctly without requiring any changes


1.21.4 Minecraft crash with v.1.6.1

Minceraft crash when joining world or enable mod ingame

because its not made for 1.21.4, has to be updated to work

Ik i mean with my bug report to ask for an update but i think i forgot to add this message xD silly me


what is 1fmHookShip.lua ? and japanese version bug report

The description says Fixed hook ship mod for JP, please tell me more about what this files.
and, how do disable autosave for SaveAnywhere?

1fmHookShip.lua causes the ship to always appear when traveling to Neverland for the first time. I added this to the readme just now.
To disable the autosave, remove this part of the code:
	if ReadFloat(soraHUD) == 1 and prevHUD < 1 then 		local f = io.open("autosave.dat", "wb") 		f:write(ReadString(continue, 0x16C00)) 		f:close() 		ConsolePrint("Wrote autosave") 	end

Thanks for your quick reply.
I forgot to mention that there is a bug that probably occurs only in the Japanese version, so I'd like to report it.
When I do a soft reset of SaveAnywhere (1fm), the text in the dialogs and cutscenes is wrong (the text around the HUD is normal).
I can work around this bug by going back to the title from the save menu by pressing L1+L2+R2+Select, but this resets the resolution from 1440p to 1080p.
I'll be glad if you can fix this bug!
https://i.imgur.com/1cH4sLw.png
https://i.imgur.com/vMGkiAz.png

The soft reset function is very simple and identical between global and japanese version, so it is unknown to me what is causing that issue. Perhaps that bug is the reason Square made the title screen button hard reset the game.

:_(
Thanks for reply :)


Understanding tocca.js events - help required

So this is how I've used tocca for touch events.
bar1 = $("#myDIv").find('.dragMarker');
bar1.on('touchstart',setCurrent);
bar1.on('mousedown',setCurrent);
function setCurrent(e,data) {
	current = 1;
	reset();
}

I want to pass an integer like this:
$.mousedown(setCurrent(i))

function setCurrent(j) {
	return function(event) {
		current = j;
		reset();
	}
}

How to combine it so that I can link both actions to the same function? ie. function setCurrent(e,data,i)
Thanks :)

I am not sure I understand what you are trying to do, but in your code there is no one events provided by Tocca.js. Can you make a simple jsfiddle please?

Yes I have just set one up: https://jsfiddle.net/Siemvandenberg/3gnz8209/
Altough some tocca beheviour might be commented out

@Vandenberg I am not sure what you are trying to do but I guess this example may help you

When I packed with r.js's   require.js,
JQuery writes tap effectively
Dian=$var ('#dian');
Dian.on ('tap', function (E, data) {//it's worker
Alert (Dian tap'jquery 1111');
});
But with the native JS, there is no effect, if I pack an error, jQuery bound tap certainly invalid
Dian2=document.getElementById var ("dian2");    //it's  don't work
Dian2.addEventListener ('tap', function (E) {
Alert (dian2 tap'js);
});

When I packed with r.js's require.js,JQuery writes tap effectively
var  dian=$('#dian');
dian.on('tap',function(e,data){      //it's  word
alert('jquery dian  tap dian');
});
But with the native JS, there is no effect, if I pack an error, jQuery bound tap certainly invalid
var dian2=document.getElementById("dian2");  //it's  don't work
dian2.addEventListener('tap',function(e){
alert('js  dian2 tap  dian2');
});

I will close this issue I provided a valid solution to pass value to the tap callback and for me it's ok so. Thank you for your questions


protein glycation parentage

https://github.com/geneontology/go-ontology/issues/12550
Could this also have
"cellular detoxification"
as a parent?
(I'm trying to pull together the detoxification terms in our slim...many of the remaining unstudied proteins seem to be 'cellular detoxification' of some sort.)
http://www.pombase.org/spombe/result/SPAC22E12.03c
@rebeccafoulger ?

I'll assign this to @rebeccafoulger for her to comment when she's back from summer leave.
Becky, feel free to assign back to me for edits then, thank you.

The degylcation enzymes I've been curating are involved in the early steps, removing MGO or GO from glycated proteins to stop them forming AGEs (advanced glycation end products). I don't think the early glycated products themselves are always toxic, but rather go on to form toxic end-products. If this still fits with 'cellular detoxification' then I'm fine with this additional parentage.
Note that it's currently 'protein deglycation ; GO:0036525' is_a: 'protein repair ; GO:0030091'.

OK, I guess it should be a broader question,  should protein  repair, DNA repair and RNA surveillance be considered 'cellular detoxification'.  Will continue to think about this, and ask some experts.


Docs on supported options for AKSNodeClass and NodePool?

Is an existing page relevant?
No response
What karpenter features are relevant?

AKSNodeClass
NodePool

How should the docs be improved?
Provide an example under https://github.com/Azure/karpenter/tree/main/examples which shows all supported configuration options.
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Until this is addressed, for a subset of settings you can refer to AKS Node Autoprovisioning (Preview) documentation.  NAP is based on this Karpenter provider, so the settings are same.


Devanagari Support

Could you please add Devanagari Script support for custom dictionaries?
Thank you,

Hi, @gautamneeraj :)
I do not plan to provide specific dictionary files as a custom dictionary. However, as discussed below, it is possible to include the dictionary files anyone created in the documentation.
https://github.com/tadashi-aikawa/obsidian-various-complements-plugin/discussions/297

Thank you, I've managed to create a custom dictionary for Nepali. Thank you!


[FIX] Remove stacktrace from view when debug mode is OFF

Add DEBUG_MODE to check when to send stacktrace and activate DEBUG

Testing with the debug_mode flag off on the link that was listed on the task (https://deploy-dev1.pinadmin.com/groups/abc/more_scaling_activities/)

Testing with the debug_mode flag on


Add test to check all methods with same seed are working as expected

The PR adds test for same seed for all methods with no args of all providers

Codecov Report

Merging #391 (ce0ed6b) into main (731780d) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #391      +/-   ##
============================================
- Coverage     93.77%   93.69%   -0.08%
+ Complexity     1970     1968       -2
============================================
  Files           217      217
  Lines          3968     3968
  Branches        385      385
============================================
- Hits           3721     3718       -3
- Misses          148      153       +5
+ Partials         99       97       -2

Impacted Files
Coverage Δ

...ker/idnumbers/pt/br/IdNumberGeneratorPtBrUtil.java
92.59% <0.00%> (-3.71%)
:arrow_down:

.../main/java/net/datafaker/service/FakerContext.java
84.61% <0.00%> (-2.57%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Nice, but how is this different from https://github.com/datafaker-net/datafaker/pull/384?

oops, didn't want to commit , just played around

No worries at all. I'm also happy if you want to replace my test with this one, it looks fine to me.


Transforming the javax.annotation root package risk braking non-Jakarta Annotations use of that package

Libraries other than Jakarta Annotations, particularly JSR 305 / com.google.code.findbugs:jsr305, utilize the javax.annotation root package. This means we can't safely transform Jakarta Annotations based on the package name alone.

I thought perhaps spelling out the details of the classes in jakarta-renames.properties would work but my experiment makes me think not.

I'm sure you found it already but a list of annotation classes that should be in the list I think is at https://jakarta.ee/specifications/annotations/2.0/apidocs/overview-tree.html

@scottmarlow FWIW https://github.com/wildfly-extras/batavia/compare/master...bstansberry:Issue_112 does not work, as those classes end up not being transformed in the class table. I believe the transformer logic using that file is meant to work with package names, not class names.

https://github.com/eclipse/transformer/blob/main/org.eclipse.transformer/src/main/java/org/eclipse/transformer/Transformer.java#L286 seems to back up what your saying:
RULES_RENAMES("tr", "renames", "Transformation package renames URL", OptionSettings.HAS_ARG,
			!OptionSettings.HAS_ARGS, !OptionSettings.IS_REQUIRED, OptionSettings.NO_GROUP),

^ RULES_RENAMES is mapped to the  jakarta-renames.properties file via:

https://github.com/eclipse/transformer/blob/main/org.eclipse.transformer.cli/src/main/java/org/eclipse/transformer/jakarta/JakartaTransformer.java#L34
https://github.com/eclipse/transformer/blob/main/org.eclipse.transformer.cli/src/main/java/org/eclipse/transformer/jakarta/JakartaTransformer.java#L44

Correct @bstansberry - it is possible to specify fully qualified class names in jakarta-renames.properties config file
but such item will not be applied to UTF-8 class file constants. For these sourcePackageName=targetPackageName
format is expected.

We are in sync WRT these annotations @scottmarlow :+1: https://github.com/wildfly-extras/batavia/blob/master/impl/ecl/src/main/resources/org/wildfly/extras/transformer/eclipse/jakarta-renames.properties#L27

@ropalka what would happen with our current rules that I pasted in above for the https://github.com/findbugsproject/findbugs/blob/master/findbugsTestCases/src/fakeAnnotations/javax/annotation/Resource.java that uses the javax.annotation package?
Will https://github.com/findbugsproject/findbugs/blob/master/findbugsTestCases/src/fakeAnnotations/javax/annotation/Resource.java be transformed to jakarta.annotation package?

Yes, it will be @scottmarlow :

Classfile /home/opalka/JAVAX_to_JAKARTA/test/to/Resource.class
Last modified Sep 14, 2021; size 443 bytes
MD5 checksum c6d133af5aafa256d3daaa08082be9e2
Compiled from "Resource.java"
public interface jakarta.annotation.Resource extends java.lang.annotation.Annotation
minor version: 0
major version: 52
flags: ACC_PUBLIC, ACC_INTERFACE, ACC_ABSTRACT, ACC_ANNOTATION
Constant pool:
#1 = Class              #19            // jakarta/annotation/Resource
#2 = Class              #17            // java/lang/Object
#3 = Class              #18            // java/lang/annotation/Annotation
#4 = Utf8               SourceFile
#5 = Utf8               Resource.java
#6 = Utf8               RuntimeVisibleAnnotations
#7 = Utf8               Ljava/lang/annotation/Target;
#8 = Utf8               value
#9 = Utf8               Ljava/lang/annotation/ElementType;
#10 = Utf8               TYPE
#11 = Utf8               FIELD
#12 = Utf8               METHOD
#13 = Utf8               Ljava/lang/annotation/Retention;
#14 = Utf8               Ljava/lang/annotation/RetentionPolicy;
#15 = Utf8               RUNTIME
#16 = Utf8               jakarta/annotation/Resource
#17 = Utf8               java/lang/Object
#18 = Utf8               java/lang/annotation/Annotation
#19 = Utf8               jakarta/annotation/Resource
{
}
SourceFile: "Resource.java"
RuntimeVisibleAnnotations:
0: #7(#8=[e#9.#10,e#9.#11,e#9.#12])
1: #13(#8=e#14.#15)

But Eclipse transformer displays this ERROR before transformation:

[main] ERROR Transformer - Approximate relocation of class; case Unknown location: initial class name [ javax/annotation/Resource ]; final class name [ jakarta/annotation/Resource ]; initial resource location [ /home/opalka/TODO/JAVAX_to_JAKARTA/dist/../test/from/Resource.class ]; final resource location [ jakarta/annotation/Resource.class ].
Archive ../test/from/Resource.class was transformed to ../test/to/Resource.class according to given transformation rules.

Okay, so we would translate any class that is in root javax.annotation (root like Brian said) package.  We would also try to transform any class in the javax.annotation.sql or javax.annotation.permission package.
I am not sure of why you got the error but I think this issue is looking to switch to more specific package.classname based mapping instead of just using the package name.
So something like the following:
javax.annotation.sql.DataSourceDefinition=jakarta.annotation.sql.DataSourceDefinition
javax.annotation.sql.DataSourceDefinitions=jakarta.annotation.sql.DataSourceDefinitions
javax.annotation.security.DeclareRoles=jakarta.annotation.security.DeclareRoles
javax.annotation.security.DenyAll=jakarta.annotation.security.DenyAll
javax.annotation.Generated=jakarta.annotation.Generated
javax.annotation.ManagedBean=jakarta.annotation.ManagedBean
javax.annotation.security.PermitAll=jakarta.annotation.security.PermitAll
javax.annotation.PostConstruct=jakarta.annotation.PostConstruct
javax.annotation.PreDestroy=jakarta.annotation.PreDestroy
javax.annotation.Priority=jakarta.annotation.Priority
javax.annotation.Resource=jakarta.annotation.Resource
javax.annotation.Resources=jakarta.annotation.Resources
javax.annotation.security.RolesAllowed=jakarta.annotation.security.RolesAllowed
javax.annotation.security.RunAs=jakarta.annotation.security.RunAs

Your last post @scottmarlow is basically what I did in https://github.com/wildfly-extras/batavia/compare/master...bstansberry:Issue_112.  I didn't bother spelling out the details of javax.annotation.sql and javax.annotation.security because I'm not aware of any libs adding classes to those packages.


Possible URL Helper

Really loving Frank so far, but trying to use it with a slightly larger site (around 20 pages) and would love a URL Helper of some sor, but didn't know if anyone had run into that before. It seems like it could be relatively simple to do based on file path, but I'm a bit of a n00b at Ruby... Any thoughts?

I've considered this several times and tend to agree that it would be nice to have a few more helpers. I've been working on a new release as I find free time. It will certainly have some sort of url helper.
What exactly were you looking for? Something like rails: link_to or something a bit different?

A link_to would be great, but actually what might be more useful, IMO, would be an array object of all the pages/articles, etc. with all the meta data associated with them. That way in the template or layout, you can just loop through them and call whatever methods you need.
Something really similar to the get_post_sumaries method you're using in your site's helper would probably suffice. Then you could call article.permalink if you needed to.
What gets interesting is the one-off link, where you're not looping through. In that case I'm not sure how you would define the page you want in a link_to method or similar because there are no routes... I'll have to think on that one more!

Ah yeah, I've been planning on doing this for a while too. I want to auto generate sitemaps, xml and html tree for development, this would require what you're talking about.
It's planned for sure.


[PT-BR-brazilian portuguese] translation

I'm a brazilian developer interested in data science. I've translated a part of the project to Brazilian Portuguese and plan on helping more, so more brazillians can have access to the lectures :)
#136

closing for staleness


⚠ Bofrost.nl has degraded performance

In 84d510c, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1439 ms

Resolved: Bofrost.nl performance has improved in ec043dd.


Pinch example with desktop emulator

I am testing pan and pinch on Chrome Mac and Windows 10 tablet. Having difficulties having a consistent behavior.
Simple implementation of 'pinch' and 'pan' (without Manager) doesn't seem to work, with the manager it does. Would be great to know why.
With the manager, the tablet fires the right events but the Mac doesn't. Tried also with the Emulator JS which I am not sure 100% if it worked since I am using es6.
Tried:
import TouchEmulator from 'hammer-touchemulator'
// react
export default class extends Component {
        componentDidMount(){

            TouchEmulator();

I am asking for support since the demos at the page: http://hammerjs.github.io/#try-it work consistently on both devices. So it would be good to see their code and understand what I am missing.
Thanks!

Ok, found a way to correctly include TouchEmulator. All good on Mac now.

Could you share correct way of including TouchEmulator?

Got it!
import * as TouchEmulator from 'hammer-touchemulator'


upgrade from 1.5.3 to 1.5.4 breaks default pub sub option

The upgrade from 1.5.3 to 1.5.4 broke our application. We didn't set the pubsub option explicitly, but relied on the default conn.private[:phoenix_endpoint]. After the upgrade, this default apparently gets overwritten by the update_config function. We noticed because subscription triggers still worked when pushing mutations via a socket, but they didn't work anymore when sending a mutation via HTTP.

Hi @woylie, are you setting pubsub explicitly in update_config? It shouldn't be overriding it with nil, can you elaborate?

We're not using update_config at all.

Here's the PR that recently changed this logic: https://github.com/absinthe-graphql/absinthe_plug/pull/243
Though quickly looking through it I don't see anything obviously wrong...

Right, @woylie how / where would update_config override things? How are you determining that pubsub is being unset?

I'm sorry, I didn't have any time to investigate this any further. I don't actually know that pubsub is being unset, that was my assumption based on:

Upgrading from 1.5.3 to 1.5.4 causes mutations sent via HTTP not triggering any subscription messages, while mutations via socket still trigger them as expected.
This problem can be reliably fixed/reproduced by down/upgrading between 1.5.3 and 1.5.4. No other changes are necessary, no other dependency versions are changed in mix.lock.
The only change between 1.5.3 and 1.5.4 is in the update_config function.

This happens with absinthe 1.5.5 and absinthe_phoenix 2.0.0.

So the actual observed behavior is this:

mutations sent via HTTP not triggering any subscription messages, while mutations via socket still trigger them as expected.

Yes. Sorry for not being clearer in the first description.

We figured it out, here's a fix: https://github.com/absinthe-graphql/absinthe_plug/pull/249

Great, thanks!


Switching from using model Metadata -> TaskMetadata

TaskMetadata will be maintained as a shadow and allows decoupling of
protocol buffer types from contributor code and user code. This allows
more flexiblity
another prime motivation behind this change is that - it allows making the interface less verbose. This makes it trivial to support default values for metadata.

I think I'm not seeing the bigger picture.  At least I don't understand how this is a decoupling?  Ultimately, at serialization time, the Python class will have to be converted into the model class right?  I feel like this is more delaying the coupling rather than decoupling.
Which is fine, but I don't understand what this enables.  +1 if you want.  I think it's okay to use the model sometimes though - like currently we use the dynamic job spec model, literal map and parameter map models, all the literal models, auth, labels, annotations, etc.  I don't think we should create parallel classes for all those.

can you take a look at test failures?


语音功能什么时候能支持window啊?

ASR是支持Windows安装的，可使用以下命令来进行安装

是支持Windows安装的，可使用以下命令来进行安装： pip install "modelscope[audio_asr]" --upgrade -f https://modelscope.oss-cn-beijing.aliyuncs.com/releases/repo.html

Reference in new issue
Reference in new issue

官网的文档上, 确实没有看到这个命令

currently  we support install modelscope[audio_asr]  which will install asr related library on windows successfully

close as this issue has already been addressed

我想问下，window 下载 【CV领域模型】能成功吗，我这里一直报这个
Successfully built zhconv fairseq sacremoses jieba seqeval audioread emoji
Failed to build healpy
ERROR: Could not build wheels for healpy which use PEP 517 and cannot be installed directly
后面就下载不了，求解决方法


⚠ Spryr has degraded performance

In ea55d3e, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 6024 ms

Resolved: Spryr performance has improved in 6e53494 after 7 minutes.


Fix Windows detection on older Chef versions

The ChefConfig class is only available on recent versions of Chef.
/cc @chef-cookbooks/delivery-migration-team

This PR has passed 'Verify' and is ready for review and approval!
Use: '@delivery approve' when code review is complete.

@delivery approve

Change: 59ad886e-6b9d-4309-9ac1-c2b8c95043d9 approved by: @yzl

This PR has passed 'Acceptance' and is ready to be delivered!
Use: '@delivery deliver' when validated in acceptance.

@delivery deliver

Change: 59ad886e-6b9d-4309-9ac1-c2b8c95043d9 delivered by: @schisamo


added sonar scanner cli dockderfile

Why another Dockerfile? And also why it is including kops, helm, terraform, etc?

There is a sonar cli image but we would still need to add to the base to include ruby, kubectl etc. There is a bunch of stuff we don't need but I left it here incase its needed in future. But I guess it would make more sense to add as and when needed. So I'll slim it down further.

@mogaal - A new Dockerfile added as this is independent to tools image.

closed as the sonar qube dockerfile is now to its own repo.


(docs) Address some issues when generating type reference docs

Several type attributes list default values in their description, but also have defaultto definitions. When generating reference docs, this results in output that contains duplicate lists of default values. Remove these redundant defaults from the descriptions.
Defaults that required additional information were intentionally left in the description, but should be minded should the defaultto value change.
Two attributes also included extraneous formatting that appeared in output as raw text. Remove or replace that formatting.

Can one of the admins verify this patch?

CLA signed by all contributors.


use APP_ENV instead of RACK_ENV

with current configuration can't run sinatra with puma if RACK_ENV is not set


On printed map title there is HTML code

Finance, Insurance & Real Estate
<_BOL> 2014 MAG Employer
Database, employers with 5 or
more employees</_BOL>

I fixed it for now by commenting out the line.  Me may want to revisit this later in the mxd and polish it up a little bit.


Issues not getting logged?

I followed the example here:
https://github.com/getsentry/sentry-dotnet-samples/blob/master/AspNetMvc5Ef6/Global.asax.cs
I am able to get a SentryId back from SentrySdk.CaptureException(exception);
But then in the UI I see nothing under Issues.
I also checked the HTTP response with Fiddler and got a 200.
Am I missing something? Are delays expected? How can I debug this further?
cc: @bruno-garcia

Lets track this here: https://github.com/getsentry/sentry-dotnet/issues/518


fix: single instance doesn't shutdown immediately

Fix #1017

Thank you! Could you also add a changefile like this https://github.com/tauri-apps/plugins-workspace/blob/v2/.changes/fix-zbus-import.md? Then it'd be good to go :)


[regression](invered index) add test null index case and fix fault injection index case

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall


multihead attention -  no converter found

❌ Validation exception on node 'MultiheadAttention':
PyTorch op: MultiheadAttention(
(out_proj): NonDynamicallyQuantizableLinear(in_features=128, out_features=128, bias=True)
)
Keras op: ChangeOrderingLayer(func=<function converter_MultiheadAttention..func at 0x75b6b7e9f380>)
Input args: ('Tensor(shape=[4096, 1, 128], dtype=torch.float32)', 'Tensor(shape=[4096, 1, 128], dtype=torch.float32)', 'Tensor(shape=[4096, 1, 128], dtype=torch.float32)')
Input kwargs: {}
Output tensors: ['Tensor(shape=[4096, 1, 128], dtype=torch.float32)', 'Tensor(shape=[1, 4096, 4096], dtype=torch.float32)']
Exception: You called set_weights(weights) on layer "multi_head_attention" with a weight list of length 8, but the layer was expecting 0 weights. Provided weights: [array([[[-0.05060041, -0.01487129,  0.10044055, ....
Traceback:
❌ Validation exception on node 'MultiheadAttentionModel':
PyTorch op: MultiheadAttentionModel(
(multihead_attn): MultiheadAttention(
(out_proj): NonDynamicallyQuantizableLinear(in_features=128, out_features=128, bias=True)
)
)
Keras op: <nobuco.layers.container.TransientContainer object at 0x75b6b7d08290>
Input args: ('Tensor(shape=[1, 128, 4096], dtype=torch.float32)',)
Input kwargs: {}
Output tensors: ['Tensor(shape=[1, 128, 4096], dtype=torch.float32)']
Exception: You called set_weights(weights) on layer "multi_head_attention_1" with a weight list of length 8, but the layer was expecting 0 weights. Provided weights: [array([[[-0.05060041, -0.01487129,  0.10044055, ....
Traceback:
[Nobuco] Converting (DONE): |████████████████████████████████████████████████████████████████████████████████| 26/26 ops [00:00]
Legend:
Green — conversion successful
Yellow — conversion imprecise
Red — conversion failed
Red — no converter found
Bold — conversion applied directly
* — subgraph reused
Tensor — this output is not dependent on any of subgraph's input tensors
Tensor — this input is a parameter / constant
Tensor — this tensor is useless

sample code is in
https://github.com/AlexanderLutsenko/nobuco/pull/64
is this a known issue - or just something that's slipped by without anyone needing?

basically crafted some keras / torch classes - mostly working now
https://github.com/johndpope/IMF/blob/feat/tensorflow-cips/tf-export2.py


TaxonomyFormat assumes first line is header

When reading TaxonomyFormat with any of the transformers, the first line is assumed to be a (non-comment) header, followed by the taxonomy mapping lines. The sniffer is very lenient and only cares that the file is two-column TSV.
Not all taxonomy files include a header (for example, Greengenes). When a transformer is invoked to read the file, the first line is interpreted as a header, causing the feature ID to be set as Index.name and the taxonomy string to be set as Series.name.
For example, suppose we have the following taxonomy.tsv file (I used the first few lines from the Greengenes taxonomy map):
228054  k__Bacteria; p__Cyanobacteria; c__Synechococcophycideae; o__Synechococcales; f__Synechococcaceae; g__Synechococcus; s__
228057  k__Bacteria; p__Proteobacteria; c__Alphaproteobacteria; o__Rickettsiales; f__Pelagibacteraceae; g__; s__
73627   k__Bacteria; p__Actinobacteria; c__Actinobacteria; o__Actinomycetales; f__Mycobacteriaceae; g__Mycobacterium; s__
378462  k__Bacteria; p__Firmicutes; c__Bacilli; o__Bacillales; f__Staphylococcaceae; g__Staphylococcus; s__
89370   k__Bacteria; p__Firmicutes; c__Bacilli; o__Bacillales; f__Bacillaceae; g__Anoxybacillus; s__kestanbolensis

Reading this file into a pd.Series (all transformers are affected, it's not limited to the pd.Series transformer):
In [1]: from qiime2.plugin.util import transform

In [2]: from q2_types.feature_data import TaxonomyFormat

In [3]: import pandas as pd

In [4]: taxonomy_series = transform('taxonomy.tsv', from_type=TaxonomyFormat, to_type=pd.Series)

In [5]: taxonomy_series
Out[5]:
228054
228057    k__Bacteria; p__Proteobacteria; c__Alphaproteo...
73627     k__Bacteria; p__Actinobacteria; c__Actinobacte...
378462    k__Bacteria; p__Firmicutes; c__Bacilli; o__Bac...
89370     k__Bacteria; p__Firmicutes; c__Bacilli; o__Bac...
Name: k__Bacteria; p__Cyanobacteria; c__Synechococcophycideae; o__Synechococcales; f__Synechococcaceae; g__Synechococcus; s__, dtype: object

In [6]: # :(

In [7]:

The Series has its Index.name set to "228054" and its Series.name to "k__Bacteria; p__Cyanobacteria; c__Synechococcophycideae; o__Synechococcales; f__Synechococcaceae; g__Synechococcus; s__".
During classification, if a query sequence is assigned to the first reference sequence taxonomy (i.e. the one that's misinterpreted as a header), I think the code will error with an IndexError coming from pandas. This happened with @nbokulich's vsearch classifier (not yet in master), and adding a header line to the Greengenes file appears to fix the issue (he's not receiving the error anymore at least).
I don't think we've seen this error with the existing classifiers because we've never had a query sequence assigned to the first reference sequence (e.g. just by chance). I suspect (and hope) that the code would fail in a similar way with an IndexError, but haven't confirmed.
I propose that we require TaxonomyFormat (both when sniffing and reading) to have the following header (we've been using this header in the unit tests, maybe elsewhere):
Feature ID<tab>Taxon

... optionally followed by other columns that are ignored. We could finesse the column names a little -- something like feature_id and taxon would be easier to access from pandas objects. This is a minor detail we can work out later.
If we go with a stricter format, such as what I'm proposing, then importing files without the appropriate header (e.g. Greengenes and other reference databases) will raise an error and the header will have to be added to the file in order to import. This is annoying, but I really think we should stop supporting tabular files without headers, especially because we want the .qza file formats to be as self-documenting as possible.
Thoughts? cc: @gregcaporaso, @nbokulich, @BenKaehler, @ebolyen, @thermokarst, @jakereps
Thanks @nbokulich for finding and reporting this bug!

:+1: I think we are using Index (which may also be a good name) somewhere else, but not sure where...
Also, I agree that this is just going to be a catastrophic failure, so I don't think we need to worry about incorrect results at least.

Also, I agree that this is just going to be a catastrophic failure, so I don't think we need to worry about incorrect results at least.

I think it's likely to error noisily, but it'd be good to confirm that the error is produced as expected, and that the reason we haven't seen this error yet is due to non-assignment of the first ref seq. I only tested this with @nbokulich's branch on an entirely new method. With Greengenes having integer IDs, coupled with pandas __getitem__ indexing instead of .loc or .iloc, there's a lot that can go wrong (e.g. using an integer ID accidentally as an index, and the indexing is off-by-one since the header is misinterpreted).

With Greengenes having integer IDs, coupled with pandas getitem indexing instead of .loc or .iloc, there's a lot that can go wrong

I hadn't even put two and two together, that is a really good point. We need to figure out metadata and indexes.

Thanks @nbokulich and @jairideout. I agree about requiring a header line in tsv files.

I hadn't even put two and two together, that is a really good point. We need to figure out metadata and indexes.

I totally agree -- this came up yesterday in a different plugin, and it required the user to rerun their entire analysis, ouch!).

I hadn't even put two and two together, that is a really good point. We need to figure out metadata and indexes.

I totally agree

Should we try to discuss this on Tuesday to start working on a plan?

That'd be great! I think CI/CD and Metadata/Indexes are top priority.

I just rediscovered this bug. The scikit-learn based classifiers are quite permissive, so this just results in the top line being missing from the training set, and that just became an issue for me.
On whether the input files should be required to have headings, looking at the greengenes and unite data sets that I have, that doesn't seem to be the convention. That doesn't mean that they shouldn't be stored with headings in the .qzas, just that I don't reckon it's what people would be expecting when they try to load up standard data sets.

Thanks @BenKaehler! We need to get this fixed for the next release (2017.5). I'm happy to help with that.

On whether the input files should be required to have headings, looking at the greengenes and unite data sets that I have, that doesn't seem to be the convention. That doesn't mean that they shouldn't be stored with headings in the .qzas, just that I don't reckon it's what people would be expecting when they try to load up standard data sets.

I suppose we could support two taxonomy formats-- one with a standardized header (this one would also be the .qza's internal format) and one without a header (to support Greengenes, etc.). What do you guys think?
cc: @gregcaporaso @ebolyen @thermokarst @nbokulich

Supporting import of headless or headed taxonomy files sounds useful. maybe add a boolean parameter headless to specify whether the input file is headless or not? If headless, append the expected header for that semantic type. One advantage of doing this is that it is flexible in case this issue may arise with other semantic types.
Either that or at least raise an error if the expected header is absent — I prefer @jairideout 's suggestion though.

maybe add a boolean parameter headless to specify whether the input file is headless or not?

👍 The framework doesn't currently support parameters to file formats / directory formats but there's been discussion in the past to support this feature.
With the current framework support, if we have two file formats (header & headerless), users would need to supply an explicit --source-format when importing the headerless file format (since the header file format would be the default format used in .qza's).
So if we define two file formats, users will still need to manually intervene when importing the headerless format. If there is a single format, users would need to intervene by adding the appropriate header to the file before importing.
Having two formats sounds a little more user-friendly but in either case the user will have to take some sort of action when supplying a headerless format (there isn't a "seamless" solution possible right now).
I don't have strong feelings either way (and happy to implement whatever we decide on).

I would definitely lean towards the two formats (and in the future collapsing to a parameter [does this potentially bump priority?]). Having users edit their input can end poorly sometimes.


Rewrite the prettyprinter

Our prettyprinter is written in a strange and inefficient style. Also, it has quite a number of bugs. The current consensus among the project maintainers is that we need to rewrite our prettyprinting module.

Duplicate of https://github.com/scalameta/scalameta/issues/567.


Remove unnecessary call to type function

The function output is ignored and type function has no side effects, so it should be safe to be removed.

@abresas as shown by failed tests that call to type is needed as pointed out by the comments right above:

Construct a new model with the new fields to allow self referential primary key to resolve to. This model won't ever be materialized as a table and solely exists for foreign key reference resolution purposes. This wouldn't be required if the schema editor was operating on model states instead of rendered models.

when a new subclass of Model is created ModelBase metaclass logic is ran, so the call to type triggers ModelBase logic which handles also foreign keys resolution.

@abresas as shown by failed tests that call to type is needed as pointed out by the comments right above:

Construct a new model with the new fields to allow self referential primary key to resolve to. This model won't ever be materialized as a table and solely exists for foreign key reference resolution purposes. This wouldn't be required if the schema editor was operating on model states instead of rendered models.

when a new subclass of Model is created ModelBase metaclass logic is ran, so the call to type triggers ModelBase logic which handles also foreign keys resolution.

I believe mentioning this in a comment would be nice.

Closing as @HMaker pointed it is mentioned in a comment.


Updates to use GitHub Action for codecov

The codecov GitHub Action replaces the uploader script for projects that
are either mostly amd64 or can consolidate uploads into one stage that
runs on amd64. func-e has no significant arm64 source files, so can use
this action instead of the uploader script.
This also bumps versions which should get rid of the dependabot warning.
Thanks to @anuraaga for noticing the uploader script is deprecated.

note: I switched the tests to use os/x 12 because recently the envoy build no longer works with multiple OS/x versions cc @cpakulski


Doesn't work actually

Uncaught ReferenceError: SharedArrayBuffer is not defined
at stockfish.js:33:131
at index.js:17:1

Looks like there is a recent behavior where SharedArrayBuffers are outlawed unless you have the right HTTP Headers
Added a ServiceWorker hack to serve the correct headers with Github Pages:
https://github.com/gzuidhof/coi-serviceworker
Clear the browser cache and try loading the page again.

Perfect !


Getting Started Guide

Guide showing steps to get started that don't assume Delphi is installed

Installing Android SDK if necessary
Installing Adopt Open JDK if necessary
Installing PythonFMXBuilder
Deploying an application
Troubleshooting

Added the guide to wiki at the following link: https://github.com/Embarcadero/PythonFMXBuilder/wiki/Step-by-step-guide-to-create-an-Android-application-using-PythonFMXBuilder