Add golangci-lint to CI

@hawkowl I like the look of this, please raise it on scrum today so we can talk over some of the wider context.

Please reopen when ready for review, doing housekeeping of older PRs :)


fix: document symbol requires non-empty name

Summary
Fix #715
VS Code now appears to require non-empty name for document symbol. Added guard check.
Testing
Test case in #715

No error 'Request textDocument/documentSymbol failed' if trying to type proc.


improve meta server to reply ERR_BUSY for config sync if is updating …

…config to zk, refer to #504
refer to https://github.com/imzhenyu/rDSN/issues/504

@imzhenyu , is it the step of committing changes to dist.service?

create pull request in rDSN.dist.service
merge the pull request in rDSN.dist.service
in rDSN, run 'git submodule update --remote', and create pull request
merge the pull request in rDSN

@qinzuoyan  the travis-ci test seems wired today and I just made some fix at https://github.com/imzhenyu/rDSN/pull/530 . May need to restart the test for this pr again after that pr is merged up to Microsoft/rDSN.
The last two steps for committing changes to dist.service are unnecessary unless you always work from that repo to include this repo. For me, I have the two repos in different directories.
git clone imzhenyu/rDSN
git clone imzhenyu/rDSN.dist.service
cd rDSN
./run.sh build
./run.sh test
./run.sh install
cd ..
cd rDSN.dist.service
dsn.run.sh build
dsn.run.sh test

Seems the failure is caused by ./run.sh stop_zk in .travis.yml. You may remove that and try this pr again.

@imzhenyu , where is the script dsn.run.sh?

it is under $DSN_ROOT/bin - which is available after rDSN installation.

@imzhenyu , please review it or merge it.


cargo run --example tour crashes with (exit code: 0xc0000005, STATUS_ACCESS_VIOLATION)

Related (duplicate?) of #98.
I'm trying to run cargo run --example tour on Windows 10, but it crashes:
$> cargo run --package tour
    Finished dev [unoptimized + debuginfo] target(s) in 0.50s
     Running `target\debug\tour.exe`
error: process didn't exit successfully: `target\debug\tour.exe` (exit code: 0xc0000005, STATUS_ACCESS_VIOLATION)

I have an additional monitor connected to my laptop, which is my primary display:

Laptop monitor
Additional monitor (primary display)

However, I can start the application successfully by I either making my laptop (1) the primary display or by disconnecting my additional monitor (2) (which makes my laptop monitor also the primary display I guess).
I have these graphic cards with the latest drivers installed:

Intel(R) HD Graphics 530
NVIDIA GeForce GTX 970M

Iced is using the Vulkan renderer and is crashing somewhere here trying to create the swap chain.

struct wgpu_native::id::Id<wgpu_native::swap_chain::SwapChain<gfx_backend_empty::Backend>> wgpu_native::device::device_create_swap_chain<gfx_backend_vulkan::Backend>(struct wgpu_native::hub::Global *, struct wgpu_native::id::Id<wgpu_native::device::Device<gfx_backend_empty::Backend>>, struct wgpu_native::id::Id<wgpu_native::instance::Surface>, struct wgpu_native::swap_chain::SwapChainDescriptor *) (~\.cargo\registry\src\github.com-1ecc6299db9ec823\wgpu-native-0.4.3\src\device.rs:2091)
struct wgpu_native::id::Id<wgpu_native::swap_chain::SwapChain<gfx_backend_empty::Backend>> wgpu_native::device::wgpu_device_create_swap_chain(struct wgpu_native::id::Id<wgpu_native::device::Device<gfx_backend_empty::Backend>>, struct wgpu_native::id::Id<wgpu_native::instance::Surface>, struct wgpu_native::swap_chain::SwapChainDescriptor *) (~\.cargo\registry\src\github.com-1ecc6299db9ec823\wgpu-native-0.4.3\src\device.rs:2125)
struct wgpu::SwapChain wgpu::Device::create_swap_chain(struct wgpu::Surface *, struct wgpu_native::swap_chain::SwapChainDescriptor *) (~\.cargo\registry\src\github.com-1ecc6299db9ec823\wgpu-0.4.0\src\lib.rs:834)
static struct wgpu::SwapChain iced_wgpu::window::swap_chain::new_swap_chain(struct wgpu::Surface *, wgpu_native::resource::TextureFormat, unsigned int, unsigned int, struct wgpu::Device *) (~\.cargo\registry\src\github.com-1ecc6299db9ec823\iced_wgpu-0.2.3\src\window\swap_chain.rs:47)
struct iced_wgpu::window::swap_chain::SwapChain iced_wgpu::window::swap_chain::SwapChain::new(struct wgpu::Surface *, wgpu_native::resource::TextureFormat, unsigned int, unsigned int) (~\.cargo\registry\src\github.com-1ecc6299db9ec823\iced_wgpu-0.2.3\src\window\swap_chain.rs:26)
struct iced_wgpu::window::swap_chain::SwapChain iced_wgpu::window::backend::{{impl}}::create_swap_chain(struct iced_wgpu::window::backend::Backend *, struct wgpu::Surface *, unsigned int, unsigned int) (~\.cargo\registry\src\github.com-1ecc6299db9ec823\iced_wgpu-0.2.3\src\window\backend.rs:64)
void iced_winit::application::Application::run<iced::application::Instance<test::Counter>>(struct iced_winit::settings::Settings<()>, struct iced_wgpu::settings::Settings) (~\.cargo\registry\src\github.com-1ecc6299db9ec823\iced_winit-0.1.0\src\application.rs:187)
static void iced::application::Application::run<test::Counter>(struct iced::settings::Settings<()>) (~\.cargo\registry\src\github.com-1ecc6299db9ec823\iced-0.1.1\src\application.rs:201)
void iced::sandbox::Sandbox::run<test::Counter>(struct iced::settings::Settings<()>) (~\.cargo\registry\src\github.com-1ecc6299db9ec823\iced-0.1.1\src\sandbox.rs:139)
static void test::main() (d:\Dev\Programming\Rust\test\src\main.rs:5)
int std::rt::lang_start::{{closure}}<()>(struct std::rt::lang_start::closure-0 *) (~\.rustup\toolchains\stable-x86_64-pc-windows-msvc\lib\rustlib\src\rust\src\libstd\rt.rs:67)
void std::rt::lang_start_internal() (@std::rt::lang_start_internal::hb00fb1ac5aa11932:57)
__int64 std::rt::lang_start<()>( *, __int64, unsigned char * *) (~\.rustup\toolchains\stable-x86_64-pc-windows-msvc\lib\rustlib\src\rust\src\libstd\rt.rs:67)
main (@main:10)
static int __scrt_common_main_seh() (@7ff6c99dc390..7ff6c99dc3f7:3)
BaseThreadInitThunk (@BaseThreadInitThunk:9)
RtlUserThreadStart (@RtlUserThreadStart:12)

I have tested the cube example in wgpu-rs which worked fine for me.

Thanks for the report!
Have you tried the game_of_life or solar_system examples?

Both work fine for me.

Both work fine for me.

For me, cargo run works if I'm in iced/examples/tour, otherwise:
andy@oceanus:~/src/rust/iced$ cargo run --example tour
error: no example target named `tour`
andy@oceanus:~/src/rust/iced$ uname -a
Linux oceanus 4.19.0-10-amd64 #1 SMP Debian 4.19.132-1 (2020-07-24) x86_64 GNU/Linux

I'm getting a similar error: (exit code: 0xc0000005, STATUS_ACCESS_VIOLATION).
Using the iced_glow renders works perfectly fine.

Windows 10 20H2, build 19042.662
AMD Radeon RX 5700 XT, driver version <IP_ADDRESS>-201125a-361677E-RadeonSoftwareAdrenalin2020

Some hopefully helpful debug info:

DxDiag.txt
vulkaninfo.txt


Jczaja/sycl except

Added a path for non-GPU aware MPI for mpi_allreduce. It can be run like this (or without MPI4JAX_uSE_SYCL_MPI):
MPI4JAX_USE_SYCL_MPI=0 time mpiexec -np 2 -ppn  1 --pmi=pmix -l ./run_mpi4jax.sh

lgtm although I would put with gil inside functions and mark them as nogil and consider putting abort outside of gil lock because I haven't seen it called with gil in mpi4jax. like here: https://github.com/jczaja/mpi4jax/pull/3/files#diff-22f47e164769f1f04c3bf44d98ac62b25f2074e4de2750c98d72ccfd931c32e1R51

Good suggesstion! I will experiment with this approach in my next PR

@jczaja great work! If possible, please write PR titles that express briefly what the PR is about.


Fixed inconsistencies in commands in kinesis lambda example

The lambda is created with a different name from that which is used in the rest of the document.
Issue #, if available:
n/a
Description of changes:
Change inconsistent names used in commands. And clarified that the put-record command no longer requires a base64 encoded string, but rather the string itself.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks!


chunk_neighbors update algorithm now count maximum number of particle…

… neighbors

Error with exaDEM (same as in CI):
/home/rp269144/ExaCodes/exaNBody/src/exanb/core/lib/domain.cpp:26:8: error: no declaration matches ‘void exanb::Domain::set_xform(exanb::Mat3d)’
   26 |   void Domain::set_xform(const Mat3d mat)
      |        ^~~~~~
