docs(agents/agent_types): Fix code example in chat_conversation_agent.mdx

Replace this comment with:

Description: Adds missing code to the example so it can be run. Copied from conversational agent example.
Issue: -
Dependencies: -
Tag maintainer: @baskaryan
Twitter handle: @finnless

this code snippet is imported in a file with other snippets that already define these variables, so doesn't the end result doc not need this? https://python.langchain.com/docs/modules/agents/agent_types/chat_conversation_agent

Fair enough, although it could be confusing if someone goes directly to that section https://python.langchain.com/docs/modules/agents/agent_types/chat_conversation_agent#using-a-chat-model
Also ConversationBufferMemory is repeated from the first notebook.
https://github.com/hwchase17/langchain/blob/2667ddc6867421842fe027f1946644f452de8eb3/docs/snippets/modules/agents/agent_types/chat_conversation_agent.mdx#L3-L8
I'll close for now.


Update dio package

Hi mate, thanks for this package, would it be possible to update the dio package to the latest ? I'm having dependency version solving problems. Thanks in advance.

Hi. Thanks for feedback.
I updated dio to 5.0.2.
Changes will available in v0.0.9  soon (already uploaded to pub, currently under review)


Tags

Our user guide mentioned add, delete, edit for tags, are we planning to keep them or remove them? Right now the only way to add tags is by listing all the tags currently on a meme then adding the tag at the end, which clearly isn't ideal.

I don’t think we need to add a tag independent of any meme. Just edit a meme and add a new tag to it. We don’t quite have a place to store the tags (my TagManager could be one but do we really have to add a tag out of no where?).

In this case the add tag was a command to append a tag to a meme's already existing tag list, not to create a new tag that has no meme attached to it.


Add issue to project

Some text

test markdown
message

@vislab2013
Add repo owner name

#2 cross issue referencing

[ task1, task2 ]

python <some_code.py>

Test add code to comments

close issue


[FP]: CVE-2023-5072 has a too broad CPE that flags all versions of json-java

Package URl
pkg:maven/org.json/json@20231013
CPE
cpe:2.3:a:json-java_project:json-java::::::::
CVE
CVE-2023-5072
ODC Integration
{"label"=>"Gradle Plugin"}
ODC Version
8.4.0
Description
The following vulnerability gets flagged in a build:
json-20231013.jar (pkg:maven/org.json/json@20231013, cpe:2.3:a:json-java_project:json-java:*:*:*:*:*:*:*:*) : CVE-2023-5072

However, this vulnability only applies to "JSON-Java versions up to and including 20230618". Version 20231013
is listed here as a patched version.
However,  even this patched version 20231013 gets flagged with this vulnerability, due to the CPE pattern being too broad (*:*:*:*:*:*:*:*) and not specifying specific version boundaries.
So for versions 20231013 and later of json-java, CVE-2023-5072 is apparently a False Positive.

This is fixed in DependencyCheck 8.4.1 see #5545 for details.


make helm authType configurable

What?
make helm authType configurable
Why?
currently authType is hardcoded to JWT_TOKEN

Reference Server Preview is available here:https://anchor-ref-pr658.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr658.previews.kube001.services.stellar-ops.com/


Create a GUI for using Reaper

Currently there is only the REST API and a CLI tool for using and configuring Reaper. It would be better for wider audience to provide a simple GUI that exposes the Reaper functionality.

There is currently one known web GUI created for Reaper, which you can check out:
https://github.com/spodkowinski/cassandra-reaper-ui


Allow creation of media assets with specified source.

This allows creation of media assets (image, document, video, or audio) with a specified source.
Sources and assets have a many-to-many relationship in the database.  Previously, admins could only persist relationships between assets and sources using the form on the sources#new view to select existing assets from a menu. This made workflow difficult for admins.
This PR introduces a way to specify which source an asset is associated with upon the asset's creation.  A source can be specified in a source_id parameter in the asset's #new route.  When the asset is saved, its association with the source is saved as well.
And important particularity of this approach is that upon creation, an asset can only be associated with one source (there are other ways to build additional associations after an asset is created).  However, the vast majority of assets are associated with only one source, so this approach should have a big positive impact on admin's workflow.
Changes include:

In sources#show, add links to asset#new routes that include the source's id as a parameter.

Allow the asset controllers to deal appropriately the source_id parameters.  For the #new method this means instantiating a Source object from the source_id param and making it accessible to the view.  For the #create method, this means permitting the source_ids param in a way that works with both the JavaScript and the asset models.

On the asset#new view, set the source_id as a JavaScript variable.  This is necessary because JavaScript intercepts the normal form-submit action, and is responsible for sending all relevant data to to the asset#create action.  Also on the asset#new view, display a message to the user telling them which source the asset will be associated with upon creation.

Update the JavaScript so that it sends the appropriate source_id data to the asset#create action.  The name of the function in which JavaScript posts to asset#create is postAssetRecord.

I have tested this locally in my browser, and it is working as expected.  I have also ensured that all Jasmine tests are passing.

PR#81 should be reviewed before this.  PR#81 establishes some patterns for Jasmine logic that are used here, and if we need to re-think those patterns, it would be better to make those changes first to PR#81.

There are some conditionals that are identical across the various *UploadHandler classes.  For example: https://github.com/dpla/primary-source-sets/blob/feature/8073-media-forms/app/assets/javascripts/avupload.js#L17-L25
Could these be factored out into a method of the UploadHandler base class, which is called by UploadHandler.handleResponseIfDone() to initialize a postdata object before passing it to createAssetRecord?  It would become necessary to pass key to this new function to initialize postdata instead of to createAssetRecord. That new function would have to set the file_name property of postdata with the value of key.
The call to a function that updates postdata could be inserted here: https://github.com/dpla/primary-source-sets/blob/feature/8073-media-forms/app/assets/javascripts/upload.js#L76

By the way, I don't know if this is the place to correct it, but there's another one of my lines where I have left out var:
https://github.com/dpla/primary-source-sets/blob/feature/8073-media-forms/app/assets/javascripts/upload.js#L75

Whatever the case, this works well, judging by what I see in my browser and in the database after adding an image.

@markbreedlove this is ready for re-review.  See commit 87cbf99 for the refactor of the javascript *UploadHandler classes.  I like your idea - let me know if I executed it how you imagined.  I also added in that var declaration that you suggested.

Sure, @AudreyAltman, looks good!  I'm ready to merge it, pending a squash.

:squash:ed


Correct the definition of wait2

wait2 will return nil if the Process::WNOHANG flag is specified
and no children are ready to be reaped.

Motivation

Test plan

See included automated tests.

I'm testing this on Stripe's codebase:
https://go/builds/bui_HMusdiQ1gM1mlX

Internal stripe build: http://go/builds/bui_HMutTlp0Uk1zmg

Great, thanks!


Clarification on "Percentage composition"

Researchers are asked for the percentage composition.
What is a percentage composition? Weight percentage? Mole fraction in percentages? Volumen-percentages?

We agreed on Weight percentage, we still need to clarify it to the user.
An alternative would be to allow the user to pick the type (volume, moles..)

Currently, it is not specified which type of composition in meant, hence, it is not clear what the user is asked for. Is the user ask for the purity corresponding the the structure provided? If this is true, it should be called purity and this is usually given in weight percentage.


Nested Navigators still getting focus traversal issues

Steps to reproduce
This is the ShellRoute example code from go_router with a couple extra controls to demonstrate the focus issues.
This was first reported on #112567 , I checked this earlier it was working after https://github.com/flutter/flutter/pull/134554 merge. But it is still reproducible now.
Click in TextField
Press Tab. Focus progresses to 'Button'
Press Tab. Focus progresses to 'View A Details'
Press Tab. Focus does NOT progress to bottomNavigationBar
Press Shift-Tab. Focus does NOT move to the 'Button'
Expected results
First Focus should be on AppBar's Button
Actual results
First Focus is on View A Details button
Code sample
Code sample
import 'package:flutter/material.dart';
import 'package:go_router/go_router.dart';

final GlobalKey<NavigatorState> _rootNavigatorKey = GlobalKey<NavigatorState>(debugLabel: 'root');
final GlobalKey<NavigatorState> _shellNavigatorKey = GlobalKey<NavigatorState>(debugLabel: 'shell');

void main() {
  runApp(ShellRouteExampleApp());
}

class ShellRouteExampleApp extends StatelessWidget {
  ShellRouteExampleApp({Key? key}) : super(key: key);

  final GoRouter _router = GoRouter(
    navigatorKey: _rootNavigatorKey,
    initialLocation: '/a',
    routes: <RouteBase>[
      ShellRoute(
        navigatorKey: _shellNavigatorKey,
        builder: (BuildContext context, GoRouterState state, Widget child) {
          return ScaffoldWithNavBar(child: child);
        },
        routes: <RouteBase>[
          GoRoute(
            path: '/a',
            builder: (BuildContext context, GoRouterState state) {
              return const ScreenA();
            },
            routes: <RouteBase>[
              GoRoute(
                path: 'details',
                builder: (BuildContext context, GoRouterState state) {
                  return const DetailsScreen(label: 'A');
                },
              ),
            ],
          ),
          GoRoute(
            path: '/b',
            builder: (BuildContext context, GoRouterState state) {
              return const ScreenB();
            },
            routes: <RouteBase>[
              GoRoute(
                path: 'details',
                parentNavigatorKey: _rootNavigatorKey,
                builder: (BuildContext context, GoRouterState state) {
                  return const DetailsScreen(label: 'B');
                },
              ),
            ],
          ),
        ],
      ),
    ],
  );

  @override
  Widget build(BuildContext context) {
    return MaterialApp.router(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      routeInformationParser: _router.routeInformationParser,
      routerDelegate: _router.routerDelegate,
      routeInformationProvider: _router.routeInformationProvider,
    );
  }
}

class ScaffoldWithNavBar extends StatelessWidget {
  const ScaffoldWithNavBar({
    required this.child,
    Key? key,
  }) : super(key: key);

