Linking PublicationChannel and Platform to classes from the domain Consumption

In the current model I cannot describe the links between a Platform or its PublicationChannels to classes like ConsumptionDeviceProfile, ConsumptionDevice and Consumer. Neither EBUCore nor CCDM provided sufficient modelling here. So, here's the current modelling situation:

For some of the concepts, I am not sure, what the benefit is and whether we need them at all (see orange marked elements and question):

Therefore, I propose to remove some of them and, more importantly, introduce some more links between the exisiting concepts (see red marked elements and hints):

Looking forward to your comments!

I am not sure, whether you @tormodv, @alexandre-rouxel and @kimviljanen have already noticed this issue. I am looking forward to your comments!

Looks good, I'll write a change that we can have a look at.

I will propose to keep the service. In English, Channels and platforms are often referred to as services, and the class will act as a superclass for the common properties of Channel and Platform.
I'm not sure that we need the rename the "compliesWith", together with the existing "Supports Consumption DeviceProfile" the classes are connected.
The device complies with some demands for capabilities required by the service/platform/cannel.
We can of course add the relations in red if we need.
What du you think?

I agree: a target platform is a platform, and the relation hasTarget Platform: does the job
About ConsumptionDevice: @JuergenGrupp. A  ConsumptionDevice is linked with a Consumer who consumes a Service, a Platform, or a publishing Channel. I don't see the connection in your diagram.
We should discuss if we want to keep the class service. If no then we can discuss the changes.

When I tried to paint the graphic with the result of our last discussion, I stumbled over the "Service" again:
In linear publication, a Service might describe the users perception, whereas the PublicationChannel better describes the technical sides of the publication, given by the platform. E.g. SWR's linear TV service is called "SWR Fernsehen" and it is published through different publication channels (defined e.g. by frequency range, Codec, etc) on different platforms (DVB-S, DVB-T), which, from the consumer perspective might also be numbered ("Channel 1") or named ("SWR Fernsehen"). Hence, to represent this, it would be good to keep the class Service. However, not as a superclass, but as an additional class, related to PublicationEvent, PublicationChannel and Platform. (Maybe even to ConsumptionDeviceProfile?).
This modelling also allows to omit an object of class Service completely, when describing a non-linear publication, e.g. a PublicationEvent on the Platform "Youtube" in the PublicationChannel "SWR".
That's my updated proposal:

The Editorial Committee agreed to this proposal and additionally to change the names to "PublicationService" and to "PublicationPlatform".
The examples for linear and non-linear uses of this model should be part of the documentation later on.


Code Release

Thanks for sharing such a great work!
I'm looking forward to the code release. It would be better if the code is implemented with PyTorch.
Thanks!

Thanks for your interest in our work. The code is in PyTorch and will be released in the next two weeks. I'm sorry for the delay!

very interested in your excellent work！
looking forward to the code release.

Thanks for your excellent work！
Looking forward to the code release, too.

Thanks for waiting patiently : )
I'm cleaning the code right now!

The code is released :)


Importing chart over API fails if the chart was built on top of a virtual dataset

Importing chart over API fails if the chart was built on top of a virtual dataset. I exported the chart with the /chart/export endpoint and then tried POSTing the resulting .zip to  /chart/import.
Expected results
Expect chart to get uploaded successfully.

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

@dpgaspar could you please take a look? thanks Daniel!


Fix documented permission requirements

Related Issue
N/A
Proposed Changes
The documentation around the minimum permissions needed to deploy the CloudFormation template drifted from the template itself and were missing several resources/permissions. This adds those resources as well as a note that the resource names might be truncated relative to the permissions listed if the stack name is long.
Reviewers
@turan18

Whoops. I missed those. I moved my notes into that section to make it more clear.


Update CI testing

Updates to testing from ioBroker Core team. Please ideally merge and do not squash. On questions check https://forum.iobroker.net/post/674030

changed manually


Finding Time Dealy using GCC-Phat - contribution doubt

I've been using an implementation of GCC-PHAT to find time delay between two sound files, and then the angle (localisation) using SciPy and NumPy. I can imagine cross correlation and identifying time delay might be useful in general, so I thought I would contribute by helping (or trying) to make them SciPy functions. How do I go about doing this?
Also, would this be useful at all to SciPy folks or should I just put up my implementation elsewhere?
Also, how do I start contributing if I want to do this?

