Spacings design tokens

Duplicates ❌

[X] I have searched the existing issues

Which Figma library is concerned?
Not related to one Figma library
Summary 💡
Define and fill in the tokens allowing to manage the spacings
Dependencies 📦
No dependencies.
Examples 🌈

nordhealth.design
Polaris Design system

Motivation 🔦
I need to be able to use spacings variables quickly, both in foundations and in a theme.
New 🆕

[x] Validated in a grooming session

Backlog 📋

[x] Has a good summary
[x] Has example link(s)

Design in progress 🏗

[x] Analysis (functional & technical)
[x] Design refinement
[x] Dev qualification (properties, breakpoints, tokens etc.)
[ ] Documentation refinement (inside the Figma branch)

Design review 👀

[ ] Figma branch merged
[ ] Documentation review _(and move the documentation into Vitamin Documentation Figma file)

Ready to dev 👍

[ ] Issues are created in all repositories affected (vitamin-<web,android,ios,compose>)

Dev in progress 💻

[ ] All issues are resolved (to replace with issues links)

This topic has been launched in beta test.
However, its status in the board no longer reflects the current reality, so it has been moved to our "New" column.
We will keep you informed of the different advances.
Thank you.


Directive naming conventions need to be clearer

I haven't found specific documentations stating EXPLICITLY (and boldly) that directives MUST be named in camelCase, I spent a long time trying to figure out why a particular directive was not working. It'd be nice to see this mentioned very clearly, at the very start of the documentation on controllers (I was reading https://code.angularjs.org/1.3.15/docs/guide/directive
In Angular 1.3.15 I can't name a directive TestDirective. It had to be named testDirective to work.
Please let me know if I am in error.

You mean https://code.angularjs.org/1.3.15/docs/guide/directive#normalization ?

I don't believe "We typically refer to directives by their case-sensitive camelCase normalized name" is the same as "We CAN ONLY refer to directives by their case-sensitive camelCase normalized name"...


The source of the MERR dataset

Hello author, I would like to ask, is the raw data of MERR you made from a combination of existing emotion datasets, or is it downloaded and filtered from the web?Besides, if you have any plans to release MERR dataset?(includes the raw data)

The MERR dataset is curated from the unlabeled video samples of the MER2023 dataset. The MERR  dataset includes only the annotations, not the original videos. If you need the source videos, you will need to apply for access from the official MER2023  dataset.


Increase the tuple length for implementing HeapSizeOf

#57 implements HeapSizeOf for tuples of length up to 5. We should increase this limit to 12, which is (e.g.) the limit on tuple Clone in stdlib.

These impls could be generated with a macro, similar to https://github.com/rust-lang/rust/blob/master/src/libcore/tuple.rs


Fix for RL <IP_ADDRESS> and various added features

I've fixed the plugin to work with the latest runelite version at this moment in time.
I've added a cached threadpool so you won't run into socket errors if you call the endpoint too often.
I've added a configurable port in case 8081 is already occupied.
I've added an idle bool to the events endpoint.
I've added special attack energy to the events endpoint.
I've updated the build file to use runelite.latest rather than a set version.
I've removed skill.overall from everything as it's deprecated.
If it turns out this project is still active i will add a lot more features

How can I add your updated version of the plugin to my RuneLite? Do I have to use IntelliJ IDEA?

@SGibson865 either wait for @MorgApps to approve the pull request or use intellij

@Christley Thanks for the updates merged them :)


Heartbeat grace period not honored

Setup: one node (both manager and agent).
When restarting the node, every single task gets re-scheduled (on itself).
IIRC we have a "grace period heartbeat" for manager re-election and restart. Should this be taken into account?
/cc @LK4D4

Thanks I have pretty much been on host acct or Vie versa so all quits together


Added --downgrade option to otiotool.

This PR adds a --downgrade option to otiotool to make it easy to write out an older OTIO schema.
From the usage statement:
  --downgrade FAMILY:VERSION
                        Downgrade OTIO schema. Only relevant when --output is used to output an OTIO file. FAMILY:VERSION specifies which
                        schema family and version to use. If FAMILY: is omitted, the default OTIO_CORE: is used. For example `--downgrade
                        OTIO_CORE:0.14.0` is equivalent to `--downgrade 0.14.0`. See the OpenTimelineIO documentation for
                        OTIO_DEFAULT_TARGET_VERSION_FAMILY_LABEL for details.

Example:
otiotool -i new_file.otio --downgrade 0.14.0 -o old_file.otio

It uses the downgrade mechanism documented here.

Could we get an option to dump the known families/versions?  I think the API is there to query that.

(apologies for the close/open, misclick)

Codecov Report

Merging #1485 (ef02f86) into main (33e0d1a) will increase coverage by 0.09%.
The diff coverage is 16.66%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1485      +/-   ##
==========================================
+ Coverage   80.09%   80.18%   +0.09%
==========================================
  Files         198      199       +1
  Lines       21703    22048     +345
  Branches     4298     4452     +154
==========================================
+ Hits        17382    17679     +297
- Misses       2174     2199      +25
- Partials     2147     2170      +23

Flag
Coverage Δ

py-unittests
80.17% <16.66%> (+0.08%)
:arrow_up:

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

Impacted Files
Coverage Δ

...-opentimelineio/opentimelineio/console/otiotool.py
72.67% <16.66%> (-0.64%)
:arrow_down:

src/opentimelineio/composable.cpp
50.00% <0.00%> (-6.67%)
:arrow_down:

src/opentimelineio/timeline.h
87.50% <0.00%> (-4.81%)
:arrow_down:

...lineio_contrib/adapters/tests/test_fcpx_adapter.py
96.03% <0.00%> (-1.15%)
:arrow_down:

...elineio_contrib/adapters/tests/test_aaf_adapter.py
97.83% <0.00%> (-0.01%)
:arrow_down:

contrib/opentimelineio_contrib/__init__.py
100.00% <0.00%> (ø)

src/py-opentimelineio/opentimelineio/__init__.py
100.00% <0.00%> (ø)

...y-opentimelineio/opentimelineio/schema/__init__.py
100.00% <0.00%> (ø)

...aselines/plugin_module/otio_mockplugin/__init__.py
100.00% <0.00%> (ø)

... and 3 more

Continue to review full report at Codecov.

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

@ssteinbach I added --list-versions which prints this:
❯ otiotool  --list-versions
Available versions for --downgrade FAMILY:VERSION
  OTIO_CORE:0.14.0
  OTIO_CORE:0.15.0
  OTIO_CORE:0.16.0.dev1

Thanks for the feedback @JeanChristopheMorinPerso. I think I addressed all of your comments.


Atomic swap smart contracts must check secret size.

Your cross-chain atomic swaps appear to be using the canonical form of smart contracts, which can make them vulnerable to an attack that I describe here:
https://gist.github.com/markblundeberg/7a932c98179de2190049f5823907c016
If Alice is providing the secret, then the smart contract created by Bob must verify the size of the secret or else he is vulnerable to having his money stolen.

Thank you for bringing this to our attention.  You are certainly correct that fraud could be performed between two currencies that have different maximum script push sizes, and that placing and verifying the actual length in the contract allows it to be audited.
We will make the appropriate changes to add the extra validation, and not accept any implementations for coins with a different maximum script size until this is resolved.  Just to reduce complexity, I think only working with a single known good secret size is best, instead of trying to support anything under the maximum.
If you haven't yet reported it yet, please bring this to the attention of the BIP0199 authors, as their example script is similarly vulnerable and the BIP will be widely referenced by implementors.

Thanks for the fast response!
I agree with what you say --- the simplest fix is that all contracts should be checked for standard form that includes a hardcoded secret size check of 32 bytes. No variation allowed.
I've sent an email to the authors you suggested, cheers!


Strings do not have an offset where they are referenced?

I just don't find such a thing. I think it is quite useful as that of a class or method. Can we add that?

yep that sounds good. Xrefs are a little bit unstructured right now...

Yes. I noticed some new sections were added recently.

In the current master, there is now a switch to get the offset of the string usage: https://androguard.readthedocs.io/en/latest/api/androguard.core.analysis.html#androguard.core.analysis.analysis.StringAnalysis.get_xref_from


Improve experience of installation

make download script more friendly

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 53.25%. Comparing base (48819eb) to head (2e79f3c).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #64   +/-   ##
=======================================
  Coverage   53.25%   53.25%
=======================================
  Files          42       42
  Lines        4792     4792
=======================================
  Hits         2552     2552
  Misses       2009     2009
  Partials      231      231

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Main page graphics have white artifacts around the edges.

This should be transparent rather than baked in white.

Also, the Facebook and Twitter share buttons:

Fixed with https://github.com/NuGet/NuGetGallery/pull/4206.


Add apply filters btn

Description
Remove watchers on filters, add an apply button instead
Changes

remove watchers for filter inputs in nodes, farms & dedicated nodes
add applyFilters function for button

Related Issues
#1775

make 'clear' button at right and color="anchor"
apply color="secondary"
space between buttons maybe mr-2

make 'clear' button at left and color="anchor"
Apply button color="secondary"
space between buttons maybe mr-2

make 'clear' button at left and color="anchor"
Apply button color="secondary"
space between buttons maybe mr-2
applied

@amiraabouhadid make clear button at left and apply at right

@amiraabouhadid make clear button at left and apply at right

perfect thanks

I think the buttons will look better at the left. What do u think ?

I think the buttons will look better at the left. What do u think ?

Agree100%

Here i chose offline nodes, then i wanted to filter on them.
but the filters applied on all nodes not offline .

Screencast from 01-01-2024 03:04:36 PM.webm

The initial value is up in the Nodes page, it should contain all nodes (up, down, and standby)

Initially the node status on nodes pages shouldn't be up, All nodes should be displayed

When i reset filters, I expect the nodes status to be reset also.

Screencast from 01-02-2024 11:30:00 AM.webm

clear btn is still loading please check it out

please check network on each click will send 2 requests

clear button shouldn't be enabled until user starts to filter

when i filter after choosing node status, the node status is cleared.

Screencast from 01-02-2024 03:16:18 PM.webm

When enabling the GPU or Gateway options and then pressing on apply button, the switches go back to being false
Screencast from 01-03-2024 09:45:50 AM.webm

The GPU option in the Dedicated Nodes screen doesn't work as expected.
Screencast from 01-03-2024 09:50:24 AM.webm

When filtering with values e.g. Node ID, when I want to display all of the nodes the button of Apply gets disabled and I can't go back.
Screencast from 01-03-2024 09:54:53 AM.webm

