Fixed alignment of faded lines when passing faded_line_ratio to :class:.NumberPlane

Overview: What does this pull request change?
Correctly align lines when creating a new NumberPlane object with faded_line_ratio property.
Before

After

Further Information and Comments
I use the code that #2894 provided.
Closes #2894.

Reviewer Checklist

[ ] The PR title is descriptive enough for the changelog, and the PR is labeled correctly
[ ] If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
[ ] If applicable: newly added functions and classes are tested

Thank you so much for this fix @imaphatduc

@behackl Thank you very much for reviewing my PR. Wow, it's really rewarding to be a contributor of a great math visualization library 🎉. Just one question, why the tests failed after I first committed my changes? (the first commit on this PR)

Which commands are needed to run to generate the control data to a binary file?

See our developer guidelines regarding graphical unit tests: https://docs.manim.community/en/stable/contributing/testing.html#graphical-unit-test -- TL;DR: pytest --set_test path/to/test_file.py::test_function_name

And this binary file is compiled from which source file(s)?

The test functions in the files in tests/graphical_unit_tests/test_*.py.

If in the future I want to contribute to ManimCE, what do I need to do before opening a PR? There seems to be no documentation referring these steps.

You did well with this PR, no need to change anything about your approach, really. There are some comprehensive instructions here: https://docs.manim.community/en/stable/contributing/development.html -- but if you have specific questions it is often easier to just ask in the #dev-chat channel of our Discord.

Great insight! Thank you.


docs: :pencil: add schemas to documentation

:tada: This PR is included in version 2.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Test suite and test cases for Azure Performance Diagnostics VM Extension

Description
These changes add a Test Suite for the Azure Performance Diagnostics VM Extension.
The initial test case verify_azure_performance_diagnostics does the following:

Creates a storage account whose name and credentials are later passed in the public and protected settings used to run the VM Extension
Install and run the Azure Performance Diagnostics VM Extension
Delete the VM Extension
Verify installation of VM Extension succeeded
Verify a report was created and uploaded to the storage account
Verify the VM Extension no longer exists on the VM after deletion

@LiliDeng LGTM


[DatePicker] w3c validator issues

Hi,
I'm using your Datepicker component.
Is there any chance to fix this?

[ ] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
When trying to validate site which uses your Datepicker component I got average about 40 errors, which says:
"Element “p” not allowed as child of element “span” in this context. (Suppressing further errors from this subtree.)"
Expected Behavior 🤔
element p should not be inside inline span block.
Steps to Reproduce 🕹
Please see Link to your Datepicker component demo site below:
https://next.material-ui.com/components/date-picker/
Your Environment 🌎

Tech
Version

Material-UI
v4.9.8

Material-UI/pickers
v3.2.10

React
16.11.0

Browser
Chrome Version 84.0.4147.105

TypeScript
3.6.4

Can I take this to do the PR of the solution?

Can I take this to do the PR of the solution?

@vicasas Could you focus on "good to take" issues? The good first issues are meant to help new contributors get started. I think that after 4 good first issues, you should now be able to gain more autonomy and demonstrate that you can add value without relying on a solution shared by a maintainer :)

@vicasas Could you focus on "good to take" issues? The good first issues are meant to help new contributors get started. I think that after 4 good first issues, you should now be able to gain more autonomy and demonstrate that you can add value without relying on a solution shared by a maintainer :)


fix issue #49

Allow multiple symbols to be defined for a single unit, e.g. "um" or "µm" for micrometre, etc.

in the csv file, the unique symbol is now a symbol list (separated by the '|' character)
added alternatives where possible
implemented a few tests to check functionality

it seems the build fails but the message is quite unhelpful...

I have not seen that error message before. Will try to merge anyway. This looks good!


composed decider for StartChildFailed

https://github.com/sclasen/swfsm/blob/master/fsm/deciders.go#L520-L536


before() method not working...

Hi, I'm having trouble getting the before() method to work. Below is my code:
<?php

namespace App\Controllers;

use \Core\View;

class Dashboard extends \Core\Controller
{
    protected function before()
    {
        echo "hello world";
        return false;
    }

    public function homepage()
    {
        // shows homepage
    }
}

When I run this code, the homepage still displays and the "hello world" text never appears. Am I missing something?

