Tooltip disappears if mouse moves and TooltipText property has been modified

Tested versions

Reproducible in v4.3.dev5.mono.official [89f70e98d]
Other versions not yet tested

System information
Windows 11 - Vulkan, Forward+
Issue description
When mousing over a control and a Tooltip popup appears, it will remain visible so long as the mouse remains hovering over the control that spawned it. This is not true if the control has had its TooltipText property modified since the tooltip was instantiated.
https://github.com/godotengine/godot/assets/22286697/98296521-3b88-44fb-9e45-0d4466050bd2
Steps to reproduce

Create a control node with some TooltipText value
Have the TooltipText value be modified after spawning the tooltip
Move your mouse (still within the bounds of the control node) and observe that the spawned tooltip will be freed

Minimal reproduction project (MRP)
Tooltip MRP.zip

Can't reproduce it.

Can't reproduce it.

Understood, I'll try to get an MRP.

Update: It actually doesn't have anything to do with redraws. Whenever the TooltipText property has changed since the tooltip popup was spawned, moving the mouse will instantly free the existing tooltip popup. In my case, this is actually undesirable, as I'm using a custom tooltip node in my game that automatically updates itself. This built-in behavior means I cannot use the real TooltipText property, unfortunately. MRP being uploaded to original post in a moment.

This is intended behavior:
https://github.com/godotengine/godot/blob/578d937927ea9735aa7a8e0535f57801406b0b82/scene/main/viewport.cpp#L1894-L1896
The tooltip is closed to avoid displaying outdated data.
If you need a tooltip with custom behavior, you can use _make_custom_tooltip().

@KoBeWi I wanted the same behavior, a tooltip that updates the text instead of disappearing and I can't seem to figure out how to do it with _make_custom_tooltip(). I believe the hiding and display logic is independent of the custom tooltip. _make_custom_tooltip might be intended for aesthetics custmizing but it will stilll hide if it's text is different than the gui_text.
Do you mind elaborating a little bit on how to achieve a custom tooltip that doesn't free when the text changes?

Don't change the text, instead keep reference to the tooltip object and update it directly. You can completely ignore the for_text argument of _make_custom_tooltip().

Here's what I did on a TextureRect (see code excerpt below). The tooltip is shown until sync_value() is called, moment it disappears.
Are you certaint the code you linked isn't called anyway? Because how does the tooltip disappear when the mouse is not hovering when it is custom? I presume it still calls _gui_cancel_tooltip() which queue_frees the custom label.
func sync_value():
    if is_instance_valid(ctooltip):
        ctooltip.set_text("%0.4f" % (self.value*1000))
        ctooltip.set_tooltip_text("%0.4f" % (self.value*1000))

var ctooltip : Label

func _make_custom_tooltip(for_text):
    ctooltip = Label.new()
    ctooltip.set_text("%0.4f" % (self.value*1000))
    #ctooltip.text = for_text
    return ctooltip

I tried without setting the text on _make_custom as well as not calling set_tooltip_text but just set_text, same disappearing behaviour the moment set_text is called on sync_value.

I copied your code and it works:
https://github.com/user-attachments/assets/c422c819-2c5f-41e6-b5c3-d36a4228a262
Make sure you are not accidentally changing tooltip_text.

I see! It must be something with my setup then. Thanks for checking it out! 🚀


POS UI Extensions: Cart's LineItem SKU is undefined for LineItem with an SKU

Please list the package(s) involved in the issue, and include the version you are using
retail-ui-extensions-react V1.1.2
POS App version: 8.11.1
Build 2080
iPadOS version 16.5.1
Describe the bug
Inside the Cart state, LineItem.sku is always undefined (even when a product has an SKU).
Steps to reproduce the behavior:

Add a product with an SKU to the cart
Use useCartSubscription() to get the Cart state and try accessing or reading the SKU.

Expected behavior
LineItem's SKU is returned.
Screenshots
Additional context

The fix for this issue will be available in POS app version 8.20


Add Starsand

Game name
Starsand
Request type
Adding a new game to the feed
Target frameworks, package dependencies

netstandard20

Allowed assembly names

Assembly-CSharp
UnityEngine.UI
Unity.TextMeshPro

Contact details
PhantomGamers#9208 @ Discord
Additional information
No response
Package ownership agreement

[X] I acknowledge and agree to the above rules

Granted without changes


Documentation error

The README says -2 means the DB doesn't exist and -1 means it's not vulnerable. The code says the opposite: https://github.com/Turr0n/firebase/blob/master/firebase.py#L55-L60
I'm guessing this is a documentation error.

Thank you very much, I didn't notice the typo.
I will update it immediately. Sorry for the delay.


feat: fix decimal places based on significant value

Overview
This PR is for #1993

Fixed decimal places based on the significant value from settings decimal places

Verifying Changes
https://imgur.com/a/SAdfW3s
Legal
This project is available under the Apache 2.0 License.

We actually have a util for this in react-components - packages/react-components/src/utils/number.ts
Let's move this into core-utils so that we can use the same rounding logic everywhere

I have made the changes as requested, not sure why all checks are failing here but those are working locally.

Closing this PR and raised the below new PR
https://github.com/awslabs/iot-app-kit/pull/2108


Add some attrs

Add some attrs

Add some attrs.

Just taking a glance and it looks good. Will test it later to confirm.


git-hook/pre-push: exec: python: not found

Does this issue occur when all extensions are disabled?: N/A

VS Code Version: 1.82.2
OS Version: macOS Ventura 13.5.2

I have a pre-push hook that runs a Python program:
#!/usr/bin/env bash
# ... irrelevant comments ...
exec python -m server.devcheck pre-push

The Python executable definitely exists on my computer, yet this stops me from being able to push commits using VS Code. Output:
2023-09-20 19:23:47.810 [info] > git push -u origin branch-name-redacted [957ms]
2023-09-20 19:23:47.810 [info] git-hooks/pre-push: line 4: exec: python: not found
error: failed to push some refs to 'github.com:EncircleInc/encircle.git'

This has to be an issue with VS Code, because I can run the above git push -u ... command from the CLI and it works with no problems. Whatever environment VS Code runs git in, it's not one that recognizes the executables in my shell.
Either this is a bug, or I need to configure my settings somehow. If it's the latter, please offer whatever help you can.
Steps to Reproduce:

Write any Python program you wish (it could be just a "Hello, world" program) and write a pre-push hook to run it like the above example.
Make a branch and add a commit to it.
Try publishing the branch through VS Code. The expectation is that it runs git push -u and successfully pushes the branch to the remote. The actual behaviour is that it fails to find Python.

@brettcannon, does this one ring a bell? Thanks!

Nope, this isn't a specific issue that I'm aware of if Python is globally installed and accessible on PATH (assuming that's how VS Code executes pre-commit hooks).

Same issue when using git in vscode on macOS, it worked when using git in terminal but not in vscode. (It previously worked but the issue happened recently)

I'm currently experiencing the same(ish) issue with a golang repo. I have a pre-commit hook that runs a linter.  I've had this issue in the past, but then got fixed when I updated vscode. Now it was reintroduced with the February 1.87 update.

How do you launch VS Code? Are you launching it from the Terminal by running code or from the dock? If you are running it from the dock you are hitting the issue that is documented here. Please use the troubleshooting steps from the documentation to further investigate the issue.


Add explorerpatcher

fixes #1392

I've had a lot of issues with explorer patcher and crashes. I don't want to include this program at this time.

I've had a lot of issues with explorer patcher and crashes. I don't want to include this program at this time.

@ChrisTitusTech Ive used explorerpatcher for a long time and it has worked great. i think the problem is people trying to run explorerpatcher on windows insider buids, which breaks explorerpatcher a lot. a warning could be added to not install it on insider builds


Trailing slash on Swagger link

http://localhost:9000/scigraph/docs/ resolves
but http://localhost:9000/scigraph/docs does not. Should resolve to the same spot.

I am still seeing this bug at 06acce3d8a185ede2797ef76b2a64bc24eb657d6 after a full rebuild.

Are you using matrix?
http://matrix.neuinfo.org:9000/scigraph/docs

Matrix is fine, but when I rebuild on kaa.neuinfo.org I still have the old behavior. Is there some way to force maven to do a clean build (I was under the impression that it was always a clean rebuild)?

I'm not sure what that host is. Is there a bamboo job for it?
maven should manage all of that for you but you could always run mvn clean package.

It is the development host I'm using for the methods ontology, there is no bamboo job I'm just following github readme at the moment (since I need to provide something that the incf/hbp folks can follow).


Inline external JS files

Hey is there a possibility to inline external JS files?
<!-- build:js inline -->
<script src="https://test.com/test.min.js"></script>
<!-- /build -->

Hi, this is not possible. I think this module is intended for local development where you would not inline remote code.
I think it would also add quite some complexity, what if the download fails or is blocked? what if there's no internet connection while developing or building on your ci, or the content is corrupted. maybe skip it and default to use


Lyrics

The idea would be to introduce a "lyrics editor" into OpenNBS to save lyrics into your songs.
The way it works would be similar to LRC: each line of lyrics is linked to the time when the line starts.
I don't know if it's very clear as I explained it, but I think it would be an amazing idea!

I like this idea! We could get the lyrics into Minecraft by printing them as chat or action bar messages in data packs.
I'm not sure how we would fit that in the current interface, however. I think this would have to wait until the layout redesign, which should come in the near future.


Issues with equality in LINQ examples

The examples in this article that are related to equality have several issues:

The implementation of DogHairLengthComparer.GetHashCode is:
https://github.com/dotnet/docs/blob/c7539d980f4e424d511b2bd30ade9ee6e7a9f3bc/docs/standard/using-linq.md#L168-L172
I don't know what "simple objects" are, but this code is wrong: when two objects are compared as equal, they have to have the same hash code and this code does not guarantee that. This is easy to fix by changing the code to return d.HairLengthType.GetHashCode();.

https://github.com/dotnet/docs/blob/c7539d980f4e424d511b2bd30ade9ee6e7a9f3bc/docs/standard/using-linq.md#L177-L178
I don't see how this code could do anything close to what the comment says. What it will do (assuming the comparer is fixed) is to return a "representative" dog for each hair length. I don't see how that would be useful, so I don't think it's a good example.

https://github.com/dotnet/docs/blob/c7539d980f4e424d511b2bd30ade9ee6e7a9f3bc/docs/standard/using-linq.md#L184-L186
It's not clear to me what the comment is trying to say or how VolunteerTimeComparer is supposed to be implemented.
The most reasonable interpretation I can think of is that the comparer considers volunteers with the same identification (e.g. Name) and time spent the same. The result would then be volunteers who spend exactly the same time in both societies.
In any case, I think this example either needs to be clarified, or rewritten.

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

ID: 89e7bc95-4062-e6df-7475-21973ed1becc
Version Independent ID: 571fd6fd-95b6-6fc9-d4df-9b44da6a01b4
Content: LINQ (Language Integrated Query)
Content Source: docs/standard/using-linq.md
Product: dotnet
GitHub Login: @cartermp
Microsoft Alias: wiwagn

Thanks, @svick, for pointing out the issues with this topic. We'll investigate and update this issue. @cartermp, do you have any context on this code?


expose the canvas context to the visualizer

The visualizers should have access to native HTML5 canvas context functionality.  As a way of explaining what I mean and why and how one normally does this, I'll go with the example I'm working on for the factoring visualizer, below.
I'm writing a visualizer that calls for a lot of gradient rectangles.  Using the p5 example for this, it is painfully slow.  Instead, you can call p5's drawingContext to get access to the native HTML5 canvas context, which does gradients quickly (example).  So visualizers should be able to do this.  However, when I try to call drawingContext from the visualizer, I get the following error:
ERROR in .../frontscope/src/visualizers/VisualizerFactors.ts(294,39):
294:39 Property 'drawingContext' does not exist on type 'import(".../frontscope/node_modules/@types/p5/index.d.ts")'.
    292 |
    293 |     grad_rect(x: number, y: number, width: number, height: number, color1: p5.Color, color2: p5.Color) {
  > 294 |         let barGradient = this.sketch.drawingContext.createLinearGradient(x, y, x, y+height);
        |                                       ^
    295 |         barGradient.addColorStop(0, color1);
    296 |         barGradient.addColorStop(1, color2);
    297 |         this.sketch.drawingContext.fillStyle = barGradient;
ERROR in .../frontscope/src/visualizers/VisualizerFactors.ts(297,21):
297:21 Property 'drawingContext' does not exist on type 'import("...frontscope/node_modules/@types/p5/index.d.ts")'.
    295 |         barGradient.addColorStop(0, color1);
    296 |         barGradient.addColorStop(1, color2);
  > 297 |         this.sketch.drawingContext.fillStyle = barGradient;
        |                     ^
    298 |         this.sketch.rect(x,y,width,height);
    299 |     }
    300 | }

