dtc-1 | test

test ai review

Code Review: AutoHotkey/test.sql
1. Code Quality and Adherence to Best Practices
General Observations:

The SQL file is creating multiple indexes on the t_student table.
The naming convention for the indexes is consistent but not very descriptive.

Best Practices:

Index Naming: Index names should be meaningful and reflect the columns they are created on. For example, idx_t_student_student_id_limit_id is more descriptive than utile_limit_wid_index1.
Consistency in Column Names: The column names used in the indexes (e.g., student_id, s_id, stuf_dent_id) seem inconsistent and possibly incorrect. Ensure that these column names are correct and consistent with the actual table schema.
Documentation: Add comments to explain the purpose of each index. This will help future maintainers understand why these indexes were created.
File Naming and Location: The file test.sql is located in the AutoHotkey directory, which seems unusual. Consider placing SQL files in a dedicated directory, such as sql/ or database/.

Example of Improved Naming and Documentation:
-- Index on student_id and limit_id for faster lookups
CREATE INDEX idx_t_student_student_id_limit_id ON t_student (student_id, limit_id);

-- Index on s_id and limit_id for faster lookups
CREATE INDEX idx_t_student_s_id_limit_id ON t_student (s_id, limit_id);

-- ... (similar comments for other indexes)

2. Potential Bugs or Issues
Column Name Consistency:

The column names stuf_dent_id, test_id, g_id, x_id, d_id, b_id, w_id, and oe_id seem unusual and might be typos or placeholders. Verify that these column names are correct and exist in the t_student table.

Redundant Indexes:

Creating multiple indexes on the same table can lead to redundancy. Ensure that each index serves a unique and necessary purpose. For example, if s_id and student_id refer to the same logical entity, you might not need both indexes.

3. Security Vulnerabilities
SQL Injection:

Since this is a static SQL script, there is no immediate risk of SQL injection. However, ensure that any dynamic SQL generated by your application is properly parameterized to prevent SQL injection.

Data Exposure:

Ensure that the t_student table does not contain sensitive information that could be exposed through these indexes. If it does, consider using views or other mechanisms to control access to the data.

4. Performance Implications
Index Overhead:

Adding too many indexes can slow down write operations (INSERT, UPDATE, DELETE) because the database needs to update all the indexes. Evaluate the trade-off between read and write performance.
Consider the selectivity of the columns used in the indexes. Highly selective columns (those with a high number of unique values) generally provide better performance.

Query Performance:

Ensure that the queries that benefit from these indexes are actually being executed. Use the EXPLAIN PLAN feature of your database to verify that the indexes are being utilized.

5. Specific Suggestions for Improvement

Review and Correct Column Names:

Verify the correctness and consistency of the column names used in the indexes.
Example: If stuf_dent_id is a typo, correct it to student_id.

Improve Index Naming:

Use more descriptive and meaningful names for the indexes.
Example: idx_t_student_student_id_limit_id instead of utile_limit_wid_index1.

Add Comments:

Document the purpose of each index to aid future maintenance.
Example: -- Index on student_id and limit_id for faster lookups.

Evaluate Index Necessity:

Review the necessity of each index and remove any redundant ones.
Example: If s_id and student_id are the same, choose one and remove the other.

Organize Files:

Move the SQL file to a more appropriate directory, such as sql/ or database/.

Monitor Performance:

After adding the indexes, monitor the performance of both read and write operations to ensure that the changes have the desired effect.

By addressing these points, you can improve the quality, security, and performance of your SQL scripts.

Code Review: AutoHotkey/test.sql
1. Code Quality and Adherence to Best Practices
General Observations:

The file .DS_Store is a macOS-specific file that stores custom attributes of a folder, such as the position of icons or the choice of a background image. It should not be included in version control. Add this file to your .gitignore to prevent it from being committed.

SQL Script:

Consistency and Naming Conventions:

The index names (utile_limit_wid_index1, utile_limit_wid_index2, etc.) are not descriptive and do not follow a consistent naming convention. It is best to use meaningful names that reflect the purpose of the index.
The column names used in the indexes (e.g., student_id, s_id, stuf_dent_id, test_id, etc.) are inconsistent and some appear to be typos (e.g., stuf_dent_id).

Redundancy:

Creating multiple indexes on the same table with similar columns can lead to redundancy and increased storage usage. For example, utile_limit_wid_index1 and utile_limit_wid_index2 both include limit_id but differ only in the first column. Consider whether all these indexes are necessary.

Formatting:

The SQL statements are not formatted consistently. Adding proper indentation and spacing can improve readability.
The file does not end with a newline, which is a common best practice for text files.

Suggested Improvements:

Use a consistent and meaningful naming convention for indexes.
Verify the column names and correct any typos.
Evaluate the necessity of each index and remove redundant ones.
Format the SQL statements for better readability.
Ensure the file ends with a newline.

2. Potential Bugs or Issues

Typos and Inconsistent Column Names:

The column name stuf_dent_id appears to be a typo. It should likely be student_id.
The presence of multiple similar column names (e.g., s_id, g_id, x_id, d_id, b_id, w_id, oe_id) without clear documentation or context can lead to confusion and potential bugs.

Index Redundancy:

Creating multiple indexes on the same table with similar columns can lead to performance issues and increased storage usage. For example, if utile_limit_wid_index1 and utile_limit_wid_index2 are not necessary, they should be removed.

Suggested Improvements:

Correct any typos in column names.
Review the necessity of each index and remove redundant ones.

3. Security Vulnerabilities

SQL Injection:

The provided SQL script does not involve dynamic SQL or user input, so there is no immediate risk of SQL injection. However, always ensure that any dynamic SQL or user input is properly sanitized and parameterized in other parts of the application.

Data Exposure:

Indexes can potentially expose sensitive data if they are not properly managed. Ensure that the indexed columns do not contain sensitive information and that appropriate access controls are in place.

Suggested Improvements:

No specific security vulnerabilities are present in the provided script, but always follow best practices for managing and securing database indexes.

4. Performance Implications

Index Overhead:

Creating multiple indexes on the same table can have significant performance implications. Each index requires additional storage and can slow down write operations (INSERT, UPDATE, DELETE) because the database must maintain the indexes.
The presence of many indexes can also impact read performance if the query optimizer has to evaluate many index options.

Query Optimization:

Ensure that the indexes are actually used by the queries. Unused indexes can add overhead without providing any benefit.
Use EXPLAIN or a similar tool to analyze the execution plans of your queries and verify that the indexes are being used effectively.

Suggested Improvements:

Evaluate the necessity of each index and remove those that are not used.
Use EXPLAIN to analyze query performance and ensure that the indexes are beneficial.

5. Specific Suggestions for Improvement

Correct Typos and Inconsistent Column Names:
create index idx_student_student_id_limit_id on t_student(student_id, limit_id);
create index idx_student_s_id_limit_id on t_student(s_id, limit_id);
-- Correct the typo in stuf_dent_id
create index idx_student_student_id_limit_id on t_student(student_id, limit_id);
create index idx_student_test_id_limit_id on t_student(test_id, limit_id);
create index idx_student_g_id_limit_id on t_student(g_id, limit_id);
create index idx_student_x_id_limit_id on t_student(x_id, limit_id);
create index idx_student_d_id_limit_id on t_student(d_id, limit_id);
create index idx_student_b_id_limit_id on t_student(b_id, limit_id);
create index idx_student_w_id_limit_id on t_student(w_id, limit_id);
create index idx_student_oe_id_limit_id on t_student(oe_id, limit_id);

Remove Redundant Indexes:

Review the necessity of each index and remove those that are not needed. For example, if idx_student_student_id_limit_id and idx_student_s_id_limit_id serve the same purpose, keep only one.

Format the SQL Statements:
CREATE INDEX idx_student_student_id_limit_id ON t_student (student_id, limit_id);
CREATE INDEX idx_student_s_id_limit_id ON t_student (s_id, limit_id);
-- Correct the typo in stuf_dent_id
CREATE INDEX idx_student_student_id_limit_id ON t_student (student_id, limit_id);
CREATE INDEX idx_student_test_id_limit_id ON t_student (test_id, limit_id);
CREATE INDEX idx_student_g_id_limit_id ON t_student (g_id, limit_id);
CREATE INDEX idx_student_x_id_limit_id ON t_student (x_id, limit_id);
CREATE INDEX idx_student_d_id_limit_id ON t_student (d_id, limit_id);
CREATE INDEX idx_student_b_id_limit_id ON t_student (b_id, limit_id);
CREATE INDEX idx_student_w_id_limit_id ON t_student (w_id, limit_id);
CREATE INDEX idx_student_oe_id_limit_id ON t_student (oe_id, limit_id);

Ensure the File Ends with a Newline:

Add a newline at the end of the file.

By following these suggestions, you can improve the quality, performance, and maintainability of your SQL script.

Code Review
1. .DS_Store

File: .DS_Store
Status: added
Changes: None

Analysis:

Code Quality and Best Practices: The .DS_Store file is a hidden file used by macOS to store custom attributes of a folder, such as the position of icons or the choice of a background image. It is not necessary for version control and should generally be ignored.
Potential Bugs or Issues: N/A
Security Vulnerabilities: N/A
Performance Implications: N/A

Suggestions for Improvement:

Add .DS_Store to your .gitignore file to prevent it from being committed in the future.

2. AutoHotkey/javatest.java

File: AutoHotkey/javatest.java
Status: added
Changes:package AutoHotkey;

class test {
  String kk() {
    return "d";
  }
}
\ No newline at end of file

Analysis:

Code Quality and Best Practices:

Class Naming: Class names should start with an uppercase letter (e.g., Test instead of test).
Method Naming: Method names should follow camelCase convention (e.g., getKk instead of kk).
No Newline at End of File: Java files should end with a newline character.
Package Structure: Ensure that the package name AutoHotkey is appropriate and follows the project's naming conventions.

Potential Bugs or Issues: The method kk() returns a hardcoded string. This may not be useful unless it serves a specific purpose.
Security Vulnerabilities: N/A
Performance Implications: N/A

Suggestions for Improvement:

Rename the class to Test.
Rename the method to getKk or something more descriptive.
Add a newline at the end of the file.
Consider adding Javadoc comments to explain the purpose of the class and method.

3. AutoHotkey/test.sql

File: AutoHotkey/test.sql
Status: added
Changes:create index utile_limit_wid_index1 on t_student( student_id, limit_id);
create index utile_limit_wid_index2 on t_student( s_id, limit_id);
create index utile_limit_wid_index3 on t_student( stuf_dent_id, limit_id);
create index utile_limit_wid_index4 on t_student( test_id, limit_id);
create index utile_limit_wid_index5 on t_student( g_id, limit_id);
create index utile_limit_wid_index6 on t_student( x_id, limit_id);
create index utile_limit_wid_index7 on t_student( d_id, limit_id);
create index utile_limit_wid_index8 on t_student( b_id, limit_id);
create index utile_limit_wid_index9 on t_student( w_id, limit_id);
create index utile_limit_wid_index10 on t_student( oe_id, limit_id);
\ No newline at end of file

Analysis:

Code Quality and Best Practices:

Index Naming: Index names should be meaningful and consistent. The current names are not very descriptive.
Column Names: Some column names (e.g., stuf_dent_id) seem to be typos or incorrectly named.
No Newline at End of File: SQL files should end with a newline character.

Potential Bugs or Issues: Creating multiple indexes on the same table can lead to performance issues if not managed properly. Ensure that these indexes are necessary and do not overlap.
Security Vulnerabilities: N/A
Performance Implications: Multiple indexes can slow down write operations (INSERT, UPDATE, DELETE) and increase storage usage. Ensure that the benefits of these indexes outweigh the costs.

Suggestions for Improvement:

Use more descriptive index names.
Verify and correct column names.
Add a newline at the end of the file.
Evaluate the necessity of each index and consider the performance impact.

4. AutoHotkey/testjs.js

File: AutoHotkey/testjs.js
Status: added
Changes:var k = "test";
let b = 'd';
const s = '3';

console.log(k + b + s);
\ No newline at end of file

Analysis:

Code Quality and Best Practices:

Variable Declarations: Use const for variables that do not change, and let for those that might. var is generally discouraged due to its function scope and hoisting behavior.
String Concatenation: Using + for string concatenation is fine, but consider using template literals for better readability.
No Newline at End of File: JavaScript files should end with a newline character.

Potential Bugs or Issues: N/A
Security Vulnerabilities: N/A
Performance Implications: N/A

Suggestions for Improvement:

Replace var k with const k.
Use template literals for string concatenation:const k = "test";
const b = 'd';
const s = '3';

console.log(`${k}${b}${s}`);

Add a newline at the end of the file.

Summary

Add .DS_Store to .gitignore.
Improve naming conventions and add Javadoc comments in javatest.java.
Use more descriptive index and column names in test.sql, and evaluate the necessity of each index.
Use const and template literals in testjs.js.

By addressing these points, you will improve the code quality, maintainability, and readability of your project.

Code Review
1. .DS_Store

Status: Added
Changes: None

Analysis:

Code Quality and Best Practices: .DS_Store files are system files created by macOS to store custom attributes of a folder, such as the position of icons or the choice of a background image. These files should not be committed to version control.
Potential Bugs or Issues: N/A
Security Vulnerabilities: N/A
Performance Implications: N/A

Suggestions for Improvement:

Add .DS_Store to your .gitignore file to prevent it from being committed in the future.

2. AutoHotkey/javatest.java

Status: Added
Changes:package AutoHotkey;

class test {
  String kk() {
    return "d";
  }
}

Analysis:

Code Quality and Best Practices:

The class name test should follow Java naming conventions and be capitalized, e.g., Test.
The method kk() is not descriptive. Consider using a more meaningful name that indicates what the method does.
The file ends without a newline, which is generally considered a best practice to include.

Potential Bugs or Issues: The method kk() returns a hard-coded string. Ensure this is the intended behavior.
Security Vulnerabilities: N/A
Performance Implications: N/A

Suggestions for Improvement:

Rename the class to Test and the method to something more descriptive, e.g., getDefaultValue().
Add a newline at the end of the file.

Example:
package AutoHotkey;

class Test {
  String getDefaultValue() {
    return "d";
  }
}

3. AutoHotkey/test.sql

Status: Added
Changes:create index utile_limit_wid_index1 on t_student( student_id, limit_id);
create index utile_limit_wid_index2 on t_student( s_id, limit_id);
create index utile_limit_wid_index3 on t_student( stuf_dent_id, limit_id);
create index utile_limit_wid_index4 on t_student( test_id, limit_id);
create index utile_limit_wid_index5 on t_student( g_id, limit_id);
create index utile_limit_wid_index6 on t_student( x_id, limit_id);
create index utile_limit_wid_index7 on t_student( d_id, limit_id);
create index utile_limit_wid_index8 on t_student( b_id, limit_id);
create index utile_limit_wid_index9 on t_student( w_id, limit_id);
create index utile_limit_wid_index10 on t_student( oe_id, limit_id);

Analysis:

Code Quality and Best Practices:

The column names like stuf_dent_id, oe_id, etc., seem to be typos or non-standard. Ensure these column names are correct and consistent with the database schema.
The file ends without a newline, which is generally considered a best practice to include.

Potential Bugs or Issues: If the column names are incorrect, the SQL statements will fail to execute.
Security Vulnerabilities: N/A
Performance Implications: Creating multiple indexes can improve query performance but may slow down write operations (INSERT, UPDATE, DELETE). Ensure that the indexes are necessary and used frequently in queries.

Suggestions for Improvement:

Verify the column names and ensure they match the actual column names in the t_student table.
Add a newline at the end of the file.

Example:
create index utile_limit_wid_index1 on t_student( student_id, limit_id);
create index utile_limit_wid_index2 on t_student( s_id, limit_id);
-- Ensure other column names are correct
create index utile_limit_wid_index10 on t_student( oe_id, limit_id);

4. AutoHotkey/testjs.js

Status: Added
Changes:var k = "test";
let b = 'd';
const s = '3';

console.log(k + b + s);

Analysis:

Code Quality and Best Practices:

Use const for all variables that do not change, including k. This makes the code more predictable and easier to understand.
The file ends without a newline, which is generally considered a best practice to include.

Potential Bugs or Issues: N/A
Security Vulnerabilities: N/A
Performance Implications: N/A

Suggestions for Improvement:

Use const for all variables that do not change.
Add a newline at the end of the file.

Example:
const k = "test";
const b = 'd';
const s = '3';

console.log(k + b + s);

Summary

.DS_Store: Add to .gitignore.
javatest.java: Follow Java naming conventions, use more descriptive method names, and add a newline at the end.
test.sql: Verify column names and add a newline at the end.
testjs.js: Use const for all immutable variables and add a newline at the end.

These changes will improve the overall quality, readability, and maintainability of your code.


Sector Simulation

This PR includes the mechanisms to allow sector-scope entities to perform simulation tasks. The bulk of the work is done by the SectorSimulationSystem, which sends events to allow sector-scope entities to work.
The SectorSimulationEvent is sent repeatedly, whether or not the chunk the entity is in is loaded. It will be sent at least once every maxDelta (defined in the SectorSimulationComponent, which holds information for calculating the simulation delta), but might be sent more often (e.g. it is always sent when the chunk is loaded). This should be used for any calculations and simulation the entity needs to do.
TODO: add another property so that the simulation rate can be adjusted when the entity is loaded
The LoadedSectorUpdateEvent is sent repeatedly, but only while the chunk the entity is in is loaded. This is where any effects on the world should take place, as the chunk is guaranteed to be loaded whenever this is sent. A SectorSimulationEvent is always sent immediately prior to this one, so no simulation needs to happen in reaction to this event.
The OnChunkLoaded and BeforeChunkLoaded also get forwarded to the appropriate sector entities when their chunks are affected, so certain things can be done on chunk load/unload (e.g. caching some data to use when the chunk is unloaded). A LoadedSectorUpdateEvent is automatically sent immediately after the OnChunkLoaded event, so this only needs to be used for things that don't normally get simulated (e.g. generating new villager entities).
See the SectorTesting module for an example of how it is implemented.

Merged in https://github.com/MovingBlocks/Terasology/pull/3009


Demo3 (todoList), TransitionMotion: "defaultStyles" and "styles"  => got object, array expected

I'm adapting react-motion (stable) demos  in our very new template app (meteor-react-es6), to transform each of them in a single component (ex: <DraggableList/>), using react-motion npm package and React.Component (instead of React.createClass).
So it only demands a syntax adaptation, but variables still the same..
The original file
demo3/
My component
here
(shortcut to TransitionMotion)
Console
Warning: Failed propType: Invalid prop "defaultStyles" of type "object" supplied to "TransitionMotion", expected an array. Check the render method of "ListAnimated"
Warning: Failed propType: Invalid prop "styles" supplied to "TransitionMotion". Check the render method of "ListAnimated"
Error Uncaught TypeError: defaultStyles.map is not a function
Any idea guys??

Same problem, is this project alive?

@MadeInMoon, I've just double-checked 0.4.1 version locally and had no issues with current demo3. The problem is definitely somewhere in your adaptation. Could you please post your component to Codepen so we can run it?

  // actual animation-related logic
  getDefaultValue() {
    const {todos} = this.state;
    return Object.keys(todos)
      .reduce((configs, date) => {
        configs[date] = {
          height:  spring(0),
          opacity: spring(1),
          data:    todos[date],
        };
        return configs;
      }, {});
  }

  getDefaultStyles() {
    return this.state.todos.map(todo => ({...todo, style: {height: 0, opacity: 1}}));
  },

getDefaultStyles returned array of styles. getDefaultValue returns object. Object does not have map method.