In file included from /home/rp269144/ExaCodes/exaNBody/src/exanb/core/lib/domain.cpp:19:
/home/rp269144/ExaCodes/exaNBody/src/exanb/core/include/exanb/core/domain.h:71:10: note: candidate is: ‘void exanb::Domain::set_xform(const exanb::Mat3d&)’
   71 |     void set_xform(const Mat3d& mat);
      |          ^~~~~~~~~
/home/rp269144/ExaCodes/exaNBody/src/exanb/core/include/exanb/core/domain.h:36:20: note: ‘class exanb::Domain’ defined here
   36 |   class alignas(8) Domain

@carrardt

It miss a member in GridChunkNeighbors:
/home/rp269144/ExaCodes/exaNBody/src/exanb/particle_neighbors/include/exanb/particle_neighbors/chunk_neighbors_execute.h:65:23: error: ‘struct exanb::GridChunkNeighbors’ has no member named ‘m_max_neighbors’
   65 |       chunk_neighbors.m_max_neighbors = 0;


Clear outputs in FunctionNode.unchain

From the documentation, I expect outputs to be cleared as well (even though they're weak refs).
Additionally I put an error message for ChainerX fallback function nodes.

Jenkins, test this please.

Jenkins CI test (for commit 03f173e87b83d2a082d56836dbffc465e72a02fb, target branch master) failed with status FAILURE.

The CI error is not related. LGTM.

Jenkins CI test (for commit 03f173e87b83d2a082d56836dbffc465e72a02fb, target branch master) succeeded!


added a  tooltip for fingertip duration for off-chain groups

Added a tooltip to display fingertip duration information, providing users with helpful insights into the duration of fingertip interactions.

Description

Related Issue
Fixes #304

Does this introduce a breaking change?
No

Hey @0xjei
I get your point. We can ask the design team for feedback and change it in the future if necessary.

Hey @0xjei
I get your point. We can ask the design team for feedback and change it in the future if necessary.

Yes and thank you! Let's merge it :)

@vplasencia @0xjei Thankyou!! Will be waiting for the future updates regarding color changes.


Autoadd team label on issue/pr assignment

When we trigger a r? <teamname> the rustbot assigns the pr/issue to someone but we don't actually check the team label for that issue/pr. This is useful for tracking its progress when we periodically check the PRs of $team.
This patch should also add the team label when the assignment to a team is requested in a github comment.
Requested by Felix on Zulip.
Tested locally (see here).
r? @Mark-Simulacrum or @spastorino

Unrelated: another thing I notice in the triagebot is this Github client. Everytime an HTTP request fails it will be retried a few times before giving up.
In some cases, ex. when I try to set the label https://api.github.com/repos/apiraino/test-triagebot/labels/t-android it will obviously never work so we can give up on the first try. Or maybe not retry on some HTTP error, like 404s. Anyway, in some cases it could be slightly smarter and bail out earlier


Run PR tests via GitHub Actions

Description of the change:
Add GitHub Actions workflow to run the Travis CI tests against PRs. This workflow will not run the Travis CI deploy stages that build and push images on pushes of master or tags.
Motivation for the change:
We have exhausted our Travis CI free usage plan, and we need a way to keep getting PRs merged. This may be temporarily until we can resolve the situation with Travis CI (or maybe its permanent if we like it enough).

/cc @estroz @jberkhahn


Quickstart fixes

@fschueler, @ParkL, @ggevay: can one of you go through the examples from the Project Setup page and then release the PR?

@ParkL is on it, but he ran into some issue, which he is looking into now.

Alright.

FYI, this PR changes the color palette of the website to Solarized Dark.
Me and @akunft were talking about using a consistent color palette across all documentation, slides, and papers in the future. IMHO Solarized is a good choice - it has both a light and a dark variant, ready-to-use IntelliJ template, and a wide range of colors (16) to derive other variants suitable for other media (e.g. presentation).

I really like the solarized theme, especially as there is a light and dark one.
@aalexandrov Should we also give the slide theme a try? The thing is that we are limited in the number of columns then.

The thing is that we are limited in the number of columns then.

Why is this an issue with this particular theme?

This would be an issue with the theme I used for the slides, as it only consists of 8 colors in the moment, not for solarized.

FYI: I already deployed the snapshots of this branch in sonatype. You don't need to manually run
mvn install

before you test the Project Setup instructions - just make sure that Sonatype snapshots are enabled in your ~/.m2/settings.xml.
<profiles>
  <profile>
    <id>allow-snapshots</id>
    <activation><activeByDefault>true</activeByDefault></activation>
    <repositories>
      <repository>
        <id>sonatype.snapshots</id>
        <url>https://oss.sonatype.org/content/repositories/snapshots</url>
        <releases><enabled>false</enabled></releases>
        <snapshots><enabled>true</enabled></snapshots>
      </repository>
      <!-- more repositories here -->
    </repositories>
  </profile>
</profiles>

:+1: for Solarized. It should be available on most editors, presentation templates, etc.


Enhancement request for a 'function' flow activity

I'm submitting a ...  (check one with "x")
[] bug report => search github for a similar issue or PR before submitting
[X] feature request
[] support request
[] general question

Current behavior (how does the issue manifest):
A user can only use existing functionality from triggers and activities, which limits the possibilities of what can be programmed.
Expected behavior:
Competitor tools like Node-red have a 'function' node, where any Javascript code can be written, which makes the tool very flexible and powerful. It would be useful to have such an activity in Flogo, that can take Go code. If it's possible to have C or Javascript, that would be even more powerful, but at a minimum, Go support would be quite useful. And for the IoT device usecase, C may be useful.
Minimal steps to reproduce the problem (not required if feature enhancement):
Functionality does not exist, no need to reproduce.
What is the motivation / use case for changing the behavior?
Competitors have this functionality, and having it makes the tool very flexible and powerful.
Please tell us about your environment (Operating system, docker version, browser & web ui version, etc):
MacOS X version 10.13.5
Docker version 18.03.1-ce, build 9ee9f40
Browser: Firefox Quantum 61.0
Flogo version (CLI & contrib/lib. If unknown, leave empty or state unknown): 0.X.X
Flogo version 0.5.3
Additional information you deem important (e.g. issue happens only occasionally):
n/a

@vijaynalawade has created an activity that can be used to execute JS: https://github.com/vijaynalawade/flogo/tree/master/activity/js


Ra no max failures

Why
Bulk file ingests should fail fast particularly when there is the same issue for all the files and all of them fail.
Dataflow and Argo could both perform a little better with more resources, which is helpful for getting imports done quickly.
This PR

Set max failures for bulk file ingests to 0 so that Jade will fail fast on file ingests
Bump up dataflow machine type and max number of worker params
Bump up parallelism for argo workflows

when there is the same issue --> can you clarify: operationally you're observing that ingests are dying consistently with the same error on all files and we're wasting time attempting to ingest the rest of the data set, hence this PR?

when there is the same issue --> can you clarify: operationally you're observing that ingests are dying consistently with the same error on all files and we're wasting time attempting to ingest the rest of the data set, hence this PR?

Yes


Python package on PyPi

Thank you for your work, I have now switched from rubber to latexrun :)
It would be also very convenient to be able to install latexrun as a PyPi package.
@aclements, do you have plans to publish it at PyPi? The setup.py file could be very simple, like this:
import setuptools

setuptools.setup(
    name='latexrun',
    author='Austin Clements',
<EMAIL_ADDRESS>    description='A 21st century LaTeX wrapper',
    keywords='latex',
    license='MIT',
    packages=setuptools.find_packages(),
    scripts=['latexrun'],
    version='0.1',
)

If you are not going to, I can do it with my own fork, but I think it's better not to produce extra entities and the best solution is if you could publish it as the developer, through your own PyPi account.
PS: You may also want to explicitly mention that latexrun uses Python 3 syntax and will not work under Python 2.7.
Thanks!

@zoresvit are you still willing to publish the package on Pypi?
Seems like this library would take great benefit from a fork with a maintainer having time to follow the issues and PRs!

@MonsieurV By now I switched to using tectonic which has very convenient features: automatically resolves and downloads dependencies, runs TeX and BibTeX as much as needed to build a document, gets rid of mess with temporary files, supports Unicode out of the box, etc. It's fresh and still in beta, but has been working great for me so far. For me tectonic completely superseded latexrun, so I've lost the interest to package it :)
Anyone still interested is more then welcome to do it though, using the skeleton of setup.py above as a base.


Add some very superficial tests

As I wrote in #38 I also started looking into phlex's deprecation of the #template method. And I also stumbled upon the missing tests to somehow verify the warning is gone but components still render.
So I wanted to add some very simple tests. They are mostly auto-generated and then adjusted manually where necessary.
The tests only try to render the components using defaults where possible or very minimal test parameters where not. Nothing fancy and barely useful, but hopefully better than nothing.
I stumbled upon two problems I had to solve:

Turns out phlex_ui implicitly depends on the json and activesupport gems. It uses #to_json in some places and a couple of AS core extensions (Object#present?, String#underscore and String#html_safe). I made these dependencies explicit. Of course, AS is a heavy dependency and it might make sense to think about removing it. But for now I made sure to require only what is needed. And in rails apps it is always there anyway.
I think the tests surfaced a little bug: PhlexUI::Select::Content had a default value of nil for the outlet_id parameter, but then goes on to call methods on the value that nil does not have. My fix was simply to remove the default.

