Added support for MSSQL to the DatabaseContextProvider

Description
I've added support for MSSQL to the DatabaseContextProvider, honestly just a few config tweaks were required.
The package dbinfoz that DatabaseContextProvider relies on has already been updated to support MSSQL.
Checklist

[x] The base branch of this PR is dev, rather than main
[x] The relevant docs, if any, have been updated or created

Screenshots
[ For visual changes, include screenshots. ]
Testing
Install SQL Server
Ensure local connectivity to SQL Server
Update continue.dev config for @database context provider and add mssql connection paramaters.
Try out @database with the mssql server

I closed my previous PR and instead made the changes on the dev branch.

It is my understanding from the thread above that we can leave this PR to just being the upgrade to dbinfoz?

Not entirely correct, the arguments for mssql is slightly different than for postgres, namely where postgres you pass 'host' argument, in mssql you pass 'server' argument instead, so a few of my changes were to add the necessary validation for it.

Hey there!
I've got the latest version installed, also tried release version but for some reason still shows "Unsupported database type" when trying to link up my local mssql db in my continue.dev config file.
Not sure how to go about fixing it, from what I can tell the dbinfoz package is indeed correct version in continue.dev's package.json ie 0.14.0.
Anyone have any idea how to fix it?

I opened up the installed extension folder and searched for dbinfo in the compiled code but cannot find the mssql adapter there.
But it does seem to be using dbinfoz package 0.14.0 so not sure what's going on.

Perhaps the old code is being cached somehow during the workflow?
https://github.com/continuedev/continue/blob/main/.github/workflows/main.yaml

Perhaps the old code is being cached somehow during the workflow?
https://github.com/continuedev/continue/blob/main/.github/workflows/main.yaml
@Patrick-Erichsen @rootedbox


Update the links which should have been latest in READMEs

This probably doesn't deserve its own release, so it's just going to sit here, haunting me, for all time :)

Indeed. This is kind of the point of https://github.com/linebender/linebender.github.io/pull/55 :)
(I was going to put it on my own blog, but I don't have my own blog 😆)
Another evidence of no overwriting is:

https://docs.rs/android_trace/0.1.1/android_trace/index.html; AND
https://docs.rs/android_trace/0.1.0/android_trace/index.html

Both of those link to the correct version of the symbol at AndroidTrace

Forgot to actually press the merge button :P


Option to disable "diff file dropped" popup

When I drag a file onto goneovim (v0.6.3), it shows this popup:

How can I make it always edit the file instead of asking to diff?
I did :help goneovim and searched for 'drop' and 'drag' and 'diff' but didn't find any relevant settings. Is there an option to disable this popup?
While, yes, I'm using my mouse to drag and drop, I don't want to mouse down to this popup to dismiss it. The popup can't be closed with Esc.

NVIM v0.10.0
Build type: Release
LuaJIT 2.1.1713484068

I tested the Windows build and it works great. I also tested turning it back on:
[Editor]
ShowDiffDialogOnDrop = true

All works well.
It seems the default value of ShowDiffDialogOnDrop is false which changes existing goneovim behaviour. That seems preferable to me, but may not be intentional?
Regardless, thanks for the quick fix!

Thanks for confirming.
Yes, that default is intentional.


Module 1 exercises [Enoch Anchebah]

Please "drag & drop" exercise 1 pdf file below.
First Assignment.pdf

Please "drag & drop" exercise 2 pdf file below.
Second Assignment.pdf

Please UPDATE THE TITLE of this "Issue" in the title bar above with your name.

If you haven't done so, please go back to Module 1 and make sure to take
the post-training survey. Participating in surveys is required for receiving
the "GALUP Certificate" by the University of Florida.

Well done, Enoch. You clearly grasped the material outlined in Module 1 and successfully applied it to these exercises. There was a minor issue the coloring on exercise 1, but this can be fixed by ensuring that the correct "reds" color ramp is chosen so that the resulting map resembles the example in the module. Thank you for your participation! Please be sure to fill out the post-Module survey before you begin the next Module.


Release 1.4.0a1

Features

Add dbt docs natively in Airflow via plugin by @dwreeves in #737
Support partial parsing to render DAGs faster when using ExecutionMode.LOCAL, ExecutionMode.VIRTUALENV and LoadMode.DBT_LS by @dwreeves in #800
Add Azure Container Instance as Execution Mode by @danielvdende in #771
Add dbt build operators by @dylanharper-qz in #795
Add dbt profile config variables to mapped profile by @ykuc #794

Bug fixes

Fix folder_dir not showing on logs for DbtDocsS3LocalOperator by @PrimOox in #856
Improve dbt ls parsing resilience to missing tags/config by @tatiana in #859
Fix operator_args modified in place in Airflow converter by @jbandoro in #835
Fix Docker and Kubernetes operators execute method resolution by @jbandoro in #849

Docs

Fix docs homepage link by @jlaneve in #860
Fix docs ExecutionConfig.dbt_project_path by @jbandoro in #847
Fix typo in MWAA getting started guide by @jlaneve in #866

Others

Add performance integration tests by @jlaneve in #827
Add connect_retries to databricks profile to fix expensive integration failures by @jbandoro in #826
Add Python 3.11 to CI/tests by @tatiana and @jbandoro in #821, #824 and #825
Fix failing test_created_pod for apache-airflow-providers-cncf-kubernetes after v8.0.0 update by @jbandoro in #854
Extend DatabricksTokenProfileMapping test to include session properties by @tatiana in #858
Fix broken integration test uncovered from Pytest 8.0 update by @jbandoro in #845
Pre-commit hook updates in #834, #843 and #852

I'd like to include the following PRs in this (alpha) release:

https://github.com/astronomer/astronomer-cosmos/pull/850
https://github.com/astronomer/astronomer-cosmos/pull/786
https://github.com/astronomer/astronomer-cosmos/pull/683

If we can get their tests passing in the next 12 hours, we can release them in this alpha release; if not - they will join the next alpha release / or they will be part of the stable 1.4 release.

https://github.com/astronomer/astronomer-cosmos/pull/850 was merged and can be included in this alpha release - thanks!

Lemme see what I can do about #786 tonight...

#786 needs a refresh on coverage tests

#786 needs a refresh on coverage tests

I've added the missing tests, @dwreeves , thanks a lot!

I believe there is an issue with the implementation of #863 - I left some feedback and we can still add it to the 1.4 release after.


jarek/create beakerx as individual component

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

I tested creating and installation package:
I used script dev_scripts/build_conda_packages.sh to create conda package. It works.
Installation conda install --use-local beakerx-all - also works for me

showEnvs works fine


Recent blog posts' titles should also link to the post

We have "read more" links, but the recent blog posts' titles should also link to the appropriate post.

Addressed in #192


Support for  Wio Terminal and  Xiao

Hello @mocleiri,
Thanks for the fantastic work. Could you please guide me on how to port tensorflow-micropython-example for both SAMD  boards(Wio Terminal and Xiao)? Here is  Ardupy which is supported by both boards, I think it might help. Thanks.

This project is a micropython C user module.  You need to get a board configuration and then build micropython for that board incorporating the microlite module.
The main issue is the build environment's support of C/C++'s standard libraries.  The ESP32 and RP2040 builds work nicely because the standard libraries are included in their respective  SDK's.
The STM32  port doesn't work right because of difficulties in integrating the standard libraries into the bare metal micropython port.
I know its possible because openmv is primarially using stm32 based boards fine; but I haven't been able to figure out how to build properly linking the C++ libraries.
Really you want to do a SAMD micropython build and include the microlite module as part of the build.  Then you want to flash and see what happens when running inference.


[WIP] Locking commands

This PR adds to @sinbad's original proposal and extends it to document some particular commands that we will have to teach the LFS client.
All of the documentation surrounding these new commands and API methods are in the contents of this PR, so I will keep this description brief 😄
Thanks in advance for reading! 👋

Hmm, after thinking more about this, I realized that will need to expand the API to cover the commit negotiation that @sinbad described in his original pull-request. I added partial support for this in 6864ac5cc439199fea06ddd3c9ee2f103a26fdb9, but I will have to think about a more flexible flow to support more implementations.

Demoting this PR to a [WIP] as I still have to consider some final thoughts for the lock-listing API. In particular:
Pagination is a bit of a tricky subject, since there is no guarantee that a client will get all of the locks they requested if a multipage response is returned and a new lock was created while paginating through the n-th page of that response. @technoweenie and I discussed that perhaps the best way around this was to eliminate the notion of "pages" and instead make it a requirement that the server return results in a reverse-chronological ordering. Clients could instead send the last UUID that they received and the server would send them n results after that, and we'd still keep the HasMore bool.
 The current filter definition that I proposed:
type Filter struct {
        // Prop is the property to search against.
        Prop Property `json:"prop"`
        // Value is the value that the property must take.
        Value string `json:"value"`
}

works well for equality operations, but fails for <, >, and != operations. This is sort of a rabbit-hole, since once you consider adding operations for < and >, it's easy to add nested filters and ANDs, ORs, and XORs. I'm not sure how far we should take this idea, but if we pursue the idea of nested filters, I could work on an API to accommodate that.

I like the type definitions, but I think it'd be helpful thinking about the API too. For example, I'm not sure unlocking will require a struct like this:
// An UnlockRequest is sent by the client over the API when they wish to remove
// a lock associated with the given UUID.
type UnlockRequest struct {
        // UUID is the identifier of the lock that the client wishes to remove.
        UUID string `json:"path"`
}

Given an LFS base url of https://git-server.com/repo/info/lfs, the unlock API call could be something like POST https://git-server.com/repo/info/lfs/locks/{uuid} -d {"operation":"unlock", ...}.
Also, I think the spec should recommend that server implementations keep locks for auditing purposes. This is why the unlock action is a POST to the resource, and not a DELETE.

On second thought, the currently proposed filter API might be alright for V1. I'm not sure the >/< operators are that useful for anything other than created_at time. It would be nice to greater support for nested filters, but I move that we adopt the current model for now, and expand on it in v2 if we find a greater need for more filtering operations.

Merging this. Great job finishing the proposal, time to get crackin on a prototype :metal:

Thanks, @technoweenie! Let's do it.


TextFormField suffix icon doesn't turn red on validation error

Steps to Reproduce
Create and validate the TextFormField below:
 TextFormField(
  decoration: const InputDecoration(
    labelText: "Password",
    border: OutlineInputBorder(),
    suffixIcon: Icon(Icons.lock_outline),
  ),
  validator: (value) {
    if (value!.isEmpty) {
      return 'Please enter some text';
    }
    return null;
  },
)

Expected results:

Actual results:

Hi @carlosfiori
Thanks for filing the issue!
Reproduced the issue using the following minimal code sample

minimal code sample
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    final appTitle = 'Form Validation Demo';

    return MaterialApp(
      title: appTitle,
      home: Scaffold(
        appBar: AppBar(
          title: Text(appTitle),
        ),
        body: MyCustomForm(),
      ),
    );
  }
}