Hi @bhargavvader, that sounds potentially interesting. It would be good to bring this up on the scipy-dev mailing list (see http://scipy.org/scipylib/mailing-lists.html); that's where decisions on adding new features are made. Some links and context information may help, I had to do a bit of digging.
The original paper from Knapp and Carter (see http://www.xavieranguera.com/phdthesis/node92.html) is heavily cited, which is a good sign. Looking at the citing papers it seems the applicability is broader than just audio correlation from multiple receivers.
This may also help: http://dsp.stackexchange.com/questions/8732/generalized-cross-correlation

When you send an email to the list, a link to code you already have would also be useful.

Cool, I'll do that!

Numpy and the Python stdlib are dependencies of Scipy, so using those is fine. Using fft functions from numpy.fft or scipy.fftpack is also fine. Using functions from other packages that aren't already dependencies of Scipy is usually not OK.

And yes, fine to ask that here. On the mailing list would also have been fine.

no update in 4 years, so I'll close this issue. thanks @bhargavvader


Export GetRegisteredWorkflowTypes so I can use in shadowtest.

Exported the GetRegisteredWorkflowTypes() function for public use.

In order to know what function types have been registered and be able to iterate them, and run the
WorkflowShadower on each exported function type.

Pull Request Test Coverage Report for Build 0184efaa-ad02-4531-914e-fe7e58dbdce6

5 of 12 (41.67%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 64.262%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/workflow_replayer.go
0
3
0.0%

internal/workflow.go
0
4
0.0%

Totals

Change from base Build 0184e8f9-af9f-4766-b6a9-3fb5e0401f70:
-0.02%

Covered Lines:
12684

Relevant Lines:
19738

💛  - Coveralls


Epic environment rebuild

Part of the epic test environment was an xa-cluster in the aws qa account that was removed by cloud custodian.
This needs to be rebuilt along with the rest of the infrastructure in gcp, though it may be possible to link the existing gcp infra to the new xa-cluster.

Matt's slack instructions

Matt's details

Request from GJ for rebuild with 11.7RC

Appliance has a release-11.7 tag, which I assume is the 11.7rc?

XA cluster build run by @ismarc

Tasks

[ ] Verify Matt's rebuild
[ ] Build 11.7 environment


Team overview page does not link to individual pages

Hey all,
I am really happy about being able to use this theme, it's really clean and fun to work with!
I found that in the teams.html, there is no link to the individual pages from anywhere on the team member's pages. This is especially confusing since the promoted teams have their bio's truncated. I was able to verify that the pages were in fact rendered and exist on the site, I just don't think that there is any link that actually points to them.
I'm not sure if that was taken into consideration, but I thought it was worth bringing up as it was really the only part of the site that has been non-intuitive thus far. Keep up the good work!

@smacz42 Yes this is by design. I'd be happy for teams.html to link to the individual pages as the default. If you have made this change already could you do a PR?


fix(chart & filters): make to padding between textarea and buttons

SUMMARY
Padding between TextArea and Buttons.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
BEFORE:

AFTER:

TESTING INSTRUCTIONS
How to reproduce issues

Go to any chart
Go to Filter options of any chart

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up


Use the latest version of webpack

The latest version of webpack has some new features (support symLinks,etc...) but we can't use it due to restriction on version <1.5.0-0
Is it possible to allow an upgrade by modifying package.json ?
After some tests I didn't see any issue between this version and gulp-webpack.
Maybe a better solution could be to set webpack in devDependencies and peerDependencies and remove it from dependencies to allow project to better manage wish version they want to use ?
Here is a pull request that explain this possibility

I dont like peer dependencies. I secretly expose a way to inject the version of webpack though if you absolutely must use a different version:
var webpack = require('webpack')
var gulpWebpack = require('gulp-webpack')
// Inject your version of webpack into gulp-webpack
gulpWebpack.webpack = webpack

I'm happy to bump the version of webpack and publish a new version though. Just not as a peer dep as they are broken and awful.

Actually scratch that, I don't. But still happy to just bump the version instead.

I just saw that the second argument of the export function is the webpack version to use:
function(options, wp, done)

So I can do this in my gulpfile:
require('gulp-webpack')(options, require('webpack'));

And this solved my issue :)

Oh cool, I guess I did expose it. :)
Updated to latest webpack anyways and published 1.2.0. Thanks!

Any chance of reconsidering this? I didn't realize for a good while that gulp-webpack was using its own version (forgot the docs said it), and it seems redundant to download webpack twice or wait for this project's dependencies to be updated. grunt-webpack is also currently using peerDependencies.

@pstoica Reconsider what?

Agh, sorry, I meant reconsider switching to peerDependencies.

@pstoica Soon npm will be removing peerDependencies so eventually it wont be an option. I'm fine just bumping the dep or letting people explicitly override the webpack instance, if absolutely needed.

That's fair, and it looks like I can access the internal webpack version if I need to pass that into webpack-dev-server. Thanks!


New version: Revise v2.5.0

Registering package: Revise
Repository: https://github.com/timholy/Revise.jl
Created by: @timholy
Version: v2.5.0
Commit: 8bee71b4501b42d360edb81f8d422219a8c3e0fa
Reviewed by: @timholy
Reference: https://github.com/timholy/Revise.jl/commit/8bee71b4501b42d360edb81f8d422219a8c3e0fa#commitcomment-36455461

Needs a retrigger to fix the conflicts [noblock].


Add translation for 'Card number or pin not valid' message

Thank you very much for your contribution @tdgroot !


Merge error publicated on docs.microsoft.com

Fixing merge error published on the offical docs removing the conflicting message and keeping the proper versión.

@jangelfdez : Thanks for your contribution! The author, @mgoedtel, has been notified to review your proposed change.


Updated U.K. Drivers License

The last 5 characters of a UK driving license are in many cases both numbers and letters. Drivers licenses in this format do still match correctly.

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@chrfox - PR has been copy edited and is ready for final review.
Thank you.
cc: @kenwith

The documentation must reflect the hard coded values in the product. Checking with the PM to see if the suggested change is accurate

requeried PM, waiting response

Per the owning PM
"I assume you are asking about the highlighted. Our code looks for 5 digits at the end and hence aligns with the documentation.
https://en.wikipedia.org/wiki/Driving_licence_in_the_United_Kingdom#:~:text=Each licence holder in England,constructed in the following way%3A&text=7–8%3A The month of,62 instead of 01–12)
Thanks,
Prateek
"
making this change would put the documentation out of alignment withe product, closing this PR


Update pluginOptions -> themeOptions

Description
When I brought the plugin for sourcing swiftype into the theme itself, I forgot to update all references of pluginOptions -> themeOptions (#201). This PR updates that and ensures we're using a consistent variable name throughout gatstby-node.js.

@jerelmiller good call, I've updated that commit. Always makes me nervous updating commits when they're already pushed up, but this feels fairly safe.

@jerelmiller good call, I've updated that commit. Always makes me nervous updating commits when they're already pushed up, but this feels fairly safe.

As long as you're using --force-with-lease, you can make sure that you have all the changes synced from remote to your local before a force push has the potential to overwrite code. I wouldn't stress about it too much unless someone else is actively working on the same branch. Anyways, thanks for updating the commit!

As long as you're using --force-with-lease, you can make sure that you have all the changes synced from remote to your local before a force push has the potential to overwrite code. I wouldn't stress about it too much unless someone else is actively working on the same branch. Anyways, thanks for updating the commit!


$and or a problem with index

Hi.
Thank you for the great plugin.
I have not been able to debug properly yet but maybe you could help.
I have a case working perfectly fine on the browser but not on an android 5 phone.
My document are simple
{
 _id: '123',
  type: 'type1',
  category: 'category1',
  name: 'My Name'
}

category can be null
and the index
db.createIndex({
				  index: {
				    fields: ['type', 'category']
				  }
				})

the query (1) works fine in chrome
db.find({
		  selector: {
		    $and: [
		    	{ type: 'type1' },
		    	{ category: 'category1' }
			]
		  }
		});

or  query (2)
db.find({
		  selector: {
		    	 type: 'type1' ,
		  }
		});

a query (3) on the wrong index and there is an error: index does not exist (normal)
db.find({
		  selector: {
		    	category: 'category1'
		  }
		});

on android 5 (built with ionic and so running in webview)
(1) returns no result..
(2) returns 25 documents out of 93
(3) return a promise but no callback, no error ..
I tried to add an index on category only (or even other fields - name never null) and query only that field and it does not return any result
I will keep looking tomorrow and might give more detail.
If you need anything, let me know

Hi @finchen, thanks for the comprehensive error report.
@nolanlawson any idea how we can debug this? I don't know anything about debugging on mobile phones.

@garrensmith If you have an Android phone, you can plug it in to your USB drive and then in Chrome go to chrome://inspect; there are instructions here: https://developers.google.com/web/tools/chrome-devtools/debug/remote-debugging/remote-debugging
Else you will need to start up an Android emulator, for which the steps are similar.

@finchen Which adapter are you using? IndexedDB, WebSQL, SQLite plugin...? Also if you could write a small web page or script to reproduce, it would be fantastic.

WebSQL on chrome and SQLlite on android.
It is working fine by removing the sqllite plugin.
Not sure if good or bad news..
At the time I did not see the warning about the sqllite plugin on the pouchdb adapaters documentation page.. I am a bit worry to have based a couple of apps on a 80mb simple database with attachment in it (I know it's not ideal, but since it is supported it is so easy.. and great)
this is the extra log info (in chrome://inspect) I get when on android
OPEN database: _pouch_mydb-mrview-b878448f9d3d65c0f259c27a219cade9
new transaction is waiting for open operation
DB opened: _pouch_mydb-mrview-b878448f9d3d65c0f259c27a219cade9

To reproduce, you'll need a cordova app with websql
npm install -g cordova
cordova create testapp com.testapp Testapp
cd testapp
cordova platforms add android
cordova plugin add org.apache.cordova.device
cordova plugin add org.apache.cordova.console
cordova plugin add org.apache.cordova.device
cordova plugin add cordova-sqlite-storage
cordova run android

once loaded you can run these blocks one by one from chrome://inspect

var script = document.createElement('script');
script.src = "//cdn.jsdelivr.net/pouchdb/5.0.0/pouchdb.min.js";
document.head.appendChild(script);

script = document.createElement('script');
script.src = "https://raw.githubusercontent.com/nolanlawson/pouchdb-find/master/dist/pouchdb.find.js";
document.head.appendChild(script);

var db = new PouchDB('test', {adapter: 'websql'});
db.put({
  _id: 'mydoc',
  type: 'type1',
  category: 'category1',
  title: 'Heroes'
});

db.createIndex({
		  index: {
		    fields: ['type', 'category']
		  }
		});

db.find({
		  selector: {
		    $and: [
		    	{ type: 'type1' },
		    	{ category: 'category1'}
			]
		  }
		}).then(function(result){

			console.log('expect 1 result but get 0', result);
});

That is good news. We are aware that the SQLite Plugin has tons of issues mostly due to a buggy implementation, so we don't recommend it anymore. Especially on Android, your code will be both faster and more reliable if you use IndexedDB or WebSQL (take your pick; they're both good on Android).


chore(msrv): Set MSRV versions in workspace member

Settings MSRV for all workspace members. 3 workspace members are still missing because of: #85

OK, can you give this one another try? :)

This should be ready, I will try to incorporate a check for it into CI in the near future.


use semistandard and remove jshintrc

https://www.npmjs.com/package/semistandard

Using standard now.


The terminal code is monolithic

All Arduino code is contained in one file, making it hard to read and maintain. Refactoring would be very beneficial.

mentioned in commit 541109328df4dfafbba47ececaf9e3b909aae26d

created branch 45-terminal-refactor to address this issue

mentioned in commit 130e17b5eed67d0b908995b79579ea3537fdd280

mentioned in commit 1a782b95bd120eee913a49eaf5b5d3e48f561114

mentioned in commit bde2fc3a690cecbdcb80f838e0696b9c374a1105

mentioned in commit 607f955484a909dc89fcaef6514e4c2195c4d2e4

mentioned in commit 6134b8892440a14a36e175f1d71d027b27d32e45

In GitLab by @alexkval on May 17, 2024, 15:39
mentioned in commit b2b509c0bc8891699da9c0694fa67cc45368907b

mentioned in commit 775df9d4b06bd49e9361b98b1aa630903cb9615c

mentioned in commit 37e4fe147ccb7e76c69a4ec1a8dd3be3ecea4ef7

mentioned in commit 84ecd58eae44c9e8647ba0f923c26db351d85c3b

mentioned in commit 51827283f32753571888c339bc04bbd69d9bd2c9

mentioned in commit 41e9f79775ac5ae4aef69b14f1779943d8ed1493

mentioned in commit 99506a83c62fb9f181344b4a57bdef3f363e4280

In GitLab by @albu-razvan on May 17, 2024, 23:40
mentioned in commit 287f62ec1b0961ac9bcc3a2aeded22952c63a5a4

mentioned in commit e5fd3cd78ecc9cad91f7c0362831f1d53072bc14


Add ability to add support to new language

It would be nice if I could quickly add support to a new language without having to create a pull request, document everything etc etc
Is this already possible today? If it is it's not documented, so that would be the actionable item here.
Thank you for the plugin! I actually was thinking about how much time I spend writting the same code to log the value of a varaible and this addresses that :)

Currently, isn't not possible for an end user to add support for a new language inside of this plugin's configuration.
I don't see it as a compeling use case, I think that any new language support would be better as a PR so any user of the plugin can benefit from it


Remove 'Node:' from card headers.

Before

After

What's the rationale for doing this?

Not a big deal but "less" is more. The word "Node" in the top of the card is superfluous since at the bottom of the card it says "Node Operational" or "Node ..".  Another solution is to have more of a header at the top of the web page which seems to have no header or intro explanation saying these are all the main nodes or discussed BIP's of Bitcoin - then "Node" is definitely not needed in the card header...  Looks better without it (btc1 still say Node btc1 in the above comment though typo maybe).

Messed up my edits but my main reasoning for this at the time was because of scaling problems with certain display sizes and div height making the "Node Operational" green backgrounds stretch out except for the largest one as seen with Bitcoin Cash in the before image.
Now that there is an extra column for scheduled forks, this horrendous layout error happens instead.

It would be better if you could figure out how to line up the header and footers in the styling itself instead of changing the strings to make the styling match.

I think each card looks too crowded and the info is repeated in each card could be put once on the left side of the page.  Might require a bit of rework.
e.g.  Just notice also questions do not have question marks "?" - Would be better if the did in my opinion -
Cross between card and table format
Has Experienced a Blockchain Reorginization**?**:       No              No             No      No    No
And / Or put fewer cards in a row so that each card can be wider reducing line wraparound...


chore: introduce dependabot

Signed-off-by: Jason Liu<EMAIL_ADDRESS>Ⅰ. Describe what this PR does
For further information, please check https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews

Codecov Report

Merging #181 (15254fa) into main (d3b3411) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #181   +/-   ##
=======================================
  Coverage   58.75%   58.75%
=======================================
  Files          94       94
  Lines        8615     8615
=======================================
  Hits         5062     5062
  Misses       3121     3121
  Partials      432      432

Flag
Coverage Δ

unittests
58.75% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 d3b3411...15254fa. Read the comment docs.

/lgtm
/approve


New package: DigitalAssetExchangeFormatIO v1.0.0

Registering package: DigitalAssetExchangeFormatIO
Repository: https://github.com/Larbino1/DigitalAssetExchangeFormatIO.jl
Created by: @Larbino1
Version: v1.0.0
Commit: 7d15c664a174f6b3a94cea8bcfe47bfcda78db00
Reviewed by: @Larbino1
Reference: https://github.com/Larbino1/DigitalAssetExchangeFormatIO.jl/issues/1#issuecomment-2263424321

@rdeits @tkoolen
Hi both, you might be interested in this - a simple dae parser for robot visualization. Once parsed into GeometryBasics meshes and colours, I can plot with GLMakie.
I would appreciate if you could lend some other files to test on, for example from the ur5 in this thread:  https://discourse.julialang.org/t/how-to-load-urdf-files-with-dae-models-into-meshcat/19509

[noblock] The reason for this is ultimately how OpenGL (and probably the other rendering APIs) work. You only get one index buffer per draw call, with one index per vertex. So you are forced to use the same vertex index for positions, normals, etc.
We are currently planning to restart https://github.com/JuliaGeometry/GeometryBasics.jl/pull/173 and refactor GeometryBasics. I'll probably move a function that does this from MeshIO to GeometryBasics with that. We could also consider adding something like MultiFace so that you can have a mesh that does not fit the OpenGL format, and then add some functionality to convert to it later.

[noblock] The reason for this is ultimately how OpenGL (and probably the other rendering APIs) work. You only get one index buffer per draw call, with one index per vertex. So you are forced to use the same vertex index for positions, normals, etc.
We are currently planning to restart JuliaGeometry/GeometryBasics.jl#173 and refactor GeometryBasics. I'll probably move a function that does this from MeshIO to GeometryBasics with that. We could also consider adding something like MultiFace so that you can have a mesh that does not fit the OpenGL format, and then add some functionality to convert to it later.

[noblock] Hi, thanks for the reply! That sounds like a great piece of work, which will really help with the Mesh ecosystem in julia for plotting.


Replace unused params with _

Description
This PR enable unused-parameter rule in linter and fixes all issues.
To perform this I enable reporting all issues and json output  in .golangci.yml
output:
  format: "junit-xml:report.xml,colored-line-number,json:out.json"

issues:
  max-issues-per-linter: 0
  max-same-issues: 0

Then with one-liner I prepared a script to replace all unused params with _
cat out.json | jq '.Issues[] | [.Pos.Line, .Text, .Pos.Filename] |@csv' -r | grep unused | sed 's/"unused-parameter: parameter \x27/ /g' | sed 's/\x27 seems to be unused, consider removing or renaming it as _"/ /g' > work.sh

I modified work.sh manually with multicursor (as I'm not proficient in sed) to make every line in format of
sed -i '<line number>s/<param name> /_ /' <file name>

Finally I applied the script with source work.sh.
This procedure was repeated multiple times until no issue was reported.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
CI

/retest

I don't want you to have to redo the work if it's not necessary, so this is just a question, not a suggestion: did you consider replacing param with _param or _param_unused or _unused_param?

@ebensh Nope, as _ (aka blank identifier) is a workaround for declared but not used params/variables/imports. If I use _param it will still be reported as unused as underscore prefix has no special meaning in Go but only in some conventions:
See:

https://go.dev/doc/effective_go#blank
https://github.com/uber-go/guide/blob/master/style.md#prefix-unexported-globals-with-_

Example:
central/activecomponent/datastore/index/indexer_impl.go:30:29: unused-parameter: parameter '_unused' seems to be unused, consider removing or renaming it as _ (revive)
func (b *indexerImpl) Count(_unused context.Context, q *v1.Query, opts ...blevesearch.SearchOption) (int, error) {


1263: fs; bfd_disasm

Checklist

[ ] Language changes are updated in docs/reference_guide.md
[ ] User-visible and non-trivial changes updated in CHANGELOG.md
[x] The new behaviour is covered by tests

@fbs related to #1263 for bfd_disasm

Needs a rebase

@danobi hi, rebased

We try to avoid merge commits in our, it makes history confusing and can make diffs weird like happens here. What I usually do is adding a --rebase to the pull command to have it rebase instead of merge. It can be bit more work but it keeps history clean.
The PR is currently unmergeable due to this, can you fix it? One way to do it is by:
# replace upstream with w/e your remote is
$ git fetch upstream
$ git checkout -b 1263_bfd_disasm_v2 1263_bfd_disasm  #easy backup just in case
$ git reset upstream/master
$ git add <your changes>
$ git commit -c 30e95232830919a9ebfc94cf63fce0d4e5495c70
$ git push yourreporemotename 1263_bfd_disasm_v2:1263_bfd_disasm --force-with-lease

Sorry for the noise, thanks!


invalid gitlab link

Describe the bug
Generated link is wrong if has start and end lines contains in hash mark.
To Reproduce
Steps to reproduce the behavior:

Go to a static php method
Click on context menu copy github link
generated url is wrong

Expected behavior

correct: https://gitlab.example.com/mygroup/myproject/blob/977ac19d4d4f3bb3ed6fce0fa78d658a7925df75/common/helpers/EnvironmentHelper.php#L429-429
wrong: https://gitlab.example.com/mygroup/myproject/blob/977ac19d4d4f3bb3ed6fce0fa78d658a7925df75/common/helpers/EnvironmentHelper.php#L429-L429

Desktop (please complete the following information):

OS: ubuntu 18.04
Browser -
Version -

Additional context

Gitlab runs on its own server (self hosted) GitLab Community Edition 12.9.4

Are you sure you have GitLab selected as the host in the plugin preferences? It can't auto detect the host so it will default to GitHub which uses the format you're seeing.

Are you sure you have GitLab selected as the host in the plugin preferences? It can't auto detect the host so it will default to GitHub which uses the format you're seeing.

I haven't done a setup (plugin preferences), but I can't find the option for that at currently (PhpStorm).

PhpStorm -> Preferences->Other Settings -> GitLink

thanks!


Application Backgrounded events send properties as null, but  properties should always be an object

The automatically collected Application Backgrounded event should send an empty properties as an empty object, and not as null.
 "event": "Application Backgrounded",
  "integrations": {},
  "messageId": "d7ee8923-280f-4224-86b7-117ae34771c3",
  "originalTimestamp": "2024-02-22T03:34:17.882927",
  "properties": null,
  "receivedAt": "2024-02-22T08:34:24.593Z",
  "sentAt": "2024-02-22T03:34:23.476Z",
  "timestamp": "2024-02-22T08:34:18.999Z",
  "type": "track",

The Segment spec specifies this should be an object: https://segment.com/docs/connections/spec/track/
As an interim workaround, a plugin can be used to change the format:
class PropertyFixPlugin extends Plugin {
  PropertyFixPlugin() : super(PluginType.enrichment);

  @override
  void configure(Analytics analytics) {
    super.configure(analytics);

    if (kIsWeb) {
      return;
    }
  }

  @override
  Future<RawEvent?> execute(RawEvent event) async {
    var local = event.type.toString();
    if (local == 'track') {
      var track = event as TrackEvent;
      final properties = track.properties;

      if (properties == null) {
        track.properties = {};
      }
    }

    return event;
  }

}

Hi @niallzato thank you for your report. We start looking into this.


Bump DMD to 2.067.0.

We really need an URL always pointing to the latest version.

The Dub and DDox versions referred to by this repo build fine with 2.067.0 then?

Oh, I didn't think of that. Why doesn't it use a pre-compiled version of Dub?

I had to update Dub to 0.9.23.

Does this fix anything for you? I can still build dlang.org fine without this pull...

Hmm, looks like I was a bit too quick to create this pull request. It actually do build for me without this PR as well. Someone had a problem in the newsgroup which I thought was related to this. I also misinterpreted how the makefile works. Apparently STABLE_DMD_VER is only used to build Dub. Initially I did have some problems building but I think that was rather a problem with me not having cleaned the dmd, druntime and phobos directories.

We really need an URL always pointing to the latest version.

http://ftp.digitalmars.com/LATEST

That's awesome, thanks.

We added those to always know the latest version in Travis-CI.


Locking Clause Support for MySQL 5.7 and below

Describe the feature
Support LOCK IN SHARE MODE clause.
Motivation
Currently, only two locking clauses are supported FOR SHARE and FOR UPDATE. However, in MySQL 5.7, FOR SHARE clause is not yet supported, instead LOCK IN SHARE MODE is used.

I can help to add in this feature and submit a pull request.

https://github.com/go-gorm/mysql/blob/61d41cd294a4609e361e4e0815f2d1d8afe79db1/mysql.go#L96
You can implement it in the above link, if current mysql version doesn't support FOR SHARE (https://github.com/go-gorm/mysql/blob/61d41cd294a4609e361e4e0815f2d1d8afe79db1/mysql.go#L74) then use LOCK IN SHARE MODE instead

PR is in https://github.com/go-gorm/mysql/pull/11

Merged, thank you.


Support auth to NSX-T with VMware Identity Manager

Description
The CPI currently does not support log in via username/password with NSX-T manager which is integrated with VMware Identity Manager (vIDM).
This pull requests adds this support via the setting of the deployment property vcenter.nsxt.remote_auth to true
Impacted Areas in Application
The authorization header would be set to Remote if the property vcenter.nsxt.remote_auth is set to ture or will default false and set the to header to Basic
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How Has This Been Tested?

[x] config_spec.rb unit test updated to include context when a valid nsxt with remote auth is passed in config
[x] nsxt_auth.rb unit test to ensure backwards compatibility of the Authorization header if the remote_auth property is nil or false
[x] nsxt_auth.rb unit test to ensure Authorization is set correctly when remote_auth property is true

Tested in lab environment against version 2.4 of VMware NSX-T and vSphere version 6.7

[x] Ensure auth header remained unchanged if remote_auth not set in deployment manifest
[x] Ensure login to NSX-T via vIDM when remote_auth set to true.

Checklist:

[x] My code follows the standard ruby style guide
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

@imukadam : Can you add the relevant unit tests?
You can run unit tests from the REPO_ROOT/src/vsphere_cpi/bin/test-unit

@imukadam : Post writing unit tests, could you squash all your commits into one? remove ssl override commit just seems like undoing some test change in the main commit.
1. git rebase -i HEAD~~~
2. Change all commits prefix to `squash` from `pick` except your first commit.
3. Save, resolve conflicts and force push

@EleanorRigby Thanks for the comments. Unit tests and commits squashed

@EleanorRigby we have received the CLA from @imukadam so the bot should be happy in 15 minutes or so

@chipchilders @imukadam : Thanks for the prompt submissions.
There is currently no setup to test vIDM integration with CPI team. The PR is going to take time to merge as we would have to automate the vIDM deployment and then write some e2e test with it.
How soon you require this?

@EleanorRigby I'll need it for our prod deployment. I have be running off the forked repo in our lab environment but looking to go to prod in a few weeks.


502 badgateway with response 401 status code - Issue only when Auth for App service turned on

We have an app-service default.htm page to which we are loading the chat bot using direct-line keys. PFB code used for same in default.html scripts.
BotChat.App({
bot: bot,
locale: params['locale'],
resize: 'detect',
// sendTyping: true,    // defaults to false. set to true to send 'typing' activities to bot (and other users) when user is typing
speechOptions: speechOptions,
user: user,
directLine: {
domain: params['domain'],
secret: key,
// token: params['t'],
webSocket: params['webSocket'] && params['webSocket'] === 'false' // defaults to true
}
}, document.getElementById('chatbotdiv'));
When the host app-service authentication is set to anonymous, The bot works fine when accessing the app-service url, we can see the chat window with results.
However when we are turning on the authentication to Login via AD for the app service, the bot is no longer working. Getting a 502 error in console and upon inspecting response can see 401 unauthorized. Kindly suggest!! Everything works fine when we remove the authentication option on Appservice, but we can't do that since we require the site to be protected.
Botbuilder versions tried: 3.15.2  &  <IP_ADDRESS>969
[BotAuthentication] attribute provided. Web config contains  values for

Hi @GayathriR2018, the issue is that the webpage and bot's endpoint are on the same service. When you enable AAD authentication to protect the webpage, it's enabling AAD auth for the bot's endpoint as well. This won't work because the service-to-service auth used to protect the endpoint is different from user authentication.
To enable AAD authentication on your website, you will need to move your bot's endpoint so it's not affected by this setting.


Setup Google Analytics for the site

Account created. The snippet we'll be using is:
<script>
  (function(i,s,o,g,r,a,m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){
  (i[r].q=i[r].q||[]).push(arguments)},i[r].l=1*new Date();a=s.createElement(o),
  m=s.getElementsByTagName(o)[0];a.async=1;a.src=g;m.parentNode.insertBefore(a,m)
  })(window,document,'script','https://www.google-analytics.com/analytics.js','ga');

  ga('create', 'UA-98793201-1', 'auto');
  ga('send', 'pageview');

</script>

Or, you know, maybe we can convince @philipwalton to give us a nice AutoTrack setup...

That would also be welcome :)

We have basic tracking in. @philipwalton if you'd like to switch to autotrack on https://github.com/GoogleChrome/workbox-microsite I'd be more than happy to take that PR ;)


