Pod GPU Metric Docs

Is it possible to export gpu metrics per pod? Is the pod gpu metric exporter still in this repo? I can't seem to find any information about it anymore. Is that a change with dcgm 2.0 ?
The instructions link on master readme is broken where should it be pointing to? https://github.com/NVIDIA/gpu-monitoring-tools/tree/master/#dcgm-exporter

Hello!
I've updated the front page with an example of the pod mapping when a GPU is used: https://github.com/NVIDIA/gpu-monitoring-tools/tree/master/#quickstart-on-kubernetes

Hello!
I've updated the front page with an example of the pod mapping when a GPU is used: https://github.com/NVIDIA/gpu-monitoring-tools/tree/master/#quickstart-on-kubernetes

Hello, I can't seem to find the mentioned example? was the page edited again?


Charts[Network]: remove mirror node and mirror node explorer sub charts and related settings and files

relates to: https://github.com/hashgraph/solo/issues/617
blocked by: - blocked by: https://github.com/hashgraph/hedera-mirror-node-explorer/issues/877

migrated to solo-charts: https://github.com/hashgraph/solo-charts/issues/18


ZEVA - Incorrect credit balance being displayed for Porsche on organisation page

Describe the Bug
On the supplier index page in prod:
https://zeroemissionvehicles.gov.bc.ca/organizations
As per the screen shot below the credit balance of B credits for Porsche is wrong. It is showing a deficit of [] B credits. Deficit are shown in brackets.

Expected Behaviour
The Vehicle Supplier index page should show the current and correct credit balance for all suppliers. For Porsche their B credit balance should be zero (displayed as a dash). On the Supplier Credit Transactions list page the value is correctly being shown as 0. Investigate the discrepancy and fix the Vehicle Supplier index page to show the correct '0' value.
Actual Behaviour
The balance of B credits being displayed for Porsche is incorrectly showing a deficit.
Implications
Not accurately displaying the number of credits held by suppliers undermines the credibility and usefulness of the ZEVA system.
Steps To Reproduce
Steps to reproduce the behaviour:
User/Role:

Go to https://zeroemissionvehicles.gov.bc.ca/organizations
Scroll down to Porsche
See incorrect B deficit credit balance

Based on my investigation here are my findings:

When the vehicle supplier does any credit transaction the transaction value gets inserted in the credit_transaction table in the DB and update the remaining balance in the account_balance table.
The latest value for Credit A and Credit B on the Vehicle supplier Index page is fetched from the account_balance table. However, on the Supplier Credit Transactions list page, the values are fetched from the credit_transaction table and calculated in the front end before being displayed to the user. This leads to a discrepancy in the values displayed on the two pages.
The backend calculation is error-free, and there are no validation errors in the front end. However, the balance might have been updated erroneously, leading to the current issue. Notably, this issue is only affecting the Porsche vehicle supplier and not any other supplier. To resolve the issue, updating the balance for Porsche for credit B to "0" could be a possible solution.


onError -- reports back URL

Hi,
Great tool. Thank you for providing it.
I am using it in Android, but I am having trouble.
I get an error message. In the onError callback shown below I get the message as the URL. It simply spits out the primary URL path. Callback is shown below, and so is the logged error message.
                @Override
                public void onError(SpeedTestError speedTestError, String errorMessage) {
                    Log.d("onError", "[ERROR] " + errorMessage);
                }

E.g. error message:
05-12 20:01:06.749 2583-3688/com.google.uxbandit D/onError: [ERROR] ipv4.bouygues.testdebit.info
What does it mean to get such an error message? When I access the URL i actually provide "http://ipv4.bouygues.testdebit.info/1M.iso" to start download it all works well elsewhere, as in I am able to download the file through my computer, as well as the Android emulator.
Thanks,
vj.

Hello, can you print the speedTestError to see what type of error shows ?

Hi,
I am running the app on the emulator and it looks like android emulator is having trouble resolving the hostname, as you can see in the attached image:

On May 13, 2017, at 1:04 AM, Bertrand Martel<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
Hello, can you print the speedTestError to see what type of error shows ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/bertrandmartel/speed-test-lib/issues/42#issuecomment-301228001, or mute the thread https://github.com/notifications/unsubscribe-auth/AGfhlCP5r6_VTGnIUkdoQuTox2j4Ial_ks5r5UfWgaJpZM4NZ6sW.
https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png https://github.com/bertrandmartel/speed-test-lib https://github.com/bertrandmartel/speed-test-lib/issues/42#issuecomment-301228001

Ah fixed it — needed to add the second line for instance in addition to the internet only .. odd.
<uses-permission android:name="android.permission.INTERNET" />
<uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />

On May 13, 2017, at 5:07 AM, Vijay Janapa Reddi<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
Hi,
I am running the app on the emulator and it looks like android emulator is having trouble resolving the hostname, as you can see in the attached image:
<PastedGraphic-1.png>

On May 13, 2017, at 1:04 AM, Bertrand Martel<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
Hello, can you print the speedTestError to see what type of error shows ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/bertrandmartel/speed-test-lib/issues/42#issuecomment-301228001, or mute the thread https://github.com/notifications/unsubscribe-auth/AGfhlCP5r6_VTGnIUkdoQuTox2j4Ial_ks5r5UfWgaJpZM4NZ6sW.
https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png https://github.com/bertrandmartel/speed-test-lib https://github.com/bertrandmartel/speed-test-lib https://github.com/bertrandmartel/speed-test-lib https://github.com/bertrandmartel/speed-test-lib https://github.com/bertrandmartel/speed-test-lib/issues/42#issuecomment-301228001 https://github.com/bertrandmartel/speed-test-lib/issues/42#issuecomment-301228001 https://github.com/bertrandmartel/speed-test-lib/issues/42#issuecomment-301228001 https://github.com/bertrandmartel/speed-test-lib/issues/42#issuecomment-301228001

Maybe it's the same as #4 , specify the DNS server with -dns-server <IP_ADDRESS>,<IP_ADDRESS>. But there is no need for ACCESS_NETWORK_STATE. Note that I had this issue with Android Studio < 2.3 and specifying network speed as other than Full, if you don't mind try change network speed to full in your emulator configuration.


Usb-Midi  when Target API level is 31

When switching to Target API level 31 it stops working.
It seems that there are no events at all.
With Api level 29 it works well.
any ideas how to fix it ?
Thanks.

Could you give me more details please ?

Are you using the plugin with Unity or only with Java ?
Are you receving the events for device attached / detached ?
Are you receving the pending intent popup for using the usb port ?

Hi,
Setting FALG_MUTABLE resolved issue with compatibility with newest Android versions, but there still issue that after NoteOff event also coming NoteOn in the next packet.
PendingIntent permissionIntent = PendingIntent.getBroadcast(_context, 0, new Intent(USB_PERMISSION_ACTION), FLAG_MUTABLE);

About above issue with NoteOn/NoteOff, adding break fix this problem:
  switch (CIN)                     {                         case 0x8:                             _midiCallback.NoteOff(byte2);                             break;                         case 0x9:                             if(byte3 == 0x00) {                                 _midiCallback.NoteOff(byte2);                             }                             else{                                 _midiCallback.NoteOn(byte2, byte3);                             }                     }

Hello,
I have fix the issue with your code in the last commit. But I didn't test it because I don't have my piano, can you test it for me ?

yes, this fix is already resolved, thanks! but I still need to update PendingIntent.FLAG_MUTABLE to make it working on API 33.
and there also issue with gradle, which I've resolved by replace task clean with following code:
clean.doFirst {     delete rootProject.buildDir }

Commited ! 7ea827662a9ebf16173dc57bb01622066bb47fb0


Revert "wait for ping OK to majority of configserver hosts"

Reverts vespa-engine/vespa#27089
@hakonhall @arnej27959

Seems servers got stuck in publiccd.


Change hover time

Really like this addon, but have 2 questions. How can i set the timer for the hover effect faster, it to slow for poping up. Im talking about the warning icons in the main window.
Also i saw on main page there is inline documentation. How or where can i view this?

Hi bazitur,
congrats for your extension for brackets
I have the same question has @schroef :
Ho can I view inline documentation?
Autocompletion and linting works fine
I already installed python 3.6.3, flake8 and docutils

Hello,
I've fixed the bug. Try updating the extension and pressing Ctrl + K.

ctrl + K on OSX deletes a line
cmd + D shows hover error "No quick docs available"

Hi @schroef,
It is done so that Quick Docs are available only when function or class is under the cursor. See:

Ps i think you adjusted the quickdos window, thats the window with the hints right?
It does seem to work when i go to location type, go to other location and return and press cmd + K
What i meant by hover time was for the error and warnings in the gutter

its the one which shows error.html cant seem to find something about time

PS i read something about you wanting to speedup the parsing. Isnt it possible to get the window height of the brackets. I think if you now the height you now howmany lines are visible. When you calculate the difference you can let the parser only do that section.
idea???

@bazitur  Where is that setting?
Overlooked that message :)

new version installed, was needed to remove your extension manually because Brackets gave error for update your extension, and now the inline documentation works fine. It will be nice if you can add a setting for users can change the font size because is the used bigger
The delay of autocompletion window, for me, is too long. Could it's possible to add  a setting for users can change the delay time too?
Thanks!

@TechJA,
The problem is that autocompletion is done via python script running in another process. Brackets API provides connection to it through Node, and there's probably no way for route Brackets → pipe → Node → pipe → Python → pipe → Node → pipe → Brackets to be faster. The bottleneck is asynchronous pipe between Node and Brackets.

Thanks for the reply
understood, no problem ;)

So hover time cant be changed, where is it controlled? Something most control the time right?

@schroef,
It simply cannot be faster because of Brackets' delay while communicating with Node.

oh thats a pitty then, to bad.


Navbar fixed

#289
Navbar Fixed