I did not go so far as to add a Github action to run these test automatically. If this is merged, it might make sense to add one. (I might even look into it, but maybe someone here is a little bit more comfortable with GH actions than me and beats me to it :slightly_smiling_face: )

Great PR. I think it would be best to merge this and #38 to update-phlex-22 and test things in that branch before we merge things into main.
Also I would like to start adding support for sinatra and other making this project not completely rails dependent so good catch on the places active support is being used. I don't think we should explicitly add them as requirements to the gemspec but instead document which calls are AS like you did and remove them in the next release preparing for support for sinatra and not making it explicitly rails.

This looks like a good starting place for tests. I'd say fix up those style format warnings to make CI happy.

thats something i am fixing on main right now. Its with an updated version of standard. @mrinterweb


Dependency Injection in Views

Fixes #128

@NTaylorMullen Please review.

Hmm, after looking over the format of this article I'm not sure the format's right. It dives into the linked samples pretty extensively but I was under the assumption this would be more of a reference doc (@danroth27 please verify)? If so I'd envision the layout being:

Intro
Inject a service into a view
Override a previously injected service into a view (HtmlHelper for example).

Each showcasing how to do inject the service and the outcome but not diving too much into detail on what's going on in the sample.

@NTaylorMullen This topic wasn't intended to be an @inject reference. It was intended to be scenario based, so I think the amount of example code is ok.
@ardalis Please do add a section about being able to override a previously injected service. For example you can override @Html to provide your own HTML helpers

Are you sure overriding works? I have this:
@inject MyHtmlHelper Html
in my sample and it says:
"Member with the same name is already declared"
@NTaylorMullen @danroth27

Hey @ardalis what bits are you using? I tried this locally with rc1-update1 bits and all seems well:
public class FooHelper
{
    public int Value { get; set; }
}

I'm using rc1-update1 bits as well. Let me know your email and I can send you the project - it's tiny.

@ardalis Taylor dot Mullen at microsoft.com

Hey @ardalis. Everything works on my box (verified runtime too). My bet is that your tooling is semi-corrupted. /cc @ToddGrun

@NTaylorMullen Thanks. Not sure how to resolve that immediately. I'll write the code I think should work and check it in and you can see if it looks good and wrap up this PR, ok?  Thanks!

@ardalis Does it work at runtime for you?

@NTaylorMullen it does now that I remembered to add the helper to DI... Should be all good now. Shipit?

@NTaylorMullen  Shouldn't we be using the latest RC2 bits at this late stage. Why have planned obsolescence?

@ardalis Just spin up a clean VM with the latest bits, it's the lowest friction approach to a clean environment. I do my development on a VM and have a scheduled task robocopy the code to my publishing VM every minute.

@Rick-Anderson RC2 bits have the core renames. If @danroth27 wants to hold off on those then RC1 bits should be fine.

:shipit:

Merged.


Reset box-shadow styles

Box-shadows are missing from styles reset section.
I catch it after using VS and Twitter Bootstrap
Before

After

I removed some trailing whitespaces
Let me know if you want a commit with a rebuild.

Great find. Thanks!


Implement conversions between char* and Path[Buf]

On unix this allows arbitrary encodings, otherwise (windows) UTF-8 is expected.

Perfect! I just wait for appveyor and I merge.


tsx watch does not work in npm script

Problem
I´ve try to use tsx to execute my server.ts file. But when i run the script, it doest not work and throws an error for me:
I tried:

Delete node_modules and install dependencies again
Uninstall and install tsx package as dev dependecy
Clear npm cache

Other point, when i change src/server.ts on dev script to other radomness name, its throws the same error in my terminal

Expected behavior
Run my server.ts file.
I change tsx to other compiler package and its works perfectly
Minimal reproduction URL
https://stackblitz.com/github/pedroaurelli/upload-ai-api?file=package.json
Version
^3.12.9
Node.js version
v16.13.1
Package manager
npm
Operating system
Windows 10
Contributions

[ ] I plan to open a pull request for this issue
[ ] I plan to make a financial contribution to this project

Your reproduction runs fine for me

FYI tsx only supports the latest version of each major. Upgrading to v16.20.2 works fine.
Also for future reference, your minimal reproduction should not include a server or prisma as they're not necessary to demonstrate the bug.


Involve daemon mode switch for up cmd

Add new --deamon-off command line parameter
for 'up' cmd instead of existing log-file workaround
Describe your changes
During debug the service, I am interested in output of the service. With this option it is easily configurable.
I tried to follow the current code style pattern, nevertheless I recommend to organize the content of the up.go into a new struct and register the upCmd into the cobra via a new function like this func registerUpCmd(rootCmd *cobra.Command). With it you can avoid global variables and init() functions.
Issue ticket number and link
Checklist

[ ] Is it a bug fix
[ ] Is a typo/documentation fix
[x] Is a feature enhancement
[ ] Created tests that fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

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

My pleasure


log in lotusdb

investigate the common used log package in Golang, and choose one for our project.
main goals:

lighweight
fast
simple to use

i think map be zap

i think map be zap

zap is a too huge project?

i think map be zap

zap is a too huge project?

more light weight?

i could choose glg to investigate log, how do i participate?

i could choose glg to investigate log, how do i participate?

investigate the commonly used log and collect their features

ok i will search for some package

cpu: 12th Gen Intel(R) Core(TM) i7-12650H
BenchmarkStandardLog
BenchmarkStandardLog-16           453703              2448 ns/op              64 B/op          1 allocs/op
BenchmarkZeroLog
BenchmarkZeroLog-16               457500              2498 ns/op               0 B/op          0 allocs/op
BenchmarkZapLog
BenchmarkZapLog-16               5814438               196.3 ns/op           130 B/op          1 allocs/op
BenchmarkOneLog
BenchmarkOneLog-16                461516              2397 ns/op              88 B/op          2 allocs/op
i would like to use zerelog as log package for its zero memory allocation.  in additionm, zerolog has more simple api to use although zap is the fastest package. is there other requirement?

Seems like zap is a better choice.

I think the base component can define a interface, the project use this interface, and we impl it use zap first, if we use other log package implement it later an we can compare them.

I think the base component can define a interface, the project use this interface, and we impl it use zap first, if we use other log package implement it later an we can compare them.

@roseduan

I think the base component can define a interface, the project use this interface, and we impl it use zap first, if we use other log package implement it later an we can compare them.

good idea

@roseduan i done this: https://github.com/lotusdblabs/lotusdb/pull/82, may be you can assign to me.


Revert "Feature/html updates"

Reverts mannyigho/foto-music#18
Hi @mannyigho ,
We shouldn't be merging to main if the rest of us haven't reviewed the pull request.
Could you explain what you're trying to do with this code?
I'm reverting the change and waiting for your approval to merge it.

I wanted to show a second API for us to use, that is why I did the merge late last night as I know everyone will be asleep

Kindly go ahead and merge, what I did was not directly on our ongoing project because I created 2 separate files, indext2.html and script2.js to work with

I wanted to show a second API for us to use, that is why I did the merge late last night as I know everyone will be asleep

That's alright we can review when we wake up ;)
Also, if you have added any extra functionality you can reuse the existing files and open a PR that adds this new implementation. There's no need to add extra files

Hi
@eduardo
, what I merged last night was 2 different files I created, script2.js and index2.html to test the second API i was able to lay my hands on. It has no directly link with the files we are working on i.e. index.html and script.js. If you look at the link https://mannyigho.github.io/foto-music/index2.html you would see what I merged last night, just a duplicate of what we are trying to do but it has no direct connection with our project files


List

Add List component to display Bullet or Numbered Lists.
Add size.bullet in theme.

@hansgx Fixed all the changes requested, should be good to merge :wink:


NIFI-461 Add processors to interact with Apache Solr

This pull request provides two processors:

GetSolr - extracts data from Solr by running a query and sending the results out in XML, the intent is to run on a timer, or cron, and only get the results since last execution, similar to GetHttp.

PutSolrContentStream - streams the content of a FlowFile to a Solr update handler, the main uses cases would be indexing json, xml, and csv, but any update handler could be used. MergeContent could be used in front of this processor to combine documents for more efficient submission to Solr. A good overview of how Solr indexes JSON is here: http://lucidworks.com/blog/indexing-custom-json-data/ and this processor could be used to post the JSON.

Both processors support a stand-alone Solr instance, as well as a SolrCloud cluster.

@madrob Can you take a look?

First a caveat - I'm less familiar with the mechanics of the ContentStreamUpdateRequest, but it looks like this should work fine. Does this handle all of insert/update/delete, or just insert? I'm more used to seeing separate docs added like you do in the test code for the GetSolr, which is what I expected the PutSolr to look like.
+1 from me.

Thanks for reviewing Mike.
With regards to the ContentStreamUpdateRequest... I actually hadn't thought about using it for deletes, but I just did a test and it worked. So it should work for almost any add/update/delete example in the Solr documentation. It would just require an upstream processor that was producing the right commands, where as for the case of adding new documents it seems more likely that you already have some data (json, csv, etc) that you can just fork into Solr.
I was thinking there eventually might be a use case for another type of "PutSolr" processor that reads in data in a standard format (maybe avro) and converted to a SolrInputDocument, but seemed like the ContentStream approach gave the most flexibility initially.

I actually hadn't thought about using it for deletes, but I just did a test and it worked.
Awesome! Can you add that test to the Unit Tests?

