Querying Unity Catalog with DuckDB UC extension requires retrying the first SQL statement

The information below has been share with the Databricks UC engineering team. They suggested the issue is likely with the DuckDB UC  extension.
Credential serving is enabled on the Databricks instance the code is connecting to.
Duckdb CLI installed on MacOS - Retry SHOW TABLES query

R - Retry SHOW TABLES query

Python - Retry SHOW TABLES query
Additionally even after a successful SHOW TABLES query no tables are found and the table queried successfully in R cannot be queried using Python.

At least part of the problem appears to be that the extension is trying to convert column types to an internal type:
https://github.com/duckdb/uc_catalog/blob/71e18a9d1ecb65acf4f9b5f0414b214a15025dab/src/uc_utils.cpp#L32-L144
But it does not have a mapping for "varchar(xxxx)", so it throws an exception:
https://github.com/duckdb/uc_catalog/blob/71e18a9d1ecb65acf4f9b5f0414b214a15025dab/src/uc_utils.cpp#L146

Hi @edwardsdm, thanks for reporting this. We will take a look at this at some point. Currently we have prioritized the development of the Delta extension over the uc_catalog extension so I can not give a timeline yet!

Hey team, this issue is also happening to me and I could reproduce exactly what was described here. I can read the metadata for all tables normally after the 3rd attempt, WHICH IS SUPER COOL. But I can't query anything due to permission errors on S3, in my org we need to query that through UC and not using the S3 path directly (that I believe is what is happening under the hood with the Delta extension).

@samansmink
Adding these 4 lines seems generally to fix the issues in our environment:
	} else if (type_text.find("varchar") == 0) {
	    return LogicalType::VARCHAR;
	} else if (type_text.find("char") == 0) {
	    return LogicalType::VARCHAR;

before:

https://github.com/duckdb/uc_catalog/blob/main/src/uc_utils.cpp#L58


SIGSEGV when closing windows on X

Bevy version
4ce37395da029eb8b58c78c32812fe8329a3a1a4
[Optional] Relevant system information
Rust version: 1.69.0
Linux 6.3.5-arch1-1, on X11
If your bug is rendering-related, copy the adapter info that appears when you run Bevy.
AdapterInfo { name: "NVIDIA GeForce GTX 1050", vendor: 4318, device: 7309, device_type: DiscreteGpu, driver: "NVIDIA", driver_info: "530.41.03", backend: Vulkan }

What you did
cargo run --example <any example>
hit close button on window
What went wrong
first logged, as normal:
2023-06-01T12:58:18.583448Z  INFO bevy_winit::system: Closing window 0v0
Then program freezes.
Sometimes program also crashes with SIGSEGV
Additional information
On Wayland (without the wayland feature, i.e. running through XWayland) window closes perfectly fine, problem is only on Xorg

@A-Walrus Is this still an issue in recent bevy? If so which window manager are you using?

I'm unable to reproduce this anymore, closing


CloudFormation now suppports Route 53's Private Hosted Zones

Hello all - looks like CF now supports private hosted zones:
http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-route53-hostedzone.html#cfn-route53-hostedzone-vpcs
but realized to my sadness that Troposphere does not.

It supported some parts of hostedzones but I just pushed some additional changes. Check master.

Mark - thanks for your incredibly fast response to this!
I'm trying the following sort of code:
from troposphere import Template
from troposphere.route53 import HostedZone

t = Template()
t.add_version("2010-09-09")
t.add_description(""" test of creating a private DNS zone """)

t.add_resource(HostedZone(
    "DNSZonePrivate",
    Name="example.com",
    VPCs={"VPCId": "foo", "VPCRegion": "us-east-1"},
))

but I'm getting this error:
TypeError: VPCs is <type 'dict'>, expected [<class 'troposphere.route53.HostedZoneVPCs'>]
Am I doing something wrong?

Here's the snippet I used for my testing last night:
t.add_resource(HostedZone("DNS",
    HostedZoneConfig=HostedZoneConfiguration(
        Comment="My hosted zone for example.com",
    ),
    Name="example.com",
    VPCs=[
      HostedZoneVPCs(VPCId="vpc-abcd1234", VPCRegion="ap-northeast-1"),
      HostedZoneVPCs(VPCId="vpc-efgh5678", VPCRegion="-west-2"),
    ],
    HostedZoneTags=Tags(
      SampleKey1="SampleValue1",
      SampleKey2="SampleValue2",
    ),
))

Note: I was debating about using positional parameters (below) for HostedZoneVPCs but left the above syntax:
HostedZoneVPCs("vpc-abcd1234", "ap-northeast-1")
I never know if new parameters will be added so at least it safeguards it for future use.

Mark - Fantastic, thank you! Works like a charm.

Can we create a public hosted zone using troposphere..? If so can you please give me a snippet on how to do that where the hosted zone should log on the query calls to the cloudwatch log group.


Build index-management that requires artifacts from a previous component.

Signed-off-by: dblock<EMAIL_ADDRESS>Description
Adds the capability for a script to consume previously built components. This is needed for index-management that needs a job-scheduler and notifications plugin to build.
If you know a way to disable certain tests in that one to avoid needing these, I'd probably prefer that.
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

I am approving, but that is a lot of copypasta - I wish I knew a better way

I know :(

Both these projects decided to continue checking in the .zips, and this was needed to run integration tests only, so we don't need this.
https://github.com/opensearch-project/index-management/pull/121
https://github.com/opensearch-project/dashboards-reports/pull/140


Relation to libsmacker

It seems nice to have independent implementation, but have you ever heard about the libsmacker lib written in C? http://libsmacker.sourceforge.net/
GH mirror / fork: https://github.com/JonnyH/libsmacker
Found it some minutes ago.  As both projects rely on information from multimedia.cx, have you thought about contributing to the lib or vice versa?

Hi @sskras!

It seems nice to have independent implementation, but have you ever heard about the libsmacker lib written in C? http://libsmacker.sourceforge.net/

libsmacker seem like a great resource, especially for implementation in C/C++ game engines. We've not looked at libsmacker specifically, but we've previously looked at the FFMPEG sources for Smacker decoding:
https://github.com/FFmpeg/FFmpeg/blob/master/libavformat/smacker.c

Found it some minutes ago. As both projects rely on information from multimedia.cx, have you thought about contributing to the lib or vice versa?

I would believe that libsmacker is feature complete? Thus, we'll probably keep implementing the Smacker decoder mostly independently, and then use it to cross-validate the implementations. If something differs we'll try to open up issues for either libsmacker or FFMPEG. As such, we'll try to refrain from looking at the source code, at least to start with :)
As with most projects, this is for educational purposes and to have fun.
In the end, I think it makes most sense to convert the *.SMK videos to modern file formats with open specifications, and then use decoders for these to implementing video playback in Devilution for instance, as indicated by @galaxyhaxz in https://github.com/diasurgical/devilution/issues/17#issuecomment-400065929

I think once we get Diablo running on linux, we could make a script which will convert all assets in the MPQ to open source formats. The script could use the Rad tools to convert the .smk to an open source format. Then repack everything back up into a .zip format, so we don't need the MPQ library at all!

BTW, the FFmpeg code is under LGPL license, while this project is under Unlicense. It seems to be illegal even to translate the source code into another language:
https://softwareengineering.stackexchange.com/questions/151515/rewrote-gnu-gpl-v2-code-in-another-language-can-i-change-a-license#151516
Which is a bit sad.

@sskras That's mostly fine I think. No code has been derived from FFmpeg. We only look at the source code in the sense that we treat it as one reference to learn about the file format specification. So, we should still be able to release the source code of this project into the public domain.
That being said, this project is on hold as other things are more exciting to explore as of current :)


Black screen after hibernation

Great job! I installed Manjaro on Microsoft Surface Laptop + your patch, everything works fine except hibernation. Everytime I suspend/hibernate I get black screen or if I leave laptop for a while so I don't how to debug this issue, I mean how to know what should I do to get hibernate working? :D

resume= was missing on grub config. so I'll close this issue.

Do you have the dGPU running, too?

I'm not sure if I have dGPU?


Add KS240 integration

Is your feature request related to a problem? Please describe.
KS240 is a new type of ceiling fan switch from the TP-Link KASA line that controls, one for the fan speed control and one for light dimmer switch.  This is not getting discovered.
Describe the solution you'd like
I have a KS220 switch and KP303 power strip that is getting discovered fine by the integration, the KS240 definition may just need to be added.
Describe alternatives you've considered
The device is currently being added to the python-kasa, https://github.com/python-kasa/python-kasa/pull/764, initially reported here: https://github.com/python-kasa/python-kasa/issues/749
Additional context
This has also been added as as request to be added to the HA core: https://github.com/home-assistant/core/issues/111238

This is the repro for tp-link decos. KS249 is not a deco, please go request in the kasa integration


Add mariadb flavor

The goal of this PR, is to be able to use the libmariadbclient to connect to a mariadb server. To be able to do it, i created a new recipe to install mysql2 gem  instead of mysql one. And by the way i use the mariadb::client recipe (dependency of mysql2_chef_gem::mariadb).
I was unable to compile the mysql gem against the libmariadbclient library.
If you think i need to change something in my way of doing, please tell me :) I do not know if i follow the right way.

Providers updated to mysql2_chef_gem, which has a mariadb library provider


TypeScript - Incorrect type signature of ElementArrayFinder::getText

Hi there!
It seems to me that the automatic application of the ElementFinder-specific WEB_ELEMENT_FUNCTIONS to the ElementArrayFinder results in an incorrect type signature of the ElementArrayFinder::getText().
Instead of declaring a thenable containing a list of strings, the signature describes a thenable containing a single string.
This gives both the IDE and the compiler a bit of a hard time:

Please let me know should you need any additional information!
Best,
Jan

This is proving to not be easy; not in a way that doesn't cause other issues, at least.
Even manually implementing methods as a proxy, rather than using dynamic assignment leads to compiler complaints.
I am elbow-deep in this, at the moment, so anything else you have hit, specifically relating to this problem, @jan-molak, would be a welcome addition.

Sure, happy to help. Do you have some branch we could look at together?

I've got a couple of them that diverge; I'll get you links for them once I pick a winner and put it in a stable state. Probably later tonight.
Part of the conflict that I run into is actually the non-compatible different Promise interfaces, woven throughout the system.

This one is also related to the area you're working on I think: #4049

Is this really still not fixed?
Is there a plan to fix it? Perhaps it would be easier after upgrading to a more recent WebDriverJS without the Promise Manager.


Recorder is not working when I upgrade to Chrome 77 version.

I have upgraded to Chrome77 version and now your recorder is not working. It throws this error

FFMPEG::Error: [gdigrab @ 0000021f04f5a540] Can't find window 'https://uat.allianz-go.com/login.max?preprocess=true - Google Chrome', aborting.
title=https://uat.allianz-go.com/login.max?preprocess=true - Google Chrome: I/O error

