FEYNMAN VOICE LEARNING SYSTEM

FEYNMAN VOICE LEARNING SYSTEM
Design Document v1.0

CORE PRINCIPLES

A. Natural Learning Flow

Speaking precedes writing
Immediate articulation reveals understanding
Real-time feedback drives improvement
Verbal expression encourages simplification

B. Learning Objectives

Improve concept understanding
Develop clear explanation skills
Identify knowledge gaps
Build teaching confidence

KEY TERMINOLOGY

Learning Components:

Session: A single learning/teaching event
Explanation: Verbal description of concept
Analysis: AI evaluation of explanation
Feedback Loop: Continuous improvement cycle

User Roles:

Learner: Primary user explaining concepts
System: AI analyzer and feedback provider
Knowledge Base: Source material reference

USER JOURNEY

Phase 1: Preparation

Select topic
Review source material
Initialize learning session

Phase 2: Verbal Explanation

Record explanation
Real-time transcription
Initial self-assessment

Phase 3: Analysis

AI processes explanation
Evaluates clarity and completeness
Identifies improvement areas

Phase 4: Review

Receive feedback
Review transcription
Save session results

PRIMARY USE CASES

A. New Topic Learning

First-time concept explanation
Basic understanding verification
Gap identification

B. Understanding Deepening

Iterative explanations
Complexity reduction
Analogy development

C. Teaching Practice

Presentation preparation
Lecture rehearsal
Concept simplification

SUCCESS METRICS

Quantitative:

Clarity score (%)
Simplicity index
Completion rate
Session duration

Qualitative:

Confidence improvement
Explanation fluency
Concept mastery
Teaching effectiveness

TECHNICAL REQUIREMENTS

Core Features:

Voice recording
Speech-to-text processing
AI analysis engine
Session management
Progress tracking

Integration Points:

Voice input
Text processing
Analysis algorithms
Storage system

GHERKIN SCENARIOS

Feature: Voice Learning Session
Scenario: Starting a New Learning Session
Given the user has opened the application
When they select "New Session"
Then the system should initialize recording capability
And display the session timer
And enable the voice input
Scenario: Recording an Explanation
Given the user is in an active session
When they press the record button
Then the system should begin voice recording
And show real-time transcription
And indicate recording status
Scenario: Completing an Explanation
Given the user is recording
When they press stop
Then the system should end the recording
And process the transcription
And initiate the analysis phase
Scenario: Receiving Analysis
Given the explanation is complete
When the analysis is finished
Then the system should display:
| Clarity score        |
| Simplicity measure   |
| Completeness rating  |
| Improvement areas    |
Scenario: Saving Session Results
Given the analysis is complete
When the user selects "Save Session"
Then the system should store:
| Recording          |
| Transcription     |
| Analysis results  |
| Session metadata  |
Scenario: Reviewing Past Sessions
Given the user has saved sessions
When they access "Session History"
Then they should see:
| Session date      |
| Topic            |
| Performance metrics |
| Improvement trend   |

NEXT STEPS

Development Phases:

MVP Implementation

Basic recording
Simple analysis
Session storage

Feature Enhancement

Advanced analytics
Progress tracking
User customization

System Integration

Knowledge base connection
Learning path optimization
Performance tracking

Would you like to focus on any particular aspect of this design document or shall we proceed with refining specific components?

DENO FEYNMAN VOICE IMPLEMENTATION
Backend Architecture Design

TECHNOLOGY STACK

Core:

Deno Runtime
Oak (middleware framework)
Deno Deploy (optional for deployment)
WebSocket for real-time communication
Deno FFmpeg for audio processing
Deno KV for storage

Frontend:

Fresh framework
Preact
Islands Architecture
WebRTC for voice capture

KEY COMPONENTS

A. Server Architecture
// app.ts
import { Application } from "https://deno.land/x/oak/mod.ts";
import { router } from "./routes.ts";

const app = new Application();

// Basic CORS setup
app.use(async (ctx, next) => {
  ctx.response.headers.set("Access-Control-Allow-Origin", "*");
  await next();
});

app.use(router.routes());
app.use(router.allowedMethods());

await app.listen({ port: 8000 });

B. Voice Processing Service
// services/voice.ts
export class VoiceProcessor {
  async processAudio(audioBuffer: ArrayBuffer) {
    // Handle incoming audio stream
    const processed = await this.convertToWav(audioBuffer);
    return processed;
  }

  private async convertToWav(buffer: ArrayBuffer) {
    // Audio format conversion logic
    return buffer;
  }
}

C. Transcription Service
// services/transcription.ts
export class TranscriptionService {
  async transcribe(audioData: ArrayBuffer): Promise<string> {
    // Integrate with preferred speech-to-text service
    // Could use Whisper API or similar
    return "transcribed text";
  }
}

D. Analysis Service
// services/analysis.ts
interface AnalysisResult {
  clarity: number;
  simplicity: number;
  completeness: number;
  suggestions: string[];
}

export class AnalysisService {
  async analyzeExplanation(text: string): Promise<AnalysisResult> {
    // Integrate with AI service for analysis
    return {
      clarity: 0,
      simplicity: 0,
      completeness: 0,
      suggestions: [],
    };
  }
}

DATA STORAGE

Using Deno KV:
// db/store.ts
interface Session {
  id: string;
  timestamp: number;
  transcription: string;
  analysis: AnalysisResult;
}

export class SessionStore {
  private kv: Deno.Kv;

  constructor() {
    this.kv = await Deno.openKv();
  }

  async saveSession(session: Session): Promise<void> {
    await this.kv.set(["sessions", session.id], session);
  }

  async getSession(id: string): Promise<Session | null> {
    const session = await this.kv.get(["sessions", id]);
    return session.value as Session;
  }
}

API ROUTES

// routes.ts
import { Router } from "https://deno.land/x/oak/mod.ts";

const router = new Router();

router
  .post("/sessions", async (ctx) => {
    // Handle new session creation
  })
  .post("/sessions/:id/audio", async (ctx) => {
    // Handle audio upload and processing
  })
  .get("/sessions/:id", async (ctx) => {
    // Retrieve session data
  })
  .get("/sessions/:id/analysis", async (ctx) => {
    // Get session analysis
  });

export { router };

REAL-TIME COMPONENTS

WebSocket Handler:
// ws/handler.ts
export class WebSocketHandler {
  private sessions: Map<string, WebSocket>;

  constructor() {
    this.sessions = new Map();
  }

  handleConnection(ws: WebSocket, sessionId: string) {
    this.sessions.set(sessionId, ws);

    ws.onmessage = async (event) => {
      // Handle real-time audio streaming
    };
  }
}

DEPLOYMENT CONSIDERATIONS

Deno Deploy for easy deployment
Edge functions for optimal latency
Consider regional data requirements
WebRTC STUN/TURN server needs

SECURITY CONSIDERATIONS

// middleware/security.ts
export async function securityMiddleware(
  ctx: Context,
  next: () => Promise<void>,
) {
  ctx.response.headers.set("X-Content-Type-Options", "nosniff");
  ctx.response.headers.set("X-Frame-Options", "DENY");
  ctx.response.headers.set("X-XSS-Protection", "1; mode=block");

  await next();
}

DEVELOPMENT WORKFLOW

# Run locally
deno run --allow-net --allow-read app.ts

# Run tests
deno test

# Deploy
deno deploy app.ts

Benefits of Using Deno:

Built-in TypeScript support
Secure by default
Modern ES modules
Built-in testing
Standard formatting
Easy deployment
Web standard APIs
Great performance

Would you like me to elaborate on any part of this architecture or focus on implementing a specific component first?


chore: Update rox-ci-image to 0.4.2

Description

change me!
User-facing documentation

[ ] CHANGELOG is updated OR update is not needed
[ ] documentation PR is created and is linked above OR is not needed

Testing and quality

[ ] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[ ] CI results are inspected

Automated testing

[ ] added unit tests
[ ] added e2e tests
[ ] added regression tests
[ ] added compatibility tests
[ ] modified existing tests

How I validated my change

change me!

/test gke-ui-e2e-tests


added seed to random choice

42 is a good seed :)

@ananyaks @aousterh Instead of caching the generated search workloads, perhaps consider caching the unique values (can keep it in the repo).
If parsing through the dataset is fast, then it doesn't make a difference. @ananyaks how long does it take for you to generate the 1000 queries?
Happy to take a stab of the above once this is merged!


Cortana Background Problem

🛑 IMPORTANT: You can get your feedback addressed faster if you use the comment section for the article in which you encountered a problem.
Link to article:
The article that cause this problem is at https://docs.microsoft.com/en-us/cortana/voice-commands/launch-a-background-app-with-voice-commands-in-cortana
Problem:
The App Service name in Package.appxmanifest and the App Service name in the VCD file is not the same. I think this may cause Cortana can't contact with the App Service.

And Cortana UI won't work but spoken message still be spoken.

@Karl-Bridge-Microsoft Can you take a look at the issue?

@superkid200 @Powerhelmsman - The snippet has been fixed. Please be sure to check the related sample for the most up to date code (linked from the doc). It sees regular updates and maintenance.


GIT sha256 support

SHA256 support comes in git-2.29, test deployment on codeberg-test does not work. Steps to reproduce:
git init --object-format=sha256
touch test
git add test
git commit -m "test"
git remote add origin git@codeberg.org:reinerh/test.git
git push -u origin main

Error: fatal: Protokollfehler: unerwartetes capabilities^{}
gitea version: 1.14.0+dev-294-g594cc4aa2

Not suggest add it now because it still an experimental feature in git.

How do we propose to set the SHA-256 setting for a repo? I assume we don't want to set this global(as this would only make sense for "new" gitea instance). Would it be a simple option on the repo creation to enable the SHA256 format?

If the repository is initialized in Gitea, we can have option in creating repository page, but if it's a repository pushing to create, we should have another method to detect it.

Hello,
NIST wanna sunset SHA1: https://www.nist.gov/news-events/news/2022/12/nist-retires-sha-1-cryptographic-algorithm
Yes, this will take some time and it is until 2030. Still, people should start it, as there are still no migration possibilities like i don't know, git make-readonly ; git migrate start --to=SHA2-256 ; git migrate clean --away=SHA1 or whatever. Also, this is also only for local git, there's no possibility for this for fetching/pulling already cloned gits.
Also, there may be more hashing algorithms in the future, or some are broken, just migrating to one will perhaps be enough for 10 years, but then we may have this issue again.  Please for now, at least test and provide sha256 as an option.
migrating data will take time.