I was thinking there eventually might be a use case for another type of "PutSolr" processor that reads in data in a standard format (maybe avro) and converted to a SolrInputDocument, but seemed like the ContentStream approach gave the most flexibility initially.
Agreed. Just file a follow-on issue and somebody (maybe you) will get to it when the need arises.

Added a test case that shows a delete by query using xml, ex:

...

<delete>
  <query>...</query>
</delete>


Named Destinations should include GoTo Actions

Clause <IP_ADDRESS> Named destinations should fully allow GoTo Actions as well as the "minified" DestDict.
ActionGoTo, ActionGoToR and ActionGotoE should all be added to DestMap and Name.
See also https://github.com/pdf-association/pdf-issues/issues/288.

ActionGoToE should be wrapped in fn:SinceVersion(1.6,ActionGoToE)


[graphviz,librsvg,libcroco] Refresh

graphviz
Properly cleanup and refresh patches. Amends #33795. Fixes #33926 which actually is my https://github.com/microsoft/vcpkg/pull/33855#issuecomment-1730173103 and which delayed feedback on #33795.
libcroco, librsvg
Revise dependencies.
Consistent use of pkgconfig for import and export. I see no benefit from unofficial exported CMake config when it relies on pkgconfig for imported dependencies.
libcroco: Use official (upstream) header installation dir. Install pc file. Drop unofficial debug postfix.
librsvg: Use official pc file name. Always build pixbufloader-svg as module (DLL), and install it to official location (https://github.com/microsoft/vcpkg/issues/8015#issuecomment-1686919918).

Add an empty comment that the tests will restart and not hangout on license/cla.

Tested usage successfully by librsvg:x64-windows:
librsvg can be imported via CMake FindPkgConfig module:

    find_package(PkgConfig REQUIRED)
    pkg_check_modules(LIBRSVG librsvg-2.0 IMPORTED_TARGET REQUIRED)
    target_link_libraries(main PRIVATE PkgConfig::LIBRSVG)

libcroco:x64-windows:
    find_package(PkgConfig REQUIRED)
    pkg_check_modules(LIBCROCO libcroco-0.6 IMPORTED_TARGET REQUIRED)
    target_link_libraries(main PRIVATE PkgConfig::LIBCROCO)


Remove version: DeaDBeeF.DeaDBeeF version 1.9.1

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/deadbeef/files/travis/windows/1.9.1/deadbeef-1.9.1-windows-x86_64.exe/download
Status Code: 200
(Automated message - build 897)

Close with reason: Package still available.;


143 memory bugs

Reduced the use of dynamic dispatch to unsafe functions and ref-from-ptr casts.

This PR cleans up a lot of code and makes the HeapPrimitive mechanic more scalable.


Add floating point guide as resource on Exact Change

Pre-Submission Checklist

[X] Your pull request targets the staging branch of FreeCodeCamp.
[X] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[X] You have only one commit (if not, squash them into one commit).
[X] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

[X] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[ ] Tested changes locally. Note: Unable to test locally as MDN Links are not currently working on the Beta.
[X] Closes currently open issue (replace XXXX with an issue no): Closes #10074

Description

I added the link to Floating Point Guide to bonfireMDNlinks.js for the Exact Change challenge and then added it to the challenge.

:sparkles: :sparkles: LGTM. :sparkles: :sparkles: Thanks.


Runners not starting

So I followed the manual, and instructions for the setup, I want to use runners for repo, on enterprise, and when runner pods are starting, every time it gets stuck in Waiting: ContainerCreating status. The logs are the following from the controller:
INFO	controllers.GithubActionRunner	Reconciling GithubActionRunner	{"githubactionrunner": "github-actions-runner-operator/runner-pool"} INFO	controllers.GithubActionRunner	Scaling up	{"githubactionrunner": "github-actions-runner-operator/runner-pool", "numInstances": 1} INFO	controllers.GithubActionRunner	Creating a new Pod	{"githubactionrunner": "github-actions-runner-operator/runner-pool", "Pod.Namespace": "github-actions-runner-operator", "Pod.Name": "runner-pool-pod-vp5fk", "result": "created"} DEBUG	controller-runtime.manager.events	Normal	{"object": {"kind":"GithubActionRunner","namespace":"github-actions-runner-operator","name":"runner-pool","uid":"976fafbb-ef42-4763-829a-78f44c022a53","apiVersion":"garo.tietoevry.com/v1alpha1","resourceVersion":"79945520"}, "reason": "Scaling", "message": "Created pod github-actions-runner-operator/runner-pool-pod-vp5fk"} INFO	controllers.GithubActionRunner	Reconciling GithubActionRunner	{"githubactionrunner": "github-actions-runner-operator/runner-pool"} INFO	controllers.GithubActionRunner	Pods and runner API not in sync, returning early	{"githubactionrunner": "github-actions-runner-operator/runner-pool"}
After this, the last two msg's repeating every half minutes or so.

Found this also:
LAST SEEN   TYPE      REASON        OBJECT                      MESSAGE
4m11s       Warning   FailedMount   pod/runner-pool-pod-vp5fk   MountVolume.SetUp failed for volume "settings-xml" : configmap "settings-xml" not found
28m         Warning   FailedMount   pod/runner-pool-pod-vp5fk   Unable to attach or mount volumes: unmounted volumes=[settings-xml], unattached volumes=[default-token-6wrzm docker-storage docker-certs runner-diag runner-work mvn-repo settings-xml]: timed out waiting for the condition
19m         Warning   FailedMount   pod/runner-pool-pod-vp5fk   Unable to attach or mount volumes: unmounted volumes=[settings-xml], unattached volumes=[settings-xml default-token-6wrzm docker-storage docker-certs runner-diag runner-work mvn-repo]: timed out waiting for the condition
49m         Warning   FailedMount   pod/runner-pool-pod-vp5fk   Unable to attach or mount volumes: unmounted volumes=[settings-xml], unattached volumes=[docker-certs runner-diag runner-work mvn-repo settings-xml default-token-6wrzm docker-storage]: timed out waiting for the condition
8m27s       Warning   FailedMount   pod/runner-pool-pod-vp5fk   Unable to attach or mount volumes: unmounted volumes=[settings-xml], unattached volumes=[runner-work mvn-repo settings-xml default-token-6wrzm docker-storage docker-certs runner-diag]: timed out waiting for the condition

Drop the settings.xml config if you don’t need it for maven.

Dropped it, now the following error appeared.
# Authentication Http response code: NotFound from 'POST https://api.github.com/actions/runner-registration' {"message":"Not Found","documentation_url":"https://docs.github.com/rest"} Response status code does not indicate success: 404 (Not Found).
I want to use it on enterprise, where can i change out the api.github.com, is it what can cause the problem?

see https://github.com/evryfs/helm-charts/blob/master/charts/github-actions-runner-operator/values.yaml#L68

I already changed that

I already changed that

Please double check. The env vars should be defined and the library should pick it up. They are named GITHUB_*

Double checked, the vars are applied on install, but the pod logs
controllers.GithubActionRunner	Pods and runner API not in sync, returning early	{"githubactionrunner": "github-actions-runner-operator/runner-pool"} controllers.GithubActionRunner	Reconciling GithubActionRunner	{"githubactionrunner": "github-actions-runner-operator/runner-pool"}

If the 404 is gone it means it can now correctly connect to the github api. Try deleting any stale runners from the admin-console of github, under your org -> settings -> actions -> general/runners

@somatczk does it work? Can this be closed?

Closing due to lack of activity.


Material Absorber and Redstone Ore

The material absorber does not absorb redstone ore

Versions please

An issue related to redstone ore dimlets was recently fixed. Can you see if 5.51 fixes the issue you're having?


Kong custom plugins not loading

kong: kong/kong-gateway:2.8-alpine
konga: pantsel/konga:0.14.9
I have installed a custom plugin in kong, and it is enabled when i checked the kong /plugins/enabled endpoint.

the issue is related to kong enterprise edition not showing the plugin in available_on_server not with konga.


Custom Pagination  Using Django rest framework and Vue.

Proposed title of article
Understanding Custom Pagination Using Django rest framework and Vue.
Proposed article introduction
On most modern web applications and websites, one will need to split the app content for better user experience and readability. This makes the app users focus on small content at a time. To do this with ease, Django comes with a pre-built class: Paginator, which creates and manages paginated data of applications using the default pagination styles.
Sometimes, one may want to customize and use a modified format to paginate data. This article will enable the reader to understand how to customize and modify the default pagination styles to suit users' needs using Django REST APIs with Vue. The reader will also learn the various ways to paginate data; Class-based views and Function-Based views and the importance of web pagination.
Key takeaways
The reader will be able to:

Understand what pagination is and its importance.
Various ways of paginating web apps using Django.
Understand how to customize and modify pagination styles using Django REST API with Vue.

Article quality
The article will help one understand how to use a modified format of the pagination styles by customizing the default pagination styles using Django REST Framework with Vue.
References
N/A
Templates to use as guides
N/A

Good afternoon and thank you for submitting your topic suggestion.
Your topic form has been entered into our queue and should be reviewed (for approval) as soon as a content moderator is finished reviewing the ones in the queue before it.

This topic may crash with another one that was closed some few weeks ago. @dentonya what do you think https://github.com/section-engineering-education/engineering-education/issues/3727


Comment in the beginning of generated files

Is your feature request related to a problem? Please describe.
No indication is given that a file is generated inside the file itself with default settings. I think that this should be the standard to clearly indicate to all developers that a file is generated.
Describe the solution you'd like
Always prepend generated files with a block comment describing that the file has been generated by graphql-code-generator, maybe even which plugins and what config that were used (This might be a security issue, since it might expose version/plugin-related vulnerabilities?).
Something like:
/*
Generated by<EMAIL_ADDRESS>Plugins:
<EMAIL_ADDRESS><EMAIL_ADDRESS>Config:
    federation: true
    useIndexSignature: true
*/

Describe alternatives you've considered
At the moment it is possible, with the help of the add-plugin, to prepend comments describing that the file is generated. This isn't optimal though.
Additional context

Why is it not optimal. Our team for example start the filename with generated, and we need the eslint disabling in the top and also add the info about autogenerating for the users we call developers with just this add block :)
Personally i think add plugin gives full flexibility.