ffmpeg.log is here

ffmpeg version N-94960-g53d31e91c5 Copyright (c) 2000-2019 the FFmpeg developers
built with gcc 9.1.1 (GCC) 20190807
configuration: --enable-gpl --enable-version3 --enable-sdl2 --enable-fontconfig --enable-gnutls --enable-iconv --enable-libass --enable-libdav1d --enable-libbluray --enable-libfreetype --enable-libmp3lame --enable-libopencore-amrnb --enable-libopencore-amrwb --enable-libopenjpeg --enable-libopus --enable-libshine --enable-libsnappy --enable-libsoxr --enable-libtheora --enable-libtwolame --enable-libvpx --enable-libwavpack --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxml2 --enable-libzimg --enable-lzma --enable-zlib --enable-gmp --enable-libvidstab --enable-libvorbis --enable-libvo-amrwbenc --enable-libmysofa --enable-libspeex --enable-libxvid --enable-libaom --enable-libmfx --enable-ffnvcodec --enable-cuvid --enable-d3d11va --enable-nvenc --enable-nvdec --enable-dxva2 --enable-avisynth --enable-libopenmpt --enable-amf
libavutil      56. 35.100 / 56. 35.100
libavcodec     58. 57.100 / 58. 57.100
libavformat    58. 32.104 / 58. 32.104
libavdevice    58.  9.100 / 58.  9.100
libavfilter     7. 58.102 /  7. 58.102
libswscale      5.  6.100 /  5.  6.100
libswresample   3.  6.100 /  3.  6.100
libpostproc    55.  6.100 / 55.  6.100
[gdigrab @ 0000021f04f5a540] Can't find window 'https://uat.allianz-go.com/login.max?preprocess=true - Google Chrome', aborting.
title=https://uat.allianz-go.com/login.max?preprocess=true - Google Chrome: I/O error

@Rajagopalan-M I don't think the Chrome version will affect the recorder. Can you share how you're providing the title to the recorder?

@kapoorlakshya
This is the way I am passing the title
@recorder = ScreenRecorder::Window.new(title: ScreenRecorder::Titles.fetch('chrome').last, output: File.expand_path("Output/Video/#{scenario}/#{testCase.first + '_' + testCase[1]}.mp4"))

@kapoorlakshya  Hi, any update on this? I couldn't upgrade to the newest version of Chrome because of this video recorder. Your recorder works fine in earlier version but not in the newer version.

Hi @Rajagopalan-M, the problem is not with Chrome or the recorder. The issue is that the recorder is attempting to start before the page in the browser is fully loaded. The actual title of the page is AllianzGo - Google Chrome, but the log you shared shows that the URL is being used. This tells us that the page is still loading when the recorder tries to get a handle on the browser window.
The following worked for me once the page was fully loaded:
rec = ScreenRecorder::Window.new(title: "AllianzGo - Google Chrome", output: 'rec.mp4')
rec.start

Try adding @browser.wait (using watir) or some other wait strategy to make sure the page is fully loaded first. Another thing you could do is start the recorder before navigating to your test app:

Launch browser (title will be "New Tab - Google Chrome")
Start recording
Navigate to the app

Hi, it's not happening while it starts the browser. it's happening while it closes the browser.

[gdigrab @<PHONE_NUMBER>b5adc0] Can't find window 'https://uat.allianz-go.com/login.max?preprocess=true - Google Chrome', aborting.
 title=https://uat.allianz-go.com/login.max?preprocess=true - Google Chrome: I/O error

Hmm... I am unable to reproduce the issue in Chrome 77. Are you stopping the recorder after closing the window?

@Rajagopalan-M I think this error is definitely happening when you start the recorder. I would recommend waiting for the page to load before starting the recorder. Give it a try and let me know if you still get this error.

Okay, I am in vacation now. Once I am back, let me try and let you know. I am using your gem in my framework. And I am using my framework in my many project. So when Chrome upgraded, all the people who uses my framework got this problem.

Sounds good! Enjoy your vacation.

I am back. But now another problem is, higher version of chrome has some problem. Can't even open chrome now. Once chrome people fix this problem, I will get back to you. Until then I use your recent version of gem with Chrome <75. Thanks for supporting.

@kapoorlakshya Hi, I started using chrome 78 and your gem works perfectly. Thanks. Your gem is very useful for me, I have incorporated inside my framework. Videos are good to review the errors.

Good :)


propertiesToHash missing from Image Service API Docs

📚 Subject area/topic
Image Service API
📋 Page(s) affected (or suggested, for new content)
https://docs.astro.build/en/reference/image-service-reference/
📋 Description of content that is out-of-date or incorrect
According to this PR, the Image Service API has a property propertiesToHash which is used to control which properties are included in the filename hash of an optimised asset.
There doesn’t seem to be a mention on the page in the docs, although I believe there should be, since it’s important information to know when creating a custom image service that supports more options than the standard one.
On a related note, I believe the built-in sharpService is not using the propertiesToHash? I couldn’t find a mention in the source code. Is that intentional and if so, why?
Edit: looking at the patch that introduced the change, it seems like if an image service doesn’t define propertiesToHash, Astro will fall back on the default ones defined in packages/astro/src/assets/consts.ts. Perhaps this could also be added to the updated documentation.
🖥 Reproduction in StackBlitz (if reporting incorrect content or code samples)
No response

It is missing yes. I said I would add it and never did, oops.
The method optionally returns an array of strings, which represents the keys that should be included to generate the hash of the image. Its default value is everything that counts for the base services, width height format quality etc.

Hi @sarah11918, I have opened a PR for this issue. I wasn’t quite sure where to put the new content in the document, since it isn’t really a hook. So I added the information in the section about local image services.

Thank you @amxmln ! I will ask @Princesseuh to review it. Really appreciate you jumping in here. :rocket:


checkbox within pptx output

Hi,
Is it possible to insert a tickable checkbox within the generated powerpoint file? or is it possible to embed/insert an excel object and use its tickable checkbox?
Please note: I need a checkbox that can be check NOT during the slide show presentation of powerpoint file. Please give me an idea of a checkbox that can be tick while editing the file.
Thank you.

Hi @roix31
There are no plans to support form fields or embedded Excel workbooks at this point in time.


Branch Tracker: Works in Progress During COVID-19

I'm using this issue to track feature branches developed during the evolving COVID-19 crisis. This has led to a lot of fast-tracked features being developed at a breakneck pace.
Bleeding Edge Integration Branch
All branches will be merged to integration/sepia, which will serve to hold the latest build with some features that may not eventually be merged to master. They are just needed right now during the COVID-19 crisis without the full review and consideration that would go into something in the master branch. This is live and in production at SEPIA (Philadelphia's AA Intergroup): https://www.aasepia.org/
The follow branches are merged to integration/sepia:

feature/online-meetings - support for online meetings using conference_url and conference_phone

This currently checks conference_url.length to trigger online meetings
We'll need to support meeting type ONL as the trigger in the future

feature/temp-closure - adds the new spec-approved meeting types TC (Temporary Closure) and ONL (Online Meeting).
feature/venmo-paypal - adds buttons to donate to Venmo and PayPal if the fields meeting.venmo or meeting.paypal are populated.

Branches With PRs to Master
The follow branches have outstanding PRs to master, as they are likely to be included since they mirror development in the TSML WordPress plugin.

feature/temp-closure

@joshreisner Closing this out, thanks to your help! :)


feat: added examples to the specification

I was pointed to the specification and wanted to just get started, which would have been possible with examples. Instead I did a lot of reading about the motivation for the spec, and the formal spec itself. I am hoping the examples will help other people see how simple the spec is and to just get started quickly.

