when will the support for attachment available for list form?

Use the following form to submit an issue only if it's related to samples in this repo. If you have an issue related to the SharePoint Framework or its documentation, please submit the issue at https://github.com/SharePoint/sp-dev-docs/issues/new. This will help us respond to your issue faster.

Thank you for reporting an issue or suggesting an enhancement. We appreciate your feedback - to help the team to understand your needs, please complete the below template to ensure we have the necessary details to assist you.
(DELETE THIS PARAGRAPH AFTER READING)

Category

[ ] Question
[ ] Bug
[x] Enhancement

For the above list, an empty checkbox is [ ] as in [SPACE]. A checked checkbox is [x] with no space between the brackets. Use the PREVIEW tab at the top right to preview the rendering before submitting your issue.
(DELETE THIS PARAGRAPH AFTER READING)

Authors

Because of the way this repository is setup, samples authors do not get a notification when you create an issue. It makes it less likely for you to get your issue resolved or to get help. For the section above @mention any author of the sample. Authors' github handle can be found on the main sample documentation page, under the "solution" section. Use the PREVIEW tab at the top right to preview the rendering before submitting your issue.
(DELETE THIS PARAGRAPH AFTER READING)

Expected or Desired Behavior

If you are reporting a bug, please describe the expected behavior. If you are suggesting an enhancement please describe thoroughly the enhancement, how it can be achieved, and expected benefit.
(DELETE THIS PARAGRAPH AFTER READING)

Observed Behavior

If you are reporting a bug, please describe the behavior you expected to occur when performing the action. If you are making a suggestion, you can delete this section.
(DELETE THIS PARAGRAPH AFTER READING)

Steps to Reproduce

If you are reporting a bug please describe the steps to reproduce the bug in sufficient detail to allow testing. Only way to fix things properly, is to have sufficient details to reproduce it. If you are making a suggestion, you can delete this section.
(DELETE THIS PARAGRAPH AFTER READING)

Submission Guidelines

All suggestions or bugs are welcome, please let us know what's on your mind.
If you are reporting an issue around any of the samples, please ensure that you have clear reference on the sample and possibly code file, which should be fixed.
If you have technical questions about the framework, we’ll be monitoring #spfx, #spfx-webparts, and #spfx-tooling on (SharePoint StackExchange)> [http://sharepoint.stackexchange.com/]. You can also alternatively submit your question to (SharePoint Developer group)> [https://network.office.com/t5/SharePoint-Developer/bd-p/SharePointDev] at Office Network.
Remember to include sufficient details and context.
If you have multiple suggestions or bugs please submit them in separate bugs so we can track resolution.

(DELETE THIS PARAGRAPH AFTER READING)

Thanks for your contribution! Sharing is caring.

@jindalv10 I used the code and changes to the files necessary. I'm able to upload the file but my files are not saved to SharePoint list. Any thoughts on it?

@Fnu-Avi Can you try to debug the code and let me know what errors are you getting on your end.
Please remove following lines from code.
Attachments: { component: SPAttachmentFormFieldEdit } in const DisplayFieldTypeMappings
I haven't tested this with display form yet.

I'll mark this issue as closed as there hasn't been any activity for a while.
Feel free to reopen it if you need more help with it.
@wrvishnu, please take the time to fill the issue template next time, as it makes it easier for everyone to help with these issues.


Escape syslinux additional initrd and arguments

Escaping these arguments so when we have multiple values for ADDITIONAL_INITRD and/or ADDITIONAL_ARGUMENTS all values are passed to the syslinux.cfg.
I'm facing an issue where not all values on usr/lib/frzr.d/bootconfig.conf are being used because only the first one is considered, I believe this is due to the shell expansion and the function expecting only 4 arguments

This issue presents itself on your machine only? Or is it present on other configurations also? Might be something else. But otherwise the patch LGTM.

It's very easy to check the issue too, just check your usr/lib/frzr.d/bootconfig.conf and your syslinux.cfg, chances are only the first value of bootconfig.conf will be present on your syslinux.cfg.

Could reproduce.

Thanks!


Add parameter verbose to kompile

Fixes #521

Is the output from kompile collected and printed at the end?

Modified accordingly.


Update Zstash to run on macOS

The command display_mode = "stat --format '%a' {}".format(file_path).split() added in hpss.py in #42 only works on Linux. There needs to be a different command for macOS.

@golaz @chengzhuzhang I'm assuming most people use Zstash on Linux, but let me know if we should try to get this into v0.4.0 (or it might need to be v0.4.1).

This can be part of a future release. Instead of relying on an external call, is it possible to rewrite this using Python built-in os.stat() function?

Officially, zstash does not support macOS. However, it would still be nice to change the command to use os.stat

Officially, zstash does not support macOS. However, it would still be nice to change the command to use os.stat


⚠ Bofrost.lu has degraded performance

In d849ca5, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1288 ms

Resolved: Bofrost.lu performance has improved in 2b9a979 after 1 hour, 27 minutes.


Search input in Ctrl-F window is not pre-populated with selected text when addSelectionToNextFindMatch is used

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.68.0
OS Version: Windows 11

Steps to Reproduce:

Assign keyboard shortcut to editor.action.addSelectionToNextFindMatch
Ensure no text is selected in any open files, close VSCode
Re-open VSCode
Use keyboard shortcut created earlier to select the text.
Press Ctrl-F
Observe that find window's text box is not prepopulated.

This happens only on clean start and no selection present earlier.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.68.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Minify CSS

Currently all CSS used in styled-components is unminified.
As @mxstbr suggested on Gitter, this Babel plugin could take care of minifying the CSS.
Perhaps only when process.env.NODE_ENV === 'production'?
Alternatively, this could be a babili plugin.
Related to https://github.com/styled-components/styled-components/issues/284

That could totally work, shouldn't be too hard actually with the base we already have! I haven't looked too deep into it, but doing s/\s/ on the CSS should kinda just work? What do you think @vdanchenkov?

It would be hard to cover all cases due to interpolations, but we definitely could support some basic stuff. s/\s/ (trim whitespace from the beginning of each line) should work fine. Maybe we could make it little more advanced like this: http://cssnano.co/optimisations/core/

True, good point about the dev! Let's try making this happen then?

I'll look into it tomorrow. Will minify css and styled.x.

any update on that topic?

We can dramatically reduce css-related code by removing template literals
const Test1 = styled.div`
  width: 100%;
`;

// transpiles with babel-plugin-transform-es2015-template-literals to

function _taggedTemplateLiteral(strings, raw) {
  return Object.freeze(Object.defineProperties(strings, { raw: { value: Object.freeze(raw) } }));
}
var _templateObject = _taggedTemplateLiteral(['\n  width: 100%;\n  color: ', ';\n'], ['\n  width: 100%;\n  color: ', ';\n']),
var Test1 = _styledComponents2.default.div(_templateObject, color);

As far as I can see we do not use raw values, so it could be transpiled with our plugin to:
var Test1 = _styledComponents2.default.div(['\n  width: 100%;\n  color: ', ';\n'], color);

// or with minification

var Test1 = _styledComponents2.default.div(['width: 100%;color: ', ';'], color);

@mxstbr Am I correct that it will work as is with styled-components lib?

raw is a publicly accessible API that babel plugins can hook into (node.value.raw, see the spec) so I doubt we can just remove it!
What we should instead do is get rid of all the whitespace in the template literals and let everybody transpile them (or not transpile them if they only support Chrome) however they want?

It's actually two cases but they are related.
Simple whitespace removal could be done, but it looks like a lot of work with little benefit. Typically it will remove 3 characters per rule if we remove newlines and spaces at the start of every line. We can not safely remove all whitespaces due to cases like content: "  ${value}  ";
The problem with template literals is that we need to remove whitespace from both raw and cooked values and strategies are little different. For example, in raw we have weird things like \\n and '\n' both representing line-break. We can even come to situation where raw and cooked values diverge significantly due to minification. That is why I don't like the idea of minifying two values separately. It would be cleaner if we minify raw and convert to cooked after minification, but I'm not sure about edge cases here.
My suggestion is similar to replacement of template literal with private API, but it can be done with no changes in the lib, I guess. I suggest to replace template literal node with regular function call.
It will give over 50% reduction of footprint if transpilation will be applied. But will slightly increase footprint if no transpilation is needed (4 chars per template literal, +1 per interpolation).

@SpaceK33z Could you check #15 with your test cases? You can paste examples of your code to fixture 08-minify-css-to-use-with-transpilation if you transpile template literals or 09-minify-css-to-use-without-transpilation if not. Run npm test to check it's output.

@vdanchenkov, looks good! I tested some code from a small project with it. I don't have much code since I was still evaluating styled-components, but it worked. It looks like keyframes aren't compressed however (red = expected, green = actual):
-const sweep = keyframes`to {transform: rotate(360deg);}`;
+const sweep = keyframes`
+    to {
+        transform: rotate(360deg);
+    }
+`;

My suggestion is similar to replacement of template literal with private API, but it can be done with no changes in the lib, I guess. I suggest to replace template literal node with regular function call.

It will give over 50% reduction of footprint if transpilation will be applied. But will slightly increase footprint if no transpilation is needed (4 chars per template literal, +1 per interpolation).

Let's do this then!

Thanks @vdanchenkov for fixing this, really awesome :).


Allow retrieval of IANA value for negotiated cipher

Problem:
s2n allows retrieval of the Openssl format cipher string for connection's negotiated cipher: s2n_connection_get_cipher(). This API works well for cases where an application simply wants to write the name of the cipher into a log.
It does not work well if the application wants to implement conditional logic based on the specific negotiated cipher. To achieve that with the current API, the application must implement string matching and parsing. This is not ideal for the following reasons:

Performance
Completeness: since we are using Openssl strings, it's not clear to an application what the total set of allowed values actually is.
Stability: Strings may change and it's not clear to an application how much they can rely on them.

Proposed Solution:
Add a getter for the IANA registry value of the negotiated cipher: s2n_connection_get_cipher_iana().

I'm not married to the IANA value as the only way to solve this, but I figured I should raise it in case other folks have opinions.

This problem is not super painful at the moment(computers are faster than we think at string comparison). but the solution might be cheap enough to implement that we should just do it.


Connect pod through SSH protocol

https://github.com/FrelDX/kube-console-on-ssh

@FrelDX thanks for sharing you neat project!
I'm going to close this issue as it doesn't appear to be something that warrants specific work to be done on kubectl.
Perhaps consider adding it to an awesome list or turning it into a kubectl plugin.
/close

@FrelDX感谢您分享整洁的项目！
我要解决这个问题，因为它似乎并不能保证需要在kubectl上进行特定的工作。
也许考虑将其添加到很棒的列表中或将其变成kubectl插件。
/关

Can you add my project to https://github.com/ramitsurana/awesome-kubernetes ？

@FrelDX thanks for sharing you neat project!
I'm going to close this issue as it doesn't appear to be something that warrants specific work to be done on kubectl.
Perhaps consider adding it to an awesome list or turning it into a kubectl plugin.
/close

Can you add my project to https://github.com/ramitsurana/awesome-kubernetes ？

