Appstudio update test-component-pac-mhjb

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-mhjb-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
12 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
1 minute
inspect-image

✅ Succeeded
1 minute
deprecated-base-image-check

✅ Succeeded
36 seconds
clamav-scan

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
12 seconds
label-check

✅ Succeeded
18 seconds
show-sbom

✅ Succeeded
15 seconds
show-summary

Pipelines as Code CI/test-component-pac-mhjb-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
19 seconds
clone-repository

✅ Succeeded
40 seconds
build-container

✅ Succeeded
25 seconds
inspect-image

✅ Succeeded
32 seconds
clamav-scan

✅ Succeeded
24 seconds
deprecated-base-image-check

✅ Succeeded
8 seconds
clair-scan

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
7 seconds
label-check

✅ Succeeded
6 seconds
show-sbom

✅ Succeeded
6 seconds
show-summary

Pipelines as Code CI/test-component-pac-mhjb-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
19 seconds
clone-repository

✅ Succeeded
40 seconds
build-container

✅ Succeeded
32 seconds
clamav-scan

✅ Succeeded
25 seconds
inspect-image

✅ Succeeded
24 seconds
deprecated-base-image-check

✅ Succeeded
8 seconds
clair-scan

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
7 seconds
label-check

✅ Succeeded
6 seconds
show-summary

✅ Succeeded
6 seconds
show-sbom


udp/tcp/dtls: Done() signalizes that connection is not more processed.

Shouldn't we add a simple test case for this new flag?

Done


Release notes link is wrong for keptn/keptn<EMAIL_ADDRESS>

Package ecosystem
GitHub Actions
Package manager version

Language version

Manifest location and content prior to update
https://github.com/keptn/keptn/blob/f01ee895f71e9055e9acc4df755d608d50e5b854~/.github/workflows/spelling.yml
name: Spell checking
on:
  push:
    branches:
      - "**"
    tags-ignore:
      - "**"
  pull_request_target:
    branches:
      - "**"
    tags-ignore:
      - "**"
    types: ['opened', 'reopened', 'synchronize']

jobs:
  build:
    name: Spell checking
    runs-on: ubuntu-20.04
    steps:
    - name: checkout-merge
      if: "contains(github.event_name, 'pull_request')"
      uses<EMAIL_ADDRESS>      with:
        ref: refs/pull/${{github.event.pull_request.number}}/merge
        fetch-depth: 5
    - name: checkout
      if: "!contains(github.event_name, 'pull_request')"
      uses<EMAIL_ADDRESS>      with:
        fetch-depth: 5
    - uses<EMAIL_ADDRESS>

https://github.com/keptn/keptn/blob/7a109e76f3fda63cede48cceda6eea65942b2a02/.github/dependabot.yml
version: 2
updates:
  - package-ecosystem: github-actions
    directory: "/"
    schedule:
      interval: daily
    open-pull-requests-limit: 5
    rebase-strategy: "disabled"

Updated dependency

