JMeter Dynatrace OSS and Data File Support

Closes #2893
Signed-off-by: Andreas Grabner<EMAIL_ADDRESS>Signed-off-by: Florian Bacher<EMAIL_ADDRESS>

Codecov Report

Merging #2931 (ed95513) into master (15a8368) will decrease coverage by 25.41%.
The diff coverage is 25.14%.

@@             Coverage Diff             @@
##           master    #2931       +/-   ##
===========================================
- Coverage   36.34%   10.92%   -25.42%
===========================================
  Files         104        3      -101
  Lines        6472      421     -6051
  Branches      197        0      -197
===========================================
- Hits         2352       46     -2306
+ Misses       3812      371     -3441
+ Partials      308        4      -304

Flag
Coverage Δ

moduleA
?

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

Impacted Files
Coverage Δ

jmeter-service/main.go
2.10% <0.00%> (ø)

jmeter-service/jmeterUtils.go
11.92% <19.69%> (-3.23%)
:arrow_down:

jmeter-service/common.go
23.77% <31.52%> (+23.77%)
:arrow_up:

bridge/client/app/_services/http-state.service.ts

...ient/app/_interceptors/http-loading-interceptor.ts

...mediation-service/handler/problem_event_handler.go

shipyard-controller/common/pagination.go

...nfiguration-service/common/mongodb_project_repo.go

.../pkg/configurationchanger/configuration_changer.go

... and 79 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #2931 (ed95513) into master (15a8368) will decrease coverage by 25.41%.
The diff coverage is 25.14%.

@@             Coverage Diff             @@
##           master    #2931       +/-   ##
===========================================
- Coverage   36.34%   10.92%   -25.42%
===========================================
  Files         104        3      -101
  Lines        6472      421     -6051
  Branches      197        0      -197
===========================================
- Hits         2352       46     -2306
+ Misses       3812      371     -3441
+ Partials      308        4      -304

Flag
Coverage Δ

moduleA
?

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

Impacted Files
Coverage Δ

jmeter-service/main.go
2.10% <0.00%> (ø)

jmeter-service/jmeterUtils.go
11.92% <19.69%> (-3.23%)
:arrow_down:

jmeter-service/common.go
23.77% <31.52%> (+23.77%)
:arrow_up:

bridge/client/app/_services/http-state.service.ts

...ient/app/_interceptors/http-loading-interceptor.ts

...mediation-service/handler/problem_event_handler.go

shipyard-controller/common/pagination.go

...nfiguration-service/common/mongodb_project_repo.go

.../pkg/configurationchanger/configuration_changer.go

... and 79 more

Continue to review full report at Codecov.

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


24 hour 12 hour clock toggle

Just a nitpick. Allow users to choose if they want 24 hour or 12 hour clock in settings.

Probably a smart idea

Shouldn't this be controlled by the time_format configuration option in webkit2-greeter's config?

It should be, but I tried playing around with it and nothing changed at all. The default value for it is LT auto, which should take the local defaults on that, with no dice. Even when I manually forced them to be US based with am pm, nothing changed. I think he has the time coded separately.

Sorry, yes, I know that. My earlier comment was meant as a suggestion for how to implement localized time (12/24-hour). That is, as opposed to worrying about something like whether AM/PM should be included, just let moment.js handle figuring out the actual text.
I assume webkit2-greeter has some kind of API call to get the localized time using the settings in its config file, but maybe the theme has to call moment.js directly? Might be worth looking to see how the default antergos theme does it.

I'm not using webkit2 greeter API for this, i'm using it for langage and i'm using webkit locale function for date, but i implemented the clock manually. I'll have a look at it

you can enable it in the Setup screen
Can you be a bit more specific? How do I get to this setup screen?

The Setup screen is only on the pre-release which you can install manually from here https://github.com/Litarvan/lightdm-webkit-theme-litarvan/releases


Support Popper v2 virtual element

Dropdowns accept a reference option for Popper however the type is restricted to string|element.
Popper itself on the other hands also allows objects which provide a getBoundingClientRect method.
This can be quite useful when one wants implement custom context menus, arbitrary popup location on maps/images or otherwise does not have an actual DOM element as a reference but instead only raw coordinates.
The only thing required should be to also accept object (or only object which have getBoundingClientRect set).

PS: I mocked the check to always pass and have not found any issues. Also a quick scan of the code showed no problematic sections. The only thing one might add is a check in _getConfig whether or not the reference has a getBoundingClientRect method if it is not of type string|element


DM's CI forget unstash some coverage files

seem in https://github.com/PingCAP-QE/ci/blob/main/jenkins/pipelines/ci/dm/dm_ghpr_new_test.groovy, we only
            unstash 'integration-cov-shardddl1'

but forget to
            unstash 'integration-cov-shardddl1_1'

please check if it's needed

/assign @Ehco1996

fixed in https://github.com/PingCAP-QE/ci/pull/253


[ECS] Add checkstyle output format

Hi,
First, thank you for ECS. I find it more natural to write ECS config than php-cs-fixer,
It would be nice to have a checkstyle output format that can be used eg. in CI with cs2pr.
I was surprised not to find it in ECS after switching from php-cs-fixer to ECS earlier this week, but maybe I did miss it?

Hi, thank you for nice words.
I though I've already replied it, but probably missed the reply button in metro.
This style would be welcomed :+1:
You need to imlement Symplify\EasyCodingStandard\Contract\Console\Output\OutputFormatterInterface to add it

Closing as not an issue and suggestion is accepted. To keep focus on relevant and opened issues.
Feel free to send PR +1


feat(ffi): add add_text_comment

While the addition of set_comment allows for the comments to be used as per the Pact specification, it did not follow the original intent for comments (see the discussion in pact-foundation/pact-compatibility-suite#8).
The new add_text_comment function allows comments to be added repeatedly and appended into an array as intended in pact-foundation/pact-specification#45.

Thanks, I missed the comments_mut() method. I've also taken the opportunity to refactor the previous set_comment method (as I did the unnecessary matching there too).
And good catch with the matching 😅


ENH: Homepage Creation/Thumbnails

Describe the solution you'd like
Homepage needs to serve thumbnails
Thumbnails need to link to video page invidually
video hash will be link and thumbnail file name
video hash will be video identifier instead of video id

More functionality required, still in testing

video hash will be link and thumbnail file name
video hash will be video identifier instead of video id

Linked to #45

Closed with PR #89


Custom data connector OAuth in Tableau Server v2021.1.18

About You:
Name: Wan Chun Chen
Company: JP Morgan
Your question:
Trying to build a custom data connector with OAuth integration with our internal OIDC provider.  It worked if I run it through Tableau Desktop.  I'm able to get the access token.  When I test the data connector on Tableau Server, I'm seeing this log in vizportal-node log file.
Request received: GET for /v1/auth/request_oauth
INFO com.tableausoftware.domain.content.oauthconnections.OAuthClientService - features.SiteLevelOAuthClient is disabled, request will not be completed
ERROR com.tableausoftware.api.webclient.rest.util.WebClientRestControllerAdvice - OAuthException in WebClientController: (errorCode: 170006)
INFO com.tableausoftware.app.vizportal.LoggingInterceptor - Request completed: /v1/auth/request_oauth with status 400
How do I enable SiteLevelOAuthClient?  Is this the cause of my error?

We have several docs that explain this, but they are specific to connectors. We will be adding a general docs that would apply to plugins soon. But the steps are basically the same. Here is an example for the Dremio connector.
https://help.tableau.com/current/server/en-us/config_oauth_dremio.htm
See the section for "Configure custom OAuth for a site"

We just upgraded Tableau server to 2022.3.  I also added my OAuth client to the registry as described in the dremio link.  I was able to get further now.  My browser re-directed me to my OIDC provider where I can enter my credential.  I also see from the logs that it called back to Tableau.  I see this.
INFO Request received: GET for /v1/auth/add_oauth_token
Now I'm getting a HTTP 400 Bad Request error when Tableau attempts to exchange authorization code for access token.  From the logs I see
POST https:///token?grant_type=authorization_code&client_id=&redirect_uri=
The URL above didn't include client_secret or code query params as required when making the /token request.  Is this because Tableau didn't include it?  Which would explain the HTTP 400 error but why wouldn't Tableau include it?
Or is it simply that the log excluded those two query params for privacy reasons?

We won't log the secrets. Client secret is optional for native/local apps because they don't have the same security issues as non-local apps, and also because there is no way to keep the client secret a secret anyways.

What OIDC provider are you using?

There is no way to configure TS to authenticate without the client_secret. It will always pass something.

It's an internal OIDC provider.  Is it possible for Tableau Server to support OAuth clients with no secret?  Client secret is optional in the OAuth spec.
https://www.rfc-editor.org/rfc/rfc7591#section-3.2.1
Don't see why Tableau Server need to make it mandatory.

It's required for confidential clients, which are all clients for Tableau Server (as opposed to Tableau Desktop).
https://oauth.net/2/client-types/

If this is an internal OIDC provider, can you just make it ignore any client_secrets that may be passed in? Then you can configured some dummy value on TS.

We will probably make this change, but it may be much faster to fix it on your end.

We have no control over the behavior of the OIDC provider so that's not possible unfortunately.  Also, a confidential client doesn't necessary have to authenticate using client secret.  There are other forms of authentication.
From https://www.rfc-editor.org/rfc/rfc6749#section-2.1
confidential
Clients capable of maintaining the confidentiality of their
credentials (e.g., client implemented on a secure server with
restricted access to the client credentials), or capable of secure
client authentication using other means.


Blocks

Need literal blocks for both variables and math equations as part of the MVP
Blocks should follow an inheritance hierarchy which is up to the developer
Blocks need a getter which returns the type of block for the whiteboard
Blocks also need a  spawning method for the block spawner, to be determined, but calling a block.type.spawn (or something like that) should spawn an instance of that block.

Blocks also need to be deleted
Blocks also need to be manipulated in VR (also talk to Julia)


SCF stability analysis does not work for stretched H2

The H2 molecule has a well-known UHF instability at separation, which is found with this input
from pyscf import gto, scf

mol = gto.M(
    atom = '''H 0 0 0
    H 0 0 10''',
    basis = 'cc-pVDZ',
    verbose = 4
    )
mf = scf.UHF(mol).run()
mf.stability()

However, when using a larger basis set, the instability is no longer found by PySCF. Instead, the wave function is deemed stable.
mol = gto.M(
    atom = '''H 0 0 0
    Ghost:He 0 0 5
    H 0 0 10''',
    basis = 'cc-pVDZ',
    verbose = 4
    )
mf = scf.UHF(mol).run()
mf.stability()

This is clearly a bug; other programs do find an instability also in this case.

This was an issue in PySCF 2.2.1. Looks like the master branch no longer has this problem.


The Jobs REST API feature should auto-retrieve objects defined as ObjectVar fields

Environment

Python version: 3.8
Nautobot version: 1.0.2

Steps to Reproduce

Create a Job with a field defined as ObjectVar
Take necessary steps to have job available to Web and Worker instances
Use the REST API to start a job (It is unclear how the ObjectVar field should be represented, but I would expect a dict with key/value pairs matching the Related Object's field/value pairs, such that calling the ORM's RelatedObjectModel.objects.get() with the data would return the single matching record (or else raise an exception).

Expected Behavior
I expected the field to automatically look up the Record in the ORM as defined in the payload, and the variable in the job to be a ModelBase instance.

Observed Behavior
The field is not looked up and is whatever data is sent via the payload. For instance, if I pass a string, say the UUID of the Related Object, it will be the string passed in via the payload. If is a dict, say something like {"id": uuid_of_related_object}, it is a dictionary with the key value pair(s) passed in via the payload. This results in later code calling attributes on the expected ModelBase instance to raise errors, since these attributes are not defined for str or dict objects.

This is related to #651 but is a distinct side effect of that behavior, and fixing that behavior will need to address how to validate ObjectVar fields, as these cannot be passed via REST.

For anyone looking for a workaround, you can use try/except or isinstance in the job to catch this before executing code, and normalize the field data in the job before executing any code using the field.
from nautobot.dcim.models.sites import Region
from nautobot.extras.jobs import Job, ObjectVar

class CustomJob(Job):
    region = ObjectVar(
        model=Region
    )

    def run(self, data, commit):
        region = data["region"]
        if isinstance(region, dict):
            region = Region.objects.get(**region)
        elif isinstance(region, str):
            region = Region.objects.get(id=region)

Thanks for the report!


Old issue in forward pass objtypespec analysis

Old issue in forward pass objtypespec analysis exposed by recent backward pass fixes. If we have a monomorphic check upstream and reach a point where a monorphic check expects a different type, we can't replace the checked type in the value info. We have to indicate a type mismatch.

@rajatd @LouisLaf

Looks good.


avoid to use reserved word 'interface'

Uncaught SyntaxError: Unexpected strict mode reserved word (at library.js:50:3)
raises while loading 'library.js' as external js module.

Thanks!


Domain support

Hi, thanks for supprting a so handsome script. I have been use ocserv for a long time, and i am willing to use domain instead of ip address. I am new in openvpn, can you consider add domain guide support to this script?

Currently you can use domain names with this installer when setting up servers behind NAT.
If your server is not behind NAT, you will not have the chance to specify a domain name, except if you edit the script to achieve this. That said, and considering the script design, using domain names will not be very useful in most cases.

Currently you can use domain names with this installer when setting up servers behind NAT.
If your server is not behind NAT, you will not have the chance to specify a domain name, except if you edit the script to achieve this. That said, and considering the script design, using domain names will not be very useful in most cases.

thanks, I will try again later. Two more questions, the first one, would you be interested in writing a script similar to this one for deploying ocserv? Second, how to set up pac or no-route (similar to ocserv)?

would you be interested in writing a script similar to this one for deploying ocserv?

No

how to set up pac or no-route (similar to ocserv)?

Read the OpenVPN manual.

alright, thank you very much


Improved the UI of Amantran Food Ordering

Description
Improved the UI of Amantran Food Ordering
@khushi-joshi-05 @sunny0625 Please review and merge this PR as soon as possible.
Related Issues
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]

Closes #1577

Type of PR

[x] Bug fix
[x] Feature enhancement
[x] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]

