7.1 [BACKLOG-15685] As a Chris, I want to be able to disable steps when r…

Backport of https://github.com/pentaho/pentaho-kettle/pull/3838
@mkambol

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -f engine/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/trans/ael/adapters/TransMetaConverter.java
  engine/test-src/org/pentaho/di/trans/ael/adapters/TransMetaConverterTest.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -.39%:small_red_triangle_down:
Complexity Change: -.51%:small_red_triangle_down:
Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change:  + 1.01%
Complexity Change:  + 1.06%
Instruction Change:  + .17%

org.pentaho.di.trans.ael.adapters.TransMetaConverter

Complexity Change:  + .17%
Instruction Change:  + .28%
Line Change:  + .35%
Method Change:  + .26%

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Instruction Change:  + .11%

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change:  + 4.17%
Complexity Change:  + 4.21%
Instruction Change:  + .28%
Line Change:  + .74%

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change:  + 1.18%
Complexity Change:  + 1.35%

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change:  + .07%

org.pentaho.di.trans.steps.singlethreader.SingleThreaderMeta

Instruction Change: -.10%:small_red_triangle_down:

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change: -.18%:small_red_triangle_down:

org.pentaho.di.trans.steps.synchronizeaftermerge.SynchronizeAfterMergeMeta

Instruction Change: -.04%:small_red_triangle_down:

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change: -.53%:small_red_triangle_down:

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.79%:small_red_triangle_down:
Complexity Change: -1.19%:small_red_triangle_down:
Instruction Change: -.07%:small_red_triangle_down:

org.pentaho.di.www.TransformationMap

Branch Change: -9.09%:small_red_triangle_down:
Complexity Change: -6.15%:small_red_triangle_down:
Instruction Change: -10.03%:small_red_triangle_down:
Line Change: -10.00%:small_red_triangle_down:


Create example program: Iterated MBQC

For an introduction to MBQC I actually recommend these lecture notes -> http://pwallden.gr/courseiqc.asp. There are reference in those lecture notes to the original papers on MBQC. Those should give you the background necessary to understand 'flow', which is how correction gates are positioned in the circuit.
For the particular circuit, in principle anything you can do in the circuit model you can do in MBQC, so you could pick your favourite application and proceed that way. I would probably recommend just implementing a small primitive for now. For example towards the end of lecture 14 in the above there is mention of a universal brickwork state. You could try to implement one 'brick' of that. Even implementing a CNOT requires relatively advanced control, so even something like that would be interesting.
A some point you may wish to start thinking about MBQC in the ZX-calculus -> https://youtu.be/0KYkTWdvNmo?si=c8G8aEYSgyG6K45P although I don't think that will be necessary just yet unless you are already familiar with ZX-calculus.


Update outdated troubleshooting.md

Juliaup is now recommended for al (supported) platforms. For scripting, the --compile=min is still valid, though I believe its use is less relevant by now. And it uses an interpreter, "min" means none in effect I believe so -O is redundant with it. I've used both just in case, but I don't recall a difference, at least simplify.

Thanks for the update! The doc was written a long time ago, so it is outdated. And this doc is used in all my packages so I need to update all of them.
I just have 1 question: How do I write Juliaup? Is it "Juliaup" or "juliaup" or "juliaup"?

I think I figured it out, thanks!


Cycle import in File datasets/pascal3d/preprocess.py

Thank you for sharing your amazing work!
I found a cycle import issue in file datasets/pascal3d/preprocess.py. It imports modules from other files that import modules from it. This will result in several ImportErrors. I found that simply restructuring the files would be a quick fix.
Cheers~

Hi @hzykent,
thanks for the interest in our work.
I managed to restructure the files, as you suggested, and now the code should work fine.
Cheers


[Game Request] Sanoba Witch

Game: Sanoba Witch
VNDB: https://vndb.org/r81584
Engine: Kirikiri
Archive file format: .xp3

When I tried to open the archive files in GARbro, I used NoCrypt, Sabbat of the Witch, and Sanoba Witch , but they all appeared as gibberish after opening.

Because GARbro have the game,so I'm not sure what's going wrong

Use krkrdump instead

Thank you for your help, now I can extract files.


Support indexing multi-level nested object arrays.

Add @Nested annotation to support indexing multi-level nested object arrays.

@Pwhxbdk thank you very much for the contribution, we'll review and send some feedback if there are any suggestions for changes. Cheers!

@Pwhxbdk this is great work, I've played with the code and realized that most of the functionality of @Embedded could be absorbed into @Indexed. My goals are for the @Indexed and @Searchable to be the primary annotations to be used with other specific ones will be deprecated at some point (@TagIndexed, @NumericIndexed, etc).
I rebase interactively and forced pushed to your repo :-) And now I'm about to merge/rebase the PR


Fix attention mask type for Flash Attention + CP + THD

Description
Fix qkv_format setting in DotProductAttention.
Use padding mask type for flash attention with THD format in CP unit tests.
Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Infra/Build change
[ ] Code refractor

Checklist:

[x] I have read and followed the contributing guidelines
[x] The functionality is complete
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/te-ci pytorch L1


allow source ns to be overridden

kubectl annotate managedclusteraddon kuadrant-addon "addon.open-cluster-management.io/values"='{"CatalogSource":"openshift-marketplace/community-operators", "CatalogSourceNS":"somens"}' -n local-cluster

/lgtm

We probably want to mention this in https://docs.kuadrant.io/multicluster-gateway-controller/docs/installation/service-protection-installation/, but that shouldn't hold this up.


[5.4] Offload writing .env file to DotEnvWriter package for key:generate

This outsources the logic to write environment files to a separate package dedicated to that responsibility (oohology/DotEnvWriter). This allows the key:generate command to write the APP_KEY to the .env file more reliably, while simplifying the KeyGenerateCommand class.
Currently, the pattern for matching an existing APP_KEY is very strict. If the .env file does not contain an exact match for Laravel's current app key in the expected format, the artisan command will fail to update the .env file. This package can handle multiple variations of the APP_KEY statement including quoted values and whitespace, as well as append a new APP_KEY if one is not present.

I don't really want to adopt an entirely new package for this tiny feature.


I faced some errors when I try to use CookieManager

[x] I have read the Get Started - Installation section
[x] I have read and done the Get Started - Setup Android section
[x] I have read and done the Get Started - Setup iOS section
[x] I have already searched for the same problem

Environment

Technology
Version

Flutter version

Plugin version

Android version

iOS version

Xcode version

Device information:
Description
Expected behavior:
Current behavior:
Steps to reproduce

This
Than that
Then

Images
Stacktrace/Logcat

We have this error in pipeline, somehow related to CookieManager:
Swift Compiler Error (Xcode): Cannot call value of non-function type 'String?'
/Users/runner/hostedtoolcache/Flutter/3.3.4/macos/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_inappwebview-5.5.0+3/ios/Classes/MyCookieManager.swift:259:43
Swift Compiler Error (Xcode): Cannot call value of non-function type 'String?'
/Users/runner/hostedtoolcache/Flutter/3.3.4/macos/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_inappwebview-5.5.0+3/ios/Classes/MyCookieManager.swift:293:43

duplicated #1369, you need to update Xcode version to 14


space key not working on ipad with bluetooth keyboard

IOS 10
with bluetooth keyboard attached (which works fine in other apps).
Tested in Safari & Chrome - same results.
I can login to the wetty, but as soon as i try enter a white space separated command, the whitespace  is not showing up. . The spacebar keyCode seems not to be passed to the server.
Thismakesworkingonipadnotpossible.

this hack fixed it. but this seems not be the proper solution to hack a generated file.
--- a/public/wetty/hterm_all.js
+++ b/public/wetty/hterm_all.js
@@ -5850,6 +5850,19 @@ hterm.Keyboard.prototype.onKeyDown_ = function(e) {
     // the modifier and think it's ok to let the browser handle the keypress.
     // The browser won't know we're trying to ignore the modifiers and might
     // perform some default action.
+
+    // IOS seems not to bubble the keyCode 32 to the keyPress event handler
+    // neither on Safari, Firefox nor Chrome. This results in a non-working
+    // spacebar. Therefore we workaround this bug here.
+    // Note this workaround does not work if you switch to "Desktop version"
+    // on the iPad
+
+    if ( e.keyCode == 32 )  {
+      var iPad = window.navigator.userAgent.match(/iPad/);
+      if (iPad)  {
+        this.terminal.onVTKeystroke(String.fromCharCode(e.keyCode));
+      }
+    }
     return;
   }


NameError: name 'torch' is not defined in python 3.5

Hi there! :smile: So I installed lutorpy python 3.5 and 2.7 using pip3 install --user lutorpy and  pip2 install --user lutorpy respectively, under ~/.local/lib/. It appears the version for Python 3.5 is not working properly.
Python 2.7
$ python2
Python 2.7.12 (default, Jul  1 2016, 15:12:24)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.

>>> import lutorpy as lua
>>> t = torch.DoubleTensor(10,3)
>>> print(t._size())
 10
  3
[torch.LongStorage of size 2]

Python 3.5
~$ python3
Python 3.5.2 (default, Jul  5 2016, 12:43:10)
[GCC 5.4.0 20160609] on linux
Type "help", "copyright", "credits" or "license" for more information.

>>> import lutorpy as lua
>>> t = torch.DoubleTensor(10,3)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
NameError: name 'torch' is not defined
>>>

Hi, thanks for the report, currently, lutorpy is only tested in python2, so please use python2 for now.

:( Is there a timeline on bringing this up to Python3.5? I know PyTorch is Py3-native, so I'll start with that, but the API on this looks really nice. Would like to evaluate both, but maintaining legacy code is a dealbreaker.
Thanks for your efforts, though, because this is really impressive work nonetheless. :)

@cathalgarvey python3 is supported now, please update with pip3.
Notice that, before you can use torch, you need to run:
require('torch')

So you need to do:
>>> import lutorpy as lua
>>> require('torch')
>>> t = torch.DoubleTensor(10,3)
>>> print(t._size())

Wow, fast turn-around! :) Thanks a million for this, I'm looking forward to playing with it.


Basic and medium level question program code of linked list

Description
GSSOC'22 contributor
I want to add solution of linked list question level basic and medium.
please assign me this issue.
Domain
Data Structure
Type of Contribution
Addition
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

Only General Problem Statements are allowed in the repo , please describe your issue or it will be closed .
Linked List is already implemented in the REPO , come up with something new and we will be happy to accept your issue .

/assign


More Documentation

It would be nice to have more documentation on setting up, types of calls, and in general.

Hi Tommy1995,
I am putting an entry into our backlog to do this.  Until the, feel free to ask specific questions on this thread.
Thanks,

Joshua