The GPU option in the Dedicated Nodes screen doesn't work as expected.
Screencast.from.01-03-2<IP_ADDRESS>.AM.webm
No GPU card on the node 148 as seen in the screen

The resolution of this comment introduces a new concern. The existing functionality for the grid proxy's GPU filter is set to "True," indicating a requirement for GPU, while "False" signifies the absence of this requirement. Consequently, when the switch is set to "False," it implies the necessity to filter all nodes with GPU filter set to "False." Consequently, nodes without GPUs will not be returned.
Refer to the Screencast from 01-03-2024 12:51:37 PM.webm for more clarity.
To address this issue, you have two potential solutions:

Rectify the situation in the grid proxy.
Resolve it by substituting the "False" keyword with "undefined" :D


Adding configuration for datacollector for chef server

Signed-off-by: Yashvi Jain<EMAIL_ADDRESS>:nut_and_bolt: Description: What code changed, and why?
Added the datacollector property configuration for embedded chef server and automate HA chef server. Now the user can use below config for adding these values in chef server along with the existing ones.
[erchef.v1.sys.data_collector]
pool_max_age=90 //In seconds
pool_cull_interval=3 //It will take this number in minutes
max_connection_duration=20 //it will also be in seconds
ibrowse_timeout=80000

:chains: Related Resources
https://chefio.atlassian.net/browse/STALWART-265
:+1: Definition of Done
Configs can be patched using the config file
:athletic_shoe: How to Build and Test the Change
rebuild components/automate-deployment/
rebuild components/automate-cli/
rebuild components/automate-cs-oc-erchef/

Add the below config in a file
 [erchef.v1.sys.data_collector]
   pool_max_age=90
   pool_cull_interval=3
   max_connection_duration=20
   ibrowse_timeout=80000

patch the above config and changes will be reflected
:white_check_mark: Checklist
All PRs must tick these:

[X] I have read the CONTRIBUTING document.
[X] All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

[X] Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
[X] Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
[X] Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
[X] Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
[X] Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

[X] Tests added/updated? (all new code needs new tests)
[X] Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.
:camera: Screenshots, if applicable

Default Values in chef-automate HA

After configuration patched.


Improve serialization efficiency of replicated Cache and CachedValue

The hz event handler itself was triggering more serialization!  Skip it, unless we are going to do something with it.

Looks great! 👍


In order to use ofbx::IScene ptr's with standard c++ lifetime constru…

…cts like std::unique_ptr or std::shared_ptr, it has to have a public destructor.

Already in master.


[BUG] Tag with underscores is styled as two separate tags

Describe the bug

To Reproduce

Type #valid_tag_with_underscores in your note
Observe how it is styled as two tags one after another: #valid_tag_with_ and #underscores

Expected behavior
Tag styled as as a single one: #valid_tag_with_underscores
Screenshots

Version [Desktop or Mobile]:

Obsidian macOS v0.13.23 from installer v0.13.19
Sanctum v0.6.2

Following Style Settings for Sanctum:
{
  "sanctum@@light-mode-palette": "sanctum-light",
  "sanctum@@dark-mode-palette": "sanctum-coldgrey-d",
  "sanctum@@head-indicator": true,
  "sanctum@@head-indicator-active": false,
  "sanctum@@heading-1": "2.5rem",
  "sanctum@@heading-2": "2.0rem",
  "sanctum@@heading-3": "1.5rem",
  "sanctum@@heading-4": "1.25rem",
  "sanctum@@heading-5": "1.00rem",
  "sanctum@@dim-panes": false,
  "sanctum@@tasks": true,
  "sanctum@@hr-options": "default",
  "sanctum@@edit-mode-tasks": true,
  "sanctum@@table-options": "default",
  "sanctum@@alt-link": false,
  "sanctum@@no-titlebar-title": true,
  "sanctum@@show-yaml": false,
  "sanctum@@topside-status-bar": true,
  "sanctum@@compact-outline": true,
  "sanctum@@note-title-left": true,
  "sanctum@@show-vault-name": false,
  "sanctum@@left-nav-arrows": true,
  "sanctum@@settings": 60,
  "sanctum@@community-theme": 90,
  "sanctum@@community-plugin": 90,
  "sanctum@@no-blur": false,
  "sanctum@@no-sanctum-icons": false
}

Editor
Bug can be observed in both legacy (the one I use on a daily basis) and new editors in the editing mode. Not present in the red mode.

Now I realised the same happens for a default Obsidian theme too, it was just not clearly apparent. On the screenshot below you can see how tag is broken into two parts (I have added some CSS to make it visible). My question is then: where to report it?

So I have noticed! You can report it in the obsidian forums, although I've made somewhat of a fix for it.

Now I again think it doesn't have to be an issue on the Obsidian side. Why? Because the last HTML element of the tag contains extra cm-hashtag-end class which could allow to improve CSS styling to capture the whole tag, I suppose.
Also, I see it's styled better in "Things" theme:

although I've made somewhat of a fix for it.

I see it now! Thanks for your commit and release! I updated locally and it works great now ❤


Accuracy and Range Limits

Can you please describe how much is Range limit of measured distance?
The original laser meter is marked as 40m, but what about the new firmware?
Can it measure distances more than 40m by prism reflectors?
Thanks

Modulation frequencies are selected so that max range is 37.5m. If you need more, you have to change firmware.
I'm not sure that you will get enough signal at such range with my firmware - it was designed for high speed measurements.
Prism reflector or just reflective plate could probably help.

Thanks for information. The phase difference method for calculating the distance is also used on equipment named Total Station. In the advertisement they say in each KM with their own special prism there is a few millimeters error.

What method do they use to first get a KM range, and also such good accuracy of mm?
Thanks heaps

As I know, total stations could use low frequency (1-10 MHz) modulation for initial (not accurate) big distance measurement.
After that, station is switched to high frequency mode to increase resolution.
Of course, total station is based on high complex electronics, and have a lot of calibrations to get high precision.


Provide argument migrate reset so that it gets used by prisma db seed

Discussed in https://github.com/prisma/prisma/discussions/21173

Originally posted by LamprosPitsillos September 21, 2023
Question
https://www.prisma.io/docs/guides/migrate/seed-database#user-defined-arguments
I use as input to my seed a json, that is passed in as an argument to the ts file.
How do i pass this to prisma migrate reset  so that it gets used by seed ?
❯ npx prisma db seed ./STjsons/test_objects_used_by_methods.json
❯ npx prisma migrate reset ./STjsons/test_objects_used_by_methods.json # doesnt work

Feature Request :
Have migrate reset propagate arguments to underlying commands such as prisma db seed

@LamprosPitsillos , Could you please explain this a bit. I'm happy to contribute :)

Of course, it's really simple.
prisma db seed  takes in arguments and passes them to the seed script.
prisma migrate reset calls seed function after resetting but takes no arguments to pass to seed.
So prisma db seed "my input file json" that seeds my database can't be called after reset because input file can't be passed

@dhayaleswaran migrate reset CLI code needs to be changed to also read any arbitrary parameters and arguments at the end, and then those need to be forwarded to where the CLI then executes db seed internally.

Thanks @LamprosPitsillos @janpio , I'll get my hands dirty in this repo first I may come here for further expectations. This is going to be my first open source contrib.

@janpio ,
I had test runs to understand the things going on. Based on that the expectation for the prisma migrate reset is to wipe out the tables from the database and reapply the migrations along with the seed.
But in actual the tables were dropped but the migrations are not applying as expected which prevents seed from executing.
So I need to fix the things between dropping and re-migrating the tables.
Can you confirm that the expectations assumed by me is correct. I just need your thoughts on this one before proceeding as I'm new to this repo.

I am pretty sure migrate reset works fine and can apply db seed - otherwise we would have hundreds of issues about this problem. You need to create a migration first with migrate dev so there is a migration history.

Here is what happens when I'm attempting to reset the db after pushing the migration files to db.
Without seed files, the reset happens as expected. but we won't have any migrations applied to the db and we can receive the seed command executed log in the terminal.
When seed file is available the script is attempting to run the seed without applying migrations that's the problem i think.

What migrations do you have in this project in prisma/migrations?