Checklist

[x] I have gone through the contributing guide
[x] I have updated my branch and synced it with project main branch before making this PR
[x] I have performed a self-review of my code
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

@sunny0625 Please merge this PR as soon as possible.

@sanjay-kv  Please merge this PR as soon as possible.


How does apache IOTDB query the time of the last change of data value?

In IoTDB, there are time series under root.device_id, where xxxx represents multiple series. I would like to query for the last changed value and the time of change under root.device_id. How should I write the query statement?
such as :1,2,2,3,3,3,3,3,3，i need find  when data change  from 2 to 3

@JackieTien97

Hi, you can see the answer in this links: https://stackoverflow.com/questions/77719939/how-does-apache-iotdb-query-the-time-of-the-last-change-of-data


Update lodash package

https://www.npmjs.com/advisories/782

Could you please update your dependency of lodash package to the latest?
I'm getting warning vulnerability message because of your dependency.

Done.

Publish the update to 'npm' please. I still download the oldest version.


fix print schema column without reftype

fix panic
$ go run ./cmd/print_schema/print_schema.go ovn-nb.ovsschema
OVN_Northbound, (5.31.0)
	 Connection([[target]])
		 target => string [M]
		 other_config => [string]string [M]
		 status => [string]string [E,M]
	 NB_Global
		 connections => %!s(PANIC=String method: runtime error: invalid memory address or nil pointer dereference)
		 options => [string]string [M]

$ go run ./cmd/print_schema/print_schema.go ovn-nb.ovsschema
OVN_Northbound, (5.31.0)
	 Connection([[target]])
		 status => [string]string [E,M]
		 target => string [M]
		 other_config => [string]string [M]
	 NB_Global
		 name => string [M]
		 connections => [] [Connection ()] (min: 0, max: -1) [M]
		 options => [string]string [M]

Pull Request Test Coverage Report for Build 863191951

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 72.609%

Totals

Change from base Build 859723497:
0.2%

Covered Lines:
1784

Relevant Lines:
2457

💛  - Coveralls

Thanks @halfcrazy.
Having recently moved all optional fields to pointers, we've made it more cumbersome to use the resulting objects.
Maybe we should add functions to access all of the optional fields so we can set their default value properly. So, I think we should do the same thing we did for ColumnSchema.Mutable
https://github.com/ovn-org/libovsdb/blob/448344537277adffe67a6b5df632c5f411ba407d/ovsdb/schema.go#L387-L393
for all the optional fields of BaseType: BaseType.RefType, BaseType.MinReal, BaseType.MaxReal, etc.
@dave-tucker wdyt?

@amorenoz yes that's a good idea

@amorenoz see #121
@halfcrazy if we go with :point_up: to fix the panic, I'd love for you to rebase and then contribute the test so we don't cause regressions in future.

ok, I'll add tests after your pr merged.

@halfcrazy we've merged changes to BaseType that should prevent the panic now.


Add support for updates with replacement objects

Closes #697
The MongoDB documentation states that when a update call is made with no operation on the update parameter, it is treated as a Replacement Object and the whole document is replaced by the contents of the update payload.
More details here:
https://www.mongodb.com/docs/manual/reference/method/db.collection.update/#std-label-update-parameter
Checklist

[x] Tests are added for new functionality or fixed bugs.
[x] task all passes.

See CONTRIBUTING.md for more details.

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

Ready for review.

@w84thesun tests, linters fail


Incompatibility with Inspecio

Latest.log - https://paste.ee/p/OIKr3
Occurs on startup, assumedly an incompatibility with tweaks that affect tooltips
With just Fabrication-1.18.2-exp2, Inspecio 1.3.0, FAPI
Also posted here https://github.com/Queerbric/Inspecio/issues/67
If it's possible to isolate the issue by banning features, this might also be useful.

Confirmed doesn't occur with all features banned, looking into it by category.

Confirmed as not occurring when all features are banned, and occurring when all features are banned except "Use items while Riding" - incompatibility can be avoided this way, however the feature also prevents players from loading into worlds (with "invalid packet") without inspecio, so it might be a related issue with 1.18.2.

exp3 has been released, which may fix this issue: https://github.com/unascribed/Fabrication/releases/download/v2.3.0/fabrication-1.18.2-2.3.1-exp3.jar

Confirmed working on exp3


kubectl logs -A

What would you like to be added:
I would like to add the option -A (--all-namespaces).
Most other kubectl subcommands have this option.
Why is this needed:
I would like to get all logs (from all namespaces) from the last N minutes.

/assign

/sig cli
/area kubectl
It will also be better to take the opinion of sig experts regarding this to get it's triage accepted.

@sbdtu5498 will you ask in #sig-cli? Please mention me, too. Thank you

@guettli sure thing!

/close
I checked some of the plugins that are available via krew and found stern. It is a plugin that could handle this feature for you. We believe this would be best served as a plugin at this stage. Please let me know if this is an acceptable solution for your use case.

@mpuckett159 of course I could use Go (or the krew plugin stern) and help myself. But I still think it would be convenient if kubectl log would have the -A option like other kubectl sub-commands have it.
I don't have time to implement it in the next weeks. But maybe someone else has time.
@mpuckett159 If there is a motivated developer, is it likely that a PR would get accepted?


Fix wialon protocol speed units

Why 218 km/h top speed did not confuse me earlier...
Same problem, it is km/h according to documentation.

Thanks for pull request.


pod库里面使用的版本有升级吗？

@chenjiang3 we already provide 1.6.beta on cocoapods. try to run pod repo update if you can not find newly version.


change add contributor modal height

Purpose
The Add Contributors Modal is too large, and goes off the page on most screens.

Changes
Override css on the add contributors table from the default treebeard size.

Side effects
It might be too awesome.
Ticket

We can get this into release/0.68.0. @billyhunt Do not merge develop into this PR.

@sloria, done (the change to require) and not done (the 💩  merge).

Great--will merge this after I'm done with the hotfix I'm working on.


NEUS frame of reference (axis orientation)

Hello, I have a question: what reference of frame is used in your project? In other words, what is the orientation of the XYZ axis for the training data? I want to use the data for training from non-Colmap photogrammetry software (e.g. Agisoft or Reality Capture). Thank you and great work!

same question with you!


[Node Finder] Clear button is not functional after opening node details dialog

Description
As we discussed earlier, Clear button should update the table if the applied fields are not in the default case
Steps to reproduce

go to node finder
select any field in this case i will select Gpu then click on apply
click on clear; you will notice that the table is not changed back to the init state

Screencast from 14 ينا, 2024 EET 02:42:00 م.webm
Logs/Alerts
Logs/Alerts if available

