Fix - Login field is focused when sign out in small screen devices

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/23797
PROPOSAL: https://github.com/Expensify/App/issues/23797#issuecomment-1655938492
Tests

On Web/Desktop

Login to any account
Click Sign out
Verify that the login input is focused
Enter any email -> click Continue to go to next step
Click on Go back
Verify that the login input is focused

On mWeb/Native apps

Login to any account
Tap Sign out
Verify that the login input is NOT focused
Enter any email -> Tap Continue to go to next step
Tap on Go back
Verify that the login input is focused

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://github.com/Expensify/App/assets/9639873/517151f3-39d9-4a0b-b5f6-12e82ae30db5

Mobile Web - Chrome

https://github.com/Expensify/App/assets/9639873/5c9784ba-a913-4c49-a154-f6797ab5b550

Mobile Web - Safari

https://github.com/Expensify/App/assets/9639873/a6254ba0-949a-41f3-b0cc-613c9eb410be

Desktop

https://github.com/Expensify/App/assets/9639873/28c0d3fb-3eb7-4a8c-bf69-0b28e9057370

iOS

https://github.com/Expensify/App/assets/9639873/8c72de96-a088-490e-8ba7-8841fe0c1e88

Android

https://github.com/Expensify/App/assets/9639873/2027cfbe-23ed-48d3-82f0-181f15b9668d

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/PauloGasparSv in version: 1.3.51-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/PauloGasparSv in version: 1.3.52-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
failure ❌

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/PauloGasparSv in version: 1.3.52-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/puneetlath in version: 1.3.52-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Enable missing Java interop tests

the "advanced" tests:
custom_metadata, unimplemented_method, status_code_and_message

@carl-mastrangelo

These are now implemented in Java and enabled in run_interop_tests.py.


fix make_docs and qsub test

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
close #1711
What are the significant changes in functionality due to this change request?

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[ ] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

Job Test qsubs sawtooth on 95fb171 : invalidated by @wangcj05
retest: set python environment look much longer than usual.

@joshua-cogliati-inl Can you take a look at this PR?

Job Test qsubs sawtooth on 95fb171 : invalidated by @wangcj05
timeout for raven run raven tests

I looked at the code and I have no objections, but I don't think any of these changes could solve any problems.
(I have also been trying to debug things at #1713 )

I agree, thanks for the review. The changes in make_docs will make SAWTOOTH pass the docs check, and the other one is just to fix the typo in tests. @joshua-cogliati-inl

Job Test qsubs falcon on dfaea7d : invalidated by @wangcj05

I found the mistake that was causing it to fail (and it was my fault).  I had left a ^ in test documentation.
I added the commit to fix it to this branch.
I also added a fix that allows --verbose to work properly for the tests directory.

@joshua-cogliati-inl tests are ok now. I will ask @mandd to merge it.

Review board is green, sawtooth machine is now down due to maintenance activities.


Not writing logs to file in release mode

Code :
Future<File> _getFileName(
      {bool forCreate = true, DateTime? requiredDate}) async {
    DateTime date = requiredDate ?? DateTime.now();
    Directory directory = Directory(
        "${(await getExternalStorageDirectory())?.path}/foreground_task_logs");
    if (!directory.existsSync()) {
      if (forCreate) {
        directory.createSync();
      } else {
        throw Exception('Directory Not Created');
      }
    }
    final String fileName =
        "${directory.path}/${DateFormat("dd-MM-yyyy").format(date)}.txt";
    print("File Path = $fileName");
    File outputFile = File(fileName);
    print("File Exists = ${outputFile.existsSync()}");
    if (!outputFile.existsSync()) {
      if (forCreate) {
        outputFile.createSync();
      } else {
        throw Exception("File Not Created");
      }
    }

    return outputFile;
  }

  Future<Logger> getLogger() async {
    try {
      return Logger(
          printer: PrettyPrinter(
            methodCount: 0,
            printTime: false,
            noBoxingByDefault: true,
          ),
          output: FileOutput(file: await _getFileName()));
    } catch (e) {
      throw Exception(e);
    }
  }

Hi, it seems that you have not provided a custom filter, by default the DevelopmentFilter is used which by design doesn't log in release mode.
You can change that by providing the following to your logger:
Logger(
  filter: ProductionFilter(),
  // ...
)

Hi, it seems that you have not provided a custom filter, by default the DevelopmentFilter is used which by design doesn't log in release mode. You can change that by providing the following to your logger:
Logger(
  filter: ProductionFilter(),
  // ...
)

Even if the code is modified, the log cannot be written to the file in release mode. However, the log can be written to the file normally in debug mode.

Hi @zzdota, I tried to reproduce your problem but wasn't able to identify any issues.
I created a FileOutput with a File in the ApplicationDocumentsDirectory, used the ProductionFilter and tried it in debug and release mode, as well as in an emulator and a real device (Android). In every case, it successfully created the file with the proper log output.
Test Code:
late File logFile;

Directory applicationDir = await getApplicationDocumentsDirectory();
logFile = File("${applicationDir.path}${Platform.pathSeparator}test.log");

// ...

static final Logger log = Logger(
    filter: ProductionFilter(),
    output: !kIsWeb
        ? MultiOutput([
            Logger.defaultOutput(),
            FileOutput(file: logFile),
          ])
        : Logger.defaultOutput(),
);

First of all, I admit that you are right, flutter build --release, this way of debugging a mobile phone with a data cable is normal;
However, if you use flutter build apk to package the apk, install the apk on your phone, and then run this application, this application uses the logger library to print logs and store the logs in files, something will appear. The log is not written to the file,

Hi @zzdota, again, I tried to reproduce it and couldn't find any problem. I built the APKs (release and debug), transferred them over to the phone, installed them manually and the log file was created just fine in both modes.
Another question, as the device file explorer isn't available for release builds, how do you check if the file exists? And in which location are you trying to save it?

So, If

the device file explorer isn't available for release builds

how can you be sure that

the log file was created just fine in both modes.?

Is the device need to be root?

So, If

the device file explorer isn't available for release builds

how can you be sure that

the log file was created just fine in both modes.?

Is the device need to be root?

Hi @magic3584, my app tells me that. I just coded a simple file check that tells me if the log file exists and what its content is.
Yeah, a rooted phone would work too.

@Bungeefan
Thanks guy, I have written a demo and you are right.


fix: document creation and deletion race with parent objects

Document object in Konnect are always associated with either a service
package or a service version.

If a service version or package is deleted, deleting a document
results in a 404, an error in decK
If a document is created before the corresponding service package or
version is created, it resulted in 400, again an error

This patch adds the appropriate barriers to prevent such races.

Codecov Report

Merging #431 (225388a) into main (f09b38e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #431   +/-   ##
=======================================
  Coverage   47.10%   47.10%
=======================================
  Files          90       90
  Lines        6088     6088
=======================================
  Hits         2868     2868
  Misses       2890     2890
  Partials      330      330

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 f09b38e...225388a. Read the comment docs.

ping @rainest specifically

holding so that #435 and #436 get in.
merging this in before them will result in a lot of unnecessary merge conflict pain

#435 and #436 are now in; anything other than the conflict needed before proceeding with this?

No. Rebased and resolved the conflict.


List of (Lat,Lon) from way_id from .osm file

Hi,
Can i get a list of (lat,lon) from way_id which is again to be extracted from map.osm(say).If yes then how?I have written two methods which separately fetches way_id and (lat,lon) from node_id but their orders are different,and i want to create some sort of dictionary with way_id as key and list of (lat,lon) as value.I am stuck quite hard.Help!
Please someone reply asap.
Thanks

Hi,
I finally got to do it myself.But i had to three dictionaries to do it.If anyone has better suggestion then please reopen it.
Thanks


added kotlin

Check if gradle build works locally for you. It should.

Hi @invasionofsmallcubes, the code is ok but I prefer to have the code instead of a dependency on a submodule. Can you fix that? :innocent:
Feel free to open another PR if you like. Alternatively, I can commit a copy of your code and cancel the PR.
Let me know. :relaxed:

I'll fix it. I'll recreate the PR.


Fix h2 multiplexing race

Fixes https://github.com/istio/istio/issues/40720
Instead of later swapping the h1 server with a multiplexed server, build
the correct one the first time. This avoids the logical race in the
issue, and a legitimate data race (swap unprotected handler).
Please provide a description of this PR:

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
3e3398b19f6afd93ba0b337c2acbd92926753e7d
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Please create nice docker-compose file

Hi,
your project is really interesting but I think its not cool to clone the whole github repo to let it run with docker-compose.
Can you please provide a docker-compose file which is able to run "standalone"?
Thank you!

I'm not sure what you mean by "nice"? You can already do this:
$ wget https://raw.githubusercontent.com/Cesura/pastey/main/docker-compose.yml
$ docker-compose up -d

Updated the README to make it more clear to all:
https://github.com/Cesura/pastey/commit/7b705b5a47483db92bd0b8d0f09b98ace8264fb7


Require data packages to have a creation date

@kbubnicki this wasn't something we discussed, but @yliefting and I think it would be a good idea to require this property at package level.

Makes sense.


[shape_poly] Add a version of jax.eval_shape that works with shape polymorphism

The use would be to find the output shapes for a function in presence of shape polymorphism, and to compute the polymorphic_shapes value that can be used in a subsequent call to jax2tf.convert.

@rxsang @maxwillzq @voutcn PTAL


pythonImportsCheckPhase: run {pre,post}PythonImportsCheck

Motivation for this change
Some packages require e.g. setting environment variables in order for them to be importable.
See https://github.com/NixOS/nixpkgs/pull/120009#discussion_r617729298.
I'm sure thise will break stuff such as when preCheck uses rm without -f because it would try to remove files twice.
Hence prePythonImportsCheck might be preferred.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] 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
[ ] Fits CONTRIBUTING.md.

