[joy-ui] Overriding Drawer Size throws browser error in Dev build

Steps to reproduce
Link to live example: https://talosic.net
Note: The error is only thrown in a dev build (served under npm run dev) and not with a production build. The live example is a production build for obvious reasons so won't manifest the error
Steps:

Static app built using @mui/joy 5.0.0-beta.48
main.tsx wraps all content in a theme component

  <React.StrictMode>
    <MaterialCssVarsProvider theme={{ [MATERIAL_THEME_ID]: MaterialTheme}}>
      <JoyCssVarsProvider theme={JoyTheme}>
        <Provider store={store}>
          <RouterProvider router={router} />
        </Provider>
      </JoyCssVarsProvider>
    </MaterialCssVarsProvider>
  </React.StrictMode>

JoyTheme includes declaration:

declare module '@mui/joy/Drawer' {
  interface DrawerPropsSizeOverrides {
    "xl": true,
  }
};

JoyTheme also includes styleOverride in the extendTheme components definition:

    JoyDrawer: {
      styleOverrides: {
        root: ({ ownerState }) => ({
          ...(ownerState.size === 'xl' && {
            "--Drawer-horizontalSize": "1600px",
          }),
        }),
      },
    },

Page includes a drawer defined as follows:

<Drawer
    size="xl"
    anchor="right"
    variant="plain"
    open={false}
    onClose={() => {
        //
    }}
    slotProps={{
    content: {
        sx: {
            bgcolor: 'transparent',
            p: { md: 3, sm: 0 },
            boxShadow: 'none',
        },
    },
    }}
    disableEscapeKeyDown={true}
>
    <Sheet
        sx={{
            borderRadius: 'md',
            p: 2,
            display: 'flex',
            flexDirection: 'column',
            gap: 2,
            height: '100%',
            overflow: 'auto',
        }}
    >

    </Sheet>
</Drawer>

When size is 'xl' the browser throws an error reporting that:
Warning: Failed prop type: Invalid prop size of value xl supplied to ForwardRef(Drawer2), expected one of ["sm","md","lg"].

The error is not thrown when 'lg' is specified.

A similar size override has been defined for @mui/joy/CircularProgress with no error generated using custom and standard sizes

Current behavior
Function is correct, and the drawer is the correct size, but an error is thrown when served in dev mode
Expected behavior
Same functionality but without warning
Context
Trying to allow the use of XL sized drawers
Your environment

npx @mui/envinfo
  System:
    OS: Linux 5.14 Red Hat Enterprise Linux 9.3 (Plow)
  Binaries:
    Node: 20.11.1 - /usr/bin/node
    npm: 10.2.4 - /usr/bin/npm
    pnpm: Not Found
  Browsers:
    Chrome: Version 126.0.6478.127 (Official Build) (64-bit)
    Edge: Version 126.0.2592.102 (Official build) (64-bit)
  npmPackages:
    @emotion/react: ^11.11.1 => 11.11.4
    @emotion/styled: ^11.11.0 => 11.11.5
    @mui/base:  5.0.0-beta.40
    @mui/core-downloads-tracker:  5.16.4
    @mui/joy: ^5.0.0-beta.2 => 5.0.0-beta.48
    @mui/material: ^5.14.14 => 5.16.4
    @mui/private-theming:  5.16.4
    @mui/styled-engine:  5.16.4
    @mui/system:  5.16.4
    @mui/types:  7.2.15
    @mui/utils:  5.16.4
    @types/react: ^18.2.15 => 18.3.3
    react: ^18.2.0 => 18.3.1
    react-dom: ^18.2.0 => 18.3.1
    typescript: ^5.1.6 => 5.5.3

Search keywords: DrawerPropsSizeOverrides JoyDrawer

@siriwatknp may I ask you to take a look at this one as it related to Joy's Drawer

Thanks for reporting the issue! Joy UI development is temporarily on hold as the maintainers focus on the next two major releases of Material UI


Fixed problem with jsoniter sorting

Refactored the jsoniter initialization code inside a single entry point in global utility package,
Modified the jsoniter configuration to use combination of ConfigFastest  and ConfigFastest  flags,
Verified that addressed both label and resource viewer issue from #2133,
Added unit test to make sure json marshaling is properly sorting fields,

Signed-off-by: Milan Klanjsek<EMAIL_ADDRESS>Fixes #2133

If we want to do this, define our own internal json package, like this.
diff --git a/internal/json/json.go b/internal/json/json.go
new file mode 100644
index 00000000..2c3c26ca
--- /dev/null
+++ b/internal/json/json.go
@@ -0,0 +1,15 @@
+package json
+
+import jsoniter "github.com/json-iterator/go"
+
+var jsonConfig = jsoniter.Config{
+       EscapeHTML:                    false,
+       MarshalFloatWith6Digits:       true,
+       ObjectFieldMustBeSimpleString: true,
+       SortMapKeys:                   true,
+}.Froze()
+
+var NewEncoder = jsonConfig.NewEncoder
+var Unmarshal = jsonConfig.Unmarshal
+var Marshal = jsonConfig.Marshal
+var MarshalIndent = jsonConfig.MarshalIndent

Thanks Wayne. I generally try to avoid shadowing packages, but on the second look I think you are right, that's better way of doing it.

@wwitzel3 PTAL


Wrong input argument description for tt_lib.tensor.embeddings in documentation

It was noticed that arguments for this operation were changed.
In documentation:
https://tenstorrent-metal.github.io/tt-metal/latest/ttnn/dependencies/tt_lib.html#tt-lib-api-through-tt-lib
It is written the following for the input tensor:
Each element greater than 0 and less than number of embeddings in table. Shape [batch_size, 1, num_rows, 1]
However, the sweep tests complain that only dims 0 and dim 3 can be different than 1. On the other side, from documentation it seems that it holds for dims 0 and 2.
However, in the unit test, another input shape was used which seems correct considering the error message
input_rows_shape = [batch_size, 1, 1, num_rows]
in tt_metal/tests/tt_eager/python_api_testing/unit_testing/test_embedding.py

This was a recent change and docs weren't updated. It was switched from num_rows being on second last dim to the last dim. Will update


edits to kinetic energy documentation

I was reading the documentation at the top of the file
cosima-cookbook/diagnostics/kinetic_energy.ipynb
I suggest the following edits to the "theory" section.
1/ For a hydrostatic ocean like MOM5, the relevant kinetic energy per mass is 0.5* (u^2 + v^2).  The vertical velocity component, w,  does not appear in the mechanical energy budget. It is very much subdominant.  But more fundamentally, it simply does not appear in the mechanical energy buget for a hydrostatic ocean.  So there should be no w as part of the documentation.
2/ It is stated that the kinetic energy per mass is 0.5rho(u^2 + v^2 + w^2).  In fact, the kinetic energy per mass has no rho factor (and the w should be dropped for hydrostatic).
Steve

Yep.  The docs for the kinetic energy were partially written ... I'll add in both of these points.


LSP/Vscode : how to connect vscode client to Ocaml server

I want to create a client side which is a vscode extension in order to communicate with the language server implemented using Ocaml by LSP. After looking at the LSP sample in vscode documentation i find those lines of code that i couldn't adapt to my situation :
`// The server is implemented in node
let serverModule = context.asAbsolutePath(path.join('server', 'out', 'server.js'));
// The debug options for the server
// --inspect=6009: runs the server in Node's Inspector mode so VS Code can attach to the server for debugging
let debugOptions = { execArgv: ['--nolazy', '--inspect=6009'] };
// If the extension is launched in debug mode then the debug server options are used
// Otherwise the run options are used
let serverOptions: ServerOptions = {
run: { module: serverModule, transport: TransportKind.ipc },
debug: {
module: serverModule,
transport: TransportKind.ipc,
options: debugOptions
}
};`
Because i don't have server.ts file in my project where can i get the server options which i need in order to create the language client?

Hi, @houdaM97. Thank you for your interest in the language server protocol.

Your question appears to be geared towards Visual Studio Code. I suggest you open your question in the Microsoft/vscode-languageserver-node repository. This repository is for discussing general matters related to the protocol and not about Visual Studio Code specifically.
You should also state which language server you are using. It's not quite clear from your title and description whether you are trying to integrate a language server with OCaml support or a language server written in OCaml.

hi @rcjsuen,
thank you for your reply,
i open my question in the repository suggested , and yes my language server is written in OCaml.


StickyWindow and sizing.

Hi Michel and thanks for this library. I'm trying to add some squares to a game window using a StickyWindow and using notepad for testing. No matter what size I tell the StickyWindow to be it always takes up the entire size of the ParentWindowHandle, is this how it's suppose to work? I planned on creating a StickyWindow for each square I need with the size I need and at the coords I need. Should I instead be creating one transparent StickyWindow that takes up the whole game window and drawing each square within that one StickyWindow? And if so then what's the sizing used for if it always takes up the entire parents window anyway? Thanks.
Process[] processes = Process.GetProcessesByName("notepad"); Console.WriteLine(processes[0].MainWindowHandle.ToString()); _window = new StickyWindow(100, 100, 200, 150) {   IsTopmost = true,   IsVisible = true,   ParentWindowHandle = processes[0].MainWindowHandle, };

