Language server error: TypeScript

Check for existing issues

[ ] Completed

Describe the bug / provide steps to reproduce it
Language server error: TypeScript
failed to run npm info
Caused by:
Permission denied (os error 13)
Environment
Zed: v0.81.1 (stable)
OS: macOS 13.3.0
Memory: 16 GiB
Architecture: x86_64
If applicable, add mockups / screenshots to help explain present your vision of the feature
No response
If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.
If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.
No response

It can be caused by a bad installation of either Node or the TypeScript language server. Could you please try the following steps:

Quit Zed completely.
In a terminal of your choice run:

$ rm -rf ~/Library/Application\ Support/Zed/node
$ rm -rf ~/Library/Application\ Support/Zed/languages/typescript-language-server

Open Zed again.

Is the issue resolved?

Yes thanks @hovsater

Great! Feel free to close the issue as well. 🙂


viewpager.setSlideBorderMode(AutoScrollViewPager.SLIDE_BORDER_MODE_CYCLE) doesn't behave properly

I have autoscrollviewpager inside recyclerview and i want it to scroll in both direction i.e. Left and Right.
when i use viewpager.setSlideBorderMode(AutoScrollViewPager.SLIDE_BORDER_MODE_CYCLE)
then it become scrollable in both direction but it shows black screen for first couple of time. And whenever i am scrolling viewpager, it doesn't listen and recyclerview takes scrolling event it scrolls vertically.
But if do not set  any slideboardMode then it scrolls to only right side but vertical scrolling issue doesn't appear that much .

Have you get Any solution regarding this issue. I also faces same issue while scrolling from last to first and vice versa. please reply if any solution found.


Error Running transformation: Mono.Cecil.AssemblyResolutionException

Outstanding work on the serenity template, Kudos to @volkanceylan and all the people responsible for this masterpiece.
I've been working with the template for almost 2 years using VS 2013 without issues. This week we decided to upgrade to VS 2017 so we upgraded to the latest serenity update as well (15/9/2018).
When I transform T4 templates the 3rd step ends failing:

In the error list I receive the following error:
Severity	Code	Description	Project	File	Line	Suppression State
Error		Running transformation: Mono.Cecil.AssemblyResolutionException: Failed to resolve assembly: 'StackExchange.Exceptional, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'
at Mono.Cecil.BaseAssemblyResolver.Resolve(AssemblyNameReference name, ReaderParameters parameters)
at Mono.Cecil.DefaultAssemblyResolver.Resolve(AssemblyNameReference name)
at Mono.Cecil.MetadataResolver.Resolve(TypeReference type)
at Mono.Cecil.TypeReference.Resolve()
at Serenity.Reflection.CecilUtils.d__8.MoveNext() in C:\Sandbox\StartSharp\Serenity\Serenity.CodeGenerator\CodeGeneration\Base\CecilUtils.cs:line 85
at System.Linq.Buffer1..ctor(IEnumerable1 source)
at System.Linq.Enumerable.ToArray[TSource](IEnumerable`1 source)
at Serenity.CodeGeneration.CecilImportGenerator.GenerateAll() in C:\Sandbox\StartSharp\Serenity\Serenity.CodeGenerator\CodeGeneration\Base\CecilImportGenerator.cs:line 147
at Serenity.CodeGeneration.ServerTypingsGenerator.GenerateAll() in C:\Sandbox\StartSharp\Serenity\Serenity.CodeGenerator\CodeGeneration\ServerTypings\ServerTypingsGenerator.cs:line 30
at Microsoft.VisualStudio.TextTemplating00664DE7EE471288B57CC2D5BFCE4748015EB922B75B3165830467F05195FAD205DB303A46524E52EF07C4A9E430F0BBE56814A01B84B303C3B60FEC63AB6A58.GeneratedTextTransformation.TransformText()	***	*******\Imports\ServerTypings\ServerTypings.tt	1
It's probably related to the StackExchange.Exceptional.dll but I'm unable to solve it or get past this error.
If anybody could shed any light on this problem, I would greatly appreciate it. Thanks in advance.

Try updating Serenity.Web.Tooling package and clear bin directory, rebuild

Thanks for your reply @volkanceylan . I finally found the real culprit & managed to fix this issue. As it turned out, it was a network issue that caused trouble only when transforming T4 templates... pretty weird huh...
Anyway, thanks again for this great product.


Make resolve() in WorkerLoader work as expected

Currently it relies on someone having called request() prior to populate the container member, which is not the typical calling pattern for a loader.

Probably not worth it at this point.


Trailing comma is invalid syntax for calling functions

EdgeDB Version: 1.0-alpha.4+dev.110.gc569d8dc
OS Version: Docker

Steps to Reproduce:
CREATE FUNCTION test(a: int64) -> int64 { USING ( SELECT a ) };
SELECT test(
  1,
)
# error: Unexpected ')'

In python this is a valid syntax:
test(
  1,
)
test (1,)

And this is not:
test(,)

Current behavior is inconsistent, trailing commas are allowed in SELECT, INSERT, etc.

Dupe of #1462?

Sorry, didn't notice that issue


downloading in werong user

it downloads in the the other user even after i change destination path to my user

Closing this because it's not something we can control.


[Event Grid] Live test pipeline broken

Since the mgmt tests were merged in https://github.com/Azure/azure-sdk-for-net/commit/0ea6efbe000c1d3a3a16719c85ebf83230ae555f, the live test runs have been timing out. We should either split into two separate pipelines and use an adequate timeout for the mgmt pipeline, or disable the mgmt live tests if they cannot be run in a reasonable amount of time.

/cc @HarveyLink @ArthurMa1978

Let's please split the pipelines.

I thought we made a broad stroke change to split all the pipelines? I am curious why this was missed and if there are others that may be in the same boat.

I thought we made a broad stroke change to split all the pipelines? I am curious why this was missed and if there are others that may be in the same boat.

I think the change was to split CI pipelines. This issue is about splitting test pipelines.

I thought we made a broad stroke change to split all the pipelines? I am curious why this was missed and if there are others that may be in the same boat.

I think the change was to split CI pipelines. This issue is about splitting test pipelines.

My bad. I think then we need to do this across SDKs too! I will talk to the engsys team and see how best to do it.

I had created a work-item to make the split the default so we don't accrue any unintentional debt  - https://github.com/Azure/azure-sdk-for-net/issues/31820 but looks like we have some clean up to do here.

I have a PR https://github.com/Azure/azure-sdk-for-net/pull/31927 here to split the test pipelines for EventGrid


Typescript Reference not working with Nuxt

Using this types reference with nuxt is not working.
/// <reference types="@spiriit/vite-plugin-svg-spritemap/client" />
However using /// <reference types="@spiriit/vite-plugin-svg-spritemap/dist/client" /> is working 🤷♂

I tried with no success to reference in the tsconfig, directly reference the package. I close this issue since there s a workaround but it is not perfect.


⚠ Bofrost.gr has degraded performance

In bc548b3, Bofrost.gr (https://www.bofrost.gr) experienced degraded performance:

HTTP code: 200
Response time: 1828 ms

Resolved: Bofrost.gr performance has improved in 228d21f.


create new line at end of program

had an issue with there not being new lines, see photos for before vs after

this was planned, thank you


Enhancement — more modular loading

First off, let me say how impressed I am with wasm-vips — amazing job 🤩👏🥳 Can't tell you how great your lib "just works" in comparison to about 8 other browser-based image manipulation libs I've tried, despite its supposed alpha quality. (If this is alpha quality, all the other libs are laughably early pre-alphas!)
The suggestion here is around more modular loading. The dynamicLibraries option is super helpful and shaves off quite a few unnecessary MB. However, I was wondering whether we could extend the concept to some of the more "core" Vips functionality. For instance, I only need webp resizing and encoding. I realize tree-shaking WASM is not yet a thing and may never be, but perhaps it's possible to further granulate wasm-vips analogously to dynamicLibraries. For instance I can imagine passing only features: ["resize", "web-encode"] or however the API might look.
Probably a big ask because ostensibly libvips itself isn't modular in the way I'm describing, but worth a discussion, I think.
Thanks again!

In practice, perhaps this is not needed due to WASM streaming.

Hi @alexandergunnarson,
Sorry for the delay. A significant portion of the vips.wasm file size comes from the foreign loaders and savers. A minimal libvips (i.e. with only core functionality) can be configured using this patch:
--- a/build.sh
+++ b/build.sh
@@ -488,11 +488,7 @@ node --version
   sed -i "/subdir('man')/{N;N;N;N;d;}" meson.build
   meson setup _build --prefix=$TARGET --cross-file=$MESON_CROSS --default-library=static --buildtype=release \
     -Ddeprecated=false -Dexamples=false -Dcplusplus=$LIBVIPS_CPP -Dauto_features=disabled \
-    ${ENABLE_MODULES:+-Dmodules=enabled} -Dcgif=enabled -Dexif=enabled ${ENABLE_AVIF:+-Dheif=enabled} \
-    -Dheif-module=enabled -Dimagequant=enabled -Djpeg=enabled ${ENABLE_JXL:+-Djpeg-xl=enabled} \
-    -Djpeg-xl-module=enabled -Dlcms=enabled ${ENABLE_SIMD:+-Dhighway=enabled} ${ENABLE_SVG:+-Dresvg=enabled} \
-    -Dresvg-module=enabled -Dspng=enabled -Dtiff=enabled -Dwebp=enabled -Dnsgif=true -Dppm=true -Danalyze=true \
-    -Dradiance=true -Dzlib=enabled
+    -Dnsgif=false -Dppm=false -Danalyze=false -Dradiance=false
   meson install -C _build --tag runtime,devel
   # Emscripten requires linking to side modules to find the necessary symbols to export
   module_dir=$(printf '%s\n' $TARGET/lib/vips-modules-* | sort -n | tail -1)

And by compiling with -Oz instead of -O3, using:
--- a/build.sh
+++ b/build.sh
@@ -111,7 +111,7 @@ done
 export RUSTFLAGS="-Ctarget-feature=+atomics,+bulk-memory,+nontrapping-fptoint"

 # Common compiler flags
-COMMON_FLAGS="-O3 -pthread"
+COMMON_FLAGS="-Oz -pthread"
 if [ "$LTO" = "true" ]; then
   COMMON_FLAGS+=" -flto"
   export RUSTFLAGS+=" -Clto -Cembed-bitcode=yes"

With that, I see:
$ ls -lh lib/*.wasm | awk '{print $5,$9}'
2.9M lib/vips-O3.wasm
2.2M lib/vips-Oz.wasm

However, most WebAssembly files are served Brotli-compressed nowadays. In the ideal case, with v0.0.8, this would mean that ~1.5MiB is transferred over the network.
$ brotli node_modules/wasm-vips/lib/vips.wasm --best -v
Compressed [node_modules/wasm-vips/lib/vips.wasm]: 5.388 MiB -> 1.509 MiB in 8.49 sec

(usually the size is ~1.8MiB due to the use of compression level 4, as shown in your screenshot)
Running the same command over the produced WebAssembly binairies above, I see:
$ brotli lib/vips-O{3,z}.wasm --best -v
Compressed [lib/vips-O3.wasm]: 2.872 MiB -> 968.115 KiB in 4.98 sec
Compressed [lib/vips-Oz.wasm]: 2.144 MiB -> 896.830 KiB in 4.26 sec

I think that modularizing every operation within wasm-vips adds unnecessary complexity to the build logic without yielding any significant benefits. We could, however, experiment with making more foreign loaders/savers as dynamic module.
Let's tag this as an enhancement.

Sorry for my own delay. Thanks for your thorough response — some great ideas here!


cosmos-tx crate

Adds a cosmos-tx crate which provides a transaction builder and signer for Cosmos-based blockchains.
This commit ports over the WIP protobuf support from:
https://github.com/iqlusioninc/crates/tree/develop/stdtx

For repos with multiple crates like this, I think it's helpful to have separate CI jobs for each crate. Here's an example:
https://github.com/RustCrypto/AEADs/tree/master/.github/workflows
I didn't do that yet, but I think it'd be a good idea.

I can split the workflows and then it should be PoC-ready, if you'd like to handle integration testing in a separate PR.

yup sounds good, integration testing should be separate as it's going to be big enough to review on it's own.


Upgrade to phpseclib3

Upgrade Tests / Code to phpseclib3
Use phpseclib3 pow method

additional changes made to branch. We will create a new PR for the phpseclib3 upgrade.


gramma server install fails

Describe the bug
command 'npx gramma server install' fails:
res.body.pipe(str).pipe(fileStream)
             ^
TypeError: res.body.pipe is not a function

To Reproduce
install gramma with node 18.16.0 & npm 9.5.1
Expected behavior
It should download the file
Screenshots

Desktop (please complete the following information):

OS: macOS
node 18.16.0

Additional context
Seems to work if I replace this slice of code in downloadFile.js (L30):
return new Promise((resolve, reject) => {
  res.body.pipe(str).pipe(fileStream)
  res.body.on("error", (err) => {
    reject(err)
  })
 fileStream.on("finish", () => {
   bar.stop()
   resolve()
 })
})

by
await pipeline(res.body, str, fileStream)
bar.stop()

with const { pipeline } = require('node:stream/promises');

Any progress here? The bug also hurts me


KnownMatrixGate should automatically implement inverse

One should not be forced to define EigenGate to have basic gate functionality in Cirq. 99.9% of users will never raise a gate to a fractional power and will find EigenGate terribly confusing. However, taking the inverse of a gate should be simple and should not require EigenGate. In the case of KnownMatrixGate, the gate inverse should be inferred directly from the matrix.

This issue is stale as these gate classes no longer exist (pre protocols).  We have a __pow__ protocol and MatrixGate implements it by default.
However +1 to users not using powering of gates, we should really audit some code to see how useful powering is, a lot of the research code I have looked at does not use this.


Add getter and setter

Add a getter and a setter to the position.
Signed-off-by: Freyskeyd<EMAIL_ADDRESS>

Why is it needed?

The getter is a useful information to display (I used it in trace!)
For the setter I don't know, maybe it's completely useless ?

The getter can be useful, but the setter would break the behaviour of the
buffer
Le 5 mai 2017 09:10, "Simon Paitrault"<EMAIL_ADDRESS>a écrit :

The getter is a useful information to display (I used it in trace!)
For the setter I don't know, maybe it's completely useless ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sozu-proxy/circular/pull/1#issuecomment-299397445,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHSABnlDcDlBWDGLS2PEx_R-xPlmvzqks5r2stIgaJpZM4NRK4s
.

That's right, I will remove it.

I've a question about how this repository is named. Circular but the main struct if Buffer. Would keep that way?

probably, it's just a crate name


Could anyone provide pre-trained model?

It takes whole day still unfinished on my MBP,I don't have ideas of how long it will takes.

Hi, I added a link for a pre-trained model (with default parameters and 10 epochs) at README.md.

Thanks ! I'll try it later.


Python hello world does not work

Describe the bug
There are two issues:

Call to non-existing function import_semantic_skill_from_directory
Depricated model text-davinci-003

To Reproduce
Steps to reproduce the behavior:

Run the example

Expected behavior
Should generate some output
Desktop (please complete the following information):
All
Additional context
Add any other context about the problem here.

It appears that many examples in this repo are based on old, deprecated code.
The examples in the main repo have a pretty different looking API.


Show Replace history for Editor find widget

Show Replace history for Editor find widget

Follow up issue from #18233

@rebornix I saw that you fixed it in #50189. I am sorry I did not inform you that I was also looking into this. Actually, I have introduced history in Search & Find in the beginning, so I thought I will finish it by fixing this :).
Anyway, I implemented in generic way here. History navigation was implemented outside the inputBox. So every time we need this feature I am duplicating it. For example, similar code is there at following places

Search input
Search replace
Search pattern inputs
Find input
Find replace
Problems filter
Terminal find

I now have a HistoryInputBox widget which has history navigation. Only thing that has to be done from outside to call APIs add/show.
Let me also tell you another cool change I am doing here. At present, we have two commands (previous/next) per each input box for navigating history. Since history navigation is encapsulated in the widget, I can now have only two commands for all above input boxes. It is similar to the List widget with commands. So basically, I will remove all commands and replace them with following two commands
input.action.historyNext, input.action.historyPrevious.
This is a breaking change but worth to have.

@sandy081 I remembered I saw this issue/task someday but when I was going through my debts I forgot about it. Your change is the right way to handle the history navigation and it's great that we have this HistoryInputBox. BTW, do you plan to adopt this component for Terminal and Webview Find box ?

Yes, Terminal & Webview find widgets were also adopted.

@rebornix @Tyriar @mjbvz
I see that you are using find history navigation feature in Editor, Terminal & Webview respectively. Currently each component defines its own commands for navigation. As an user if I want to customise, I have to update all these commands which is cumbersome. Hence, I have merged all these commands into two generic commands input.action.historyNext and input.action.historyPrevious. Please read comment for more information. Impact of this change is that the current history navigation commands will be deprecated in favour of above.
For one milestone, I would be showing following deprecating warning message when user triggers current commands and will remove them in the following milestone.

I like this change, thanks for doing this. Please just give me a FYI @mention for stuff like this in the future, I thought it was broken for a bit :)

cc @cleidigh .

LGTM 👍

@roblourens Oops.. Sorry Rob.. How could I forget you... Not next time 👍

@roblourens @rebornix @Tyriar @mjbvz
FYI: Removed following commands
[

		'search.history.showNextIncludePattern',
		'search.history.showPreviousIncludePattern',
		'search.history.showNextExcludePattern',
		'search.history.showPreviousExcludePattern',
		'search.history.showNext',
		'search.history.showPrevious',
		'search.replaceHistory.showNext',
		'search.replaceHistory.showPrevious',

		'find.history.showPrevious',
		'find.history.showNext',

		'workbench.action.terminal.findWidget.history.showNext',
		'workbench.action.terminal.findWidget.history.showPrevious',

		'editor.action.extensioneditor.showNextFindTerm',
		'editor.action.extensioneditor.showPreviousFindTerm',

		'editor.action.webvieweditor.showNextFindTerm',
		'editor.action.webvieweditor.showPreviousFindTerm'

	]


Prevent model saving in disk

Is there a way to avoid saving of model in the disk? I checked the docs but if the path to set to None instead of not saving, it saved it to a time-stamped based folder.

Not at present. This is to simplify the code and avoid nasty out-of-memory errors. If the amount of disk usage is the problem, the upcoming v0.2.0 reduces Tabular's disk usage by ~10x+ in many cases.

Closing as wontfix for now. Adding this functionality adds a great deal of code complexity for not much benefit. Please re-open if you can highlight a use-case where saving to disk is not an option.

Hi @Innixma ,
I met the same problem here. My use case is to build a distributed ML pipeline system, and all the in-middle steps will be pickled and saved by the system. Here I use Autogluon as a good operator, but since it will auto-save the model after the fit process, I have to delete it by my self (or if there is a better way, like a custom predictor base on tablular and omit the saving process, please tell me :)
I mean, it's ok to use disk before the final training process, but I do not want to save the model by Autogluon, and prefer to deal with it myself. So I think in this case, giving a flag to disable model save would be good.


Type inference problem (regression ?)

TypeScript Version:  any > 2.1.4
I have been unable to reproduce the problem easily using only built-in objects, so I made a test case with the library I am building/using which exhibits the bad behaviour.
There is a repository here with the code ready to be run : https://github.com/ceymard/typescript-inference-problem
Code
/**
 * This example cannot compile with typescript < 2.1.4 where the error is already present.
 */

import {Repeat, Observable, d} from 'domic'

/**
 * In domic, an Observable is a container object that holds a value and
 * alerts subscribers whenever this value (or a subvalue) changes
 */
var arr = new Observable({a: [1, 2, 3]})

/**
 * Here, arr.p('a') gives us a PropObservable<{a: number[]}, number[]>,
 * which is itself an Observable<number[]> since PropObservable is a subclass of it.
 *
 * The Repeat() function acts as some kind of a map() method, taking an Observable<T[]>
 * and receiving a callback expecting a T. So far, the inferer never made any problem
 * and always guessed right.
 *
 * Unfortunately, since at least 2.1.4, the Repeat call is typed as
 * Repeat<any>(...) instead of the correct type.
 *
 * This is a problem in itself alright, but more upsetting is the fact that even
 * with noImplicitAny there is no error !
 *
 * I introduced someNonExistentMethod() on purpose to try to check whenever
 * value is not typed as any.
 */
document.body.appendChild(Repeat(arr.p('a'), value =>
  d('div', {}, value.someNonExistentMethod()) // value is a non-reported implicit 'any' instead of a number
))

/**
 * When helping the type inferer by telling it that prop_a is in fact
 * an Observable<number[]>, the Repeat() call gets typed correctly.
 */
document.body.appendChild(Repeat(arr.p('a') as Observable<number[]>, value =>
  d('div', {}, value.someNonExistentMethod()) // this is correctly flagged as an error.
))

Expected behavior:
I should have two error messages, one for each bogus call of someNonExistentMethod()
Actual behavior:
There is only one error reported, and the whole block where Repeat is typed as Repeat does not even tell that there was an implicit any.
Note : here is a very similar use case of the code I'm using, which works as intended !
class Obj<T> {
  constructor(public val: T) { }

  p<K extends keyof T>(prop: K): SubObj<T, T[K]> {
    return new SubObj<T, T[K]>(this.val, this.val[prop])
  }
}

class SubObj<U, T> extends Obj<T> {
  val2: U // just to have some U stuff
  constructor(public orig: U, new_val: T) {
    super(new_val)
  }
}

var test = new Obj({a: [1, 2, 3]})
var test2 = test.p('a')

function Test<T>(obj: Obj<T[]>, callback: (a: T) => void): void {
  // !!!
}

// Here, someBogusMethod is flagged as erroneous, even though test2 is a SubObj and
// not an Obj. value is correctly typed as number.
Test(test2, value => value.someBogusMethod())

Reduced repro that works in Playground
export declare function Repeat<T>(ob: Observable<T[]>, render: (e: T) => void): T;
export declare type Observer<T> = { foo: T }

export declare class Observable<T> {
    _value: T;
    constructor(value: T);
    set(value: T): boolean;
    set<K extends keyof T>(prop: K, value: T[K]): boolean;
    set<V>(this: Observable<V[]>, idx: number, value: V): boolean;
    addObserver(fn: Observer<T>): void;
    addObserver<U>(this: Observable<U[]>, fn: Observer<U>, prop?: number): void;
    addObserver<P extends keyof T>(fn: Observer<T[P]>, prop?: P): void;
}
export declare abstract class DependantObservable<T> extends Observable<T> {
    addObserver<U>(this: Observable<U[]>, fn: Observer<U>, prop?: number): void;
    addObserver<P extends keyof T>(fn: Observer<T[P]>, prop?: P): void;
    addObserver(fn: Observer<T>): void;
}
export declare class PropObservable<T, U> extends Observable<U> {
    set<K extends keyof U>(prop: K, value: U[K]): boolean;
    set<A>(this: Observable<A[]>, idx: number, value: A): boolean;
    set(value: U): boolean;
}

// Works (a: number)
declare const m1: Observable<number[]>;
var a = (Repeat(m1, value => null));

// Can't infer T
declare const m2: DependantObservable<number[]>;
var b = (Repeat(m2, value => null));

// T is silently any (??)
declare const m3: PropObservable<{a: number[]}, number[]>;
var c = (Repeat(m3, value => null));

If I change the methods to have the same ordering as the base class, it works:
export declare function Repeat<T>(ob: Observable<T[]>, render: (e: T) => void): T;
export declare type Observer<T> = { foo: T }

export declare class Observable<T> {
    _value: T;
    constructor(value: T);
    set(value: T): boolean;
    set<K extends keyof T>(prop: K, value: T[K]): boolean;
    set<V>(this: Observable<V[]>, idx: number, value: V): boolean;
    addObserver(fn: Observer<T>): void;
    addObserver<U>(this: Observable<U[]>, fn: Observer<U>, prop?: number): void;
    addObserver<P extends keyof T>(fn: Observer<T[P]>, prop?: P): void;
}
export declare abstract class DependantObservable<T> extends Observable<T> {
    addObserver(fn: Observer<T>): void;
    addObserver<U>(this: Observable<U[]>, fn: Observer<U>, prop?: number): void;
    addObserver<P extends keyof T>(fn: Observer<T[P]>, prop?: P): void;
}
export declare class PropObservable<T, U> extends Observable<U> {
    set(value: U): boolean;
    set<K extends keyof U>(prop: K, value: U[K]): boolean;
    set<A>(this: Observable<A[]>, idx: number, value: A): boolean;
}

// Works (a: number)
declare const m1: Observable<number[]>;
var a = (Repeat(m1, value => null));

// Works
declare const m2: DependantObservable<number[]>;
var b = (Repeat(m2, value => null));

// Works
declare const m3: PropObservable<{a: number[]}, number[]>;
var c = (Repeat(m3, value => null));

This may, unfortunately, be by design due to erasure of constraints during inference -- when the signatures don't line up, we may end up relating an erased-to-any type parameter to a T

Are you talking about overloading order or method declaration order ?
Shouldn't the compiler "sort" them internally to keep some kind of logic ?

The order of overloads matters - there is no "sorting". We tried to sort in the past and it created more problems than it solved.

Okay, I'll fix the ordering in observables then.
Shouldn't the "match to any" be reported under noImplicitAny however ? I missed some errors because tsc said nothing.

I need to reduce the problem further but agree that we ought to implicit-any error here if possible. We might be able to fix the inference depending on what the smallest repro looks like

This repro is bananas; reordering these signatures at all removes the any. I also can't make any of the signatures simpler - even removing idx fixes the inference.
declare function Repeat<T>(ob: Observable<T[]>): T;

declare class Observable<T> {
    _value: T;
    set(value: Date): void;
    set<K extends keyof T>(prop: K, value: T[K]): void;
    set<V>(idx: number, value: V): void;
}
declare class PropObservable<T> extends Observable<T> {
    set<K extends keyof T>(prop: K, value: T[K]): void;
    set<V>(idx: number, value: V): void;
    set(value: Date): void;
}

// Works (a: number)
declare const m1: Observable<number[]>;
var a = (Repeat(m1));

// c is silently any (??)
declare const m3: PropObservable<number[]>;
var c = (Repeat(m3));

Removed as many type parameter uses as I could. The fact that changing T[K] in both keyof signatures to T or K causes the issue to no longer occur makes me believe this is a bug -- doing that should not change inference at all as far as I can tell.
declare class Observable<T> {
    _value: T;
    m(value: Date): void;
    m<K extends keyof T>(prop: Element, value: T[K]): void;
    m<V>(idx: number, value: V): void;
}
declare class PropObservable<T> extends Observable<T> {
    m<K extends keyof T>(prop: Element, value: T[K]): void;
    m<V>(idx: number, value: V): void;
    m(value: Date): void;
}

declare function Repeat<T>(ob: Observable<T[]>): T;

// Works (a: number)
declare const m1: Observable<number[]>;
var a = (Repeat(m1));

// T is silently any (??)
declare const m3: PropObservable<number[]>;
var c = (Repeat(m3));

declare class Observable<T> {
    _value: T;
    m<K>(idx: number, value: K): void;
    m<K extends keyof T>(prop: Element, value: T[K]): void;
}
declare class PropObservable<T> extends Observable<T> {
    m<K extends keyof T>(prop: Element, value: T[K]): void;
    m<K>(idx: number, value: K): void;
}

declare function Repeat<T>(ob: Observable<T[]>): T;

// Works (a: number)
declare const m1: Observable<number[]>;
var a = (Repeat(m1));

// T is silently any (??)
declare const m3: PropObservable<number[]>;
var c = (Repeat(m3));

Where are we at on this bug ? It would seem it is still present.

This seems to have been fixed - none of the reduced repros display the problem anymore.


2.0 run contain fail

Rancher versions:
rancher/server: server:preview
rancher/agent: v2.0-alpha4
Infrastructure Stack versions:
healthcheck:
ipsec:
network-services:
scheduler:
kubernetes (if applicable):
Docker version: (docker version,docker info preferred)
1.12.6
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Steps to Reproduce:
Results:
i want to run nginx container

There have been major changes since this tech preview, please try the latest (v2.0.0-alpha23, https://forums.rancher.com/t/rancher-tech-preview-update-3-26-2018/9893) and file a new issue if you run into trouble.


update citation

I want to cite this R package, but I noticed that the citation is a little out of data (e.g. old version, missing authors), compared to what is in the DESCRIPTION file. I looked for a CITATION file, but I could not find one. I'm happy to update it if you point me to where the changes need to be made.
> citation("datos")

To cite package ‘datos’ in publications use:

  Edgar Ruiz, Riva Quiroga, Mauricio Vargas and Mauro Lepore (2019). datos: Traduce al
  Español Varios Conjuntos de Datos de Práctica. R package version 0.1.0.
  https://CRAN.R-project.org/package=datos

A BibTeX entry for LaTeX users is

  @Manual{,
    title = {datos: Traduce al Español Varios Conjuntos de Datos de Práctica},
    author = {Edgar Ruiz and Riva Quiroga and Mauricio Vargas and Mauro Lepore},
    year = {2019},
    note = {R package version 0.1.0},
    url = {https://CRAN.R-project.org/package=datos},
  }

Hi @raynamharris! The citation is build upon the DESCRIPTION file.

I'm closing this issue as it can be solved through #139


feat: Add condense() on all supported indicators

Description

Add condense() on all supported indicators
Bump up minimum version of typing_extensions to 4.4.0 for @override

Issue

pythonnet does not support extension method yet

https://github.com/pythonnet/pythonnet/issues/55

Checklist

[ ] My code follows the existing style, code structure, and naming taxonomy
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have performed a self-review of my own code and included any verifying manual calculations
[ ] I have added or updated unit tests that prove my fix is effective or that my feature works, and achieves sufficient code coverage.  New and existing unit tests pass locally and in the build (below) with my changes
[ ] My changes generate no new warnings and running code analysis does not produce any issues
[ ] I have added or run the performance tests that depict optimal execution times
[ ] I have made corresponding changes to the documentation

@DaveSkender  I think Codacy config is reset. Please exclude /docs and /tests from its coverage target.

That's the part I'm worried about recently. As I mentioned above, extension method is not supported, so we may need to find another approach to implement those features(chaining, streaming)


add direct download info

It's not the same thing that is below "How to use"? "When downloading a file replace https://mega.nz with https://directme.ga". Maybe we can make it easier to users by making a form where they enter the original MEGA URL and it returns the Direct MEGA URL...
By the way: you don't need to replace the "#" with a "?" as it's handled for you since this change.

However, one gives you a direct download, one links you with a direct
download. The direct download link is a necessity for my use.
On 25 Nov 2017 1:10 pm, "Gustavo Rodrigues"<EMAIL_ADDRESS>wrote:

It's not the same thing that is below "How to use"? "When downloading a
file replace https://mega.nz with https://directme.ga". Maybe we can make
it easier to users by making a form where they enter the original MEGA URL
and it returns the Direct MEGA URL...
By the way: you don't need to replace the "#" with a "?" as it's handled
for you since this change
https://github.com/qgustavor/direct-mega/commit/522fb9397437dc49c6ebfa3d5d55ea0be7acb20e#diff-aa50e0dbefe3259b6e91300af13d3479R183
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/qgustavor/direct-mega/pull/13#issuecomment-346939707,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHEE5iahZ5uP5hBjSDkg5wKHU6vefTB9ks5s6BHCgaJpZM4QqZcy
.

As it runs in a browser there's no difference between those: loading with "#" will redirect to "?" automatically. Trying to load it outside a browser will download this HTML page.
Because as Foreign Fetch will be removed from the spec if you try to load content from Direct MEGA using something like <img src="https://directme.ga/?..."> the Service Worker will not be called and it will not work (it will load the HTML page).
There is some situation where the redirect don't works? Also, how changing the readme will improve that?

Closing as there wasn't a response.


Modal wrong action routing

https://github.com/the-control-group/voyager/blob/0b11c4c09d494ca72497ec0fc2e66dd21fa457d6/resources/views/menus/browse.blade.php#L111
Should be:
$('#delete_form')[0].action = '{{ route('voyager.'.$dataType->slug.'.index') }}/' + id;

Because if not, it's appending the id of the menu constantly.
Steps:

Click delete in any menu
Click cancel on the modal.
Click delete in the same menu or any other.

Result, sometime like that: http://voyager.dev/admin/menus/3/1/3

I'll try!

Pr submitted.


I would like to ask how to use this model in a java project

I would like to ask how to use this model in Java project and how to get the results from web in Java. Thank you very much!

I’m not too versed in Java so I’m not sure, but I assume you would need to convert the model to TensorFlow lite

I’m not too versed in Java so I’m not sure, but I assume you would need to convert the model to TensorFlow lite


Possible incompatibility with 'WD Minimap' extension

I had installed 'WD Minimap' and then I installed your plugin. It's works well, but when I press the right button for 'hide toolbar' the 'minimap' dissapear. When I try to press again the button for 'show toolbar', toolbar is showed, but the minimap still not working. The button for 'show worklist' doesn't work too.

"WD Minimap" not well developed, I will find a solution to this, I also hope that the developer "WD Minimap" improve its extension.

I can not find solution for this extension :unamused:


[BUG] Unhandled exception. System.IO.FileNotFoundException: The configuration file 'configuration.prod.yaml' was not found and is not optional.

Release version
v6.0.1
Describe the bug
Following the current instuctions for v6.0.1 I didn't see references to create this file. configuration.prod.yaml. I have a github environment with the secrets from the documentation but the GH action fails until you create the file.
Expected behavior
action should continue if you do not have any GH environment overrides and the file does not exist or the documentation may need to be updated.
Actual behavior
created a blank file and the error goes away.
Reproduction Steps
run through the APIOPs documentation in a new GH repo. publish your first change to an API and promote up to prod.

We have a sample file on the home repo. Also our wiki goes into specifics on how to create it. Finally, you can reference the official documentation of the APIM restful API, which basically tells you what the configuration should look like as APIops follows their specification.

Example configuration override
API Management REST API

I'm getting same error
Unhandled exception. System.IO.FileNotFoundException: The configuration file '' was not found and is not optional.
If the configuration file is not optional then why there's this condition
      ${{ if ne( parameters['CONFIGURATION_YAML_PATH'], '' ) }}:
        CONFIGURATION_YAML_PATH: ${{ parameters.CONFIGURATION_YAML_PATH }}

& https://github.com/Azure/apiops/blob/6e9f9b73d1a76c1811e6e578aced64615b17591a/tools/azdo_pipelines/run-publisher-with-env.yaml#L155C1-L156C75


Add code club 1 sample

add code snippet for session 1

@danfishgold I guess let's just put this swift code in - as decision made for session one. Then follow this pattern for adding code.pdf to the other 2 sessions?
with 2 of our best examples in python, js, or java


[MM-27231]: e2e for T1880 - deleted token cannot be used to post

Summary

Update apiAccessToken to return full token response
Add  apiRevokeAccessToken method to revoke token
cypress test for T1880 - deleted token cannot be used to post

Ticket Link

https://mattermost.atlassian.net/browse/MM-27231

/update-branch

/update-branch

@ogi-m I'll go ahead and merge this PR and update the TM4J and JIRA tickets


disconnect event doesn't trigger when close the app, not by exiting

*title*

not from raknet

it does trigger


Extend support for other package formats (ipk and rpm)

Implemented in #63. Meta-tedge doesn't use install scripts from thin-edge.io. Instead, yocto specific script is delivered that is compatible with all package formats.


Congestion control

I'm currently working on congestion control & loss dection. I'm basing my work on  draft-ietf-quic-recovery-06 [1].
Progress can be somewhat followed in [2].
[1] https://tools.ietf.org/html/draft-ietf-quic-recovery-06
[2] https://github.com/pietdevaere/minq/tree/congestion_control

@ekr, what do you think of the idea to split up the outputClearQ and the outputProtectedQ in to two queues each: outputClearQ and outstandingClearQ. When frames are handed off by the stream to the connection, they are placed in outputClearQ. When they are transmitted, they are moved to outstandingClearQ. When the loss detector considers them lost, they are moved back to outputClearQ, and when they are acked they are removed from both queues. That way sendQueuedFrames() (which I splitted of from queueStreamFrames) does not have to iterate through the output queue to find packets that it can transmit. It can just take the top ones from outputClearQ.
To make things more efficient we could maybe also add a map[pn][]*frame kind of structure tracking which frames a packet contains. That way the loss detector & ack handler don't have to do lengthy linear searches to find the frames corresponding to a packet. We could then also remove that info from the frame struct.


Update newfile

👍

hooray!

test

👍

👍

hmm, will fix

test

test

test

👍

👍

👍

👍

👍

👍

ahhhhhh why are there so many errors

okay now it will work

👍

YESSS IT WORKED


Extend error reporting

Hi, thanks for the gem! It made setting up JSON Patch easy for us.
We had a requirement to have some more detail in the error messages when things went wrong, so this PR was made to accomplish that goal, but the scope did end up creeping a bit. This PR:

Adds messages to existing errors that were being raised.
Adds more error handling for some cases that weren't being caught. The two cases we found here were when path was not specified on operations other than add, and when the path tried to index into a string, boolean, or number (so if the path was /birthDate/foo and /birthDate was a string).
Refactors the replace function to use rm_op and then add_op, instead of replacing the value directly. We did this because we noticed that if a replace operation was performed on target location which was on an object that existed, but didn't itself exist (so if foo was an object that didn't have a bar attribute, and the path was /foo/bar), replace would add the value, but by our reading of the spec the replace should fail in this case.

We merged this as one PR on our fork, because we needed the error messaging soon, but if you'd prefer this PR be broken up into smaller ones that are more specific, let me know and we can definitely do that.

This seems fine to me.  I'll merge and ship a new release.  Thank you!


Support for Mutiple Links enabled

https://user-images.githubusercontent.com/93882347/193349988-37edc1a7-378e-45d2-a67b-b60e67a2e4be.mp4

@akshatjainei Please resolve the branch conflicts


2.5.1 stubs are unreadable by WireMock 2.1.7

Obviously it's my bad that I haven't bumped up WireMock but anyways... The problem looks as follows.
At some point WireMock has started adding id and uuid fields to the JSON. Unfortunately old versions of WireMock are very strict about what should be there in the StubMapping so they are not happy to see those fields there. Example:
WireMock 2.1.7
{
  "request" : {
    "method" : "GET"
  },
  "response" : {
    "status" : 200
  }
}

WireMock 2.5.1
{
  "id" : "77514bd4-a102-4478-a3c0-0fda8b905591",
  "request" : {
    "method" : "GET"
  },
  "response" : {
    "status" : 200
  },
  "uuid" : "77514bd4-a102-4478-a3c0-0fda8b905591"
}

If you have an app using WireMock 2.1.7 and you get the second stub you get:
com.fasterxml.jackson.databind.exc.UnrecognizedPropertyException: Unrecognized field "id" (class com.github.tomakehurst.wiremock.stubbing.StubMapping), not marked as ignorable (7 known properties: "response", "requiredScenarioState", "priority", "uuid", "request", "newScenarioState", "scenarioName"])
 at [Source: {
  "id" : "77514bd4-a102-4478-a3c0-0fda8b905591",
  "request" : {
    "method" : "GET"
  },
  "response" : {
    "status" : 200
  },
  "uuid" : "77514bd4-a102-4478-a3c0-0fda8b905591"
}; line: 2, column: 11] (through reference chain: com.github.tomakehurst.wiremock.stubbing.StubMapping["id"])

	at com.fasterxml.jackson.databind.exc.UnrecognizedPropertyException.from(UnrecognizedPropertyException.java:62)
	at com.fasterxml.jackson.databind.DeserializationContext.handleUnknownProperty(DeserializationContext.java:834)
	at com.fasterxml.jackson.databind.deser.std.StdDeserializer.handleUnknownProperty(StdDeserializer.java:1093)
	at com.fasterxml.jackson.databind.deser.BeanDeserializerBase.handleUnknownProperty(BeanDeserializerBase.java:1477)
	at com.fasterxml.jackson.databind.deser.BeanDeserializerBase.handleUnknownVanilla(BeanDeserializerBase.java:1455)
	at com.fasterxml.jackson.databind.deser.BeanDeserializer.vanillaDeserialize(BeanDeserializer.java:282)
	at com.fasterxml.jackson.databind.deser.BeanDeserializer.deserialize(BeanDeserializer.java:140)
	at com.fasterxml.jackson.databind.ObjectMapper._readMapAndClose(ObjectMapper.java:3798)
	at com.fasterxml.jackson.databind.ObjectMapper.readValue(ObjectMapper.java:2842)
	at com.github.tomakehurst.wiremock.common.Json.read(Json.java:38)
	at com.github.tomakehurst.wiremock.stubbing.StubMapping.buildFrom(StubMapping.java:62)

Repo (just build and it will fail) https://github.com/marcingrzejszczak/wiremock_217_vs_251
So I guess the only way on our (library users) side is to try to parse the thing and if it fails due to UnrecognizedPropertyException then we should try to do sth about it (for example remove those unrecognized fields). WDYT?

Right, so the problem is that you're trying to use stub generated by a newer version of WM with an older one rather than the other way round?
Generally speaking I'm don't think forwards compatibility is a guarantee I'd ever make (backwards compatibility, yes).
This specific problem won't happen in future as an "ignore unrecognised" flag has been added to the Jackson config since 2.1.7. But obviously I can't go back and change 2.1.7 now, so I'm not sure I can offer advice other than "upgrade to 2.5.1".

Awesome - that's what I was expecting. I'll program this on my side (in the code that is using 2.1.7) to remove those fields and everything should work fine. Thanks!


bpaf_derive command on enum

#[derive(Debug, Clone, Bpaf)]
#[bpaf(command)]
enum Command {
    /// Parse and dump parsed messages
    Dump,
    /// Dump sliced feed without parsing
    Slice,
}

This should derive a selection between two commands: "dump" and "parse", currently it generates a single command: "command" with value that depends on a flag

Actually it should be like this and it is more consistent this way...
#[derive(Debug, Clone, Bpaf)]
enum Command {
    /// Parse and dump parsed messages
    #[bpaf(command)]
    Dump,

    /// Dump sliced feed without parsing
   #[bpaf(command)]
    Slice,
}


Draft PR to run validations for new resources to cdn-2024-06-01-preview

Choose a PR Template
Switch to "Preview" on this description then select one of the choices below.
Click here to open a PR for a Data Plane API.
Click here to open a PR for a Control Plane (ARM) API.

Closing this draft PR


train.py stop at 3321step

how can i train start from 3000 step?what command is?thanks

it stop at 1969 step this time……

@fangg2021 I cannot make any sense from this screenshot. Check your system log perhaps?

it does't have any errors, just Automatically stopped, sometimes ran fine. I just want to know, has any way to start from 3000 steps?
Now,i haved modify the train.xml's file params save_step=1000. But a little troublesome by this way...

@fangg2021 I have not faced this condition before...

try  add "--restore_step 1969 "?


Add HID module descriptions from corresponding header files.

Also see related issue https://github.com/MicrosoftDocs/windows-driver-docs-ddi/issues/1234

@DJm00n : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Extracting landmarks

We're trying to extract the 3D landmarks from the hand tracking (GPU) example. However, we're having a hard time figuring out how we can access the landmark data so we can process it in our software.
Unfortunately, we couldn't figure out how to do this by reading the documentation. If somebody could point us to the right direction, that would be greatly appreciated!
We want to process the 3D landmarks coming from mediapipe in a C++ application. What is the preferred/recommended way of doing this?

Please take a look at https://github.com/google/mediapipe/issues/200#issuecomment-599971551 and https://github.com/mgyong/mediapipe-issue200.

Call CalculatorGraph::ObserveOutputStream to register a packet callback, then use Packet::Get,especially Get<NormalizedLandmarkList>, to read data.
If you want to interfere the graph's execution, just write a Node and put it in the graph pbtxt.

Thank you for the information, guys.

@Tectu Were you able to extract the 3D landmarks from the hand tracking example. Can you share the steps?

Call CalculatorGraph::ObserveOutputStream to register a packet callback, then use Packet::Get,especially Get<NormalizedLandmarkList>, to read data.
If you want to interfere the graph's execution, just write a Node and put it in the graph pbtxt.

Can you please elaborate the process? Thanks


Update undici

Summary
Updates undici.
Implementation details
Should be a transparent dep bump.
How to validate this change
Make sure recorder works and all CI processes are green.

:broken_heart: Build Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2023-03-29T16:08:05.141+0000

Duration: 3 min 0 sec

Steps errors
Expand to view the steps failures

Error signal

Took 0 min 0 sec . View more details here
Description: githubApiCall: The REST API call https://api.github.com/repos/elastic/synthetics-recorder/pulls/390 return the message : java.lang.Exception: httpRequest: Failure connecting to the service https://api.github.com/repos/elastic/synthetics-recorder/pulls/390 : Ambiguous method overloading for method org.apache.commons.io.IOUtils#toString.Cannot resolve which method to invoke for [null, class java.lang.String] due to overlapping prototypes between:[class [B, class java.lang.String][class java.io.InputStream, class java.lang.String][class java.net.URI, class java.lang.String][class java.net.URL, class java.lang.String]

:robot: GitHub comments
Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

/test all : Re-trigger the build for all the stages.

/test all


1justingilmore seo working branch

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

@traya1

Hi @1justingilmore,
The weird errors we encountered have been resolve. Feel free to continue using this PR to store your commits.

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

Hi @1justingilmore,
My suggestions were based on the MS Style Guide, but seeing that you did not touch the rest of the content in the topic and it has been merged as is, I am good with it. Thanks.


Application Structure - Shared Directives

I apologize if this has already been asked and answered.
I am curious in the application structure layout where directives would be placed that are shared across components and features.
Would the recommendation be to add a common/ folder to place these pieces of shared code?
Thanks for the advice!

I'd like to build upon this with an additional question.  We had something that resided in a module, and then it became more commonly used.  Do you refactor at this point, or leave it in the original module?

@ralberts I'd create a reusable set of widgets in a module. Like a component library.
@voor Yes, I would refactor once I know a thing can be reused.


Add "jti" (JWT ID) to JWT

Token format  default JWT is too long ，and  JWT-EMPTY info is too little information.
It is recommended to add a unique identifier  "jti" (JWT ID), Developers can associate session context through jti。
The JWT spec mentions a jti claim which allegedly can be used as a nonce to prevent replay attacks:
The "jti" (JWT ID) claim provides a unique identifier for the JWT. The identifier value MUST be assigned in a manner that ensures that there is a negligible probability that the same value will be accidentally assigned to a different data object; if the application uses multiple issuers, collisions MUST be prevented among values produced by different issuers as well. The "jti" claim can be used to prevent the JWT from being replayed. The "jti" value is a case-sensitive string. Use of this claim is OPTIONAL.

Can the table "token" ower + name be used as "jti"

@KricYi maybe. Can you make a PR?

@RyaoChengfeng


fix(Entry): add missing curly bracket

Latest update relating to the malformed URLs left me with this syntax error:

This fixes it.

isch, thanks! looks like editing directly on github was a bad idea after all


refactor(linter): use the same miette graphical reporter for tests

I'm a bit worried that none of the tests failed though - would that be expected?

This is the forked graphical reporter, the testers are using the one from miette. Let me clean this up a little bit.

If it's not urgent, can I have a go at that?

If it's not urgent, can I have a go at that?

Yes of course!


Feature/ci

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 4 committers have signed the CLA.:x: awarde96:x: mathleur:x: jameshawkes:x: Adam WardeAdam Warde seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


solve_all_conditions condition parameter not used

LossFunction.cache_by_condition calls solve_all_conditions to compute a Solution for each condition. However, the conditions argument is not being used, and instead the function is using model.required_conditions to determine the required conditions. Is this a bug or is this expected behavior? If it's expected, then removing the conditions argument would be good so that it's not misleading.
source: model.required_conditions

Great find, thanks for reporting this.  I just committed a change which gets rid of the unnecessary argument.


[Feature] Hide Photok Icon and Open the Photok by phone Dialer

Is your feature request related to a problem? Please describe.
Not really a problem but just for more stealth and privacy.
Describe the solution you'd like
For even more Stealth some apps give the option to hide the Icon of the app as if it is not installed and it can be opened by dialing a code on the phone dialer app which then brings the vault app and ask for password like normal. So if a phone is shared the other user won't even know that there is a Photok Installed unless he make a deep inspection :)
Additional context
I am aware that this not really something to prioritize but in the future it would be nice addition to Photok

This should't be too hard to implement. I'll do some research tomorrow.
Cloud even fit in this release.

This should't be too hard to implement. I'll do some research tomorrow.
Cloud even fit in this release.

Did some research, should be able to add the feature in 1.2.0

Did some research, should be able to add the feature in 1.2.0

Wow Didn't expect this so fast your are the Best 👍 👍 👍

Wow Didn't expect this so fast your are the Best 👍 👍 👍

Refarences for implementation:

https://stackoverflow.com/questions/8398514/hide-application-icon
https://stackoverflow.com/questions/56900946/how-to-detect-outgoing-call-number-in-android-q

Refarences for implementation:

https://stackoverflow.com/questions/8398514/hide-application-icon
https://stackoverflow.com/questions/56900946/how-to-detect-outgoing-call-number-in-android-q

So I was implementing the feature when I found out that google is really strict about call permissions.
I don't know if we can really do this feature. Need to do some more research.

So I was implementing the feature when I found out that google is really strict about call permissions.
I don't know if we can really do this feature. Need to do some more research.

@fux0r2009 do you have any other idea on how to open the app, when the icon is hidden?

@fux0r2009 do you have any other idea on how to open the app, when the icon is hidden?

So I was implementing the feature when I found out that google is really strict about call permissions.
I don't know if we can really do this feature. Need to do some more research.

Turns out there is a way with a static secrect code like ##123## without any permission.

So I was implementing the feature when I found out that google is really strict about call permissions.
I don't know if we can really do this feature. Need to do some more research.

Turns out there is a way with a static secrect code like ##123## without any permission.

So I was implementing the feature when I found out that google is really strict about call permissions.
I don't know if we can really do this feature. Need to do some more research.

Turns out there is a way with a static secrect code like '##123##' without any permission.

Yes that's what I had in mind and it's also what is used in other apps with the same feature! looking forward to the next release :)

So I was implementing the feature when I found out that google is really strict about call permissions.
I don't know if we can really do this feature. Need to do some more research.

Turns out there is a way with a static secrect code like '##123##' without any permission.

Yes that's what I had in mind and it's also what is used in other apps with the same feature! looking forward to the next release :)

@fux0r2009 do you have any other idea on how to open the app, when the icon is hidden?

You could also make a feature to disguise app icon with a different generic app name like File Explorer or QR code Scanner... etc for those who don't want to hide the app but just make it less obvious. just a proposition as I personally still prefer to completely hide the app but some people like the other way of disguising the icon too.  anyway It not as important as hiding so up to you @leonlatsch

@fux0r2009 do you have any other idea on how to open the app, when the icon is hidden?

You could also make a feature to disguise app icon with a different generic app name like File Explorer or QR code Scanner... etc for those who don't want to hide the app but just make it less obvious. just a proposition as I personally still prefer to completely hide the app but some people like the other way of disguising the icon too.  anyway It not as important as hiding so up to you @leonlatsch

@fux0r2009 do you have any other idea on how to open the app, when the icon is hidden?

You could also make a feature to disguise app icon with a different generic app name like File Explorer or QR code Scanner... etc for those who don't want to hide the app but just make it less obvious. just a proposition as I personally still prefer to completely hide the app but some people like the other way of disguising the icon too. anyway It not as important as hiding so up to you @leonlatsch

This might be the way to go. As of Android 9, the app icon will always be shown, unless its a system app. Source

@fux0r2009 do you have any other idea on how to open the app, when the icon is hidden?

You could also make a feature to disguise app icon with a different generic app name like File Explorer or QR code Scanner... etc for those who don't want to hide the app but just make it less obvious. just a proposition as I personally still prefer to completely hide the app but some people like the other way of disguising the icon too. anyway It not as important as hiding so up to you @leonlatsch

This might be the way to go. As of Android 9, the app icon will always be shown, unless its a system app. Source

@fux0r2009 do you have any other idea on how to open the app, when the icon is hidden?

You could also make a feature to disguise app icon with a different generic app name like File Explorer or QR code Scanner... etc for those who don't want to hide the app but just make it less obvious. just a proposition as I personally still prefer to completely hide the app but some people like the other way of disguising the icon too. anyway It not as important as hiding so up to you @leonlatsch

This might be the way to go. As of Android 9, the app icon will always be shown, unless its a system app. Source

I see well disguising the app is better than nothing :)

@fux0r2009 do you have any other idea on how to open the app, when the icon is hidden?

You could also make a feature to disguise app icon with a different generic app name like File Explorer or QR code Scanner... etc for those who don't want to hide the app but just make it less obvious. just a proposition as I personally still prefer to completely hide the app but some people like the other way of disguising the icon too. anyway It not as important as hiding so up to you @leonlatsch

This might be the way to go. As of Android 9, the app icon will always be shown, unless its a system app. Source

I see well disguising the app is better than nothing :)


Add DK roskildegymnasium.txt

The official website of the gymnassium / school: https://www.roskilde-gymnasium.dk/

For reference email format for student<EMAIL_ADDRESS>For reference email format for student<EMAIL_ADDRESS>

Example course with informatics & programming classes https://www.roskilde-gymnasium.dk/bliv-elev/stx/politik-og-samfund/12-politik-og-samfund/

Hi, turns out we have another mail domain: roskilde-gym.dk
Example<EMAIL_ADDRESS>

@Seba244c lib/domains/dk/roskilde-gym.dk: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@philipto ups, that was a mistake. It matches the format now.


Configuration file seems not to be loading

Action is not loading configuration options
I've placed my configuration at .github/config/rubocop_linter_action.yml with:
rubocop_version: '0.79.0'

rubocop_extensions:
  - 'rubocop-rspec': 'latest'

check_scope: 'modified'

and got this info from build:
Successfully installed rubocop-0.80.0
8 gems installed
cannot load such file -- rubocop-rspec

Could You please help me I don't know what I'm doing wrong with this one.

My first guess is that you aren’t requiring that extension in your .rubocop.yml

In my .rubocop.yml file I got:
require: rubocop-rspec

Rails:
  Enabled: true

also in configuration I specified rubocop version to '0.79.0' but it is instaling '0.80.0'

@DominikAlberski can you create a reproduction? Click here and add me as a contributor to it?

Closing due to no response.


Model->update() has no effect if model does not include ID

Laravel Version
10.26.0
PHP Version
8.1
Database Driver & Version
No response
Description

I retrieve a collection (of users) and took only the "email" and "name" fields. I just need these for an API operation.
Afterwards, I wanted to update a third field using $model->update([$field => $value]).
The code ran without error, but the field never updated. I took me a while to notice it was caused by the absence of the id field.

I am not sure if this has been addressed/discussed before, but at least some error message will be useful.
Steps To Reproduce

Grab a collection:

$users = User::where($someCondition)->get(["name", "email"]);

Do mass update:

$users->each(fn($user) => $user->update(["processed" => true]));  // No effect, no warning.

Hours looking at the $fillable array, database migration, checking column names 😅

#29194

Hi there,
Thanks for reporting the problem you are encountering, but it looks like this is a question which may be better suited for a support channel. We only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repository you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

Thanks for your reply.
I do feel this can be improve because this special case is not documented, unlike more common gotchas such as $fillable/$guarded. Perhaps it could be added to the documentation of $model->get() parameters.

The query that is executed if the id is missing is :
PDOStatement {#8482
  +queryString: "update `users` set `name` = ?, `users`.`updated_at` = ? where `id` is null"
}

Thank you @lk77 for your investigation. That's very useful to know.
Suppose we are using a Model that does not use ID (but UUID, for example) as the default primary key. Then at least running update on those instances will result in an error, won't it?
So, I do feel an error should be thrown when updating Models without supplying the ID field. Or some documentation, like for the $fillable/$guarded property.

@hktang no because it will use whatever field you define as a primaryKey on the model
i think laravel should always select the primaryKey field


Library files not found in web applications after upgrading from 1.0.12 to 2.2.0 (with suggested fix)

After upgrading from v1.0.12 to v2.2.0 in a MVC 5 web application where tesseract wrapper is installed in an assembly other then the web application the library liblept168.dll and libtesseract302.dll can not be found with the error message, "Failed to find library "liblept168.dll" for platform x64.".
The error is caused by the logic in the LibraryLoader.cs file method LoadLibrary which then calls out to CheckCurrentAppDomain. In the CheckCurrentAppDomain it uses the following to search as a base directory
var baseDirectory = Path.GetFullPath(AppDomain.CurrentDomain.BaseDirectory);

The issue is that AppDomain.CurrentDomain.BaseDirectory returns the base directory of the application but with web apps the location of the deployed code dlls will actually be {base directory}\bin and so the files will not be found. The issue is hidden a little because if you had tesseract installed in the main web application assembly then the libraries would be found when debugging as they would be available at base directory because the source files are available but once deployed they would not and the error would show up.
Suggested solution:

Change LoadLibrary method as per below
if (dllHandle == IntPtr.Zero)
dllHandle = CheckCurrentAppDomain(fileName, platformName);
if (dllHandle == IntPtr.Zero)
dllHandle = CheckCurrentAppDomainBin(fileName, platformName);
if (dllHandle == IntPtr.Zero)
dllHandle = CheckWorkingDirecotry(fileName, platformName);

Add the new method CheckCurrentAppDomainBin to the class as below
private IntPtr CheckCurrentAppDomainBin(string fileName, string platformName)
{
var baseDirectory = Path.Combine(Path.GetFullPath(AppDomain.CurrentDomain.BaseDirectory), "bin");
return InternalLoadLibrary(baseDirectory, platformName, fileName);
}

In an ideal world you would probably know that it's a web application and add the bin location if it was rather than have two calls almost identical however I'm not sure you would really want to introduce something like a HttpContext.Current != null check and drag in the web libraries to the helper when this simple fix will do the trick.
Let me know your thoughts.
Chris

I'll have a think about this. We did use to detect if it was a web app
using the httpcontext trick you previously mentioned however I was never
happy with this approach.
Another concern is that asp.net / iis creates a shadow copy of the managed
dlls but not the unmanaged ones which means the approach you recommended
may not work. I'm assuming the app domain base directory will return the
shadowed directory in this case though I haven't tested this.
Another approach might be to allow users to specify a directory in which to
search. This would also allow more advanced use cases such as storing the
tesseract dlls as embedded resources and extracting them to the app data
directory on init.
On 8 Mar 2015 08:50, "chtay1"<EMAIL_ADDRESS>wrote:

After upgrading from v1.0.12 to v2.2.0 in a MVC 5 web application where
tesseract wrapper is installed in an assembly other then the web
application the library liblept168.dll and libtesseract302.dll can not be
found with the error message, "Failed to find library "liblept168.dll" for
platform x64.".
The error is caused by the logic in the LibraryLoader.cs file method
LoadLibrary which then calls out to CheckCurrentAppDomain. In the
CheckCurrentAppDomain it uses the following to search as a base directory
var baseDirectory = Path.GetFullPath(AppDomain.CurrentDomain.BaseDirectory);
The issue is that AppDomain.CurrentDomain.BaseDirectory returns the base
directory of the application but with web apps the location of the deployed
code dlls will actually be {base directory}\bin and so the files will not
be found. The issue is hidden a little because if you had tesseract
installed in the main web application assembly then the libraries would be
found when debugging as they would be available at base directory because
the source files are available but once deployed they would not and the
error would show up.
Suggested solution:

Change LoadLibrary method as per below
if (dllHandle == IntPtr.Zero)
dllHandle = CheckCurrentAppDomain(fileName, platformName);
if (dllHandle == IntPtr.Zero)
dllHandle = CheckCurrentAppDomainBin(fileName, platformName);
if (dllHandle == IntPtr.Zero)
dllHandle = CheckWorkingDirecotry(fileName, platformName);
2.
Add the new method CheckCurrentAppDomainBin to the class as below
private IntPtr CheckCurrentAppDomainBin(string fileName, string
platformName)
{
var baseDirectory =
Path.Combine(Path.GetFullPath(AppDomain.CurrentDomain.BaseDirectory),
"bin");
return InternalLoadLibrary(baseDirectory, platformName, fileName);
}
In an ideal world you would probably know that it's a web application and
add the bin location if it was rather than have two calls almost identical
however I'm not sure you would really want to introduce something like a
HttpContext.Current != null check and drag in the web libraries to the
helper when this simple fix will do the trick.
Let me know your thoughts.
Chris
—
Reply to this email directly or view it on GitHub
https://github.com/charlesw/tesseract/issues/157.

I'd be happy supplying the location, as you said it does add flexibility to support other scenarios as well.  I guess worst case scenario of the approach I suggested is that it doesn't find the library and moves on which is wasted time but not too much of an issue. In the meantime I'm going to deploy my application using my cloned version of the repository with the fix in place.
FYI, on the solution, in Tesseract.csproj, Tesseract.Tests.csproj and Tesseract.Tests.Console.csproj the properties\AssemblyVersionInfo.cs file is missing from the git repository. Is this deliberate?

Yes, it's dynamically generated by the prepare batch script. Probably
should add this to the vs projects as a prebuild step.
On 9 Mar 2015 12:30, "chtay1"<EMAIL_ADDRESS>wrote:

I'd be happy supplying the location, as you said it does add flexibility
to support other scenarios as well. I guess worst case scenario of the
approach I suggested is that it doesn't find the library and moves on which
is wasted time but not too much of an issue. In the meantime I'm going to
deploy my application using my cloned version of the repository with the
fix in place.
FYI, on the solution, in Tesseract.csproj, Tesseract.Tests.csproj and
Tesseract.Tests.Console.csproj the properties\AssemblyVersionInfo.cs file
is missing from the git repository. Is this deliberate?
—
Reply to this email directly or view it on GitHub
https://github.com/charlesw/tesseract/issues/157#issuecomment-77787889.

Yep, sorry didn't look at the full build process, was just in a hurry debugging so I could get the upgrade finished.

Okay, I've implemented both approaches (checking in the bin directory and allowing users to specify a custom path). This will be in the next release which should be out in the next hour :)

Thanks, I've installed and tested. Working fine.


feat: User-specified index creation

Allow users to specify their own index weightings for individual variables or categories of variables. Allow users to categorize variables.

Whoever picks this up, the toxpiR docs are very helpful. The most important thing to understand is slices & how they combine metrics (="allow users to combine variables")


Refactor snapshotCacheExpiryTimeoutMs to maximumCacheDurationMs

An refactor made in relation to this comment https://github.com/microsoft/FluidFramework/pull/8763#discussion_r786210145

Timing of the original change means it already made its way into the 0.55 branch.  Since it's a new property and shouldn't be used externally yet I guess you can get away with making this change in both the release branch and main and having a patch of driver-definitions?


React to #158 (DataToken for GetVirtualPath) in Routing

@rynowak, @Eilon
This is the PR for the react to https://github.com/aspnet/Routing/pull/158.
Thanks

:shipit:

Looks good to me too.

https://github.com/aspnet/Mvc/commit/27bdec40a6a26c83e8219b94f99c692f15dd126f
and
3693002b0234a7df2c08be694b1514426e057c16


Open a file by Ctrl+Click-ing on the file location string

Description of the new feature/enhancement
If there is a file location (absolute or relative) string on terminal, ctrl+clicking on that location string will open the file in the default application for that file type.

Proposed technical implementation details (optional)

Thanks for the suggestion! Please make sure to search for duplicates when filing new issues. This is already being tracked by another issue on our repo - please refer to #574 for more discussion.
/dup #574


Revert "Add support for iterables in the serializer (#991)"

Fixes #1028 "After 2.3.2->2.4.0 update, Behat stops on the first feature involving an exception"
Reverts #991 "Use is_iterable instead of is_array" and adds a non-regression test

About broken CI, why is b586900f6912a69c7743dbed268981b10500e586 only on develop and cd80b9b681b92716e807a08a19a84d0c3355a79a only on hazat/am (not master)?

(rebased on top of master updated with #1032, CI green)

@ste93cry: Thank you for the careful review. Does it look good now?

It does! Thank you for your effort, LGTM

We forgot the changelog entry :(

Ops, I will never stop forgetting to update the CHANGELOG with each PR :disappointed: Anyway, I will do it once we prepare the release


Log output in json format

Structured json log may be useful for search and analysis in AWS CloudWatch and GCP Logging.
It would be nice to be able to output logs in json format by giving the option, like --json.
$ skeema lint
2023-10-05 12:41:43 [INFO]  Linting /goo/bar

$ skeema lint --json
{"time": "2023-10-05 12:41:43", "level": "INFO", "message": "Linting /goo/bar"}

Thank you for the feature request! I agree that JSON output in Skeema could be generally useful, but I am still undecided on some of the specifics. One main open question is where/how JSON should appear:

a) STDERR structured logging in JSON format (just time/level/message for logs, as in your example above)
b) STDOUT structured output in JSON format (automation-friendly output for schema diffs and linter annotations, following stable JSON Schema definitions)
c) perhaps both of the above (controlled by separate options?)

I do have a couple concerns regarding offering it for STDERR logs:
Timing: a move to JSON log output should really coincide with a move to Go 1.21's log/slog package in the stdlib. However since we still support Go 1.20, we can't use log/slog here until Go 1.22 is released, presumably in 4 to 5 months.
Automation uses: Processing JSON logging is easier for automation of course, but the current STDERR logging is geared for human consumption, and we don't make any guarantees that the STDERR log messages remain unchanged between releases. So if users try to build automation around the message field of any new JSON format logging, some users will effectively just pin their Skeema version forever to avoid upgrade problems, which is not ideal.
Some of the design difficulty comes from Skeema being positioned as both a manual CLI as well as an automation/CI/pipeline-friendly tool. There are some plans for building out a separate automation-specific product, but it's still early-stage.
Anyway, that all said -- if you could expand on your specific use-case / what concrete problems you'd like to solve using JSON logging, that would be helpful in weighing the various options on how to proceed. Thanks!

Hi @hayashikun, just following up again. Can you please expand on your specific use-case / what concrete problems you'd like to solve using JSON logging?

@evanelias
I appreciate your consideration.
I'm facing a problem: GCP Logging shows unintended behavior when the log is not in JSON format.
e.g., The severity of the message shown after the skeema push is assigned to ERROR, but it is not an error.
This may be due to this INFO message being sent as STDERR.

If the log output from skeema is in JSON, it can be processed to GCP parsable format by jq.
I would prefer structured output in JSON format for both STDERR and STDOUT.
Structured output is easier to search and analyze in a logging platform, even for STDOUT.
Regarding the timing, using log/slog is a good idea, and it is not a very urgent issue for me.
It would be nice to have support for JSON format when moving to log/slog.

Thanks, that's good context.
As a temporary workaround for now, I believe you can use a custom fluentd configuration to process unstructured logs on GCP. Or you could pipe Skeema's STDERR to a small custom script which converts each line to JSON.
Definitely agree that built-in support would be better, but this may end up being a bit longer-term while the separate automation-specific product takes shape...


New release

Is it possible to release a new version please? I want to start using the recent changes for supporting pyproject.yaml.

I presume this is blocked on https://github.com/timothycrosley/isort/issues/763 at the moment.

Status？

Before you release the new version, you might want to know that this feature isn't working well:
rr-@tornado:~/src/maintained/bubblesub$ cat pyproject.toml
[tool.black]
line-length = 79
py36 = true

[tool.isort]
known_third_party = mpv
multi_line_output = 3
include_trailing_comma = true
rr-@tornado:~/src/maintained/bubblesub$ isort -rc -y **/*.py

Running isort recognizes mpv as third party module. This is good.
rr-@tornado:~/src/maintained/bubblesub$ black .
Usage: black [OPTIONS] [SRC]...

Error: Error reading configuration file: invalid literal for int() with base 10: 'mpv'

Whoops! Looks like this isn't a valid TOML file. Let's fix this.
rr-@tornado:~/src/maintained/bubblesub$ sed -i 's/mpv/"mpv"/' pyproject.toml
rr-@tornado:~/src/maintained/bubblesub$ black .
All done! ✨ 🍰 ✨
113 files left unchanged.

So far so good…
rr-@tornado:~/src/maintained/bubblesub$ isort -rc -y **/*.py
Fixing /home/rr-/src/maintained/bubblesub/bubblesub/api/media/media.py
Fixing /home/rr-/src/maintained/bubblesub/bubblesub/api/media/video.py

…but now isort incorrectly looks for import "mpv" statements.
rr-@tornado:~/src/maintained/bubblesub$ cat pyproject.toml
[tool.black]
line-length = 79
py36 = true

[tool.isort]
known_third_party = "mpv"
multi_line_output = 3
include_trailing_comma = true
rr-@tornado:~/src/maintained/bubblesub$

@jdufresne Thanks for https://github.com/timothycrosley/isort/pull/798! Can we please get a new release now?

@ofek While I can help out by reviewing and merging PRs, I do not have permission on PyPI to submit new releases. That will require @timothycrosley

@jdufresne Ah I see. Btw, does the config loader traverse up through the directory tree just like black?

Hi, any progress on this?

Hi Everyone!
Good news:
https://github.com/timothycrosley/isort/releases/tag/4.3.5
4.3.5 is now officially out and includes many of the improvements and fixes you have been waiting for :crossed_fingers: without any major regressions. Officially this is the last Python2 release, however, if there does happen to be a large number of regressions I'm open to doing some hotfixes under 4.3.x, but am fully unwilling to port backwards any new features.
I'm deeply sorry for the unreasonably long delay to wrap up all these awesome improvements made by users and contributors to isort. I want everyone to know that I'll be working diligently to open deploy access to more contributors to avoid this from occurring in the future.
Thank you!
~Timothy

@timothycrosley We all appreciate this very much, thank you!

I suggest a major version bump for the drop of python2 support.

@das7pad Good call! I'll update the changelog to list it as the 5.0.0 release. I want to combine the next release with strict profile support - so that we can do things like ensure full-strict coverage with projects that use black with an isort black profile.

A quick update on this: auto PYPI deploy has been set up against the master branch, which enables others to also do deploys starting with @jdufresne who has done a tremendous amount of work and a great job acting as a co-maintainer - even without all the tools to fully perform that role (the ability to make releases!).  I'd like to open it up to more core contributors in the future, and this is the first step to enable that to happen. If this is something that interests you, please let me know.
Due to at least in part, the large time lapse since the last release, it took many hotfix releases before isort felt stable based on incoming issues. However, I feel it is now stable enough to start planning completely the first 3.x only release. I'm going to let this work week pass by, and if no major new issues are reported I'll aim to deploy the Python3 release shortly after.
Thanks again, everyone!
~Timothy


Heart  disease :  | 1. Data preparation And Pipeline Creation

Hi @TAHIR0110 I would like to do further work on this Dataset on the  EDA file These are proposed steps :
1.Imputing Outliers and NULL values
2.Normalization of skewed values and One Hot coding Text attributes
3.Comparing KNN.Random Forest and SVC
4.Validatiing the Evaluated models on the validation set using metrics like accuracy, precision, recall, and F1 score
5.Including visualizations (e.g., training curves, confusion matrices) to better illustration of the model performance.
Further :
Creation of Automated Pipeline and Custom Tranformer For the Datset each steps proposed .

HI @TAHIR0110 can you also specify what other datset you would like same .

Hi @TAHIR0110, I am a new contributor can you please guide me and can you assign me something. I have made some projects on image classification and also data prediction.

@akv2011 assigned

will you please asign it to me?

Hi @TAHIR0110 @akv2011 , I have been working on Deep Learning for quite a long time now, kindly assign this issue to me. I will love to work on it.


Make calcChoiceProbs more accurate, and make simultaneous evalution f…

…aster (and more accurate).
The commit message should say it. It makes the simultaneous evaluation faster and more accurate, and the choice prob evaluation more accurate on its own.
Improvements where needed for https://github.com/econ-ark/REMARK/pull/9 to fully replicate DCEGM paper.

@mnwhite would it be possible for you to look at these changes? It'd probably make sense for them to make it into HARK before the next version, as the way they're calculated here is quite important to be able to replicate the DCEGM paper for example. With the old way of handling overflow, DCEGM figures do not match those published (and the fault is ours)
cc @shaunagm as I feel like this is slightly release blocking

Yes, I think these should be in the release -- but I thought they'd already
been merged into the master branch?
On Fri, Apr 5, 2019 at 5:05 AM Patrick Kofod Mogensen <
<EMAIL_ADDRESS>wrote:

@mnwhite https://github.com/mnwhite would it be possible for you to
look at these changes? It'd probably make sense for them to make it into
HARK before the next version, as the way they're calculated here is quite
important to be able to replicate the DCEGM paper for example. With the old
way of handling overflow, DCEGM figures do not match those published (and
the fault is ours)
cc @shaunagm https://github.com/shaunagm as I feel like this is
slightly release blocking
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/242#issuecomment-480203839, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABQlf3_h5b-IzCgkzN8dqZ_azqgYiliVks5vdxHvgaJpZM4b76ja
.

--

Chris Carroll

Yes, I think these should be in the release -- but I thought they'd already been merged into the master branch?

Some version of the collection of functions are there, but this PR hasn't been merged, so the improvements are not.

Ack, I'm in Bordeaux right now, flying to Dublin momentarily. I'm sorry
this keeps slipping. I will try to merge in tonight (European).
On Fri, Apr 5, 2019, 10:05 AM Patrick Kofod Mogensen <
<EMAIL_ADDRESS>wrote:

@mnwhite https://github.com/mnwhite would it be possible for you to
look at these changes? It'd probably make sense for them to make it into
HARK before the next version, as the way they're calculated here is quite
important to be able to replicate the DCEGM paper for example. With the old
way of handling overflow, DCEGM figures do not match those published (and
the fault is ours)
cc @shaunagm https://github.com/shaunagm as I feel like this is
slightly release blocking
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/242#issuecomment-480203839, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANUQFaCfI8DSscIA2PmJYMly1JGTA_bVks5vdxHwgaJpZM4b76ja
.

The loop at 3395 can be done with an indexing scheme (and no loop).  But (from my own experience with this exact step) it makes the code harder to read and doesn't actually result in a speedup-- the number of choices is usually like 2-5, so it's not much of a loop to avoid.
Merging now.

@pkofod @mnwhite - can one of you suggest a one line summary for the release notes?

"Improve accuracy and performance of functions for evaluating the integrated value function and conditional choice probabilities for models with extreme value type I taste shocks."


Make "system:" prefixed ClusterRoles and ClusterRoleBindings optionally visible

What would you like to be added:
Make system: prefixed ClusterRoles and ClusterRoleBindings optionally visible under Access Control. Currently they are filtered out with no obvious way to make them visible.
Why is this needed:
Many users and service accounts are mapped to system: prefixed ClusterRoles, and it useful to be able to view them to see what they do and do not allow. Likewise, it is useful to see what is already mapped to those roles.

You can already see those by displaying roles and bindings from all namespaces.

@nevalla Thank you for pointing out that some ClusterRoles and ClusterRoleBindings are visible. I have updated the issue accordingly.


Revert "Add a default delete override for the interface"

This reverts commit e8a61d63b5a6db0f81b688481a9485d412e5a41e.

Just talked with @vjpai and we are going to keep this and fix the tests...
Thanks for doing this. Closing for now.


pageserver: test_statvfs_pressure_usage instability

test_statvfs_pressure_usage and test_statvfs_pressure_min_avail_bytes are among the most frequent flaky failures.

There seems to be two modes:

needing to use global LRU

pg14 at least

evicting too much

pg14
pg16

Downloads are caused by initial logical size calculation, it's a bit of a toctou fight. I'll add a wait until we have calculated all timelines sizes.


prioritize_no_change GUI option

Describe the solution you'd like
https://github.com/bitcoin/bitcoin/pull/23475 is likely to be merged soon.  A GUI PR is a good followup.  Just filling this so it is not forgotten.
Additional context
A tickbox option under the send page.

I don't see any sense in this, I think it would be viable to keep the addresses to be available for consultation, it makes no sense to delete the addresses since they are part of the wallet, even if they are no longer used.

I don't understand your point.  Did you mean to comment on this issue?


fix: could not find package gitconfig-mode

Fixes #5668

@catap oh, I don't know it has been duplicated with yours, going to cancel the PR right now


Automator: update proxy@master in istio/istio@master

Generated by Automator - 2021-11-11T21:16:18+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-multicluster-tests_istio
4882cec0a1ed7b5ddecc38cd36a3874879408ece
link
true
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-pilot-multicluster-tests_istio


Implement counts record on Overview Tab.

Increased number of heroes in order to make the page alignment look cleaner.
Logic may not be the best, so definitely looking for some review/optimization. (had some difficulty understanding the transforms that were occurring to the data and why certain fields weren't accessible at certain stages in processing)
Container title probably deserves a better name, or at least it's own variable if we decide to change it in the future.

tip: If you make your branch on this repo it's easier for the rest of us to clone without having to add your repo to remote :)

General comments:

Currently seems a bit buried (user has to scroll down a lot).  Could we maybe move to a three column layout?
Probably should separate the sections (radiant/dire, patch, region, mode)
Context links to the matches filter page when number is clicked on (like heroes/players?)  More generally this could be done for the rest of the counts

Ah, okay. I'll create the branches on the main repo from now on. Will make my workflow easier too :)
The three column layout makes things look super busy and will only get more filled if the counts get split into individual tables. This layout is also pushing it to show content without horizontal scrolling (below 1540px the counts would need to be pushed down)

Maybe we want to represent this data similar to the "Summary" section?

Currently seems a bit buried (user has to scroll down a lot). Could we maybe move to a three column layout?

I think that 3 columns is bad

Context links to the matches filter page when number is clicked on (like heroes/players?) More generally this could be done for the rest of the counts
Instead of a table, maybe something a little more eye-catching? c3 has gauge charts: http://c3js.org/samples/chart_gauge.html or maybe donut charts: http://c3js.org/samples/chart_donut.html

👍

Yeah, maybe another row under the summary section would be good

Still working out some parts of this, but before I go much further I'd like some feedback.

Maybe we could shorten the last 20 games summary and put them both on one line?  IMO having two additional rows on top is pushing down the "primary" content a little.

I wonder if it would make sense to put this section under the existing content, rather than over.

replaced by #1635


Create contents

Dream Guide TOC is approved.
Even though this is approved, please check again and discuss it with the group as there might be changes and so we can do collective decisions.

[ ] 1. Create your markdown notes inside the notes folder.
[ ] 2. Create a per chapter markdown notes that follows the name of the chapter found from this Dream Guide TOC for example, one chapter for the Introduction, another for specialization and so on and so forth.

Hello @SuzyQuant
1. Introduction
2. Mission statement
3. How We Work:
• What We Do
• Who We Work With
• Why We are Good At It
4. The Dream Play Book
5. The five foundations of Ikigai
6. How Dream Hive can help you
7. Kaizen
8. SuperNOVA
9. Reasons to work together
10. How To Do What You Love For a Living
○ Passion
○ Excitement to work
11. Goals

Ikigai
Dream System
• Dream Wave
○ 7 D's of Dream Unalome
* Discover
* Devote
* Define
* Develop
* Deliver
* Divide
* Direct
• ORBITAL
• SUPER NOVA
• STAR
Dream Playbook
7 Steps to perfection
Vision Board
Dream Sourcing
• Awaken The OM
• Feel Tantric Flow
• Light The Path
• Create Divine

@SuzyQuant, I have shared two different TOCs of Dream Guide and Focus Grading System.
The one I have shared here and the one that you shared are totally different. Can you tell me which one we are going to go with? As the one that I shared are straight from OneNote and Mr. Hackney approve them.


Methods with pointer to pointer arguments using old signature

A commit was made to better support pointer to pointer arguments. This is commit https://github.com/progrium/macdriver/commit/6a98f82290e53bc5d82cb59436644f705156d7cb. Running 'go generate ./macos/appkit' would update all the methods in a selected folder to use the new function signature.
Example:
// Before running 'go generate ./macos/appkit'
func (n_ Nib) InstantiateWithOwnerTopLevelObjects(owner objc.IObject, topLevelObjects []objc.IObject) bool {
// After running 'go generate ./macos/appkit'
func (n_ Nib) InstantiateWithOwnerTopLevelObjects(owner objc.IObject, topLevelObjects unsafe.Pointer) bool {
The '[]objc.IObject' type is replaced with an 'unsafe.Pointer' type.
I ask that the Macdriver repo have all its supported frameworks updated by running 'go generate ./macos/framework-name-here'
Thank you.

Or is this fine? I will run generate soon either way.

This touched a lot, but it's in. All examples are working. Let me know if it's what you expected so I can make any fixes before finally releasing 0.5.0 ... thanks for all your help @programmingkidx


Handle precise orbits

Create a precise orbit data struct in the SARprocessing Module
Create a loader for sentinel 1 precise orbits.
Create a function create a orbit interpolate function based on the precise orbits data

The orbit interpolation function should be another issue.


MGRS and the South Pole

Please fill out as much known and relevant information as possible.
Version Information:

MGRS Java Version: 212
MGRS Java Source: (e.g. Central Repository, Release, Source Code Build) maven
IDE Name & Version: Android Studio
Maven Version: (mvn -version)
Java Version: (java -version) 11
Platform & OS: Android 7+
Other Relevant Libraries:

Expected Results:

What did you expect to happen?

Create an MGRS object near the Poles. Example: AQE 33377 29769
Using the GEOTrans utility/source code, this value can be parsed as valid MGRS
Observed Results:

What happened instead?
MGRS.parse() throws a ParseException
How often does this occur?
Anytime a valid location near the poles is used, it fails the regex in use ^(\\d{1,2})([C-HJ-NP-X])(?:([A-HJ-NP-Z][A-HJ-NP-V])((\\d{2}){0,5}))?$

Output:

Any logs, errors, or output messages?
ParseException("Invaliid MGRS")

Steps to Reproduce:

Try to parse an MGRS string that represents a location at the poles
ParseException

Relevant Code:
MGRS mgrs = MGRS.parse("AQE3337729769");

Test Files:

Links to any files needed for testing?

Additional Information:

Any additional configuration, data, or information that might help with the issue?

MGRS uses UPS project to cover the poles, where A|B cover the South Pole, and Y|Z cover the North Pole. I didn't see references to ups in the source for this project. Are there plans to add support for the poles?

Possibly related to an issue I encountered with mgrs-android which uses this repo:
https://github.com/ngageoint/mgrs-android/issues/3
It would appear that the polar regions are not being handled correctly by this library, which is unfortunate

These libraries don't currently support polar regions.  PRs are welcome if you would like to take a stab at it.

If this is not a bug (i.e. it is expected behaviour), this behaviour should be documented.