I agree with @maapteh on that. Maybe we can offer a "built-in" plugin called debug or something similar, to print those (you can also create it, should be pretty simple).

@emilwihlander fyi I was able to use the @graphql-codegen/add plugin to prepend a comment at the top of the generated file.


Carto::User does not fallbacks to user Google api key if organization is blank

From https://github.com/CartoDB/cartodb/issues/11655
STR

Set an organization google key to '' (empty string)
Set the user api key to something
Open Builder, you should be able to see the google maps script being loaded

This is currently broken in the AR model only:
  def google_maps_api_key
    if has_organization?
      self.organization.google_maps_key || self.google_maps_key
    else
      self.google_maps_key
    end
  end

If the organization key is blank but not nil, the organization key will be used, even if the user has a proper one. The old models handles this correctly.

Reviving this. This has affected enterprise users in the past, although there doesn't seem to be anyone affected now. But we should address this. Maybe RT if there is a slot there, if not, we'll get to this eventually in Backend.
cc @ramiroaznar

Let's move this to RT then.

Isn't this already fixed here?

^^ @javitonino ?

Yeah! :tada:


Add a Gitter chat badge to README.md

jrudolph/sbt-dependency-graph now has a Chat Room on Gitter
@jrudolph has just created a chat room. You can visit it here: https://gitter.im/jrudolph/sbt-dependency-graph.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Johannes Rudolph » sbt-dependency-graph #23 SUCCESS
This pull request looks good
(what's this?)


Complete Heroku application configuration and deployment

What Issue Are You Seeing or What Feature Needs To Be Added?

[x] Add the following environment variables: ENV, LOGIN_SECRET_KEY, SQLALCHEMY_DATABASE_URI
[ ] Sync repository with Heroku
[ ] Add deploy on Tag creation
[ ] Add Heroku script to make sure application runs
[ ] Make sure database user is setup
[ ] Confirm application runs on Heroku

Are There Any Links That Would Be Helpful?

@anisfeld I'll take care of this tomorrow, apologies for the delay.

Thanks for the update.
On Sun, Dec 10, 2023, 9:48 PM Michael Plunkett @.***>
wrote:

@anisfeld https://github.com/anisfeld I'll take care of this tomorrow,
apologies for the delay.
—
Reply to this email directly, view it on GitHub
https://github.com/prijatelilab/PrijateliTree/issues/78#issuecomment-1849279457,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGAZYZ5OUZXLFLCLKUQXBB3YIZ673AVCNFSM6AAAAAA74UIWC2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBZGI3TSNBVG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


[DOCFIX] Update Chinese Minio Doc

Can one of the admins verify this patch?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10861/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10938/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/11201/
Test PASSed.

alluxio-bot, merge this please


A question about the test output

Hi,
Sorry to bother you.
I have a question about the test.py output.
I found that when I test the model with a sequence, frames indexing from 1 to 300, the output sequence's frames only range from 3 to 297. Frame 1, 2, 298, 299, 300 are lost.  Why does this happen? What should I do If I want to get the lost frames?

Why does this happen?

Because the default model use features from two future frames and two past frames.

What should I do If I want to get the lost frames?

You can duplicate the start and end frames twice, making the sequence have 304 frames.

Why does this happen?

Because the default model use features from two future frames and two past frames.

What should I do If I want to get the lost frames?

You can duplicate the start and end frames twice, making the sequence have 304 frames.

A simple and easy to understand reply, thank you @zzh-tech

A simple and easy to understand reply, thank you @zzh-tech


Set load_balancing_weight with a valid value

Following #9627
envoy api LocalityLbEndpoints.load_balancing_weight  proto definition has a limit, the valid value range is [1, 128], otherwise the config will not be accepted.
https://github.com/envoyproxy/envoy/blob/master/api/envoy/api/v2/endpoint/endpoint.proto#L115-L116
  google.protobuf.UInt32Value load_balancing_weight = 3
      [(validate.rules).uint32 = {gte: 1, lte: 128}];

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: hzxuzhonghu
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
If they are not already assigned, you can assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [hzxuzhonghu]
tests/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hzxuzhonghu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
6206be367d799cb7d1cb582734acb456939bf32b
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
6206be367d799cb7d1cb582734acb456939bf32b
link
/test istio-integ-k8s-tests

prow/istio-integ-local-tests.sh
6206be367d799cb7d1cb582734acb456939bf32b
link
/test istio-integ-local-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
6206be367d799cb7d1cb582734acb456939bf32b
link
/test istio-integ-k8s-tests

prow/istio-integ-local-tests.sh
6206be367d799cb7d1cb582734acb456939bf32b
link
/test istio-integ-local-tests

prow/e2e-mixer-no_auth-mcp.sh
6206be367d799cb7d1cb582734acb456939bf32b
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
6206be367d799cb7d1cb582734acb456939bf32b
link
/test istio-integ-k8s-tests

prow/e2e-mixer-no_auth-mcp.sh
6206be367d799cb7d1cb582734acb456939bf32b
link
/test e2e-mixer-no_auth-mcp

prow/istio-integ-local-tests.sh
65a4846b5429605bd8812939202594ef20b9a424
link
/test istio-integ-local-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
6206be367d799cb7d1cb582734acb456939bf32b
link
/test e2e-mixer-no_auth-mcp

prow/istio-integ-local-tests.sh
65a4846b5429605bd8812939202594ef20b9a424
link
/test istio-integ-local-tests

prow/istio-integ-k8s-tests.sh
65a4846b5429605bd8812939202594ef20b9a424
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-local-tests.sh
65a4846b5429605bd8812939202594ef20b9a424
link
/test istio-integ-local-tests

prow/istio-integ-k8s-tests.sh
65a4846b5429605bd8812939202594ef20b9a424
link
/test istio-integ-k8s-tests

prow/e2e-mixer-no_auth-mcp.sh
65a4846b5429605bd8812939202594ef20b9a424
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hzxuzhonghu, rshriram
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [hzxuzhonghu,rshriram]
tests/OWNERS [rshriram]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


adding html file

Adding a simple calculator.

Sure
On Fri, 8 Oct 2021, 11:47 1dharode, @.***> wrote:

Hi Dear @Abijitchettri99 https://github.com/Abijitchettri99
Great code
Thanks for your contribution......
Show your support on my youtube channel web Pixar(The institute of coding)
like , share , subscribe
youtube - https://youtu.be/5qnQjgdpFPg
telegram-channel - https://t.me/WebPixar
join our telegram channel for more goodies , swag packs
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/1dharode/Hacktoberfest2021/pull/168#issuecomment-938375769,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARHJXDBPRB345GIXNQYKFULUF2EI3ANCNFSM5FNLCLSA
.


[hsqldb] Invalid DDL Generation

Using v4.2.2. The ddl generation of tables that contain default values produces incorrect. This appears to be consistent in various places where ddl is created for tables.
CREATE TABLE PUBLIC."SESSION" (
ID BINARY NOT NULL DEFAULT X'00000000000000000000000000000000',
ENTITYID BINARY NOT NULL,
CREATEDON DATE DEFAULT CURRENT_DATE,
ACCESSEDON DATE DEFAULT CURRENT_DATE,
CONSTRAINT SESSION_PK PRIMARY KEY (ID),
CONSTRAINT SESSION_PROPRIETOR_FK FOREIGN KEY (PROPRIETORID) REFERENCES PUBLIC.PROPRIETOR(ID)
) ;
Expected behaviors:

Output the default specifier before the constraint like so:
ID BINARY DEFAULT X'00000000000000000000000000000000' NOT NULL
Output the length for types such as BINARY, VARBINARY, VARCHAR, etc:
SESSIONID BINARY(16) NOT NULL

Hope this helps, and thanks for publishing this tool!

Yes, it is a bug in DDL generator. Thanks for report

My pleasure to help. Thanks for the great tool.


Automatic Type Acquisition based on filename matching pulls in too many irrelevant typings

Consider a simple JavaScript project with files like the following
/flight.js
/platform.js
/myfile.js

The names flight and platform seem innocuous enough, but their mere presence causes Automatic Type Acquisition to pull in @types/flight and @types/platform. Because these words are so common, I would argue it's highly unlikely that the user actually needs the typings for these libraries - much more likely that the files are just regular files in the user's project. I think ATA's behavior is a little too eager, and causes unnecessary files and symbols to be added to the project context. There's also no visual indication that this is actually happening (but you can confirm it by checking TS Server logs).
This behavior happens in VS Code if you have the files open in the editor. But it occurs in VS if you have them present in your project at all.
I propose one or more of the following solutions:

Disable this filename-matching behavior entirely
Trim back the allow-list considerably to only include obvious libraries like jquery.
Give a visual indicator to the user that ATA is doing this, and give them a way to disable it

@DanielRosenwasser

This has heuristic that is at:
https://github.com/microsoft/TypeScript/blob/31f495749452c7f3046ea3d621282f72312a7a73/src/server/project.ts#L1711
where it does ATA if the resolved module is not ts or json and is referenced using non relative name. This is been around for some time but i think the reason was that eg if you resolved to node_modules\mocha you instead want .d.ts file for it instead. May be adding check for isExternalLibraryImport might be a good addition to check there.

It's been like this forever yes, I think the original intent (@RyanCavanaugh would know) was to detect third party libraries in non-module based code, e.g. a project that just has jquery.js or jquery.min.js copied in. There are lots of existing VS projects that do this. How do we detect that that file is a dependency, and not user code? At the time we assumed filename matching with an allow-list might be a good heuristic. I'm not so sure of that anymore.

isExternalLibraryImport property of the resolution would let us know that since that indicates the resolution is from node_modules

Thanks for the pointer. To be clear the scenario I describe, there is no node_modules, and we still intend to do ATA.

I think a good solution is a combination of bullet points 2 + 3.

Give a visual indicator to the user that ATA is doing this, and give them a way to disable it

VSCode already shows a status bar for this message and I intend to add one to VS:

Contributors for other editor will likely find this issue and if they are also trying to address ATA notifications.

Trim back the allow-list considerably to only include obvious libraries like jquery.

We have data from npm and VS that tells us which packages are downloaded most frequently. We can shorten the list to the top 10-20 from those lists. These include the expected popular packages like lodash, jquery, and bootstrap.
With both changes, users would be getting typings only for popular packages that are probably the jqueryies they are looking for, and if not, be aware of what they are getting through the notification.


[docker-compose] Bump up version to 3.1

Ref.: https://github.com/backmeupplz/shieldy/issues/119
Thank you for the suggestion, @DolanKivert

Thank you!


Log ring in stress tests runs

I have not created the column in kusto yet
https://github.com/microsoft/FluidFramework/pull/15054

There is no need to create anything in Kusto - columns are auto-created once events start flowing.
But clearly this PR is not enough, as proper implementation should get rid of frsCanary term from repo. The way it's structured right now will break existing flow.


[Migration] It's impossible to migrate entities if their parent folders contains "." or not allowed symbols at the end

EPAM AI DIAL chat version
0.7.0
What happened?
It's impossible to migrate entities if their parent folders contains "." or not allowed symbols at the end
Confidential information

[X] I confirm that do not share any confidential information

Verified on staging successfully. Important to check on Sales env.- waiting for 0.6.13 patch there

Documented in user guide


GD script debugger jumping around weird places

Godot version: 3.1.1
OS: Windows 7 Ultimate 64bit
I am having a issue where the godot debugger is reading code inside a condition before actually checking the condition.
I don't know if it actually executes it, but currently this makes debugging really hard.

download this project: https://drive.google.com/open?id=1fo3fr8KEY3eLLqnh1DI_n5TZexbnZeg9

make sure breakpoint is in Player.gd line 31.

debug Player scene and press up.

note debugger position

use debugger to step into code

note its now at the if statement instead of the next line where it sets the animation.

use debugger to step into code (again)

note its now at where it sets its animation.

Seems to be working correctly in 3.2.3


fix the broken link to a true branch

What this PR does / why we need it:
Which issue this PR fixes(if applicable):
fixes #
Special notes for reviewers:

Release note:

NONE

/kind: bug
fix a broken url to a true url

why it's broken?

why it's broken?
I can't find the file in master branch


[profile] broken social login via Torus

Creating a profile or editing an existing one with Torus is currently failing. Some info from Selim and Jabby: "It says i rejected the approval with my torus wallet even though the torus page isn't even done loading".
First screenshot shows wallet loading loop on attempted profile creation, second screenshot is console while trying to edit an existing profile.

For what it's worth: no issues when creating/editing profiles with Metamask.

@0xtapi @sprrwhwk has mentioned that this is working again? Can you double check?

@0xtapi @sprrwhwk has mentioned that this is working again? Can you please double check?

Just tested, seems to work. Created a profile and edited a profile, no issues. The bugs have begun fixing themselves... wonderful ;)

