Unable to convert TensorFlow ssd_mobilenet_v2_coco

Hi,
I trained the model mobilenet_v2_coco, and froze it using the following command on host machine:
python object_detection/export_inference_graph.py \
input_type=image_tensor \
--pipeline_config_path=... \
--trained_checkpoint_prefix=... \
--output_directory=...

Then I moved the generated file (frozen_inference_graph.pb) to VM where OpenVINO is installed and ran the following command:
python3 mo_tf.py --input_model ~/vehicle_model/frozen_inference_graph.pb
 --tensorflow_use_custom_operations_config extensions/front/tf/ssd_v2_support.json

and got the following error:
Model Optimizer arguments:
Common parameters:
	- Path to the Input Model: 	/home/niko/vehicle_model/frozen_inference_graph.pb
	- Path for generated IR: 	/home/niko/intel/computer_vision_sdk_2018.5.455/deployment_tools/model_optimizer/.
	- IR output name: 	frozen_inference_graph
	- Log level: 	ERROR
	- Batch: 	Not specified, inherited from the model
	- Input layers: 	Not specified, inherited from the model
	- Output layers: 	Not specified, inherited from the model
	- Input shapes: 	Not specified, inherited from the model
	- Mean values: 	Not specified
	- Scale values: 	Not specified
	- Scale factor: 	Not specified
	- Precision of IR: 	FP32
	- Enable fusing: 	True
	- Enable grouped convolutions fusing: 	True
	- Move mean values to preprocess section: 	False
	- Reverse input channels: 	False
TensorFlow specific parameters:
	- Input model in text protobuf format: 	False
	- Offload unsupported operations: 	False
	- Path to model dump for TensorBoard: 	None
	- List of shared libraries with TensorFlow custom layers implementation: 	None
	- Update the configuration file with input/output node names: 	None
	- Use configuration file used to generate the model with Object Detection API: 	None
	- Operations to offload: 	None
	- Patterns to offload: 	None
	- Use the config file: 	/home/niko/intel/computer_vision_sdk_2018.5.455/deployment_tools/model_optimizer/extensions/front/tf/ssd_v2_support.json
Model Optimizer version: 	<IP_ADDRESS>d067a0
[ ERROR ]  Exception occurred during running replacer "ObjectDetectionAPIPreprocessorReplacement" (<class 'extensions.front.tf.ObjectDetectionAPI.ObjectDetectionAPIPreprocessorReplacement'>): To convert the model specify path to the pipeline configuration file which was used to generate the model. Please use "--tensorflow_object_detection_api_pipeline_config" option:
--tensorflow_object_detection_api_pipeline_config "<path_to_pipeline.config>"
If you have downloaded the model file from the Object Detection Model zoo repository then this file is located in the archive with frozen model and called "pipeline.config".
If you did not use this command line parameter before that means that you are using currently deprecated TensorFlow* Object Detection API models conversion mechanism.

I also tried to add pipeline.config file but got the following error:
python3 mo_tf.py --input_model ~/vehicle_model/frozen_inference_graph.pb --tensorflow_use_custom_operations_config extensions/front/tf/ssd_v2_support.json --tensorflow_object_detection_api_pipeline_config ~/vehicle_model/pipeline.config
Model Optimizer arguments:
Common parameters:
	- Path to the Input Model: 	/home/niko/vehicle_model/frozen_inference_graph.pb
	- Path for generated IR: 	/home/niko/intel/computer_vision_sdk_2018.5.455/deployment_tools/model_optimizer/.
	- IR output name: 	frozen_inference_graph
	- Log level: 	ERROR
	- Batch: 	Not specified, inherited from the model
	- Input layers: 	Not specified, inherited from the model
	- Output layers: 	Not specified, inherited from the model
	- Input shapes: 	Not specified, inherited from the model
	- Mean values: 	Not specified
	- Scale values: 	Not specified
	- Scale factor: 	Not specified
	- Precision of IR: 	FP32
	- Enable fusing: 	True
	- Enable grouped convolutions fusing: 	True
	- Move mean values to preprocess section: 	False
	- Reverse input channels: 	False
TensorFlow specific parameters:
	- Input model in text protobuf format: 	False
	- Offload unsupported operations: 	False
	- Path to model dump for TensorBoard: 	None
	- List of shared libraries with TensorFlow custom layers implementation: 	None
	- Update the configuration file with input/output node names: 	None
	- Use configuration file used to generate the model with Object Detection API: 	/home/niko/vehicle_model/pipeline.config
	- Operations to offload: 	None
	- Patterns to offload: 	None
	- Use the config file: 	/home/niko/intel/computer_vision_sdk_2018.5.455/deployment_tools/model_optimizer/extensions/front/tf/ssd_v2_support.json
Model Optimizer version: 	<IP_ADDRESS>d067a0
The Preprocessor block has been removed. Only nodes performing mean value subtraction and scaling (if applicable) are kept.
[ ERROR ]  Cannot infer shapes or values for node "ToFloat".
[ ERROR ]  NodeDef mentions attr 'Truncate' not in Op<name=Cast; signature=x:SrcT -> y:DstT; attr=SrcT:type; attr=DstT:type>; NodeDef: ToFloat = Cast[DstT=DT_FLOAT, SrcT=DT_UINT8, Truncate=false](image_tensor_port_0_ie_placeholder). (Check whether your GraphDef-interpreting binary is up to date with your GraphDef-generating binary.).
[ ERROR ]
[ ERROR ]  It can happen due to bug in custom shape infer function <function tf_native_tf_node_infer at 0x7f6341caf400>.
[ ERROR ]  Or because the node inputs have incorrect values/shapes.
[ ERROR ]  Or because input shapes are incorrect (embedded to the model or passed via --input_shape).
[ ERROR ]  Run Model Optimizer with --log_level=DEBUG for more information.
[ ERROR ]  Stopped shape/value propagation at "ToFloat" node.
 For more information please refer to Model Optimizer FAQ (<INSTALL_DIR>/deployment_tools/documentation/docs/MO_FAQ.html), question #38.

Does anyone know how to successfully convert trained ssd_mobilenet_v2_coco model?

After upgrading TensorFlow to version 1.12, the conversion works.


fix($LogProvider) : Allow console methods overriding

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix
What is the current behavior? (You can also link to an open issue here)
If a user override a console method on its personal code, such as console.error
console.error = (function(cerror) {
  return function() {
    const args = Array.from(arguments);
    args.unshift('[my custom prefix] ');
    cerror.apply(console, args);
  };
})(console.error);

And then Angular logs something, such as an error:
scope.$apply(function() { throw new Error('boom'); })
// logs "boom"

We don't see the [my custom prefix]  appended to the error message. The original console.error is being used.
What is the new behavior (if this is a feature change)?
This tiny change will make Angular use the overriden console methods.
scope.$apply(function() { throw new Error('boom'); })
// logs "[my custom prefix]  boom"

Does this PR introduce a breaking change?
No
Please check if the PR fulfills these requirements

[X] The commit message follows our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:
If a user overrides the methods console.log, console.error, etc, this
change will use them rather than the default ones.

I signed it! 👍

First if we were to make this change, we should be consistent and do the same thing on IE as well.
But I'm not too eager to support this usecase - it doesn't seem like a good idea to dynamically overwrite global objects. If this is something specific to your Angular app, you should be decorating the $log service instead.
In any case, you can achieve the same result by overwriting the methods before loading Angular.

Alright, agreed, I as a user should find a work around. I'll check the $log service, thanks.
I'm going to close it then.


P3 images not rendered

Using v1.1.4 of the extension
I was following the Ray tracing in one weekend tutorial but the images generated using the sample code provided there are not rendered.

I get this blank screen on opening the ppm file.
The contents of the ppm file are as per requirement of P3.
Here is the code being used to generate the P3 file.
#include <iostream>
#include<random>
#include<ctime>
#include "vector.h"
int main() {
    srand(time(NULL));
    int nx = 300;
    int ny = 200;
    std::cout << "P3\n" << nx << " " << ny << "\n255\n";
    for (int j = ny-1; j >= 0; j--) {
        std::cerr<<"Scanlines remaining :"<<j<<std::flush;
        for (int i = 0; i < nx; i++) {
            float tmp = rand() / static_cast<float>(RAND_MAX);
            vec3 col = vec3(tmp,tmp,tmp);
            int ir = int(255.99*col[0]);
            int ig = int(255.99*col[1]);
            int ib = int(255.99*col[2]);
            std::cout << ir << " " << ig << " " << ib << " ";
        }
        std::cout<<"\n";
    }
}

You can view the specific image.ppm file here and reproduce the issue.

Hey @prakharpratapmall please email me a copy of the problematic P3 file as well and I will take a look at it when I have a bit more time. Thanks!
<EMAIL_ADDRESS>

@prakharpratapmall Apologies I'll take a look at the file you linked

@prakharpratapmall Your file has null bytes between each character. Please make sure that the file contains the right bytes using a hex viewer tool such as https://hexed.it
Please take a look at our sample file for p3 in /images
Instead of redirecting std::cout to a file in the terminal I would recommend using something like fstream

Oh no I just checked my uploaded file. While it does have null bytes in between it is not the original ppm file which I am having issue with. Apparently uploading it to drive changes something in the file. I am not sure how to send you the original file. It looks something like this.

which should be fine? The only difference between the sample images you have provided and my file is the line endings. I tried changing that too but it did not help.
I will mail you the file in hopes that you receive it unchanged.
I am also attaching it here by renaming it to image.txt since github doesnt allow '.ppm' files
image.txt
I will also try to zip and then upload it so that the file is not changed. Check that here

Ah ohkay got it! thanks for the quick response. For future people facing problems specifically while redirecting std::cout can refer to this. This worked out for me without having to switch to fstream.


Clarify docs for radio and checkbox conditionals

The documentation for the conditional Nunjucks macro option for checkboxes and radios is confusing.
The checkboxes documentation lists conditional as a boolean, and the radios documentation lists it as a string.
Neither are correct.
The conditional option is an object with a single nested option, html. The nested html option is a string, used to specify the content that is revealed when the checkbox or radio is checked.
Update the component YAML so that the documentation reflects this.
Checkboxes
Before

After

Radios
Before

After

Closes #1903

Thanks @36degrees
You probably know I've written types for the YAML component data here:
https://github.com/alphagov/govuk-frontend/blob/main/shared/lib/files.js#L163
This is so we can get autocomplete (and checks) in tests using getExamples() etc
But the example data just has a generic {object} type getComponentData() isn't that helpful, but if we're bringing things in line it might be good to start typing it with the various flavours of component data
Might reveal other places where we've got the object structure wrong


Update css to better match website

A bigger makeover is needed someday, but for now the following would go a long way:

Use Lato and Poppins fonts for body/headings
Set background colors to match palette of company website.

@micahalcorn - I know you're busy, but could you take stab at this? I can also do it.