I've swapped it all to one StickyWindow and drawn everything I need on there instead of drawing seperate windows. Still curious about the size arguments for StickyWindow() though.

Yes it is supposed to take the entire parent window size. The client size of a window is its size subtracted any borders and the title bar.
Can you name the arguments you are curious about?
It is much better to just create a single window. You can manage the bounds of the rectangles yourself.

The arguments I was curious about were the size and position args for public StickyWindow(int x, int y, int width, int height) if it's always going to take up the whole size and position of the parent anyway.

this constructor is inherited from the base class

Thanks for the replies, I'm quite new to c# and this lib has helped a lot with my first project..

you can leave the issue open for a while if you may have any new questions. i will close it at some point when it's inactive


⚠ ClamAV has degraded performance

In 2143d3c, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 439 ms

Resolved: ClamAV performance has improved in c5d268d.


Pretrained models do not give results on evaluation

I am trying to run DeepLoc_eval.py script on pretrained models such that --logdir parameter is set to pretrained_DeepLoc/pretrained_models where pretrained models are saved. I ran this script on 2 batches (implementation is not changed).  I get total accuracy of 0.09375 and total loss 0.291712125391, what looks like model would not be trained. Is that normal or am I doing anything wrong?

Can you provide more details of the commands you ran?

Sorry for bothering you. The bug happened when I tried to cast scripts to Python 3.


bugfix: attach hanging of cri container

Signed-off-by: YaoZengzeng<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
If we want to get the log of cri container, the step as follows:

create container

attach to the io of the container

start the container

Then the IO of the container will be created in step 2
So we should decide whether open the Stdin of Container IO, otherwise it will never be opened
which will result the hanging of attach
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #2130 into master will decrease coverage by 0.04%.
The diff coverage is 72.22%.

@@            Coverage Diff             @@
##           master    #2130      +/-   ##
==========================================
- Coverage   64.88%   64.84%   -0.05%
==========================================
  Files         208      208
  Lines       16458    16465       +7
==========================================
- Hits        10679    10676       -3
- Misses       4451     4459       +8
- Partials     1328     1330       +2

Flag
Coverage Δ

#criv1alpha1test
33.04% <55.55%> (-0.12%)
:arrow_down:

#criv1alpha2test
33.8% <16.66%> (-0.02%)
:arrow_down:

#integrationtest
39.5% <0%> (-0.06%)
:arrow_down:

#unittest
23.93% <0%> (-0.02%)
:arrow_down:

Impacted Files
Coverage Δ

cri/v1alpha2/cri.go
64.63% <100%> (ø)
:arrow_up:

cri/v1alpha2/cri_utils.go
82.36% <100%> (+0.02%)
:arrow_up:

cri/v1alpha1/cri.go
63.35% <50%> (-0.79%)
:arrow_down:

cri/v1alpha1/cri_utils.go
83.33% <69.23%> (-0.32%)
:arrow_down:

apis/server/utils.go
61.9% <0%> (-4.77%)
:arrow_down:

ctrd/image.go
76.75% <0%> (-2.2%)
:arrow_down:

daemon/mgr/container.go
56.04% <0%> (ø)
:arrow_up:

ctrd/watch.go
80.3% <0%> (+4.54%)
:arrow_up:


Bolt decided to delete all the previous changes to fix a bug.

Describe the bug
Bold decided to delete all the previous changes to fix a bug and delete the latest work and take it as an instruction when I clicked on fix bugs.
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-z633vj
Steps to reproduce

Create a feature
Get an error
Fix the error
Bolt deleted the previous work

Expected behavior
Take in consideration the previous work and make sure to not get back
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 129
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 154569432,
  "usedJSHeapSize": 150534136,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = nans64
Chat ID = b4302a48f4b3

Additional context
No response

You're right; this happens to me quite often. Trying to fix those bugs consumes a lot of tokens, which makes them run out quickly. Even when it's specified not to delete something that has already been done, it insists on removing it, which is really frustrating

Appreciate your patience during this beta period as we work to improve the platform! This issue, some powerful temporary workarounds, and it's resolution is being tracked in issue #24


Update README.md with audio generation steps

Added a section on the steps taken to generate AAC / AC4 / EAC3 audio content.

Thanks!
The text is fine. However it doesn't allow to fully reproduce (based on pre-encoded content for properietary codecs). Personally I would have found great that it re-uses the existing code to package.
Waiting for others' approvals to merge.

My main comment is that, for Dolby encoding, the reference to Hybrik is to a tool available to anyone under commercial terms. There's not really enough information there for someone else to regenerate the content.

Hi @rbouqueau, thanks for the feedback. I understand your position and agree that a single script would be the ideal. Unfortunately it's not the way we have generated the content and changing now would require significant re-work. (Our commercial obligations were to produce appropriate audio for tests - which has been done).
Apologies for this, but I would be grateful if you can merge this PR. I can then update with further details re Dolby encoding using Hybrik as requested by @jpiesing.
Thanks, Richard.

Hi @rbouqueau please can you merge the changes in patch-2 (added more info on encoding Dolby). Many Thanks, Richard.

@rcottingham sorry for the delay. I'm concerned about the reproducibility and maintenance. Thus would it be ok to add yourself/your-company as a maintainer at the top of these sections?


1.16.5?

you really need to stop creating tonnes of issues in different repos asking for 1.16 support. in almost every case, if its not for the version you want, it will likely never be. 1.16 is rather redundant at this point when there is 1.18+ and thus developers choose to focus on those versions rather than having to maintain support for multiple different versions.


Convert grpc_ares_wrapper to C++

The primary change here is to have grpc_dns_lookup_ares_locked return a unique_ptr<AresRequestInterface>.
Conversion of the remaining C-code in grpc_ares_wrapper.cc is left for more followups.

This change is

Comments addresses (though with the ref counting scheme still in discussion).
Thanks for the review so far. A heads up that there still seems to be something happening wrong on Windows, but I think this PR can still otherwise be good for another look.

Comments addresses (though with the ref counting scheme still in discussion).
Thanks for the review so far. A heads up that there still seems to be something happening wrong on Windows, but I think this PR can still otherwise be good for another look.

Manually squashed commits and force pushed

artifact build windows: https://github.com/grpc/grpc/issues/25545

basic tests c#: #25748
python windows: https://github.com/grpc/grpc/issues/25749
portability tests windows: https://github.com/grpc/grpc/issues/25751


Strange(?) behaviour of Tupler on HList of HList

I am kind of confused about the behaviour of Tupler on nested HList:
val l = 1::2::HNil ::3::4::HNil
l: shapeless.::[Int,shapeless.::[Int,shapeless.::[shapeless.HNil.type,shapeless.::[Int,shapeless.::[Int,shapeless.HNil]]]]] = 1 :: 2 :: HNil :: 3 :: 4 :: HNil
l.tupled
(1,2,HNil,3,4)

Whereas I'd expect something like
l.tupled
((1,2),(3,4))

Is this the intended behaviour?

I don't think your l is what you think it is. You are actually putting the HNil object into the middle of an HList.
I think what you meant to do was something like this:
l: shapeless.::[shapeless.::[Int,shapeless.::[Int,shapeless.HNil]],shapeless.::[shapeless.::[Int,shapeless.::[Int,shapeless.HNil]],shapeless.HNil]] = 1 :: 2 :: HNil :: 3 :: 4 :: HNil :: HNil

scala> l.tupled
res6: (shapeless.::[Int,shapeless.::[Int,shapeless.HNil]], shapeless.::[Int,shapeless.::[Int,shapeless.HNil]]) = (1 :: 2 :: HNil,3 :: 4 :: HNil)

The .tuple command only goes one layer deep. So if you wanted to also turn each of the two HLists into a tuple, you would have to call .tupled on them.

@ceedubs is spot on.

Right, my oversight. However, is there a reason why .tupled only does go one layer deep? Disclaimer - I haven't looked at the implementation, just wondering.

I think a one layer operation makes sense as a primitive. Would you expect size on a List[List[Int]] to give you the number of List[Int] in the outer list or the total number of Ints in the structure?

No, but on the same token, would you expect flatten on nested Lists to
only flatten one level? Incidentally, this is what List.flatten
does...Which in my opinion doesn't make sense. This came up during the
implementation of flatten for HList - hence the opinionated view I
guess.
I am happy for this to be closed - it was just for clarification.
On 3 September 2015 at 21:21, Miles Sabin<EMAIL_ADDRESS>wrote:

I think a one layer operation makes sense as a primitive. Would you expect
size on a List[List[Int]] to give you the number of List[Int] in the
outer list or the total number of Ints in the structure?
—
Reply to this email directly or view it on GitHub
https://github.com/milessabin/shapeless/issues/454#issuecomment-137547224
.