@FrelDX thanks for sharing you neat project!
I'm going to close this issue as it doesn't appear to be something that warrants specific work to be done on kubectl.
Perhaps consider adding it to an awesome list or turning it into a kubectl plugin.
/close


Dash Panel Keeps Loading

What version are you using?
v0.2.10
What happened?
Disable the extension and open any workspace and enable the extension. Now go to the Dash Panel. It will keep loading.
The same error has also been reported by Barış KAÇİN in the marketplace review:

sometimes that window is not working its like loads stuff on background

We need to figure all cases where the issue is reproducible and push the correct fix.
@wadhia-yash could be reached here or in slack for any questions.
Steps to reproduce
1. Go to '...'
2. Click on '...'
3. Scroll down to '...'
...

Supporting info to reproduce
No response
Relevant log output
2024-02-23 10:38:51.609 [info] ExtensionService#_doActivateExtension WelltestedAI.fluttergpt, startup: false, activationEvent: 'onLanguage:dart'
2024-02-23 10:38:51.912 [error] Activating extension WelltestedAI.fluttergpt failed due to an error:
2024-02-23 10:38:51.912 [error] TypeError: Cannot read properties of undefined (reading 'analyzer')
    at activate (/Users/mr.minnu/.cursor-nightly/extensions/welltestedai.fluttergpt-0.2.10/out/extension.js:55:50)
    at u.n (/Applications/Cursor Nightly.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:141:6342)
    at u.m (/Applications/Cursor Nightly.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:141:6305)

I'm ready take this up 😇

assigned @yahu1031. 👍

This happened because Dart-Code stopped exposing _privateApi for the Dart extension. This led to the failure to call the LSP analyzer. The latest version, v3.83.20240225 has been published to support this issue as a temporary fix.
CC: @samyakkkk @KevalPrajapati


AVL Tree in Java

In the Tree folder create another package named AVLTree and in that folder create a java file and add the code.
Make sure to create a folder as I said above.
Comment on the issue to get assigned.

@aritra-tech

AVL tree what to do in it could you please say it? Meaning we have to add basic AVL tree code?

Yah basic code of AVL tree

Please do assign it to me!

Please do assign it to me!

Ok go for it


PostCSS transformation for stylesheets loaded from <link> tags

Is your feature request related to a problem? Please describe.
Based on this part of the README:

Vite automatically applies your PostCSS config to all styles in *.vue files and imported plain .css files. Just install necessary plugins and add a postcss.config.js in your project root.

my understanding is that the PostCSS config will be applied to stylesheets that are imported from within a javascript file, e.g.
// main.js
import './main.css';

but it doesn't state that the PostCSS config will be applied to stylesheets that are directly requested via <link> tags, e.g.
<!-- index.html -->
<link rel="stylesheet" href="/src/main.css">

I went ahead and tried that locally with<EMAIL_ADDRESS>and a basic PostCSS config:
module.exports = {
  plugins: [
    require('postcss-nested')
  ]
};

and confirmed that the stylesheet was not transformed via postcss-nested when loaded from a <link> tag.
Describe the solution you'd like
I might be in the minority but I still use <link> tags to load stylesheets rather than importing them in javascrpt files. It would be nice if they could still be processed through PostCSS. Is this possible or are there implementation details that would make this impractical?

Whoever wants to implement this, here's what you need to do.
After this if block, add an else statement that calls compileCss on the ctx.body string:
https://github.com/vitejs/vite/blob/963614bff7039207c0b2d9e8c2c5780b74ee176a/src/node/server/serverPluginCss.ts#L33
I think that vite build will process <link> stylesheets with PostCSS already. Can you confirm that, @rmacklin?

I think that vite build will process <link> stylesheets with PostCSS already. Can you confirm that, @rmacklin?

I didn't get that working either, though I'm not sure if something in my setup was wrong. I've pushed up a demo repository here (it's just a handful of commits on top of yarn create vite-app):
https://github.com/rmacklin/vite-css-link-tag-demo
This commit was my attempt to use a <link> tag to load the stylesheet. As you can see, the index.css file in dist didn't get processed (it still has nested rules, so postcss-nested didn't run, and it's not minified or fingerprinted), and index.html ends up with undefined for the <link>'s href:
https://github.com/rmacklin/vite-css-link-tag-demo/blob/ab1abfaed3d41e2dd6d7ca34ff0cfe79491880ee/dist/index.html#L6

This is fixed in 2.0.

This is fixed in 2.0.


Support reading Postgres array as JSON

Fixes: #682

Issue:  https://github.com/prestosql/presto/issues/682

cc @guyco33

Applied comments

Merged, thanks!


Added stronger UUID generation algorithm

It seems that lately we had some UUID collision, it's unclear to me where the collision is created (or rather why), but i changed the algorithm to be a bit stronger (and to use true random if available on platform)

Closing this one as super-seeded for now


Increase concurrency capability

I want to increase concurrency capability, but i see source code like:
    with model_lock:
        segments = []
        text = ""
        segment_generator, info = model.transcribe(audio, beam_size=5, **options_dict)
        for segment in segment_generator:
            segments.append(segment)
            text = text + segment.text
        result = {"language": options_dict.get("language", info.language), "segments": segments, "text": text}

can i remove the lock?

From my work with the code, the lock prevents multiple accesses to the same model instance. If you wanted to run multiple in parallel, you would need to initialize multiple models. As the model, when running, will more or less saturate the processing capability of the GPU/CPU it is running on, this likely wouldn't be as useful as you hope. It is much easier/configurable to spawn multiple instances of the docker container instead. For example, using --gpus device=n where n is the id of a compatible GPU allows you to run the model on a specific GPU. I have used this to run testing on two Nvidia Tesla GPUs as well as the system CPU


Support option equivalent to SkipTemplateParameterPrompt from PowerShell for ARM template validation

Is your feature request related to a problem? Please describe.
While validating ARM template and parameter file, there is no way to avoid prompting for missing parameters.
When this is automated in a pipeline to proceed to template validation, the pipeline will hang until timeout.
By having an option to disable prompting for missing parameters, this will allow to get validation result right away when we expect parameter file or command line to contains all required parameters.
Describe the solution you'd like
Add a new parameter to az group deployment validate to support not prompting for missing parameter.
Describe alternatives you've considered
Only alternative right now is to decrease the task timeout in the pipeline to get it to fail more quickly... but this still does not indicate the real issue.
Additional context
It looks like the Az PowerShell module already support this kind of option with SkipTemplateParameterPrompt for both Test-AzResourceGroupDeployment and New-AzDeployment

add to S168


Permission denied when trying to Process.start a program with execute only permissions.

It appears that the dart runtime (on Linux at least) is checking the executable passed to Process.start for read access which fails for programs like /usr/bin/sudo which only have execute access.   The access(2) with X_OK seems like a more appropriate check.
dart --version
Dart SDK version: 2.19.2 (stable) (Tue Feb 7 18:37:17 2023 +0000) on "linux_x64"

output from strace dart run test/tests.dart
876842 openat(AT_FDCWD, "/usr/bin/sudo", O_RDONLY|O_CLOEXEC) = -1 EACCES (Permission denied)
output from program
  ProcessException: Permission denied
    Command: /usr/bin/sudo /usr/bin/sh

  dart:io-patch/process_patch.dart 401:33            _ProcessImpl._start
  dart:io-patch/process_patch.dart 38:20             Process.start

I have worked around the issue by not specifying the absolute path and letting the runtime search the PATH which does not seem to check for read access to the executable first.  That is problematic as the program cannot guarantee that it's running the correct executable.

Hello,
I have same problem. I use csharp_rpc package. I created Windows console application. I want to start it in my android application but it gives me this error. It is working with windows application. Do you have any solution?


Microsoft Access unable to import generated CSV formatted output

Microsoft Access fails to import FileSystemSearcher output.  At least one of the problems is that commas embedded in music file names causes the commas in the file name to be interpreted as field delimiters which would be expected.

Made simple fix.  Able to load 2 million record file generated by running on 4TB HD holindg backups into MS Office Access.  The file will import but is so close to Access DB size limits that the creation of a single index on the table causes the DB to fail and no recovery possible other than reloading.
Future work needs to include making dialect='excel' and delimiter='|' settable via command line options.


absl/base/dynamic_annotations.h conflicts with glog when using TSAN

Describe the issue
I'm using abseil-cpp and glog(0.5.0) with COPTS -fsanitize=thread and  get the following error:
external/_main~data_deps_ext~com_google_absl/absl/base/dynamic_annotations.h:218:6: error: conflicting declaration of C function 'void AnnotateBenignRaceSized(const char*, int, const volatile void*, size_t, const char*)'   218 | void AnnotateBenignRaceSized(const char* file, int line,       |      ^~~~~~~~~~~~~~~~~~~~~~~ In file included from external/_main~data_deps_ext~com_github_brpc_brpc/src/butil/logging.h:36,                  from external/_main~data_deps_ext~com_github_brpc_brpc/src/bthread/types.h:27,                  from external/_main~data_deps_ext~com_github_brpc_brpc/src/brpc/controller.h:31,                  from taishan_common/profile_service.cpp:3: bazel-out/k8-opt/bin/external/glog~0.5.0/_virtual_includes/glog/glog/logging.h:1006:17: note: previous declaration 'void AnnotateBenignRaceSized(const char*, int, const volatile void*, long int, const char*)'
Steps to reproduce the problem
build with glog
What version of Abseil are you using?
a3020c763c12bd16bbf00804abe853afa5778174.zip
What operating system and version are you using?
ubuntu
What compiler and version are you using?
gcc 9.1
What build system are you using?
6.1.1
Additional context
No response

glog fixed this problem nearly 2 years ago: https://github.com/google/glog/commit/ffb40018a50e86d3a9e518405cee59e76e5430d9
You need to update.


Centos OpenSSL problem.

20:46:43 [INFO] Could not load native library: /tmp/bungeecord-native-cipher810503535089700859.so: libcrypto.so.1.0.0: cannot open shared object file: No such file or directory 20:46:43 [INFO] Using standard Java JCE cipher. To enable the OpenSSL based native cipher, please make sure you are using 64 bit Ubuntu or Debian with libssl installed.
`[root@bungeeberry Bungee]# yum install openssl-devel
Loaded plugins: fastestmirror
Loading mirror speeds from cached hostfile

base: less.cogeco.net
extras: centos.mirror.rafal.ca
rpmforge: mirror.team-cymru.org
updates: less.cogeco.net
Package 1:openssl-devel-1.0.1e-51.el7_2.5.x86_64 already installed and latest version
Nothing to do`

Please switch to a less crappy distribution, in this case Debian or Ubuntu

It's faster native library.

Thank you ;)

Just to note,You have to do
git submodule update --init
before applying patches

Yeah forgot about that ;)


Use a standard data encoding

We currently use Amino, we should use a standard serialization format instead, e.g. Protobuf. However, Protobuf does not support random access and always decodes e.g. whole nodes, which may be too expensive when we're simply traversing the tree. Other encodings may be more appropriate.

@erikgrinaker can you elaborate on random access? What are some alternatives? What is the relationship between encoding and the merkle root? If there is none, we can consider gob with streaming APIs.

