Custom filename should be allowed in download_single

[x] Using latest version as provided on the master branch
[x] Searched for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug
[x] Feature Request
[ ] Question
[ ] Other

Description
In the main function download_single, we should be able to specify a custom filename and use that as opposed to one of the options that is presented. We cannot assume that these options cover all use cases. One simple example: If I want to save the file as the Spotify ID of a song, this is currently impossible. There are other conceivable use-cases that involve custom sequences for which a custom name override would be vary handy

If this looks like something you want to support, I can submit a PR

First of all, this use case is specific to calling spotdl from another python file. An example:
url = 'my spotify url'
output_dir = '.'
from spotdl.spotdl import download_single
from spotdl import const as spotdl_const
from spotdl import handle as spotdl_handle
spotdl_const.args = spotdl_handle.get_arguments(raw_args=args.spotdl_base_args + [
    '-f', output_dir,
    '-s', url
])
download_single(raw_song=spotdl_const.args.song)

This approach is still a bit hacky because you use parsed global args instead of passing them to the download_single function, but it works.
Now, say we also want to save as a custom filename, for instance choosing the spotify URL as filename. This is currently not possible, but it will become possible if we do:
download_single(raw_song=spotdl_const.args.song, custom_file_name='{}.mp4'.format(url))

And in download_single on line 138 we do:
if custom_name is None:
    if meta_tags is not None:
        refined_songname = internals.format_string(const.args.file_format,
                                                   meta_tags,
                                                   slugification=True)
        log.debug('Refining songname from "{0}" to "{1}"'.format(songname, refined_songname))
        if not refined_songname == ' - ':
            songname = refined_songname
    else:
        log.warning('Could not find metadata')
        songname = internals.sanitize_title(songname)
else:
    songname = custom_name

after which we can just pass our custom name and it overwrites the default behavior.
Let me know what you think. Alternatively, we could refactor download_single to accept parsed arguments instead of getting them from the global const.args

Wow, never expected people would use this tool as a library. This is really creative!
Making this approach further hacky, you could use -ff option here to set custom filenames. So, it would look this for your case:
url = 'https://open.spotify.com/track/2DGa7iaidT5s0qnINlwMjJ'
output_dir = '.'
base_args = ['-ll', 'DEBUG', '-o', '.m4a', '--overwrite', 'skip']
from spotdl.spotdl import download_single
from spotdl import const as spotdl_const
from spotdl import handle as spotdl_handle
spotdl_const.args = spotdl_handle.get_arguments(raw_args=base_args + [
    '-f', output_dir,
    '-s', url,
    '-ff', url.split('/')[-1]
])
download_single(raw_song=spotdl_const.args.song)

It will automatically append .{extension} to whatever you pass in -ff option.

download_single(raw_song=spotdl_const.args.song, custom_file_name='{}.mp4'.format(url))

We could have something like this but we have a lot of options (--dry-run, etc.) and most of them would possibly be passed to other functions (--avconv, to convert.Converter class, etc.).
Also, download_list would need to have this options as well, so they would be passed to download_single whenever one downloads tracks from a file. This will get messier than our current approach (defining arguments in const.args) real quick.

Alternatively, we could refactor download_single to accept parsed arguments instead of getting them from the global const.args

We would have to pass parsed arguments to each function with this as well to which I'd rather prefer the way it currently is.
We might have to look into this again if someday, we officially support using this tool as a library.

We might have to look into this again if someday, we officially support using this tool as a library.
Agreed, that is indeed really creative - and doesn't look that bad considering the API wasn't meant to be used outside of spotdl itself so far... :smile:

Your view on the package is interesting to me, because I'm not used to think that way. I always see a Python package as a well-defined collections of modules, that can contain functions, classes, or attributes, and should be able to be used kind of independently. In that sense it's not weird that we can use spotdl's main functions.
I agree with you that it would be better to support all options if you start adding them as keyword arguments. However, I also think the current approach of global args is a bit hacky. Wouldn't it be better to pass the args namespace to functions that need them explicitly?

As you say, a well-written executable program should also be easy to use as a library. However, I'm still not sure whether or not passing all the appropriate arguments to each function is a good idea as I mentioned in https://github.com/ritiek/spotify-downloader/issues/359#issuecomment-424802508.
I'm going to close this issue assuming you were able to save the track with filename as Spotify URL with passing '-ff', url.split('/')[-1] as an argument when using it as a library. You should be able to solve most such scenarios with this option.


RtLogs.java:40-45: Continue implementing this class, take...

The puzzle 135-7b38f89f from #135 has to be resolved:
https://github.com/amihaiemil/docker-java-api/blob/ab0d78ec783de53306d63a66dd459598d58dc8c4/src/main/java/com/amihaiemil/docker/RtLogs.java#L40-L45
The puzzle was created by amihaiemil on 12-Jun-18.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@amihaiemil please pay attention to this new issue.

@amihaiemil/z please, pay attention to this issue


Add Elasticsearch integration tests

In this PR, I added instructions for Travis to pull different versions of ES, spin up a 1-node cluster, and seed it with some test data. I also added a basic integration test just to be sure this is working.

Codecov Report

Merging #35 into master will increase coverage by 11.25%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master      #35       +/-   ##
===========================================
+ Coverage   59.66%   70.91%   +11.25%
===========================================
  Files           2        2
  Lines         533      533
===========================================
+ Hits          318      378       +60
+ Misses        215      155       -60

Impacted Files
Coverage Δ

R/elasticsearch_parsers.R
75.94% <0%> (+13.36%)
:arrow_up:

Continue to review full report at Codecov.

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


[MM-23612] Added proper permissions around add_channel_dropdown menu items

Summary
I had overlooked permissioning for the add channel dropdown on the new sidebar. This PR adds the correct permissions for those 2 menu items so they will only display if the user has permissions.
Ticket Link
https://mattermost.atlassian.net/browse/MM-23612

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-5191.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

Test server destroyed


Theme constants in common module

cc @domasn

Would be nice to return to this 🙂

Closing in favor of #25


Update Autosubmit GUI Icon

In GitLab by @ jberlin on Aug 12, 2022, 17:56
Currently the icon of the GUI is the default React framework icon, we should create a new one and add it to the app
@ggcr @mcastril

In GitLab by @ jberlin on Aug 25, 2022, 17:49
tomorrow I will try to add it to the app

In GitLab by @manuel-g-castro on Aug 25, 2022, 12:10
Steve Jobs would be amazed over such a display of design prowess

In GitLab by @ jberlin on Aug 25, 2022, 11:59
looking very good !! I support the idea, a masterpiece of app icon engineering

In GitLab by @ jberlin on Aug 26, 2022, 10:21
yes, it is a very nice design, I will find a proper place to store these files in the source code so we can reference these in the app

In GitLab by @ggcr on Aug 26, 2022, 08:24
Looks good at the GUI Navbar :)

In GitLab by @ jberlin on Aug 26, 2022, 16:49
mentioned in commit 2ec15c92bedfd726e613ac9909959945e3fcc964

In GitLab by @ jberlin on Aug 26, 2022, 16:53
I added the tab icon, it does it work in Firefox, but for some reason I still see the old one.

In GitLab by @ jberlin on Nov 10, 2022, 15:07
Validated in the development VM

In GitLab by @mcastril on Aug 26, 2022, 10:19
Thanks for the initiative guys, I think it's in a good direction.
If I have the chance I'll hand it to a graphic designer to give it a look. Anyway, I'd be happy with this version.

In GitLab by @mcastril on Aug 26, 2022, 10:26
Yes, and we have to modify the GitLab and Trello icons too. But please wait 1 week in case that we have a v2.

In GitLab by @manuel-g-castro on Aug 26, 2022, 10:29
I made a Carles move and already changed the Trello card! Hahahahha


Posible bug