@Anjalis14 Already fixed. Not needed 😂

@swaraj-das you just assign me that thats why i did

@swaraj-das you just assign me that thats why i did

You created the issue a while ago, but before you could make a PR, other contributors submitted their PRs and resolved the issue


ArcGIS Editor for OpenStreetMap 10.8 (64-bit) Trojan Alerts

Guys i just upload the setup.exe file to virus total for (ArcGIS Editor for OpenStreetMap 10.8 (64-bit)\setup.exe) check. VT is prompting trojan alerts for 3 engines. I am not sure is this due to VT is getting confused as i have seen previous version also had similar issue and later got whitelisted by Mcafee. But this time it is alarmed by yandex,Zilya and jiangmin engines. I am new to this and anyone happy to share some knowledge on this at all. Thanks
ArcGIS_Editor_OSM_10_8Desktop.zip

@malfdo I'm looking into this, I'll let you know what I find. Do you get this error with the .zip downloaded from both Github and ArcGIS Online?

@malfdo I'm looking into this, I'll let you know what I find. Do you get this error with the .zip downloaded from both Github and ArcGIS Online?

Hello Steve;
Thanks for getting back to me on this. I only downloaded this from here
https://www.arcgis.com/home/item.html?id=2f58a77a65c546149dd1ea425a680ad3
I am only working by the information i got from one of our user who want to use this software.
Kind regards
Malaka Fernando
On Thursday, 28 January 2021, 03:24:45 am AWST, Steve Moore<EMAIL_ADDRESS>wrote:
@malfdo I'm looking into this, I'll let you know what I find. Do you get this error with the .zip downloaded from both Github and ArcGIS Online?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hello Steve;
Thanks for getting back to me on this. I only downloaded this from here
https://www.arcgis.com/home/item.html?id=2f58a77a65c546149dd1ea425a680ad3
I am only working by the information i got from one of our user who want to use this software.
Kind regards
Malaka Fernando
On Thursday, 28 January 2021, 03:24:45 am AWST, Steve Moore<EMAIL_ADDRESS>wrote:
@malfdo I'm looking into this, I'll let you know what I find. Do you get this error with the .zip downloaded from both Github and ArcGIS Online?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


member_limit on createChatInviteLink doesn't work

I am using the latest version of the library.
And when i call the createChatInviteLink  i use member_limit only 1 person, but when i shared the link to another user of telegram,
he can open the link of group..
Is this a bug of telegram?

Invite links have a curious use case. Imagine that you create a link for 1 person and that person uses it by entering the group (the link reached its maximum of 1 use), but then this same person leaves the group. Well, now the link has an available use again.
In my opinion, the best thing is that you put an expiration date on the link and as soon as the user enters, delete that link.


Update Spring property description

Description
Fix https://github.com/Azure/azure-sdk-for-java/issues/39084
update max-concurrent-calls property
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


python38Packages.spyder-kernels: 1.10.3 -> 2.1.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.spyder-kernels is: "Jupyter kernels for Spyder's console"
meta.homepage for python38Packages.spyder-kernels is: "https://docs.spyder-ide.org/current/ipythonconsole.html"
meta.changelog for python38Packages.spyder-kernels is: "https://github.com/spyder-ide/spyder-kernels/blob/master/CHANGELOG.md"
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

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 2.1.0 with grep in /nix/store/0lzc0zkyzs6cbw25nlh7dfnhklf171fk-python3.8-spyder-kernels-2.1.0

found 2.1.0 in filename of file in /nix/store/0lzc0zkyzs6cbw25nlh7dfnhklf171fk-python3.8-spyder-kernels-2.1.0

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

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

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

15 total rebuild path(s)

4 package rebuild(s)

4 x86_64-linux rebuild(s)
4 i686-linux rebuild(s)
3 x86_64-darwin rebuild(s)
4 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
cq-editor
python38Packages.spyder
python38Packages.spyder-kernels
python39Packages.spyder-kernels

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/0lzc0zkyzs6cbw25nlh7dfnhklf171fk-python3.8-spyder-kernels-2.1.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python38Packages.spyder-kernels https://github.com/r-ryantm/nixpkgs/archive/2bccdaea5e40098a46b0cb5b0649e8689c3b5983.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/0lzc0zkyzs6cbw25nlh7dfnhklf171fk-python3.8-spyder-kernels-2.1.0
ls -la /nix/store/0lzc0zkyzs6cbw25nlh7dfnhklf171fk-python3.8-spyder-kernels-2.1.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 45m and timed out

Maintainer pings
cc @gebner for testing.

Result of nixpkgs-review pr 135289 at 2bccdaea run on aarch64-linux 1

1 package failed to build:

python38Packages.spyder

1 package skipped due to time constraints:

cq-editor

2 packages built successfully:

python38Packages.spyder-kernels
python39Packages.spyder-kernels

1 suggestion:

warning: python-include-tests
Consider adding a checkPhase for tests, or if not feasible, pythonImportsCheck.
Near pkgs/development/python-modules/spyder-kernels/default.nix:25:0:
   |
25 |     description = "Jupyter kernels for Spyder's console";
   | ^

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

https://github.com/NixOS/nixpkgs/commits/master/pkgs/development/python-modules/spyder-kernels/default.nix
already in master


Metrics / Exception Reporting - LetsEncrypt / Dynect flow, ELB source sync

Provide better metrics and exception handling for LetsEncrypt flow, particularly with Dyn DNS.
For source-syncing, provide more details via metrics/sentry

Coverage decreased (-0.2%) to 61.114% when pulling e2b59622819f565f5c5e58a2ebcfd8ed3e3335f3 on castrapel:metrics_errors_acme into 88ed9581b64b8504755e7957c938cfd96d31c538 on Netflix:master.


Is attention-mask needed in pre-training in tokenize_function_seq_desc?

While padding="max_length", truncation=True, I feel that it would be better to use an attention mask here, right?

Hello:
This function is only used for biotranslation task in pre-training and will not be used in fine-tuning.

Sorry，I mean pre-training

Thank you for bringing this to our attention! You're right about the need for an attention mask for biotrainslation task in pre-training. This oversight is a bug in our code.
We appreciate your contribution in identifying this issue, and we will work on fixing it promptly. Your feedback helps us improve. Thanks again!

I add attention_mask to batch in the newest commit.
Thanks for your kindly help~


Using multiple providers

I have a project that uses both Node.js and Python. With buildpacks, I could specify heroku/nodejs and heroku/python and it would do the right thing (Node build/install followed by Python build/install).
With nixpacks, the project is identified as Node.js automatically and does the right thing there, but if I add --pkgs=python39 to nixpack build it only installs Python, but doesn't use the default build/install command.
Is there a way to "stack" providers similar to how it is done with buildpacks? If not, that would be nice to have so I don't have to manually specify the additional build/install commands.

Is there a way to "stack" providers similar to how it is done with buildpacks? If not, that would be nice to have so I don't have to manually specify the additional build/install commands.

ATM no, you'd have to manually define those additional commands which I could see as being not so great. Simply haven't run into in a way that doesn't "rollup" to framework specific handling (e.g Ruby runs some node stuff but that's invoked from the Ruby provider)

I don't want to jinx it, but we might be able to accomplish this by simply:

Changing [matched_provider](https://github.com/railwayapp/nixpacks/blob/59ba5914e411b778b015da4bb6374826ffc16825/src/nixpacks/plan/generator.rs#L89) from an Option to a Vec
Writing a function to "join" each stage

Would be open to a PR or we can look at it once there's more demand in that category

This is a pretty common use case for Django (and I assume Python in general). Use a standard Node.js pipeline for your static frontend assets and then build your Python project. For Django, you'd typically run manage.py collectstatic --noinput at the end to grab all the Node-built files and put them where Django expects. The Heroku buildpack runs this collectstatic step by default, but it has a toggle to turn it off because it can be problematic in some scenarios.

I think that's a common usecase for many web related projects.
One half the actual software and the other half some nodejs to build assets.

This is a pretty common use case for Django (and I assume Python in general). Use a standard Node.js pipeline for your static frontend assets and then build your Python project. For Django, you'd typically run manage.py collectstatic --noinput at the end to grab all the Node-built files and put them where Django expects. The Heroku buildpack runs this collectstatic step by default, but it has a toggle to turn it off because it can be problematic in some scenarios.

Right. Like I said, the Ruby/Rails Provider generally handles this, so I wouldn't be opposed to adding one of:

Logic for multiple providers to contribute
The above into the Django provider itself

I will start working on this soon. It is a bit more complicated then just converting it to a Vec and the ordering of these providers will be important.
It would also be nice if you could specify the exact providers to use via a config variable or something.


[Bug] histogramChart crosshair

Version
1.1.0-beta2
Link to Minimal Reproduction
null
Steps to Reproduce
https://www.visactor.io/vchart/demo/histogram-chart/histogram-different-bin
Current Behavior
最后一根柱子的crosshair区域不对
Expected Behavior
最后一根柱子的crosshair区域应该显示正确
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

This was caused by the fieldX2 does not participate in data statistics, resulting in incorrect scale domain of the x-axis.


Problem Android - not showed map when i try getMyLocation

I'm submitting a ...  (check one with "x")

[x] question
[ ] any problem or bug report

OS:  (check one with "x")

[x] Android
[ ] iOS
[ ] Browser

cordova information:  (run $> cordova plugin list)
com.googlemaps.ios 3.8.0 "Google Maps SDK for iOS"
cordova-plugin-device 2.0.2 "Device"
cordova-plugin-geolocation 4.0.2 "Geolocation"
cordova-plugin-googlemaps 2.7.1 "cordova-plugin-googlemaps"
cordova-plugin-inappbrowser 3.2.0 "InAppBrowser"
cordova-plugin-ionic-keyboard 2.2.0 "cordova-plugin-ionic-keyboard"
cordova-plugin-ionic-webview 4.2.1 "cordova-plugin-ionic-webview"
cordova-plugin-network-information 2.0.2 "Network Information"
cordova-plugin-splashscreen 5.0.2 "Splashscreen"
cordova-plugin-statusbar 2.4.2 "StatusBar"
cordova-plugin-whitelist 1.3.3 "Whitelist"
cordova-sqlite-storage 5.0.0 "Cordova sqlite storage plugin - cordova-sqlite-storage plugin version"

If you use @ionic-native/google-maps, please tell the package.json (only @ionic-native/core and @ionic-native/google-maps are fine mostly)
@ionic-native/core : ^5.26.2
@ionic-native/google-maps : ^5.5.0

Current behavior:
I´m trying to show a map with my location with Ionic.
On Apple, it´s seems ok, but in android the map show blank when i try this:
this.map.getMyLocation().then((location: MyLocation) => {
//...
}).catch(err => {
this.showToast(err.error_message);
});
On Android studio the map is blank, and the Toast is not showed. I can see on logcat
D/PluginLocationService: ---->The last location is expired. Let's get the latest location...
D/PluginLocationService: ---->_requestLocationUpdate (isRetry = true)
On Android Device, the app show me the toast with the text "Since this device does not have any location provider, this app can not detect your location."

Please share your project files on Github repository (not zip file or Google Drive etc)

Since this device does not have any location provider, this app can not detect your location

As the message says your Android device does not have GPS receiver or network is offline

Oh, I´m sorry but when i activated the location on the device, everything has worked. (Thanks wf9a5m75)
But in the emulator, the location is active, the app show me: "Allow APP to access this device´s location?". Next I touch on ALLOW button, but the map dosent works
The console said: "The last location is expired. Let's get the latest location".
I can see that this message is from:
https://github.com/mapsplugin/cordova-plugin-googlemaps/blob/master/src/android/plugin/google/maps/PluginLocationService.java#L459
If in the same emulator I open google maps app, this app has worked, but my app continue not working

If you find a problem, please fix the code, then send your code as pull request. That's open source contribution :)

Any updates?

Hi again.
I just upload a clean project in https://github.com/u01ser/test-map00 (obviusly i delete API KEY)
On iOS, everything works.
On Android, the  maps when i use:
this.map.getMyLocation()
The logcat shows:
Android Studio said:
2020-06-18 10:12:07.440 19132-19132/io.ionic.starter D/PluginLocationService: ===> onConnected  PluginLocationService1079793405
2020-06-18 10:12:07.440 19132-19132/io.ionic.starter D/PluginLocationService: --->regularAccuracyRequestList.size = 1
2020-06-18 10:12:07.440 19132-19132/io.ionic.starter D/PluginLocationService: ---->_requestLocationUpdate (isRetry = false)
2020-06-18 10:12:07.451 19132-19132/io.ionic.starter D/PluginLocationService: --->highAccuracyRequestList.size = 0
2020-06-18 10:12:07.513 19132-19132/io.ionic.starter D/PluginLocationService: ---->The last location is expired. Let's get the latest location...
2020-06-18 10:12:07.513 19132-19132/io.ionic.starter D/PluginLocationService: ---->_requestLocationUpdate (isRetry = true)
And the map is shown blank.
If i don't use "his.map.getMyLocation()" everything is ok.

Please try to insert this.platform.ready() before initmap()

Please try to insert this.platform.ready() before initmap()

https://github.com/u01ser/test-map00/commit/ac2c97e705eb1b3b2fad66bb2a966eaec92e24a8
Unfortunately, it still not works. Same results

Ok, I'll check it later

Hmm, I have no problem

Ok, i just install a APK in the device and all works fine. Is strange, i think is problem of my emulator, and the plugin its ok.
How do you made the gifs? I think is very util for me in next times.

Record from Android Studio, then save a mp4 file. After that, use online free video to gif service.


[PE-D][Tester A] Archiving Data Files feature is empty

Upcoming feature is empty, can include something for users to look forward to.

Labels: type.DocumentationBug severity.VeryLow
original: Darren159/ped#15

Add a short description in the UG


CLOUDSTACK-10032 : Database entries for templates created from snapshots disappear after management-server service restart

ISSUE
This issue occurs only with KVM hypervisor. Database entries for templates created from snapshots disappear after management-server service restart
STEPS TO REPRODUCE

Create a ACS setup and add KVM hypervisor as host.
Create snapshot of any disk (root or data disk) of an instance.
Create template using disk snapshot.
Verify that template got downloaded completely and is in Ready state.
Also, verify that entry for this template is present in template_store_ref table in database.
Now restart management server.
Once management server is restarted completely and web UI is available, check the template status. It will be in Active state instead of downloaded.
Also, entry for this template vanishes from template_store_ref table in database.

Fix for the Issue
In NfsSecondaryStorageResource.java class, inside method copySnapshotToTemplateFromNfsToNfs() bufferwriter which was created for writing data in template.properties file is not closed and hence few properties were not getting written in template.properties. As few properties were absent in template.properties file, so after management server restart, this template is not loaded and hence it goes into Active state.
Screenshot of Template.properties before fix :

Screenshot of Template.properties after fix :

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-926

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-1351)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 37515 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2224-t1351-kvm-centos7.zip
Intermitten failure detected: /marvin/tests/smoke/test_network.py
Intermitten failure detected: /marvin/tests/smoke/test_nic.py
Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermitten failure detected: /marvin/tests/smoke/test_routers.py
Intermitten failure detected: /marvin/tests/smoke/test_templates.py
Intermitten failure detected: /marvin/tests/smoke/test_vpc_router_nics.py
Test completed. 51 look OK, 6 have error(s)

Test
Result
Time (s)
Test File

test_01_VPC_nics_after_destroy
Failure
378.14
test_vpc_router_nics.py

test_06_router_advanced
Failure
0.02
test_routers.py

test_05_router_basic
Failure
0.03
test_routers.py

test_04_rvpc_privategw_static_routes
Failure
391.25
test_privategw_acl.py

test_04_extract_template
Error
5.10
test_templates.py

test_03_delete_template
Error
5.08
test_templates.py

test_01_create_template
Error
50.49
test_templates.py

test_09_reboot_router
Error
5.11
test_routers.py

test_08_start_router
Error
45.41
test_routers.py

test_04_restart_network_wo_cleanup
Error
40.37
test_routers.py

test_03_restart_network_cleanup
Error
65.53
test_routers.py

test_03_vpc_privategw_restart_vpc_cleanup
Error
320.17
test_privategw_acl.py

test_01_nic
Error
167.41
test_nic.py

test_reboot_router
Error
257.56
test_network.py

test_change_service_offering_for_vm_with_snapshots
Skipped
0.00
test_vm_snapshots.py

test_09_copy_delete_template
Skipped
0.02
test_templates.py

test_06_copy_template
Skipped
0.00
test_templates.py

test_static_role_account_acls
Skipped
0.05
test_staticroles.py

test_11_ss_nfs_version_on_ssvm
Skipped
0.02
test_ssvm.py

test_01_scale_vm
Skipped
0.00
test_scale_vm.py

test_01_primary_storage_iscsi
Skipped
0.04
test_primary_storage.py

test_vm_nic_adapter_vmxnet3
Skipped
0.00
test_nic_adapter_type.py

test_nested_virtualization_vmware
Skipped
0.00
test_nested_virtualization.py

test_06_copy_iso
Skipped
0.00
test_iso.py

test_deploy_vgpu_enabled_vm
Skipped
0.03
test_deploy_vgpu_enabled_vm.py

test_3d_gpu_support
Skipped
0.04
test_deploy_vgpu_enabled_vm.py

Looks like we've some template related test failures, which may or may not be related to this PR changes. I'll work next week towards fixing test failures both on 4.9 and master, and can revisit this after the baseline branches are passing most tests.

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian.  I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-968

ACS CI BVT Run
Sumarry:
Build Number 1167
Hypervisor xenserver
NetworkType Advanced
Passed=105
Failed=7
Skipped=12
Link to logs Folder (search by build_no): https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0
Failed tests:

test_non_contigiousvlan.py

test_extendPhysicalNetworkVlan Failed

test_list_ids_parameter.py

ContextSuite context=TestListIdsParams>:setup Failing since 59 runs

test_routers_network_ops.py

test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 31 runs

test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 158 runs

test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 153 runs

test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 153 runs

test_03_RVR_Network_check_router_state Failed

Skipped tests:
test_vm_nic_adapter_vmxnet3
test_01_verify_libvirt
test_02_verify_libvirt_after_restart
test_03_verify_libvirt_attach_disk
test_04_verify_guest_lspci
test_05_change_vm_ostype_restart
test_06_verify_guest_lspci_again
test_static_role_account_acls
test_11_ss_nfs_version_on_ssvm
test_nested_virtualization_vmware
test_3d_gpu_support
test_deploy_vgpu_enabled_vm
Passed test suits:
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_vm_snapshots.py
test_over_provisioning.py
test_global_settings.py
test_router_dnsservice.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_loadbalance.py
test_routers.py
test_reset_vm_on_reboot.py
test_deploy_vms_with_varied_deploymentplanners.py
test_network.py
test_router_dns.py
test_login.py
test_deploy_vm_iso.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_metrics_api.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_volumes.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_vm_life_cycle.py
test_disk_offerings.py

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1048

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-1464)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 33782 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2224-t1464-kvm-centos7.zip
Intermitten failure detected: /marvin/tests/smoke/test_iso.py
Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
Test completed. 54 look OK, 4 have error(s)

Test
Result
Time (s)
Test File

test_01_vpc_remote_access_vpn
Failure
50.94
test_vpc_vpn.py

test_05_rvpc_multi_tiers
Failure
186.72
test_vpc_redundant.py