build should fail when broken links found

🚀 Feature
I would expect npm run build to break upon broken links found, but it just exits with code 0.
Have you read the [Contributing Guidelines on issues]
yes
Motivation
It would not allow publication of broken docs (doh!)
Pitch
Self explanatory

I think this should throw an error code when CI=true

That's already the case and we even have config for that.
https://v2.docusaurus.io/docs/docusaurus.config.js/#onbrokenlinks
100% sure it works if you use latest Docusaurus 2 versions.

Pfff...totally overlooked. Thanks!


KAFKA-6604; ReplicaManager should not remove partitions on the log directory from high watermark checkpoint file

Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Ping @becketqin @ijuma @hachikuji for review.

@lindong28 Thanks for the PR, meta-comment: you are supposed to replace the text in italics in the PR description. :)

@becketqin @ijuma Thanks much for your review!
I have updated the patch and the PR description as suggested.

@lindong28 : Thanks for the patch. I am not sure if this patch is the best way to fix this issue. Currently, when a broker is restarted, we expect the very first LeaderAndIsrRequest to contain all the replicas assigned to this broker. However, this is not strongly guaranteed, which could lead to this particular issue. We discussed another way while working on controller improvement. We can let broker register an epoch in ZK during restart. The epoch changes on every broker restart. The controller includes the broker epoch in the LeaderAndIsrRequest. The broker rejects a LeaderAndIsrRequest if the broker epoch in the request doesn't match its own. This is a more complicated fix, but solves a broader set of of issues including this one. For example, this will also address the issue that Jiangjie raised.