Work Completed:
we decided that the current logic will not scale as wanted, we drafted the old pr and now we are enhancing the filters, almost done.
Investigation and solution:
will apply the new filters to the dedicated machine page
add query route to the filter component

verified working fine now
env used : devnet
version :  9d076f1
Screencast from 22 ينا, 2024 EET 11:07:48 م.webm
testlodge : https://app.testlodge.com/a/26076/projects/40893/suites/234374?expand_section=426661#case_4150464

Verified:

Devnet
7b55dd4


Add the ability to configure the port used the the postgresql server

HI @ricardozanini, it looks like I can't add reviewers, would you mind review?

@fjtirado please review too

@fjtirado can you please create a 0.0.x branch based on 0.0.9 tag that is not upgraded to Quarkus3, so we can cherry-pick there and release 0.0.10?
BTW, which version are we using on Kogito?

@fjtirado can you please create a 0.0.x branch based on 0.0.9 tag that is not upgraded to Quarkus3, so we can cherry-pick there and release 0.0.10?
BTW, which version are we using on Kogito?

we are using 0.0.9

@fjtirado can you please create a 0.0.x branch based on 0.0.9 tag that is not upgraded to Quarkus3, so we can cherry-pick there and release 0.0.10?
BTW, which version are we using on Kogito?

we are using 0.0.9

@wmedvede @ricardozanini Im not sure if you are aware that there is already a quarkus2 branch for this extension where this change shoud be cherry picked to.


has-danger and col-form-label

The validation feedback for errors (.has-danger) has not been applied to the col-form.label.

    Name
    <input type="text" id="name" name="name" class="form-control">
    The name field is required.

I'd changed the class for the validation error message to .form-control-feedback, and it now works it, but the issue still remains for the input label. What's the correct class?

In other parts of the docs, there is a reference to .form-text class. What's the correct one: .form-text or .form-control-feedback?

If you're using v4, make sure you have the latest code—this should be working just fine with Alpha 5.
From our documentation:

v3's validation docs
v4's validation docs

If I am using the form on the grid, as in the docs:
http://v4-alpha.getbootstrap.com/components/forms/#using-the-grid
Do I have to use both col-form-label and form-control-label classes on my label?

Ah, sorry, yes, right now you do. I think we need to revisit that. Reopening and retitling the issue so we can tackle that.

Thanks!


Fix memory leak by explicit calling on plan frament dtor in task

In some edge case, task destruction doesn't call planFragment_ dtor
which will hold the vectors in plan node such as Values. The latter
can cause memory leak which can be detected by the memory tracker.
The subclass member destruction should be called auto by compiler
generated code so we don't know in that case it is not invoked.
This PR fixes the problem by explicitly call planFragment_ dtor, and
verified the fix with ApproxDistinctTest.groupByIntegers test.
Also include memory check PR#2606 from @mbasmanova

@xiaoxmeng Thanks. Should we add the check for allocated memory to be 0 in memory pool dtor in this PR?

Sure. Updated.


wire run_scam on "run scam on all unchecked images"

Currently when running scam on all images, the web ui calls run_scam_file on each file appearing in the UI, it would be more efficient to just call run_scam and use the resulting json

let's drop this


[SPARK-33907][SQL] Only prune columns of from_json if parsing options is empty

What changes were proposed in this pull request?

As a follow-up task to SPARK-32958, this patch takes safer approach to only prune columns from JsonToStructs if the parsing option is empty. It is to avoid unexpected behavior change regarding parsing.
This patch also adds a few e2e tests to make sure failfast parsing behavior is not changed.
Why are the changes needed?

It is to avoid unexpected behavior change regarding parsing.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Unit test.

Test build #133523 has started for PR 30970 at commit 5c5616a.

Reminder: this was already in branch-3.1 first. So don't need to backport it. Thanks.

Test build #133524 has started for PR 30970 at commit 333fbcb.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38111/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38113/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38111/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38113/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38111/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38113/

Test build #133523 has finished for PR 30970 at commit 5c5616a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133523/

Test build #133524 has finished for PR 30970 at commit 333fbcb.

This patch fails from timeout after a configured wait of 500m.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133524/

The second commit is only comment. Merging to master. Thanks @dongjoon-hyun @HyukjinKwon


Notifications not respecting filters

Describe the bug
I keep receiving notifications for changes that should be filtered out.
Version
v0.45.16
To Reproduce
Steps to reproduce the behavior:

Create a new watch with a discord notifications and add filters
Wait for a change of the website that includes something in the filters
Receive the notification

My instance is self-hosted so I don't have a link to share.
Expected behavior
I would expect the change to be filtered out and a notification not be triggered.
Screenshots

Additional context
Not sure if it's a bug or if I'm understanding something incorrectly.
I have also tried to use filters without the regex pattern.

the filters will filter WHAT is triggered on, but the actual notification will include everything that changed.
you should consider these other tokens/placeholder in the notification body template

Thanks for the explanation, but I'm not sure I follow. How is filtering what triggers a change detection distinct from what gets sent as a notification?
I'm trying to set this up so I only get notified on what matters to me.


Make consistent links to repository in the Slack/Mattermost notificiations

Fixes #9195

Codecov Report

Merging #9205 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #9205      +/-   ##
==========================================
- Coverage   41.06%   41.05%   -0.01%
==========================================
  Files         553      553
  Lines       72041    72046       +5
==========================================
- Hits        29586    29582       -4
- Misses      38724    38732       +8
- Partials     3731     3732       +1

Impacted Files
Coverage Δ

modules/webhook/slack.go
0.91% <0%> (-0.02%)
:arrow_down:

modules/indexer/indexer.go
44.73% <0%> (-10.53%)
:arrow_down:

modules/migrations/migrate.go
23.78% <0%> (-1.63%)
:arrow_down:

models/repo_list.go
73.14% <0%> (-0.93%)
:arrow_down:

modules/migrations/gitea.go
8.82% <0%> (-0.64%)
:arrow_down:

models/repo.go
47.17% <0%> (-0.05%)
:arrow_down:

models/error.go
33.86% <0%> (+1.18%)
:arrow_up:

modules/task/migrate.go
28.94% <0%> (+3.94%)
:arrow_up:

Continue to review full report at Codecov.

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

This fixes the link to the repo but leaves the comment links broken:

That is a separate issue not related to this PR

And it does work in Slack:

@cornelk Please submit separate issue for mattermost links


Only show close button on last page

Not an issue really, just a desired scenario. Can I only show the close button on the last walkthrough page? Thanks so much for awesome repo!

Sure, you can link the closeButton property to your viewController, then, you need to setup a BWWalkthroughViewControllerDelegate and you can use its walkthroughPageDidChange(pageNumber:) function to hide the closeButton if the received page is the last one (just doing something like pageNumber == walkthrough.numberOfPages -1).


Add .net50

Try to solve issue https://github.com/gmarokov/dotnet-trx2sonar/issues/1

Hi @nesc58 thanks for the PR, the pipeline failed with missing target framework. I think instead of net50 should be net5.0. Can you look at this? Cheers.

@gmarokov  I have changed the framework to net5.0. Locally this works as well as the commits before.
I have changed the pipeline too because I think that the problem is there.
The pipeline before executes the commands side-by-side. But we need to install both versiosn: 3.1.x and 5.0.x

When this works I would solve #3 next


Bug: Sidemenu Content Scrollable When Side Menu Exposed

Type: bug
Platform: mobile webview
When you have native scrolling enabled on 1.0.0, and you scroll a list then drag to vie the sidemenu list you can still scroll the small ion-view and interact with its clickable stuff. It should normally be non intractable.

Can you provide a codepen for this @saniyusuf

Greetings @saniyusuf!
My sensors indicate a reply was requested, but we have not received one yet, I am using my robot powers to close this issue. However, if you are still experiencing this issue, please feel free to reopen it by creating a new issue, and include any examples or necessary information.
Thank you for allowing me to assist you.

Greetings @saniyusuf!
My sensors indicate a reply was requested, but we have not received one yet, I am using my robot powers to close this issue. However, if you are still experiencing this issue, please feel free to reopen it by creating a new issue, and include any examples or necessary information.
Thank you for allowing me to assist you.


Workspace trust should move closer to config service init

I feel this code should move into the createWorkspaceService:
https://github.com/microsoft/vscode/blob/6779f85321456851940ea2b79233089d78596f1f/src/vs/workbench/electron-sandbox/shared.desktop.main.ts#L269-L271
https://github.com/microsoft/vscode/blob/6779f85321456851940ea2b79233089d78596f1f/src/vs/workbench/browser/web.main.ts#L210-L212

WorkspaceTrustManagementService has a dependency on StorageService and StorageService is not available when createWorkspaceService is called - So I am not sure how we can move to createWorkspaceService

I see. I think my confusion was that WorkspaceTrustManagementService seems to need both storage and config service to work. I would have probably expected that the configuration service gets the WorkspaceTrustManagementService passed in to handle the updateWorkspaceTrust from within instead of doing this from the outside.

The WorkspaceTrustManagementService needs the configuration service as it needs to check whether the feature is enabled or not by reading a user setting. Do we have a patten for reading the value of a setting without taking a dependency on the configuration service?

No I don't think so. I don't have strong feelings so feel free to close.

Ok. Will close it for now, as in the future WorkspaceTrustManagementService will not need the configuration service any more.


Add meta to SemanticModels

resolves #8511
Problem
No meta attribute on SemanticModels
Solution
Add meta under configs
Checklist

[ ] I have read the contributing guide and understand what's expected of me
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX
[ ] This PR includes type annotations for new and modified functions

Opened a new issue in dbt-labs/docs.getdbt.com: https://github.com/dbt-labs/docs.getdbt.com/issues/4180


MWPW-159427: Milo Catalog: Ipad specific issues

On iOS version 15 and lower there is a different way of managing the vertical stacking context, which puts the modal below the merch card. Adding z-index on the merch-sidenav resolves this problem.
Related CC PR: https://github.com/adobecom/cc/pull/422
Resolves: MWPW-159427
Test URLs:

Before: https://main--cc-–adobecom.hlx.live/products/catalog?martech=off
After: https://mwpw-159427-ipad-issues --cc-–adobecom.hlx.live/products/catalog?martech=off (added a space to hide the link from PSI check)

For PSI check:
https://mwpw-159427-ipad-issues--milo--mirafedas.hlx.live/?martech=off&georouting=off