There will be an ADR on this.

can you elaborate on random access?

We may not always want to load the entire node, but only a field or two. For example, when we're checking if a given key exists it's not necessary to also load the value, which can be arbitrarily large. Protobuf does not support reading a single field, it always reads the entire message (or node, in this case).
Of course, the OS may use read-ahead anyway, so it may not matter - we'll have to look into it.

What are some alternatives?

As far as I know, IAVL currently uses Amino only for encoding of scalar values, which are laid out linearly. We may want to use a similar approach instead of a message-oriented one. In principle, any length-prefixed scalar encoding would do.

What is the relationship between encoding and the merkle root

I don't believe it matters, except for possibly the encoding of the integer node version. We'll have to look into it.

we can consider gob with streaming APIs

I feel like Gob is too tied to Go, I'd prefer something broader so that it's at least possible to access the data in other languages.

I see. Well, if the encoding isn't tied to the merkle-ization of the tree, then being language specific doesn't really matter (WRT to gob). Looking forward to reading the ADR 👍

Well, this turned out to be a bit of a red herring, since we obviously have to load the entire serialized value from the KV store - Protobuf will do just fine.
Sorry about the noise, I'd been thinking about on-disk B-tree layouts all week.

So the Amino-encoding in IAVL really just wraps binary.PutVarint and related varint methods in the Go standard library, and uses varint-prefixed byte slices. It would be trivial to remove Amino (the library) and retain the same encoding using the Go standard library.
I'm not convinced that moving to Protobuf (which would be mostly the same, varints and varint-prefixed byte slices) is worth the breakage, at least not unless we're making other changes at the same time.


MudDataGrid | Hide Column with IF statement in razor not working

Things to check

[X] I have searched the existing issues for this bug
[X] To rule out a caching problem I made sure the bug also happens in an incognito tab

Bug type
Component
Component name
MudDataGrid
What happened?
Can not hide DataGrid column with IF in razor files, it worked before.
When change the condition with a button, then adds the column to the end.
Link: https://try.mudblazor.com/snippet/mawovbcbepWclYeR
Expected behavior
Hide columns inside the if statement when condition is false.
Reproduction link
https://try.mudblazor.com/snippet/mEwSPbcvIfpyszfZ
Reproduction steps

Open the reproduction link
Run the code
Try to play with IF condition or hidden parameter or with the commented code lines in OnInitializedAsync()

Relevant log output
No response
Version (bug)
7.15.0
Version (working)
Worked before, don't know the version.
What browsers are you seeing the problem on?
Firefox, Chrome
On which operating systems are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

it worked before.

When did it worked?

It works strange because when i use prop the it adds the column to the end.
In v7.10.0 as I remember.

In v7.10.0 as I remember.

I will test it later, but I really doubt it ever worked. When you dynamically remove a column, it gets removed from RenderedColumns. Then, when you make it visible again using if (_isVisible), it gets added back to RenderedColumns, but without any information about its original order. In Blazor, there’s no way to get the index of an element within a RenderFragment, so all items are added sequentially when they render. This is a general problem with RenderFragments.
To preserve the order, we would need to add an Order property to PropertyColumn, and you would have to explicitly set it.
Why don't you just use:
 <PropertyColumn Property="x => x.Name" Title="Name with IF" Hidden="@_isVisible"/>

https://try.mudblazor.com/snippet/QaGevbcPeLXoVkvP
It was intended for that case, because in that case it lives inside RenderedColumns, just not visible.

Why don't you just use:
 <PropertyColumn Property="x => x.Name" Title="Name with IF" Hidden="@_isVisible"/>

I will use Hidden and will convert all of my previos code. Thanks for the help!


Find a way to add horizontal scrolling without making the divider disappear

When wrapping a container view with dividers in a HorizontalScrollView, the dividers are not shown.

maybe due to it having a width matching the parent.

All issue are moved to:

https://app.nostromo.io/work/sample-project?e21vZHVsZTp3b3JrLHByb2plY3Q6NTcwNn0=


Update Cilium to v1.8.2

What this PR does / why we need it:
Updates Cilium to v1.8.2
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Cilium is now running on v1.8.2.

/invite @DockToFuture


Feature to copy emoji to clipboard

Upon receiving the results, user can copy the emoji onto their clipboard to use it elsewhere

I guess user can copy the emojis. When user clicks on the emoji that specific emoji gets copied in your clipboard and  there is a message saying "Copied" on the emoji box.
If there is an issue with the copying can you please elaborate in comments and solve the issue. I will assign it to you.

oh okay!


fails on revspec I suspect the issue is a hard coded origin/master that fails to take into account origin/main is a thing.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
[sage] building binary and generating Makefiles...
[convco-check] checking git commits...
[format-yaml] formatting Yaml files...
[format-markdown] formatting Markdown files...
[convco-check] revspec 'origin/master' not found; class=Reference (4); code=NotFound (-3)
[convco-check] exit status 1
make: *** [Makefile:52: default] Error 1
Expected behavior
no error
System (please complete the following information):
https://github.com/einride/sage
Additional context
nothing in the makefile or sage seems to be referring to origin/master directly
https://github.com/einride/sage

I found where it was hard coded to origin/master in the sage stuff in some generated code, that is why I could not find it in the codebase. sorry.


containerd-apply-patch: updated to work with upstream ebuilds

The containerd auto-updater script needs to be updated as we are now basing containerd on upstream ebuilds.
Test action run for the change: https://github.com/flatcar/scripts/actions/runs/6747181933/job/18342745813
PR generated by the change's action run: https://github.com/flatcar/scripts/pull/1336
Test build of that PR: https://github.com/flatcar/scripts/actions/runs/6747213855/job/18342850495?pr=1336

Waiting for the test build of the auto-generated PR to conclude: https://github.com/flatcar/scripts/actions/runs/6745498333/job/18337419310?pr=1333

Tests in containerd update PR concluded, all green: https://github.com/flatcar/scripts/pull/1336


Option to hide first stickied comment

Should hide first comment chain if it is a moderator sticky

Done! Now it's a button


Media being re-imported with each subsequent export

An Airstory customer has reported a case where the WordPress plugin is creating duplicates of images in the media library each time they export from Airstory to WordPress. This shouldn't be the case, so we need to investigate why it might be happening.
No further details are available from the customer at this time, but contact information is available in the Airstory Slack channel.
Response headers from the user's server:
$ curl -I http://wildstory.com/wp-json/airstory/v1

HTTP/1.1 200 OK
Date: Wed, 28 Jun 2017 18:32:25 GMT
Server: Apache/2.2.29 (Unix) mod_ssl/2.2.29 OpenSSL/1.0.1e-fips mod_bwlimited/1.4
X-Powered-By: PHP/5.4.33
Expires: Thu, 19 Nov 1981 08:52:00 GMT
Cache-Control: no-store, no-cache, must-revalidate, post-check=0, pre-check=0
Pragma: no-cache
X-Robots-Tag: noindex
Link: <http://wildstory.com/wp-json/>; rel="https://api.w.org/"
X-Content-Type-Options: nosniff
Access-Control-Expose-Headers: X-WP-Total, X-WP-TotalPages
Access-Control-Allow-Headers: Authorization, Content-Type
Allow: GET
Set-Cookie: PHPSESSID=991b83810f0373e8376589535a06f921; path=/
Vary: Accept-Encoding,User-Agent
X-UA-Compatible: IE=edge
X-Content-Type-Options: nosniff
Content-Type: application/json; charset=UTF-8

I've been unable to reproduce this behavior, but have noted that the "download a separate copy of the original media passed to Cloudinary" logic (see Airstory\Formatting\retrieve_original_media() does result in duplicate copies of images) — it's not being done every time a post is exported, but it does result in two separate attachments.

Reached out to Marc, the Airstory customer who reported the issue and it appears to not be with subsequent exports but rather the original + resized media being imported as two separate attachments.
Ideally, the original image should be what's imported as an attachment, with the resized version as a thumbnail.

Work has begun in the fix/duplicate-media branch, but as I'm digging into this I'm seeing that it's going to require a larger change to the way media is treated, in a way I'm not quite comfortable approaching in a patch release.
Since there's no immediate bug (it's functioning as designed, just not in the most optimal way), I'm going to bump this to the 1.2.0 milestone (the next minor release).


docs: fix links in tool_guide.md

Description: Update the links from /toolkit to /cohere-toolkit
Issues: None
Dependencies: None

Thank you for your contribution! We appreciate it and hope to see more of your contributions in the future 😄
BTW you'll need to update your branch to be able to merge it.

@lakshyaag fixed the merge conflicts for you

Thank you @lusmoura and @tianjing-li - I think the merge is blocked because of review approval.


cmd: Introduce runtime and k8s columns

#1702 added support for k8s metadata in JSON/YAML but the support for columns was still missing. This PR allows the same information to be available as columns as well.
Related: #737
Idea

Before this change both ig and kubectl-gadget would filter k8s and runtime columns respectively rather then hiding. So we will switch to marking them hidden as needed by each client.
The hidden columns can be displayed using -o columns as needed by the user.
list-containers needs an extra work but k8s metadata can also be displayed using -o custom-columns

Tesing done
$ make minikube-start-containerd
$ make ig
$ docker cp ig minikube-containerd:/bin/ig
$ docker exec -it minikube-containerd bash
$ ig list-containers # same as before
RUNTIME.RUNTIMENAME RUNTIME.CONTAINERID                                              RUNTIME.CONTAINERNAME   RUNTIME.CONTAINERIMAGENAME
containerd          0a9cbc23a8409cf015e389fa71f2422f28dbea1129e66a40e38685fb67424ef6 coredns                 k8s.gcr.io/coredns/coredns:v1.8.6
containerd          5e48d5e717d89f2237eb0b364d9c8b6265f949e5600f642d55420e1a4922e2e2 etcd                    k8s.gcr.io/etcd:3.5.3-0
$ ig list-containers -o custom-columns=runtime.containerid,runtime.containername,k8s.pod,k8s.namespace
RUNTIME.CONTAINERID                                              RUNTIME.CONTAINERNAME   K8S.POD                                     K8S.NAMESPACE
0a9cbc23a8409cf015e389fa71f2422f28dbea1129e66a40e38685fb67424ef6 coredns                 coredns-6d4b75cb6d-94hkg                    kube-system
5e48d5e717d89f2237eb0b364d9c8b6265f949e5600f642d55420e1a4922e2e2 etcd                    etcd-minikube-containerd                    kube-system

$ ig trace open -o columns=runtime.containername,k8s.pod,k8s.namespace,pid,comm
RUNTIME.CONTAINERNAME                                                K8S.POD                                                             K8S.NAMESPACE                                                       PID        COMM
gadget                                                               gadget-p4kvb                                                        gadget                                                              448891     runc:[2:INIT]
gadget                                                               gadget-p4kvb                                                        gadget                                                              448891     runc:[2:INIT]

It should also make the implementation of the --k8s and --runtime flags