@tchaffee thank you for this work; just some quick drive by comments (I'm finally catching up on some of my massive backlog of git issues phew).
I love the idea of adding some example; I'm excited to see this work to conclusion.
a couple random notes:

as a matter of process, should we update the version of the document when we make significant changes like this; perhaps call this 1.0.0-beta.2?
we should also update the documents in the translation folders (perhaps we can loop @Morishiri back in to provide the updated translation).

@Morishiri perhaps you have some thoughts on this pull too; given that you've made some significant contributions to this site.

I will update polish language as well with some additional changes which I've noticed after I added this translation, but I think I will do that as separate pull request? Or should I add it here?

@Morishiri @tchaffee this has been hanging out for a long time, shall we get it over the finish line?

I think this should be good now, but if there are any other minor changes please let me know as soon as you can and I'll get this finished up.

@damianopetrungaro thanks for the review and I've made the changes you requested. Hopefully this is ready to be merged now.

Is there anything holding this up from being merged? I think I've made all the requested changes.

@tchaffee please tag owner of the repo 😄

@damianopetrungaro Do you know who that is?

@bcoe @stevemao

Selfless plug: if you need more examples, which are valid for tools, take a look at this section on a post I wrote.

@bcoe Any feedback here? I'd like to get this closed out and move on before I get distracted again with other projects and can't revisit this for a while.

Yay! Ready! Let's get this sucker merged 🎉
bump @bcoe @stevemao 🙂

@bcoe @stevemao also, if you think this project could benefit from more maintainers to share the load, perhaps we could open another ticket to do a "Call for co-maintainers" with a lightweight nomination process?

@patcon please, move to another issue, this is not the right place.

@damianopetrungaro @patcon would definitely love more maintainers chipping in. Full disclosure, my wife passed away at the start of 2018 and it's been taking me a while to get back into open-source again -- hoping I'll be a bit more on the ball again in the near future.
I have a slack I've been managing for open-source projects like conventional commits:
http://devtoolscommunity.herokuapp.com/
This is a good place to come have a real-time conversation about better maintaining this projects (and to express an interest in being a maintainer).

@bcoe I'm so sorry to hear that. It can take a long time to get back to anything even resembling a normal routine. I'm also happy to help out in any way I can.

@bcoe So sorry to hear about this man, very sorry for your loss.

@patcon @tchaffee @blackxored thanks for your thoughts 👍 I'm happy to add more folks to the project, I think we all have the same goals: getting the language and rules around conventional commit messages as straight-forward as possible, and making sure the other libraries in this repo match those rules.
Private message me in http://devtoolscommunity.herokuapp.com/, and I'll get a few more folks added as contributors.


Fix filer dependency

cmsplugin-filer 1.1 must depend on filer 1.2 as it no longer ships with ThumbnailOption model
#238 must also be merged to fix an issue with ThumbnailOption migrations

@mkoistinen @stefanfoulis #242 and #238 should have been included in 1.1 release as they fix issues with ThumbnailOption migrations

Fixed in #243


Correctly read in input from a non-interactive stream for the CLI

If you pipe in a file to the influx CLI, it will not try to open the
interactive line reader, but instead just send the contents of the
entire file to the server.
Fixes #6896.

I believe I am running into an issue caused by this pull request related to the TTY check introduced in one of the commits. When executing influx CLI without a TTY, commands hang probably waiting for input from STDIN.
With the version of the CLI built from master i get the following behavior:
/inlflux-from-master/influx -database telegraf -execute 'show databases'
name: databases
name
----
telegraf
_internal
kapacitor_example

While simulating no tty allocation using ssh the command hangs indefinitely:
ssh localhost "/influx-from-master/influx -execute 'show databases'"

The same query succeeds using one of these two:
ssh -t localhost "/influx-from-master/influx -execute 'show databases'"
ssh localhost "echo 'show databases' | /influx-from-master/influx"

Notice that the query was successful using CLI 1.0.2:
$ /usr/bin/influx
Visit https://enterprise.influxdata.com to register for updates, InfluxDB server management, and monitoring.
Connected to http://localhost:8086 version 1.0.2
InfluxDB shell version: 1.0.2
> exit
$ ssh localhost "/usr/bin/influx -execute 'show databases'"
name: databases
---------------
name
telegraf
_internal
kapacitor_example

We run into this issue when some queries executed using the latest influx CLI from a Cloud Foundry start up script for which a tty is not allocated started to fail. In that case the error was ERR: missing required parameter "q". It can reproduced by:
ssh localhost "echo '' | /influx-from-master/influx -database telegraf"
ERR: missing required parameter "q"

This makes me believe that the influx CLI is still trying to read from STDIN in the second case.


Add macFUSE check for MacOS

What changes are proposed in this pull request?
Add macFUSE check for MacOS.
Why are the changes needed?
The method isFuseInstalled can not check macFUSE.
Does this PR introduce any user facing changes?
no.

@yyongycy MacFuse version has been tested.

@LuQQiu any more comments?

@LuQQiu Please help review this pr, thank you!

alluxio-bot, merge this please


README has screen shots and walk-through video

Finished the README.md file. Added screen shots, walk through video, GitHub address and others

updated readme - linked screenshot in issue instead of adding files to repo - #44


felt test --unit-tests-only leaves .js/.deps/.map files when test compilation fails

felt test --unit-tests-only leaves .js/.deps/.map files when test compilation fails.
Steps to reproduce.

Introduce a compilation error in web engine code
Run felt test --unit-tests-only.
Wait for felt to fail with a compiler error. For example, in my case I had this error:

test/canvaskit/canvaskit_api_test.dart:320:21:
Error: The method 'height' isn't defined for the class 'SkAnimatedImage'.
 - 'SkAnimatedImage' is from 'package:ui/src/engine.dart' ('lib/src/engine.dart').
    expect(animated.height(), 1);
                    ^^^^^^
Error: Compilation failed.
ERROR: Failed to compile test /home/yjbanov/code/flutter/engine/src/flutter/lib/web_ui/test/canvaskit/canvaskit_api_test.dart. Dart2js exited with exit code 1
Failed to compile tests.

Run git status, and observe left-over files:

Untracked files:
  (use "git add <file>..." to include in what will be committed)
	test/alarm_clock_test.dart.browser_test.dart.js
	test/alarm_clock_test.dart.browser_test.dart.js.deps
	test/alarm_clock_test.dart.browser_test.dart.js.map
	test/canvas_test.dart.browser_test.dart.js
	test/canvas_test.dart.browser_test.dart.js.deps
	test/canvas_test.dart.browser_test.dart.js.map
	test/canvaskit/frame_timings_test.dart.browser_test.dart.js
	test/canvaskit/frame_timings_test.dart.browser_test.dart.js.deps
	test/canvaskit/frame_timings_test.dart.browser_test.dart.js.map
	test/canvaskit/image_test.dart.browser_test.dart.js
	test/canvaskit/image_test.dart.browser_test.dart.js.deps
	test/canvaskit/image_test.dart.browser_test.dart.js.map
	test/canvaskit/path_test.dart.browser_test.dart.js
	test/canvaskit/path_test.dart.browser_test.dart.js.deps
	test/canvaskit/path_test.dart.browser_test.dart.js.map
	test/canvaskit/shader_test.dart.browser_test.dart.js
	test/canvaskit/shader_test.dart.browser_test.dart.js.deps
	test/canvaskit/shader_test.dart.browser_test.dart.js.map
	test/canvaskit/skia_objects_cache_test.dart.browser_test.dart.js
	test/canvaskit/skia_objects_cache_test.dart.browser_test.dart.js.deps
	test/canvaskit/skia_objects_cache_test.dart.browser_test.dart.js.map
... etc ...

felt first compiles code into the source directories, then copies everything from test/ to build/, then deletes the extra files. There are a few flaws in this approach:

If the process is interrupted for any reason, it leaves garbage in the source directories.
It leaves opportunities for garbage leaks in the future, as the build must be kept in sync with the clean-up.
If there's a bug in the clean-up function, it may accidentally delete files that shouldn't be deleted (_cleanupExtraFilesUnderTestDir is basically rm -rf with a filter, which makes me scared).

A better approach would be one that does not require clean-up. For example, the build could output straight into the build/ directory, without temporarily writing into the source directories.
/cc @nturgut

--unit-tests-only is gone.


shell: Add support for custom line2argv callback

This allows to define shells which are using different syntax for
commands parsing eg. foocmd=param1,param2.
This is usefull for providing compatibility with existing external
tools while allowing to use Zephyr's shell subsystem.
Signed-off-by: Szymon Janc<EMAIL_ADDRESS>

Codecov Report

Merging #9774 into master will decrease coverage by <.01%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master    #9774      +/-   ##
==========================================
- Coverage   52.24%   52.23%   -0.01%
==========================================
  Files         212      212
  Lines       25950    25952       +2
  Branches     5593     5594       +1
==========================================
  Hits        13557    13557
- Misses      10159    10160       +1
- Partials     2234     2235       +1

Impacted Files
Coverage Δ

subsys/shell/shell.c
46.81% <33.33%> (-0.36%)
:arrow_down:

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 1359992...800a694. Read the comment docs.

ping


Update attack-surface-reduction-rules.md

Updated the document as ASR Rules are working against Server 2016

@JohanFreelancer9 Please copy review

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/6413

@jweston-1 Please merge this PR. Thanks

@tiburd Kindly suggest, who can help us here. Thanks

Won't fix.
The table containing this information has been changed, moving those columns (Server 2012 and Server 2016) into a separate table, per the product team. Accordingly, this update would conflict with existing 'approved' published content.

Thank you for the udpate.


[Keep On Death]

I suggest that the Cars can be keep on Death.

You could do this with other plugins?

If you mean keep them in your inventory on death, then just enable keep inventory in the normal Minecraft gamerules.


[Bug Report]: Key Vault Role assignments should use more inputs to guid

Module path
avm/res/key-vault/vault
Bicep version
<IP_ADDRESS>444
Describe the bug
If I re-run a deployment I'm getting a conflict on the role assignment used here:
https://github.com/Azure/bicep-registry-modules/blob/main/avm/res/key-vault/vault/main.bicep#L267C14-L267C92
One way around that is to use more inputs to the guid parameter like we do here:
https://github.com/Azure/azure-dev/blob/main/templates/common/infra/bicep/core/security/role.bicep#L15
To reproduce
Run a template with the following, multiple times.
module keyVault 'br/public:avm-res-keyvault-vault:0.1.0' = {
  name: 'keyvault'
  scope: rg
  params: {
    name: !empty(keyVaultName) ? keyVaultName : '${abbrs.keyVaultVaults}${resourceToken}'
    location: location
    tags: tags
    roleAssignments: !empty(principalId) ? [
      {
        principalId: principalId
        principalType: 'User'
        roleDefinitionIdOrName: '4633458b-17de-408a-b874-0445c86b69e6'
      }
    ] : []
  }
}

Code snippet
.

Relevant log output
.

Unfortunately, if there is a pre-existing role assignment with a different name, there is not much we can do from a module perspective. This is an unfortunate "design" of role assignments in Azure. The only solution I know of is to remove the role assignment in Azure once and redeploy. Adding more inputs doesn't solve the issue (the ID is already unique thanks to its elements). If you have a pre-existing role assignment and change the name of the role assignment in any way (e.g., by adding different segments) the effect is simply that ARM will tell you that there already is a role assignment with a different name.

Hey @jongio ,
I've now been deploying the same template 3 times and cannot reproduce the issue.
Instead of conflicts the only issue that I got was that the provided roleDefinitionId is in an incorrect format. RoleDefinitionIds are not just GUIDs but look like this:
roleDefinitionIdOrName: '/subscriptions/{subscriptionId}/providers/Microsoft.Authorization/roleDefinitions/14b46e9e-c2b7-41b4-b07b-48a6ebf60603'

or
roleDefinitionIdOrName: subscriptionResourceId('Microsoft.Authorization/roleDefinitions', '14b46e9e-c2b7-41b4-b07b-48a6ebf60603')

for short.
Another alternative is the reference:
resource roleDef 'Microsoft.Authorization/roleDefinitions@2022-04-01' existing = {
  name: '14b46e9e-c2b7-41b4-b07b-48a6ebf60603'
  scope: tenant()
}

(...)
   roleDefinitionIdOrName: roleDef.id

For reference: https://learn.microsoft.com/en-us/azure/role-based-access-control/built-in-roles#contributor
The template variant of yours that I did use for testing, was this one
targetScope = 'subscription'

param location string = deployment().location
param keyVaultName string = 'alsehrkvtestvault'
param principalId string = '<myUserId>'

resource rg 'Microsoft.Resources/resourceGroups@2023-07-01' = {
  name: 'rg-alsehr'
  location: location
}

module keyVault 'br/public:avm-res-keyvault-vault:0.1.0' = {
  name: 'keyvault'
  scope: rg
  params: {
    name: keyVaultName
    location: location
    tags: {}
    roleAssignments: !empty(principalId) ? [
      {
        principalId: principalId
        principalType: 'User'
        roleDefinitionIdOrName: subscriptionResourceId('Microsoft.Authorization/roleDefinitions', '14b46e9e-c2b7-41b4-b07b-48a6ebf60603')
      }
    ] : []
  }
}

I switched to
        roleDefinitionIdOrName: subscriptionResourceId('Microsoft.Authorization/roleDefinitions', '4633458b-17de-408a-b874-0445c86b69e6')

And it works.
Maybe we need to more clear that this is "resource id" not the "Role id"

I switched to
        roleDefinitionIdOrName: subscriptionResourceId('Microsoft.Authorization/roleDefinitions', '4633458b-17de-408a-b874-0445c86b69e6')

And it works.
Maybe we need to more clear that this is "resource id" not the "Role id"

I agree @jongio . Would could also add an example in the test files to make it a bit more obvious.
It is actually described in the parameter description for role assignments:

Optional. Array of role assignment objects that contain the 'roleDefinitionIdOrName' and 'principalId' to define RBAC role assignments on this resource. In the roleDefinitionIdOrName attribute, you can provide either the display name of the role definition, or its fully qualified ID in the following format: '/providers/Microsoft.Authorization/roleDefinitions/c2f4ef07-c644-48eb-af81-4b1b4947fb11'.

But I'll suggest to move that to the 'roleDefinitionIdOrName' parameter metadata.
@eriqua , any thoughts on the matter? I'd suggest we do both:

Update the parameter descriptions (and in turn the AVM spec)
Update the test cases of the existing modules to also have a role assignment using a role definition id

What about calling the param roleDefinitionResourceId

What about calling the param roleDefinitionResourceId, there would have been no confusion for me if resourceid was in the name

Fundamentally I have no objection to that, but it would need to be updated in the Bicep & Terraform schema. I assume the reason it's not called a resource ID is that it's not a resource? Just a guess 😄
Let's see, @eriqua & @jtracey93 what's your take on this?

We opted to not diviate from the specs for this (name-wise) - however, we did implement a PR that not only enables all schemas possible (name / guid / full ID ref), but also updated the tests accordingly to show the different options in #663 . This should resolve also the original issue. Hence I'll go ahead and close it. If there are any further questions, please let me know.


DOC Update SilverStripe to Silverstripe

Only Lessons this PR
Apply consistent formatting (120 chars etc)

Should it have been 'Silverstripe CMS' in some of those places?

Possibly, though it's a step in the right direction =)

Yeah, not a huge deal. My view was that it may make finding the places where it should be Silverstripe CMS and it's not SilverStripe anymore a bit harder, so we ideally get it right when we change it from SilverStripe to what it should be.
Updating the 3 instances here is not a dealbreaker, compared to the other PR where it was tens of changes.

Thanks for accepting this PR.
I will take a look at #9993, which has merge conflicts, I noticed it also needs to be updated to Silverstripe CMS too I'll add this file to that PR and squash. Hopefully I will got it right 😉


Binding Convertert does not get triggered when ObservableCollection is changed

Describe the bug
bind to a ObservableCollection and do a value aggregation in a binding converter does not work. the binding does not get invalidated and reevaluated when the collection changes
To Reproduce
see example project
Expected behavior
the binding should be invalidated and updated whenever an item is added or removed in the observable collection
Additional context
ObservableCollectionConverter.zip

That's not what ObservableCollection does. It doesn't fire PropertyChanged, it fires CollectionChanged which is a different event.

I'd suggest to add a read-only property Sum in the ViewModel instead and raise PropertyChanged for it, as you already have. I'd just make the setter private.
Alternative:
public MainWindowViewModel()
{
    RemoveCommand = ReactiveCommand.Create<int>(Remove) ;
!   // Added a listener to CollectionChanged, which will notify the UI that collection has changed.
!   // Don't do it. It's evil :-)
+   _items.CollectionChanged += (sender, args) => this.RaisePropertyChanged(nameof(Items));
}

As this is by design, I'll close the issue for now.

oh i see. i thought in itemscollection it is also done through the binding that it updates correctly when the collection changed. but i see it is an explicit listener that is registered


Set means to zero in online covariance GPU if assume_centered=True

Description

Set means to zero in online covariance GPU if assume_centered=True to align with all other covariance versions.


expt.binfun creates some trouble

Hi,
I am trying use this package on my data but for one trial I get an error due to the exp.binfun function.
It calculates the number of bins by dividing duration of trial with experiment binsize, and gives the right result compared to the number of samples in my behavior vector, for example, my vector is 251 elements, and duration/binSize = 251.00000... However, due to the ceil function it gets rounded up to 252 (The diff from 251 is 2.8422e-14).
I run into the same problem in line 14 of compileSparseDesignMatrix.
Is there any reason for using ceil instead of round?
Best regards, Eivind Hennestad

It depends on how you define bin edges. The default binfun currently uses ceil to make sure the first bin starts at 0 and ends at the end of the binwidth. You can use a custom binfun if you'd like. All code should be calling exp.binfun, so you can replace the function handle.


urlteam_2020-12-28-03-17-02 Archive already exists!

I have download this repo.
I have download xz-5.2.5-windows.zip. Install files to c:\Windows\System32.
I have goo message from: > xz --help
First time run was downloaded the archives to archives\urlteam_2020-12-28-03-17-02
But then i have errors: go run main.go -date latest -keywords keywords.txt -o out.txt
Search starting for: latest
urlteam_2020-12-28-03-17-02 Archive already exists!
panic: runtime error: index out of range [1] with length 1

goroutine 1 [running]:
main.searchFile(0xc0001a5380, 0x37, 0xc0001adbc0, 0x14, 0xc000010118, 0x7)
        ../urlhunter/main.go:201 +0xc5c
main.getArchive(0xc000300000, 0x172e4, 0x1fe00, 0xc0000100a8, 0x6, 0xc0000100f0, 0xc, 0xc000010118, 0x7)
        ../urlhunter/main.go:194 +0xb2f
main.main()
        ../urlhunter/main.go:94 +0x387
exit status 2

Could you please try to use the compiled-binary for Windows? https://github.com/utkusen/urlhunter/releases

Downloaded and unpack urlhunter_0.1.1_Windows_amd64.zip.
Copied the archives\urlteam_2020-12-28-03-17-02\ to the new folder.
Copied the keywords.txt.
Run the command urlhunter.exe -date latest -keywords keywords.txt -o out.txt
And then i have the same error.
Search starting for: latest urlteam_2020-12-28-03-17-02 Archive already exists! panic: runtime error: index out of range [1] with length 1

Who are you?
On Mon, 28 Dec 2020, 11:03 am Damian Viktorovich Morozov, <
<EMAIL_ADDRESS>wrote:

Downloaded and unpack urlhunter_0.1.1_Windows_amd64.zip.
Copied the archives\urlteam_2020-12-28-03-17-02\ to the new folder.
Copied the keywords.txt.
Run the command urlhunter.exe -date latest -keywords keywords.txt -o
out.txt
And then i have the same error.
Search starting for: latest urlteam_2020-12-28-03-17-02 Archive already
exists! panic: runtime error: index out of range [1] with length 1
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/utkusen/urlhunter/issues/12#issuecomment-751640392,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR46FUZCBEFUODKNAMFYDKLSXBCW3ANCNFSM4VLYDUPQ
.

Can you solve that issues?

I will check it soon

Hello, do you have any news?

Hello, do you have any news?

@utkusen Could you please release a new version of URLHunter with the patch in #13 included?


iOS 11 localhost http access

[x] I've read, understood, and done my best to follow the *CONTRIBUTING guidelines.

What did you do?
Updated to Xcode 9 with the latest Alamofire pod, and sending GET request to http://localhost:8080/storys
What did you expect to happen?
200 response with normal payload as it always does.
What happened instead?
Alamofire request gets error:
Error: Optional(Error Domain=NSURLErrorDomain Code=-1004 "Could not connect to the server." UserInfo={NSUnderlyingError=0x1c0255db0 {Error Domain=kCFErrorDomainCFNetwork Code=-1004 "(null)" UserInfo={_kCFStreamErrorCodeKey=61, _kCFStreamErrorDomainKey=1}}, NSErrorFailingURLStringKey=http://localhost:8080/storys, NSErrorFailingURLKey=http://localhost:8080/storys, _kCFStreamErrorDomainKey=1, _kCFStreamErrorCodeKey=61, NSLocalizedDescription=Could not connect to the server.})
Note that:

https request works for my real server
I verified curl & postman & Chrome works on the localhost url, and added ATS keys to info.plist:

<key>NSAppTransportSecurity</key>
    <dict>
        <key>NSAllowsArbitraryLoads</key>
        <true/>
		<key>NSAllowsLocalNetworking</key>
		<true/>
        <key>NSExceptionDomains</key>
        <dict>
            <key>localhost</key>
            <dict>
                <key>NSExceptionAllowsInsecureHTTPLoads</key>
                <true/>
                <key>NSExceptionRequiresForwardSecrecy</key>
                <false/>
                <key>NSIncludesSubdomains</key>
                <true/>
                <!-- Optional: Specify minimum TLS version -->
                <key>NSTemporaryExceptionMinimumTLSVersion</key>
                <string>TLSv1.1</string>
                <key>NSRequiresCertificateTransparency</key>
                <false/>
            </dict>
        </dict>
    </dict>

Could the iOS 11 deprecations of some cyphers hashes and protocols?
https://stackoverflow.com/a/46481875/1032900
Alamofire Environment
Alamofire version: 4.0.0
Xcode version: 9.0
Swift version: 4
Platform(s) running Alamofire: iOS 11
macOS version running Xcode: 10.12.6

Strange. That error is very generic but I'm pretty sure it's not an ATS or other issue like that. Instead it's more likely the server isn't running at all. Is this a server that's running locally on device? Or are you connecting to a server on the local network? I asked, because cURL, Postman, and Chrome wouldn't work with a server on a local device. Please verify that's the server is actually running.
While this is unlikely to be an Alamofire issue, I won't close the issue yet, as I'm interested in seeing whether local server behavior has changed in iOS 11.

For some reason it works today o.O I did update macOS to High Sierra and not sure whether there was a bug with Xcode 9 on Sierra - also please note I was running on simulator not a local device, and the server was definitely running when I reported.
Thanks for the reply and hopefully the above information provides some insights.

@jshier Thanks for the response to this query. I also faced a similar problem and learned that it was a server issue.
@hyouuu Thanks for making your post well detailed.

Just put your  local IP instead of localhost in url .
e.g. http://<IP_ADDRESS>:3000/getUserDetail
That should be work....

Best way to resolve this is using ngrok


face_recognition only works with obama.

face_recognition version: 1.2.2
Python version: Python 3.10.12
Operating System: Linux mint

Description
I copied the [Recognize faces in live video using your webcam - Simple / Slower Version to a file changed obama & biden to the faces I want irl and I just get this error (IndexError: list index out of range) but when I use the defaults it works fine....
here is the code that I added my own pictures in
import face_recognition
import cv2
import numpy as np

video_capture = cv2.VideoCapture(2)

#obama_image = face_recognition.load_image_file("obama.jpg")
#obama_face_encoding = face_recognition.face_encodings(obama_image)[0]

MY FACE_image = face_recognition.load_image_file("MY FACE.jpg")
MY FACE_face_encoding = face_recognition.face_encodings(MY FACE_image)[0]

#biden_image = face_recognition.load_image_file("biden.jpg")
#biden_face_encoding = face_recognition.face_encodings(biden_image)[0]

known_face_encodings = [
    MY FACE_face_encoding
]
known_face_names = [
     "My face"

face_locations = []
face_encodings = []
face_names = []
process_this_frame = True

while True:
    ret, frame = video_capture.read()

    if process_this_frame:
        small_frame = cv2.resize(frame, (0, 0), fx=0.25, fy=0.25)

        rgb_small_frame = small_frame[:, :, ::-1]

        face_locations = face_recognition.face_locations(rgb_small_frame)
        face_encodings = face_recognition.face_encodings(rgb_small_frame, face_locations)

        face_names = []
        for face_encoding in face_encodings:
            # See if the face is a match for the known face(s)
            matches = face_recognition.compare_faces(known_face_encodings, face_encoding)
            name = "Unknown"

            face_distances = face_recognition.face_distance(known_face_encodings, face_encoding)
            best_match_index = np.argmin(face_distances)
            if matches[best_match_index]:
                name = known_face_names[best_match_index]

            face_names.append(name)

    process_this_frame = not process_this_frame

    for (top, right, bottom, left), name in zip(face_locations, face_names):
        top *= 4
        right *= 4
        bottom *= 4
        left *= 4

        cv2.rectangle(frame, (left, top), (right, bottom), (0, 0, 255), 2)

        cv2.rectangle(frame, (left, bottom - 35), (right, bottom), (0, 0, 255), cv2.FILLED)
        font = cv2.FONT_HERSHEY_DUPLEX
        cv2.putText(frame, name, (left + 6, bottom - 6), font, 1.0, (255, 255, 255), 1)

    cv2.imshow('Video', frame)

    if cv2.waitKey(1) & 0xFF == ord('q'):
        break

video_capture.release()
cv2.destroyAllWindows()

mb i have actually no idea how to put the code in its own thing

you're most probably getting the error while calculating face_encodings - I had the same issue, it got resolved after I removed extra information from my image (metadata). This library seems to not work when your image has metadata.

@AbdurNawaz
hello，how to remove extra information from my image(metadata) ?

Maybe we can try cv2.imread to load imgs, as blow:
`
import cv2
import face_recognition
import os
root = './imgs/known_peoples/'
for name in os.listdir(root):
img_path = os.path.join(root, name )
img = cv2.imread(img_path)[:,:,::-1]
num_face = len(face_recognition.face_encodings(img))
print(name,  num_face)
The above code can correctlyface_encodings face from personal face. The reason is, as @AbdurNawaz , thatcv2.imread' gets only the image data.


Backport 2.16: build with gcc11

https://github.com/ARMmbed/mbedtls/pull/4462 (development)
https://github.com/ARMmbed/mbedtls/pull/4493 (development_2.x)

Please rebase to resolve the merge conflict.

The latest iteration of the pr-merge job failed due to a network glitch. The previous iteration passed, so we can ignore this failure.


License for open source

Aren't we supposed to have an open source license declaration of some sort?

http://choosealicense.com/ :)


Incomplete type declaration for KeyExclusions [pymodsecurity]

Describe the bug
Hello, guys! Nice work on version 3.0.3. We have been working towards compatibility. We have an initial support, but we noticed that KeyExclusions declared in src/variables/variable.h is not published in the final/installed headers for version 3.0.3. I couldn't find a way to handle this in pybind without the complete declaration, therefore we're having compiling issues with the new methods: resolveMultiMatches and resolveRegularExpression.
Can you guys help us with that?
Here's our issue: https://github.com/actions-security/pymodsecurity/issues/6
Please let me know if there anything else we can add.

Hi, @GustavoKatel
Indeed the  KeyExclusions class is not part of the public headers. The declaration of such class is forwarded here:
https://github.com/SpiderLabs/ModSecurity/blob/74eee9330bb4c1908145cee5011dd1b50c791acc/headers/modsecurity/collection/collection.h#L41
and here:
https://github.com/SpiderLabs/ModSecurity/blob/74eee9330bb4c1908145cee5011dd1b50c791acc/headers/modsecurity/anchored_set_variable.h#L45
As you may notice it is not used on the public headers, just to declare parameters. That is why is not part of it.
Are the bindings just including the public headers? or they also refer to ModSecurity source code?


Add ANCM props to VS editor intellisense

I don't get editor intellisense in my csproj file for the AspNetCoreHostingModel and AspNetCoreModuleName.

@shirhatti any guidance on this?

Copying comment from #891:
Another thing I noticed when looking at VS schema is that hostingmodel values are CamelCased there while the ones I added to templates are all lowercase, we need to decide which one we prefer because schema validation is case sensitive.

@shirhatti  sample PR you gave me is not for csproj intellisense, it's for web.config.

We still need to update it for the web.config with handlerSettings

@shirhatti Do you prefer CamelCased or all lowercase for inprocess/outofprocess?

Whatever casing the webSDK uses today to remain consistent. I believe it's all lowercase today, but you should confirm

we set it in what ever case the msbuild property is set.

This was done by CommonStartupTests.StartupStress

Hm?

Copy paste error.


Create a While operation

Not sure about the name of the operation, 'While' - it needs some thought. We could also call it something like 'ConditionalRepeat'.
It would be useful to repeat operations, such as GetAdjacentIds until some condition is met. To do this we should create a While operation similar to:
public class While implements InputOutput<Object, Object>, Operations<Operation> {
    private Operation operation
    private Conditional predicate;
    private int maxRepeats = DEFAULT_MAX_REPEATS
}

where Conditional is a simple POJO like:
public class Conditional {
    private Operation transform;
    private Predicate predicate;
}

We would need a WhileScoreResolver to calculate the score based on the operation score multiplied by the value of 'maxRepeats'.
In the WhileHandler we should have a maximum number of times allowed to prevent users from repeating a large number of times.

Merged into develop.


check-model-has-description fails with macros with description

Describe the bug
I get the following error when running the pre-commit hook check-model-has-description:
Check the model has description...............................................Failed
- hook id: check-model-has-description
- exit code: 1

macros/generate_schema_name.sql: does not have defined description or properties file is missing.

my .pre-commit-config.yaml file looks like:
repos:
- repo: https://github.com/offbi/pre-commit-dbt
  rev: 607cb07a1918442f5963662a9aa19da8984931e6
  hooks:
    - id: check-model-has-description

the macros yaml file (macros/macros.yml) has all the descriptions:
version: 2

macros:
  - name: generate_schema_name
    description: >
      A macro to generate the custom schema names based on the environment:

      * In `prod`:
        - If a custom schema is provided, a model's schema name should match the custom schema, rather than being concatenated to the target schema.
        - If no custom schema is provided, a model's schema name should match the target schema.

      * In other environments (e.g. `dev` or `ci`):
        - Build all models in the target schema, as in, ignore custom schema configurations.
    arguments:
      - name: custom_schema_name
        type: string
        description: The configured value of schema in the specified node, or none if a value is not supplied
      - name: node
        type: string
        description: The node that is currently being processed by dbt

and the manifest.json has the macro's description as well:
"macro.dwh.generate_schema_name": {
      "unique_id": "macro.dwh.generate_schema_name",
      "package_name": "dwh",
      "root_path": "/path/to/dbt/project/dwh",
      "path": "macros/generate_schema_name.sql",
      "original_file_path": "macros/generate_schema_name.sql",
      "name": "generate_schema_name",
      "macro_sql": "{% macro generate_schema_name(custom_schema_name, node) -%}\n    {{ generate_schema_name_for_env(custom_schema_name, node) }}\n{%- endmacro %}",
      "resource_type": "macro",
      "tags": [],
      "depends_on": {
        "macros": [
          "macro.dbt.generate_schema_name_for_env"
        ]
      },
      "description": "A macro to generate the custom schema names based on the environment:\n* In `prod`:\n  - If a custom schema is provided, a model's schema name should match the custom schema, rather than being concatenated to the target schema.\n  - If no custom schema is provided, a model's schema name should match the target schema.\n\n* In other environments (e.g. `dev` or `ci`):\n  - Build all models in the target schema, as in, ignore custom schema configurations.\n",
      "meta": {},
      "docs": {
        "show": true
      },
      "patch_path": "macros/macros.yml",
      "arguments": [
        {
          "name": "custom_schema_name",
          "type": "string",
          "description": "The configured value of schema in the specified node, or none if a value is not supplied"
        },
        {
          "name": "node",
          "type": "string",
          "description": "The node that is currently being processed by dbt"
        }
      ]
    }

Version:
pre-commit-dbt 607cb07a1918442f5963662a9aa19da8984931e6
dbt v0.19.0
python 3.8.6

Thanks, @PabloPardoGarcia I did not know that it is even possible to document macros like this. Will look at that!


add a "mature" channel aimed at new student users

As an instructor teaching a university course involving Julia-based practicals, I'm not sure which of the existing juliaup channels I should recommend to my students to use:

release may at times catapult them onto a very fresh release (patch level 1.x.0, just a few days old, where the ecosystem has not yet had time to discover and iron out new incompatibilities and glitches).
lts may at times lead to a very old release, lagging years behind the state of the art.

Students in courses are usually first-time Julia users without any experience of making such judgement calls themselves. They deserve the smoothest possible experience as new Julia users, and neither release nor lts are in practice ideal for that.
I would like to see a mature channel that tracks release with a cautious/conservative delay, such that it roughly

updates to a new “minor” release version only once per year, during the summer break (late July / early August), well before the start of the next academic year in the northern hemisphere
avoids updating to new “minor” 1.x releases until issues have calmed down (e.g., there hasn't been a significant new patch release for at least ~5 weeks)
delays updating to new patch releases for ~2–3 weeks, to leave some time for monitoring of new issues introduced by those

In case the current lts release is reasonably recent, mature might also linger on that until the end of the academic year.
(There might of course also be demand for a mature-south channel more in sync with the academic years of schools in the southern hemisphere.)
Managing such a mature channel is of course a bit more involved than the other channels, as it involves making judgement calls and liaising with various release managers and triaging teams in the Julia community, to get a sense of how “mature” a minor or a patch release has become.

This seems way too arbitrary. Just tell students what version to use in that case imo.

I agree with @KristofferC, I think that is too specific, and I'm sure there are many, many more similar use-cases with slightly different requirements out there.
Medium term I expect that the best way to support these teaching scenarios where an instructor wants everyone to be on the same version to be solved by https://github.com/JuliaLang/juliaup/pull/1059. Essentially at that point you can distribute a project to your students, ask the students to run Julia with that project, and it will automatically ensure that they all run on exactly the right Julia version.

I agree with the sentiment for teaching, but I think in an environment where you want least surprises, it is best to just tell students to do juliaup add 1.10 or juliaup add lts or something specific for the duration of the course.
Note that you also don't want this version to change during the semester, and different universities in different parts of the world have different start and end times. As you note, this would be significantly challenging. Also different courses may have different things they need from Julia, which will lead them to use different versions.
I am closing this for now.


WIP: Issue 463

Basing this on issue-481 since there's already a lot of work around aiohttp's stub.

Some other interesting bits:

requests will append a / while aiohttp does not, so I had to change this to /1 to make it somewhat consistent
This is missing my changes that converted the headers to a list in YAML (I'll add those shortly)
requests adds some default headers like user agent whereas aiohttp does not


gray bottom-border to navbar on sample page??

2/6/12: updated the topic title and content after following markdotto's suggestion to use the repo, but am now experiencing different issues.
Example at http://visualwave.com/bootstrap/mls2-fluid.html
Thank you, Tom

Instead of saving it from the browser, try downloading the repo instead and see if that solves your problem. Unsure of what your exact issue is without spending more time to dig deeper.
Have you modified the responsive CSS file at all?

http://visualwave.com/bootstrap/mls-fluid2.html -- now the icon bars work but an unexpect gray bottom border to the nav bar, cannot find any style for this...
Just noticed in IE9 the page gets a top black border that should not be there and is not there in FF / Chrome...any idea why??? -- it appears after allowing scripts to run...
The navbar should NOT be so difficult to customize -- I do realize it was constructed for Twitter first and others second but it would be nice/good/awesome/wonderful to see clear and extensive documentation on how to change different parts of the nav bar: coloring it, centering it, change color of the little square with 3 bars, get rid of any/all borders around the nav bar regardless of where they come from, resizing navbar width and height...
I understand that @mdo and @fat are busy and that Twitter is their 1st priority, perhaps someone knowledgeable about TWBS could contribute this to the docs??
Thank you, Tom

The style you're referencing is the box-shadow on .navbar-inner, line 2335 in your bootstrap.css.

cause of the black line in nav was     padding: 0px; /* 2/6/12 was 5px; */ in responsive.css -- something still causes it to have a black background for navbar-inner
now to figure out the padding around the brand thing which I replaced with an image...
Thank you, Tom

Since the page is no longer accessible and your comments seem to indicate this is resolved, closing this out.


Always pick Event Builder V2

Coverage remained the same at 100.0% when pulling 025a2aa6b65253e7042f4027522f3ade0a978853 on roman/v2 into 090136e675cc44ecff941b1d72781ea16121215f on devel.


assistant chat window does not support markdown base64 image

Feature Request
Is your feature request related to a problem? Please describe.
Currently assistant chat window uses markdown format to display texts. And many markdown editors supporting embedding base64 images in markdown. But open-webui does not support it.
I used Functions to try to display some images on response, but it failed.
Describe the solution you'd like
support hard coded base64 images displayed on assistant
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context

my filter code:
def outlet(...):
     ....
     for i in img_list:
       processed_images.append(f"data:image/png;base64,{new_img_data}")
     content = assistant_message["content"]
     for i in processed_images:
     content += f"\n\n![image{i}]({processed_images})"
     body["messages"][-1]["content"] = content

It seems __event_emitter__ works instead of the response rewriting


Revert "Update iOS RN header paths"

Reverts evollu/react-native-fcm#209

looks like it is not yet published


Revert "Default admin dependency removal - operand changes (#156)"

This reverts commit 9ad2dff7e975590bd41f5b6ef6ad208e3f4cda96.

Facing issues in make install. @posriniv please help or I shall revert these changes.


What do I do when I get this

Review #30839 as asked and consequently:

not post an image but only pasted text
actually use the correct tracker ("... if you were actually running yt-dlp ..."), where there is an existing issue for your diagnostic.


Bad result  train with a voice

I've trained many popular voices perfectly, but for some reason and for this voice only it doesn't do a good job.
https://whyp.it/tracks/113084/01?token=DVdVc (audio)
I have 20 minutes dataset with this high quality audio.

If your dataset consists entirely of this serious and formal broadcasting tone, then in order for the trained model to achieve good inference results, the input source should also mimic this broadcasting tone as closely as possible.

well even if i use the same voice, it's not that good
I want this voice: https://www.whyp.it/tracks/113084/01?token=DVdVc
but the result is this(bad): https://www.whyp.it/tracks/113106/none?token=Lz00f
And for example i tried another voice like elon musk (works perfectly)
elon musk voice: https://www.whyp.it/tracks/113107/elon?token=n6Q7I


handlers to submit events in kafka topics

Need to be able notify remote kafka from handler when processing POST request from user

@7flash I am working on this right now. There will be a publishEvent method to publish events to a Kafka topic.


Metadata for nodes and edges

addition of user defined metadata to nodes and edges
addition of source column names to nodes
updated README
add owlready2 to Poetry (required by src/preprocessing)
Fix errors in docstring

Good to go, just fix the conflicts and we can merge.


Testing with Docker: GitLab OAuth Callback URL is Read-Only

It looks like the callback URL is hard-coded, I can't update - since it defaults, in this case, to http://localhost/oauth/gitlab authentication doesn't work.
Investigating, but wanted to track this here in case someone has the same issue later.

Replaced line 5 in Template/config/integration.php with the following:
<?= $this->form->text('gitlab_oauth_url', $values) ?>

And was able to change the value of this field successfully. Still unable to connect - investigating...

For clarification, my callback URL has been set to http://<IP_ADDRESS>/oauth/gitlab which is the IP of the node the docker container is on and which I can hit without issues / which can reach the GitLab server without issue.
Not sure what the problem is - I saw the issue with url_rewrite but that seems to be set by default in the docker container, if I am seeing things correctly

Reverted to using the default localhost callback, which does let me authenticate - however, now I get the error message The OAuth2 state parameter is invalid
I don't see how localhost callback will work from anywhere else, but at least I'm further. Investigating this new error now.

I made sure to check the box for Allow Account Creation, same error. Tried manually adding my account as an external one, same error.

 I discovered the problem. If you update the Application URL in Settings -> Application Settings this updates the callback URL in Settings -> Integrations

The authentication problems are also resolved.


modestbranding not working in android.Does not remove the youtube logo.

How to remove youtube logo(in bottom right) in react-native-youtube library in android.
modestbranding does not work in android.

YouTube disabled this option quite a while ago

YouTube disabled this option quite a while ago
@johnckendall
Is there any way we can do some changes in the library to implement this ?

Is modestbranding working only with ios and not android and if so then why it is not working with android?

@hvaidsain This is the respinsibility ot the underlying Library


How to get SessionID?

Added the image to jump to a page with instructions on how get the SessionID.

I like the idea of this, but the wiki article is ancient, I don't even think the cookie name it refers to is correct. In order to merge something like this there would need to be an up to date article on the github wiki that it links to.

I expected that someone would correct wiki.
Because I'm because there are no English abilities fatally.
But I'll try my best.

How about with this?
https://github.com/bule2015/Procurement/wiki

Looks good apart from the steam section - it reads a little bit like you can login to procurement with steam, which isn't the case.
I would write:
・Can I log in to pathofexile.com using my steam account?
Yes, Please login using "Sign in through STEAM". The POESESSID retrieval will be the same.

It was rewritten as follows.
・Can Steam account also acquire SessionID?
　　Yes, When logging in by a official site, SessionID can be acquired.

I've copied your wiki page to https://github.com/Stickymaddness/Procurement/wiki/SessionID, if you link to it in your PR I can merge it.


Compact identifier DRS URIs should be cached in TES

Problem:
DRS URIs come in two styles (Hostname-based DRS URIs and Compact Identifier-based DRS URIs. The second style is resolved to the first type, and the resolves are shared with all DRS clients globally.
Due to that, "For performance reasons, DRS clients SHOULD cache the URL pattern..., with a suggested 24 hour cache life."
Solution:
Identify and resolve compact DRS into hostname DRS in TES, cache each result for 24 hours, and pass the resolved DRS to the compute node. There should be no need to continue resolving hostname DRS in TES as we intend to have the compute nodes perform that resolution (as they do today).
Describe alternatives you've considered
Resolving compact DRS in the compute node isn't useful because any caching performed there cannot be shared nor kept.
Code dependencies
This will require code changes in unit and/or integration tests.
Additional context
From what I've read, resolving compact identifier to hostname does not ever require authentication.
https://github.com/broadinstitute/cromwell/blob/acf584099b68c35083bffa1815404b36bc7c363e/cloud-nio/cloud-nio-impl-drs/src/main/scala/cloud/nio/impl/drs/DrsCloudNioFileSystemProvider.scala#L31

@MattMcL4475 Should this fall under the TES P1, P2 or P3 issues?


Split: Train Test Validation

We need a good way to split up data into train/validation/data sets.
Would like a function that takes a a list of timestamps (List[pd.Timestamp]) and splits according to a few methods. Note we are dealing with timeseries data so we have to be a little careful about sampling. This is so as not to give samples that are too similar in different sets. Because Solar generation is zero in the night, we can in general make sure we split data around midnight.
Function inputs:

List[pd.Timestamp]: all time series
method:str, the method to use
train_test_validation_split: List[int] - This gives the ratio of data that should be put in the different sets

Methods:

[ ] Day random - Divide the dataset into days groups, and randomly take samples according the 'train_test_validation_split'
[ ] Week random  - Divide the dataset into weeks groups, and randomly take samples according the 'train_test_validation_split'
[ ] Year random  - Divide the dataset into year groups, and randomly take samples according the 'train_test_validation_split'
[ ] Specific year -  takes an input which tells which years to go into which datasets
[ ] Full - return full dataset to all new datasets. This might be useful for unit-testing with a small amount of data
Any others?

Bonus:

[ ] Might be nice to have a option to turn on random sampling, as described above, or more repetitive sample e.g 3 weeks train, 1 week validation, 1 week test and repeat. This ensures that we have data from all seasons in each dataset

Return  dict[str:List[pd.Timestamp]] - return three datasets. The keys should be 'train','validation' and 'test'
Unittest

[ ] This would be really great to have unit test with all these methods too

One perspective on how to split time series data is here: https://niklasriewald.com/2021/06/29/paper-review-metnet-a-neural-weather-model-for-precipitation-forecasting/

Started on https://github.com/openclimatefix/nowcasting_dataset/tree/issue/127-split-data

Decided not to do 'year random' as not sure we have enough data for this


After signing the message for the discovery token, the prompt remains

And I cannot leave the screen after the prompt shows up, forcing me to kill the app.

Didn't get authorisation due to an error, hence the prompt did not dismiss.

I/chromium: [INFO:CONSOLE(122)] "Uncaught TypeError: window.web3CallBacks[id] is not a function"

Solved in #912


A problem about EMMC support

When flash a partition with 'sparse="true"' flag,  no error returned.
But device can't boot up successfully and it reports corresponding partition mount failed.
According to the Windows tool, it has split image up into 4 partitions and then write 1 by 1.
It seems we don't obtain such mechanism.
Test log as below from windows QFIL:
xml:

QFIL log:
INFO: File system.img is a sparse file, being split up into 4 separate XML tags
......
INFO: system.img is a sparse image
INFO: Looking for file 'system.img'
INFO: system.img is a sparse image
INFO: Looking for file 'system.img'
INFO: system.img is a sparse image
INFO: Looking for file 'system.img'
INFO: system.img is a sparse image
INFO: Looking for file 'system.img'
......
system.img is a sparse image
{ (68.80 MB) 140912 sectors needed at location 3031040 on LUN 0}
{ (8.00 KB) 16 sectors needed at location 3293184 on LUN 0}
{ (8.00 KB) 16 sectors needed at location 3293320 on LUN 0}
{ (119.97 MB) 245696 sectors needed at location 3297336 on LUN 0

Actually this is issue was caused by the sparse image format. We shall transfer it to RAW image format.


unsafeHTML not working well outside of render()

Calling unsafeHTML in a plain string does not return the plain string unescaped.
console.log((html`Foo`).getHTML()); //  'Foo'
console.log((html`${unsafeHTML("Foo")}`).getHTML()); // '<!--{{lit-8186002727240103}}-->'

From what I could find out, unsafeHTML rather than returning a template with the HTML content in .values, it returns a function that takes a Part.
How can I get the "final" input when using unsafeHTML?

unsafeHTML() is a lit-html directive, you have to use it inside render(). If you want HTML outside of lit-html, just use the standard .innerHTML setter on an element.

Then if I want to test unsafeHTML without rendering, what would be the way to go? In that example, how can I test that unsafeHTML is returning a plain string Foo?
console.log((html`${unsafeHTML("Foo")}`).getHTML())

Why without rendering? You can use a simple utility to get the resulting HTML of using the directive.
function getHtml(template: TemplateResult) {
  const div = document.createElement('div');
  render(template, div);
  return div.innerHTML;
}

console.log(getHtml(html`${unsafeHTML("Foo")}`));

unsafeHTML is a directive and doesn't "return" anything. It creates some DOM from the HTML and appends it into the DOM where the template is. Directives only run on render.
You can check the rendered DOM instead.

Thank you for the clarification!


The Client will block forever when the host address is faked.

What version of gRPC and what language are you using?
The master branch, C++.
What operating system (Linux, Windows, …) and version?
ArchLinux.
What runtime / compiler are you using (e.g. python version or version of gcc)
$ gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-pc-linux-gnu/7.1.1/lto-wrapper
Target: x86_64-pc-linux-gnu
Configured with: /build/gcc/src/gcc/configure --prefix=/usr --libdir=/usr/lib --libexecdir=/usr/lib --mandir=/usr/share/man --infodir=/usr/share/info --with-bugurl=https://bugs.archlinux.org/ --enable-languages=c,c++,ada,fortran,go,lto,objc,obj-c++ --enable-shared --enable-threads=posix --enable-libmpx --with-system-zlib --with-isl --enable-__cxa_atexit --disable-libunwind-exceptions --enable-clocale=gnu --disable-libstdcxx-pch --disable-libssp --enable-gnu-unique-object --enable-linker-build-id --enable-lto --enable-plugin --enable-install-libiberty --with-linker-hash-style=gnu --enable-gnu-indirect-function --disable-multilib --disable-werror --enable-checking=release --enable-default-pie --enable-default-ssp
Thread model: posix
gcc version 7.1.1 20170630 (GCC)
What did you do?
I modify greeter_client.cc:
        GreeterClient greeter(grpc::CreateChannel(
          "localhost:50051", grpc::InsecureChannelCredentials()));

to use a fake host name, such as "badhost":
        GreeterClient greeter(grpc::CreateChannel(
          "badhost:50051", grpc::InsecureChannelCredentials()));

What did you expect to see?
The client's SayHello() can return.
What did you see instead?
The client blocks the SayHello() for ever.

This looks similar to https://github.com/grpc/grpc/issues/12272.
cc @y-zeng - I believe the issue is that failed DNS resolution (with the native resolver) doesn't follow fail fast semantics. (i.e., if an RPC is made with the default  setting of fail_fast to true, and the hostname fails to resolve, then the resolver will repeatedly retry name resolution rather than setting connectivity state to transient failure and ending the call).

@apolcyn
Thanks for your comment!
The Client't main thread is blocking in grpc::CompletionQueue::Pluck:
#0  0x00007ffff6619076 in epoll_pwait () from /usr/lib/libc.so.6
#1  0x00007ffff7145b78 in ?? () from /usr/lib/libgrpc.so.4
#2  0x00007ffff7144b24 in ?? () from /usr/lib/libgrpc.so.4
#3  0x00007ffff714c25f in grpc_pollset_work () from /usr/lib/libgrpc.so.4
#4  0x00007ffff717482c in ?? () from /usr/lib/libgrpc.so.4
#5  0x00007ffff7172f23 in grpc_completion_queue_pluck () from /usr/lib/libgrpc.so.4
#6  0x00007ffff74f746e in grpc::CoreCodegen::grpc_completion_queue_pluck(grpc_completion_queue*, void*, gpr_timespec, void*) () from /usr/lib/libgrpc++.so.1
#7  0x000055555556b36d in grpc::CompletionQueue::Pluck (this=0x7fffffffe4a0, tag=0x7fffffffe550)
    at /usr/include/grpc++/impl/codegen/completion_queue.h:224
#8  0x000055555556c869 in grpc::BlockingUnaryCall<helloworld::HelloRequest, helloworld::HelloReply> (
    channel=0x5555557975d0, method=..., context=0x7fffffffe760, request=..., result=0x7fffffffe6d0)
    at /usr/include/grpc++/impl/codegen/client_unary_call.h:59
#9  0x0000555555569254 in helloworld::Greeter::Stub::SayHello (this=0x555555790e10, context=0x7fffffffe760,
    request=..., response=0x7fffffffe6d0) at helloworld.grpc.pb.cc:32
#10 0x000055555557251a in GreeterClient::SayHello (this=0x7fffffffe928, user="world") at greeter_client.cc:58
#11 0x000055555557221b in main (argc=1, argv=0x7fffffffea98) at greeter_client.cc:82

While another thread is keeping on retrying getaddrinfo():
#0  0x00007ffff6604604 in getaddrinfo () from /usr/lib/libc.so.6
#1  0x00007ffff714ee1e in ?? () from /usr/lib/libgrpc.so.4
#2  0x00007ffff714f38c in ?? () from /usr/lib/libgrpc.so.4
#3  0x00007ffff714d020 in ?? () from /usr/lib/libgrpc.so.4
#4  0x00007ffff714cf12 in ?? () from /usr/lib/libgrpc.so.4
#5  0x00007ffff71fff57 in ?? () from /usr/lib/libgrpc.so.4
#6  0x00007ffff7755049 in start_thread () from /usr/lib/libpthread.so.0
#7  0x00007ffff6618f0f in clone () from /usr/lib/libc.so.6

BTW, how can I build libgrpc.so with debug information? (I have posted it in group, but unfortunately no response.) Thanks in advance!

I don't think we would really consider this a bug and that stack trace looks like I would expect in the situation you described. The library isn't "stuck" or anything; it is simply still trying to make your request. If you set a deadline, then after that deadline it will stop trying, and the SayHello call will return.

@murgatroid99 I've heard the other argument - I'm wondering what wait_for_ready should mean for name resolution.

@murgatroid99 I find set_deadline() has a problem: for example, if I set 100 seconds, even it is communicating now, the connection is still cut off. So I think what I need is when the connection is in "IDLE" for 100 seconds, then the connection should be over.
BTW, can gRPC do some retry count limitaion? Such as if it has tried 10 times and can't get the correct DNS result, report to caller function it is failed?
Thanks!

@apolcyn @murgatroid99 After debugging, I find it is because in executor_thread:
if (ts->shutdown) {
    if (GRPC_TRACER_ON(executor_trace)) {
      gpr_log(GPR_DEBUG, "EXECUTOR[%d]: shutdown",
              (int)(ts - g_thread_state));
    }
    gpr_mu_unlock(&ts->mu);
    break;
  }

The ts->shutdown is always false, so retry getaddrinfo() again and again.
Thanks!

I suspect that this is related to #9646 and/or #12867.

Actually, I take back what I said -- I don't think those issues are relevant.
I don't think wait_for_ready is relevant in this case; that affects behavior when an initial connection attempt fails, but in this case we are not even getting to try a connection, because the resolver is not returning any results.
It is actually intentional behavior that the resolver does not return any results for an unknown address.  This is because even if an address is unknown at the moment the channel is created, the name could potentially spring into existence later, at which point the channel would start connecting and everything would start working again.
That having been said, we should honor the RPC deadline even if the resolver doesn't return anything, and I suspect that we're not doing that right now.  So I think that's the thing that needs to be fixed here.

@apolcyn @murgatroid99 @markdroth Is there any update on this issue? Thanks!

I'm the right person to fix this, but it may be a couple of weeks before I have time to look at it.  I'll update the bug when I do.

@markdroth Got it, thanks!

I can't reproduce this.  I changed the greeter client to change localhost to badhost and added the following lines:
   context.set_wait_for_ready(true);
   context.set_deadline(
       std::chrono::system_clock::now() + std::chrono::seconds(10));

When I ran it, the RPC did indeed return DEADLINE_EXCEEDED after 10 seconds.
Please re-open if this is still happening for you.


SCRIPT5009: 'FormData' is undefined in IE9

Is this on purpose or am I expected to include another third-party lib to support it on IE9?
I am using Request with Browserify (latest versions), which in turns depends on form-data.

Hey,
Didn't know request works in a browser. :) I thought it's server-side project, and form-data  definitely is.
So for browserify we fallback to native browser FormData object (this is what it was implemented after).
In your case, something like es5-shim (haven't tried myself though) might help to bring old browsers up to speed with modern tech.
Or looks like es5-shim isn't enough either, check out this thread:
http://stackoverflow.com/questions/15475215/formdata-in-ie8-9
PS. Do you know what is request's maintainers' take on using it in a browser?
/cc @simov

Yes, that makes sense, I think it's mostly Browserify's job to ensure the browser compatibility, I just reported this bug with them. I think request is also mostly or at least initially server-side and Browserify works to ensure it works too in the browser.


Removing sleep from build_and_redeploy.sh

Motivation
There's a wait to tell helm commands to wait until things are initialized. See @christos-h's comments on #1073
Proposal
Stop using sleep and use --wait instead
Test Plan
CI

reviewer checklist

Current dependencies on/for this PR:

main

PR #1062

PR #1073

PR #1078   👈

PR #1075

This comment was auto-generated by Graphite.


Change "g-cloud" string to variable from url

Hi @ArturMichalak! Thanks for submitting a contribution. Can I ask how you found this repo and this particular issue?
Usually we ask contributors to raise an issue first, which we can then investigate and prioritise. Feel free to raise an issue you've found with the repo, or if it's a security issue, please follow the GDS vulnerability reporting steps. Thanks!

Hey @katstevens,
at work I'm using code based on digitalmarketplace and I want to learn the basics because I have a problem with flow of Supplier offers search. https://www.digitalmarketplace.service.gov.uk/digital-outcomes-and-specialists/opportunities.
Sorry for doing pull request before making an issue,  I wanted to trace the process of writing changes.


Fix many small issues

Several fixes / updates:

Pins datastore and google-cloud-logging
Only start Prometheus if configured to
Update how we call libcloudforensics to match current release
Add authentication message if datastore client initialization fails for auth reasons
Only load google_cloud if configured
Add block device checks to preprocessor
Fix up some local_path in evidence to always point to the last processed path.
Add required states for binary extractor Job so correct preprocessors run
Move dependency check output to debug logs
Adds some more debug log lines
Fix some linting

Small nits, LGTM


[tests-only] Bump core commit id for tests

Similar to https://github.com/owncloud/ocis/pull/1111
Gets latest test code and scenarios from owncloud/core for TUS and other tests.

@labkode @ishank011 if you are back online today, this can be safely merged. It passed drone CI before Christmas.
Then when the drone hardware is running again, I can move forward with more test-suite things.

@labkode @ishank011 if you are back online today, this can be safely merged. It passed drone CI before Christmas.
Then when the drone hardware is running again, I can move forward with more test-suite things.


MM-T2372 Deleting: Thread does not move when deleting post

Summary
Cypress test: MM-T2372 Deleting
Ticket Link
https://automation-test-cases.vercel.app/test/MM-T2372

Hello @yulyanaR,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

/release-note none

/release-note-none

/update-branch

Please fix step and assertion indicators

Thanks, it's been updated.


esp8266 serial output says its connected to my wifi, but it's not

I'm having issues getting this set up.
I can connect to the wifi config page by grounding GPIO10, I set this up and save it.
Serial port shows the below indicating it has connected to my WiFi network, but  access via IP or .local addresses does not work and no sign of any connection being made in my router logs.
11:33:50.892 -> *WM: <IP_ADDRESS>
11:33:50.892 -> *WM: static gateway
11:33:50.892 -> *WM: <IP_ADDRESS>
11:33:50.892 -> *WM: static netmask
11:33:50.892 -> *WM: <IP_ADDRESS>
11:33:50.926 -> *WM: Sent wifi save page
*WM: Connecting to new AP
11:33:52.894 -> *WM: Connecting as wifi client...
11:33:52.928 -> *WM: Custom STA IP/GW/Subnet
11:33:52.928 -> *WM: <IP_ADDRESS>
11:33:52.928 -> *WM: Already connected. Bailing out.
Should save config
11:33:52.965 -> WiFi connected! User chose hostname 'irblaster' passcode 'xxx' and port '8080'
11:33:52.965 ->  config...
11:33:52.965 -> {"hostname":"irblaster","passcode":"pass","port_str":"8080","user_id":"xxx","ip":"<IP_ADDRESS>","gw":"<IP_ADDRESS>","sn":"<IP_ADDRESS>"}
11:33:53.002 -> Writing config file
11:33:53.002 -> Config written successfully
11:33:53.002 -> *WM: freeing allocated params!
11:33:53.002 -> WiFi configuration complete
11:33:53.002 -> Local IP: <IP_ADDRESS>
11:33:53.002 -> URL to send commands: http://irblaster.local:8080
11:33:53.002 -> ArduinoOTA started
11:33:53.002 -> MDNS http service added. Hostname is set to irblaster.local:8080
11:33:53.002 -> HTTP Server started on port 8080
11:33:53.002 -> Starting UDP
11:33:53.036 -> Local port: 8888
11:33:53.036 -> Waiting for sync
11:33:53.036 -> Transmit NTP Request
11:33:53.036 -> time.google.com: <IP_ADDRESS>
No NTP Response :-(
11:33:54.505 -> No errors detected with security configuration
11:33:54.505 -> Epoch time from timeServer is unexpectedly old, probably failed connection to the time server. Check your network settings
11:33:54.540 -> 25
11:33:54.540 -> Invalid EPOCH time, security checks may fail if unable to sync with NTP server
11:33:54.540 -> Ready to send and receive IR signals
Turning off the LED to save power.

If I reset the esp8266 without GPIO10 set to ground it just boots back into the WiFi config mode.
11:40:46.922 -> ESP8266 IR Controller
11:40:46.922 -> Config pin GPIO10 set to: 1
11:40:46.922 -> mounted file system
11:40:46.922 -> reading config file
11:40:46.922 -> opened config file
11:40:46.922 -> {"hostname":"irblaster","passcode":"xxx","port_str":"8080","user_id":"xxx","ip":"<IP_ADDRESS>","gw":"<IP_ADDRESS>","sn":"<IP_ADDRESS>"}
11:40:46.922 -> parsed json
11:40:46.922 -> *WM: Adding parameter
11:40:46.922 -> *WM: hostname
11:40:46.922 -> *WM: Adding parameter
11:40:46.922 -> *WM: passcode
11:40:46.922 -> *WM: Adding parameter
11:40:46.922 -> *WM: port_str
11:40:46.922 -> *WM: Adding parameter
11:40:46.922 -> *WM: user_id
11:40:46.922 -> Using Static IP
11:40:46.922 -> *WM:
11:40:46.922 -> *WM: AutoConnect
11:40:46.922 -> *WM: Connecting as wifi client...
11:40:46.922 -> *WM: Custom STA IP/GW/Subnet
11:40:46.922 -> *WM: <IP_ADDRESS>
11:40:46.922 -> *WM: Using last saved values, should be faster
11:40:46.922 -> *WM: Connection result:
11:40:46.922 -> *WM: 0
11:40:46.922 -> Entered config mode
11:40:46.922 -> <IP_ADDRESS>
11:40:46.922 -> IR Controller Configuration
11:40:46.922 -> *WM:
11:40:46.922 -> *WM: Configuring access point...
11:40:46.922 -> *WM: IR Controller Configuration
11:40:46.922 -> *WM: AP IP address:
11:40:46.922 -> *WM: <IP_ADDRESS>
11:40:46.922 -> *WM: HTTP server started

I've re-flashed and gone through the config multiple times ensuring everything is correct but have the same issues. I've even put in the wrong wifi password to see if it does anything different, but nope it says it is connected.

Can you post your build settings?
What version of the ESP8266 core are you using?
What version of lwIP are you building against?
What board do you have selected?

My settings are:

Originally, I left things on default which was IwIP set to v2 Lower Memory, but since changing to v1.4 Higher Bandwidth I have managed to get a bit further and access the webserver via my WiFi. The serial output reports my external IP correctly meaning it did manage to connect to my WiFi and the outside internet. I managed to load the blue header of the IR Controller webpage but the rest of the page didn't load.
After resetting the board a few times I noticed I can ping it for a short while but it consistently stops after 10/11 pings and is then no longer reachable. This happens even if I just reset the board and don't even attempt to load the webpage.

My first thought was a power issue since I was still powering it via a USB cable from my Macbook (which I assume can easily power it). But I've since tried running via USB from a 2A phone charger and also via a 5v source on a breadboard - all display the same issue.
I have 2 boards and both behave in the same way.
I'm new to ESP8266/Arduino so may be missing something really obvious!
Thanks

You need to downgrade to 2.4.0 for the ESP8266 version, you’re probably using 2.4.2 or 2.4.1 which has some kind of networking memory leak issue with WiFiManager

On Oct 28, 2018, at 4:00 PM, Brendan<EMAIL_ADDRESS>wrote:
My settings are:
https://user-images.githubusercontent.com/7316130/47621030-9eb99e00-dae9-11e8-8bbe-27c75b4d42b3.png
Originally, I left things on default which was IwIP set to v2 Lower Memory, but since changing to v1.4 Higher Bandwidth I have managed to get a bit further and access the webserver via my WiFi. The serial output reports my external IP correctly meaning it did manage to connect to my WiFi and the outside internet. I managed to load the blue header of the IR Controller webpage but the rest of the page didn't load.
After resetting the board a few times I noticed I can ping it for a short while but it consistently stops after 10/11 pings and is then no longer reachable. This happens even if I just reset the board and don't even attempt to load the webpage.
https://user-images.githubusercontent.com/7316130/47621023-7df14880-dae9-11e8-8096-4cea7c527e4d.png
My first thought was a power issue since I was still powering it via a USB cable from my Macbook (which I assume can easily power it). But I've since tried running via USB from a 2A phone charger and also via a 5v source on a breadboard - all display the same issue.
I have 2 boards and both behave in the same way.
I'm new to ESP8266/Arduino so may be missing something really obvious!
Thanks
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/mdhiggins/ESP8266-HTTP-IR-Blaster/issues/225#issuecomment-433736400, or mute the thread https://github.com/notifications/unsubscribe-auth/ADcO6q8uvRRBvhMHKkaoeBnJdIIJ9FIAks5upgzFgaJpZM4X9Xdq.

Thanks, I was indeed on 2.4.2 - I just installed the latest from https://github.com/esp8266/Arduino# so have downgraded and it now works.
Thanks again


crash tester test suite

Motivation:
It's often very important to test that in certain cases we actually
crash with a certain error message whilst having access to all of NIO.
Modifications:
Add a crash tester test suite.
Result:
We can test crashers.

This is really cool :)


Custom recipient / custom to address on swaps

Overview
Currently we offer the ability to enter a custom address when a user has a wallet connected that doesn't support a specific chain (EVM wallet connected but attempting to swap to BTC, for example).
We should extend this functionality to always allow for a custom recipient in the swapper.  This isn't always supported by all routes, so we may need to add a bit of help text about this if a user attempts to enter a custom recipient.
References and additional details
Some risk here of a user entering an incorrect address. We can validate the format, but not much else
Acceptance Criteria

ability to input custom recipient to swapper
tooltip / info text alerting the user that not all routes support this functionality
validate the address is correctly formatted / valid addy for the specific output chain
on confirm screen we should link out the custom recipient address to a block explorer so people can easily confirm its validity
add flag on event for mixpanel so that we can track the usage of this feature

Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

Spec is a lil out of date https://www.notion.so/shapeshift/Input-custom-recipient-address-on-Trades-b1ccaf4df489421e8e60c63b1f80dce0?pvs=4 need to

finish technical due diligence for wallet types (since snap launched)
spike through tracking and how we'd like to see it
articulate success/failure metrics.

@twblack88 - did the spec for this get finished off? might be able to slot it in if its ready from a product perspective.

I think it just needs a spike on the technical bit. the product experience is pretty dialed. Hop does this pretty well and a few others we can look to. Will bring up on roadmap.

@reallybeard to design component for a custom address dropdown on the primary swapper input screen.

Flow

Recipient address shows the address we get from the wallet.
The user can click the edit icon, if the user hovers over the edit button they will see a tooltip with the text "Enter a custom recipient address for this trade"
The custom recipient text input box appears.
The user pastes their custom address, if the address is invalid we show an error message below the text field in yellow.200
Once the user has entered a valid address, they can click either the check mark button to save or the x mark button to cancel.
After the user has saved their custom address, the row changes from text to a tag with an x mark button on it. And also note that the label of the row has now changed from Recipient Address, to Custom Recipient Address
If the user hovers over the tag they will see a tooltip that says, "This is your custom recipient address"
If the user decides they want to remove the custom recipient address, they click the tag and the field then resets back to the original state with the wallet address.

thanks @reallybeard!