@mirafedas - can we get this tested? thanks!


Cleaning up a little bit

cc @yeti-or

Coverage remained the same at 100.0% when pulling c9e1e06ac3e2e26e0079e3b6cfe24448b7bd15d8 on zxqfox.chores into fcf2c7db771ff4d992c6b45ea7f2c0c6ce7354e6 on master.

Coverage remained the same at 100.0% when pulling c798fe09a7a0bf727cb3decc78d476c145f8b4c6 on zxqfox.chores into fcf2c7db771ff4d992c6b45ea7f2c0c6ce7354e6 on master.


add row length and string_view of cell

(char, not cells) row length is useful to skip empty lines (often the last row!)
adding a string_view on cell contents to speed up processing

Oops. Sorry. Yes it does. Sorry @reder2000. Best to recreate the PR, perhaps change the API to be C++11 compatible (e.g., const char* return that can be converted to string_view with C++17 support).

No worries and thanks for the super-quick revert (... and sorry @reder2000 ! 🙈 ).

No prob- i was unsure about it.
I'll change that (check C++ standard).
const char* is not that interesting, consumer has to find string length whereas we already have it computed (hence the string_view)


Create endpoints for Pins

Useful endpoints could include:

Get a pin by ID (include all fields)
Get all pins associated with an Investigation (include minimal fields like ID + location data)

I put in #61 to add the second one, but we already had the first. For now, I don't think we need two separate serializers for Pins like we have for Investigations. There aren't enough fields on a Pin to warrant a small serializer and a full one.

Should the second endpoint replace the general GET all pins endpoint? It may be reasonable to remove the "all pins" endpoint entirely, since pins would only ever be needed in the context of their investigation.

Should the second endpoint replace the general GET all pins endpoint? It may be reasonable to remove the "all pins" endpoint entirely, since pins would only ever be needed in the context of their investigation.

I agree that it would be reasonable to remove the GET all endpoint but I don't think it should hold up #61


Update start page for top up

Removed example - this was confusing users. Have added some information about how it's calculated.
Trello: https://trello.com/c/twwy5H0W/383-state-pension-top-up-calculator-and-information

This smart answer has been retired, so closing this Pull Request.


YARN-7243. Moving logging APIs over to slf4j in hadoop-yarn-server-resourcemanager.

Backport https://issues.apache.org/jira/browse/YARN-7243 to branch-3.2 to fix compile failure caused by https://issues.apache.org/jira/browse/YARN-9873

All the conflicts are import sections.
Ran {{mvn install -DskipTests}} successfully.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
531
Docker mode activated.

_ Prechecks _

+1
dupname
6
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 74 new or modified test files.

_ branch-3.2 Compile Tests _

0
mvndep
17
Maven dependency ordering for branch

-1
mvninstall
399
root in branch-3.2 failed.

-1
compile
367
root in branch-3.2 failed.

-0
checkstyle
207
The patch fails to run checkstyle in root

-1
mvnsite
29
hadoop-yarn-server-resourcemanager in branch-3.2 failed.

-1
shadedclient
550
branch has errors when building and testing our client artifacts.

+1
javadoc
83
branch-3.2 passed

0
spotbugs
103
Used deprecated FindBugs config; considering switching to SpotBugs.

-1
findbugs
27
hadoop-yarn-server-resourcemanager in branch-3.2 failed.

_ Patch Compile Tests _

0
mvndep
18
Maven dependency ordering for patch

+1
mvninstall
91
the patch passed

+1
compile
906
the patch passed

-1
javac
906
root generated 517 new + 804 unchanged - 8 fixed = 1321 total (was 812)

-0
checkstyle
219
The patch fails to run checkstyle in root

+1
mvnsite
134
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
707
patch has no errors when building and testing our client artifacts.

+1
javadoc
114
the patch passed

+1
findbugs
227
the patch passed

_ Other Tests _

+1
unit
536
hadoop-common in the patch passed.

-1
unit
4292
hadoop-yarn-server-resourcemanager in the patch failed.

+1
asflicense
49
The patch does not generate ASF License warnings.

9655

Reason
Tests

Failed junit tests
hadoop.yarn.server.resourcemanager.metrics.TestSystemMetricsPublisherForV2

hadoop.yarn.server.resourcemanager.metrics.TestCombinedSystemMetricsPublisher

Subsystem
Report/Notes

Docker
Client=19.03.3 Server=19.03.3 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1634/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1634

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux c9a61fad2458 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
branch-3.2 / ae8ca70

Default Java
1.8.0_222

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-1634/1/artifact/out/branch-mvninstall-root.txt

compile
https://builds.apache.org/job/hadoop-multibranch/job/PR-1634/1/artifact/out/branch-compile-root.txt

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-1634/1/artifact/out//home/jenkins/jenkins-slave/workspace/hadoop-multibranch_PR-1634/out/maven-branch-checkstyle-root.txt

mvnsite
https://builds.apache.org/job/hadoop-multibranch/job/PR-1634/1/artifact/out/branch-mvnsite-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt

findbugs
https://builds.apache.org/job/hadoop-multibranch/job/PR-1634/1/artifact/out/branch-findbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt

javac
https://builds.apache.org/job/hadoop-multibranch/job/PR-1634/1/artifact/out/diff-compile-javac-root.txt

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-1634/1/artifact/out//home/jenkins/jenkins-slave/workspace/hadoop-multibranch_PR-1634/out/maven-patch-checkstyle-root.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1634/1/artifact/out/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1634/1/testReport/

Max. process+thread count
1532 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: .

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1634/1/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

The test failures are not related to the patch. Merging this into branch-3.2.


added jothiprasath.com

This PR is:

[x] Adding a new domain
[ ] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[x] I used the uncompressed size of the site
[x] I have included a link to the GTMetrix report
[x] The domain is in the correct alphabetical order
[x] This site is not a ultra lightweight site
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[x] Check to confirm

- domain: jothiprasath.com
  url: https://jothiprasath.com
  size: 376
  last_check: 2022-07-09

GTMetrix Report: https://gtmetrix.com/reports/jothiprasath.com/ZJf1l1fJ/

Thanks for adding your site!


Entities not Updating

Communication with the weather station (WS-5000) appears to be working properly but the entities are all showing Unavailable. I've checked the logs and there are no errors pertaining to AWNET to HASS.

Transferring issue here since the work to fix is in the add-on.


feat(range): adding the resolution property for the range

Enhancement
Add of a resolution property in the ion-range component in order to select the number of decimals needed by the developper.
Ionic Version: 2.x
Issue: #6812

Any news about this ?

Hello and thank you for contributing to Ionic! We have been working on porting all of the Ionic components to web components and have recently updated master to reflect this. This significant change has caused this pull request to break. While we really appreciate the time and effort you put into creating this, we are not able to merge it because of the newly introduced conflicts. We are extremely sorry about this. We will not be merging any more features in to v3. If this is a feature and you have the time, please resubmit this PR against the master branch. If this is a critical security issue in v3, we would greatly appreciate it if you would resubmit the PR against the new v3 branch. Thanks so much for your time!


fix attachment_tag no longer having cdn host

https://github.com/refile/refile/commit/acb99c2f42efa596b9f2d9a3a5f124b5c2df049a
introduced a bug where attachment_tag would previously pass down nil to
attachment_url, but now this is no longer overwritten as Refile.host
also included a failing test :)
@jnicklas

I think the test is a strange mix of integrated (using a real ActiveRecord object) and mocked (mocking the Rails parts). Personally I'd prefer a more integrated tests. We had some mock-heavy unit tests for the helper before and ended up removing them because they weren't helpful.

all integration now, better ?
On Sun, May 10, 2015 at 2:01 AM, Jonas Nicklas<EMAIL_ADDRESS>wrote:

I think the test is a strange mix of integrated (using a real ActiveRecord
object) and mocked (mocking the Rails parts). Personally I'd prefer a more
integrated tests. We had some mock-heavy unit tests for the helper before
and ended up removing them because they weren't helpful.
—
Reply to this email directly or view it on GitHub
https://github.com/refile/refile/pull/220#issuecomment-100605803.

Yes, nice, thanks!


ORecordDuplicatedException: 2 Node Distributed

I have a 2 node orient distributed setup with the following default distributed settings:
{
    "autoDeploy": true,
    "hotAlignment": false,
    "executionMode": "asynchronous",
    "readQuorum": 1,
    "writeQuorum": 2,
    "failureAvailableNodesLessQuorum": false,
    "readYourWrites": true,
    "clusters": {
        "internal": {
        },
        "index": {
        },
        "*": {
            "servers" : [ "<NEW_NODE>" ]
        }
    }
}

I'm using the tinkerpop API to create the database on node1. After it's created, I'm inserting TONS of records very quickly from multiple threads. No errors are returned from the API, but on the second node a bunch of com.orientechnologies.orient.core.storage.ORecordDuplicatedException: Cannot index record #16:379: found duplicated key '<redacted>' in index 'Address.email' previously assigned to the record #16:65 RID=#16:65  are being logged. Also, no data is being committed to node1...
Is it possible to do mass parallel insertion into a distributed environment like this?
For reference, here is a sample of how I'm inserting items:
for (int i = 0; i < MAX_TX_RETRIES; i++) {
    try {
        g.begin();
        Vertex thread;
        thread = g.getVertexByKey("Thread.messageId", threadMsgId);
        if (thread == null) {
            thread = g.addVertex("class:Thread");
            thread.setProperty("messageId", threadMsgId);
        }
        g.addEdge(null, msg, thread, "threaded_by");
        g.commit();
        break;
    } catch (ONeedRetryException | ORecordDuplicatedException | ODatabaseException ex) {
        g.rollback();
        if (i == MAX_TX_RETRIES - 1) {
            System.err.println("Hit MAX_TX_RETRIES");
            throw ex;
        }
    }
}

Which release? Please try with a recent version, namely 2.0.15, 2.1.2 or 2.2.0-SNAPSHOT.

Hi @lvca. I gave up on working with it a while back. This was found using version 2.0.3. If I have a chance, I'll retry using something later, but I doubt I'll get around to it. Thanks for the response!

The issue has been closed because no response has been received after a while. In case the issue is still valid, please reopen it or write a comment that explains why it's still valid and a way to reproduce it. Thank you.


Address spdm_hmac_* and spdm_hash_* return values