I'm wondering about what's the idea there. Provide independent flags to enable the runtime and k8s columns? IMO we should avoid adding so many flags, what about having something like -o columns=runtime? (we can try to find a different syntax for making it obvious it's for the full "kind" of columns.)

It's missing to update all gadgets guides (sorry, I now it's to annoying to do it, perhaps a good regexp to replace can help)

Done. Agreed, it wasn't fun to do but I realized we were missing RUNTIME columns for ig in docs as well so thanks for catching it!

It should also make the implementation of the --k8s and --runtime flags

I'm wondering about what's the idea there. Provide independent flags to enable the runtime and k8s columns? IMO we should avoid adding so many flags, what about having something like -o columns=runtime? (we can try to find a different syntax for making it obvious it's for the full "kind" of columns.)

My initial idea was to add --k8s only to ig and --runtime only to kubectl-gadget, but it was just my proposal, we can ask for opinions in the slack channel 🙂.

Thanks for the reviews!


Contrib requests

contrib.requests
Add-On classes to create request bodies for several endpoints like OrderCreate

Coverage decreased (-0.2%) to 99.046% when pulling 1dbb21851cd4cf752c73ab9a1ad5338a13ddfe80 on contrib-requests into c3d54d611759c111f2a64806616f680b56f19fdc on master.

Coverage decreased (-0.1%) to 99.102% when pulling 2d1f1a5a314029b9bda7c22291a2fa6c53752240 on contrib-requests into c3d54d611759c111f2a64806616f680b56f19fdc on master.

Coverage decreased (-0.2%) to 98.988% when pulling 45497e4576b977040c3b65f34b6246b9d7c56a69 on contrib-requests into c3d54d611759c111f2a64806616f680b56f19fdc on master.

Coverage decreased (-0.2%) to 99.048% when pulling a4eda62c6070b5ead91735f194928eff5a6c9184 on contrib-requests into d71bae01cb1546c1f9c8f2fabaab90a918265a07 on master.

Coverage decreased (-0.3%) to 98.966% when pulling 933709e5b57dace734e27946a3e1143ebf55210f on contrib-requests into d71bae01cb1546c1f9c8f2fabaab90a918265a07 on master.

Coverage decreased (-0.1%) to 98.513% when pulling b12ce76a98ba96ea73c4e9c1440f106d84407bca on contrib-requests into d1b3924ddad81e4c7bec738b5137acaf0a491f1f on master.


Patch the latest NMS implementation from TF master

@anhappdev  @freedomtan, please help review, thanks!

Looks good to me. @anhappdev or @d-uzlov, is iOS built failed a real problem?


Fix hang on scene load, when on Mac OS X

When loading a scene, TexturePacker is used which loads some awt classes.
When these awt classes are loaded, the whole awt event loop is started and vis-editor is blocked.
Fix is simple, run awt in headless mode, which allows using Image related functions.

Thanks!


Design: creating intervals.

Current API
In the 10.0.0 release, intervals (both "atomic" and "full") will be constructed via Interval static class, with factory methods. Right now the API looks like this:
public static class Interval
{
    public static Interval<T> CreateAtomic<T>(BoundaryType lowerBoundaryType, T lowerBoundaryValue, T upperBoundaryValue, BoundaryType upperBoundaryType);
    public static Interval<T, TComparer> CreateAtomic<T, TComparer>(BoundaryType lowerBoundaryType, T lowerBoundaryValue, T upperBoundaryValue, BoundaryType upperBoundaryType);

    public static Interval<T> CreateClosedAtomic<T>(T lowerBoundaryValue, T upperBoundaryValue);
    public static Interval<T, TComparer> CreateClosedAtomic<T, TComparer>(BoundaryType lowerBoundaryType, T lowerBoundaryValue, T upperBoundaryValue, BoundaryType upperBoundaryType);

    public static Interval<T> CreateOpenAtomic<T>(T lowerBoundaryValue, T upperBoundaryValue);
    public static Interval<T, TComparer> CreateOpenAtomic<T, TComparer>(BoundaryType lowerBoundaryType, T lowerBoundaryValue, T upperBoundaryValue, BoundaryType upperBoundaryType);

    public static Interval<T> CreateAtomicSingleton<T>(T value);
    public static Interval<T, TComparer> CreateAtomicSingleton<T, TComparer>(T value);

    public static Interval<T> Join<T>(ICollection<Interval<T>> intervals);
    public static Interval<T, TComparer> Join<T, TComparer>(ICollection<Interval<T>> intervals);
    public static Interval<T> Join<T>(IReadOnlyCollection<Interval<T>> intervals);
    public static Interval<T, TComparer> Join<T, TComparer>(IReadOnlyCollection<Interval<T>> intervals);
}

Why factory methods?
Before 10.0.0, interval types in Accretion.Intervals were created via constructors, but this design proved to be a problem, mainly as a maintainability burden. Constructors are, by design, nameless. This means that if the intent of an internal constructor is to provide a "shortcut" initialization without some of the consistency checks, this cannot be done descriptively. This even lead to the weird situation in which internal Boundary structs had factory methods like CreateChecked and CreateUnchecked, but the public ones had to use constructors for the same purpose, with comments like: "Ensure the array is copied before invoking this constructor!". Also, constructors are not quite as flexible as static methods in their implementation, although in our case this is not such a big concern, since all of our interval types will be structs.
This issue will track the evolution of the design.

Creating composite intervals
Methods to create composite intervals:
public static Interval<T> CreateComposite<T>(IList<Interval<T>> intervals);
public static Interval<T, TComparer> CreateComposite<T, TComparer>(IList<Interval<T>> intervals);
public static Interval<T> CreateComposite<T>(IReadOnlyList<Interval<T>> intervals);
public static Interval<T, TComparer> CreateComposite<T, TComparer>(IReadOnlyList<Interval<T>> intervals);

I decided to provide only IList<T> and IReadOnlyList<T> overloads because the implementation will use the number of intervals when creating CompositeInterval (the alternative was to use Enumerable.Count(), which seems unnecessarily inefficient). IEnumerable<T> overload is still under consideration though, as are Span<T>-derived overloads.

The naming of the "full" interval factories has been changed: Join instead of CreateFull. Also under consideration: Combine and Union.

Since the naming for interval types has been decided, factory methods creating "atomic" intervals will drop the "atomic". 'Join' has been decided as the final name for the method creating "full" intervals: while Union seems logical, it will potentially be present on both interval types an instance method, thus using it as a factory name seems questionable.


35 Panther X miners in same location. Winkleigh, England

Hotspot b58 Addresses
https://explorer.helium.com/hotspots/cities/d2lua2xlaWdoZW5nbGFuZHVuaXRlZCBraW5nZG9t
Discord Handle
Kris B#6031
Reason(s)
I have found 35 Panther x miners all in the same area and only witnessing themselves. Please can you look into this as I think they are spoofing the system.

Can be closed. Already in the denylist.


Batch in migrations ignored on rollback

For a module we've created several migrations. All these migrations got their own batch number. As soon as we preform a rollback the batch number is ignored and all migrations will be rolled back.
Also the order of the rollbacks on the migrations are wrong, as they are prefromed ascending while you'd expect descending.

Posted the issue on my work account, anyhow, I created a PR for this issue #264


Support queuing a bulk export job through [base]/$export endpoint

User story
As an engineer, I want to queue a bulk export job so that it can be picked up and processed in the future.
In this user story, we will enable the ability to queue a job through [base]/$export endpoint only.
Acceptance criteria

Verify 202 Accepted is returned if a job is queued successfully.
Verify Content-Location header is returned with the location of the job if a job is queued successfully.  The returned URI does not need to work yet.
Verify 500 Internal Server Error is returned if a job fails to be queued.
Verify a new job record is inserted into the database.

Moved tracking of user story level items to Azure DevOps.


OpenJ9 image missing debug symbols

looks like both JRE and JDK versions are missing native debug symbols. any chance to include them?
I am trying to debug a JVM crash and so far native callstacks are useless...

@ninja- Which platform are you looking for the symbols for? While I'll admit it doesn't seem to be particularly accessible on the web site if you go to e.g. the 11.0.9+11.1 OpenJ9 release in Github there are the debugimage downloads which likely contain what you need.

@sxa isn't the newest openj9 version java13 instead of java11?

I pointed you to the latest LTS release which is 11 as I wasn't sure which version you wanted. The latest java release is 15. 13 is out of support for OpenJ9 (Only the in-support releases are listed on the front of https://adoptopenjdk.net)

OK, I see the proper image would be here: https://github.com/AdoptOpenJDK/openjdk15-binaries/releases/download/jdk15u-2020-11-20-06-22/OpenJDK15U-debugimage_x64_linux_openj9_2020-11-19-20-27.tar.gz

If you're using the nightly build that's correct. If it's the latest 15 GA code then https://github.com/AdoptOpenJDK/openjdk15-binaries/releases/jdk-15.0.1%2B9_openj9-0.23.0 is the place to pick it up from

Oh there are nighly builds? Even better for me. thanks for the info :)

Yep they're at https://adoptopenjdk.net/nightly.html ... Standard disclaimers apply of course - they could be broken but they are built from the latest code. if you need the debug symbols make sure you're using the ones that match the build you're using. I'll close this for now but your issue has highlighted that the web page doesn't have the debug symbols easily accessible so I'll raise that separately in the website repo. Thanks and good luck debugging :-)

@sxa these symbols don't work for me :/ they are still unresolved in javacore*.txt - expect a few ones
Stacktrace:
3XMTHREADINFO3           Native callstack:
4XENATIVESTACK               (0x00007FAC3A667992 [libj9prt29.so+0x58992])
4XENATIVESTACK               (0x00007FAC3A634D73 [libj9prt29.so+0x25d73])
4XENATIVESTACK               (0x00007FAC3A667A0E [libj9prt29.so+0x58a0e])
4XENATIVESTACK               (0x00007FAC3A667B04 [libj9prt29.so+0x58b04])
4XENATIVESTACK               (0x00007FAC3A634D73 [libj9prt29.so+0x25d73])
4XENATIVESTACK               (0x00007FAC3A66786B [libj9prt29.so+0x5886b])
4XENATIVESTACK               (0x00007FAC3A663BFF [libj9prt29.so+0x54bff])
4XENATIVESTACK               (0x00007FAC3BA243D0 [libpthread.so.0+0x143d0])
4XENATIVESTACK               pthread_cond_wait+0x1fa (0x00007FAC3BA1FA7A [libpthread.so.0+0xfa7a])
4XENATIVESTACK               (0x00007FAC3914DA48 [libj9jit29.so+0x164a48])
4XENATIVESTACK               (0x00007FAC3915E168 [libj9jit29.so+0x175168])
4XENATIVESTACK               (0x00007FAC3915E1FA [libj9jit29.so+0x1751fa])
4XENATIVESTACK               (0x00007FAC3A634D73 [libj9prt29.so+0x25d73])
4XENATIVESTACK               (0x00007FAC3915E654 [libj9jit29.so+0x175654])
4XENATIVESTACK               (0x00007FAC3AAAD2F2 [libj9thr29.so+0xe2f2])
4XENATIVESTACK               (0x00007FAC3BA18F00 [libpthread.so.0+0x8f00])
4XENATIVESTACK               clone+0x3f (0x00007FAC3B94445F [libc.so.6+0xfe45f])

