Script is unable to find "Next" button and throws AtributeError when done with gallery

Most recently receiving the following error:
Traceback (most recent call last):
  File "furaffinity-dl.py", line 200, in <module>
    next_button = s.find('button', class_='button standard', text="Next").parent
AttributeError: 'NoneType' object has no attribute 'parent'

All gallery pages were downloaded, and I assume there's supposed to be a catch to see if this is the last page in a gallery or not?

This should be fixed with PR #65


ovs: Support user space only interface

The OVS bridge and OVS port is user space only iface which can hold the
duplicate name against kernel interfaces. In order to support that use
case, introduced:

OvsBridgeIface.is_user_space_only = True
OvsPortIface.is_user_space_only = True
Ifaces.all_ifaces()
Ifaces.all_kernel_ifaces
Ifaces.all_user_space_ifaces
Ifaces.get_iface(iface_name, iface_type)
Ifaces.get_cur_iface(iface_name, iface_type)
Ifaces.add_ifaces()

The limitation of user space interfaces are:

Cannot be used subordinate of other kernel interfaces.
Due to limtation of nmstate, currently cannot be used as
subordinate of other user space interfaces.

Integration test case added.

Wait on PR #1405


Remove Rubinius from allowed failures list

As the bug that made issues here: https://github.com/roidrage/lograge/pull/121 is fixed in the latest release we can remove RBX again from the allowed failures.
The JRuby builds are also fine again. Should we do the same there?

Unless I'm mistaken, this would require a new release of RBX first.
With regards JRuby, sure - we should move those out. :+1:
We should probably add ruby-head and 2.2.2 also, perhaps retiring some of the older 2.X series rubies at the same time?

All green, ignore me! 😂

Yes we should also clean those up.
I will see what I can do during the day (:


Accelerometer scale

Hi,
There is something I do not understand. There is a scaling made here for the accelerometer but it is just a divider by 100 instead values given in the datasheet.

1 LSB = 3.91 mg (2-g range)
1 LSB = 7.81 mg (4-g range)
1 LSB = 15.63 mg (8-g range)
1 LSB = 31.25 mg (16-g range)

Is it intended ? I am in the process of implementing it for the interrupts, should I put the right scale here too ? From my point of view it will be much cleaner but it is also a breaking change.

Oops, it is only for interrupt thresholds. For acc values the datasheet says:

1 mg = 1 LSB

That said why dividing by 100 and to 1000 as 1 g = 1000 mg ?

I used this repo as an example https://github.com/ShellAddicted/BNO055ESP32 and looks like I got these values from it.
1000 makes more sense as milli = 10^-3


502 @ auth0.nuxt.org

The link in the repo description points to a page that says

502 | An error occured with your deployment.

Best karfau

Confirm, demo is not working.

It's fixed :+1:


double-imported esm and cjs modules in the esbuild dist for RapiD

Here's a view of the docs/statistics page currently:

All the pixi modules are double-imported, once for ecma script modules (ESM) and once for commonJS (CJS). This leads to a very large bundle size, because we're effectively importing the entirety of pixi twice.  Need to investigate why this is.

Fixed in 8b3761b70770161a711dc9f00425888690507b74 while upgrading to v7.
In the course of upgrading to v7, I decided to wholly gobble up the pixi-dashed-line repo and rehost / rebuild it with our country-coder techniques as the last release was well over a year ago and there has been no movement on the original repo.
The new repo is at https://github.com/Bonkles/pixi-dashed-line.
This had the added benefit of driving out the double dependencies!


Real merkle proofs

Fix up merkle proof generation and proving to match latest real proofs being generated by the relay chain.
Enable real proof verification and validator set updates on the validator registry

working :D - e2e tests all pass
coming up in next PR, enabling parachain merkle proofs on the parachain verification and enabling the full e2e verification for messages

@vgeddes @philipstanislaus update: e2e tests all pass. ready for review :)

included in https://github.com/Snowfork/snowbridge/pull/481


Extension Pack not really needed?

Had the Oracle Extension Pack installed when setting up a 10.14 Mojave and a 10.15 Catalina VMs, as required by this script.
But was able to have them run properly for many weeks without an Extension Pack. I think I only had to downgrade the USB controller to USB 1.1 (OHCI) Controller for them.
Just a FYI… Maybe worth adding an entry to README? Not sure if the installation would also work without the Extension Pack, but that would be great.

Last time I checked the extension pack is needed for mouse/keyboard support.

Had these VMs running with latest VirtualBox 6.0 under macOS 10.15 and with latest VirtualBox 5.2 under Ubuntu 18.04. No issues at all running them and using mouse and keyboard to connect. I guess USB 1.1 is good enough.


Fix misc RBAC issues on 1.6.6

Any way to get a test for this (failing without the fix and passing with it) ?
Also can you describe a bit the fix/symptoms etc by editing the PR description

Jenkins job istio/presubmit passed

Create a cluster in GKE with 1.6.6 version and deploy istio, then do kubectl logs pilot-... , kubectl logs ingress-....


[Bug]: JSONForm select field does not open when used in an nested field (Array/Object)

Is there an existing issue for this?

[X] I have searched the existing issues

Description
The select field menu does not open when it is added to an Array Item/Array of Object/Object or any nested structure.
Steps To Reproduce

Add a JSON Form
Add a new custom field
Change the type to Object
Add a new field under that object
Change the newly added field type to select
Try clicking the select field on the form, the menu won't open.

Public Sample App
No response
Version
Cloud

Godspeed @ashit-rath 🤗 💟


web.lua crashed after 10 hours runtime

I noticed the web.lua crashed after 10 hours of running, with this message:
/root/torch/install/bin/luajit: /usr/local/share/lua/5.1/turbo/ioloop.lua:518: attempt to index local 'handler' (a nil value)
stack traceback:
/usr/local/share/lua/5.1/turbo/ioloop.lua:518: in function '_run_handler'
/usr/local/share/lua/5.1/turbo/ioloop.lua:436: in function '_event_poll'
/usr/local/share/lua/5.1/turbo/ioloop.lua:425: in function 'start'
web.lua:204: in main chunk
[C]: in function 'dofile'
/root/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:131: in main chunk
[C]: at 0x004064d0

I guess the turbo.lua has some bugs.
I am using upstart to respawn a server process.
see appendix/waifu2x.upstart.conf.


Dev

Merge the great debloat into main

Objection, your honour. It is to my attention that pixdoet has been signing commits without gpg keys.


BUG: NPSP Data Importer ignores Home Address for an import record that only has Account1 and no Contacts.

Should either generate an error and marked the di as failed, or at least mark the Home Address Import Status field as "ignored".
It might be generating this error, even though the Account gets created:
Update failed. First exception on row 0; first error: MISSING_ARGUMENT, Id not specified in an update call: []
I'm not sure since this is marked on the record with failed for import status, and yet the Account is created and the lookup set on the import record.

**lurch: attach W-032951


Add support for root models to Laravel Breeze

Laravel Version
10.28.0
PHP Version
8.2.11
Database Driver & Version
MySQL 8.0.32
Description
Sorry, this is arguably a feature request, but let's see...
After the change in Laravel 8 to place models in App/Models by default, some of us have held onto the support for storing models in the App folder.
Thankfully the framework supports this (so php artisan make:model knows where to put new models), but the default Laravel Breeze installation still doesn't...
Obviously the user can easily search and replace App\Models\User for App\User, but it would be great if they didn't have to.
There is one controller (RegisteredUserController.php) and one request (ProfileUpdateRequest.php), as well as several tests that Breeze adds, that reference the default location.
Steps To Reproduce

Install a new Laravel project
Move User.php out of the app/Models folder to the app/ folder
Delete the app/Models folder
Run php artisan make:model Test and ensure it is created in the right place
Install Laravel Breeze (composer require laravel/breeze --dev and php artisan breeze:install)
Try and register on the default app

You will get an error because RegisteredUserController.php is still referring to app/Models instead of app/.
I guess the solution would be for Breeze to detect the absence of the models folder?

I’m sorry but we have no plans for this. Breeze is a scaffold tool and by default references the location of the place where the model is by default in the skeleton.

Ok, but please don't remove this support from Artisan. Many of us still prefer the Laravel 7 folder structure! Thanks


Possibly invalid reference to 'NotUnderContentRootModuleInfo.project'

Hello! I am a maintainer of the Kotlin IntelliJ IDEA plugin.
During the plugin refactoring, I found out that the JBang plugin uses the NotUnderContentRootModuleInfo.project method.
Here is the exact call site:
https://github.com/jbangdev/jbang-idea/blob/af96dbaf65d1fdf5a3c23f81e8f39b725dfdc5a0/src/main/kotlin/dev/jbang/idea/actions/CreateFromTemplateAction.kt#L71
I suppose using NotUnderContentRootModuleInfo.project there is unexpected, as the method always returns null.
It would be great if you review this code fragment.

@yanex  thanks for your remind.  NotUnderContentRootModuleInfo.project was introduced by auto import, and I have fixed it already.


Cannot create Host on EC2.  Spins on "Installing Rancher Agent"