test_04_rvpc_privategw_static_routes
Failure
326.35
test_privategw_acl.py

test_05_iso_permissions
Failure
0.07
test_iso.py

test_02_edit_iso
Failure
0.05
test_iso.py

test_change_service_offering_for_vm_with_snapshots
Skipped
0.00
test_vm_snapshots.py

test_09_copy_delete_template
Skipped
0.03
test_templates.py

test_06_copy_template
Skipped
0.00
test_templates.py

test_static_role_account_acls
Skipped
0.02
test_staticroles.py

test_11_ss_nfs_version_on_ssvm
Skipped
0.03
test_ssvm.py

test_01_scale_vm
Skipped
0.00
test_scale_vm.py

test_01_primary_storage_iscsi
Skipped
0.04
test_primary_storage.py

test_vm_nic_adapter_vmxnet3
Skipped
0.00
test_nic_adapter_type.py

test_nested_virtualization_vmware
Skipped
0.00
test_nested_virtualization.py

test_06_copy_iso
Skipped
0.00
test_iso.py

test_deploy_vgpu_enabled_vm
Skipped
0.03
test_deploy_vgpu_enabled_vm.py

test_3d_gpu_support
Skipped
0.04
test_deploy_vgpu_enabled_vm.py

LGTM, other failures are known intermittent issues


Local Draft of Tiyaro Support

@ijonglin  Can we also add a blurb under the 'Quick Prediction' section of the README.

@maheshpatil2021 Added an extra section in quick prediction.


Table-of-contents toggling support

This PR adds a card UI which allows for removing/adding the table of contents. It relies on https://github.com/lblod/ember-rdfa-editor/pull/313.

Looks good, maybe we don't want that many cards in the editor? Check with Dieter if we should move this elsewhere

@Dietr should we add the toggle in a seperate card (like it is now) or do we add it somewhere else?


"deadliness" of an BusinessService

everytime we send a message to the RestBS business service:
from flask import Flask, request

app = Flask(__name__)

from iop import Director

@app.route('/interop', methods=['GET', 'POST', 'PUT', 'DELETE'])
def interop():

    bs = Director.create_python_business_service('BS')

    rsp = bs.on_process_input(request)

    return jsonify(rsp)

The business service does its job but it also dies:

even if the service dies it works but we have to wait a second until we are able to send the request again
if we send two requests very fast we get a 500

Fix in 3.1.4


If Plasmic Is Open Source, Why the Need to Provide API Token?

I've skimmed through the docs trying to evaluate Plasmic, and I am lacking the understanding of what I am getting.
My assumption is that I get a drag-and-drop website builder + CMS + low-code tool that plugs into data sources, all in one, open source, self-hostable.
However, I see mentions of an API token in docs (e.g. here https://docs.plasmic.app/learn/react-quickstart/), and I do not quite understand, why the token is needed.
I also hadn't seen a mention of an API token in the New Plasmic codebase (CLI) refernce, would that be the (only?) solution for a fully self-hostable Plasmic?
Thanks in advance.

Hello @disbolog. This API token is used to get readonly access to the project code generation within plasmic.
When you generate a new code base using the CLI, we automatically fill the API token based on your user permissions within  the closest .plasmic.auth file.
The self host also include the code generation backend. This means that your local project will also have an API token that will give permission to your local codegen.
Let me know if you have any more questions.


Pipelines Issue

Hoping someone will see this.  I am having an issue with Pipelines.  When I try to save a new topic (or anything for that matter)  It isnt saved.  There is a graceful finish, but nothing has happened.  When debugging I see that there are no pipes in the pipeline.  This is due to when the Pipeline.Register function is called it doesnt happen because the "if (allTopicPipes.ContainsKey(pipe))" is false.  therefore nothing is added to the collection when the actual save happens.  Can someone please help?  Many thanks!
Here is the add attempt (No Key)

And finally nothing happens here

Issue Resolved.  The pipelines look for assemblies that are loaded on app start.  The default directory for these assemblies is app_data. But.. Initially there is nothing in there. You must first compile the project MvcForums.Plugins before running the web project.  This will create the assembly needed to properly save to the db. Note:  I created the Plugins folder manually.


[INTEGRATION][AIRFLOW] Add new extractor for GCSToGCSOperator

Signed-off-by: Kengo Seki<EMAIL_ADDRESS>Problem
Currently, there's no extractor corresponding to GCSToGCSOperator.
Solution
This PR adds a new extractor for GCSToGCSOperator.
Checklist

[x] You've signed-off your work
[x] Your pull request title follows our guidelines
[x] Your changes are accompanied by tests (if relevant)
[x] Your change contains a small diff and is self-contained
[x] You've updated any relevant documentation (if relevant)
[ ] You've updated the CHANGELOG.md with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary)
[ ] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)
[ ] You've added a header to source files (if relevant)

SPDX-License-Identifier: Apache-2.0
Copyright 2018-2023 contributors to the OpenLineage project

I ran the following command locally after putting my own gcloud-service-key.json in integration/airflow/tests/integration/gcloud and confirmed the gcs_dag integration test succeeded.
$ env AIRFLOW_IMAGE=apache/airflow:2.4.2-python3.7 \
  GOOGLE_CLOUD_STORAGE_SOURCE_URI=gs://gcp-public-data-landsat/LC08/01/044/034/LC08_L1GT_044034_20130330_20170310_01_T2/LC08_L1GT_044034_20130330_20170310_01_T2_ANG.txt \
  GOOGLE_CLOUD_STORAGE_DESTINATION_URI=gs://sekikn/output \
  tests/integration/docker/up.sh


High CPU usage when merging UE? node with custom node

As you asked, I create an issue just for that.
In the testing video you can see that the issue only involves Anything Everywhere? node, the other UE nodes work fine.
Also in my case (gaming laptop), it increases integrated GPU usage as well.
https://github.com/chrisgoringe/cg-use-everywhere/assets/161304036/3c95ff45-1f8f-4f83-b664-dd2db48da28e

TL;DR - I think this has been fixed as a side effect of the fix of #144. Could you check?
It's odd: when you read certain information from a group node, as a side effect it writes the values of the widgets back onto them (even though they haven't changed). But writing to a widget invalidates the UE network (because if you change a regex, the network changes). Which means the UE network ends up self-invalidating, so it is perpetually recalculating!
You'd think it would be easy to say "if the new value is the same as the old value, just ignore", but place that can be done is in the core code (I've submitted https://github.com/comfyanonymous/ComfyUI/pull/3093)
When I fixed #144 I discovered a side effect was that the node highlight wasn't working - and this was why.
Anyway, I've set a flag in the UE code to disable invalidation while reading the list, so now the read->write->update->callback->invalid sequence is broken.

Maybe @LukeG89 could give a more definitive answer on this.
Before the latest update, I did noticed the fan noise from my CPU cooler, when I was building this bad boy ...
But now it seems better.

Yes! It's perfect now! Great job @chrisgoringe! 🔥🔥🔥


Per Database Disk Space Usage

We need to monitor and report on database size (disk space usage per database).
I've looked through the options in SHOW STATS / DIAGNOSTICS / DATABASES / SHARDS and cannot find anything suitable.
Only option right now seems to be a disk level 'du influx/data/....' type query, but this poses security problems and does not fit well into our (docker based) stack.  Ideally we'd like to read this info from the server by a standard api call not some file system hack.  It seems like the server should know what disk space it is using, and if it's necessary to account for multiple nodes in a cluster then the server should also be aware of this.
Similar feature, for reference: in ElasticSearch, the indices stats API reports index size out of the box.  https://www.elastic.co/guide/en/elasticsearch/reference/current/indices-stats.html
Could this be added to SHOW DATABASES / SHOW DIAGNOSTICS or similar?

Fixed via #5758 and #5810


Initial Implementation

Merging prematurely to test some automation things.
This library / tool should still be considered experimental.

Looks like this is why typescript isn't working: https://github.com/microsoft/TypeScript/issues/46900


Post calls not working on PHP 71.

I Get 404 not found error when calling post functions as below
PHP Deprecated:  Non-static method ------ should not be called statically
Did this packages work on  PHP 7.1
Did any updates to overcome the issue?

Thanks for reporting @midhunraj can you describe a bit better what you're doing? I'm not sure I understand.

Thanks for the quick response.
When submitting a form with normal post method, I got an error as 404 - not found even the route extits.
When I checked the error log got an error message as
[08-Oct-2019 09:06:13 UTC] PHP Deprecated:  Non-static method App\Admin\Login::route() should not be called statically in /home/ivendqatar/public_html/admin/vendor/foil/foil/src/Engine.php on line 130
[08-Oct-2019 09:06:13 UTC] PHP Deprecated:  Non-static method App\Admin\Login::route() should not be called statically in /home/ivendqatar/public_html/admin/vendor/foil/foil/src/Providers/Kernel.php on line 82

Now the PHP version is 7.1, Could you help me to sort this out?

Hello..
Is there any updates on this?


glitter: 1.5.15 -> 1.6.0

Description of changes
https://github.com/Milo123459/glitter/compare/v1.5.15...v1.6.0

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

1 package built:

glitter

Hello!
FYI, I just saw that glitter --version returns the following output on x86_64-linux:
$ glitter --version
glitter 1.6.0Error:

Is this the expected output for this command?

I think I've caused this bug upstream, I'll have to fix it upstream

I think I've caused this bug upstream, I'll have to fix it upstream

Ok thanks!
I was unsure whether it was something caused by a build error or a bug in glitter :)

1.6.1 released: https://github.com/Milo123459/glitter/releases/tag/v1.6.1

@figsoda Can you update?

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

1 package built:

glitter


Integrate ffmpeg 4.1

I have also created a branch legacy/ffmpeg-3.4, so people can still build against old ffmpeg version if they like. All legacy/* branches are automatically protected.

FFmpegUniversal has been updated yesterday.
https://github.com/M2Team/FFmpegUniversal/releases/tag/4.1

@MouriNaruto  seems like ffmpegUniversal is doing exactly what ffmpeginterop does for uwp ? or is there a difference?

@touseefbsb FFmpegUniversal is a merged FFmpeg dynamic linked library for the Windows Universal Platform.
You can integrate FFmpegUniversal to FFmpegInterop.
Sample:
https://github.com/lukasf/FFmpegInterop/tree/FFmpegUniversal

@MouriNaruto  what are benefits of using ffmpeguniversal with ffmpeginterop, instead of using ffmpeg with ffmpeginterop ?

There are no functional benefits. But with FFmpegUniversal, you only need to reference and ship one ffmpeg dll, instead of 6 separate dlls. So file handling is a bit easier in the app where you use it. But FFmpegInterop needs quite a lot of changes when using it with the merged dll, so unfortunately, it is not easy to switch between original and FFmpegUniversal approach. There is a branch of FFmpegInterop which has the changes for FFmpegUniversal, but it is currently out of date.
I am not sure about the licensing implications of the dll merging used in FFmpegUniversal. ffmpeg consists of multiple libraries and it is usually licensed as lgpl (library gpl), which means that it must be possible for users to replace the shipped dlls with custom built dlls. I am not sure if the lgpl license even allows us to change the library structure. Even if it is allowed, we are then definitely forced to provide build tools and instructions to build such a merged dll, and we have to keep those hosted as long as our product is out.
Personally, I prefer to just use the original ffmpeg dlls. Then I can just point to the official ffmpeg compilation guide. Everyone can replace the ffmpeg dlls with custom built ones or dlls from nuget or whatever, and does not even need to know about FFmpegUniversal. The file handling with FFmpegUniversal is a bit easier, but it's not such a big deal for me. For me, the benefits of sticking to original dll structure are more important.

There is a branch of FFmpegInterop which has the changes for FFmpegUniversal, but it is currently out of date.
@lukasf I have updated that branch two days ago.

I am not sure about the licensing implications of the dll merging used in FFmpegUniversal. ffmpeg consists of multiple libraries and it is usually licensed as lgpl (library gpl), which means that it must be possible for users to replace the shipped dlls with custom built dlls.
@lukasf You can read the section 2 of LGPL (https://www.gnu.org/licenses/lgpl.html), and you will know if you modified a project which distributed under the LGPL, you need to distribute your modification under the LGPL or GPL. But FFmpegUniversal is a independent project or the modified version of FFmpeg, distributed under the same license as FFmpeg. For example, FFmpegUniversal uses LGPL 2.1 if you choose LGPL 2.1 to compile FFmpeg. Everyone can use FFmpegUniversal 's open source script to build FFmpeg and replace other one's FFmpegUniversal. So I think you don't need to worry about that.

@MouriNaruto Thanks for the clarifications. And good to hear that the branch is up to date again.
It is just my personal preference to stay with the original dlls. Using FFmpegUniversal sure is an alternative, if simple file handling and deployment is important.

Hi @lukasf
It seems it works fine. I tested filters as well and they work properly.

I also attached a profiler, which turned out no memory leaks.

hi @lukasf
I have added my review. Been using the library in a targeted fraction of my users (that's my non-dev devices + a few other people I know they use the player). It all checks out OK.
Maybe @MouriNaruto  can throw a review as well, but from my part, we are good to merge.

@lukasf @mcosmin222
I think we are good to merge. Hope it can bring us a better performance.

I have also tested with a lot of files in the last days. I did not notice any issues.
What do you think about adding a version check for ffmpeg dll? This branch will only work with at least V4.0 of ffmpeg. Older versions will cause threading problems due to missing lock_manager.

Maybe something like this? Check latest commit.

It is opt-in, so dev can decide if he wants version checking, and which version to check for (minimum version, which is 4.0, or recommended version, which is 4.1 right now).

Perhaps it should be moved to a separate class, to not clutter the main class interface.

Sounds good. Maybe we should have a check in the build script as well? So this branch cannot compile 3.0 but will work for 4.0

Then it would not be opt-in anymore. I'd rather let the build script build any version (and it is capable to build both 3.x and 4.x). And then there is also the fact that I don't know how to do partial string comparisons in build script ;)

Maybe it shouldn't be an opt-in. We should do a one time check at process start to see if dlls match the version, otherwise throw exception. I don't see why you'd want the 3.0 branch using the 4.0 dll or the other way around.
We can expose the property in a class called ffmpeginfo or something if it needs to be consumed somewhere else.

Maybe like this? Minimum version check is now mandatory (done when first using FFmpegInteropMss). Recommended version check is optional and must be called on FFmpegVersionInfo.

Looks good. And maybe we can change the minimum check for each branch: the 3.0 checks minimum for 3.0, this one checks for 4.0

I don't really plan to bring improvements to the 3.x branch. If you want to do it, feel free. But I'd only add this on master and use it for future versions.

Ok. I will do it after the merge.

Ok, merged!

Nice, now we have 0 issues. xD

Yes! :)


Update dependencies

Codecov Report

Merging #19 (454904c) into master (7f46a1d) will decrease coverage by 0.03%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
- Coverage   95.42%   95.39%   -0.04%
==========================================
  Files           1        1
  Lines         153      152       -1
==========================================
- Hits          146      145       -1
  Misses          7        7

Flag
Coverage Δ

unittests
95.39% <66.66%> (-0.04%)
:arrow_down:

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

Impacted Files
Coverage Δ

basedir.lua
95.39% <66.66%> (-0.04%)
:arrow_down:

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 7f46a1d...454904c. Read the comment docs.


React Look Example

Add react-look example.
The ReactPlugin doesn't work at all, probably because the wrapper around the entire app (which is needed) doesn't get picked up:
<LookRoot config={Presets['react-dom']}>
  {/* Root Component Here */}
</LookRoot>

I have no idea how we could fix that, honestly.
Closes #140

Wrapping the root node in a LookRoot component sounds like the worst idea I've heard for CSS-in-JS, and is only doable by our ReactPlugin if we add a special case for it, which I don't think is worth it. Closing for now!


Time out for double clicks is handled incorrectly.

I found that the handler set using onMouseDoubleClick is always called when a GShape object is clicked on twice using a mouse even if the time between clicks is several seconds. The cause is due to the value of MouseInputData.time being in seconds while the value of MouseInputData.doubleClickTime is in milliseconds. The comparison at line 949 in display_object.dart ends up using an interval of 250 seconds as a result and always succeeds.
Changing line 9 in pointer_data.dart so that the value of doubleClickTime is in seconds fixed the issue for me:
static double doubleClickTime = 0.250;

Thank you so much for finding the bug and providing a solution @JohnDeptuch. Will try to merge the change soon.


Discussion: how do we handle missing Babel config files in v11

From https://github.com/babel/babel-eslint/pull/711#issuecomment-453157175
Original question:

It looks like babelCore#parseSync will still parse without a config file, it just won't enable any plugins. Does this seem like a reasonable default? I'm worried about it being confusing behavior since Babel itself won't run any transformations without a config file. An alternative would be to run a check to see if a config file can be found (not sure if @babel-core provides an API for this already or not) and throw an error with a helpful message, if not.

Sounds like there might issues with any approach that doesn't just parse using defaults, as described here by @loganfsmyth :

The issue is that not all files in your project may have a config. If you have a .babelrc in src/ but didn't bother with any Babel config for the rest of your project, you should still be able to lint those files using the parser's defaults.

I guess we have to decide if babel-eslint is quite literally only for code that is transformed by Babel or if it can also be used for non-transformed code in a project that mostly consists of files that are transformed by Babel. Concrete example I can think of off the top of my head is a tooling config file (.eslintrc.js, babel.config.js, etc.) that might not have a config.
From a philosophical standpoint, I actually don't think it's unreasonable to say that those files should be parsed with the default parser (which will always be available since it's a dependency of ESLint and is totally possible with ESLint's glob-based configuration). Practically speaking, though, I'm not sure it's a good idea. It increases the overhead for every new project that uses babel-eslint and has any non-transformed JS code.

I got the error @loganfsmyth proposed above in 11.0.0-beta.0 and had to solve it by passing babelOptions into parserOptions because I'm working in a monorepo. This is specifically documented on the README (👍) but just posting this here in case anybody else comes to this issue in the same situation.


Update the MHLO Input Conversion pass to create func.call for mhlo.custom_call

https://github.com/vmurali/iree/tree/CustomCallInputConversion

@aaron-schneider @vmurali Can we remove this from the inbox? Needs scheduling, or in progress?

@allieculp These are sub-issues of https://github.com/openxla/iree/issues/12410, and I have scheduled these sub-issues in the dropdown menu on the right. If that timeline can be replicated in the sprints/taskflow, that will be great.


toEqual/toStrictEqual does not check custom Error properties

Describe the bug
Hi, I'm using the effect library to construct class instances, and I encounter the following issues, where the 2 assertions below pass (which is wrong):
import { Schema } from "@effect/schema";
import { Data } from "effect";
import { describe, expect, it } from "vitest";

class DataTaggedError extends Data.TaggedError("DataTaggedError")<{
  msg: string;
}> {}

class SchemaTaggedError extends Schema.TaggedError<SchemaTaggedError>()(
  "SchemaTaggedError",
  {
    msg: Schema.string,
  }
) {}

describe("testing", () => {
  it("debugging", () => {
    //Data.TaggedError
    expect({
      error: new DataTaggedError({ msg: "a" }),
    }).toStrictEqual({
      error: new DataTaggedError({ msg: "b" }),
    });

    //Schema.TaggedError
    expect({
      error: new SchemaTaggedError({ msg: "a" }),
    }).toStrictEqual({
      error: new SchemaTaggedError({ msg: "b" }),
    });
  });
});