@junrao Thanks for the suggestion. Just to clarify , the suggested approach requires a znode for maintaining global broker epoch, controller should include broker epoch in the LeaderAndIsrRequest and broker should reject the request if brokerEpoch is not expected. In comparison to the approach in this patch, the immediate benefit of using brokerEpoch is that we can delete log directories for partitions that are stopped when the broker is offline. There maybe other benefits in the future.
If it sounds correct, I will draft a KIP to propose this change.

@lindong28 : Yes, your understand is correct. About the broker epoch. Onur mentioned the following in KAFKA-1120.
"James Cheng I think Jun's comments and the redesign doc in KAFKA-5027 are sort of saying the same thing. The broker-generation concept has two use cases which was sort of implied:

the controller using broker generations to distinguish events from a broker across generations.
controller-to-broker requests should include broker generation so that brokers can ignore requests that applied to its former generation.

While I think czxid's will work for the 1st use case, I don't think we can naively reuse czxid for the 2nd use case. The reason is a bit silly: zookeeper's CreateResponse only provides the path. It doesn't provide the created znode's Stat, So you have to do a later lookup to find out the znode's czxid.
If we want to solve both use cases with the same approach, I think we have a couple of options:

maybe we can get away with using czxids by doing a multi-op when registering brokers to transactionally create a znode and read that same znode to read the czxid of the znode it just created.
we can instead use the session id as the broker generation. The controller can infer the broker's generation by observing the broker znode's ephemeralOwner property. Brokers can determine their generation id by looking up the underlying zookeeper client's session id which is just ZooKeeper.getSessionId(). The ephemeralOwner of an ephemeral znode its the client's session id which is why this would work."

So, potentially, we could use the ZK session id. If we go with this route, perhaps we can just fix both issues mentioned in KAFKA-1120.


ISPN-13500 CLI install

https://issues.redhat.com/browse/ISPN-13500
Adds a dependency on org.apache.commons:commons-compress

@tristantarrant I will test this with quarkus

It needs the image patch too

I have integrated Don's commit too

I have fixed the issue with installation from the image https://github.com/infinispan/infinispan-images/pull/157#discussion_r754352016

I've added a -r/--retries=num option. If a cached/downloaded artifact fails the checksum verification, it is discarded and redownloaded up to num times.

Added a --clean option which cleans the server/lib directory

I've made a small change so that the maven settings.xml can also be referenced as a URL and it will be downloaded, e.g.
./bin/cli.sh install --maven-settings=https://raw.githubusercontent.com/infinispan/infinispan/main/maven-settings.xml ...

Thanks @tristantarrant


Added Library Mappings and VCS Root

These two files are automatically modified each time the project is opened (specially in different OSs), and do not affect the project if ignored.

Thanks!


Better function name for process_span