@AndreasKostler I wonder how the signature of List.flatten would look like if it could flatten nested lists?

This is modelled after Miles' nice example for Tuples...
trait LowPriorityFlatten extends Poly1 {
    implicit def default[T] = at[T](_::HNil)
}
object flatten extends LowPriorityFlatten {
    implicit def caseTuple[L <: HList](implicit fm: FlatMapper[flatten.type, L]) = at[L](_.flatMap(flatten))
}

The current Tupler/.tupled are behaving as designed, and I think they should stay that way, so I'm going to close this issue for now.
But a recursive operation would also be useful, and I'd be happy to be on the receiving end of a PR that added one :-)

;) I'll see what I can do
On 10 September 2015 at 11:04, Miles Sabin<EMAIL_ADDRESS>wrote:

The current Tupler/.tupled are behaving as designed, and I think they
should stay that way, so I'm going to close this issue for now.
But a recursive operation would also be useful, and I'd be happy to be on
the receiving end of a PR that added one :-)
—
Reply to this email directly or view it on GitHub
https://github.com/milessabin/shapeless/issues/454#issuecomment-139176342
.


auto-nag is moving bugs back into components they were moved out of by humans

e.g. https://bugzilla.mozilla.org/show_bug.cgi?id=1580658#c2
There were a few of these today. New Tab Page is definitely not the right place for this bug, so unsure why bugbug is insisting it needs to go there.

The Bugzilla bug has since been fixed.


fix code sections

Requested review from @timja for a check on the regex expression,  IIRC we wanted it to be a bit more verbose, and postponed the discussion for later.

I would say it's up to the users but my suggestion for the schema matching and what I will use for our use case would be
{
  "yaml.schemas": {
    "jcasc-schema.json": "*.jcasc.yml"
  }
}


Add support to the battor cli so that way it honors the passed in tty parameter

Also, show the character for the default gain in usage.

Any updates on this? I'm happy to pull it in once those two changes are made.

Comments addressed.

lgtm


[Python] [LGTM.com] False-positive: py/hash-unhashable-value when indexing pandas.DataFrame with list

Description of the false positive
LGTM flagged this snippet
data = data[list(vectors)]

