error: cannot assign value of type string to a value of type json

var json : JSON = []
var data = [
"name":"asd",
"age":"23"
]
for (key,value) in data {
       json[key] = value
}

error: cannot assign value of type string to a value of type json

var json : JSON = [:]  // data is a Dictionary
var data = [
    "name":"asd",
    "age":"23"
]
for (key,value) in data {
    json[key].stringValue = value // or json[key] = JSON(value)
}

This should work.

Out of curiosity why does this error occur? What about wrapping it in a for loop forces us to explicitly type it to a string?


Fix Program Brief leftovers

Packages impacted by this PR

@azure-tools/communication-short-codes

Issues associated with this PR
--
Describe the problem that is addressed by this PR
NOTE: This is a private SDK intended for internal use only. This SDK will not be publicly supported
This fixes a problem with tests leaving draft leftovers. The idea here is to do a cleanup before running the tests and also to delete the program briefs if the test fails for some reason.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
--
Are there test cases added in this PR? (If not, why?)
--
Provide a list of related PRs (if any)
--
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[x] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

/azp run js - communication-short-codes - tests

/azp run js - communication-short-codes - tests

/azp run js - communication-short-codes - tests


SSL Issue.. (windows 10)

Not able to download the course.........facing SSL certificate issue...............
[0316/150902.789:ERROR:gpu_process_transport_factory.cc(1009)] Lost UI shared context.
[0316/150902.794:ERROR:instance.cc(49)] Unable to locate service manifest for metrics
[0316/150904.749:ERROR:service_manager.cc(890)] Failed to resolve service name: metrics
Login with your credential
Downloading course: javascript-hard-parts
[0316/150932.653:INFO:CONSOLE(0)] "The SSL certificate used to load resources from https://embedwistia-a.akamaihd.net will be distrusted in the future. Once distrusted, users will be prevented from loading these resources. See https://g.co/chrome/symantecpkicerts for more information.", source: https://frontendmasters.com/courses/javascript-hard-parts/ (0)
Retrieving: javascript-hard-parts/JavaScript_The_Hard_Parts/Introduction
Video URL: http://placehold.it/500x500
Retrieving: javascript-hard-parts/Principles_of_JavaScript/Global_Execution_Context
[0316/150957.634:ERROR:instance.cc(49)] Unable to locate service manifest for metrics
[0316/150957.636:ERROR:service_manager.cc(890)] Failed to resolve service name: metrics

[0319/105302.417:ERROR:gpu_process_transport_factory.cc(1009)] Lost UI shared context.
[0319/105302.419:ERROR:instance.cc(49)] Unable to locate service manifest for metrics
[0319/105302.419:ERROR:service_manager.cc(890)] Failed to resolve service name: metrics
DevTools listening on ws://<IP_ADDRESS>:12673/devtools/browser/f0ee8643-f054-4e66-a4ae-d3c1fbead141
[0319/105303.025:ERROR:instance.cc(49)] Unable to locate service manifest for metrics
[0319/105303.025:ERROR:service_manager.cc(890)] Failed to resolve service name: metrics

Login with your credential
Downloading course: js-fundamentals-functional-v2
Retrieving: js-fundamentals-functional-v2/JavaScript_From_Fundamentals_to_Functional_JS/Course_Introduction
[0319/105322.683:INFO:CONSOLE(0)] "The SSL certificate used to load resources from https://embedwistia-a.akamaihd.net will be distrusted in the future. Once distrusted, users will be prevented from loading these resources. See https://g.co/chrome/symantecpkicerts for more information.", source: https://frontendmasters.com/courses/js-fundamentals-functional-v2/ (0)
Traceback (most recent call last):
File "frontendmasters-dl.py", line 24, in
downloader()
File "C:\Python27\lib\site-packages\click\core.py", line 722, in call
return self.main(*args, **kwargs)
File "C:\Python27\lib\site-packages\click\core.py", line 697, in main
rv = self.invoke(ctx)
File "C:\Python27\lib\site-packages\click\core.py", line 895, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "C:\Python27\lib\site-packages\click\core.py", line 535, in invoke
return callback(*args, **kwargs)
File "frontendmasters-dl.py", line 17, in downloader
spider.download(course, high_resolution, video_per_video)
File "C:\webdrivers\OS_FrontendMaster-dl-master\extractor\spider.py", line 52, in download
course_downloadbale = self._get_downloadable_links(course_detailed_list, high_resolution, video_per_video)
File "C:\webdrivers\OS_FrontendMaster-dl-master\extractor\spider.py", line 178, in _get_downloadable_links
resolution_button.click()
File "C:\Python27\lib\site-packages\selenium\webdriver\remote\webelement.py", line 80, in click
self._execute(Command.CLICK_ELEMENT)
File "C:\Python27\lib\site-packages\selenium\webdriver\remote\webelement.py", line 628, in _execute
return self._parent.execute(command, params)
File "C:\Python27\lib\site-packages\selenium\webdriver\remote\webdriver.py", line 312, in execute
self.error_handler.check_response(response)
File "C:\Python27\lib\site-packages\selenium\webdriver\remote\errorhandler.py", line 242, in check_response
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.ElementNotVisibleException: Message: element not visible
(Session info: headless chrome=64.0.3282.186)
(Driver info: chromedriver=2.37.543627 (63642262d9fb93fb4ab52398be4286d844092a5e),platform=Windows NT 10.0.16299 x86_64)

C:\webdrivers\OS_FrontendMaster-dl-master>[0319/105334.817:ERROR:instance.cc(49)] Unable to locate service manifest for metrics
[0319/105334.819:ERROR:service_manager.cc(890)] Failed to resolve service name: metrics


readme issues

This link (code_runner.lua) is dead:

And if I understand this correctly, mode is not type bool but string:

Pleace send PR


Update documentation with 2.1-preview.2 new features

Update samples
Update Readme

Invoices Readme and samples documentation have been added.
For the remaining features, we decided not to expose them in the Readme or specific samples until there is a concrete design on the SDK


Bugfix/1540/refactoring

Description
Fixed threw https://docs.google.com/spreadsheets/d/12P5AgSMLZywGaZsVRWR87HAODDG04-F7YM35R2qGeh4/edit#gid=1661224271

Refactored code of pages Login, GEnerate Password, Restore Password to meet the one standard.
 Please revise the file structure for them.
Changed background
Added adaptive design
Fixed unpretty loader Rectangle

Screenshots

Original
Updated

**  **
**  **

MOBILE

Fixed Loader Background

Checklist

[x ] 🔽 My branch is up-to-date with "development" branch
[x ] ✅All tests passed locally
[x] ✨My changes working with up-to-date admin and back-end part locally, like charm
[x ] 🔗 Link pull request to issue

Updated. After creating a new brunch I got an error message about missing
this module. So I installed it to avoid seeing errors. I will backup
package-json before merging, thank you.
On Fri, Dec 24, 2021 at 7:25 PM Filiczini @.***> wrote:

@.**** commented on this pull request.
What happend with i18next ?
—
Reply to this email directly, view it on GitHub
https://github.com/ita-social-projects/horondi_client_fe/pull/1551#pullrequestreview-839973032,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF6D6C6UAASVDM6KXOOXKADUSSUG7ANCNFSM5KW3FRJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***
com>


[Firebase Storage] getReferenceFromUrl returns wrong storage reference path

I have a file which resides in the following path in Firebase storage: /users/{userid}/avatar.jpg
I'm putting this file in storage, then retrieve its download URL via getDownloadURL method. Afterwards, I'm saving this download URL in Firestore.
Whenever I want to delete this file from storage, I'm using this download URL already saved in Firestore to retrieve the storage reference of this file. So, I'm using this method:
StorageReference avatarRef =
        await _firebaseStorage.getReferenceFromUrl(user.avatarUrl);

When I debug this avatarRef object, I see that its path field has the value: /users/{userid}/{userid}/avatar.jpg
As you can see the path involves duplicate {userid}s. Therefore avatarRef.delete() operation fails due to this wrong reference path.
I tested this on an Android device. And here is my flutter doctor output:
[✓] Flutter (Channel beta, v1.17.0, on Mac OS X 10.15.4 19E287, locale en-TR)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
[✓] Android Studio (version 3.6)
[✓] VS Code (version 1.45.0)
[✓] Connected device (1 available)

• No issues found!

Hi @ugurcany
can you please provide your flutter run --verbose?
Also, to better address the issue, would be helpful if you could post a minimal code sample to reproduce the problem
Thank you

Hi @TahaTesser
The following is the code snippet:
//DEFINE AVATAR PATH ON STORAGE
1.    StorageReference avatarRef =
        await FirebaseStorage.instance.getReferenceFromUrl(aValidDownloadUrl);

//DELETE AVATAR IMAGE FILE
2.    await avatarRef.delete();

If you put debug point on the line 2 and run the code, you will see the path parameter inside avatarRef object has a wrong value.

It turned out that it's not related to storage sdk itself, thus closing.


Adding the option for unsub command.

Hi, as mentioned in the issue #58 I added an unsubscribe command to the project. It is something relatively frequent in Mqtt commands.  The option number is 4. So [4,"topicToUnsubscribeTo"] is now a valid serial message to send to the serialToMqtt software.

nice feature


Issue/exception when using GitHub 3

Need to fix Test structure by refactoring:

Make universal format for test database creation.
Fix tests :3


Git: Cannot read property 'toString' of undefined

Issue Type: Bug
HI!
Everything was fine until I added two keyboard shortucts to my Keyboard Shortcuts settings page (Ctrl+k, Ctrl+s)
I added:

Ctrl + Shift + G , Ctrl + A to Stage All Changes
and
Ctrl + Shift + G , Ctrl + S to Stage Change

and now when I open source control pane, I select a file &  do Ctrl + Shift + G , Ctrl + S, I get this error:

I have done this shortcuts with a previous installation of windows 10 + code insiders and am sure it worked until yesterday morning. But I reinstalled my windows 10 yesterday (same version as before) and installed vs-code-insiders again (same version as before), But this time I'm getting this error
I also tried with my add-ons/extensions disabled, still the same error!
P.S.
But the Ctrl + Shift + G , Ctrl + A  shortcut (Stage All Changes) works perfectly! Strange :)
THANKS!
VS Code version: Code - Insiders 1.31.0-insider (ece3a90fb58986570d8afdfb16530c4dd8c743af, 2018-12-14T06:19:42.976Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 x 2808)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.88GB (3.94GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (7)

Extension
Author (truncated)
Version

gitlens
eam
9.2.1

tslint
eg2
1.0.41

prettier-vscode
esb
1.7.2

camelcasenavigation
map
0.3.51

vsliveshare
ms-
0.3.1071

vscode-docker
Pet
0.4.0

vscode-icons
rob
8.0.0

Unfortunately there is no support to configure the Stage Change command to a keybinding.
/duplicate #24916

@joaomoreno Thanks for your reply!
But as I mentioned I did it already in a previous version of Windows & VS-Code (it was just last week) and it worked fine.
I did the same thing exactly with Ctrl + Shift + G , Ctrl + S
And yes! I'm pretty sure

Oh I missed that, let me check...

Are you sure that you are not confusing 'Stage Change' with 'Stage Changes'. The latter works as expected for me. I cannot find where the former is used.

I get the same error when I execute the "Git: Revert Change" command (VSCode 1.40, 1.39). It worked on previous VSCode versions too (I'm not sure when it broke exactly). There is no error output in the Git Log.

@edespong sorry I missed your question.
I don't encounter this issue anymore and don't know how to reproduce it now. I don't remember that it was stage change or stage changes

I get the same error when I execute the "Git: Revert Change" command (VSCode 1.40, 1.39). It worked on previous VSCode versions too (I'm not sure when it broke exactly). There is no error output in the Git Log.

Same on my windows 10 (VSCode 1.40.1, Git 2.21.0.windows.1)

I get also the same problem on Linux, insider's edition, any pointers on where to look to try and fix it?

Same issue here. Get the error when pressing "CTL+g" after setting that as hotkey for "Git: Stage changes" .   Pressing F1 and then selecting that operation however does work as intended.
I like "CTRL+g" to stage changes it seems intuitive to me.
Though being able to map a hotkey to stage a specific block vs the entire fire would be better; in that case I'd probably map CTRL+shift+g to whole file and CTRL+g to single block.

I get the same error when I execute the "Git: Revert Change" command (VSCode 1.40, 1.39). It worked on previous VSCode versions too (I'm not sure when it broke exactly). There is no error output in the Git Log.

I found out that the "Git: Revert Change" command only works when there is an active selection. In IntelliJ it also works without selection, and just resets the hunk at the current cursor position, which is very helpful. Should I create a separate ticket for this?

Is there any update on this ?
I also had this issue when binding shortcut for git.revertChange


feat(proxy): Support UNSELECT

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 57.678%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
477

Relevant Lines:
827

💛  - Coveralls


Issues with docker

| 2021-03-07 20:39:27 0 [Note] InnoDB: Starting crash recovery from checkpoint LSN=61030
mariadb_1  | 2021-03-07 20:39:28 0 [Note] InnoDB: 128 out of 128 rollback segments are active.
mariadb_1  | 2021-03-07 20:39:28 0 [Note] InnoDB: Removed temporary tablespace data file: "ibtmp1"
mariadb_1  | 2021-03-07 20:39:28 0 [Note] InnoDB: Creating shared tablespace for temporary tables
mariadb_1  | 2021-03-07 20:39:28 0 [Note] InnoDB: Setting file './ibtmp1' size to 12 MB. Physically writing the file full; Please wait ...
mariadb_1  | 2021-03-07 20:39:28 0 [Note] InnoDB: File './ibtmp1' size is now 12 MB.
mariadb_1  | 2021-03-07 20:39:28 0 [Note] InnoDB: 10.4.18 started; log sequence number 61039; transaction id 20
mariadb_1  | 2021-03-07 20:39:28 0 [Note] InnoDB: Loading buffer pool(s) from /var/lib/mysql/ib_buffer_pool
mariadb_1  | 2021-03-07 20:39:28 0 [Note] Plugin 'FEEDBACK' is disabled.
mariadb_1  | 2021-03-07 20:39:28 0 [Note] Server socket created on IP: '::'.
mariadb_1  | 2021-03-07 20:39:28 0 [Note] InnoDB: Buffer pool(s) load completed at 210307 20:39:28
mariadb_1  | 2021-03-07 20:39:28 0 [Note] Reading of all Master_info entries succeeded
mariadb_1  | 2021-03-07 20:39:28 0 [Note] Added new Master_info '' to hash table
mariadb_1  | 2021-03-07 20:39:28 0 [Note] mysqld: ready for connections.
mariadb_1  | Version: '10.4.18-MariaDB-1:10.4.18+maria~focal'  socket: '/var/run/mysqld/mysqld.sock'  port: 3306  mariadb.org binary distribution
redis_1    | 1:C 07 Mar 2021 20:39:27.574 # oO0OoO0OoO0Oo Redis is starting oO0OoO0OoO0Oo
redis_1    | 1:C 07 Mar 2021 20:39:27.574 # Redis version=6.2.1, bits=64, commit=00000000, modified=0, pid=1, just started
redis_1    | 1:C 07 Mar 2021 20:39:27.574 # Warning: no config file specified, using the default config. In order to specify a config file use redis-server /path/to/redis.conf
redis_1    | 1:M 07 Mar 2021 20:39:27.575 * monotonic clock: POSIX clock_gettime
redis_1    | 1:M 07 Mar 2021 20:39:27.575 * Running mode=standalone, port=6379.
redis_1    | 1:M 07 Mar 2021 20:39:27.575 # Server initialized
redis_1    | 1:M 07 Mar 2021 20:39:27.575 # WARNING overcommit_memory is set to 0! Background save may fail under low memory condition. To fix this issue add 'vm.overcommit_memory = 1' to /etc/sysctl.conf and then reboot or run the command 'sysctl vm.overcommit_memory=1' for this to take effect.
redis_1    | 1:M 07 Mar 2021 20:39:27.575 * Ready to accept connections
acs_1      | 2021-03-07 20:39:28,398 INFO Included extra file "/etc/supervisor/conf.d/goacs.conf" during parsing
acs_1      | 2021-03-07 20:39:28,398 INFO Set uid to user 0 succeeded
acs_1      | 2021-03-07 20:39:28,416 INFO RPC interface 'supervisor' initialized
acs_1      | 2021-03-07 20:39:28,417 CRIT Server 'unix_http_server' running without any HTTP authentication checking
acs_1      | 2021-03-07 20:39:28,417 INFO supervisord started with pid 1
acs_1      | 2021-03-07 20:39:29,419 INFO spawned: 'goacs' with pid 9
acs_1      |
acs_1      | Warning: require(/app/vendor/autoload.php): Failed to open stream: No such file or directory in /app/artisan on line 18
acs_1      |
acs_1      | Fatal error: Uncaught Error: Failed opening required '/app/vendor/autoload.php' (include_path='.:/usr/local/lib/php') in /app/artisan:18
acs_1      | Stack trace:
acs_1      | #0 {main}
acs_1      |   thrown in /app/artisan on line 18
acs_1      | 2021-03-07 20:39:30,464 INFO success: goacs entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
acs_1      | 2021-03-07 20:39:30,464 INFO exited: goacs (exit status 255; not expected)
acs_1      | 2021-03-07 20:39:31,469 INFO spawned: 'goacs' with pid 10
acs_1      |
acs_1      | Warning: require(/app/vendor/autoload.php): Failed to open stream: No such file or directory in /app/artisan on line 18
acs_1      |
acs_1      | Fatal error: Uncaught Error: Failed opening required '/app/vendor/autoload.php' (include_path='.:/usr/local/lib/php') in /app/artisan:18
acs_1      | Stack trace:
acs_1      | #0 {main}
acs_1      |   thrown in /app/artisan on line 18
acs_1      | 2021-03-07 20:39:31,515 INFO exited: goacs (exit status 255; not expected)
acs_1      | 2021-03-07 20:39:32,519 INFO spawned: 'goacs' with pid 11
acs_1      |
acs_1      | Warning: require(/app/vendor/autoload.php): Failed to open stream: No such file or directory in /app/artisan on line 18
acs_1      |
acs_1      | Fatal error: Uncaught Error: Failed opening required '/app/vendor/autoload.php' (include_path='.:/usr/local/lib/php') in /app/artisan:18
acs_1      | Stack trace:
acs_1      | #0 {main}
acs_1      |   thrown in /app/artisan on line 18
acs_1      | 2021-03-07 20:39:32,567 INFO exited: goacs (exit status 255; not expected)
acs_1      | 2021-03-07 20:39:34,572 INFO spawned: 'goacs' with pid 12
acs_1      |
acs_1      | Warning: require(/app/vendor/autoload.php): Failed to open stream: No such file or directory in /app/artisan on line 18
acs_1      |
acs_1      | Fatal error: Uncaught Error: Failed opening required '/app/vendor/autoload.php' (include_path='.:/usr/local/lib/php') in /app/artisan:18
acs_1      | Stack trace:
acs_1      | #0 {main}
acs_1      |   thrown in /app/artisan on line 18
acs_1      | 2021-03-07 20:39:34,613 INFO exited: goacs (exit status 255; not expected)
acs_1      | 2021-03-07 20:39:37,618 INFO spawned: 'goacs' with pid 13
acs_1      |
acs_1      | Warning: require(/app/vendor/autoload.php): Failed to open stream: No such file or directory in /app/artisan on line 18
acs_1      |
acs_1      | Fatal error: Uncaught Error: Failed opening required '/app/vendor/autoload.php' (include_path='.:/usr/local/lib/php') in /app/artisan:18
acs_1      | Stack trace:
acs_1      | #0 {main}
acs_1      |   thrown in /app/artisan on line 18
acs_1      | 2021-03-07 20:39:37,649 INFO exited: goacs (exit status 255; not expected)
acs_1      | 2021-03-07 20:39:38,651 INFO gave up: goacs entered FATAL state, too many start retries too quickly

Hi,
Same issue here.
I use Docker Desktop for macos.
Do you have a workaround/a quickfix for this ?
Thanks for your work :)

I need to build a full image with the whole application and publish it in docker hub....  I will do it this month


Search not working when typing "script"

After typing "script" in the search box, nothing is changed and still all emojis are listed.
Screenshot:

Thanks for reporting. I will look into this.

I think the script here is actually description, same result when typing that


New labels annotation tool and tensorstore

🐛 Bug Report
Using the new annotation tool in the labels layer with tensorstore doesn't provide any feedback when the saving operation is unsuccessful.
💡 Steps to Reproduce
A test showing interaction with a tensorstore array (numpy array for comparison).
The array has two slices to easily demonstrate re-reading of the annotation.
import napari
import zarr
import tensorstore as ts
import numpy as np

array_size = (2,2000, 2000)
np_array = np.zeros(array_size, dtype='uint32')

zarr_path = r'd://example.zarr'
z = zarr.zeros(array_size, chunks=(1,1000, 1000), dtype='uint32')
zarr.save(zarr_path, z)

spec = {
  'driver': 'zarr',
  'kvstore': {
    'driver': 'file',
    'path': zarr_path,
  },
}

ts_array = ts.open(spec).result()

viewer = napari.Viewer()
viewer.add_labels(ts_array,name='ts')
viewer.add_labels(np_array,name='np')

https://github.com/napari/napari/assets/7549583/e30eb1d6-e657-403f-9584-c590c601597f
💡 Expected Behavior
Maybe an error message that saving to the zarr array failed in this situation?
🌎 Environment
napari: 0.5.0a2.dev606+gb3e15c51
Platform: Windows-10-10.0.19045-SP0
Python: 3.10.13 | packaged by conda-forge | (main, Dec 23 2023, 15:27:34) [MSC v.1937 64 bit (AMD64)]
Qt: 5.15.2
PyQt5: 5.15.10
NumPy: 1.26.4
SciPy: 1.12.0
Dask: 2024.3.1
VisPy: 0.14.2
magicgui: 0.8.2
superqt: 0.6.2
in-n-out: 0.2.0
app-model: 0.2.5
npe2: 0.7.4
💡 Additional Context
It's related to the performance and when the classical drawing tool (brush) is responsive, the new tool is working correctly too. Yet, this performance issue seems unavoidable with an HDD drive (in the example that I provide, a small 2k x 2k px array is not performant) and when the brush is used the problem is obvious. In contrast with the new tool, the annotation is displayed correctly but stored incorrectly in the underlying zarr.

I can reproduce this.
Using the script in the OP painting in the tensorstore array is slow, but not as bad as in the video.
But when I use the polygon labels tool to draw two rectangles (on the lower right) I get two glitches instead.

I tested with a zarr on disk without tensorstore, using:
z2 = zarr.open('test_zarr.zarr', mode='w', shape=(2, 2000, 2000), chunks    ...: =(1, 1000, 1000), dtype='i4')
Performance of painting felt similar to tensorstore and the polygon tool worked!

I was playing with this pattern:
https://google.github.io/tensorstore/python/api/tensorstore.Transaction.html
Script:
import numpy as np
import napari
import tensorstore as ts
import zarr
array_size = (2,2000, 2000)
np_array = np.zeros(array_size, dtype='uint32')

zarr_path = r'd://example.zarr'
z = zarr.zeros(array_size, chunks=(1,1000, 1000), dtype='uint32')
zarr.save(zarr_path, z)

spec = {
  'driver': 'zarr',
  'kvstore': {
    'driver': 'file',
    'path': zarr_path,
  },
}

ts_array = ts.open(spec).result()

txn = ts.Transaction()

viewer = napari.Viewer()
viewer.add_labels(ts_array.with_transaction(txn), name='ts')
viewer.add_labels(np_array,name='np')

Painting is much more performant and glitches with the polygon tool less frequent:

The catch is you need to commit the changes to the array using txn.commit_async() but that will break making changes to the open layer, so you need to re-open the store.
Maybe this could be made more graceful via a plugin? Widget with commit button that commits and re-opens with a new transaction?

Playing a bit further, not sure if this is too hacky, but it does work:
paint, commit, refresh transaction, point layer data at the new transaction (just refreshing doesn't suffice):
In [13]: txn = ts.Transaction()

In [14]: viewer.add_labels(ts_array.with_transaction(txn))
Out[14]: <Labels layer 'Labels' at 0x2af2b0e90>

In [15]: txn.commit_async()
Out[15]: <tensorstore.Future at 0x2ae3bdd80>

In [16]: txn = ts.Transaction()

In [17]: viewer.layers[0].data = ts_array.with_transaction(txn)

In [18]: txn.commit_async()
Out[18]: <tensorstore.Future at 0x2b1d61d80>

In [19]: txn = ts.Transaction()

In [20]: viewer.layers[0].data = ts_array.with_transaction(txn)

I assume you could go on painting indefinitely	before doing the commit, so it's actually not bad from a UX point of view. It's just save basically, consisting of commit, make a new transaction, re-home the layer.data. But too specific for napari to handle, still fine for a plugin/script/notebook.
CC: @jni not sure who else uses tensorstore and my google-fu hasn't come up with much in this area.

@psobolewskiPhD I haven't played that much with Tensorstore beyond getting plain-old-paint to work, and in fact you are now the expert. 😂 But that exploration seems very interesting. I've thought about a general approach here in which we paint only the view pixels synchronously, and paint into the data asynchronously.
Anyway, the performance issue is, I think, secondary to the polygon glitching issue. I wonder whether it's related to my issues with triangulation in #6654 #5673. I don't think the two issues are related, though it's interesting that you can reduce the glitching by using transactions...
My suspicion is that there's something a bit weird going on with tensorstore indexing — it is a little bit different from NumPy indexing in that it maintains the index of sliced dimensions:
In [4]: arr = ts.array(np.arange(5*10*12).reshape((5, 10, 12)))

In [5]: b = arr[1:3]

In [6]: b[0]
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
Cell In[6], line 1
----> 1 b[0]

IndexError: OUT_OF_RANGE: Checking bounds of constant output index map for dimension 0: Index 0 is outside valid range [1, 3) [domain='{origin={1, 0, 0}, shape={2, 10, 12}}'] [transform='Rank 2 -> 3 index space transform:   Input domain:     0: [0, 10)     1: [0, 12)   Output index maps:     out[0] = 0     out[1] = 0 + 1 * in[0]     out[2] = 0 + 1 * in[1] '] [source locations='tensorstore/index_space/internal/propagate_bounds.cc:109\ntensorstore/index_space/internal/propagate_bounds.cc:109\ntensorstore/index_space/internal/compose_transforms.cc:113\ntensorstore/index_space/index_transform.h:106']

In [7]: b[1]
Out[7]:
TensorStore({
  'array': [
    [120, 121, 122, 123, 124, 125, 126, 127, 128, 129, 130, 131],
    [132, 133, 134, 135, 136, 137, 138, 139, 140, 141, 142, 143],
    [144, 145, 146, 147, 148, 149, 150, 151, 152, 153, 154, 155],
    [156, 157, 158, 159, 160, 161, 162, 163, 164, 165, 166, 167],
    [168, 169, 170, 171, 172, 173, 174, 175, 176, 177, 178, 179],
    [180, 181, 182, 183, 184, 185, 186, 187, 188, 189, 190, 191],
    [192, 193, 194, 195, 196, 197, 198, 199, 200, 201, 202, 203],
    [204, 205, 206, 207, 208, 209, 210, 211, 212, 213, 214, 215],
    [216, 217, 218, 219, 220, 221, 222, 223, 224, 225, 226, 227],
    [228, 229, 230, 231, 232, 233, 234, 235, 236, 237, 238, 239],
  ],
  'context': {'data_copy_concurrency': {}},
  'driver': 'array',
  'dtype': 'int64',
  'transform': {'input_exclusive_max': [10, 12], 'input_inclusive_min': [0, 0]},
})


[LOW] Zod denial of service vulnerability

Github reported a new dependabot security alert at: https://github.com/cdk8s-team/cdk8s-cdktf-resolver/security/dependabot/2

Fixed


feature request: Allow lazy loading

Now, the rule raises errors with the following codes
const LazyComponent = React.lazy(
  () => import("./MyComponent"),
);

Fast refresh only works when a file only exports components. Move your component(s) to a separate file.eslint(react-refresh/only-export-components)

Can we modify to determine this as the correct code?

Can you give more context? This code creates a component so it should follow some Fast refresh rules

I made a repo to repro
https://github.com/januswel/react-refresh-sample

This problem may occur with RouteObjects react-router v6

This is the same issue as https://github.com/ArnaudBarre/eslint-plugin-react-refresh/issues/25
I need to dig deeper to see how it really behaves at runtime. Given that SWC and Babel generate some fast refresh code for that for now it's safer to consider that React.lazy calls are like creating a wrapper component (which is the case)

@ArnaudBarre
Thank you for your comment
I saw #25 and consider to use workarounds from you

I'm closing this one for the same reason as #25, see https://github.com/ArnaudBarre/eslint-plugin-react-refresh/issues/25#issuecomment-2496199809


refactor: rely on aws-sdk being available

Current dependencies on/for this PR:

main

PR #64   👈

This comment was auto-generated by Graphite.


[Question] Why is "localhost.tiangolo.com" accessible?

Hey guys,
I am confused. Even though in my .env file the line "DOMAIN=localhost.tiangolo.com" is commented out, I can reach the frontend if I visit http://localhost.tiangolo.com.
I used the global search in my IDE to find an appropriate line in the project code that enables this, but haven't found anything.
Would be happy if someone could explain it to me :) Thank you!

Hey there! That domain is configured to point to <IP_ADDRESS>, your own localhost. That's why it works. 🤓
Thanks for closing the issue!

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


how to install ffmpeg universal package

how to install ffmpeg universal package?
the ffmpeg seems support the universal build ,but how to achieve this?

You can install it by command .\vcpkg.exe install ffmpeg:x64-uwp.

thanks ,this is the command on Windows
what is the corresponding command on Macos?

The installation command for macos is: .\vcpkg.exe install ffmpeg. You can try to install using command. I think this issue can be closed, if you has new quesion, plase reopen this issue.

thanks ,this is the command on Windows what is the corresponding command on Macos?

Please operate according to the tutorial.


[in_app_purchase] [android] Add a field in PurchaseDetails, that maps the purchaseToken found in Android Purchase

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
In the in_app_purchase package, PurchaseDetails class, does not contain a field that maps the purchaseToken, which is available in the Purchase class of Android's BillingClient API. This purchaseToken is in fact the most important field when working with androidPublisher APIs on the backend, for subscriptions. Not having this token as a field is a big drawback, as the androidPublisher API consistently throws 400 error.
I have tried with the purchaseId field many times, but it seems that it maps to the orderId field in the Purchase class of Android.
Please consider adding this field.
Purchase class (Android)
Acknowledge API in androidPublisher where purchaseToken is required as param
Proposal
Add a new field that maps to the purchaseToken field in the Purchase class in Android.

PurchaseToken field is present in verificationData.serverVerificationData, hence closing this issue


WebSocket Connection error

Hi,
I know there use to be a bug with "websocket connection error", but as i could see on a changelog from Ubiquiti, they fixed it "long" ago.
I'm currently running goofball222/unifi:latest which is 5.10.21 (Build: atag_5.10.21_11661)
I currently mount these ports to the container:
<IP_ADDRESS>:6789->6789/tcp, <IP_ADDRESS>:8443->8443/tcp, <IP_ADDRESS>:8843->8843/tcp, <IP_ADDRESS>:8880->8880/tcp, <IP_ADDRESS>:3478->3478/udp, <IP_ADDRESS>:8888->8888/tcp, 8080/tcp, 10001/udp
And running Mongo in a separated container.
Any idea how to fix this?

Are you accessing the controller via cloud access when you get the websocket error (https://unifi.ubnt.com)?

Are you accessing the controller via cloud access when you get the websocket error (https://unifi.ubnt.com)?

i'm accessing through my own url on the server.
an Nginx is in front proxy to 8443.
My user is connected to the cloud access.

This is a fairly old issue, please try with a newer version of the Debian-based image and UniFi software and open a new issue if the problem persists.


only reconnecting after inverter restart

Version of the custom_component
0.0.26
SMA Tripower X 25 firmware 03.06.19.R

Configuration
Inverter not active on <IP_ADDRESS>:502
Failed to connect to <IP_ADDRESS>:502 slave id 126

Describe the bug
After restarting HASS or restarting the integration, SunSpec wont connect to Inverter.
Stoping and starting Modbus Server or replugging Ethernet dosnt help.
When restarting Inverter, I get data as intended without problem.
But again, when I wanna change the config SunSpec tells me it can't connect to Inverter even though data is recieved and everything works as intended.
Debug log

2024-08-22 15:11:36.089 WARNING (MainThread) [custom_components.sunspec] Inverter not active on <IP_ADDRESS>:502
2024-08-22 15:12:25.419 WARNING (MainThread) [custom_components.sunspec] Failed to connect to <IP_ADDRESS>:502 slave id 126

Same. Reverted to 0.0.25

Temp fix for me was downgrading to 0.0.25 and replacing sensor.py with sensor.py from 0.0.26 (Constants in use by 0.0.25 get removed Jan 2025).
Suspect "Bump pysunspec2 from 1.0.8 to 1.1.5" is the source of this new issue.

That worked for me as well.  Thank you for pointing me in the right direction @SmileyMan

I see the error "Failed to connect to host (hostname):502 slave 1 - Inverter not active on (hostname):502".


coder: add update script

Description of changes
Adds an updateScript to automatically update coder as new releases become available.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

I suggest you to not update the package. Just add the update script. And then, let the bot run the update script to update the package. So you can be sure it will work all right.

Sure. Will change!

@superherointj fixed!

I'd recommend testing to run script locally in the update script directory and also at top-level of nixpkgs.
Depending on how the script is triggered, it may be meaningful. IIRC, the bot uses top-level. And maintainers often trigger locally.

@superherointj fixed to cd first.

Because GitHub imposes a limit on API requests, it's good practice to add GITHUB_TOKEN to API requests. See:
https://github.com/NixOS/nixpkgs/blob/master/pkgs/applications/networking/cluster/fluxcd/update.sh#L10

@superherointj fixed!

@superherointj seems like the build failed: https://r.ryantm.com/log/coder/2024-04-14.log
I'm unsure what to do in this situation; it looks like something infrastructure-related. I'd appreciate any help!

@superherointj fix here: https://github.com/NixOS/nixpkgs/pull/304314


Export MaskRegions

I would like to be able to use the MaskRegions type in my code that interfaces with pdf-visual-dff

I'd love to use a new version with this in if possible! 🙇♂

I'd love to use a new version with this in if possible! 🙇♂

Sure thing! Published as 0.9.0 :)


[bug] HTTPS support for on_publish

Expected behavior
I can use https-urls in on_publish and on_publish_done
Actual behavior
I get the following error complaining about invalid port.
nginx: [emerg] invalid port in url "https://<IP_ADDRESS>/api/stream/on_publish" in /opt/nginx/conf/nginx.conf:45
Steps to reproduce the behavior

Use the following config in the rtmp application.
on_publish https://<IP_ADDRESS>/api/stream/on_publish; on_publish_done https://<IP_ADDRESS>/api/stream/on_publish_done;

Hi,
I can also confirm this is the case for push directive.
For example, the following:
push rtmps://live-api-s.facebook.com:443/rtmp/abcd;

Returns:
nginx: [emerg] invalid port in url "rtmps://live-api-s.facebook.com:443/rtmp/abcd" in /opt/nginx/conf/nginx.conf:10

Encrypted HTTP is not suppported in commands on_xxx.
@TheRealMattLear you can use stunnel to convert rtmp to rtmps.

@TheRealMattLear you can use stunnel to convert rtmp to rtmps.

For push targets i would imagine this to be requirement, especially since facebook ONLY accepts secure connections. Using a 3rd party tool to achieve a secure connection is a work around rather than a complete solution. Could I create a feature request for this instead?

@TheRealMattLear
Well, secure RTMP feature will not be supported in this open source project.
In fact, I've developed this new feature since I saw Facebook announced that it would ONLY accept RTMPS on May 1st, 2019 and now it works well as a server or downstream server. I've spent so much time on it and it's cost me much on cloud server, domain name (some sites, Facebook for example, and encryption certificates doesn't support naked IP) and encryption certificates to test it. So, I will not open the source and it is non-free.
However, it can be provided for some one who has contributed important features to this open source project for free.
For more details, you can contact me via E-Mail :)

Hi,
For now we have managed to have this working by using ffmpeg to pass-thru the stream. This uses minimal resources on the system. Thank you
Best Regards
Matthew Lear – Director

From: winshining<EMAIL_ADDRESS>Sent: Thursday, 9 April 2020 2:46 AM
To: winshining/nginx-http-flv-module<EMAIL_ADDRESS>Cc: Matthew Lear<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [winshining/nginx-http-flv-module] [bug] HTTPS support for on_publish (#130)
@TheRealMattLearhttps://github.com/TheRealMattLear
Well, secure RTMP feature will not be supported in this open source project.
In fact, I've developed this new feature since I saw Facebook announced that it would ONLY accept RTMPS on May 1st, 2019 and now it works well as a server or downstream server. I've spent so much time on it and it's cost me much on cloud server, domain name (some sites, Facebook for example, and encryption certificates doesn't support naked IP) and encryption certificates to test it. So, I will not open the source and it is non-free.
However, it can be provided for some one who has contributed important features to this open source project for free.
For more details, you can contact me via E-Mail :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/winshining/nginx-http-flv-module/issues/130#issuecomment-611068131, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACFTTLTSPWFIRTPM2WKY54LRLSS77ANCNFSM4JAAU3FQ.


New user guide: Which DEA Services to use?

Preview: https://pr-308-preview.khpreview.dea.ga.gov.au/guides/access/which-dea-services/
For Knowledge Request: Log request for minor KH updates
Based on Claire P's word document: https://geoscienceau.sharepoint.com/:w:/r/sites/DEGroup/_layouts/15/Doc.aspx?sourcedoc={4BD0E288-ECD4-436B-A207-931DA89F65C7}&file=DEA Services on a page - draft.docx&wdOrigin=TEAMS-WEB.teams_ns.rwc&action=default&mobileredirect=true

[ ] I have spellchecked my content using Microsoft Word, Grammarly, or otherwise.
[ ] If required, update the DEA Tech Alerts and Changelog and the DEA Sandbox Updates banner.

Hey @benji-glitsos-ga, this is looking awesome. Just one thought: the user guides page is already busy and packed with material... is there any way we could streamline/combine any of the following docs? There's already some overlap here, and I just think the cleaner we can get these guides, the easier it will be for users to digest them...

Hi @robbibt , yep, I'll look into combining this into the Getting Started with DEA guide.

Hi @erialC-P , this is now ready for review. See the preview: https://pr-308-preview.khpreview.dea.ga.gov.au/guides/setup/README/


Added rules,cloud credentials,schedule policy share helper functions

What this PR does / why we need it:
Added helper function to update ownership for rules, cloud credentials, and schedule policy.
Which issue(s) this PR fixes (optional)
Closes #PA-1342
Special notes for your reviewer:

@ak-px Please resolve the merge conflict. Also please check why all the checks are failing.

Can we run a test to confirm that these functions are working as intended?

@mkoppal-px / @sn-px , please the find the logs for testing the function added. I'll be removing the script added after final approval.
https://jenkins.pwx.dev.purestorage.com/job/portworx-backup/job/system-tests/job/byoc/job/px-backup-on-demand-system-test-byoc/2005/
https://jenkins.pwx.dev.purestorage.com/job/portworx-backup/job/system-tests/job/byoc/job/px-backup-on-demand-system-test-byoc/2006/console


Axis exercise

feat: Axis Exercise Component
Summary
Closes #13
Created axis exercise component that takes in an orientation, markers and labels to generate an axis.

Currently incomplete! Just wanted to get some feedback on how its looking etc.

Currently waiting on the turtle SVG to be centred at the shell to fix positioning issue.


support SET Timezone

Which issue does this PR close?

Closes #4106
Rationale for this change

a POC for #4106
to support set time zone
❯ show time zone;
+--------------------------------+---------+
| name                           | setting |
+--------------------------------+---------+
| datafusion.execution.time_zone | UTC     |
+--------------------------------+---------+
1 row in set. Query took 0.069 seconds.

❯ set time zone = '+08:00';
0 rows in set. Query took 0.001 seconds.

❯ show time zone;
+--------------------------------+---------+
| name                           | setting |
+--------------------------------+---------+
| datafusion.execution.time_zone | +08:00  |
+--------------------------------+---------+
1 row in set. Query took 0.007 seconds.

casting to timestamptz should use the timezone instead of the fixed UTC
❯ select arrow_typeof('2000-01-01T00:00:00'::timestamp::timestamptz);
+------------------------------------------+
| arrowtypeof(Utf8("2000-01-01T00:00:00")) |
+------------------------------------------+
| Timestamp(Nanosecond, Some("+08:00"))    |
+------------------------------------------+
1 row in set. Query took 0.007 seconds.

What changes are included in this PR?

Are there any user-facing changes?

Thank you again @waitingkuo  for driving this forward

Thanks again @waitingkuo


vscode, vscodium: 1.62.3 -> 1.63.0

Motivation for this change
It builds and launches but please just consider me as samuela's nixpkgs-upkeep bot.
https://code.visualstudio.com/updates/v1_63
Closes #149842
Closes #149843
Things done

[x] Fits CONTRIBUTING.md.
[x] Runs the updateScript.
[x] Makes it builds on x86_64-linux:

- substituteInPlace $unpacked/sudo-prompt/index.js \
+ substituteInPlace<EMAIL_ADDRESS>\

@ofborg build vscodium
@ofborg test vscodium

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

5 packages built:

vscode
vscode-fhs
vscode-with-extensions
vscodium
vscodium-fhs

Executable tested and works

I prefer this over #149842 & #149843 because this actually seems to work ;)

@ofborg test vscodium

Thanks for this @bobby285271 !


Updating DTree and LR schema to allow string labels.

This has been resolved by two other pull requests that have since been merged to master:

https://github.com/IBM/lale/pull/189
https://github.com/IBM/lale/pull/190


Integrate LLVM at llvm/llvm-project@cae4067ec1cd

Update BUILD files and submodule for
cae4067ec1cd

I'm a little uncomfortable with the zero-review merges with actual BUILD file updates.... do you actually want review? I was starting to look at this and noticed it had already been merged.

I've been following the LLVM standard of submitting without review and getting post-commit review for simple changes. These are necessary to keep the build green, so I think merging them quickly is high-value. Most of these are just ported from internal patches that were authored by someone related to the commit they were for and seem non-controversial.


Update Cycle Primitives

CyclePrimitives
Update the following to use ### RobotElementNames::State_USAGE

CyclePrimitives
PrimitiveParams
PrimitiveParser
auton.dtd

This is duplicate so closing this one


rigid body bullets?

i like it when bullets collide with enemy bullets- it gives you a sense of control.

qurobullet is not designed to work this way, unfortunately; it's built to handle huge numbers of bullets with high performance. As such, they're written as plain Objects for speed, and even just implementing them as plain Nodes drastically increases the processing power each bullet requires, let alone as RigidBody nodes that would take additional physics processing and have to check for collisions with each other... I have no plans to implement a feature like this anytime soon.
That said, it's not impossible to accomplish what you want with this module. Since in most games, you don't need players to be capable of firing complicated volley patterns like enemies do, you could use qurobullet specifically for the enemy bullets, and write your own simpler script for spawning bodies (or areas for better performance) for player bullets that can collide with qurobullet bullets.
You could even make use of a spawner detached from the bullet server and use its signals to spawn your custom bullets, if you wanna get fancy with it.


Outlook warning about unsafe link for resend email

anyone else getting this for outlook / office 365 recipients?
Domain is verified in resend and all DKIM / SPF  keys set... do I need to set fully qualified DMARC as well?
Issue doesnt occur for other common email clients -- Gmail / Yahoo / Proton Mail.

More information:

user is also using Gmail

Mail app on windows and McAffee antivirus which is triggering the warning

I also hit this and turned click tracking off for the moment


uninitialized constant Haml::Filters::SassImporter

Hello,
I've stumbled across this issue when upgrading my app to Rails 4.2.
It can be reproduced with a new rails project.

add following gem dependencies:

gem 'haml-rails', '~> 0.5.3'
gem 'compass-rails', '~> 2.0.3'

rename application.css in application.css.scss and add @import 'compass';

add the following config in application.rb:
config.assets.enabled = true (important part!)

generate a model, a controller and view. In the view add inline scss:

:scss
  h1 {
    color: yellow;
  }

When trying to access the view, the following error will be raised:
ActionView::Template::Error (uninitialized constant Haml::Filters::SassImporter):
  compass-rails (2.0.3) lib/compass-rails/patches/sass_importer.rb:58:in `sass_importer_artiy'
  compass-rails (2.0.3) lib/compass-rails/patches/sass_importer.rb:63:in `sass_importer'
  compass-rails (2.0.3) lib/compass-rails/patches/sass_importer.rb:18:in `block in evaluate'
  compass-rails (2.0.3) lib/compass-rails/patches/sass_importer.rb:18:in `map'
  compass-rails (2.0.3) lib/compass-rails/patches/sass_importer.rb:18:in `evaluate'
  tilt (1.4.1) lib/tilt/template.rb:103:in `render'
  haml (4.0.6) lib/haml/sass_rails_filter.rb:8:in `render'
  (eval):5:in `render_with_options'
  haml (4.0.6) lib/haml/filters.rb:184:in `block in compile'
  haml (4.0.6) lib/haml/filters.rb:162:in `instance_eval'
  haml (4.0.6) lib/haml/filters.rb:162:in `compile'
  haml (4.0.6) lib/haml/filters.rb:141:in `internal_compile'
  haml (4.0.6) lib/haml/compiler.rb:273:in `compile_filter'
  haml (4.0.6) lib/haml/compiler.rb:20:in `compile'
  haml (4.0.6) lib/haml/compiler.rb:22:in `block (2 levels) in compile'
  haml (4.0.6) lib/haml/compiler.rb:22:in `each'
  haml (4.0.6) lib/haml/compiler.rb:22:in `block in compile'
  haml (4.0.6) lib/haml/compiler.rb:95:in `compile_root'
  haml (4.0.6) lib/haml/compiler.rb:22:in `compile'
  haml (4.0.6) lib/haml/engine.rb:66:in `initialize'
  haml (4.0.6) lib/haml/template/plugin.rb:25:in `new'
  haml (4.0.6) lib/haml/template/plugin.rb:25:in `compile'
  haml (4.0.6) lib/haml/template/plugin.rb:30:in `call'

As a workaround, I prevented the load of sass_rails_filter by removing config.assets.enabled = true in application.rb (it's supposed to be by default now anyway). If I understand correctly it only prevents me from using rails helpers in inline sass.
As a sidenote, the test app.config.assets.enabled in haml-4.0.6/lib/haml/railtie.rb seems buggy since app.config.assets.enabled is nil for a new rails app even though asset pipeline is enabled by default...
Please tell me if you need any more information.
You might also consider that the issue belongs to compass-rails... Let me know!

+1
I get the error in a view rendered by ActionMailer

I downgraded sass-rails to 4.0.5 and compass-rails to '1.1.7' and it solved the problem on my side.  Strange.

Oh well, now I'm having this error Unsupported rails environment for compass in some of my tests.  I would have to move back up to compass-rails v2.

I believe this is a bug compass-rails, as sass-rails changed out from underneath it. I'll try and get a solution in.


Under windows, mono runs Avalonia with errors

Describe the bug
A clear and concise description of what the bug is.
Under windows, mono runs Avalonia with errors，such as mono xxxxx.dll
To Reproduce
Steps to reproduce the behavior:
mono xxxxx.dll
Expected behavior
"C:\Program Files\Mono\bin\mono.exe" --debug --debugger-agent=transport=dtsocket,server=y,suspend=y,address=<IP_ADDRESS>:55556,setpgid=y E:/Debug/Avalonia.NETCoreApp/Avalonia.NETCoreApp/bin/Debug/net5.0/Avalonia.NETCoreApp.dll
Can't find custom attr constructor image: E:\Debug\Avalonia.NETCoreApp\Avalonia.
NETCoreApp\bin\Debug\net5.0\System.Reactive.dll mtoken: 0x0a00000a due to: Could
not resolve type with token 0100000b from typeref (expected class 'System.Resou
rces.NeutralResourcesLanguageAttribute' in assembly 'System.Runtime, Version=5.0
.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a') assembly:System.Runtime
, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a type:System.
Resources.NeutralResourcesLanguageAttribute member:(null)
Unhandled Exception:
System.TypeInitializationException: The type initializer for 'Avalonia.Controls.ApplicationLifetimes.ClassicDesktopStyleApplicationLifetime' threw an exception. ---> System.TypeInitializatio
nException: The type initializer for 'Avalonia.Controls.Window' threw an exception. ---> System.TypeInitializationException: The type initializer for 'Avalonia.Controls.Primitives.TemplatedC
ontrol' threw an exception. ---> System.TypeInitializationException: The type initializer for 'Avalonia.Controls.TextBlock' threw an exception. ---> System.TypeInitializationException: The t
ype initializer for 'System.Reactive.Linq.Observable' threw an exception. ---> System.TypeInitializationException: The type initializer for 'System.Reactive.Linq.QueryServices' threw an exce
ption. ---> System.TypeLoadException: Could not resolve type with token 010000e0 from typeref (expected class 'System.Diagnostics.Debugger' in assembly 'System.Runtime, Version=<IP_ADDRESS>, Cult
ure=neutral, PublicKeyToken=b03f5f7f11d50a3a')
at System.Reactive.Linq.QueryServices.Initialize () [0x00000] in //Rx.NET/Source/src/System.Reactive/Internal/QueryServices.cs:18
at System.Reactive.Linq.QueryServices..cctor () [0x00000] in //Rx.NET/Source/src/System.Reactive/Internal/QueryServices.cs:11
--- End of inner exception stack trace ---
at System.Reactive.Linq.Observable..cctor () [0x00000] in //Rx.NET/Source/src/System.Reactive/Linq/Observable.cs:13
--- End of inner exception stack trace ---
at Avalonia.Controls.TextBlock..cctor () [0x001f7] in //src/Avalonia.Controls/TextBlock.cs:142
--- End of inner exception stack trace ---
at (wrapper managed-to-native) System.Object.icall_wrapper_mono_generic_classinit(intptr)
at Avalonia.Controls.Primitives.TemplatedControl..cctor () [0x0002d] in //src/Avalonia.Controls/Primitives/TemplatedControl.cs:38
--- End of inner exception stack trace ---
at (wrapper managed-to-native) System.Object.icall_wrapper_mono_generic_classinit(intptr)
at Avalonia.Controls.Window..cctor () [0x001f0] in //src/Avalonia.Controls/Window.cs:198
--- End of inner exception stack trace ---
at (wrapper managed-to-native) System.Object.icall_wrapper_mono_generic_classinit(intptr)
at Avalonia.Controls.ApplicationLifetimes.ClassicDesktopStyleApplicationLifetime..cctor () [0x00000] in //src/Avalonia.Controls/ApplicationLifetimes/ClassicDesktopStyleApplicationLifetime
.cs:22
--- End of inner exception stack trace ---
at (wrapper managed-to-native) System.Object.icall_wrapper_mono_generic_classinit(intptr)
at Avalonia.ClassicDesktopStyleApplicationLifetimeExtensions.StartWithClassicDesktopLifetime[T] (T builder, System.String[] args, Avalonia.Controls.ShutdownMode shutdownMode) [0x00000] in
//src/Avalonia.Controls/ApplicationLifetimes/ClassicDesktopStyleApplicationLifetime.cs:128
at Avalonia.NETCoreApp.Program.Main (System.String[] args) [0x00000] in E:\Debug\Avalonia.NETCoreApp\Avalonia.NETCoreApp\Program.cs:13
[ERROR] FATAL UNHANDLED EXCEPTION: System.TypeInitializationException: The type initializer for 'Avalonia.Controls.ApplicationLifetimes.ClassicDesktopStyleApplicationLifetime' threw an excep
tion. ---> System.TypeInitializationException: The type initializer for 'Avalonia.Controls.Window' threw an exception. ---> System.Type

You are trying to run an app compiled for net5.0 with Mono that implements net48 API. That just can not work.


Tn/unsafe handling

This PR is more a proposal on how to improve our handling of unsafe code.

I added/removed some unsafe depending on if the function has unsafe code. Also see #982
The more interesting part is in get_update_info.rs. Currently both methods ocall_get_update_info and get_update_info have unsafe code. I refactored it such that one function is safe and the other unsafe. The safe one also has a more rust like interface.

The parameters are quite easy to convert. The return value and error handling is a bit more involved but still doable.

Also added nullptr checks in get_update_info.rs. See #1113
I think we should also already consider RFC-2585: No longer treat the body of an unsafe fn as being an unsafe block.

@murerfel Maybe you also want to have a look


feat(errorHandlingConfig): add an option not to encode stack in URL

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Feature
What is the current behavior? (You can also link to an open issue here)
When there is an error in loading a module then all modules that are dependent will fail also in
loading. The throwed error contains the URL for each error. The URL contains encoded stack which leads sometimes to long URL.
What is the new behavior (if this is a feature change)?
Adding an option so that the generated URL contains only the erorr code which is informative enough
to find out the problem. closes #14744.
Does this PR introduce a breaking change?
Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:

I'm pretty sure I have signed a CLA for our company and @aalmkhieber is one of our employees. I'm not quite sure how to resolve the CLA issue. Please advise.

As discussed, it's generally nice to have at least some context, and a bit of stack can definitely help.
So what about an option to set the stack length (not the url length)? Then you can set it to 0 when you really don't want it, but provide a sensible default to show some context at least.

I'm pretty sure I have signed a CLA for our company and @aalmkhieber is one of our employees. I'm not quite sure how to resolve the CLA issue. Please advise.

I can be wrong here, but afaik your company has nothing to do with this as the contribution is done by a single user, it's that particular user who needs to sign the CLA (I guess not all future contributions of him will be related to your company, right ?).

@frederikprijck, n that is not correct. Company CLAs are a thing :smiley:
But they might need to be manually verified.
@IgorMinar, can you confirm/verify the CLA?

@oliversalzburg  @gkalpak Okay, ignore that part. Made (still makes) little sense to me if the commit is done using a single github account. But nvm I guess 🙈

@frederikprijck It's not done using a single account. I have my own account and so does every other employee. We have signed a corporate CLA with Google, so that all our employees are allowed to contribute to AngularJS.
I neglected to add @aalmkhieber to the Google Group associated with our CLA, but I have done so in the meanwhile. At least, I think I did, as the process is not as clear as I would like it to be :D
If someone could offer some feedback on the implementation, that would be awesome.

If someone could offer some feedback on the implementation, that would be awesome.

I did, here:  https://github.com/angular/angular.js/pull/16283#issuecomment-336946533

I've left more info in the first thread: https://github.com/angular/angular.js/pull/15707#issuecomment-338659805

I suggest that we can add two more options to error configuration to control the behaviour of loadModules function in terms of catched errors:

isModuleErr: specifies whether an error will be throwed for each failed module or not.
isStack: specifies whether the stack of a failed module will be passed to Minerr or not.

Implementing isModuleErr through just rethrowing the catched error will cause the stack to be changed in chrome:
https://bugs.chromium.org/p/chromium/issues/detail?id=60240
Please advise!

Implementing isModuleErr through just rethrowing the catched error will cause the stack to be changed in chrome

I couldn't reproduce it and that issue is closed as fixed 😕

I don't think adding even more config options is very developer friendly.
I think the main problem is really that modulerr is adding the stack to the message, which then gets passed to nomod, and so on.
So I added an option to remove the errorUrl from the message: http://plnkr.co/edit/6zfFojrkH4xha9HmXodj?p=preview (flatErrors for now).
Voila, the message is much easier to read.
And as far as I can tell, you don't lose much stack info, because the stack only points at AngularJS internals anyway.

@Narretz I don't get it. I complain about the error message and I'm being told "Hey, we're adding configuration capabilities. That's the way to solve this problem."
Now we're adding configuration options to that capability and suddenly it's not the way to go?

@oliversalzburg
I explained why I think those specific config options that you suggested aren't a good fit.
a) the first one is removing all context from the error url, even if it's just a short string = too aggressive
b) the second and third options are too specific. They only affect one type of error message and if they are not the default, it's imo too difficult for a dev to find out that these specific config options affect their error messages.
This is my opinion and other members might disagree, however I feel like we should discuss the options, but instead I didn't get any clear response to the points I made, which I find rather strange tbh, especially if the answer is simply "more code".

I agree that the second and the third one is too specific but they offer us a solution to this too long messages. And I think the output of the script:
http://plnkr.co/edit/6zfFojrkH4xha9HmXodj?p=preview
is very resonable for both the long and the information and making it as default would be fit. For the first option,I disagree because many developers read the console before clicking on the link so that they have a little bit information about the error and with the short link they can figure out the problem.

I will look into how my proposal affects other error messages, and put it up for discussion at the next team meeting. I guess an extra option to turn off all arguments doesn't hurt, either.

@Narretz Great!

We've put this on the back burner as we are trying to get up to speed with the 1.7 release, but I think in general we'll implement it with an option to restrict the args length (so you can set it to 0, and get the plain error url)

...


taxon_ids[] param not properly filtering observations

noticed issue in this year long plant bioblitz http://www.inaturalist.org/projects/joshua-tree-national-park-wildflower-watch/
'all observations' uses this param which is not restricting obs to plants (taxon_id 47126)
http://www.inaturalist.org/observations?d1=2015-01-01T01%3A00%3A00-07%3A00&d2=2015-07-31T02%3A00%3A00-06%3A00&place_id=3680&taxon_ids[]=47126

The underlying issue here is that Joshua Tree's place boundary can't be indexed by elasticsearch for some reason, so these queries are going straight to the database, and there we're not filtering by taxon_ids[] b/c that could doing an arbitrary number of taxon descendant clauses could potentially be horrible (that's one the advantages of using ES). I can fake this kind of behavior by looking for single rules or rules involving iconic taxa, which will solve this particular problem, but it won't solve it for a project that has more than 1 taxon rule including non-iconic taxa.

got it - this must be the ),( vs )),(( kml upload issue we found with HAVO. I'll make a separate issue for that

I think this is resolved now that we're no using the elasticsearch geospatial queries. Scott, can you confirm?

Sorry - already closed. My bad


Secure protocol for xmlns

Secure protocol for xmlns

@hiohiohio sure, no problem. Thanks for your time.
I'm closing this PR now


Unable to modify and save cell value in umya_spreadsheet 0.9 version

Issue: Unable to modify and save cell value in umya_spreadsheet 0.9 version
Description:
The provided code is unable to modify and save cell value in umya_spreadsheet 0.9 version. The code works fine in version 0.8.7 where it can modify A1 cell value and save it in the file "aaa2.xlsx". However, when the code is run in version 0.9, it fails to modify A1 cell value, and hence cannot save it in the file "aaa2.xlsx".
extern crate umya_spreadsheet;

fn main(){

    let path = std::path::Path::new("./aaa.xlsx");
    let mut book = umya_spreadsheet::reader::xlsx::read(path).unwrap();

    let sheet = book.get_sheet_by_name_mut("Sheet1").unwrap();
    let cell = sheet.get_cell_mut("A1");
    // work on 0.87
    // cell.set_value("test".to_string());
    // work on 0.9
    cell.set_value_from_string("test");
    let path = std::path::Path::new("./aaa2.xlsx");
    let r = umya_spreadsheet::writer::xlsx::write(&book, path);
    println!("{:?}", r);
}

Expected Behavior:
The code should be able to modify the A1 cell value and save it in the file "aaa2.xlsx" when run on umya_spreadsheet 0.9 version, similar to how it works in version 0.8.7.
Steps to Reproduce:
Install umya_spreadsheet 0.9 version.
create empty excel aaa.xlsx
Run the provided code on version 0.9.
Actual Behavior:
The code runs without any error, but the A1 cell value is not modified and hence not saved in the file "aaa2.xlsx".

The fix for this is in, but as of now there's no release that includes the fix.  You can build the library locally from master until there's a new release.  That's what I'm doing, but it's not a solution that's free of its own problems by any means.
When API changes have been made, it's a particularly bad time for the library to be left in an unusable state like this.  At least in my own application, this bug causes my code to generate nothing but empty sheets.  If I hadn't changed my code to use the new API, it would have been a simple matter to change the dependency back to the previous version.  I hope you can put an end to this situation by making a new release soon.

I understand the situation.
It's unfortunate that the fix hasn't been released yet.
I have been using the previous version of the library and have found it to be very reliable.

We apologize for any inconvenience caused.
We will release 0.9.1 in the near future.


Create a lint for the rules of hooks

Specific Demand
It would be nice if the Dioxus CLI could report when the rules of hooks are not being followed. Eventually we could expand this to lint other antipaterns like using an index as an iterator key.
Implement Suggestion
Eventually adopting some of the tooling from the Clippy tool would be helpful to add lints.
For the initial implementation we can use syn to search for any functions that start with use_ that are being used within a if statement of loop. (Specifically the Visit trait could be useful here).
That will only allow us to detect issues with the current crate, but that is ok for a MVP.
We could run linting before the build and serve commands and also add a new dioxus check command that just runs the linting.

I plan to try implementing this.


Synthesis failed for java-monitoring-dashboards

Hello! Autosynth couldn't regenerate java-monitoring-dashboards. :broken_heart:
Here's the output from running synth.py:
dashboard-v1-java/src')] were copied. Does the source contain files?
2020-07-16 17:38:49,483 autosynth [ERROR] > Synthesis failed
2020-07-16 17:38:49,483 autosynth [DEBUG] > Running: git reset --hard HEAD
HEAD is now at bab0afc chore(deps): update dependency com.google.cloud:libraries-bom to v8.1.0 (#154)
2020-07-16 17:38:49,491 autosynth [DEBUG] > Running: git checkout autosynth-googleapis
Switched to branch 'autosynth-googleapis'
2020-07-16 17:38:49,496 autosynth [ERROR] > Command '['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']' returned non-zero exit status 1.
2020-07-16 17:38:49,630 autosynth [DEBUG] > Running: git checkout bab0afc5e9533da1e0ce2ad819291c78a6505c74
Note: checking out 'bab0afc5e9533da1e0ce2ad819291c78a6505c74'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at bab0afc chore(deps): update dependency com.google.cloud:libraries-bom to v8.1.0 (#154)
2020-07-16 17:38:49,636 autosynth [DEBUG] > Running: git checkout 21f1470ecd01424dc91c70f1a7c798e4e87d1eec
Previous HEAD position was 4f2c9f7 ci(java): run dependency test on Java 8 and 11 (#639)
HEAD is now at 21f1470 chore: add config files for cloud-rad for node.js (#667)
2020-07-16 17:38:49,645 autosynth [DEBUG] > Running: git checkout b56aa06cc99050861c9a9b14d9569bd18eb09604
HEAD is now at b56aa06c feat: Adds AuditLog proto for reading IAM admin logs.
2020-07-16 17:38:49,664 autosynth [DEBUG] > Running: git branch -f autosynth-58
2020-07-16 17:38:49,668 autosynth [DEBUG] > Running: git checkout autosynth-58
Switched to branch 'autosynth-58'
2020-07-16 17:38:49,673 autosynth [INFO] > Running synthtool
2020-07-16 17:38:49,673 autosynth [INFO] > ['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']
2020-07-16 17:38:49,673 autosynth [DEBUG] > log_file_path: /tmpfs/src/logs/java-monitoring-dashboards/58/sponge_log.log
2020-07-16 17:38:49,675 autosynth [DEBUG] > Running: /tmpfs/src/github/synthtool/env/bin/python3 -m synthtool --metadata synth.metadata synth.py --
2020-07-16 17:38:49,880 synthtool [DEBUG] > Executing /home/kbuilder/.cache/synthtool/java-monitoring-dashboards/synth.py.
On branch autosynth-58
nothing to commit, working tree clean
2020-07-16 17:38:50,017 synthtool [DEBUG] > Ensuring dependencies.
2020-07-16 17:38:50,029 synthtool [DEBUG] > Cloning googleapis.
2020-07-16 17:38:50,030 synthtool [DEBUG] > Using precloned repo /home/kbuilder/.cache/synthtool/googleapis
2020-07-16 17:38:50,034 synthtool [DEBUG] > Generating code for: //google/monitoring/dashboard/v1:google-cloud-monitoring-dashboard-v1-java.
2020-07-16 17:38:51,771 synthtool [SUCCESS] > Generated code into /tmpfs/tmp/tmpbbzb0g5b.
2020-07-16 17:38:51,773 synthtool [WARNING] > No replacements made in [PosixPath('/tmpfs/tmp/tmpbbzb0g5b/google-cloud-monitoring-dashboard-v1-java/proto-google-cloud-monitoring-dashboard-v1-java/src/**/*.java')] for pattern // Generated by the protocol buffer compiler.  DO NOT EDIT!, maybe replacement is no longer needed?
2020-07-16 17:38:51,776 synthtool [WARNING] > No replacements made in [PosixPath('/tmpfs/tmp/tmpbbzb0g5b/google-cloud-monitoring-dashboard-v1-java/proto-google-cloud-monitoring-dashboard-v1-java/src/**/*Name.java'), PosixPath('/tmpfs/tmp/tmpbbzb0g5b/google-cloud-monitoring-dashboard-v1-java/proto-google-cloud-monitoring-dashboard-v1-java/src/**/*Names.java')] for pattern /\*
 \* Copyright \d{4} Google LLC
 \*
 \* Licensed under the Apache License, Version 2.0 \(the "License"\); you may not use this file except
 \* in compliance with the License. You may obtain a copy of the License at
 \*
 \* http://www.apache.org/licenses/LICENSE-2.0
 \*
 \* Unless required by applicable law or agreed to in writing, software distributed under the License
 \* is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express
 \* or implied. See the License for the specific language governing permissions and limitations under
 \* the License.
 \*/
, maybe replacement is no longer needed?
2020-07-16 17:38:51,777 synthtool [WARNING] > No replacements made in [PosixPath('/tmpfs/tmp/tmpbbzb0g5b/google-cloud-monitoring-dashboard-v1-java/grpc-google-cloud-monitoring-dashboard-v1-java/src/**/*.java')] for pattern ^package (.*);, maybe replacement is no longer needed?
2020-07-16 17:38:51,778 synthtool [DEBUG] > Wrote metadata to synth.metadata.
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/synthtool/__main__.py", line 102, in <module>
    main()
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/tmpfs/src/github/synthtool/synthtool/__main__.py", line 94, in main
    spec.loader.exec_module(synth_module)  # type: ignore
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/kbuilder/.cache/synthtool/java-monitoring-dashboards/synth.py", line 35, in <module>
    bazel_target=f'//google/monitoring/dashboard/{version}:google-cloud-monitoring-dashboard-{version}-java',
  File "/tmpfs/src/github/synthtool/synthtool/languages/java.py", line 318, in bazel_library
    destination_name=destination_name,
  File "/tmpfs/src/github/synthtool/synthtool/languages/java.py", line 193, in _common_generation
    required=True,
  File "/tmpfs/src/github/synthtool/synthtool/transforms.py", line 208, in move
    f"No files in sources {sources} were copied. Does the source "
synthtool.transforms.MissingSourceError: No files in sources [PosixPath('/tmpfs/tmp/tmpbbzb0g5b/google-cloud-monitoring-dashboard-v1-java/gapic-google-cloud-monitoring-dashboard-v1-java/src')] were copied. Does the source contain files?
2020-07-16 17:38:51,824 autosynth [ERROR] > Synthesis failed
2020-07-16 17:38:51,824 autosynth [DEBUG] > Running: git reset --hard HEAD
HEAD is now at bab0afc chore(deps): update dependency com.google.cloud:libraries-bom to v8.1.0 (#154)
2020-07-16 17:38:51,830 autosynth [DEBUG] > Running: git checkout autosynth
Switched to branch 'autosynth'
2020-07-16 17:38:51,835 autosynth [DEBUG] > Running: git clean -fdx
Removing __pycache__/
Traceback (most recent call last):
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 367, in synthesize_loop
    synthesize_inner_loop(fork, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 411, in synthesize_inner_loop
    synthesizer, len(toolbox.versions) - 1
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 266, in synthesize_version_in_new_branch
    synthesizer.synthesize(synth_log_path, self.environ)
  File "/tmpfs/src/github/synthtool/autosynth/synthesizer.py", line 120, in synthesize
    synth_proc.check_returncode()  # Raise an exception.
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 389, in check_returncode
    self.stderr)
subprocess.CalledProcessError: Command '['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']' returned non-zero exit status 1.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 657, in <module>
    main()
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 506, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 637, in _inner_main
    commit_count = synthesize_loop(x, multiple_prs, change_pusher, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 375, in synthesize_loop
    synthesize_loop_single_pr(toolbox, change_pusher, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 401, in synthesize_loop_single_pr
    synthesize_inner_loop(toolbox, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 411, in synthesize_inner_loop
    synthesizer, len(toolbox.versions) - 1
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 266, in synthesize_version_in_new_branch
    synthesizer.synthesize(synth_log_path, self.environ)
  File "/tmpfs/src/github/synthtool/autosynth/synthesizer.py", line 120, in synthesize
    synth_proc.check_returncode()  # Raise an exception.
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 389, in check_returncode
    self.stderr)
subprocess.CalledProcessError: Command '['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']' returned non-zero exit status 1.

Google internal developers can see the full log here.

Autosynth is still having trouble generating java-monitoring-dashboards. :sob:
Here's the output from running synth.py:
c')] were copied. Does the source contain files?
2020-07-17 03:05:02,756 autosynth [ERROR] > Synthesis failed
2020-07-17 03:05:02,756 autosynth [DEBUG] > Running: git reset --hard HEAD
HEAD is now at bab0afc chore(deps): update dependency com.google.cloud:libraries-bom to v8.1.0 (#154)
2020-07-17 03:05:02,762 autosynth [DEBUG] > Running: git checkout autosynth-googleapis
Switched to branch 'autosynth-googleapis'
2020-07-17 03:05:02,767 autosynth [ERROR] > Command '['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']' returned non-zero exit status 1.
2020-07-17 03:05:02,908 autosynth [DEBUG] > Running: git checkout bab0afc5e9533da1e0ce2ad819291c78a6505c74
Note: checking out 'bab0afc5e9533da1e0ce2ad819291c78a6505c74'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at bab0afc chore(deps): update dependency com.google.cloud:libraries-bom to v8.1.0 (#154)
2020-07-17 03:05:02,913 autosynth [DEBUG] > Running: git checkout 21f1470ecd01424dc91c70f1a7c798e4e87d1eec
Previous HEAD position was 4f2c9f7 ci(java): run dependency test on Java 8 and 11 (#639)
HEAD is now at 21f1470 chore: add config files for cloud-rad for node.js (#667)
2020-07-17 03:05:02,923 autosynth [DEBUG] > Running: git checkout bb46f87724ddd2fd1ce9f9dcb71f147427775592
HEAD is now at bb46f877 Adds bazel build file for public version of AuditData proto for bigquery.
2020-07-17 03:05:02,941 autosynth [DEBUG] > Running: git branch -f autosynth-59
2020-07-17 03:05:02,946 autosynth [DEBUG] > Running: git checkout autosynth-59
Switched to branch 'autosynth-59'
2020-07-17 03:05:02,951 autosynth [INFO] > Running synthtool
2020-07-17 03:05:02,951 autosynth [INFO] > ['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']
2020-07-17 03:05:02,951 autosynth [DEBUG] > log_file_path: /tmpfs/src/logs/java-monitoring-dashboards/59/sponge_log.log
2020-07-17 03:05:02,953 autosynth [DEBUG] > Running: /tmpfs/src/github/synthtool/env/bin/python3 -m synthtool --metadata synth.metadata synth.py --
2020-07-17 03:05:03,160 synthtool [DEBUG] > Executing /home/kbuilder/.cache/synthtool/java-monitoring-dashboards/synth.py.
On branch autosynth-59
nothing to commit, working tree clean
2020-07-17 03:05:03,293 synthtool [DEBUG] > Ensuring dependencies.
2020-07-17 03:05:03,305 synthtool [DEBUG] > Cloning googleapis.
2020-07-17 03:05:03,306 synthtool [DEBUG] > Using precloned repo /home/kbuilder/.cache/synthtool/googleapis
2020-07-17 03:05:03,310 synthtool [DEBUG] > Generating code for: //google/monitoring/dashboard/v1:google-cloud-monitoring-dashboard-v1-java.
2020-07-17 03:05:04,994 synthtool [SUCCESS] > Generated code into /tmpfs/tmp/tmpnfojhowx.
2020-07-17 03:05:04,995 synthtool [WARNING] > No replacements made in [PosixPath('/tmpfs/tmp/tmpnfojhowx/google-cloud-monitoring-dashboard-v1-java/proto-google-cloud-monitoring-dashboard-v1-java/src/**/*.java')] for pattern // Generated by the protocol buffer compiler.  DO NOT EDIT!, maybe replacement is no longer needed?
2020-07-17 03:05:04,999 synthtool [WARNING] > No replacements made in [PosixPath('/tmpfs/tmp/tmpnfojhowx/google-cloud-monitoring-dashboard-v1-java/proto-google-cloud-monitoring-dashboard-v1-java/src/**/*Name.java'), PosixPath('/tmpfs/tmp/tmpnfojhowx/google-cloud-monitoring-dashboard-v1-java/proto-google-cloud-monitoring-dashboard-v1-java/src/**/*Names.java')] for pattern /\*
 \* Copyright \d{4} Google LLC
 \*
 \* Licensed under the Apache License, Version 2.0 \(the "License"\); you may not use this file except
 \* in compliance with the License. You may obtain a copy of the License at
 \*
 \* http://www.apache.org/licenses/LICENSE-2.0
 \*
 \* Unless required by applicable law or agreed to in writing, software distributed under the License
 \* is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express
 \* or implied. See the License for the specific language governing permissions and limitations under
 \* the License.
 \*/
, maybe replacement is no longer needed?
2020-07-17 03:05:04,999 synthtool [WARNING] > No replacements made in [PosixPath('/tmpfs/tmp/tmpnfojhowx/google-cloud-monitoring-dashboard-v1-java/grpc-google-cloud-monitoring-dashboard-v1-java/src/**/*.java')] for pattern ^package (.*);, maybe replacement is no longer needed?
2020-07-17 03:05:05,000 synthtool [DEBUG] > Wrote metadata to synth.metadata.
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/synthtool/__main__.py", line 102, in <module>
    main()
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/tmpfs/src/github/synthtool/synthtool/__main__.py", line 94, in main
    spec.loader.exec_module(synth_module)  # type: ignore
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/kbuilder/.cache/synthtool/java-monitoring-dashboards/synth.py", line 35, in <module>
    bazel_target=f'//google/monitoring/dashboard/{version}:google-cloud-monitoring-dashboard-{version}-java',
  File "/tmpfs/src/github/synthtool/synthtool/languages/java.py", line 318, in bazel_library
    destination_name=destination_name,
  File "/tmpfs/src/github/synthtool/synthtool/languages/java.py", line 193, in _common_generation
    required=True,
  File "/tmpfs/src/github/synthtool/synthtool/transforms.py", line 208, in move
    f"No files in sources {sources} were copied. Does the source "
synthtool.transforms.MissingSourceError: No files in sources [PosixPath('/tmpfs/tmp/tmpnfojhowx/google-cloud-monitoring-dashboard-v1-java/gapic-google-cloud-monitoring-dashboard-v1-java/src')] were copied. Does the source contain files?
2020-07-17 03:05:05,041 autosynth [ERROR] > Synthesis failed
2020-07-17 03:05:05,042 autosynth [DEBUG] > Running: git reset --hard HEAD
HEAD is now at bab0afc chore(deps): update dependency com.google.cloud:libraries-bom to v8.1.0 (#154)
2020-07-17 03:05:05,047 autosynth [DEBUG] > Running: git checkout autosynth
Switched to branch 'autosynth'
2020-07-17 03:05:05,053 autosynth [DEBUG] > Running: git clean -fdx
Removing __pycache__/
Traceback (most recent call last):
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 367, in synthesize_loop
    synthesize_inner_loop(fork, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 411, in synthesize_inner_loop
    synthesizer, len(toolbox.versions) - 1
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 266, in synthesize_version_in_new_branch
    synthesizer.synthesize(synth_log_path, self.environ)
  File "/tmpfs/src/github/synthtool/autosynth/synthesizer.py", line 120, in synthesize
    synth_proc.check_returncode()  # Raise an exception.
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 389, in check_returncode
    self.stderr)
subprocess.CalledProcessError: Command '['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']' returned non-zero exit status 1.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 657, in <module>
    main()
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 506, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 637, in _inner_main
    commit_count = synthesize_loop(x, multiple_prs, change_pusher, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 375, in synthesize_loop
    synthesize_loop_single_pr(toolbox, change_pusher, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 401, in synthesize_loop_single_pr
    synthesize_inner_loop(toolbox, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 411, in synthesize_inner_loop
    synthesizer, len(toolbox.versions) - 1
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 266, in synthesize_version_in_new_branch
    synthesizer.synthesize(synth_log_path, self.environ)
  File "/tmpfs/src/github/synthtool/autosynth/synthesizer.py", line 120, in synthesize
    synth_proc.check_returncode()  # Raise an exception.
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 389, in check_returncode
    self.stderr)
subprocess.CalledProcessError: Command '['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']' returned non-zero exit status 1.

Google internal developers can see the full log here.

Autosynth passed, closing! :green_heart:


VS Code Connection To SAS Server Using Remote SSH Fails

Hi
I use a locally installed VS Code editor installed on a Windows 11 machine and try to connect to a remote SAS 9.4 M7 engine on a Windows 2019 server in AWS. I have v1.10.0 of the SAS Extension for VS Code installed. I set up an SSH connection with public and private key. I can connect to the SAS server from my workstation over SSH. I have configured a specific SSH connection profile as well. When I run my simple program I get an 'All configured authentication methods failed' message (see screenshot 2). I have the impression I am signed in (see screenshot 1), but I am not really sure. I would expect a pop up to provide username and password when I run the program as well.
Could you help to troubleshoot this issue?
Thanks
Jan

are you following the steps in https://sassoftware.github.io/vscode-sas-extension/Configurations/Profiles/sas9ssh?

Yes, I followed these steps. Is there anything I can do to get more insight in what is going wrong? Thanks.

Is there any error/warning in the console? (Click Help > Toggle Developer Tools from the top menu bar.)

Yes, I enclosed them.

Sorry, the screenshot enclosed below came before the previous one in the console and the first error matches the popup in the GUI.

Would you also post a screen of the console above the 'All configured authentication methods failed' error? Thanks.

Yes, but it is a repetition of the same sequence.

Would you please try input vscode.process.env.SSH_AUTH_SOCK in the console and see if it replies the correct env variable you've set?

@smorrisj for thoughts

A few things to look at:

Does ssh-agent have the private key file that was created? You can run ssh-add -L to list out the private key files that have been registered.

Are you able to successfully connect to the target server outside of the vscode/sas extension environment via password-less ssh using that same key file? Normally when we see the error mentioned above, this step will also fail, which means that something with the password-less setup isnt working correctly.

Hi
As you can see in the screenshot below the key is present, and I can connect to the server over SSH in the terminal. I have to enter my password, though. Is there a way to revert to the state with the blue sign into SAS button  as displayed in the 2nd screenshot?
Thanks
Jan

Thanks for the updates. A few answers below:

I have to enter my password, though

Can you double-check your ssh config and that the public key that was used has been copied over to the 94m7 server? Passwordless ssh will need to work for the ssh user being used, outside of vscode and the extension, in order to get rid of the auth error in the extension itself. Those items would be item 6 and item 9 in the ssh setup guide listed above.

Is there a way to revert to the state with the blue sign into SAS button as displayed in the 2nd screenshot?

This login method is mainly for SAS Viya connections. It's not really used for making the connection in this case, which is why you are seeing the "Your connection does not support...." messaging.

I am seeing the failure via macOS and Windows 10 when connecting to Linux.
I ran the ssh server in debug mode and it looks like you are using ssh2js1.15.0 as the ssh client.
Once this client fails at user authentication, it does not try the other methods and disconnects.
debug3: userauth_finish: failure partial=0 next methods="method1,method2,etc"
It would be great if you did not require people to use keys with the SSH_AUTH_SOCK variable and inherited their environment.

I am seeing the failure via macOS and Windows 10 when connecting to Linux. I ran the ssh server in debug mode and it looks like you are using ssh2js1.15.0 as the ssh client.
Once this client fails at user authentication, it does not try the other methods and disconnects. debug3: userauth_finish: failure partial=0 next methods="method1,method2,etc"
It would be great if you did not require people to use keys with the SSH_AUTH_SOCK variable and inherited their environment.

Feature request is tracked in #1059 separately.

I do have a further question, though: you indicated that this authentication method is passwordless and there is indeed no popup requesting SAS user credentials, but how does SAS authenticate the incoming SSH connection targeting the executable? I do not see any SAS specific item in the whole Remote SHH configuration setup. Thanks for your answer.

how does SAS authenticate the incoming SSH connection targeting the executable

In the ssh2 library that we use, we're delegating authentication to ssh-agent by setting the agent property of the config to the value that was set in SSH_AUTH_SOCK:
https://github.com/sassoftware/vscode-sas-extension/blob/main/client/src/connection/ssh/index.ts#L75
The ssh-agent service will usually set this when the openssh service starts on windows.
The agent is looking at $HOME/.ssh/config entries to determine connection level details. The only thing we track in the extension's connection profile is the username itself. As @scnwwu mentioned, we do have some feature requests on the book to enhance the auth methods that we support when using the ssh connection method, through either using additional fallbacks, as well as supporting a more basic username/password authentication to ease friction.
Once authentication succeeds, we're interacting with SAS over the ssh session via interactive line mode:
https://documentation.sas.com/doc/en/vdmmlcdc/8.1/pgmcli/n16ui9f6dacn8pn1t0y2hgxgi7wa.htm

We also support connecting to a SAS 9 Remote installation using IOM if that's an option for you?
https://sassoftware.github.io/vscode-sas-extension/Configurations/Profiles/sas9iom
It uses a traditional username/password auth method and is generally easier to setup. The password gets entered once and then stored in the extension's secret storage area. It requires the ITC free download to work however, since we rely on the COM objects that it lays down to interact with the IOM apis.

Okay, thanks for clarifying that. We agree that the user name of the client (my PC on which I run VS Code) and the user name on the server (user name of my privileged account for server admin, without access to SAS) do not HAVE to match to make a successful SSH connection, as long as the public key is correctly set on the server (in the user directory of my privileged account)?

IOM and the SAS Integration Technology Client would be an awkward possible solution since we try to minimize the installation of additional components as it increase the maintenance and admin activities. Moreover, the ultimate solution for our business users would be to have the VS Code web app connecting to SAS 9.4 from our Linux Workbench server on code server. I have tried to setup the SSH connection in that environment as well but from your documentation I deduced that it is not a support use case, correct?

I'm glad that it's working for you now!
@snlwih I think that as we discuss enhancements to the remote/ssh connection type, we should also have a spike to look into how we might add fallback support for the different ssh auth modes.
The ssh library we use supports an authHandler property on the config object that we might leverage to handle some of this.
Perhaps we could use this issue to track that @jbreitman?

Sorry, for us neither the SSH connection
from VS Code on a Windows 11 PC to the SAS 9.4 M7 Windows 2019 server, nor the SSH connection
from VS Code on code-server Workbench (as a web app, see screenshot) to the same SAS 9.4 M7 Windows 2019 server
works correctly.

as long as the public key is correctly set on the server (in the user directory of my privileged account)?

The key needs to also be setup in ssh-agent by using ssh-add. In order for that to happen, you will also need the relevant entry in $HOME/.ssh/config that specifies the identity file for the host in question.

the ultimate solution for our business users would be to have the VS Code web app connecting to SAS 9.4 from our Linux Workbench server on code server

Would that be a single hosted code-server deployment that multiple users would connect to, and then use the sas extension from there? If so, then Microsoft doesn't support code-server running in that manner:
https://code.visualstudio.com/docs/remote/vscode-server#_is-the-vs-code-server-designed-for-multiple-users-to-access-the-same-remote-instance

Hi @smorrisj,
Thanks for your input.
I've tried to follow what @jandewitte has done, but I haven't gone that far yet. I went through https://learn.microsoft.com/en-us/windows-server/administration/openssh/openssh_install_firstuse?tabs=powershell
But now that I am in VS Code and have installed the extension, I am stuck at the json.settings part which is beyond my understanding. Jan will be back from holiday next Monday and I am sure he will get back to you on this and catch the train.

Could anyone confirm that we cannot use the SAS extension for VS Code to connect the SAS 9.4. exe installed on a remote Windows server? According to @ShuguangSun it is mandatory that the remote SAS 9.4 is installed on Linux server. Thanks.

Could anyone confirm that we cannot use the SAS extension for VS Code to connect the SAS 9.4. exe installed on a remote Windows server? According to @ShuguangSun it is mandatory that the remote SAS 9.4 is installed on Linux server. Thanks.

There are a few remote connection types that are supported. The 9.4 remote (ssh) connection type can only connect to SAS installed in a linux environment. This is because underneath, the integration is built on top of interactive line mode, which is only offered for linux environments. Apologies @jandewitte, I see that you were trying to target a windows server installation. I got my wires crossed with the different user issues that were reported above.
Would the remote IOM option work for you? See the capabilities matrix and capabilities by connection type here.

Thanks @smorrisj for confirming.
If remote IOM works for creating a connection from the SAS VS Code Extension installed on a Windows PC to the SAS 9.4 executable sitting on a Windows servers, than this is indeed the only available option for us. The disadvantage of this approach is that we then need to install the SAS Integration Technologies Client on the PC of every user that want to use the SAS Extension in VS Code and connect to SAS 9.4 for running programs. This obviously creates some system management overhead.
Is this the correct picture?

I installed the SAS IT client on my Windows machine and created an IOM connection profile. This setup works as expected. We can close this issue. Thanks for your help.

@jandewitte glad you were able to get the ITC method to work. I'm going to close this out. Subsequent issues can be addressed on separate github issues.


19750 Fixed error handling

Issue #: bcgov/entity#19750
Description of changes:

app version = 5.9.2
fixed error handling bug (missing await)
removed unneeded code in buildAmalgamationFiling()
updated some comments

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

/gcbrun

/gcbrun

Temporary Url for review: https://business-create-dev--pr-657-or2ets8j.web.app