@wanderingstan certainly. It may be this evening, but otherwise no big deal.

Nice work on the docs. Looking way better.
Are you seeing the halo around the logo? Can we get rid of that?

@joshfraser I did see that. It's actually part of the image file. I'll replace it and make another PR.

This looks great, @micahalcorn . Than you!


useFakeTimers returns timeout with react 18 and waitFor

Describe the bug
Describe the bug
useFakeTimer returns timeout error when used with waitFor in react 18.
I've been trying to use wairFor and fake timers in a unit test but i get this error message:
FAIL  src/__tests__/ExampleComponent.vitest.test.jsx > ExampleComponent > changes message after 1 second
Error: Test timed out in 5000ms.
If this is a long-running test, pass a timeout value as the last argument or configure it globally with "testTimeout".

I created a repo in github to show this problem and here it is.
Unit test:
import "@testing-library/jest-dom";
import { act, render, screen, waitFor } from "@testing-library/react";
import ExampleComponent from "../ExampleComponent";

describe("ExampleComponent", () => {
  beforeEach(() => {
    vi.useFakeTimers();
  });

  afterEach(() => {
    vi.restoreAllMocks();
    vi.clearAllTimers();
  });

  test("changes message after 1 second", async () => {
    render(<ExampleComponent />);
    act(() => vi.advanceTimersByTime(1000));
    expect(await screen.findByText("Hello, world!")).toBeInTheDocument();
  });

  test("changes message after 1 second --- 2", async () => {
    render(<ExampleComponent />);
    act(() => vi.advanceTimersByTime(1000));
    await waitFor(() =>
      expect(screen.getByText("Hello, world!")).toBeInTheDocument()
    );
    expect(await screen.getByText("Hello, world!")).toBeInTheDocument();
  });
});

React component:
import { useState, useEffect } from "react";

function ExampleComponent() {
  const [message, setMessage] = useState("");

  useEffect(() => {
    const timeoutId = setTimeout(() => {
      setMessage("Hello, world!");
    }, 1000);
    return () => clearTimeout(timeoutId);
  }, []);

  return <div>{message}</div>;
}

export default ExampleComponent;

The error is exactly the same with the findBy method. Note that if jest is executed the tests pass. It is also worth mentioning that with react 17 I have not had this problem.
Reproduction
https://github.com/maximinetto/vitest-issue
System Info
System:
   OS: Linux 6.1 Manjaro Linux
   CPU: (6) x64 Intel(R) Core(TM) i5-8400 CPU @ 2.80GHz
   Memory: 5.15 GB / 15.17 GB
   Container: Yes
   Shell: 5.1.16 - /bin/bash
Binaries:
   Node: 18.14.2 - ~/.nvm/versions/node/v18.14.2/bin/node
   npm: 9.5.0 - ~/.nvm/versions/node/v18.14.2/bin/npm
Browsers:
   Firefox: 111.0
npmPackages:
   @vitejs/plugin-react-swc: ^3.0.0 => 3.2.0
   vite: ^4.2.0 => 4.2.1
   vitest: ^0.29.8 => 0.29.8

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I worked around this by adding the following to a setupFile:
// Simulate Jest for waitFor()
// see https://github.com/testing-library/dom-testing-library/blob/0ce0c7054dfa64d1cd65053790246aed151bda9d/src/helpers.ts#L5
// and https://github.com/testing-library/dom-testing-library/blob/0ce0c7054dfa64d1cd65053790246aed151bda9d/src/wait-for.js#L53
global.jest = {
  advanceTimersByTime: (ms: number) => vi.advanceTimersByTime(ms),
} as any;

Doesn't seem like this is a Vitest problem. vi.advanceTimersByTime does what you asked it to do. For better compatibility with jest you can add a global jest variables in your setupFiles:
import { vi } from 'vitest'
globalThis.jest = vi

I'm not convinced by your answer, besides I don't plan to use jest but let's leave it at that.


Apk zip recompiling store vs deflate

how apk zip recompiling is handled?
I found .png and .jpg archived with deflate, which there's no need of (being those filetypes already de facto compressed), while resources.arsc as store, which on the other hand could deflate nicely.
Some deflated packed .pngs are even slighty bigger than original, due to compression overhead.

Looks like you are looking for this block of code - https://github.com/iBotPeaches/Apktool/blob/master/brut.apktool/apktool-lib/src/main/java/brut/androlib/Androlib.java#L164
Compression or not is the source of plenty of problems and a variety of pain points and discussions around the tracker.

just edited apktool.yml - do not compress section and added - png


Error when expanding thumbnails in get Item request

Hi,
I'm getting an ImportError when I try to get a Item with expanded thumbnails field.
Here's the code to reproduce it:
returned_item = client.item(drive='me', id='file_id').request(expand="thumbnails").get()
I would like to obtain the Item and thumbnails information in a single request and this seems the right way to do it.
BTW I cloned the repo and patched the code adding the missing import statement and got the function to work, am I missing something?
I use python 2.7, onedrivesdk 1.1.4 .
Thank you

Hey! Just wanted to let you know that we are looking into this. While your case does only involve that fix, and will work for you, there were a few other related issues uncovered while looking into it.

This is fixed with #65. Keep in mind that paging has now changed for collections, See the README for an example on how to do this now.


PHOENIX-6586 Set NORMALIZATION_ENABLED to false on salted tables

Sorry for the additional review requests, I missed that @ankitsinghal  has already approved. (Thanks!)


CNA calls

We should have the data in more ready to use format. Gistic calls (-2,-1,0,1,2) are much more usable than log ratios.  Can we  have a conversion or a way to take the "raw" log ratios into discrete calls? I know we talked about keep the data as raw and reproducible as possible, but these data are just not as usable.

All CNA-datasets now have GISTIC or GISTIC-like calls instead of log-ratios.


Future support of reading first columns along/instead of first rows?

Hello!
I currently extract first row and column of my worksheet by parsing through the cells, but I would like to limit the file reading so only needed data (first column and/or row) is extracted.
I already know about using XLSX.read(myFile, { sheetRows: 1 });, but I didn't find any issue discussing a column variant sheetColumns. Is such a feature planned for later? Or not at all?
Thanks for your time! :)

It's definitely interesting but lower priority.  Some notes:
Most file formats store data in row-major order.  For example, in the CSV format:
A1,B1,C1,D1,...
A2,B2,C2,D2,...
A3,B3,C3,D3,...
A4,B4,C4,D4,...
...
A1000000,B1000000,C1000000,D1000000,...

The file starts with the first row of data, followed by the second row, followed by the third row, etc.  For sheetRows: 1, the parser can stop after the end of the first row.
To extract the data from the first column, the parser would have to scan almost the entire file (stopping at cell B1000000 in the last row)
.
The dual sheetCols parameter is interesting because some formats are column-major order.  The XLS file from issue #2432 is in column-major order.  BIFF8 XLS also has optional special records that let you jump around the file.  This approach would not work for XLSB or XLSX.

Dupe of #335

@reviewher related but not really duplicate, my issue is not with the format order of the json (row-major) but with limiting the parsed (or stored) data to the initial columns.

drop a comment in #475 that a column limit feature should also be supported


qt5.12

I wonder what people think about updating this feedstock to qt5.12.
We see different problems with qt5.6.2 on osx and linux (freecad-feedstock) and think updating qt would solve some of them. So we are definitely interested into qt5.12.
I know qt is one of the most difficult packages in conda-forge, but it's more likely to get done if more people are interested in this task. So maybe other maintainers interested in qt5.12 can show up here and give their opinon about how and when this conda-forge qt5.12 should happen. I guess  one of the main interests of anaconda to provide qt is spyder. So maybe asking @ccordoba12 what he thinks about updating this feedstock.
@mingwandroid has already build a package of qt5.12 for osx. So maybe he can give some insights on what has to be done for the other platforms.

So maybe asking @ccordoba12 what he thinks about updating this feedstock.

We in Spyder are already well covered by the excellent packages created by @mingwandroid for Anaconda. conda-forge is a community effort, so it depends on the time and good will of volunteers to run.
Until @mingwandroid updates Qt to 5.12 in Anaconda, I think there's no point in trying to do that here. The builds don't pass on Azure even after 6 hours, as PR #88 showed (except for macOS), so there's no way to see if things are working. Besides, it's an extremely time consuming effort for volunteers.
The good thing about PR #88 is that it showed that the recipes for Qt 5.9 are working very well, so we need to resort to what we've done several times in the past: compile locally for Linux and Windows and upload the packages to Anaconda.
I could help with Linux. @gillins, do you have time to help us with Windows?

@gillins, do you have time to help us with Windows?

:+1: I don't have a lot of expertise with Qt on Windows, but will help where I can. I have Windows hardware to do builds on.

Great!! Thanks a lot @gillins! Let's work on PR #88 to see if we can manage to get this done.

Can anybody give me ssh access to a windows server to debug qt 5.12 on windows?

I can try to set you up with an RDP connection.  The computer is not great, though - old Phenom II x6.  You'd also need to set up compilers and stuff.  Unfortunately my nicer win machine only has win 10 home.

The computer is not great, though - old Phenom II x6.

Sounds better than my current solution (VM on top of ubuntu)

Unfortunately my nicer win machine only has win 10 home.

It's been a while since I used windows. What's the difference?

