Using Nginx ingress controller for Kong Gateway

Hi,
Is there a way that I can use Nginx ingress controller for my Kong gateway to proxy the traffic instead of using Kong Ingress controller.
I see this option in the values CR file.
# Kong proxy ingress settings.
# Note: You need this only if you are using another Ingress Controller
# to expose Kong outside the k8s cluster.
ingress:
# Enable/disable exposure using ingress.
enabled: false
hosts: []
# TLS section. Unlike other ingresses, this follows the format at
# https://kubernetes.io/docs/concepts/services-networking/ingress/#tls
# tls:
# - hosts:
#   - 1.example.com
#   secretName: example1-com-tls-secret
# - hosts:
#   - 2.example.net
#   secretName: example2-net-tls-secret
# Map of ingress annotations.
annotations: {}
# Ingress path.
path: /
externalIPs: []

In our case, we have nginx controller running in different namespace and Kong Gateway running in another one. Will that work if I make any changes to the CR?

I would like to do the same here. I have kong proxy, kong admin , and kong-manager deployed within Google Kubernetes Engine and when I try to secure the endpoints I get a 400 Bad Request error, The plain HTTP request was sent to HTTPS port. My assumption is this has to do with the Kong ingress controller. No matter what configs I place in the yaml file I'm getting that same 400 Bad Request Error. So I want to use nginx ingress class instead of kong's. I believe nginx will finally allow http to https redirect and I can reach kong-admin and kong-manager securely.


Introduce a move command for moving cursor in the editor

Introduce a move command for moving cursor at a logical view position in the editor
This will allow the extension writers to place the cursor at a provided position in the editor
Usage:

Command Id: cursorMove
Arguments: {to: $ViewPosition, select: boolean, by: $By, amount: number}
Defaults: amount =  1. by based on to value.

ViewPosition

left: Moves the cursor left by n characters.

right: Moves the cursor right by n characters.

up: Moves the cursor up by n lines or wrapped lines.

down: Moves the cursor down by n lines or wrapped lines.

wrappedLineStart: Places the cursor at the start of the current line in the view.

wrappedLineFirstNonWhitespaceCharacter: Places the cursor at the first non white space character of the current line in the view.

wrappedLineEnd: Places the cursor at the end of the current line in the view.

wrappedLineLastNonWhitespaceCharacter: Places the cursor at the last non white space character of the current line in the view.

wrappedLineColumnCenter: Places the cursor at the center of the line in the view.

viewPortTop: Move the cursor to the first non white space character of Nth line from the top of the view.

viewPortCenter: Move the cursor to the first non white space character of the center line of the view.

viewPortBottom: Move the cursor to the first non white space character of Nth line from the bottom of the view.

By

line - Default for line specific commands
wrappedLine
character - Default for character specific commands
halfLine - Only for character specific commands.

Set select arg to true, to select text for above

@rebornix I am going to implement a command name 'move' which can take arguments. Based on the arguments command will move the cursor appropriately. Let me know if you have any specifications so that I can make them part of it.
As a beginning, this will let to move the cursor to

first character in the screen line
first non white space character in the screen line

// set anchor=active for just positioning the cursor (no selection)
editor.selections = [new vscode.Selection(anchor, active)]

@siegebell No, above will place the cursor as per the editor model. Instead this new command will allow you to move it to a position you wanted on the screen. For eg: If a line is wrapped and if you want to move the cursor to the beginning of the line on the screen (not to the beginning of the complete line) above javascript will not able to do that.

@siegebell We can but this will introduce view properties on the document model which is not a good idea. Hence we intended to provide commands so that users can execute them.

@sandy081 these commands look good to me and really thank you! For now I still need time to check whether they are suitable to Vim integration, whether it's a direct support or a workaround.

@sandy081 just played around with these APIs, thank you very much as they make the implementation really simple our side. I bet other extensions can also benefit from these APIs.
There is some unexpected behaviors while running  cursorMove, the first thing is lineEnd actually moves the cursor the first character of next screen line. The position line and character are both zero-based, so I'm guesssing maybe we forget to decrease by 1 while calculating.
Another one is lineCenter is just keeping the cursor where it was. Not sure did I miss anything.

@rebornix I see lineEnd and lineCenter commands are behaving as expected for me. I guess you would be calling these arguments just similar to others. If you can provide any information that can help in reproducing would be great.

@sandy081 I used the keywords put in this issue description (up, down), while it's lineUp and lineDown per your code. It was my misunderstand and lineCenter, lineUp and lineDown works perfectly.
But I still have problem with lineEnd, will sync with @sandy081 offline. All others are really awesome.

@rebornix Yah, I forgot to update the description with updated keywords. Thanks for pointing this.

@rebornix It would be great if you can give me an updated list of VIM commands (all variations) which are implemented / implementable (with existing API) and which are still not.  I have seen the VIM extension road map but I think it is not up to date or does not have all variations.. we can also discuss it offline.
Thanks

@rebornix Commands H, M and L are supported with selection.

@sandy081 I just found that Vim's H, M, L are not referring to Screen Line. They are referring to real line numbers on the gutter. I'm not sure whether it's confusing putting both screen line and real line in the same API all together, what do you think?

@rebornix above commands are moving to Screen line not to real line. Can you please check by having a very small wrapping size. I think the issue you are seeing is if the screen line above top screen line is little visible then H command jumps to that. Also there is a similar issue for bottom of the view - #9542. It looks to me an issue while computing the lines layout. Since Alex is on vacation we have to wait on that.
Otherwise, H, M and L commands are moving to screen line. Please double check with a line wrapped into more than 2 lines.

@rebornix

Re-Implemented viewPortTop, viewPortBottom and viewPortCenter commands by real lines instead of wrapped lines.
Implement left and right commands to move cursor left and right respectively by 1, n or halfLine characters
Changed the key word names

Adopt to the above changes and let me know the feedback.
Thanks

@rebornix Please note the small change done to the API for consistency and clean

Renamed argument parameter amount to value.

Please adopt to these chages

@sandy081 cursorMove does not respect already selected text.
Vjjjj ing works find but Vkkkkkk ing will break selected code.
    await vscode.commands.executeCommand("cursorMove", {
     to: "up",
     by: "wrappedLine",
     select: true,
     value: count
   });

I've spent a couple of hours on this, it seems that activeTextEditor.slection is not handled properly.
@sandy081 Please give us the same API without moving the cursor, Vim extensions are masters on how to move the cursor themselves.

@sandy081  This is how we can practically use your api, moveDown record the position then moveUp. This makes life hard and moving around slow and if you move pass the first line or the last line some wired visual feedback can be seen (you can see cursor moving up and then down and them up) ❗
Let's face it, no other plugin is going to use these commands other than VIM extensions 😉
And we already know that no internal editor stuff needs to be exposed for folding info.
These makes both vim extension writer life and vscode team easier.
So i'm going to ask once more hoping that you reconsider this wrong design approach 😄

@sandy081
I would like to chime in along with @aminroosta here.
Unfortunately, the current cursorMove API is not powerful enough to implement most core features of Vim. Simply moving the cursor is not enough - we actually need the position. Without the position, there's no practical way to use the value any of the visual modes (which effectively take the position, then apply some transformations on it, and then create a new selection object). This means that visual mode, visual line mode and visual block mode cannot be implemented using this API.
Again, we are all super appreciative of all the work that you guys do on such a great editor. Let us know if there's anything we can do to help!

@aminroosta

@sandy081 cursorMove does not respect already selected text.
Vjjjj ing works find but Vkkkkkk ing will break selected code.

It is working for me very smoothly. Please see the attached video. I call the following command just like you did
vscode.commands.executeCommand("cursorMove", {
     to: "up",
     by: "wrappedLine",
     select: true,
     value: count
   });

Unfortunately, the current cursorMove API is not powerful enough to implement most core features of Vim.

@rebornix, In regards to @johnfn comment,  may I know what is missing here? I remember that this can be used to implement most of cursor movement functionality in Vim. In fact, I implemented in our [Vim sample extension] (https://github.com/Microsoft/vscode-extension-samples/tree/master/vim-sample).

@sandy081 Thanks for taking time and checking this issue. So there should be something wrong with my code. If your code is available online on github please let me know.

@aminroosta here is our [Vim sample extension] (https://github.com/Microsoft/vscode-extension-samples/tree/master/vim-sample).

@sandy081 No no no, vkkkk (lower case v) is perfectly fine, i'm having a hard time with Vkkkk (upper case V) that selects lines.

@sandy081 It seems only lower case vk can be implemented with cursorMove, i can't fine any information in the documentation (first comment above) that suggests how to implement Vkkk ing.

@aminroosta Just to note that the sample vim extension does not have all complete vim features. This is an example/reference implementation showing how some of the vim features can be implemented using the editor commands.
With regards to upper case V, sample supports moving to visual mode only on lower case v. You can refer to this to implement the other.
@rebornix might help you in how to consume this.

@sandeep thanks for the tip.
@aminroosta I'm adding these command sin my separate PR but it does hacking to the view updating phase so it's still WIP. I'll let you know if I solve the issues you met.

I see the problem, I only used cursorMove for screen lines moving but not regular lines, that might describe I don't see any issues @aminroosta ran into. But @aminroosta is doing the right thing, implementing all Vim cursor moving commands by cursorMove instead of doing things in different ways. Let me give them a check real quick.


show durations and published date on the video thumbnails

This pr will close multiple issues -
close #28

add ffprobe package for getting video metadata
add @ffprobe-installer/ffprobe for platform independent binary installer
add durations filed in videos collection to store video durations in second
get video durations in seconds using ffprobe and store that while storing video info in db
use react-moment package to format(HH:mm:ss) video durations.

close #29

shows recordingDate as published date in video thumbnails.
use react-moment package to format(DD/MM/yyyy)  recordingDate.

thanks for the pull request brother. I would like to request you to allow me some time (2 weeks) to complete my current refactoring on the server side (making genric repo thing). then i can accept this PR. Thanks 🙏

@aninda052 please have a look
https://github.com/foyzulkarim/mern-video-streaming/issues/51

@aninda052  please resolve the conflicts.. thanks 👍

@aninda052 please resolve the conflicts.. thanks +1

@foyzulkarim conflict resolve

what is the difference between this work and #47 ?
either use different code in different PR, or close either of the PR please.

closing this pr as i already merge  feature/video-durations into feature/update-video-status


fix(gateway): make close messages bypass the command ratelimiter

Only Discord gateway events are ratelimited, see https://discord.com/developers/docs/topics/gateway#rate-limiting.

#1946 is now merged


Expose recursive flag

Addresses #114  by exposing a flag allowing the user to choose whether they want to list recursively or not.

clang-tidy seems to be confused/lying, so I am going to ignore it

typo in the first commit message 'paramter' -> 'parameter'
looks good

Force-pushed only to amend base commit message.


About paper

Thank you very much for your team's open source code, from which I have learned a lot. However, regarding the content of the paper, I have some concerns and I hope you can help me clarify them.

Does your paper propose a method to filter grab poses based on task instructions? In other words, does the network in your paper have limitations in generating a large number of grab poses from point clouds?

Hi @toolren thanks for the qn and interest in the paper. The paper/GCNGrasp network concerns itself with task-oriented grasping based on task instructions (e.g. from language). It decomposes the problem into 1) stable grasp pose generation and 2) discriminative task-oriented evaluation, and GCNGrasp only does the latter. It needs a separate network that proposes stable grasp poses (GPD in this case but you can use others).
As such, yes, the network is restricted in that way. It is upper bounded by the diversity of grasps generated by the underlying stable grasp generation algorithm