I've initially raised an issue on the effect, but it result that by comparing with the native assert it works as expected:
import { Schema } from "@effect/schema";
import { Data } from "effect";
import * as assert from "node:assert";

class SchemaTaggedError extends Schema.TaggedError<SchemaTaggedError>()(
  "SchemaTaggedError",
  {
    msg: Schema.string,
  }
) {}

class DataTaggedError extends Data.TaggedError("DataTaggedError")<{
  msg: string;
}> {}

// no error
assert.deepStrictEqual(
  { error: new DataTaggedError({ msg: "a" }) },
  { error: new DataTaggedError({ msg: "a" }) }
);

// error report properly
assert.deepStrictEqual(
  { error: new DataTaggedError({ msg: "a" }) },
  { error: new DataTaggedError({ msg: "b" }) }
);

// no error
assert.deepStrictEqual(
  { error: new SchemaTaggedError({ msg: "a" }) },
  { error: new SchemaTaggedError({ msg: "a" }) }
);

// error report properly
assert.deepStrictEqual(
  { error: new SchemaTaggedError({ msg: "a" }) },
  { error: new SchemaTaggedError({ msg: "b" }) }
);

Reproduction
reproduction on StackBlitz with the code above
System Info
System:
    OS: macOS 12.2.1
    CPU: (8) arm64 Apple M1 Pro
    Memory: 476.78 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 20.3.0 - ~/.volta/tools/image/node/20.3.0/bin/node
    Yarn: 1.22.19 - ~/.volta/bin/yarn
    npm: 9.6.7 - ~/.volta/tools/image/node/20.3.0/bin/npm
    pnpm: 8.3.1 - ~/.volta/bin/pnpm
  Browsers:
    Chrome: 121.0.6167.184
    Safari: 15.3
  npmPackages:
    vitest: ^1.2.2 => 1.2.2

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

This happened as both DataTaggedError and SchemaTaggedError are extended from Error, and for Error objects, now Vitest will only checks the message prop. https://github.com/vitest-dev/vitest/blob/c692f76e58ae11fcc13631adcb4b80055843d447/packages/expect/src/jest-utils.ts#L98
Perhaps Vitest need to do more checks when there is no message prop in Error objects, I'm willing to make a PR to fix this.

I think jest still has the same code. There's one similar issue and even PR too, but they got closed as they didn't receive attention:

https://github.com/jestjs/jest/issues/13604
https://github.com/jestjs/jest/pull/13607

If there were no back-compat or jest-compat concern, then personally it makes sense to do the same treatment as NodeJS's assertion https://nodejs.org/api/assert.html#comparison-details_1
But in practice, changing this condition might have some larger implication.

@steffanek Thanks for raising an issue btw.
The current behavior doesn't seem intuitive for me as well, but the code we're seeing here is originally from Jest https://github.com/jestjs/jest/blob/e6d60cb1ab04a2d691dcbf025ed53d7a07327889/packages/expect-utils/src/jasmineUtils.ts#L86-L88 and the comment says it's based on underscore, so it could be like more than 10 years old.
What I just wanted to add is that, there could be some reason people have been adopting this behavior, so I think it's worth spending time surveying some history and more comparison.
Like you've already shown, node:assert is different (and personally it makes sense to me). But, we might want to also check others like lodash's isEqual, other test frameworks and assertion libraries.
In terms of documentation, there's at least a warning about this for toEqual (and internally toStrictEqual doesn't treat Error differently):

https://vitest.dev/api/expect.html#toequal
https://jestjs.io/docs/expect#toequalvalue

I explored expect.addEqualityTesters based approach and I think this should work for your use case for the time being:
https://stackblitz.com/edit/vitest-dev-vitest-944rfr?file=test%2Fbug.spec.ts
On Vitest side, probably what we can do is to just put more warning on documentation https://github.com/vitest-dev/vitest/pull/5253
Maybe if this behavior is desired so much, then we could introduce some opt-in flag to automatically enable this custom equality tester, but we'll probably need more discussion based on use cases.

even they are extending from Error, I would expect from the toStrictEqual to check if all key values matches. Does that make sense?

@steffanek Yes, It makes sense to me to be honest. And I think @hi-ogawa has provided the appropriate solution at this time.

Thanks @hi-ogawa and @fenghan34 for your quick response and solution.
Another solution would be to simply import * as assert from "node:assert" and use it directly in test files, whenever we want to check a deepStrictEqual in case of any instances. Do you guys see some downsides? What can comes to my mind is: performance (comparing to expect.addEqualityTesters) or wrong testing report?

Another solution would be to simply import * as assert from "node:assert" and use it directly in test files

Just a note, Vitest supports extended assert API: https://vitest.dev/api/assert.html

Another solution would be to simply import * as assert from "node:assert" and use it directly in test files

Just a note, Vitest supports extended assert API: https://vitest.dev/api/assert.html

@sheremet-va assert.deepStrictEqualfrom vitest does not solve this issue.

Another solution would be to simply import * as assert from "node:assert" and use it directly in test files, whenever we want to check a deepStrictEqual in case of any instances. Do you guys see some downsides? What can comes to my mind is: performance (comparing to expect.addEqualityTesters) or wrong testing report?

@steffanek Using node:assert should work fine, but the downside is simply that it's not a part of expect API, which would mean for example:

it might be prone to forget switching to assert.deepStrictEqual(...) if you are using expect(...).toStrictEqual(...) around the code.
some expect API works in a "stateful" way, such as expect.hasAssertions doesn't take into assert.deepStrictEqual account when counting the number of assertions.
there's no expect.soft version of assert.deepStrictEqual

Regarding performance, I would guess node:assert would be faster since expect/toEqual does more stuff underneath. But I don't think such raw performance difference would practically matter for normal usages.

Makes sense @hi-ogawa thanks for your contribution. Let's see if this behavior is really desired, to enable it by default.

Hi @hi-ogawa it seems that I do not need anymore to use the custom matcher that you've initially provided.
However, now I got the following issue (see below the last test where I compared two errors values inside an Exit data type :
import { it, expect } from "vitest";
import { Schema } from "@effect/schema";
import { Exit } from "effect";

class SchemaTaggedError extends Schema.TaggedError<SchemaTaggedError>()(
  "SchemaTaggedError",
  {
    msg: Schema.String,
    other: Schema.String,
  }
) {}

class SchemaTaggedClass extends Schema.TaggedClass<SchemaTaggedClass>()(
  "SchemaTaggedClass",
  {
    msg: Schema.String,
    other: Schema.String,
  }
) {}

it("Error type", () => {
  expect(new SchemaTaggedError({ msg: "a", other: "other" })).toStrictEqual(
    new SchemaTaggedError({ msg: "a", other: "other" })
  );
});

it("Exit with SchemaTaggedClass", () => {
  expect(
    Exit.fail(new SchemaTaggedClass({ msg: "a", other: "other" }))
  ).toStrictEqual(
    Exit.fail(new SchemaTaggedClass({ msg: "a", other: "other" }))
  );
});

//This should not pass..
it("Exit with SchemaTaggedError", () => {
  expect(
    Exit.fail(new SchemaTaggedError({ msg: "a", other: "other" }))
  ).toStrictEqual(
    Exit.fail(new SchemaTaggedError({ msg: "b", other: "other" }))
  );
});


Inputting the audio stream directly will report an error

When I get the audio from the microphone through speech_recognition and send it directly to faster-whisper for transcription, the following error will be reported.
Traceback (most recent call last):   File "D:\Works\PythonProjects\RealTime\main.py", line 19, in listenAudioStream     whisperHandle(audio)   File "D:\Works\PythonProjects\RealTime\main.py", line 32, in whisperHandle     segments, info = model.transcribe(audio, beam_size=5, vad_filter=True,   File "D:\Software\Anaconda\envs\RealTime\lib\site-packages\faster_whisper\transcribe.py", line 239, in transcribe     audio = decode_audio(audio, sampling_rate=sampling_rate)   File "D:\Software\Anaconda\envs\RealTime\lib\site-packages\faster_whisper\audio.py", line 45, in decode_audio     with av.open(input_file, metadata_errors="ignore") as container:   File "av\container\core.pyx", line 401, in av.container.core.open   File "av\container\core.pyx", line 246, in av.container.core.Container.__cinit__   File "av\container\pyio.pyx", line 44, in av.container.pyio.PyIOFile.__cinit__ ValueError: File object has no read() method, or readable() returned False.
What is the reason for this? Does it not support direct audio streaming?
If I want to do real-time transcription, does this project support direct input of audio stream and then transcribe? Just don't save it as a file, just stream it directly to the audio.
I will be very grateful if you can answer my question.

Hi, seems the type of audio is incorrect. From the documentation:
'audio: Path to the input file (or a file-like object), or the audio waveform.'
So, it should be one of: 1) path to the file, 2) BinaryIO obj with the full file inside (see io package) 3) raw samples in a numpy array (with framerate 16000 hz).
I don't know the type of audio that comes from this 'sr. ' but if it does not have a .read() method it can be probably a bytes string. Then it should be converted to samples (with a correct framerate). I use the Pydub package for such kinds of manipulations.
Also, it's better to initialize the model once and pass it to the function each time you want to transcribe.


Fix newlines

adds a _sanitization step with (only, for now) removal of \n newline characters that was choking IRC output

I think this is replaced by #10. I will close for now.


Use treemacs-magit package

I thought this might be useful to setup and install when using both treemacs and magit. Cheers.

Whoops, sorry about the extra treemacs checks. Still new to emacs and lisp. Cheers, thanks!

Not a problem! Thanks for the quick fixes (and thanks again for your help).


Rewrite test_disturl_can_be_checked_out to not rely on osc

The test either does not work because osc is not installed, or it fails because osc is available but its dependency are not available inside the venv that the testsuite is running under. We can easily rewrite that with just using requests and avoiding the conflict with venvs