Hi,
you need to suffix your action methods names with "Action":
public function homepageAction()
(I haven't added this to the documentation yet as I'm currently rewriting the framework so that you don't have to do this!)

Hi Daveh I’ve followed your course on Udemy and really appreciated your help. Do you know when the new version of the framework will be released? Will it be updated on Udemy and will it be easy to upgrade from the current framework? Thanks for your work

Hi,
you need to suffix your action methods names with "Action":
public function homepageAction()
(I haven't added this to the documentation yet as I'm currently rewriting the framework so that you don't have to do this!)

@daveh I noticed that without this suffix, most everything still works properly. Is this suffix only a requirement for the before & after filters?

@vince844 I don't have a date I'm afraid, but I'm working on it as much as I can! It's an update to both the framework and the Udemy course, and there will be an upgrade guide from the current version to the new one.
@badalyan Yes, the suffix is only needed for the before and after filters to work (it's a consequence of how this works by using the __call method)

@daveh thanks — it worked!


[#card-program-manager] - [May 1] New resource to publish

Publish date: May 1
DRAFT

I'm ready to be assigned!

unassigned myself after chatting this one through!

Ok, I reworked this a bit. There are so few steps to actually upgrading, but a bit of information otherwise. @joekaufmanexpensify I left some comments, if you could take a look! Thanks!

Went through and left some suggestions/comments. LMK what you think!

Ok @ren-jones this is ready for another review!

Not overdue. I think still pending @ren-jones review

Still pending

This is live! 🎉
But @ren-jones, noticing a couple things:

This should be under Expensify Classic rather than New Expensify, since this is only relevant to domains who enabled the card program on Classic

I don't think those two bullet points are supposed to be in that box?

No per-PR payment needed here

This is still in the wrong place w/ bad formatting. @rushatgabhane do you know how to fix this formatting? Then we can delete the article under the New Expensify section and move it to Expensify Classic..

This issue should fix the weird formatting, and then we can move that article to the Expensify Classic section.

checking if I need to pay this or if the process is outdated


Export Image

I often use tikzcd to make images for blog posts (which aren't written in LaTeX). It would be awesome if I could directly save an SVG, PDF, or really anything from the app.

This is a duplicate.


If you cancel a Hold, we need to be able to mark an item as Missing

In Sierra, a user can "Cancel & Mark as Missing".

Steps in Folio:

See steps for cancelling a request then do the following
App: inventory
Change search type as necessary (barcode, title, etc) and enter item identifier
Click on the item to open a new pane
Click on the "holdings" menu and click on the relevant holding
Click on the item record
In the "actions" drop down and select "mark as missing"

@jwfuller this one is gtg!


Add unit tests for model package

Issue: #34
Adding unit tests for model package.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #38 (e193959) into main (7e2c176) will increase coverage by 4.25%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #38      +/-   ##
==========================================
+ Coverage   22.76%   27.02%   +4.25%
==========================================
  Files           4        4
  Lines         470      470
==========================================
+ Hits          107      127      +20
+ Misses        349      330      -19
+ Partials       14       13       -1

Impacted Files
Coverage Δ

pkg/model/types.go
87.50% <100.00%> (+41.66%)
: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 7e2c176...e193959. Read the comment docs.


upload_arduino_target does not work (avrdude problem?)

Hello out there,
I am trying to upload my hex file via SPI - having the latest avrdude version and latest Arduino SDK version - sure this looks more like an avrdude problem ... But is there a way to specify an own configuration file for avrdude?
This works like a charme for me - how could I do this as upload target?
avrdude -c linuxspi -p m328p -P /dev/spidev1.0 -v -U flash:w:controller-arduino.hex
I am getting
-- Arduino SDK version 1.8.8: /opt/arduino/arduino-1.8.8
-- Determined Platform Header: /opt/arduino/arduino-1.8.8/hardware/arduino/avr/cores/arduino/Arduino.h
-- Arduino SDK version 1.8.8: /opt/arduino/arduino-1.8.8
-- Determined Platform Header: /opt/arduino/arduino-1.8.8/hardware/arduino/avr/cores/arduino/Arduino.h
-- Arduino SDK version 1.8.8: /opt/arduino/arduino-1.8.8
-- Determined Platform Header: /opt/arduino/arduino-1.8.8/hardware/arduino/avr/cores/arduino/Arduino.h
-- Configuring done
-- Generating done
-- Build files have been written to: /tmp/tmp.LL6xRi9W6k/build
[ 86%] Built target nano_atmega328_core_lib
[ 93%] Built target SPI
[ 96%] Linking CXX executable controller-arduino.elf
Generating EEP image
Generating HEX image
Calculating controller-arduino size
Firmware Size: [Program: 1922 bytes 5.9% ] [Data: 184 bytes 9.0% ] on atmega328p
EEPROM Size: [Program: 0 bytes 0.0% ] [Data: 0 bytes 0.0% ] on atmega328p

Uploading controller-arduino target

avrdude: Version 6.1-svn-20130917, compiled on Dec 31 2018 at 14:06:08
         Copyright (c) 2000-2005 Brian Dean, http://www.bdmicro.com/
         Copyright (c) 2007-2009 Joerg Wunsch

         System wide configuration file is "/opt/arduino/arduino-1.8.8/hardware/tools/avr/etc/avrdude.conf"
avrdude: syntax error at /opt/arduino/arduino-1.8.8/hardware/tools/avr/etc/avrdude.conf:1095
CMakeFiles/controller-arduino.dir/build.make:86: recipe for target 'controller-arduino.elf' failed
make[2]: *** [controller-arduino.elf] Error 1
make[2]: *** Deleting file 'controller-arduino.elf'
CMakeFiles/Makefile2:73: recipe for target 'CMakeFiles/controller-arduino.dir/all' failed
make[1]: *** [CMakeFiles/controller-arduino.dir/all] Error 2
Makefile:83: recipe for target 'all' failed
make: *** [all] Error 2

cmake_minimum_required(VERSION 3.2)

#set(ARDUINO_SDK_PATH /opt/arduino/arduino-linux-1.8.5)
set(ARDUINO_SDK_PATH /opt/arduino/arduino-1.8.8)

set(CMAKE_TOOLCHAIN_FILE /opt/arduino/Arduino-Toolchain.cmake)
#set(ARDUINO_DEFAULT_PORT /dev/ttyUSB0)  # Default Port, when not specified
#set(ARDUINO_DEFAULT_BOARD pro)          # Default Board ID, when not specified
#
#set(ARDUINO_BOARD_NAME "ProMini3V328")

# fully optimize debug code
set(CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG} -O2")
set(CMAKE_C_FLAGS_DEBUG "${CMAKE_C_FLAGS_DEBUG} -O2")

set(CMAKE_CXX_FLAGS "-std=c++11")

project(controller-arduino C CXX ASM)

#generate_arduino_firmware(controller-arduino
#        SRCS src/main.cpp include/queue.h
#        include/com.h src/com.cpp
#        include/util.h src/util.cpp
#        include/process.h src/process.cpp
#        include/clock.h src/clock.cpp
#        BOARD mega)
#
get_board_id(board_id nano atmega328)
add_arduino_executable(controller-arduino ${board_id} src/main.cpp)
link_platform_library(controller-arduino SPI ${board_id})

upload_arduino_target(controller-arduino "${board_id}" SPI)

Umm I don't know why do you get this error, honestly - Have nothing to say about avrdude as I'm not really familiar with it (nor with the Arduino ecosystem... ironic right?)
However, what I can tell from your attached CMakeLists.txt file is that you're doing some things wrong:

You shouldn't set flags manually like that.
It seems you're migrating code from the old arduino-cmake project, where manually setting flags were an ordinary behavior, but here it's a bit different. Please read this wiki page regarding build flags in this framework.
Your ARDUINO_SDK_PATH shouldn't be set locally in the cmake file, rather at the command-line or configured as an environment variable. You can read about it more here.

At last - Please state your configuration details as requested here.
After doing this we could investigate the issue further.

"Your ARDUINO_SDK_PATH shouldn't be set locally in the cmake file,"
Why is this? Doesn't sound right at all. You can set an ENV anywhere including on the cmake line. As a newbie to this (arduino cmake) and completely incapable of getting it working (upload_arduino_target not defined) I need educating. Thank!

@rileyrg Hi, I'm afraid you're referring to a completely different issue which is not related to this one.
I believe that #77 is the place to talk about it.

In my case I had a problem with avrdude installed from system repository (Ubuntu 18.04), as it was unable to parse configuration included in Arduino SDK:
avrdude: error at /home/user/Apps/arduino-1.8.9/hardware/tools/avr/etc/avrdude.conf:1133: programmer type jtagice3_updi not found

To fix this, I set cmake to use avrdude included in SDK and it uploaded successfully.
set(ARDUINO_CMAKE_AVRDUDE_PROGRAM $ENV{ARDUINO_SDK_PATH}/hardware/tools/avr/bin/avrdude)

@ghost - your's avrdude is older than version provided with sdk (6.1 vs 6.3-20171130) - I suspect that this causes that syntax error.


Support for VS 2022 Preview 1

Hi Oleg,
I love, Love, LOVE your extension! I've been using it for years. Thank you so much for creating this extension!!!
Are you able to add support for VS 2022 Preview 1? It's driving me nuts to have all the tabs display text instead of their icons.
It's such an essential feature that it really should be an option in VS itself. That way there wouldn't be a bit of lag in showing the icons sometimes.
It's no fault of your extension, because I know that you don't the ability to load at the same time as the IDE. It can only load after the IDE is displayed and VS starts to load extensions. Even if it's an async extension there's no autoload guid to load with the IDE, only after it's displayed itself.

Hi @yannduran! Yes, I plan to install VS 2022 preview in a couple of days and figure out what's needed in order to support it.
The problem with icons not showing immediately is actually not related to async loading. In order to show icons, I need to modify WPF component tree, which can be reset to its initial state by Visual Studio at any time. Unfortunately, I have no easy way of intercepting this event, so all I can do is monitor the tree every second and apply modifications when necessary. Hence the visible lag in showing icons.

Hi @yannduran! You can try the new version that supports VS 2022: https://marketplace.visualstudio.com/items?itemName=OlegTarasov.Iconizer2022. Unfortunately, I couldn't figure out a way to make single package for VS 2017-2022.

It's working great Oleg! Thank you very much for doing this so quickly.


[Feature] Modify streamx old style images

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
Replace streampark with streamx old style images
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

finish

finish


Need API doc to test api`s of DMaaS feature

Test cases I am working on :-

TCID-DIR-DMAAS-SCHD-CSTOR-MINIO-BUCKET
TCID-DIR-DMAAS-SCHD-CSTOR-AWS-BUCKET
TCID-DIR-DMAAS-SCHD-CSTOR-GCP-BUCKET

Basic requirement that should be present in API doc

URL where we need to give post request
Request body
Expected response body.
Flow of DMaaS schedule creation should be understood by the API doc.
Any NOTE that should be taken care by Automation Engineer at the time of API testing.

I need your help on this. Can we do this over a call where I can show you how to send requests and meaning of each API/parameter?


Update CONTRIBUTING.md

Improve contributing.md file.

Can I work with this?

Assigned to you! Add good documentation. Contribute well.🚀

Assign this issue to me I can do this.

Go ahead! Contribute well🚀

Can I contribute as well?

Go ahead, @ayushigeorge! Contribute well🚀

Is this issue open?
I would like to work on it.


Add description to composer.json

Description required for published packages.
composer validate output:
./composer.json is valid for simple usage with composer but has
strict errors that make it unable to be published as a package:
See https://getcomposer.org/doc/04-schema.md for details on the schema
description : The property description is required

See this problem in your other package: #58

Fixed in 210ff935adb2ad9bc7668f6f551e131891b76c2c.


User defined property default for array type causes exception

Describe the bug
I'm trying to add a user-defined property of string[] or just string type and use it in one of my registers with its default value.
When I try to access it by using get_property(<udp>) I get 'None'
Details like these can be helpful:
property myUDP    { type = string ; component = reg; default = "STR";};

RDL code:
reg reg1 {
      myUDP;
};
reg1 reg1_instance;

I expect that if no value was assigned to this UDP, I would get "STR" as a result of the method.

I confirm that the compiler throws an exception if a UDP with type = string[] is used. That should be supported and is definitely a bug.
However I do not see any issue with the non-array example you show. I do indeed see that with type = string, the property resolves as expected: reg1_instance->myUDP == "STR"

Fixed in 1.26.1


Support for multiple emails

Many mainstream web apps tend to support multiple email addresses (that is, a primary one plus other emails supported for logging-in) as well as, sometimes, something like a username.
Is this something you'd consider supporting in Sorcery v1 ?
I don't know any authentication lib supporting that, and that'd be a pretty cool feature.
Thanks for the good work!

Great suggestion! My immediate focus for v1 is getting it live with 1:1 support of the existing Sorcery functionality, with the main additions of an improved configuration and better support for community made plugins. I'll keep this in mind though, and if it's simple enough implement while I'm digging through the code. Otherwise, this could be a reasonable addition for v1.1, or a community made plugin.


How can customers create reviews on the product page?

How can customers create reviews on the product page?
Can this demo only create reviews in the app behind Shopify?

By checking Allow unverified review submissions in the embeded app integration within your theme editor.


shattering the grand illusion

In a world of AI art gen, it's nice to see an actual algorithm. It harkens back to, well: a simpler time...

Definitely it is not an OpenAI =)


[*.py] Lint

I'm a little worried that this PR changes too much and thus will never be merged.
The first commit is from:
fd -HIepy -x bash -c 'autoflake --remove-all-unused-imports -i "$0" && isort --atomic "$0" && python -m black "$0"' {} \;

The goal is to have all these lint checks passing:
flake8 . --count --select=$(printf '%s,' {A..Z}) --ignore='W503,E203,E402,E731,E741,C901' --show-source --max-line-length=119 --statistics
pylint flax -rn --load-plugins=pylint.extensions.check_docs --accept-no-param-doc=n

This is related to this PR #933

We would have to review this carefully as it would break the git blame chain, and that would happen every time we change lint configuration going forward

We would have to review this carefully as it would break the git blame chain, and that would happen every time we change lint configuration going forward

@avital Yep, I agree completely. So that's why it's important to get linters setup as early as possible in the project. Introducing these to TensorFlow for example would be completely untenable [without some executive decision: good luck], as there are over 200 open PRs.
In terms of what linters pass currently, all the flake8 tests pass. Just wanted to wait for your feedback before proceeding with updating [almost] all the docstrings.

@avital Yep, I agree completely. So that's why it's important to get linters setup as early as possible in the project. Introducing these to TensorFlow for example would be completely untenable [without some executive decision: good luck], as there are over 200 open PRs.
In terms of what linters pass currently, all the flake8 tests pass. Just wanted to wait for your feedback before proceeding with updating [almost] all the docstrings.

We should probably update the pylint configuration with the style that is used right now.
We need at least 2 space indent instead of 4 space otherwise pretty much every line changes

We should probably update the pylint configuration with the style that is used right now.
We need at least 2 space indent instead of 4 space otherwise pretty much every line changes

@jheek One can just add this line to the GitHub Action:
flake8 . --count --select=$(printf '%s,' {A..Z}) --ignore='W503,E203,E402,E731,E741,C901' --show-source --max-line-length=119 --statistics

In terms of two spaces versus four, I suppose I can modify that easily, but not sure that's good practice. With the exception of Google repositories, most every repo I see is four spaces (not tabs). Anyway I'm happy if a little surprised that you're actually considering merging this that changes so many lines.
Good!

@jheek One can just add this line to the GitHub Action:
flake8 . --count --select=$(printf '%s,' {A..Z}) --ignore='W503,E203,E402,E731,E741,C901' --show-source --max-line-length=119 --statistics

In terms of two spaces versus four, I suppose I can modify that easily, but not sure that's good practice. With the exception of Google repositories, most every repo I see is four spaces (not tabs). Anyway I'm happy if a little surprised that you're actually considering merging this that changes so many lines.
Good!

I think it's best to pick the dominant style of coding as source of truth for the linting rules for now.
That way the issue doesn't become too invasive.
We can always switch some style in the codebase to a different indentation or string quote but that shouldn't be in the linting PR.
Otherwise we are going to mix a discussion about consistency (which we obviously want) with subjective discussions like indentations (which obviously everyone is going to disagree on)

I think it's best to pick the dominant style of coding as source of truth for the linting rules for now.
That way the issue doesn't become too invasive.
We can always switch some style in the codebase to a different indentation or string quote but that shouldn't be in the linting PR.
Otherwise we are going to mix a discussion about consistency (which we obviously want) with subjective discussions like indentations (which obviously everyone is going to disagree on)

Closing this due to inactivity. Also, we are incrementally fixing linting errors in our codebase, but this is quite a difficult process due to how we set up our linting infrastructure, so submitting it all in one PR is unfortunately not feasible.

Darn

Hi @SamuelMarks, I was going over all PRs and issues and realize now I probably have closed this PR too soon.
Apologies! I didn't mean to disregard all your hard work. We are trying to minimize the number of open issues so we can keep maintaining our repo effectively.
I have now read both this PR and the discussion in #841 in more detail.

So that's why it's important to get linters setup as early as possible in the project. Introducing these to TensorFlow for example would be completely untenable [without some executive decision: good luck], as there are over 200 open PRs.

I share this sentiment! One problem is that we are internally using pylint (which is a google product), but the external version is slightly different, so if we decide to enforce it we will run into problems.
The pylint team recommended using something like this: https://github.com/google/styleguide/blob/gh-pages/pylintrc
Anyway I am reopening this issue, sorry again!


Build rework for release. Split examples out into separate project.

Tagging @jbellis for review just to make sure we're on the same page with what's getting pulled into jvector-examples as a separate project. Don't worry about the build parts any more than you want to.

#collaborating my thought was that we'd include SimpleMappedReader (I guess we should include a SimpleMappedSupplier too) and leave the rest of it in example, both to keep the surface area small and to avoid the impression we're trying to provide something production-ready.  Do you think it makes more sense to include the MMapReader too?

structure LGTM


feat: allow clients to pass boolean, Date, and number values

Changes

Create ClientJsonApiQuery and ServerJsonApiQuery types. The former provide better type safety while the latter remain string.
Client-side, support boolean, Date, and number types in addition to string.
Add deepEqual asserts to the examples to quickly verify correctness.
More complete tests
Better 🐶 names

i googgled around for make the server type safe with zod schema. But it involves writing hacks/workarounds in zod to achieve this. not something out of the box

@fasidOnGit, I'd rather use ts-rest query params server-side, which supports this already; example here. For Clipboard, this library is helpful for the client-side. I provided the server-side for completeness and because once I did client-side, it took Claude Sonnet five minutes to generate the inverse for me 😄


Provide indication in the code model what context schema is used in

Input/output
XML/Json
Useful to decide what serializers/deserializers need to be generated.

The body parameter does have the knownMediaType (json/xml/form) that the body is expected to be in.
Input/output is duplicated with #23

The body parameter does have the knownMediaType (json/xml/form) that the body is expected to be in.

Can we add knownMediaType to all the schemas?

You want to know if a given model could ever be asked to be be serialized or deserialized to a specific format?
That's so you don't have to generate serialization code for a given model + format when it's never programmatically possible?
I can see why that'd be useful.
Yes, I've added that to #23.


Dynamic Services?

Can it be possible to make it dynamic?
For example :
.....
  data:
    image: mongo:3.4
    networks:
      - mongo
    volumes:
      - data:/data/db
    command: mongod --shardsvr --replSet datars --smallfiles --port 27017
    deploy:
      mode: replicated
      replicas: 3
      placement:
        constraints:
          - node.labels.mongo.role == data

There is no StatefulSets support in Swarm so you can't get a DNS record per Mongo node like you do in Kubernetes. Even if you would query the tasks records in Swarm DNS to get the containers IP, when you restart a container that IP will change and the cluster will get broken.


Fix sample yaml

Yaml provided as a sample have missing property
and are confusing the users
Fix them to avoid confusions and also used the
consistent naming
Fixes https://github.com/tektoncd/pipeline/issues/755

awesome 👍

/ok-to-test

/test pull-tekton-pipeline-integration-tests

This is great, thanks so much for catching and fixing this @piyush-garg !!! :heart::heart::heart:
/approve
/meow space


Update typing of the browser messaging service

Type

Proposal
Bug fixes

Description
This PR types the callback to the subscribe function properly, and makes sure that the callback will receive the payload associated with the appropriate action from the possible action/payload combinations.
The proper typing of the subscribe function helped find bugs in several subscribers (e.g. they were expecting the action to be a part of payload).
Note: we'll have to be less ambitious with the subscription function, which won't be able to subscribe a listener to an array of actions, but only to a single action at once. Given the way we are using the subscription, this should not be a problem for us.
Demo
See typescript playground demonstrating a minimal working example of the subscribe function that is aware of the association between an action and the payload that is passed to the listener function.

@azangru the typescript playground link seems broken and also the tests are failing. Apart from these two, the PR looks good.

@msmanoj I think I fixed the typescript playground link.
As for the tests, they are broken both in this branch and in  bms-improvement branch. I think we could fix them in the bms-improvement branch once we merge this branch in?


make ovs socket file path as configurable property

The ovs-cni currently hardcodes the ovs unix socket file path to /var/run/openvswitch/db.sock.
But this would change across deployments. Hence introducing a global flat file configuration so that
socket_file  path can be specified and this is applied for all network attachment definition objects.
Signed-off-by: Periyasamy Palanisamy<EMAIL_ADDRESS>

/release-note-none

/cc @JanScheurich

/lgtm
/approve


logging option + version

there it is.. logging option for program start, token error, token accepted, score above 500, and restart notice... logged in log.txt file with date time

the logging of point is still a bit buggy, it keeps writing the score linewhen you hit 500

Mon Mar 28 2016 21:34:43 GMT+0200 (Rom, sommertid) | *** HIGHEST SCORE ABOVE 500 ***
Mon Mar 28 2016 21:34:44 GMT+0200 (Rom, sommertid) | *** HIGHEST SCORE ABOVE 500 ***
Mon Mar 28 2016 21:34:45 GMT+0200 (Rom, sommertid) | *** HIGHEST SCORE ABOVE 500 ***
Mon Mar 28 2016 21:34:46 GMT+0200 (Rom, sommertid) | *** HIGHEST SCORE ABOVE 500 ***

will be fixed unless someone else fixes this in a pull :)

I can't send out a pull because I'm behind :-1: Just gotta wait for Crazy to accept and I can start my work on it :3

you can merge with mine and continue editing and adding to this PR :)

I just figured it out, I accepted your merge, now all I have to do is edit mine and you can merge mine with yours so it get's added :+1:

;-)
Den 28/03/2016 21.57 skrev "Makan Dey"<EMAIL_ADDRESS>
I just figured it out, I accepted your merge, now all I have to do is
edit mine and you can merge mine with yours so it get's added
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

@Salvationdk Made a PR. :+1:

Amazing work guys!! :+1:


Feature/route visibility

Feature should be implemented correctly. Some tests fail because of a missing table "tasks_registration__old", which appears to be unrelated.

Some tests fail because of a missing table "tasks_registration__old", which appears to be unrelated.

This is an incompatibility issue with newer SQLite versions and Django 1.11. Should be fixed when we migrate to 2.2

Your solution is fine.
However, I would suggest moving the code that assigns the human readable text to the route from the view code to a new method in the Review class.
This way, we can re-use that code elsewhere later on.
In addition, we might want to look at using .annotate() (see the Django documentation) on the QS instead. That way we could annotate the proposal object with the latest review object, and call the method I described earlier in the view instead. (And get_queryset could then actually return a QS, which is preferable).
That could be quite hard, so ignore this if you deem it so.

Comments duly noted. I've applied changes, and have added some help-text reflecting the desire to move route info into the Review class in the future. I think it's best to get this one out the door for now, and start applying annotate() on my next pass.

Fine with me!
Merge when you can!
Could you make an issue for moving the code? Before we forget to do it....


Adding trove classifiers.

Adding trove classifiers.

[Current coverage][cc-pull] is 45.87%

Merging [#1547][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master      #1547   diff @@
==========================================
  Files            52         52
  Lines          5790       5790
  Methods           0          0
  Messages          0          0
  Branches       1116       1116
==========================================
  Hits           2656       2656
  Misses         2927       2927
  Partials        207        207

Powered by Codecov. Last updated by [8e206f9...f4865d1][cc-compare]
[cc-base-branch]: https://codecov.io/gh/graphite-project/graphite-web/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/graphite-project/graphite-web/compare/8e206f9ce6a6ef59cdd3bbb879c2c154a1df8259...f4865d1277fcaf3c44a056e244205bbc3ecfdaec
[cc-pull]: https://codecov.io/gh/graphite-project/graphite-web/pull/1547?src=pr

Looks like there are some additional ones we could include. Mind comparing to https://github.com/graphite-project/carbon/commit/e9904725e371974036ea8312231b092715a4b88c as a baseline?

Nice, thanks! 🍰 🌴 🍧


Definition of Severity

#74130
Let me know the definition of "Severity' such as high/medium.
It's unclear in this Azure doc. - How are alerts classified?
Is there any quantitative definition like CVSS?

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

ID: ff54d88b-b294-cbbb-7034-04e1f64a6cbc
Version Independent ID: 48e954b7-b94c-ce9a-baf0-748e42195542
Content: Security alerts and incidents in Azure Security Center
Content Source: articles/security-center/security-center-alerts-overview.md
Service: security-center
GitHub Login: @memildin
Microsoft Alias: memildin

@TakashiTerasawa Thank you for your feedback . We will try to get more information on this and update the thread.

@TakashiTerasawa I do not think the quantitative logic behind how Azure security center exactly classifies the alerts into High/Medium/Low is available publicly as you have in case of CVSS . I do not think exact specification like CVSS can be made public but , let me check internally if there is something like that and I will update the thread.
Thank you.

@TakashiTerasawa  - can you please elaborate what is unclear in the doc?

@TakashiTerasawa As we have not received any response from your side, we will close this issue by EOD today . Should you still have queries on this , please open a new issue referencing this one and let us know the details as to what exactly seem unclear as in the section How are alerts classified? to you as asked by above .


[ODS-5379] Support PS 7.2 and Unix - Script group 10

Note that the next modified scripts don't belong to this group:

create-database-template.psm1
prep-package.ps1
run-smoke-tests.ps1

... I modified them to call GetExeExtension instead of Get-IsWindows since it's a bit more readable.

create-database-package.ps1 is being left alone, since the only places it is referenced are the EdFi.Database.Security and EdFi.Database.Admin builds in TeamCity/Github, and the builds in Github will remain on windows-latest due to the requirement of needing the sql binaries for their database packages to be created.

Is there a ticket to look into changing that in the future? Surely there is an alternate method to get what we need.

create-database-package.ps1 is being left alone, since the only places it is referenced are the EdFi.Database.Security and EdFi.Database.Admin builds in TeamCity/Github, and the builds in Github will remain on windows-latest due to the requirement of needing the sql binaries for their database packages to be created.

Is there a ticket to look into changing that in the future? Surely there is an alternate method to get what we need.

Yes @vimayya and I were just talking about how there is a sqlpackage available now in linux, so when we look at moving the relevant workflows from Windows to Ubuntu, we will see if using that will work so that the workflow does not have to remain on Windows.


ModuleNotFoundError: No module named 'tensorflow.python.saved_model.model_utils

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):NO
OS Platform and Distribution :Linux Ubuntu 16.04
TensorFlow installed from (source or binary):conda install tensorflow-gpu=1.13.1
Python version:3.6

I have installed tf=2.0.0 before on another conda env ,but after uninstalling tf=2.0.0 the whole tensorflow on every conda env has been ruined. Trying different envs/python-version doesn't work at all.
pip uninstall/install tensorflow_estimator in/out of conda envs doesn't work as well

@Charlie4zc
How did you solve it?
I am facing same problem while working with Tensorlfow object detection API.


Customize home page

Hi, how can I customize the home screen?

I want to see only camera photos when start the app:

It's not possible at the moment, but I'd like to implement it.

Actually, why don't you use a shortcut? You can create a shortcut to any filtered collection from the Collection page menu > "Add shortcut", and then launch this shortcut instead of the normal app icon.

I am using the widget to access this app.

Please add an option for widget settings.

Ah, i see. Then opening the app by the widget should open it to the collection configured for that widget. For example if the widget is configured with the "image" and "camera" filters it would open the app to the Collection page with "image" and "camera" filters already there. Is that your expectation?

Yes 🤩


Reimporting a specific invalid Collada will crash the editor

Godot version: v3.1.2.stable.mono.official
OS/device including version: Windows 10 Home 1903
Issue description: When I try to import a specific Collada file into Godot the editor crashes when I try to reimport it. the only way to stop it from crashing is to delete the Collada through the file manager. It gives me this error message in the console: "Skeleton parenthood graph is cyclic"
Steps to reproduce: Create a new project, import the file below, and reimport it.
Minimal reproduction project: ColladaGlitch.zip

@qarmin a couple things could be wrong (as far as Godot is concerned) with your file:
if you have a null/group interrupting a bone hierarchy (bone hierarchy with null/other transform nodes inside that also have bone children thereof... nulls can be leaf nodes but can't have bone children with skinning), this could result in a crash or animations that appear broken.
if you do not have at least ONE skinned object with all the bones in your skeleton hierarchy in the skinning algorythm (even if the bones are zero weighted they need to be listed in at least one of your skinned objects), you will get an error similar to this:
ERROR: Collada::_remove_node: ERROR: Not found node to remove?    At: [your project path here]\editor\collada\collada.cpp:2181
That error will hang and your editor will never fully boot up, if your editor is already booted up and you export a .dae to your project path, this will crash the editor upon trying to load.
But I agree that said "broken" collada files should not crash Godot.

Crash still reproducible in 3.2.4 RC 1.

I get the same error when I tried making some simple meshes children of a new skeleton node on godot 3.3. Is this not common for everyone? The skeleton properties don't show up in the inspector either and the meshes become invisible despite being set to visible.

p.s. don't worry about the camera error, it's intentional. There are 2 players in the scene so there are 2 cyclic errors.

Seems fixed in the current master branch (as of 4.0 alpha 12).

Omg how old was that post?Im 3 companies and 2 engines separated from Godot.On Jul 17, 2022 8:13 AM, Rémi Verschelde @.***> wrote:
Seems fixed in the current master branch (as of 4.0 alpha 12).
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

2 years, which doesn't seem to be a big deal for a rare crash caused by an invalid file :)
It's probably not worth keeping around for 3.x specifically so closing as fixed in 4.0.


Opening multiple windows using react router v6.8.1

Would anyone please show how to implement opening multiple windows?
Under development I can open an about screen on a new electron window using createHashRouter.
if (is.dev && process.env['ELECTRON_RENDERER_URL']) {
childWindow.loadURL(process.env['ELECTRON_RENDERER_URL']);
} else {
childWindow.loadFile(join(__dirname, '../renderer/index.html'));
}
But in production I get a blank screen.
I appreciate any help. Thanks!
-Ari

Can you provide a minimal reproduction repo?

Can you provide a minimal reproduction repo?

Sure thing. thank you!
https://github.com/agf2000/my-vite-react-electron-app.git

https://github.com/alex8088/electron-vite
You should submit your question in this repository, it has nothing to do with me and I resent it


Updating failure due to migration message with link to docs

Let's merge this to master and not the docs/lakefs-kv-migration.

Let's merge this to master and not the docs/lakefs-kv-migration.

Yeah... that would be problematic as it branched from the docs/lakefs-kv-migration branch and contains some of its modifications. I will create a new PR on top of master and close this one

See #4074 instead


Added WinRT concurrency support.

Performance test reports can be found here
Increase performance / reduces execution time for cases when tasks can be effectively paralleled and load is enough for the parallel execution to result into substantial benefit for example for imgproc and features modules. No impact or limited negative impact for cases when tasks are not effectively paralleled or the gain from parallel execution is less that management expenses.

Quite peculiar since this doesn't affect Mac "gcd" concurrency logic. I'll take a look, would appreciate your advice.

@mkostin, don't worry, this test fails sometimes on mac. I've restarted the build.

@mshabunin nice one, thank you.

very nice contribution, thanks! :+1:


az storage blob upload-batch outputs misleading error if --account-key is not attached

(I realize this may be obvious to most, especially as most other commands -h don't outline certain parameters as required even if they are but that doesn't make it a good practice and as a first time user of the CLI it can lead to some unnecessary trial and error, especially when the errors don't actually specify the correct error)
Description
Basically, when running "az storage blob upload-batch -d https://.blob.core.windows.net/test-container -s C:\Users<myuser>\storage-blob-integration-with-cdn-search-hdi\clinical-trials\clinical-
trials-small"
it outputs "The specified resource does not exist."
This is misleading, as the resource does in-fact exist (I created it myself after-all) and it can see it and work with it. After some trial and error and reading through the -h I determined it's probably failing because the command is missing the account key, after adding --account-key and my information, the command is accepted and executes correctly, filling the container with the files. Which I've verified over in the UI just to be sure the command is actually executing.
So I'd suggest adding an additional error message which identifies that the --account-key parameter is missing in the event that it is.

Environment summary
Install Method: How did you install the CLI? (e.g. pip, interactive script, apt-get, Docker, MSI, nightly)
Answer here: I downloaded and ran the windows MSI.
CLI Version: What version of the CLI and modules are installed? (Use az --version)
Answer here:
"azure-cli (2.0.15)
acr (2.0.11)
acs (2.0.14)
appservice (0.1.14)
batch (3.1.2)
billing (0.1.4)
cdn (0.0.7)
cloud (2.0.7)
cognitiveservices (0.1.7)
command-modules-nspkg (2.0.1)
component (2.0.7)
configure (2.0.10)
consumption (0.1.4)
container (0.1.9)
core (2.0.14)
cosmosdb (0.1.12)
dla (0.0.11)
dls (0.0.13)
eventgrid (0.1.3)
feedback (2.0.6)
find (0.2.6)
interactive (0.3.8)
iot (0.1.11)
keyvault (2.0.8)
lab (0.0.10)
monitor (0.0.9)
network (2.0.13)
nspkg (3.0.1)
profile (2.0.11)
rdbms (0.0.6)
redis (0.2.8)
resource (2.0.13)
role (2.0.11)
servicefabric (0.0.2)
sf (1.0.7)
sql (2.0.10)
storage (2.0.13)
vm (2.0.13)
Python (Windows) 3.6.1 (v3.6.1:69c0db5, Mar 21 2017, 17:54:52) [MSC v.1900 32 bit (Intel)]
Python location 'C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\python.exe'
Legal docs and information: aka.ms/AzureCliLegal"
OS Version: What OS and version are you using?
Answer here: Windows 10 Enterprise
Shell Type: What shell are you using? (e.g. bash, cmd.exe, Bash on Windows)
Answer here: powershell

Supposedly the account key is not required if the full qualified share/container name is given. The command is supposed to fetch the account name from the URL and fetch the account key automatically if the CLI is logged in. (Is it logged in here?)
The error message indeed can be improved. Thanks for the advice.

For what it's worth I (thought?) I provided the URL (I omitted parts of it when I put it here, which is why it looks incomplete), I copied it from the primary service endpoint over in the portal. Is that the correct URL? (sorry, again, new user)
If so: it didn't seem like it was able to parse it correctly (and yes it was logged in, I wouldn't imagine the command would work if it wasn't but I haven't tried).

https://.blob.core.windows.net/test-container

This URL on paper doesn't look correct. The account name is missing here.

I'm aware, I omitted my account when I put it here, I wrote in tags but it looks like those were deleted when I posted somehow. On reflection there was no reason to omit it in the first place so I'll post it, I've since deleted that particular storage account but it was something like this:
https://examplestorageacct.blob.core.windows.net/

Thank you. The url looks correct. I will revisit this scenario in a bit. The bottom line is that we can improve the message, but I'm curious what stop the command from fetching the key.

Fixed in https://github.com/Azure/azure-cli/pull/4624.
The account key should be automatically fetched when it is missing in this situation.


WhatsApp business, how to get cart details

Hi,
As we know, WhatsApp Business has catalog feature.

The recipient of the order ( the WhatsApp business ) can see the order details from the web.
Once user clicks on the order details button.
WhatsApp web sen unencrypted soket request.
Something like this..
745.--26,,["query","order",{"id":"35206.44710-1","orderId":"961297227737359","imageWidth":"80","imageHeight":"80","token":"AR5zzk+vCTfMZNGd7sbYCJiLXWUMqLZaC8S+SjDABOZX1g=="}]
Now, most of the variables are captured from the handler via this PL
https://github.com/Rhymen/go-whatsapp/pull/518
the problem is the {"id":"35206.44710-1"}  is not included with the handler returned result.
Ps: let's say you have 5 orders in the chat.
If you click on any one on the first time, The latest number will be one, -1 now if you click on any other order, the number will be +1..which will be 2.
The id it self is not sate, but each time you refresh the chat you will nit a different number and different token.
In order to know what user's asking fir un their orders we need to send this json to grep the results but what is the correct method?
Regard

See if it helps you!
func (wac *Conn) Orders() (<-chan string, error) { 	data := []interface{}{"query", "order", map[string]string{ 		"orderId":     "275046030837098", 		"id":          "17582.4263-5", 		"imageHeight": strconv.Itoa(80), 		"imageWidth":  strconv.Itoa(80), 		"token":       "AR6DX3HdQbd2fh7p7LlpUXXgyBkHAPdXEMAtl3504NifEw==", 	}} 	return wac.writeJSON(data) }
func (wac *Conn) Products() (<-chan string, error) { 	data := []interface{}{"query", "bizCatalog", map[string]string{ 		"catalogWid"<EMAIL_ADDRESS>		"limit":      strconv.Itoa(10), 		"height":     strconv.Itoa(100), 		"width":      strconv.Itoa(100), 		"stanza_id":  "17582.4263-4", 		"type":       "get_product_catalog_reh", 	}} 	return wac.writeJSON(data) }

@Romerito007 Thank you for your reply but I don't understand, how can we get the "id": "17582.4263" !

@ Romerito007 Obrigado pela sua resposta, mas não entendi, como podemos obter o "id": "17582.4263"!

if changing the id makes no difference, test it by changing this id!
Sorry about my English.

Do you mean it is a random id? cos I test random id and it returns an error 400

Look, this is not going to work this way, You made the PL, and thank you so much for that.
Can you make the handler return the order details as well along with the order?

@Romerito007 I test your code but something does not correct it returns error 400.
I am not sure if it work with you.
Anyway, can you Update your PL to return order information?

Não tenho certeza se funciona com você.
De qualquer forma, você pode atualizar seu PL para retornar as informações do pedido?

it's working, I don't get 400 status, just 500 or 401

@Romerito007
thin it is not working you have to get the result of the request with a 200 status code!
Did you get the order json code with the response?

yes returns status 200

Great, can you make it as a function with the main function which it makes
the call after the handler takes the first request, so the final return
will include the order and the order information? if yes I will merge it
ASAP
On Sat, Mar 27, 2021 at 10:26 PM Romerito Alexandre <
@.***> wrote:

yes returns status 200
[image: Captura de tela de 2021-03-27 16-22-50]
https://user-images.githubusercontent.com/13719169/112732478-efcb5780-8f18-11eb-8731-da364894b698.png
[image: Captura de tela de 2021-03-27 16-24-09]
https://user-images.githubusercontent.com/13719169/112732480-f528a200-8f18-11eb-8173-c2dedb78b0c9.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Rhymen/go-whatsapp/issues/539#issuecomment-808790122,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDLT273U7KFQH6UCDNCDJDTFYWNRANCNFSM4Z2LD4QQ
.

Dear friend.. @Romerito007 in your example you know the ID and you are sending via  "Products" but that is not the case I am looking to.
Lets say i want to listen if any one send me a product cart. I need to get the information of the order.
Now, in the handler you did not return the "id": "17582.4263-5", which i don't find it anyware , I know it may a different number.
Now, What I am looking for NOT send a product. but if any one send me a product i want to handle and read the details.
Now to do this you need the ID. which is not included in the incoming request

I think it works for me
SOLVED

How did you do to work?

func randInt(min int, max int) int {
    return min + rand.Intn(max-min)
}
       rand.Seed(time.Now().UTC().UnixNano())
	id:= strconv.Itoa(randInt(11111, 99999))+"."+strconv.Itoa(randInt(11111, 99999))+"-1"

	datas:= []interface{}{"query", "order", map[string]string{"id":id ,"orderId":message.OrderId,"imageWidth":"80","imageHeight":"80","token":message.Token}}

    order_details,_ := wh.Wac.WriteJson(datas)
	output := <-order_details
	log.Warnf("JSON %v", output)

@beshoo and you are using the functions from @Romerito007's PR?

Yes, but he has to modify his PL to add this to the response context @Romerito007 please can you add it so I can merge the PL?


dataset

How the dataset is made

Hi, you can refer to the paper of The Pile (https://arxiv.org/abs/2101.00027). We use OpenWebText2 selected from The Pile.

I want to know what format of file is in 'enwik8'

Hi, sorry for confusing. it is a txt file.

我想知道你用的python环境和cuda，我跑train，一直说Traceback (most recent call last):
File "/home/lab/SpikeGPT/train.py", line 94, in
model = GPT(GPTConfig(train_dataset.vocab_size, train_dataset.ctx_len, model_type=model_type,
File "/home/lab/SpikeGPT/src/model.py", line 355, in init
self.blocks = nn.Sequential([Block(config, i)
File "/home/lab/SpikeGPT/src/model.py", line 355, in
self.blocks = nn.Sequential([Block(config, i)
File "/home/lab/SpikeGPT/src/model.py", line 316, in init
self.lif1 = neuron.MultiStepLIFNode(tau=2., surrogate_function=surrogate.ATan(alpha=2.0), backend='cupy',
File "/home/lab/SpikeGPT/src/spikingjelly/clock_driven/neuron.py", line 823, in init
check_backend(backend)
File "/home/lab/SpikeGPT/src/spikingjelly/clock_driven/neuron.py", line 30, in check_backend
assert cupy is not None, 'CuPy is not installed! You can install it from "https://github.com/cupy/cupy".'
AssertionError: CuPy is not installed! You can install it from "https://github.com/cupy/cupy".
可是我已经安装了cuda对应版本的cupy，我用的是4090的显卡

Hello,
We've tested multiple CUDA and cupy versions, ranging from 10.2 to 12.0. Additionally, we've experimented with PyTorch versions spanning from 1.9.0 to 2.0.0 – all of which have exhibited proper functionality. If you require additional assistance, kindly open a new issue to address your inquiries related to environment configuration, rather than commenting on this existing issue.

thank you


Trump approval ratings API or raw data endpoint?

FR: Is there any endpoint to the raw data similar to this: https://github.com/fivethirtyeight/data/issues/72
But for the trump approval ratings? https://projects.fivethirtyeight.com/trump-approval-ratings/
Thanks

I think these are all of them. Use inspect element and network tab to view interactive visualization data sources.
https://projects.fivethirtyeight.com/trump-approval-ratings/historical-approval.json
https://projects.fivethirtyeight.com/trump-approval-ratings/approval.json
https://projects.fivethirtyeight.com/trump-approval-ratings/polls.json
https://projects.fivethirtyeight.com/interactive-recirc/interactive-recirc.json

Genius!


Handring of Encodings that replaces Backslash

I'm using this editor on Japanese Windows to write C (MSVC) code.
Since Japanese defaults Shift_JIS, a Backslash character 0x52 on Japanese font shows as a Yen sign.
(display only, behavior in programs are as same as Backslash)
But when I load a Shift_JIS file, the editor converts 0x52 characters to Unicode's Yen sign.
This breaks things that rely on Backslash (like LSP).

This patch fixes it, but...
diff --git a/data/core/doc/init.lua b/data/core/doc/init.lua
index edc17cd1..7c325f80 100644
--- a/data/core/doc/init.lua
+++ b/data/core/doc/init.lua
@@ -84,6 +84,20 @@ function Doc:set_filename(filename, abs_filename)
 end

+---@param encoding string
+---@param content string
+---@return string fixed_content
+local function ecoding_conversion_fixes(encoding, content)
+  if encoding == "SHIFT_JIS" then
+    -- on Japanese \ is shown as Yen sign ¥, iconv do this conversion
+    -- which is problematic because it changes the original codepoint,
+    -- we are insterested on keeping the original to prevent issues on
+    -- code that uses escape sequences like '\n', '\r', etc...
+    content = content:ugsub("¥", "\\")
+  end
+  return content
+end
+
 function Doc:load(filename)
   if not self.encoding then
     local errmsg
@@ -108,6 +122,7 @@ function Doc:load(filename)
       strict = false,
       handle_from_bom = true
     }))
+    content = ecoding_conversion_fixes(self.encoding, content)
     for line in content:gmatch("([^\n]*)\n?") do
       if line:byte(-1) == 13 then
         line = line:sub(1, -2)

Here comes the but... I wonder if this would be the correct behavior since a legitimate ¥ codepoint would also get converted back to a \. Do you think such a fix would cause issues? Is it safe to always convert ¥ to \ ? I ask because I'm not familiar with the language. Sounds to me like this issue should be redirected to the iconv library since it shouldn't be doing such conversion when the ¥ character already has a specific codepoint and japanese fonts do the conversion automatically as you mentioned.
If it isn't safe to always convert ¥ to \, wouldn't it be better to switch to utf-8 encoding for coding purposes and leave SHIFT_JIS for other text editing activities non-code related?
Anyways, I smell this to be improper behavior from the iconv library side...

Was curious and searched a bit of history around this issue, leaving it here for future reference:

In Shift-JIS (a variant of JIS X 0208), the character ¥ (Yen sign) has a special representation. In the encoding scheme, ¥ is encoded as \x81\x9F. This might seem unusual, but it's because the original design of Shift-JIS was influenced by the way Japanese characters were represented in older systems.
In pre-1980s Japan, the Yen sign (¥) was often represented using a combination of ASCII characters: \\ (backslash-backslash). To accommodate this convention, the designers of Shift_JIS decided to encode ¥ as \x81\x9F, which is equivalent to the original \\ representation.
When you use iconv with Shift_JIS encoding and pass a string containing \, it attempts to convert it according to the rules defined for that encoding. In this case, since \ has a special meaning in Shift-JIS (representing ¥), iconv replaces the original \ characters with the equivalent ¥ character (\x81\x9F).

Added a proper fix on #106 which replaces every \ into a special char sequence that after conversion to a ¥ gets converted back to a \. Will merge right away since this solution shouldn't cause any issues (I think :smile:).
Thanks for reporting!

Sorry for comment, but this solution seems having issues when multi-byte characters are shown.

(text from https://www.type-labo.jp/KumimihonREAD.html)

Sorry for comment, but this solution seems having issues when multi-byte characters are shown.

@Hinaxnitro
Could you please try this change on core/encoding.lua and let me know if it fixes it for you?
diff --git a/data/core/encoding.lua b/data/core/encoding.lua
index 51adf9e4..bf4f5252 100644
--- a/data/core/encoding.lua
+++ b/data/core/encoding.lua
@@ -9,7 +9,7 @@ encoding.convert = function(tocharset, fromcharset, text, options)
     -- we are insterested on keeping the original to prevent issues on
     -- code that uses escape sequences like '\n', '\r', etc...
     local errmsg
-    text = text:gsub("\\", "{\\\\\\}")
+    text = text:ugsub("\\", "{\\\\\\}")
     text, errmsg = encoding_convert(tocharset, fromcharset, text, options)
     if text then
       text = text:gsub("%{¥¥¥%}", "\\")

Also could you open a new issue for this with a title like:
Wrong replacement of backslashes belonging to multi-byte sequences in Shift_JIS

And also link it to this issue by adding #94 as part of the description


Allow to remove a project from the UI

There is no way to remove a project from the UI, so users can only add things and not cleanup. This inconvenient in the case of a hosted service, as ssh + shell access is more complex than the UI.

Done in 1119082a76ca59bf609e0683d70fa3921b461791 and e5834fac836a017a9e9cf0fe32467af5c15b7182, apologies for not referencing the ticket in it.


What is the latest version of Blazor?

[ I searched a lot but couldn't find it. Is  blazor is related to the .net release? Such as dot net 7, ASP dot net 7, Blazor ASP dot net 7. In fact, I want to know what is the latest version of blazor now? Release date or end support etc.]

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

ID: 5bc28995-6784-a7d2-5c0b-9811a8d19c1a
Version Independent ID: 1668c8fb-024c-c86d-d4d5-e22e3355c04b
Content: ASP.NET Core Blazor
Content Source: aspnetcore/blazor/index.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @DebasisPaul ... Blazor is part of ASP.NET Core/.NET and follows the same release cycle. Therefore, you can get that information at ...
https://dotnet.microsoft.com/platform/support/policy/dotnet-core
The current release is .NET 6.
.NET 7 is in development now, and I think the latest release of that is Preview 4. Preview 5 is probably close to release. If you check out Dan Roth's .NET Blog posts, you can see the announcements for what's in these early previews of ASP.NET Core 7.0 ...
https://devblogs.microsoft.com/dotnet/author/danroth27/
The roadmap that the product unit maintains shows all of the items under consideration and complete ...
https://github.com/dotnet/aspnetcore/issues/39504
That roadmap is updated as they go. Things are added and subtracted at any time all the way up to the Release Candidate.


[Podofo] Update to 0.9.7

Library name:  Podofo
Latest version: 0.9.7
Portfiles and other necessary updates required to get an updated podofo version binaries.

Duplicate of #16358

Thanks Team!


[FEA] Support StaticInvoke to convert from string to varchar

I wish we can support StaticInvoke to convert from string to varchar.
eg.
create table src(id string) stored as parquet;
create table tgt(id varchar(200)) stored as parquet ;
insert into src values ('abc');
insert into tgt select id from src;

Fallback message:
      ! <StaticInvoke> staticinvoke(class org.apache.spark.sql.catalyst.util.CharVarcharCodegenUtils, StringType, varcharTypeWriteSideCheck, cast(id#25 as string), 200, true, false, true) cannot run on GPU because GPU does not currently support the operator class org.apache.spark.sql.catalyst.expressions.objects.StaticInvoke

There are a lot of things about this that are new. We have never done anything with StaticInvoke before. We also do not have support for the varchar data type. is not something that we have supported before. For this very specific case it looks do-able but because we have not done a number of these things before I want to be a bit cautious about that.
Please note that StaticInvoke is a very special operator and we will not be able to support it generally.


Shopee trying to fetch webp images and it fails

Hi, I am trying to upload products using Shopee Integration for WooCommerce plugin but during product upload shopee is trying to fetch webp images from cloudflare servers and not from the origin. This causes the upload to fail since shopee doesn't support using webp images for the products. I have tried setting up page rules to bypass example.com/wp-content/uploads/ to bypass cache but it doesn't work and webp images are still being served.

Plugin version is 4.4.0, WP version is 5.7 and Php version is 7.4.16

I am specifically using Cloudflare APO for WordPress.

As it turned out, webp express was the cause of images getting converted and not cloudflare.


method_whitelist failure'

ailed to download https://noaa-gfs-bdp-pds.s3.amazonaws.com/gfs.20240606/18/wave/gridded/gfswave.t18z.atlocn.0p16.f000.grib2: init() got an unexpected keyword argument 'method_whitelist'


Configuration/Companion App failing authentication

I entered my email and password for Frontier API access, received the verification code, but when I enter that I get this error:

I can't finish configuration.  I can't even load the configuration app any more until I delete credentials.json.
I've tried this a few times with the same results each time.
Any suggestions?  There's no eddi.log at this point.

Hi I had similar I requested 3 times same issue then I closed EDDI reopened it just to double check it wasn't a glitch and it worked on 4th attempt.  just for the record, the error I got was this:

Are you running the latest beta? If not, you can enable beta updates via the EDDI app.
If that fails then you may have a corrupted file. Close EDDI (and VoiceAttack as applicable), then open %APPDATA%/EDDI and delete the "credentials.json" file. Restart EDDI and refill your credentials.


Conditional check around ecr create repo CLI command

In response to #29
Modify the checking (in the form of an "if" now) around the "aws ecr describe-repositories" and conditional docker repository creation and hide confusing error message from the user.

Thanks very much. Issue https://github.com/gchq/sleeper/issues/29 actually refers to the confusing error message from the attempt to create a bucket for the jars. In this PR, you've fixed a similar issue for the creation of the ECR repos (unless I'm missing something?), which is great. We can either merge this PR and leave https://github.com/gchq/sleeper/issues/29 open, or you could update this PR to fix the issue with the jars bucket? Happy with either option - let me know.

This should now actually address #29.  Sorry about that!

Thank you! Merged into main.


Django Util

@jonparrott
Long overdue PR, this is pre-review before official PR.
Some notes:

Docs are still being finished
Django is really opposed to the idea of any global variables  http://stackoverflow.com/questions/2680902/python-django-global-variables which means I really avoided having a single "instance" where things get initiated. Instead you put things in the global Django settings. The place where this is most awkward is the authorization callback, which I temporarily removed.
The more Django-nic way is to put things like URLs in urls.py, views in views.py, statup in apps.py. However between the Django way and the oauth2client one-file way, I picked the latter, but not sure if it's better.
Haven't generalized storage yet, that's next
The existing django test, test_django_orm, makes a fake django settings module. I tried to duplicated this, and it worked in isolation, but broke when run as part of the full test suite. This is because Django loads the settings at import time and caches a ton of stuff. Trying to force all those imports to reload became a nightmare, so I reluctantly refactored the settings into a common file. The other big reason this is necessary is because Django settings is a module, but sort of treated like a map. What I mean by that is that you can modify fields on the settings object if they existed at import time, but if they didn't exist at import time you will get an AttributeError. This is why the shared settings module just defines a bunch of fields that can be changed later.

Not ready for merge into official repo yet, this was supposed to be against wparin/oauth2client until Jon takes a look, sorry!


No issue :p

I just want to say congratulation with your cool project!
Keep up the good work and make it a great project, so others can benefit from your great work ;-)

Thanks for the nice feedback @ustad-nordin !


[VS 2015 Build Tools] Microsoft.Build.Utilities.Core and dependency issue

The solution that I am trying to build and deploy on Azure comprises of ASP.NET MVC 5 website developed using RC version of VS 2015 and references .NET 4.5.2 framework
When this solution is checked-in to VSO and build is triggered to deploy to Azure Web App, it builds fine and deployment succeeds.
When this solution is checked-in to GitHub and build is triggered using Kudu, it gives following error
error MSB4062: The "Microsoft.CodeAnalysis.BuildTasks.Csc" task could not be loaded from the assembly
D:\home\site\repository\packages\Microsoft.Net.Compilers.1.0.0\build\..\tools\Microsoft.Build.Tasks.CodeAnalysis.dll.
Could not load file or assembly 'Microsoft.Build.Utilities.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies.
The system cannot find the file specified. Confirm that the <UsingTask> declaration is correct, that the assembly and all its dependencies are available, and that the task contains a public class that implements Microsoft.Build.Framework.ITask.

The packages.config for has Microsoft.Net.Compilers which has developmentDependency=true
 <package id="Microsoft.Net.Compilers" version="1.0.0" targetFramework="net452" developmentDependency="true" />

Considering that this solution is building correctly on VSO, I believe VS 2015 Build Tools are required to successfully compile this solution using Kudu.
Also, shouldn't this package be ignored for builds triggered outside of Visual Studio?

+1 VS2015 RC Webjob targeting .Net 4.5.1  Also using Microsoft.Net.Compilers 1.0.0 so that I can use C#6 features with a kudu build.  I have similar preexisting projects that still build fine with kudu (although I might not have another webjob with a direct dependency on Microsoft.Net.Compilers 1.0.0), but I can't get the new one I set up today to build.
You have my permission to check anything you need on lwf-email-webjob-master.azurewebsites.net

Could either of you please share a repo that demonstrates the issue? Thanks!

@davidebbo You can try this repo  https://github.com/punitganshani/NuGetViz
This repo builds fine when I use VSO builds, but fails with Kudu

Yes, I see the issue with your repo. Interestingly, I do see Microsoft.Build.Utilities.Core in D:\Program Files (x86)\MSBuild\14.0\Bin in Kudu, but something must be not quite right.
As a test, does the issue go away if you set developmentDependency to false?

I've changed as suggested developmentDependency=false, but it still failed.

you can set an appSetting for your app MSBUILD_PATH = D:\Program Files (x86)\MSBuild\14.0\Bin\MSBuild.exe and that should get you unblocked. Though I'm also not yet sure why the other msbuild is failing

Thanks, I can confirm that the MSBUILD_PATH workaround works.

@ahmelsayed Strange, I did not have to do this to make it work for VSO :-) Plus, everyone using VS2015 will face this issue and will have to change their appSettings
Is it possible to override the .targets files on the build server to correctly work with the path?

yes we need to figure out why this is failing and make it work without requiring app settings changes. I just mentioned the app setting as a workaround to get you unblocked :)
I'll take a look at the real reason for the failure as soon as I can

I think the real fix is to install Microsoft Build Tools 2015 onto the VM, which we are planning. That adds Microsoft.Build.Utilities.Core to the GAC, which I think is what makes it work.

To add- If anyone is using packages from pre-release packages of ASP.NET, they will also have to add NuGet.config in the solution directory to make ASP.NET 5.0 builds work as the package source is different from nuget.org

VS 2015 Build Tools should be on all machines now.

Is https://github.com/projectkudu/kudu/blob/master/Kudu.Core/Infrastructure/PathUtility.cs published? With that the workaround still needs to be present.

No, the change to use the new msbuild is not yet published.


Drag Drop into move mode

Hi
I am trying to get dragging a item from a list into dashboard and have it automatically add the new item and also go into moving mode without having to let go of the mouse button. So i can drag into the postion i want.
I have got the dragging working and the auto creating the dash-item, but i am having trouble going directly into move mode.
Here is a sample of where i am up to, you can drag the item at the top into the dashboard and it creates the dash-item, but if you continue to move the mouse around without letting of the mouse button, it doesnt drag the new dash-item around. (Not at least unles you let go and click mouse down again - which i am trying to avoid)
https://codesandbox.io/s/vue-responsive-dash-wn52w
Note i am using a simulate mouse function to try and trigger the change to move mode, but the mouse event doesn't seem to do the trick.
Do you have any suggestions on how to go into move mode from javascript ?
Thanks
Peter

Hi @peterjslater ,
Interesting application! The reason you mouse event is not working is its actually another div inside the component that monitors the mouse click. The div has an id of the items id + "-overlay"
i.e. if the item id is 4 the div monitoring clicks has an id of "4-overlay". However simply changing this in the example unfortunately does not give the desired result as the dash item then darts across the screen which is most likely due to the initial position being outside the layout? (thats a guess im not 100% sure")
Here is what i modified:
methods: {     onDragOver(item) {       console.log("Drag Over");       console.log(item.data);       if (!this.newOne) {         this.newOne = { id: "4", x: 0, y: 0, width: 1, height: 1 };         this.ditems.push(this.newOne);       }       this.$nextTick(() => {         // get the new dash-item         var newdashitem = document.getElementById("4-overlay")         console.log(newdashitem);         // try and simulate a mouse down to get the item into moving mode         this.simulate(newdashitem, "mousedown");       });     }   },

Edit it runs off the screen due to the code running over and over again.

Not a perfect example but should give you enough to go on. https://codesandbox.io/s/vue-responsive-dash-hw81b

Thanks, that helps a lot, will try it out.

I have almost got it all working, just having an issue where the positions get out of sync (prob because of the simulated mouse click)
To fix I just have to do a single mouse click on one of the dash-item, and the positions correct themselves.
Is it possible to have a javascript call on dashbord object to recalc all the positions just like a click on a dash item does ?
I have tried simulating mouse click but am having lots of trouble, if there was a method I could call it would make it a lot easier.
Thanks
Peter

Just to see if by calling this method it solves your issue try adding a dashItem with zero width/height and at random position. (when you would expect to want to call the method.) Then remove it again after a next tick event.

ok will try it, thanks.

The issue seems to have gone away with the latest update, thanks.

Hi the issue above is back (once i fixed the key using index it came back)
I can replicate the issue below,
https://codesandbox.io/s/vue-responsive-dash-wn52w?file=/src/App.vue
I am trying to drag an item into dashboard and then cancel the drag by moving the mouse out of the drag area without lifting the left mouse button. Once the drag is outside area, it should remove the dash-item that was just added and return the orignal dash-items to their original places.
However after the drag out, there is a "hole" left where the dash-item was. As single click on one of the existing dash-items will trigger a "recalc' and move them back into correct position.
I tried your suggestion adding an empty dash-item (h:0 w:0) and that seemed to fix it.
you can see this by uncommenting the code on lines 101-105
So if there was a call that did the same, that would be a bit cleaner than adding a dummy dash-item.
Thanks
Peter

Currently the dash Items are not compacted when an item is removed. Ill do some testing in adding this functionality in when i get some time. This will solve your issue.

https://codesandbox.io/s/vue-responsive-dash-tjqxq using the most recent version fixes this issue by re-compacting the layout on removal of an item.

Thanks, that works great!
p.s. for anyone using the codesandbox pages in this thread, the latest version of vue-easy-dnd (1.5.2) causes issues with the mouse moves, looks like they are not getting passed to the dashboard. It all works well with version "vue-easy-dnd": "1.3.17"

Good to hear!


CamelCase movement (sub word movement)

Hi!
I know this is not officially supported in vim, but rather a plugin (http://www.vim.org/scripts/script.php?script_id=1905) - but this is something that would make me move so much faster.
I am not suggesting you add this feature, but rather ask you to think about it for a few minutes and write a comment about how it could be done - I will do the change in my own repo.
Thanks! :)
PS: This should be a feature request/suggestion, but I can't find a way to tag it as such. I assume it has to be done by an admin or some such.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

hey @codeluggage did you get a chance to implement this? That'd be awesome.

Sorry @MarcoSero, no I didn't. My workflow doesn't involve much objc or swift lately, so I never pursued it. Best of luck if you do!

Did #826 implement this?


[5.1] Fixing scheduler switch user with withoutOverlapping together not working

Fixes #13332

What about windows support?

Ah... Sorry didn't test Windows.
Does sudo -u work on Windows anyway?
Also what terminal on Windows, the native one?

We need to get it to ignore the user flag on windows.

ok, so this be ok? or do you think there should also be a check in the user() function for if windows throw exception?
if (windows_os()) {
    return $command;
}

return $this->user ? 'sudo -u '.$this->user.' -- sh -c \''.$command.'\'' : $command;

I was just thinking: return $this->user && ! windows_os() ? 'sudo -u '.$this->user.' -- sh -c \''.$command.'\'' : $command;

Cool, done that. Thanks.


Removing API deleted dynamoDB data

I am troubleshooting another amplify issue and I deleted my API to try setting it up again and that deleted all my dynamo DB data without warning.
My wife had already created a bunch of new posts on her brand new blog I built her and now I have to let her know that's all gone. Very unhappy customer here.

Hi @jknapp25
Sorry for your troubles. Currently removing the api will delete @model dynamoDB tables.
I will mark this as enhacement to make it more clearer in future.

Thank you @akshbhu, I appreciate the response 🙂


MetricRecorder thread async update

Update the Metric Recorder to support asynchronous timing events.
Resolves #65

Can one of the admins verify this patch?

Jenkins, test this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/bdg-utils-prb/79/
Test PASSed.

Sorry for missing this @Georgehe4! Merged. Thank you!


Highlight code blocks

Hey @sindresorhus 🖖,
I've create a module that parse the twitter stream and match the markdown code block, and substitute that with highlighted html 😉
The color and the theme are in beta 😄

Here the example tweet 👇

Very cool! :)
Make sure you run npm test locally and fix the lint issues.

I've made some fixes today 😉
The last thing I miss is to do the aliases for the supported languages.

Cool. Can you also add a mention of the feature to the Highlights section in the readme?

Hi 🖖, I finished the last fixes but there is one last problem,
When I click on the tweet, the pop-up ends rendering after my function run.
How do you think about adding an event for example onPopupFinishLoading() ?

@tomma5o I've had the same problem when inlining instagram photos (see #19).
I've created a new event onSingleTweetOpen() and I'm waiting on @sindresorhus comments on it. I guess we can wait to have just this one event that you'll be able to hook your function to. :)
Nice work on this, btw! 👍

Oh, perfect 👍  @filipekiss. Thanks for the update!

#19 just landed, so you can now use the onSingleTweetOpen() method added there. The merge conflict also needs to be fixed.

🤙 Ok Perfect!
Thanks for the reminder 😉

Ok, I've added the new languages 👍
Sorry for the readme but my Diff tool solved it in this way 🙃
anyway Merry Christmas 🎄😀
@filipekiss @sindresorhus


guide/template-syntax: port to new doc design scheme

Use Jade extends with blocks rather than copying prose.
For the record, here are the differences in the HTML: diff-ts-dart-html.txt

Are you making a PR?

Working on it now.


Use healthcheck to make sure DB container starts first

Docker compose 2.1 added support to healthcheck. A statement that can add a test to see when the container is healthy.
In combination with depends_on, it can be used to assert that the db container starts before the tomcat container, avoiding start up problems in ThingWorx.

Cool thanks!


/api/datasources endpoint not accessible in 3.0.1 for non-Admin users

Hi anryko! 😃
After upgrading Grafana to 3.0.1 unfortunately the scripted dashboard doesn't load (gives blank page), when logged in without admin privileges.
That seems to be caused by the "/api/datasources" endpoint not being publicly accessible anymore. It returns:
{"message":"Permission denied"}
with such a user.
Are you aware of this? How could this be worked around?
Maybe providing an API key in getdash configuration would be an option?
Cheers,
Tobias.

Hey Tobias!
I think I fixed the issue. Please confirm.

Confirmed! 👍
Thanks for fast response.


Update README.md with sample usage

Small code sample might get someone to pickup the library easier.

@mbalous Thank you for the PR! My apologies we didn't pick up on this and merge it in sooner.


Reverted install.sh to previous.

@AnanthaKrish this is ready to be merged. Ignore that travis failure as the side branch credentials need to be in main for it to run properly.


amazon support

i want to be able to do my online shopping on discord

out of scope for this project


target .NET Framework 4.5.2

4.5 was end of life as of 2016-01-12

Codecov Report

Merging #55 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #55   +/-   ##
=======================================
  Coverage   21.49%   21.49%
=======================================
  Files         110      110
  Lines        1475     1475
  Branches      105      105
=======================================
  Hits          317      317
  Misses       1099     1099
  Partials       59       59

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 fabca1f...9f5264f. Read the comment docs.


Facet values list retrieval

Add a button to copy the list of facet values to the clipboard \n delimited

The facet counts were requested by @durack1 so keep this open, it was not on the board


Ensure case is normalized on Windows when comparing paths from XML coverage report

This fixes a subtle problem on Windows, which is case-preserving but case-insensitive.
Depending from where the user executed the coverage command to produce the XML file, the <sources> tag might contain a different case in the drive letter: w:\foo\bar vs W:\foo\bar (note the case of the first letter). On Windows both paths should be considered the same.
This patch calls the os.path.normcase function, which changes all paths to lower-case on Windows and is a noop on Linux.
I experienced this problem myself and with this patch this fixes the issue for me.

Coverage decreased (-0.04%) to 99.139% when pulling 0fca8c525d7df97fec3cb8e3dcbaa5986123c623 on nicoddemus:normcase-windows into 901cb3fc986982961785e841658085ead453c6c9 on Bachmann1234:master.

Coverage decreased (-0.04%) to 99.139% when pulling 0fca8c525d7df97fec3cb8e3dcbaa5986123c623 on nicoddemus:normcase-windows into 901cb3fc986982961785e841658085ead453c6c9 on Bachmann1234:master.

Missing line because my test suite does not test windows. Ill file an issue to add windows to travis

Thanks for the PR

https://pypi.org/project/diff-cover/2.0.1/

Great, thank you!


Access Access ModelState In Middleware?

Is it possible to access the ModelStateDictionary in a middleware class? I am looking to write a "catch all" ModelState.IsValid handler for every request for my API via middleware so I do not have to do the check in the controller actions all over the place.

@kspearrin ModelStateDictionary is a MVC-specific feature. It's not filled in until model binding and validation occur -- just before invoking an action method.
However it's possible to write catch-all ModelState.IsValid checks other ways. For example the MVC functional test web sites include a couple of ActionFilterAttribute subclasses that do this -- ModelStateValidationFilterAttribute and the more complicated / specific ValidateBodyParameterAttribute. ActionFilterAttribute is not the only approach: E.g. IActionFilters and IAsyncActionFilters can be added through the application model.

Right, a global MVC filter is the way to do this. That would apply the filter to every MVC controller action method. Middleware is simply the wrong layer for this.

Totally makes sense now that I think about it. Thanks for the tips.


Added spanish

No worries, and thanks for the translation.
Maybe you want to look into installing an extension for "editorconfig" support. Editorconfig is a way to supply configuration settings like indentation style, newline style and such to the editor on a per-project basis and if a project contains such a configuration file that config will be used instead of the default configuration that's put into your editors. Many projects supply these .editorconfig files, which then prevents such a thing from happening.

With what name would you like to be attributed for the translation in the readme and changelog. Your github name, your real name or some other name?

@Viriato139ac#0342


Terminal colors are different (lighter) after upgrading to macOS Catalina

VSCode Version: 1.39.2
OS Version: macOS Catalina 10.15

Steps to Reproduce:

Upgrade to macOS Catalina and notice the color change right away.
If you are already on macOS Catalina, open terminal / iTerm and compare with VSCode's terminal.

Images:
VSCode:

iTerm2 / Terminal:

Back in Mojave there was no such difference. VSCode's terminal displayed the same color.

The colors the terminal draws is based on the vscode theme. You can try change the renderer to see if that changes the colors https://github.com/microsoft/vscode/wiki/Terminal-Issues#rendering-problems

Tried both renderers, neither seem to impact the color. With the dom-based renderer the font looks slightly bigger, but that's all, nothing else changes.

@ay0o pretty sure your theme is saying to use that color then? You can set them in settings manually:

yes @Tyriar , you are right that I can manually force them to match the color I want by manually setting each of them, but this wasn't the behavior prior to the update to macOS Catalina, that's why I opened this ticket.
Before, both VSCode and iTerm2 had the same scheme without manual changes. In either case, I'm closing the ticket as the solution is as simple as:

Open iTerm2 / Terminal
Copy the color code (in my case, ansi blue and green)
Force that setting in VSCode's settings.json


validate all map values

Can someone have an idea how can I validate all map's values (regardless the map keys)?
For example:
tasks:
  "0":
    id: "0"
    color: "red"
    type: start
  "1":
    id: "1"
    color: "blue"
    type: middle
  "2":
    id: "1"
    color: "green"
    type: end
...

I would like to validate all keys with the same partial schema:
schema;task_schema:
  type: map
  mapping:
    id:
      type: str
      required: yes
    color:
      type: str
      required: yes
    type:
      type: str

Is that possible?
Thanks

Find an answer..
It can be done with regexp;(..) & matching-rule properties
tasks:
    type: map
    required: yes
    matching-rule: 'all' # can also be 'any' since only one regex is supplied
    mapping:
      regex;(^[0-9]+$): # numeric regex
        type: map
        mapping:
          id:
             type: str
             required: yes
          color:
              type: str
              required: yes
          type:
              type: str


Fix PaymentServer::handleURIOrFile warning message

GUIUtil::parseBitcoinURI does not check address validity. Therefore, an invalid address cannot cause parsing failure.
An invalid address case is covered by its own message.
This PR fixes the warning messages.

Rebased 45ebd9e1825886767efc38ddc3e300c638fcae6c -> 5b20d46f8be6b7fc08012d80a9fc42e9dbc812ff (pr225.02 -> pr225.03) on top of the recent CI changes.

@hebasto Can you share any examples that I can try for which GUIUtil::parseBitcoinURI will return false and https://github.com/hebasto/gui/blob/5b20d46f8be6b7fc08012d80a9fc42e9dbc812ff/src/qt/paymentserver.cpp#L254 will be executed ?


Update README spelling and grammar

Everyone knows the most important part of a project is the stuff the user never sees! Gotta start with a good foundation. 😎

haha great :)


Can't use secret to auth

Hi, I'm Mind
I'm trying using secret for auth feature.
At first, it works after I put username and password direct with the secret, and then since we have user/pass on vault. For security, I need to call it from the vault.
`username=$( echo $1 | base64)
password=$(echo $2 | base64)
VALUES_PATH=$3
sed -i "s/USER_NAME:/& $username/" $VALUES_PATH
sed -i "s/USER_PASSWORD:/& $password/" $VALUES_PATH
`
Here is my code. It was simple after I get the username and password. I insert it to the end of USER_NAME and USER_PASSWORD on the Yaml file and it doesn't work. The pod run and URL can access normally, but can't log in. It seems like an invalid credential.
I put AUTH_TYPE: "LOGIN_FORM" in configmap and
SPRING_SECURITY_USER_NAME:   SPRING_SECURITY_USER_PASSWORD:  on secret
I exec to the pod by kubectl exec -it <pod name> -n <namespace> -- sh  and use command 'env' found that both username and password are corrected.
If I put SPRING_SECURITY_USER_NAME:   SPRING_SECURITY_USER_PASSWORD:  to configmap, it will work.
I don't why either, but I need to put it on secret. Thank you for your help in advance.

It's back to works for some reason. I guess it doesn't works since vault function on jenkinsfile. Anyway, will close this.


QA: Item page

🐛 Bug

[x] Image gets squished (incorrect aspect ration) in Safari

[ ] Image controls to overlay image (#1204)

[ ] Image should stretch fully to the edge of the dark background area

[ ] "Back to results" CTA overlay image

[ ] apply latest "info" icon (svg code below)

[x] Bottom margin on "Curator's Intro" paragraph is tight

[x] Add 60 pixels above the footer

[x] (Mobile) spacing between CC icons and "Author", make it 24px

[x] 950 breakpoint: take title font down one style

Info icon code:

Updated icon svg sode:
Info_Icon.txt


Performance related questions about query_posts/pagination and ACF get_field.

Just getting started on Timber and as I read through the documentation I have a few questions.
When you use query_posts to alter pagination, does this requery the database an additional time in addition to get_posts()?
Also, I could be wrong, but I believe the example "pre_get_posts" action in the documentation needs a conditional restricting it to altering the "home" query as indicated.
Additionally, if I wanted to use ACF's get_fields instead of querying each field separately, is there an easy way to do this, or would I be looking at setting up a separate class for each post type and a separate property for field, following the Extending Objects documentation?
Would it be possible to making create something like AcfPost which houses the returned array in an "acf" property, and then access via {{ post.acf.field_name }}?
Another possible minor documentation correction: for the first Extending Objects example, I assume that {{ post.issue.title }} uses the return of the function issue(). If this is the case {{ post.issue.name }} should be used as a term object is returned.
Thanks!

I got the AcfPost class working using
class AcfPost extends TimberPost {

    var $_acf;

    public function acf(){
        if(!$this->_acf){
            $this->_acf = get_fields($this->ID);
        }
        return $this->_acf;
    }

}

Fields can be access in twig templates via {{ post.acf.field_name }}

Going to close this as you seem to have solved it, feel free to comment if it's still an issue and ill re-open.


Build fails with Go 1.23

What happened:
Build with Go 1.23 rc1 & rc2 fails.
What you expected to happen:
Build to succeed.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?
Build error
  ...
  # github.com/grafana/pyroscope-go/godeltaprof/internal/pprof
  /Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/grafana/pyroscope-go/godeltaprof@v0.1.6/internal/pprof/delta_mutex.go:30:20: undefined: runtime_cyclesPerSecond
  /Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/grafana/pyroscope-go/godeltaprof@v0.1.6/internal/pprof/proto.go:320:8: undefined: runtime_expandFinalInlineFrame
  make: *** [juicefs] Error 1

Indicating the upstream problems

https://github.com/grafana/pyroscope-go/issues/103
https://github.com/golang/go/issues/67548

The upstream fixes still needs to be released.
Environment:

JuiceFS version (use juicefs --version) or Hadoop Java SDK version: 1.2.0
Cloud provider or hardware configuration running JuiceFS:
OS (e.g cat /etc/os-release):
Kernel (e.g. uname -a):
Object storage (cloud provider and region, or self maintained):
Metadata engine info (version, cloud provider managed or self maintained):
Network connectivity (JuiceFS to metadata engine, JuiceFS to object storage):
Others:

Build attempt https://github.com/Homebrew/homebrew-core/pull/175310
Formula: https://github.com/Homebrew/homebrew-core/blob/master/Formula/j/juicefs.rb
Go 1.23 no longer allows //go:linkname * runtime.* link instructioins, see
Go 1.23 release notes: https://tip.golang.org/doc/go1.23#linker
Proposal: https://github.com/golang/go/issues/67401

Thanks, we will fix our code after the upstream fixes

FYI, upstream pyroscope-go has released a fix for Go 1.23:
https://github.com/grafana/pyroscope-go/releases/tag/godeltaprof%2Fv0.1.8


Add ArtifactInfo into design-time build

Allows aspnet/universe to acquire a list of expected outputs (artifact info) without building aspnet/SignalR

Two test failures in Travis-CI Linux, but they appear to be unrelated to this change. Is this known?
[xUnit.net 00:00:01.8281447]   Starting:    Microsoft.AspNetCore.SignalR.Tests (parallel test collections = on, max threads = 16)
Application startup exception: System.NotSupportedException: Duplicate definitions of 'OverloadedMethod'. Overloading is not supported.
   at Microsoft.AspNetCore.SignalR.HubEndPoint`1.DiscoverHubMethods() in /_/src/Microsoft.AspNetCore.SignalR.Core/HubEndPoint.cs:line 524
``

Thats not the actual failing test :P
But don't block your change on the 2 failing tests

I'll take that as a :shipit:?

One, sec, I'll look at the actual change :D


Update sbt-scala-native to 0.4.2

Updates org.scala-native:sbt-scala-native from 0.4.1 to 0.4.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-native", artifactId = "sbt-scala-native" } ]

labels: sbt-plugin-update, early-semver-minor, semver-spec-patch

superseded by #176


Release 19c request

Hi,
I admire your work and I use your images in many projects. Could you create a docker image for Oracle 19c (Version <IP_ADDRESS>.0)?
Thanks for all your work!

Thank you very much, @zachy!
Unfortunately, there is no version of Oracle Database Free for 19c and the version in question (19.21) is only available for Oracle Database Enterprise Edition and Standard Edition 2.
Neither of these editions, Oracle Database Enterprise Edition and Oracle Database Standard Edition 2, are available under Oracle's Free Use Terms and Conditions (FUTC) and hence I cannot build images of these editions nor would I be allowed to redistribute them.
Additionally, the .21 is a Release Update (RU), which is only available to customers with an active support contract.
However, the good news is that Oracle provides images of that version over at their Container Registry (container-registry.oracle.com) where you can also find the particular version in question: https://container-registry.oracle.com/ords/ocr/ba/database/enterprise_ru
Thanks,