TST: make assertion messages more understandable

Closes #10373. Also, this is based on #10500 and #10501.
The fix also did some refactoring related to assert_index_equal. Added check_exact and check_less_precise, and moved logics from assert_series_equal and assert_frame_equal for cleanups.
Followings are the list of tested and what the output looks like
Index

Shape (size)
Dtype
Values
Metadata

tm.assert_index_equal(pd.Index([1, 2, 3]), pd.Index([1, 2, 3, 4]))
# AssertionError: Index are different
#
# Index length are different
# [left]:  3, Int64Index([1, 2, 3], dtype='int64')
# [right]: 4, Int64Index([1, 2, 3, 4], dtype='int64')

tm.assert_index_equal(pd.Index([1, 2, 3]), pd.Index([1, 2, 5]))
# AssertionError: Index are different
#
# Index values are different (33.33333 %)
# [left]:  Int64Index([1, 2, 3], dtype='int64')
# [right]: Int64Index([1, 2, 5], dtype='int64')

tm.assert_index_equal(pd.Index([1, 2, 3]), pd.Index([1, 2, 3], name='x'))
# AssertionError: Index are different
#
# Attribute "names" are different
# [left]:  [None]
# [right]: [u'x']

Series

Shape (size)
Dtype
Index (same as above)
Values
Metadata (same as above)

tm.assert_series_equal(pd.Series([1, 2]), pd.Series([1, 2, 3]))
# AssertionError: Series are different
#
# Series length are different
# [left]:  2, Int64Index([0, 1], dtype='int64')
# [right]: 3, Int64Index([0, 1, 2], dtype='int64')

tm.assert_series_equal(pd.Series([1, 2, 3]), pd.Series([1, 2, 4]))
# AssertionError: Series are different
#
# Series values are different (33.33333 %)
# [left]:  [1, 2, 3]
# [right]: [1, 2, 4]

DataFrame

Shape (size)
Dtype
Index (same as above)
Column (almost same as above, but different summary)
Values

tm.assert_frame_equal(pd.DataFrame([[1, 2], [3, 4]]), pd.DataFrame([[1, 2, 3], [4, 5, 6]]))
# AssertionError: DataFrame are different
#
# DataFrame shape (number of columns) are different
# [left]:  2, Int64Index([0, 1], dtype='int64')
# [right]: 3, Int64Index([0, 1, 2], dtype='int64')

tm.assert_frame_equal(pd.DataFrame([[1, 2], [3, 4]]), pd.DataFrame([[1, 2], [3, 4], [5, 6]]))
# AssertionError: DataFrame are different
#
# DataFrame shape (number of rows) are different
# [left]:  2, Int64Index([0, 1], dtype='int64')
# [right]: 3, Int64Index([0, 1, 2], dtype='int64')

tm.assert_frame_equal(pd.DataFrame([[1, 2], [3, 4]], columns=['A', 'B']), pd.DataFrame([[1, 2], [3, 4]], columns=['a', 'b']))
# AssertionError: DataFrame.columns are different
#
# DataFrame.columns values are different (100.0 %)
# [left]:  Index([u'A', u'B'], dtype='object')
# [right]: Index([u'a', u'b'], dtype='object')

tm.assert_frame_equal(pd.DataFrame([[1, 2], [3, 4]]), pd.DataFrame([[1, 2], [3, 5]]))
# AssertionError: DataFrame.iloc[1, :] are different
#
# DataFrame.iloc[1, :] values are different (50.0 %)
# [left]:  [2, 4]
# [right]: [2, 5]

@sinhrks What is the status of this? Is this ready for review?
Does it implement the full description you gave in https://github.com/pydata/pandas/issues/10373#issuecomment-114851494

@jorisvandenbossche Rebased, and it is ready for review. I've updated the top description to show each outputs.

lgtm. only issue I would have is if an exception message were to actually fail because of a code issue in the exception code itself, but you are mostly testing that.

@jorisvandenbossche @shoyer @TomAugspurger Pls let me know if anything.

Let me merge this if there is no pointing out during the weekend.

@sinhrks lgtm
nice doc strings!

@sinhrks this is nice, e.g. (in this case its a sorting difference)
AssertionError: Series.index are different

Series.index values are different (96.66667 %)
[left]:  DatetimeIndex(['2000-02-09', '2000-01-20', '2000-01-10', '2000-01-24',
               '2000-01-07', '2000-02-11', '2000-01-25', '2000-01-03',
               '2000-02-03', '2000-02-01', '2000-02-07', '2000-01-19',
               '2000-02-02', '2000-01-14', '2000-01-31', '2000-01-05',
               '2000-02-10', '2000-01-06', '2000-01-18', '2000-01-11',
               '2000-01-21', '2000-01-13', '2000-01-28', '2000-01-17',
               '2000-02-08', '2000-01-27', '2000-01-12', '2000-02-04',
               '2000-01-04', '2000-01-26'],
              dtype='datetime64[ns]', freq=None, tz=None)
[right]: DatetimeIndex(['2000-01-03', '2000-01-04', '2000-01-05', '2000-01-06',
               '2000-01-07', '2000-01-10', '2000-01-11', '2000-01-12',
               '2000-01-13', '2000-01-14', '2000-01-17', '2000-01-18',
               '2000-01-19', '2000-01-20', '2000-01-21', '2000-01-24',
               '2000-01-25', '2000-01-26', '2000-01-27', '2000-01-28',
               '2000-01-31', '2000-02-01', '2000-02-02', '2000-02-03',
               '2000-02-04', '2000-02-07', '2000-02-08', '2000-02-09',
               '2000-02-10', '2000-02-11'],
              dtype='datetime64[ns]', freq='B', tz=None)


SPM demo does not show the Monitor tab in the UI

Describe the bug
The SPM demo described in https://github.com/jaegertracing/jaeger/blob/main/docker-compose/monitor/README.md should be showing the Monitor tab in the UI (it is enabled in the UI config), but it does not.
cc @albertteoh

I tried it out just now, making sure that I delete the existing jaegertracing/all-in-one image in case a bug was introduced:
docker rmi -f jaegertracing/all-in-one

but I'm still able to see the Monitor tab.

Is there any other step I can try to reproduce the issue?

hm, yes, after deleting the image it seems to work. Not sure if my "latest" was very old, I usually run versioned images.  Perhaps we can improve the instructions in case people run into this issue.
BTW, another issue, but I think it's for the UI only - https://github.com/jaegertracing/jaeger-ui/issues/969


No License file on pypi

Would it be possible to include it? It makes it easier to successfully and properly repackage such as for conda-forge.

Hi @carver, this issue was fixed and can be closed.


*: simplify network metrics

The previous rafthttp metrics are too fine grained and not suitable for stable metrics. Also I am not convinced that they are very useful for debugging purpose. We have been debugging etcd extensively and never rely on these msgType related info.
Bandwidth is more important.

lgtm

@heyitsanthony All fixed.

lgtm


All PDFs are assigned to a PDF that could not exist

The API states that every problem has a PDF file, it might not.
The API should check if the PDF exists and properly return the URL to display in the fronted.

Pending validation 46b281b


oranda build: "failed to read asset from" No such file or directory (os error 2)

I tried oranda build (0.4.1) as shown on the website and the book on https://github.com/konstin/gourgeist 28c8126fad9b465b4ba6bd9170721594b6a04ed3:
$ oranda build
↪ >o_o< INFO: Building components: artifacts, changelog
  × failed to read asset from
  ╰─▶ No such file or directory (os error 2)
  help: Make sure your path is relative to your oranda config or project manifest file.

I'm missing the information about what failed because it doesn't show the missing path

heya! sorry for that terrible error - that was an oversight with how we handle path fallbacks. I'll send a PR fixing that error to actually display the missing filename, which in your case was the readme file. one current "hard" requirement of oranda is that you kind of need a README file, this stems from the very first version, when literally all it did was render README files. oranda can do way more now, so I don't think a readme should be a requirement anymore.

Posted https://github.com/axodotdev/oranda/pull/640, this should allow you to use oranda without a README present, and have much nicer error messages for all other paths that it expects to exist (mostly when you tell it explicitly that something should be at a specific path).

I just released a new prerelease that contains the above PR, feel free to test it out, and let us know if you have any further issues or suggestions! https://github.com/axodotdev/oranda/releases/tag/v0.5.0-prerelease.1

That fixes it, thank you!


Fix: Handle CUDA device-side error

CUDA error: device-side assert triggered happens from time to time in the loop function. Once it's triggered, all following validation will keep failing due to this. We need a bash script to monitor and restart the loop process based on the error.

validate function should exit the process whenever it captures this exception with a special exit code
The bash script should capture the exit code and decide whether to restart the loop process
If it's caused by this error, we should not mark the assignment as failed, because it's not actually caused by OOM
The bash script should have a usage() defined
The bash script should take in necessary ENVs using --hf_token , --flock_api_key, and etc.
Update the README

Here is a screenshot of the CUDA error:


QualityCheck.json - net.ipv4.ip_local_port_range definition in conflict with SAP HANA recommendation

The NetApp recommendation is in conflict with the SAP HANA recommendation. In fact the SAP Host Agent will set the correct values, but only if this range is not defined within sysctl.conf.
{
    "Type": "OS",
    "Enabled": true,
    "RootRequired": true,
    "PreProcessingCommand": "",
    "ProcessingCommand": "sysctl net.ipv4.ip_local_port_range",
    "PostProcessingCommand": "",
    "ExpectedOutput": "net.ipv4.ip_local_port_range = 40000 65300",
    "ErrorMessage": "sysctl parameter missing or missconfigured",
    "HelpMessage": "see more details at https://docs.microsoft.com/en-us/azure/virtual-machines/workloads/sap/sap-hana-scale-out-standby-netapp-files-suse"
},

2382421 - Optimizing the Network Configuration on HANA- and OS-Level