Description
This PR renames the function name : process_span to better reflect the functionality, adds additional logging for ground truth to help debug the latest grammar changes.
Type of change
Please check the options that are relevant.

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] Proposes a change (non-breaking change that isn't necessarily a bug)
[x] Refactor
[ ] New feature (non-breaking change that adds a new functionality)
[ ] Breaking change (fix or feature that would break some existing functionality downstream)
[ ] This is a unit test
[ ] Documentation only change

Type of requested review

[ ] I want a thorough review of the implementation.
[x] I want a high level review.
[ ] I want a deep design review.

Testing
Tested the unit test suite locally.
Checklist:

[x] I have performed manual end-to-end testing of the feature in my environment.
[ ] I have added Docstrings and comments to the code.
[ ] I have made changes to existing documentation where needed.
[ ] I have added tests that show that the PR is functional.
[x] New and existing unit tests pass locally with my changes.
[ ] I have added relevant collaborators to review the PR before merge.

Added a comment, to be clear : process_spans isn't a new fix. This function used to always substitute the string values for indices that come in the logical form.


[core-amqp] close session after sender and receiver in requestResponseLink

Fixes #9615
Description
This change ensures that when a RequestResponseLink is closed, the sender and receiver are closed before the session.
What was happening before?
Previously, when this.sender.close() was called, the session was also closed.
Why was this a problem?
Interestingly, after the sender and session were closed, I'd see a new see a new message arrive on the (cbs) receiver followed immediately by a new session being created. This appears to be in response to the EventHubConsumerClient calling receiveBatch, and triggering a race condition where the cbs receiver hasn't been explicitly closed yet, but the session has been.
Next this.receiver.close() is called, and viewed as already being closed.
I can tell from the logs that the session the cbs receiver was attached to was session 0, and the error from the service says it can't find the session channel '0', so I believe we're hitting a race condition where there's still some activity on the receiver link just after/while the session is being closed.
The fix?
rhea-promise let's us opt out of closing the session when calling sender.close() or receiver.close(). This leads to a more graceful close of the RequestResponseLink as we can be sure all in-flight bytes on the sender and receiver links are handled before closing the session.

How is a cbs session set up for an AQMP receiver link behind a receiveBatch() call related to closing the RequestResponseLink for the $management link?

I'll merge so you can test if this fixes the issue in your service-bus PR, but will continue root causing what exactly is going on here.

I was able to dive deeper into the rhea/core-amqp logs to better understand what was happening. Turned out I needed some of the core-amqp and event-hubs logs to fully make sense of it.
In the sample script, here's the general sequence of events without the fix:

Create cbs session as part of creating ManagementClient.

Connection opened.

cbs session, sender, receiver created and opened.

Call negotiateClaim for $management.

Send $cbs request and receive accepted response.

$management session, sender, receiver created and opened.

ManagementClient gets HubRuntimeInfo

EventHubConsumerCleint creates an EventProcessor, creates a new PartitionPump for partition '0'.

Receiver for partition '0' uses existing cbs session to negotiateClaim (audience is the partition address).

maxWaitTime of 0 is hit, processEvent is called, we now call subscription.close().

EventProcessor is shutdown. Note however that we don't support cancelling receiver initialization, so this will continue.

Now we call client.close().

The cbs RequestResponseLink is closed first.

this.sender.close(): sends detach frame on session '0', receives detach frame on session '0', sender closed.
closeSession is not set, so now session is closed as well. send end frame on session '0'.
cbs receiver receives the message from the partition receiver's negotiateClaim call.
partition receiver's initialization creates a new session (this will eventually time out after ~60 seconds) <- Seems to be the issue
receives end frame on session '0' from service.
this.receiver.close() and this.session.close() called, both effectively no-ops.

The management RequestResponseLink is closed.

Start receiving detach frames on session '1' indicating that session channel '0' cannot be found in connection.

Why does explicitly closing the session after closing the sender/receiver help?
Steps 1-12 happen exactly the same.
In step 13, after we close the sender, the session is not closed. Instead, we close the receiver next.
Right after sending the detach frame for the cbs receiver, we still receive the message from the partition receiver's negotiateClaim call. However this time since the cbs receiver link is being closed, the message is not forwarded to the partition receiver's initialization and a new session is never created.
What does this mean?
It seems that attempting to create the new session for the partition receiver is causing problems when the cbs session has been removed. My theory is that since the new session is being created using the claim from the cbs session that no longer exists, the service is coming back with an error saying it can't find the cbs session (which is channel 0).
What else did you try?
In event-hubs, I changed the order such that the managementSession is closed before the cbsSession and this error also goes away. The reason it goes away though is because the partition receiver session has time to be fully opened while the managementSession is closing down, before the cbsSession starts closing.
I also tested what happens when the managementSession is not closed, and only the cbsSession is closed. I still saw detach frames coming back about the missing session channel '0' in on session '1' (the management session), so I think these errors are coming in out-of-band from the 'close' calls we're making on the managementSession sender/receiver/session.

My theory is that since the new session is being created using the claim from the cbs session that no longer exists, the service is coming back with an error saying it can't find the cbs session (which is channel 0).

Let's confirm this with someone from the service team. We know for sure that the error for missing channel is coming from the service right?


Refine Extensions

Current coverage is 86.80%

Merging #58 into master will increase coverage by +0.33% as of a7c6269

@@            master    #58   diff @@
=====================================
  Files            4      4
  Stmts          451    447     -4
  Branches         0      0
  Methods
=====================================
- Hit            390    388     -2
  Partial          0      0
+ Missed          61     59     -2

Review entire Coverage Diff as of a7c6269

Powered by Codecov. Updated on successful CI builds.


publish new version crate

would be nice if you guys could publish a new release crate with a version bump. i tried running the examples in the dir and they don't compile on 0.1.0, i had to check out the version from latest master branch. thanks

@tdudz I did not get a notification for this issue. It seems I have to adjust the settings. I plan to release a new version the next weeks.

it's ok, i wont be using this anymore


Question about highlight

"Add PDF level highlighting (instead of using a pen tool, select and highlight PDF text) ".
Is this distant to an early implementation? Is that it seems extremely important to be able to generate annotations more personalized I have been working on your pod and I find it difficult to continue without this I have taken a look at your code but I am not so expert yet, in the language.

The annotation system on whole is currently being reworked to allow for greater flexibility and performing document level writing of the annotations. This is the main priority currently. Enabling highlighting my selecting text is further down the road because it involves parsing the text elements as well to be able to gather their location on the page. What is the use case you are trying to accomplish?


Access Participant Registration Information

It's not possible to access the registration information (name, email) for teams on TIRA. This makes it impossible for external organizers to contact their participants.

[ ] Add a way for organizers to download a list of registrants
[ ] Add a way for organizers to see contact information (in the run-inspect component?)

How urgent is this? We can implement this very fast via an cli export, it would require a bit more time to get this as API to production.

The other (external) organizers at PAN might need this ad-hoc and without access to our internals.
I think a simple list-download via button would suffice for now (if this is not too much work; I don't want to dictate your priorities)

I have added the REST endpoint that makes this export to the API.
Deploying this might take a while, because one of the recent tickets (I think it was this one) introduced a regression on the task page that we would have to fix first.
Hence, I exported the data locally and did send it via mail, I think this is ok for the moment?
Best regards,
Maik


Detect the usage of Message.Builder and advice using ...VBuilder in the codebase

The validating builders are created to eliminate potential programming mistakes on building the Message violating the validation rules. Therefore we need to provide a way to track the cases where a regular Builder is used and warn the end-user on that.
It seems that creating a custom Error Prone check is the way to do it.
Ideally we should be able to configure a severity of the bug (warning, error etc) per Gradle module or even per package, if possible.
Also a way to suppress this check per-expression/per-class is needed — in case we really need to use a Protobuf-generated builder of Message. For instance the test code of our validation framework has some invalid Message instances created on purpose using the Message.Builder mechanism.

Closed via https://github.com/SpineEventEngine/base/pull/180.


update prop, open to visable

What does this PR do and why?
https://ant.design/components/dropdown/
Prop change is needed for new API:
Why we need align pop component with open prop?

For historical reasons, the display names of the pop components are not uniform, and both open and visible are used. This makes the memory cost that non-tsx users encounter when developing. It also leads to ambiguity about what name to choose when adding a feature. So we want to unify the attribute name, you can still use the original visible and it will still be backward compatible, but we will remove this attribute from the documentation as of v5.

Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] I have tested this PR in all supported browsers.
[ ] I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #2104

PR #2098

PR #2090

PR #2089

PR #2091

PR #2093

PR #2095

PR #2167   👈
PR #2097

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: OhSplG4sLw5ccXyOTEUT


macos下使用3.8cmake和3.2opencv用build.sh时出现问题

[ ] help wanted
yuboxideMacBook-Pro:EasyPR-master  cavendishforscut$ ./build.sh
CMake Warning (dev) at CMakeLists.txt:50:
Syntax Warning in cmake code at column 32
Argument not separated from preceding token by whitespace.
This warning is for project developers.  Use -Wno-dev to suppress it.
CMake Warning (dev) at CMakeLists.txt:50:
Syntax Warning in cmake code at column 38
Argument not separated from preceding token by whitespace.
This warning is for project developers.  Use -Wno-dev to suppress it.
CMake Warning (dev) at CMakeLists.txt:51:
Syntax Warning in cmake code at column 26
Argument not separated from preceding token by whitespace.
This warning is for project developers.  Use -Wno-dev to suppress it.
CMake Warning (dev) at CMakeLists.txt:51:
Syntax Warning in cmake code at column 30
Argument not separated from preceding token by whitespace.
This warning is for project developers.  Use -Wno-dev to suppress it.
CMake Warning (dev) at CMakeLists.txt:52:
Syntax Warning in cmake code at column 36
Argument not separated from preceding token by whitespace.
This warning is for project developers.  Use -Wno-dev to suppress it.
CMake Warning (dev) at CMakeLists.txt:52:
Syntax Warning in cmake code at column 41
Argument not separated from preceding token by whitespace.
This warning is for project developers.  Use -Wno-dev to suppress it.
CMake Warning (dev) at CMakeLists.txt:53:
Syntax Warning in cmake code at column 26
Argument not separated from preceding token by whitespace.
This warning is for project developers.  Use -Wno-dev to suppress it.
CMake Warning (dev) at CMakeLists.txt:53:
Syntax Warning in cmake code at column 30
Argument not separated from preceding token by whitespace.
This warning is for project developers.  Use -Wno-dev to suppress it.
CMake Warning (dev) at CMakeLists.txt:64:
Syntax Warning in cmake code at column 80
Argument not separated from preceding token by whitespace.
This warning is for project developers.  Use -Wno-dev to suppress it.
CMake Error at CMakeLists.txt:65:
Parse error.  Function missing ending ")".  Instead found unterminated
string with text ")
".
-- Configuring incomplete, errors occurred!
make: *** No targets specified and no makefile found.  Stop.

应该是cmake版本或者是编码的原因

如果这个问题依然存在，可以open这个issue。


[FLINK-1168] Adds multi-char field delimiter support for CsvInputFormats

This commit includes parts of Cbro's pull request and subsumes PR #247
This closes #247

Are there other opinions on this?

Since you added a charset for the delimiter, it would be good to have this consistent with the String conversion.

Either both cannot be configured and are only applicable to UTF-8
Or we define a charset for the input format as a whole, which is used for delimiter and StringParser

That's a good point. I copied the code from the DelimitedInputFormat which allows to specify the charset for the record delimiter.
So if we go with

we should also fix the DelimitedIF to UTF-8.
we need to extend the StringValue which does not support charsets.

would be the easy fix. Charset support could be added later. Any objections with going for 1)?

+1 for going for (1)

Updated the PR and will merge once Travis completed the build.


[Feature request] image_to_image+contorlNet with onediff & deepcache

Hello,
It seems like there's no deepcache version of image to image in the examples.
Here's the text_to_image version:
onediff_diffusers_extensions/examples/text_to_image_deep_cache_sd.py
Is that possible to provide a image to image with ContorlNet example?
Thank you in advance.

there's no deepcache version of image to image in the examples.

Yes. deepcache is an experimental feature so we only put it in the text to image.

image to image with ContorlNet example

https://github.com/siliconflow/onediff/blob/main/onediff_diffusers_extensions/examples/image_to_image_controlnet.py


Add quotes to TTA promo page

Trello card
Trello-4012
Context
We want to add a list of quotes to the TTA promo page to replace the accordion component that was recently removed.
Changes proposed in this pull request

Remove unused TTA page partials

Add quotes to the TTA promo page

Add QuoteListComponent

We want to display a list of quotes on some of our content pages; this is similar to the PhotoQuoteListComponent but without the photos and with the heading half overlapping the containing box. I initially tried to combine the two quote list components but it became awkward due to the overlapping heading on the non-photo variant, so I've kept them separate.
Guidance to review
TTA page

Desktop
Mobile

@gemmadallmandfe do we need to run these past Andy first?


How to serialize/save and load/deserialize

