MTU for openstack tap interfaces not set

Expected Behavior
MTU of the tap interfaces should reflect openstack related options to enable instances to use the specified MTUs.
Current Behavior
MTU of the tap interface is the default 1500
Possible Solution
The ml2 plugin should consume the related MTU option values, and make them available for Felix to consume and set on the tap interfaces when created.
Steps to Reproduce (for bugs)
Set the following:
neutron.conf: global_physnet_mtu = 9000
ml2 conf: path_mtu = 9000

Restart neutron-server, and create an instance.
Context
Instance interface MTUs that reflect the underlay can be configured, and all the correct openstack config is in place, but traffic is fragmented when it passes the tap interface.  Changing the relevant tap interface MTU manually to the required value prevents further packet fragmentation.
Default MTU:
# ip l show tapb1d13698-13
37: tapb1d13698-13: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UP mode DEFAULT group default qlen 1000
    link/ether 00:61:fe:ed:ca:fe brd ff:ff:ff:ff:ff:ff

# tcpdump -i tapb1d13698-13 icmp -n
tcpdump: verbose output suppressed, use -v or -vv for full protocol decode
listening on tapb1d13698-13, link-type EN10MB (Ethernet), capture size 262144 bytes
18:39:22.615684 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 1418, seq 1, length 8908
18:39:22.616299 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo reply, id 1418, seq 1, length 1480
18:39:22.616307 IP <IP_ADDRESS> > <IP_ADDRESS>: ip-proto-1
18:39:22.616310 IP <IP_ADDRESS> > <IP_ADDRESS>: ip-proto-1
18:39:22.616312 IP <IP_ADDRESS> > <IP_ADDRESS>: ip-proto-1
18:39:22.616314 IP <IP_ADDRESS> > <IP_ADDRESS>: ip-proto-1
18:39:22.616317 IP <IP_ADDRESS> > <IP_ADDRESS>: ip-proto-1
18:39:22.616319 IP <IP_ADDRESS> > <IP_ADDRESS>: ip-proto-1
18:39:23.617092 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 1418, seq 2, length 8908
18:39:23.617763 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo reply, id 1418, seq 2, length 1480
18:39:23.617781 IP <IP_ADDRESS> > <IP_ADDRESS>: ip-proto-1
18:39:23.617784 IP <IP_ADDRESS> > <IP_ADDRESS>: ip-proto-1
18:39:23.617786 IP <IP_ADDRESS> > <IP_ADDRESS>: ip-proto-1
18:39:23.617788 IP <IP_ADDRESS> > <IP_ADDRESS>: ip-proto-1
18:39:23.617790 IP <IP_ADDRESS> > <IP_ADDRESS>: ip-proto-1
18:39:23.617792 IP <IP_ADDRESS> > <IP_ADDRESS>: ip-proto-1

Increaed MTU:
# ip link set tapb1d13698-13 mtu 9000

# tcpdump -i tapb1d13698-13 icmp -n
tcpdump: verbose output suppressed, use -v or -vv for full protocol decode
listening on tapb1d13698-13, link-type EN10MB (Ethernet), capture size 262144 bytes
18:40:44.845442 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 1423, seq 1, length 8908
18:40:44.846135 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo reply, id 1423, seq 1, length 8908
18:40:45.847037 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 1423, seq 2, length 8908
18:40:45.847782 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo reply, id 1423, seq 2, length 8908

Your Environment

Calico version - 3.2.1
Orchestrator version - Openstack Mitaka
Operating System and version: Ubuntu Xenial

@neiljerram presumably this is a relatively straightforward tweak in networking-calico?

@caseydavenport @jocado No, I think it might need a Nova patch, and that more debugging is needed to understand why this is not already working.
Nova code (as of 17.0.5, which happens to be the tag I currently have checked out) has this, which looks good to me:
    def plug_tap(self, instance, vif):
        """Plug a VIF_TYPE_TAP virtual interface."""
        dev = self.get_vif_devname(vif)
        mac = vif['details'].get(network_model.VIF_DETAILS_TAP_MAC_ADDRESS)
        linux_net.create_tap_dev(dev, mac)
        network = vif.get('network')
        mtu = network.get_meta('mtu') if network else None
        linux_net._set_device_mtu(dev, mtu)

That code was exactly the same as of 13.0.0, which is the Mitaka release version.


fix macos docker-compose test

[x] fix macos docker-compose test

Comments

Issue Number

bors r+


JS snippet: anonymous function variable assignment.

Closes #524
@lpil anything else?

My bad, totally opened this in the wrong place =)


New-EnterpriseApplication.ps1 calls cmdlet without proper scopes

Expected Behavior
Please describe the behavior you are expecting.
New-EnterpriseApplication.ps1 is able to run the cmdlet Get-MgOrganization without any issue
Current Behavior
What is the current behavior?
New-EnterpriseApplication.ps1 connects to MS Graph with the scopes Application.ReadWrite.All,AppRoleAssignment.ReadWrite.All, and then later in the script calls the cmdlet Get-MgOrganization, which will give an error if the proper permissions aren't configured.
The error given is "Insufficient privileges to complete the operation"

Opened this issue in the wrong repository


feat: add langchain support

Add langchain backend (#36)

Note: Currently it's just for openai, I think for general llm need to make the backend base more dynamic or conform to some settings by langchain framework. We can follow this example which utilizes from langchain.schema and langchain.output_parsers but not sure if it's desired since we might have less freedom to define some prompts etc but in terms of programming it's less work since we will have some out-of-box functions.

either making a wrapper for them to work as a backend or making a custom agent class that can be extended.

Either way is fine with me, let me know which way you guys decide to go and I can implement accordingly. cc @yuxiang-wu

Thank you for your contribution @dexhunter ! I think putting it as a backend is good for now. We can work on a more complex agent later. Thank you @elliottower for the discussion too!


IP in screen shot is not the same as ssh public IP

In the browser screen shot, IP is <IP_ADDRESS>, which is different then the IP used for ssh (<IP_ADDRESS>) - this can be misleading.

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

ID: 814b92ed-f712-53c8-644b-d3be513a4d06
Version Independent ID: 79d8998f-1b47-40ec-e975-8eeda383b0cf
Content: Quickstart - Create a Linux VM in the Azure portal
Content Source: articles/virtual-machines/linux/quick-create-portal.md
Service: virtual-machines-linux
GitHub Login: @cynthn
Microsoft Alias: cynthn

Thanks for the feedback! We are currently investigating and will update you shortly.

@MachLearnPort thanks for pointing this out. The reason for this is that we use the same screenshot for multiple docs at times to make things simpler and easier to update. However I can see how that can cause confusion.
At this point, this is the first time we have heard feedback on this. So at this point we have decided to keep the doc as is. However, your comment will remain on this document so if others feel the same they can comment and let us know and we can always revisit this in the future.
CC @cynthn, this issue does bring up a valid point. Possibly going forward we should reconsider when to reuse some screenshots. Just sharing for future awareness :)

@MachLearnPort Oops, thank you for catching that one! The one for Instance details was also old and used Ubuntu 16.04 instead of 18.04. I've fixed them and they should go live later today :)

@MicahMcKittrick-MSFT I'm going to blame my summer cold for this one :D  I updated this topic a couple days ago, along with a couple screenshots, but missed these two. We really should share art as much as we do, so I'll try to fix that as I touch files!


Ubuntu 16.04 LTS is approaching its "End of Standard Support" in April 2021

Just wanted to share that Ubuntu 16.04 LTS is approaching its "End of Standard Support" in April 2021 - that's not too far away.
Not sure if that just means an update to the documentation where Ubuntu 16 in mentioned multiple times or if this has more serious implications to change over to Ubuntu 18 LTS or directly to Ubuntu 20 LTS.
While at it, it would be lovely to see a simple containerised environment for production :-)

Good observation!
You can see that there has been an official Dockerfile added already which affirms support for Ruby 2.6 which means 18.04 is covered (and there has been an install script added recently for it) however others have reported problems with some features in Ruby 2.7 so 20.04 is not really an option IIRC.
I've been making use of the Dockerfile in local testing which has been a massive time saver compared to maintaining a real server for it but some more documentation on it would be nice.

Good observation!
You can see that there has been an official Dockerfile added already which affirms support for Ruby 2.6 which means 18.04 is covered (and there has been an install script added recently for it) however others have reported problems with some features in Ruby 2.7 so 20.04 is not really an option IIRC.
I've been making use of the Dockerfile in local testing which has been a massive time saver compared to maintaining a real server for it but some more documentation on it would be nice.

Great to see the Dockerfile. Now we only need a tiny bit of documentation ;-)
According to docs Postal needs MySQL/MariaDB, RabbitMQ, NodeJS and Nginx.
This could all be included in a Docker Compose file or a simple shell script.

Great to see the Dockerfile. Now we only need a tiny bit of documentation ;-)
According to docs Postal needs MySQL/MariaDB, RabbitMQ, NodeJS and Nginx.
This could all be included in a Docker Compose file or a simple shell script.

Quite, I've done some messing around with a docker-compose file over here https://github.com/SynergiTech/postal/blob/docker-experimentation/docker-compose.yml
This is mostly being discussed in #974

Quite, I've done some messing around with a docker-compose file over here https://github.com/SynergiTech/postal/blob/docker-experimentation/docker-compose.yml
This is mostly being discussed in #974

Hi,
I have Postal running on Ubuntu 18.04 Bionic (LTS), there is few more step to resolve some issue.
Please refer to #777 for more details.
Thank you


Added an interactive trivia feature

The command I added makes an API request to "Open Trivia Database" and fetches a question. I use the response to display the questions and options (in a random order). I attached a screenshot below describing how the command works.

I made this PR in response to #1, and I had a fun time adding this command. I hope you find it useful!

Looks good, thanks!


alt+enter doesn't enter fullscreen

Pressing alt+enter won't change a window to fullscreen. If I stop streaming to my Steam Deck, the shortcut will start working again.

Hello, this issue isn’t related to my AIO tool. It is probably related to Sunshine or Moonlight directly. Check with them so


Support for a CLI?

I think a CLI would be pretty useful. I've got a few different use cases where I'd like to do something like:
getcss http://foo.bar > foobar.css

Do you think this would be worthwhile to incorporate into this project? I'd be happy to submit a PR with the functionality above if you think it's worth it.