net.ipv4.ip_local_port_range  => 9000-65499
As HANA uses a considerable number of connections for the internal communication, it makes sense to have as many client ports available as possible for this purpose.
At the same time, you need to ensure that you explicitly exclude the ports used by processes and applications which bind to specific ports by adjusting parameter net.ipv4.ip_local_reserved_ports accordingly.
If configured correctly, the SAP Host Agent takes care of adjusting this parameter and setting it manually is neither recommended nor required.
For further details about the SAP Host Agent please see the section below.
The SAP Host Agent typically increases the port range typically to 9000-65499. If your port range is significantly different, for example when your lower port range starts with port 40000, please check the SAP Host Agent section below.

Thanks Peter, I will discuss the issue with the team responsible for defining the parameters.

Thanks Peter, I will discuss the issue with the team responsible for defining the parameters.

Please re-open as this is still not fixed.
In the meantime this section has been added
https://docs.microsoft.com/en-us/azure/virtual-machines/workloads/sap/sap-hana-scale-out-standby-netapp-files-suse

Tip
Avoid setting net.ipv4.ip_local_port_range and net.ipv4.ip_local_reserved_ports explicitly in the sysctl configuration files to allow SAP Host Agent to manage the port ranges. For more details see SAP note 2382421.

included in new version, will keep it open until script is GA

V2 is GA now


Add theme section to website

More about themes of the hackathon => ultrahack.org/ultrahackindia

@chocos10 @suhailsinghbains bhai krdo jra :P

Working on it now


How are duplicated parameters being handled?

dotnet build can be called like dotnet build /p:IncludeSymbols=True /p:PackAsTool=True or dotnet build /p:IncludeSymbols=True;PackAsTool=True I did not find anything about this in the docs. Is that usage scenario covered?
@drewburlingame  Since you're busy with docs atm, i figured you might be aware of it if that were the case.

Hi @taori, thanks for the questions.
When possible, if I can answer them by updating docs, I would like to do that to help drive the documentation I'm working on.  To that end, I've added a doc for Token Transformation, but Travis doesn't seem to be running builds atm.  Perhaps they'll have run and deployed the updated docs by the time you read this.  Look under Extensiblity \ Token Transformations
CommandDotNet does not support the syntax you've posted.  It doesn't support the /p: or splitting on ;
I think it would be simple enough to support by creating a TokenTransformation.  You can see examples in the ___Transformation.cs classes found here
There would be two steps.

inspect all Value type tokens.  If they start with /p, convert them to Option token, trimming away the /p for the value.
split on ;, creating a new Option token per part.  Splitting could be a separate transformation.

The framework will already split on assignment so you won't need to handle that here.
I don't plan to add support for those personally, but if you'd like to submit a PR with tests, I'll review it since this seems to be a MS pattern. Or you could create a nuget package and we can create a help page to reference 3rd party packages.

https://bilal-fazlani.github.io/commanddotnet/V3/Extensibility/token-transformations/

That sounds great. Thanks for your answer.


High-level API

Moving forward the idea is to have two APIs:

a high level API as close as possible to keras:

allowing user to more clearly express what they want to achieve with secure computation (along the lines of how TF2 wraps distribution up into a DistributionStrategy object)

making it easier for us to guide users in how to get good performance out of TFE, in some sense making it harder for them to shoot themselves in the foot

a low level API similar to what exists currently:

gives full power but also the ability to shoot yourself in the foot by eg naively using expensive operations everywhere

See also Standardizing on Keras: Guidance on High-level APIs in TensorFlow 2.0

model = tf.keras.Sequential([
  layers.Dense(64, activation=’relu’)
  layers.Dense(64, activation=’relu’)
  layers.Dense(10, activation=’softmax’)
])

model.compile(
  optimizer=’adam’,
  loss=’sparse_categorical_crossentropy’,
  metrics=[‘accuracy’])

model.fit(x_train, y_train, epochs=5)

model.evaluate(x_test, y_test)

Federated models:

model = tf.keras.Sequential([
  tf.keras.layers.Dense(64, activation=’relu’)
  tf.keras.layers.Dense(64, activation=’relu’)
  tf.keras.layers.Dense(10, activation=’softmax’)
])

model.compile(
  optimizer=’adam’,
  loss=’sparse_categorical_crossentropy’,
  metrics=[‘accuracy’],
  computation='aby3')

model.fit(x_train, y_train, epochs=5)

model.evaluate(x_test, y_test)

model.predict(x)

Custom Keras backends:

In Keras it is possible to load more backends than "tensorflow", "theano", and "cntk". Keras can use external backends as well, and this can be performed by changing the keras.json configuration file, and the "backend" setting. Suppose you have a Python module called my_module that you wanted to use as your external backend. The keras.json configuration file would be changed as follows: ... An external backend must be validated in order to be used, a valid backend must have the following functions: placeholder, variable and function.

See also here

I had this same idea about just creating a new keras backend. I mostly worry about the keras frontend being flexible enough for our uses and how much we'd have to implement to get simple models to work correctly. It could be something we could test fairly easily though. We just implement those three functions (placeholder, variable and function) and then implement a simple Sequential model with a single Dense layer. We could try running that and see what breaks. Perhaps a medium-term goal rather than short-term.

Note: let's make sure that tfe.keras matches tf.keras as much as possible, ideally such that one can simply replace tf with tfe.

One small note: it seems safe to follow design patterns that users will be familiar with from TF already, even if there might potentially be better ways of doing things. One concrete example is how to make this Keras implementation agnostic to the underlying computation protocol, which can be done in at least two ways:

define abstract Keras elements that can be created without knowing the computation protocol and later instantiated in an explicit call; one issue here is the Keras functional API which might require us to replicate a lot of dummy elements and patch up;

an alternative is to take inspiration from TF distribution strategies and allow the use of contexts; to still be agnostic to the computation protocol the user can wrap model creation in a function

cc @yanndupis @jvmancuso

Closing, with protocol requirements distilled into https://github.com/tf-encrypted/tf-encrypted/issues/558.


fix(radio): host element unable to receive focus events

In #13323 we started clearing the tabindex from the host element in order to avoid having two elements with the same tabindex when the consumer sets a custom tabindex. As a result, the mat-radio-button can no longer receive focus and forward it to the input. These changes always reset the tabindex to -1 so the element is focusable, but not tabbable.
Fixes #13953.
Note: this is a resubmit of #13956.

Caretaker note: causes a handful of failures in g3 due to assertions about tabindex that are no longer true


⚠ Processors has degraded performance

In 72b0541, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 72 ms

Resolved: Processors performance has improved in c039598.


[BUG] Can't install charts when all nodes of a cluster have a taint

Rancher Server Setup

Rancher version:

v2.6-b66dd0f614f9d60ef54dfb82da12b9b8b194021b-head

Installation option (Docker install/Helm Chart):

Docker

Proxy/Cert Details:

N/A

Information about the Cluster

Kubernetes version:

1.24.2+rke2r1

Cluster Type (Local/Downstream):

DigitalOcean

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

Admin

Describe the bug
If all nodes of a cluster have a taint it's not possible to install a chart from the UI. The application never appears in the installed apps section and there's never an error.
Example taint:

Example pod failure:

It looks like UI needs a way to specify tolerations  for the "apps & marketplace pod" as it's currently hardcoded. Or, we should be able to pass the tolerations as a part of the install post request /v1/catalog.cattle.io.clusterrepos/rancher-charts?action=install.
To Reproduce

Create a cluster of 1 node
Set a taint on the single node in the cluster