Hi,
I have attempted to serialize/save using the following code:
std::ofstream outfile("outfile.dat", std::ofstream::binary);
outfile.write(reinterpret_cast<const char*>(compressed_output.data() /* or &v[0] pre-C++11 */), sizeof(uint32_t) * compressed_output.size());
outfile.close();```

 I am not sure this is the right way because I am ending with a file that is larger (~2x) than what is reported by:
```c++
  std::cout << std::setprecision(3);
  std::cout << "You are using "
            << 32.0 * static_cast<double>(compressed_output.size()) /
                   static_cast<double>(mydata.size())
            << " bits per integer. " << std::endl;```

For instance, I get `You are using 9.11 bits per integer` with `96504` integers but my output file is `-rw-r--r--  1 root root 215K May 10 18:58 outfile.dat`

What am I doing wrong? Also, how would you load/deserialize from this file?

Thanks for your help.

What is the value of sizeof(uint32_t) * compressed_output.size()?

The value is 219700

Hence compressed_output.size() is 54925.
What is mydata.size()? You wrote that the program computed that 32.0 * static_cast<double>(compressed_output.size()) /  static_cast<double>(mydata.size()) is 9.11.
It should be that mydata.size() is about 193008. But you wrote that you had 96504 integers.
One of these numbers is wrong.

 Found the issue in my code now I am getting a larger bits per integer (18.1) which is expected. Thanks!


Codegen storage markers as assignments to __CPROVER_dead_object

This changes our handling of storage markers to be marking is-alive only rather than treating StorageLive as creating a new object. That is, object instances are now tied to their Mir-provided declarations (which, at present, only appear once per function). To still account for when Rust scopes deem an object to be alive, we use StorageLive and StorageDead to update __CPROVER_dead_object. This (global) variable is used by CBMC's pointer checks to track when a pointer may not be safe to dereference for it could be pointing to an object that no longer is in scope.
Resolves: #3099
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

This still requires a fix on the CBMC side for contracts instrumentation not to fail.

Closing as the changes are now included in #2995.


Add initial dump of drupal contrib SA

Here is an initial dump of Drupal contrib SA.
This fixes #366
I will clean up the script and publish as its own when I get the time.
Edit: Just to clarify: I made the script to parse the latest advisories published. It could theoretically go back in history more, but the format changed some time in 2017. So instead of trying to add parse exceptions, I am going to assume that people with projects not updated for a couple of years, in spite advisories, would not use this repo and its tooling anyway. Open to comments and suggestions on that :)

Probably we should revert back this PR considering we got https://github.com/drupal-composer/drupal-security-advisories for this which has more details of drupal packages and causing issues like https://github.com/Roave/SecurityAdvisories/issues/54

I think this PR is okay because it explicitly defines which repository the packages come from, which Roave/SecurityAdvisories#54 is going to key off of.

Thanks @colinodell.  thought it's an overlap as https://github.com/drupal-composer/drupal-security-advisories reading these details from drupal.org. I will follow progress at https://github.com/Roave/SecurityAdvisories/issues/54

reverted


Cover page: typo and consistency

Fix typo: "One can become a contributor by joining the Rule Book Working Group as member of International Data Spaces Associatoin."
Also we should use "IDSA Rulebook" consistently. Now there are different versions "IDSA rulebook",  "IDSA Rule Book", etc.

Should be "IDS Rulebook" as the name of the document to be used.


Exception when getting element from .NETCore that are added by .NET.

Hi, I'm trying to setup and environment where a console app, based on .NET461, adds elements to a cache, elements that are getted by a .NETCore 3.1 web app. The console app uses the ICache interface, while the .NETCore web app uses the IDIstributedCache interface. All packages are updated to the latest available. When the .NETCore web app executes the GetString() method, an exception arises from the NCache subsystem and no element returns.
The relevant piece of code of the console app is:
string cacheKey = $"elm:pagestree|idcn:{idCN}";
string cacheItem = Newtonsoft.Json.JsonConvert.SerializeObject(shadowPagesTreeList, Newtonsoft.Json.Formatting.None);
if (cacheManager.Contains(cacheKey))
{
cacheManager.Remove(cacheKey);
}
cacheManager.Insert(cacheKey, cacheItem);
The relevant piece of code of the web app is:
string cacheKey = $"elm:pagestree|idcn:{idCN}";
var cacheItem = _distributedCache.GetString(cacheKey);
if (cacheItem != null)
{
l = System.Text.Json.JsonSerializer.Deserialize<List<Models.PagesTreeElement>>(cacheItem);
}
The exception arises when executing the GetString() or Get() methods and is:
_System.Configuration.Provider.ProviderException
HResult=0x80131500
Message=Unable to cast object of type 'System.String' to type 'System.Byte[]'.
Source=Alachisoft.NCache.DistributedCache
StackTrace:
at Alachisoft.NCache.Caching.Distributed.NCacheDistributedCache.RaiseException(Exception exc)
at Alachisoft.NCache.Caching.Distributed.NCacheDistributedCache.LogError(Exception exception)
at Alachisoft.NCache.Caching.Distributed.NCacheDistributedCache.Get(String key)
at Microsoft.Extensions.Caching.Distributed.DistributedCacheExtensions.GetString(IDistributedCache cache, String key)
at IsibetPRO.IsiPlanner.ServiceWebApp.Controllers.MainController.GetPagesTree(Byte appType) in D:\Users\andre\Source\Repos\IsiPlanner\IsibetPRO.IsiPlanner.ServiceWebApp\Controllers\MainController.cs:line 262
at Microsoft.Extensions.Internal.ObjectMethodExecutor.Execute(Object target, Object[] parameters)
at Microsoft.AspNetCore.Mvc.Infrastructure.ActionMethodExecutor.SyncActionResultExecutor.Execute(IActionResultTypeMapper mapper, ObjectMethodExecutor executor, Object controller, Object[] arguments)
at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.InvokeActionMethodAsync()
at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.InvokeNextActionFilterAsync()
This exception was originally thrown at this call stack:
[External Code]
Inner Exception 1:
InvalidCastException: Unable to cast object of type 'System.String' to type 'System.Byte[]'._
The cache, server side, is configured as in file attached.
Please note that other combinations of data-format and serialization parameters don't resolve.
If the web app itself adds and gets elements from the cache, all works as expected.
Thanks in advance, Andrea Valori.
config.ncconf.txt

@MeValori, As per our discussion over the support ticket, I have executed the application on my end again in Debug mode and here are my findings.
During the testing, I was unable to reproduce the error that you're seeing on your end. However, I noticed a behavior where after adding the items in the Cache through the .NET Framework  4.6.1 console app, I get a "null" value from the "idistributedcache" using the .NET Core 3.1 web app which uses NCache IDistributedCache Provider.
The IDistributedCache Provider approach is used by those users who want to stick to it. I've noticed that you are adding the items in the Cache using NCache Direct API i.e. "_cache.Insert()" and then trying to retrieve those items using IDistributedCache Provider. I need to discuss this with the Engineering team if they know about any known specific behaviors in this particular scenario.
Moreover, I tried to modify your sample application where it uses NCache Direct APIs (i.e. _cache.Get() & _cache.Insert()) in both applications (.NET Framework 4.6.1 & .NET Core 3.1) and it is working without any issues. You can easily retrieve the items from Cache in this case. Here are some code snippets that I have used on my end for this.
// Insert the item in the Cache using Insert() API.
if (cacheManager.Contains(cacheKey))
{
     LogInstance.Debug($"66D43421-> Item with key {cacheKey} in cache {Properties.Settings.Default.NCacheName} will be updated.");
}
cacheManager.Insert(cacheKey, cacheItem);

// Retrieve the item from NCache using Get() API.
string cacheItem;
try
{
     cacheItem = distributedCache.Get<string>(cacheKey);
}

I have also noticed that you're using the JSON serialization/deserialization process explicitly while adding and retrieving the items from the cache. This could be a requirement for IDistributedCache but please note that it is not needed for NCache. All items are serialized/deserialized automatically by NCache during the insertion/retrieval process. In the case of Binary serialized cache, you need to mark the entities that you want to cache as "Serializable" and NCache will handle everything else itself from there. NCache also supports "JSON" serialization mode for the Cache which does not require the entities to be marked as "Serializable".
Hope this helps.


PRO-2086 documentation-driven development (README comes first)

Sample report at the end. This isn't meant to be perfect, just some meaningful output to give developers a reassurance that it did some work and also illustrate the code migration task before them.
Complete!

Doc types inserted:

@apostrophecms/archive-page
@apostrophecms/global
@apostrophecms/image
apostrophe-blog
apostrophe-blog-page
apostrophe-redirect
author
blog-category
case-studies-page
case-study
custom-cta
default
home
styleguide
zapier-pieces

Widget types inserted:

@apostrophecms/image
@apostrophecms/rich-text
@apostrophecms/video
big-button
breaker-cta
button
callout
card
case-study
code-editor
cta
custom-ctas
fancy-list
full-width
image
landing-callout
landing-marquee
link-cards
list
logos
marquee
mega-button
navigation
newsletter
one-column
one-column-large
padded
side-by-side
simple-cta
single-testimonial
tabs
terminal
testimonials
three-five-four
tiles
two-col-tabs
two-column
two-column-marquee
two-up
video-marquee
video-set


Adding the possibility to use different sender id

Sometimes we need to use different sender id for different messages

I'm confident you have opened a PR to the wrong repository.

I'm confident you have opened a PR to the wrong repository.

Yes you are right. Thank you

Open on the wrong project. Was supposed to open this pull request on a fork of this project


SingleHtmlBuilder is broken w/ patch

Details
I am trying to produce a project with as i would with Sphinx's make singlehtml, however with my internal build of RTD.org, the effect is the same as make html. I found the issue in backends/spinx.py which was created by this commit: https://github.com/rtfd/readthedocs.org/commit/ed12559ab3ee3e867387c8cbe75d6471a8db79b3
In the previous commit, an __init__ method was added to the HtmlBuilder and the HtmlDirBuilder builders, however the SingleHtmlBuilder was unmodified. Since SingleHtmlBuilder inherits from HtmlBuilder , HtmlBuilder.initis executed and sets thesinglehtml_builder_instance.sphinx_buildtoreadthedocsorreadthedocs-comments. Which makes it impossible to get the correct value of shpinx_builderfrom the class;SingleHtmlBuilder.sphinx_buidler == 'readthedocssinglehtml'`.
The fix, found below, is to add a __init__ method to SingleHtmlBuilder and set self.sphinx_builder appropriately.

Project URL: n/a
Build URL (if applicable): n/a
Read the Docs username (if applicable): n/a

Expected Result

Set a project's Documentation type: to Sphinx Single Page Html.
Build docs.
Expect to see a single page of documentation.

Actual Result

Actual result is the normal Sphinx Html page is built.

Patch
---
 readthedocs/doc_builder/backends/sphinx.py | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/readthedocs/doc_builder/backends/sphinx.py b/readthedocs/doc_builder/backends/sphinx.py
index 3bd95b8..780034b 100644
--- a/readthedocs/doc_builder/backends/sphinx.py
+++ b/readthedocs/doc_builder/backends/sphinx.py
@@ -183,7 +183,10 @@ class HtmlDirBuilder(HtmlBuilder):

 class SingleHtmlBuilder(HtmlBuilder):
     type = 'sphinx_singlehtml'
-    sphinx_builder = 'readthedocssinglehtml'
+
+    def __init__(self, *args, **kwargs):
+        super(SingleHtmlBuilder, self).__init__(*args, **kwargs)
+        self.sphinx_builder = 'readthedocssinglehtml'

 class SearchBuilder(BaseSphinx):
--

Closing this as fixed since #2527 has already been merged.