Sorry @janpio It was my mistake. I used db-push which skips the generation of migration. Now everything works as expected.
I need to make migrate reset to read the parameters (I'm assuming like --seed-files ) and use them when seeding. Is this the expectaion ?

@janpio , I made the required changes but I'm not able to push it to this repo. Can you help me on this ?

I think you figured it out, you need to fork and then send a pull request.
Let's continue the discussion there: https://github.com/prisma/prisma/pull/21271


Windows: Can't find datasets file, despite setting full path and dataset path in Preferences

[X] Did you get wrong results?

Impact

[X] Low - User productivity trivially degraded

To Reproduce

Go to 'Open Session' in Windows version of Vapor
Select file in or out the dataset path folder (no difference)
See error
Import WRF-ARF dataset manually loads the file

Expected behavior
Vapor should search the path declared in the session file and in the dataset path.
Attachments

attached screenshot relevant to understanding the issue.

Desktop

OS and version: Windows 11
Version: 3.7.0.

Please note that Windows slash is backwards, and not forward as in Linux, That might have an impact.
Excerpt of .vss file
<DataSetPathsTag Type="String">
G:/Downloads/ExchangeWSL/concat.nc
</DataSetPathsTag>

Log
Vapor 3.7.0.<debug>
OS: Windows 6.2 (9200)

-------------------
This session links to the dataset concat.nc which was not found. Please open this dataset if it is in a different location
-------------------
"G:/Downloads/ExchangeWSL/concat.nc" not found.
------------------
Vapor 3.7.0.<debug>
OS: Windows 6.2 (9200)

File is present in folder.

I can provide the session file, but the dataset is HUGE.

How big?  I might be able to try to find a similarly sized dataset and edit the session to point to it.  Either way, the session file might provide a clue.

4gig. Session is on the way.

Sorry, I forgot to mention that my path is what under Windows is called "Special folder" or "Known folder" which is actually a redirect. I think that might be the reason why.

Sorry, I forgot to mention that my path is what under Windows is called "Special folder" or "Known folder" which is actually a redirect. I think that might be the reason why.

I answer myself, I moved the whole thing in a regular folder and still same error occurs.

@Padanian - Could you send that session file?  Also, 4GB isn't that big.  It will be more likely that we can find the root of the issue if I can test it in an environment that's as close to yours as possible.

I think I sent it via email. Please have a look at your spam folder. Or I just attach it here. The dataset can be downloaded here
prove con italia.zip

@Padanian - Thank you - I am able to reproduce your issue.
We're currently in a code freeze to release version 3.8 and we will be testing and hopefully releasing through December.  I'm going to tag this as a high priority bug that we will take on after that.  When we get a fix, you should get an email notification that regards this thread.  We will make you a custom installer to try at that point.  This is a weird one.  Thank you for reporting it.

@Padanian - Followup question:
I am unable to import concat.nc as a WRF-ARW file entirely due to an Unrecognized time stamp error.  I'm assuming you do not see this.  Could you please confirm?

I don't. Works for me.

Prerelease of 3.8.0 still has this bug.
Vapor 3.8.0.
OS: Windows 6.2 (9200)

This session links to the dataset concat.nc which was not found. Please open this dataset if it is in a different location
"G:/Vapor/ExchangeWSL/concat.nc" not found.
Vapor 3.8.0.
OS: Windows 6.2 (9200)

@padanian - this bug didn't get fixed for 3.8.  We will get around to it once we release.  We have another windows user who may be experiencing the same bug here.

Ok, thanks
Not sure though it's the same bug. The other user has got colons in the filenames of the zipped files. Colon is an invalid character for filenames in Windows.
Before zipping the files, it is necessary to replace the colons with dashes or underscores, as in:
find /path/to/directory -type f -name "*:*" -exec rename -n 's/:/-/g' {} +

With #3414, we now support relative dataset paths with respect to session files.  Please re-open this issue if this does not solve your problem.


Decide how to handle 100-continue in HTTP/2

RFC 9113 8.8.5 Gives examples of 100-continue:
  HTTP/1.1 100 Continue            HEADERS
  Extension-Field: bar       ==>     - END_STREAM
                                     + END_HEADERS
                                       :status = 100
                                       extension-field = bar

  HTTP/1.1 200 OK                  HEADERS
  Content-Type: image/jpeg   ==>     - END_STREAM
  Transfer-Encoding: chunked         + END_HEADERS
  Trailer: Foo                         :status = 200
                                       content-type = image/jpeg
  123                                  trailer = Foo
  {binary data}
  0                                DATA
  Foo: bar                           - END_STREAM
                                   {binary data}

                                   HEADERS
                                     + END_STREAM
                                     + END_HEADERS
                                       foo = bar

...mostly as an example of how to translate HTTP/1.1? I suppose that's a question for HTTP/1 => HTTP/2 proxies, which.. doesn't happen that often? I don't know. There's not a fully-featured proxy built on top of fluke right now anyway, only trivial ones, so that's a question for later, but: for now, H2Encoder::write_response has this assert:
        // TODO: don't panic here
        assert!(
            !res.status.is_informational(),
            "http/2 does not support informational responses"
        )

Which is probably wrong.

That also means in the HTTP/2 client state machine, we shouldn't move on to "expecting body" if we receive an HTTP 1xx response — this may already be the case, I haven't checked, but we should be consistent on both sides.


ActiveRowColor is not working in android release mode

Describe the bug
When I set the ActiveRowColor properite it's working fine in debug mode, but not in realese nor in real device.
Tried to set that propertie in different ways but neither was succesfull.
Codes what I tried
<!--In xaml--!>
<dg:DataGrid.ActiveRowColor>
    <Color>#FFB0C4DE</Color>
</dg:DataGrid.ActiveRowColor>

<dg:DataGrid x:Name="dataGrid"
                Grid.Row="1"
                SelectionEnabled="True"
                ItemsSource="{Binding SomeItems}"
                SelectedItem="{Binding Selecteditem}"
                RowHeight="50"
                HeaderHeight="50"
                BorderColor="Gray"
                HeaderBackground="LightGray"
                HeaderBordersVisible="True"
                RefreshingEnabled="False"
                ItemSelected="OnRowSelected"
                ActiveRowColor="LightSteelBlue">

//In code behind
dataGrid.ActiveRowColor = Colors.LightSteelBlue;

Is this fixed now with the latest versions of MAUI and the latest DataGrid?


Tamper Chrome is not working

Hey, I was trying out Tamper Chrome, but I am not able to get it to work. It is always saying "Could not find Tamper Chrome Application

I'm having the exact same problem on:
Chrome Version 57.0.2987.133 (64-bit)
OSX 10.11.6 (15G1217)

This was my initial issue too, but if you read the second line...
'... extension requires the Taper Chrome packaged application...'
In reality Tamper Chrome is seemingly two parts: the extension and the packaged application.
Once you download and install both, it should work.
Perhaps the README could be updated to reflect the requirement?

ah, actually i read it. But when I clicked the link and ended up on the chrome web store i though i was back to the plugin page. My bad. But could probably be even more explicit :)

Thanks. Its done 👍


Update Composer dependencies (2022-07-30-00-02)

Visual regression test passed!
View the visual regression test report


WIP: Extend Cop metadata #5978

This PR is intended to extend the cop metadata as described in this issue https://github.com/rubocop-hq/rubocop/issues/5978. Currently only the empty VersionAdded meta data is added here, the others meta data fields will follow. However, I would like to clarify once whether my implementation corresponds to this as it is intended. And whether I should continue or what I should change or improve.
ToDo

[x] Add VersionAdded - the RuboCop version in which it was added

[ ] Fill VersionAdded

[ ] Add VersionChanged - the RuboCop version in which it was changed in a user-impacting way (new config, updated defaults, etc)
[ ] Add VersionRemoved - the RuboCop version in which the cop was removed for whatever reason.
[ ] Add Safe (true/false) - indicates whether the cop can yield false positives (by design) or not.
[ ] Add SafeAutoCorrect (true/false) - indicates whether the auto-correct the cop does is safe (equivalent) by design.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[ ] Squashed related commits together.
[ ] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

@bbatsov I see, I'm afraid there was some misunderstanding. But let me get this straight, in the rubocop.yml are not all cops are shown, so you don't mean the config/default.yml?
And then it should look like the following example:
Style/StringLiteralsInInterpolation:
  EnforcedStyle: single_quotes
  SupportedStyles:
    - single_quotes
    - double_quotes

@bbatsov I see, I'm afraid there was some misunderstanding. But let me get this straight, in the rubocop.yml are not all cops are shown, so you don't mean the config/default.yml?
And then it should look like the following complete example (in case Style/StringLiteralsInInterpolation cop will be removed in the next version):
Style/StringLiteralsInInterpolation:
  VersionAdded: v0.27.0
  VersionChanged: v0.27.1
  VersionRemoved: v0.58.0
  Safe: true
  SafeAutoCorrect: true
  EnforcedStyle: single_quotes
  SupportedStyles:
    - single_quotes
    - double_quotes

And if e.g. the VersionChanged or VersionRemoved does not exist the corresponding metadata key should also not exist either.

@bbatsov I see, I'm afraid there was some misunderstanding. But let me get this straight, in the rubocop.yml are not all cops are shown, so you don't mean the config/default.yml?

Yeah, it's default.yml. Too many config files. :D

And if e.g. the VersionChanged or VersionRemoved does not exist the corresponding metadata key should also not exist either, right?

Yep.

And if e.g. the VersionChanged or VersionRemoved does not exist the corresponding metadata key should also not exist either, right?

Exactly.
Looking at all the keys maybe we should renamed Safe to SafeInspection so it's more clear what this refers to. (inspection that's guaranteed not to yield false offenses).
For the safety levels in the interest of brevity I'd suggest to assume that if they are not explicitly added they default to true. This will require us to add them only to the unsafe cops, which are relatively few overall.

VersionAdded: v0.27.0

We can probably omit the v here.

P.S. Thanks for tackling this, btw - it's one of the most important tasks on the road to RuboCop 1.0!

Okay, all right, then I'll remodel my PR in the hope that we've cleared up all misunderstandings. 😉
And yes, I'm very happy and honored if I can help rubocop to get to the first major version. 😊

@Poisoned Any progress here?

Hey @bbatsov sorry for the late feedback, unfortunately I didn't find any spare time to resume my work on this PR and then I was also on vacation. But I started working on it again and I'm just about to finishing the script that extracts the versions from the release notes.

Great!
On Sat, 4 Aug 2018 at 17:49, Christoph Hugo<EMAIL_ADDRESS>wrote:

Hey @bbatsov https://github.com/bbatsov sorry for the late feedback,
unfortunately I didn't find any spare time to resume my work on this PR and
then I was also on vacation. But I started working on it again and I'm just
about to finishing the script that extracts the versions from the release
notes.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rubocop-hq/rubocop/pull/6034#issuecomment-410454728,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGVyg7Hchkn9uH7PmwTQeAQPH7RZKcMks5uNbR6gaJpZM4UyUwL
.

--
Best Regards,
Bozhidar Batsov
http://www.batsov.com

So I have discarded the old code again and started to design logic to search the release notes for the corresponding versions. I've designed it as a class for now, but I think I'll put it back into a rake task and make it accessible there.

Sounds good to me. Likely we’ll need this task just once anyways.
On Sat, 11 Aug 2018 at 15:56, Christoph Hugo<EMAIL_ADDRESS>wrote:

So I have discarded the old code again and started to design logic to
search the release notes for the corresponding versions. I've designed it
as a class for now, but I think I'll put it back into a rake task and make
it accessible there.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rubocop-hq/rubocop/pull/6034#issuecomment-412276776,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGVyou7WMhEpz6cszGQ48DlhMe2utGlks5uPuKWgaJpZM4UyUwL
.

--
Best Regards,
Bozhidar Batsov
http://www.batsov.com

@Poisoned Any progress with this?

Sorry, I just can't find time to continue working on this PR at the moment, so I prefer to close the PR. If the issue remains open and I find time I will reopen this PR. 😕


Better error message for pattern variable declarations where they are not allowed

The new patterns feature lets you use patterns in variable declarations, but only for local variables. Declarations of top-level variables and class fields can't use patterns. Right now, the syntax error you get from attempting to use a pattern in one of those latter locations is pretty unclear:
lib/foo.dart:9:5: Error: Expected an identifier, but got '('.
Try inserting an identifier before '('.
var (lat, long) = geoCode('Aarhus');
    ^
lib/foo.dart:9:1: Error: The return type can't be 'var'.
Try removing the keyword 'var', or replacing it with the name of the return type.
var (lat, long) = geoCode('Aarhus');
^^^
lib/foo.dart:9:17: Error: Expected a function body or '=>'.
Try adding {}.
var (lat, long) = geoCode('Aarhus');

