SFC setup script does not make imported components available to template with TypeScript + Pug

Version
3.2.2
Reproduction link
https://github.com/spenserblack/setup-script-typescript-and-pug-issue
Steps to reproduce

Create an SFC with <template lang="pug"> and <script setup lang="ts">
Import a component
Use component in template

What is expected?
Imported component is rendered.
What is actually happening?
Imported component is not rendered, receive warning
[Vue warn]: Invalid vnode type when creating vnode: undefined

This seems to happen specifically when TypeScript and Pug are used together. As you can see in the reproduction, when <template> <script setup lang="ts">, component renders, and when <template lang="pug"> <script setup>, component renders. Only <template lang="pug"> <script setup lang="ts"> seems to cause the issue.

I have facing a similar issue with vuedraggable ^4.0.3

Regardless this little problem, it is still one of the best feature Vue provides, would be even better if we could have a global defineEmits/defineProps etc. without needing import every single time. 😀

would be even better if we could have a global defineEmits/defineProps etc. without needing import every single time.

The implementation in 3.2 is providing exactly that. Are you already on 3.2.1 ?

would be even better if we could have a global defineEmits/defineProps etc. without needing import every single time.

The implementation in 3.2 is providing exactly that. Are you already on 3.2.1 ?

I am using Vue 3.2.2 and Vue cli 4.5.13, just tried to do it without defineEmits, however it shows an error

Last night I just keep converting all files to

That's an eslint errror. You need to tell eslint that these globals exist.

I have facing a similar issue with vuedraggable ^4.0.3

Regardless this little problem, it is still one of the best feature Vue provides, would be even better if we could have a global defineEmits/defineProps etc. without needing import every single time. 😀

I've encountered the similar issue with vue-draggable.
But I've found a workaround. Make your import Uppercase!
I think it's a bug introduced in 3.2.0-beta.5

I have facing a similar issue with vuedraggable ^4.0.3

Regardless this little problem, it is still one of the best feature Vue provides, would be even better if we could have a global defineEmits/defineProps etc. without needing import every single time. 😀

I've encountered the similar issue with vue-draggable.
But I've found a workaround. Make your import Uppercase!
I think it's a bug introduced in 3.2.0-beta.5

Verified, after changing to the upper case, it works for me.  Thanks!

same issue +1

Results of preliminary investigation, the AST that lives in the resolveTemplateUsageCheckString function represents a template that has not been processed by pug, so the imported component is checked as not being used in the template, i.e. isUsedInTemplate is always false.

In my project this problem only occurs in dev mode and work without problem in production build

I wanted to give my 2 cents on this issue since I was about to create one myself.
This issue does not only occur when using script <setup> TypeScript + Pug, for me this happens when using it with just Typescript (<script setup lang="ts">), without Typescript it works like expected.
Error: [Vue warn]: Invalid vnode type when creating vnode: undefined.
Vue version: 3.2.2

Same issue in new scaffold with yarn create vite + ts but no Pug used.
Error: [Vue warn]: Invalid vnode type when creating vnode: undefined.
Vue version: 3.2.x

@cutsin Did you test it with all 3.2.x versions? Downgrading to version 3.2.1 resolved the issue for me, while downgrading to 3.2.0 did not.

@joostdelange if your issue is not related to pug, please open a separate issue with your own reproduction since they are likely not the same problem.

Note: the fix skips the import usage check since with a custom lang, @vue/compiler-sfc won't be able to reliably analyze the template to determine whether an import is used in the template or not. This means if you are using Pug or other template pre-processors with <script setup> + TS, you will need to explicitly import types with import type { xxx } from '...' to avoid them from being returned from setup() during dev.

Thank you for the fix! :tada:

This means if you are using Pug or other template pre-processors with <script setup> + TS, you will need to explicitly import types with import type { xxx } from '...' to avoid them from being returned from setup() during dev.

Because of this, would it be recommended to avoid template pre-processors when using <script setup> + TS?


[FLINK-25601][state backends][config] Update the 'state.backend' option

What is the purpose of the change
The value and comments of 'state.backend' in flink-conf.yaml is deprecated.
Brief change log

Change the 'state.backend' option to new value

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (no)

@Myasuka Thanks Yun Tang merge into master. Could I close this PR?

@deadwind4 , sorry it's my bad to forgect to trigger the auto closing this PR in the commit message.
Let's close this now.


Sum balance overflow

Describe the bug
There is no range check in the circuit for the sum of balances, which poses a risk of overflow.
Furthermore, since N_BYTES is not exposed in the contract, users must run examples/gen_inclusion_verifier.rs to obtain a warning message about the risk of overflow.
To Reproduce
Steps to reproduce the behavior:
examples/gen_inclusion_proof.rs set N_BYTES to 32 and change one of MBlfbBGI's balance to<PHONE_NUMBER>1839275222246405745257275088548364400416034343698204186575808495617 in csv/entry_16.csv.
A valid proof is generated and pass verification.
Expected behavior
A clear and concise description of what you expected to happen.
Cannot generate valid proof
Additional context
Add any other context about the problem here.
Perhaps we could use a new lookup argument to achieve acceptable performance?
Lasso
https://github.com/privacy-scaling-explorations/halo2/issues/194
https://github.com/DoHoonKim8/halo2-lasso/pull/4

I think that this issue and #14 are basically the same thing and, in my opinion, do not reflect a bug.
We deliberately decided to design the range check inside the circuit such that the only way that it can lead to an overflow is for some combinations between N_BYTES and LEVELS (check the presentation I gave at the beginning at the cohort).
Following the principle that everything that can be verified outside of the circuit should be verified outside the circuit, the user has all the tools (https://summa.gitbook.io/summa/v/1/smart-contract/summa.sol#verifier-contract-validity) to assess if, given the compiled verifier contract, the combination poses a risk of balance overflow. If the combination is fine, we believe there's no any way that a prover can generate a valid proof. Only in that case, I would consider it as an high vulnerability.


Service's description disappears when the ingestion task runs

Affected module
UI and Ingestion
Describe the bug
When a created ingestion for a Database service runs, it deletes the service's description.
To Reproduce

Create a DB service - e.g., MySQL
Create a metadata ingestion
Run the ingestion task
The description disappears

Expected behavior
The description should not be updated by the ingestion.
Version:

OpenMetadata version: 0.11.4

This scenario is already covered on 0.12


Tests start executing before page is loaded

I am using karma-jasmine: 0.3.5. I have tests that run perfectly from a HTML runner that loads jasmine 2.2 but when I run them through karma using karma-jasmine they fail. I have investigated the problem and it seems that the tests start executing before the DOM is ready (which is not the case when run from a HTML runner). After updating the createStartFn function in the adapter.js file to the code below which waits for the DOM ready the tests started passsing.
function createStartFn(karma, jasmineEnv) {
  // This function will be assigned to `window.__karma__.start`:
  return function () {
    $(function () {
      jasmineEnv = jasmineEnv || window.jasmine.getEnv();

      jasmineEnv.addReporter(new KarmaReporter(karma, jasmineEnv));
      jasmineEnv.execute();
    });
  };
}

Look at the discussion about the issue here

Guys should we wait event DOMContentLoaded or load before starting tests?
\cc @karma-runner/contributors

Yes please :-) :+1:

Yes please

+1
Or karma can be configured to not start automatically.

+1

+1


Q23 Initial Harness Design

Initially, I was going to try RapidHarness but the projects are limited to 30 connections. Altium is adding a harness design feature as well so I thought I would start with this project and transition over if we can get full access.
NOTE: This will only be merged once we are happy with the design and software CAD tools used. The initial commits will be to just capture what we have so manufacturing can start in a timely manner.

TODO

Add a gear position sensor to this schematic (will go to an analog pin on the ECU)
Verify Shutoff circuit against the rules
Update any BSPD connections based off of "inline breakout box" notes the rules.

Another Note: Map steering wheel to PDM power.

NOTE: also remove 5V supply from the engine speed sensor connection

March 16

Added kill switch signal and return to the dash bulk head. Need this replicated on the front loom. I have placed a female two pin on the kill switch itself.
Placed the driver kill switch in the elec panel harness box.
Joe Will need to help with front loom replication


Provide support for multiple file-types to the new filetype config

FAQ

[X] I have checked the FAQ and it didn't resolve my problem.

Issues

[X] I have checked existing issues and there are no open or closed issues with the same problem.

Neovim Version
NVIM v0.6.1
Minimal reproducible config
No applicable.
Description
The new filetype option appears to be singular in nature as noted in this example from the README:
cmp.setup.filetype('markdown', {
    sources = {
      { name = 'path' },
      { name = 'buffer' },
    }
  })

This replaced:
autocmd FileType markdown lua require('cmp').setup.buffer {
  \   sources = {
  \     { name = 'path' },
  \     { name = 'buffer' },
  \   }
  \ }

In my case, I want to limit nvim-cmp to a list of development filetypes, in autocmd mechanism I did this:
   augroup CmpPluginEvents
     autocmd!
     autocmd FileType dart,eruby,html,javascript,ruby,rust lua require('cmp').setup.buffer(
     \  {sources = {
     \    {name = 'nvim_lsp', keyword_length = 3, max_item_count = 10},
     \    {name = 'vsnip', keyword_length = 3, max_item_count = 5},
     \    {name = 'buffer', keyword_length = 4, max_item_count = 10},
     \  }
     \})
   augroup END