Problem solved.. Man, thank you so much for helping..
The problem
I was adapting the demo3/todo from this commit, after reading that in master demo readme:
Note: since this is the master branch, the demos might be a bit ahead of the current stable API. This commit contains the [stable demos](https://github.com/chenglou/react-motion/tree/0627243316c564f6c2f480bf615b82135f649a0a/demos).
Your Solution
the demo3/todo from master  demos

Question
In the stable commit :
- demo8 :  re-adaptable in a few seconds
- demo3 : seems to be un-usable because of object/array formats
Is it better to go in master  demos or stable commit demos?

Master of course.
Why changing demos to es6 classes?

All my template app components are ES6 consts or classes. So why should stay in ES5?

@MadeInMoon ah, sorry, I thought you are going to PR these changes to react-motion.
I am closing this for now, feel free to ping me in https://gitter.im/nkbt/help or Reactiflux if you need more help!


Keep Alive functionality does not work unless a message has been received on the websocket

What is the current behavior?
When opening a websocket and "holding" it open by sending Keep Alive messages, the websocket closes after 30 seconds. Note that no audio data is sent.
If a tiny amount of audio data is sent, Deepgram will not respond with a message over the websocket. In this scenario, the websocket also closes after 30 seconds.
If enough audio data is sent over the websocket so that Deepgram responds, the websocket stays open (which is the correct behavior, and sending the keep alive messages should result in the same behavior).
Steps to reproduce
Below is the reproduceable C# code:
using System;
using System.Text;
using Deepgram;
using Deepgram.Models;
namespace SampleApp;

class SampleApp
{
    public static async Task<int> Main()
    {
        // Set up the DG live client
        var credentials = new Credentials("<api_key>", "https://api.deepgram.com", null);
        var deepgramClient = new DeepgramClient(credentials);
        var deepgramLive = deepgramClient.CreateLiveTranscriptionClient();
        deepgramLive.ConnectionOpened += (object? sender, Deepgram.CustomEventArgs.ConnectionOpenEventArgs e) => { Console.WriteLine("Deepgram Connection Opened"); };
        deepgramLive.TranscriptReceived += (object? sender, Deepgram.Models.TranscriptReceivedEventArgs e) => { Console.WriteLine("Deepgram Message Received"); };
        deepgramLive.ConnectionClosed += (object? sender, Deepgram.CustomEventArgs.ConnectionClosedEventArgs e) => { Console.WriteLine("Deepgram Connection Closed"); };
        deepgramLive.ConnectionError += (object? sender, Deepgram.CustomEventArgs.ConnectionErrorEventArgs e) => { Console.WriteLine("Deepgram Connection Error"); };

        // Open the websocket
        var options = new Deepgram.Models.LiveTranscriptionOptions()
        {
            Tier = "base",
            Model = "general",
            Language = "en",
        };
        await deepgramLive.StartConnectionAsync(options);

        // Send keep-alive messages (without sending any audio) to keep the websocket open
        for (int i = 0; i < 60; i++)
        {
            Task.Delay(2000).Wait();
            if (System.Net.WebSockets.WebSocketState.Open == deepgramLive.State())
            {
                Console.WriteLine("Sending KeepAlive " + i);
                deepgramLive.KeepAlive();
            }
            else
            {
                // The websocket is no longer open... why???
                Console.WriteLine("Websocket changed state unexpectedly, it's " + deepgramLive.State() + " (iteration " + i + ")");
                break;
            }
        }

        return 0;
    }
}

To run this code/repo in GitHub's codespaces, edit the devcontainer.json file with the contents below:
// For format details, see https://aka.ms/devcontainer.json. For config options, see the
// README at: https://github.com/devcontainers/templates/tree/main/src/dotnet
{
	"name": "C# (.NET)",
	"image": "mcr.microsoft.com/devcontainers/dotnet:0-7.0",
	"features": {
		"ghcr.io/devcontainers/features/dotnet:1": {
			"installUsingApt": true,
			"version": "7"
		}
	}
}

You can then modify the SDK to debug directly.
Expected behavior
The websocket should not close after 30 seconds because Keep Alive messages are being sent.
Please tell us about your environment
I used GitHub codespaces - see above.
Other information
I implemented the same functionality in Python and the issue does not occur. This implies the issue is with the .NET code.
I dug into the SDK code carefully, and there is no indication of the Keep Alive messages not being sent to Deepgram. Additionally, if you comment out the line deepgramLive.KeepAlive();, the websocket closes after 12 seconds (which is intentional). This implies the Keep Alive is being sent over the websocket.
My best guess is that C# is closing the websocket if the websocket does not receive a message after 30 seconds - but that would be extremely surprising behavior.
Below is the Python code that is "identical" to the C# code. The websocket stays open during the entire lifetime of the code.
from deepgram import Deepgram
import asyncio
import time
import os

async def main():
    DEEPGRAM_API_KEY = os.environ["DEEPGRAM_API_KEY"]
    deepgram = Deepgram(DEEPGRAM_API_KEY)
    deepgramLive = await deepgram.transcription.live({
        "tier": "base",
        "model": "general",
        "language": "en",
    })

    # Add handlers
    deepgramLive.registerHandler(deepgramLive.event.CLOSE, lambda c: print(f"Connection closed with code {c}."))
    deepgramLive.registerHandler(deepgramLive.event.TRANSCRIPT_RECEIVED, print)

    # Listen for the connection to open and send streaming audio from the URL to Deepgram
    for i in range(60):
        time.sleep(2)
        if deepgramLive._socket.open:
            deepgramLive.keep_alive()
            print(f"Keep alive {i}")
        else:
            print("Deepgram connection closed unexpectedly!")
    time.sleep(1)

    # Indicate that we've finished sending data by sending the customary zero-byte message to the Deepgram streaming endpoint, and wait until we get back the final summary metadata object
    await deepgramLive.finish()

asyncio.run(main())

The issue is with the Keep Alive's WebSocketMessageType. It needs to be set to WebSocketMessageType.Text rather than WebSocketMessageType.Binary. PR incoming.


[WIP] Add community groups (SIGs/WG/UG)

Sources community groups with sig-list.md serving as the default page.
WIP for now. Can remove the group readme's and merge just the fix if needed.

/assign @vishakhanihore
/assign @parispittman

I've removed the wip and made this PR just the bug fix to get it merged. Will do a PR later to add community groups after we fix ordering / naming in k/community

/lgtm
/approve


getCropRect() wrong coordinates

On samsung devices the coordinates are confused, although in the correct orientation of the bitmap

Example?

This happened to me today. An image had width of 2218. But, getCropRect() says that rect.right is 2220.
I have attached a sample image from dropbox.
https://www.dropbox.com/sh/94fxu6hnpyfxabk/AAAqYtQnRrpVmxcCEVRzSKyea?dl=0


Adding how log is MFA usage log stored

My customer insists to ask us to adding how long is MFA usage log stored.

@Castermeek : Thanks for your contribution to the Azure documentation! The author, @MarkusVi, has been notified to review your proposed change.

It seems All checks have been passed.
To this be published, what else I have to do?

#sign-off


CodeBlock appearance stay dark when switching to light mode.

🐛 Bug Report
Have you read the [Contributing Guidelines on issues]
Yes
To Reproduce

Start a new Docusaurus project and open on a page which has a code block on it or open the Docusaurus sandbox demo page at: https://codesandbox.io/s/docusaurus . The page will appear in dark theme mode.
Open Tutorial page from the menu which has a code block on it.
Switch the theme to light theme mode froom the navbar. The code block appearance stays in dark theme mode.

Expected behavior
I am not sure if this is the correct behavior of the theme switching regarding the code block of Docusaurus v2. But I am expecting it is going to work as in Docusaurus official documentation site (https://docusaurus.io/docs/next/). When switching the theme from dark to light vice-versa, the code block appearance is switching properly.
Docusaurus online documentation light theme:

Docusaurus online documentation dark theme:

Actual Behavior
The code block appearance stays in dark theme mode, regardless the dark/light theme.

Your Environment

Public source code: N/A
Public site url: N/A (local development)
Docusaurus version used: 2.0.0-beta.0
Environment name and version (e.g. Chrome 78.0.3904.108, Node.js 10.17.0):
Operating system and version (desktop or mobile): macOS Catalina 10.15.7

Reproducible Demo

Open https://codesandbox.io/s/docusaurus
Open the Tutorial page on Browser panel
Switch theme to dark and light
The code blocks stay in dark appearance

https://docusaurus.io/docs/api/themes/configuration#theme
You can set distinct codeblock Prism themes for light/dark mode
Maybe we should also document it here?
https://docusaurus.io/docs/markdown-features/code-blocks#syntax-highlighting
Eventually, add a distinct dark theme in the init templates?

I see, it is my mistake for not reading and understanding the documentation thoroughly.
So it is related to the configuration of prism after all.
I wonder what else can we do with prism other than setting the theme?
  themeConfig: {
    prism: {
      theme: require('prism-react-renderer/themes/github'),
      darkTheme: require('prism-react-renderer/themes/dracula'),
      // how about other settings documentation/information?
      // possibly includes the default and possible values for each configuration setting.
    },
    // ...
  },

Do you think the documentation can also be improved by adding more information about what we can do with prism from the configuration file documented here?
https://docusaurus.io/docs/docusaurus.config.js
Thank you for the response. I think we can close this issue for now.

There's nothing more we can do than what is documented here afaik: https://docusaurus.io/docs/api/themes/configuration#codeblock
Also it's not a good idea for me to document on the config doc, because the config can use a custom theme that has totally different theme config object than the theme config object of the theme-classic. I would rather remove/reduce that themeConfig example in the docusaurus config doc, and ask the user to. visit the theme-classic themeConfig doc directly instead


Activities Option in YAP

Would like to see the option where users call in and have an option for activities like dances, conventions,etc. This should be read off as an MP3 file.
Our old activities line was a phone option where someone would just read off flyers a group or area provided. This could stay the same with the new option with MP3.
Lastly,  it would be nice to just upload PDF files of flyers to a specific location and have a computer voice read off the content in each of the pdf files effectively creating that MP3 (text to speech engine)

Reading PDFs is very unreliable, unless formatted correctly.  Most likely this would require reformatting as most folks do not understand the semantics how to do this.  I get the intention of this feature though, and there seems like there could be a way to automate potentially some other way.

Can we do the mp3 option where the flyers are read off by a person and updated on a regular basis?

Not currently.  There would have to be a change to support this.

Custom extensions option will be available in version 3.7.0.  Details are here: https://github.com/bmlt-enabled/yap/wiki/Custom-Extensions
You could do a few things with this.

Set the custom extension option to play back a recorded MP3 with the announcements.
Add a custom extension that redirects to another phone number in Twilio that has a TwiML bin that plays back announcements.
With a few future enhancements to sub, we could create the ability to automatically text-to-speech announcements (assuming you are using it).

https://github.com/bmlt-enabled/sub/issues/4
https://github.com/bmlt-enabled/sub/issues/5
https://github.com/bmlt-enabled/sub/issues/6

This is in master and testable https://archives.bmlt.app/yap/yap-1379-ae2fa62c5507aa4b79b36dc4e698cb8a8e7b1483.zip


Add dialout group to sync, for use with programming microcontrollers

This lets the crouton environment work with IDEs like Arduino, NodeMCU, etc. I've only tested on an Asus C100 Flip (ARM.)

Are tape and tty also needed?  Or just dialout?

They were already in the file; I just moved them down a line to keep the line length about the same.

haha I can't read
Yes, this looks good.  @smibarber can you think of any downsides here?

Nope, seems fine to me.


Wrong version number being reported

When I run npm outdated I get a different output for latest as the one shown when I hover over a package in package.json.
I can confirm that I only have package-lock.json in my project.

I am unable to reproduce. Could you provide an example of a package this is happening for?

This turned out to be an issue with the built-in npm extension.


Please update to 1.21.1

Please update plugin to 1.21.1

Thanks for creating this issue! I have updated Simple Stack to 1.21.1, you can find the download here. Let me know if you are experiencing any issues with the update.

Thanks for creating this issue! I have updated Simple Stack to 1.21.1, you can find the download here. Let me know if you are experiencing any issues with the update.

Thanks for the update!


Added OverlayCenter method

Equivalent to PasteCenter, but using Overlay instead. Useful for creating thumbnails from transparent images.

I think it might be useful. Thank you.


The Notes Sometimes Don't Input in Psych 0.7.2 And Above

Describe your bug here.
Mind you, if you are modding, please try testing your bug in a clean version of the engine instead, this ensures the bug is actually caused by the engine itself.
Basically, I have a problem where in 0.7.2 and above, the notes in a song won't input even when I've pressed down on the specific key. I don't know if the input timing has changed or not, so lemme know if that's the case. Anyways, I even tried on different devices, same problem.
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
Which branch of the engine are you using?
Latest (main)
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No, I haven't

Nevermind, I saw that they have a new input system


BTC in crowdsales

Hey guys,
As requested I'm working on allowing BTC in crowdsales.
I noticed in the spec we have the text:

Effective with version 1 of Transaction type 51 and block #(TBD), a single crowdsale is able to accept multiple currencies, including bitcoins (currency id 0), for purchases of a Smart Property in a single crowdsale.

Two Q's:

The requirement at the top of roadmap is BTC in crowdsales, but the roadmap doesn't mention multiple properties in crowdsales.  Are we sticking to the roadmap, or do we want to try and do multi-currency crowdsales per spec?

Note my vote is a resounding 'no' to multiple currency crowdsales, as it means we need to allow multiple crowdsales to run side by side for a single address, and I'm not really sure what the point is (though happy to be educated!).

What exactly does constitute a v1 crowdsale?  We have plenty of valid v1 crowdsales already on-chain, and these certainly don't allow BTC or multiple currencies.

Thanks
Z

Do BTC payments to crowdsales need to carry an Omni marker (Exodus P2PKH or OP_RETURN marker)?

I vote yes.

Should sender identification for BTC payments to crowdsales be based on the older "largest input by sum" logic or the newer "first vin is sender" logic?

Active BTC crowdsale.  Can't participate yet as early days, but progress...
{
    "propertyid" : 3,
    "name" : "Z_IndivisTestProperty",
    "active" : true,
    "issuer" : "mhVgSK3zaPdm752cZ7teXZaBJcMHH3ibXo",
    "propertyiddesired" : 0,
    "tokensperunit" : "10",
    "earlybonus" : 0,
    "percenttoissuer" : 0,
    "starttime" :<PHONE_NUMBER>,
    "deadline" :<PHONE_NUMBER>,
    "amountraised" : "0.00000000",
    "tokensissued" : "0",
    "addedissuertokens" : "0"
}

The requirement at the top of roadmap is BTC in crowdsales, but the roadmap doesn't mention multiple properties in crowdsales. Are we sticking to the roadmap, or do we want to try and do multi-currency crowdsales per spec?

In my opinion these are two different requirements: accepting BTC, and accepting multiple properties. While I can see the use of having multiple properties (e.g. Omni and BTC at the same time), it seems more complex, and I'd rather go step by step (i.e. only implement BTC crowdsales now).

What exactly does constitute a v1 crowdsale? We have plenty of valid v1 crowdsales already on-chain, and these certainly don't allow BTC or multiple currencies.

That's a mess-up from the old days (see: https://github.com/mastercoin-MSC/mastercore/issues/118). We need to use v2.

Do BTC payments to crowdsales need to carry an Omni marker (Exodus P2PKH or OP_RETURN marker)?

Absolutely. However, in my opinion the mechanism is very similar to BTC payments of DEx orders, so we probably should reuse that code in terms of marker selection.

Should sender identification for BTC payments to crowdsales be based on the older "largest input by sum" logic or the newer "first vin is sender" logic?

Assuming we reuse the marker identification from DEx payments, then we're stuck with Class B, thus "largest input by sum". Then again, I'd consider it as flaw that we can't use an empty OP_RETURN marker in these cases. Though also another point: BTC payments are very different from normal payload transactions.

Absolutely. However, in my opinion the mechanism is very similar to BTC payments of DEx orders, so we probably should reuse that code in terms of marker selection.

Hmm... I actually really disliked this model (no explicit payload describing the function) on the DEx and am sort of heading in a different direction with the BTC crowdsales stuff; requiring an explicit payload.  I've always strongly favored being explicit rather than implicit about Omni Layer interactions.
For example what happens if an address has a DEx sell for BTC and a crowdsale for BTC both active, which open transaction should the BTC credit be applied against?
It'll need the "Investment Send" transaction to be brought back (renamed "Participate In Crowdsale" or similar, we should avoid nomenclature that has the term "invest" included).  I realize we dropped this because participation in a crowdsale could be accomplished just with "Simple Send" but I think it's the right way to go (I also think in the DEx phase 1 revamp we should require a "Pay For Accept" payload but that's another topic).

Then again, I'd consider it as flaw that we can't use an empty OP_RETURN marker in these cases.

To be honest bud I kind of consider an empty OP_RETURN marker a flawed methodology anyway - why not use a payload with an explicit instruction such as "Pay Crowdsale" or "Pay Accept" - the cost is negligible and it removes ambiguity about the users intent (which IMO can only be a good thing).

Though also another point: BTC payments are very different from normal payload transactions.

I think that's my biggest issue - they don't need to be.  Why not make BTC payments just plain Omni transactions (with payload)?.  To note this is kind of what I'm playing with in BTC crowdsales at the mo, an Omni transaction "Pay Crowdsale" which carries BTC.
Thoughts?

Perhaps to simplify, instead of having multiple "Pay Crowdsale" or "Pay Accept" and so forth, we could have a single "Bitcoin Payment" payload.  Something like:
Transaction Type: 80 (Bitcoin Payment)
Transaction Ver: 0
TargetTxType: uint16
vOut: uint16
The vOut specifies exactly which output is considered the payment, and to whom (explicit).
The TargetTxType allows to specify what the payment is for - if the target tx type is 22 we know the payment is for an accept, if it's 51 we know the payment is for a crowdsale.  This allows BTC payments in other (eg future) tx types without expressly engineering for it each time.
Just thinking out loud as I'm prototyping :)

It'll need the "Investment Send" transaction to be brought back (renamed "Participate In Crowdsale" or similar, we should avoid nomenclature that has the term "invest" included).

Yes, yes, yes! :)
I'd really like this. Not only for BTC crowdsales, but also for crowdsales in general and DEx payments.

Something like: ...

What if we use some other identifier, such as the hash of the crowdsale/... transaction, which even more explicitly points to the actual target, instead of TargetTxType? In this case we may even allow multiple DEx orders from the same address (not saying we necessarily should).

Just thinking out loud as I'm prototyping :)

I love the direction!

I like this direction.

https://github.com/OmniLayer/omnicore/pull/399


Increase Receive Speed

Do you have any recommendations on set up to help increase receive distance using 3 anchors and one tag?

Hi @FXUDarkNova ,
by now (with DWM1000 as anchors and tag), there are few possibilities:

Following the specification about the PCB board. See dw1000 user manual.
Only use vertical polarization.
Disable "smartPower" and manually set the power at a step up than the default. To perform this, set false the third parameter from up in device_configuration_t struct. So, use setTXPower() API to set manually the transmission power. See dw1000 user manual before trying to do everything about power.

By the way, for every question, there is a response inside Decawave's manuals or application notes.
PS. maybe the title of this issue could change in "Increase receive distance".

Also be careful to check the current regulatory power limits for your country as you can easily get into illegal range if you don't know what you are doing.


[Inspector]: Show untokenized properties

@six7 proposing to close as its stale and if its needed it will come back up (:


can tc works with tpa?

we usually use tc to emulate network abnormal cases, since libtpa is a userspace tcp stack, so tc can't work with it ?

Yes, it won't work. However, Libtpa ships pktfuzz (located at src/pktfuzz), which can be used to simulate some very basic abnormal cases. Note that it might not work perfectly (it may have bugs), as we seldom run it. We don't really rely on it for the stableness. Instead, we use unit test to simulate all the abnormal inputs we think of.


Crash when rendering some pdfs.

I downloaded the pdf from https://tools.ietf.org/pdf/rfc7519.pdf . Then replace adobe.pdf with rfc7519.pdf.  It is very slow when rendering rfc7519.pdf. And crashed after a while. I found the culprit：
return renderer.openPage(position);
But other pdfs is fine, witch is evan larger than rfc7519.pdf.
You can reproduce it with the rfc7519.pdf.

This one is the same.
https://github.com/barteksc/AndroidPdfViewer/files/329537/pdfSample.pdf

This one is the same. pdfSample.pdf

Hello,
I've reproduced the issue in my phone, with your PDF. It also crashes in my phone. We will have a look to see if we can fix the issue in any way.

I found a similar issue about this, https://github.com/googlesamples/android-PdfRendererBasic/issues/1
Are you reading your pdf files directly from assets.
I suggest you to copy the files in your own SD card

No difference，from assets or SD card , all crash. But i found it not crash in some phones. I count not find the cause. When it crash , here is the log :
es.voghdev.pdfviewpager A/libc: Fatal signal 11 (SIGSEGV), code 1, fault addr 0x0 in tid 16545 (ev.pdfviewpager)
it look like it is missing '.so' libs.

Hello,
it looks like an internal error of PdfRenderer class. If the original PdfRenderer can't render your PDF file, I guess this library can't do anything about it, as it is mounted on top of it, sorry about that :-/
I suggest you to stay tuned on the issue that @josearmandoam linked, and let me know if they find a solution. I'll update the library with any changes that may be necessary


[cbs.com:playlist] Extract playlists of CBS shows.

[x] New feature

Support extracting playlists of full episode as well as episode clips from cbs.com.

Corrected error caught by Travis CI core tests, overlapping regexps.

Check code with flake8.

OK! I think we're set. Are we ready to squash?

One if (1 line) vs try except + RegexNotFoundError import (4 lines).

Done. But it's only one line shorter (try + except are 2, if + else are 2) because of the import. But why would we sacrifice code clarity for lines? Lines are cheap. Now somebody has to figure out how the default machinery is working...

If both are present it makes sense to extract both as well. This comment may be useful for end users but not for developers and it steels attention and takes time to read. So I vote for removing it from the source code.

I removed it as you directed, but I think it's really unfortunate to not have a place to put information like this. Sure, most endusers don't read the source code, but for the few that do, it's great to have it there. Maybe they can find it from the git history now :)
QUESTION: How do I run only_matching tests? I just commented out the only_matching line during development, but it seems like there should be a better way. I could not find one.

Seems incomplete. Testing with big bang theory only pulls a few episodes. Unrestricted access for episodes 15-19 is available, and the "subscription" links for 13 and 14 will work. But only episodes 16-19 are found.
keybounceMBP:CBS michael$ ./yt-dl --get-filename http://www.cbs.com/shows/big_bang_theory
[debug] System config: []
[debug] User config: ['-k', '-o', '%(title)s.%(ext)s', '-f', '\nbest[ext=mp4][height>431][height<=576]/\nbestvideo[ext=mp4][height=480]+bestaudio[ext=m4a]/\nbest[ext=mp4][height>340][height<=431]/\nbestvideo[ext=mp4][height>360][height<=576]+bestaudio/\nbest[height>340][height<=576]/\nbestvideo[height>360][height<=576]+bestaudio/\nbestvideo[height=360]+bestaudio/\nbest[ext=mp4][height>=280][height<=360]/\nbest[height<=576]/\nworst', '--ap-mso', 'Dish', '--ap-username', 'PRIVATE', '--ap-password', 'PRIVATE', '--write-sub', '--write-auto-sub', '--sub-lang', 'en,enUS,en-us', '--sub-format', 'ass/srt/best', '--convert-subs', 'ass', '--embed-subs', '--mark-watched', '--download-archive', 'downloaded-videos.txt']
[debug] Custom config: []
[debug] Command-line args: ['-v', '--hls-prefer-native', '-o', '%(series)s/s%(season_number)02d-e%(episode_number)02d - %(title)s.%(ext)s', '--get-filename', 'http://www.cbs.com/shows/big_bang_theory']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.07.02
[debug] Python version 3.6.1 - Darwin-13.4.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.2.4, ffprobe 3.2.4, rtmpdump 2.4
[debug] Proxy map: {}
WARNING: Failed to download m3u8 information: HTTP Error 404: Not Found
WARNING: enUS subtitles not available for 0Xvz7OlfV56M_4ejYwX2IMX6dcKefSml
WARNING: en-us subtitles not available for 0Xvz7OlfV56M_4ejYwX2IMX6dcKefSml
The Big Bang Theory/s10-e16 - The Big Bang Theory - The Allowance Evaporation.mp4
WARNING: Failed to download m3u8 information: HTTP Error 404: Not Found
WARNING: enUS subtitles not available for A7DCA92A-187E-CA0E-A3B9-626A0E0F5B31
WARNING: en-us subtitles not available for A7DCA92A-187E-CA0E-A3B9-626A0E0F5B31
The Big Bang Theory/s10-e17 - The Big Bang Theory - The Comic-Con Conundrum.mp4
WARNING: Failed to download m3u8 information: HTTP Error 404: Not Found
WARNING: enUS subtitles not available for HyZVvrlBo_Dx1yW4N3Dw5aUKReVm9Dq3
WARNING: en-us subtitles not available for HyZVvrlBo_Dx1yW4N3Dw5aUKReVm9Dq3
The Big Bang Theory/s10-e18 - The Big Bang Theory - The Escape Hatch Identification.mp4
WARNING: Failed to download m3u8 information: HTTP Error 404: Not Found
WARNING: enUS subtitles not available for _DwiBhcrlbDLeQqjJ_4R9MT0z9xS90ah
WARNING: en-us subtitles not available for _DwiBhcrlbDLeQqjJ_4R9MT0z9xS90ah
The Big Bang Theory/s10-e19 - The Big Bang Theory - The Collaboration Fluctuation.mp4

... and it fails with an error on The Late Show
keybounceMBP:CBS michael$ ./yt-dl --get-filename http://www.cbs.com/shows/the-late-show-with-stephen-colbert/
[debug] System config: []
[debug] User config: ['-k', '-o', '%(title)s.%(ext)s', '-f', '\nbest[ext=mp4][height>431][height<=576]/\nbestvideo[ext=mp4][height=480]+bestaudio[ext=m4a]/\nbest[ext=mp4][height>340][height<=431]/\nbestvideo[ext=mp4][height>360][height<=576]+bestaudio/\nbest[height>340][height<=576]/\nbestvideo[height>360][height<=576]+bestaudio/\nbestvideo[height=360]+bestaudio/\nbest[ext=mp4][height>=280][height<=360]/\nbest[height<=576]/\nworst', '--ap-mso', 'Dish', '--ap-username', 'PRIVATE', '--ap-password', 'PRIVATE', '--write-sub', '--write-auto-sub', '--sub-lang', 'en,enUS,en-us', '--sub-format', 'ass/srt/best', '--convert-subs', 'ass', '--embed-subs', '--mark-watched', '--download-archive', 'downloaded-videos.txt']
[debug] Custom config: []
[debug] Command-line args: ['-v', '--hls-prefer-native', '-o', '%(series)s/s%(season_number)02d-e%(episode_number)02d - %(title)s.%(ext)s', '--get-filename', 'http://www.cbs.com/shows/the-late-show-with-stephen-colbert/']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.07.02
[debug] Python version 3.6.1 - Darwin-13.4.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.2.4, ffprobe 3.2.4, rtmpdump 2.4
[debug] Proxy map: {}
WARNING: enUS subtitles not available for K5pgOWUaj_44NtRQ5XgMYGXOtXg2K0uL
WARNING: en-us subtitles not available for K5pgOWUaj_44NtRQ5XgMYGXOtXg2K0uL
The Late Show with Stephen Colbert/s02-e170 - The Late Show - 6_30_2017 (Michael Keaton, John McEnroe).mp4
WARNING: enUS subtitles not available for SpPBSIgnOvV6wcLuH2QmPHvPs3QY_S_4
WARNING: en-us subtitles not available for SpPBSIgnOvV6wcLuH2QmPHvPs3QY_S_4
The Late Show with Stephen Colbert/s02-e169 - The Late Show - 6_29_2017 (Naomi Watts, Ari Graynor, Swet Shop Boys).mp4
WARNING: enUS subtitles not available for E5uA9ix_atu_X4o9_DHPlbnScR0Mnj9b
WARNING: en-us subtitles not available for E5uA9ix_atu_X4o9_DHPlbnScR0Mnj9b
The Late Show with Stephen Colbert/s02-e168 - The Late Show - 6_28_2017 (Josh Duhamel, Justin Bartha, Brian Greene).mp4
WARNING: enUS subtitles not available for 2gzMOBnjAf_k0iKFrhJTQlmgubrogRnC
WARNING: en-us subtitles not available for 2gzMOBnjAf_k0iKFrhJTQlmgubrogRnC
The Late Show with Stephen Colbert/s02-e167 - The Late Show - 6_27_2017 (Eric Stonestreet, John McEnroe, Lillie Mae).mp4
WARNING: enUS subtitles not available for _PA2rOl8J85BsOlgvPXOuqftGXjKDShO
WARNING: en-us subtitles not available for _PA2rOl8J85BsOlgvPXOuqftGXjKDShO
The Late Show with Stephen Colbert/s02-e166 - The Late Show - 6_26_2017 (Michael Keaton, Zoe Kazan, Tom Shillue).mp4
WARNING: enUS subtitles not available for 5vCxVEjeeBLaV_eEr7srVsjkajLJB9ow
WARNING: en-us subtitles not available for 5vCxVEjeeBLaV_eEr7srVsjkajLJB9ow
The Late Show with Stephen Colbert/s02-e165 - The Late Show - 6_21_2017 (Jeffrey Tambor, John Benjamin Hickey, Lake Street Dive).mp4
ERROR: None of the available releases match the specified AssetType and/or Format preferences
Traceback (most recent call last):
  File "/Users/michael/bin/youtube-dl.new/youtube_dl/YoutubeDL.py", line 762, in extract_info
    ie_result = ie.extract(url)
  File "/Users/michael/bin/youtube-dl.new/youtube_dl/extractor/common.py", line 433, in extract
    ie_result = self._real_extract(url)
  File "/Users/michael/bin/youtube-dl.new/youtube_dl/extractor/cbs.py", line 190, in _real_extract
    return self._extract_video_info(content_id)
  File "/Users/michael/bin/youtube-dl.new/youtube_dl/extractor/cbs.py", line 169, in _extract_video_info
    'Downloading %s SMIL data' % asset_type)
  File "/Users/michael/bin/youtube-dl.new/youtube_dl/extractor/theplatform.py", line 42, in _extract_theplatform_smil
    raise ExtractorError(error_element.attrib['abstract'], expected=True)
youtube_dl.utils.ExtractorError: None of the available releases match the specified AssetType and/or Format preferences

keybounceMBP:CBS michael$

Feature request: a way to only fetch the episodes, and not the clips/interviews.

Feature request: a way to only fetch the episodes, and not the clips/interviews.

Already covered:
https://github.com/johnhawkinson/youtube-dl/blob/80df786df990bef1e801f56dcc3a8f48d51dddd0/youtube_dl/extractor/cbs.py#L112-113
Specifically youtube_dl --match-filter 'episode_number' http://...

Thank you

Did not work for me -- still wanted to fetch all 29 videos and clips for Colbert.
keybounceMBP:CBS michael$ ./yt-dl -i -v --match-filter 'episode_number' http://www.cbs.com/shows/the-late-show
-with-stephen-colbert/
[debug] System config: []
[debug] User config: ['-k', '--hls-prefer-native', '-o', '%(series)s/s%(season_number)02d-e%(episode_number)02d-%(title)s.%(ext)s', '-f', '\nbest[ext=mp4][height>431][height<=576]/\nbestvideo[ext=mp4][height=480]+bestaudio[ext=m4a]/\nbest[ext=mp4][height>340][height<=431]/\nbestvideo[ext=mp4][height>360][height<=576]+bestaudio/\nbest[height>340][height<=576]/\nbestvideo[height>360][height<=576]+bestaudio/\nbestvideo[height=360]+bestaudio/\nbest[ext=mp4][height>=280][height<=360]/\nbest[height<=576]/\nworst', '--ap-mso', 'Dish', '--ap-username', 'PRIVATE', '--ap-password', 'PRIVATE', '--write-sub', '--write-auto-sub', '--sub-lang', 'en,enUS,en-us', '--sub-format', 'ass/srt/best', '--convert-subs', 'ass', '--embed-subs', '--mark-watched', '--download-archive', 'downloaded-videos.txt']
[debug] Custom config: []
[debug] Command-line args: ['-v', '--hls-prefer-native', '-o', '%(series)s/s%(season_number)02d-e%(episode_number)02d - %(title)s.%(ext)s', '-i', '-v', '--match-filter', 'episode_number', 'http://www.cbs.com/shows/the-late-show-with-stephen-colbert/']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.07.02
[debug] Python version 3.6.1 - Darwin-13.4.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.2.4, ffprobe 3.2.4, rtmpdump 2.4
[debug] Proxy map: {}
[cbs.com:playlist] the-late-show-with-stephen-colbert: Downloading webpage
[cbs.com:playlist] Downloading clips carousel: Downloading JSON metadata
[cbs.com:playlist] Downloading episodes carousel: Downloading JSON metadata
[download] Downloading playlist: The Late Show with Stephen Colbert
[cbs.com:playlist] playlist The Late Show with Stephen Colbert: Collected 29 video ids (downloading 29 of them)
[download] Downloading video 1 of 29
[download] 7/10/2017 (Woody Harrelson, Coble Smulders, Emmylou Harris) has already been recorded in archive
[download] Downloading video 2 of 29
[download] 6/30/2017 (Michael Keaton, John McEnroe) has already been recorded in archive
[download] Downloading video 3 of 29
[download] 6/29/2017 (Naomi Watts, Ari Graynor, Swet Shop Boys) has already been recorded in archive
[download] Downloading video 4 of 29
[download] 6/28/2017 (Josh Duhamel, Justin Bartha, Brian Greene) has already been recorded in archive
[download] Downloading video 5 of 29
[download] 6/27/2017 (Eric Stonestreet, John McEnroe, Lillie Mae) has already been recorded in archive
[download] Downloading video 6 of 29
[download] 6/26/2017 (Michael Keaton, Zoe Kazan. Tom Shillue) has already been recorded in archive
[download] Downloading video 7 of 29
[download] 6/21/2017 (Jeffrey Tambor, John Benjamin Hickey, Lake Street Dive) has already been recorded in archive
[download] Downloading video 8 of 29
[CBS] Ve1PQPLiu8GD5Uj1YP4Sv34W2K0NDBWM: Downloading XML
[CBS] Ve1PQPLiu8GD5Uj1YP4Sv34W2K0NDBWM: Downloading OnceURL SMIL data
ERROR: None of the available releases match the specified AssetType and/or Format preferences
Traceback (most recent call last):
  File "/Users/michael/bin/youtube-dl.new/youtube_dl/YoutubeDL.py", line 762, in extract_info
    ie_result = ie.extract(url)
  File "/Users/michael/bin/youtube-dl.new/youtube_dl/extractor/common.py", line 433, in extract
    ie_result = self._real_extract(url)
  File "/Users/michael/bin/youtube-dl.new/youtube_dl/extractor/cbs.py", line 190, in _real_extract
    return self._extract_video_info(content_id)
  File "/Users/michael/bin/youtube-dl.new/youtube_dl/extractor/cbs.py", line 169, in _extract_video_info
    'Downloading %s SMIL data' % asset_type)
  File "/Users/michael/bin/youtube-dl.new/youtube_dl/extractor/theplatform.py", line 42, in _extract_theplatform_smil
    raise ExtractorError(error_element.attrib['abstract'], expected=True)
youtube_dl.utils.ExtractorError: None of the available releases match the specified AssetType and/or Format preferences

[download] Downloading video 9 of 29
[download] 6/19/2017 (Seth Rogen, Kumail Nanjiani, Paul Shaffer & The World's Most Dangerous Band) has already been recorded in archive
[download] Downloading video 10 of 29
[download] 6/16/2017 (Scarlett Johansson, Bill Burr, Fleet Foxes) has already been recorded in archive
[download] Downloading video 11 of 29
[download] 6/15/2017 (Milo Ventimiglia, Judy Greer, Keith Alberstadt) has already been recorded in archive
[download] Downloading video 12 of 29
[download] 6/14/2017 (Trevor Noah, Ilana Glazer, Sam Richardson) has already been recorded in archive
[download] Downloading video 13 of 29
[download] 6/13/2017 (Olivia Wilde, Eddie Izzard, Nick Cave) has already been recorded in archive
[download] Downloading video 14 of 29
[download] 6/12/2017 (Uzo Aduba, T.J. Miller, Oliver Stone) has already been recorded in archive
[download] Downloading video 15 of 29
[download] Mr. President Wasn't Mr. Popular At The G20 Summit has already been recorded in archive
[download] Downloading video 16 of 29
[download] Cobie Smulders Is Totally Not A Made-Up Name has already been recorded in archive
[download] Downloading video 17 of 29
[download] Emmylou Harris & Her Red Dirt Boys Perform 'Pilgrim' has already been recorded in archive
[download] Downloading video 18 of 29
[download] Woody Harrelson Gets 'Han Solo' Details Squeezed Out Of Him has already been recorded in archive
[download] Downloading video 19 of 29
[download] North Korea's Missiles Can Reach Alaska. Alaska Doesn't Care. has already been recorded in archive
[download] Downloading video 20 of 29
[download] Kellyanne Conway Gives It To Stephen Straight (As In, Straight Up BS) has already been recorded in archive
[download] Downloading video 21 of 29
[download] Stephen Stars As Donald Trump Jr. In 'Collusion' has already been recorded in archive
[download] Downloading video 22 of 29
[download] One Week Older, June 30 has already been recorded in archive
[download] Downloading video 23 of 29
[download] Michael Keaton Interview, Part 2 has already been recorded in archive
[download] Downloading video 24 of 29
[download] People Can't Stand Martin Shkreli's Face has already been recorded in archive
[download] Downloading video 25 of 29
[download] Trump Has United The Planet... Against Him has already been recorded in archive
[download] Downloading video 26 of 29
[download] This Season's Must-Have Fireworks has already been recorded in archive
[download] Downloading video 27 of 29
[download] What Is A 'Bona Fide Relationship,' Anyway? has already been recorded in archive
[download] Downloading video 28 of 29
[download] Swet Shop Boys Perform 'T5' has already been recorded in archive
[download] Downloading video 29 of 29
[download] Ari Graynor Went To Prom With A Kennedy, Humblebrag has already been recorded in archive
[download] Finished downloading playlist: The Late Show with Stephen Colbert
keybounceMBP:CBS michael$

Has this functionality been added to the master branch at all?


Webhook adapter to allow custom headers

This allows web hooks to send things like X-App-Event or whatever.

Probably should allow different verbs for web hooks too.


Move numeric_types.py from fud/verilator to calyx-py (another attempt at #1715 )

This is another attempt at #1715. I merged #1715 without realizing that it would mess up Docker (see this error).
Therefore, I reverted the merge. I have reopened another PR that does the exact same thing as #1715. I would like to merge it, but first have to figure out why the Docker stuff was failing.
Apologies for this being so confusing.

@calebmkim Anything blocking us from merging this?

Docker fails when I tried to push this.
When it tries to build Docker it gives an error: ModuleNotFoundError: No module named 'calyx'. Which is from the new fud dependency on calyx.numeric_types

@calebmkim let's try to get this merged? Hopefully the new docker image stuff allows this to be tested correctly?


vcpkg，并没有用github下载

我刚刚在用vcpkg的时候，发现并没有在用github下载，是不是哪里出错了

@RainbowTaro2022, Thanks for posting this issue, could you please provide me a screenshot of your issue?

In fact, partial packages are downloaded from github, like fmt or gsl.

右边是我正在进行的任务，左边没有显示
我用的是fastgithub

右边是我正在进行的任务，左边没有显示
我用的是fastgithub

It's normal,gsl is downloaded from gnu for everyone

This is normal, not all vcpkg libraries are downloaded from github, and some libraries are downloaded from gitlab or other official websites.


more fixes for anchor not found errors

I've just run the website build and the link checker and did not find any broken links or any warnings or errors pointing to any link issues. The only issues from Antora were the The WARNING: skipping reference to missing attribute which will be fixed with the changes you proposed but only temporarily as the next documentation generation from the Javadoc will switch those escaped braces back to their un-escaped variants, so those need to be handled differently, either by changing the Javadoc or adding additional escaping logic to the Maven plugin that generates the Asciidoc files.
You can take a look at the escape method in MvelHelper and tweak it there:
https://github.com/apache/camel/blob/eba3b85f55176d412d5de4ed4e239bd2595686c0/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/MvelHelper.java#L33-L42
And you can add one of the escape cases to the unit test here:
https://github.com/apache/camel/blob/eba3b85f55176d412d5de4ed4e239bd2595686c0/tooling/maven/camel-package-maven-plugin/src/test/java/org/apache/camel/maven/packaging/MvelHelperTest.java#L56-L66
From this I think we can take the anchor naming changes in spring-boot.adoc, test-spring.adoc, contributing.adoc and the link correction in unmarshal-eip.adoc.
How did you find out about these issues here?

Hello Mr.Zoran, I did add additional escaping logic to the Maven plugin that generates the Asciidoc files. But there is a little issue I would like to clarify, I search for escaped '$' signs in the camel/docs there are 10 files have escaped dollar signs '$' so I changed them to unescaped dollar signs to check whether It gives any error or warning in yarn build. There were no errors or warnings came. So when my additional escaping logic finds such a  string like '${hello world}' it will convert it to '${hello world}'. If we remove escape dollar sign logic it will be like '${hello world}'. I would like to know your thoughts on this matter.


No tweets returned from Twitter, BS4 fails parsing

After setting up the project und running server.py, the python script returns an error saying that "title" is "NoneType", but expecting it to be a string!
I guess the reason is that Twitter is not returning any tweets!
Here is the stack trace:
`[...]
File "/home/kali/Desktop/twitter-rss/twitter-rss/server.py", line 34, in feed_to_xml
return twitter_rss.FeedManager(config).get(kind, feed)
File "/home/kali/Desktop/twitter-rss/twitter-rss/twitter_rss.py", line 214, in get
tweets = self.HANDLERS[kind](feed, self.pictures)
File "/home/kali/Desktop/twitter-rss/twitter-rss/twitter_rss.py", line 158, in init
super(UserTweetGetter, self).init("https://twitter.com/{0}", username, pictures=pictures)
File "/home/kali/Desktop/twitter-rss/twitter-rss/twitter_rss.py", line 121, in init
self.parse_twitter()
File "/home/kali/Desktop/twitter-rss/twitter-rss/twitter_rss.py", line 132, in parse_twitter
self.title = soup.title.string
AttributeError: 'NoneType' object has no attribute 'string'`
Maybe we can find a quick solution?

If you debug the app and look inside of the variable "content.text" which is the HTML page, you will see a lot crap, but no tweets:

We've detected that JavaScript is disabled in your browser. Would you like to proceed to legacy Twitter?

Yes

Something went wrong, but don’t fret — let’s give it another shot.


OSError: [Errno 24] Too many open files (with num_parallel 64)

WIth --num_parallel 64, too much? I have 32 cores in my machine.
Traceback (most recent call last):
  File "/home/tiny/build/nv_isa_solver/nv_isa_solver/instruction_solver.py", line 1339, in analyse_operand_interactions
    interaction_data, self.operand_interaction_raw = analyse_live_ranges(
  File "/home/tiny/build/nv_isa_solver/nv_isa_solver/life_range.py", line 153, in analyse_live_ranges
    result = get_live_ranges(tmp.name)
  File "/home/tiny/build/nv_isa_solver/nv_isa_solver/life_range.py", line 123, in get_live_ranges
    proc = subprocess.run(
  File "/usr/lib/python3.10/subprocess.py", line 503, in run
    with Popen(*popenargs, **kwargs) as process:
  File "/usr/lib/python3.10/subprocess.py", line 837, in __init__
    errread, errwrite) = self._get_handles(stdin, stdout, stderr)
  File "/usr/lib/python3.10/subprocess.py", line 1643, in _get_handles
    errread, errwrite = os.pipe()
OSError: [Errno 24] Too many open files

Less parallel seems okay.

Fixed with less parallel

We automatically spawn nvdisasm processes based on the number of cores you have. the --num_parallel here specifies how many instructions we should solve in parallel. You should probably use something like 3


Add support for PseudoDojo PAW pseudos

Discussed in issue #40
Because only the Lanthanides are missing cutoffs in the PseudoDojo PAW families, which in general are not available in the other PseudoDojo families, I simply remove the elements with missing (placeholder, value -1) cutoffs from the PseudoDojoFamily during installation and warn the user that I'm doing it.

Superseded by #42

Superseded by #42


[TW#13322] Typo in documentation

in /components/driver/include/driver/gpio.h
line 139:
@note GPIO34..29 are input mode only. */
should be changed to
@note GPIO34..39 are input mode only. */

Hi @elfalko,
Thank you for spotting this typo.
Krzysztof


crash when no payload package is received

Crash occurred while block-wish was receiving data

@okwendell  Could you test it with branch fixSegBWT if it solve the issue ?

ok

It's working fine now. thx.


Propagate permission denied errors

Simply emit an event when this occurs.

I will elaborate on this in the morning based the feedback from the other PR and fix the lint error as well.

Check the last changes in the repository. Lint errors are fixed and an error event implemented. However I'd like to hear opinions on whether throwing the error if no error event handlers are found is a good idea or not.

Ah, looks good to me!


System Lifecycle Rework: all Systems are initialized by definition

Note: This is just very rough experimentation. I (currently) have no intention to move this forward as-is ... I just created it because people are discussing reworking system descriptors and scheduling right now and this change is something that relates / that I've been wanting to explore for awhile.
This does the following:

Systems are "initialized" by definition. The "system initialization" lifecycle has been completely removed. IntoSystem takes a world reference and is responsible for initializing the system.
FunctionSystems now use SystemState internally, unifying them as alluded to in #2283. The impl is much smaller / simpler now
SystemDescriptors / ExclusiveSystemDescriptors now store functions that take a World and return a System.
Adding a SystemDescriptor to a Schedule therefore requires a World reference. For Apps, this means users don't need to care because we can abstract this out. For raw Schedules / Stages, naively this means they need to accept a World reference, which significantly affects ergonomics. If we were to move forward with this, we would (1) remove all IntoSystem-related methods from Schedule and Stage in favor of accepting "boxed ready-to-go" systems. This would make them much simpler / might improve compile times (2) for direct bevy_ecs users that can't use App, add relevant StageBuilder and ScheduleBuilder types that abstract out the World requirement. alternatively we could just merge bevy_app into bevy_ecs.

The things this accomplishes:

One less thing Systems need to do
Simpler / smaller implementation that allows code reuse
Systems no longer need to store their param state in an Option or unwrap it on every run
Systems no longer need to store their Config indefinitely. It is only in memory when the app is being built.
Things running systems and run criteria (like the parallel executor) no longer need to worry about ensuring everything is initialized, which makes the code smaller/simpler/mistake proof.

This comes at the cost of:

Systems types cannot be created and stored without a World. This makes some patterns harder (ex: see the changes made to the old renderer's graph init ... this was quick and dirty. Fortunately we dont need that pattern in the new renderer).  People that really need this pattern (which honestly should be discouraged generally) can just use SystemDescriptors instead of boxed Systems.
Schedules and Stages cannot add new systems without a World reference. This makes sense to me because they are the "baked and ready to go" types. In situations where a world isn't available, we could have intermediate types like StageDescriptor. But I expect App to meet basically everyone's needs in practice.

This touches a lot of stuff and most of it is just noise. The most relevant changes are in function_system.rs,  the System trait, system descriptors, and the executor.

alternatively we could just merge bevy_app into bevy_ecs.

I'd prefer not to do this. The standalone bevy_ecs users that I've talked to tend to have very unusual data-flow patterns and the current Schedule abstractions tends not to work well for them.
They also tend to care about compile sizes and dependencies in ways that other Bevy users don't.

Worth considering but this is way out of date and would need significant reworks at this point.


windows: no nifs found compilation failures when running tests

If I try to test a file or simply run mix test, it will give a compilation error one of the test files.  You can keep trying and get a different file most times (due to the parallel compiler).
❯ mix test .\test\support\zigtest\passing_tests.ex
Compiling 35 files (.ex)

11:45:29.921 [warn]  windows is not supported, but may work.

If you find an error in the process, please leave an issue at:
https://github.com/ityonemo/zigler/issues

11:45:29.928 [warn]  windows is not supported, but may work.

If you find an error in the process, please leave an issue at:
https://github.com/ityonemo/zigler/issues

11:45:29.935 [warn]  windows is not supported, but may work.

If you find an error in the process, please leave an issue at:
https://github.com/ityonemo/zigler/issues

11:45:29.942 [warn]  windows is not supported, but may work.

If you find an error in the process, please leave an issue at:
https://github.com/ityonemo/zigler/issues

11:45:29.950 [warn]  windows is not supported, but may work.

If you find an error in the process, please leave an issue at:
https://github.com/ityonemo/zigler/issues

== Compilation error in file test/support/zigtest/documentation.ex ==
** (CompileError) test/support/zigtest/documentation.ex: no nifs found
in the module Elixir.ZiglerTest.Integration.Documentation
    expanding macro: Zig.Compiler.__before_compile__/1
    test/support/zigtest/documentation.ex:1: ZiglerTest.Integration.Documentation (module)
    (elixir 1.12.2) lib/kernel/parallel_compiler.ex:319: anonymous fn/4 in Kernel.ParallelCompiler.spawn_workers/7

again
❯ mix test .\test\support\zigtest\passing_tests.ex
Compiling 35 files (.ex)

...

== Compilation error in file test/support/zigtest/failing_test.ex ==
** (CompileError) test/support/zigtest/failing_test.ex: no nifs found in the module Elixir.ZiglerTest.ZigTest.FailingTest
    expanding macro: Zig.Compiler.__before_compile__/1
    test/support/zigtest/failing_test.ex:1: ZiglerTest.ZigTest.FailingTest (module)
    (elixir 1.12.2) lib/kernel/parallel_compiler.ex:319: anonymous fn/4 in Kernel.ParallelCompiler.spawn_workers/7

again
❯ mix test .\test\support\zigtest\passing_tests.ex
Compiling 35 files (.ex)

...

== Compilation error in file test/support/types.ex ==
** (CompileError) test/support/types.ex: no nifs found
in the module Elixir.ZiglerTest.Types
    expanding macro: Zig.Compiler.__before_compile__/1
    test/support/types.ex:1: ZiglerTest.Types (module)
    (elixir 1.12.2) lib/kernel/parallel_compiler.ex:319: anonymous fn/4 in Kernel.ParallelCompiler.spawn_workers/7

It is because the parser does not handle CRLF, just LF.  As a quick and dirty workaround I added the below to Parser.parse().  I'll try update the parser to handle both line endings.
code =
  if :os.type() |> elem(0) == :win32 do
    String.replace(code, "\r\n", "\n")
  else
    code
  end

I got this error after that change.
== Compilation error in file test/support/zigtest/passing_tests.ex ==
** (CompileError)  this zig compiler warning hasn't been incorporated into the parser.
Please file a report at:
https://github.com/ityonemo/zigler/issues
Assertion failed at /home/andy/dev/bootstrap-zig/zig/src/stage1/analyze.cpp:8061 in get_builtin_value. This is a bug in the Zig compiler.
Unable to dump stack trace: debug info stripped

    lib/zig/parser/error.ex:54: Zig.Parser.Error.parse/2
    lib/zig/command.ex:36: Zig.Command.compile/2
    lib/zig/compiler.ex:103: Zig.Compiler.compilation/2
    expanding macro: Zig.Compiler.__before_compile__/1
    test/support/zigtest/passing_tests.ex:1: ZiglerTest.ZigTest.PassingTests (module)

and then the LLVM error, which should be fixed in 0.8.0 🤷
== Compilation error in file test/support/zigtest/transitive.ex ==
** (CompileError)  this zig compiler warning hasn't been incorporated into the parser.
Please file a report at:
https://github.com/ityonemo/zigler/issues
LLVM ERROR: Bad $ operand number in inline asm string: ' movl %fs:0x18, $104801965141131264'

    lib/zig/parser/error.ex:54: Zig.Parser.Error.parse/2
    lib/zig/command.ex:36: Zig.Command.compile/2
    lib/zig/compiler.ex:102: Zig.Compiler.compilation/2
    expanding macro: Zig.Compiler.__before_compile__/1
    test/support/zigtest/transitive.ex:1: ZiglerTest.ZigTest.Transitive (module)

oh wow cool, thanks for the catch.  As it turns out cr/lf is prohibited in zig code itself, so I'll have to process that anyways.  are you by chance on windows?

I am.
I think we might need to upgrade to 0.8.0 to get us past the next error though.

after that, if you look at erl_nif.h there are some things that are "windows" only, for example this line:
memcpy(&WinDynNifCallbacks,callbacks,sizeof(TWinDynNifCallbacks))

which will need to be implemented in zig.

ok I'm playing around with it, 0.8.1 might support windows, gonna give one shot at testing it manually, but otherwise going to YOLO it through CI, I'll tag you if you want to take a gander at it.

Well we get a new error now 😃
❯ mix test
Compiling 34 files (.ex)

14:24:58.138 [warn]  windows is not supported, but may work.

If you find an error in the process, please leave an issue at:
https://github.com/ityonemo/zigler/issues

14:24:58.111 [warn]  windows is not supported, but may work.

If you find an error in the process, please leave an issue at:
https://github.com/ityonemo/zigler/issues

14:24:58.124 [warn]  windows is not supported, but may work.

If you find an error in the process, please leave an issue at:
https://github.com/ityonemo/zigler/issues

14:24:58.131 [warn]  windows is not supported, but may work.

If you find an error in the process, please leave an issue at:
https://github.com/ityonemo/zigler/issues

14:24:58.118 [warn]  windows is not supported, but may work.

If you find an error in the process, please leave an issue at:
https://github.com/ityonemo/zigler/issues

14:24:58.213 [debug] compiling nif for module ZiglerTest.ZigTest.FailingTest in path C:/Users/simon/AppData/Local/Temp/.zigler_compiler/test/Elixir.ZiglerTest.ZigTest.FailingTest

14:24:58.213 [debug] compiling nif for module ZiglerTest.ZigTest.PassingTests in path C:/Users/simon/AppData/Local/Temp/.zigler_compiler/test/Elixir.ZiglerTest.ZigTest.PassingTests

14:24:58.247 [debug] copied c:/Users/simon/code/zigler/test/support/zigtest/imported.zig to c:/Users/simon/AppData/Local/Temp/.zigler_compiler/test/Elixir.ZiglerTest.ZigTest.Transitive/imported.zig

14:24:58.250 [debug] copied c:/Users/simon/code/zigler/test/support/zigtest/namespaced.zig to c:/Users/simon/AppData/Local/Temp/.zigler_compiler/test/Elixir.ZiglerTest.ZigTest.Transitive/namespaced.zig

14:24:58.253 [debug] copied c:/Users/simon/code/zigler/test/support/zigtest/nonpublic.zig to c:/Users/simon/AppData/Local/Temp/.zigler_compiler/test/Elixir.ZiglerTest.ZigTest.Transitive/nonpublic.zig

14:24:58.253 [debug] compiling nif for module ZiglerTest.ZigTest.Transitive in path C:/Users/simon/AppData/Local/Temp/.zigler_compiler/test/Elixir.ZiglerTest.ZigTest.Transitive

14:24:58.328 [debug] wrote build.zig to C:/Users/simon/AppData/Local/Temp/.zigler_compiler/test/Elixir.ZiglerTest.ZigTest.Transitive/build.zig

14:24:58.329 [debug] wrote build.zig to C:/Users/simon/AppData/Local/Temp/.zigler_compiler/test/Elixir.ZiglerTest.ZigTest.PassingTests/build.zig

14:24:58.331 [debug] wrote build.zig to C:/Users/simon/AppData/Local/Temp/.zigler_compiler/test/Elixir.ZiglerTest.ZigTest.FailingTest/build.zig

== Compilation error in file test/support/zigtest/transitive.ex ==
** (CompileError)  this zig compiler warning hasn't been incorporated into the parser.
Please file a report at:
https://github.com/ityonemo/zigler/issues
C:\Users\simon\code\zigler\zig\zig-windows-i386-0.8.1\lib\std\builtin.zig:13:28: error: container 'builtin' has no member called 'is_test'
pub const is_test = builtin.is_test;
                           ^
C:\Users\simon\code\zigler\zig\zig-windows-i386-0.8.1\lib\std\special\compiler_rt.zig:8:24: note: referenced here
const is_test = builtin.is_test;
                       ^
C:\Users\simon\code\zigler\zig\zig-windows-i386-0.8.1\lib\std\special\compiler_rt.zig:17:25: note: referenced here
    const linkage = if (is_test) builtin.GlobalLinkage.Internal else builtin.GlobalLinkage.Weak;
                        ^
C:\Users\simon\code\zigler\zig\zig-windows-i386-0.8.1\lib\std\builtin.zig:18:27: error: container 'builtin' has no member called 'target'
pub const target = builtin.target;
                          ^
C:\Users\simon\code\zigler\zig\zig-windows-i386-0.8.1\lib\std\target.zig:1251:32: note: referenced here
    pub const current = builtin.target;
                               ^
C:\Users\simon\code\zigler\zig\zig-windows-i386-0.8.1\lib\std\special\compiler_rt\atomics.zig:8:24: note: referenced here
const arch = std.Target.current.cpu.arch;
                       ^
C:\Users\simon\code\zigler\zig\zig-windows-i386-0.8.1\lib\std\special\compiler_rt\atomics.zig:17:37: note: referenced here
const supports_atomic_ops = switch (arch) {
                                    ^
C:\Users\simon\code\zigler\zig\zig-windows-i386-0.8.1\lib\std\special\compiler_rt\atomics.zig:123:9: note: referenced here
    if (supports_atomic_ops) {
        ^
C:\Users\simon\code\zigler\zig\zig-windows-i386-0.8.1\lib\std\start.zig:30:16: error: container 'builtin' has no member called 'zig_is_stage2'
    if (builtin.zig_is_stage2) {
               ^

    lib/zig/parser/error.ex:54: Zig.Parser.Error.parse/2
    lib/zig/command.ex:30: Zig.Command.compile/2
    lib/zig/compiler.ex:99: Zig.Compiler.compilation/2
    expanding macro: Zig.Compiler.__before_compile__/1
    test/support/zigtest/transitive.ex:1: ZiglerTest.ZigTest.Transitive (module)

unhandled zig compiler warnings are due to the error parser failing on Windows paths, as they have : in them and then find_file fails as they don't start with /.


Display HOLC value on the chart

Good day,
is there any way to display the HOLC values on the left corner of the lightweight chart?
Thank you

I would recommend looking at this tutorial guide as a starting point: https://tradingview.github.io/lightweight-charts/tutorials/how_to/legends

@SlicedSilver thank you for your support, it is done!


Do you have a Discord account?

subject
I have a driver that I want to see if you can work on it.

yes<EMAIL_ADDRESS>

I sent one to smokingpc. Is that you?


Stream layer sample: ClientWebSocket.ReceiveAsync may need to loop over multiple chunks

https://github.com/Esri/arcgis-maps-sdk-unity-samples/blob/d3cedb81c55d939299b4a452300ee1691e3335bc/samples_project/Assets/SampleViewer/Samples/StreamLayer/StreamLayerWebSocketSubscribe.cs#L148
Should probably be implemented as a loop in order to make sure all the payload for a message is gathered.
More info here: https://stackoverflow.com/questions/23773407/a-websockets-receiveasync-method-does-not-await-the-entire-message

@markdeaton Thanks for raising this issue. The fix got merged.

forgot to attach the PR but here was it https://github.com/Esri/arcgis-maps-sdk-unity-samples/pull/90 @markdeaton

Closing out this issue as it has been fixed!


ng serve failing after updating CLI to latest version

Bug Report or Feature Request (mark with an x)
- [x ] bug report -> please search issues before submitting
- [ ] feature request

Command (mark with an x)
- [ ] new
- [ ] build
- [x ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Versions

Angular CLI: 6.2.3
Node: 8.11.3
OS: win32 x64
Angular: 6.1.9
... common, compiler, compiler-cli, core, forms, http
... platform-browser, platform-browser-dynamic, router
Package                           Version
@angular-devkit/architect             0.8.3
@angular-devkit/build-angular     0.8.3
@angular-devkit/build-optimizer  0.8.3
@angular-devkit/build-webpack   0.8.3
@angular-devkit/core                    0.8.3
@angular-devkit/schematics         0.8.3
@angular/cli                                   6.2.3
@ngtools/webpack                        6.2.3
@schematics/angular                    0.8.3
@schematics/update                     0.8.3
rxjs                                                 6.3.3
typescript                                       2.9.2
webpack                                        4.20.2
Repro steps

Updated local version of the CLI to the latest version and updated npm to latest.
Fixed all dependency errors it was complaining about and then update core to latest.
The log given by the failure

91% additional asset processing scripts-webpack-plugin× ｢wdm｣: Error: EISDIR: illegal operation on a directory, read
at Object.fs.readSync (fs.js:675:18)
at tryReadSync (fs.js:540:20)
at Object.fs.readFileSync (fs.js:583:19)
at Storage.provideSync (C:\Projects\Tradify\TradifyWeb2\node_modules\enhanced-resolve\lib\CachedInputFileSystem.js:98:13)
Desired functionality

Mention any other details that might be useful

This seems like a bug but we'll need to look at a reproduction to find and fix the problem. Can you setup a minimal repro please?
You can read here why this is needed. A good way to make a minimal repro is to create a new app via ng new repro-app and adding the minimum possible code to show the problem. Then you can push this repository to github and link it here.
This might be related to your directory structure so its really important to get an accurate repro to diagnose this.

I am having the same issue. Upgrading from 0.8.1 to 0.8.3, building or serving with or without --aot give the error:
Cannot find module 'merge-descriptors'
If I proceed to add that module manually, I get another error:
Cannot find module 'finalhandler'
And then again:
Cannot find module 'path-to-regexp'
And again:
Cannot find module 'methods'
Yet again:
Cannot find module 'content-disposition'
Then I honestly stopped and reverted back to 0.8.1.

This seems like a bug but we'll need to look at a reproduction to find and fix the problem. Can you setup a minimal repro please?
You can read here why this is needed. A good way to make a minimal repro is to create a new app via ng new repro-app and adding the minimum possible code to show the problem. Then you can push this repository to github and link it here.
This might be related to your directory structure so its really important to get an accurate repro to diagnose this.

Sure thing, i will get on that. If this does not work, i will share the full repo.

@DeonBonthuys I m facing the same error could you please point me to that line where u find the folder instead of file?
angular.txt


Put up as AAR in maven central

Excellent library..love this concept. But can you put this up as AAR in mavencentral?

+1


update scripts mod name

update scripts and ui mod name for compiling with PD.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@6358594). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1117   +/-   ##
=========================================
  Coverage          ?   23.12%
=========================================
  Files             ?      102
  Lines             ?     9981
  Branches          ?        0
=========================================
  Hits              ?     2308
  Misses            ?     7523
  Partials          ?      150

Flag
Coverage Δ

be_integration_test_latest
7.05% <0.00%> (?)

be_integration_test_v4.0.1
7.05% <0.00%> (?)

be_unit_test
24.13% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 6358594...5038b70. Read the comment docs.

Close this PR because we fix the PD compile issue by the better solution in PR https://github.com/tikv/pd/pull/4502


CMS Toolbar Not Working on Mobile

Summary
In the mobile and tablet views the 'More...' link in the Django CMS toolbar,

,that opens up the tools for editing the page does not work, therefore pages cannot be edited, published, unpublished, etc on mobile

Expected behaviour
Clicking the 'More...' link expands editing tools for page
Actual behaviour
Clicking the 'More...' link has seemingly no effect, no tools are opened
Environment

Python version: 2.7.10
Django version: 1.9.9
django CMS version: 3.3.2

Hi, could you provide a reduced test case to reproduce and a bit more information on browser / OS ?
I've just tried at http://demo.django-cms.org and it seems to be working correctly.

Hiya,
I'm seeing this in Chrome 60.0.3112.113 // Macintosh OS X 10.10.5 personally, but have had a few others with different set ups test and be able to reproduce.
I have a paired down example on hand, but as it requires logging in to the CMS is it all possible to do a screenshare? Thanks

Could you try removing the css from your site and try to see if it's accessible like that? If it is, then the problem is there.
Sadly screenshare won't be possible


Update limits-specifications-teams.md

Added notes to the table for clarity

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.


:bug: Adding tags back after fixing builtin config for binary analysis

Notice that this requires https://github.com/konveyor/analyzer-lsp/pull/671 and I will run a test there with both PR's and then we can merge.

Merging as the changes need github.com/konveyor/analyzer-lsp/pull/671


Validation of required non-nullable model properties

Is this a Bug or Feature request?:
Bug as it seems to contradict official documentation and is surprising behavior.
Steps to reproduce (preferrably a link to a GitHub repo with a repro project):
Create a new project dotnet new webapi and modify a controller as follows:
    [Route("api/[controller]")]
    public class ValuesController : Controller
    {
        [HttpPost]
        public IActionResult Post([FromBody] TestRequest request)
        {
            return Ok(new {ModelState.IsValid, request?.Value});
        }
    }

    public class TestRequest
    {
        [Required, BindRequired]
        public int Value { get; set; }
    }

Description of the problem:
"Required" non-nullable properties will never raise a corresponding model validation error if the property is entirely omitted from the request body.  Given the following request body sent to the Post action:
{ }

The controller responds with:
{
    "isValid": true,
    "value": 0
}

Per the docs for binding and similarly the docs for validation it is unexpected that a missing property in the json passes validation.

[BindRequired]: This attribute adds a model state error if binding cannot occur.
[Required]: Makes a property required.

It also appears to be difficult to replace the built-in convention and achieve the desired behavior (which is a validation error if a required or binding required property is omitted).
One of the top search results for this issue is this blog post where the author explains a workaround which no longer works:
public class RequiredBindingMetadataProvider : IBindingMetadataProvider
{
    public void GetBindingMetadata(BindingMetadataProviderContext context)
    {
        if (context.PropertyAttributes.OfType<RequiredAttribute>().Any())
        {
            context.BindingMetadata.IsBindingRequired = true;
        }
    }
}

Code within the default model binders declare that binding is successful if the property is non-nullable and the bound value is null:
https://github.com/aspnet/Mvc/blob/c922b0b90dc95407e1f59510f953d220442f96a1/src/Microsoft.AspNetCore.Mvc.Core/ModelBinding/Binders/SimpleTypeModelBinder.cs#L141-L151
Similarly, it seems none of the usual hooks for custom validation can tell if the value was bound to a value in the request or if it just assumed the default value for the non-nullable type.  I've explored:

Using IModelValidatorProvider / IModelValidator to check the ModelValidationContext (the context has a Model value of the assumed default 0).
Using IBindingMetadataProvider to make sure IsBindingRequired is true (as per the above example).
Using IValidationMetadataProvider to make sure IsRequired is true (no effect).

The only work-around I've found is to change the property to a nullable type (int?).  This seems like a poor solution since:

Usage of the model property is more verbose: Value.Value.
Static analysis tools will identify this usage as a possible NRE.

This has also been discussed in issues #6825, #5391, and #3971, but none provide a resolution.
Is there any way to achieve this desired behavior by convention, and should this not be the default behavior?
Version of Microsoft.AspNetCore.Mvc or Microsoft.AspNetCore.App or Microsoft.AspNetCore.All:
I've tried this on both:

2.1.0-rc1-final
2.0.5

Thanks for contacting us, @mpetito.

This is not a problem we can solve.
[BindRequired] is a feature inextricably tied to the MVC model binding system and none of the supported input formatters supports it. But for JSON.NET, use [JsonProperty(Required = true)] instead.
[Required] is a .NET attribute with very specific, documented semantics:

A validation exception is raised if the property is null, contains an empty string (""), or contains only white-space characters.

I've spent the better part of the day running through most of @mpetito steps above (while introducing [ApiController]). I think if anything, the documentation for BindRequiredAttribute should be updated to call out the limitations and workaround @dougbu mentioned. The distinction between model binding and input formatters is far from obvious until you start to really dig into this.
IMHO, the TestRequest model above really should fail validation when MVC handles posted JSON of {}.


Time slot is stuck when patient is trying to book a doctor appointment

when we login through patient login after registeration and try to book an appointment we are only able to change dates but not time. Time slot is stuck at 10:00 only and it does not change

Currently the appointment system is based on first come first serve basis. So the patient could only select the date and according to the doctor's availability the time is automatically allocated by the system, which could not be modified currently. It will be good to provide user an option to select the appointment time as well.
Are you willing to work on this issue @Bhaal-G

yes i would love to but could you explain what i need to do?
On Fri, Oct 13, 2023 at 10:31 AM Aman Dwivedi @.***>
wrote:

Currently the appointment system is based on first come first serve basis.
So the patient could only select the date and according to the doctor's
availability the time is automatically allocated by the system, which could
not be modified currently. It will be good to provide user an option to
select the appointment time as well.
Are you willing to work on this issue @Bhaal-G
https://github.com/Bhaal-G
—
Reply to this email directly, view it on GitHub
https://github.com/Aman-Codes/MedBuddy/issues/6#issuecomment-1760822474,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARTKSJ3X5X3TVMMUZQAPAC3X7DDKVANCNFSM6AAAAAA56O53PA
.
You are receiving this because you were mentioned.Message ID:
@.***>


self.【ability(YYY).XXX】YYY cannot be input correctly, a BUG will be prompted.

ability(奥术风暴)[!weather(奥术之风)]
ability(奥术风暴)[ weather(奥术之风) & weather(奥术之风).duration < 3 & self.ability(法力澎湃).duration <= 1]
ability(法力澎湃)[weather(奥术之风)]
ability(奥术风暴)[ self.ability(奥术风暴).strong]
ability(#1) [ self.ability(#1).strong]
ability(奥术风暴)[ self.ability(#1).weak]
ability(#1) [ self.ability(奥术风暴).weak]
ability(奥术风暴)[enemy(#1).dead & enemy(#2).dead & !self.ability(奥术风暴).weak]
ability(#1)

Interface/AddOns/tdBattlePetScript/
Core/Util.lua:73: bad argument #1 to
max'(number expected, got nil)

This really should never happen, honestly.

What fight does it happen on?
Are you able to reproduce it consistently or did it happen once and never again? If yes,  please get to the point of error happening and run /run print("---")local o,s,a;for o=1,2 do for s=1,3 do for a=1,3 do print(o,s,C_PetBattles.GetPetSpeciesID(o,s),a,C_PetBattles.GetAbilityInfo(o,s,a),C_PetBattles.GetAbilityState(o,s,a))end end end, screenshot/copy the data shown in your chat window
If you can't reproduce it, do you have a PetBattleLogKeeper log for that fight?