It would be great if the parser could have recovery that parses the pattern and then report a higher-level error saying that patterns can only be used in local variable declarations.
cc @jcollins-g @natebosch

@jensjoha


No support for PHP 7.4 due to old Sentry

When using this plugin with PHP 7.4, the following error is thrown:
PHP Fatal error:  Uncaught yii\base\ErrorException: Array and string offset access syntax with curly braces is deprecated in vendor/sentry/sentry/lib/Raven/Client.php:331
Stack trace:
#0 vendor/craftcms/cms/src/web/ErrorHandler.php(74): yii\base\ErrorHandler->handleError()
#1 vendor/composer/ClassLoader.php(444): craft\web\ErrorHandler->handleError()
#2 vendor/composer/ClassLoader.php(444): include()
#3 vendor/composer/ClassLoader.php(322): Composer\Autoload\includeFile()
#4 [internal function]: Composer\Autoload\ClassLoader->loadClass()
#5 vendor/lukeyouell/craft-sentry/src/services/SentryService.php(69): spl_autoload_call()
#6 vendor/lukeyouell/craft-sentry/src/Sentry.php(37): luke in vendor/sentry/sentry/lib/Raven/Client.php on line 331

The relevant lines from sentry/lib/Raven/Client.php::
if ($path{0} === DIRECTORY_SEPARATOR && substr($path, -1) !== DIRECTORY_SEPARATOR) {
    $path .= DIRECTORY_SEPARATOR;
}

Seems like the Sentry SDK is used in version 1.x which uses the curly brace syntax for string offset. This syntax has been deprecated with PHP 7.4 and throws a runtime error.
I guess the only way to get it to work with PHP 7.4 would be updating to Sentry 2.

Agree, would be great to upgrade to Sentry 2 sdk indeed.

@lukeyouell are any updates planned?

Needed this as well. Happy to jump in with a PR if that helps @lukeyouell

There's an alternative plugin that uses the latest SDK: https://github.com/born05/craft-sentry


[ez] Check for nullable system prompt

[ez] Check for nullable system prompt
We should really have automated tests more in our cookbooks to catch these issues, weird
This is NOT a thorough solution, just quickly unblocking. Filed an issue to investigate deeper and make sure we have better null-checks!!!

test plan

Before
After


Add timeout to python-requests

python-requests doesn't have a timeout by default (source) :scream:

e2e :heavy_check_mark:


Fix info dict key error with latest MetaDrive

In this PR, I will fix:

Wrong state observation. The path to StateObservation is wrong with latest MetaDrive and we have to change the import code for this.
Wrong info dict. In latest MetaDrive, in the first step of a newly spawn agent the info dict contains velocity entries like: {velocity=0.0}. Therefore in the DrivingCallback, the criterion to check whether the info dict is came from a newly spawn agent if "velocity" in info become invalid since velocity is there even the agent is just born. I changed this criterion to if "step_reward" in info now!

Related to #14

thanks


Memory leak - Base class instance

I have noticed a memory issue with express:
I have create nodejs typescript app having get method which will create an object welcome to implement  my business logic.
I have run node --inspect and get see that the base class instance existing in memory after api request completed.

App.ts :
import express from 'express';
import { welcome } from './welcome';
const app = express();
const port = 3000;

app.get('/', (req, res) => {
	const screen: any = {}
	screen.test = new welcome();
	res.send('Hello World!');
});

app.listen(port, () => {
	return console.log(`Express is listening at http://localhost:${port}`);
});

basecontroll.erts
export class BaseController {
	constructor() {
		console.log('BaseController');
	}
}

welcome.ts
import { BaseController } from "./basecontroller";

export class welcome extends BaseController {
	constructor() {
		super()
		console.log("welecome");

	}
}

I have tried to delete welcome object forcefully like  delete screen.test  but still baseclass instance existing in memory.
I have tried below command but we can't do this for each class
delete screen.test.__proto__.__proto__.constructor
This is working but it just work around.
Package.json:
{
  "name": "nodejsmemoryleak",
  "version": "1.0.0",
  "description": "",
  "main": "dist/index.js",
  "types": "dist/index.d.ts",
  "scripts": {
    "start": "tsc && node --inspect dist/app.js",
    "build": "tsc"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "devDependencies": {
    "@types/express": "^4.17.1",
    "nodemon": "^3.0.1",
    "ts-node": "^10.9.1",
    "typescript": "^5.1.6"
  },
  "dependencies": {
    "express": "^4.17.1"
  }
}

JavaScript is a garbage collected language, so objects may not be freed immediately after they leave the scope. Does the number of instances of this object grow when you spam requests (e.g. by calling fetch(`http://localhost:${port}`) in setInterval with a low interval)?
I've added a large memory allocation (100 MiB Buffer) with some random reads and writes (to prevent optimization) to the base class instance and the memory used by Node.js never exceeded 200 MB, which it would do if 2 base classes were allocated at the same time. However, I can see just one BaseController class in memory snapshots, so it may be that I can't reproduce the bug.
You can also check if the same behaviour can be reproduced without Express:
import http from "node:http";
const port = 3000;

class BaseController {
    constructor() {
        console.log("BaseController");
    }
}
class Welcome extends BaseController {
    constructor() {
        super();
        console.log("welcome");
    }
}

http.createServer((req, res) => {
    const screen = {};
    screen.test = new Welcome();
    res.end("Hello World!");
}).listen(port, () => {
    console.log(`Server is listening at http://localhost:${port}`);
    const url = new URL(`http://localhost:${port}`);
    // WARNING: this will send requests really fast
    setInterval(() => fetch(url).catch(), 10);
});

If you create multiple snapshots you can get a summary or comparison of objects allocated between these snapshots and you can check what keeps them alive.

Thank you  krzysdz for the detail.
yes it is not increasing the count of BaseController and we can reproduce the same issue with http code which you have provide. Below is the comparison between initial memory allocation immediately after starting server and after calling api.

My concern is number of such scenario in an application. we are inheriting base class in many classes and here it create base class instance for each class object inherited it.
I have created welcome1 class which is also inheriting same BaseController. below is the memory snapshot after executing below code


refactor!: Merge cosmwasm-bindings and contract features

As I investigated the entry_point attribute macro, it appears to only add exports only when the target is WebAssembly. As a consequence of that even when the integration tests pull the contract packages with the contract feature it won't affect the workflow, as the integration tests are in a way prohibited from targeting WebAssembly.
The result is that, contract and cosmwasm-bindings can be freely combined, which is the intent of this PR.

Full workflow run, including optimized builds: https://github.com/nolus-protocol/nolus-money-market/actions/runs/7058585327

Manually verified the entry points are exported.


Add README to storage/write_ahead_log

Once #370 gets merged, we should have documentation explaining the log manager and its tasks

Instead of a README, a wiki page has been added


Async/Await does not work out of the box on iOS/Android with a modern babel config

🐛 Bug Report

With the new JSC for Android, apps with minimum iOS 10 support can work with a modern babel config that removes unnecessary babel plugins from the metro RN babel preset. This also means that async/await, which is supported in both those base versions, should work out of the box. However, on both platform, async calls only seem to be "flushed" if a user does a tap gesture.
This might be similar to an issue reported before: https://github.com/facebook/react-native/issues/19572 But the cause in that issue doesn't seem to be the case.
Here's a gif explaining the behaviour on ios and android:

To Reproduce

Clone: git@github.com:karanjthakkar/ModernBabelConfigIssue.git
Run npm install
Run the following for each platform to see the broken build:

iOS: RCT_NO_LAUNCH_PACKAGER=1 FORCE_BUNDLING=1 react-native run-ios
Android: react-native run-android --no-packager

You can then checkout the branch named working and that will work.

The only difference between both is the babel config.
The culprit here seems to be the absence of @babel/plugin-transform-regenerator which compiles async/await code.
Expected Behavior

Multiple async/await calls should work the same without compilation via babel.
Code Example

Repo with code: https://github.com/karanjthakkar/ModernBabelConfigIssue
Repro steps mentioned above.
Environment

  React Native Environment Info:
    System:
      OS: macOS 10.14.1
      CPU: (8) x64 Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
      Memory: 26.45 MB / 16.00 GB
      Shell: 5.3 - /bin/zsh
    Binaries:
      Node: 10.15.3 - ~/.nvm/versions/node/v10.15.3/bin/node
      npm: 6.9.0 - ~/.nvm/versions/node/v10.15.3/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
      Android SDK:
        API Levels: 23, 24, 25, 26, 27, 28
        Build Tools: 23.0.1, 25.0.0, 25.0.2, 26.0.2, 26.0.3, 27.0.3, 28.0.3
        System Images: android-26 | Google APIs Intel x86 Atom_64, android-26 | Google Play Intel x86 Atom, android-27 | Google APIs Intel x86 Atom, android-28 | Google APIs Intel x86 Atom
    IDEs:
      Android Studio: 3.3 AI-182.5<IP_ADDRESS>64788
      Xcode: 10.1/10B61 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.8.3 => 16.8.3
      react-native: 0.59.2 => 0.59.2
    npmGlobalPackages:
      react-native-cli: 2.0.1

Have you tried removing the Promise polyfill? That may be causing issues too, since it might conflict with the Promise type built into the JSC and (potentially) used by async/await.

It's not recommended to change babel config unless you know what you're doing. we're working on to reduce babel plugins and polyfills used. Thank you

@newyankeecodeshop Hey! I tried removing the polyfill from here: https://github.com/facebook/react-native/blob/4148976a83ab96029de1b26b515ff95d3cb58c10/Libraries/Core/InitializeCore.js#L35 But I still face the same issue with a fresh metro cache.
@dulmandakh Do you have any links to conversations around this? I'd love to keep an eye out.
Using rn-snoopy I can see that the message describing the new UI is sent to the native side:
04-02 10:20:00.790  5264  5297 D ReactNative: ReactInstanceManager.attachRootViewToInstance()
04-02 10:20:01.184  5264  5296 I ReactNativeJS: Running application "ModernBabelConfigIssue" with appParams: {"rootTag":1}. __DEV__ === false, development-level warning are OFF, performance optimizations are ON
04-02 10:20:01.196  5264  5296 I ReactNativeJS: JS->N : 6.2([3,"RCTRawText",1,{"text":"Loading...."}]) *
04-02 10:20:01.198  5264  5296 I ReactNativeJS: JS->N : 6.2([5,"RCTText",1,{"fontSize":20,"textAlign":"center","margin":10,"accessible":true,"allowFontScaling":true,"ellipsizeMode":"tail"}]) *
04-02 10:20:01.199  5264  5296 I ReactNativeJS: JS->N : 6.18([5,[3]]) *
04-02 10:20:01.206  5264  5296 I ReactNativeJS: JS->N : 6.2([7,"RCTView",1,{"flex":1,"justifyContent":"center","alignItems":"center","backgroundColor":-656129}]) *
04-02 10:20:01.206  5264  5296 I ReactNativeJS: JS->N : 6.18([7,[5]]) *
04-02 10:20:01.206  5264  5296 I ReactNativeJS: JS->N : 6.2([9,"RCTView",1,{"collapsable":true,"pointerEvents":"box-none","flex":1}]) *
04-02 10:20:01.206  5264  5296 I ReactNativeJS: JS->N : 6.18([9,[7]]) *
04-02 10:20:01.206  5264  5296 I ReactNativeJS: JS->N : 6.2([13,"RCTView",1,{"flex":1,"pointerEvents":"box-none"}]) *
04-02 10:20:01.206  5264  5296 I ReactNativeJS: JS->N : 6.18([13,[9]]) *
04-02 10:20:01.211  5264  5296 I ReactNativeJS: JS->N : 6.18([1,[13]]) *
04-02 10:20:01.213  5264  5296 I ReactNativeJS: JS->N : 30.0([1,0,1554196801213,false]) *
04-02 10:20:01.221  5264  5296 I ReactNativeJS: JS->N : 30.0([2,1,1554196801221,false]) *
04-02 10:20:01.253  5264  5296 I ReactNativeJS: JS->N : 6.22([3,"RCTRawText",{"text":"Welcome to React Native mate!"}]) *

Its just not getting flushed from what I can imagine. Do you have any ideas on why this could be happening?

FYI, https://github.com/facebook/react-native/commit/06cf7face24d0f06e7b690dfc30912ec1e3e1bfb commit removing various polyfills that are no longer required.

@dulmandakh This is unrelated to whether the polyfills are present or not and still exists on the latest master where the mentioned change has landed. Can you please reopen this?

@dulmandakh This is unrelated to whether the polyfills are present or not and still exists on the latest master where the mentioned change has landed. Can you please reopen this?

It's not recommended to change babel config unless you know what you're doing, so if the issue is caused by babel config change we cannot fix it. Please use babel config RN provides.


Markers are not shown in the Android app only

Hi Guys,
i have a problem with official  google maps package
the markers are not shown in the android app only(iOS is working fime)
Steps to Reproduce

implement google maps
use your markers
in my case markers are not visible in android app

Expected results:
the markers should be visible in both iOS and Android
Actual results:
iOS working version

Android broken map
post pictures

E/flutter (13818): 	at android.os.MessageQueue.next(MessageQueue.java:326) E/flutter (13818): 	at android.os.Looper.loop(Looper.java:165) E/flutter (13818): 	at android.app.ActivityThread.main(ActivityThread.java:6810) E/flutter (13818): 	at java.lang.reflect.Method.invoke(Native Method) E/flutter (13818): 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:547) E/flutter (13818): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:873) E/flutter (13818): ) E/flutter (13818): #0      StandardMethodCodec.decodeEnvelope package:flutter/…/services/message_codecs.dart:582 E/flutter (13818): #1      MethodChannel._invokeMethod package:flutter/…/services/platform_channel.dart:159 E/flutter (13818): <asynchronous suspension> E/flutter (13818): #2      MethodChannel.invokeMethod package:flutter/…/services/platform_channel.dart:332 E/flutter (13818): #3      MethodChannelGoogleMapsFlutter.updateMarkers package:google_maps_flutter_platform_interface/…/method_channel/method_channel_google_maps_flutter.dart:245 E/flutter (13818): #4      GoogleMapController._updateMarkers package:google_maps_flutter/src/controller.dart:115 E/flutter (13818): #5      _GoogleMapState._updateMarkers package:google_maps_flutter/src/google_map.dart:287 E/flutter (13818): <asynchronous suspension> E/flutter (13818): #6      _GoogleMapState.didUpdateWidget package:google_maps_flutter/src/google_map.dart:265 E/flutter (13818): #7      StatefulElement.update package:flutter/…/widgets/framework.dart:4815 E/flutter (13818): #8      Element.updateChild package:flutter/…/widgets/framework.dart:3314 E/flutter (13818): #9      RenderObjectElement.updateChildren package:flutter/…/widgets/framework.dart:5647 E/flutter (13818): #10     MultiChildRenderObjectElement.update package:flutter/…/widgets/framework.dart:6246 E/flutter (13818): #11     Element.updateChild package:flutter/…/widgets/framework.dart:3314 E/flutter (13818): #12     ComponentElement.performRebuild package:flutter/…/widgets/framework.dart:4652 E/flutter (13818): #13     StatefulElement.performRebuild package:flutter/…/widgets/framework.dart:4800 E/flutter (13818): #14     Element.rebuild package:flutter/…/widgets/framework.dart:4343 E/flutter (13818): #15     BuildOwner.buildScope package:flutter/…/widgets/framework.dart:2730 E/flutter (13818): #16     WidgetsBinding.drawFrame package:flutter/…/widgets/binding.dart:913 E/flutter (13818): #17     RendererBinding._handlePersistentFra W/DynamiteModule(13818): Local module descriptor class for com.google.android.gms.googlecertificates not found. I/DynamiteModule(13818): Considering local module com.google.android.gms.googlecertificates:0 and remote module com.google.android.gms.googlecertificates:6