This seemed to be a torus issue based on the error message. If this problem reoccurs we can try upgrading @toruslabs/torus-embed in lib package to the latest version (4.0 -> 4.2.1)


Implement loading justfile from stdin

I think it would be nice to have the possibility to pipe justfiles into just, like so:
cat input.txt | just --dump --justfile - > output.txt

It would work by detecting a special symbol "-" as the -f / --justfile argument.
Benefits:

Some code editors like helix require this in order to make auto formatting to work,
Many command line tools work like this and it seems it will not hurt to have,

The plan:

Move Search{} functionality into Source{}
Implement Source::from_stdin()
Add "-" detection to the argument parser.

Here is the first of the commits, more commits will be added while work progresses.

This seems reasonable, but this PR looks a bit too complicated and I think some of the changes are unnecessary.
I would create two new SearchConfig variants, WithStdin, and WithStdinAndWorkingDirectory, and handle those two cases separately in Search::find.

@casey I have added the variants, now we need to change Search.justfile type because it is a PathBuf which does not fit to the stdin stream. It seems to me that changing it to an Rc<dyn std::io::Read> or a custom JustfileSource would be some options. What do you think?

There are two options I can think of:

Add an additional field to Search, which is a string containing the loaded source
Make Search::justfile an enum, like JustfileSource

I would try out 1 first and see how it looks and see if it causes any problems, or requires a big refactor.

Sorry for closing and reopening. It was by mistake.
@casey Please have a look at the new commit. I have added a JustfileKind enum to the Search struct. It does not feel right, but I have no idea how to make it better without a bigger refactor. Let me know what you think.

This looks like a good approach to me!

Hi @casey I have changed a little the implementation of this PR. Previous approach did not felt right for me. I think the new approach makes more sense. However, I stumbled upon a failing integration test: working_directory::justfile_without_working_directory. It seems it runs the command in a subshell, which makes it hard to find out what's the problem. Any advice on how to trace this?
Let me know what you think about the new changes.
Thanks!


returned with exit code 2 error

Hi,
Have followed the instruction https://github.com/Lookingglass/Maltego but get the "returned with exit code 2 error" when i run the transform am using private virus total api and Carbon CE 3.5.3
Please advice
Mike

Hi Mike,
What URL are you trying to scan? Also, can you confirm this private API key
is one that is paid for through their subscription service?
Thanks,
Steven
On Sun, May 3, 2015 at 4:38 AM, MikeLinney<EMAIL_ADDRESS>wrote:

Hi,
Have followed the instruction https://github.com/Lookingglass/Maltego but
get the "returned with exit code 2 error" when i run the transform am using
private virus total api and Carbon CE 3.5.3
Please advice
Mike
[image: maltego-error]
https://cloud.githubusercontent.com/assets/11976463/7444395/12ef5c24-f178-11e4-9810-986acefd4cac.PNG
—
Reply to this email directly or view it on GitHub
https://github.com/Lookingglass/Maltego/issues/7.

--
Regards,
Steven Weinstein
Malware Researcher
Lookingglass Cyber Solutions, Inc.

Hi Steven,
Have tried a few urls including www.nullit.net. Virus Total issued a  private key with special academic researcher privileges.
Thanks
Mike

Hi Mike,
Trying that example worked for me, with the expected output of " Scan
request successfully queued, run urlToDetectionRatio later".
If you can confirm that you are trying to run that transform with that URL
having a "domain" entity type (domain entity type is required for this
transform), you may want to ask support at VirusTotal if the API key they
have provisioned you allows for URL submissions.
Thanks,
Steven
On Sun, May 3, 2015 at 5:47 AM, MikeLinney<EMAIL_ADDRESS>wrote:

Hi Steven,
Have tried a few urls including www.nullit.net. Virus Total issued a
private key with special academic researcher privileges.
Thanks
Mike
—
Reply to this email directly or view it on GitHub
https://github.com/Lookingglass/Maltego/issues/7#issuecomment-98460902.

--
Regards,
Steven Weinstein
Malware Researcher
Lookingglass Cyber Solutions, Inc.

Steven,
Thanks very much for all you help. Done a complete re-install of Maltego and problem solved, not sure what caused the error..but its all working fine now.
Regards
Mike

Hi Mike,
Glad to hear that solved the problem, sometimes that's what's needed!
Steven
On Thu, May 7, 2015 at 7:06 PM, MikeLinney<EMAIL_ADDRESS>wrote:

Steven,
Thanks very much for all you help. Done a complete re-install of Maltego
and problem solved, not sure what caused the error..but its all working
fine now.
Regards
Mike
—
Reply to this email directly or view it on GitHub
https://github.com/Lookingglass/Maltego/issues/7#issuecomment-100043722.

--
Regards,
Steven Weinstein
Malware Researcher
Lookingglass Cyber Solutions, Inc.


Removed dependence on jQuery except in tests