Apologies, I have now lost the log sigh   However, my environment gave a log error trying to publish a message where the value was supposed to be a string or buffer and was in fact a number (It was a valid reading, but it might not be a smartthings proper sensor reading - don't ask -:) )
I changed the following code in  your sendMQTT Function in mqttout.js to guard this and it seems to have worked
Your code line 220
 mqttClient.publish(topic, msg_tosend.event.value, pub_options, (err) => {
My change
const valToSend = (typeof(msg_tosend.event.value) == "number") ? Number(msg_tosend.event.value).toString() : msg_tosend.event.value;    mqttClient.publish(topic, valToSend, pub_options, (err) => {

Thanks!  I'll make a similar change to the posted code after some regression tests of my own.


Monsters don't attack

Hi All,
First time getting involved in an open source project so excuse any mistakes. Also, new user of Minecraft!
Anyway, my problem is that all the monsters seem to be harmless.
They don't attack and I don't take damage when I walk into them. Is there an option I haven't turned on or is there a problem in the code?
Running windows x64 r173.
Thanks!

The problem is that MCServer has no AI for monsters, apart from movement (sometimes). I think this issue is a dupe though.

Ah sorry about the dupe.
I'll look into the AI then, I'm a bit of a programmer anyway, but I'm not promising anything! :)
Thanks for the response

Here's a relevant forum thread for you: https://github.com/mc-server/MCServer/issues/910

Awesome, thanks again!

For me, hostile mobs walk towards you really fast, then sometimes hit you.. And if they do hit you, you get knocked way back.

This was fixed with the new AI and PathFinder.
Beware that some monsters are still a bit weird, but it's much better now.


Lifecycle hooks typings not showing

Version
2.6.11
Reproduction link
https://null
Its more of typing issue than functionality bug
Steps to reproduce

Setup a typescript project
Create class component using vue-property-decorator
In class enter crea and invoke completions in vscode (use the CTRL   SPACE)
Erase crea and again invoke completions

What is expected?
It should list all the list of lifecycle hooks when requested completions like other functions $forceUpdate..
What is actually happening?
No typings for lifecycle hooks

Here the typings of external package vue-toast is being shown in vue class but not of vue lifecycle hooks

Hi, thanks for your interest but Github issues are for bug reports and feature requests only. You can ask questions on the forum, the Discord server or StackOverflow.

This is not directly related to Vue but rather the plugins or the editor


Implement filter() on collection objects

We need to be able to filter the base collection objects to reduce the items in the list to ones that match the criteria.
See [https://github.com/F5Networks/f5-common-python/wiki/API-Requirements#requirements](Requirement 2).

This was closed by the same merge that fixed #207.    The way to filter is to use 'requests_params'.


[Bug] RangeSlider touch not working on WPF

Description
Steps to Reproduce

Crate a new Xamarin Forms App in VS.
Clear everything in the Main Page and add a RageSlider (set in, max...)
Add a WPF project for the Xamarin Forms App you just created
The slider will work on the other platforms but not in WPF
In WPF, I can't grab the slider point and drag them.

Expected Behavior
-- in the reproduction steps
Actual Behavior
-- in the reproduction steps
Basic Information
-- in the reproduction steps

Version with issue: 1.1.0
Last known good version: ----
IDE: VS
Platform Target Frameworks:

WPF: .Net Framework 4.72

Android Support Library Version:
Nuget Packages:
Affected Devices:

Workaround

Reproduction imagery

Reproduction Link

Hi @RwSgxx, thanks for the report
Could you please check if PanGestureRecognizer works on WPF?
RangeSlider is cross-platform control and it based on PanGestureRecognizer. Perhaps there are some bugs with WPF implementation of this gesture.

It is because of the PanGestureRecognizer. It is designed to work with touch events not mouse events.

@RwSgxx, unfortunately, I don't think this will be implemented in near future then(

@AndreiMisiukevich I know. The iOS build is broken since 3 weeks and the Xamarin Team and Microsoft don't give a flying ... (you know what).
https://developercommunity.visualstudio.com/t/xamarinforms-ios-ioexception-sharing-violation-on/1355811?from=email&viewtype=all#T-N1450332
Totally disappointed. I regret investing so much time in this platform.

@RwSgxx sorry to hear this :(
But unfortunately, I can't help with that. I am even not an MS employee.

@RwSgxx this is not the place to share issues for other products. Also, there are multiple workarounds given and even a message in the link that you shared that it should be fixed: https://developercommunity.visualstudio.com/t/xamarinforms-ios-ioexception-sharing-violation-on/1355811#T-N1450827

@jfversluis you're right.  The part with "don't think this will be implemented in near future then" hit a nerve :)

So back to the problem it self.

The PanGestureRecognizer for WPF is design to work with touch events, not mouse events -> On a touch screen it will work.
You need to create a custom MouseDragGestureRecognizer for WPF, it should expect 3 inputs DragStarted, DragContinue, DragCompleted (each of them receiving also the mouse position).  This should expose the same event as the PanGestureRecognizer (PanUpdated) so that it can be easily integrated in the RangeSlider.
The last piece is a custom renderer for the ThumbFrame for WPF. This should attach to the MouseDown, MouseMove and MouseUp events of the native control and forward them to the MouseDragGestureRecognizer. Here don't forget to capture and release the mouse, on MouseDown and MouseUp.
In the RangeSlider add the MouseDragGestureRecognizer to the two Thumbs.

@RwSgxx thanks for the suggestion)
Feel free to grab this part and submit a PR :) Since this toolkit has "Community" word in the name ;)


feat: allow task name editing for v0.4.x

Reference: https://github.com/alanvardy/tod/pull/245
Tests are missing, I just tested the feature with my account. I would like to have feedback on the code organization as I mainly focused on making things work (which was actually straightforward!). Should I add a method to projects for editing the tasks, instead of using request directly?

Also, bigger problem is on how to display the current content of the task. My idea was to print the current task name and make it editable from the user, but from a quick overview of the inquire lib it's not possible

Also, bigger problem is on how to display the current content of the task. My idea was to print the current task name and make it editable from the user, but from a quick overview of the inquire lib it's not possible

I haven't tried it but initial_value might be what you are looking for
https://docs.rs/inquire/0.6.2/inquire/struct.Text.html
If that doesn't work you could use the Editor option as it allows predefined text
https://docs.rs/inquire/0.6.2/inquire/struct.Editor.html

@BobToninho @titoOdUA I'm going to hold off merging my work for a couple of days or until this gets merged because some of my refactors will conflict with this.

Great! Will finish this in the following days

This is great, thank you!
I have PRs after this that will make it easy to test. Are you ready for me to merge?

So we'll add tests after those PRs? If yes, you can merge!


failed to load tests

Steps to reproduce
create new flutter project and follow the instructions on the following link to setup patrol tests.
add sample test for testing as follows.
patrolTest(
   'counter state is the same after going to home and switching apps',
   nativeAutomation: true,
   nativeAutomatorConfig: NativeAutomatorConfig(
     packageName: 'com.example.patrol_test',
     bundleId: 'com.example.patrol_test',
   ),
   ($) async {
     // Replace later with your app's main widget
     await $.pumpWidgetAndSettle(
       MaterialApp(
         home: Scaffold(
           appBar: AppBar(title: const Text('app')),
           backgroundColor: Colors.blue,
         ),
       ),
     );

     expect($('app'), findsOneWidget);
     await $.native.pressHome();
   },
 );

Actual results

Logs
Connecting to VM Service at http://<IP_ADDRESS>:64687/SGbnY_qqipw=/ws
Failed to load "/Users/madhura/Downloads/patrol_test-b4034526946954849886cf26be3b0fc6e366a5ac/integration_test/example_test.dart": 'package:flutter/src/foundation/binding.dart': Failed assertion: line 149 pos 12: '_debugInitializedType == null': is not true.
dart:core                                                                                _AssertionError._throwNew
new BindingBase
binding.dart:149
package:flutter_test/src/binding.dart                                                                                new _TestWidgetsFlutterBinding&BindingBase&SchedulerBinding
binding.dart:1
package:flutter_test/src/binding.dart                                                                                new _TestWidgetsFlutterBinding&BindingBase&SchedulerBinding&ServicesBinding
binding.dart:1
package:flutter_test/src/binding.dart                                                                                new _TestWidgetsFlutterBinding&BindingBase&SchedulerBinding&ServicesBinding&GestureBinding
binding.dart:1
package:flutter_test/src/binding.dart                                                                                new _TestWidgetsFlutterBinding&BindingBase&SchedulerBinding&ServicesBinding&GestureBinding&SemanticsBinding
binding.dart:1
package:flutter_test/src/binding.dart                                                                                new _TestWidgetsFlutterBinding&BindingBase&SchedulerBinding&ServicesBinding&GestureBinding&SemanticsBinding&RendererBinding
binding.dart:1
package:flutter_test/src/binding.dart                                                                                new _TestWidgetsFlutterBinding&BindingBase&SchedulerBinding&ServicesBinding&GestureBinding&SemanticsBinding&RendererBinding&PaintingBinding
binding.dart:1
package:flutter_test/src/binding.dart                                                                                new _TestWidgetsFlutterBinding&BindingBase&SchedulerBinding&ServicesBinding&GestureBinding&SemanticsBinding&RendererBinding&PaintingBinding&WidgetsBinding
binding.dart:1
package:flutter_test/src/binding.dart                                                                                new _TestWidgetsFlutterBinding&BindingBase&SchedulerBinding&ServicesBinding&GestureBinding&SemanticsBinding&RendererBinding&PaintingBinding&WidgetsBinding&TestDefaultBinaryMessengerBinding
binding.dart:1
package:flutter_test/src/binding.dart                                                                                new TestWidgetsFlutterBinding
binding.dart:1
package:flutter_test/src/binding.dart                                                                                new LiveTestWidgetsFlutterBinding
binding.dart:1
package:integration_test/integration_test.dart                                                                       new IntegrationTestWidgetsFlutterBinding
integration_test.dart:1
package:patrol/src/binding.dart                                                                                new PatrolBinding
binding.dart:1
new PatrolBinding.ensureInitialized
binding.dart:107
patrolTest
common.dart:66
Users/madhura/Downloads/patrol_test-b4034526946954849886cf26be3b0fc6e366a5ac/integration_test/example_test.dart 6:3  main
Patrol version
patrol: ^2.1.0
Patrol Doctor output

Patrol Doctor output
Patrol CLI version: 2.1.0
Program adb found in /Users/madhura/Library/Android/sdk/platform-tools/adb
Env var $ANDROID_HOME is not set
Program xcodebuild found in /usr/bin/xcodebuild
Program ideviceinstaller not found (install with `brew install ideviceinstaller`)

Flutter Doctor output

Flutter Doctor output
[✓] Flutter (Channel stable, 3.7.3, on macOS 12.5.1 21G83 darwin-arm64, locale en-LK)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.2)
[✓] VS Code (version 1.80.2)
[✓] Connected device (3 available)
[✓] HTTP Host Availability

Hi @madura7, thank you for the issue. I need some more info about it:

Which command did you use to run the tests?
Could you provide minimum reproducible example? I would be easier if we could debug this locally

I also have the same error, I'm using version ^1.1.0 and flutter test integration_test/app_test.dart command.

Hi @eliezer-castro, to run Patrol tests that use the native automation feature (like $.native.pressHome()), you have to use patrol_cli:
$ patrol test --target integration_test/app_test.dart

Install it with dart pub global activate patrol_cli. It's also all described in our getting started guides.

I see the issue now. I have used native commands but not run with  run it with patrol_cli. it worked when runs with the cli.


Isn't clear which version of Symfony is supported

Hello, I would like to use your library, but I tried with Symfony 3.0 and 2.7 and both aren't working. It says:
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - creads/api2symfony 1.1.1 requires symfony/filesystem ~2.6 -> no matching package found.
    - creads/api2symfony 1.1.0 requires symfony/filesystem ~2.6 -> no matching package found.
    - Installation request for creads/api2symfony ^1.1 -> satisfiable by creads/api2symfony[1.1.0, 1.1.1].

I don't really know which version of symfony has symfony/filesystem 2.6. Are you going to support some latter versions of Symfony, or can you at least state the required version here at GitHub?
Thanks and thanks for your work.

Hello @maral !
I just tried with a fresh new project under symfony 2.7 and it works as i expected.
It is normal that you can't install it under symfony >= 3 because it uses symfony.filesystem: self.version and we do not handle version > 2.x.x.
You are right, we should manage it as well if test pass.
@pitpit

Hi @qpautrat, thanks for quick answer, I will try it with 2.7 then. Are you going to support Symfony 3.0 in future though?

I honestly don't know. I don't have write access, I'm not working at Creads anymore and I will not maintain this repository. However I pinged @pitpit, will see.

Hi @maral,
Unfortunatly, we don't plan to maintain effort on this bundle for now. However It should not be really hard to migrate code to SF3. Would you like to submit a PR ?
@qpautrat I restored your write access on the bundle. It was a mistake

Hey @pitpit, I would love to, but I'm afraid it would take me way too long,
I'm not Symfony well-spoken...
2016-04-13 14:07 GMT+02:00 Damien Pitard<EMAIL_ADDRESS>
Hi @maral https://github.com/maral,
Unfortunatly, we don't plan to maintain effort on this bundle for now.
However It should not be really hard to migrate code to SF3. Would you like
to submit a PR ?
@qpautrat https://github.com/qpautrat I restored your write access on
the bundle. It was a mistake
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/creads/api2symfony/issues/31#issuecomment-209398983

--
S přáním krásného dne
Marek Lisý
mail<EMAIL_ADDRESS>|  phone: +420 702 143 850 <+420702143850>
|  blog: www.mareklisy.cz http://www.mareklisy.cz/

Is it possible to jump to Symfony3?


biome / cavity / erosion data not handled

On the biome front, optional output to VSPro would be super. Would need to make a polygon from the map weights. See: this method in VSProExtensions


No images render on Post cards in Posts section

No image on Post card in Posts section.  Even though I have a featured image in every Post , it seems to dont see the source url...
post:  {
  id: 85,
  date: '2024-10-02T05:12:45',
  date_gmt: '2024-10-02T03:12:45',
  guid: { rendered: 'http://<IP_ADDRESS>:8080/?p=85' },
  modified: '2024-10-13T14:36:18',
  modified_gmt: '2024-10-13T12:36:18',
  slug: 'post-4',
  status: 'publish',
  type: 'post',
  link: 'http://<IP_ADDRESS>:8080/post-4/',
  title: { rendered: 'Post 4' },
  content: { rendered: '', protected: false },
  excerpt: { rendered: '', protected: false },
  author: 1,
  featured_media: 74,
  comment_status: 'open',
  ping_status: 'open',
  sticky: false,
  template: '',
  format: 'standard',
  meta: { footnotes: '' },
  categories: [ 1, 98 ],
  tags: [],
  class_list: [
    'post-85',
    'post',
    'type-post',
    'status-publish',
    'format-standard',
    'has-post-thumbnail',
    'hentry',
    'category-bez-kategorii',
    'category-solidity'
  ],
  _links: {
    self: [ [Object] ],
    collection: [ [Object] ],
    about: [ [Object] ],
    author: [ [Object] ],
    replies: [ [Object] ],
    'version-history': [ [Object] ],
    'predecessor-version': [ [Object] ],
    'wp:featuredmedia': [ [Object] ],
    'wp:attachment': [ [Object] ],
    'wp:term': [ [Object], [Object] ],
    curies: [ [Object] ]
  }
}

Did you add your host to next.config?
https://nextjs.org/docs/messages/next-image-unconfigured-host

Same problem, but by default of the template it's already set and still no image rendered on page post.
`/** @type {import('next').NextConfig} */
const nextConfig = {
images: {
remotePatterns: [
{
protocol: "https",
hostname: process.env.WORDPRESS_HOSTNAME,
port: "",
pathname: "/**",
},
],
},
};
export default nextConfig;`

Same problem, but by default of the template it's already set and still no image rendered on page post.
My next.config.mjs
/** @type {import('next').NextConfig} */
const nextConfig = {
  images: {
    remotePatterns: [
      {
        protocol: "https",
        hostname: process.env.WORDPRESS_HOSTNAME,
        port: "",
        pathname: "/**",
      },
    ],
  },
};

export default nextConfig;

I change my protocol to "http" and works for me in local development.

Same problem, but by default of the template it's already set and still no image rendered on page post.
My next.config.mjs
/** @type {import('next').NextConfig} */
const nextConfig = {
  images: {
    remotePatterns: [
      {
        protocol: "https",
        hostname: process.env.WORDPRESS_HOSTNAME,
        port: "",
        pathname: "/**",
      },
    ],
  },
};

export default nextConfig;

I change my protocol to "http" and works for me in local development.
/** @type {import('next').NextConfig} */
const nextConfig = {
  images: {
    remotePatterns: [
      {
        protocol: "http",
        hostname: process.env.WORDPRESS_HOSTNAME,
        port: "",
        pathname: "/**",
      },
    ],
  },
};

export default nextConfig;

Cheers thanks for figuring this out @pcteilor


Bug: PRs are not populated with the PR template's contents

What happened?
When creating a new PR, its content is not automatically filled by the PR template.
We probably need to create a pull_request_template.md markdown file under PULL_REQUEST_TEMPLATE/ instead of the existing pr.yml YAML file.
Reference

https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/creating-a-pull-request-template-for-your-repository

What steps did you take?
Go to Pull requests -> New pull request -> Choose different branches or forks -> Create pull request
What behavior did you expect?
The PR body should be filled with the contents of https://github.com/repository-service-tuf/repository-service-tuf-api/blob/main/.github/PULL_REQUEST_TEMPLATE/pr.yml
Relevant log output
No response
Code of Conduct

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

I can take care of this.


Add support for NOT operator in MySQL

Add support for NOT operator in MySQL similarly as it is implemented in PostgreSQL. This enables NOT EXISTS subqueries in MySQL.

NOT operator is now included in latest v2.5.0 release.


Add some strictness annotations

PR checklist:

[ ] Test coverage for the proposed changes
[ ] PR description contains example output from repl interaction or a snippet from unit test output
[ ] Any changes that could be relevant to users have been recorded in the changelog

Additionally, please justify why you should or should not do the following:

[ ] Confirm replay/back compat
[ ] Benchmark regressions
[ ] (For Kadena engineers) Run integration-tests against a Chainweb built with this version of Pact

Where are the performance numbers to show that this is beneficial in terms of either memory or CPU?

Generally I'm wary of tossing {-# StrictData #-} in places as a performance improvement because it's mostly a parser-phase ! insertion on records are sums. I would like any benchmarking to include some analysis of predictability of the priors instead of just the blunt instrument of "goes faster/looks nicer". We need to know our cost centers!

Sure, here are the benchmark outputs:

for the current master: gist
for the proposed changes: gist

Here are also the relative differences (with the denominator being the average of the old and new number), 118 out of 122 benchmarks in favor of the proposed changes: gist (and the not-in-favor being less -4.6%, -2.3%, -0.9% and -2.4% differences). So it's mostly two-digit relative performance improvements, up to 80% on some tests.
It's also worth noting that these benchmarks might be hard to interpret unambiguously. nfAppIO forces each result of all the runEvalMs to normal form, but that result contains the interpreter state, which, in particular, includes the esLoaded field. This state is not really the output of interpretation, but IMO it's worth including the time to evaluate it in the benchmarks (as it indeed currently happens), otherwise it's harder to reason about performance of the compositions of the functions (since esLoaded is used during evaluation, and some parts of esLoaded might be lazily delayed to be evaluated later when a subsequent function uses them, so not counting them might be sort of cheating).
What's ambiguous about all that? Well, esLoaded includes non-trivial (Haskell) terms, so even with the strictness annotations it takes a non-trivial amount of time just to ensure that it's in normal form. For example, replacing
C.bench "(+ 1 2)" $ C.nfAppIO (runEvalM ee' es' . Eval.evalNormalForm env') term'

with
C.bench "(+ 1 2)" $ C.nfAppIO (fmap (second force) . runEvalM ee' es' . Eval.evalNormalForm env') term'

(which gives the same result, but with an extra nf'ing pass over the interpreter state) increases the run time from 290-300 ns to about 440-460 ns (all with strictness annotations), which might be a good ballpark estimate of the overhead of just traversing the interpreter state to ensure it's in NF (although, for this particular test!). So, for this particular case it's arguably more correct to interpret all these numbers as traversing the interpreter state taking about 150 ns, and thus pure interpretation time for the master version is 380 - 150 = 230 ns and for the "strict" version it's 290 - 150 = 140 ns, giving about 48% perf improvement (instead of 26%, if criterion report numbers are taken as is).
State traversal overhead is different for different tests for a bunch of reasons, so the best approach to account for that would be to have a version of each benchmark with and without an additional force, but I'm not sure it's worth investing time right now.
@emilypi

Generally I'm wary of tossing {-# StrictData #-} in places as a performance improvement because it's mostly a parser-phase ! insertion on records are sums.

That's in fact exactly what I want here, since I caught myself just inserting lots of !s in record fields that are transitively involved in Loaded/EvalState.

We need to know our cost centers!

This particular case appears to have optimizations being quite sensitive to profiling builds. The difference between proposed changes and master are much less pronounced (and sometimes around 0%) when compiled with profiling (and all the usual auto-SCC jazz), even if profiling is disabled at runtime (that is, no -p and friends). I have recorded cost centers for some of our tests, but they aren't very telling (the test function itself typically being the biggest cost center), and the aforementioned change of behavior makes me consider them less reliable.


No Licence file

HI @Horkyze!
I'd like to give this a go in a school project I'm working on, however there is no licence.md or information on reuse.
May I use it?

Hi, yes sure. Do whatever you want with the source. :wink:


iOS excel doesn’t hold data in clipboard

When you copy or cut an item in the iOS or osx world, the data you copied or cut is supposed to stay in the clipboard after (if) you paste it until you copy or cut new data.
This is so you can post the same data multiple times (until you copy or cut new data).
This does not happen in iOS excel. After copying and pasting info, the data is either cleared from the clipboard or something triggers the the paste option in iOS excel to become unavailable until you copy new or the same data again.
Bad UI.

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


Multiple Sell orders for the same asset can be placed.

Steps to reproduce:
Go to Assets page
Click 3 Dots on an asset and select sell
Enter auction details
Confirm transaction
View "Find" Page
Confirm same asset is there multiple times
(Note that I did this with an Ethercraft item, but I only possessed 1 of that item from pre-sale)

This is true; it's not unintentional - once one order is matched, the other will be automatically removed (and can no longer be matched even if the user had a cached page).
I realize it may be confusing. The orderbook server could block multiple sell-side submissions of the same asset.

If that does function as described, then the main concern shifts to a potential "attack" on the exchange, where an asset is deposited, and then the owner spams the order book with the same asset over and over. Since this only requires a signature, and not any gas cost, the order book would be overwhelmed.

The orderbook server rate limits requests (order submission requests moreso), and there is no incentive for a party to submit multiple identical orders. Further limiting measures could be imposed if necessary.
At the moment, however, I agree that the use cases requiring multiple orders for the same asset are unlikely, and this behavior is confusing.
Moved to https://github.com/ProjectWyvern/example-orderbook-server/issues/2 (this must be implemented server-side).


Two way communication in run mode.

I needed to send some commands to the embedded target over serial. I found the path of least resistance was to extend bobbin-cli since I'm already using it for normal development. Feel free to use it, not use it, copy paste the code, request changes or commit directly to my repo (I've "allowed edits from maintainers") as you wish (or any other thing you might want to do with this PR).
I've only tested this with the JLink firmware but I would assume it works the same for other debuggers as well.

@jcsoo
Have you had a chance to look at this?
Is there anything more i can do to make this feature happen?

Thanks for pinging me on this, I meant to take a look at this a while back and then got caught up in a lot of other stuff.
Overall it looks great. I just need build it and test it out myself.


Feature Request: Configurable validation

Could you add a validation that, like the validation of string format parameters, checks the usage of characters like pipes or other configurable character(s)?

How could we check the usage of pipes?

@Llarian any feedback what exactly you mean with this?

Currently there is a validation regarding the usage of string format parameters: you get a warning if the number of parameters used differs from one language to another. I'd also liked a warning for the usage of pipes: When used in a FileopenDialog in the filter string to seperate the values, .NET will throw an exeption, in case the pipe is missing. Our translators do not all know the importance of pipes and we (developers) do not check the translations of languages we don't know.
For this it might be useful to set the "source" language, which is the developers own language; this will most probably be correct. The "tartet" languages should be checked against the source language.
This is a very specific example, but I gues there might be other scenarios. So something configurable would be the best solution.

Unlike common string format parameters, this is a very special use case.
I would rather recommend not including the pipes in the resources at all, but only the translatable part(s).


site 404 on quick start

https://kpt.dev/book/01-getting-started/01-quickstart
This is due to navbar being stale.

Related fix: https://github.com/GoogleContainerTools/kpt/pull/1637/


A key can only consist of ASCII letter, digit or '_' (underscore) characters.

Hello everyone,
Ad func.py
How come you have got for example this:
schema = cfg["db-schema"]?
Given the fact that hyphens are not allowed.
Fn: Error updating function configuration: Service error:InvalidParameter. Invalid config. http status code: 400.
Thanks and Regards,
Vaclav

Hi Vaclav,
Sorry about the delayed reply.
Yes, it looks like a bug (to be confirmed). Earlier we used to allow hyphens "-" in function config variable names. Can you replace the hyphens in function config variable names with underscores please "_" ?
Regards
Sachin Pikle.

Hi Vaclav,
This issue is fixed. Can you try again and confirm?
Regards
Sachin Pikle.

Hi Sachin,
It seems to be ok from the Cloud Shell perspective. However, there is still the same warning in the GUI form.
Thanks and Regards,
Vaclav

Hi Vaclav,
The GUI fix is awaiting deployment. Will let you know once that's in production.
Regards
Sachin Pikle.

Hi Vaclav,
The console/GUI fix has been deployed in production. I am closing the issue. If you still see this issue, please add a comment and reopen this ticket. Thank you!
Regards
Sachin Pikle.

Hi Sachin,
Thumbs up!
Thanks a lot.
Regards,
Vaclav


Some interpreter tests don't work with non-V1CompatibilityRules, but work on Canvas

We have a set of tests that don't work when run through the interpreter without V1CompatibilityRules, a part of PowerFxV1.  One would think that is a problem, but alas, Canvas doesn't use the interpreter and the tests pass on Canvas.  Rather than fixing the bug in the short term, we need to skip these tests here and run them on Canvas in another way.
For example, DateTime(1900,1,1,12,0,0) <= Time(12,0,0) and Time(1,0,0) <= DateTime(1899,12,30,1,0,0) with all comparison operators including =.

Fixed by #1694


Bug reported by community user

Please see the bug reported by the community user Hayes3d at https://powerusers.microsoft.com/t5/Power-Apps-Governance-and/COE-Starter-Kit-Developer-Compliance-Center-Description-Field/td-p/433405

Fixed with new release


Improve handling of debugger aborts

This checkin improves the error message that we show when GDB exits or we otherwise loose the connection to the target process.
This also fixes a hang that could happen if we lost our connection after DebuggedProcess.Initialize, but before DebuggedProcess.ResumeFromLaunch. The issue was that nothing would send the program destroy event.

@chuckries please review

👍

👍

👍


Add Swift-YouTube-Player

Swift library for embedding and controlling YouTube videos in your iOS applications!
Project URL
https://github.com/gilesvangruisen/Swift-YouTube-Player
Category
Video
Description
Swift library for embedding and controlling YouTube videos in your iOS applications!.
Why it should be included to awesome-ios (optional)
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[ ] Supports Swift 4 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Warning

:warning:
Found 2 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

5
Dupe
https://gitter.im/vsouza/awesome-ios

2698
503
https://boombox.io/

  Generated by :no_entry_sign: Danger


Feature Request: LiveHunts

We use VirusTotal LiveHunts heavily for users to have their yara rules scanned against incoming binaries.  We'd like this feature on AssemblyLine.  Here are some thoughts regarding a LiveHunt feature.

The rules would be run against all incoming submissions
The rules would be user submitted / managed (unlike the Admin defined Yara Source Rules)
The user would be alerted when the rule hits on a submission
Expiration date w/ notification - Submissions should have a timeframe option (set by admin) that expires the rule unless renewed
The submitting user gets a note that their submission hit on a LiveHunt rule submitted by {username}
The rule could be accompanied with a note - such as what action the person may want, contact info, etc.
Have the option to run the rule as a retro hunt.
Ability to submit binaries to AL and not run any of the standard scans - just a submission for the purpose of having it processed and available for LiveHunt / RetroHunt.

We're almost done with our first version of retro-hunting, this will not come on day 1 but is a feature that will be easy enough to build in.


Add Adherent relation to App request and new command to sync all cand…

…idates

Bella PR :ok_hand:

[ch8668]


Fix Typo

for verification, see here: https://dict.leo.org/german-english/welches

Hello @nilsmartel 👋
Can you please sign the CLA? I will approve your PR then.

I signed the contribution document now, how can. I rerun the CLA test?

Thanks. The test runs automatically :)
/lgtm
/approved

/approved

/approve


Add Spark Benchmarks for Perf Testing

It would be good to measure performance on various Spark workloads. We can run benchmarks such as Hibench and Databricks.

HiBench is a big data benchmark suite that helps evaluate different big data frameworks in terms of speed, throughput and system resource utilizations. It contains a set of Hadoop, Spark and streaming workloads, including Sort, WordCount, TeraSort, Sleep, SQL, PageRank, Nutch indexing, Bayes, Kmeans, NWeight and enhanced DFSIO, etc. It also contains several streaming workloads for Spark Streaming, Flink, Storm and Gearpump.

Packages to Use
We'll need the following packages in order to run perf tests.
Hadoop: https://hadoop.apache.org/
Spark: https://spark.apache.org/
HiBench: https://github.com/Intel-bigdata/HiBench
Supported Hadoop & Spark releases for HiBench

Hadoop: Apache Hadoop 2.x
Spark: Spark 1.6.x, Spark 2.0.x, Spark 2.1.x, Spark 2.2.x

I have experience in using the following: hadoop-2.7.3, spark-dk-<IP_ADDRESS> and hibench-6.0 and tests work without any issues. Though it would be preferred to use the latest compatible binaries.
TODO
I wrote a script few years back that we still use for our internal perf analysis. We'll need to create the build and playlist files in order to get all the dependencies needed to runs the tests.

Close this as tracking addition of additional perf benchmarks via #1112.


fix(icon): workaround for webkit double render (ssr)

What I did

forcibly remove double-rendered icons in ssr scenarios (webkit, mostly)

Testing Instructions

check navigation-secondary analytics demos, possibly others (compare to main)

Notes to Reviewers
Related to #2043

It's worth noting that this CTA double icon rendering bug does not exist on uxdot / main—only on staging/cubone. Benny speculated that this might be due to the typescript changes that have been merged to main but, as of this writing, have not made their way back to staging/cubone.
Before merging this PR, update staging/cubone with main and re-test.

This is the behavior this PR is looking to correct (or that may already be corrected on main). See the CTA in the Secondary Nav demo via the rh-disclosure DP on staging/cubone.


stop the listener in the server unit tests

Codecov Report

Merging #1527 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #1527      +/-   ##
=========================================
+ Coverage   89.18%   89.2%   +0.02%
=========================================
  Files         123     123
  Lines        8642    8642
=========================================
+ Hits         7707    7709       +2
+ Misses        633     631       -2
  Partials      302     302

Impacted Files
Coverage Δ

server.go
86.51% <0%> (+0.93%)
:arrow_up:

Continue to review full report at Codecov.

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


Parameter moving under cursor cancels field editing

Describe the bug

(GIF courtesy of @PeterMathijssen)
Affects

[ ] Designer
[x] Blocks editor
[ ] Companion
[ ] Compiled apps
[ ] Buildserver
[ ] Debugging
[ ] Other... (please describe)

Expected behavior

The flydown should not open when the edit field is focused.
Steps to reproduce

See GIF.

Hi @ewpatton, I would like to work on this. Could you please let me know which file this issue lies in?

Hi @ewpatton, I would like to work on this. Could you please let me know which file this issue lies in?

@Siddharth1010 I haven't looked into it yet, but I expect that what needs to be done is in to add another check in field_flydown.js in showFlydownMaker_ that tests whether Blockly.WidgetDiv.DIV is non-null.

@Siddharth1010 I haven't looked into it yet, but I expect that what needs to be done is in to add another check in field_flydown.js in showFlydownMaker_ that tests whether Blockly.WidgetDiv.DIV is non-null.

@ewpatton There does not seem to be a Blockly.WidgetDiv.DIV component in field_flydown.js  . I had added Blockly.WidgetDiv.Div != null in showFlydownMaker_ . But on executing ant tests the Blockly tests seem to be failing.

Also, I am unable to reproduce this exact issue using a procedure block. Could you please guide me?

@ewpatton There does not seem to be a Blockly.WidgetDiv.DIV component in field_flydown.js  . I had added Blockly.WidgetDiv.Div != null in showFlydownMaker_ . But on executing ant tests the Blockly tests seem to be failing.

Also, I am unable to reproduce this exact issue using a procedure block. Could you please guide me?

You need to add an input variable to the procedure. Click on that blue badge in the upper left corner of the block.

You need to add an input variable to the procedure. Click on that blue badge in the upper left corner of the block.

@ewpatton @patryk84a , Yes I have added an input variable and I am able to recreate the issue. But isn't this issue related to Blockly.FieldTextInput as opposed to Blockly.WidgetDiv.DIV?

@ewpatton @patryk84a , Yes I have added an input variable and I am able to recreate the issue. But isn't this issue related to Blockly.FieldTextInput as opposed to Blockly.WidgetDiv.DIV?

@ewpatton @patryk84a , Yes I have added an input variable and I am able to recreate the issue. But isn't this issue related to Blockly.FieldTextInput as opposed to Blockly.WidgetDiv.DIV?

Hi @Siddharth1010 =) The widget div is non-null whenever a text input field is being edited (the place where your text goes when you're typing is the widget div). So If you only want the flydown to open when a text input field is /not/ being edited, you should be able to use the widget div to check for that state. I hope that helps!

@ewpatton @patryk84a , Yes I have added an input variable and I am able to recreate the issue. But isn't this issue related to Blockly.FieldTextInput as opposed to Blockly.WidgetDiv.DIV?

Hi @Siddharth1010 =) The widget div is non-null whenever a text input field is being edited (the place where your text goes when you're typing is the widget div). So If you only want the flydown to open when a text input field is /not/ being edited, you should be able to use the widget div to check for that state. I hope that helps!

Hi @BeksOmega , thank you so much :) . So logically, if we check Blockly.WidgetDiv.DIV===null (not being edited) in the showFlydownMaker_ function, it should work. And it does, sort of. The problem is, now the flydown does not show up at all. Even if I am not editing the text field.
Fix:

Problem:

Please let me know if I am doing something wrong

Hi @BeksOmega , thank you so much :) . So logically, if we check Blockly.WidgetDiv.DIV===null (not being edited) in the showFlydownMaker_ function, it should work. And it does, sort of. The problem is, now the flydown does not show up at all. Even if I am not editing the text field.
Fix:

Problem:

Please let me know if I am doing something wrong

Hi @BeksOmega , thank you so much :) . So logically, if we check Blockly.WidgetDiv.DIV===null (not being edited) in the showFlydownMaker_ function, it should work. And it does, sort of. The problem is, now the flydown does not show up at all. Even if I am not editing the text field.

Hmmm you're right it looks like DIV never gets set back to null after it's created... maybe try the Blockly.FieldTextInput.htmlInput_ instead? It looks like it gets set and unset properly hehe
Thank you for looking into this!

Hi @BeksOmega , thank you so much :) . So logically, if we check Blockly.WidgetDiv.DIV===null (not being edited) in the showFlydownMaker_ function, it should work. And it does, sort of. The problem is, now the flydown does not show up at all. Even if I am not editing the text field.