`Flutter Doctor -v
`Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel unknown, 1.22.6, on Mac OS X 10.14.6 18G8012 darwin-x64, locale en-EG)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
[✓] Android Studio (version 3.6)
[✓] VS Code (version 1.55.2)
[✓] Connected device (1 available)
• No issues found!
Mohammeds-MacBook-Pro:saloni_salon mohammedadelalisorour$ `

Hi @mohadel92
I cannot reproduce the issue.

Please upgrade your Flutter version and provide your flutter doctor -v, your flutter run --verbose, your pubspec.yaml and a minimal reproducible code sample.
Thank you


[Question] Is it possible to both support legacy and new kernels by writing code once?

I know it is possible to use BPF on 4.x kernel, but some features are not always supported, such as global var, /sys/kernel/btf/vmlinux, and tp_btf.  If I want to write some tools to support from 4.x to 5.x kernels, I have to deal with these discord:

Define some macros to generate the suitable code for the target kernel like this.
I can use  BTFHub to get some BTFs or use pahole to generate the BTF files for the ones not provided in the hub, and then let the tool load it by an option.
Define two or more entry points to hook at the same tracepoint using tp_btf, raw_tp, kprobe, or else, and then choose to load one by detecting the kernel's version or feature.

But the first one is not CORE, I still have to build twice to generate 2 tools with the same usage. Another way is to compile two bpf.o of the same code with global var or not, and then let the userspace decide which one to load. The second and third one needs some code change for tools from BCC like this.
So, is it possible to fill the gaps between different kernels? Are there some more discords? And does the community want this ability?

I don't think there is any magical solution. So what you describe is probably the way to do it. You can write your code to use the lowest common denominator set of features (e.g., don't use global vars at all, use BPF maps directly, etc). You'll lose some of the nice features and developer productivity, but you'll be able to support older kernels.
There is no magic bullet, in short.

Got it. Thank you for the explanation.


Display event banners, use fallbacks

Related to #97. The mobile app will display an event banner if exists, otherwise use our generic fallback image.

check approve. Looks good


[SPARK-31420][WEBUI][FOLLOWUP] Make locale of timeline-view 'en'

What changes were proposed in this pull request?

This change explicitly set locale of timeline view to 'en' to be the same appearance as before upgrading vis-timeline.
Why are the changes needed?

We upgraded vis-timeline in #28192 and the upgraded version is different from before we used in the notation of dates.
The notation seems to be dependent on locale. The following is appearance in my Japanese environment.

Although the notation is in Japanese, the default format is a little bit unnatural (e.g. 4月9日 05:39 is natural rather than 9 四月 05:39).
I found we can get the same appearance as before by explicitly set locale to 'en'.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

I visited JobsPage, JobPage and StagePage and confirm that timeline view shows dates with 'en' locale.

NOTE: #28192 will be backported to branch-2.4 and branch-3.0 so this PR should be follow #28214 and #28213 .

Test build #121291 has started for PR 28218 at commit 20bfd4d.

Merged build finished. Test PASSed.

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

Test build #121291 has finished for PR 28218 at commit 20bfd4d.

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

Merged build finished. Test PASSed.

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

I'm OK with it. I would have thought localized display is a little better, but the whole UI is clearly written for locale=en anyway. Maybe it's more consistent to standardize that rather than have random bits localized.

+1

@srowen
Yeah, At first, I thought it's better to make locale customizable but any other parts in the UI i s not like so.
So, I explicitly set 'en' for the consistency.

+1

Thanks @srowen , @HyukjinKwon and @gatorsmile !
Merging this into master, branch-3.0 and branch-2.4.

Yeah, I noticed that when I confirmed the log message. Thanks for the notice!


Workday session cookies are disappearing after the initial get/set cycle.

After the initial get/set cycle, the Workday session cookies are no longer found on subsequent chrome.cookies.getAll() calls.

Fixed: https://github.com/NathanLaan/prevent-workday-timeout/commit/1f4dcd7527a378a25fc490570685db1e060bc6ec


Fix broken links to API docs

The link to .someWhere in mount.md was broken due to a capitalization error
All of the other links were broken in the web version of the docs due to absolute paths (they did work w/in Github)

LGTM, thanks!

Whoa that was quick! Thanks as well @ljharb!


fix(chat): fix export button color for prompt preview mode and add truncating to publish modal (Issues #1327, #1464)

Description:
fix export button color for prompt preview mode and add truncating to publish modal
Issues:

#1327
#1464

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review


Make MDETR demo without modifications running properly

Since the subfiles like transformers.py and mdetr.py are running through the cache the modification to subfiles has been applied even when running the original mdetr demo code.

To solve this issue, in the mdetr_demo.py I added another key when loading the model from the torch hub to force it to reload the model instead of using the cached files. Refer to torch.hub for additional details.
model, postprocessor = torch.hub.load('ashkamath/mdetr:main', 'mdetr_efficientnetB5', pretrained=True, return_postprocessor=True, force_reload=True)

This makes the MDETR demo run using the original transformer.py and mdetr.py. However, the issue is not completely solved. If I were to run the GazeMDETR on another computer since the modules are being loaded directly from the torch hub and not the GitHub repo folder, it would cause an issue. To solve this issue refer to #3.

Temporary solution to do comparisons
The transformer.py and mdetr.py files that are the modified ones are copied. when running the GazeMDETR, I replace them with the ones in the .cache path.

suggested solutions

Create a condition to run the parts related to either the MDETR or GazeMDETR


Enhancement: Resizable tables

Would be a nice thing if tables would be resizable. For example some tables with a lot of not so important columns takes too much space, so it could be resized, eventually with inline scrolling to view all the columns if needed? Thanks!

Interesting idea. Apart from the obvious "grab lower-right corner to resize" UI stuff, we would have to also provide a UI/UX-friendly way to revert back to "auto-width" mode, because users will want to go back to the previous automatic state. Perhaps a double-click on this corner can do this?

I agree with your idea of reverting. Double click will make it, if we provide this info to the users how they  do it. Or an additional option when double click the table where you edit the table name and comment eventually. Or both ways. Thanks!


Remove StateStream types

[Docs] Remove StateStream from the README
[Docs] Move strongly typed examples into HeaderDoc blocks
[Tests] Test for and fix disposable.disposed getter
[StateStream] Remove StateStream from core
[StoreType] StoreType is no longer a subtype of MiddlewareApiType

Current coverage is 99.28%

Merging #20 into develop will increase coverage by +4.09% as of ef3f5a1

@@            develop     #20   diff @@
=======================================
  Files            15      14     -1
  Stmts           416     422     +6
  Branches          0       0
  Methods           0       0
=======================================
+ Hit             396     419    +23
  Partial           0       0
+ Missed           20       3    -17

Review entire Coverage Diff as of ef3f5a1

Powered by Codecov. Updated on successful CI builds.


Forfeiting a kata should enable forking for all languages

Currently, it is only possible to fork a language if one solved the kata in that language.  Forfeiting should enable forking for all available languages of a given kata, since you can see all solutions anyway and you will no longer gain honor/rank from that kata. This would allow users to propose fixes and improvements in a language B after they solved the kata in language A but are not interested in solving it in language B.

Modifying the algorithm for calculating honor and ranking points based on completed and forfeited translations requires extreme caution. This is a delicate area, and previous bugs have caused significant issues, including instances where users were unable to accumulate ranking points due to these errors. Any changes to this system should be approached with utmost care and thorough testing to avoid repeating past problems.


the schema of hive_cli_conn_id will not be apply in HiveOperator

If I do not input schema, to apply the schema of hive_cli_conn_id is better reasonable

This should do it: https://github.com/airbnb/airflow/pull/200
I just want to make sure this won't affect our production jobs (by setting our schema to empty string in our connections) before merging


Config client hints

Fixes gh-354

Please let me know when you think we should merge this PR.

@sdeleuze I'm done.

Merged via c6e4d7239db63f05e76d5f6c053b9024107db10f, thanks.


Don't recompute AuxPoW when serving peer requests

I didn't test it yet, just applied my patch. I'm using it along with other minor patches on 1.14.2. It would be interesting to feed it with a block or just header with incorrect AuxPoW and immediately request it from a different node and see how it behaves.

Sorry, it is supposed to be a draft.

Sorry, it is supposed to be a draft.


vscode-css-languageservice ships 2.8 MB of coverage files

pushed a fix through https://github.com/Microsoft/vscode/commit/a2d1bbbe6f88f4fe31e5afe01cee3884a58517d2

Nice!


scale as vector

Just reposting previous suggestion in TWHL as GitHub issue: https://twhl.info/thread/locate-post/345813

Make the macro_insert scale property a vector, so you could stretch one axis, and flip another axis.

This is a bit tricky, because scaling also adjusts the 'scale' attributes of entities, and those are just a single number.
One possible solution would be to introduce a separate 'geometry_scale' attribute. This would be a vector, and it would only apply to geometry. The existing 'scale' attribute would remain a number, and would apply to entities, and to geometry if the 'geometry_scale' attribute is left empty. This should produce backwards-compatible behavior and allow geometry to be stretched/flipped along different axis. I'll see what I can do.

This will be available in v1.2, with the new instance_geometry_scale attribute. I also fixed a bug with negative scales producing invalid brushes.


Notarizing macOS build

Which version of kubefirst are you using?
ALL
Which cloud provider?
None specific
Which installation type?
None specific
Which distributed Git provider?
None specific
What is the issue?
We should notarize our macOS build to add some legitimacy to the application, and prevent some security software to look at our CLI as suspicious caused it's not signed. It's possible to do it with a GitHub Action when we release as long as we have an Apple Developer Account + created the proper signing certificate.

Code of Conduct

[X] I agree to follow this project's Code of Conduct

A tutorial on how Espanso did it https://federicoterzi.com/blog/automatic-code-signing-and-notarization-for-macos-apps-using-github-actions/

@jarededwards @johndietz if you will setup the apple dev subscription for kubefirst I will take care of the rest and get it signed.
Not sure many devs will really appreciate this but as we move towards the gui it might make more sense.


deleting a Team that still has Pipelines causes pipeline destroy to fail

What
k delete team x should delete any Pipeline resources associated with team x
Why
Otherwise the Pipeline resources will be undestroyable unless you remove the finalizer from their metadata. The pipeline-operator will be stuck in a loop attempting to destroy a pipeline that doesn't exist.
Context / thoughts

The Pipeline resources should probably set the Team resource as it's owner reference so they are dependent and get GC'd correctly
The destroy step for Pipeline resource should correctly handle the case when the pipeline has already been destroyed

I'm happy that this happens so irregularly that it's not worth doing the work until/unless it starts becoming a pain.


Update Consul to 1.0.0

We should do #270 first. Then release marathon-consul and finally perform same steps to update to 1.0. There might be some breaking changes so the code might need some fixes. Finally, we should check if Marathon-Consul is backwards compatible with Consul 0.9.3. And if everything is ok we may create a new release.

any plans on merging this PR and cutting a release?
Consul 1.1.0 removed the deprecated script checks which is breaking marathon-consul checks.

We're already on Consul 1.2.0 as of #288.


Create a data generator to populate an avro file with coco personnas based on the Palisade beans

No plans to work on this further currently. Will re-open if required


Changes to make package names suit standard practice

Changes to make package naming consistent with standard

Changes to make package names suit standard practice


Able to load complex data arrays.

Hi,
I modified the code to load and save complex datatype arrays from/to .mat files. When loading, it creates a tuple of python builtin complex values.
I've implemented a test related to complex values, using the same format of the other tests (as close as I could). Now it passes all the tests.
But regarding tests, I had to modify the json module, from "json" to "json-tricks", to be able to serialize complex data, cause the standar json.dumps didn't work with complex numbers. Any problem with that?
Regards,
Ion

Is there a way to achieve this without json-tricks?


BackgroundIsolateBinaryMessenger.ensureInitialized web issue

Steps to reproduce
Unable to run web application because of BackgroundIsolateBinaryMessenger.ensureInitialized
Expected results
should be running smoothly
Actual results
not able to run
Code sample
Code sample
   if(!AppConstant.isWeb) {
      BackgroundIsolateBinaryMessenger.ensureInitialized(rootInstanceToken!);
    }

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
This will generate a JSON format file containing all messages that
need to be translated.
Waiting for connection from debug service on Chrome...
lib/isolate/http_isolate_service.dart:49:40: Error: Member not found: 'BackgroundIsolateBinaryMessenger.ensureInitialized'.
      BackgroundIsolateBinaryMessenger.ensureInitialized(rootInstanceToken!);
                                       ^^^^^^^^^^^^^^^^^
Failed to compile application.```