Tommy1995,
We recently merged some additional docs to the README.rst file.  Can you see if these meet your needs?  If you want an example of usage, you could always look at how the Docker Registry Cleaner works.
Let me know,
- Joshua


GROOVY-8114: Include correct time unit in timeout error message

PR for patch as supplied by Jochen Kemnade. Planned for 2.5+.

+1


Consider porting to Kinetic

No one knows what this would entail right now.
This will also not happen for a long time, until we've at least done a public demo.

I'm closing this issue and opening another one which is just porting to kinetic. The camera driver things that were blocking us are no longer an issue as a few months ago libusb_camera got ported to kinetic.  Being on Indigo has made things harder for new members. Working on porting install  script now.


Changing pricing page and adding request demo button

Description
Goal is to simplify "access to demo" on our website and better show difference between plans on our pricing page.
Pricing Page

Request Demo Button on top menu (links to the intro call form)

Risk

overloading ourselves with non-qualified leads (e.g. pro plans) ?

@Adugre @clmrn @nchinot @ahammour what do you think about the following proposals?
Happy to change wording, etc. This is a V0 to show what's possible.

@thib-martin looks neat!
A few comments:

'restricted spaces' is not self-explanatory IMO
Single Sign On (Okta, Entra ID, Jumpcloud)
'(soon) Salesforce connector' in Enterprise

Aligned with @clmrn , thanks for taking the lead on this.

Thanks for the feedback. Here's an updated view, does this look better to you?
Note: we'll probably iterate on the contact form (moving to typeform or a hubspot form) but 'growth hacking' is not the top priority for now.

@gabhubert @spolu ok with this?

LGTM 👍

What's the wording on a US IP? WE should also sell "US data hosting" in the US.
I don't think we need the "soon" for data hosting given our time to close deals


warning message if kubectl command not found

Fix: #441

Codecov Report

Merging #464 (44568cb) into master (8cc7bd0) will decrease coverage by 0.02%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master     #464      +/-   ##
==========================================
- Coverage   67.40%   67.38%   -0.03%
==========================================
  Files          74       74
  Lines        4915     4921       +6
  Branches      867      868       +1
==========================================
+ Hits         3313     3316       +3
- Misses       1602     1605       +3

Impacted Files
Coverage Δ

src/tkn.ts
66.48% <50.00%> (-0.14%)
:arrow_down:

Continue to review full report at Codecov.

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


Enhance BatchSequenceGenerator with support for IdGeneratorType

Hibernate 6.5 has deprecated @GenericGenerator see https://discourse.hibernate.org/t/deprecation-of-genericgenerator-in-hibernate-6-5/9482. Instead the @IdGeneratorType meta-annotation should be used https://docs.jboss.org/hibernate/orm/current/userguide/html_single/Hibernate_User_Guide.html#identifiers-generators-IdGeneratorType
The result is a lot less verbose, what previously was
  @Id
  @GenericGenerator(
          name = "parent_id_generator",
          strategy = "com.github.marschall.hibernate.batchsequencegenerator.BatchSequenceGenerator",
          parameters = {
              @Parameter(name = SEQUENCE_PARAM, value = "SEQ_PARENT_ID"),
              @Parameter(name = FETCH_SIZE_PARAM, value = "50")
          })
  @GeneratedValue(generator = "parent_id_generator")
  private Long parentId;

Is now simply
  @Id
  @BatchSequence(name = "SEQ_PARENT_ID", fetchSize = 50)
  private Long parentId;

Here is an example what the changes may look like  https://github.com/marschall/hibernate-batch-sequence-generator/commit/d88975a0be56703e4431d2bb829b47be6b409692
I'm not sure this can be retrofitted into the 6.3 module or a new 6.5 module is needed.

Fixed.


Checkbox Sort

When sorting the checkbox column it doesn't sort correctly.

dbfolder 2.3.6
obsidian 0.16.2

The 2.3.6 move checkbox from 0 - 1 to true-false. , so maybe is mixed right now in your ddbb.
I will add an option to choose how to persist it

https://github.com/RafaelGB/obsidian-db-folder/issues/158

solucionado en al 2.4.0. Lo cierro como wont fix


Change server_info endpoint error message to be verbose

Add handling for case in which API version is not supported by the Tableau server. While trying to explore this library, I ran into an error when trying to call server.server_info_get()
>>> server.version = "2.5"
>>> server.server_info.get()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/jojacob/development/dasre-tableau-insights_trunk/build/dasre-tableau-insights/venv/lib/python3.7/site-packages/tableauserverclient/server/endpoint/endpoint.py", line 108, in wrapper
    return func(self, *args, **kwargs)
  File "/Users/jojacob/development/dasre-tableau-insights_trunk/build/dasre-tableau-insights/venv/lib/python3.7/site-packages/tableauserverclient/server/endpoint/server_info_endpoint.py", line 25, in get
    server_info = ServerInfoItem.from_response(server_response.content, self.parent_srv.namespace)
UnboundLocalError: local variable 'server_response' referenced before assignment

This stems from the code here:
https://github.com/jacobj10/server-client-python/blob/e4ec849b19f02f016f90e641756d9bd81d23d026/tableauserverclient/server/endpoint/server_info_endpoint.py#L17-L21
which can catch an error without setting the variable, leading to a confusing and non descriptive UnboundLocalError. Right now I just added an extra check for my specific error (version unsupported), but it my be worth just raising whatever error comes from the response.
This is my first time contributing so please let me know if you have any questions, comments, or concerns.

I haven't reviewed in detail yet, but I think the problem is that we handle this a level above at https://github.com/jacobj10/server-client-python/blob/e4ec849b19f02f016f90e641756d9bd81d23d026/tableauserverclient/server/server.py#L76
In this case, you're manually setting the API version to 2.5, avoiding the 'use legacy version check' fallback. I agree we should handle this, and I'll take a look at your PR!

Ahh I see, I think I was manually updating the version due to confusion when reading the documentation. The correct way is to use the class methods?

Hey!
Yeah, the recommendation is to just use the methods like this:
auth = TSC.TableauAuth(USERNAME, PASSWORD)
server = TSC.Server("http://SERVER", use_server_version=True)

# OR
server = TSC.Server("http://SERVER")
server.use_server_version()

... I'm not sure how that test is failing given you didn't touch sorting/filtering.
Can you make sure your fork is updated tot he latest development branch?

Weird, seems like the fork is up to date with the origin dev branch? I tried kicking off another CI run but to no avail :(

The test fails locally on my master branch as well

The only explanation I have is some update in a library we depend on changed the encoding behavior?
Feel free to take a look at that and include it in your PR, or ignore it and I won't hold it against you :)

Ack I'll try to include in this PR, I'll let you know if I can't figure it out.

Yep that did it. More of a single issue fix. It might be worth seeing if there are any utils that the request module uses to URI decode/encode the response and use that accordingly with all relevant assertions, but that seems like it should be addressed in its own PR if at all. LGTM!

Thanks, I'll take it!
Can you confirm you've sent in the CLA? (I don't have an easy way to check on my end) I'll wait a day and then merge this.

Sent :)

CLA approved.


Rectangular core shape

If we made the core a rectangle 10:8 then we'd always get full lines?

yeah, guess so since the standard core sizes tend to be 8x10^n win!


Ideas for Improving Code Structure and Collaboration

Hello everyone,
It is exciting to see that the project is now on GitHub!
I wanted to share some thoughts on possible directions to improve both the codebase and future collaboration.
While there are some broader, long-term goals I personally find interesting, I believe it would be helpful to figure out some immediate steps that would help with readability, testability, and ease of contribution. Below are some high-level ideas for the long term, followed by more short-term actionable steps.
Long-Term Ideas:
1. Disentangle Model Code from Testing Code
Problem: The current implementation intertwines model handling and benchmarking logic, making it difficult to test and manage benchmarks.
Proposal: Separate the model interaction code (e.g., API, model loading) from the benchmarking logic.
Benefits: This will streamline the process of testing benchmarks, making development faster and more understandable, without the complexity of machine learning-specific code.
2. Introduce Prompt Templates
Problem: Generative models rely on specific instruction layouts, affecting the benchmark results. This variety makes it hard to ensure consistency across different models. On top of that, popular benchmarks have a specific structure of instruction prompts they use to do the actual evaluation.
Proposal: Create prompt templates to make those axes of variation explicit. I think this would be even worth some amount of code/template duplication such that for some model/benchmark combinations it is clear what's happening without combining different templates and puzzling together the final prompt given to the model.
Benefits: This would improve the consistency, transparency, and comparability of benchmarks across various models, and also benchmarks.
3. Improve Result Transparency and Aggregation Workflow
Problem: Results are currently logged as JSON files in a bottom-up workflow, which makes it somewhat difficult to trace metrics back to specific inputs and adjust benchmarks iteratively.
Proposal: Introduce better traceability in the logging system and explore workflows inspired by tools like ZAP, a proxy tool used for testing web applications.  Usage of such user interfaces would support manual and automated testing, providing flexible result management and improved transparency.
Benefits: Enables easier aggregation of results, allowing recalculation and comparison across different benchmark subsets, providing better insights and adaptability.
Short-Term Actionable Ideas:
I could see it as beneficial to start with some basic steps to lay the foundation for broader changes and  ease of addition of other benchmarks. For example:

Create Tests for the Benchmarking Code: This could provide a reliable starting point for further changes and improvements across the codebase.
Update the README: It would be helpful to outline these first steps and the initial direction in the top-level README.md to arrive at a clearer picture of what to do next and be aligned in those steps.

Next Steps:
I would love to hear your thoughts on these ideas and suggestions and also what you think should be the next steps. If others are on board, it could be useful to start add tests for the benchmark code and updating the documentation to reflect the current state and next steps.
What do you think?
I am looking forward to contributing!
Best, Philipp

Hi Philipp,
thanks for the nice suggestions. We are looking into some of these already and I'll post a larger updated a bit later with more details.
best,
Pavol


I get this error in some images

I get this error when i call human.detect, not all images but some, i couldnt find anything to solve these, it is probably about conversition setting of image, conversiyon function and image are below; also logged final tensor data which is the parameter of human.detect(tensor)
Function;
export async function urlImageHuman(url: string): Promise<undefined | any> {
    const fetch = require("node-fetch");
    const res = await fetch(url);
    if (res && res.ok) {
        let buffer = await res.buffer();
        if (!buffer) {
            return undefined;
        }
        let decoded = human.tf.node.decodeImage(buffer);
        let casted = decoded.toFloat();
        let tensor = casted.expandDims(0);
        decoded.dispose();
        casted.dispose();
        return tensor;
    }
    return undefined;
}

Image;