Hmmm you're right it looks like DIV never gets set back to null after it's created... maybe try the Blockly.FieldTextInput.htmlInput_ instead? It looks like it gets set and unset properly hehe
Thank you for looking into this!

@BeksOmega Yes  Blockly.FieldTextInput.htmlInput_ seems to be working fine :)

Shall I create a pull request?

@BeksOmega Yes  Blockly.FieldTextInput.htmlInput_ seems to be working fine :)

Shall I create a pull request?

Shall I create a pull request?

@Siddharth1010 I'm not an official maintainer hehe, but I'd say it looks like you're good to go!

Shall I create a pull request?

@Siddharth1010 I'm not an official maintainer hehe, but I'd say it looks like you're good to go!

@BeksOmega Oh okay. Thank you for your help  :)

@BeksOmega Oh okay. Thank you for your help  :)

Will the fix work for other blocks as well?

Will the fix work for other blocks as well?

Will the fix work for other blocks as well? The same situation is also with the block of the procedure returning result. The same applies to local variables when we add more than one variable to the same block.

Yeah it should fix all of the relevant blocks because the fix is in the root cause (the field). So you should be good! Be sure to write back if you find anything else that isn't working though :D


Propose a boolean field in ClientStatusRequest to indicate whether xds server should exclude detailed config(xds_config filed in GenericXdsConfig) but keep the rest of fields in response.

