I'm not able to run code developed using dbt directly in Databricks.

Describe the bug
I'm not able to run code developed using dbt directly in Databricks.
I'm trying to configure a CI/CD process using Databricks jobs to automatically run the SQL codes. We usually do this using Airflow and this is the first time we are configuring it directly in Databricks. However, when trying to connect, DBT (or Python, I don't know) gives an error that it doesn't have the logs function.
Steps To Reproduce

Create a venv with Python 3.8.10
py -m venv venv
source venv/Scripts/activate

In this case, I'm on Windows 10, but I don't think it will impact the next steps for reprodute.

Install and upgrade dbt-databricks

pip install dbt-databricks
pip install --upgrade dbt-databricks

Generate a dummy project

dbt init jaffle_shop

Check if connection is working

dbt debug

Commit all changes in a repository test

Add repository in Repos Databricks and give a pull refresh

Create a notebook in same diretory for test

Install requirements and debug dbt
In this case, we have only dbt-databricks to install

%pip install dbt-databricks
%sh dbt --version

%sh dbt debug

Expected behavior
The expected result of dbt debug is the connection indicator working.
Like this:

Screenshots and log output
If applicable, add screenshots or log output to help explain your problem.
System information
The output of dbt --version:

Additional context
We tried to install another version of dbt-databricks, but every time we try, the installed version always comes 1.0.4, as if it didn't obey the command directed.
As happen in this case:

@ac-garcia thanks for opening the issue. If you are trying to run a dbt project as a job, we are actually previewing this capability right now. Would you be interested in trying it out? Then there's no need to run a notebook.

@ac-garcia thanks for opening the issue. If you are trying to run a dbt project as a job, we are actually previewing this capability right now. Would you be interested in trying it out? Then there's no need to run a notebook.

Hi @bilalaslamseattle ! Thanks for the answer. I'm interested in testing it. Just help me with the steps I should follow to do this experiment, please.

hi @bilalaslamseattle I would also like to test this, thanks!

Hi @bilalaslamseattle , I would love to give it a go as well. Can you please share how we can test it?

@ac-garcia @binhnefits @guizsantos I'm going to set up a "drop in" demo for next week. I'd love to get your feedback on the experience and, if you like it, we can enroll you in the preview right there.
Here's are a couple of times next week that you can drop by:

EMEA-friendly
North America-friendly

Looking forward to seeing you there!

I'm going to close this issue as this is not really a bug, and we have a feature under development (dbt task type in Databricks Workflows) which solves for the requirement.


Update fhir-meter README with search examples

Is your feature request related to a problem? Please describe.
fhir-meter has a wealth of advanced search cases that the community might be interested in.
however, to know what is in there, you need to download jmeter, import the script, and then inspect it.
it would be great if we had an overview in the README so that users will know if this is something they are interested in or not
Describe the solution you'd like
Update the fhir-meter README with search examples, potentially grouped/categorized in some useful way.
Describe alternatives you've considered
Have some page on our project site with the searches and their results over time.
Acceptance Criteria
Additional context

the searches are added


Support for parameter choices

I really like your library so far, thanks for the good work.
You could make it more useful by adding support for parameters with limited set of acceptable values. This typically maps to using enums as the java type for the parameter. I've tried to find other CLI parsers for Java that support this and I've only found argparse4j.
This can be viewed as a special case of parameter validation, but in my opinion it's common enough so that implementing this would help a lot of people (since there is no proper validation). Of course, you can solve the problem in the client code but having this in the library could prove so much better (possibility to print the acceptable values in help text). The values could be specified in the annotation or determined automatically in case of an enum.

Your timing is impeccable, sir!
Please take a look at the release notes file in master.
One the items for the upcoming 3.2 release is that the description for an option or positional parameter can include a ${COMPLETION-CANDIDATES} variable.
For enums, this expands to a list of the enum values. Enums also validate values during type conversation.
Options and positional parameters can also specify a class that generates completion candidates. This is intended to be used for command line completion, especially for jline2 integration.
However, picocli will not do validation against completion candidates. It considers completion candidates like suggestions for the user. For example, a --file option (with type File) can show all files in the current directory as completion candidates, but the option may accept a file path outside that directory.
Generally, I haven’t added a Validator interface for options because I cannot see how this would add much to the application doing the validation. Also, such an interface would only be a partial solution since many applications need to validate combinations of options. I’m open to be convinced otherwise but for now I’m not focusing on validation.

Great news indeed!
The thing for validation is you often need just a set of values. The set need not be unnecessarily small (e.g. enum), but can be a (larger) subset of a primitive type (let's say integers from 0 to 100; impractical for enums). You're right that you won't solve complicated validation use cases and you probably even shouldn't. Keeping it simple has its own appeal. When I think about it, my example could be probably solved even now by having a custom converter.
The use case I had in mind for validating enums is something more (end)user friendly. Trying a value not in the given enum type as a parameter gives the user a not really nice java exception. Good for a java dev, bad for the user. I was thinking the user could get more useful error message like

Invalid 'algorithm' specification: 'AES-129'. Valid values are: 'AES-128', 'AES-256'

Maybe giving the exceptions more structure like adding reference to problematic parameter and a non-string cause-of-the-parsing-error specification would allow me to implement it myself in my client code. Or maybe there's another way that I'm missing.
Anyway, the 3.2 release goes a long way (especially with the completions).

About error handling: picocli throws a ParameterException if the user specified invalid input (e.g. value could not be converted, missing mandatory parameters etc).
If you use CommandLine.run(new YourRunnable(), args) in your main method, the default exception handler will catch the ParameterException, print the message of that exception and show the usage help message. No stack trace is shown.
However, if picocli called run or call on a command and the run or call method threw an exception, picocli will throw an ExecutionException wrapping the original exception. I believe this is a reasonable compromise.
So, the error message is very important since that will be shown to the user if they specified an invalid crypto algorithm, for example.

Please consider my pull request #406 to address this feature request.

@dhait  Thanks for the PR! It has been merged into master. I may make some changes to combine some constructors.

I combined some of the constructors and updated the release notes. Thanks again for the PR!

LOL, I haven't been at a computer for a couple of days. What a progress - commit, PR, merge. I applaud you, gentlemen. Seems like chosing picocli has been a good idea.
Anyway, I've renamed the issue.

If you like the project, please star it on GitHub and tell your friends! :-)


About the RegDB dataset

could you piease share me the RegDB dataset?  I cannot find it at the official web.
email<EMAIL_ADDRESS>

completed


Delayed focus added for mweb transition

Details
Navigation to IOUConfirmationList auto focuses field on mweb even before the transition is ended, which makes animation buggy
Fixed Issues

$ #10698
Tests / QA Steps

Login at staging.new.expensify.com
Click the FAB button again and select Request Money
Input any amount and click Next
Select any user account
Input any text in the field "What's it for?"
Click "Back" button <
Select any user account again
Verify that the cursor is in right place and that the selection is not shifted

