Official NPO apps have livestreams for NPO 1, NPO 2 and NPO 3

Would be cool to have them in here. Not sure how hard that would be though.

Should be relatively easy to add. We've implemented them before for Roku here: https://github.com/roonie/roku-npo-nederland1

This would be a welcome addition, but I don't think I'll get around to implementing it. I'll accept a pull request for this, if somebody else wants to get their hands dirty.
Maybe a showcaseTemplate would be a good fit; https://developer.apple.com/library/prerelease/tvos/documentation/LanguagesUtilities/Conceptual/ATV_Template_Guide/ShowcaseTemplate.html#//apple_ref/doc/uid/TP40015064-CH27-SW1

I made an Apple TV app for this: https://github.com/Mauricevb/NPO-Live-Apple-TV-4

Hey @Mauricevb, awesome work!
@Sjoerdjanssenen, @johanb and @marijnvdwerf have been working on getting live streams into this app in #15, but I think it might be a good idea to move efforts on that to your project.
That way, we don't have to reinvent the weel (although that would now be easier by looking at your code), and we'd have a separate project and a separate maintainer for the live TV app, which should make development faster (I don't watch live TV, so I wouldn't notice if anything breaks) and maintaining both projects easier.
Also, I think having these separate makes more sense. I'd even try to make a separate app for each stream, so you can select it directly from the home screen, but that'd probably make it cumbersome to get everything side loaded on devices, so I think this is our best bet for now.
So, everyone, what do you think?

I wouldn't want a separate app for each channel. Especially not before apple TV gets folders. I would like the ability to switch channels while watching another channel without needing to go back to the menu (swiping left / right for example)
But it does make sense to have separate app for live and gemist.


[FEATURE] Could you please share, how the dataset visualization image was generated as shown in the following screenshot from Cellpose 1.0 paper?

Please see #666.


Put all symmetricizing function into class & documented it

Let me know if I should put more documentation. Also updated requirements file to include packages installed.

Alright, so I pushed in the changes. Chiefly among them, I totally rewrote the Symmetricize.py file to be more OOP-like and have an init method. I also split TestingNotebook into two: one for testing just Symmetricize.py and one for my development. Lastly, I removed the little extra blurb in requirements_rtd.txt


feat: 7 bit long note type id

Making note type id 7 bits long.

#10899  👈 (View in Graphite)
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Closing in favor of https://github.com/AztecProtocol/aztec-packages/pull/10951
After discussions with Nico does not make sense to tackle the optimization of note type id encoding yet. Opened a new PR such that the work on the encoding does not get completely lost.


Generating tokens is not convenient

Just installed the docker image and I wanted to create a token right inside my config files for local docker dev env but the fact that
docker exec -u warp10 sencrop-warp10 sh -c 'warp10-standalone.sh worf test<PHONE_NUMBER>0'

returns several lines
default options loaded from file:/opt/warp10/etc/.conf-standalone.conf.worf
{"read":{"token":"***","tokenIdent":"****","ttl":31536000000,"application":"test","applications":["test"],"owners":["***"],"producer":"f****","producers":[]},"write":{"token":"****","ttl":31536000000,"application":"test","owner":"*****","producer":"*****"}}

is not nice. I think that the default options loaded from file:/opt/warp10/etc/.conf-standalone.conf.worf text should not appear to easily load the JSON or maybe be outputted in stderr.
FYI, here is what I had to do do parse the JSON data:
const WARP10 = JSON.parse(
  execSync(
    "docker exec sencrop-warp10 sh -c 'warp10-standalone.sh worf sencrop<PHONE_NUMBER>0'"
  )
    .toString()
    .split(/\r?\n/g)[1]
);

The fact that tokens has to be generated is not convenient too for a quick start. I like how the MySQL docker image exports the MySQL PWD on an environment variable, maybe that something similar can be done here behing an env var or an option flag.
See: https://hub.docker.com/_/mysql/
And how I can use it in my config scripts:
const MYSQL_PWD = execSync(
  "docker exec sencrop-mysql sh -c 'echo $MYSQL_ROOT_PASSWORD'"
)
  .toString()
  .trim();

Hope it helps

Tokengen is now the preferred mode


New UI and UX need 💡::🎨 Request for Major UI Changes 🚀

### Description:
The current user interface (UI) of our application needs significant enhancements to improve usability, accessibility, and overall user experience.
Required Changes:

Implement a new navigation system with simplified menus or sidebar navigation.
Update the visual design with a modern color scheme and typography.
Ensure responsive design across all devices, prioritizing mobile optimization.
Enhance accessibility features for improved inclusivity.
Streamline workflows by redesigning UI elements based on user feedback and analytics.

Let's work together to transform the UI of our application and deliver a more engaging and user-friendly experience for our users. 🌟

Required Team for Developing the UI/UX for Major Changes , Intrested !? Reply to this Issue.

Assign this issue to me
As a full stack Developer i can write optimized code and responsive and i can integrate with backend also.

Sir, I can solve the issue for the frontend ui/ux in a short period of time.
I have studied about typography and graphic design so it's easier for me to integrate amazing color sense into the website to make it look magical.
I will also keep in mind the various types of users that may visit the website so everyone's specific needs are fulfilled.

@PatilHarshh Please could you assign me this issue under GSSoC'24

@Shyambarua @r7projects-shayan @Prithwi32  Can you Share the New UI for our site ? it will easy for me to decide further !

@Shyambarua I will share it with you soon.

@PatilHarshh Assign this issue to me for update UI if you have any figma please provide me I have done it

Hi @PatilHarshh
You have correctly identified the need of redisgning UI/UX for better usability of the website.
The requirements listed by you are completely on point to accomplish the goal of a better User Experience. I think I will be able to handle all these correctly and properly as I have previously developed projects which includes all these things.
How will I do it?

Navbar Redesign: A better looking and completely responsive Navbar (with drop-menus on phone width).
Redesigning of Color Theme and Typography to make it more subtle and match with the theme of Job Seeking and Hustling Culture.
Redisgning the boxes of Jobs Posted to make them look more appealing and updating them in a way so that those boxes contain more useful info in a concise manner.
Updating and resdisgning the Job Uploading forms: converting it into a multi-step form which will enhance the UI by a mile and also make it seem like less over-whelming for the person who is filling it.
Complete redesign of components related to Company Profiles so that Job seekers get better knowledge about the Company which they are applying in.
Inclusion of Google Translate to make the website usable accessible for more number of Job Seekers.
Redesign of About Page to clearly highlight the Goal of this website and how is it different than the others. (an interactive About Page)