</details>

### Flutter Doctor output

<details open><summary>Doctor output</summary>

```console
 flutter doctor -v
[√] Flutter (Channel stable, 3.16.2, on Microsoft Windows [Version 10.0.23606.1000], locale en-IN)
    • Flutter version 3.16.2 on channel stable at C:\Users\sachin\Documents\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 9e1c857886 (3 weeks ago), 2023-11-30 11:51:18 -0600
    • Engine revision cf7a9d0800
    • Dart version 3.2.2
    • DevTools version 2.28.3

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at C:\Users\sachin\AppData\Local\Android\sdk
    • Platform android-34, build-tools 33.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[X] Visual Studio - develop Windows apps
    X Visual Studio not installed; this is necessary to develop Windows apps.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components

[√] Android Studio (version 2023.1)
    • Android Studio at D:\Android Studio Preview
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10353782)

[√] Android Studio (version 2022.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = C:\Program Files\Android\Android Studio
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)

[√] Connected device (4 available)
    • SM M215G (mobile) • RZ8R81NYF2E • android-arm64  • Android 13 (API 33)
    • Windows (desktop) • windows     • windows-x64    • Microsoft Windows [Version 10.0.23606.1000]
    • Chrome (web)      • chrome      • web-javascript • Google Chrome 120.0.6099.110
    • Edge (web)        • edge        • web-javascript • Microsoft Edge 121.0.2277.4

[√] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Thaks for filing this @sachinbpatil619.
This is similar to https://github.com/flutter/flutter/issues/136886.
Closing as this is WAI per this https://github.com/flutter/flutter/issues/136886#issuecomment-1781620302


env: remove refinement types from default environment

Commit message:

This dramatically lowers the performance cost of type checking collections in some cases.

The difference is staggering in some cases.
~/github.com/sanctuary-js/sanctuary (master)
$ time node -e 'const S = require("./"), xs = []; for (let id = 1; id <= 8; id += 1) xs.push({id}); S.I(xs)'

real	1m9.221s
user	1m8.379s
sys	0m0.790s

~/github.com/sanctuary-js/sanctuary (davidchambers/env)
$ time node -e 'const S = require("./"), xs = []; for (let id = 1; id <= 1000; id += 1) xs.push({id}); S.I(xs)'

real	0m0.450s
user	0m0.456s
sys	0m0.021s

Increasing the length of xs from 8 to 9 on master results in the heap consuming all available memory on my computer. On davidchambers/env increasing the length of xs from 8 to 1000 only adds about 200 milliseconds to the run time.
Removing refinement types from the default environment will result in less informative error messages in some cases (but much clearer error messages in certain other cases). @Avaq has proposed that sanctuary-def should explicitly support refinement types (sanctuary-js/sanctuary-def#162). Were it to do so, refinement types could be ignored when resolving type variables, but could be considered when enumerating a value's types in an error message. The best of both worlds!
Many thanks to @mfidemraizer for providing a minimal test case which made it easy to diagnose this long-standing issue. :clap:

@davidchambers No problem :D

Closing in favour of sanctuary-js/sanctuary-def#177


use equals to compare OptionalInt instances

@kkraune please review
@jonmv @bratseth FYI

One reason why optional should be used with care...

Thanks!


[colm] eradicate the use of replItemList

The replacement, accumulator, and string concat sections of the grammar all use this global (to Parser). This is problematic in the face of recursion. It should be replaced with code that passes the list up through the productions.

verified fixed in 0.13


[Event Request] Codeunit 90 "Purch.-Post" - OnRoundAmountOnBeforeIncrAmount

Hi, would it be possible to modify the OnRoundAmountOnBeforeIncrAmount event in codeunit 90 (Purch.-Post) by adding the xPurchLine parameter, like this:
    local procedure RoundAmount(PurchHeader: Record "Purchase Header"; var PurchLine: Record "Purchase Line"; PurchLineQty: Decimal)
    var
        CurrExchRate: Record "Currency Exchange Rate";
        NoVAT: Boolean;
    begin
        OnBeforeRoundAmount(PurchHeader, PurchLine, PurchLineQty);

        with PurchLine do begin
            IncrAmount(PurchHeader, PurchLine, TotalPurchLine);
            Increment(TotalPurchLine."Net Weight", Round(PurchLineQty * "Net Weight", UOMMgt.WeightRndPrecision));
            Increment(TotalPurchLine."Gross Weight", Round(PurchLineQty * "Gross Weight", UOMMgt.WeightRndPrecision));
            Increment(TotalPurchLine."Unit Volume", Round(PurchLineQty * "Unit Volume", UOMMgt.CubageRndPrecision));
            Increment(TotalPurchLine.Quantity, PurchLineQty);
            if "Units per Parcel" > 0 then
                Increment(TotalPurchLine."Units per Parcel", Round(PurchLineQty / "Units per Parcel", 1, '>'));

            if "VAT Calculation Type" = "VAT Calculation Type"::"Reverse Charge VAT" then begin
                "Amount Including VAT" := Amount;
                "VAT %" := 0;
            end;

            xPurchLine := PurchLine;
            PurchLineACY := PurchLine;
            OnRoundAmountOnBeforeCalculateLCYAmounts(xPurchLine, PurchLineACY, PurchHeader);
            if PurchHeader."Currency Code" <> '' then begin
                if PurchHeader."Posting Date" = 0D then
                    Usedate := WorkDate
                else
                    Usedate := PurchHeader."Posting Date";

                NoVAT := Amount = "Amount Including VAT";
                "Amount Including VAT" :=
                  Round(
                    CurrExchRate.ExchangeAmtFCYToLCY(
                      Usedate, PurchHeader."Currency Code",
                      TotalPurchLine."Amount Including VAT", PurchHeader."Currency Factor")) -
                  TotalPurchLineLCY."Amount Including VAT";
                if NoVAT then
                    Amount := "Amount Including VAT"
                else
                    Amount :=
                      Round(
                        CurrExchRate.ExchangeAmtFCYToLCY(
                          Usedate, PurchHeader."Currency Code",
                          TotalPurchLine.Amount, PurchHeader."Currency Factor")) -
                      TotalPurchLineLCY.Amount;
                    "Line Amount" :=
                      Round(
                          CurrExchRate.ExchangeAmtFCYToLCY(
                          Usedate, PurchHeader."Currency Code",
                          TotalPurchLine."Line Amount", PurchHeader."Currency Factor")) -
                      TotalPurchLineLCY."Line Amount";
                "Line Discount Amount" :=
                  Round(
                    CurrExchRate.ExchangeAmtFCYToLCY(
                      Usedate, PurchHeader."Currency Code",
                      TotalPurchLine."Line Discount Amount", PurchHeader."Currency Factor")) -
                  TotalPurchLineLCY."Line Discount Amount";
                "Inv. Discount Amount" :=
                  Round(
                    CurrExchRate.ExchangeAmtFCYToLCY(
                      Usedate, PurchHeader."Currency Code",
                      TotalPurchLine."Inv. Discount Amount", PurchHeader."Currency Factor")) -
                  TotalPurchLineLCY."Inv. Discount Amount";
                "VAT Difference" :=
                  Round(
                    CurrExchRate.ExchangeAmtFCYToLCY(
                      Usedate, PurchHeader."Currency Code",
                      TotalPurchLine."VAT Difference", PurchHeader."Currency Factor")) -
                  TotalPurchLineLCY."VAT Difference";
                "VAT Base Amount" :=
                  Round(
                    CurrExchRate.ExchangeAmtFCYToLCY(
                      Usedate, PurchHeader."Currency Code",
                      TotalPurchLine."VAT Base Amount", PurchHeader."Currency Factor")) -
                  TotalPurchLineLCY."VAT Base Amount";
                "Pmt. Discount Amount" :=
                  Round(
                    CurrExchRate.ExchangeAmtFCYToLCY(
                      Usedate, PurchHeader."Currency Code",
                      TotalPurchLine."Pmt. Discount Amount", PurchHeader."Currency Factor")) -
                  TotalPurchLineLCY."Pmt. Discount Amount";
            end;

            // Modified Event begin
            // OnRoundAmountOnBeforeIncrAmount(PurchHeader, PurchLine, PurchLineQty, TotalPurchLine, TotalPurchLineLCY);
            OnRoundAmountOnBeforeIncrAmount(PurchHeader, PurchLine, PurchLineQty, TotalPurchLine, TotalPurchLineLCY, xPurchLine);
            // Modified Event end

            IncrAmount(PurchHeader, PurchLine, TotalPurchLineLCY);
            Increment(TotalPurchLineLCY."Unit Cost (LCY)", Round(PurchLineQty * "Unit Cost (LCY)"));
        end;

        OnAfterRoundAmount(PurchHeader, PurchLine, PurchLineQty);
    end;

    [IntegrationEvent(false, false)]
    local procedure OnRoundAmountOnBeforeIncrAmount(PurchaseHeader: Record "Purchase Header"; var PurchaseLine: Record "Purchase Line"; PurchLineQty: Decimal; var TotalPurchLine: Record "Purchase Line"; var TotalPurchLineLCY: Record "Purchase Line"; var xPurchaseLine: Record "Purchase Line")
    begin
    end;

#11756
Thanks!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Translation for "Current" and "Upcoming"

Hello
and thank you for your work and the possibility of using your module.
May I ask you how to change the language of those 2 terms on the Mirror?

It seems that they are "hard coded", or I do not know how to show it in the right language.
Thank you for your interest and your Help.
Kind regards,
Cédric

Hi @1Cedric1 !
Please take a look at https://github.com/MMM-CalendarExt2/MMM-CalendarExt2/issues/79#issuecomment-627224236 - where would you expect that documented, so that you'd found it?

Thank you @klaernie,
I will try it.
I suppose https://github.com/MMM-CalendarExt2/MMM-CalendarExt2/blob/master/docs/Localization.md is the right place to put this information, as it is in fact already the case.
Thank you very much for your job and have a good week-end.


loop back address getting reported by running crust example on Ubuntu

On Ubuntu, I can still see loop back address getting reported by running crust example and not the local address.
This is seen even after the fix for it already in the code (https://github.com/maidsafe/crust/pull/104)
$ cargo run --example crust_node
   Compiling crust v0.0.8 (file:///home/prakash/rust/maidsafe/crust)
     Running `target/debug/examples/crust_node`
Listening for new connections on Tcp(V4(<IP_ADDRESS>:34005)), and listening for UDP broadcast on port 9999.
Didn't bootstrap to an existing network - this is the first node of a new network.

Enter command (stop | connect <endpoint> | send <endpoint> <message>)>

This is fixed with addition of getifaddrs


Decide on full functionality of UI

Functional discussions completed. See wireframe image for reference to full feature list.


Unable to upload large size files to Azure Page Blob

Hi, I am unable to upload large sized files (> 5 GB) to the Page Blob service. We are using the
put_page_blog_from_path method to do this, and consistently getting the following error:
ERR 104: Connection reset by peer
On one occasion, the connection was reset 68 times.
I want to know if there's anything we're doing incorrectly or is it an API issue. The same file got uploaded using the Windows Azure Powershell tool.

Are you using the latest from the master branch?  If not, you should try that.  If yes, how many connections are using?  Do you get more/less/same number of errors as you lower the number of connections?

Hi Hugues, I tried the upload with the latest master branch. The image of size 2 GB got uploaded in ~20 minutes with 5 connections.
The connection reset error we faced was with the older version of the SDK.
Let me run multiple uploads to validate the new branch, and will get back to you.
Thanks

Hi , I also tried to upload with the latest master branch. 5 GB got uploaded in ~35 minutes with 8 connections.
We faced issues of connection reset with old sdk.
Thanks

So it sounds like it's working for both of you, but that's kind of slow.  Is it faster with powershell or azcopy?  Of course I don't know how fast of an internet connection you have between your machine and your storage account...
Last week, when I tried from a Linux Azure VM to a storage account in the same data center (which should be super fast), I got around 60-70MB/sec upload, 50MB/sec download.
The latest in master branch has retries, so it's possible you are still getting connection reset, but since it retries, it ends up succeeding.  If that's the case, the errors will slow down your upload (it sleeps 1 sec between retries, you can control that via parameter).
You could modify the code to make it print when there are retries, so you'll see if that's the case.
https://github.com/Azure/azure-sdk-for-python/blob/master/azure/storage/__init__.py#L1035


high GPU memory usage with tf.data.Dataset.from_tensor_slices

System information

OS Platform and Distribution: Linux 5.13.0-28-generic
TensorFlow installed from: source
TensorFlow version: 2.8.0
Python version: 3.8.10
Installed using: conda
CUDA version: 11.5
GPU model and memory: NVIDIA GeForce RTX 3090 24 GB
CPU model: Intel(R) Core(TM) i9-10980XE CPU @ 3.00GHz

When I run the following code on Jupyter Notebook, it uses 22651 MB GPU memory when viewed with nvtop:
import tensorflow as tf
dataset = tf.data.Dataset.from_tensor_slices([1, 2, 3])

It also prints the following message:
2022-02-16 11:55:26.100430: I tensorflow/core/platform/cpu_feature_guard.cc:151] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 AVX512F FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2022-02-16 11:55:26.576039: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1525] Created device /job:localhost/replica:0/task:0/device:GPU:0 with 22232 MB memory:  -> device: 0, name: NVIDIA GeForce RTX 3090, pci bus id: 0000:17:00.0, compute capability: 8.6

I found other issues with similar messages, but they did not have the same memory issue.

I made a typo by passing lowercase 'gpu' instead of 'GPU' as the device_type, but it went unnoticed as there was no error. Fixing that solved the issue.
gpus = tf.config.experimental.list_physical_devices(device_type='gpu')
for gpu in gpus:
    tf.config.experimental.set_memory_growth(gpu, True)


Missing Ui Elements

On implemented pdp (branch pdp) the elements for priceOld and the label for the color chooser are missing.
They are not empty, but are also missing if given a hardcoded String.

@JulianSamarjiev can you have a look please?

@peter-gerhard Is this still the case after the design update?

@schleichardt I fixed the color label, the element for priceOld is still missing

@JulianSamarjiev can you please check if the priceOld is still missing? thank you


updated naming for iOS -> Touch ID and Face ID

Description

Motivation and Context

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation fix (typos, incorrect content, missing content, etc.)

Checklist

[ ] I have read the contributing document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Yep! Done!
On Fri, Aug 20, 2021 at 8:48 AM Nancy Sweeney @.***>
wrote:

@.**** approved this pull request.
This looks good to me, @linafarid https://github.com/linafarid.
@lauramthomas https://github.com/lauramthomas -- are you cool with
merging this update?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/saucelabs/sauce-docs/pull/609#pullrequestreview-735048353,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANMRAYZG2ZXYDZNZS6YJUALT5ZMKRANCNFSM5CIIGT5A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

--
Laura Thomas


add something like semantic.gs?

I like semantic.gs approach to the grid system. https://github.com/twigkit/semantic.gs
Wouldn't this be a nice feature to add to the bootstrap?

Mark's working on some cool grid stuff for 2.0 - responsive, etc. But we'll take a look at what they're doing, thanks!

Just to be clear. You can use makeColumn to make a column.
// makeColumn can be used to mark any element (e.g., .content-primary) as a column without changing markup to .span something
.makeColumn(@columnSpan: 1) {
  .gridColumn();
  .columns(@columnSpan);
}

and to make a row just .row I guess.

gagarine, this doesn't work. The closest I could get was this for twitter bootstrap 2:
//custom-mixins.less
.col (@col) {
#gridSystem > .gridColumn(@gridGutterWidth);
#gridSystem > .columns(@gridGutterWidth, @gridColumnWidth, @gridRowWidth, @col);
}
//styles.less
.greetings {
.col(3);
}
lessc will generate the following in styles.css
.greetings {
float: left;
margin-left: 20px;
width: 220px;
}

Did you try?:
.myclassCoumn{
makeColumn(2)
}

What CSS this will generate?
I will like to reopen the issue because @fat didn't really answer the question but I don't have the right for that.


CSS theme causing conflict

First of all, congrats for the theme, it's very clean and elegant!
Using the theme I realized it's CSS it's not well encapsulated. For example, I was building a bootstrap for a list, and the css of the theme conflicted with my own.
Here's a screenshot:

How can I fix this?

Hi @victornovais
Thanks for reporting this. I've improved the encapsulation of the theme styles. Can you test your styleguide using this theme's master? I hope it fixes your issues will release a new version if everything is okay.
Mathis

Hi - echoing above, it's a really cool theme and saving me a bunch of time so thanks and congrats!
I also have a small issue with the encapsulation of  tags in the content though, I guess caused by this:
.hgt-pagehead-nav a, .hgt-content a, .hgt-footer a {
  color: #4183C4;
  text-decoration: none;
}

.hgt-content a:hover, .hgt-footer a:hover {
  text-decoration: underline;
}

For example if I was using something like:

    <button class="btn-default">login</button>
    login

For now, I've just removed the CSS affecting the a tags in .hgt-content as follows:
.hgt-pagehead-nav a, .hgt-footer a {
  color: #4183C4;
  text-decoration: none;
}

.hgt-footer a:hover {
  text-decoration: underline;
}

But if you have a better idea, let me know.

Yeah, same issues with fonts etc in the .hgt-content div.

Although this comment is roughly 4 months old, this issue still exists today. From what I've found, there aren't too many suggestions out there.
Thought I'd throw my hat in the ring.
I see this problem often with hologram themes; that is, issues related to CSS specificity. It can result in:
1) The hologram theme inheriting application styles:

2) The code examples inheriting the hologram theme styles:

Both are bad.
The only way to assuredly avoid this by relying solely on CSS specificity would be to ensure uniqueness of every style definition, both in the hologram theme as well as in the application CSS. This, of course, is not a real solution. Themes can be developed by anyone, who may adhere to whatever style guidelines they desire. Application code needs to consider CSS inheritance among other things for scaling purposes, and can not feasibly specify unique style definitions for every.single.element.
For past projects, my "workaround" was to avoid contributed hologram themes entirely, and use the application's CSS to build out the hologram theme. This alleviates the concern of conflicts, but adds overhead for each new project. Additionally, there are really nice themes that already exist, such as the hologram-github-theme.
Finally, I designed a solution that works consistently, requires little effort, can be reused for all projects, and can be applied to any hologram theme. It guarantees complete isolation between the theme CSS and the application CSS. Sound too good to be true? Then you might laugh at the solution:
1) First, replace the default markup_example_template.html.erb, which looks like this:

    <%= rendered_example %>

<%= code_example %>

...with a new file by the same name, that uses an iframe's srcdoc attribute to build a completely isolated document context:

    <iframe srcdoc='
      <link rel="stylesheet" href="content/main.css">
      <%= rendered_example %>
    '>

<%= code_example %>

You will notice a reference to the app's stylesheet in the  section of the isolated document. This only exists because it was copied over as a dependency:
# @file hologram_config.yml
dependencies:
  - ../build/development/content

Note that css_include should NOT reference the app's stylesheet, else it will be loaded into the context of the hologram theme.

2) Next, include a script file to handle correctly sizing the iframes:
# @file hologram_config.yml
js_include:
  - support/globals.js

// @file globals.js
window.onload = resizeFrames;

function resizeFrames() {
  var frames = document.getElementsByTagName('iframe');

  // set the height of the iframe to the height of its content
  for (var i = 0; i < frames.length; i++) {
    frames[i].height = 0; // Allows frames to shrink if content height < default frame height
    frames[i].height = frames[i].contentWindow.document.body.scrollHeight + 'px';
  }
}

3) Finally, include a css file to correct the width of the iframe, and make it vanish from sight:
# @file hologram_config.yml
css_include:
  - support/globals.css

/* @file globals.css */
iframe {
  border: 0;
  width: 100%;
}

That's it! The document scope for each individual rendered example block is completely isolated from the hologram theme. There is also the added benefit of the example blocks being isolated from one another. This allows for example-specific CSS that doesn't have to be wildly specific to ensure it doesn't leak over. For example, you might have this as your html_example block:

I am left-of-center

I am right-of-center

These are class definitions that really only apply to the single example, and can make the example more clear by not having inline styles.
This may not be the most elegant solution, but it gets the job done, and does it well. This should enable everyone to focus on more important problems.

Hi Justin. Thank you for your detailed input! Looks like a viable solution to me. I will try it out and then discuss with @hupf

I'm already working on a solution with more specific CSS rules that I'll be able to push when I find some more time. We might also try to approach with the ``ìframe```, that sound legit (indeed, some elements are not easily addressable).