hope that clarifies!

OH, thanks for your reply, I got it.


Feature/35 contact page

Descriptions

Link to UI design (if possible): TaoTech

Provide screenshots or videos that the function is working.
https://user-images.githubusercontent.com/110534329/222376099-77700a71-f5f5-4848-99c1-6c17337e342e.mov
Provide screenshots of no warning, or error in the console.

[x] All unit tests passing and coverage of testing is more than 90% unless special needs.
[x] No merge conflict with the main branch, make sure your code is up to date with main.
[x] Check PR title is aligned with your ticket.
[x] Check your app will run without crashes.
[x] Check if there are hard-coded values.
[x] No //eslint-disable-next-line no-unused-vars, unless otherwise specified.
[x] Check you are using camelCase && PascalCase correctly.
[x] Remove unused comments.

Before reviewing, please check the new feature:
https://user-images.githubusercontent.com/110534329/223355864-46c93f3f-950b-458c-ab51-1f9268595822.mov


Use config platform to check php compatibility

@GrahamCampbell this is not working. PHPUnit 5.0 is still added on requirements:
https://travis-ci.org/Soullivaneuh/php-cs-fixer-styleci-bridge/jobs/83450934#L116

Looks like bug in composer then.

Composer's require command should be able to deal with that without messing up.

@Seldaek what do you think about this?


blockpy-installation

Local installation of BlockPy. Everyone should have this. See #10.
Submit picture of BlockPy running locally (see picture in comment as an example).

Whoops didn't mean to do that, sorry!


Error while running GitHub Scan

I am getting this error when I am running Github Scan:
[GIN-debug] [WARNING] Running in "debug" mode.
Web interface available at http://<IP_ADDRESS>:9393
We have these orgs: [a]
We have these users: []
We have these repos: []
Gathering github organizations...

Retrieved repository a/objectivec-quickstart-platform from org a
 Retrieved repository a/python-quickstart-platform from org a
 Retrieved repository a/kotlin from org a
 Retrieved repository a/graphql-a-subscriptions from org a
panic: sync: negative WaitGroup counter

goroutine 1 [running]:
sync.(*WaitGroup).Add(0xc000245690, 0xffffffffffffffff)
        /usr/local/go/src/sync/waitgroup.go:74 +0x147
wraith/core.AnalyzeRepositories(0xc0000c9600)
        /home/humayl/go/src/wraith/core/analysis.go:96 +0xea
wraith/cmd.glob..func1(0x1654840, 0xc000081b30, 0x0, 0x5)
        /home/humayl/go/src/wraith/cmd/scanGithub.go:79 +0x7c8
github.com/spf13/cobra.(*Command).execute(0x1654840, 0xc000081ae0, 0x5, 0x5, 0x1654840, 0xc000081ae0)
        /home/humayl/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:854 +0x2c2
github.com/spf13/cobra.(*Command).ExecuteC(0x16545a0, 0x44736a, 0x15b16c0, 0xc000000180)
        /home/humayl/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:958 +0x375
github.com/spf13/cobra.(*Command).Execute(...)
        /home/humayl/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:895
wraith/cmd.Execute()
        /home/humayl/go/src/wraith/cmd/root.go:22 +0x31
main.main()
        /home/humayl/go/src/wraith/main.go:29 +0x25

make sure you include --num-threads in command line arguments, for some reason default number of threads set to -1 which causes the error
core/session.go line 57 has the -1 value b

make sure you include --num-threads in command line arguments, for some reason default number of threads set to -1 which causes the error
core/session.go line 57 has the -1 value b

@malinoismalware thanks this fixed the issue but i now that i am trying to use the config file i am getting error here is the screenshot of my config and the error that i am getting

@malinoismalware thanks this fixed the issue but i now that i am trying to use the config file i am getting error here is the screenshot of my config and the error that i am getting

@malinoismalware Thanks for clearing that up. I am trying to work through that piece. -1 in the codebase means full bore, threads are calculated based on the number of repos, commit depth goes back to the beginning of time, etc. There is a bug where if only a single repo is given both the commit depth and number of threads will puke at times. Still trying to trace it through as I can only replicate it sometimes.

@osama-set Is the token scoped correctly. I have seen this several times, and generally, it is because either the user has not given the token the correct access rights or is trying to reach a repo/org they don't have access rights to. The simplest way is to test this is to create a God token with full rights and try again.
If it works you know the repo/org permissions are correct and it was the token cope. If it does not work then you need to look closer at the repo/org permissions assigned to the token user.

@malinoismalware @osama-set Looking closer at the debug output, thanks for that, it shows no sigs added, and no valid repo's and orgs. I am almost 100% sure looking at the output and tracing the code, the root problem was no orgs or repos. That is where the waitgroup is unhappy as it can't find targets to thread. Setting a specific thread count means it will not be dynamically set and will ignore the count of repos and orgs, in this case, 0
Test it by doing a scan with no repos and orgs, that should give you the same error, just leave the lists empty. After that, test it by setting the count to 1 and still give it no repos or orgs, the problem should not happen. Loading no sigs should not give an error at all, it should just pass through, not finding anything.
Let me know the results if you can confirm mine I will dig in and through some code around the issue to keep it from panicking.


Rework SpikeCosim::step() to handle exception on 1st ISR instr

Seperate concerns so the flow of stepping spike and checking against
the ibex RVFI data is clearer. One 'step' of either system produces
different amounts of progress, and this conditional-checking-and-stepping
is needed to tie up the flows.
Closes #1687

Force-pushed new changes which : ran clang-format, addressed nit, reword some longer prose-like comments to be clearer.


Executed Steps in NaN hour

It displays NaN after running all the tests. See http://i.imgur.com/0W2b4YG.png

Could you clone this repository and try to reproduce this issue with this example: https://github.com/bcaudan/jasmine-spec-reporter/tree/master/examples/node?

Closed due to lack of information. If you are facing this behavior, please fill a new issue.

Sorry for not being able to reply lately. Already found the reason why. Thank you.


onRunSuccess/onThreadComplete hooks called in timeout flow

In 1.3.x, under thread isolation, when a HystrixCommand times out, the onRunSuccess/onThreadComplete hooks are invoked by the Hystrix thread eventually, even if the calling thread has already moved through the fallback flow.
In master/1.4, only the onThreadComplete hook is invoked (eventually).
/cc @benjchristensen, @KoltonAndrus What should the semantics be?  I'd like to decide that and then enforce it in both 1.3.x and master

I think it should call eventually whenever it does complete. A thread always ends (unless it's an infinite loop).
It is clearer for it to eventually be called than to have cases where it only starts but never "ends". Also I don't want to break 1.3 behavior.

Yeah, that's the way I was leaning, but didn't want to bias your response :)  This will be a change in master only then.  I'll also add some Javadoc to clarify this case, as it's not obvious.

Added Javadoc in #512


The output NetCDF files are not identical between runs with MPI and without MPI

@rmclaren
I tested converting the satwind BUFR file using bufr2netcdf.x with and without  MPI.
The output files are not consistent (identical) from tests with MPI and without MPI.
The output files from NO MPI run
17067320 gdas.t00z.satwind_abi_goes-16.tm00.nc --- data count:  965550
30718782 gdas.t00z.satwind_abi_goes-17.tm00.nc --- data count:  1799711
18687  gdas.t00z.satwind_abi_goes-18.tm00.nc  --- data count: 0

The output files from MPI run ( srun -n 12)
 57349351 gdas.t00z.satwind_abi_goes-16.tm00.nc --- data count: 3115825
 76874918 gdas.t00z.satwind_abi_goes-17.tm00.nc  --- data count: 4301847
 18687  gdas.t00z.satwind_abi_goes-18.tm00.nc  --- data count:  0

Also, MPI runs with 2, 4, or 8 processors did not run to completion successfully. They were hung until they ran out of the node allocation time.  The test ran to completion with 12 processors.  However, the output files are not consistent with the NO MPI output.
The input, configuration, mapping, and script files can be found on HERA:
/scratch1/NCEPDEV/da/Emily.Liu/EMC-bufr-query/run_satwind
./process_bufr2netcdf
./process_bufr2netcdf_mpi  4
./process_bufr2netcdf_mpi 12

The CrIS and IASI output files are consistent between MPI and No MPI runs.
But, output files are not consistent for satwind.
The satwind mapping file is similar to CrIS and IASI, except that the satwind mapping has subsets defined as the following:
bufr:
  subsets:
    - NC005030
    - NC005031
    - NC005032
    - NC005034
    - NC005039

  variables:
    # MetaData
    timestamp:
      datetime:
        year: "*/YEAR"
        month: "*/MNTH"
        day: "*/DAYS"
        hour: "*/HOUR"
        minute: "*/MINU"
        second: "*/SECO"

    latitude:
      query: "*/CLATH"

    longitude:
      query: "*/CLONH"

    satelliteId:
      query: "*/SAID"

    satelliteZenithAngle:
      query: "*/SAZA"

         ------

Is it possible that the MPI is not implemented correctly for data mapping with subsets defined?

Thanks, I'll take a look