Addresses #75.
Addresses most remaining functions - these functions use allocated contexts heavily, so have to be more careful when checking for and handling error, so that we do not unintentionally disrupt state.
Note: I did NOT address the libspdm_append_message_k and libspdm_append_message_f buffers yet. These functions are not written in a way to handle errors gracefully. If we fail at any point during these functions, the message hashes and buffers can enter an unrecoverable state. The changes required to gracefully handle this error would need close care when reviewing, and so I will defer it to a later PR. This probably deserves its own issue.

it appears I clobbered this in a misguided attempt to merge latest. Will reattempt and push new PR.


Feature request - Focus on project

I really like the ability to load/unload projects ("NX load/unload apps and libs), and the way it also loads/unloads dependencies of the project.
I thought that it'd be nice to take it one step forward - An easy way to "focus" on a single project. It would work like so:

User would choose a project to focus on from right-click context menu (or from a filterable popup)
All unrelated projects/libraries would unload. All related ones would be loaded.
You'll have an ability to see/run the tasks only of the focused project

How does that sound?

I will see what I can do.

I did some work. This is how it will looks.
https://mobile.twitter.com/iguissouma/status/1422962363515510784

Awesome! Looks like exactly what I was talking about, thanks!

Just something to consider - Besides an ability to do it from the context menu, it'd be great if I can also open it from a keyboard-friendly action dropdown. Something like "Recent Files"/"Recent Project"


2.1?

Please make sure that this is a feature request. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:feature_template
System information

TensorFlow version (you are using):
Are you willing to contribute it (Yes/No):

Describe the feature and the current behavior/state.
Will this change the current api? How?
Who will benefit with this feature?
Any Other info.

tf2.1 pb

Hi @Jie-Yuan , can you please be more explicit on your demand? Thank you

The README does not provide any information on what version of TF does this java api support. Is it for TF2.0, TF2.1, or earlier versions?

Currently the API targets TF 2.2 rc2.

I've added this section to the README.md


Launching Firefox doesn't work with FF 3.6.13, SeLib 2.5

Originally submitted to Google Code by spielman... on 20 Jan 2011

I'm trying to run robot tests using Selenium on my local machine. I use Firefox 3.6.13 and Selenium Library 2.5. As soon as the Selenium server says "Launching Firefox...", it does start Firefox with two tabs, but that's all. The first tab is empty, the second tab points to
chrome://src/content/RemoteRunner.html?sessionId=2ebd0bb91949489faf15d66ffa316640&multiWindow=true&baseUrl=http%3A%2F%2Fsomehost%2Fxy%2Fz%2F&debugMode=true&driverUrl=http://localhost:4455/selenium-server/driver/
but nothing happens. Firefox sits there, on a seemingly half-initialized Selenium session, and doesn't do anything else. I've waited for quite some time if anything would happen, but it didn't.
Can anyone confirm this?

Originally submitted to Google Code by spielman... on 20 Jan 2011

This happens on Windows 7 Professional, 64 bit. Just in case this matters ;)

Originally submitted to Google Code by @pekkaklarck on 20 Jan 2011

I have exactly same Firefox and SeLib versions in WinXP and they work fine.
Does this happen for you with all tests? Have you, for example, tried running the SeleniumLibrary demo?

Originally submitted to Google Code by spielman... on 24 Jan 2011

Funny situation. When I tried, I was able to run the demo just fine, even though I already had an instance of Firefox running. However, I just read about Selenium Grid, and in the documentation I found this little paragraph:
"Note: I recommend quitting all running instances of the Firefox browser before running the demo. Sometimes Selenium has trouble launching Firefox when an instance is already running."
Thus, I will do another test tomorrow with no instances of Firefox running when I start the test, and then see if it works.

Originally submitted to Google Code by @pekkaklarck on 27 Jan 2011

Any more news about this?

Originally submitted to Google Code by spielman... on 1 Feb 2011

It worked with the demo tests, but it never worked with my project-related tests. With SeLib 2.6, it works fine under all circumstances, so I suppose that there is no value in going after the cause.

Originally submitted to Google Code by @pekkaklarck on 1 Feb 2011

If 2.6 fixed your problem then it was probably caused by CustomFirefoxProfile introduced in 2.5. We noticed Firefox was launched twice when using it ( issue 171 ) and that was fixed. It could be that your problem was another way for this problem to appear or we may have fixed your problem by accident.
If you still want to investigate this, you can start Selenium Server with option -firefoxProfileTemplate DEFAULT when using 2.5. If Firefox works then, this issue should be resolved as a duplicate for 171. Now I just close this.

Originally submitted to Google Code by jollych... on 2 Mar 2011

when I user the selenium Library (382:491d0094aa97)'s firefoxprofile,the RC server can't open the firefox 3.6.13

/usr/bin/java -jar /Library/Python/2.6/site-packages/SeleniumLibrary/lib/selenium-server.jar -port 4444 -firefoxProfileTemplate DEFAULT -userExtensions /Library/Python/2.6/site-packages/SeleniumLibrary/lib/user-extensions.js -firefoxProfileTemplate /Library/Python/2.6/site-packages/SeleniumLibrary/firefoxprofile

after set the -firefoxProfileTemplate DEFAULT,it's works fine.
Maybe this issue should reopen

Originally submitted to Google Code by @pekkaklarck on 2 Mar 2011

I don't know what version 382:491d0094aa97. Have you tried with the latest version?


[Windows][Desktop]: onLongPress gesture events not firing on touch screen device

Steps to Reproduce

Run the below code in windows machine with touchscreen compatibility.

Code snippet
import 'package:flutter/material.dart';

void main() {
  return runApp(App());
}

class App extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData(primarySwatch: Colors.blue),
      home: _MyHomePage(),
    );
  }
}

class _MyHomePage extends StatefulWidget {
  // ignore: prefer_const_constructors_in_immutables
  _MyHomePage({Key? key}) : super(key: key);

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<_MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
        appBar: AppBar(
          title: const Text('Syncfusion Flutter chart'),
        ),
        body: Container(
          child: GestureDetector(
            onLongPress: () {
              print("longPress tapped");
            },
            child: Center(
              child: Container(
                height: 300,
                width: 300,
                color: Colors.blue
              ),
            ),
          ),
        ));
  }
}

Perform long press gesture on the touch screen, the event won't get fired.
We have also attached a screen recording below for reference.

Recording
https://user-images.githubusercontent.com/63577117/122738800-f0695500-d29f-11eb-9c6a-21a599142eca.mp4

Logs

Doctor summary
[√] Flutter (Channel stable, 2.2.1, on Microsoft Windows [Version 10.0.19041.1052], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Professional 2019 16.10.2)
[√] Android Studio (version 4.1.0)
[√] VS Code, 64-bit edition (version 1.57.1)
[√] Connected device (3 available)

• No issues found!

@SriramKiranSenthilkumar
Can you try adding behavior: HitTestBehavior.translucent and see if it helps ?

@darshankawar
I have tried setting the HitTestBehavior value as tranclucent and still the event didn't get fired.

This seems to be specific to touch screen based device, which I don't have to verify and replicate the issue, so I am unable to identify the root-cause and pinpoint the issue. Keeping this open for further investigation from team.
Note: Verified on Windows 10 machine on which the long press using mouse is registered properly.

Since we have no way to reproduce this ourselves, we cannot really debug it.
I would encourage you to step through the code with a debugger, or enable the debugPrintGestureArenaDiagnostics feature, to determine the cause of the issue here, and then file a new issue with the precise problem, and maybe even a PR to fix it. :-)

(Closing per last comment.)


[Feature] Squeezelite binaries version bump

Change url and point to the latest binaries from sourceforge

Available in release 2022-11-26


[Feat] Add a Password Visibility Toggle

Description
Add a password visibility toggle to the login page and also add a margin top to the Sign-in/Login card
@mdazfar2  Assign me this issue under GSSoC'24-Extd and Hacktoberfest'24 to make my contribution count.
Solve:

Add Password Visibility Toggle
Add Margin Top to the Sign-in/Login Card

The Screenshot is provided below
Screenshots

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

I have starred your repo and join your discord
Hope you will assign me to this issue as mentioned above under label GSSoC'24-Extd and Hacktoberfest'24

Hey @Priyesh017 great issue found! You are going to be assigned.

Hope you read the CONTRIBUTING Guidelines.
And also follow us on Linkedin

@mdazfar2 The issue is fixed, could you help me contribute it as it's my first time contributing ?

Hey @Priyesh017, no worries! It is very easy to contribute to open source. I will provide you with the guidelines below:

Firstly you have to fork our repo

Then create issue which you already raised.

clone your fork repository and setup in your local machine and open terminal by which file you want to keep as see the below command.

 git clone https://github.com/priyesh017/ezyshop.git

then all the Ezyshop project code will come in your file

then changes from your vs code and complete the changes, open terminal and do the below commands

 git status

 git add .

 git commit -m "write your command here"

 git push origin main

that's all good now open forked GitHub of Ezyshop

Click on the contribute button

click on Create new pull request

then write description and titile and scroll down and then final click again create pull request

Congrats! 🥳 You've made your first pull request to this project repo.

Hope you understand if you stuck any where then you can react out to me on Discord or Linkedin.

Hey @Priyesh017, no worries! It is very easy to contribute to open source. I will provide you with the guidelines below:

Firstly, you have to fork our repo.

Then create an issue if you haven't already raised one.

Clone your forked repository, set it up on your local machine, and open the terminal in the directory where you want to work. See the command below:
git clone https://github.com/priyesh017/ezyshop.git

The Ezyshop project code will be cloned to your local machine.

Make changes in VS Code, and after completing the changes, open the terminal and run the following commands:
git status

git add .

git commit -m "write your commit message here"

git push origin main

That's all good! Now open your forked GitHub repository of Ezyshop.

Click on the Contribute button.

Click on Create new pull request.

Write the description and title, scroll down, and finally click Create pull request.

Congrats! 🥳 You've made your first pull request to this project repo.

Hope you understand! If you get stuck anywhere, feel free to reach out to me on Discord or LinkedIn.

After "git push origin main" it's showing an error:  "! [rejected]        main -> main (fetch first)
error: failed to push some refs to 'https://github.com/Priyesh017/Ezyshop_Copy.git'
hint: Updates were rejected because the remote contains work that you do not
hint: have locally. This is usually caused by another repository pushing to
hint: the same ref. If you want to integrate the remote changes, use
hint: 'git pull' before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details." how to solve it ?

The error is fixed by using "git push --force origin main"

Hey @Priyesh017 great issue found! You are going to be assigned.