  final Widget child;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Column(
        children: [
          Column(
            children: [
              const TextField(),
              TextButton(
                onPressed: () {},
                child: const Text('Button'),
              ),
            ],
          ),
          Expanded(child: child),
        ],
      ),
      bottomNavigationBar: BottomNavigationBar(
        items: const <BottomNavigationBarItem>[
          BottomNavigationBarItem(
            icon: Icon(Icons.home),
            label: 'A Screen',
          ),
          BottomNavigationBarItem(
            icon: Icon(Icons.business),
            label: 'B Screen',
          ),
        ],
        currentIndex: _calculateSelectedIndex(context),
        onTap: (int idx) => _onItemTapped(idx, context),
      ),
    );
  }

  static int _calculateSelectedIndex(BuildContext context) {
    final GoRouter route = GoRouter.of(context);
    final String location = route.location;
    if (location == '/a') {
      return 0;
    }
    if (location == '/b') {
      return 1;
    }
    return 0;
  }

  void _onItemTapped(int index, BuildContext context) {
    switch (index) {
      case 0:
        GoRouter.of(context).go('/a');
        break;
      case 1:
        GoRouter.of(context).go('/b');
        break;
    }
  }
}

class ScreenA extends StatelessWidget {
  const ScreenA({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: Center(
        child: Column(
          mainAxisSize: MainAxisSize.min,
          children: <Widget>[
            const Text('Screen A'),
            TextButton(
              onPressed: () {
                GoRouter.of(context).go('/a/details');
              },
              child: const Text('View A details'),
            ),
          ],
        ),
      ),
    );
  }
}

class ScreenB extends StatelessWidget {
  const ScreenB({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: Center(
        child: Column(
          mainAxisSize: MainAxisSize.min,
          children: <Widget>[
            const Text('Screen B'),
            TextButton(
              onPressed: () {
                GoRouter.of(context).go('/b/details');
              },
              child: const Text('View B details'),
            ),
          ],
        ),
      ),
    );
  }
}

class DetailsScreen extends StatelessWidget {
  const DetailsScreen({
    required this.label,
    Key? key,
  }) : super(key: key);