A couple of working ones from the javacore:
4XENATIVESTACK               JLI_Launch+0x148b (0x00007F2D0B66391B [libjli.so+0x791b])
4XENATIVESTACK               omrthread_monitor_wait+0x287 (0x00007FAC3AAA77E7 [libj9thr29.so+0x87e7])
4XENATIVESTACK               Java_openj9_internal_tools_attach_target_IPC_waitSemaphore+0x55 (0x00007F2D03B41D25 [libjclse29.so+0x10d25])
4XENATIVESTACK               Java_java_lang_ClassLoader_defineClassImpl1+0x11e (0x00007F2D03B427DE [libjclse29.so+0x117de])


Could you please share the source for the heroku hosted code that actually does the GaneshaSpeaks calls?

Could you please share the source for the heroku hosted code that actually does the GaneshaSpeaks calls?

Horoscope-API is using pyhoroscope package. This repository has all the code for Heroku.
Closing this, reopen if needed.


Consider Adding L4S and NQB Support

Hi - Some networks are working to test/deploy the IETF L4S standard for low latency networking, which creates a 2nd network queue at bottleneck links for packets marked with ECT(1) or DSCP-45 to use a low latency queue with a shallow queue depth. This should in theory improve working latency dramatically.
Perhaps it is worth considering in the future adding support for L4S and NQB. Initially it may just be probing to see if the marks work E2E (a la Is My BGP Safe Yet from CF). Then later test the performance effects.
Timing - The Comcast network is planning to trial L4S from mid-June so one way to test it may come up soon (you can also test in a lab of course).
References:
RFC 9330, 9331, 9332, draft-ietf-tsvwg-nqb

It would be interesting to surface such details but I'm not sure that would be workable from a web browser, where such low level connection control is not usually available from JavaScript.
If those network changes are effective, I'd expect them to result in improved numbers without any changes to the measurement components. And that would be observable (over time) in the public speed test results (which currently only tests performed from speed.cloudflare.com) available through M-Lab's BigQuery:
https://www.measurementlab.net/blog/cloudflare-aimscoredata-announcement/
https://www.measurementlab.net/data/docs/bq/quickstart/


LCS Trial Link Broken

The LCS link does not create a proper LCS trial,  instead, it takes you to the 30-day trial with is not helpful at all.

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

ID: 0eb4c106-c0e3-81ba-db99-1fcdaa5158dd
Version Independent ID: 0f4a76f7-d071-e9ee-4271-866a4ae417f9
Content: Upgrade from AX 2012 to Finance and Operations
Content Source: articles/fin-ops-core/dev-itpro/migration-upgrade/upgrade-overview-2012.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @tariqbell
Microsoft Alias: tabell

Thank you for your feedback. @sericks007 can you please take a look?

@tariqbell or @laneswenka, please update this content as necessary.

BUG 427710 created to track.

I unfortunately don't own trials so not sure how this works today, will follow up with the team.

Hi,
Please can we get this LCS Trial link corrected/updated. Community members trying to initiate upgrade projects are getting stumped at step 1 - I know, I've been on a call with one this week.
Please, if anyone knows the current and correct procedure or link for a customer to get their LCS trial project going please can you share it. This is to enable a customer to perform the Analyze activities for an upgrade before actually purchasing Finance and Operations, which lets them deploy their own Finance and Operations environments and gives access to the tools in LCS that are used to evaluate their AX 2012 environment and existing custom code.
Thanks.

@laneswenka, any update on this?  I believe you were going to check with your team.

This content has been updated.  It now directs you to the Sign up for preview subscriptions article for more information -- which was also updated yesterday, as well.


[Bug Report]  Could not bind to '<IP_ADDRESS>:9000'

Describe the bug
i download the last binary 0.10.0 and i try to lunch for the fist time the app
To Reproduce
Steps to reproduce the behavior:

Go to the install folder
make mango executable
write ./mango on shell
error :

The config file /home/nodoka/.config/mango/config.yml does not exist. Do you want mango to dump the default config there? [Y/n]
Y
The config file has been created at /home/nodoka/.config/mango/config.yml.
[INFO]    2020/08/14 19:03:05 | The queue DB directory /home/nodoka/mango does not exist. Attepmting to create it
[NONE]    2020/08/14 19:03:05 | Initial user created. You can log in with {"username" => "admin", "password" => "blup"}
Unhandled exception: Could not bind to '<IP_ADDRESS>:9000': Address in use (Socket::BindError)
  from ???
  from ???
  from ???
  from ???
  from ???
  from ???

and the process finish
Expected behavior
A clear and concise description of what you expected to happen.
expected to see mango server start
Environment (please complete the following information):

OS: Manjaro rolling Realease
Mango Version v0.10.0

Additional context
also tried in root user. same problem

The error message says that the default port 9000 is already being used. Please edit the config file at /home/nodoka/.config/mango/config.yml to change the port number.


Fix Venoshock 4x effect on poison

Currently Venoshock calculates 4x on "Poisoned" status due to the bonus damage calculation ocurring twice, once in the lines removed from the commit and the second time in this check.
Also the removed lines had the issue of ignoring "Badly Poisoned" status.

Thanks! :+1:


Mention the need to define a reasonable MSRV

The guideline should make mention of defining a reasonable MSRV, how to go about doing so (explain how to install a specific Rust version using rustup and use it for buildling) and why (edition 2015 crates developed on a recent stable might inadvertently start using a recent API or backwards compatible feature).
Inspired by this conversation https://twitter.com/ekuber/status/1438867907388362757

Linking previous discussions: https://github.com/rust-lang/api-guidelines/discussions/123

i find cargo-msrv to be a pretty easy way to determine an MSRV

If we were to add something like this, it should probably start with something like, "If you'd like to define an MSRV, then ..."

I think users should be encouraged to specify cargo-readable MSRV when it lands:
https://github.com/rust-lang/rust/issues/65262

(edition 2015 crates developed on a recent stable might inadvertently start using a recent API or backwards compatible feature)

This is something that had not occurred to me until now (in fact, I'm not sure if I understood the problem completely!), so it would be immensely important for this page to exist in the guidelines with an explanation of this phenomenon, perhaps even with examples.
With that said, I would be inclined to emphasize on the pros and cons of an MSRV, including a possible strategy for choosing when and how to embrace one.

I agree with what's been said. I think the important part is making sure that there's intent behind what the library author is doing and to have a reference on what can be done when they do desire to support old versions, as well as the reasons for doing and not doing them.

Concretely, I'd like to see advice provided for the following situation:
Suppose:

I'm verifying that my library works with Rust 1.234, and intend to continue to do so at least until my next major version; and

I believe that at present it works with some older versions, but I'm not testing that and don't plan to put any effort into keeping that the case.

Should I put rust-version = "1.234" in Cargo.toml or not?
It feels like I should, so as to document my intent, but (if I understand correctly) doing so would also make life less convenient for someone who has reason to use the library with Rust 1.233 right now and would prefer to delay dealing with with any future trouble that causes until it comes.

To be clear; if a dependency updates to use a newer compiler version in a minor update, that could break the minimum version policy without your crate even updating, correct? If so, then either

You need to make sure that every dependency has a compatible MSRV policy. If any dependency can update the MSRV in a minor update, this is not an option.
Or, you must fix the versions in the dependency specification, "1.2, <1.2.3". This is not good. For example, it means security patches require an update of all dependent crates in the dependency tree.

So, having a strict MSRV is only really possible in very select circumstances, namely when all dependencies have it. On the other hand, if you have it, you don't impose anything on dependent crates.
Maybe the advice should then be: Define a MSRV if

It is possible because all your dependencies define it, and
You're willing to handle the additional work associated with developing for and testing with older versions of the compiler.

I think there's a value in specifying MSRV in Cargo.toml as a warning to users of old compiler — such explicit error is much better than a compilation error that makes rustc babble about nighty features and syntax errors.
It may have value someday when Cargo learns to use it for dependency resolution.
But apart from that I think maintaining MSRV more than 3 versions behind is a futile excercise.
Here's the MSRV data for all crates.io crates:
https://lib.rs/stats#rustc

Crates that are compatible with old compilers are mainly because they're also old and abandoned.

Crates that are actively maintained drop old compilers very quickly.

Editions push MSRV up more than any other feature. I expect very soon Rust older than 1.56 will be unusable.

Breakage is very "viral". Even if 99 out of 100 dependencies are compatible, the project is still broken.

Almost all crate incompatibility is caused by dependencies. Few popular crates dictate MSRV for the entire ecosystem (e.g. zeroize for cryptography, and object for error handling crates).

Right, my comment assumed that the MSRV was actually a supported version, but it is a necessary requirement, but possibly not sufficient.

My own opinion on this is that if you don't have a lock file then you shouldn't be concerned with MSRVs. As far as real utility is concerned they're ceremonious at best. As a library author today you have no tools for managing a MSRV. Until that situation changes and there's real mechanisms for maintaining library toolchain compatibility that's in-line with Rust's release train model and can be applied consistently I don't think we should recommend any particular policy around MSRVs.
I don't think that's necessarily the same as deciding to pick and document a specific toolchain in your Cargo.toml and CI though.


[TASK] FMI/FMU exporter of ROM/ExternalModel

Issue Description
Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
RAVEN should be able to export as FMI/FMU ROMs and ExternalModels in order to be used in
external FMI/FMU compatible platforms
Describe alternatives you've considered
N.A

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or task?
[x] 2. Is it tagged with a priority: critical, normal or minor?
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[x] 1. If the issue is a defect, is the defect fixed?
[x] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[x] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[x] 4. If the issue is a defect, does it impact the latest release branch? If yes, is there any issue tagged with release (create if needed)?
[x] 5. If the issue is being closed without a pull request, has an explanation of why it is being closed been provided?

For the record, here are the changes made to PythonFMU:
diff --git a/framework/contrib/PythonFMU/build_pythonfmu b/framework/contrib/PythonFMU/build_pythonfmu
new file mode 100755
index 000000000..290ac5286
--- /dev/null
+++ b/framework/contrib/PythonFMU/build_pythonfmu
@@ -0,0 +1,82 @@
+#!/bin/bash
+# This script is aimed to check and build
+# PythonFMU RAVEN compatible
+# With this script we can support all the OSs
+# we support in RAVEN
+# @author, Andrea Alfonsi
+# @date, 05/05/2021
+
+# establish conda
+../../../scripts/establish_conda_env.sh --load
+# fail if anything here fails
+set -e
+echo "start PythonFMU (RAVEN compatible) installation ......"
+# firstly lets check if the resources/binaries folders are empty or not
+# (some binaries are released with the code). At least for the current
+# operating system. Supporter are Win64, linux64, darwin64
+
+OSDIR=""
+DIRB="./pythonfmu/resources/binaries"
+if [[ "$OSTYPE" == "linux-gnu"* ]]; then
+  OSDIR="linux64"
+elif [[ "$OSTYPE" == "darwin"* ]]; then
+  OSDIR="darwin64"
+elif [[ "$OSTYPE" == "cygwin" ]]; then
+  OSDIR="linux64"
+elif [[ "$OSTYPE" == "msys" ]]; then
+  OSDIR="win64"
+elif [[ "$OSTYPE" == "win64" ]]; then
+  OSDIR="win64"
+else
+  echo "OS $OSTYPE not supported!"
+  exit 1
+fi
+
+BINARYFOLDER="$DIRB/$OSDIR"
+BUILDIT=0
+# look for empty dir
+if [ "$(ls -A $BINARYFOLDER)" ]; then
+  echo "** PythonFMU binary folder for OS '$OSDIR' has been found at $BINARYFOLDER. No build is required!"
+else
+  echo "** PythonFMU binary folder for OS '$OSDIR' has NOT been found at $BINARYFOLDER. BUILD is required!"
+  BUILDIT=1
+fi
+
+if [[ $BUILDIT == 1 ]];
+then
+  echo "** Building RAVEN version of PythonFMU ..."
+  DIR="./pythonfmu/pythonfmu-export"
+  cd $DIR
+  if [ -d "build" ]
+  then
+    echo "** build directory exists. Not creating a new one ..."
+  else
+    # make the directory
+    mkdir "build"
+  fi
+
+  cd build
+
+  echo "***** MAKING BINARIES *"
+  OUTPUT=$(cmake .. -DCMAKE_BUILD_TYPE=Release)
+  echo "${OUTPUT}"
+  OUTPUT=$(cmake --build .)
+  echo "${OUTPUT}"
+  echo "***** MAKING BINARIES SUCCEDED*"
+  cd ../../../
+fi
+
+echo "** Check binaries in resource folder *"
+if [ "$(ls -A $BINARYFOLDER)" ]; then
+  echo "***** binaries successfully installed at '$BINARYFOLDER'*"
+else
+  echo "***** ERROR: binaries not found at '$BINARYFOLDER'*"
+  exit 1
+fi
+
+echo "** Installing RAVEN compatible PythonFMU library *"
+# FIXME: uncomment when new version of pip is installed by RAVEN
+#pip install . --use-feature=in-tree-build >$(tty)
+OUTPUT=$(pip install .)
+echo "${OUTPUT}"
+echo "...... PythonFMU (RAVEN compatible) installation done!"
diff --git a/framework/contrib/PythonFMU/pythonfmu/builder.py b/framework/contrib/PythonFMU/pythonfmu/builder.py
index a96d51b3e..4988d6887 100644
--- a/framework/contrib/PythonFMU/pythonfmu/builder.py
+++ b/framework/contrib/PythonFMU/pythonfmu/builder.py
@@ -66,6 +66,7 @@ class FmuBuilder:
         dest: FilePath = ".",
         project_files: Iterable[FilePath] = set(),
         documentation_folder: Optional[FilePath] = None,
+        file_name: Optional[FilePath] = None,
         **options,
     ) -> Path:
         script_file = Path(script_file)