Hope you read the CONTRIBUTING Guidelines.
And also follow us on Linkedin

@mdazfar2 Thank you for assigning me to this issue.


[Hacktoberfest]: Add "JSNation" to the Conferences Section

🆕🐥☝ Beginners Only:
This issue is reserved for people who are new to Open Source. We know that the process of creating a pull request is the biggest barrier for new contributors. This issue is for you 💝
To do:

Add JSNation to the Conferences Section
See below for link to step-by-step tutorial
Go down to Conferences section - it starts at around line 173
Using the following template, put in the correct info for that conference (Search for it on the internet to find the correct details) underneath the last row in the table, ensuring there is still an empty row before the next table.

| [Conference Name](Website Address) | Conference Name or Organising Org | 10 | Description of the Conferences, found on their website |

Add your details in the contributors section at the bottom of the page using the same formatting as the last one

📋 Step by Step
Step-By-Step Tutorial
Remember to observe the Code of Conduct - by which we mean, in short - Be Kind, Be Positive, Be Helpful

Could I work on this issue?


MessageChatMemoryAdvisor adds empty user message

Bug description
When calling
ChatClient.builder(chatModel)
        .defaultAdvisors(new MessageChatMemoryAdvisor(chatMemory))
        .build()
        .prompt()
        .call()
        .chatResponse()

MessageChatMemoryAdvisor adds an empty UserMessage in the chat memory (see line 94).
In our use case we send the user message in a POST, store it in the conversation, then send a GET to benefit from SseEmitter. Hence the empty call to prompt().
Environment
1.0.0-M3
Expected behavior
When using an empty prompt MessageChatMemoryAdvisor should probably not add an empty UserMessage in the chat memory.

Fix PR https://github.com/spring-projects/spring-ai/pull/1708

@fmunch thanks for reporting this. Could you elaborate a bit more on this use case? I'm asking because I see the empty prompt and, in particular, the absence of a user message in ChatClient as an incorrect state (unless it's part of a function calling special flow). Indeed, the current snapshot version of Spring AI throws an exception if there's no user message (whereas in M3 an empty message was created, as you noticed) after adding null-safety to the ChatClient and Advisor APIs. Supporting a null/empty user message would make the behaviour of ChatClient unpredictable and hide pesky errors from developers.
If I understood correctly, your application is adding user messages to the ChatMemory object explicitly, outside the ChatClient workflow.  The MessageChatMemoryAdvisor is designed to handle the entire lifecycle of a chat memory for each conversation using an internal ChatMemory object. It might not be the best candidate for handling only parts of the lifecycle.
Since you have already a ChatMemory object where you're storing a user message explicitly, you could pass directly the messages from the memory via the .prompt() clause. And then save the response back into the memory.
Something like this:
var chatResponse = ChatClient.builder(chatModel).build()
                .prompt(new Prompt(chatMemory.get(conversationId, lastN)))
                .call()
                .chatResponse();
chatMemory.add(chatResponse.getResult().getOutput());

You can also use the .messages()  clause, if you prefer.
var chatResponse = ChatClient.builder(chatModel).build()
                .prompt()
                .messages(chatMemory.get(conversationId, lastN))
                .call()
                .chatResponse();
chatMemory.add(chatResponse.getResult().getOutput());

Full, standalone example:
@Bean
CommandLineRunner chat(ChatClient.Builder chatClientBuilder) {
    return _ -> {
        var chatClient = chatClientBuilder.build();
        var chatMemory = new InMemoryChatMemory();
        var lastN = 10;

        var conversationId = "007";

        var userMessage1 = new UserMessage("My name is Bond. James Bond.");
        chatMemory.add(conversationId, userMessage1);

        var assistantMessage1 = chatClient
                .prompt(new Prompt(chatMemory.get(conversationId, lastN)))
                .call()
                .chatResponse().getResult().getOutput();
        chatMemory.add(conversationId, assistantMessage1);

        System.out.println(assistantMessage1.toString());

        var userMessage2 = new UserMessage("What's my name?");
        chatMemory.add(conversationId, userMessage2);

        var assistantMessage2 = chatClient
                .prompt(new Prompt(chatMemory.get(conversationId, lastN)))
                .call()
                .chatResponse().getResult().getOutput();
        chatMemory.add(conversationId, assistantMessage2);

        System.out.println(assistantMessage2.toString());
    };
}

@ThomasVitale Thank you for taking the time to reply to this.
To summarize, our Spring AI application has a REST API to add messages in a conversation (via POST calls) and to submit the conversation and stream the text/event-stream response (which requires a GET call when using EventSource AFAIK). The POST requests update the chat memory and the text/event-stream request does not send anything, that's why the prompt is empty, everything is already stored in the chat memory.
Your suggestion of using .messages(chatMemory.get(conversationId, lastN)) would work great for us. I initially decided to use the advisor because it was convenient and to benefit from the rest of it (handling the chatMemoryRetrieveSize for example and what might come in the future).


feat: :sparkles: Change the Graph creation to allow subclassing

Currently it is not possible to subclass a Flow as in the example snippet below.
This PR alters the graph construction that enables this flow subclassing. Instead of using the ast.NodeVisitor class to collect the steps and their ast.FunctionDef's.  I have instead iterated on the dir(flow) and then extracted the function ast.FunctionDef's.
from metaflow import FlowSpec, step

class BaseFlow(FlowSpec):
    @step
    def start(self):
        print("this is the start")
        self.next(self.step1)

    @step
    def step1(self):
        print("base step 1")
        self.next(self.end)

    @step
    def end(self):
        print("base step end.")

class SubFlow(BaseFlow):
    @step
    def step1(self):
        print("sub step 1")
        self.next(self.step2)

    @step
    def step2(self):
        print("sub step 2")
        self.next(self.end)

if __name__ == "__main__":
    SubFlow()

As referenced in this issue: https://github.com/Netflix/metaflow/issues/144
and discussed in this issue: https://github.com/Netflix/metaflow/issues/245
I have not yet added tests for this PR as the approach to this might take some discussion. The current Test harness setup uses the FlowFormatter class to generate flows on the fly but all of these assume the flow directly inherits off the FlowSpec. How might we go about testing this kind of subclassing?
Note this PR was originally raised as https://github.com/Netflix/metaflow/pull/2035 but from the wrong branch

Alright, I finally got around to this and there is an issue that is minor but nonetheless:

This line: https://github.com/Netflix/metaflow/blob/master/metaflow/cli.py#L173 breaks

The reason for this (I think) is that:

line number is always the same due to the new processing
so that causes the DAGNode to be compared (which does not work)
Prior to this, the line number was properly computed and always not equal so we would never compare DAGNodes.