Bumps [check-spelling/check-spelling](https://github.com/check-spelling/check-spelling) from 0.0.17-alpha to v0.0.18.

What you expected to see, versus what you actually saw
commit f01ee895f71e9055e9acc4df755d608d50e5b854
Author: dependabot[bot]<EMAIL_ADDRESS>Date:   Mon May 3 08:49:51 2021 +0000

    Bump check-spelling/check-spelling from 0.0.17-alpha to v0.0.18

    Bumps [check-spelling/check-spelling](https://github.com/check-spelling/check-spelling) from 0.0.17-alpha to v0.0.18.
    - [Release notes](https://github.com/check-spelling/check-spelling/releases)
    - [Commits](https://github.com/check-spelling/check-spelling/compare/0.0.17-alpha...08f08a6ff6b9ebae06cb8fe463374a8a5a37e03c)

The release notes link should have been to:
https://github.com/check-spelling/check-spelling/releases/tag/v0.0.18
Native package manager behavior

N/A
Images of the diff or a link to the PR, issue or logs

🕹 Bonus points: Smallest manifest that reproduces the issue

n.b. I don't understand why this PR was made in the first place as noted in https://github.com/keptn/keptn/pull/3930#issuecomment-831226270 -- the tag is for a pre-release, and I wouldn't expect dependabot to offer pre-releases.

This code doesn't seem to care about releases:
https://github.com/dependabot/dependabot-core/blob/c3c87bb90042cf5b3d3af50d42b0e83fe3267041/github_actions/lib/dependabot/github_actions/update_checker.rb#L54-L71
Compare:
https://github.com/dependabot/dependabot-core/blob/b1f81765c32de788182ae204c802c3c83e081efd/cargo/lib/dependabot/cargo/update_checker/latest_version_finder.rb#L34-L39
https://github.com/dependabot/dependabot-core/blob/b1f81765c32de788182ae204c802c3c83e081efd/cargo/lib/dependabot/cargo/update_checker/latest_version_finder.rb#L50-L54
I think that if dependabot first checked to see if there were releases, and if there were, used those, and failing that looked for git tags, I'd be much happier.

Had a quick look at this. Release notes link is now correctly, but we have actually regressed here with recent update logic fixes, because 0.0.17-alpha is not considered the same precision as 0.0.20 (latest version).
A fix for that could be something like
diff --git a/common/lib/dependabot/git_commit_checker.rb b/common/lib/dependabot/git_commit_checker.rb
index 736427633..f808503c4 100644
--- a/common/lib/dependabot/git_commit_checker.rb
+++ b/common/lib/dependabot/git_commit_checker.rb
@@ -184,7 +184,7 @@ module Dependabot
     end

     def precision(version)
-      version.split(".").length
+      [version.split(".").length, 3].min
     end

     def most_specific_version_tag_for_sha(commit_sha)

I also observed (after applying that change), that the PR still has weird wording:
--commit--
Bump check-spelling/check-spelling from 0.0.17.pre.alpha to 0.0.20

Bumps [check-spelling/check-spelling](https://github.com/check-spelling/check-spelling) from 0.0.17.pre.alpha to 0.0.20.
- [Release notes](https://github.com/check-spelling/check-spelling/releases)
- [Commits](https://github.com/check-spelling/check-spelling/compare/0.0.17-alpha...v0.0.20)
--/commit--

It should read 0.0.17-alpha instead. The problem is due to some internal transformation Gem::Version does on prerelease versions with dashes.
So at least two issues here that should be addressed, although not related to the issue reported originally, which seems fine now.

Yeah, dependabot currently shows only the "numeric version" here, I guess we could switch the message to what originally specified in the workflow.

@deivid-rodriguez can you poke things?


A question regarding to HMR

I turned the reactHotLoader ON and it seems everytime I change a literal in a UI component, it does a full-reloading.
And I figured it's related to this line.
https://github.com/sysgears/apollo-fullstack-starter-kit/blob/master/src/client/index.jsx#L31
To prevent that, I need to turn frontendRefreshOnBackendChange flag off, which seems not right by the name of the flag.
I changed one of front end components, and it recognized it as a back-end change.
It may be related to SSR but I am not 100% sure.
Thanks!

@zirho When you change literal in UI component, do you see message like this in backend console?
Force front-end current page refresh, due to change in backend at: ...

@zirho Okay, I see some problem with this logic too, looking...

@zirho I have reworked front end reload on back end change logic. Should work as expected now

Cool thanks


remove unused json files

Signed-off-by: x893675<EMAIL_ADDRESS>
What type of PR is this?

/kind cleanup
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for reviewers:

Does this PR introduced a user-facing change?

None

Additional documentation, usage docs, etc.:

/milestone v1.2.0
/assign @lixd
/cc @lixd

Codecov Report

Merging #130 (12d3475) into master (ae86c48) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #130   +/-   ##
=======================================
  Coverage   11.79%   11.79%
=======================================
  Files          98       98
  Lines       15107    15107
=======================================
  Hits         1782     1782
  Misses      13090    13090
  Partials      235      235


Titanic Survivals Predictions

#38  Predicted titanic survivals using Decision Tree Classifier which gives us approx. 67% accuracy.

@all-contributors please add @gargnityansh

@all-contributors please add @gargnityansh

@all-contributors please add @gargnityansh for Code

@all-contributors please add @gargnityansh for Code


Fix bug with initializing service with channel parameter

Summary
Details and comments
Fixes #551

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

9 of 12 (75.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 66.585%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_ibm_runtime/accounts/management.py
9
12
75.0%

Files with Coverage Reduction
New Missed Lines
%

qiskit_ibm_runtime/accounts/management.py
1
95.58%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
3246

Relevant Lines:
4875

💛  - Coveralls


Include coding aspect to assignments for week 7 and 8

Update Week 7 and 8 assignments to include a coding element based on either

aima-python/learning-app.ipynb,  but only if the bug producing a ValueError in cell 8 when initializing a DataSet object with the MNIST dataset is fixed

sklearn(http://scikit-learn.org/stable/modules/generate /sklearn.tree.DecisionTreeClassifier.html) and you can also export it as a graphviz: http://scikit-learn org/stable/modules/generated/sklearn.tree export_graphviz.html

Material no longer included in course

Course no longer covers these topics. So, I am closing this issue.


A more readable code?

Hey, you don't have to merge this.
I just thought I would share the modifications I did to your code. You can add what you liked to your code if you liked any of them!

I changed the initialization of the states array to use the Array constructor and the .fill() method to set all positions to 1.
I changed the name of the "r" variable to "steps" and made a small while to find the number of the dice rolled (and the index we hit in the states array).
With that index in hand, now we can update every state using a .map() call, setting the target index to 1 and all others to their previous state + 1.

Let me know if you have any better ideas too! I loved your idea and had fun looking at your code so I decided to maybe add some stuff to it!
Edit: I just noticed that I could've left the "sum" as const!

This looks very promising, when I get home tonight I'll look it over in more detail. Thanks for your contribution!

I made some minor adjustments and merged it in here.


Add support for esbuild 0.23.1

package.json is currently prohibiting use of esbuild 0.23.1:
  "peerDependencies": {
    "esbuild": ">=0.14.0 <0.23.1"
  },

esbuild 0.23.1 is out now:

Thanks for that! I'll do an update automation to avoid similar situations in the future.


Form encoding for submissions

In the Hubspot::FormConnection.submit method, the content type is set to 'application/x-www-form-urlencoded', but the body is still encoded to JSON. Per the documentation, the body should be form encoded, eg<EMAIL_ADDRESS>

I confirmed that this isn't working in the HubSpot dashboard. When I submit using this library as-is, I get a 204 response, but a blank entry shows up in form submissions:

I then monkey-patched the gem to not use to_json:
require 'hubspot/connection'

module Hubspot
  class FormsConnection < Connection
    follow_redirects false

    def self.submit(path, opts)
      url = generate_url(path, opts[:params], { base_url: 'https://forms.hubspot.com' })
      post(url, body: opts[:body], headers: { 'Content-Type' => 'application/x-www-form-urlencoded' })
    end
  end
end

And the submission appeared correctly!:
![image](https://cloud.githubusercontent.com/assets/91577/8434100/6bf33ddc-1f17-11e5-9424-2c5c3b1c0a73.pn


Support for non-Apple keyboards

I am aware that currently, fluor supports only Apple keyboards. What is the reason for this, and what would be required to add support for other keyboards, e.g. Logitech MX Keys? Is it simply a matter of finding the right way to tell the keyboard to switch modes, or is it more difficult than that?
I'm not a macOS / Swift programmer so it would probably be quite difficult for me to find this out by myself...
Thanks, Victor

It seems to be a complicated issue, as reported in this reddit thread

Would be nice to have a setting to set an arbitrary shortcut to toggle.


Occluded landmark

How are you handling the loss for occluded landmarks?
I was looking for this detail in the paper but I didn't find anything.

How are you handling the loss for occluded landmarks?
I was looking for this detail in the paper but I didn't find anything.

According to the face annotation of widerface, we filter the occluded face in the landmark task.

How are you handling the loss for occluded landmarks?
I was looking for this detail in the paper but I didn't find anything.

According to the face annotation of widerface, we filter the occluded face in the landmark task.

I meant more the case when you have one or two occluded landmarks so a very partial occlusion of the face.

I meant more the case when you have one or two occluded landmarks so a very partial occlusion of the face.

This case doesn't matter, and it can be used for training.

So here:

Different from Box regression, the regression of the five facial landmarks adopts the target normalization method
based on the center position.

So if you have on a face just 4 visibile landmarks are you simply going to exclude that one from the regression?

So here:

Different from Box regression, the regression of the five facial landmarks adopts the target normalization method
based on the center position.

So if you have on a face just 4 visibile landmarks are you simply going to exclude that one from the regression?

Invisible points still regression. And this can improve the robustness of landmarks detection.

This Is valid only if you have occluded points in GT.


show many <<<<<<

hi i am start using bracket gits today, in the beginning page show many <<<<<< in the beginning and end of my script file, thanks for you apreciation

Brackets 1.4.0-16380 (linux), Git 1.9.1 & Brackets-Git 0.14.32

https://www.google.com.au/search?q=git+merge+arrows

ok, thanks


Add ARM 32 and 64 cross-build

Enables use on RaspberryPI and similar devices.
Based on https://github.com/ckulka/docker-multi-arch-example
You have to configure build to something like that:

@andrusha Could you just correctly prefix image tags? I’ll be reviewing this on Monday again, great job so far! 💯👌

Updated, after its merged the first build will fail until all architecture images become available


[Epic] React Native .73 Compatibility

### Tasks
- [ ] https://github.com/wix/Detox/issues/4338
- [ ] https://github.com/wix/Detox/issues/4339
- [ ] https://github.com/wix/Detox/issues/4340
- [ ] https://github.com/wix/Detox/issues/4341
- [ ] https://github.com/wix/Detox/issues/4342
- [ ] https://github.com/wix/Detox/issues/4343
- [ ] https://github.com/wix/Detox/issues/4345
- [ ] https://github.com/wix/Detox/issues/4346

Any reason this is still open? Is this supported yet? Are their plans to close it?


Initial commit to setup downstream repo dockerfile

Signed-off-by: Mohamed Mahmoud<EMAIL_ADDRESS>Add Dockerfile.openshif and owner fiiles required by ART ticket 3029

/overide ci/prow/e2e-aws

/lgtm

/approve
/lgtm


Install guide?

hi lads, im on an opensuse system and everytime i launch hyprland it tells me about installing hyprland-qtutils but i cant find any documentation or anything about how this should be installed, what dep it needs, etc.... can we please get a guide on that?

same as all hyprland apps, just use cmake.
See the instructions @ xdph for example: https://github.com/hyprwm/xdg-desktop-portal-hyprland
(obviously deps are different though)


Bump null-label to 0.17.0 for Terraform 0.13 support

what

Bumped the pinned version of null-label to the version that supports Terraform 0.13

why

Module doesn't work on 0.13 without this change :)

note

This is a PR for #32
This PR is currently blocking updates on the terraform-aws-cloudfront-s3-cdn module (and likely others) as it depends on this module

/test all


what if we want this library to use in an single project so we done have to put the files under the htdocs folder ?

its done


Uninitialized constant using Rails 4.2

Hi,
I'm use different fields for my address in database. So I need to concat LibAddr, LibZipc, LibCity and LibCtry.
Using this return an uninitialized constant :
geocoded_by :full_address
after_validation :geocode
def full_address
[LibAddr, LibZipc ,Libcity, LibCtry].compact.join(', ')
end
Trying :LibAddr and so on return nothing and trying only geocoded_by :LibAddr work fine.
Precision : Informations are posted to the controller as a json post (API)
Thanks
Update : Trying with a view form in another project work, so, what's the method for an api. Thanks.

Hi (again lol)
I found a way but i'm not sure it is the best. So I added a LibFullAddr param in my database.
Now i use geocoded_by :LibFullAddr
And in my controller I merge  LibFullAddr in posted params using
def set_full_address
[params[:site][:LibAddr], params[:site][:LibZipc] ,params[:site][:LibCity], params[:site [:LibCtry]].compact.join(', ')
end
It seems to work fine.
Let me know if there is a simple way when using json post, patch or put
Best regards.

What does this have to do with the functionality provided by Geocoder? I think StackOverflow is a better forum for this kind of question.


Added F1 score Feature

✨ Feature: F1 Score Calculation & Improved Test Output Readability
Description of the Feature
This update introduces the F1 Score calculation feature alongside improvements
in the readability and clarity of test outputs for Exact Match, Precision,
Recall, and F1 Score tests. The new test structure ensures that results are
presented in a more accessible and informative format, making it easier to evaluate
the performance of each test case and metric.
Key Features:

F1 Score Calculation:

The F1 Score metric is added to measure the harmonic mean of precision and recall,
providing a balanced evaluation of test performance where both metrics are considered.

Improved Test Case Readability:

Each test case is clearly labeled and presented in an organized format, making it
easier to track input/output and the corresponding calculated metrics (e.g., precision, recall, F1 score).

Detailed Metric Display:

For each test case, both the input and expected output are displayed alongside the calculated
metric (precision, recall, or F1 score), making comparison of actual vs. expected behavior clear.

Metric Summaries:

After evaluating all test cases, the mean value of the metric being tested is displayed in a
summary, offering a quick overview of the test’s overall performance.

Enhanced Debugging Support:

Each test provides detailed information about the predicted and expected values, including the
calculated metrics, making it easier to identify potential issues in case of mismatches.

Example Test Output:

========== F1 Score Test Start ==========

Predicted: paracetam, Actual: paracetamol, Precision: 0.0, Recall: 0.0, F1 Score: 0.0
Predicted: aspirin, Actual: aspirin, Precision: 100.0, Recall: 100.0, F1 Score: 100.0
Predicted: ibu, Actual: ibuprofen, Precision: 0.0, Recall: 0.0, F1 Score: 0.0
Predicted: ibuprofen, Actual: paracetamol, Precision: 0.0, Recall: 0.0, F1 Score: 0.0

Test Case 1:
  Input: paracetam
  Expected: paracetamol
  F1 Score: 0.00%
-------------------------------------------------------

Test Case 2:
  Input: aspirin
  Expected: aspirin
  F1 Score: 100.00%
-------------------------------------------------------

Test Case 3:
  Input: ibu
  Expected: ibuprofen
  F1 Score: 0.00%
-------------------------------------------------------

Test Case 4:
  Input: ibuprofen
  Expected: paracetamol
  F1 Score: 0.00%
-------------------------------------------------------

Summary: Mean F1 Score = 25.00%

========== F1 Score Test End ==========

See comment on PR #62
Also, #56 gives the formula for a more general $F_\beta$ score, which is pretty easy to implement, so could we have that instead of the more specific $F_1$ score?


fix(material-experimental/mdc-card): remove extra margin if header doesn't have an avatar

The card header margin was set up assuming that there would always be an avatar element, but that's not always the case. These changes add an extra check to avoid the extra margin.
Fixes #19069.

Does the mdc-based card do something similar?

Looks like it does, but we'd have to handle it differently since it uses padding to do the spacing. Do you remember the context behind it @jelbourn (you're on the blame)?  https://github.com/angular/components/blob/master/src/material-experimental/mdc-card/card.scss#L33

IIRC, this was the best I could do to preserve the existing spacing for the content regions without adding spacing to the card itself. Padding contained the spacing to only the specific content regions, which made the styles much simpler; when it was margin, the styles needed to be more aware of adjacent siblings and the like.

I've pushed a fix to cover the MDC card @jelbourn.

Can you add a demo app example (one for each version) that demonstrates this situation?

Done.

This breaks about 61 targets worth of screenshot tests. All of them are cases where it looks like they were depending on this padding, so it breaks the app layout.

@jelbourn Whats the plan for this fix then, will this be considered for v10?

Due to how breaking this is, we probably won't land it at all.

This makes mat-card pretty annoying for us, as we have to work around this on every usage with a title, as we never use the avatar element :/

This makes mat-card pretty annoying for us. We have to work around this on every usage with a title, as we never use the avatar element :/

You could fix it in your styles.scss file globally till mdc-card arrives.

I've reduced the scope of these changes to only apply to the MDC card.

Ran presubmit last night, looks like this has about 41 presubmit failures.


Renovate/eslint plugin react 7.x

@kristastarr Hey! Is this something I can review? Or did you want to hold off for Taylor to take a look?

@johnbister Thanks, I need to look into these bot PRs a little further but I will let you know!

This one it's pinning the dep instead of using the range. I deleted the duplicated config file,  I suggest closing this one for now and letting renovate pick it up again when the next minor version comes out.


Templating

Might be worth taking a look at this for some inspiration if you haven't already seen it https://github.com/tadzik/Template-Mojo/

Thank you. I'm going to add some template feature.
Template-Mojo looks good, but It does not escape HTML metacharacters.

just a trial https://github.com/shoichikaji/Frinfon/tree/template-trial


Do not trigger_error in production environment when refreshing the pl…

Questions
Answers

Why?
When you want to deploy code with a ci server to different servers without any user interaction in the backend you need to refresh the plugins during the deployment (php bin/console sw:plugin:refresh). When you delete a plugin that registers a cli command from your code it is not possible to refresh the plugins to force a database cleanup (Event Subscribers etc.).   So instead of removing the code completely, I thinks it is a good compromise to not trigger the error in a production environment. If you refresh the plugin list in the backend there is not error or warning and everything is working as expected.

BC breaks?
in some cases maybe

Tests exists & pass?
no

Related tickets?

How to test?
Install and activate a plugin that registers a cli command (SwagImportExport). Remove the code of the plugin and call the php bin/console sw:plugin:refresh command.

Requirements met?
yes

Thanks @nlx-sascha. The solution you provide is too broad since it would omit the errors for all customers in production. This would lead to hidden errors.
Sadly, we have to decline your PR. Thanks for your effort!


[$50] mWeb - Workspace - In offline, unable to save workspace name/ description

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

Version Number: 9.0.79-0
Reproducible in staging?: Yes
Reproducible in production?: No
Issue reported by: Applause Internal Team
Device used: Redmi note 10s Android 13
App Component: Workspace Settings
Action Performed:

Go to https://staging.new.expensify.com/home
Go to workspace settings
Turn off internet connection in the device and wait until header shows no internet connection
Create a new workspace
Try to rename and save workspace name/ description

Expected Result:
In offline, must be able to save workspace name/ description.
Actual Result:
In offline, unable to save workspace name/ description.
Workaround:
Unknown
Platforms:

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

Screenshots/Videos
https://github.com/user-attachments/assets/d424d25f-e5c7-4fdb-937e-303bbe438435
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021872619802972851604
Upwork Job ID:<PHONE_NUMBER>972851604
Last Price Increase: 2024-12-27
Automatic offers:

suneox | Reviewer | 105490775
huult | Contributor | 105490778

Issue OwnerCurrent Issue Owner: @ZhenjaHorbach

50$ to identify the offending PR and confirm with the revert.
I have demoted as I have tested and this is not a problem if the workspace is created online. Its very rare that the workspace would be created offline and user needs to update the name straight after

This is caused by https://github.com/Expensify/App/pull/53529

@huult @suneox can you please have a look or should we revert your PR and address it later?

I'm checking

@mountiny I found the root cause of this issue. It was due to an update to eslint for getCustomUnitRateID, which caused this issue to occur.
https://github.com/Expensify/App/blob/670440364067251e855d0a9cd7a7aad5c5a3a666/src/libs/DistanceRequestUtils.ts#L293-L297
May I revert this commit?

@mountiny I found the root cause of this issue. It was due to an update to eslint for getCustomUnitRateID, which caused this issue to occur.
https://github.com/Expensify/App/blob/670440364067251e855d0a9cd7a7aad5c5a3a666/src/libs/DistanceRequestUtils.ts#L293-L297
May I revert this commit?

@huult Have you got solution to resolve this issue instead of only revert commit?

yes @suneox I will create pull request soon

@suneox @mountiny The pull request https://github.com/Expensify/App/pull/54634 is ready. Could you take a look?

@huult @suneox can you please have a look or should we revert your PR and address it later?

@mountiny The PR doesn't resolve this issue, so I think we can revert PR until we find the root cause.

I haven't really dig into the issue, but I found that it's caused by the keyboard utils dismiss function of index.website.ts

@suneox Could you check again? thanks

@suneox Could you check again? thanks

Can you provide the RCA for this issue? In your new update, it only ignores iOS Safari, but I’ve double-checked this issue also occurs on the latest staging with iOS Safari (real device).
https://github.com/user-attachments/assets/bec2eacd-dc09-4c27-963d-f1d0432f0d91

@suneox Yes, we have two issues here:

We updated ESLint and added a return early condition, which caused the send button to become unclickable. I have fixed this by removing the return early condition.

For dismissing the keyboard, we used visualViewport to detect and hide the keyboard. This should only work on Safari.

I tested it on the iOS version of the mobile web, and it’s working well.
https://github.com/user-attachments/assets/db473bc6-5109-47b0-b173-f3b6092ca12b

We updated ESLint and added a return early condition, which caused the send button to become unclickable.

@huult Your RCA isn't correct

For dismissing the keyboard, we used visualViewport to detect and hide the keyboard. This should only work on Safari

The issue occurs on a real device when the smart banner app is displayed.

Yes, I will check this case (the smart banner is displayed).

@suneox When the smart banner is displayed, the button cannot be clicked. This issue is not related to this ticket or my pull request. I tested other cases, such as changing the display name on iOS, and the button still cannot be clicked.
https://github.com/user-attachments/assets/63d74209-52c3-47be-ab1e-42902e8ae74c

@suneox  I have found the RCA of this issue. I will update the pull request in a few hours. The problem occurs because isVisible is not updated to false when the smart banner is displayed, causing the condition currentHeight === initialViewportHeight to fail
https://github.com/Expensify/App/blob/968a125793147d8b9b7a5378913b2ee7eb0592c2/src/utils/keyboard/index.website.ts#L18-L20

@suneox I have finalized the pull request. Could you check it again? Thank you!


Remove Ternary and CTWildcard

Co-authored-by: Max Kießling<EMAIL_ADDRESS>

We kept the default only for the very common Var case that is used in too many tests. For every other case we deliberately opted for removing those defaults to make it more visible that the correct type needs to be supplied


Can we Connect to RDBMS - SQL server instead of Mongo DB

Hi,
I wanted to know if we can connect to SQL server instead of Mongo DB.
Since Repository pattern is not used and it is tightly coupled with Mongo DB , can you tell me if there is a way to connect to SQL sever . I just want to try it out and have a flat table structure.
Any inputs on this are appreciated.
Regards,
Vinuth

@vinuth02 thanks for reaching out. I have previously commented on this in issue #163.
Adding a library for SQL-compliant databases is something I would like to include, but have not had time to do.
If one would like to add support for another data store like SQL or create a FHIR facade one would implement the following interfaces:

IFhirStore – provides basic create/read functionality
ISnapshotStore – provides create/read functionality of Snapshot objects (used for making result search links non-volatile)
IIndexStore – manages the search index
IFhirIndex – used for search

Spark.Engine is a the core library where all the mentioned interfaces and other core functionality lives. Spark.Mongo is the concrete storage implementation for MongoDB using the mentioned interfaces from Spark.Engine.
This commit https://github.com/FirelyTeam/spark/commit/9ff12ed5aac39babe6e116755b5127ce32e49d20 was a POC for using Spark as a FHIR facade, it currently only implements IFhirStore. This would be the exact same way you would go forward to create a SQL-compliant storage implementation.

@kennethmyhra
Thank you for your inputs . I will try this and let you know.
However , i have a question - Should we make changes in HomeController as Mongo DB instance is created there.
Thanks,
Vinuth

I recommend that instead of patching on the reference implementation you use the NuGet packages as described here: https://firelyteam.github.io/spark/quickstart
To inject your own store you would use dependency injection. For inspiration you can look at how that is done in Spark.Mongo: https://github.com/FirelyTeam/spark/blob/master/src/Spark.Mongo/Extensions/IServiceCollectionExtensions.cs

@kennethmyhra
Thanks for your inputs.
The reference provided (https://firelyteam.github.io/spark/quickstart) is pointing to MongoDB.
However I am looking for integrating SPARK with RDBMS instead of MongoDB.
As mentioned earlier, Mongo is tightly coupled with the SPARK design. I didn't find any helpful approach to bring in RDBMS.
Please let me your thoughts on the same.
Thanks,
Vinuth

My recommended approach if you want create an integration with a RDBMS, which there currently is no storage library for, you would have to implement that using the mentioned interfaces:

IFhirStore – provides basic create/read functionality
ISnapshotStore – provides create/read functionality of Snapshot objects (used for making result search links non-volatile)
IIndexStore – manages the search index
IFhirIndex – used for search

Then you add a similar method to AddMongFhirStore, for your RDBMS. In this method you need to add your implementations for the mentioned interfaces above IFhirStore, etc:
static void AddSqlFhirStore(this IServiceCollection services, StoreSettings settings)

Then in your ConfigureServices method you could switch AddMongoFhirStore with AddSqlFhirStore like this:
public void ConfigureServices(IServiceCollection services)
{
    // Sets up DI context and adds neccessary infrastructure, like ASP.NET MVC
    services.AddFhir(new SparkSettings
    {
        Endpoint = new Uri("https://localhost:5001/fhir")
    },
    options => options.EnableEndpointRouting = false
    ).SetCompatibilityVersion(CompatibilityVersion.Version_3_0);

    // Adds support for RDMBS Store
    services.AddSqlFhirStore(new StoreSettings
    {
        // Connection string for your MongoDB Store
        ConnectionString = "mongodb://localhost/spark-r4"
    });
}

@kennethmyhra
Thank you . I shall try this out.
Can you please provide me inputs on below query

I'm trying to filter out queries. Foe example , if there are 10 Patient records , i want get only few of them . As of now "Search" method fetches all records and there is no way to filter it out.

So wanted to know if there is a way to filter out the query to get few records instead of all.

Thank you.
P.S - Also wanted to know if we can have a call , so that we can discuss in detail. I'm available of Skype (vinuth02). Please let me know
Regards,
Vinuth

If you want to limit the number of results you receive you can add the query string parameter _count. i.e: /Patient?_count=5
Also here are the links to the FHIR search specification and REST specification.

@kennethmyhra
Thank you . Giving count will limit the number of records.
But is there an option to pick few only . For example , i have 5 patient records and i want pick patientid's - 1.3,5 only.
Where can i query this or where can i put a condition to pick only these?
Thanks,
Vinuth

Then you would do it like this: /Patient?_id=1,3,5
_id parameter is documented here: http://hl7.org/fhir/search.html#id
Composite search parameters OR, AND, etc: https://www.hl7.org/fhir/search.html#combining
Spark implements most of the specification I recommend you go through the links in my comment above

Hi @kennethmyhra
Thank you . This will work fine for few patient records but in other scenarios ,if i want to fetch more than100  records , i wont be able to add it to the Query String. Do we have any alternative to this ? Where in if we can fetch selected patient records for an User.
Thanks,
Vinuth

As I said, you would need to go through the search documentation to find what part fits your use case, there are many ways to you can query using search. Also each resource has a set of search parameters, search parameters for example for Patient they are here: http://hl7.org/fhir/patient.html#search
There is also FHIR community where you can ask questions and learn more: https://chat.fhir.org/


Constant blue light on board after upgrading to 2.3.3

I'm getting constant blue light on board after upgrading to 2.3.3. Tried flashing many times again with Konnected Flash tool but no luck.

Issue can be closed as it might be an issue with my USB port or laptop but I tried two different laptops . @heythisisnate Please note I tried flashing with Konnected Flash tool many times and it didn't work, then tried with esptool.py using standard command mentioned in scripts/flash, also did not work. WHy would the Konnected Flash tool suddenly not work, I tried on two different laptops.
This is the command that worked and it only works when flashing filesystem first then firmware.
esptool.py --port=COM6 --baud 460800 write_flash --flash_mode dio --flash_size=detect 0 konnected-filesystem-0x100000-2-3-1.img

This problem can be caused by a filesystem flashed at the wrong memory location.
The filesystem file needs to be flashed at address 0x100000. Command should look like this:
esptool.py --port=COM6  write_flash --flash_mode dio 0x100000 konnected-filesystem-0x100000-2-3-3.img

Next release we'll be combining the FW and FS images into one to make it simpler to flash all at once.


iOS 10 Problem

Browser: Safari iOS
OS: iOS 10
Clappr Version: 0.2.63
Steps to reproduce:

first step

 var player = new Clappr.Player({
            source: "https://hls.***.com/ios/smil:live.smil/playlist.m3u8",
            parentId: "#player",
            autoPlay: true,
            width: '100%'
        });

then second step
The system was working on iOS 9 perfectly but when I updated to iOS 10 it suddenly gives the error No playable sources found  and no error on the browser console.

Did you try to reproduce this issue at http://cdn.clappr.io/
I've tried on the cdn it worked perfectly.

@DKarakis did you try to use http://www.streambox.fr/playlists/x36xhzz/x36xhzz.m3u8 as source just to test?

@leandromoreira thank you for replying back. I solved the problem. It turned out to be I was failing while my security check. Nothing related with the product itself. Have a good day!


Customize Dialogue -> Specifications -> linux_host_name => {}

I am experiencing an issue where the customization specifications that MIQ is pulling from a vCenter is populating the Host name[linux_host_name] string with a pair of curly brackets {}.  This is what it shows  when I goto VM Provision=>Choose my Template=>Customize Tab=>Customize(Specification)=>Choose my specification file.  In the Naming section for "Host Name" it shows a pair of curly brackets.
Currently in my VMWare specs object, I have it set to set hostname based on the vmware hostname.  If I specify a hostname in the VMWare specifications, instead of just empty brackets, I get the hashstring { "host"=>"testvmname" }.   Normally, I have to override the selection and remove the brackets to complete my build using the vmname preprov and my ipam information.  I am not using DHCP, I use the "Prompt user for input" for my VMware specs file, and provide the type and IPADDR information through MiQ querying my IPAM server.
If I click my Specification object, but do not override, it returns that I am using an "Unsupported Customization Spec option detected: User must prompt for IP address".  I believe that the {} is causing MIQ to pass too many parameters to VMWare resulting in incorrect information passed maybe?
If I click my Specification object, and click override but leave the brackets, I receive a specification error about hostname being in bad format (which I would expect).
I need to either remove the brackets from the field, or perhaps adjust how I am provisioning a VM through MIQ to VMWare?

Still unable to determine what is causing the curly brackets to appear on the Customization page from a VMWare spec.

@gmcculloug Who should look at this?

@syncrou I have seen this in the UI and can help you recreate.

@syncrou if this is still a relevant issue, can you remove the "stale" label. Otherwise we'll see about closing.


Comments: Account for custom comment types when calculating comment counts

Fixes #1069.
The benefit of this approach is that it doesn't add a DB query to front end requests.
The downsides are that if another plugin uses the same filter, there's a potential for conflict, and comment counts won't change until WordPress finds a reason to update the count, like approving/unapproving a comment.
One more consideration:
We might want to add an upgrade routine so we can grab all posts with a like or reblog and update their counts, after the plugin was upgraded (example).
There is almost certainly more work needed to make this work correctly.
Proposed changes:

Adds callback that queries comment counts without Activitypub custom comment types.

Other information:

[x] Have you written new tests for your changes, if applicable?

Testing instructions:

Apply the diff.
Go to the comments list on your development site.
Approve/unapprove a like or repost.
Make sure the new comment count reflects that number of actual comments (and pings).

I've done some testing, how these changes behave with different settings and it seems that if you enable "paging" for comments, it shows no comments at all. Will see how this happens.

I fixed paging for classic themes, but FSE themes seem still be broken.


No result

Hello,when I  execute the following command:
python apis/run.py -d options/examples/default_configuration.yml
The terminal  print this message：
Save result in result/tmp/0/? [y/n]y
==> Initialize LiteFlowNet with [model_zoo/optical_flow/LiteFlowNet/network-default.pytorch]:
==> Initialize Depth-CNN with [model_zoo/depth/kitti_odom/stereo/]
==> Start DF-VO
==> Running sequence: 10
Start with frame: 25
0it [00:00, ?it/s]
=> Finish!
The result is saved in [result/tmp/0/].
Save VO map.
Trajectory saved.
----- time breakdown -----
Group [single]:
But the map.png in the result folder is black and .txt file only save one line of data.Is the dataset not imported and run?
Thank you for your help!

Hi genius-hl, could you check the data directory/data loading to make sure if the images have been loaded?

Got the same error. Any solution for this?

Yes, because I have converted KITTI data from png to jpg. Please update image: ext in the configuration file.

Got the same error. The issue was fixed after changing "jpg" to "png."
Thank you for your tips !

Got the same error. The issue was fixed after changing "jpg" to "png."

Thank you for yout tips!

Yes, because I have converted KITTI data from png to jpg. Please update image: ext in the configuration file.

Thank you for your reply ! I got the result.


GVFSService: platforms should use appropriate pipe name

On POSIX platforms the pipe name should be a rooted path so that we have
full control over the pipe path. If a relative path is used, then the
pipe is created in a temporary dictory, which can be a different
location depending on the user's context (i.e. running as root vs normal
user). This can result in an process running as root not being able to
communicate with the GVFS Service.
This change enables the different platforms to create a pipe path that
is suitable for the platform, and encapsulates the logic for creating
the pipe name for client / server in a single location.

This is not immediately relevant to this PR, but the changes here reminded me to note that the current code which generates named piped names on POSIX platforms (Mac, Linux) does not constrain them to the limits imposed on domain sockets of about 100 characters.
One can generate a stack trace like the following by simply mixing up the order of arguments on the gvfs clone command line:
Unhandled Exception: System.ArgumentOutOfRangeException: The path '.../.gvfs/GVFS_NetCorePipe' is of an invalid length for use with domain sockets on this platform.  The length must be between 1 and 108 characters, inclusive.
...
   at System.IO.Pipes.NamedPipeClientStream.Connect(Int32 timeout)
   at GVFS.Common.NamedPipes.NamedPipeClient.Connect(Int32 timeoutMilliseconds) in .../VFSForGit/GVFS/GVFS.Common/NamedPipes/NamedPipeClient.cs:line 29
   at GVFS.CommandLine.GVFSVerb.IsExistingPipeListening(String enlistmentRoot) in .../VFSForGit/GVFS/GVFS/CommandLine/GVFSVerb.cs:line 360
   at GVFS.CommandLine.CloneVerb.CheckNotInsideExistingRepo(String normalizedEnlistmentRootPath)
...

I could put this into an issue if it's not something you'd like to address in this PR, but I thought I'd mention it here because creating pipe names using absolute paths may also sometimes trigger this exception, depending on the path.

I could put this into an issue if it's not something you'd like to address in this PR, but I thought I'd mention it here because creating pipe names using absolute paths may also sometimes trigger this exception, depending on the path.

Thanks for pointing this out! If I understand the issue correctly, I think we are most susceptible to this in the pipe for the mount process, which depends on the enlistment path. I think the paths for the GVFS Service's named pipes are more constrained / controlled, as they do not depend on a mount's directory.
Let's file a separate issue for this - would you mind filing it? If not, I can go ahead and file it to make sure we track it.


node-exporter: 1.0.0 -> 1.0.1

Motivation for this change
https://github.com/prometheus/node_exporter/releases/tag/v1.0.1
Things done

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

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

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

This change is

Result of nixpkgs-review pr 90614 1

1 package built:
- prometheus-node-exporter

@GrahamcOfBorg test prometheus-exporters


@BeforeClass doesn't get run on class with @Factory method -- expected behavior?

Here's a distilled version of my code: the parent class launches my GUI program in the init() method, and then the factory method creates inner test classes that access the GUI via the parent class instance.  TestNG didn't handle this how I expected; 1) did TestNG execute it properly?  2) Is there a better way for me to tie all the factory-created tests to the same GUI instance?
public class TestFactory {
    @BeforeClass
    public void init() {
        System.out.println("BeforeClass");
    }

    @Factory
    public Object[] createTests() {
        System.out.println("Factory");
        return new Object[] { new Test1("GOOG"), new Test1("FB") };
    }

    public void enterSymbol(String symbol) {
        System.out.format("TestFactory %x: entering %s\n", hashCode(), symbol);
    }

    public class Test1 {
        private String  symbol;

        public Test1(String symbol) {
            this.symbol = symbol;
        }

        @Test
        public void test() {
            System.out.println("Test1(" + symbol + ")");
        }
    }
}

Output:
Factory
[TestNG] Running:
  C:\Users\james.a.edwards\AppData\Local\Temp\testng-eclipse--1490678808\testng-customsuite.xml

Test1(FB)
TestFactory 1264eab: entering FB
Test1(GOOG)
TestFactory 1264eab: entering GOOG
PASSED: test
PASSED: test

Expected Output:
BeforeClass
Factory
[TestNG] Running:
  C:\Users\james.a.edwards\AppData\Local\Temp\testng-eclipse--1490678808\testng-customsuite.xml

Test1(FB)
TestFactory 1264eab: entering FB
Test1(GOOG)
TestFactory 1264eab: entering GOOG
PASSED: test
PASSED: test

Closing this issue with resolution as Working as designed because the example shared is not a correct one. @BeforeClass will be executed only if the class has atleast one @Test method. If this is still a problem, please share a proper example that can be used to reproduce the problem.


Feature/nx examine controller

Change the Inspect Mouse controls to mimic CAD-Style navigation.

Any further objections?


[🐛 in esl-event-listener]: host is missed in debug information of empty subscription

Add host to debug information for EventListener in case of any warnings from ESL side

:tada: This issue has been resolved in version 5.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 4.15.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Event Request] Page 9800 Users - OnBeforeSetUserName

Please add the following Event OnBeforeSetUserName in Page 9800 Users
    local procedure SetUserName()
    var
        IsHandled: Boolean;
    begin
        //>> New event
        IsHandled:= false;
        OnBeforeSetUserName(Rec, XRec, WindowsUserName, IsHandled);
        if IsHandled then
            exit;
        //<< End
        "User Name" := WindowsUserName;

    [IntegrationEvent(true, false)]
    local procedure OnBeforeSetUserName(var User: Record User; xUser: Record User; WindowsUserName: Text[208]; var IsHandled: Boolean);
    begin
    end;

Please provide explanation why you need this change. Any manipulation with use names can be a security concern.

Closing due to inactivity. Please create a new issue if needed.


[Feature]: VSCode - Add schema support for .taq/config.json

When editing the .taq/config.json file in VS Code, we should utilize VSCode to make it aware of the schema so that it can do things like autocomplete, show errors when the configuration is invalid and underline the problem, etc.

@mweichert, is Zod able to generate a JSON schema?
We would want to automate the generation of the schema so that there is no maintenance burden


fixes template renaming while renaming a partitioned table

the template prefix which is used to match against a given index name
was not renamed and so this template would still match on the old table
name. this results in that partitions will belong to the wrong table.
fixes: https://github.com/crate/crate/issues/5823

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@10d2764). Click here to learn what that means.
The diff coverage is 86.36%.

@@            Coverage Diff            @@
##             master    #5839   +/-   ##
=========================================
  Coverage          ?   82.91%
  Complexity        ?    12089
=========================================
  Files             ?     1428
  Lines             ?    46421
  Branches          ?     5719
=========================================
  Hits              ?    38490
  Misses            ?     5251
  Partials          ?     2680

Impacted Files
Coverage Δ
Complexity Δ

...io/crate/analyze/CreateTableAnalyzedStatement.java
88.88% <100%> (ø)
26 <0> (?)

...src/main/java/io/crate/metadata/PartitionName.java
84.94% <100%> (ø)
36 <1> (?)

.../crate/executor/transport/AlterTableOperation.java
86.94% <85%> (ø)
76 <3> (?)

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 10d2764...fcb19a1. Read the comment docs.


How to use an "icon name included in the project" for the action image

Hi, it says in the README:
   * - Allows icon name included in project or system (Android) resources drawables and
   * in SF Symbol (iOS)

Does it mean you add the icon to the native project, and then you can refer to it? How do you do this?
Thanks

You can find all the icons on Hotpot.

For Android Icons, click the Material Design option.
For iOS Icons, click the SF Symbols option.

Then to select an Icon, just click it, this copies the icon name to your clipboard.
Here's an example to get the Safari (Browser) icon, for iOS only.
{         id: 'open',         title: 'Open',         image: Platform.select({             ios: 'safari',         }), }
Hope this answers your question 😁

Thanks @kayode0x.
Sorry, I wasn’t clear enough - I know about using SF Symbols and Material Design icons.
The readme implies that you can alternatively use an “icon name that is included in the project” - or am I misreading it?
Thanks

it is not working for android
the is creashing when i use the name

For Android Icons, click the Material Design option.
For iOS Icons, click the SF Symbols option.

Hi there, I'm experiencing the same issue as @hanishbasheer, my android icons are not working (they are working on iOS) It simply doesn't render any image even using the example code in the docs

Same issue on android
React Native: 0.71.2
@react-native-menu/menu": "0.7.3"

Same for me, for android:
Using eg 'ic_menu_add' works and gives me a plus sign in a circle.  But none of the names on for android Hotspot work in the code.  And none of the icons for android on Hotspot are called 'ic_menu_add'.
So it looks like the code is using a different set of icon names for android than Hotspot is.
Where do I find the list of icon names, and their images, that work in this code?

@willxy you need to add the icons via android studio In Android Studio, right-click on the “drawable” folder and select “New” » “Vector Asset”.

@willxy you need to add the icons via android studio In Android Studio, right-click on the “drawable” folder and select “New” » “Vector Asset”.

Thanks this helped.

@willxy you need to add the icons via android studio In Android Studio, right-click on the “drawable” folder and select “New” » “Vector Asset”.

This kinda worked for me but instead of using right-click on the "drawable" folder, I clicked on the "Resource Manager" tab on the left and then clicked the plus button and selected Vector Asset.

For those wondering, the default icons present for android can be found here (those that don't need an extra-install step): https://developer.android.com/reference/android/R.drawable
For instance: android: "ic_menu_gallery"


fix: replace and deleteOrphaned always default

Fixes issue #165

Looks good, thanks for the fix!
Could you resolve the conflicts by removing the package-lock changes? Will merge after that.

Thanks,
Changes to lock file reverted.

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


Adopted build system to hosts without /bin/bash

Some linux hosts don't keep their bash binary under /bin but in other
places. One example of such a host is nixos that keeps it somewhere
under /nix/store.
Instead of assuming the location of the binary, use /usr/bin/env to
determine it.

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

practically, is this change really necessary? is env guaranteed to always be in /bin/usr/env?
I've only heard of /bin/usr/env being used in the shebang when someone wants to reference the default perl or python interpreter. Bash seems pretty standard.

@davidvossel I use my bare metal host that is nixos right now, so it's needed for me. I don't know about others. (I used a fedora host before but currently traveling with my laptop; and due to multi layer virtualization that is used for our builds it's not possible to run make inside another VM).
Of course, it's helpful for a minority of developers, but I don't plan to change my host system to be able to develop for kubevirt (at least not now), so I may need to carry the patch locally if it's not merged.

As to whether env is always in /usr/bin, I've never seen a system where it's not, it's pretty standard approach when you want to be cross platform to rely on it.

Surprising to see a technically legit PR being struck down this way, even if it addresses an issue for a minority, but ok.

@booxter this change is fine. if you feel like rebasing and updating, i'm fine with ack/merging it.


var

Hello! And have a look at this CLI tool made of Deno

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.17 out of 18 committers have signed the CLA.:white_check_mark: ry:white_check_mark: crowlKats:white_check_mark: denobot:white_check_mark: jasonjgardner:white_check_mark: lino-levan:white_check_mark: vgarciaf:white_check_mark: xnuk:white_check_mark: lambtron:white_check_mark: MariusVatasoiu:white_check_mark: kt3k:white_check_mark: bastilian:white_check_mark: roj1512:white_check_mark: bradenmacdonald:white_check_mark: digitaldesigndj:white_check_mark: GJZwiers:white_check_mark: lucacasonato:white_check_mark: SpencerIsGiddy:x: denyzhirkovYou have signed the CLA already but the status is still pending? Let us recheck it.


Whitelist wildcard domains

Summary
It would be great to be able to whitelist sites by wildcard. For example, if I want to whitelist baz.foo.com and bar.foo.com, I should be able to whitelist *.foo.com.
Motivation
This would be particularly useful for the slew of work websites. AdBlock has this feature.

@JohnLKramer thanks for the request! Agree this would be a nice feature, no reason why we can't implement it.


STATION_MODE doesn't work after NULL_MODE

I stumble upon weird behaviour. If I set wifi mode to NULL_MODE the station mode will not work no mater what I do.
Station mode will also not work if NULL_MODE is remembered from the previous life cycle and not updated in user_init.
I tried the same trick with SOFTAP_MODE but it works. NULL_MODE doesn't seem to influence SoftAP mode.
void task(void *pvParameters)
{
    while (1) {
        sdk_wifi_set_opmode(STATION_MODE);
        sdk_wifi_station_set_config(&config);

        // At this point WiFi connection works

        vTaskDelay(10000 / portTICK_RATE_MS);

        sdk_wifi_set_opmode(NULL_MODE);

        // After this point WiFi connection will not work

        vTaskDelay(10000 / portTICK_RATE_MS);
    }
}

void user_init(void)
{
    uart_set_baud(0, 115200);
    // WiFi mode must be not NULL_MODE in user_init for station to work at all
    sdk_wifi_set_opmode(STATION_MODE);

    xTaskCreate(task, (signed char *)"task", 1024, NULL, 2, NULL);
}

I figured out why I can't connect to an AP after NULL_MODE :)
I just have to manually call sdk_wifi_station_connect();
The examples that demonstrate WiFi station mode doesn't include this call because the start-up code calls sdk_wifi_station_connect(); itself right after user_init if the mode is STATION_MODE.


loadUiType usage

Hey,
I'm going over some code that we are converting and it uses loadUiType instead of loadUi.
In the examples folder there is some code to implement a load_ui_type wrapper it looks  like.
Do you have any plans for that code to make it's way into the core Qt.py module?

Do you have any plans for that code to make it's way into the core Qt.py module?

We already support one way of importing .ui files which doesn't require uic or pysideuic; QtCompat.loadUi. We'd like to steer users in this direction since e.g. older Nuke versions doesn't come with pysideuic. I'd be hesitant to implement loadUiType support, to be honest, and I would strongly suggest trying to use QtCompat.loadUi instead, if possible.
@Ahuge However, I'd be happy to know more if you are unable to use QtCompat.loadUi for some reason.
Those examples you found are "leftovers" from implementing Qt.load_ui (now QtCompat.loadUi). Initially, we didn't want to maintain such code but still wanted to provide some alternatives to users who wanted to load .ui files, and so we kept them in the "examples" folder.
I see we should update the README for loadUi:

mapped to a convenience function in Qt.py called loadUi (part of the QtCompat module; QtCompat.loadUi). This function only accepts the filename argument of the .ui file.

This is no longer true, as it does indeed support a baseinstance argument as well.

IMHO I'm trying to steer people away from loadUiType towards simply loadUi because it's a lot cleaner to use and implement.
loadUi unfortunately has a slightly higher overhead though if you're instantiating many ui objects because it's loaded multiple times whereas loadUiType can be loaded once in a module and reused.
I'm not sure how many people are really using that to their advantage though because I have yet to encounter it.

Yeah I did some looking into it for our usage cases and we can just use loadUi.
Thanks for the explanation

Cool. I'll close this then.


Edit schedules in website instead of in Google sheets

custom table editor
transfer data via API
remove code to request a link to Sheet

#4 fixed

reading and writing schedules using the API now works
still need to add a "recover" button for if you delete your schedule on accident

accounts are now JSON
only need to check API for 'recover' and 'delete' actions and make the buttons functional for them

API and buttons fixed

still need to add time editor

update released - only small updates required now


MQTT Publishing Callback not being called

Tested with demo sketch on Teensy 3.1, using hardware Serial1 to communicate with ESP.
Other callbacks (wifiCb, connectedCb, disconnectedCb and dataCb) seem to be working fine.

It seems that this feature is not implemented yet: ELClientMqtt.h#L30


Bm link fix

Changelog

Move "credentials for submission" to Access and rename it to "access key generation"
Set up an additional re-direct, update old redirect

 Submission access key documentation rename


Use logger from loguru

Could use the logger from loguru rather than importing the root logging (which we need to configure further). The default logger from loguru is already formatted to show timestamp, originating line number etc.
Also get rid of print lines and use logging instead.

Will also get rid of import * from the codes. Usually it's better to import exactly what we need.


Tests fail when upgrading from ruff 0.4.8 to 0.4.10

Attempting to package ruff 0.4.9 and later for Gentoo, I hit the following test failures.
    Finished release [optimized] target(s) in 19m 03s
     Running `/var/tmp/portage/dev-util/ruff-0.4.10/work/ruff-0.4.10/target/release/deps/red_knot-60a25f0ffe9edd60`

running 75 tests
test files::tests::four_files ... ok
test files::tests::insert_path_twice_same_id ... ok
test module::tests::first_party_module ... ok
test files::tests::insert_different_paths_different_ids ... ok
test module::tests::module_search_path_priority ... ok
test module::tests::regular_package_in_namespace_package ... ok
test module::tests::package_priority_over_module ... ok
test module::tests::resolve_package ... ok
test module::tests::resolve_dependency ... ok
test module::tests::first_party_precedence_over_stdlib ... ok
test semantic::symbol_table::tests::add_child_scope_with_symbol ... ok
test semantic::symbol_table::tests::insert_different_named_symbols ... ok
test semantic::symbol_table::tests::bigger_symbol_table ... ok
test semantic::symbol_table::tests::insert_same_name_symbol_twice ... ok
test module::tests::typeshed_zip_created_at_build_time ... ok
test semantic::symbol_table::tests::scope_from_id ... ok
test semantic::symbol_table::tests::symbol_from_id ... ok
test semantic::tests::assign ... ok
test semantic::tests::annotation_only ... ok
test semantic::tests::class_scope ... ok
test semantic::tests::empty ... ok
test semantic::tests::dupes ... ok
test semantic::tests::func_scope ... ok
test semantic::tests::expression_scope ... FAILED
test semantic::tests::generic_class ... ok
test semantic::tests::generic_func ... ok
test semantic::tests::import ... ok
test semantic::tests::import_as ... ok
test semantic::tests::import_from ... ok
test semantic::tests::import_sub ... ok
test semantic::tests::reachability_trivial ... FAILED
test semantic::tests::simple ... ok
test module::tests::namespace_package ... ok
test module::tests::stdlib ... ok
test semantic::types::infer::tests::if_elif ... FAILED
test module::tests::symlink ... ok
test semantic::types::infer::tests::follow_import_to_class ... FAILED
test semantic::types::infer::tests::if_elif_else ... FAILED
test semantic::types::infer::tests::ifexpr_nested ... ok
test module::tests::typing_stub_over_module ... ok
test semantic::types::infer::tests::ifexpr ... ok
test module::tests::sub_packages ... ok
test semantic::types::infer::tests::ifexpr_walrus ... FAILED
test semantic::types::infer::tests::join_paths ... FAILED
test semantic::types::infer::tests::ifexpr_walrus_2 ... FAILED
test semantic::types::infer::tests::literal_int_arithmetic ... FAILED
test semantic::types::infer::tests::narrow_none ... FAILED
test semantic::types::infer::tests::none ... ok
test semantic::types::infer::tests::maybe_unbound ... FAILED
test semantic::types::tests::add_class ... ok
test semantic::types::tests::add_function ... ok
test semantic::types::tests::add_intersection ... ok
test semantic::types::tests::add_union ... ok
test semantic::types::tests::flatten_intersection_one_negative_element ... ok
test semantic::types::tests::flatten_intersection_one_positive_element ... ok
test semantic::types::tests::flatten_intersection_zero_elements ... ok
test semantic::types::tests::flatten_nested_intersection ... ok
test semantic::types::tests::flatten_nested_union ... ok
test semantic::types::tests::flatten_union_one_element ... ok
test semantic::types::tests::flatten_union_zero_elements ... ok
test semantic::types::infer::tests::resolve_visible_def ... FAILED
test semantic::types::infer::tests::resolve_base_class_by_name ... FAILED
test semantic::types::infer::tests::resolve_union ... ok
test semantic::types::infer::tests::resolve_module_member ... FAILED
test semantic::types::infer::tests::resolve_method ... ok
test typeshed_versions::tests::invalid_typeshed_versions_bad_colon_number ... ok
test typeshed_versions::tests::invalid_typeshed_versions_bad_hyphen_number ... ok
test semantic::types::infer::tests::walrus ... ok
test semantic::types::infer::tests::resolve_literal ... ok
test typeshed_versions::tests::can_parse_vendored_versions_file ... ok
test typeshed_versions::tests::invalid_typeshed_versions_bad_period_number ... ok
test typeshed_versions::tests::invalid_typeshed_versions_non_identifier_modules ... ok
test typeshed_versions::tests::invalid_typeshed_versions_non_digits ... ok
test typeshed_versions::tests::invalid_huge_versions_file ... ok
test typeshed_versions::tests::can_parse_mock_versions_file ... ok

failures:

---- semantic::tests::expression_scope stdout ----
thread 'semantic::tests::expression_scope' panicked at crates/red_knot/src/semantic/symbol_table.rs:243:31:
index out of bounds: the len is 0 but the index is 1
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- semantic::tests::reachability_trivial stdout ----
thread 'semantic::tests::reachability_trivial' panicked at crates/red_knot/src/semantic/flow_graph.rs:59:28:
index out of bounds: the len is 0 but the index is 2

---- semantic::types::infer::tests::if_elif stdout ----
thread 'semantic::types::infer::tests::if_elif' panicked at crates/red_knot/src/semantic/flow_graph.rs:59:28:
index out of bounds: the len is 0 but the index is 11

---- semantic::types::infer::tests::follow_import_to_class stdout ----
thread 'semantic::types::infer::tests::follow_import_to_class' panicked at crates/red_knot/src/semantic/flow_graph.rs:59:28:
index out of bounds: the len is 0 but the index is 0

---- semantic::types::infer::tests::if_elif_else stdout ----
thread 'semantic::types::infer::tests::if_elif_else' panicked at crates/red_knot/src/semantic/flow_graph.rs:59:28:
index out of bounds: the len is 0 but the index is 17

---- semantic::types::infer::tests::ifexpr_walrus stdout ----
thread 'semantic::types::infer::tests::ifexpr_walrus' panicked at crates/red_knot/src/semantic/flow_graph.rs:59:28:
index out of bounds: the len is 0 but the index is 12

---- semantic::types::infer::tests::join_paths stdout ----
thread 'semantic::types::infer::tests::join_paths' panicked at crates/red_knot/src/semantic/flow_graph.rs:59:28:
index out of bounds: the len is 0 but the index is 7

---- semantic::types::infer::tests::ifexpr_walrus_2 stdout ----
thread 'semantic::types::infer::tests::ifexpr_walrus_2' panicked at crates/red_knot/src/semantic/flow_graph.rs:59:28:
index out of bounds: the len is 0 but the index is 10

---- semantic::types::infer::tests::literal_int_arithmetic stdout ----
thread 'semantic::types::infer::tests::literal_int_arithmetic' panicked at crates/red_knot/src/semantic/flow_graph.rs:59:28:
index out of bounds: the len is 0 but the index is 5

---- semantic::types::infer::tests::narrow_none stdout ----
thread 'semantic::types::infer::tests::narrow_none' panicked at crates/red_knot/src/semantic/flow_graph.rs:59:28:
index out of bounds: the len is 0 but the index is 12

---- semantic::types::infer::tests::maybe_unbound stdout ----
thread 'semantic::types::infer::tests::maybe_unbound' panicked at crates/red_knot/src/semantic/flow_graph.rs:59:28:
index out of bounds: the len is 0 but the index is 3

---- semantic::types::infer::tests::resolve_visible_def stdout ----
thread 'semantic::types::infer::tests::resolve_visible_def' panicked at crates/red_knot/src/semantic/flow_graph.rs:59:28:
index out of bounds: the len is 0 but the index is 4

---- semantic::types::infer::tests::resolve_base_class_by_name stdout ----
thread 'semantic::types::infer::tests::resolve_base_class_by_name' panicked at crates/red_knot/src/semantic/flow_graph.rs:59:28:
index out of bounds: the len is 0 but the index is 0

---- semantic::types::infer::tests::resolve_module_member stdout ----
thread 'semantic::types::infer::tests::resolve_module_member' panicked at crates/red_knot/src/semantic/flow_graph.rs:59:28:
index out of bounds: the len is 0 but the index is 1

failures:
    semantic::tests::expression_scope
    semantic::tests::reachability_trivial
    semantic::types::infer::tests::follow_import_to_class
    semantic::types::infer::tests::if_elif
    semantic::types::infer::tests::if_elif_else
    semantic::types::infer::tests::ifexpr_walrus
    semantic::types::infer::tests::ifexpr_walrus_2
    semantic::types::infer::tests::join_paths
    semantic::types::infer::tests::literal_int_arithmetic
    semantic::types::infer::tests::maybe_unbound
    semantic::types::infer::tests::narrow_none
    semantic::types::infer::tests::resolve_base_class_by_name
    semantic::types::infer::tests::resolve_module_member
    semantic::types::infer::tests::resolve_visible_def

test result: FAILED. 61 passed; 14 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.09s

error: test failed, to rerun pass `-p red_knot --lib`
 * ERROR: dev-util/ruff-0.4.10::gentoo failed (test phase):
 *   cargo test failed

$ rustc --version
rustc 1.77.1 (7cf61ebde 2024-03-27) (gentoo)

Oh, we got reports for this earlier but we haven't been able to reproduce but I'm able to reproduce now thanks to you sharing the build command. This is a problem with running tests in release builds!
The good news (for you) is that the red_knot code isn't part of ruff. So you can safely ignore these tests for now.

Thanks. I backported the fix as I wasn't sure how to guarantee all tests for crates that get used in the final binary are run.


Ability to customise log behaviour

I think that we need ability to customise log behaviour, for example to turn off logs.
The best solution will be ability to set custom logger class.

NodeKit


Proxima client authentication performance update

Proxima should only check basic users against basic creds, likewise for digest and jwks.

Complete


fix: improve the detection and loading of default certificates

Follow up PR of https://github.com/IBM/python-sdk-core/pull/196. Turned out in some cases - especially in containers - the certificate verification is still failing due to the missing certs. It's because the location of those files really depend on the system and the OpenSSL configuration.
This PR adds a workaround for this problem, by loading the default certs from the location that the certifi package reports. The requests package uses the same logic, so it should cause no issues.

That's a really good point! About the requirements, I did not add the certifi package on purpose, but now it doesn't matter anymore as we've stopped referencing to that.

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


Update build plugins and cds-services / cds-dk

Successfully tested: https://cdsruntime-pg-test-pg-test-bookshop-pg-app.cfapps.us31.hana.ondemand.com/app/fiori.html


优化多注册中心动态切换serverAddr配置还是保留旧注册中心地址

Describe what this PR does / why we need it
我们的场景是在多注册中心不重启实例的情况下动态切换注册中心。
Does this pull request fix one issue?
NONE
Describe how you did it
实例已经在线的情况下，通过post /actuator/env 或由配置中心更改了${spring.cloud.nacos.discovery.server-addr}配置,NacosDiscoveryProperties Bean重载完static NamingService还是保留旧的注册中心地址。导致无法注册到新的注册中心
Describe how to verify it
通过post /actuator/env 或由配置中心更改了 查看注册中心实例是否注册
Special notes for reviews

 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<EMAIL_ADDRESS>seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

你改了项目的代码风格。。

你改了项目的代码风格。。

感谢提醒！已经调整。


Codabench login

Could not find a support link, hence raising an issue here, please pardon if I have missed relevant information. I signed up for Codabench a couple of days back but have not activated the account.
Now, when I try to log in, it says to activate account with given link. I tried doing, but the activation link has expired. If I try to sign up again, it says the account already exists with the email ID. I tried a forgot password, but since my account is not activated, the resent link is not sent to my email ID.
How should I resolve it?
Thanks!

Please send an email to<EMAIL_ADDRESS>Send from the email you have used to create account on Codabench.

I close this issue as its resolution will continue by email.


Eco mode property not set

I have a Daikin Altherma 8 MT F, indoor model ETVH12S18EJ9W. I use the ALTHERMA(BIZONE_CB_04-08KW).h setup.
When comfort mode is selected, the Storage comfort property goes properly to ON. When mode switches to Eco, however, Storage eco property stays OFF.
{0x62,2,306,1,-1,"Storage eco ON/OFF"}
{0x62,2,305,1,-1,"Storage comfort ON/OFF"}

Any clue where this could be rooted?

The only thing I can think of is the combination of the setup file and your heat pump?  I'd say have a look in other setup files for the storage eco address that isn't 306 as above and maybe edit this new address into your current code?  Not sure tbh but that's the first thing that springs to my mind.

It's 306 for all configurations. Also strange that it works for comfort, but doesn't for eco...


pwm/pwm_da1469x: Preserve timer state on frequency change.

Read the current timer state (timer control reg) and preserve its running state when setting frequency. The only bits that should be set in the control register are the bits to enable the timer in the proper mode and enable the clock.
The rational behind this change is that some users may want to be able to change the frequency "on the fly" without having to diable pwm, change frequency and then re-enable pwm. I do realize that with this chip if you change frequency the duty cycle changes so changing frequency on the fly also changes the duty cycle but this was always the case.
Note that this should not change the previous functionality (other than adding some extra code to read the control register and mask out the  SYS_CLK_EN bit. This change could have been done using a syscfg value to preserve the old code and make this new code present only present if the syscfg was set but it seems unnecessary. However, if folks think there should be a syscfg to make this behavior optional that would be fine.

another change in that should not be.


Sorting of needs and connections

It would be nice if the node would add aggregation-infos to the connection containers, e.g. "timestamp of latest event of type X", that together with the list of unread-events allow sorting the feed-entries without fetching every single event.
The same applies for needs and fetching connections. However then the owner would also need to be able to persist and serve information about unread events per connection.
Particullarly when using paging and on the feed this is of critical importance as it determines which entries people get to see at all.

connected to #723

The part about the unread events is postponed.


timer.h: Add handy timed_thread_wedge()

It's for helping with benchmarks where you want to see how something scales
with number of threads.
Example output:
threads    time   speedup  efficient  its/thread   range (best of 1)
   1        2.3     1.00x    100.00%   20000000     0.00
   2        1.2     1.97x     98.50%   10000000     0.00
   4        0.7     3.59x     89.86%    5000000     0.00

Update after @fpsunflower warned me against the overhead of std::function<>, which I was not previously aware of. Some google searching turned up a discussion and implementation of function_view, which is a much lighter-weight way to encapsulate a callable function-like object to pass around (but with the limitation of being non-owning, so just like a string_view or array_view, it is critical to make sure the view's lifetime does not exceed that of the functor it wraps).
So the review is now amended to include function_view.h (which I suspect we'll find more uses for), and then the timed_thread_wedge is changed to pass function_view's around rather than std::function.

LGTM
Did you measure any performance difference over the std::function implementation?

I didn't, specifically. I took it for granted that, by being the thinnest possible wrapper around a function pointer, it would have the same perf as func ptr, and that std::function<> couldn't possibly be faster and might be slower, as is widely documented. It also aligns well with my current tendency toward using string_view and array_view more, feels like the analogous thing for function objects.


xtensa: fix issues with userspace

(see individual commits for more details)

Make arch_user_string_nlen() work with invalid pointers.
Stash EPC, DEPC and EXCCAUSE correctly for later processing as any TLB misses will override these registers.
Have a "semantic" triple faults to avoid bouncing between kernel/user exception and double exception indefinitely.
Other misc fixes.

Compliance check failures are false positive on assembly code with jump and labels.

@nordicjm @nashif This PR increased ram usage in a few bytes, causing mcuboot to fail in esp32 and esp32s3 SoCs. Is CI testing mcuboot build somehow?

1KB is quite a lot. Could you compare and see where are the increases?

This is the output for west build -b esp32s3_devkitm/esp32s3/procpu ../bootloader/mcuboot/boot/zephyr:
Attention to the fact that mcuboot linker moves all ROM to IRAM/DRAM, so the increased value is converted to "iram/dram" segments in the following table:

Before this PR:

Memory region         Used Size  Region Size  %age Used
        iram_seg:       38788 B        40 KB     94.70%
 iram_loader_seg:        6264 B       6656 B     94.11%
        dram_seg:       24928 B        25 KB     97.38%
        IDT_LIST:          0 GB         8 KB      0.00%

After this PR (I had to increase ram/iram to make it build)

Memory region         Used Size  Region Size  %age Used
        iram_seg:       38960 B        40 KB     95.12%
 iram_loader_seg:        6264 B         7 KB     87.39%
        dram_seg:       26984 B      27392 B     98.51%
        IDT_LIST:          0 GB         8 KB      0.00%

So the diff is 2056 bytes in dram_seg (which is probably .text or .rodata).
Anyway, we have increased the sizes in #74687.

Hm... weird... I can understand the code size changes... but this PR should not increase data size that much...

Hm... weird... I can understand the code size changes... but this PR should not increase data size that much...

Perhaps something in esp32 linker file might have this side effect. We will check later.


FPS Control Plugin can cause the Runelite sidebar to become unresponsive

Describe the bug
When using the FPS Control plugin to limit the unfocused FPS it will immediately reduce the Runelite client refreshrate when clicking on the sidebar outside of the osrs game window. With a low enough unfocused FPS target this will cause the runelite sidebar to be poorly responsive as it will stutter on all input.
To Reproduce
Steps to reproduce the behavior:

Go to the FPS Control plugin settings menu
Set the Unfocused FPS target to 5 and enable the unfocused FPS target limit
Apply the settings
Click in the game window to get the regular FPS target
Click on the RL sidebar and try to search for settings by typing or interact with it in other ways. It is especially apparent when you try to highlight your search string with the mouse.
Inputs will be missed and the experience is generally poor

Expected behavior
The osrs game client window will be unfocused and capped but the RL client window should still refresh at an independent rate.
Environment:

OS: [macOS] (M1)
RuneLite version: [1.18.10]
Launcher version: [2.3.0]
GPU plugin enabled. It seems like this issue isn't present with the GPU plugin disabled.

We have recently replaced the GL library used by the GPU plugin. Given that change and that we are not able to reproduce this issue, we are closing this. Please comment if you still experience this issue.


Fix failing partial-no-import test

This fixes a test that is currently failing on master branch.
ping @dryoma

Hmm, that's strange. Everything is ok on my side. Maybe it has something to do with whether it's windows/linux and the slashes used?

Hmm, that's strange. Everything is ok on my side. Maybe it has something to do with whether it's windows/linux and the slashes used (I use Windows)?

That might be, I'm using osx and it is also failing on my machine.

The problem was in hard-coded path slashes I used to test dirs starting with _. I'm afraid if (/\\/.test(fileName)) { return } won't help here.
I pushed a fix in #73

ok, thanks! Let's close this then.


Question: How can I check the mount point for dvdcli in mountinfo

When I run the command "dvdcli mount", I can see that I can mount successfully, but the problem is how can I check the mount point via namespace?
I was using convoy as backend.
root@mesos002:~# convoy list
{}
root@mesos002:~# dvdcli mount --volumedriver=convoy --volumename=dvd-mesos
INFO[0000] /root/nfs-vol1/dvd-mesos
/root/nfs-vol1/dvd-mesos
root@mesos002:~# cat /proc/self/mountinfo  | grep dvd-mesos
root@mesos002:~#

Why did there is no mount point in my host mountinfo? How can I check this mount point in mount namespace? Thanks!

@jay-lau Have you tried the dvdcli path --volumedriver=convoy --volumename=dvd-mesos command?
I typically look to cat /proc/mounts to view the mounts which should include the one listed as long as you have access to the PID1 mount namespace.

@clintonskitson Yes, I know that I can use dvdcli path --volumedriver=convoy --volumename=dvd-mesos, but what I want to do is I want to see how can I check this via mount info, such as mounts, mountinfo etc, but neither of them can show me the info, do you know why?
root@mesos002:~# cat /proc/mounts | grep dvd-mesos
root@mesos002:~# cat /proc/self/mountinfo | grep dvd-mesos
root@mesos002:~#

It is possible that the mount namespace that convoy is using isn't the same as what you are trying to reference here. Can you try doing a docker exec containerpid cat /proc/mounts from the convoy container?

Thanks @clintonskitson , now two questions want to get your help:

root@mesos002:/tmp# docker exec  ebadbe5114c9 cat /proc/mounts |grep dvd-mesos
root@mesos002:/tmp# docker exec  ebadbe5114c9 cat /proc/self/mountinfo |grep dvd-mesos
107 89 8:1 /var/lib/docker/volumes/dvd-mesos/_data /tmp rw,relatime - ext4 /dev/disk/by-uuid/017a354d-62b9-4ca2-aca2-12df8da6c82e rw,errors=remount-ro,data=ordered

From above, you can see that I can see the mount point from /proc/self/mountinfo but not /proc/mounts, do you know why?

But how can I check the mountpoint if I did not create a container? If I do not create a container using this volume, but run convoy delete dvd-mesos, it will be failed with following errors. I know that the mountpoint was in use, but I do not know in which namespace can I find the mountinfo for the volume of dvd-mesos, any comments on this? Thanks!

root@mesos002:/proc/3069# convoy delete dvd-mesos
ERRO[0000] Error response from server, Cannot delete volume 8b416bd0-4df8-444f-a56b-73b089bd6081. It is still mounted

{
	"Error": "Error response from server, Cannot delete volume 8b416bd0-4df8-444f-a56b-73b089bd6081. It is still mounted\n"
}

If you look at the mapped volumes, I am assuming the listing you are getting is from the PID1 mount space.

You could traverse the namespace's mountinfo files and find which owned the volume?

awk '$9 == "proc" {print FILENAME,$0}' /proc/*/task/*/mountinfo | sort -k2 -u

Thanks @clintonskitson , for 1), can you please show me some command to clarify?
For 2), not quite catch your  point, in which namespace shall I find? Because there is no container now.

docker inspect containerid and find the bind/volumes to see if the /proc folder is coming in.

On the second, the command that I sent will list the mountinfo files for each container. If a container has a volume, you could review it there.

@jay-lau Can you close this if satisfied?

Thanks @clintonskitson!


Delete obsolete option.

Delete --self option from update command.

TODO

[ ] README.md
[ ] doc/guide/ja/README.md
[ ] doc/zplug.txt
[ ] doc/txt/zplug-update.txt


Again

Worked out the bugs and finished the new cool Ocean Menu.   and Images to match queries.  Works fine down to 290px.  Needs to be test on mobile now live to work out any remaining quirks before going forward with more advanced things.

dito


How to get this working in a devcontainer?

Thanks for making this! Works nice locally, but I got into trouble making this work in a devcontainer because of bat
Using something like:
 "postCreateCommand": "sudo apt-get update && sudo apt-get install -y bat fzf ripgrep",

in the devcontainer.json will not work, although it does make bat usable as a cli tool.
Making the symbolic link also does not help here:
  "postCreateCommand": "sudo apt-get update && sudo apt-get install -y bat fzf ripgrep && mkdir -p ~/.local/bin && ln -s /usr/bin/batcat ~/.local/bin/bat"

This wsl fix also did not do it for me.
The error is always: Failed to active the plugin , bat not found on your PATH.
Any ideas?

I have zero experience with devcontainers so I can't contribute much here. I think it'd be good to know that the extension really just runs in a fairly vanilla terminal within VS Code, so if you can figure out more about that environment (e.g. PATH, etc.) you might be able to figure out why it's not working.
Also, here's a post I've linked in related issues before regarding how to set additional env vars: https://stackoverflow.com/a/45637716/888916, that should help correct any e.g. PATH issues to help get things running.
Hope this helps!


request for weight files

@shraddhavijay
First, wonderful project and paper. I have manage to run the web application , but the weight model is missing.
Can the author kindly grant me request for access to image and video weight google drive.
My gmail<EMAIL_ADDRESS>Please grant me access in
https://drive.google.com/drive/folders/1B4ODeK_QQ6XMFo6i6EEup1nZC6PllVfu?usp=sharing
https://drive.google.com/drive/folders/1irYZbRnr4Y7jKieSyhjxHxwk43oSMqh-?usp=sharing
I'm doing for my school research purpose, it would be super helpful.
Thank you guys for this amazing project.

@shraddhavijay thank you very much , i have accessed the folders.

@shraddhavijay
Great project and paper! I've managed to run the web application, but the weight model is missing. Could you please give me access to the image and video weight files on Google Drive?
My gmail<EMAIL_ADDRESS>Please grant me access in
https://drive.google.com/drive/folders/1B4ODeK_QQ6XMFo6i6EEup1nZC6PllVfu?usp=sharing
https://drive.google.com/drive/folders/1irYZbRnr4Y7jKieSyhjxHxwk43oSMqh-?usp=sharing
I'm doing this for my personal and school. It would be really helpful.


[Docu] get docker ip address

$ ifconfig docker0 | grep 'inet' | cut -d: -f2 | awk '{ print $1}' | head -n1

With this you will get only the IP-Address without any other informations.

@Gummibeer fantastic 👍
Would you like to send a PR updating the readme, or I do it myself?

@Mahmoudz PR is done - check if it is ok for you. ;)

@Gummibeer Thank u.
Closed in #68


BERT embeddings

Hi,
Reading your paper I noticed you used BERT embedding in it and (I suppose) because of that, got better results, especially on the ,multilingual settings. Do you have any plan to incorporate the usage of BERT embeddings to the code in this repository?
Thanks

The old version of the code uses tensorflow bert (https://github.com/google-research/bert) to extract fixed bert embeddings and save them to the disk, which makes running inconvenient. So I am not planning to release the code. But I will release a version of incorporating the usage of pytorch bert (https://github.com/huggingface/pytorch-pretrained-BERT)  in the near future.


Rmc jij cycle fix

for #172

maybe change the name of context to something descriptive ... perhaps job_names?
Also are there unit tests for this function?

The context is actually in some sense more descriptive than "job names" in regards to the expansion process as a whole, as it describes the context under which the current job is expanding- ie the jobs it has already expanded from, which allows it to detect if a cycle would occur.

not that I'd be opposed to a name change, but I think job_names is a bit underdescriptive, but not sure what else would make sense and also be a succinct enough variable, ie jobs_expanded_from is a bit verbose, I think

How about ancestor_jobs?

I say :+1: to that. for it being descriptive and succinct. I'll work on a separate PR for that.

Gonna add a unit test for this case in the meantime

Unit test added!

works for me then

Looks good to me. :+1:

looks good


Addition of Graph Algorithms.

There is no such folder for Graph related algos.
Please assign this to me.
@pankaj-bind

@Shariq2003 ; there is a folder for Graph related algos. Look at it again


init new codebase

created a new folder for the new codebase
started with creating dataclasses for the grid environment
the main code that connects everything in the end is currently in activelearning.ipynb

adapted the code according to your comments.
added classes for datahandler + branin specific datahandler.
regarding the jupyter notebook, I am using it to test the implementation and find it far more handy than using a script for development purposes...


[3.x] hessian upgrade infactor

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Environment

Dubbo version: 3.0.12
Operating System version: Mac OS
Java version: JDK8

Steps to reproduce this issue

POJO which name com.xxx.dto.response.RowResponse without implement java.io.Serializable
invoke provider which return this POJO
get warning log with  below message and invoke result return HashMap class type.

com.alibaba.com.caucho.hessian.io.SerializerFactory.getDeserializer 679: Hessian/Burlap: 'com.xxx.dto.response.RowResponse' is an unknown class in sun.misc.Launcher$AppClassLoader@18b4aac2:
java.lang.IllegalStateException: Serialized class com.xxx.dto.response.RowResponse must implement java.io.Serializable

Pls. provide [GitHub address] to reproduce this issue.
Expected Behavior
invoke result return type is com.xxx.dto.response.RowResponse
Actual Behavior
return HashMap
Cause By
this problem cause by hessian version upgrade from 3.2.12 to 3.2.13.
this is the change log link: https://github.com/apache/dubbo-hessian-lite/compare/v3.2.12...v3.2.13
new version hessian add limitation for deserializer method, it require POJO implement java.io.Serializable
it seems not reasonable for this block upgrade, can we remove this limitation?

Object pass in Dubbo should implement java.io.Serializable interface. This should be a programming specification, and this can prevent send unexpected object to remote, which may cause a lot of serious leaks.
If you want to send the object, which not implement java.io.Serializable interface, please add -Ddubbo.hessian.allowNonSerializable=true to jvm arguments.


URL change FlowerManga.com

Source information
FlowerManga.com 1.4.36
Source language
Portuguese Brazilian
Source new URL
https://flowermangas.com
Other details
No response
Acknowledgements

[X] I have updated all installed extensions.
[X] I have opened WebView and checked that the source URL has not changed yet.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I will fill out all of the requested information in this form.

The issue closer mistakenly thought the name of the extension (FlowerManga.com) was the new URL.


Missing ADA

Hi guys,
Please can you help me with an issue on Nami? It appears in my wallet that i have 301 ADA in my wallet when I should have 3191 ADA.
In transaction ID 30fb2777a6816566798628992441e5a26c68ab92f1e69e62f865fb480dbafaf8, I sent 2000 ADA to my other sub-wallet in Nami but it appears that another 2869 ADA has gone. It shows in my "http://pool.pm" that the ADA is still in my wallet but I am unable to see it in Nami?
(https://pool.pm/addr1q9szxdf0neuz8zvhsxyuuyj63r4y08jvfwtph57a937l2s7meczvtpxskk3hg2ra8r3ak546g2rc3gfa0xqhe03fl9hsk65m9g)
Please help :)

You probably have done transactions with another wallet app than Nami.
All other wallet apps use a lot of addresses per wallet, while Nami only sees and uses the first one. Transactions in other wallet apps move things away from where Nami can see them. But it should show the real balance in a pop-up at the small “i” next to the balance.
You can move everything back to that first address (the one Nami shows in “Receive”) with a transaction in the other wallet app.
ccvault has a “Single address mode” in the settings to prevent this moving away and cooperate nicely with Nami.
(Also try “Collateral” on the same page if you set a collateral in Nami. But do not set it in ccvault. ccvault will recognise the collateral UTxO done by Nami, but Nami will not if ccvault does it. Both knowing it should prevent ccvault from using the collateral UTxO for normal transactions. Otherwise, Nami would prompt to set it again and again costing transaction fees.)

Yes! this sorted it, sent all ADA to my OG address and it's fixed, thank you!

From: Benjamin Braatz @.>
Sent: 22 January 2022 22:55
To: Berry-Pool/nami-wallet @.>
Cc: Dineen94 @.>; Author @.>
Subject: Re: [Berry-Pool/nami-wallet] Missing ADA (Issue #282)
You probably have done transactions with another wallet app than Nami.
All other wallet apps use a lot of addresses per wallet, while Nami only sees and uses the first one. Transactions in other wallet apps move things away from where Nami can see them. But it should show the real balance in a pop-up at the small “i” next to the balance.
You can move everything back to that first address (the one Nami shows in “Receive”) with a transaction in the other wallet app.
ccvault has a “Single address mode” in the settings to prevent this moving away and cooperate nicely with Nami.
(Also try “Collateral” on the same page if you set a collateral in Nami. But do not set it in ccvault. ccvault will recognise the collateral UTxO done by Nami, but Nami will not if ccvault does it. Both knowing it should prevent ccvault from using the collateral UTxO for normal transactions. Otherwise, Nami would prompt to set it again and again costing transaction fees.)
—
Reply to this email directly, view it on GitHubhttps://github.com/Berry-Pool/nami-wallet/issues/282#issuecomment-1019372477, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXNITDRCW6SVR3HSYCXXJNLUXMYXDANCNFSM5MRTMBAA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.***>

That's good to hear!
Would you close the issue, so that the Nami developers have less to go through, when they return here?