Is there a reason why pythonImportsCheckPhase runs before preCheck? I'd think it should be the other way around, since it's "just another check".
IMHO, a better patch would be to fix that ordering issue, rather than possible running preCheck multiple times, or adding yet another pre* phase.

IMHO, a better patch would be to fix that ordering issue, rather than possible running preCheck multiple times, or adding yet another pre* phase.

That doesn't fully solve the problem since there are packages without a checkPhase but with a pythonImportsCheckPhase.

Hm, fair.
Then I'd be more in favor of adding {pre,post}PythonImportsCheck -- running things twice seems like a recipe for disaster (or at least, a recipe for potentially-bad-things happening).

I'm sure thise will break stuff such as when preCheck uses rm without -f because it would try to remove files twice.

And we do not want to add the -f to it because then we wouldn't notice when the file is moved/gone.

@ofborg eval

Yeah I think package authors assumptions that the hooks will run in a simple linear order will trip us up here.

Honestly, I think having to have a write-able home just to import a module is just bad design. I'm not really familiar with the package in question, but just doing something like import scancode should not cause file IO in my $HOME dir. I can understand import scancode; scancode.doSomething(), but there I have an expectation that a function/method may have side effects.

@ofborg eval

@jonringer I guess you could argue that it's okay to load some configuration upon importing.
Anyway, I think there is no real disadvantage to merging this.

I have another case where this would be helpful: https://github.com/NixOS/nixpkgs/pull/121825/files#diff-1b43d6607a8882b1a7b152335be67d4c08bf3c8949f7e5b8d99d387786040a7fR31-R36

This is a useful feature that we are also missing

Do you need it for a specific package?

This is a useful feature that we are also missing

Do you need it for a specific package?

This is for experiments with ASAN in private fork, but I can see how this can help to package some broken packages that do something during import (like the one mentioned here).


⚠ Bofrost.fr has degraded performance

In f39812f, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1176 ms

Resolved: Bofrost.fr performance has improved in 128c46a after 2 hours, 18 minutes.


chore: update CHANGELOG.md

Remove bullets not related to end-user consumption of the unstructured library.

These removals make sense 👍 Klaijan on the 0.10.26 item, since it's already released we probably won't change it retroactively, but if decided otherwise that can be a separate PR

Agreed, @Klaijan , we can leave the CHANGELOG for older releases intact.


ask for train model

Hi,
Could you help to provided the model you have trained ?
We wants to do some inference performance test.
Thanks very much!

Hi @rongzha1 we don't provide pre-trained models at this time.
You can train a model yourself by following the tutorials or the scripts in the arxiv_1217 branch.


Fixed undefined variables + improved logo for darkmode github users

I added variables that were previously not defined for whatever reason and also improved the logo and changed its text color to grey instead of black like previously, so darkmode users see the text better (lightmode users will still see it because it's grey, not white).
I hope I could help :)

Oh, and in the installation -> Packages section i changed the pytube package, because pytube3 should not be used.

@namechangezero Thank you for contributing 🎉


[WIP] Add argument handler class

What?
Move argument handling and config setup out of main.py into its own class. WIP for now as there may be more changes to main.py
Why?
This makes argument handling and config creation testable and included in coverage checks.
Testing/Proof
Automated testing

oh, is this still a WIP? I approved it, but I just noticed you may want to keep working on this

oh, is this still a WIP? I approved it, but I just noticed you may want to keep working on this

Happy to go ahead with this as is, but I made it WIP for now in case we want to make more changes to main.py as discussed in #41

I rather cohesive units of change. This is good as is now. If you want to do further improvements, let's open new PRS instead


Change API URLs to avoid triggering ad blockers

As we discovered in #896, some adblocker configurations can block XHR requests made by the Conduit web UI, causing the UI to not work as expected. We should investigate how hard it would be to add adblocker detection and warn the user that the dashboard won't work with adblockers enabled.

It seems likely to me there is a way to avoid the ad blockers breaking things. It seems likely we're doing something that seems like something an ad would do, that a normal (React) app that uses XHR wouldn't do. Do we know why exactly the ad blockers block XHR requests made by the Conduit web UI? Which ad blocking rule are we triggering? Can we change our XHRs to avoid trigger that ad blocking rule? Can we submit a tweaked version of the ad blocking rule upstream?

@briansmith I believe that one of our XHR requests has a path that includes /api/stat, which apparently triggers a blocking rule in uBlock Origin. See https://github.com/runconduit/conduit/issues/896#issuecomment-390124997.