// Create a Form widget.
class MyCustomForm extends StatefulWidget {
  @override
  MyCustomFormState createState() {
    return MyCustomFormState();
  }
}

// Create a corresponding State class.
// This class holds data related to the form.
class MyCustomFormState extends State<MyCustomForm> {
  // Create a global key that uniquely identifies the Form widget
  // and allows validation of the form.
  //
  // Note: This is a GlobalKey<FormState>,
  // not a GlobalKey<MyCustomFormState>.
  final _formKey = GlobalKey<FormState>();

  @override
  Widget build(BuildContext context) {
    // Build a Form widget using the _formKey created above.
    return Form(
      key: _formKey,
      child: Column(
        crossAxisAlignment: CrossAxisAlignment.start,
        children: <Widget>[
          TextFormField(

            decoration: const InputDecoration(
              labelText: "Password",
              border: OutlineInputBorder(),
              prefixIcon: Icon(Icons.lock_outline),
              suffixIcon: Icon(Icons.lock_outline,),
            ),

            validator: (value) {
              if (value!.isEmpty) {
                return 'Please enter some text';
              }
              return null;
            },
          ),
          Padding(
            padding: const EdgeInsets.symmetric(vertical: 16.0),
            child: ElevatedButton(
              onPressed: () {
                // Validate returns true if the form is valid, or false otherwise.
                if (_formKey.currentState!.validate()) {
                  // If the form is valid, display a snackbar. In the real world,
                  // you'd often call a server or save the information in a database.
                  ScaffoldMessenger.of(context)
                      .showSnackBar(SnackBar(content: Text('Processing Data')));
                }
              },
              child: Text('Submit'),
            ),
          ),
        ],
      ),
    );
  }
}

preview

Reproduced on the following channels

stable
master

✅
✅

Check flutter doctor -v outputs for each channel below

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.0, on macOS 11.3.1 20E241 darwin-x64, locale en-GB)
    • Flutter version 2.2.0 at /Users/tahatesser/Code/flutter_stable
    • Framework revision b22742018b (4 days ago), 2021-05-14 19:12:57 -0700
    • Engine revision a9d88a4d18
    • Dart version 2.13.0

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.5, Build version 12E262
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.2)
    • 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 11.0.8+10-b944.6916264)

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

[✓] Connected device (4 available)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E            • ios            • iOS 14.6
    • iPhone 12 (mobile)     • D5A130FB-A402-4C16-8F33-7806BCE32682 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)        • macos                                • darwin-x64     • macOS 11.3.1 20E241 darwin-x64
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 90.0.4430.212

• No issues found!

[✓] Flutter (Channel master, 2.3.0-13.0.pre.33, on macOS 11.3.1 20E241 darwin-x64, locale en-GB)
    • Flutter version 2.3.0-13.0.pre.33 at /Users/tahatesser/Code/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 84734ecc54 (7 hours ago), 2021-05-18 23:09:03 -0700
    • Engine revision 2398dea918
    • Dart version 2.14.0 (build 2.14.0-119.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.5, Build version 12E262
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.2)
    • 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 11.0.8+10-b944.6916264)

[!] Android Studio
    • Android Studio at /Volumes/Extreme/Android Studio Preview.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
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] IntelliJ IDEA Community Edition (version 2021.1.1)
    • IntelliJ at /Volumes/Extreme/IntelliJ IDEA CE.app
    • Flutter plugin version 55.1.5
    • Dart plugin version 211.7233

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

[✓] Connected device (4 available)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E            • ios            • iOS 14.6
    • iPhone 12 (mobile)     • D5A130FB-A402-4C16-8F33-7806BCE32682 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)        • macos                                • darwin-x64     • macOS 11.3.1 20E241 darwin-x64
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 90.0.4430.212

! Doctor found issues in 1 category.

✅ : Reproducible ❌: Not Reproducible

@bleroux Did you fix this recently or am I misremembering? I know you've recently made changes to this kind of theming.

@bleroux Did you fix this recently or am I misremembering? I know you've recently made changes to this kind of theming.

@justinmc I have not yet filed fixes for the prefix/suffix icons. One of the current PRs, I'm working on is the M3 tests migration for prefix/suffix. It will probably be ready after Google/IO.

FYI, when running the given code sample, the icon color turns to red for Material3 and it does not for Material2. I had a quick look to M2 and M3 spec and in both cases the icon color is supposed to turn red.
I will work on a fix.


[Observer] Avoid calling webhook twice if you send transaction to yourself

Fixed in https://github.com/TrustWallet/blockatlas/commit/ddfce6736b37be7c5fbb9a80d48e3bf8210399aa


NPM ReadME out of date