where data is a pandas.DataFrame as an alert "This instance of list is unhashable." (py/hash-unhashable-value) which I believe is a false positive.
I guess the analysis believes that the list used to index data will be hashed (I can't explain this alert otherwise)? However this is clearly not true as this is simply part of pandas API (see here).
I don't believe this alert was already raised as a false positive in another issue. If I overlooked something I apologize.
URL to the alert on the project page on LGTM.com
https://lgtm.com/projects/gl/tud-mst/ptvpy/snapshot/8523c500d4ec81cb64766e9a57efd801c6ffebe9/files/src/ptvpy/process.py?sort=name&dir=ASC&mode=heatmap#x58400f6141863731:1

Looking at the query in question it simply seems like pandas.DataFrames are not yet accounted for in the same way numpy.ndarrays are.


[Event Request] codeunit 7152 "Export Item Analysis View" - OnbeforeFillOutItem

🕮 Describe the request
This codeunit is used to export Item Analysis View.  If the event could be placed on FillOutItem Function or AddAcc Function, we would easily achieve the result without having to rebuild this complex report by developer. In Event please pass ItemVariables as well either as variant or the record itself and also IsHandled Parameter.
Additional context
Most of our customer wants to add additional details not just the Item No. and Item Description. And the separator used here by Microsoft is space character. After exporting to excel, space does not help if the customer wants split Item No. and Item Description to different columns because Item Description also might have spaces. So we want an events here so we can hook our code and add any splitting characters between Item No and Item Description. And also we would be able to add more Item Information if this event could be placed.

Hello @yuranshrestha
We can add it in procedure AddAcc(), just provide us where in the procedure and it would the best if you send us exact event structure.

Hello @AleksandricMarko, following is the event structure we would like to have in this AddAcc() function. I hope this will be available on all SAAS localization. Currently we require this for GB Localization on cloud.

    [IntegrationEvent(false, false)]
    local procedure OnBeforeAddAcc(ShowName: Boolean;var Account: Text;var AccName: Text; var IsHandled: Boolean; Item: Record Item; var TempExcelBuffer: Record "Excel Buffer" temporary)
    begin
    end;

    [IntegrationEvent(false, false)]
    local procedure OnAfterAddAcc(ShowName: Boolean;var Account: Text;var AccName: Text;var CellValueAsText: Text; Item: Record Item; var TempExcelBuffer: Record "Excel Buffer" temporary)
    begin
    end;

    local procedure AddAcc(ShowName: Boolean; Account: Text; AccName: Text)
    var
        CellValueAsText: Text;
        IsHandled: Boolean;
    begin
        OnBeforeAddAcc(ShowName, Account, AccName, IsHandled, Item, TempExcelBuffer);
        if IsHandled then
            exit;

        if Account = '' then
            CellValueAsText := ''
        else
            if ShowName then
                CellValueAsText := Account + ' ' + AccName
            else
                CellValueAsText := Account;

        FillNextCellInRow(CellValueAsText);

        OnAfterAddAcc(ShowName, Account, AccName, CellValueAsText, Item, TempExcelBuffer);
    end;

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. 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.
Build ID: 13321.


How to use lid_kaldi

How to use lid_kaldi for language identification?

You can find example here https://github.com/igorsitdikov/lid_kaldi/blob/master/python/example/test_simple.py

Where to get lid-model in the example.
I am getting this error in the example:
from lid import Model, KaldiRecognizer
ModuleNotFoundError: No module named 'lid'

Hi! Thx @igorsitdikov for your work! I have related question.
I installed whl on my PC. But when I try to run example, I got an error
ERROR ([5.5.873~1-75ec]:ReadConfigFile():parse-options.cc:462) Cannot open config file: lid-model/mfcc.conf

I suppose I need to get somewhere lid-model folder. Where I can get it?

@asadullah73-ce install whl file for your platform from releases page.
For example
pip install https://github.com/igorsitdikov/lid_kaldi/releases/download/1.0.1/lid-1.0.1-py3-none-manylinux2010_x86_64.whl

Ok, I figured out with my problem. I was misled because actual model was in version 1.0.0 and actual whl in 1.0.1.

@igorsitdikov thx a lot!

One more question please.
Does this detect other languages e.g Hindi or Urdu or Arabic?

Okay, I was able to detect Urdu language from the code.


Error on nightly: Unions in const fn are unstable

I'm getting this error on recent nightlies:
error[E0658]: unions in const fn are unstable (see issue #51909)
  --> /Users/josef/.cargo/registry/src/github.com-1ecc6299db9ec823/heapless-0.3.6/src/__core.rs:43:9
   |
43 |         U { none: () }.some
   |         ^^^^^^^^^^^^^^^^^^^
   |
   = help: add #![feature(const_fn_union)] to the crate attributes to enable

error: aborting due to previous error

I'm now using the master branch in my project which allows disabling the const-fn crate feature.

@japaric This is awesome! Thanks!


MicroHack/03-Azure/01-03-Infrastructure/02_Hybrid_Azure_Arc_Servers/Challenge 2

Walkthrough references now outdated legacy method, should be moved to Data Collection Rules.
To mitigate, update screenshots to reflect portal changes:
Portal:

Currently in docs:

@adriandiver is it possible to update the screenshot to reflect the new portal experience?


SOF-1423 Revert updateFunction back to not fetch .value

After we are able to decide wether or not the EditAttributeDropdown component will save the data with a label or not, we need to revert this usage of the EditAttributeDropdown back since it does no longer receive { value: string, label: string }

Codecov Report

Merging #239 (12d1eed) into staging (a47683c) will decrease coverage by 5.51%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##           staging     #239      +/-   ##
===========================================
- Coverage    68.15%   62.64%   -5.51%
===========================================
  Files          280       13     -267
  Lines        34083     2613   -31470
  Branches      4569      188    -4381
===========================================
- Hits         23228     1637   -21591
+ Misses       10391      975    -9416
+ Partials       464        1     -463

see 267 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[elixir] fix: further filter objects coming in for ingestion

Because S3 treats folders created as objects, we need to be able to filter them out, so we don't try to ingest them. This PR addresses this issue.

What does the S3 data look like from the API? I'm surprised we didn't come across this issue before, and I thought the Delimiter part of the query caused prefixes to be returned separately?

They look like regular objects, but with a size of 0, and the key being the folder path. We didn't run into it because we mostly moved objects in through aws s3 cp or some other mean where the actual files were being moved/copied. What Cubic/Qlik did is that they created the folder first and then uploaded. It's similar to using the 'Create Folder' button on the S3 AWS console.


Upgrade template verified

What was wrong?
The project template was updated. This pulls in those changes. This is a re-do of PR #49, fixing the a gpg signing issue in the template history.

[ ] Add entry to the release notes

Cute Animal Picture

Wrong history used, closing


Wrap task submission for transfer into tasks

This is a pretty large refactoring of the architecture, but I think that it is worth it and necessary in the long run as we add features. In short, the change wraps the TaskSubmitter logic that submits all of the tasks to execute a transfer into a SubmissionTask itself and these submission tasks are then executed by an executor specific for running submission tasks. This is really helpful for the following reasons:

The submission of tasks are no longer handled in the main thread, which makes it quicker to enqueue transfer request and get a transfer future back. For example, in downloads and copies you are no longer doing a HeadObject in the main thread. This is also going to be needed for streaming uploads (i.e. we do not want to stream the entire file in the main thread).
Exception handling is more robust such that if there was an error during submission before the final task was submitted, bad things could have happened such as hanging or cleanup tasks not being ran. This is no longer the case.
on_queued callbacks happen in the submission thread pool. This allows you to make slower calls in these callbacks without slowing down your speed in adding more transfer requests. This will be critical if you want to try to do complete copies for multipart which could require multiple API requests.

As to the changes that were made in the refactoring, they include:

Remove the TaskSubmitter class in place of a SubmissionTask class. They relatively have the same functionality but the SubmissionTask subclasses from the Task class so it can be ran by an executor.
Added a new executor to complete SubmissionTask that does not interfere with the main executor.
Moved the running of failure cleanups and done callbacks into the TransferCoordinator. This was changed because logically it made sense that coordinator runs the callbacks associated to it when it announces done especially since it is no longer the responsibility of a single know task to announce that the transfer is done (This can happen in the submission task if it fails or the designated final task).

Let me know if you have any questions. I tried my best to add comments and documentation at the trickier parts of the code.
cc @jamesls @JordonPhillips

Overall pretty straightforward.  Two general pieces of feedback:

It would be helpful to add .debug() logs to the various task submission/queueing parts.  There's likely to be tricky concurrency issues, and the only insight we generally get is by asking for debug logs.  Being able to see how things are getting queued/processed will make this easier to troubleshoot.
Getting to the bottom of the perf discrepancy between the develop branch as well as aws s3 cp.  Would also be worth checking if this is only affecting downloads or if uploads is affected as well.

I will add more debug logs. That makes sense.
As to the performance comparison, I will put some numbers up, but I am not seeing too big of a difference between this PR and the develop branch. So I think addressing it in a seperate PR would be more appropriate. It must be as fast as or faster than the CLI before we start migrating it.

@jamesls I tested the uploads out and they are fine. They are actually faster than the CLI.
The downloads I am still testing. I found one issue that gets the develop branch back to the speed of the CLI, where our default configuration is not inappropriate for downloads.
Consider the case where we have a fast network and are bounded more by io. Given that our default io queue size is a thousand and the default io chunk size is 8KB, then the most amount of data that can be stored in the queue is around 8 MB. However, this is a problem because each downloaded part is 8 MB as well so if the threads are downloading it really quick there is a bottleneck in the queue where the queue may fill up because the enqueuing is much quicker than the io thread is processing them. So I think we need to bump up the value of the io chunksize or io queue size to something closer to the value of (max_request_threads * request_chunksize). This makes sense too why the CLI is faster because for the CLI the max io queue size is 1000 while the io chunks are 1MB.
What is your thoughts on this? Should we bump up the default io queue size of io chunksize? After testing bumping the io chunksize to 64 KB is the best, but I see similar results by increasing the io queue size. So I would be inclined to pick the bumping of chunksize even though we may lose some granularity in the progress callbacks.
However, with this change my PR is still twice as slow as the CLI, but it is definitely faster than before. I need to do more digging on that.
Also on a side note, I commented on two other comments you had.

Hmm, after testing changing the io chunksize to 64 KB, it is actually as fast as the CLI now, I may have just not been using the correct io chunk size in the first place.

@jamesls I addressed all comments by updating the code or by responding back to the comment. Should be good to look at again.

Ah, your explanation makes a lot of sense, good deep dive on figuring that out.  I think the math works out nicely as well with the defaults:

1k max queue * 64kb IO chunks = 62.5MB max mem allocated to IO queue
8MB part sizes with 64kb UI chunks = 128 parts per thread = 1280 possible IO slots needed before threads can move on to the next part.  I'd also be cool with bumping the default queue to 1280, but I don't know if it makes a noticeable difference because that's a worst case scenario.

Looking over the rest of the changes now.

:shipit:  Looks good to me.

I updated the status documentation for running as well. Merging.


Code for "Update the Delete Razor page" is incorrect

The code shown for "Update the Delete Razor page" is incorrect.
On line 36 it has @Utility.GetLastChars(Model.Department.ConcurrencyToken) while the existing code in my project has @Html.DisplayNameFor(model => model.Department.ConcurrencyToken). The code in my project results in a more correct display when deleting a department.
The code in the tutorial is also highlighted incorrectly with line 42 highlighted instead of line 45.

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

ID: 16a5d321-117b-c37c-6ac9-b56009e7ead7
Version Independent ID: e337d2e1-7447-634a-d11a-511247e0f897
Content: Part 8, Razor Pages with EF Core in ASP.NET Core - Concurrency
Content Source: aspnetcore/data/ef-rp/concurrency.md
Product: aspnet-core
Technology: aspnetcore-data
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Associated WorkItem - 61143

@MartinNr5 Thanks again for reporting that!


Basic Validation of data fields

To ensure data consistency and integrity, we need to build in validation checks on the different data fields. This will need to be thought through thoroughly. At the database level, we could implement NOT NULL and other constraints.

I think the constraints should also be at the App level, because I think the app will shut down if the DB returns an error.

@bhsi-snm to update with status (function test results)

Level of effort: 'medium'


Expose Keymap command in Keyboard Shortcuts toolbar

Usertesting showed that keymap extensions are not easy to discover due to a mix of issues. Users have many different terms for them, don't know that they exist or don't look in the right places.
To improve organic discovery the idea here is similar to Turn on Settings Sync button in Settings, showing a Migrate your Keyboard Shortcuts from other Editors in the Keyboard Shortcuts page.
To focus on new users, it could be hidden when a user installed a keymap.

To clarify, would the command just bring up the filtered keymaps extensions view?
One possible option would be an additional icon added the toolbar e.g.

We don't yet have the right icon in codicons so consider the new icon a placeholder for now

The command is workbench.extensions.action.showRecommendedKeymapExtensions (Preferences: Keymaps)

One possible option would be an additional icon added the toolbar e.g.

I am skeptical that am icon solves the discoverability problem. The UI is already pretty dense and an icon might just blend in with everything. Could we explore a non-primary labeled button as well?

I agree. Without changing the existing toolbar I can see that the commands can also feature a label. However I've never seen this used in VS Code so I'm not sure it's the right way to go.
I'll take a look at some other options and will report back.

Oh ok, I see what you're referring to now re: Settings Sync. Hadn't noticed this since it's always been on for me :)

Secondary button inline with search

With H1
I realize this one is a bit of scope creep here. I'm actually curious how we're getting away without an H1 on this or similar pages. @misolori would you have any context there?

We tend to rely on the file tab as the "heading" for most of our single pages (settings, running extensions, etc.) and the extensions detail page is one of the exceptions of this:

Closing for now as we lost traction on this.


Duplicate ids in multi-threaded

I tried the generator (1 generator) for 1000 times for 3 threads, and it gives me duplicated values
// Create a new instance of the generator
      Generator gen1 = new Generator(128, new DateTime(2019, 05, 1));

      int max = 1000;
      List<string> ids = new List<string>();

      for (int i = 0; i < max; i++) {
        ids.Add(gen1.Next());
      }

      Task t1 = Task.Factory.StartNew(() => {
        for (int i = 0; i < max; i++) {
          ids.Add(gen1.Next());
        }
      });

      Task t2 = Task.Factory.StartNew(() => {
        for (int i = 0; i < max; i++) {
          ids.Add(gen1.Next());
        }
      });

      Task.WaitAll(t1, t2);

      var grouped = (from i in ids group i by i into g select new { Id = g.Key, Count = g.Count() });

      var duplicated = grouped.Count(g => g.Count > 1);
      Console.WriteLine("duplicated ids  " + duplicated);
      //Failed
      Debug.Assert(0 == duplicated);

Hi @mschuler, would you like the community to solve this issue?

This is by design. You have to create a generator for each thread / application.
What's the reason for creating multiple generatores? If you have a distributed microservice application, you have one generator with unique thread-id per service instance. If your application is not distributed, do you really need multiple generators?

Thanks @mschuler for your prompt response. Makes sense.

List ids = new List();

@mschuler
I have tried this issue, and i think duplicate ids is because of List are not thread safe, use ConcurrentDictionary instead List, there are no duplicate ids.


LBFGS does not work with SVI

LBFGS requires us define a closure to be called in its step method. Defining a closure to call it with loss_and_grads is easy but I can't figure out how to incorporate it with the way PyroOptim handles parameters (creating an optimizer for each param).

@eb8680 will we support it? LBFGS is a very convenient optimizer for models with small dataset! I have used it to optimize a lot of models. Many of them can not be trained with SVI.

@fehiepsi sorry, I only closed the issue because I thought it was resolved by the PR #1518.
pyro.optim.PyroOptim is a klugy wrapper for handling varying amounts or orders of parameters.  I don't see how we could modify it to take a closure argument, but I also think the way you're using torch.optim in your linked example is fine - if anything, we should be encouraging users to use that idiom instead of svi.step, since it's more explicit.

Please find a solution for this issue and discussions why such solution is not appropriate to merged in #1562.


Project status unclear, especially regarding container image releases

Hi,
I'm currently investigating potential backends for Prometheus and noticed timescale + pg_prometheus being an option.
However, I'm unsure about the project status. The last release on DockerHub was in January, which correlates with the 0.2.1 release, but isn't up to date with timescaledb releases at all. It's not very clear to me which timescaledb release is going to be included with that image, for example. Tags are very different from the main project releases. The DockerHub page hasn't any documentation too.
Is the Docker image considered actively maintained? Is building it from source with an updated timescaledb version the recommended way for production usage?

Hey @embik,
sadly it's true that Docker image is behind TimescaleDB releases. There was a lot going on... Anyways, it should be fairly easy to build a Docker image locally. I haven't personally tested it with the latest TimescaleDB but I think it should be fine.

Hey @niksajakovljevic,
thanks for the clarification, I appreciate it! I'll try what you suggested.


SA1646: IncludedDocumentationXPathDoesNotExist

The compiler already checks this (SA1589). See #165.

:question: Would it make sense to include unit tests proving it solves the intended problem? It might be possible to locate some reference tests in the Roslyn repository.

I agree with disabling this. It helps keep this project out of the business of probing the file system. :+1:


add StarCIO article

The test will pass when we merge this PR: https://github.com/mindee/devrel/pull/273

The test will pass when we merge this PR: #273

Alright


Missing latest updates

As of 12/01/2020 20:00, koroonakaart.ee still shows data uploaded on 11/01/2020. The cause of this is failing to build the docker image.

docker build log
❯ docker build -t koroonakaart .

Sending build context to Docker daemon  15.96MB
Step 1/13 : FROM python:3.7-buster AS fetch_data
 ---> 978a47fa762f
Step 2/13 : WORKDIR /app
 ---> Using cache
 ---> ffd8f1a1a824
Step 3/13 : COPY . .
 ---> Using cache
 ---> 79d19418b4f7
Step 4/13 : RUN cd TEHIK_Open_Data_Loading_Scripts     python -m pip install --upgrade pip     python main.py
 ---> Using cache
 ---> d62100a5a68b
Step 5/13 : FROM node:erbium-buster AS build_frontend
 ---> b247258c051b
Step 6/13 : WORKDIR /app
 ---> Using cache
 ---> 7b1f526f0a58
Step 7/13 : COPY . .
 ---> Using cache
 ---> 719256a4eb8a
Step 8/13 : COPY --from=fetch_data /app/data/data.json /app/data/data.json
COPY failed: stat /var/lib/docker/overlay2/6ba00adfa145cd2452879bb064dcbea0aa84d71d7774465df77c3cc6a208d42a/merged/app/data/data.json: no such file or directory

The build might fail because the latest commit aa1b5713 removed some death data, but didn't fix all references, since running the python download manually fails with

tehik download log
❯ python main.py
Traceback (most recent call last):
  File "main.py", line 67, in <module>
    json_deaths = read_json_from_file(DEATHS_FILE_LOCATION)
  File "main.py", line 43, in read_json_from_file
    with open(path) as f:
FileNotFoundError: [Errno 2] No such file or directory: '../data/deaths.json'
❯

or maybe my investigation is totally wrong and your build setup is supposed to be broken and the real cause why the server hasn't updated is something else.
@dorkster100 @okestonia any thoughts?

Hello,
Thank you for the report, i will investigate the issue further. The problem is probably not related to the error you provided. I suspect that the server did not have enough power to compute all the data, i will have to check that, Sadly TEHIK data is petty inconsistent lately.
Since docker was added by other people i am not maintaining it, server is not running on docker, so currently it can be used solely for local development, however since the stack is so simple i do not see any need for that personally.
For data update on server update_data.sh is used. For local i am simply running python3 main.py and then npm run serve.
You can probably run update_data.sh locally, however it will create a backup folder it parent directory.
I will update the README later to reflect these changes..

Hello,
Thank you for the report, i will investigate the issue further. The problem is probably not related to the error you provided. I suspect that the server did not have enough power to compute all the data, i will have to check that, Sadly TEHIK data is petty inconsistent lately.
Since docker was added by other people i am not maintaining it, server is not running on docker, so currently it can be used solely for local development, however since the stack is so simple i do not see any need for that personally.
For data update on server update_data.sh is used. For local i am simply running python3 main.py and then npm run serve.
You can probably run update_data.sh locally, however it will create a backup folder it parent directory.
I will update the README later to reflect these changes..

Oh yeah, thanks! I see now the docker file is out of date with update_data.sh.
And thank you for fixing the web page!

Oh yeah, thanks! I see now the docker file is out of date with update_data.sh.
And thank you for fixing the web page!


I don't know if it's a bug or a configuration error

Question
I don't know if it's a bug in the JRE or if it's some Tomcat configuration.
Should I add some extra configuration at the level of the default tomcat?
Note this is the first time I see this error. And that causes a complete fall of the demon.

A fatal error has been detected by the Java Runtime Environment:

SIGBUS (0x7) at pc=0x00007efdb10523b0, pid=21085, tid=0x00007efd71def700

JRE version: OpenJDK Runtime Environment (8.0_372-b07) (build 1.8.0_372-b07)
Java VM: OpenJDK 64-Bit Server VM (25.372-b07 mixed mode linux-amd64 compressed oops)
Problematic frame:
v  ~StubRoutines::jlong_disjoint_arraycopy

Failed to write core dump. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again

If you would like to submit a bug report, please visit:
https://github.com/adoptium/adoptium-support/issues

---------------  T H R E A D  ---------------
Current thread (0x00007efd8d692800):  JavaThread "ajp-nio-8009-exec-81" daemon [_thread_in_Java, id=11082, stack(0x00007efd71bef000,0x00007efd71df0000)]
siginfo: si_signo: 7 (SIGBUS), si_code: 2 (BUS_ADRERR), si_addr: 0x00007efda8018000
Registers:
RAX=0x00007efda801a000, RBX=0x0000000000000000, RCX=0x00000000a4f860d0, RDX=0xfffffffffffffc04
RSP=0x00007efd71deddd0, RBP=0x00007efd71deddd0, RSI=0x00000000a4f840d8, RDI=0x00007efda8019ff8
R8 =0x00000000a495bd80, R9 =0x00007efda8018000, R10=0x00007efdb1052940, R11=0x0000000000000010
R12=0x0000000000000000, R13=0x00000000a4f840c8, R14=0x0000000000002000, R15=0x00007efd8d692800
RIP=0x00007efdb10523b0, EFLAGS=0x0000000000010282, CSGSFS=0x002b000000000033, ERR=0x0000000000000004
TRAPNO=0x000000000000000e
Context
Java version:
openjdk version "1.8.0_372"
OpenJDK Runtime Environment (Temurin)(build 1.8.0_372-b07)
OpenJDK 64-Bit Server VM (Temurin)(build 25.372-b07, mixed mode)
Your operating system and platform:
Ubuntu 18.04.6 LTS x86_64
Tocmat 8.5.39-1ubuntu1~18.04.3

please upgrade to Tomcat 8.5.89 and reopen this if you get another crash


WIP Ds with affinity

WIP; needs some tests mainly.
What this PR does / why we need it:
Which issue(s) this PR fixes

Fixes #

Special notes for your reviewer:
Release note:
release-note

I manually tested this slightly with just a DoesNotExist case to run a plugin just on my worker nodes and it worked well. It should be quite easy to test these functions.


Server side rendering

Preliminary testing looks pretty promising. Many tests are already passing. It seems that if the author is precise how they define components, things work. So no jQuery changes of DOM, and state local to the component instance, without global state.
This pull request would help that we do not have to copy too much code from the templating package: https://github.com/meteor/meteor/pull/5903

Some questions with server side rendering:

what to do with *.html files which add things to <head> and <body>
can we use this for spiders?
and that content is rendered on the server, send to the client, and then replaced with the client dynamic version?
how to integrate routing, how to support rendering without routing?
when rendering the page, how to know what should be the value of Meteor.userId()
what to do with subscriptions from the component? should we just make component.subscribe immediately return, be a no-op, and pretend it is subscribed? how do we provide then things like subscription scope? also, publish functions tend to limit the fields available to the client, and client might assume that no extra fields are available and render everything available, with no-op implementation of subscribe, all fields would be available and this might reveal sensitive information (imagine a component which simply calls EJSON.stringify on documents)

I have a general question about SSR in Meteor in combination with BlazeComponents.
Let's say I have a shop and want everything to be static (SSR or Caching p.e. thru Ngnix), except the "add-to-shopping-cart"-functionality and "your-last-viewed-products-history"-functionality.
How would you approach this as a pro? is this currently possible in Meteor at all?

Not really sure. I have not looked yet much into server-side rendering.

I created a topic for discussion on https://forums.meteor.com/t/how-to-migrating-from-all-in-packages-to-1-3-modules/20681


Add GCE workflow and instructions

GCE deployment currently focuses only on container-optimized VMs as noted in the instructions.

GitHub ate one of your comments but I also addressed it. There were nested lists in the doc but they were not rendered correctly due to bad formatting.


[v3] navbar-fixed-* don't get right padding on modal open

When opening a modal and there is a vertical scrollbar on body,
body gets added a pading right of scrollbar width and overflow hidden,
because without it content will move left and right.
Same applies for navbar-fixed-top and navbar-fixed-bottom,
When body has overflow hidden, navbar-fixed-* is moving left and right on modal toggle

Linux 4.15.0-34-generic #37-Ubuntu SMP x86_64 x86_64 x86_64 GNU/Linux
Google Chrome Version 69.0.3497.100 (Official Build) (64-bit)
https://codepen.io/iamandrewluca/pen/VEZNdY

Fixed in #27331


Files Scene

Resolves #295 and #297
Overview
Implements file uploading and a "Project Files" scene that displays a table of files associated with the project. We use a /projects/:projectId/files/:folderId routing scheme for routing the user to different folder views based on their selection of a given folder in the UI. The :folderId is checked to make sure it is a folder associated with the :projectId
We show breadcrumbs for the current folder and its parents, up to the root "Files" page for the project. This also re-implements the Table component created in PR #256, which is still blocked and in draft mode.
Project Files Scene
The Project Files scene has the following features:

"Upload files" and "New folder" buttons that open dialogs to perform the respective actions
Clicking the name of a folder opens the folder using the routing scheme as above. Clicking a file does nothing at the moment. We will implement file previews in a future PR
Each file/folder row in the table has a "more" button with actions that can be performed for that type of item. All these actions are implemented, although changes will probably need to be made after feedback from the design team:

Folders:

Rename
Delete

Files:

Rename
Download
Delete
View History
Create New Version

The version history for a file displays the following:

Date of version
User who created version
File name of version
Icon representing category of version
Download button for version

Uploads
File uploads are implemented through an Dropzone field component and an UploadManager for viewing the progress of uploaded files. We do not currently offer the option to rename a file during the upload process; files are uploaded immediately upon being dragged into the dropzone.
The Upload Manager opens automatically when an upload is started. It can be collapsed into the toolbar, repositioned on screen by dragging the toolbar, and closed—which makes a button appear in the Profile Toolbar to reopen it.
The Upload Item shows the percentage of the file that is uploaded in a progress bar, along with the name of the file. If an error occurs, an error message will be shown underneath the progress bar; otherwise the text below the progress bar shows messaging to indicate what stage the upload is at.
I think we have a few unanswered questions about some of the behaviors. For example: collapsing the Upload Manager currently collapses it down into the bottom-right corner and makes it no longer draggable. But I figured these things might be better left for after the designers have done a pass at this.

@CarsonF I think this is ready to review again. I believe I've done all the changes we discussed. I didn't add any features that weren't already in this branch, so no file previews, and "File History" only offers a download link, not the full set of actions. This is just the UI/UX changes we discussed. Let me know if I'm missing anything

@CarsonF This is rebased and ready again


Printing with PhantomJS 1.9.1

Hi,
I'm not sure if this is an issue with js-sequence-diagrams, Raphael, PhantomJS or with my diagram, but I'm getting errors when using PhantomJS to print a page with a js-sequence-diagrams generated diagram.
Sample command:
phantomjs rasterize.js file:///path/to/seq-diag.html test.pdf

I was originally rasterizing/printing a HTML document that used js-sequence-diagrams. That was quite a large document, so I pulled out one of the generated SVG elements and gradually removed sub-elements until I was getting the error with the minimum SVG diagram.
seq-diag.html
<!DOCTYPE html>

<html>

<body>

<script src="../src/site/resources/lib/jquery-1.9.1.js"></script>

<script src="../src/site/resources/lib/raphael-min.js"></script>
<script src="../src/site/resources/lib/underscore-min.js"></script>
<script src="../src/site/resources/lib/sequence-diagram-min.beautified.js"></script>

<pre class="sequence-diagram">
Andrew->China: Says Hello
Note right of China: China thinks\nabout it
China-->Andrew: How are you?
Andrew->>China: I am good thanks!
</pre>

<script>
(function() {
    console.log("Before render");
    try {
        $(".sequence-diagram").sequenceDiagram({ theme: "simple" });
        console.log("Render success");
    } catch (e) {
        console.log(e);
    }
}());
</script>

</body>

</html>

seq-diag.html error

simple2.svg
Command:
phantomjs rasterize.js file:///path/to/simple2.svg test.pdf

SVG:
<!--
Is this the minimum diagram that fails to load in Adobe Reader?
-->
<svg style="overflow: hidden; position: relative;" height="318" version="1.1" width="475" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink">
    <desc style="-webkit-tap-highlight-color: rgba(0, 0, 0, 0);">
    Created with Raphael 2.1.0</desc>
    <defs style="-webkit-tap-highlight-color: rgba(0, 0, 0, 0);">
        <path style="-webkit-tap-highlight-color: rgba(0, 0, 0, 0);" stroke-linecap="round" d="M5,0 0,2.5 5,5z" id="raphael-marker-block">
        </path>
        <marker style="-webkit-tap-highlight-color: rgba(0, 0, 0, 0);" id="raphael-marker-endblock55" markerHeight="5" markerWidth="5" orient="auto" refX="2.5" refY="2.5">
            <use style="-webkit-tap-highlight-color: rgba(0, 0, 0, 0);" xlink:href="#raphael-marker-block" transform="rotate(180 2.5 2.5) scale(1,1)" stroke-width="1.0000" fill="#000" stroke="none">
            </use>
        </marker>
    </defs>
    <path style="-webkit-tap-highlight-color: rgba(0, 0, 0, 0);" fill="none" stroke="#000000" d="M50,96C50,96,204.95384871959686,96,234.99128912080778,96" stroke-width="2" marker-end="url(#raphael-marker-endblock55)" stroke-dasharray="0">
    </path>
</svg>

Did you ever dig deeper into this problem? or find a solution? If not I'm going to close this bug in a few days.

Closing, feel free to reopen if the problem persists.


Unable to load dynamic library 'intl.so'

Describe the bug
When I try installing the intl extension on PHP 8.2, I get the following error:
PHP Warning:  PHP Startup: Unable to load dynamic library 'intl.so' (tried: /usr/lib/php/20220829-zts/intl.so (/usr/lib/php/20220829-zts/intl.so: undefined symbol: executor_globals), /usr/lib/php/20220829-zts/intl.so.so (/usr/lib/php/20220829-zts/intl.so.so: cannot open shared object file: No such file or directory)) in Unknown on line 0

This issue sounds similar to #722, but I'm not sure if it's the same problem.
Version

[x] I have checked releases, and the bug exists in the latest patch version of v1 or v2.
[x] v2
[ ] v1

Runners

[x] GitHub Hosted
[ ] Self Hosted

Operating systems

Ubuntu

PHP versions

PHP 8.2

To Reproduce

Here's the step I'm using in my GitHub Actions yml file:
      - name: "Install PHP"
        uses<EMAIL_ADDRESS>        with:
          php-version: "latest"
          extensions: "intl-73.1"
          coverage: "none"
        env:
          debug: true
          phpts: "ts"

Expected behavior

The intl extension should load.
Screenshots/Logs

Additional context

I've tried this with ICU version 67.1 and 73.1 (trying to see if maybe 67.1 had trouble on PHP 8.2).
You can see the failed runs for both, here:

https://github.com/ramsey/php-ecma-intl-ext/actions/runs/5195176011/jobs/9367634410
https://github.com/ramsey/php-ecma-intl-ext/actions/runs/5195212718/jobs/9367702771

Are you willing to submit a PR?

I'm willing to submit one if you give me a few pointers for where to get started. 🙂

Hi Ben, thanks for the sponsorship 🙂
I have added support in for zts intl builds in https://github.com/shivammathur/setup-php/commit/12282c9597ef0c7329c0f382c5953c6248a43a62 and https://github.com/shivammathur/icu-intl/commit/d6ee42cd39943b532979d8f8ad6223042347b5cc.
This will be in the next release.

Thanks!

Released 2.25.3 with the fix.
https://github.com/shivammathur/setup-php/releases/tag/2.25.3


[Bug]: Footer issue AGAIN

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Footer section is messed up again!!

Add Screenshots
No response
What browsers are you seeing the problem on?
Chrome
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

This issue has already been assigned to me  #192

Okay, Work on this.
Actually, I fixed that section so many times. but again, the same issue is happening.

@4F24L yeah same here, even I fixed it twice. May be the contributors are not syncing or fetching the latest updates.

@4F24L yeah same here, even I fixed it twice. May be the contributors are not syncing or fetching the latest updates.

@mansiruhil13 You should add an automated message to tell the contributors to push in a newly created branch. So, the only changes can be added to the codebase.

i have no idea about automated message?

@mansiruhil13 I can fix this, I have an idea

I'll raise a sepearate issue for this

Like the Github Bot is replying when someone is raising an issue. When someone is assigned for any issue, it will reply them that this is the rules and you have to follow it when you make a PR.

Yeah that would be a nice idea

As this issue is not solved yet, should I work on this?

I will resolve this

Ok buddy !!


Add a way to express proxy connections

It would be great to show communication links passing through proxies - e.g. by adding a "via" field (or even list) to the communication_links entries.

Yep, that's something useful for reverse proxies or stuff like that...
There's already something like that in the queue to have a technical component act like a transparent element in risk analysis. Ideas exist for being able to have "n" incoming communication links to a reverse-proxy (or WAF) like thing and then "m" connections outgoing but still being able to address what input leads to what specific output connection.
Current state is: haven't fully thought through about how to add this seamless and intuitively (for the user) to the model, but will post some ideas soon (currently prepping a conference talk about Threagile) and look forward for feedback then.


nixos/switch-to-configuration: support shells which aren't POSIX compliant

fish' doesn't support the ENVVAR=foo my cmd' notation which breaks
`nixos-rebuild switch':
fish: Unsupported use of '='. To run '/nix/store/dzn7zyf1dhxlvrbpgn yn7di58zvfblrq-systemd-239/bin/systemctl'
with a modified environment, please use
'env XDG_RUNTIME_DIR=/run/user/1000 /nix/store/dzn7zyf1dhxlvrbpgnyn7di58zvfblrq-systemd-239/bin/systemctl…'

The issue can be solved by adding an `export' statement and run the
command after that.
Fixes #45897
Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Duplicate of https://github.com/NixOS/nixpkgs/pull/45932

This would still not solve it in case of say csh.

Maybe `su -s /bin/sh?


Update holiday normalization

This appears to be a Java 8 vs. Java 11 problem.  At one point I had to update Jenkins to use Java 11, because that is the lowest version supported.  It is somehow causing a problem.

Some problems are described here: https://github.com/svendiedrichsen/jollyday/issues/88

https://github.com/focus-shift/jollyday says that JDK 11 is required.

How about we pull the plug and remove the corenlp subproject?

:-)  I may be able to simply shade it.  I can also try to use jollyday differently so that it doesn't throw the exception.  After all, stanford uses it and those tests aren't failing.  Updating to a new corenlp will probably not help, because they use the same old version of jollyday.

I was serious about removing it. It depends on older Java, introduces many differences, and causes all these conflicts...

The holiday tests have passed on Linux with Java 8, 11, 14, 17, 18, on Windows with Java 8, 11, 14, 17, 18 and Mac with java 8, 11.  There were other tests that failed intermittently on Mac (8, 11) and Windows (18).  They went away when I retested.  There's a chance that the dependency change has some kind of transitive effect which will cause users of the library problems,  Look here to the cause.  Holiday handling could be a lot better and an issue will be filed with some details.

Is that a thing?


Fixes #296 Cannot find valid target when multiple drop types specified

Fixes the issue from #296 where drop types are compared by reference only. If both drop types are arrays, an additional array comparison is performed.

This fix happens to work, but this is not the underlying issue.
Description in https://github.com/gaearon/react-dnd/issues/296#issuecomment-165277413 is incorrect:

Although the types are not changed occurs initialization below and takes some time.

JavaScript is single threaded (from user’s perspective) so a race condition is not the problem here. The problem is in DnD Core removing handlers immediately which can cause problems during nested updates: for example, a hover() action causing removeTarget().
The solution would be to change HandlerRegistry in dnd-core to delay deleting old handlers until the next tick.

Fixed in dnd-core via f1893a64965d2a294611d4d0449e5250a58308e0.


Replaced EOL component javax.mail with com.sun.mail:javax.mail

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

@david-leifker
Can you please have a look into this javax.mail vulnerability fix?


Add support for ATIME_NOW and MTIME_NOW

@zargony any feedback on this PR? Happy to make changes if you'd like

Hey @cberner.
Thanks a lot for adding atime/mtime support! Though there are two reasons why I'd like to delay this PR a bit.

I'd like to finish #126 first which significantly changes how request and replies are handled in rust-fuse
I'm not sure if we want to have abi version related feature flags on the main crate. I feel like it would be more appropriate to have higher level feature flags. This isn't well thought through or discussed yet though. Having flags like feature(atime_mtime) would be nice, but that could lead to a lot of feature flags and might become confusing quickly. Flags for major fuse release versions would be nice, but that seems to be hard to map to different versions of Linux/macOS/FreeBSD. I'm not sure what would be best here (we should probably discuss that in general in a separate issue).

Ok, that's fine. I'll follow #126 to see when it's merged, and then rebase on it. Sure, named feature flags sounds easier to use. It doesn't seem like it will create too many, since most new FUSE versions seem to only add one or two new features

@zargony just checking in. Any estimate when you will merge #126?

I'm so sorry about the slow progress, I don't get a lot of free time these days to work on this. I just added the new filesystem trait and started to work on porting the session (which is the last missing piece besides a lot of other open questions). I'm afraid that it'll still take a few weeks to get it to a working state.
If you'd like to help, there are still several pieces calling for discussion/implementation, like a modern reply type for directories or xattr lists or a channel implementation that can be used asynchronously. Or more general things like how to do feature gates that require specific abi versions :)

Ya, I'm happy to help out! The feature gates sound like an easy place for me to start, since I already looked at those a bit. I think it makes the most sense to have a feature flag for each ABI version, and then link to the fuse documentation, given that each version may add many features.
If you prefer having a flag for each logical feature that then maps to the cooresponding ABI version, I can certainly implement that instead.
Let me know!


Ability to establish more than 60k simultaneous connections to a backend IP

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
When handling large numbers of long-lived TCP connections, a fundamental limit is the unique tuple of (src ip, src port, dst ip, dst port). When src ip/dst ip/dst port are fixed, as they usually will be, this only allows you roughly 63k simultaneous connections (due to the lower 1k typically being privileged. And in fact due to some issues around hashing is handled in the linux kernel, keeping the number lower at 10-30k is typically much better for performance.
The way around this is to have multiple IPs on the same interface and to send from those to multiple IPs on the remote side. You can do this just using say 5 IPs on the remote side but then your limitation is 560k, if you have 3 on source and 3 on remote then you get 33*60k - ie scaling the number of IPs on both sides produces much better scaling than just on 1 side of the connection.
Unfortunately there is no way (that I know of) in Linux to say "send from all IPs on an interface" it will only send from the primary one. You can potentially do some stuff with iptables marking a given connection with a certain fwmark and then use multiple routing tables to specify source IP based on fwmark, but this usually requires the conntrack module to be loaded (which causes its own issues, especially under large numbers of connections), and is very messy and hard to set up correctly. You could alternatively try using multiple alias interfaces each with their own outbound IP address, but linux doesn't really support ECMP so even on the same route metric for the same subnet it doesn't work particularly reliably.
For this reason, most loadbalancers have support for specifying a set of source IPs to use for a connection. For example haproxy has an optional src-ip parameter for each backend connection, and so we just iterate src/dst ip pairs to generate a whole load of backend servers via templating at present.
An alternative may be to use an approach like tsung does which allows you to specify an outbound interface (eg eth0), but from what I understand of the mechanism behind that it effectively just issues a syscall to get the IPs of an interface and then adds them into an internal pool. This is likely to not be portable between OS's (unlike the source ip parameter of net.Dial), and restricts some of the configurability that is available via simply specifying a list of source IPs to use for a given backend connection.
I have generated a PR at https://github.com/traefik/traefik/pull/10277 which should address this issue.

Thank you for taking the time to open this proposal.
I believe it would be beneficial to keep this proposal open even after your pull request has been merged. There are potential improvements that could enhance its reliability and simplify its configuration.
One challenge I foresee in manual source address configuration is the difficulty in selecting the appropriate address. If addresses are chosen sequentially or randomly, the resulting distribution may be suboptimal. Additionally, it becomes challenging to predict the capacity for handling connections based on the number of configured source addresses.
Another promising solution worth considering is the utilization of SO_REUSEADDR in conjunction with a bind + connect approach. This would enable the reuse of ephemeral ports as long as the destination differs.


Autocomplete doesn't exist in the ui kitten library files

Warning: React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but
got: %s.%s%s, undefined,  You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.

🐛 Bug Report

To Reproduce
Steps to reproduce the behavior:
Expected behavior

Link to runnable example or repository (highly encouraged)

UI Kitten and Eva version

Package
Version

@eva-design/eva

@ui-kitten/components

Environment information

What UI Kitten / Eva versions you run?
Did you update from previous versions? In case yes 👇
https://github.com/akveo/react-native-ui-kitten/issues/930

What UI Kitten / Eva versions you run?
Did you update from previous versions? In case yes 👇
#930

my @ui-kitten/components is 4.3.2

Autocomplete was released in 4.4


onChange is called once per character in edit window

Version
"@toast-ui/react-editor": "^1.0.0",
Test Environment
mac mojave 10.14.5
chrome 80.0.3987.100 (Official Build) (64-bit)
Current Behavior
Using the onChange function is called once per character in edit window
ie. type 'a' -> onChange is called once
type 'a' (now you have aa) -> onChange is called twice
type 'a' (now you have aaa) -> onChange is called n times
... if you have 20 characters '12345678901234567890 and type 'a' -> onChange is called a lot of times
Here's a image with 63 calls with only adding an 'a' to the string above.

I've had this installed for 9 months with no changes and recently updated and now I am seeing this issue.  It's possible this was a latent issue due to my setup.  If you see something wrong with my setup please let me know.
This is a live app so I can show you.  Let me know when you have time.
Here's my component
import React from 'react';
import PropTypes from 'prop-types';

import 'codemirror/lib/codemirror.css';
import 'tui-editor/dist/tui-editor.min.css';
import 'tui-editor/dist/tui-editor-contents.min.css';
import 'tui-editor/dist/tui-editor-extTable';
import {Editor} from '@toast-ui/react-editor';
import {Typography} from '@material-ui/core';

import './index.css';

class WhlEditor extends React.Component {
   constructor(props) {
      super(props);

      this.editorRef = React.createRef();
   }

   static propTypes = {
      onChange: PropTypes.func,
      toolbarItems: PropTypes.arrayOf('string'),
      content: PropTypes.string,
      maxLength: PropTypes.number,
   };

   state = {};

   onChange = event=> {
      const content = this.editorRef.current.getInstance().getValue();

      if (content == this.state.content) {
         // skip when the same
         console.info('same...');
         return;
      }

      console.info('onChange content', content,event);
      this.setState({content});
      (typeof this.props.onChange === 'function')  && this.props.onChange(content, this.state.content);

   };

   toolbarItems = [
      'heading',
      'bold',
      'italic',
      'underscore',
      'strike',
      'divider',
      'hr',
      'quote',
      'divider',
      'ul',
      'ol',
      'task',
      'indent',
      'outdent',
      'divider',
      'table',
      'image',
      'link',
      'divider'
      // 'code',
      // 'codeblock'
   ];

   ToastEd = ()=> {
      const self = this;
      const {toolbarItems = this.toolbarItems, content} = self.props;
      return (
         <>
            <Editor
               ref={this.editorRef}
               previewStyle="none"
               height="25vh"
               width="80vw"
               initialEditType="wysiwyg"
               initialValue={content}
               onChange={ self.onChange }
               onFocus={ self.updateMaxLength }
               useCommandShortcut={true}
               toolbarItems={toolbarItems}
               exts={[
                  {
                     name: 'chart',
                     minWidth: 100,
                     maxWidth: 600,
                     minHeight: 100,
                     maxHeight: 300
                  },
                  'scrollSync',
                  'colorSyntax',
                  'uml',
                  'mark',
                  'table'
               ]}
            />
            { this.props.maxLength > 0 &&
               <Typography
                  id="editor-maxlength"
                  align="right"
                  style={ {
                     position: 'relative',
                     height: '0px',
                     bottom: '35px',
                     right: '5px'
                  } }
               >
                  { this.props.maxLength - (content && content.length || 0) }
               </Typography>
            }
         </>
      );
   };

   render() {
      return <this.ToastEd {...this.props} />;
   }
}

export default WhlEditor;

Expected Behavior

onChange should be called only once per change

One more thing that I forgot.  My app created two instances of the editor because I have multiple places this is being used.  One is for editing tips the other is for articles.

Hello, I believe that this is normal behaviour. onChange is being called each time it is changed. When you add a new character in the editor, that newly added character is registered as a change to the function.
What problems are you facing when onChange is being called each time a new character is added?


Experiment: Replace dialog with details-dialog

@koddsson and @keithamus brought up the idea of replacing the use of @reach/dialog with the GitHub details-dialog web component. The @reach/dialog package and it's associated dependencies currently represent about 11% of PRC's bundle size for just one component. This would also pave the way for a single source of truth for our JS behaviors across the design system's implementations.
Open Questions

[ ] How will we get around rendering these components in an SSR application? I'd like PRC to remain compatible with applications that are server side rendered in frameworks like Next.js or Gatsby (our docs sites are all using PRC within Gatsby)

I've started this work in the branch details-dialog but am blocked by the SSR issue

Update - regarding importing the custom element in the React component itself @keithamus suggested:
have a separate file like browser.js which you encourage users to import in their browser environment, and that file can import all of the custom elements
That would work, but I'm concerned about having invisible dependencies in Primer React Components. One of our main goals in PRC is developer experience and I'm worried that having some components that depend on a separate js bundle and some components that don't would be confusing to our users 🤔


Updating user email creates a new identity

Hi, not sure if that's the right place to ask such a question, but updating user email creates a new secondary identity instead of changing the primary one, at least that would be expected behaviour.
user = client.users.first
user.update(email<EMAIL_ADDRESS>user.save!

Thanks.

Currently, you can only update user identities through the user identity update API.
Something like should work:
user = client.users.first
id = user.identities.first # or user.identities.detect {|id| id.value == ...}
id.value =<EMAIL_ADDRESS>id.save!


fix: restore newsreader import feature

./gradlew :overlays:NewsReaderPortlet:dataImport -Ddir=...

Checklist

[x] the individual contributor license agreement is signed
[x] commit message follows commit guidelines
[ ] tests are included
[ ] documentation is changed or added
[ ] message properties have been updated with new phrases
[ ] view conforms with WCAG 2.0 AA

Description of change

Well done Benito ! All test are passing, so this could be merged ?


[DRAFT] Add capacity memory overhead generation

Fixes #N/A
Description
How was this change tested?
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Going to close this out since I don't think we're going to go with the auto-gen method and we will most likely go with the launch discovery method that's proposed in newer PRs.


Map correct

Moved logic from routes.js into controller.js

Looks good. Just clean up some commented out code.

I'm having a little bit of difficulty parsing the code. I think it needs more comments.

Could you add some comments describing what's going on, especially in the google.maps stuff. It'd make it easier to understand when we look back to it later in the week.

Cool. I took care of it.


Fixes some keyboard absolute move guideline bugs.

Problem:
There are some minor issues with the absolute keyboard move strategy:

Pressing the alt key prevents moving with the keyboard completely.
Once a move has been started pressing the alt key doesn't show the distance guidelines.
The snapping guidelines don't show at all when moving with the keyboard.

Fix:
The fixes for the above respectively are:

Remove the check on the alt key from the fitness function of the strategy.
Changed the check in the guidelines which prevented them from showing up if any interaction has been started.
Used some existing code for creating guidelines from the mouse absolute move strategy to include the appropriate guidelines during a keyboard move.

Additionally an issue was found when the timeout cleared the interaction strategy but resulted in the snapping guidelines persisting because foldAndApplyCommands was accumulating the transient patches as well as permanent ones.
Commit Details:

The alt key is now ignored by keyboardAbsoluteMoveStrategy, as it should
be independent of toggling the distance guidelines.
Includes the activateSnap guidelines while moving with the keyboard
without snapping to them.
snapDrag moved into shared-absolute-move-strategy-helpers.ts.
foldAndApplyCommands does not accumulate patches for commands that are transient.
Added key and id to the element underlying GuidelineControl.
DistanceGuidelineControl changed from not displaying on any interaction being
active, to not displaying on a drag interaction being active.
CLEAR_INTERACTION_SESSION now wipes the entire controls field of
EditorStateCanvas.

Changed the check in the guidelines which prevented them from showing up if any interaction has been started.

this may potentially collide with alt-drag-to-duplicate. I guess we want to keep the distance guidelines hidden while alt-drag-to-duplicating? cc @maltenuhn

Changed the check in the guidelines which prevented them from showing up if any interaction has been started.

this may potentially collide with alt-drag-to-duplicate. I guess we want to keep the distance guidelines hidden while alt-drag-to-duplicating? cc @maltenuhn

From the commit details:

DistanceGuidelineControl changed from not displaying on any interaction being active, to not displaying on a drag interaction being active.