Liam and I tried updating p5 to 1.4.0 but that didn't help.
You can also circumvent this by getting the context from the canvas itself, but you need the HTML id in order to do this.  Here's an example of how to do that.  The problem here is that the visualizer doesn't know the canvas element id (in the browser it looks like 'DefaultCanvas0' or something like that when I inspect it), or I don't know the variable name it is stored in, if any.  When p5 creates a canvas using createCanvas, it is possible to set an id.  But this seems to happen in src/components/CanvasArea.vue and src\components\bundleCard.vue and I don't know how to pass it from there back to the visualizer.
So, in short, the visualizers should have access to native HTML5 canvas context functionality, and the above gives some ideas how one might start figuring out how to do that.

My thought is that canvasArea.vue calls methods of the activeViz, so when you set the canvas id via id(elementId), canvasArea.vue can pass the elementId used to the activeViz, or simply set it when instantiating.
So here, hold the created canvas in some variable, create an id string, set it with canvas.id(elementIdString) and then somewhere below you can set activeViz.setCanvasId(elementIdString).
Then the activeViz could write to it directly. If I recall correctly, this would "simply" require adding a new property to the vizualizer interface (something like canvasElementId) which can then be written to by anything that can control a vizualizer.
I also came across one SO thread with a similar issue: https://stackoverflow.com/questions/67348174/property-noise-does-not-exist-on-type-typeof-types-p5-index. Even the accepted solution speculates it may not be a clean solution, so I'd try the above solution first.

Apprently there's a notion of parentElement: const parentDiv = sketch.canvas.parentElement.id; (see here and here)

Part of @katestange's work on https://github.com/numberscope/frontscope/issues/115.

Try it again based on current main, the problem may have fixed itself. The very stupid code below replacing the usual draw method of VisualizerDifferences.ts just worked for me and drew a visible gradient at the top left of the visualizer area both as a bundle card and in the full-screen view:
    draw() {
        console.log(
            'Hi',
            this.sketch,
            this.sketch.noCanvas,
            this.sketch.drawingContext
        )
        this.drawDifferences(this.n, this.levels, this.seq)
        const bg = this.sketch.drawingContext.createLinearGradient(
            0,
            0,
            0,
            100
        )
        bg.addColorStop(0, 'white')
        bg.addColorStop(1, 'black')
        this.sketch.drawingContext.fillStyle = bg
        this.sketch.drawingContext.fillRect(0, 0, 50, 100)
        this.sketch.noLoop()
    }

This has fixed itself, yay!  Thanks Glen.  I'm going to close this issue.


spring-cloud-sleuth-instrumentation:3.0.4 is incompatible with reactor-core:3.4.5

Describe the bug
spring-cloud-sleuth-instrumentation:3.0.4 uses Scannable.Attr.LIFTER in ReactorHooksHelper:
https://github.com/spring-cloud/spring-cloud-sleuth/blob/602749a8d148c281e9f519a1ef362744eeff1f44/spring-cloud-sleuth-instrumentation/src/main/java/org/springframework/cloud/sleuth/instrument/reactor/ReactorHooksHelper.java#L171
This doesn't exist in reactor-core:3.4.5 (Spring Boot 2.4.x) and throws an exception when running.
Sample
Create a new Maven project with spring-cloud-starter-parent as the parent:
<parent>
    <groupId>org.springframework.cloud</groupId>
    <artifactId>spring-cloud-starter-parent</artifactId>
    <version>2020.0.4</version>
</parent>

How is this possible? We're using build 3.0.x that uses boot 2.4.x. We're not setting reactor version explicitly so we're getting a reactor version from boot.

My bad, I had an additional dependency in the pom.xml that I didn't notice that brought in an older version of reactor-core

How is this possible? We're using build 3.0.x that uses boot 2.4.x. We're not setting reactor version explicitly so we're getting a reactor version from boot.

Sorry if I am missing something but this is the issue I believe.
spring-cloud-sleuth-instrumentation:3.0.4 needs a version post this commit
https://github.com/reactor/reactor-core/commit/ab16d1c7fdd4d7c7aa18531a3f9b7ee06ad27edb#diff-b75ea62daae61a2e6aab3e2c5a565c6a91d6918b84187c6357ea789a432e55d9

How is this possible? We're using build 3.0.x that uses boot 2.4.x. We're not setting reactor version explicitly so we're getting a reactor version from boot.

Sorry if I am missing something but this is the issue I believe.
spring-cloud-sleuth-instrumentation:3.0.4 needs a version post this commit (3.4.6)
reactor/reactor-core@ab16d1c#diff-b75ea62daae61a2e6aab3e2c5a565c6a91d6918b84187c6357ea789a432e55d9

thank you, 2 days with this problem, i solved it thanks to your help


Test coverage

Additional unit tests to increase coverage.
Related Issues
#190

will decrease coverage by 8.51%.

The code coverage actually went up quite a bit by this PR, but one file that is now referenced is not yet tested: https://github.com/adobe/helix-rum-collector/blob/main/src/unpkg.mjs
Other source files have a statement coverage of over 90%.

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

v2.2.1
v2.2.1
GitHub release

Your semantic-release bot :package::rocket:


feat(Datagrid): rename useInlineEdit to useEditableCell in v2

Contributes to #2889
This PR renames the useInlineEdit hook to useEditableCell in v2. It is still possible to use useInlineEdit, but enabling via a feature flag is now required. I renamed the InlineEdit storybook directory to EditableCell to reflect this change as well.
Feature flag usage:
import React from 'react';
import { Datagrid, useDatagrid, useInlineEdit, pkg } from '@carbon/ibm-products';

const MyInlineEditDatagrid = () => {
  pkg.feature['Datagrid.useInlineEdit'] = true;
  const [data, setData] = useState(gridData);
  const datagridState = useDatagrid(
    {
      columns,
      data,
      onDataUpdate: setData,
    },
    useInlineEdit
  );
  return <Datagrid datagridState={datagridState} />
}

What did you change?
packages/cloud-cognitive/src/components/Datagrid/Datagrid.mdx
packages/cloud-cognitive/src/components/Datagrid/Extensions/InlineEdit/InlineEdit.stories.js
packages/cloud-cognitive/src/components/Datagrid/index.js
packages/cloud-cognitive/src/components/Datagrid/useEditableCell.js
packages/cloud-cognitive/src/components/Datagrid/useInlineEdit.js
packages/cloud-cognitive/src/global/js/package-settings.js
packages/core/story-structure.js

How did you test and verify your work?
Storybook

Marked as draft until naming conventions are clarified.


Add feature to support model design for wizard host form and pages.

Add wizard model for host form.
Add wizard page model for pages.

Wizard model supports to host all page models, and wizard can be initialized by wizard model. User only to bind page models with host wizard model.
overwrite  Wizard.CreatePage(WizardPageModel m) method, to support create page with page model.

Add wizard model and page model
Add WPF wizard page support
Add HTML/JS wizard page support


Does it read documentation?

Describe the feature
I think it would improve the quality if the model first goes for example to the package.json and searches the web for the documentation of each library used then vector embeds them to be used to solve the problem.
What do you think?
Potential Solutions
We can discuss that and if applicable I can make this contribution

not yet. If you implement this, run it on swe-bench lite, and if it improves performance we will integrate it into swe-agent


[Feat]: Add a new tool to the website

What feature?
I would like to add a new tool to this website, which will be a tool for paraphrasing. This will be very valuable to the website's resources.
@JasonDsouza212 , I'd like to work on this under GSSOC'23 if you approve.
Add screenshots
Add screenshots
Code of Conduct

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

you can start working on this @mukul314


WIP: More csp fixes

What does this PR do?
Still showing errors for eval/inline. Have:

set node: false for service worker script
turned off html minification (as the csp report has a line number which isn't very useful when it's all on one line :) )
specified netlify headers so hopefully can debug this in branches and don't have to merge to master each time

Links to related issues
#1068
Checklist

[ ] added a test, or explain why one is not needed/possible...
[ ] no unrelated changes
[ ] asked someone for a code review
[ ] joined #karrot-dev channel at https://slackin.yunity.org
[ ] added an entry to CHANGELOG.md (description, pull request link, username(s))

Firefox still complains, but doesn't seem to actually try sending a report.
@tiltec I suggest we merge this. And then maybe removing the "report-only" part for dev and see how it goes...

Do we need to turn html minification back on?