Light With castShadow enabled, models in the scene are no longer visible

Hi,
I tried to add a cast, but it didn't work.
The version used is Android Studio Giraffe | 2022.3.1 Patch 3.
Running on a Huawei HONOR 9X real machine.
The code is as follows:
`
Future setup() async {
threeJs.camera =
three.PerspectiveCamera(30, threeJs.width / threeJs.height, 1, 2200);
threeJs.camera.position.setValues(cameraPos.x, cameraPos.y, cameraPos.z);
threeJs.scene = three.Scene();
threeJs.camera.lookAt(threeJs.scene.position);
threeJs.scene.add(threeJs.camera);
final light = three.DirectionalLight(0xffffff, 1);
light.position.setValues(5, 10, 7);
// light.castShadow = true; // You can't see it if you open it here
threeJs.scene.add(light);

three.PlaneGeometry planeGeometry = three.PlaneGeometry(100, 100);
three.MeshBasicMaterial planeMaterial =
    three.MeshBasicMaterial({three.MaterialProperty.color: 0xaaaaaa});
three.Mesh plane = three.Mesh(planeGeometry, planeMaterial);
plane.receiveShadow = true;
threeJs.scene.add(plane);

three.MeshStandardMaterial material =
    three.MeshStandardMaterial({three.MaterialProperty.color: 0x00ff00});
three.Mesh model = three.Mesh(three.BoxGeometry(1, 1, 1), material);
model.castShadow = true;
model.position.y = 1;
threeJs.scene.add(model);

// threeJs.renderer!.autoClear = false;
threeJs.renderer!.shadowMap.enabled = true;
threeJs.postProcessor = ([double? dt]) {
  threeJs.renderer!.setViewport(0, 0, threeJs.width, threeJs.height);
  threeJs.renderer!.clear();
  threeJs.renderer!.render(threeJs.scene, threeJs.camera);
  threeJs.renderer!.clearDepth();
};

}
`

When not turned on, the plane and model display normally.

Turn on the castShadow of the light, neither the plane  nor the model  is displayed

Hi @vincentChen02,
Sorry to hear you are having an issue.
Please remove the post processor from threeJs and initialize it to be
   threeJs = three.ThreeJS(
      onSetupComplete: (){setState(() {});},
      setup: setup,
      settings: three.Settings(
        useSourceTexture: true
      )
    );

Hope this helps.

Thank you very much for your reply @Knightro63 .
Your answer solves the problem perfectly.
Have a good day!

Hi!@Knightro63
I'm sorry to bother you again.
Q1:Why do I need to turn on useSourceTexture to see the light set shadows?
Q2:At present, after useSourceTexture is enabled in the Settings, the shadow can be displayed normally, but the model lighting is slightly different, showing a dark red situation, and adjusting attributes such as roughness has no effect.
If useSourceTexture is not set, the following information is displayed:

If useSourceTexture is set to true, the following is displayed:

Try to modify the light intensity and position, the roughness, metalness, emissive and other properties of the material, can not be adjusted to the same as when not opened, especially the face is dark red.
thank you!

Hi @vincentChen02,
I am sorry you are having an issue with this. The useSourceTexture adds the shadow layer onto the scene. This is required for every platform except for web. I have tried to fix the issue but was unsuccessful.
I will look into this further. The creation of the shadow layer might have too high of a red hex value.
I will update you if I solve the issue.

Look forward to your follow-up version, thank you very much, wish better and better!


[Feat]: Add ability to re-use CSS classes

Description
First off, thank you for your contribution of this library, fantastic stuff and looking forward to v3!
Along that line, I'm currently working on adopting v2 and one thing that would be nice is to have the ability to re-use the CSS classes elsewhere within a site.  Unfortunately, many of the styles are tied to element IDs (e.g., #s-bl) making it (theoretically) impossible to re-use (since id's should be unique on a page).
Possibly this approach is necessary but if not, would like to see CSS classes that can be re-used implemented.
The use case is having a Cookie Policy page that contains the exact same table as the Preferences Modal including the same styling.  To achieve currently, I'm copy/pasting the CSS to work around but this results in performance impact for pages (larger payload size & processing) in addition to maintenance overhead when new versions are released.
Appreciate your consideration!
Proposed solution
Change (or enhance) CSS selectors from element id's to classes (e.g. #s-bl table might become .cc-s-bl table or #s-bl table, table.cc-ct)
Additional details
No response

Sorry, this won't be worked on.
The id is there to raise the CSS selector specificity/strength so that the style of the plugin is not (easily/unintentionally) overridden by other CSS rules. The other (uglier) option would have been to use the !important directive on all CSS rules.


arguments that can accept content area and indicator shouldn't be called content_area

-Need to have argument name that shows it can accept indicator

completed in v1.0.0


Update type signature for SubscriptionHandler. Update associated docs.

This PR would fix #319. As @mjadczak points out, the void type is a mistake there - handler is called in useSubscription with (s.data, data), where s.data is set initially by useImmediateState to undefined: https://github.com/FormidableLabs/urql/blob/master/src/hooks/useSubscription.ts#L36. It also fixes the signature of the mutable ref inside of useRequest, which is initialized to undefined .
I'm also not sure if we were attempting to take advantage of undefined and null being sub-types of void, so feel free to push back on this change (but it seems to make sense to me).

Makes sense 👍
Void isn't actually a type so I think it should be avoided where possible. The only place I've found it valuable is when stating "I don't care about this value" which is almost always exclusive to function arguments and their return types.
interface Props {
  onClick: (e: MouseEvent) => void;
}

@JoviDeCroock yep, the change for useRequest was included as well, it's at the very bottom of the diff 👍

@JoviDeCroock yep, the change for useRequest was included as well, it's at the very bottom of the diff 👍

I looked over it, my bad

Looks like Sail CI may be down?

@parkerziegler seems to be working now. Just a failing snapshot; not sure what happened there but the stack might be inaccurate? I've got an alternative implementation for that since we're launching devtools soon and want that to be super nice and accurate :+1:

@kitten do you want me to merge this and we'll assume that #329 will fix the build once it gets merged? Or wait until #329 is merged, rebase off of it, and then merge?

cc/ @kitten @andyrichardson @JoviDeCroock merging this with the snapshots updated to pass the build. When #329 gets merged we can update those snapshots in tests.


fix: handle filepath error checks differently

in relation to https://github.com/open-rpc/schema-utils-js/issues/447

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ac52ac5). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master        #6   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?         2
  Lines             ?        50
  Branches          ?         5
==========================================
  Hits              ?        50
  Misses            ?         0
  Partials          ?         0

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 ac52ac5...422f855. Read the comment docs.

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

GitHub release
1.0.2
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


removeSelected removes any pre-selected item from dropdown

Bug description:
When linking the ui-select component to a model which points to an item in the items array, the preselected item is not displayed in the dropdown list.
Link to minimally-working plunker that reproduces the issue:
http://plnkr.co/edit/bMlvGYc0Jfq3PX8NQcpv?p=preview
Notice that although 'Amalie' is preselected, she is not displayed in the dropdown list.
Version of Angular, UI-Select, and Bootstrap/Select2/Selectize CSS
Angular: 1.5.0
UI-Select: 0.18.0
Bootstrap/Select2/Selectize CSS (if applicable): 3.1.1

It's worth noting that this situation works fine with 0.17.1.
A temporary fix is to add a remove-selected="false" attribute to the <ui-select> tag.

I think it's not a bug but feature ;-) https://github.com/angular-ui/ui-select/commit/3ad084f
You can switch it back to false globaly in configuration section:
app.config(function(uiSelectConfig) {
  uiSelectConfig.removeSelected = false;
});

@p10 I tried this but it didn't work... I think the config block is run after the uiSelectController is created, and so this code which caches the removeSelected flag is still true.

@dondi any chance you could take a look at this?

OK I'll investigate.

Quick update: I have been able to replicate this behavior and have written a unit test for it. Based on what I can replicate, the previous version (0.17.1) indeed does not remove the item from the dropdown, but the highlighted item when the dropdown is displayed for the first time does not match the pre-selected object either (initial highlighted item is always the first one).
Is that what others saw in the previous release? If this is the behavior that everyone is used to, then even if it is not entirely correct, it will be useful to know that just reverting to this will suffice. However if I’m not seeing the correct prior behavior, or if the preference is to get this working completely right, then that changes the code trajectory a bit.
Please advise—thanks!

From my point of view, I've upgraded from 0.14.1 where the preselected item
was correctly displayed and highlighted, my vote would be to revert to this
expected behaviour. Thanks!

OK, potential fix available at https://github.com/angular-ui/ui-select/pull/1690 …I toggled between master and this to make sure that the incorrect behavior reported here was definitely gone in the new version. All unit tests still pass, plus one more unit test was added specifically for this situation. Hope this does the trick, let me know :)

BTW @fiznool I was able to get this PR to implement the fully correct behavior where the pre-existing selection in the model is indeed highlighted from the get-go in the ui-select element. Thank you!

@p10 For me, it works when changing the value to false in configuration. Thanks ;)

@dondi I've just tested with a manually created build using gulp and all works nicely, thanks!
Could you cut a new release? I'm installing using bower and so I need the dist folder to contain the latest code in order to update my codebase properly.

Greetings @fiznool —Glad to hear your tests look good! I will have to defer to @user378230 with regard to releases.

I have used LimitTo filter in ui-select where i have millions of items in my dropdown. So preselected


Bug: No feedback while saving workspace changes

Describe the bug
When saving changes in a workspace there is no indicator that anything is happening until it is done saving.
Perhaps a loading/disabled state can be added to the button? Or a message indicating that something is happening?
Steps to reproduce

Go to workspaces
Create a new workspace
Give it some details
Make some changes to it
Click "update workspace"
Notice it takes a minute before any feedback is received

Do you have any images or screen captures?
https://github.com/open-sauced/app/assets/67555014/c857cbb6-9dc4-4cb5-a13e-2edcd2724b88
Code of Conduct

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

Contributing Docs

[X] I agree to follow this project's Contribution Docs

We can probably add a processing overlay screen @isabensusan. This wasn't an issue initially as all our tests were with small datasets. Now that you can import an orgs repositories, it really looks likes nothing happening for a bit.

For now, @isabensusan, I opted for a button loading state instead. We can revisit a saving overlay.


texlive-new: build of trivial packageset fails