Attempt to install an application/chart (I've been using monitoring) you can specify tolerations in the values if you want but it won't make a difference.
After the installation takes you to the installed charts page observe that the chart is not installed
Go to the workload > pods list page and see that you have a failed pod

Result
Shown above in the description.
Expected Result
We should be able to specify tolerations from the UI so that an application can install on a cluster that has taints on all nodes.
Additional context
This issue is currently preventing the UI from resolving a customer reported issue https://github.com/rancher/dashboard/issues/4737

Internal reference: SURE-4019

Associated Jira was closed by support though the issue is still present.


[$500] Web - Expense - Deleted expense on OD is still displayed on report on ND

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.33-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal team
Slack conversation:
Action Performed:
Pre-requisite: user must have created a collect workspace and have invited a member as employee.

As employee, create an expense on OD.
On the same report, create another expense.
As the admin on ND, verify the report and the expenses (there should be 2 expenses).
As the employee again on OD, delete the first expense you created.
As the admin on ND, verify the report again.

Expected Result:
The expense deleted by the employee should not appear on the report on ND as the admin.
Actual Result:
The expense deleted by the employee is still displayed on the report on ND as the admin.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/16f9866c-55aa-448f-aa30-6ac1a5e38ee5
View all open jobs on GitHub

We think that this bug might be related to #wave6-collect-submitters
CC @greg-schroeder

This should be handled here: https://github.com/Expensify/Expensify/issues/350244


fix: add FundingSource & ParcelGuard as valid options for InsuranceProvider

per https://www.shipengine.com/docs/shipping/add-insurance-to-shipment/

@semarj If you wouldn't mind reviewing. This'll be my 2nd PR. How many do I need before I get a T-shirt?

@jpill Can I get this tiny change merged? I'm a paying customer trying to use Parcel Guard in my dotnet app.

Thanks!

I just tried using the new release. The new values, parcelguard and funding_source don't work in the sandbox. When I pass either value, the response has the following error
{
  "RequestId": null,
  "ErrorSource": 1,
  "ErrorType": "business_rules",
  "ErrorCode": "unspecified",
  "ResponseMessage": null,
  "TargetSite": null,
  "Message": "A shipping carrier error occurred: Wallet not found for currency USD",
  "Data": {},
  "InnerException": null,
  "HelpLink": null,
  "Source": null,
  "HResult": -2146233088,
  "StackTrace": null
}

It works in Prod, but it seems like it should work in Test/Sandbox too.
I've validated this with Postman too. Is there a better place to file server bugs?


Windows: Docker top implementation

Signed-off-by: John Howard<EMAIL_ADDRESS>A picture speaks a thousand words :smile:

Now with test cases. Ready for full review. Removed [wip] from title.

@jhowardmsft tests failures seems related to top changes here

@runcom oops. Thanks for pinging. Looking...

@runcom Should be fixed now - waiting for Jenkins to finish.
@icecrime PTAL too as to fix the tests, I updated the logic of what was the defaultSleepCommand. There was some incorrect logic in it as it the value was set by build tag according to the platform where the client part of the test was running. It should have been the platform of the daemon instead.

The two experimental failures are networking related and nothing to do with these changes - seems to have started happening today on PRs across the board.  <-- @icecrime FYI

All green :green_heart: on CI. Ready for review.

Code LGTM (I don't have a Win machine to test this out) ping @icecrime @dnephin

I don't really know anything about windows, but code LGTM

Anyone else? Or good to merge with 2 LGTMs? :smile:

@icecrime Anyone who has a Windows machine want to test? Or OK to merge?

Merging (SGTM from @icecrime on IRC)

@jhowardmsft this needs rebased first


Support for generating Andmore projects

Since Android ADT plugin is no longer developed by Google, it's been forked into the Andmore project. Andmore's ADT fork uses a different package for the project builders and natures. Here's a typical Andmore .project file:
<?xml version="1.0" encoding="UTF-8"?>
<projectDescription>
	<name>WebApps</name>
	<comment></comment>
	<projects>
	</projects>
	<buildSpec>
		<buildCommand>
			<name>org.eclipse.andmore.ResourceManagerBuilder</name>
			<arguments>
			</arguments>
		</buildCommand>
		<buildCommand>
			<name>org.eclipse.andmore.PreCompilerBuilder</name>
			<arguments>
			</arguments>
		</buildCommand>
		<buildCommand>
			<name>org.eclipse.jdt.core.javabuilder</name>
			<arguments>
			</arguments>
		</buildCommand>
		<buildCommand>
			<name>org.eclipse.andmore.ApkBuilder</name>
			<arguments>
			</arguments>
		</buildCommand>
	</buildSpec>
	<natures>
		<nature>org.eclipse.andmore.AndroidNature</nature>
		<nature>org.eclipse.jdt.core.javanature</nature>
	</natures>
</projectDescription>

It would be great if there was an option for Gradle Eclipse AAR plugin to generate these projects instead of the old ADT version.

@tobykurien Thank you for reporting this. I'll look into it.

I've added a new option andmore to support Andmore.
You can try this with 0.2.0-SNAPSHOT.
buildscript {
    repositories {
        mavenCentral()
        maven {
            url 'https://oss.sonatype.org/content/repositories/snapshots/'
        }
    }
    dependencies {
        classpath 'com.github.ksoichiro:gradle-eclipse-aar-plugin:0.2.0-SNAPSHOT'
    }
}

When andmore is set to true, .classpath and .project will be generated for Andmore.
eclipseAar {
    andmore = true
}

Thank you! I've tested it, and it seems to work. I did notice some weird things though:

Andmore throws up errors about the output bin folder not existing after importing the projects. If I close Eclipse and re-open it, they then compile fine. I think it's a bug in Eclipse as it did this with ADT as well.
The AAR projects are not marked with the Andmore "a" badge, instead they are marked with the Java "j" badge. When I previously converted an ADT project to Andmore, it had the "a" badge. I'm not sure why this is since the .classpath and .project files have the right content in them (including Andmore nature). It doesn't seem to cause problems though...

Thank you for testing and your feedback!
I haven't noticed those points.

Andmore throws up errors about the output bin folder not existing after importing the projects

Unfortunately, I couldn't have confirmed this issue,
but I'll modify the plugin to create bin and gen folders if they're not created.
Generating those folders by the plugin will not cause problems, I think.

The AAR projects are not marked with the Andmore "a" badge, instead they are marked with the Java "j" badge.

I found that it's because the first nature in the generated .project file is org.eclipse.jdt.core.javanature.
If I change the order of natures, "a" badge is shown; AndroidNature should be first.
This also happens with ADT, and creating Android project with ADT also defines AndroidNature first,
so I'll change the order for both Andmore and ADT.
Please wait for a while.

I've fixed those issues and updated the SNAPSHOT, so you can try it if you have time.

I've tested it, and it is working flawlessly! Great work :+1: Please feel free to make a release.

Thank you for testing! I'll release it soon.

I've released v0.2.0.
It'll be synced to Maven Central later.
With plugins notation, you can use it right now.
plugins {
  id 'com.github.ksoichiro.eclipse.aar' version '0.2.0'
}


[MM-23414] Fix up multiselect styling

Summary

The original issue was just that the dark mode styles were too dark - the cause of which i traced back to https://github.com/mattermost/mattermost-webapp/pull/5011
Turns out the select input of this modal never modified its colours to match the current theme so I made some changes to its styles to look good in both dark and light themes
A lot of the styles were borrowed heavily from /components/widgets/inputs/user_emails_input.scss (from the invite team members select list)

Ticket Link
https://mattermost.atlassian.net/browse/MM-23414
Before

After

@fm2munsh I submitted a PR to fix the dark theme issue and the height.
I think we can still take the rounded thing and the stylsting changes from this PR.
See if you can resolve the conflicts.
https://github.com/mattermost/mattermost-webapp/pull/5079

@asaadmahmood Thanks - I didnt realize you already had a fix for this - I merged my changes with yours and kept the styling changes I made

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-5087.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

Test server destroyed

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

@fm2munsh
Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Fetching origin
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-#5087-upstream-release-5.22-1585151061
Branch 'automated-cherry-pick-of-#5087-upstream-release-5.22-1585151061' set up to track remote branch 'release-5.22' from 'upstream'.
+++ Downloading patch to /tmp/5087.patch (in case you need to do this again)

+++ About to attempt cherry pick of PR. To reattempt:
  $ git am -3 /tmp/5087.patch

Applying: Improve direct messages multiselect styling
Using index info to reconstruct a base tree...
M	components/multiselect/__snapshots__/multiselect.test.tsx.snap
M	components/multiselect/multiselect.tsx
Falling back to patching base and 3-way merge...
Auto-merging components/multiselect/multiselect.tsx
CONFLICT (content): Merge conflict in components/multiselect/multiselect.tsx
Auto-merging components/multiselect/__snapshots__/multiselect.test.tsx.snap
CONFLICT (content): Merge conflict in components/multiselect/__snapshots__/multiselect.test.tsx.snap
Patch failed at 0001 Improve direct messages multiselect styling
Use 'git am --show-current-patch' to see the failed patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

+++ Conflicts detected:

UU components/multiselect/__snapshots__/multiselect.test.tsx.snap
UU components/multiselect/multiselect.tsx

+++ Aborting in-progress git am.

+++ Returning you to the master branch and cleaning up.


http://www.getmdl.io/templates/blog/index.html

At 1024 px things start shuffling in different orders. Don't do that :)

I'm pretty sure this is a bug, but will wait for a confirmation from @sgomes or @surma

@surma Any idea what's happening here?

Working as intended.

@sgomes :


Whitespace is incorrectly stripped for calc expressions containing plus signs

-moz-calc(100% + 1em);

incorrectly gets converted to
-moz-calc(100%+1em);

because of this regex which strips superfluous whitespace around the  + adjacent-sibling selectors.

It has just been fixed: https://github.com/GoalSmashers/clean-css/commit/5e56238e826b0292dafe511f78e7d31aaddc2e03
A new clean-css 0.2.5 with this patch is available in the npm registry.
Thanks!
Jakub Pawlowicz
GoalSmashers.com

We've just released clean-css 0.2.6 with enhanced support for mixed calc() and adjacent selectors. Sorry if you've run into any issues with the previous one.

thanks very much for addressing this so promptly!


📖 add docs for releasing cluster-stack-operator

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes https://github.com/SovereignCloudStack/cluster-stack-operator/issues/62
Special notes for your reviewer:
Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
TODOs:

[ ] squash commits
[ ] include documentation
[ ] add unit tests

Hi @kranurag7, is that still a Draft?

Hi @kranurag7, is that still a Draft?

I wanted to add a note and I've done that via https://github.com/SovereignCloudStack/cluster-stack-operator/commit/94b951a5ff4ff0ec48ce8c4cbe38a5b8c2be8e9e
Now, I'll mark this ready for review.

Hi @kranurag7, is that still a Draft?

I wanted to add a note and I've done that via 94b951a
Now, I'll mark this ready for review.

Thanks @kranurag7. Question to you and @batistein (as requested for review)

Yes, I think the approvals are still up to date here but still let's go through it again.
@chess-knight @michal-gubricky I added on more commit about updating metadata.yaml when we change the minor version for the release.
https://github.com/SovereignCloudStack/cluster-stack-operator/pull/79/commits/76681e2001106e404047058d09dc1aa1f31a7b59
Please review it again and upon approval, let's get this patch merged into main.


How can I turn down the memory optimization option?

It seems that the macro definition of NNVM_EXEC_ENABLE_INPLACE has been removed in the 0.93 version. How can I turn down the memory optimization option?

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Rubocop fixes

Add Rubocop fixes that are forward-compatible with newer versions of
Rubocop.

Current coverage is 88.04% (diff: 100%)

Merging #190 into master will not change coverage

@@             master       #190   diff @@
==========================================
  Files            36         36
  Lines          1146       1146
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1009       1009
  Misses          137        137
  Partials          0          0

Powered by Codecov. Last update 3eea618...df35591


markdown workflow: add optional spell checker

Description
add optional spell checker to markdown workflow
Changes Made

add spell checker boolean flag
execute spell check if flag true
add configuration file and empty dictionary

Related Issues
Fixes #43
Checklist

[x] I have used a PR title that is descriptive enough for a release note.
[x] I have tested these changes locally.
[x] I have added appropriate tests or updated existing tests.
[ ] I have tested these changes on a dedicated VM or a customer VM [name of the VM]
[ ] I have added appropriate documentation or updated existing documentation.

to be rebased after merge of #42


Improve Timeseries Chart Hover

I find the timeseries chart hover to be lacking in functionality to the point that I almost want it to just not exist.  But, I really do want it to exist.  What do you think about it operating like:

Even if you don't hover over the line, it still generates a hover over the points in the same "plane" on the x-axis
Include the actual time in the hover text
Provide the "actual" number and then the human readable in parens (something like 1,284,374,372 (1.28b) )

Am I just totally off-base?

1: Just to clarify, do you mean that you would want it to generate a hover for all measures and not just one?
2: Should be coming soon.
3: Right now it shows the digits the same way it does in the table I believe. Would adding a digit or two everywhere help? Or, would you like a way to have much higher accuracy? Or, do you specifically need the "actual" number?
Anyway, right now out of these #2 is definitely on the roadmap.

I want all of the points for whatever time point I'm hovering over to show up.  Basically, the fact that I have to find the exact point to hover over is a little annoying.  This isn't super horribad and I hate you for it, it just might be nice to have more of a "proximity" hover.
Cool
Generally, I'm hovering because I want better accuracy than just the eye-ball I get out of the graph, so I think there is an argument for getting higher accuracy out of the tooltips.  I'm not too adamant about this though.

OK sorry but I am still not completely sure so I will ask more clearly. It sounds to me like you want to see the hover info for all measures at once and not just one at a time, but it also sound like there is some problem with the hover bubble not appearing unless you hit a very specific area. If the second thing is correct then it's an unfamiliar bug - there should not be a place within the axes area which does not generate a hover. Please confirm if that's the case.
Good point!

(also if you are encountering this bug, then what browser / os are you using?)

Chrome on mac.  The only place I get the tooltip to show up is when I hover over the "bend" in the graph.  Well, actually I just noticed that if I hover over that exact point lower down in the graph, it also shows me the tooltip, but if I'm off by even just a little bit, I don't get the tooltip.  This is especially noticeable when aggregating to daily granularity on a weekly chart.  I'd give you a screenshot, but when I take them, my cursor disappears.

Definitely a bug. I can't reproduce right now but we are looking into it.

+1 on all three things, but especially 1 & 2

I am having a tough time looking at telemetry-y datasources (where I want to do a lot of time drill down) with the current hover stuff

Update - we have reproduced and figured out the bug in #1, so a fix should be coming soon.
As for #2, it is top priority and should be coming really soon as well.

@asherbitter on github when you write #1, #2 it auto links them to the issues of that number.

The PR above addressed 1 and 2.
As for 3, we are taking it under consideration.
Closing this issue because the core of the problem has been fixed.


Add raw option to DataMatrix

I need to generate a DataMatrix barcode with special control codes and C40 encoding, at least until #41 is added. 😃 This is for programming Symbol barcode scanners, which seem to require both to accept the barcode as a programming barcode.
It's easy enough to generate the raw codes for C40, but the parse option mangles the output converting to ASCII. This seems to be solved with the raw option for other symbologies.
Ideally I would like to have the following:
Bar text: ^234^230^170^054^129
Options: raw
Output would decode to: {Reader programming}{C40 start}N59{End of message}

Added with e5adc6d1151e4918d22d3ae1335e829dcd871b9d.
Pseudo random padding is added automatically so you could use ^234^230^170^054 in place of your example above.

Tagged 2018-08-27 and released here: https://github.com/bwipp/postscriptbarcode/releases/tag/2018-07-27.
Hi @metafloor: @T3hUb3rK1tten is presenting at Def Con 26 and might appreciate if BWIP-JS could synced with the just-released BWIPP.
https://www.defcon.org/html/defcon-26/dc-26-speakers.html#West
https://barcowned.com/app/
@T3hUb3rK1tten May the demo gods be with you :-)


fx_cast_bridge: 0.2.0 -> 0.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/hensm/fx_cast/releases.
meta.description for fx_cast_bridge is: "Implementation of the Chrome Sender API (Chromecast) within Firefox"
meta.homepage for fx_cast_bridge is: "https://hensm.github.io/fx_cast/"
meta.changelog for fx_cast_bridge is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.3.0 with grep in /nix/store/zjl5vpw34ziwxijrb6lhbimy3wc1i2wm-fx_cast_bridge-0.3.0
found 0.3.0 in filename of file in /nix/store/zjl5vpw34ziwxijrb6lhbimy3wc1i2wm-fx_cast_bridge-0.3.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
fx_cast_bridge

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/zjl5vpw34ziwxijrb6lhbimy3wc1i2wm-fx_cast_bridge-0.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A fx_cast_bridge https://github.com/r-ryantm/nixpkgs/archive/724d059d750b7eaae89aa7d3ae810ae17036048b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/zjl5vpw34ziwxijrb6lhbimy3wc1i2wm-fx_cast_bridge-0.3.0
ls -la /nix/store/zjl5vpw34ziwxijrb6lhbimy3wc1i2wm-fx_cast_bridge-0.3.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

fx_cast_bridge

Maintainer pings
cc @dtzWill @kevincox for testing.

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

1 package built:

fx_cast_bridge

@ofborg eval

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/hensm/fx_cast/releases.
meta.description for fx_cast_bridge is: "Implementation of the Chrome Sender API (Chromecast) within Firefox"
meta.homepage for fx_cast_bridge is: "https://hensm.github.io/fx_cast/"
meta.changelog for fx_cast_bridge is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.3.1 with grep in /nix/store/1jlaz7p1wx4i95wnsqib5f86byfz0hjv-fx_cast_bridge-0.3.1
found 0.3.1 in filename of file in /nix/store/1jlaz7p1wx4i95wnsqib5f86byfz0hjv-fx_cast_bridge-0.3.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
fx_cast_bridge

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/1jlaz7p1wx4i95wnsqib5f86byfz0hjv-fx_cast_bridge-0.3.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A fx_cast_bridge https://github.com/r-ryantm/nixpkgs/archive/aeda2014ea97755dc8c02dbe845e046a85cd0ac0.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1jlaz7p1wx4i95wnsqib5f86byfz0hjv-fx_cast_bridge-0.3.1
ls -la /nix/store/1jlaz7p1wx4i95wnsqib5f86byfz0hjv-fx_cast_bridge-0.3.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

fx_cast_bridge

Maintainer pings
cc @dtzWill @kevincox for testing.


Updating the Mgt latest version in the tab-graph-toolkit Node.js sample

Build Pipeline -
https://github.com/OfficeDev/Microsoft-Teams-Samples/actions/runs/8370002347


Refactor netplugin CLI

This commit refactors netplugin CLI, it uses "github.com/urfave/cli"
now in order to provide configuration from CLI and environment vars,
and provides easy way to load from config files too.
In details, this commit introduced:

New configs for netmode and fwdmode settings, which needs to
match what in etcd;
Improved validation for all CLI/ENV inputs before
starting netplugin;
Removed requirement of running as root user, as long as the running
user has privilege to run ovs and most networking related CLIs, is good
enough;
Option "-cluster-store" has been reworked, replaced by "--etcd-endpoints"
and "--consul-endpoints", so that it will be easy to enable https endpoints
and adding more than one endpoints for HA;
Option "-config" which reads configuration from stdin has been removed,
it will soon be replaced by reading configuration from a real file instead;
Option "-debug" has been replaced by "--log-level", this will allow user
to set loglevel to higher level if doesn't want to use INFO level;
Option "-syslog" has been replaced by "--use-syslog" and "--syslog-url",
this will make it more clear on syslog settings and endpoint.
Removed Option "-net-driver", because netplugin currently only supports
OVS, will add it back when there are second backend type.
Rework state driver and objdb client, make them more flexible to support
multi-endpoints model of etcd and consul, and http/https as URL schema.
Updated testing cases in react of all above changes.

Signed-off-by: Wei Tie<EMAIL_ADDRESS>

build PR

build PR

build PR

build PR

build PR

Nice work! Had some comments. Also, did not review the 21 new files vendored in vendor/github.com/urfave.

The error Error message did not yield stack trace with CONTIV_TRACE set in core/error_test.go need to refer CONTIV_TRACE as CONTIV_TRACE is not used anymore.  Can be corrected in a follow-up PR.


Do we want a config file?

Right now we allow defaults via the environment.
@andrewjankie makes a passing mention of config file defaults

Hm.. Needlessly complex? What was the use case you had in mind?
Either way, we definitely still want configuration via environment
variables so that we can use qbatch with environment-modules.

Just kinda worried about the proliferation of environment variables.
I'm just thinking about how to handle all cases of defaults for the "I don't want to type this every day" type

If we ever do consider it https://docs.python.org/2/library/configparser.html

As an alternative to "environment-variables" support, we could do what pydpiper does, which is define a "default" config file via an ENV var. This would reduce the env var set to one.

And so in the environment-modules file you could do something like:
set-env  QBATCH_CONFIG  /quarantine/qbatch/1.01/build/config

I'm okay with that. I also don't see a compelling case to change from
using environment variables. What do you have in mind?

I can see a case for allowing 'most' of our command line arguments to have settable defaults. This presents an explosion of environment variables. If future options come this could continue.
A config file would collapse all these info one, and allow for documentation of options inside the config file.

Right, I see what you mean. I'm weirdly attached to the convenience of
env variables but no matter... config file does the trick nicely.

Looks like ConfigArgParse is ideal.
Mostly drop-in replacement for ArgParse, with config file and environment variable features in one set:
https://github.com/bw2/ConfigArgParse


Cannot install pi_piper on minibian (Raspberry Pi)

I'm trying to do a fresh install using the minibian distro.
It won't let me install pi_piper saying that it requires RubyGems version >- 2.0.0
I have apt-get updated
It suggests gem update --system but I get an error saying this is disabled in Debian as it can break Debian.
Any advice on how to proceed?
(I then over-rode this to force a gem update and now get the following error when I run the printer script pi-printer.rb)
/usr/lib/ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in `require': cannot load such file -- serialport (LoadError)
	from /usr/lib/ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in `require'
	from ./pi-printer.rb:5:in `<main>'

In my case, i just need to synchronize between ruby and gem version.
you should check your ruby and gem version, they have to on the same version.

check it with command 'which ruby' and 'which gem'
and then check it on /usr/bin with 'ls -ltr /usr/bin/ruby*' 'ls -ltr /usr/bin/gem*'
after that you should change the asymlink to the exact version for both ruby and gem


Looks like diagnostics module's icons can't be customized

Thanks a lot from this awesome plugin.
Self Checks

[x] I'm using the latest lualine.
[x] I didn't find the issue in existing issues or prs.

How to reproduce the problem
I've successfully customized other modules (such as branch and filename), but I'm failing at configuring the diagnostics icons. I tried:
{
  'diagnostics',
    icons_enabled = true,
    -- symbols = {    -- I tried with and without this wrapping
    icons = {
      error = '', -- xf659, shows up in my terminal
      warn = '',  -- xf529, idem
      info = '',  -- xf7fc, idem
      hint = '⚡',  -- xf835, idem
    }
    -- }
}

I checked that setting icons_enabled = false results in the text equivalents being shown, so I guess my config is not completely south.
I'm a lua newbie, but I suspect in this code the customized icons are always overridden by the default ones or text replacements.
https://github.com/nvim-lualine/lualine.nvim/blob/master/lua/lualine/components/diagnostics/init.lua#L25-L30
Expected behaviour
I would expect my custom icons to be displayed instead of the default ones.
Actual behaviour
The default icons are always displayed.
Minimal config to reproduce the issue

Config
Sorry, I'll first try posting this issue without doing this intimidating task. I'll do it if you really want it to solve this issue, but my guess is that the initialization code has a small issue.
<!--
  =================
  ### IMPORTENT ###
  =================
<p>You need to provide a way to reproduce the issue without influence
of other plugins.
Issues without proper format and a proper way to
reproduce might be ignored.
This script can help you with that.
https://github.com/nvim-lualine/lualine.nvim/blob/master/scripts/nvim_isolated_conf.sh
Create an isolated config directory with:
`nvim_isolated_conf.sh -c DirectoryName`

Then modify the DirectoryName/.config/nvim/init.vim
so you can reproduce the issue .
You can load the config to see if the issue is occuring with
`nvim_isolated_conf.sh -l DirectoryName`

Paste the DirectoryName/.config/init.vim without comments below
-->

Additional information

The docs says
{
  'diagnostics',
      symbols = {
      error = '', -- xf659, shows up in my terminal
      warn = '',  -- xf529, idem
      info = '',  -- xf7fc, idem
      hint = '⚡',  -- xf835, idem
    }
}

did you try that ? there's no icons tabel.

No, I missed this. It's working now, thanks!


fix FE collection runtime error

Doing a better response check to avoid runtime error!
I forgot BE would return error response like the following, so I need to update how I check for empty datasets:
{
  detail: "None is not of type 'string'↵↵Failed validating 'type' in schema['properties']['obfuscated_uuid']:↵    {'type': 'string'}↵↵On instance['obfuscated_uuid']:↵    None",
  status: 500,
  title: "Response body does not conform to specification",
  type: "about:blank"
}

Thank you!

Just merged this one since it's a low risk fix! Please still help review in case I missed something lol Thank you!


Remove redundant type casts

Getting rid of redundant type casts makes the code a lot easier to read, and there is no downside to removing them.

@JamieMagee thank you for your contribution


create command app not working

hey there, I've just tried to build an app with the create command. However, running the code nothing more than this happens:
C:\tmp> mantra create test -v
  create  test/
  invoke  init

the root folder is created without content.
If possible I would like to debug this and help with solving this issue as I really like this tool.
Did I miss something? Do other experience the same issue?
Also I'm using windows, I assume this can be a problem.

Problem solved:
C:\tmp> mantra c ./asdf
  create  asdf/
  invoke  init

needed a qualified path: https://github.com/shelljs/shelljs/issues/41


Minor spelling correction

Hi realsaraf, thankyou for the PR.  can you please make the change to this file under the /template/ folder rather than /generated/ folder.
kind regards,
nick

hi @nickharris
i don't see the corresponding content in the /templates . Am i missing something?


🐞[bug]: Footer icons in the Cart page is not changing on hover

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug

Expected behavior
smooth enhancement on hovering over footer icons (in cart page)which is similar to all other pages
Add ScreenShots
No response
What browsers are you seeing the problem on?
Chrome, Microsoft Edge
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24EXTD contributor
[X] I have starred the repository

@laxmikandivalasa go ahead.


2.11: Use c6gn.large in place of c6gn.16xlarge for EFA tests

The test is useful to verify EFA kernel module is installed correctly on ARM instances with alinux2 and ubuntu1804 and that openmpi is installed correctly but it's not needed to use the biggest c6gn instance type.

Closed because EFA can be used on c6gn.16xlarge only, see: https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/efa.html#efa-instance-types


Misc1

Shall I go ahead and merge this PR?

Shall I go ahead and merge this PR?

It is better if the PR are not merged by its author when possible, for review quality purpose.


WMS ID:11024 - Updating to use the "paid tier" option

requesting the pull request to update the use of "tenancy" as workshop can only be run for paid tenancy option

Please replace the following files in index.html
https://oracle.github.io/learning-library/common/redwood-hol/js/jquery-1.11.0.min.js
https://oracle.github.io/learning-library/common/redwood-hol/js/jquery-ui-1.10.4.custom.js
https://oracle.github.io/learning-library/common/redwood-hol/js/main.min.js
with this. as we will be using oracle-livelabs repo
https://oracle-livelabs.github.io/common/redwood-hol/js/jquery-1.11.0.min.js
https://oracle-livelabs.github.io/common/redwood-hol/js/jquery-ui-1.10.4.custom.js
https://oracle-livelabs.github.io/common/redwood-hol/js/main.min.js

updated all the changes for the index.html pointing to the oracle-livelabs repo


Go Modules support

Hi, I'm currently working on updating draft to work with go modules.
This is the current progress:

[x] Building and running draft outside the gopath

Following the official documentation, I went through the following process:
# clone outside GOPATH
$ git clone https://github.com/Azure/draft
$ cd draft
$ go mod init github.com/Azure/draft

go mod will then copy the dependency requirements from the existing Gopkg.lock.
$ go mod tidy

to clean up the dependencies , add missing ones and mark indirect dependencies.
The results are this go.mod and this go.sum.
Now I can build and run draft outside the gopath.
$ go build ./...
$ go run ./cmd/draft

(I previously mentioned some dependency issues here but it turns out these were based on wrong assumptions - I thought I was already importing my fork with modules support but I was actually importing the remote version.) For documentation purposes I've stored them here)

Remaining issues with running the project outside of the gopath:

make bootstrap won't work - but it is also not required, as go modules will make sure all the dependencies are installed. It can still be used

failing unit tests (make test-unit)

--- FAIL: TestStoreCreateBuild (0.00s)
    store_test.go:128: failed equality for CreateBuild
--- FAIL: TestStoreUpdateBuild (0.00s)
    store_test.go:128: failed equality for UpdateBuild

I'm not entirely sure why they fail. I haven't been able to spot anything usually handled by make bootstrap that would've been required.
The structs appear equal when I have a look at their fields.
got: buildID:"foo1" release:"bar1" contextID:"foobar1" created_at:<seconds:1557756090 nanos:889887465 >
obj: buildID:"foo1" release:"bar1" contextID:"foobar1" created_at:<seconds:1557756090 nanos:889887465 >

I'm not sure how to approach this issue yet.
Would you like me to open a PR anyway?

Some quick thoughts here:

We're currently experimenting with migrating to go modules in Helm and have some concerns about maintaining the project compared to dep's approach to dependency management, so we haven't invested much time in here. Once we're more familiar with go mod we should be able to provide more opinions here
make bootstrap can go away with go mod. That's not an issue :)
feel free to PR what you have. Perhaps someone from the community could take a look and provide some insight to your failed tests

Thanks for taking a look into this!

@nicmr
I looked into the failing tests and the reason they are failing is a field called XXX_sizecache within CreatedAt and for the obj it has a value of 12 and for got is has a value of 0.

This is the issue for both

TestStoreUpdateBuild
TestStoreCreateBuild


Can you choose which GPU to use. like  tf inference  device_map="cuda:0"

As the title suggests

https://github.com/vllm-project/vllm/blob/e221910e77087743a50560e4ae69c3c2a12beb53/vllm/engine/arg_utils.py#L285
I think it is not supported. But it shouldn't be too hard to support

you may use CUDA_VISIBLE_DEVICES / HIP_VISIBLE_DEVICES to isolate GPU. PyTorch picks which GPUs to use.

you may use CUDA_VISIBLE_DEVICES / HIP_VISIBLE_DEVICES to isolate GPU. PyTorch picks which GPUs to use.

The CUDA_VISIBLE-DEVICES variable is available, but I would like to start three different models in a fastAPI process, each occupying one GPU. CUDA_VISIBLE-DEVICES can only be set once, and subsequent settings will become invalid.

https://github.com/vllm-project/vllm/blob/e221910e77087743a50560e4ae69c3c2a12beb53/vllm/engine/arg_utils.py#L285
I think it is not supported. But it shouldn't be too hard to support

        parser.add_argument("--device",
                            type=str,
                            default=EngineArgs.device,
                            choices=["auto", "cuda", "neuron"],
                            help='Device type for vLLM execution.')

I have looked at the source code here, it only sets the hardware type and cannot select which GPU it is

torchrun should be able to solve your problem? each process runs 1 vllm...

same issue, keep this open pls.


Bump production to 0.74.0

Release #198 to production

Will fix the tfsec issue before bumping prod

Fixed the tfsec issue in #201 and #204


Video

Any way to do a video recording with this? Any plans on adding that feature?

Anyone luck here @jishindev ? We need this too and will look into adding it if no one has done so.


Failure at installation

Issue Type: Bug
tried to install vs live share audio extension together vs live share  extension
Extension version: 1.0.2354
VS Code version: Code - Insiders 1.47.0-insider (269f0fdd83f1e6611dd0c6d225a1685719eafedb, 2020-06-22T05:53:42.894Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7500T CPU @ 2.70GHz (4 x 2712)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.89GB (4.56GB free)

Process Argv

Screen Reader
no

VM
0%

Same here with Extension Version: v1.0.2377
Win 64bin Enterprise.
Left Icon for LiveShare is created but no  "live share" icon in status line.
When I install the Extension pack (v0.4.0) a message comes: LiveShare Audio needs  Extension pack. !?
And again: left Icon is created, status line has no icon

After Installation of Extension pack v0.4.0 following notification: "VS LiveShare Audio extension requires VS Live Share extension to be installed. Please install VS Live Share extension or update to latest"
In .vscode the 3 folders ms-vsliveshare* are generated and are filled (ms-vsliveshare.vsliveshare-pack-0.4.0 contains only some files).
Windows 10 Enterprise, Version 1803

Resolving as a dupe of #3136


Makefile: isolated makefile portion to manage the transition to pattern based system.

Just for the record; should be nothing unsafe about merging this!

@dosumis


Tests using toImage in Firefox fail in headless mode due to no WebGL

Tests using Picture.toImage do not work in Firefox since we run it in headless mode and headless mode has no WebGL support, which is needed for toImage in order to support texture-backed images.
Perhaps it would be better to disable browser image decoding if there is no WebGL support?

Looks like the issue is simply that we broke the CPU mode by adding a requirement for Picture.toImage  to use a WebGL surface. If so, should we fix the CPU mode?


Add registering text input

Temporary fix for https://github.com/leancodepl/patrol/issues/2110

@jBorkowska that change seems to singlehandledly break the behavior of the text input field in our app tests (run in the Android emulator) - downgrading to 2.0.1+1 resolves issues

the keyboard does not show up upon calling enterText if the field is not tapped first (less of a problem, though awkward)
the keyboard does not close when the text field edit is done / the focus is lost - it stays on forever (can't really close it even when tapping back in the emulator)

It doesn't look at first glance that there's something incorrect with the app, esp. as it behaves as expected during regular operations. Also, these tests worked fine with previous Patrol (2.3.2) and as mentioned they work fine if 2.0.1+1 version of the package is used.


Kupo retry mechanism tweaks

The following retry mechanism can be improved implemented in this commit: https://github.com/Plutonomicon/cardano-transaction-lib/commit/13adac22532a9bbedc7f52a11a975a6a53cdee21
It's just exponentially increasing the backoff time up to infinity.
There should be a maximum backoff time specified (e.g. 32 seconds) + delay randomisation should be introduced, too.

It's just exponentially increasing the backoff time up to infinity.

That's exactly how it should work. Only then we can guarantee that you will not self-DOS the server and if it has a fixed and non-zero capacity, eventually it will serve everyone.

delay randomisation should be introduced, too.

No need to, because the clients are not going to hit the server at the same time.


need help

can you help me with how to setup this rclone fork on ubuntu
i'm new at this and i really need to learn step by step how to do that. thank you

I don't have Ubuntu, so I can't really assist you. Check out official rclone sources :)


DNM: testing apple/swift-installer-scripts#120

testing removal of arch in layout

Please test with following PRs:
apple/swift-installer-scripts#120
@swift-ci please build toolchain Windows platform

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/120
@swift-ci please build toolchain Windows platform


Apply C++11 atomic feature to lock operations for the integrated LLVM OpenMP Library

Original author: Seonmyeong Bak
Original issue: https://charm.cs.illinois.edu/redmine/issues/1266

The integrated LLVM OpenMP Library reverted the patch applying c++11 atomic features for better compatibility with old compilers.
We'll keep the codes for backward compatibility but should maintain the codes in the base repo as well not to be divorced from the upstream.
Probably, two implementations of lock operations in the library should be chosen by a compiler flag such as STDC_VERSION >= 201112L(C11) or __cplusplus >=201103L(C++11)

Original author: Seonmyeong Bak
Original date: 2016-10-25 22:17:18

Seonmyeong Bak wrote:

The integrated LLVM OpenMP Library reverted the patch applying c++11 atomic features for better compatibility with old compilers.
We'll keep the codes for backward compatibility but should maintain the codes in the base repo as well not to be diverged from the upstream.
Probably, two implementations of lock operations in the library should be chosen by a compiler flag such as STDC_VERSION >= 201112L(C11) or __cplusplus >=201103L(C++11)


Restructured and set up travis.

Looks good! Seems like you have tests for most of the new functions, which is something I need to get on. If you give make me a collaborator on the project, I'll merge the pull request all above-board like.


[cpu][python backend]crash in python backend

System Info
test-embeddings-inference==v1.5.0
python==3.9
Run with CPU device
Information

[ ] Docker
[X] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction
Follow https://github.com/huggingface/text-embeddings-inference?tab=readme-ov-file#local-install to but with "-F python"
and start service with
model=BAAI/bge-large-en-v1.5
revision=refs/pr/5
./target/release/text-embeddings-router --model-id $model --revision $revision --port 8080 --dtype float32

Expected behavior
    cpu_results = embedding.view(-1).tolist()
RuntimeError: view size is not compatible with input tensor's size and stride (at least one dimension spans across two contiguous subspaces). Use .reshape(...) instead.

More explanations
I am trying to use the python backends and I met some error because of these two lines:
https://github.com/huggingface/text-embeddings-inference/blob/661a77ffba48f92fccda8c7b7302f6a973570016/backends/python/server/text_embeddings_server/models/default_model.py#L44-L45
I inserted some logs to see the shapes:
        logger.info(f"Python backend: input_ids {kwargs['input_ids'].shape}")
        output = self.model(**kwargs)
        logger.info(f"Python backend: output {output[0].shape}")
        embedding = output[0][:, 0]
        logger.info(f"Python backend: embedding {embedding.shape}")
        cpu_results = embedding.view(-1).tolist()

2024-07-15T08:32:45.772527Z  INFO text_embeddings_router: router/src/lib.rs:257: Warming up model
2024-07-15T08:32:45.778553Z  INFO python-backend: text_embeddings_backend_python::logging: backends/python/src/logging.rs:37: Python backend: input_ids torch.Size([32, 512])

2024-07-15T08:32:50.829333Z  INFO python-backend: text_embeddings_backend_python::logging: backends/python/src/logging.rs:37: Python backend: output torch.Size([32, 512, 1024])

2024-07-15T08:32:50.829401Z  INFO python-backend: text_embeddings_backend_python::logging: backends/python/src/logging.rs:37: Python backend: embedding torch.Size([32, 1024])

I am an newbee for text embedding. From my understanding here,
the batch size should be 32, seq_len should be 512, I wish to understand what is expected output embeding shapes?
The output should be torch.Size([32, 512, 1024])
This slice means we only choose the first embedding for 1 batch? Does this expected? We have 512 tokens per batch but we only want the first token's embeddings?
embedding = output[0][:, 0]

If it is expected, we may just modify
cpu_results = embedding.view(-1).tolist()

to reshape from the suggest in the error msg/
If it is not expected, should we use embedding = output[0] instead of embedding = output[0][:, 0]
and also correct
        return [
            Embedding(
                values=cpu_results[i * self.hidden_size : (i + 1) * self.hidden_size]
            )
            for i in range(len(batch))
        ]

With this example usage
curl <IP_ADDRESS>:8080/embed \
    -X POST \
    -d '{"inputs":"What is Deep Learning?"}' \
    -H 'Content-Type: application/json'

The input_id's shape is [1, 7] which means BS=1, seq_len=7, hidden_size=1024
Do we expected output embeddings shape=[1024] or [7168](7 * 1024)?

We have 512 tokens per batch but we only want the first token's embeddings?

Yes, that's called class pooling.
It should be a reshape instead of a view yes. Reshape forces the tensor to be contiguous.

We have 512 tokens per batch but we only want the first token's embeddings?

Yes, that's called class pooling.
It should be a reshape instead of a view yes. Reshape forces the tensor to be contiguous.

@OlivierDehaene, thanks!


Hosting to heroku not working. Help needed

Nothing seems to be working. I created a config.ru file for heroku and even filled in environment variables. It doesn't seems to bulge. Logs ain't even showing anything meaningful.
require 'middleman/rack'
run Middleman.server

Hey @osazemeu,
Middlemana is not intended to be ran as a server, it's a static site generator, therefore no need for a running server. Using GitHub Pages or Amazon S3 + some caching would be far more efficient.
Still if you want to proceed with the Heroku route, following this tutorial might be helpful https://jordanelver.co.uk/blog/2014/02/17/how-i-deployed-middleman-to-heroku/
Hope that helps
Cheers

thanks for the heads up


Upload file to GCS with presigned URL

Question
Hi, i want to upload file using resumable presigned url from client to GCS. I want to know if it's possible using this library.
If so can someone provide a details or example.

tus-js-client does not directly upload to GCS. tus uploads always have to go through a tus server, which can then store it on cloud providers. For example, https://github.com/tus/tusd is able to upload to GCS.


Roll Flutter Engine from dd37cefd4a94 to 5d97d2bcdffc (5 revisions)

https://github.com/flutter/engine/compare/dd37cefd4a94...5d97d2bcdffc
2024-06-20<EMAIL_ADDRESS>Roll Dart SDK from cbd933e707e7 to 366eb1b4b308 (1 revision) (flutter/engine#53487)
2024-06-20<EMAIL_ADDRESS>Roll Skia from 17626ca22729 to 4471ee07e223 (1 revision) (flutter/engine#53486)
2024-06-20<EMAIL_ADDRESS>Roll Dart SDK from e403519a4436 to cbd933e707e7 (1 revision) (flutter/engine#53485)
2024-06-20<EMAIL_ADDRESS>Roll Skia from 4acebac47ea5 to 17626ca22729 (1 revision) (flutter/engine#53484)
2024-06-20<EMAIL_ADDRESS>Roll Skia from 199e1a49b091 to 4acebac47ea5 (1 revision) (flutter/engine#53482)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jsimmons@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


rustic-buffer-workspace does not find the workspace root

If you have workspace with multiple crates, it gives you the directory of the crate which contains buffer's file, not the workspace root. The name and docstring indicate that this may not be intended.

It might be best to rely on the field workspace_root with the built in cargo metadata command.

I hope this helps #294


Adding null

This pull request was made by the Gatsby Branch Editor.

and i also did thwis one by accident too, sorry about that.


Invalid link representation

nodes: [
        {
          id: "1",
          text: "A",
          link: ["-- yes -->", "-- no -->"],
          next: ["2", "3"],
          editable: true
        },
        { id: "2", text: "B" },
        { id: "3", text: "C"}
      ],

Expected: https://raw.githubusercontent.com/robin1liu/vue-mermaid/HEAD/img/links.jpg
Actual:
graph LR
1[A]-- yes -->,-- no -->2[B]
1[A]-- yes -->,-- no -->3[C]
2[B]
3[C]

I found a workaround. When defining a separate node, it can be realized.
nodes: [
        {
          id: "1",
          text: "A",
          link: ["-- yes -->"],
          next: ["2"],
          editable: true
        },
        {
          id: "1",
          text: "A",
          link: ["-- no -->"],
          next: ["3"],
          editable: true
        },
        { id: "2", text: "B" },
        { id: "3", text: "C"}
      ],

But then why would the link attribute be an array if it can only have an length of 1? That doesn't male sense to allude to multiple links via that attribute's data type but then not be able to render it as expected

Of course It is definitely a bug since readme.md shows a node with multiple links are allowed.
https://github.com/robin1liu/vue-mermaid#different-link-values-of-next
So, my solution is just a workaround.

Any news about this bug?

I found a comment https://github.com/robin1liu/vue-mermaid/issues/11#issuecomment-569527049 mentioning that modifying vue-mermaid/package.json manually will resolve the bug.

Thanks, but i will prefer your workaround


Rename ModelChain test fixture system

This should be renamed to sapm_dc_snl_ac_sapm_temp_system follow the pattern for other fixtures.

@cwhanse I'm happy to take a stab at this. Also, what do you think about naming it to sapm_dc_snl_ac_system? it seems to be more consistent with the other fixture names. Thanks!

Ok with me to be sapm_dc_snl_ac_system and only have the temperature model terms added when that's important for the fixture.


Does the Filter Apply to the properties of the User Signing in or the Device?

Does the Filter Apply to the properties of the User Signing in or the Device?
My policy that is supposed to apply to Devices that aren't AD Registered is not working for my test user. The Device is AD Registered but for a different user than the one signing in now.

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

ID: fcc79933-58e1-66d1-81ae-f43802d41223
Version Independent ID: 5be1fefc-356e-d1eb-ab97-8cbe5654ff43
Content: Filter for devices as a condition in Conditional Access policy - Azure Active Directory
Content Source: articles/active-directory/conditional-access/concept-condition-filters-for-devices.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

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

@Dburke225
Thank you for your time and patience!
Based off our documentation, the condition filter for devices apply to devices.
For more info - Supported operators and device properties for filters.
When it comes to troubleshooting your policy, I'd recommend leveraging our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.


[release-v0.12] SHIP-0038: Run CI on Release Branches

Changes
Update our CI-oriented GitHub actions to run when commits merge in a release-v* branch, or a pull request is opened against a release-v* branch. With this change, future release branches will automatically have CI checks enabled. This commit should be backported to enable CI in a prior release branch.
This implements a portion of SHIP-0038.
See also https://github.com/shipwright-io/community/issues/85
Submitter Checklist

[ ] Includes tests if functionality changed/was added
[ ] Includes docs if changes are user-facing
[x] Set a kind label on this PR
[x] Release notes block has been filled in, or marked NONE

See the contributor guide
for details on coding conventions, github and prow interactions, and the code review process.
Release Notes
NONE

/kind cleanup

gosec picked up a few items that should not block merge of this.
Once this PR merges, I can set up the branch protection rules.

@adambkaplan @qu1queee  Can we merge #1547 before? Should fix golangci-lint errors.

@adambkaplan @qu1queee Can we merge #1547 before? Should fix golangci-lint errors.

This is done.
@adambkaplanI am okay with either: you rebase and this PR gets fully green or we red-merge it.

Doing the "red button" merge on this, so that I can then set up the branch protection rules.


update package.json file with new packages

I'm confident of solving this issue please assign me , I will create the pull request after 1st oct to be counted for hacktoberfest if you are okay with it ?

Can you please explain more about this issue??

Hello @yashmangal112  sir I Want To Contribute My 1st Contribution Into Your Project And I Am A CLg Student Can You Pls Assing me this Project . I will be so much glad to contribute in this project.


Files with filenames starting with a dot is not deployed

TL;DR
Files with filenames starting with a dot (e.g. .npmrc) is not deployed.
Expected behavior
Deployed like any other file.
.npmrc is required to install private node modules.
https://cloud.google.com/functions/docs/writing/specifying-dependencies-nodejs#using_private_modules
Observed behavior
The deply action succeeds, but fails to build functions because the file is not uploaded.
Reproduction
Action YAML

on: workflow_dispatch

jobs:
  deploy:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2

      - name: Add npm credential
        run: echo "//npm.pkg.github.com/:_authToken=${{ secrets.TOKEN }}" >> .npmrc

      - id: deploy
        uses: google-github-actions/deploy-cloud-functions@main
        with:
          name: *********************
          runtime: nodejs14
          region: asia-northeast1
          credentials: ${{ secrets.CREDENTIALS }}
          event_trigger_type: google.pubsub.topic.publish
          event_trigger_resource: ************************

Thanks for the report @egs33
Do you have a .gcloudignore file that is ignoring .npmrc?

I have a .gcloudignore, but it doesn't ignore a .npmrc

Hey @egs33 - I just pushed up an integration test that shows files beginning with a dot are correctly added to the deployment. If you're still experiencing this on main, can you comment with more information so we can try to reproduce? Thanks!


Test Aptomi installation methods in CI

Install script (compact) + end-to-end tests
Install on k8s + end-to-end tests

still have to test Helm-based install into k8s


feat(fscomponents): support more styling options for MultiCarousel

Feature
This add more styling options for MultiCarousel component.
Example Screenshots

(contentContainerStyle controls ScrollView content container style)

This is out of date with develop due to force pushes.


Only 24 images of an artist are shown

Currently when visiting the pixiv.js.org website and browsing trough the posts of an artist, only 24 max are shown and here's no option to show more.

The author @Dragon-Fish initially did not implement pagination, so there are only 24 images shown.

If I have enough free time. Maybe I will try to make it works in the 2023 Chinese New Year.


Small fixes

Fixes a broken link (in a long json file, only removed a /is.
Fixed linurit by making the category selector into a flexbox instead of a grid.
Switches back packages size small from 2 to 10.

also updates the babel-plugin-inline-react-svg package, this has a dependency to js-yaml that reports security issues on older versions.


condor_platform_select.sh: Fix Debian detection (#313)

Instead of parsing /etc/lsb-release (which has been obsoleted before Debian 9 Stretch), use /etc/os-release
This should fix #313

@steffen-AEI I was looking at https://github.com/glideinWMS/glideinwms/issues/313 and it looks like you found the right place, but did you mean to make this PR against the upstream glideinwms repo?

This PR is breaking the RHEL platform detection.
See https://github.com/glideinWMS/glideinwms/pull/320 for a possible solution

@jasoncpatton I think the issue should be resolved as upstream as possible
@mambelli whatever keeps the ball rolling is fine with me - I'm a bit unhappy with the "fallback logic" in the platform detector itself (making it necessary to change the detection once the true value can be handled: Debian 12 has been released weeks ago, and there's HTCondor for it, I've been told)


.streamrole Comand doesnt see mixer people online status

The .streamrole Comand doesnt see mixer people online status when streaming

@worldofmadness do you have a mixer stream? If you do, you could DM me and we could test it together. I don't know anyone who streams over there but I thought it worked from the tests I've done.

DM Sent

.streamrole only works on people who have streaming status on discord (purple one with stream name)


Fix gh action failures from forked repo

On moving this repo to redhat-developer org we need to fix the gh actions

Fixed by #250


Add hover effect to send button in contact us section

I want to modify send button a little bit and add a hover effect on it.
Please assign it to me under hacktoberfest.

@AnandSY This is already done , one who made the contact us actually did it just forgot to add the class name which I fixed it. But you can join our discord and suggest  custom issues and we'll create it for you.


search in the PDF

Hi, I'll like to know how can I search in the PDF file like Google PDF viewer.
Thank you.

I also want it

most needed feature.. please enhance this feature

Hi
anyone can help me how to highlight text or paragraph while reading ? plz let me know how to code for that

Hi, I'll like to know how can I search in the PDF file like Google PDF viewer. Thank you.

This app is all ready done anyone can use reverse engineering.
https://play.google.com/store/apps/details?id=pdf.reader.pdfviewer.pdfeditor

I also want it

Hi, I'll like to know how can I search in the PDF file like Google PDF viewer. Thank you.

This app is all ready done anyone can use reverse engineering.
https://play.google.com/store/apps/details?id=pdf.reader.pdfviewer.pdfeditor

@vimalcvs Was this app created by you?

pdfium should have this function, the problem is to implement it for android.  I remember the old app "google pdf viewer" which however was based on pdfium and had this function
I did some research and found some libraries that have implemented the search, but I don't know if they work, they should be tested.
It would take someone to test them and maybe fork and keep an updated fork.
Test them and let me know if they work in any way
https://github.com/gulshan98185/AndroidPdfViewer
https://github.com/beeline09/AdvancedPdfView
https://github.com/beeline09/android-support-pdfium
https://github.com/benjinus/android-support-pdfium


refactor(transformer/typescript): determine whether to remove ExportSpeicifer by ReferenceFlags

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#4513  👈
#4511
main

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

Join @Dunqing and the rest of your teammates on  Graphite


video crashing on book change from header

Apk crashes when plays the video.first time when selects the video from top menu, it plays well. Then if do change in book and select video from top menu bar apk crashes
[
https://user-images.githubusercontent.com/34941343/169771566-95c92341-866e-400b-b534-59dcaace1a7a.mp4
](url)

Completed Testing and this issue is fixed. As discussed moving this issue to done.


big: attempt to index field 'view' (a nil value)

Did you check docs and existing issues?

[X] I have read all the edgy.nvim docs
[X] I have searched the existing issues of edgy.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
v0.9.1
Operating system/version
13.0.1 (22A400)
Describe the bug
Pursuant to attmpting to resolve #27, I added symbols-outline.nvim as a dependency, but, no dice. Now the error is:
vim/_editor.lua:0: nvim_exec2(): Vim(lua):E5108: Error executing lua ...e/nvim/lazy/symbols-outline.nvim/lua/symbols-outline.lua:307: attempt to index field 'view' (a nil value)
stack traceback:
	...e/nvim/lazy/symbols-outline.nvim/lua/symbols-outline.lua:307: in function 'toggle_outline'
	[string ":lua"]:1: in main chunk
	[C]: in function 'nvim_exec2'
	vim/_editor.lua: in function 'cmd'
	.../andy/.local/share/nvim/lazy/edgy.nvim/lua/edgy/view.lua:128: in function <.../andy/.local/share/nvim/lazy/edgy.nvim/lua/edgy/view.lua:127>
	[C]: in function 'pcall'
	.../andy/.local/share/nvim/lazy/edgy.nvim/lua/edgy/util.lua:6: in function 'try'
	.../andy/.local/share/nvim/lazy/edgy.nvim/lua/edgy/view.lua:127: in function <.../andy/.local/share/nvim/lazy/edgy.nvim/lua/edgy/view.lua:122>

Also worth noting: I am unable to quit nvim with :q.  It simply toggles the first edgy window. I have to :qa for it to work.
Steps To Reproduce

Start from vanilla lazyvim installation
Enable edgy in lazyvim, using the standard ui plugin
create a ~/.config/nvim/lua/plugins/coding.lua file with the following contents:

return {
  {"simrat39/symbols-outline.nvim"},
}

save everything and open some code.

Result:

Expected Behavior
No error
Repro
As above.

nvm, needed to call setup.


Documentation should state the default output location

-nt-

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

ID: 910cd365-d63a-00dc-417c-db017f83bf6f
Version Independent ID: 5e495ddd-140e-de5c-a793-d4a9f88be51e
Content: dotnet build command - .NET Core CLI
Content Source: docs/core/tools/dotnet-build.md
Product: dotnet-core
GitHub Login: @mairaw
Microsoft Alias: mairaw

For reference it seems the default path is ./bin/<configuration>/<framework>/


HBASE-25285 [branch-1] Fix two methods removed problems reported by compat-check

Because they are HBaseInterfaceAudience.COPROC.
Let's keep them, just in case any coprocessor uses.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
docker
10m 40s
Docker failed to build yetus/hbase:b249092a5f.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/2658

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2658/1/console

versions
git=2.17.1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
docker
0m 11s
Docker failed to build yetus/hbase:b249092a5f.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/2658

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2658/2/console

versions
git=2.17.1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Archive JUnit-formatted test results
[2020-11-16T07:23:14.835Z] Recording test results
[2020-11-16T07:23:14.982Z] No test report files were found. Configuration error?

any hint?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
docker
0m 12s
Docker failed to build yetus/hbase:b249092a5f.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/2658

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2658/3/console

versions
git=2.17.1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Triggered new build, we should see QA result this time.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
12m  0s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ branch-1 Compile Tests _

+1 :green_heart:
mvninstall
9m 53s
branch-1 passed

+1 :green_heart:
compile
0m 40s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
compile
0m 45s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
checkstyle
1m 42s
branch-1 passed

+1 :green_heart:
shadedjars
3m  2s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 48s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javadoc
0m 41s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+0 :ok:
spotbugs
3m  8s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m  4s
branch-1 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 58s
the patch passed

+1 :green_heart:
compile
0m 41s
the patch passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javac
0m 41s
the patch passed

+1 :green_heart:
compile
0m 45s
the patch passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
javac
0m 45s
the patch passed

+1 :green_heart:
checkstyle
1m 32s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
2m 49s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
4m 38s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2.

+1 :green_heart:
javadoc
0m 30s
the patch passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javadoc
0m 41s
the patch passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
findbugs
2m 54s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
125m 25s
hbase-server in the patch passed.

+1 :green_heart:
asflicense
0m 35s
The patch does not generate ASF License warnings.

178m 33s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2658/4/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2658

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux db8abcbc8621 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-home/workspace/Base-PreCommit-GitHub-PR_PR-2658/out/precommit/personality/provided.sh

git revision
branch-1 / edccfe4

Default Java
Azul Systems, Inc.-1.7.0_272-b10

Multi-JDK versions
/usr/lib/jvm/zulu-8-amd64:Azul Systems, Inc.-1.8.0_262-b19 /usr/lib/jvm/zulu-7-amd64:Azul Systems, Inc.-1.7.0_272-b10

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2658/4/testReport/

Max. process+thread count
3352 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2658/4/console

versions
git=1.9.1 maven=3.0.5 findbugs=3.0.1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
7m 52s
Docker mode activated.

-1 :x:
patch
0m  8s
https://github.com/apache/hbase/pull/2658 does not apply to branch-1. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2658/5/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2658

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2658/5/console

versions
git=1.9.1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Buttons do not respond to clicks in HTML export

I created a simple demo using the first script tutorial - click a button to change some text.
This demo works in preview, as well as an OSX export, however in the HTML export the button does not respond to clicks at all.
Here's the code

And the layout

Pressing the button changes the text, however this does not happen in the HTML exported version.
Tried this in both Mac OSX Chrome and Firefox.

Since there was quite some work done on the HTML5 exporter for 2.0, and the bug report being quite old, I'll assume it's fixed ans archive it. Feel free to reopen if it's not the case in the current master branch.


Feature Request: Add non-GPU build / docker instructions

I do not currently have a NVIDIA GPU but would like to test out the bostontwin_demo.ipynb inside a docker container. It would be great if there were instructions on how to do that in the README.
Thanks!

Hi @Blake-Haydon,
thanks for your input. I'll add some instructions as soon as possible. Are you trying to use the Docker container, or do you use a Python virtual environment (e.g., conda or venv)?

Same issue as @Blake-Haydon. I am trying to run this on a MAC OS system without any gpu support.
This command fails ---
b. Run the container binding the boston_twin directory to the container: docker run --privileged=true --gpus=all --mount type=bind,src=".",target="/home/root/boston_twin" --env NVIDIA_DRIVER_CAPABILITIES=graphics,compute,utility --rm -it bostontwin
Please provide an alternative at the earliest.

Hi @hiteshPoddar,

Without a error log, it's hard to tell what's the problem
On MAC OS, there are differences in the tensorflow installation with/without silicon chips (M1-3), and I'm not sure how Mitsuba (the underlying library that Sionna uses for RT) deals with the M-* chips.
I would suggest you try the Sionna Docker image or local installation first, and see if that runs on your device. The additional libraries required by Bostontwin should not have any compatibility issue with MAC OS.

In general, to run it on CPU, it should be enough to remove the --env and the --gpus=all flags


Enhance: year selection

Is there a way to make year selection easier? Not just by typing it.
I mean if I want to select date of birth I should go through all years down to mine one. For some elder users it could be annoying.

Hello,
I don't see any problems with typing the year into the input.
Its much faster than having the user go through multiple unnecessary views via plethora of clicks.

chiming in just FYI because I'm facing the same situation. the issue is that it's not clear to the user that they can type the year. if there's an easy way to change the input field for the year into a select list, I'd love to hear about it.

Just wanted to comment as well, our client came with this exact issue; She thought she'd had to press 300 times to go back to a date back in 1992.
For your info, after figuring out how it actually works I think it's probably the fastest and cleanest way I've seen in a date picker. I'd suggest we just need to look for a way to communicate better that It's possible to change the year there. Perhaps making it look more like an 'input field' or 'clickable' object, even when not hovering it.

Thanks for chiming in @crashkonijn
Yes, it'd be great if users knew they could simply input the date. As a first step, I've added a text cursor to the year input. Next, I'll probably give the year an off-white background to make it stand out.
If you've got any more ideas on how to improve this, please feel free to do it right here :)

@pirco selects  are possible with plugins, but I've opted against them for many reasons. Out of all calendars I've worked with, none had a simple and efficient way of year input. Nearly all of them involve clicking around multiple times to select a year. Year selection should be so user-friendly that its invisible.

we just need to look for a way to communicate better that It's possible to change the year there.

thanks @chmln. in my opinion, and considering that day and month selection are made via mouse/trackpad, I find selects the better UIX here because users wouldn't have to switch to keyboard and type. a select list is just a click-scroll-click away form any year.
but, again, it's just my opinion and I'm glad there's an option to use a select by way of plugin.
I'm just hoping that someone could help me with that (since this would have to be a new plugin, right?)
thanks for the awesome product BTW.

@chmln Thanks four your response!
How about something like this:

Just a quick test, basicly make the hover background always visible and a little extra offset from the left.


seq66: fix build against gcc-13

Without the chnage build fails on master as:
In file included from os/daemonize.cpp:64:
../include/os/daemonize.hpp:58:8: error: 'uint32_t' does not name a type
   58 | extern uint32_t daemonize
      |        ^~~~~~~~
../include/os/daemonize.hpp:36:1: note: 'uint32_t' is defined in header '<cstdint>'; did you forget to '#include <cstdint>'?
   35 | #include "seq66_platform_macros.h"      /* for detecting 32-bit builds      */
  +++ |+#include <cstdint>

As upstream changes are quite invasive for backport I am only injecting missing headers.
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Great catch! I think it's a side-effect of a missing plugin path. As a result qt fails to find any rendering backends. Attempted to fix as seq66: enable Qt wrapper to fix plugin loading. This allows starting GUI for me.

Seems to run successfully against pipewire for me:
nix run github:NixOS/nixpkgs/pull/280768/head#seq66
[Reading rc configuration /home/slyfox/.config/seq66/qseq66.rc]
[Reading MIDI control file /home/slyfox/.config/seq66/qseq66.ctrl]
[32 loop-control lines]
[32 mute-group-control lines]
[50 automation-control lines]
[Reading mute group file /home/slyfox/.config/seq66/qseq66.mutes]
[Filter-by-channel off]
[Reading user configuration /home/slyfox/.config/seq66/qseq66.usr]
[Non-I/O port 'PipeWire-System']
[Non-I/O port 'PipeWire-RT-Event']
[Initialized, running without JACK sync]
3 rtmidi ports created:
Input ports (2):
  [0] 0:1 system:announce (system)
  [1] 14:0 Midi Through:Midi Through Port-0
Output ports (1):
  [0] 14:0 Midi Through:Midi Through Port-0


Accelerated single-step system hardware config

Co-Authored-By: Richard Pioso<EMAIL_ADDRESS>Change-Id: Iccdef973d4456a4475ea04e10c52f9adccd23953
Story: 2003594
Task: 24914

@ajya, this pull request contains my feedback up to the Export configuration subsection in Proposed change.
Please let me know what you think.