Tensor :
{"kept":false,"isDisposedInternal":false,"shape":[1,480,640,4],"dtype":"float32","size":1228800,"strides":[1228800,2560,4],"dataId":{},"id":10762,"rankType":"4","scopeId":9028}
Error;

(node:18860) UnhandledPromiseRejectionWarning: Error: Human: Input tensor shape must be [1, height, width, 3] and instead was 1,480,640,4
at j1 (C:\Development\hemsor\functions\node_modules@vladmandic\human\dist\human.node.js:15:822)
at C:\Development\hemsor\functions\node_modules@vladmandic\human\dist\human.node.js:734:8060
at processTicksAndRejections (internal/process/task_queues.js:97:5)

your image is RGBA (it has alpha channel) and tensorflow requires pure RGB input
try something like this:
(sorry for any typos, i wrote it from head without testing)
const tensor = tf.tidy(() => {
  let decoded = human.tf.node.decodeImage(buffer);
  let casted = decoded.toFloat();
  let res;
  if (tensor.shape[3] !== 3) {
    const channels = tf.split(tensor, 4, 2); // split rgba to channels
    const rgb = tf.stack([channels[0], channels[1], channels[2]], 2); // stack channels back to rgb and ignore alpha
    res = tf.reshape(rgb, [1, decoded.shape[0], decoded.shape[1], 3]); // move extra dim from the end of tensor and use it as batch number instead
  } else {
    res = casted.expandDims(0); // just add batch number
  }
  return res;
})

@okeoke85 any updates on this issue?

thanks to your help seems to be working


differences in look between fontawesome and fontello (e.g. handle of trash is squashed).

I switched from fontawesome 4.7.0 to using the same icons exported by fontello from fontawesome 4.7.0.
Here is the pure fontawesome version followed by the fontello version for comparison:

You can see that the handle of the trash can has been squashed (but the rest of the trash bin is identical). The entire floppy disk has been vertically squashed. The globe and the "x" also seem to have gotten shifted up and to the right a bit. The only icon that looks the same in both is the chevron.

When I turned off hinting they match, sorry for the noise.

Sorry I was wrong, turning off hinting stops the disk getting squashed and the shifting of the x, but the handle of the bin is still squashed and the globe is shifted up and to the right a bit.

Go to advanced settings and play with font accent/decent. Fontello defaults can be different.

@puzrin I played with them a bunch, I set them to what I believe font-awesome uses. Could that explain the handle of the bin getting squashed without the rest of the bin being affected?

Yes. Process of rendering high resolution images to low resolution screen is not trivial. Sometime renderer can change details to improve contrast. And it's difficult to predict where it happens (can vary in OS-es and browsers). The only thing you can do - try to reproduce similar initial conditions as in original - metrics, hinting.

Is this still a problem?

@puzrin Sorry I'm not sure, I stopped using fontello slightly after hitting this problem.


Downloads Links aren't being updated again

The download links are pointing to an old binary again.  Please consider redirecting to the official download source instead.

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

ID: 17880b9d-6b98-3194-cc33-fe8067ce241e
Version Independent ID: 643bcf2f-507b-e449-73db-923d8a368ad9
Content: Teams for Virtualized Desktop Infrastructure - Microsoft Teams
Content Source: Teams/teams-for-vdi.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@ekkonomix Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hi @ekkonomix, thank you for your feedback and help us to improve docs.microsoft.com.
@LanaChin @sarossma could you kindly help here? Thanks!

Please update the links of the msi download. Version is more then 2 months old!!!

There is a more recent download version on https://docs.microsoft.com/en-us/microsoftteams/msi-deployment#vdi-installation

Hi @ekkonomix. Thanks for your feedback. We've updated the MSI links in the article.


[Feat] Support pg_up and pg_down key events in preview windows

Hello,
I was wondering if it would be possible to add Pg_Up and Pg_Down support for the preview window buffer which shows up when searching for files and/or live grepping.
For reference, here's my plugin configuration:
return {
    "jake-stewart/jfind.nvim",
    branch = "2.0",
    config = function()
        require("jfind").setup({
            exclude = {
                ".git",
                ".idea",
                ".vscode",
                ".sass-cache",
                ".class",
                "__pycache__",
                "node_modules",
                "target",
                "build",
                "tmp",
                "assets",
                "dist",
                "public",
                "*.iml",
                "*.meta",
            },
            windowBorder = true,
            tmux = true,
            maxWidth = 180,
            maxHeight = 34,
        })
    end,
    keys = {
        {
            "<C-P>",
            ":lua require('jfind').findFile({preview = true, previewPosition = 'right'})<CR>",
            desc = "find files",
            silent = true,
        },
        {
            "<leader>,",
            ":lua require('jfind').liveGrep({ caseSensitivity = 'smart', previewPosition = 'right' })<CR>",
            desc = "live grep with ,,",
            silent = true,
        },
    },
}

I can navigate through the list using up and down arrows but cannot do Pg_Up and Pg_Down.

i'll add these keys as defaults to jfind in a bit

i added the keys to main branch of jfind
recompile for these new changes
it scrolls a full page. you can also use alt-d and alt-u to scroll only 10 lines.


Update project files

Thank you for contributing! Please check the following things before submiting your PR:
Required:

[x] I have read and followed the contributing guidelines

If necessary:

[ ] I have updated the README and documentation.
[ ] I have updated the ChangeLog with the changes I have made.

Update project files with changes to labels

Closes #

Note: we will close your PR without comment if you do not check the required boxes above and provide ALL requested information.

Are the labels updated?


[Firebase Core] FutureBuilder With Firebase.initializedApp() Not Resolved

I'm trying to initialize Firebase with FutureBuilder as the documentation recommends, but I'm having problems. With a simple application placing the FutureBuilder in a Widget or even in the main void, the promise is never resolved and remains pending.
firebase_core: "0.5.0-dev.2"
firebase_analytics: "^6.0.0-dev.1"
firebase_auth: "0.17.0-dev.1"

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, v1.17.5, on Mac OS X 10.15.6 19G73, locale es-419)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
[✓] Android Studio (version 4.0)
[✓] VS Code (version 1.47.3)
[✓] Connected device (1 available)

• No issues found!

### Code Example..
void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter',
      debugShowCheckedModeBanner: false,
      home: HomeScreen(),
    );
  }
}

class HomeScreen extends StatefulWidget {
  @override
  _HomeScreenState createState() => _HomeScreenState();
}

class _HomeScreenState extends State<HomeScreen> {
  @override
  Widget build(BuildContext context) {
    return FutureBuilder<FirebaseApp>(
      future: Firebase.initializeApp(),
      builder: (BuildContext context, AsyncSnapshot<FirebaseApp> snapshot) {
        if (snapshot.hasData) {
          // this never happens the promise remains unresolved.
          return CatalogScreen();
        } else {
          return Container(
            color: Colors.white,
            child: CircularProgressIndicator(),
          );
        }
      },
    ),
  );
}

Another problem that I notice is that when the application is initialized for the first time, the animation of the CircularProgressIndicator widget never occurs, it remains static.

Hi @andrefedev
Can you please explain why you checking data in initializedApp. Did you mean to Firetore to heck data for catalog screen?

There could be an error being thrown but you don't check for that, try something like:
class FlutterFireInit extends StatelessWidget {
  final Future<FirebaseApp> _init = Firebase.initializeApp();

  @override
  Widget build(BuildContext context) {
    return FutureBuilder(
      future: _init,
      builder: (context, snapshot) {
        // Check for errors
        if (snapshot.hasError) {
          return Center(
              child: Text(
                  "Something went wrong during initalization: ${snapshot.error}"));
        }

        // Once complete, show your application
        if (snapshot.connectionState == ConnectionState.done) {
          return CatalogScreen();
        }

        // Otherwise, show something whilst waiting for initialization to complete
        return Center(child: CircularProgressIndicator());
      },
    );
  }
}


improve testing

In GitLab by @NicoleSchwartz on Apr 6, 2020, 22:08

In GitLab by @NicoleSchwartz on Apr 6, 2020, 22:08
added to epic gitlab-org&2769

In GitLab by @NicoleSchwartz on Apr 6, 2020, 22:08
marked this issue as related to #516

In GitLab by @NicoleSchwartz on Apr 6, 2020, 22:08
closed

In GitLab by @NicoleSchwartz on Apr 6, 2020, 22:09
removed from epic gitlab-org&2769


Update Azure WebSubHub improvements proposal with the latest requirements

Description:

$subject

Following details should be included:

How to embed event-profile information (IdP, SCIM etc) into the WebSub topic ?
How are we going to revise the WebSub Hub deployment strategy within the Asgardeo system based on the latest topic mapping requirements ?

Proposal doc can be found here [1]
[1] -  https://docs.google.com/document/d/1Oz7yb8j3CahtCbv4_-lJuqpaKcIGuJHCfNthXJfzx_w/edit?usp=sharing

The proposal doc has been updated with the relevant information and hence marking the task as complete.


Support for configuring default and maximum command timeouts

I would ideally like to configure these settings via terraform:

I know there is not currently an API available for this either but it would be nice to have the API -> terraform capability.
Thanks!

Hey @dgollahon!
Tanks for the request - and hope you had a great New Year to welcome in 2024!
As of now, the provider doesn't have that functionality in place as you've seen as the relevant APIs dont exist yet for setting the upper limit timeouts (UI only at present). Shall will keep you in the loop on if/when that can be added for the provider to also maintain in a stateful manner.

I've raised this internally to be supported in the API(s), we can't move forward with this without API support. Once that support is complete then we can card this up.


Add a one-time in-context tooltip for people looking for the nutrition table

Problem

We don't show the nutrition table by default
We have an option in settings to reenable it

Proposed solution

[ ] Add a one-time in-context tooltip for people looking for the nutrition table
[ ] It should offer to reenable nutrition inline (in a modal ?)

Mockups
Part of

#561

I would also add a "show nutrition data" button.
I don't think we're supposed to perform actions when clicking on a tooltip - beyond closing the tooltip.

My wording was bad, basically a dismissible way to tweak the settings to show the ingredients. Eg it should not clutter the product page if the user is not interested


Cleanup

Resolves #1 (spine-lazy EphemeralQueue is not included because the only affected operation would be map and that function is already rather useless).
Documentation for this version is available at Github Pages.
A big tacky rewrite that reorganizes everything in a sensible manner:

All data structure modules are now called Data.*.

All data structures now expose internals in *.Unsafe modules and debug utilities in *.Debug.

EphemeralQueue is now called Steque (stack-ended queue). It uses strict lists underneath for performance.

Queue uses a strict list as its back for performance.

Queues no longer guarantee that Foldable's "left" and "right" match "front" and "back". All folds are now written to run in O(n).