NoMethodError (undefined method `force_encoding' for nil:NilClass)

@coorasse  i try out this Gem via console for calling sample open json api but while calling   log = OutboundRequestLog.from_request(request) its give me an error
  uri = URI("http://jsonplaceholder.typicode.com/users/")
  http = Net::HTTP.start(uri.host, uri.port)
  request = Net::HTTP::Get.new(uri)
  log = OutboundRequestLog.from_request(request)
  response = http.request(request)
  log.response_body = response.body
  log.response_code = response.code
  log.save!

NoMethodError: undefined method `force_encoding' for nil:NilClass
from /home/hp/.rvm/gems/ruby-2.5.0@dummt-api/gems/rails_api_logger-0.2.5/lib/rails_api_logger/request_log.rb:15:in `from_request'

@coorasse  after digging into code i found out that i need to pass empty request.body and now its working. Thanks guys Gem.

@coorasse is there any option through which we can store headers of request or response.

Thanks for the issue. I released version 0.2.6 where you can log even if the body is not present so you don't need to do request.body = {}.to_json anymore.

Regarding the headers no, not at the moment, but I will gladly work on it. 👍 I think is a good idea!

@coorasse  i did modify RequestLog  class to enable storing headers. Thanks for awesome gem


it can't be run

when the code ran in my computer Macmini(M1), it  displays an error'Torch not compiled with CUDA enabled', can you help me, Thank you.

ok, thank you


[BUG] - Error [ERR_INTERNAL_ASSERTION]

[Describe the bug]
When making an axios GET request to "http://localhost", we get the following error:
node:internal/assert:14
    throw new ERR_INTERNAL_ASSERTION(message);
    ^

Error [ERR_INTERNAL_ASSERTION]: This is caused by either a bug in Node.js or incorrect usage of Node.js internals.
Please open an issue with this stack trace at https://github.com/nodejs/node/issues

    at new NodeError (node:internal/errors:406:5)
    at assert (node:internal/assert:14:11)
    at internalConnectMultiple (node:net:1118:3)
    at defaultTriggerAsyncIdScope (node:internal/async_hooks:464:18)
    at GetAddrInfoReqWrap.emitLookup [as callback] (node:net:1510:7)
    at GetAddrInfoReqWrap.onlookupall [as oncomplete] (node:dns:130:8) {
  code: 'ERR_INTERNAL_ASSERTION'
}
Node.js v20.8.0

[To Reproduce]
import axios from "axios";
const url = "http://localhost";
axios
  .get(url, { httpAgent: ssrfFilter(url), httpsAgent: ssrfFilter(url) })
  .then((response) => {
    console.log(`Success`);
  })
  .catch((error) => {
    console.log(`${error.toString().split("\n")[0]}`);
  })
  .then(() => {});

[Platform]
I tested on both windows 11 & ubuntu 22.
Nodejs version: v20.8.0
ssrf-req-filter: 1.1.0
[Additional context]
I'm not quite sure if this is actually a bug within the ssrf-req-filter library or a bug in nodejs.

It seems to be related to having http://localhost specifically.
http://<IP_ADDRESS> seems to work just fine.
Doing things like http://localhost:8080/some/path will also lead to this crash.

By the looks of it this is not an issue with this library, but nodejs.
I tested another library for ssrf filtering called request-filtering-agent and that has the same issue when using localhost.
I was able to solve this on my end by just downgrading from nodejs 20.8.0 to 18.


TAB

How to remove an NPC from a tab, it sometimes flickers in the tab.

This cannot be fixed and you can see this on every Server with the only diffrence being the time the NPC is on the Tablist.


Polylines in blocks are not parsed correctly

The polylines in the block in the attached file are not parsed correctly. The block "Block02" contains 1 polyline, with 4 vertices. See test-case: test4.zip
The file was generated with Rhinoceros 3d in an Autocad 2004 format. Similar effects were to be seen with different export versions, and files generated with Adobe Illustrator.
Reproducing can be done using the following code:
        using (var fs = new FileStream("data/test4.dxf", FileMode.Open))
        {
            var dxfFile = DxfFile.Load(fs);
            var blocks = dxfFile.Blocks;
            var exampleBlock = blocks.Where(blk => blk.Name == "Block02").First();
            foreach (var item in exampleBlock.Entities)
            {
                Console.WriteLine(item.EntityType);
            }
            var polyline = (DxfPolyline)exampleBlock.Entities.First();

            Console.WriteLine("Polyline has {0} vertices", polyline.Vertices.Count);
        }
        Console.ReadLine();

Will give the following output:
Polyline
Vertex
Vertex
Vertex
Vertex
Seqend
Polyline has 0 vertices

Where of course 1 Polyline with 4 vertices is expected (without vertices as seperate entities, or seperate Seqend entities)

Good find!  I've fixed the issue in master, the change is here: cb2e0bf6d7630cbdda168250190b6fd01624e70f.


Fix relational mapping

@map is not supported for relational fields

@ridafkih any chance on taking a look at this, currently blocking a new model we are introducing?

Yes, will have a look and release this in the next release.


error where it seems like the keras_model.h5 isn't downloading/unzipping and the code is moving on before that process completes

I experienced that error where it seems like the keras_model.h5 isn't downloading/unzipping and the code is moving on before that process completes. Not sure there is much to be done about this without some ugly custom code.
Originally posted by @WillRNaylor in https://github.com/SINTEF/pyopia/pull/35#pullrequestreview-1179086189

current workaround is to re-run the notebook if you get this error. It is probably just because python has run the unzipping asynchronously and moved on to the next step in the notebook without the files being properly unpacked.

i think i might have found the reason for this:
The first time you run 'model_path = exampledata.get_example_model()', 'model_path' returns 'keras_model.zip'.
The second time you run it, 'model_path' returns 'keras_model.h5'


Drop dependency on deprecated gulp-util

Hi,
gulp-util is deprecated. It should be replaced by dependencies on the individual components used by gulp-eslint. The README lists alternatives for the different components.
See https://github.com/gulpjs/gulp/issues/2065

gulpjs recommend using ansi-colors instead of chalk. I tried it on, it works good and passes all test.

chalk could still be used w/o any much problem i guess, it's gutil.colors from gulp-util that's deprecated

Can someone please merge this, thanks!

Please release!!

Closed by https://github.com/dlmanning/gulp-sass/pull/667


Add docc file type support

Resolves https://github.com/tuist/tuist/issues/3650
Short description 📝
Adds support for including Docs.docc files.
Solution 📦
Inspected the project file to determine fileType for the new extension.
Implementation 👩💻👨💻
Add docc and file type to Xcode constants for the project.

Codecov Report

Merging #660 (4b69815) into main (8998757) will decrease coverage by 0.00%.
The diff coverage is 90.24%.

@@            Coverage Diff             @@
##             main     #660      +/-   ##
==========================================
- Coverage   84.66%   84.66%   -0.01%
==========================================
  Files         154      157       +3
  Lines        8878     8987     +109
==========================================
+ Hits         7517     7609      +92
- Misses       1361     1378      +17

Impacted Files
Coverage Δ

Sources/XcodeProj/Project/Xcode.swift
100.00% <ø> (ø)

...ests/Objects/Project/PBXProjIntegrationTests.swift
100.00% <ø> (+1.29%)
:arrow_up:

.../XcodeProj/Scheme/XCScheme+TestableReference.swift
72.88% <62.50%> (-1.63%)
:arrow_down:

Sources/XcodeProj/Scheme/XCSchemeManagement.swift
87.69% <87.69%> (ø)

...s/XcodeProjTests/Extensions/XCTestCase+Shell.swift
92.85% <92.85%> (ø)

...codeProjTests/Scheme/XCSchemeManagementTests.swift
100.00% <100.00%> (ø)

Sources/XcodeProj/Errors/Errors.swift
36.11% <0.00%> (-11.12%)
:arrow_down:

Sources/XcodeProj/Utils/JSONDecoding.swift
72.58% <0.00%> (+3.22%)
:arrow_up:

Continue to review full report at Codecov.

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

@all-contributors add @Jake-Prickett for code


Revert "Display blocking view and remove download button when user have no access to the report"

Reverts Expensify/App#42312

Straight revert because of failing test.


Corrupted HTML in the "SEmi-Annual (Targeted) links

In the section #change-the-update-channel-after-you-enable..., there is broken HTML (apparently) in the URL and a comment following it, directly before this "Feedback" section of the page.  Confirmed on Chrome and Edge.

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

ID: d1da48af-4783-bd17-67f3-f265df93503f
Version Independent ID: 17432b22-04cb-f47f-1634-e5b7fc99c37d
Content: Manage Office 365 ProPlus updates - Configuration Manager
Content Source: sccm/sum/deploy-use/manage-office-365-proplus-updates.md
Product: configuration-manager
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Closing this issue. @jocasti submitted a private pull request to fix the comment. It should publish later today or tomorrow at the latest.

Thanks.  I tried to make the update myself, but apparently it didn't take/save, or more likely I didn't submit it properly.


Update AutoRest C# version to 3.0.0-beta.20231102.2

Update AutoRest C# version to 3.0.0-beta.20231102.2

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.ChangeAnalysis

Issues resolving at https://github.com/Azure/azure-sdk-for-net/pull/37927


Duel accept issue

Whenever a user accept the duel challenge bot edit that message to "." and doesn't start a game

I already got this error too, most likely you are doing this:
JSON.parse(<ButtonInteraction>.customId);

When the customId is "yes" or "no", JSON.parse throws an error
The solution I found was to write a function with try/catch
function JSONparse(customId) {
  try {
    JSON.parse(customId);
  } catch {
    return;
  }
}

With that, you won't have any more problems.

I already got this error too, most likely you are doing this:
JSON.parse(<ButtonInteraction>.customId);

When the customId is "yes" or "no", JSON.parse throws an error
The solution I found was to write a function with try/catch
function JSONparse(customId) {
  try {
    JSON.parse(customId);
  } catch {
    return;
  }
}

With that, you won't have any more problems.

Hello @Gorniaky, I don't understand where to fix this in the module source code?

@utarwyn the problem is not in the module itself, but when we load a customId that is not encoded in JSON and try to decode it with JSON.parse, an error occurs, but there is a solution to this problem: if you use numbers to confirm the duel , "1" for "yes" and "0" for "no", JSON.parse has no problem with numbers.


Failed to retrieve azure-arc-onboarding pods

Describe the bug
failed to retrieve  azure-arc-onboarding data
To Reproduce
Steps to reproduce the behavior:

create cluster
create a Service Principle
Install-AksHciArcOnboarding -clusterName $clusterName -resourcegroup $resourceGroup -location $location -subscriptionid $subscriptionId -clientid $appId -clientsecret $password -tenantid $tenant
kubectl logs job/azure-arc-onboarding -n azure-arc-onboarding --follow

Expected behavior
it show ARC related the pods.
Screenshots

Successfully retrieved cluster information.
addon.msft.microsoft/arc-onboarding-arctestwcluster created
Arc Onboarding Agent has been installed to the cluster
To watch progress for the Arc Agents Onboarding run: kubectl logs job/azure-arc-onboarding -n azure-arc-onboarding --follow
Unable to connect to the server: x509: certificate signed by unknown authority (possibly because of "crypto/rsa: verification error" while trying to verify candidate authority certificate "kubernetes")
PS C:\E2EWorkload>  kubectl logs job/azure-arc-onboarding -n azure-arc-onboarding --follow
Unable to connect to the server: x509: certificate signed by unknown authority (possibly because of "crypto/rsa: verification error" while trying to verify candidate authority certificate "kubernetes")
PS C:\E2EWorkload>

Environment (please complete the following information):
Name                           Value

cloudConfigLocation            c:\clusterstorage\volume1\Config
controlplaneVmSize             Standard_K8S3_v1
deploymentType                 MultiNode
vnetType                       Transparent
skipUpdates                    False
vlanID                         0
nodeConfigLocation             C:\programdata\wssdagent
nodeAgentPort                  45000
clusterRoleName                ca-4f04d38a-4490-48ef-89b6-4032f7ca790e
wssdImageDir                   c:\clusterstorage\volume1\Images
vnetName                       extSwitch
stagingShare                   http://netapplinux.guest.corp.microsoft.com/AzureEdge
vippoolStartIP
forceDnsReplication            False
loadBalancerVmSize             Standard_K8S_v1
useStagingShare                False
nodeAgentAuthorizerPort        45001
skipHostLimitChecks            False
useStagingCR                   False
macPoolEnd
cloudServiceCidr
sshPublicKey                   C:\Users\wolfpack.ssh\id_rsa.pub
akshciVersion                  <IP_ADDRESS>
insecure                       False
vippoolEndIP
wssdDir                        C:\wssd
macPoolStart
cluster version:
Version         : v1.18.8

You need to do ...
Get-AksHciCredential -clusterName $clusterName

Before attempting to view these logs. Need to add this to the script output. thanks.


Update Close-SmbOpenFile.md

updated syntax, Description for example and added SMB instance

Docs Build status updates of commit ce8ef77:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2012-ps/smbshare/Close-SmbOpenFile.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.

Docs Build status updates of commit 9ea74ac:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2012-ps/smbshare/Close-SmbOpenFile.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/smbshare/Close-SmbOpenFile.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.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@dariomws306 @get-itips @tseward
Peer review needed for this PR. Thanks.

Docs Build status updates of commit 49518fe:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2012-ps/smbshare/Close-SmbOpenFile.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/smbshare/Close-SmbOpenFile.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.


Add cloud:provider to cloud_provider_contents for Salt API call

Without this change cloud:provider won't be saved in Salt as pillar.

Just fix the test and this should be good to merge.

I've updated tests

Rubocop is not happy with the changes...

Hopefully this will fix rubocop complains

 Write user case for cloud provider UI part in Velum

tested it and works fine


Add IndexingParameteres Helper Class

Add a helper class for IndexingParameters that adds convenience methods for setting its configuration, such as file name extensions and file contents. This is a port of IndexingParametersExtensions.

Move to preview 5 as it is an addition changes.

This issue may be fixed when #13881 is resolved.


Fix tlsglobals migration on macOS

#2545 and #2736 are both blocked on a verification failure that this PR fixes.


1.2.0 needs some minor cleanup

source/release.cpp: Lines 15 and 16, you need to use a forward slash for include paths or you break compilation on Linux/Mac:
#include "minizip/ioapi_mem.h"
#include "minizip/unzip.h"
====
source/arnutil.cpp: In function 'bool renameRecursive(const FS_Archive&, const string&, const string&)':
source/arnutil.cpp:87:21: warning: converting to non-pointer type 'Handle {aka long unsigned int}' from NULL [-Wconversion-null]
Handle directory = NULL;
This should probably be 0, not NULL.
====
source/release.cpp: In function 'bool extractZip(u8*, size_t, u8**, size_t*)':
source/release.cpp:282:10: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
if (res != *payloadSize) {
You can avoid the error with a cast of either (unsigned int)res or (int)*payloadSize, it's dealers choice, just be aware that you're mixing signed and unsigned in here, and things will get strange if you ever use this code with a > 2GB file.

Thanks for the reports, I’m about to go hunt for some memory issues so that’s gonna be helpful.
Can’t believe I didn’t notice the include path problem yet again.
Also as the payload size has a maximum size which is lower than megabytes I don’t think I’ll have many issues with the int/size_t checks


Feature/2

DIsplay screen on Electron.

Use relative homepage to load resources on Electron.
Some wait required to render transparent page on Linux

close #2


chore disable rowid conversion check for mow table by default (#27482)

Proposed changes
Issue Number: close #xxx

cherry-pick from: #27482
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Preventing AWS Lambdas from self-terminating when an ack() is sent

Description
Hi! For context, I have been working on a slackbot that sends a survey form to a user via DM and records their response. In this issue here I read AWS lambdas can self terminate after sending an ack, a behavior I did notice. I tried to solve this problem by calling ack() as soon as the event was received but not awaiting the promise until just before the handler returned which seemed to resolve the issue. Wondering why this was not suggested as a potential solution in the issue thread, it might just have been an oversight but I wanted to make sure I wasn't missing anything since it's a lot simpler than spinning off additional lambdas to handle expensive logic. Thanks!
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] example code related
[ ] testing related
[x] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Hi @sanchitcop19 , thanks for writing in!

I tried to solve this problem by calling ack() as soon as the event was received but not awaiting the promise until just before the handler returned which seemed to resolve the issue.

I don't think that it is guaranteed that this approach always works well. The async ack() call may work as you expected in some cases. However, there is still a certain risk that the rest of your code executions can be aborted by the sudden termination of the lambda runtime. Thus, I have to say that the approach is not a fundamental solution for eliminating the FaaS difficulty.
As you may already notice, the recommended way in bolt-js is to use processBeforeResponse: true option and ensure the total execution time of your listener function to be less than 3 seconds. If you have some time-confusing tasks in response to Slack events, your app needs to have asynchronous execution mechanism (say, have a queue to save tasks, invoke a different offline function to run long operations).
As you linked, bolt-python offers Lazy Listener feature, which enables developers to more easily run the offline functions with Bolt listener arguments. We are not planning to implement the same functionality in bolt-js mainly due to the limitation of bolt-js framework design. If you need a handy way, consider using bolt-python instead.
I hope this helps! Do you need more help here?

This is really helpful, thanks for the detailed explanation!

I'm glad to hear that! Let me close this issue now but please feel free to write in when you have more to ask/discuss regarding this.


Why does this error appear to me? Uncaught Error: Cannot find module "react-bootstrap-date-picker"

I'm importing it this way:
import DatePicker from 'react-bootstrap-date-picker';

have you installed it into your project?
npm install react-bootstrap-date-picker --save

For some reason on npm it says no dependencies but there's actually react-bootstrap required

Thanks man!


Prime Factorisation

Added prime factorisation problems

When I tested your code I found that it rarely worked. Here's 10 tests:
('Find prime factors of 7', '[2, 2, 2, 2, 2, 7]')
('Find prime factors of 19', '[2, 5, 19]')
('Find prime factors of 13', '[2, 2, 5, 13]')
('Find prime factors of 3', '[2, 2, 2, 2, 2, 3]')
('Find prime factors of 761', '[761]')
('Find prime factors of 43', '[17, 43]')
('Find prime factors of 613', '[613]')
('Find prime factors of 47', '[3, 47]')
('Find prime factors of 349', '[2, 349]')
('Find prime factors of 367', '[367]')

Be sure to test your code before submitting a pr. You can include
for i in range(0, 10):
    print(primeFactors())

in the last line of your code to test it multiple times.

Good work

Thank you for your feedback, I have adjusted the problem since it was previously being adjusted itself in the algorithm, as well as adjusted the difficulty to be between 1 and 200

Thank you, it looks perfect now.


test(e2e,playwright): SEO component

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #51705

camperbot cant code


[6.3.0] Please ignore - this is for automation testing

Forked from #191

Cherry-pick in https://github.com/bazelbuild/bazel/pull/18996


VID namespace verify allow for precomputation

          Optional TODO in the future PR: save this computation.

Originally posted by @chancharles92 in https://github.com/EspressoSystems/jellyfish/pull/389#discussion_r1377950064

I don't think this is relevant any more.


Use pip for installation

python setup.py install is deprecated in favor of installation with pip. The use of setup.py appears to be responsible for compilation and linking errors with the simulator (#13). The proposed changes do the following:

add cmake to the build system requirements in pyproject.toml
use a consistent and reduced set of dependencies which work with pip 23
recommend pip installation in  README.md

Thanks for the pull -- I've simplified and cleaned up the build process in the latest version of evogym (2.0.0). In particular, evogym is now pip installable.
In my testing, I did not find it necessary to add cmake to the build system requirements in pyproject.toml. I can add it if you continue to experience issues!


[BUG] Resolutions property not populated

Library name and version
Azure.AI.TextAnalytics 5.3.0-beta.1
Describe the bug
Please close this if this is known or if creating bugs on beta packages isn't expected. While the latest API returns these values, they are not populated into the CategorizedEntity.Resolutions property
Expected behavior
CategorizedEntity.Resolutions contains the resolutions returned by the API
Actual behavior
CategorizedEntity.Resolutions is an empty collection
Reproduction Steps
var client = new TextAnalyticsClient(new Uri(""), new AzureKeyCredential(""));
var entities = await client.RecognizeEntitiesAsync($"Test Location Atlanta, GA and Time 5:00PM.");

Note that entities.Value[0] does not contain anything for the Resolutions property
Environment
No response

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Hello, @rattrick1! NER resolutions is currently only supported in model version "2022-10-01-preview" (see: https://aka.ms/azsdk/language/ner-resolutions).
Could you please try the following and let me know if it works for you?
var client = new TextAnalyticsClient(new Uri(""), new AzureKeyCredential(""));
var options = new TextAnalyticsRequestOptions() { ModelVersion = "2022-10-01-preview" };
var documents = new List<string>() { $"Test Location Atlanta, GA and Time 5:00PM." };
RecognizeEntitiesResultCollection results = await client.RecognizeEntitiesBatchAsync(documents, options: options);

Then, the CategorizedEntity objects in results[0].Entities should include a non-empty Resolutions collection where applicable. In the example you provided above, I believe you should see one DateTimeResolution referring to "5:00PM".

Hi Jose, thank you for the quick reply. That did work for me. I think my confusion comes from the following links:
https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/textanalytics/Azure.AI.TextAnalytics/CHANGELOG.md
https://www.nuget.org/packages/Azure.AI.TextAnalytics/5.3.0-beta.1#readme-body-tab
Both of which say that for 5.3.0 2022-10-01-preview is the default.

Gotcha. The docs are referring to the API version (which does default to 2022-10-01-preview) but they fail to mention the model version (e.g., what it is, how it differs from API version, what it defaults to, etc.).
Briefly, the model version refers to the AI model that is used by the service to do the text analysis. This means that users can benefit from improved AI models by simply updating the model version without having to change any of their code or logic (i.e., they can keep using the same API version). The default model version of the library is latest, which is a way of saying "use whichever is the latest Generally Available (GA) model". In this case, to use NER resolutions, model version 2022-10-01-preview is needed, which is newer than the latest GA model, and therefore the model version must be set manually for now. Here's more information on model versions and lifecycle: https://learn.microsoft.com/en-us/azure/cognitive-services/language-service/concepts/model-lifecycle
I have created the following issue to make our docs clearer based on this conversation: https://github.com/Azure/azure-sdk-for-net/issues/33326.
Thank you for bringing this to our attention!


test_mainwindow.py opens too many file descriptors and sockets

Hi,
First, I want to clarify that I am fully aware of the fact, that you do not support running unit tests outside of your own development environments and CIs. However, because I think this might be related to what happens in #12247, I am reporting anyway. Please, don't feel obliged to dig too much into it if you feel this does not apply to the scenarios you are officially supporting.
In my endeavor to stabilize Spyder for the OpenSUSE Linux distro package which I am maintaining, I try to run as many unit tests as possible in order to detect possible package incompatibilites.
Fact is, the execution of python3 runtests.py --run-slow always fails with a segfault or breakpoint trap. This happens not always at the same test but consistently a few dozen tests into the test_mainwindow.py suite. Sometimes there is a Too many open files error message associated with it.
I added a few debug statements to the main_window fixture and found out that for each test, the number of open files and sockets increases. Specifically the pdb_history.sqlite file is opened many times but not closed at teardown of the test. I suspect this is because of the reuse of the window as introduced in #10444. This could also be the reason why #12247 cannot open any new connections.
I will dig more into this, but any hints would be appreciated.

Specifically the pdb_history.sqlite file is opened many times but not closed at teardown of the test

Interesting. @impact27, could you take a look at this one?

I can confirm there is a leak:

pdb_history.sqlite is a file open on the frontend, not the kernel, so I am not sure it is related.

In the internal console after closing the ipyconsole 100 times:
import psutil
proc = psutil.Process()
len(proc.open_files())

shows that pdb_history.sqlite is open 100 times

I fixed it in #12134 because I was already separating the pdb_history work there.

Thanks for looking into this!
I will check if I can apply the whole PR to the 4.1.2 package or whether just the last two commits are enough.

If you need to fix that now:
In spyder/plugins/ipythonconsole/widgets/client.py find def shutdown(self):
Under
self.shellwidget._pdb_history_file.save_thread.stop()

Add
self.shellwidget._pdb_history_file.db.close()

If you need to fix that now:

@impact27, I think this is important enough to have it in 4.1.3 (our next release). Could you make a PR for that?

Done in  #12340


Make previousLocation customizable in Archi4Migrator

Hi,
Like some other (enterprise) users, I maintain an internal package for Archi which exhibits some specific configuration. Until now, I was using a custom location for Archi's config files and user "profile". Now that these have changed, I'd like to use the new defaults.
My issue is that the Archi4Migrator use hard-coded path for previousLocation, so it is not able to migrate data in my context. Would it be possible to make it customizable in some ways so that it could work also in my context?
Ping #925

The idea for the Archi4Migrator was to handle the typical user who has defaults for previousLocation. I was hoping that power users might do a manual copy to the new destinations.
The problem is, how would the migrator know what the previous locations were? When Archi 5 is launched it only knows its own locations for workspace etc.
Unless perhaps it takes some arguments?

I was hoping that power users might do a manual copy to the new destinations.

People (like me) who designs such custom configs are power users, but our end users are not :-(

The problem is, how would the migrator know what the previous locations were? When Archi 5 is launched it only knows its own locations for workspace etc.
Unless perhaps it takes some arguments?

Yes, IMHO the best way to do it would be through a new argument passed through Archi.ini (because if one face this issue, that's because he/she already changed this file).
Is this something possible ?
(another option for me is to simply stick with my custom setup)

If it was just one location instead of the default user.home/AppData/Roaming/Archi4 that could be specified in the Archi.ini file like -DmigrationLocation=/path/to/Archi4 but would get complicated if you've set more than one location for workspace and dropins and config.

So the answer is "yes" if it's just one location. Note that the migrator creates a marker file called migrated in the previous location so it knows not to ask again.

BTW, I might have another related issue : for coArchi users, migrating Archi4 folder also means moving thousands of XML files, and this is a very long operation. Asof now, there's no feedback for people to know the operation is ongoing, and some might think Archi is stuck and kill it while the operation is not finished. Maybe some visual feedback to know that Archi does something would be helpful.

[...] but would get complicated if you've set more than one location for workspace and dropins and config.

...which is my case :-(
I do use a path using "plugins" instead of "dropins" because this is easier to understand for people (if at some points they have to do something manually with this folder).
Maybe a set of options:

-DworkspaceMigrationLocation=/path/to/Archi4
-DdropinsMigrationLocation=/path/to/Archi4
-DconfigMigrationLocation=/path/to/Archi4

?

Well I can do it if there are just two options:
-DworkspaceMigrationLocation=/path/to/Archi4
-DdropinsMigrationLocation=/path/to/Archi4

If the first option is set that is used instead of the inbuilt one.
If the second option is set it will copy that directory, if it exists, to a dropins folder in the new workspace location. But if you have also set -Dorg.eclipse.equinox.p2.reconciler.dropins.directory to something else then things are going to get really complicated and that is something I don't want to do (See code in DropinsPluginHandler for how complicated that is.)

But if you want to migrate the old dropins folder that implies that you want it to be copied to the new default dropins location, right? Otherwise you would keep the existing -Dorg.eclipse.equinox.p2.reconciler.dropins.directory setting?

But if you want to migrate the old dropins folder that implies that you want it to be copied to the new default dropins location, right? Otherwise you would keep the existing -Dorg.eclipse.equinox.p2.reconciler.dropins.directory setting?

Not necessarily. In my case I use a custom dropins folder named "plugins" and under "Archi4", so I might want to migrate it outside this "Archi4" folder, but not into the default dropins folder.

But if you have also set -Dorg.eclipse.equinox.p2.reconciler.dropins.directory to something else then things are going to get really complicated

~Not sure to see why this would be complicated.~
Ok, I've looked at the logic coded into DropinsPluginHandler and see the issue related to % variables substitutions. Btw, this would also impact workspaceMigrationLocation and dropinsMigrationLocation as they would also rely on % variables.
Would it be possible to make DropinsPluginHandler#getUserDropinsFolder() and DropinsPluginHandler#substituteVariables() public and static and to call them from Archi4Migrator ? This would make it possible to "decode" workspaceMigrationLocation and dropinsMigrationLocation, and to find the right target dropins folder, whatever its path/name.
If such approach makes sense for you, I can work on it myself and let you review the code after.

It's not the problem of substituteVariables (which actually is only needed for the dropins folder). It's the way that the dropins folder can be one of many things (user set, "plugins", etc...)
But I think I can find a way to do this - move substituteVariables into ArchiPlugin, add a getUserDropinsFolder method there (which returns null if not set) and then use that when copying to a user set dropinsMigrationLocation

I can work on it myself and let you review the code after.

I'll do it.

BTW - work so far is in migrator branch.

which actually is only needed for the dropins folder

Well, if workspaceMigrationLocation is set to @user.home\somepath, something has to happen for this @user.home to be converted to something useful (unless the magical eclipse framework does it for us?).

I'll do it.

Thank you !

(I'll just be glad when everyone has migrated to Archi 5)

OK, I think I've done it now. You can set these in Archi.ini before launching Archi  for the first time:
<EMAIL_ADDRESS>-DdropinsMigrationLocation=/oldPath/to/dropins
-Dorg.eclipse.equinox.p2.reconciler.dropins.directory=%user.home%/somewhere/else

Dropins will be copied from dropinsMigrationLocation to whatever is set in org.eclipse.equinox.p2.reconciler.dropins.directory (assuming they're not the same).
Note that we use the %user.home% for the dropins path because that's what Eclipse parses when it sets the dropins folder. In order for us to also parse the dropins folder we have to do the same thing (there's no simple Eclipse API to get this so we have to re-invent the wheel with our copy of substituteVariables to get it. Hey, Eclipse rocks! 😢  )

Not necessarily. In my case I use a custom dropins folder named "plugins" and under "Archi4"

Now it gets complicated if that plugins folder is in user.home/AppData/Roaming/Archi4.  When copying files and folders to the new destination from this folder the logic is:

If it's a file, copy it to target workspace (files like models.xml, templates.xml)
If it's the .config folder, ignore it
If it's the dropins folder, copy it to target workspace
If it's the .metadata folder, copy it to target workspace
If it's any other folder (could be model-repository, jasper-reports or any user created folder) copy it to user.home/Documents/Archi

So your plugins folder will get copied as in the last case.

It might be easier if you tell me what your folder layout is exactly, so I can test it. It's all getting rather complicated.

It might be easier if you tell me what your folder layout is exactly, so I can test it.

Here's my current configuration:
<EMAIL_ADDRESS><EMAIL_ADDRESS>-Dorg.eclipse.equinox.p2.reconciler.dropins.directory=%user.home%/Archi4/plugins

Now it gets complicated if that plugins folder is in user.home/AppData/Roaming/Archi4. When copying files and folders to the new destination from this folder the logic is:
[...]
So your plugins folder will get copied as in the last case.

I  guess it is possible to check if current file/folder to be copied is the same as dropinsMigrationLocation, and if yes skip it.

It's all getting rather complicated.

You don't have to implement all this by yourself, I can easily work on it in the coming days if you want.

You don't have to implement all this by yourself, I can easily work on it in the coming days if you want.

I'm almost done with it, but getting a fix out for HTML/Jasper not working should come soon.

Here's my current configuration:

And what will be the settings for the new Archi 5 configuration, including the workspaceMigrationLocation and dropinsMigrationLocation?

I guess it is possible to check if current file/folder to be copied is the same as dropinsMigrationLocation, and if yes skip it.

Yes, that should work
Please try this build - https://www.archimatetool.com/downloads/temp/Archi-Win64-5.0.2.zip

Please try this build

Checking it tomorrow morning :-)

Checking it tomorrow morning :-)

I've checked and it almost work. For some reasons my dropins folder (name plugins) didn't get copied. I'm trying to understand why (checking the code and my folders right now)

Please post args you're using and folder structure

It now works !

The migration ones will be:
<EMAIL_ADDRESS><EMAIL_ADDRESS>

In fact dropinsMigrationLocation should be defined as %user.home%/Archi4/plugins instead (variables pattern is different).

In fact dropinsMigrationLocation should be defined as %user.home%/Archi4/plugins instead (variables pattern is different).

No, dropinsMigrationLocation is parsed using @user.home The %user.home% construct only works in org.eclipse.equinox.p2.reconciler.dropins.directory

Wait, it seems to work, but there might be an issue anyway. Digging....

Please share your Archi.ini...

No, dropinsMigrationLocation is parsed using @user.home The %user.home% construct only works in org.eclipse.equinox.p2.reconciler.dropins.directory

That's what I've seen just after.
I've solved it. The root cause was an issue with my Archi.ini in which (due to several changes done), I had<EMAIL_ADDRESS>so using the wrong variable.
At some point I thought it worked with -DdropinsMigrationLocation=%user.home%/Archi4/plugins, but that was just a side effect of Archi4Migrator not able to find the old dropins folder, so copying everything (including my old plugins dir) into the target workspace.
So in short: was my fault and eclipse using multiple variables patterns sucks ;-)
It definitely works !

BTW: I think this Archi4Migrator can be helpful also in the long term (in fact each time someone has to change the default configuration)

This is now in Archi 5.0.2.


Dangerous public servers LUA eval

I read in discord in "Public servers" about dangerous LUA eval ability:
Only join servers from people you trust, malicous server owners can send a packet to execute lua code on your computer (RCE).
We are not responsible for anything that happens to your computer when you join public servers.

Servers can deliver new script payloads to the client, and the client will execute them in an environment that can launch other programs on the host machine. It's not GC-specific, so any protections in GC are useless, protections will need to be clientside to have any security.
@hartie95 says: If I remember it correctly akebi also has protection for that build in
This is really not funny when the server can send the code to your computer and it will execute it, maybe probably add a client patch against the server code execution, some option to do that?

this is one of the larger concerns surrounding private servers, and something the team has thought of many times. currently the focus is on getting UserAssembly patching into Cultivation since this can already be done when injecting akebi. it's definitely a feature that will be implemented at some point though!


Returning NULL on released version

I implemented the device_callendar 0.2.2 and everything works great on debug but when I release the app on android, the device asks for the permission to access the calendar just fine but the list of calendars come back null. The amount of calendars retrieved from the release version come back the same as in debug but the .id and the .name come back null for each calendar. On iOS released version everything works great as in debug. Am I missing something?
I trippled checked my android manifest and the permisions are there:

<uses-permission android:name="android.permission.READ_CALENDAR"/>
<uses-permission android:name="android.permission.WRITE_CALENDAR"/>
<uses-permission android:name="android.permission.INTERNET"/>

I seems the problem is proguard. Once I enable it, the device_calendar returns null.

-keep class com.builttoroam.devicecalendar.** { *; }

@MaikuB did we add info somewhere on the use of proguard - can we elevate it to the Readme so it's easy for developers to find
I'm reopening this item just so we can track the update to the docs

Yes another issue was raised on this and the change is currently in #112

Closing this item as it's addressed in linked PR


When the ItemClicked event is enabled, a margin is added to the bottom of the page when Day view and Month view.

Hello, When the ItemClicked event is enabled, a margin is added to the bottom of the page when Day view and Month view.

Duplicate of #130.


Cleanup bulk uploader sample data

fixes #18
also referenced in #120
-- geodesicsolutions.com-->geodesicsolutions.org
-- Emails all go to @noreply.com now
-- Downloaded free to use sample images from pixabay.com. Files are included in a folder bulk_uploader/docs/samplepics with a README. Probably best keep everything self-contained as far as the distro.
I guess the images also need to go on:
http://www.geodesicsolutions.org/demos/photos/
unless you have a different plan.

I guess the images also need to go on:

http://www.geodesicsolutions.org/demos/photos/

No need for the longer path, lets call it what it is, something like bulk-uploader-sample-images/ or similar.  I'd say we should add it there first, if you do a PR then I can review/merge it.  Once it is on the website you can test the sample CSV import files to make sure the import works as expected using the updated url.

do a PR then I can review/merge it

Done!
https://github.com/geodesicsolutions-community/website/pull/17/files

@vicos59 the images are merged in the website and updated, be sure to test the bulk upload with the sample CSV and let me know if it works now.

example_upload_auctions.txt fails at the final step:
http://localhost:8081/admin/index.php?page=addon_bulk_uploader_main_config&p=3
TypeError: array_merge(): Argument #1 must be of type array, null given in /var/www/html/src/addons/bulk_uploader/admin.php:2007
Stack trace:
#0 /var/www/html/src/addons/bulk_uploader/admin.php(2007): array_merge(NULL, Array)
#1 /var/www/html/src/addons/bulk_uploader/admin.php(1915): addon_bulk_uploader_admin->alterUploadForm()
#2 /var/www/html/src/addons/bulk_uploader/admin.php(416): addon_bulk_uploader_admin->displayBulkUploader()
#3 /var/www/html/src/admin/php5_classes/Admin.class.php(543): addon_bulk_uploader_admin->display_addon_bulk_uploader_main_config()
#4 /var/www/html/src/admin/index.php(16): geoAdmin->load_page()
#5 {main}

P.S. I know I need to mod .gitignore to exclude files from the 2 new folders. I am doing this last or else it makes a mess on Windows. Will remove the temp files before final commit.

By the way, I don't think you need a readme in every empty folder...  If there is a specific reason to sure, just mentioning it isn't like a requirement or anything, like there is none in user_images or templates_c etc.

I am a long time practitioner of self-documenting code. It makes life much easier for me and the next guy who has to make changes x+ years from now.
I see the README.md as essential in those folders because user-generated data is being stored in a place where normally there would only be code. In my apps, I typically put data files in a data/ folder parallel to webroot/ where they can never be directly accessed by a URL.

In my apps, I typically put data files in a data/ folder parallel to webroot/ where they can never be directly accessed by a URL.

That is actually one of the things I was hoping to improve in a future version... Not the initial release.  But it would be nice to have better separation.  Like code would go in src, web accessible in webroot, etc. like you said.  I'm used to cakephp so one option might be to use it to handle the framework, maybe finally get rid of that huge switch in the index.php 😆 .  Though I think we would need a lot more people using it to warrant doing a refactor of that size.


WIP: Initial sketch of streaming tail workers

Implements the key pieces of the new streaming tail workers model. Does not yet introduce the changes to make it active. This PR focuses on the core data model and restructuring of parts of the current tail workers impl to accomodate both models. Look at the tests (specifically trace-streaming-test.c++) to get a good feel for how the streaming trace is structured.
Among other changes included here...

Enables tail workers in local dev. A workerd worker can now be configured to push tail events to a tail worker locally. In the current form this only supports the Version 1 tail workers but will soon support the streaming tail workers also.
Adds an autogate that will be used to gate the use of the new streaming tail workers functionality

Can we please get a design doc showing what the API looks like from the application developer's perspective? It's hard to figure that out from a PR.

Can we please get a design doc showing what the API looks like from the application developer's perspective...

There's already an internal wiki doc discussing this that has already been shared with you previously. Check your email. Was shared about a month ago and updated yesterday

Overall LGTM

@danlapid ... thanks for the review! I updated a couple of those. On the naming concern, I'm going to leave the names as they are for now. Those will be trivial to change later if necessary before this goes live.

@fhanau ... updated here to incorporate the new ExecutionModel info

Additional commits added to address feedback. PR rebased to address merge conflict.

Closing now that this has gone through a few rounds of discussion. The actual implementation work will be spread out over several smaller PRs


[request] Feature output .xml.gz

add file.xml.gz output feature to compress file size

Currently there is no plan to do this. You can easily use gzip to achieve this.
py-epg-cli local -f input.json -o tv.xml -d 7 -w 10
gzip -f tv.xml


Bills: support bill action related entities in output

Adds support for Bill Action Related Entities when the client requests Bill data with the Actions include.
Goal here is to open up more ability to identify things like Committee relationships to actions at the scraper level, of which clients can then take advantage.
Also added some architecture notes to the README to make understanding the code a bit easier.

Great, this has apparently blown up 16 tests somehow :roll_eyes: will work on that next session smh


testing data for perception

I wanna individually run perception module on my my local machine that test the ability to detect obstacles or traffic lights recognition then visualize the simulation process on Dreamviewer. But, where I could get sensor data or testing data feed into perception module?
Any helpful answer is appreciated!

Hi Zhonggen,
At modules/perception/README.md, you can see more details information about that. We have opened data set which could be found at http://data.apollo.auto.
Also, Please check up the related staff from modules/perception/data/

Hi Jie,
Thanks for your response. Your prompts are very useful. I got better understood about perception module and data platform.
But for files under modules/perception/data/, I have no clue about those and is there any docs for this folder.
Thanks in advance.

you can get the usage example in apollo/docs/demo_guide/README.mdpython3 docs/demo_guide/record_helper.py demo_3.5.recordcyber_recorder play -f demo_3.5.record --loopget demo_3.5.record first and then play it. ——Reply reference comes from the Apollo developer community issue master contest https://apollo.auto/developer/questiondata_cn.html?target=636


plugin does not work

I have installed the plugin, but it does not work.

my tree-sitter config:
require("nvim-treesitter.configs").setup {
  ensure_installed = { "vim", "vimdoc", "bash", "c", "cpp", "json", "lua", "python", "markdown", "markdown_inline",
 "make", "cmake" }, -- one of "all" or a list of languages

  highlight = { enable = true },
  indent = { enable = true },
}

Do I forget to config other thing?

The recommended way to setup the plugin is not via the setup function (see the readme), but if you prefer to use the setup function, you need to use require('rainbow-delimiters.setup').setup (note: setup twice, not configs for the first one).

Try this
{
  'nvim-treesitter/nvim-treesitter',
  event = 'VeryLazy',
  build = ':TSUpdate',
  main = 'nvim-treesitter.configs',
  opts = {},
},
{
  'HiPhish/rainbow-delimiters.nvim',
  lazy = false,
  main = 'rainbow-delimiters.setup',
  opts = {},
},


LIFX broken on Windows in HA 0.41

Home Assistant release (hass --version): 0.41.0
Python release (python3 --version): Windows
Component/platform: LIFX
Description of problem: It seems that #6584 broke Windows compatibility for LIFX because hass.loop.create_datagram_endpoint is not supported.
Expected: I expected that it would work :-)
Additional info: I want to fix this, but I am not sure how to proceed; what is the proper way to support all three of asyncio, Windows and UDP?
@balloob @frawau do you have any hints?

There is no way. You can move back to threadding pyton module or use docker/virtualbox on windows.
Please add a comment to doc, that windows are not supported at the moment.

Bummer. I see three paths forward:

start from scratch with a sync module
include the buggy old version and (automatically?) use it on Windows
declare Windows unsupported

I don't like any of them but what is the preferred way?

I do not have a Window$ machine and I don't want to touch that thing with a 10 metre
pole, but
Can't you jusr use asyncio.SelectorEventLoop instead of asyncio.ProactorEventLoop?
The only drawback is that you cannot use "Pipe"
Regards,
François
On Monday, 27 March 2017 22:24:51 ICT Anders Melchiorsen wrote:
Bummer. I see three paths forward:
1. start from scratch with a sync module
2. include the buggy old version and (automatically?) use it on Windows
3. declare Windows unsupported
I don't like any of them but what is the preferred way?
—You are receiving this because you were mentioned.Reply to this email directly, view it
on GitHub[1], or mute the thread[2].[3]

[1] https://github.com/home-assistant/home-assistant/issues/
6820#issuecomment-289666545
[2] https://github.com/notifications/unsubscribe-auth/AORvqcVImYf1BLep3ZspfrmSKpxb-NWXks5rqJmjgaJpZM4Mq0Hn
[3] https://github.com/notifications/beacon/AORvqcpCMq5UmMqm_mUbw-YHv48i4PXUks5rqJmjgaJpZM4Mq0Hn.gif

No  ProactorEventLoop is fix for home-assistant.
I think we can it set to posix only. It is not the only platform/component they will not work with Windows.
@amelchio a fallback for Windows is no option inside our code, that is to wired.

Ooooooh no 😢
I think that the best solution for now is to add the old platform back as lifx_windows (or lifx_legacy). Don't use it automatically, instead:

throw a warning if new one is used on Windows
mention it in the docs

@balloob Thanks for weighing in. That solution is my preference as well but I was unsure about HA project priorities. I'm really sad that I didn't know about this limitation, I really wanted this to be an improvement for all users.
I will go for lifx_legacy as I expect the new implementation to grow additional features soon.


Update scalajs-linker to 1.13.0

Updates org.scala-js:scalajs-linker from 1.0.1 to 1.13.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-js", artifactId = "scalajs-linker" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-js", artifactId = "scalajs-linker" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #636.


Fixes issue with null values in sortBy() when using a callback function

fix #220

I've also done some kind of refactoring in this method. I hope this is a good one.

Based on this changes it would be easy to implement #219 as a feature. Ready to do it if this gets merged. ✌


Extending The Model

i dont really understand how this model really works besides the basic stuff that all frameworks has.
I can no longer make the business logic in the models
I dont have a good example how an many to many relationship works.
Example :
table : UserModel.
foreign_table : UserCommentsModel.
I need a way based on those two models somehow to retrieve all the info without recoding all the functions.
Do Codeigniter members agree that we need an ORM like MODEL ?
Or it is just me wanting this feature inside CI4 ?

This is probably better suited for the forums, GitHub is for issues and bugs. I think the decision was that CI4 won’t have ORM as part of its core, but there are a couple projects to extend CI4 for such functionality. Here is one from the forums:
https://github.com/hlohrenz/CI4-Relationship-Query-Builder


Problems with performance testing (stops responding)

Hello guys,
Ive been writing a while ago about problems with server to respond to siege stress tests and ur recommendation to use keep-alive header worked, but now it seem to be broken again and even keep alive does not help.
Having siege set to run with 15 users i see first 15 requests to be served and thats it, no more request is being processed. I have keep-alive enabled.
Any suggesting? Was something changed in last 4 days?

It seems not even required to run it on Linux. I have tried to run web endpoint frm Visual Studio 2015 and behavior is same - having started benchmarking utility server stops responding after few requests processed.

project.json and resolved package version used would be helpful

Project.json:
{
"version": "1.0.0-*",
"compilationOptions": {
"emitEntryPoint": true
},
"dependencies": {
"Microsoft.AspNet.IISPlatformHandler": "1.0.0-rc1-final",
"Microsoft.AspNet.Mvc": "6.0.0-rc1-final",
"Microsoft.AspNet.Mvc.Formatters.Xml": "6.0.0-rc1-final",
"Microsoft.AspNet.Server.Kestrel": "1.0.0-rc1-final",
"Microsoft.AspNet.StaticFiles": "1.0.0-rc1-final",
"Microsoft.Extensions.Configuration.FileProviderExtensions": "1.0.0-rc1-final",
"Microsoft.Extensions.Configuration.Json": "1.0.0-rc1-final",
"Microsoft.Extensions.Logging": "1.0.0-rc1-final",
"Microsoft.Extensions.Logging.Console": "1.0.0-rc1-final",
"Microsoft.Extensions.Logging.Debug": "1.0.0-rc1-final",
"System.Net.Http": "4.0.0",
"System.Xml.XDocument": "4.0.11-beta-23516"
},
"commands": {
"web": "Microsoft.AspNet.Server.Kestrel --server.urls http://<IP_ADDRESS>:5000"
},
"frameworks": {
"dnx451": {
"dependencies": {
"System.Net.Http": "4.0.1-beta-23516"
}
},
"dnxcore50": {
"dependencies": {
"System.Xml.XmlDocument": "4.0.1-beta-23516",
"System.Xml.XPath.XmlDocument": "4.0.1-beta-23516"
}
}
},
"exclude": [
"wwwroot",
"node_modules"
],
"publishExclude": [
".user",
".vspscc"
]
}
What package u mean?
One more notice. While testing i found strange coincidence with return value size.
Problem happens when the return json document size is around 130Kb. I have changed to return empty document instead - about 12 bytes (rest of logic remain same - all the calculations and data manipulation) and it works much more stable...

/cc @yuwaMSFT

Was the server running on Windows or Linux? Behind any HTTP server?
I did a quick test with kind of latest runtime 1.0.0-rc2-16453. The server is on Linux with Nginx as proxy. It's an MVC with one action returning about 225KB json.
I ran 500 clients for 10 minutes, with avg about 220 req/s. All seemed fine...Will let it run longer and see.

Hi @yuwaMSFT ,
Its both, for windows and ubuntu, same behavior. I have Web Api project.
What u use for testing? Is RC2 available yet?

Hello,
Any feedback on that, is it a bug and should be fixed sometime soon or am i doing somethin wrong?

i have some way of replicating and issue and play around with server settings to let it get timouted and keep workin until next timeout. Having a parallel environment with Kestrel running behind of Nginx i found nginx configuration option called "keepalive_timout" to have affect on Kestrel stucks.
Changing this value i can push Kestrel to shoot a portion of failed connection and then start working once again before next timeout...

Was something changed in last 4 days?

Considering that you are using the rc1-update1 packages, certainly not. Was the same scenario working correctly before that point.
If you don't mind testing on rc2, you can update your NuGet.Config to include "https://www.myget.org/F/aspnetcidev/api/v3/index.json" as a package source.
I assume your using this tool for load testing? https://github.com/JoeDog/siege
Could you give us which parameters your using for siege, your nginx configuration (are you using nginx to reproduce on Windows too?), and the application you're testing against?
The EPIPE errors shouldn't be a big concern. It means there was a write failure, but that shouldn't bring Kestrel to a halt. In rc2 we no longer throw an exception for these kinds of error by default, we simply log them.

Yes, i do use siege as:
siege -t1M -c150 -d1s -v http://endpoint
Failing configuration: 1. Ubuntu VM with DNX or 2. Run application from Visual Studio 2015 - Kestrel endpoint (web).
When I do mentioned above siege, both installations fail regardless of number of cores or RAM assigned to VM. In case of Visual Studio I run on local machine which is i7 Xenon with 16GB RAM...
A successful configuration is:
Ubuntu VM (2 cores/1Gb RAM) with DNX and Nginx installed. Nginx is used to proxy requests. Nginx.conf has 2 processes with 512 connections allowed in each.  Nginx is set to keep-living connection with 5s for keep-livinf timeout.
No idea where the problem is, but sieging Kestrel directly failing. Is there some way I can forward you my app directly?

@ArkadyB If you can't push your app to a github repo, you can email it to me. My email address is set to public on my github profile. Did you try rc2?


Add basic AppVeyor configuration

Run mysql_test and odbc_test (against PostgreSQL) tests.
Some of cases of odbc_test.cpp do not pass against PostgreSQL, so we likely need pgsql_test.cpp variant.

@lexicalunit If you agree to merge this, you will need to follow logon to https://ci.appveyor.com and add nanodbc in https://ci.appveyor.com/projects to enable AV service. No configuration tweaks should be necessary in the Settings panel as all is include in the appveyor.yml.

This is first stab at the AppVeyor setup to enable builds with VS2015.
Next steps include:

HOT: Check $LastExitCode in PowerShell scripts to avoid false-positive build results.
Install SQLite3 library
Install ODBC drivers for SQLite
Add test for SQL Server (the odbc_test.cpp is not compatible)

I'm going to take care of the outstanding points soon.

Added AppVeyor badge (5d8d5fac46dbc9cdf76daea13938c0b600e5ce04)


starting http server for minion worker conditionally

Allow to start http server for minion worker conditionally, following the pattern used for controller/broker/server, so that one can only start a https server for minion worker with the default port 9514 w/o worry about port conflict.

Codecov Report

Merging #9542 (7f98b66) into master (c145ac8) will decrease coverage by 1.22%.
The diff coverage is 33.33%.

@@             Coverage Diff              @@
##             master    #9542      +/-   ##
============================================
- Coverage     69.86%   68.63%   -1.23%
- Complexity     4795     4864      +69
============================================
  Files          1927     1927
  Lines        102697   102730      +33
  Branches      15586    15593       +7
============================================
- Hits          71747    70513    -1234
- Misses        25878    27161    +1283
+ Partials       5072     5056      -16

Flag
Coverage Δ

integration1
25.94% <33.33%> (+0.02%)
:arrow_up:

integration2
?

unittests1
67.26% <0.00%> (+0.04%)
:arrow_up:

unittests2
15.58% <0.00%> (+0.02%)
:arrow_up:

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

Impacted Files
Coverage Δ

...org/apache/pinot/core/util/ListenerConfigUtil.java
50.00% <33.33%> (-31.19%)
:arrow_down:

...t/core/plan/StreamingInstanceResponsePlanNode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ore/operator/streaming/StreamingResponseUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...server/starter/helix/SegmentReloadStatusValue.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../operator/blocks/results/MetadataResultsBlock.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ager/realtime/PeerSchemeSplitSegmentCommitter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...urces/ServerReloadControllerJobStatusResponse.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...he/pinot/common/utils/grpc/GrpcRequestBuilder.java
0.00% <0.00%> (-90.91%)
:arrow_down:

...ator/streaming/StreamingSelectionOnlyOperator.java
0.00% <0.00%> (-90.00%)
:arrow_down:

...he/pinot/core/plan/StreamingSelectionPlanNode.java
0.00% <0.00%> (-88.89%)
:arrow_down:

... and 143 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Prefix after number

Hi, is it possible to put text after currency number, e.g. $ 1,000,000.00 USD ?

Hello. At this moment it's not possible. I will work on it.

This requirement is now available in version 1.4.0. You need to use the suffix attribute.


Support for #log [log] and (log)

It would be nice to have a way to explicitly mark a PR or a commit as "meaningful for the log".
For jQuery / angular style, I think this would be the format:
(log) <type>(<scope>): <subject>

For Ember, I think it would be this:
[LOG <TYPE> <channel>]

For all commits, I think there should be an additional option via #:
Angular: <type>(<scope>): <subject containing #log>
Ember: [<TYPE> <channel>] <subject containging #log>

@seawatts this is the issue, turns out I just need to fill it out ;)

cc @seawatts I've filled it in :)

lol 👍 nice!


Bump golang.org/x/net to v0.33.0

What type of PR is this?
(bug)
What this PR does / why we need it?
This PR updates the golang.org/x/net dependency to more recent version
Pre-checks (if applicable):

[x] Tested latest changes against a cluster

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

Project coverage is 64.22%. Comparing base (1eea813) to head (44e02d9).

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #158   +/-   ##
=======================================
  Coverage   64.22%   64.22%
=======================================
  Files          22       22
  Lines        1504     1504
=======================================
  Hits          966      966
  Misses        459      459
  Partials       79       79

/lgtm


onboard @pmoorman, growth marketer for early-stage tech startups

@pmoorman suggested a marketing audit in #234; that resonates with some gaps I see, and he's clearly more qualified to address them than I am, so I'm doing what I can to show him around.

[x] how he got involved: "@traviagio asked me to get involved with RChain"
[x] filled out introduction / skills inventory (#222) 1/25/2018 10:47:52

See also on-boarding issues #15, #253
cc @traviagio, @pmoorman @patrick727 @drbloom @kennyrowe

I'll write another comment later outlining the history of how I got here (similar to your own issue, @dckc). That could be helpful for modeling future onboarding.
For now, I thought it might help to outline a few things that I'm still stuck with. At this point I've been actively involved with the RAM for 2-3 weeks.
Key confusion / friction areas:

How to define good scopes for projects.  ::  I'm probably better at this than most marketers already, but for any non-dev this will be confusing. It's something I see all sorts of people do badly. Related to: defining SMART goals.

How does consensus work?  ::  I might be a bit impatient sometimes, but I sometimes struggle to figure out whether we have enough of a consensus to proceed with a suggested approach, or rather wait for more "approval". (see #219 for instance) More clarity here would help me be more productive.

[BIG ONE!] How is RChain structured?  ::  We have RChain, RChain Coop (and Ventures and Holding), and then we have within the coop the RAM, but the RAM is only part of it....

Making bigger contributions  ::  there are 3-5 areas of marketing that will make an outsized difference in RChain marketing effectiveness. The nature of the RAM is such, that it's much easier to just work on small issues, rather than kill the elephant. But solving 100 small problems/tasks isn't the same as solving one of those big things. I currently have a hard time finding the right approach towards making big contributions.

I don't know for sure if any or all of these problems are problems to others, also. But here's how I think we can maybe tackle each one of them:
Suggested improvements:
Scope setting

Education on how this works should be included in the onboarding. Once you understand the basics of how RAM and the coop works, this could be part of the "practical" education. It's also a valuable skill to have for other projects.

Experienced people helping you. I appreciate the effort of more experienced people (mostly @dckc) to help me set better scopes for issues. This is a bit tiresome for the mentor, but really helps. The more people get good at setting scopes, the better this will work.

Consensus

Clear guidelines on how this works would help

Clearer understanding of "who decides on what". Could also be part of onboarding education.

RChain structure

Maybe we can make an organisational chart, or some form of diagram that would take away much of the confusion (something like this?)

Naming conventions help. See the issue about "members != members", for instance (we decided to change it to RAM, which is better already)

Maybe this is not an issue to most people. If you're in the RAM and want to do translation work, or fix bugs as a developer, or write blog posts, I don't think you'd particularly care about this. Which leads to the 4th issue...

Contributing on big issues

I think this might be more something personal, rather than an issue everyone experiences. Still, it's interesting to think what we can do to improve it.

Having a bit of a 'political mentor' helps navigate those waters

Better solving of the RChain structural stuff would also really help.

++++++++
Pfff.... that's it for now!!
Cheers,
–Pieter
P.S. I think @dckc deserves a BIG shoutout for patiently helping others get up to speed, when he has undoubtedly more glamorous things to do.  I really appreciate it.

In reverse order, i.e. most useful to least useful answers:

Making bigger contributions

In your case: get in touch with @kennyrowe . He's busy, so be patient and persistent.
I'm not sure this advice generalizes... perhaps it does: make time to talk with people that have been around for a while.

How is RChain structured?

That one we should be able to deal with in 5 to 7 wiki topics:

[ ] Greg Meredith
[x] RChain cooperative

[ ] RChain Board of Directors
[ ] RChain Officers and employees

[ ] Core Dev team, Pyrofex: Medha, Nash, Mike Stay
[ ] Holdings

How does consensus work?

One of the great mysteries of life. I spent 15 years compiling "The Art of Consensus" at W3C. I wonder if I can get a copy to share publicly... of course, I wonder how much of it is applicable outside the particular context of chairing a W3C working group.

How to define good scopes for projects. ... It's something I see all sorts of people do badly.

That's the nature of the beast, as far as I know. There's a reason organizations hire project managers. It's a skill that typically takes time to develop.

In #253 @dckc wrote:

If you haven't spent hours onboarding others, please document your own experience coming on board in an issue like #293 or a blog item or whatever.

So here we go, so we can add that to the library of cases there. In roughly chronological order:

I learned about crypto about 4 years ago

In 2017 I had some more time to dedicate to new projects, so I decided "I should get involved with crypto stuff"

Georgi (@traviagio) and I work together more regularly on projects, and he introduced me to RChain.

When I entered, I was completely confused about what RChain actually is trying to achieve. The upside was: they had lots of marketing problems to solve. Good for me.

Due to @traviagio more or less holding my hand, onboarding was rather smooth. He crucially pointed out that I should call with Ian to get my KYC / passport verification done.

@ian is actually the first one that gave me some more background about RChain during the verification call, and the enthusiasm was contagious (well done, Ian!).

@traviagio had actually invited me with the specific instructions to do a marketing audit (which became #234), so that's where I started.

Once that was on the rails, I had a good understanding of the main issues marketing-wise, and started contributing to a bunch of other issues. Multiple people picked up on my enthusiasm and involvement, and showed me around, such as @dckc

Honorable mention of where we got totally confused:  I sat with @traviagio about his December invoice, and together we deciphered how the financial spreadsheet worked. Took forever to understand that (Bali internet on such a massive spreadsheet didn't help, either).

That's where we're at today.
Things that went well (onboarding-wise):

@traviagio pulling me inside the project

A clear 1st project to work on (marketing audit)

More senior members seeing my work, and helping me really get up to speed (Dan, Ian, Kit, etc.)

Main friction points:

Setting up all the infrastructure (Discord, membership payment, Github (which I otherwise never use), Google Groups access, etc.)

Scheduling the passport verification with Ian

The financial spreadsheet

'Political' navigation: "who should sign off on this?", "Who is in charge of marketing?" and similar sort of questions/frustrations

Btw, thanks everyone  :+1:

Researching new blockchain project I could get involved in to help understand blockchain technology - Rchain came up
I requested for access to one of the google doc and I got an email from @lapin7 which was my guide to getting on rchain coop. This was very good.
$20 to sign got me to think twice and whated to verify this was legit.
I looked up coop members on LinkedIn, contacted @Ojimadu just to validate his experience on the coop. @Ojimadu  sounded convincing, so I decided to join.
When I paid, I thought I would get a notice for verification, so I called +1<PHONE_NUMBER>  (the number on the reciept to call if I have got any questions). I did not get a respond.
I literally had to keep asking to be verified, I did not get an invitation to get verified.
I jot on Github and there were tons of issues, I was reading through to get a sense of what was going on. Next I wanted to get involved so I commented on issues I wanted to work on a task, I wasn't sure what the norms are.
@pmoorman sent me a private message on Discord because I had mentioned I was interested in being involved in a task. @pmoorman helped guide me through this process.
I contacted @jimscarver that I'd like to be part of the BYOID project and he sent me link to it.
Areas I am still unclear of:
-Budgetting, voting and Invoice
-Creating SMART issues.
-Identifying SMEs in various areas (maybe not applicable in this org)
-What the coop mission, vision, strategy is and if or where it is stated
-How Personal Information is secure and what privacy policy is in place (area I would consider working on if not in place)

@makys Your story shows that you're very motivated. Even after going through an onboarding hell, like you did. Anyway you're all set now to make things better at RChain. :-)

Are we done here? The only thing left that I see is perhaps to expand the scope (change the title) to include the story from @makys and perhaps some part of her role / title: "Global Information Security and Privacy Professional"

Let's close it. If we want to include the scope of Makys, we should spin it out into a separate issue. One person, one issue (to keep things clean).


"Leave blank to render without a title" not working

Check for existing bug reports before submitting.

[X] I searched for existing Bug Reports and found no similar reports.

Expected Behavior
Hi, I don't know if this is a bug, but the option to "Leave blank to render without a title" when adding a callout seems to not do anything. I would expect the callout to look like this

Current behaviour
However, this is what currently happens

As you can see the title is still being rendered. In both reading and editing view.
Reproduction
Just insert a callout via admonition.
Which Operating Systems are you using?

[ ] Android
[ ] iPhone/iPad
[ ] Linux
[ ] macOS
[X] Windows

Obsidian Version Check
1.6.2 (sandbox)
Plugin Version
10.3.2
Confirmation

[X] I have disabled all other plugins and the issue still persists.

Possible solution
No response

Check this: answer.
> [!NOTE] Lorem ipsum dolor sit amet, consectetur adipiscing elit. Nulla et euismod nulla.

Thanks for your answer :)