Commit Message: Propose a boolean field in ClientStatusRequest to indicate whether xds server should exclude detailed config(xds_config filed in GenericXdsConfig) but keep the rest of fields in response.
Additional Description: Made some changes according to comments in https://github.com/envoyproxy/envoy/pull/25354
Risk Level: Low
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/assign @markdroth

/lgtm api

/assign-from @envoyproxy/maintainers

/assign @htuch

/retest


Fix -fPIC/-fPIE travis

Fixes the bug where travis complained about needing to compile with -fPIC. The reason for this was because it was compiling with both -fPIE -fPIC, which seems to default to just -fPIE. You cannot link Position Independent Executable code into a shared library. It has to be Position Independent Code (source).
This extra -fPIE flag was added by ${Qt5Widgets_EXECUTABLE_COMPILE_FLAGS}. As we are making a library, not an executable, we don't want this. The fix is to remove it altogether, as it's added automatically when the user links with the Qt libraries (whether directly or transitively through us)
Additionally, in .travis.yml, I clarified some comments, and added a command to make it easier to get a verbose build (make VERBOSE=1). Now, if the build fails, you can go to https://travis-ci.org/paceholder/nodeeditor/settings and add a VERBOSE_BUILD environment variable. When VERBOSE_BUILD is set to ON, it will act as if you had make VERBOSE=1. When VERBOSE_BUILD is unset or set to OFF, it will act normally (non-verbosely). You can then trigger a rebuild (no commit needed) to get the verbose information (or you can just leave VERBOSE_BUILD=ON and always have verbose information)

Thanks a lot. Interesting, previously we added the flags from ${Qt5Widgets_EXECUTABLE_COMPILE_FLAGS} as well.  And it is somehow gets compiled and linked on my laptop (Linux, Qt 5.10, gcc 7.3)

@paceholder It compiles and links just fine on my Ubuntu 16.04 system as well. On my system, ${Qt5Widgets_EXECUTABLE_COMPILE_FLAGS} expands to -fPIC rather than -fPIE. Likely has to do with the version of Qt, perhaps it was a bug in Qt. Either way, that was the problem.


TTS-Engine from trained model

How to convert a trained model into a tts - engine to be used with windows?

Your question is a bit vague to me. what is the specification of your trained model? Did you find a well-trained TTS-engine? to apply it in windows os?

Your question is a bit vague to me.

That's fine, don't worry. There is a lot of resources on the internet that help you learn understand it.


Binary operations in BlockDataContainer

What do we return here?

I believe this line says that the result of the operation that is executed is saved already in each out, so after the execution there's no need to do anything else and the method returns, i.e. stops.


Failed to reload settings terminal

Windows Terminal version (or Windows build number)
1.6.10571.0
Other Software
No response
Steps to reproduce
I deleted the old settings.json file
Expected Behavior
I have such a problem. Namely, when he wants to make such modifications https://www.bleepingcomputer.com/news/microsoft/windows-terminal-now-has-a-hidden-graphical-settings-editor/ in the windows terminal, he receives this error 'Failed to reload settings'.And now my question. Where does such an error come from if I do everything in accordance with what is given in the guide? How to fix it?
Actual Behavior
This error continues 'Failed to reload settings'
This is the contents of the old settings.json file
// This file was initially generated by Windows Terminal 1.0.1401.0
// It should still be usable in newer versions, but newer versions might have additional
// settings, help text, or changes that you will not see unless you clear this file
// and let us generate a new one for you.
// To view the default settings, hold "alt" while clicking on the "Settings" button.
// For documentation on these settings, see: https://aka.ms/terminal-documentation
{
    "$schema": "https://aka.ms/terminal-profiles-schema",

    // "defaultProfile": "{61c54bbd-c2c6-5271-96e7-009a87ff44bf}",

    // You can add more global application settings here.
    // To learn more about global settings, visit https://aka.ms/terminal-global-settings

    // If enabled, selections are automatically copied to your clipboard.
    "copyOnSelect": false,

    // If enabled, formatted data is also copied to your clipboard
    "copyFormatting": false,

    // A profile specifies a command to execute paired with information about how it should look and feel.
    // Each one of them will appear in the 'New Tab' dropdown,
    //   and can be invoked from the commandline with `wt.exe -p xxx`
    // To learn more about profiles, visit https://aka.ms/terminal-profile-settings
    "defaultProfile": "{00000000-0000-0000-ba88-000000000002}",
    "profiles": {
        "defaults": {
            // Put settings here that you want to apply to all profiles.
        },
        "list": [{
                "guid": "{00000000-0000-0000-ba88-000000000002}",
                "commandline": "C:\\Program Files\\Git\\bin\\bash.exe",
                "name": "Git Bash",
                "background": "#012456",
                "hidden": false
            },
            {
                // Make changes here to the powershell.exe profile.
                "guid": "{61c54bbd-c2c6-5271-96e7-009a87ff44bf}",
                "name": "Windows PowerShell",
                "commandline": "powershell.exe",
                "hidden": false
            },
            {
                // Make changes here to the cmd.exe profile.
                "guid": "{0caa0dad-35be-5f56-a8ff-afceeeaa6101}",
                "name": "Command Prompt",
                "commandline": "cmd.exe",
                "hidden": false
            },
            {
                "guid": "{2c4de342-38b7-51cf-b940-2309a097f518}",
                "hidden": false,
                "name": "Ubuntu",
                "source": "Windows.Terminal.Wsl"
            },
            {
                "guid": "{b453ae62-4e3d-5e58-b989-0a998ec441b8}",
                "hidden": false,
                "name": "Azure Cloud Shell",
                "source": "Windows.Terminal.Azure"
            }
        ]
    },

    // Add custom color schemes to this array.
    // To learn more about color schemes, visit https://aka.ms/terminal-color-schemes
    "schemes": [],

    // Add custom keybindings to this array.
    // To unbind a key combination from your defaults.json, set the command to "unbound".
    // To learn more about keybindings, visit https://aka.ms/terminal-keybindings
    "keybindings": [

        // Copy and paste are bound to Ctrl+Shift+C and Ctrl+Shift+V in your defaults.json.
        // These two lines additionally bind them to Ctrl+C and Ctrl+V.
        // To learn more about selection, visit https://aka.ms/terminal-selection
        {
            "command": {
                "action": "copy",
                "singleLine": false
            },
            "keys": "ctrl+c"
        },
        {
            "command": "paste",
            "keys": "ctrl+v"
        },

        // Press Ctrl+Shift+F to open the search box
        {
            "command": "find",
            "keys": "ctrl+shift+f"
        },

        // Press Alt+Shift+D to open a new pane.
        // - "split": "auto" makes this pane open in the direction that provides the most surface area.
        // - "splitMode": "duplicate" makes the new pane use the focused pane's profile.
        // To learn more about panes, visit https://aka.ms/terminal-panes
        {
            "command": {
                "action": "splitPane",
                "split": "auto",
                "splitMode": "duplicate"
            },
            "keys": "alt+shift+d"
        }
    ]
}