Seems reasonable to me :+1:


PHPExcelHelper could not be found.

Hi guys,
I am trying to use this plugin, but the problem " PHPExcelHelper could not be found. "
In AppController
$this->loadComponent('RequestHandler',['viewClassMap' => ['xlsx' => 'Cewi/Excel.Excel']]);
In my Controller
public $helpers = ['Cewi/Excel.Excel'];
in my view:
$this->Excel->addWorksheet($matches, 'controllersStatus');
phpexcel is in:
vendor/phpoffice/phpexcel
i think this can be solved with
require_once(ROOT. DS .'vendor'.DS.'phpoffice/phpexcel/Classes');
but nothing happen i dont where to put the require_once sentence! some help will be amazing.
thanks

In cakephp, you use autoloading. No require_once needed.
Did you load the plugin?
You have to add
Plugin::load('Cewi/Excel', ['bootstrap' => true, 'routes'=>true]);
in your bootstrap.php file.

Yes this is my last line in bootstrap.php
Plugin::load('Cewi/Excel', ['bootstrap' => true, 'routes'=>true]);
I dont have problem when i use $this->loadComponent('Cewi/Excel.Import');
the problem is with public $helpers = ['Cewi/Excel.Excel']; when the helper call PHPExcel Helper.
No so much information i had check all the link in a google search. :(

Hi Cewi,
I am  trying something today. i followed this steps as you described
router.php
Router::extensions(['xlsx']);
ArticlesController:
public $helpers = ['Cewi/Excel.Excel'];
add a Folder 'xlsx' in Template/Articles and create the file 'index.ctp' in this Folder.
$this->Excel->addWorksheet($articles, 'Articles');
create the link to generate the file somewhere in your app:
Html->link(__('Excel'), ['controller' => 'Articles', 'action' => 'index', '_ext'=>'xlsx']); ?>
'_ext'  is working as index.xlsx
and of course i am getting the error
Missing Method in ControllersStatusController   public function index.xlsx()
Some advice please

Can be something about this:
Deprecated since version 3.1.0: As of 3.1.0 the viewClassMap() method is deprecated. You should use config() to change the viewClassMap at runtime.
???

Got it guys,
in router.php the line
Router::extensions(['xlsx']);
must be before the method Router::scope('/', function (RouteBuilder $routes) {}
Something like this:
router.php:
use Cake\Core\Plugin;
use Cake\Routing\RouteBuilder;
use Cake\Routing\Router;
use Cake\Routing\Route\DashedRoute;
Router::extensions(['xlsx']);
Router::defaultRouteClass(DashedRoute::class);
Router::scope('/', function (RouteBuilder $routes)
{
..
..
}

that's true. If you want to use it inside a scope, you've to write $routes->extensions(['xlsx']); Will clear that in the doku.
Have fun!


Dual Screen not working correctly

I recently updated the version of FreeRDP that we are using.  In the process I found that Dual screen operation is broken.  The 2nd monitor does not show what should be on the desktop.  I can drag an Icon or window on to the other screen but it is not visible.  I found that the check in 719a0fd3e9877cc9f0c2031d0e68fc5680834c9c  (Thu Mar 12 07:27:29 2015 -0400) is where the break happens.

@davidleake did you test with the latest master already? There were recent changes regarding multimon.

@davidleake in case it fails with current master please also let us know which window manager you're using, to which rdp server you're connection (the remote Windows version/edition) and your exact xfreerdp command line.

I have tried it with the current master.  We are using a derivative of the icewm as our WM.  I have connected to Server 2008R2, Windows 8, and Server 2012 all with the same results.
command line is
xfreerdp /t:rdp-win8fx /v:win8fx -grab-keyboard /gdi:hw /sound:sys:alsa /microphone:sys:alsa /cert-ignore -sec-nla -toggle-fullscreen /drive:TCshare-sdb1,/media/sdb1 /bpp:24 /network:lan +aero +window-drag +menu-anims +compression /multimon /monitors:0,1 /f
I have also tried just to
xfreerdp /t:rdp-win8fx /v:win8fx  /multimon /monitors:0,1 /f

@davidleake I think this is a dulicate of issue #2707. Icewm does not support the NET_WM_FULLSCREEN_MONITORS property.
For a detailed explanation see https://github.com/FreeRDP/FreeRDP/issues/2707#issuecomment-153721965)


Disabling drag when mouse is over a div that sits above the zoompan element

Hi Timmy (duplicate with example)
https://jsfiddle.net/wxgvqkan/6/
on the example i cant scroll the popup window using the scroll bars (i can with the mouse wheel)
you recently described how to disable zoom with mouse when the pointer is over a different window that sits in front of the panzoom element. I used this and it works when i am wanting to scroll the target window and not the panzoom element.
i am having an issue when dragging the scroll bar of the target window, it does scroll the window but it also pans the map behind it. Can you help?
many thanks.

Thanks for opening an issue. Is this the behavior you're looking for?
https://jsfiddle.net/timmywil/5n10Lx2q/1/

Thanks for opening an issue. Is this the behavior you're looking for?
https://jsfiddle.net/timmywil/5n10Lx2q/1/

Hi Timmy, Unfortunately i need the the popup div to be part of the panzoom element so it pans and zooms with the rest of the image.
i just dont want it to pan when the mouse drag is on the popup scroll bar.

Hi Timmy, Unfortunately i need the the popup div to be part of the panzoom element so it pans and zooms with the rest of the image.
i just dont want it to pan when the mouse drag is on the popup scroll bar.


Program should ignore workflow files

Allowing the program to copy workflow files in .github/workflow could create an infinite loop of running GitHub Actions that invoke other actions.
More generally, .github is for repository-specific content that is largely independent of the contents of the repository. It never makes sense to sync this directory.

Fixed in v0.3


ell, iwd: update

Motivation for this change

ell: 0.35 -> 0.36
iwd: 1.10 -> 1.11

ver 0.36:
Fix issue with strict netlink attribute validation.
Fix issue with DUID generation for DHCPv6.
Add support for Address Conflict Detection.
Add support for PKCS#12 algorithms handling.
Add support for PKCS#12 key derivation algorithm.
Add support for RC2 and ARC4 cipher suites.

https://git.kernel.org/pub/scm/libs/ell/ell.git/tree/ChangeLog?h=0.36

ver 1.11:
Add support for ACD client for static configuration.
Add support for intelligent scan of all frequencies.

https://git.kernel.org/pub/scm/network/wireless/iwd.git/tree/ChangeLog?h=1.11
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 110275 run on x86_64-linux 1

4 packages built:

ell
iwd
modem-manager-gui
ofono

Result of nixpkgs-review pr 110275 run on x86_64-linux 1

4 packages built:

ell
iwd
modem-manager-gui
ofono

/marvin opt-in
/status needs_reviewer

/marvin opt-in
/status needs_reviewer

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110275 run on x86_64-linux 1

4 packages built:

ell
iwd
modem-manager-gui
ofono

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110275 run on x86_64-linux 1

4 packages built:

ell
iwd
modem-manager-gui
ofono

Thanks!

Thanks!


radius of ROUNDED_RECTANGLE shape

Hello
I am creating a rounded rectangle by following the code. Is there any way to change the radius of the shape?
from pptx import Presentation
from pptx.util import Inches
from pptx.enum.shapes import MSO_SHAPE

pr = Presentation(os.path.join(os.getcwd(),pptx_figure))
slide_template = pr.slide_layouts[3]
slide = pr.slides.add_slide(slide_template)

left = Inches((13.33-(0.33*2))/2+0.33)
top = Inches(3.84)
width = Inches((13.33-(0.33*2))/2)
shape = slide.shapes.add_shape(
    MSO_SHAPE.ROUNDED_RECTANGLE,
    left=left,
    top=top,
    width=width,
    height=width*(10/24)
)

pr.save(pptx_figure)

In md2pptx I have the following code:
            if cardShape == "rounded":
                # Rounded Rectangle for card
                cardBackgroundShape = slide.shapes.add_shape(
                    MSO_SHAPE.ROUNDED_RECTANGLE,
                    Inches(0),
                    Inches(0),
                    Inches(0),
                    Inches(0),
                )

                # Rounding adjustment works better with different values for horizontal and vertical cards
                if cardLayout == "horizontal":
                    # Make the rounding radius small. This is 1/4 the default
                    cardBackgroundShape.adjustments[0] = 0.0416675
                else:
                    # Make the rounding radius smallish. This is 1/2 the default
                    cardBackgroundShape.adjustments[0] = 0.083335

which does indeed get me a rounded rectangle with a specific radius. The operative line is
                    cardBackgroundShape.adjustments[0] = 0.0416675

I would play with the actual value - to get one you want.
(Note: My purpose isn't to "sell" you md2pptx; Rather to share the relevant piece of code.)

In md2pptx I have the following code:
            if cardShape == "rounded":
                # Rounded Rectangle for card
                cardBackgroundShape = slide.shapes.add_shape(
                    MSO_SHAPE.ROUNDED_RECTANGLE,
                    Inches(0),
                    Inches(0),
                    Inches(0),
                    Inches(0),
                )

                # Rounding adjustment works better with different values for horizontal and vertical cards
                if cardLayout == "horizontal":
                    # Make the rounding radius small. This is 1/4 the default
                    cardBackgroundShape.adjustments[0] = 0.0416675
                else:
                    # Make the rounding radius smallish. This is 1/2 the default
                    cardBackgroundShape.adjustments[0] = 0.083335

which does indeed get me a rounded rectangle with a specific radius. The operative line is
                    cardBackgroundShape.adjustments[0] = 0.0416675

I would play with the actual value - to get one you want.
(Note: My purpose isn't to "sell" you md2pptx; Rather to share the relevant piece of code.)

works exactly like I wanted. :)


Introduce global default settings

Fixes #20

Codecov Report

Merging #31 into master will increase coverage by 0.55%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #31      +/-   ##
==========================================
+ Coverage   95.67%   96.23%   +0.55%
==========================================
  Files          18       18
  Lines         370      398      +28
  Branches       14       12       -2
==========================================
+ Hits          354      383      +29
+ Misses         16       15       -1

Impacted Files
Coverage Δ

src/main/scala/AppConfig.scala
100.00% <100.00%> (+2.56%)
:arrow_up:

Continue to review full report at Codecov.

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


Function return values get erased when entering a new function, essentially locking up the program

example :
`func exit int -- void in
60 syscall1
end
func bar void -- int in
69
end
func foo void -- int in
bar
end
func main void -- void in
foo exit
end
`
in this example, main calls foo, which places it's return address into r15, then calls bar, which replaces foo's return address with it's own, then return, then foo tried to exit, but r15 is empty, so it cannot, and locks up.

well this has been fixed since


[Bug]: getCurrentNode() not working

What happened?
on the photo-sphere-viewer docs, under the virtualTour plugin, there's a method called getCurrentNode which is supposed to return the current node. I used this in react-photo-sphere-viewer after getting an instance of the virtualTour plugin and got an error. I have attached an Image containing the error

What should have happened?
return current node
Code
// create an instance of the plugins
const handleReady = (instance) => {
// marker plugin intance
const markersPluginInstance = instance.getPlugin(MarkersPlugin);
// virtualTour plugin instance
const virtualTourPluginInstance = instance.getPlugin(VirtualTourPlugin);

virtualTourPluginInstance.setNodes(vtplg)
const recNode = virtualTourPluginInstance.getCurrentNode()
console.log(recNode);

if (markerPlug) {
  markerPlug.addEventListener("select-marker", () => {
    console.log("marker clicked");
  });
}

};
Sandbox Link
No response
Library Version
"^3.4.0-psv5.4.0"
What operating system are you using?
macOS
What browser are you using?
Chrome
Logs
No response
Interest to fix the bug

[ ] I would like to fix this bug!

Hello! thanks I'll try to figure it out.
I will let you know :)

thank you. i have already used the "node-changed" event listener to get my nodeId which is what i was after but it'll still be great to have the method fixed


Add Better Homebrew Browser

https://github.com/hackmyvita/hackmyvita.github.io/pull/1
So that it works, you have to accept the pull request from there, otherwise, it won't work.
Also If there's something wrong just tell me and I can fix it.

Oh, and the vpk's I can send it.
Vitadeploy_v.2.0.zip

will add in the next update

will add in the next update

Okay 👍

wontadd, added vitadb downloader, you can download bhbb via it


Fix javadoc for verifyNew

From<EMAIL_ADDRESS>on November 02, 2009 10:54:55
Javadoc refers to verifyStatic
Original issue: http://code.google.com/p/powermock/issues/detail?id=187

From<EMAIL_ADDRESS>on December 09, 2009 09:27:06
Fixed
Status: Fixed


some suggestions

1，Is it possible to extend the if/else judgment?
2，Is it possible to add data index when data-repeat?
besause without these functions, most time i need to do more work with javascript.

@vopjie This is probably best served over at https://github.com/mustache/spec as it contains the spec of what mustache.js implements (along with other implementations in different languages). I will warn you that the spec hasn't been updated in many eons unfortunately.
If you want something less opinionated about logic in templates then you may be interested in http://handlebarsjs.com/ which is quite compatible with the mustache spec but has the features you requested.
Hope this helps

ok,thansk


Use monolith for local run

Description of your changes
While testing a new resource in provider-aws, after running make run in my local, the ready and synced status of the resource remained empty. I didn't encounter any error message in the description and when I used monolith package instead of provider in local run, the problem was solved.

I have:

[ ] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Hi @turkenf,
Could you please fix the reported linter issue? It's been introduced in an earlier PR but because we had issues with the linter, we could not probably spot it.


Multiple cucumber_output (json) into Single Report with different Metadata

Hi @wswebcreation
Now I have 2 set of test executions

puppeteer chrome
puppeteer firefox
both produce 2 different JSON output.

Able to produce a single report by pointing outputDir.
But I want to feed different metadata for each output.
What I see we have only metadata which applied for all output files
I made a small change in collect-json.js to adopt different metadata for each output file
metadata sample
metadata: {
    "cucumber_report_chrome": {
      browser: {
        name: 'chrome',
        version: '68.0.3419.0'
      },
      device: 'Local Test Machine',
      platform: {
        name: 'linux',
        version: 'x86_64'
      },
    },
    "cucumber_report_firefox": {
      browser: {
        name: 'firefox',
        version: '65.0'
      },
      device: 'Local Test Machine',
      platform: {
        name: 'linux',
        version: 'x86_64'
      },
    }
}

cucumber_report_chrome
cucumber_report_firefox
these are output filenames (without json) which produced by cucumberjs execution.
How do you see this? should raise the PR for the changes.
I made it as additional if block to accept if there is multiple metadata otherwise it will go as global one which applied all the files.
( I don't find an easy way to add metadata inside cucumberjs report )

Hi @muthukumarse,
I think there is some misunderstanding on the metadata part. Please check this part in the docs
https://github.com/wswebcreation/multiple-cucumber-html-reporter#metadata
and especially this remark

If you provide the metadata when instantiating multi-cucumber-html-reporter the metadata will be added to each feature. If you already have metadata in your JSON then the metadata in the JSON will take precedence

This means that you need to provide it yourself per instance, so per generated JSON file in your framework, or you provide a capability for generating the report, but then it will be used for all
I'm going to close it for now, but it's still open for discussion

Hi @wswebcreation
I understood and was using the same way you pointed out.
now I have a situation that, I am producing more than one output (JSON output) and when I am producing HTML report I want each file should have own metadata instead of same.
in my case 1 output which ran in puppeteer chrome, another one puppeteer-firefox. once I generate the HTML output, I want different metadata info for each of those.

That can be reached by having it in per json file, the module will print it to a nice report, just check the example files here. There you will see that all these files have their own metadata and it will be printed to this report which you can find here

@wswebcreation
this is what I am trying to get it done, is there way to push the metadata inside report json through cucumber-js command

I don't know the tool you are using, but you should be able to do that in a post-process when the json file has been stored, open it and add the data

@wswebcreation Yes can be achieved by post-process step. But I see that collect-json.js does the same in "multiple-cucumber-html", I thought we should extend the feature instead of doing one more pre-process as post-process before generating the report.

The mode will never have the understanding which JSON file was from which browser instance. This means that the framework should add this data during or after runtime.
I also want to keep it that way because the core is being used my multiple different frameworks and languages that use cucumber and I want to keep this module as simple as possible

@wswebcreation well I used post-process to append metadata based my project before generating the report.

@wswebcreation I am creating separate json per run and in the report whatever the last browser was it will cascade that to all the previous runs and thats despite the fact that i am generating separate jsons (filename: example-test-suite chrome Automated Test Report-1570468288493.json)

Hi @larryg01
Can you explain it a bit more in detail what is happening? The feature you are referring to is working as expected

ok, I run the test twice on different browsers so i have
example-test-suite chrome Automated Test Report-1570521833027.json and
example-test-suite firefox Automated Test Report-1570521851043.json
but on my report its records the last browser that was passed for both runs

like its just over writing the values

@larryg01
Can you provide the 2 json files so I can check it here?
Can you also provide the version you are using ?

I am using v1.11.6
I just change the extension from .json to .txt so that I can send them here
example-test-suite chrome Automated Test Report-1570547715909.txt
example-test-suite firefox Automated Test Report-1570547743237.txt

@wswebcreation
Do you have an update on this issue please

@larryg01
Sorry for the delay, I've checked your files but they don't have the metadata in it. I've checked them in the report itself and they look like this

The metadata that can be provided like here https://github.com/wswebcreation/multiple-cucumber-html-reporter#cucumber-js-2x-and-lower
The downside if this is that it will use that metadata for ALL files, even though the are not created by the browser. Like I explained in this comment https://github.com/wswebcreation/multiple-cucumber-html-reporter/issues/82#issuecomment-473173160, the module will never have an understanding which browser generates which file because this report is a post process, when all workers are done. You need to add the metadata in a post process per worker in your own framework

I am very confused because as you are seeing something different to what i am seeing, you are seeing 'not known' where as i am seeing the actual metadata value (previous image above).
Can you point me in the right direction for adding that metadata in a post process per worker, it would be greatly appreciated

In my setup I'm not adding the metadata when the report is being build, I assume you have it set in your configuration.
About adding itinerary a postprocess, it all depends on your framework. I don't know your framework so I can't tell you how to do it.

https://github.com/larryg01/klassi-js
runtime/world.js bout ln 337 (its commented out, but you will understand it)
Thats how and where I am adding the metadata, so it means i will have to remove it from there and add it as a postproecss

This is always overwriting the data with the latest. What you need to do is to get hold of the process in your framework that creates the JSON file, then read it and add the metadata data to it.
I can't do more for you in this case, I'm busy enough with maintaining my own modules that I don't have enough time to maintain/fix others, sorry

Shouldn't it be Appending the latest data as opposed to Overwriting it so as to write the next line in the report.
As it stands it over writes the data from all previous runs with the last run and thats only for the OS Device and Browser columns as it records the correct data for the rest ie feature, tag, status etc.

When you call the method to create the report it will scan all files and add the metadata to all files. That's how it's build now

@larryg01

if thats the case my friend how then would you get the metadata to be appended to the file so as to preserve the previous run.

As said, this needs to be done by your framework, not by this module. The purpose of this module is to collect data from a folder, enrich it if needed with predefined data and then create the report

Hi am using the "saveCollectedJson: true" option and getting this error (see below) when I run the test a second time, the enriched and merged files are being created but its not generating the report
" UnhandledPromiseRejectionWarning: TypeError: JSON.parse(...).map is not a function
at files.map.file (/Users/lgoddard/Projects/klassi-js/node_modules/multiple-cucumber-html-reporter/lib/collect-jsons.js:23:30) "
can you assist please?
NOTE: on first run the report is produced, its on the second run thats its failing with that error


[zero3] Registering External Parameters doc clarification

In the doc Registering External Parameters

at the very end it gives an example:

class ModuleZ3(torch.nn.Module):
    def __init__(self, *args):
        super().__init__(self, *args)
        self.layer1 = SomeLayer()
        self.layer2 = OtherLayer()
        deepspeed.zero.register_external_parameter(self, self.layer1.weight)

    def forward(self, input):
        x = self.layer1(input)
        # self.layer1.weight is required by self.layer2.forward
        y = self.layer2(x, self.layer1.weight)
        return y

If I understand the situation correctly that last comment:

self.layer1.weight is required by self.layer2.forward

doesn't quite fit to to what actually happens.
Isn't the point here that we need to have the weights synced in the current forward and if so shouldn't it say:

self.layer1.weight is required by ModuleZ3.forward

because this is where it's really used. The fact that it's then passed to layer2 is irrelevant because by that time it is already synced and fully present. layer2.forward is oblivious to the arg being actually a weight elsewhere.
I mean that's why ModuleZ3.__init__ calls register_external_parameter and not OtherLayer.__init__
The way I interpret the current comment is that register_external_parameter should actually happen inside OtherLayer.__init__ which I don't think is right.

And then going back to the beginning. The section starts with a nice practical example and then it drops off and shows a different example. All the suspense was lost ;)

But I find the prose after it hard to understand - it goes:

The tensor embeddings.weight is used in both embeddings.forward() and compute_logits(). We call embeddings.weight an external parameter because it is used in the training loop outside of its owning module’s forward pass.

Am I still keeping the intention correct if I rephrase it as:

The tensor embeddings.weight belongs to embeddings and is used in embeddings.forward() and LanguageModel.forward(). We call embeddings.weight an external parameter in this situation because it's used by LanguageModel.forward() to which it doesn't belong,
[all the rest of the doc]
[and then before the examples start let's finish the first example]
and therefore we need to adjust our initial example to:

class LanguageModel(torch.nn.Module):
    ...
    def __init__(self, ...):
        [....]
        self.embeddings = nn.Embedding(...)
        deepspeed.zero.register_external_parameter(self, self.embeddings.weight)

    def forward(self, inputs):
        embeds = self.embeddings(inputs)
        ...
        logits = compute_logits(output, self.embeddings.weight)
        ...

Here again compute_logits() makes things slightly more difficult to understand because it potentially may mean that this function could live on another layer or something like that, but again it's irrelevant since it doesn't know where its argument came from. i.e. it's only relevant inside the LanguageModel.forward. compute_logits() though is definitely less confusing than self.layer2() call from the other example,  because the latter does refer to yet another sub-module.

The mantra I think is: if layer A's forward refers directly to layer B's weights, layer A needs to register layer's B's weights in A.__init__.
I, of course, could be wrong - just trying to match the prose to the code and vice versa.
I'd be happy to send a PR if either or both of my suggestions may help improve clarity around this topic.
Thank you!

Actually, reading the section one before it's quite confusing with regards to it again referring to external parameters. Is it the case where the last section shows how to do the same during __init__ whereas the first section does the same at the point of use in forward? i.e. Are these identical functionality-wise:
original example
def forward(self, input):
    x = self.layer1(input)

    # self.layer1.weight is required by self.layer2.forward
    with deepspeed.zero.GatheredParameters(self.layer1.weight,
                                           fwd_module=self):
        y = self.layer2(x, self.layer1.weight)
    return y

modified using register_external_parameter instead:
def __init__(self, ...):
    [...]
    deepspeed.zero.register_external_parameter(self, self.layer1.weight)

def forward(self, input):
    x = self.layer1(input)
    y = self.layer2(x, self.layer1.weight)
    return y


Create a self-flash event

I'm wondering if it is possible to create a flash grenade event (for myself without affecting other players) when there is no real flash grenade is dropped (it's a long story why I need this).
I know for sure that changing flashMaxAlpha or flash duration variables (e,g, setting the duration to a typical value during a flash event of 4.0f) don't result in anything. Can someone pls give me a hint on where I should be looking?

Not sure if I understand your question, but how I understand it, you need to fill your screen with a "white image" and then have it fade?

yes, - you got it right. And it should be done by tapping onto CS GO functionality.

check


St 1711 dummy branch

Change description
testing preview
JIRA link (if applicable)
ST-1711
Before merging a pull request make sure that:

[ ] tests have been updated / new tests has been added (if needed)
[ ] README and other documentation has been updated / added (if needed)

If this ticket will have any visible impact on users and is not behind a feature toggle, make sure that:

[ ] this ticket been reviewed by QA
[ ] the user story been signed off by the PO

Note that bug fixes, dependency updates and technical tasks do not directly impact the user experience and can be merged without QA and PO review.
If this user story cannot be immediately merged find a way to put it behind a feature toggle and get it merged.

dummy PR to test preview. Closing.


Relax any checks that block transformers needing to see target in training.

See https://github.com/alan-turing-institute/MLJ.jl/issues/874

Resolved by https://github.com/JuliaAI/MLJBase.jl/pull/732 waiting for 0.20 release.


Disable disallowed features

Add feature flags. These cannot be easily changed by users.
Feature flags affect the rendering (and toolbar usage of): navigation, markers, friends.
Move settings refreshing from Frame.tsx to App.tsx. Just a bit of maintenance.
Add a notice that some features have been disabled:

Closing the message will store a setting, stating that the message has been read. It will not be shown again.

Could you also bump the version?


APP: update app rarity constants

https://github.com/webaverse/app/blob/22e88566c92e760bb672e434c49ec59aee79a500/constants.js#L4-L10

added https://github.com/webaverse/app/commit/c6923d149cd3d91356cfbf4b639e403b9b8293ef

added https://github.com/webaverse/app/commit/c6923d149cd3d91356cfbf4b639e403b9b8293ef

fixed

fixed


DomainAdminPassword Should Be SecureString Param Type

@leovms - Do we have a setup to test this change?


Suggestion from Jesse for future work: keystroke timing analysis

@jesse asked to convey suggestion for future work (see the topic).

Related research for background information: https://people.eecs.berkeley.edu/~daw/papers/ssh-use01.pdf ... Jesse was envisioning to use similar approach for more generic attack/attacker fingerprinting instead of traffic (content) analysis itself.


An error occurred while placing player slot items! java.lang.NullPointerException: null

Logs are attached as a link
in game screenshot

Database for builders

https://pastebin.com/PXks3ZCi

Are those slot columns' default values actually "0" or is that just your program displaying "null" as "0"?

Are those slot columns' default values actually "0" or is that just your program displaying "null" as "0"?
I just opened up terminal and yes it seams like those values are 0

Looks like there is a problem with default values in the SQL statements...

specifically with default NULL values

specifically with default NULL values

if i infer correctly the values should all be (NULL), that may have been my program changing the values when i was manually setting the lang column

Issue has been fixed after manually setting all the 0's to (NULL)


Update answers to When is CockroachDB a good / not good choice

Update answers to:
When is CockroachDB a good choice?
Adds: Multi-region deployments, Cloud migrations
Question: When is CockroachDB not a good choice?
Removes: Complex SQL Joins
cc @awoods187 @petermattis @nstewart

This change is

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

@jseldess Good point. Adding a PTAL for @nstewart, as well.
And yes, please take over and apply it to the 2.0 FAQ. Much appreciated!

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


feat: create eslint shareable config module

https://github.com/SAP/open-ux-tools/issues/983

Create eslint plugin module
Run npm run int for TS projects when using UX_DEBUG=true UX_DEBUG_FULL=true  npm test in FE and FF writer
Fix lint issues in templates

Build has completed.  But status is not reported. Merging.

Thanks @docirl @tobiasqueck @IainSAP


feat: Port Data.Finset.MulAntidiagonal

[ ] depends on: #2152

This PR/issue depends on:

leanprover-community/mathlib4#2152
By Dependent Issues (🤖). Happy coding!

This PR/issue depends on:

leanprover-community/mathlib4#2152
By Dependent Issues (🤖). Happy coding!


OAuth authentication not working

The flow as described in the documentation is working fine till JiraClient.oauth_util.swapRequestTokenWithAccessToken method call. I get the access token but creating the JiraClient  instance with consumer_key, private_key, token and token_secret in the 4th step gives oauth_problem=token_rejected error when making a API call.
Alternatively I tried to use the access_token return by JiraClient.oauth_util.getAuthorizeURL method call to make the API request like http://<API_path>/?access_token=<access_token> this works fine.
Can anyone help me to clarify this issues.

I'm having the same issue:
{ statusCode: 401, data: 'oauth_problem=token_rejected' }

Oh nvm, I was using a wrong host on the swapRequestTokenWithAccessToken method.


ImportError: No module named nototools

I cloned the nototools repo and ran brew install --with-cairo harfbuzz cairo pango pygtk imagemagick, pip install -r requirements.txt, and python setup.py develop. And then I cloned this repo and ran make, getting the following message:
Traceback (most recent call last):
  File "./flag_glyph_name.py", line 25, in <module>
    import add_emoji_gsub
  File "/Users/4cm4k1/Code/noto-emoji/add_emoji_gsub.py", line 28, in <module>
    from nototools import font_data
ImportError: No module named nototools

I've tried a few things to try to fix it, but I've had no luck. Would anyone have a suggestion?

Python versions: 2.7.14, 3.6.3
macOS version: 10.13.1 Beta 2

hello @4cm4k1
can you successfully import nototools from the Python interactive shell or does it give you ImportError there as well?
How did you install python? Homebrew, or Apple's built-in python, or pyenv, etc?
If you have multiple pythons, make sure you run the noto-emoji scripts with the same python used to install nototools.

What does pip --version show as python version and site-packages folder? Make sure it is linked with the expected python binary.
Try to call pip as python -m pip ... (or python3 -m pip ...) instead of just pip, as that explicitly runs pip for a specific python binary and may avoid this confusion.

Please reopen if you're still having trouble with this.


Added of 'Get Window Position' and 'Set Window Position' keywords

Added 'Get Window Position' and 'Set Window Position' keywords matching the Selenium functionality.
#366
#367

The 'Double Click Element' test case failed, but 'Get and Set Window Position' passed. Can someone help me with the failed test?

I don't think this is actually a failure with your PR. The same thing happened with mine (#365).

@ktarasz I somewhat traced it down. It's a known failure though it doesn't actually seem to be a problem with this library. It seems like it's an actual bug in selenium. Regardless, I've added a PR to make it a non-critical failure so you test will pass after that gets merged. (See #371)

@ktarasz, if you'll update this I'll merge it.


Use the latest chatbot UI to support streaming output

Hello, I am a code review bot on flows.network.
It could take a few minutes for me to analyze this PR. Relax, grab a cup of coffee and check back later. Thanks!


Created functionality for Login Authentiaction

The login authentication logic is structured at AuthContext class and useAuth hook.

Added faculty number validation
Better error messaging
Private routing
Login, Logout, Dashboard redirection added
Moved useAuth into hooks directory
Moved AuthContext into context directory


java.net.UnknownHostException

Hey,
Sometimes I get an UnknownHostException related to address i.instagram.com when I'm trying to make a DirectGetPresenceRequest. This does not happen often but it happened a couple of times already. I even added some logic to just repeat the request in case it fails due to this reason for a maximum of 5 times and after it failed the first first, it immediately failed the next 5 attempts. After running the application again, it worked fine tho. Any ideia what's causing this? Thanks.

I'm not sure, could be a problem on Instagram's or your end.
I say it's good solution to retry the requests.


Remove raise immediately statements

This commit removes raise only statements in try-except branches
found by pylint. Although the purpose was to re-raise the same
exception, it makes sense to be explicit about it. The change
basically replaces the empty raises with an explicit raising of
the same exception that the try branch brakes on.
Fixes #201
Signed-off-by: Nisha K<EMAIL_ADDRESS>

Looks good, just had one question about catching the APIError.


Spec incorrectly claims that predeclared bindings cannot be reassigned

In the Name binding and variables section, it says:

The set of predeclared names includes the universal constant values None, True, and False, and various built-in functions such as len and list; these functions are immutable and stateless. [...] Starlark programs cannot change the set of predeclared bindings or assign new values to them.

However both Bazel and https://github.com/google/starlark-go allow reassignment of True:
True = 3
print(True)

I initially considered this to be a bug in the implementations rather than the spec, since it seems like a good idea to prevent rebinding of the predeclared names. But then I realized that this would make it impossible to add new predeclared names in a backwards-compatible way. So perhaps this is a necessary evil.

There is shadowing. Values in the "predeclared" block can be shadowed by values in the module or file block. It's all resolved statically (unlike Python).
Indeed, this was designed with backwards compatibility in mind.

Ah, my bad. I can see how this is considered to be shadowing rather than rebinding. A note to clarify could help, but I see now that this is not a bug in the spec.

The spec mentions later:

The outermost block of the Starlark environment is known as the "predeclared" block. It defines a number of fundamental values and functions needed by all Starlark programs, such as None, True, False, and len, and possibly additional application-specific names.
These names are not reserved words so Starlark programs are free to redefine them in a smaller block such as a function body or even at the top level of a module. However, doing so may be confusing to the reader. Nonetheless, this rule permits names to be added to the predeclared block in later versions of the language (or application-specific dialect) without breaking existing programs.

But I agree that the first quote can cause confusion.
On a side-note, this code shows the difference with Python:
print(len)
len = 2

Python3 will print <built-in function len> while Starlark has an error before the evaluation (which proves that we are not modifying len).

Thanks for the explanation.
I found another issue -- running it by you here before I open another issue, in case I'm misunderstanding again.
The spec says:

The sets of names bound in the file block and in the module block do not overlap: it is an error for a load statement to bind the name of a global, or for a top-level statement to bind a name bound by a load statement.

I take this to mean that the following Starlark should be rejected:
# In a .bzl file, this code is rejected.
# In a BUILD file, it prints "3"
load(":test.bzl", "a")
a = 3
print(a)

In my testing, I find that this fragment is rejected in a .bzl file (good), but accepted in a BUILD file. This seems contrary to the spec. Am I missing something?

Correct, this should be rejected.
When we decided on this behavior, there were many existing BUILD files, so we added an exception for them and postponed the migration. Ideally, a migration flag should be added to Bazel and later switched.
It's worth filing an issue on Bazel.

That makes sense.
I also notice that the spec makes no mention of the leading-underscore-is-private convention used by Bazel. I guess this is already mentioned in https://github.com/bazelbuild/starlark/issues/37.

See https://github.com/bazelbuild/starlark/blob/master/spec.md#load-statements

The literal string ("x"), which must denote a valid identifier not starting with _, specifies the name to extract from the loaded module. In effect, names starting with _ are not exported.


Add subspace method for learning LDS

The EM method for learning LDS in PRML is hardly working without a good initialization. The subspace method can be used for a good initialization. This method is not in PRML but BRML.

done


problems with installation on Python 3.11

Hi! In Python 3.11, the installation does not seem to work

(venv) localcreator@localcreator-hp:~/pythonProject$ pip install fracdiff
Collecting fracdiff
Downloading fracdiff-0.6.0-py3-none-any.whl (13 kB)
Requirement already satisfied: numpy<2.0.0,>=1.18.0 in ./venv/lib/python3.11/site-packages (from fracdiff) (1.25.0)
Collecting scikit-learn<0.25.0,>=0.24.0
Downloading scikit-learn-0.24.2.tar.gz (7.5 MB)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 7.5/7.5 MB 1.4 MB/s eta 0:00:00
Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... error
error: subprocess-exited-with-error
× Preparing metadata (pyproject.toml) did not run successfully.
│ exit code: 1
╰─> [73 lines of output]
:17: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
Partial import of sklearn during the build process.
:116: DeprecationWarning:
    `numpy.distutils` is deprecated since NumPy 1.23.0, as a result
    of the deprecation of `distutils` itself. It will be removed for
    Python >= 3.12. For older Python versions it will remain present.
    It is recommended to use `setuptools < 60.0` for those Python versions.
    For more details, see:
      https://numpy.org/devdocs/reference/distutils_status_migration.html

  INFO: C compiler: x86_64-linux-gnu-gcc -Wsign-compare -DNDEBUG -g -fwrapv -O2 -Wall -g -fstack-protector-strong -Wformat -Werror=format-security -g -fwrapv -O2 -fPIC

  INFO: compile options: '-c'
  INFO: x86_64-linux-gnu-gcc: test_program.c

  [Errno 2] No such file or directory: 'x86_64-linux-gnu-gcc'

  Traceback (most recent call last):
    File "/home/localcreator/pythonProject/venv/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
      main()
    File "/home/localcreator/pythonProject/venv/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
      json_out['return_val'] = hook(**hook_input['kwargs'])
                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    File "/home/localcreator/pythonProject/venv/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 149, in prepare_metadata_for_build_wheel
      return hook(metadata_directory, config_settings)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    File "/tmp/pip-build-env-ug1kwehm/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 380, in prepare_metadata_for_build_wheel
      self.run_setup()
    File "/tmp/pip-build-env-ug1kwehm/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 488, in run_setup
      self).run_setup(setup_script=setup_script)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    File "/tmp/pip-build-env-ug1kwehm/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 338, in run_setup
      exec(code, locals())
    File "<string>", line 301, in <module>
    File "<string>", line 297, in setup_package
    File "/tmp/pip-build-env-ug1kwehm/overlay/lib/python3.11/site-packages/numpy/distutils/core.py", line 135, in setup
      config = configuration()
               ^^^^^^^^^^^^^^^
    File "<string>", line 188, in configuration
    File "/tmp/pip-build-env-ug1kwehm/overlay/lib/python3.11/site-packages/numpy/distutils/misc_util.py", line 1050, in add_subpackage
      config_list = self.get_subpackage(subpackage_name, subpackage_path,
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    File "/tmp/pip-build-env-ug1kwehm/overlay/lib/python3.11/site-packages/numpy/distutils/misc_util.py", line 1016, in get_subpackage
      config = self._get_configuration_from_setup_py(
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    File "/tmp/pip-build-env-ug1kwehm/overlay/lib/python3.11/site-packages/numpy/distutils/misc_util.py", line 958, in _get_configuration_from_setup_py
      config = setup_module.configuration(*args)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    File "/tmp/pip-install-our_u18v/scikit-learn_950f7f89433040a78a59d64776b761d8/sklearn/setup.py", line 83, in configuration
      cythonize_extensions(top_path, config)
    File "/tmp/pip-install-our_u18v/scikit-learn_950f7f89433040a78a59d64776b761d8/sklearn/_build_utils/__init__.py", line 45, in cythonize_extensions
      basic_check_build()
    File "/tmp/pip-install-our_u18v/scikit-learn_950f7f89433040a78a59d64776b761d8/sklearn/_build_utils/pre_build_helpers.py", line 106, in basic_check_build
      compile_test_program(code)
    File "/tmp/pip-install-our_u18v/scikit-learn_950f7f89433040a78a59d64776b761d8/sklearn/_build_utils/pre_build_helpers.py", line 66, in compile_test_program
      ccompiler.compile(['test_program.c'], output_dir='objects',
    File "/tmp/pip-build-env-ug1kwehm/overlay/lib/python3.11/site-packages/numpy/distutils/ccompiler.py", line 91, in <lambda>
      m = lambda self, *args, **kw: func(self, *args, **kw)
                                    ^^^^^^^^^^^^^^^^^^^^^^^
    File "/tmp/pip-build-env-ug1kwehm/overlay/lib/python3.11/site-packages/numpy/distutils/ccompiler.py", line 369, in CCompiler_compile
      single_compile(o)
    File "/tmp/pip-build-env-ug1kwehm/overlay/lib/python3.11/site-packages/numpy/distutils/ccompiler.py", line 328, in single_compile
      self._compile(obj, src, ext, cc_args, extra_postargs, pp_opts)
    File "/tmp/pip-build-env-ug1kwehm/overlay/lib/python3.11/site-packages/numpy/distutils/ccompiler.py", line 91, in <lambda>
      m = lambda self, *args, **kw: func(self, *args, **kw)
                                    ^^^^^^^^^^^^^^^^^^^^^^^
    File "/tmp/pip-build-env-ug1kwehm/overlay/lib/python3.11/site-packages/numpy/distutils/unixccompiler.py", line 58, in UnixCCompiler__compile
      raise CompileError(msg) from None
  distutils.errors.CompileError: Command "x86_64-linux-gnu-gcc -Wsign-compare -DNDEBUG -g -fwrapv -O2 -Wall -g -fstack-protector-strong -Wformat -Werror=format-security -g -fwrapv -O2 -fPIC -c test_program.c -o objects/test_program.o" failed with exit status 127
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.
PS. This solution helped: pip install git+https://github.com/fracdiff/fracdiff.git

yeah https://github.com/fracdiff/fracdiff/issues/153


Adds feature roadmap table

Updates README to include details about this fork and a detailed feature status table

Current dependencies on/for this PR:

main

PR #4   👈

This comment was auto-generated by Graphite.


Single binary target with subcommands, using argh and fuchsia's CLI conventions

I originally had the split binaries when I was embedding data into some of them; I don't really need to bother any more. These seem like a decent set of opinions so let's follow them instead of independently bikeshedding: https://github.com/google/argh https://fuchsia.dev/fuchsia-src/development/api/cli#execution

done by c3345e787109b797910dede18f5d5da8257f36dc


Add school King Edward VI School (Southampton)

@kes-jjb First stage of verification passed, official domain is kes.school 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://senior.kes.school/6/contact-kes

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


Enable the creation of an identity from a KeyPair in the Account

Description of change
The IOTA Streams team requested the ability to create an identity from an existing KeyPair. Due to the pre-existing KeyType field on the IdentityCreate type, only a SecretKey was added to that type, to avoid adding the KeyType field a second time (in the KeyPair itself) and the possibly resulting confusion.
As an example, developers can create an identity from an ed25519 KeyPair, by doing:
let keypair: KeyPair = KeyPair::new_ed25519()?;

let id_create = IdentityCreate::new()
    .key_type(KeyType::Ed25519)
    .secret_key(keypair.secret().clone());

account.create_identity(id_create).await?;

Alternatively, it's also possible to add a new method with a custom secret key:
let keypair: KeyPair = KeyPair::new_ed25519()?;

let command: Command = Command::create_method()
  .type_(MethodType::Ed25519VerificationKey2018)
  .fragment("my-next-key")
  .secret_key(keypair.secret().clone())
  .finish()?;

account.update_identity(did, command).await?;

Note: A stronghold upgrade was necessary, to include a bugfix.
Links to any relevant issues
Part of #325
Type of change
Add an x to the boxes that are relevant to your changes.

[ ] Bug fix (a non-breaking change which fixes an issue)
[x] Enhancement (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Fix

How the change has been tested
Three tests were added:

A stronghold test, test_store_secret_key to make sure that inserting a secret key into stronghold works
Two account tests

test_create_identity_from_secret_key
test_create_method_from_secret_key

I modified the account_stronghold example with the above code to test the change, but no automated test was added. It seems like we don't have high-level account-stronghold tests in the first place.
Change checklist
Add an x to the boxes that are relevant to your changes.

[x] I have followed the contribution guidelines for this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

In addition to what you've added already here, is it possible to easily add the ability to use an existing keypair not as the base for the account (CreateIdentity command), but as the base for a method (CreateMethod)? This is important for allowing existing account owners to add a new verification method to longer lasting identities. For example I made this for testing purposes on my end, i'm sure there's a better way to go about it though:
https://github.com/DyrellC/identity.rs/commit/9427a528196c8e3b6fd5e86b168644ae3253da2a

is it possible to easily add the ability to use an existing keypair as the base for a method (CreateMethod)?

Sounds like a reasonable addition to me - we can add this. And thanks for providing the code!

@DyrellC The PR is done now, feel free to take another look and confirm that this is what you expected.

@DyrellC The PR is done now, feel free to take another look and confirm that this is what you expected.

Yup looks good to me! Thank you!


【AT】 commit transaction of phase two

What would you like to be added:
Why is this needed:

assign me

assign me

looking forward your contributing


Retire systemdMinimal

Description of changes
This PR retires systemdMinimal. It's a follow-up to #261798. The goal is to avoid having multiple sets of systemd binaries in a typical system closure and save everyone some disk space. This is especially relevant for smaller image-based systems (see #263462).
We replace systemdMinimal with:

systemd, if systemd commands or udevadm is needed (bluez) or when systemd is already in the closure (dbus)
systemdLibs

for dependencies of systemd itself to break dependencies
for packages that just need libudev or libsystemd that don't already have systemd in their closure.

Some of the uses of systemdLibs could just be systemd, if we manage to give pkgs.systemd a lib output that doesn't drag in lots of unneeded binaries.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

I unfortunately have no time to finish this. :-1:


Added High School

High schools official website URL: http://teslabg.edu.rs/
Page that shows a long term IT course(link at the bottom of the paragraph shows what classes the student takes during 4 years of studying): http://teslabg.edu.rs/strana.aspx?id=136&parentid=342
I have no proof that this is the domain for emails the school has given us, as we have got these at the start of this school year.

Actually i should have some proof let me dig it up

Censored names of my classmates and me, this is all we got from our head teacher. Passwords were changed when we logged in so i left those

@BogiBruh Pull request merged. Thank you!
I deleted your last comment, because it shows that you had identical passwords, and a stranger might try and login before an actual inbox user, which is a potential security issue.


Added roles field to node info

This will add the Roles field to node info.
This is exactly the same change made in #745, but in the Elasticsearch 5.X branch.

Thanks for supporting Elastic. I hope to release a new Elastic version over the weekend, including this backport.

Thank you.


Edge Case Where Foobar's Presence Never Disappears

If you close foobar2000 before setting a song, it will remain in that initialized presence even if you close foobar2000, but if a song is set and you close, it acts as normal.

I'll give it a look, thanks.

It does have to set the initialization state first - but if you close it in between that and selecting a song, then it gets stuck. I've done it on occasion where I accidentally launch foobar, close it and then discord thinks I'm still in foobar.

Okay, that's very odd. If the associated process of a rich presence ends, discord should normally clear everything related.
I'll try to reproduce it during the next days.
Maybe I can fix that, I kind of have an idea.

Right? It might just be our order is funky, but I don't have access to my normal desktop to test right now.

Have you been able to reproduce? I can always record a video too.

No, I havent been able to reproduce.
Yes, you can if you like to.

Solved in latest version


WIP: Kleros Protocol Integration

This is the first draft (still work-in-progress) of the Kleros Protocol integration as a decentralized Dispute Resolver (DR).

Thanks @0xartem, this is interesting and indeed one of our important milestones ahead. Proposal looks great as draft, please add Rationale section as well.

Observation 1
For a couple of reasons, we should not change the resolveDispute method signature.
Reason A

It represents a breaking change, and from a versioning standpoint it would require a major version of 3.0.0. Of course we can make major version changes till the cows come home, the number line is infinite. But dropping 3.0.0 this close to 2.0.0 sends a signal that we’re not really on top of things. We need to keep this in mind for all ongoing changes. Maintaining backward compatibility will become more important over time, so when proposing and voting on changes, we must strive to do so in a backward compatible way.

Reason B

The method requires the signature of the party that isn’t calling, so that we know the two are in accord. This signature is important and can't be dropped, otherwise -- without consulting each other -- the seller could resolve disputes immediately with a zero buyer percent or the buyer could resolve with 100%.

We could add an alternative method to preserve backward compatibility, but from the specification, I don’t understand why it is even being changed. It's not necessary to call it before doing an escalateDispute call.

An alternative that doesn't take the signature but requires that both parties call would be more complex, requiring state to track which party has called already and what the proposal was. Again though, not sure why this is even part of the proposal, which should start with escalateDispute and end with decideDispute or refuseEscalatedDispute.

Observation 2
In the diagram, a “resolution proposal” is being created before escalation. It's purpose isn't explained anywhere in the specification. Also, I don’t see any new proposeResolution method being added and resolveDispute is not a proposal, it is an action. The diagram doesn’t say what method is being called.
Observation 3
There is no rationale section that describes why this architecture is being proposed. The specification section just says “In order to implement integration of Kleros Protocol the following changes have to be made.”
I would just ask for more detail in the specification, a rational section that explains why this approach is being taken, what tradeoffs or alternatives were considered, and a rework with an eye toward backward compatibility.

Thanks for your comments @cliffhall. I appreciate the time you spent on it. I will work on the BPIP more and update it taking into account your observations.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I love this proposal, and I agree with @cliffhall's comments that we can't make a breaking change to the API. I think @0xartem's proposal to add in an onchain serialisation of the mutual resolution proposals is the right thing to not break the existing APIs.
Futhermore, i feel like we should merge this into the set of BPIPs, being able to rely on decentralised dispute resolution,l like Kleros, for protocol DR is definitely something that we need to explore. And having a proposal that extends the protocol to enable such activity seems like a positive step for the protocol.
THIS LGTM ! :shipit:

Let's get this merged in !


Cannot GetProcAddress for Py_DontWriteBytecodeFlag

When running docker compose, I get the following error:
Cannot GetProcAddress for Py_DontWriteBytecodeFlag
I've now gotten this error on both Windows 10 and Windows 7 (with Docker Toolbox). This has worked on other Windows machines without this error.
Diagnostic uploaded:
7F49371E-5D55-4825-99F2-9BDEF11B5F94/2016-10-20_15-29-54

Steps to reproduce the issue:
docker-compose --version
Describe the results you received:
Cannot GetProcAddress for Py_DontWriteBytecodeFlag
Describe the results you expected:
docker-compose version ...
Additional information you deem important (e.g. issue happens only occasionally):
Happens always on some Windows machines, but not others.
Output of docker version:
Docker version 1.12.1, build 23cf638

Output of docker info:
Containers: 1
 Running: 0
 Paused: 0
 Stopped: 1
Images: 11
Server Version: 1.12.1
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 14
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host overlay null
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: seccomp
Kernel Version: 4.4.20-moby
Operating System: Alpine Linux v3.4
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 1.942 GiB
Name: moby
ID: YYX3:44FZ:3HDN:GMCQ:KBGL:PYVQ:ZUPI:R4FE:NR5O:ZWSG:IWD5:6ZMZ
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: 14
 Goroutines: 23
 System Time: 2016-10-20T14:37:21.8635547Z
 EventsListeners: 0
Registry: https://index.docker.io/v1/
Insecure Registries:
 <IP_ADDRESS>/8

Additional environment details (AWS, VirtualBox, physical, etc.):

@aanand Can you take a look at this, please? It seems to be related to docker-compose.

can you reopen this in the docker-compose issue tracker? https://github.com/docker/compose/issues
That's way it'll get the attention of the docker compose team (and if there's a bug, it can be resolved there)
I'll close the issue in this issue tracker, but thanks for reporting!


Folder for sample nb_funcs

I thought it might be a good idea to have a directory that has a collection of sample nb_funcs that people can use. I went ahead and included an updated version of the hydrophobic spring one.

I like this idea! But could we make a file nma/nb_func.py so users can import from nma.nb_func?
Also could you rebase and merge changes from master? I can help with this tomorrow if you're around.


particles are not working

The particles only work for me when I first open Atom. Opening another document within a project causes the particles no longer appear. Only the shake happens. Switching back to the first document does not fix the issue, and particles only appear after closing and re-opening atom.

The same thing happens for me.

restarting the editor worked for me, weird

Particles worked for me a couple minutes after I toggled it. After those few minutes, the particle effect stopped.

@mattds9 can you describe your issue?

Closing issues on old versions of this package.
Open again if the problem persists with the latest release.


⚠ Bofrost.lu has degraded performance

In 6e13ef3, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1860 ms

Resolved: Bofrost.lu performance has improved in b1a6bb8 after 6 minutes.


rsync: fix missing ipv6 support (again)

Description of changes
The patch was removed in 21604e8d2bf4 despite not being included in rsync v3.3.0, which broke IPv6 on at least darwin again.
cc @kashw2
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Is rsync so deeply embedded that this needs to be merged into staging? Seems like it could benefit from a merge to master if it doesn't cause too many downstream rebuilds, although I can totally imagine 70% of nixpkgs indirectly depending on rsync 😂

#305294 went to staging and it has a lot of rebuilds:

Yeah makes sense. 🤷

Looking at https://github.com/RsyncProject/rsync/compare/v3.2.7...v3.3.0, it does indeed seem to be missing fd268fc9c2a46edfcb5bc5cb218bf63153701054 which introduced this fix upstream.
My apologies for the regression, I'd assumed a point in time release methodology instead of picking commits into a release.


Support for changing payment without a checkout flow

Hi,
Any plans to add a feature that allows users to change a payment method without a "buy button" just with picking a credit card/Apple pay etc..? so we can save it for later usage using the pfToken ?
Thanks
Guy.

This is part of the ShopperConfiguration feature and should be fully supported in the next release.


Rename migration docs

Description:
Currently, the MI migration documents follow the initial title pattern used when the documents were created since MI was released as a part of APIM.
For example:
Upgrading from WSO2 API-M 4.0.0 to WSO2 API-M 4.2.0
However, since MI is a separate product, we can rename the files to follow the format below:
For example:
Upgrading from WSO2 MI 4.0.0 to WSO2 MI 4.2.0
Reference:
https://github.com/wso2-enterprise/wso2-mi-internal/issues/816#issuecomment-2184828429

Hi @DinithiDiaz,
Any update on this task?
Regards,
Nadee


KDF life-cycle documentation

[x] documentation is added or updated
[ ] tests are added or updated

I'd appreciate a review of the KDF parts of this before I dive into the rest.  Getting agreement on the formatting etc up front would avoid a lot of manual rework.

Merged to master.


Add openFilters configuration option

The default behavior of the filter display is to show values only when there is one filter with six or fewer values. This PR adds a configuration option called openFilters that allows users to include a list of filter names that will open by default, regardless of the number of filters or values present.
new PagefindUI({
    element: "#search",
+    openFilters: ['Tags','Type']
});

Nice one! Thanks for the contribution 😄


Pass templated shell server hostname in problem description

So that a GET on /problem routes will generate problem description URLs based on shell server configured hostname.
This enables generated instance data (via shell_manager deploy) to be used between multiple shell servers

This would also improve issues outlined in #65 with two possible URLs as result of http/s proxy from #366


Build single attribution page

Ala Who's On First sources.

Ideally it's published on openaddresses.io as well, but first round can just be a README.md markdown file in the repo.

This data can help drive the page: http://results.openaddresses.io/latest/licenses.json


Make Jessie's email a clickable link on research highlight

Challenging @mrueschman to do a pull request for this one!
Hint from
You can email me at<EMAIL_ADDRESS>to contact me.

You can email me at
= mail_to<EMAIL_ADDRESS>to contact me.

Closed in 7-2-stable


Ignore broken CSS

Broken CSS inside a page will throw an error on our behalf. This should
be ignored because we are not in control of the CSS on the page we are
recording.

@dphuang2 Thanks! This looks important.


Correct race condition in AsyncTargetWrapperExceptionTest

Modify AsyncTargetWrapperExceptionTest to avoid race condition with AsyncWrapper thread timer.

Thanks!
Can you explain why the sleep isn't needed?

Sure. The Thread.Sleep() call doesn't guarantee that the AsyncTargetWrapper actually processes its queue. The timer that is used may fire, but never actually be scheduled to run. If that happens the test will fail and seems to eventually lead to an exception in xUnit. Calling targetWrapper.Close will force the queue to be processed and stop the timer, which will ensure that the internal logger is written to.

Thanks for the explanation :+1: :+1:


watchFuture tanks performance

Hello,
I have this function that returns a future of whether a species is favorited by the user.  (The favorite status is stored on the device in a local database, Isar)

I then have a widget like this where the favorite status is watched using watchFuture inside of the build method. The favorite status is then used in the widget to show an icon (not shown in image)

This works functionally, and when I remove the favorite status manually, the UI is instantly updated.
However, this completely tanks performance: I see that the widgets are rebuilt every frame.

I understand that a rebuild needs to happen so that the widget can be instantly updated to show the new favorited status.
But how can I improve the performance?
I'm thinking that only really the favorite status widget needs to be rebuilt (I use a ChoiceChip to display the favorite status)

But the rest of the widgets should not be rebuilt? How can I achieve this?
Should I move the ChoiceChip to its own widget, call watchFuture only there in the build method so that the other widgets do not need to be rebuilt?
I'm pretty new to flutter and state management so any advice would be great. Or if there is a better way to use watch_it to achieve this, basically my use case seems pretty common.
Thanks a lot!

I think your problems has different aspects

yes putting the Chip into its own widget would only rebuild that widget which would improve performance.
The bigger problem in my opinion is that you get e new future on every rebuild because you call that function that calls into  ISAR.

Doesn't ISAR also offer a Stream based API that would send a new data item whenever its data changes?
otherwise I would recommend adding a business logic layer in between that checks the status of that field in regular intervalls and then updates a ValueListenable that then is watched by watchValue.
I hope that helps :-)


Need to be able to disable/uninstall plugins

There are sometimes problems with a particular plugin that I would like to disable temporarily, or when one decides to stop using a plugin, it is nice to be able to remove it. There does not appear to be a way to do that with dein because it puts all the vim files in one directory ~/.vim/dein/.dein. This makes it very difficult both to to disable something temporarily as well as to remove completely.

lazy load plugin will not be moved to ~/.vim/dein/.dein,you can use lazy load feature.also call dein#recache_runtimepath() can recache ~/.vim/dein/.dein.

for uninstall plugin or disable plugin,I do not know this feature will be added to dein.

@matthauck You can disable the merge feature by 'merged': 0.
And I will add disable plugins feature.

Implemented.

Wow, that was fast, thanks!
On Friday, April 8, 2016, Shougo<EMAIL_ADDRESS>wrote:

Implemented.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/Shougo/dein.vim/issues/71#issuecomment-207683327

--
Matt

Question: So if there is no uninstall that will be added to dein, how would you suggest we uninstall things? The only options seem to be:

Keep a list forever of things disabled
Blow away the .dein directory and reinstall everything

Neither of these seem optimal. Is there another workflow that you have for removing plugins that you can share?

@matthauck
You can get the directories by dein#check_clean().
So, you can remove the directories like this.
call map(dein#check_clean(), "delete(v:val, 'rf')")

I think it's still complicated. I will never remember this command to remove disabled plugins.
What do you think of a dein#uninstall([{plugins}]) or dein#remove([{plugins}]) function? Every package manager out there has it. It could call disable() and then filter check_clean()'s result for the plugin to remove. And then delete it.
Also, I have a question on dein#disable(): The documentation says

Note: This command must be executed before dein loads
the plugins.

Does this mean I have to put the call to disable() in my vimrc between dein#begin() and dein#end()? Or can I also call it in a running vim/nvim?

@schoettl Why this feature is needed?
I can add dein#uninstall(), but you must remove the dein#add() line from your vimrc.
It is the same behavior with Bundle.

Does this mean I have to put the call to disable() in my vimrc between dein#begin() and dein#end()? Or can I also call it in a running vim/nvim?

You cannot disable the plugin after it is loaded.
Because, Vim cannot unload the loaded plugins.

I don't know Bundle. Ok, I don't understand how to disable a plugin. Is deleting the dein#add() in the vimrc enough?

Yes.

@schoettl if it is not a lazy load plugin, you need run :call dein#recache_runtimepath() after start vim I think.

@wsdjeg
Dein.vim detects the vimrc changes and calls dein#recache_runtimepath() automatically.

@Shougo thanks
but will dein detect these changes?

call dein#add('foo/abc') to call dein#add('foo/abc',{'on_i':1})   ===> not sure dein will remove plugin from cache dir
call dein#add('zaa/abc') to call dein#add('foo/abc')     ===> I think dein could rechche this by enable checkinstall feature at vim startup.
call dein#add('foo/abc',{'on_i':1}) to call dein#add('foo/abc')   ===> for this, how dein works?

Thank you two! I got it; it's really simple.
I was confused earlier because it seemed to not work, but that was obviously my mistake.

@wsdjeg Yes.
But you must specify dein#begin() 2nd argument.

@Shougo thanks, maybe I should add compared vimrcs in to my config

I just want to note that if you have your package configuration in another file besides ~/.vimrc (vundle recommended it for first installation, but it is probably not as useful for dein), dein will not detect the removal of the dein#add, so you will need to run call dein#recache_runtimepath() anyway.

@trisys3 Thank you for saving me time! I was going nuts trying to figure out why plugins were lingering after I was using a file ~/testvimrc even when I had removed the dein#add lines as well as called
call map(dein#check_clean(), "delete(v:val, 'rf')")

as suggested by @Shougo above. I was not aware of this caching and I found it surprising and unintuitive that a plugin still seemed to be installed even though I had called the "recommended" uninstallation method.

So what's the conclusion? I ran all these commands and the plugin still remains in ~/.config/nvim/.cache/.
Is the solution to just rm -rf ~/.config/nvim/.cache/ and restart nvim?

Is the solution to just rm -rf ~/.config/nvim/.cache/ and restart nvim?

It removes all installed plugins.
You should not do this.

Is it safe to include call map(dein#check_clean(), "delete(v:val, 'rf')") in init.vim?

Safe.
But it must be after plugins loaded.
I don't recommend to uninstall automatically.
Other plugin/package managers don't do it.


Fix code scanning alert - Inefficient regular expression

Tracking issue for:

[ ] https://github.com/noplriv/test-etherpad-lite/security/code-scanning/1

opening, fixing now.


render_markdown_path: Refactor and test pure_markdown rendering logic.

@timabbott: FYI :)

Nice, verified that the test failed with the old code, and merged, thanks @eeshangarg!


Draft Facility ranking mockup page

-Do all work for this task in a branch called issue-06
-Mockup of Facility ranking page

Merged into main


Exception in Microsoft.Data.SqlClient 1.1.3 when opening Sql Server Connection when run from the c program

Describe the bug
When running the dotnet library using Microsoft.Data.SqlClient 1.1.3 from the c program using nethost.h/dll approach I get the following error when trying to open connection. I don't use any configuration for the library.
Unhandled exception. System.TypeInitializationException: The type initializer for 'Microsoft.Data.SqlClient.SqlAuthenticationProviderManager' threw an exception.
 ---> System.InvalidOperationException: Failed to read the config section for authentication providers.
 ---> System.Configuration.ConfigurationErrorsException: Configuration system failed to initialize
 ---> System.PlatformNotSupportedException: Operation is not supported on this platform.
   at System.Configuration.ClientConfigPaths..ctor(String exePath, Boolean includeUserConfig)
   at System.Configuration.ClientConfigPaths.GetPaths(String exePath, Boolean includeUserConfig)
   at System.Configuration.ClientConfigurationHost.get_ConfigPaths()
   at System.Configuration.ClientConfigurationHost.GetStreamName(String configPath)
   at System.Configuration.ClientConfigurationHost.get_IsAppConfigHttp()
   at System.Configuration.Internal.DelegatingConfigHost.get_IsAppConfigHttp()
   at System.Configuration.ClientConfigurationSystem..ctor()
   at System.Configuration.ConfigurationManager.EnsureConfigurationSystem()
   --- End of inner exception stack trace ---
   at System.Configuration.ConfigurationManager.EnsureConfigurationSystem()
   at System.Configuration.ConfigurationManager.PrepareConfigSystem()
   at System.Configuration.ConfigurationManager.GetSection(String sectionName)
   at Microsoft.Data.SqlClient.SqlAuthenticationProviderManager..cctor()
   --- End of inner exception stack trace ---
   at Microsoft.Data.SqlClient.SqlAuthenticationProviderManager..cctor()
   --- End of inner exception stack trace ---
   at Microsoft.Data.SqlClient.SqlInternalConnectionTds..ctor(DbConnectionPoolIdentity identity, SqlConnectionString connectionOptions, SqlCredential credential, Object providerInfo, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance, SqlConnectionString userConnectionOptions, SessionData reconnectSessionData, Boolean applyTransientFaultHandling, String accessToken, DbConnectionPool pool, SqlAuthenticationProviderManager sqlAuthProviderManager)
   at Microsoft.Data.SqlClient.SqlConnectionFactory.CreateConnection(DbConnectionOptions options, DbConnectionPoolKey poolKey, Object poolGroupProviderInfo, DbConnectionPool pool, DbConnection owningConnection, DbConnectionOptions userOptions)
   at Microsoft.Data.ProviderBase.DbConnectionFactory.CreatePooledConnection(DbConnectionPool pool, DbConnection owningObject, DbConnectionOptions options, DbConnectionPoolKey poolKey, DbConnectionOptions userOptions)
   at Microsoft.Data.ProviderBase.DbConnectionPool.CreateObject(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)
   at Microsoft.Data.ProviderBase.DbConnectionPool.UserCreateRequest(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)
   at Microsoft.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, UInt32 waitForMultipleObjectsTimeout, Boolean allowCreate, Boolean onlyOneCheckConnection, DbConnectionOptions userOptions, DbConnectionInternal& connection)
   at Microsoft.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, TaskCompletionSource`1 retry, DbConnectionOptions userOptions, DbConnectionInternal& connection)
   at Microsoft.Data.ProviderBase.DbConnectionFactory.TryGetConnection(DbConnection owningConnection, TaskCompletionSource`1 retry, DbConnectionOptions userOptions, DbConnectionInternal oldConnection, DbConnectionInternal& connection)
   at Microsoft.Data.ProviderBase.DbConnectionInternal.TryOpenConnectionInternal(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOptions)
   at Microsoft.Data.ProviderBase.DbConnectionClosed.TryOpenConnection(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOptions)
   at Microsoft.Data.SqlClient.SqlConnection.TryOpen(TaskCompletionSource`1 retry)
   at Microsoft.Data.SqlClient.SqlConnection.Open()
   at Gen3Host.EntryPoint.Initialize(IntPtr argsPointer, Int32 argsLength) in C:\dev\Astro\repl\servers\platform-invoke\Gen3Host\EntryPoint.cs:line 23

To reproduce
Create a dotnetcore3.1 or dotnet standard 2.0 library that use Microsoft.Data.SqlClient. Then call the library from the c code using nethost dll.
public static class EntryPoint
    {
        public static int Initialize()
        {
            SqlConnection sqlConnection = new SqlConnection("Server=sqlserver_hostname;Database=test;User Id=test;Password=test;");
            sqlConnection.Open();
       }
   }

Expected behavior
Connection can be established
Further technical details
Microsoft.Data.SqlClient version: (found on the nuget or Microsoft.Data.SqlClient.dll) 1.1.3
.NET target: (e.g. Framework 4.7.1, Core 3.1.8)
SQL Server version: (e.g. SQL Server 2017)
Operating system: (e.g. Windows 10 Enterprize

@troglas Can you override the Microsoft.Data.SqlClient version to 2.1.0-preview1.20235.1 and see if the issue comes up?

Hi JRahnama,
I obeserved this error in EF Core 3.1.8 which uses Microsoft.Data.SqlClient which uses version 1.1.3. When I add the 2.0.1 to the my custom library which references EF Core 3.1.8 the issue does not appear.

Hi @troglas
Closing this issue as Microsoft.Data.SqlClient v2.0.1 contains the fix. You should be able to direct reference this version in your application and wait for later EF Core versions to take a dependency on it.


Start working on reading adc_sums

These seem to be raw adc sums that need to be calibrated, we need to figure out how to correctly convert to pe.
They also have both gains, so we also somehow need to perform a gain selection.
Maybe @ParsonsRD knows? If not I'll contact konrad

This is intended  to fix #1777
If we don't want this, we should close #1777 as "won't fix"

This is intended to fix https://github.com/cta-observatory/ctapipe/issues/1777

Ah, right, in the raw data, this has no useful meaning (it's just the full sum of the waveform), however, for DST-style output (which is closer to DL1), it's the DL1 image.   So if we want to use it, we have to do the same calibration as for the simulated waveform, but using the integrated  pedestal.  But in that case the EventSource should then also report that it outputs DL1_IMAGE information, and not DL0 since the waveforms are no longer there.
Is it worth supporting that? Not sure - do we have a lot of Simtel DST data where we don't already have the raw data?

Is it worth supporting that? Not sure - do we have a lot of Simtel DST data where we don't already have the raw data?

Have? No. I don't think so. But I think for data storage reasons, @ParsonsRD uses it in the ImPACT training workflow.

Closing for now. Will reopen if need arises and we have a solution.


Fixed warning message in log.

WARNING (MainThread) [homeassistant.loader] '1.4a' is not a valid version for custom integration 'prusa_connect'. Please report this to the maintainer of 'prusa_connect'

Thanks for fix, semas to work on clear installation. I need to change my versioning scheme....


M3-1995: switch volume columns

Switch "Region" and "Attached to" columns for Volumes

Coverage remained the same at 32.169% when pulling 54bfbdb522c688001a221945b13c85d34d8c7c05 on rulikrulit:feature/M3-1995-volume-columns into c82ec89a1f5eb0ccc54f391e525effd55a35663d on linode:develop.


Longint

Added a-loption into printf to handle long numbers and a helper function to make that happened.

LGTM!


Add Navbar section

I want to add a navbar which is very much needed for a portfolio.Please assign me this issue

Please see #17 , we will add navbar after all the sections are ready


Training on another big datasets

Have you plans training this network on ImageNet, COCO, or another datasets? Can model demonstrate good results in common objects photo (nature, city) after that?

I am sorry not to train this model with a natural image dataset like COCO. According to the paper results and our faces datasets experiments, it should also have a not bad performance.

I'm trying learn on COCO in next week

Waiting for your good news.

32x32 -> 256x256, dataset was prepared from COCO with sliding window, PSNR ~21, inference on 256x256 crops and merge it back


Make pronto able to run as a micro server for the GitHub integration

The idea would be to have commands like rackup (with a config.ru file) or pronto server or pronto watch. This micro server would listen to Github webhooks when PR events happen, and analyze the PR.
This would allow people to easily have their own free version of https://houndci.com by deploying it to a free Heroku dyno or any other server. This also has the advantage of being independent from a CI server and it does not require too much effort to give it a spin.
I do not know if this is in the scope of the pronto project though. This small 60-lines Sinatra server I wrote basically just wraps pronto, so I was thinking it would not be too much effort to integrate it in this project. The server works quite well, it is on a free Heroky dyno and we use it every day at my job.
If this is not in the scope of this project and these kinds of servers should be independent, please close this enhancement request. If it is, I could maybe find some time to make a PR (although I do not guarantee anything).

Thanks for creating this issue and keeping it open.
I was disappointed to discover that CircleCI doesn't very well support running Pronto inline because it doesn't allow you to wait until a PR is opened to begin a build against a branch on your origin (cf. here, here, and here.
Looking forward to playing with your Sinatra script.

For reference, there's also this Pronto-as-a-Service project by @doomspork: https://github.com/wombatsecurity/nag

I'm gonna close this PR for now, since there does not seem to be much demand for it.
Also, my main use case for using a Pronto-as-a-Service is moot, since these days I am using pronto directly with my CI server (Codeship) thanks to the trick explained in this blog post.
As explained in the post, Codeship doesn't provide a pull request environment variable, but you can use git ls-remote to query remote PR branches made by Github, and do a bit of extraction with sed:
export PRONTO_PULL_REQUEST_ID="$(git ls-remote -q origin pull\*\head | grep $CI_COMMIT_ID | sed 's/.*refs\/pull\/\([0-9]*\)\/head/\1/g')"