Upgrade sane and use FSEventsWatcher on Darwin

This is to fix regressions with the NodeWatcher on macOS.
More info here: https://github.com/amasad/sane/pull/97
Test plan

yarn test
on macOS create a new dir and install a preposterous number of node_modules
run jest tests in that directory using this branch

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3902   +/-   ##
=======================================
  Coverage   58.01%   58.01%
=======================================
  Files         195      195
  Lines        6741     6741
  Branches        6        6
=======================================
  Hits         3911     3911
  Misses       2827     2827
  Partials        3        3

Impacted Files
Coverage Δ

packages/jest-haste-map/src/index.js
92.59% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Anybody wants to send a fix for this so we can get it into the next Jest release?

Ask and you shall receive: #3918

Thanks! I sent this on go so couldn't update. Feel free to close this one.

Anybody know when the next Jest version will be released with this fix?

@kirkstrobeck you can use prerelease tagged<EMAIL_ADDRESS>

@thymikee Unfortunately, getting errors on that
Error: Cannot find module './state'
    at Function.Module._resolveFilename (module.js:485:15)
    at Function.Module._load (module.js:437:25)
    at Module.require (module.js:513:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/Users/kirkstrobeck/dropbox-personal/Sync/git/healthline/frontend/node_modules/jest-snapshot/build/index.js:24:14)
    at Module._compile (module.js:569:30)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)
    at Function.Module._load (module.js:458:3)
error Command failed with exit code 1.

Had this same error. I ran yarn add once again and it clicked

I can repro that fix..


Possibly Missing Interpolation in SMTLibInteractiveTheoremProver

Hi,
I was reading the source code of the SMTLibInteractiveTheoremProver class, and I saw that line 812 has the following context:
foreach (var relaxVar in relaxVars)
{
  var resp = await SendVcRequest("(get-value ({relaxVar}))").WaitAsync(cancellationToken);
...

I saw this as highly suspicious, prompting me to open this issue. Should there not be a $ before "(get-value...?

I saw this as highly suspicious, prompting me to open this issue. Should there not be a $ before "(get-value...

Oh, I think you're right! Fortunately that's easy to fix. It makes me think that no one has been using this functionality for a while.

Hi, happy to hear that it is not a false positive :) I saw that only Houdini uses it but it is still good to keep things correct.


4D53880C - Halo 3 Beta

Marketplace N/A
Tested on https://github.com/xenia-canary/xenia-canary/commit/6181321fd974272edc2ad1f0cf4e848af3e75d25 (import-libraries)
Issues:
Initially would guest crash on launch with an IO error message, merging in the cache commit & modifying it to link the cache:\ drive after cache0:\ let it get passed this error, but only to a black screen...
Using the unlocked XEX patches by Lord Zedd allows it to run properly, AFAIK everything mostly seems to work fine besides some audio problems (eg. garbage before menu shows up). I didn't test for that long though, so its possible that it'll host crash like retail Halo 3 after some time.
The TU2 XEX also appears to work too, though sadly the full TU2 doesn't seem to be available on the net anywhere, so I might not have tested the complete update...
Log:
(TODO: will update this with the log soon..)
Screenshot(s):

Labels:
apu-garbage
state-gameplay
marketplace-delisted
tech-engine-blam

how did you merge and modified the link the cache:\ drive thing to get passed that error,


error submitting build... already submitted

I provided the wrong buildpack and the wrong path when building my droplet.
$ ltc build-droplet dora go --path=.
Submitted build of dora
08/10 11:40:35.69 [BUILD|0] Successfully created container
Build failed: Downloading failed

When I try again, I get an error
$ cd dora/

$ ltc build-droplet dora ruby --path=.
Error submitting build of dora: build-droplet-dora has already been submitted

$ ltc list-droplets
Droplet		Created At	Size

Now what do I do?

Diego eventual consistency at play here.
$ ltc list
------------------------------= Apps =-------------------------------
No apps to display.

------------------------------= Tasks =------------------------------
Task Name		Cell ID		Status		Result		Failure Reason
build-droplet-dora	cell-01		COMPLETED	N/A		N/A

$ ltc list
------------------------------= Apps =-------------------------------
No apps to display.

------------------------------= Tasks =------------------------------
No tasks to display.

$ ltc build-droplet dora ruby --path=.
Submitted build of dora
08/10 11:50:59.48 [BUILD|0] Successfully created container
08/10 11:51:11.13 [BUILD|0] -------> Buildpack version 1.6.2
...

Just following up,
This is due to the fact that we submit builds with a predictable task name, build-droplet-FOO so that the user can simultaneously follow logs of the build process without having to figure out a GUID name or other.
Unfortunately, Diego does not reap tasks immediately, so in the case of a typo'd buildpack or broken app, you need to wait a few minutes to try again.
Alternatively, you can brute-force tell Diego to wipe out all task information, but this is Not Recommended outside of a Dev context:

curl -L http://<IP_ADDRESS>:4001/v2/keys/v1/task/?recursive=true -XDELETE

--
Marco Nicosia
Product Manager
Pivotal Software, Inc.

That curl is really dangerous. You can almost always use ltc cancel-task build-droplet-xxx (if it's running) or ltc delete-task build-droplet-xxx (if it's completed).


Ensure schema consistency when configuring replication

Before this change we would attempt to subscribe to any database that we were pointed at if we could connect to it
If the local and remote databases had different schemas we would likely fail to complete the data sync as in https://bugzilla.redhat.com/show_bug.cgi?id=1331114
This change will check both the local and remote databases against schema.yml file before attempting to save the subscription and will raise an exception if either does not match

@miq-bot add_label replication, core, bug


Fixes #469

Fixes #469

@alfredmyers,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/Consume-Packages/Finding-and-Choosing-Packages.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@karann-msft can you review?

Verified with changes to nuget.org; thanks for the edits.

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

Line 357: [Error] Error happened in build.entrypoint.ps1. Please build your target branch first and then reopen this pull request to do another round of verification. If the issue still happens, open a ticket in http://SiteHelp and include the following error message for our team to troubleshoot: The process cannot access the file 'tmp_pack_QqZNTa' because it is being used by another process.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Thanks; I verified the changes with the current nuget.org UI. I appreciate you taking the time to make the edits!


Please create a full sample run in background for Xamarin.Forms?

Thank you for your plugin!
It's help me easy when create a geofence. But main mission of geofence is running in background.
If you have free time, please create a full sample Geofence run in Background for Xamarin.Forms (Android, iOS and UWP).
Thank a lot!

listen man, I've tried to be polite.  I provide the code.  You use it at your own risk.  Background does work, you just need a sticky service to keep everything alive.

Thank you for polite.
I very hate when you close after one reply. You busy or you don't want support user. Please do not create anything when you don't want support user!

I can do whatever I want with my code.  I was polite and you reply with a rude comment because I don't hold your hand through everything.  I can do whatever I like with my time and my code.
You are not welcome back.

@aritchie thanks for such an amazing library and contribution and you sure can do whatever you want with your time. It's remarks like these that make open source toxic when people come demanding fixes, changes and support with no understanding at all.


Clean data chart

There is a function by which a chart is built for a certain pair of currencies, and when switching this pair, in theory I need to clear the chart and load new data into it. If I add the line chart.removeSeries (lineSeries); then the chart is always empty, what's the problem?
function cd(){
    const lineSeries = chart.addLineSeries();
    let data_chart = [];
  //  chart.removeSeries(lineSeries);
    if (typeof(Data_for_chart[CoinVal]) != "undefined"){
        if (typeof(Data_for_chart[CoinVal][CoinProd]) != "undefined"){
            if ((typeof(Data_for_chart[CoinVal][CoinProd]['Price']) != "undefined")&&(typeof(Data_for_chart[CoinVal][CoinProd]['Time']) != "undefined")){
                console.log(Data_for_chart[CoinVal][CoinProd]['Price'])
                for(var i = 0; i < Data_for_chart[CoinVal][CoinProd]['Price'].length; i++){
                    data_chart.push({time:Data_for_chart[CoinVal][CoinProd]['Time'][i], value:Data_for_chart[CoinVal][CoinProd]['Price'][i]});
                }
                lineSeries.setData(data_chart);
            }
        }
    }
}

You don't need to remove a series if you just want to update its data, setData replaces all the data of the series

@EffMining
	reCreateChart() {
			this.chart.removeSeries(this.series)
			this.chart = null
			this.createChart() // create chart again
		},

@nikitamarcius recreating the whole chart might cause performance issues. I'm sure you know what you're doing, it's just a note :) If you want to replace series, it's better to remove that series and create another one.

@timocov sure. While I have such a solution. When a trading pair changes, the OHLC data is rebuilt, while the chart remains in the same coordinates along X and Y.
It is possible to align the graph only horizontally chart.timeScale().resetTimeScale(), but not vertically, I do not know this possibility.

@nikitamarcius it looks like an feature request for API to get/set price range...

@timocov 😉

What is the solution for pushing marginal new data rather than rerendering the whole chart every 2 seconds? @timocov