The rule comes from the EasyPrivacy list, which forbids:
/api/0/stats
/api/analytics/*
/api/log?
/api/lt/ref?
/api/pageview?
/api/pixel?
/api/stat?
/api/tracking/*
/api/v1/stat?
/api/x.gif?

@briansmith I believe that one of our XHR requests has a path that includes /api/stat, which apparently triggers a blocking rule in uBlock Origin. See #896 (comment).

Thanks I see "/api/stat?" in this blocklist: https://easylist.to/easylist/easyprivacy.txt. It seems like all we have to do is change that URL and everything will work. That seems much better than letting the ad blocker block the XHR and then reporting on it.

Some URL proposals (all of these pass the easylist at least):
/api/control-plane/stat
/api/details
/api/traffic
/control-plane/stat
/backend/stat
/api/v2/stat

My vote is /api/v2/stat. /api feels like a good base and I like keeping the routes the same as the CLI commands they're mirroring.

Given that /api/v1/stat? is in the blocklist, I would lean towards using another non-versioned namespace (since this isn't really a versioned api anyway, is it?).  Given that the protobuf api is in the conduit.public namespace, perhaps /api/public/stat?

I suggest we try to think of something more creative (unique) than "stat" to use in the URL. Any generic URL similar to the ones we've mentioned so far seem very likely to be put into add blocker blacklists in the future. I suggest "/api/v1/tps-reports".
Regarding whether or not the API is versioned: As we get closer to the 1.0 release I think we need to make sure all APIs are versioned one way or another. Whether that happens in the URL I don't know.

@briansmith any suggestions for a better named stat endpoint?

Current winner: /api/tps-report


Cant create the shortcut

Describe the bug
A clear and concise description of what the bug is.
Hi Team, I can not download the shortcut from the VBS file. WHen I double click, my mac asks me to decide which application to open the VBS file from.
Any help?
Best,
Shehroz
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Please read the Readme properly, it has mentioned that if you are a mac or linux user, then this shortcut releated process is needed to be skipped and need to follow other steps metioned in the readme.

How to run on Mac or Linux OS
https://github.com/pishangujeniya/instagram-helper/issues/22#issuecomment-774589015


Krysalisbeauty

Karma is a tale that is written by the oppressed and the sufferance of the weak and vulnerable .  Called #al


Loading image from External Storge

I'm  trying to show in my app an image loaded from the external storage, however, it is not showing up.
I have already added the permission and it is working with other photos

So, just want to be sure I have this right and I am assuming this is Android. But you:
1.) are using PickPhotoAsync()
2.) You have a photo on an external SD card
3.) You pick it from the gallery on the external SD Card
Does it return null? Or does it return a path?

Sorry, I should have been more specific
var file = await CrossMedia.Current.PickPhotoAsync(new Plugin.Media.Abstractions.PickMediaOptions()
{
CompressionQuality = 50,
});

Yes, I have a photo on my external SD card
3)I pick the photo from the gallery on the external SD card
It returns the path and bytes, however, when I set the data to be shown in my UI image control it does not appear. If i pick any other photo it works perfectly

How are you attempting to load the image into the UI?

I populate my property Image (ViewModel)
_post.Image = file.GetStream().ToByte();
In my ContentPage
image = new Image
{
HorizontalOptions = new LayoutOptions(LayoutAlignment.Center, true),
VerticalOptions = new LayoutOptions(LayoutAlignment.Center, true),
Aspect = Aspect.AspectFit,
Source = ImageSource.FromStream(() =>
new MemoryStream(AddPostViewModel.Post.Image)
)

any idea?

Haven't had any time to test this out sorry. Marking as low priority as it most likely has been an issue since the original launch and no one has reported.

On emulators the default picker doesn't show external sd cards so goign to install a picker and hope that I can get it working. Does it pick from a photo gallery app?

Hi,
I have been testing it on my phone (android) and the behavior is a bit weird. Some photos from my camera that im picking it from(internal and SD) are being shown and some are not

@henriqueteles I had a similar issue. Try to define a var outside the lamba like:
var stream = new MemoryStream(AddPostViewModel.Post.Image);
stream.Position = 0 // Sometimes I noticed that the Position is at the end
Source = ImageSource.FromStream(() => stream)

Why don't you just do that ?
Source = ImageSource.FromStream(() =>file.GetStream())

I am pretty sure that would be it via @ericbrunner , can you validate?


⚠ Bofrost.fr has degraded performance

In ece9dbc, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1176 ms

Resolved: Bofrost.fr performance has improved in 95e6d04.


EventName.info missing id

Hello, could you please add the reqId and errorCode to the ib.on(EventName.info ?
The reqId (-1) and ErrorCore (2158) is being received so it should be easy for you to add it
received [
  [
    '4',
    '2',
    '-1',
    '2158',
    'Sec-def data farm connection is OK:secdefil',
    ''
  ],
  '4\x002\x00-1\x002158\x00Sec-def data farm connection is OK:secdefil\x00\x00'
]

You may ask why? Its simple, I am building a bot for people and I need to know about every info, and there are many languages in the TWS and also on its white label, so message only is not practical because it would be different in each language, however code is always the same.
Right now there is only message listener but the error code is very important for me and I would be glad if you could add it there.
Thank you

You are welcome to push PR

Seems ok, the compiler eats it xD

Perfect, thank you for fast merge. We can close this issue :)


feat: added the handling of duplicate project names

Summary
Added the handling of duplicate project names

Related issue (if exists)

Types of changes

[ ] Docs change / Dependency upgrade
[ ] Bug fix
[x] New feature / Improvement
[ ] Refactoring
[ ] Breaking change

Checklist

[x] I have added changeset via pnpm run changeset.
[ ] I have added tests to cover my changes.

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


建议把 FullscreenPopup 改成 public

@LZRight123 We make a decision to make FullscreenPopup internal, as it is not a primary part of the library. If you want to use it, feel free to copy-paste it in your project, it is only ~150 LoC without imports.


haskellPackages.cut-the-crap: use ffmpeg instead of ffmpeg_3

Motivation for this change
https://github.com/NixOS/nixpkgs/issues/120705
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] 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.

I only made sure it builds, not that it works.
cc @jappeace

/rebase haskell-updates

We'll merge before branch-off.

ffmpeg isn't used for building at all, but the cli api is stable so this shouldn't cause any issues.


[DNM]*: Add circleci.ym

@siddontang @huachaohuang  PTAL
Please add pingcap/pd on https://circleci.com

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

LGTM

@overvenus PTAL

@overvenus PTAL

LGTM


ord server not working (testnet)

I ran $ ord -t server --http
I can see that the process is running

But it's not creating anything on the network side and I can't access it
$ sudo lsof -i -P -n | grep LISTEN
systemd-r  492 systemd-resolve   14u  IPv4  15108      0t0  TCP <IP_ADDRESS>:53 (LISTEN)
node       613         bitcoin   20u  IPv6  17706      0t0  TCP *:3000 (LISTEN)
bitcoind   704         bitcoin   11u  IPv6  16700      0t0  TCP [::1]:18332 (LISTEN)
bitcoind   704         bitcoin   12u  IPv4  16702      0t0  TCP <IP_ADDRESS>:18332 (LISTEN)
bitcoind   704         bitcoin   28u  IPv4  18231      0t0  TCP <IP_ADDRESS>:18334 (LISTEN)
bitcoind   704         bitcoin   29u  IPv6  18232      0t0  TCP *:18333 (LISTEN)
bitcoind   704         bitcoin   30u  IPv4  18233      0t0  TCP *:18333 (LISTEN)
sshd       720            root    3u  IPv4  17684      0t0  TCP *:22 (LISTEN)
sshd       720            root    4u  IPv6  17695      0t0  TCP *:22 (LISTEN)

Also unable to access "wallet/server" or "wallet/preview" on testnet
And ordinals (dot) com tesnet site is not up-to-date / fully synced

$ ord -t preview --http works though..
what difference would i see between preview and server?
preview    Run an explorer server populated with inscriptions
  server     Run the explorer server

My preview works too, but is showing regtest results rather than testnet results, which the btc and ord servers are set to

yeah.. same..
my bitcoind is indexing testnet data.
but preview is showing regtest.
My command to run preview is:
$ ord -t preview --http --address localhost
but here's what I see:

Same here. ord is also ignoring parameters:
sudo ord -t --config ~/blockchain/bitcoin-core/data/bitcoin.conf --bitcoin-data-dir ~/blockchain/bitcoin-core/data preview --http --address localhost
2023-03-28T14:17:32Z Bitcoin Core version v24.0.1 (release build)
2023-03-28T14:17:32Z Default data directory /Users/somoza/Library/Application Support/Bitcoin
2023-03-28T14:17:32Z Using data directory /tmp/.tmpek0Dd2/bitcoin/regtest
2023-03-28T14:17:32Z Config file: /tmp/.tmpek0Dd2/bitcoin/bitcoin.conf (not found, skipping)

Running server fails with:
sudo ord -t server --http --address localhost
error: failed to connect to RPC URL
because: I/O error: No such file or directory (os error 2)

Does anyone have a testnet explorer running?

There is: testnet.ordimint.com.


tree value

Use root.count or root.sum and expose a node:value somehow
(From https://github.com/observablehq/plot/pull/841)

Being able to preprocess the tree with a function that gets passed the root node might be a more general solution than specifically calling count or sum (eg. maybe the user wants to do something more involved using a particular traversal).
The API already exposes the tree node objects as nodes if you use the { node: d => ... } form for an option value so I don't think this would involve a bigger commitment to the d3 tree representation than already exists.

I suppose that if we define a value option it could accept built-ins ("sum" and "count") as well as a more generic function.

an example https://observablehq.com/@john-guerra/tree-value


SSR rendering twice in next branch

Not sure if this is an issue but I noticed that when I console.log in render in Home or About for the branch next its getting called twice. If this is the expected behavior can someone explain to me why this happens? Thanks!
I tested the master branch and it seemed to be fine.

Hey @kennethtruong the reason you're seeing a double console.log has to do with asynchronously loading the component.
Allow me to explain. Replace one of those logs with a console.trace('hi').
 The first time, essentially, the component is being explored in order to find out info before rendering.
The second trace illustrates the actual rendering of the component into the DOM.
@ctrlplusb Might be able to explain a bit better. It's late and Ive gotta hit the pillow. Hopefully that helps.

Ohh I see! I had a tiny suspicion that it could have been related to the asyncComponent since that was one of the things that changed from master to next. Thanks for taking the time to my question!


Making AES_128_GCM_SHA256 have a higher priority over CHACHA20_POLY1305_SHA256 on HTTP2

Motivation:
From the recent benchmark using gRPC-Java based on Netty's HTTP2, it appears that it prefers ECDHE_ECDSA_WITH_CHACHA20_POLY1305_SHA256 over ECDHE_ECDSA_WITH_AES_128_GCM_SHA256 since it uses the Netty HTTPS Cipher list as is. Both are considered safe but ECDHE_ECDSA_WITH_AES_128_GCM_SHA256 has a good chance to get more optimized implementation. (e.g. AES-NI) When running both on GCP Intel Haswell VM, TLS_ECDHE_ECDSA_WITH_CHACHA20_POLY1305_SHA256 spent 3x CPU time than TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256. (Note that this VM supports AES-NI)
From the cipher suites listed on Intermediate compatibility (recommended) of Security/Server Side TLS, they have a cipher preference which is aligned with this PR.
0x13,0x01  -  TLS_AES_128_GCM_SHA256         TLSv1.3  Kx=any   Au=any    Enc=AESGCM(128)             Mac=AEAD
0x13,0x02  -  TLS_AES_256_GCM_SHA384         TLSv1.3  Kx=any   Au=any    Enc=AESGCM(256)             Mac=AEAD
0x13,0x03  -  TLS_CHACHA20_POLY1305_SHA256   TLSv1.3  Kx=any   Au=any    Enc=CHACHA20/POLY1305(256)  Mac=AEAD
0xC0,0x2B  -  ECDHE-ECDSA-AES128-GCM-SHA256  TLSv1.2  Kx=ECDH  Au=ECDSA  Enc=AESGCM(128)             Mac=AEAD
0xC0,0x2F  -  ECDHE-RSA-AES128-GCM-SHA256    TLSv1.2  Kx=ECDH  Au=RSA    Enc=AESGCM(128)             Mac=AEAD
0xC0,0x2C  -  ECDHE-ECDSA-AES256-GCM-SHA384  TLSv1.2  Kx=ECDH  Au=ECDSA  Enc=AESGCM(256)             Mac=AEAD
0xC0,0x30  -  ECDHE-RSA-AES256-GCM-SHA384    TLSv1.2  Kx=ECDH  Au=RSA    Enc=AESGCM(256)             Mac=AEAD
0xCC,0xA9  -  ECDHE-ECDSA-CHACHA20-POLY1305  TLSv1.2  Kx=ECDH  Au=ECDSA  Enc=CHACHA20/POLY1305(256)  Mac=AEAD
0xCC,0xA8  -  ECDHE-RSA-CHACHA20-POLY1305    TLSv1.2  Kx=ECDH  Au=RSA    Enc=CHACHA20/POLY1305(256)  Mac=AEAD
0x00,0x9E  -  DHE-RSA-AES128-GCM-SHA256      TLSv1.2  Kx=DH    Au=RSA    Enc=AESGCM(128)             Mac=AEAD
0x00,0x9F  -  DHE-RSA-AES256-GCM-SHA384      TLSv1.2  Kx=DH    Au=RSA    Enc=AESGCM(256)             Mac=AEAD

Modification:
Moving up AES_128_GCM_SHA256 in the CIPHERS of HTTPS so that it gets priority.
Result:
When connecting to the server supporting both ECDHE_ECDSA_WITH_AES_128_GCM_SHA256 and ECDSA_WITH_CHACHA20_POLY1305_SHA256 and respecting the client priority of cipher suites, it will be able to save significant cpu time when running it on machines with AES-NI support.

@ejona86 Thanks for the comment. I updated the description accordingly.

Friendly ping! If there is something I can do, please let me know.

@veblush did you sign our icla ?
https://netty.io/s/icla

@normanmaurer Oh I just did it now.

@netty-bot test this please

It appears that it passed all checks. Is it now ready for merge?

@veblush sorry for the long turn around... I just returned from vacation and merged it :)

@normanmaurer No worry about that! Thank you for the review and approval.


Configuration for BIPM/ephemeris version should be stored in the data repository

Currently, the checks that pint_pal does to make sure the "correct" BIPM and ephemeris versions are being used are based on a file tracked as part of pint_pal, namely src/pint_pal/defaults.py. This was appropriate when this code was only being used for a single data release (the NANOGrav 15-year), but different data releases (say IPTA DR3 and NG20) might want to make different choices for these values.
The best solution to this would seem to be to store this configuration in the corresponding data repository instead of the pint_pal repository.

Resolved by #63.

Resolved by #63


Revert "Support passing number only values as string directly through helm.va…"

Reverts GoogleContainerTools/kpt-config-sync#153
This change does not work for all cases, revert it for now, need a generic solution to support all special value cases.

/retest


feat: golang-github-go-macaron-inject SRC

utilities for mapping and injecting dependencies
Log: golang-github-go-macaron-inject SRC would be added.
Issue:
https://github.com/deepin-community/sig-deepin-sysdev-team/issues/353

/lgtm


Invoke-DbatoolsFormatter - remove tabs from code comments

Invoke-DbatoolsFormatter - remove tabs from code comments

Type of Change

[x] Bug fix (non-breaking change, fixes #7127 )
[ ] New feature (non-breaking change, adds functionality, fixes # )
[ ] Breaking change (effects multiple commands or functionality, fixes # )
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
Ensure that tabs are removed from code comments, since the dbatools style check will fail if there are tabs in the command documentation comments block.
Approach

The fix proposed here is to replace tabs with 4 spaces during the last step where the whitespace trim is being done.

thanks for the approval, @niphlod ! and thank you for the PR, @lancasteradam !


Any comparison with imagenet inception v3 for transfer learning / visual search

Hi I have integrated Open Images inception into Deep Video Analytics [1] for generating text tags for an image/frame. However currently I am using imagement pre trained inception v3 for visual search. Have you tried using Open Images inception for transfer learning / visual search is it better than the imagenet trained model?
Thanks,
[1] https://deepvideoanalytics.com/

Better is subjective, and one cannot do an apples-to-apples comparison since the openimages trained model is trained with another set of labels.   but the set of annotation labels is bigger, so you would have more coverage for visual search.


fix: improve lobby creation query speeds

Changes

#866  👈
main

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

Join @AngelOnFira and the rest of your teammates on  Graphite

Dupe #868


Favor exceptions and Optionals over boolean and nulls

There's an anti-pattern in the code of returning nulls when an exceptional case occurs. This causes any context from the source of the exceptional case to be lost and it results in lots of null checks and logging at all higher stacks of the code. Ideally methods never return null.
Favor Exceptions over nulls
If a method should always return a value, except during an exception, mark it as @NotNull and throw an exception if an exceptional case occurs. The thrown exception could be a custom exception, defined for the situation. This provides the following benefits:

Lowered risk of NPEs
More concise code written to the happy path, without null checks.
Cleaner logging, since everyone who handles the null doesn't need to blab about it (aside: when handling exceptions you should either log or throw but not both).
More context about the root of the issue (e.g., rich exception messages, stack traces, etc)
A more informed caller, since the calling code can handle different exception types or exception response codes differently.

Favor Exceptions over booleans
If an action should always succeed under normal circumstances, favor a void return type and indicate failure with an exception. Using a boolean true/false to indicate success/failure has similar downsides to using null to indicate failure. NPE's are no longer an issue with this approach, but returning false will be equally void of meaningful context about the failure. Should the calling code retry? Should it give up? Custom exceptions or response codes can help the caller decide.
Favor Optional over null
If a method may or may not return a value as part of normal behavior, favor returning an Optional of the given return type. This article explains why.
Caveat: Optional was introduced in Java 8, so it should not be used to maintain Java 7 compatibility. Instead we should use guava's Optional.

Is a "Best Practices" doc page a expected deliverable for this issue?

@ashvina - that is a start. But we need to continue to be mindful of it during the code review. Then we need to clean up our existing code.

Favor immutability (#1323) is another best practice.


Added expanding menu to FloatingActionButton

Added expanding menu
#80
#103
Sample

bump!! any issues please tell

Nice job, maybe you should update the README too, to let people know how it works.
I hope it get merged soon!

I think, that will be good add auto hide menus when is FAB binding to listview ...
public void onScrollUp() {
if (isMenuShowing) { triggerMenu(90); } // Add to hide menus

I try it on API 10 and I dont know if is problem with combination ListView and SwipeRefreshLayout but cant click on menus

It's probably worth noting that this change only works if the FAB's parent is a FrameLayout, which won't be the case for everybody.

Thanks a lot for your contribution but decided to keep this library simple and focused (without floation action menus but only the FAB itself).


add SkipToContent button to SubdomainNavBar

Summary
Adds a SkipToContent button to the SubdomainNavBar
List of notable changes:

Adds a SkipToContent button to the SubdomainNavBar

What should reviewers focus on?
Does the button work?
Does it properly hide itself when not in focus?
Does it jump to the content of the page.
Steps to test:
You can test this component on any of the SubdomainNavBar stories, e.g. https://primer-6e6587907b-26139705.drafts.github.io/storybook/?path=/story/components-subdomainnavbar--playground
Supporting resources (related issues, external links, etc):
Solves #404
Contributor checklist:

[ ] All new and existing CI checks pass
[ ] Tests prove that the feature works and covers both happy and unhappy paths
[ ] Any drop in coverage, breaking changes or regressions have been documented above
[ ] New visual snapshots have been generated / updated for any UI changes
[ ] All developer debugging and non-functional logging has been removed
[ ] Related issues have been referenced in the PR description

Reviewer checklist:

[ ] Check that pull request and proposed changes adhere to our contribution guidelines and code of conduct
[ ] Check that tests prove the feature works and covers both happy and unhappy paths
[ ] Check that there aren't other open Pull Requests for the same update/change

Screenshots:

Please try to provide before and after screenshots or videos

 Before  After

@joseph-lozano I'd suggest use banner stretching under the header to avoid having the button awkwardly overlap the header UI like this:

Since this would be a custom additions we could consider using our button component and stretch it full width (like Linear does). Since our CTA component is very chunky it would look like this - ideally we'd want to remove the border radius for it too look more like the banner above.

Happy to see/review whatever is feasible here

After chatting at the Accessiblity Eng+ Office Hours, it sounds like we should first look for a <main> element before falling back to creating an empty div for the start-of-content.
Additionally, if we create the start-of-content empty div, we should make sure to give it tabindex="-1".
https://github.com/github/accessibility/issues/5240

@rezrah thanks for helping me sort out running the test:axe. It turned out that it was missing a directory so mkdir apps/docs/public/static which was breaking the run of npm run build:prod --workspace=apps/storybook.  So adding the empty dir like you said made it run.
But then this wasn't what I needed, what I needed was to get that storybook-static directory. So you helped me by mentioning npx storybook build so I did this:
cd apps/storybook
npm i # which I think doesn't do anything cause node_modules are in the root directory
npx storybook build

This gave me the storybook-static directory with the stories.json that is needed for the packages/e2e/scripts/playwright/axe-clean.spec.ts to run when called by npm run test:axe --workspace=packages/e2e since this was the CI that was failing and since we don't list all of the violations, I couldn't know this before I got this working locally. **I offer to try to write the code for listing the violations, shouldn't be that hard ☺, so we can see directly from the CI what AXE is telling us. 🪓 **
So I curated the stories.json to run the two stories in question and found out what it was complaining about, which is this 👇

These stories include <main> tags which automatically have the role="main" and AXE doesn't allow us to have any other role for the <main> tag. But also we have a default Storybook div role="main" which was killing our buzz. 👇

I wonder if we can load the desired story independently of the story chooser. I think we should be able to do so. I'll try to update the code.
But as the instant remedy I added these two stories in question to the ignore list. Since they are just for testing the ability to scroll to <main>.
P.S. The reason I needed to curate the stories.json is because I was getting an AXE violation for this 👇 😆 It's a banner telling us to update storybook. I think loading the stories independently of the chooser will ignore this violation.


[Faculty] Graduation times - licentiates

Investigate licentiates, why so few are shown currently.

There just are so few, all check's out


osx builds not reflecting openssl constraints correctly

The openssl dependency is marked # [not osx] for reasons I don't claim to understand; however, there are still two separate CI jobs being generated, as well as two builds (with different hashes, because openssl still somehow ends up in there).
Either we should:

reflect the respective openssl build in the dependencies
only have one build on osx (resp. osx-arm)

It may be due to things like comments and selectors in patches:
https://github.com/conda-forge/qt-main-feedstock/blob/main/recipe/meta.yaml#L32
Can we apply this patch unconditionally?

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/qt-main-feedstock#88.

Can we apply this patch unconditionally?

It's not about the patch AFAICT, especially because that selector is not arch-specific.
IMO it's about the fact that the output has a # [not osx] for openssl

See rerendered reipe in #88. I conda-smithy scans for occurances of openssl that may affect the build. It isn't too smart.

See rerendered recipe in #88. I conda-smithy scans for occurances of openssl that may affect the build. It isn't too smart.

OK, I think we were talking past each other to an extent. I understand why openssl still ends up in the list of host deps (we could add ignore_run_export). My point was more that I consider it curious that only osx wouldn't require openssl...

It is currently explicitly disabled in https://github.com/conda-forge/qt-main-feedstock/blob/9c43d53232c1bd572f241d1ed9cb0a2612627dfe/recipe/build.sh#L247 - I can't recall why, though. Might be worth a shot using the same flags for osx that we use on linux and see what happens?!?

I think this is because in Mac you can use the

native secure transport instead of OpenSSL

as described here (see last message).
However, the conda-forge build is missing the securetransport option.

From the build logs in #88
2022-12-11T19:18:56.1823310Z Qt Network:
2022-12-11T19:18:56.1823820Z   CoreWLan ............................... yes
2022-12-11T19:18:56.1824370Z   getifaddrs() ........................... yes
2022-12-11T19:18:56.1824940Z   IPv6 ifname ............................ yes
2022-12-11T19:18:56.1825500Z   libproxy ............................... no
2022-12-11T19:18:56.1826070Z   SecureTransport ........................ yes
2022-12-11T19:18:56.1826640Z   OpenSSL ................................ no
2022-12-11T19:18:56.1827190Z     Qt directly linked to OpenSSL ........ no
2022-12-11T19:18:56.1827750Z   OpenSSL 1.1 ............................ no

Thanks @hmaarrfk! So, I'd say OpenSSL on Mac is not really necessary.

Should we merge #88 as a cleanup? I left the build number the same so that it shouldn't create new packages. At least for linux and windows.

+1 from me. I'd also suggest to add the -securetransport option before -no-openssl on build.sh and leave a comment about this discussion so future maintainers have an easier job understanding why that option is used on Mac.

ok. i added a comment. Feel free to merge when you feel like it went through enough of the build process on mac.

now that imerged #88 can we close this?


Refusing to install react-image-annotate as a dependency of itself

npm-debug.log

I can't seem to replicate this. Can you give some details of your operating system and what commands you ran?

What version of node are you using? What's the output of node --version? Mine is v12.16.1

Hi all,  sorry i guess it was fault from my side,  it was silly i did not
then understood how npm installations work.  UDT works perfectly..  Thanks
for being helpful and responsive.
On Wed, Aug 26, 2020, 6:00 AM Severin Ibarluzea<EMAIL_ADDRESS>wrote:

What version of node are you using? What's the output of node --version?
Mine is v12.16.1
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/UniversalDataTool/react-image-annotate/issues/79#issuecomment-680358237,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMTZDMLVJFW73KYB5BO7SQDSCRJUBANCNFSM4PZJKIOQ
.

--
Verificient Technologies, Inc.
https://www.verificient.com/
https://www.verificient.com/ | New York, USA | Pune | Delhi NCR, India
*

_
_
This email and any files transmitted with it are confidential and
intended solely for the use of the individual or entity to whom they are
addressed. If you have received this email in error please notify the
system manager. This message contains confidential information and is
intended only for the individual named. If you are not the named addressee
you should not disseminate, distribute or copy this e-mail. Please notify
the sender immediately by e-mail if you have received this e-mail by
mistake and delete this e-mail from your system. Verificient Technologies
accepts no liability for the content of this email, or for the consequences
of any actions taken on the basis of the information provided.

no problem!!


Train myself

I am trying to use the process.py , but I have this error. Is that code still maintained?
Also I had to edit some python imports about hubert.
python process.py --path /opt/docker/compose/discord-tts-bot/bark/config/vocab.txt --mode prepare2
2024-08-08 17:14:34 | INFO | fairseq.tasks.text_to_speech | Please install tensorboardX: pip install tensorboardX
WARNING: einx with PyTorch requires PyTorch version >= 2, but found
Traceback (most recent call last):
  File "/opt/projects/bark-voice-cloning-HuBERT-quantizer/process.py", line 4, in <module>
    from prepare import prepare, prepare2
  File "/opt/projects/bark-voice-cloning-HuBERT-quantizer/prepare.py", line 8, in <module>
    from bark_hubert_quantizer.pre_kmeans_hubert import CustomHubert
  File "/opt/projects/bark-voice-cloning-HuBERT-quantizer/bark_hubert_quantizer/pre_kmeans_hubert.py", line 20, in <module>
    from audiolm_pytorch.utils import curtail_to_multiple
  File "/opt/projects/bark-voice-cloning-HuBERT-quantizer/.venv/lib/python3.10/site-packages/audiolm_pytorch/__init__.py", line 8, in <module>
    from audiolm_pytorch.audiolm_pytorch import AudioLM
  File "/opt/projects/bark-voice-cloning-HuBERT-quantizer/.venv/lib/python3.10/site-packages/audiolm_pytorch/audiolm_pytorch.py", line 23, in <module>
    from audiolm_pytorch.soundstream import SoundStream
  File "/opt/projects/bark-voice-cloning-HuBERT-quantizer/.venv/lib/python3.10/site-packages/audiolm_pytorch/soundstream.py", line 20, in <module>
    from vector_quantize_pytorch import GroupedResidualVQ
  File "/opt/projects/bark-voice-cloning-HuBERT-quantizer/.venv/lib/python3.10/site-packages/vector_quantize_pytorch/__init__.py", line 1, in <module>
    from vector_quantize_pytorch.vector_quantize_pytorch import VectorQuantize
  File "/opt/projects/bark-voice-cloning-HuBERT-quantizer/.venv/lib/python3.10/site-packages/vector_quantize_pytorch/vector_quantize_pytorch.py", line 12, in <module>
    import einx
  File "/opt/projects/bark-voice-cloning-HuBERT-quantizer/.venv/lib/python3.10/site-packages/einx/__init__.py", line 5, in <module>
    from . import backend
  File "/opt/projects/bark-voice-cloning-HuBERT-quantizer/.venv/lib/python3.10/site-packages/einx/backend/__init__.py", line 1, in <module>
    from .register import register_for_module, register, get, backends, numpy
  File "/opt/projects/bark-voice-cloning-HuBERT-quantizer/.venv/lib/python3.10/site-packages/einx/backend/register.py", line 51, in <module>
    register_for_module("torch", _torch.create)
  File "/opt/projects/bark-voice-cloning-HuBERT-quantizer/.venv/lib/python3.10/site-packages/einx/backend/register.py", line 19, in register_for_module
    register(backend_factory())
  File "/opt/projects/bark-voice-cloning-HuBERT-quantizer/.venv/lib/python3.10/site-packages/einx/backend/register.py", line 30, in register
    raise ValueError("Backend must be an instance of einx.backend.Backend")
ValueError: Backend must be an instance of einx.backend.Backend

Looks like the environment you're running this in is using an old pytorch version.
Make sure you're using a pytorch version at least 2.0.0
pip install -U torch torchvision torchaudio will update pytorch to the latest version
Also, looking at your command, the --path flag takes a directory, not a txt file, if you want to train, you'll need to generate a dataset first (https://github.com/gitmylo/bark-data-gen)
And last thing, going off the project name "discord-tts-bot", FYI you don't need to train a whole new quantizer model if you just want to clone a voice. The different models are just to support different languages. The huggingface repo for the official models can be found here, but using the HuBERTManager will also download the models automatically.

Looks like the environment you're running this in is using an old pytorch version. Make sure you're using a pytorch version at least 2.0.0 pip install -U torch torchvision torchaudio will update pytorch to the latest version
Also, looking at your command, the --path flag takes a directory, not a txt file, if you want to train, you'll need to generate a dataset first (https://github.com/gitmylo/bark-data-gen)
And last thing, going off the project name "discord-tts-bot", FYI you don't need to train a whole new quantizer model if you just want to clone a voice. The different models are just to support different languages. The huggingface repo for the official models can be found here, but using the HuBERTManager will also download the models automatically.

Thank you. But I just runned the code without changing anything on that code/requirements.txt
Anyways I think I'll need some example on how to train a model e and where to find one for Italian language, could you provide some?

Looks like the environment you're running this in is using an old pytorch version. Make sure you're using a pytorch version at least 2.0.0 pip install -U torch torchvision torchaudio will update pytorch to the latest version
Also, looking at your command, the --path flag takes a directory, not a txt file, if you want to train, you'll need to generate a dataset first (https://github.com/gitmylo/bark-data-gen)
And last thing, going off the project name "discord-tts-bot", FYI you don't need to train a whole new quantizer model if you just want to clone a voice. The different models are just to support different languages. The huggingface repo for the official models can be found here, but using the HuBERTManager will also download the models automatically.

Thank you. So if I want to use italian do i have to train another quantizer or not?
I am trying to clone my voice (i am Italian), I don't understand if these quantifier are good for italian also
https://huggingface.co/GitMylo/bark-voice-cloning/tree/main

Guess i solved it by doing so:
https://github.com/gitmylo/bark-data-gen/issues/3
now i just have to wait for the npy and wav generations, next step is to train my dataset.
I am doing so on 1600 ebooks.. I hope it wont take ages!!

@gitmylo Just a question, in your opinion, how many data do I have to generate with (https://github.com/gitmylo/bark-data-gen) to train an other languuage?
I triied with 5000 npy/wavs but the result was bad, maybe just because i stopped the training at 0.3 loss. I don't know....
By the way I made a new version of  create_data, with correspondent launch script, Do you think we can integrate that in the code?
create_data_and_wavs.py
import os.path
import random
import uuid
import sys
import hashlib
from random import randrange

import numpy

from bark import text_to_semantic
from bark.generation import load_model, SAMPLE_RATE
from bark.api import semantic_to_waveform

from data import load_books, random_split_chunk
from scipy.io import wavfile

loaded_data = load_books()

print('Loading semantics model')
load_model(use_gpu=True, use_small=False, force_reload=False, model_type='text')
print('Loading coarse model')
load_model(use_gpu=True, use_small=False, force_reload=False, model_type='coarse')
print('Loading fine model')
load_model(use_gpu=True, use_small=False, force_reload=False, model_type='fine')

output = '/mnt/t/projects/npy_ita'
output_wav = '/mnt/t/projects/wav_ita'

if not os.path.isdir(output):
    os.mkdir(output)

loop_count = 10
try:
    if len(sys.argv) > 1:
        loop_count = int(sys.argv[1])
        print('Loop count found in cmdline:', str(loop_count))
except:
    pass

for i in range(loop_count):
        text = ''
        while not len(text) > 0:
            text = random_split_chunk(loaded_data)  # Obtain a short chunk of text
        if text is not None and text != '':
            text = text.strip()
            filename = hashlib.md5(text.encode('utf-8')).hexdigest() + '.npy'
            file_name = os.path.join(output, filename)
            if not os.path.isfile(file_name):
                print('Loop count:', str(i))
                print('Generating semantics for text:', text)
                semantics = text_to_semantic(text, temp=round(random.uniform(0.6, 0.8), ndigits=2))
                numpy.save(file_name, semantics)

                base_name = os.path.basename(file_name)

                real_name = '.'.join(base_name.split('.')[:-1])  # Cut off the extension
                out_file = os.path.join(output_wav, f'{real_name}.wav')
                if not os.path.isfile(out_file):  # Don't process files that have already been processed
                    print(f'Processing {file_name}')
                    wav = semantic_to_waveform(numpy.load(file_name), temp=round(random.uniform(0.6, 0.8), ndigits=2))
                    wavfile.write(out_file, SAMPLE_RATE, wav)

print('Done!')

create_data_and_wavs.sh
#/usr/bin/bash
cd /opt/projects/bark-data-gen
source .venv/bin/activate; python create_data.py $1


Preparing code for v2 release

Opted for Major Branch approach instead of Major Subdirectory.
We might consider creating a new v2 branch for clarity, and to avoid breaking clients that are not using go mod.

I wouldn't worry too much about folks not using go modules. Last I checked there weren't any non-NFLX projects using spectator-go other than openconfig/gnmi-gateway, which is something we open sourced, we no longer support, and it uses modules.

Fixed on #96 with code rebased


Can we use this library in app engine?

Can we use this library in the app engine?  I would like to receive frames in the app engine server.

Discuss simple queries on Gitter chat : https://gitter.im/vidgear/community?utm_source=notification&utm_medium=email&utm_campaign=unread-notifications


Custom image component for use in MDX

Additional context: https://prisma-company.slack.com/archives/C5Z9TH6N9/p1699035865780689

@ruheni Could you please elaborate on how this component should be? Use the issue template if needed

We'd like to have a custom image component which we could define custom styling for alignment like we do on this page. The regular img HTML tag

Context on the Slack thread

doesn't render correctly on first load.

Closing this as not planned since no real use cases for this came up besides the Turso logo, which I'm also suggesting to remove from the docs now to keep the page a bit cleaner:


Moving cdn (2) resources to v1beta1 version

Description of your changes
Adding the following resources to cdn:

[x] azurerm_cdn_endpoint
[x] azurerm_cdn_profile

Fixes #122
I have:

[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
azurerm_cdn_endpoint - manually and uptest
azurerm_cdn_profile - manually and uptest

examples="examples/cdn/endpoint.yaml"

examples="examples/cdn/profile.yaml"

/examples="examples/cdn/endpoint.yaml"

/test-examples="examples/cdn/endpoint.yaml"

/test-examples="examples/cdn/profile.yaml"


MNT: rerender

Hi! This is the friendly automated conda-forge-webservice.
I've rerendered the recipe as instructed in #363.
Here's a checklist to do before merging.

[ ] Bump the build number if needed.

Fixes #363

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

This seems to be a no-op. Is there some additional context on why this is needed somewhere?

I marked a bunch of packages I maintain for myself broken in the UI since I was in a rush. Needs a rebuild here to propagate.

Are we able to just restart CI here? Happy to do that if you'd like

The PR does the same thing as a CI restart but bumps folks less. I'll do that in the azure interface next time.

Added a build here

Hopefully that helps get out even faster 🐎
Thanks Matt! 🙏

Thank you!

Ofc. FWIW the build just finished.


[Bug]: Version 2.7.0 extra793 fails for network LBs

What happened?
After upgrading to prowler 2.7.0 existing network LBs are now reported aus failure from extra793. Network LBs where skipped before this version. This was introduced by a copy past mistake in https://github.com/prowler-cloud/prowler/commit/7b848c04c417b44b5fa7db8e4ee9a6e8af79e686#diff-b1664cb40c3cbbbda77c412cc03924209c78e46bbb71967b64701b079f902b71
Expected behavior
7.93 [extra793] Check if Elastic Load Balancers have SSL listeners  - elb [Medium]
Actual behavior
7.93 [extra793] Check if Elastic Load Balancers have SSL listeners  - elb [Medium]
FAIL! eu-central-1:  has non-encrypted listeners
From where are you running Prowler?
Please, complete the following information:

Resource: EC2 instance
OS: Amazon Linux 2
AWS-CLI Version [aws --version]: 1.22.42
Prowler Version [./prowler -V]: 2.7.0
Shell and version: bash
Others:

It is now fixed in master. Thanks for the heads up.

@toniblyx sorry, I noticed a different line and created another PR to fix this

I see, thanks!

@toniblyx is there any planned release date for a 2.7.1 version? I don't want to use master...

Let´s see if we can fix some more issues and we make the minor release. We will check it out later this week.


wanted to understand how to deploy graphql on AWS EC2 instances using graphql mesh and hive gateway

I have few doubts regarding using graphql mesh and deploying is using hive gateway without any subcription or cloud account of hive.

I can create mesh config and can generate the graphql file using mesh. Will also create gateway config for caching etc.
I can start the server using npx hive-gateway supergraph xyz.graphql, now if i write a script to run this server on a EC2 instance, it will serve as a graphql server right? Or do i need hive token etc to deploy it etc?
Also their is Dashboard section with explorer, laboratory, are those features only for hive organisation membership? or it can be used without any membership and how? could not find any documentation or same.
What is the best way to deploy it for best performance? using npx command and js package or as a express server or using binary cli etc?

Yes that's how you should do!
You can start the server exactly like that. You need Hive CDN Token if you use Hive CDN. If you need to report to Hive Dashboard, you need Hive Token. Other than, you don't need any token if you just want to serve your API.

https://the-guild.dev/graphql/hive/docs/high-availability-cdn
https://the-guild.dev/graphql/hive/docs/schema-registry/usage-reporting

You can also self-host Hive Dashboard on your own too, if you don't want to sign up Hive.
https://the-guild.dev/graphql/hive/docs/self-hosting/get-started
For the best performance, you can use Hive Gateway CLI or our official Docker image. We recommend you to avoid any complicated server setup with Express etc.

Hey there! Thanks for writing. Let me jump straight in:

GraphQL Mesh outputs a supergraph SDL that the Hive Gateway consumes. It does not generate a config file for the gateway.
Hive Console (the dashboard) and Hive Gateway are separate products and can be used fully independently. So yes, you can serve any Federation supergraph without ever depending, or even having an account, on the Hive Console.
All features of Hive Console (the dashboard) are available for every type of account. Here's the relevant documenation for the two:
a. Explorer
b. Laboratory
The Hive Gateway CLI (npx hive-gateway) already has all performance related features implemented. You're recommended to use either the CLI or the official Docker image from this repo.

Oh, @ardatan beat me to it! @vibhor-d11 pick and choose. 😄


camleyon16 test data corrupt

Hi
Thank you for the wonderful work.
After downloading camleyon16 test data this is what I get trying to unzip it.
unzip c16-dataset-test.zip -d .
Archive:  c16-dataset-test.zip
warning [c16-dataset-test.zip]:<PHONE_NUMBER> extra bytes at beginning or within zipfile
(attempting to process anyway)
error [c16-dataset-test.zip]:  start of central directory not found;
zipfile corrupt.
(please check that you have transferred or created the zipfile in the
appropriate BINARY mode and that you have compiled UnZip properly)
Is it actually corrupted or my internet and downloading is not working properly?

I have tested both the function that download.py uses and normal unzipping tools available on macOS 13.2 and Ubuntu 22.04 LTS and also unzipping for TCGA test data is ok so I don't think my tools are the problem. Nonetheless what is the tool you used to unzip them so I can try it?


Crash in dev environment

After pointing the mod to the correct directory when it is loading, I get the following crash:
http://paste.ubuntu.com/11270269/

I've seen this a few times and I'm not quite sure what causes it. Is it possible that you have NEI added to your dev classpath and in the mods folder?

NEI and CCC(the dev and src jars of both) are in the forgeroot/eclipse/mods folder, and the dev jars of both are added to the build path.

That's probably the issue. You need to make them compile-only (so they're not added to the runtime classpath)

And how exactly do I do that? I can't find anything on Google about how to.
I, for now, have removed it from the build path, and am just ignoring the "errors" in my NEI compat classes.

I presume you're using eclipse, in which case, find the "order and export" tab and don't export nei

Alright, thanks!

Ok, I have re-added them to the build path, gone to the Order and Export tab, and unchecked the boxes to export NEI and CCC. It resulted in the crash again. So, I restarted Eclipse and tried again, with the same result. I then checked both of them, restarted Eclipse(One post I found about similar issues said Eclipse doesn't update those until it restarts), and tried running it again, with the same crash. Then, I tried leaving CCC checked and NEI unchecked, and vice versa, and still got the crash.
tl;dr: Tried how you said to do it, and no matter how I do it, it crashes again.

Unfortunately I don't use eclipse, so I don't know how to specify it properly. Try unchecking both, and sending me the full fml log.

Here you go:
http://paste.ubuntu.com/11382954/

On the second line, where it says the java classpath
C:\Users\Lance\Documents\unlogic-ii\eclipse\mods\NotEnoughItems-1.8-<IP_ADDRESS>-dev.jar
eclipse isn't removing them from the runtime classpath properly

perhaps http://stackoverflow.com/questions/9282440/how-do-i-edit-the-runtime-classpath-in-eclipse

alternatively (and I have no idea if this will work) try moving NEI to a place that's not the mods folder

The stack overflow answer looks good, I will try it when I get the chance. As for your alternative suggestion, which you aren't sure about, I have tried that before, it results in a crash.

A different crash, out of curiosity, which one?

Ok, I tried what the stack overflow, that only allows you to add and remove projects, not the jars.

I just booted up eclipse, and may have figured it out.
http://puu.sh/i2rjv/702657a6c4.png
Click debug configurations and go to the classpath tab
http://puu.sh/i2ret/e6fc15b33d.png
select the project, click edit
http://puu.sh/i2rhH/f7c304cc7a.png
check the Only include exported entries box
Then you can go to the Order and Export tab and check everything but
NEI/CCC/stuff in your mods folder
http://puu.sh/i2rlJ/37f77f91ce.png
hopefully that works

A bit of fiddling it appears you may need to delete and re-add the project
to the debug classpath window, I'm not entirely sure, but you should be
able to find something in there

Deleting and re-adding the project let me fix it. I will let you know if I have any other issues. As for the other crash, which you wanted to know about, I will edit in a link to it when I get home.


Run on Dimensionality of Nations data

This is a real world test/use for Model 1.  The HIRM paper ran on this data, but with the continuous variables thresholded at their means and one-of-n coding for the categorical variables.  We should be able to use both variables as-is.
Data is from
R. J. Rummel. Dimensionality of nations project: Attributes
of nations and behavior of nation dyads, 1950-1965. Technical Report ICPSR 5409, Inter-university Consortium
for Political and Social Research, 1999
and can be downloaded at https://www.icpsr.umich.edu/web/ICPSR/studies/5409

That download link will only actually give you the data if you are affiliated with a ICPSR member instutition.  MIT is so affiliated; Google is not.
This means that we probably can't put this data in our repository, either.


Glimmer components

allows for block params and more advanced mustache on % syntax
adds deprecation API and deprecations for:

legacy Ember.View syntax
explicit mustache without =

Depends on #271 and #270

@machty I could use your advice on how to move forward with this branch.  I think the refactoring was important for making the glimmer component logic work, but it also helped clarify the overall logic a good deal for me.  As a result, in the process of working on it it made several bug fixes trivial.  (And unfortunately, it would take some extra work to backport them to a pre-refactor version).
I would really like to see the bugfixes and refactoring merged.  As far as I know, the work on glimmer components is in line with the current syntax, but I don't think much more can be done until the Ember team finalizes that syntax.  I think there is a good case for merging the current work, but with the obvious understanding that as the syntax finalizes some changes may need to be made.
What would you like to see from this branch to get it merged?  If it would help I can split up this branch into its different parts (bugs, refactoring, etc.).
Thanks!

@thec0keman could you e-mail me at<EMAIL_ADDRESS>I'd like to discuss giving you commit bit for this repo :)


Datagrid Feature Comparison Matrix

Feature Request
Description of Problem:
regular-table offers some unique features and a simple API compared to many Data Grid libraries available.  It would be helpful to see a more granular breakdown of how regular-table compares.
These features make regular-table unique in this regard:

It's just a <table>!
Virtualized rendering for high performance.
async friendly for easy remote data models.
Easy to style, works with vanilla CSS for <table>.
Small.
Simple API.

Potential Solutions:
From #3, which lists many such alternatives that would be a good place to start:
curated lists of grid/table projects:

awesome-grid

vanilla javascript + DOM (similar to this project):

Clusterize

React-based:

react-table
react-datasheet
Griddle
material-table
rsuite-table
frameworks:

react-sortable-hoc
react-virtualized

for Angular/Vue:

primeng
ngx-datatable

other:

jqGrid
tabulator
tui.grid

@texodus How are you calculating that minified size?


Fail importing from Teslafi

Hi I am having issues importing history from Teslafi to teslamate. I succeeded importing 1 of 6 months and it shows in grafana. I have tried multiple times importing other months in a bundle and always get 1 green checkbox (false positive, not showing in grafana) and the rest of the files gets a blue status “looks a little like a WiFi logo”
When I try importing one file at the time I get a green checkbox but data is not showing in grafana.
I noticed from the logs I was first having issues with memory (raspberry pi) but it helped stopping grafana docker container wile importing.
00:20:01.722 [info] Already up
00:20:49.242 [info] Version: 1.16.0
00:20:50.679 [info] Refreshed api tokens
00:20:50.680 [info] Scheduling token refresh in 40d
00:20:50.752 [info] Running TeslaMateWeb.Endpoint with cowboy 2.7.0 at :::4000 (http)
00:20:50.766 [info] Access TeslaMateWeb.Endpoint at http://localhost00:21:42.216 [info] GET /
00:21:42.304 [warn] Elixir.Cldr.NoMatchingLocale: No configured locale could be matched to "sv-se"
00:21:42.319 [info] Sent 302 in 103ms
00:21:42.429 [info] GET /import
00:21:42.435 [warn] Elixir.Cldr.NoMatchingLocale: No configured locale could be matched to "sv-se"
00:21:42.571 [info] Sent 200 in 141ms
00:22:29.140 [info] Importing 3 file(s) ...
00:22:29.333 [info] Starting logger for 'Tezz'
00:29:00.425 [info] GET /import
00:29:00.434 [warn] Elixir.Cldr.NoMatchingLocale: No configured locale could be matched to "sv-se"
00:29:00.449 [info] Sent 200 in 22ms
00:29:02.618 [info] GET /import
00:29:02.623 [warn] Elixir.Cldr.NoMatchingLocale: No configured locale could be matched to "sv-se"
00:29:02.642 [info] Sent 200 in 23ms
00:38:59.704 [info] Import complete!
00:42:34.724 [info] GET /import
00:42:34.732 [warn] Elixir.Cldr.NoMatchingLocale: No configured locale could be matched to "sv-se"
00:42:34.755 [info] Sent 200 in 29ms
00:42:40.057 [info] GET /import
00:42:40.064 [warn] Elixir.Cldr.NoMatchingLocale: No configured locale could be matched to "sv-se"
00:42:40.093 [info] Sent 200 in 36ms

The blue icon is meant to indicate that the file was skipped because data already exists for an earlier period. I'd recommend to restore the backup that was made before importing the first CSV file first. Afterwards import all CSV files at once.

Hi! thank you I have succedded importing all teslafi history.
I just did a docker system prune and compose again and started off importing the data.
thank you!

Any idea why most of these TeslaFi imports are being skipped? I uploaded all of the CSV files to the import folder and ran the import, but it seems like it failed after the first two.


Bug check rf dependency is set and fix wrong throw status

What problem does this PR solve?

check rf dependency is set
fix wrong throw status
catch throw exception on buildsink::close

Release note
None
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[ ] No.
[ ] Yes.

Does this need documentation?

[ ] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall


bzrtp: 4.4.9 -> 4.4.24

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/bzrtp/versions.
meta.description for bzrtp is: "An opensource implementation of ZRTP keys exchange protocol"
meta.homepage for bzrtp is: "https://gitlab.linphone.org/BC/public/bzrtp"
meta.changelog for bzrtp is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 4.4.24 with grep in /nix/store/y50p1lxfalmq4ssw6shi7k8dyjpp7j3v-bzrtp-4.4.24

directory tree listing: https://gist.github.com/e3c8baded8ea830f70c21d40d92ec1f2

du listing: https://gist.github.com/a85c87b734b83c5403f7b4ff61237e21

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

19 total rebuild path(s)

6 package rebuild(s)

6 x86_64-linux rebuild(s)
6 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
6 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
bzrtp
liblinphone
linphone
mediastreamer
mediastreamer-openh264
msilbc

Instructions to test this update (click to expand)

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

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

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

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

6 packages built:

bzrtp
liblinphone
linphone
mediastreamer
mediastreamer-openh264
msilbc

Maintainer pings
cc @jluttine for testing.

Closing in favor of #116768


Port to Scala.js

java.io.File is now scala.meta.io.AbsolutePath.
The io module abstracts over the platform, making it possible
to hook in node.js modules to write JavaScript command-line apps.
The Input.File(file: java.io.File) constructor is still kept,
usage of those constructors will lead to linking errors in a JS app.
The scalametaJS/test will very likely guard us against using those
constructors in the implementation of the scalameta projects.
The JVM api remains otherwise unchanged :tada:
Lit.{Float,Double}.syntax is different on JVM and JS. Still WIP.
Binary serialization/deserialization of semanticdb is available on
both JVM and JS, thanks to Scala.js support in scalapb.
WeakHashMap is replaced with java.util.Map on JS, we could port the
implementation to js.WeakMap if memory/GC becomes a problem in
scala.meta js applications.
Reflection tests only run on the JVM.
Scalahost remains JVM only.

Okay, I think I've managed to clean up everything unexpected that came up in CI :)
For the record, the CI was green at least once here: https://platform-ci.scala-lang.org/scalameta/scalameta/11 The latest commits have been to clean up some remaining quirks in the port.
It would be great to try and merge this before we merge other large changes in the repo. This PR moved most .scala files in the repo so I'd prefer to avoid conflict hell when rebasing.

The CI is green now. The tests failed last night because of OOM errors.

The most urgent issue is to fix https://github.com/scalameta/scalameta/issues/762 I gave it a try but hit on cryptic errors in the @ast macros.

I added an experimental solution for Lit.Float/Doublein https://github.com/scalameta/scalameta/pull/761/commits/4649537366ced32647ec79214cfe1d618d6c9db4 the tests are still failing, just wanted to "float" the idea around 😅

I fixed the Lit.Float/Double.syntax problem by making Lit.Float/Double accept a string. All unit tests pass now on JS and JVM. This PR is ready for another round of review @xeno-by

I took the liberty to fix the previously broken pretty printing for doubles and floats,
// before
Lit.Float(Double.NaN).syntax == "NaNd"
// after
Lit.Float(Double.NaN).syntax == "Double.NaN"

@olafurpg Some of my comments are marked as outdated. No idea why, because I've just submitted them :)

I've updated the PR address all points except the custom Lit.Float/Double.unapply.
Github review UI is not suited for larger PRs like this :'(

It seems the new Lit encoding doesn't work yet with macros, I bumped into this error while porting scalafmt to scala.js
ConfigReader.scala:50: scala.meta.Lit.type does not take parameters
[error]       val x = q"""val x = "string""""
[error]               ^

False alarm, the quasiquote error came from a half-baked local release. It seems to work fine with the new Lit.X encoding :)
I'm still gonna take the liberty to add Lit.apply(Int/String/Double/...) methods for backwards source compability. I believe it will be easy to then provide a scalafix rewrite to migrate existing code to the new encoding.

Let's merge then :) LGTM! 👍


mvn clean compile jib:build not work for war project.

Environment:

Jib version:  3.0.0
Build tool: maven 3.6.3,Java version: 1.8.0_282,
OS:  ubuntu20.04

Description of the issue:
mvn clean compile jib:build  not work for war project.
have to package the war project. mvn clean package jib:build
The documents and the log output do not mention that.
Expected behavior:
work as jar project.
Steps to reproduce:

create simple springweb  app in https://start.spring.io/; set the package to war
run mvn clean compile jib:build

Log output:
Obtaining project build output files failed; make sure you have compiled your project before trying to build the image. (Did you accidentally run "mvn clean jib:build" instead of "mvn clean compile jib:build"?)

Yeah, the message is misleading. (It used to be mvn clean package jib:build, but I guess this is a regression.)
BTW, I highly recommend <packaging>jar instead of war if feasible.

Hi. I am new to this space, and  would like to pick up this issue.

Thank you @chanseokoh . I will take a look into it :)

/take

@chanseokoh @saptaswapal I noticed this task hasn't been worked on for some time now and I'd like to contribute. May I take this over?

I assumed I can start working on the task and submitted a PR. @chanseokoh, feel free to  review at your free time! #3306


It takes forever to run ollama (llama2 or mistral) in wsl

Hi I just installed ollama in wsl using curl. I tried ollama run llama2 and ollama run mistral.
It doesn't enter into the prompt. A tiny rotating snake like objects appears in the terminal and it has been several hours like this. Nothing much happened. please help me fix it.
My system:
Intel i7
RTXA2000 Laptop GPU : 4GB vRAM (0 GB used)

Hi @allmin
I think that you don't have enough memory. 4GB is nice, but not for IA. Try phi model. It's only a 1.6GB Gigabit file size, but can take more in GPU.
ollama run phi

Thank you! phi works!, I was indeed looking for smaller models but the
model library page: library (ollama.ai) https://ollama.ai/library was
lacking the information on memory requirements.
On Thu, Dec 21, 2023, 11:00 PM Igor Schlumberger @.***>
wrote:

Hi @allmin https://github.com/allmin
I think that you don't have enough memory. 4GB is nice, but not for IA.
Try phi model. It's only a 1.6GB Gigabit file size, but can take more in
GPU.
ollama run phi
—
Reply to this email directly, view it on GitHub
https://github.com/jmorganca/ollama/issues/1654#issuecomment-1866982942,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACNQU4PUQTMGRZC7CAJLGEDYKSWPHAVCNFSM6AAAAABA6K6FKKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRWHE4DEOJUGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

BTW, It'd be helpful to also share here that I tried running llama2 through docker and it works perfectly on my 4GB GPU (Nvidia RTXA2000)!


Expand ability to import LAS point cloud format

Tool to convert a las file into an entwine data structure that potree can use:
https://entwine.io/en/latest/quickstart.html#building-the-data


Update to latest shipyard/admiral versions

Signed-off-by: Janki Chhatbar<EMAIL_ADDRESS>

Superseded by https://github.com/submariner-io/submariner/pull/1314.


net/url: RawPath unset even though Path differs from input

What version of Go are you using (go version)?

$ go version
go version go1.16 darwin/amd64

Does this issue reproduce with the latest release?
✔
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/caselman/Library/Caches/go-build"
GOENV="/Users/caselman/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/caselman/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/caselman/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.16"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/caselman/code/goepf/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/t9/cyl4762s64d7mcb1zg67yt1r0000gn/T/go-build1746458354=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

https://play.golang.org/p/RTw-u_iiYfK
package main

import (
	"fmt"
	"net/url"
)

func main() {
	a,_ := url.Parse("file:///some%20path")
	fmt.Println(a.Path, a.RawPath)

	b,_ := url.Parse("file:///some%2Fpath")
	fmt.Println(b.Path, b.RawPath)

}

What did you expect to see?
/some path /some%20path
/some/path /some%2Fpath

What did you see instead?
/some path
/some/path /some%2Fpath

I did not expect RawPath to be unset in the first case, when the value set in Path is different than the input to setPath
Wondering if https://github.com/golang/go/blob/master/src/net/url/url.go#L669 is meant to be:
if escp := escape(p, encodePath); p == escp {

and if not, why the discrepancy based on the input path containing a %20 vs a %2F?
Thanks

cc @neild @rsc

I think it works as documented
https://pkg.go.dev/net/url#URL.EscapedPath

In general, code should call EscapedPath instead of reading u.RawPath directly.

The EscapedPath returns escaped value in both cases:
https://play.golang.org/p/8eY6NeIrkle

RawPath is only set when it's different from the output of EscapedPath, which is what you should be using. It is documented to only be a hint:

RawPath     string    // encoded path hint (see EscapedPath method)

Closing as working as intended.


Could not add github status. 403: Resource not accessible by integration

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Running bundlesize on Github Actions on a PR from fork (with env set GITHUB_TOKEN) fails with:
> web3.js@ bundlesize /home/runner/work/web3.js/web3.js
> bundlesize

 ERROR  Could not add github status.
        403: Resource not accessible by integration

If the current behavior is a bug, please provide the steps to reproduce.
See ci run here: https://github.com/ethereum/web3.js/pull/3622/checks?check_run_id=847192039#step:6:224

What is the expected behavior?
Should work as expected
Please mention other relevant information.

node version

12.x

npm version

latest

Operating system

linux

bundlesize version

0.18.0

CI you are using

Github Actions

Hey!
Forks are tricky because they don't get access to environment tokens of the source repository. Can you confirm that the action has the required token set?

hey @siddharthkp, thanks for your quick response :)
yes forks are tricky, but since I saw that GITHUB_TOKEN is supported in token.js I thought it would work across forks. e.g. that's how our coveralls github action works. we are passing GITHUB_TOKEN as an env in this step which then runs bundlesize here.
thanks for the link to bundlesize2, looks great and i didn't see it before!


Universal build fails with warning SHAs are not identical

Error: Expected all non-binary files to have identical SHAs when creating a universal build but "Contents/Frameworks/Electron Framework.framework/Versions/A/Libraries/vk_swiftshader_icd.json" did not

Which is weird, since when running shasum manually it tells me that the files are identical:
shasum Salonhub-darwin-*64/Salonhub.app/Contents/Frameworks/Electron\ Framework.framework/Versions/A/Libraries/vk_swiftshader_icd.json
8734c383f004e365904068c9cde7999aab0b54c6  Salonhub-darwin-arm64/Salonhub.app/Contents/Frameworks/Electron Framework.framework/Versions/A/Libraries/vk_swiftshader_icd.json
8734c383f004e365904068c9cde7999aab0b54c6  Salonhub-darwin-x64/Salonhub.app/Contents/Frameworks/Electron Framework.framework/Versions/A/Libraries/vk_swiftshader_icd.json

It appears the problem is present in 1.0.2 and 1.0.3. Rolling back to 1.0.1 fixes the issue.

Hm, I wonder if my rewrite to make the sha method async by using streams has a flaw.  It worked with my local apps

A fix is available in PR #11 😀

Fixed in  477a52e

Contents/_CodeSignature/CodeResources always different

Are you perhaps signing the binaries first? That would cause them to be different.
The universal binary should be created from unsigned arm64 and x64 binaries. And finally the universal binary needs to be signed and notarized.

no, not signed first, i see the xml in Contents/_CodeSignature/CodeResources , they are always different
 		<key>Frameworks/Electron Framework.framework</key> 		<dict> 			<key>cdhash</key> 			<data> 			zftqvls6kWOPLaw3yZLtpnJCzFs= 			</data>

i think : they come from different release https://github.com/electron/electron/releases/download/v11.0.3/electron-v11.0.3-mas-arm64.zip and https://github.com/electron/electron/releases/download/v11.0.3/electron-v11.0.3-mas-x64.zip

it seems electron-builder will sign...

but, how can i get the .app before electron-builder build

but, how can i get the .app before electron-builder build

You can use electron-packager for generate .app file.


mavros interface for trajectory tracker

Hi,
I want to pass the SO3 command to the FCU via mavros, I checked the AttitudeTarget message, but that message only supports normlised thrust in z-axis, but SO3Command has forces in x, y and z axis. how can I convert SO3Command to Attitude target message @kartikmohta
thanks

This is an old thread so hopefully you have gotten the info you need, but I'm putting a response here in case someone has the same question.
This should be handled already in kr_mavros_interface. Check out line 123 in particular. You will need to update the kf parameter based on your robot. https://github.com/KumarRobotics/kr_mav_control/blob/master/interfaces/kr_mavros_interface/src/so3cmd_to_mavros_nodelet.cpp

Thanks @jessweakly!!