Hello, NIST wanna sunset SHA1: https://www.nist.gov/news-events/news/2022/12/nist-retires-sha-1-cryptographic-algorithm
Yes, this will take some time and it is until 2030. Still, people should start it, as there are still no migration possibilities like i don't know, git make-readonly ; git migrate start --to=SHA2-256 ; git migrate clean --away=SHA1 or whatever. Also, this is also only for local git, there's no possibility for this for fetching/pulling already cloned gits.
Also, there may be more hashing algorithms in the future, or some are broken, just migrating to one will perhaps be enough for 10 years, but then we may have this issue again. Please for now, at least test and provide sha256 as an option.
migrating data will take time.

I second this. Also even if SHA-1 wasn't broken, it would still only be secure up to 80 bits, which shouldn't be considered safe nowadays. You should aim for 100 bits or preferably 128 bits which SHA-256 will give you.

https://medium.com/@v3ai/how-to-use-sha-2-git-repositories-6c2a6ed5d580 - mentions how few git tools support sha256

What's the status of this? Git's documentation in 2.42.0 no longer calls the use of sha256 experimental, and states that no backward-incompatible changes are expected.

Follow #23894


update change in mendablesearch.js

Not including 'window.' might lead to a scope resolution error, ensuring it's accessed within the global scope.

@AashiGuptaShorthillsAI, Please resolve your conflicts and remove the path changes


Adding handling for system tz not found exception

Pull Request Type

[ ] Feature addition
[✓] Bugfix
[ ] Documentation update
[ ] Code refactor
[ ] Tests improvement
[ ] CI/CD pipeline enhancement

Description
It handles a rare exception that was found while testing locally in a Windows 10 environment.
Changes

A try-except catch for the timestamp.astimezone() function in the return
Allows fallback handling to UTC, if the system tz isn't found at any time

I'm somewhat against this idea as by all means, this is a Python and/or operating system bug. It's also a fallback that results in behavior that isn't necessarily consistent with how things usually acts (since UTC is far from being guaranteed as the user's timezone).
Not entirely against it, but still.

@AstreaTSS The whole library should handle a fallback globally, possibly every connection refresh or initialization? But this fixes the only related bug I've noticed.

Gateway issues are usually machine-dependent, but they're usually down to things we wouldn't consider as bugs and usually happen in normal situations - IE connection being lost briefly.
Meanwhile, the fact this exists suggests the datetime library is getting an edge case on a - specific OS.
That being said, having done more research, what's likely happening here is that the datetime library is being passed a datetime that happens very near to the UNIX epoch. I found issues that suggest that using astimezone() on Windows like this makes things freak out.
 This solution doesn't actually fix that specific issue - it just works around them.

@AstreaTSS Well considering Windows users, the Exception is caught every time because Windows has the unix timestamp in milliseconds. Considering this function runs like 5 times a second (in a big server at least), maybe an overall runtime system check is needed?

That's not... quite correct. See my comments on Discord - this issue actually likely has to do with ActivityTimestamp's behavior.

Superceded by #1626.


systems in module

Hi,
is it normal, that ocicl also creates a systems.csv and directory in a module/subdirectory?

contents is:
asdf-package-system, ghcr.io/ocicl/asdf-package-system@sha256:bafeb9d6e67cb280100080a2382f24558964ecf37029ff904b932c6c7d0e9c33, asdf-package-system-20230809-61776f2/asdf-package-system.asd

Marc

I could never reproduce this.


Extension tests: debugger does not disconnect when tests finishes

Steps to Reproduce:

yo code to produce a TS extension
run the extension tests

=> after the tests finish, the debug toolbar is still showing

most likely a duplicate of #48442

@weinand if this is a duplicate and that one has been verified, why is this one marked as verification needed?

Because of the "most likely" in my comment above. I wasn't really sure whether the fix for #48442 fixes this issue too.

@weinand yup verified

great, thx!


Azure App Configuration doesn't work with Managed Identity if OfflineCache is specified