I want to translate to the new mechanism as follows:
cmp.setup.filetype({ "dart", "eruby", "html", "javascript", "ruby", "rust" }, {
...
...

But that is not working since it appears only a single filetype string can be specified as an argument to the new filetype function.
Can the filetype function be enhanced to take a string (as it does now) and a table of strings? If given a table of strings, then iterate over the table as a list of filetypes?
Steps to reproduce
Not applicable.
Expected behavior
Not applicable.
Actual behavior
Not applicable.
Additional context
No response

I'll do it.


Generated code tries to extend an interface. TS2689

Summary
Generated code by the Kiota dotnet tool (1.10.1) tries to extend the interface 'BaseRequestBuilder'.
Environment info
Platform: macOS Sonoma 14.3 (23D56) ARM64
node v20.10.0
npm 10.3.0
kiota dotnet tool version: 1.10.1+fa5da75b8b4f362d821032021bc5bfcf063cb522
dotnet SDK: 8.0.101 (arm64)
Steps to reproduce
The 'Build API clients for TypeScript' Quickstart instructions were used.
npm init npm install -D typescript ts-node npx tsc --init
The following dependencies are added:
kiota-abstractions: ^1.0.0-preview.37
kiota-http-fetchlibrary: ^1.0.0-preview.36
kiota-serialization-form: ^1.0.0-preview.26
kiota-serialization-json: ^1.0.0-preview.37
kiota-serialization-multipart: ^1.0.0-preview.16
kiota-serialization-text: ^1.0.0-preview.34
posts-api.yml was used
Followed by the following command:
kiota generate -l typescript -d posts-api.yml -c PostsClient -o ./client.
Then, the index.ts file from the quickstart was used.
Actual result
Running npx ts-node index.ts resulted in the following error:
<redacted>/tempdir/node_modules/ts-node/src/index.ts:859
    return new TSError(diagnosticText, diagnosticCodes, diagnostics);
           ^
TSError: ⨯ Unable to compile TypeScript:
client/postsClient.ts:14:34 - error TS2689: Cannot extend an interface 'BaseRequestBuilder'. Did you mean 'implements'?

14 export class PostsClient extends BaseRequestBuilder<PostsClient> {
                                    ~~~~~~~~~~~~~~~~~~
client/postsClient.ts:19:45 - error TS2339: Property 'pathParameters' does not exist on type 'PostsClient'.

19         return new PostsRequestBuilder(this.pathParameters, this.requestAdapter);
                                               ~~~~~~~~~~~~~~
client/postsClient.ts:19:66 - error TS2339: Property 'requestAdapter' does not exist on type 'PostsClient'.

19         return new PostsRequestBuilder(this.pathParameters, this.requestAdapter);
                                                                    ~~~~~~~~~~~~~~
client/postsClient.ts:26:50 - error TS7006: Parameter 'x' implicitly has an 'any' type.

26         super({}, requestAdapter, "{+baseurl}", (x, y) => new PostsClient(y));
                                                    ~
client/postsClient.ts:26:53 - error TS7006: Parameter 'y' implicitly has an 'any' type.

26         super({}, requestAdapter, "{+baseurl}", (x, y) => new PostsClient(y));
                                                       ~
client/postsClient.ts:37:14 - error TS2339: Property 'pathParameters' does not exist on type 'PostsClient'.

37         this.pathParameters["baseurl"] = requestAdapter.baseUrl;

Hi @jonasmercelis
Thanks for using kiota and for reaching out.
TypeScript still being an experimental language for kiota, we often ship breaking changes. This will not be the case once typescript becomes a stable language.
This is caused because we released updates to the typescript packages yesterday, and the corresponding changes haven't been released for the generator.
You can either:

wait until thursday afternoon for the next preview of kiota which will have those changes.
install the correct versions of the typescript dependencies (kiota info -l typescript will give you the details)


RefreshControl UI Issue iOS - Takes up space after refresh completes

Description
After refresh completes and the flag is set to false, the RefreshControl hides but the space it occupied remains. I noticed this issue on a FlatList component. I was able to reproduce this on expo iOS. Android sees to work fine.
I also noticed,

The first refresh, you can see the loading indicator on the right side of the screen when it is supposed to be on the top.
If you fresh multiple times, the empty space keeps growing.

Demo of issue
https://github.com/facebook/react-native/assets/38050123/2bc81678-1975-4e6c-89d6-d8cd899edef1
I first reproduced this issue on a non-expo project. Just regular React Native.
React Native Version
0.71.6
Output of npx react-native info
System:
OS: macOS 13.1
CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
Memory: 786.46 MB / 16.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 14.17.0 - /usr/local/bin/node
Yarn: Not Found
npm: 9.6.3 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.11.3 - /Users/jordanwagner/.rvm/gems/ruby-3.0.2/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.4, iOS 16.4, macOS 13.3, tvOS 16.4, watchOS 9.4
Android SDK: Not Found
IDEs:
Android Studio: 2022.1 AI-221.6008.13.2211.9619390
Xcode: 14.3/14E222b - /usr/bin/xcodebuild
Languages:
Java: 11.0.13 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.2.0 => 18.2.0
react-native: ^0.71.6 => 0.71.8
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce

Load app
Swipe down to trigger the RefreshControl component
Wait for it to complete
Repeat steps 2-3 multiple times

Snack, code example, screenshot, or link to a repository
Snack
Demo of issue
https://github.com/facebook/react-native/assets/38050123/6874ce78-9d07-4c62-8776-e562ba8b74fa

I found my issue. I had initially added this line of code which is very incorrect and conflicted with the onRefresh and refreshing properties.
ListHeaderComponent={() => <RefreshControl refreshing={!refreshing} />}


Not fetching 1000 records in Select Dropdown? and showing Blank Page

We want to fetch 1000 Customer Names in Select Dropdown Like this But Whole Page is blank???
$grid->addColumn('Customer_No', 'Customer', 'string', fetch_pairs($mysqli,'SELECT No, Name FROM customers'),true ); in loaddata.php

Any error in the browser's console or in the webserver's error log file ?

Please check these screenshots
Browser Screenshot with Error :- http://prntscr.com/b32k2s
Code Screenshot :- http://prntscr.com/b32kir
(Working Fine)Browser Screenshot after commenting out that 1000s customer record line :- http://prntscr.com/b32lfg
Code Screenshot :- http://prntscr.com/b32llg
Using 'limit 900' its working http://prntscr.com/b32omk
Using 'limit 1000' its not working http://prntscr.com/b32p2z
Can we not use something like this?
$grid->addColumn('Customer_No', 'Customer', 'string', fetch_pairs($mysqli,'SELECT No, Name FROM customers where Name=??'),true );
Thankful to you in advance :)

Sol : Char encoding problem.

Did you find an answer ?


2.0.0RC4

Coverage decreased (-0.6%) to 40.926% when pulling 3099f0077418ead2346e855f7ccf8d5f75f14b15 on lmazuel:RC4 into bda629da37a7e8d224b4a2222149eddf1baf5ac0 on Azure:dev.

Coverage decreased (-0.5%) to 40.989% when pulling 3099f0077418ead2346e855f7ccf8d5f75f14b15 on lmazuel:RC4 into bda629da37a7e8d224b4a2222149eddf1baf5ac0 on Azure:dev.


improve documentation

This PR adds some basic instructions on how to run the KiD_driver to the README.md. It also improves our documentation

bors r+


Provisioning modal  fit and finish

Electron, OS, v1.4.0

@JaskeeratBedi-cmk Are you not signed in to Azure when you get to this page?  The reason it is on the left is because the sign-out and badge is missing.

@GeoffCoxMSFT this is the Hand off to admin scenario, where technically I do not need to signed in. Regardless, I would like the main CTA's to be on the left hand side, where they usually are for our modals..

@VamsiModem I think this is a partial dupe of the sign out issue you are working on already.  When the sign-out is hidden, it seems the buttons move to the left.  We should update the layout/css for each of the footers to make sure the buttons stay on the right.

Shiproom - you guys merged a PR that didn't have a ship room approved issue.  let's follow the process or I'm going to ask you to back these out in the future.
I've assed Shiproom-Approved to keep the bookkeeping correct.  :-(


Clone repo from GitHub to local machine

Install local copy of project from GitHub

[x] Use git clone to pull repo's code
[x] Create SSH keys and set it into GitHub profile section with SSH Keys. Help article
[x] Use git config to set your credentials

done


Web player websocket port and handshake issues

The web player keeps defaulting back to port :9000 when accessed directly from its URL (or the page is reloaded):

Enable web player and set HTTP and websocket ports to different values
Open web player (I am using a domain name in my case, tunneling my computer with ssh -R to a Raspberry Pi server connected to a DNS entry).
Visit the web interface
Click the Player tab, then this works the first time (the dev tools Network tab shows the websocket connection is the newly chosen one)
Reload the player page, now the websocket connection is back to :9000 and fails.
It appears every time /player is loaded directly, it requests :9000 while every time it is accessed by clicking the Player tab, the port is correct.

The web player fails to establish a websocket connection when visiting the Player link a second time:

Visit the web interface home page and hard refresh (Ctrl Shift R)
Click the Player tab, the connection should be working and videos play
Hit the back button in the browser
The console shows this error and eventually the music for the currently queued karaoke video plays despite not even being on the player:

Uncaught TypeError: Cannot read property 'callback' of undefined
    at r (1.karaoke-rs.js:1)
    at Module.Ve (1.karaoke-rs.js:1)
    at __wbg_cargowebsnippetf5302c0143e9bd10d39262404fc2b4c9660c7f9d_645c5ca38dda53ac (karaoke-rs.js:1)
    at <yew::services::resize::ResizeTask as core::ops::drop::Drop>::drop::hf45e7f94856691f9 (:my-http-port/wasm-function[768]:0xbbe35)
    at core::ptr::real_drop_in_place::h76a109ebe0812206 (:my-http-port/wasm-function[673]:0xb2ce2)
    at <yew::html::scope::DestroyComponent<COMP> as yew::scheduler::Runnable>::run::h82153cfbd3015bc8 (:my-http-port/wasm-function[358]:0x823cd)
    at yew::scheduler::Scheduler::put_and_try_run::h654711681d8a2ef3 (:my-http-port/wasm-function[576]:0xa7e23)
    at yew::agent::AgentLink<AGN>::callback::{{closure}}::h164a1611c4638d27 (:my-http-port/wasm-function[860]:0xc32ba)
    at <stdweb::webcore::newtype::Newtype<(stdweb::webcore::serialization::FunctionTag, (A12,)),stdweb::webcore::mutfn::Mut<F>> as stdweb::webcore::serialization::FuncallAdapter<F>>::funcall_adapter::h3f17d9ef7d5dc8f9 (:my-http-port/wasm-function[243]:0x5aa58)
    at Object.n.STDWEB_PRIVATE.dyncall (1.karaoke-rs.js:1)

If you fool around with it enough, like remove the /player from the URL and hit enter, or reload the page on the home page, sometimes (often more consistently than other times) this error appears instead of the currently-queued video:

karaoke-rs.js:1 WebSocket connection to 'ws://my-domain:my-websocket-port/' failed: Connection closed before receiving a handshake response

Good news, after I merge https://github.com/tarkah/karaoke-rs/pull/19, this should fully fix all issues related to navigating away from the player page. If you click back, the audio will end properly and the page you were previously on will render properly.
No idea if any of this will fix the issue around the below error, but if not, it should be the ONLY error left related to this issue
Uncaught (in promise) DOMException: Failed to set the 'buffer' property on 'AudioBufferSourceNode': Cannot set buffer to non-null after it has been already been set to a non-null buffer

@Keavon please try the newest release, 0.9.1 and see if any of these issues still exist. Thanks!

@tarkah I downloaded 0.9.1 and I am getting this error when I open the Player page. I also see [2020-02-07T08:52:26Z INFO  karaoke_rs::websocket] Connection from <IP_ADDRESS>:63241 show up in the karaoke-rs console and [2020-02-07T08:52:26Z INFO  karaoke_rs::websocket] Connection from <IP_ADDRESS>:63241 when closing the browser page. The web player is entirely nonfunctional because of this error.

Huh, does it do this for all your songs, or just a specific song? I'll try to fix it so the program doesn't panic and the web page is still usable. If it can't decode your mp3 file, I'm not sure why that'd be.
Sorry for all these errors, I'm not experiencing any of them. Appreciate you uncovering them all!

@Keavon #20 should do the trick. The player will no longer panic, however it will obviously not play the song because it can't decode that audio file. A WARN will be logged to console notifying the user to remove that song from the queue. Since no panic occurs, you can easily backspace -> Queue page, hit next song, then go back to player (or have queue open on another tab / device)


Unity Support

Doesn't work with Unity 2021.3.1f1 (and probably with any other version).
Will you consider supporting it?

What exactly does not work?
I have no experience with Unity so far, so either you can supply a PR which adds support or you can let me know what needs to be done. If it is not too much effort, I will consider maintaining it.

There are errors when I try to import the project. It seems that Unity's backend (Mono) version has a bit outdated .NET libraries. I'll try to fix it on my own later, I just hoped that you might know the solution.
Starting: C:\Program Files\Unity 2021.3.1f1\Editor\Data\bee_backend.exe --profile="Library/Bee/backend_profiler0.traceevents" --stdin-canary --dagfile="Library/Bee/1900b0aEDbg.dag" --continue-on-failure ScriptAssemblies
WorkingDir: ~
ExitCode: 0 Duration: 0s908ms
[725/730    0s] Csc Library/Bee/artifacts/1900b0aEDbg.dag/Assembly-CSharp.dll (+2 others)
warning CS8032: An instance of analyzer Lombok.NET.PropertyGenerators.NotifyPropertyChangedGenerator cannot be created from ~Assets\Packages\Lombok.NET.2.1.0\analyzers\dotnet\cs\Lombok.NET.dll : Could not load file or assembly 'Microsoft.CodeAnalysis, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'. The system cannot find the file specified..

...

Reloading assemblies after forced synchronous recompile.
Begin MonoManager ReloadAssembly
Symbol file LoadedFromMemory doesn't match image<EMAIL_ADDRESS>Symbol file LoadedFromMemory is not a mono symbol file
Symbol file LoadedFromMemory doesn't match image<EMAIL_ADDRESS>Assembly 'Assets/Packages/System.Reflection.Metadata.5.0.0/lib/netstandard2.0/System.Reflection.Metadata.dll' will not be loaded due to errors:
System.Reflection.Metadata references strong named System.Collections.Immutable Assembly references: <IP_ADDRESS> Found in project: <IP_ADDRESS>.
Assembly Version Validation can be disabled in Player Settings "Assembly Version Validation"
Assembly 'Assets/Packages/Microsoft.CodeAnalysis.Common.4.3.1/lib/netstandard2.0/Microsoft.CodeAnalysis.dll' will not be loaded due to errors:
Reference has errors 'System.Reflection.Metadata'.
Assembly 'Assets/Packages/Microsoft.CodeAnalysis.CSharp.4.3.1/lib/netstandard2.0/Microsoft.CodeAnalysis.CSharp.dll' will not be loaded due to errors:
Reference has errors 'Microsoft.CodeAnalysis'.
Assembly 'Assets/Packages/Microsoft.CodeAnalysis.Workspaces.Common.4.3.1/lib/netstandard2.0/Microsoft.CodeAnalysis.Workspaces.dll' will not be loaded due to errors:
Unable to resolve reference 'SQLitePCLRaw.core'. Is the assembly missing or incompatible with the current platform?
Reference validation can be disabled in the Plugin Inspector.
Unable to resolve reference 'SQLitePCLRaw.batteries_v2'. Is the assembly missing or incompatible with the current platform?
Reference validation can be disabled in the Plugin Inspector.
Assembly 'Assets/Packages/Microsoft.CodeAnalysis.CSharp.Workspaces.4.3.1/lib/netstandard2.0/Microsoft.CodeAnalysis.CSharp.Workspaces.dll' will not be loaded due to errors:
Reference has errors 'Microsoft.CodeAnalysis.Workspaces'.
Assembly 'Assets/Packages/Lombok.NET.2.1.0/lib/netstandard2.0/Lombok.NET.dll' will not be loaded due to errors:
Reference has errors 'Microsoft.CodeAnalysis.CSharp'.
Native extension for WindowsStandalone target not found

What .NET and VS version (or Rider) are you using? If it's VS, it needs to be at least v17.3.

And I can see that you're using C# 10.0 and current version of Unity supports C#9.0 only

Yes, the generators generate C# 10 code. Is there no way to use C# 10 in Unity?

Have you tried the steps described in the "Installing an existing Roslyn analyzer or source generator" section of https://docs.unity3d.com/Manual/roslyn-analyzers.html?

Yes, the generators generate C# 10 code. Is there no way to use C# 10 in Unity?

nope, Mono doesn't support it yet

Have you tried the steps described in the "Installing an existing Roslyn analyzer or source generator" section of https://docs.unity3d.com/Manual/roslyn-analyzers.html?

I've also found it but haven't tried it yet, perhaps a little later, I'll tell you about the results here.

@CollinAlpert Okay I tried to follow those steps. It seems that you have newer versions of Microsoft.CodeAnalysis dependencies than it is allowed by Unity.
Unity suggests using Microsoft.CodeAnalysis.CSharp 3.8.0 in the manual. And it doesn't contain:

GeneratorAttributeSyntaxContext from Microsoft.CodeAnalysis.Common v4.3.0. Unity's latest package is 3.8.0
IIncrementalGenerator from Microsoft.CodeAnalysis.Common v4.0.1. Unity's latest package is 3.8.0
IncrementalGeneratorInitializationContext from Microsoft.CodeAnalysis.Common v4.3.0. Unity's latest package is 3.8.0
On top of that you use FileScopedNamespaceDeclaration from C# 10. Unity uses C# 9.0

I see! If you want to fork this library and downgrade the dependencies, I could help you maintain it. As for IIncrementalGenerator compared to ISourceGenerator, I made the switch in the commit https://github.com/CollinAlpert/Lombok.NET/commit/ee788bb01a6e62eaa1bf5923853479b0b57a6241, so you could look at how the generators were constructed before that.

I don't think I'll dive into that soon enough or ever at all but thank you for your help! Perhaps someone would work on that someday later at least knowing what he has to do after we investigated the issue.
Anyway, great project!

No worries, I get it :)
I hope you manage to use the library at some other point.


MNT: Re-rendered with conda-smithy 1.4.5

and add pip to build requirements now that it's needed.
closes #2

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Tutorial does not work for GPT-4

Hello,
Thank you for the great tutorial. After deploying a GPT-4 model to my Azure OpenAI resource, and calling it in the script, the responses are the following errors.
How can we use GPT-4 with this tutorial?
Thank you
(.venv) PS C:\Users\me\Documents\projects\gpt_sts_poc> python .\gpt-test.py

Azure OpenAI is listening. Say 'Stop' or press Ctrl-Z to end the conversation.
Recognized speech: Test test test.
Encountered exception. The completion operation does not work with the specified model, gpt-4. Please choose different model and try again. You can learn more about which models can be used with each operation here: https://go.microsoft.com/fwlink/?linkid=2197993.

After some research, I found that the ChatCompletions() API should be used with GPT-4. I tried changing the script to:
# Ask Azure OpenAI
    response = openai.ChatCompletion.create(engine=deployment_id, prompt=prompt, max_tokens=100)

Then wrapping the speech output in the correct format by editing...
...
# Get audio from the microphone and then send it to the TTS service.
speech_recognition_result = speech_recognizer.recognize_once_async().get()
prompt = [{"role": "user", "content": {speech_recognition_result.text}}]
...

# Call the model
ask_openai(prompt)

This fails with...
(.venv) PS C:\Users\me\Documents\projects\gpt_sts_poc> python .\gpt-test.py

Azure OpenAI is listening. Say 'Stop' or press Ctrl-Z to end the conversation.
Recognized speech: Test test test.
Encountered exception. Resource not found

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

ID: 1370a1c1-3686-3810-ccd2-648780aa69c9
Version Independent ID: 93ff06b1-e74b-f680-18f9-d9b8dd283e24
Content: Azure OpenAI speech to speech chat - Speech service - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/openai-speech.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @eric-urban
Microsoft Alias: eur

@ericthomas1
Thanks for your feedback! We will investigate and update as appropriate.

@ericthomas1 you're using wrong api version, switch to 2023-03-15-preview and follow this guide: https://learn.microsoft.com/en-gb/azure/cognitive-services/openai/how-to/chatgpt?pivots=programming-language-chat-completions

That is helpful to know. Changing the API version worked.
Thank you

@coder-pm doesnt work for us fails with the same error - Tried changing the api version to 2023-03-15-preview and also please provide link on where to refer the api versions.
InvalidRequestError: The completion operation does not work with the specified model, gpt-4-32k. Please choose different model and try again. You can learn more about which models can be used with each operation here: https://go.microsoft.com/fwlink/?linkid=2197993.

@abhishek-ravikumar-kulkarni, please show your code

@abhishek-ravikumar-kulkarni, please show your code

@coder-pm now i am getting this -
InvalidRequestError: The API deployment for this resource does not exist. If you created the deployment within the last 5 minutes, please wait a moment and try again.
The deployment i have

Please help

@abhishek-ravikumar-kulkarni, please show your code

@coder-pm now i am getting this -
InvalidRequestError: The API deployment for this resource d <img width="805" alt="Screenshot 2023-04-26 at 10 21 48 AM" src="https://user-images.githubusercontent.com/91946307/234472961-d876855c-e1f6-4459-a46d-812c36884321.png"> oes not exist. If you created the deployment within the last 5 minutes, please wait a moment and try again.
The deployment i have

Please help

@coder-pm
The code

@coder-pm
InvalidRequestError: The API deployment for this resource does not exist. If you created the deployment within the last 5 minutes, please wait a moment and try again.
When i am doing,
indexAzure = GPTSimpleVectorIndex.from_documents(documents, service_context=service_context)

How do you build a service context? Do you pass the deployment name there?

@coder-pm  This is the code, its working for other deployment not for gpt4-32k

Your code looks good. Just make sure there are no typos in your deployment name and API base.

@coder-pm yep no typos though

@coder-pm , I'm stuck on the prompt to GPT-4.
I'm not sure how to pass speech INTO the prompt
Example:

Here is the ask_openai() function. How do I pass voice into the user role's content field?

# Prompts Azure OpenAI with a request and synthesizes the response.
def ask_openai(prompt):

    prompt = [
            {
                "role": "system",
                "content": "<system-prompt>",

                "role": "assistant",
                "content": "<asst-prompt>'

                "role": "user",
                "content": "<user-prompt>"  #<--How do I pass in recognized speech into the prompt?
                """
            }
        ]

    # Ask Azure OpenAI
    response = openai.ChatCompletion.create(
        engine=deployment_id,
        messages=prompt,
        max_tokens=100
    )
    text = response['choices'][0]['message']['content']
    print('Azure OpenAI response:' + text)

Thank you

Nevermind, I got it!

Hi @abhishek-ravikumar-kulkarni, I'm facing the same issue for gpt4 deploymen with Azure OpenAI . Did you find any solution?

i have again this problem can ur help me ?
data: {
error: {
code: 'OperationNotSupported',
message: 'The embeddings operation does not work with the specified model, gpt-4. Please choose different model and try again. You can learn more about which models can be used with each operation here: https://go.microsoft.com/fwlink/?linkid=2197993.'
}
}
i am using gpt4

@abhishek-ravikumar-kulkarni is this error resolved?
You should actually use AzureChatOpenAI class for GPT4 usage. Following code should work.
from langchain.chat_models import AzureChatOpenAI

llm = AzureChatOpenAI(deployment_name="<insert deployment name from azure>", model_kwargs={
    "api_key": openai.api_key,
    "api_base": openai.api_base,
    "api_type": openai.api_type,
    "api_version": openai.api_version
})
llm_predictor = LLMPredictor(llm=llm)

embedding_llm = LangchainEmbedding(
    OpenAIEmbeddings(
        model="text-embedding-ada-002",
        deployment="<insert EMBEDDING model deployment name from azure>",
        openai_api_key= openai.api_key,
        openai_api_base=openai.api_base,
        openai_api_type=openai.api_type,
        openai_api_version=openai.api_version,
    ),
    embed_batch_size=1,
)

documents = SimpleDirectoryReader('data').load_data()

service_context = ServiceContext.from_defaults(
    llm_predictor=llm_predictor,
    embed_model=embedding_llm,
)

index = VectorStoreIndex.from_documents(documents, service_context=service_context)

query = '<YOUR QUERY>'
query_engine = index.as_query_engine()
answer = query_engine.query(query)

print(answer.get_formatted_sources())
print('query was:', query)
print('answer was:', answer)

此错误是否已解决？
实际上，应该使用 AzureChatOpenAI 类来使用 GPT4。以下代码应该可以工作。
from langchain.chat_models import AzureChatOpenAI

llm = AzureChatOpenAI(deployment_name="<insert deployment name from azure>", model_kwargs={
    "api_key": openai.api_key,
    "api_base": openai.api_base,
    "api_type": openai.api_type,
    "api_version": openai.api_version
})
llm_predictor = LLMPredictor(llm=llm)

embedding_llm = LangchainEmbedding(
    OpenAIEmbeddings(
        model="text-embedding-ada-002",
        deployment="<insert EMBEDDING model deployment name from azure>",
        openai_api_key= openai.api_key,
        openai_api_base=openai.api_base,
        openai_api_type=openai.api_type,
        openai_api_version=openai.api_version,
    ),
    embed_batch_size=1,
)

documents = SimpleDirectoryReader('data').load_data()

service_context = ServiceContext.from_defaults(
    llm_predictor=llm_predictor,
    embed_model=embedding_llm,
)

index = VectorStoreIndex.from_documents(documents, service_context=service_context)

query = '<YOUR QUERY>'
query_engine = index.as_query_engine()
answer = query_engine.query(query)

print(answer.get_formatted_sources())
print('query was:', query)
print('answer was:', answer)

hi,  which package,   could you give me those package?  I need to install it
from llama_index import VectorStoreIndex？
from langchain.indexes.vectorstore import VectorStoreIndexWrapper as VectorStoreIndex

You can check latest notebook/ reference here - https://docs.llamaindex.ai/en/stable/examples/llm/azure_openai.html


Add Error derive to all Bevy error types

          Not sure how important but would be nice to derive Error on all error things :)

Originally posted by @killercup in https://github.com/bevyengine/bevy/pull/8603#discussion_r1350777416

I can have a look at this!
I'll try to derive Error on every type defined by us that's used as an error/2nd parameter in Result.


Clarify documentation / examples for OIDCCryptoPassphrase

Currently the documentation just mentions OIDCCryptoPassphrase in examples, usually like this:
OIDCCryptoPassphrase <password>

From this I'm left with the question what should I substitute <password> with. My understanding is that this is just used to encrypt / sign the cookie set by the module and can be any random value (with the mention that if there are multiple servers which could handle the request - like behind a load-balancer - they all need to have the same value so that they can read each-others cookies), but perhaps it would be good to explicitly document this somewhere (in the README, Wiki, etc).
PS. Thank you for maintaining this module!

FWIW: there's a line of doc on it here: https://github.com/pingidentity/mod_auth_openidc/blob/master/auth_openidc.conf#L16

I am guessing it should be this one https://packages.debian.org/sid/libapache2-mod-auth-openidc, since my docker image starts with FROM httpd:2.4 which is based on FROM debian:stretch-slim.


SCSS files use relative paths for images, which doesn't work on themes

Current Behavior
A theme created using build-custom-theme.js has relative paths for images. That doesn't work because it ends up pointing to /theme/css/images/, where the correct should be /red/images/.
Expected Behavior
Since those are the default UI image files, I believe there's no need to create a copy of them for each theme. That would be a waste of resources.
Steps To Reproduce
Create a theme using the default colors.scss file, and then load that theme.
Example flow
No response
Environment

Node-RED version: 2.0.5
Node.js version: v14.17.3
npm version: 6.14.13
Platform/OS: macOS Big Sur 11.5.2
Browser: Chrome and Safari

I think that a possible solution would be replacing those paths with absolute ones.
For example:
.red-ui-icons-flow {
-    background-image: url('images/subflow_tab.svg');
+    background-image: url('/red/images/subflow_tab.svg');
    background-repeat: no-repeat;
    background-size: contain;
    filter: brightness(2.5);
}

Would this work?

We cannot use absolute paths because we cannot assume the editor is being served from / - if the user has set httpAdminRoot to /admin (for example), the full path would be /admin/red/images/subflow_tab.svg.
If I understand the issue properly, I think the fix is to strip out any CSS that references a default image instead of including it in the theme.

I removed the references to SVG files from the generated theme file in the updates for Node-RED 2.0.6. It seems to be working fine.
I haven't removed the references to grip.png because they are overwritten by the themes.
Where is this one used? I couldn't find it.

Closing this since everything seems to be working fine.


Link to GitHub should go straight to issue queue & be less prominent

I made the github image smaller and linked to issue queue in commit b2992816704fa8c0fbe051fbc0ff416ca804361e


[#159] Moved Java Tokenizer to its own module

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 70.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 48.55%. Comparing base (30b224f) to head (332407d).

Files
Patch %
Lines

...n/java/net/zscript/tokenizer/TokenArrayBuffer.java
70.00%
0 Missing and 3 partials :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #160      +/-   ##
============================================
- Coverage     49.43%   48.55%   -0.89%
+ Complexity      788      775      -13
============================================
  Files           131      131
  Lines          4978     4978
  Branches        820      820
============================================
- Hits           2461     2417      -44
- Misses         2311     2356      +45
+ Partials        206      205       -1

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


'TimeOut' object has no attribute 'optional'

Hello,
I'm currently running the scenario_runner with openscenario files created by the traffic generation editor.
But the same error keeps happening..
Scenario 1:  two vehicles(one is an ego vehicle) with initial speeds > 0. Both using Waypoint Maneuvers but without Start Triggers.
The error shows up when one vehicle reaches the first waypoint.
Scenario 2: only one non-ego vehicle in the scenario with initial speeds > 0, using Waypoint Maneuvers and without Start Triggers. The error shows up before the vehicle shows up. (I guess it's because I need to add an ego vehicle? but is it possible to have my own ego vehicle in a separate file controlled by a reinforcement learning agent?)
Error :

Traceback (most recent call last):
File "scenario_runner.py", line 408, in _load_and_run_scenario
self._analyze_scenario(config)
File "scenario_runner.py", line 259, in _analyze_scenario
if not self.manager.analyze_scenario(self._args.output, filename, junit_filename, json_filename):
File "C:\Users\Desktop\scenario_runner-0.9.11\scenario_runner-0.9.11\srunner\scenariomanager\scenario_manager.py", line 216, in analyze_scenario
if (not criterion.optional and
AttributeError: 'TimeOut' object has no attribute 'optional'
'TimeOut' object has no attribute 'optional'
No more scenarios .... Exiting

Any idea how to solve this problem? Thanks:)

OS: Windows 10
CARLA Version [0.9.11]
Scenario_Runner Version [0.9.11]
Python version [3.7]

Hi,
can you switch to 0.9.11 branch (not tag) of scenario runner? This should resolve this issue.

ok thanks! solved


Word Cloud links have trailing spaces

Just a small cosmetic issue. Word cloud items have trailing spaces or some type of padding afterwards to make them appear separated from one another. Only issue is that when hovering the cursor over them, the hyperlink underline shows in the trailing space

It appears that there is an em space hard coded inside the hyperlink?

Word Cloud has been removed.


Enable E226 linting and fix all E226 errors

E226 (missing whitespace around arithmetic operator) is considered optional in default flake8 configuration. In this PR we enforce E226 checks in invoke lint and fix all E226 errors. More details about E226 can be found at the actual PEP-8.

Thank you!


AST tiny and small pretrained models

Hi Yuan,
I have tested your AST mode pretrained on Audioset on my own dataset and I noticed that it achieves similar performance as EfficientNet pretrained on Audioset using psla pipeline.
I was wondering if maybe a smaller AST model could achieve same performance while using less parameters (my dataset is not as big as Audioset). Did you train also tiny and small AST? Would it be possible to have AST tiny and small models pretrained on Audioset? I would like to compare them to the other models I tested (AST base, EfficientNet) and analyse the benefit of Audioset pretraining on other audio classification tasks.
I will cite your work in future publications of course.
Thank you in advance.
Annalisa

Hi Annalisa,
Unfortunately, I don't have tiny&small ImageNet+AudioSet pretrained models. The SSAST repo has but it is based on a different pretraining scheme. Without AudioSet pretraining, you are free to use tiny & small models with only ImageNet pretraining.

I have tested your AST mode pretrained on Audioset on my own dataset and I noticed that it achieves similar performance as EfficientNet pretrained on Audioset using psla pipeline.

They can be similar, but I usually see a small improvement in my experiments. The AST model needs a 10X smaller learning rate than the EfficientNet model.
-Yuan


Example for tension and friction config

Are tension and friction the same as stiffness and dampening?  It would be neat to have an example like react-motion to demonstrate the attributes.

In addition to the grid from that example, it may be nice to have a separate row for the pre-defined settings:  default, gentle, wobbly, stiff, slow.
Unfortunately, I will be unable to try making the example until the semester ends (that, and I am still wrapping my head around how react-spring works).

Also, one other question, is it possible to set your own defaults for the config without having to pass the config object to the react-spring components every time?  (similar to how you can create a custom instance of axios)

Yes, they're the same. Could i interest you in porting it?


Allow custom import of db and files in provider interface (ddev pull)

The Problem/Issue/Bug:
Discussed in https://github.com/drud/ddev/pull/3223 thanks to @jonaseberle
The ddev pull provider interface can be a little too inflexible. It assume one database, for example, when there might be more than one. It also assumes that you don't know where the files are going to go, and you might want to send them somewhere else.
How this PR Solves The Problem:
Allow db_import_command and files_import_command to take over the job of actually doing the import.
Manual Testing Instructions:

[ ] Experiment with the updated localfile.yaml.example, which demonstrates the new usage.

Automated Testing Overview:
TestLocalfilePull now tests these feature
Related Issue Link(s):
Replaces #3223
Release/Deployment notes:

This can be tested in gitpod using https://gitpod.io/#https://github.com/drud/ddev/pull/3223
Artifacts can be downloaded for local experimentation at https://github.com/drud/ddev/actions/runs/1469618009

That is great! Thank you so much!
I tested locally.
It absolutely stays as easy as it has been before while still allowing for more complex setups.
Functionally I wasn't able to make it misbehave in any way. --skip-db / --skip-files / --skip-import behave as before/expected, too.

Some docs/example additions and improvable command output I noticed so far (nothing serious or blocking - I am happy to go through it and do PRs):
command output:

When giving a custom _pull_command, it still outputs Database/Files downloaded to: /.../.ddev/.downloads/... which might not be what actually happened in that custom command.

docs:

In order to have a step do nothing but still prevent the default import (for example if the _pull_command already downloaded files to the final locations), add a _import_command without the command: declaration.

Thanks @jonaseberle -
I didn't understand the comment

command help text: misses descriptions for --skip*/-y

because
$ ddev pull localfile -h
Pull with localfile

Usage:
  ddev pull localfile [flags]

Examples:
ddev pull localfile
ddev pull localfile -y
ddev pull localfile --skip-files -y

Flags:
  -h, --help                help for localfile
  -y, --skip-confirmation   Skip confirmation step
      --skip-db             Skip pulling database archive
      --skip-files          Skip pulling file archive
      --skip-import         Downloads file and/or database archives, but does not import them

Global Flags:
  -j, --json-output   If true, user-oriented output will be in JSON format.

Oh, haven't noticed that it is there already when called like that.
I was looking for it with ddev pull --help or ddev help pull.


API blueprint documentation in different language

Hi,
is there any way to write API documentation in two languages?
My comments in the code are in english, but I need to create API documentation in different language.
Any ideas?
So far I can only translate the output from api:docs manually and then generate HTML file, but maybe somebody had already this problem and knows better solution?
For example when I create "@Response(422, body={"error": {"username": {"Username is already taken."}}})"
tis that possible that this string "Username is already taken" will come from Laravel lang file?
thank you!

I don't think I want to over complicate the doc generation process too much.
Feel free to investigate how plausible it would be and send a PR.


Python range() Function

Proposed title of article
Python range() Function
Proposed article introduction
Range () function is a Python built-in function. It returns a number sequence that starts at zero and increments by one by default, stopping before the provided number.
In this article, we will learn how to utilize the range() function in a variety of ways, including with explicit start and finish indices, custom step sizes, and negative step sizes and explained code examples of each.
Key takeaways
-Overview of Python range() Function
-How to Loop Through Any Iterable Using Python's range() Function
-How to Use Python's range() Function with Explicit Start and End Indices
-How to Use Python's range() Function with a Custom Step Size
-How to Use Python's range() Function with a Negative Step Size
-How to Use Python's range() and reversed() Functions to Reverse a Sequence
References
N/A

Unfortunately, this topic is already covered adequately in already published articles and official documentation. Feel free to suggest another topic. Thanks


Investigate state management delta (de)serialization

Presently, the statemgmt deltas use a custom serialization process.  In state_delta.go, there is a comment
// marshalling / Unmarshalling code
// We need to revisit the following when we define proto messages
// for state related structures for transporting. May be we can
// completely get rid of custom marshalling / Unmarshalling of a state delta

Now that the message protos are defined, unless there is a compelling reason to continue to use the custom serialization, it would make the message protocol specification more clear, and would also simplify the state transfer API in issue #299
Assuming this can be done, the CPI interface and associated plugin usage will need to be fixed as well, feel free to assign to me (@jyellick) if needed for the CPI related work.

@jyellick @srderson I presume this is still relevant? If so, please transfer to Jira and close this one. Thanks

@christo4ferris Actually, I think this is dead in the new architecture, as we will only be moving blocks, and not state deltas.

ok, closing thanks @jyellick


Fixed the leak of Cohttp_lwt fd

TelegraML often prints warnings "Cohttp_lwt: body not consumed - leaking stream!".
This commit fixes this issue: clear_update now consumes the body using Cohttp_lwt.Body.drain_body.
Related with https://github.com/mirage/ocaml-cohttp/issues/730

sorry for late response, don't use github much anymore. looks good to me!


Reasoning speed is too slow

In the code, I need to run llama every time I reply to a word, is this a normal phenomenon? By the way, is the final assessment a validation collection or a test collection? Is it possible to finally evaluate the selected test set? I doubt that the evaluation of too many samples will bring a lot of computational burden.

Closing as no further response. The test server is open.


Move default language setting to Tokenizer

~1. Add a preserve feature to CodeNode~
~In the original implementation, CodeNode is removed when the backward deletion occurs at the beginning of the Node.~
~In some use cases, implementing a code editor, such behavior is not needed.~
~The "Preserve" setting is for making it controllable.~
2. Move the default language setting to Tokenizer
The former PR #3243 enabled to use arbitrary Tokenizer other than Prism and 'javascript' is not always appropriate to the default so make it a Tokenizer setting.
This PR contains 2 changes but I'm happy to divide them if required.

I have no idea how to fix the failed e2e tests...
It seems to be occurring in the main branch randomly.

@thegreatercurve
Thanks for your kind review!
Thanks to #3367, the code something like below works as  I expected.
Thank you so much for the great work!
I've removed commit 1. which is no longer needed as you mentioned.

export class CodeEditorNode extends CodeNode {
  static getType(): string {
    return 'code-editor';
  }

  collapseAtStart(): true {
    return true;
  }
}

export const $createCodeEditorNode = (language?: string | null | undefined): CodeEditorNode => {
  return new CodeEditorNode(language);
}

const editor = createEditor({
  nodes: [
    CodeEditorNode,
    CodeNode: {
      replace: CodeNode,
      with: (node: CodeNode) => $createCodeEditorNode(node.getLanguage())
    },
    CodeHighlightNode,
  ]
})

I'm curious if it's better to have default language as a param to registerCodeHighlighting and CodeHighlightingPlugin. Then you can have single tokenizer but custom default language based on specific surface needs

I think it's also OK, but I put the setting here because the supported languages depend on Tokenizer's implementation. The setting 'javascript' is one of the proper choices for PrismTokenizer because Prism supports it.
In terms of the use of the library, the difference is that users should specify the default language even when they use PrismTokenizer by default.


Hide search until this feature is fully implemented

The search box gives the idea that something can be searched but actually this feature isn't there yet.
It should be hidden for now until it's picked up and implemented

#12 related

Closed by #37


Add methods to insert and display interviews

Task completed

add use case and view model methods to insert interviews to the database and display interviews in the dashboard screen.

Tests
https://github.com/Audienix/InterPrep/assets/67870437/4c27afca-dff0-42c4-b6f9-b22a748123b4

@YANLING-H  @sev3rus I have fixed almost all the issues and removed lot of unnecessary code.
1> AlertDialog for not filling up mandatory details - Can be handled in next PR.
2> I am approving this but plz fix the below issue before merging this PR.
Issue - I added one interview at 12:53 AM 8th October. But it went to Past Interview. It is very easily reproducible issue.


Can we not flag comments in ARM JSON files as errors?

I'm doing a lot of work with Azure Resource Manager templates, and tend more towards the Ops side of DevOps. Over on our side of the fence, reading infrastructure-deploying JSON files would be much easier with inline comments, since we're building longer-lived IaaS stuff with them and tend to human-edit them more.
Can we have a setting where JSON comments in ARM templates aren't flagged as illegal? The ARM parser does strip them out if you submit a template with comments. I know you can add a "description" or "comment" element, but when you're reading 1000 lines of template it's nice to see a comment that clearly states "change this parameter for Behavior X" in a distinct color the way you might do in a traditional XML or config file.

You have to associate your file with the 'jsonc' language mode. You can do that in the status bar (on the language mode), and also in the user settings.

That works, thank you!


Export Prop type in main types file (vue/types/index.d.ts)

What problem does this feature solve?
When attempting to declare a type for a complex object-type prop for a component, it's difficult to understand (without knowing exactly how the Props typing works) how to cast that specific prop to the interface you create.
The best way of doing it currently is to do this:
import Vue from 'vue'
import { Prop } from 'vue/types/options'

interface FooBar {
  foo: string,
  bar: string
}

export default Vue.extend({
  props: {
    foobar: Object as (() => FooBar)
  }
})

What does the proposed API look like?
It would be nice to expose the Prop generic type in vue/types/options.d.ts in the default types export so that we can simply use this:
import Vue, { Prop } from 'vue'

// boilerplate
  props: {
    foobar: Object as Props<FooBar>
}

This also extends to complex callback types if need be so that whenever the regular prop types via the primitive type constructors don't work, we can create an interface and cast it as Prop<ICustomType> and it just works.

Did you tried vue-property-decorator?
import Vue from 'vue'
import { Prop, Component } from 'vue-property-decorator'

interface FooBar {
  foo: string,
  bar: string
}

@Component
export default class App extends Vue {
  @Prop() foobar: FooBar;
}

I like the idea. Prop type would hide the internal detail of type declaration and looks more intuitive syntax.
@sandangel This proposal is not related with decorators. It would help us declare props type easier with canonical API.

Yep, it's not obvious from that the type declaration uses generic constructors or factory functions to tease out the generic type, and exposing the Prop type makes it much easier to use custom interfaces

I can do a PR if it's easier since this would be a very simple change.


Updated the 1.6 branch with more info and fixes

Added a nix shell for an easier dev environment (including things like libsocketcan and zeromq headers)
Updated the python bindings to support 3.10
Fixed a compilation warning in a debug file
Added toliman related info to the readme

Ignore, I selected the wrong PR destination when making the PR


Bugfix/abstractprojectors

Fixes a bug that prevented calculating the ProjectedEnergy with UniformProjector().

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.2%) to 88.691%

Files with Coverage Reduction
New Missed Lines
%

src/BitStringAddresses/fermifs.jl
2
95.16%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
3882

Relevant Lines:
4377

💛  - Coveralls


Snapshot Testing Version Mixmatch

error: the library 'HtmlSnapshotTesting' requires macos 10.13, but depends on the product 'SnapshotTesting' which requires macos 10.15; consider changing the library 'HtmlSnapshotTesting' to require macos 10.15 or later, or the product 'SnapshotTesting' to require macos 10.13 or earlier.

I'm also seeing this same issue on MacOS Sonoma. Specifically when using swift-html-vapor v0.4.0.


Google Login is hard-coded for Offline Access

This might be a problem.  Today, when using our express-angular sample application, the Google log-in dialog was blatantly asking me for "Offline Access", and not outlining the scopes it wanted.
However, as with most google login issues, I only saw it two or three times (revoking the app access in-between each attempt).  I can no longer reproduce.
Has anyone else seen this issue?
This makes me concerned that GoogleUser.grantOfflineAccess() is the wrong API to be using, because we definitely do not want offline access as our default scope request.  By default we should only ask for email.

Yeah, if you request scopes with offline, they always just show "requests offline access" and gives you the ability to refresh the access token.


facing issue while importing setfit classifier

I am facing some issues while trying to import SetFitClassifier
code - from setfit import SetFitClassifier
Error - RuntimeError: Failed to import transformers.trainer_callback because of the following error (look up to see its traceback): module 'lib' has no attribute 'X509_V_FLAG_NOTIFY_POLICY'.
will be really helpful if anyone can help me to troubleshoot this.

Hi @Abhaysharma53,
Try to upgrade pyopenssl. Here's how:
python -m pip install pip --upgrade
pip install pyopenssl --upgrade


Make ng test --environment=prod fail when tests contain fdescribe, fit, xdescribe or xit

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

Versions.
@angular/cli: 1.4.4
node: 8.1.1
os: win32 x64
@angular/animations: 4.3.6
@angular/common: 4.3.6
@angular/compiler: 4.3.6
@angular/core: 4.3.6
@angular/forms: 4.3.6
@angular/http: 4.3.6
@angular/platform-browser: 4.3.6
@angular/platform-browser-dynamic: 4.3.6
@angular/router: 4.3.6
@angular/cli: 1.4.4
@angular/compiler-cli: 4.3.6
@angular/language-service: 4.3.6
typescript: 2.3.4
Repro steps.
create a test with fdescibe, fit, xdescribe or xit
The log given by the failure.
ng test runs only the tests with fdescribe or fit, or ignores tests with xdescribe or xit (which is fine is you are in development mode)
Desired functionality.
if environment prod is defined as build environment in ng test (ng test --environment=prod), make the build/tests fail when they contain fdescribe, fit, xdescribe or xit tests.

That is unexpected behavior for Jasmine. You can include a TSLint rule that will catch the usage of fdescribe, fit, xit, and xdescribe. When your CI runs, have it run lint first. If it fails, you can fail the build. If it passes, you know there are no unwanted test functions and proceed to run ng test.

Hi @timvl,
We will not special case tests with the environment. This is just not a feature we're interested in.
Two things you can do in the meantime;

use linting to fail tests on CI, or
in code or your karma.conf, check the environment configuration and replace those functions with error throwing ones. This is probably the closest solution to your feature request.

Closing this as it's really not in the scope of the CLI. Extensibility will probably fix this.

Thanks, that will work for me

for (2), is there a recipe? am trying to use middleware & beforeMiddleware but am not able to get hold of fdescribe / fit.


Added Perspective to list of widgets, and added gif to assets.

name is just "perspective", thanks!

Done! @timkpaine

would it make sense to use this as the gif https://camo.githubusercontent.com/f69d1132e2984e2a97209fc21520e9812c464873/68747470733a2f2f70657273706563746976652e66696e6f732e6f72672f696d672f64656d6f5f736d616c6c2e676966

Yes, I've added it in the image section. Does it look better now?

it should probably be copied locally into assets/perspective.gif

I'm not sure if what I've done now is right. Does it look okay? Added the gif as .gif in assets.

yep looks good!
@blink1073 does this look ok?

@blink1073 and @ruthussanketh thanks!


Improper permissions on Portus init script

It's currently impossible to run opensuse/portus:2.5 since the /init script used as the entrypoint lacks the execute bit.
~/Desktop $ docker run -it opensuse/portus:2.5
Unable to find image 'opensuse/portus:2.5' locally
2.5: Pulling from opensuse/portus
ab9e8de21875: Pull complete
760a221d5360: Pull complete
Digest: sha256:fd014f18cef904c5d85b37b7aca36c1d62a2f36e4b93d75c8cc039ea93b359bd
Status: Downloaded newer image for opensuse/portus:2.5
docker: Error response from daemon: OCI runtime create failed: container_linux.go:346: starting container process caused "exec: \"/init\": permission denied": unknown.
~/Desktop $ docker run -it --entrypoint= opensuse/portus:2.5 bash
cd447a2256f7:/ # ls -la
[snip]
-rw-r--r--   1 root root 3089 May  6 11:42 init
cd447a2256f7:/ #

ping @mssola

Hey   @Ashtonian @flavio @akx
I am scanning all portus issues to give this to all who have had problems with portus images
Successfully repaired opensuse/portus:2.5 OCI container image
Hi @mdeboer @bretep @invy @stefankip @dkowis   This time it's done, I have a solution to use, all details https://github.com/SUSE/Portus/issues/2241#issuecomment-593370789
It solves many of the issues I've scanned on portus project, like garbage collection keep latest feature, proper sync, I also have a working clair scanner (I think you might be interested Ash , remembering from the comments of your docker-compose.yml :) )
:)
I will read any feedback from you all with interest

I think this is still an issue:
# docker run -it opensuse/portus
Unable to find image 'opensuse/portus:latest' locally
latest: Pulling from opensuse/portus
ab9e8de21875: Already exists
760a221d5360: Already exists
Digest: sha256:fd014f18cef904c5d85b37b7aca36c1d62a2f36e4b93d75c8cc039ea93b359bd
Status: Downloaded newer image for opensuse/portus:latest
docker: Error response from daemon: OCI runtime create failed: container_linux.go:370: starting container process caused: exec: "/init": permission denied: unknown.
ERRO[0002] error waiting for container: context canceled

+1


given and event descriptor generate a json schema for its events

Because once we know the datakeys, we know how to completely lock down the Event.

I need this too! Are you working on this already?

No, just noting that it needs to be done.
On Tue, Sep 8, 2015, 09:58 Arman Arkilic<EMAIL_ADDRESS>wrote:

I need this too! Are you working on this already?
—
Reply to this email directly or view it on GitHub
https://github.com/NSLS-II/metadatastore/issues/168#issuecomment-138567974
.

Ported to https://github.com/NSLS-II/databroker/issues/178


producer and consumer connect  broker different  ip address

rocketmq version 4.6.1
producer use  broker public ip address.  Form some resaon ,  consumer need use broker another private ip addess.
so I build two group namesrv ，one for producer ，other for comsumer，
i modified  DefaultRequestProcessor  getRouteInfoByTopic method
after
‘TopicRouteData topicRouteData = this.namesrvController.getRouteInfoManager().pickupTopicRouteData(requestHeader.getTopic());’
i add repalce brokerAddress method
//TODO ADD replace brokerAddress adress
replaceBrokerAddress(topicRouteData,this.namesrvController.getKvConfigManager().getKVMapByNamespace(BROKER_IP_REPLACE_CONFIG));
private static void replaceBrokerAddress( TopicRouteData topicRouteData, Map<String/* source*/, String /* target */ > map ) {
log.info("repalce map:{}" ,map);
List brokerDataList =  topicRouteData.getBrokerDatas();
if (brokerDataList !=null && map != null) {
for(BrokerData brokerData : brokerDataList) {
HashMap<Long, String> addrsMap = brokerData.getBrokerAddrs();
if( addrsMap !=null) {
for(Map.Entry<Long, String> entry : addrsMap.entrySet()) {
String value = entry.getValue();
int index = value.lastIndexOf(":");
if(index != -1) {
String sourceIp = value.substring(0,index);
String targetIp = map.get(sourceIp);
if(targetIp != null) {
entry.setValue(value.replaceFirst(sourceIp,targetIp));
}
}
}
}
        }
    }
    log.info("repalce topicRouteData:{}" ,topicRouteData);
}

the consumer can get the  replaced address .
i'm not sure ,anthor place i need to modify , or have some better method .

Just change here，the producer can get the replaced address too.

@xunmengh it would be better to implement more customized requirements by implementing org.apache.rocketmq.remoting.RPCHook.


Index.md, Overview.md

We have
https://github.com/SciML/SciMLDocs/blob/main/docs/src/index.md
and
https://github.com/SciML/SciMLDocs/blob/main/docs/src/overview.md
The link https://docs.sciml.ai/Overview/dev/
currently goes to index.md?

I think back then the overview.md was not included anywhere, but this has been fixed by now, it is here:
https://docs.sciml.ai/Overview/dev/overview/


Unsupported engine (latest)

While installing on Debian, i have this error:
Unsupported engine {
package<EMAIL_ADDRESS>required: { node: 'latest' },
current: { node: 'v17.8.0', npm: '8.5.5' }
}

Isn't it just a warning? I think it should still work.
Anyway, I am updating min version to a lower value soon.


M3-1980 Update Radio Input Label text size on 'Create Volume' drawer

Update Radio Input Label text size on 'Create Volume' drawer
Type of Change

Non breaking change ('update', 'change')

Note to Reviewers
View this change from the linode details page under the volumes tab.

Coverage remained the same at 33.463% when pulling 0573888cb80f4a26c08f98f2a53987f522592bd7 on WilkinsKa1:M3-1980 into 4ffafff1f54186227bbc8478efc8baffadbfa2fd on linode:develop.


CI: Adds explicit runtime versions in job matrix

CI: Adds explicit runtime versions in job matrix.
Updates appveyor.yml.
Related: #1096

CI: Stores build artifact (binding.node) for job.

The artifact will be available under job details in Artifacts tab
URL format: https://ci.appveyor.com/project/sass/node-sass/build/<build-id>/job/<job-id>/artifacts

Package: Updates to latest versions.

/cc @xzyfer, @saper, @nschonni
This is related to #56.
The artifacts are publically accessible. To wit: https://ci.appveyor.com/project/sass/node-sass/build/1180/job/1sjh8wsp5rfb1n95/artifacts, which means no more Windows machine required for releases.
Random thoughts:
We can configure the TravisCI do the same for Mac and Linux. As I have understood it, TravisCI doesn't provide out of the box storage but encourages users to configure an external storage like S3 bucket (which makes sense IMO, as this approach promises storage permanency).
After that is done, we can actually make our install script serve the binary from the CI server directly. For instance:
https://ci.appveyor.com/api/buildjobs/1sjh8wsp5rfb1n95/artifacts/win32-ia32-14_binding.node. If we come across issue of "artifact expiry" (which I am not sure whether or not release artifacts get deleted from AppVeyor cloud after some time), we can push the artifact to the same instance of S3 bucket which will contain FreeBSD, Mac and Linux binaries (this can be script within appveyor.yml).
To consume the binaries from CI artifacts directly, we can probably define a new item "binaries": { } in package.json, which will contain the URL (or the unique part of the URL) for each combination. For instance:
"binaries": {
  "linux32-ia32-44": "<s3-url>",
  "win32-ia32-14": "1sjh8wsp5rfb1n95"
}

If this is the direction we want to take (as opposed to configuring node-pre-gyp strategy; which IMO possesses some technical debt as their advertised approach of building binaries etc. is slightly incompatible with our architecture), I can close #56 and open a new issue with the proposed strategy; in order to make CI do all the heavy-lifting for us in future.

Replied in #56

What do you mean by saying node-pre-gyp advertised strategy is not compatible with us?

That we don't need to make changes to configure node-pre-gyp in our setup, as CIs are flexible enough.


I wonder if it's possible for ufldv2 to detect more than 4 lanes

Hi~ My custom dataset contains up to 8 lanes, can ufldv2 tackle this problem like ufldv1?

@dragonhaha
Yes, you can refer to the model of CurveLanes. It could support 10 lanes.


[Bug] [dolphinscheduler-tools] [mysql] bugs are thrown when init&upgrade meta-schema

Search before asking

[x] I had searched in the issues and found no similar issues.

What happened
exceptions is thrown while running UpgradeDolphinScheduler with profle 'upgrade':

Duplicate key exceptions are thrown when upgrading mysql-meta-data by executing ver. 2.0.3~ver. 2.0.4 dolphinscheduler_ddl.sql
,like this:

An exception occurred while dropping an index that does not exist by executing ver. 3.1.0 dolphinscheduler_ddl.sql

Duplicate column exceptions are thrown when upgrading mysql-meta-data by executing ver. 3.1.1 dolphinscheduler_ddl.sql
,such as :

What you expected to happen
I think the upgrade process is important and shouldn't have any exceptions, and appended upgrade processes must take into account the case of duplicates of indexes or columns or tables
How to reproduce
I recommend defining mysql procedures to determine whether an index or column already exists, and if it exists, delete it and then create a new one;
then exception dropping an index that does not exist occurs because of a sql logic error and easy to fix it;

Anything else
I recommend that subsequent versions of the mysql sql script also need to avoid these exceptions
Version
dev
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I think the upgrade process is important and shouldn't have any exceptions, and appended upgrade processes must take into account the case of duplicates of indexes or columns or tables

Yes, agree with you.


[AutoPR] cost-management/resource-manager

Created to accumulate context: cost-management/resource-manager

(message created by the CI based on PR content)

@AutorestCI rebuild https://github.com/Azure/azure-rest-api-specs/blob/master/specification/cost-management/resource-manager/readme.md

Working on generating https://github.com/azure/azure-rest-api-specs/blob/master/specification/cost-management/resource-manager/readme.md for you!!!

Nothing to rebuild, this PR is up to date


Update asserts

#14

looks ok, let me do final checks before mergin

added new parameters to order such as version and expire_at + new events for upgrade methods in the main contract.
+oracle_settle_qty to mitigate any unexpected issues with rollups


Change default port 3000?

Hi
The port 3000 is taken in my system. How can I use another port properly with your setup?
Thanks

Hi
The port 3000 is taken in my system. How can I use another port properly with your setup?
Thanks

hi!
change docker-compose.yml
caddy:
image: stefanprodan/caddy
container_name: caddy
ports:
- "80:3000"# 80 your port  or another
- "9090:9090"
- "9093:9093"
- "9091:9091"

@gerroon pretty much as per @letnab's comments, or you can look at some of the previous issues like: https://github.com/stefanprodan/dockprom/issues/133#issuecomment-477899012


Pagination doesn't seem to work

I've indexed some 700k records and checked in elasticHQ what the query *tux*, that returns 114 records, same happens if I paginate a number higher than the expected numbers. but when I do
$players = User::search('tux')->paginate(24);

I get 24 results and an empty paginator.

Might add that I'm currently on dev-master version of the engine.

Hello @thorerik, I will check on this later in the day when I have time. Not to boycott myself but if you need this working ASAP, Laravel Scout has its own elastic engine now that was recently merged, you can check that out.

Thanks, that must have passed right under my radar while I've been working on our own stuff, thanks for giving me a headsup on a 1st party solution :)

NP 😄  I made the driver because there wasn't an elastic integration at that time, so this package has become a bit obsolete.

maybe worth updating packagist to indicate that as well since most resources (laracast and the docs) kinda indicate that Algolia is the only driver around.

To add to this in case someone else wanders by this issue, issue is caused by scout itself:
https://github.com/laravel/scout/pull/52

@thorerik I see a fix has been merged in the Scout repo, is this solved?

Yes, I'm just waiting for them to tag it and maybe extend it to make scout able to do a lengthAwarePaginator instead.

Having the same problem here. Thanks @thorek for adding the getTotalCount but who will update the builder paginate method?
This solves my problem after adding these lines of code in laravel scout src/Builder.php.
        $paginator = (new LengthAwarePaginator($results, $engine->getTotalCount($rawResults), $perPage, $page, [
            'path' => Paginator::resolveCurrentPath(),
            'pageName' => $pageName,
        ]));

@harlekoy: if you could make a PR to laravel/scout that'd be great, I'd rather not make any changes to upstream packages(maintenance hell)


run bazel failed:FATAL: ExecuteProgram(C:\Users\Administrator/_bazel_Administrator/install/9470fb600225157f1e34c45c6d4dc834/_embedded_binaries/embedded_tools/jdk/bin/java.exe) failed: 6

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): win7
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:no
TensorFlow installed from (source or binary):source
TensorFlow version: 2.0
Python version: 3.6
Installed using virtualenv? pip? conda?:conda
Bazel version (if compiling from source): 0.27
GCC/Compiler version (if compiling from source):no
CUDA/cuDNN version:no
GPU model and memory:no

Describe the problem
After I installed bazel 0.27.1, I try to run the command "bazel", it show the error
"FATAL: ExecuteProgram(C:\Users\Administrator/_bazel_Administrator/install/9470fb600225157f1e34c45c6d4dc834/_embedded_binaries/embedded_tools/jdk/bin/java.exe) failed: 6"
but when move into the path "C:\Users\Administrator/_bazel_Administrator/install/9470fb600225157f1e34c45c6d4dc834/_embedded_binaries/embedded_tools/jdk/bin/", it is ok to run "bazel", do you have idea on it? and how to solve it.
Thanks,
Provide the exact sequence of commands / steps that you executed before running into the problem
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

@cj741 Just to verify, did you follow the steps mentioned in official Tensorflow Website and did bazel installed properly? Thanks!

@gadagashwini  Thanks for your response, I have try to install bazel from Chocolatey and download the Bazel binary (bazel--windows-x86_64.exe), both of them are still have the same error.

Could you downgrade Bazel to 0.26.0 as that is the max version allowed. Let us know if this resolves your problem. Thanks!

I have tried Bazel 0.23.0, since on the stackoverflow, someone say the higher versions are not work well with tensorflow, but still got the same problem.

From what I can see, this is a bazel bug.
Please reach out to bazel team.

ok, I send the problem to the bazel team, thanks~

@cj741 Can you please let us know if you are happy to close this issue. Thanks!


To make compatible with external gateway structure changed for MA-5, MA-6

retest this please

SonarQube analysis reported 30 issues

 2 critical
 13 major
 6 minor
 9 info

Watch the comments in this conversation to review them.
9 extra issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 HubSubRequestBuilder.java#L45: Define a constant instead of duplicating this literal "yyyy-MM-dd'T'HH:mm:ss.SSSXXX" 3 times.
 AbsractQueryBuilder.java#L65: This block of commented-out lines of code should be removed.
 HubSubRequestBuilder.java#L26: Move the contents of this initializer to a standard constructor or to field initializers.
 HubSubRequestBuilder.java#L41: Remove this unused method parameter "userProfile".
 HubSubRequestBuilder.java#L175: Change this condition so that it does not always evaluate to "true"
 DefaultAppRequestBuilder.java#L10: Remove this unused import 'org.workflow.core.execption.WorkflowExtensionException'.
 HubSubRequestBuilder.java#L33: Remove the declaration of thrown exception 'com.wso2telco.core.dbutils.exception.BusinessException', as it cannot be thrown from method's body.
 AbsractQueryBuilder.java#L154: Reduce the number of returns of this method 2, down to the maximum allowed 1.
 HubSubRequestBuilder.java#L168: Reduce the number of returns of this method 2, down to the maximum allowed 1.

few suggestions
1.Define a constant instead of duplicating this literal "yyyy-MM-dd'T'HH:mm:ss.SSSXXX" at the abrract layer and reuse it to all concrete classes.

SonarQube analysis reported 25 issues

 2 critical
 9 major
 5 minor
 9 info

Watch the comments in this conversation to review them.
8 extra issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 HubSubRequestBuilder.java#L45: Define a constant instead of duplicating this literal "yyyy-MM-dd'T'HH:mm:ss.SSSXXX" 3 times.
 AbsractQueryBuilder.java#L66: This block of commented-out lines of code should be removed.
 HubSubRequestBuilder.java#L26: Move the contents of this initializer to a standard constructor or to field initializers.
 HubSubRequestBuilder.java#L41: Remove this unused method parameter "userProfile".
 HubSubRequestBuilder.java#L175: Change this condition so that it does not always evaluate to "true"
 HubSubRequestBuilder.java#L33: Remove the declaration of thrown exception 'com.wso2telco.core.dbutils.exception.BusinessException', as it cannot be thrown from method's body.
 AbsractQueryBuilder.java#L155: Reduce the number of returns of this method 2, down to the maximum allowed 1.
 HubSubRequestBuilder.java#L168: Reduce the number of returns of this method 2, down to the maximum allowed 1.

SonarQube analysis reported 27 issues

 1 critical
 11 major
 6 minor
 9 info

Watch the comments in this conversation to review them.
2 extra issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AbsractQueryBuilder.java#L67: This block of commented-out lines of code should be removed.
 AbsractQueryBuilder.java#L156: Reduce the number of returns of this method 2, down to the maximum allowed 1.


android: [WindowSDL] No running App found, exit.

Sometimes android app is being paused  during sinitialization, before the kivy.app.App instance is created. In such case it exits with error [WindowSDL] No running App found, exit.
It happens with some apps, when --window arg is passed to p4a.
Essentially what happens is:

SDL2 Window is created
App paused
kivy tries to find App instance to handle pause, and fails
stopTouchApp() called from window_sdl2.py do_pause()
App instance is created
stopTouchApp() forces app to exit.

Log:
06-25 19:27:20.316 I/python  (27304): [INFO   ] [Window      ] Provider: sdl2
06-25 19:27:20.346 V/SDL     (27304): setOrientation() orientation=-1 width=800 height=600 resizable=true hint=
06-25 19:27:20.356 I/python  (27304): [INFO   ] [GL          ] Using the "OpenGL ES 2" graphics system
06-25 19:27:20.356 I/python  (27304): [INFO   ] [GL          ] Backend used <sdl2>
06-25 19:27:20.356 I/python  (27304): [INFO   ] [GL          ] OpenGL version <b'OpenGL ES 3.1<EMAIL_ADDRESS>(GIT@I041b18199e)'>
06-25 19:27:20.366 V/SDL     (27304): surfaceDestroyed()
06-25 19:27:20.366 V/SDL     (27304): nativePause()
06-25 19:27:20.396 I/python  (27304): [INFO   ] [WindowSDL   ] No running App found, exit.
06-25 19:27:20.406 V/SDL     (27304): surfaceCreated()
06-25 19:27:20.406 V/SDL     (27304): surfaceChanged()
06-25 19:27:20.406 V/SDL     (27304): pixel format RGBA_8888
06-25 19:27:20.406 I/python  (27304): [INFO   ] [Window      ] auto add sdl2 input provider
06-25 19:27:20.406 V/SDL     (27304): Window size: 1080x1845
06-25 19:27:20.406 V/SDL     (27304): Device size: 1080x1920
06-25 19:27:20.406 I/python  (27304): [INFO   ] [Window      ] virtual keyboard not allowed, single mode, not docked
06-25 19:27:20.406 V/SDL     (27304): nativeResume()
06-25 19:27:20.576 I/python  (27304): [INFO   ] [Text        ] Provider: sdl2
06-25 19:27:23.186 I/python  (27304): [INFO   ] [GL          ] NPOT texture support is available
06-25 19:27:23.466 I/python  (27304): [WARNING] [Lang        ] constants.kv has already been included!
06-25 19:27:24.386 I/python  (27304):  2019-06-25 19:27:24.396 | INFO     | __main__:build:107 - Build started
06-25 19:27:24.416 I/python  (27304): [WARNING] [Base        ] Unknown <android> provider
06-25 19:27:24.416 I/python  (27304): [INFO   ] [Base        ] Start application main loop
06-25 19:27:24.426 I/python  (27304): [INFO   ] [Base        ] Leaving application in progress...
06-25 19:27:24.796 I/python  (27304): [INFO   ] [WindowSDL   ] exiting mainloop and closing.
06-25 19:27:24.926 I/python  (27304): Python for android ended.

I was facing the same problem and did the changes in your commit. My app is now being paused instead of exit. However, my main app is still not working :/


Refactor: Rename BlockType to Block and Block to BlockResponse

The enum previously named BlockType was used to manage block types, and the old Block struct represented the common parts returned from the Notion API, with BlockType as one of its fields. Since this naming was unclear, the following changes have been made:

Renamed struct Block to struct BlockResponse
Renamed enum BlockType to enum Block

From now on, the API will use these new names.

I will self-merge with repository owner privileges.


Possible to use also TB Lists

First I may thank you for this very useful add-on!
I have setup a list of 4 persons in my TB contacts. Can this list be used for fast forward? I try but for me don't work.
Thank you!

Good point!
This is fixed now in 1.0.2, should be published soon once Mozilla is done reviewing it. In the meantime you can directly install the XPI, if you want: https://addons.thunderbird.net/thunderbird/downloads/file/1026850/one_click_forward-1.0.2-tb.xpi?src=devhub


How to open settings?

I can't seem to open the settings pane? I've tried opening the app through spotlight and finder, but nothing happens. The app is showing notifications as expected. Also note that the app icon is NOT visible in either my dock or menu bar.

macOS Sequoia 15.2
spotify-notify v0.5.1

I experience the same issue on my end (same OS and app versions).


Theme Lost on Firmware Upgrade (Flash)

Hi,
Most settings are preserved when a new image is flashed, but it seems the theme (Design) setting is lost. Is this expected?
Thanks!

the theme (Design) setting is lost. Is this expected?

Theme is not just a setting, but an installable package
Losing is

expected, if you install a release or snapshot build and you have not used Bootstrap, the default theme.
unexpected if you have built a personal firmwre image where the theme is built-in.

Hi,
It's a personal build, where the theme is built-in ... so the unexpected case 😆. I agree with you, if an installed packaged, this would / should go away (but in this case, seems it shoudl stay).
Thanks!


Time Out Error When Installing stash-code-plugin

Hi,
I have built the project and tried to install the jar on stash 3.5.1 and 3.11.1. Tried both the version i get the following error.
Please could you suggest how we to solve this error.
2016-09-16 09:59:10,111 ERROR [UpmAsynchronousTaskManager:thread-3] venkatasrinadh @1T11LC6x597x81x0 1rxhr0s 0:0:0:0:0:0:0:1 "POST /rest/plugins/1.0/ HTTP/1.1" o.s.o.e.i.u.c.RunnableTimedExecution Closing runnable for context NonValidatingOsgiBundleXmlApplicationContext(bundle=com.palantir.stash.stash-code-search, config=osgibundle:/META-INF/spring/*.xml) did not finish in 10000ms; consider taking a snapshot and then shutdown the VM in case the thread still hangs
2016-09-16 09:59:10,130 ERROR [UpmAsynchronousTaskManager:thread-3] venkatasrinadh @1T11LC6x597x81x0 1rxhr0s 0:0:0:0:0:0:0:1 "POST /rest/plugins/1.0/ HTTP/1.1" c.a.plugin.manager.PluginEnabler Unable to start the following plugins due to timeout while waiting for plugin to enable: com.palantir.stash.stash-code-search

found the problem, it with version of elastic search that i am using.. works  fine on 1.3.3


Purpose of PropTypes and Interfaces

Looking through the Button source, I see that there is a Typescript interface for it's props as well as redundant usage of the PropTypes package.
Is there a reason for using both?

PropTypes are useful for run-time checks for expected props and are particularly useful for JS consumers
TypeScript interfaces offer compile time checks and all the goodies of a compiled code (easy access to available API, type safety, auto-complete, etc.)
@levithomason and @mnajdova might have more insight on how to unify these moving forward

Why have PropTypes?

A small subset of the world currently uses TypeScript.  We hope that changes in the coming years.  PropTypes exist primarily for the majority of those users who don't have any other type safety.

they can do cool things with values, like this:

Value checks cannot be done in TypeScript.  You also cannot get help at this level ^ with TS.  Even when using typescript, there are cases where this kind of thing comes up at run time.

Currently our docs and component info files are generated from the PropType info.  That AST crawling logic needs to be moved to the crawling the TS interfaces before we can remove them.  Otherwise, we can't generate docs or run our automated conformance tests.

I was going to say propTypes are stripped in production, however, it looks like we lost that feature at some point after forking off of Semantic UI React 😕 We should fix that...

Why remove them?

It is a pain to maintain both.  We could generate simplified typings from our interfaces someday.  Perhaps we will.  Until then, we have both.

Would love to hear other's thoughts and input on this as well.

Closing for housekeeping.  Let us know if there are still questions around this and we'll answer them.


Temporarily turn off NB.GV on Mac

Until https://github.com/AArnott/Nerdbank.GitVersioning/issues/123 is fixed.

@KirillOsenkov,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Codecov Report

Merging #3 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #3   +/-   ##
=======================================
  Coverage   83.63%   83.63%
=======================================
  Files          70       70
  Lines        5333     5333
  Branches      834      834
=======================================
  Hits         4460     4460
  Misses        647      647
  Partials      226      226

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e6d5eac...c7acd07. Read the comment docs.

@AArnott how about this one please? ;)

To be honest, I'd rather take the 30-60 minutes it would require to fix NB.GV. But I might not get to it before this weekend. Will that be a problem for you?

No, not really. It can wait.

Please keep the branch though since I'm going to include this branch in the workaround I provide to the Mono runtime devs.

@KirillOsenkov the branch is in your fork. It's not going anywhere unless you delete it. :)


Setting dialog audio v2

Purpose
Second iteration on the Audio tab under settings
Proposal
Added the active speaker indicator to help users easily determine if their mic is active. Refactored the layout to match Gmeet's design and improved the responsiveness of several elements.

Improvements I suggest for the V3 https://github.com/numerique-gouv/meet/issues/142


Fix memory management in tests

For each run of each test is created instance of XCTestCase. But that instance is not deallocated until whole XCTestCase is finished.
So it’s necessary to cleanup resources manually in the tearDown()  method or in the tearDownWithError() method.
And currently this is not done in majority of tests. CompactBlockProcessor has internal timer which periodically execute sync process. And when instances of CompactBlockProcessor aren't removed from memory on end of each tests these instances are piling. And sooner of later of of those instances fires it's internal timer and execute sync. And two sync processes are in progress in paralel. Which shouldn't happen because the SDK isn't designed for this situation and it cause mayhem.

Fixed in #797


Add the list of removed events to migration guide

Description

Add the list of removed events to migration guide, and improve the readability of the event table in readme.md

Checklist

[ ] Does npm test pass with this change (including linting)?
[ ] Is the new or changed code fully tested?
[ ] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

The PR only changes some documents.

@simonprickett THX a lot! I merged the changes.


Request for release into maven repository

Appreciate if you can consider release your jarjar libraries into the official maven repository, so aTalk can refer to it.
aTalk release the apk into android playstore and Fdroid. Fdroid has restriction on inclusion of any pre-built jar into the source.
aTalk is an open source: https://github.com/cmeng-git/atalk-android
Currently aTalk build.gradle is making reference to 'org.anarres.jarjar' jarjar, but seems this tool has not been updated for a while and is facing some problem i.e. NullPointerException in LambdaClass.createLambdaImplMethodTarget
=================
buildscript {
repositories {
google()
jcenter()
}
dependencies {
    classpath 'org.anarres.jarjar:jarjar-gradle:1.0.1' // working jarjar.repackage version
}

}
apply plugin: 'org.anarres.jarjar'

@shevek we'd also love to get a new release of your plugin. We need to pick up ASM8 support upgrade
Please release 1.0.2 from current master.

I hope this will help you https://github.com/shevek/jarjar/pull/20 but there is current no publish included

Thanks for the new input. Would you be able to provide an example on how to use the jarjar 1.0.3 in gradle.
When I change the aTalk build.gradle to
buildscript {
repositories {
google()
mavenCentral()
}
dependencies {
    classpath "org.anarres.jarjar:jarjar-gradle:1.0.3"
}

}
Gradle sync immediately exit with the following errors:
A problem occurred configuring project ':buildSrc'.

Could not resolve all artifacts for configuration ':buildSrc:classpath'.
Could not find org.anarres.jarjar:jarjar-gradle:1.0.3.
Searched in the following locations:
- https://dl.google.com/dl/android/maven2/org/anarres/jarjar/jarjar-gradle/1.0.3/jarjar-gradle-1.0.3.pom
- https://repo.maven.apache.org/maven2/org/anarres/jarjar/jarjar-gradle/1.0.3/jarjar-gradle-1.0.3.pom
- https://plugins.gradle.org/m2/org/anarres/jarjar/jarjar-gradle/1.0.3/jarjar-gradle-1.0.3.pom
Required by:
project :buildSrc

===============
For jarjar v1.0.1 it was released in the maven repository, hence there is no problem when v1.0.1 is used.
https://mvnrepository.com/artifact/org.anarres.jarjar/jarjar-gradle
I also made refer to the following file, but none help.
https://github.com/shevek/jarjar/blob/master/jarjar-gradle/example/build.gradle

@shevek Any chance to have an update released soon? Especially in the light of #22

I tried on aTalk project but facing problem. Any advice will be appreciated.
Unable to use shadow to achieve the same result as jarjar tools for aTalk project #761


[Feature Req] ServiceBusMessageSender should refresh maximum message size when service settings change

Library name and version
Azure.Messaging.ServiceBus
Describe the bug
When using a ServiceBusMessageSender instance while there's a change to the entity's maximum message size, the Batch API doesn't allow message adding despite the message being within the allowed size. The client holds on to the old maximum message size it has retrieved from the entity, which is confusing.
Expected behavior
Messages that previously couldn't be sent to be sent w/o the need to restart the process.
Actual behavior
The process has to be restarted to get the updated max message size.
Reproduction Steps
On a premium namespace, a queue of size S1. Start a process that sends a message larger than S1.
Increase the entity maz size to S2 (S2 > S1). Adding messages to a batch messageBatch.TryAddMessage(message) be unsuccessful.
Environment
No response

Hi @SeanFeldman.  Thanks for letting us know.   The current implementation is based on the now outdated service limitation that the maximum message size was based on the SKU and was not configurable.  As the service feature set changed, this was overlooked in the client.
There's a bit of a tricky balance here.  I don't think that we want to force every call to CreateMessageBatchAsync to be checking the link for the unlikely case that the message size could change.   I do think that it would be reasonable to reevaluate the maximum message size on send, where we're already network bound.   This would mean that the client won't refresh in real-time but would adjust before each send operation.  I think that's a reasonable trade-off to avoid impacting performance each time a batch is created, but I'd appreciate your thoughts.
Regardless of how the client adjusts, we'd also need a service-side feature for this to work.  Currently, the entity does not detatch existing AMQP connections/links when settings change.   Since the maximum message size is communicated to the client via metadata on the link, for the client to be aware of changes, it needs to open a new link on which the service has set that updated metadata.   I've opened #705 for the Service Bus team.
In the meantime, the proposed client changes are safe to make, so we'll look at getting them in.

@jsquire, that's a good approach 👍

Client portions addressed in #44328.    It should help a bit, but won't be fully effective until the service forces client disconnection when the size is updated.

Changes merged and will be available in our next release.