A lot of functions were renamed to match containers, more specifically Seq, as it itself is a (double-ended) queue. F and B are used as suffixes in functions that refer to front and back of the queue respectively to match the Foldable change.

Removed Semigroup and Monoid instances because these queues do not have fast appends.

Removed Queue's Functor and Traversable as these functions would either cause the schedule to diverge or require forcing the queue, neither approach makes sense.

Moved Queue's (<|) to the *.Unsafe module because it makes the schedule diverge.

Steque's reverse lives in the *.Unsafe module because it breaks asymptotics.

Additionally I implemented Okasaki's list-based real-time Deque, however there's no good reason to include it in the package per benchmarks, I expect it to be removed before merging.
All code has been simplified to ease GHC's job, base range is now more liberal, all the datatype explanations have been moved to README and the relevant modules. Everything is documented and tested.

Benchmarks
Backport of the benchmarks to queues-1.0.0 used here can be found at https://github.com/BurningWitness/queues/commit/f6b05c82d482b194db36ae257e5ea5a0d7d6bd25. Benchmarks for new Queue and Steque were run with Deque commented out so that they're as close to backports as possible.
Described as:

insert: insert <size> elements sequentially at the back of an empty queue;
reduce: empty a queue of <size> elements by removing elements from the front;
wobble: over a base <size>, add one element at the back and remove one element at the front 100000 times.

 New Queue is on par with the old one.
Deque is much slower than Seq, as unlike Seq it has to constantly reevaluate the backs of both lists. Since any tree-based deque gets to skip that process by encoding element order into the tree, list-based solutions are inferior.

Hi, I'm sorry but I'm not willing to merge a "big tacky rewrite" that contains superficial changes to names of modules, functions, and types. Would you still like to pursue some subset of these changes?

I don't consider the changes superficial, otherwise I'd change only the function names, not definitions. Though I can't deny that my request is rather rude, just that the ecosystem is so small I either write what I need, or nothing ever gets done.
I already have the [as I believe] superior version of the package to myself, so I don't see a point in backporting it one slice at a time. You're free to incorporate any of the aforementioned changes yourself if you feel like it.

Um, ok!


Set search conditions to query parameters

This great OSS always helps me.
I use this every day, and I thought it would be more convenient if it was reflected in the query parameters when changing paging or limits.
This makes it easier to share search results with others.
I tested it in my local environment, but I don't know all the features so I may be missing something.
I'm not a front-end expert, so please let me know if you have any concerns.
only paging: /?start=0&limit=100

with search condition: /search?q=python+99&start=25&limit=25

tag search: /search?q=tag%3A10&start=50&limit=50

This is awesome @yktakaha4, thank you! There are a few bugs I have identified which I hope you can maybe fix in your PR:

The idea of a shareable URL is great, except it doesn't work for the start or limit (they just default to 0 and 50 regardless what the URL says). Load Mailpit, go to a page 2, 3 etc (eg: start > 50) and refresh the page (the URL stays the same of course, eg: http://localhost:8025/?start=250&limit=50 but the page actually loads start=0 & limit=50.
Links to tags in either the side nav or the small tags under messages in the overview should always go back to start 0 (they currently use whatever the start currently is). I think that maybe toTagUrl() requires an optional second boolean parameter which sets pagination.start = 0 before the params....
The "inbox" link (that calls reloadInbox()) should include a this.$router.push('/') before this.loadMessages() to "reset" the URL
While viewing any page > 0  - as new messages are received Mailpit, offsets the "current view" in relation to the new messages so that the current view doesn't get affected. You can see the start changing at the top right but the URL remains the same:

Ideally the URL needs to change too (dynamically)