I have used the UI multiple times to try and add a host to a newly created Master from EC2.  The process hangs at "Installing Rancher Agent".  This is what I see in the logs:
time="2017-01-06T18:55:31Z" level=info msg="stdout: Creating CA: /var/lib/cattle/machine/9d099016-8f03-4a33-a76b-c7d81f98a2f2/certs/ca.pem" resourceId: =1ph7
time="2017-01-06T18:55:31Z" level=info msg="stdout: Creating client certificate: /var/lib/cattle/machine/9d099016-8f03-4a33-a76b-c7d81f98a2f2/certs/cert.pem" resourceId: =1ph7
time="2017-01-06T18:55:31Z" level=info msg="stdout: Running pre-create checks..." resourceId: =1ph7
time="2017-01-06T18:55:31Z" level=info msg="stdout: Creating machine..." resourceId: =1ph7
time="2017-01-06T18:55:31Z" level=info msg="stdout: (hostx01) Launching instance..." resourceId: =1ph7
time="2017-01-06T18:55:46Z" level=info msg="stdout: Waiting for machine to be running, this may take a few minutes..." resourceId: =1ph7
time="2017-01-06T18:55:46Z" level=info msg="stdout: Detecting operating system of created instance..." resourceId: =1ph7
time="2017-01-06T18:55:46Z" level=info msg="stdout: Waiting for SSH to be available..." resourceId: =1ph7
time="2017-01-06T18:56:34Z" level=info msg="stdout: Detecting the provisioner..." resourceId: =1ph7
time="2017-01-06T18:56:34Z" level=info msg="stdout: Provisioning with ubuntu(systemd)..." resourceId: =1ph7
time="2017-01-06T18:57:16Z" level=info msg="stdout: Installing Docker..." resourceId: =1ph7
time="2017-01-06T18:58:56Z" level=info msg="stdout: Copying certs to the local machine directory..." resourceId: =1ph7
time="2017-01-06T18:58:57Z" level=info msg="stdout: Copying certs to the remote machine..." resourceId: =1ph7
time="2017-01-06T18:58:58Z" level=info msg="stdout: Setting Docker configuration on the remote daemon..." resourceId: =1ph7
time="2017-01-06T18:59:00Z" level=info msg="stdout: Checking connection to Docker..." resourceId: =1ph7
time="2017-01-06T18:59:00Z" level=info msg="stdout: Docker is up and running!" resourceId: =1ph7
time="2017-01-06T18:59:00Z" level=info msg="stdout: To see how to connect your Docker Client to the Docker Engine running on this virtual machine, run: docker-machine env hostx01" resourceId: =1ph7
time="2017-01-06T18:59:00Z" level=info msg="Machine Created" machineExternalId=9d099016-8f03-4a33-a76b-c7d81f98a2f2 resourceId=1ph7
time="2017-01-06T18:59:00Z" level=info msg="Creating and uploading extracted machine config" resourceId=1ph7
time="2017-01-06T18:59:00Z" level=info msg="Machine config file created and encoded." destFile="/var/lib/cattle/machine/9d099016-8f03-4a33-a76b-c7d81f98a2f2/hostx01.tar.gz" resourceId=1ph7
time="2017-01-06T18:59:00Z" level=info msg="Activating Machine" eventId=f216e108-a808-458d-a7c4-e179304aff05 resourceId=1ph7
time="2017-01-06T18:59:01Z" level=info msg="pulling rancher/agent:v1.1.3 image."
time="2017-01-06T18:59:12Z" level=info msg="Container created for machine" containerId=a97e356298822a60cedec92f67b188169f67b92c52cdb628e14576a72f74041c machineId=1ph7 resourceId=1ph7
time="2017-01-06T18:59:12Z" level=info msg="Rancher-agent for machine started" containerId=a97e356298822a60cedec92f67b188169f67b92c52cdb628e14576a72f74041c machineExternalId=9d099016-8f03-4a33-a76b-c7d81f98a2f2 resourceId=1ph7
time="2017-01-06T18:59:12Z" level=info msg="Creating and uploading extracted machine config" resourceId=1ph7
time="2017-01-06T18:59:12Z" level=info msg="Machine config file created and encoded." destFile="/var/lib/cattle/machine/9d099016-8f03-4a33-a76b-c7d81f98a2f2/hostx01.tar.gz" resourceId=1ph7

Useful
Info

Versions
Rancher v1.3.0 Cattle: v0.175.2 UI: v1.3.5

Access
github admin

Orchestration
Cattle

Route
host.containers

nvm...I needed to open another port on my server.

Hi @lwjameson - can I please ask what your port/security config is that fixed your problem? I'm currently having this exact same issue and trying to find a fix. thanks!

Hi Onyx.  I needed to open port 8080 on the security group for the master server.  I had set it to only allow connections from myself.  Since I am mostly just evaluating Rancher at this point I opened it to everyone and am relying on the authentication on Rancher for security.  I am sure it can be set up more elegantly. One way or the other, the host machines in the cluster need access to 8080 to communicate with the rancher API.
Lorin


Use symfony translations for title in xml templates

A property in the xml templates currently look like the following:
<property name="title" type="text_line" mandatory="true">
    <meta>
        <title lang="de">Titel</title>
        <title lang="en">Title</title>
     </meta>
     <tag name="sulu.rlp.part"/>
</property>

Instead of the meta tag with the titles it would be nice to use the translation mechnism from symfony.

Please for all stuff, which can have a translation.

nice it would be to provide both of it!

jup :+1:

I think this issue should be prioritized even higher now, since we also define forms like contacts and accounts using this style now.
If we do not support that now, external contributors won't be able to translate these fields without touching our files, and that's actually what we would like to avoid and use some other online tools instead (i.e. https://lingohub.com/)


New Release?

@michaelhelmick It's been a over a year since the last release. Time for a new one perhaps?

Sorry about that, 3.5.0 is now available! 🍰


Record exception if SearchParameterStatus out of sync with resource

Description
It is possible with failures during certain race conditions between multiple FHIR instances to get the SearchParameterStatus out of sync with the underlying SearchParamterResource object.  Currently the service throws an exception, but we will now log the error.
Also added integration tests which replicate the issue.
Related issues
Addresses [issue AB#87426].
Testing
New integration tests added which simulate a second FHIR service instance.
FHIR Team Checklist

[X] Update the title of the PR to be succinct and less than 50 characters
[X] Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
[X] Tag the PR with the type of update: Bug, Dependencies, Enhancement, or New-Feature
[X] Tag the PR with Azure API for FHIR if this will release to the Azure API for FHIR managed service (CosmosDB or common code related to service)
[X] Tag the PR with Azure Healthcare APIs if this will release to the Azure Healthcare APIs managed service (Sql server or common code related to service)
[X] CI is green before merge
Review squash-merge requirements

Semver Change (docs)
Patch|Skip|Feature|Breaking (reason)

/azp run


Logged In Navbar Improvements

Changes

Restructured the logged in navbar
Added the sign out button in the manage wallet modal

thanks jakz!!! feels 100% better. couple comments:

let's remove the address option in the dropdown

let's rename this to say "Manage Account"

@Robinnnnn Updated the changes 👍

thank you @jakzaizzat !!!


Thank you!

Thank you!
Thank you for your project and software!
Thank you for maintenance in the past and in the future!
Kind regards and season's greetings!

Thank you very much! These kind words are highly appreciated :)


ENH: Implementing current density as output of empymod

Message from @efinden

To understand more of the free and polarization currents in the ground it would be useful to adjust the output of empymod simply by multiplying the electric field by the eta H-function, in order to obtain the total current density in the frequency domain and then use model.tem to transform it to the time-domain.
In other words, just to use Ohms law in the frequency domain: J(w) = E(w)*etaH(w).

Thank you very much for the new implementation!
I did some testing today and have a question on which locations in space that etaH is computed.
Is the etaH working on the output of the electric field for every receiver-location?
I tested one case with resisivity=200 Ohm-m, for a halfspace over air of resistivity 1e40. The air layer has a nearly zero permittivity, and thus etaH for the air layer should be very low (I used res=1e-40). The configuration is a rectangular bipole source in the air layer at z=-0.1 m. Electric source. Electric receiver.  Receivers are spaced out between x,y,z = [-5,5]. The current density of the air layer is very low, as expected, but the current density is also very low for the 200 Ohm-m halfspace.
I would perhaps expect the y-component of the electric field multiplied with the etaH-function in the ground of 200 Ohm-m to be much larger. Maybe it is something in my missing understanding of the implementation that is causing this results.

I pushed a change, could you test again? Now it should multiply it with the etaH from the receiver layer

Thanks! It is fine, then I understand it as well!
A loop over the receiver depths will make it!
I would like to do it but I am not quite sure where.

A test of a pure resistive  200 Ohm-m halfspace, in the diffusional model (which means setting the permittivity to zero) is done. When asking for receivers to show the y-component of the electric field, and with the ecurrent = True, the modified code seems to reproduce exactly the current density as obtained when the conductivity is multiplied with the electric field outside empymod (on the output of empymod).
Thank you very much!
I look forward to investigate the polarization currents now for an IP case. Also, I will try to model a larger domain to see if the "smoke ring" of currents caused by the TEM system behaves as expected with depth.

Great! As I said, I only implemented according to your description - I haven't done any background research/read-up yet or thought deeper about it, so it is good to keep being curious about the feasibility of the outcome.

Thank you! I did a bit of background check on the physics before I asked and it seems all right, but other considerations are always welcome.
I post below an image of the current density due to a TEM switch off from a 2x2 loop lying 10 cm above the surface for a 200 Ohm-m halfspace.


remove mmio_region parameter from handle::open

Remove mmio_region parameter from handle::open
Add handle::map_csr which calls fpgaMapMMIO

uint8_t* map_csr(uint32_t csr_space)

See https://github.com/OPAE/opae-sdk/pull/263

#266 removed the need for mapping MMIO from C++ and #271 removes the csr_space from handle::open


Storyshots with Storybook 4 for React fails with Angular import.

Framework Loaders tries to load Angular stuff in React environment
After updating from 4.0-alpha.4 to -alpha.9 I see the following happens:
Cannot find module '@storybook/angular/options' from 'loader.js'

      at Resolver.resolveModule (node_modules/jest-resolve/build/index.js:210:17)
      at Object.<anonymous> (node_modules/@storybook/addon-storyshots/dist/angular/loader.js:7:16)

This is the not touched Story.test.js which is used to execute the test runner:
import initStoryshots, {
  multiSnapshotWithOptions
} from "@storybook/addon-storyshots"
import { dirname } from "path"

initStoryshots({
  // Delegate to centralized config
  configPath: dirname(require.resolve("edge-storybook")),

  // Storing seperate snapshots for each individual story
  test: multiSnapshotWithOptions({}),

  // Ignore all containers, only snapshot pure components
  // without any app logic or data fetching
  storyKindRegex: /^((?!container).)*$/i
})

Steps to reproduce
Executing jest on the command line.
Please specify which version of Storybook and optionally any affected addons that you're running

storybook/react 4.0.0-alpha.9
storybook/addon-actions 4.0.0-alpha.9
storybook/addon-links 4.0.0-alpha.9
storybook/addon-storyshots 4.0.0-alpha.9
storybook/addon-viewport 4.0.0-alpha.9
jest 23.1.0

It will be fixed with #3745

The issue seems relatively obvious after starting studying the current code base:
This is the code from src/frameworkLoader.js:
import loaderReact from './react/loader';
import loaderRn from './rn/loader';
import loaderAngular from './angular/loader';
import loaderVue from './vue/loader';
import loaderHTML from './html/loader';

const loaders = [loaderReact, loaderAngular, loaderRn, loaderVue, loaderHTML];

function loadFramework(options) {
  const loader = loaders.find(frameworkLoader => frameworkLoader.test(options));

  if (!loader) {
    throw new Error('storyshots is intended only to be used with storybook');
  }

  return loader.load(options);
}

export default loadFramework;

Three of these loaders are trying to import @storybook/angular/options:

angular
html
vue

The loader system might need some rework that there is no assumption for frameworks which are not installed or needed.

This is a bug that happened recently, and after the refactoring, it will be fixed.

@igor-dv may we get a new alpha version with this fix?

I assume it will be soon

Any news on the next alpha version? Will it be<EMAIL_ADDRESS>?


Random carouselId breaks Jest-Snapshot tests

Concerns mjml-carousel only.
This line](https://github.com/mjmlio/mjml/blob/master/packages/mjml-carousel/src/Carousel.js#L45) generates a fully random id which is used on many css-classes and elements. On each render run it is different, so all snapshot tests involving a file that renders a carousel, brake.

Would be nice to have a static version for testing ?!
Or actually why use that random id in the firstplace ?

Well the id is needed to ensure that you can use multiple carousel in a single email. You can just stub the crypto package for that https://github.com/mjmlio/mjml/blob/master/packages/mjml-carousel/src/Carousel.js#L45

@iRyusa  thanks so much for the super quick reply.
and: True! 🤦♂ stubbing is the way to go ..


[v2] got an error in console on login-screen-close

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Framework7 Version: 2.0.0-beta1

Platform and Target: all

Live Link or JSFiddle/Codepen: none

What you did
Open login-screen demo as overlay
Expected Behavior
it works well without any message in console
Actual Behavior
it works with this message in console:
TypeError: Argument 1 ('node') to Node.appendChild must be an instance of Node

Fixed by https://github.com/nolimits4web/Framework7/commit/643d9f3696a192eeb461ab488e3a572ebb4a7f53


ipsec: Deprecate global IPsec keys

Using a single IPsec key for all IPsec tunnels is insecure. It was only preserved to allow for a smooth switch to per-tunnel keys. Per-tunnel keys have been released in v1.13, v1.14, and v1.15, so we can now deprecate the insecure alternative for v1.16.

/test

/test

I think this would benefit from a mention in the upgrade notes. And maybe a few lines in the IPsec docs to describe the new key system, so that users understand what the deprecation actually means?

Done at https://github.com/cilium/cilium/pull/33564.


add @Deprecated to all generated overloads of method with defaulted params

Currently, the backend only adds a @Deprecated annotation to one of the generated overloads of a deprecated method with defaulted parameters. This isn't perfect from the perspective of Java clients, and it's causing me a minor bug in the IntelliJ IDE. I think it should add @Deprecated to all overloaded versions.

Thanks Tom!


Add test in Test Script to test the CASJobs Button

Add test in Test Script to test the CASJobs Button.

One way in the application to make this clear is to change the drag icon to the RED icon - right now it uses the Green one, suggesting that it should be possible (when it is not).

We note that when trying to create a NEW file in the root folder, a message pops up saying that you are not allowed to do so.

Decided to remove CasJobs button. Therefore no need for adding test script.


Adding 'New College Lanarkshire'

Main website: https://www.nclanarkshire.ac.uk/
List of computing related courses available at this College: https://www.nclanarkshire.ac.uk/courses/computing-and-digital-technologies
Email login recognising official domain for students:

@CJ-Resdiary First stage of verification passed, official domain is nclanarkshire.ac.uk as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://www.nclanarkshire.ac.uk/your-student-portal

@CJ-Resdiary Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Replace Decentraleyes by LocalCDN

Decentraleyes has not been updated in almost a year and I heard development has stopped. LocalCDN can do exactly the same but it's updated way more often. It even passes Decentraleyes testing utility.


Story on embedded resources seems to be rather outdated

In http://fsharp.github.io/2015/09/29/fsharp-compiler-guide.html#how-f-handles-embeddedresource-items-differently-to-c-details there's a long description about differences between how F# handles embedded resources. It links to several tracking issues, all of them resolved last year, one of them as won't fix.
I'm not quite sure which part of that story still apply, but at a minimum, I think someone knowledgeable about the subject could update it to contain the latest information. Maybe the whole section can be removed completely if the bugs in F# resource management have disappeared (with only a note saying the xBuild is not working properly).
Not sure this should be reported here, I'm happy to move it elsewhere if needed.
Relevant related issues:

#922, fixed, I think via https://github.com/fsharp/fsharp/pull/761, but if it is a 100% fix I cannot tell
https://github.com/fsharp/fsharp/issues/562, not fixed for xbuild, quote from @dsyme:

A workaround is to use msbuild on Linux (xbuild is basically being deprecated slowly).

#1050, fixed via https://github.com/Microsoft/visualfsharp/pull/3352

cc @dsyme to update that section


Mm 43418 deliquency modal subscription

Summary
This PRs add the delinquency modal for admin users. This modal will only display after 90 days, workspace and admin console. The modal will appear each time the user login until they select update billing information or stay on starter plan.
Ticket Link
https://mattermost.atlassian.net/browse/MM-43418
Screenshots
Downgrade Modal

No limits surpassed Modal

Boards limits Modal

Storage limits Modal

Messages limits Modal

Integrations limits Modal

Multiples limits Modal

Release Note
After 90 days since the day of missing payment, admins will see a modal where they have to choose between update billing status or stay on Starter Tier.

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

@nickmisasi @johndavidlugtu @annelieseklein gentle ping for review this one

I am also wondering if the designs changed before the screenshots were taken. The screenshot seem to be different from the figma designs attached to the ticket

Can we make this "Cloud Starter"?

@Afsoon

the workspace has been downgraded modal needs to have rounded corners like the other modals

@AGMETEOR For the modal of cloud limits we talked on a sync to reuse the cloud limits that Nathaniel did instead of do it again.
@johndavidlugtu and @annelieseklein noted to be changed.

@Afsoon the test server seems to have been downgraded to Cloud Starter already. Can you confirm that this: https://github.com/mattermost/mattermost-webapp/pull/11095#issuecomment-1248190800
has been resolved?

@johndavidlugtu Yes, the change is applied. I have just this morning pushed the changes.

@annelieseklein the downgrade modal design update

manual test case added for to test in release.
MM-T5190

@Afsoon @yasserfaraazkhan delinquent_since value has been set to a value > 90 days ago

Hello @annelieseklein. Gentle ping

@annelieseklein we are going to go ahead and merge this as it needs to be cherry picked to cloud for full feature testing. @nickmisasi will provide a test server with different states in delinquency for final review.

/cherry-pick cloud

/cherry-pick cloud

Cherry pick is scheduled.


Use Choosewin on Open File

I like to use the choosewin plugin. And I found it useful to combine it with NERDTree on open a file. Case when open a file from within NERDTree straight forward, it choose the last active window to show the new buffer. Often I've open multiple windows and rly wanna specify in which one the buffer should been opened.
With this simple adjustment this is possible. If the plugin can be found by name in the runtime path, the activateFileNode() function first call choosewin, so the user can select a window.
Cause the following procedure in NERDTree use local buffer variabels, it was necessary to backup the window ID of the NERDTree window and jump back to it, after the user has choosen the window.
Despite choosewin hasn't that much stars (if that is an indicator), this new functionality should do not bother anyone, cause it only works for these, which have installed this additional plugin. And I guess these will love it.
Maybe the README can be adjusted to promote this relation to choosewin.

@weilbith, thanks for the submission, but I am going to pass on this because it couples NERDTree with another plugin, and that sort of dependency could lead to compatibility issues down the line.
A better option for this is to turn it into a NERDTree plugin. Save this code as ~/.vim/nerdtree_plugin/choosewin_open.vim, and it will do the same thing as your pull request. Keeping that file out of the nerdtree folder enables you to keep NERDTree up to date with master.
call NERDTreeAddKeyMap({ 'key': '<2-LeftMouse>', 'scope': "FileNode", 'callback': "ChooseWinOpen", 'override': 1 })
call NERDTreeAddKeyMap({ 'key': g:NERDTreeMapActivateNode, 'scope': "FileNode", 'callback': "ChooseWinOpen", 'override': 1 })

function ChooseWinOpen(node)
    " Select the window where to open the file, if the 'choosewin' plugin is
    " in the runtime path.
    " It is necessary to jump back the NERDTree window, cause local buffer
    " variables are required for the following procedure.
    if &rtp =~ 'vim-choosewin'
        let l:nerdwindow = win_getid()
        call choosewin#start(range(1, winnr('$')))
        call win_gotoid(l:nerdwindow)
    endif

    call a:node.activate({'reuse': 'all', 'where': 'p'})
endfunction

You could even package it up nicely as another Vim plugin others could install from github. Something like this:
+- nerdtree_choosewin_plugin  (This is the name of your repo.)
   +- nerdtree_plugin
   |  +- choosewin_open.vim
   +- README.md

@PhilRunninger u've good arguments.
Thanks for the further instructions! :+1:


Dispatch tool available as NuGet package

We are creating a bot app for a client. To run the dispatch tool manually is not really handy for the client so we're investigating the possibilities of running the dispatch tool in azure as a function, opposed to from CMD prompt.
I would like to request to have this dispatch tool available as a NuGet package so we can use azure functions to run or dispatch files.
If anybody has a solution, I would greatly appreciate it. I have tried importing the Dispatch.dll in our azure function project without success. We're using c# functions, would creating a JS functions project be a solution to somehow?

Will be fixed when all tools are published for Ignite.

@tsuwandy can you please give a status update of this?
Also, this feature request is related: https://github.com/Azure/azure-sdk-for-net/issues/5278

Any update about this?


container CPU headroom widget calculation is not correct

@natoscott is it possible to add a cgroup.cpusched.quota metric to the cgroup agent? cpu.cfs_quota_us should be the correct file in the pseudo-filesystem.

@spiermar yep, sure, will look into that this week.

cgroup.cpusched.cfs_{period,quota} metrics have been added to the pcp master branch.

Awesome! Thanks @natoscott ! Once that is out, I'll change the Vector widgets.


CLI timeout too short

Rancher Versions:
1.5.8
Server:
1.5.8
healthcheck:
ipsec:
network-services:
scheduler:
kubernetes (if applicable):
Docker Version:
OS and where are the hosts located? (cloud, bare metal, etc):
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single node
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Setup Rancher CLI, when there're lots of containers, commands timeout. Timeout value seems too small.
Results:
ubuntu@ip-172-31-32-163:~/netstress$ time rancher ps -c --format json
FATA[0016] Get http://ec2-52-23-171-71.compute-1.amazonaws.com:8080/v2-beta/projects/1a5/containers?limit=-2&removed_null=1&state_ne=inactive&state_ne=stopped&state_ne=removing&system=false: net/http: request canceled (Client.Timeout exceeded while awaiting headers)

real    0m16.422s
user    0m0.108s
sys    0m0.016s

Configurable client timeout would be nice.

Thanks for your report!
With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.
If you think we should keep this issue open, please let me know.


Incorrect expected chunk count

Version
latest
Platform
Darwin MacBook-Pro.local 22.5.0 Darwin Kernel Version 22.5.0: Mon Apr 24 20:53:19 PDT 2023; root:xnu-8796.121.2~5/RELEASE_ARM64_T6020 arm64
Description
When executing a command to retrieve multiple datasets, cryo returns an incorrect expected chucks, thus an incorrect ETA.
For example, a command of
cryo blocks txs traces --blocks 18183000:latest

latest = 18219277 @ execution time
Expects 37 chunks, but the actual is 74:
collection summary
──────────────────
- total duration: 1513.587 seconds
- t_start: 2023-09-26 12:12:50.922
- t_end:   2023-09-26 12:38:04.509
- chunks errored:   0 / 37
- chunks skipped:   0 / 37
- chunks collected: 74 / 37
- blocks collected: 72,556
- blocks per second: 47.9
- blocks per minute: 2,876.2
- blocks per hour:   172,571.2
- blocks per day:    4,141,709.6

Example of incorrect ETA:

fixed in 322d665c42bec408342555396f9bcbfc9ab5629c


Thinkware has never warned about line safety

While Thinkware F800 https://github.com/130s/hut_10sqft/issues/299 has been recording front view camera really great (confirmed the other day by copying the saved files from SD card into a computer), I've never heard it warned me for any safety issues it detects, other than occasional ping about speed meter being ahead.

I logged on its mobile app to the Thinkware device (first time since...I installed it?) then noticed safety features were "disabled". I enabled them. We'll see if this fixes.

Since then the device seems to start alerting me by a few different sounds:

Nearing or crossing lane lines?
Distance closer with the car in front.
Distance too closer with the car in front.
So far I found the notification for sideline is helpful as I don't see them, esp. the right lane, very well while driving.


Track - Playback speed menu from Track your expenses modal is cut off when exit full screen

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.28-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4913022&group_by=cases:section_id&group_order=asc&group_id=309130
Issue reported by: Applause Internal Team
Action Performed:

Go to FAB> Track
Verify "Track your expenses" modal appears
Open this modal in full screen
Exit full screen
Open the Playback speed option from 3-Dot menu

Expected Result:
Playback speed menu from Track your expenses modal should be fully visible
Actual Result:
Playback speed menu from Track your expenses modal is cut off when exit full screen
Workaround:
Unknown
Platforms:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/2c18386c-11f5-4105-b14d-f62f604d9fb9

View all open jobs on GitHub

@adelekennedy  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

I can't quite reproduce this one - I'm on a slightly later version (v9.0.29-6) and this is a fairly minor bug so closing!
https://github.com/user-attachments/assets/69ca4c68-9e36-4714-85cf-0b4dd95888e7


Fix #337 with normal plus and css transform

Reasons for making this change
My attempt to fix #337 as described in my comment there.

Checklist

[x] I'm adding or updating code

I close this in favor of #375.


Tail-recursive fibonacci

The naive implementation is extremely inefficient. Not only is it slow, on my computer it simply cannot compute the 30th number due to Exception in thread "main" java.lang.OutOfMemoryError: GC overhead limit exceeded after nearly 4 minutes.
This tail recursive implementation computes up to the 92nd number instantly and correctly.

@yegor256/z everybody who has role REV is banned at #177; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@yegor256/z everybody who has role REV is banned at #177; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@WiebeCnossen thanks!

@WiebeCnossen thanks!


Some fixes

Sorry for the confusion around the commits. I'm new to Git and didn't fully understand what I was doing.
I'll learn learn the basics before making further requests!
And also I'm taking this opportunity to say it: thank you for all your work! I'm glad I stumbled on your project :)
PS: Oh and btw I didn't mention I was running Linux, that's great to see everything is working there too ;)
Cheers mate!


