[HOLD for payment 2023-05-25] [HOLD for payment 2023-05-23] senders avatar and name are missing on create new chat and send message

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

Action Performed:

click on + button
select new chat option
type email id with whom you don’t have any chat history
go to chat
send message

Expected Result:
senders avatar and name are missing
Actual Result:
should show senders avatar and name
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Version Number: v1.3.14-8
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/ff5c7366-6a71-48ed-ad40-97f3ca316b8e

https://github.com/Expensify/App/assets/93399543/a11e1225-b154-4422-8c1d-79c24124d33d
Expensify/Expensify Issue URL:
Issue reported by: @gadhiyamanan and Applause - Internal Team
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1684141166139909
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Thank you for taking care of this guys!

I don't think any additional regression tests are needed since Applause caught this

I'll had to create an Upwork job manually for payment on this: https://www.upwork.com/jobs/~01d55baf443fdbd072
Seems like @0xmiroslav was C and @rushatgabhane are C+, yeah?

@stephanieelliott reporting bonus is pending

@0xmiroslav you're been paid!
@gadhiyamanan and @rushatgabhane I have pending offers for you two, please accept the job on Upwork and I will pay out ASAP. Thanks!

@stephanieelliott accepted, thanks!

Everyone has been paid! Thank you


Monitoring of lowla based services/apps?

We are wondering if there is any plans for the inclusion of monitoring capabilities within the lowla framework to enable SLA and service availability adherence.
For example, are there any possibilities around utilising any of the following:

cron jobs/scripts to run scheduled replication/availability tests
API to pull monitoring triggers/stats, etc, from lowla based apps and serviceas
Testing of active connections to backend DBs, in this case mongodb
Testing of external availability, i.e. mobile client connections
Performance timings and usage statistics

At the moment many of these things are possible but you'd need to roll your own. All of the protocols within lowladb are based on HTTP and documented so it would be simple to address anything to do with availability with periodic polling of syncer and adapter URLs. Both the syncer and adapter need to access their respective databases so that would implicitly test database availability.
There are a number of hooks with the default syncer/adapter implementations that we provide that are in place to support the future collections of metrics for monitoring and managing servers. Our long-term intention is to create a management console that exists independently of the syncer and adapter to collect and display this type of data. The management console will likely be offered on a commercial basis.
So on your list of 5 items, numbers 1, 3 and 4 can be done today with simple URL polling. The remainder are planned for inclusion in a future offering. I'm going to close this issue as it isn't a specific change that we can track to a release, but feel free to continue the discussion!
Mark


Custom Sub-Domains

Want ability to use subdomains:
docs.johnliu.net
currently the custom domains settings only validates www.
this is strange since AzureFunctions or Static Site both supported subdomains.

This sounds like a DNS issue. Have you tried creating a CNAME from docs.johnliu.net to the name of your static web app?
Static Web Apps supports more than just the www subdomain.

Hi, you are right, I might have done it wrong last time and mistyped my cname or something, it's working as I expect today sorry!


FEATURE: PAAS-7831 add posibility to use serverspec docker driver

@martin-ducar-gd you probably want to update rpm spec right away and test the build


Context Manager namespace enforcement and namespace support

This PR enforces that inputs and outputs are 0-70 lowercase hex characters, or fails the transaction. It also makes it so that addresses under a namespace can be correctly found using 'get' on the context manager. Prior to this PR, it was possible to set an address that was partially specified in the outputs, but it wasn't possible to 'get' the address and always find the correct value for that context.
STL-121

Here's that PR. #638

@ojalatodd I fixed the subject/verb agreement problems, but did so while rebasing so I still only have 3 commits.


menu improvements

Add missing separator after About.
Use role instead of deprecated selector for menu items
Add undo and redo menu items. Both Terminal and iTerm have these.
Remove indent of the report issue template. The indent ended up in the issue body, which looks weird.
Add separator after New Tab. This is how iTerm does it.

Thanks @sindresorhus <3


Activity Indicator Not Working

I am using alamofire to call API. For Activity Controller I am installed this pod. In AppDelegate I import AlamofireNetworkActivityIndicator and in didFinishLaunchingWithOptions I wrote NetworkActivityIndicatorManager.shared.isEnabled = true. But when  API is called from Alamofire the Indicator is not shown. Please let me know what wrong I have done?

Not displaying in my app, too.
iOS 11, iPhone 7.

@tinu22 maybe your API request is too fast, try to change startDelay to 0 to test...

@bcorrea2-luizalabs, it's wired to me it defaults to 1.0, for the simplest use cases it will never be shown with a delay this long! For others, here's a fix:
NetworkActivityIndicatorManager.shared.startDelay = 0


Bug 1911470: Creating secrets do access registry through routes

This commits adds registry external routes to created docker
configuration secrets.
It hooks the controller manager into Image config changes.

/bugzilla refresh

/bugzilla refresh

/assign @dmage @adambkaplan

/assign @dmage @adambkaplan

AFAIK all OCM configuration parameters should be wired through its operator. Otherwise it'll be more difficult to investigate problems as we won't know the controller state.

AFAIK all OCM configuration parameters should be wired through its operator. Otherwise it'll be more difficult to investigate problems as we won't know the controller state.

I agree with @dmage here - the better approach is to have openshift-controller-manager-operator wire the external hostnames in via the mounted OCM configuration file.
It seems that the pull secrets controller already has a configuration API [1]. I'm not 100% clear on how RegistryURLs is used - we could either take advantage of that field or create a new field for the external URLs.
[1] https://github.com/openshift/api/blob/a9e731090f5ed361e5ab887d0ccd55c1db7fc633/openshiftcontrolplane/v1/types.go#L232

We already wire in the internal registry hostname here - https://github.com/openshift/cluster-openshift-controller-manager-operator/blob/master/pkg/operator/configobservation/images/observe_images.go#L15-L54

@adambkaplan I have checked the code and seems like RegistryURLs is not in use. I am closing this PR and opening a new one with the patch to start using this config directive. PTAL at https://github.com/openshift/cluster-openshift-controller-manager-operator/pull/197


It doesn't merge popups

I was looking to replace the identically-named Merge Windows extension with this one, but apparently that one also merges popups.
Should this one behave the same?

Sweet!


[BUG] - CloudFormation template not found

Describe the bug
I am trying to add more teams group using demo files in thos project as a guide however, the installation doesn't find my cloudformation template called (dev-juma-creator-cfn-template.yaml). I tried putting it in several places but It can't pick it up.
Latest file structure but still not working. I don't where to put the file so it can be picked by installation
❯ cat juma-creator-plugins.yaml

PluginId: code_commit
Module: code_commit
PluginId: custom_cfn
Module: custom_cfn
Parameters:
CfnTemplatePath: ./juma/dev-juma-creator-cfn-template.yaml

To Reproduce
Steps to reproduce the behavior:

create a cloudformation template for adding user group
update manifest file to include new teams group
orbit deploy teams -f manifest.yaml
See error

Expected behavior
I am expecting "orbit deploy teams -f manifest.yaml" to be able to find the cloudformation file
Screenshots

Additional context
Add any other context about the problem here.

what does your directory and file layout look like? where is the manifest.yaml file in relation to the dev-juma-creator-cfn-template.yaml file? what directory are you executing the orbit cli from?
i would recommend:

create the juma/ directory in the top-level of the orbit project (at the same level as the cli/ directory)
ensure you execute the orbit cli from the top-level of the orbit project, giving a relative path to the manifest.yaml

we generally recommend keeping all the manifests and cfn templates in the same directory to simplify locating them.

Hi Chamcca,
I am executing the orbit command inside " ~/HSDAP/orbit/aws-orbit-workbench/cli" folder like so "orbit deploy teams -f juma/latest-manifest.yaml --debug".  "juma" folder contains my latest-manifest.yaml file and dev-juma-creator-cfn-template.yaml, juma-creator-plugins.yaml, common_plugins.yaml files. however the error still persist.
previously I was executing the orbit command inside conf/ folder and this is where all my files were before copying them into cli/juma/ folder.

because we bundle and push the entire project directory to codebuild, the relative path used for the CfnTemplatePath must be relative to the top-level of the project directory. if your juma/ directory is in the cli/ directory, then you should use:
CfnTemplatePath: ./cli/juma/dev-juma-creator-cfn-template.yaml


Dark mode not eneabled on cleck profile

Maybe need to change import { UserProfile as ClerkUserProfile } from "@clerk/nextjs" to import { ClerkProvider as ClerkUserProfile } from "@clerk/nextjs" like in https://clerk.com/docs/components/customization/overview?utm_source=www.google.com&utm_medium=referral&utm_campaign=none


Looking for a complete tutorial for starting from a scratch workflow

I used matterport-mrcnn and detectron2 in the last couple of months so far and quite acquainted to the workflow (prepping datasets, configs , training an so on). However I seem to struggle with mmdetection probably I can't concentrate on to it after them. Reacthedocs and colab tutorials seems to cover a lot but I just can't merge them into something useful.
Here is what I have and need to accomplish:

I do have a coco annotated json files for train and valid sets for instance segmentation.
I want to train a mask rcnn model for instance segmentation lets say r101-fpn.

I'm much confused about customdataset class, dataset registration and configs. Initially, do I have to create a new dataset as indicated here? Or is it just for non coco datasets preprocessing stuff. Do I have to register my database by creating a py file in dataset folder or can I do it under main config file.
Actually what I need is a complete example of workflow from scratch that involves dataset creation or registration, config setting and training routine. I heard many times that indeed mmdet is an easy to learn and flexible to implement framework but looks like I'm not able to find my way inside.
Thanks.

Hi @colt18 ,
Thanks for your kind suggestion. We are reorganizing the documentation, which may be released in v2.6.0, your suggestion and question are very valuable for us.
Since there will be a while before the new release of documentation, here is the answer to your question:

If you have coco annotated JSON files, you do not need to implement a new dataset because you can directly use the CocoDataset in your config. For the dataset, you only need to modify the paths related to your annotation and data including ann_file, img_prefix, and classes.
As for the model, after modifying the dataset, you only need to modify the num_classes in box head and mask head.

Here is an example of the config for your case:
_base_ = [
    '../_base_/models/mask_rcnn_r50_fpn.py',
    '../_base_/datasets/coco_instance.py',
    '../_base_/schedules/schedule_1x.py', '../_base_/default_runtime.py'
]
# Use r101 model
model = dict(
    pretrained='torchvision://resnet101',
    backbone=dict(depth=101),
    roi_head=dict(
        bbox_head=dict(num_classes=N),
        mask_head=dict(num_classes=N)))

classes = ('a', 'b', 'c')
data = dict(
    train=dict(
        ann_file='path/to/your/annotation',
        img_prefix='root path/to/your/image'
        classes=classes),
    val=dict(
        ann_file='path/to/your/annotation',
        img_prefix='root path/to/your/image'
        classes=classes),
    test=dict(
        ann_file='path/to/your/data',
        img_prefix='root path/to/your/image'
        classes=classes))

# in case you want to use mask r101 pretrain
load_from = 'http://download.openmmlab.com/mmdetection/v2.0/mask_rcnn/mask_rcnn_r101_fpn_1x_coco/mask_rcnn_r101_fpn_1x_coco_20200204-1efe0ed5.pth'  # noqa

With this config, you should be able to train mask r101 on your own dataset without any further modification.

As the documentation have been refactored, now we have quick run sections for training and testing models on existing datasets or on customized datasets.

As the documentation have been refactored, now we have quick run sections for training and testing models on existing datasets or on customized datasets.


[TE4] Enable Lighting Devices types cluster and attributes

Problem
Color control cluster is not enable and is incomplete
Occupancy sensor cluster is not enable
Change overview
Enable occupancy Sensor cluster to Controllers, All-cluster-app and Lighting-app example
Enable Color control cluster to Controllers, All-cluster-app and Lighting-app example
Process EnhancedHue commands received
Update xml to match spec (missing arguments in command)
Include Color-control.cpp to Nordic and Telink lighting example
NOTE : there is 2 commits that are only regenerated files. You can focus the review on the two others.
TODO

ColorLoop not implemented
Apply Level and/or color to a Interface (Led/Lightbulb)

Testing
Rebuild chip-device-ctrl
Complete commissioning flow with EFR32
Test cluster commands
zcl LevelControl
zcl ColorControl
Validate value sent with logs and verify attributes values changes with
zclread LevelControl CurrentLevel
zclread ColorControl CurrentHue
zclread ColorControl CurrentX
zclread ColorControl CurrentY

@bzbarsky-apple I fixed what you pointed out in your prior review and reimplemented the Enhanced color command callbacks to match spec as discussed with Rob. It also remove duplicated code as Standard Hue and Enhanced Hue commands share the same functions with some condtions.
@andy31415 @Damian-Nordic @msandstedt @chrisdecenzo @hawk248 I would need your review as this needs to be merge for Friday for TE4 branch. Thank you
I will be Out of Office tomorrow as it is Canada day. Hopefully we can get this merged by friday

@jmartinez-silabs Sorry conflicts here :(

@mspang @andy31415 @LuDuda @msandstedt @saurabhst @Damian-Nordic @jepenven-silabs Please take a look?

@Damian-Nordic @LuDuda ?

@mspang ?

TE4 cherrypick failed with conflicts on
Unmerged paths:
  (use "git add/rm <file>..." as appropriate to mark resolution)
	deleted by us:   src/app/common/gen/callback.h

i believe this will compound as we add more files with zap-related changes in them (I see #7183 and #8138 to have potential issues). Need a path forward: maybe @jmartinez-silabs could create a patch on the TE4 branch or asking some help from @vivien-apple / @bzbarsky-apple if they are more familiar with the regen (Thinking that this may be as easy as "cherrypick and run zap regen, however I am afraid that I cannot validate the result myself)
@franck-apple FIY

We also have the option to pull #8065 in since that seems to have moved files around. That would probably resolve this merge conflict but unsure about any future issues.

I cherrypicked and done the following to resolve conflicts:
cd third_party/zap/repo
npm install
cd -
./scripts/tools/zap_regen_all.py

and added/updated all files

@andy31415 Thank you Andrei


NullPointerException  at tell @a

NullPointerException at tell @a from console

Thank you for the PR!
Can you please branch of from beta or rebase your commit on top of it.
Also please add a test to check the fix.

Closed in favor of issue #44


[24.05] Backport kernel changes from #350757

See #350757
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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

I have to run the latest kernel because of hardware driver issues together with zfs. Zfs still runs up to 6.10, which is now removed.
Please, increase the time between kernel EOL and kernel removal of nixpkgs on the stable branch of NixOS from 2 week to 8 weeks.


Add drag and dragStart to outputs array

Not sure if this was overlooked but the current version the drag and dragStart outputs do not work

thx


Show execution time below node when displaying an execution plan

Type: Bug
When I enable the Actual Exceution Plan I don't see the timings below the node as I do in SMSS. Is there a certain option I need to enable to show the timings or is this a issue with Azure Data Studio on macOS?

Azure Data Studio version: azuredatastudio 1.44.1 (8f53a316fa00a98264f1ab119641cd540b5af25c, 2023-06-01T02:13:21.084Z)
OS version: Darwin x64 22.5.0
Restricted Mode: No
Preview Features: Enabled
Modes:

I was wondering if this issue is already fixed in the latest version of Azure Data Studio?


[ESP32-S3][eFuse] OPTIONAL_UNIQUE_ID has no documentation. Why does it exist? (IDFGH-7225)

in esp_efuse_table.csv, there a reference to an optional 128-bit ID.
# SYS_DATA_PART1 BLOCK# - System configuration
#######################
    OPTIONAL_UNIQUE_ID,                   EFUSE_BLK2,    0,  128,     Optional unique 128-bit ID

But there is no documentation for this:

am I supposed to program this myself?
how is this ID used by the system? I dont see it used anywhere
does ESP-IDF intend to use this for something? If not, then I imagine it might get removed in a future ESP-IDF, so I don't want to depend on it.

Currently it is used by esp-rainmaker: https://github.com/espressif/esp-rainmaker/blob/f57a5ec62d6430a4f517de39a9121fc533f7f17d/components/esp_rainmaker/src/core/esp_rmaker_claim.c#L286
We don't have plans to remove this unique ID.

Thanks =) My google skills failed I should have googled ESP_EFUSE_OPTIONAL_UNIQUE_ID, not just OPTIONAL_UNIQUE_ID.

In what way is it "optional"? Does optional mean it could for some devices be all-zero? Or does optional just mean it's optional to use the ID in the application?
Also, what bit in EFUSE_WR_DIS does actually protect this field? Is it bit 20 per the Technical Reference Manual, or is it bit 21 per mem_definition.py in espefuse repository?
When looking at that file, why does the other BLOCK2 fields as specified in https://github.com/espressif/esp-idf/blob/master/components/efuse/esp32s3/esp_efuse_table.csv#L154 not match with https://github.com/espressif/esptool/blob/master/espefuse/efuse/esp32s3/mem_definition.py#L232, regarding the bit positions?
Regarding the link to esp-rainmaker, it says in the comments in the code it's for ESP32-S2. But its TRM does not say this field exists...


first version of cutscene engine

This is the first version that I managed to commit. Is not exactly what I wanted since most of the system is all show but little substance in the code. I may redo the whole system, but keep certain visual elements like the scrolling text and next icon animation. Thoughts?

Any status update on this?

Any status update on this?

Hey, sorry for the late reply. @Telvy is working on a followup to this at the moment. I'm not sure how much of what he is working on now depends on this code but he is new to git so I think the second PR will include everything done in this. We should probably close this.
@Telvy did you use the code in this PR in the new stuff you're working on now? Have you made significant changes to the code in this PR since you submitted it?

Heyya. Sorry for the late reply. For the past week or so I've been creating a completely new system that has more classes implemented to have the whole system working.
The whole system reads a .json file with information on the respective characters, expressions and lines. Most the code in my first pull request is practically dead at this point. I'll scrap the code for the visual text scrolling affects and that's it.
Also first time comment replying on github. Never did this before but I hope I'm doing this right.

@Telvy all good, sounds exciting! Let's close this PR then in favour of the new one.


iOS blue indicator on status bar reported by users.

Some users are reporting that they are seeing the blue status bar when not using the app. Not all users and I cant reproduce on 9-10 phones but I have found someone with the issue and dont know why
Your Environment

Plugin version: 0.5.0-alpha.34
iOS
OS version: 11.4.1
Device manufacturer and model: iPHONE X
React Native version: 55.2

BackgroundGeolocation.configure({
      desiredAccuracy: BackgroundGeolocation.HIGH_ACCURACY,
      stationaryRadius: 50,
      distanceFilter: 50,
      //notificationTitle: 'Background tracking',
      //notificationText: 'enabled',
      debug: false,
      startOnBoot: false,
      stopOnTerminate: false,
      locationProvider: BackgroundGeolocation.ACTIVITY_PROVIDER,
      interval: 10000,
      fastestInterval: 5000,
      activitiesInterval: 10000,
      stopOnStillActivity: true,
    });

Context
Most users are not experiencing this, but some are. have tried uninstall and reinstall, restart phone, toggle all preferences in location settings for the app. Some users see the blue status bar when the app is in background but most do not see the bar
Expected Behavior
When app is in background blue bar should not be there.
Actual Behavior
When app is in background blue bar appears for subset of users
Steps to Reproduce

Download Hooked Deals App
Sign Up
Enable Geo
Put App in Background
Either see the blue bar or not.

Context
I dont ever want to see the blur bae
Debug logs
N/A

I could be wrong here but i think the blue bar shows up if you select 'only while using app' for the location services if you say always it doesn't show. This has been my experience so far.

@Cumquat @mauron85 So how do I stop locations tracking when user selects "When app in use". I used every option but was not able to fix that.

@Cumquat @mauron85  Cannot update the location when I select the 'Always' for the location services. Also, the Blue bar does not appear on the screen which means location is getting updated.
I have configured with the below values.
 desiredAccuracy: 10, //Desired accuracy in meters. Possible values [HIGH_ACCURACY, MEDIUM_ACCURACY, LOW_ACCURACY, PASSIVE_ACCURACY]. Accuracy has direct effect on power drain. Lower accuracy = lower power drain.     stationaryRadius: 5, //Stationary radius in meters. When stopped, the minimum distance the device must move beyond the stationary location for aggressive background-tracking to engage.     distanceFilter: 5, // 50, meter , The minimum distance (measured in meters) a device must move horizontally before an update event is generated.      notificationTitle: 'Background tracking',     notificationText: 'enabled',     notificationsEnabled: true,     debug: false,     startOnBoot: false,     stopOnTerminate: true,     locationProvider: isAndroid ? BackgroundGeolocation.ACTIVITY_PROVIDER : BackgroundGeolocation.DISTANCE_FILTER_PROVIDER,     // locationProvider: BackgroundGeolocation.provider.ANDROID_ACTIVITY_PROVIDER,     interval: 10000, // 5000, ms, android only      fastestInterval: 1000, //android only, Fastest rate in milliseconds at which your app can handle location updates.      activitiesInterval: 1000, //android only,Rate in milliseconds at which activity recognition occurs. Larger values will result in fewer activity detections while improving battery life.     stopOnStillActivity: false,     enableHighAccuracy: true,     pauseLocationUpdates: false,     // saveBatteryOnBackground: true,//NOTE: Enable only if we want to RCTBackgroundGeoLocation paused posting location. //Only iOS, Switch to less accurate significant changes and region monitory when in background     // maxLocations: 1000,     // url: 'http://<IP_ADDRESS>:3000/location',     // customize post properties     postTemplate: ['@time', '@latitude', '@longitude', '@accuracy', '@bearing', '@speed']

How did you fix the blue bar issue when background location service should be stopped?

This issue has been automatically closed, because it has not had recent activity. If you believe this issue shouldn't be closed, please reopen or write down a comment requesting issue reopening with explanation why you think it's important. Thank you for your contributions.

this is crap, this issue is still happening, users of an app using this library will be like "WTF" 'cause the blue bar always appear


openplayground error

I couldn`t use the open playground for many hours. Anyone has the same issue?

+1

It has something to do with credits and payment. Despite having a lot of $ in my account, openplayground seems to think that the account balance is too low to process the question.

+2.  I have tried funding up On top of a sufficient balance I already had, using different browsers, logging out and logging in again, clearing my history In the open playground, And yet I still continue to receive the same error message.


Configuring a SSL-Truststore is no longer working

Description
We use a database which uses a custom SSL-Certificate. In the past we were able to connect to it by adding this SSL-Certificate to a truststore and start DBeaver with the following command:
"C:\Program Files\DBeaver\dbeaver.exe" -nl en -vmargs -Djavax.net.ssl.trustStore=D:\dbeaver-truststore.jks
This no longer works after upgrading from version <IP_ADDRESS>207240747 to <IP_ADDRESS>210231354. On connect, DBeaver displays this error: The application requester cannot establish the connection. (PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target)
Configuring the truststore via dbeaver.ini does not work either.
After downgrading to 22.1.3, connecting works again.
dbeaver-logs.zip
DBeaver Version
Community Edition <IP_ADDRESS>210231354
Operating System
Windows 10 Enterprise 20H2
Database and driver
DB2 iSeries/AS 400 in unknown version with driver net.sf.jt400:jt400-11.0
Steps to reproduce

Configure Database with custom SSL-Certificate
Add custom SSL-Certificate to a truststore
Start DBeaver with "C:\Program Files\DBeaver\dbeaver.exe" -nl en -vmargs -Djavax.net.ssl.trustStore=D:\dbeaver-truststore.jks
Connect to the database. After supplying the password, DBeaver displays this error: The application requester cannot establish the connection. (PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target)

Additional context
No response

Could you try switching the preference Use Windows truststore on/off as a possible workaround?

Also logs would be very useful. You can find instructions how to send log files here https://github.com/dbeaver/dbeaver/wiki/Log-files

Hello @E1izabeth ,
thank you for your fast response. Indeed, the Setting Use Windows trust store was enabled. After toggling it to off and restarting, the connection works again. It seems as this option was added in 22.2.2 and enabled by default, which is incompatible with the JKS-Truststore we use.
Logs are linked in the original text: https://github.com/dbeaver/dbeaver/files/9918016/dbeaver-logs.zip
From my side, this issue is solved, however I am leaving this open in case you want to update your documentation or tweak when this setting enables itself.

DBeaver used windows trust store by default (it was written in dbeaver.ini file). In the last release we removed this setting from dbeaver.ini and made a preference for that, because meet some certificate issue after one of the Windows update. We used windows truststore earlier and enable this option by default now. So nothing was supposed to be changed from DBeaver side.
We will investigate it. Thank you.

Seems we need to check if javax.net.ssl.trustStore or javax.net.ssl.trustStoreType options were set in the dbeaver.ini file and do not set Use Windows trust store in this case.

Windows Trust Stores don't seem to work anymore.
When simply using the Windows Trust Store, dbeaver cannot validate a certificate issued from an intermediate Issuing CA coupled with the trusted Root CAfrom the Windows Trust Store.
Scenario:
Root CA Certificate installed/trusted in Computer's & User's "Trusted Root Certificate Authorities" windows store.
Intermediate Issuing CA certificate installed/trusted in the Computer's and User's "Intermediate Certificate Authorities" windows store.
Certificate issued from the Intermediate Issuing CA installed on the database we are connecting to from DBeaver. Windows shows the certificate can be validated, yet DBeaver throws the error mentioned above.
There is something still off when using the new setting under Preferences --> Connections --> Use windows trust store checkbox. When I setup the cacerts file for the java trust store and uncheck the "User Windows trust store" things work as expected. But when setting up the Windows Trust stores properly the connections cannot be established.
We have tested using an internal Certificate Authority and Intermediate Issuing Authority running Microsoft Certificate Services.
In powershell (and certificate control panel snapin) the Certificate locations are:
Root CA ---                  Cert:\LocalMachine\Root   (Trusted Root Certification Authorities)
Intermediate CA ----   Cert:\LocalMachine:\CA    (Intermediate Certificate Authorities)
Root CA ----                Cert:\CurrentUser\Root   (Trusted Root Certification Authorities)
Intermediate CA ----- Cert:\CurrentUser\CA       (Intermediate Certificate Authorities)
We also tried adding all certs (root and intermediate into the Root (Trusted Root Certification Authorities)  and that didn't work either.
What am I missing or is this an outstanding bug due to the changes identified above that haven't been accepted yet?

@ADegele please, try early access version https://dbeaver.io/files/ea/. Has the problem been solved?

@E1izabeth I had the checkbox "Use Windows trust store" unchecked before the upgrade to 22.3.0-preview. After the upgrade, the connection to the DB still works, and I cannot check the checkbox again:

From my side everything is fine and I hope this is the behaviour you expect. Thank you!

Thank you for checking. Yes, that's expected behaviour.

@E1izabeth should this checkbox also recognize -Djavax.net.ssl.TrustStoreType has already been set too?
https://stackoverflow.com/questions/44745261/why-do-jvm-arguments-start-with-d
Over my head, but ran into this problem today

It wouldn't override settings, which was set not by application itself.

Windows Trust Stores don't seem to work as expected.
When simply using the Windows Trust Store, dbeaver cannot validate a certificate issued from an intermediate Issuing CA coupled with the trusted Root CAfrom the Windows Trust Store.
Scenario: Root CA Certificate installed/trusted in Computer's & User's "Trusted Root Certification Authorities" windows store. Intermediate Issuing CA certificate installed/trusted in the Computer's and User's "Intermediate Certification Authorities" windows store. Certificate issued from the Intermediate Issuing CA installed on the database we are connecting to from DBeaver. Windows shows the certificate can be validated, yet DBeaver throws the error mentioned above.
There is something still off when using the new setting under Preferences --> Connections --> Use windows trust store checkbox. When I setup the cacerts file for the java trust store and uncheck the "User Windows trust store" things work as expected. But when setting up the Windows Trust stores properly the connections cannot be established.
We have tested using an internal Certificate Authority and Intermediate Issuing Authority running Microsoft Certificate Services. In powershell (and certificate control panel snapin) the Certificate locations are: Root CA --- Cert:\LocalMachine\Root (Trusted Root Certification Authorities) Intermediate CA ---- Cert:\LocalMachine:\CA (Intermediate Certificate Authorities) Root CA ---- Cert:\CurrentUser\Root (Trusted Root Certification Authorities) Intermediate CA ----- Cert:\CurrentUser\CA (Intermediate Certificate Authorities)
I did validate that this works when all certificates (Root and Intermediates) are in one container such as the "Trusted Root Certification Authorities" however this isn't really how Windows is designed to work. It takes advantage of the different certificate containers. Example of this is where Root certificates are trusted in the "Trusted Root Certificate Authorities" container and intermediaries are stored in the "Intermediate Certification Authorities" container. When the entire chain isn't in a single container DBeaver doesn't validate the certificate properly. To use the dbeaver.ini file you specify -Djavax.net.ssl.trustStoreType=Windows-ROOT, there needs to be a way to specify the Windows-ROOT plus the Windows-CA containers or just have dbeaver use the Certificate store as windows is designed to by taking into consideration all of the trust chains from the different containers, Trusted Certification Authorities, Intermediate Certification Authorities, Third-Party Root Certification Authorities, etc. along with the ability to check for "Untrusted Certificates" specified in that store.

I found myself with exactly this problem. I am in a corporate computer in Windows, I cannot edit my DBeaver installation and therefore I cannot change the cacerts to import a certificate, nor the dbeaver.ini file to use an external JVM where I have write access to the cacerts file. I can install certificates in my user trust stores, though. However, as this is an intermediate authority (not a root authority), the authentication still fails.
I see this comment sort of went unnoticed. @E1izabeth (tagging you as you were the person handling this fix), was this comment ever taken into consideration?
Thanks!

@sinhote please create a new issue. This one is not about that and is closed


Rename eAtlas

Proposal:

[ ] eAtlas -> tgvejs
[ ] eatlas-template -> tgve-app
[ ] full-template -> tgve-app-full?

Once we commit fully to TGVE, I propose tgvejs.

@layik See top of issue for current proposal.

Thank you @rolyp. One more suggestion to make it even more difficult:

eAtlas -> tgvejs
eatlas-template -> app
full-template -> app-full?

As we have /tgve from the org name anyways.

@layik I like your suggestion better -- updated the proposal at the top.

In relation to first checkbox, this is where I have been hung/stuck:
https://stackoverflow.com/a/28841779/11101153

RE third item (rename full-template -> full-app), there is the feeling that the repo name still does not reflect it fully. So had to use tgve-full-app in the repo's README.

In short, the logic (which I proposed) of relying on GH organisational naming tgve/app does not automatically reflect onto README/docs/papers/presentations.

In relation to first checkbox (renaming eAtlas -> tgvejs), this is where I have been hung/stuck:
https://stackoverflow.com/a/28841779/11101153

Perhaps we can just deprecate the old component and release a new one with the new name.

@layik Yes, let's revert to tgve-app and tgve-full-app then after all.

Thanks @rolyp. Please let me know your preference of the two.

@layik I feel like we're good to delete
            sed -i "s/layik/${a[0]}/" package.json
from gh-pages.yml (in app and full-app), since there are no longer any references to layik in the baseline package.json.

@layik Unless you tell me otherwise planning to rename eatlas-data to example-data or similar.

@layik I feel like we're good to delete

            sed -i "s/layik/${a[0]}/" package.json

from gh-pages.yml (in app and full-app), since there are no longer any references to layik in the baseline package.json.
(Btw: how are ${a[0]} and ${a[1]} resolved?)

Sorry @rolyp for late response to this. I think you edited the file(s) but this is important for people who "fork the template". So app and full-app would need to be replaced with their repository names.


[CMO]: After the list of kits is successfully completed, the summary has a blank page

not reproducible anymore


Clear out Open issues

@katiewhite360 - as you're able can you please look through all open issues on the newsletter repo and close out anything that doesn't need further discussion? Note you can always reopen these if you need to. I closed out several but if you can help with the remaining that would be great. Also do you have a cue of topics/write ups for future newsletters is being stored? Can you share this or create an issue?

Thanks @willingc for clearing out a few of these also!

@katiewhite360 @Ruv7 I'm closing since there appears to be no further action now.


Zephyr 4.0 module update overview

Umbrella issue for modules which may require an update before Zephyr 4.0.

[ ] https://github.com/zephyrproject-rtos/acpica (@najumon1980)
[x] https://github.com/zephyrproject-rtos/babblesim-manifest (@aescolar) (https://github.com/zephyrproject-rtos/zephyr/pull/80510)
[x] https://github.com/zephyrproject-rtos/canopennode (@henrikbrixandersen)
[ ] https://github.com/zephyrproject-rtos/chre (@yperess)
[ ] https://github.com/zephyrproject-rtos/cmsis (@stephanosio, @microbuilder, @povergoing)
[ ] https://github.com/zephyrproject-rtos/cmsis-dsp (@XenuIsWatching, @stephanosio)
[x] https://github.com/zephyrproject-rtos/cmsis-nn (@XenuIsWatching, @stephanosio)
[x] https://github.com/zephyrproject-rtos/edtt (@aescolar, @wopu-ot, @thoh-ot)
[ ] https://github.com/zephyrproject-rtos/fatfs (@de-nordic)
[ ] https://github.com/zephyrproject-rtos/hostap (@krish2718, @jukkar)
[ ] https://github.com/zephyrproject-rtos/liblc3 (@Casper-Bonde-Bose, @MariuszSkamra)
[ ] https://github.com/zephyrproject-rtos/libmetal (@carlocaione, @arnopo)
[ ] https://github.com/zephyrproject-rtos/loramac-node (@Mani-Sadhasivam)
[x] https://github.com/zephyrproject-rtos/lvgl (@faxe1008, @brgl, @pdgendt)
[ ] https://github.com/zephyrproject-rtos/lz4 (@parthitce )
[ ] https://github.com/zephyrproject-rtos/mcuboot (@d3zd3z, @de-nordic, @nordicjm)
[ ] https://github.com/zephyrproject-rtos/nanopb (@pdgendt)
[ ] https://github.com/zephyrproject-rtos/open-amp (@carlocaione)
[ ] https://github.com/zephyrproject-rtos/openthread (@rlubos)
[ ] https://github.com/zephyrproject-rtos/percepio (@eriktamlin, @aronlander-pe)
[ ] https://github.com/zephyrproject-rtos/picolibc (@keith-packard, @stephanosio)
[ ] https://github.com/zephyrproject-rtos/sof (@kv2019i)
[ ] https://github.com/zephyrproject-rtos/tflite-micro (@laurenmurphyx64)
[ ] https://github.com/zephyrproject-rtos/thrift (@cfriedt)
[ ] https://github.com/zephyrproject-rtos/zcbor (@de-nordic)
[ ] https://github.com/zephyrproject-rtos/zephyr-lang-rust (@d3zd3z)
[x] https://github.com/zephyrproject-rtos/zscilib (@microbuilder)

Folks that are tagged please comment and confirm the status of your modules with respect to readiness for the 4.0 release.

Zephyr 4.0 Release Checklist #76338
Zephyr 4.0 security related modules update overview #80489

@dkalowsk EDTT is already the latest version. For bsim, I just qeued a PR with a minor update with some minor improvements. It is not required for 4.0 but does not harm either.

@dkalowsk hostap update is planned after 4.0 is released (tracked in Wi-Fi project page https://github.com/orgs/zephyrproject-rtos/projects/50)

There is a PR for zscilib, but waiting on the PR author to comment, and can be updated post 4.0 release.

In picolibc, there are some fixes for clang, arc and xtensa upstream which should probably get pulled in for 4.0. We'll need to sync with the SDK and see when that might get updated.

In picolibc, there are some fixes for clang, arc and xtensa upstream which should probably get pulled in for 4.0. We'll need to sync with the SDK and see when that might get updated.

@keith-packard Unless those are critical bug fixes, we will likely have to target 4.1 since releasing and integrating another version of SDK will likely take more than a week and, at that point, we will be at rc2 or even rc3 where we cannot risk toolchain-related regressions.

CMSIS (v5) is already at the latest version (5.9.0). As for CMSIS v6, it will likely have to be added as a separate module: https://github.com/zephyrproject-rtos/zephyr/issues/77220.

@keith-packard Unless those are critical bug fixes, we will likely have to target 4.1 since releasing and integrating another version of SDK will likely take more than a week and, at that point, we will be at rc2 or even rc3 where we cannot risk toolchain-related regressions.

Sounds fine -- none of them are critical.

MCUboot will be updated with a bug fix which is currently pending on the submitted fixing their commit, a sync will then be opened in zephyr

Created #80555 to sync nanopb with upstream.

No need to update OpenThread.

Regarding following modules:

https://github.com/zephyrproject-rtos/fatfs - will not be updated prior to 4.0; there will be update after, bringing fixes in features that are not yet enabled in Zephyr
https://github.com/zephyrproject-rtos/littlefs - missing from list, at this point this is at version latest offered by upstream (https://github.com/littlefs-project/littlefs/commits/master/)
https://github.com/zephyrproject-rtos/zcbor  - @oyvindronningstad, can you comment here, the project is under your control.

No need to update percepio module, as far as we can tell.
Best regards
Erik Tamlin
On Tue, 29 Oct 2024 at 12:41, Dominik Ermel @.***>
wrote:

Regarding following modules:

https://github.com/zephyrproject-rtos/fatfs - will not be updated
prior to 4.0; there will be update after, bringing fixes in features that
are not yet enabled in Zephyr
https://github.com/zephyrproject-rtos/littlefs - missing from list,
at this point this is at version latest offered by upstream (
https://github.com/littlefs-project/littlefs/commits/master/)
https://github.com/zephyrproject-rtos/zcbor - @oyvindronningstad
https://github.com/oyvindronningstad, can you comment here, the
project is under your control.

—
Reply to this email directly, view it on GitHub
https://github.com/zephyrproject-rtos/zephyr/issues/80491#issuecomment-2443976944,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APNUTSJMNLYKR7OSBL2BIYDZ55X5BAVCNFSM6AAAAABQWSR5W6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBTHE3TMOJUGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Sounds like we'll leave picolibc alone for 4.0 as the SDK is already set.

MCUboot: https://github.com/zephyrproject-rtos/zephyr/pull/80607

No updates for Thrift yet. I'm following up about why this was reverted and how to get it back in.
https://github.com/apache/thrift/pull/2871
https://github.com/apache/thrift/pull/2899

MCUboot has been updated to bring in a fix for a user reported issue

No plans to update liblc3

@JordanYates Since you last updated the module, do you think the Tensorflow Lite Micro module needs an update?

LoRaMAC-node is in maintenance mode and has no updated

@JordanYates Since you last updated the module, do you think the Tensorflow Lite Micro module needs an update?

Only if required by #80495, which was the reason I had to update it last time.


Where examples for all style classes?

I see other css style features, toast, tooltip, popover, carousel, but i not see one examples!

Hello, this template is based on Bootstrap 4. If you want to see some examples of popover, carousel, etc., check the Bootstrap documentation. Best luck!

Hello, this template is based on Bootstrap 4. If you want to see some examples of popover, carousel, etc., check the Bootstrap documentation. Best luck!

I'm waiting for examples on the layout style sb2, not bootstrap examples.

Good luck waiting then :)

Anything that has been modified past the Bootstrap framework has been documented in the theme for the most part, but everything in the Bootstrap docs will get slightly restyled due to some global variable changes. For sake of brevity, I don't plan on re-stating everything in the Bootstrap docs within the theme, only things that are custom built exclusive to the theme or things that have been significantly changed. Hopefully this helps explain things!


Real time classification

What is the timeline for the real time classification capability?

Since I have other things to do, I am sorry that I do not plan to dive into this capability anytime soon.
If you are interested in it, I hope you will have a great time developing it.


chore: upgrade tekton to 0.33.4

Signed-off-by: ankitm123<EMAIL_ADDRESS>

/retest

Important: after this is merged, we only support kubernetes 1.21+.

/retest

/retest


smart routing config setting

[placement]
mode = "smart"

https://developers.cloudflare.com/workers/configuration/smart-placement/
This mades a huge difference when you need to connect to your own DB like I do.
I use PB / SQLITE

MySQL Blog server example should contain that config.
https://github.com/syumai/workers/tree/main/_examples/mysql-blog-server


character editior is not working

Describe your bug here.
hey I noticed character editor is not working

4b42-8c48-df2a696a99d5
Command Prompt/Terminal logs ()
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
Base mod, nothing was changed.

Fixed in 1.1


docs: Add requirement for ginkgo version

Recently, ginkgo v2 is released, and has some breaking changes, such as
the below. Hence, we should update development docs for the same. By default,
go install github.com/onsi/gingko/ginkgo@latest will only install the
latest v1 version, at the time of writing, this will be v1.16.5.
Thanks to @thejosephstevens for highlighting this issue in development
channel.
$ ginkgo -h
Ginkgo Version 2.1.3

$ ginkgo --focus="Runtime" --tags=integration_tests
level=info msg="environment variable \"K8S_VERSION\" was not set; setting to default value \"1.23\""
flag provided but not defined: -ginkgo.timeout
Usage of /home/tammach/go/src/github.com/cilium/cilium/test/test.test
...

Signed-off-by: Tam Mach<EMAIL_ADDRESS>

Did you check if it would be hard to support the new version? Ideally, if we can do that, it's probably better than updating the docs.

I didn't, but AFAIR there was a convo moving away from ginkgo in favour of native go test :thinking:

Did you check if it would be hard to support the new version? Ideally, if we can do that, it's probably better than updating the docs.

I didn't, but AFAIR there was a convo moving away from ginkgo in favour of native go test thinking

Like any big test refactoring, it's a very slow convo, so best to update our ginkgo version IMO.

Marking this as Draft to explore upgrade path as suggested by Paul.


UnexpectedImageLayout { allowed: Undefined, requested: PresentSrc }

Version of vulkano: 0.33.0
OS: Win 10 x64
GPU (the selected PhysicalDevice): NVIDIA GeForce RTX 3060
GPU Driver: 531.61
Upload of a reasonably minimal complete main.rs file that demonstrates the issue: N/A

Issue
I've been having issues with incorrect image layout transitions (Undefined -> X) since the start of my Vulkano journey about a year ago. After an update to Vulkano version 0.33.0, the software patch I've been using was rendered useless, and as such, I had to come up with a new patch to enable my continued usage of Vulkano. While doing so, I happen to stumble accross this document:
https://github.com/vulkano-rs/vulkano/blob/master/DESIGN.md#image-layouts
In the last paragraph of the linked section, it targets the very issue I've been facing the whole time. However, it seems the document is out-of-date and not applicable anymore.
Is this is common issue with Vulkano? Is there a documented way to resolve it?
All help is welcome!

This looks very much like a bug in Vulkano, but I can't say offhand where the problem is, it will need some investigating.

Not sure if 0.34 did it, but this seems to have been solved. Version 0.34 did solve a couple of other barrier issues aswell 👍 Closing.


aix-explainer: update versions to fix CVEs

What this PR does / why we need it:
Fixing critical CVEs in latest kserve/alx-explainer version
Comparing this to latest stable and v0.9.0 kserve/aix-explainer images:

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Not sure there's an open issue for this
Type of changes

[x] Bug fix (non-breaking change which fixes an issue)

Feature/Issue validation/testing:
kserve/alx-explainer image builds ok with no issues

Logs

Special notes for your reviewer:

This PR only changes image versions

Release note:
fixing vulnerabilities (CVEs) in kserve/aix-explainer

/lgtm

/approve


Installation via npm gives 404 error

Problem
npm install -g tocdoc
npm ERR! code E404
npm ERR! 404 Not Found: tocdoc@latest

npm ERR!      C:\Users\Clare\AppData\Roaming\npm-cache\_logs\2019-04-11T07_14_08_424Z-debug.log

Things Tried
I tried the steps listed in https://stackoverflow.com/questions/47520181/installing-npm-package-fails-with-404

npm config set registry http://registry.npmjs.org
Removing C:\Users\Clare\.npmrc

But it turns out that both these URLs give {"error":"Not found"} for me:

http://registry.npmjs.org/tocdoc
https://registry.npmjs.org/tocdoc

And searching https://www.npmjs.com/search?q=tocdoc only shows grunt-tocdoc

I’ve tried those URLs via two different networks and both give Not Found.

Sorry - I see I had typed the project name wrong!
-tocdoc
+doctoc


gyro allows adding the same repo twice

i accidentally added my repo twice:
$ gyro add travisstaloch/art.zig -g

This resulted in this error:
$ gyro build run
./deps.zig:8:9: error: redefinition of 'art'
    pub const art = std.build.Pkg{

Not sure what should happen here.  Maybe just show an error saying the repo already exists.

I tried to check this today with a fresh gyro and zig but it seems to give the same error. Still allows the repo to be added twice and shows the same error message.
My terminal commands to reproduce are below.  Let me know if I got anything wrong.   I checked at the bottom to make sure I have a current gyro which was just pulled and built.
home:/tmp/zig-test
$ zig init-exe
info: Created build.zig
info: Created src/main.zig
info: Next, try `zig build --help` or `zig build run`
home:/tmp/zig-test
$ gyro add travisstaloch/art.zig -g
debug: default_branch: master
home:/tmp/zig-test
$ gyro add travisstaloch/art.zig -g
debug: default_branch: master
home:/tmp/zig-test
$ ls
build.zig  gyro.zzz  src
home:/tmp/zig-test
$ ls gyro.zzz
gyro.zzz
home:/tmp/zig-test
$ cat gyro.zzz
deps:
  art:
    src:
      github:
        user: travisstaloch
        repo: art.zig
        ref: master
  art:
    src:
      github:
        user: travisstaloch
        repo: art.zig
        ref: master
home:/tmp/zig-test
$ cat build.zig
const Builder = @import("std").build.Builder;
const pkgs = @import("gyro").pkgs;

pub fn build(b: *Builder) void {
    // Standard target options allows the person running `zig build` to choose
    // what target to build for. Here we do not override the defaults, which
    // means any target is allowed, and the default is native. Other options
    // for restricting supported target set are available.
    const target = b.standardTargetOptions(.{});

    // Standard release options allow the person running `zig build` to select
    // between Debug, ReleaseSafe, ReleaseFast, and ReleaseSmall.
    const mode = b.standardReleaseOptions();

    const exe = b.addExecutable("zig-test", "src/main.zig");
    pkgs.addAllTo(exe);
    exe.setTarget(target);
    exe.setBuildMode(mode);
    exe.install();

    const run_cmd = exe.run();
    run_cmd.step.dependOn(b.getInstallStep());
    if (b.args) |args| {
        run_cmd.addArgs(args);
    }

    const run_step = b.step("run", "Run the app");
    run_step.dependOn(&run_cmd.step);
}
home:/tmp/zig-test
$ gyro build run
./deps.zig:8:9: error: redefinition of 'art'
    pub const art = std.build.Pkg{
        ^
./deps.zig:3:9: note: previous definition is here
    pub const art = std.build.Pkg{
        ^
./build.zig:2:29: note: referenced here
const pkgs = @import("gyro").pkgs;

home:/tmp/zig-test
$ ls -l $(which gyro)
-rwxr-xr-x 1 travis travis 5711384 Mar  4 14:03 /home/travis/Documents/Code/zig/gyro/zig-cache/bin/gyro

actually closed now after #59


extend MvuContentPanel

Allows lazy loading of the content now:
export class MyPanel extends AbstractMvuContentPanel {

	createView(model) {
		const { active } = model;

		return active ?  ...

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

33 of 33 (100.0%)  changed or added relevant lines in 3 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 99.464%

Files with Coverage Reduction
New Missed Lines
%

src/modules/menu/components/mainMenu/MainMenu.js
1
98.04%

src/modules/olMap/handler/draw/OlDrawHandler.js
1
98.23%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
11063

Relevant Lines:
11069

💛  - Coveralls


Feedback error while running langchain

Trying to run langchain with open ai api, it works fine with short paragraphs but when I tried longer ones I got this error:
openai.error.InvalidRequestError: This model's maximum context length is 4097 tokens, however you requested 13214 tokens (12958 in your prompt; 256 for the completion). Please reduce your prompt; or completion length.
I don't know if I get the setting right or not, here is my code:
import os
from langchain.document_loaders import TextLoader
from langchain.text_splitter import CharacterTextSplitter
from langchain.embeddings import OpenAIEmbeddings
from langchain.vectorstores import FAISS
from langchain.chains.question_answering import load_qa_chain
from langchain.chat_models import ChatOpenAI
os.environ["OPENAI_API_KEY"] = "sk-xxxxxxxxxx"
def main():
global db, chain, entry, output  # Add entry and output to the global variables
file_path = r"F:\langchain\doc1.txt"
loader = TextLoader(file_path, encoding='utf-8')
documents = loader.load()

text_splitter = CharacterTextSplitter(chunk_size=1000, chunk_overlap=0)
docs = text_splitter.split_documents(documents)

embeddings = OpenAIEmbeddings()

db = FAISS.from_documents(docs, embeddings)

llm = ChatOpenAI(openai_api_key="sk-xxxxxxxxxx", model_name="gpt-3.5-turbo",
             max_token=200)

chain = load_qa_chain(llm, chain_type="stuff")

Standard error message when the text provided is too long and based on your max_token setting you've set the response length to be quiet small as well
Couple of options for you to try - see what works best for your use case but note you won't be able to override the model limit.

Shorten text - as you've tested it works with smaller paragraphs
Increase the 'chunk_overlap' value
Increase max_token parameter - you'll only get a short response with your current configuration

Use the tiktoken library to count tokens if you need to do some sanity checking before making the API call


cmd/compile: emitptrargsmap looks buggy

Perhaps I'm missing something, but this sequence of calls in emitptrargsmap looks suspect:
	if fn.IsMethod() {
		onebitwalktype1(fn.Type.Recvs(), 0, bv)
	}
	if fn.Type.NumParams() > 0 {
		onebitwalktype1(fn.Type.Params(), 0, bv)
	}

If I'm reading it correctly, this will overwrite the Recvs bitmap with the Params bitmap.
Also, after emitting that bitmap, we do:
	if fn.Type.NumResults() > 0 {
		onebitwalktype1(fn.Type.Results(), 0, bv)
		off = dbvec(lsym, off, bv)
	}

But we don't clear bv first, so any set bits from recvs/params will still be set.
emitptrargsmap is only called from assembly functions, which are unlikely to have methods and which are often nosplit, which is possibly why this hasn't caused visible problems yet (?).
cc @mdempsky -- does this code look right to you, and if so, what am I missing?

I think that code is fine.
We create that bitmap to implement GO_ARGS and GO_RESULTS_INITIALIZED from runtime/funcdata.h. The first bitmap (recvs+params) is to represent at function entry (when only params are live), and the second bitmap (recvs+params+results) is to represent once the results have been initialized within the assembly code.
It's okay to call them all like that because function parameter fields have their Xoffsets set according to their frame offset.

Thanks, @mdempsky. We should add a comment in the code; re-opening this for a doc fix in 1.12.


[Bug]: Roles cannot be checked as they are not defined

Summary
Roles, like
:sql:`SELECT * FROM table`

are not defined, so typos or unsupported roles cannot be warned about, like (mind the g instead the q):
:sgl:`SELECT * FROM table`

In Sphinx, custom roles have to be defined:
https://www.sphinx-doc.org/en/master/usage/restructuredtext/roles.html#inline-code-highlighting
On rendering, Sphinx displays an error about an unknown role:
 ERROR: Unknown interpreted text role "sgl".

This way, the user can spot the error and correct it.
Guides holds no definition of custom roles and therefore cannot warn about it.

@linawolf it looks like the \phpDocumentor\Guides\RestructuredText\TextRoles\DefaultTextRoleFactory we do collect the roles that are known by the parser. I think we can use trigger a warning when we fall back to the generic role. But that seems not always valid to me as you can set the default role via a directive. Which makes this expected behavior. On the other hand it would be nice if we could detect typo's what do you think?


const-qualify interp_instance structs in interpolation functions

Interpolation functions take as an input an interp_instance struct.
As an example,
https://github.com/ARM-software/CMSIS-DSP/blob/99e15d3d95084b6868c5579b6791aff5cbeefcd6/Source/InterpolationFunctions/arm_linear_interp_f32.c#L79-L81
As far as I understand, this interp_instance is not modified by the function, and so the pointer can be declared as pointer to const.
float32_t arm_linear_interp_f32(
  const arm_linear_interp_instance_f32 * S,
  float32_t x)

In this way the code is more expressive, and a const struct can be passed as an input without any warning about casting-away its const-ness.

Corrected in latest commit


Uri.origin issue for empty host

  var uri = new Uri(scheme: "http", host: "", port: 8080);
  print(uri.toString()); // http://:8080
  print(uri.origin); // Exception Bad state: Cannot use origin without a scheme: http://:8080

Origin is described at https://www.w3.org/TR/2011/WD-html5-20110405/origin-0.html#origin and I didn't find any limitations for an empty host there. Therefore, I believe, there should be no error.
If, though, an empty host is prohibited, then an error message Cannot use origin without a scheme is wrong

The error message is wrong.
HTTP Uris don't generally allow omitting the host or authority. It's not something we check in the Uri class, but here we are handling an HTTP specific thing (since origin only works for HTTP[S] at all), and it doesn't make sense to have an HTTP origin with no host.

Now has specific error message for the no-host case.


Conflict with Leugi's Monopoly++

Describe the bug
CQUI seems to conflict with being able to purchase the two new units added by Monopoly++, the investor and the tycoon. AI can use them just fine, but they are unable to be purchased by players.
To Reproduce
Steps to reproduce the behavior:

Enable all Leugi's Monopoly++ mods
Enable CQUI
Watch as you are unable to purchase investors and later tycoons, while the AI builds all the monopolies and industries.

Expected behavior
Being able to purchase investors and tycoons.
General informations

OS: macOS Mojave
Version <IP_ADDRESS>
Expansion: All
Other mods: All of Leugi's Monopoly++ mods

heh, actually saw the comments on the Monopoly++ page before seeing this issue here.  I found the problem and will be creating a PR in a few minutes.  Thank you for reporting this!

I saw there was a PR by @HeroOfOlympus after I had made the changes that is now PR #316

this is fixed now


Find or write a method to convert a method signature into a human-readable form

This would be for Exceptions.java for "NoSuchMethodException" and "NoSuchMethodError" (and maybe the field errors too).
Method signatures are described here: https://udaniweeraratne.wordpress.com/2016/07/10/how-to-generate-jni-based-method-signature/
I haven't been able to find source code anywhere for doing the reverse conversion (apparently "FindBugs" has some, but it's not in a library).

The javap -s _class_name_ command shows the signatures of methods in that class.

Here's an example of the error message from the "NoSuchMethodError":
info.rlwhitcomb.calc.CalcParser$TimeThisStmtContext.stmtBlock()Linfo/rlwhitcomb/calc/CalcParser$StmtBlockContext;


Add function to log cuda devices

PR that adds a lightweight function to check for CUDA device availability using a lightweight library https://github.com/gpuopenanalytics/pynvml. The advantage is avoiding to install heavy libraries in express such as torch or TF

Thanks @PhilippeMoussalli
Why do we need this function? It only logs information, so I'm not sure if it's worth adding a dependency.

Thanks @PhilippeMoussalli
Why do we need this function? It only logs information, so I'm not sure if it's worth adding a dependency.

This will be used at the beginning of every component as a sanity check if there is a GPU device running (check if GPU drivers are installed properly on the underlying nodes running the job). It will be probably be repeated across different component.
Alternatively we can create a soft dependency check for it (similar to pandas, HF datasets) and then include in the requirements.txt of the components that will use it. Wdyt?

But it only logs information, right? Shouldn't it either return a value or raise an error in some cases?
If we just want to check if an Nvidia GPU is available, we can do that without a dependency as well:
import subprocess

try:
    subprocess.check_output('nvidia-smi')
    print('Nvidia GPU detected!')
except Exception: # this command not being found can raise quite a few different errors depending on the configuration
    print('No Nvidia GPU in system!')

But it only logs information, right? Shouldn't it either return a value or raise an error in some cases?
If we just want to check if an Nvidia GPU is available, we can do that without a dependency as well:
import subprocess

try:
    subprocess.check_output('nvidia-smi')
    print('Nvidia GPU detected!')
except Exception: # this command not being found can raise quite a few different errors depending on the configuration
    print('No Nvidia GPU in system!')

makes sense! Thank you for the suggestion. Implemented :)

Mainly since it's going to be repeated across components that have GPU. A bit similar to the functions that we have that check for soft dependencies. Alternatively we can just add it at the try except at beginning of each component if you see it more fit.

My main issue is that it only logs things. It does not return a value or raise an error, so I don't see how this is useful.

Mainly since it's going to be repeated across components that have GPU. A bit similar to the functions that we have that check for soft dependencies. Alternatively we can just add it at the try except at beginning of each component if you see it more fit.

My main issue is that it only logs things. It does not return a value or raise an error, so I don't see how this is useful.

I see your point, I think then we'll just include them in the components themselves. I'll close the PR.


[WIP] chore(models): Enforce validations

Coverage increased (+0.004%) to 67.267% when pulling 71e79d724d6f5699df808612cefb5ae07c50d0f6 on chore/models-validations into 7bdfdd00fd443064cf619b60af326ad52566294c on master.


Some hotspot data not available on map

Not all hotspots are being fetched by the MapScreen. This means the mouseover feature which draws a line from a packet to the hotspot that received it does not work for all hotspots. It looks like only 10,000 hotspots are loaded out of the current 15,000+ per this code.
https://github.com/helium/cargo-elixir/blob/adc92d58efa896e1418af951b6080e290f029a2a/assets/js/pages/MapScreen.js#L153
This is a nice to have issue and by no means critical functionality. If the limit of 10,000 is intentional, perhaps the list of hotspots can be filtered based on the current view of the map instead of pulling from all hotspots which returns the most recent 10,000 regardless of whether they are on the map or no.

Thanks for pointing this out @PaulVMo!

Thanks for pointing this out @PaulVMo!


Link issue & author on CHANGELOG

Problem
Current CHANGELOG does not link the issue nor the author.
Solution
Would be cool to have links, e.g:

Implemented ABC (#123, roby)

Although this would be nice to have, we decided not do to do this earlier because it is really cumbersome to setup, and also makes the 'raw' markdown really hard to read.

Closing this I guess? If you think we should follow with this feel free to reopen.


Fixed naming and other code smell issues

Fixing of the issues found here: https://sonarcloud.io/dashboard?id=TeamInterject-164531_BillChop_FE

Coverage remained the same at 6.482% when pulling b2a6b97a2dffebce70abe49a431cc9954d413108 on hotfix/code-smell-issues-fe into 628b4d88618fd81314068f59440cbae8ea265f3f on master.

 Task 13: Fix frontend code smells


segfault in all connext request-response tests on Windows

All of the connext based request-response tests and examples crash in the std::string implementation with this error:
Unhandled exception at 0x00007FFF11C61332 (vcruntime140d.dll) in test_server__rmw_connext_cpp.exe: 0xC0000005: Access violation reading location 0xFFFFFFFFFFFFFFFF.

The back trace (of the connext version of test_server):
>	userland_msgs__rosidl_typesupport_connext_cpp.dll!std::char_traits<char>::copy(char * _First1, const char * _First2, unsigned __int64 _Count) Line 529	C++
 	userland_msgs__rosidl_typesupport_connext_cpp.dll!std::basic_string<char,std::char_traits<char>,std::allocator<char> >::assign(const std::basic_string<char,std::char_traits<char>,std::allocator<char> > & _Right, unsigned __int64 _Roff, unsigned __int64 _Count) Line 1132	C++
 	userland_msgs__rosidl_typesupport_connext_cpp.dll!std::basic_string<char,std::char_traits<char>,std::allocator<char> >::assign(const std::basic_string<char,std::char_traits<char>,std::allocator<char> > & _Right) Line 1115	C++
 	userland_msgs__rosidl_typesupport_connext_cpp.dll!std::basic_string<char,std::char_traits<char>,std::allocator<char> >::operator=(const std::basic_string<char,std::char_traits<char>,std::allocator<char> > & _Right) Line 1003	C++
 	userland_msgs__rosidl_typesupport_connext_cpp.dll!connext::ReplierParams<userland_msgs::dds_::AddTwoIntsRequest_,userland_msgs::dds_::AddTwoIntsResponse_>::service_name(const std::basic_string<char,std::char_traits<char>,std::allocator<char> > & service_name) Line 72	C++
 	userland_msgs__rosidl_typesupport_connext_cpp.dll!userland_msgs::service_type_support::create_replier__AddTwoInts(void * untyped_participant, const char * service_name, void * * untyped_reader) Line 74	C++
 	rmw_connext_cpp.dll!rmw_create_service(const rmw_node_t * node, const rosidl_service_type_support_t * type_support, const char * service_name) Line 640	C++
 	test_server__rmw_connext_cpp.exe!rclcpp::node::Node::create_service<userland_msgs::AddTwoInts>(const std::basic_string<char,std::char_traits<char>,std::allocator<char> > & service_name, std::function<void __cdecl(std::shared_ptr<rmw_request_id_t> const &,std::shared_ptr<userland_msgs::AddTwoIntsRequest_<std::allocator<void> > > const &,std::shared_ptr<userland_msgs::AddTwoIntsResponse_<std::allocator<void> > > &)> callback_with_header, std::shared_ptr<rclcpp::callback_group::CallbackGroup> group) Line 219	C++
 	test_server__rmw_connext_cpp.exe!main(int argc, char * * argv) Line 36	C++

It's my belief that this is related to https://github.com/ros2/ros2/issues/31, because when researching that issue I remember a discussion online about how the libc++ developers intentionally caused a link time error even though they could have made it work in order to avoid a run time error which would occur when creating a std::string in one library and passing it to a library built with a different version. Here is the summary:
http://stackoverflow.com/questions/8454329/why-cant-clang-with-libc-in-c0x-mode-link-this-boostprogram-options-examp/8457799#8457799

In order to turn this run time crash into a link time error, libc++ uses a C++11 language feature called inline namespace to change the ABI of std::string without impacting the API of std::string. That is, to you std::string looks the same. But to the linker, std::string is being mangled as if it is in namespace std::__1. Thus the linker knows that std::basic_string and std::__1::basic_string are two different data structures (the former coming from gcc's libstdc++ and the latter coming from libc++).

And I think that the VS2013 - VS2015 Preview headers do not use this trick, but potentially do change the implementation of basic_string, which would cause an issue. This might be expected since they do not encourage you to use binaries built with one VC with binaries built from another.
So, it looks to me that we have no bug in our code, but simply that we need newer or from source versions of the RTI libraries to make this work.

Also, I'd be happy to walk anyone who's interested through the setup I have for a second opinion.

Here's the back trace for the connext version of test_client too:
 	appcrt140d.dll!00007fff0652d7f1()	Unknown
 	appcrt140d.dll!00007fff0652d48a()	Unknown
 	appcrt140d.dll!00007fff0654dea6()	Unknown
 	vcruntime140d.dll!00007fff11c670f6()	Unknown
 	test_client__rmw_connext_cpp.exe!__scrt_unhandled_exception_filter(_EXCEPTION_POINTERS * const pointers) Line 78	C++
 	KernelBase.dll!00007fff1bf41c72()	Unknown
 	ntdll.dll!00007fff1ecdfb33()	Unknown
 	ntdll.dll!00007fff1ecc2896()	Unknown
 	ntdll.dll!00007fff1ecd3f0d()	Unknown
 	ntdll.dll!00007fff1ec94887()	Unknown
 	ntdll.dll!00007fff1ec93aad()	Unknown
 	KernelBase.dll!00007fff1be68b9c()	Unknown
 	msvcr120.dll!00007fff060de3d9()	Unknown
 	ntdll.dll!00007fff1ecd3573()	Unknown
 	rticonnextmsgcpp.dll!`std::basic_string<char,std::char_traits<char>,std::allocator<char> >::_Copy'::`1'::catch$1() Line 2219	C++
 	msvcr120.dll!00007fff060e6920()	Unknown
 	msvcr120.dll!00007fff060de36d()	Unknown
 	ntdll.dll!00007fff1ecd3573()	Unknown
 	rticonnextmsgcpp.dll!std::basic_string<char,std::char_traits<char>,std::allocator<char> >::_Copy(unsigned __int64 _Newsize, unsigned __int64 _Oldlen) Line 2215	C++
 	rticonnextmsgcpp.dll!std::basic_string<char,std::char_traits<char>,std::allocator<char> >::assign(const std::basic_string<char,std::char_traits<char>,std::allocator<char> > & _Right, unsigned __int64 _Roff, unsigned __int64 _Count) Line 1138	C++
>	rticonnextmsgcpp.dll!connext::RequesterParams::service_name(const std::basic_string<char,std::char_traits<char>,std::allocator<char> > & name) Line 66	C++
 	userland_msgs__rosidl_typesupport_connext_cpp.dll!userland_msgs::service_type_support::create_requester__AddTwoInts(void * untyped_participant, const char * service_name, void * * untyped_reader) Line 43	C++
 	rmw_connext_cpp.dll!rmw_create_client(const rmw_node_t * node, const rosidl_service_type_support_t * type_support, const char * service_name) Line 566	C++
 	test_client__rmw_connext_cpp.exe!rclcpp::node::Node::create_client<userland_msgs::AddTwoInts>(const std::basic_string<char,std::char_traits<char>,std::allocator<char> > & service_name, std::shared_ptr<rclcpp::callback_group::CallbackGroup> group) Line 161	C++
 	test_client__rmw_connext_cpp.exe!request<userland_msgs::AddTwoInts>(std::shared_ptr<rclcpp::node::Node> node, std::function<void __cdecl(std::shared_ptr<userland_msgs::AddTwoIntsRequest_<std::allocator<void> > > &,unsigned __int64)> set_data_func, unsigned __int64 number_of_requests) Line 31	C++
 	test_client__rmw_connext_cpp.exe!main(int argc, char * * argv) Line 83	C++

The back trace is different, but the problem traces to this line in our code:
https://github.com/ros2/rmw_connext/blob/master/rosidl_typesupport_connext_cpp/resource/srv_ServiceTypeSupport.cpp.template#L56
And stepping through code I can confirm that the service_name c-string is valid and that the requester_params object looks valid as well.

As an experiment, I've ported our custom implementation of services from OpenSplice to Connext (only for the static variant) and the tests now pass:
http://ci.ros2.org/job/ros2_batch_ci_windows/534/testReport/(root)/test_services_cpp__rmw_connext_cpp/
As a side benefit, OpenSplice and Connext can interoperate.
I also agree with @wjwwood that this is due to an change in std::basic_string in VS2015.

Holding off on this hoping for an upstream fix to work with VS2015

The first engineering build does not magically make it work (as we had hoped). I have debugged some more on the Windows machine (running test_services_server_cpp__rmw_connext_cpp). The application crashed with an access violation within the connext::details::EntityParams. I have asked our contacts at RTI for support.

This has been addressed by building Release on Windows: ament/ament_tools#72
The followup ticket ros2/rmw_connext#89 will allow to build Debug again by selecting the corresponding Connext libraries.

Amazing: http://ci.ros2.org/job/ci_windows/922/testReport/


[New game]: Math Game

🎮 Game Request
This is a game based on simple mathematics.
Point down the features
On clicking the start button on the screen shown to the user, the user will get a mathematical equation with a blank space. The user has to fill that blank space with appropriate integers or operators. Then a user clicks the submit button after entering their response.
After clicking the submit button, a result screen is shown. On the result screen, we display the result and a restart button.
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @vishalkumarmahto !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @vishalkumarmahto ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


chore: update Talisman logo

Hello,
We noticed that the Talisman logo is not the one that we use to designate the wallet.
Would be great if you could accept this PR which changes it to the correct one.

Thank you @0xKheops and apologies for the delay

No worries, thanks to you!


Unable to start session due to Appium Server errors

Hi,
I'm writing you because I would like to try this library with my application written in Flutter.
Actually, after having established the connection to the Appium Server using the following command:
appium -a <IP_ADDRESS> -p 4723 --allow-cors --session-override
I've tried to run the Appium Inspector with the following configuration:

The application doesn't open, and in the Appium Server I see the following logs:
New FlutterDriver session created successfully, session 16faeda2-1e6f-4d98-ac35-ba03a6471c0b added to master session list
[debug] [AppiumDriver@7e1a] Event 'newSessionStarted' logged at<PHONE_NUMBER>038 (12:35:10 GMT+0200 (Ora legale dell’Europa centrale))
[AppiumDriver@7e1a] Promoting 1 sessionless plugins to be attached to session ID 16faeda2-1e6f-4d98-ac35-ba03a6471c0b
[debug] [FlutterDriver@a2f4 (16faeda2)] Cached the protocol value 'W3C' for the new session 16faeda2-1e6f-4d98-ac35-ba03a6471c0b
[debug] [FlutterDriver@a2f4 (16faeda2)] Responding to client with driver.createSession() result: {"capabilities":{"subcommand":"server","address":"<IP_ADDRESS>","allowCors":true,"port":4723,"sessionOverride":true,"usePlugins":["execute-driver"],"extraArgs":[],"allowInsecure":[],"basePath":"","callbackPort":4723,"debugLogSpacing":false,"denyInsecure":[],"keepAliveTimeout":600,"localTimezone":false,"loglevel":"debug","logNoColors":false,"logTimestamp":false,"longStacktrace":false,"noPermsCheck":false,"relaxedSecurityEnabled":false,"strictCaps":false,"useDrivers":[],"tmpDir":"C:\\Users\\endrim\\AppData\\Local\\Temp","platformName":"Android","platformVersion":"13","deviceName":"R9JRC0EL0YT","automationName":"Flutter","ensureWebviewsHavePages":true,"nativeWebScreenshot":true,"newCommandTimeout":3600,"connectHardwareKeyboard":true,"fastReset":true,"skipUninstall":true}}
[HTTP] <-- POST /session 200 6550 ms - 837
[HTTP]
[HTTP] --> GET /session/16faeda2-1e6f-4d98-ac35-ba03a6471c0b/screenshot
[HTTP] {}
[debug] [FlutterDriver@a2f4 (16faeda2)] Calling AppiumDriver.getScreenshot() with args: ["16faeda2-1e6f-4d98-ac35-ba03a6471c0b"]
[debug] [FlutterDriver] Executing Flutter driver command 'getScreenshot'
[debug] [FlutterDriver@a2f4 (16faeda2)] Encountered internal error running command: TypeError: Cannot read properties of null (reading 'call')
[debug] [FlutterDriver@a2f4 (16faeda2)]     at FlutterDriver.getScreenshot (C:\Users\endrim\.appium\node_modules\appium-flutter-driver\lib\commands\screen.ts:4:39)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at commandExecutor (C:\Users\endrim\AppData\R1oaming\npm\node_modules\appium\node_modules\@appium\base-driver\lib\basedriver\driver.ts:106:18)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\node_modules\async-lock\lib\index.js:171:12
[debug] [FlutterDriver@a2f4 (16faeda2)]     at AsyncLock._promiseTry (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\node_modules\async-lock\lib\index.js:304:31)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at exec (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\node_modules\async-lock\lib\index.js:170:9)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at AsyncLock.acquire (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\node_modules\async-lock\lib\index.js:187:3)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at FlutterDriver.executeCommand (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\node_modules\@appium\base-driver\lib\basedriver\driver.ts:122:39)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at processTicksAndRejections (node:internal/process/task_queues:95:5)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at FlutterDriver.executeCommand (C:\Users\endrim\.appium\node_modules\appium-flutter-driver\lib\driver.ts:167:16)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at defaultBehavior (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\lib\appium.js:684:14)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at AppiumDriver.executeWrappedCommand (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\lib\appium.js:774:16)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at AppiumDriver.executeCommand (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\lib\appium.js:696:17)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at asyncHandler (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\node_modules\@appium\base-driver\lib\protocol\protocol.js:393:19)
[HTTP] <-- GET /session/16faeda2-1e6f-4d98-ac35-ba03a6471c0b/screenshot 500 54 ms - 662
[HTTP]
[HTTP] --> GET /session/16faeda2-1e6f-4d98-ac35-ba03a6471c0b/window/rect
[HTTP] {}
[debug] [FlutterDriver@a2f4 (16faeda2)] Calling AppiumDriver.getWindowRect() with args: ["16faeda2-1e6f-4d98-ac35-ba03a6471c0b"]
[debug] [FlutterDriver] Executing Flutter driver command 'getWindowRect'
[debug] [FlutterDriver@a2f4 (16faeda2)] Encountered internal error running command: NotYetImplementedError: Method has not yet been implemented
[debug] [FlutterDriver@a2f4 (16faeda2)]     at FlutterDriver.executeCommand (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\node_modules\@appium\base-driver\lib\basedriver\driver.ts:100:13)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at processTicksAndRejections (node:internal/process/task_queues:95:5)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at FlutterDriver.executeCommand (C:\Users\endrim\.appium\node_modules\appium-flutter-driver\lib\driver.ts:167:16)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at defaultBehavior (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\lib\appium.js:684:14)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at AppiumDriver.executeWrappedCommand (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\lib\appium.js:774:16)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at AppiumDriver.executeCommand (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\lib\appium.js:696:17)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at asyncHandler (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\node_modules\@appium\base-driver\lib\protocol\protocol.js:393:19)
[HTTP] <-- GET /session/16faeda2-1e6f-4d98-ac35-ba03a6471c0b/window/rect 405 17 ms - 1057
[HTTP]
[HTTP] --> GET /session/16faeda2-1e6f-4d98-ac35-ba03a6471c0b/source
[HTTP] {}
[debug] [FlutterDriver@a2f4 (16faeda2)] Calling AppiumDriver.getPageSource() with args: ["16faeda2-1e6f-4d98-ac35-ba03a6471c0b"]
[debug] [FlutterDriver] Executing Flutter driver command 'getPageSource'
[debug] [FlutterDriver@a2f4 (16faeda2)] Encountered internal error running command: NotImplementedError: Not implemented yet for find.
[debug] [FlutterDriver@a2f4 (16faeda2)]     at FlutterDriver.getPageSource (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\node_modules\@appium\base-driver\lib\basedriver\commands\find.ts:54:11)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at commandExecutor (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\node_modules\@appium\base-driver\lib\basedriver\driver.ts:106:18)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\node_modules\async-lock\lib\index.js:171:12
[debug] [FlutterDriver@a2f4 (16faeda2)]     at AsyncLock._promiseTry (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\node_modules\async-lock\lib\index.js:304:31)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at exec (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\node_modules\async-lock\lib\index.js:170:9)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at AsyncLock.acquire (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\node_modules\async-lock\lib\index.js:187:3)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at FlutterDriver.executeCommand (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\node_modules\@appium\base-driver\lib\basedriver\driver.ts:122:39)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at processTicksAndRejections (node:internal/process/task_queues:95:5)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at FlutterDriver.executeCommand (C:\Users\endrim\.appium\node_modules\appium-flutter-driver\lib\driver.ts:167:16)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at defaultBehavior (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\lib\appium.js:684:14)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at AppiumDriver.executeWrappedCommand (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\lib\appium.js:774:16)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at AppiumDriver.executeCommand (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\lib\appium.js:696:17)
[debug] [FlutterDriver@a2f4 (16faeda2)]     at asyncHandler (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\node_modules\@appium\base-driver\lib\protocol\protocol.js:393:19)
[HTTP] <-- GET /session/16faeda2-1e6f-4d98-ac35-ba03a6471c0b/source 405 25 ms - 1906
[HTTP]
[HTTP] --> GET /session/16faeda2-1e6f-4d98-ac35-ba03a6471c0b/timeouts
[HTTP] {}
[debug] [FlutterDriver@a2f4 (16faeda2)] Calling AppiumDriver.getTimeouts() with args: ["16faeda2-1e6f-4d98-ac35-ba03a6471c0b"]
[debug] [FlutterDriver] Executing Flutter driver command 'getTimeouts'
[debug] [FlutterDriver@a2f4 (16faeda2)] Responding to client with driver.getTimeouts() result: {"command":3600000,"implicit":0}
[HTTP] <-- GET /session/16faeda2-1e6f-4d98-ac35-ba03a6471c0b/timeouts 200 7 ms - 42

I specify I have installed Appium 2.0 and the latest versions of Appium Flutter Driver and Appium Flutter Library.
I have searched for these errors on Google but I haven't find anything about them.
Am I missing something in my configuration? Could you help me in resolving my issue?
Thanks a lot in advance.

Btw, below means a new session creation itself succeeded.
[debug] [AppiumDriver@7e1a] Event 'newSessionStarted' logged at<PHONE_NUMBER>038 (12:35:10 GMT+0200 (Ora legale dell’Europa centrale))
[AppiumDriver@7e1a] Promoting 1 sessionless plugins to be attached to session ID 16faeda2-1e6f-4d98-ac35-ba03a6471c0b
[debug] [FlutterDriver@a2f4 (16faeda2)] Cached the protocol value 'W3C' for the new session 16faeda2-1e6f-4d98-ac35-ba03a6471c0b
[debug] [FlutterDriver@a2f4 (16faeda2)] Responding to client with driver.createSession() result: {"capabilities":{"subcommand":"server","address":"<IP_ADDRESS>","allowCors":true,"port":4723,"sessionOverride":true,"usePlugins":["execute-driver"],"extraArgs":[],"allowInsecure":[],"basePath":"","callbackPort":4723,"debugLogSpacing":false,"denyInsecure":[],"keepAliveTimeout":600,"localTimezone":false,"loglevel":"debug","logNoColors":false,"logTimestamp":false,"longStacktrace":false,"noPermsCheck":false,"relaxedSecurityEnabled":false,"strictCaps":false,"useDrivers":[],"tmpDir":"C:\\Users\\endrim\\AppData\\Local\\Temp","platformName":"Android","platformVersion":"13","deviceName":"R9JRC0EL0YT","automationName":"Flutter","ensureWebviewsHavePages":true,"nativeWebScreenshot":true,"newCommandTimeout":3600,"connectHardwareKeyboard":true,"fastReset":true,"skipUninstall":true}}
[HTTP] <-- POST /session 200 6550 ms - 837

Hi,
Thank you, does a solution for this "problem" actually exist?
If Appium Inspector doesn't actually work with this plugin, could I use the plugin with Robot Framework simply including it in my project and using its functions, using automationName = Flutter, or isn't it compatible with Robot Framework or should I use additional configuration?
Thank you in advance.

I'm not sure if Robot Framework supports this driver, but Appium Python client can use this. e.g. https://github.com/appium-userland/appium-flutter-driver/blob/main/example/python/example.py

Hi,
Thanks for the explanation.
I've tried to execute a test with Robot Framework.
The test opens the app, then I try to use the Input Text function on an input field using its key.
In the Appium Server logs, these logs are printed:
[FlutterDriver] Attempt #1
[debug] [FlutterDriver] Error: No observatory url was found in the device log. Please make sure the application under test logs an observatory url when the application starts.
[debug] [FlutterDriver]     at processLogToGetobservatory (C:\Users\endrim\.appium\node_modules\appium-flutter-driver\lib\sessions\observatory.ts:197:11)
[debug] [FlutterDriver]     at getObservatoryWsUri (C:\Users\endrim\.appium\node_modules\appium-flutter-driver\lib\sessions\android.ts:62:43)
[debug] [FlutterDriver]     at connectSocket (C:\Users\endrim\.appium\node_modules\appium-flutter-driver\lib\sessions\observatory.ts:33:34)
[debug] [FlutterDriver]     at startAndroidSession (C:\Users\endrim\.appium\node_modules\appium-flutter-driver\lib\sessions\android.ts:33:18)
[debug] [FlutterDriver]     at processTicksAndRejections (node:internal/process/task_queues:95:5)
[debug] [FlutterDriver]     at FlutterDriver.createSession (C:\Users\endrim\.appium\node_modules\appium-flutter-driver\lib\sessions\session.ts:43:47)
[debug] [FlutterDriver]     at AppiumDriver.createSession (C:\Users\endrim\AppData\Roaming\npm\node_modules\appium\lib\appium.js:346:35)
[FlutterDriver] Connecting to Dart Observatory: undefined
[FlutterDriver] {}
[FlutterDriver] Check Dart Observatory URI no URI found in the device log
[FlutterDriver] Waiting 3 seconds before trying...

What should I do?
Maybe I should configure the "appium:observatoryWsUri" Device Capability, but how could I know which uri should I set?
Thank you in advance.

Did you configure https://github.com/appium-userland/appium-flutter-driver#usage ? Set  flutter_driver for the app dependency and set enableFlutterDriverExtension?
observatoryWsUri is for advanced usage. Usually this driver gets the url automatically, but the error means probably your app hasn't exposed the expected url because of not configured as the usage

What I know of is basically that's all.
Did you confirm the observatory url exist in the logcat when you launch the app? If not, maybe your DartVM hasn't been exposed. For example, a release build does not work.


Filename Compatibility with MultiMC

It seems that multimc has difficulty extracting filenames with special characters in them. Specifically it seems to choke with the § section symbol.
24.162 W Failed to extract file "resourcepacks/Better-Leaves-6.0-1.13+\u00156.zip" to "resourcepacks/Better-Leaves-6.0-1.13+\u00156.zip"
this is reported a couple times on their discord server and also in an open issue on github https://github.com/MultiMC/Launcher/issues/4182

Fixed.


add the pip installation strategy to the documentation

Is your feature proposal related to a problem?
I wanted to use the lib and was discovering the installation section of the documentation. The only documented way is conda, even though the lib is published and up-to-date on pipy. It didn't took me long to figure it out but that was still a missing information.
Describe the solution you'd like to propose
I would like to update the documentation file to add the second way of installing osmnx: pip

In a sense, we do document the pip installation strategy. The docs state:

you can install OSMnx via pip if you already have all of its dependencies installed and fully tested on your system. Note: installing the dependencies with pip is nontrivial. If you don’t know exactly what you’re doing, just use conda as described above.

We don't document it any further than that, because there's nothing else to document on our end. OSMnx is pure-Python and trivially easy to install if you have the dependencies all installed and tested. The devil's in the details, and some of the dependencies' dependencies have C extensions and rely on other compiled libraries and require system-specific compilation, installation, and configuration. To install the dependencies, you have to go to each dependency's web site and follow their instructions, which is a nontrivial (and often-changing) process. If someone needs/wants to install OSMnx via pip, it is possible but difficult.
Long story short: we don't (and really can't) support pip installation, and don't document it accordingly.

thanks for taking the time to answer 👍

now that GDAL has been removed from the deps of osmnx (https://github.com/gboeing/osmnx/issues/916) does it means that a pure pip installation is possible again or is there still other C-like dependencies ?

The latest docs note that pip is an installation option, but not "officially supported" by the OSMnx project because we can't personally guarantee the availability, compilation, and installation of C dependencies on every system. In other words, pip installation usually works on most systems, but if it it doesn't work on your system, you'll have to seek support from the relevant dependency that's not working.
It's worth noting that OSMnx has an optional dependency of rasterio, which itself depends on GDAL. So a full installation of OSMnx still installs GDAL, though it is no longer a direct requirement. Additionally, OSMnx requires shapely (which depends on GEOS) and GeoPandas (which depends on pyproj which in turn depends on PROJ). So there are still compiled sub-dependencies. Note that GeoPandas uses similar language as us, noting that pip installation is possible but potentially nontrivial.


Nested transaction support via savepoint

Make sure these boxes checked before submitting your pull request.

[x] Do only one thing
[x] No API-breaking changes
[x] New code/logic commented & tested

For significant changes like big bug fixes, new features, please open an issue to make an agreement on an implementation design/plan first before starting it.
What did this pull request do?
Support simulated nested transaction via savepoint

recheck

Codecov Report

Merging #2516 into master will decrease coverage by 0.07%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #2516      +/-   ##
==========================================
- Coverage   79.34%   79.26%   -0.08%
==========================================
  Files          24       24
  Lines        3452     3487      +35
==========================================
+ Hits         2739     2764      +25
- Misses        612      618       +6
- Partials      101      105       +4

Impacted Files
Coverage Δ

utils.go
86.5% <100%> (+1.29%)
:arrow_up:

main.go
79.77% <54.83%> (-1.23%)
:arrow_down:

Continue to review full report at Codecov.

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

Will this be merged any time soon?

will merge it sonn?

Codecov Report

Merging #2516 into master will increase coverage by 0.32%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##           master    #2516      +/-   ##
==========================================
+ Coverage   79.01%   79.33%   +0.32%
==========================================
  Files          24       24
  Lines        3540     3528      -12
==========================================
+ Hits         2797     2799       +2
+ Misses        642      632      -10
+ Partials      101       97       -4

Impacted Files
Coverage Δ

main.go
82.44% <78.94%> (+1.34%)
:arrow_up:

utils.go
86.71% <100.00%> (+1.50%)
:arrow_up:

logger.go
26.56% <0.00%> (-5.33%)
:arrow_down:

dialect_common.go
40.59% <0.00%> (-3.34%)
:arrow_down:

callback_query.go
82.14% <0.00%> (-0.62%)
:arrow_down:

callback_create.go
86.08% <0.00%> (-0.24%)
:arrow_down:

scope.go
87.00% <0.00%> (-0.04%)
:arrow_down:

callback.go
97.41% <0.00%> (+0.02%)
:arrow_up:

dialect_mysql.go
1.37% <0.00%> (+0.05%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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

Really looking forward for this MR to get merged


Add dark theme

[x] Base dark theme
[x] Add support for prefers-color-scheme which will be used as the default if the theme if undefined in the localstorage.
[ ] Still a fair few variables that need tweaking for dark theme

Looks great, like the top right toggle

I see two issues with the switcher. More of a wishlist than issues.

If you manually set a theme you cannot go back to automatic selection without clearing local storage. This is not a very user friendly approach. I would prefer it if the switcher was tri-state with the default state being "auto".
When my browser changes from light to dark or vice versa the site remains in the light/dark state it loaded with. I do not know of a good way to fix this with JavaScript. I imagine there must be a better way than setting an interval timer every 1-2 seconds to check the browser preference.

Should I file issues about them or should I go away quietly? :smiley:

Thanks Nic, I'll add a "system preference" (middle) state to the toggle

PR: https://github.com/C-Lodder/lightning/pull/77
Closing this for now. If any additional aspects of the page are missing their respective dark colours, separate issues can be opened.