I'm trying to follow along the sample of accessing App Configuration with Managed Identity in AspNetCore 3.1 app but getting runtime exception with error details:
Message: An Azure App Configuration connection string is required.
Source: Microsoft.Extensions.Configuration.AzureAppConfiguration
StackTrace:    at Microsoft.Extensions.Configuration.AzureAppConfiguration.OfflineFileCache.EnsureOptions(AzureAppConfigurationOptions azconfigOptions)
   at Microsoft.Extensions.Configuration.AzureAppConfiguration.OfflineFileCache.Export(AzureAppConfigurationOptions options, String data)
   at Microsoft.Extensions.Configuration.AzureAppConfiguration.AzureAppConfigurationProvider.<LoadAll>d__20.MoveNext()
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.ConfiguredTaskAwaitable.ConfiguredTaskAwaiter.GetResult()
   at Microsoft.Extensions.Configuration.AzureAppConfiguration.AzureAppConfigurationProvider.Load()
   at Microsoft.Extensions.Configuration.ConfigurationRoot..ctor(IList`1 providers)
   at Microsoft.Extensions.Configuration.ConfigurationBuilder.Build()
   at Microsoft.Extensions.Hosting.HostBuilder.BuildAppConfiguration()
   at Microsoft.Extensions.Hosting.HostBuilder.Build()
   at App.Web.Program.Main(String[] args) in E:\App\src\App.Web\Program.cs:line 45

The only difference between doc's sample code and my code is that of OfflineCache. This is what I have in my code:
options.SetOfflineCache(new OfflineFileCache(
                        new OfflineFileCacheOptions { Path = $"{contentRoot}\\OfflineConfigStore\\OfflineCache.json" }));

If I remove those 2 lines of SetOfflineCache, the code works fine.
I'm not sure how connection string is linked with OfflineCache setting.
Is it broken?
P.S. My nupkg ver of Microsoft.Azure.AppConfiguration.AspNetCore is 4.1.0.

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

ID: e0b5a566-084c-d77f-9d78-3bfd72b01080
Version Independent ID: b22f2729-2f31-a0cd-fc61-0ead183b0aab
Content: Use managed identities to access App Configuration - Azure App Configuration
Content Source: articles/azure-app-configuration/howto-integrate-azure-managed-service-identity.md
Service: azure-app-configuration
GitHub Login: @AlexandraKemperMS
Microsoft Alias: alkemper

@sameer-kumar -
Thanks for brining this to our attention. We will investigate it further and update you shortly.

@sameer-kumar -
Thanks for brining this to our attention. We will investigate it further and update you shortly.

@sameer-kumar -

Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can update the document accordingly.

hi @sameer-kumar, thank you for reaching out. This is expected behavior. App Configuration supports offline caching, but only with connection strings. The work to support offline caching with managed identity is currently in progress, and not yet available. In order to use the offline cache, you will also need to create a connection string.

Thanks @AlexandraKemperMS for following up. I think this is a bug and not thought through while designing. At one side you recommend to use MSI and at the same time you deprive enterprise from such a key functionality. I strongly believe and request you should put this in your priority backlog.

Hi @sameer-kumar, yes, I hear you. To build a better experience this is something we have prioritized and are working on.

Thank you for confirming @AlexandraKemperMS. Looking forward to hear soon on this.

Glad to help. I will now close this thread, if you have any further feedback or questions, please feel free to create a new thread anytime.


Error in versions newer than 1.7.0: useHref() may be used only in the context of a  component

What version of Remix are you using?
1.7.4
Steps to Reproduce
Here's my root component, largely copied from Remix examples:
export default function Root() {
	return (
		<html lang="en">
			<head>
				<Meta />
				<Links />
			</head>
			<body>
				<Sidenav />
				<Outlet />
				<ScrollRestoration />
				<Scripts />
				<LiveReload />
			</body>
		</html>
	);
}

Sidenav (which causes the error) looks like this:
export default function Sidenav() {
	return (
		<SideNav aria-label="Side navigation" isRail>
			<SideNavItems>
				{Object.keys(FunctionData).map((cardName) => (
					<Link key={cardName}
						to={cardName}>
						{FunctionData[cardName].cardTitle}
					</Link>
				))}
			</SideNavItems>
		</SideNav>
	);
}

SideNav (capital N) is an IBM Carbon UI component (similar to Google Material etc.), an implementation of a left-side navigation menu.
FYI, it fails even WITHOUT the Carbon wrapper, so Carbon is not the issue. For example, this fails the exact same way:
export default function Sidenav() {
	return (
		<>
			{Object.keys(FunctionData).map((cardName) => (
				<Link key={cardName}
					to={cardName}>
					{FunctionData[cardName].cardTitle}
				</Link>
			))}
		</>
	);
}

Expected Behavior
Sidenav should render and the links should work. Remix 1.7.0 in fact does work fine with this implementation, and for now I'm prevented from updating to newer Remix versions until this is resolved. I'm open to code changes on my end of course, but haven't found any way yet to get it to work in 1.7.4...e.g., moving Sidenav into a sample child route that is rendered via Root's Outlet still gives the same error. The Sidenav component renders fine and the links work fine in 1.7.0, but not in 1.7.4 (pretty certain 1.7.2 and/or 1.7.3 similarly broke this, haven't tried 1.7.1).
Actual Behavior
This is the error dump when I try to view any page of the app in 1.7.4 (no error and everything works great in 1.7.0):
Error: useHref() may be used only in the context of a <Router> component.
    at Object.invariant (C:\GitClones\ci-react-ts\node_modules\@remix-run\router\utils.ts:757:11)
    at Object.useHref (C:\GitClones\ci-react-ts\node_modules\react-router\lib\hooks.tsx:53:3)
    at Object.LinkWithRef [as render] (C:\GitClones\ci-react-ts\node_modules\react-router-dom\index.tsx:372:16)
    at ReactDOMServerRenderer.render (C:\GitClones\ci-react-ts\node_modules\react-dom\cjs\react-dom-server.node.development.js:3872:44)
    at ReactDOMServerRenderer.read (C:\GitClones\ci-react-ts\node_modules\react-dom\cjs\react-dom-server.node.development.js:3690:29)
    at renderToString (C:\GitClones\ci-react-ts\node_modules\react-dom\cjs\react-dom-server.node.development.js:4298:27)
    at handleRequest (C:\GitClones\ci-react-ts\build\index.js:1786:49)
    at handleDocumentRequest (C:\GitClones\ci-react-ts\node_modules\@remix-run\server-runtime\dist\server.js:431:20)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at requestHandler (C:\GitClones\ci-react-ts\node_modules\@remix-run\server-runtime\dist\server.js:72:18)

@ecrvich could you create a little repository reproducing the issue? Hard to tell what's going wrong without that.

In trimming down the code for a simpler test case, I discovered the problem.
I had been using this import in Sidenav (and this works fine in Remix 1.7.0):
import { Link } from "react-router-dom";
I changed this to:
import { Link } from "@remix-run/react";
And everything now works fine with 1.7.4 as well.
I always try to use Remix imports over React Router imports where they exist in both places, but I guess I missed this one. And there are still some cases where components only seem to exist in React Router (like some hooks, I think), so it's seemingly a trial-and-error thing as to where to import from. No biggie. Sorry for the false alarm, feel free to close if you think that's appropriate!


IMN-254 Fix interface deletion when deleting a draft descriptor in catalog-process

[To be merged after #201]
This pull request is for fixing the deleteDraft operation. When deleting a draft descriptor, we also have to delete the uploaded interface file and documents. However, we were using a wrong logic to retrieve the interface path to delete.

Closed because the branch name was wrong.


AtlantisHelper crashes when using websockets and Intercom's new package

Xcode 14.3.1 (14E300c)

Atlantis 1.21.1
iOS 15.4 and 16.4

We have narrowed it down to moving to Intercom's new iOS package.
https://github.com/intercom/intercom-ios   <--- Atlantis worked just fine
https://github.com/intercom/intercom-ios-sp   <--- Atlantis crashed as shown above

When I revert back to the previous version Atlantis works again with websockets.
We were also having issues with the debugger not working while using the newer package:
(lldb) po hourlySummaries.count
error: expression failed to parse:
error: stat cache file '/Users/distiller/project/DerivedData/archive/SDKStatCaches.noindex/iphonesimulator16.4-20E238-.sdkstatcache' not found
error: stat cache file '/Users/distiller/project/DerivedData/archive/SDKStatCaches.noindex/iphonesimulator16.4-20E238-.sdkstatcache' not found

error: couldn't IRGen expression. Please check the above error messages for possible root causes.

but reverting to the previous version of Intercom fixes that as well. Very surprising it would cause issues like this. Going to send them a support message now.

Thanks for the update. Let me know if it's Atlantis bug, then I can check it.

Having the same issue here, @chadparker did you had a chance to fix this issue?

No, @yuravake we sent a support message to Intercom, and they suggested to try their next release but it didn't fix the issue. We reverted our Intercom version for now. Could you create a support ticket with Intercom to raise visibility into this?

@chadparker yes, will do.

Hi @chadparker @yuravake do you think that I should have a function to enable/disable the WebSocket Interceptor on Atlantis ? So, you can opt-out and prevent internal crashes from the Intercom app.

By default, Atlantis performs the Method Swizzling from all available classes, including the main app, 3rd-party frameworks, and libraries.
I suggest:

Only perform the method swizzling for the current app.
Ignore 3rd-party libs.

It might solve all problems.

@NghiaTranUIT That sounds like a good solution; that would be a great function to have! I'm not very familiar with the Atlantis code, but if you could give me things to try I'll definitely let you know if it fixes things.

@chadparker @yuravake can you check with the latest version of Intercom ?
Here is my setup:

Atlantis v1.22.0 (latest)
Intercome v16.0.3 (latest - https://github.com/intercom/intercom-ios-sp)
Proxyman v4.12.0 (latest)

Result:

No crash ✅
Atlantis can intercept Intercom traffic

@NghiaTranUIT That sounds like a good solution; that would be a great function to have! I'm not very familiar with the Atlantis code, but if you could give me things to try I'll definitely let you know if it fixes things.

Turn out it's technically impossible:

Atlantis tries to intercept the URLSession from Apple framework (CFNetwork.framework)
However, I couldn't query if the URLSession is called from the main app or 3rd-library. There is no such information in runtime (Or I haven't found yet).

@NghiaTranUIT unfortunately, we can still reproduce this crash on the latest versions(

@yuravake can you share with me your setup?

iOS version
iOS device or simulator
Xcode version.
Atlantis version
Intercom version

Have the same exact crash.
Some info:

iOS 17.0 (21A328)
Simulator (iPhone 15 Pro)
Xcode 15.0 (15A240d)
Atlantis 1.22.0
Intercom 16.2.3

Hix, not sure why I can't reproduce the crash:
@djbe can you share your setup?

Is Atlantis installed via Cocoapods or SPM?
Is Intercom installed via Cocoapods or SPM?
What func is called that causes the crash? (You can trace it from the Xcode's call stack)

Thanks in advance 🙇

My setup: Install Atlantis and Intercom (Same version) via SPM. It works fine, no crash.

Both are installed via CocoaPods
By trace, do you mean bt?

(lldb) bt
* thread #6, queue = 'com.apple.NSURLSession-delegate', stop reason = EXC_BAD_ACCESS (code=2, address=0x16d618000)
    frame #0: 0x00000001051b3840 libsystem_platform.dylib`_platform_memmove + 112
    frame #1: 0x00000001800c2ce4 libsystem_blocks.dylib`_Block_copy + 92
  * frame #2: 0x00000001047443d4 Atlantis`closure #1 in static AtlantisHelper.swizzleWebSocketReceiveMessage(message=0x00006000000213d0, error=nil, responseHandler=0x0000000104761230 Atlantis`partial apply forwarder for closure #1 (Swift.Optional<Swift.String>, Swift.Optional<Foundation.Data>, Swift.Optional<Swift.Error>) -> () in closure #1 (Swift.AnyObject, Swift.AnyObject) -> () in Atlantis.NetworkInjector._swizzleURLSessionWebSocketReceiveMessageSelector(Swift.AnyObject.Type) -> () at <compiler-generated>, originalHandler=0x000000016d6127f8) at AtlantisHelper.swift:75:13
    frame #3: 0x00000001047449cc Atlantis`thunk for @escaping @callee_guaranteed (@guaranteed NSObject?, @guaranteed NSError?) -> () at <compiler-generated>:0
    frame #4: 0x00000001074780f0 libdispatch.dylib`_dispatch_call_block_and_release + 24
    frame #5: 0x000000010747993c libdispatch.dylib`_dispatch_client_callout + 16
    frame #6: 0x0000000107481bd8 libdispatch.dylib`_dispatch_lane_serial_drain + 916
    frame #7: 0x000000010748291c libdispatch.dylib`_dispatch_lane_invoke + 420
    frame #8: 0x000000010748f2f8 libdispatch.dylib`_dispatch_root_queue_drain_deferred_wlh + 324
    frame #9: 0x000000010748e754 libdispatch.dylib`_dispatch_workloop_worker_thread + 488
    frame #10: 0x000000010549f924 libsystem_pthread.dylib`_pthread_wqthread + 284

Thanks @djbe . I'm working on the fix now.

Provide a flag to turn off the method swizzling in WebSocket classes => Prevent unexpected crashes 👍

Hmmm, definitely better than what I'm doing currently (disable Intercom in debug builds).
Any idea on the root cause? Presumably it's something that Intercom and Atlantis are both doing (with web sockets), but what 🤷

Sorry for the delay.
@djbe @yuravake @chadparker If you don't mind, let's update Atlantis to v1.23.0 and use this code:
Atlantis.start(hostName: "<Your host name>", shouldCaptureWebSocketTraffic: false)

It will skip the Method Swillzing on the WS/WSS connection, which prevents the unexpected crash on the Intercom package 👍

Any idea on the root cause? Presumably it's something that Intercom and Atlantis are both doing (with web sockets), but what 🤷

I'm still investigating it, but not found a solution yet. This fix can help you workaround it 👍

Can confirm that with the flag set to false, the crash no longer occurs 🎉

Hello @NghiaTranUIT and thanks for the fix 👍 For your information we have the same issue with the iOS SDK of Gleap (https://docs.gleap.io/ios/) and using the flag fixes it, maybe it could help you to locate the root cause.

For what it's worth, we are on Intercom 16.4.0 and that was the reason Atlantis wouldn't work for us. Once we removed Intercom, Atlantis worked great.

@chadparker if you want to keep the Intercom, you can use v1.23.0, and opt-out the WebSocket interception 👍

@chadparker Thank you! I have opted out of websockets for Atlantis and it works. Thanks again!

Hey! I'm a new Atlantis user and I'm encountering this same problem, and just found this issue. I can also confirm that adding the shouldCaptureWebSocketTraffic to false fixes the app crash! However, as the name implies, I now cannot see messages from Websocket traffic in Proxyman, which is what I want to use Atlantis for. Is there any other solution planned, or workaround that others have found?

@markanderson-underdog unfortunately, there is no workaround atm because I can't reproduce the crash on my side with Atlantis + Intercome.
I suggest disabling the Intercome if possible to see the WS/WSS traffic.
May I ask:

What Intercome lib version you're using? and steps to reproduce the crash?

@NghiaTranUIT awesome! By removing the line of code where we init() the Intercom SDK by calling Intercom.setApiKey(), the app crash no longer occurs and I can view Websocket traffic using Atlantis 🎉
The Intercom version is 1.6.5 (the latest), and we're using SPM for both Intercom and Atlantis.
The STR is as simple as just launching the application, and letting the SDKs initialize. Below is the full variable debugger when the crash in Atlantis. Please let me know if there's anything else I can provide!


Apply changes from #11032 departing from a clean state

@mrsdizzie I don't expect to see anything new, but it doesn't hurt to try.
IMPORTANT
This PR is not for merging because I naively applied the differences with master from 11032, so this would revert the work on master that's not present on that PR.

@guillep2k heh I had the same idea in https://github.com/go-gitea/gitea/pull/11171 but no luck : ( All of the builds are making it a bit further today maybe in a few days will just fix itself 🤞

Didn't work 🙁


Looking for an easy way to mock a ConnectException

Was looking for ways to mock not so common exceptions as Guzzle/ConnectExceptions
I think closures could work as in laravel HttpClient
https://laravel.com/docs/9.x/http-client#fake-callback
Also guzzle handlers allow these kind of mocked exceptions
https://docs.guzzlephp.org/en/stable/testing.html#mock-handler
Any ideas?

Propossing something like:
new MockClient([
    fn() => new MockResponse(['name' => 'Sam'], 200),
    new MockResponse(['name' => 'Alex'], 200),
    new MockResponse(['error' => 'Server Unavailable'], 500),
]);

Thank you for mentioning this issue. I hadn't considered mocking exceptions but I have now created a PR for them #49 and here's how it will work:
$mockClient = new MockClient([
    MockResponse::make(['name' => 'Sam'])->throw(new Exception('Custom Exception!')),
]);

// Or using Guzzle

$mockClient = new MockClient([
    MockResponse::make(['name' => 'Sam'])->throw(fn ($guzzleRequest) => new GuzzleException('Cannot connect!', $guzzleRequest)),
]);

It will be in the next release 🚀

Released in 0.11.0 / 0.12.0 (Laravel)


all models except davinci 2 + not working...

Describe the bug
all models except davinci 2 + not working...
I get axios errors when trying to use models such as ada / babbage / etc EXCEPT davinci 2+ -> these models are not working; throws error.
To Reproduce
Fetch using redux.
Error snip:
response: {
status: 404,
statusText: 'Not Found',
headers: {
date: 'Fri, 02 Dec 2022 00:49:31 GMT',
'content-type': 'application/json; charset=utf-8',
'content-length': '158',
connection: 'close',
vary: 'Origin',
},
config: {
transitional: [Object],
adapter: [Function: httpAdapter],
transformRequest: [Array],
transformResponse: [Array],
timeout: 0,
xsrfCookieName: 'XSRF-TOKEN',
xsrfHeaderName: 'X-XSRF-TOKEN',
maxContentLength: -1,
maxBodyLength: -1,
validateStatus: [Function: validateStatus],
headers: [Object],
method: 'post',
Code snippets
import { OpenAIApi, Configuration } from 'openai';
import { Ratelimit } from '@upstash/ratelimit';
import { Redis } from '@upstash/redis';
import getIP from '../../../utils/get-ip';

const configuration = new Configuration({
    apiKey: process.env.OPENAI_API_KEY,
});
const openai = new OpenAIApi(configuration);

const redis = new Redis({
    url: process.env.UPSTASH_REST_API_DOMAIN,
    token: process.env.UPSTASH_REST_API_TOKEN,
});

const ratelimit = new Ratelimit({
    redis: redis,
    limiter: Ratelimit.slidingWindow(3, '1 d'),
});

export default async function response(req, res) {
    const ip = getIP(req);
    const result = await ratelimit.limit(ip);
    res.setHeader('X-RateLimit-Limit', result.limit);
    res.setHeader('X-RateLimit-Remaining', result.remaining);

    if (req.method !== 'POST') {
        res.status(405).json({ error: 'Method not allowed' });
        return;
    }

    if (!req.body.projectId) {
        res.status(400).json({ error: 'Missing projectId' });
        return;
    }

    if (!result.success) {
        res.status(429).json({
            error: 'You have reached your daily limit of 3 free completions. Try again tomorrow or upgrade your plan in account settings to continue using services regularly.',
        });
        return;
    }

    const completion = await openai.createCompletion({
        model: 'text-davinci-002',
        prompt: req.body.prompt,
        temperature: 0.6,
        max_tokens: 2000,
        presence_penalty: 0.5,
        // frequency_penalty: 0.5,
    });

    try {
        const completeModeration = await openai.createModeration({
            input: completion.data.choices[0].text,
            model: 'text-moderation-latest',
        });
        const moderationRes = completeModeration.data.results[0].flagged;
        if (moderationRes === false) {
            res.status(200).json({ response: completion.data.choices[0].text });
        } else {
            res.status(500).json({
                error: 'Sorry. The output has been flagged for inappropriate content. Please try again.',
            });
        }
    } catch (error) {
        res.status(500).json({ error: error.message });
    }
}

OS
mac
Node version
v18.12.0
Library version
3.1.0

Having same issue

Hi @temrb and @doverradio – The following code works for me on library version 3.1.0:
// script.js
// Run with: node script.js
import { Configuration, OpenAIApi } from "openai";

const configuration = new Configuration({
  apiKey: process.env.OPENAI_API_KEY,
});
const openai = new OpenAIApi(configuration);

async function init() {
    try {
        const resp = await openai.createCompletion({
            model: 'ada',
            prompt: 'Hello world',
        });
        console.log(resp.data);
    } catch (error) {
        console.log(error.response.status);
        console.log(error.response.data);
        console.log(error.message);
    }
}

init();

Does that work for you? If not, can you provide exact code that I can execute to replicate the issue you're experiencing?

Seems to be working now when I replace the model input in my original posted code from text-davinci-2 to just using the model name without numbers ; 'ada'
Where as in the documentation it says to use text-ada-001
When I opened the issue I was trying both ways and it was still giving issue. Not sure if it was an update that changed either way docs should be updated imo.

Where as in the documentation it says to use text-ada-001

text-ada-001 works for me too if I replace 'ada' with 'text-ada-001' in my code snippet. I just assumed you were using the older legacy ada model based on your original post, but yes you should be using 'text-ada-001'.

Going to resolve this issue – feel free to open a new issue if you continue to run into problems.

Hey @schnerd! I believe this error (where users can't use any model besides davinci) is still happening. My node version is 16.16. My library version is 3.2.0. Here is how I'm trying to call the ada model:
        await openaiClient.createCompletion({
          model: 'text-ada-001',
          prompt,
          temperature,
          n,
          presence_penalty,
          frequency_penalty,
          max_tokens,
        });

I tried passing in ada to the model parameter, but that didn't work either. If you could please advise I'd greatly appreciate it. Thanks!

Nevermind, mistake was in my code. Turning it to ada from text-ada-001 worked. Thank you! @schnerd


correction for 'used/unused' filters in iam-roles resource

correcting issue where asg & ec2 roles were not being properly reported (#1511)

previously only collecting instance profile
now collecting roles attached to instance profile

adding state param to iam-role used filter

will allow for used/unused via boolean value

deprecating iam-role filter unused

pointing any unused called to the used filter & specifying state param

also reported in #2036


add yarn instructions to installation

Is that really necessary? I guess yarn users know how to install a package from npm install chart.js --save? So I think it's a bit useless but if we really want to add it to our docs, then it should come after the npm section.

My initial reaction was the same. I feel it's a bit redundant

Hi @jayqui. I'm going to close this issue since we believe that most developers would know how to install yarn packages if they've decided to use yarn. Thank you for the suggestion though. Let us know if you encounter anything else that you think should be updated


Change Password Bug

When signing up in the demo with an email the change password feature does not seem to work. Based on the feedback given by the site it wrongly identified my account as an OAuth account and thus didn't allow changing the password. Using the forgot password feature did work.

Working fine for me. Can you provide some detail: steps, screenshot of error, ideally email address on account (can DM to me if you want).

Yes sure. So pretty straight forward, I did a sign up with an email address and a randomly generated password (length 16), clicked the confirm email link I got send, went to settings and clicked change password. I'll DM you the address I used, initially I deleted the account afterwards, but I redid the process just now. The supplied image is the message shown in /account/settings/change_password

I can repo. Will fix. Thanks for report!


Moe Sync

This code has been reviewed and submitted internally. Feel free to discuss on the PR and we can submit follow-up changes as necessary.
Commits:
 Make @RequestScoped use @Scope instead of @ScopeAnnotation.
06256ad3abe926df07ec906084e60a3191873a93

@sameb it looks like the ant build is failing. Do we still use that? If so, it appears that we may need to install it ourselves (perhaps it's no longer installed?)
https://ant.apache.org/bindownload.cgi

Yah looks like Travis stopped supplying it by default. I'll try to fix
tomorrow.
On Mon, Jun 17, 2019, 8:59 PM Ron Shapiro<EMAIL_ADDRESS>wrote:

@sameb https://github.com/sameb it looks like the ant build is failing.
Do we still use that? If so, it appears that we may need to install it
ourselves (perhaps it's no longer installed?)
https://ant.apache.org/bindownload.cgi
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/guice/pull/1251?email_source=notifications&email_token=AAQLWEWP4KLSALWGPM53UILP3AXPVA5CNFSM4HY2NWW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX43K4Q#issuecomment-502904178,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAQLWESYMU3VNT6C4LU4VFLP3AXPVANCNFSM4HY2NWWQ
.


Fixed improper OnDestroy implementation

See https://github.com/Microsoft/MixedRealityToolkit-Unity/commit/290cec0854c67b923e810a8a440c55151bcb8a85

@timGerken yes, you will need to always call base.OnDestroy

Of course. Didn't see it in the thumbnail of code GitHub decided to show me on my phone. 😉

Yeah I know that feeling!


Convert int to string when execute insert query if the type of mysql column is 'bigint unsigned'

What steps will reproduce the problem?

Create a MySQL table and mapped ActiveRecord.

create table foo(id bigint unsigned);

Save a model
    $f = new Foo;
    $f->id = 1;
    $f->save();

What is the expected result?
The  query should be
INSERT INTO `foo` (`id`) VALUES (1)

What do you get instead?
The executed query is
INSERT INTO `foo` (`id`) VALUES ('1')

The range of MySQL unsigned bigint is larger than PHP_INT_SIZE, but it's not reasonable to convert int attribute manually set to string when execute sql, which may slow the query performance.
Additional info

Q
A

Yii version
2.0.12

PHP version
7.1.3

Operating system
CentOS 7

MySQL
5.7.16

Have you checked that it is not related to PDO ?

This is not a bug.
The maximum int value of php depends on the system. 32 bit systems have a maximum signed integer range of -2147483648 to<PHONE_NUMBER>. So for example on such a system, intval('1000000000000') will return<PHONE_NUMBER>.
see http://php.net/manual/en/language.types.integer.php for more details

This is not a bug.
The maximum int value of php depends on the system. 32 bit systems have a maximum signed integer range of -2147483648 to<PHONE_NUMBER>. So for example on such a system, intval('1000000000000') will return<PHONE_NUMBER>.
see http://php.net/manual/en/language.types.integer.php for more details

I think it's the programmer's responsibility to take care of the risk of int overflow.
If the framework user set the int value of a attribute explicitly,  I don't think it's appropriate to cast string to int implicitly by framework

The executed query is

how did you verify the type of the value bound on query execution time? These are bound as parameter values and are converted inside of PDO, Yii has no logic to convert int to string by default in that case. I can not reproduce the issue. Please provide the CREATE TABLE statement of your table.

how did you verify the type of the value bound on query execution time

I find some slow query in my production MySQL slow log which the type of value is not as expected. Maybe the performance of type casting is not the key reason for mysql, but I still think it's better not to cast the type of value if user set it explicitly.

These are bound as parameter values and are converted inside of PDO, Yii has no logic to convert int to string by default in that case

I think the following code do the casting.
https://github.com/yiisoft/yii2/blob/1501c659aca7c50fa5e4f6fbe5ca549acc946c86/framework/db/mysql/QueryBuilder.php#L261

Please provide the CREATE TABLE statement of your table.

create table foo(id bigint unsigned);

Insert table which has bigint unsigned type would case the casting.

yeah, the reason for this is that bigint unsigned is bigger than integer in PHP so the string representation is save to use, while integer may overflow.

I find some slow query in my production MySQL slow log which the type of value is not as expected.

can you provide more information about this? What exactly is logged in MySQL?

can you provide more information about this? What exactly is logged in MySQL?

The table has a union index with two unsigned bigint columns and has ten million of rows. There are some inserted sql logged at MySQL slow query log. Although I think it's not a heavy work doing type casting for mysql, I still migrated the unsigned bigint to signed.

yeah, the reason for this is that bigint unsigned is bigger than integer in PHP so the string representation is save to use, while integer may overflow.

The overflow only happened when string casted to int, but if programmer set the type as int explicitly, my point is that maybe it's not necessary to cast int back to string.

The overflow only happened when string casted to int, but if programmer set the type as int explicitly, my point is that maybe it's not necessary to cast int back to string.

it is probably not necessary, but we need to find the condition to find that case.

I'm check query for this code
        $model = new Order();
        $model->customer_id =<PHONE_NUMBER>000;
        $model->save(false);

Result query is
``sql
INSERT INTO order (customer_id, created_at) VALUES<PHONE_NUMBER>000,<PHONE_NUMBER>)

for the
$model->customer_id =<PHONE_NUMBER>0000000000000;

sql
INSERT INTO `order` (`customer_id`, `created_at`) VALUES<PHONE_NUMBER>423024128,<PHONE_NUMBER>)

PHP_INT_MAX check required?

PHP_INT_MAX check required?
You may add rules to model. But generated sql query is correct.

@darkdef What is the type of customer_id column? Try convert it to unsigned bigint. The inserted value is irrelevant here, incorrect typecast comes from column schema, you will get the same result even for $model->customer_id = 1 (as explained in first comment).

You right. Reproduced the problem

phpunit --filter testInsertInteger tests/framework/db/mysql/QueryBuilderTest.php
Failed on PHP 5.6:

Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'INSERT INTO customer (profile_id) VALUES<PHONE_NUMBER>000)'
+'INSERT INTO customer (profile_id) VALUES<PHONE_NUMBER>)'

The value<PHONE_NUMBER> is correct - because type of customer.profile_id is INT (mysql.sql). And it's maximum in MySql is<PHONE_NUMBER>.
So behaviour is ok. Test should be fixed.

Closed via 17742cb1466cf6ca64ef4dd8cb9e6f706798564b


How can I remove future weeks from the calendar?

version: 7.0.1
Me again.
After https://github.com/patchthecode/JTAppleCalendar/issues/384 I got a weekly calendar with horizontal scrolling feature.
However, my app does not want users to select the future dates. So I want to disable the scroll to the future weeks.
How can I implement this feature?
For example.
Today is Apr. 19, so my calendar looks like
|  16  17  18  (19) 20 21 22  |
And if the user want to scroll back (left), it will become
|   9  10  11  12  13  14  15  |
While, if the user want to scroll to the future (right), I want the calendar stay on the week start from 16.
I tried to set the end date as today, but I still can scroll to the end of this month.
I tried to use deleteSections() or deleteItems(), but I cannot get the index for the target from date.
I also considered whether there is a shouldScroll delegation function, but I failed to find one.
Could you show me a way to work this out?

sounds like you want a date boundary

Thank you very much! This is exactly what I need.
Seems the data structure of visibleDates.monthDates had been changed from [Date] to [(Date, IndexPath]
Paste my code here to reduce the confusion.
    func scrollDidEndDecelerating(for calendar: JTAppleCalendarView) {
        guard let end = self.visibleEndDate else {
            return
        }

        let visibleDates = calendar.visibleDates()
        if visibleDates.monthDates.contains(where: {$0.date >= end}) {
            calendar.scrollToDate(end)
        }
    }

Awesome. Glad to help!
cheers 🍻


chore: use go generate to generate test PKIs

Currently to test the project, users are forced to call make generate-test-keys
This target is not intended to be used by users of the library.
Instead a more general go generate command can be called to create test keys
closes issue #13

Codecov Report

Merging #14 into master will increase coverage by 7.77%.
The diff coverage is 85.97%.

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
+ Coverage   77.77%   85.54%   +7.77%
==========================================
  Files           1        3       +2
  Lines           9      173     +164
==========================================
+ Hits            7      148     +141
- Misses          1       13      +12
- Partials        1       12      +11

Impacted Files
Coverage Δ

pkg/didcomm/transport/http/outbound.go
83.54% <83.54%> (ø)

pkg/didcomm/transport/http/tls/certpool.go
88.23% <88.23%> (ø)

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 ba773ce...c25a7b4. Read the comment docs.

@Baha-sk @troyronda if this PR is merged as-is, and #9 is also merged, it will leave the make generate-test-keys target

@Baha-sk @troyronda if this PR is merged as-is, and #9 is also merged, it will leave the make generate-test-keys target

yes this is why this PR should be merged after #9 is merged, I created branch off of #9, I should have created it off of master to remove the dependency


Use external endorser service

The Traction repo has code for an endorser that is out of date - https://github.com/bcgov/traction/tree/main/services/endorser.
We should switch to reference and use the code in https://github.com/hyperledger/aries-endorser-service instead.

Other alternative could be to just use an endorser agent without the service on top? Is that possible/something that's lighter? Or are the service integrations needed.
On the other hand, having the service running when starting up the whole docker compose is then creating an ecosystem that matches our deployed/production real world Traction cases, so maybe that's better.

Other alternative could be to just use an endorser agent without the service on top? Is that possible/something that's lighter? Or are the service integrations needed.

Actually that might be a better idea for Traction in Docker, since we just need to be able to endorse: we can run an ACA-Py agent with auto flags set and that would be good enough and simpler - good point!

Hello,I have a doubt,Do i need to deploy endorser separately in Kubernetes or there is another way of directly giving ledger genesis and endorser did to traction while deploying using helmcharts,so it manages automatically by itself of endorsing transactions sent by tenants using the provided endorser did and ledger genesis.

because in docker there is separate endorser agent but i am unable to find any scripts for endorser agent in helmcharts

@shanmukh-777 you would have to deploy an endorser agent separately, or connect to an existing one. The endorser service included in the docker-compose file is just there to facilitate local development.

@esune thanks for the confirmation,but is there any charts for aries-endorser-service kubernetes deployment

@esune thanks for the confirmation,but is there any charts for aries-endorser-service kubernetes deployment

We do not have native k8s charts for it, but there are OpenShift templates you may be able to take inspiration from here

Resolved by #1344


Remove indy-sdk dependency in IndyDidResolver

The BaseLedger interface ((it should really be called IndyLedger) is sufficient and allows indy-vdr to substitute.
This PR also changes the requirement checks in the demo agents to allow running without indy-sdk.

Codecov Report

Merging #1283 (e1a589a) into main (57b82d4) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1283   +/-   ##
=======================================
  Coverage   94.46%   94.46%
=======================================
  Files         476      476
  Lines       28855    28855
=======================================
  Hits        27259    27259
  Misses       1596     1596


Remote: loosen a cast

cl objects to the use of the reinterpret_cast for the casting between
integral types.  Loosen to a static_cast.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test

CC: @dcci


[PORT] Merge 4.9 into master

Port this change from botbuilder-dotnet/master branch:
https://github.com/microsoft/botbuilder-dotnet/pull/3890

Fixes #3884
Fixes #3889 against master

Merges 4.9 into master, "cherry-picking" #3822, #3891, #3892, and #3893
(and #3894)
Changed projects

AdaptiveExpressions
Microsoft.Bot.Builder.AI.QnA
Microsoft.Bot.Builder.Dialogs.Adaptive
Microsoft.Bot.Builder.Dialogs.Debugging
Microsoft.Bot.Builder.Dialogs.Declarative
Microsoft.Bot.Builder.Dialogs
Microsoft.Bot.Builder.Dialogs.Adaptive.Profiling
Microsoft.Bot.Builder.Dialogs.Adaptive.Templates.Tests
Microsoft.Bot.Builder.Dialogs.Adaptive.Tests
Microsoft.Bot.Builder.TestBot.Json

The only bit applicable from this issue was tracked and resolved in https://github.com/microsoft/botbuilder-js/issues/2213.
Closing.


Readonly field handling

What're we doing here
In preparation for the "readonly field handling" feature as outlined in #9 (sorta at the end of the thread) I had to refactor a lot of stuff; primarily the way Schemata are handled.
So as a side-effect the VersionTolerance feature got cleaned up a lot. Also there have been huge performance improvements to it (preventing duplicate Schema entries, caching generated dynamic formatters, ...)
Once this feature is done Ceras will be able to handle readonly fields in various ways.
The current idea is that it should work like this (copied from tutorial step10)
		 * Situation:
		 *
		 * You have an object, which has a 'readonly Settings CurrentSettings;'.
		 * Of course that can't (normally) be serialized or deserialized.
		 * But Ceras can still deal with it.
		 *
		 * Default: readonly fields are completely ignored
		 *
		 * Members: save/restore the content of the variable itself
		 *
		 * Forced: like members, but force a fix if there's a mismatch. Uses reflection, so should be used sparingly.

A mismatch is when the target field is null but the data says it shouldn't be (or vice versa), or when the object in the field has a different type than what the data says it shoudl be.
Properties:
get-only properties cannot be changed through reflection as they literally have no way set-method. Like  a computed property (example: public int A => Environment.TickCount;).
As for get-only-auto-properties: Setting them would require setting the compiler-generated backing field.
While it's very easy to demonstrate that it it's possible, in practice (like once you start to really investigate it) you'll find that it is 100% madness to even attempt it.
See the issue about delegate serialization #11 to get an idea what awaits anyone who thinks about messing with compiler-generated fields.

With the latest commit this feature is considered complete now.
There is one thing that keeps me from merging it:
While implementing this a lot of stuff was refactored.
Some refactorings had to introduce binary-breaking changes (meaning data from older ceras versions will not load correctly, the "ceras format" has changed).
In the end that is for the better, but I would like to keep the number of breaking changes to a minimum.
Now, the next thing (version tolerance) also has its own set of breaking changes; and while that is not done the version-tolerance feature is not really usable anyway.
So because of that it would be quite good if I could merge all the breaking changes into one version.
Like not even publishing a new version of this branch right now and waiting until the next feature is complete as well.


Allow PATCH as a valid initial method for uploads

This LGTM. In a separate thread it sounded like you were running into some issues when trying to use it. Just ping here or on that separate thread once you feel comfortable with the change and I'll merge it.
Thanks for adding the tests.

I verified it works with direct/multipart uploads. There is a separate issue with resumable uploads, but that appears to be an issue on the backend and is reproducible with other client libraries too.

Hi,
@ejona86 , is there any repo with the release candidate containing this fix available ?
Thx

@ihommani, this repo contains the fix. Do you mean Maven repo? No, but I can make a snapshot and I will be doing a new release soon.

@ejona86 , yes a maven repo. But it's ok for now, I used our own nexus server to host the snapshot.  Waiting for the new release though :)
Thx


Cleanup dex market page

Don't read feed channel from pg.selectedMarketOrderbook
Remove unused code
Calculate dex order by lot
Display order Amount: this will make it less confusing for user when calculating what they get with amount of lots they want.

Screenshot is stale: Quantity -> Amount

Hey @ukane-philemon I've briefly took a look at https://github.com/crypto-power/cryptopower/issues/690 and noticed there is quite a bit of concurrent interaction happening with DEXMarketPage struct but not enough syncronization between go-routines involved,
I don't plan to investigate those myself, just pointing out this is probably needs addressing and that it can cause https://github.com/crypto-power/cryptopower/issues/690 and similar issues.


Listening / receiving sms does not work

Copied the entire example into a new flutter project.
android manifest minsdk 19,
permissions set, receiver included
connected my phone via usb with android studio.
loaded app and got a friend to send me an sms.
app only ever says 'latest: null'
receiver never seems to fire - adding print('hi') doesn't fire in onBackgroundMessage(), or onMessage()
any ideas?
edit: RCS was the issue.

Hello @ShadeBlack Can you tell me what was the issue ? and if possible can you share your code ?

My issue was with RCS.
If the message was sent by someone with Google RCS enabled, it wouldn't be received as an SMS, but instead an RCS message.
Same problem if the receiving phone has RCS enabled.
I have to have both phones disable RCS completely in order for it to go through correctly.


OPT 175B inference using accelerate

What is your question?
We have ran the OPT 30B model for inference, using the accelerate library, with multi GPU configuration. Reference notebook - Accelerate_OPT.
So, can we use accelerate to run OPT 175B model, for inference, by loading consolidated shards?
What's your environment?
-metaseq Version (e.g., 1.0 or master):
-PyTorch Version (e.g., 1.0):
-OS (e.g., Linux): Linux
-How you installed metaseq (pip, source): source
-Build command you used (if compiling from source): Referred this link : setup
-Python version: 3.7.12
-CUDA/cuDNN version: 11.2
-GPU models and configuration: 16 A100 40GB
-Any other relevant information:

@stephenroller @patrickvonplaten Have you tried running the 175B model with HF?

I haven't.

Haven't found the time yet either sadly

@BalajiAJ Just FYI -- if by any chance you haven't figured out how to serve the 175B model using the accelerate library, maybe try using Alpa to serve it following this guide .
With Alpa, you can actually serve that model using older generations of GPU  as long as your total memory suffices. You can try it out and let us know what you think about it!

Accelerate_OPT. has lead to the below errors. Can you please help.
ValueError: /home/ubuntu/.cache/huggingface/hub/facebook--opt-30b.main.463007d7da4e87fe962909a027811a8c0b32ede8 containing mote than one .index.json file, delete the irrelevant ones.

@BalajiAJ Just FYI -- if by any chance you haven't figured out how to serve the 175B model using the accelerate library, maybe try using Alpa to serve it following this guide .
With Alpa, you can actually serve that model using older generations of GPU as long as your total memory suffices. You can try it out and let us know what you think about it!

@zhisbug Sure, we will try hosting using Alpa. Thank you!

Hi @BalajiAJ ,
Accelerate_OPT. has lead to the below errors.
load_checkpoint_and_dispatch( model.model, weights_path, device_map=device_map, offload_folder=offload_folder, dtype='float16', offload_state_dict=True ) model.tie_weights()
ValueError: /home/ubuntu/.cache/huggingface/hub/facebook--opt-30b.main.463007d7da4e87fe962909a027811a8c0b32ede8 containing mote than one .index.json file, delete the irrelevant ones.
Can you please help.

In case you haven't figured it out, for OPT-30B snapshot_download currently downloads 3 versions of the model (flax, tf, and pytorch). To run in Huggingface you just need the pytorch version, so you can move/delete the others, including the flax and tf index.json files.

@punitkoura has been working on this and just landed a PR with #214.


Add convert script and table of ROM info

Simple shell script and CSV table of ROM info. The table is based on the text file from TerraOnion and therefore includes definitions for every ROM set supported. This table contains the common ROM set name, TO's proprietary screenshot number, the NGH number, the English title, genre, publish year, developer, and a parent ROM (if applicable). I've tested with about 2 dozen games but the table is as filled out as I could make it.
The script takes a single ROM set archive or directory containing multiple ROM set archives, looks them up in the table, and runs neosdconv against them with the proper arguments.

I'm not sure if this belongs as a part of your project, feel free to reject and state it as such. This is definitely outside of the scope of neosdconv, as well as your goal for the project. However, on the other side, the script and table are pretty much useless without neosdconv. If you don't think its in scope here, I'll slap these in their own repo and call it good.

The script is pretty rough, it was meant as a quick way to not have to type in the args every time to neosdconv. For that purpose it works. It does not support child ROMs that require a parent, and I have no plans to take it to that point for a number of reasons. On a technical level, the ROM set distributions and file names are too variable to be handled by this simple script. It would be more manageable if the script and table used checksums. Using a more powerful scripting language would probably be wise here too.

In order to implement checksums, the script and the table would need to be expanded quite a bit. In this case, it would probably be a worthy goal to scrape the info from NeoBuilder or a site like the arcade database. But, as it is, the script and table have worked for every parent ROM I've tested. This includes a couple ROMs that were not in the NeoBuilder text file originally.

This is great! Thanks for doing this. I don't mind this being in the main repo. I like that the script just uses neosdconv's command line interface, keeping the two nice and separate.

I added a note about this script in the README.
Adding checksums would be nice down the road. But this is still super useful as-is, so I'm not too worried about them right now.

Cool :D
But also, I wanted to note that your changes to the README are not quite correct. The .csv needs to be in the same folder in the script, OR more args passed and the csv specified as the last argument e.g. cd scripts && ./convert.sh <dir of ROMs> or ./scripts/convert.sh <dir of ROMs> <output dir for .neo files> ./scripts/neosd_romset.csv
That behavior is the same when specifying a single ROM set zip as well. e.g. cd scripts && ./convert.sh input.zip or ./scripts/convert.sh /path/to/input.zip /path/to/output/dir/ ./scripts/neosd_romset.csv
I can change this behavior if you think thats easier. Would add a search path of ./ and ./scripts/ first

Ah ok. I added an extra sentence to the README:
The script is flexible on the locations of the input roms, output folder and the location of the csv file. For more information, run `convert.sh` without any arguments for help.


Internal QA Feedback enhancements

Describe the feature you would like to request
In the acceptance criteria, we've defined features we deemed important and easy to implement from the feedback we gathered from our QA round.
Provide a brief explanation of why this feature is necessary
Explain why is this feature necessary here.
Acceptance Criteria
List down criterias that should be included in this issue. This is to help reviewers look out for expected changes in PR.

[x] Disable Coming soon buttons in homepage (show disabled state)
[x] Move “My drops” to be a full nav item (similar to doc and get in touch) (make sure to check if the nav is responsive in mobile, if its breaks, it should remain in the menu)
[x] Change “All Drops” to “My Drops”
[x] Redirect to “My Drops” after user has connected with the wallet
[x] use outline variant for secondary cta button (e.g. Master key modal cancel button)
[x] replace the existing Master key icon in the menu dropdown with a key icon
[x] disable export cancel for empty table

Provide an estimation of how long the developers should spend on implementing this feature
4-5 hrs
References
Include any URLs, screenshots, or source code that would be helpful as reference or inspiration towards implementing the feature.
https://www.notion.so/airfoilhq/Internal-QA-Feedback-3d3b713ad7fb4e7a98a0238b8ec007f3

also solve #159


ngMock doesn’t support inject from before hooks.

Both Jasmine 2.1+ and Mocha support before hooks that run setup code once before all beforeEach hooks and test cases are run. angular-mocks.js does not use these hooks to configure currentSpec (the way, for example, it uses beforeEach to do so). When inject is called from a before hook, it returns without invoking workFn and therefore does not invoke the specified function block, configure the injector, etc.
We would like to use before hooks with ngMock to avoid repeating common, state-agnostic, possibly expensive test fixture setup work.

Whatever you put in before cannot be injected as, at that time, there is
no injector nor app running

I assume @lgalfaso's answer and the closed PR settle this.


Add LocalSingleStoreElimPass

This is another pass in a series of passes designed to reduce the size of
SPIR-V files.
For each entry point function, this pass eliminates loads and stores for
function scope variable that are stored to only once, where possible. Only
whole variable loads and stores are eliminated; access-chain references are
not optimized. Replace all loads of such variables with the value that is
stored and eliminate any resulting dead code.
Currently, the presence of access chains and function calls can inhibit this
pass, however the Inlining and LocalAccessChainConvert passes can make it
more effective.
This pass will reduce the work needed to be done by LocalSingleBlockElim
and LocalSSARewrite and can improve the effectiveness of other passes such
as DeadBranchElimination which depend on values for their analysis.

Please rebase against master

Rebased.

I think everything has now been addressed.

Rebased, squashed, and pushed into master as 0c5722fc019


[WIP] data factory

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/Azure/azure-cli-extensions.git@initial-datafactory#subdirectory=src/$EXT&egg=$EXT"


Remove myself from the maintainers

I'm afraid, but I'm not an active maintainer anymore.

I'll happily add you to the list again if you feel like doing some ES stuff. Thanks for your contributions and discussions so far!


Edited the cards in join us section

Use to install font awesome for react
npm install react-icons --save

can u send the ss of changes?

Is it okay to be merged or any changes required ??


Invalid book tag

i dont know why, but if i create a book or try to /ib open example_book it will show a book on screen but it only states invalid book tag

how do i fix this?

Please post the book config, probably there is an error with the syntax.


Remove extra stars add cv

This PR removes useless Ldwarf and Carbon templates but adds CV PCA templates.
Thanks to the archetype logic Ldwarf and Carbon templates are now used in archetypes.

@sbailey, please merge https://github.com/desihub/redrock-templates/pull/7 first, then let me do the required changes.


OStatus

We should support remote follow using OStatus.

I do some research, register myself in the ostatus group @evanp created but found no information about the progress of this issue.
there are some block? because if not I would suggest it as a very important issue to bring more users to pump.io

+1. same as @txt-file

Would definitely love to see this :)

It would be brilliant if I could follow people on indenti.ca from mastodon.social - see https://github.com/Gargron/mastodon/issues/138

Heya! Here's the deal:
This is almost certainly never going to make it into pump.io core since as Evan pointed out above, it makes more sense as a separate bridge. Anyone who wants this feature badly enough is welcome to step up and volunteer to write such a bridge (and I'll be happy to help out whoever does however I'm able), however this won't be high priority for pump.io roadmap for a looooong while. I.e. if no one does this independently, it simply won't happen. I'm really hoping that someone steps up to do this, since it would be pretty sweet. (+1s here are, as usual, extremely unlikely to change the above.)
Thanks for your interest in pump.io :)

Duplicated in: https://framagit.org/compa/compa/issues/3

At this point in time, I'd suggest writing an OStatus-to-ActivityPub bridge, since it'd be more useful.

@Serkan-devel we are cranking on it right now. I'd expect it for the next version of pump.io, either 5.2.0 or 6.0.0 depending on whether deprecation is backwards-compatible.

Any news on support for ActivityPub in pump.io?

is it still the plan?
any idea when it might become a reality?

As one of the pioneers of federated social networking, we'd love to feature pump.io prominently on fediverse.party. But the criteria include being able to federate with at least one other federated packages. AFAIK pump.io sites currently only federate with each other, is this correct?

@strypey the correct issue for that question is: https://github.com/pump-io/pump.io/issues/1241

Thanks a lot @vxcamiloxv ! I'll update our internal wiki to follow that issue instead of this one. But can you (or anyone) confirm that pump.io sites currently only federate with each other, or if this is not correct, let me know what other packages pump.io can currently federate with?

@strypey  currently only federate with each other, but probably @strugee  has more information


USHIFT-1395: Fix custom-rpm failure in build.sh

Custom RPMs are only used if specified explicitly from the command line.
Closes USHIFT-1395

/lgtm

/test verify


Backend test is failing.

Describe the bug
Test case "test/posts.test.js" is failing.
Test Suites: 1 failed, 1 total
Tests:       2 failed, 11 passed, 13 total
To Reproduce
Steps to reproduce the behavior:

Go into backend folder and run "npm test test/posts.test.js"
See error

Expected behavior
Should pass all test.
(to be specific, should return the correct response code)
Screenshots

Desktop (please complete the following information):

OS: MacOS
Browser : terminal/command line
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

I'm not getting the same error so I have no idea if this will help but,
try:
const authToken = await Authentication.generateAuthToken(user1)
instead of:
const authToken = Authentication.generateAuthToken(user1)
inside the posts.test.js file for the test cases that you failed.
The error you're getting right now is because you are failing to get the auth token in those two tests.

This was due to me not correctly following the README instructions.
The migration of database 'sequelize db:migrate --url "mysql://updogDev:password@localhost:3306/updog"' fixed the problem for me.


cardano-sl crashes during sync

Hi,
I have built a container as of this guide in a VM (using cardano-sl-explorer-static. in the nix-build command). I use version 3.0.2. I start it with the cardano-explorer command.
It starts fine and runs for about 30 minutes but then it crashes and the container stops. Sometimes it says "Failed to add headers from ... to block retrieval queue: queue is full" in the log but not always.
What I've tried so far:

restart container
add more RAM, reboot, restart container
add more CPU, reboot, restart container
None of them works, I cleared the database before each restart. Please help me, logs attached
Cardano_sync_error_log.txt
Cardano_sync_error_log_2.txt
Many thanks.

I also meet this problem. Do you resolve it? Anyone can help me?

How much RAM does the VM have? The IOHK devops team runs one on a 64Gig VM and if still runs out of memory, crashes and is restarted.
The existing explorer is of very poor quality and we are working on providing a new one.

How much RAM does the VM have? The IOHK devops team runs one on a 64Gig VM and if still runs out of memory, crashes and is restarted automatically.
The existing explorer is of very poor quality and we are working on providing a new one.

But during syncing data, it has bugs as picture

The "attempted to subtract" problem is know and has been looked at. We could not get to the bottom of it, and decided that since the explorer is going to be replaced anyway, it was not worth putting any more dev effort into this one.
Address balances pulled from the explorer can be incorrect. Data about block and transactions is thought to be 100% correct.


style: Improve closing state of the menu

Describe the bug
" It appears the animation on the menu are jerking a little bit. Thought I’d show you in case it was suppose to be a smooth close. And the part I’m seeing is the onclick to close the menu, it doesn’t appear to match the animation of the onclick to open the menu. "
To Reproduce
Steps to reproduce the behavior:
open the menu -> opening is smooth
close the menu -> closing the animation is abrupt
Expected behavior
add the same -but inverted- animation on the closing of the button
Screenshots

Desktop (please complete the following information):

OS: all
Browser all
Version [e.g. 22]

Smartphone (please complete the following information):

Device: all
OS: all
Browser all
Version [e.g. 22]

Additional context
Feedback from Slack from a very very good friend 👍

resolved in #57


Blocks ads for sure, but won't skip it

As I've used host files before, I know I've used them properly, but spotify still tries to play ads. When this happens, the program goes silent and on where the album cover is, I see the blank disc symbol with "Learn More" next to it. clicking pause or play makes no difference other than on the button itself. When I try to click on another song, the banner pops on top saying "The selected song will play after the ads"
But it certainly blocks the playable ads
Edit: I restarted my computer to see if it would help fix the problem, and it does but not in a way I'd prefer. Because ads play

I restarted my computer to see if it would help fix the problem, and it does but not in a way I'd prefer. Because ads play

Hi,
Also info on what ads are being played?!

they are the typical Spotify premium ads, pima institute, state farm, charmin, and army recruitment are a few that I've caught
By the way, thank you so much for your work on this! after restarting my computer, I've noticed fewer ads. It'd usually be about 4 ads each time ads played, but now it's only one or two each time

Thanks for information, will try to intercept those...
Also you are welcome.


_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m 31s
master passed

+1 :green_heart:
compile
1m  9s
master passed

+1 :green_heart:
checkstyle
0m 45s
master passed

+1 :green_heart:
spotbugs
1m 41s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m  7s
the patch passed

+1 :green_heart:
compile
1m  4s
the patch passed

+1 :green_heart:
javac
1m  4s
the patch passed

-0 :warning:
checkstyle
0m 45s
hbase-thrift: The patch generated 1 new + 3 unchanged - 0 fixed = 4 total (was 3)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
20m 51s
Patch does not cause any errors with Hadoop 3.1.2 3.2.2 3.3.1.

+1 :green_heart:
spotbugs
1m 48s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 16s
The patch does not generate ASF License warnings.

45m 51s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3984/3/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3984

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile

uname
Linux d50c57e45d99 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 9e469e4a2a

Default Java
AdoptOpenJDK-1.8.0_282-b08

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3984/3/artifact/yetus-general-check/output/diff-checkstyle-hbase-thrift.txt

Max. process+thread count
96 (vs. ulimit of 30000)

modules
C: hbase-thrift U: hbase-thrift

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3984/3/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 26s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m 31s
master passed

+1 :green_heart:
compile
1m  9s
master passed

+1 :green_heart:
checkstyle
0m 45s
master passed

+1 :green_heart:
spotbugs
1m 41s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m  7s
the patch passed

+1 :green_heart:
compile
1m  4s
the patch passed

+1 :green_heart:
javac
1m  4s
the patch passed

-0 :warning:
checkstyle
0m 45s
hbase-thrift: The patch generated 1 new + 3 unchanged - 0 fixed = 4 total (was 3)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
20m 51s
Patch does not cause any errors with Hadoop 3.1.2 3.2.2 3.3.1.

+1 :green_heart:
spotbugs
1m 48s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 16s
The patch does not generate ASF License warnings.

45m 51s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3984/3/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3984

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile

uname
Linux d50c57e45d99 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 9e469e4a2a

Default Java
AdoptOpenJDK-1.8.0_282-b08

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3984/3/artifact/yetus-general-check/output/diff-checkstyle-hbase-thrift.txt

Max. process+thread count
96 (vs. ulimit of 30000)

modules
C: hbase-thrift U: hbase-thrift

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3984/3/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Newer Safari versions not recognized for iOS 13.6-7

When I run a Google Analytics import, "Safari & Chrome for iOS 13.6" and "Safari & Chrome for iOS 13.7" show up in the Browsers Not Included column. Right now that accounts for just under 30% of my 30-day traffic. Any way we can get these recognized?

Looks like I missed those versions being added. Should now be fixed.

Yep! They import properly now. Thanks 😄


Ensure TWYS of longitude and latitude for shipping is preserved and continued

Geoff: please edit as need be:
Pat to establish the extent to which longitude and latitude was captured for ships at sea,.
The transcriptions of shipping at sea (usually? mostly?) contains latitude and longitude The current work to allocate ships at sea to the place where the census is returned will mean that we loose these transcriptions, which are needed for future developments (Pat to create details) for researchers, and may be needed by academic researchers.
Ships at anchor tend (mostly?) to have their position recorded in relation to a land mark or similar place (e.g. a lighthouse).
Options for how to preserve this data, and enable the censuses being transcribed now to capture it  include creating two new fields "TWYS Latitude" and "TWYS Longitude".
Or four - one for the figures (only figures can be entered) and one for the north-or-south only N or S can be entered and one for the east-or-west only E or W can be entered)

Pat--lat and long or any other info will not be lost---please refer to file RG9_4445.
Lat and long was sometimes recorded.
Ships at anchor sometimes recorded their position in relation to a landmark.
There would seem to be insufficient data to proceed as you suggest.

This appears to be the same as #1537.
Where the latitude and longitude are given they are already recorded in the where census taken field. If the position is given as text (e.g. Bristol Channel off Portishead) then that is entered in the where census taken field.
No additional field is necessary.
Geoff

Pat to explaine whiy it is different to #1537 and what else is needed.

#1537 is about using the degrees, minutes, seconds lat-long to create a digital lat-long.
This story is about not over-writing or otherwise losing the twys lat-long when #1537 is implemented (degrees, minutes, seconds)

@PatReynoldsFUG this should be addressed by Appendix D. If you review this and think there's further development needed, please note in the story

Appendix D is currently undergoing its final rewrite. I will send Pat a copy when this is completed.
Geoff


rust: ignore RUSTSEC-2020-0091 until dependency bumps

I already bumped arc-swap in #3511 (see last commit).


Prl 5301 citizen sos

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/PRL-5301
Change description
Citizen SOA - Statement of service
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

Merge into https://github.com/hmcts/prl-cos-api/pull/2544/
Can be tested as part of that PR
So closing this


add unit_of_measurement and device_class

Hi, please add this to Code
sensor.lightning_azimuth:
  unit_of_measurement: °
sensor.lightning_distance:
  device_class: distance

Already implemented, should be closed.


command 'opens1.refresh' not found

All commands not found.

It looks like casing by some vulnerabilities.After i using "npm audit fix",it was work.

It looks like casing by some vulnerabilities

No, it's probably not.
From my logs, the command is not found because the extension failed activation. And the reason is VS Code removed keytar in v1.83. See https://code.visualstudio.com/updates/v1_83#_keytar-removed-from-vs-code @NessajCN you may want to switch to use SecretStorage api.
2023-10-26 10:46:04.908 [error] Activating extension nessaj.opens1 failed due to an error:
2023-10-26 10:46:04.908 [error] Error: \\?\c:\Users\yanzh\.vscode\extensions\nessaj.opens1-0.5.3\node_modules\keytar\build\Release\keytar.node is not a valid Win32 application.
\\?\c:\Users\yanzh\.vscode\extensions\nessaj.opens1-0.5.3\node_modules\keytar\build\Release\keytar.node

@NessajCN BTW if you insist using keytar, it's ok. Then you need to publish platform-specific extensions for each supported platform. From my perspective, it's cheaper to adopt SecretStorage API.

@Eskibear Thanks for feedback. I'll find a replace.

I unblock myself by adopting SecretStorage API, and I can send out a sample PR for your reference, which simply demonstrates how to use it. Looking forward to a new version with the fix! (which is important for me to moyu 😆  )

The new release f54e156 should work. Update the extension to 0.6.0


create-service-key does not create IAM user

It creates the IAM group, but not a user.
Later, when deleting the service key, an error is in the logs for its inability to delete the missing user.
Binding to an application generates IAM user & credentials correctly.

FWIW, this works with https://github.com/cloudfoundry-community/s3-broker.


ciao-image: Add BAT testing for CIAO Image Service

We need to cover all ciao-image functionality in BAT.

Addressed with #1123


Small updates to LICENSE

Correct file content + make sure file ends with with a line feed

Replaced by https://github.com/dart-lang/setup-dart/pull/28


Territories which are disconnected don't count towards breakage

Fix the bug

Would be great to have some tests for this.

While having a test case may have prevented this issue in the first place, since we have now fixed this and the lock-out system isn't going to have any changes before it's removal at the end of this year the addition of a test case at this point seems of little benefit.

While having a test case may have prevented this issue in the first place, since we have now fixed this and the lock-out system isn't going to have any changes before it's removal at the end of this year the addition of a test case at this point seems of little benefit.

I appreciate this that it feels like this may not change, however I'm not sure this is the right approach:

experience shows that things change much more than you'd think and often in ways that you don't predict. The number of times I've thought "nah, this won't change, I don't need to write tests" is nonzero. The number of times I've been right about that assumption is zero.
as well as helping developers have confidence in correct behaviour, tests help readers understand the expected behaviour of the code and how it should be used

I've added tests for cut-off territories and locking territories through self-claiming

New territory names (assignment your discretion):

FL: Flailsbury
YL: Yamlot
HA: Hattersly
TH: Tudor Hammer
PL: Port Lucent
SF: Scarborough Fire


Format option missing in context menu of drives in overview.

What feature or improvement do you think would benefit Files?
The format option is a core feature of a file explorer. The ability to format usb sticks, sd cards and hard drives is crutial. The current workaround for me is the location list on the left where you can access the format drive option, I think it's confusing to have 2 different context menus for the same item (drives on the sidebar and drives on the main overview).
Requirements
No requirements
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.19045.2364
Windows Version
Windows 10 22H2 19045
Comments
No response

This already exists under more.../Format

Merging with #5888


Fix typo in name of output topic

This change fixes inconsistent naming for the output topic in the documentation.

@mmerdes Thanks for this contribution!


Update deptools

This PR updates the deptools collection
Installation sequence for tool-installers

[x] Test using @galaxybot test this
[x] Inspect CI output for expected changes
[ ] Deploy using @galaxybot deploy this if test install was successful
[ ] Merge this PR

@galaxybot test this

@galaxybot deploy this


TSX Support Not Working For Some .tsx Files

After updating VSCode, All JSX tags in .tsx files are all displayed as errors in my editor. Meanwhile, Webpack can still work without error.
An example of the errors, <div> is recognised as an error and the error message reads cannot find name 'div'. ts(2304). It seems that the parser interprets <...> as type casting instead of JSX tag even though the files name ends with .tsx.
All such error messages will disappear if I change the file name extension to .jsx.
I updated both August 2020 update and September 2020 update together. The error may occur from August 2020 update.
Steps to reproduce:

Open an empty folder
Run command npx create-react-app my-app --template typescript
Open file my-app/src/App.tsx. The file should look good.
Create another file under src folder and name it as App1.tsx.
Copy everything in App.tsx into App1.tsx. You should see the error messages. (A mystery here is that the error messages won't show if you use command line cp App.tsx App1.tsx.)

The issue is no longer reproducible after resetting the ESLint extension.


Improvements to main application with empahsis on documentation and usability

Also:

updated bootloaders and main application configuration in preparation for Mbed OS 5.11.
new targets support

I'm not happy that this adds 50 lines of code (33% more!) on main.cpp.


_embed does not work when using a custom id

I'm mocking up a .NET API which uses Pascal case so I have to use a custom Id.  In doing so, _embed will not populate due to the custom Id.
db.json
{
  "Posts": [
    { "Id": "1", "Title": "json-server"}
  ],
  "Comments": [
    { "Id": "1", "Body": "some comment", "PostId": "1" }
  ]
}

Run cmd
> json-server -w db.json -i Id

Request
GET /posts?_embed=Comments

Response
[
  {
    "Id": "1",
    "Title": "json-server",
    "Comments": []
  }
]

Request
GET /posts/1?_embed=Comments

Response
{
  "Id": "1",
  "Title": "json-server",
  "Comments": []
}

I think the issue is on ./src/server/router/plural.js line: 21
// The id will prob have to come from resource[db._.id]
query[`${singularResource}${opts.foreignKeySuffix}`] = resource.id;

I tried to fix this but it broke a bunch of tests.  Unfortunately, I'm in the middle of a project and don't have enough time to inspect and submit a PR.  I'd appreciate it if someone could fix this.  If not, I'll submit a PR at a later time.  Thanks!

I'm facing the same issue. Are there any updates? Thank you.


Update native-modules-setup.md

Hi @LadyGaga1208,
Nice meeting you as well, but I hope you understand we won't merge this.


Rounded Corners not working.

Using Jellyfin Version: 10.7.6 | Operating System: Linux
Corners are not rounded when using multiple import lines. rounding.css as well as rounding_circlehover.css are not working.
These are the import lines that I'm using-
@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/fixes.css');
@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/jf_font.css');

@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/base.css');
@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/accentlist.css');

@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/rounding_circlehover.css');

@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/smallercast.css');

@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/episodelist/episodes_grid.css');

@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/header/header_transparent-dashboard.css');

@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/login/login_minimalistic.css');

@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/fields/fields_border.css');

@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/cornerindicator/indicator_corner.css');

@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/type/colorful.css');

@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/titlepage/title_simple.css');

@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/bottombarprogress.css');

@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/effects/hoverglow.css');

@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/effects/glassy.css');

@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/effects/pan-animation.css');

Screenshots

You need to actually set the amount of rounding you want, as shown in step 14.

Ahh, sorry. Looks like I had missed out that point. Thanks!


adding support for Anthropic/Cohere/Llama2

Hi @jedan2506 @Tarraann  ,
Noticed y'all are calling multiple LLM Providers (Palm, Openai). I'm working on litellm (simple library to standardize LLM API Calls - https://github.com/BerriAI/litellm) and was wondering if we could be helpful.
Added support for Anthropic, Llama2, and Cohere by replacing a new Lite class with completion from litellm. The code is pretty similar to the OpenAI class - as litellm follows the same pattern as the openai-python sdk.
Would love to know if this helps.

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

bump on this?  @jedan2506 @Tarraann

Yes, please!


fix reported issue #61, non-audo-cd bug fix and add timestamp to logs

Corrects error reported to root@localhost in log reported in issue #61 when a non-audio disc is inserted
Also added a timestamp to each logfile (line 20)

Thanks, @cbunt1


Error when searching for a single parenthesis in the search box

What is the current behavior?
When searching for ( in the search panel, the system raises an error:
Something went wrong. Please ask your system administrator to share the logs by creating an issue on Github or sending a message via Slack
What is the expected behavior?
All the texts containing a single parenthesis should appear in the results results on the right panel
Steps to reproduce the problem:

Open a workspace
Search for ( in the search panel

Environment:

OS: MaxOS
Web browser: Chrome
Python version: 3.9
Label Sleuth version (as it appears on the bottom left of the UI): 0.14

Suggested solution (if any):

Resolved by #572.


cloudfoundry-cli: 6.41.0 -> 6.45.0 (fixes CVE-2019-3781)

Motivation for this change
Update, also fixing CVE-2019-3781
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

BTW I'm planning on bumping 19.03 to 6.43.0 rather than directly bumping this as it still fixes the CVE yet has less risk of behaviour change.

BTW I'm planning on bumping 19.03 to 6.43.0 rather than directly cherrypicking this as it still fixes the CVE yet has less risk of behaviour change.

Sounds good to me :+1:


feat: Added buttons add all roles and clear all roles for user

Task Link: https://app.asana.com/0/1194051119161697/1199997172288575/f
Description: Display add_all_roles and clear_all_roles buttons on user page
Gif:

I guess there is only an 'if statement' removed @hrwX @meshramsaroj

Yes that is correct. And it works perfectly fine since they were hidden for user doctype


Clarification Question

Hi there - this project looks really cool. Short question - is it possible to deploy pytorch as a lambda layer s.t. it only hosts the pytorch package and can thus be imported from other layers/funcs using torchlambda?
Thanks!

Statically compiled PyTorch C++ frontend is included when you run torchlambda build and it is deployed as a package to AWS's Lambda. You can use anything the C++ frontend (libtorch) can use in your C++ deployment code and can use most (any?) models compiled by torchscript provided they fit on a single layer (up to 50Mb size, at least currently).
Does that answer your question?

hmm i see - ok I should dive more into how to best use torchscript in that case. Thanks for getting back!


Create Song “2024-09-02/index”

Automatically generated by Netlify CMS

👷 Deploy Preview for sprightly-palmier-1465e0 processing.

Name
Link

🔨 Latest commit
1c8050de1214da830ab10114360dbd3514a5b6a1

🔍 Latest deploy log
https://app.netlify.com/sites/sprightly-palmier-1465e0/deploys/66d4e97eab186900089b5efa


Throwable capture spheres

The capture crystals are good but wouldn't it be cool if you could throw them and have a capture chance based on combat power and so you could cap at a distance

The throwable part of this is done, but not so much the combat power bit. That's covered by the idea of graded spheres though, so I think we can close this out.

The throwable part of this is done, but not so much the combat power bit. That's covered by the idea of graded spheres though, so I think we can close this out.