for pushing marginal new data

@johnworthley what do you mean?

Believe @johnworthley is looking for update()

What works perfectly for me is this:
chart.remove();
Then you can call createChart again.


PageBase.position attribute not pre-populated by admin

issue introduced in https://github.com/SexualHealthInnovations/django-wizard-builder/pull/30

closing in favor of https://github.com/SexualHealthInnovations/django-wizard-builder/issues/97


Add Involuntary Conversions test cases

This PR covers the test cases for Involuntary Conversions:

create involuntary project
validation checks (few tests are skipped as we have a raised bug for those)
change school and trust (these are failing atm due to the performance issue as the search dropdown is taking long to display. This might be resolved once performance issue is resolved)
There are few classes used instead of cypress selector , will ask Dev to update the same ( Specially for Change button)

P.S. This PR should not be merge until Performance Issue resolved.

Updated cypress selectors and added and explicit wait for the test case when user change the school/trust(It looks like javascript is taking bit long to load the page thus cypress test need to wait for an extra ti

me )

@rdogra-nimble the performance on dev now is as good as we're going to get I'm afraid.


Add support for Position Independent Code

Signed-off-by: Kemal Akkoyun<EMAIL_ADDRESS>Fixes https://github.com/parca-dev/parca/issues/326

There is a regression, in the branch. I'm trying to find it. Thus, I've marked it as draft again.


sales_insight example

WIP and lot TBD here:

[x] Generate fake sales transcripts
[x] chunk, embed and load sales transcripts
[x] conversation with transcript API using Modal
[x] Conversation with sales transcript, UI
[x] Try to get key points out of transcripts - customer pain points, next steps
[ ] voice summary
[ ] Read sales call out loud

Still stuff missing, but merging and will pick up from here later.


Obtaining present state of a pixel

First of all, thanks for this awesome library. I am using it with an Adafruit 128x128 grayscale oled connected to a Pi Pico. It works great. Quick question: is there a way to interrogate the display, returning the present state of a specific pixel?(something akin to lcd.get(x,y)). Apologies if this is obvious, I am a noob to micropython.
Thanks again!

(or alternatively reading the state of the whole array of pixels, for further use. I can update these via display.pixel(x,y,col) followed by display.show(), but I cannot find any obvious way in the examples or the library to read rather than write. Apologies if this is noob blindness. For reasons of code optimization I would prefer to have this option to interrogate the display when needed, rather than keeping programmatic track of what was previously written to the display)

Hopefully you've found a workaround in the 5 months since this was posted, but if you're still interested, here's how you could do it.
One (hacky) way you can do this is to query the pixel value via the framebuf field that the SSD1327 library wraps around.
So if you set a pixel at (x,y) with value c this way: lcd.pixel(x,y,c), then you can get c back by calling lcd.framebuf.pixel(x,y).
Example with a Zio.cc 128x128x16 grayscale display:
>>> display.pixel(0,0,15)   # brightest grayscale value
>>> display.pixel(16,16,8)  # half brightness
>>> display.pixel(32,32,1)  # lowest brightness
>>> display.show()          # 3 pixels are displayed
>>> display.framebuf.pixel(0,0)
15
>>> display.framebuf.pixel(16,16)
8
>>> display.framebuf.pixel(32,32)
1

Using the internal framebuf object so obviously not ideal, so you could also add a method to the SSD1327 class, like this:
def get_pixel(self, x, y):
    return self.framebuf.pixel(x, y)

Hope it helps!

I did find a workaround, but with some sacrifices (had to reduce the size of my matrix, making pixels aggregates of four, to reach the processing speed intended). Your comment is now inspiring me to go back to the original purpose, using the whole 128x128 matrix. Definitely a project for Xmas break. Thanks for the excellent input!

Normally, displays like this are write-only. You can't get the pixel state back from the display.
If you maintain a copy of the display data (ie use a framebuf), then you can get the data from there instead, as demo'd above. But if you write data directly to the display, then sadly no, it's fire and forget.


Feature/speech

Issue # 28
added speech to the next and previous buttons.

I noticed I accidentally deleted, from a past commit, which Rook or Knight goes to the square if both are able.   I will put that in soon.

This seems ready to pull.   I will check into en passant.

Manually merged


Generate local svg with white background

Hi,
first, congrats for this amazing tool! :)
Since I work with dark themed VSC but the browser setting is user dependent, it would be nice to add the option to allow a white background in the locally saved SVG to always have contrast for light and dark setting of browser or VSC.
Best

I had the same problem, as a workaround you can add follow
"markdown.styles": ["markdown-styles.css"]

to your ~/.config/Code/User/settings.json then create a css file ~/.config/Code/markdown-styles.css with this content
img {
    background-color: white;
}

for a result like follow

from markdown containing svg eq

Feature implemented as of version v0.1.0, closing.

Awesome, thanks!

Awesome, thanks!


list(Architecture)[x] where x>len(list(Architecture)) crashes

list(Architecture)[x] where x>len(list(Architecture)) crashes