PR Review Checklist

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] iOS / native
[x] Android / native
[x] iOS / Safari
[x] Android / Chrome
[x] MacOS / Chrome
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product was added in all src/languages/* files
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

PR Reviewer Checklist
The reviewer will copy/paste it into a new comment and complete it after the author checklist is completed

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] iOS / native
[ ] Android / native
[ ] iOS / Safari
[ ] Android / Chrome
[ ] MacOS / Chrome
[ ] MacOS / Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots

Web

https://user-images.githubusercontent.com/70144379/195667866-795f5d92-245f-4ad6-9444-4c8c27edb6af.mov
Mobile Web - Chrome

https://user-images.githubusercontent.com/70144379/195668515-b6957883-5425-41c9-8a31-647047a3c033.mov
Mobile Web - Safari
https://user-images.githubusercontent.com/70144379/195024214-0f262013-7239-4770-9549-9e014bacd473.mov
Desktop

https://user-images.githubusercontent.com/70144379/195669630-ca8981e3-1e0a-4806-bbd2-793e13de9d3d.mov
iOS

https://user-images.githubusercontent.com/70144379/195667497-7ddf4cfa-674b-41b7-bdcb-66396b5fde35.mov
Android

https://user-images.githubusercontent.com/70144379/195668362-148b4784-b514-4e78-a7f7-609140ce2d51.mov

I have read the CLA Document and I hereby sign the CLA

@Uros787 you'll need to get this PR finished up before I will begin reviewing it:

We need videos confirming that this works on all platforms, not just mobile web. I know the original issue was present in iOS Safari, but testing on all platforms helps ensure that we aren't going to introduce a regression with this change.
Please fill out the PR author checklist to the best of your ability. If you find that some items are not relevant to this PR, check them off anyways as recognition that you've read and considered their content.
You're missing testing steps. If the testing steps and QA steps are the same here, you can simply combine them under a single header Tests / QA Steps

Thanks

hi @roryabraham thanks for the help.
Just went through the checklist, and also added videos for all platforms except ios and desktop.
I will add ios in the morning but I am having trouble running desktop.
Could you help me out with that?

@roryabraham all done

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @roryabraham in version: 1.2.16-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @sketchydroide in version: 1.2.16-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


copr: remove nullable signature for expression

Signed-off-by: Edward Elric<EMAIL_ADDRESS>
What problem does this PR solve?
What is changed and how it works?
Remove Option from bit_count signature
Related changes
Check List
Tests

Unit test

PTAL @TennyZhuang

/merge


Cacheable() not working

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[x] Support request
[ ] Other... Please describe:

Current behavior
Actually I've written a dataservice which contacts my api to get data from it. For this case a service is extended from the data.class.ts:
import { HttpClient } from '@angular/common/http';
import { cacheable, EntityState, EntityStore, QueryEntity } from '@datorama/akita';
import { Observable, of } from 'rxjs';
import { tap } from 'rxjs/operators';

export class Data<T> {

  private readonly API_URL = 'http://localhost:3000/api';
  private _request: Observable<any>;

  constructor(private query: QueryEntity<EntityState<T>, T>, private endpoint: string, private http: HttpClient) {

  }
  public get(_id?: string): Observable<T> {
    return this.call(HttpMethod.get, _id);
  }

  private call(method: HttpMethod = HttpMethod.get, id?: string, data?: T): Observable<T> {
    let request$: Observable<any>;
    const mockData = { name : 'test-product' }

    if (this.query.getHasCache()) {
      console.log(`${this.endpoint} from cache`);
    } else {
      console.log(`${this.endpoint} not from cache`);
    }

    switch (method) {
      case HttpMethod.get: request$ = of(mockData).pipe(tap(this.updateStore)); break;
    }
    return cacheable(this.query.__store__, request$);
  }

  private updateStore = (data: any) => this.query.__store__.update(data);
}

enum HttpMethod {
  get,
  post,
  patch,
  delete
}

A proper Service would be created by me like this (product.service.ts):
import { Injectable } from '@angular/core';
import { HttpClient } from '@angular/common/http';
import { Data } from '../classes/data.class';
import { Product } from './models/product.model';
import { ProductQuery, ProductStore } from './stores/product.store';

@Injectable({
  providedIn: 'root'
})
export class ProductService extends Data<Product> {
  constructor(_store: ProductStore, _query: ProductQuery, http: HttpClient) {
    super(_query, 'config', http);
  }
}

My product.store.ts:

import { Injectable } from '@angular/core';
import { EntityState, EntityStore, QueryEntity, StoreConfig } from '@datorama/akita';
import { Product } from '../models/product.model';

export interface ProductState extends EntityState<Product> {

}

@StoreConfig({ name: 'products' })
@Injectable({
  providedIn: 'root'
})
export class ProductStore extends EntityStore<ProductState, Product> {
  constructor() {
    super();
  }
}

@Injectable({
  providedIn: 'root'
})
export class ProductQuery extends QueryEntity<ProductState, Product> {
  constructor(protected store: ProductStore) {
    super(store);
  }

  get(): Promise<ProductState> {
    return this.selectFirst().toPromise();
  }
}

My product.model.ts
import { ID } from '@datorama/akita';

export interface Product {
  _id: ID;
  name: string;
}

export function createProduct({ name }: Partial<Product>) {
  return {
    name
  } as Product;
}

How I query the data:
import { Component } from '@angular/core';
import { Observable } from 'rxjs';
import { ProductQuery, ProductState } from './shared/stores/product.store';
import { ProductService } from './shared/product.service';

@Component({
  selector: 'app-root',
  templateUrl: './app.component.html',
  styleUrls: ['./app.component.scss']
})
export class AppComponent {

  constructor(private productQuery: ProductQuery, private productService: ProductService){}
  productFromService: Promise<ProductState>;
  productFromQuery: Promise<ProductState>;

  ngOnInit(){

    // Product is filled
    this.productFromService = this.productService.get().toPromise();

    // Product is not filled
    this.productFromQuery = this.productQuery.get();
  }
}

In 99% of my use case, the data is retrived by the dataservice, so I prefer the hard wired way to get them over a query but with this implementation, there is actually data in the localStorage (because I presisted it) but it seems there is no data in the store (when it's cached). The service always returns the data from the dataservice, but the query seems to be empty (especially the store).
Expected behavior
The cached instance of the data should be returned.
Minimal reproduction of the problem with instructions
https://stackblitz.com/edit/github-kfnhy2
What is the motivation?
I want to reduce the boilerplate code for a small project.
Environment
@angular-devkit/build-optimizer   0.13.9
@angular-devkit/build-webpack     0.13.9
@angular-devkit/core              7.3.9
@angular-devkit/schematics        7.3.9
@angular/cli                      7.3.9
@ngtools/webpack                  7.3.9
@schematics/angular               7.3.9
@schematics/update                0.13.9
rxjs                              6.3.3
typescript                        3.2.4
webpack                           4.29.0
Browser:

[x] Chrome (desktop) version 77.0.3865.120
[ ] Chrome (Android) version XX
[ ] Chrome (iOS) version XX
[ ] Firefox version XX
[ ] Safari (desktop) version XX
[ ] Safari (iOS) version XX
[ ] IE version XX
[ ] Edge version XX

For Tooling issues:

Node version: v10.16.3
Platform:  Windows x64

You need to call setHasCache(true):
private updateStore = (data: any) => {
   this.query.__store__.setHasCache(true);
   this.query.__store__.update(data);
};

Only the set method calls it internally.

private updateStore = (data: any) => {
   this.query.__store__.setHasCache(true);
   this.query.__store__.update(data);
};

Oh wow, thanks for the really fast reply. Really appreciate this!
Updated the stackblitz according to this. but as you can see, the data is empty from the servicethe data from the query is null.

You have several issues:

You need to change the idKey to _id. https://netbasal.gitbook.io/akita/entity-store/entity-store/entity-id

You are using toPromise which resolved once - remove it.

You have several issues:

Based on your model, you need to change the idKey to _id. https://netbasal.gitbook.io/akita/entity-store/entity-store/entity-id

For simplicity I've changed this to the default id.

You are using toPromise, which resolved once - remove it.
I actually removed all toPromise but the only thing which has changes it, that there is not data from the service.

The query returns nothing as well.
I guess there is something strange with the store, but I don't get whats missing at all.

You are working with entities. You need to call this.query.__store__.set/add(data), not update.

You are working with entities. You need to call this.query.__store__.set/add(data), not update.

Doh! That makes sense. Sorry for bothering you with that 😀

You need to call setHasCache(true):
private updateStore = (data: any) => {
   this.query.__store__.setHasCache(true);
   this.query.__store__.update(data);
};

Only the set method calls it internally.

@NetanelBasal Can you update the docs from Store.setCache to Store.setHasCache?
https://netbasal.gitbook.io/akita/entity-store/cache-functionality

Sure, thanks.


"Start MO2" starts with the top profile (not the profile selected)

To solve this, the ModOrganiser.ini might need editing dynamically to edit the selected_profile

Fixed with https://github.com/Wildlander-mod/Launcher/issues/293


Newline characters are stripped from Say text

Reported by Cadmus (via twitter)

Turns out Unity's textarea automatically converts '\n' to '\n' so you can't enter escape codes directly in a text area. Fortunately, you can just add newlines in the actual text which do display as expected in the Say dialog.


Ignore time intervals

Hello！
Is there an option to ignore time intervals smaller than a certain time period and incorporate that time period into the previous scene? Or in order to keep the api simple, now can only filter according to the judgment conditions, and then use for example -- split_video_ffmpeg?
Thanks

There is the -m, --min-scene-len option you can give to scenedetect (documentation here). This takes a timecode as an argument and will ignore any detected scene transitions that happen less that that amount of time or number of frames since the previous transition.
For example
scenedetect -i video.mp4 -m 5s detect-content

would ignore any detected scene transitions that happen less than 5 seconds from the previous detected scene transition. So, it would just be part of the scene before it.

There is the -m, --min-scene-len option you can give to scenedetect (documentation here). This takes a timecode as an argument and will ignore any detected scene transitions that happen less that that amount of time or number of frames since the previous transition.
For example
scenedetect -i video.mp4 -m 5s detect-content

would ignore any detected scene transitions that happen less than 5 seconds from the previous detected scene transition. So, it would just be part of the scene before it.

Ok, thank you very much. Could you plz tell me the python call api equivalent to this CLI？

Using the python api, you would set min_scene_len per detector when you create it (default is 15 frames, see here for definition in the __init__ function of the ContentDetector).
One difference between the cli and python api is, I believe, the python api expects min_scene_len to be an int corresponding to a minimum number of frames. By contrast, using the cli, you can either specify the number of frames or a timecode. If you need to programatically find the framerate of your video, you should be able to access that through a property of the VideoStream object. So, let's say you open video.mp4 using
from scenedetect import open_video

video = open_video("video_file.mp4")  # This returns a VideoStream object

you can then access the framerate of the video through video.framerate.
@Breakthrough I am just realizing from this conversation how the cli and python api treat min_scene_len differently. Is my understanding of this correct? And, if so, should min_scene_len be made more flexible for the python api in a future version or set at the SceneManager level rather than per detector?

Your understanding is correct!  The current API could be updated to use a FrameTimecode for detectors.
The idea I have for the long-term 1.0 API would be something where detectors simply generate a set of events (e.g. Event.CUT, Event.START for fade in, Event.END for fade out), and feed those to a SceneManager.  Then, SceneManager could be responsible for actually respecting min_scene_len by dropping CUT events that are too close together when you call get_scene_list().  One benefit of doing this as a post-processing step is the ability to call multiple times with different lengths without needing to run detection again.  This would break how callbacks currently work, but perhaps that is okay as doing this kind of filtering online in a callback is relatively easy (e.g. ignore Event.CUTs that are just too close to the last one).
That being said your suggestion is a good idea and might bring us one step closer to that vision.  It's certainly possible to deprecate min_scene_len on detectors, and add a new argument for that purpose to SceneManager.get_scene_list().  One downside of this approach though is it will not work correctly with the callback function that detect_scenes() takes, so this may need to be an intrinsic property of a given detection run (e.g. passed to detect_scenes() or made a property of SceneManager) and handled as an online filter.

thanks！close


server: correctly return and close ch from exits routine

Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>

LGTM

LGTM
/test all


HealthChecks class in website assembly not detected

Hi,
I don't know if it is a wanted behavior or not. The HealthCheck defined in the web application are not found when adding healthchecks.
Version of AppMetrics : 1.0.0-beta4
Steps to reproduce :

Create a new Solution
Add a new dotnet core library project,  with a HealthCheck class labeled "lib"
Add a new dotnet core asp.net project, with basic appmetrics configuration (like the mvc.sample project)
Add reference to previous lib project
Add new HealthCheck class labeled "web".
Run
When looking at /health, we can see only the "lib" healthcheck, not the "web" one.

Thanks for identifing this. This issue has been fixed in 1.0.0-beta5, tested using the latest samples.
I'll be releasing 1.0.0-beta5 sometime this week, otherwise you can try the dev build on myget


Define a memory slab type to ensure that memory allocated with malloc() is freed with free()

valgrind turned up an issue that the default unique_ptr deleter was using delete[] on memory that was allocated using malloc()-like aligned_alloc().  This defines a memory_slab_t with a custom deleter to ensure that free() is called.

Coverage increased (+0.02%) to 29.788% when pulling 2073a7b11ac98e3c886f6c0a391b4d47f2a59e19 on memory_slab_t into 80579e11214b1c278ed27eb26c7aa139fdedec93 on master.


Rework ThemeUtils.java

Particularly:

Activity's theme should be managed in RiotAppCompatActivity, cause new Activity can forget to add an entry in ThemeUtils.setActivityTheme()
Activity can be restarted by ActivityLifecycleCallbacks implementation, but what happen if Activity has been started with startActivityForResult()? I think the result will be lost

For instance AccountCreationCaptchaActivity has been forgotten, the theme is not set properly

AccountCreationCaptchaActivity has now a theme set properly


WIP - Pages inheritance from layout.html

Each page extends the layout.html in the block tag
The content of each page stays the same
Each page content is wrapped by a 'block' tag

#66
Co-Authored-By: ilanKushnir<EMAIL_ADDRESS>

looks great to me!


GLRIDB-493

This fix looks fine. I only line commented for curiosity. However, it also curios, when the story said it used to work at one point, that there was additional display required.

Please include a more detailed message in the commit.


feat(react-components): add Text

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[textanalytics] add back extractive summarization

Resolves https://github.com/Azure/azure-sdk-for-python/issues/26186
Adds back extractive summarization from the 5.2.0b4 version of the library.
No active endpoint for this API version so tests will fail.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-textanalytics

Looks good. Do you plan to add the bespoke method in another PR?

First trying to get all the new features in, then will add bespoke methods if time permits for this beta.


[TOMEE-4354] Implicitly create ContextService when needed

Resolves TOMEE-4354
Makes the following configuration work again on TomEE 10 (note that Context is omitted):
  <Resource id="Mes" type="ManagedExecutorService">
    Core = 5
    Max = 25
    KeepAlive = 5 s
    Queue = 15
    ThreadFactory = org.apache.openejb.threads.impl.ManagedThreadFactoryImpl
    Lazy = true
  </Resource>

Also ensures that §<IP_ADDRESS>, §<IP_ADDRESS>, §<IP_ADDRESS> and §<IP_ADDRESS> of Concurrency 3.0 spec are fulfilled

maybe just change the default of openejb.environment.default instead of the code which prevent to disable it (main case)

These services were added unconditionally before as well until https://github.com/apache/tomee/commit/2c2fa2d95a68ded5ef52526b27838915730ff339 to fix ConfigurationFactoryTest, I reverted this again and fixed the test differently

Think the original intention of the flag was to prevent creartion of JMS stuff (if one looks into the history)

Not really, this 9 years old behavior intent is to not create default resources when it is not needed (in all apps except TCK ;)) so think it is sane to revert back to the original behavior and just use the toggle for tests/tcks.
My 2cts

@rmannibucau I see. Just looked in 8.x and it was like that before, so I agree, that we might just want to change the default and people not needing it, can still disable.

Well there are a lot of things there. If you don't care of the detail just read the last sentence ;).

It is not about the TCK/tests, it is more or less about the requirement to have a context service available to have an managed executor service.

Hmm I understand "it is 100% about TCK" - cause requirements are just about them for me and embedded defaults/coded behavior about usability/users.
I didn't check recently but when it was added (in 7?) I also added a Lazy support for resources but it is way different than this default which disables undesired resources by default (sane default for all cases from embedded daemon to prod ear) and it was still impacting negatively the startup (tests)).
So from my perspective - I still support it but use it way less today - it is wrong to keep it by default until bundled (it is ok to set the toggle to something else in the .zip/.tar.gz since people are unlikely to use it if you follow me.

Or would you say, the solution for people needing to configure a custom managed executor service is: configure the toggle and be happy about it? (so actually a doc update, which mention it?)

Personally I think we shouldn't enable any default resource but ultimately people willing a tck compatible instance could set it up (this is actually what is done, a ton of flags are needed to respect the spec requirements and nobody cares).
Or we just provide a default system.properties in distro which is compliant (people rarely use it I think if they install tomee properly with base/home).
So I would enphasize the doc, probably explain a bit more the reasoning and why being explicit there is important and explain that we handle default resources - it is openejb default behavior - as soon as a resource of the same type matches or is known it works (tests of openejb-core use that for ex).
Note the name is not important there until you enable exact matching (linking) - another toggle nobody uses probably.
The issue specifically with thread pool: there is no default correct thread pool in java. In a pure world where everything is reactive you are either a worker thread (computing intensive) or IO thread (latency friendly but low need) so from there you already can't get a single thread pool or banalised tasks as in EE concurrency. Now in Java we are still with some blocking tasks which are yet another kind of tasks. When you mix it all with a default executor service algorithm - mainly queue based - you can interlock and you can't - as a container - configure it right for any application, so default never works except in rare test cases but it hides the fact you have to configure it for prod (often multiple instances).
Side note: tomee keeps a default auto resource creation normally (in AutoConfig) from service providers so it should just work smoothly - as tests show.
The issue is more an implementation of the factory - wrapper of the actual executor instance - which does a lookup assuming all resources are bound (https://github.com/apache/tomee/blob/main/container/openejb-core/src/main/java/org/apache/openejb/resource/thread/ManagedExecutorServiceImplFactory.java#L74).
This one is quite trivial to fix, instead of failling is context is empty, null or missing in JNDI, just do a new ContextServiceImpl (maybe with a lazy logger info log line to still catch configuration errors with logs - easy to do using from application resources which have an offset).

I was thinking about two ways to fix this, either fix the factory so it just works (like you suggested Romain) or to make sure there is a ContextService bound in JNDI
Do you think we should still have to make sure to bind the ContextService (so not just new ContextServiceImpl() and call it a day) in the ManagedExecutorService factory? Otherwise there would be a bunch of implicitly created ContextService instances if one configures more than one MES/SMES

fair enough, I have next to no experience using JNDI so I guess you are right ;)
Singleton in factory sounds like a good option

reworked now

LGTM, too :) - thanks @rmannibucau for review, feedback and (long) explanation. It is really appreciated!
The CI gave some test failures originating from the change:
Testergebnis (20 fehlgeschlagene Tests / +18)
org.apache.openejb.arquillian.ManagedExecutorServiceGetPrincipalInTaskTest.run
org.apache.openejb.config.BuiltInEnvironmentEntriesTest.testRed
org.apache.openejb.config.BuiltInEnvironmentEntriesTest.testBlue
org.apache.openejb.threads.CompletableFutureTest.testSupplyAsyncAndCompleteExceptionally
org.apache.openejb.threads.CompletableFutureTest.testSupplyAsync
org.apache.openejb.threads.CompletableFutureTest.testRunAsync
org.apache.openejb.threads.InjectionTest.checkUserTransactionIsAccessible
org.apache.openejb.threads.InjectionTest.checkInjections
org.apache.openejb.threads.ManagedExecutorServiceTest.checkCdiContext
org.apache.openejb.threads.ManagedExecutorServiceTest.runnable
org.apache.openejb.threads.ManagedExecutorServiceTest.checkEjbContext
org.apache.openejb.threads.SecurityPropagationTest.checkItIsTrue
org.apache.openejb.server.cxf.rs.SuspendedTest.run
org.superbiz.executor.ManagedScheduledServiceTest.singleFixedDelayTaskWithException
org.superbiz.executor.ManagedScheduledServiceTest.singleFixedDelayTask
org.superbiz.executor.ManagedScheduledServiceTest.periodicFixedDelayTaskWithException
org.superbiz.executor.ManagedScheduledServiceTest.periodicFixedDelayTask
org.superbiz.executor.ManagedServiceTest.managedInvocationTest
org.superbiz.executor.ManagedServiceTest.managedInvocationTestWithException
org.superbiz.executor.ManagedServiceTest.managedInvocationTestWithTimeout

Full logs here: https://ci-builds.apache.org/job/Tomee/view/tomee-10.x/job/pull-request-manual/113/#showFailuresLink
FYI: @jungm

New build is green except for a test I can't get to fail locally: https://ci-builds.apache.org/job/Tomee/view/tomee-10.x/job/pull-request-manual/116/


bot framework error code 407

hi, I have this message when I try to debug my bot deployed in azure:
[17:41:15] -> POST Refresh access token failed with status code: 407

407 is a Proxy Authentication status code.  Is the emulator behind a proxy?


removed wedbio.com - seems like site isn't available

I've made a pull request to remove "wedbio.com" from the JSON data as it appears to be unavailable. The commit message is: "Removed wedbio.com - seems like site isn't available."
Could you please review my contribution and let me know if it's valid? Also, if approved, I'd appreciate being added to the list of contributors.

Thank you @AJWorkData for the update.

It has been a rewarding experience to contribute to the project and see my work making a positive impact.
Considering my contributions, I would like to kindly request your consideration for inclusion in the project's contributors list. It would be an honor to have my name listed among the talented individuals who have contributed to the project's success.


execinfo is a separate library on alpine

Hi,
Quick tip.  Your Dockerfile says (line 71):
# NOTE execinfo.h not available on musl -> lldb and compiler-rt:fuzzer/sanitizer/profiler cannot be built!

It is available, it's just implemented in a separate library.  apk install libexecinfo-dev to get it.
https://pkgs.alpinelinux.org/package/v3.15/main/x86_64/libexecinfo-dev
Hope this helps.

👍 Thanks for your information. I will try it as soon as possible.


⚠ Bofrost.at has degraded performance

In b70dbe6, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1270 ms

Resolved: Bofrost.at performance has improved in c3d3499 after 21 minutes.


UF-5833: Implemented CRD-API for labels

Implemented CRD-API for labels
Added label structure to resource for fetching all metadatas
Added skeleton for handling labels in service component

Acted on PR-comments and also added validations of unique sibling names and equal sibling classification values


update with usage examples

Some usage examples for the kotlin newbi user (like me), also note the BC jar version dependency

Merged. Thanks!


A valid Mapbox access token is required

Am getting an error.
"A valid Mapbox access token is required to use Mapbox GL JS. To create an account or a new access token, visit..."
Am using selfhosted vector tiles generated by Openmaptiles.
I have tried the following code:
<VectorTileLayer       styleUrl="https://mymap.com/osm-liberty/style.json?key=no-token"     />

Maps loads in a second and then disappears

@ted-piotrowski I tested and it work perfectly. Thanks once more.


News items have no age based cut-off

News items will be shown right back to network inception. This may not be sustainable or desirable. A max age would sort this but might result in an empty ticker so should be combined with a min number of items. Remove oldest items as long as they're older than max_age and leave at least min_items other news items.

Closed by https://github.com/vegaprotocol/ticker-service/commit/3e09242fd9850cc407e071fe38e68d1d92699b64


Add axis labels to network view

Hmm, this looks wrong: https://demo.myria.cs.washington.edu/profile?queryId=64839
Check out F2 -> F1


Win app driver not working in Selenium Webdriver Typescript

Trying to use Win App Driver with Typescript to automate a Windows Desktop App. And when I run the below code in any test framework I get a 400 error  in win app driver and I see a error message browser is mandatory eventhough am testing a desktop app.
And when I give a some value for browserName it still gives 400 error in win app driver with error message either app or appLevelTopWindow should be given
app is Root so I can iterate through all available apps. The same approach works with C# Webdriver and with C# as coding language but not working with Typescript with Selenium Webdriver and Selenium Appium
const capabilities = {
  platformName: 'Windows',
  deviceName: 'WindowsPC',
  app: 'Root'
};

await driver.startWithCapabilities(capabilities);

Below are libraries used in node js
Selenium-webdriver: 4.18.1
Selenium-appium: 4.1.22

Selenium 4 will not work, you need to downgrade to selenium 3

After downgrading to Selenium "selenium-webdriver": "3.6.0" and "selenium-appium": "1.0.2" But its very slow in finding new windows which appear in app. But while using .net C# it is very fast in find the same windows with same selectors.
Is there a reason why Windows App Driver is quite slow (over 2 minutes) in finding new windows  with Typescript but faster with C# .net Webdriver? C# .Net webdriver version am using is Selenium.WebDriver - 3.141.0 and Appium.WebDriver - 4.4.5

@anunay1 Is Selenium 4 totally out of support? I am writing a Python script to automate a Win app that uses MS WebView2, and i want to automate that webview using Selenium 4's Attach to Session feature, similar to this issue

Yes it is unfortunately.

And python language binding has other issues like action classes does not work properly. You might face issues like right click and all.

@anunay1 Yep, I realised problams in locating elements using the Selenium webdrivers when I upgrade to Selenium 4. For example,
    time_and_language_menu_bar = settings_driver.find_element_by_xpath(config_file["xpaths"]["TimeAndLanguageMenuBar"])   #returns a dictionary instead of the WebElement
    print("time_and_language_menu_bar: ", time_and_language_menu_bar)
    time_and_language_menu_bar.click()

Are there any ways to automate the WebView2 using Selenium 3?

I don't know what your compulsion is with python is? Is it your project call? But moving to C# will be a good option.

@anunay1 hi, my project is able to switch to C#, but I would still have to use selenium 3 as winappdriver does not support selenium 4. Will automating Microsoft webview be a problem with selenium 3 (I am referring to this link https://learn.microsoft.com/en-us/microsoft-edge/webview2/how-to/webdriver#attaching-microsoft-edge-webdriver-to-the-launched-webview2-app)

I don't think so.

@anunay1 Hi, Selenium 3 does not actually support webview2 according to  https://github.com/MicrosoftEdge/WebView2Feedback/discussions/3892#discussioncomment-7244130 . Will WinAppDriver work with Selenium 4 in C# Nunit?

I don't know I have not tried.


Won't load

Failed to load mods: Error while running setup for entity prototype "infinity-chest-active-provider" (infinity-container): next_upgrade target (logistic-chest-active-provider-2) must have the same bounding box.

All I can tell you is that by disabling Bob's logistics mod will allow it to load.

Seems to be because bob's mods add more tiers of logistic chest and the upgrade planner functionality of the basic logistic chests gets copied to the infinity chests. Setting chest.next_upgrade = nil in the prototype definition for the logistic chests should solve the problem.

Hmm, this doesn't happen to me. I'll implement zero's suggestion and you'll have to let me know if it works.

v0.2.2 is out, does it work now?

I'm out at dinner right now but I'll check when I get back.

Yes it worked, thank you!

No problem!


Update Gradle to 8.5

Update Gradle Wrapper to 8.5 and plugins to latest versions.
Apply the common plugins and remove now redundant configurations.
Update GitHub Actions in workflows.
Address warnings with Java 21.
Apply Spotless formatting changes to .properties files.

Just noticed that the build is failing.


url in tree page + hack for crawler

Hi @gruppin, once CI unittest failed on crawler test- can you please take a look?
@florpor

?
@GalGend did everything turn out ok?

i rebased the branch over master and the tests passed.
On Wed, 30 Jun 2021 at 11:10, florpor @.***> wrote:

?
@GalGend https://github.com/GalGend did everything turn out ok?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/meirim-org/meirim/pull/390#issuecomment-871190104,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEMETVXVU63JIC6NQU6GB7LTVLGPLANCNFSM47EIO6JA
.


add flxw to the org and to helm charts repo

Summary

add flxw to the org and to helm charts repo
@flxw will help in the helm chart repo

sponsor:

@sabre1041

I think it's great to add more folks to the helm effort within sigstore, but this change appears to increase permissions from nothing
straight to admin. Is there more context here (in light of our maintainer ladder) that I'm missing, or should we be fixing the permissions on the helm group first?

oh! i can fix that permission, that should be just a write permission and not admin

@bobcallaway done the change and Felix did several contributions to the repo and will be a good addition to help us

@sigstore/core-team please weigh in, will merge via lazy-consensus in 5 days unless there is discussion to the contrary


Link to Documentation is landing on a 404 Error page

Your link to the newest documentation "https://docs-beta.luxonis.com/develop/ros/depthai-ros-intro/" is landing on a 404 Error page.

Hi, thanks for notifying, it seems things moved around a little bit lately.


Diversity report shows data for each exercise stage

Is your feature request related to a problem? Please describe.
I need to be able to see diversity information at each stage of exercise processing.
Describe the solution you'd like
The current dashboard but it shows the numbers at each exercise stage.
Describe alternatives you've considered
Some kind of export but this is more efficient.
Additional context

A few refinements to the Diversity report:

[ ] Include 'Last refreshed' date near the Refresh button
[ ] Include 'Summary' tab containing text 'Summary coming soon' as a placeholder for future comparison/summary data
[ ] Include 'Export data' button on each tab
[ ] Include 'Export all data' at the top of the report


Relocate RPCErr to ripple/protocol - fix previous PR

High Level Overview of Change
Adds STCurrency.h to libxrpl for Clio
Fixes PR #4885 botched by force push

Context of Change
Force push in updating PR #4885 undid intended changes.
Latest Clio build requires STCurrency.h in libxrpl package.

API Impact

[ ] Public API: New feature (new methods and/or new fields)
[ ] Public API: Breaking change (in general, breaking changes should only impact the next api_version)
[x] libxrpl change (any change that may affect libxrpl or dependents of libxrpl)
[ ] Peer protocol change (must be backward compatible or bump the peer protocol version)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 61.65%. Comparing base (22b7518) to head (b39f06f).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #4940   +/-   ##
========================================
  Coverage    61.65%   61.65%
========================================
  Files          806      806
  Lines        70967    70967
  Branches     36690    36690
========================================
+ Hits         43752    43753    +1
- Misses       19856    19857    +1
+ Partials      7359     7357    -2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

2. beast/unit_test/main.cpp

@ximinez This is one thing I was unsure about whether it actually belonged or not.
I think @thejohnfreeman has the intention to export all these via globs so that none are forgotten about anymore but that would entail the audit Ed refers to.
Should I just close this PR or is that audit/export on too hazy of a timeline?

This is one thing I was unsure about whether it actually belonged or not. What would be the utility of providing this or not whether installed to a system or provided in a Conan package with libxrpl?

I can't think of much utility. IIRC, if one were to build and run beast/unit_test/main.cpp, it would only be able run the unit tests that are part of beast. Those are all already included in rippled AFAIK. It might be useful as sample / example code, but that can just as easily be obtained from Github.

Should I just close this PR or is that audit/export on too hazy of a timeline?

I vote to keep this PR open. It's a smaller subset of changes that can be reviewed and merged quickly, and I believe unblocks something that @thejohnfreeman is working on.

@ximinez we're already installing some headers in impl directories. I would like to hold off on moving them around. My plan is to remove all impl directories in the PR that separates libxrpl and xrpld sources.


Optimize audio node rendering

This improves node rendering performance by reusing PIXI objects that represent how they are rendered in WebGL.
There are two types of objects cached with this change:

PIXI.Text are cached as textures that are reused as PIXI.Sprites
PIXI.Graphics representing the node's rectangle and unconnected ports create a PIXI.GraphicsGeometry that can be reused by other PIXI.Graphics objects

Here are the types of work this change reduces.

PIXI.Text renders text with a 2d canvas. This is now only done once per unique text displayed in the graph.
PIXI.Graphics computes a mesh based on calls to its drawing methods. This is now only done once per unique node type in the graph.
PIXI.Text and PIXI.Graphics upload the rendered texture and mesh to webgl. Reusing the texture through PIXI.Sprite and mesh through PIXI.GraphicsGeometry means this is now done once.
Ports that are connected to another node are represented as their PIXI object and don't require the background graphics to be recomputed, instead they are rendered on top of the unfilled ports rendered in the background.
AudioGraphRender determines when nodes and edges are not in the given graph context and should be removed from its presentation. This is changed to reduce memory allocation (by creating strings) and reduce time spent doing this comparison.

@mzgoddard Do you want me to merge this to main? I see a series of PRs are waiting in the list, but wasn't sure about your plan.

@mzgoddard Do you want me to merge this to main? I see a series of PRs are waiting in the list, but wasn't sure about your plan.

Un. I'll go ahead and merge this and merge other PRs bottom up in the list that you've approved.


Parallel pull from V2 registry may lead to corruption of shared ancestor images

Description
Pulling images with common ancestors from a V2 private registry seems to have some probability of corrupting the images, and the issue seems to stem from the client side. Serializing download of all images made the problem go away, but is not a good user solution because (A) it cannot be done correctly outside of the docker daemon and (B) it slows downloads which could otherwise be done in parallel.
Due to the severity of this issue, we have been unable to use the docker registry in production, for fear of corrupted images leading to hard-to-trace failures.
Bug report
List the steps to reproduce the issue:

Execute many "docker pull" requests in parallel against images in a V2 private registry which share several common ancestors.
docker run all of them after downloads have been completed.

Describe the results you received:
Nondeterministically, this produces corrupt images. If ubuntu is corrupted, for instance, we may receive (during build time of subsequent images):
System error: exec: "/bin/bash": stat /bin/bash: no such file or directory
In other cases, we've seen aufs mount failed exceptions while running images which had been fully downloaded and built without error, but unfortunately I do not have one of these errors recorded.
Describe the results you expected:
Pulling and building should deterministically work with parallel operations.
Provide additional info you think is important:
We worked around the issue on our test machines and development environment by serializing all "docker pull" operations behind a global lock. This caused both the /bin/bash error and the aufs mount failed error to go away, and the build returned to relative stability.
docker version:
1.6.2
docker info:
Containers: 88
Images: 1715
Storage Driver: aufs
Root Dir: /var/lib/docker/aufs
Backing Filesystem: extfs
Dirs: 1891
Dirperm1 Supported: false
Execution Driver: native-0.2
Kernel Version: 3.11.0-26-generic
Operating System: Ubuntu 13.10
CPUs: 4
Total Memory: 15.38 GiB
WARNING: No swap limit support
uname -a:
Linux kitties 3.11.0-26-generic #45-Ubuntu SMP Tue Jul 15 04:02:06 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
Provide additional environment details (AWS, VirtualBox, physical, etc.):
Reproducible on AWS, boot2docker on Mac, and Linux laptop.

This should be covered by #14193.

Closing as fixed, for now. If 1.7.1 rc still sees issue, please re-open.

Hi there,
I think I am still seeing this issue:
Building images on docker 1.8.2, build 0a8c2e3
Pushing to registry:2
Pulling images on 1.7.1, build 2c2c52b-dirty (coreos 766.3.0)
It looks like I am missing the layer where I installed java because some containers complains about it. If I run the container directly on the machine that build it (without pulling from registry:2) everything works fine. registry v1 works fine too.

Hi,
same problem here, pulled with 1.7.1 (RH) - so far it is okay if this is fixed in later releases, but after upgrading to 1.8.2 (RH) the problem did still not "go away" or got automatically fixed, we had to prune all images and pull again - would be nice if there would be an option to check/verify pulled images.

@CybotTM Please avoid commenting on closed issues.
Docker 1.10 has done a lot to address these kinds of problems. Specifically, the download manager addressed the concurrency problems leading to corruption. I'd recommend trying it out.


Assertion failed in resize.cpp

Hey guys, I'm having an error while trying to run botty, I've filled out param.ini and pressed f9 to auto set the d2 settings, but then I press f10 or f11 and I'm getting this assertion error:

Do you know what I am missing?
PS: I'm using the main.exe compiled version

looks like a access rights issue. So the path to the template exists but still opencv could not load that image.

FYI it's working well by running the python code directly, so as far as I'm concerned you can close the ticket, cheers


Add strict route prop for path-to-regex option

A fix for #4170
Can now add a strict boolean prop to Route (if the strict prop is omitted it defaults to true):
// this will match both /foo and /foo/
<Route exact path="/foo" strict={false} component={foo} />

I ran the tests and it passes all of them, and in my limited testing it works both inside and outside of <Switch />

Thanks, @rafrex :) Right now our automated tests on v4 are not very complete, unfortunately. Would you mind booting the website and clicking through the examples to make sure everything still works?

I clicked through all the examples and they seem to all work.
Note that I locally changed the default for strict to false in the router, and then booted the site so I  wouldn't have to add strict={false} to every route in the examples. I also tested a few routes by adding the strict prop as well.

Curious, how come strict is enabled by default?
Not matching trailing slashes seems like the edge case here, which would be in line with the default behavior of path-to-regex (which mimics express' behavior).

Sounds good, @rafrex. Can we get a few tests around this too? Ideally we'd have 2 tests: one for strict = false and another for strict = true.

@simenbrekken I left the default as per how it worked previously. I agree it might make sense to change it to false.

Just a nitpicky 2 cents: the fact that the internal implementation (the path-to-regexp) uses an argument called "strict" is kind of an implementation detail, and that terminology isn't super intuitive for a user of the router. (like, wtf is "strict...oh, its a param for an internal dependency!".
if that gets exposed as a public api, it probably needs a better name.

I actually don't mind the strict prop name because RRv4 doc says refer to path-to-regex to figure out how path="..." will match. But also don't have a strong preference for it either.
Maybe the default could be false and the prop could be matchTrailingSlash or something.
@mjackson what do think the default for strict should be? And what do you think the prop should be called? (I think both of these are high level api decisions).

I think strict is a fine name and we should probably use the same default as path-to-regexp does. I'm sure those folks have good reasons by now for choosing the defaults they have.

Change the default for strict to false.
I don't have experience with karma/mocha, someone with more experience want to write the tests?

All you have to do is copy/paste some of the existing tests. Look at integration-test.js for hints.

Also, we need to make sure we have tests for cases like https://github.com/ReactTraining/react-router/issues/4130 where two routes have similar URLs. @SimeonC created https://github.com/ReactTraining/react-router/pull/4133 with some tests that we may be able to use here (with attribution, ofc).

I'll just reply here - I can't update/create PRs currently as I'm without a PC for a month or two while moving so feel free to repurpose any of my PRs as needed.

@mjackson I added tests for the strict prop with trailing slash.
I also added tests for similar paths (/some and /some-path). The problem in #4130 was with active links, which are no longer part of v4 (as far as I can tell), so I tested it using children and making sure the match boolean is correct (which seems to be how one would do active links now).
<Route path="/some-path" children={({ match }) => (
  <h1>{match ? TEXT2 : TEXT1}</h1>
)}/>

Thanks, @rafrex! Looks like there are some conflicts here, probably because I moved some stuff around when organizing the monorepo last week. Would you please rebase?
Also, sorry for all the churn. I'm trying to get things to settle down as quickly as I can.

@mjackson I rebased, should be all set!

Thanks, @rafrex. I'll take it from here ;)


TerraformCLI does not support 'fmt' command as documented

Describe the bug
TerraformCLI task does not support 'fmt' command as documented. I manually installed the extension for Azure DevOps server but can confirm on the Microsoft hosted version also does not have the 'fmt' option either.
To Reproduce
Steps to reproduce the behavior:

Setup pipeline as YAML
Add a Task (via the assistant)
Select TerraformCLI
'fmt' is not in the list (also manually writing the task in YAML, fmt is highlighted as an unsupported value)

Expected behavior
To be able to use the 'fmt' command to lint Terraform configuration files.
Screenshots
If applicable, add screenshots to help explain your problem.
Pipeline Logs
Include logs that help demonstrate the problem. Please make sure to redact any sensitive info such as secrets.
Agent Configuration

OS: [e.g. ubuntu debian]
Hosted/Self Hosted
Terraform version used (Default for hosted agent is acceptable)
AzureCLI version used (Default for hosted agent is acceptable or N/A)

Additional context
Add any other context about the problem here.

@someoneelsescloud thanks for reporting this. It looks like all that needs to be modified is the task.json file such that "fmt":"fmt" is added to the list of options. All the rest of the code is in place to support this command. If you or someone else would be interested in submitting a PR for this, I can help get it merged in so it can be used as the docs indicate.

@charleszipp done https://github.com/charleszipp/azure-pipelines-tasks-terraform/pull/66

merged with #66. closing. Thanks @someoneelsescloud for the contribution!


[CI][FLINK] Flink update next version in flink/examples

Problem
gradle.properties in flink/examples/stateful is not being updated with the next version when new-version.sh runs.
Solution
This PR would update the version there with the next version.

Note: All schema changes require discussion. Please link the issue for context.

[ ] Your change modifies the core OpenLineage model
[ ] Your change modifies one or more OpenLineage facets

If you're contributing a new integration, please specify the scope of the integration and how/where it has been tested (e.g., Apache Spark integration supports S3 and GCS filesystem operations, tested with AWS EMR).
Checklist

[x] You've signed-off your work
[x] Your pull request title follows our guidelines
[ ] Your changes are accompanied by tests (if relevant)
[x] Your change contains a small diff and is self-contained
[ ] You've updated any relevant documentation (if relevant)
[ ] You've updated the CHANGELOG.md with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary)
[ ] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)
[ ] You've added a header to source files (if relevant)

@merobi-hub can you manually bump those versions in Flink for now too?

@mobuchowski yes, no problem

Thanks @merobi-hub!


don't ignore hidden files in pages/

I'd like to generate .htaccess files using templates.
Unfortunately this line:
https://github.com/koenbok/Cactus/blob/master/cactus/utils/filesystem.py#L23
ignores all files that start with a period so I'm not able to.
Is this check guarding against anything in particular?

Does this this issue lend any help?

@moomoohk -- thanks but I don't think so. This issue describes whether or not cactus will treat a file in pages/ as a page to be compiled while as I understand the issue you linked -- cactus deletes the .build folder on every re-build so users can't maintain .build as a git repository.

Oh yes, I got the folders confused...

Hey @rryan
Just a quick note that I've been very busy with work as of late, but that I hope to take a look at this when I have more free time, most likely in a week or so. Sorry about the silence here.
Cheers,

@krallin -- no problem at all! For now I'm simply symlinking .htaccess to htaccess but it'd be nice if I could avoid that.


blue background and white vertical lines after executing nvim on centos7

when i execute nvim --clean blue background and white vertical lines disappear. a problem may be old centos7 but could anybody confirm that, please? nvim has been compiled based on latest sources from github using gcc-12.1.0 and python-3.6 is used.
you can see a screen-shot here
https://www.dropbox.com/s/63b0l8bpeainhkh/Screenshot from 2022-12-19 13-09-50.png?dl=0

What terminal are you using? It must not support true colors would be my guess.

$ nvim --debug
nvim: Unknown option argument: "--debug"
More info with "nvim -h"

i'm using defaut centos7 terminal ad it should support true colors because my vim doesn't have any problem...
$ echo $TERM
xterm-256color

and if i use it in the screen it uses settings below:
# 256 colors
attrcolor b ".I"
termcapinfo xterm* 'is=\E[r\E[m\E[2J\E[H\E[?7h\E[?1;4;6l'
termcapinfo xterm 'Co#256:AB=\E[48;5;%dm:AF=\E[38;5;%dm'
term screen-256color
defbce "on"

well if you're using screen, then it's not actually xterm, so you probably want to change your $TERM value. There are quite a few issues and mentions of this in neovim/neovim: https://github.com/neovim/neovim/wiki/FAQ#nvim-shows-weird-symbols-2-q-when-changing-modes
I'll close this cause I don't think we'll be able to fix this -- feel free to use a different color scheme and remove termguicolors from the options.


Implement: package page - package data area - cross linking

The API references generated from the jsii assembly should support creating links between different packages.

This is already supported


[Feature] one and only dtd file

Reported by andosa on 2004-05-19 14:15 UTC
Make the program use just one dtd file, not a sperate
one for each xml

Updated by pgrigorenko on 2007-10-16 22:24 UTC

status: open --> closed


operator sampler

Make sure that the built equations are valid

Already done in methods, e.g. node replacement.


Integrate DETM

This is a work in progress PR I am doing about the integration of DETM.
This will hopeful fixes #1

@silviatti I managed to make some progress on this ... I think you can start having a look at it and let me know if I am in the right way.
I have few questions on the data representation :

I have hacked the DETM model and make it work with the data as TF-IDF matrix and the timestamps as an array, do you think it is better to use this as the input of the model or keep using the bag of word version in the original version of the DETM model.?

Hi, sorry if I didn't reply to you earlier. I'm going to review your PR next week.
About your question, I think using tf-idf would work fine as well. I've seen this already in another implementation (but I don't remember where). Maybe we can make it a parameter so that a user may choose between using counts or tf-idf.
Do you have other questions? :)


Add execute context that can handle promises

Description
POC for a custom graphql-core ExecutionContext class that handles Promises (from https://github.com/syrusakbary/promise) so that you can use a sync dataloader. Couple of notes:

This is a just a rough exploration of a potential solution. It needs a lot more testing and I've probably missed some things.
There is no easy way to extend some of the ExecutionContext methods so I had to duplicate them and add custom logic (I also got rid of the code paths that handle async code). This will make maintenance a bit of a pain if those code paths change
Regardless of whether we end up going down this route I think expanding the execute and execute_sync methods to allow passing of the execution_context_class would be beneficial

TODO

[ ] Use https://github.com/celery/vine instead of promise (+ add optional dependency)
[ ] Test lists
[ ] Docs

Types of Changes

[ ] Core
[ ] Bugfix
[ ] New feature
[x] Enhancement/optimization
[ ] Documentation

Issues Fixed or Closed by This PR

Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

@patrick91 @BryceBeagle this is ready for review now

@patrick91 @BryceBeagle this is ready for review now

Hi, thanks for contributing to Strawberry 🍓!
We noticed that this PR is missing a RELEASE.md file. We use that to automatically do releases here on GitHub and, most importantly, to PyPI!
So as soon as this PR is merged, a release will be made 🚀.
Here's an example of RELEASE.md:
Release type: patch

Description of the changes, ideally with some examples, if adding a new feature.

Release type can be one of patch, minor or major. We use semver,so make sure to pick the appropriate type. If in doubt feel free to ask :)

Apologies for the delay with this PR. Life events have meant I haven't had much time lately 👶
I think the last thing that needs doing is to add support for lists.

Apologies for the delay with this PR. Life events have meant I haven't had much time lately 👶
I think the last thing that needs doing is to add support for lists.

Awesome! Thanks for the update 😊 I was looking at fixing the tests but I got distracted by other things 😊


fatal error: X11/extensions/XTest.h: No such file or directory

Getting error in build.
Current Behavior
11:58:07     toggleKeyCode(key, down, flags);
11:58:07                                   ^
11:58:07   CC(target) Release/obj.target/robotjs/src/deadbeef_rand.o
11:58:07   CC(target) Release/obj.target/robotjs/src/mouse.o
11:58:07 ../src/mouse.c:12:35: fatal error: X11/extensions/XTest.h: No such file or directory
11:58:07   #include <X11/extensions/XTest.h>
11:58:07                                    ^
11:58:07 compilation terminated.
11:58:07 robotjs.target.mk:117: recipe for target 'Release/obj.target/robotjs/src/mouse.o' failed
I have tried to run sudo apt-get install libxtst-dev libpng++-dev. But got apt-get command not found error.
Need your help to resolve the issue.

Can someone help me to resolve this issue please

But got apt-get command not found error.

Which Linux distribution are you running?

This issue has been closed because it has been inactive for > 180 days.
You may reopen this issue if it has been closed in error.

But got apt-get command not found error.

Which Linux distribution are you running?

Hey. I am having this issue when I run npm i robotjs.
My Linux distro is Ubuntu 20.04 LTS.


Update version - v1.1.17

Signed-off-by: MoonkiHong<EMAIL_ADDRESS>Update list

Update dependencies (PR#606)
[ScoringMgr] Amendment of scoring API (PR#615)
Update golang ver. - v1.19 (PR#611)

Still failed. Any reason for this?


Dreame bot X10

Describe the bug
do not Room entities for customized cleaning
Additional Information (please complete the following information)

Model Name [e.g. dreame.vacuum.r2235]:
Firmware Version [e.g. 1156]: 4.3.3_1215
Home Assistant Version:  2023.1.2
Configuration Type [With or without map support]: with map
Errors or warnings shown in the HA logs (if applicable): do not Room entities for customized cleaning
https://github.com/Tasshack/dreame-vacuum/blob/ee71c8c608fa4cca9bc85629dcc86e2d5b83b891/docs/room_entities.md#rooms-card


Problem Documenting Required Properties

Hello!!
I am trying to create more documentation in some APIs,
I want to change the optional field to a required field.

But when I add the DataAnnotations [Required] in a property, swagger returns me a 500 error.

I read from a lot of documentation and this is supposed to work, but it'snot working.

Try navigating to the Swagger JSON page directly - if you’re using the default route config for Swashbuckle I think you’ll find this at at /v2/swagger/v1/swagger.json. If something is going wrong with the Swagger generation the full stack trace should be visible there

There is not a swagger.json file.

Navigate to the url in the white box at the top of the swagger page - that’s the swagger.json page

By “navigate”, I mean type it into your browser address bar

Sorry. I am new with swagger and I thought you were referring to the swagger browser.
This is the error:

I already checked that issue and added the Microsoft.Web.Infrastrure package and right now a have this error:
message="Could not load file or assembly 'System.ComponentModel.Annotations, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified." source="mscorlib"
I added the System.ComponentModel.Annotations library but I have the same error.

I changed the System.ComponentModel.Annotations version from 4.7.0 to 4.4.1 and this fixed my problem.
Thanks for your help.


[k3d] Missing TLS SAN for initializing server node

Environmental Info:
K3s Version:

Docker Image rancher/k3s:v1.19.2-k3s1

Cluster Configuration:

3 servers

Describe the bug:
When creating a multi-server k3s cluster with k3d (v3.1.2), the secondary server node (server-1) fails to join the cluster of the primary server node (server-0) with the following logs:
$ docker logs -f k3d-test-server-1
time="2020-10-08T09:01:48.098912215Z" level=info msg="Starting k3s v1.19.2+k3s1 (d38505b1)"
time="2020-10-08T09:01:48.124151595Z" level=fatal msg="starting kubernetes: preparing server: server https://k3d-test-server-0:6443/cacerts is not trusted: Get \"https://k3d-test-server-0:6443/cacerts\": x509: certificate is valid for kubernetes, kubernetes.default, kubernetes.default.svc.cluster.local, localhost, not k3d-test-server-0"

Steps To Reproduce:

Install k3d v3.1.2: e.g. wget -q -O - https://raw.githubusercontent.com/rancher/k3d/main/install.sh | bash (Follow https://github.com/rancher/k3d/#get)
Try to create a multi-server cluster with k3s v1.19.2: k3d --trace cluster create test -s 2 --k3s-server-arg="--debug" --k3s-agent-arg="--debug" --no-rollback --image rancher/k3s:v1.19.2-k3s1

Expected behavior:
Multi-Server Cluster should be created without the secondary master node crashing.
Actual behavior:
The secondary master node crashes with the logs pasted above.
Additional context / logs:

This can be "fixed" in k3d by adding --tls-san <server name> to the k3s server command in the initializing server node

From Slack discussions:

potentially, the issue has its origin in the Go HTTP Client that doesn't do SNI by default

Server would usually add the requested hostname to the certificate when the client makes a request with the server name being in the SNI Hello message
the issue could've been introduced by the Go version update
Related issue on Go: https://github.com/golang/go/issues/22704#issuecomment-344137683

Related k3d issue: https://github.com/rancher/k3d/issues/365
cc @brandond & @erikwilson

Following @brandond 's investigation, k3d is just trying to add additional server nodes too quickly, before the API Server is even running. This could already be fixed on k3d's side by waiting a little longer until the initializing server node logs k3s is up and running indicating, that all components are ready.

I added some additional tracing and confirmed that the SNI ServerName is being set by the joining client. The issue is that the k3s-serving cert is created at startup with a preconfigured list of SANs, including those requested via the --tls-san flag. However, additional SAN entries to match the system hostname cannot be added to the certificate until after the datastore and apiserver are available. k3d was waiting for a 'Running kubelet' message as an indication that k3s was done starting and was ready to have additional nods join, when in reality the "k3s is up and running" message is probably the one to look for.


isl_0_20: do not guess -march=/-mtune=

See #106587
Strictly speaking, this change would also make sense for previous versions of isl.

Motivation for this change
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-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.

/rebase staging


Updates the HA doc, adding HPA and PDB sections

Adding the horizontal pod autoscaler and pod disruption budget sections to the HA doc
Signed-off-by: Shalier Xia<EMAIL_ADDRESS>

PRing this into main is great, and also if it's intended to be in the release branch for v0.9.0, please add it to https://github.com/openservicemesh/osm-docs/tree/release-v0.9


Anyone tried to create an expo dev client against an ignited app? I'm getting this error....

I ignited an app recently using the expo option. Then I upgraded to expo 43. Now I am attempting to build a dev client so i can include some custom native code:
https://docs.expo.dev/development/getting-started/#creating-and-installing-your-first-development-build
I am running the build command:
eas build --profile development --platform android
And getting the following error:
✖ Config sync failed
[stderr] [04:27:32] [android.dangerous]: withAndroidDangerousBaseMod: Could not add expo-dev-client dependencies to existing file /root/workingdir/build/react-native.config.js. See expo-dev-client installation instructions to add them manually: https://docs.expo.dev/clients/installation/
[stderr] [04:27:32] Error: [android.dangerous]: withAndroidDangerousBaseMod: Could not add expo-dev-client dependencies to existing file /root/workingdir/build/react-native.config.js. See expo-dev-client installation instructions to add them manually: https://docs.expo.dev/clients/installation/
[stderr]     at addReactNativeConfigAsync (/root/workingdir/build/node_modules/expo-dev-client/plugin/build/withDevClient.js:35:19)
[stderr]     at action (/root/workingdir/build/node_modules/@expo/config-plugins/src/plugins/withMod.ts:227:29)
[stderr]     at interceptingMod (/root/workingdir/build/node_modules/@expo/config-plugins/src/plugins/withMod.ts:108:27)
[stderr]     at action (/build/node_modules/@expo/config-plugins/src/plugins/withMod.ts:228:14)
[stderr]     at interceptingMod (/build/node_modules/@expo/config-plugins/src/plugins/withMod.ts:108:21)
[stderr]     at interceptingMod (/build/node_modules/@expo/config-plugins/src/plugins/withMod.ts:108:21)
[stderr]     at interceptingMod (/build/node_modules/@expo/config-plugins/src/plugins/withMod.ts:108:21)
[stderr]     at action (/build/node_modules/@expo/config-plugins/src/plugins/createBaseMod.ts:82:21)
[stderr]     at interceptingMod (/build/node_modules/@expo/config-plugins/src/plugins/withMod.ts:108:21)
[stderr]     at evalModsAsync (/build/node_modules/@expo/config-plugins/src/plugins/mod-compiler.ts:172:25)
[stderr]     at compileModsAsync (/build/node_modules/@expo/config-plugins/src/plugins/mod-compiler.ts:82:10)
[stderr]     at configureManagedProjectAsync (/build/node_modules/expo-cli/src/commands/eject/configureProjectAsync.ts:64:12)
[stderr]     at prebuildAsync (/build/node_modules/expo-cli/src/commands/eject/prebuildAsync.ts:90:21)
[stderr]     at actionAsync (/build/node_modules/expo-cli/src/commands/prebuildAsync.ts:37:3)
node exited with non-zero code: 1

---

`npx ignite-cli doctor` results:
System
  platform           win32
  arch               x64
  cpu                8 cores       Intel(R) Core(TM) i7 CPU         950  @ 3.07GHz
  directory          PaymentsApp   C:\Users\razor\projects\PaymentsApp

JavaScript (and globally-installed packages)
  node               16.13.0      C:\Program Files\nodejs\node.EXE
  npm                8.1.0        C:\Program Files\nodejs\npm.CMD
    eas-cli          0.38.2
    expo-cli         4.13.0
    prettier         2.4.1
    yarn             1.22.17
  yarn               1.22.17      C:\Users\razor\AppData\Roaming\npm\yarn.CMD
    eas-cli          0.38.2
    expo-cli         4.13.0

Ignite
  ignite-cli         7.8.0        C:\Users\razor\AppData\Local\npm-cache\_npx\e31027f3785124a8\node_modules\.bin\ignite.CMD
  ignite src         build        C:\Users\razor\AppData\Local\npm-cache\_npx\e31027f3785124a8\node_modules\ignite-cli\build

Android
  java               17.0.1       C:\jdk-17.0.1\bin\java.EXE
  android home       -            undefined

Hey, I just gave this a try. After a bit of trial and error, turns out you really need<EMAIL_ADDRESS>at least.

expo-dev-client will add react-native.config.js, but if you have that already, then it can't. Here is how I added it.
module.exports = {
dependencies: {
...require('expo-dev-client/dependencies'), // to remove
// ...
},
};

@razor2022 have you tried updating to Expo 46 and making a custom dev client according to the Expo docs?
I just did it yesterday successfully, let me know if you have tried the new version.
Expo 46 will be baked into the next version of ignite, but for now you can upgrade like @bendoerr mentions in step 3.

I'm going to close this issue due to inactivity. However, if this problem still persists and you have updated to the latest version of ignite, please open a new issue stating the problem and the output of npx ignite-cli doctor.
Also, you can check out https://community.infinite.red to get help with us (and a bunch of other RN devs!) directly in Slack.


SearchProvider API for file search broken

VSCode Version: 1.24.0-insiders
OS Version: Windows 10

I had implemented SearchProvider from vscode.proposed.d.ts to search for file using Ctrl+P. I have seen that the API changed removing query from provideFileSearchResults and it no more calls this method on Ctrl+P. Has the functionality changed?

Have discussed over email


New adapters for 2.0

The goal is the move the adapters except Campfire and Shell out of the core of Hubot into their own npm packages maintained by the origin contributer, so we can keep the dependencies in Hubot to a minimum. This branch is the start of the changes.
I've updated the Campfire, IRC and shell adapters for the new core. The next part of the changes is to get the tests working for them. So I was hoping @technoweenie could glance over and point me in the right direction due to the changes with Robot and Robot.Adapter.

If you're happy to rock without working tests then :metal: Hoping to work on the requiring third-party adapters at some point so we can get the deps out.


UR3 robot in Unity3D did not change joint states as the URSim robot.

Hello, I've successfully connect the Unity3D with URSim with the IP address and it's possible to utilize the "Joystick Panel" to control URSim robot remotely. But the robot in Unity3D did not change its condition to be the same as that in URSim.
Then when I close the Unity running mode, I obtained the following issue:
ObjectDisposedException: Cannot access a disposed object.
Object name: 'System.Net.Sockets.NetworkStream'.
System.Net.Sockets.NetworkStream.Read (System.Byte[] buffer, System.Int32 offset, System.Int32 size) (at <0463b2ef957545c0a51b42f372cd4fbb>:0)
ur_data_processing+UR_Stream.UR_Stream_Thread () (at Assets/Scripts/UR3/ur_data_processing.cs:234)
System.Threading.ThreadHelper.ThreadStart_Context (System.Object state) (at <695d1cc93cca45069c528c15c9fdd749>:0)
System.Threading.ExecutionContext.RunInternal (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, System.Boolean preserveSyncCtx) (at <695d1cc93cca45069c528c15c9fdd749>:0)
System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, System.Boolean preserveSyncCtx) (at <695d1cc93cca45069c528c15c9fdd749>:0)
System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state) (at <695d1cc93cca45069c528c15c9fdd749>:0)
System.Threading.ThreadHelper.ThreadStart () (at <695d1cc93cca45069c528c15c9fdd749>:0)
UnityEngine.<>c:<RegisterUECatcher>b__0_0(Object, UnhandledExceptionEventArgs)

How can I fix the problem, thank u very much.

Hi Roman Parak,
I encountered the same problem as mentioned by 2000222. And my situation is exactly the same.

I can successfully connect Unity3D with URsim with the IP.
After IP is connected, the robotic arm in Unity3d is not successfully initialized.
UR3 robotic arm in the simulator can be controlled through Joystick Panel in the Unity3D. But a Unity3D-to-URsim control fails.
After exiting Play mode, ObjectDisposedException error comes out. Sometimes another error SocketException occurs together.

I attached the debug info and some snapshots regarding the issue for your reference.
Error 1: ObjectDisposedException
System.Net.Sockets.NetworkStream.Write (System.Byte[] buffer, System.Int32 offset, System.Int32 size) (at <0463b2ef957545c0a51b42f372cd4fbb>:0)
ur_data_processing+UR_Control.UR_Control_Thread () (at Assets/Scripts/UR3/ur_data_processing.cs:354)
System.Threading.ThreadHelper.ThreadStart_Context (System.Object state) (at <695d1cc93cca45069c528c15c9fdd749>:0)
System.Threading.ExecutionContext.RunInternal (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, System.Boolean preserveSyncCtx) (at <695d1cc93cca45069c528c15c9fdd749>:0)
System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, System.Boolean preserveSyncCtx) (at <695d1cc93cca45069c528c15c9fdd749>:0)
System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state) (at <695d1cc93cca45069c528c15c9fdd749>:0)
System.Threading.ThreadHelper.ThreadStart () (at <695d1cc93cca45069c528c15c9fdd749>:0)
UnityEngine.<>c:<RegisterUECatcher>b__0_0(Object, UnhandledExceptionEventArgs)

Error 2: SocketException
SocketException: A blocking operation was interrupted by a call to WSACancelBlockingCall.

System.Net.Sockets.Socket.Receive (System.Byte[] buffer, System.Int32 offset, System.Int32 size, System.Net.Sockets.SocketFlags socketFlags) (at <0463b2ef957545c0a51b42f372cd4fbb>:0)
System.Net.Sockets.NetworkStream.Read (System.Byte[] buffer, System.Int32 offset, System.Int32 size) (at <0463b2ef957545c0a51b42f372cd4fbb>:0)
Rethrow as IOException: Unable to read data from the transport connection: A blocking operation was interrupted by a call to WSACancelBlockingCall.

 Any help from you will be really appreciated!

Hey, guys,
sorry for the late response, but as always, I'm busy.
I have tested the app and everything works properly (simulation only). Since I only have the CB version in my lab, I have not tested the latest UR-e version in real-world. However, I have received a lot of positive feedback that everything works without problems.
First of all, If you are using the UR-e version of the robot, you need to change lines 93 and 97 in the C# script.
Link: https://github.com/rparak/Unity3D_Robotics_UR/blob/main/Universal_Robots_Unity_App/Assets/Scripts/UR3/ur_data_processing.cs
Because the communication rate of the CB-version of the robot I used is 125 Hz (8 milliseconds) and the new one is 500 Hz (2 milliseconds). Just change the time step value from 8 to 2.
But I suppose that's not the main problem. You will probably need to enable the remote control.
Remote Control option on the UR robot controller if you have a UR-e robot (UR3e, UR5e or UR10e):
1.Select the Settings menu from the top right button.
2.Select System-Remote Control.
3.Select Enable.
4.Change the mode of operation to Remote Control
Note:
The first generation of UR robots (non UR-e) such as UR3, UR5 and UR10 already has the Remote Control option enabled by default.
I hope everything is clear, if not, contact me at:
<EMAIL_ADDRESS>Have a great day.
RP


[ADD]:  THE RAISING FIGHTING SPIRITS GAME 👾

PR Description 📜

This PR is about adding The Raising Fighting Spirits Game

Fixes #1076

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have specified the respective issue number for which I have requested the new game.

Screenshots 📸

Thank you soo much for contributing to our repository 💗

Thank you @0xabdulkhalid ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

@0xabdulkhalid Resolve the conflicts

@Arun9739, Conflicts are resolved!

@Arun9739, Conflicts are resolved!

Thank you  @0xabdulkhalid , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


add day 1 solution

pls merge or my 2d waifu will leave me

It doesn’t look like your email set when authoring the commit, so it won’t show up in you GitHub account details and your account won’t show up in the contributors section of the repo.


Old kubernetes dashboard deployment

Hi,
I noticed the current version of the default deployment for kubernetes-dashboard is v1.6.0 which is a bit old. Guess it's probably because the dashboard enforced its security and can only be accessed over HTTPS by default since v1.7?
To live with an old dashboard may be inconvenient in some cases. Although the dashboard URL can be customized, there's a bug if I choose newer version. When call dind::component-ready to detect the dashboard readiness, it uses an old label app in 1.6 which has been changed to k8s-app after then.
To have a default dashboard URL for people who don't care it is great. However, can it support new version as well in case some people like me may need it? (app vs. k8s-app)  :-)
Also, according to rawgit.com, the site will be shut down soon. There needs to be a new place to put the deployment YAML for the old one if that's needed by people. Alternatively, we could even disable dashboard deployment by default, and let people customize DASHBOARD_URL by themselves explicitly. Seeing someone was asking a similar feature while ago.
I'd be glad to contribute code if that makes sense!

Hi, I've submitted a PR for this at #294 . It also addresses #65 Feel free to comment on it...

I am gonna close it since the PR #294 has been merged! Cool...


Kleunen master mg2

Yes, this looks ok. Only coding style changes, all test pass. Indeed it has become a massive PR.

I force updated the PR.
See https://github.com/redboltz/mqtt_cpp/pull/921/commits/85cfa6f694b1905efd92bd7c7c1792ed755aed39

Looks good

Merged. Thank you for the great work !!
Could you update https://github.com/redboltz/mqtt_cpp/wiki/Authentication-and-Authorization ?
I think that you have write permission.

Excellent.
I will update the documentation


capture screenshots of the selector matches

This way, people will be able to see the matched elements without having to open the sample HTMLs in their browser.

this would require rough changes. it would be implemented later, but not now.