@@ -122,7 +123,10 @@ class FmuBuilder:
                 temp_dir.absolute() / script_file.name, module_name
             )

-            dest_file = dest / f"{model_identifier}.fmu"
+            if file_name is None:
+                dest_file = dest / f"{model_identifier}.fmu"
+            else:
+                dest_file = dest / file_name

             type_node = xml.find("CoSimulation")
             option_names = [opt.name for opt in FMI2_MODEL_OPTIONS]

Closure checklist passed...
Closing approved via #1481


Dev/events-calendar

https://github.com/rit-sse/WebsiteTheSSEquel/assets/111908488/bb1147b2-328c-446d-980b-8e80cdf6bba8
As a visitor, I want to view a calendar of SSE events so that I can plan ahead to stay engaged with the organization and aware of its activities.

IT LOOKS GOOD, I like the responsive mobile stuff 💯 💯 💯 💯 💯 💯 💯 💯


How to Understand BERT Model with Multiple [CLS] Output？

class ExtSummarizer(nn.Module):
    def forward(self, src, segs, clss, mask_src, mask_cls):
        top_vec = self.bert(src, segs, mask_src)
        sents_vec = top_vec[torch.arange(top_vec.size(0)).unsqueeze(1), clss]
        sents_vec = sents_vec * mask_cls[:, :, None].float()
        sent_scores = self.ext_layer(sents_vec, mask_cls).squeeze(-1)
        return sent_scores, mask_cls

i guess the above code extract serveral BERT [CLS] outputs, but in my understanding, BERT model only have one [CLS] output?
hope someone can help me figure it out

The paper said that for extractive summarization to work, it needs multiple CLS tokens to tell the model that there are many labels that we're predicting. So the output of the same position of CLS token is the probability that we should select ith sentence as a part of the summary.


Linux build support

Only tested on 6.95 as it's the only version I have available atm. Build is successful and can confirm the plugin works with idaq and idaq64.

Hi, thanks for contributing! Please check out the develop branch, I completely moved the build logic to an extra repo, having it as a submodule in order to allow other plugin authors to reuse the CMake stuff, I just forgot to merge that stuff to master. Would you mind creating a PR in the other repo?


fix(website): adjust header color on light theme

What is it?

[ ] Feature / enhancement
[ ] Bug
[x] Docs / tests / types / typos

Description
Quick adjustment of header colors when light mode is active.
Current

Adjusted

Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

@zanettin you beat me to it! I think this diverges from the designs though, what we want is a blue header like I was just adding (for both modes)
https://github.com/BuilderIO/qwik/pull/4007
your SVG CSS was helpful to save me some time though, ported that over to this PR ^

ah nice 🙏 happy to close this hacky solution pr 😆


controller: Separate ps and worker pods

/assign @ScorpioCPH
We do not separate the pods of PS and workers in the controller, then it has bugs when the replicas are succeeded.
If the PR LGTM, I will file another PR for service.
Signed-off-by: Ce Gao<EMAIL_ADDRESS>

This change is

Coverage increased (+0.2%) to 59.071% when pulling 62630d7429b00c9304ca657deff21fb7a756b6cc on gaocegege:succeeded into cb5b9941663b33f623bbcfad8f90dd44c4a3370f on kubeflow:v1alpha2.


How and where to change the spark history server port number in dr-elephant

Spark 2.4.4 version
emr release 5.29
Due to some constraint, i can not use port number 18080 for spark history server. Spark history server points to other than 18080(example: 18480).
I am getting below error message in dr-elephant log.
04-29-2020 05:31:54 ERROR [ForkJoinPool-1-worker-29] com.linkedin.drelephant.spark.fetchers.SparkRestClient : error reading applicationInfo http://master-dns-name:18080/api/v1/applications/application_1588136450759_0001. Exception Message = java.net.SocketTimeoutException: connect timed out
I have followed this guide to install dr.elephant in EMR- https://aws.amazon.com/blogs/big-data/tune-hadoop-and-spark-performance-with-dr-elephant-and-sparklens-on-amazon-emr/

Hi @selvamraman,
You can make these changes to the config named spark.yarn.historyServer.address that is generally present in the spark-defaults.conf file of the conf folder of Spark.
@selvamraman if possible can you provide some insight regarding your usage of Dr.Elephant, like

what Fetcher are you using for Spark applications?

what version of Spark and Hadoop you are using?

This information would be helpful for us in providing a resolution to your issues.

@ShubhamGupta29
AWS EMR Cluster details:
Release label:emr-5.29.0
Hadoop distribution:Amazon 2.8.5
Applications:Spark 2.4.4, Hive 2.3.6
FetcherConf.xml:::

spark
com.linkedin.drelephant.spark.fetchers.SparkFetcher

<use_rest_for_eventlogs>true</use_rest_for_eventlogs>
<should_process_logs_locally>true</should_process_logs_locally>

@selvamraman hope the reply resolved your issue, if so then can I close this issue?

@ShubhamGupta29
Thanks for the help. i have changed the port number in the conf and now dr.elephant try to fetch the information from 18480. But, still i am facing the problem
05-05-2020 19:30:53 ERROR [ForkJoinPool-1-worker-5] com.linkedin.drelephant.spark.fetchers.SparkRestClient : error reading applicationInfo http://dns_name:18480/api/v1/applications/application_1588700272512_0001. Exception Message = java.net.SocketException: Unexpected end of file from server
The spark history server address is not listening http and it is https -
https://dns_name:18480/api/v1/applications/application_1588700272512_0001.
how to include https for history server in the config?
is it something i can change in the spark-defaults.conf
spark.yarn.historyServer.address=https://dns-name:18480
Also, what are the environment variable do i need to setup before start the EMR?

spark conf
spark home
etc

Hi @selvamramam,
For using HTTPS, if your  spark.yarn.historyServer.address has HTTP then change that to HTTPS and you also need to make some changes to the Dr.Elephant's SparkRestClient.scala class. There is an else condition in the calculation for an attribute named historyServerUri.
Environment Variable needed before setup:

SPARK_HOME
HADOOP_HOME
HADOOP_CONF_DIR

I will let you know the final result. Thanks you so much @ShubhamGupta29 your prompt response.

@ShubhamGupta29  and @selvamraman
Hi there. I am facing the same issue because the spark that I am using on the cluster only supports https URL. And I also could not make it.
Here is the detailed description.
After the compilation and bin/start.sh. I got some dr-elephant logs like this:
WARN  [dr-el-executor-thread-1] com.linkedin.drelephant.spark.fetchers.SparkFetcher : Failed fetching data for application_1592940011846_0014. I will retry after some time! Exception Message is: Illegal character in authority at index 7: http://${hadoopconf-yarn.resourcemanager.address}:50509
It seems that dr-elephant did not parse that ${hadoopconf-yarn.resourcemanager.address} correctly. Since the actual spark history server URL on my cluster is https://aaa.bbb.ccc.ddd:50509.  So I firstly modify the spark configuer file $SPARK_CONF_DIR/spark-defaults.conf from
spark.yarn.historyServer.address ${hadoopconf-yarn.resourcemanager.address}:50509 to spark.yarn.historyServer.address https://aaa.bbb.ccc.ddd:50509.
And I went to app/com/linkedin/drelephant/spark/fetchers/SparkRestClient.scala, changes the historyServerUri  related condition check from
    if (historyServerAddress.contains(s"http://")) {
        new URI(historyServerAddress)
    } else {
        new URI(s"http://${historyServerAddress}")
    }

to
    if (historyServerAddress.contains(s"http://") ) {
        new URI(historyServerAddress)
    } else if (historyServerAddress.contains(s"https://")) {
        new URI(historyServerAddress)
    } else {
        new URI(s"http://${historyServerAddress}")
    }

Now, after recompile and start the dr-elephant, dr-elephant still cannot fetch the spark jobs. I checked logs, it says
WARN  [dr-el-executor-thread-0] com.linkedin.drelephant.spark.fetchers.SparkFetcher : Failed fetching data for application_1592940011846_0016. I will retry after some time! Exception Message is: javax.net.ssl.SSLHandshakeException: java.security.cert.CertificateException: No name matching aaa.bbb.ccc.ddd found

I am not an expert on the scalar. It seems there are still issues about SSL connection.
Any ideas? Thanks in advance.
P.S.

