Slower performance in threaded implementation?

I've noticed slower performances doing big matrix multiplies in multiple threads, even when each thread uses a different GPU. I made a simplified snippet to illustrate the issue:
require 'nn'
require 'cunn'
require 'cutorch'
local threads = require 'threads'

cutorch.setDevice(8)

local pool = threads.Threads(7, function()
    require 'nn'
    require 'cunn'
    require 'cutorch'
    print ('done requires')
end)
pool:synchronize()
pool:specific(true)

for i=1,7 do
    pool:addjob(i, function()
        cutorch.setDevice(i)
        net = nn.Linear(4000,4000):cuda()
        closure = function()
            local timer = torch.Timer()
            for j=1,1000 do
                inp = torch.CudaTensor(4000)
                out = net:forward(inp)
            end
            print ('Execution time in thread '..tostring(__threadid), timer:time().real)
            return out
        end
    end)
end
pool:synchronize()

for i=1,7 do
    pool:addjob(i, function()
        return closure()
    end
    -- , function(out)
    --     print ('finished')
    -- end
    )
end
pool:synchronize()

local mynet = nn.Linear(4000, 4000):cuda()
local timer = torch.Timer()
for j=1,1000 do
    inp = torch.CudaTensor(4000)
    out = mynet:forward(inp)
end
print ('Single thread execution time:', timer:time().real)

This script yields the output
Execution time in thread 7	0.64696097373962
Execution time in thread 3	0.65926384925842
Execution time in thread 4	0.68375587463379
Execution time in thread 6	0.69135785102844
Execution time in thread 2	0.69343709945679
Execution time in thread 1	0.70243215560913
Execution time in thread 5	0.70486903190613
Single thread execution time:	0.21251583099365

where the execution in each of the threads is about 3.5x slower. Interestingly, if I only run the job using 3 of the threads, this changes:
Execution time in thread 3	0.32128405570984
Execution time in thread 1	0.47071886062622
Execution time in thread 2	0.48118710517883
Single thread execution time:	0.21262502670288

I'm not sure what the bottleneck resource could be here - if somebody could point that out, it'd be a huge help. Thanks in advance.

There's a number of issues here- (1) the first forward pass will always have an overhead due to needing to allocate memory, and this shouldn't be included in the timing (2) cutorch synchronization seems to incur some overhead, (3) garbage collection slows this down, so to have a realistic comparison, we should make sure that we aren't reallocating memory inside the script
For those interested, here's a new version of the script
require 'nn'
require 'cunn'
require 'cutorch'
local threads = require 'threads'

cutorch.setDevice(8)
local timerB = torch.Timer()
cutorch.synchronizeAll() -- Why does this take 10seconds?
print ('Time for first synchronize:', timerB:time().real)
local pool = threads.Threads(7, function()
        require 'nn'
        require 'cunn'
        require 'cutorch'
    end,
    function()
        cutorch.setDevice(__threadid)
    end,
    function()
        net = nn.Linear(4000,4000):cuda()
        inp = torch.CudaTensor(4000)
        closure = function()
            local timer = torch.Timer()
            for j=1,1000 do
                inp:fill(torch.random() % 1000 / 1000)
                net:forward(inp)
            end
            print ('Execution time in thread '..tostring(__threadid), timer:time().real)
            return out
        end
        closure() -- allocate inside network memory
    end
)
pool:synchronize()
pool:specific(true)
print ('-----------Done initializing threads-------------')

local timerA = torch.Timer()
for i=1,7 do
    pool:addjob(i, function()
        return closure()
    end)
end
pool:synchronize()
print ('After pool sync', timerA:time().real)
cutorch.synchronizeAll() -- This still takes .15 seconds which is troubling
print ('After torch sync', timerA:time().real)
cutorch.synchronizeAll()
print ('Sync again', timerA:time().real)

local mynet = nn.Linear(4000, 4000):cuda()
local inp = torch.CudaTensor(4000)
local timer = torch.Timer()
for j=1,1000 do
    inp:fill(torch.random() % 1000 / 1000)
    mynet:forward(inp)
end
print ('Single thread execution time:', timer:time().real)

with some representative output
Time for first synchronize:	12.22967505455
Execution time in thread 2	1.2082839012146
Execution time in thread 4	1.2250549793243
Execution time in thread 1	1.2145731449127
Execution time in thread 6	1.2097449302673
Execution time in thread 3	1.2016780376434
Execution time in thread 7	1.1553859710693
Execution time in thread 5	1.0837528705597
-----------Done initializing threads-------------
Execution time in thread 2	0.3775360584259
Execution time in thread 4	0.39866209030151
Execution time in thread 1	0.40224695205688
Execution time in thread 6	0.40711092948914
Execution time in thread 3	0.41298985481262
Execution time in thread 7	0.41581201553345
Execution time in thread 5	0.42192792892456
After pool sync	0.42342686653137
After torch sync	0.56986784934998
Sync again	0.56992602348328
Single thread execution time:	0.40415096282959

where now the execution time in each of the threads matches the execution time in a single thread.


Updated to master branch & Improved compiler

Not much to be said about it. You don't have to merge the CMake file, but it should point to master at this point.
Compiler is still in a rough shape and needs some work, but should compile most of the inputs thrown at it.
Also improved compile command to auto-create output folder for compiled binary files

Thanks! I updated the cmake to point to an absolute ref (better for reproducing builds in the future).


Example of adjusting environment variable via CLI to disable server

Is your feature request related to a problem? Please describe.
I want to run tests in parallel however due to the port conflict I cannot.
Error: listen EADDRINUSE: address already in use <IP_ADDRESS>:2121
Describe the solution you'd like
An example of how to do this.
Describe alternatives you've considered
I have tried the following without success:
cypress run --env "cypress-plugin-snapshots.serverEnabled=false"
cypress run --env "cypress-plugin-snapshots.serverEnabled"="false"
cypress run --env "env.cypress-plugin-snapshots.serverEnabled"="false"
cypress run --env cypress-plugin-snapshots.serverEnabled=false
cypress run --env "cypress-plugin-snapshots".serverEnabled=false
cypress run --env serverEnabled=false
Additional Context

Windows 10
WebStorm
Cypress 5.5

I ran into the exact same issue, but it worked when setting serverEnabled to false in cypress.json:
"env": {
    "cypress-plugin-snapshots": {
      "serverEnabled": false
    }
}


put-file -i should be able to add files to a directory

I want to create a dir with a set of images in it, but pachctl put-file images master set1/ -i ~/pachyderm/doc/examples/opencv/images.txt currently creates a file set1 that is all of the images in images.txt (urls in this case) appended together.
With -i, we should probably have any file name that specified be a prefix to the files that are being added.

Side thought -- as nice as it is for demos, do we want to get rid of -i and simplify put-file a bit? I've almost never seen any users use it. Just a thought.


Support for Next13 appDir

Love next-translate! Are you planning to implement support for Next13 appDir?

YES! and soon 😊 Today or tomorrow I'm going to do the PR about this part, it's almost ready, I only want to increase the number of tests

@aralroca, that is AMAZING! Thank you very much!!! ❤ 🚀 🚀