All the things mentioned above will fulfill the requirements mentioned by you.
Please assign me this issue so that I can starting working on it ASAP. (under GSSOC'24)
Looking forward to a positive response.

@r7projects-shayan I appreciate your efforts but the design created by you does not meet industry standards at all and would not be good according to the User Experience as well.
Just a feedback, please don't take it in an offensive manner. 🙂

@rikhiltaneja Sure, I will recreate it, thanks for the feedback

Hi Admin,
I'm interested in contributing to your project as part of GSSoC. I've been following your work and believe my skills align well. You can find my portfolio here: (https://sites.google.com/view/uibymehtab) and LinkedIn (https://www.linkedin.com/in/mehtabarkhan/)
Thanks,
Mehtab

Hi Admin,
I'm interested in contributing to your project as part of GSSoC. Please assign this task to me and also add the level tag to this issue.

This can be further modified according to the specificness . Kindly assign this to me.

This can be further modified according to the specificness . Kindly assign it.

This can be further modified according to the specificness . Kindly assign it.

@Caliber-Ruchi  Not bad ! @r7projects-shayan @rikhiltaneja   Can we move to this and modify more to this or you have your design ready ?

@PatilHarshh yes you can move to the new design

@PatilHarshh , design by @Caliber-Ruchi is great! We can proceed with this and can design all other pages on the same theme.
You can assign some of the partcipants (of your choice) of this issue and then all the assignees will divide the work themselves and raise PRs.

Hey @PatilHarshh  , I am Sourav Mohanta , 3rd year Student of NIT Durgapur, I have previously contributed to p5js (Processing Foundation) ,
I understand the issue and work on this challenging issue , Plz assign it to me

If this issue is still pending, I can work on this under GSSoC'24. Lemme know :)


Is it possible to configure Commment.nvim to add a new comment on a new line

Assuming I have the following:
// one
// two
// three
foo
bar

and my cursor is on 'three', then I would like to be able to press o and for every new line created in insert mode below this one to be automatically-commented. If, however, I want to add an uncommented line after // three then I just move to foo and press O to create an uncommented new line in insert mode above foo. This is commenting behaviour that I'm used to, and it's a lot less work than pressing gco > type comment esc``gco > type comment esc``gco > type comment.
Is this currently possible in Comment.nvim?

≥ then I would like to be able to press o and for every new line created in insert mode below this one to be automatically-commented. If, however, I want to add an uncommented line after // three then I just move to foo and press O to create an uncommented new line in insert mode above foo.
This is vim native behaviour. I don't understand what do you exactly want?

Oh didn't realise it was native behaviour. For some reason this behaviour doesn't occur for me. Assumed it was due to Comment.nvim. I need to figure out what's causing this. Perhaps it's a vim option. I'll update once I've done some more investigation.


OnBeforeClose does not run till end

I added OnBeforeClose event handler similar to
https://github.com/chromelyapps/Chromely/blob/81b8231cf4321c05e2689be4407c3c85076f68db/src/Chromely/ChromelyEventedApp.cs
But when chromely is closed the method OnBeforeClose() does not run till end if code in the event hadler take some time to execute. You can catch it by adding
    protected virtual void OnBeforeClose(object sender, BeforeCloseEventArgs eventArgs)
    {
          Thread.Sleep(10000); //Or Task.Delay(10000).Wait();
          Console.WriteLine("Closing");
    }

Console.WriteLine() will never be executed
This is because of parallel execution of closing window and OnBeforeClose event at this line
https://github.com/chromelyapps/Chromely/blob/81b8231cf4321c05e2689be4407c3c85076f68db/src/Chromely/Windows/Window.cs#L70
If make it synchronous then Dispose() ends after OnBeforeClose() event and all works as expected

@xplicit great catch!
It's been an issue that has been bugging me for a while, and never had the chance to properly zoom in on it. At some point I thought it was just happening locally for me.
I will see about making the change over the weekend.
Thanks.

It would be great, if we can to cancel closing inside this event. For example, user press Alt+F4, we cancel the closing and ask user ("You have unsaved work, do you really want to close?" [OK], [Cancel]).

@xplicit  I won't advise that. CEF processes are tricky - even this OnBeforeClose approach for now is a hack  - https://github.com/chromelyapps/Chromely/blob/6b02086ba3360d3baa706ebddb21158b3d85d8da/src/Chromely/Windows/Window.cs#L74
It was discovered late after the release, and I have personally not have enough time to dig deeper because a fix in Windows may break Linus/MacOS and vice-versa. It needs time.
What you are trying to do I will suggest you do in the native host but that will involve creating a custom one.
I think a more comprehensive  OnClose  feature should be added for a major release.

Thanks, your suggestion helped to resolve the issue with properly tracking "close" event. We use Close event from WinAPIHost - it's called synchronously. If we need to cancel the event, we can override WndProc() for now.


add option to prevent additional arg quoting

#837

will add documentation as needed

forgot vim-async part

What about not quoting strings, but only arrays?  (i.e. if args is already a string leave it as-is)

i'm just concerned about other makers, I tried something like that and broke vint

I think some makers depend on the extra quoting, vint for example

I can't find a way to get vim-async to run my js2 maker.

Hmm, vint returns an array for args?
To have it return a string you would have to change it:
diff --git i/autoload/neomake/makers/ft/vim.vim w/autoload/neomake/makers/ft/vim.vim
index e6d2e8e..00e719a 100644
--- i/autoload/neomake/makers/ft/vim.vim
+++ w/autoload/neomake/makers/ft/vim.vim
@@ -13,7 +13,7 @@ function! neomake#makers#ft#vim#vint() abort
     endif

     return {
-        \ 'args': l:args,
+        \ 'args': join(map(l:args, 'shellescape(v:val)')),
         \ 'errorformat': '%I%f:%l:%c:style_problem:%m,%f:%l:%c:%t%*[^:]:%m'
         \ }
 endfunction

And as for the change I had in mind:
diff --git i/autoload/neomake.vim w/autoload/neomake.vim
index b0b7b97..82f8964 100644
--- i/autoload/neomake.vim
+++ w/autoload/neomake.vim
@@ -121,12 +121,16 @@ function! s:MakeJob(make_id, maker) abort
     else
         let args = a:maker.args
     endif
-    let append_file = a:maker.file_mode && index(args, '%:p') == -1 && get(a:maker, 'append_file', 1)
-    if append_file
-        call add(args, '%:p')
-    endif
+    let args_list = type(args) == type([])

-    call neomake#utils#ExpandArgs(args)
+    if a:maker.file_mode && get(a:maker, 'append_file', 1)
+        if args_list
+            call add(args, '%:p')
+            call neomake#utils#ExpandArgs(args)
+        else
+            let args .= ' '.fnameescape(expand('%:p'))
+        endif
+    endif

     if has_key(a:maker, 'cwd')
         let old_wd = getcwd()
@@ -144,14 +148,14 @@ function! s:MakeJob(make_id, maker) abort
     endif

     try
-        let has_args = type(args) == type([])
         let error = ''
         if neomake#has_async_support()
-            let argv = [exe]
-            if has_args
-                let argv += args
+            if args_list
+                let argv = [exe] + args
+            else
+                let argv = exe . (len(args) ? ' ' . args : '')
             endif
-            if has('nvim')
+            if has("nvim")
                 let opts = {
                     \ 'on_stdout': function('neomake#MakeHandler'),
                     \ 'on_stderr': function('neomake#MakeHandler'),
@@ -228,14 +232,17 @@ function! s:MakeJob(make_id, maker) abort
             let r = jobinfo.id
         else
             call neomake#utils#DebugMessage('Running synchronously')
-            if has_args
-                if neomake#utils#IsRunningWindows()
-                    let program = exe.' '.join(map(args, 'v:val'))
+            let program = exe
+            if len(args)
+                if args_list
+                    if neomake#utils#IsRunningWindows()
+                        let program = exe.' '.join(map(args, 'v:val'))
+                    else
+                        let program = exe.' '.join(map(args, 'shellescape(v:val)'))
+                    endif
                 else
-                    let program = exe.' '.join(map(args, 'shellescape(v:val)'))
+                    let program .= ' '.args
                 endif
-            else
-                let program = exe
             endif

             call neomake#utils#LoudMessage('Starting: ' . program)

Tests are passing (make docker_test DOCKER_VIM=vim-master), but the Windows case is not covered.
Should I create a new PR based on that, or do you want to pick it up from here?

I am kind of unsure how we should do this.
jobstart({cmd}[, {opts}])				{Nvim} *jobstart()*
		Spawns {cmd} as a job.  If {cmd} is a |List| it is run
		directly.  If {cmd} is a |String| it is processed like this: >
		  :call jobstart(split(&shell) + split(&shellcmdflag) + ['{cmd}'])
<		NOTE: read |shell-unquoting| before constructing any lists
		with 'shell' or 'shellcmdflag' options.  The above call is
		only written to show the idea, one needs to perform unquoting
		and do split taking quotes into account.

from what i can see all args are being additionally quoted, here is debug with vint:
Neomake [3.268]: Starting: vint '--style-problem' '--no-color' '-f' '{file_path}:{line_number}:{column_number}:{severity}:{description} ({pol icy_name})' '/Users/mac/.vimrc'
I don't know whether or not that is fine with some shell syntax, but the maker using emacs failed when it had those quoted ( and also failed with your patch ).

As for vint this is fine (and worked for me).
Only the argument to -f needs to be quoted really, but it's not meant to use a string for vint anyway - I was just trying it out.

the maker using emacs failed when it had those quoted ( and also failed with your patch ).

Can you construct a minimal maker where it fails?  Would be good to use just echo or something more portable than emacs for it.
You are not using Windows, are you?

I use osx and bash
maybe this helps?
let g:neomake_echo_maker = {
      \ 'exe': 'echo',
      \ 'args': ['in','array','in array'],
      \ 'errorformat': '%f %m',
      \ 'mapexpr': 'expand("%:p") . " " . v:val',
      \ }

Neomake [3.470]: [1] Running makers: ['echo']
Neomake [3.472]: [1] Starting async job: ['echo', 'in', 'array', 'in array', '']
Neomake [3.476]: [1.1] stdout: echo: ['in array in array ', '']
Neomake [3.477]: [1.1] exit: echo: 0
Neomake [3.477]: [1.1] echo: processing 1 lines of output.
Neomake [3.478]: Could not place signs for 1 entries without line number.
Neomake [3.479]: [1.1] echo: completed with exit code 0.
Neomake [3.479]: File-level errors cleaned in buffer 1

I think I get your patch now. would it be able to expand '%:p' in the string?

@bounceme
Not sure if I get your question, but the idea is to only expand %:p if args is a list, otherwise you would have to to it when returning the string.
btw: https://github.com/neomake/neomake/pull/845

    if a:maker.file_mode && get(a:maker, 'append_file', 1)
        if args_list
            call add(args, '%:p')
            call neomake#utils#ExpandArgs(args)
        elseif args =~# '%:p'
            let args = substitute(args,'%:p','\=expand(submatch(0))','g')
        else
            let args .= ' '.fnameescape(expand('%:p'))
        endif
    endif

This is straight from my patch, right?
I'll create a PR based on that, and we work from there.

In case you have not seen it already, from Vim's :h job_start:

{command} can be a String.  This works best on MS-Windows.  On
Unix it is split up in white-separated parts to be passed to
execvp().  Arguments in double quotes can contain white space.
{command} can be a List, where the first item is the executable
and further items are the arguments.  All items are converted
to String.  This works best on Unix.

@bounceme
Please try https://github.com/neomake/neomake/pull/848.


AAD Authentication with Guest Accounts

Azure Data Studio Version: Version 1.3.4-insider (1.3.4-insider)

Steps to Reproduce:

Use an account from a different AAD that is a guest member of the directory for the database.
Attempt to login to the database.
Returns NT AUTHORITY\ANONYMOUS LOGIN error.

This is resolved in SSMS by the option to specify the AD Domain Name or TenantID in the connection properties tab.

@aaomidi  do you have any context on this request and whether your recent auth updates will help here?

I think so - will validate once the work is complete.

So I think this issue has been covered by the auth updates, and the changes Matt made here.


Check for X-Powered-By ( X-Generator, ... ) headers

Potential attacker can use it to fingerprint the server and target attacks accordingly.

Another header is "server" which should be checked too.


4.x: Upgrade graphql-java

In Helidon 2.x and 3.x we are on the graphql 17.x branch.
In 4.x we should take the opportunity to upgrade to a newer branch. As of this writing the latest release is 20.3 (https://github.com/graphql-java/graphql-java/releases)

Trying with with following version:
<version.lib.graphql-java>20.3</version.lib.graphql-java>
<version.lib.graphql-java.extended.scalars>20.2</version.lib.graphql-java.extended.scalars>

I'm upgrading to graphql-java latest, 20.3, and seeing the tck failing on the errorHandling tests because the error messages have changed. There are also some changes in graph-ql java as well.
As an example in unknownMutation.log, previously the tck expected
"message": "Validation error of type FieldUndefined: Field 'createNewHeroCat' in type 'Mutation' is undefined @ 'createNewHeroCat'",
But with latest graphql-java i see
"message": "Validation error (FieldUndefined@[createNewHeroCat]) : Field 'createNewHeroCat' in type 'Mutation' is undefined"

This requires a change to the mp-graphql tck to allow multiple outputs.

Created issue on mp-graphql https://github.com/eclipse/microprofile-graphql/issues/520

Create issue to do 20.x upgrade #6963 , this issue will upgrade to latest 18.x

Graphql has been upgrade to 18.6 in 4.x. We will use the issue mentioned above to track upgrade to 20.x.


Question regarding file naming

Could you please clarify the best practice for naming the files - is the prefix/suffix is needed in the filename?
If I have prefix "XYZ" specified in the AppSourceCop.json and I'm creating "Sales Header" table extension what should the filename be? XYZSalesHeader.PageExtension.al or just SalesHeader.PageExtension.al?
The documentation states that it should be "MyPrefixSalesperson.PageExt.al" - this looks like the naming convention that was introduced after release of AL but it was changed later and now it's reverted - is this correct?

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

ID: 3c7b7057-f9c5-6587-5d8b-267935685262
Version Independent ID: 11a70e81-f0a2-ec66-d63e-d1b0b5edeb5c
Content: Best Practices for AL code - Business Central
Content Source: dev-itpro/compliance/apptest-bestpracticesforalcode.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

I think #1077 was the wrong descission.
It only needs a reference to AppSourceCop.json settings.
The MS default apps also ignores Suffix and Prefix in file names (one sample):
https://github.com/microsoft/ALAppExtensions/blob/master/Apps/DE/Elster/app/src/Codeunits/Management.Codeunit.al
File names with suffix/prefix makes the projects more unreadable.
Maybe @JesperSchulz can deliver a clear response from the developer team here.

Is it possible to get an answer here? This quetion is now over three months old.
The closed Issue #1495 asked the same.
The article for AA0215 refers to this article https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/compliance/apptest-bestpracticesforalcode#file-naming
But the code cop rule still allows to remove the prefix/suffix from the filename. Is there any technical reason why we should add the prefix/suffix to the filename?
The guideline was only changed on the issue request from @JulianTillmann in Issue #1077:

The CodeCop rule AA0125 Forces the Suffix or Prefix in the filename, here it's explicitely removed from the filename: .al

But the CodeCop Rule AA0215 never forced the suffix/preffix, it only wants the objectname in the filename. If the prefix/suffix is declared in the AppSourceCop.json a filename without prefix/suffix is no longer a warning in the code cop analyser.

Sorry, I didn't not monitor this repository! Let me see if I can dig out an answer for you :-) We've been back and forth on file naming. Need to get up to speed with the latest best practices on those.

I understand the confusion - the information on file names is not perfectly clear. Let's start with the facts: what you name your files, is entirely up to you. If you want to know about the rules in general you need to follow before submitting apps to AppSource, please check here: aka.ms/checkbeforeyousubmit. That list does not mention anything about file names and file names hence don't need to follow any rules.
We do have best practices though and those can be checked by CodeCop. From what I can tell, CodeCop only suggests that the object name is part of the file name - which indeed is a good best practice. Whether you want to add any affix to the file name is up to you. That is not what our documentation says though, and we should adjust that.
I hope that clarifies things a little? You decide on the file names yourself. If you want to follow best practices, include object name in file name. Whether you want affixes in the files names, is your call. If you don't care about our best practices at all, feel free to disable CodeCop rule AA0215.
@SusanneWindfeldPedersen, could you update the docs?

Thanks for the feedback and discussion. This topic will be updated with the next doc refresh.


Little docu bug

Too small for a PR (afraid to make sth wrong):
~/Documents/GitHub/attrs/docs $ git diff
diff --git a/docs/extending.rst b/docs/extending.rst
index 11f2a74..7b185e8 100644
--- a/docs/extending.rst
+++ b/docs/extending.rst
@@ -13,6 +13,7 @@ So it is fairly simple to build your own decorators on top of ``attrs``:
    >>> import attr
    >>> def print_attrs(cls):
    ...     print(cls.__attrs_attrs__)
+   ...     return cls
    >>> @print_attrs
    ... @attr.s
    ... class C(object):

Thank you! In such simple cases you can just just press the edit button on the files. :) It's the CI's job to make sure nothing breaks.


MarkovChain: state_values related

Change

(simulate, simulate_values) to (simulate_indices, simulate)
(simulation, value_simulation) to (simulation_indices, simulation)?

For the new simulate (current simulate_values), init should be specified as a state value, not index. Need a method to return the index in state_values given a state value (for the Python version, see https://github.com/QuantEcon/QuantEcon.py/pull/237/commits/5bc78d5706462cde82dcba09dfec114c2a3e4680).
Let recurrent_classes and communication_classes return state values, and add *_indices.

simulate_values!(mc::MarkovChain, init_state::Vector{Int}, X::Matrix) currently works as follows:
julia> mc = MarkovChain([0 1; 1 0]);

julia> X = Array(Int, 5, 2);

julia> init_state = [1, 2];

julia> simulate_values!(mc, init_state, X)
5x2 Array{Int64,2}:
<PHONE_NUMBER><PHONE_NUMBER>
          2           1
          1           2
          2           1
          1           2

It should be better to fill X[1, :] with the initial state values.

@oyamad I'm confused.  What's the difference between simulation and simulate?   It seems confusing to have both for instances of MarkovChain.

simulation returns a Vector, while simulate returns a Matrix (even when num_reps=1) (https://github.com/QuantEcon/QuantEcon.jl/pull/99#issuecomment-197412344). I agree it can be confusing, but I have no good idea for alternatives.

Thanks.  I guess I knew that and forgot.  Let's open this up to discussion.
First, I like @oyamad 's plan to unify names with what we have in Python.  The Python conventions are simple and clear.  In Python we only have simulate and simulate_indices, each of which generate a single time series.  So my preference is:

Change (simulate, simulate_values) to (simulate_indices, simulate) and have them return flat arrays.

Drop simulation and value_simulation.

To clarify, in the Python version, simulate (and simulate_indices) returns a 1d array if num_reps is not supplied, and a 2d array otherwise (http://quanteconpy.readthedocs.io/en/latest/markov/core.html#quantecon.markov.core.MarkovChain.simulate). We need a clever idea if we want to follow this in the Julia side with one function name while keeping type stability.

I see.
How about simulate gives a single chain and returns a 1d array, while simulate_panel generates a 2d "panel"?
(This corresponds to the meaning of panel data)
Similarly, simulate_indices gives indices in 1d, while simulate_panel_indicies gives the 2d version?

Partly related question:  Is this line
https://github.com/QuantEcon/QuantEcon.jl/blob/master/src/markov/mc_tools.jl#L426
with init_state::Int we are passing an index.  In the Python version we pass a value.
I guess both are OK as long as the documentation is clear.

@albep @vgregory757 If either or both of you are able to contribute to finishing the debate and closing out this issue I would be grateful.
Apart from simulate_panel, simulate_cross_section could also be a good option.  It's a bit verbose but I think the meaning is clear from the name.
Whatever decisions are made this will affect the lecture site and applications repo, so please coordinate with me.
Thanks, J.

Am I correct in saying that in the Julia simulate function we need an actual Matrix to be returned if either of 3 things happens: i) a vector of initial conditions is supplied (but num_reps = 1); ii) only 1 initial condition is supplied but num_reps > 1; iii) a vector of initial conditions is supplied and num_reps > 1. I'm looking at this line https://github.com/QuantEcon/QuantEcon.jl/blob/master/src/markov/mc_tools.jl#L315. My question then is: when are cases i) and iii) relevant? Otherwise we could use only 2 functions (simulate and simulate_indices) that return a flat array or a 2d array based on 2 methods, depending on whether init is a number or an array - where the array corresponds to cases when we want more than 1 repetition. This cannot handle on its own situations where we want some assigned initial conditions to be repeated multiple times, but we can just generate the appropriate initial conditions outside of the function, and then apply the method on that vector of initial conditions

I agree that the current setup of the functions is confusing and that the names simulation and value_simulation are unclear. I like @albep's suggestion -- would it mean that there would be no num_reps argument? If so, one downside to that might be that it's departing from the way the Python code is set up.
Whatever we decide, I'd be happy to help make any changes to the functions/existing code that calls them.

@spencerlyon2 probably knows a bit more about what is happening behind the scenes, so he can comment on this thought (and probably improve it).
I agree with John with respect to having the exposed function names match up to the Python code. It might not be that big of a deal to make the simulate function consistent in its return type if the returned value is generated within a different function (which does consistently deal with the same types so will compile nicely). In my mind I have something like the following:
function simulate(mc::MarkovChain; npers::Int, nreps::Int=1)
    # Farm out to the right method
    if nreps == 1
        out = simulate_single_chain(mc, npers)
    else
        out = simulate_panel(mc, npers, nreps)
    end

    return out
end

function simulate_single_chain(mc::MarkovChain, npers::Int)
   # Do stuff......
   # Return a 1d array of the simulation
end

function simulate_panel(mc::MarkovChain, npers::Int, nreps::Int)
  # Do stuff
  # Return a 2d array with each simulation down column
end

All the work is happening within functions that return the same type of thing each time, so I don't think we would suffer much slow down. Additionally, we would only export the single simulate function so users don't even have to know about these other pieces.

@cc7768 That sounds like a smart way to go.  As you say, each function handles a different case, so there should be no performance issues.
I should say I also like @albep 's suggestion that the function the user calls figures out whether it's a single chain or a cross-section of multiple changes from the init argument.  If it's a single element then you get back a single chain.  If it's an array, then you get back a matrix with one chain per element of the array.

Thanks @cc7768. Your analysis of the type inference implications within the simulate function is exactly right.
However, the issue is that the simulate function itself will not be type stable. This means anyone that calls that function will have to deal with the type instability. My rule of thumb is that I try to figure out how to make library functions type stable so that users of the library can trust that calling our routines will still allow their code to be compiled efficiently.
I haven't thought of a clever way to do this, but when I do I'll post here.
A non-clever way to do it would be to always return a 2 dimensional array, where the size would be (N, 1) in cases where we previously returned a 1d array. I don't love that solution though, so let's keep thinking

This would be a way to implement what I had in mind, using only method inference:
`function _simulate(mc::MarkovChain, ts_length::Int, init::Int)
return 1d simulation of mc with init as initial index
end
function _simulate(mc::MarkovChain, ts_length::Int, init::Vector{Int})
return 2d simulation of mc with init as vector of initial indeces
end
function _simulate(mc::MarkovChain, ts_length::Int; num_reps::Int = 1)
init = rand(1:size(mc.p, 1), num_reps)
_simulate(mc, ts_length, init)
return 1d(2d) simulation of mc with random initial index(ces)
end
function _simulate(mc::MarkovChain, ts_length::Int, num_reps::Int, init::Vector{Int})
_simulate(mc, ts_length, repmat(init, num_reps))
return 2d simulation of mc with init as vector of initial indeces, repeated num_reps times
end`
It covers all cases as the current code. However it doesn't address @spencerlyon2 point, which I think is important: so far we discussed returning flat or 2d arrays, but the user would then have to face the type instability induced by the library function simulate. So the goal is really to always return the same type. Unless we create a new type to achieve that type stability (a type which is always the same, regardless of whether we have 1 or multiple series simulated),  the only alternative I see is a 2d array as @spencerlyon2 said

Always returning a 2d array doesn't sound a bad idea.

Personally I would say that the case of wanting to generate a cross section is significantly less common than wanting to generate a single chain.  So, rather than always returning a 2D array, my vote would be to just drop the methods that generate multiple chains in cross section.  Instead we should only have routines that generate a single chain returned as a flat array, and any user who wants a cross section just sets up a loop.  Julia being Julia, there will be no issue with performance.
Generating a cross section is probably more important in a language where vectorization is necessary.  That's not the case here.

Here's an idea.
What if we have simulate and simulate_indices always return a flat array (Vector), but also expose methods simulate! and simulate_indices! that will fill a pre-allocated array with as many chains as there are columns in the pre-allocated array.
I think that the init argument for these methods should do the following:

if not given, create a new random initial condition for each chain
If a scalar is given, apply the same initial condition to each chain
If a vector is given, then apply each element of the vector as an initial condition for one chain, repeating the conditions as necessary. Specifically I have the following in mind:

real_init = collect(take(cycle(init), size(out, 2)))

An example of this behavior is here:
julia> out = zeros(10, 5);

julia> init = [1, 2, 3];

julia> real_init = collect(take(cycle(init), size(out, 2)))
5-element Array{Int64,1}:
 1
 2
 3
 1
 2

The alternative to this more robust, more magical proposal would be to throw an error whenever 1 < length(init) < size(out, 2)
Any comments/suggestions on this proposal?

I like this idea.
Either of the alternative approaches to treating init is fine, as long as it's clearly documented.

Sounds good to me. @vgregory757 or @albep are either of you willing to take this on?

Sure. I'll start by implementing @spencerlyon2's idea(s) and submit a PR asap.


Please brief the flow of pact provider jvm junit.

I would like to understand the flow of Pact-jvm-provider-junit. I have explored the example of ContractTest and it's fine but suppose if we need to verify the new pact file (.json) then where should I write the verification code? If we have multiple pact files with different format then how we can verify the pact files?
Please brief high level so that I can move further. Thanks!

Pact-jvm-provider-junit allows you to validate pact files via a JUnit test. You don't write the verification code, the Pact-jvm-provider-junit does that. You just define the source of the pact files (URLs, directories on disk or pact files stored in a pact broker), and the target to validate the requests against (currently on HttpTarget has been implemented).
You're test class is responsible for making sure your provider is started before and torn down afterwards. You can use the standard JUnit @Before and @After annotated methods for that.

Thanks Ronald,
I was exploring HTTPTarget and found It is comparing actualResponse with interaction.getResponse() and interaction response is what defined into pact json(contract.json) but I am not getting how jetty server returning actual response. is it already defined there? could you brief about how other request /data ,/second will work ?

I'm not sure I understand this statement: but I am not getting how jetty server returning actual response
This is how jetty will work. If you make a HTTP request to a running jetty server, it will return an HTTP response.
I think what you are trying to ask is how do you get your provider to be running for the test, and this depends on how your provider is implemented. Is it a web application, or are you using a micro-service framework like SpringBoot or DropWizard?
The Pact-jvm-provider-junit library will make each request it finds in all the pact files provided and compare the actual response back from the running provider based on the HttpTarget annotation. It will then report back on all the requests made and fail the test if any did not match.


Small wifi reliability enhancements

After talking with @tyeth, we've identified two changes to make to increase wifi reliability:

For PicoW, there's an issue where it fails to connect to WiFi

Within this sample sketch, the WiFi object's setTimeout function is called with a timeout of 20 seconds.
We should try this and implement it if it fails less often

To help find mobile hotspots or other mobile APs, we could increase the number of times WipperSnapper attempts to scan for the wireless network described in secrets.json.

@tyeth  Since you can replicate the PicoW bug, do you want to take this issue on (at least the first bullet)?

@tyeth  Do we still want to take a look at this, or close it out?

I still have problems connecting pico-w's, but my local network setup is currently in flux, transitioning to a mesh which brings its own fun/problems.
Think it's worth leaving open, and as you previously said, for me to test with a 20second timeout.


how to handle EventEmitter for a component created through Modal

Support Question
Sorry I am asking this question here because I haven't gotten any response from the forum:
Hi, I have a dump component(i.e only has Input and Output). I am creating the component through a ModalController. I can easily pass in the required input params as per the docs.
The issue I have now is how to subscribe to the component's output event.
for ionic v3 this was easily done, see related stackoverflow
Q&A(https://stackoverflow.com/questions/42259421/ionic2-how-to-catch-events-from-within-a-modal-component-created-from-modalcon)
basically how would to realise this ionic v3 implementation in ionic 4
const modal = this.modalCtrl.create(MyComponent);
modal.didEnter.subscribe(() => {
    modal.instance.myAwesomeEvent.subscribe(....)
});

Note this is different from the use case where you get teh data returned onDidDismiss(). I have looked through the source code(https://github.com/ionic-team/ionic/blob/master/core/src/components/modal/modal.tsx) but can seem to arrive at a solution

Thanks! But I don't understand your question. Please see docs -> https://github.com/ionic-team/ionic/tree/master/core/src/components/modal
To return data, just do the following:
const modal = await modalController.create({...});
const { data } = await modal.onDidDismiss();
console.log(data);

Inside your modal you have to do this:
// Dismiss the top modal returning some data object
modalController.dismiss({
  'result': value
})

console.log would now return

{
  'result': value
}

Thanks @paulstelzer , yes I have seen this use case. which I don't think applies to my scenario. I will have to create a feature request.
My use case in summary:
I have a TagAdminComponent(a smart component that is able to call a Tagservice or state management store).This parent component houses  reusable;  list(generic-list) and form(generic-form) child components. These serve a dumb reusable component which communicates with the parent through  inputs and  output:
  <ion-content padding>
    <ion-grid>
      <ion-row>
        <ion-col size="12" size-md="6">
            <generic-list
              [hasCustomTemplate]="false"
              [isApp]="isApp"
              [itemList]="modelList"
              (viewItem)="onViewItem($event)"
              (updateItem)="onUpdateItem($event)"
              (deleteItem)="onDeleteItem($event)"
              (archiveItem)="onArchiveItem($event)"
            >
            </generic-list>
        </ion-col>
        <ion-col *ngIf="!isApp" size="12" size-md="6">
          <generic-form [config]="config" (created)="onCreated($event)"> </generic-form>
        </ion-col>
      </ion-row>
    </ion-grid>

    <ion-fab *ngIf="isApp" vertical="bottom" horizontal="end" slot="fixed" (click)="presentItemForm(config)">
      <ion-fab-button> <ion-icon name="add"></ion-icon> </ion-fab-button>
    </ion-fab>
  </ion-content>

so on a larger screens the two child components are rendered side by side but on small sreens(usually on mobile phone), I render just the list component and launch the form component through modal and I need to be able to handle the created event(in the child form component)  through the oncreate method defined in the parent. I am using these generic child components accross the application hence the dumb-smart components design approach.
In ionic v3 I could easily handle this implementation with the code snippet below. Not sure why this ability missing ionic 4
const modal = this.modalCtrl.create(MyComponent);
modal.didEnter.subscribe(() => {
    modal.instance.create.subscribe( //call this.onCreate(event) )
});


Microsoft.SqlServer.Dacpacs.SSISDB missing

Is your feature request related to a problem? Please describe.
Lot's of our existing Visual Studio driven Database projects have database references to SSISDB.
I'd like to convert them to dotnet based database projects.
Currently there are dacpacs for:

master https://www.nuget.org/packages/Microsoft.SqlServer.Dacpacs.Master
msdb https://www.nuget.org/packages/Microsoft.SqlServer.Dacpacs.Msdb

However, there is no Microsoft.SqlServer.Dacpacs.SSISDB
Describe the solution you'd like

I'd like to have SSISDB as a dacpac so that all my database projects can be ported to Azure Data Studio.
otherwise https://github.com/microsoft/DacFx/blob/main/src/Microsoft.Build.Sql/docs/Functionality.md#database-references should state that this is not supported

Describe alternatives you've considered
Stay with Visual Studio based SSDT ...
Additional context
Add any other context or screenshots about the feature request here.

Hi @fmms -
We've currently shipped dacpacs for the SQL system databases, and the integration services catalog is handled as a user database.
Can you expand on how you're currently adding the database references to SSISDB? I understand its in Visual Studio, but do you have a dacpac for SSISDB from your own environment?
I'm checking into some options here, and will update the documentation to reflect current state more clearly shortly.

Hi @fmms - We've currently shipped dacpacs for the SQL system databases, and the integration services catalog is handled as a user database.
Can you expand on how you're currently adding the database references to SSISDB? I understand its in Visual Studio, but do you have a dacpac for SSISDB from your own environment?
I'm checking into some options here, and will update the documentation to reflect current state more clearly shortly.

HI @dzsquared we have SSISDB as a database reference in our database projects in Viusal Studio as we do have custom mail notifications sent out on success an error parsing the logs from SSISDB to sent out some nice HTML formatted tables.
In the simplified version imagine a database project with the following file (it does a lot more than just selecting on those tables ... ;) ):
CREATE VIEW [dbo].[ssislog] AS
SELECT * FROM [$(SSISDB_SERVER)].[$(SSISDB)].[catalog].[executions] e

I see - so you have a dacpac locally for SSISDB that you're setting the database location for in SSDT.
(thanks for bearing with my questions, wanting to make sure the docs and future solutions are applicable and clear)


Reinstate snippet language

cf https://github.com/meteor/hexo-theme-meteor/pull/3

Our syntax highlighting still needs some work, namely JSX, but closing this per https://github.com/meteor/hexo-theme-meteor/pull/3#issuecomment-367287097

👋 Tom.


[Microsoft Purview - Catalog] API Review

New API Review meeting has been requested.
Service Name: Microsoft Purview - Catalog
Review Created By: Yifan Zhou @yifan-zhou922
Review Date: 05/04/2023 04:00 PM PT
PR:
Hero Scenarios Link: Not Provided
Core Concepts Doc Link: Not Provided
Description:
Detailed meeting information and documents provided can be accessed here
For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

Meeting updated by Yifan Zhou
Service Name: Microsoft Purview - Catalog
Review Created By: Yifan Zhou
Review Date: 05/11/2023 04:00 PM PT
PR:
Hero Scenarios Link: Not Provided
Core Concepts Doc Link: Not Provided
Description:
Detailed meeting information and documents provided can be accessed here
For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

Meeting updated by Yifan Zhou
Service Name: Microsoft Purview
Review Created By: Yifan Zhou
Review Date: 05/11/2023 04:00 PM PT
PR: https://github.com/Azure/azure-rest-api-specs/pull/23095
Hero Scenarios Link: Not Provided
Core Concepts Doc Link: Not Provided
Description:
Detailed meeting information and documents provided can be accessed here
For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

Meeting updated by Yifan Zhou
Service Name: Microsoft Purview
Review Created By: Yifan Zhou
Review Date: 05/11/2023 04:00 PM PT
Onboarding Record:
PR: https://github.com/Azure/azure-rest-api-specs/pull/23095
Hero Scenarios Link: Not Provided
Core Concepts Doc Link: Not Provided
Description:
Detailed meeting information and documents provided can be accessed here
For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

Meeting updated by Yifan Zhou
Service Name: Microsoft Purview
Review Created By: Yifan Zhou
Review Date: 05/11/2023 04:00 PM PT
Onboarding Record:
PR: https://github.com/Azure/azure-rest-api-specs/pull/23095
Hero Scenarios Link: Not Provided
Core Concepts Doc Link: Not Provided
Description: We plan to introduce several new APIs entity, including entityDeleteHierarchy and entity import/export APIs.
Detailed meeting information and documents provided can be accessed here
For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

Note from API review meeting 5/11/23

New APIs are Microsoft-specific -- not taken from Apache Atlas
Consider changing parallelCount to an enum that the service maps to a particular number

or leave it out

Error response is not Azure standard -- is that because it matches Purview Catalog standard
Should guid be entityIds ?  At a minimum it should be guids.  Maybe not ... because it is collectionFormat=multi.
And consistent with the rest of the service, so perhaps we leave it as is.
Consider moving hierarchy to a query parameter, to allow delete of single entities w/ LRO operation

Include "bulk" in the path for consistency with Atlas delete entity operation (non-LRO)

Consumes multipart/form-data -- need to fix that
What are the semantics of cancel? Is it "atomic"?  No.  That needs to be documented.
For export, will you ever want something other than csv?

This can probably be done w/o breaking

If you ever want export to be LRO, you should make it LRO now.

If you make export sync now, it must stay sync forever.

How big can the CSV files be?  100K rows, but may be larger in the future.

What about BYOS support? You can add this later. Simple upload still a good option

List on operations, collection name should be operations -- plural

@yifan-zhou922 please post a comment here when you have addressed the issues above in the PR and are ready for sign off.

Thanks @mikekistler for the comments & suggestions. We will address the issues above and leave a note when they are all ready for signoff.

@yifan-zhou922 Please provide an update on this API review. Are you still working on addressing the issues or has work on this version been suspended?

Hi @mikekistler , for these APIs, we are still working on the feedbacks from the meeting review, for example, design an async export API instead releasing a sync version, changing to plural form for some API paths.
We are still working on the api version, here is the PR. https://github.com/Azure/azure-rest-api-specs/pull/25164
In this PR, we have added APIs for bulk edit glossary terms and bulk get glossary terms. These APIs have been reviewed in June. For other APIs reviewed in June, some of the comments have been updated but these APIs are not ready for public preview. So, these APIs will not be included in this API version update. Thanks.
Could you please help review this PR  https://github.com/Azure/azure-rest-api-specs/pull/25164? Many thanks.


[firebase_messaging] push messaging not delivering when app is killed

Description:
firebase_messaging plugin does not deliver FCM messaging to app or system tray when app is killed.
Using:

environment:
  sdk: ">=2.15.0 <3.0.0"

firebase_core: ^1.10.2
firebase_messaging: ^11.2.0

[√] Flutter (Channel stable, 2.8.1, on Microsoft Windows [Version 10.0.19044.1415], locale ru-RU)
    • Flutter version 2.8.1 at D:\.Important\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 77d935af4d (4 weeks ago), 2021-12-16 08:37:33 -0800
    • Engine revision 890a5fca2e
    • Dart version 2.15.1

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at D:\.Important\AndroidSDK
    • Platform android-31, build-tools 30.0.2
    • ANDROID_HOME = D:\.Important\AndroidSDK
    • Java binary at: D:\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

initialization code
void main() async {
  WidgetsFlutterBinding.ensureInitialized();

  await Firebase.initializeApp();
  FirebaseMessaging.onBackgroundMessage(firebaseMessagingBackgroundHandler);
  FirebaseMessaging.onMessage.listen(onFirebaseMessage);
  // FirebaseMessaging.onMessageOpenedApp.listen(_onFirebaseMessage);

  runApp(AssemblerApp());
}

While app is in foreground and background, I receive onBackgroundMessage / onMessage.
If I kill the app the messages are no longer being delivered to user while FCM reports success
Push message data
    "notification": {
        "title": "\u041d\u043e\u0432\u044b\u0439 \u0437\u0430\u043a\u0430\u0437",
        "body": "\u041f\u043e\u0441\u0442\u0443\u043f\u0438\u043b \u043d\u043e\u0432\u044b\u0439 \u0437\u0430\u043a\u0430\u0437 \u2116 [%]. \u041d\u0435\u043c\u0435\u0434\u043b\u0435\u043d\u043d\u043e \u043d\u0430\u0447\u043d\u0438 \u0435\u0433\u043e \u0441\u0431\u043e\u0440\u043a\u0443."
    },
    "data": {
        "type": "ORDER_INFO",
        "title": "\u041d\u043e\u0432\u044b\u0439 \u0437\u0430\u043a\u0430\u0437",
        "body": "\u041f\u043e\u0441\u0442\u0443\u043f\u0438\u043b \u043d\u043e\u0432\u044b\u0439 \u0437\u0430\u043a\u0430\u0437 \u2116 [%]. \u041d\u0435\u043c\u0435\u0434\u043b\u0435\u043d\u043d\u043e \u043d\u0430\u0447\u043d\u0438 \u0435\u0433\u043e \u0441\u0431\u043e\u0440\u043a\u0443.",
        "order_id": "100000853",
        "success": "NEUTRAL",
        "deeplink": "\/order\/100000853"
    },
    "to": "eK5LBortQYWVxQN6pDeYIu:APA91bH7SgA4d1gQFW7emmFeHx2ATSDmXhaokt6FhDDRr-LiAo6svwaNMKdPuhvuiVJoAQK4SVNpT9b6B7HFj8kVCTUz1f46ICRNobFj1vqnBQjY2TRg8Yd7shZrNreM9l5-ajqMxbiP",
    "android": {
        "priority": "high"
    },
    "apns": {
        "payload": {
            "aps": {
                "contentAvailable": "true"
            }
        },
        "headers": {
            "apns-push-type": "background",
            "apns-priority": "5",
            "apns-topic": "io.flutter.plugins.firebase.messaging"
        }
    }
}
{"multicast_id":8906492871425850351,"success":1,"failure":0,"canonical_ids":0,"results":[{"message_id":"0:1641897546671072%d7e81b5ad7e81b5a"}]}```

Message w/o data or notification is not delivering also.
If there is any data I could provide to debug the issue please tell.

@Anfet
As mentioned in the documentation https://firebase.flutter.dev/docs/messaging/usage#receiving-messages when the app is killed on Android the notifications can't be received or listened to.
How are you terminating the app ?

By swiping it away.
Why you are stating that notifications can't be received? In documentation we have written
Based on your applications current state, incoming payloads require different implementations to handle them:
Terminated: onBackgroundMessage
I've found the solution to my problem by adding  "priority":"high" to raw body of the push message. (per this documentation )
That conflicts with flutter documentation example where
  android: {
    priority: "high",
  },

block is added which has no effect.
There is also undesired effect of doubling notifications. As I understand, system is receiving push data containing notification block and making a notification itself and forwarding data to app service which makes another notification.

@Anfet
Can you try the official plugin example and see if you get the same behavior ?

I also have the problem that when the android app is killed the background messages aren't received anymore.
I even get the
FLTFireMsgReceiver: broadcast received for message
log but the background message handler isn't triggered.
I also tried removing the high priority check to have same behavior for all messages: https://github.com/FirebaseExtended/flutterfire/blob/master/packages/firebase_messaging/firebase_messaging/android/src/main/java/io/flutter/plugins/firebase/messaging/FlutterFirebaseMessagingBackgroundService.java#L39
but even with it always to true no messages are received as soon as the app is killed for longer than a minute.

The problem is that notification isn't delivered to system as well (at least not showing up in notification popup).

@darshankawar this might be related to #6956.

The problem is that notification isn't delivered to system as well (at least not showing up in notification popup).

I don't understand how to  reliably deliver.
Even with "priority":"high" the messages are being delivered randomly or with unknown delay.

@Anfet
Do you mean that the delivery is intermittent, sometimes it delivers and sometimes not or is it consistent that everytime you push, it doesn't deliver ?
Per above comment, can you try on Android 10 and < 10 to confirm ?

@Anfet Do you mean that the delivery is intermittent, sometimes it delivers and sometimes not or is it consistent that everytime you push, it doesn't deliver ? Per above comment, can you try on Android 10 and < 10 to confirm ?

When I send push and the app is in foreground or background - the push data is delivered and received 100%.
When app is killed (swiped away) - the messages are delivered on unknown schedule. I suppose there is some delay. As from time to time, I see in logcat that app is being started and message processed.
Android 7 have normal behavior. Push messages are delivered reliably.

@kanekv
This might be related to native firebase android sdk since per your comment it works well on Android 7 but not on Android 10 or later and not with the firebase_messaging plugin. I would advise you to open this issue in https://github.com/firebase/firebase-android-sdk/issues.
Flutterfire plugin are just wrappers around native firebase sdks.


"The namespace or module '...' is not defined" despite module existing

Describe the bug

There is syntax highlighting with the error message of "The namespace or module '...' is not defined" despite the module existing in that exact version in my local files. When I build the program or run the tests (the file with the error is my test.fs file), everything works fine, but I get this annoying syntax highlighting.
Steps to reproduce

Have a file that imports packages.

Link to sample reproduction

Expected behaviour

No syntax highlighting
Screenshots

Module highlighting:

Resulting highlighting:

Machine info

OS: Windows 11
.NET SDK version: Appears on 6.0.309 and 7.0.201
Ionide version: 7.5.1

Additional context
If you want the exact files, I can share them with you. Just let me know.

Hello @CanePlayz,
Does the error goes away if you quit and restart VSCode?
As a workaround can you try the following:
Workaround 1

Save/Commit the work you don't to lose
Clean your temporary files git clean -xdf
Run dotnet build to try forcing Ionide to update

Workaround 2

Save/Commit the work you don't to lose
Clean your temporary files git clean -xdf
Quit and restart Ionide

Sometimes I need to do stuff like that on some of my projects.

Does the error goes away if you quit and restart VSCode?

Unfortunately, no. Sometimes, it does find the modules. And something that is different in those scenarios is that there is a file called something like NET Testing SDK...However, that might just be a side effect of the project finding the file, because that file only gets auto-generated when the module is found, and not the other way around (the file helping Ionide).
Thanks for the workarounds, however, they don't apply in this case since this is not a Git project. It's basically many very tiny projects, just exercises from university, only consisting of a .fsproj. and two to three .fs files.

The clean command is because in general you add the bin and obj folders to the gitignore file.
If you don't use Git, you can always remove the bin and obj folders manually or using the CLI.

I had the same issue with the default SAFE template, enabling Adaptive Lsp Server fixed it for me (https://github.com/ionide/ionide-vscode-fsharp/issues/1852#issuecomment-1469005519)

Issue still persists with Adaptive Lsp Server enabled, unfortunately. It's also weird that on some of the exercise projects, it finds the modules, despite them not being any different from the other ones as far as I can see.

After running "dotnet test" once, it finds the modules in the test files.

Can you submit a repository that reproduces this issue?

You can find the project here: https://github.com/CanePlayz/Ionide-Test

After a dotnet restore I cannot get this to reproduce.
https://user-images.githubusercontent.com/1490044/227225283-f949fc26-fa1e-44b6-a9fc-3a381781ea0b.mp4

Maybe there's something weird about your install of dotnet. What's dotnet --info ?

After a dotnet restore I cannot get this to reproduce.

Me neither anymore. Maybe something fixed it. It always finds the modules now after a dotnet restore. But why does it need that dotnet restore if the packages are already installed?

What's dotnet --info?

.NET SDK:
 Version:   7.0.202
 Commit:    6c74320bc3

Laufzeitumgebung:
 OS Name:     Windows
 OS Version:  10.0.25314
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\7.0.202\

Host:
  Version:      7.0.4
  Architecture: x64
  Commit:       0a396acafe

.NET SDKs installed:
  6.0.407 [C:\Program Files\dotnet\sdk]
  7.0.202 [C:\Program Files\dotnet\sdk]

.NET runtimes installed:
  Microsoft.AspNetCore.App 6.0.15 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 7.0.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 6.0.15 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 7.0.4 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 6.0.15 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 7.0.4 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

Restore doesn't just download nuget packages, more importantly it creates several metadata files (like project.assets.json) in the Intermediate Output Path of each project that are required in order to provide intellisense for the files in that project. The fact that performing a restore outside of the editor cleared things up for you makes me think that our automatic restore-triggering in FSAC isn't working consistently.

Thank you for this thread. I've been losing my mind. "dotnet restore".

@MangelMaxime I cannot reproduce this anymore. Should we close this issue?

I still get this issue. It's always after I add a new .fs file to my .fsproj. The reported error is usually to do with an included project like this:
    <ProjectReference Include="../../../sutil/src/Sutil/Sutil.fsproj"/>

I would discount installation of .NET etc because it still happens even after installing a brand new laptop (the thinking being that having followed standard installation procedures on a new machine, there's been little to no chance for the .net install to get funky).
I'd be happy to try and make a repro project if that would help?

Yeah, I also just remembered that this was not just about modules not being found in general, but also about not finding them before doing a manual dotnet restore, which is still necessary but shouldn't be.


Standardize Merging - Logic, Approach, Testing

With the addition of hashing and the update method this library can now be used as to combine or merge a set of messages. The logic for message merging should be agreed upon by all interested parties and validate through extensive testing. This issue is to hold such a discussion.
It is important that when we make decisions about how message merging will work, that the speed of the implementation is considered.

To state initial objects:
Each TRAPI object may have a different set of criteria for how they are merged together.
The following should be guaranteed for all TRAPI object merges:

Merging is symmetrical, so merge(m1, m2) == merge(m2, m1)
Merging will raise errors iff provided values cannot be merged AND the provided values are different
Some TRAPI objects allow extra properties. When these are present, they must all be identical for merging to be allowed

The above may result sub-optimal speed. Consider these generic objects
QNode1: { “ids” : [], “deeply” : { “nested” : { “extra” : { “field” : {}}}} }
QNode2: { “ids” : [], “deeply” : { “nested” : { “extra” : { “field” : {“foo” : “bar”}}}} }
These two objects cannot be merged using the above criteria because of the mismatch of deeply.nested.extra.field. The only way to know that these objects cannot be merged is by traversing the object hierarchy and then identifying the mismatch.

Restating objectives to more closely match the current implementation and provide a faster runtime.
Each TRAPI object has a different set of criteria for how they are merged together.
The following should be guaranteed for all TRAPI object merges:

In the case of a conflict in merge(m1, m2), where conflict is any field for which no specified merging rule exists, the output value will be the value on m1. The only exception to this is if the value on m1 is null, in which case the value on m2 will be used.
Because of the above property, merging is not guaranteed to be symmetrical, so merge(m1, m2) != merge(m2, m1)

Removing symmetry enables a faster implementation but may lead to inconsistencies. For example, if an ARS made several async requests to ARAs and merged the output. The order of that the responses are received can potentially impact that resulting merged message.


c# response headers support and code simplification

-- Expose ResponseHeadersAsync property to access response headers from client-side calls.
-- simplify client-side call lifecycle handling
-- Introduce internal INativeCall interface to allow mocking the native layer in tests
-- A bit of code cleanup and polishing.
Fixes #1423

LGTM


Nested counters support

JUnit QueryCounter should probably have its own query counter, which won't be affected to Sniffer.reset calls inside of it

Implemented using a new Spy abstraction which provides and independent set of queries counters


:recycle: chore: Move docs generation to package

Description
Move doc generation to package level instead of generating it in a different package. Each package reference docs will be a standalone site
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[x] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #596

PR #597

PR #599
PR #598

PR #602   👈
PR #601

This comment was auto-generated by Graphite.


Add asciidoctor to website.

Bump website dependencies (hugo, npm)
Add Asciidoctor to website.
Update deployment definition for Netlify.

Part of #75

cc @rolfedh

Yes. As I understand it, with this enhancement, with can use markdown and asciidoc content.
I would avoid having duplicate content in both file types. For example, with build.adoc and build.md, contributors might not know which file to modify.

In my tests, if you have two content files such as build.adoc and build.md, the markdown file "wins."

In my tests, if you have two content files such as build.adoc and build.md, the markdown file "wins."


Research - Build tool

Describe the solution you'd like
Decide on an appropiate build tool for the project.
This will be dependent on the language we use.
Describe alternatives you've considered
Additional context

Check out pull panda https://github.co/31G3b4T

Makefiles to get moving?

^^ Makefiles.


For more information section only uses examples

Hey folks,
the security advisory at https://github.com/helm/helm/security/advisories/GHSA-c38g-469g-cmgx has a section called "For more information".
The repo link  and the email address both points to example addresses.
A solution could be to delete that question or to fill in valid addresses.
I do not have further questions or comments about the advisory.
Thanks for your work,
Matthias

Thanks for the heads-up. We have to wait until the CVE is issued before we can update that document. In the meantime, the SECURITY doc for the project is the best source of information. https://github.com/helm/community/blob/master/SECURITY.md

The CVE was just issued and the notice was unlocked. So I have fixed the document.
Thanks again.


How to set independent intrinsic of render cameras?

Thanks for your wonderful work!
When I trying to set different intrinsic to the cameras in render_path.npy, an error occurred when I runing the render script.

So what can I do to flexible set intrinsic and make it work!

Hi! Have you solved it?


Scalar api, ndarray.sum fixed, backend casting.

Hey, commit's descriptions describe exactly what was done.
np.ones, np.zeros were rewritten to fallback into a single method and np.ones implementation fixed. it failed in different dtype cases like np.ones(np.uint8, ...).
I had to change NDArray.Storage to internal (from protected) to gain access in the backend engine.
I've added a backend method name Cast as progress with upcasting during math operations (#281).
Note: backend.Sum is still broken for types other than int32 (no due to my changes).

Amazing work. Too much changes. Might need @henon help review.

looks all very professional. lots of test cases too, great. I like the idea of linking to the numpy docs.
one hint: when fixing a bug that wasn't covered by the test suite add a simple test so that it stays fixed forever.

@Nucs: can you add an issue about the broken Sum ?

Well, the backend is a mess.
No unit tests at all.
Implementations are limited to rank 2.
The types handling covers only half, sometimes just two or three types.
This has to be done next, I'm looking for a proper and clean way to handle code generations except for T4 templates.
Manually writing it all will take couple of straight days of work.
If anyone knows what tool the dev used here - that would be great.

They use F# to generate the code.

Found nest loop for np.ones. I've fixed it.

@Nucs Please always keep in mind the performance. for example.
           Single SumSingle(Single[] arr)
            {
                Single sum = 0;
                checked
                {
                    for (int i = 0; i < arr.Length; i++)
                    {
                        sum += arr[i];
                    }
                }

                return sum;
            }

At most of time, the Linq sum would faster than raw loop.
You can write a performance benchmark:

Didn't even have that project loaded, definitely will do.


Remove version: FineprintSoftware.pdfFactory.Pro version 9.07

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: Duplicate;


BytesN SSZ Alias is Misleading

Hi all, currently the SSZ spec alias section mentions:
BytesN to Vector[byte, N] (this is not a basic type)

and we also have the type Byte[N] in some other places across the repo including the test generators for ssz generic containers here. This tricky nomenclature led us to some painful debugging and quite some time spent fixing broken tests in Prysm. The alias doesn't add much utility and we at @prylabs would just prefer to stick to the suggested naming conventions for lists vs. vectors:
List[byte, N]
Vector[byte, N]

How is it misleading? Other than that we should probably add Byte[N] to our list of aliases?

Why need the alias and more complexity? Why not just stick to
List[byte, N]
Vector[byte, N]

It was just a point of frustration when trying to pass spec tests and got confused between the types. Would have been just easier if we didn't introduce those custom aliases. Spent a good chunk of time fixing broken tests due to confusion, unfortunately.

See #1480


SQL function LOG(number, base) is not implemented

The following code results in exception:
CREATE TABLE table1 (
double_col DOUBLE,
smallint_col SMALLINT);
SELECT LOG("double_col", 2) AS tmp FROM table1;

Exception: Exception occurred: org.apache.calcite.runtime.CalciteContextException: From line 1, column 8 to line 1, column 29: No match found for function signature Log(<NUMERIC>, <NUMERIC>)
Ibis test test_complex_math_functions_columns fails because of this.

the query can be rewritten with ln(2)/ln("double_col").
adding an extension function to add this functions is an easy task

Thanks for letting me know. In this case it is necessary to change Ibis to generate such code for the corresponding operation.

We should support this too.

We should support this too.

it's trivial to support btw


[BUG] Clean library button is not working

Description of the Bug
Clean library button shows the following error: 405 Method Not Allowed
Expected Behavior
Clean library button should remove all books in a library with missing files.
To Reproduce
Steps to reproduce the behavior:

Go to Library Settings
Go to Deletion section
Click "Clean library" button
See error

Screenshots

Debugging Information
Looks like the Method should be PUT If I'm reading this right: library.rs.
Maybe codegen for the sdk is not up to date?
From browser request:
Request URL: [REDACTED]/api/v1/libraries/08a2071e-ca65-47cf-8c2a-e27edbe367c4/clean
Request Method: POST
Status Code: 405 Method Not Allowed
Referrer Policy: strict-origin-when-cross-origin

Environment:

OS: [Brave Browser]
Device: [Android]

Build Details:

Version: [experimental]
Docker: [yes]

Thanks for the bug report! Your write-up is very helpful, it is almost certainly a small, lingering regression from https://github.com/stumpapp/stump/pull/457.
I'll push up a fix either today or tomorrow and update this ticket once the experimental build is updated

@xendke This should be fixed in the updated experimental build. I didn't have time to validate myself, but swapped out the request method to the correct one (PUT) so it should be good. If you are able to re-test, let me know and I can close this out afterwards

that did it, thanks @aaronleopold 🎉


Create a Google Compute Engine init package.

Fixed #277
The google-compute-engine package is identical to the Python package and
does not install distribution specific init files. The new package is
necessary to automatically run daemons and instance setup.

LGTM

@dmichel1 - please take a look. I'd like your input before I merge.

Merging - if there are additional comments we can resolve them in a followup pull request.

LGTM +1


Tensors should only depend on System.Memory (not Buffers.Primitives)

We don't actually use Buffers.Primitives now that Memory (previously Buffer) has been moved into System.Memory.

@dotnet-bot test Innerloop Ubuntu16.04 Debug Build and Test


Is there a way to ban a users ID from queueing songs?

Just need to know if it's possible because of really shitty memes :/

Check out the 'blacklist' command

https://github.com/Just-Some-Bots/MusicBot/wiki/Commands#blacklist-status-user1


gnu-pw-mgr: 2.3.3 -> 2.4.2

Motivation for this change
Things done

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

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

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

@GrahamcOfBorg build gnu-pw-mgr

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: gnu-pw-mgr
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: gnu-pw-mgr
Partial log (click to expand)
/nix/store/hnq16rq7lszasrqb87314h7c5qdq1zir-gnu-pw-mgr-2.4.2
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/hnq16rq7lszasrqb87314h7c5qdq1zir-gnu-pw-mgr-2.4.2
shrinking /nix/store/hnq16rq7lszasrqb87314h7c5qdq1zir-gnu-pw-mgr-2.4.2/bin/gnu-pw-mgr
shrinking /nix/store/hnq16rq7lszasrqb87314h7c5qdq1zir-gnu-pw-mgr-2.4.2/bin/sort-pw-cfg
gzipping man pages under /nix/store/hnq16rq7lszasrqb87314h7c5qdq1zir-gnu-pw-mgr-2.4.2/share/man/
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/hnq16rq7lszasrqb87314h7c5qdq1zir-gnu-pw-mgr-2.4.2/bin
patching script interpreter paths in /nix/store/hnq16rq7lszasrqb87314h7c5qdq1zir-gnu-pw-mgr-2.4.2
checking for references to /build in /nix/store/hnq16rq7lszasrqb87314h7c5qdq1zir-gnu-pw-mgr-2.4.2...

Success on aarch64-linux (full log)
Attempted: gnu-pw-mgr
Partial log (click to expand)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/p9nrvjzcig490wmv5wkgzzqzf8h2gqjg-gnu-pw-mgr-2.4.2
shrinking /nix/store/p9nrvjzcig490wmv5wkgzzqzf8h2gqjg-gnu-pw-mgr-2.4.2/bin/sort-pw-cfg
shrinking /nix/store/p9nrvjzcig490wmv5wkgzzqzf8h2gqjg-gnu-pw-mgr-2.4.2/bin/gnu-pw-mgr
gzipping man pages under /nix/store/p9nrvjzcig490wmv5wkgzzqzf8h2gqjg-gnu-pw-mgr-2.4.2/share/man/
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/p9nrvjzcig490wmv5wkgzzqzf8h2gqjg-gnu-pw-mgr-2.4.2/bin
patching script interpreter paths in /nix/store/p9nrvjzcig490wmv5wkgzzqzf8h2gqjg-gnu-pw-mgr-2.4.2
checking for references to /build in /nix/store/p9nrvjzcig490wmv5wkgzzqzf8h2gqjg-gnu-pw-mgr-2.4.2...
/nix/store/p9nrvjzcig490wmv5wkgzzqzf8h2gqjg-gnu-pw-mgr-2.4.2


Fix inappropriate SPDX license tag

SPDX-License_Identifier --> SPDX-License-Identifier
Added one more # to the first line for doxygen.
Signed-off-by: MyungJoo Ham<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1982. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


Investigate dcr conformance suite failures for dev

The dcr conformance suite is failing for dev and nightly:
https://g.codefresh.io/build/65448b9e2db095851e0d601a?step=run_tests_private_key_jwt&tab=output&logs=terminal

The following test is failing:
=== Scenario: DCR-005 - Dynamically retrieve a new software client
        Test case: Register software client
                PASS Generate signed software client claims
                PASS Software client register
                PASS Output x-fapi-interaction-id
                PASS Assert status code 201
                PASS Decode client register response
        Test case: Retrieve client credentials grant
                PASS Client credentials grant
        Test case: Retrieve software client
                PASS Software client retrieve
                PASS Assert status code 200
                FAIL Validate client response schema: schema invalid: software_statement: cannot be blank, software_statement: cannot be blank

The software_statement field is missing from the GET registration response (GET .../register/7e08b66a-776c-41ab-ac9a-b73541dca13e)
This field is added by the ProcessRegistration script, the value is extracted from the IDM ApiClient object's ssa field. Further investigation is required.

Issue is in ProcessRegistration script:
groovy.lang.MissingPropertyException: No such property: ssa for class: com.forgerock.sapi.gateway.dcr.models.ApiClient
at org.codehaus.groovy.runtime.ScriptBytecodeAdapter.unwrap(ScriptBytecodeAdapter.java:67)
at org.codehaus.groovy.vmplugin.v8.IndyGuardsFiltersAndSignatures.unwrap(IndyGuardsFiltersAndSignatures.java:161)
at org.codehaus.groovy.vmplugin.v8.IndyInterface.fromCache(IndyInterface.java:321)
at ProcessRegistration$_run_closure4.doCall(ProcessRegistration.groovy:277)

A change was made to store the com.forgerock.sapi.gateway.dcr.models.ApiClient object in the attributes context, previously a json Map was being stored. The software_statement value needs to be extracted from the object by calling: apiClient.softwareStatement.getB64EncodedJwtString()


Style import fix

Addresses #159.
Also changed spaces to tabs, as that seemed to be the convention.

This would be a breaking change for .less files, since providing paths to less-loader disables Webpack's ~ resolution. For sass-loader, though, this is not the case.
@developit I can just remove all of the tildes from .less files, but wanted to know if you think this is the right approach.

Hmm - I've never used less-loader with the path option.   Does it fully bypass the built-in path handling in less?

@lukeed I think that was my initial reaction too, though this is a lot of moving parts so I wasn't sure.  In the linked #159 issue it seemed like he was hoping to support importing npm modules from .scss files without using the ~ prefix.

Ah right. I'm inclined to think that moving to resolve.modules will get picked up by sass-loader (didn't check). But if not, then leaving the includePaths will be fine.
The important part is not breaking less-loader behavior.

@lukeed @developit is this still valid?

Crap, I botched that merge.

We should also add paths to the stylus-loader, now that we have it.


Firefox: bezel insets are the color of the lcds instead of taking on

It's an interesting effect, not sure if I like it.

it makes sense that the inset bezels would 'reflect' the light of the lcds.

Also occurs in Internet explorer. Seems fine to me, no biggie.


Support f-strings

It looks like flake8-sql fails to parse queries built with f-strings (the same string with .format() works)
Below is a compact example tested with: python -m flake8 flake8_sql_check.py
columns = 'col1,col2'
view = 'View1'

# Works, catches Q440
query = 'select {columns} From [{view}]'.format(columns=columns, view=view)

# Throws "TypeError: unsupported operand type(s) for -: 'NoneType' and 'int'"
# query = f'select {columns} From [{view}]'

Full traceback:
C:\Users\king.kyle\Developer\project>poetry run python -m flake8 flake8_sql_check.py
Traceback (most recent call last):
  File "c:\users\king.kyle\miniconda3\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "c:\users\king.kyle\miniconda3\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\king.kyle\Developer\project\.venv\lib\site-packages\flake8\__main__.py", line 4, in <module>
    cli.main()
  File "C:\Users\king.kyle\Developer\project\.venv\lib\site-packages\flake8\main\cli.py", line 22, in main
    app.run(argv)
  File "C:\Users\king.kyle\Developer\project\.venv\lib\site-packages\flake8\main\application.py", line 363, in run
    self._run(argv)
  File "C:\Users\king.kyle\Developer\project\.venv\lib\site-packages\flake8\main\application.py", line 351, in _run
    self.run_checks()
  File "C:\Users\king.kyle\Developer\project\.venv\lib\site-packages\flake8\main\application.py", line 264, in run_checks
    self.file_checker_manager.run()
  File "C:\Users\king.kyle\Developer\project\.venv\lib\site-packages\flake8\checker.py", line 323, in run
    self.run_serial()
  File "C:\Users\king.kyle\Developer\project\.venv\lib\site-packages\flake8\checker.py", line 307, in run_serial
    checker.run_checks()
  File "C:\Users\king.kyle\Developer\project\.venv\lib\site-packages\flake8\checker.py", line 589, in run_checks
    self.run_ast_checks()
  File "C:\Users\king.kyle\Developer\project\.venv\lib\site-packages\flake8\checker.py", line 494, in run_ast_checks
    for (line_number, offset, text, _) in runner:
  File "C:\Users\king.kyle\Developer\project\.venv\lib\site-packages\flake8_sql\linter.py", line 49, in run    initial_offset = _get_initial_offset(node, self.lines)
  File "C:\Users\king.kyle\Developer\project\.venv\lib\site-packages\flake8_sql\linter.py", line 178, in _get_initial_offset
    first_physical_line = physical_lines[query_end_lineno - len(logical_lines)]
TypeError: unsupported operand type(s) for -: 'NoneType' and 'int'

I can possibly look into a PR later, but wanted to report the error since I've run into it a few times.
I just want to live dangerously:

I went back to revisit this issue, but I oddly can no longer replicate it?
@thomasaarholt, can you share a snippet that fails for you? Do you happen to be using Windows?

@KyleKing I am able to reproduce this error using Python 3.9.9, flake8 4.0.1 and flake8-sql 0.4.1 when running python -m flake8 flake8_sql_format_check.py on this file:
table_name = "mytable"
sql = f"SELECT id FROM {table_name} WHERE foo = 'bar'"

This gives the following traceback:
Traceback (most recent call last):
  File "/Users/brox/.pyenv/versions/3.9.9/lib/python3.9/runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/Users/brox/.pyenv/versions/3.9.9/lib/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/Users/brox/.pyenv/versions/sandbox/lib/python3.9/site-packages/flake8/__main__.py", line 4, in <module>
    cli.main()
  File "/Users/brox/.pyenv/versions/sandbox/lib/python3.9/site-packages/flake8/main/cli.py", line 22, in main
    app.run(argv)
  File "/Users/brox/.pyenv/versions/sandbox/lib/python3.9/site-packages/flake8/main/application.py", line 375, in run
    self._run(argv)
  File "/Users/brox/.pyenv/versions/sandbox/lib/python3.9/site-packages/flake8/main/application.py", line 364, in _run
    self.run_checks()
  File "/Users/brox/.pyenv/versions/sandbox/lib/python3.9/site-packages/flake8/main/application.py", line 271, in run_checks
    self.file_checker_manager.run()
  File "/Users/brox/.pyenv/versions/sandbox/lib/python3.9/site-packages/flake8/checker.py", line 311, in run
    self.run_serial()
  File "/Users/brox/.pyenv/versions/sandbox/lib/python3.9/site-packages/flake8/checker.py", line 295, in run_serial
    checker.run_checks()
  File "/Users/brox/.pyenv/versions/sandbox/lib/python3.9/site-packages/flake8/checker.py", line 597, in run_checks
    self.run_ast_checks()
  File "/Users/brox/.pyenv/versions/sandbox/lib/python3.9/site-packages/flake8/checker.py", line 500, in run_ast_checks
    for (line_number, offset, text, _) in runner:
  File "/Users/brox/.pyenv/versions/sandbox/lib/python3.9/site-packages/flake8_sql/linter.py", line 49, in run
    initial_offset = _get_initial_offset(node, self.lines)
  File "/Users/brox/.pyenv/versions/sandbox/lib/python3.9/site-packages/flake8_sql/linter.py", line 178, in _get_initial_offset
    first_physical_line = physical_lines[query_end_lineno - len(logical_lines)]
TypeError: unsupported operand type(s) for -: 'NoneType' and 'int'


ensure id='defaultHttpEndpoint' attribute is added to httpEndpoint

Ensure id='defaultHttpEndpoint' attribute is added to httpEndpoint when generating a new entry.

Looks good! :+1:


fix: fix example build bug

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Add more Activation Function options.

As per this discussion: #70
This pull request adds the following features to the library.

The duplicate static method - which duplicates matrix passed into it.
Unit tests for the duplicate static method.
Many new activation functions, including relu and leaky_relu.

My javascript auto-formatter cleaned up the code, but it obscured the relavent changes in the most recent commit.  It might be more informative to look at an older commit...

Lol. Just exploring your code to add more activation function into my code and found that out. Now all of them should work fine

Ran some more tests on XOR.
Everything works great! There was a 100% pass rate and most of them were quick.
Observations below:

leaky_relu had the best balance of consistency and speed.
relu was the fastest, but not consistent. Still exhibiting "dying relu" problem.
relu was generating an interesting step patterns in the solution space. Here's a picture.
arctan was solving the problem consistently, albeit not as fast as the others. The solution space looked like a large open field with folded corners. Here's a picture.
softplus had the most difficult time. But it consistently solved the problem. Lots of fluctuations and BOLD decision making. The solution space looks like a glowing blob! Here's a picture.

I encourage everyone to check it out. It's really interesting!


[1.19.2] Jei 11.5.0 and up crash on world join

Jei versions 11.5.0.x and up crash with MoreOverlays in 1.19.2 when connecting to a world.
Tested with:
Forge: 43.2.2
moreoverlays-1.21.3-mc1.19
jei-1.19.2-forge-<IP_ADDRESS>0 - No crash
jei-1.19.2-forge-<IP_ADDRESS>1 - Crash
Latest.log: https://gist.github.com/YOYOK9/00393c50bd8ee0f817405c23c5089f67
Debug.Log: https://gist.github.com/YOYOK9/b5dc1b1d30381aa0d0bf3f34af718fe8
Steps to reproduce:

Add JEI 11.5.0.x+ and MoreOverlays
Try to create/join a SP world

Extra notes:
I also tested this with jei-1.19.2-forge-<IP_ADDRESS>7
latest.log: https://gist.github.com/YOYOK9/b906e6c846e91312ab275046235db938

Same problem as you. I can confirm this incompatibility issue

Can confirm. JEI made some changes that break More Overlays. I'll try to fix it in the next few days. For now you'll have to use an older version of JEI.

Currently having some trouble using the JEI integration for 11.5.0 so I've created an issue on their repo.
https://github.com/mezz/JustEnoughItems/issues/3093

I've released a version that should fix this issue: here
It should be available once it gets accepted on Curseforge.
Please let me know if you encounter any problems with this new version.


Can you query an AWS AppSync API from GraphQL PlayGround?

Hey guys,
I would love to use the Playground for test queries and such. Any experience in setting this up? AWS AppSync gives you a config file in the following format, but I suspect this is wired to AWS's Apollo Client and probably not something you can use as a generic URL endpoint with Playground:
export default {
    "graphqlEndpoint": "https://12345.appsync-api.region-code.amazonaws.com/graphql",
    "region": "region-code",
    "authenticationType": "API_KEY",
    "apiKey": "12345"
}

When you try to enter the endpoint, you get the following error message:
{
  "error": "Response not successful: Received status code 401"
}

I tried to paste parts of the config file to the HTTP headers, but that didn't help either (no idea how to format that though). Any ideas?

Looking a the AWS AppSync docs for API_KEY, you pass the key in a header called x-api-key:

https://docs.aws.amazon.com/appsync/latest/devguide/security.html#api-key-authorization

Using the 'Http Headers' section of GraphQL Playground, it appears it wants the headers as a JSON object:
{
  "x-api-key": "da2-FOOMYKEY"
}

This seemed to allow me to successfully query my API.

Awesome! That works for me, too. Thank you! 👍

How about AMAZON_COGNITO_USER_POOL ?


[quickwin] Improve Search bar

What problem does this new feature solve?
• Add Cancel to the search ; it would give the opportunity user to quickly erase his search without taking to much space as a button
• Make Typing indicator a bit bolder to fit rounded style
Describe the solution you'd like
Component update here
https://www.figma.com/file/jjt28S7DTuKxVAYLdb8vmn/Web3Modal?type=design&node-id=6-1530&t=XH3Xef4EiVgJHSzX-4

hi there can someone assign this to me

Implemented in v3


how is orbitcontrols updated

I'm having issues with getting the template to work with my set-up but OrbitControls don't work. How is it updated exactly?
Expected something along the lines of controls.update().

What do you mean with this? 🤔

OrbitControls don't work

Also

Expected something along the lines of controls.update().

The update() method is supposed to be called after any manual changes to the camera's transform, or in the update loop if the autoRotate or enableDamping properties are enabled, which is not the case in my starter.
Maybe you missed this part in the doc? 🙂