env variables SPARK_HOME, HADOOP_HOME and HADOOP_CONF_DIR are all set correctly.
MapReduce jobs could be fetched by dr-elephant.  Spark jobs could not.
spark configuration is correct. And can run spark jobs on the cluster.
using the following setting in app-conf/FetcherConf.xml<fetcher>
      <applicationtype>spark</applicationtype>
      <classname>com.linkedin.drelephant.spark.fetchers.SparkFetcher</classname>
      <params>
          <use_rest_for_eventlogs>true</use_rest_for_eventlogs>
          <should_process_logs_locally>true</should_process_logs_locally>
          <event_log_dir>webhdfs:///mapred/sparkhistory</event_log_dir>
      </params>
</fetcher>

My MapReduce History server also using https URL. But MapReduce jobs are fetched correctly. I hope this might help.


[Minor] Fix runnability of RLHF example in examples/rlhf

Description
Small amount of bit rot on the RLHF example.

Correct passing of configuration in train transformer and reward steps
Fix typo on prompt logger
Update log_scalar to match class spec.

Motivation and Context
There was one larger issue which I didn't address, which was that tensordictmodule wasn't playing well with dynamo, due to the optimizedmodule not forwarding getattr calls to the underlying module: I didn't get round to verifying that was fixed in nightlies (though I think it might be), but I did think about about updating the train/train_reward configs to default compile to be off as it will fail in the stable release version with TypeError: _forward_unimplemented() got an unexpected keyword argument 'input_ids' or similar. Its easy enough to disable compile in config or on command line, but it might be worth disabling in the base config as well.
Types of changes
What types of changes does your code introduce? Remove all that do not apply:

[ x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds core functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (update in the documentation)
[ ] Example (update in the folder of examples)

Checklist
Go over all the following points, and put an x in all the boxes that apply.
If you are unsure about any of these, don't hesitate to ask. We are here to help!

[ x] I have read the CONTRIBUTION guide (required)
[ ] My change requires a change to the documentation.
[ ] I have updated the tests accordingly (required for a bug fix or a new feature).
[ ] I have updated the documentation accordingly.

which was that tensordictmodule wasn't playing well with dynamo, due to the optimizedmodule not forwarding getattr calls to the underlying module

Yep! tensordict (and related modules) compatibility with dynamo and the rest of the compile stack is a big TODO. Happy to chap about what the roadmap for this looks like offline.


Doc skeleton

note: please let me know if this should be placed in Substra/substra-documentation instead
This is to be a skeleton for the new documentation of Substra as discussed with @RomainGoussault and @Esadruhn
The sections not filled here will be taken care of in a different PR, here:

we set the Sphinx structure for the documentation (using .rst files)

to be filled with text + references

set the Example gallery, here we are using Sphinx-gallery (therefore existing Example directory is renamed to code_examples)

include 1 Titanic example (to be filled with more comments and references)

The Theme has been chosen randomly and will be updated in the future
preview of the main page:

shifting this PR to substra-documentation repo


Add test that private keys are not extractable

Description
Adds test that RSA and EC private keys cannot be extracted.
Updates C_GetAttributeValue to return the correct error code when RSA private key is queried. (Previously returned CKR_ATTRIBUTE_TYPE_INVALID).
Checklist:

[ ] I have tested my changes. No regression in existing tests.
https://amazon-freertos-ci.corp.amazon.com/job/master/job/custom_ide/268/console
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run checks


Suggest using simple port forwarding instead of LoadBalancer for cloud deploy in User Guide

From @dakoner on May 23, 2018 16:49
The User Guide documents adding a LoadBalancer to expose the tf-hub in a cloud deployment with the stated caveat it opens the port to the internet.  There is an additional document for using IAP, which is more secure but also more complex.
I suggest documenting a simpler port forward, which just makes the tf-hub available to the local machine:
kubectl port-forward --namespace kubeflow $(kubectl get pod --namespace kubeflow --selector="app=tf-hub" -o jsonpath='{.items[0].metadata.name}') 8080:8000
After running this command, I was able to connect to the tf-hub, start a new jupyter instance, create a new notebook, and evaluate a simple python statement.  The resulting hub port forward is available on localhost of the forwarding machine, so any user on the machine has access.  That's not perfect but it's better than opening the hub to the internet.
Copied from original issue: kubeflow/kubeflow#860

From @jlewi on May 23, 2018 17:8
This makes sense.
I think the general approach we want to recommend is that users connect to the ambassador reverse proxy so they get access to all the services.
This is blocked though on #817 which will put JupyterHub behind the ambassador.
Once everything is available through ambassador we can provide suitable recipes for connecting to ambassador e.g.

Port forward
Expose ambassador on loadbalancer or node port ( this is safe for on prem/minikube if you are on a protected network accessible from corp vc but not externally)

/area docs

This is documented here:
https://www.kubeflow.org/docs/guides/accessing-uis/


Rubix size based eviction triggering too early

I am seeing unexpected SIZE based evictions in cluster with Rubix caching enabled:
2020-05-05T17:49:00.908Z        WARN    20200505_174449_00008_sug9p.1.0-2278-82 com.qubole.rubix.bookkeeper.FileMetadata        Evicting<EMAIL_ADDRESS>due to SIZE
2020-05-05T17:52:18.308Z        WARN    20200505_175007_00009_sug9p.1.0-1301-98 com.qubole.rubix.bookkeeper.FileMetadata        Evicting<EMAIL_ADDRESS>due to SIZE
2020-05-05T17:58:58.931Z        WARN    20200505_175802_00010_sug9p.1.0-287-54  com.qubole.rubix.bookkeeper.FileMetadata        Evicting<EMAIL_ADDRESS>due to SIZE

The cluster has single worker.
The cluster is configured in non-parallel-warmup mode.
The cluster runs a query in a loop which reads whole table.
The table data consists of 5 files. Each ~35GB in size. Total size ~170GB.

The size of cache partition is 485GB.

And eviction threshold is set to 80%.
In such situation evictions are unexpected. My guess is that those are happening because of best-effort nation of size-based evictions in Guava cache but I need to investigate more.

cc: @stagraqubole

This too is fixed in https://github.com/qubole/rubix/pull/368, specifically by https://github.com/qubole/rubix/pull/368/commits/918bf1ab667883805a8c2962ebf7aaee2a7731c7

Fixed via: https://github.com/prestosql/presto/pull/3772


[fontbakery fix-dsig] fake dsig table is broken.

Observed behaviour
Microsoft FontValidator:

<Report ErrorType="E" ErrorCode="E0502" Message="The digital signature of the file did not verify" Details="Offset and length were out of bounds for the array or count is greater than the number of elements from index to the end of the source collection." TestName="DSIG_VerifySignature" />

first reported in https://github.com/googlefonts/fontbakery/issues/1430#issuecomment-315734410
With a response of @HinTak:

Well, that's what it says, I'd say your fake dsig table is broken... there should also be a command-line tool called DSIGInfo, which takes one or more "-v" 's and may give you more info. Anyway, the bulk of the DSIG related code is in https://github.com/HinTak/Font-Validator/blob/master/DSIGInfo/DSIGInfo.cs , and please free to play with it and see which part of your dsig table it does not like :).

Expected behaviour
A none broken fake DSIG table created by fontbakery fix-dsig

https://github.com/googlefonts/fontbakery/issues/1430#issuecomment-315767549:

The rest of the dsig code is in:
https://github.com/HinTak/Font-Validator/blob/master/OTFontFileVal/val_DSIG.cs
which wraps around the other file. Your file seems to have fallen into the block around line 119 to 123, which is the catch-all "shit happens" clause that your dsig table is doing something entirely unexpected...

Looks like our implementation goes back to this typedrawers post by Ben Kiel http://typedrawers.com/discussion/comment/9651/#Comment_9651
getting this ttx dump:
<?xml version="1.0" encoding="UTF-8"?>
<ttFont ttLibVersion="3.6">

  <DSIG>
    <!-- note that the Digital Signature will be invalid after recompilation! -->
    <tableHeader flag="0x1" numSigs="1" version="1"/>
    <SignatureRecord format="1">
-----BEGIN PKCS7-----
0000000100000000
-----END PKCS7-----
    </SignatureRecord>
  </DSIG>

</ttFont>

I think for an dummy dsig table, you want numSigs=0, not 1, at least!

If you want a dummy but valid NumSigs = 0 example, the amiri-*.ttf from http://www.amirifont.org (I am just looking at the Fedora shipped version of those fonts and where they are from according to the rpm database) do it correctly. They are not the only one - I am just looking at all the fonts with a DSIG table on fedora, as I did test the code with all such in fedora 24(?) over a year ago, and remember seeing that. And this one starts with a "a" . This is what it dumps - note NumSigs = 0:
  <DSIG>
    <!-- note that the Digital Signature will be invalid after recompilation! -->
    <tableHeader flag="0x0" numSigs="0" version="1"/>
  </DSIG>

To create this DSIG table the fontTools code looks like this:
          font = ttLib.TTFont(path)
          newDSIG = ttLib.newTable("DSIG")
          newDSIG.ulVersion = 1
          newDSIG.usFlag = 0
          newDSIG.usNumSigs = 0
          newDSIG.signatureRecords = []
          font.tables["DSIG"] = newDSIG
          font.save(path)

And I can confirm that it passes FontValidator.exe. Though, I don't know about compatibility with i.e. directwrite or what else is looking at DSIG. I remember though that @carrois mentioned at TypoLabs 2017 that his DSIGs are just empty. Checking FiraSans-Regular.ttf
I indeed get:
  <DSIG>
    <!-- note that the Digital Signature will be invalid after recompilation! -->
    <tableHeader flag="0x0" numSigs="0" version="1"/>
  </DSIG>

Has anyone strong feelings about this or can I prepare an updated fontbakery-fix-dsig.py?
CC: @davelab6 @m4rc1e @felipesanches

I was just cleaning up my git stash and found this lying around. Before we merge, in the issue @HinTak writes:

However, you probably want to check that this works for the undocumented use of it in directwrite ...

So, what are these applications that need a (dummy) DSIG to be present and who can test this?


Improve graphics quality and performance

Describe the issue
Improve graphics quality and performance.
I compared the quality and performance of UTM vs VMWare Fusion with both Windows 11 and Ubuntu 23.10 as guests and the graphics quality in UTM could be improved vs Fusion.
It also feels more sluggish and see some tearing on UTM when simply dragging windows around or opening menus.
I used virtio-ramfb-gl (GPU Supported) and did not try any other emulations.
Configuration

UTM Version: 4.4.5
macOS Version: Sonoma 14.3
Mac Chip: M2

This not a fault of UTM, it uses QEMU under the hood which does not currently emulate any GPU that Windows haz proper 3D drivers for.
You are going to want to set the Spice QXL GPU as the display adapter and install UTM’s guest tools to get basic 2D support with automatic resolution switching.

@ThatStella7922 thank you. I do have UTM Guest Tools installed in Win 11. Which one is Spice QXL GPU in UTM? I use the default:

Automatic resize works and it's usable but compared to VMware Fusion it feels slightly sluggish and lower quality in terms of sharpness (I have Retina Mode enabled in UTM and use 2x scaling in Win 11).

Seems like the gpu you have selected now supports all of the features of the old Spice GPU. My bad for not keeping up to date!
unfortunately there is no real fix to make it feel less sluggish, with the current gpu drivers in Windows you have to do CPU rendering

Thank you for clarifying, this makes sense and at least now I know it's not a configuration issue on my side.


Added More Functionality in Fraction Calculator

Fixes Issue🛠

Closes #1467
Description👨💻

Here is a summary of the changes made to the Fraction Calculator project:

HTML Update :

Added a new input field in the form to display the decimal result.

JavaScript Update :

Updated the solve function to calculate and display the decimal equivalent of the resulting fraction.
Included a check in the display function to alert if the result is not a real number.

CSS Update :

Styled the new input field for the decimal result to match the existing input fields.

README Update :

Updated the project description to include the new functionality of converting fractions to decimals and checking if the result is a real number.
Added detailed instructions on how to use the updated features.

Type of Change📄

[x] Bug fix (non-breaking change which fixes a bug)
[x] New feature (non-breaking change which adds functionality)
[x] Style (non-breaking change which improves website style or formatting)
[x] Documentation (non-breaking change which updates or adds documentation)
[x] Responsiveness (non-breaking change which improves UI/UX on different screen sizes)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project
[x] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

@Dark-S8uL Actually you have passed decimal result to Number.isFinite(decimalResult) method which will always return true.

@Rakesh9100 PR looks good.


feat(Gamut)!: Added underlines to Text links

Overview
PR Checklist

[x] Related to Abstract designs: https://app.abstract.com/projects/a4a24600-c8c9-11e8-a177-81a5fa367641/branches/328ef9fe-09cf-43f4-8dd1-40947f5d4fb4/commits/3a51a13f42daf13efbfd15dc96f34ad00a72052d/files/FA9AF2C4-35B2-4183-8E93-A9D84A525613/layers/7598FF01-E0C7-4FAD-A505-B6136FA4DC91?mode=design
[x] Related to JIRA ticket: A11Y-377
[x] I have run this code to verify it works
~[ ] This PR includes unit tests for the code change~

Description
Per the designs, we want our general-purpose links to be purple and have underlines. Our global "reboot" styles already apply the purple color but removes the underline; this adds in the underline. If we get rid of "reboot" we can get rid of the underline here too probably.
We talked on Slack about how applying to <Markdown> initially would be a good entry into accessible links: that's in #793. This starts on our other general-purpose body text container.

📬Published Alpha Packages:@codecademy/gamut-labs@1.7.9-alpha.62583e.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5ec5c5716fe15b01734f9428--codecademy-styleguide.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5ec5e196fd1cc68bdfc61dc3--codecademy-styleguide.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@1.7.9-alpha.fba349.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5ec6973a0154b5017977de40--codecademy-styleguide.netlify.appDeploy Logs

Closing because we're leaning towards using the Anchor component or some equivalent.


Alex

Additions include code surrounding deployment to AWS infrastructure.

settings.py should have * in allowed hosts until domain is purchase

@agiannini you should rebase on main, there have been lots of changes there so there's a risk that you overwrite things otherwise. That will also get the linting and Django tests to run which I would like to see pass before merging this.

closing


Scope system rework

Continuously updated
Tracker for details regarding the current and future form of Bait's scope system.
Changes will be split up into multiple PRs.
Status Quo
Hierarchy:
global scope
+-- file 1
  +-- blocks
    +-- ...
  +-- ...
+-- file 2
+-- ...

Storage:
table.Table.pkg_scopes map[string]&ast.Scope
Contents high-level:

universe

primitive types

FFI

all C / JS declarations depending on the backend

package scope

globals
constants
functions

block scope (file toplevel)

import aliases

block scope (sub blocks)

everything else

ScopeObject value variants:
// normal
table.pkg_scopes[full.pkg]Obj{
  name = name
}

// builtin
table.pkg_scopes["builtin"]Obj{
  name = name
}

// ffi
table.pkg_scopes["FFI"]Obj{
  name = pkg.name
  lang = lang // this might not even be needed
}

Progress:

#248

constants in FFI scope
globals in pkg scope

As of #257 the reworked scope system has fully replaced all functionality of the old one.
Next up I will expand it's usage for symbols that are currently tracked in some other messy way.


Remove unneeded code that was causing error

Fixes error: Cannot assign to read only property 'data' of object '[object ImageData]'
Error appears to have appeared now after updating TS library in project. The issue has been identified in original library since 2017, and PR have been created to fix this issue but author hasn't merge them.

@pa7 Is there a chance you could approve this PR?

Is this going to get merged?

This project appears to be abandoned. For anyone else experiencing this issue - switching to 'leaflet.heat' was easy enough.
https://www.npmjs.com/package/leaflet.heat


SmartRecruiters appears glitched

Hello!
I'm running Windows 10, Mozilla Firefox Developer Edition 41.0a2 (2015-07-30) (Aurora update channel) and I experienced a graphical glitch upon scrolling to the SmartRecruiters section while browsing the main page.
The section of the website:

Closeup of the graphical glitch:

Thanks,
Jack

Thanks, fixed. (Should show up on the site soon)


PI 4?

I haven't heard of any mention of if this would work on a pi 4 or is it to early. I didn't see anything posted about it so thought I'd give it a go.

In theory, it should work, I don't have one to try it on but feel free to give it a shot and if you do report back here!

How did you go with pi 4 @bugilt?
In theory can we expect better performance on pi 4 with more fps?  Any chance it would be able to run gui window at same time as visualisations?

I can't seem to find my usb audio. I've ordered a new one. should be here tomorrow.

I'm getting average 41 fps with desktop and gui running over remote desktop.

How u can run it remotely? Im trying to archive that with no success
El jue., 1 de agosto de 2019 05:30 PM, bugilt<EMAIL_ADDRESS>escribió:

I'm getting average 41 fps with desktop and gui running over remote
desktop.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/scottlawsonbc/audio-reactive-led-strip/issues/222?email_source=notifications&email_token=ALHKYG52B3ZMNMKUBU55SV3QCNPWTA5CNFSM4IFRQZ22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3MCDVI#issuecomment-517480917,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALHKYGYBVKYVDU67FK2QM4TQCNPWTANCNFSM4IFRQZ2Q
.

I'm running it with VNC.

Seems like this is possible to run on the pi 4 so I will close this issue. Reopen if there is something else you would like to address.


Bye-bye blaze

This is a bit premature, but will help unblock https://github.com/http4s/http4s/pull/6403.
The open PRs seem like they might need another cycle or two, so I'll start preparing a PR for the blaze repo with what's currently on series/0.23.
Edit: also, a blaze maintainer will have to transfer the issues.

I can make you a blaze maintainer, but some would view that as more of a threat than an offer.

Yep definitely a threat 😆 hmm yeah it's a lot of issues, surely somebody else would like to transfer them? Otherwise I will click as needed, and then unsubscribe.

Voila:

https://github.com/http4s/blaze/pull/628


Icon component "description" property should not be required

Summary
The Icon component requires the description property, and uses a default value if none is given, using the property for an aria-label and alt attributes and also as the text of a title element. I would like to request that the Icon component not require this property and that it only populate the other fields if a description is given. This will allow the Icon component to be used in ways that do not involve direct user interaction.
Justification
Our product uses the Icon component, and we have run into situations where the fields that this component fills in with the default are not useful to us. (E.g. when we're using an Icon as a purely presentational component or when we're using tooltips in place of title attributes)

The title attribute is the part that's really giving us trouble in particular, because we don't want text to appear in that way on hover (since we already have a tooltip on hover). If you want description as required and keep the aria-label and alt attributes, that should work for us.

Thank you for providing us with more details @ColbyJohnIBM! We can introduce title prop to <Icon> where you can set null or an empty string to remove the SVG <title>. Wondering if it will make sense to you.

I think that makes sense and should work for our needs.

Figured out that there is we have #726, associated with a PR with the code exactly like what I imagined (#1319). Closing, but don't hesitate to speak up in case #1319 wasn't what you wanted. Thanks!


Bugfix/aer backend info

Refactored aer.py to hopefully make it clearer what settings are being used for AerBackend, AerStateBackend, and  AerUnitaryBackend.

set supports_fast_feedforward=True for  AerBackend, Resolves #81

use same gate set for backend_info.gate_set, gate set predicate, and rebase pass, Resolves #75

Do I need to update the extension version?

Do I need to update the extension version?

yes and it would be great if you could add something to the changelog in /docs/changelog.rst to explain what is changed.
You can just add a new section with a "unreleased" 0.37.0

I wrote a small docstring for the Noise model characterisation dataclass. While looking over it I realized i could clean up it's usage a bit and then also get rid of its __post_init__ function.


Migrating to Github CI

This should allow us to migrate to Github CI

This has been tested on a separate branch and does seem to deploy correctly. We'll see if that's actually true ^^

This has been tested on a separate branch and does seem to deploy correctly. We'll see if that's actually true ^^

It seems to work locally with act, but requires a custom Docker image (see https://github.com/nektos/act/issues/107)
FROM ubuntu:20.04

RUN apt-get update \
    && apt-get upgrade -y \
    && DEBIAN_FRONTEND=noninteractive apt-get install -y \
    build-essential \
    curl \
    git \
    nodejs \
    npm \
    && rm -rf /var/lib/apt/lists/*

followed by sudo act -P ubuntu-latest=ubuntu-builder

It seems to work locally with act, but requires a custom Docker image (see https://github.com/nektos/act/issues/107)
FROM ubuntu:20.04

RUN apt-get update \
    && apt-get upgrade -y \
    && DEBIAN_FRONTEND=noninteractive apt-get install -y \
    build-essential \
    curl \
    git \
    nodejs \
    npm \
    && rm -rf /var/lib/apt/lists/*

followed by sudo act -P ubuntu-latest=ubuntu-builder

Is this still intended to be merged? If so, I can take a look.

Yeah. I would like to merge it after a review. I don't trust myself to do CI right ^^

Would you mind if I made a PR with a different pipeline setup?

I am closing this for now because we have merged #821 instead


Start elegant - Keep it elegant

The schema representation is just elegant, better than this it could not be.
What about making the query in the same elegant way? Take a look at what Ponyorm did.

We're keeping it very simple - folks should use an ORM for that kinda thing.


preset/wind -> grid-template-columns value

I am not sure, but it looks like bad implementation of grid-cols

as we can see, it puts comma between values, but in the windi docs, there is no comma

https://windicss.org/utilities/layout/grid.html

UnoCSS uses _ instead of , for seperator

ok, didn't know, I was searching here https://uno.antfu.me/?s=grid-cols
Thanks for clarify

UnoCSS uses _ instead of , for seperator

@antfu I think this information should be posted on the main page

UnoCSS uses _ instead of , for seperator

Ah nice! I just spent an hour debugging everything 😅! On the plus side I looked into every packages and really like the project.
I think it should be part of the Tailwind migration guide too as it works the same in tailwindCSS !