@mr0b0t0 You can already try it in [2.0.0-canary.3](https://github.com/aralroca/next-translate/releases/tag/2.0.0-canary.3. Please, if you can provide me with any feedback before the 2.0.0 release will be very welcome 😊
I wrote the documentation about how to use it inside the canary README.md, or also in the PR description.

Thank you! 😍 Going to try it ASAP

Working great so far! 💯 🚀  @aralroca

Working great so far! 💯 🚀 @aralroca

Nice to know it! Thanks! 😊👍 2.0.0 will be released soon


FancyZones Windows Terminal Taskbar Button and Window on Different Monitors

Environment
Windows build number: Microsoft Windows [Version 10.0.19042.330]
PowerToys version: v0.19.1
PowerToy module for which you are reporting the bug (if applicable): FancyZones

Steps to reproduce

Requires 2 monitors
1a. May require using a non-admin account
Show taskbar buttons on: Taskbar where window is open
Open Windows Terminal and use FancyZones to "snap" to zone on non-default monitor
Reboot computer (or possibly just sign out and back in)
Open Windows Terminal

Expected behavior
Windows Terminal taskbar button and window will both appear on the non-default monitor. Window will be "snapped" to same zone as before.
Actual behavior
Windows Terminal taskber button appears on the default monitor while the window will be "snapped" to the correct zone on the non-default monitor.

@jkane-csshealth
this is currently by design, it will be changed in 0.20 https://github.com/microsoft/PowerToys/issues/4357
Closing as duplicate of https://github.com/microsoft/PowerToys/issues/4357
Thanks,

@enricogior I'm confused: what I'm describing is clearly a bug/undesirable behavior while what #4357 describes sounds more like a  feature request. It doesn't at all mention a taskbar button being on a different monitor from where the window is opened. The issue I'm reporting also doesn't impact all programs; Outlook, Slack, and Microsoft Teams all open up in their previous snapped-to zones on a non-default monitor with their taskbar buttons on the correct monitor,

@jkane-csshealth
I link the wrong issue, anyway this should be fixed in 0.20.

 I tested your scenario by:

Quitting Windows Terminal
Quitting PowerToys
Restarting PowerToys from the start menu
Reopening WIndows Terminal from the Start Menu
It was a little weird though, while Windows Terminal immediately opened in the original fancy zone on the non-default display, the taskbar button initially slid in on the default display before disappearing and reappearing on the correct, non-default one.

@jkane-csshealth
this is supposed to be fixed in https://github.com/microsoft/PowerToys/releases/tag/v0.21.1
Is it working for you?

@jkane-csshealth
this is supposed to be fixed in https://github.com/microsoft/PowerToys/releases/tag/v0.21.1
Is it working for you?

Yeah, Windows Terminal now opens on the main monitor regardless of the last monitor it was on. The Windows Terminal Taskbar button is always on the same monitor as the window now.


Errors being trapped by merge operations like switchMaps

RxJS version: 5.5.5
Code to reproduce:
Reproduction JSBin Here

of('test').switchMap(() => {
  return timer(0).do(() => {
    throw new Error('hahah');
  })
})
.subscribe({
  error(err) {
    throw new Error('final error');
  }
});

Expected behavior:
Error "final error" should be reported in console
Actual behavior:
Error is squeltched.
Additional information:
This only happens if the observable in the merge operation is synchronous. The error thrown by the subscription unwinds the stack back to the  switchMap's subscriber (via parent.notifyError from InnerSubscriber), and it's swallowed because the switchMap subscriber isStopped.

cc @IgorMinar

maybe related: https://github.com/ReactiveX/rxjs/pull/3161

@IgorMinar can you give #3161 a try? (the example above doesn't seem to suffer from the described error)
I've attached a build from that PR you could copy/pasta from into your node_modules
dist.zip

Apparently my minimal repro doesn't repro like I thought. :\

@jayphelps that build didn't work because it's not complete (all add/operator/* files are missing), but I was able to diff it and extract this patch that I applied to my current build:
--- node_modules/rxjs.patched/src/Subscriber.ts 2017-12-06 09:58:33.000000000 -0800
+++ node_modules/rxjs/src/Subscriber.ts 2017-12-05 01:29:16.000000000 -0800
@@ -68,6 +68,7 @@
         }
         if (typeof destinationOrNext === 'object') {
           if (destinationOrNext instanceof Subscriber) {
+            this.syncErrorThrowable = destinationOrNext.syncErrorThrowable;
             this.destination = (<Subscriber<any>> destinationOrNext);
             (<any> this.destination).add(this);
           } else {

--- node_modules/rxjs.patched/src/Observable.ts 2017-12-06 09:58:33.000000000 -0800
+++ node_modules/rxjs/src/Observable.ts 2017-12-05 01:29:16.000000000 -0800
@@ -199,7 +199,7 @@
     if (operator) {
       operator.call(sink, this.source);
     } else {
-      sink.add(this.source ? this._subscribe(sink) : this._trySubscribe(sink));
+      sink.add(this.source || !sink.syncErrorThrowable ? this._subscribe(sink) : this._trySubscribe(sink));
     }

     if (sink.syncErrorThrowable) {

With this patch applied to the JS files by hand, I was able to confirm that the errors now get propagated correctly, so if I extracted the change correctly, then what you did resolves the problem.


Should master node different with worker node?

Organization Name: EnjoyMusic Technology

Short summary about the issue/question:
Should master node different with worker node? In other words, master node can only use CPU but can't with GPUs?
2020-05-26 06:28:21,485 [INFO] - openpai-generator.py:120 : AMD-Device-Plugin is not ready.
2020-05-26 06:28:21,485 [INFO] - openpai-generator.py:121 : Total: 1
2020-05-26 06:28:21,485 [INFO] - openpai-generator.py:122 : Ready: 0
2020-05-26 06:28:21,485 [INFO] - openpai-generator.py:168 : AMD-Device-Plugin is not ready yet. Please wait for a moment!
2020-05-26 06:28:31,588 [ERROR] - openpai-generator.py:196 : No worker node is detected. username and password to log in.

Brief what process you are following:
The handbook's installation guide

How to reproduce it:
Provide only one machine with GPU

OpenPAI Environment:

OpenPAI version: v1.0.0
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release): ubuntu 18.04
Kernel (e.g. uname -a):
Hardware (e.g. core number, memory size, storage size, GPU type etc.):
Others:

Anything else we need to know:

@k1ngcyk - would you mind share your Org's name? we are clustering issues with org names to better manage issues.

According to the installation guide, you should have 3 separate machines: dev box, master and worker. The worker should have GPU. For dev box and master machine, you can use CPU-only nodes.

@k1ngcyk - would you mind share your Org's name? we are clustering issues with org names to better manage issues.

ok, edited

According to the installation guide, you should have at least 3 separate machines: dev box, master and worker. The worker should have GPU. For dev box and master machine, you can use CPU-only nodes.

ok, though I use this one machine for test. What I really care is that if I have GPU on the master machine, would these gpu be available in the PAI?

You will not be able to use the GPU within PAI system. Because all computing jobs submitted in PAI will be scheduled to workers (not master.)

You will not be able to use the GPU within PAI system. Because all computing jobs submitted in PAI will be scheduled to workers (not master.)

thanks for declaration. But I found same master and worker in your add node example in handbook: here which contains lines like
...
  children:
    kube-master:
      hosts:
        origin1:
    kube-node:
      hosts:
        origin1:
        origin2:
...

so the master machine can also be a worker machine?

It is k8s master & nodes, not PAI master & worker. PAI should have separate master and worker. They are different.

ok, thanks again for your reply.

I hereby close the issue. Feel free to reopen it if you have further questions.


useClientEffectQrl is not a function

Code Not Working On  Above Qwik version 0.25.0

Hey, in the "lottie.tsx" you have to change "useClientEffect" to "useVisibleTask". Thatll fix your problem

great
Thanks for the tip!
On Sat, 12 Aug 2023 at 01:46, dezzi @.***> wrote:

Hey, in the "lottie.tsx" you have to change "useClientEffect" to
"useVisibleTask". Thatll fix your problem
—
Reply to this email directly, view it on GitHub
https://github.com/qwikverse/qwik-lottie/issues/3#issuecomment-1675317278,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF733CMOGVDV24MPEJKZ7NTXU2HIJANCNFSM6AAAAAAZPURWD4
.
You are receiving this because you authored the thread.Message ID:
@.***>

@notdezzi  Hi, I have the same issue.
I don't have "lottie.tsx" file


Issue

Hello, Im new in R, Im very interesting in your script, but is does not work:
I m in ubuntu
I follow this https://www.r-bloggers.com/how-to-install-r-on-linux-ubuntu-16-04-xenial-xerus/ for install R.
I run in console:
toto@titi:~/Rencoder/k-means-auto-encoder$ Rscript clustering.r
[1] 10
[1] 20
[1] 50
[1] 100
[1] 200
[1] 500
[1] 1000
[1] 2000
null device
          1
null device
          1
Error in is.data.frame(data) : object 'x' not found
Calls: cube.root.fig ... <Anonymous> -> model.frame.default -> is.data.frame
Execution halted

Any ideas what am I doing wrong?

On Thu, May 11, 2017 at 5:37 AM, MELNYK<EMAIL_ADDRESS>wrote:

Error in is.data.frame(data) : object 'x' not found

That is very odd.
I see the same problem and am worried that I missed pushing a last change
or something.
I will figure this out.

It is also very odd that I don't have an older copy of that project.
On Thu, May 11, 2017 at 10:26 AM, Ted Dunning<EMAIL_ADDRESS>wrote:

On Thu, May 11, 2017 at 5:37 AM, MELNYK<EMAIL_ADDRESS>wrote:

Error in is.data.frame(data) : object 'x' not found

That is very odd.
I see the same problem and am worried that I missed pushing a last change
or something.
I will figure this out.

I pushed a fix. IT was my sloppiness as I suspected.
I rearranged code at the last minute without checking that it worked
outside of my edit session. Unfortunately, inside the session I had a
convenient variable with the same name and desired structure hanging around.
Fixed now and the picture is a bit more informative.
On Thu, May 11, 2017 at 10:27 AM, Ted Dunning<EMAIL_ADDRESS>wrote:

It is also very odd that I don't have an older copy of that project.
On Thu, May 11, 2017 at 10:26 AM, Ted Dunning<EMAIL_ADDRESS>wrote:

On Thu, May 11, 2017 at 5:37 AM, MELNYK<EMAIL_ADDRESS>wrote:

Error in is.data.frame(data) : object 'x' not found

That is very odd.
I see the same problem and am worried that I missed pushing a last change
or something.
I will figure this out.

@melnarte Can you confirm that the latest fix solves your problem?


Struggling with dynamic data

Hi, I am try to create a graph but having problems converting the data to a format compatible with Chart JS. I'd like a graph like the one below.

The Data returned from SQL via Ajax/Json looks like this.

This data will dynamically change so I can't manually create the datasets. The data should be dynamically generated from the database.
Any help would be appreciated.

Please include an example of your issue on a website such as JS Bin, JS Fiddle, or Codepen with your current code. (Template)

Closing since there is no test case


Doesn't seem to read "SteamLibrary" folders as valid

Trying to setup the games directory, the direct steam folder works fine but the SteamLibrary folders where games can also be installed doesnt seem to be recognised

I take it you have l4d2 installed on another drive? I completely overlooked this possibility, in all honesty. I'll take a look.

fixed in 4033f0981cc2896546be060018d65af725227037, i'll ship it in 0.2.1 soonish


alias for nvim :Z -> :cd

hey i was using zoxide lately for my terminal navigation and it seems awsom i have alias it to the cd  ( eval "$(zoxide init --cmd cd fish | source)" ) using this i use fish btw
i installed the zoxide.vim and it uses :Z as it command is there a way to do the same :Z -> :cd
it will be really awsome

The closest thing to a fish alias in vim is probably a command-line abbreviation (:h cabbrev)
You could try something like along those lines:
cabbrev <expr> cd getcmdtype() is# ':' && getcmdline() is# 'cd' ? 'Z' : 'cd'


[cdc] Supports taking a column of a table from MYsql and processing it as a table partition field for paimon table

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
<FLINK_HOME>/bin/flink run
-c org.apache.paimon.flink.action.FlinkActions
/path/to/paimon-flink-**-0.4-SNAPSHOT.jar
mysql-sync-table
--warehouse hdfs:///path/to/warehouse
--database test_db
--table test_table
--partition-keys pt
--primary-keys pt,uid
--mysql-conf hostname=<IP_ADDRESS>
--mysql-conf username=root
--mysql-conf password=123456
--mysql-conf database-name=source_db
--mysql-conf table-name='source_table_.*'
--catalog-conf metastore=hive
--catalog-conf uri=thrift://hive-metastore:9083
--table-conf bucket=4
--table-conf changelog-producer=input
--table-conf sink.parallelism=4
As shown above, only existing columns of mysql table can be selected as partitioning fields of paimon table. However, in production, time partitioning is generally adopted, and the time field is usually of datetime type, which needs to be processed into dt/hr partitioning field
Solution
No response
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

My idea is:

We can add a new argument for specifying computed column, for example:

--partition-keys dt,hr
--computed-column dt=date(ts)
--computed-column hr=hour(ts)

ts is exsiting column, we can specify that dt is day of ts and hr is hour of ts.
2. The newly computed column can be specified as partition key and will be auto created in Paimon table.
3. Introduce two class to handle computed column:
class ComputedColumn: store column's name, type, argument column names, and aColumnValueComputer to compute value from given argument.
interface ColumnValueComputer:  has a method Object computeValue(Object... input) to compute value. We can implement several concrete computers.

Pass ComputedColumns to EventParser. Every time extractRow is invoked, generate computed column values.

Solved by #1109.
If we need more processing functions, we can create other issues.


Swift Package Manager [WIP]

Testing notes:

cd /path/to/opencv
First build the xcframework python ./build-xcframework.py platforms/apple/xcframework-build/ or place your prebuilt opencv2.xcframework at platforms/apple/xcframework-build/opencv2.xcframework
swift build

Currently dealing with some interface issues with the existing Swift module from the objc module:
/Users/chrisbal/Documents/opencv/.build/x86_64-apple-macosx/debug/opencv2.framework/Modules/opencv2.swiftmodule/x86_64-apple-macos.swiftinterface:676:81: error: ambiguous type name 'Rect' in module 'opencv2'
  public func detectMultiScale(img: opencv2.Mat, foundLocations: inout [opencv2.Rect], foundWeights: inout [Swift.Double], hitThreshold: Swift.Double)
                                                                        ~~~~~~~ ^
opencv2.Rect:1:12: note: found candidate with type 'Rect'
open class Rect : NSObject {
           ^
Darwin.Rect:1:15: note: found candidate with type 'Rect'
public struct Rect {
              ^

I think one possible solution could be to remove the Swift stuff from the objc module and only include it via the Swift Package, but it would be a breaking change for consumers of the existing module.
TODO:

[ ] Fix swift module interface so it compiles
[ ] Get tests in modules/core/misc/objc/test to run via swift test

Resolves https://github.com/opencv/opencv/issues/18398

Related: https://github.com/opencv/opencv/issues/18843


Inconsistent punctuation use in comments

There doesn't appear to be a standard when it comes to whether to have a periods in the comments above contract functions.
ERC721.sol
/**
  * @dev Gets the owner of the specified token ID
  * @param tokenId uint256 ID of the token to query the owner of
  * @return address currently marked as the owner of the given token ID
  */
/**
  * @dev Approves another address to transfer the given token ID
  * The zero address indicates there is no approved address.
  * There can only be one approved address per token at a given time.
  * Can only be called by the token owner or an approved operator.
  * @param to address to be approved for the given token ID
  * @param tokenId uint256 ID of the token to be approved
  */

ERC20.sol
/**
  * @dev Gets the balance of the specified address.
  * @param owner The address to query the balance of.
  * @return A uint256 representing the amount owned by the passed address.
  */
/**
  * @dev Transfer token to a specified address
  * @param to The address to transfer to.
  * @param value The amount to be transferred.
  */

I don't know why I noticed...

Thanks for spotting this!
All @dev documentation should have punctuation. I'm not sure about @param. We are planning to write a style guide for this soon, but this simple criteria should be enough if someone is interested in making a PR for it.

I don't mind doing it if I have time this afternoon.

We've updated all of the @dev comments (which now have consistent punctuation). A style guide is missing still (though to be fair it may be too early to write one).


Gazebo model issues

gazebo simulation model issues fixed
1.dynamic friction reduced
2.joint torque added
3.robot feet collison points and friction added
4. gazebo test world file configuration changed

Sorry for reopening that issue. However I wanted to ask if the model issues have actually been fixed? I tried using Robinroy-peters files but it is still sliding on the ground? I am using Ros2 Humble with Gazebo 11 btw.


error in segment_path_lengths fxn

Hi,
I think there is potentially a missing np.cumsum() around the sectionfunc.segment_lengths(section) in line 293 of neuritefunc.py:
result = [_get_pathlength(section) + sectionfunc.segment_lengths(section)
for section in iter_sections(neurites, neurite_filter=neurite_filter)]
Cheers
PS. thanks for this package, it has made handling the swc file and writing custom functions a lot easier :)

Hi, thanks for opening this issue! I have fixed the code and will close this issue.
If you like NeuroM, you might be interested by a few of the other code we recently open-sourced (I sound like an add, lol)
MorphIO a lower level IO library for reading/writing morphology files
morph-tool a "toolbox" to perform various operation on morphologies
NeuroR a software for the repair of experimental morphologies


doesn't work on custom video

Thanks for the amazing work. I am getting an error while running on a custom video. Its time duration is 6 seconds and I changed the parameters. but it shows me an error in the attached image.
Your help would be highly appreciated.

Issue-Label Bot is automatically applying the label bug to this issue, with a confidence of 0.67. Please mark this comment with 👍 or 👎 to give our bot feedback!
Links: app homepage, dashboard and code for this bot.


getApplicationBasicInfo SOAP service returns 0 as the applicationId for all the applications

Description
When calling getApplicationBasicInfo SOAP service returns 0 as the applicationId for all the applications.
IS Version: 5.8., 5.9.

auto increament applicationId is part of the internal implementation logic of the Identity Server. So need it is to be exposed outside.

@gayashanbc can you link the relevant pull requests here?


Only fit psf for relevant sources

@rgiordan should review the second commit 5b88c06 to make sure it still does what its supposed to.
The basic idea here is that we:

initialize model parameters without fitting the PSF (same as before)
compile a list of all relevant sources (get_all_relevant_sources(mp))
for each relevant source, replace the corresponding SkyPatch in mp with a new one that has a fitted PSF.

This includes #163, which didn't actually turn out to be too relevant.

Yes, I will look at it tomorrow morning!  Sorry, my Thursday evenings are
always away from my computer.
On Thu, Mar 17, 2016 at 10:06 PM, Kyle Barbary<EMAIL_ADDRESS>wrote:

@rgiordan https://github.com/rgiordan should review the second commit
5b88c06
https://github.com/jeff-regier/Celeste.jl/commit/5b88c0642fda9e89fbf5a1156dd2bfbea09d6767
to make sure it still does what its supposed to.
The basic idea here is that we:

initialize model parameters without fitting the PSF (same as before)
compile a list of all relevant sources (get_all_relevant_sources(mp))
for each relevant source, replace the corresponding SkyPatch in mp
with a new one that has a fitted PSF.

This includes #163 https://github.com/jeff-regier/Celeste.jl/pull/163,
which didn't actually turn out to be too relevant.
You can view, comment on, or merge this pull request online at:
https://github.com/jeff-regier/Celeste.jl/pull/164
Commit Summary

store patch radius in pixels in SkyPatch in initialization
infer(): only fit psf for relevant sources

File Changes

M src/ModelInit.jl
https://github.com/jeff-regier/Celeste.jl/pull/164/files#diff-0 (38)
M src/Types.jl
https://github.com/jeff-regier/Celeste.jl/pull/164/files#diff-1 (3)
M src/api.jl
https://github.com/jeff-regier/Celeste.jl/pull/164/files#diff-2 (28)
M test/test_images.jl
https://github.com/jeff-regier/Celeste.jl/pull/164/files#diff-3 (23)
M test/test_misc.jl
https://github.com/jeff-regier/Celeste.jl/pull/164/files#diff-4 (27)

Patch Links:

https://github.com/jeff-regier/Celeste.jl/pull/164.patch
https://github.com/jeff-regier/Celeste.jl/pull/164.diff

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/jeff-regier/Celeste.jl/pull/164

This looks good to pull.  Thanks, Kyle!  After this is all done we should clean up this whole initialization process (it's my fault it's a mess), but this will be good for now.

Comments addressed. In the interest of consistency I've changed source index to s. I've left initialize_objid as is, since it no longer has an analog in infer() and so it isn't obvious how to factor it out. Do what you will with it in future work.

SGTM, go ahead and merge if you like, since I'm in a meeting.
-- Ryan.  Sent from a phone.  Nevertheless, all typos are deliberate.
On Mar 18, 2016 1:03 PM, "Kyle Barbary"<EMAIL_ADDRESS>wrote:

Comments addressed. In the interest of consistency I've changed source
index to s. I've left initialize_objid as is, since it no longer has an
analog in infer() and so it isn't obvious how to factor it out. Do what
you will with it in future work.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/jeff-regier/Celeste.jl/pull/164#issuecomment-198519641


python3Packages.qcs-api-client: 0.20.3 -> 0.20.4

Motivation for this change
Update to latest upstream release 0.20.4
Change log: https://github.com/rigetti/qcs-api-client-python/releases/tag/v0.20.4
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
22.05 Release Notes (or backporting 21.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 148333 run on x86_64-linux 1

8 packages built:

python38Packages.cirq
python38Packages.cirq-rigetti
python38Packages.pyquil
python38Packages.qcs-api-client
python39Packages.cirq
python39Packages.cirq-rigetti
python39Packages.pyquil
python39Packages.qcs-api-client


📚 Documentation Improvements: Typos and Readability Enhancements ✏

Overview
This pull request addresses various documentation issues, focusing on fixing typos, improving readability, and ensuring consistency throughout the documentation. ✨📚 This PR fixes issue #643 and also addresses redundancies identified in issue #677.
Changes Made

Fixed Links: Corrected the generative-ui link for proper navigation. 🔗
Capitalization Consisitency: Changed copilots to Copilots in relevant sections. 🛠
Added Title: Added "Bring Your Own LLM" title for clarity. 📄
Adjusted Font Weight: Corrected font-weight from "a" to "a". ✏
Terminology Normalization:

Changed Self Hosting to Self-Hosting.
Updated Router-mode to Router mode.
Adjusted Agent Lock to Agent-Lock.

Impact
These changes enhance the readability and consistency of the documentation, making it easier for users to navigate and understand the material. 📈
Related Issues
This pull request addresses:

Issue #643: Documentation Cleanup: Fix Typos and Improve Readability. 📝
Issue #677: Fixed redundancies identified in the documentation.

hey @arielweinberger @mme @ataibarkai Can you check my PR which can be resolving #643

The merge conflicts are arising due to the recent repo updating but I have enhanced the documentation as per the issue #643 and the redundancies from the #677
###Please have a check on it @ataibarkai @mme @arielweinberger

Hello sir! any updates?? @arielweinberger @mme @ataibarkai

There are a bunch of conflicts @LitZeus  - we moved to a new docsite just a few days ago. Please resolve the conflicts and we'll review.

looking into the new docs, will update you when the work is done

There are a bunch of conflicts @LitZeus - we moved to a new docsite just a few days ago. Please resolve the conflicts and we'll review.

@arielweinberger
I reviewed the new docsite and found some issues.
I am not raising another issue as there are 2-3 documentation issues.
I am updating you in the new PR along with the changes in the documentation if that is okay to do!

@LitZeus, are you still working on the conflicts?


add first message for Iris

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[ ] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[ ] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[ ] I chose a title conforming to the naming conventions for pull requests.

Server

[ ] Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
[ ] I followed the coding and design guidelines.
[ ] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[ ] I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
[ ] I documented the Java code using JavaDoc style.

Client

[ ] Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data.
[ ] I followed the coding and design guidelines.
[ ] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[ ] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[ ] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[ ] I documented the TypeScript code using JSDoc style.
[ ] I added multiple screenshots/screencasts of my UI changes.
[ ] I translated all newly inserted strings into English and German.

Changes affecting Programming Exercises

[ ] I tested all changes and their related features with all corresponding user types on Test Server 1 (Atlassian Suite).
[ ] I tested all changes and their related features with all corresponding user types on Test Server 2 (Jenkins and Gitlab).

Motivation and Context

Description

Steps for Testing

Prerequisites:

1 Instructor
2 Students
1 Programming Exercise with Complaints enabled

Log in to Artemis
Navigate to Course Administration
...

Exam Mode Testing

Prerequisites:

1 Instructor
2 Students
1 Exam with a Programming Exercise

Log in to Artemis
Participate in the exam as a student
Make sure that the UI of the programming exercise in the exam mode stays unchanged. You can use the exam mode documentation as reference.
...

Review Progress

Performance Review

[ ] I confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance
[ ] I confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test 1
[ ] Test 2

Test Coverage

Screenshots

Codecov Report

Merging #6699 (f7e822c) into feature/develop-iris-frontend (5536992) will increase coverage by 4.34%.
The diff coverage is 87.50%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@                         Coverage Diff                         @@
##             feature/develop-iris-frontend    #6699      +/-   ##
===================================================================
+ Coverage                            76.12%   80.46%   +4.34%
- Complexity                               0    13687   +13687
===================================================================
  Files                                 1292     2432    +1140
  Lines                                48293    92418   +44125
  Branches                              8429    12932    +4503
===================================================================
+ Hits                                 36762    74367   +37605
- Misses                                5863     9913    +4050
- Partials                              5668     8138    +2470

Impacted Files
Coverage Δ

...rcise-chatwidget/exercise-chat-widget.component.ts
90.00% <87.50%> (-0.91%)
:arrow_down:

... and 1192 files with indirect coverage changes

Flag
Coverage Δ

client
76.19% <87.50%> (+0.07%)
:arrow_up:

server
85.12% <ø> (?)

Continue to review full report in Codecov by Sentry.

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

Tested on ts9, works as expected.

But I would suggest rephrasing the last sentence to "You can learn more about me here"
Also, the here is not a link or something so the student doens't know where he can learn more about iris. Will that be developed in another PR?
Will the first message be translated to German if the language is set to german? Currently, It stays in English.

Thanks for the feedback. And yes, after the 'About Iris' page implemented, we will add the link at the end. And for the translation I need to discuss with other teammates.


Enable CodeQL

Enable CodeQL
Summary
Enable CodeQL - https://codeql.github.com/.
Description
Added GitHub workflow with CodeQL Jobs
Additional Reviewers
@birschick-bq
@xiazcy
@alexey-temnikov
@kylepbit

Codecov Report

Merging #216 (44c2ae4) into develop (98b1f90) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #216   +/-   ##
========================================
  Coverage    67.53%   67.53%
========================================
  Files          121      121
  Lines         5874     5874
  Branches       585      585
========================================
  Hits          3967     3967
  Misses        1556     1556
  Partials       351      351

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Guest users

Do Guest Users in Teams require a Power Apps license to run the app?

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

ID: a0367a42-3b99-3362-2ea7-1a821d66d4eb
Version Independent ID: 5bda16c0-b231-29f6-9da1-766ddd72b374
Content: Embed an app in Teams - Power Apps
Content Source: powerapps-docs/maker/canvas-apps/embed-teams-app.md
Service: powerapps
GitHub Login: @matthewbolanos
Microsoft Alias: mabolan

My guess is Guest Users will not see the app if its embedded in Teams

Thank you for reaching out. The answer is Yes.


initial Pull

@mrbojangles3 Is this something we are still wanting to do?  Or should we close this and archive the repo?

Closing due to inactivity.


[improvement][mow] Add profile for delete_bitmap get_agg function

Proposed changes
now profile could be
I20240412 11:08:51.078223 2771963 plan_fragment_executor.cpp:640] Fragment xxxx:(Active: 12s048ms, non-child: 0.85%)

DorisBeVersion: 7206fa868e
BlocksProduced: 40
FragmentCpuTime: 867.503ms
RowsProduced: 99.89K
VOlapTableSink:(Active: 11s946ms, non-child: 98.33%)

xxx
VNewOlapScanNode(stress_source(stress_source)) (id=0):(Active: 99.112ms, non-child: 0.82%)

xxx
VScanner:
-xxx

DeleteBitmapGetAggTime: 6.957ms
xxx
SegmentIterator:

xxx
:

xxx
LoadChannel load_id=xxx:

xxx
TabletsChannel (xxx):

xxx
RowsetBuilder 512174:

xxx
DeltaWriter 512174:
xxx
MemTableWriter 512174:
-xxx
PeakMemoryUsage:
xxx

Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall


Add WebXR Light estimation support

Reference Issue
This PR adds WebXR AR Light estimation support.

Question after reading the code: it looks like this isn't configurable. I think there should be an option on model-viewer to either keep or replace the environment (e.g. an additional "xr-environment" attribute). It's not always desired to have the real-world environment affect the AR scene as that can change the perception of brand colors etc.

Question after reading the code: it looks like this isn't configurable. I think there should be an option on model-viewer to either keep or replace the environment (e.g. an additional "xr-environment" attribute). It's not always desired to have the real-world environment affect the AR scene as that can change the perception of brand colors etc.

I would like to support the idea of light estimation being an option through a flag setting or smth.
I tried the estimation in scene viewer and it is not always producing pleasing effects, so often it is safer to set a neutral lighting or neutral hdri.

I agree with the other commenters that this should be configurable, because ARCore's lighting estimation is often a disaster. I agree a boolean xr-environment attribute that defaults to false so as not to be a breaking change is a good way to go.

One advantage of this being a separate attribute would be that there could be (future) improvements and new modes, e.g. "xr-environment: none", "xr-environment: estimated" (?), "xr-environment: mixed" to combine low-res env + high-res sharp reflections from another (the existing?) env map (good idea!).

Thanks for the comments.
Regarding low resolution environment image (on ARCore), I didn't know that, interesting. I only tested with [the space suit model](space suit model). Maybe we need to test more to determine the best solution.
I added boolean xr-environment attribute (default is false) so far. If it's false estimated environment will be ignored. I think it's an easy and good start. Feel free to make follow up PRs for the "mixed" (or other) approach.
I'm not sure the boolean xr-environment attribute will be the final solution, so I added xr-environment attribute to only the top page example of modelviewer.dev and I didn't update the document yet. We can update the document when we determine the solution.

One question to those possible future improvements, if xrEnvironment starts as boolean, what will happen once/if it becomes an enum later? Will there be a graceful transition (e.g. xrEnvironment is then equivalent to xrEnvironment="estimated") or will this be a breaking change?


Trouble reading the github page for this project

Hi there! I am a visually impaired gamer and i just found out about the decompile project for Banjo Kazooie. I use a screen reader, which is a program to read all text on websites and speak it bak with a text to speech. I have tried to navigate this github page for Banjo Kazooie, but unfortunately, it is not possible. I can't view the progress of the project, like see how much percent is done. All i get is a bunch of links which my screen reader can't do anything with. I get nothing read at all.
So would it be possible to find a way to make the site more easier for me to read using a screen reader?

The progress percentage was added as text instead of just svg to the readme a while ago. Does this solve your problem? If so, I would recommend closing this issue as solved.

Hi!
Ah, i was not aware of that. I have seen that it was changed. It does solve my problem.
How would i close it? I am not used to how that works.


Fix visibility of cursor on iPad and VR

Don't show moving cursor on iPad, only on click.
Don't show the cursor at all in VR. (fix #51)
Hide cursor when pressing a key (using wasd controls)

I have those changes on my version since some times, but please take a look @kfarr if it's fixing all what you want and I didn't change the behavior.


Send email to all registry users prompting them to update their projects with their correct contacts

We want to ensure that registry users are receiving communications, we rely that contacts in each project are up to date.
Context
Some of our users have not been receiving updates because their contact details are updated.
We discuss ways in which we can prompt users to update their contacts. We decided that an email to all users would be good idea
Will discuss this further in the next sprint meeting

Moved to the Platform Experience board


react google map zoom_changed event

I want to hide and present something else while the map has zoom in or out. please, would you mind to let me know how can I add zoom_changed to the map?

You can use the onZoomChanged property to handle this, and get the zoom value with getZoom().
...
const [map, setMap] = useState(null);
const [zoomValue, setZoomValue] = useState(10);

<GoogleMap
  defaultZoom={10}
  defaultCenter={{ lat: xx.xxxxx, lng: -x.xxxxx }}
  ref={googleMap => setMap(googleMap)}
  onZoomChanged={() => setZoom(map.getZoom())}
>
  ...
</GoogleMap>
...


Slicer - Missing property "multi-select with control"

Hi,
I was happy when I found your repo with all those templates!
I was planning on using the one for slicers, but I noticed a setting I use is not there: Multi-select using control.
Do you know what is the property name for the JSON theme?

Just found it here: https://community.powerbi.com/t5/Desktop/JSON-Label-for-Slicer-Selection-Control-Setting-Multi-select/m-p/816201
It is "multiSelect" and it is a boolean property.
I'll let you update your template!


Enhancement Proposal

Hello Abarta
I really like your plugin and would like to contribute.
Here is what I would like to work on:
Ability to change the texts in the layout
Right now, we have 3 texts which are hard-coded:
Add files
Drop multiple files here.
Space for your instructions
I would like to add them as props, which can be configurable, if needed. The defaults would remain the same.
If this sounds like a good idea, I would do the same for the div with class="uploadBoxMain"
Let me know what you think

Hi Mushood,
Thanks for using it. I'm glad you like it. Yes it sounds nice to have the text messages as props, so they can be translated or changed. You can also remove the div that holds that says "Space for your instructions". Just don't change variable names. Do a PR when done and I will review it and merge if good.
Andrei

Hello
I sent you a PR for the aforementioned proposal.
I would like to add the following enhancements in the future:

Ability to add a file one by one
Ability to remove files one by one
Add a hover class  when files are being dropped

Let me know if those are changes you'd be interested in

Hi Mushood,
Thanks for the PR, seems fine, I will look at it in detail next week and merge it if the are no issues.
Yes, what you are describing is useful extra functionality, both adding bulk and one by one. The hover class I don't understand but as it is a css class, that can be easily modifying by people using it.
Please for next PR, also update the README.md file so others know what you have added/changed.
Have a nice weekend!
Andrei

Hello Abarta,
By the hover class, I meant that when people are dropping their files, they have a visual confirmation that the files can be dropped here.
As for the README.md, understood. Just confirm to me the naming convention and I will do the necessary for the README.md
Have a nice weekend too!

Thanks for clearing that up.
Yes, both naming convention and hover class are fine.
Andrei

Hello Abarta
I updated the README file. Do check it and let me know if any changes need to be done.
Have a good week ahead

Tested, all good, pushed to npm, just removed the header prop which was not used anymore. Thanks a lot, looking forward to next PR.

A good week to you as well in Mauritius! Wish I had the same weather here.

Good week to you too in sunny Mauritius!

Sorry Abarta, was quite busy late week and things are looking busy this week as well. I'll be working on the project next week ;)
Its too hot in Mauritius right now! :P

Take your time to develop and test. Have a good busy week.

Closing this thread, if you do the PR, please open new one.
Happy Holidays!


Custom fields (feature request)

Hi,
Is there a possibility to add arbitrary custom fields to each image like, say, title, description, tags or other? Without such feature, image uploader is of little use for anything but very simple projects.

Thanks for your good suggestion. I will add theses features soon 👍


Add alternate io class

This adds an io class the converts message/commands/etc.. to json encoded strings on the terminal. Useful for external tools control, ie. vscode extension.
It might be nice to change the base aider functionality to something like this were the api is a set of commands with responses and the terminal could be implemented from that (and other applications could just use the base api), however for the moment this is the least intrusive mode for this functionality.

 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.


[Backport 2.x] Add method to Engine to fetch max seq no of last refresh

Backport https://github.com/opensearch-project/OpenSearch/commit/750bfc6216e1b67f865595bb37123d29d659ac2f from https://github.com/opensearch-project/OpenSearch/pull/5970.

Compilation error:
> Task :server:compileJava
/var/jenkins/workspace/gradle-check/search/server/src/main/java/org/opensearch/index/engine/Engine.java:311: error: cannot find symbol
        org.apache.lucene.document.Document document = searcher.storedFields().document(docs[0].doc);
                                                               ^
  symbol:   method storedFields()
  location: variable searcher of type IndexSearcher

Compilation error:
> Task :server:compileJava
/var/jenkins/workspace/gradle-check/search/server/src/main/java/org/opensearch/index/engine/Engine.java:311: error: cannot find symbol
        org.apache.lucene.document.Document document = searcher.storedFields().document(docs[0].doc);
                                                               ^
  symbol:   method storedFields()
  location: variable searcher of type IndexSearcher

This would be due to difference in Lucene Version in 2.x and main. I will check and fix.

Codecov Report

Merging #6038 (1f267b7) into 2.x (78f72ff) will increase coverage by 0.32%.
The diff coverage is 92.85%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##                2.x    #6038      +/-   ##
============================================
+ Coverage     70.13%   70.46%   +0.32%
- Complexity    58618    58958     +340
============================================
  Files          4768     4768
  Lines        282609   282623      +14
  Branches      41166    41168       +2
============================================
+ Hits         198219   199138     +919
+ Misses        67649    66867     -782
+ Partials      16741    16618     -123

Impacted Files
Coverage Δ

...rg/opensearch/index/fielddata/ScriptDocValues.java
52.63% <ø> (ø)

...in/java/org/opensearch/index/shard/IndexShard.java
69.83% <ø> (-0.30%)
:arrow_down:

.../main/java/org/opensearch/index/engine/Engine.java
75.07% <92.85%> (+1.69%)
:arrow_up:

...rch/test/junit/listeners/ReproduceInfoPrinter.java
9.52% <0.00%> (-68.26%)
:arrow_down:

.../java/org/opensearch/node/NodeClosedException.java
50.00% <0.00%> (-50.00%)
:arrow_down:

...ava/org/opensearch/action/NoSuchNodeException.java
0.00% <0.00%> (-50.00%)
:arrow_down:

...opensearch/persistent/PersistentTasksExecutor.java
22.22% <0.00%> (-44.45%)
:arrow_down:

...n/admin/indices/readonly/AddIndexBlockRequest.java
35.71% <0.00%> (-35.72%)
:arrow_down:

...nsearch/index/shard/IndexShardClosedException.java
66.66% <0.00%> (-33.34%)
:arrow_down:

...search/search/aggregations/pipeline/EwmaModel.java
24.44% <0.00%> (-31.12%)
:arrow_down:

... and 507 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


mgmt, AKS, support disk encryption set

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API changes have been detected in com.azure.resourcemanager:azure-resourcemanager-containerservice. You can review API changes here
API changes
+         String diskEncryptionSetId()
-             interface WithCreate extends Creatable<KubernetesCluster>, WithAgentPool, WithNetworkProfile, WithDnsPrefix, WithAddOnProfiles, WithAccessProfiles, WithAutoScalerProfile, WithManagedServiceIdentity, WithRBAC, WithAAD, WithLocalAccounts, Resource<DefinitionWithTags, WithCreate> {
+             interface WithDiskEncryption {
+                 WithCreate withDiskEncryptionSet(String diskEncryptionSetId)
+             }
+             interface WithCreate extends Creatable<KubernetesCluster>, WithAgentPool, WithNetworkProfile, WithDnsPrefix, WithAddOnProfiles, WithAccessProfiles, WithAutoScalerProfile, WithManagedServiceIdentity, WithRBAC, WithAAD, WithLocalAccounts, WithDiskEncryption, Resource<DefinitionWithTags, WithCreate> {


How to use "analyse.sh" and "filter_dataset.py"?

Hello together
First of all, thank you for sharing the code of this project.
I was able to train the model and make some predictions, but now I want to find the shortcomings of the model, so I want to analyze on which questions/queries the model performs well.
I found the "analyse.sh" script and the "filter_dataset.py".
Now I want to ask you what's the purpose of these files and how to use them.
Thank you for your time
Kind regards
Nicolas

Thanks @ml-nic and sorry for the delayed reply.
You can use the analyse.sh script to get insights on how predicates and single tokens were translated:
sh analyse.sh <dataset> <NL test set> <SPARQL test set>

The filter_dataset.py script is used to generate smaller training sets. You need to pass some parameters. The json file contains a dictionary with a counter for each resource,  is minimum number of occurrences per resource and the last parameter is a switch for requiring a minimum for all/any resources in the query.
python filter_dataset.py --dataset <dataset> --used_resources used_resources.json --minimum <min> --comp <all|any>


frame change bug

Changing the frame of the easytableview after it is been created, does not update the internal tableview properly

EasyTableView was just recently updated. The interface has changed slightly so you may need to make changes to your code to support EasyTableView 2.0. Please update and let me know if the same behavior still exists. Thanks.


Support new Docker Community Edition, with Stable & Edge channels on RHEL

Starting with Docker 17.03, Docker changed how they distribute Docker packages, and introduced the concept of Docker Enterprise Edition (Docker EE) and Docker Community Edition (Docker CE). It would be great if the garethr/docker module could support the new Docker CE repositories, including both the Docker CE Stable and Edge channels.
Docker CE now has two repositories, and provides both Quarterly 'stable' releases and Monthly 'edge' releases. These quarterly/monthly channels are provided via different repositories, according to https://docs.docker.com/engine/installation/linux/centos/ .
As part of the CE/EE quarterly/monthly split, Docker changed their installation procedure. The new repo file is named /etc/yum.repos.d/docker-ce.repo, and includes multiple repository channels. The stable repo is enabled by default, but the edge repo can be easily enabled for folks who want it.
Currently, garethr-docker points us to the older upstream repositories at https://yum.dockerproject.org/repo/main/centos/  which are deprecated. If I continue to use this module unmodified, my systems will end up with the Docker 17.0.4 , which I believe is the latest monthly Edge release. The latest Docker version in the Stable channel is 17.03.1 , according to https://download.docker.com/linux/centos/7/x86_64/stable/Packages/ .
The monthly Edge release cycle might be too unstable for many admins.

And the are the instructions for Debian/Ubuntu:
https://store.docker.com/editions/community/docker-ce-server-debian
https://store.docker.com/editions/community/docker-ce-server-ubuntu

@stefanlasiewski Thank you for the very important hint.
I was able to change the Repo source with the following Hiera settings:
docker::package_name: docker-ce
docker::package_key_source: https://download.docker.com/linux/centos/gpg
docker::package_source_location: https://download.docker.com/linux/centos/7/$basearch/stable
docker::package_key: 060A61C51B558A7F742B77AAC52FEB6B621E9F35

Docker version output:
 docker version
Client:
 Version:      17.03.1-ce
 API version:  1.27
 Go version:   go1.7.5
 Git commit:   c6d412e
 Built:        Mon Mar 27 17:05:44 2017
 OS/Arch:      linux/amd64

Server:
 Version:      17.03.1-ce
 API version:  1.27 (minimum version 1.12)
 Go version:   go1.7.5
 Git commit:   c6d412e
 Built:        Mon Mar 27 17:05:44 2017
 OS/Arch:      linux/amd64
 Experimental: false

yum list installed output:
yum list installed|fgrep docker
docker-ce.x86_64                  17.03.1.ce-1.el7.centos            @docker
docker-ce-selinux.noarch          17.03.1.ce-1.el7.centos            @docker

I downloaded the Repo-File from https://download.docker.com/linux/centos/docker-ce.repo and used it for the hiera values.
curl https://download.docker.com/linux/centos/docker-ce.repo

Note that for Ubuntu/Debian you have to override a couple more hieras:
docker::package_name: docker-ce
docker::package_key_source: https://download.docker.com/linux/ubuntu/gpg
docker::package_source_location: https://download.docker.com/linux/ubuntu
docker::package_key: 9DC858229FC7DD38854AE2D88D81803C0EBFCD88
docker::package_repos: stable
docker::package_release: trusty

This project has been migrated to puppetlabs/docker . I'm closing this ticket in favor of https://github.com/puppetlabs/puppetlabs-docker/issues/11 .

This project has been migrated to puppetlabs/docker . I'm closing this ticket in favor of https://github.com/puppetlabs/puppetlabs-docker/issues/11 .


Document JSON API

Once this is merged, I’d like to make a new release. 🎉

Hey, I tried to address your comments. 🙇

I like the changes.


Pre-release packaging: Windows .zips bundle duplicate binaries

Choose any recent pre-release build (e.g. this one) and download the Windows .zip file
Extract it, then enter the directory and extract that too
You'll find himalaya.exe as well as another himalaya.zip

When you open up the second .zip file, you'll find a duplicate copy of himalaya.exe (the file names, sizes, and hashes match identically)

So the application is being bundled twice

There is in fact 2 problems:

The tar archive ends up being archived with the zip one (not the opposite)
The release is bundled twice. It looks like the GitHub action that uploads artifacts does a zip on its own, and it's not customizable:

When an Artifact is uploaded, all the files are assembled into an immutable Zip archive. There is currently no way to download artifacts in a format other than a Zip or to download individual artifact contents.

Thank you for pointing it out!

Fixed on master, see https://github.com/pimalaya/himalaya/actions/runs/11596856458.


Assign labels to events

Add ability for users to create custom labels and assign them to events. Then, users can filter their weekly recurrences by label.
Todo

[x] SQL: create tables to record user labels
[ ] API: create labels endpoints

Users can assign multiple labels to an event.

HTML Color Picker Libraries

https://simonwep.github.io/pickr/

For the SQL tables, need to make 2 tables: Labels and Event_Labels

Api Endpoint for labels:
api.tasks.com/labels

Try using the new base response class for the services, and using IEnumerable instead of a list.

Sql command to modify an Event_Label
REPLACE INTO Event_Labels (event_id, label_id, created_on)
SELECT
    '049a10f7-53f1-4cdf-b906-31cfb4e8a148' AS event_id,
    '7bb42426-c172-4ffe-a973-954ed6723557' AS label_id,
    now()
WHERE
    EXISTS (
        SELECT
            1
        FROM
            Events e
        WHERE
            e.id = '049a10f7-53f1-4cdf-b906-31cfb4e8a147'
            AND e.user_id = 'b3406658-9721-4497-9eb6-784ed8555ac8'
    )
    AND EXISTS (
        SELECT
            1
        FROM
            Labels l
        WHERE
            l.id = '7bb42426-c172-4ffe-a973-954ed6723557'
            AND l.user_id = 'b3406658-9721-4497-9eb6-784ed8555ac8'
    );

When you open the labels modal, js needs to make 2 calls to the api:

a list of all the user's labels html for the form
a list of all the assigned labels for the event html

Batch insert for API
PUT: /events/:eventId/labels

removes all existing label assignments for the event
request body should be a json list of label ids to assign to the event

I think I'll need to make another endpoint for fetching all the event-label assignments in 1 api call

To filter out the recurrences board by labels, add a query parm to recurrences request: labels.
This will be a comma separated list of label ids.

Next step: update the js board controller to take in the labels url parms to send to the api


Synonym for gastrin receptor activity

Hello,
Could you please add the exact
synonym 'cholecystokinin-B receptor activity' to the GO
term 'gastrin receptor activity' (GO:0015054).
And for proof that gastrin=cholecystokinin-B, see:
http://www.copewithcytokines.de/cope.cgi?003963
and PubMed ID:10913157
cheers,
Emily
Reported by: edimmer
Original Ticket: "geneontology/ontology-requests/1672":https://sourceforge.net/p/geneontology/ontology-requests/1672

thanks www.movied.org


Added a dm that send user a dm if (s)he is given items or emeralds

idk python so please just recheck it twice or thrice

maybe you should do something like this:
if item not in ('emerald', 'emeralds', ':emerald:',)::
    await self.bot.send(user, ctx.l.econ.give.gave.format(ctx.author.mention, amount, db_item['name'], user.mention)

so people wont get spammed

maybe you should do something like this:
if item not in ('emerald', 'emeralds', ':emerald:',):
    await self.bot.send(user, ctx.l.econ.give.gave.format(ctx.author.mention, amount, db_item['name'], user.mention)

so people wont get spammed

idk python really so just copied the code from pillage command i hope ur method works but like even for emerlads would be good why not if it is more than 50 emeralds?

Denied, this is a good idea but there needs to be far more work put into it.

There should be a config option for this which updates the db and cache
There needs to be a cache (the data should be loaded from the db) for each user who has other than the default option (for example, if the default option is ON, only users who have it OFF should be in the cache) This is to save memory
There needs to be a new DB column for whether people have this on or off
text.json needs to be edited to add text for the new config option

Because of 4, let's hold off on this till the translators are done with the version I've given them.


Unable to view offline player's echests and inventories

Describe the bug
If I remember correctly you were able to view offline player's echests and inventories in 1.16. This does not seem to be the case in 1.17 from my tests. Entering the command results in no action, not even a message and nothing in the server logs.
Expected behavior
Be able to view offline player's echests and inventories.
Screenshots
If applicable, add screenshots to help explain your problem.
Device Info(please complete the following information):

OS: Mac
1.17
1.4.2

Additional context
I am using /player from carpet mod to set up some starting items for players on the whitelist. I tested if altering the echest of a fake player would transfer to the real player, and it does. I tried viewing the inventory of my alt account that had actually been logged on and that didn't work either. The server is being locally hosted.

this repro's for me too

Im having the same issue

It definitely works so it's most likely some sort of compat issue. Not much I can do without logs.

this seems to cause a crash on hosted servers if this is attempted, seems to be a incompatibility with Player Roles mod?
the server was running Origins mod too
this is what gets printed out if attempted to access an offline players inventory:
10.08 07:57:17 [Server] Server thread/INFO Error while reading data: player roles were not initialized 10.08 07:57:17 [Server] Server thread/ERROR Encountered an unexpected exception 10.08 07:57:17 [Server] INFO java.lang.NullPointerException: player roles were not initialized 10.08 07:57:17 [Server] INFO at com.google.common.base.Preconditions.checkNotNull(Preconditions.java:787) ~[custom.jar:?] 10.08 07:57:17 [Server] INFO at net.minecraft.class_3222.getPlayerRoleSet(class_3222.java:1720) ~[intermediary-server.jar:?] 10.08 07:57:17 [Server] INFO at dev.gegy.roles.PlayerRoles$1.byEntity(PlayerRoles.java:58) ~[player-roles-1.4.2.jar:?] 10.08 07:57:17 [Server] INFO at dev.gegy.roles.api.RoleLookup.byPlayer(RoleLookup.java:25) ~[24fd5af1-2dd5-4946-989e-ad5598af7a37.jar:?] 10.08 07:57:17 [Server] INFO at net.minecraft.class_3222.method_5476(class_3222.java:2199) ~[intermediary-server.jar:?] 10.08 07:57:17 [Server] INFO at io.github.apace100.apoli.power.factory.action.EntityActions.lambda$register$24(EntityActions.java:310) ~[18478006-321c-438c-9736-0865a2c1b1ab.jar:?] 10.08 07:57:17 [Server] INFO at io.github.apace100.apoli.power.factory.action.ActionFactory$Instance.accept(ActionFactory.java:41) ~[18478006-321c-438c-9736-0865a2c1b1ab.jar:?] 10.08 07:57:17 [Server] INFO at io.github.apace100.apoli.power.factory.action.EntityActions.lambda$register$3(EntityActions.java:69) ~[18478006-321c-438c-9736-0865a2c1b1ab.jar:?] 10.08 07:57:17 [Server] INFO at io.github.apace100.apoli.power.factory.action.ActionFactory$Instance.accept(ActionFactory.java:41) ~[18478006-321c-438c-9736-0865a2c1b1ab.jar:?] 10.08 07:57:17 [Server] INFO at io.github.apace100.apoli.power.factory.action.EntityActions.lambda$register$8(EntityActions.java:113) ~[18478006-321c-438c-9736-0865a2c1b1ab.jar:?] 10.08 07:57:17 [Server] INFO at io.github.apace100.apoli.util.Scheduler.lambda$new$0(Scheduler.java:49) ~[18478006-321c-438c-9736-0865a2c1b1ab.jar:?] 10.08 07:57:17 [Server] INFO at net.fabricmc.fabric.api.event.lifecycle.v1.ServerTickEvents.lambda$static$2(ServerTickEvents.java:62) ~[d9182cfe-7c8b-46a0-96c8-29746296c107.jar:?] 10.08 07:57:17 [Server] INFO at net.minecraft.server.MinecraftServer.handler$zna000$onEndTick(MinecraftServer.java:3327) ~[intermediary-server.jar:?] 10.08 07:57:17 [Server] INFO at net.minecraft.server.MinecraftServer.method_3748(MinecraftServer.java:873) ~[intermediary-server.jar:?] 10.08 07:57:17 [Server] INFO at net.minecraft.server.MinecraftServer.handler$bdm000$modifiedRunLoop(MinecraftServer.java:4890) ~[intermediary-server.jar:?] 10.08 07:57:17 [Server] INFO at net.minecraft.server.MinecraftServer.method_29741(MinecraftServer.java:676) ~[intermediary-server.jar:?] 10.08 07:57:17 [Server] INFO at net.minecraft.server.MinecraftServer.method_29739(MinecraftServer.java:270) ~[intermediary-server.jar:?] 10.08 07:57:17 [Server] INFO at java.lang.Thread.run(Thread.java:831) [?:?]


Refactor HTML5 simpletable accessibility for modern approach

Signed-off-by: Robert D Anderson<EMAIL_ADDRESS>This pull request modifies the way simpletable headers are marked up for accessibility.
The original approach specified @id for every header cell, and used @headers on body cells to reference each header. This was the recommended approach when DITA-OT 1.0 came out and was used in XHTML.
There are a number of problems with this:

It used generated IDs, which are a problem for many systems that use diff to compare output
The output is extremely verbose, while many HTML5 developers prefer clean files / minimal noise
In our implementation, generated IDs for cells inside sthead often override specified IDs, so links to those entries do not resolve
Even when specified IDs are preserved today, the actual ID is used instead of topic__id - so again, links to those IDs do not resolve
In 2.5, when a header row is specified for <choicetable>, the same generated ID is used for each cell of the header, so those are broken today (it only works when the header row is generated, or when the header entries specify their own ID)

The updated code here switches to use the @scope attribute to mark header cells. It is always used for entries inside sthead (with scope="col"), and is also used for any designated key column (with scope="row"). The code to generate these values is much simpler, which makes it easy to fix items 3/4/5 above. By removing extra noise from the HTML5 output, we also move to the now-recommended approach for data tables. See: https://webaim.org/techniques/tables/data#headers

The scope attribute tells the browser and screen reader that everything within a column that is associated to the header with scope="col" in that column, and that a cell with scope="row" is a header for all cells in that row.
All  elements should generally always have a scope attribute. While screen readers may correctly guess whether a header is a column header or a row header based on the table layout, assigning a scope makes this unambiguous.
...
Another way to associate data cells and headers is to use the headers and id attributes. This method is NOT generally recommended because scope is usually sufficient for most tables, even if the table is complex with multiple levels of headers.

A few notes / caveats:

This update is straightforward for simpletables, where entries cannot span and headers are easy to calculate. I'd like to make the same update for complex tables but am not sure I'll have time before 3.0 is frozen. The updates for complex tables are closely related to #1779 so should probably be done together.
I've only done this for HTML5. Given that the existing support is valid (although outdated), and given the churn this would cause for existing XHTML table overrides, I'd prefer to leave that alone (much higher cost for a lower return).
The update here covers simpletables and OASIS specializations of simple tables (<properties>, <choicetable>).
When integrated, this should close #2448 and #2449 that both suggest removing generated headers in simple tables

Tested using the following files, which cover simpletable (with / without headers and keycol), properties (with/without generated headers and keycol), and choice tables (with / without headers and keycol).
1779.zip

@robander Doesn't this make #2449 obsolete.

Yes - I'll mark it closed.


[Feature Request] Ignore files and folders

Feature: In the setting, specify files and folders to ignore parsing todos
Usecase: I have templates with todos that are being parsed by the grind manager. I would like the grind manager to skip these todos in templates.

Hi, @eduongAZ! Thanks for a suggestion. I'll try to implement it in the near future.

I added this feature in ab77c95b35bbaae82da4169fea14cb9a5cea646d, it will be available in 1.1.0 (Release 15 April)


[BugFix] Fix week of the year behavior of date_format function inconsistent with mysql

Why I'm doing:
1 sr data_format behavior:
+----------------------------------+
| date_format('20241231', '%x-%v') |
+----------------------------------+
| 2025-01                          |
+----------------------------------+
the fe date_format behavior diff with be
+-------------------------------+
| date_format('20241231', '%v') |
+-------------------------------+
| 53                            |
+-------------------------------+
2 mysql data_format behavior:
+-------------------------------+
| date_format('20241231', '%v') |
+-------------------------------+
| 01                            |
+-------------------------------+
What I'm doing:
fix the behavior
Fixes #issue
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[x] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

@trueeyu @imay
Please help review code

add a test case plz?

add a test case plz?

OK

@Seaven
hi Seaven, the UT has been added.

@Seaven hi seaven, Please help review code

@Seaven
hi seaven
Quality Gate passed

@Seaven hi seaven, Please help review code

Sorry, I'm late


Missing a reference of the alert parameter in RMUniversalAlertCompletionBlock on v0.8

Recently I recognize the actionSheet is not working after updating RMUniversalAlert to 0.8.
As I check the issue, I found the issue was caused by the Pull Request #18.
    __weak __typeof(self) weakSelf = self;

    [RMUniversalAlert showActionSheetInViewController:self
                                            withTitle:nil
                                              message:NSLocalizedString(@"message", @"")
                                    cancelButtonTitle:NSLocalizedString(@"cancel", @"")
                               destructiveButtonTitle:nil
                                    otherButtonTitles:@[NSLocalizedString(@"ok", @"")]
                   popoverPresentationControllerBlock:^(RMPopoverPresentationController *popover) {
                       __weak __typeof(weakSelf) self = weakSelf;
                       if (!self) return;

                       // Do something
                   }
                                             tapBlock:^(RMUniversalAlert *alert, NSInteger buttonIndex) {
                                                 __weak __typeof(weakSelf) self = weakSelf;
                                                 if (!self) return;

                                                 if (buttonIndex == alert.cancelButtonIndex) {
                                                     // Do something on clicking cancel button
                                                 } else if (buttonIndex == alert.destructiveButtonIndex) {
                                                     // Do something on clicking destructive button
                                                 } else if (buttonIndex >= alert.firstOtherButtonIndex) {
                                                     // Do something on clicking alert button
                                                 }
                                              }];

I usually use it in this way.
After updating onto 0.8, the alert parameter on tab block was always passed by nil.
I hope this issues would be fixed.
Thank you/

Hi, thanks are you able to submit a PR?


Maintainers outside of base16-project

@danielsz you have write access to this repo but aren't on the http://github.com/base16-project team. Should I add you to the team (as a maintainer) or remove you from the project? I'm not sure if you're aware of base16-project or the changes happening to base16 in general? Additionally, if you'd like to be part of the team and maintain different projects, let me know which ones and I can add you ad a maintainer there.

I'm going to remove you from the organization @danielsz, but let me know if you want to contribute in future and I'll add you.


what is the good way to destroy the recorder and release the resource it use?

question:
what is the good way to destroy the recorder? It seems that the memory usage keep rising during recording.
In beforeEach:
recorder = new PuppeteerScreenRecorder(pyBell.page);
await recorder.start(./report/video/a.mp4);
after finish executing test case:
await recorder.stop();
just repeat the above steps,After loads of tests,the memory usage rise to unacceptable level,and cause test to fail.
Also tried stream,it turns out the same.

Any update for this? This library is unusable due to memory usage not being released. Any alternatives?


ambiguity="explicit" returns None

Describe the bug
When I set ambiguity="explicit", it returns None when parsing. Not setting it makes it parse normally
To Reproduce
#!/usr/bin/env python3
# -*- coding: utf-8 -*-
"""
Initial author: Bryan Hu.

The main program that will run.
"""
from pathlib import Path

import lark

here = Path(__file__).parent
lark_grammar = Path(here / "grammar.lark")

parser = lark.Lark(lark_grammar.read_text(), ambiguity="explicit")
DOC = R"""
func main (argv:List<Str>, argc: int) {
    obj Greeter {
        private var _name
        var name: property<Str>
        func init(name: str) -> Void {
            self._name = name
            self.name = property(
                get=(func() -> Str {return self._name})
            )
        }
        func __str__() -> Str {
            return "hello, " + self._name
        }
    }
    if not (len(argv) > 1) {
        print("You gotta give me a name")
    } else {
        var maing: Greeter = Greeter(argv[1])
        print(maing)
    }

}
"""
print(parser.parse(DOC).pretty())

And the very ambiguous, nondeterministic grammar (still trying to reduce ambiguity):
%import common.WS
%import common.NEWLINE
%import common.CNAME -> IDENTIFIER
%import python.STRING
%import common.SIGNED_NUMBER -> NUMBER
%import common.SH_COMMENT -> COMMENT

%ignore WS
%ignore ";"
%ignore NEWLINE
%ignore COMMENT

start: _statement+
////////////////////////////////////////////////////////////////
_statement:  _definition | _operation | _ifs
////////////////////////////////////////////////////////////////
_definition: function_definition | class_definition | variable_definition
// FUNCTION //
function_definition: "func" IDENTIFIER "(" [_sequence{functype, ","}] ")" ["->" _type] braced_code
anonymous_func: "(" "func" "(" [_sequence{functype, ","}] ")" ["->" _type] braced_code ")"
// VARIABLE //
?variable_definition: ("var"|"const") IDENTIFIER ((":" _type) | ([":" _type] "=" value))
// CLASS //
class_definition: "obj" IDENTIFIER [":" NAMESPACABLE] class_code
class_code: "{" (class_defs | _statement)+ "}"

?class_defs: meth_def | class_var_def

?meth_def: [PRIVATE|PUBLIC] function_definition
class_var_def: [PRIVATE|PUBLIC] variable_definition

PRIVATE: "private"
PUBLIC: "public"
////////////////////////////////////////////////////////////////
_operation:  _binary | _logic | _misc | value
// BINARY //
_binary: add
       | sub
       | div
       | mul
       | bor
       | band
       | bnot
       | bxor
add: _bop{"+"}
sub: _bop{"-"}
div: _bop{"/"}
mul: _bop{"*"}
bor: _bop{"|"}
band: _bop{"&"}
bnot: "!" _operation
bxor: _bop{"~"}
// LOGIC //
_logic: gt
      | lt
      | eq
      | not
      | and
      | or
gt: _bop{">"}
lt: _bop{"<"}
eq: _bop{"=="}
and: _bop{"and"}
or: _bop{"or"}
not: "not" _operation
// MISCELLANEOUS //
_misc: assign | return | index | call_op
assign: _bop{"="}
return: "return" [_sequence{value, ","}]
index: value "[" value "]"
call_op: (NAMESPACABLE|anonymous_func) "(" [arguments] ")"

//! PRIVATE !//
_bop{thing}: value thing _operation
arg.2: [IDENTIFIER "=" ] value
arguments: _sequence{arg, ","}  // To prevent clashing with assignment
// FILLER: "..."
////////////////////////////////////////////////////////////////
_ifs: if (elif)* else?
?if: "if" value braced_code
?elif: "else" if
?else: "else" braced_code
////////////////////////////////////////////////////////////////
?value: NAMESPACABLE
      | STRING
      | NUMBER
      | BOOLEAN
      | anonymous_func
      | paranthesised
      | _operation

NAMESPACABLE: /[a-zA-Z_]+(?:\.[0-9a-zA-Z_]+)*/
BOOLEAN: "true" | "false"

braced_code: "{" _statement+ "}" // TODO USE INDENT-BASED

_type: NAMESPACABLE | generic_type
generic_type: NAMESPACABLE "<" _type ">"
functype: (IDENTIFIER ":" _type) ["=" value]
paranthesised.2: "(" value ")"  // Will clash with call-op

_sequence{x, sep}: x (sep x)* sep?

Terminal output:
Traceback (most recent call last):
  File "__main__.py", line 40, in <module>
    print(parser.parse(DOC).pretty())
AttributeError: 'NoneType' object has no attribute 'pretty'

If that is impossible, provide clear steps to reproduce the behavior.

Poetry lock file (if useful): https://gist.github.com/ThatXliner/e900393295d8337e87acd2902734c484

Poetry lock file (if useful): https://gist.github.com/ThatXliner/e900393295d8337e87acd2902734c484

(note, a mcve would be nice. This is a bit much to figure what is happeing)
Using git bisect, I found that this is related to #745 by @chanicpanic . Note that this wasn't a regression. Before that PR, this would error with index out of bounds, now it returns None.
This correctly indicates that there is an infinite ambiguity, which you can check by calling lark.logger.setLevel('DEBUG'), and passing debug=True to Lark. This infinite ambiguity is in this specific case between value and _operation. You should probably restructured your grammar to not contain this. But lark still shouldn't return None

(note, a mcve would be nice. This is a bit much to figure what is happeing)
Using git bisect, I found that this is related to #745 by @chanicpanic . Note that this wasn't a regression. Before that PR, this would error with index out of bounds, now it returns None.
This correctly indicates that there is an infinite ambiguity, which you can check by calling lark.logger.setLevel('DEBUG'), and passing debug=True to Lark. This infinite ambiguity is in this specific case between value and _operation. You should probably restructured your grammar to not contain this. But lark still shouldn't return None

Here a (I think) minimal example:
GRAMMAR = """
start: _operation
_operation:  value
value: "b"
      | "a" value
      | _operation
"""

parser = lark.Lark(GRAMMAR, ambiguity="explicit", debug =True)
print(parser.parse("ab"))

Observation: This works correctly when start: value is used instead. @ThatXliner that might also apply for you larger grammar, essentially keeping the infinite ambiguity, but not using _operation outside of value

Here a (I think) minimal example:
GRAMMAR = """
start: _operation
_operation:  value
value: "b"
      | "a" value
      | _operation
"""

parser = lark.Lark(GRAMMAR, ambiguity="explicit", debug =True)
print(parser.parse("ab"))

Observation: This works correctly when start: value is used instead. @ThatXliner that might also apply for you larger grammar, essentially keeping the infinite ambiguity, but not using _operation outside of value

This correctly indicates that there is an infinite ambiguity, which you can check by calling lark.logger.setLevel('DEBUG'), and passing debug=True to Lark. This infinite ambiguity is in this specific case between value and _operation. You should probably restructured your grammar to not contain this. But lark still shouldn't return None

Why not make it so when it does return None, raise an exception instead

This correctly indicates that there is an infinite ambiguity, which you can check by calling lark.logger.setLevel('DEBUG'), and passing debug=True to Lark. This infinite ambiguity is in this specific case between value and _operation. You should probably restructured your grammar to not contain this. But lark still shouldn't return None

Why not make it so when it does return None, raise an exception instead

I have a fix for this. I'll open a PR shortly.

I have a fix for this. I'll open a PR shortly.

@ThatXliner Please verify that latest master version fixes it.

@ThatXliner Please verify that latest master version fixes it.

You fixed it. Yep.

@ThatXliner Please verify that latest master version fixes it.

You fixed it. Yep.

@ThatXliner Please verify that latest master version fixes it.


Upgrade Apache Commons Collections to v3.2.2

Upgrade Apache Commons Collections to v3.2.2
Version 3.2.1 has a CVSS 10.0 vulnerability. That's the worst kind of
vulnerability that exists. By merely existing on the classpath, this
library causes the Java serialization parser for the entire JVM process
to go from being a state machine to a turing machine. A turing machine
with an exec() function!
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2015-8103
https://commons.apache.org/proper/commons-collections/security-reports.html
http://foxglovesecurity.com/2015/11/06/what-do-weblogic-websphere-jboss-jenkins-opennms-and-your-application-have-in-common-this-vulnerability/

We are not actually using commons collections, so I completely removed the references to it instead.
Thanks for pointing out the problem, though.


BCGPLVM and infer_newX

Hi,
When I try to infer_newX using BCGPLVM I get the following error. Is there any reason for this?
Note: the infer_newX is working fine with me for other models
z_test[indx, :]= m.infer_newX(x_test[i].reshape(1,-1))[0].mean[0]

File "/usr/local/lib/python2.7/dist-packages/GPy/core/gp.py", line 571, in infer_newX
return infer_newX(self, Y_new, optimize=optimize)
File "/usr/local/lib/python2.7/dist-packages/GPy/inference/latent_function_inference/inferenceX.py", line 22, in infer_newX
infr_m = InferenceX(model, Y_new, init=init)
File "/usr/local/lib/python2.7/dist-packages/paramz/parameterized.py", line 49, in call
self = super(ParametersChangedMeta, self).call(*args, **kw)
File "/usr/local/lib/python2.7/dist-packages/GPy/inference/latent_function_inference/inferenceX.py", line 82, in init
self.X = self._init_X(model, Y, init=init)
File "/usr/local/lib/python2.7/dist-packages/GPy/inference/latent_function_inference/inferenceX.py", line 114, in _init_X
X = Param('latent mean',(model.X[idx].values).copy())
AttributeError: 'numpy.ndarray' object has no attribute 'values'
Thanks in advance

What version is your GPy and Paramz? Could you update the GPy and Paramz
and try again?
Zhenwen
On 31/05/2016 20:46, kabdelfatah wrote:

Hi,
When I try to infer_newX using BCGPLVM I get the following error. Is
there any reason for this?
Note: the infer_newX is working fine with me for other models
|z_test[indx, :]= m.infer_newX(x_test[i].reshape(1,-1))[0].mean[0] |
File "/usr/local/lib/python2.7/dist-packages/GPy/core/gp.py", line
571, in infer_newX
return infer_newX(self, Y_new, optimize=optimize)
File
"/usr/local/lib/python2.7/dist-packages/GPy/inference/latent_function_inference/inferenceX.py",
line 22, in infer_newX
infr_m = InferenceX(model, Y_new, init=init)
File "/usr/local/lib/python2.7/dist-packages/paramz/parameterized.py",
line 49, in call
self = super(ParametersChangedMeta, self).call(/args, */kw)
File
"/usr/local/lib/python2.7/dist-packages/GPy/inference/latent_function_inference/inferenceX.py",
line 82, in init
self.X = self._init_X(model, Y, init=init)
File
"/usr/local/lib/python2.7/dist-packages/GPy/inference/latent_function_inference/inferenceX.py",
line 114, in _init_X
X = Param('latent mean',(model.X[idx].values).copy())
AttributeError: 'numpy.ndarray' object has no attribute 'values'
Thanks in advance
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/SheffieldML/GPy/issues/396, or mute the thread
https://github.com/notifications/unsubscribe/AFwBWnLNpj7aE14TgOB7zhbYR1IbfC7kks5qHJAlgaJpZM4Iq41s.


fix load circles into canvas from library

data is not being read properly, [0] index is undefined?
data also needs to be objectified then deconstructed to pass into new circle element.


Added function pretty for pretty printing JSON

Signed-off-by: larsewi<EMAIL_ADDRESS>

Thanks for submitting a PR! Maybe @vpodzime can review this?


MediaMTX crashes occasionally when publishing AV1 over WEBRTC

Which version are you using?
v0.23.8
Which operating system are you using?

[x] Linux amd64 standard
[ ] Linux amd64 Docker
[ ] Linux arm64 standard
[ ] Linux arm64 Docker
[ ] Linux arm7 standard
[ ] Linux arm7 Docker
[ ] Linux arm6 standard
[ ] Linux arm6 Docker
[ ] Windows amd64 standard
[ ] Windows amd64 Docker (WSL backend)
[ ] macOS amd64 standard
[ ] macOS amd64 Docker
[ ] Other (please describe)

Describe the issue
I have not been able to reproduce this crash reliably so a journalctl crash logs are all I have. This crash happens when I publish AV1 video over with RTSP and open it with WebRTC. I am not sure if it's the opening and closing of the browser window that's causing the crashes or something else.
I have two different logs:
crashlog.txt
crashlog2.txt
mediamtx.yml:
mediamtx.yml.txt
Describe how to replicate the issue

I do not know

Did you attach the server logs?

yes
Did you attach a network dump?

no

Hello, thanks for reporting the crashes. The second one is caused by these non-standard HTTP requests with an absolute URL in place of the path, and is fixed with by #2169
OPTIONS http://localhost HTTP/1.1
Host: localhost:8889
Accept-Encoding: gzip
User-Agent: Go-http-client/1.1

The bug that causes first crash is not present on v0.23.8 but is present on v0.23.7, since line webrtc_manager.go:395 is empty in v0.23.8, and has already been fixed by #2072 in v0.23.8.
Next time please report the two bugs in two separate issues.

They arose from similar circumstances so I assumed they were related. Anyways, thank you. I'll compile the latest version and report any new crash I encounter.


Fall back to read-only credentials on EACCES errors

This change fixes a bug where multiple threads and/or processes
using multistore_file on Windows to access the same backing
store could raise IOError errno.EACCES to the calling application.
This change causes a fallback to read only mode and refreshes
credentials if necessary.
Fixes #386

Can you add a test case for this?

I did in test_multistore_file?

Oh, of course. I get an 'F' for reading comprehension.
@dhermes @nathanielmanistaatgoogle this change looks good to me, please take a look.


Column autoHeight doesn't adjust without first clearing the store

If editing a field changes the row height, you currently need to clear the store before loading the data in order to get the height to adjust without needing to refresh the browser. This happened when editing a task in the todoApp.
//  Clear store so description field's auto height readjusts after edit
this.gridModel.clear();
this.gridModel.loadData(tasks);

Thanks for catching and filing!
I've opened the following PRs which should address the issue:
https://github.com/xh/hoist-react/pull/2471/files
https://github.com/xh/toolbox/pull/485


Faculty of Engineering Tanta University

Faculty of Engineering Tanta University
URL of college: https://eng.tanta.edu.eg/en/

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

@MohameddEzzatt I am sorry, the domain f-eng.tanta.edu.eg is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


certifier.so: Convert to TSDbg

The TSDbg works better in shutdown conditions. The certifier plugin with the old logging interface was experiencing a shutdown crash. Converting to TSDbg addresses this crash.
Fixes #9794

[approve ci autest]

[approve ci rat]

Might as well take all the changes from #9975 as well.

The certifier plugin has been converted to DbgCtl via #10390:
https://github.com/apache/trafficserver/pull/10390/files#diff-14664431da891cb710c866874630968fad88531ac96f9e6e7b2558ee13988f7e
Closing this PR.