Since Ember itself seems to be moving away from the use of jQuery in the core library, this PR goes ahead and removes the dependence for the main addon (though I didn't remove it from tests). This allows using ember-islands with a project that doesn't otherwise include (or want to include) jQuery.

This looks great thanks! I'll merge this in and then probably follow up with a minor release.

Awesome! Thanks! 😁

Released as 1.5.0


TCP/IP printer connection suddenly breaks

Hi
I am trying to use a wifi/serial converter between printer and repetierHost (TCP/IP connection, not a Repetier Server).
using: RepetierHost version 2.1.3 w/ linux mono v4.6.2 (mono --verify-all RepetierHost.exe)
Connection works for simple things (switch on fans, temp, simple moves).
Once printing has started, the connection is broken from repetierHost side few seconds later.
The converter itself is quite stable and able to transmit megabytes @ 115k for hours with a loopback wire and data checker on tcp side.
All is fine when using only usb connection. Printer is reprap (w/ eMotronic board).
A tcpdump / wireshark shows the issue (net.8 is repetier on the capturing pc, net.249 is the converter):
Complete dump for wireshark: repetierHost-FIN.pcapng.gz
We can see that repetier is closing its sending way by sending a TCP-FIN twice.
The converter is correctly answering by TCP-RST (closes both sides).
Anything that could help debugging this issue is welcome,
Thanks!

I also tried to cheat by using a virtual serial port under linux using
socat -d -d pty,raw,echo=0 tcp:<IP_ADDRESS>:8888

That creates a dynamic serial port like /dev/pts/24 that I set into repetierHost instead of usual /dev/ttyACMx or /dev/ttyUSBx. Socat would manage virtualserial<->tcp transmission to the wifi tcp<->serial converter plugged to the printer.
But repetierHost / mono do not want to use that virtual device, saying:
Opening serial failed on second try:System.IO.IOException: Invalid argument
  at System.IO.Ports.SerialPortStream.ThrowIOException () [0x00012] in <bd46d4d4f7964dfa9beea098499ab597>:0
  at System.IO.Ports.SerialPortStream.SetSignal (System.IO.Ports.SerialSignal signal, System.Boolean value) [0x00042] in <bd46d4d4f7964dfa9beea098499ab597>:0
  at (wrapper remoting-invoke-with-check) System.IO.Ports.SerialPortStream:SetSignal (System.IO.Ports.SerialSignal,bool)
  at System.IO.Ports.SerialPortStream..ctor (System.String portName, System.Int32 baudRate, System.Int32 dataBits, System.IO.Ports.Parity parity, System.IO.Ports.StopBits stopBits, System.Boolean dtrEnable, System.Boolean rtsEnable, System.IO.Ports.Handshake handshake, System.Int32 readTimeout, System.Int32 writeTimeout, System.Int32 readBufferSize, System.Int32 writeBufferSize) [0x00057] in <bd46d4d4f7964dfa9beea098499ab597>:0
  at (wrapper remoting-invoke-with-check) System.IO.Ports.SerialPortStream:.ctor (string,int,int,System.IO.Ports.Parity,System.IO.Ports.StopBits,bool,bool,System.IO.Ports.Handshake,int,int,int,int)
  at System.IO.Ports.SerialPort.Open () [0x00078] in <bd46d4d4f7964dfa9beea098499ab597>:0
  at RepetierHost.model.ProtectedSerialPort.Open () [0x00037] in <4b6b78851490464eaa6ae55615a8e4c5>:0
  at (wrapper remoting-invoke-with-check) RepetierHost.model.ProtectedSerialPort:Open ()
  at RepetierHost.connector.SerialConnector.Connect (System.Boolean failSilent) [0x00220] in <4b6b78851490464eaa6ae55615a8e4c5>:0

Then I came across this repository https://github.com/JTrotta/MonoSerialPort that might help using socat. I have no idea whether it could be tried, but it sounds like designed for that purpose.
Again, anything that could help debugging this issue is welcome.
Thanks


Update Fork - consistency

Description
This pull request includes changes to the scenarios/Assistants/api-in-a-box directory, primarily focusing on improving the documentation and code in the README.md and .ipynb files for the api-in-a-box scenarios. The changes are aimed at enhancing the clarity and readability of the documentation, as well as refining the code for better functionality and understanding.
Documentation improvements:

scenarios/Assistants/api-in-a-box/README.md: Modified the description of the use case to make it more concise and clear.
scenarios/Assistants/api-in-a-box/README.md: Updated the description of the Failed Banks and added a new Sales Assistant in the list of use cases.
scenarios/Assistants/api-in-a-box/math_tutor/README.md: Updated the description of the objective of the provided Python file.

Codebase improvements:

scenarios/Assistants/api-in-a-box/failed_banks/assistant-failed_banks.ipynb: Removed unnecessary newline characters and updated the comments for better understanding. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]
scenarios/Assistants/api-in-a-box/math_tutor/assistant-math_tutor.ipynb: Updated the description of the provided Python file and removed unnecessary newline characters. [1] [2] [3] [4] [5]

Please note that these changes are aimed at enhancing the user experience and understanding of the api-in-a-box scenarios.

Checklist

[ ] I have read the contribution guidelines
[ ] I have coordinated with the docs team<EMAIL_ADDRESS>if this PR deletes files or changes any file names or file extensions.
[ ] This notebook or file is added to the CODEOWNERS file, pointing to the author or the author's team.

Thank you for the contribution!
Please see this section in the contributing guide to get pre-commit set up, so you can remedy your CI issues: https://github.com/Azure-Samples/azureai-samples/blob/main/CONTRIBUTING.md#3-set-up-pre-commit


Remove PHPStan from pre-commit hook

This removes PHPstan from our precommit hook. While that was well intentioned, I believe from discussions we've had that most of the team actually finds that more trouble than it is worth. I believe it causes a choke point for potential contributors who get stuck trying to pass that and never actually submit anywhere. With it gone, they can submit and open up a discussions about the code much easier.
At one point discussions mentioned removing the entire pre-commit hook, but I find value in the linting and auto-fixing of style issues where it can, and believe that stan was the biggest pain point in the process.

If we accept this, why don't we go with develop?

Rebased onto develop branch.


[CARBONDATA-1837] Reusing origin row to reduce memory consumption

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[X] Any interfaces changed?
ADDED TWO INTERFACE FOR CARBON_ROW_BATCH
[X] Any backward compatibility impacted?
NO
[X] Document update required?
NO
[X] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
NO. ONLY FOR OPTIMIZATION
- How it is tested? Please attach test report.
TESTED IN CLUSTER OF 3 NODES
- Is it a performance related change? Please attach the performance test report.
PERFORMANCE SHOULD BE ENHANCED A BIT, SINCE MEMORY CONSUMPTION IS REDUCED
- Any additional information to help reviewers in testing this change.
NO
[X] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
NOT RELATED