I think this is probably one of those BOM related issues.

More specifically?

See #9310
btw you may want to attach a screenshot shot of the special error you run into. That would be much more helpful sometimes.

FYI, you can just keep replying on your original thread. You don't need to make a new one just because the bot closed it.
/dup #9309


SITE-BUSTING ERROR: "render_template() called before await ds.invoke_startup()"

My https://list.saferdisinfectants.org/disinfectants/listN page (linked from https://SaferDisinfectants.org ) has been running beautifully for a year and a half, including a scrape workflow that's been routinely updating the database.

I received a recent report that the list page is down. I don't know when it went down, but the content is replaced with: "render_template() called before await ds.invoke_startup()"

The local datasette repo runs without incident.

The site is hosted on vercel, linked to my github repo. Perhaps  some vercel changes were made, but not by anyone on our side. Here is a screenshot of the current project settings:

Here a screenshot of the latest deployment status:

This is my repository:
https://github.com/mroswell/list-N
(I notice: datasette==0.59 in my requirements.txt file)
Because it's been long while since I actively worked on this or any other datasette project, I forget a lot of what I knew at one point. Perhaps some configuration file could be missing? Or perhaps I just need to know the right incantation to add to that vercel settings page.
Help is welcome as the nonprofit org is soon hosting its annual conference, and we'd love to have the page working again.

should this issue be under https://github.com/simonw/datasette-publish-vercel/issues ?
Perhaps I just need to update:
datasette-publish-vercel==0.11
in requirements.txt?
I'll try that and see what happens...

I updated datasette-publish-vercel to  0.14.2 in requirements.txt
And the site is back up!
Is there a way that we can get some sort of notice when something like this will have critical impact on website function?

Good spot fixing that!
Sorry about this - it was a change in Datasette 0.63 which should have been better called out.
My goal for Datasette 1.0 (which I aim to have out by the end of the year) is to introduce a formal process for avoiding problems like this, with very clear documentation when something like this might happen.


import valeriano manassero chart version improvements

this is a first proposal to import improvements we have in https://github.com/valeriano-manassero/helm-charts/tree/main/valeriano-manassero/trino before depreciating it in favour of the official one.
I know this PR is pretty big but I tested before submitting and seems to work on my testing environment, any thought is appreciated.

@valeriano-manassero Should we retry :-) ? I would like to help.

I updated my fork with my chart again. I understand the PR is quite big since implementations are quite different; I checked file by file and importing features one by one is time consuming for me so I'm not sure this PR will be good for the maintainers.

@valeriano-manassero you need to sign the CLA if you want it to be considered for inclusion.

whoops, I totally missed it, mail sent.

Since I see features importing is an ongoing process I don't see value on maintaining this PR :-)

Thanks @valeriano-manassero for keeping the changes incoming in your chart. We seem to have ported over most of the big-ticket items but there are obviously a few more quality of life changes remaining.
I'll take a look at the missing features this weekend and decide which ones to port in what order accordingly.