win 10 pro allows RDP, win 10 home does not. =(

I think I can find another way.  Nomachine or vnc or something.


Add getObjectDefinitions() to DMS SDK

Hi @nschoellhorn,
thanks for your contritution! I will look into your changes asap!

Hi @nschoellhorn,
sadly this route is not part of the public API yet. I have no choice but to close this issue.


Allow DeprecatedScopeResolvers to be aware of the node being processed

This stems from work in https://github.com/mglaman/phpstan-drupal/pull/714 and https://github.com/mglaman/phpstan-drupal/pull/659 in phpstan-drupal.
It's complicated, but it's based on this code:
$result = DeprecationHelper::backwardsCompatibleCall(
        currentVersion: \Drupal::VERSION,
        deprecatedVersion: '10.3',
        currentCallable: fn() => Role::loadMultiple(),
        deprecatedCallable: fn() => user_roles(),
      );

We want to use a DeprecatedScopeResolver to ensure the code called in the deprecatedCallable is within a deprecated scope.  That's been accomplished in the PR with DeprecationHelperScope so far. However, it is including currentCallable (we want to find and error on deprecated code within that callable.)
Using a NodeVisitor it is possible to traverse the children in the deprecatedCallable tree and set a deprecated attribute. But, this attribute cannot be read using the Scope alone.
There are problems with backwards compatibility to modify \PHPStan\Rules\Deprecations\DeprecatedScopeResolver::isScopeDeprecated. So, this is my idea.
Introduce NodeAwareDeprecatedScopeResolver which DeprecatedScopeResolver implementations can also implement.
interface NodeAwareDeprecatedScopeResolver
{

	public function withNode(Node $node);

}

DeprecatedScopeHelper needs to be modified to support this:
	public function isScopeDeprecated(Scope $scope, Node $node): bool
	{
		foreach ($this->resolvers as $checker) {
			if ($checker instanceof NodeAwareDeprecatedScopeResolver) {
				$checker->withNode($node);
			}
			if ($checker->isScopeDeprecated($scope)) {
				return true;
			}
		}

		return false;
	}

So what is the problem here? You have this code:
$result = DeprecationHelper::backwardsCompatibleCall(
        currentVersion: \Drupal::VERSION,
        deprecatedVersion: '10.3',
        currentCallable: fn() => Role::loadMultiple(),
        deprecatedCallable: fn() => user_roles(),
      );

And when a deprecated function is called inside deprecatedCallable argument, it should not be reported, but when a deprecated function is called from inside other arguments, it should still be reported?

Correct. Given that user_roles is deprecated, but we want to support multiple minor versions,
// This is OK, callable in `deprecatedCallable` considered
DeprecationHelper::backwardsCompatibleCall(
    currentVersion: \Drupal::VERSION,
    deprecatedVersion: '10.3',
    currentCallable: fn() => Role::loadMultiple(),
    deprecatedCallable: fn() => user_roles(),
  );

// This is _not_ OK because the deprecated code is in `currentCallable.`
// It's not common here, but could happen when _not_ using named parameters and mixing up the arguments.
DeprecationHelper::backwardsCompatibleCall(
    currentVersion: \Drupal::VERSION,
    deprecatedVersion: '10.3',
    currentCallable: fn() => user_roles(),
    deprecatedCallable: fn() => Role::loadMultiple(),
  );

So how about https://github.com/phpstan/phpstan-src/commit/b87e5c4c7e33a91c61341f0335a221c32df603b2 ? Would calling Scope::getFunctionCallStackWithParameters() work for you?

🤔 maybe! I didn't see that method call, only the getFunctionCallStack. I will try that and report back.

Because I just added it.

It worked! mglaman/phpstan-drupal@23bf3ef (#714)
This allowed me to target the specific parameter in the function call stack. I love it because this proposal felt super messy.

I'll close this issue, then :) An example below for anyone who happens to come across this issue.
    public function isScopeDeprecated(Scope $scope): bool
    {
        if (!class_exists(DeprecationHelper::class)) {
            return false;
        }
        $callStack = $scope->getFunctionCallStackWithParameters();
        if (count($callStack) === 0) {
            return false;
        }
        [$function, $parameter] = $callStack[0];
        if (!$function instanceof MethodReflection) {
            return false;
        }
        if ($function->getName() !== 'backwardsCompatibleCall'
            || $function->getDeclaringClass()->getName() !== DeprecationHelper::class
        ) {
            return false;
        }
        return $parameter !== null && $parameter->getName() === 'deprecatedCallable';
    }

Awesome! You can expect PHPStan 1.10.56 released in the next few days.

Thank you @ondrejmirtes, amazing as ever. I appreciate it!

Yay! Great work, and thank you @ondrejmirtes! <3

Released: https://github.com/phpstan/phpstan/releases/tag/1.10.56


Document GitHub Open Source Strategy for Catalyst Projects

Develop and document a GitHub Open Source Strategy for Catalyst Projects as part of F8 Automate Open Source Project Standards.

Added @stephen-rowan as Treasury Guild Organization member in order to add and design Projects

Potential Issues going forward

[ ] Will clients have access to Project Board or just Treasury Guild
[ ] What to do if they have their own Project system

Should Catalyst proposal texts be stored on a repo with a Open Source Licence

Resolved Issues

If the Client does not have an existing GitHub Org then we set one for them with a Co-ordination repo with a Project Board
Agree OS licensing with them
Archive the proposal text

Document this strategy in the Treasury Advisory Service GitBook
What Treasury Guild's approach to OS is
What this mean practically - licenses, repo location etc
Context - reference to Dirk and Fred's fireside chat - de-risking projects
An explainer for OS licences - Andre
How does OS effect collaboration and the culture of Catalyst

Useful links

https://www.youtube.com/watch?v=OnmWFxlG2GA
https://www.youtube.com/watch?v=8nLfGcLXK3A
https://www.youtube.com/watch?v=UMIG4KnM8xw

Created page in Treasury Guild Advisory Service on Open Source Standards
https://quality-assurance-dao.gitbook.io/treasury-advisory-service/educational-materials/open-source-standards

[ ] Produce material for GitBook OS section

How will this material change given the governance changes of the last 3 months

Resume work on https://github.com/treasuryguild/Treasury-Advisory-Service/issues/35#issuecomment-1249360428

[ ] Create F8 ABC of Governance content for this.

Andre posted an OS Licence bot - https://ora.sh/environmental-beige-hdpf/open-source-license-bot
Aiming for June 2023 to complete


Fully configurable tooltip function.

A couple of notes looking at the changes:

The user needs to know that they have to set d3-flame-graph-tip class (this can be considered a good feature as user can pick their own class -- but easier for users if documented in readme that this "default" class is available for use)
The tooltip is configurable, but not the label. But I'm not sure what your use case is for the label.

For my personal case, I don't care about the titles, only the tooltips, and since you allow to set arbitrary HTML it's perfectly suitable for me. Remembering to set direction, offset, and default class in addition is not a big issue for me since it's a one-time piece of code. So I think this case is also good for everyone if the intent is if not using tooltips, the users are forced to a fixed string formatting in title tag.
I do see one issue, I think if you set tooltip to a function, tip is set but tooltp was previously set to false, you'll get double tooltips again as you'll have both titles and tips. That could be fixed such that when setting tip, you also set tooltip to true. I added a "line comment" in the commit where I am talking about.

Great! I'll include the extra info in the README. For the label, it also affects the details content, under the graph, and the title. Will probably affect the search result list I have planned. Change is simple, so I'll include it too.
About the issue you mentioned, I can't repro it. Update is called and it "should" remove title if tooltip is set to either a function to "true". Can you share your example?

I'm setting tooltip to true either way. It's more readable.

@gillius just merged. Have a look and let me know what you think. ;-)

What I meant is tooltip=true inside of the typeof function check. If the user called (I realize this is contrived) .tooltip(false) and then called .tooltip(function) then tooltip would be false and tip would be set to something, so you'd get no tooltips. At the time originally I thought it would double up the tooltips, but I was wrong it actually would skip d3-tip entirely. In the latest merge that contrived example works, but now I don't think .tooltip(false) would work at all, because you always set it true (I haven't tried it -- just reviewing code)...

@spiermar btw, I checked out the other commits too, great job on everything and of course merging my earlier PRs is much appreciated. When 2.4.2 comes out we'll be able to use it to display a profile of code executed on each of our web app page loads!

What's 2.4.2 a version of?

That's a great question and I have no clue how I got to 2.4.2. I meant to say 0.4.2, since 0.4.1 is out, or whatever is the next release to be put up on Bower for d3-flame-graph.


Adf in key metrics

Description of changes:
Add link_to_ADF.ipynb to the key_metrics example. This required a little modification to generate_adf_config_file.py to account for the fact that off-line regridding happened in 092 (CESM output on the CAM-SE grid, and ADF needs fields on the FV grid to plot them)

[x] Have you followed the guidelines in our Contributor's Guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you made sure that the pre-commit checks passed (#8 in Adding Notebooks Guide)?
[x] Have you hidden the code cells (#8 in Adding Notebooks Guide) in your notebook?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you successfully tested your changes locally?

This may be something on my end, but when I ran this, everything was computed with no issue but an additional (undesirable) key_metrics directory appeared here: CUPiD/examples/key_metrics/computed_notebooks/key_metrics and thus I ran into some issues generating the webpage. I'll try starting with a clean version since it's possible something is outdated in my sandbox.

Two more questions to consider:

Should we add the ADF comparison table to the Link to ADF Output notebook?
Should we add TAUY to the variable list so that we generate a lat-lon vector plot? This should also add a second plot to the output in the notebook

Based on our conversation I think it sounded like these were two of the key items that users will want to look at, so I would think yes to both of those questions.

From 02cb0ce:

Looks good @mnlevy1981! I guess the only improvement would be to work on the number formatting. Thanks for doing this

eff58b9 adds a row to the table and the vector map:

Based on our conversation I think it sounded like these were two of the key items that users will want to look at, so I would think yes to both of those questions.

Done!

Looks good @mnlevy1981! I guess the only improvement would be to work on the number formatting. Thanks for doing this

Good catch! That's probably an option in the pandas df.to_html() function... I'll look closer at the documentation...

I found the ADF call to to_html(), and added float_format='{:6g}'.format to my argument list. In 5c20647, it looks like:


Allow spawning environments from multiple vaults

After having some out of band discussions with folks, I'm not convinced we should implement this feature right now. Closing for the time being.


[iOS 12.2] Crash in first run

I think is problem of compare with new iOS version
Crash log
Swift class extensions and categories on Swift classes are not allowed to have +load methods

I'm pretty sure it's related to this issue: https://github.com/facebook/react-native/issues/24139

I'm pretty sure it's related to this issue: facebook/react-native#24139

yep, i think that there will be a fix shortly
https://github.com/facebook/react-native/pull/24155

This is fixed in 0.59.3

This is fixed in 0.59.3

So old project must be update to RN 0.59.3?

For Xcode 10.2 yes. https://github.com/facebook/react-native/releases

Hello,
I've the same error but also old libraries that doesn't work with react native 0.59.3 :(
The maximum version is 0.58.5 :(
What can I do?
I've tried to install version 1.1.4 but still same error :(
Thanks!

Any news for me? :( ?

how to solve? i use rn0.52


Create interfaces

Is your feature request related to a problem? Please describe.
None
Describe the solution you'd like
As an user, I need to create and update interfaces
Describe alternatives you've considered
None
Additional context

Feature complete in "feature/forms_interface". Postponing PR/merge until demo of application is complete.


Description
Start  the project by creating login and register pages
Suggested Solution
create using nextjs server side features
Alternatives
No response
Additional Context
No response

@Bhanu-code Done !
Please merge it

Could you please assign me the task of creating the login and registration pages for the project? I'm ready to get started on building these user authentication components.

@Jasmeetkaur-13 The pages already made , if you can please make the backend of signup and registeration and make it work

Ok, I'll check it out.
On Wed, Oct 16, 2024, 11:37 AM Anurag Maurya @.***>
wrote:

@Jasmeetkaur-13 https://github.com/Jasmeetkaur-13 The pages already
made , if you can please make the backend of signup and registeration and
make it work
—
Reply to this email directly, view it on GitHub
https://github.com/Bhanu-code/OpenLMS/issues/22#issuecomment-2415811575,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCFBXWQPWOXMCXBKSUTZOH3Z3X7AVAVCNFSM6AAAAABPYAIYLGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJVHAYTCNJXGU
.
You are receiving this because you were mentioned.Message ID:
@.***>


dds_write crash in ThreadSafePolice

Required information
Operating system:
E.g. Ubuntu 16.04

crash when send chunk
cyclonedds link
https://github.com/eclipse-cyclonedds/cyclonedds/issues/1481

@sgf201 thanks for the bug report. For the time being I would wait until the cyclone dds side has made some debugging and analysis before proceeding with this bug report. If it turns out that this originated really in iceoryx I would take it from there but in my opinion the best approach is to analyse it from top to bottom and start with cyclone dds.
If nothing is happening in the next week please ping me again then I would try to look at it.

@sgf201 do you have the output from the terminal where RouDi runs? I think the application did not send the heartbeat in time and RouDi might have assumed that the application died and cleaned up the resources which invalidated a mutex. You can start RouDi with -m off to disable the monitoring. In this case the heartbeat signal is not evaluated and the cleanup only happens when RouDi shuts down or when the exact same application tries to register at RouDi again.

@sgf201 do you have the output from the terminal where RouDi runs? I think the application did not send the heartbeat in time and RouDi might have assumed that the application died and cleaned up the resources which invalidated a mutex. You can start RouDi with -m off to disable the monitoring. In this case the heartbeat signal is not evaluated and the cleanup only happens when RouDi shuts down or when the exact same application tries to register at RouDi again.

writer app stderr print those sentence.
a little confuse about cleaned up the resources which invalidated a mutex
I think the mutex instance is init by writer ,how could roudi which is running seperatly cleaned up the mutex?

instead of disable the monitoring,can I change the heartbeat time required?

do you have a similar log message in the RouDi terminal
2022-11-17 19:33:07.007 [Warn ]: Application iox-cpp-user-header-publisher not responding (last response 1537 milliseconds ago) --> removing it

This would confirm my theory.
Unfortunately the heartbeat timeout cannot be changed via parameter and one would need to change the source code.
There is a mutex in the publisher itself. It is usually uncontested, only when new subscriber are added there is contention.

do you have a similar log message in the RouDi terminal
2022-11-17 19:33:07.007 [Warn ]: Application iox-cpp-user-header-publisher not responding (last response 1537 milliseconds ago) --> removing it

This would confirm my theory.
Unfortunately the heartbeat timeout cannot be changed via parameter and one would need to change the source code.
There is a mutex in the publisher itself. It is usually uncontested, only when new subscriber are added there is contention.

do you accept this judgment? I think there are two different sign between the two crash. 1.in 1643 it rease segment fault ,but in1790 it rease sigabort 2.in 1790 the writer is alive and do not plan to exit

@elBoberido

@sgf201 I do not have enough knowledge about the Cyclone DDS codebase in order to make any assessment. I think it is best to wait till that PR is merged and see whether the bug still occurs.


generated snippets seems to be old, not in line with the provider's current documentation

I have issues
Some generated snippets seems to be old, not in line with the provider's current documentation.
I'm submitting a

[x] bug report
[ ] feature request
[ ] support request

What is the current behavior
writing tf-oci-resource-core_vcn generates the code block below (pre-HCL2 syntax):
resource "oci_core_vcn" "test_vcn" {
    #Required
    cidr_block = "${var.vcn_cidr_block}"
    compartment_id = "${var.compartment_id}"

    #Optional
    defined_tags = {"Operations.CostCenter"= "42"}
    display_name = "${var.vcn_display_name}"
    dns_label = "${var.vcn_dns_label}"
    freeform_tags = {"Department"= "Finance"}
}

Whereas the current documentation have the following code block in the documentation: https://registry.terraform.io/providers/hashicorp/oci/latest/docs/resources/core_vcn
resource "oci_core_vcn" "test_vcn" {
    #Required
    compartment_id = var.compartment_id

    #Optional
    cidr_block = var.vcn_cidr_block
    cidr_blocks = var.vcn_cidr_blocks
    defined_tags = {"Operations.CostCenter"= "42"}
    display_name = var.vcn_display_name
    dns_label = var.vcn_dns_label
    freeform_tags = {"Department"= "Finance"}
    ipv6cidr_block = var.vcn_ipv6cidr_block
    is_ipv6enabled = var.vcn_is_ipv6enabled
}

If this is a bug, how to reproduce? Please include a code sample
In VSCode with the Terraform-doc-snippets extension enabled, type tf-oci-resource-core_vcn in a file with a .tf extension and compare the output to the current documentation.
https://registry.terraform.io/providers/hashicorp/oci/latest/docs/resources/core_vcn
What's the expected behavior
resource "oci_core_vcn" "test_vcn" {
    #Required
    compartment_id = var.compartment_id

    #Optional
    cidr_block = var.vcn_cidr_block
    cidr_blocks = var.vcn_cidr_blocks
    defined_tags = {"Operations.CostCenter"= "42"}
    display_name = var.vcn_display_name
    dns_label = var.vcn_dns_label
    freeform_tags = {"Department"= "Finance"}
    ipv6cidr_block = var.vcn_ipv6cidr_block
    is_ipv6enabled = var.vcn_is_ipv6enabled
}

Environment

Affected module version: 0.4.0
OS: N/A
Terraform version: N/A

Other relevant info

Updated them this week in response to your email. Sorry for the delay - working to make the code a little easier to operate regularly.


How Do I activate an installed Python Environment?

How can I activate a specific Python environment without creating a virtual environment? For example, I use: uv python list.
It shows that I have installed two versions of Python:

..\cpython-3.12.7-windows-x86_64-none\python.exe
..\cpython-3.8.20-windows-x86_64-none\python.exe

Now I want to perform operations directly on the 3.12 environment, what should I do?  What if I don't want to create a virtual environment?

If you install Python with uv, we intentionally don't support manipulating those managed Python installations -- we require you to create a virtual environment. You can probably do uv pip list --python  ..\cpython-3.12.7-windows-x86_64-none\python.exe, but it's not recommended.
If you're using some other Python installation, you can use uv pip install --system to opt-in to manipulating the system environment.

I tried to use uv pip install --python "cpython-3.12.7-windows-x86_64-none" pyside6, and it prompted me:
This Python installation is managed by uv and should not be modified
That is, this directory is not allowed to be modified. But if I want a continuously integrated Python environment, what should I do?
For example, if I use uv venv 312, it will create a Python virtual environment of 312, then can I use the uv command to activate this environment?
Now I want to install to this directory, I must use uv pip install --python "../312" pyside6.
I have to navigate to the specified Python virtual directory first, but I want something like this:
$ uv venv ../python/312 # Activate a Python virtual environment instead of creating one
$ Administrator in python/312/Scripts via 🐍 v3.12.7 (312)
# Now all commands are executed in the python/312 environment, and then I can easily use all uv commands, for example:
uv pip list
uv install pyside6
pyside6-rcc rec.qrc -o res.py
...

In this way, although I am not in the python/312 directory now, I can use the functions of the 312 virtual environment because I don't need pyside6 in my working directory.

I understand. I can indeed enter the python/313 directory and activate the virtual environment by executing activate.
However, if I want a Python environment for continuous integration, I have to create a new independent virtual environment instead of using the installed one.
Now uv does not allow you to modify the Python environment it installs, but it is also not possible to use it to create a python environment that is not managed by it. As long as you use uv install python, this environment cannot be modified.
For convenience, I should not download a system Python environment separately. I want uv to help me download it, but I also want this environment to be modifiable and accept continuous integration.
Similar commands:
# Specify system to install this python in the system directory, which is not subject to uv,
# Can be modified by commands such as `uv pip install`.
uv install --system python 3.13

Then I can specify the installation directory:
uv venv python 3.8 --system # Activate 3.8 system directory
uv venv python 3.12 --system # Activate 3.12 system directory

Otherwise, if I want a continuous integration Python environment, using uv, I need to create a separate virtual environment. If so, if I use 3.8, 8.12, I will install two more virtual environments for continuous integration.

I'm having some trouble following. Are you referring to a continuous integration pipeline, like on GitHub Actions? Or continuous development on your local machine? I'd suggest using a virtual environment in both cases.

It can be said that it is a global Python environment, which is separate from the virtual environment of the workspace, but some packages in this global environment may be used frequently, so they are directly installed in the global environment.
As I said, if I use several Python versions for development, and if the Python version I use is managed by uv, then in order to make such a global environment, I have to create two new Python virtual environments.
Since uv allows the modification of the system's Python environment, and the system's Python environment can also be managed by uv, right? Then why not add a function that allows downloading system Python environment? Because now I can only download an independent python environment by myself, and then use uv to manage it?
Can't it be more convenient? Allow me to download the system Python environment directly, which I don't want uv to constrain it.


datepicker doesn't open to correct month for non US date format

Bug, feature request, or proposal:
Bug
What is the expected behavior?
When reopening the date-picker, it should always open on the selected month
What is the current behavior?
If a date can be parsed as mm/dd/yyyy it will open at the month in that format, even if though it has correctly parsed the date and has the correct date highlighted if you scroll to the correct month
What are the steps to reproduce?

Select a date with the date picker that can be parsed as mm/dd/yyyy e.g. 10th July 2017 (10/07/2017).

Re-open the date picker, see that it is open on October, not July.

Scroll to July and see that the correct date is highlighted, so date picker has correctly parsed the date at some point, just not when deciding which month to open.

Providing a Plunker (or similar) is the best way to get the team to see your issue.
Plunker template: https://goo.gl/DlHd6U
This behaviour can be seen in the angular material examples: https://material.angular.io/components/datepicker/examples
What is the use-case or motivation for changing an existing behavior?
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
angular 4.2.4
material 2.0.0-beta.8
Windows 10, Chrome 59.0.3071.115
TS 2.4.1
Is there anything else we should know?

If faced the same issue when trying to use the datepicker with the Dutch date format (which is dd-MM-yyyy). When opening the date picker with a date from a previous month, it opened on the current month instead of the month of the selected date. Scrolling to the previous month showed the selected date correctly (highlighted), as you stated above.
My fix was to the set the LOCALE_ID to the Dutch language in app.module.ts:
providers: [   { provide: LOCALE_ID, useValue: 'nl' } ]
After that, the datepicker opens with the month of the date (instead of the current month), showing the highlighted date correctly at once.

Thanks for the comment and workaround for this Peter. I've seen similar workarounds, however I feel this should be addressed as a bug as the date picker clearly knows how to correctly parse the date in the correct browser without the work around, however it seems to parse it again when deciding which month to open, rather than opening the month of the date that it currently has highlighted.

+1, please do address this. The workaround proposed by petertangelder didn't unfortunately work for me. No idea why, but I'm using Estonian locale.

There's a similar workaround for setting locale which does work (as opposed to providing a LOCALE_ID in AppModule) in issue #4832, however this does not resolve the problem of mm/dd/yyyy vs dd/mm/yyyy.
It would be superbly awesome if somebody suggested either an alternative solution so that when the datepicker input shows 01/02/2000, the date opened is 1st February, not 2nd January, as it does now.

Take a look at this: https://plnkr.co/edit/uPNESrA5VbKDrIjP9BaS?p=preview
It's configured to Brazilian Portuguese and I think it's a workaround for all the issues pointed here.

Any updates on this? Was about to report the same bug, just before i noticed this issue.

I think it's being handled over there : #6785 , with the PR #6798

FYI #6798 makes the breakage less noticeable because it removes unnecessary calls to DateAdapter.parse, but it does not fix the problem. If the user manually types input it will still need to go through DateAdapter.parse and unless you're using a DateAdapter that can handle dd/mm/yyyy format it will not be interpreted correctly.
In order to make this work correctly you will need to use a custom DateAdapter that can parse this date format (@julianobrasil has an example above), or wait for the official MomentDateAdapter which is being worked on now (#6860).

Closing this since MomentDateAdapter has been added now


Bug fix/fix undeterministic traversal test

Scope & Purpose
Fix a test that fails undeterministically.
The test runs a DFS traversal with an undeterministic result. The asserts in the tests didn't sufficiently account for that.

http://jenkins.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/7672/

LGTM. Jenkins green. Ready to merge.


Custom Vehicle Head Lights and Adjustable vehicle light parameters through Python API as future features

Hi, Thank you for developing a great simulator.
I am interested in utilizing CARLA specifically for car headlights simulation with MATLAB/Simulink models.
In order to imitate ADB(Adaptive Driving Beam) and AFS(Adaptive Front Lighting System) in CARLA, I'm very happy that if CARLA has a feature such as adjusting the head lights parameters like the Light intensity through Python API as same as the street light does. Can I achieve this by editing the c++ codes before building the carla?
Do you plan to add such a feature for a future release?

Also, maybe it is already possible but I would like to add head lights onto the custom vehicle added to carla and make it turn on/off through python API during the simulation . If it is possible, I would appreciate if anyone helps me to do this.

The Python API only lets you change the light state (turn on/off low beam, high beam, fog lights, etc). AFAIK you have to write your own C++ code to change the intensities.
You can add lights to a custom vehicle (I've done it successfully for a Mustang I imported), the full instructions are in the tutorial for adding a new vehicle but basically, you first need to create a mesh shaped like the lights, then apply the proper UV texture, export it to .fbx, import it in UE4, apply the light material to the mesh, add the mesh to the blueprint and tag it as emissive, and finally add spot lights similar to the ones in existing blueprints.

Hello! You can follow the documentation to add new lights to your vehicles:
https://carla.readthedocs.io/en/latest/tuto_content_authoring_vehicles/#lights
Regarding the lights intensity, for now, you can only change the streetlights, if you want to change the vehicle intensity, you should modify the vehicle blueprint, rightnow you can't use the api for that, sadly.

I'm closing the issue, since the problem seems solved.
Thanks for your time.


Dynamic Port Access Without Use of CLI

This change modifies the SonicPi class so that it will find the
dynamic port that Sonic Pi is running on. By doing this, you can
instantiate SonicPi in irb or other ruby code.
The existing change to find the dynamic port would execute when running
sonic_pi from the command line tool; this extends that availability to
be wherever the SonicPi class can be used.

Thanks for this, just had to move the regex over as well but no troubles aside from that.
Just released v0.2.0 with these changes 🙂

Thank you for catching that. Sorry I missed it.


feat: create release and add APK to CI release

Whenever we build APKs for main, we create a new release with the date format and attach the APKs
Untested... YOLO.

Had a quick chat with @luckysori : we created a separate repository for this purpose: https://github.com/get10101/10101-test-apks . We will create the releases there to not pollute this repository.


[Nerdpack] Dashboard and runbook final touches

Once the nerdpack is deployed to production we can add some final touches to our dashboard and runbook
Acceptance Criteria

[x] Update nrql queries in dashboard with production env (currently they query for any tessen events)
[x] Update feature flag list in the runbook with production and EU flags
[ ] Update any feature flags in the runbook with changed names if applicable
[ ] Consider if we want more nuanced error charts in the dashboard

Product Feedback

[ ] ensure widgets focused on usage are added to the global Observability Packs: Marketplace Usage Dashboard https://staging-url-here/-/0GbRmvn8Vjy
[ ] any widgets added to this dashboard for the internal usage should be labels internal:measurement
[ ] any existing widgets that measure external data should be renamed external:measurement
[ ] name the new dashboard used for alerting and troubleshooting Observability Pack UI: troubleshooting https://staging-url-here/-/0PoR8nNLoQG

@jpvajda doesn't look like the first dashboard exists anymore, do we have a link to the new one?

@rudouglas I don't know what dashboard this used to point to, let's ask @LizBaker which one she was referring to in standup today. 🅿

My opinion here is since we are handing this off to devex, let's put it in their primary DB called [DevEx] New Relic I/O

@jpvajda @rudouglas  we have a dashboard called [DevnEn] I/O Quickstart detail that has a couple error widgets and a couple usage metrics widgets. I feel like we can call this good, and any other usage metrics widgets folks want in the big dashboard can be put in there. I feel like we can close this ticket


Losing connection sometimes crashes PicoTorrent

Most of the time, if and when I lose my internet connection, PicoTorrent crashes.
Also, torrent progress is lost when this happens. Sometimes it removes the torrent completely, other times it just reverts back to lower progress.

Very odd - do you have a reliable way of reproducing this?


KAFKA-17756: Added dynamic share group configs share.heartbeat.interval.ms and share.session.timeout.ms

This PR adds the 2 configs share.heartbeat.interval.ms and share.session.timeout.ms in GroupConfig. These can be dynamically set for a share group without restarting the server

@chirag-wadhwa5 There's a relevant test failure:
FAILED ❌ KafkaApisTest > testDescribeConfigsConsumerGroup()


Reorganize keys into projectSettings and buildSettings

Also adds tests to make sure project works as expected.

@jrudolph Would it be possible for you to look into this wrt the designation of projectSettings vs buildSettings vs globalSettings? Additional clarification was provided at https://github.com/sbt/sbt/issues/3652#issuecomment-1429898230 but I want to make sure that I am on the right track and I think you have a lot more experience with sbt plugins

So I am going to go ahead and merge this, at least the important thing regarding globalSettings has been clarified.


Support for ABAPGIT after SSO enablement of enterprise github

By the end of January 2020, our organisation is enabling SSO for the enterprise version of github. I use ABAPGIT for pushing code to the repositories created in the enterprise version of github.
How can I adapt or what configurations are required in SAP GUI to enable SSO verification?
Thanks in advance.

Need to mention that our SSO is based on private SSL certificates.

Need to do something like described in https://help.github.com/en/github/authenticating-to-github/authorizing-a-personal-access-token-for-use-with-saml-single-sign-on
I dont have access to any SSO/SAML setup, so its difficult to test for me

Is there a way how to get my client SSL cert from Operating system (Windows) without the need to save it in STRUST?

Our start screen opens an URL which does SSO autohrization too. By accident, I have 2 SSO certs and  SAPGUI asks me to single out one. So, I guess there must by some way how to get the cert also for abapGit.

pull requests welcome, its difficult to test and build without a SAML setup

I just tried https://docs.github.com/en/github/authenticating-to-github/authenticating-with-saml-single-sign-on/authorizing-a-personal-access-token-for-use-with-saml-single-sign-on and it seems to work in the setup I have access to

closing


Pre-trained model on MetFaces Dataset

Thanks for sharing your code, could you please share the pre-trained checkpoints trained on MetFaces dataset?

You can get a different met faces model (trained with the original StyleGAN, but you should be able to convert it) here: https://github.com/justinpinkney/awesome-pretrained-stylegan2/#painting-faces

You can use this checkpoints: https://drive.google.com/file/d/1g6URSv_0K3G0hy6s7ZLW5Sb1Bpaqd6CE/view?usp=sharing
But it is the results from my intermediate experiments with non leaking augmentations, and without FFHQ pretraining. So the result will be not very optimal. (For examples, I found "leaking" of augmentations behavior.)

Thanks for the replies!


Better performances and thread compliance

Better performances and thread compliance with publisher
RosConnector can now be inherited from in an easier way

Thank you @flonou . Theese changes make a sense.
UnityPublisher also becomes similar to UnitySubscriber.


[SPARK-47511][SQL][FOLLOWUP] Rename the config REPLACE_NULLIF_USING_WITH_EXPR to be more general

What changes were proposed in this pull request?

With is not only used by NullIf, but also Between. This PR renames the config REPLACE_NULLIF_USING_WITH_EXPR to be more general, and use it to control Between as well.
Why are the changes needed?

have a conf to control all the usages of With.
Does this PR introduce any user-facing change?

no, not released yet
How was this patch tested?

existing tests
Was this patch authored or co-authored using generative AI tooling?

no

cc @kelvinjian-db


Add keyboard shortcut for “Add to Reading List”

I largely triage NNW using keyboard shortcuts (A/Z K/L). One of the few things that requires me to take my hands off the keyboard is sending something to Reading List. Would be cool to have a keyboard shortcut for this.

@karan301 What do you mean by "Reading List"?  Are you using a read later service like Instapaper?

@vincode-io Sorry, should have specified, Safari Reading List (Share → Add to Reading List).

In Mac, you can add your own shortcuts from system preferences > keyboard > shortcuts

I did not know that, though I’ve pretty much only used an iPad for like 6 months now. 😅 Would love to see it there!

@specter78 What command should I add a shortcut to? I don't see an "Add to Reading List" menu item in NNW. I tried adding Control-S to 'Instapaper' as seen in the Share menu, but it doesn't work -- presumably because the Toolbar's Share menu is not a proper menu to the system Keyboard Shortcuts feature.

Interestingly I see ^S by Instapaper in the Toolbar menu, but hitting Control-S doesn't do actually anything.


polyhedral simulation example mesh is bad

Yields an incorrect meshLooks like the 'polyhedral' zone in the middle is connected incorrectly.
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 1875
Status: Resolved
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: polyhedral simulation example mesh is bad
Assigned to: Kathleen Biagas
Category: -
Target version: 2.8
Author: Kathleen Biagas
Start: 06/13/2014
Due date:
% Done: 100%
Estimated time: 4.00 hours
Created: 06/13/2014 01:45 pm
Updated: 06/26/2014 05:28 pm
Likelihood: 3 - Occasional
Severity: 4 - Crash / Wrong Results
Found in version: 2.7.1
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
Yields an incorrect meshLooks like the 'polyhedral' zone in the middle is connected incorrectly.
Comments:
Tried stepping through the tesselation code, but got lost. M /src/avt/Database/Database/avtPolygonToTrianglesTesselator.C /** preserve node ordering, was commented out /M /src/databases/SimV2/SimV2GetMesh.C / send the face normal to the tesselator **/


Add basic structure to support multiple locales in CV

This PR is to set the basic skeleton of Internationalization, UX and coverage need to be improved in further PRs

Coverage increased (+0.05%) to 84.783% when pulling 5c81a71a1c359f68b4b78faa80b2de5dc8cee871 on stories/i18n into de4b5790f84cf8b14fd263911f1cfdbe40fbcec3 on master.

Coverage increased (+0.05%) to 84.783% when pulling 5c81a71a1c359f68b4b78faa80b2de5dc8cee871 on stories/i18n into de4b5790f84cf8b14fd263911f1cfdbe40fbcec3 on master.


Not able to run F# Core MVC projects

Version: 9.31.2

.NET Core Version: 3.1
Node.js Version: 12.13.0

Target: Windows (10)

First of all, this is a very interesting project that I'm excited to experiment with for building small projects.
The application doesn't work for F# ASP.NET Core MVC applications, or Blazor (using Bolero). While electronize init throws an exception due to only .csproj projects being searched, manually filling the fields of the manifest file to match an equivalent C# project caused electronize start to hang just after outputting stdout: Use Electron Port:
Steps to Reproduce:

Run dotnet new mvc -lang F#
Add ElectronNET project with dotnet add package ElectronNET.API
Add webBuilder.UseElectron(args) |> ignore to Program.fs
Add async { do! Electron.WindowManager.CreateWindowAsync() |> Async.AwaitTask |> Async.Ignore  } to Startup.fs
Run electronize init, manually correct "executable", "name", "apiId" and "projectName" to match equivalent C# project
Add CopyToOutputDirectory for electron.manifest.json in *.fsproj
Run electronize start

I realised that step 4 didn't actually run the async code, so it should be:
async {
    do!
        Electron.WindowManager.CreateWindowAsync()
        |> Async.AwaitTask
        |> Async.Ignore
} |> Async.RunSynchronously

I'll make a pull request to fix the electronize init issue with .fsproj projects.


Add VAT number examples

Can be used on placeholder or error message.

Usage:
import { getVATExample, austria } from 'jsvat';

getVATExample(austria) // ATU12345678

Based on Stripe examples https://stripe.com/docs/billing/customer/tax-ids
These numbers are not valid, only the format

Since this repo seemed so abandoned I've created a fork on:
https://github.com/mathieumagalhaes/jsvat-next
I was wondering if the community wants this change in this library, or if this is out of scope.


Implement async SpiBusDriver

Starts fixing #162 .
This PR adds a SpiBusDriver that takes exclusive access to a SpiDriver and provides the SpiBus trait.
Async SpiDeviceDriver will come in future commits or another PR.
This is draft atm as it is untested and IRAM attributes haven't been specified everywhere it's needed.
The example isn't complete as I haven't looked into how to run it yet.
If you want to comment on something and there's a TODO comment about it, hold on to your comment until the TODO is gone.

@Dominaezzz What is the status of this PR? It is still listed as "draft"?

Isn't it better to both of us work on the async-drivers branch, as that is what will land soon? As in, if you could help me see the Operation alternative implementation rather than me doing it, and also if you can double check your idea with the non-global Notification does work?

My last push was just to try out the pseudo code I showed you for putting the notification on the stack.
For some reason esp-idf-sys won't build locally so I'm using CI to test.


Suggestion of Custom Splitting and Nuetralisation

Im working a Scaffold based project and it involves a lot of custom bond splitting.
I currently have a molecule " *C1CC(=O)c2ccccc2C1 "

I tried filling that * location with a Hydrogen atom in order to complete the valency using this piece of code
>> mol.GetAtomWithIdx(n_at-1).SetAtomicNum(1)

You can see that an extra Hydrogen atom is added but for some reason when I convert it to smile string it explicitly shows the Hydrogen atom - "[H]C1CC(=O)c2ccccc2C1"
Are there ways to neutralize or complete the valency of a broken bond with trivial molecules like Hydrogen?

@VIGNESHinZONE That works. You can do something like:
for a in mol.GetAtoms():
    if (a.GetAtomicNum() == 0):
        a.SetAtomicNum(1)
        a.SetIsAromatic(False)
        a.SetIsotope(0)

(depending on the method you use to cleave bonds aromatic flags or isotopic labels can be left on the dummy atoms, this is why I reset those)
Finally, convert the hydrogens in the molecule graph to implicit:
mol = Chem.RemoveHs(mol)

and you'll have accomplished your goal.

Oh great, thanks a lot!


[Bug]  PyTorch and TVM loading problem due to conflicting LLVM symbols

Apparently, the new PyTorch release crashes with symbols loaded by TVM, so the following trivial code crashes with invalid pointer Aborted (core dumped) upon exit:
import tvm
import torch

We can workaround this by swapping the import order, but as pointed out in https://github.com/apache/tvm/issues/9349#issuecomment-950685224 this may not always be possible.
Another solution is to remove the use of RTLD_GLOBAL in https://github.com/apache/tvm/blob/dfe4cebbdadab3d4e6e6ba3951276a51a4ffeaf6/python/tvm/_ffi/base.py#L57
See related issues in other repos that moved away from using RTLD_GLOBAL.
https://github.com/dmlc/dgl/issues/2255
https://github.com/pytorch/pytorch/pull/28536
https://github.com/pytorch/pytorch/issues/3059
Is there any particular reason we are using RTLD_GLOBAL? @tqchen @areusch

OK, digged a bit into this. I think I know the possible cause. This is because of the conflict of LLVM symbols(due to different versions of LLVM being used). PyTorch also starts to ship with LLVM. To avoid the problem, we need to do two things

Turn on static linking of LLVM, this will directly link llvm code into libtvm without relying on dynamic library (that creates global symbols)

set(USE_LLVM "llvm-config --link-static")

Turn on set(HIDE_PRIVATE_SYMBOLS ON). This will effectively hide the LLVM related symbols when we load globally from pytorch.

I did a quick experiment locally and when we turn both options ON, things are good, and there will be conflict with either option off.

Thanks @tqchen, I confirmed that your solution worked on both of my envrionements too, and also both static link and HIDE_PRIVATE_SYMBOLS are required.
Also I realized that when I said "I cannot reproduce the original failure anymore" in https://github.com/apache/tvm/issues/9362#issuecomment-951574017, my cmake config is pointing to a different, custom LLVM build that has only static libs. Moreover, apparently these custom libs were built in a way that HIDE_PRIVATE_SYMBOLS doesn't need to be enabled.
So no mystery on my end anymore.

@tqchen  I modified the CMakeLists.txt,
tvm_option(USE_LLVM "/path/to/llvm-config --link-static" ON)
tvm_option(HIDE_PRIVATE_SYMBOLS "Compile with -fvisibility=hidden." ON)
But I still found the bug "free(): invalid pointer",

@Jie-KUN you need to set those configurations in config.cmake instead of CMakeLists.txt

@tqchen , thank you sincerely. I still have a question that I tried the code "from_pytorch.py". But I always found the tips:
"One or more operators have not been tuned. Please tune your model for better performance. Use DEBUG logging level to see more details."
Is that normal?

Yes that's normal. Please post other questions to the discuss forum.

@masahi  Ok， thank you！

cc @leandron @areusch for awareness, let us update tlcpack config

* Turn on static linking of LLVM, this will directly link llvm code into libtvm without relying on dynamic library (that creates global symbols)
  * `set(USE_LLVM "/path/to/llvm-config --link-static")`
* Turn on `set(HIDE_PRIVATE_SYMBOLS ON)`. This will effectively hide the LLVM related symbols when we load globally from pytorch.

Thanks for letting us know. It seems that currently, --link-static is already there in tlcpack. I added https://github.com/tlc-pack/tlcpack/pull/81 for the workaround discussed here.

reopen, can be closed once we updated the installation guide


Can I use storage server like ceph

Hello dear,
vCloudStack MGMT : <IP_ADDRESS>
CloudStack install : Centos 7.7
Compute Nodes : Centos 7.7
Virtualization host : kvm
ceph install : Centos 7.8
Can I use Storage Server like ceph for my private cloud. Please see attachment for my design.

Thanks.
@alamintech

@alamintech please use the users mailing list (http://cloudstack.apache.org/mailing-lists.html) to ask questions, only report issues related to Primate on this repo. Closing on this remark.


Rollup 3.x.x

This plugin is not compatible with Rollup 3.x.x:
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/rollup
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer rollup@"1 || 2" from<EMAIL_ADDRESS>npm ERR! node_modules/rollup-plugin-css-only
npm ERR!   rollup-plugin-css-only@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /home/sergio/.npm/eresolve-report.txt for a full report.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/sergio/.npm/_logs/2022-10-17T16_03_05_981Z-debug-0.log```

Rollup 3 support has been added in v4 🎉


makes expiration optional for request endpoints

Overview
Fixes #557. This makes expiration optional for any requests that are created.
Description
Changes a default behavior according to the issue.
How Has This Been Tested?
Unit and integration tests pass.

Codecov Report

Merging #568 (226f435) into main (df7b007) will increase coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #568      +/-   ##
==========================================
+ Coverage   23.93%   23.95%   +0.02%
==========================================
  Files          46       46
  Lines        5311     5305       -6
==========================================
  Hits         1271     1271
+ Misses       3806     3800       -6
  Partials      234      234

Impacted Files
Coverage Δ

config/config.go
26.72% <ø> (ø)

pkg/server/router/manifest.go
7.80% <0.00%> (+0.13%)
:arrow_up:

pkg/server/router/presentation.go
2.54% <0.00%> (ø)


Linter is not working for Android libraries

Project has 2 modules - one is main application module and one is android library - linter tasks are created for library module, but doesn't report any issues in Kotlin files. I've checked by introducing "Needless blank line(s)" lint error.

Thanks for bringing it to my attention Yahor. I haven't tested against more complex android projects so I might be missing some things. I'd like to figure out all the cases though and get them working!
Can you paste the output from gradle tasks --all | grep lintKotlin from the top of the project? This should help me see which sourceSets are being detected in the library module if any.
Are the lint tasks applied correctly to the source in the application module?
Can you run gradle -d lintKotlin | grep linting? This should show each file that is being detected and linted by the tasks. Let me know which source dirs it appears to be picking up and which ones it is missing.
Thanks for your help!

I've found the issue and have a pull request to fix. Thanks for the report and I don't need any further info from you!

https://github.com/jeremymailen/kotlinter-gradle/pull/8

Should be fixed in 0.9.3. Please give it a try and let me know if it works for you.


Timestamps completely wrong

This is all from 1 dsr pull and everyone died in a ~30 second time interval
https://i.imgur.com/tSU9jzI.png

Only appeared after today's update, forgot to clarify that

Oh I do not have timestamps enabled, I'll look into that


feat(ci): set motd based on final workflow status

Description
Set/unset motd based on workflow final status:

success() check if every preceding step succedded
failure() check if any of the preceding steps failed

Based on those conditions, now:

set host as free so it can be reused
set host as failed, maybe util for debugging. [*]

[*] discussion to follow: should we do anything more if the check fail? Maybe gather logs, mark the runner as inactive until debug is performed, etc.
Type of change
Please select the appropriate options:

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

Checklist

[x] I have performed a self-review of my own code
[ ] If a change is adding a feature, it should require a change to the README.md and the review should catch this.
[ ] If the change is a fix, it should have an issue. The review should make sure the comments state the issue (not just the number) and it should use the keywords that will close the issue on merge.
[ ] A change should not be merged unless it passes CI or there is a comment/update saying what testing was passed.
[ ] PRs should not be merged unless positively reviewed.

The test failed due to mirror issues and actually that was the best that could happen on this case.
The workflow skipped unsetting the motd and finally set a motd acknowledging the failure.

:tada: This PR is included in version 1.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Debug log has state class total_increasing, but its state is not strictly

Describe the bug
A clear and concise description of what the bug is.
2024-10-27 16:45:10.537 WARNING (Recorder) [homeassistant.components.sensor.recorder] Entity sensor.ward_coysman_total_current_day_energy from integration fusion_solar has state class total_increasing, but its state is not strictly increasing. Triggered by state 16.73 (16.76) with last_updated set to 2024-10-27T16:43:07.089449+00:00. Please create a bug report at https://github.com/tijsverkoyen/HomeAssistant-FusionSolar/issues
** Debug information **
Answer the questions below before submitting a new issue.

Are you using the latest version of Home Assistant?: yes
Are you using the latest version of this integration? yes
Have you looked into the closed issues and did not find a similar issue. : yes
Are you using Kiosk or Northbound / OpenAPI mode?: OpenAPI mode

This is related to Huawei returning faulty data. I can't do anything about it.

Hi,
We are also encountering the same warning message about the daily energy not being strictly increasing.
Could it be possible that Huawei is subtracting self-consumption from the total generated energy? This might explain why the sensor occasionally shows a decrease. I’ve also observed that Yield Today remains at 0.0 kWh for an unusually long time in the mornings, even when the PV system starts generating power.
Since the detection logic seems to me a part of Home Assistant Core and cannot be modified in the integration, would it be possible to adjust the sensor’s classification to one that supports non-strictly increasing values? This might better reflect the data behaviour without triggering unnecessary warnings.
What do you think about this approach?
Kind regards,
Jordi


bagit creation: investigate multi-threaded zip generation

Currently the use of only one CPU core appears to be a bottle neck, we should look into this a bit more and see if or how it can be improved.

I think this would still be nice to have.
Since we use autoscaling for the bagit workernodes, this is probably not a high priority. It might be worth estimating the needed time though.


add video files

add them to the project
Optimise their load

https://medium.com/canal-tech/how-video-streaming-works-on-the-web-an-introduction-7919739f7e1


Replace ellipsoidal_to_cartesian with ERFA function

Our ellipsoidal_to_cartesian is exactly equal to gd2gce from SOFA/ERFA in one simple test I did:
In [1]: a, b = 6378137.0, 6378137.0

In [2]: lat, lon = 58, 25  # Arbitrary

In [3]: f = (a - b) / a

In [4]: from poliastro.czml.utils import ellipsoidal_to_cartesian

In [5]: from astropy import _erfa as erfa

In [6]: x, y, z = ellipsoidal_to_cartesian(a, b, lat, lon); x, y, z
Out[6]:<PHONE_NUMBER>635349, -100606.81643041565, 6332677.773035965)

In [7]: x, y, z = erfa.gd2gce(a, f, lon, lat, 0); x, y, z
Out[7]:<PHONE_NUMBER>635349, -100606.81643041565, 6332677.773035965)

@Juanlu001 I would like to work on the issue. Basically gd2gce converts geodetic coordinates to geocentric coordinates for a reference ellipsoid.
I have to just replace function definition in czml/utils.py by
return erfa.gd2ce(a,(a-b)/a,lon,lat,0) right? Should we keep the comments inside function same?

Hi @abhijeetmanhas, thanks for your interest in poliastro! Yes, only changing that should be enough.

pyerfa was splitted! https://github.com/liberfa/pyerfa

Juan, I can take this one as I think it was left.
We currently don't have PyERFA as a dependency, since it split from Astropy. Do you think it is ok to add it?

@vmos1 noticed that, for a != b, the results are not the same!
In [23]: lat, lon = 58, 25

In [24]: a, b = Earth.R.to(u.m).value, Earth.R_polar.to(u.m).value

In [25]: x, y, z = ellipsoidal_to_cartesian(a, b, lat * u.deg, lon * u.deg); x, y, z
Out[25]:
(<Quantity 3055830.60159627>,
 <Quantity 1424957.21154874>,
 <Quantity 5432271.27401853>)

In [26]: x, y, z = gd2gce(a, f, (lon * u.deg).to_value(u.rad), (lat * u.deg).to_value(u.rad), 0); (x * u.one, y * u.one, z * u.one)
Out[26]:
(<Quantity 3063227.38217292>,
 <Quantity 1428406.38697732>,
 <Quantity 5408966.60065562>)

This turned out to be not so trivial, since our code assumed geocentric coordinates. A fix is in progress in #995.

Fixed in #995.


Simplified syntax for the 'name' parameter decorator

Is your feature request related to a problem? Please describe.
The current authoring experience for Bicep parameters requires you to specify the 'name' metadata property using the full syntax. In contrast, you can use a much shorter and cleaner approach for the' description' property. For example, if I want to define both the 'name' and 'description' properties, the resulting syntax looks a bit awkward :
@metadata({
  name: 'Target management group'
})
@description('A management group where referenced policy definitions are deployed.')
param targetManagementGroup string

Describe the solution you'd like
It would be great to define the same using the following syntax:
@name('Target management group')
@description('A management group where referenced policy definitions are deployed.')
param targetManagementGroup string

Out of curiosity, where is this metadata.name property on the parameter consumed/displayed?

For example, it's used to display a human-readable name for an ARM parameter when doing template deployments.
If you compile the following bicep definition into an ARM template: https://github.com/andrewmatveychuk/azure.policy/blob/master/other-samples/initiatives/definitions/tagging/bicep/azuredeploy-to-management-group.bicep
Then deploy the resulting ARM template using the Template deployment on the Azure portal, you should see that the 'name' metadata property is displayed as parameter name:

I believe the actual name of the parameter, if camelCased, will be turned into a space-separated name when deployed via portal. Have you tried this?

Wow, that works! Thank you, @alex-frankel! I wasn't aware of such behavior. Is it documented somewhere?

@tiftzmac / @mumian - do you happen to know if we have this captured somewhere?

Should it also go in https://docs.microsoft.com/en-us/azure/azure-resource-manager/templates/template-parameters?tabs=bicep?

Updated 3 articles - https://github.com/MicrosoftDocs/azure-docs-pr/pull/157354

Docs should be published. Closing.


Enhancements for UID, TLM, found/lost, and distance

Includes #9 and the following:

TLM data is OR’ed with old, if present
events changed to discover, found, and lost
use simplified distance formula for now (same as examples in google/eddystone repo)
only emit lost when duplicates is enabled

@christopherdro @jacobrosenthal @ukBaz comments and feedback welcome ...

:+1: Just testing it out and it looks great! Awesome work.

Nice work @sandeepmistry @christopherdro

Nice! Thanks for taking the time for your great additions @sandeepmistry @jacobrosenthal


[New Metric] Specificity

🚀 Feature
In addition to Precision and Recall it would be nice to have a Specificity metric.
For the implementation I think it would be enough to make a copy of Recall (class und function) and adapt numerator and denominator in _precision_compute.
Alternatives
For binary classification Specificity is the same as Recall with 0 as true label.
For multiclass classification this is not as easy as this though.

@Borda , can I take up this issue?

@Borda , for the test cases, since there is no specificity metric in sklearn, the randomized test cases cannot be used. Should I just write some test cases with fixed values?


feat(api): add namespace

:tada: This PR is included in version 0.10.0 :tada:
The release is available on:

GitHub release
v0.10.0
v0.10.0

Your semantic-release bot :package::rocket:


Fix response in unified cache sample

Addresses #1558

Coverage remained the same at 79.833% when pulling 20823c2ac4595a5d813a232624d95477ef727755 on fix-sample into 920911e14f2bc5b146e1229162929f9ae1324860 on dev.


Docs review - batch 3

Hello !
Next batch of docs review 📝
Today, pages:

Column Spanning
Row IDs
Row Sorting

+ and minor modification of pages Column Pinning

Here is the changelog:
Column Pinning

Remove §Jump To & Pinning as this feature is not in the example (and I didn't find a way to include it for now)
(I thought I removed it in the last commit)

Column Spanning

Modify h3 titles by h2
Add §Column Spanning Simple Example (inserted before the existing example)

Add text from AG Grid docs
Add example for §Column Spanning Simple Example
Add css style in columns.css (*docs\assets* and *docs\examples\columns\assets*) and add code block before
example
Add js function in dashAgGridFunctions.js (*docs\assets* and *docs\examples\columns\assets*) and add code
block before example

Modify title "Example Column Spanning" to "Column Spanning Complex Example"
Add code blocks for css classes and js functions above the example §Column Spanning Complex Example:
Modify title of example §Column Spanning Complex Example
Fix little typo in comment of the example §Column Spanning Complex Example

NOTE:
While adding the js function in docs\assets\dashAgGridFunctions.js, I noticed (thanks Pycharm 😌) in the class
dagfuncs.DMC_Select, in the function React.createElement the arg style is def twice
style: {width: params.column.actualWidth} and style: params.style

Row IDs

Modify h3 titles by h2
Add some text formatting

Row Sorting

Modify h3 titles by h2
Add/modify text formatting
Fix example title §Example 1: Sorting

Based on AG Grid docs:

Add description of options:
dashGridOptions = {'suppressMultiSort': True} and dashGridOptions = {'alwaysMultiSort': True}
Add §Sorting Animation, §Sorting Order and §Example: Sorting Order and Animation
Add new Sorting Order and Animation example
Add *§Sorting API including new example
Add *§Accented Sort including new example
(I added a switch and the Row ID col to better show the difference)
Add *§Post-Sort including new example (with new js function put in docs\assets\dashAgGridFunctions.js and
docs\examples\columns\assets\dashAgGridFunctions.js)
(I just changed country === 'Ireland' by athlete === 'Michael Phelps', I think it is better for the example, as there are too many 'Ireland' rows blocked at the top, and he deserves it! 😁)

NOTE:
For §Sorting API, the example I added do the job, but I am not really satisfied with the save/restore of sort
state callbacks I created, but that's the best I found.
If anyone has a better idea that would be awesome!
However, the functions of the example from AG docs seem really simples...
They save the sort state like (as I did):
savedSort = [
    {
        "colId": "athlete",
        "sort": "asc",
        "sortIndex": 1
    },
    {
        "colId": "age",
        "sort": "asc",
        "sortIndex": 0
    }
]

Then to restore, they simply call:
.applyColumnState({
      state: savedSort,
      defaultState: { sort: null },
    })

But like in the Column Pinning example, I don't know how the columnState of Dash AG Grid is translated to AG Grid using .applyColumnState
To be continued... 😄

This all look great!  Thanks so much for the excellent edits and new examples.
:woman_dancing:


support search in factories and add whoosh as a store backend

#318 is related.

@abom if good to go please resolve the conflict and merge


Item position offset on move begin and end

Hi everyone,
Not a huge deal, but there seems to be an unwanted position offset when selecting/releasing an item. As seen on the GIF attached, the item moves upwards when the long press is detected, as well as when it is released (before moving to its final position). Any thoughts ?
This is the code from the example, running on an android phone through Expo.

+1

In my current app, if I've scrolled any, whenever the longPress activates, it sends the active item to the top of the screen, instead of remaining in place.
Dont know if this is the same issue, or a different one. Havent debugged it too much yet. :-/

If you're using react-navigation (especially with the IOS 13 Modal Presentation animation or the card animation), the calculated container offset will be incorrect as it's measured during the screen transition. I've fixed it by adding an InteractionManager.runAfterInteractions(...) around the code which measures the container offset.

If you're using react-navigation (especially with the IOS 13 Modal Presentation animation or the card animation), the calculated container offset will be incorrect as it's measured during the screen transition. I've fixed it by adding an InteractionManager.runAfterInteractions(...) around the code which measures the container offset.

Could you please write exactly where you added InteractionManager.runAfterInteractions(...)? I am using react-native-navigator and I have a similar offset problem when I start dragging.

This seems to be fixed in v2 for me

I'm using react-native-navigation 2.27.7 and react-native-draggable-flatlist 2.0.1


Add data types for notifications and improve lock methods

Add NcNotificationType enum
Add NcSubscriptionEndEventData type
Remove oid from wait methods

Reviewed on the call on the 14th of June 2022


Example with styled components and scoping

Description
Add styled-components example with scoping of playgrounds enabled

Awesome ! Thanks :)


Busboy 'field' never triggered

Hi, I am having this problem that the field or file functions never get triggered, and here is my code:
const Busboy = require('busboy');

exports.test = functions.https.onRequest((req, res) => {
    console.log("start");
    console.log(req.rawBody.toString());
    if (req.method === 'POST') {
        var busboy = new Busboy({ headers: req.headers});
        busboy.on('field', (fieldname, val, fieldnameTruncated, valTruncated, encoding, mimetype) => {
            console.log('field');
        });
        busboy.on('finish', function() {
            console.log('finish');
            res.json({
                data: null,
                error: null
            });
        });
        req.pipe(busboy);
    } else {
        console.log('else...');
    }
});

And the output of my code is:
Function execution started
start
--Boundary-43F22E06-B123-4575-A7A3-6C144C213D09
Content-Disposition: form-data; name="json"

{"name":"Alex","age":"24","friends":["John","Tom","Sam"]}
--Boundary-43F22E06-B123-4575-A7A3-6C144C213D09--
finish
Function execution took 517 ms, finished with status code: 200

As you can see, the file function was never triggered, how can I fix this?
Also, I have tried POSTMAN to test my function, and the output was the same except the boundary.

If you've already read the request data (as evidenced by the existence of a req.rawBody), then there is nothing for busboy to parse. You will need to either disable request body buffering or you will need to manually feed the buffered data to the Busboy instance via busboy.end(req.rawBody) instead of req.pipe(busboy).

Just updated my code like you suggested, and it still not working. Any idea? Here is my code:
exports.test = functions.https.onRequest((req, res) => {
    console.log("start");
    console.log(req.rawBody.toString());
    if (req.method === 'POST') {
        var busboy = new Busboy({headers: req.headers});
        busboy.on('field', (fieldname, val, fieldnameTruncated, valTruncated, encoding, mimetype) => {
            console.log('field');
        });
        busboy.on('finish', () => {
            console.log('finish');
            res.json({
                data: null,
                error: null
            });
        });
        busboy.end(req.rawbody);
    } else {
        console.log('else...');
    }
});


Loading GLBs doesn't show vertex colours

Forum https://forum.playcanvas.com/t/vertex-color-doesnt-show-on-compressed-glb
Our viewer:

Babylon viewer

I believe this only happens when the model is draco compressed.


Should close the iostream (self.content)

In GetContentFile and GetContentString, after the content is read, the iostream needs to closed; otherwise, it could lead to serious memory leak.

This should be fixed by now.


Discussing Tags for commits

Although this isn't as important with a more manual changelog
Suggestions (and examples):

Fix | Bugfix
Docs

Docs: `disallowParenthesesAroundArrowParam` fixes

New | New Rule

New: `disallowArrowFunctions`

Update | Rule Update - new rule values, support that isn't a bug fix

Update: `disallowMultiLineTernary` - add `nested` option

Internal: - something for jsfile, string-checker

Internal: `CLI` - implement `extract` option

Config | Configuration - change related to .jscsrc

Config: strip BOM from config files

Dependency | Package | Upgrade - updating a dependency

Upgrade<EMAIL_ADDRESS>

Chore | Build - related to dev-ops, CI, or dependencies
Misc
Preset
Test
Feat
Perf
Cleanup | Refactor
Security
 Breaking

Don't need that many but just some thoughts

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/26990925-discussing-tags-for-commits?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github).

I'm just looking at changelog. Not sure that we need so detailed list of tags. Breaking can be a flag, like: Fix: BREAKING etc.
Docs: Documentation only changes
Feat: A new feature
Fix: A bug fix
Chore: Changes to the build process or auxiliary tools and libraries such as documentation generation
Preset
New Rule
Update Rule
Perf
Cleanup | Refactor
Misc ← Test, Config, Internal, Security
Chore ← Dependency | Package | Upgrade, ?

Less categories is good - just shouting out possibilities.
General: Docs, Fix, Chore, Misc
JSCS: New Rule, Update Rule, Preset

We would need to use those tags with changelog generator, since primarily we want to do it for changelog itself.
Like what it would generate if we would use Fix:Breaking: ...?
Also we need to make sure we can add such checks in githook.

Similar to https://github.com/jscs-dev/node-jscs/issues/407 so goina close that one again.
https://github.com/jzaefferer/commitplease, http://gitcop.com/, etc

I've been using Fix, Misc, Docs so far - New Rule, Update Rule, Preset sound good. If we want a flag then we can do it like angular fix(rule-here): description, chore(package-here): description

Did you implemented commitplease? Since you moved that ticket here but not implemented it

We also need to document this


Update migration docs

[ ] root readme in getting-started
[ ] Add js to js folder in getting started
[ ] Fix js sample links to link to new getting started
[ ] Add single page for TTK setup for js and link to for all samples

Closing PR: https://github.com/microsoft/teams-ai/pull/383


Map event selector to event name

Based on the following PR: https://github.com/software-mansion/protostar/pull/330

map selector to name
refine output

Please merge the changes, I will be easier to review

@MaksymilianDemitraszek merged master


Cargo Telepad Unpowered at Round Start

No real steps to reproduce, the telepad just begins the round unpowered. Unlike the previous issue involving the telepad, it does not start unanchored. Moving the telepad doesn't seem to power it up either.

supposedly closed by #3559


Function completions always returned as snippets

Environment

Elixir & Erlang versions (elixir --version): Elixir 1.9.4, Erlang/OPT 22
Operating system: Linux
Editor or IDE name (e.g. Emacs/VSCode): Vim
LSP Client name: vim-lsp

Troubleshooting

[x] Restart your editor (which will restart ElixirLS) sometimes fixes issues
[x] Stop your editor, remove the entire .elixir_ls directory, then restart your editor

NOTE: This will cause you to have to re-run the entire dialyzer build

The LSP client that I'm using specifies snippetSupport as false, but all function completions are still being returned with snippets.
For example, elixir-ls gives Enum.join(${1:enumerable}, ${2:joiner \\\\ ""}) for the completion of Enum.join.
Looking into the code, it seems like function completion always expands to snippet form.
The regex for checking if a completion is a snippet  also does not account for when the snippet has a placeholder.
My ideal behaviour is to provide an empty argument list if the LSP client does not support snippets.

Marking this as "help wanted" and "good first issue" because this should be a fairly straightforward fix that won't require looking at any code outside of https://github.com/elixir-lsp/elixir-ls/blob/81057be222204a461aad132f31d4666b3ea082b2/apps/language_server/lib/language_server/providers/completion.ex

A good approach might be to define two versions of the completions, one that supports snippets and another that does not.


Add JuliaChemicalReactions to organizations.txt

Hi @ViralBShah, I have updated with the name of my "new-born" Organization: JuliaChemicalReactions, focused on the development of new tools and parsers to deal with chemical reactions computationally.

Thanks for submitting!


fix(tree): the symbols of image type do not display at the first rendering. close #12279.

Brief Information
This pull request is in the type of:

[x] bug fixing
[ ] new feature
[ ] others

What does this PR do?
Fixes #12279
Fixes #11860
Fixed issues

#12279
#11860

Details
Before: What was the problem?
The symbols of image type do not display at the first rendering. Please refer to #12279, #11860 for more details.

After: How is it fixed in this PR?

After creating the instance of SymbolClz, update its data immediately but not wait until the next update rendering if the type of symbol is image.
Related test cases
Refer to test/tree-image2.html for a test case.

Looks good


bitcoind: descriptor walled disabled

While working with a testnet node I get that the descriptor wallet get disabled.
2023-08-05T21:01:49.042Z DEBUG bitcoincore_rpc JSON-RPC error for fundrawtransaction: RpcError { code: -4, message: "Transaction needs a change address, but we can't generate it. Error: No bech32 addresses available.", data: None }
2023-08-05T21:01:49.043Z ERROR lampod JSON-RPC error: RPC error response: RpcError { code: -4, message: "Transaction needs a change address, but we can't generate it. Error: No bech32 addresses available.", data: None }

We should find a reason of why this is disabled and re-enable it

related to https://github.com/vincenzopalazzo/lampo.rs/issues/128


Edit complete

Q
A

content fix?
yes

New article?
no

What's in this Pull Request?
Added numbered lists, corrected punctuation and grammar, changed some passive to active.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/spfx/extensions/get-started/building-simple-field-customizer.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/spfx/extensions/get-started/building-simple-field-customizer.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/spfx/extensions/get-started/building-simple-field-customizer.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

thx Linda, appreciated.