Fixed some time ago:
>>> list(Architecture)[10000]
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
IndexError: list index out of range
>>> ```


Package Manager usecase benchmarks for #ipfs-test-infra project

Related to #76, #77 and #79
One of the Infrastructure/IPFSaaS team's goals this quarter is to be able to test potential IPFS releases against real-world usecase benchmarks to catch any potential regressions early.
Package managers are one of those usecases, we should package up some of those benchmarks for the @ipfs/wg-infrastructure team to integrate into their test suite.

We can also explore whether we should do this work as part of https://github.com/ipfs/benchmarks //cc @alanshaw

The test scenarios we have are documented here https://github.com/ipfs/benchmarks/tree/master/tests#nodejs-and-go - happy to talk with either of you about augmenting them for your needs

There's two kinds of things to think about here:

tests - integration level tests that cover our usecases that either pass/fail
benchmarks - measure the performance of running our usecases over time and ensuring there aren't regressions

Note that @ipfs/wg-infrastructure is not working on test suites. I'm not sure the pipeline that is being setup by @jimpick or @alanshaw etc can accomodate adding TBs of data. Maybe you will need a separate pipeline. Also, worth checking if the problems seen with multiple repositories of 1TB can already be seen with test-sets of 1 GB.
The bottleneck is probably disk througput on write. This will become slower with flatfs and probably statys reasonably constant with badger (assuming enough memory). All the tests listed by @alanshaw probably grasp a lot of metrics which, if improved, will help adding large repos.
Having tracing for the "add" process is probably the first place to get an idea of what is taking longest.

The existing ipfs/benchmarks setup is using a single bare hardware minion for the tests, I believe. But I'm also doing work with https://github.com/libp2p/testlab which sets up a cluster ... we could design a test target cluster with the resources required for some heavy tests.


Compare to odo

Can someone compare and contrast intake to blaze/odo?
Do they both have similar goals i.e. ingestion of data? can they both support chunking/streaming?

(odo has not seen significant development in some time)

Yeah, I've been wanting to add that history section, since what Intake is trying to be is informed by Blaze (and odo), but not attempting to be a replacement.
Intake is trying to be extremely simple, make data cataloging easy, but also allow going very broad in terms of kinds of data plugins.  Since all of the compute is handled by existing systems (Pandas, Dask, NumPy, etc), the author of a new data source plugin has to do much less work, which will hopefully inspire more of them (and a greater variety of them, see #58).


Can you git rid of pulling new image every time?

Description:
Hi there, I am a Chinese developer located in Shenzhen City.
You guys may know the Gov use GreatFireWall to block internet connection between China and the world.
Recently docker.io and other dockers related website are blocked.
This caused a problem for our team, every time we invoke sam local function, we ended with http error. We can not access docker.io to get the newest images. :/
Expected result:
Is there any way to prevent sam local to pull the newest images?

sam local start-api --skip-pull-image

--skip-pull-image               Specify whether CLI should skip pulling down
the latest Docker image for Lambda runtime.

thankyou
On Sun, Oct 7, 2018, 02:04 Tanbouz<EMAIL_ADDRESS>wrote:

sam local start-api --skip-pull-image
--skip-pull-image Specify whether CLI should skip pulling down
the latest Docker image for Lambda runtime.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-sam-cli/issues/695#issuecomment-427594547,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABmNJpTWYlDggtH6bk6fke_lcIY7ayt1ks5uiPC-gaJpZM4XLOXC
.

--
from android

@Tanbouz is correct. You can use --skip-pull-image to force SAM CLI not to call docker to download a new image.
We have another issue for making this experience better at #529.
Closing


Improve formatting of ghc and hlint haskell linters

For ghc, it seemed that the conditional
if l:corrected_lines[-1] =~# ': error:$'
    let l:line = substitute(l:line, '\v^\s+', ' ', '')
endif

was never being reached. It's actually better to unconditionally
collapse whitespace anyway and so I simply removed the conditional
check.
For hlint I added more information about the error. This changes the
reported error from Error: Avoid lambda to something like:
 Error: Avoid lambda. Found: \ x -> foo x Why not: foo

I see no problems here. :+1: If you want to go the extra mile, it would be good to have some Vader tests to cover these functions. The Vader tests aren't too hard to write once you get your head around them. Just avoid using Then unless you are using newer versions of Vader.


add unicode encode when add ceph pool

agent cannot load Chinese words directly, so send Unicode then encode it in agent.
for https://github.com/zstackio/issues/issues/2527


luarocks: 3.0.4 -> 3.1.3

Motivation for this change
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[X] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"- I ran this command but I don't think all Lua packages were rebuild so I'm not sure about this one.
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Assured whether relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Also: Updated darwin.patch for new version of the patched file.

It seems this is already updated on master?

This fails for me non-nixos linux x86_64
building '/nix/store/a79j22ib4b1ynd5qn5f2vi4hfimid4k6-luarocks-nix-3.1.3.drv'...
unpacking sources
unpacking source archive /nix/store/bdia5298mxqqxgz7i0pv0z70821629gh-source
source root is source
patching sources
applying patch /nix/store/djx2aqx0rdwbp4kaw45ga8xgd9759dpj-darwin-3.0.x.patch
patching file src/luarocks/core/cfg.lua
Hunk #1 FAILED at 425.
Hunk #2 succeeded at 436 (offset 2 lines).
1 out of 2 hunks FAILED -- saving rejects to file src/luarocks/core/cfg.lua.rej
builder for '/nix/store/a79j22ib4b1ynd5qn5f2vi4hfimid4k6-luarocks-nix-3.1.3.drv' failed with exit code 1

Thanks for catching that @worldofpeace , @Shados was faster then me :) Closing.

Yeah we can sometimes have competing PR's since people round here are pretty attentive.


Typo in ResponseValidator._processClaims error

The following mentions "access_token" whereas it should be "id_token" :

ResponseValidator.js#L154

"ResponseValidator._processClaims: sub from user info endpoint does not match sub in access_token"

ResponseValidator.js#L155

"sub from user info endpoint does not match sub in access_token"

possibly. i'll have a look.

Fixed, albeit very late. Thanks.

Fixed, albeit very late. Thanks.


[ADLSinASE] Can not find value type when create new vertex.

Build:
StorageExplorer-ia32
Repro Steps:

Open Explorer
Find graph tab then click it.
Click New Vertex button.
Click Add Property

Result:
Can not find and seclect value type.

Dupe of #1237


Defining an "otherwise" route fails to redirect

Steps to reproduce and a minimal demo of the problem
http://plnkr.co/edit/b6dkMFAIRpNHepDDP4l0?p=preview
Open plunk -- notice that path is not redirected to '/home'
Current behavior
Per a previous gitter conversation, you can specify a "default" route by using:
 { path: ‘/**’, redirectTo: ‘/home’ }

However this appears to be broken in 1.0.0-beta.1
Expected/desired behavior
A route path could be added as a catch all, and force a redirect to specified path. Similar to the "otherwise" functionality from angular-ui-router.
https://github.com/angular-ui/ui-router/blob/1e46c8d/src/urlRouter.js#L102
Other information
Workaround is available by redirecting the '/' route.
export const routes = [
  {
    path: '/',
    redirectTo: '/home'
  },
  {
    path: '/home',
    component: Home
  },
  {
    path: '/away',
    component: Away
  },
  {
    path: '/level1',
    component: Level1,
    index: {
      component: Level1Home
    }
  },
  {
    path: '/**',
    redirectTo: '/home'
  },
];

Confirmed. You said as of beta 1, is this a regression in your app?

I haven't used previous versions, just trying to be explicit with the version I was using.

I had same issue


Update to latest ELK

Update to latest stable Elastic 5.2.1

[x] Udate to latest Elastic - check elasticdsl queries in web interface
[x] Update to latest Logstash
[x] Update to latest Kibana

Note: create new branch from master

Update required adding RAM to consumer, because elasticsearch needs to allocate 2GB of RAM.

Ok, agree, please update the vagrant file and the README file (Requirements) accordingly.

Found error during deployment:
TASK [elk : Restart Elasticsearch] *********************************************
fatal: [consumer]: FAILED! => {"changed": false, "failed": true, "msg": " * Stopping Elasticsearch Server\n   ...done.\n * Starting Elasticsearch Server\n[2017-03-13T10:33:45,317][WARN ][o.e.c.l.LogConfigurator  ] ignoring unsupported logging configuration file [/etc/elasticsearch/logging.yml], logging is configured via [/etc/elasticsearch/log4j2.properties]\n   ...fail!\n"}

Please fix it

fixed, closed


about iBims dataset

We plan to refer to your paper, but in the part of testing iBims data set, we found that your inference code simple_inference.py did not include the calculation methods of pe and dbe indicators. And we noticed iBims data set the official download link https://dataserv.ub.tum.de/index.php/s/m1455541 seems to be no longer available, don't know whether you can provide the iBims backup data set.
Best wishes

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Thank you for your reply. We clicked on the link, which seems to be the download of a compression program. Does it have anything to do with our problem?


Please help to add About page into the nav bar

Hi Varun,
Appreciate your effort to create this simple and fast hugo theme. I am writing this to ask ABOUT page into a nav bar, this is not an issue. Thank you for your hard work.

@cloudmiracle Support for custom menu is now added
Please refer to exampleSite/config.toml for details on how to add a new menu

thank you for your effort @apvarun. I can see the External Item on the menu. Can you help to keep the logo, nav bar and logo with SPA concept (dont open new link) in your free time. Thank you!

thank you for your effort @apvarun. I can see the External Item on the menu. Can you help to keep the logo, nav bar and logo with SPA concept (dont open new link) in your free time. Thank you!

Is there any progress about this? I want to create a about/team page as well, but the nav bar has to be there.

@cloudmiracle Could you rephrase your need here? I don't understand the SPA part in your query.

Closing this due to inactivity


Add back Grunt and set it to default deploy

This Pull Request has been closed by GitConsensus.
Vote Totals

Yes
No
Abstain
Voters

2
3
0
5

Vote Breakdown

User
Yes
No
Abstain

tedivm
✔

sparr
✔

lurr

✔

ags131

✔

TheRealMaxion

✔

Vote Results

Criteria
Result

Has Quorum
True

Has Votes
False


[Win 10] Build failure compiling v2 branch

Still getting the same error:
Via/modules/sync3/sync3_modes.cpp: In member function 'void ViaSync3::handleButton2ModeChange(int32_t)':
Via/modules/sync3/sync3_modes.cpp:18:15: error: 'scales' was not declared in this scope
   18 |  numerators = scales[mode].numerators;
      |               ^~~~~~
make: *** [../../compile.mk:68: build/./Via/modules/sync3/sync3_modes.cpp.o] Error 1

For some reason updating the v2 branch on my local repo with git pull didn't update Via submodule properly.
Fixed my end.


Add tree endpoint and parent children relationship management to material

addresses the last of https://vizzuality.atlassian.net/browse/LANDGRIF-189 I think

Adds ability to pass parentId when creating a material
optionally allows requesting children when loading materials list
Adds new /material/tree endpoint to load data in a tree-like format
Adds optional depth limiter on the tree endpoint (implementation will be reviewed but API contract wont change)

Thanks @tiagojsag, this is brilliant.
I have some questions that I'd like to clarify:

Swagger shows that depth query is required for the request, which is not accurate.

Commented out tests: Why are these tests commented out? As an example of possible use cases regarding to this new functionality?


Fix doc comment for switcher_current_thread.

As an aside: do we have a convention for return values from library calls and cross compartment calls? This library call returns false on failure, which is fine, but that is not a good idea for compartment calls since they return -1 on forced unwind. This could cause confusion if we ever change a library call into a compartment call or vice versa, so wouldn't it be better to be consistent?

I also notice that the doc comment for trusted_stack_has_space references trusted_stack_index, which doesn't exist.


build: Update Vulkan-Headers to 1.2.201

CI Vulkan-ExtensionLayer build queued with queue ID 9157.

CI Vulkan-ExtensionLayer build # 101 running.


Check for unsupported connection headers.

Checks for unsupported "connection" headers in SDK authorization policy. If found, returns an appropriate invalid policy error.

This change is

We could choose to let them through the SDK API and just not match them, but given how they are defined in the API this seems fair.

Exactly, so we don't want to do the policy evaluation here, instead since its mentioned in the schema, usage of these headers would mark that SDK policy as "invalid".

:scheme and grpc--prefixed are disallowed (NACKed in RBAC) in both equivalently

So by RBAC here, I am assuming you are referring to xDS path, and this validation failure on finding :scheme or grpc- prefixed header would take place before the call to RBAC Engine's evaluate functionality?


Replace obj with discriminated unions for better type safety and type discoverability

Some properties are bound like obj in the json file to allow implicit conversion just like we would do in XAML.
E.g. Image.Source can be set as a string or as an ImageSource.
Or GridRowDefinition can be either "*" or GridLength.Star
Same with View.Margin
But when using such types, IntelliSense only shows that the property is obj.
Developers don't know about the implicit conversions (if they didn't check the samples) and must look into the documentation of Xamarin.Forms to know what type exactly is allowed.
Wouldn't changing those obj to discriminated unions be better?
It would make typing explicit to developers, while allowing the compiler to check if there's nothing funny going on.
And cherry on top, converters would output warnings if we're missing a case (which is not the case currently with type matching)
What do you think?

I think F# should have type-tagged union types (string|ImageSource) (erased to obj).  I am hoping we can add those in F# 5.0, partly driven by exactly this question

That would be great!
Is it something that will come in the foreseeable future (e.g. ~6 months)?
Or should we do something in the meantime and replace later once out?

I think it will take longer.  I feel using "obj" is likely ok for now, as people nearly always do view elements via cut-and-paste snippets, but I know it's not ideal.

Closing due to #341


Support for identifiers of things in CSVW

Currently, there is no "iri" column generated for identification of resources. This is similar to #94 , only for CSV. It also needs to be discussed with @sstenchlak because it is regarding the overall treatment of identifiers (IRIs).

I have added a provisional column for identifiers, because I needed that for foreign keys. We should figure out some final form for the identification.


Bad link - Use The Web API with Dynamics 365 Metadata

Describe the bug
On this page https://docs.microsoft.com/en-us/dynamics365/customer-engagement/web-api/about?view=dynamics-ce-odata-9
Link  - Use The Web API with Dynamics 365 Metadata in the image below has href
assetid:///a0edc029-c6db-48ac-9538-b0270fe94440.
It probably should be linking to
https://docs.microsoft.com/en-us/powerapps/developer/common-data-service/webapi/use-web-api-metadata

To Reproduce
Steps to reproduce the behavior:

Go to https://docs.microsoft.com/en-us/dynamics365/customer-engagement/web-api/about?view=dynamics-ce-odata-9
Click on  - Use The Web API with Dynamics 365 Metadata

Expected behavior
Link navigation
Desktop (please complete the following information):

OS: Windows
Browser [e.g. chrome, safari] Chrome
Version [e.g. 22] 83.0.4103.61

@tonyhallett Please move this issue to https://github.com/MicrosoftDocs/dynamics365-docs-odata-apis/issues


Emulator doesn't read most keyboard symbols

Using the emulator, it doesn't register many non-letter keys like "+", "[", "]", "{", "}". I pressed every key, but none give me a "+".
The real thing works OK, however.

Same here; IU guess this is really an issue for the emulator:

https://github.com/Zeal8bit/Zeal-WebEmulator

But it makes changing drives impossible, due to the inability to enter :.

My bad. It fails with Mozilla Firefox, my main browser, so much loaded with extension to block trackers, cookies, javascript and everything under the Sun.
It works with Chromium, my alternative browser with no extension added.

My bad. It fails with Mozilla Firefox, my main browser, so much loaded with extension to block trackers, cookies, javascript and everything under the Sun.
It works with Chromium, my alternative browser with no extension added.

New information:

Using Linux Mint 21.2

It fails with my main browser, Mozilla Firefox with anti-tracker, no-script, anti-cookies, uBlock, and others extensions for protection.

I tested with Chromium, pristine with no extensions, and the problem almost vanishing, but still:

some symbols are missing, like colon :
have to use "Ctr + number key" instead of "Shift + number key" to produce #, $, % ...
some chars, like {, appears on various places.
the two keys at right of M key produces < and > when pressed. To get dot and comma have to use Shift.

My keyboard:


Contact Section

create a contact me section with email, linkedin, github, socials or number

-center using bootstrap


mbsync: extra config goes at end of .mbsyncrc, should go at beginning

Cause here.
The issue is that the .mbsyncrc format is section-based; if you have any accounts/channels/etc., the given configuration will not be global, but rather happen to go in the last account/channel defined.

Good point, I have tried sorting this out in the above commit.

Looks perfect, will verify that the change works but assume it did if you don't hear from me!


Is it possible to pass token via request header

I need to make post request with body is json format so that I think passed the token on the header is better.
Is your tool already support this way ?

Great question, try it now!
https://github.com/josephspurrier/csrfbanana/commit/b30971a69aea585b5445f49d92ffb8e8fdf13367


Remove position: relative from grid columns

Prompted by #25254, I dug into our Git history to see why we add position: relative to our grid classes and mixins. Turns out, I have no idea and added it to v3 in a commit that fixed an unrelated issue—see https://github.com/twbs/bootstrap/commit/0f17f9b7f7fe15bb4ce6301caa2600bfd30e63d7. This change would restore a default, and implicit, position: static, allowing you to opt into whatever positioning you need.
I don't see this as a breaking change—we change properties and values as needed—but I could see this being confusing and unnecessary either way.
Thoughts from folks?
Closes #25254.

Todos if moving forward

[ ] Update docs snippets around mixins
[ ] Verify docs don't mention relative position

Going to have to punt this to v5 because it breaks form validation tooltip positioning. We currently rely on that to align the form feedback message.


CANNOT SWITCH TO DARK MODE

when I touch apply dark mode, it revert back to light mode. even I restart the application
I touch apply and switch like this

after I touch X on setting window, it back to light mode

even I restarted the app, no changes

Changes are saved only when you press the OK button at the bottom.
Please refer to the comment on issue #11

Applied to v 0.3.6


[AVM Module Issue]: DCR 'kind' should allow being undefined to support both Windows and Linux.

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Bug
Module Name
avm/res/insights/data-collection-rule
(Optional) Module Version
0.2.0
Description
Property Kind is optional but defined with default value Linux in the template.
This property should be optional without a default value to support xPlat (all) scenarios.

(Optional) Correlation Id
No response

@jagiraud thanks for reporting, I will take care of it

@jagiraud It turns out to be more complicated because the Resource provider does not accept the value kind: null or kind: '', but expects the "kind" to not be present in the module at all when creating a cross-platform Data Collection Rule. I've created a ticket with the product group. I will keep the issue updated

@jagiraud The update is implemented in the PR (see above) in review. Once the module is published, you can deploy a cross-platform DCR using the param dataCollectionRuleProperties.kind: 'All'


upgrade gomodule

just a simple upgrade to latest

I read diffs carefully before upgrading depenencies.  I'll upgrade the dependencies when I have time to read the diffs.


Add redirect method from @pcdinh

@pcdinh seems to have abandoned #118 so I've picked it up and added some more tests around the None bug that was mentioned.

Yay for pytest fixtures!


align asr output and llm input without using orchestrator

Description
align asr output and llm input without using orchestrator, allow k8s to use directly
Issues
340
Type of change
List the type of change like below. Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would break existing design and interface)
[x] Others (enhancement, documentation, validation, etc.)

Dependencies
None
Tests
existing CI

We need a e2e test_asr.sh for this case.

@chensuyue https://github.com/opea-project/GenAIComps/blob/main/tests/test_asr_whisper.sh already there. The CI just skips that test.

@chensuyue https://github.com/opea-project/GenAIComps/blob/main/tests/test_asr_whisper.sh already there. The CI just skips that test.

This is for comps/asr/whisper/Dockerfile,  not for comps/asr/Dockerfile.


[Composer] Shutting down of the Composer closes the entire browser window

Description:
When a browser window instance is automatically started as a result of starting the composer, shutting down of the composer closes that entire window. It is inconvenient as all the tabs which were opened in that browser window get closed. However, this issue cannot be seen if there is an already opened browser window instance when the composer is started.
Steps to reproduce:

Make sure there is no running instance of the default browser.
Run the composer using the terminal.
Go to the browser window which will be automatically opened by the composer and open new tabs within that window.

Affected Versions:
0.975.1
OS, DB, other environment details and versions:
OS - Ubuntu 17.10, Ubuntu 18.04
Browser - Mozilla Firefox, Google Chrome
Suggested Labels (optional):
Bug

@Gathika94 Could you mention the browser versions which this occurred?

Experienced the mentioned issue in Chrome Version 67.0.3396.99 (Official Build) (64-bit) @DimalChandrasiri

@DimalChandrasiri  The browser version is Mozilla Firefox Quantum 61.0 (64-bit).

Was able to reproduce this issue in ubuntu 18.04. Since the composer is moving to a standalone application IMO this wouldn't be a problem in the future. As for a suggestion, could you try opening up the composer using the nohup command?
nohup sh composer

@DimalChandrasiri  I tried with nohup composer command, but this issue was still there. If we have an already opened instance of the default browser when the composer is started,  shutting down of the composer does not close the browser window. When there is no running instance of default browser  at the time of starting the composer,  even if the composer is started using nohup composer command, shutting down of composer closes the entire browser window. Agree that this would not be a problem if composer is moving to a standalone application.

@Gathika94 please try with the version 0.980 latest release? if not could u try with nohup composer & to separate the browser from the console.

@Gathika94 Were you able to try out @DimalChandrasiri 's suggestion. Please note that 0.981.0 is released therefore you can try on that.

@Manuri - Yeah, I tried with 0.981.0. The ballerina composer started as a standalone application. Therefore, I believe this issue is resolved.

Thanks @Gathika94. Closing the issue based on your feedback.


Change search script include from HTTP to HTTPS

This PR changes some script includes in legacy templates from HTTP to HTTPS.
This is a followup to #2455.
Changes

Changed references from http://search.usa.gov to https://search.usa.gov.

Testing

Passes all unit tests.
Run an HTTPS localhost (./runserver ssl) and then navigate to https://localhost:8000/paying-for-college/ (requires the college-costs app). Note there are no mixed-content warnings.

Review

@sebworks @cfpb/cfgov-frontends

Notes

This will prevent mixed-content warnings when cf.gov moves to HTTPS itself.

Checklist

[X] Changes are limited to a single goal (no scope creep)
[X] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[X] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

👍


Some suggestions

so this does a few things

uses TargetFrameworks instead of TargetFramework` since it is more future proof fro when people multi target
replaces existing nuget references with PackageReference