Integration test that dashboard OTLP endpoint can be configured with TLS

The dashboard OTLP endpoint is unsecured by default. It should support being secured with TLS to give deployers the option of securing the connection.

Done with https://github.com/dotnet/aspire/pull/2223


Music does not work on portait on phone

Hi so I can't click on the play button for the music
On my phone in portrait landscape works though the res of my Samsung galaxy a13 is 720x1600  here is the link for the image https://files.catbox.moe/yrcl8e.jpg
I think the issue is that the horizontal resolution is so small that the ui your using just does not work
It should be pretty easy to reproduce this issue
Thanks and have a good day? Well right now for me it's 1:53 anyways
And I was using chrome

Unfortunately, this probably can't be fixed because the UI for audios is handled by the browser itself and not by WebTV HD. I could try adding a min-height attribute to the audio, but it might not solve the problem


Adjust Image Resizer string to remove translation clarity issue

Microsoft PowerToys version
0.56.1
Utility with translation issue
Image Resizer
🌐 Language affected
German
❌ Actual phrase(s)

"Passt in [resolution] pixel"
✔ Expected phrase(s)
Only the resolution.
Format: "[width] x [height] [unit]"
e. g.: 1920 x 1080 Pixel (no the capital letter at the beginning of the word "Pixel" is not a spelling mistake)
CRUTKAS possible solution: Mode - Width x Height unit
ℹ Why is the current translation wrong
Sounds wrong in my ears. It sounds better when you simply put the new resolution of the new picture there

You have different modes for resizing, that is why they are there.  Fit, fill, stretch

/needinfo

Still the current translation sounds weird although I also understand your point

What would be a better way then?  Simple as shifting the "in" to be a dash?
Mode - Width x Height unit
/needinfo

I think the words "Fits inside" would be even better.

@jay-o-way I would suggest that being a different scope.  I think the pattern solves the problem


Champion swaps are not handled

Trades do not seem to be handled yet; that's a major issue. #1 priority.

Resolved by 44292a6.


add Vanier College

Sorry for the delay! I'm adding the domain.

No problem.
Thanks.
On 2 March 2015 at 11:57, Elena Ganenkova<EMAIL_ADDRESS>wrote:

Merged #760 https://github.com/JetBrains/swot/pull/760.
—
Reply to this email directly or view it on GitHub
https://github.com/JetBrains/swot/pull/760#event-244502314.


License

There is currently no license associated with this repository.
For those of us who would like to use it without legal uncertainty can a license be added to the repo ( LICENSE.md ) and/or as a header comment to ngx_http_aclog_bypass_module.c?
OpenSource.org has templates for popular licenses:

http://opensource.org/licenses/BSD-2-Clause
http://opensource.org/licenses/LGPL-2.1
http://opensource.org/licenses/GPL-2.0

Cheers,
Mike

added


[PanneauPocketBridge] Use dynamic list of cities

Hi,
I think I spotted an issue on the AbstractBridge
I added some comment why I was forced to copy/paste a core method or maybe I misunderstood the logic.
Feel free to add comments

I've just tried it, and it works well for me! Thanks

lint

thanks for the contribution flovi.
this pr needs improvement.

this pr increases the frontpage size from 1.7M to 2.3M (25%) because of the 13000 options in the list. this not good

the bridge does http call in constructor. not good

use cache to cache the big list of cities

i agree there is a bug in parent method regarding static::PARAMETERS. if you feel brave can you please fix it so we dont have to copy paste the method.

i dont really see a need to do work in the constructor

Hi,
For item 4, here is the PR https://github.com/RSS-Bridge/rss-bridge/pull/3858

sorry dont have mental energy to followup this one