@RolandStuder - Good luck!
@hupf - Thanks for responding. Completely unique style definitions will certainly help mitigate the issue of the code examples inheriting the hologram theme styles, but there is still the issue of the theme inheriting the application styles.
The only way I could find true isolation was with an iframe, and I don't have change how I architect my CSS.

Alright, I made the css rules more specific, now they should no more conflict with the content css. I think this solution is fine, since the ``ìframe``` approach is quite hacky.
@justinhelmer: can you test your styleguide with the version on master? I'll create a new release if it's fine.

I understand the iframe solution feels hacky, I would not make it the default, but it could be a viable option. Thanks @hupf for the update.


[Watson][a156521d-a6b8-9b7c-c5be-df434f98fb1c]CLR_EXCEPTION_System.IO.FileNotFoundException_80070002_System.Diagnostics.FileVersionInfo.dll! System.Diagnostics.FileVersionInfo.GetVersionInfo

Generated issue from Watson
Watson Link:  https://watsonportal.microsoft.com/Failure?FailureSearchText=a156521d-a6b8-9b7c-c5be-df434f98fb1c
ADO Link: https://microsoft.visualstudio.com/DefaultCollection/OS/_workitems#id=27174620&triage=true&fullScreen=true&_a=edit
Process: powerlauncher.exe,
Module: system.diagnostics.fileversioninfo.dll v4.700.19.60801

This is similar to #6420 and is already fixed in #6456


README typo fix

fixed typo in line 9 of README.md
changes:
Sate -> State

fix: Typo in line 9 of README.md
changes Sate to State


{docs} List modules owned by CLI team

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

docs

@dbradish-microsoft, thanks for the suggestion. We'll put your suggestion as an improvement in our backlog and revisit it when we have bandwidth.


Comment on: "Pex doesnt add Users to the file"

i dont want to edit the user permissions. i want that users were added automatically to the user list.

Then change the create-user-entries option in the config to true


Slime Chunk limit

Slime Chunk doesn't show up in 10240 blocks away from spawn
forgot to screenshot it :<
Game Version: 1.20.73
Platform: Mobile

Yeah. It won't. In 2 years with 10,000+ downloads you are the 2nd person to
complain about that. We have to pick a distance where it stops as the map
is precomputed. And it gets exponentially more work as the radius is
increased.
On Sun, May 12, 2024, 3:03 AM Creeperuuu @.***> wrote:

Slime Chunk doesn't show up in 10240 blocks away from spawn
forgot to screenshot it :<
Game Version: 1.20.73
Platform: Mobile
—
Reply to this email directly, view it on GitHub
https://github.com/RavinMaddHatter/Bedrock-Technical-Resource-Pack/issues/12,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBUC5AP642KRXALBWJ5R7TZB446DAVCNFSM6AAAAABHSVUZAGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI4TCMRWGI3DINY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>