API docs for scoring a question set are out of date:
View Readme here: https://www.npmjs.com/package/blockscore
It says:
var data = {
    person_id: response.person_id,
    question_set_id: response.id,
    answers: [
      {
        question_id: 1,
        answer_id: 1

It should be
var data = {
    id: question_set.id,
    answers: [
      {
        question_id: 1,
        answer_id: 1

Thanks for letting us know, Kevin! I'll try to get this fixed.  In the meantime, our hosted docs are more regularly kept up to date.

@kingkevin I'm pretty sure it is updated now. However, npmjs.com is serving me different READMEs depending on the page load.  Can you confirm it is updated?


PyPI page still shows readme of old version

Hi @yamatt ,
Happy new year!
Thanks for maintaining this fork.
However it seems the PyPI page on https://pypi.org/project/homoglyphs_fork/ still shows the readme of the old original version as the project description, not loading your README.md contents specified by the pyproject.toml
Not sure quite what's causing that, but maybe the setup.py needs to be updated ?
I'm not familiar with pyproject.toml specification yet - it seems something is causing PyPI to load the description from the readme of the original version, even though the other pieces are correct.
p.s. do you think it would make sense to make this project an org, move it to homoglyphs/homoglyphs ?
That way there could be several maintainers added to the project.
There is also a forker at https://github.com/AnatolyTimakov/homoglyphs/commit/a70aa9ff975d149ec992815431cf4518ba6dfdd0 that added some content.
Thanks,

Hi @wesinator thanks for telling me.
I think PyPI didn't get updated as I had done things out of sequence and I updated the README after the GitHub Action had been run. I'll see if I can trigger a manual update but I remember it being a faff involving versioning which is why I didn't bother.
I would expect it to be updated once a new version of this fork gets published.
As for the Organisation I'm not sure owning the entire Homoglyphs group is entirely useful. I'm happy to add people to this repo as is warranted for now in the same one way the original project was run.
I'll definitely check out the other fork.

The description on PyPI is now updated because of the v2.1.0 release


Calling /productmodal throws an exception

The page /productmodal is indexed by search enginges an throws an exeption on calling directly. This has shown up on Google Search Console for me.
Is there a way that this redirects to home if this is called directly?

I've pushed up a fix for this:
https://github.com/concrete5-community-store/community_store/commit/da78aee48517e20e0464f98c57d4ccbe66d1ffc8

This works great! Many thanks!


Changes resources

Purpose: To improve the functionality and user experience of the resources section.
I worked on zenodo.py to ensure extraction of all necessary information, for this I also made some changes in objects.py.
I worked on the resources page such that it displays all the relevant details about a resource appearing in the results section. For this I edited the resources.html file and  main.py file.
I worked on the resources-details section to ensure that on clicking a particular resource, its details page is opened. For this, apart from main.py, zenodo.py and resource.html , I edited resources-details.html. Below files might give insights about the changes I made.

.vscode should not be pushsed to git
changes in .github/workflows/main.yml should be rolled back
what is a.py file? this seems to be some kind of test code file. shouldn't be pushed to git.
changes in sources/data_retriever.py should be rolled back
we no longer have dblp.py. we are maitaining separate file for each dblp endpoint such publications, or researchers. Make necessary changes in those source files.
same applies to openaire, openalex, and wikidata
roll back the changes in orcid.py
remove zen.py. should not be pushed to git.
roll back the changes in utils.py


Integrate MongoDB to Store User and Media Data

Description:
To manage users, images, and videos efficiently, we need to integrate MongoDB into our backend. This involves setting up schemas for users and media (images/videos), and modifying API endpoints to interact with the database.
Acceptance Criteria:

Users can create accounts where their information is securely stored in MongoDB.
Images and videos uploaded by users are associated with their account in the database.
API endpoints for listing/deleting images and videos retrieve/update data in MongoDB.

Tasks:

[x] Set up MongoDB connection in the backend.
[x] Define Mongoose schemas for users, images, and videos.
[x] Update API endpoints to interact with MongoDB.
[x] Test CRUD operations for users and media.

Changed to Supabase. Completed


Error executing "hpcr_contract_encrypted" on MAC

I ran nginx-hello terraform on Mac (Monterey) and it resulted in the below error. I then had to move over to using Ubuntu.
╷
│ Error: openSSL Version [LibreSSL 2.8.3] is unsupported
│
│   with hpcr_contract_encrypted.contract,
│   on terraform.tf line 132, in resource "hpcr_contract_encrypted" "contract":
│  132: resource "hpcr_contract_encrypted" "contract" {
│
Can we use a version of openSSL that is more widely used ?. Thanks.

Main reason for using the openssl binary is to allow using implementations that meet special security claims (such as FIPS compliance) which are not met via the underlying golang crypto APIs.
Unfortunately the default version of openssl on mac does not meet the requirements. There are two ways to get over this:

install an openssl version (not libressl) and configure it as defined here https://github.com/ibm-hyper-protect/terraform-provider-hpcr#prerequisite
as a fallback solution the provider will first determine of the configured version of openssl is acceptable. If not it will fallback to the golang crypto APIs instead of bailing out. So the provider will run on mac without changes to the system.
see https://github.com/ibm-hyper-protect/terraform-provider-hpcr/pull/8


Implemented Firebase and the avatar upload function

@trofej if you remove the EditProfile page, which I don't think we use anyway, then you should pass the quality gate here


"Berlești" is listed in the drop-down menu

Description:
"Berlești" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Berlești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Berlești, GORJ".


Alpha stage

Checklist

[ ] I have updated the changelog.txt, if applicable.
[ ] I have bumped the software version in _version.py, if applicable.
[ ] I have checked that the unit tests passed.
[ ] I have checked that Minnie and/or RoboMinnie passed.
[ ] I have resolved merge conflicts.

I think this PR is not needed anymore. Feel free to re-open


add domain mercatorum.it

University Mercatorum Italy.

@nbattista89 First stage of verification passed, official domain is unimercatorum.it as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@nbattista89 Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

Hi,
Below all informations that you need:
https://www.unimercatorum.it/
Piazza Mattei, 10, 00186 Rome RM Italy
Link of courses of degree (three or five years) :
https://www.unimercatorum.it/corsi-di-laurea
My course of degree (Big data) :
https://www.unimercatorum.it/corsi-di-laurea/statistica-e-big-data
My email of student university: @.***
Thanks,
Best regards
Nicola Battista
Il Gio 16 Feb 2023, 21:51 Philip Torchinsky @.***> ha
scritto:

@nbattista89 https://github.com/nbattista89 Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an
IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are
submitting as an official email domain for the students.

—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/16594#issuecomment-1433698963,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFQMYIDVZPFADBN2CIHOK53WX2HO3ANCNFSM6AAAAAAU43KJEU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@nbattista89 Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Replace snackbar with native alerts for signup for errors

Changes
When pressing "Sign Up" with an empty form:

Old Snackbar
New Native Alert

Why?
Google's material design (which I assume is where the snackbars are inspired from) only reccomends using snackbars for low priority messages. Instead, these signup form errors should be converted to native alerts (they are high priority as unless the user takes action the user cannot progress). Reason being it is easier for users to overlook snackbar style alerts, whereas, fullscreen-alerts prevent the user from doing anything until the alert is acknowledged.
See material design dialogs usage for more information. In particular the "When to use" and "Cross-platform adaptations" subsections.
Also the margins around the text on the old snackbar are inconsistent, which gives an unprofessional look.
Apply this change for other error messages?
If you like the spirit of this change I would be willing to go though and find other usages of the snackbar for error messages and replace with alerts where appropriate.

hey @ShadowGravity , thanks for the PR and congrats on the first contribution with us!  Would be awesome if we did that everywhere else too - wanna take a stab?
Can we show you some appreciation?


[docs] Change screenshot path pattern command line syntax to single quotes

\cc @AndreyBelym @kirovboris
closes #2952

what about using the path in .testcaferc.json?
"JSON standard does not allow single quoted strings"

@mornatan, you can use both types of quotes in the configuration file.
You are right, standard JSON supports only double quotes, but TestCafe configuration file uses JSON5, a superset of JSON. It adds support for single quotes.

Some boring technical TLDR:
The type of quotes is important in the command line only because bash (Linux & macOS command line interpreter) will try to substitute values for path pattern placeholders if you use double quotes instead of single quotes. Since the configuration file is not processed by your command line interpreter and based on the JSON5 standard, you can use any type of quotes in it.


style: alert log level

What type of this PR
Add one of the following kinds:
/style
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #your-issue_number
[Erda Cloud Issue Link](paste your link here)

Specified Reviewers:
/assign @sfwn @luobily
Need cherry-pick to release versions?

/approve


Added EXPOSE command

In reference to the issue #38

Thanks @NishkarshRaj I appreciate the compliment! It's one of the first issues I have ever contributed so it's definitely encouraging to hear good words. I will take a look in a few days and try to provide an advanced example as well. 👍 😄

Sure @Arnau-Ninerola 🙂
Take your time.
And if you need any help regarding open source contributions, feel free to contact me 😄


Sweep: add JWT docs

Please generate detailed docs in code for the jwt.cpp and jwt.hpp files, including all the structs, methods, attributes etc.

💎 Sweep Pro: I'm using GPT-4. You have unlimited GPT-4 tickets. (tracking ID: None)

[!TIP]
I can email you when I complete this pull request if you set up your email here!

Actions (click)

[ ] ↻ Restart Sweep

I am currently looking into this ticket! I will update the progress of the ticket in this comment. I am currently searching through your code, looking for relevant snippets.

Step 1: 🔎 Searching
I'm searching for relevant snippets in your repository. If this is your first time using Sweep, I'm indexing your repository. You can monitor the progress using the progress dashboard

🎉 Latest improvements to Sweep:

New dashboard launched for real-time tracking of Sweep issues, covering all stages from search to coding.
Integration of OpenAI's latest Assistant API for more efficient and reliable code planning and editing, improving speed by 3x.
Use the GitHub issues extension for creating Sweep issues directly from your editor.

💡 To recreate the pull request edit the issue title or description.
Something wrong? Let us know.


Allow defining package description and compat data in config file

// in jsr.json or deno.json
{
  "description": "...",
  "compat": {
    "browsers": true,
    "deno": true,
    "node": true,
    ...
  }
}

Looks like we have an issue about defining the compatibility data in jsr.json already https://github.com/jsr-io/jsr/issues/147

Closing as duplicate


Use classification training, boxing does not work

Current model PotHoleDetector does not work for proper identification of potholes

We need to aquire better hardware or use cloud computing since a local execution to classify images takes more than 8 hours in a intel i7 11th generation


Use ANONYMOUS flag to check whether a type is an anon type

Description:
$title. ref: [1].
Describe your problem(s) / solution(s)
There are some cases where we generate types/names but they don't have the anon prefix. So, we might need to use the ANONYMOUS flag for such types.
Related Issues (optional):
[1] https://github.com/ballerina-platform/ballerina-lang/pull/29792#discussion_r610338082

https://github.com/ballerina-platform/ballerina-lang/issues/28262

Fixed in 3da69b3 of https://github.com/ballerina-platform/ballerina-lang/pull/29792


Add Rpi zero support

Thanks

I believe the available armv7 release should be compatible - https://github.com/pocketbase/pocketbase/releases
(https://github.com/pocketbase/pocketbase/releases/download/v0.7.8/pocketbase_0.7.8_linux_armv7.zip).

no it doesn't work i tried it before i open an issue, says Illegal instruction
when i compile it for v6 everything works

Ah, I was confused by this spec - https://cdn.sparkfun.com/assets/learn_tutorials/6/7/6/PiZero_1.pdf, which ended up to be wrong per the comments in https://forums.raspberrypi.com/viewtopic.php?t=253166.
We can add a armv6 target to the, but I'm not sure how many users will use it and I'm afraid it may increase the build time during releases (currently the github release action takes ~ 12min to complete).
If there is a users demand for armv6 target I'll consider adding it to the gorelease config, but for now I'll leave it aside because it will increase the build time during releases and test runs (currently the github release action takes ~10min to complete after tagging).


Rule idea: overload-coverage

Promiscuous types and type unions in signatures are legacy JavaScript code-smell. TypeScript comes with an Expressive Type System that allows for expressions in type signatures. Any Haskell developer who has previously scoffed at the procedural nature of ES will feel right at home.
New rule
I propose introducing a rule that enforces good form by demanding a minimum overload coverage.

overload-coverage.

Example:
"overload-coverage": [
  true,
  {
    "require-returntype": true,
    "minimum-coverage": 0.8
  }
]

Options

minimum-coverage: Require at least this ratio of type signatures to be over,loads of another signature.
require-returntype: Require the return type to be different for a signature to count as different

This is horrific, I love it


The -p option (progressive) with jpg input files results in error and no image being put out, even when it is just on its own without any other options.

Describe the bug
The -p option (progressive) with jpg input files results in error and no image being put out, even when it is just on its own without any other options. But when input files are png images, then this option works fine.
To Reproduce
Steps to reproduce the behavior:
run command: .\cjxl.exe ......\Pictures\BingImageOfTheDay_20210523.jpg .\result5.jxl -p
Expected behavior
Image is put out properly.
Screenshots
this is the test image that i used:
Environment

OS: Windows 11 with all latest updates.
Compiler version: [e.g. clang 11.0.1]  This software was downloaded as an x64 binary from the releases of this github repo. I did not compile it.
CPU type: amd ryzen 5900hs x64 CPU.
cjxl/djxl version string: JPEG XL encoder v0.8.1 c27d499 [AVX2,SSE4,SSSE3,Unknown]
Additional context

.\cjxl.exe .\BingImageOfTheDay_20210523.jpg .\result5.jxl -p -v
The output (error?) message:
JPEG XL encoder v0.8.1 c27d499 [AVX2,SSE4,SSSE3,Unknown]
Note: Implicit-default for JPEG is lossless-transcoding. To silence this message, set --lossless_jpeg=(1|0).
Read JPEG image with 345077 bytes.
Encoding [Container | JPEG, lossless transcode, effort: 7 | JPEG reconstruction data],
D:\a\libjxl\libjxl\lib\jxl\enc_frame.cc:908: JXL_CHECK: enc_state_->passes. Size() == 1
Maybe related to the problem: there is no D: drive mounted on my pc, and the error still occurs even when the jpg is in the same dir as the cjxl.exe executable.
Kind regards.

Just for some info, the failed progressive on the jpeg is due to it trying to losslessly transcode it, which uses different techniques than normal and likely is what's breaking it.
The png getting massive and taking longer is because the jpeg transcode is almost instant and skips a lot of steps that normal encoding needs. If you try to compress it with only -d 0 -p then it should be smaller than the png at least, a lot of those options are redundant or for jpeg only.
If you want to progressively encode the jpegs before it's fixed, then try -p -d 0 -j 0  for lossless or -p -d 1 -j 0 for lossy, bear in mind the results will probably be worse than the transcode.
Best for the experts to chime in, but hopefully this explains and helps somewhat

This should be fixed with https://github.com/libjxl/libjxl/pull/2479
Feel free to reopen if the issue is still there.


WFLY-18647: Upgrade HAL to 3.6.16.Final

https://issues.redhat.com/browse/WFLY-18647

@hpehl This would be good for 30.x as well

@bstansberry PR for WildFly 30.x: https://github.com/wildfly/wildfly/pull/17327


NO-ISSUE: Bump JDT.LS to 1.27

@yesamer @tiagobento The full CI build is currently broken for several PRs. Based on the log, might be related to this PR. Could you check please? For example: https://github.com/kiegroup/kie-tools/actions/runs/5883339840/job/15958054262

@paulovmr The build was fixed... it was a problem @thiagoelg identified and fixed as part of https://github.com/kiegroup/kie-issues/issues/398


IE11 the selection is  invalid

Cann't it select words and then format them in IE ??

I am sorry,editor hadn't test in IE.I will optimize later.

hi, any news on this?

hi.
I've investigated a little. The problem seems to be in the
saveCurrentRange method. The condition
	            if (this.$els.content.contains(range.startContainer) && this.$els.content.contains(range.endContainer)) { 	                this.range = range; 	            }
is never true on IE11. Both the conditions in && stay always false.
Thus this.range = range is never executed.


[WIP] New optional --remote flag for 'istioctl version'

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

Needs approval from an approver in each of these files:

istioctl/OWNERS [esnible]

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


feature Support SELECT * REPLACE clause

Proposed changes
Issue Number: close #36777

run buildall

run buildall

@morrySnow please review again, thanks

run buildall

Hi @morrySnow, do you have any time to review my code?


spdk: nvme-cli should be equal to or greater than 1.12

go-spdk-helper can support nvme-cli v2.0+.
Longhorn/longhorn#5744

@mergify backport v1.5.x


Unable to add several hosts to a network

Hi,
i am unable to add several hosts to a network through the GUI;
I click on a "domain" which then select all the hosts int his domain.
I click then new, enter the name of the network then click save.
At this time only one host is selected and added to the network. It is then impossible to add a new host.
The workaround is to manually add it directly using mongoDB interface  which is efficient but not recommended i suppose.

While creating a new network, if you select one host it will show you all the hosts in the same domain allowing you to add more hosts from that same domain. Previously, that first host you selected was highlighted with a different color. However, now in the latest version that is not happening.
Anyways, if you ignore the highlighting issue and just click the rest of the hosts you want to add BEFORE saving your new network, those hosts would be added properly but again the highlight will not occur...

It turns out that some minor changes to the project had unexpected impacts on certain things, the highlighting of hosts during network instantiation being one of them. I've created a patch that's currently waiting to be pulled in (https://github.com/mitre/adversary/pull/47), that should fix this issue by restoring the highlighting.

@p57p57 @EnJohnGee If the issue still persists after updating to the most recent version of Caldera and its plugins, feel free to reopen this ticket or create a new one. Otherwise, it appears #47 fixed the root issue here.


should balance the game

In my opinion it should be more difficult from the beginning

Maybe we should lower the extra time added, it's pretty easy keep the bar full during all the game


[YouTube] JSON Videos + Subtitles

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 2016.12.09. 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 2016.12.09

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?

[ ] 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)
[x] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
$ youtube-dl -v <your command line>
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2016.12.09
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Description of your issue, suggested solution and other information
How to get a list of all the media files and subtitles in one JSON response?

You can get subtitle URLs via this hack:
$ youtube-dl -j "https://www.youtube.com/watch?v=RgKAFK5djSk" --write-sub | jq .subtitles
{
  "und": [
    {
      "url": "https://www.youtube.com/api/timedtext?lang=und&v=RgKAFK5djSk&name=&fmt=ttml",
      "ext": "ttml"
    },
    {
      "url": "https://www.youtube.com/api/timedtext?lang=und&v=RgKAFK5djSk&name=&fmt=vtt",
      "ext": "vtt"
    }
  ],
  "en": [
    {
      "url": "https://www.youtube.com/api/timedtext?lang=en&v=RgKAFK5djSk&name=&fmt=ttml",
      "ext": "ttml"
    },
    {
      "url": "https://www.youtube.com/api/timedtext?lang=en&v=RgKAFK5djSk&name=&fmt=vtt",
      "ext": "vtt"
    }
  ]
}

Thank you. My problem was that the subs were downloaded because I was using "--print-json" and I should use only "-j".


Re-deploy the site so that Date() in Footer shows 2023

Hi,
I was going through Powerloom for Blockchain Engineer and/or Frontend-related roles and found the project and vision of the company quite interesting! This made me check out the website and some documentation at the basic level. I thought of adding this as a Discussion instead of an issue, but Github didn't allow it for this repo for some reason.
I found out that the copyright year at the footer of the docs site showed 2022, and thought of updating it via a PR, only to find out that the code was correct, but the re-deployment hasn't been done in 2023.
Also, to further optimize the powerloom.io website, we can also use AVIF instead of PNGs and also resize the images (and crop; e.g., divergence ventures logo) for even further optimization!
Let me know if I can assist you guys with anything regarding the above or for any other issue related to Frontend, Solidity, and JavaScript in general! I am relatively a beginner in Blockchain/Solidity but I have done open-source contributions to React, Hardhat, Wagmi, Preact, Lenster, and many other impactful projects!
Have a great weekend!

Thanks for flagging this. We're about to push some changes to the docs and hence the hold up on deployment.
Wrt powerloom.io, will take a note. That repo needs some cleanup before going public!
As for contributing, you are welcome to contribute whatever way you can. We just started open sourcing all our code yesterday, watch out for a call to action next week. Meanwhile, you can start tinkering around with the code in other repos :)

Congratulations on open-sourcing your code! Will be waiting for CTA next week as well, till then let me see if I can provide any assistance to your codebase 👨🏻💻
Closing this issue now 😄


Not working anymore?

Hi,
I think about developing some route planning tool for location based games. For example for Ingress. At a first step I want to try out the browser plugin IITC. Maybe my Idea could become a plugin.
But after I installed Greasmonkey on Firefox and hit the download button on https://iitc.me/desktop/ the intel site isn't working anymore. It looks like the old one but doesn't loads a map.

Try changing the map later to the Default Ingress Google Map, I've had an issue where the default map for IITC doesn't load for me

https://intel.ingress.com/intel

Yes, exactly that site. If I click on the link I come to a site that looks like the old site but does not load anything. I don't thnk I can change there any settings. If I deactivate the script it is working again. (The standrd version.)

Ah, got it. Simply leave this site and continue here: https://github.com/IITC-CE/ingress-intel-total-conversion

You may have to edit the script to include the intel.ingress.com url, I don't know what version you have

The one from https://iitc.me/desktop/.


weird test failures on s390x-unknown-linux-gnu with rustc + llvm 13 only

For context: I'm the primary package maintainer for Rust crates in Fedora Linux. We're running test suites for all crates, where possible, in an effort to improve the quality of the packages that we ship.
I came across a very weird test failure in concurrent-queue 1.2.3 and 1.2.4:
running 10 tests
test close ... ok
test capacity ... ok
test len ... ok
test len_empty_full ... ok
test linearizable ... ok
test mpmc ... FAILED
test smoke ... ok
test drops ... ok
test zero_capacity - should panic ... ok
test spsc has been running for over 60 seconds

And then the test runner is stuck forever.
This problem with the mpmc and spsc tests is limited to s390x, and only when the crate is compiled with Rust on LLVM 13 (such as on Fedora 35, where LLVM 14 is not available).
I cannot determine for certain which change might have started to cause this problem, because we don't include s390x in our CI, due to limited builder capacity, but I can provide these data points:
Last known successful build:

concurrent-queue 1.2.2
rust version: 1.58.1 built against llvm 13
build host: Fedora 35 running Linux 5.15.6

First known unsuccessful build:

concurrent-queue 1.2.3
rust version: 1.62.0 built against llvm 13
build host: Fedora 36 running Linux 5.18.9

But this is also successful:

concurrent-queue 1.2.3
rust version: 1.62.0 built against llvm 14
build host: Fedora 36 running Linux 5.18.9

concurrent-queue 1.2.3

1.2.3 has been yanked due to a bug: https://github.com/smol-rs/concurrent-queue/blob/master/CHANGELOG.md#version-123
Please try with 1.2.4.

Oh, sorry, you already said 1.2.4 has the same issue.

I came across a very weird test failure in concurrent-queue 1.2.3 and 1.2.4:

I cannot find any changes between 1.2.2 and 1.2.4 that might affect s390x or big-endian targets: https://github.com/smol-rs/concurrent-queue/compare/v1.2.2...v1.2.4
So, the rustc version or the distro version may be relevant. Actually, the rustc 1.60+ changes its behavior on s390x depending on the LLVM version: https://github.com/rust-lang/rust/pull/94764/files#diff-9f3bfd2fdc228dcb1b352082a7ebc39b19c8bea15b73141f72d9f5021aa3c66c

rust version: 1.58.1 built against llvm 13
build host: Fedora 35 running Linux 5.15.6

rust version: 1.62.0 built against llvm 13
build host: Fedora 36 running Linux 5.18.9

So, the rustc version or the distro version may be relevant. Actually, the rustc 1.60+ changes its behavior on s390x depending on the LLVM version

Right, that could be a possible culprit :( I'll try to follow up with Red Hat Rust / LLVM maitnainers.

Closing as an upstream bug per https://github.com/smol-rs/concurrent-queue/issues/21#issuecomment-1197570359.


Validate Wrapper only on push to master

Updating the Validate Gradle Wrapper action to make sure it runs only once for a pull request.
The list of active checks is already long and this action was fired twice.

Codecov Report

Merging #2548 into master will not change coverage by %.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #2548   +/-   ##
=========================================
  Coverage     80.85%   80.85%
  Complexity     2265     2265
=========================================
  Files           376      376
  Lines          7355     7355
  Branches       1206     1206
=========================================
  Hits           5947     5947
  Misses          809      809
  Partials        599      599

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 be4a0fd...8212dd4. Read the comment docs.


WIP: Linux litex auto generated

This PR adds a linux litex auto-generated test to the set of existing litex tests for xc7.
It generated the devicetree and emulator binary.
It's WIP as we still need to build the kernel and rootfs (buildroot)

Travis CI is red on python formatting:
yapf needs to reformat /home/travis/build/SymbiFlow/symbiflow-arch-defs/xc/xc7/tests/soc/litex/generate_linux.py
yapf needs to reformat /home/travis/build/SymbiFlow/symbiflow-arch-defs/xc/xc7/tests/soc/litex/linux/soc_linux.py
make[3]: *** [CMakeFiles/check_python.dir/build.make:76: CMakeFiles/check_python] Error 2

@litghost @acomodi I reran the CI several times, but seems that the fail is infrastructure related?


New rstack release?

The cfg-if dependency was bumped here but no new release was made. Could one be created?

Done!

Done!

That was fast, thanks!


mailtivarite time series classification for example mixture of categorical and continues data

will it work for
mailtivarite time series classification for example mixture of categorical and continues data
for example at time t1 we have observation: red, 2.4 , 5, 12.456 and time t2: green, 3.5, 2, 45.78; time t3: black, 5.6, 7, 23.56; t4: red, 2.1, 5, 12.6 ?

Hello @Sandy4321
I didn't know multivariate time series yet (I guess mailtivarite is just a typo). Your example is in my eyes no problem if you are able to define a distance measure between

red, 2.4 , 5, 12.456
green, 3.5, 2, 45.78

for example. Did that hel you?
Best Regards
Gordon


New TLDs as email

I would like to use<EMAIL_ADDRESS>as our e-mail, but time off says, it is not a valid e-mail. So at least .digital should be added as a valid email, or all new TLDs could be accepted.

@saschafoerster The current email validator is doing some naive regex testing rather than actually checking against a list of canonical TLDs.
The pull request I've submitted simply adjusts the regex to allow all the currently valid TLDs to be used, so you should apply that commit if you are urgently in need of using emails like<EMAIL_ADDRESS>

Great, thanks a lot!


[Enum Request] Table 99000880 "Order Promising Line"

Hi Team,
In table 99000880 "Order Promising Line" the field 20 "Source Type" is still an Option field. To be able to include own tables to the order promising functionality it is needed to change it to an Enum.
Thanks in advance!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Uncaught promise error in 1.1.0

Hi,
Firstly, I want to thank you for your library, it adds a nice touch to our application.
As I am a developer, it's sometime nice when we know how our creation is used, so I've attached a partial screenshot to show to you our use case.
Secondly, I encouters a little issue since I've upgrade progressbar.js from 1.0.1 to 1.1.0.
In my browser console, there is sometime one or multiple uncaught promise errors:
Uncaught (in promise) {offset:<PHONE_NUMBER>019968}
Promise.then (async)
animate	@	progressbar.js:1183
animate	@	progressbar.js:1406

It does'nt seem to affect the UI. Unfortunatly, I have not been able to find exactly why it occurs.
By by looking into the diffs from 1.0.1 to 1.1.0,
I have been able to find that the error may be caused since the upgrade of the shifty dependency from the Pull Request #196 (commit).
By adding an empty catch function to R118-R121, the promise error is handled and it seem to not affect the UI. But, I'm not sure that it is the right way to fix this problem...
.catch(function(state) {})

I hope it helps you, thank you very much!

Thanks for reporting. Let's see if this is related to https://github.com/kimmobrunfeldt/progressbar.js/issues/256

Will you please just catch it until you'll figure it out? its really hazardous..

Same problem here!

Thanks for the fix. Here the command for installing from the commit:
npm install --save "git://github.com/Zellerich/progressbar.js.git#0978bd7a45c1e1fcc8cf9b0890caddb5362941f6

Can you test the latest progressbar.js from latest master?
I think this should work similarly as @dmarman commented before:
npm install --save "git://github.com/kimmobrunfeldt/progressbar.js.git#master

The root cause of the problem is that animate calls shifty tween multiple times before the previous animation is completed. Shifty expects the client to never do that, but progressbar.js doesn't put any protection in place. Part of the reason is because the error message and documentation aren't clear why that is happening (improvement here: https://github.com/jeremyckahn/shifty/pull/115)
The solution is to make sure to .stop(true) the animation between animating again.

FYI, I have incorporated @wparad's jeremyckahn/shifty#115 fix into the latest v.2.9.0 release.
Thank you @wparad!

Is there any timeframe for having this issue solved?

FYI, I have incorporated @wparad's jeremyckahn/shifty#115 fix into the latest v.2.9.0 release.
Thank you @wparad!

Now I'm having the stop() executed while tween isPlaying. error.
Is this repository abandoned?

Just to get some clarity on the previous two comments, were they directed at @kimmobrunfeldt or myself?

Just to get some clarity on the previous two comments, were they directed at @kimmobrunfeldt or myself?

Mine was directed to progressbar.js so it's @kimmobrunfeldt

As a temporary workaround, you can set option duration for animate() to almost zero time


Raspberry pi 3 B+ boot problem

Prebuilty zipped disk image is not work on Raspberry pi 3 B+. Flashes red light and wont boot.

wow, you already have one. I'm working on updating the disk image and will post a script here.. https://github.com/wroscoe/donkey/blob/wr/disk_img_script/install/make_pi_disk_img.sh

I was waiting for this device and got one immediately. Thanks for fast
response.
On Sat, 24 Mar 2018, 23:13 Will Roscoe<EMAIL_ADDRESS>wrote:

wow, you already have one. I'm working on updating the disk image and will
post a script here..
https://github.com/wroscoe/donkey/blob/wr/disk_img_script/install/make_pi_disk_img.sh
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/wroscoe/donkey/issues/230#issuecomment-375921576, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AjHzm_I1gUHQTAgnJpzfo1g_IQ2gWdFvks5thqjVgaJpZM4S54hY
.

hello again i have been getting this error "ImportError: numpy.core.multiarray failed to import", while using "donkey calibrate --channel 0" this line. i guess something wrong about numpy. i deleted and reinstalled numpy but no solution.

https://stackoverflow.com/questions/20518632/importerror-numpy-core-multiarray-failed-to-import
try numpy 1.8?

https://stackoverflow.com/questions/17709641/valueerror-numpy-dtype-has-the-wrong-size-try-recompiling
Answer of RexFuzzle works for me. thanks

Now calibrate is working, my motor and servo are calibrated. however new error while "python manage.py drive" :
tensorflow.python.framework.errors_impl.NotFoundError: /home/pi/env/lib/python3.5/site-packages/tensorflow/contrib/data/python/ops/../../_prefetching_ops.so: undefined symbol: _ZN6google8protobuf8internal26fixed_address_empty_stringB5cxx11E

It would be helpful to see where in donkey that was getting called. On my Pi B the drive script is working. Haven't tried autopilots yet with this TF 1.5.

tensorflow 1.4.1 works well. thanks

Hey guys completely new to Raspberry pi's and donkeycars and im glad to have seen this issue posted. I am having the same issue where the prebuilt zipped disk image is not booting with our Raspberry pi so it is blinking red. Power supply is goood, Wroscoe where can i find a newer/updated version of this file? Or how can i do this properly? I am a complete noob!! Thank you

I am having the same issue.

Just to say I was getting a similar but slightly different error:
tensorflow.python.framework.errors_impl.NotFoundError: /home/pi/env/lib/python3.5/site-packages/tensorflow/contrib/data/python/ops/../../_dataset_ops.so: undefined symbol: _ZN6google8protobuf8internal9ArenaImpl10AddCleanupEPvPFvS3_E
Following onukadri's suggestion of uninstalling tensorflow (I was on 1.7.0) and installing tesorflow 1.4.1 fixed it.
pip uninstall tensorflow
tf_file=tensorflow-1.4.1-cp35-none-linux_armv7l.whl
wget https://github.com/lhelontra/tensorflow-on-arm/releases/download/v1.4.1/${tf_file}
pip install ${tf_file}
rm ${tf_file}

I wasn't using an image, but instead installed by using this script as a guide:
https://github.com/wroscoe/donkey/blob/wr/disk_img_script/install/make_pi_disk_img.sh

I used that script too. Almost ready to test, we'll see if it works.... I installed tensorflow 1.7 I think, on raspbian stretch, so I don't know if that will work.

I’m on stretch lite if that helps
On Tue, 22 May 2018 at 03:58, Benjamin<EMAIL_ADDRESS>wrote:

I used that script too. Almost ready to test, we'll see if it works.... I
installed tensorflow 1.7 I think, on raspbian stretch, so I don't know if
that will work.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/wroscoe/donkey/issues/230#issuecomment-390847306, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAe_Y-z-gyEQrC6bgwQtPufyXk7aLOkOks5t0368gaJpZM4S54hY
.

--
Meurig

It worked! I successfully installed opencv 3.4.1, tensorflow 1.4.1, and donkey on raspbian stretch.

I've taken opencv out of the image. We're going pure tensor! :)  Here is the new install script... https://github.com/wroscoe/donkey/blob/dev/install/pi/install.sh


WEBAPP-311: Add simple rating functionality

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword WEBAPP.
I've added two buttons two the categories toolbar (happy smiley and sad smiley), which are posting a feedback if clicked.
Also I moved some files in the layout module to a separate header/footer/toolbar package.

Most of the changes are just because I moved some files in the layout module to a separate header/footer/toolbar package.

Wrong branch prefix: WEBAPP-311 instead of 311 :)


fsGroup addition to security context for v3.10

Fixes errors related to reading serviceaccount token by adding fsGroup for the non-root user. This issue is described here kubernetes/kubernetes#82573
2019-10-10 17:34:52.521 [ERROR][6] daemon.go 227: Failed to connect to datastore error=open /var/run/secrets/kubernetes.io/serviceaccount/token: permission denied
Run typha in fsGroup to allow access to serviceaccount token

Deploy preview for calico ready!
Built with commit bb7c283a3de28f4b7132d849734259cb83f73523
https://deploy-preview-2928--calico.netlify.com


HIVE-25919: ClassCastException when pushing boolean column predicate in HBaseStorageHandler

What changes were proposed in this pull request?
The predicate analyzer can only handle expressions of the form:

col comp_op constant
constant comp_op col
and these are always ExprNodeGenericFuncDesc.

The new if clause ensures that we are never going to handle anything that is not supported.
Why are the changes needed?
Avoids the ClassCastException during predicate pushdown.
Does this PR introduce any user-facing change?
No apart from solving the problem.
How was this patch tested?
mvn test -Dtest=TestHBaseCliDriver -Dqfile="hbase_ppd_boolean_cols.q"

+1, very neat, @zabetak


docs: [IAC-2094]: IaCM Security documentation

Description

Add new IaCM Security doc with interactive diagram
JIRA

Screenshots
https://github.com/harness/developer-hub/assets/167115273/d263986a-9bec-4a25-89ba-606b6eb9fe66
PRs must meet these requirements to be merged:

[x] Successful preview build.
[x] Code owner review.
[x] No merge conflicts.
[x] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6675824c3bf94c84fa4ac863--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://667582b7769ada817158e299--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66793f0583621c6565373a30--harness-developer.netlify.app


Improve Dropout guide

What does this PR do?
WIP

Codecov Report

Merging #2846 (35fe82d) into main (1d669bc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2846   +/-   ##
=======================================
  Coverage   81.76%   81.76%
=======================================
  Files          55       55
  Lines        5906     5906
=======================================
  Hits         4829     4829
  Misses       1077     1077

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


devicetree: replace DT_HAS_DRV_INST with DT_INST_FOREACH

Make drivers multi-instance wherever possible using DT_INST_FOREACH.
This allows removing DT_HAS_DRV_INST in favor of making drivers just
do the right thing regardless of how many instances there are.
There are a few exceptions:

some SoC drivers with ifdefs inside code for each enabled instance.
These could be converted to IF_ENABLED() or COND_CODE_1() eventually
to use DT_INST_FOREACH. For the sake of expediency, just inline the
DT_HAS_DRV_INST expansion for now.

SoC drivers which are explicitly single-instance. Again for the sake
of expediency, drop a BUILD_ASSERT in those cases to make sure the
assumption that all supported SoCs have at most one available
instance is valid, failing fast otherwise.

I've fixed the designware DMA driver bugs discussed on slack. According to @galak , @tbursztyka  agrees the devices weren't being instantiated correctly. I assume that means that it's safe to fix them and use DT_INST_FOREACH.

WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon

Do we care? I don't personally, but I can use the workaround people have been doing where they end the instantiation macro in a function instead.

@henrikbrixandersen curious about the kinetis temperature driver only supporting a single instance, but nxp_ke1xf.dtsi having 3.  Is that correct?

WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon

Do we care? I don't personally, but I can use the workaround people have been doing where they end the instantiation macro in a function instead.

Does it break if you remove the ;.  At least for the ones that call DEVICE_AND_API_INIT() you should be able to safely remove it.

At least for the ones that call DEVICE_AND_API_INIT() you should be able to safely remove it.

DEVICE_AND_API_INIT needs a trailing semicolon

LGTM. We should probably open a issue to cleanup uses of DT_HAS_NODE_STATUS_OKAY(DT_DRV_INST(x)).

Will do after this gets merged.

@henrikbrixandersen curious about the kinetis temperature driver only supporting a single instance, but nxp_ke1xf.dtsi having 3. Is that correct?

@galak Essentially, they all access the same, internal temperature sensor. The temperature can be read using either of the available ADCs, so all ADCs have their own (disabled) temperature sensor instance. It is up to the user of the temperature sensor to decide, which one to use (have the application share the ADC with the temperature sensor to reduce code size, use a dedicated ADC for reading temperature to avoid interfering with application code, ...).
I am okay with changing the driver to allow for multiple, enabled instances although from a user point of view it doesn't make much sense.

I am okay with changing the driver to allow for multiple, enabled instances although from a user point of view it doesn't make much sense.

Please let me know if there's something nonsensical here and I'll put it back.

@galak Essentially, they all access the same, internal temperature sensor. The temperature can be read using either of the available ADCs, so all ADCs have their own (disabled) temperature sensor instance. It is up to the user of the temperature sensor to decide, which one to use (have the application share the ADC with the temperature sensor to reduce code size, use a dedicated ADC for reading temperature to avoid interfering with application code, ...).
I am okay with changing the driver to allow for multiple, enabled instances although from a user point of view it doesn't make much sense.

mostly curiosity on my part.

Ok w/the checkpatch errors, going to merge w/override.


Add a raw socket API

It would be nice to implement protocols like ICMP and DHCP on top of NIO, in order to do this NIO needs to provide some sort of raw socket API.

Yup, this would be a nice feature.

Implemented in https://github.com/apple/swift-nio/pull/2320


OTA example issues due to callback defined inside loop?

Hiya folks!
The OTA examples seem to work fine for me, but when adapting them to larger programs, I kept having a series of issues.
I found that the const OTA_Update_Callback callback was being declared in loop() (https://github.com/thingsboard/thingsboard-client-sdk/blob/22265f5b2d1d336af16de149e81ec743ce640d5f/examples/0011-esp8266_esp32_subscribe_OTA_MQTT/0011-esp8266_esp32_subscribe_OTA_MQTT.ino#L338).  I think maybe it should be in a larger scope, so that when loop() exits and restarts the pointers are still good.  When I changed this in my larger programs, it seemed to instantly start to work reliably.
It might even be a cause of some of the issues like https://github.com/thingsboard/thingsboard-client-sdk/issues/183.

Aformentioned issue is fixed in the linked Pull Request and has been fixed already because of #159.


Really simple force baim

Force baim if health is < X

feel like this is unneeded atm, but works and does improve the (extremely limited) ragebot we have rn, so i'll merge


Merge landingpage redesign

This PR is for merging in all the recent changes to the Cal-Adapt home page, the new grants page and the new blog posts. This branch has been deployed to https://beta.cal-adapt.org/ for review

 Add Grants block

 Add Grants block

 Update home page


Upgrade highlight.js

I get a security warning on my app because vue-markdown requires 9.X for highlight.js but the vulnerability is only fixed in 10.4.1
See below

I have updated the dependencies and you can install using
npm i<EMAIL_ADDRESS>Also, I am planning to maintain this package and have already merged most of the PR in my repository. Any contribution is welcomed to new repository.
Please check https://github.com/adapttive/vue-markdown/
master: current version (with few fixes)
next: updated dependencies and few features added.
Use:
npm install @adapttive/vue-markdown
https://www.npmjs.com/package/@adapttive/vue-markdown


Suggestion to add docs/ and sandbox_venv*/ in .gitignore

docs/ is the default path where docs are storer when running app locally
sandbox_venv*/ are created when installing python

Thanks @yohanboniface !


Travis CI: Lint Python code for syntax errors and undefined names

15 files with Python syntax errors https://travis-ci.org/intel/mavlink-router/builds/587837498#L334

Sorry for the long time. We switched to Github Actions, so this needs an update


kade engine not open

when i build the game, it not open the game. When i try to open the exe file, it open the windows, loading to 20/21 and close the windows
can someone help me
https://user-images.githubusercontent.com/87105792/136309652-b2d1b7fe-fc47-4d47-bd38-508547a560a0.mp4
?

oh okay


Grafana-operator multi-namespace support from OLM via ENV VAR instead of container args

Is your feature request related to a problem? Please describe.
Right now,  to have grafana-operator watching from multiple namespace, as per docs at https://github.com/integr8ly/grafana-operator/blob/master/documentation/multi_namespace_support.md#watching-for-dashboards-in-all-namespaces, you need to:

Add extra ClusterRole
Modify container args with --scan-all

For the required extra cluster permissions, we are already managing them manually (although would be great to come by default with OLM).
But for the required containers args, we don't want to manage manually a deployment that is already being managed by OLM.
When you use OLM to install an operator with a Subscription custom resource, you can modify the operator deployment resource by adding extra ENV VARs, updating the resources requests/limits... but you cannot update the container args.
Subscription Custom Resource: https://github.com/operator-framework/operator-lifecycle-manager/blob/master/deploy/upstream/manifests/0.17.0/0000_50_olm_00-subscriptions.crd.yaml
Example of tunning etcd operator deployment with the Subscription CR:
apiVersion: operators.coreos.com/v1alpha1
kind: Subscription
metadata:
  name: etcd-config-test
  namespace: openshift-operators
spec:
  config:
    env:
    - name: HTTP_PROXY
      value: test_http
    - name: HTTPS_PROXY
      value: test_https
    - name: NO_PROXY
      value: test
  channel: clusterwide-alpha
  installPlanApproval: Automatic
  name: etcd
  source: community-operators
  sourceNamespace: openshift-marketplace
  startingCSV: etcdoperator.v0.9.4-clusterwide

(If applicable)If your feature request solves a bug please provide a link to the community issue
Describe the solution you'd like
It would be nice to be able change the operator behaviour with ENV VARs instead of container args, so that way you can
for example enabling that functionality of multi namespace support without manually managing a deployment already managed by OLM because it is needed to specify a container arg --scan-all.
Describe alternatives you've considered
N/A
Additional context
N/A
Existing solutions

Right now we were re-packaging the operator on our own OLM catalogue, by updating the CSV installModes MultiNamespace and MultiNamespace from false to true, but this method is very manual and does not scale.
In addition, it seems there is an alternative way of having multi-namespace support  without modifying container args :

Default OLM installMode on single namespace
Adding extra ClusterRole manually
Not modifying the container args
Using Grafana CR field dashboardNamespaceSelector --> I see this field does not exist on Grafana CR

We have exactly the same problem with the multi/namespace support (v 3.9.0). The Operator seems to be able to watch dashboards only from the namespace where it runs.
Perhaps this should be reported as "bug"

Maybe it could be implemented a new ENV VAR called EXTRA_ARGS (which I'd say is pretty common on containers), so that way, it could be added any required extra args via ENV VAR instead of modifying the container args directly? that way it would work for both manual or OLM operator deployment

Ideally we would only have a CR field to enable multi namespace support (e.g. dashboardNamespaceSelector). Unfortunately that won't work because the Operator needs to know on startup which namespaces to watch (and not then the CR is created). The ENV VAR solution proposed by @slopezz seems workable. Can you pass those env vars also when installing from OLM? As far as i know it does not allow you to edit the subscription. I think those two solutions could work:

When installing from OLM, default to multi namespace mode: the --scan-all flag is set in the deployment and we add the required permissions to the CSV.
Remove --scan-all and use the env var approach suggested here. Will probably not work for installations from operatorhub.io, but should work for custom subscriptions.

Please let me know what you would prefer.

In order to finetune an operator deployment, you need to use the Subscription resource, where you can set some envvars that will be propagated to the operator deployment.
If the --scan-all or the --namespaces variables were exposed as envvars we could fine tune them using the subscription, so that would work for us (although we may still require to manually add some extra permissions).
Nevertheless, if the goal is to make it completely available from OLM without requiring any extra permission and making it easier to use by any user, there are other possibilities.
Maybe instead of having those --scan-all and --namespaces custom flags, @pb82 have you consider the possibility of using official operator-sdk WATCH_NAMESPACE envvar (the one which decides to watch a single namespace, multi namespace, o watching all namespaces)?

Affecting behaviour of the Operator from the Subscription YAML would be geat, i think. Only because I have to edit the CSV bei adding  a single line args: ["--scan-all"] I had to Change the Approval to "Manual" to be able to repeat this manual Change every time a new Operator Release is out.
Should not one one in the long-term be able to customize Opertaor Config from some sort of ConfigMap supplying /etc/grafana-operator/config.yaml  or similar? Then we would not be limited by the Syntay of Subscription
Objects ..

This is a very important feature to be able to run in clusterScope via OLM. We have a central grafana and manage all dashboards in teams with grafana orgs, so they should be able to create they own dashboards in their namespaces.


lgogdownloader segfault during download

There are repeatable core dumps when using lgogdownloader from git head to download darksiders.
Previously I had also seen that with hotline miami, but not investigated further. All other games seem to download fine.
A build with -DDEBUG -g -O0 run in gdb ends with:
DEBUG INFO (API::getInstallerLink)
{
        "file" :
        {
                "available" : 1,
                "link" : "http://wpc.11DF.edgecastcdn.net/0011DF/secure/darksiders_2/win/setup_darksiders2_<IP_ADDRESS>.exe?SOME_HASH_DATA_DELETED&fileExtForIe=.exe",
                "message" : "",
                "name" : "Windows Installer, English",
                "type" : "installer"
        },
        "result" : "ok",
        "timestamp" :<PHONE_NUMBER>
}
Repairing file ./darksiders_ii/setup_darksiders2_<IP_ADDRESS>.exe
XML: Using remote file
XML: Valid XML
XML: Parsing finished

setup_darksiders2_<IP_ADDRESS>.exe
        MD5:    56b59c909e8424657b8e9d85bccb5a93
        Chunks: 4
        Size:   34964608 bytes

File doesn't exist ./darksiders_ii/setup_darksiders2_<IP_ADDRESS>.exe
Downloading: ./darksiders_ii/setup_darksiders2_<IP_ADDRESS>.exe
 54% ▕█████████████▉            ▏ 17.92/33.34MB @ 2.25MB/s ETA: 6s
Program received signal SIGSEGV, Segmentation fault.
[Switching to LWP 1]
0x0000784f5aa26249 in addbyter () from /usr/pkg/lib/libcurl.so.4
(gdb) bt
#0  0x0000784f5aa26249 in addbyter () from /usr/pkg/lib/libcurl.so.4
#1  0x0000784f5aa265f5 in dprintf_formatf () from /usr/pkg/lib/libcurl.so.4
#2  0x0000784f5aa2796c in curl_mvsnprintf () from /usr/pkg/lib/libcurl.so.4
#3  0x0000784f5aa1516d in Curl_failf () from /usr/pkg/lib/libcurl.so.4
#4  0x0000784f5aa0a9af in Curl_resolv_timeout () from /usr/pkg/lib/libcurl.so.4
#5  0x0000000000000000 in ?? ()
(gdb)

I'm using curl-7.51.0. There are no other threads active. This looks like name resolution error handling, but this doesn't really make sense to me, because why would there be name resolution after the download already started?

I just tried again with hotline miami, and it still dumps core for me.
Backtrace is a bit longer but not much more helpful:
(gdb) thread apply all bt

Thread 1 (LWP 1):
#0  0x0000783d39226249 in addbyter () from /usr/pkg/lib/libcurl.so.4
#1  0x0000783d392265f5 in dprintf_formatf () from /usr/pkg/lib/libcurl.so.4
#2  0x0000783d3922796c in curl_mvsnprintf () from /usr/pkg/lib/libcurl.so.4
#3  0x0000783d3921516d in Curl_failf () from /usr/pkg/lib/libcurl.so.4
#4  0x0000783d3920a9af in Curl_resolv_timeout () from /usr/pkg/lib/libcurl.so.4
#5  0x0000000000000018 in ?? ()
#6  0x00007f7fff952d10 in ?? ()
#7  0x0000000000000002 in ?? ()
#8  0x0000000000003032 in ?? ()
#9  0x0000783d368bc4df in malloc (size=132204319001632) at /usr/src/lib/libc/stdlib/jemalloc.c:3795
#10 0x00007f7fff952da9 in ?? ()
#11 0x00007f7fff952da9 in ?? ()
#12 0x00007f7fff952da0 in ?? ()
#13 0x00007f7fff952da0 in ?? ()
#14 0x00007f7fff952daf in ?? ()
#15 0x0000783d3778d200 in ?? () from /usr/lib/libstdc++.so.7
#16 0x0000000000000018 in ?? ()
#17 0x00007f7fff952da0 in ?? ()
#18 0x0000000000000009 in ?? ()
#19 0x3531303630313032 in ?? ()
#20 0x0000783d36700033 in ?? ()
#21 0x0000783d37784028 in vtable for std::ios_base () from /usr/lib/libstdc++.so.7
#22 0x0000000000000006 in ?? ()
#23 0x0000000000000000 in ?? ()

There are some games where this happens reliably, other games (or times?) it happens a couple of times then works.
master_of_orion started failing this morning.

The presence of Curl_resolv_timeout in the backtraces strongly suggests that this should be fixed by #96. Could you test the patch and report if it fixes things?

Seems to be gone, thank you!
(I'll re-open in case it happens again.)


Improve and fixed guide style code from outdated cmd

Hi,
I separate these part from this PR https://github.com/bundler/bundler/pull/5058.
Regards.

:umbrella: The latest upstream changes (presumably #5176) made this pull request unmergeable. Please resolve the merge conflicts.

@ryanfox1985 my apologies, I didn't even know about this PR, and it looks like my recent merge stomped on it as well.
It does seem most of these could stay with a small update on the same sort of method extraction you did that was in the recent merged commit.
I'd love to get @indirect's opinion on these changes. The project already uses rubocop to monitor style issues, and I don't feel like I have the history with the codebase to have a valuable opinion on making these changes. (Part of me doesn't want these changes due to disruption into the git history on the various lines).

@chrismo rebased.

@chrismo r?

xD

@chrismo A part of combine the dependency array and sort one time, I improve calculate filter_options_patch the old version is checking the three variables for each gem.

@bundlerbot r+


[Bug] retry_policy not availably in activity info

What are you really trying to do?
From withing an activity I want to get the attempt number
Describe the bug
I pass in a retry_policy to workflow.execute_activity and expect to be able to get that policy from within the execute activity with activity.iinfo().retry_policy but I get None instead.
Minimal Reproduction
Not entirely self contained example, but I hope you get the idea.
I tried to make a failing test in this repo, but I'm still unable to run the development environment. I guess this is a separate issue, but running the tests fails with ImportError: dlopen(/Users/nob/repos/sdk-python/temporalio/bridge/temporal_sdk_bridge.abi3.so, 0x0002): tried: '/Users/nob/repos/sdk-python/temporalio/bridge/temporal_sdk_bridge.abi3.so' (mach-o file, but is an incompatible architecture (have 'x86_64', need 'arm64e')). My guess is that rust is cross compiling for x86_64 rather than the arm64e I need for my M1 Mac for some reason 🤷♂.
from datetime import timedelta
import uuid
from temporalio import activity, workflow
from temporalio.client import Client
from temporalio.common import RetryPolicy
from bots_interface.temporal.server import temporal_worker_for_test

@activity.defn
async def return_retry_policy() -> RetryPolicy:
    activity_info = activity.info()
    # This is None!!!
    return activity_info.retry_policy

@workflow.defn
class Workflow:
    @workflow.run
    async def run(self) -> RetryPolicy:
        policy = await workflow.execute_activity(
            return_retry_policy,
            retry_policy=RetryPolicy(maximum_attempts=3),
            schedule_to_close_timeout=timedelta(seconds=5),
        )
        return policy

async def test_activity_info_retry_policy(temporal_client: Client):
    async with temporal_worker_for_test(
        temporal_client, Workflow, activities=[return_retry_policy]
    ) as worker:
        result = await temporal_client.execute_workflow(
            Workflow.run,
            id=str(uuid.uuid4()),
            task_queue=worker.task_queue,
            execution_timeout=timedelta(seconds=10),
            retry_policy=RetryPolicy(maximum_attempts=1),
        )
    assert result.maximum_attempts == 3  # This fails

Environment/Versions

OS and processor: M1 Mac, MacOS 12.4
Temporal Version: temporalio==0.1a2
Are you using Docker or Kubernetes or building Temporal from source?: Using embedded golang temporal server as used in the repo. So compiling myself I guess

Thanks! Will investigate why retry policy is not there. Looks like I didn't properly assert for it at https://github.com/temporalio/sdk-python/blob/0392468f21d9604c828d50d34daf8c9be03b31cd/tests/worker/test_activity.py#L78. There may be a bug or two with what the server is returning on activity execution.
As for the cross-compile issue, you'll need to check rustup and Rust in general is defaulted to the arm target and not the x64 one.

Ah, thank you. This explains it:
rustup show
Default host: x86_64-apple-darwin
rustup home:  /Users/nob/.rustup

stable-x86_64-apple-darwin (default)

I switched my default toolchain with rustup default stable-aarch64-apple-darwin, re-ran poe build-develop and now I can run the tests. Nice to learn a little something about rust.

I have confirmed via https://github.com/temporalio/sdk-core/blob/15ea5d3ee580e2a226eefdcf0d8444b2ae30a997/protos/local/temporal/sdk/core/activity_task/activity_task.proto#L55-L58 that we should not even expose this. None of our SDKs provide the original retry policy via the activity info (ref https://pkg.go.dev/go.temporal.io/sdk/internal#ActivityInfo, https://typescript.temporal.io/api/interfaces/activity.Info, https://www.javadoc.io/static/io.temporal/temporal-sdk/1.14.0/io/temporal/activity/ActivityInfo.html, etc) and Python shouldn't either.

Hmm, OK is there a different way for an activity to understand what retry policy it's operating under?
My use case is that my existing tasks (run using Dramatiq) log a Sentry errors and HTTP request logs on the last retry, which reduces logged errors to those that were not solved by retry. At some point I would also like to ask your opinion on the best way to do a Sentry integration (my guess is an interceptor?) but I don't want to ask too many questions in one day 🙂.
If there's a different approach you think could solve this problem, I would love to hear that as well.

Hmm, OK is there a different way for an activity to understand what retry policy it's operating under? [...] My use case is that my existing tasks (run using Dramatiq) log a Sentry errors and HTTP request logs on the last retry, which reduces logged errors to those that were not solved by retry.

The problem is that an activity may not even get called for its last attempt (what if it times out before execution?). However, if you still want to do something from the activity side on last attempt, you probably need to send that as a parameter to the activity, e.g.:
@dataclass
class MyActivityParams:
    some_other_param: str
    send_error_after_attempt: int

@activity.defn
async def my_activity(params: MyActivityParams):
    try:
        ...
    except Exception as err:
        if activity.info().attempt >= params.send_error_after_attempt:
            send_error(err)
        raise

This is untested of course, I just typed it here in issues.
But it can often be clearer and more accurate to react to activity failures when you get them back from the workflow instead of in the activity (which yes may mean starting another activity to send off the failure).

At some point I would also like to ask your opinion on the best way to do a Sentry integration (my guess is an interceptor?) but I don't want to ask too many questions in one day 🙂.

Ask away. Though I would recommend the forums at https://community.temporal.io/ or the #python-sdk Slack channel at https://temporal.io/slack for questions instead of GH issues.

The problem is that an activity may not even get called for its last attempt (what if it times out before execution?). However, if you still want to do something from the activity side on last attempt, you probably need to send that as a parameter to the activity, e.g.:

In practice, I do not find this to be an issue. For timeouts, all I really want to track is what percentage of the time a given task times out, I don't want Sentry errors because the details are not usually important and as a timeout can happen at any time you get a whole bunch of different error reports for one reason: the task is timing out.
The issue with logging activity failures from the workflow is I would lose all of the great context that Sentry collects, like local variables and breadcrumbs. For actually control flow due to the activity failures, I agree doing that in the workflow is best but I'm thinking more of observability of errors here.
It would be nice to have the retry policy because then you can have error handling contexts that can react to to the retry policy in a generic way without passing this for every activity. However, if it's not there it's not there and thank you for the example. At least I have the attempt number and I can make it work.
I'll do some experiments with Sentry setups and ask away when I'm a bit more oriented.

Closing. Feel free to reopen as necessary.


OpusDependencies.c ⇒ implicit declaration of function is invalid in C99

I have one remaining error when I try to use libnyquist with WebAssembly.
Repro steps:
Install emscripten SDK. Then:
mkdir build_emscripen
cd build_emscripten
emcmake cmake ..
make -j

Errors:
In file included from /home/sonzogna/programmation/download/libnyquist/src/OpusDependencies.c:96:
/home/sonzogna/programmation/download/libnyquist/third_party/opus/celt/celt_decoder.c:102:11: error:
      implicit declaration of function 'opus_custom_decoder_get_size' is invalid in C99
      [-Werror,-Wimplicit-function-declaration]
   return opus_custom_decoder_get_size(mode, channels);

I am not quite sure what is the problem.

@ArthurSonzogni is it required that you compile with -Wimplicit-function-declaration? It's treating the warning as an error in this case

Hmm, I wasn't able before. (I tried).
Things seems to have improved recently. I am now able to make:
https://github.com/ArthurSonzogni/smk
to depends on:
https://github.com/ddiakopoulos/libnyquist
instead of:
https://github.com/ArthurSonzogni/libnyquist
I guess it means this bug can be closed. I am no more depending on this to be resolved.


rendering via an S3 URL

i'm trying render an image via an S3 url in the following form:
"'https://s3.amazonaws.com/bucket-name/'+ req.file.key"
where req.file.key is my storage's dirname attribute; however, i keep getting the following error when i navigate to said URL: "This XML file does not appear to have any style information associated with it. The document tree is shown below." i'm not sure if this is an issue with multer itself or this module for storing to my s3 bucket. i do indeed have my storage acl set to 'public-read'.
if this helps, logging 'req.file' to the console returns an object with all the proper metadata, where the size is "undefined." if you could let me know what you think Duncan, that'd be great!

This could probably still have something to do with acl since that can be set per request. Could you try to run curl https://s3.ama... and then paste the xml output, that should give you a more specific error.

i'm pretty sure the error is not related to the acl, but rather that amazon s3 isn't recognizing the content type of the image even though multer itself is recognizing as mimetype 'image/jpeg' -- i.e. all my s3 files are of content type 'application/octet-stream.' i'm not sure where exactly the fix needs to be made however.. maybe specify content type within multer's storage option?
however, i'm not entirely sure how logically to fix this error.

yeah you're right -- it was an acl problem. i could've sworn that i had set public privileges but apparent not 0.o... thanks for being helpful, Linus.

Hehe, no problem :)


Remove the use of "here"

See https://lehors.wordpress.com/2009/01/29/linking-the-proper-way/
Thanks @lehors

Done in https://github.com/slsa-framework/oss-na24-slsa-workshop/commit/cb679284d43cd888e58f7deb465b94bcdee01a2e


I can't seem to open godot

Godot version
3.3.4 flathub
System information
Chromebook 32bit ran on linux
Issue description
As soon as I open up Godot it has a tab (and this is the only tab) stating that Error: Can't run project: no main scene defined in the project.
It's just a white box that states that message and no matter what I do I cannot get past this stage, any help?
Steps to reproduce
To reproduce these circumstances you need to on a Chromebook open a new project once recently download it, leave it empty and leave (I was checking out the software and then left) after that this error message will pop up.
Minimal reproduction project
[https://docs.google.com/presentation/d/1LyO5Yk52BFnr1MXw4soUyyKtkWC20i3Dj7SF7u_hLps/edit#slide=id.p](url for the error message)

When you run Godot executable inside a project directory, it will run the project instead of opening the editor (unless you run with -e argument).

You probably created a project folder directly within your home directory (/home/<username>). Don't do this, as Godot will try to run the project in the default working directory for applications that are started.

Thanks, I've managed to open godot again.

Reopening, as we should still prevent creating projects in locations that can "brick" the editor.

Hmm, I am definitely going to implement something to prevent users from creating a project as their home directory / executable directory. I vaguely remember importing my entire home directory with Godot Engine™, I forget if it broke my install :laughing:
However, I feel like this could be an engine bug. On linux, we can already pass --project-manager using a desktop file, to avoid this completely. What steps would need to be taken to automatically use this flag on Windows/Mac?

Fixed by https://github.com/godotengine/godot/pull/74964.


Bump dependencies

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

test

test

test

test


List unofficial Erlang client

https://github.com/deadtrickster/prometheus.erl

Thanks!
Only a float API should be offered. In your histogram example you should have the units in seconds. orders_summary should be orders, don't put types in metric names.


[cargo.sh] Auto-install toolchains and components

cargo.sh detects missing Rust toolchains and the rust-src component. When missing, it offers to install them automatically via rustup.

You can also just simply do:
rustup toolchain install $TOOLCHAIN -c rust-src

If the toolchain isn't installed, it installs it. If the component isn't installed, it installs it. If the toolchain and component are installed, it does nothing.

I'm not sure we should install without user confirmation. Is there a way to have rustup confirm first?

You can also just simply do:
rustup toolchain install $TOOLCHAIN -c rust-src

If the toolchain isn't installed, it installs it. If the component isn't installed, it installs it. If the toolchain and component are installed, it does nothing.

I'm not sure we should install without user confirmation. Is there a way to have rustup confirm first?

Discussed offline; this still allows us to simplify the code even while prompting the user.


[BUG] IonMenu is swipable through backdrop

Describe the bug
IonMenu is swipable through the Cupertino-Pane Backdrop
To Reproduce
Just create Ionic App with Menu and open Drawer.
Expected behavior
It should not be swipable.

@BerkeAras Thank you. I tried, but can't reproduce with both opened menu and drawer.
Please share some code.

@roman-rr https://jsfiddle.net/umy4vn91/

@BerkeAras
Please try this solution (Disable/Enable gesture)
<ion-menu [swipeGesture]="swipeGesture">

And controller
public swipeGesture:boolean = true;
constructor() {}

ngOnInit() {
    this.drawer = new CupertinoPane('.ion-drawer', {
      ...
      onWillPresent: () => this.swipeGesture = false,
      onDidDismiss: () => this.swipeGesture = true
    });
    this.drawer.present({animate: true});
}

@BerkeAras Already able touchMoveStopPropagation settings with the latest commit, and will be able in next release within 1 week.
Simple set
let settings = {
  ...
  touchMoveStopPropagation: true
}

To fix this issue.


eo-teams-python-upd-del-messages

Added Python snippets.

Docs Build status updates of commit 69c2954:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/update-and-delete-bot-messages.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 709b20e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/update-and-delete-bot-messages.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 5b86fef:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/update-and-delete-bot-messages.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.