Hi,
Maybe we can keep this PR open.
I made a PR to accept a new type for parameters: dynamic_list (@see #4361).
I updated this PR accordingly. Merge it only if #4361 is merged.
Kind regards


Update client commands in quick start

This was missed when renaming the client commands

Codecov Report

Merging #312 (43ebcd7) into master (15048c4) will decrease coverage by 0.22%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #312      +/-   ##
============================================
- Coverage     71.36%   71.13%   -0.23%
+ Complexity      913      912       -1
============================================
  Files           135      135
  Lines          2787     2796       +9
  Branches        340      340
============================================
  Hits           1989     1989
- Misses          676      684       +8
- Partials        122      123       +1

Impacted Files
Coverage Δ

...ss/model/order/OrderContainsKeywordsPredicate.java
76.92% <0.00%> (-7.70%)
:arrow_down:

src/main/java/seedu/address/model/order/Order.java
64.86% <0.00%> (-7.45%)
:arrow_down:

...ava/seedu/address/model/order/UniqueOrderList.java
68.62% <0.00%> (-0.61%)
:arrow_down:

...rc/main/java/seedu/address/model/ModelManager.java
75.34% <0.00%> (ø)

.../java/seedu/address/model/util/SampleDataUtil.java
15.38% <0.00%> (ø)

.../seedu/address/logic/commands/AddOrderCommand.java
86.66% <0.00%> (+0.95%)
:arrow_up:

Continue to review full report at Codecov.

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


PATCH: Allows adaptiveHeight option with multiple slides shown.

The current carousel implementation of adaptiveHeight only works for single-slide carousels. [fiddle]
This patch allows for multiple slides, adapting to the height of the tallest item. [fiddle]

Thanks for this solution. Going with it for now. Would love to see this merged.


Adjust Windows Scaling

Description of the new feature / enhancement
It would be amazing to have a shortcut key to adjust Windows Scaling on the fly.  With so many UHD & 4K laptops available it would be really beneficial to be able to easily change the Windows Scaling settings.  The ability to just toggle between two or three settings (100%, 150% & 200%) would be ideal.
Scenario when this would be used?
Depending on what a laptop or Microsoft Surface is being used for and how it is being used differently.  For example, I tend to change my scaling depending if I'm using mine as a tablet vs. as a laptop because of the difference in how far the screen is from my face.
Supporting information
Examples of other people trying to solve this issue
https://www.tenforums.com/graphic-cards/171539-hotkey-scaling-windows-10-a.html

/dup #1052


create app icons for example apps

Created the icon.
logo-RNMP copy.pdf

We need to actually put the icon into the app. Will open a new issue


Cmake always builds generator binary

I had quite a struggle to develop in CLion with the current CMake with both generator and testsuite binaries. So I changed CMake so it always builds the generator binary. It is possible to build both testsuite and generator binary in one run. This simplifies a development for me as I can have both enabled and debug.
Thanks for considering a merge.

Thank you for this, when I was adding testsuite I wanted to do it this way, but I was getting some errors so I did it little bit ugly.
I think we should modify also travis to run only 2 jobs instead of 4. I will do it and add it to PR.

read for merge


Added Neural Tanh Layer and corresponding unit test

The Tanh squasher is a very popular alternative to the sigmoid layer. It's widely used, although not as much as before thanks to relu's. It's definitely important for a complete neural nets package. :)

this is a vanilla implementation. please consider using our linalg library and openmp at the least.

I was wondering why you guys didn't use the linalg library for the logistic layer so I just stuck with the same implementation. I'll modify this one to use it now. Thanks.

Nice patch.
I agree with @vigsterkr please use linalg, eigen, and openmp to make this stuff fast

news?

@karlnapf if you are okay with it, I think you can merge it now. When refactoring the nn package with linalg and openmp, I think it'd be better if all the modules are refactored at once. What do you think?

Few things that block this being merged.

travis is failing, your code does not compile
Shogun's NN should be refactored 1by1, not all at once (small PRs!)
Put all the open mp stuff already in. Once something is merged, it usually is unlikely that it is touched again, so we want to make it as good as we can in the first merge


Unknown column '$1' in 'where clause'

My code is in postgres:
let r = sqlx::query!("select * from menu where id = $1", id).fetch_one(&self.pool).await?;

I don't know why.
THe following code is right.  can i not use macro in postgres？
match sqlx::query(r#"insert into menu (id, created_by, created_date, flag, modified_by, modified_date, parent_id, authority_id
, controller_class, icon_path , mark, menu_classify, menu_desc, menu_index, menu_name, menu_url,remark, small_icon_path)
values ($1, $2, $3, $4, $5, $6, $7, $8, $9, $10, $11, $12, $13, $14, $15, $16, $17, $18 )"#)
.bind(&id)
.bind(&menu.created_by)
.bind(Utc::now().naive_utc())
.bind(menu.flag).bind(&menu.modified_by).bind(menu.modified_date)
.bind(&menu.parent_id).bind(&menu.authority_id).bind(&menu.controller_class)
.bind(&menu.icon_path).bind(&menu.mark).bind(&menu.menu_classify)
.bind(&menu.menu_desc).bind(&menu.menu_index).bind(&menu.menu_name)
.bind(&menu.menu_url).bind(&menu.remark).bind(&menu.small_icon_path)
.execute(&self.pool).await{
Ok(count),
Err(e) => Err(anyhow!("{:?}", e)),
}

@Jeeesie what version are you using? The published 0.3.5 or the code that's on master right now?

@Jeeesie what version are you using? The published 0.3.5 or the code that's on master right now?

sqlx-core = { version = "0.3", default-features = false, features = [ "runtime-tokio" ] }
sqlx = { version = "0.3", default-features = false, features = [ "runtime-tokio", "macros", "postgres", "mysql", "sqlite", "uuid", "chrono", "bigdecimal", "json", "ipnetwork", "tls" ] }

@Jeeesie what version are you using? The published 0.3.5 or the code that's on master right now?

note that I also have a mysql impl. after i finished mysql impl, I want to impl postgres,  but the macros always pop out Error.
for example:

You can't use $1, $2, $3, etc. as bind parameters in MySQL; that syntax is specific to Postgres. You have to use ? which doesn't work with Postgres, which means you have to use different query strings for Postgres and MySQL.
The macros can also only use one kind of database (Postgres or MySQL) at a time; whatever kind of database you give in DATABASE_URL is the one they emit code for and it is not interchangeable with other databases currently.

I am using $1for postgres. but it doesn't work.  The Error accurs as above.
I change the program feature to "postgres" for the postgres impl.

It looks like you have DATABASE_URL set pointing to a MySQL database.

It looks like you have DATABASE_URL set pointing to a MySQL database.

It looks like you have DATABASE_URL set pointing to a MySQL database.

the url is postgres.
when I use bind, it seems right.  when use macro, the error appears.

This error:

assuming it follows a sqlx::query!() invocation, would only occur if you have DATABASE_URL in either your .env file or your shell environment set to a URL that starts with mariadb:// or mysql://.

assuming it follows a sqlx::query!() invocation, should* only occur if you have DATABASE_URL in either your .env file or your shell environment set to a URL that starts with mariadb:// or mysql://.

Yes! Thanks! the problem is solved.   the .env file like this.

This error:

assuming it follows a sqlx::query!() invocation, should* only occur if you have DATABASE_URL in either your .env file or your shell environment set to a URL that starts with mariadb:// or mysql://.

  It seems solved.

Closing this then.

Closing this then.

Closing this then.

One more question:
the .env file :

the code is:
let url = env::var("POSTGRES_URL").expect("POSTGRES_URL must be set");
It only read the DATABASE_URL??


Fix service worker path for reverse proxy

Reverse proxy: service worker with base path fails due to incorrect scope
@code-asher may have more details.
Introduced in #4414

To test:

Spin up NGINX/Caddy/etc
Serve code-server at a base path like /code
Observe the service worker error

I cannot reproduce this.

It appears that the service worker uses a relative path for the scope so maybe this is no longer an issue?

Ahhh this is probably because I was browsing /code and not /code/ (trailing slash).
I thought we supported this (no trailing slash on a base path) but I checked 3.10.2 and we do not.  In hindsight that makes sense since we do not know what the base path is so we require the trailing slash (on the base path at least, we can still support no trailing slash on our own routes if we want).


Last match of tournament isn't recording

Describe the bug
The result of the last match of the tournament isn't being recorded. This is a known issue as mentioned on the README / code TODO, but I just wanted to make it an official issue for discussion to fix it.
Log
I added print statements that output when game_state != previous_game_state or game_mode != previous_game_mode
Currently in Tournament with 1 matches remaining.  Game state is open.
Player 1 chance to win: 54.34%
Current Balance is: $25,421
Bet placed: $11 on player1
Current Tier is: 2
game_state != previous_game_state //  game_state_lies = False  // state locked // prev state open
Currently in Tournament with 1 matches remaining.  Game state is locked.
True Streaks are: (3, 5)
game_mode != previous_game_mode // game_state_lies = True // current mode: Exhibition // prev mode: Tournament
game_state != previous_game_state // game_state_lies = False // state 1 // prev state locked
In Exhibition.  No bets are placed, and nothing is recorded.  25 matches left.

Notes
I think what might be causing this is that the game mode is being updated (From Tournament -> Exhibitions) before (or at the same time) the match result is detected (Game State: locked -> 1). The program is correctly detecting that player 1 won (game_state = 1), but because the game mode changed, the if statement  detects that Exhibitions mode is active and instead logic proceeds to this elif statement. As a result, the program will not record the match.
Unsure if it helps, but here is what the twitch chatbot WAIFU4u posts in the following order when the last match of the tournament ends:
WAIFU4u: Exhibitions will start shortly. Thanks for watching!
WAIFU4u: Chopper wins! Payouts to Team Red. 25 exhibition matches left!
WAIFU4u: Bets are OPEN for Koumakan sakuya vs Haizilyo! (S Tier) (Requested by Lookbeneathyou) (exhibitions)

Thank you for documenting this officially outside of the #TODO in the program itself.
Yes, you're correct in that the way that the JSON updates, it updates the game mode first, then the game state.  Because of this, we're in the "no last match of tournies recording" issue here.
Fix incoming probably later this week.
Keep 'em coming if you find them.

This should now be fixed!
Please let me know if it isn't, and I'll reopen.


useParams somehow not working with Solid-router

I am trying solid-router for routing in solid application. Equivalent Typescript code works without any issue. But F# / Fable code is not working. Generated code looks all ok to me.
Here is fable code.
type SolidRouter =
    [<ImportMember("@solidjs/router"); JSX.Component>]
    static member Router(children: JSX.Element list) : JSX.Element = jsNative

    [<ImportMember("@solidjs/router"); JSX.Component>]
    static member Routes(children: JSX.Element list, ?fallback: JSX.Element) : JSX.Element = jsNative

    [<ImportMember("@solidjs/router"); JSX.Component>]
    static member Route(path : string , ``component`` : JSX.Element) : JSX.Element = jsNative

    [<ImportMember("@solidjs/router")>]
    static member useParams () : 'T = jsNative

    [<ImportMember("@solidjs/router")>]
    static member useIsRouting () : unit -> bool = jsNative

all other than useParams works without any issue.
SolidRouter.Router [
        Html.fragment [
            Html.p "Header"
            SolidRouter.Routes [
                SolidRouter.Route("/", View1())
                SolidRouter.Route("/someitem/:id?", View2())
            ]
        ]
    ]

[<JSX.Component>]
    let View2 () =
        let isRouting = SolidRouter.useIsRouting()
        printfn $"{JSON.stringify( SolidRouter.useParams())}"
        let par : {| id : string  |} = SolidRouter.useParams()
        let id = par.id
        Html.div [
            Html.text $"Is routing {isRouting()}"
            Html.text $"Specific item with {id}"
        ]

and here is generated code.
import { createHotContext as __vite__createHotContext } from "/@vite/client";import.meta.hot = __vite__createHotContext("/dist/pages/SpecificModel.jsx");import { template as _$template } from "/node_modules/.vite/deps/solid-js_web.js?v=2764a46a";
import { insert as _$insert } from "/node_modules/.vite/deps/solid-js_web.js?v=2764a46a";
import { esm as _esm } from "/@solid-refresh";

const _tmpl$ = /*#__PURE__*/_$template(`<div></div>`, 2);

import { useParams, useIsRouting } from<EMAIL_ADDRESS>import { toConsole } from "/dist/fable_modules/fable-library.4.0.0-theta-007/String.js";
export const $$registrations = {};

const _Hot$$View1 = function View1() {
  const isRouting = useIsRouting();
  toConsole(`${JSON.stringify(useParams())}`);
  const par = useParams();
  const id = par.id;
  return (() => {
    const _el$ = _tmpl$.cloneNode(true);

    _$insert(_el$, () => `Is routing ${isRouting()}`, null);

    _$insert(_el$, `Specific item with ${id}`, null);

    return _el$;
  })();
};

$$registrations._Hot$$View = {
  component: _Hot$$View,
  id: "_Hot$$View"
};

const {
  handler: _handler,
  Component: _Component
} = _esm($$registrations._Hot$$View, !!import.meta.hot);

if (import.meta.hot) import.meta.hot.accept(_mod => {
  _handler(_mod) && import.meta.hot.invalidate();
});
export const View = _Component;

Any pointers to debug would be so helpful.

@kunjee17 Both the binding and the generated code look fine to me, what error you were getting?

@alfonsogarciacaro no error and that was the issue. I was just getting undefined that is it. Without error I can't figure it out where I was making mistake

Can you please tell me where did you get the undefined or provide logs from the browser dev tools? Do you have any commit that I can check out to reproduce the issue?

@alfonsogarciacaro no logs or no error. But it is working now. We can safely close this issue.

Perfect, thanks for confirming @kunjee17!


Update mongo tests

🗣 Description

These changes update tests that connect to a mock Mongo database.
💭 Motivation and context

Resolves #9

🧪 Testing

I modified the tests that connect to a mock Mongo database to use the new connection method. After my changes, all tests passed locally.

✅ Pre-approval checklist

[ ] This PR has an informative and human-readable title.
[ ] Changes are limited to a single goal - eschew scope creep!
[ ] All relevant type-of-change labels have been added.
[ ] I have read the CONTRIBUTING document.
[ ] These code changes follow cisagov code standards.
[ ] Tests have been added and/or modified to cover the changes in this PR.
[ ] All new and existing tests pass.

✅ Pre-merge checklist

[ ] Revert dependencies to default branches.
[ ] Finalize version.

✅ Post-merge checklist

[ ] Create a release.

Should we drop support for Python 3.6?

These changes look good to me, but I'll wait for the GH Actions checks to pass before actually approving.

Yes, please let us know when the GH Actions all pass and then we will finalize our reviews.

I will add a test to boost coverage and make the Coveralls check happy.