When i spiked this i wasnt really intending it to be merged, it was just some stuff i wanted. so feel free to pulling any bit u like or not :)

Thanks for the PR (and sorry about the delay in getting back). The changes around replacing nuget references look interesting and i'll work on pulling them in.
As for TargetFramework vs TargetFrameworks - while it's true that TargetFrameworks sets you up for multi-targeting in the future, it has downsides. Primarily, for all projects with TargetFrameworks, msbuild will evaluate the project twice, once to get the set of TFs and then once for each TF. Apart from the perf downside, it also add some complications around extending the project with custom targets. So I don't think TargetFrameworks is a good default.


TODO: Bump compatibility for Foundry VTT V9

Apart from #3, Bullseye NG seems to work fine with the new Foundry VTT version.
Blocked by #3, though.

Done with v1.2.0


Update catalog mgmt service

PR summary
This PR contains the update for Catalog Management SDK after the Api recent changes.
Important to note the following:

the tests where I had issues (user is not granted, no suitable data or lack my domain knowledge) has comments about the details, even though it is ugly but hopefully informative to the team, so they can find solutions easily
Integration test data and parameters are conherent across SDKs except the cases where  the language name was used
as the amount of catalogs grows the listCatalogs method will fail more and more due to that the database query has a limit on the result set. Many times the newly created catalog is not in the result. It is discussed with Jon. In the java tests checking whether the list contains the newly created catalogs is commented out as it stops the execution.
the order of the test cases are the same in layout and execution* as the other 3 SDK have.
I run Optimize imports on the Catalog Mgmt module and unused models with their tests are also removed