I suspect that this would make line-number incorrect so we should probably store line and file or something. This is used in the graph object and we actually rely on it as well for stuff like debug integration (see here: https://github.com/Netflix/metaflow-nflx-extensions/blob/1c534c567beb90de584db7cc2e2b624f08d8498d/metaflow_extensions/netflix_ext/cmd/debug/debug_script_generator.py#L395)
So we somehow need to get equivalent information (probably file and line number) and that should be ok.

Hi @romain-intel,
Previously it was the case that all steps were defined in the same module and the NodeVisitor was used to get the ast.FunctionDef for each "step". In this implementation the lineno for the step and for the tail of the step are stored.
In the implementation I used the inspect module to get the source code for each function in the dir(flow) and then call ast.parse on each function source to get the func_ast: ast.FunctionDef. However as you point out the func_ast.lineno in this case is relative only to the function and not the file. The image below shows line numbering lineno for the file and the func_ast.linenofor the step.
e.g.
from metaflow import FlowSpec, step # line 1
# line 2
...

# line 20
class BaseFlow(FlowSpec): # line 21
    @step # line 22 - line 1 in func_ast
    def start(self): # line 23 - line 2 in func_ast
        print("this is the start") # line 24 - line 3 in func_ast
        self.next(self.step1) # line 25 - line 4 in func_ast

To resolve the issue I updated the constructor of the DAGNode to take the source_file and the lineno of the function (in the example above this is line 22). There is one slight complication is that the line number we want is line 23 (the line with the def on it not the line where the decorators for the function start). To get this we can simply use lineno + func_ast.lineno - 1 which in words is the line number of the the start of function (including decorators) in the file + the number of lines between this and the def.
However by allowing subclassing of flows one step may be in a different file from another and hence we could still get a func_lineno clash which in turn would through the error you mentioned in https://github.com/Netflix/metaflow/blob/master/metaflow/cli.py#L173.
To resolve this I changed to iterated on the sorted nodes based on traversing the Dag.
I also notice in the codebase we use the linenumber in a lot of LintWarn exceptions e.g.
if re.search("[^a-z0-9_]", node.name) or node.name[0] == "_":
            raise LintWarn(msg.format(node), node.func_lineno)

We might want to consider extending these warnings to log the source_file as well as the lineno to give more context. I am happy to add that in this PR but wanted to discuss with you first.

OK, I like where this is going. Yes, I think you are right that we should indicate the file and line number. I think we also need to add the source file in the node_to_dict function so that we can pick it up in the debug integration. Looping @talsperre to check on this.

Hi @romain-intel, I am going on holiday for a couple of weeks now but I made a few more changes based on  what you suggested:

Added source_file to the base MetaflowException class
Updated the lint warnings to include the node.source_file
Updated the cli method to display the filename in the warning as in the image below
Added the source_file to the node_to_dict method

Let me know what you think 😄

Hey @romain-intel , Just wondered if you had any time to get to this?

Alright. Why are the tests not running here. I'll run them internally and check.

OK, the changes look good to me, I just need to probably make a few changes here: https://github.com/Netflix/metaflow-nflx-extensions/blob/main/metaflow_extensions/netflix_ext/cmd/debug/debug_script_generator.py#L409 (and probably a few other places) so that it doesn't break when this change merges. I should be able to test this out and make the requisite changes. @talsperre for viz as well.

OK, the changes look good to me, I just need to probably make a few changes here: https://github.com/Netflix/metaflow-nflx-extensions/blob/main/metaflow_extensions/netflix_ext/cmd/debug/debug_script_generator.py#L409 (and probably a few other places) so that it doesn't break when this change merges. I should be able to test this out and make the requisite changes. @talsperre for viz as well.

Great - thanks @romain-intel. Let me know if there are any outstanding tasks I can help with.

From the previous comment i can see that 12 failures happened somewhere. Is there anyway I can see these and fix them?

Coincidentally, this is now going to be needed for configs too. I've made a tiny change to make it compatible and will merge as soon as the tests pass (or at least nothing unexpected happens). Yes, finally!


How to Use this for x => (x == "test")?

This looks very promising for what I'm doing, but I can't get it to work for this use case:
x => (x == "test")
I also need to handle ?. syntax if at all possible and ternary operators.
Will these work?

Right now, all C# syntax for expression is supported, except of params.
Do you want to convert expression to Func<string, bool>? Check the example below.
using System;
using TagBites.Expressions;

var func1 = ParseFunc<string, bool>("x == \"test\"");
Console.WriteLine(func1("test"));
Console.WriteLine(func1("some"));

var func2 = ParseFunc<string, int>("x == \"test\" ? 100 : x.Length");
Console.WriteLine(func2("test"));
Console.WriteLine(func2("some"));

private static Func<T, TReturn> ParseFunc<T, TReturn>(string expression)
{
    var options = new ExpressionParserOptions
    {
        Parameters =
        {
            (typeof(T), "x")
        },
        ResultType = typeof(TReturn)
    };
    var lambda = ExpressionParser.Parse(expression, options);
    return (Func<T, TReturn>)lambda.Compile();
}

Output:
True
False
100
4

Test on dotnetfiddle.net
If you want to parse an expression that returns a function like Func<Func<string, bool>>, then it is not supported. I think it is unnecessary wrapper.


Fix check style problem with class BasePageRequest

The class org.apache.shardingsphere.elasticjob.lite.console.dto.request.BasePageRequest is break rule of check style.
There are use // CHECKSTYLE:OFF to avoid checkstyle validate, but protected field modifiers do not allow.
Please remove // CHECKSTYLE:OFF and provide protected getter and setter method to instead of protect fields.

I can make it, please assign to me. @terrymanu

already fixed


Update the public bucket example URL

Change object.pouta to a3s.fi
Add https prefix
I don't understand what this statement means so I removed it for clarity:
"it can be shared only to the wanted audience."

The new text mentions the project identified in UUID format. That's new to me. Would a link to wikipedia or some part of our own documentation help here?

The new text mentions the project identified in UUID format. That's new to me. Would a link to wikipedia or some part of our own documentation help here?

Fair point! Added a link, this is currently documented under s3cmd. Project UUID is an OpenStack Keystone term and in a way would deserve a more generic page. Something for another day perhaps.


Update .sol templates

Issue To Be Solved
Contract fields on .sol templates are inconsistently name - flow* instead of cadence* - and the ERC20 should implement ERC165.

Closed by #88


The pixel information obtained by TIOCGWINSZ is incorrect.

Windows Terminal version
1.22.2912.0
Windows build number
10.0.22631.0
Other Software
My application.
termsize.c:
#include <fcntl.h>
#include <termios.h>
#include <stdio.h>
#include <unistd.h>
#include <sys/ioctl.h>

void main() {
  struct winsize ws;
  int fd = open("/dev/tty", O_RDONLY);
  if(fd == -1) {
    fprintf(stderr, "Can't open tty");
    return;
  }
  int ret = ioctl(fd, TIOCGWINSZ, &ws);
  close(fd);
  fprintf(stderr, "cols %d rows %d, xpixel %d, ypixel %d.\n", ws.ws_col, ws.ws_row, ws.ws_xpixel, ws.ws_ypixel);
}

Steps to reproduce

Run gcc termsize.c -o termsize && ./termsize
Output stderr cols 163 rows 60, xpixel 0, ypixel 0. † xpixel and ypixel is fixed value zero.

(I was recommended to ask here in the comments at https://github.com/microsoft/WSL/issues/12265, so I am asking here as well.)
Expected Behavior
Output xpixel and ypixel is correct value.
Actual Behavior
Output xpixel and ypixel is fixed value zero.

Thanks for filing! We're going to mark this as a /dup of #8581


bazel: building java targets fails because bazel downloads ijar binary

Issue description
Bazel fetches a binary at runtime and tries to execute it, which of course fails on NixOS:
ERROR: /home/philip/.cache/bazel/_bazel_philip/cccb2d2da8416067607b8c5cff088df0/external/org_apache_spark_spark_core_2_10/BUILD:7:1: Extracting interface @org_apache_spark_spark_core_2_10//:org_apache_spark_spark_core_2_10 failed (Exit 1) ijar failed: error executing command external/remote_java_tools_linux/java_tools/ijar/ijar external/org_apache_spark_spark_core_2_10/spark-core_2.10-1.6.0.jar ... (remaining 3 argument(s) skipped)

Use --sandbox_debug to see verbose messages from the sandbox
src/main/tools/linux-sandbox-pid1.cc:427: "execvp(external/remote_java_tools_linux/java_tools/ijar/ijar, 0x62e860)": No such file or directory

Related to https://github.com/bazelbuild/bazel/issues/8575
Steps to reproduce
Build any Java rule with bazel 0.26.1 in nixpkgs on NixOS.
cc @groodt @kalbasit

I'm having the same issue. They have a new target for the java_runtime, called @bazel_tools//tools/jdk:absolute_javabase: https://github.com/bazelbuild/bazel/blob/cfc6800c64bc97f433e602d0e9d0753b3036502e/tools/jdk/BUILD#L67-L75. I hardcoded the ABSOLUTE_PATH, but it did not work.
diff --git a/src/.bazelrc b/src/.bazelrc
index d67eb3c81..5a2bcc9ce 100644
--- a/src/.bazelrc
+++ b/src/.bazelrc
@@ -48,20 +48,11 @@ test --features=race
 # https://docs.bazel.build/versions/master/be/java.html#java_toolchain
 # https://github.com/bazelbuild/bazel/blob/17794d58cc493d918a98c3250302ca44b1d53e3f/tools/jdk/default_java_toolchain.bzl#L84-L93

-build --host_java_toolchain='@bazel_tools//tools/jdk:toolchain_hostjdk8'
-build --host_javabase='@local_jdk//:jdk'
-build --java_toolchain='@bazel_tools//tools/jdk:toolchain_hostjdk8'
-build --javabase='@local_jdk//:jdk'
-build --proto_toolchain_for_java='@bazel_tools//tools/jdk:toolchain_hostjdk8'
+build --host_javabase='@bazel_tools//tools/jdk:absolute_javabase'
+build --define=ABSOLUTE_JAVABASE=/nix/store/543yiclmkvmpy8b9fg00y0bw4xwh3865-openjdk-8u212-ga/lib/openjdk

I suspect it's coming from here which is called from here. The absolute_javabase java_runtime defines the java_home but not the ijar target.

@Profpatsch have you made any progress on this yesterday?

tbh I was hoping someone else would fix it. :P
I don’t have much time allocated to bazel work for next week, so it might be ~10 days until I can take an earnest stab.

I was honestly expecting the following patch to work, but :man_shrugging:! I'm not really sure why it's not working.
diff --git a/tools/jdk/BUILD b/tools/jdk/BUILD
index d99fb7adb9..b582f7ba67 100644
--- a/tools/jdk/BUILD
+++ b/tools/jdk/BUILD
@@ -327,24 +327,12 @@ default_java_toolchain(

 alias(
     name = "toolchain",
-    actual = select({
-        "//src/conditions:darwin": "@remote_java_tools_darwin//:toolchain",
-        "//src/conditions:darwin_x86_64": "@remote_java_tools_darwin//:toolchain",
-        "//src/conditions:windows": "@remote_java_tools_windows//:toolchain",
-        "//src/conditions:linux_x86_64": "@remote_java_tools_linux//:toolchain",
-        "//conditions:default": "@bazel_tools//tools/jdk:legacy_toolchain",
-    }),
+    actual = "@bazel_tools//tools/jdk:legacy_toolchain",
 )

 alias(
     name = "remote_toolchain",
-    actual = select({
-        "//src/conditions:darwin": "@remote_java_tools_darwin//:toolchain",
-        "//src/conditions:darwin_x86_64": "@remote_java_tools_darwin//:toolchain",
-        "//src/conditions:windows": "@remote_java_tools_windows//:toolchain",
-        "//src/conditions:linux_x86_64": "@remote_java_tools_linux//:toolchain",
-        "//conditions:default": "@bazel_tools//tools/jdk:legacy_toolchain",
-    }),
+    actual = "@bazel_tools//tools/jdk:legacy_toolchain",
 )

 # The 'vanilla' toolchain is an unsupported alternative to the default.
diff --git a/tools/jdk/BUILD.java_tools b/tools/jdk/BUILD.java_tools
index 7d4ee45498..30d26ebd5c 100644
--- a/tools/jdk/BUILD.java_tools
+++ b/tools/jdk/BUILD.java_tools
@@ -18,7 +18,7 @@ java_toolchain(
     genclass = [":Genclass"],
     header_compiler = [":Turbine"],
     header_compiler_direct = [":TurbineDirect"],
-    ijar = [":ijar"],
+    ijar = [":ijar_cc_binary"],
     javabuilder = [":JavaBuilder"],
     javac = [":javac_jar"],
     javac_supports_workers = 1,
@@ -52,7 +52,7 @@ java_toolchain(
         "-g",
         "-parameters",
     ],
-    singlejar = [":singlejar"],
+    singlejar = [":singlejar_cc_bin"],
     source_version = "8",
     target_version = "8",
     tools = [
diff --git a/tools/jdk/BUILD.java_tools.old b/tools/jdk/BUILD.java_tools.old
index ff438af752..6b6e2aea3f 100644
--- a/tools/jdk/BUILD.java_tools.old
+++ b/tools/jdk/BUILD.java_tools.old
@@ -10,7 +10,7 @@ java_toolchain(
     genclass = [":Genclass"],
     header_compiler = [":turbine"],
     header_compiler_direct = [":turbine_direct"],
-    ijar = [":ijar"],
+    ijar = [":ijar_cc_binary"],
     javabuilder = [":javabuilder"],
     javac = [":javac_jar"],
     tools = [
@@ -52,7 +52,7 @@ java_toolchain(
         # Restrict protos to Java 7 so that they are compatible with Android.
        "proto": ["-source", "7", "-target", "7"],
     },
-    singlejar = [":singlejar"],
+    singlejar = [":singlejar_cc_bin"],
     bootclasspath = ["@bazel_tools//tools/jdk:platformclasspath"],
 )

This issue is typical on bazel with nixos: every "prebuilt" binary fetched by any bazel rule will fail with the same issue.
We can fix each issue manually as they arise, but can we do better in a potential more generic way?
From what I know, the only place bazel may generate binaries from the outerworld are in repository rules, and especially the following rules:

https://docs.bazel.build/versions/master/skylark/lib/repository_ctx.html#download
https://docs.bazel.build/versions/master/skylark/lib/repository_ctx.html#download_and_extract
https://docs.bazel.build/versions/master/skylark/lib/repository_ctx.html#extract

What are your thoughts about applying a recursive patchelf to patch at least the dynamic loader on the result of these rules?
Another option may be to just patch https://docs.bazel.build/versions/master/skylark/lib/repository_ctx.html#execute to rewrite the call to /path/to/know/ld/loader binary_path instead.

This issue is typical with bazel on nixos: every "prebuilt" binary fetched by any bazel rule will fail with the same issue.
We can fix each issue manually as they arise, but can we do better in a potential more generic way?
From what I know, the only place bazel may generate binaries from the outerworld are in repository rules, and especially the following rules:

https://docs.bazel.build/versions/master/skylark/lib/repository_ctx.html#download
https://docs.bazel.build/versions/master/skylark/lib/repository_ctx.html#download_and_extract
https://docs.bazel.build/versions/master/skylark/lib/repository_ctx.html#extract

What are your thoughts about applying a recursive patchelf to patch at least the dynamic loader on the result of these rules?

It would be wonderful if that would work, as we don't have to keep specifying Nix provided toolchains. Making this work well might not be a trivial task, as you would most likely have to replicate autoPatchelfHook.

Another option may be to just patch https://docs.bazel.build/versions/master/skylark/lib/repository_ctx.html#execute to rewrite the call to /path/to/know/ld/loader binary_path instead.

How would this allow it to find, say go or javac?

It would be wonderful if that would work, as we don't have to keep specifying Nix provided toolchains. Making this work well might not be a trivial task, as you would most likely have to replicate autoPatchelfHook.

Yes. That's the most difficult solution.

How would this allow it to find, say go or javac?

I don't understand. The only things I'm thinking about is that, instead of using execute(arguments=foo) we rewrite it as execute(arguments=["path/to/a/known/ld/linker", call_to_which(foo[0])] + foo[1:]).

It would be wonderful if that would work, as we don't have to keep specifying Nix provided toolchains. Making this work well might not be a trivial task, as you would most likely have to replicate autoPatchelfHook.

Yes. That's the most difficult solution.

How would this allow it to find, say go or javac?

I don't understand. The only things I'm thinking about is that, instead of using execute(arguments=foo) we rewrite it as execute(arguments=["path/to/a/known/ld/linker", call_to_which(foo[0])] + foo[1:]).

I see what you mean now. I think the autoPatchelfHook is more elegant. Given the amount of work that might require, it might be prudent to just fix the immediate issue right now. @guibou would you be able to take a stab at it?

@kalbasit I'm on it.

Progress: I've built a local package with fixed ijar and I'm using --override_repository to point to that package. This setting is done inside a system-wide bazelrc file.
Hopefully I'll be able to open a PR today.

That's great news @guibou! Thank you!

That's great news @guibou! Thank you! Please include a Java test (similar to current python test) with your PR.

AFAIK there is a java test in the installCheck phase that I also repaired.


Filter with Wildcard Searches

I'm submitting an Extension Method for SolrExpress.Search.Parameter.Extension

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[x] Feature request
[ ] Documentation issue or request

Current behavior

Using Solr.Express I could not complete the request for Code:*123*, for example, just Code:"*123*", but it's not work for me
Expected behavior

I need smth like this or recommendation how I can make such a request using available tools
public SearchQuery Contains(string value)
{
return this.AddValue($"*{value}*", false);
}
// using
documents
.Select()
.Filter(q => q.Code, q=>q.Contains("123"))
It already works for me locally
Minimal reproduction of the problem with instructions
query - string with Solr query
I used to
documents
.Select()
.Query(query)
but now it does not work like this

Environment

Connect to a SOLR version: 7.2.0
Connect to a SOLR in cloud mode? [ ] Yes [x] No
I'm coding using platform:  Windows
Using .net version: netcoreapp2.0
Using SolrExpress package: SolrExpress.Solr5
Using SolrExpress package version: 5.1.2
Using SolrExpress.DI package: Native .Net Core
Using SolrExpress.DI package version: 5.1.2

Other

With this i'd like to add a 'endswith' query aswell.. There's currently only support for 'query.startswith()'


Nuxt Server prefixed files do not respond with correct status code 405

Environment

Operating System: Linux
Node Version:     v20.6.0
Nuxt Version:     3.7.2
CLI Version:      3.8.3
Nitro Version:    2.6.3
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/nuxt-server-retcode-bug?file=app.vue
Describe the bug
When employing an HTTP method-prefixed file handler, if the prefix does not align with the request method, it is anticipated that the server should respond with a 405 Method Not Allowed status code, as mentioned in the docs. However, it currently responds with a 404 Not Found status code. Furthermore, in the Stackblitz development environment, when attempting to issue a GET request to a route designated for POST requests only, it erroneously renders the HTML page for the / route.
This was previously tracked in #22382.
Additional context
No response
Logs
No response

Would you be able to open an issue upstream in https://github.com/unjs/nitro? 🙏

@danielroe I have opened an issue in unjs/nitro#1727. Also note a previous pull request  unjs/nitro#1504 tried to fix this.

I really appreciate this - thank you ❤


ft:initial commit landing-page

initial commit for angular front end for landing page

added donation form features


print instruction to exec.log

Is there a way for me to print RISC-V instructions along with clock cycle into exec.log?
Thanks in advance.

it does - first number in the exec.log  is the clock cycle, then executed instruction number and hart ID (0), then instruction address and then opcode. Using .dis you can find mnemonics
You can also try incorporate dasm.svi from EL2 DB to print mnemonics in exec.log

it does - first number in the exec.log  is the clock cycle, then executed instruction number and hart ID (0), then instruction address and then opcode. Using .dis you can find mnemonics
You can also try incorporate dasm.svi from EL2 DB to print mnemonics in exec.log

@agrobman  Thank you!
I assume that exec.log from EL2 will contain the mnemonics after running the simulation, does it?

@agrobman  Thank you!
I assume that exec.log from EL2 will contain the mnemonics after running the simulation, does it?


No possible to select base-branch for New branch

The screenshot that shows how to create a branch starting from a work item (git-dev-create-branch.png) shows that it should be possible to select

the repo
the branch in that repo to start from

However, when I create a 'New branch' from a work item, all I can select is

the repo

So I have no possibility to select the starting branch.
It seems to create the branch starting from the branch marked as  'Default' in the repo, which is not always desired.
Anything I'm missing?
Thanks

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

ID: 1d125435-dbfb-c199-65ed-34ad33b1f522
Version Independent ID: 04cae7ee-1e54-a002-bb7b-c93a606e982f
Content: Drive Git development from user story and requirements - Azure Boards
Content Source: docs/boards/backlogs/connect-work-items-to-git-dev-ops.md
Service: azure-devops-boards
GitHub Login: @chcomley
Microsoft Alias: chcomley

@WardLootens -- Ward, thank you for your feedback and question.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@chcomley -- Chrystal, please look into this potential documentation issue.

@WilliamAntonRohm it was previously possible to select a base branch and this functionality seems to have been removed in the last week or two. It is also still possible when navigating through the Branches tab.
Instead of looking at this as a documentation issue, could it instead be looked at to put this functionality back in? This change has broken the general workflow of creating branches off a feature branch while updating a work item/user story.

@WilliamAntonRohm We are experiencing the same issue since this morning, so no longer able to select the correct base branch of a repo.
This broke our workflow we've been using for years of creating work items, and then create a branch which is linked automatically.
As @teddy5 requested: would it be possible to put this functionality back in?

@teddy5, @KoenVanhove -- thank you for your follow-up.  Please add this issue here:

Developer Community

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


Arrow Keys Code don't get registered on using Keyboard event.

Problem
KeyboardEvent::key() and ::code() always returns an empty string. I tried logging of keypress the value by adding a batch_callback to a an input box, everything else logs but the arrow keys. Tried on Chrome and Edge.
Steps To Reproduce
Steps to reproduce the behavior:
Relevant Code:
let onkeypress = ctx.link().batch_callback(|event: KeyboardEvent| {
            if event.key() == "Enter" {
                Some(Msg::Submit)
            } else {
                console_log!(event.code());
                None
            }
});
html! {
    <input type="text" onkeypress={onkeypress.clone()} oninput={ ctx.link().callback(|e: web_sys::InputEvent| {
    let input: web_sys::HtmlInputElement = e.target_unchecked_into();
    Msg::OnInput( input.value() ) })} />
}

Expected behavior
Some value must be logged on pressing the keys.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment:

Yew version:  "0.19.3"
Rust version: "1.61.0"
Target, if relevant: wasm32-unknown-unknown
Build tool, if relevant: trunk
OS, if relevant: MacOS M1
Browser and version, if relevant: Chrome, Edge

Questionnaire

[x] I'm interested in fixing this myself but don't know where to start
[ ] I would like to fix and I have a solution
[ ] I don't have time to fix this right now, but maybe later

The onkeypress event is deprecated. Have you tried using the keydown event or another related one instead?

attaching it to onkeydown works thanks a lot!


Bump to 1.2.0 for release

Codecov Report

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

@@            Coverage Diff            @@
##             master      #36   +/-   ##
=========================================
  Coverage     59.23%   59.23%
  Complexity      138      138
=========================================
  Files            15       15
  Lines           601      601
  Branches         91       91
=========================================
  Hits            356      356
  Misses          223      223
  Partials         22       22

Continue to review full report at Codecov.

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


[WIP]: Add deprecation message for deploy tools

Motivation
Since MMDeploy has been released, the deployment tools scattered in each OpenMMLab codebase will be deprecated in future.
Modification
Add deprecation message in deployment tools
BC-breaking (Optional)
None
Use cases (Optional)
And the message would look like it in Ubuntu18.04 terminal:

@lvhan028 Hi, any comment on this PR?


chore: release  service 0.1.12

:package: Staging a new release
0.1.12 (2023-07-28)
Bug Fixes

service: extract the label part from what is returned by the metadata action (#71) (2243f0f)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/carbynestack/ephemeral/releases/tag/service-v0.1.12 :sunflower: