Firebase messaging -> Foreground notification not showing on iOS

On Android everything works fine. But on IOS, when app in foreground no notification shown, and also onMessage is not called. if app in background or terminated, everything works fine.
Flutter Doctor:
[✓] Flutter (Channel stable, 2.0.4, on macOS 11.2 20D64 darwin-x64, locale en-AE)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.1)
[✓] VS Code (version 1.57.1)
[✓] Connected device (2 available)
Flutter Dependencies:

firebase_core: "^1.3.0"
firebase_auth: ^2.0.0
firebase_messaging: ^10.0.3
firebase_analytics: ^8.1.2

I am sending notification from firebase console.
CODE:
  if (Platform.isIOS) {
      NotificationSettings settings = await messaging.requestPermission(
        alert: true,
        announcement: false,
        badge: true,
        carPlay: false,
        criticalAlert: false,
        provisional:
            false, //to show message at lock screen  by default it is false
        sound: true,
      );
      if (settings.authorizationStatus == AuthorizationStatus.authorized) {
        print('User granted permission');
      } else if (settings.authorizationStatus ==
          AuthorizationStatus.provisional) {
        print('User granted provisional permission');
      } else {
        print('User declined or has not accepted permission');
      }
      //for foreground
      await messaging.setForegroundNotificationPresentationOptions(
        alert: true, // Required to display a heads up notification
        badge: true,
        sound: true,
      );
    }

 FirebaseMessaging.onMessage.listen((RemoteMessage message) {
      print("foreground message::::$message");
    });

Facing same issue, push message not receiving in iOS device. Have tried to send directly from firebase console.

same issue

Hi @flutter-developer-lab
Have you requested notification permissions on the device? I'm not seeing it in the code you provided.
Could you try running the official example?
Thank you

Hi @markusaksli-nc
I follow flutterfire documentation. You can See in my code i request for permission. I also get allow notification dialog. if is there any other permission request. Could you please provide me some specific code.

Nevermind, I just read over it for some reason. Have you tried running the example?

I had this problem, I fixed it changing the file AppDelegate.swift like this:
@objc class AppDelegate: FlutterAppDelegate {
override func application(
_ application: UIApplication,
didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?
) -> Bool {
FirebaseApp.configure()
GeneratedPluginRegistrant.register(with: self)
return true
// original line was:
// super.application(application, didFinishLaunchingWithOptions: launchOptions) //this line was working on older versions of
// iOS and stop working with iOS 14.7.1, at least in my case.
}
}
Tested with firebase console and using postman https://fcm.googleapis.com/fcm/send with json

I had this problem, I fixed it changing the file AppDelegate.swift like this:
@objc class AppDelegate: FlutterAppDelegate {
override func application(
_ application: UIApplication,
didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?
) -> Bool {
FirebaseApp.configure()
GeneratedPluginRegistrant.register(with: self)
return true
// original line was:
// super.application(application, didFinishLaunchingWithOptions: launchOptions) //this line was working on older versions of
// iOS and stop working with iOS 14.7.1, at least in my case.
}
}
Tested with firebase console and using postman https://fcm.googleapis.com/fcm/send with json

YOU ARE GENIUS!!!

Thanks @Glexium,
It's Working


Release 2.7.0

Auto Height and Unbolded Headers
Added: Auto height layout class.
Reset all header elements to normal font-weight by default.

lgtm


runtime: do not rely on a single bitmap for all Go heap?

See also #9311 for Android L and #7284 for windows/386. This might also affect
the darwin/arm port, but so far my testing hasn't revealed anything (I think darwin/arm
kernel disallows memory overcommit, so it might poses problems for devices with
less than 1GB of physical memory)
For 32-bit ports, we probably want to support a scheme where there could be separate
bitmap for each large chunk of VM reservation.
Thoughts?

Let's just muddle along until we can stop worrying about 32-bit Android, iOS, or Windows.

iOS stopped allowing new 32-bit apps shortly after @rsc's last comment.
And as of ~yesterday, the new iOS beta says 32-bit apps won't even run on iOS anymore.
So, Android and Windows.
@eliasnaur, is this a problem in practice for you?
@aclements, you want to keep this open?

Since Go 1.7 we've been able to accept allocations anywhere in the 32-bit address space without the need for multiple bitmaps (actually, it was @eliasnaur who did that work), so think we can just close this.
Let me know if I've misunderstood the issue and we can reopen.


First version of Talessya sheet

Changes / Comments
This is the first version of the sheet for Talessya setting for D&D 5ed
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[x] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

Hey there @jorsermar , before I proceed with merging you PR, I need to ask you: Are there any major difference in the rules between Talessya and the other versions of the 5e sheets that already exist in our repo? I am asking you that because while there have been duplicate public sheets allowed in the past, going forward Roll20 wants to limit this practice. Thanks for all the work you do!

Hi,
The main difference is the inclusion of different features coming from the reign of origin of the characters that will join race, class and other feature sources. Also characters will have arcane tattos that will grant them another full family of features, both temporal and permanent.
Regards,

Hi @jorsermar, it sounds to me that the features you've mentioned above could be added on top of a existing sheet and toggled on/off in the way Piety (a rule only available at the Theros settings) can be turned on/off in the options tab of the official 5e sheet. Would you kindly consider following that route? Thanks for all the work you do!

There is also a cosmetic factor, which is quite important to the author, in the form of the dragons that appear in the background, the shape of boxes and stat inputs... and of course, the fact that it might be confussing for the players having to use a sheet with a different name.

I'm willing to back this sheet entering the repository as it's in support of specifically published content, and the precedent has been set by the likes of Esper Genesis to split into secondary sheets for substantially changed work.
Merging this now.


Optional Config Option to Export Filename to Index Matches

Request is quite simple in theory. For each image that is processed and exported within the outputs section, return a series of defines, using the file name in conjunction with the output name.
#define TrekGFX_deflected 0
#define TrekGFX_icon_frequency 1
#define TrekGFX_icon_health 2
...

You could optionally include the group name of the sprite as well.
*This is one of those relatively useless features that would be more helpful to me than most others due to the ridiculous complexity of TI-Trek, but could also prove useful in other use cases where code needs to know where a particular subset of sprites start.

Added in 2652023


Feat(aws-ebs-csi-driver): add capability to deploy into self managed mode

What does this PR do?
add capability to deploy aws-ebs-csi-driver addon into self managed mode
Motivation
Currently the aws-ebs-csi-driver managed addon does not allow any configuration changes.
For example, deploying aws-ebs-csi-driver on "tainted" nodes is not supported and this can be problematic in some cases.
Until the modification of the addon in EKS managed mode is possible, I propose to be able to deploy "ans-ebs-csi-driver" in self managed mode with its helm chart
More

[x] Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
[x] Yes, I have added a new example under examples to support my PR
[ ] Yes, I have created another PR for add-ons under add-ons repo (if applicable)
[x] Yes, I have updated the docs for this feature
[x] Yes, I ran pre-commit run -a with this PR

Note: Not all the PRs required examples and docs except a new pattern or add-on added.
For Moderators

[ ] E2E Test successfully complete before merge?

Additional Notes

Thank you for creating this PR, but I do not see any related issue/s to this PR.
Please ensure to follow our Contributing Guidelines.
Thanks.

Thank you for creating this PR, but I do not see any related issue/s to this PR. Please ensure to follow our Contributing Guidelines. Thanks.

Hello,
Thank you for your feedback, I was going to create an issue but I saw that an issue already existed for a problem similar to mine.
I have just added the issue to my PR.
Does this look right to you?

Please use #558 as a reference since it follows the same pattern. Note that we'll need to add more documentation around the two different routes users can use to deploy this addon, etc.

Hello @bryantbiggs .
Thank you for your feedback, I have just taken it into consideration and updated my PR accordingly.


Missing roles required(?) by the extension

Hi, how is it that there are no roles required by the extension in the extension.yaml? Does it not require any?

Currently it doesn’t require any. I will try to set up roles for accessing Firebase storage so that one doesn’t need to make their storage public.

I added storage.admin role, feel free to reopen if your concern is not addressed yet.
Relevant commit: https://github.com/pdfplum/pdfplum/commit/ed70637c52e8645df6916f4e09479d2a94feac76


feat(plugin): add farcaster plugin

Add Farcaster Plugin to GOAT SDK
Relates to:

Add Farcaster Protocol Support

Background
What does this PR do?
This PR adds a new plugin for Farcaster protocol integration in the GOAT SDK. The plugin enables AI agents to interact with the Farcaster social protocol through the Neynar API, including:

Reading and creating casts
Managing conversations and threads
Searching content
Handling authentication via Signer UUID

What kind of change is this?

[x] New feature (non-breaking change that adds functionality)
[x] This change requires a documentation update
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Documentation changes needed?
Yes, documentation changes have been made:

Added full README with setup instructions and examples
Added JSDoc comments for all public methods
Updated main GOAT SDK docs to include Farcaster plugin section

Testing
Test Environment Setup
pnpm install
pnpm run build

Detailed testing steps

Basic Functionality Tests
// Get a cast
await tools.get_cast({
    identifier: "0x123...",
    type: "hash"
});

// Publish a cast
await tools.publish_cast({
    signer_uuid: "test-uuid",
    text: "Test cast"
});

Error Handling Tests

Test with invalid API key
Test with malformed parameters
Test rate limiting behavior

Integration Tests

Test interaction with other GOAT plugins
Verify proper event handling
Check response type consistency

Plugin-specific Testing

[x] Tested with key pair wallets
[x] Tested with Crossmint Smart Wallets
[x] Tested with both mainnet and testnet configurations
[x] Verified proper error handling and rate limiting
[x] Checked compatibility with latest GOAT SDK core

Test Coverage

Unit Tests: 95%
Integration Tests: 90%
End-to-end Tests: Added for main flows

Discord username
@codingshthefounder
Additional Notes

All dependencies are properly versioned
Performance impact has been measured
Security considerations have been reviewed
Breaking changes are documented

Ah prob need to delete the pnpm lock file and run pnpm install again! 🙏

This is awesome! :) Requested one small change and then good to merge!!
Also you might need to delete the pnpm-lock file and run pnpm install again. Also run pnpm lint:fix before pushing!

done!

@developerfred i think u didnt push the new pnpm-lock file! :)


Add Header Toolbar

Add a header toolbar for day, week, month, and previous and next buttons

Completed a header toolbar:


Create generic script to allow multiple scripts for an operator

AS discussed, threading will be implemented but will be limited to one thread for each script server instance to avoid decline in performance


Missing credential in multi_pool.md

I'm testing oracle-ords-operator/docs/examples/multi_pool.md
The init_ords.sh scripts fails with the following error on pdb1
Found Pool: pdb1...
db.password in pool pdb1, not defined
db.adminUser.password in pool pdb1, not defined
db.cdb.adminUser.password in pool pdb1, not defined
FATAL: db.password must be specified for pdb1
Pool: pdb1, Exit Code: 1

it seems that some credential is missing in the multi_pool.md example
echo "
apiVersion: database.oracle.com/v1
kind: RestDataServices
metadata:
  name: ords-multi-pool
spec:
  image: container-registry.oracle.com/database/ords:23.4.0
  forceRestart: true
  globalSettings:
    database.api.enabled: true
  poolSettings:
    - poolName: pdb1
      autoUpgradeORDS: true
      autoUpgradeAPEX: true
      db.connectionType: tns
      db.tnsAliasName: DB1
      tnsAdminSecret:
        secretName: multi-tns-admin
      restEnabledSql.active: true
      feature.sdw: true
      plsql.gateway.mode: proxied
      db.username: ORDS_PUBLIC_USER
      db.secret:
        secretName: pdb1-ords-auth
      db.adminUser: SYS
      db.adminUser.secret:
        secretName: pdb1-priv-auth" | kubectl apply -f -

There is nothing wrong with yaml file , I  mistakenly forgot to set the environment running the init_script. I'm testing the above yaml  using on premises database ; in my case, the init_script fails when we check adb.
SQLERROR:
Error starting at line : 9 in command -
                SELECT COUNT(*) FROM (
                        SELECT JSON_VALUE(cloud_identity, '$.DATABASE_OCID') AS database_ocid
                        FROM v$pdbs) t
                WHERE t.database_ocid like '%AUTONOMOUS%'
Error at Command Line : 10 Column : 22
Error report -
SQL Error: ORA-00904: "CLOUD_IDENTITY": invalid identifier
00904. 00000 -  "%s: invalid identifier"
*Cause:    The identifier or column name entered was invalid.
*Action:   Ensure the following

Thanks, was just testing this and couldn't reproduce.  However, I did catch the ORA-00904 on 23ai databases.  Fix for that on its way up.

v0.3.0 avail to resolve ORA-00904


How to prevent converting px into rem

Hi all,
My root html font size was set to percent. I upgraded clarity lib in my project based on Angular. As an effect my template is extremaly different than before becaouse all properties based on pixels are automatically converted into rem. How to prevent that? Thank you for help in advance.

Hi @falcon-eu
If you could provide a plunkr with the general idea of what you are trying to do by setting the html font-size to a percent, we could probably help you figure something out.

Okay my problem was that my project includes styles from file cliarity-ui.css located in node_modules and lots of styles were inherited from there. I have just copy old css file and now everything is okay.


Create an example for the documentation

@baiddd Could you add an markdown file to give an example for how people should make documentation for their dataset? I think you can take Manish SINGH's data as an example. And place in this repo (in the docs folder).
Another note is that, if you have several images (e.g. <60) that belongs to the same experiment or a group of experiment, you should  upload them in the same deposit.

Ok I will do that, by the way it’s not possible to upload wide field images with localisation table because a screenshot is mandatory (but I can visualize by the kaibu plugin that you implemented)

Ok I will do that, by the way it’s not possible to upload wide field images with localisation table because a screenshot is mandatory (but I can visualize by the kaibu plugin that you implemented)

I will look into that, would be good to create separate issue for each different issue.

Fixed


Update the meeting note for March 2024

📚 Documentation preview 📚: https://docs-community--110.org.readthedocs.build/

Thanks!


Remove type safety from Interface

The type safety of the interface causes some functionality to be unavailable. I'll remove the compile-time type safety of the interface and check it when I call Schema::new. It doesn't have any performance loss, but there is no guarantee that a compile-time type safety, I think it is worth to do so.

The side effect is improved runtime performance.😂

I'm not very happy with this idea.

If the interface field types do not match, there is now a panic at Schema::new and a reason is given.
The following code didn't compile well before, but it does now.
struct MyObj;

#[Object]
impl MyObj {
    async fn a(&self) -> &str {}
}

#[Interface(field(name = "a", type = "String"))]
struct MyInterface(MyObj);

Panic is not desirable. I believe there is a solution that involves traits but I'm unable to work on it currently.

Panic is not desirable. I believe there is a solution that involves traits but I'm unable to work on it currently.

Thanks for your suggestion.😁 I will make a version as soon as possible and see how it works.

@sunli829 is it possible to use a macro to add additional methods to a trait, including a default implementation? That is the solution I am considering.

@phated  I'm actually hoping to solve this problem #29. I think procedural macros may not be possible.

I am hoping to find time to understand macros better, but my other open source responsibilities are requiring my time right now.
Tomorrow I am giving a presentation on async-graphql and I wanted to show type safety on interfaces.

@phated I tried this on a new branch.
In addition, thank you very much for the support of async-graphql. I was even more excited when I found out that you are the main maintainer of glup.js.

🙇 Thank you! Your work on this project is amazing and I very much appreciate it! I am excited to help out more.

@sunli829 I think it is a great Idea especially when it panics at application start. I never really liked the implementation of Interface. I think it is somehow reversed. Isn’t there a more natural way where Interface behaves more like a trait? So that Objects and SimpleObjects can "implement" it rather than beeing passed into the interface as struct fields? It just feels wrong and doesn’t allow shareing an interface over crate boundaries.
@phated Is your presentation being recorded? I would love to see it.

@nicolaiunrein  The interface definition is not natural, but it is simple enough to use.😁


update tests to run Ubuntu/Windows/Linux for 3.7, 3.8, and 3.9. 3.10 …

Updates to run tests on Linux, Mac, Windows using Python versions 3.7, 3.8, and 3.9. 3.10 fails due to some unrelated errors in the Collections module.

This also looks great.  No reason to wait, merging.


Add instructions on how to open this project in intellij

This may sound like a stupid question, but it is not obvious how to open this project in intellij. Do we use "Import Project" or "Open".
If I choose open, I'm prompted to import gradle project. Then it asks:
Please select the modules/data to include in the project. If you accept the default (all of them) you are then prompted to remove something from the project:

Should it be this complex to open a project in IntelliJ?

Then after accepting all of the defaults, I end up getting this message:

But if do as instructed, I get more errors:

I am guessing that many Kotlin early adopters are also IntelliJ users. So it would be a good idea to make it simple for these folks.

So after all that, I went to line 29 of HomeView.kt and changed:
h2 { +"Thoughts" }
to
h2 { +"Thoughts 2" }
Then I hit Command + S.
Went back to browser, hoping to see hot reload. No hot reload. So next I tried doing Command + F9. Then I hit refresh in the browser. And that caused the client to crash, with this message in the JavaScript console:
Uncaught Error: Module build failed: Error: ENOENT: no such file or directory, open '/Volumes/repos/kotlin-fullstack-sample/frontend/build/js/frontend.js'

Hi! I'm having the same troubles you had opening this project. How did you solve them?

@dabeschte  I solved that error by going to errors in backend, and a "Select SDK" popped up, I selected 1.8 and building's been working ever since.


Please add water pump support 🙏

Hi,
I've bought a wifi-water pump but it's not yet supported either by you, the official repo or home assistant plugin, can you add this device type please ?
Thanks
{
"active_time":<PHONE_NUMBER>,
"biz_type":0,
"category":"ggq",
"create_time":<PHONE_NUMBER>,
"icon":"smart/icon/ay15662905541765SKtA/473a410865ed1a3b8b4dce342f41a07d.png",
"id": "aaaaaaaaaaa",
"ip": "xxxxxxxxxxxxxxxx",
"lat": "30.30286857361191",
"local_key": "xxxxxxxxxxxxx",
"lon": "120.0639743842656",
"model":"106W",
"name":"Wi-Fi Water Timer with Pump",
"online":true,
"owner_id": "34794909",
"product_id": "yfemiswbgjhddhcf",
"product_name":"Wi-Fi Water Timer with Pump",
"status":[
{
"code":"switch",
"value":false
}
],
"sub": false,
"time_zone": "+08:00",
"uid": "xxxxxxxxxxxxxxxxxxx",
"update_time":<PHONE_NUMBER>,
"uuid": "xxxxxxxxxxxxxxxxxx"
}
{
"category":"ggq",
"devices":[
"aaaaaaaaaaa"
],
"functions":[
{
"code":"switch",
"desc":"switch",
"name":"switch",
"type":"Boolean",
"values":"{}"
}
],
"product_id":"xaabybja"
}

OK, I will add to my todos

Please confirm if it's working.

It's working !! Thanks

I have this message in homebrige logs
[TuyaPlatform] TuyaOpenMQ warning: message received with wrong order.
lastMessage: dataId=ae13981e-8b54-4feb-9822-6f1f6b8d1038, t=1667129372, 1970-01-20T07:05:29.372Z
currentMessage: dataId=64dc7d95-c9a3-4f6b-a078-01cc65abd0fe, t=1667129371, 1970-01-20T07:05:29.371Z

This is a warning, MQTT message not very stable. I've already submit a ticket for this in Tuya Platform. It's not related to water pump.

#32


Memory leak

I trained with raw data,  but memory usage rose from 60% to 70%+ in the first epoch, and it keeps going up .
Whatsmore,I found that the training speed of the new code decreased a lot, may I ask why ?

it's not memory leak, the new IO also caches and prefetchs data  by design.
The speed should be comparable to the old IO.

it's not memory leak, the new IO also caches and prefetchs data by design. The speed should be comparable to the old IO.
When training data more than 2W hours（data type is raw）,  memory usage continues to rise during training.
It feels like an iterative read without releasing the previous data.

it's not memory leak, the new IO also caches and prefetchs data by design. The speed should be comparable to the old IO.

When training data more than 2W hours（data type is raw）, memory usage continues to rise during training.
It feels like an iterative read without releasing the previous data.

I meet the same problem.
When trainning data more than 6 epoches, memory usage continues to rise.


github actions: codefmt.yml

@jschwe just trying a workflow to have the formatting checked.

@jschwe I added isort. The PR is ready to be reviewed.


[MM-36792] Consistent query param names

Summary

Be consistent with query parameter names

Related PRs:

https://github.com/mattermost/mattermost-server/pull/18260
https://github.com/mattermost/mattermost-webapp/pull/8700

Ticket Link

https://mattermost.atlassian.net/browse/MM-36792

Checklist

[ ] Added or updated unit tests (required for all new features)
[ ] Has UI changes
[ ] Includes text changes and localization file updates

Device Information
This PR was tested on:
Screenshots

Release Note

NONE

This looks ok but why the change on the per_page query parameter name?

in short: to make the name consistent with other API calls. This is an off-shoot of a server fix to limit query results on the server side. More details in the linked server PR - https://github.com/mattermost/mattermost-server/pull/18260


Enables batch DDPG agents to be trained.

This PR changes two Q-functions: FCBNLateActionSAQFunction and FCLateActionSAQFunction. A simple search in the ChainerRL repository:
https://github.com/chainer/chainerrl/search?q=FCBNLateActionSAQFunction&unscoped_q=FCBNLateActionSAQFunction
https://github.com/chainer/chainerrl/search?q=FCLateActionSAQFunction&unscoped_q=FCLateActionSAQFunction
shows that the DDPG example is the only place these are used. Furthermore, looking at the __init__ functions for those networks, it appears that neither of them need recurrence. Since
https://github.com/chainer/chainerrl/blob/15d7cbbc8a1fe5b73d8c93cd3fc5cf2c1361f579/chainerrl/links/mlp.py#L16
and
https://github.com/chainer/chainerrl/blob/15d7cbbc8a1fe5b73d8c93cd3fc5cf2c1361f579/chainerrl/links/mlp_bn.py#L31
are not recurrent.


Simon Game

Describe the project

Simon is an electronic memory game with colored buttons. Repeat the sequence of lights and sounds to advance and test your memory skills.
This project will be made using HTML, CSS and Javascript

Functionality

Sequence generationScoring
Button activation
Scoring

I would love to work on this issue. Can you please assign it to me under SSoC '23 ?
Thank You :)

Hey @TusharKesarwani
I would love to work on this issue. Can you please assign it to me under SSoC '23 ?
Thank You :)
It would be more understandable it you could provide some kind of pictures and other stuff for this project.

@patnaikankit games are not allowed


User settings to filter redirect modules in auto-completion

Problem
Presently, we have more than 1000 candidates for modules completion. Almost half are redirect modules and as far as I understand, they are not really useful when you write new playbooks.
Furthermore, although having a lower priority, redirect modules sometimes appear at the top of the list displayed to the user (depending on the sorting done by the client from the search string).
Solution
Add settings to let the user configure the server to provide (or not) redirect modules in auto-completion.
Alternatives
No response
Additional context
For an example of redirect modules at the top of list, you can see the first video (with "foreman" search string) in the previous issue : https://github.com/ansible/ansible-language-server/issues/329

If you're OK with the idea, I can write the solution. I just to know if the new setting must in the generic ansible category (near useFullyQualifiedCollectionNames) or elsewhere.


remove confusing vendor entry in CommonsChunkPlugin example

I found it pretty confusing to have the vendors chunk set to moment when using the CommonsChunkPlugin. Shouldn't the plugin do this for us, so there is no need to add moment explicitly?

On running webpack now, we see that two bundles have been created. If you inspect these though, you will find that the code for moment is present in both the files!

oops, edited wrong code snipped. Reopening ... :)


promptless datmo init

We'd like to use datmo for running model prediction tests in a container. My current solution is to use the yes command.
yes n | datmo init

What's the exact use case over here? and would you like to initialize the project with defaults without passing parameters?

 I think it would make sense to initialize a project with no prompts whatsoever. In our case, we do not need to setup an environment because we have a customized Dockerfile.

Got it! We can add datmo init -f or datmo init --force to initialize without prompts.

Have added the datmo init -f or datmo init --force command to initialize the project without prompts https://github.com/datmo/datmo/pull/267

has been added into the lastest master.


Make line/col always unsigned

Our examples scopes aren't properly sorted right now, but original line/column should always be ascending, and the same for generated line/column.

Thanks! Look like there is a merge-conflict somewhere. For the formatting changes, I used deno fmt and called it a day.

Should be resolved now

I'll have to fix the terser prototype to sort the scopes.


Issue with loading images in the embedded viewer.

Hi,
I am trying to integrate our PACS server with the embedded version of OHIF Viewer. I have got it working till loading the list of studies using WADO and QIDO. However when I double click on a study to open images I am getting the error as shown in attachment 1. I have also attached the index.html file. Can you please help me figure out how to load the images correctly?

index.html.txt
Thanks and congrats on the amazing work being done.

Can you either create a CodeSandbox, GitHub gist, or CodePen with your index.html content and link to it? I'm trying to avoid downloading a file to assist.

We were able to update the library to the latest version and and it now works well through we need to use WADOURI instead of DicomWeb. Thanks for the assistance.


Poor performance using Safari on iOS devices (Low FPS)

When using the app on an iOS device with Safari browser, we get very low FPS count.
Steps to reproduce the behavior:

Enter a room using the Safari browser on an iOS device
See bad performance on peer's screen
Incoming video is smooth as expected on the iOS device itself

Expected behavior
Video should be smooth and performant on all supported devices and platforms
Video
https://user-images.githubusercontent.com/34302982/113510142-f9307180-9561-11eb-81b6-0b62866ea317.mp4
Environment (please complete the following information):

OS: iOS 14.4.1 (on iPad 8)
Browser: Safari 14.4.1
App Version: 0.2.4
SDK Version: 2.13.1
Node.js version: 12.21.0

More info
-I've also tried running the "build" version (non-development) in order to narrow down the possibility of the issue only occurring when using the development code
-No changes were made to the code
-I think this issue is related to the twilio-video repo, but let's make sure
-I've used 2 different networks in order to check, and in the video I use my cellular connectivity, and on the mac I'm on WIFI
-When using https://appr.tc/ I get a smooth 30 FPS at all times (well, not considering the occasional frame drop). Both Safari on mac and on iOS

My colleague @JonnieDev suggested that I run twilio-video quickstart and see if the issue is reproducible there as well:
https://user-images.githubusercontent.com/34302982/113557255-63025700-9606-11eb-8003-691100873d3a.mp4
At the start it's stuck and the FPS is low, but it gets better as time progresses.
Could it be related to simulticast?

Thanks for the issue @Galzk!
I tried to reproduce this issue by connecting to a room (with simulcast enabled) on an iPhone 11, and the framerates seemed reasonable to me. Unfortunately I don't have an iPad that I can try this out on.
Since this issue is reproducible in two different apps, I'll see if I can get this issue transferred to the twilio-video SDK repo. This way the SDK engineers will be able to take a look.
In the meantime, is there a room SID that you can provide?


Look up classes on published apps in another workspace

Will be used by client to look up classes in published apps in another workspace.

close for https://github.com/modal-labs/modal-client/pull/1190


Blurry marker sting produced by threex-arpatternfile.js (pr)

I want to add pull-request to fix blurry marker parts generated by threex-arpatternfile.js (will do it tomorrow if everything is ok with this change).
What is the current behavior?
i visualized marker string onto canvas and found that rotated versions are blurry.
here on screenshot first three small images are channels of unrotated marker image, and then goes rotated 90deg and it was blurry:

What is the expected behavior?

zoomed version:

you can open it, of course :D


Customised URLs

solved #5

#5

@madjaqk By chance do you know enough to check this?

This will allow customized URLs, but:

It requires slugs to be globally unique, which feels wrong (there can only be one my-puzzle-hunt or similar)
It allows the slug to be null, which is the simplest way to add a field to an existing model, but doesn't make sense in this context.  (I believe unique=True and null=True also interact oddly, possibly because NULL != NULL in SQL, but I don't remember the details and could be thinking of some other corner case.  At the very least, without testing, there's a chance that as written this will only allow one Hunt with slug NULL, which will break in prod where multiple hunts already exist.)
This will break all existing URLs
This changes models but doesn't include the new migration.  (I'm not familiar with Heroku or the project's deployment process, maybe it's not required here.)

I think 3 and 4 are easy to fix. We just need to adjust one hunt link for it, and I believe migration is auto-included with any release so the Database stuff willl be resolved without much trouble.
1 is a no go unless we adjust slugs to be 3-myus-hunt where 3 comes your primary key from myus_hunt table or something.
I dont understand 2 to tell what we should do there

For 2, the general solution is to use a data migration to set the value for the new field based on the existing record.  (Django docs, higher-level StackOverflow answer, StackOverflow answer with nitty-gritty details)  I don't know how that interacts with Heroku auto-generating the migrations, though.