You are are able to take a look at those issues for then then great (I suspect the last point I made will be "too hard" so just leave that), but if the rest are too complicated for you (I think you've done a great job so far) then let me know and I'll do some work on it (I'm really busy at the moment, so I'm not sure when I will get to this). Just let me know please so we don't end up both doing the same thing?

@axllent Thank you for your kind review and sorry for not checking enough.
I fixed it in the following commit.
https://github.com/axllent/mailpit/pull/303/commits/807c058a6f6d594e4c530dde82737a0a60e59152
When accessing http://localhost:8025/?start=250&limit=50

Initialize start in the tag URL.

Initialize URL when Inbox is clicked.

I also fixed the last issue, but I am not confident.
https://github.com/axllent/mailpit/pull/303/commits/7e039df342643cf847088681088cd5fbf12ed81e
https://github.com/axllent/mailpit/assets/20282867/64683e8b-381a-4ae4-8372-e1db5ec3dc45

It's working now pretty nicely for me, well done and thank you @yktakaha4! I will merge this into a new feature branch on Mailpit, and I will make a few minor changes (for instance I do not think one needs to convert the start & limit to a string every time).
Thank you very much for all your hard work :heart:

@yktakaha4 I have made a number of small changes to the feature/query-parameters branch which I would really appreciate your feedback (and testing) on - if possible:

The start URL parameter is only set > 0 (shorter URLs)
The limit URL parameter is only set if != 50 (the default)
The URL is now only updated at a maximum of 2x per second when receiving many messages (and having a start > 0) - I was getting a browser error of Too many calls to Location or History APIs within a short timeframe.

The only issue I have found so far (which I don't think can be resolved without a lot of complicated work) is the back browser button doesn't trigger a reload of messages (eg: you click next page, next page, next page, and then click the back browser button). This did not work before your changes either, so it's not new, but it is the only "issue" I can find now. Apart from that, I think this implementation is working very well.

I tested it locally and worked fine, thanks.

the back browser button doesn't trigger a reload of messages

As far as I know, we can suppress history generation by using $router.replace instead of $router.push.
Although it is not a fundamental solution, I think it will reduce user confusion caused by the browser's back button.
https://router.vuejs.org/guide/essentials/navigation.html#Replace-current-location

Thanks for the suggestion, but I don't think it actually resolves anything or prevents any confusion (in this case). The back browser button still loads a URL which does not change the message listing. Don't worry, it's not an issue unless someone makes it an issue, so I think I'll do a bit more testing and get this branch merged into develop. I just released a new version this weekend so I'll probably wait a few days before releasing another version (OS package maintainers sometimes get grumpy with me when I release too often :laughing: ) :+1:
Great work, and thank you for your contribution!

@yktakaha4 I managed to solve the browser back/forward navigation, shifting around some of the logic plus a few other changes :partying_face: I haven't found any other issues in my testing. Thanks again for your hard work, I really appreciate it!
This functionality has now been released in v1.18.5.


Trim field names

Most of the time we do not want to have whitespace at the start or end of a field
Also prevent characters like tab / LF
How much of this does Django handle?

Looks like this is all handled automagically! Sweet


SoftConstraint subclass of Constraint

Application
Right now all the constraints are soft constraints when converted to BQMs. It would be good though to formalize that and allow users to specify the strength of their constraints

That's right! It would allow for a high-level way of setting truly soft constraints.


Move the dataframe operator together with others

Codecov Report

Merging #396 (564a30e) into main (b555f16) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #396      +/-   ##
==========================================
- Coverage   90.46%   90.45%   -0.01%
==========================================
  Files          54       53       -1
  Lines        1793     1792       -1
  Branches      280      280
==========================================
- Hits         1622     1621       -1
  Misses        131      131
  Partials       40       40

Impacted Files
Coverage Δ

src/astro/__init__.py
100.00% <100.00%> (ø)

src/astro/sql/__init__.py
100.00% <100.00%> (ø)

src/astro/sql/operators/load_file.py
96.92% <100.00%> (ø)

Continue to review full report at Codecov.

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


Allow non device type template parameter for output views in BVH::query()

Fix #270
This will produce an example of bad error message as requested in https://github.com/arborx/ArborX/issues/269#issuecomment-615190740

Yeah, I thought a way to fix this would be through templating on view types. The unfortunate side effect of this is that the view signatures are no longer easily distinguished (for example, value types).

Well I was not necessarily planning on opening that PR yet because I am necessarily convinced whether I want to keep the Details::BoundingingVolumeHierarchyImpl:: namespace and what is the better way to template for the bvh and output views arguments in the queryDispatch() but I might as well push the current workaround I have

By the way I fail to see how this would be qualified as a bug.


Adding authorization rules

This pull request has been created against below mentioned new implementations into the project:

[ ]
[x]  Installed CanCanCan gem.
[x]  Generated a migration to add a role column to the users table.
[x]  Ran the migration to apply the changes to the database.
[x]  Modified the user model to include the :cancan flag.
[x]  Implemented authorization rules for post deletion based on user roles.
[x]  Implemented authorization rules for comment deletion based on user roles.
[x]  Updated views to display "Delete Post" and "Delete comment" conditionally based on user authorization.


RaisedButton is deprecated

'RaisedButton' is deprecated and shouldn't be used. Use ElevatedButton instead. See the migration guide in flutter.dev/go/material-button-migration-guide). This feature was deprecated after v1.25.0-8.1.pre..
Try replacing the use of the deprecated member with the replacement.

Alternative buttons such as TextButton, ElevatedButton, and OutlinedButton are not quite easy like before. We can still use those legacy buttons using the legacy_buttons package
https://pub.dev/packages/legacy_buttons/

why cant we use raised button no more p.s i am new to flutterr


consider programmatic creation of parms info

In FC1, for each new year/county, someone has to manually look at the corresponding information on the national archives website and manually create a specifically-formatted .csv file with the information for each piece in that year/county, then convert the completed .csv into a PARMS.DAT file to be used by FreeCen. The process is time-consuming and it is easy to make mistakes.
It may be possible to crawl the national archives site once, automatically parsing the necessary information to be ingested once into freecen2 so that Brenda and her coords do not have to continue manually entering the information for each new year/county/piece.
We would need to check the archives' robots.txt file to see if they allow indexing of their site, and also look at the format of their pages to see if directly parsing the data is practical.

Issue with amending rather than creating a new file - specifics for this error sent over email. Awaiting Doug's input.

Just to clarify. We have 2 Issues with PARMS in FreeCEN 2.

The issue is with creating a new PARMS file using FreeCEN 2. (Waiting
for Doug to return)

Trying to delete a specific PIECE number in a current PARMS file.
We can amend details within a PIECE number, but cannot delete a PIECE
number.
These mostly relate to the Records that were lost over 100 years ago before
they reached the National Archives.
Doug suggested we could possible mark the number as "Data lost" instead.
Therefore the system will not need to account for it when all the other
PIECES for that Census Year x County are complete and will then show as
100% done.

We can however still use the FreeCEN 1 current system for the time being.
Brenda
On Mon, 6 Feb 2017 at 15:49, Rich Pomfret<EMAIL_ADDRESS>wrote:

Issue with amending rather than creating a new file - specifics for this
error sent over email. Awaiting Doug's input.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/FreeUKGen/FreeCENMigration/issues/195#issuecomment-277722732,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANC91tJ89G-7W46c4hKksb5fDfcjxvg_ks5rZ0D0gaJpZM4LwAop
.

See email to Richard Light 10/3/2017 for suggestion that we may already have this data

Update.
Richard has done all the PARMS using the NA as Doug described.
I am in the process of editing them as one column has only 21 digits to use
so some places have to be reduced to fit.
There should be some ready to send to Lemon hopefully in the next few weeks.
There are quite a few of them.
In fact Richard did everyone for all the Census Years including 1881 that
are not as yet on the database.
The problem with London/Middlesex/Kent etc  re earlier  Census I will leave
till the last.
On Tue, 16 Oct 2018 at 10:52, PatReynolds<EMAIL_ADDRESS>wrote:

Assigned #195 https://github.com/FreeUKGen/FreeCENMigration/issues/195
to @FreecenBren https://github.com/FreecenBren.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/FreeUKGen/FreeCENMigration/issues/195#event-1906326287,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANC91sQsNTv6LFnbCssNslIJhn9O-A4lks5ulaxmgaJpZM4LwAop
.

This only got the PARMS files into FC1 (and Brenda still converting from CSV to DAT and giving to Lemon for uploading) - need them also in FC2

The (unedited) files need adding to FC2.

Will the edited field need bulk converstion to the actual place names in order to be properly displayed / used on FC2, link with Pelagios, etc.?

@Vino-S is currently working on the upload of theparms file to CEN2

Upload no longer required and we are evolving towards a fixed parms/piece structure based on TNA. Members will have the ability to access diece and civil parish names on line. Will there be a bneed for a download?


Go To Definition and Breakpoints

vscode version: 1.13.0
I am currently working in a PHP environment on Windows 7.
When I use any action to "Go To Definition" the editor now opens up a new tab even when the definition is within the same file. In this duplicated file that has navigated to the definition, I am not able to add a breakpoint for debugging. The only way to put a breakpoint in place is to go back to the original tab for the file, and use a different approach to navigation to the line I need to get to.

Are you using any extensions for php?

Can you file this on the repo for the PHP extension that you're using?

I am using the extension PHP Debug, however, I think there is still an issue here with VSCode. Perhaps I should ignore the breakpoint part of this issue for a moment.
My understanding of the behaviors are as follows:

File A exists in the Open Editors
File A is triggered to be opened again through some action.
A check is made to confirm which side of the editors is currently active
A check for File A in the editors is made
4.1.1. A tab for File A is found in the active side
4.1.2 File A's tab becomes the active file --OR--
4.2.1  A tab for File A is not found in the active side
4.2.2  File A is opened in the active side

If this understanding is correct, using "Go To Definition" should not create a new tab in the same side of the editor when the definition is found within the same file.
Coming back to the breakpoint issue. When File A is open on two sides of the editor, I am able to put breakpoints in both files. The issue only occurs when "Go To Definition" creates a duplicate of File A on the same side of the editor. To the best of my ability, I have not been able to find another way to duplicate File A in this manor. This creation of the duplicate is what appears to go against the normal behavior. I also believe this a new behavior introduced in the latest version of VSCode.
If I have made any mistakes in my analysis and you still feel the extension should be handle this behavior, I would be glad open an issue in that repo instead.
Thank you for your time!

It sounds like the extension is probably returning the wrong path or something for "go to definition", since this doesn't happen with typescript or anything else, so I'd send it to the extension for investigation.


 support

Forgive me if this is covered, but I've searched the documentation and the issues and see no reference to this.
Is <foreignObject> supported or on the roadmap?

What's the use case of a foreignObject?  Does it work if wrapping a RN  as foreignObject?

Handling rich-text in SVG is really difficult, so some people embed HTML elements in foreignobject to be able to represent richly-formatted text in a hassle-free way.
As for the second question, I wouldn't know, I'm a RN neophyte (though I do have some experience with react) looking into this particular use-case atm.

You can use a WebView instead, almost the same functionality/end-result. SVG 2 supports multi line and shape-inside line-breaking. So when the layout algorithm for that is properly implemented, it should be much better. I've already started it, and just a few things to do left and then it can be integrated: https://github.com/msand/react-native-svg/blob/6.0.0/android/src/main/java/com/horcrux/svg/TextLayoutAlgorithm.java

Otherwise, just position view and svg elements absolutely.

does it have support now?

I would recommend splitting your svg into a foreground and background layer, and place a native view with what ever you need in between. This will be more performant (reuses the resulting bitmaps of the svgs) than using the svg rendering logic to make bitmaps of other native components (rerendering all of it on any change in the foreignobjects) + handling hit testing and gestures etc.
Or what would be your use case?

I realized that clipping and masking of native content might be useful, and once we have filter support, that could also be used for all kinds of purposes. Reopening this.

12.5. The ‘foreignObject’ element
‘foreignObject’
Categories:
Graphics element, renderable element, structurally external element
Content model:
Any elements or character data.
Attributes:

aria attributes — ‘aria-activedescendant’, ‘aria-atomic’, ‘aria-autocomplete’, ‘aria-busy’, ‘aria-checked’, ‘aria-colcount’, ‘aria-colindex’, ‘aria-colspan’, ‘aria-controls’, ‘aria-current’, ‘aria-describedby’, ‘aria-details’, ‘aria-disabled’, ‘aria-dropeffect’, ‘aria-errormessage’, ‘aria-expanded’, ‘aria-flowto’, ‘aria-grabbed’, ‘aria-haspopup’, ‘aria-hidden’, ‘aria-invalid’, ‘aria-keyshortcuts’, ‘aria-label’, ‘aria-labelledby’, ‘aria-level’, ‘aria-live’, ‘aria-modal’, ‘aria-multiline’, ‘aria-multiselectable’, ‘aria-orientation’, ‘aria-owns’, ‘aria-placeholder’, ‘aria-posinset’, ‘aria-pressed’, ‘aria-readonly’, ‘aria-relevant’, ‘aria-required’, ‘aria-roledescription’, ‘aria-rowcount’, ‘aria-rowindex’, ‘aria-rowspan’, ‘aria-selected’, ‘aria-setsize’, ‘aria-sort’, ‘aria-valuemax’, ‘aria-valuemin’, ‘aria-valuenow’, ‘aria-valuetext’, ‘role’
core attributes — ‘id’, ‘tabindex’, ‘lang’, ‘xml:space’, ‘class’, ‘style’
conditional processing attributes — ‘requiredExtensions’, ‘systemLanguage’
global event attributes — ‘oncancel’, ‘oncanplay’, ‘oncanplaythrough’, ‘onchange’, ‘onclick’, ‘onclose’, ‘oncuechange’, ‘ondblclick’, ‘ondrag’, ‘ondragend’, ‘ondragenter’, ‘ondragexit’, ‘ondragleave’, ‘ondragover’, ‘ondragstart’, ‘ondrop’, ‘ondurationchange’, ‘onemptied’, ‘onended’, ‘onerror’, ‘onfocus’, ‘oninput’, ‘oninvalid’, ‘onkeydown’, ‘onkeypress’, ‘onkeyup’, ‘onload’, ‘onloadeddata’, ‘onloadedmetadata’, ‘onloadstart’, ‘onmousedown’, ‘onmouseenter’, ‘onmouseleave’, ‘onmousemove’, ‘onmouseout’, ‘onmouseover’, ‘onmouseup’, ‘onmousewheel’, ‘onpause’, ‘onplay’, ‘onplaying’, ‘onprogress’, ‘onratechange’, ‘onreset’, ‘onresize’, ‘onscroll’, ‘onseeked’, ‘onseeking’, ‘onselect’, ‘onshow’, ‘onstalled’, ‘onsubmit’, ‘onsuspend’, ‘ontimeupdate’, ‘ontoggle’, ‘onvolumechange’, ‘onwaiting’
document element event attributes — ‘oncopy’, ‘oncut’, ‘onpaste’
graphical event attributes — ‘onfocusin’, ‘onfocusout’
presentation attributes —

Geometry properties:

x
y
width
height

DOM Interfaces:

SVGForeignObjectElement

SVG is designed to be compatible with other XML languages for describing and rendering other types of content. The ‘foreignObject’ element allows for inclusion of elements in a non-SVG namespace which is rendered within a region of the SVG graphic using other user agent processes. The included foreign graphical content is subject to SVG transformations, filters, clipping, masking and compositing. Examples include inserting a MathML expression into an SVG drawing [MathML3], or adding a block of complex CSS-formatted HTML text or form inputs.
The HTML parser treats elements inside the ‘foreignObject’ equivalent to elements inside an HTML document fragment. Any svg or math element, and their descendents, will be parsed as being in the SVG or MathML namespace, respectively; all other tags will be parsed as being in the HTML namespace.
SVG-namespaced elements within a ‘foreignObject’ will not be rendered, except in the situation where a properly defined SVG fragment, including a root ‘svg’ element is defined as a descendent of the ‘foreignObject’.

Hi, do we have progress / workaround on this?
Thanks

There is experimental support for foreignObject in the develop branch now: https://github.com/react-native-community/react-native-svg/pull/1242#issuecomment-573347807
Many thanks to @SaeedZhiany

Also, for multi-line text, you can use the inlineSize attribute on TSpan elements, and it will cause line-breaks to ensure the text fits inside that width

:tada: This issue has been resolved in version 10.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

This library is so awesome and have a good support as well, i have a problem with ForeignObject as it doesn't render transform is style on View, anyone have the same issue or only me?

This library is so awesome and have a good support as well, i have a problem with ForeignObject as it doesn't render transform is style on View, anyone have the same issue or only me? , is it ok as well to ask this question here or should i ask it on a new issue? thank you in advance.

@youssefali424
Please open a new issue with more detail and a reproducible example (preferably an expo project to see the problem quickly).
Thanks

@msand Could you provide an example of inlineSize. I try to use it but without any effect


WIP: Fix DeepSpeed checkpointing issues

Still very broken; please take a look later.
Does not yet support loading a VAE DeepSpeed checkpoint for the DALL-E model.

Add DeepSpeed checkpointing.
Add compatibility to load non-DeepSpeed checkpoints for resuming training.
When DeepSpeed is used, we only save DeepSpeed checkpoints. These can be converted to standard checkpoints using scripts in the DeepSpeed repo (look for "consolidating checkpoints").
A nice side effect is that we are now also able to initialize models that don't fit into memory.

Need testing to see whether this allows loading the broken ZeRO stage 3 checkpoint in #211. @afiaka87 pretty please? :sparkles::eye::lips::eye:

I probably won't be able to change my CUDA install locally for awhile as I'm using pytorch to process a bunch of data for the next couple of days.
@lucidrains Any chance you can spin it up?

Sorry, I don't think I'll find time to look into this further soon. I tried to fix the issues without touching the rest of the code base too much using metaprogramming black magic but it never quite worked out.
I will close this for now, keeping the branch as some documentation on what I tried out.


Open metals' logs relatively to LSP-root

Describe the bug
In the following project structure:
app
  ui/
  server/
    build.sbt
    src/main/scala/Main.scala

Start nvim with app as a root directory
Go to Main.scala. Metals launches with a root at app/server
Toggle Metals logs. A buffer with a tail of app/.metals/metals.log opens

The use case is caused by a desire to develop multiple components (potentially in different languages) in a single nvim instance
Expected behavior
A buffer with a tail of app/server/.metals/metals.log opens
Operating system
macOS
Version of Metals
v0.11.10
Commit of nvim-metals
c552c96b837ce8b1318cf3eccbda41a73760cb21

Thanks a lot for opening this @susliko. While trying to reproduce this I actually hit on another issue, which I think is sort of pointing out a flaw somewhere, but I need to dig in a bit further to figure it out. For example if I have
.
├── build.sc
├── minimal
│  ├── src
│  │  └── Main.scala

And I open build.sc all is fine and the .metals dir gets created there like I'd expect. If I then exit and cd minimal/src and open there, I also see another .metals/ created there. The log there still says it recognizes the right root, and that the logs are going to the other .metals/metals.log, but there shouldn't be another .metals/ created. I'm not sure why that's happening now especially seeing that the correct root is still being detected. This might be an issue in Metals actually. I want to confirm that first and ensure we're not getting extra .metals/ dirs created, and then we can address the issue you mentioned @susliko.

@ckipp01 I can confirm that running nvim from ./minimal/src in your example (in sbt project though) and entering Main.scala results in an extra ./minimal/src/.metals directory. Its metals.log file contains the following lines:
2023.01.24 12:51:55 INFO  tracing is disabled for protocol LSP, to enable tracing of incoming and outgoing JSON messages create an empty file at PATH_TO_REPRO/repro/minimal/src/.metals/lsp.trace.json or ....
2023.01.24 12:51:55 INFO  logging to file PATH_TO_REPRO/repro/.metals/metals.log

Alright, @susliko are you able to try this again using 0.11.10+79-399623cf-SNAPSHOT? I believe this should be fixed.

@ckipp01 Sorry for the delay
Tested on locally published metals from commit https://github.com/scalameta/metals/commit/85e9f3acafeec7cc9fb4cab65c5578b88c593f15 (because sonatype timeouted when downloading snapshot 🤷🏻)

The case when build.sbt is collocated with the project root and we enter ./minimal/src is handled correctly: no extra .metals directories created, nvim-metals opens log file from the root
The original case of this issue, when build.sbt if located in one of a few modules of the project, still reproduces: both app/.metals and app/server/.metals directories are created and nvim-metals shows logs from the first one. These logs reference to the nested log file:

2023.01.27 11:20:51 INFO  logging to file  ..../app/server/.metals/metals.log

@susliko just to double-check, can you give #533 a try?

@ckipp01 Yep, the fix works, nvim-metals opens a more nested log file 🙇🏻
What about .metals directory, which is still created in the project root? Is it an acceptable behaviour or should it be fixed in Metals?

What about .metals directory, which is still created in the project root? Is it an acceptable behaviour or should it be fixed in Metals?

This shouldn't be happening. Metals should only create it either in the global log file or in the root uri that is passed in. However, again you'll need the nightly for this fix. With the latest snapshot if this is still happening, do you mind creating a reproduction where this happens that I can use to mimic it?

Just for the history: I was unable to reproduce it with the latest snapshot of Metals


Row height determined based on actual node height instead of line-height.

At the original code the line-height of the grid rows taken from the “line-height” css style from the first node.
If the contents of a node is multiline, its “line-height” doesn't represent real height anymore.
I suggest this replacement to rely on the node actual “height” instead of “line-height”.

I am having an issue with this one. I find that sometimes it messes up the row heights. See the picture below.

It probably has something to do with timing and when it calculates outerHeight().

Ah, OK, it isn't your code at all, which just exacerbates an existing problem.
In the sample file https://github.com/deitch/jstree-grid/blob/master/treegrid.html there are 5 trees, one of which ("Third Tree", div#jstreec) uses the theme small, which creates a smaller tree. But the code referenced applies a single style for every tree cell on the page, hence the issue.
This first needs to be fixed in general, then your code can be pulled in.

Ok, thank you for review :)

OK, that has been fixed. Now I can add yours.

Done. I had to do it manually because of the prior change, but all good now v3.2.9


dotnetCorePackages.combinePackages: fails with dotnet-sdk_6

Describe the bug
As of #194061 combinePackages fails when sdk_6_0 is added to the package list.
The LICENSE.txt and ThirdPartyNotices.txt files are no longer in the root of the package as that derivation expects which breaks the build if sdk_6_0 is included.
Steps To Reproduce
Steps to reproduce the behavior:

Add (with dotnetCorePackages; combinePackages [ sdk_6_0 sdk_3_1 ]) to your packages or inputs.
Build

Notify maintainers
Not sure who to notify
@mdarocha made the commit and is the last commit to be made
Metadata

system: "x86_64-linux"
host os: Linux 5.19.12, NixOS, 22.11 (Raccoon), 22.11pre413974.59d2991d425
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.11.0
channels(root): "nixos, nixos-hardware"
channels(zyl): ""
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

I'm still not great at making issues. I hope this is the correct way to do it.
Thank you for your patience

I already have a fix :) https://github.com/NixOS/nixpkgs/pull/194797