NOTES:
In data converting process of data loading, Carbondata will convert each row to another by batch.
Currently, it will create a new batch to store the converted rows, which I think can be optimized to reuse the old row batch's space, thus will reduce memory consumption and GC related overhead.
Tests have been done in my local cluster, enhancement gained is not obvious (maybe the tests didn't reach the memory shortage of the cluster), but the performance didn't become worse than that of before.

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1581/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1979/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1589/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1665/

retest this please

Build Success with Spark 2.2.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/428/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1699/

Build Success with Spark 2.2.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/492/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1755/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/2138/

LGTM, thanks for working on this


Might be good to also show how to update Properties to something else instead of setting it to @()

OR at least a small comment of:

$kv = Get-AzKeyVault -VaultName myvault
$kv # this outputs all properties of object, including accesspolicies.

'# $vault.properties.accesspolicies  can be updated with real applications/users/rights so that it does not need to be done after this whole activity.'

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

ID: dbe3afaa-c19a-65f3-f562-e9c698e4c6a3
Version Independent ID: 66556728-72ef-769a-d666-c1a67d7c39fe
Content: Change the key vault tenant ID after a subscription move - Azure Key Vault
Content Source: articles/key-vault/key-vault-subscription-move-fix.md
Service: key-vault
GitHub Login: @amitbapat
Microsoft Alias: ambapat

@peledins Thank you for the feedback! We are investigating this and will get back to you shortly.

@peledins thanks again for letting us know about this. We will look into this and try to add this into the docs accordingly.

@peledins can you actually clarify what you mean in regards to 2.?
$vault.properties.accesspolicies can be updated with real applications/users/rights so that it does not need to be done after this whole activity.
Are you suggesting showing how to update the access policies at this point in time?

I am suggesting that at place of initializing the variable to empty array, you could at least add a comment saying “Rights can be updated directly in to this array”.
There is some positive effect out of this: there is no interruption where the vault is unfunctional due to no accesspolicies (whereas running set-accesspolicies later on gives a time where vault has 0 access policies and is not functional).
Peteris
From: Frank Hu<EMAIL_ADDRESS>Sent: Wednesday, July 31, 2019 5:48 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Peteris Ledins<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Might be good to also show how to update Properties to something else instead of setting it to @() (#36059)
@peledinshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpeledins&data=02|01|peledins%40microsoft.com|b156f760735e4e2daef008d71619dd32|72f988bf86f141af91ab2d7cd011db47|1|0|637002172656836347&sdata=1%2BKyUe6kpyDuARaj4iGLRiPQRo7N6M2Absii%2Ba%2Fi6tc%3D&reserved=0 can you actually clarify what you mean in regards to 2.?
$vault.properties.accesspolicies can be updated with real applications/users/rights so that it does not need to be done after this whole activity.
Are you suggesting showing how to update the access policies at this point in time?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F36059%3Femail_source%3Dnotifications%26email_token%3DAK7MHLQXKW4W4RFNUPX7UN3QCIXC7A5CNFSM4IHVWOQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3I66BQ%23issuecomment-517074694&data=02|01|peledins%40microsoft.com|b156f760735e4e2daef008d71619dd32|72f988bf86f141af91ab2d7cd011db47|1|0|637002172656836347&sdata=srI%2F8fFoQGLK4K3MjRG8PDeHHtrTudW%2BPWsAnBlT19Y%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAK7MHLTWBO45S4LL2EK6JVDQCIXC7ANCNFSM4IHVWOQA&data=02|01|peledins%40microsoft.com|b156f760735e4e2daef008d71619dd32|72f988bf86f141af91ab2d7cd011db47|1|0|637002172656846341&sdata=qGJPiLpfDU1VTvRlElc19Db9R1RRmMDLl7sAHQRc1mg%3D&reserved=0.

I see, okay thanks for that! We're currently looking into this issue, as we have made a note of this issue, I will go ahead and close this issue out by end of day tomorrow. Please let us know if you'd like to keep this open further.

Hey, similar to this git issue : https://github.com/MicrosoftDocs/azure-docs/issues/35886
Please file a new git issue if you have anymore concerns. I'll go ahead and close this issue out as the doc change is currently in progress.
Please reopen it if you have any concerns.


Travis Update

Thanks for keeping these links up to date...
Really thanks a lot

:tada:

And the build is passing again!
Good job!

:sweat_smile:


[ Suggestion ] Pinned elements should follow as you scroll.

The elements you pin should follow as you scroll through the elements box.

Before implementing this feature, I actually had a talk with active people in the Infinite Craft discord server, and asked whether pinned elements should follow as you scroll. We settled on "it probably shouldn't".
We now have people with pinned element lists so long that they take up their entire sidebar, and implementing this would render their sidebar unusable. I think it was the right call.


Feat:  Add "Forgot Password" Functionality

Describe the feature
Implement a "Forgot Password" feature to allow users to reset their passwords if they have forgotten them. This feature should include a link on the login page that guides users through the process of resetting their password.
Add ScreenShots
Implement the Forgot Password Feature
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Kindly, explain how you are going to proceed?

I have an intuition for this question ...
will use the nodemailer package to send mail to the person have that email with an OTP...
and will create a page to seek the OTP from the user ...
If it is right will proceed to the next step of changing the password of the user
I am a gssoc contributor...
Do assign me this issue along with the required labels so that I could start working on it

Ok, go ahead! @ROKUMATE

@hustlerZzZ
Sir there are many error in the repo while running also
which are caused by recent 2-5 days pull requests

There were more in the components/nav.jsx file also
I would recommend create a new issue and merge a pr for solving recent all this errors so it will be easy for others to fork and code


Fix leftover std::holds_alternative usage

Use hpx::holds_alternative instead.

Performance test reportHPX PerformanceComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch++InfoPropertyBeforeAfterHPX Commit254db57ab5e998619e49744062d14609f1b466daf41dd7b45ccc1f02f4712cf33421deb8491e08d9HPX Datetime2021-08-21T11:38:52+00:002021-11-05T10:30:57+00:00ClusternamedaintdaintHostnamenid00446nid01359Compiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0Datetime2021-08-23T15:21:22.855927+02:002021-11-05T13:40:27.086382+01:00EnvfileComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add-(=)-Stream Benchmark - Scale(=)(=)(=)Stream Benchmark - Triad---Stream Benchmark - Copy(=)(=)(=)InfoPropertyBeforeAfterHPX Commit916f4d8a700533b3273012ecf811d63617592bb2f41dd7b45ccc1f02f4712cf33421deb8491e08d9HPX Datetime2021-10-22T15:54:13+00:002021-11-05T10:30:57+00:00ClusternamedaintdaintHostnamenid01196nid01359Compiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0Datetime2021-10-25T11:52:24.316784+02:002021-11-05T13:40:42.477422+01:00EnvfileExplanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…


UI Responsiveness Fixes - modify breakpoints

Describe the bug
Some breakpoints need to be tweaked. See screenshots below.
To Reproduce
Modify window width on Bonds & Stake screens until you see the view overflow the screen width. Modify the breakpoints to fix.
Expected behavior
The view should not overflow the horizontal width of the window.
Screenshots
Bonds (Bonds buttons are overflowing):

Staking (TVD column is overflowing):

Hey @JanaTod, sorry I didn't see that you were also working on this issue. We can compare solutions when you are done if you want


Units don't seem to work correctly

Hello,
Here is a small example with units:
using ModelingToolkit, MethodOfLines, OrdinaryDiffEq, DomainSets
using Unitful

function ModelingToolkit.get_unit(a::Pair{Num, Float64})
    ModelingToolkit.get_unit(a[1])
end

@parameters x [unit=u"m"]
@parameters t [unit=u"s"]
@variables u(..) [unit=u"kg"]
Dt = Differential(t)
Dx = Differential(x)

x_min = t_min = 0.0
x_max = t_max = 1.0

@parameters α = 10. [unit=u"m/s"]

eq = [Dt(u(x,t)) ~ α * Dx(u(x,t))]

@assert ModelingToolkit.get_unit(eq[1].lhs) == u"kg/s"
@assert ModelingToolkit.get_unit(eq[1].rhs) == u"kg/s"

domains = [x ∈ Interval(x_min, x_max),
              t ∈ Interval(t_min, t_max)]

bcs = [u(x,t_min) ~ 0.0,
       u(x_min,t) ~ 0.0,
       u(x_max,t) ~ sin(1.0),
]

@named pdesys = PDESystem(eq,bcs,domains,[x,t],[u(x,t)], [α => 10.])

discretization = MOLFiniteDifference([x=>6], t)
@time prob = discretize(pdesys,discretization)

It works if there aren't any units, and as you can see the units on the two sides of the equation match. However, running the code to the end gives this error: ERROR: ModelingToolkit.ValidationError("Some equations had invalid units. See warnings for details.")
Here are the warnings in question:
┌ Warning:  in eq. #1left, in comparison >, units [m s⁻¹] and [] do not match.
└ @ ModelingToolkit ~/.julia/packages/ModelingToolkit/rrbUl/src/systems/validation.jl:152
┌ Warning:  in eq. #2left, in comparison >, units [m s⁻¹] and [] do not match.
└ @ ModelingToolkit ~/.julia/packages/ModelingToolkit/rrbUl/src/systems/validation.jl:152
┌ Warning:  in eq. #3left, in comparison >, units [m s⁻¹] and [] do not match.
└ @ ModelingToolkit ~/.julia/packages/ModelingToolkit/rrbUl/src/systems/validation.jl:152
┌ Warning:  in eq. #4left, in comparison >, units [m s⁻¹] and [] do not match.
└ @ ModelingToolkit ~/.julia/packages/ModelingToolkit/rrbUl/src/systems/validation.jl:152

It looks like what is happening is that the upwind differencing is comparing the velocity to zero, but the zero doesn't have any units, which causes the problem. I also tried advection_scheme=WENOScheme(), but it gave a similar error.

Running into the same issue - unit validation passes on the PDESystem but not on the discretized system. I can post an MWE for my case if it'd help

I guess the immediate successful case would be that the example above would run without error. As far as I can tell it's just an issue of the constants that are added being units (e.g. when the upwind scheme checks for u > 0, the 0 is unitless but u is not).


The change of all the Titles (AboutUs, ContactUs,Home,Rate,Contributors)

Title and Issue number

Title :The change of all the Titles (metaverse)
Issue No. :#941
Tech Stack : CSS,JS,HTML
Close #<issue_no>

Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Add WEB3 project
[ ] Other (specify): _______________

Video (mandatory)

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have gone through the  contributing.md file before contributing
[ ] I have Starred the Repository.

Checklist:(In case of Adding New WEB3 project only)

[ ] I have made my Commits(of new project) under the  Projects folder
[ ] I have created a helpful and easy to understand README.md
[ ] i have added the image of my project under asstes/img folder

Additional context (Mandatory )
Are you contributing under any Open-source programme?

[X] I'm a GSSOC'24 extended contributor
[ ] I'm a Winter of Blockchain(WOB'24) contributor
[ ] I'm a Hactoberfest'24 contributor

@apu52 hey i made the changes please change !!


README minor typo fix

Thanks for the typo fix. We should probably have this listed, but can you pull req this into the develop branch instead of master.


Integrate Tallycoin Connect

Closes #1450

Thax :) Marked for final testing & to be part of v1.8.0 release.

Marked for final testing for v1.7.2 release.


Navbar enhancing

Navbar is looking so dull and simple. I have a nicer idea to make it more beautiful and eye-catchy. Kindle please assign me this task @khushi-joshi-05 . Here's my LinkedIn profile : https://www.linkedin.com/in/bharti-jangir-9a719a264/

i would like to resolve this issue

Navbar is not sticky. It is not simple and responsiness. I have good idea to make attractive contact and cart make to left @khushi-joshi-05 . So,Kindly please assign to me.