Looks like this is failing for Tumbleweed

that's fixed now.


Update to Angular 13's new package format and compile with partial-ivy

With Angular 13 a new package format was rolled out. They're also encouraging library developers to publish libraries with partial ivy compilation mode, which removes the need for ngcc. This library is not being published with either partial-ivy or the new package format.
`Processing legacy "View Engine" libraries:

@ng-idle/core [es2015/esm2015] (git+https://github.com/moribvndvs/ng2-idle.git)
@ng-idle/keepalive [es2015/esm2015] (git+https://github.com/moribvndvs/ng2-idle.git)
Encourage the library authors to publish an Ivy distribution.`

If you use ngpackagr and Angular 13 to build the library, then this should be solved naturally. If not, here are the details about the new package format: https://angular.io/guide/angular-package-format

any update on this issue? now we have<EMAIL_ADDRESS>

Would be nice to have this warning removed, any info on that?
Is there still a maintenance issue (as I've seen a lot of people volunteering to help maintain this package)?

Would be nice to have this warning removed, any info on that?
Is there still a maintenance issue (as I've seen a lot of people volunteering to help maintain this package)?

I accept pull requests.
As for a help maintaining, I've had people stop by to help but they don't tend to stick around for unknown reasons.

Ok, thanks for the quick response!
I'll see what I can do in terms of PR for this.
If anyone in this thread want to beat me to it feel free :-)

I've opened the following PR:
https://github.com/moribvndvs/ng2-idle/pull/169
Besides running the tests is there another way to make sure everything is as expected?


Cleanup code smells from sonarcloud

Take some time to do a cleanup of the code smells as found in sonarcloud:
https://sonarcloud.io/project/issues?id=bitmaelum_bitmaelum-suite&resolved=false

Hi, this project looks interesting, I'm eager to contribute.
This issues presents itself as a nice starting point into your codebase, mind if take it?

Hi, this project looks interesting, I'm eager to contribute.
This issues presents itself as a nice starting point into your codebase, mind if take it?

welcome! We're always happy to have new people contribute. You're more than welcome to pick up issues. The code might be a little hard to grasp maybe because there is a lot going on, but most of the sonarcloud issues are doable enough.

welcome! We're always happy to have new people contribute. You're more than welcome to pick up issues. The code might be a little hard to grasp maybe because there is a lot going on, but most of the sonarcloud issues are doable enough.

https://github.com/bitmaelum/bitmaelum-suite/pull/171

https://github.com/bitmaelum/bitmaelum-suite/pull/171

Pretty nice work done!
I leave this issue open for now, so we can tackle the rest later

Pretty nice work done!
I leave this issue open for now, so we can tackle the rest later


Add Swiftlint #trivial

Replaces #642
Issue ref: #394
Integrates SwiftLint into DangerBot.
You may want to append a ?w=1 to the end of the files-changed url to exclude whitespace-only changes.

[ ] All tests pass. Demo project builds and runs.
[ ] I added tests, an experiment, or detailed why my change isn't tested.
[ ] I added an entry to the CHANGELOG.md for any breaking changes, enhancements, or bug fixes.
[ ] I have reviewed the contributing guide

#trivial

@rnystrom -- still having trouble getting this imported. any updates?

Ok so this is adding about 35mb to the repo from including the SwiftLint framework 3x from each of the examples. Wow. Our internal tools flagged this and wont let us import it b/c of the size.
I'm not totally sure what the best course of action here should be. Obviously the main issue is having 3x separate example projects, each with their own Podfile, and the fact that we check in all Pod data.
Couple options:

We drop all the SwiftLint stuff and just keep the lint changes to .swift files
We update all of the examples to be a single project/workspace so we only SwiftLint import once
We drop this altogether ☹

@rnystrom surprised it's 35mb.
I think short-term, the right move may be to take the changes to the .swift files and drop SwiftLint from the example projects -- it won't flag errors during development of the Example project, but the bot should still flag errors on the PR.
Restructuring into one xcworkspace sounds like the right longer-term move to me.

Another option is to get the framework (1 version of it) store that in a directory (e.g. Examples/Swiftlint) and then reference that with a built script from all 3 projects. (So stop using CocoaPods to get Swiftlint).
I'd honestly prefer this!
@rnystrom  @heshamsalman

I think @Sherlouk 's idea is good.
That's how the local IGListKit pod is working -- all examples just reference ../../IGListKit

Occam's Razor 😎
Seems like the right move to me. I'll make the change late Friday or Sunday 👍

cc @rnystrom ?

On board w/ @Sherlouk's plan, sounds great!

@Sherlouk @jessesquires @rnystrom
Should be a good deal smaller now. 👍

Changes look a lot nicer, will leave it to the insta crew to check on Phabricator! Hopefully we can get it in 🙏

@rnystrom @jessesquires Think we could try and see if Phabricator will except this some time this week? Would be nice to get this in (and I get that it's a pain to keep rebasing 😅)

@Sherlouk - yep, everything looks alright to me!

Ayyy 👍

Ok, bad news:
So imported succeed but land failed internally because the binary files are too large. 😞
The best path forward is probably to run this on CI, but not include SwiftLint in the repo? Not 100% on what the best option is.
@rnystrom is out for a bit, when he returns we can sort this out. 👍
Sorry for the churn @heshamsalman ! 😅

@jessesquires I could alternatively change the SwiftLint script to rely on a user's HomeBrew installed copy of SwiftLint (instead of a locally referenced binary), and echo installation instructions if they don't have it installed?

@heshamsalman -- that sounds awesome! Let's definitely do that.
That's how we handle jazzy. Use must have it installed.

@heshamsalman -- because of how the importing system works, we'll need to open a new PR 😄

@jessesquires haha classic. I'll re-open soon

thanks so much @heshamsalman ! 🎉

User must have it installed. (Though we don't provide a helpful message about how to install. that would be nice)

Might be a good idea to create an issue for that? :)

@BasThomas Yep! 😊

Created #744. :)


[1.9] Shift clicking recipe for shield template causes crash

rftools-1.9.0-4.45
jei_1.9-<IP_ADDRESS>
Forge 1865
Shift clicking recipe (the +) in JEI for shield template causes client disconnect with the following message - http://i.imgur.com/hrBQ5c3.png
Log traps this which might be a Forge error ? http://i.imgur.com/TpOpnzX.png

I would report this to the JEI guys. Doesn't seem to be a RFTools but but I may be mistaken

It's not, I did further invetigation into the crashing and it appears isolated to Refined Storage crafting tables, sorry to bother you with this.


 The external interface of Cargo is not affected by this change.
Previously each Target had a Profile embedded within it. Conceptually a Target
is an entry in the manifest (a binary, benchmark, etc) and a Profile controlled
various flags (e.g. --test, -C opt-level, etc). Each Package then contained many
profiles for each possible compilation mode. For example a Package would have
one target for testing a library, benchmarking a library, documenting a library,
etc. When it came to building these targets, Cargo would filter out the targets
listed to determine what needed to be built.
This filtering was largely done based off an "environment" represented as a
string. Each mode of compilation got a separate environment string like "test"
or "bench". Altogether, however, this approach had a number of drawbacks:

Examples in release mode do not currently work. This is due to how examples
are classified and how release mode is handled (e.g. the "release" environment
where examples are meant to be built in the "test" environment).
It is not trivial to implement cargo test --release today.
It is not trivial to implement cargo build --bin foo where only the binary
foo is built. The same is true for examples.
It is not trivial to add selective building of a number of
binaries/examples/etc.
Filtering the list of targets to build has some pretty hokey logic that
involves pseudo-environments like "doc-all" vs "doc". This logic is duplicated
in a few places and in general is quite brittle.
The TOML parser greatly affects compilation due to the time at which profiles
are generated, which seems somewhat backwards.
Profiles must be overridden, but only partially, at compile time becuase only
the top-level package's profile is applied.

In general, this system just needed an overhaul. This commit made a single
change of separating Profile from Target and then basically hit make until
all the tests passed again. The other large architectural changes are:

Environment strings are now entirely gone.
Filters are implemented in a much more robust fashion.
Release mode is now handled much more gracefully.
The unit of compilation in the backend is no longer (package, target) but
rather (package, target, profile). This change had to be propagated many
location in the cargo_rustc module.
The backend does not filter targets to build at all. All filtering now
happens entirely in the frontend.

I'll test issues after this change lands, but the motivation behind this is to
open the door to quickly fixing a number of outstanding issues against Cargo.
This change itself is not intended to close many bugs.

r? @huonw
(rust_highfive has picked a reviewer for you, use r? to override)

This is me finally following up to #1325

cc @wycats

This will also unblock some work related to rust-lang/rust#23533 I've got pending.

r? @brson

I can also squash any number of commits as desired or break up the first commit if necessary. I started by lumping as much as possible into one commit but eventually started making one fix per mistake, so the first commit is somewhat biasedly large!

@bors r+

:pushpin: Commit 3ce79da has been approved by brson

:hourglass: Testing commit 3ce79da with merge 0428b88...

:broken_heart: Test failed - cargo-win-32

@bors: r=brson acd1304

:hourglass: Testing commit acd1304 with merge 060c59f...

:broken_heart: Test failed - cargo-win-32

@bors: r=bson 0c28783

:hourglass: Testing commit 0c28783 with merge c6b9324...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-32, cargo-win-64

:umbrella: The latest upstream changes (presumably #1441) made this pull request unmergeable. Please resolve the merge conflicts.


Add timeout for SOLR download

The default of 10 minutes was not enough for me.

Looks good to me!  Thanks!


Comparison against six layer BERT

Hi
I read your paper and found it very interesting, I was wondering if you had any ablation results which compared the performance of a 6 layer Bert-of-Theseus (compressed from 12 layer BERT) against the performance of a 6 layer BERT trained from scratch? If not do you have any intuition for whether the module replacement of a larger model would surpass the performance of that same smaller model trained from scratch?
Many thanks
David

Hi David, thanks for your interest in our work.
Yes, of course a model trained from scratch performs way worse than any model compression method. This is the basic idea of model compression and we do not have this ablation because we have stronger baselines.

Thanks for the reply! I was hoping that was the case, but even if it wasn't, a secondary motivation for model compression could be to compress pre-trained models which would be infeasible to retrain from scratch, even if the resultant performance was only equivalent to a smaller model trained from scratch, but it seems that is not the case.


Polishing differential abundance

Parent ticket for many (hopefully small!) polishing tasks for the differential abundance app:

[x] Check to ensure no values overlap in groupA and groupB. So if "peanuts" chosen in group A, disable it in group B and add tooltip that says "Values cannot overlap between groups" or something  like that (credit @jernestmyers ) https://github.com/VEuPathDB/web-monorepo/pull/388
[x] Add swap button to group A group B https://github.com/VEuPathDB/web-monorepo/pull/396
[x] Add empty vizx plot. Otherwise the panel looks really weird before the viz data is ready. https://github.com/VEuPathDB/web-monorepo/pull/390
[x] Add legend (driven by VolcanoPlot, always the same labels, no need to have it be clickable) https://github.com/VEuPathDB/web-monorepo/pull/392
[x] Add axis controls to x and y axis. Component is ready. Just need axis range control, no log transform or anything! https://github.com/VEuPathDB/web-monorepo/pull/404
[x] Add opacity slider for markers. https://github.com/VEuPathDB/web-monorepo/pull/404
[x] Add axis truncation indicators (remember use props+data to derive this instead of state!) IP here https://github.com/VEuPathDB/web-monorepo/pull/325
[x] update numberInputRange to take increment prop (right now changes by 1, but we need it to change by 0.001 or so) https://github.com/VEuPathDB/web-monorepo/pull/384
[x] Layout inputs and such nicer (may require update to LabelledGroup) https://github.com/VEuPathDB/web-monorepo/pull/396

Closing now that all sub-tasks have been merged.


Change all links to fd_set in nf-winsock2-select.md to point to the structure instead of the macro

Thank you!


Description of the architecture

I am just familiarizing with the repository and it seems that there are many moving parts. I think it would be useful to have a diagram or a description of what is what and how the different pieces are combined together.

On a related node: I started documenting the samples: https://github.com/modelix/modelix/wiki/Samples
@slisson will create a high level arch document.

Why is it that for some nodes (e.g. the ones in org.modelix.samples.entities.sandbox) I get a read-only darkula-styled editor, and for some (e.g. org.modelix.model.runtimelang.structure) I get editable white-background editors.

Why is it that for some nodes (e.g. the ones in org.modelix.samples.entities.sandbox) I get a read-only darkula-styled editor, and for some (e.g. org.modelix.model.runtimelang.structure) I get editable white-background editors.

There is an override for AbstractConceptDeclaration in org.modelix.ui.sm.transformations.conceptPage

@slisson thank you for describing the architecture. It's worth reading the doc in the linked commit.
I have one more question. Does this project have a relation to Conveton?
I'm citing the linked blog article.

Conceptually, Convecton can be seen as a descendant of MPS. Like MPS, it relies on projectional editing. But it also approaches a couple of things quite differently to make it more suitable for non-technical business users:
The Convecton client, the part used for modeling by domain experts, is a modern web application and not an IDE optimized for developers.
Collaboration among multiple end users is achieved via a central repository and real-time synchronization (like Google Docs) instead of files, git, and diff/merge.
Realizing that the value of models to domain experts is often more in their ability to run/test/simulate them, Convecton puts a much bigger emphasis on execution of models (think: live programming) in the sense that interpreters and upstream dependency tracking are built-in.
And finally, the deployment effort is reduced, because everything runs in the browser and on servers, in contrast to rich clients apps (like MPS).

Unfortunately, the link to the Convecton homepage is dead.

The convecton project is also dead, but the vision continues to live in this project. For me convecton is one of the many prototypes that was useful to collect experience, but didn't survive. With this project we just try a different approach to reach the same goal.


Create lo2kk.txt

@fedehusk First stage of verification passed, official domain for II Liceum Ogólnokształcące im. Mikołaja Kopernika w Kędzierzynie-Koźlu is lo2kk.edu.pl as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@fedehusk Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Update READ.me

Fixed typo. Re-arranged items.
Removed the broken link.

Fixed typo. Re-arranged items.
Removed the broken link.


RA-GRS - How to restore backups from RA-GRS

This page doesn't mention how to restore a backup from an RA-GRS storage account.  Is there information available on this?

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

ID: 54961bd6-2a57-66b8-95ef-4b390263db82
Version Independent ID: 213d141c-af2c-1ac5-8964-adee28f6fc87
Content: Back up to URL best practices & troubleshooting - SQL Server
Content Source: docs/relational-databases/backup-restore/sql-server-backup-to-url-best-practices-and-troubleshooting.md
Product: sql
Technology: backup-restore
GitHub Login: @cawrites
Microsoft Alias: chadam

@seanalexander -- Seann, thank you for your question.  Please consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

@cawrites -- please look into this potential documentation issue.

@seanalexander -- Seann, thank you for your question.  Please consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

@cawrites -- please look into this potential documentation issue.

Thanks @WilliamAntonRohm !
It seems it's not supported, but I'd like for it to say that in the documentation so that we have something to refer to.

Thanks @WilliamAntonRohm !
It seems it's not supported, but I'd like for it to say that in the documentation so that we have something to refer to.

@seanalexander This is one reference that you may find useful. https://docs.microsoft.com/en-us/sql/sql-server/stretch-database/backup-stretch-enabled-databases-stretch-database?view=sql-server-ver15

@seanalexander This is one reference that you may find useful. https://docs.microsoft.com/en-us/sql/sql-server/stretch-database/backup-stretch-enabled-databases-stretch-database?view=sql-server-ver15

#please-close

#please-close

To clarify, there is no way to restore from a RA-GRS account on Hosted SQL Server 2019?

To clarify, there is no way to restore from a RA-GRS account on Hosted SQL Server 2019?


Don't omit any used fields when encoding people

When building a person object to send to the API, don't omit empty fields. Otherwise, you couldn't set them to empty value.
Fixes VEGA-1404 #patch

Codecov Report

Merging #100 (e9a16dd) into main (72e3029) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #100   +/-   ##
=======================================
  Coverage   78.50%   78.50%
=======================================
  Files          53       53
  Lines        1261     1261
=======================================
  Hits          990      990
  Misses        170      170
  Partials      101      101

Impacted Files
Coverage Δ

internal/sirius/person.go
75.00% <ø> (ø)

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 72e3029...e9a16dd. Read the comment docs.


bug(Android): NFC write URI or absolute URI record not working correctly

Plugin version:

0.2.0

Platform(s):

Android 12 / macOs Monterey 12.6.1
Current behavior:

Record the incomplete URL and it is not possible to read the card

Expected behavior:

Save URL correctly

Steps to reproduce:

Write a URI or absolute URI on an android device

Related code:

Other information:

Capacitor doctor:

💊   Capacitor Doctor  💊

Latest Dependencies:

  @capacitor/cli: 4.6.2
  @capacitor/core: 4.6.2
  @capacitor/android: 4.6.2
  @capacitor/ios: 4.6.2

Installed Dependencies:

  @capacitor/cli: 4.0.1
  @capacitor/android: 4.0.1
  @capacitor/ios: 4.0.1
  @capacitor/core: 4.0.1

[success] Android looking great! 👌
[error] CocoaPods is not installed.
See this install guide: https://guides.cocoapods.org/using/getting-started.html#installation

@javier-cuevas Thank you for reporting this! I will take a look this weekend.

I published a new version 0.3.1 with a fix. I will also update the demo app.


Flash to white on hot update - do we mind?

Removed proactive destroy call - does this still happen?


test_utils: use ext2 instead of ext3

Images should never be generated from a journaled filesystem. Attempting
to shove the 4MB journal in the test filesystem normally isn't a big
deal but sometimes it appears to tickle something problematic in the
bsdiff algorithm, taking as long as 15-20 minutes to run.
So lets just not do that shall we? :-P

LGTM


added post save current data event

I added an onPostCurrentSaveDataEvent to be able to add some data after saving. Relates to issue #212

@placid2000, do you still see a need for this?

Yes ticket can be closed in my opinion since I took another approach.
It'd have just been nice if it was possible to add custom data (for properties that are not set in the form type) to the storage during the process because I end up persisting my the data to the database so this Bundle became kind of obsolete in my implementation I guess. that's a pity ..

It is possible to use the storage directly.

Ok. I had a very specific case that didn't work. I end up using a workaround like written above.


Ntp external source fast config load

Good Morning All.
Working with customer we followed this configuration and is correct. In addition, in order to update the configuration in a fast way we can create a scheduled task to run the Time Sync after the system startup. Tested in Lab and customer and works fine.
Thank you

Hello Sensei @NDVALPHA , can you please review?
Thanks

@NDVALPHA

Can you review this PR?

IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Docs Build status updates of commit 11780c9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/virtual-machines/windows/external-ntpsource-configuration.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 00e1fab:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/virtual-machines/windows/external-ntpsource-configuration.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

#sign-off

Docs Build status updates of commit a7409f0:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed]

Cannot sync git repo to specified commit because ref +refs/pull/97651/merge has been deleted or has been force pushed.
 If this is pull request, please ensure it is mergeable (no merge conflict) and open (not closed).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Avoid redundant warning when encoding NumPy array as Image

Avoid a redundant warning in encode_np_array by removing the identity check as NumPy dtypes can be equal without having identical ids.
Additionally, fix "unreachable" checks in encode_np_array.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.