As prior work, I believe (teammate's?) Puzzup has URLs of the form /puzzle/<id>/<slug>, where /puzzle/<id> redirects to the slug URL -- though it doesn't actually use the slug for anything, just the ID.
Agreed with jaqk here on writing a migration function. The given migration function is stored in the migration file, and Heroku applies it.
See 1.
Heroku doesn't auto-generate migrations -- it applies the migrations tracked in myus/migrations on each deploy. Migrations should be in version control.

Going to continue in #34 . Thanks @HyoungSooo


[Question]: SQLTableRetrieverQueryEngine does not support streaming in async mode

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
This is the BaseSQLTableQueryEngine which some other engines inherit from
class BaseSQLTableQueryEngine(BaseQueryEngine):
    """Base SQL Table query engine.

    NOTE: Any Text-to-SQL application should be aware that executing
    arbitrary SQL queries can be a security risk. It is recommended to
    take precautions as needed, such as using restricted roles, read-only
    databases, sandboxing, etc.
    """

    def __init__(
        ....

    def _query(self, query_bundle: QueryBundle) -> Response:
        """Answer a query."""
        retrieved_nodes, metadata = self.sql_retriever.retrieve_with_metadata(
            query_bundle
        )

        sql_query_str = metadata["sql_query"]
        if self._synthesize_response:
            partial_synthesis_prompt = self._response_synthesis_prompt.partial_format(
                sql_query=sql_query_str,
            )
            response_synthesizer = get_response_synthesizer(
                llm=self._llm,
                callback_manager=self.callback_manager,
                text_qa_template=partial_synthesis_prompt,
                refine_template=self._refine_synthesis_prompt,
                verbose=self._verbose,
                streaming=self._streaming,
            )
            response = response_synthesizer.synthesize(
                query=query_bundle.query_str,
                nodes=retrieved_nodes,
            )
            cast(Dict, response.metadata).update(metadata)
            return cast(Response, response)
        else:
            response_str = "\n".join([node.node.text for node in retrieved_nodes])
            return Response(response=response_str, metadata=metadata)

    async def _aquery(self, query_bundle: QueryBundle) -> Response:
        """Answer a query."""
        retrieved_nodes, metadata = await self.sql_retriever.aretrieve_with_metadata(
            query_bundle
        )

        sql_query_str = metadata["sql_query"]
        if self._synthesize_response:
            partial_synthesis_prompt = self._response_synthesis_prompt.partial_format(
                sql_query=sql_query_str,
            )

            response_synthesizer = get_response_synthesizer(
                llm=self._llm,
                callback_manager=self.callback_manager,
                text_qa_template=partial_synthesis_prompt,
                refine_template=self._refine_synthesis_prompt,
            )
            response = await response_synthesizer.asynthesize(
                query=query_bundle.query_str,
                nodes=retrieved_nodes,
            )
            cast(Dict, response.metadata).update(metadata)
            return cast(Response, response)
        else:
            response_str = "\n".join([node.node.text for node in retrieved_nodes])
            return Response(response=response_str, metadata=metadata)

I notice when run in async mode (calling aquery instead of query) the streaming mode is not passed on into the response synthesizer causing stuff to break. Adding the streaming gets it to work.
Also, if we are streaming, why are is it casting to Response. (return cast(Response, response)). Shouldn't it be to either a StreamingResponse or an AsyncStreamingResponse?

@dosu Should I make a PR or something then?

@nazim-ashman-oc would love a PR if you have time

@nazim-ashman-oc would love a PR if you have time

@logan-markewich
https://github.com/run-llama/llama_index/pull/15129
First PR to an open source thing, hope its ok.


Correct the capitalization of Xcode in README

This pull request corrects the capitalization of Xcode :sweat_smile:
https://developer.apple.com/xcode/
Created with xcode-readme.

ok

:beers:


chore: modify package name of releases

AsIs
The package name has been low-mid-girl, which is previous one, while proper name of brandings is 6ow3idgirl.
Because of this, names of the artifacts in GitHub Releases has been also wrong ones.
ToDo

[x] update entries of package name in dependency files (Ref)

(Optional) Acceptance criteria

build and deploy will be still successfully done after renaming
confirm GitHub Release name will be fixed after merge of release PR #64

With merging of #65, we found that release-please had raised another release PR for renamed packages, which includes the changes of existing one.
Release PR for renamed package: https://github.com/hwakabh/6ow3idGirl/pull/66


Firefox Horizontal Scrolling w/ Shift

Holding shift and mousewheel scrolling in firefox doesn't work, but it does in Chrome. I know a similar issue was reported in #5 but this one seems like an easy fix in Firefox.

Closed for the inactivity.
Please feel free to reopen.


Optimize - chess/core

Pull Request Test Coverage Report for Build 101

24 of 25 (96.0%)  changed or added relevant lines in 9 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 86.168%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/chess/core/applySpecialActions.js
0
1
0.0%

Totals

Change from base Build 100:
-0.2%

Covered Lines:
265

Relevant Lines:
294

💛  - Coveralls


Implemented steam-library for library theming

Pretty much entirely reworks the plugin-- the code is now significantly more readable, and a lot easier to modify, as well as taking very little effort to change to another color scheme (or fix any errors in the colors that there almost certainly is, because I just hand-typed a ton of RGB values and I am in pain :))))))))
This implements my suggestion (#5), as well as updating the readme to include steam-library, and changing out the image for the cool split up image that most of the implementations have.

I have brain damage I forgor to add the files, one sec


[cloudflare-exporter] Update image tag

Update image tag in chart. It is actually 0.0.7
https://hub.docker.com/r/lablabs/cloudflare_exporter/tags?page=1&ordering=last_updated

@ymrsmns thanks!


[[Fix]] Speed issues with linting very large files

Fixes #2532
linting the example file goes from 60 seconds to 2 seconds with these changes!

Merged to master at commit b33e2d7a9a457733fc76c7af30da3dd8f1aa79a9
Sorry for the delay here. I'm interested in a more formal method for enforcing expectations on performance, but that work is taking longer than expected. I don't want to hold this up any longer, especially because the changeset is so minimal.
Thanks!


Cannot create new folder

There is written, that Creating folder “New Folder“ failed.  File or directory can nott be used.

@fildanovo Could you provide me the console debug log by the steps? https://github.com/yoichiro/chromeos-filesystem-cifs/wiki/How-to-Capture-the-Console-Log-of-DevTools

Here

I have the same problem.
Cannot creat a folder
3221225485: NT_STATUS_INVALID_PARAMETER

I have the same problem.
Cannot creat a folder
3221225485: NT_STATUS_INVALID_PARAMETER

Same here.


After signing-in, we stay in the login screen

The sign-in process seems to work, it queries google for a password and prints User Signed In, but there is no change in screen, we remain in the login screen.

Please see https://github.com/CodePath-Parse/MiAR/pull/25 for this.


Is it possible to add this to bloxstrap's intergrations?

I think it's possible, but only manually, and when starting a new roblox instance a warning will appear (If you want I can add a new setting to disable it)

I think it's possible, but only manually, and when starting a new roblox instance a warning will appear (If you want I can add a new setting to disable it)

Would be cool to have it integrated. Is this a viable method though? As the setup currently already works with bloxstrap.


Add support for Switchbot Lock

Add command to lock/unlock device
:recycle: Current situation
You can only view the status of the Switchbot lock, you cannot control it.
:bulb: Proposed solution
Added the correct command for the lock so that it can be locked/unlocked
:gear: Release Notes
Added support for locking/unlocking the switchbot lock
:heavy_plus_sign: Additional Information
Tested on my Switchbot lock at home.
Reviewer Nudging
Change is not very large, it was just changing the command from "turnOn" -> "lock" and "turnOff" -> "unlock"
Closes #226

closes #381

Where did you find these commands?

Where did you find these commands?

I reverse engineered them using postman and my own Switchbot lock

Please rebase to beta. And I will release a new beta and merge which will publish a new beta.

Done on #382

@Tom-Todd, in the latest beta


Name for Aluminium

Hello,
Your chemical elements webservice is great and fill with tasty data.
I just regret that Aluminium is named "Aluminum or Aluminium". It's not really representing the name and can broke display.

I think you sould state for one or the other.

I chose "Aluminum" and submitted the change to NPM, please update your dependencies. Thank you for bringing this to my attention.


What does ValidatingWebhookConfiguration validate?

What happened:
This is more a question than a bug.
I was trying to figure out what ValidatingWebhookConfiguration (in helm) was validating, and found the following

the generated url in clientConfig didn't work. It looks http://oidc-webhook-authenticator./webhook/validating. Is there something else I need to do before enabling this?
It checks that issuerURL, clientID, and usernameClaim are set. Everything else seems to be optional. This is probably fine, but if I have a provider (an oath2 provider) that requires jwks.keys, it didn't complain. Is that expected?
If we can make sure that id-providers have all the data and the right format. We don't actually need it, correct? Since it requires a caBundle for each instance, it is a little hard to do it automatically in scale.

What you expected to happen:
If it could detect that an oath2 provider requires jwks.keys, it would be great.
How to reproduce it (as minimally and precisely as possible):
deploy ValidatingWebhookConfiguration, and deploy id providers with jwks.keys commented out.
Anything else we need to know:
Environment:

the generated url in clientConfig didn't work.

I am not sure what values are you providing when installing the helm chart. Usually the validating webhook is referenced by its service https://github.com/gardener/oidc-webhook-authenticator/blob/5405bf01f80aff069fad737f0a914b70dcd7b580/charts/oidc-webhook-authenticator/charts/application/templates/validating-webhook-configuration.yaml#L36

It checks that issuerURL, clientID, and usernameClaim are set. Everything else seems to be optional.

You can see what the validating webhook is actually checking directly in its implementation. But basically it performs some sanity checks on openidconnect objects.
https://github.com/gardener/oidc-webhook-authenticator/blob/5405bf01f80aff069fad737f0a914b70dcd7b580/apis/authentication/v1alpha1/openidconnect_webhook.go#L85

If we can make sure that id-providers have all the data and the right format. We don't actually need it, correct?

Well, yes. But this is actually the purpose of the validating webhook - to perform validation on the openidconnect objects.

If it could detect that an oath2 provider requires jwks.keys, it would be great.

How can this be done? Usually OIDC providers are publicly accessible and there is no need to provide their jwks.keys. If there is the need for non-generic validation an additional validating webhook can be implemented and installed to take care of these additional requirements.

Since it requires a caBundle for each instance, it is a little hard to do it automatically in scale.

I am not sure that I get this. Do you mean that it needs a different caBundle for each deployment/installation? I would say that this is pretty common scenario and certificate management/generation is not something specific to this component.


[FLINK-35256][runtime] Fix transform node does not respect type nullability

This closes FLINK-35256.
Flink CDC 3.1.0 brought transform feature, allowing column type / value transformation prior to data routing process. However after the transformation, column type marked as NOT NULL lost their annotation, causing some downstream sinks to fail since they require primary key to be NOT NULL.
Here's the minimum reproducible example about this problem:
source:
  type: mysql
    ...

sink:
  type: starrocks
  name: StarRocks Sink
    ...

pipeline:
  name: Sync MySQL Database to StarRocks
  parallelism: 4

transform:
  source-table: reicigo.\.*
  projection: ID, UPPER(ID) AS UPID

MySQL source table schema definition is as follows:
CREATE TABLE table1 (
  ID VARCHAR(17) NOT NULL,
  ...
  PRIMARY KEY (ID)
);

The underlying reason is Calcite SqlToRelConverter used in DDL parsing removes redundant NOT NULL annotation after conversion. Backfilling this data from pre-transform columns should fix this problem.

I've tested that if namespace of TableId is not set and __namespace_name__ column will be null string, so it won't cause error if MetadataColumns are not null.

@yuxiqian Could you also back port the patch onto release-3.1? Thanks

@PatrickRen Sure! Done in #3277.


Revert 'fix: Fix export of json backup'

Motivation / Background

This Pull Request ...
Details

Additional information

Current dependencies on/for this PR:

main

PR #5278   👈

This stack of pull requests is managed by Graphite.


Invisible blocks?

Running lastest Glowstone++, patched with MysticStone, but It should not make a difference. Also runnung behind bungee.

Looks like a chunk error - are you able to reproduce with vanilla Glowstone++?

Happens only after respawn, maybe related to bungee

Try to eliminate variables; try it without bungee, or without MysticStone. See if not using either of those fixes it.

Above was without both, just after a respawn

Alright, paging doctor @mastercoms
I'm not really sure what to have you check; I'm assuming you've already looked at the logs, so I'll see what he wants us to look at.

Its so strange, maybe glowstone is sending the chunks weird? To early or something?

Hard to say without some more info, just not yet sure what to look for.

Will try to debug the client now..

I have experienced this bug as well. I've noticed if you break a block in that area, it will show the blocks.

So that is what if got, dont know that todo with it thou :D https://gist.github.com/Paulomart/f8aa774c129a1b51b365

That doesn't look too relevant, honestly.. And there's nothing in your server logs?

Just getting #199 sometimes, but in the moment I respawn or something

It's probably related to that, then? Seems likely anyway.

Sometimes chunks get invisible even if they loaded before. (Sorry gif is too big)

Well, maybe rolling through #199 might net us some answers for this one as well - never know.

I can reproduce it 99% on my server, if someone wants to take a look, ping me at gitter

I might have fixed this inadvertently. Could you confirm?

Closing this as I haven't been seeing this bug recently when I had been before. (and bug reporter seems to be inactive :()


fix(go/adbc/driver/flightsql): should use ctx.Err().Error()

As pointed out by @zeroshade here, this should be handled by doing adbc.Error{Msg: ctx.Err(), Code:....}.

@cocoa-xu looks like you need to update some of the python tests which were expecting specific error text results which are being changed by this

Sorry but I didn't find there're any tests (in fact any code) that are expecting any error message that is (or contains) "Cancelled by request" or "Deadline exceeded".
Plus, even if there're some tests that should be updated because of the changes made here, it doesn't make sense that only Python 3.9 (and perhaps Python 3.11) failed on macos-13 instead of all of them (i.e., on macos-latest, ubuntu-latest and windows-latest for both Python 3.9 and 3.11). CI results.
Furthermore, the same test also failed in other CI runs, https://github.com/apache/arrow-adbc/actions/runs/8843931798/job/24285879645#step:14:1213, in PR #1762. It's unlikely that the changes in this PR has caused the above failed CI jobs.

@zeroshade there's something in the Flight SQL client that is treating io.EOF as an actual error, I think


Remove eval code, add eval function

If constant value then provide that value otherwise yield error!
Part of #133

Why add this duplicate ?

I splitted #133 into separate tasks for traceability. Since now we have a burndown chart based on number of issues, splitting such combined issues into separate tasks make our chart closer to actual situation.

What are we going to do now the milestone has passed?

What are we going to do now the milestone has passed?

I would ask ourselves how important this is for us and our stakeholders. Kerem and I have tasks for the coming weeks, and after that we can see what's on our backlog and reprioritize.


Common questions

hi,

Could you inform me of your environment? It seems the version of clang is a little bit old.

@HidenoriMatsubayashi
got similar error while cross building flutter test application
Downloading elinux-x64-release tools...                          1,608ms
💪 Building with sound null safety 💪
Building an eLinux application with wayland backend in release mode for arm64 target...        17.4s
Failed to cmake:
-- Toolchain file defaulted to '/opt/tdx-wayland/5.1.0/sysroots/x86_64-abcsdk-linux/usr/share/cmake/OEToolchainConfig.cmake'
-- The CXX compiler identification is Clang 10.0.1
-- Check for working CXX compiler: /opt/ABC-wayland/5.1.0/sysroots/x86_64-abcsdk-linux/usr/bin/clang++
-- Check for working CXX compiler: /opt/ABC-wayland/5.1.0/sysroots/x86_64-abcsdk-linux/usr/bin/clang++ -- broken
-- Configuring incomplete, errors occurred!
See also "~/test_app/build/elinux/arm64/release/CMakeFiles/CMakeOutput.log".
See also "~/test_app/build/elinux/arm64/release/CMakeFiles/CMakeError.log".
CMake Error at /opt/tdx-wayland/5.1.0/sysroots/x86_64-abcsdk-linux/usr/share/cmake-3.16/Modules/CMakeTestCXXCompiler.cmake:53 (message):
could you help me in fixing it?
my flutter-elinux doctor says
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel unknown, 2.4.0-4.2.pre, on Ubuntu 20.04.1 LTS 5.8.0-63-generic, locale en_IN)
[✓] eLinux toolchain - develop for embedded Linux devices
[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
✗ cmdline-tools component is missing
Run path/to/sdkmanager --install "cmdline-tools;latest"
See https://developer.android.com/studio/command-line for more details.
✗ Android license status unknown.
Run flutter doctor --android-licenses to accept the SDK licenses.
See https://flutter.dev/docs/get-started/install/linux#android-setup for more details.
[✓] Chrome - develop for the web
[✓] Linux toolchain - develop for Linux desktop
[✓] Android Studio (version 4.1)
[✓] Android Studio
[✓] Connected device (4 available)
regsrds
Nagendra

@Nagendra-Bankupalli You try to cross-compile, right? If so, please see https://github.com/sony/flutter-elinux/wiki/Building-flutter-apps#2-cross-building-from-x64-to-arm64

@HidenoriMatsubayashi ,
Yes , the  Link suggests to use tool-chain (sdk) specific to the target . I am using sdk built using yocto for my target , which includes flutter engine as well. the same tool chain worked flawlessly to build flutter-embedder "flutter-client" .

I understood. Is it possible to try the build by specifying only your sysroot without using the Yocto SDK's toolchain? It means
using the host toolchain with the target sysroot.
e.g:
$ flutter-elinux build elinux --target-arch=arm64 --target-sysroot=<absolute_path_to_yocto_sdk_sysroot>

@HidenoriMatsubayashi , below is the error
Building an eLinux application with wayland backend in release mode for arm64 target...        17.5s
Failed to cmake:
-- The CXX compiler identification is Clang 10.0.0
-- Check for working CXX compiler: /usr/bin/clang++
-- Check for working CXX compiler: /usr/bin/clang++ -- broken
-- Configuring incomplete, errors occurred!
See also "/home//test_app/build/elinux/arm64/release/CMakeFiles/CMakeOutput.log".
See also "/home//test_app/build/elinux/arm64/release/CMakeFiles/CMakeError.log".
CMake Error at /usr/share/cmake-3.16/Modules/CMakeTestCXXCompiler.cmake:53 (message):
The C++ compiler
"/usr/bin/clang++"

is not able to compile a simple test program.

Thanks. Can you pass self-build?
$ flutter-elinux build elinux

It works for x64 target (weston) and I can even run profile (dev tool) as well.

Did you install required tools to cross-build?
$ sudo apt install gcc-aarch64-linux-gnu g++-aarch64-linux-gnu
$ sudo apt install libstdc++-8-dev-arm64-cross

No, I thought target tool chain would be sufficient , will try and let you know

Yes, we need these tools to cross-build when we use host's toolchain (clang/llvm).

@HidenoriMatsubayashi , thank you for the inputs,
now the build is success, and able to run the app on target using (from host)
flutter-elinux  run -d
on profiling, I have updated .flutter_custom_devices.json with all required info about my target
but flutter-elinux attach says  (running on another host window)
"Waiting for a connection from Flutter on eLinux..."

Sounds good.
$ flutter-elinux run -d My-device is debug-mode (not profile mode), and currently, custom-devices support only debug mode. If you want to use the observatory, write forwardPort and forwardPortSuccessRegex in .flutter_custom_devices.json. When we attach debugger on remote devices, we need to port-forwarding.
      "runDebug": [
        "ssh"<EMAIL_ADDRESS>"/tmp/${appName}/${appName} -b ."
      ],
      "forwardPort": [
        "ssh", "-o", "ExitOnForwardFailure=yes",
        "-L", "<IP_ADDRESS>:${hostPort}:<IP_ADDRESS>:${devicePort}"<EMAIL_ADDRESS>      ],
      "forwardPortSuccessRegex": "Linux"

Yes, forwardPort and forwardPortSuccessRegex have been added to my flutter_custom_devices.json
According to
https://github.com/sony/flutter-elinux/wiki/Remote-target-devices
below commands can be used to install and uninstall app from the target
flutter-elinux install -d  my-device ,flutter-elinux uninstall -d my-device
in the above commands uninstall and forwardPort does not work for me throwing unknown commands ,
Tried running app on target and shows below line, does this means we can observe it on host, if portForward is success, if yes please let me know how to do it? am I missing anything ?
flutter: Observatory listening on http://<IP_ADDRESS>:41285/Q3dpAt3Ta3g=/

Could you inform me of your trying command and error messages?

with below commands I could get observatory data on host chrome from target (my-device)
target :
root@my-device:/tmp/test_app# ./test_app  -b . flutter: Observatory listening on http://<IP_ADDRESS>:34549/CA1Zk5rr-LQ=/
host
ssh -o ExitOnForwardFailure=yes -L <IP_ADDRESS>:34549:<IP_ADDRESS>:34549 root@ip-my-device
open chrome on host  "http://<IP_ADDRESS>:34549/CA1Zk5rr-LQ=/"

Yes, $ flutter-elinux run -d My-device does the same thing automatically as above.
Internal steps:

[uninstall comamnd] uninstall your old flutter app from My-device
[install command] install your flutter app to My-device
[runDebug command] run your flutter app on My-device
[forwardPort command] portforwarding
You will get log message like "http://<IP_ADDRESS>:34549/CA1Zk5rr-LQ=/" and you can open it on host web-browser

Yes,I expected the same, but the command did not provide any log , so I tried it manually , it worked
between , this is not actual profile data right ? like DevTool ?
host $  flutter-elinux -v run -d my-device
Launching lib/main.dart on eLinux in debug mode...
Uninstall test_app from my-device.
Uninstallation Success
Install test_app (build/elinux/arm64/debug/bundle) to my-device
Installation Success
Launch test_app.name on my-device

`

In my environment, it works fine.
$ flutter-elinux run -d raspberry-pi4
Launching lib/main.dart on eLinux in debug mode...
Uninstall sample from raspberry-pi4.
<EMAIL_ADDRESS>password:
Uninstallation Success
Install sample (build/elinux/arm64/debug/bundle) to raspberry-pi4
<EMAIL_ADDRESS>password:
Installation Success
Launch sample.name on raspberry-pi4
<EMAIL_ADDRESS>password:
<EMAIL_ADDRESS>password:
Syncing files to device eLinux...                                  202ms

Flutter run key commands.
r Hot reload. 🔥🔥🔥
R Hot restart.
h List all available interactive commands.
d Detach (terminate "flutter run" but leave application running).
c Clear the screen
q Quit (terminate the application on the device).

💪 Running with sound null safety 💪

An Observatory debugger and profiler on eLinux is available at: http://<IP_ADDRESS>:43237/xd6sW5hdEoE=/
Activating Dart DevTools...                                      1,830ms
The Flutter DevTools debugger and profiler on eLinux is available at:
http://<IP_ADDRESS>:9100?uri=http://<IP_ADDRESS>:43237/xd6sW5hdEoE=/

I couldn't reproduce this issue. So, could you do debugging by adding log messages to around https://github.com/sony/flutter-elinux/blob/main/lib/elinux_device.dart#L141?

between , this is not actual profile data right ? is profile mode supported in future ?

Why do you want to use profile mode? I think debug mode is well enough. The difference between debug and profile mode is that debug mode is debugging + profiling, whereas profile mode is profiling only.

is profile mode supported in future ?

I added it just now. #34

@HidenoriMatsubayashi ,

Thank you for updating profile mode
Yes, I could get timeline info in debug mode, I observed that the profile mode is more sophisticated than debug mode, let me check again.
Regarding "flutter-elinux run" command, Could be my host settings? I will get back with more details later as it works with manual commands  for me now

@Nagendra-Bankupalli

Yes,I expected the same, but the command did not provide any log (app runs on target successfully), so I tried it manually , it worked

I can reproduce the same issue using i.mx8m device, but if I use raspberry-pi4, it's no problem. So, it looks depending on the target device or environment.

Thanks for updating, Yes mine is similar platform, I did not look into it as it is not blocking.

@HidenoriMatsubayashi , Have you tried flutter-gallery app on embedded target ?

Yes, I have. It worked fine.

@HidenoriMatsubayashi ,
its not working at my end not sure what is the issue, the application loads but no widgets visible , its white , except the flutter animation. any issues with GPU support ?, or how do we know that app is GPU driven? any inputs on this would be helpful.

Proverbially, I guess you haven't set LANG environment var. Could you try the following on your target device?
$ export LANG=en_US.UTF-8

Hi,I created a flutter custom device, but when I entered the command "flutter-elinux devices" I found that instead of one device, two sailfish-devices are displayed. Because of this, when I enter the command "flutter-elinux run -d sailfish-device", debugging does not work correctly for me. What could this be related to?
![Снимок экрана от 2021-11-16 15-40-22](https://user-images.githubuser

content.com/71459422/141989426-ff9d24fb-e65a-48d4-a029-80dff15b9219.png)

Hi,I created a flutter custom device, but when I entered the command "flutter-elinux devices" I found that instead of one device, two sailfish-devices are displayed. Because of this, when I enter the command "flutter-elinux run -d sailfish-device", debugging does not work correctly for me. What could this

@Rustam-Askerov
Your target device (sailfish-device) looks localhost, it means you want to install your host machine instead of remote custom devieces, right?

I run virtualbox on my pc and as a custom device I use a virtual machine on which sailfish os is installed. In the instructions for the Sailfish SDK, it says ssh
the connection to the virual box is done with the command "ssh -p 2222 -i ~ / SailfishOS / vmshare / ssh / private_keys / sdk root @ localhost". So I use localhost.

Also I tried to build the app from x64 host for arm64 using cross compilation but when compilation reaches 100% the error appears:
"/usr/bin/aarch64-linux-gnu-ld: warning: libxkbcommon.so.0, needed for
/home/raskerov/test_app/elinux/flutter/ephemeral/libflutter_elinux_wayland.so, not found
(try using -rpath or -rpath-link)"

flutter-elinux build elinux --debug --target-arch=arm64 --target-sysroot=/opt/poky/2.1/sysroots/aarch64-poky-linux --system-include-directories=/usr/aarch64-linux-gnu/include/c++/10/aarch64-linux-gnu

💪 Building with sound null safety 💪

Building an eLinux application with wayland backend in debug mode for arm64 target...             12,1s
Failed to cmake build:
Scanning dependencies of target flutter_assemble
[  0%] Built target flutter_assemble
Scanning dependencies of target flutter_wrapper_app
[  7%] Building CXX object
flutter/CMakeFiles/flutter_wrapper_app.dir/ephemeral/cpp_client_wrapper/core_implementations.cc.o
[ 15%] Building CXX object
flutter/CMakeFiles/flutter_wrapper_app.dir/ephemeral/cpp_client_wrapper/standard_codec.cc.o
[ 23%] Building CXX object
flutter/CMakeFiles/flutter_wrapper_app.dir/ephemeral/cpp_client_wrapper/flutter_engine.cc.o
[ 30%] Building CXX object
flutter/CMakeFiles/flutter_wrapper_app.dir/ephemeral/cpp_client_wrapper/flutter_view_controller.cc.o
[ 38%] Linking CXX static library libflutter_wrapper_app.a
[ 38%] Built target flutter_wrapper_app
Scanning dependencies of target flutter_wrapper_plugin
[ 46%] Building CXX object
flutter/CMakeFiles/flutter_wrapper_plugin.dir/ephemeral/cpp_client_wrapper/core_implementations.cc.o
[ 53%] Building CXX object
flutter/CMakeFiles/flutter_wrapper_plugin.dir/ephemeral/cpp_client_wrapper/standard_codec.cc.o
[ 61%] Building CXX object
flutter/CMakeFiles/flutter_wrapper_plugin.dir/ephemeral/cpp_client_wrapper/plugin_registrar.cc.o
[ 69%] Linking CXX static library libflutter_wrapper_plugin.a
[ 69%] Built target flutter_wrapper_plugin
Scanning dependencies of target test_app
[ 76%] Building CXX object runner/CMakeFiles/test_app.dir/flutter_window.cc.o
[ 84%] Building CXX object runner/CMakeFiles/test_app.dir/main.cc.o
[ 92%] Building CXX object
runner/CMakeFiles/test_app.dir/__/flutter/generated_plugin_registrant.cc.o
[100%] Linking CXX executable test_app

/usr/bin/aarch64-linux-gnu-ld: warning: libxkbcommon.so.0, needed for
/home/raskerov/test_app/elinux/flutter/ephemeral/libflutter_elinux_wayland.so, not found
(try using -rpath or -rpath-link)
/usr/bin/aarch64-linux-gnu-ld: /opt/poky/2.1/sysroots/aarch64-poky-linux/usr/lib/libEGL.so.1:
.dynsym local symbol at index 3 (>= sh_info of 3)
/usr/bin/aarch64-linux-gnu-ld: /opt/poky/2.1/sysroots/aarch64-poky-linux/usr/lib/libEGL.so.1:
.dynsym local symbol at index 4 (>= sh_info of 3)
/usr/bin/aarch64-linux-gnu-ld: /opt/poky/2.1/sysroots/aarch64-poky-linux/usr/lib/libEGL.so.1:
.dynsym local symbol at index 5 (>= sh_info of 3)
/usr/bin/aarch64-linux-gnu-ld: /opt/poky/2.1/sysroots/aarch64-poky-linux/usr/lib/libEGL.so.1:
.dynsym local symbol at index 6 (>= sh_info of 3)
/usr/bin/aarch64-linux-gnu-ld: /opt/poky/2.1/sysroots/aarch64-poky-linux/usr/lib/libEGL.so.1:
.dynsym local symbol at index 7 (>= sh_info of 3)
/usr/bin/aarch64-linux-gnu-ld: /opt/poky/2.1/sysroots/aarch64-poky-linux/usr/lib/libEGL.so.1:
.dynsym local symbol at index 8 (>= sh_info of 3)
/usr/bin/aarch64-linux-gnu-ld: /opt/poky/2.1/sysroots/aarch64-poky-linux/usr/lib/libEGL.so.1:
.dynsym local symbol at index 9 (>= sh_info of 3)
/usr/bin/aarch64-linux-gnu-ld: /opt/poky/2.1/sysroots/aarch64-poky-linux/usr/lib/libgbm.so.1:
.dynsym local symbol at index 3 (>= sh_info of 3)
/usr/bin/aarch64-linux-gnu-ld: /opt/poky/2.1/sysroots/aarch64-poky-linux/usr/lib/libgbm.so.1:
.dynsym local symbol at index 4 (>= sh_info of 3)
/usr/bin/aarch64-linux-gnu-ld: /opt/poky/2.1/sysroots/aarch64-poky-linux/usr/lib/libgbm.so.1:
.dynsym local symbol at index 5 (>= sh_info of 3)
/usr/bin/aarch64-linux-gnu-ld: /opt/poky/2.1/sysroots/aarch64-poky-linux/usr/lib/libgbm.so.1:
.dynsym local symbol at index 6 (>= sh_info of 3)
/usr/bin/aarch64-linux-gnu-ld: /opt/poky/2.1/sysroots/aarch64-poky-linux/usr/lib/libgbm.so.1:
.dynsym local symbol at index 7 (>= sh_info of 3)
/usr/bin/aarch64-linux-gnu-ld: /opt/poky/2.1/sysroots/aarch64-poky-linux/usr/lib/libgbm.so.1:
.dynsym local symbol at index 8 (>= sh_info of 3)
/usr/bin/aarch64-linux-gnu-ld: /opt/poky/2.1/sysroots/aarch64-poky-linux/usr/lib/libgbm.so.1:
.dynsym local symbol at index 9 (>= sh_info of 3)
/usr/bin/aarch64-linux-gnu-ld:
/home/raskerov/test_app/elinux/flutter/ephemeral/libflutter_elinux_wayland.so: undefined
link to<EMAIL_ADDRESS>/usr/bin/aarch64-linux-gnu-ld:
/home/raskerov/test_app/elinux/flutter/ephemeral/libflutter_elinux_wayland.so: undefined
link to<EMAIL_ADDRESS>/usr/bin/aarch64-linux-gnu-ld:
/home/raskerov/test_app/elinux/flutter/ephemeral/libflutter_elinux_wayland.so: undefined
link to<EMAIL_ADDRESS>/usr/bin/aarch64-linux-gnu-ld: /opt/poky/2.1/sysroots/aarch64-poky-linux/lib/libpthread.so.0:
undefined link to «__libc_dl_error_tsd@GLIBC_PRIVATE»
/usr/bin/aarch64-linux-gnu-ld: /opt/poky/2.1/sysroots/aarch64-poky-linux/lib/libpthread.so.0:
undefined link to «__libc_vfork@GLIBC_PRIVATE»
/usr/bin/aarch64-linux-gnu-ld:
/home/raskerov/test_app/elinux/flutter/ephemeral/libflutter_elinux_wayland.so: undefined
link to<EMAIL_ADDRESS>/usr/bin/aarch64-linux-gnu-ld:
/home/raskerov/test_app/elinux/flutter/ephemeral/libflutter_elinux_wayland.so: undefined
link to<EMAIL_ADDRESS>/usr/bin/aarch64-linux-gnu-ld:
/home/raskerov/test_app/elinux/flutter/ephemeral/libflutter_elinux_wayland.so: undefined
link to<EMAIL_ADDRESS>/usr/bin/aarch64-linux-gnu-ld:
/home/raskerov/test_app/elinux/flutter/ephemeral/libflutter_elinux_wayland.so: undefined
link to<EMAIL_ADDRESS>/usr/bin/aarch64-linux-gnu-ld:
/home/raskerov/test_app/elinux/flutter/ephemeral/libflutter_elinux_wayland.so: undefined
link to<EMAIL_ADDRESS>/usr/bin/aarch64-linux-gnu-ld:
/home/raskerov/test_app/elinux/flutter/ephemeral/libflutter_elinux_wayland.so: undefined
link to<EMAIL_ADDRESS>/usr/bin/aarch64-linux-gnu-ld:
/home/raskerov/test_app/elinux/flutter/ephemeral/libflutter_elinux_wayland.so: undefined
link to «wl_proxy_marshal_constructor_versioned»
/usr/bin/aarch64-linux-gnu-ld:
/home/raskerov/test_app/elinux/flutter/ephemeral/libflutter_elinux_wayland.so: undefined
link to<EMAIL_ADDRESS>/usr/bin/aarch64-linux-gnu-ld:
/home/raskerov/test_app/elinux/flutter/ephemeral/libflutter_elinux_wayland.so: undefined
link to<EMAIL_ADDRESS>/usr/bin/aarch64-linux-gnu-ld:
/home/raskerov/test_app/elinux/flutter/ephemeral/libflutter_elinux_wayland.so: undefined
link to<EMAIL_ADDRESS>/usr/bin/aarch64-linux-gnu-ld:
/home/raskerov/test_app/elinux/flutter/ephemeral/libflutter_elinux_wayland.so: undefined
link to<EMAIL_ADDRESS>clang: error: linker command failed with exit code 1 (use -v to see invocation)
gmake[2]: *** [runner/CMakeFiles/test_app.dir/build.make:136: runner/test_app] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:219: runner/CMakeFiles/test_app.dir/all] Error 2
gmake: *** [Makefile:149: all] Errors 2

But libxkbcommon0 library is installed:

hi,
on host device:
flutter-elinux run -d mixpad-x
Launching lib/main.dart on eLinux in debug mode...
Uninstall hello_remote_device from mixpad-x.
Uninstallation Success
Install hello_remote_device (build/elinux/arm64/debug/bundle) to mixpad-x
Installation Success
Launch hello_remote_device.name on mixpad-x

no flutter hot load
on target device:
./video_player_example -b ./
arm_release_ver of this libmali is 'g2p0-01eac0', rk_so_ver is '3'.embedder.cc (1933): 'FlutterEnginePostDartObject' returned 'kInvalidArguments'. Engine not running.
[ERROR:engine/src/flutter/runtime/dart_vm_data.cc(18)] VM snapshot invalid and could not be inferred from settings.
[ERROR:engine/src/flutter/runtime/dart_vm.cc(256)] Could not setup VM data to bootstrap the VM from.
[ERROR:engine/src/flutter/runtime/dart_vm_lifecycle.cc(84)] Could not create Dart VM instance.
[FATAL:engine/src/flutter/shell/common/shell.cc(297)] Check failed: vm. Must be able to initialize the VM.

Hi, @HidenoriMatsubayashi, i got this when i try to cross-compiling...

it was ok when i run flutter-elinux run -d elinux-x11, thx.

Activating Dart DevTools...                                      1,830ms
The Flutter DevTools debugger and profiler on eLinux is available at:
http://<IP_ADDRESS>:9100?uri=http://<IP_ADDRESS>:43237/xd6sW5hdEoE=/

How can we setup for activating Dart DevTools on your RPi board?


I268 make admin stats page mobile friendly

Change Summary
Adjusted the styling of the admin/index and stats pages to be mobile friendly, and changed the table layout of the statistics.
Change Form
Fill this up (NA if not available). If a certain criteria is not met, can you please give a reason.

[x] The pull request title has an issue number
[x] The change works by "Smoke testing" or quick testing
[ ] The change has tests
[ ] The change has documentation

Other Information
[Is there anything in particular in the review that I should be aware of?]
Related Issue

Resolve #268

Maybe add some padding for the first box to match the second box

They're now the same size on medium and large phones, but the table is larger on laptops so its more readable.


Do you have a timeline to support Windows 2022, 23H2 and 2025?

Tell us about your request
From https://github.com/Azure/karpenter-provider-azure/blob/d83a94caa672fe6f93b7767b7a2c3d928cdfd9a1/pkg/apis/v1alpha2/labels.go#L112-L115, it seems like that karpenter does not support Windows. Do you have any timeline for Windows support?
const (
	Ubuntu2204ImageFamily = "Ubuntu2204"
	AzureLinuxImageFamily = "AzureLinux"
)

Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
Windows support
Are you currently working around this issue?
no.
Additional Context
No response
Attachments
No response
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

cc @comtalyst


nodeBuiltins depend on @types/node

Like non-built-in modules, add a dependency (on @types/node in this case) given the following (unless there's already a dependency, on @types/node or on the specified module):
import * as http from "http";

Otherwise it errors:
error TS2307: Cannot find module 'http' or its corresponding type declarations.

It's been a while since I looked at this code. Can you give a larger example of a project whose publish is wrong before this change and would be fixed by it?

I think @types/standard-engine and @types/undertaker are current examples. This change adds the missing dependency on @types/node to both:
@types/standard-engine
$ npm install @types/standard-engine
$ > index.ts
$ tsc index.ts
node_modules/@types/standard-engine/index.d.ts:7:22 - error TS2307: Cannot find module 'os'.

7 import { type } from 'os';
                       ~~~~

:x:
$ npm install @types/node
$ tsc index.ts

:heavy_check_mark:
@types/undertaker
$ npm install @types/undertaker
$ > index.ts
$ tsc index.ts
node_modules/@types/undertaker/index.d.ts:9:24 - error TS2307: Cannot find module 'stream'.

9 import { Duplex } from "stream";
                         ~~~~~~~~

node_modules/@types/undertaker/index.d.ts:10:30 - error TS2307: Cannot find module 'events'.

10 import { EventEmitter } from "events";
                                ~~~~~~~~

node_modules/@types/undertaker/index.d.ts:25:56 - error TS2503: Cannot find namespace 'NodeJS'.

25         (done: (error?: any) => void): void | Duplex | NodeJS.Process | Promise<never> | any;
                                                          ~~~~~~

:x:
$ npm install @types/node
$ tsc index.ts

:heavy_check_mark:

On second thought, while a dependency on @types/node (or @types/ the specified module) is necessary, it's not sufficient:
$ npm install @types/standard-engine
$ > index.ts
$ tsc --types standard-engine index.ts
node_modules/@types/standard-engine/index.d.ts:7:22 - error TS2307: Cannot find module 'os'.

7 import { type } from 'os';
                       ~~~~

$ npm install @types/node
$ tsc --types standard-engine index.ts
node_modules/@types/standard-engine/index.d.ts:7:22 - error TS2307: Cannot find module 'os'.

7 import { type } from 'os';
                       ~~~~

The types must also /// <reference types="node" />.
I've updated this change to throw an error where before it just added the missing dependency. Linting all files in a package, not just the tests, would also catch these deficiancies?

fyi: fixing import { type } from 'os'; in DefinitelyTyped/DefinitelyTyped#44213 as being result of simple mistype,
kudos to @jablko though for his work!

I opened microsoft/dtslint#286, which is an attempt to address this PR in a more general way: Instead of catching libraries that import a nodeBuiltins module without a /// <reference types="node" /> (or a dependency on @types/ the specified module), that PR attempts to catch libraries that error on import.

I've updated this PR again, this time to add dependencies on @types/${dependency} (just like all other dependencies) instead of @types/node. The @types/${built-in module} packages are introduced in DefinitelyTyped/DefinitelyTyped#44429

The present state of this is an interesting idea, but it depends on DefinitelyTyped/DefinitelyTyped#44429 to split up @types/node into @types/buffer (for example).
Since this PR isn't ready to go yet, and has changed substantially since its creation anyway, can you re-create it at https://github.com/microsoft/DefinitelyTyped-tools? It's the new monorepo that contains all of DT's publishing infrastructure.
I'm going to look at your other PRs and merge+port those that are simple enough and ready to go. I'll comment on any others that I'd like you to recreate the PR at DefinitelyTyped-tools.


Ability to disable playerlist

Is there a way to disable the playerlist in LiveAtlas or is there a way to do that with dynmap that I haven't found yet?
I have checkd the dynmap wiki and issues, but I have only found ways that modify the javascript of the dynmap's javascript code directly.

Not a full fix, but you should be able to comment out the playermarkers section in the dynmap/configuration.txt file and dynmap won't show that layer's information. Dymap and LiveAtlas both still show the player icon/heading, but shows 0 players.
#  - class: org.dynmap.ClientComponent
#    type: playermarkers
#    ...

LiveAtlas will now hide the player list when playermarkers is turned off.


add support for parameter store

I started working on this but it is the first time I've written a Chef resource or really written anything in Ruby.  I've based my code on some of the other resources in this project.  My thought is to put the parameters into environment variables.  Does that make sense?  My code looks like this
action :get do
    request = {
      name: name,
      with_decryption: with_decryption,
    }
    response = ssm_client.get_parameter(request)
    ENV["#{name}"]="#{response.parameter.value}"
    Chef::Log.debug "Get parameter #{name}"
end

If anyone wants to work on this with me please let me know.

@chefbob and @runningman84, how do you see the retrieved Parameter being used? For example, in Parameter Store, I keep some state configuration files to identify similar nodes, but I also keep secrets.
In the case of secrets, I'd want to use them in Chef templates, rather than writing them directly to files--which is what I'd do with the state config files, though.

Yes I would also use them in templates to populate settings like db_user, db_password and so on.
I could also imagine to use it in the recipe, for example to use aws_s3_file with a s3 bucket name from a ssm value.

I agree.  The idea would be to have them available to both templates and recipes.  I've spoken to a few people that the get should be done using a library instead of a resource.  I'm trying to figure out how to do that.

I think this can be closed, since #339 has merged in?


How to obtain model blobs?

I've an issue in the following line of code
model_blob = (model_config["blob"] + "_sh" + str(model_config["shaves"]) +                   ".blob")
Because there is no blob file present in the repository. Where can I obtain these blob files?

Have a look through the readme of the project. In the setup section you'll find that the model blobs can be downloaded from here: https://drive.google.com/file/d/1AUszSCMSc5dCATnZn1jK8PzMMyLQ__5M/view?usp=sharing

Ok thank you very much, it was a silly question since now I've noted the drive link in the README.


[SPARK-33671][SQL] Remove VIEW checks from V1 table commands

What changes were proposed in this pull request?
Remove VIEW checks from the following V1 commands:

SHOW PARTITIONS
TRUNCATE TABLE
LOAD DATA

The checks are performed earlier at:
https://github.com/apache/spark/blob/acc211d2cf0e6ab94f6578e1eb488766fd20fa4e/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala#L885-L889
Why are the changes needed?
To improve code maintenance, and remove dead codes.
Does this PR introduce any user-facing change?
No
How was this patch tested?
By existing test suites like v1/ShowPartitionsSuite.

@imback82 @cloud-fan Please, review this PR.

Test build #132275 has started for PR 30620 at commit 14b6558.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36876/

Test build #132275 has finished for PR 30620 at commit 14b6558.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132275/

jenkins, retest this, please

Test build #132277 has started for PR 30620 at commit 14b6558.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36878/

Test build #132277 has finished for PR 30620 at commit 14b6558.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132277/

Could you give us some pointer for test coverage for LOAD DATA and TRUNCATE TABLE:

LOAD DATA:
https://github.com/apache/spark/blob/acc211d2cf0e6ab94f6578e1eb488766fd20fa4e/sql/core/src/test/scala/org/apache/spark/sql/execution/SQLViewSuite.scala#L176-L179
TRUNCATE TABLE:
https://github.com/apache/spark/blob/acc211d2cf0e6ab94f6578e1eb488766fd20fa4e/sql/core/src/test/scala/org/apache/spark/sql/execution/SQLViewSuite.scala#L180-L183


Use soft linebreaks for constant doc styles

Fixes cases like KeyBindingHandler constant entry in Readline docs.

I've force-pushed changes with @GloverDonovan's suggestion.

Since .entry-const is used in just one place throughout the codebase (src/compiler/crystal/tools/doc/html/type.html) there shouldn't be any regressions. Is it GTG now?

Maybe the title should be updated to reflect what this pull request actually does?

@GloverDonovan I updated the PR title accordingly(?)

@ysbaddaden @jhass @oprypin ready for another review

Maybe constant values shouldn't be shown in the docs.

Values would be fine, it's the chunks of source code that's the problem.

What are values? You mean just show literals like numbers and strings? Maybe that's a good compromise.

@asterite @oprypin do you see any problem with proposed solution? It fixes problem at hand with no downsides. Better options could be discussed in another issue/PR.

I see it as exacerbating the problem. Before, we have an unwanted chunk of code taking up 3 lines, and after, we have it taking up 8 lines.

I think what @oprypin says is exactly the thing. There's a problem: constant values look ugly. So a fix is to make it beautiful. Another fix is to hide them, because they are irrelevant.
I originally included values in constants because of enum members, to know their values (I don't know why, though). So instead of rushing and "fixing" this we are taking some time to discuss what's the real problem and what a good solution might be.

@asterite I strongly disagree on the value of constants being irrelevant for API docs. They are certainly not always meaningful, but on many occassions it's super useful to know. Especially when it's just a plain literal:
MAX_CODEPOINT = 0x10FFFF

It wouldn't hurt to have some lines about the details alongside, but you shouldn't need to wrap that plain value in prose.
If it's more complicated than that, for sure, there is no point in printing all this:
STDERR = (IO::FileDescriptor.new(2, blocking: (LibC.isatty(2)) == 0)).tap do |f| f.flush_on_newline = true end

Or the Readline example mentioned in the OP.
In such cases, it is essential to have a doc string describing all the details about this constant, it's value and usage. Perhaps it would be nice to add the inferred type alongside the name, but nothing else.

@asterite Before now there was no discussion about constants being irrelevant in docs. This PR fixes only the presentation layer (you have new lines - then show 'em!). ATM we have a mess - we see the values, yet we see them garbled. How bad is fixing this?

From https://github.com/crystal-lang/crystal/pull/6118#issuecomment-391318938 by @asterite:

This is an improvement over the old code. It might not fix all problems, but that's how we move in Crystal, fixing things slowly.

Shall we use that approach, merge this and discuss future improvements in another issue/pr?

Actually, what we can do is to change the doc generator to only show simple constants, like numbers, booleans, chars and strings. Everything else could be hidden.

@asterite Personally I'd find it counter-intuitive and misleading. Anyhow, let's discuss this in a new issue/pr please.

@Sija Do you find the value of STDOUT useful?
We can always show a "show source" that leads to where the constant is defined, and there you can see the value.

Personally I'd find it counter-intuitive and misleading. Anyhow, let's discuss this in a new issue/pr please.

It's impossible to track if this is the right change, it's just CSS so lets merge it and see if anything breaks with existing <code> blocks.


Agree build sprint schedule

Hi all, this is what I was thinking for our structure over the next 3 weeks:

13th to 19th - Front End Sprint (4.5 days) - potential user testing on afternoon of 19th
20th to 27th - Back End Sprint (6 days)
28th to 2nd - Debugging and Stretch Goals sprint (3 days)
3rd December - Handover

I thought it best to get the front end done first as we can then review the overall style and actions with Elysabeth (and potentially users).
We would then have more time designated to do the back end, with a final few days left over as contingency for further work and debugging.
What do people think? @arrested-developer - it'd be great to see if you feel this is sensible too :)

I think this looks pretty sensible 👍🏼
Time does feel a bit tight, but I'd be tempted to try and get the front-end sprint done more quickly. If you've got user testing on 19th, you could aim to have front-end fully functional by end of 18th, then you have 1/2 day contingency if needed, which can ideally be used to refactor the frontend and bring everything together.
I think having the last 3 days as contingency time is a great idea - hopefully we'll have thorough tests and no need for extra bug fixing 🐛so you can use this time to refactor.
Out of interest, what time would you draw on if the user testing on 19th shows a lot of changes are needed?

that's good :+1:

I like that, agree :+1:

yeah as discussed this afternoon this setup makes a lot of sense and will enable us to work slightly more independently. I also think the timings should work as they are set out by @thejoefriel

That's good :+1:
Did you add this user story to the core sprint?

Export results (email / pdf download)

If yes, I should add it to user stories.

Export will be in core+ so the second sprint. Will add it though so we don't forget! Thanks!


Sharded row index for Tabular dataset (MLDB-1762)

Also fixes issues with row name generation when offset and limit aren't at their default values.
This reduces by about 80% the time elapsed creating the row index of a large tabular dataset post-import.

+1


Boards filters are removed on refresh

To reproduce:

go to board
filter by category (url should display the current filter)
refresh

The url should be back to not having a filter. This is wrong, the filter should stay.

Fixed by https://github.com/BobaBoard/boba-frontend/pull/96, which however needs some further debugging. Currently paused and up for grabs.


Force the start of qemu-guest-agent service

What does this PR change?
Some VMs are not deployed on my main.tf, showing this error:
Error: Error: couldn't retrieve IP address of domain.Please check following:
1) is the domain running proplerly?
2) has the network interface an IP address?
3) Networking issues on your libvirt setup?
 4) is DHCP enabled on this Domain's network?
5) if you use bridge network, the domain should have the pkg qemu-agent installed
IMPORTANT: This error is not a terraform libvirt-provider error, but an error caused by your KVM/libvirt infrastructure configuration/setup
 timeout while waiting for state to become 'all-addresses-obtained' (last state: 'waiting-addresses', timeout: 5m0s)

  on /home/jenkins/jenkins-build/workspace/manager-4.0-qam-setup-cucumber/results/sumaform/backend_modules/libvirt/host/main.tf line 65, in resource "libvirt_domain" "domain":
  65: resource "libvirt_domain" "domain" {

VMs not deployed:
sles15-sshminion
ubuntu1604-sshminion
ubuntu2004-minion
controller

Might it fix the issue?
I tried from the workspace of my Jenkins job, and apparently didn't work, but I want to share it with you to discuss it.

Please try logging in one of those non-responding VMs from their host via virsh console, then inspect the system.
Is an ethernet interface present, does it get the expected IP address? Is qemu-guest-agent installed? Is the corresponding service running?
Depending on the answers to these questions fix could be different (also on a case-by-case basis).
Feel free to post observations here for more guidance!
Thanks for looking into this.

Eric did the proper fix for that.


operator [O] keycloak-operator (22.0.4)

/merge possible

/merge possible

/merge possible

/merge possible


Disable or reduce log level

Hi! I have installed Lens App 6.0.1 and running it on a Fedora 35 with gnome desktop.
I have noticed that it is generating a lot of logs entries on messages files. It is all the time writing logs with information that I think is not necessary. For example, is logging all the time that I access to a cluster, or open a pod to check information, etc.
Below you can find some examples of logs:
Sep 15 11:18:03 computer01 lens.desktop[4061571]: info:    ┏ [CLUSTER]: refresh +26s
Sep 15 11:18:03 computer01 lens.desktop[4061571]: info:    ┃ [1] {
Sep 15 11:18:03 computer01 lens.desktop[4061571]: info:    ┃ [2]   id: '1fd3838f862addc1eeb5ce615849c708',
Sep 15 11:18:03 computer01 lens.desktop[4061571]: info:    ┃ [3]   name: 'mycluster',
Sep 15 11:18:03 computer01 lens.desktop[4061571]: info:    ┃ [4]   ready: true,
Sep 15 11:18:03 computer01 lens.desktop[4061571]: info:    ┃ [5]   online: true,
Sep 15 11:18:03 computer01 lens.desktop[4061571]: info:    ┃ [6]   accessible: true,
Sep 15 11:18:03 computer01 lens.desktop[4061571]: info:    ┃ [7]   disconnected: false
Sep 15 11:18:03 computer01 lens.desktop[4061571]: info:    ┗ [8] }
Sep 15 11:18:33 computer01 lens.desktop[4061571]: info:    ┏ [CLUSTER]: refresh +30s
Sep 15 11:18:33 computer01 lens.desktop[4061571]: info:    ┃ [1] {
Sep 15 11:18:33 computer01 lens.desktop[4061571]: info:    ┃ [2]   id: '1fd3838f862addc1eeb5ce615849c708',
Sep 15 11:18:33 computer01 lens.desktop[4061571]: info:    ┃ [3]   name: 'mycluster',
Sep 15 11:18:33 computer01 lens.desktop[4061571]: info:    ┃ [4]   ready: true,
Sep 15 11:18:33 computer01 lens.desktop[4061571]: info:    ┃ [5]   online: true,
Sep 15 11:18:33 computer01 lens.desktop[4061571]: info:    ┃ [6]   accessible: true,
Sep 15 11:18:33 computer01 lens.desktop[4061571]: info:    ┃ [7]   disconnected: false
Sep 15 11:18:33 computer01 lens.desktop[4061571]: info:    ┗ [8] }
Sep 15 11:18:37 computer01 lens.desktop[4061571]: warn:    ┏ [METRICS-ROUTE]: failed to get metrics for clusterId=1fd3838f862addc1eeb5ce615849c708: Metrics not available +4s
Sep 15 11:18:37 computer01 lens.desktop[4061571]: warn:    ┃ [1] Error: Metrics not available
Sep 15 11:18:37 computer01 lens.desktop[4061571]: warn:    ┃ [2]     at /opt/Lens/resources/app.asar/static/build/main.js:1:404041
Sep 15 11:18:37 computer01 lens.desktop[4061571]: warn:    ┃ [3]     at runMicrotasks (<anonymous>)
Sep 15 11:18:37 computer01 lens.desktop[4061571]: warn:    ┃ [4]     at processTicksAndRejections (node:internal/process/task_queues:96:5)
Sep 15 11:18:37 computer01 lens.desktop[4061571]: warn:    ┃ [5]     at async Promise.all (index 0)
Sep 15 11:18:37 computer01 lens.desktop[4061571]: warn:    ┃ [6]     at async /opt/Lens/resources/app.asar/static/build/main.js:1:404789
Sep 15 11:18:37 computer01 lens.desktop[4061571]: warn:    ┃ [7]     at async Object.route (/opt/Lens/resources/app.asar/static/build/main.js:1:396830)
Sep 15 11:18:37 computer01 lens.desktop[4061571]: warn:    ┗ [8]     at async c.route (/opt/Lens/resources/app.asar/static/build/main.js:1:396518)
Sep 15 11:19:03 computer01 lens.desktop[4061571]: info:    ┏ [CLUSTER]: refresh +26s
Sep 15 11:19:03 computer01 lens.desktop[4061571]: info:    ┃ [1] {
Sep 15 11:19:03 computer01 lens.desktop[4061571]: info:    ┃ [2]   id: '1fd3838f862addc1eeb5ce615849c708',
Sep 15 11:19:03 computer01 lens.desktop[4061571]: info:    ┃ [3]   name: 'mycluster',
Sep 15 11:19:03 computer01 lens.desktop[4061571]: info:    ┃ [4]   ready: true,
Sep 15 11:19:03 computer01 lens.desktop[4061571]: info:    ┃ [5]   online: true,
Sep 15 11:19:03 computer01 lens.desktop[4061571]: info:    ┃ [6]   accessible: true,
Sep 15 11:19:03 computer01 lens.desktop[4061571]: info:    ┃ [7]   disconnected: false
Sep 15 11:19:03 computer01 lens.desktop[4061571]: info:    ┗ [8] }

Most of the messages are info level, so it's not necessary. Its flooding the messages files with lens logs and is impossible to debug other issues due to this.
is it possible to disable log or at least change log level?
Thanks!

You can set the LOG_LEVEL env var to warn or even error if you would like.


Increase pageSize

At least for the get gender call

Maybe for dynamic start small and then make a judgement on size based on num of pages