ClassDB.class_get_method_list("class") returns a method Array of a Dictionary with only the name key

Godot version:
Godot 3.2.3 Stable
Issue description:
Calling ClassDB.class_get_method_list("Node") in a Windows Release Exe will return an Array of dictionaries that
only contain the name key. All other keys that you can see in the editor don't seem to appear here.
Steps to reproduce:
func _on_Button_pressed():
	var method_list = ClassDB.class_get_method_list("Node")
	for m in method_list:
		# Adding method list
		var l = Label.new()
		l.text = m as String
		$ScrollContainer/VBoxContainer.add_child(l)

In Editor Builds

In Release Builds

Minimal reproduction project:
DBTest.zip

Seems to be done on purpose:
https://github.com/godotengine/godot/blob/8158d17edf18e3d6ec847d2cb710012361e7b12e/core/core_bind.cpp#L2163-L2169
Should be documented probably.

Seems to be done on purpose:
https://github.com/godotengine/godot/blob/8158d17edf18e3d6ec847d2cb710012361e7b12e/core/core_bind.cpp#L2163-L2169
Should be documented probably.


Don't typedef bool for c23 (closes #240)

Seems to fix the issue.

I've solved it differently, by broadening the scope its real limit beyond the cgo interface.


Improve panics

Resolves #227
Changes:

[X] Introduce env variable in config file
[X] Initiate the logger according the env variable value so the devs can use Dpanic instead
[ ] Remove panic instructions in the code where it should not panic
[ ] Enforce rule in CI/CD that only allows panic() at bootstrap via global logger
[ ] Add goroutine catching global panic (Useful when memory leak occurs and so on...)

Types of changes
What types of changes does your code introduce?

[ ] Bugfix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[X] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[X] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Other (please describe):

Testing
Requires testing

[X] Yes
[ ] No

In case you checked yes, did you write tests??

[ ] Yes
[ ] No

I am closing this PR as it's a duplicate of #268.


Typo in command line

Set-ProcesMitigation -Name c:\apps\lob\tests\testing.exe -Enable AuditDynamicCode
should be
Set-ProcessMitigation -Name c:\apps\lob\tests\testing.exe -Enable AuditDynamicCode

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

ID: 00cee114-b948-5896-fe9e-8aef56281345
Version Independent ID: dffaa17d-04ca-aef2-0a9e-5af81cbdb3b9
Content: See how exploit protection works in a demo
Content Source: windows/security/threat-protection/windows-defender-exploit-guard/evaluate-exploit-protection.md
Product: w10
Technology: windows
GitHub Login: @levinec
Microsoft Alias: ellevin

@officedocsbot assign @jvsam

Hi @ericlaw1979 thank you so much for bringing this up. I will get this over to the Windows content writing team and request for the correction of the misspelled cmdlet.  We always strive to improve the quality of the technical documentation of Microsoft Docs so your feedback is appreciated!
Thank you for being part of the Microsoft Docs Community!

Hi again @ericlaw1979, I just want to inform you that @martyav already pushed a commit (see 31cbdee) into the doc repository to fix the misspelling. Update will be displayed here at docs.microsoft.com site in the next scheduled publishing run.
Thank you again for taking the time to share your feedback and for being part of the Microsoft Docs community!


Missing folder/files (?)

The README suggest there is an 'example' folder.

Afterwards you can start the example by executing the following in the example folder:

However there doesn't seem to be such a folder (at least not in the master branch).

Thanks!
We removed the examples from this project to create a new examples project but somehow missed to set it up. This project has been stale for a while however I've been working on a new version which is not yet available on github. (I'll push it on here including examples as soon as I think it's stable)
For now you can take a look here:
http://gamedog.chromatin.eu/chromajs/
And here is a node example:
https://gist.github.com/WolfspiritM/2748bc547c9553ad6c1698a981d7c206
If you need some help and informations feel free to join #developers on our Discord if you have questions:
https://insider.razerzone.com/index.php?threads/come-join-us-on-discord.25768/

Thanks, the example you provided works excellent!


How to set insecure registry for docker?

With docker-machine, I am able to pass the argument --engine-insecure-registry <registry> when creating the vm.

Current workaround is to set EXTRA_ARGS with --insecure-registry in /var/lib/boot2docker/profile  and restart the docker-machine.

You could always create the machine directly by calling docker-machine create <opts...> dinghy, but for ease-of-use maybe it's worth adding the ability to specify custom docker-machine create flags when calling dinghy create, something like dinghy create -p virtualbox -- --insecure-registry XX

:+1: for custom flags on dinghy create

+1 Would be useful

:+1:


chore(noir): Release 0.7.1

:robot: I have created a release beep boop
root: 0.7.1
0.7.1 (2023-06-22)
Features