> nix-build -E 'with import <nixpkgs> {}; with texlive; combine {} '
these derivations will be built:
  /nix/store/si8lxq99l9kmrd631fmi4ldjw91l62ld-hyphens.sed.drv
  /nix/store/0syl7by3hxvrk3n04hxdd27ypija251j-texlive-combined-2015.drv
building path(s) ‘/nix/store/bymh855b6bz3ab79di4dxlk5n7kaf7gb-hyphens.sed’
building path(s) ‘/nix/store/rmsz6f1kvai0iwda6hfl55w97148xs4l-texlive-combined-2015’
created 1 symlinks in user environment
/nix/store/xqlf0dmmzi79mhn7b51shmfqw6lq59y3-stdenv/setup: line 849: cd: ./share/texmf/web2c/: No such file or directory
builder for ‘/nix/store/0syl7by3hxvrk3n04hxdd27ypija251j-texlive-combined-2015.drv’ failed with exit code 1
error: build of ‘/nix/store/0syl7by3hxvrk3n04hxdd27ypija251j-texlive-combined-2015.drv’ failed

(it also fails with a package in the combine attrset)
cc @vcunat

Yes, that's known. See the preliminary user docs: https://github.com/NixOS/nixpkgs/blob/master/pkgs/tools/typesetting/tex/texlive-new/default.nix#L1

Are you sure that’s how it should work? I used exactly the same statement around half a year ago and it worked fine.
Especially those two lines weren’t there the last time I checked:
/nix/store/xqlf0dmmzi79mhn7b51shmfqw6lq59y3-stdenv/setup: line 849: cd: ./share/texmf/web2c/: No such file or directory
builder for ‘/nix/store/q7jhlcqn9cmbdqjkgnax67qk1qwxd2hf-texlive-combined-2015.drv’ failed with exit code

Some things were changed probably, but the TeX stuff is probably useless if you don't plug any packages at all into it. Better fail early.

As I said, `nix-build -E 'with import  {}; with texlive; combine { inherit minted; } ' fails with the same error.

Yes, as that one wouldn't even contain the TeX binaries. (Upstream TeX Live has very holey dependencies among the packages.)

To explain better... the current texlive is a rather thin layer around the upstream packaging. Yes, we could improve it a lot, but I "only" invested a few full weeks of time into this and ATM I'm not inclined into more such TeX endeavors. That certainly doesn't prevent others from improving it.

You mean it’s missing a few dependencies and so the build fails? Can we include those dependencies manually if CTAN doesn’t provide them?

It is possible to override the dependency information manually, and we have a couple of adjustments in there already. CTAN doesn't provide any dependency information, I think. It's TeX Live where I took the information and the packages themselves. (And IIRC Fedora's srpm has some magic code that tries to detect the dependencies.)
To kill the problem that most packages depend on some of the core ones, I recommend in the user docs to always include one of the schemes, e.g. scheme-basic; that's what upstream TeX Live does anyway.


Remote Config version 21.1.1+ in Unity SDK

Unity editor version: 2020.3.33f1
Firebase Unity SDK version: 9.1.0
Source you installed the SDK: .unitypackage
Problematic Firebase Component: Remote Config
Other Firebase Components in use: Analytics, Crashlitics
Additional SDKs you are using:
Platform you are using the Unity editor on: Windows 10 Pro
Platform you are targeting: Android
Scripting Runtime: IL2CPP

When uploading new version to Google Play Console this issue is reported:
Critical issues have been reported with the following SDK versions:
com.google.firebase:firebase-config:21.1.0
What the SDK developer told us:
Bug affecting some locales causing HTTP errors Please downgrade to 21.0.2 or, if available, update to the next version which will contain a fix. For more info please see: https://github.com/firebase/firebase-android-sdk/issues/3757
I didn't find in your changelog which Firebase Unity SDK version includes Remote Config version 21.1.1 that fixes the problem as described here
Will latest version fix this or we need to wait for future versions?

Hi @TellmewowBilbao,
The latest Unity releases typically point to the latest version of the build SDK at the time of release. Version 9.4.0 is currently pointing to Remote Config version 21.1.1, so an SDK update should solve the issue.
That being said, I'll be closing this ticket. Let me know if an issue arises.


Add stale app config

Add configuration YAML for https://github.com/apps/stale.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #38   +/-   ##
=======================================
  Coverage   99.86%   99.86%
=======================================
  Files           8        8
  Lines         738      738
  Branches       99       99
=======================================
  Hits          737      737
  Partials        1        1

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 276c814...ab48313. Read the comment docs.


SUBTOTAL ISSUE STILL IN THERE

Everytime user faced to Doubled Amount subtotal even order just single QTY. Please get this fix.

@ianbresnan - we've consistently been unable to reproduce this issue on a clean instance of magento. Perhaps you can share the amazon diagnostic output? Or, you can share any specific add on extensions you are using that might be conflicting?

@islandskater43 mate... I try the same with Fresh MAGENTO SETUP too. When Add to Cart any product went to AMAZON payout and back to Cart to update something every time this extension Doubled MY CART sub Total.

@islandskater43 mate... I try the same with Fresh MAGENTO SETUP too. When Add to Cart any product went to AMAZON payout and back to Cart to update something every time this extension Doubled MY CART sub Total.

@islandskater43 mate... I try the same with Fresh MAGENTO SETUP too. When Add to Cart any product went to AMAZON payout and back to Cart to update something every time this extension Doubled MY CART sub Total.

@islandskater43 mate... I try the same with Fresh MAGENTO SETUP too. When Add to Cart any product went to AMAZON payout and back to Cart to update something every time this extension Doubled MY CART sub Total.

@islandskater43 mate... I try the same with Fresh MAGENTO SETUP too. When Add to Cart any product went to AMAZON payout and back to Cart to update something every time this extension Doubled MY CART sub Total.

@islandskater43 mate... I try the same with Fresh MAGENTO SETUP too. When Add to Cart any product went to AMAZON payout and back to Cart to update something every time this extension Doubled MY CART sub Total.

@islandskater43 mate... I try the same with Fresh MAGENTO SETUP too. When Add to Cart any product went to AMAZON payout and back to Cart to update something every time this extension Doubled MY CART sub Total.

Can you detail out the steps you take to create a fresh instance? Here are the steps I've followed where I'm unable to replicate:

Launch a new Bitnami instance of Magento 1.9 on AWS.
Install the Pay with Amazon plugin

@ianbresnan - can you answer the above? If we can find a way to successfully replicate the issue, then we can try to resolve. Thanks!

Closing as duplicate of #155. @ianbresnan - please provide additional details in issue #155.


fix(support): accordion slamming on loading was fixed (#1547)

Pull Request Test Coverage Report for Build 1661

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 53.533%

Totals

Change from base Build 1659:
0.0%

Covered Lines:
9421

Relevant Lines:
15619

💛  - Coveralls


Add test command

This adds a command like build called test, for running tests.
This also refactored the BuildSupervisor up into the RunSupervisor. Fixed a few little things like the output printing, and the build-error.log that was always printed out. Thats only there now in --verbose mode.
I also updated the build cache, to cache per runCommand, and to properly load and reload from disk.
These commands now work:
yarn test

yarn test packages/examples/lorem-ipsum

This fixes #35 and #32


fuse-overlayfs: 0.7 -> 0.7.2

Motivation for this change
upstream update - works here.
Things done

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

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

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

Notify maintainers
cc @furrycatherder

looks good to me, lgtm


feat(apps/whale-api): Add consortium transaction history endpoint

What this PR does / why we need it:
Adds an endpoint to be used in defiscan.live
Additional comments?:
Add transactions endpoint - This will cater the transaction history page under consortium

What happens if it searches by TxId that is not part of the consortium or not a mint/burn transaction?
Are there transactions other than mint or burn in the test container? If not, then can we add some noise to the data to check if it searches correctly?

We already have some fund transfering txes in the setup which creates sent type transactions and also blockReward transactions, I guess we can count on those for this query -- however I've added one more test to specifically search for a type: sent txid to see if it return empty list. 👍🏼


Working on new data - same hashes when downloaded one by one or .tar

Hello @mingujo , @ye-zhi
You told me you managed to work with the new data. As I said yesterday, when I do data = img.get_data() I got an error.
Can you put the link of the scripts you did that ?
I must do something wrong.
Also, I checked the hashes of the files you sent me. They are the same as the files I can download one by one. So I will keep the download data one by one since thus takes only 10s/data.
I was thinking I could just unzip them in the same ds005 directory as the original data. what do you think ?
Thanks!

Actually,
I think I have a lead on the data thing.
thanks

https://github.com/berkeley-stat159/project-epsilon/blob/multi/code/utils/scripts/multi_beta_script.py
So in this script, you'll see how i'm using it in the middle.
For me, I downloaded the whole filtered data set under the data folder, unzipped, and named as "ds005_2"
I'm not sure downloading one by one would work.
As long as the hashes are same, they would work.
Approximately, one data is large as ~250MB
Downloading that large file in 10sec must be either your internet is super fast or something is wrong...
@soazig

Great,
I changed the server where I run my code and it is working now.
Also working on the downloaded data one by one, as I said, they have the same hashes so I think it is safe. We can discuss later.
Thank you !


Use OC stackdriver exporter to capture self observability metrics as GCM protos

Collect the OC self observability metrics using the same MetricTestServer (GCM mock server) + stackdriver exporter instead of a bespoke proto format. They now get serialized as they would be exported to GCM.

Why does using the OpenCensus exporter help us? Won't the metrics still have inconsistent values?

Yes it doesn't fix that problem, but i was going down a rabbit hole of trying to add metric type and histogram bucket options to the SelfObservabilityMetric and figured it's easier to just use the GCM proto.
This is also better if/when we switch these self obs metrics to OTel instrumentation, we can just use our OTel exporter to capture the same metrics rather than write conversion to a bespoke protobuf format.

it's easier to just use the GCM proto.

I think i've almost connected the dots.  How does using opencensus relate to using the GCM proto?

How does using opencensus relate to using the GCM proto?

Do you mean using the OpenCensus stackdriver exporter? It's just the way to serialize OC metrics to GCM protobufs. I may be misunderstanding your question

Ah, ok.  I got it now.  We are essentially changing from opencensus to GCM protos.

Ah, ok. I got it now. We are essentially changing from opencensus to GCM protos.

Yup. Probably should have explained that better, sorry