*not really, but the solution mimics the same, having exactly the same execution order would be an unnecessary and enormous effort which doesn't worth it
About coverage:

except 4 operations all operation has at least one test exercising either the happy path or a negative case
I had a discussion with the team about the 4 operations (GET/PUT/DELETE Offering Instance and Replace Offering Icon which is disabled)

The special cases:

Go and Java SDK returns 200 when we'd like to delete a catalog which doesn't exist, Node and Python returns 404, I couldn't figure out why

PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

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


Fix minor typos

Fix two minor formatting errors.

Hi @mcollier, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (mcollier). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@mcollier it looks like someone already fixed these typos, if not can you please resolve the merge conflicts and I will merge it for you.

@mcollier I went ahead and fixed the conflict and merged it in another PR. Thank you for your contribution to this topic. We appreciate your help improving the quality of Azure docs.


:up: playwright to v1.27.1

Since @bpasero is ooo, @jrieken might be a good reviewer.
The 1.27 release implements a workaround for the issue that the browser downloads where not working as expected which yielded to "executable not found" when launching the browser.
Issue: https://github.com/microsoft/vscode/issues/161039
Upstream issue: https://github.com/microsoft/playwright/issues/17394

There is still a problem. you helped me

for process reasons thiscontinues in https://github.com/microsoft/vscode/pull/164238

OK is there a way

I downloaded version 1.27.1 and it still doesn't work. I see from the logs why it is not working


Allow the search index class to be provided by the user in index.

