RuntimeWarning: invalid value encountered in intersection on M1 Mac

Expected behavior and actual behavior.
intersection unexpectedly raises a RuntimeWarning.
This script:
import shapely
a = shapely.from_wkt('POLYGON ((110 110, 110 100, 100 100, 100 110, 110 110))')
b = shapely.from_wkt('LINESTRING (100 -100, 100 100)')
print(a.intersection(b))

should print POINT (100 100) without warnings. This works as expected on Intel macs. On M1 macs, the result is correct but I get a warning:
/Users/runner/micromamba/envs/env/lib/python3.12/site-packages/shapely/set_operations.py:131: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
POINT (100 100)

Steps to reproduce the problem.
Full example here. The test runs on Github Actions on macos-13 (Intel) and macos-14 (M1) (https://github.com/emlys/demos/actions/runs/8913541967/job/24479230749).
Operating system
Mac OSX 14.4.1
Shapely version and provenance
2.0.4 from conda-forge

We have had several of such reports in the past (eg https://github.com/shapely/shapely/issues/1709, https://github.com/shapely/shapely/issues/1345 for some context). Most of them have been fixed in the meantime, and also typically involved some special case such as empty geometries as input or as result.
Now, for this specific example, I can't reproduce this on Linux (Ubuntu), also using shapely 2.0.4 from conda-forge (using GEOS 3.12.1).

I can reproduce that on M1.

Hmm, then we will need someone with a (M1) Mac to debug this ..
In the past, what I did to debug such warnings is building GEOS locally with the following snippet added somewhere:
#include <fenv.h>

feenableexcept(FE_INVALID);

(although now I am writing this, I am wondering if that would also work if you add that to ufuncs.c in shapely itself with a released build from GEOS, which would be easier).
Now, that might be something gcc specific and not work with clang: https://stackoverflow.com/questions/37819235/how-do-you-enable-floating-point-exceptions-for-clang-in-os-x

I did try to include the code above in shapely but it won't compile. And stuff that comes via Google search or that SO link is way beyond my understanding. The warning is reproducible in GitHub Actions but I understand that it is not the fastest way of development...

Hmm... I've had some unit tests failing in another project since a few last week for macos with small differences in results for union,... operations. Apparently it is since macos-latest switched to macos14 on arm64 that they started failing, so it seems there are more differences that can be expected...

Any progress on this? I'm running into using geopandas to select features based on intersection characteristics, and I see this error even if I first filter based on GeoSeries.intersects to be sure I am only looking at intersections of geometries that actually intersect.


Propose new CI build solution

Motivation
We currently perform builds locally and upload them to Dockerhub, where the CI pulls from to run system tests. The same process is used for running testnets. This has never been a problem thanks to Go's cross compilation - developers were able to create builds for all platforms, regardless of the platform they're developing on.
Now that we start using CGO in order to support GPU-PoST and later SVM the build process becomes more involved, with dependencies that are not trivial to keep in sync on different platforms. Building binaries for different platforms than the platform they're being built on is also a challenge.
High level solution
The "industry standard" is to create builds remotely as part of the CI process. We should consider all the different use-cases that are in use at Spacemesh and design a process that will support them. The high-level idea is to spin up machines running different platforms and create each platform's binary on a development machine running that platform. We should consider how we make the artifacts available to the next steps in the CI flow, how artifacts from other repositories (in other languages) are referenced and pulled, etc.

@noamnelke still relevant?


#1 Docker compose up does not run properly

#Issue 1: Docker compose up does not run properly
Error:
Preparing metadata (pyproject.toml) ... error error: subprocess-exited-with-error when running docker compose up --build.
Fix:

Read through Dockerfile, find duplicated codes and make modification (commenting out the codes that is not needed). In specific:

There were two "Set enviornment variable" sections that were duplicated. Commentted out:
 Set environment variable to ensure all python commands run inside the virtual environment ENV PATH="/.venv/bin:$PATH" \ PYTHONUNBUFFERED=1 \ PYTHONFAULTHANDLER=1 \ QR_CODE_DIR=/myapp/qr_codes

Duplicated Update system and specifically upgrade libc-bin to the required security patch version RUN apt-get update && apt-get install -y --no-install-recommends \ gcc \ libpq-dev \ && apt-get install -y libc-bin=2.36-9+deb12u7 \ && apt-get clean \ && rm -rf /var/lib/apt/lists/*

Duplicated WORKDIR /myapp

Check other codes that are duplicated

Issue resolved.

Issue resolved and merged to main with branch #1.


[BUG] After copying a numbered list from the view mode, the list numbering is reset

Has this bug been raised before?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Description
There was a formatting issue when pasting a numbered list after copying from Preview mode.
Steps to Reproduce

Create numbered list
Set view mode
Copy list
Insert into notes, for example

Environment

OS: all
Browser: all

Screenshots

Do you want to work on this issue?
No
If "yes" to above, please explain how you would technically implement this
No response

Can you assign this to me?

@mutant138 let's go!
assigned

Thank you ! Issue will complete it soon !


Standard lock format for python deps

makePythonPypiEnvironment requires providing a .yaml file with a custom format
a custom format limits the use of SCA scanning tools and may require more maintenance as people request missing features compared to popular package managers
this issue proposes switching to poetry's own poetry.lock

Addressed in https://github.com/fluidattacks/makes/issues/1171


SPARK-12619 Combine small files in a hadoop directory into single split

When a directory contains too many (small) files, whole spark cluster will be exhausted scheduling tasks created for each file. Custom input format can handle that but if you're using hive metastore, it could hardly be an option.

Test build #48661 has started for PR 10572 at commit 055f613.

Test build #48661 has finished for PR 10572 at commit 055f613.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class SimpleCombiner<K, V> implements InputFormat<K, V>
 public static class InputSplits implements InputSplit, Configurable

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48661/
Test FAILed.

Merged build finished. Test FAILed.

Test build #48804 has started for PR 10572 at commit e056332.

Test build #48804 has finished for PR 10572 at commit e056332.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class SimpleCombiner<K, V> implements InputFormat<K, V>
 public static class InputSplits implements InputSplit, Configurable

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48804/
Test FAILed.

Maybe we might have to correct the title just like the others, [SPARK-XXXX][SQL] (this is described in https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark).

This is fixed in 2.0, could you close this PR?

@davies do you have the commit that fixes this in 2.0 ?

Is that https://github.com/apache/spark/pull/12095?

@HyukjinKwon
To merge small files, should I tune spark.sql.files.maxPartitionBytes? But IIUC it only works for FileSourceScanExec. So when I select from hive table, it doesn't work.

@jinxing64 Yup but I think I intended to say most cases within Spark datasources are covered by it. Empty files could be skipped by spark.hadoopRDD.ignoreEmptySplits and probably most other cases could be covered by InputFormat control though.


[Loc] Small addition to sql.xlf

This PR fixes #

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.681%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
26709

Relevant Lines:
58054

💛  - Coveralls


Cannot read property 'ref' of undefined in sliding-pages.html

I am seeing this exception when I try to swipe vertically on my screen containing sliding-pages
Uncaught TypeError: Cannot read property 'ref' of undefined
    at HTMLElement._alterClasses (sliding-pages.html:206)
    at HTMLElement._handleUp (sliding-pages.html:156)
    at HTMLElement._handleTrack (sliding-pages.html:101)
    at HTMLElement.SlidingPages.Polymer.Gestures.addListener.e (sliding-pages.html:77)
    at Object._fire (gestures.html:535)
    at Object._fire (gestures.html:764)
    at Object.touchend (gestures.html:750)
    at HTMLElement._handleNative (gestures.html:333)

Error occurs at sliding-pages.html line 206
_alterClasses(page, transition) {
        page.ref.style.transform = '';

Any help would be appreciated.

this is probably the same bug causing #1


Add typescriptreact support

When I try to use any refactor command, I get the next error:
E5108: Error executing lua: ...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:14: The coroutine failed with this message: .../packer/start/refactoring.nvim/lua/refactoring/query.lua:23: No treesitter parser found. Install one using :TSInstall <language>
stack traceback:
        [C]: in function 'error'
        ...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:14: in function 'callback_or_next'
        ...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:40: in function 'step'
        ...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:43: in function 'execute'
        ...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:113: in function <...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:112>
        ...start/refactoring.nvim/lua/refactoring/pipeline/init.lua:70: in function 'run'
        .../start/refactoring.nvim/lua/refactoring/refactor/119.lua:117: in function <.../start/refactoring.nvim/lua/refactoring/refactor/119.lua:105>
        ...k/packer/start/refactoring.nvim/lua/refactoring/init.lua:29: in function 'refactor'
        /.config/nvim/plugin/mappings.lua:193: in function </.config/nvim/plugin/mappings.lua:193>

File type is "typescriptreact". Probably that is the reason, why it can't get language from TS. But TS is working fine, also TS Playground shows all types, without problem. Is something wrong with function that get languages from TS in query.lua:23 or it is my setup fault?
I've manually switched language from "typescriptreact" to "typescript", and try to extract a variable. It showed "119: What is the var name >" input, but when I enter variable name and press enter, I get the following error:
E5108: Error executing lua: ...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:14: The coroutine failed with this message: .../start/refactoring.nvim/lua/refactoring/refactor/119.lua:77: statements is nil! Something went wrong
stack traceback:
        [C]: in function 'error'
        ...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:14: in function 'callback_or_next'
        ...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:40: in function 'on_confirm'
        ...ar/neovim/HEAD-39a35dd/share/nvim/runtime/lua/vim/ui.lua:68: in function 'input'
        ...ker/start/refactoring.nvim/lua/refactoring/get_input.lua:8: in function 'returned_function'
        ...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:26: in function 'callback_or_next'
        ...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:40: in function 'step'
        ...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:43: in function 'execute'
        ...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:113: in function <...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:112>
        ...start/refactoring.nvim/lua/refactoring/pipeline/init.lua:70: in function 'run'
        .../start/refactoring.nvim/lua/refactoring/refactor/119.lua:117: in function <.../start/refactoring.nvim/lua/refactoring/refactor/119.lua:105>
        ...k/packer/start/refactoring.nvim/lua/refactoring/init.lua:29: in function 'refactor'
        /.config/nvim/plugin/mappings.lua:193: in function </.config/nvim/plugin/mappings.lua:193>

@fragov, thanks for opening an issue. Like you said, the plugin does not currently know how to handle typescriptreact filetypes. Can you attach the exact code you tried to refactor when you switched the filetype to typescript as well as what you visually highlighted? That way, I can try to reproduce it and get back to you.

import React from 'react';

export function SomeComponent() {
const b = 1;
const c = 2;
const a = b * c + b * c;

return <>{a}</>;
}

I try to extract b * c, and then I get that error.

Thanks, I'll take a look and see what I can do :+1:

@fragov this seems to be an issue with the treesitter. If you open a tsx file, it will automatically register the filetype as typescriptreact, which means the treesitter parser used is still typescriptreact. Setting the filetype manually will not immediately change the treesitter parser used, which is what causes this problem. Until the plugin gains support for the typescriptreact filetype, a temporary workaround would be to put an autocmd in your init.vim/lua, something like this:
autocmd FileType typescriptreact set ft=typescript

This will make it use the right treesitter parser off the bat, and enable refactor operations. It is worth noting that this might mess with the LSP, so in the end it is your choice if you want to do this.

I'm not sure this will work without a full typescriptreact setup as it looks like typescriptreact has a different extract function signature with export at the beginning of the function, so it's not the same as normal typescript extract. I think ideally we have to setup a new extra path similar to typescript, but with slight differences for for typescriptreact.

Did anyone have a chance to look into adding support for typescriptreact? I don't want to use an autocommand to set  the file type to typescript because I don't want to break the LSP.

Ran into the same issue today. Does anyone have a workaround?

Also running into this, if there is a setup config change that could fix this, it would be appreciated.

IME, setting the filetype to typescript doesn't help much. Refactorings throw errors like refactoring.nvim/lua/refacto ring/pipeline/init.lua:61: Scope is nil
I'm interested in investigating adding language support here but not really sure where to start. Is there any documentation or a solid PR one could link me to that illustrates what's involved?
I suspect I'd need to implement lua/refactoring/treesitter/langs/typescriptreact.lua and (perhaps) lua/refactoring/code_generation/langs/typescriptreact.lua but any guidance would be appreciated. 🙏

Hi @semanticart, we don't have any documentation right now on adding a new language (perhaps we should add some), but I'd be happy to give some guidance on how you can get started if you'd like to contribute.

You'd also need to add typescriptreact to the init.lua file of the refactoring/config directory.
Then, coming up with treesitter queries for various different cases. You'll find this documentation very helpful in understanding the basics of treesitter queries. I highly suggest using Neovim's Treesitter Playground and its query editor for this; it'll make life a lot easier. You'd need to follow the structure of the treesitters for other languages and implement these queries in lua/refactoring/treesitter/langs/typescriptreact.lua, as you said, then add it to the treesitter init.lua file.
We would also definitely need lua/refactoring/code_generation/langs/typescriptreact.lua, which is how the plugin knows what a constant looks like or what a function declaration looks like for this filetype. Same thing, you'd need to add it to the code generation init.lua file for it to work. I suppose you could use a lot of TypeScript's code generation by importing it (see codegen for filetype c).
Finally, and probably the bulk of the work: tests for the existing functionality. Don't forget to add typescriptreact to minimal.vim, or else it won't know to install the correct parser during the testing process.

The latest language added was Ruby, so I suggest taking a look at #326 to see a practical implementation of this list of steps. If you have any questions, you can ask in the #refactoring channel in Prime's Discord server. Hope this helps in getting started :)

@pranavrao145 Thanks for the thorough reply! I'll try to take a look over the holidays. If I follow through here, I'll also try to build on your comments to document the process for future contributors as a separate PR.

Hi everyone! I found this neovim plugin: https://github.com/napmn/react-extract.nvim which extracts JSX stuff. I think it might be useful for anyone trying to add TSX/JSX functionality, sadly I don't have the time nor the experience to do it myself.

@alexpetrean80 Thanks for this! It looks like this plugin covers 99% of my use case for the refactoring plugin anyways :)

@alexpetrean80 Thanks for this! It looks like this plugin covers 99% of my use case for the refactoring plugin anyways :)

Same, but I gotta admit I'd love to have a single refactoring plugin for everything I do. Glad I was of help tho :D

Hi, I am using refactoring.nvim version 6d0315d.
I noticed that JavaScript React files (*.jsx) are also returning error.
E5108: Error executing lua ...e/pack/lazy/opt/plenary.nvim/lua/plenary/async/async.lua:18: The coroutine failed with this message: ...factoring.nvim/lua/refactoring/treesitter/treesitter.lua:434: attempt to index local 'parser' (a nil value)
stack traceback:
        [C]: in function 'error'
         ...e/pack/lazy/opt/plenary.nvim/lua/plenary/async/async.lua:18: in function 'callback_or_next'
        ...e/pack/lazy/opt/plenary.nvim/lua/plenary/async/async.lua:45: in function 'step'
        ...e/pack/lazy/opt/plenary.nvim/lua/plenary/async/async.lua:48: in function 'execute'
        ...e/pack/lazy/opt/plenary.nvim/lua/plenary/async/async.lua:118: in function <...e/pack/lazy/opt/plenary.nvim/lua/plenary/async/async.lua:117>
        ...y/opt/refactoring.nvim/lua/refactoring/pipeline/init.lua:77: in function 'print_var'
        [string ":lua"]:1: in main chunk


support eslint

The most popular linting framework for frontend code - seems valuable to support :-)

I poked at this one a bit already @JonathanWThom, but the problem I ran into is that the way you run eslint isn't consistent. Most projects seem to specify a bunch of the config for the tool by creating one or more package.json script entries that runs eslint with additional arguments supplied, and they name them just whatever they feel like.


chore(test): only call handle_timeout when necessary

As measured by running perf on our tests, a big part of why they are slow is that we are calling handle_timeout basically on every iteration of the advance loop in the test. Similar as in production, there is no need to do that. Instead, we only call handle_timeout of a particular component (client, gateway or relay) if they indicate that they have something they are waiting for (as defined by poll_timeout).
Simply doing that makes the tests fail for certain scenarios where we handle IP packets that aren't mean for the tunnel (such as DNS queries or STUN messages for the relay / ICE agent). To fix this, we call handle_timeout whenever encapsulate returns None. This is fairly common across sans-IO systems: When a function that usually 1-to-1 transforms a packet instead handles it internally, it must have changed internal state. To make code-organisation easier, handle_timeout is treated as the "work-horse" of a sans-IO system: It is where all the code goes that needs to perform processing upon multiple conditions.
Making this change drops the runtime of tunnel_test from ~33s to ~12s on my machine (tests compiled with opt-level 2).

Making this change drops the runtime of tunnel_test from ~33s to ~12s on my machine (tests compiled with opt-level 2).

About 8 of the remaining 12 seconds are spent in tracing: Checking spans etc, writing the actual files is surprisingly fast. I am not sure we can optimise that away. 12s are a pretty good improvement for now, at least we know that the test-suite themselves isn't unnecessarily slow.


[Bug]: Launching/attaching Flutter through DAP fails with more than 1 device

Describe the bug
More than one device connected; please specify a device with the '-d <deviceId>' flag, or use '-d all' to act on all devices.
macOS (desktop) • macos  • darwin-arm64   • macOS 11.4 20F71 darwin-arm
Chrome (web)    • chrome • web-javascript • Google Chrome 92.0.4515.131
Exited (1)

To Reproduce

Ensure 1+ devices are connected
lua require('dap').continue()

Expected behavior
Ideally a devices window should popup; or otherwise we should be able to specify extra flags in the config to use a specific device.
System information

OS: macOS Big Sur 11.4
Flutter version: 2.2.2 stable
Is flutter in $PATH: yes
neovim version: 0.5

Thanks for developing this great plugin, it really helped me to fully switch my workflow to neovim. I look forward to improving my lua skills enough to someday contribute!

@rlch thanks for raising this. I was actually aware of this as I've hit it myself. Currently I don't really know how to fix it tbh. I need to interact with dap in some way to have it return the error to me programatically so I can then show a picker and then pass the selected id back to dap somehow.
Not really sure exactly how to do this and also not really got the time to do it currently.

@akinsho No problem :)
I'll try have a look into it when I get some time!

This should be working with the dap-based runner introduced in https://github.com/akinsho/flutter-tools.nvim/pull/108


Configure DiagnosticsProfile on VM creation

Currently, it's not possible to setup DiagnosticsProfile when using Azure.virtualMachines().define() method.

@jkubrynski
DiagnosticsProfile can be enabled by adding an extension for the same to the virtual machine. Refer this https://github.com/Azure/azure-sdk-for-java/pull/1160/files unit test to see how to define this extension for a linux virtual machine.
It looks like it it make sense to have a convenience layer to enable this for a virtual machine, we'll add that if it is feasible.
The unit test adds the extension as a part of VM update (Update). We can do that during VM creation time as well (Define), we need to find the id of the to be created VM to do this. The format of the id is: /subscriptions/{subscription_id}/resourceGroups/{resourceGroup_name}/providers/Microsoft.Compute/virtualMachines/{vm_name}

Thanks @anuchandy - this extension works like a charm. But as I understand it's not the same as BootDiagnostics which could be set in DiagnosticProfile?

@jkubrynski the wither to enable boot diagnostics is not implemented, created a work item to track this https://github.com/Azure/azure-sdk-for-java/issues/1163
Seems boot diagnostics cannot be enabled via extensions, only runtime diagnostics can be configured through extensions.

This support is added and will be available as part of 1.2 release.

Feature is available as of v1.2 hence closing this.


[Bug]  'devicePixelRatio' must be provided after installing main branch

Steps to reproduce the bug
flutter --version
Flutter 3.7.0-17.0.pre.24 • channel master • https://github.com/flutter/flutter.git
Framework • revision 957781a108 (58 minutes ago) • 2023-01-11 04:04:39 -0500
Engine • revision 6256f05db1
Tools • Dart 3.0.0 (build 3.0.0-109.0.dev) • DevTools 2.20.0

im using pluto_grid ver: 5.4.9

When compiling on windows
../../AppData/Local/Pub/Cache/hosted/pub.dev/pluto_grid-5.4.9/lib/src/widgets/pluto_scrollbar.dart(482,39): error G036AE10F: Required named parameter 'devicePixelRatio' must be provided. [C:\Users\souku\MM\multi_modal_cs\build\windows\flutter\flutter_assemble.vcxproj]
../…/widgets/pluto_scrollbar.dart:1
../../AppData/Local/Pub/Cache/hosted/pub.dev/pluto_grid-5.4.9/lib/src/widgets/pluto_scrollbar.dart(500,39): error G036AE10F: Required named parameter 'devicePixelRatio' must be provided. [C:\Users\souku\MM\multi_modal_cs\build\windows\flutter\flutter_assemble.vcxproj]
../…/widgets/pluto_scrollbar.dart:1
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.CppCommon.targets(247,5): error MSB8066: Custom build for 'C:\Users\souku\MM\multi_modal_cs\build\windows\CMakeFiles\c6da59db4288df8274ad09a95dd22609\flutter_windows.dll.rule;C:\Users\souku\MM\multi_modal_cs\build\windows\CMakeFiles\1549ab607f84496e655c13ebce2c5ccf\flutter_assemble.rule' exited with code 1. [C:\Users\souku\MM\multi_modal_cs\build\windows\flutter\flutter_assemble.vcxproj]
Exception: Build process failed.

it is basic code from tutorial:
class ScreenList extends StatefulWidget {
  @override
  State<StatefulWidget> createState() {
    return ScreenListState();
  }
}

class ScreenListState extends State<ScreenList> {
  final GlobalKey<FormState> _formKey = GlobalKey<FormState>();

  @override
  Widget build(BuildContext context) {
    return PlutoGrid(
        columns: columns,
        rows: rows,
        onChanged: (PlutoGridOnChangedEvent event) {
          print(event);
        },
        onLoaded: (PlutoGridOnLoadedEvent event) {
          print(event);
        });
  }
}

not including here ROW and COL definition
before i switched to main branch of flutter my code worked fine.
PlutoGrid version
PlutoGrid version is 5.4.9
OS
Windows 10

Currently, the latest version of PlutoGrid operates on Flutter stable 3.3 version.
Behavior in master or beta versions is not yet supported.
We plan to support that version when it becomes stable.


watcher_id is the ful hash not a portion of it as expected.

process_id                            watcher_id
9760  4038e9ab-3d51-461c-ac1c-8c0e7ce49b21
9744  7830f19a-0ef9-440d-ac27-bd1c3a8963d9

@faical-yannick-congo, I'm closing this as I think that it isn't an issue anymore. The watcher ID is just 12 characters right now.


HDDS-9499. Migrate tests in ozone-s3gateway to JUnit5

What changes were proposed in this pull request?
Migration of Junit tests in ozone-s3gateway to JUnit5
Please describe your PR in detail:
Migration of Junit tests in ozone-s3gateway to JUnit5 and Some refactoring to remove the try-catch/expected and replace it with assertThrows for the following files:
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/protocolPB/TestGrpcOmTransport.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/TestEmptyContentTypeFilter.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/TestOzoneClientProducer.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/TestS3GatewayAuditLog.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/TestSignedChunksInputStream.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/TestVirtualHostStyleFilter.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/commontypes/TestObjectKeyNameAdapter.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestAbortMultipartUpload.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestBucketAcl.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestBucketDelete.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestBucketHead.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestBucketList.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestBucketPut.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestBucketResponse.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestCompleteMultipartUploadRequestUnmarshaller.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestEndpointBase.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestInitiateMultipartUpload.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestListParts.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestMultiDeleteRequestUnmarshaller.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestMultipartUploadComplete.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestMultipartUploadWithCopy.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestObjectDelete.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestObjectEndpoint.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestObjectGet.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestObjectHead.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestObjectMultiDelete.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestObjectPut.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestPartUpload.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestPartUploadWithStream.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestPermissionCheck.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestRootList.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestUploadWithStream.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/exception/TestOS3Exceptions.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/metrics/TestS3GatewayMetrics.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/signature/TestAuthorizationV2HeaderParser.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/signature/TestAuthorizationV4HeaderParser.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/signature/TestAuthorizationV4QueryParser.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/signature/TestStringToSignProducer.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/util/TestContinueToken.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/util/TestRFC1123Util.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/util/TestRangeHeaderParserUtil.java
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/util/TestS3Utils.java
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9499
How was this patch tested?
All of the test cases are checked manually before adding it in commits.

I converted the '@Test(expected=XXX)' to assertThrows and all the applicable try-catch to assertThrows.
In some case I see the exception handling is done but the catch block is not part of expected flow and hence in such cases it is kept as it is.

Thanks @raju-balpande for updating the patch.
There are a few JUnit4 assertions left in s3gateway:
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/metrics/TestS3GatewayMetrics.java
59:import static org.junit.Assert.assertEquals;
60:import static org.junit.Assert.assertTrue;
61:import static org.junit.Assert.fail;

hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/TestAuthorizationFilter.java
50:import static org.junit.Assert.fail;
53:import org.junit.Assert;

hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/exception/TestOS3Exceptions.java
21:import static org.junit.Assert.assertEquals;

If the PR replaced these, too, then you could also add the <allow.junit4>false property to ban the old version.

Thanks @raju-balpande for updating the patch.  There are some checkstyle failures:
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestBucketHead.java
 66: Line is longer than 80 characters (found 98).
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/metrics/TestS3GatewayMetrics.java
 160: Line is longer than 80 characters (found 97).
 210: Line is longer than 80 characters (found 95).

@raju-balpande thank you for the patch this is looking good to me. (Minor advice that you can do to make reviewing your changes easier is to describe what your commit does in the commit message instead of repeating the same message for all commits)

Thanks @raju-balpande for the patch, @Galsza, @hemantk-12 for the review.


Logger changes. Update error code spreadsheet.

The logging system has been updated. Now -s (--silent) can be used to disable all elevator messages. Also, the enable and disable flags now work. Updated the spreadsheet and assigned the corresponding error code numbers throughout the code. Fixes to issues #10, #12

Merging all branches into master


Wrong depending on same version detection

Depending on the same crate twice using two different, but similar, versions gives me an error about depending on the same version. Changing it to depend on a more different version fixes it.
Example:

[dependencies]
glium =  "0.25"
imgui =  "0.1.0"
imgui-glium-renderer = "0.1.0"
imgui-winit-support = "0.1.0"
imgui_copy = { version = "0.1.0-rc.1", package = "imgui" }

Output:
error: the crate test-imgui v0.0.2 (C:/_)depends on crateimgui v0.1.0 multiple times with different names
Changing one to version "0.0.X" works.

I think this is to be expected.  The semver requirement ^0.1.0-rc.1 will resolve to 0.1.0, and you can't have two of the same version.  You can use =0.1.0-rc.1 to force the use of a pre-release, but that still won't work in your situation because it is still considered semver-compatible to 0.1.0 and it won't let you do that.

I'm going to close this as somewhat expected, and a duplicate of #6016.  Pre-releases don't quite work very well now (see #2222 which covers some of what is wrong).


HAWQ-23. Fixed building gpfdist error

Please enable test case "errortbl" in install check good. It was disabled when gpfdist was disabled.

Others are good to me.

+1


Game crash at end of song

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Install Game
Add a songpack
Start game
Press start > create profile > select song > speed 550 and Reverse
Start song and complete
song ends, game crash

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

Windows 10
crashinfo.txt

64bit

Additional context
Add any other context about the problem here.

duplicate #603


Add response-policy to the dns::view

Could you please add response-policy to the dns::view defined type, so it can be called from view.pp or else, simillar option like additional_directives and additional_options.
As an example
view "example" {
  match-clients { <IP_ADDRESS>/24; };
  response-policy {
    zone "rpz";
  };
  zone "rpz" {
    type master;
    file "/var/cache/bind/zones/db.rpz";
  };
};

Thank you.

A PR for this is welcome.


Review eq and hash in Ref

Refs are used as keys in many dictionaries and sets and both behavior and performance are important.
Currently mutations of internal functions are allowed in refs and manager although not inherently safe.
Refs __eq__ and __req__ return an BinOpRef. Not clear if this can break expectations in collections since hash and eq are linked.
__hash__ of  Ref ultimately rely on the id of the root object. It is probably not needed since refs define a formal relation.
A review is required at some point...

Obsolete


Secrets error to warn

update to<EMAIL_ADDRESS>adjust secrets warning message
secrets rule disabled for now
adjust no-loop error message

Let's also note that secrets rule is disabled.


Introduce pipeline templating for centralizing changes

https://github.com/MAFIN-Squad/workflows/pull/1


Optimized sgemv_t for small N based on AVX512

This patch is to optimized sgemv_t for N=1-8, as original algorithm would divied M and N no matter how big M and N is. This patch is to customize kernel for individual N size to get  much better performance with vectorization .
The performance improvement is up to 8.16x per the verification on Cascade server

Unfortunately there appears to be a problem with the sgemv_t microkernel for SkylakeX that you introduced with this PR, causing about 1060 failures in the LAPACK testsuite. Apologies for not catching this earlier, and I wonder if you could recheck your contribution ? Running make lapack-test after a successful build should be sufficient to demonstrate the issue.


Clean up and enhance TreeMap

I've started work on a replacement at https://github.com/apasel422/tree.

For what it's worth, I'd be happy to move my crate to contain-rs if it's deemed worthy, now or in the future.

For now we should still migrate the old design, yeah?

Sure. One problem is that my crate is using tree on crates.io, so I don't know how we'll publish it. (I probably shouldn't be using that identifier anyway.) It'd be nice if we could have a contain-rs namespace of some kind.

CC @alexcrichton on namespaces

Well we can leave it here for now and when yours gets big and strong migrate it.

I can migrate the collect::tree to a bst crate under contain-rs if you approve that name.

Works for me.


Update zh_cn.json

fix one error

"commands.tpa.request": "Tpa request from %s",
"commands.tpa.info": "/tpaccept /tpyes or /tpdeny /tpno",
"commands.tpa.error": "You can not request your self ",
"commands.tpa.youdenied":"youdenied",
"commands.tpa.gotdenied":"gotdenied",
"commands.tpa.nonetodeny":"nonetodeny",
"commands.tpa.youaccepted": "youaccepted",
"commands.tpa.gotaccepted": "gotaccepted",
"commands.tpa.notonline": "notonline",
"commands.tpa.nonetoaccept":"nonetoaccept"
can you make the translation for the tpa command please

oh?I didn't check that you updated the text, no problem, wait a minute.

I finished

Thanks


skinmesh example is a black screen on windows

I get the following error message on windows when trying to run the skinmesh example:
PS C:\Users\fredr\GitHub\Ambient\guest\rust\examples\basics> ambient run .\skinmesh\
[2023-06-14T07:22:18Z INFO  ambient] Building project "project"
[2023-06-14T07:22:19Z INFO  ambient] Using bundled certificate and key
[2023-06-14T07:22:19Z INFO  ambient::server] Created server, running at <IP_ADDRESS>:9000
[2023-06-14T07:22:19Z INFO  ambient::server] Starting server
[2023-06-14T07:22:19Z INFO  ambient_network::native::server] Connecting to proxy server
[2023-06-14T07:22:20Z INFO  ambient_network::native::client] Handling client connection
[2023-06-14T07:22:20Z INFO  ambient_network::native::client] Opening control stream
[2023-06-14T07:22:20Z INFO  ambient_network::native::server] server;
[2023-06-14T07:22:20Z INFO  ambient_network::native::client] Opened control stream
[2023-06-14T07:22:20Z INFO  ambient_network::native::client] Attempting to connect using "user_5zMreeuvDxnm4JkF3Nmgnm"
[2023-06-14T07:22:20Z INFO  ambient_network::native::client] Accepting control stream from server
[2023-06-14T07:22:20Z INFO  ambient_network::native::server] Waiting for connect request
[2023-06-14T07:22:20Z WARN  wgpu_core::device] Requested size 1424x714 is outside of the supported range: Extent3d { width: 800, height: 600, depth_or_array_layers: 1 }..=Extent3d { width: 800, height: 600, depth_or_array_layers: 1 }
[2023-06-14T07:22:20Z INFO  ambient_network::proto::server] User connected
[2023-06-14T07:22:20Z INFO  ambient_network::proto::server] Player connected user_id="user_5zMreeuvDxnm4JkF3Nmgnm" id=EntityId(N4UUujkQSEvbpkf9OtLhMQ,<PHONE_NUMBER>6267735842917170129690461495)
[2023-06-14T07:22:20Z INFO  ambient_network::native::client] Entering client loop
[2023-06-14T07:22:20Z INFO  ambient_network::native::client] Waiting for server to accept connection and send server info
[2023-06-14T07:22:20Z INFO  ambient_network::native::client] Accepting diff stream
[2023-06-14T07:22:20Z INFO  ambient_network::native::client] Client connected
[2023-06-14T07:22:20Z WARN  ambient_wasm::client] Failed to load client bytecode from url: Failed to download http://<IP_ADDRESS>:8999/content/client/client_skinmesh.wasm

    Caused by:
        0: error sending request for url (http://<IP_ADDRESS>:8999/content/client/client_skinmesh.wasm): error trying to connect: tcp connect error: The requested address is not valid in its context. (os error 10049)
        1: error trying to connect: tcp connect error: The requested address is not valid in its context. (os error 10049)
        2: tcp connect error: The requested address is not valid in its context. (os error 10049)
        3: The requested address is not valid in its context. (os error 10049)
[2023-06-14T07:22:20Z INFO  ambient_audio::stream] Audio stream config: StreamConfig { channels: 2, sample_rate: SampleRate(48000), buffer_size: Default }
[2023-06-14T07:22:20Z INFO  ambient_network::native::server] Allocating proxy endpoint
[2023-06-14T07:22:20Z INFO  ambient_network::native::server] Proxy sees this server as <IP_ADDRESS>:9000
[2023-06-14T07:22:20Z INFO  ambient_network::native::server] Proxy allocated an endpoint, use `ambient join eu.proxy.ambient.run:9003` to join
[2023-06-14T07:22:20Z WARN  quinn_udp] sendmsg error: Os { code: 10040, kind: Uncategorized, message: "A message sent on a datagram socket was larger than the internal message buffer or some other network limit, or the buffer used to receive a datagram into was smaller than the datagram itself." }, Transmit: { destination: <IP_ADDRESS>:7000, src_ip: None, enc: None, len: 1452, segment_size: None }

There does not seem to be a log for the http interface starting up at <IP_ADDRESS>:8999.
Additionally, perhaps the <IP_ADDRESS> ip addres is not bound due to e.g IPv6 or other networking configuration.
Previously we attempted to resolve the primary network interface address using local-ip-address but we have since removed the dependency and instead use the ip address the server socket was bound to, which is with the current configuration <IP_ADDRESS> which isn't a valid connect ip, only a valid listening ip address as it listens for all network cards, but is to unspecific when connecting as it can mean any one of the network interfaces

I'm going to make connecting default to <IP_ADDRESS> when there's not address specified (or it's <IP_ADDRESS>). For other cases (including IPv6) we can rely on address being explicitly provided.


Add for me and deploy me buttons

Fork me is a github button
Deploy me is a heroku button
Try to find a button for github pages

I'd also add options for well-known hosting services if possible:

https://vercel.com/
https://www.netlify.com/

Looks a bit marketing-y to me.

Looks a bit marketing-y to me.

What do you mean by "marketing-y"? They're too specific deployment services?

Yes, I'd say GH Pages and Heroku are the "go-to" vendors for this use case but these are less known.

Gotcha. By the way, they're not less known services. In fact, Next.js recommends using their own platform for deployments.

As a compromise: https://github.com/c-hive/dotdev/commit/7ce17f1ffb662fadd4234b9048832d1c45d1aa04
I'd consider this done.


Review the SDS

SDS page is available here

please review rapidly

Nice work, some comments:
Deployment diagram: this is a place where you can describe the connection with the surrounding wordpress system
I think you should investigate the database issue some more


Bug 1985997: Enable startup-monitor for SNO

Enable startup-monitor static pod for SNO topology as described in https://github.com/openshift/enhancements/pull/833.
This also leaves unsupportedConfigOverrides.startupMonitor in place for testing and development in HA cluster.

/hold
for e2e test.

/retest

/close
in favour of https://github.com/openshift/cluster-kube-apiserver-operator/pull/1198


Avoid recreating the legend view

Thanks for getting this in shape. I found a couple jquery scope issues where multiple filters would inadvertently get updated (fixed in 5aa184c). If that looks good to you, we can merge.
This is still a tangle, though, and we should rewrite to be model-driven asap.


[Schema Registry] Fix Avro Serializer README links

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


How to check ERRORLEVEL

Description
Dear SCT experts,
I have noticed that bash functions (.sh) in the SCT 3.2.1 return in every cases variable error level ($? or $ERRORLEVEL) with value equals to 0. In previous versions of SCT this variable had various values depending if function ended with or without error.
In my automated bash pipeline where I have many functions one after another I used value in $ERRORLEVEL to stop my pipeline in cases when $ERRORLEVEL was not equal to 0.
How is possible to check error level now?
Thanks, Jan

A regression had been introduced when computing total processing time in the launcher, the script return code was discarded in the process. Fix available for pull.

Should be fixed by #1902 ; @valosekj you may use the master branch until the next release. Please confirm resolution & close issue, and thanks for the report!


InvalidLicense: License check failed: Unable to communicate with service.

Hey All,
I have recently registered on Coursera
I have also received a product key to use graphlab. But I am facing below error while using graphlab. Can you please help ?
I have set the product key in jupyter notebook and getting below error while running graphlab commands.
InvalidLicenseTraceback (most recent call last)
 in ()
----> 1 graphlab.set_runtime_config('GRAPHLAB_DEFAULT_NUM_PYLAMBDA_WORKERS', 4)
/opt/conda/lib/python2.7/site-packages/graphlab/util/init.pyc in set_runtime_config(name, value)
474     """
475     from ..connect import main as _glconnect
--> 476     unity = _glconnect.get_unity()
477     ret = unity.set_global(name, value)
478     if ret != "":
/opt/conda/lib/python2.7/site-packages/graphlab/connect/main.pyc in get_unity()
158     """
159     if not is_connected():
--> 160         launch()
161     assert is_connected(), ENGINE_START_ERROR_MESSAGE
162     return UNITY_GLOBAL_PROXY
/opt/conda/lib/python2.7/site-packages/graphlab/connect/main.pyc in launch(server_addr, server_bin, server_log, auth_token, server_public_key)
89         if server:
90             server.try_stop()
---> 91         raise e
92     server.set_log_progress(True)
93     # start the client
InvalidLicense: License check failed: Unable to communicate with service. Verify Internet connection and try again or visit https://turi.com/support for support options.

I tried installing this on my another laptop which is on MacOS and it worked fine.


Stop non-existent channels causing Infinite loops on upload_video and create_playlist

A Google account can lack a public-facing YouTube channel. Though such an account can still authenticate with YouTube scopes without error, video uploads and playlist/playlistItem creation/deletion fail. (However, interestingly, subscriptions succeed, though are not publicly shown.)
When a channel doesn't exist, deleting a playlist will fail with a RequestError, with reason "channelNotFound". However, in the same circumstances, account#upload_video and account#create_playlist enter infinite token-refresh loops because the API is returning an Net::HTTPUnauthorized error that YT treats as token expiry.
This pull request adds the channel#is_linked? attribute that allows a channel's existence to be tested. The PR also uses this attribute to prevent the infinite loops by changing the Unauthorized errors into RequestErrors with "channelNotFound" reasons, matching the behaviour of delete_playlist.
I haven't added tests yet, both because I'm not sure how to have the automated tests operate on a user without a channel, and because I don't want to waste time writing them if my approach isn't accepted.

Hi there,
My name is Derek and I'm now one of the maintainers of the Yt gem. I want to both thank you for your contribution and apologize for the lack of feedback on your pull request. Maintaining an open source project like Yt is a pretty big job, and we're taking steps to prioritize maintenance and development of the project. We're hoping to review community contributions on a regular basis, as well as provide a roadmap for our development plans.
As part of this process we're going through the open PRs and taking action on each one, and for this PR we've chosen to close it. Unfortunately due to the sheer number of PRs and amount of time that's passed, we can't address your PR individually. If you'd like our thoughts, feel free to comment on your PR, and we'll try and get back to you in a timely manner. In lieu of that, some of the reasons we've closed PRs are:

a lack of tests. We strive for comprehensive coverage; this is important to us
the passage of time: things that were relevant when the PR was opened might not be as important anymore
feature need. Since adding features requires us to maintain them, we want to add features that many people will use

Thanks again for your contribution, and please let us know if you have any questions or comments.


Web Manifest Density is a float, not a string

As per https://w3c.github.io/manifest/#density-member
R: @PaulKinlan @petele

There are a lot of confusion out there regarding density string.
I'm trying to fix these. See https://github.com/GoogleChrome/multi-device/pull/21 for instance.


WIP: Operation pewetomy

Supercedes #2161. Remove all dependence on Pew, and replace all functionality with our own implementation.
Most are quite straightforward (but still needs testing), only new and shell requires particular attention. new is converted to use virtualenv directly, with a little post-creation patching to make it compatible with Pew (mostly the project association stuff).
shell is the coolest part. I rewrote all shell detection as pipenv.shelltools (except the NT backend is 99% _win_utils from the previous implementation), and re-implemented pew.workon_cmd. I’m not that familiar with the compat shell, and largely did copy-pasting job in that part, so more testing are probably required to avoid regression.
We need to manually test the hell out of this before it can land in a stable release.

Moved part of this to https://github.com/pypa/pipenv/projects/3. Purge can be done later.


remove outdated css / update expression editor toolbar styling

General cleanup of unused css
replaced old "searchtoolbar" class with PF toolbar styling in expression editor

https://www.pivotaltracker.com/n/projects/1613907/stories/129462395

@miq-bot add_label ui, technical debt

@miq-bot assign @dclarizio


Pods not available

[!] Unable to find a specification for Serrata

Hi! @Neeraj204 .
Sorry for the late reply.
Please try the following on the command line and try.
It may be broken.
sudo rm -fr ~/.cocoapods/repos/master
pod setup


Add batching and retries to insertion of host announcements

background threads are bad. We want to avoid them if at all possible. I'd rather perform the insert inside ProcessConsensusChange, e.g.:
announcements = append(announcements, ann)
if len(announcements) >= batchThreshold {
    insertAnnouncements(announcements)
    announcements = announcements[:0]
}

Also, does it makes sense to retry at all? Why would an insertion fail initially, but succeed later? IMO a failure to write to the DB should result in an immediate shutdown.


[AIRFLOW-1387] Add unicode string prefix

Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-1387] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-1387

Description

[x] Here are some details about my PR, including screenshots of any UI changes:
Python's format function has a feature of encoding the supplied
variable with the same encoding as a string where substitution will
take place. In case the string is not originally specified as unicode
default encoding will be used. This will yield an error if
sys.getdefaultencoding() is not 'utf-8' because it will try to encode
previously utf8 decoded string (with unicode chars) as non unicode.
Solution based on SO 5082452.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
Does not add any unit tests as it affects function's side effects (not sure how to test it). I tested it on default vagrant ubuntu 16.04 machine using the task which was crashing before and it worked.

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #2426 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2426      +/-   ##
==========================================
+ Coverage   69.24%   69.25%   +<.01%
==========================================
  Files         146      146
  Lines       11230    11230
==========================================
+ Hits         7776     7777       +1
+ Misses       3454     3453       -1

Impacted Files
Coverage Δ

airflow/task_runner/base_task_runner.py
72.41% <100%> (ø)
:arrow_up:

airflow/models.py
87.31% <0%> (+0.04%)
: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 0fc4504...83ed0dd. Read the comment docs.


Create new example using Kaggle's Titanic dataset

Description

We love hosting examples of how Kedro is used and this activity is focused on creating an example using Kedro with Kaggle's Titanic dataset
The output of this task is a PR with a Kedro project with a completed ML pipeline for the Kaggle's Titanic dataset

Context

These examples help so many users understand how to use Kedro
We really do care about expanding our example range

Possible Implementation

Use the latest version of Kedro indicated in Releases
Here's a link to the dataset: https://www.kaggle.com/c/titanic/data
Here's a link to refactored modular code that you could use: https://github.com/FilippoBovo/production-data-science/blob/master/tutorial/d-refactor/titanic/models.py
We also have found a user who's done a similar exercise using Kedro here: https://github.com/tatamiya/kedro_titanic

Possible Alternatives

You could decide to use another dataset if you want

I am happy to work on this one!

Thank you very much for mentioning my repository https://github.com/tatamiya/kedro_titanic.
I'd like to contribute to this issue, too!

I forked the repository and created a new branch.
Are you considering to make two example sub-directories under the titanic-example directory like the link below?
https://github.com/tatamiya/kedro-examples/tree/titanic-example/kedro-tutorial

Hi @tatamiya! Thank you so much for working on this! I'd actually suggest putting the example here: https://github.com/tatamiya/kedro-examples/tree/titanic-example/kedro-exercises
And then just create the PR and we're ready to commit your amazing work.

An update on this ticket, we'd actually love for this to be turned into a Kedro Starter, see more details here: https://kedro.readthedocs.io/en/latest/02_get_started/06_starters.html

I'm going to close this ticket, as we no longer require a Titanic example to show Kedro users. You can also check out this video to see an example Titanic dataset Kedro example explained.


[GWCA] EncodeSkillTemplate missing character

this becomes [1] but should become [2]
OgkTY57g5B6sdQOAAgMAAAAA
OgkTY57g5B6sdQOAAgMAAAAAAA (generated by game for the same build)

shows as

but loads fine

fixed


Add CLI to demo

This add a simple CLI for the demo. Can be extended in the future.

clap derive is awesome! I have been working with the author a bit, very knowledgeable and dedicated.  The structopt has merged into clap, so its a nobrained i guess :)

I'm a little bit of a divergent opinion on the derive API. I basically found two issues while using it which make it a not so pleasant experience:

There are runtime errors which theoretically could be caught during compile time. This goes slightly against Rusts idea: If it compiles it works!
The derive API is not that nicely documented imo. I'm having a hard time to add for example a description for a subcommand.

Agree on the docs - might be good to spend some time helping the Clap author with it. WRT runtime errors - is there an issue for that? I would love to start discussing it, and hopefully address it too. I like Clap too much not to help it :)

I haven't looked deeper into it :D I though of it as expected behavior so i did not report it.
I just have a much better experience with the non-derive API.
Documentation is abolutely the first thing to improve I would say :D
Last time I was bruteforcing derive attributes :D


Compare with InternImage on semantic segmentation task?

https://github.com/OpenGVLab/InternImage

Hello, thank you for your question. Currently, the semantic segmentation performance of InternViT-6B is approximately at a level comparable to InternImage-H. Because the model is quite large, we used a resolution of 504x504 in the experiments with InternViT-6B, without scaling up to 896x896. Additionally, InternImage's pre-training includes supervised signal for mask image modeling, which is beneficial for semantic segmentation, whereas InternVL does not incorporate this feature.

Hello, thank you for your question.
Currently, the semantic segmentation performance of InternViT-6B (UperNet + 504x504 = 58.9 mIoU) is approximately comparable to InternImage-H (UperNet + 896x896 = 59.9 mIoU). Because the model is quite large, we used a resolution of 504x504 in the experiments with InternViT-6B, without scaling up to 896x896.
Additionally, InternImage's pre-training includes supervised signal for mask image modeling, which is beneficial for semantic segmentation, whereas InternVL does not incorporate this feature.

ok, thank you!
@czczup


Add ability to override image+name for check group.

Please reopen it with more detailed information.
What does "image" and "name" refer to?


XamlParseException with no explanation for a UserControl

Describe the bug
After converting my UWP app to WinUI Desktop, one of the controls fails to load its XAML at runtime throwing a XamlParseException with no context or explanation.
Steps to reproduce the bug

Steps to reproduce the behavior:

Build a UWP app with WinUI 2.x and validate it works
Do all necessary steps to convert this to WinUI Desktop and see a vague XAML parse exception

Expected behavior
The control should load and not break for exception. Also, the exception should provide better insight about what could cause the problem.
Screenshots

Version Info

NuGet package version: Microsoft.WinUI 3.0.0-preview4.210210.4

Windows app type:

UWP
Win32

YES

Windows 10 version
Saw the problem?

Insider Build (21318)
YES

October 2020 Update (19042)

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
YES

Xbox

Surface Hub

IoT

Additional context

NavigationToolbarXAML.txt
stack.txt

Can you provide a repro app for us to see the issue in?

@StephenLPeters Sure, here's a repro app: https://github.com/duke7553/files-preview

@StephenLPeters The stack trace of this bug looks similar to what's been reported here https://github.com/microsoft/microsoft-ui-xaml/issues/4235 . I'm not sure if they're the same.

@alwu-msft If you run into an issue with App.xaml while compiling this repro app, you should add/remove a single space to trick the XAML compiler into accepting the changes in that file.
This is because of a separate bug that was reported.

@duke7553 files-preview/Files.Extensions/Files.Extensions.vcxproj includes files from ..\..\files-community\Files\Files.Extensions\ that don't exist, which prevents it from successfully compiling. Is that intentional? The relative path indicates that files are being pulled from outside of the repository.

@alwu-msft Sorry about that. This is an early version of the WinUI desktop version of the project. I expect the Files.Extensions c++ project to be unneeded at this point for getting the controls to load as we don't depend on it anywhere. However, there is a C++ project included with the repository which means it could be referenced incorrectly. I was able to compile and run the package project on my end, but the XamlParseException was blocking us.

Fixed in Reunion v0.5. Closing this now


add 'onblocked' handler

The current 'onblocked' handler just throws, where it is also possible to close the db, which will allow the potentially blocked operation to proceed.

Many thanks for finding and reporting the issue. I have made the required adjustments to fix the problem.


Update basic function names for Kindex update

Depends on https://github.com/HigherOrderCO/HVM/pull/249

Closes  #572

I'm going to change the target of this PR to a separate branch based on the issue just to fix somethings.


Improve WebSocket handshake validation (fixed)

Fixed #5327 to the changes to HTTP::Server API from #5776

@RX14 Any idea about the broken pipe error in the specs? I can't reproduce it locally.

It's a linux thing so you'll have to reproduce it on linux

I know 😁
It seems the error doesn't reproduce when this spec is run alone. And it looks very much similar to the error in #5958, probably related to HTTP::Server#close...

It is indeed the same as the error discovered in #5958. For some reason on Linux there seems to be some strange behaviour when HTTP::Server#close closes the socket servers.
For now I'll disable closing the server in the specs, then it should be all green. It's certainly not optimal, but the error with HTTP::Server needs to be fixed independently from this.


[Feature request] More integration with windows context menu

Add optional integration with windows context menu (e.g. in 7zip), when right clicking on files. It will be look like that:

Menu explanation
Encrypt - encrypt selected file;
Decrypt -  decrypt selected file;
Encrypt and replace - encrypt selected file and replace it. This will be work like this:

you right-click on some new.txt file;
click this menu item;
typing password for encryption;
new.txt file will be deleted;
instead of new.txt file will be a new, encrypted file, in the same place - new.cnp;

Feature was added. Will be available in the next release.


pull base_fee from horizon, update text to pull from there too

@vcarl @stellar/nebula

Preview is available here:http://laboratory-pr460.previews.kube001.services.stellar-ops.com/

Preview is available here:http://laboratory-pr460.previews.kube001.services.stellar-ops.com/

Also, @ire-and-curses IIRC there was some discussion about changing the fee_stats endpoint – can you confirm that this solution is future-proof?

Yeah this is fine. The changes have already happened. They were around the accepted_* names, which are now renamed max_*.

Preview is available here:http://laboratory-pr460.previews.kube001.services.stellar-ops.com/


webpack: add aliases via application config

Note: adding this method to inject Webpack aliases doesn't cancel out https://github.com/inveniosoftware/pywebpack/pull/27, which can still be useful for generally adding aliases.

Tests seem to be failing because Flask-WebpackExt docs are not properly built?

@slint we. had that issue several times during the sprint. read the docs are built in py27 in many cases. If you change it to python 3 they built again properly.


the burst algorithm

What would you like to be added?
When is the code implementation of the burst algorithm expected to be released?
// TODO
BurstPortraitType PortraitType = "Burst"
Why is this needed?
“Algorithm which detects abnormal traffic or potential capacity risks, and suggests a safe replica count proactively.”
I originally thought the burst algorithm mentioned in the project introduction was already supported.

Currently no ETA, will update here if we have one.


The offical page of C++ set up guide is missing

Hi, I want to build MXNet 1.5.1 with cpp bindings on Windows.
But I found that the previous official guide ( https://mxnet.incubator.apache.org/get_started/cpp_setup ) is invalid. How can I get the instruction in detail, which prefers VS2017 and CUDA8/9.

It was merged with https://mxnet.incubator.apache.org/get_started/build_from_source

Thanks for reply.
It does lose something. I remember the previous guide emphasize to change the version of MSVC toolset in VS2017. Maybe there are more setting issues when building specific MXNet version but I don't remember details.

@terrencewayne you can find the old page at https://github.com/apache/incubator-mxnet/blob/24d88a2cdec3e0ab8f4fe0e436eb0015e9ccfd47/docs/static_site/src/pages/get_started/c_plus_plus.md
But note that it does not contain any reference to the MSVC toolchain etc. I think the merged page accurately contains the contain from the C++ page. It's merged for consistency (none of the other language bindings has their extra install page; it's not needed) and maintainability (previously information was duplicated among multiple guides).
For the MSVC setup, I think you refer to https://github.com/apache/incubator-mxnet/blob/24d88a2cdec3e0ab8f4fe0e436eb0015e9ccfd47/docs/static_site/src/pages/get_started/windows_setup.md
Previously we had separate guides for every operating system, which was not really maintainable.
Would you be able to try follow the new guide and port any gaps from the old guide? Specifically, you're totally right that https://github.com/apache/incubator-mxnet/blob/24d88a2cdec3e0ab8f4fe0e436eb0015e9ccfd47/docs/static_site/src/pages/get_started/windows_setup.md#build-from-source contains more lower-level instructions. Would you like to help improve the new guide?
Another issue is that we may want to keep the guides for separate versions separate. The current guide is mainly written for the upcoming MXNet 2 which requires VS2019. But the website doesn't enable users to retrieve old guides at this point in time. One idea to fix this is to make use of the
https://mxnet.apache.org/get_started?platform=windows&language=python&processor=gpu&environ=build-from-source&
which currently states "Please follow the build from source instructions linked above."
Instead, it could display a version-specific build from source guide based on which version the user has selected in the "MXNet Version" selector.
Would you like to help with this? This can be achieved by editing the files in https://github.com/apache/incubator-mxnet/tree/master/docs/static_site/src/_includes/get_started and specifying which files should be shown on the website in https://github.com/apache/incubator-mxnet/blob/cb54a4a99463b23b8abaa2629661954c4ba3c60b/docs/static_site/src/_includes/get_started/get_started.html#L326-L399

Thanks for help! I got the detail instruction from (https://github.com/apache/incubator-mxnet/blob/24d88a2cdec3e0ab8f4fe0e436eb0015e9ccfd47/docs/static_site/src/pages/get_started/windows_setup.md)

Thanks @terrencewayne. Please post here if you run into any problems


LEARNER-8637 - Remove Travis from the repo

Description
LEARNER-8637
Delete .travis.yml
Notes
How to test this PR

@colinbrash The PR looks fine but can you please update the commit messages to conventional messages? and it would be awesome if you can also update the branch name to the naming that we are following colin/LEARNER-8637.

Will open a new PR with the correct branch name


So how do I use this thing?

I have DateTime data which comes with extra fields indicating the time zone in offsets from UTC and whether daylight saving is on. How do I construct a timezone-aware DateTime-like object?

The day of reckoning has come. Guess I better get this package in shape. Honestly I haven't touched it since I moved it out of the Dates module last summer. I was going to work on https://github.com/JuliaLang/julia/pull/8718 first, but I guess I can get this up and running beforehand. I don't think there's a ton of work I need to do, but let me dig into it over the next few days.

+1
I have to handle a large high frequency data project and I can be a good beta tester of the implementation of DataTime + TimeZones.

The danger of writing code is that someone might actually try to use it. ;-)

@quinnj – would be great to have some docs for this; I'm finding myself need it. Let me know if there's anything I can do to help.

Alright, alright, I'll do it :) Thankfully most of the dirty work is done and it's mainly designing the interface. You should at least be able to do
Pkg.clone("https://github.com/quinnj/TimeZones.jl")
Pkg.build("TimeZones")

as of now. I'll open up another issue where peeps can chime in on desired interface (we're starting from scratch here, so I'd appreciate any input).

Any progress on this issue? It'd be great to be able to use this package. I am working on a project that badly needs timezoned datetime and if it does not work yet in Julia, I need to fall back to R.
I just tried the Warsaw example and it is broken:
warsaw = TimeZone("Europe/Warsaw")
ZonedDateTime(DateTime(2014,1,1), warsaw)
LoadError: Unknown timezone Europe/Warsaw
while loading In[60], in expression starting on line 1
in call at /Users/szf/.julia/v0.4/TimeZones/src/TimeZones.jl:33
Any suggestion is welcome as to how to get it running.

What's the output of running
Pkg.clone("https://github.com/quinnj/TimeZones.jl")
Pkg.build("TimeZones")

?

Closest candidates are:
Base.Dates.DelimitedSlot{T}(!Matched::Type{T}, !Matched::Char, ::Int64, !Matched::Union{AbstractString,Regex})
call{T}(::Type{T}, ::Any)
convert{T}(::Type{T}, !Matched::T)
while loading /Users/szf/.julia/v0.4/TimeZones/deps/build.jl, in expression starting on line 43
================================================================================
================================[ BUILD ERRORS ]================================
WARNING: TimeZones had build errors.

packages with build errors remain installed in /Users/szf/.julia/v0.4
build the package(s) and all dependencies with Pkg.build("TimeZones")
build a single package by running its deps/build.jl script

================================================================================

@omus, do we need a fix in TimeZones after https://github.com/JuliaLang/julia/pull/12799 was merged?

And I am using Julia v0.4, but the package is only for the 0.4 version anyway, right?

I believe so yes. Are you using a very recent 0.4 master? Like, compiled within the last week? We just merged some changes to 0.4 master that will make TimeZones a little better to work with and I think it may have broken the package here.

Yes, I downloaded  and installed v0.4 a few days ago.

For completeness:
using TimeZones
gives these warnings
WARNING: module TimeZones should explicitly import == from Base
WARNING: module TimeZones should explicitly import + from Base
WARNING: module TimeZones should explicitly import - from Base
WARNING: module TimeZones should explicitly import + from Base
WARNING: module TimeZones should explicitly import + from Base
WARNING: module TimeZones should explicitly import - from Base
WARNING: module TimeZones should explicitly import - from Base
WARNING: Method definition call(Type{Base.Dates.DateFormat}, AbstractString) in module Dates at dates/io.jl:86 overwritten in module TimeZones at /Users/szalmaf/.julia/v0.4/TimeZones/src/timezones/io.jl:63.
WARNING: Method definition call(Type{Base.Dates.DateFormat}, AbstractString, AbstractString) in module Dates at dates/io.jl:86 overwritten in module TimeZones at /Users/szalmaf/.julia/v0.4/TimeZones/src/timezones/io.jl:63.
WARNING: Method definition parse(AbstractString, Base.Dates.DateFormat) in module Dates at dates/io.jl:148 overwritten in module TimeZones at /Users/szalmaf/.julia/v0.4/TimeZones/src/timezones/io.jl:85.
WARNING: Method definition lastdayofyear(Base.Dates.DateTime) in module Dates at dates/adjusters.jl:42 overwritten in module TimeZones at /Users/szalmaf/.julia/v0.4/TimeZones/src/timezones/adjusters.jl:17.
which is a bit scary.

@quinnj we just need to merge in #5. I was going to double check it and merge it tomorrow. Once merged TimeZones shouldn't have any more warnings about overwritten methods.

@szalmaf if you want to use the package right now you can do Pkg.checkout("TimeZones", "8b5d530") then Pkg.build("TimeZones"). You'll get the warnings about the explicit imports but the scary warnings should be gone.

@omus OK, I just tried the  Pkg.checkout("TimeZones", "parsing") then Pkg.build("TimeZones") and it still has problems building the package:
Pkg.checkout("TimeZones", "parsing")
INFO: Checking out TimeZones parsing...
INFO: Pulling TimeZones latest parsing...
INFO: No packages to install, update or remove
Pkg.build("TimeZones")
INFO: Building TimeZones
INFO: Downloading TZ data
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100 13018  100 13018    0     0   8404      0  0:00:01  0:00:01 --:--:--  8409
100 48014  100 48014    0     0  29272      0  0:00:01  0:00:01 --:--:-- 29258
100 76319  100 76319    0     0  35845      0  0:00:02  0:00:02 --:--:-- 35847
100 72533  100 72533    0     0  31388      0  0:00:02  0:00:02 --:--:-- 31399
100  121k  100  121k    0     0  53250      0  0:00:02  0:00:02 --:--:-- 53254
100  136k  100  136k    0     0  59901      0  0:00:02  0:00:02 --:--:-- 59924
100  140k  100  140k    0     0  55246      0  0:00:02  0:00:02 --:--:-- 55226
INFO: Pre-processing TimeZone data
==============================[ ERROR: TimeZones ]==============================
LoadError: MethodError: convert has no method matching convert(::Type{Base.Dates.DelimitedSlot{T}}, ::Int64, ::Type{Base.Dates.Year}, ::Int64, ::Int64, ::ASCIIString)
This may have arisen from a call to the constructor Base.Dates.DelimitedSlot{T}(...),
since type constructors fall back to convert methods.
Closest candidates are:
Base.Dates.DelimitedSlot{T}(!Matched::Type{T}, !Matched::Char, ::Int64, !Matched::Union{AbstractString,Regex})
call{T}(::Type{T}, ::Any)
convert{T}(::Type{T}, !Matched::T)
while loading /Users/szalmaf/.julia/v0.4/TimeZones/deps/build.jl, in expression starting on line 43
================================================================================
================================[ BUILD ERRORS ]================================
WARNING: TimeZones had build errors.

packages with build errors remain installed in /Users/szalmaf/.julia/v0.4
build the package(s) and all dependencies with Pkg.build("TimeZones")
build a single package by running its deps/build.jl script

================================================================================

@szalmaf can you let me know the output of versioninfo()?

versioninfo()
Julia Version 0.4.0-pre+7105
Commit 349a4e1 (2015-08-30 15:16 UTC)
Platform Info:
System: Darwin (x86_64-apple-darwin13.4.0)
CPU: Intel(R) Core(TM) i7-2760QM CPU @ 2.40GHz
WORD_SIZE: 64
BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY Sandybridge)
LAPACK: libopenblas
LIBM: libopenlibm
LLVM: libLLVM-3.3

@szalmaf I'm having difficulties reproducing the error you are getting. Can you run import TimeZones; TimeZones.Olsen.compile()?

import TimeZones; TimeZones.Olsen.compile()
LoadError: MethodError: convert has no method matching convert(::Type{Base.Dates.DelimitedSlot{T}}, ::Int64, ::Type{Base.Dates.Year}, ::Int64, ::Int64, ::ASCIIString)
This may have arisen from a call to the constructor Base.Dates.DelimitedSlot{T}(...),
since type constructors fall back to convert methods.
Closest candidates are:
Base.Dates.DelimitedSlot{T}(!Matched::Type{T}, !Matched::Char, ::Int64, !Matched::Union{AbstractString,Regex})
call{T}(::Type{T}, ::Any)
convert{T}(::Type{T}, !Matched::T)
while loading In[17], in expression starting on line 1
in call at essentials.jl:58
in call at /Users/szalmaf/.julia/v0.4/TimeZones/src/timezones/io.jl:76
in call at /Users/szalmaf/.julia/v0.4/TimeZones/src/timezones/io.jl:63
in parsedate at /Users/szalmaf/.julia/v0.4/TimeZones/src/timezones/Olsen.jl:108
in zoneparse at /Users/szalmaf/.julia/v0.4/TimeZones/src/timezones/Olsen.jl:205
in anonymous at /Users/szalmaf/.julia/v0.4/TimeZones/src/timezones/Olsen.jl:459
in open at iostream.jl:114
in tzparse at /Users/szalmaf/.julia/v0.4/TimeZones/src/timezones/Olsen.jl:440
in load at /Users/szalmaf/.julia/v0.4/TimeZones/src/timezones/Olsen.jl:483
in compile at /Users/szalmaf/.julia/v0.4/TimeZones/src/timezones/Olsen.jl:490 (repeats 2 times)

Thanks. Ok it appears you are still running the old version of the package. Try running Pkg.checkout("TimeZones", "parsing") which should get you the working version. You can confirm that Julia updated the package via Pkg.status("TimeZones"). This issue will go away once I merge #5.

After doing
Pkg.checkout("TimeZones", "parsing")
the
Pkg.status("TimeZones")
gives

TimeZones                     0.0.1+             parsing

Is this 0.0.1+ the right version? Also, I didn't build it this time. Should I?

@szalmaf that is the right version. You should be able to run Pkg.build("TimeZones") now without running into an exception.

Pkg.build("TimeZones")
still produces this:
INFO: Building TimeZones
INFO: Downloading TZ data
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0    0     0     0        0     0    0   -0- :- - : - -0  - - : - - :0- -   - - : -0- :----: - - : - -0 --:--:-- --:--:--     0curl: (56) Recv failure: Connection reset by peer
curl: (56) Recv failure: Connection reset by peer
curl: (56) Recv failure: Connection reset by peer
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100 13018  100 13018    0     0  10225      0  0:00:01  0:00:01 --:--:-- 10234
100 48014  100 48014    0     0  36157      0  0:00:01  0:00:01 --:--:-- 36182
100 76319  100 76319    0     0  39904      0  0:00:01  0:00:01 --:--:-- 39894
100 72533  100 72533    0     0  36922      0  0:00:01  0:00:01 --:--:-- 36912
100  121k  100  121k    0     0  60072      0  0:00:02  0:00:02 --:--:-- 60085
100  136k  100  136k    0     0  69179      0  0:00:02  0:00:02 --:--:-- 69194
100  140k  100  140k    0     0  57814      0  0:00:02  0:00:02 --:--:-- 57800
INFO: Pre-processing TimeZone data
==============================[ ERROR: TimeZones ]==============================
LoadError: MethodError: convert has no method matching convert(::Type{Base.Dates.DelimitedSlot{T}}, ::Int64, ::Type{Base.Dates.Year}, ::Int64, ::Int64, ::ASCIIString)
This may have arisen from a call to the constructor Base.Dates.DelimitedSlot{T}(...),
since type constructors fall back to convert methods.
Closest candidates are:
Base.Dates.DelimitedSlot{T}(!Matched::Type{T}, !Matched::Char, ::Int64, !Matched::Union{AbstractString,Regex})
call{T}(::Type{T}, ::Any)
convert{T}(::Type{T}, !Matched::T)
while loading /Users/szalmaf/.julia/v0.4/TimeZones/deps/build.jl, in expression starting on line 43
================================================================================
================================[ BUILD ERRORS ]================================
WARNING: TimeZones had build errors.

packages with build errors remain installed in /Users/szalmaf/.julia/v0.4
build the package(s) and all dependencies with Pkg.build("TimeZones")
build a single package by running its deps/build.jl script

================================================================================

I'll just get this new release out. This is strange as the exception you are getting would be from running an older version of the code.

I restarted the 0.4 kernel in Jupyter and built TimeZones again, and it seems like it helped:
Pkg.build("TimeZones")
INFO: Building TimeZones
WARNING: module TimeZones should explicitly import == from Base
WARNING: module TimeZones should explicitly import + from Base
WARNING: module TimeZones should explicitly import - from Base
WARNING: module TimeZones should explicitly import + from Base
WARNING: module TimeZones should explicitly import + from Base
WARNING: module TimeZones should explicitly import - from Base
WARNING: module TimeZones should explicitly import - from Base
INFO: Downloading TZ data
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100 48014  100 48014    0     0  33044      0  0:00:01  0:00:01 --:--:-- 33067
100 13018  100 13018    0     0   8722      0  0:00:01  0:00:01 --:--:--  8725
100 76319  100 76319    0     0  37178      0  0:00:02  0:00:02 --:--:-- 37192
100 72533  100 72533    0     0  33199      0  0:00:02  0:00:02 --:--:-- 33211
100  121k  100  121k    0     0  56517      0  0:00:02  0:00:02 --:--:-- 56528
100  140k  100  140k    0     0  63166      0  0:00:02  0:00:02 --:--:-- 63182
100  136k  100  136k    0     0  57585      0  0:00:02  0:00:02 --:--:-- 57600
INFO: Pre-processing TimeZone data
INFO: Successfully processed TimeZone data

Awesome. Everything should work as intended now. I'll get this fixed today so that others don't run into the same issue.

Great. Thanks for your help.
Just a quick check:
warsaw = TimeZone("Europe/Warsaw")
ZonedDateTime(DateTime(2014,1,1), warsaw)
now outputs
2014-01-01T00:00:00+01:00

Working as intended. Check out the README for a basic usage walkthrough.

Wouldn't it be a good idea to have in core a DateTime constructor signature corresponding to DateTime(2014,1,1, warsaw) (also extended to h, m, s, etc.) instead of writing ZonedDateTime(DateTime(2014,1,1), warsaw)? Just a suggestion.

There are plenty of features to be added including convenience constructors. I've updated the package to work with the Julia v0.4-pre release. The updated version of TimeZones just needs be merged into METADATA. https://github.com/JuliaLang/METADATA.jl/pull/3257

Closing issue as basic usage has been addressed in the README.


Unexpected error fetching when user is offline

Step to reproduce-

User in live
Connecting Tiktok Live then event data is comming
User is offline in the mean while we fetch this issue -

Package Version
My version of TikTokLive is 6.0.3

Closing if stale in 48h


ApplyVocabularyMapping can now handle plans containing filter operators.

Functionality for filter operations in the plan rewrite of ApplyVocabularyMappings. Necessary to get all the tests to run when building the package after uncommenting the lines required for the rewriting package build:
return createQueryOptimizerWithoutOptimization(ctxt); uncommented in HeFQUINEngineConfig.java
heuristics.add( new ApplyVocabularyMappings()); uncommented in LogicalOptimizerImpl.java

Simon, thanks a lot for fixing this problem! I see, however, that your fix is suitable only for cases in which the filter conditions would not be affected by the vocabulary mappings. I guess that this is okay for the queries in your evaluation. Therefore, I am fine accepting this PR as is, but with the understanding that it is only addressing half of the problem. For the rest of the problem, I have created issue #257


Create TRIDIAGONAL.c

I am creating a code, which can solve a system of matrices in tridiagonal form. If it is unclear, what this code does, I will be happy to explain

This repository has chosen to disable pull requests.


use different busy flag to prevent screen fill by importexportbusy

I first took out the setBusy entirely, but that seemed risky.  This flag may be reset by the next dataChanges, but at least we're indicating we're busy...


Imputation of missing values using ML models. (Enhancement and Bug fix opened in #477)

I have implemented the functionality for imputation of missing values using ML models.
Now below additional options are provided in healthcareai.common.transformers.DataFrameImputer class:
imputeStrategy : string, default='MeanMode'
It decides the technique will be used for imputation of missing values.

If imputeStrategy = 'MeanMode'
Imputation is done using Mean/Mode
If imputeStrategy = 'RandomForest'
Imputation is done using ML models.

tunedRandomForest : boolean, default=False
If set to True, RandomForestClassifier/RandomForestRegressor to be used for imputation of missing values are tuned using grid search and K-fold cross validation.
******************************************** Bug Fix ***********************************************
In existing code there was no provision to handle the columns which are of type int/float but by nature they are categorical. < ex JobCode ( Levels : 1,2,3,4,5,6)  >. Therefor this type of column were imputed using Mean value ( ex. 2.8, 3.6 etc) which can be very hazardous.
I handled this problem as well for both imputation strategy i.e 'MeanMedian' and 'RandomForest'.
Now user can use below parameter to explicitly mention such type of columns.
numeric_columns_as_categorical : List of type String, default=None
List of column names which are numeric(int/float) in dataframe, but by nature they are to be considered as categorical.
For example:
There is a column JobCode( Levels : 1,2,3,4,5,6)
If there are missing values in JobCode column, panadas will by default convert this column into type float.

If numeric_columns_as_categorical=None
	Missing values of this column will be imputed by Mean value of JobCode column.
	type of 'JobCode' column will remain float.
If numeric_columns_as_categorical=['JobCode']
	Missing values of this column will be imputed by mode value of JobCode column.
	Also final type of 'JobCode' column will be numpy.object

The existing approach of missing value imputation( using Mean/Mode) is preserved with one fix.

Raised the same pull request from different account. Therefore closing this pull request


Migrate division test from E2E to Integration

What does this PR (Pull Request) do?

[x] This PR is not related to any Issue
Note: Most of the tests were already migrated. There was one remaining and this PR fixes that.

Which Traffic Control components are affected by this PR?

Traffic Portal
CI tests

What is the best way to verify this PR?

Required: Local TO or CIAB TO running
Go to traffic_portal/test/integration
Update specs, apiUrl and baseUrl in config.json as follows:
"specs": [
    "specs/Divisions.spec.js"
  ],

"params": {
    "apiUrl": " https://localhost:9090/api/4.0",
    "baseUrl": "https://localhost:9090/",
....
}

and then run the below two commands in a separate terminal window:
1. For me chrome v90 doesn't work, so, I ran the following cmd: webdriver-manager start --versions.chrome 86.0.4240.22
2. npm test -- --params.baseUrl https://localhost:9090 --capabilities.shardTestFiles true

localhost port may differ for every user
All division integration test cases should pass.
If this is a bug fix, what versions of Traffic Control are affected?

The following criteria are ALL met by this PR

[x] This PR includes tests
[x] I have explained why documentation is unnecessary
[x] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[x] This PR includes any and all required license headers
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Additional Information

There is no logic change to require an update in documentation

Code looks good to me. Same like Cachegroup check CSV test.


Can I use encrypted DLL under the unity Editor

@Liudeke
Hi, I think yes you can, depends on the protection level, if it's smth like BitDotnetNet probably your file is not even gonna reference like at all? But I think you can try.
Why not put that encrypted DLLs in the output folder once you built your Unity Project?

In Unity Project use normal DLL.
Once built replace that DLL in output folder with encrypted one.


Website cached

Describe the bug
This link doesn't work: (link to highlight)
But, the link is correct in the codebase, which means your caching system is doing something
Reproduction
Open:
Open the codebase at the same place
Compare the links
System Info
N/A

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Looks like a SPAM. I've deleted links and closed the issue.


Adding a title to a question set

I hope this dirty schema explains the idea :

As there is a title to the panel, there'd be one on the set.
Or I missed the option in the docs

Hi @bbnnt.
You cannot currently add a title to a question set, but you can add one to a questionPanel and a question. If this is something you think would be worthwhile, it can be done!

Maybe the demo provided confused me; can you have all your panels displayed at the same time ?
That'd solve it too

Panels are not supposed to be displayed all at once no, panels are like pages. Question sets however are all rendered one after another.

That's what I wasn't sure of;
what do you think, worthwhile to add titles to questions sets ?

I imagine so If you wanting this and have a case for it.
Moving forward I can add it in sometime soon or you can PR it if you feel comfortable doing so!

I'd like to look into it; maybe you help point me where it might happen
Have to check the code a bit

@bbnnt Fixed. Information in the README. Will be live in the next release, which I am getting round to very soon.

could not get enough time to go into this;
but thanks a lot for this improvement !


"Follow" links in the (left) sidebar does not show on iPad  with Safari.

Hi,
Thank you very much for creating this awesome theme. Since I don't have any experience with Github, and know only a little bit of html/css, my question might be trivial. But here we go.
I just forked mmistakes/minimal-mistakes and started re-configuring (for an academic homepage) a week ago and noticed that the "Follow" links of the author info on the (left) side-bar does now show up when viewing the page with my iPad pro (with Safari). The screen size is 2224-by-1668. When I use the screen horizontally, the photo and the description below the photo shows up on the left (like in any large screen), but the list of links below them do not show up. If I use the screen in the portrait direction, the "Follow" button shows up, but nothing happens if I push it. The page that I am working on is here:
https://github.com/makotonakajima/makotonakajima.github.io
I thought I screwed up something when I was tweaking parameters, but the same thing happens with, for example, here:
https://mmistakes.github.io/minimal-mistakes/year-archive/
So this is not a problem of the tweaked version of mine. I tried my old iPad with an old (outdated) Safari and screen size of 2048 x 1536, and there is no problem. Since both have a larger screen than the max size of the theme, I suspect it is related to the (most) recent version of Safari. I suspect this is related to how Breakpoints work, together with the size of the screen obtained from the browser, but I don't have any deep knowledge in terms of how this work. Any help would be appreciated.
Thank you again for this great theme.
Makoto

Do you have any content blockers enabled in Safari?

Bingo! I am so ashamed that I didn't think about it. Yes, I had AdGuard with various custom filters, and the problem went away when I turned them off. I didn't think about it because various content blockers of Chrome on my PC did not cause a problem. Thank you so much for a quick help!

No problem.


3/17 Publish

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-autogen/index.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/network/application-gateway/url-path-map.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/vm.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/vm/disk.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/vm/unmanaged-disk.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/vmss.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/vmss/disk.pycliyml
:white_check_mark:Succeeded
View

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


How to filter for a specific asset/account in the get-A2A-password.sh

Hi.
I have a customer that  is trying to credential retrieval via bash for a certificate user that has 48 accounts, and only wants the pw for one assetaccount.
In Powershell, you are able to filter to select the asset/account:
“$myapikey = $myAvailableAccts | where {(($.assetname -eq $mysystem) -and ($.Accountname -eq $myaccount))} | % APIKey”
The customer want sto know how can they handle the same in bash script.
Sorry I am not a bash expert-- hopefully this is an easy question.  We checked the other A2A shell scripts and they all use an API key for the specific account. What is needed is a script or additional commands to request the API key for the specific asset/account, so that API key can be passed to get-A2A-password.sh

You can use the https://github.com/OneIdentity/safeguard-bash/blob/master/src/get-a2a-retrievable-account.sh script if you have A2A registration visible to certificate users:

By default, the get-a2a-retrievable-account.sh script will get all accounts available using that credential, but you should be able to filter the results from there.  We could take an enhancement to allow searching for a specific account.


Test OpenVPN connect times on 2048 bit RSA, consider default variable change from 1024 bit

completed here: https://github.com/robbintt/popup-openvpn/commit/f3bdf04b7799f5d648c0c692738be711fcd65391