Awesome, I didn't notice you already made the pull request.
Thank you!


emacs30: 30.0.92 -> 30.0.93

Announcement: https://lists.gnu.org/archive/html/emacs-devel/2024-12/msg00869.html
diff: https://github.com/emacs-mirror/emacs/compare/emacs-30.0.92...emacs-30.0.93

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

nixpkgs-reviewd workflow started

Extra args: ``
Free space: true
Who asked: lucasew

TARGETS

x86_64-linux: true
x86_64-darwin: true
aarch64-darwin: true

Status and logs: https://github.com/nixosbrasil/nixpkgs-reviewd/actions/runs/12447957153

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 367148

aarch64-darwin

:white_check_mark: 4 packages built:

emacs30
emacs30-gtk3
emacs30-nox
emacs30-pgtk

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 367148

x86_64-linux

:white_check_mark: 4 packages built:

emacs30
emacs30-gtk3
emacs30-nox
emacs30-pgtk

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 367148

x86_64-darwin

:white_check_mark: 4 packages built:

emacs30
emacs30-gtk3
emacs30-nox
emacs30-pgtk


getCurrentDevice() Does Not Return a Custom Name

I just updated our link code to use the new - (void)registerUser:withDevice:withSuccess:withFailure: method. It worked, but the DevicesViewController -getCurrentDevice method does not return the custom name. Specifically, my device is named "Artie's iPhone", and that name does appear in the list of devices fetched via -getDeviceList, but the call to getCurrentDevice() returns iPhone8,1.

Are you passing "Artie's iPhone" to the call?
The NSString passed to the call will be the name of the device. So it's not using the UIDevice class to get the name of the device on a system level. The -(void)registerUser:withDevice:withSuccess:withFailure: will set the name of the device to whatever you pass to it, otherwise it will be named based off the device type. And -getCurrentDevice returns that same name.

Yes:
WhiteLabelManager.sharedClient().registerUser(
    code,
    withDevice:UIDevice.currentDevice().name,
    withSuccess: {
        self.submitting(false)
    },
    withFailure: { (errMsg, errCode) in
        NSLog("Link Failed: %@ - %@", errCode, errMsg)
        self.submitting(false)
    }
)

Note that the name "Artie's iPhone" correctly appears in the list of devices retrieved via getDeviceList(), but not in -getCurrentDevice(). Which also means I can't filter it out of the list of devices.

Strange. I just tried passing the current device name the same way you are, in both Objective-C and Swift, and was able to retrieve the correct device name with the -getCurrentDevice call. I even called it within the success block of the -registerUser:withDevice:withSuccess:withFailure: call and it worked.
I will look into this call further to see what could be causing these inconsistencies.

Okay, I just discovered that I was constructing a DevicesViewController at app launch — even before the device was linked. I then went into the view that displays devices and used that DevicesViewController. Which I guess hasn't been updated for the link. So I changed things so that I only create a DevicesViewController when I actually use it (it was left over from a use case where I was showing the list of devices on the main screen, which we're not currently doing). That appears to have fixed this problem as well as #5.
I apologize for the goose-chasing, @ashiranian24.
This came about because I'm not using DevicesViewController as a view controller, but as a model. I've written my own view controller. This will be something to keep in mind when rejiggering things to separate models from view controllers: Sometimes models might be hanging around! For such cases, it'd be useful to add notifications that every copy of a model could listen for. So if I have two DeviceCollection model objects, for example, and I unlinked a device from one, the other would get the notification and remove the device from its storage, as well. Even better if said models automatically trigger the updating of any tables they're linked to (CoreData has some magic like this).
Anyway, just a thought, and apologies again for the misdirection.


[FIX] Make report context manager stop swallowing exceptions

You can open HTML reports with a context manager, e.g.:
with mne.open_report('report.h5') as report:
    # Do things with report

When I implemented this, in my ignorance about how to properly implement context managers, I made mne.Report.__exit__ return self. Returning truthy values causes the context manager to suppress all exceptions within the context block! mne.Report.__exit__ should not return anything.

Returning truthy values causes the context manager to suppress all exceptions within the context block! mne.Report.exit should not return anything.

I had no idea! Nice catch


0.1.18

new experimental support for choropleth maps
refactor chart properties and docs
bugfixes

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Does QtAVPlayer work with FFMPEG 7 ?

When looking through qtmultimedia changes update below hints for possible differences with Ffmpeg 7 in hw accelerated decoding and rendering:
https://codereview.qt-project.org/c/qt/qtmultimedia/+/560858

Old FFmpeg versions expose supported hw formats in codec->pix_fmts,
whereas FFmpeg 7.0, and partially FFmpeg 6.1, expose the formats in
hw_config.
The patch uses both of them to support the best compatibility.
Pick-to: 6.7 6.5