Defaults to SearchIndex otherwise.
Could be a very simple fix for #96.
(If this kind of change is desirable I'll be more than happy to supplement it with documentation updates.)

I was actually going to do something like this. In fact it can solve sooo many problems such  fuzzy search (https://github.com/twitter/typeahead.js/issues/525) which in fact I did using a library called Fuse.js. I would make a pull request, but currently this repository has been inactive for quite some while!

 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.

Whoop! 5 years and counting!


handle only valid files

additional fix for #5590

It seems that main problem is pileup still getting invalid files from dbs.
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/WMSpec/Steps/Fetchers/PileupFetcher.py#L56
I updated the patch to filter out invalid files. However, this is a hacky way to do it. I think dbs api should return valid files only by default and invalid files if the fag is set. However, that api changes could break other code, we can have default value set to return including invalid block as it is. Need to discuss with Yuyi. @yuyiguo

Also need to check returns all the files (invalid ones) ?
listFileParents
listFileLumis
listFileSummaries

listFileLumis(validFileOnly=1, ...) return valid file only.
listFileSummaries(validFileOnly=1, ...)  return valid file only.
listfileParents (logical_file_name=Child-lfn) if Child-lfn is valid, all prents have to be valid otherwise, there must be something wrong in the DB. If Child-lfn is invalid, no call needed.

@ticoann , it seems this patch is really solving the invalid files issue.
In our 2 test workflows, I have not seen any failure related to the invalid/bad PU file:
/store/mc/TP2023SHCALGS/MinBias_TuneZ2star_14TeV-pythia6/GEN-SIM/DES23_62_V1-v1/20000/4EA851D4-776C-E411-9D45-842B2B186684.root
IMO we can now abort those test workflows and already push it to production. What do you think?

Thanks Alan, go ahead please.


Arrow animation glitch

With the rework of the card in v0.2.0, the animations for the arrows aren't working as expected. The arrows are somewhat jumpy on the animation instead of smooth.

Fixed with release v0.2.2.


Get information about linters being run

There are various states ALE can be in that a user might want to check.

ALE is disabled
ALE is disabled for this filetype
ALE is enabled for this filetype but there are either no linters specified or none of the linters specified are available.
ALE is enabled for this filetype, there are linters available but there are no diagnostics. E.g. 0 errors, warnings, info.
Usual stuff, there are diagnostics to be shown.

I need 3. for deciding whether to show linting information in the statusline or not. It is different to show that there are 0 errors than to show that no checks are currently being run.

Also is there another way of knowing if ALE is disabled for this filetype other than looking at g:ale_filetype_blacklist (which is not documented btw)

Did you try
:ALEInfo
?

The filetype blacklist isn't documented because I don't support anyone changing it.

Please describe what you hope to achieve in more detail.

@w0rp I just read the documentation for g:ale_pattern_options, I don't know how did I miss that. This covers disabling ale for certain filetypes.
What I want to achieve is to display useful information in the statusline. No checkers being run is not the same as no errors present. Under the current system, I have no option but to display "0 errors", which is ambiguous.

Have a look at this. https://github.com/w0rp/ale/pull/1203 You can use autocmd commands to do whatever when ALE starts or stops linting.

@w0rp Yes that does it. Thank you


Caller Identification (CALL.Options.disclose_me | bool := true)

I tried to find any properties for Caller Identification feature but not found. Do you have any plans to support this?

I see this functionality in fork https://github.com/ichrome/jawampa/commit/f792f320e91c8312e3c2033ba2a31930ca29f4e0

Yes, this is now merged.


No source for code: '/x/y/z/'. (couldnt-parse)

Describe the bug
When I run the coverage xml -i command, I get a number of errors of the form:
/usr/local/lib/python3.10/dist-packages/coverage/report.py:87: CoverageWarning: Couldn't parse '/main/srhaque/...not an actual file': No source for code: '/main/srhaque/...not an actual file'. (couldnt-parse)
  coverage._warn(msg, slug="couldnt-parse")

where the path printed (twice) has a valid dirname(path), but the basename(path) of has values like Works id, Units, Date, bankdetail.sort_code. These names are not the names of any files, but do exist in the codebase as strings.
To Reproduce
How can we reproduce the problem?  Please be specific. Don't link to a failing CI job. Answer the questions below:

What version of Python are you using? 3.10
What version of coverage.py shows the problem? 6.4.1 The output of coverage debug sys is helpful.

$ coverage debug sys
-- sys -------------------------------------------------------
               coverage_version: 6.4.1
                coverage_module: /usr/local/lib/python3.10/dist-packages/coverage/__init__.py
                         tracer: -none-
                        CTracer: available
           plugins.file_tracers: -none-
            plugins.configurers: -none-
      plugins.context_switchers: -none-
              configs_attempted: .coveragerc
                   configs_read: /main/srhaque/Innovatieltd/source/.coveragerc
                    config_file: /main/srhaque/Innovatieltd/source/.coveragerc
                config_contents: b'# .coveragerc to control coverage.py\n[run]\nbranch = True\nparallel = True\nomit =\n    /usr/*\n    paiyroll/migrations/*\nconcurrency =\n    thread\n    multiprocessing\n'
                      data_file: -none-
                         python: 3.10.4 (main, Apr  2 2022, 09:04:19) [GCC 11.2.0]
                       platform: Linux-5.15.0-37-generic-x86_64-with-glibc2.35
                 implementation: CPython
                     executable: /usr/bin/python3
                   def_encoding: utf-8
                    fs_encoding: utf-8
                            pid: 3840835
                            cwd: /main/srhaque/Innovatieltd/source
                           path: /usr/local/bin
                                 /usr/lib/python310.zip
                                 /usr/lib/python3.10
                                 /usr/lib/python3.10/lib-dynload
                                 /usr/local/lib/python3.10/dist-packages
                                 /usr/lib/python3/dist-packages
                    environment: HOME = /main/srhaque
                   command_line: /usr/local/bin/coverage debug sys
                sqlite3_version: 2.6.0
         sqlite3_sqlite_version: 3.37.2
             sqlite3_temp_store: 0
        sqlite3_compile_options: ATOMIC_INTRINSICS=1, COMPILER=gcc-11.2.0, DEFAULT_AUTOVACUUM,
                                 DEFAULT_CACHE_SIZE=-2000, DEFAULT_FILE_FORMAT=4,
                                 DEFAULT_JOURNAL_SIZE_LIMIT=-1, DEFAULT_MMAP_SIZE=0, DEFAULT_PAGE_SIZE=4096,
                                 DEFAULT_PCACHE_INITSZ=20, DEFAULT_RECURSIVE_TRIGGERS,
                                 DEFAULT_SECTOR_SIZE=4096, DEFAULT_SYNCHRONOUS=2,
                                 DEFAULT_WAL_AUTOCHECKPOINT=1000, DEFAULT_WAL_SYNCHRONOUS=2,
                                 DEFAULT_WORKER_THREADS=0, ENABLE_COLUMN_METADATA, ENABLE_DBSTAT_VTAB,
                                 ENABLE_FTS3, ENABLE_FTS3_PARENTHESIS, ENABLE_FTS3_TOKENIZER, ENABLE_FTS4,
                                 ENABLE_FTS5, ENABLE_JSON1, ENABLE_LOAD_EXTENSION, ENABLE_MATH_FUNCTIONS,
                                 ENABLE_PREUPDATE_HOOK, ENABLE_RTREE, ENABLE_SESSION, ENABLE_STMTVTAB,
                                 ENABLE_UNLOCK_NOTIFY, ENABLE_UPDATE_DELETE_LIMIT, HAVE_ISNAN,
                                 LIKE_DOESNT_MATCH_BLOBS, MALLOC_SOFT_LIMIT=1024, MAX_ATTACHED=10,
                                 MAX_COLUMN=2000, MAX_COMPOUND_SELECT=500, MAX_DEFAULT_PAGE_SIZE=32768,
                                 MAX_EXPR_DEPTH=1000, MAX_FUNCTION_ARG=127, MAX_LENGTH=1000000000,
                                 MAX_LIKE_PATTERN_LENGTH=50000, MAX_MMAP_SIZE=0x7fff0000,
                                 MAX_PAGE_COUNT=1073741823, MAX_PAGE_SIZE=65536, MAX_SCHEMA_RETRY=25,
                                 MAX_SQL_LENGTH=1000000000, MAX_TRIGGER_DEPTH=1000,
                                 MAX_VARIABLE_NUMBER=250000, MAX_VDBE_OP=250000000, MAX_WORKER_THREADS=8,
                                 MUTEX_PTHREADS, OMIT_LOOKASIDE, SECURE_DELETE, SOUNDEX, SYSTEM_MALLOC,
                                 TEMP_STORE=1, THREADSAFE=1, USE_URI

What versions of what packages do you have installed? The output of pip freeze is helpful.
What code shows the problem?  Give us a specific commit of a specific repo that we can check out. If you've already worked around the problem, please provide a commit before that fix.
What commands did you run?

Coverage is run programmantically via the API on a set of code including Django, Celery, subprocesses.
coverage combine -q
coverage xml -i
Expected behavior
The XML file should be produced without needing the "-i" option.

Additional context

The overall coverage result seems to be missing some covered lines of some files, but is generally plausible.
The coverage.xml file contains none of the strings I mentioned, suggesting this is related to file scanning.
I tried debugging, and can see the bad entries being read from the database at https://github.com/nedbat/coveragepy/blob/2cf1981a06ac597aacf826b56db0d50a2ee0ea0b/coverage/sqldata.py#L295
But I have not been able to determine where (and thus why) the database entry is created.

I could probably track the issue down if I could be helped on this point.

Hi this seems a continuation of #1318.  Can you give me a way to reproduce the problem?
Typically, phantom file names like this are due to dynamically compiling code.  The compile() builtin takes a "file name", but when there is no actual file, people create a fake one based on something about the origin of the code.  Perhaps your project is also doing this?
Coverage will ignore a file name if it starts with an angle bracket: <.  This is a convention used by many people when dynamically compiling code.
As an example, see #1393.

I had completely forgotten filing #1318! At any rate, the big difference from then is that I'm now able to get somewhat plausible coverage numbers having hacked around the Celery fork-vs-spawn issue (see #689).
You are indeed correct, I do use compile(), and the filename is exactly the strings I cited. I added the "<" prefix you suggested and the errors are gone!


[1.20.1 Forge] Crash with MCA (minecraft comes alive) mod (conflict)

Describe the bug
Crash upon joining server, likely due to initial screen conflict between the mods
To Reproduce
Steps to reproduce the behavior:

Run with https://www.curseforge.com/minecraft/mc-mods/minecraft-comes-alive-reborn/files?version=1.20.1
Join server
Crash

Version
MCA Version, Minecraft version, Modloader
minecraft-comes-alive-7.5.3+1.20.1-universal
tried on latest as well
minecraft-comes-alive-7.5.8+1.20.1-universal
1.20.1 Forge 47.1.3
Crash log:
crash-2023-09-12_21.27.31-client.txt

Issue may actually be with Radium, will test again soon

closed - issue was with a third mod


Enhance Date64 type documentation

Which issue does this PR close?

Closes #5288
Rationale for this change

What changes are included in this PR?

Update docs of Date64 to hopefully clear up any confusion on its details.
Are there any user-facing changes?

I think doc update is sufficient to close the issue, can add in optional validation later if there is a use case for it


Support adding files to hierarchy

Why was this change made? 🤔
Allow users to work with items that have file hierarchies.
How was this change tested? 🤨
⚡ ⚠ If this change involves consuming from or writing to another service (or shared file system), run integration test create_object_h2_spec.rb and/or test manually in [stage|qa] environment, in addition to specs. ⚡
Unit

Some early thoughts:

Looks like we need a rebase now (my PR was merged into main).
I wonder if we should remove the labels for the description and the hide checkbox since when it is rendered in the new list view, it is inline with the existing column that already says "hide"? See below:

If we do 2, we may need to split the component since the ZIP dropzone would still need those labels.

Uploading Screen Recording 2023-01-24 at 4.02.46 PM.mov…

https://user-images.githubusercontent.com/588335/214419391-77ed6997-088e-445d-a803-9ffdf2f81ada.mov


Swift 2.0 compatibility

Hi,
I updated your code to work with Xcode 7 beta 5. Test are passing. Feel free to pull these changes if you want them :) Other projects seem to use a swift-2.0 branch to facilitate people using their Pods with the Xcode beta and not get mixed up with the latest stable version.
Cheers.

Thanks! :+1:

@jgarth Sorry for my mistake. I've added a new branch swift-2.0, so would you create a new pull request to the branch?

Hey @iCodesign,
I can't create a new PR because my commits are already in the swift-2.0 branch :) You can just git revert your commit 45e85cc3378b046f085bb29c8fe44b31196fb5f3 (the one that reverted the changes in the first place) and so the changes will be in the branch again. :+1:

@jgarth Done!


[2.0 Alpha] [bug] [New scene system] Changing name of node that is a part of the base scene duplicate this node instead

If you change name of the node that is part of the base scene, save inherited scene close it and reopen then this node will be duplicated (with new name). I know that there was no plans for allowing to change these names, but this behaviour don't seems to be expected as well.
Also don't know if call it a bug or a feature (since that might be handy) but for sure it feels like not planned behaviour.
To recreate:

Create scene from option "new inherited scene"
Change name of one of the nodes from base scene
Save and close scene
Reopen closed scene

Quick 40s wid that show this process: https://youtu.be/zqDUXy39d80

uh.. it should not allow you to rename.. i totally missed that...
On Sat, Oct 17, 2015 at 6:13 PM, kubecz3k<EMAIL_ADDRESS>wrote:

If you change name of the node that is part of the base scene, save
inherited scene close it and reopen then this node will be duplicated (with
new name).
Don't know if call it a bug or a feature (since that might be handy) but
for sure it feels like not planned behaviour.
To recreate:

Create scene with from option "new inherited scene"
Change name of one of the nodes from base scene
Save and close scene
Reopen closed scene

Quick 40s wid that show this process: https://youtu.be/4eTSSHn2EmE
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/2615.


.Net Framework vs .Net Core might be confusing

"Note
Instructions referring to ApplicationInsights.config are only applicable to apps that are targeting the .NET Framework, and do not apply to .NET Core applications."
With .Net 6, this might be confusing and need a clearer explanation.
Is targeting .Net 6 considered targeting ".Net Framework"?
You can target a .Net Core Application to .Net 6.

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

ID: b189efce-ce4e-f8f8-ecf8-8803a18f9b65
Version Independent ID: aec54012-dae3-ce18-0f8c-30749c46a79c
Content: Azure Application Insights for Console Applications - Azure Monitor
Content Source: articles/azure-monitor/app/console.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @AaronMaxwell
Microsoft Alias: aaronmax

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

Thanks for your dedication to our documentation. The note regarding .NET Core's compatibility with the .config file has been simplified. #please-close
https://github.com/MicrosoftDocs/azure-docs-pr/pull/212520


Detect if Get-SFTPFile failed

I am being asked to delete files on the remote host after get. How can I know if Get-SFTPFile failed? I am not able to catch the exception. In this case, I am specifying a filename that I know does not exist on the remote host.
try {
    Get-SFTPFile -SFTPSession $Session `
        -NoProgress `
        -RemoteFile "/DATA/$LinuxPhaseDir/extracts/RX_340B/$Channel/abc.txt" `
        -LocalPath $LocalPath
}
catch {
    "failed to retrieve $file.FullName"
}

===
[DBG]: PS M:\DW_Devl\FileTransfer\340b>>
Get-SFTPFile : /DATA/PHS_UAT/extracts/RX_340B/Sentry/abc.txt does not exist.
At M:\DW_Devl\FileTransfer\340b\atest.ps1:80 char:13
+             Get-SFTPFile -SFTPSession $Session `
+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (SSH.SftpSession:SftpSession) [Get-SFTPFile], SftpPathNotFoundException
    + FullyQualifiedErrorId : /DATA/PHS_UAT/extracts/RX_340B/Sentry/abc.txt does not exist.,SSH.GetSftpFile

Because Get-SFTPFile does not produce a terminating error when the file is not found, -ErrorAction Stop must be used in order to catch it. Then, the error will be caught. In this case, being in a loop to get all files, the error needs to be cleared in order to continue.
foreach ($file in $files) {
    if ((-not $file.IsDirectory) -and ($File.FullName -match $FilenameFilter)) {
        $localfilename = $LocalPath + $file.Name
        if (Test-Path $localfilename) { Remove-Item $localfilename }

        try {
            Get-SFTPFile -SFTPSession $Session `
                -NoProgress `
                -RemoteFile $file.FullName `
                -LocalPath $LocalPath `
                -ErrorAction Stop
            Remove-SFTPItem -SFTPSession $Session -Path $file.FullName
            $FileCount++
        }
        catch {
            "failed to retrieve $file.FullName"
        }
        finally {
            $error.Clear()
        }
    }
}

Thanks to Carlos Perez for pointing me in the correct direction.


Clarify how failOnNonReproducibleResolution() should work if constraints with ranges are published

Issue type
Missing information
Problem description
Creating this as documentation issue as I am not sure what the expected behavior of failOnNonReproducibleResolution() is in this case.
The problem came up when using latest versions of Spotbugs. In its metadata, it publishes a dependency with a clear requiredVersion:
Inside dependencies blocks:
        {
          "group": "org.apache.logging.log4j",
          "module": "log4j-core",
          "version": {
            "requires": "2.22.0"
          }
        },

In addition it publishes constraints with ranges. In dependencyConstraints block:
        {
          "group": "org.apache.logging.log4j",
          "module": "log4j-core",
          "version": {
            "strictly": "[2.17.1, 3[",
            "requires": "[2.17.1, 3["
          },
          "reason": "CVE-2021-44228, CVE-2021-45046, CVE-2021-45105, CVE-2021-44832: Log4j vulnerable to remote code execution and other critical security vulnerabilities"
        }

I would think that this publishing of ranges is okay, because there is a clear version defined and thus the resolution result is reproducible. However when turning on failOnNonReproducibleResolution the resolution fails with:
Could not resolve org.apache.logging.log4j:log4j-core:2.22.0: Resolution strategy disallows usage of dynamic versions

Reproducer: https://github.com/boris-petrov/gradle-spotbugs-bug
Build Scan: https://scans.gradle.com/s/yowv7hwhqd7sm/dependencies?focusedDependency=WzAsMiwzLFswLDIsWzIsM11dXQ&toggled=W1swXSxbMCwyXSxbMCwyLFsyXV1d
Right now failOnNonReproducibleResolution is unusable as soon as you get any external library with such constraints into your build. We wonder, what the right solution for this is:

Is this a bug in failOnNonReproducibleResolution? To me it looks like that. Then this issue should be turned into a bug.
Should libraries not publish such constraints? What would be the argument for not doing so? (If there is a clear answer, we can open an issue at Spotbugs.)

Context (optional)
No response
Page with the problem
https://docs.gradle.org/current/userguide/resolution_strategy_tuning.html#reproducible-resolution

Thank you for your interest in Gradle!
This issue needs a decision from the team responsible for that area. They have been informed. Response time may vary.

I think this makes sense.  The devil's in the details about how we can detect the scenario where:

There's a range that makes the request dynamic
There are other constraints that make the request always resolve to a specific version

In that scenario, I don't think we should fail because there are no dynamic versions that can change the result.
This isn't something we're going to get to soon, do you have some ideas where this could be done?

I agree with what you say about how this feature should work.
I have no urgent need for this. There are much more severe bugs in dependency management that I think need to be addressed (like #22326).
I created this issue also on behalf of @boris-petrov who discovered this (see https://github.com/gradlex-org/java-ecosystem-capabilities/issues/62#issuecomment-1915805718).
I think leaving this feature in its current state is more hurting than helping. As I see it, the benefit of using this feature right now is very small.  You can only use it if you are "lucky". As soon as you have any dependency that adds constraints with ranges, it is useless. Which is somehow paradoxic: on the one hand we advertise for libraries to publish more and better metadata for years, but now if they do that, we run into these kinds of problems which leave users puzzled.
So why would you want this feature? To get reproducible builds. So it is "nice" to use it to get an early warning, but this feature alone won't make sure that your build is reproducible. And if you setup other solutions to check your build for reproducibility, those will most likely catch such problems as well. Also the configuration cache lets you know if it is out of date because of dynamic versions.
In my opinion, if there are no resources to fix this feature, I would rather think about discouraging/deprecating it instead of just leaving things as they are. At least I would add a big note about this to the documentation.


fix(android): Fix crash on deny permission request on Android M

On Android 6.0 when asking for permission to access location, answering Deny will cause application crash

@ochakov could you please create associated JIRA issue or post repro steps here.
http://cordova.apache.org/contribute/issues.html
I don't see app crash when I deny location access and can see error callback is correctly called in this case. From the code context is set in execute method so technically this should never happen or there is some tricky case (background/foreground or something) and should be investigated further to find root cause.

Cordova CI Build has completed successfully.
Commit     - Link
Dashboard - Link

Builder Name
Console Output
Test Report
Device Logs

Windows 8.1 Store
Link
Link
Link

Windows 10  Store
Link
Link
Link

Windows 8.1 Phone
Link
Link
Link

iOS
Link
Link
Link

Android Mac
Link
Link
Link

"On Android 6.0 when asking for permission to access location, answering Deny will cause application crash"
Today i lost 4 hours trying to make it working on the emulator(Genymotion, Google Nexus 5X 6.0.0 - API 23), but on my case was when answering Allow which causes the application to "crash". With "crash" i mean it will show the android logo like it was starting and after 10 seconds it will be back to the main menu with the app running in background. Trying to debug was useless.
Now on Genymotion, Google Nexus 6 5.0.0 - API 21.
Don't know if it helps anyone but i though it would be good to share.

I can confirm that pretty much exactly what @iursevla said is happening to me. I click 'Approve' and it looks like it restarts the emulator.

I am experiencing the same issue as @gkrizek and @iursevla .
When I click allow permission on android 6 in the emulator, it crashed the emulator.

If this is needed, I have the logcat output  from when it dies. I don't have a physical device to test this on, only an emulator. @xiaoerge or @iursevla, do you guys have a physical device?
While it's asking me to accept the location, this is output many times a second:
10-20 23:34:27.732  1708  1708 W art     : Attempt to remove non-JNI local reference, dumping thread
10-20 23:34:27.782  1708  1708 W art     : Attempt to remove non-JNI local reference, dumping thread
10-20 23:34:27.832  1708  1708 W art     : Attempt to remove non-JNI local reference, dumping thread
10-20 23:34:27.884  1708  1708 W art     : Attempt to remove non-JNI local reference, dumping thread

Then the actual failure:
10-20 23:34:27.919  1763  1776 E Surface : getSlotFromBufferLocked: unknown buffer: 0xeb0ed380
10-20 23:34:27.924  1763  1776 D OpenGLRenderer: endAllStagingAnimators on 0xf2c2b480 (RippleDrawable) with handle 0xeea529d0
10-20 23:34:27.944  1708  1708 W art     : Attempt to remove non-JNI local reference, dumping thread
10-20 23:34:27.989   758  1462 I ActivityManager: Killing 896:com.android.externalstorage/u0a7 (adj 15): empty #17
10-20 23:34:27.993   758   758 D AndroidRuntime: Shutting down VM
10-20 23:34:27.993   758   758 E AndroidRuntime: *** FATAL EXCEPTION IN SYSTEM PROCESS: main
10-20 23:34:27.993   758   758 E AndroidRuntime: java.lang.IllegalArgumentException: provider doesn't exist: network
10-20 23:34:27.993   758   758 E AndroidRuntime: 	at com.android.server.LocationManagerService.requestLocationUpdatesLocked(LocationManagerService.java:1637)
10-20 23:34:27.993   758   758 E AndroidRuntime: 	at com.android.server.LocationManagerService.requestLocationUpdates(LocationManagerService.java:1616)
10-20 23:34:27.993   758   758 E AndroidRuntime: 	at android.location.LocationManager.requestLocationUpdates(LocationManager.java:880)
10-20 23:34:27.993   758   758 E AndroidRuntime: 	at android.location.LocationManager.requestLocationUpdates(LocationManager.java:496)
10-20 23:34:27.993   758   758 E AndroidRuntime: 	at com.android.location.fused.FusionEngine.enableProvider(FusionEngine.java:142)
10-20 23:34:27.993   758   758 E AndroidRuntime: 	at com.android.location.fused.FusionEngine.updateRequirements(FusionEngine.java:195)
10-20 23:34:27.993   758   758 E AndroidRuntime: 	at com.android.location.fused.FusionEngine.setRequest(FusionEngine.java:118)
10-20 23:34:27.993   758   758 E AndroidRuntime: 	at com.android.location.fused.FusedLocationProvider$1.handleMessage(FusedLocationProvider.java:98)
10-20 23:34:27.993   758   758 E AndroidRuntime: 	at android.os.Handler.dispatchMessage(Handler.java:102)
10-20 23:34:27.993   758   758 E AndroidRuntime: 	at android.os.Looper.loop(Looper.java:148)
10-20 23:34:27.993   758   758 E AndroidRuntime: 	at com.android.server.SystemServer.run(SystemServer.java:285)
10-20 23:34:27.993   758   758 E AndroidRuntime: 	at com.android.server.SystemServer.main(SystemServer.java:170)
10-20 23:34:27.993   758   758 E AndroidRuntime: 	at java.lang.reflect.Method.invoke(Native Method)
10-20 23:34:27.993   758   758 E AndroidRuntime: 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
10-20 23:34:27.993   758   758 E AndroidRuntime: 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)
10-20 23:34:28.005  1708  1708 W art     : Attempt to remove non-JNI local reference, dumping thread
10-20 23:34:28.006   758   758 I Process : Sending signal. PID: 758 SIG: 9
10-20 23:34:28.025  1763  1763 E JavaBinder: !!!   Can we get any help on this?

This PR wraps the following lines
https://github.com/apache/cordova-plugin-geolocation/blob/7e7a473cfadf9e41034ca82042298f093db281c9/src/android/Geolocation.java#L73-L74
in a condition if (context != null)
Master already has this condition here:
https://github.com/apache/cordova-plugin-geolocation/blob/7e7a473cfadf9e41034ca82042298f093db281c9/src/android/Geolocation.java#L68-L69
So I believe this PR is no longer relevant.

Thanks for investigating @breautek - and thanks to @ochakov for creating this PR back in the day 🚀 👵


Use natsort in place of regex for neuralynx load

look into using this instead to get properly sorted files https://pypi.org/project/natsort/

No problem with using regex as is currently done, I more just wanted to let you know about this package. Feel free to use if you'd like

natsort is now being used for all neuralynx interfaces


calling htmldialog-inputbox from UI::HtmlDialog

Hi there!
For a while I am trying to call an inputbox from a UI::HtmlDialog
When I use the following simple example from a UI::HtmlDialog the called InputBox has just a frame and displays without any further shown data (no buttons, no textbox) modal and can't be closed.
Due to the modal shown frame one can just end the SketchUp -process / -task.
options = {
        title: 'test',
        accept_button: 'Ok',
        cancel_button: 'Cancel',
        inputs: [
          HtmlInputBox::HtmlUI::Textbox.new('some data'),
        ],
      }

test_inputbox = HtmlInputBox::HtmlUI::InputBox.new(options)

test_inputbox.prompt

I am on a MacBook (Apple Silicon M2) using SketchUp 2024.
When the shown example is called directly (for example by an exported function in the module from the console) and not 'above' a UI::HtmlDialog everything works perfect.
I just need to catch some additional data within my UI::HtmlDialog dialog. So I added some buttons that should call InputBox dialogs.
I can't get it working and would be very thankful for some help.
Thanks,
Dirk

So, the snippet above is not enough to reproduce? That works fine?
If so, can you provide a full repo example please?


[Feature Request] Allow searches on discontinuous characters

In my case, I'd like the picker to show the cs589-hw4 venv when I type 5894. main branch can do this, but not regexp branch.

Hi,
Thanks for the suggestion! I did change the default filtering in the regexp branch to match all characters, but going back to the previous filtering can easily be made into an option you can set. Thanks for the suggestion!

Thank you! Really appreciate it.


polymer app does not render correctly in Firefox and Internet Explorer

This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?

Start fish.dart as local server
2.Open serviceonline.html of the deployable version
3.

What is the expected output? What do you see instead?
I created to custom elements with polymer. The first one is an input field with a refresh button. After typing in some text into the input field and pressing refresh button 3 Links will appear. When clicking the first link the second custom element with data binding will be created and displayed. When opening the site with dartium everything works fine. But when I open the file with IE or Firefox all custom elemts are displayed immediately and also without data binding. I attached the deployable version of the app as zip file.
What version of the product are you using? On what operating system?
Dart Editor version 1.0.0_r30798 (STABLE)
Dart SDK version <IP_ADDRESS>_r30798
Windows 7
Please provide any additional information below.

Attachment:
build.zip (2.94 MB)

This issue has been moved to dart-lang/polymer-dart#376.


Activity Log

#11 done.
(I know it could look better, but I hardly have time for it.)

Thanks @Michalw101 ! Much appreciated!


File archive app - option to write logs in compact JSON format

Serilog's compact JSON format is a great archival format for Seq logs. We should add a Boolean option to Seq.App.FileArchive to write logs in compact JSON rather than text.

This is now implemented in the new Seq.App.JsonArchive app.


pythonPackages.django-sampledata-helper: init at 0.4.1 (+dependency sampledata 0.3.7)

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] Linux

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

this package is now part of a combined PR: #28861


Support Cordova 5

The Cordova docs say:
To include a plugin from npm specify npm in the source attribute.
<gap:plugin name="example-plugin" source="npm" />

I have tried this with
 <gap:plugin name="cordova-plugin-websocket" source="npm" />

The build service failes, because it does not respect the "source" parameter.
According to this announcement, the Cordova plugins are moving to npm:
https://cordova.apache.org/announcements/2015/04/21/plugins-release-and-move-to-npm.html

Tabris.js 1.0 was tested and is compatible with Cordova 4.x. The build service currently does not support 5.x
We will investigate and upgrade asap. Thanks for reporting

We have updated to Cordova 5.1.1. You can already use the new version when setting the Tabris.js Version to nightly within the online build service. Please note that you don't need to use gap:plugin anymore. Cordova 5 support a simple plugin element in the config.xml now. A sample usage looks like this:
<?xml version='1.0' encoding='utf-8'?>
<widget
    id="my.first.app"
    version="1.0.0"
    xmlns="http://www.w3.org/ns/widgets"
    xmlns:cdv="http://cordova.apache.org/ns/1.0">
    ...
  <plugin name="cordova-plugin-camera" version="1.2.0" />
  <plugin name="cordova-plugin-dialogs" version="1.1.1" />
</widget>

I'm seeing this error on the tabris build server now, this worked fine a month ago
Failed to install 'cordova-plugin-camera':CordovaError: Plugin doesn't support this project's cordova-android version. cordova-android: 4.0.2, failed version requirement: >=5.0.0-dev

The cordova version we support is 5, but each platform has its own version number. Therefore you should reference the older plugin name as stated here: https://github.com/apache/cordova-plugin-camera until we have added support for the cordova android 5.

how would I do that from the tabris.js build service? what is the proper syntax for referencing older plugin version from config.xml?

Have you tried a concrete version number in you config.xml?
<plugin name="cordova-plugin-camera" version="1.2.0" />

See here for details https://tabrisjs.com/documentation/latest/build#adding-plugins

this works, not sure why I had the older plugin tag code that used repo directly and not npm
thanks!

Great to hear that it worked. Thanks for the info.