brillig: foreign calls with dynamic-size objects (#1705) (fe7bb99)
update acvm to 0.15.1 (#1764) (b52f25d)

Bug Fixes

lsp: Ensure LSP can compile on Windows (#1794) (2992915)
Methods called after being passed through a generic type were not being detected (#1785) (e560cd2)
nargo: Update acvm-backend-barretenberg to allow wasm backend compilation (#1771) (97da745)
old ssa: fix to_be_bits (#1765) (2541fbd)
ssa refactor: ACIR gen NOT integer (#1749) (af749a0)
ssa refactor: allow simplified call inserts & fix const radix arg handling (#1774) (46facce)
ssa refactor: Fix flattening pass inserting loads before stores occur (#1783) (4293b15)
ssa refactor: Fix panic in acir-gen from multiplying values of different types (#1769) (1f9a132)
ssa refactor: function inlining orphans calls (#1747) (f30a90f)
ssa refactor: Ignore array out of bounds errors when enable_side_effects is false (#1797) (7b7682a)
ssa refactor: Implement merging of array values during flattening pass (#1767) (8f24751)
ssa refactor: recursion_level decrement (#1745) (e449b92)
ssa refactor: recursive branch analysis (#1759) (635b574)
ssa refactor: Speedup acir-gen (#1793) (1e75f0e)
ssa refactor: Speedup find-branch-ends (#1786) (861e42c)
Update array type when processing staged memory (#1751) (27eb748)

noir_wasm: 0.7.1
0.7.1 (2023-06-22)
Miscellaneous Chores

noir_wasm: Synchronize noir versions

This PR was generated with Release Please. See documentation.

superseded by #1807


image picker: Suggest to stop using UIAlertView but use UIAlertController with a preferredStyle of UIAlertControllerStyleAlert instead

Everything is working well. But there is just an annoying warning from Xcode every time.
/Users/ME/development/flutter/.pub-cache/hosted/pub.flutter-io.cn/image_picker-0.7.3/ios/Classes/FLTImagePickerPlugin.m:149:20:
'UIAlertView' is deprecated: first deprecated in iOS 9.0 - UIAlertView is deprecated. Use UIAlertController with a preferredStyle of UIAlertControllerStyleAlert instead

So may I suggest that can we follow the suggested message to Use UIAlertController with a preferredStyle of UIAlertControllerStyleAlert instead of UIAlertView or if there is other ways?
Thank you

flutter doctor -v
[✓] Flutter (Channel stable, 2.0.2, on macOS 11.2.2 20D80 darwin-x64, locale en-US)
    • Flutter version 2.0.2 at /Users/ME/development/flutter
    • Framework revision 8962f6dc68 (6 days ago), 2021-03-11 13:22:20 -0800
    • Engine revision 5d8bf811b3
    • Dart version 2.12.1
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Volumes/ZK/Applications/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Volumes/ZK/Applications/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.54.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.20.0

[✓] Connected device (2 available)
    • Zenko iPho (mobile) • b3ebbe93864b12b2d71234da9ac512fc4e • ios            • iOS 14.4
    • Chrome (web)        • chrome                                   • web-javascript • Google
      Chrome 89.0.4389.90

• No issues found!

Didn't find already open issue to address this, except this open PR (but it seems to use AlertViewController instead of UIAlertControllerStyleAlert), https://github.com/flutter/plugins/pull/3372

UIAlertController is available in iOS 8, so it looks like we could do this without breaking iOS 8 (which we still support), unlike many iOS 9 deprecation warnings.

Implementation of UIAlertController will be ready soon! @zenkog


Fix local variable shadow in WaitForMultipleEvents

This fixes a bug where WaitForMultipleEvents on Windows would indicate the wrong event was signaled if milliseconds was greater than UINT32_MAX. It was caused by the local variable result, which is used to set index and the return value being shadowed inside the else case for large timeouts.

Nice catch!


En and es in route

I have noted that  en and es are generating an error if used together like below
$route['^(en|es|fr)$'] = $route['default_controller'];
if i visit a url like app/es/go/here i get app/en/es/go/here

did you setup  $languages in MY_Lang.php properly? Also what you mentioned should only affect the default controller and not other controllers. Is  "go/here" you default controller?

Yes, I did setup everything right. I have seen the same problem in meteor
js routes so its definitely a regex error.
On 13 Jul 2017 15:38, "Walid"<EMAIL_ADDRESS>wrote:

did you setup $languages in MY_Lang.php properly? Also what you mentioned
should only affect the default controller and not other controllers. Is
"go/here" you default controller?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/waqleh/codeigniter-language-in-url-internationalization/issues/2#issuecomment-315065322,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANZAwSihWDe-HIvKXUqUM3oGljm_IbZUks5sNg_ggaJpZM4OWxun
.

the URL are tying to access is incorrect. The language should be before the controller.

language/controller/method

e.g.:
index.php/en/welcome/index
index.php/es/welcome/index
index.php/fr/welcome/index


Move common functions and types to backupstore lib

https://github.com/longhorn/longhorn/issues/6514

cc @derekbit


Update to use of Eigen and Address transformation problem w/ TR cards

Summary
This PR intends to remove the armadillo dependency from the project and replaces it with eigen (hopefully resolving #51) and correct lattice transformations (not only translations like in #41) originally reported by @YeChengKSU.
Other notable changes:

adds a CMake test related to the characterization of GQ surfaces using the following reference form Radford University
adds a few representative GQ tests for both axis-aligned and transformed ellipsoids and elliptic cylinders

My apologies for the relatively large PR, but it was really easiest (and most time-efficient on my side) to combine these efforts into one.
Verification
I've tested all of the files in both /tests/gqs and /tests/nested-lattice-tests with both the master branch and eigen_update using both Trelis and OCE. No changes were observed in the resulting CAD files when using Trelis, but when testing OCE I noticed a few issues:

elliptic cylinders seem malformed
the rotated elliptic cylinder failed to generate using the master branch
curve artifacts were seen when generating lattice files

Poorly formed Elliptic Cylinder

Highlighted Curve Artifact

I'd really appreciate some verification from @zxkjack123 and @YeChengKSU of what I saw when testing the files mentioned above. Someday we'll have automated tests for this maybe...
Finally, thank you to @samuelstern for the bulk of the leg work in our switch to eigen!

I'm ready to merge after updates to the README

Thanks @pshriwise - now to see how this works in Windows!!!


There was a strange computation error between standard attention and flash-attention2

import time
import torch
from torch.nn import functional as F
from flash_attn import flash_attn_func
from einops import rearrange
import math

def standard_attention(query_layer, key_layer, value_layer, attention_mask,scaling_attention_score=True):
    if scaling_attention_score:
        query_layer = query_layer / math.sqrt(query_layer.shape[-1])
    attention_scores = torch.matmul(query_layer, key_layer.transpose(-1, -2))

    attention_scores = torch.mul(attention_scores, attention_mask) + \
                           torch.finfo(query_layer.dtype).min * (1.0 - attention_mask)

    attention_probs = F.softmax(attention_scores, dim=-1)

    context_layer = torch.matmul(attention_probs, value_layer)
    return context_layer

def flash_attention(q, k, v, causal=False):
    o = flash_attn_func(q, k, v, causal=causal)
    return o

def test(func_name, q, k, v, *args, **kwargs):
    if func_name in ["standard_attention", "pytorch_func"]:
        q = rearrange(q, "a b c d -> a c b d")
        k = rearrange(k, "a b c d -> a c b d")
        v = rearrange(v, "a b c d -> a c b d")

    o = globals()[func_name](q, k, v, *args, **kwargs)
    if func_name in ["standard_attention", "pytorch_func"]:
        o = rearrange(o, "a c b d -> a b c d")
    return o

if __name__ == "__main__":
batch_size = 1
seq_len = 4096
num_head = 32
hidden_units = 128
f_attn_mask = torch.ones(batch_size, num_head, seq_len, seq_len, dtype=torch.float16, device="cuda")
f_attn_mask.tril_()
query = torch.rand((batch_size, seq_len, num_head, hidden_units), dtype=torch.float16, device="cuda")
key = torch.rand((batch_size, seq_len, num_head, hidden_units), dtype=torch.float16, device="cuda")
value =  torch.rand((batch_size, seq_len, num_head, hidden_units), dtype=torch.float16, device="cuda")
o = test("standard_attention", query, key, value, attention_mask=f_attn_mask, scaling_attention_score=True)
fa_o = test("flash_attention", query, key, value, causal=True)

I have observed a peculiar computation error between standard attention and flash-attention2. When i run the code above, the maximum error value between the two is 0.0005, but once i multiplied the query-key and value by 100, the maximum error value became 64.8125. Is this the expected numerical error?

The right thing to compare to is standard attention in fp32. In this case FlashAttention is actually more accurate than the standard implementation in fp16:
torch.manual_seed(0)
batch_size = 1
seq_len = 4096
num_head = 32
hidden_units = 128
f_attn_mask = torch.ones(batch_size, num_head, seq_len, seq_len, dtype=torch.float16, device="cuda")
f_attn_mask.tril_()
query = torch.rand((batch_size, seq_len, num_head, hidden_units), dtype=torch.float16, device="cuda") * 100
key = torch.rand((batch_size, seq_len, num_head, hidden_units), dtype=torch.float16, device="cuda") * 100
value =  torch.rand((batch_size, seq_len, num_head, hidden_units), dtype=torch.float16, device="cuda") * 100
o = test("standard_attention", query, key, value, attention_mask=f_attn_mask, scaling_attention_score=True)
fa_o = test("flash_attention", query, key, value, causal=True)
o_ref = test("standard_attention", query.float(), key.float(), value.float(), attention_mask=f_attn_mask, scaling_attention_score=True)
print((o - o_ref).abs().max())  # 65.7
print((fa_o - o_ref).abs().max())  # 32.6

The numerical error here is probably unavoidable because some computation is done fp16, which has limited numerical precision.

Thank you for your reply.
I have another question. I am working on enabling prefix attention mask support for flash-attention. I have already completed the forward propagation part, but I have some doubts about the backward propagation. What does this code mean? In what scenarios would it execute this logic?https://github.com/Dao-AILab/flash-attention/blob/main/csrc/flash_attn/src/flash_bwd_kernel.h#L647

leizhao1234

hello.  、
I had the same problem with bf16, do you have a solution?

Are you referring to the bf16 computation error issue?

Are you referring to the bf16 computation error issue?
yes，once multiplied the f_attn_mask by 100, the same computation error issue.


Command fully appear and then only half of them appear.

Describe the bug
So if you invite this bot, you will see all of the commands those are registered in console. But after a while, only half of them are appeared. It's a strange behavior hard to be reproduced.
To Reproduce
Steps to reproduce the behavior:

Invite this bot to your server
Check and re-check all registered commands in the console and the integration menu.
If all of them are complete, then wait for some hours.
Half of the command were missing

Expected behavior
All of the commands are exist.


How to transcribe audio files?

Hello,
How can I to transcribe audio files like OpenAI PHP?
Thanks

I created a PR for that:
https://github.com/theodo-group/LLPhant/pull/215

Merged


Logbook: add Engine hours to summary

List engine hours for all engines on Log book Summary page
Description
If propulsion.*.runTime property is provided, list engine hours on Summary page

Note: if there are multiple engines (e.g. catamaran) then we add name of engine to display. Needs to be tested with such boat.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Improvement/refactoring (non-breaking change that doesn't add any feature but make things better)

Thanks great idea.


image pull fails to retrieve k8s.gcr.io/coredns:1.1.3

Output of containerd --version:
containerd github.com/containerd/containerd v1.1.1-rc.0 395068d2b7256518259816ae19e45824b15da071

$ crictl --version
crictl version 1.0.0-beta.1

The k8s.gcr.io/coredns:1.1.3 image fails to pull
$ sudo crictl pull k8s.gcr.io/coredns:1.1.3
FATA[0001] pulling image failed: rpc error: code = Unknown desc = failed to pull image "k8s.gcr.io/coredns:1.1.3": failed commit on ref "manifest-sha256:a6f64e1af34ada82c3837e641fcc50f5fbfd9969c2b0f37ce5b78ba14b14884a": "manifest-sha256:a6f64e1af34ada82c3837e641fcc50f5fbfd9969c2b0f37ce5b78ba14b14884a" failed size validation: 946 != 752

The image on docker is fine
$ docker pull k8s.gcr.io/coredns:1.1.3
1.1.3: Pulling from coredns
88286f41530e: Pull complete
f7a3e79b147d: Pull complete
8d6991e77daf: Pull complete
Digest: sha256:9a239e9d3060aa5acd3d6e898fe405031594676e15e3b1b471f460eeef462583
Status: Downloaded newer image for k8s.gcr.io/coredns:1.1.3

And crictl can pull the docker version of core-dns ok too
$ sudo crictl pull coredns/coredns:1.1.3
Image is update to date for sha256:b3b94275d97cb24e34af9bb70e8582c312596eaa33716b98b46e0dffdab2f6a4

This seems to be https://github.com/containerd/containerd/pull/2364. And it will be fixed in 1.1.1-rc.1

Doesn't look like it is fixed in rc.1
ubuntu@srv-kymi3:~$ sudo crictl pull k8s.gcr.io/coredns:1.1.3
FATA[0002] pulling image failed: rpc error: code = Unknown desc = failed to pull image "k8s.gcr.io/coredns:1.1.3": failed commit on ref "manifest-sha256:a6f64e1af34ada82c3837e641fcc50f5fbfd9969c2b0f37ce5b78ba14b14884a": "manifest-sha256:a6f64e1af34ada82c3837e641fcc50f5fbfd9969c2b0f37ce5b78ba14b14884a" failed size validation: 946 != 752
ubuntu@srv-kymi3:~$ containerd -v
containerd github.com/containerd/containerd v1.1.1-rc.1 cbef57047e900aeb2bafe7a634919bec13f4a2a5

I can reproduce this. @dmcgowan @abhi Any idea on this?

If this is a containerd issue, we should fix it.
If this is a registry issue, we should at least understand it. :)

It looks like the manifest list is invalid. It lists amd64 manifest (sha256:ab8aec91742937058f6b307efbd411ba8b49c5392a757431b00fda4bdd4fa588) as having a size of 756 bytes but when that manifest is downloaded 950 bytes are returned. I think this is the expected behavior when the size of an object in a manifest differs from the actual size.

@dmcgowan Is this an image build problem or registry problem?
If it is the latter, I'll file issue to gcr.

Is this an image build problem or registry problem?

Should be a build problem, not a registry problem. The registry is returning the right content, the content is just wrong. I don't know what you are using to build the manifest list though.
You can see the problem with these curl commands
curl -v -H 'Host:k8s.gcr.io' -H 'User-Agent:Go-http-client/1.1' -H 'Accept:application/vnd.docker.distribution.manifest.list.v2+json, *' -H 'Accept-Encoding:gzip' 'https://k8s.gcr.io/v2/coredns/manifests/sha256:9a239e9d3060aa5acd3d6e898fe405031594676e15e3b1b471f460eeef462583'

curl -v -H 'Host:k8s.gcr.io' -H 'User-Agent:Go-http-client/1.1' -H 'Accept:application/vnd.docker.distribution.manifest.v2+json, *' -H 'Accept-Encoding:gzip' 'https://k8s.gcr.io/v2/coredns/manifests/sha256:ab8aec91742937058f6b307efbd411ba8b49c5392a757431b00fda4bdd4fa588'

https://gist.github.com/dmcgowan/d59d6dba116b6ff42c24794dbb57687d

Hopefully not manifest-tool! 😅

Filed issue to Kubernetes. And I think we can move the issue out of 1.1.1 milestone for now.

Hopefully not manifest-tool!

@estesp, yes CoreDNS uses manifest-tool.   Moving to docker manifest is on the to-do list, but since it's still in beta, it hasn't been a priority yet.
What are the possible issues with using manifest-tool?

@chrisohaver actually, I just realized the size mismatch/validation issue with containerd, as @dmcgowan pointed out, is with the actual entry in the manifest list for a particular architecture.
manifest-tool doesn't write those manifests; those are references to an existing (non-manifest list image) that was either created via a docker build -> docker push combination, or a custom image build tool that potentially speaks Docker v2 registry HTTP API.
manifest-tool will do a registry query ("inspect") of each architecture/platform specific image and record the size, digest, and media type in a manifest object which will then be referenced within the target repo (in case you are doing 'cross-repo' assembly of the final manifest list image). However, because manifest-tool doesn't actually pull the manifest object from the registry, it can't do size validation today. It does do digest calculation validation because after the manifest reference is pushed into your target repo, it does check--and will fail the manifest list creation--if the digests don't match between query and push.
So, I don't think manifest-tool is the size mismatch culprit here unless the registry query is actually getting an invalid result from gcr.io.  A debug run of manifest-tool would confirm the sizes being returned from the registry interaction.

Thanks... see https://github.com/kubernetes/kubernetes/issues/65253#issuecomment-398779159 ... when we inspect the manifest list with docker manifest, we get a size of 950 for the amd5 manifest in the manifest list.  So, k8s.gcr.io seems to be reporting the sizes differently depending on the method of manifest list inspection?

Hmm; interesting. Using manifest-tool shows a (correct) size of 950 for the amd64/linux entry:
$ ./manifest-tool inspect k8s.gcr.io/coredns:1.1.3
Name:   k8s.gcr.io/coredns:1.1.3 (Type: application/vnd.docker.distribution.manifest.list.v2+json)
Digest: sha256:9a239e9d3060aa5acd3d6e898fe405031594676e15e3b1b471f460eeef462583
 * Contains 10 manifest references:
1    Mfst Type: application/vnd.docker.distribution.manifest.v2+json
1       Digest: sha256:ab8aec91742937058f6b307efbd411ba8b49c5392a757431b00fda4bdd4fa588
1  Mfst Length: 950
1     Platform:
1           -      OS: linux
1           - OS Vers:
1           - OS Feat: []
1           -    Arch: amd64
1           - Variant:
1           - Feature:
1     # Layers: 3
         layer 1: digest = sha256:88286f41530e93dffd4b964e1db22ce4939fffa4a4c665dab8591fbab03d4926
         layer 2: digest = sha256:f7a3e79b147dd0a1392288e06fee0b7a23a8f0fb6ec3e2db9a4944d26b8cbe7e
         layer 3: digest = sha256:8d6991e77daf7c1f6879e84f58460b26e9d821aa27e64d8f944fbb230be845b8

Using direct curl returns the wrong size, but using docker manifest inspect gives the correct values:
jbelamaric@ANN-M-JBELAMARIC [k8s:minikube] ~$ curl -s -H 'Accept:application/vnd.docker.distribution.manifest.list.v2+json, *' 'https://k8s.gcr.io/v2/coredns/manifests/1.1.3'  | grep size
         "size": 756,
         "size": 752,
         "size": 752,
         "size": 752,
         "size": 752,
         "size": 756,
         "size": 752,
         "size": 752,
         "size": 752,
         "size": 752,
jbelamaric@ANN-M-JBELAMARIC [k8s:minikube] ~$ docker manifest inspect k8s.gcr.io/coredns:1.1.3 | grep size
         "size": 950,
         "size": 946,
         "size": 946,
         "size": 946,
         "size": 946,
         "size": 950,
         "size": 946,
         "size": 946,
         "size": 946,
         "size": 946,
jbelamaric@ANN-M-JBELAMARIC [k8s:minikube] ~$

@estesp manifest-tool was used for the list on Docker hub, but docker manifest was used for the grc.io manifest list.

@johnbelamaric yeah; so we are getting the same experience, and containerd's way of talking to gcr.io seems to be getting the curl-like response (and therefore the size mismatch). Can we get someone from gcr (Jon Johnson) to look at the backend and see if there is a gcr.io interaction problem here that generates two different responses for the same image?

Just to confirm, I tried again, pushing to a different repo, and get the same size mismatch.  I don't know the guts of docker manifest - anyone?  Can we take that apart to see what it is doing differently?

what does "size" represent?  what is it the size of?

The size of the manifest - the manifest list points to manifests which describe the images. This is essentially the Content-Length header you would get if you pulled down the manifest via http: https://gist.github.com/dmcgowan/d59d6dba116b6ff42c24794dbb57687d#file-curl-sh-L128

The 950 number is the byte count of an individual image manifest in pretty-formatted JSON.
Removing all whitespace gets to the 756 number.
I can't find anything in the spec that says what the rules are for formatting.  That explains the difference in size between tools (one is counting after formatting the other is reporting what the API is holding).  it seems pretty wrong to take JSON, reformat it, and then expect it to exactly match the source.  Either compare size directly or make the spec demand a particular formatting.

@tonistiigi just let me know he has also experienced this, apparently the docker manifest create tool is completely broken. It should NEVER change the content of the manifests. Will file an issue and get this resolved, that is pretty bad.

@dmcgowan How does docker pull work then? Does it do this kind of size validation?

I am a little wrong.
$ curl -s -H 'Accept:application/vnd.docker.distribution.manifest.list.v2+json, *' 'https://k8s.gcr.io/v2/coredns/manifests/1.1.3__amd64_linux' | wc -c
950
It looks like the API is holding formatted JSON and the manifest list is reporting the size of compacted JSON.

So I guess docker manifest is the culprit, compacting the JSON before sizing it?

I don't get it, is compacted or formatted JSON the right value for size here?
Is docker just ignoring the size validation as it works there :thinking:?

As far as I can tell, the docker manifest command is fine. The error comes from docker/distribution, package schema2:
https://github.com/docker/distribution/blob/749f6afb4572201e3c37325d0ffedb6f32be8950/manifest/schema2/manifest.go#L92-L93
And well, funny thing, it is NOT canonical, since it is blindly copied from a byte array which might store indented or non-indented JSON:
https://github.com/docker/distribution/blob/749f6afb4572201e3c37325d0ffedb6f32be8950/manifest/schema2/manifest.go#L109-L111
Here's a suggested fix to ensure it's canonical:
https://github.com/flx42/cli/commit/8d54e7c9445004bffa38ac798cf984bb243c2d73
(yes, I know this is not the right repo to submit this change, but it's easier to test).

Hi all.  Are there any viable workarounds in the mean time?
On Fri, Jun 22, 2018 at 1:18 PM Tõnis Tiigi<EMAIL_ADDRESS>wrote:

Manifest command should never unmarshal/marshal the individual manifests.
Doesn't matter if they are canonical or in whatever form, or whatever is
inside them. They need to be considered immutable bytes.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/containerd/containerd/issues/2401#issuecomment-399569935,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFVgVEvOUA6A55I4VxVoNzJ4NPhtwSg0ks5t_VD8gaJpZM4UpWzu
.

@flx42 This is getting offtopic (cli issue is tracked in https://github.com/docker/cli/issues/1135), but where does guarantee the canonical representation? I agree that variable names/comments could be improved in that code but there is only one public Payload() method with a comment "Payload returns the raw content of the manifest. The contents can be used to calculate the content identifier.". Comment for DeserializedManifest is "DeserializedManifest wraps Manifest with a copy of the original JSON.". That is exactly what the old code does.

from @thockin : re-pushed with manifest-tool and it seems good - please confirm
@NeilW , @Random-Liu : can you make a try now ?

It works now!
$ crictl version
Version:  0.1.0
RuntimeName:  containerd
RuntimeVersion:  1.0.0-75-ge3d57d2-TEST
RuntimeApiVersion:  v1alpha2
$ crictl pull k8s.gcr.io/coredns:1.1.3
Image is up to date for sha256:b3b94275d97cb24e34af9bb70e8582c312596eaa33716b98b46e0dffdab2f6a4
$ curl -v -H 'Host:k8s.gcr.io' -H 'User-Agent:Go-http-client/1.1' -H 'Accept:application/vnd.docker.distribution.manifest.list.v2+json, *' -H 'Accept-Encoding:gzip' 'https://k8s.gcr.io/v2/coredns/manifests/sha256:db2bf53126ed1c761d5a41f24a1b82a461c85f736ff6e90542e9522be4757848'
{
   "schemaVersion": 2,
   "mediaType": "application/vnd.docker.distribution.manifest.list.v2+json",
   "manifests": [
      {
         "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
         "size": 950,
         "digest": "sha256:ab8aec91742937058f6b307efbd411ba8b49c5392a757431b00fda4bdd4fa588",
         "platform": {
            "architecture": "amd64",
            "os": "linux"
         }
      },
      {
         "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
         "size": 946,
         "digest": "sha256:c8e2c9210cae0dc5ab5bbd4f96a97786494e88d3725dbe44e8ad50223731e06e",
         "platform": {
            "architecture": "arm",
            "os": "linux"
         }
      },
      {
         "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
         "size": 946,
         "digest": "sha256:68c20062c707a4b94092014db3f9f440e357110d0bca97dc8064488fc081ffd3",
         "platform": {
            "architecture": "arm64",
            "os": "linux"
         }
      },
      {
         "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
         "size": 946,
         "digest": "sha256:96eaf2c9f4979d91ded1932b8ea7c47a1fdcb30457990b63476c10b9dd7d9786",
         "platform": {
            "architecture": "ppc64le",
            "os": "linux"
         }
      },
      {
         "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
         "size": 946,
         "digest": "sha256:a6f64e1af34ada82c3837e641fcc50f5fbfd9969c2b0f37ce5b78ba14b14884a",
         "platform": {
            "architecture": "s390x",
            "os": "linux"
         }
      }
   ]
}

I am now getting what appears to be the same issue. I am on MacOS 12.3.1
I am working in a colima/lima environment, which is running this version of containerd:
containerd github.com/containerd/containerd v1.5.8 1e5ef943eb76627a6d3b6de8cd1ef6537f393a71
I am getting the following error while building a Docker image from a base node image (node:14.19.1 for building and node:14.19.1-alpine3.15 for runtime)
 => ERROR [stage-1  1/17] FROM docker.io/library/node:14.17-alpine3.14@sha256:7964eefba059e1536cca5c8da0181457b62d32f88a66fa44cb52734cd3b28c29  277.2s
 => => resolve docker.io/library/node:14.17-alpine3.14@sha256:7964eefba059e1536cca5c8da0181457b62d32f88a66fa44cb52734cd3b28c29                    0.0s
 => => sha256:1e738e76d3209f29b5a2a3284864656e933431bb77be4997155954afa813f0e8 2.36MB / 2.36MB                                                    2.9s
 => => sha256:7964eefba059e1536cca5c8da0181457b62d32f88a66fa44cb52734cd3b28c29 1.43kB / 1.43kB                                                    0.0s
 => => sha256:3f4c922d5f29da10fc2a5f1ab6a9812e7aadfc88ecc9e150ef86066f449e55ec 1.16kB / 1.16kB                                                    0.0s
 => => sha256:6c8d3cd2ef0c192a8ec333ca1043574e5e6c04f85265349dac7ec8aaed59e32e 6.53kB / 6.53kB                                                    0.0s
 => => sha256:a0d0a0d46f8b52473982a3c466318f479767577551a53ffc9074c9fa7035982e 2.81MB / 2.81MB                                                    1.7s
#### This one never matches. The download is attempted 3 times ######
 => => sha256:ac0b39f3af6d33c53223dc20d75bc9a69155cdce8bf1eaa84683507cde187863 35.65MB / 36.40MB                                                277.2s
######################
 => => extracting sha256:a0d0a0d46f8b52473982a3c466318f479767577551a53ffc9074c9fa7035982e                                                         0.3s
 => => sha256:064b7e3f032bee1f118f3ee4743c3a4fd09b0180e49e36e3f88839a1a7c5b935 281B / 281B                                                        1.9s
 => [internal] load build context                                                                                0.1s
 => => transferring context: 43.67kB                                                                                0.0s
 => CACHED [builder 2/6] WORKDIR /app                                                                                0.0s
 => CACHED [builder 3/6] COPY package*.json ./                                                                                0.0s
 => [builder 4/6] COPY . .                                                                                0.2s
 => [builder 5/6] RUN npm install                                                                                65.4s
 => [builder 6/6] RUN npm run build:prod                                                                                98.3s
------
 > [stage-1  1/17] FROM docker.io/library/node:14.17-alpine3.14@sha256:7964eefba059e1536cca5c8da0181457b62d32f88a66fa44cb52734cd3b28c29:
------
failed commit on ref "layer-sha256:ac0b39f3af6d33c53223dc20d75bc9a69155cdce8bf1eaa84683507cde187863": "layer-sha256:ac0b39f3af6d33c53223dc20d75bc9a69155cdce8bf1eaa84683507cde187863" failed size validation: 36397077 != 36398486: failed precondition

I have tried different combinations of the node and alpine versions in the FROM line, but the problem persists. Any idea how to fix this?


Create test case for Demo Health Check App

Describe the Issue
Create a test case to deploy and test the Demo Health app works as expected
Additional Information
This app covers a different functionality than the serverless and containers app, as such, it provided supplementary information about the working of the SEA
See ticket #2007
Acceptance Criteria

Get up to speed with this demo app: how it is deployed, how it work, what is supposed to do.
Write test case to deploy and test the app for new/existing account
Add the test case to the test plan
.
.
.
.
.
.
.
.
.
.
.

Extra Template Info
Definition of Ready
These set of conditions will need to be met in order to bring a ticket into the sprint and start work. Protects the team from unclear requirements.
Issues (Task/Story/Spike) aligned to an EPIC and linked appropriately.
Assigned to the appropriate CPF MEMBER (and is not left blank in ZenHub) at the start of sprint (1st day of the sprint)
Acceptance has been defined for the issue and has been reviewed with CPF team and approved by the necessary approver.
Has been sized and estimated by the delivery team.
Detailed breakdown of the steps required to complete the story in the additional context
Any additional specifications have been documented, reviewed with CPF and approved by the necessary approver.
Acceptance Criteria
A set of pre-defined requirement that need to be met in order to mark the user story as “done”.
It should be testable with no room for interpretation
It should be either “pass” or “fail”
It should be clear enough for business stake holders to understand
As part of the user story, it should be written from the user perspective
A well written acceptance criteria is great for

managing expectations
defining scope
reducing ambiguity
establishing testing criteria for QA
Given (Pre-condition)
When (Action)
Then (Outcome)

Definition of Done
These set of conditions are met for work to be considered as complete on an issue.
All acceptance criteria(s) have been validated.
All the necessary documentation for the issue has been uploaded to Teams.
Functionality has been tested when applicable.
Functionality has been demonstrated to the relevant stakeholders (where applicable).
Story has been scheduled for a Sprint Demo/community update and impacted teams invited to the Demo.
Stories accepted by PO and documented for potential sharing with impacted users.

Documented = Captured in Community Update (CU) deck
Release notes = aspirational goal of providing a bulleted list of features and changes that our users can touch. Was going to store them in our Cloud Pathfinder repo. Can also display to users in the CU

The title of the provided link to #2007 suggest a more complex artifact (app) that really is. That is not the case, the real work was to add some text so it does not require to develop any test.
In this context, this task will not be executed and can be closed.


Upgrade to Python 3

Python 2.7 will hit EoL at the beginning of next year:
https://github.com/python/devguide/pull/344
DartPad's serving setup needs to be updated to Python 3 in between now and then.

dart-pad repo still has web/main.py and build/main.py??? CC @johnpryan

web/main.py is the important one, build/main.py is copied from the web/ directory when you run the build. (along with app.yaml and other files)

This update was completed in https://github.com/dart-lang/dart-pad/commit/73a4d1440b74a44f9f9e6f193c1ee38571340b21 and Python/AppEngine usage here was removed in https://github.com/dart-lang/dart-pad/commit/7477326887f8c2149585c0bb1807ecda18964c42.


Support itk with custom namespace

Note that this commit is backward compatible and allows to build against ITK with/without
custom namespace.

See https://github.com/Slicer/Slicer/issues/6587

Please merge this. Without this the library is no longer usable in Slicer.

Moving the integration, we will address additional issues reported on the dashboard after the upcoming round of nightly builds


[Question] Document Size and Performance

I'd love to try this plugin out, but I'm slightly worried that due to the eager evaluation model (it seems to try and parse all code blocks in a page eagerly) it won't cope with large files? I don't necessarily want to use it in massive files, but I'm more worried about opening a huge file in general and having things bog down. Are there any plans to make it less eagerly evaluated?

Yeah, https://github.com/victorb/obsidian-wielder/issues/2 would alleviate this concern a bit I believe. Another way could be to not do anything with any code block until the user presses  or a "Run" button next to the code block.
Another thing you could do already is changing the Code-block Language setting to something else than clojure (like clojure-eval), and then Wielder would only look for code-blocks defined with clojure-eval and evaluate those instead, so if you have a large document with tons of clojure blocks already, this could help you.
Personally, I tend to split my documents over many pages once they get over a certain size (because it becomes easier to manage and less jumping up/down to find stuff), so I haven't hit this issue myself yet.
But if it's a issue for you, we definitely should try something that could help prevent any performance issues. You think the above solutions could work in your case?

Ooh! The code block language setting sounds perfect, but my install doesn't seem to have any settings at the moment? Is there something I have to do specially to add the settings pane?

Sorry for the delay @iamrecursion, somehow didn't get notified of your comment...
The settings panel should appear somewhere in the bottom of the general settings panel for Obsidian.
Looks like this for me:

Strange. I'm not seeing it.

I wonder if it's related to the first error in #5?


Incompatability crash With Possibly Waila or Walwla

Possible crash with the two aforementioned mods.
Not entirely sure if this is your problem as removing said mods prevents crash.
Crash occurs when you try to fell a tree, soon as all the tree bits fly bam, it crashes.
Crash log below
Pastebin

Its probably calling the NBT functions on the client side, which it isn't really setup for atm.

What is the plan for getting this fix to players?

and me
https://gist.github.com/36d3114889f5e616135fe8a549590f71

Well, it seems to be working with the previous 1.10.2 version, so I'm currently reverting back to that one.


Top-level function to take queries and return responses

We have parsing, validation, resolving, and a response object. We need to string them all together.
Notes:
the 'schema' is defined by the server types, an initial value, and any supported directives (although we'll probably skip those for first release) -- so that's the server's input
the client's input is a query document, a variable map, and an optional name of an operation to run. Not sure how these are actually delivered to us.
the output is the Response object in our Output.hs module
overall process is:

parse query (if it fails, respond with PreExecutionFailure)
transform the AST into something valid, ignoring type checks, but making sure that arguments aren't duplicated, no circular references. (if fail, PreExecutionError)
Note that we don't need the schema or the variables yet.
substitute in the variables. now everything is literal.
apply the directives. probably hard-code @skip & @include for the first release. if fail, execution error, and continue with fragments without bad directives
We now have something that's just objects & fragments
"collect the fields" to avoid duplicate processing (c.f. https://facebook.github.io/graphql/#sec-Field-Collection). still not 100% sure about this.
delegate to server-supplied code, collecting any results and errors
wrap whatever we've got off in a Response

#66 touches on this.

#59 is item 5 in the original list.


Referencing a function that has not been defined fails unhelpfully.

If I have a state using function myaction in a workflow that does NOT include the function definition, the workflow (understandably) fails but the resulting logs are not helpful.
Good YAML:
id: request
description: "Get a dad joke from the net."
functions:
- id: myaction
  image: vorteil/request
states:
- id: myrequest
  type: action
  action:
    function: myaction
    input: '{
      "method" : "GET",
      "url"    : "https://icanhazdadjoke.com/",
      "headers": {
        "Accept": "application/json"
      }
    }'

Bad YAML:
id: request
description: "Get a dad joke from the net."
states:
- id: myrequest
  type: action
  action:
    function: myaction
    input: '{
      "method" : "GET",
      "url"    : "https://icanhazdadjoke.com/",
      "headers": {
        "Accept": "application/json"
      }
    }'

Logs:
[9:33:14] Beginning workflow triggered by API.
[9:33:14] Running state logic -- myrequest:1 (action)
[9:33:14] Workflow crashed due to an internal error.

Good catch, Fixed: https://github.com/vorteil/direktiv/commit/cec6b723b3ae5a44355c6ce136b1079f6ae60366


Please add support for Groups

Hi,
Please add support for
http://docs.powerbi.apiary.io/#reference/groups
Thanks

Feature is avaible on v1.2 branch. You can use the classic library to test it : https://github.com/Vtek/PowerBI.Api.Client/tree/v1.2/dist/Classic
Beware, due to this feature, the Api url define in configuration must be changed. In the 1.1 version it was :

Datasets is now remove in the url :

1.2 alpha version avaible on NuGet (Pre version).
Please read the 1.2 wiki page : https://github.com/Vtek/PowerBI.Api.Client/wiki/Version-1.2-alpha
A++
Sylvain


pull request for e2e test at 12/16/2016 03:15:22

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


TypeScript definitions are missing

Describe the bug
The package.json of the dropbox package contains:
  "types": "types/index",
  "typings": "types/index",

But the entire types folder is missing in the code that's being deployed to npm. This might be similar to #296. But from looking at the source I don't know how these files should be generated.
Additionally, according to the TypeScript docs the value of the property types should include the file extension .d.ts → "types": "types/index.d.ts". I don't know if it's also working without the extension. But I think it's worth mentioning to avoid further potential problems.
To Reproduce
‒
Expected Behavior
The file types/index.d.ts should be created and deployed to npm.
Actual Behavior
It's missing.
Screenshots
Versions

What version of the SDK are you using? 6.0.1
What version of the language are you using? -
Are you using Javascript or Typescript? TypeScript
What platform are you using? (if applicable) -

Additional context
‒

Thanks for reporting this, I see this is a bug, it appears that the typings folder was not included in the list of files to include when uploading.  I will put out a fix by the end of the day

This should be fixed as of v6.0.2.  If this is not, please reopen this issue!


Extracting Content Automatically

To make it easy to maintain, it should be possible to extract content automatically, formatted for inclusion.

Commits made during the week.
Newly added tasks and diffs.
Tasks marked as resolved.

This script gets formats using RST for easy copy-pasting.
START="HEAD~20"
END="HEAD"

git log origin/master \
        --no-merges --first-parent \
        --stat \
        --decorate \
        --format="%n- %C(yellow):rev:\`%s <B%H>\`%C(reset)%n%n%w(120,2,2)%b" \
        $START..$END | \
        perl -e "s/\bT(\d+)\b/:task:\`\$1\`/" -p | \
        perl -e "s/\bD(\d+)\b/:diff:\`\$1\`/" -p

We can generate a list of new diffs by sorting https://developer.blender.org/api/differential.revision.search and using the title and maybe authorPHID which can be ran through https://developer.blender.org/conduit/method/user.search/ to get the real name/username a bit cumbersome but will save a lot of time if we end up using this. This can be done either through arc/php/python(https://github.com/disqus/python-phabricator) curl can also be used but there are length limits with that.


Use only seconds to keep track layer access stats instead of combining both seconds and nanoseconds

Currently we use SystemTime there, which consists of number of seconds and number of nanoseconds. As discussed with @koivunej we shouldnt need the nanosecond resolution there. So most probably seconds will be enough.
Having just seconds allows to cut memory usage a bit, squeeze the type into usize so we can use atomics instead of mutexes, etc.
Originally mentioned here:
https://github.com/neondatabase/neon/pull/3809#discussion_r1150976522

These days there is more stuff in LayerAccessStats, but this ticket still points out a sensible optimization: we should use a simple atomic for the access time, so that record() doesn't have to lock/unlock a mutex in the hot read path.


Base_value not matching

I am trying to match the base value with the force plot. I believe the base value is the weighted average of prediction in training data. The below example is for a weighted regression problem with lightgbm
`import numpy as np
import pandas as pd
import lightgbm as lgb
categorical_features =  ['c1','c2']
np.random.seed(0)
matrix = np.random.choice(['a','b','c','d'],100*2)
matrix.resize(100,2)
df = pd.DataFrame(matrix,columns = categorical_features)
df['target'] = np.where(df.c1=="a",np.random.normal(1,0.2),np.where(df.c2=="b",np.random.normal(2,0.5),np.random.normal(3,0.5)))
def as_cat(df):
for feature_name in categorical_features:
df[feature_name] = df[feature_name].astype('category')
return df
def model_train(X_train):
X_train = as_cat(X_train)
valid_features=categorical_features
parameters={
            'objective': 'regression', "num_leaves":3,
             'boosting_type': 'gbdt','leanring_rate':.1}

lgb_train_data = lgb.Dataset(X_train[valid_features], label=X_train.target,
                            categorical_feature=categorical_features,weight=X_train["weight"].values,
                                      free_raw_data=False)
print "running "

model = lgb.train(parameters,
    lgb_train_data,verbose_eval=0,
    num_boost_round=1,valid_sets=[lgb_train_data],
                         valid_names=['train'] )

return model

df["weight"]=np.random.normal(2,3,df.shape[0])
df = as_cat(df)
model = model_train(df)
explainer = shap.TreeExplainer(model)
shap_values = explainer.shap_values(df[categorical_features])
shap.force_plot(explainer.expected_value, shap_values[2,:], df[categorical_features].iloc[2,:])`
In my case
Weighted_average=2.8460
Simple_average=2.60427
base_value=2.818
Could you please help me to understand this or I am missing something? FYI, it's matching for unweighted cases

Well, after plotting the tree, I think the base value is the weighted mean of leaf node prediction and weight being the leaf_count.
So during lightgbm model building "leaf node prediction " has been done using pre defined weighing scheme. However while calculating base_value the weighing is just the count. Is this correct?

That's right. TreeExplainer currently does not consider any of the weighting that was in the training dataset. This is because the node coverage values recorded by LightGBM in the tree do not reflect any sample weighting. I am adding a to-do tag, since being able to handle weighting seems like a good feature to have. Note that right now you could redefine the explainer.model.node_sample_weight matrix yourself and then set self.model.model_type to "internal". That would allow you to run TreeExplainer on a model that you got to pick the weighting scheme for.

@slundberg  thanks for the reply. Could you please help me which code I need to modify?


Add context menu item for "convert to struct *"

Add a context menu item for "convert to struct *"

Bonus points if it suggests one that fits the uses and offsets in the current function, building on the code from #1886


fix ntp port

What does this PR do?
Fixes default ntp port which would cause below errors if taken in as an example.
      Core Check Loader:
        Could not configure check ntp: yaml: unmarshal errors:
  line 3: cannot unmarshal !!str `ntp` into int

The blurb regarding /etc/services was applicable to python-based NTP check for Agent5
https://github.com/DataDog/integrations-core/blob/5.32.x/ntp/datadog_checks/ntp/ntp.py#L29-L34
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

The system probe kitchen failures are known intermittents.


feat: delete creatable items

@yurii-korolchuk Do we need delete button only for creatable select?

@vlacher12 Yes, I believe so

@vlacher12 Yes, I believe so, but do you mean on QIT or overall ?

PUI can be used anywhere, we need to foresee any usage. I'd add delete button for default SelectInput, not only for creatable
cc @lazzystep @kaliahinartem @kkurylchenko

@vlacher12 Yes, I believe so, but do you mean on QIT or overall ?

PUI can be used anywhere, we need to foresee any usage. I'd add delete button for default SelectInput, not only for creatable cc @lazzystep @kaliahinartem @kkurylchenko

Will add, no problem, just need some time to refactor the existing solution


How to change language?

On config\nova\froala-field.php I tried:
    'options' => [
        'toolbarButtons' => [
            'bold',
            'italic',
            'underline',
            '|',
            'formatOL',
            'formatUL',
            '|',
            'insertImage',
            'insertFile',
            'insertLink',
            'insertVideo',
            '|',
            'html',
        ],
        'language' => 'pt_br',
    ],

and
    'options' => [
        'toolbarButtons' => [
            'bold',
            'italic',
            'underline',
            '|',
            'formatOL',
            'formatUL',
            '|',
            'insertImage',
            'insertFile',
            'insertLink',
            'insertVideo',
            '|',
            'html',
        ],
    ],
    'language' => 'pt_br',

, nothing happened.
Btw, is it possible to translate the Show Content and Hide Content?

It uses native Laravel localization services. You can find more information in Nova documentation

Yup, I did.
I think we have to import the translations the same way we are importing the plugins

Could you please clarify, in which way?
If you did these things it should works properly. I have tested it.

same issue. Any docs to setup language?

Same issue here. I tried inserting the translations straight into my sv.json (since i have the swedish translation which works for the rest of nove). But the editor is still not translated.
Am i missing something here? I have read Novas translation docs and got it to work with nova. I have placed translation for the froala editor in the same file, but it does still not work.


Add robot name

I think this is a useful exercise to add, as it requires mutable state for its solution. No other exercise I've seen does this.
This could be made an easier/less challenging exercise by removing the enforcing of unique names (the final test). It would still let learners find out about mutable state & randomness.
What do you think?

Exploring the specific strengths and weaknesses of a language is very interesting. So I agree that it would be interesting to see how this pans out

I rolled back a bit from requiring all unique names for the Robots. It's a little peripheral from what I think this exercise can teach (reference cells and mutable record fields), and makes the exercise significantly harder.
However I think it's worth keeping as an option if anyone wants a bit more of a challenge, so I left a test in which is by default not enabled (I really wish ounit had the idea of a pending or ignored test!)

I'll remove the dependency on the re2 regex library. It's just being used in the tests. Will be a shame to force this dependency on the student, as they will almost certainly not want it for the implementation.

Rolling back uniqueness is probably better, because it could burden people a lot.

Removed the re2 dependency, so I think is good to go.