I can confirm that QtAVPlayer is working with Qt 6.7.2 (PR #490 required) and FFmpeg 7.0.1-full_build-www.gyan.dev.

Yes, I can confirm too, I have it running on Qt 6.6.3 with (self compiled) FFmpeg 7 too, latest from git, on Linux and MacOS. I had to make 3 very minor changes in QtAVPlayer code, easy.


Warn about a weak API-Key at startup

Readiness checklist

[ ] I added/updated tests.
[x] I ensured that the PR title is good enough for the changelog.
[x] I labeled the PR.

Description
Additionally it is not possible anymore to start the connector without an API Key in production mode.

Hmm, some downside - the warning is now spammed in the tests. Maybe it should be disabled when tests are detected.

the warning is now spammed in the tests

Why don't we have a strong API-Key? 🧐

@Milena-Czierlinski also an Option, let me think about this.

I'm absolutely no fan of introducing an external password-check library for such a low-level check. I would recommend removing this external dependency for security reasons and match the following regexp which still needs to be fit to our requirements (see https://stackoverflow.com/questions/5142103/regex-to-validate-password-strength).

^                         Start anchor
(?=.[A-Z].[A-Z])        Ensure string has two uppercase letters.
(?=.[.!@#$&])            Ensure string has one special case letter.
(?=.[0-9].[0-9])        Ensure string has two digits.
(?=.[a-z].[a-z]) Ensure string has two lowercase letters.
.{8}                      Ensure string is of length 8.
$                         End anchor.

passwordPolicy = new RegExp("^(?=.*[A-Z].*[A-Z])(?=.*[!@#$&*])(?=.*[0-9].*[0-9])(?=.*[a-z].*[a-z]).{30,}$")
if (!passwordPolicy.test(password)) {
    console.warn(...)
    // bye bye baby
}

I'm absolutely no fan of introducing an external password-check library for such a low-level check

That lib was well maintained and had more features than a simple regex, but I removed it.
I added some more special characters to make the feature way more usable.

LGTM


Model do not throw error on wrong ID

I wrote wrong code:
var id = 32;
new MyModel(id);

where I used number ID instead of string. As result Swarm didn’t sync models.
But I think it is common mistake and we should throw a error on wrong type of ID.

To avoid confusion, 1.0 dispenses known-id objects from Host.get.
new Type() makes new objects.


Support triggers

Support triggers feature implemented here https://github.com/mindsdb/mindsdb/pull/6819

I want to give it a try.


Cursor Does Not Re-appear if RPCS3 Crashes

I hid my cursor with KAMI and RPCS3 Crashed. When it did, my cursor remained invisible. It was only visible when I passed it over text fields because it turned into a capital I.
Revealing my cursor with KAMI after RPCS3 had crashed was not possible. I had to relaunch RPCS3 and boot the game I was playing again to reverse the process, all while using an invisible cursor.

You should not use the hide cursor option with RPCS3 unless you must, it is a bad feature and you just experienced why. but if something goes wrong we must be able to still toggle KAMI and reset the cursor, I have added handling such that if you close KAMI while the cursor is hidden it should reset.

Yeah, I also discovered that it stays hidden if KAMI crashes, too. So instead of using KAMI's hide cursor feature, I've been using this AutoHotKey script to hide my cursor: https://www.autohotkey.com/docs/commands/DllCall.htm#HideCursor. No problems so far. Maybe we could use that as a workaround until KAMI's hide cursor feature is more stable.


Unable to deserialize lockfile

$ RUST_BACKTRACE=1 multitool --lockfile multitool.lock.json update
thread 'main' panicked at src/main.rs:206:41:
Unable to deserialize lockfile: Error("invalid type: string \"https://raw.githubusercontent.com/theoremlp/rules_multitool/main/lockfile.schema.json\", expected struct Binary", line: 2, column: 100)
stack backtrace:
   0: _rust_begin_unwind
   1: core::panicking::panic_fmt
   2: core::result::unwrap_failed
   3: multitool::main
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

{
  "$schema": "https://raw.githubusercontent.com/theoremlp/rules_multitool/main/lockfile.schema.json",
...

I can provide the rest of the lockfile if needed, but I am not sure think it matters—even when I strip away everything else, it still fails. Looks like the parsing expects there to be only Binary declarations, not the "$schema".

Released in 0.3.0.

Thank you!


fix: [#2398] Support Bots that use Single Tenant AAD Applications

Fixes #2398
Description
This PR enables the tenant id value to authenticate Emulator with Single Tenant apps.

This feature requires bots with the last updates of botbuilder-dotnet and botbuilder-js:
PR#6769
PR#4643

Specific Changes

Added tenantId property to the classes involved in authentication.
Added logic to get the access token for the Single Tenant login endpoint.
Added method to use tenant ID value to create a dynamic issuer.

Testing
These images show the new field Tenant ID in the Open Bot and New Bot Configuration views.

This is the new message error related to authentication.

The following image shows the Emulator working with a bot using Single Tenant authentication.

coverage: 67.547% (-0.004%) from 67.551%
when pulling 24cb7de1de9d3dc98549fa25a78c9e20269b1e1c on southworks/add/support-singletenant-apps
into 012a6641077fe96c8155687f8e6ecb1ebffdc0e3 on main.

@tracyboehrer is there anywhere I can get a build of the emulator with this PR in it?  The nightlies repo doesn't have any recent builds.  I could build it myself, looking to avoid the time spent getting all of the build tools stood up so I can focus on testing a bot. I'm hoping this exists and I just missed it.
Thanks!

@GaryIrick At the moment there isn't.  This also relies on a change on the SDK side to take the tenant into account when validating the token.  That has just now made it to the DotNet SDK.


phpunit error

when I run phpunit, it says
Could not use "Codedungeon\PHPUnitPrettyResultPrinter\Printer" as printer.

Hmm done in fresh install?


Create and take range orders

It would be nice to be able to create and take range orders and have the quantity that has not been taken republished

This has been implemented here 2d263fa3ed1e1c04a5f26d879f7eb33eaad680e3 and in subsequent commits. It's coming up in the next release.


Uncaught ReferenceError: vendor_ is not defined when using DllReferencePlugin

Firstly, apologies for raising yet another issue around this, I've already tried everything in this one with no luck.
I've put together a sample application which displays this issue for me, it can be found here. The steps to reproduce are as follows:
Clone the above repo
dotnet restore
npm install
npm run webpack
dotnet run
I receive the following error: NodeInvocationException: vendor_19f7ef894c06af9bbdef is not defined
I've tried everything in the issue linked above, as well as playing around with the DllPlugin settings in both webpack.config.js and webpack.config.vendor.js. The only way to get the app working that I've found so far is to remove the following from webpack.config.js:
new webpack.DllReferencePlugin({
  context: __dirname,
  manifest: require("./wwwroot/dist/vendor-manifest.json")
})

Any ideas what I'm missing? I'm sure it's something very simple but I can't figure it out!
I'm running node v8.9.3 / npm v5.6.0 / dotnet v2.1.3 if it helps.
Cheers

Since this is a Webpack config question (and you have a custom Webpack configuration), would you be able to raise it with the Webpack community? Thanks!

Hi @SteveSandersonMS - this was originally generated using the dotnet CLI (VueJS template). All I've done is remove TypeScript for personal preference, and add a couple of extra parts around SCSS and ES6 support. The DllReference code is all standard dotnet template as far as I can recall? Cheers.

DllReferencePlugin is a standard Webpack feature, not a .NET Core feature. I don't think we ever had support for SSR in the Vue template.

No the VueJS template doesn't have SSR, I'm just trying to add it in but as soon as I render on the server I get this issue. Client rendering is fine. I'll jump on SO and raise it there.
Thanks for your help.

hey @sturatcliffe did you ever get this working? I'm facing a similar issue (linked above) perhaps you can offer some insight?


fontsize needs to be quoted

In pydot 3.0.1 fontsize attribute is not quoted, while it should - if it is string.
Here is an example:
import pydot

graph = pydot.Dot("my_graph", graph_type="graph")

# Add nodes
graph.add_node(pydot.Node("A", fontsize=12))  # works
graph.add_node(pydot.Node("B", fontsize="\"12pt\""))  # works
graph.add_node(pydot.Node("C", fontsize="12pt"))  # doesn't work

# Add an edge
graph.add_edge(pydot.Edge("A", "B"))
graph.add_edge(pydot.Edge("B", "C"))

print(graph.to_string())
# Generate the SVG content
svg_content = graph.create_svg()

dot file:
graph my_graph {
A [fontsize=12];
B [fontsize="12pt"];
C [fontsize=12pt];
A -- B;
B -- C;
}

Note that Node B is fine as it is created withwith graph.add_node(pydot.Node("B", fontsize=""12pt""))
but Node C is not:
graph.add_node(pydot.Node("C", fontsize="12pt"))
Error produced:
stdout, stderr:

b''
b"Warning: syntax ambiguity - badly delimited number '12p' in line 4 of /var/folders/20/2whws7mn5hdcjbrdmqbyfph00000gn/T/tmpf6psb8kv splits into two tokens\nError: /var/folders/20/2whws7mn5hdcjbrdmqbyfph00000gn/T/tmpf6psb8kv: syntax error in line 4 near ']'\n"

Ah!! Interesting corner case that we seem to have missed, yeah.
Pydot will quote string-format attributes unless they're all-alphanumeric, which sounds like it should be good enough (and covers things like label=node1 which is fine)... but mixed-alpha/numeric strings that start with numerals should also be quoted.

This is actually kind of confusing, because the type of attribute fontsize is documented as "double" — yet, graphviz clearly accepts a string value containing units, not just a double.
Same thing for other sizes like height=, which is a "double" in inches, but setting it to either height=2 or height="2in" is equivalent. (And presumably, it would convert other units, like the equivalent height="144pt"?)
Anyway, #411 should fix it.


Relax version number to major only

It seems that the Rsyslog apt repo often loses minor versions, we should update the version to be 8.* instead of 8.{minor}.*.

Closed by #11


Bring over DevSecOps Features

All the initial updates from the DevSecOps Repo.  I've created the https://compai.aiaadeic.org url for your site.    Once you merge this PR, I'll fix up the github pages to make sure it's deploying.
It will watch the main branch which I've protected.  You'll be able to push directly to the main branch.  If you want to give anyone else that capability you'll need to add them in the Settings -> Branches -> Branch Permissions for the main branch.
Essentially, anything that gets added to the main branch will get built into the resulting deployed website automatically.  So if you want to manage additions in a controlled way you'll want to use pull requests to main that you can review (like this one!)
If you want me to review updates to ensure they'll build correctly let me know.  I need to write a short document about running the docs site locally for big additions.  I'll do that and add it to this PR if you haven't merged it yet, or make a new PR if you have. Added this to the contributing file.

Trying to commit merge. Confused at the thrashing.

Trying to commit merge. Confused at the thrashing.

Hmm... Strange when there were only a couple files before...


Friday 09-06-2023 roll call

Roll Call!
Leave a comment with these two things:

A summary of your study using only emojis
Something you'd like to share, anything goes! (within respect)

emoji cheat sheet

Good Morning

Present

morning

good morning

hi

good morning

Good morning

Good morning

Good morning

Present 🙋♂

Good morning 🌞

Good morning

morning

present

hi

🥲


Concrete Playback playback subcommand picks up regular unit tests and fails to compile

When there are other unit tests present, running cargo kani playback -- kani_concrete_playback_harness_i64_13477596495325479973 for example, gives compilation errors that are picked up from unit tests unrelated to kani. The command breaks completely and displays an empty output.

After discussing offline, the problem seems to be with the integration tests, not with the playback back mode. The integration tests were using functions that were not defined when kani config was set. We can provide options for the user to disable the integration tests though.


Liquid Tank

Description (Required)
The liquid tank - doesn't calculate liquids going in correctly, being placed correctly, and water that is displaced doesn't flow back correctly.
Steps to reproduce the Issue (Required)

Expected behavior (Required)

Server Log / Error Report

Environment (Required)

Minecraft Version:
CS-CoreLib Version:
Slimefun Version:
Plugin Version:

you guys figured this one out already?

Kinda, we have the cause and the fix, it's linked to the other open issue around blocks not updating and will be in the fix for that. the issue is how to approach it as it's awkward


Scan through all NICs on the VMWare instance, looking for the configured one

This fixes #3388 by borrowing from a newer helper function in govmomi, but I'm not sure if updating to get WaitForNetIP would be enough.  I need to force IPv4 but would it need to fall back & let others still work with just IPv6? @frapposelli?
Signed-off-by<EMAIL_ADDRESS><EMAIL_ADDRESS>

Closing in favour of #3436, which is a cleaner fix and the govmomi upgrade fixes another small bug.


src/ should not be part of the include path

Description
Files such as glfw_window.cpp include private headers with right angle brackets full paths, this is misleading. All private headers should be included with relative paths.
With this change, src/ could be removed from the include path

Issue 600 wow


Error happens when use webp type image on server, but normal on local

Server environment: CentOS 7.2   LNMP
Local environment: Windows10    WNMP
example:
when use {{ 'assets/images/theme/bg-presence.webp'|theme }}, both server and local worked well.
when use {{ 'assets/images/theme/bg-presence.webp'|theme|resize(200,200) }}, server throws error, but local works well.
Error info:
# /www/wwwroot/octobercms/vendor/october/rain/src/Database/Attach/Resizer.php line 527

        break;
    case 'image/png':
        $img = @imagecreatefrompng($filePath);
        $this->retainImageTransparency($img);
        break;
    case 'image/webp':
        $img = @imagecreatefromwebp($filePath);               ##here line 527##
        $this->retainImageTransparency($img);
        break;
    default:
        throw new Exception(sprintf('Invalid mime type: %s. Accepted types: image/jpeg, image/gif, image/png, image/webp.', $this->mime));
        break;
}

WebP seems not to be supported by your GD installation.
https://www.php.net/manual/en/intro.image.php

To find out which formats that was available to GD during compilation, use the gd_info() function


Update job listing in about.jade

Changed job description on the about#careers page (/app/templates/about.jade) from Software Engineer (China) to Administrative Assistant.

@Aliensinthesky Is this ready for code review? There's some issues with change history here that we'll want to resolve.

If you don't mind I'll close this as it's not required any more. We can re-open if becomes required again. Thank you!


Not having yet investigated further, I suspect that something may be calling os.chdir into a tmpdir, and then that tmpdir is deleted asynchronously by the pytest teardown.

Did you ever narrow down this problem? We've found a reproducible 'test case' on EL9 hosts, but ... for some reason only in subdirectories, not in the top level mount.
But the failure scenario is fairly straightforward so we can reproduce it, and it seems to be something happening in kernel space - with rpcdebug enabled we get nfs_revalidate_inode failures
Apr 24 10:46:50 build-2 kernel: nfs_revalidate_inode: (0:56/3839636522) getattr failed, error=-512
I know it's a few years later, but this is the most relevant 'hit' when trying to look for related issues.


Unable to arm with Counterforce

Hello
I am trying to arm my system with your integration but it does not work. However I am perfectly able to disarm, including inside an automation.
Since I don't need to use my code fo arming my system, I did not added it in my configuration. I tried with the code without success.
I am subscribed with Counterforce in Canada. I'm using 2FA.
Here is my configuration.yaml:
alarm_control_panel:
  - platform: alarmdotcom
    username: scrambled_user
    password: scrambled_pwd
    # The below parameters are optional
    # code: 1234
    # force_bypass, no_entry_delay, and silent_arming are not supported on all systems/providers. See the description section below.
    force_bypass: "true"
    no_entry_delay: "home"
    silent_arming: "false"
    # two_factor_cookie is only used if your portal is forcing 2FA
    two_factor_cookie: "A0KD871991SSJS..."

Extra information in case you nee it, from my home assistant.log:
2021-12-09 09:06:15 ERROR (MainThread) [pyalarmdotcomajax.pyalarmdotcomajax] Can not load state data from Alarm.com
2021-12-09 09:06:44 ERROR (MainThread) [pyalarmdotcomajax.pyalarmdotcomajax] Unable to extract state data from Alarm.com

2021-12-09 10:26:49 ERROR (MainThread) [pyalarmdotcomajax.pyalarmdotcomajax] Arm+Stay failed with HTTP code 422
2021-12-09 10:26:49 ERROR (MainThread) [pyalarmdotcomajax.pyalarmdotcomajax] Arming parameters: force_bypass = True, no_entry_delay = True, silent_arming = False
2021-12-09 10:36:13 ERROR (MainThread) [pyalarmdotcomajax.pyalarmdotcomajax] Arm+Stay failed with HTTP code 422
2021-12-09 10:36:13 ERROR (MainThread) [pyalarmdotcomajax.pyalarmdotcomajax] Arming parameters: force_bypass = True, no_entry_delay = True, silent_arming = False

Current state of my entity:
code_format: null
changed_by: null
code_arm_required: true
sensor_status: System OK, Clavier Portable is Fermé, Détecteur de fumée is Non réinitialisé, Mouvement Sous-sol is Activé, Porte Avant is Fermé, Porte Patio is Fermé, Porte Patio D is Fermé, Sous-sol is Fermé, Transmetteur is Fermé
friendly_name: Alarm.com
supported_features: 3

Please see #57 and #32 as well as the comment about the force_bypass, no_entry_delay, and silent_arming parameters here: https://github.com/uvjustin/alarmdotcom#description-of-configuration-parameters
I don't mean to be rude, but I can't keep spending time answering the same questions over and over again. Please use the HA forums for tech support.


Types don't colorize at declaration

Install latest master of F# and Roslyn in a single hive.
Create a Console App.
Enter the following:

// Learn more about F# at http://fsharp.org
// See the 'F# Tutorial' project for more help.

type BST<'T> =
    | Empty
    | Node of 'T * BST<'T> * BST<'T>

let rec flip (tree: BST<'T>) =
    match tree with
    | Empty -> tree
    | Node(x, left, right) -> Node(x, flip right, flip left)

[<EntryPoint>]
let main argv =
    printfn "%A" argv
    0 // return an integer exit code

Note that BST<'T> isn't colorized at the first declaration, but is colorized in subsequent uses.

Also, generic parameters don't colorize, either.
let x = new HashSet<SomeType>()
//                  ^^^^^^^^ this isn't colorized

let y = new HashSet<int>()
//                  ^^^ neither is this

Fixed in https://github.com/Microsoft/visualfsharp/pull/1841

@vasily-kirichenko Thank you for this fix


[Question]: filters doc doesn't include GRPCProxy

This is not Bug report or Feature request

[X] I confirm that this is not bug report or feature request.

Is this question answered in the documentation?

[X] I have searched the documentation

Is there an existing issue for this?

[X] I have searched the existing issues

Your question
Filter GRPCProxy isn't included in filter docs. Author could write this for grpc is a popular protocol over the world. I found this feature by making a global search in the code.

Thanks for your reminder.
I will write and submit the filter doc as soon as possible after the PR is merged.

PR，doc is submitted.


ENHANCEMENT Allow extensions to intercept incorrect deletes on unpublish

The issue with the existing behaviour is that some queried tables are NOT a part of the hierarchy, and the subsequent delete could unintentionally delete objects outside of this object.
For instance, fluent uses a localisation table as an inner join. This existing behaviour deletes these locales, but using the ID of the record, not the ID of the localisation (which is a separate ID itself).
With the given extension points extensions are able to intercept these queries and rewrite the delete safely without violating database integrity.

For instance, this is a page that has three failover locales. When deleted it performs these deletes:
"SiteTree_Live, ID = 3"
"SiteTree_Localised_Live, ID = 3"
"SiteTree_Localised_Live, ID = 3"
"SiteTree_Localised_Live, ID = 3"

Those following three deletes should be RecordID = 3 not ID = 3.

Seems reasonable, and it doesn't change anything if there were no extensions :)


add tool to create and download the DEM derived data

As ArcticDEM derived data (relative elevation, slope) is required for running, we should provide a tool to download those.
Tasks:

Define workflow for end user to select spatial extend (RapidEye Tile IDs, bounding box, arbitrary vector data ??) of data to download
think about a tiling scheme - if we use RapidEye Tiles, we need to provide that data as well in some way
use already existing processing chain for download (get_ArcticDEM_slope(), get_ArcticDEM_elevation() )
create CLI tool and/or Notebook (parameters: spatial extend, download folder)

current implementation: https://github.com/initze/thaw_slump_preprocessing/blob/main/DEM_01_ProcessArcticDEM.ipynb

good point, I have a notebook for that, will add the link shortly to formalize this a bit better!


Customizer does not update border-color for button hover and panels.

When making a theme with the bootstrap customizer the border-color does not get overridden in the :hover state by the theme, so it will remain blue.
To reproduce:

Go to customize
Change brand-primary to some other color like green

As you can see the border-color is still blue, while it should be green.
This happens on buttons in the :hover move and on panels in generally.

I wasn't able to duplicate this. My entering #FF0000 as brand primary without any other changes correctly resulted in the following red .btn-primary:
.btn-primary {
  color: #ffffff;
  background-color: #de0000;
  border-color: #c40000;
}
.btn-primary:hover,
.btn-primary:focus,
.btn-primary.focus,
.btn-primary:active,
.btn-primary.active,
.open > .dropdown-toggle.btn-primary {
  color: #ffffff;
  background-color: #ab0000;
  border-color: #870000;
}

I believe you are mixing a vanilla bootstrap.css with a customized bootstrap-theme.css, which shouldn't be done. If you're customizing, then customize both files.