  final String label;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Details Screen'),
      ),
      body: Center(
        child: Text(
          'Details for $label',
          style: Theme.of(context).textTheme.headlineMedium,
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
https://github.com/flutter/flutter/assets/58659088/62bba334-ef52-4298-bae2-525d9389f796

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[!] Flutter (Channel [user-branch], 3.16.0-8.0.pre.4, on macOS 14.2 23C64 darwin-x64, locale en-IN)
    ! Flutter version 3.16.0-8.0.pre.4 on channel [user-branch] at /Users/admin/fvm/versions/master
      Currently on an unknown channel. Run `flutter channel` to switch to an official channel.
      If that doesn't fix the issue, reinstall Flutter by following instructions at https://flutter.dev/docs/get-started/install.
    ! Upstream repository unknown source is not a standard remote.
      Set environment variable "FLUTTER_GIT_URL" to unknown source to dismiss this error.
    • Framework revision ff73448f33 (2 months ago), 2023-10-09 09:10:23 -0700
    • Engine revision 22ce5c6a45
    • Dart version 3.3.0 (build 3.3.0-2.0.dev)
    • DevTools version 2.28.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/admin/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C65
    • CocoaPods version 1.14.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] VS Code (version 1.85.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.78.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 14.2 23C64 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 120.0.6099.109

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

@chunhtai can you please look into it.

Thanks for filing this.
This seems to have been fixed on the latest master channel, are you able to check this out on the latest master 3.18.0-14.0.pre.46?
https://github.com/flutter/flutter/assets/53122008/4cfbf327-2dbb-4f07-824d-1dd0368a333c

@dam-ease Thanks for the quick response, yes it is working on the version you mentioned, any idea when these changes https://github.com/flutter/flutter/pull/134554 will be available in stable?

Thanks for checking that out.
You can read about it here: https://github.com/flutter/flutter/wiki/Flutter-build-release-channels#will-a-particular-bug-fix-be-provided-in-a-hotfix-release (and whole that page for more info)
Closing this as it has been fixed in newer versions.


download_strategy: clear referer by default with curl

This fixes the situaiton when a user .curlrc contains e.g. --referer ';auto' which in turn may break download URLs unless reset to the default (empty) value. Such download URLs are anything under https://downloads.sourceforge.net/.
Also switch to the long-form of the -e option, which is --referer, for readability/discoverability.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

Test:
$ brew install --cask bibdesk
==> Downloading https://downloads.sourceforge.net/bibdesk/BibDesk/BibDesk-1.8.2/BibDesk-1.8.2.zip
==> Downloading from https://sourceforge.net/projects/bibdesk/files/BibDesk/BibDesk-1.8.2/BibDesk-1.8.2.zip/download?use_mirror=kumisystems
######################################################################## 100.0%
Error: SHA256 mismatch
Expected: f5db9daad0239e6e851efbcc45e51d5ffb20003790d7784500307930be16cfab
  Actual: 081a53e1c2aa802fcc6c52be54c19428d60f224990e380fd50f682534148790a
    File: ~/Library/Caches/Homebrew/downloads/[...]--BibDesk-1.8.2.zip
To retry an incomplete download, remove the file above.

This fixes the situaiton when a user .curlrc contains e.g. --referer ';auto' which in turn may break download URLs unless reset to the default (empty) value. Such download URLs are anything under https://downloads.sourceforge.net/.

Do you have HOMEBREW_CURLRC set? I'm assuming so in which case: I don't think it's worth modifying this behaviour unconditionally if we don't have any (more) user reported issues for this?

Is there a disadvantage or cost with merging this? I've intentionally not referred to my personal setup, but one that I accidentally found on the internet, which means it's not my personal issue (though I have HOMEBREW_CURLRC=1 set, which is intentional as I'd like to control some curl options.)

Is there a disadvantage or cost with merging this?

It's a behaviour change that may break some configurations.

but one that I accidentally found on the internet

Where?

I linked to the config in the original post. (Via the word "contain")
How is it realistic that a formula/cask outright expects a user-supplied, via .curlrc, explicit referer to make a download work at all? In such rare case, wouldn't it be realistic to expect that formula to use the existing fake referer option?
On the other hand nearly all existing formula/cask implicitly expect a default (= empty) referer for them to work correcly (unless overridden on a per-formula-basis), so enforcing that the empty value is indeed empty, to me appears to be a net win, and highly unlikely to cause any harm.

On the other hand nearly all existing formula/cask implicitly expect a default (= empty) referer for them to work correcly (unless overridden on a per-formula-basis), so enforcing that the empty value is indeed empty, to me appears to be a net win, and highly unlikely to cause any harm.

I don't believe in making changes to fix speculative bugs that have known workarounds.

Passing on this, sorry.

I don't understand what speculative is about this when clearly SourceForge (and since then GitHub) breaks with curl autoreferrer feature enabled. Formula/cask upgrades DO break if someone uses this option. There is no option to present a different .curlrc to fix this Homebrew issue, so the only workaround is to patch Homebrew after each brew update update. This what I'm doing, but it's clearly bad and quite a disappointing solution.
I'd also have been pleased to see any argument why passing an empty referer would be bad, and how that could possibly break something.

I don't understand what speculative is about this

Well:

I've intentionally not referred to my specific setup, but one that I accidentally found on the internet, which means it's not my personal issue

I don't know if you're being somewhat intentionally difficult here but: it's unclear that this issue affects literally any Homebrew users.

Here's the monkey patch for those unlucky to want to configure their curl and use Homebrew:

No, only those who have HOMEBREW_CURLRC set.
I'm sure this isn't your intention but this interaction just makes me want to remove HOMEBREW_CURLRC entirely.
Look, Homebrew has some settings that you can use in some configurations to break your use of Homebrew. If you do that: that's on you. It's a slippery slope and bad use of time to work around every possible user misconfiguration here.


feat: perform most operations on temporary entities

fixes #6

I did look quickly through this and it looks good!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Inline CSS class specification

For instance the Rust documentation (reference.md) uses markdown that specifies CSS classes for fenced code, like ```{.classa .classb} which produces unbalanced fenced code currently. This should fix it so those classes are parsed and added to the resulting HTML, and the fenced code isn't made unbalanced.
Also removed that LANG_TAG substitution, since assuming the name of the class attribute in HTML to be "class" is pretty safe to make, plus not all CSS classes are only for specifying language highlighting.
No syntax errors in python2.7, python3.4 or pypy3

and of course another file in the Rust doc also decides to use some weird ```lang,class,class syntax, but I give up. -_-

@cyisfor that you for your contribution. However, before I accept your patch, I have a few concerns I would like to address:
What do you mean by unbalanced? I don't see any invalid HTML so you must be referring to something else.
Also, could you include a test to demonstrate that your change works. Specifically, a test that fails before the change is applied and passes after.
Additionally, is appears that you broke an existing test. There are a large number of existing users who are expecting the current output (and they already have existing CSS which works with it). If we change the output, we will break their sites when they upgrade. Is the proposed change worth it?
It also appears that you need to clean up your code to be flake8 compliant.
And finally, read carefully over my analysis of how code blocks should be marked up in HTML. Anything that does not fit that model is likely to be rejected.

Given that lack of response on this issue, I'm closing this for now. I'll reconsider if and when the issues I raised are addressed.

Yeah sorry, I just don't have a reason to do this outside of one rather stupid documentation package that just isn't worth it.


Move the cursor down when sending a line to FSI

I'm not sure if this is a known issue, but several months ago, something broke the functionality that moves the cursor down when sending a line to FSI. I believe this PR fixes this issue by executing the cursorMove command per the vscode documentation: https://code.visualstudio.com/api/references/commands (see screenshot below):

Thanks!


Fix sql and readme

Codecov Report

Merging #328 (fc79689) into master (efe124f) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #328      +/-   ##
==========================================
- Coverage   66.13%   66.10%   -0.04%
==========================================
  Files          90       90
  Lines        6127     6127
==========================================
- Hits         4052     4050       -2
- Misses       1520     1521       +1
- Partials      555      556       +1

Impacted Files
Coverage Δ

plugin/link/httplink/httplink.go
67.85% <0.00%> (-3.58%)
: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 efe124f...fc79689. Read the comment docs.


Intellisense!

Ace has plugins for TypeScript code completion (http://hi104.github.io/typescript-playground-on-ace/). I have heard there will be a move to Monaco anyway.

Monaco is unclear at this point. Post beta, we should look at adding Ace completion.

@davidebbo The Monaco text editor is well supported, its the entire IDE that has more issues. Post beta, we should have a serious discussion about monaco text editor vs ACE.

Yep, we'll look at available options.

@davidebbo - can you assign someone?

First, I think we need to decide whether we want to go down the Monaco route (and figure out where the sources are, the license, ...), or try getting fancier with the ACE route.

Definitely not a decision to be taken lightly!

Let's investigate ACE's intellisense support. I'll try and see what we can do with Monaco, but that's a much bigger change and should only be made if there is equivalent value.

Some interesting links:
https://github.com/ajaxorg/ace/wiki/How-to-enable-Autocomplete-in-the-Ace-editor
https://github.com/sevin7676/Ace.Tern - DEMO: http://sevin7676.github.io/Ace.Tern/demo.html#javascript
Not finding a lot of things for C# auto-complete

Yep, this is the level that I expected, pretty good for js and not much for c#. It seems like a very easy step to just enable this so we should just do it, regardless of whether we end up switching away from ACE.

FYI - Javascript autocomplete with Tern is live on the next environment. It's actually pretty good! Give it a try.

Tried it. Awesome!

Some things to consider - a Javascript intellisense solution would preferably use TypeScript definitions (.d.ts files) - that way we get free, reliable intellisense for most dependencies too. It's my understanding that tern does not use .d.ts files.
I thought that Monaco supported C# intellisense, but apparently not, so tern definitely wins in that regard.

So much excitement!!!

Me likey :)

Closing this since we have tern.js support and ace auto-complete turned on. We should open a subsequent issue to track the C# and other language intellisense (as well as improvements to the Node experience/monaco migration)

#438


display a large loklak icon at the top of the left column of the wall

This feature has the target to reach the milestone at
https://groups.google.com/forum/#!msg/loklak/7azwyuslmTU/q9ueCSgwjZUJ
it also is based on https://github.com/loklak/loklak_webclient/issues/31
The large yellow loklak icon shall be in the top left corner of the wall, inside the left column. Right below the icon shall be the line  loklak wall. Both shall be clickable and linked to loklak.net

please use https://github.com/loklak/loklak_server/blob/master/html/artwork/loklak_icon_yellow.png


cni: enable pprof, stop memory spikes

Turns out once you can see whats going on, its really simple to optimize
functions that are obviously inefficient and have been around for ~5
years!
Fixes https://github.com/istio/istio/issues/49053

In response to a cherrypick label: #50393 failed to apply on top of branch "release-1.20":
Applying: cni: enable pprof, stop memory spikes
Using index info to reconstruct a base tree...
M	cni/pkg/cmd/root.go
M	pkg/file/file.go
Falling back to patching base and 3-way merge...
Auto-merging pkg/file/file.go
CONFLICT (content): Merge conflict in pkg/file/file.go
Auto-merging cni/pkg/cmd/root.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 cni: enable pprof, stop memory spikes
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #50399

In response to a cherrypick label: new pull request created: #50400


Confirm presence and relevance of examples

Part of end-stage-review QC work (ref notes from 18th May 2023).
Confirm that there are appropriate examples in place for every class and property listed in ltc_standard_terms_draft.csv, excluding any ltc:hasFoo terms because examples in that case are given under the Foo docs. Examples should be wrapped in backticks and comma-delimited.

[x] Address
[x] Chronometric Age
[x] Latimer Core Scheme
[x] Collection Status History
[x] Contact Detail
[x] Event
[x] Geographic Context
[x] Geological Context
[x] Identifier
[x] Measurement or Fact
[x] Object Classification
[ ] Object Group
[x] Organisational Unit
[x] Person
[x] Person Role
[x] Record Level
[x] Reference
[x] Resource Relationship
[x] Role
[x] Scheme Measurement Or Fact
[x] Scheme Term
[x] Storage Location
[x] Taxon
[x] Temporal Coverage

Examples are now setup for most, and as new cases come up, we will add new examples


Certify Rancher Container Crontab

Validation completed for  Container Crontab catalog version v0.2.0. No new issues found and https://github.com/rancher/rancher/issues/8979 was validated.


Coordination of documentation of feature integration of third party apps/space for show-off of community projects

I commented in issue #186 that a lot of the recent feature request can be solved through third-party apps (and often through running scripts or node-red in termux).
I assume that several people would be willing to share/show-off their set-ups here. I thought I could open this issue to help coordinating these documentation efforts.
I am working on a couple of home automation youtube videos featuring android based wall panels (running wallpanel and sshd in termux), mycroft, rhasspy, and node-red (I am not personally using hass, even if I often recommend it in my home autmation classes. Therefore, wallpanel is usually just running node-red dashboards for me), I will look if I can make some documentation cross-link suggestions, but things are moving very slowly at the moment, so it might still take a couple of weeks until I have something meaningful to share here.
Rhasspy has a show-off group in their discourse - @thanksmister do you have something like that, or should we use this issue here to collect ideas? I am not sure if github offers a way for that (maybe the wiki?) or if that requires some external tool like discourse. Maybe, having a special doc-subfolder against which we could make pull-requests could be an easy solution for creating "community"-project-documentation? Please add other ideas to realize this as response to this issue.

@ulno I think I will start a discourse group or do something like this for discussion of all my projects.  I started working on a home alarm system recently using Node-Red running on an Android device using termux with a custom Android application in kiosk mode.
The idea is to have a stand alone alarm hub that talks using MQTT to the alarm app and other security sensors that can also work offline.   I like your option of having a node-red dashboard running in WallPanel as well.  I would look forward to any documentation you can share and will let you know when I setup a chat channel for projects.

@ulno finally got discourse up and running, here is the link https://community.thanksmister.com.  I will be adding this to all the repositories so users can go there to discuss or share information.


fix Vagrant multimaster setup

Signed-off-by: samuel.elias<EMAIL_ADDRESS>

Coverage decreased (-0.1%) to 75.261% when pulling 395253d1647d652acbfb30e3f13d40bf3858179d on samelias:multi_master into 882bc4b3041fc1a054ddd054b85bbcd4657a6b0a on contiv:master.


Export to Mac produces 0-length files

Godot version:
3.2.2.stable.official
OS/device including version:
Fedora 32 64bit
Issue description:
Exporting using the standard templates. Linux X11 and Windows exports work. But when I export Mac I get a zip file in which the zipped files are zero length, so it cannot be run.
Steps to reproduce:
Open the project, export to Mac, check out the exported zip and see that though the directory structure is correct the files are empty.
Minimal reproduction project:
Attached is the "Dodge the Creeps" tutorial as I created it.
dodge.zip

Duplicate of #39931, this is fixed in the 3.2 branch and I will soon release 3.2.3 including it and other fixes.
In the meantime, since the problem is with the Zip flags and not the files themselves, you should be able to work it around by unzipping with a different tool than macOS' default archiver, make the Godot binary executable (which is missing) and re-zip.

I don't think this is a duplicate of that issue. I am not unzipping on Mac, but rather on Linux. The files are there in the zip but they are zero length.

See https://github.com/godotengine/godot/issues/39931#issuecomment-651340043
It's a symptom of the same bug, which was fixed in #39977.

Awesome, thanks! And I found a workaround and more details. Will post it here in case someone searches and find this:
The issue in Fedora Linux, specifically, was the GNOME Archive Manager app. It seems to have the same or similar problem as the Mac default archiver, whereby zip entries with bad attributes don't get read correctly. In the case of GNOME Archive Manager it does show those entries, but as length 0. And if you try to extract them, they will be skipped.
However, the "unzip" command line tool is more forgiving. The workaround then is simple in Linux: unzip from the command line, and then you can zip again (you can zip again using GNOME Archive Manager if you want, the files are good now).


DDD-7 Asynchronous Debezium Embedded Engine

https://issues.redhat.com/browse/DBZ-7073

Thanks @jpechane for the comments. I didn't mentioned it in my previous comment, but yes, the main drawback I see with this approach is that sooner or later we will end up with re-implementing most of the Kafka stuff, so I completely agree  on this part.
As for JDBC sink support - in general (read: in undefined future) I'd say yes, IMHO we should support it (the logic behind it is that we should provide the same level of support as for other connectors and as we provide runtime for source connectors, we should do it also for the sink). But definitely would be good to wait for the community first to see if there is any interest for it at all.
Anyway, I have no strong opinion about that, so no problem with keeping most of the logic in the engine itself. If we later on decide to support also sink connector, it should be quite easy do the refactoring.

Is there a possibility that the engine could provide both parallelism and ordering?  In a manner akin to how Kafka handles its partitions, each message would get hashed by its key.  The hashes would be assigned to a queue/thread.  For the use case of an event whose key is the table name, this would gain parallelism while maintaining order at the table level.
Perhaps a pluggable impl to determine the hashes/queue id, where

serial = 1 queue defined and one hash / queue Id always returned
no order =  N number of queues and random or smallest queue hash/id returned
hash based = N number of queues where message key determines hash / queue id

Perhaps this makes things overly complex though.  Anyways, just a thought.

Thanks for your input @jeremy-l-ford , I think this a very good idea! I'll think more about it, but it shouldn't be that hard to implement it.
We probably cannot use it as the default and should still provide total order of messages as the default (e.g. when one table references values in another table with foreign key constraint, insert in such table would fail if it arrives sooner than the insert in referenced table), but it can be another option between running consumer in a single thread and running consumer in multiple threads. Basically what we would need here is to expose ExecutorService (or, in this case, set of ExecutorServices created as newSingleThreadExecutor()) to RecordProcessor, which would be little bit dangerous as we would pass ExecutorService to potentially unknown code (which may either shut down the service or handle exception in an unexpected way), but the benefits may outweigh the risks. Or possibly we can expose ExcutorService only to our implementation of RecordProcessor and if we decide in the future to provide RecordProcessor via SPI, we won't provide ExecutorService to these implementations.
Any thoughts on this @jpechane ?


docs: update README

This PR applies a couple of changes to the README, improving formatting and adding more badges.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
145

Relevant Lines:
172

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
145

Relevant Lines:
172

💛  - Coveralls


[PluggableDevice] Add TF_IsHostMemoryInput and TF_IsHostMemoryOutput to the C API

This reduces boilerplate logic in pluggable devices code that decouple operator registration from kernel implementation. This also makes the transition from TF 1.15 to 2.x easier since the same device code in 1.15 would have been able to use OpKernelContext::input_memory_type.

@penpornk Can you please review this PR ? Thanks!

@penpornk Can you please review this PR ? Thanks!

@penpornk Can you please review this PR ? Thanks!

I'm sorry I get to this PR so late. Is this required for TF-DirectML for TF 2.9? Asking because we are having an API freeze starting tomorrow.

@PatriceVignola Could you please help address @rohan100jain's comment? Thank you very much!

@rohan100jain Thanks for the review! I removed the fatal checks and added a status instead.

Hi @rohan100jain Can you please review this PR ? Thank you!

Hi @rohan100jain Can you please review this PR ? Thank you!


Update and add translations for de_DE

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[x] Other... Please describe: i18n translations added and updated (de_DE)

What is the current behavior?
Console warning "Missing translations for "Text" in language "de".You can use
NzI18nService.setLocale" as a temporary fix. Welcome to submit a pull request to help us optimize the translations!"
Issue Number: N/A
What is the new behavior?
Translations added and updated
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Danke! Thank you for submitting a PR.
Please change your commit message to meet our committing guideline. https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit-message-format


Description of AZURE_FUNCTIONS_ENVIRONMENT is inaccurate

The description says
> When AZURE_FUNCTIONS_ENVIRONMENT isn't set, it defaults to Production.
This may be true in Azure, but it is not true for development in VS. It defaults to "Development".

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

ID: 18aa54fd-3aa1-37c5-925c-51a0195b1030
Version Independent ID: 60198016-1ea9-e7ae-b00b-2c665d5ad8ca
Content: App settings reference for Azure Functions
Content Source: articles/azure-functions/functions-app-settings.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hi @MisinformedDNA - Thank you for your feedback! We will review and update as appropriate.

Hi @MisinformedDNA - We're making an update to the statement to read as follows:

When AZURE_FUNCTIONS_ENVIRONMENT isn't set, it defaults to Production on Azure and Development when developing locally.

@MisinformedDNA We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

I just created a new Function app in VS and if I add the variable to local.settings.json
{
  "IsEncrypted": false,
  "Values": {
    "AzureWebJobsStorage": "UseDevelopmentStorage=true",
    "FUNCTIONS_WORKER_RUNTIME": "dotnet",
    "AZURE_FUNCTIONS_ENVIRONMENT": "Staging"
  }
}

And run locally, then I get the error

Azure Functions Core Tools (2.7.1434 Commit hash: eac7c2888f315da0f2b51831b47f134330b501fd)
Function Runtime Version: 2.0.12555.0
An item with the same key has already been added. Key: AZURE_FUNCTIONS_ENVIRONMENT

How do I set AZURE_FUNCTIONS_ENVIRONMENT when testing in VS?

I just created a new Function app in VS and if I add the variable to local.settings.json
{
  "IsEncrypted": false,
  "Values": {
    "AzureWebJobsStorage": "UseDevelopmentStorage=true",
    "FUNCTIONS_WORKER_RUNTIME": "dotnet",
    "AZURE_FUNCTIONS_ENVIRONMENT": "Staging"
  }
}

And run locally, then I get the error

Azure Functions Core Tools (2.7.1434 Commit hash: eac7c2888f315da0f2b51831b47f134330b501fd)
Function Runtime Version: 2.0.12555.0
An item with the same key has already been added. Key: AZURE_FUNCTIONS_ENVIRONMENT

How do I set AZURE_FUNCTIONS_ENVIRONMENT when testing in VS?

I also experience this. Both on .net core 2.1 and .net core 3.0:

Azure Functions Core Tools (2.7.1846 Commit hash: 458c671341fda1c52bd46e1aa8943cb26e467830)
Function Runtime Version: 2.0.12858.0
An item with the same key has already been added. Key: AZURE_FUNCTIONS_ENVIRONMENT

I can still set ASPNETCORE_ENVIRONMENT on local development machine, and it reflects the given environment when starting up the Azure Functions runtime. I don't know if this reflects in same way in Azure, can someone confirm this? Can I set the ASPNETCORE_ENVIRONMENT in Azure and expect the hosting to be resolved properly, or do I need to make some code to automatically not add the AZURE_FUNCTIONS_ENVIRONMENT when on local development?

I still have the same problem as @kepikoi and cannot enforce to set AZURE_FUNCTIONS_ENVIRONMENT to production on my local machine.
The things I tried:

Set "AZURE_FUNCTIONS_ENVIRONMENT": "Development" in local.settings.json this results in error "can't set key AZURE_FUNCTIONS_ENVIRONMENT twice"
Set environment variable "AZURE_FUNCTIONS_ENVIRONMENT": "Development" in Visual Studio. This does nothing and I receive a message.
Set global environments via Powershell but this only says "Skipping 'AZURE_FUNCTIONS_ENVIRONMENT' from local settings as it's already defined in current environment variables.".

Can someone please help me, because I want to use context.HostingEnvironment.IsDevelopment() and now that is not possible, because it is always set to Production


UI of homepage needs imporvement

The Interface of the homepage should be fancier and not just plain text...

I suggest making a separate homepage that has an image of the event and some details, the date, the location and so on. The tracks is a separate section and shouldn't be the landing page of the user.

i would love to work on this issue. :)

Go code! @saketkumar95

@saketkumar95 Any progress?

@mananwason Can you give me some UI tips, Like what we can make have on the homeView?

@mananwason Hi. I guess did it in other issue. https://github.com/fossasia/open-event-android/issues/991 Please check it out.

@saketkumar95 Please go through material guidelines and come up with a good solution for the usecase.
@heysadboy I have added some comments. Please have a look.

Hey!
Is this issue still open?

No @shreyasnbhat., this has been fixed in #1017. @heysadboy Please refer the correct issue. In #1017 you have referred another PR of yours instead of this issue.

@mananwason Got it. Changed the issue name.


[FEAT]: Investigate Server-Rendering/Static Site Generation

Initial Requirements

Developers will primarily build server-rendered apps
After initial render, the client-side app takes over routing
Also want the ability to statically generate pages of the app
This should be configurable per page/route

SSR/SSG is implemented. Will open a new issue for being configurable per route


driver: gpio: Fixing Pin Direction Setting.

Change Summary:
Moving the setting of Pin direction before setting/clearing the pin configured as output for the change to correctly take place.
Signed-off-by: Ravik Hasija<EMAIL_ADDRESS>

@ravikh19 - just one clarifying question, but otherwise LGTM

@ravikh19 - just one clarifying question, but otherwise LGTM

@cfriedt Thanks for the review. I have also updated API to update direction to use base_addr instead of port_base_addr, since port_addr_base will not work for cases when since IP is split into multiple DTS nodes. Also, looks like writing to direction register was the only odd one out that was using port_base_addr. Please have a look & let me know. Thanks.

@ravikh19 - just one clarifying question, but otherwise LGTM

@cfriedt Thanks for the review. I have also updated API to update direction to use base_addr instead of port_base_addr, since port_addr_base will not work for cases when since IP is split into multiple DTS nodes. Also, looks like writing to direction register was the only odd one out that was using port_base_addr. Please have a look & let me know. Thanks.

I can't say for certain if port base or base should be used because it's hard to say if I have a board i can test this out on.
Just to minimize scope though, i would suggest  keeping the original change and maybe adding a separate commit for port base vs base.
If you're testing on a board, please run through tests/drivers/gpio

@ravikh19 - just one clarifying question, but otherwise LGTM

@cfriedt Thanks for the review. I have also updated API to update direction to use base_addr instead of port_base_addr, since port_addr_base will not work for cases when since IP is split into multiple DTS nodes. Also, looks like writing to direction register was the only odd one out that was using port_base_addr. Please have a look & let me know. Thanks.

I can't say for certain if port base or base should be used because it's hard to say if I have a board i can test this out on.
Just to minimize scope though, i would suggest keeping the original change and maybe adding a separate commit for port base vs base.
If you're testing on a board, please run through tests/drivers/gpio

Sure, Will create a new PR for base_addr change.

@jmnlcruz - your input here would be very helpful, or perhaps Dirk Brandewie? @tbursztyka ?
I doubt I have any easy way to test any of the proposed changes

@henrikbrixandersen - PTAL - I don't have a board to test this on, personally, but it seems like a valid fix.


Transaction support (Undo-redo history)

The canvas state update system needs to be refactored into a reducer for this. Actions would include: clear, drawSingle, drawMany (fill or pencil stroke or eraser), and import maybe

Partially added while improving the drawing efficiency of bucket and drag operations


Update scala-library to 2.12.15

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

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

labels: library-update, semver-patch

Waiting for https://github.com/scoverage/scalac-scoverage-plugin/pull/370


Text Filter with Pagination?

I can't seem to get a normal text filter working with the pagination directive. I am missing something?
<input type="text" data-ng-model="searchText" />
<tr dir-paginate="image in images | itemsPerPage: 10 | filter:searchText">

Thanks for the assistance!
K.

Disregard, as values hidden in my array were causing it to create false positives.

Okay. Also, putting the itemsPerPage filter last in the expression is usually recommended (unless you have a particular reason not to).

Makes sense. Thanks for the assistance.


Component preview

Pull Request for Issue #26
Title
Displaying a preview of the components posted by the users
Description
Now, along with the code, the users will have a toggle option to view a preview of the component posted on the website
Related Issues
N/A
Changes Made
Changes are mainly limited to the Post.tsx file wherein I have added the iframe to display a component preview. I also utilized the DOMPurify package to sanitize the code uploaded by the user for security purposes.
Checklist

[x] I have tested the changes locally
[ ] Documentation has been updated (if necessary)
[ ] Changes are backward-compatible

Screenshots (if applicable)

Additional Notes
N/A
Footer

Yup, just did them


Add colombian holidays

Hey @adanselm cool! When do we plan to release to public? Here is how we do it: https://github.com/Teamweek/holidefs/blob/master/CONTRIBUTING.md#releasing

Hey @adanselm cool! When do we plan to release to public? Here is how we do it: https://github.com/Teamweek/holidefs/blob/master/CONTRIBUTING.md#releasing

Wdyt I also merge https://github.com/Teamweek/holidefs/pull/11 into master?

Yeah, let's merge it, if you could just run the tests for it locally before merging though, it would be awesome 😀

Also, feel free to do #13 if you get time


packager: make prs draft

In last Thursday's pipeline meeting, @yanone mentioned that he'd like the packager to submit draft prs. We liked the idea since a PR made by the packager made need further tweaking.

Tested on https://github.com/google/fonts/pull/7169

Please include a print statement that informs the user about the draft status of the PR.


Viki Error 104 "Unable to download JSON metadata: [Errno 104] "

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.05.01. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.05.01

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

the purpose of this issue is a bug report
provide the full verbose output as follows: (replace it with your log inserted between triple ```):
$ youtube-dl --write-sub --sub-lang 'fr,en' --verbose --sub-format 'srt' --proxy <IP_ADDRESS>:45554 https://www.viki.com/videos/1088385v-cupid-factory-episode-1
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--write-sub', u'--sub-lang', u'fr,en', u'--verbose', u'--sub-format', u'srt', u'--proxy', u'<IP_ADDRESS>:45554', u'https://www.viki.com/videos/1088385v-cupid-factory-episode-1']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.05.01
[debug] Python version 2.7.12 - Linux-4.4.0-77-generic-x86_64-with-Ubuntu-16.04-xenial
[debug] exe versions: avconv 2.8.11-0ubuntu<IP_ADDRESS>, avprobe 2.8.11-0ubuntu<IP_ADDRESS>, ffmpeg 2.8.11-0ubuntu<IP_ADDRESS>, ffprobe 2.8.11-0ubuntu<IP_ADDRESS>
[debug] Proxy map: {u'http': u'<IP_ADDRESS>:45554', u'https': u'<IP_ADDRESS>:45554'}
[viki] 1088385v: Downloading video JSON
ERROR: Unable to download JSON metadata: '' (caused by BadStatusLine("''",)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 498, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2106, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python2.7/urllib2.py", line 429, in open
    response = self._open(req, data)
  File "/usr/lib/python2.7/urllib2.py", line 447, in _open
    '_open', req)
  File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
    result = func(*args)
  File "/usr/local/bin/youtube-dl/youtube_dl/utils.py", line 925, in http_open
    req)
  File "/usr/lib/python2.7/urllib2.py", line 1201, in do_open
    r = h.getresponse(buffering=True)
  File "/usr/lib/python2.7/httplib.py", line 1136, in getresponse
    response.begin()
  File "/usr/lib/python2.7/httplib.py", line 453, in begin
    version, status, reason = self._read_status()
  File "/usr/lib/python2.7/httplib.py", line 417, in _read_status
    raise BadStatusLine(line)
...
<end of log>

<IP_ADDRESS>:45554 is socks proxy, needless to say it appears to be dead.


Download gpg key from https server, since the public key server doesn't ...

...have it.
Based on https://forge.puppetlabs.com/puppetlabs/apt#types

closing because this has conflicts, resolve any conflicts and reopen.

closing because this has conflicts, resolve any conflicts and reopen.


Javascript errors in some browsers

Hi,
I'm using APEX 21.1.2 and have installed CKEditor4 in order to provide a way to edit email messages in an app. It is working perfectly in Chrome (latest version) in Windows 10, but yesterday my colleague told me of problems using Firefox and Chrome on Mac. I checked using MS Edge on Windows and encountered the same problem.
Here's a screenshot from your demo app:

Basically, that is all you get. I have pasted below the JS errors from the console. I don't know why the same issues don't happen in Chrome on Windows.
Thank you for having a look at this and if possible please email me at<EMAIL_ADDRESS>if this is caused by a configuration issue or anything like that.
And thanks for providing your excellent free plugins!
Console errors:
Refused to load the script 'https://cdn.ckeditor.com/4.15.1/full-all/ckeditor.js' because it violates the following Content Security Policy directive: "default-src 'self' 'unsafe-inline' 'unsafe-eval' ajax.googleapis.com ajax.aspnetcdn.com code.jquery.com". Note that 'script-src-elem' was not explicitly set, so 'default-src' is used as a fallback.
Refused to load the script 'https://cdn.ckeditor.com/4.15.1/full-all//adapters/jquery.js' because it violates the following Content Security Policy directive: "default-src 'self' 'unsafe-inline' 'unsafe-eval' ajax.googleapis.com ajax.aspnetcdn.com code.jquery.com". Note that 'script-src-elem' was not explicitly set, so 'default-src' is used as a fallback.
desktop_all.min.js?v=21.1.2:5
ReferenceError: CKEDITOR is not defined
at Object.FOS.item.ckeditor4 (script.js:85:23)
at f?p=114:4010:::::::214:22
at HTMLDocument. (f?p=114:4010:::::::219:5)
at Object.e.init (desktop_all.min.js?v=21.1.2:14:5863)
at HTMLDocument. (f?p=114:4010:::::::210:11)
at e (desktop_all.min.js?v=21.1.2:2:30005)
at t (desktop_all.min.js?v=21.1.2:2:30307)
e.error @ desktop_all.min.js?v=21.1.2:5
desktop_all.min.js?v=21.1.2:5 console.trace
e.error @ desktop_all.min.js?v=21.1.2:5

I accidentally closed  comment


Add missing elements in gh-pages landing page

Have been learning a lot by these examples lately; today I was curious to see some of the samples live, then I've added them to the landing page so that I could run them locally. I figured I'd share it, in case someone else can benefit.
Basically, I've added some of missing ui elements to the index.html page. I left out a couple, since I couldn't find a way to have them working without modifying the original code (like the advanced-router)
I apologize if this wasn't supposed to be done, I figured it would've been an overkill to both open an issue and submit a PR since it's just a couple of lines of code.
Hope it makes sense to you!
Keep up ;)

LGTM. Thanks a lot, @mendaomn. Gonna wait for a +1 by @paullewis to make sure :)

Allllll good! Thank you 👍


Add support for unix domain sockets to TCP endpoint

Quick stab at implementing unix domain socket support. I deliberately wanted to use a SOCK_STREAM instead of SOCK_DGRAM to have bi-directional communication from the start on. (Somehow it didn't work with datagram first.)
I extended the TCP endpoint because I wanted to have the re-connect logic. Somehow it sending didn't work using sendto because of "Transport endpoint is already connected".
The code is not great, but may be a starting point.

Why adding it as client? Wouldn't it make more sense as server?

Why adding it as client? Wouldn't it make more sense as server?

You mean having it the same as TcpServerPort? In general, yes. But, I need to use endpoint groups (PR #330) with these endpoints in my use case. And that doesn't work with the server on mavlink-router's side.

Sorry, I think this is too much if/else to consider extending TcpEndpoint. We could move the re-connection logic to a helper class and make both TcpEndpoint and UnixEndpoint to use it

I think we should leave this for v4. We don't have to take a long time to release it after v3

Definitely don't wait for this PR with the release of v3. I'm not sure though, what the best versioning scheme and release cycle would be, but that's better discussed in #344.

I'm sorry to say this, but I don't have any time in the foreseeable future to rework this. We changed the software, which needed the UNIX domain socket support in the first place to standard TCP server because of several other reasons. So there is no urge to get this done from our side any more, unfortunately.

Several years ago before I knew about mavlino router, I wrote my own router which utilized Unix sockets for coms. If it's still a welcome feature I can take a look at what I had and see if I can can bring it in as a feature. I believe I used a single unix socket as a server node which would send a string notifying the client of the file descriptor it should use for its own private connection. I'll take a look and see if I can dig it up.

Using FD-passing? Yeah, I think that would ok.


Fix storyboard warnings

Description
Fix warning: Automatically Adjusts Font requires using a Dynamic Type text style
Fix warning: Plain Style unsupported in a Navigation Item [9]

Congrats on your first PR in this Project 🎉


Proposal: Move data types to a separate file within FDRS_install

I was wondering if it wouldn't be easier for maintainance and cleaner code if the data types for sensors would go to a separate file within the FDRS_Install folder, e.g.:
FDRS_Install\FDRS_datatypes.h
#define STATUS_T    0  // Status
#define TEMP_T      1  // Temperature
#define TEMP2_T     2  // Temperature #2
#define HUMIDITY_T  3  // Relative Humidity
#define PRESSURE_T  4  // Atmospheric Pressure
#define LIGHT_T     5  // Light (lux)
[...]

This way it could be easily included within the sensor sketches by using
#include <FDRS_datatypes.h>
This would shorten the fdrs_sensor.h's for each sensor a little bit. But the great advantage would be that if the data types are extended it does not have to be edited in every single sensor sketch to match the newest version. Only updating the <FDRS_datatypes.h> file would be sufficient.
Or am I overseeing something?

I guess this would also be a good Idea for the fdrs_functions and fdrs_sensor files.
Can one make a library out of this repo which one could install in the libraries folder?

I would suggest moving all the files containing the same code to a central directory which is to be placed in the libraries folder.

The types as globals is also something that Andreas pushed for as well. My reluctance was because I thought those should only be changed project-wide. That is, I wanted people to contact me to add new types, that way we didn't end up with a bunch of conflicting new ones.  The other issue that I always stuck to was that I wanted the system to stand alone. Like, you can run it out of its sketch folder with no installation. I was happy to have someone see the repo, let alone move files around just to try it out.
Soon, the library approach will make this a non-issue, as things will be installed as the norm. Unless anyone objects. For now, I think I could have it say:
   #ifdef USE_GLOBALS
   #include <library.h>
   #else
   #include "library.h"
   #endif

This would choose the header in the project folder if globals is not available.  Seem good?

I forgot to wrap up my initial thought, which was that the "FDRS_Install\FDRS_datatypes.h" idea seems like a good one.

In favor of moving any duplicate code to the root of the project. Was my original idea with #17 and #20.
It will make code maintenance much easier.
However keep the original examples as is due to the video that Andreas made. Would force any new code to follow the library method. The other option is to create a branch at 3818d88dd5fdbed1f2521bdb624bf6ee088c29f4 and direct people to that branch if they want the version from the video.

The other issue that I always stuck to was that I wanted the system to stand alone. Like, you can run it out of its sketch folder with no installation.

You already have 4-5 dependency's that require installation.
ArduinoJson
LoRa
pubsubclient
ESP32 or/and ESP8266 boards
P.S. This topic will keep popping up.

Okedoke that all sounds reasonable. Here is my plan:

We'll make installing the root directory into the "libraries" folder preferred.
I'll move the new sensor files and "fdrs_sensor.h" into the root.
I'll remove the now-duplicate files among everything except the examples. For now the sensors will use fdrs_sensor, but I hope to test the new sensor sketch thoroughly over the next few days and switch them the the universal sketch soon. In the future I might make a specific folder for just the video examples, as they now go against the addressing standards I want to set. There is a tag marked v2.000 that I made to mark the state where the video was released, if that helps.
We'll do the thing with the data types.
I'm going to sleep and will do this stuff tomorrow.
I always appreciate these kinds of suggestions! I'm figuring this out as I go. I just did a deep-dive into best practices for Arduino libraries and APIs and it's been very interesting. I didn't even realize i was designing an API, let alone such a crappy one! 😅

Okedoke that all sounds reasonable. Here is my plan:
[...]
* We'll do the thing with the data types.

Done :)

* I always appreciate these kinds of suggestions! I'm figuring this out as I go. I just did a deep-dive into best practices for Arduino libraries and APIs and it's been very interesting. I didn't even realize i was designing an API, let alone such a crappy one! 😅

I had a look at how to deploy code as a library yesterday and found the following article very helpful: https://roboticsbackend.com/arduino-create-library/


transition from the circle to the square is smooth now

Changing the circle radius check value will actually change the desire UI which will remove the overlap rectangles

The reason why this will look smooth is because the circle and rounded rect are of the same radius which which will change the design ouput

Not this


Support for hash files more than 512Mb

Now hashers support hashing only 32bit message length in bits.
This is about 512Mb. After that it calculates wrong hash.
I'm ignoring this, but now https://nf404.github.io/crypto-api/example/hasher-file.html calculates 1Mb per second on my core I5, so 600Mb calculates about 10min. This may be not so rare case to calculate hash from 1Gb file in browser )))

This would be a useful feature in CyberChef  as we improve our support for large files.
It would also be useful if your hashers could accept data from an ArrayBuffer instead of having to convert to a string first as this increases processing time.

When I tested ArrayBuffers was slowly than Arrays and Arrays much slowly than strings.
I understand that I need to migrate from strings to ArrayBuffers at least for webasm but for now this would decrease speed.
So the solution will be to create encoder fromArrayBuffer. I created issue #7 for this.

Thanks for the swift response.
I would argue that the best way to read a large binary file into a browser is by using FileReader.readAsArrayBuffer(). At this point, the data is already in ArrayBuffer format and converting it to a string will take a substantial amount of time. This conversion overhead probably means that hashing the file as a string will take longer than hashing it as an ArrayBuffer, even if the ArrayBuffer method is currently slightly slower. I would hope that ArrayBuffer performance will improve in future. In theory they should probably be faster than normal arrays and strings due to their fixed sizes, but obviously a lot of optimisation has gone into arrays and strings in the past.

I use readAsBinaryString for example https://github.com/nf404/crypto-api/blob/master/example/hasher-file.js
I don't compare speed of this two functions but it work pretty fast


Truncate failing for different adapters

I have a remote SQL Server and a Postgres database. Truncate commands are executed in the PostgreSqlAdapter syntax even for SQL Server models and therefore data is not being truncated for SQL Server. The truncate commands is in the format truncate table "public"."Table1"; which is obviously not going to work for SQL Server. I don't know what to do. This is my configuration.
The same configuration was working fine in a previous project where I used multiple MariaDB databases but here the adapter is different.
RSpec.configure do |config|
    config.before(:suite) do
      DatabaseCleaner.add_cleaner(:active_record, SQL_SERVER)
      DatabaseCleaner.clean_with(:truncation)
      DatabaseCleaner.strategy = :transaction
   end

   config.before(:each) do
      DatabaseCleaner.start
   end

  config.after(:each) do
    DatabaseCleaner.clean
  end
end

The problem is with the Truncation class which loads the default connection from database.yml rather than preferring the connection from the individual models. Ideally the problem is in the below code:
    # lib/database_cleaner/active_record/truncation.rb

    class Truncation
       include ::DatabaseCleaner::ActiveRecord::Base
       include ::DatabaseCleaner::Generic::Truncation

      def clean
         connection = connection_class.connection
         ....

After changing it to this.
      # lib/database_cleaner/base.rb

     def clean_with(*args)
      opts = args.pop
      strategy = create_strategy(*args)
      set_strategy_db strategy, self.db

      strategy.clean(opts)
      strategy
    end

     # lib/database_cleaner/active_record/truncation.rb

     def clean(opts = {})
      connection = ActiveRecord::Base.establish_connection(opts).connection
      .....

and changing the configuration into this it is working perfectly fine.
    RSpec.configure do |config|

        config.before(:suite) do
          cleaner = DatabaseCleaner.add_cleaner(:active_record, SQL_SERVER)
          cleaner.clean_with(:truncation, SQL_SERVER)
          DatabaseCleaner.clean_with(:truncation, Rails.configuration.database_configuration[Rails.env])
          DatabaseCleaner.strategy = :transaction
        end

       config.before(:each) do
         DatabaseCleaner.start
       end

      config.after(:each) do
        DatabaseCleaner.clean
      end
   end

@railsmith I understand the problem. Here is an alternative solution: https://gist.github.com/mgreenly/1109325
I'm not convinced that your solution is the best solution.
If you see the documentation, you can specify a single connection like this: https://github.com/DatabaseCleaner/database_cleaner#how-to-use-with-multiple-orms
I'd be more interested in adding a feature like this:
# for a single connection
DatabaseCleaner[:active_record, { :connection => :two }]

# for multiple connections (this is my proposed solution)
DatabaseCleaner[:active_record, { :connection => [:one, :two] }]

@etagwerker
I did not intend to provide the best solution otherwise I would have opened a pull request. I posted the above solution for someone who wants a quick fix by forking the repo and using that fork in an ongoing project. I will open a pull request sometime later.
Thank you for your suggestion.


Null pointer exception on Ghidra plugin

Hello I want to rename a function, then I right click on reva->Rename over the function name and I obtain this:

if you need the program I can provide it

Thank you for this bug report.
The issue is that the right click rename command should detect a high symbol or a global symbol. I will update the handler to detect this case and finish the rename global symbol command to allow renaming functions.
I apologise for the inconvenience, you will have a better time with the chat functions. I am currently working hard on the Ghidra plugin, so there will be bugs and unfinished features for a few more weeks.

The global symbol rename should work now. If we don't support renaming a specific thing, we will print to the log.
If something fails to rename, please let me know!
You should be able to right click and press rename on global symbols (functions, variables) and on things in the decompilation. I do not support everything in the listing view yet, but this should cover most cases.

This weeknd I'll give it a try , thank you


doc: add type casts to README

try to fix #26 by adding type casts to queries in README create table section

@all-contributors Please add odysa for the doc contribution.


Promote suitable instructions to constant instruction

For example, the following composite construct:
[[d0:%\d+]] = OpCompositeConstruct %v3float %float_6 %float_7 %float_8
[[d1:%\d+]] = OpCompositeConstruct %v3float %float_9 %float_10 %float_11
[[d:%\d+]] = OpCompositeConstruct %mat2v3float [[d0]] [[d1]]

Could be turned into
[[d0:%\d+]] = OpConstantComposite %v3float %float_6 %float_7 %float_8
[[d1:%\d+]] = OpConstantComposite %v3float %float_9 %float_10 %float_11
[[d:%\d+]] = OpConstantComposite %mat2v3float [[d0]] [[d1]]

This will further enable const folding passes.

Ah, if making a composite out of only constants, then make the whole thing a constant.  Good idea.

This is an "instcombine"-like optimization.

Yep. I am having this issue right now in spiregg codegen for variable initializers.
It seems the frontend lacks const folding support for HLSL specific types in DirectXShaderCompiler; so it ends up when we are generating intializers for some variables like vectors/matrices, we are always resolving to OpStore, even if the initializer is a constant.

And that seems suggesting another pass: using initializers to replace OpStores. :)

Can they go all the way to just an SSA  value and eliminate the local variable?  (Function storage class)?
@greg-lunarg contributed a number of passes recently that combine to do that.  E.g. see local_access_chain_convert_pass, local_single_block_elim_pass, local_ssa_elim_pass.    They have a few restrictions (e.g. they only work on entry point functions), but that could be generalized with some work.

True if not written to I think.

Yes, there's a local_single_store_elim_pass  for when the store is written to only once.  Then all dominated loads can just use that one value.

Done with #1283.


DOC change doc plot kinds

Corrects the help of plot kinds and closes #124

If you write
closes #124 in pr description so when we merge it will close the issue automatically

Codecov Report

Merging #129 (13e03bb) into master (58215d9) will decrease coverage by 0.25%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #129      +/-   ##
==========================================
- Coverage   74.45%   74.20%   -0.26%
==========================================
  Files          28       28
  Lines        1190     1190
  Branches      180      180
==========================================
- Hits          886      883       -3
- Misses        243      246       +3
  Partials       61       61

I did this instead :
https://github.com/ngazagna/benchOpt/blob/6d62edc3f4d38c0e32448b81e3cc551f75293851/benchopt/cli.py#L158-L160
Can we remove the "+" ?

What we did with @tomMoral

What we did with @tomMoral

Thx a lot @ngazagna for taking the time to do this! :fire:

Thx a lot @ngazagna for taking the time to do this! :fire:


Upgrade frontier to support polkadot-v0.9.16

Motivation
Currently my team uses frontier and is a parachain. We want to do testing on the latest version of live Rococo (polkadot-v0.9.16). Currently we utilize the branch worked on by @riusricardo frontier-para. This branch is utilizing polkadot-v0.9.13.
Suggested Solution
Checkout frontier-para branch and implement similar upgrade to support latest polkadot version polkadot-v0.9.16. This would be my first time upgrading or altering any frontier related code. Is this the right approach? Any guidance would be appreciated!
Alternatives
Not sure what an alternative would be.
Additional Information
Currently if you want to use frontier for a parachain I do not know any other way then to continuously implement these upgrades.

Will do this koushiro! Thank you for the feedback!

As koushiro said, this can be solved by replacing all substrate dependencies with branch = "polkadot-v0.9.16"
then delete the template folder or if you want to fix the code in it

I got up to 15 working, but it brakes with 17

https://github.com/flipchan/frontier-pallet-upstream

Previously I cloned Frontier's main branch that has the substrate version I needed, pasted that into my own repo with a branch name like polkadot-v0.9.17, then made my parachain to use this repo instead of using Frontier's official repo as the following:
In my parachain's root Cargo.toml
[patch."https://github.com/paritytech/frontier"]
pallet-evm = { git = "https://github.com/your_username/frontier-mod", branch = "polkadot-v0.9.17" }


Resolves #21. Adds aggregate currency repository.

Current coverage is 100% (diff: 100%)

Merging #22 into master will not change coverage

@@           master   #22   diff @@
===================================
  Files           3     3
  Lines          73    88    +15
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
+ Hits           73    88    +15
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update e1b0ece...f869b28


kup list does not show locally installed versions of tools

If I do kup install kontrol --version . inside the kontrol repository, then do kup list, it shows kontrol as not installed. It should instead point to the path it was installed from, or the commit it was installed from, or COMMIT-dirty if the commit is dirty.
Discovered by several members of the Kontrol team.

completed in #102


Log reason for unready model mesh container

I'm currently debugging constantly unready model mesh containers, which do not directly indicate the problem in the logs. Only through reading the code I discovered abortStartup, which fails the ready probe without logging the reason https://github.com/kserve/modelmesh/blob/c4fc0417b6a7a4cce332630a564fe99f80da9dce/src/main/java/com/ibm/watson/modelmesh/ModelMesh.java#L1311-L1313
ModelMesh.abortStartup indicates an unrecoverable failure, which can only be resolved with an restart of model mesh container.
If the readiness probe fails because of abortStartup it should be logged, to allow debugging the root cause of the issue.
Alternatively, the Liveness probe of the container should fail to indicate an unrecoverable application failure.

@Legion2 -- would you like to propose a code change in a PR?

@ckadner I created #132


Add packages to runtimes.

Adding some popular packages to the base image - I floated the patch number so that as fixes are released we pick them up as the images are built via Travis.

What do you think about adding a jsonwebtoken which is an industry standard RFC 7519?
https://www.npmjs.com/package/jsonwebtoken

Feedback on dev list was mostly not in favor of maintaining packages in the runtimes. I may publish an image separately.


Condensed

I don't think this was intended to be committed. If it was, please reopen with a description of the changes you'd like to suggest.


merge package from arlac77/npm-package-template

package.json

chore(scripts): cover@#overwrite c8 --temp-directory build/coverage ava && c8 report -r lcov --temp-directory build/coverage
chore(engines<EMAIL_ADDRESS>chore(package): add xo from template
chore(package): add nyc from template
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/-test.js,tests/-test.mjs' as in template
chore(package): set $.ava.extensions='js,mjs' as in template

.travis.yml

chore(travis): remove node versions 10.14.1
chore(travis): add node versions 10.14.2

Codecov Report

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

@@           Coverage Diff           @@
##           master     #316   +/-   ##
=======================================
  Coverage   84.11%   84.11%
=======================================
  Files           1        1
  Lines         214      214
=======================================
  Hits          180      180
  Misses         34       34

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 2bf6196...c30e8a1. Read the comment docs.

Coverage remained the same at 84.81% when pulling c30e8a1ad93d9803100b31169e804368d924a2d5 on template-sync-1 into 2bf6196e8925f0dd8d32c3b2abffdb425dba0b1d on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


URL SVG loading result in: 'Content is not allowed in prolog'

Hi, maybe im doing something wrong but when i try to load the following svg  from an URL : https://svgs.scryfall.io/sets/fdn.svg?1721016000
SVGLoader.load("https://svgs.scryfall.io/sets/fdn.svg?1721016000");
I got the following error: Content is not allowed in prolog
After some research, it seems to came from a character that may be declared before the XML declaration.
I then tried to download the SVG and load it from a file and it worked like a charm, is there a known limitation?

Sorry for opening this issue it was me doing something wrong:
I used the wrong load method:
SVGLoader.load("https://svgs.scryfall.io/sets/fdn.svg?1721016000");
And should have used the following:
SVGLoader.load(URI.create("https://svgs.scryfall.io/sets/fdn.svg?1721016000").toURL());


In comment Title, "NO." -> "No." to agree with GSS Release 23 (Jan 2023)

To agree with GS1 General Specifications Release 23 (Jan 2023), comment titles containing "NO." -> "No."

@PetaDing-GS1 For your consideration and approval.

Reviewed and approved. Thank you!


(SSR) groupElement is null

I didn't look at all the docs but i have succeed to use Next and React on the old version that i'm using (17 and 12) with react-rezisable-panels using the linked demo https://github.com/bvaughn/react-resizable-panels-demo-ssr so this is not blocking i will migrate to this repo. However i had an issue so why not explain it.
After refresh the ui error because server and client doesn't have the same id. i tried using onLayout but i didn't worked, i even copied paste the component didn't work either. But as i said i didn't have the time and skill to look into it, plus i can still go on on my little project. but i though i could open an issue, might be useful

There isn’t enough info here for me to help. You’ll need to share a way to reproduce the error above.

I made a pr with the last commit having this bug here -> https://github.com/jocelinqueau/RELIST/pull/14/commits/5c0c3d8833983aaf977397543dd9744b26ae7ba4#diff-da6f79e0a52169a08ce0c0921c3518bdd9f1b732d32b3c393ac3b26496dc327c
If you prefer having the info in a different way, i can try to do it, just tell me what you want..

@jocelinqueau seeing the code is more helpful than not, but how do I reproduce the error? (What are the steps for me to follow that will show me the same error you copy pasted above?)

ha yes sorry

install the dependencies(i used pnpm not sure why)
go to the page http://<IP_ADDRESS>:3000/styling/liked
refresh the page

$ pnpm install
 ERROR  /<EMAIL_ADDRESS>is an invalid relative dependency path

well sorry, i see that i didn't mention the page is only on the branch of the commit / pr i mentioned feature/update-layout and should be accessible without login to spotify. Anyway i tried to replicate on codesandbox in the following way, again sorry for the trouble

Checking out that branch and loading the link you showed shows the following error in the server terminal:
ReferenceError: localStorage is not defined
    at V2 (webpack-internal:///./src/layouts/v2.tsx:28:9)

This can be fixed by changing your v2 code to:
  let storage;
  if (typeof localStorage !== 'undefined') {
    storage = localStorage;
  }

Looking in React DevTools I also see:

So it looks like you haven't added a PanelResizeHandle component, which you'll need. (I confirmed that by searching the branch you mentioned to see if it was imported anywhere.) I added one, and only then was I able to see the error you reported.

Looks like the error is happening because the group id in useWindowSplitterBehavior doesn't match the group id that was rendered in the DOM. I guess that's because my attempt to poorlyfill useId for older React versions isn't working.
The work around for that is to provide ids as props.
That seems to be enough to get things working.
diff --git a/src/layouts/v2.tsx b/src/layouts/v2.tsx
index 91208c6..1c6b3f4 100644
--- a/src/layouts/v2.tsx
+++ b/src/layouts/v2.tsx
@@ -3,32 +3,42 @@
 import { Links } from '@/components/v2/Sidebar/Links';
 import { PlayerV2 } from '@/components/v2/Sidebar/Player/Player';
 import { ReactNode } from 'react';
-import { Panel, PanelGroup } from 'react-resizable-panels';
+import { Panel, PanelGroup, PanelResizeHandle } from 'react-resizable-panels';

-
-
-export const V2 = ({ children, defaultLayout = [33, 67] }: { children: ReactNode, defaultLayout?: number[] | undefined }) => {
+export const V2 = ({
+  children,
+  defaultLayout = [33, 67],
+}: {
+  children: ReactNode;
+  defaultLayout?: number[] | undefined;
+}) => {
   const onLayout = (sizes: number[]) => {
     document.cookie = `react-resizable-panels:layout=${JSON.stringify(sizes)}`;
   };

   let storage;
-  try {
+  if (typeof localStorage !== 'undefined') {
     storage = localStorage;
-  } catch (error) {
-    console.log(error);
   }

   return (

-      <PanelGroup onLayout={onLayout} direction="horizontal">
-        <Panel  minSize={20} defaultSize={defaultLayout[0]}>
+      <PanelGroup id="panel-group" onLayout={onLayout} direction="horizontal">
+        <Panel
+          minSize={20}
+          defaultSize={defaultLayout[0]}
+          id="left-panel"
+          storage={storage}
+        >

             <PlayerV2 />
             <Links />

         </Panel>
-        <Panel minSize={50} defaultSize={defaultLayout[1]}>
+        <PanelResizeHandle id="resize-handle">
+

+        </PanelResizeHandle>
+        <Panel id="right-panel" minSize={50} defaultSize={defaultLayout[1]}>

             {children}

@@ -38,4 +48,4 @@ export const V2 = ({ children, defaultLayout = [33, 67] }: { children: ReactNode
   );
 };

-export default V2;
\ No newline at end of file
+export default V2;

Ok thank you for your answers


Navigating an OptionButton's PopupMenu using keyboard or controller causes flashing items

Tested versions
Godot_v4.3-stable, Godot_v4.4-dev5
System information
Godot v4.3.stable - Windows 10.0.19045 - Vulkan (Forward+ [This bug also appears using the compatibility renderer]) - dedicated NVIDIA GeForce RTX 4070 (NVIDIA; <IP_ADDRESS>90) - AMD Ryzen 5 7600 6-Core Processor (12 Threads)
Issue description
Navigating an OptionButton's PopupMenu with a keyboard's arrow keys or a controller ocationally causes the items in the menu to quickly flash before settling on a seemingly random item. This glitchy behavior only happens durring the first input on that menu.
This issue is similar #42230
Steps to reproduce
Use a keybaord's arrow keys to navigate an OptionButton's PopupMenu.
This can be quickly done in Godot's project manager in the top right where it says: "Sort: Last Edited".
Reproducing this bug is inconsitant so it may take a few attempts.
Minimal reproduction project (MRP)
N/A

Sound to be same as this issue https://github.com/godotengine/godot/issues/94712#issuecomment-2282093214

Thanks for sending that link Giganzo, I belive this is the same issue as the one you mentioned.
I searched and searched for a long time but coudln't find anyone with the same issue.
This issue could probably be closed / merged with the one you mentioned unless anyone has anything else to add to this.


[Extra Metata Video] Importing video for trailer from file deletes original file

I don't know if this has been reported, so I'm reporting it.
The options "Set trailer video for the selected game" and "Set microtrailer video for the selected game" don't work, and delete the original file in the process. This can be reproduced on any game :

Right click on game and select one of the two options
Set the video file, in my case an MP4 video

Process is finished, but there is no trailer to be found, and the original file disappeared

The issue seems to be present on any game, and so far has only been tested with .mp4 files.

Fixed a while ago but forgot to close


HOG not able to detect when is apply to a roi of the same image

I have trained a hog detector(scan_fhog_pyramid<pyramid_disable>), when I passed it an image of the same size as the images used in the training, it is able to detect, but is I select a ROI of the same image it is not able to detect anything.
After googling and looking into the code and not found any clue. I thought if it is any with the slide window size.
Is the slide window size dependent of the input image dimension? And this generates slide windows of size much less small that my 'features' and then it is not able to found it?
Can you help me?
Greeting.

I'm not sure exactly what you are asking, but you probably don't want to
use pyramid_disable.  Use the recommended default and it will be able to
detect things regardless of their size in the image.

I have the following image(640.380):

And in it the feature is detected(in this image is visible only the feature,) and if I crop a little area that contains just the feature as the next image(128x76):

I crop this image, not resize, not scale and the HOG detector can't detect it.
What is wrong with this?
Why is it unable to find it?

It really depends on your training data.  There are border effects that
cause the edge of the image to look like it has a black stripe around it.
So if the training data never has any objects next to the edge of the
image, or next to a perfect black stripe, it will probably not learn to
detect things like that.

Thanks @davisking, but I think this is not the case. In the sample I show you, the light blue color is to hide the other part of the image. My main question is why in the big image I am able to detect the hog and in the little one that is a part of the big one I am not able to detect it. I think is cause of the slide window size using to detect the hog in the little one. Can you explain to me what are the possible sizes of the slide window in the little image and how the pyramid_disable affect it?


Fix failing Jest tests

Description
Some Jest tests were failing because of this error
Test suite failed to run

    TypeError: Cannot destructure property 'access' of 'fs_1.promises' as it is undefined.

      at Object.<anonymous> (node_modules/@actions/core/lib/summary.js:45:3)
      at Object.<anonymous> (node_modules/@actions/core/lib/core.js:429:17)

Found the solution here: https://github.com/actions/toolkit/issues/1075
Related Issue

Motivation and Context

How Has This Been Tested?
npm run test
Screenshots (if appropriate):
Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

@AmyJZhao can you link the build results to the pull request?


Usage with TextField

Currently there aren't perfect solution for the most common form field.
TextFormField
Works. However formini state resets on hot reload and TextFormField's state doesn't causing the input show previous (mismatching with state) value.
FormControlBuilder(
  control: user.email,
  builder: (context, state) => TextFormField(
    initialValue: state.value,
    onChanged: user.email.setValue,
    decoration: InputDecoration(
      labelText: 'Email address',
      errorText: state.dirty ? state.error?.toString() : null,
    ),
  ),
)

TextField
Works. However the TextEditingController also controls the cursor state so creating a new control for every value causes the cursor to go to the beginning of the value.
FormControlBuilder(
  control: user.email,
  builder: (context, state) => TextField(
    controller: TextEditingController(text: state.value),
    onChanged: user.email.setValue,
    decoration: InputDecoration(
      labelText: 'Email address',
      errorText: state.dirty ? state.error?.toString() : null,
    ),
  ),
)

Formini state reseting on hot reload is probably something we want to fix. I still really don't like the fact the view widget (TextFormField) has a duplicate state of the field.


Poor jpeg output quality if gamma correction is enabled

I had some quality issues on certain jpeg images scaled with php-image-resize. Even with a quality setting of 100, part of the images showed ugly artifacts.
I found that disabling gamma correction restores the image to an acceptable quality. I do not know what gamma correction exactly is supposed to do, so can't comment on whether changing the parameters of the gamma correction would result in better images.
Here is the code I used for testing. The original image is named orig.jpg. It produces 3 scaled down images (from 2000px width to 1600px width). For 3 quality settings (80,90,100). Once with gamma correction enabled and once without. So a total of 6 images are produced.
$destination_width = 1600;
foreach([true,false] as $gamma) {
	foreach([80,90,100] as $quality) {
		$image = new \Gumlet\ImageResize(__DIR__.'/orig.jpg');
		$image->quality_jpg = $quality;
		$image->gamma($gamma);
		$image->resizeToWidth($destination_width, false);
		$destination = __DIR__.'/'.$quality.'_'.($gamma?'gammatrue':'gammafalse').'.jpg';
		$image->save($destination);
	}
}

Here is the original image.

For the quality 90 scaled down image, here is the result with gamma correction enabled. Notice the artifacts on the floor (you may need to view the image in a new tab for the full size)

For the quality 90 scaled down image, here is the result without gamma correction. The artifacts are gone

So the question is: is there some setting I am missing that would allow to use gamma correction without having these quality issues?

You should disable Gamma correction if you are facing such issues.


Unable to use RESTful API to post message

I try to use RESTful API to post messages to Azure SignalR from a thrid-party App, but the API doesn't work.
My signalR version is 1.0, here is my API format:
 https://{mysignalRname}.service.signalr.net/api/v1/hubs/{hubname}

message is:
{
        'target': 'newResult',
        'arguments': [{
            'tag': xxx",
            'url': "xxxxxxxxx"
        }]
    }

I get the 401 response. I learned that Jwt token may be used but I don't find any official document related that. How can I solve this problem? Thanks in advance

I got this error info after using JWT:
Bearer error="invalid_token", error_description="The audience is invalid"

My aud is the API: https://{mysignalRname}.service.signalr.net/api/v1/hubs/{hubname}“
What should the aud be?


README: php requirement has been bumped to 7.2

see https://github.com/messagebird/php-rest-api/pull/108#discussion_r365210940

@webmaster777 Thanks for catching this, we'll actually keep support now back up to 7.1, I've included this readme update in #111


feat: add error handling lib for each possible revert

The errors will always return a bytes(hex) value, that can be converted into legible data when using:
contract.interface.decodeFunctionResult("param", response)

The bytes look like this error at "return data:":

An example can be seen at tasks-service.ts:

The task is to display the error in a readable manner through alerts or tooltips in the front-end

This was fixed in #97 and #67


Please provide some more information about restore scenario's

Hi,
Just had to restore my authenticator app because of a new phone. For the re-verification this documentation page provides clear information.
I didn't find any information though on how to do this for external tenant where my work account is a Guest user, eventually figured out that you need to go to: https://account.activedirectory.windowsazure.com/r#/profile switch to the profile and if everything if ok you can find an option here to "Edit security info". This process wasn't fluid, since I am a Guest account in multiple other Azure AD's and switching between those tenants so that I could modify my security information didn't always work.
For the case where my Gmail account was a guest user, I didn't find any instructions at all on how to re-authenticate using the authenticator app. Luckely I'm also the global admin of the tenant in which my personal account was a guest, so I could do a MFA reset from there.
Adding documentation for these scenario's would really help -
Thanks in advance,
/Kenneth
[Enter feedback here]

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

ID: 8b52a210-1be6-4e5f-d95c-8ea7b87919ca
Version Independent ID: 02a43ebb-7471-d7ff-4f7e-84c9f7a66326
Content: Back up and recover accounts with the Microsoft Authenticator app - Azure AD
Content Source: articles/active-directory/user-help/user-help-auth-app-backup-recovery.md
Service: active-directory
Sub-service: user-help
GitHub Login: @curtand
Microsoft Alias: curtand

@kennethvs
Thanks for your feedback! We will investigate and update as appropriate.

+1 for this issue. The documentation is severely lacking for Microsoft Work accounts and in particular Guest accounts are difficult to figure out.
As @kennethvs points out for guest accounts you can go to https://account.activedirectory.windowsazure.com/r#/profile and switch org in the top right. But only for some of my guest accounts do I see the "Edit security info". On most it is just blank page - I have been unable to regain access to these. I can see no reason why some appear and some do not.
Surely the purpose of the Restore process is to restore the accounts? This process is actually "go set everything up again" - and you are lucky if you can with some accounts. I can understand if you need the old device to approve the transfer but this is really broken as it stands.