@rmclaren Tested the satwind again with MPI using your latest update for feature/in_parallel
Test with MPI = 12
ls -l ../bufr_backend_mpi12/*
-rw-r--r-- 1 Emily.Liu da 17067320 Jul 24 16:22 gdas.t00z.satwind_abi_goes-16.tm00.nc
-rw-r--r-- 1 Emily.Liu da 30718782 Jul 24 16:22 gdas.t00z.satwind_abi_goes-17.tm00.nc
-rw-r--r-- 1 Emily.Liu da    18687 Jul 24 16:22 gdas.t00z.satwind_abi_goes-18.tm00.nc

Test without MPI
ls -l ../bufr_backend/*
-rw-r--r-- 1 Emily.Liu da 17067320 Jul 24 16:19 ../bufr_backend/gdas.t00z.satwind_abi_goes-16.tm00.nc
-rw-r--r-- 1 Emily.Liu da 30718782 Jul 24 16:19 ../bufr_backend/gdas.t00z.satwind_abi_goes-17.tm00.nc
-rw-r--r-- 1 Emily.Liu da    18687 Jul 24 16:19 ../bufr_backend/gdas.t00z.satwind_abi_goes-18.tm00.nc

The size and number of observations of the output files matched with and without MPI.
I will check if the results are reproducible with various MPI numbers
I will test script backend with and without MPI
Will report back soon.

@rmclaren The output files are reproducible with varying MPI numbers from 4, 8, 12 and 24
[Emily.Liu@hfe06 bufr_backend]$ ls -l
total 46688
-rw-r--r-- 1 Emily.Liu da 17067320 Jul 24 16:19 gdas.t00z.satwind_abi_goes-16.tm00.nc
-rw-r--r-- 1 Emily.Liu da 30718782 Jul 24 16:19 gdas.t00z.satwind_abi_goes-17.tm00.nc
-rw-r--r-- 1 Emily.Liu da    18687 Jul 24 16:19 gdas.t00z.satwind_abi_goes-18.tm00.nc
[Emily.Liu@hfe06 bufr_backend]$ ls -l ../bufr_backend_mpi4
total 46688
-rw-r--r-- 1 Emily.Liu da 17067320 Jul 24 16:30 gdas.t00z.satwind_abi_goes-16.tm00.nc
-rw-r--r-- 1 Emily.Liu da 30718782 Jul 24 16:30 gdas.t00z.satwind_abi_goes-17.tm00.nc
-rw-r--r-- 1 Emily.Liu da    18687 Jul 24 16:30 gdas.t00z.satwind_abi_goes-18.tm00.nc
[Emily.Liu@hfe06 bufr_backend]$ ls -l ../bufr_backend_mpi8
total 46688
-rw-r--r-- 1 Emily.Liu da 17067320 Jul 24 16:30 gdas.t00z.satwind_abi_goes-16.tm00.nc
-rw-r--r-- 1 Emily.Liu da 30718782 Jul 24 16:30 gdas.t00z.satwind_abi_goes-17.tm00.nc
-rw-r--r-- 1 Emily.Liu da    18687 Jul 24 16:30 gdas.t00z.satwind_abi_goes-18.tm00.nc
[Emily.Liu@hfe06 bufr_backend]$ ls -l ../bufr_backend_mpi12
total 46688
-rw-r--r-- 1 Emily.Liu da 17067320 Jul 24 16:22 gdas.t00z.satwind_abi_goes-16.tm00.nc
-rw-r--r-- 1 Emily.Liu da 30718782 Jul 24 16:22 gdas.t00z.satwind_abi_goes-17.tm00.nc
-rw-r--r-- 1 Emily.Liu da    18687 Jul 24 16:22 gdas.t00z.satwind_abi_goes-18.tm00.nc
[Emily.Liu@hfe06 bufr_backend]$ ls -l ../bufr_backend_mpi24
total 46688
-rw-r--r-- 1 Emily.Liu da 17067320 Jul 24 16:31 gdas.t00z.satwind_abi_goes-16.tm00.nc
-rw-r--r-- 1 Emily.Liu da 30718782 Jul 24 16:31 gdas.t00z.satwind_abi_goes-17.tm00.nc
-rw-r--r-- 1 Emily.Liu da    18687 Jul 24 16:31 gdas.t00z.satwind_abi_goes-18.tm00.nc
[Emily.Liu@hfe06 bufr_backend]$

The run with MPI=12 finished in 12 seconds
The run without MPI finished in 144 seconds

The script backend for satwind also worked with MPI.
The 24 processors took 15 seconds.  3 seconds more than the bufr backend because python is involved and three new variables were added.
This is great! Good job!!

@emilyhcliu Thanks for your patience and for testing everything :)


support UE5.3

a commit from #27 and a bugfix

Thanks @matyalatte for accepting the change and for adding that fix on top of it! ❤
I'm sorry I didn't see your message yesterday.
Very happy to see UE5.3 support in the official UE4-DDS-Tools. 🥳


Add notebook for Multi Tenancy RAG

Description
PR to add Multi-Tenancy RAG with LlamaIndex notebook.
Fixes # (issue)
Type of Change
Please delete options that are not relevant.

[ ] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Added new unit/integration tests
[ ] Added new notebook (that tests end-to-end)
[ ] I stared at the code and made sure it makes sense

Suggested Checklist:

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I ran make format; make lint to appease the lint gods

Added it under new folder so that for different vectordb's we can add multi-tenancy rag implementation notebook.


fix:rcBuildLayerRegions missing areaType

fix:rcBuildLayerRegions missing areaType

This fix is absolutely vital to get area IDs working correctly. Here is what happens without it:

The black blocks use a different area ID and should show up in red rather than blue. As you can see, the blue area ID takes over in many places. The dark blue grid lines are tile borders, so that's why the area ID is correct for half the black block, but not the other half, because a tile border separates them.
With the fix, the area IDs work as expected:

It would be great, if this PR could be merged to the main branch.


change ddb to be on demand billing method

• Does any part of it require special attention?
• Does it relate to or fix any issue?
closes #9
Standard checks

Unit tests: Any special considerations?
Docs: Do we need to update any docs, internal or public?
Backwards compatibility: Will this break existing apps? If so, what would be the extra work required to keep them working?

We'll need to notify customers that use this on the change of behaviour. Especially those that sed the capacity before deploying.

Do we know who that is? I agree we should, although if they are really only using the 1 read/write capacity it wont get too expensive since they barely use it anyways. Agreed though just have no idea who is using it


[UWP] RichTextBlock is not a valid type

Platform

UWP

Version of SDK
master
Issue
I get this warning when running the visualizer with the RichTextBlock.json payload

This is because the visualizer validates cards against the schema which has not been updated.

Looks like this could be fixed once #2407 is fixed

Resolving as dup of # #2407


Add override keyword in codebase

For #120675
This uses a script to add the override keyword to places that need it in the codebase
Note that we can't enable the --noImplicitOverride setting yet since there are still around 200 errors that require further attention

Merging this since it is a large PR and quickly starts having merge conflicts.
Will continue following up separately on remaining 200 or so cases


Some improvements to Mix visual shader function

Changed default port values to more useful eg. mix(0, 0, 0) to mix(0, 1, 0.5)
Renamed c port to weight (which makes it less confusing to user)
Added new overload: mix(vector, vector, scalar) as (VisualShaderNodeVectorMixScalar)

These changes will not break compatibility. For 4.0, I think, its reasonable to rename VisualShaderNodeVectorInterp and VisualShaderNodeScalarInterp to the VisualShaderNodeVectorMix and VisualShaderNodeScalarMix respectively.

Renamed VisualShaderNodeVectorMixScalar to VisualShaderNodeVectorScalarMix to make it compatible with other similar functions

Thanks!


Create requirements.txt for python venv installations

Document python package versions in requirements.txt file and standardize python environments for users.

Resolved in commit 727b1f4b655eef17133978ed59a4d556addb0ca5


Use go modules

From golang 1.14 modules are defacto standard

Let's setup an hour meeting to discuss what we learned moving the plugin repos to go modules and help the controller contributors get up to speed on this.

Closed https://github.com/konveyor/mig-controller/pull/439 due to staleness. Still plan to switch to go modules in the future.


Bug 1769015: Add kubebuilder annotations to the network types

Add missing annotations and validation markers for network types.

/assign @danwinship

My earlier comments still hold.

There are a few elements missing:
1. You need to copy the crd yamls from network-operator into `network/v1` directory, similarly how it's done in `operator/v1`, for example.

2. You need to add this rule to Makefile:

$(call add-crd-gen,network,./network/v1,./network/v1,./network/v1)

next to similar lines for operator.
The above 2 steps will ensure we will update the yamls inside this project, and you'll be able to consume it nicely in openshift/cluster-network-operator#398

For 1, is there a naming scheme convention? Right now the file in the cluster-network-operator repo is called 001-crd.yaml.
I have made the change for calling add-crd-gen

/test verify

https://github.com/openshift/api/pull/527/files/6d8451fea22ffaed53bea1e362b6da937e41d58a#diff-d22a7c6691ae44dcb021a880f462f6f9 is critical. Otherwise lgtm.

@soltysh @sttts can one of you approve and lgtm this?


v-validate does not work with v-class

My html code

        <input type="text" v-model="current_page" number v-validate="min: 1, max: 9" v-on="keyup: onGoToPage | key 'enter'">

Whatever numbers typed in the field, the v-class won't work.

I tried the your issues on the jsfiddle.
https://jsfiddle.net/kazupon/q7xcbuxd/6/
It seems to work normally.
Can you give me your issue code on the jsfiddle ?

@kazupon
Hi, in your sample, VueJS file been blocked since it request by http resouce, jsfiddle now has supported https.

I updated sample.
https://jsfiddle.net/kazupon/q7xcbuxd/8/

@kazupon  It's a great sample for vue-validator. :+1:
I fork one for adding an error effect to make it clear. https://jsfiddle.net/vz5rLzny/

I see, I didn't get this line in my code.
Vue.use(window['vue-validator'])


Logging does not work on local dev docker

This has been a long-standing issue (in fact, just found a duplicate here https://github.com/makeabilitylab/makeabilitylabwebsite/issues/815). Perhaps I can discuss with Jason. Appears to be another one of those pesky permissions issues:
website_1  | Traceback (most recent call last):
website_1  |   File "/usr/local/lib/python3.8/logging/config.py", line 563, in configure
website_1  |     handler = self.configure_handler(handlers[name])
website_1  |   File "/usr/local/lib/python3.8/logging/config.py", line 744, in configure_handler
website_1  |     result = factory(**kwargs)
website_1  |   File "/usr/local/lib/python3.8/logging/handlers.py", line 148, in __init__
website_1  |     BaseRotatingHandler.__init__(self, filename, mode, encoding, delay)
website_1  |   File "/usr/local/lib/python3.8/logging/handlers.py", line 55, in __init__
website_1  |     logging.FileHandler.__init__(self, filename, mode, encoding, delay)
website_1  |   File "/usr/local/lib/python3.8/logging/__init__.py", line 1147, in __init__
website_1  |     StreamHandler.__init__(self, self._open())
website_1  |   File "/usr/local/lib/python3.8/logging/__init__.py", line 1176, in _open
website_1  |     return open(self.baseFilename, self.mode, encoding=self.encoding)
website_1  | PermissionError: [Errno 13] Permission denied: '/code/media/debug.log'
website_1  |
website_1  | The above exception was the direct cause of the following exception:
website_1  |
website_1  | Traceback (most recent call last):
website_1  |   File "manage.py", line 10, in <module>
website_1  |     execute_from_command_line(sys.argv)
website_1  |   File "/usr/local/lib/python3.8/site-packages/django/core/management/__init__.py", line 381, in execute_from_command_line
website_1  |     utility.execute()
website_1  |   File "/usr/local/lib/python3.8/site-packages/django/core/management/__init__.py", line 357, in execute
website_1  |     django.setup()
website_1  |   File "/usr/local/lib/python3.8/site-packages/django/__init__.py", line 19, in setup
website_1  |     configure_logging(settings.LOGGING_CONFIG, settings.LOGGING)
website_1  |   File "/usr/local/lib/python3.8/site-packages/django/utils/log.py", line 76, in configure_logging
website_1  |     logging_config_func(logging_settings)
website_1  |   File "/usr/local/lib/python3.8/logging/config.py", line 808, in dictConfig
website_1  |     dictConfigClass(config).configure()
website_1  |   File "/usr/local/lib/python3.8/logging/config.py", line 570, in configure
website_1  |     raise ValueError('Unable to configure handler '
website_1  | ValueError: Unable to configure handler 'file'
website_1  | Traceback (most recent call last):
website_1  |   File "/usr/local/lib/python3.8/logging/config.py", line 563, in configure
website_1  |     handler = self.configure_handler(handlers[name])
website_1  |   File "/usr/local/lib/python3.8/logging/config.py", line 744, in configure_handler
website_1  |     result = factory(**kwargs)
website_1  |   File "/usr/local/lib/python3.8/logging/handlers.py", line 148, in __init__
website_1  |     BaseRotatingHandler.__init__(self, filename, mode, encoding, delay)
website_1  |   File "/usr/local/lib/python3.8/logging/handlers.py", line 55, in __init__
website_1  |     logging.FileHandler.__init__(self, filename, mode, encoding, delay)
website_1  |   File "/usr/local/lib/python3.8/logging/__init__.py", line 1147, in __init__
website_1  |     StreamHandler.__init__(self, self._open())
website_1  |   File "/usr/local/lib/python3.8/logging/__init__.py", line 1176, in _open
website_1  |     return open(self.baseFilename, self.mode, encoding=self.encoding)
website_1  | PermissionError: [Errno 13] Permission denied: '/code/media/debug.log'

Interestingly, when I went in and fixed https://github.com/makeabilitylab/makeabilitylabwebsite/issues/985, it somehow also fixed this issue. So, clearly a permissions issue of some sort.
The key was to make a bash shell inside the container and make sure /code/media had correct permissions.

OK, so I figured out how to solve it on Windows WSL2 with ubuntu. The problem is a permissions issue with /code/media in the Docker container.
By default, it looks like /code/media has a permission of drwxr-xr-x  2 root root  4096 Oct 13 02:59 media. I just changed this to 777 and it worked. Here are the steps:

Open the Docker container in a bash shell: > docker exec -it makeabilitylabwebsite_website_1 bash
Make sure you are in the /code directory. Try to change the permissions by $ chmod 777 media
If that doesn't work, you'll need sudo access. So, try:apache@e86363656952:/code$ su
Password:
root@e86363656952:/code# chmod 777 media

Restart the Docker container (e.g., from Ubuntu, > docker compose-up

I tested this on a separate system and it worked


"an item with the same key has already been opened" is shown when opening an F# project in Visual Studio 2017

Whenever you open an F# source file from an F# project in Visual Studio 2017 for the first time, and sometimes later again, you will get a static dialog popup with:

an item with the same key has already been opened

Repro steps
Not sure, but I wasn't the first to notice (see for instance this comment: https://github.com/Microsoft/visualfsharp/issues/3267#issuecomment-311551337). But this warrants its own issue, as it keeps happening with most recent previews.
I have one project where it reproduces after I remove all NuGet packages from disk, but only for one project in the solution, in this case one using UnQuote, which compiled previously, but in the editor and from MsBuild command prompt, it didn't load the library at all.
Not sure the one has to do with the other. I see the duplicate-key popup happen on any project, whether or not it was saved through Visual Studio 2017 before or not. The only common denominator seems to be that:

It happens whenever you open a source file for the first time on that computer with that version of VS2017
It happens with F# projects only
It seems to happen occasionally after the first time, or after removing all NuGet packages (but again, this may not be related)

Expected behavior
No error dialogbox.
Actual behavior
The error as shown above.
Known workarounds
I don't know of any workarounds (apart from clicking the messagebox away, you can continue working).
Related information

Seen on Windows 10 Creative Ed and Windows 7 latest SP
Visual Studio, at least since Preview 3 through Preview 7 (15.3.0 Preview 7.0)
F# 4.0, 4.1 (or internally, version craziness: Core dlls having <IP_ADDRESS> or <IP_ADDRESS>)

This should be fixed here: https://github.com/Microsoft/visualfsharp/blob/master/vsintegration/src/FSharp.Editor/LanguageService/LanguageService.fs#L465
The only way to test this out at the moment is to use a self-built install of the Visual F# tools in VS 2017 Update 3.  You can do that by following the instructions in DevGuide.md to get prerequisites, then running build vs release.  If you run the VISX under bin/release/net40, then it will replace your current F# tools with the latest master.  I'll re-open this if you can still reproduce it.

@cartermp: I haven't seen this for a while now (probably since 15.3 Preview 6 or so) and I for sure haven't seen it with <IP_ADDRESS> Preview 1 and today's nightly, so I consider this confirmed fixed unless it resurfaces.

@abelbraaksma Good to hear!


ACTIVITYOG_SERVICE_ACCOUNT_SECRET

Describe the bug
https://owncloud.dev/services/activitylog/ mentions ACTIVITYOG_SERVICE_ACCOUNT_SECRET as configuration option. All others actually have the ACTIVITYLOG_ prefix.
Steps to reproduce

visit https://owncloud.dev/services/activitylog/

For ease of identifying the issue, the character L is missing:
ACTIVITYLOG_
ACTIVITYOG_SERVICE_ACCOUNT_SECRET


[battery] fixed bugs related to lifecycle and invalid reading

Description
Due to Activity destruction and flutter not cancelling EventChannels subscriptions automatically, there is this need to unregister the BroadcastReceiver by hand so we can avoid leaking that reference.
Reading the docs for getIntProperty I noticed that -1 as not a really good invalid return to the battery level calculation, so I changed to 0 and updated the calling site.
Is this a breaking change?
Related Issues
https://github.com/flutter/flutter/issues/36520#issuecomment-514489072
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[X] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[X] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See [Contributor Guide]).
[X] All existing and new tests are passing.
[X] I updated/added relevant documentation (doc comments with ///).
[X] The analyzer (flutter analyze) does not report any problems on my PR.
[X] I read and followed the [Flutter Style Guide].
[X] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[X] I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy].
[X] I updated CHANGELOG.md to add a description of the change.
[X] I signed the [CLA].
[X] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[X] No, this is not a breaking change.

NOTE:
I would recommend to move EmptyActivityLifecycleCallbacks to the engine or somewhere else and avoid replicating this file or leaving the code dirty with a bunch of replicated empty methods.

If we don't unregister the BroadcastReceiver, what would be the problem? Is there a filed bug report?
This isn't a breaking change as it doesn't change any public API signature nor expected behavior.

There is no bug filled.
The problem is that, is considers that the registration will happen only one time per app creation, but that is a false believe. Even if the Activity is destroyed, the application may be kept alive by the SO and  the old BroadcastReceiver will still be in memory, that BroadcastReceiver has an implicit reference to the plugin class, as it is an anonymous class.
So every time the plugin registration happens, the old BroadcastReceiver/plugin instance will leak.
This is easier to think if you think a add-to-app case where the flutter Activity is open multiple times.

@digiter I asked about the breaking change because, 0 used to be a valid number for a positive measurement, now the plugin will raise it is an error.

@digiter , could you take another look, the CI is green now.

In general this looks good to me, but I'm not confident enough, so I'm leaving approval to Amir.
I'm not sure about breaking change neither.
One thing worth considering is that, should Flutter cancel EventChannels subscriptions automatically? If yes, then please find or file a bug. @Hixie

In general this looks good to me, but I'm not confident enough, so I'm leaving approval to Amir.
I'm not sure about breaking change neither.
One thing worth considering is that, should Flutter cancel EventChannels subscriptions automatically? If yes, then please find or file a bug. @Hixie

This bug was reported
https://github.com/flutter/flutter/issues/32818
While flutter does not implement a call to onCancel plugin developers need to do that themselves.

This PR needs tests.

This PR needs tests.

Hey @Hixie , I agree that some tests would be great, however this plugin has zero android unit tests at the moment and to write propor tests some refactor in the code would be needed (proper DI and inversion of control to a delegate class). Could you slide this requirement to an issue to be tacked in another PR?

@BugsBunnyBR Your contributions are highly appreciated! Please do not close this.


Sync Unity meta data with dbt model.yml metadata on run

Describe the feature
dbt tracks table meta data (name, description, column name, column description) in the model.yml. Unity also has these meta data labels. A dbt run should sync the metadata to Unity.
Describe alternatives you've considered
Writing my own python notebook to parse manifest.json for these meta data items and iterating over it myself to add the metadata to Unity.
Additional context
Both the create and alter statement have the ability to add table comments at table or column level.
Who will this benefit?
Everyone using model documentation. We're creating dbt documentation as we go, Unity should benefit.
Are you interested in contributing this feature?
I have my own method of syncing the data but I don't have any time in the foreseeable future to look into how to get this working inside the dbt-databricks framework.

Hi @EtherJfox,
Could you elaborate what kind of metadata you mean?
Are they different from what persist_docs does?

Didn't know the feature existed. Looks like I'm getting the table level comment on some tables, but not others, and no column comment.
Sample config
version: 2
models:
  - name: finance_datamart__dim_date_accounting
    description: A date dimension used in the Finance data model representing accounting periods
    columns:
      - name: date_key
        description: An integer representation of the date
    config:
      enabled: true
      file_format: delta
      +persist_docs:
        relation: true
        columns: true

Do we need + before persist_docs in the model property file?
According to the General configurations, it's necessary for Project file, but not for Property file?
What if moving the config to the project file?

@ueshin I feel like the persist_docs functionality could be significantly improved for the dbt-databricks implementation.
The way it currently works is by first creating the table, then sending one alter table ... change column ... comment after another for every single table. This takes an unnecessary long time to complete. E.g.

However, there are two alternative approaches here that should be vastly faster.
1. Add comments directly in create (or replace) statements
Databricks supports the following syntax, which takes as long time to complete as any regular create (or replace) statement
create or replace table kumulus_dev.sindre__std.test
(id INT comment 'description', name STRING comment 'description', age INT comment 'description')
COMMENT 'this is a comment'

2. Utilize the Unity Catalog API
With unity catalog, using the unity catalog api could also be a fast option.
https://api-docs.databricks.com/rest/latest/unity-catalog-api-specification.html
In summary
These are only meta operations and should be possible to update in bulk rather than comment by comment. The way it is currently, it's simply not feasible to use persist_docs, but it shouldn't have to be this way.

cc @andrefurlan-db

Could we open a new issue for this performance issue with syncing metadata? I think the initial topic has been resolved with persist_docs.

https://github.com/databricks/dbt-databricks/issues/507


[Bug]: Timeout Exceeded for Retrieving Location on Emulator

Please check the following before submitting a new issue.

[X] I have searched the existing issues.
[X] I have carefully read the documentation and verified I have added the required platform specific configuration.

Please select affected platform(s)

[X] Android
[ ] iOS
[ ] Linux
[ ] macOS
[ ] Web
[ ] Windows

Steps to reproduce
I am following the App Brewery's Flutter tutorial for how to create an application that can retrieve the user's current location and output it. This is the backend code that makes all the magic happen:
class Location {
  double latitude = 0.0;
  double longitude = 0.0;

  Future<void> getCurrentLocation() async {
    if (!await isLocationServiceEnabled()) {
      return Future.error(
        'Location services are disabled. Please activate them.',
      );
    }

    if (isPermissionGranted(await getLocationPermission())) {
      await fetchAndSetCurrentLocation();
    }
  }

  Future<LocationPermission> getLocationPermission() async {
    LocationPermission locationPermission = await Geolocator.checkPermission();

    if (locationPermission == LocationPermission.unableToDetermine) {
      throw Future.error(
          "Unable to determine if location permissions are enabled.");
    } else if (locationPermission == LocationPermission.denied ||
        locationPermission == LocationPermission.deniedForever) {
      locationPermission = await Geolocator.requestPermission();
    }

    return locationPermission;
  }

  Future<bool> isLocationServiceEnabled() async {
    return await Geolocator.isLocationServiceEnabled();
  }

  bool isPermissionGranted(LocationPermission locationPermission) {
    return locationPermission == LocationPermission.whileInUse ||
        locationPermission == LocationPermission.always;
  }

  Future<void> fetchAndSetCurrentLocation() async {
    Position position = await Geolocator.getCurrentPosition(
        desiredAccuracy: LocationAccuracy.lowest,
        timeLimit: const Duration(seconds: 10));

    latitude = position.latitude;
    longitude = position.longitude;
  }
}

The function awai Geolocator.getCurrentPosition() never returns. Even when I increase the time limit to 180 seconds, it still fails to return. I am not sure why this is happening. I am running this on a Simulated Device inside Android Studio.
Expected results
I expect the function to return.
Actual results
The function should return.
Code sample
Code sample
class Location {
  double latitude = 0.0;
  double longitude = 0.0;

  Future<void> getCurrentLocation() async {
    if (!await isLocationServiceEnabled()) {
      return Future.error(
        'Location services are disabled. Please activate them.',
      );
    }

    if (isPermissionGranted(await getLocationPermission())) {
      await fetchAndSetCurrentLocation();
    }
  }

  Future<LocationPermission> getLocationPermission() async {
    LocationPermission locationPermission = await Geolocator.checkPermission();

    if (locationPermission == LocationPermission.unableToDetermine) {
      throw Future.error(
          "Unable to determine if location permissions are enabled.");
    } else if (locationPermission == LocationPermission.denied ||
        locationPermission == LocationPermission.deniedForever) {
      locationPermission = await Geolocator.requestPermission();
    }

    return locationPermission;
  }

  Future<bool> isLocationServiceEnabled() async {
    return await Geolocator.isLocationServiceEnabled();
  }

  bool isPermissionGranted(LocationPermission locationPermission) {
    return locationPermission == LocationPermission.whileInUse ||
        locationPermission == LocationPermission.always;
  }

  Future<void> fetchAndSetCurrentLocation() async {
    Position position = await Geolocator.getCurrentPosition(
        desiredAccuracy: LocationAccuracy.lowest,
        timeLimit: const Duration(seconds: 10));

    latitude = position.latitude;
    longitude = position.longitude;
  }
}

Screenshots or video

Screenshots or video demonstration
None needed.

Version
10.0.0
Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.10.2, on Ubuntu 22.04.3 LTS 6.2.0-26-generic, locale en_CA.UTF-8)
    • Flutter version 3.10.2 on channel stable at /home/mahyar/snap/flutter/common/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 9cd3d0d9ff (3 months ago), 2023-05-23 20:57:28 -0700
    • Engine revision 90fa3ae28f
    • Dart version 3.0.2
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /home/mahyar/Android/Sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /snap/android-studio/127/android-studio/jbr/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✗] Chrome - develop for the web (Cannot find Chrome executable at google-chrome)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[✓] Android Studio (version 2022.2)
    • Android Studio at /snap/android-studio/126/android-studio
    • Flutter plugin version 74.0.2
    • Dart plugin version 222.4582
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] Android Studio (version 2022.3)
    • Android Studio at /snap/android-studio/127/android-studio
    • Flutter plugin version 75.1.2
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] Connected device (1 available)
    • Linux (desktop) • linux • linux-x64 • Ubuntu 22.04.3 LTS 6.2.0-26-generic

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Dear @mahyarmirrashed,
Double check if this is working on the example app in the repository. A emulator is not a suitable solution when testing locations. It also depends on the proper permissions and the AndroidSettings.
Kind regards,

Hi @TimHoogstrate , where can I find the example application in the repository? Also, I would have thought that the emulator would be a great way to test since I can set the location to wherever I want in the world to test that my application works as expected. Is there perhaps a guide on how to test with an emulator correctly for this repository? Regards,

I am pretty sure that my Android Emulator is working correctly. I just set the location to a very precise location and opened up Google Maps. Maps was then able to identify the correct location of where I had set the location. I tried the same thing more than a few times and it would go to the updated location. There was about a 1-second delay. I also gave it the same permissions that I gave my sample application ("While in use").

Hi @mahyarmirrashed,
I took the liberty to also look into this issue and I noticed that you set the desiredAccuracy to LocationAccuracy.lowest when making the call to getCurrentPosition.
On Android accuracy settings will translate into a priority on the native SDK. The LocationAccuracy.lowest will translate in to PRIORITY_PASSIVE which means:

Ensures that no extra power will be used to derive locations. This enforces that the request will act as a passive listener that will only receive "free" locations calculated on behalf of other clients, and no locations will be calculated on behalf of only this request.

This will explain why your application is not returning a result, since it is configured to not actively request a location from the device but instead wait for some other application to get a location and return that same particular location.
We are aware that this seems a bit counter intuitive, however it is the only way to over a shared API between all the different platforms.
The mapping is as follows:

Location accuracy
Android priority
Description

lowest
PRIORITY_PASSIVE
Ensures that no extra power will be used to derive locations. This enforces that the request will act as a passive listener that will only receive "free" locations calculated on behalf of other clients, and no locations will be calculated on behalf of only this request.

low
PRIORITY_LOW_POWER
Requests a tradeoff that favors low power usage at the possible expense of location accuracy.

medium
PRIORITY_BALANCED_POWER_ACCURACY
Requests a tradeoff that is balanced between location accuracy and power usage.

high+
PRIORITY_HIGH_ACCURACY
Requests a tradeoff that favors highly accurate locations at the possible expense of additional power usage.

I think if you would set the desiredAccuracy to LocationAccuracy.medium (or higher) you should get a correct result back from the Geolocator plugin.
In the meantime we will double check if we can improve our documentation and make sure we mention how accuracy is translated into a priority on Android devices.


VLAZ: Watch Marontoto

VLAZ requests the watch of the watch_keyword Marontoto. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Marontoto has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by cigien in Charcoal HQ


Add MockCredential

Context? Why do we need it?

I'd like to use it in our Testing Tutorial since we are writing a unit test for a library that only supports AAD. I think it is a useful thing to expose for such circumstances.


Removed outdated callbacks.
Removed unused variable marker

Hi! I see you've got a couple different changes here but I'm not sure if they're related. (Maybe I don't have the full picture?)
Just in case they aren't related, I want to give you an early heads up that we'd prefer separate pull requests instead of submitting many different changes in bulk in a single PR.


Rename from VW Optimizer/Visual Website Optimizer -> VWO

They want us to use VWO all over.

Needs to include market.


Fix for #337: Also check if Cordova is loaded.. Debugging with the device toolbar breaks…

…as it exposes itself as android, and without StatusBar defined.

It is working on android device without this change.

Works fine on the Android emulator on my machine, without this change. After discussion with @jbdtky , I'm closing this as unnecessary.

Can confirm that it works fine on my real Android device.

This issue was not regarding the Android emulator, I meant the Device toolbar in the Google Chrome console (which emulates the user agent, so you can simulate the device specific css/code)


Suggestion: add "--daemonize" parameter to web command

A --daemonize parameter will allow better integration with daemon watchers because it doesn't blocks. For example I'm trying to configure monit with:
check process gitea
  with pidfile /var/run/gitea.pid
  start program = "/usr/bin/sudo -iu user /bin/bash -c 'cd /home/user/applications/gitea && /home/user/applications/gitea/gitea web start'"
  stop program = "/usr/bin/sudo -iu user /bin/bash -c 'cd /home/user/applications/gitea && /home/user/applications/gitea/gitea web stop'"
  if 5 restarts with 5 cycles then timeout

But it fails because it blocks and doesn't returns.

Just for the record (and for the next soul trying  to do the same) I ended up creating a systemd wrapper around gitea, and I use that as an interface to start / stop gitea with monit. Instructions: https://gogs.io/docs/intro/faqs#how-do-i-run-gogs-at-startup-with-systemd%3F .

We could add that possibility in the mean time you could also simply :
"/usr/bin/sudo -iu user /bin/bash -c 'cd /home/user/applications/gitea && /home/user/applications/gitea/gitea web start &'"

Well I've tried that before, but it doesn't works (even it says the exit status is 0).
Here is my monit.log:
[ART Aug 21 13:01:01] error    : 'gitea' process is not running
[ART Aug 21 13:01:01] info     : 'gitea' trying to restart
[ART Aug 21 13:01:01] info     : 'gitea' start: /usr/bin/sudo
[ART Aug 21 13:01:31] error    : 'gitea' failed to start (exit status 0) -- /usr/bin/sudo: 2017/08/21 13:01:01 [T] Custom path: /home/vizcay/applications/gitea/custom
2017/08/21 13:01:01 [T] Log path: /home/vizcay/applications/gitea/log
2017/08/21 13:01:01 [I] Gitea v1.1.3 built with: bindata, sqlite

in this case try nohup you could also use deamon-start-stop (available on most distrib) that is used for this case in many (old) init.d file.

@sapk nohup wasn't working either, but I was able to solve it with damonize (https://github.com/bmc/daemonize) in the meantime. The only thing is that I havent been able to find the PPA of it still (installed the old way).

yeah would be create if we could run gitea as a daemon
further it would be create if gitea would create a pidfile itself
currently i'm don't get it running with supervisord and a pidfile

Essentially, Go doesn't really do deamonizing well https://github.com/golang/go/issues/227#issuecomment-254735160 and the package I see "highly recommended" https://github.com/sevlyar/go-daemon is IMO not an option since it's 1) POSIX only, 2) very poorly implemented and generally a bad solution.
And in general I'm with the golang devs on this, there's at least 10 different projects for deamonizing a process in linux/BSD, pick one of them.

@bkcsoft Go as a language doesn't has anything to do with daemonizing a process, because it's a behaviour of the process. As far as I can tell, if something is meant to run as a service in the unix world, you can expect at least some of that behaviour (at least dont-block going into the background and write a pidfile), everything else is a hack.

Running gitea web -P ./gitea.pid & works fine for me. Still prints to stdout because it's configures to do so, but it works.


Error about Flownet2 installation

When I tried to execute example command, I encountered with this issues.
Could you please help to solve this problem?
I guess there is a problem regarding the 'resample2d_cuda.cpython-36m-x86_64-linux-gnu.so' file

Where did you get pretrained_model_liteflownet?

@yuyang16101066 I've linked them in the README of the fork or in this issue

@yuyang16101066 I've linked them in the README of the fork or in this issue

Thanks for your reply. I have generated flow with LiteFlowNet.


Mobile id - can't change the field from uin to vid while generating id

Describe the bug
can't change the field from uin to vid while generating id
To Reproduce

Open the app
Click on add UIN
Add a vid number under uin field and click Add ID
It will show an error as invalid uin
Don't clear the vid number that you added and change the field to vid

Current behavior
The field is not changing to vid any more
Expected behavior
The dropdown feild should allow us to change the feild
Screenshots

Device: Android
OS: [e.g. iOS8.1]
Inji app version: [e.g 0.3.0]
Mimoto version: [e.g 1.2.x]
MOSIP Version: <IP_ADDRESS>
Mimoto server: [e.g. https://.....com]
MOSIP server: [e.g. https://...mosip.com]
Additional context
Env: qa5.mosip.net

Changing the dropdown to VID should clear the error message as well as the value entered previously.

Now both the error message and the previous value were clearing when the option changed to either UIN or VID in both cases.

@santhoshsunder I have gone through this bug to solve it. However, by adding a VID number under the UIN field and clicking Add ID, The VC was generated without showing any error. The same happens in the vice-versa case also.

This needs to be retested in the latest IOS 16 build

Verified this in qa5 env, issue is not occoring any more, iam attaching the screen recording bellow, hense closing this bug
Video :
https://user-images.githubusercontent.com/102220709/208162252-ff1f0248-0035-4dd1-8627-5c944c851e09.mp4
Smartphone :

Device: PIXEL 6
OS: Android 13
MOSIP Version: <IP_ADDRESS>
MOSIP server: qa5.mosip.net
APP link; https://mosip.atlassian.net/browse/DSD-1816


container/controller: avoid cancellation with forked pull context

Context cancellations were previously causing Prepare to fail
completely on re-entrant calls. To prevent this, we filtered out cancels
and deadline errors. While this allowed the service to proceed without
errors, it had the possibility of interrupting long pulls, causing the
pull to happen twice.
This PR forks the context of the pull to match the lifetime of
Controller, ensuring that for each task, the pull is only performed
once. It also ensures that multiple calls to Prepare are re-entrant,
ensuring that the pull resumes from its original position.
Signed-off-by: Stephen J Day<EMAIL_ADDRESS>Further mitigation to #24447.

cc @cpuguy83 @tiborvass @aluzzardi @aaronlehmann

Just gave this a full test and we are looking solid.

A test timed out.
04:36:47 --- FAIL: Test (1187.00s)
04:36:47 panic: DockerSuite.TestRunTmpfsMountsWithOptions test timed out after 5m0s [recovered]
04:36:47 	panic: DockerSuite.TestRunTmpfsMountsWithOptions test timed out after 5m0s

Unrelated?

Unrelated?

Completely, restarted builder.

LGTM

LGTM


New languages

I added a few Pakistani and all officially recognised Guatemalan languages. I am submitting a new pull request as I can't seem to be able to rebase my original request (ref #3369) and have a clean working tree.
I hope that this pull request will be acceptable!

Thanks for reworking your branch! This may generate a migration, we should add that here as well if so. Does makemigrations give a project migration?

Thank you! I will try on Monday and let you know.
Just to be sure that I don't make another mistake: all I should do is run
python manage.py makemigrations

...and then see if any migrations are made (and commit & push them if necessary)?
Thanks!
-Julien

દ્વારા: Anthony<EMAIL_ADDRESS>મોકલ્યું: 09 માર્ચ 2018 20:28:40
પ્રતિ: rtfd/readthedocs.org
Cc: Julien Malard; Author
વિષય: Re: [rtfd/readthedocs.org] New languages (#3759)
Thanks for reworking your branch! This may generate a migration, we should add that here as well if so. Does makemigrations give a project migration?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/rtfd/readthedocs.org/pull/3759#issuecomment-371990766, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKMb6xdS-SMeoaxNdWK4I-KB2K2KBQFLks5tcyxIgaJpZM4SkgBp.

Hello,
I can't seem to be able to get a successful migration run on my computer (it says that I need to install packages which I have in fact installed)...any chance you could help me out with the migration?
Many thanks,
Julien Malard

@agjohnson Is there a document on how to do this, somewhere?

@RichardLitt @julienmalard I think the best place to read about this is the Django official documentation at https://docs.djangoproject.com/en/2.0/topics/migrations/
@julienmalard in case you still have problems, please paste the output of the command that it's failing so we can see what it's happening in your side.

Hello all,
Many thanks for your support! I figured out the mistake (seems that some build dependencies were not explicitly listed in the requirements file).
Here is the output from the migration run. I think it means that it worked? I will push the repository now.

(venv) C:\Users\jmalar1\PycharmProjects\readthedocs.org>python manage.py makemigrations
System check identified some issues:
WARNINGS:
?: (1_8.W001) The standalone TEMPLATE_* settings were deprecated in Django 1.8 and the TEMPLATES dictionary takes precedence. You must put the values of the following settings into your default TEMPLATES dict: TEMPLATE_DEBUG.
?: (guardian.W001) Guardian authentication backend is not hooked. You can add this in settings as eg: AUTHENTICATION_BACKENDS = ('django.contrib.auth.backends.ModelBackend', 'guardian.backends.ObjectPermissionBackend').
Migrations for 'builds':
0005_auto_20180315_1604.py:
- Alter field error on build
- Alter field output on build
- Alter field state on build
- Alter field type on build
- Alter field privacy_level on version
- Alter field slug on version
- Alter field type on version
- Alter field from_slug on versionalias
- Alter field to_slug on versionalias
Migrations for 'comments':
0002_auto_20180315_1604.py:
- Alter field decision on moderationaction
- Alter field date on nodesnapshot
Migrations for 'gold':
0002_auto_20180315_1604.py:
- Alter field level on golduser
Migrations for 'integrations':
0003_auto_20180315_1604.py:
- Create proxy model BitbucketWebhook
- Create proxy model GenericAPIWebhook
- Create proxy model GitHubWebhook
- Create proxy model GitLabWebhook
- Alter field integration_type on integration
Migrations for 'oauth':
0009_auto_20180315_1604.py:
- Alter field clone_url on remoterepository
- Alter field ssh_url on remoterepository
- Alter field vcs on remoterepository
Migrations for 'projects':
0024_auto_20180315_1604.py:
- Alter field canonical on domain
- Alter field count on domain
- Alter field allow_promos on project
- Alter field comment_moderation on project
- Alter field conf_py_file on project
- Alter field default_version on project
- Alter field documentation_type on project
- Alter field has_valid_webhook on project
- Alter field language on project
- Alter field privacy_level on project
- Alter field programming_language on project
- Alter field python_interpreter on project
- Alter field repo_type on project
- Alter field suffix on project
- Alter field theme on project
- Alter field version_privacy_level on project
Migrations for 'redirects':
0002_auto_20180315_1604.py:
- Alter field redirect_type on redirect
- Alter field to_url on redirect

@julienmalard oops, it looks like you got all of the field changes in a migration set! Make sure you run the migrate step of installation before generating and new migrations:
http://docs.readthedocs.io/en/latest/install.html

Hello,
I reverted the commit, ran python manage.py migrate, followed by python manage.py makemigrations. Is it ok now?
Thanks!
-Julien

દ્વારા: Anthony<EMAIL_ADDRESS>મોકલ્યું: 15 માર્ચ 2018 19:54:33
પ્રતિ: rtfd/readthedocs.org
Cc: Julien Malard; Mention
વિષય: Re: [rtfd/readthedocs.org] New languages (#3759)
@julienmalardhttps://github.com/julienmalard oops, it looks like you got all of the field changes in a migration set! Make sure you run the migrate step of installation before generating and new migrations:
http://docs.readthedocs.io/en/latest/install.html
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/rtfd/readthedocs.org/pull/3759#issuecomment-373559349, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKMb60eIkVF98Fed3fGIpsvdvPYoGeZiks5tev85gaJpZM4SkgBp.

Hrm, that doesn't seem to have helped. There are still a large number of spurious migrations included.

That is very odd... is there any chance you could clone my fork and then run the migrations on your system, if you think that could work? If there is anything I could do on my system instead, please do let me know.
-Julien

દ્વારા: Anthony<EMAIL_ADDRESS>મોકલ્યું: 20 માર્ચ 2018 19:44:21
પ્રતિ: rtfd/readthedocs.org
Cc: Julien Malard; Mention
વિષય: Re: [rtfd/readthedocs.org] New languages (#3759)
Hrm, that doesn't seem to have helped. There are still a large number of spurious migrations included.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/rtfd/readthedocs.org/pull/3759#issuecomment-374794889, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKMb6wgwrWBwGw6XrEZVwdj_i9GBUMgLks5tgZRVgaJpZM4SkgBp.

If you can, just create a migration for the projects application and remove any unwanted migrations from that file. There are likely skipped migrations currently around help_text updates.

Better? :)

Hello,
I just wanted to ask whether what I have done was correct/sufficient or if I would need to do something else for this pull request to be acceptable.
Thanks!
Julien

Looks like there is a problem with the migration. Can you rebase your branch to the current master and re-generate the migration? Also can you use a named migration? python manage.py makemigrations --name changed_my_model your_app_label

Sure, will do that now. (By the way, does the migration have to be done on Python 2, or should it not make a difference?)
Thanks for the help!

It shouldn't make any difference

I'm assign this to myself to merge after the migration that is taking place this weekend.

NOTE: tests are not passing, we need to rebase with master and probably update the migration.

Unassigning for now, cleaning this PR up is going to take a bit of work and not something core team can focus on ATM.

Fixed up this migration, and it should be ready to merge. 👍

Many thanks!


Apple M1 support

I'm creating this issue to track support for Apple M1 silicon machines.
Currently, there are two outstanding issues which need to be solved:

winit = "0.26"
This will come with the glium "0.29" release which should be released next week.
libusb1-sys needs to be vendored (features = ["vendored"])
The only "real" package manager for macOS is brew, but it's not fully supported on Apple aarch64 yet. That means there is no "easy" way to get libusbinstalled for libusb1-sys to link against.

The first item is obviously just needed to get the mirror.rs example working, so it's not vital to the library itself. Would you be fine with making libusb1-sys vendored? Maybe we could just enable it automatically when the "vendor" feature is enabled for this crate (uvc).

winit is the offending component here. I mentioned glium = "0.29" as our target to watch out for, because that will pull in glutin = "0.26" which in turn depends on winit = "0.26" ;)
Speaking of which - glium = "0.29" has been tagged and pushed to crates.io - I'll prepare a PR.

Merged in https://github.com/mulimoen/libuvc-rs/commit/41392b770c7a205c722a4a1a131ed30544909633.


Translations stopped working

Greetings, it appears the translation feature ceased to work.

For a while, what appeared to be a server side issue affected translations. It seems this has now been resolved. You can also track translation issues in #435

Translations are broken again, either that or the API is rate limited


[WFLY-17890] forces jacorb's migrate() op to use elytron security

Issue: https://issues.redhat.com/browse/WFLY-17890

@emmartins @tadamski I think a change right after this line could be good:
https://github.com/wildfly/wildfly/blob/main/legacy/jacorb/src/main/java/org/jboss/as/jacorb/MigrateOperation.java#L156
Check 'jacorbModel' (not the transformed 'openjdkModel')  for 'identity', 'IDENTITY', 'client' or 'CLIENT' and if present add something to 'warnings'.  IIOPSubsystemAdd rejects client and identity at runtime so presumably there's a behavior difference between those and 'elytron' that justifies a warning.

@bstansberry From what I understood "client" and "identity" on jacorb were options using legacy security, which originally we would migrate to "identity" on iiop, but then we introduced Elytron and "elytron" option was added to iiop... And now we have no other option than to use "elytron" since we have no legacy security alternatives.
Anyway I will leave up to @tadamski to decide with you how the op should behave, maybe it would even make sense for him to takeover on the PR.

Agreed; this is Tomek's decision. If a config used 'client' or 'identity' and us changing it to 'elytron' isn't going to affect the server behavior, then for sure we should not include a warning message. If there is some difference the user might want to be aware of, the warning message is the way to provide that info without just failing the migration.

As there are differences in behavior between client, identity and elytron interceptors my vote would be to add such warning. I have opened a PR with it in https://github.com/emmartins/wildfly/pull/1. ElytronSASClientInterceptor has good javadoc so I decided to refer to it in the warning message.

Thanks @emmartins and @tadamski. I'm going to merge this one as-is and will push up a separate PR for the https://github.com/tadamski/wildfly/tree/WFLY-17890 commit.


Using TryFrom/TryInto generics with anyhow::Error

hi!
I'm trying to write a function that uses TryFrom generics:
pub fn extract_pkg<T: TryFrom<NewPkg>>(bytes: &[u8]) -> Result<Vec<T>>
    where <T as TryFrom<NewPkg>>::Error: std::error::Error + Sync + Send + 'static
{

The TryFrom is using anyhow::Error:
impl TryFrom<NewPkg> for DebianSourcePkg {
    type Error = anyhow::Error;

    fn try_from(new: NewPkg) -> Result<Self> {
        Ok(DebianSourcePkg {
            base: new.base.ok_or_else(|| format_err!("Missing pkg base field"))?,
            binary: new.binary.ok_or_else(|| format_err!("Missing binary field"))?,
            version: new.version.ok_or_else(|| format_err!("Missing version field"))?,
            directory: new.directory.ok_or_else(|| format_err!("Missing directory field"))?,
            architecture: new.architecture.ok_or_else(|| format_err!("Missing architecture field"))?,
            uploaders: new.uploaders,
        })
    }
}

The error looks like this:
error[E0277]: the trait bound `anyhow::Error: StdError` is not satisfied
   --> tools/src/schedule/debian.rs:144:5
    |
144 |     extract_pkg(bytes)
    |     ^^^^^^^^^^^ the trait `StdError` is not implemented for `anyhow::Error`
...
149 | pub fn extract_pkg<T: TryFrom<NewPkg>>(bytes: &[u8]) -> Result<Vec<T>>
    |        ----------- required by a bound in this
150 |     where <T as TryFrom<NewPkg>>::Error: std::error::Error + Sync + Send + 'static
    |                                          ----------------- required by this bound in `extract_pkg`

For more information about this error, try `rustc --explain E0277`.

Possibly related to #153.

The appropriate bound if you need to construct an anyhow::Error from the TryFrom impl's error type is <T as TryFrom<_>>::Error: Into<anyhow::Error>. That would work with any std::error::Error + Sync + Send + 'static error type as well as anyhow::Error.


registrarForPlugin  return nil

I create a PipeLine Plugin, registered at AppDelegate of Runner.xcproject. It works fine.
In Main.story board. I  change the Initia ViewController to a UINavigationController, whose rootViewController is FlutterViewController.
Then the plugin register failed.
In Appelegate.m

NSObject<FlutterPluginRegistrar> *strar = [self registrarForPlugin:@"PipePlugin"];

//strar is nil

I upload the demo :  https://github.com/taojigu/learn_plujgin
[✓] Flutter (Channel beta, v1.1.8, on Mac OS X 10.14.3 18D42, locale zh-Hans-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
[✓] Android Studio (version 3.2)
[✓] VS Code (version 1.32.3)
[✓] Connected device (2 available)

Hi, could you please confirm if this problem still exists with the latest versions on stable or master?
Also does this happen on Android and iOS?
Thanks a lot

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


Improved device checking for import

Added a better mechanism to check if the current OS device is supported

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


Sudoshred Bio

Sudoshred is a Linux Sys Admin and fully believes that Linux and open source can change the world.
He streams Linux native games and distro releases on Twitch.  He started his Twitch streaming in an effort to dispell the myth that Linux has no games and to educate people on Linux.
His goal is to help people move to Linux and grow Linux.

Also need an avatar at least 512x512 for your profile

I don't really have an avatar, so how does this do:
http://ih1.redbubble.net/image.198448355.4265/flat,800x800,075,t.jpg

Social Media
https://www.twitch.tv/sudoshred
https://twitter.com/sudoshred


docs: ✏ use false investor uniqueness for create asset example

JIRA Link
N/A
Changelog / Description
Default create asset to not require investor uniqueness. If enabled claims have to be made before settlements succeed, and it can be difficult to debug why they fail.
Checklist -

[ ] New Feature ?
[ ] Updated swagger annotation (if API structure is changed) ?
[ ] Unit Test (if possible) ?
[ ] Updated the Readme.md (if required) ?

:tada: This PR is included in version 2.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Mainly SQL Updates

Add comment to SQL Filtering (Section 8.1)
Add NAME to ALTER TYPE (Section 8.2)
Refactor CREATE EDGE docu (Section 8.2)
Add BY KEY and BY VALUE to CEATE INDEX (Section 8.2)
Remove Java11 compatibility (Section 11.9)
Add note on duplicate JSON keys to FAQ (Section 11.10)

@lvca is this OK?

Perfect, thanks!!!


(New Feature) Added basic integration with Travis CI service

Added basic config for Travis CI service, including 4 basic
platforms: Ruby 1.8.7, Ruby 1.9.2, Ruby 1.9.3 and Ruby
Enterprise Edition. Also moved rSpec options from deprcated
spec.opts file to .rspec, so rSpec output in Travis CI
will be mush more prettier.

We have our own CI system at https://jenkins.puppetlabs.com/ - thanks for the submission, but we would generally prefer this runs on systems that we own and manage.


Does it work with https://github.com/liquidpele/nvd3

Hello active users of angular-nvd3,
does someone already tested this directive against recent development of nvd3 in what would become the new home for nvd3:  https://github.com/liquidpele/nvd3
Thanks.

Update: I tried again adding nvd3-community to my project, and so far I see no issues with angular-nvd3.
The only problem is that angular-nvd3 will still install the original nvd3 via bower, as referenced by @AgDude . Unless this repo is updated, and given that the library itself is just a single, relatively simple/short js file, my best bet is to just fork this repo and include the file directly into our angular projects.

I tried to register it with bower as "nvd3-community"... you can try that and let me know if there are any issues...

Thanks to all for your activity. It's required some fixes to use this directive with nvd3-community. I will make new release in the near future.

Thanks for being on top of it krispo.  I'm brand new to NVD3 so am still experimenting with it in my app. I was having similar issues to others such as leaflet interaction and a wiredep issue when manually installing NVD3-community.  To tide me over in the experimental phase, I installed klangfarbe's fork with bower and am rolling.

@krispo, FYI, Novus has re-kindled work on the original nvd3 repo, merging in all changes from nvd3-community.

As 1.0.0-BETA is targetting novus nvd3-1.7.1 that include community updates,
I suppose we can close this one.


NDRS-942: Delegation rate for validators in accounts.toml

Ref: https://casperlabs.atlassian.net/browse/NDRS-942
This PR changes the structure of accounts.toml to separate bonded_amount so delegation_rate could be a part of it. If [accounts.validator] field in TOML is not specified for given array element, then such account would be normal user without stake. Otherwise it will be considered an account with a stake.
Thanks to this separation of stake related attributes both bonded_amount and delegation_rate can be properly validated.

bors r+


242 3.25 accept rejected without alternatives

This PR contains

the last requirement for #242  - the possibility to accept rejected applications, when there are enough free places and no alternative applications
some small refactorings/tests that were made more precise because of the introduced changes

Coverage remained the same at 100.0% when pulling 18d918b5edb8fd220819c2a8981535c8281c9873 on 242_3.25_AcceptRejectedWithoutAlternatives into 64aca2ddb218616c654ba7088a57c18143223569 on dev.


unused -v still says 0.1.0

Using unused 0.2.0:
$ unused --version
unused-rs 0.1.0

@lrworth whoops; I'd changed the wrong file for picking up versioning. I've just released a patch version to address this: 850d586dd28e473ea847741a66f28dba860e2090


PR: Fix QProcess error messages in Completions plugin

Description of Changes
This PR fixes two error message issued by Qt which are related QThread/QProcess. The error messages appears on the console when Spyder is closed.
The QProcess error has been observed both with PyQt (Qt 5.15.2/PyQt 5.15.4) and PySide (Qt 5.15.0/PySide 5.1.50) on the current 5.x branch and with v5.1.5. The QThread error has been observed with PySide (PyQt not checked).
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct:
rear1019

Hey @rear1019, I think leaving only the QProcess here is fine, so could you revert your changes to fix the QThread issues? That way @impact27 implement his ideas in another PR and we could merge this one.

Hey @rear1019, I think leaving only the QProcess here is fine, so could you revert your changes to fix the QThread issues? That way @impact27 implement his ideas in another PR and we could merge this one.

I also think that we should leave out the QThread changes. At least for now, mainly due to #16324 which makes massive changes to the IPython plugin.

Hey @rear1019, I think leaving only the QProcess here is fine, so could you revert your changes to fix the QThread issues? That way @impact27 implement his ideas in another PR and we could merge this one.

I added this to https://github.com/spyder-ide/spyder/pull/14025 because it was already touching the shutdown code


no png

There is no module named png in the flowlib of lib.

Please download pypng package using apt-get if you are a linux user.
Thank you


User and Authentication 1.1

Tasks

[x] Refactor Nitro_Repo to allow for different database types(SQLIte and Postgres) and to allow different user storage methods
[x] Session Token Storages methods such as Redis and In-Memory
[x] Complete Login Tokens #100

Token Types
Session Tokens nrs_{} for sessions.

Created upon first connection to website. Stored in Cookies
Stored in Memory or Redis
Will reference the Authentication Token
24 life time.
Authentication Token nrp_{}
Stored in Database
Life Span is specified upon creation. So 24 if for session token.  or even like 90 days if you are going to use this for like your deploying from Maven
Can be sent with API requests instead of session tokens. For API stuff
Will be an alternative for deploying using usernames and passwords

Completed in the 1.1 Dev Branch


nixos-rebuild: don't run sudo in sudo

Someone just didn't read the code while making changes? run0 sets SUDO_USER which is broken because of this check. I think the code in buildHostCmd is redundant after this change, needs checking.
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Closing as a duplicate of https://github.com/NixOS/nixpkgs/pull/331741.


BCLib 0.4.2 crash on startup

MC 1.17.1
Fabric loader 0.12.4; API 0.41.3
BCLib 0.4.2
Latest.log: https://gist.github.com/Footage2-Amply-Pounce/04d7be1f83b59d379e135c1f1ec6f69c
Crash report: https://gist.github.com/Footage2-Amply-Pounce/f862d96140fcce653406344e8eb22148

Thanks for the report. We'll investigate...

Possible fix implemented in b7c83923ea70de3e59fdd8364df4bfb03a32cbdc

Fixed in 0.4.3

You should really really put that in the change log that 0.4.3 requires modloader 0.12 and 0.4.2 requires modloader 0.11
Or at least add a required dependacy

It is not supposed to be like this, and we will release an update shortly. I'll update the change-logs in the meantime.

This is also an old issue tracker, please use the new one

that error log tells them nothing. you need to post the entire stack trace @Miskler

I'm too lazy to look for a log. excuse me)

that error log tells them nothing. you need to post the entire stack trace @Miskler

I'm too lazy to look for a log. excuse me)

Good luck getting any help with that attitude

I'm not interested in your mod


Include namespace label in prometheus metrics

This is a feature request
Traefik can be configured to expose a metrics endpoint in the Prometheus format. An end-user is free to choose between:

an entry point format:
traefik_entrypoint_request_duration_seconds_bucket{code="404",entrypoint="traefik",method="GET",protocol="http",le="0.1"} 14
and a service format:
traefik_service_request_duration_seconds_bucket{code="200",method="GET",protocol="http",service="whoami-whoami-9ab4060701404e59ffcd@kubernetescrd",le="0.1"} 2

Typically, those metrics are produced by a Traefik controller according to how IngressRoute is defined (how entryPoints and services properties are set). If we have, for example, the following IngressRoute definition:
kind: IngressRoute
apiVersion: traefik.containo.us/v1alpha1
metadata:
  name: myservice
  namespace: mynamespace
spec:
  entryPoints:
    - web
  routes:
  - match: PathPrefix(`/myservice`)
    kind: Rule
    services:
    - name: myservice
      port: 80

Traefik configured with --metrics.prometheus=true --metrics.prometheus.buckets=0.100000, 0.300000, 1.200000, 5.000000 --metrics.prometheus.addEntryPointsLabels=true --metrics.prometheus.addServicesLabels=true will produce the following metrics:
traefik_entrypoint_request_duration_seconds_bucket{code="200",entrypoint="web",method="GET",protocol="http",le="0.1"} 2
traefik_entrypoint_request_duration_seconds_bucket{code="200",entrypoint="web",method="GET",protocol="http",le="0.3"} 2
traefik_entrypoint_request_duration_seconds_bucket{code="200",entrypoint="web",method="GET",protocol="http",le="1.2"} 2
traefik_entrypoint_request_duration_seconds_bucket{code="200",entrypoint="web",method="GET",protocol="http",le="5"} 2
traefik_entrypoint_request_duration_seconds_bucket{code="200",entrypoint="web",method="GET",protocol="http",le="+Inf"} 2
traefik_entrypoint_request_duration_seconds_sum{code="200",entrypoint="web",method="GET",protocol="http"} 0.002430748
traefik_entrypoint_request_duration_seconds_count{code="200",entrypoint="web",method="GET",protocol="http"} 2

traefik_service_request_duration_seconds_bucket{code="200",method="GET",protocol="http",service="mynamespace-myservice-9ab4060701404e59ffcd@kubernetescrd",le="0.1"} 2
traefik_service_request_duration_seconds_bucket{code="200",method="GET",protocol="http",service="mynamespace-myservice-9ab4060701404e59ffcd@kubernetescrd",le="0.3"} 2
traefik_service_request_duration_seconds_bucket{code="200",method="GET",protocol="http",service="mynamespace-myservice-9ab4060701404e59ffcd@kubernetescrd",le="1.2"} 2
traefik_service_request_duration_seconds_bucket{code="200",method="GET",protocol="http",service="mynamespace-myservice-9ab4060701404e59ffcd@kubernetescrd",le="5"} 2
traefik_service_request_duration_seconds_bucket{code="200",method="GET",protocol="http",service="mynamespace-myservice-9ab4060701404e59ffcd@kubernetescrd",le="+Inf"} 2

While having all relevant context data, the output still lacks a clear notion of what exact namespace the related resources (IngressRoute, Service, TraefikService) are defined in.
Why is namespace so important?
So far, I've come up with several use cases:

Grafana dashboards usually contain namespace selectors. Users are willing to distinguish their services between different namespaces nicely.
Integration: to integrate with HPA through k8s-prometheus-adapter it requires at least to know a namespace when the HPA scaling is based on an object type.
Further automation: if there's some consumer service that makes decisions based on observed metrics, it somehow needs to know where to find the original resource to request its specs or state.

But the service latency metric already contains the namespace entry in its service label.
That's true. But the way it's implemented (namespace + "-" + name) doesn't cover all possible scenarios.
Imagine if you have the my-namespace namespace and the the-resource resource. In this case, even a powerful Prometheus metric_relabel_configs won't help you much. With the relabeling rule like this:
    metric_relabel_configs:
      - source_labels:
          - service
        regex: ^([^-]+)-.*$
        replacement: ${1}
        target_label: namespace
      - source_labels:
          - service
        regex: "^[^-]+-(.*)-[^<EMAIL_ADDRESS>        replacement: ${1}
        target_label: entrypoint

You'll get a processed metric:
traefik_service_request_duration_seconds_bucket{code="2000",entrypoint="namespace-the-resource",instance="<IP_ADDRESS>:8082",job="traefik",le="0.3",method="GET",namespace="my",protocol="http",service="my-namespace-the-resource-df34efd3b36ac5e9a216@kubernetescrd"}

Which is not exactly the original intention.
All of this just comes down to a proper convention
Having a standardized number of dashes in your namespaces would totally resolve the issue. Still, nothing beats a distinct self-explanatory label name namespace in a metric description.
traefik_entrypoint_request_duration_seconds_count{code="200",entrypoint="web",namespace="mynamespace",method="GET",protocol="http"} 2
traefik_service_request_duration_seconds_bucket{code="200",method="GET",protocol="http",namespace="mynamespace",service="mynamespace-myservice-9ab4060701404e59ffcd@kubernetescrd",le="0.1"} 2

It would be better to have pod-level metrics
It totally would! Instead of measuring latency on a Traefik's load balancer and averaging it, how about measuring it independently on every individual pod and producing the metrics that would include a pod label that later can be consumed by k8s-prometheus-adapter and handed over to the HPA which is quite successful at balancing the scaling parameters by evaluating the pod-level metrics.

Agree with @iliazlobin
For me the use case is heavily on taking advantage of namespaces implementation to be sued in HPA metrics. Specifically if i want to scale based on request count.  It's easier to write a "clean" service name (which is this namespace introduction) instead of the generated ingressroute name.


Document compound key features affect on changefeeds

Abbey Russell (amruss) commented:
Hash sharded indexes, regional by row tables, etc.

the compound key comes through the changefeed

changes to key cause delete/insert

causes backfill of table

Jira Issue: DOC-1215

Here’s a Knowledge Base (internal) and a Slack conversation (internal) with more details.


lookup server=localhost: getaddrinfow: No such host is known

lookup server=localhost: getaddrinfow: No such host is known
but,i can connect it by java.

my connect string is:server=localhost;user id=sa;password=123456

is this working for you:
server=<IP_ADDRESS>;user id=sa;password=123456
?

@denisenkom i have try it,and i use my ip.but it does not work.

hello @haochun please verify that the server accepts tcp connections.
see: https://msdn.microsoft.com/en-us/library/bb909712(v=vs.90).aspx

@dimdin thank you ,i will try it.but,i can connect it by JDBC,i does not know why.

Try adding encrypt=disable


opt(torii): batch query execution in sync_range

Stack:

#2381
#2378
#2374 ⬅
#2373

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

✓ Commit merged in pull request #2381


Redesign share sensor page to be scrollable and visually pleasing

Description:
Redesign share sensor page to be scrollable and visually pleasing

Duplicate


Mark 19-ea as experimental.

I'm hoping that this will address a sporadic(?)
failure:
Could not open settings generic class cache for settings file '/home/runner/work/jspecify/jspecify/settings.gradle' (/home/runner/.gradle/caches/7.5.1/scripts/8fxsx7a5az16qabak9o2v29h1).
> BUG! exception in phase 'semantic analysis' in source unit '_BuildScript_' Unsupported class file major version 63

Gradle doesn't clearly support Java 19 yet:
https://docs.gradle.org/current/userguide/compatibility.html
I'm basing this config on similar configuration in Error
Prone.

Maybe this will become obsolete after https://github.com/jspecify/jspecify/issues/271#issuecomment-1223320796, but for now, I'm hoping that it eliminates any flakiness in the CI.

Thank you! I should have pulled up https://github.com/google/error-prone/commit/50e2e7b54215f8fc0e2b6d82e3667a12f0f6e530 when I was writing this PR, but obviously I did not.


Updated pyOpenSSL dependency to 16.2.0

This fixes an AttributeError: 'module' object has no attribute 'SSL_ST_INIT' error on startup.

Wew, it's been a while since that dependency was updated apparently. Thanks!