[INLONG-11148][Sort] Fixes Log4j2 of sort-end-to-end-test-v1.15 not working properly

Fixes [Bug] Log4j2 configuration of sort-end-to-end-test-v1.15 not working properly #11148
Motivation
The issue occurs when running tests in the sort-end-to-end-test-v1.15 module, where Log4j2 fails to load the configuration from the log4j2-test.properties file, leading to incomplete or missing logs during the test runs. This issue is particularly significant as the tests are long-running, and prompt log feedback is essential for monitoring and debugging purposes.
The goal of this fix is to ensure that Log4j2 correctly loads the log4j2-test.properties file during the test phase so that proper logging is restored for the end-to-end tests.
Modifications
Note: This is a temporary workaround, as the root cause of the issue (likely related to Maven’s resource loading or dependency exclusions) still needs further investigation and a permanent solution.

Updated the maven-surefire-plugin configuration in the pom.xml to explicitly specify the path to the log4j2-test.properties file using the system property log4j.configurationFile.
This modification ensures that the Log4j2 configuration file located in src/test/resources is correctly loaded during the test phase, thereby restoring proper log output during test execution.

Verifying this change
(Please pick either of the following options)

[ ] This change is a trivial rework/code cleanup without any test coverage.

[x] This change is already covered by existing tests, such as:

All existing end-to-end tests that rely on logging, where log output will be restored and displayed correctly during test runs.

[x] This change can be verified as follows:

Run mvn test in the sort-end-to-end-tests-v1.15 module and confirm that logs are being correctly displayed as per the configuration in log4j2-test.properties.

[ ] This change added tests and can be verified as follows:

Documentation

Does this pull request introduce a new feature? No
If a feature is not applicable for documentation, explain why? This is a configuration fix and does not introduce new functionality.
If a feature is not documented yet in this PR, please create a follow-up issue for adding the documentation: Not applicable.

Correctly printed log:

@PeterZh6 please fix the failed workflows, thanks.

@PeterZh6 please fix the failed workflows, thanks.

Thanks for following up with this. However, I am unable to reproduce the test failure locally, so may still need some time

@PeterZh6 please fix the failed workflows, thanks.

Thanks for following up with this. However, I am unable to reproduce the test failure locally, so may still need some time

The failure is linked to [Bug][Sort] Mongodb2StarRocksTest Failure Due to Potential Dependency Conflicts #11166
I am currently working on a solution and will submit a pull request to address the problem as soon as possible.


⚠ Bofrost.at has degraded performance

In fb2e76a, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2793 ms

Resolved: Bofrost.at performance has improved in cee533e after 24 minutes.


build(flux): update flux to v0.171.0

@jsternberg Can you cast another eye on this one? I needed to update the skip list mechanism now that we have converted all of our acceptance tests to the new format.


No module named 'Queue'

pipenv install --three leads to an error:
root@kali:/home# pipenv install
Locking [dev-packages] dependencies���
python2.7/dist-packages/pipenv/resolver.py", line 11, in <module>
    import pipenv.utils
  File "/usr/local/lib/python2.7/dist-packages/pipenv/../pipenv/__init__.py", line 17, in <module>
    from .cli import cli
  File "/usr/local/lib/python2.7/dist-packages/pipenv/../pipenv/cli.py", line 10, in <module>
    import delegator
  File "/usr/local/lib/python2.7/dist-packages/pipenv/vendor/delegator.py", line 8, in <module>
    from pexpect.popen_spawn import PopenSpawn
  File "/usr/local/lib/python2.7/dist-packages/pipenv/vendor/pexpect/popen_spawn.py", line 14, in <module>
    from Queue import Queue, Empty  # Python 2
ModuleNotFoundError: No module named 'Queue'

What's odd is if I open the python2 interpreter then that import statement is fine. The python3 import statement in that file is also fine in the python3 interpreter. Additionally, using the python3 interpreter from the created virtualenv also doesn't have a problem importing Queue, Empty from queue.
pipenv install --two works fine but this is a Python3 script.
Describe your environment

Kali Rolling
Python version: 2.7.14+
Pipenv version: 11.0.2

Expected result
I expect pipenv install to install all the libraries in the Pipfile.
Actual result
Pipenv errors out on seemingly all its commands like pipenv install and pipenv clean.
Steps to replicate
Run pipenv install.
Pipefile:
[[source]]

url = "https://pypi.python.org/simple"
verify_ssl = true

[packages]

netaddr = "*"
pexpect = "*"
asyncio = "*"
python-libnmap = "*"
netifaces = "*"
requests = "*"
termcolor = "*"
libtmux = "*"

Even weirder, if I pipenv install --two it works. If I then pipenv install --three and agree to delete the old virtualenv via pipenv prompt, it works. Wtf.

Any idea why this would occur?

Had the same error, pipenv was installed with pip install --user pipenv with system Python 2 and called under Python 3 virtual environment. Reinstalling with pip3 fixed the problem for me.

I get this error when I delete the icebreaker folder and reclone it.
Pavel's probably right about it being messed up pip and/or virtualenv
installs but I notice it goes away if you do pipenv install --two then do
pipenv install --three.


Respond to private messages does not work

Code
await driver.subscribeToMessages();
await driver.respondToMessages(..., {dm: true});

Does not work as expected. Private messages sent to the bot are totally ignored and does not arrive.

nevermind, it started magically working


Add certs regenerations counter metric

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

/test all

/test all


Static method cannot be named "getInstance"

We have a static method getInstance, used to access a singleton class in Umpire: https://github.com/LLNL/Umpire/blob/develop/src/umpire/ResourceManager.hpp#L41
When describing this in the shroud yaml file as:
- decl: class ResourceManager
  declarations:
     - decl: static ResourceManager& getInstance()

we hit the following problem:
There is already a procedure with binding name 'get_instance' for the derived type 'umpire_resourcemanager'

Shroud automatically creates a getter and setter method for the C++ object which is shadowed by the derived type.  The default names are 'get_instance' and 'set_instance'.  The  C++ name getInstance is changed to get_instance for Fortran by default (since Fortran is not case sensitive).  The name conflicts and the compiler complains.  The easiest work around is to suppress the creation of the getter and setter by setting their names to blank use a format block on the class:
- decl: class ResourceManager
  format:
     F_name_instance_get:
     F_name_instance_set:

You can also explicitly name them something else if you want to keep them.  The other option is to change the name of the Fortran wrapper for getInstance to something else using a format block on the method:
- decl: static ResourceManager& getInstance()
  format:
    F_name_impl: non_conflicting_name

Thanks! Are there any side effects from disabling the getter/setters?

Not really.  A typical user should not need access to the raw C++ pointer.

Perfect, I will give that try. I think I tried using F_name_impl before and something was still amiss, but maybe it did work.
In any case, it's nice to able to have the names match across C++/C/Fortran. Thanks again.

Suppressing the getter/setter creation worked, closing.


When property is not getting added when Yup to Json

HI Team,
I am trying to get the below Json format
"RegistrationNumber": {
"type": "string",
"when": {
"SearchRegoOrManual": { "is": "RegistrationNumber", "then": { "required": true } },
},
}
by using yup.string().when("SearchRegoOrManual", {is: "RegistrationNumber". then : yup.number().required() })
but in the output we are getting the expected output.
Please help

@DaddyWarbucks Could you please look into this Issue


Check for empty string before adding elements to TLSCerts

As part of ClientTLSSettings, getTLSCerts func in pilot-agent appends TLSCerts elements (CaCertificates, ClientCertificate, PrivateKey) to the tlsCerts [] string. These are essentially file names which are waited for in waitForFile and timeout after 2 mins if file is not available.
In some cases (as in ALS Simple TLS mode), only CaCertificates would be available and the other two client elements ClientCertificate, PrivateKey would be empty. Currently these are blindly added as "" string array and waitForFile waits on a file with name "" for two minutes before timing out. This results is istio-proxy start getting delayed unnecessarily.
This fix checks for "" string before adding to tlsCerts[] so that waitForFile will not have "" element. This is a very local fix.

Hi @deva26. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@deva26: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
272e5efc1b0121b77bdbf0f1bf27d0e3c90680b9
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/cherrypick release-1.6

@rshriram: once the present PR merges, I will cherry-pick it on top of release-1.6 in a new PR and assign it to you.

In response to this:

/cherrypick release-1.6

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@rshriram: new pull request created: #23866

In response to this:

/cherrypick release-1.6

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


bump version; update release log

Codecov Report

Merging #99 into master will decrease coverage by 2.92%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #99      +/-   ##
==========================================
- Coverage   95.76%   92.83%   -2.93%
==========================================
  Files           9        9
  Lines         684      684
==========================================
- Hits          655      635      -20
- Misses         29       49      +20

Impacted Files
Coverage Δ

txaio/tx.py
95.74% <ø> (ø)
:arrow_up:

txaio/_version.py
100% <100%> (ø)
:arrow_up:

txaio/__init__.py
73.52% <0%> (-20.59%)
:arrow_down:

txaio/aio.py
94.11% <0%> (-5.43%)
:arrow_down:

txaio/_iotype.py
45.83% <0%> (-4.17%)
:arrow_down:

Continue to review full report at Codecov.

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


Handle lightning: and keyauth: protocols

Handle click on links using lightning: and keyauth: protocols and prompt user to open Zerologin mobile app.
When user uses a browser and click on a link starting with lightning: or keyauth:, Android or iOS should prompt to open this link in Zerologin.

To increase support of keyauth://, Zerologin will not implement lightning: scheme.
Also, lightning: is used for bolt11 invoice and people will not want to see Zerologin appears when clicking on a link lightning:lnbc...


Can't store release json file

I have installed phpbrew on Debian Wheezy.  However when I try to install a version of PHP I get the following message:
$ sudo phpbrew install php-5.4.36 +default +mysql +gettext +mcrypt +intl +iconv +ftp +exif +dba +openssl +soap +apxs2=/usr/bin/apxs2 -- --with-libdir=lib/x86_64-linux-gnu --with-gd=shared --enable-gd-native-ttf --with-jpeg-dir=/usr --with-png-dir=/usr --enable-wddx --with-mysql-sock=/var/run/mysqld/mysqld.sockPHP Warning:  file_put_contents(/root/.phpbrew/php-releases.json): failed to open stream: No such file or directory in phar:///usr/bin/phpbrew/PhpBrew/ReleaseList.php on line 2
Can't store release json file

You can't use phpbrew with sudo, it's because PHPBrew depends on the environment variable, when you use sudo, you're missing the settings. If you want to install phpbrew to system-wide environment, please check our wiki:
https://github.com/phpbrew/phpbrew/wiki/Cookbook#install-phpbrew-into-system-wide-environment

Thank you that did the trick.


Finished chat functionality, updated style sheet, and updated server.js

Changes:

Added support for multiple users
Added chat messages for user join/leave


Fluid LLM - SharedTree Branching & Merging

Overview
Adds an new experimental proof of concept library for simplifying the process of allowing an LLM to collaborate live with humans by making changes directly to the state of a SharedTree.
Essentially, what we're doing here is sending the current state of a shared tree as JSON to an LLM and asking it to make changes.
The LLM returns the current state of the SharedTree you provided with any modifications as JSON - essentially a divergent branch of the shared tree. This library compares the two branches, produces a series of diffs and can apply them, essentially merging the two branches.
Eventually, SharedTree will have this behavior natively with branching & merging but until then this library is a short term solution.
File Structure & Class Explanation

fluid-llm
|-> object-diff: Old package that is a fork of micro-diff with some small tweaks. You can ignore this -- it will probably get removed.
|-> shared-tree-diff: core package of this library

|-> SharedTreeBranchManager.ts : Enforces schema between branches, produces diffs, and handles applying diffs.
|-> sharedTreeDiff.ts : Produces a series a diff when comparing two SharedTree nodes or javascript objects.

Quick Overview
You can take a look at /test/SharedTreeBranchManager.spec.ts for examples of how everything comes together. E.G:

sharedTreeDiff.ts
This file provides sharedTreeDiff() which produces a list of "diffs" when comparing two different SharedTree's. The core logic was originally forked from micro-diff https://www.npmjs.com/package/microdiff.
How each type of diff (Create, Change, Delete) are create is straight forward with the exception of the "move" diff.
For move diffs we use the unique identifier of an object (e.g. an ID field) to provide object permanence. At this time it's only applicable to objects within an array but it allows you to track a node being moved to a difference index. Without "moves", a node at a new index in a separate branch would be considered a Create.
SharedTreeBranchManager.ts
This class handles producing diffs using sharedTreeDiff() as well as actually applying said diffs. It also uses ZOD to enforce schema to ensure that branches returned by LLM's don't contain unexpected changes. Applying diffs is not as straightfoward as you think and there is a proper order to applying them in order to create the expected end state.
Without this library, the naive method would be to simply completely replace the old SharedTree with the new one. This is not performant and doesn't play well with SharedTree event listeners which tie into the front-end re-render event loop. In contrast, this library allows to apply changes piecemeal, preserving the original tree and producing a series of changes that is very similar to human changes.
Reviewer Guidance

Not all of the code is fully tested. I will wait for some review rounds to solidify the design approach before adding the remaining testing. Particularly around the SharedTreeBranchManager methods.

We should definitely get Taylor or someone from the Tree team to look this over too, since it is related to the tree APIs.

Work on this space continued in https://github.com/microsoft/FluidFramework/pull/22732

@seanimam is there anything from here that we still want/need to bring in to ai-collab?


Clock time picker component

Pull Request Title:
Sharing is Caring Power App Sample
Pull Request Body:
Category

[x] New Sample

What's in this Pull Request?
Power Apps Clock Time Picker Component

Thanks @Laskewitz!
Just to confirm, will someone from the PnP maintainers package the source code into a solution msapp file or should i submit another pull request for the solutions folder included?

@NatiTurts: Whatever you prefer, I can do it, but you're welcome to do it yourself. If you do it yourself, you know how it will look - if I do it, it might not look as fancy as you want it to be! 😉


SEO / update Hreflang & Canonical tags

On the dismoi.io website, update the Hreflang and Canonical tags whenever required.
All necessary info on this supporting file: https://docs.google.com/spreadsheets/d/1IAkAy3JfJqFFa0ZvSwfC_abGEd3o181iAf7oL3mMr_A/edit#gid=2058169503

As an example: on "https://www.dismoi.io/fr/aide" page, the following code should be present between the  tags:
`

`


Update to latest supported version list as of 2024-04-02.

This is an update to the list of Azure OpenAI versions.  While this constant appears to be purely informational, it would be good if it were kept up to date or removed.
This PR updates the list, adding several missing entries and removing entries being retired as of April 2, 2024 (next Tuesday)

awesome! can you generate a patch changeset with pnpm changeset?

@EmanuelCampos Added.  Hopefully I did that right.


[Question] How to run codegen using gradle?

 Don't forget to press Record button in Playwright Inspector
Good luck!

plugins {
application
kotlin("jvm") version "1.5.1"
}

What is application in plugin block?

Folding into https://github.com/microsoft/playwright-java/issues/571


[PyTorch] Add aten::view_as

support aten::view_as op for the frontend of pytorch. The info of this op is available at this link.

@junrushao
Thanks for your reply. BTW, what is tensor aliasing, and could you provide an example?


HTML

#HTML
##User Story
As a bootcamp student
I want the pre work notes to be structured on a webpage
so that I can easily find and read information
##Acceptance Criteria
Given a Prework Study Guide website
When I visit the wesite in my browser
Then I see four boxes titles HTML, CSS, Git, and JavaScript with associated note listed

done


fix missing days with 0 contribution

Since color for legend and graph differs in github page, the days for 0 contribution are missing in widget.
For a few days the github page has given this as a legend for 0 contributions:
var (- color-calendar-graph-day-bg)
Unlike the graph where there is the color code.
Which gives a widget like this:

When the github page displays:

Confirming this fixes the issue,
Before

After

Is this still happening to you guys? I also noticed it for some days, but today I checked and it was ok.

Confirmed, issue seemed to have resolved itself. Live AppStore app is showing empty days as expected.

GitHub page has been fixed with same words in graph and legend. So this PR is now useless.


Package group support

We could identify group based on github/gitlab/... organization.
This way we could support query build order for a group (example: build order of jupyter related packages).
More complex, but we could also identify groups based on the dependency graph, by identifying clusters. Gephi would do that, but there are likely some better suited python tools readily available

Could use https://scikit-learn.org/stable/modules/clustering.html#affinity-propagation or https://scikit-learn.org/stable/modules/clustering.html#spectral-clustering
This SO post has an example from a networkx graph: https://stackoverflow.com/questions/23684746/spectral-clustering-using-scikit-learn-on-graph-generated-through-networkx


archetype=Asset freezing bug, stores non-JSON string into cache

I'm seeing a bug using this plugin with webpack 3 --watch.
When freezing an asset (for example, from emitFile) the value to be frozen is a string, so a non-JSON string is stored into cache.  When cache attempts to retrieve and parse this value as JSON, it throws an error.
Detailed breakdown of the flow through code:
store()...
https://github.com/mzgoddard/hard-source-webpack-plugin/blob/master/index.js#L1561-L1565
...calls freeze():
https://github.com/mzgoddard/hard-source-webpack-plugin/blob/master/index.js#L1519-L1529
...which triggers '_hardSourceFreezeAsset', which is tapped here:
https://github.com/mzgoddard/hard-source-webpack-plugin/blob/master/lib/hard-asset-plugin.js#L9
It calls asset.source() which returns a string according to the docs:
https://github.com/webpack/webpack-sources#source-1
Back in store(), the result of calling freeze() (a string) is passed to cache.set()
This cached value is a string of the asset's source code.
Later, when we try to retrieve that cached value via cache.get(), it attempts to JSON.parse the string, but the string is not valid JSON, so I see an error:
https://github.com/mzgoddard/hard-source-webpack-plugin/blob/master/index.js#L1409-L1410
I think this a logic error but I'm not sure what the intended behavior is because the codebase doesn't have any sort of type annotations.  Can someone help me debug and write a PR to fix this behavior?

I'm using this as a workaround, but I'm not 100% sure it's working:
class HardSourceFixBug {
    apply(compiler) {
        compiler.plugin('_hardSourceAfterFreezeAsset', (frozen, item, extra) => {
            if(typeof frozen === 'string') {
                console.log('WRAPPING string in FROZEN WRAPPER');
                return {___frozenString: true, ___frozenStringValue: frozen};
            }
            return frozen;
        });
        compiler.plugin('_hardSourceBeforeThawAsset', (thawed, asset, extra) => {
            if (!thawed) {
                thawed = asset;
                if(thawed && thawed.___frozenString) {
                    thawed = thawed.___frozenStringValue;
                }
                if (thawed.type === 'buffer') {
                    thawed = new Buffer(thawed);
                }
                if (!(thawed instanceof RawSource)) {
                    thawed = new RawSource(thawed);
                }
            }

            return thawed;
        });
    }
}


[BUG] NullPointerException in KafkaOpenMetadataTopicConnector if no "group.id" or "local.server.id" are supplied in config.

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
NullPointerException in KafkaOpenMetadataTopicConnector if no "group.id" supplied in config.
Expected Behavior
If group.id is a required field then if it is not specified then it should error.
Ifgroup.id is option then start should complete successfully.
Steps To Reproduce
conbfigure a Kafka connector without a group.id and without a local.server.id
Environment
- Egeria:
- OS:
- Java:
- Browser (for UI issues):
- Additional connectors and integration:

Any Further Information?
In KafkaOpenMetadataTopicConnector line 171 the code does this :
                if (StringUtils.isEmpty((String)consumerProperties.get("group.id"))) {
                    consumerProperties.put("group.id", serverId);

the issue here is if there is no serverId then putting a null value in the map null pointers
I marked this as a good first issue, as I hope one of the supplied configurations in the Dojo or labs could be amended to remove the group.id and have no local.server.id to reproduce this issue.

https://egeria-project.org/release-notes/3-11/?h=local.server.id may have introduced this. fyi @planetf1

I did change this code in 3.11 as mentioned
Did you get an NPE or is this by inspection?
According to https://commons.apache.org/proper/commons-lang/apidocs/org/apache/commons/lang3/StringUtils.html#isEmpty-java.lang.CharSequence, calling StringUtils.isEmpty(null) should be fine (and is why that class was used)

in doc
[isEmpty](https://commons.apache.org/proper/commons-lang/apidocs/org/apache/commons/lang3/StringUtils.html#isEmpty-java.lang.CharSequence-)([CharSequence](https://docs.oracle.com/javase/7/docs/api/java/lang/CharSequence.html?is-external=true) cs)
Checks if a CharSequence is empty ("") or null.

@planetf1 it is the put of a null value that is causing the null pointer - I have amended the words

Ok got it, I believe in my testing I never saw a scenario where serverId was null at that point in the code (cue discussion on more checks ;-) )
But I wonder, should we ever get here with serverId null?

serverId is the identity of the server, so if this is NPEing it makes me think there's also a bug elsewhere - either in how the connector is being used here or elsewhere in the code when setting up the server?

I did not know what value to put in local.server.id, so had removed it from my config, it sounds like the code should check the config and ensure we have either a group.id or the local.server.id , if we have neither then the start should fail (gracefully not npe).

see https://github.com/odpi/egeria-docs/issues/586

Agreed, but I think there may be other problems besides the connector if no server.id (I may be wrong). The server.id always gets set when creating a new server, so I think the path that prompted this was the manual removal from the config.
So it does make sense to add the check here.. but also look at where else serverid gets used.

I can't see it gets used that much. If so then doing the check here, and ensuring the server fails initializing if it can't find a group.id to set makes sense

ConnectorConfigurationFactory.java also has a similar code fragment

so to be safe would also add a check here, again ensuring the init fails

Also would check the code in DataEngineProxyOperationalServices. The constructor gets passed the serverId, and it gets used within this module - including for similar checks. I've not tested it but suspect if the id were removed from the config, it could also NPE.

Removed good first issue since ths is difficult to test.


Change full Home page design

Description :
I want to change Home page design and responsiveness . and make it eye catching and attractive.
Hello @codervivek5 ,
Please assign me this issue , i want to work on it.

can you share any design

@codervivek5 ,
I have not Specific Design , But i want to change Vector Png ,some font size and color and responsiveness.

If you give me this issue  , then i  will send my progress with Screenshots.

Redesigned it accordingly your creativity... Remember it that this will be a evo friendly product selling website who will sell handmade eco-friendly product so use color similar to eco friendly theme

@codervivek5 hey please sign this issue to me , i want use some pastel colors to make it more attractive , once done i'll share it with you

Hello @codervivek5
I have made this design for home page . Is it okay???

Please assign this task to me. I want to work on this

Hello @codervivek5 ,
please checkout new pull request  i have mad changes for home section  - https://github.com/codervivek5/VigyBag/pull/1105#issue-2304760733.

Hello @codervivek5 ,
please checkout new pull request i have mad changes for home section - https://github.com/codervivek5/VigyBag/pull/1105#issuecomment-2123893076


Fix for "Uncaught Error: Firebase\JWT\JWT::decode(): Argument #3 ($headers) cannot be passed by reference"

This is unnecessary and this issues is already resolved when headers are not passed. All it needs is a new release.


Windows release builds missing

Have they gone on purpose? the last two releases are missing windows versions
https://github.com/ndmitchell/hlint/releases

My bad - the Linux/Mac releases are automated, but the Windows one I do myself, and I forgot. I went back and added them. Not sure what to do going forward, other than trying to remember.

Can't it be automated on appveyor? What needs a human touch?

I once tried automating it on AppVeyor, and quickly came to the conclusion that it was easier to do it by hand that to do it via hand. I should probably take another go at some point...


[ionic2] Nested list

I've tried to make a nested lists. But all styles in the inner list seemed to be broken.
The below code is a pseudo-code of what I've tried to make.
<ion-list>
  <ion-item>
    <ion-label>LABEL</ion-label>
  </ion-item>
  <ion-item>
    <ion-list>
      <ion-header>Language</ion-header>
      <ion-header>Language</ion-header>
      <ion-header>Language</ion-header>
      <ion-header>Language</ion-header>
    </ion-list>
  </ion-item>
</ion-list>

But it was rendered as

Thanks.

what about something like this?
<ion-list>
   <ion-item>
      <ion-label>LABEL</ion-label>
   </ion-item>
   <ion-list>
      <ion-item>
         <ion-label>INNER LABEL</ion-label>
      </ion-item>
      <ion-item>Language</ion-item>
      <ion-item>Language</ion-item>
      <ion-item>Language</ion-item>
      <ion-item>Language</ion-item>
   </ion-list>
</ion-list>

<ion-header> is not an Ionic component, you can use <ion-list-header> or <ion-item-divider>. Please post questions on using the framework on the forum: https://forum.ionicframework.com/c/ionic-2
http://ionicframework.com/docs/v2/components/#list-headers
Closing as it isn't an issue with the framework itself. :smile:

@cyprusglobe Your source is working, but needs some fixes for indented styling. I think I can handle it. Thanks.
@brandyscarney In this example, ion-header is still used. I think it should be fixed soon to prevent misuses.

@pjc0247 Thanks for pointing that out! Created an issue here to update it: https://github.com/driftyco/ionic-site/issues/434

:smile:


Has dependencies?

I'm a little confused. It says that this package has no dependencies and that you can include the CSharp files in your project, however, I'm unable to get this working. It seems to be using / referencing CSharpx and RailwaySharp which both appear to be unrelated projects. I'm producing a simple portable utility so would ideally like to avoid having to install a separate dll for this, but could anyone provide some guidance on how you're supposed to embed the .cs files directly in a .net framework project?

See #251 - the 'no dependencies' promise only applies to the final NuGet package, not the intermediate build steps. CSharpx and RailwaySharp are both written by the original author of commandlineparser and I don't know why he decided to make them separate repositories. You should be able to copy over those files too into your project
I would also highly recommend utilizing ILMerge - it will take multiple DLLs and combine them into a single DLL/EXE - that way you can use any Nuget package and still retain a single EXE when deployed.


Delete ScriptEngineFactory service

Service is already defined in the optional artifact com.tinkerpop.gremlin:gremlin-groovy. When also defined in this artifact and optional jar is not available, an confusing warning is logged: ScriptEngineManager providers.next(): javax.script.ScriptEngineFactory: Provider com.tinkerpop.gremlin.groovy.jsr223.GremlinGroovyScriptEngineFactory not found

Merged, thanks!


Handle empty string when converting external reference values

Fix for HumanCellAtlas/ingest-central#533

Codecov Report

Merging #51 into master will increase coverage by <.01%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master      #51      +/-   ##
==========================================
+ Coverage   73.67%   73.68%   +<.01%
==========================================
  Files          37       37
  Lines        3130     3131       +1
==========================================
+ Hits         2306     2307       +1
  Misses        824      824

Impacted Files
Coverage Δ

ingest/importer/conversion/conversion_strategy.py
98.37% <100%> (+0.01%)
:arrow_up:

ingest/importer/importer.py
68.94% <66.66%> (ø)
: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 88e02a9...0b94e6c. Read the comment docs.


Bump fronius to 2.0.2 at stable

https://github.com/iobroker-community-adapters/ioBroker.fronius/issues/330

2.0.2 created 28.6.2023
300 user
OK


Triggers for Keyboard and MIDI not saving key param in local storage

Hi @raphaelameaume! First, congrats on the awesome work done to build this tool! 👏
I've noticed that the triggers for Keyboard and MIDI using the UI are not working.
The key param doesn't seem to be persisted in the local storage as you can see on the screenshot below.

Thanks for your help!

Thanks for the kind words Fabien, and the reporting of this issue.
Indeed, there was a mixed-up between the custom and native events firing from <Input.svelte>. In this case, the callback to register the trigger expects a custom event and receive a native one, being unable to split the keys properly before saving to localStorage.
Considering <TextInput> is only used in <Params> and <FieldTrigger>, I think we can safely dispatch a CustomEvent from Input.svelte to mimic the behaviour of the on:change callback.
But this gets me thinking into having both CustomEvent and the native one. Currently the on:change event is overridden in order to align every kind of input and being able to apply those change to whatever props we pass through from the sketch file. If we change this behaviour to keep the native one and dispatch a more "custom" one (so with a different name), this would have more consequences overall but maybe more future proof.
Will work on this for the next minor release

Thanks @raphaelameaume for the detailed explanation!

Solved in #21


Add server deployment steps for Nathan

instructions in Readme.md, supervisor.conf file also.

Could we make a docs/ and devops/ directory to mimic the structure of https://github.com/transitmatters/new-train-tracker - README and LICENSE can go in the former and supervisor conf file (and the ansible playbook I'll add later) in the latter

Content looks good - we might want to cleanup and formalize the whole README at some point closer to release

Someone else should add the license.
Sorry for dropping the ball here. Is it good to go?

I opened a PR to add an MIT license: https://github.com/transitmatters/regional-rail-explorer/pull/24


Wire frame sketch and lo fi

created wire frame sketch

https://www.figma.com/file/LEKdq0JqyuzzOGxv0tZWjh/WalletFriendly?node-id=1892%3A4560


Abot is not able to crawl a password protected web site

When we are trying to crawl password protected site (with basic authentication) crawler fails with message that page has no content.  Url That I am passing is http://username:password@siteurl

Hi Steven
My Apology for such a long delayed response,
Due to some legal restriction I won`t be able to share fiddler log but
Today I got some time to look into this  It seems that I got the issue
ABOT is using HttpWebRequest that by default do not support url pattern like<EMAIL_ADDRESS>In fact it does`t work even in IE by default without Registry hacks
Here is what I tested
Code var responce = HttpWebRequest.Create("http://username:password@mysite.com").GetResponse(); throws an 401 exception
But when I passed basic authentication like below it works perfect
var credCache = new CredentialCache();
credCache.Add(new Uri("http://mysite.com"), "Basic", new NetworkCredential("User", "Pw"));
        var request = HttpWebRequest.Create("http://mysite.com");
             request.Credentials = credCache;
         var responce=request.GetResponse();

Then I looked ABOT source code and it seem there is not explicit support for  passing 401 credential
Unfortunately I am not able to complile the code on may machine so not able to test so thought to let you know may be you can help
regards
Ashwini

Reopening to consider a way to allow basic authentication through configuration.

Thanks for accepting as feature request , Let me know if I can contribute.

Hi. This feature will be much appreciated. If possible it should be made as a 'plugin' thereby allowing to develop other authentication methods such as federated login.
If I can help in testing this, please let me know.

I will Implement basic authentication through configuration.

I'm using authentication for crawling some specific websites. You can take a look at the following issue:
https://github.com/sjdirect/abot/issues/81

Hi,
I had done coding & implementing unit  tests,
But when trying to commit I get
remote: Permission to sjdirect/abot.git denied to obender.
fatal: unable to access 'https://github.com/sjdirect/abot.git/': The requested URL returned error: 403

Hi,  you can submit your changes through a pull request.
Steven
On Sep 15, 2015 10:53 PM, "obender"<EMAIL_ADDRESS>wrote:

Hi,
I had done coding & implementing unit tests,
But when trying to commit I get
remote: Permission to sjdirect/abot.git denied to obender.
fatal: unable to access 'https://github.com/sjdirect/abot.git/': The
requested URL returned error: 403
—
Reply to this email directly or view it on GitHub
https://github.com/sjdirect/abot/issues/37#issuecomment-140634664.

Yes, but I need to commit & push the changes somewhere no ? (Sorry for the nubby questions, never contributed on GitHub, first time :-) )
Do i need to fork the project ?
Or there a branch that I can commit...

Yes you would need to fork then commit changes to your fork. Then you
submit a pull request from your repo to abots repo master branch.
Lots of tutorials online on how to do that.
Thanks
On Sep 16, 2015 7:11 AM, "obender"<EMAIL_ADDRESS>wrote:

Yes, but I need to commit & push the changes somewhere no ? (Sorry for the
nubby questions, never contributed on GitHub, first time :-) )
Do i need to fork the project ?
Or there a branch that I can commit...
—
Reply to this email directly or view it on GitHub
https://github.com/sjdirect/abot/issues/37#issuecomment-140753024.

Yes,
No problem,

pull request above now allows basic authentication through config values

Thank you, I climbed over ten thousand more interesting links this afternoon. I don't need an account password. I see that the CrawledPage object has a HttpWebRequest property, which is set up in the PageCrawlCompleted event after the first request, CrawledPage.HttpWebRequest? I am a Chinese student, English is a bit bad, I hope to understand.... Thank you.0.0

Didn't got your question....
Do you want to setup the request object on  PageCrawlCompleted event  ?

yes 0.0

Why... ?
PageCrawlCompleted  it's after the request was done.

我犯迷糊了，对不起

我犯迷糊了，对不起
I am confused, sorry


NullReferenceException in ExternallyControlledLifetimeManager

[TestMethod]
public void NullReferenceExceptionWhenGettingAndSettingsValuesFromMultipleThreads()
{
var e = new ExternallyControlledLifetimeManager();

Parallel.For(
  0,
  100000,
  i =>
  {
    if (i % 2 == 0)
      e.SetValue(new Foo());
    else
    {
      var foo = e.GetValue();
    }
  });

public class Foo
{
  public int[] Value { get; } = new int[1024 * 1024];
}
}

If we resolve values registered with ExternallyControlledLifetimeManager from multiple threads we quite frequently get a NullReferenceException in ExternallyControlledLifetimeManager.GetValue. There are no locks as far as I can tell anywhere so it boils down to what happens with the code above.
System.NullReferenceException: Object reference not set to an instance of an object.
   at Unity.Lifetime.ExternallyControlledLifetimeManager.GetValue(ILifetimeContainer container)
   at Unity.Strategies.LifetimeStrategy.PreBuildUp(BuilderContext& context)
   at Unity.UnityContainer.<>c.<.ctor>b__73_1(BuilderContext& context)
   at Unity.UnityContainer.Unity.IUnityContainer.Resolve(Type type, String name, ResolverOverride[] overrides)
   at Unity.UnityContainerExtensions.Resolve[T](IUnityContainer container, String name, ResolverOverride[] overrides)

This happens with Unity.Container <IP_ADDRESS> and Unity.Abstractions <IP_ADDRESS>.
It did not happen with Unity.Container <IP_ADDRESS> and Unity.Abstractions <IP_ADDRESS>.

The externally controlled lifetime manager is not supposed to be used like that. It is initialized only once during instance registration and holds on to that value for the rest of its life.
Theoretically, if the object it holds reference to is already collected, it might create another one, but it all should happened inside the Unity.

This was just to show how the error occurs. The same thing happens if you resolve something registered using ExternallyControlledLifetimeManager from multiple threads as you can sort of see from the stack trace.

If you could create a test case with the container I will investigate.

Attached a small program that triggers the issue. Start the release build without a debugger attached and press the start button. On my machine it usually takes somewhere between 2-10 iterations until you get a NullReferenceException.
WpfApp2.zip

A possible improvement would be to make ExternallyControlledLifetimeManager a SynchronizedLifetimeManager to avoid creating multiple instances unnecessarily.

I agree, I’ll make the change in next version

After playing with it a bit, I think there are more to it than meets the eye. I see two cases where ExternallyControlledLifetimeManager could be used.

external instance
type or factory registration

In case of instance registration, container can use the instance until it is collected by the GC. Once collected, container should throw an exception if the instance is requested. This behavior should be enforced because container does not know how to create new instance (if at all possible) and not throwing the exception will destabilize environment and create invalid state.
If type or factory is registered with this manager it should only create new instance if it is either never been created or already collected. (This scenario is useful in all kinds of cache scenarions)
I do not think this behavior could be implemented in one manager. I think ExternallyControlledLifetimeManager should be reserved for instance registrations and new manager WeakReferenceLifetimeManager should be created to cover  type and factory registrations.

I've created this issue to track progress of the fix

Would still be useful to fix ExternallyControlledLifetimeManager in a 5.x release and then do your major refactoring in 6.x.

I do not believe it is broken. It was never intended to be used as you do, look at the original implementation.
As I suggested, create your own WeakReferenceLifetimeManager and perhaps send a PR to develop branch?

The original implementation is thread-safe and so works just fine even if you resolve from multiple threads in parallel.

After thinking a bit more about it, I think you are right.

Fixed in 5.11.3


add property for struct value

gen.InitStruct("myStruct").Address()

method to add inits for this block

make it a statement as well


AccountId / Lookup fields have a backslash in the table

When using the collection datatable component I noticed that when displaying 'AccountId' from a vanilla Get Records on Contact, it appears as such:

I'm using the latest version of collection datatable.

@adwhite75 you will need to give more repro steps than this, this is working for me using a scratch org with the master branch.
One thought here is that your user does not have access to an LDS wire - in which case, I cannot query the account name on your behalf.
Please see the attached working scratch org example image.


Panic when skinned mesh has a bone which drives a blend shape/morph target (GLTF and FBX)

If a skinned mesh has a bone that drives a blend shape / morph target, either GLTF or FBX, the asset is not imported.
The bone is keyframed only on Location and drives a blend shape value from 0 to 1.
https://user-images.githubusercontent.com/248383/236671476-e1d807ff-2da9-4f38-8ff4-bbc694d25d75.mp4
GLTF Error
thread 'tokio-runtime-worker' panicked at 'not implemented', <...>\7daa5ef\crates\model_import\src\gltf\mod.rs:109:26
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
[2023-05-07T10:13:10Z ERROR ambient_build] In pipeline pipeline.json/0, at file AnimationDriver.glb

    Caused by:
        task 30 panicked

FBX Error
thread 'tokio-runtime-worker' panicked at 'called `Option::unwrap()` on a `None` value', crates\model_import\src\fbx\mesh.rs:115:108
[2023-05-07T10:13:10Z ERROR ambient_build] In pipeline pipeline.json/0, at file AnimationDriver.fbx

    Caused by:
        task 29 panicked

Tried to load in the skinmesh sample:
const START: (&str, &str) = (
    "TurnOnMachine",
    "assets/AnimationDriver.glb/animations/TurnOnMachine.anim",
);
const END: (&str, &str) = ("TurnOnMachine", "assets/AnimationDriver.glb/animations/TurnOnMachine.anim");

  let unit_id = Entity::new()
      .with_merge(make_transformable())
      .with(
          prefab_from_url(),
          asset::url("assets/AnimationDriver.glb").unwrap(),
      )
      .with(name(), "Machine".to_string())
      .spawn();

Example
.blend, fbx and gltf sample files: AnimationDriveBlendShapeBone.zip
This mesh contains a very simple armature with one animation "TurnOnMachine" that moves on the Y up and down, and drives the blend shape "TurnOn" of this cube (from the sample GIF above).
Validation
As a validation, here's the model in the Khrono's glTF Viewer:
https://user-images.githubusercontent.com/248383/236680315-634c3b99-21af-440d-afd7-33d2197bedcb.mp4

Looked into this; it appears morph targets are not currently implemented.
@FredrikNoren Looks like we need your input on this one, too. Are morph targets planned?

@philpax Not in the immediate future. @alfredbaudisch (or anyone else interested in it) if you'd like to take a stab at implementing morph targets we would be happy to review a PR for it.


Tests using _MULTIPLE_SECRETS_PATCH do not pass if cassettes are removed

Environment

ggshield version: 1.13.5
Operating system (Linux, macOS, Windows): any
Operating system version: any
Python version: 3.8.10

Describe the bug
If one removes the multiple_secrets.yaml cassette, affected tests no longer pass.
Steps to reproduce:
pipenv shell
rm tests/unit/cassettes/multiple_secrets.yaml
export TEST_GITGUARDIAN_API_KEY=<a valid API key>
pytest tests/unit

Actual result:
The following tests fail:

tests/unit/cmd/test_ignore.py::test_cache_catches_nothing
tests/unit/cmd/test_ignore.py::test_cache_catches_last_found_secrets
tests/unit/output/test_json_output.py::test_json_output[_MULTIPLE_SECRETS]

Expected result:
All tests should pass.
This is caused by the secret used in the _MULTIPLE_SECRETS_PATCH being ignored in our .gitguardian.yml.

It works now.


benchmarking read and write performance

@rmg55 put together a nice notebook here looking at s3 versus http access: https://github.com/rmg55/CloudDAAC_Binders , would be great to include more benchmarking information and suggestion in this repository
also wanted to link over to https://github.com/pangeo-data/pangeo-integration-tests/issues/1

@scottyhq I haven't run your notebook but couple weeks ago we had a discussion about the difference between s3/https URL in the cogeotiff slack. We found out that using S3 url was saving one request (GetFileSize)
$ tilebench profile s3://rio-tiler-dev/data/eu_webAligned_256pxWEBP.tif --tile 5-10-9| jq
{
  "LIST": {
    "count": 0
  },
  "HEAD": {
    "count": 0
  },
  "GET": {
    "count": 2,
    "bytes": 32768,
    "ranges": [
      "0-16383",
      "229376-245759"
    ]
  },
  "Timing": 0.4968528747558594
}
$ tilebench profile https://rio-tiler-dev.s3.amazonaws.com/data/eu_webAligned_256pxWEBP.tif --tile 5-10-9 | jq
{
  "LIST": {
    "count": 0
  },
  "HEAD": {
    "count": 1
  },
  "GET": {
    "count": 2,
    "bytes": 32768,
    "ranges": [
      "0-16383",
      "229376-245759"
    ]
  },
  "Timing": 0.4858889579772949
}

ahhhhh, so GetFileSize is used on http sources, if you use s3 GDAL get the filesize from the response of the first GET request

ref: https://cogeotiff.slack.com/archives/C01DE57GLHE/p1613141057016300


Update dependencies

requires.io doesn't seem to be doing this, so manual it shall be
The one notable change is the jump from lxml v3 to v4. All the tests appear to pass, and I'm unclear from the lxml changelog what's happened that is not backwards-compatible. As such, this seems like a reasonable change so that we can gain the benefits of future updates.

requires.io has come through eventually - see #184


Added help menu to FavoritesActivity

This change is

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Default Property Values Incorrect in Documentation

In the documentation, the default values for fillWeight, hachureGap, and simplification are incorrect.
fillWeight is listed as strokeWidth, but the default is -1
hachureGap is listed as strokeWidth, but the default is -1
simplification is listed as 1, but the default is 0

You're right.  As it stands simplification 0 means no simplification. Setting it to 1 will actually simplify the curves a little bit.
As for fillWeight, hachureGap - the value is dynamically calculated at render time. A value of -1 will cause the current value of strokeWidth to be used.

Thanks for clarifying. The reason I'm not still not 100% sure about fillWeight and hachureGap is because passing 1 for both of them and strokeWidth results in different behavior, but I'm doing some DPR scaling, so it may something unrelated to the defaults anyway.

https://jsfiddle.net/9j64h2r1/6/ may help
fillWeight by default is 0.5 * strokeWidth
hachureGap by default is 4 * strokeWidth
if you set strokeWidth = 1, fillWeight =4, then you should update your hachureGap to be > 4 or it will be too tight.

Just noticed a bug which you may be running into. {fillWidth: 4}, and {fillWidth: 4, strokeWidth: 1} should produce the same results. But it is not. In the later, the hachureGap needs to be set as well. That's definitely a bug. I'll fix that.

That one may not be a bug, in that fiddle, the rough canvas has a default strokeWidth so its being overridden in the drawing https://jsfiddle.net/he85xaua/4/
I think I just misread the initial values for hachureGap and fillWeight when I was first reading the docs.

Documentation updated.


Aqua/refactor

Description
This PR contains the first step of the refactoring work. Breaking down the change into step by step for convenient review. There will be another PR later for more refactor. To avoid breaking change, I haven't remove the original version for all moving. We will remove it in the last step.
Major Change

Provided centralized management for aqua job/model resources.
Moved constants to constants.py.
Moved all enums to enums.py.
Refactor evaluation.py to leverage new module.
Restructure evaluation.py

aqua
|
--evaluation
       |
       --- __init__.py
       |
       --- evaluation.py
       |
       --- config.py
       |
       --- errors.py
       |
       --- request_params.py
       |
       --- response_model.py

Next step

Use wrapper in FT modules.

Test

Many tests are failing https://github.com/oracle/accelerated-data-science/actions/runs/9036162812/job/24832484555
Something wrong with mocking the auth?

Many tests are failing https://github.com/oracle/accelerated-data-science/actions/runs/9036162812/job/24832484555 Something wrong with mocking the auth?

The failing test for opctl probably from merging conflict. I will update it one more time to check.
The rest tests are failing in feature/aquav1.0.2 before this PR : https://github.com/oracle/accelerated-data-science/actions/runs/9025038228


Allow database connectors to transform WKT to geometry during import

Currently, when user import a table using the MySQL connector with a column named "the_geom" set as a VARCHAR data type that contains geometries in WKT text like (LINESTRING(15.24 51.23, 12.43 51.41)), during the import, the table is imported correctly but the_geom column is not
being populated with the geometries of the WKT text, instead it adds an invalid_the_geom column with the WKT values.
According to  @jgoizueta , this is the expected behaviour because the geometries are not automatically detected. You'll need to cast the column after importing.
It would be good to improve the connectors in order to be able to cast WKT to geometry types during the import.

Summoning @kevin-reilly for prioritization.

If we implement this we could accept and convert to geometry various formats:

WKT, EWKT
WKB, EWKB as hex string

That would work if the data select as the_geom is a geometry column in a PostGIS external database, so it would be a workaround also for the case of users with geometries in their external dbs.

Hey, on the BigQuery connector branch I had to enable geometry guessing. It works with our regular georeferencer. Take a look at this commit
Let me know if that commit is suitable for preparing a PR.
cc @jgoizueta

Thanks @alrocar that looks good!
I think that using the Georeferencer will detect WKT in the_geom columns and cast it to geometry, but if the contents is WKB (as an hex string) it won't. We should handle that as well, since when a PostGIS geometry column is imported through a connector it ends up as WKB text string.
It seems that supporting WKB in Georeferencer was considered at some point: https://github.com/CartoDB/cartodb/blob/master/services/importer/lib/importer/column.rb#L157

Hey @jgoizueta thanks for pointing that out
Adding support for WKB/EWKB in column.rb seems quite straight forward. Do you know of any reason it is not there yet? (just wondering if I'm missing something not obvious)

I don't know why it wasn't implemented. One reason could be the current identification of WKB by regular expression is too simplistic (which may not be a problem, since we're only looking at columns with names that suggest they contain geometries). Checking properly for a WKB would be more complex/costly, and non-geometry numeric/binary columns would still be likely accepted as geometries.
I would go for trying to convert anything that looks like a WKB (just checking the proper number of hex digits, and maybe the header bytes) and is named the_geom, wkb_geometry, etc.

Yep, it looks like we are supporting wkb by casting to geometry See this test as well.
Just tested locally and it properly imports a table with a string column that contains a WKB as long as it is named like one of these
Adding a PR -> #12753
Not adding support for ewkt/ewkb yet, but IMO we should tackle that in a different issue.
Thoughts? @jgoizueta

Oh, I had missed the unconditional cast to geometry, good point.
I agree to deal with ewkb/t in a separate ticket, let's deploy the current PR here first, which is a huge improvement in its own.

@oriolbx #12765 merged and deployed in production.
Feel free to close this when you consider.

Thanks! 😃

☝ @alasarr @alvaro some interesting background here related to detecting and casting geometries during imports from databases (via @oriolbx). @jgoizueta take a look at https://github.com/CartoDB/cartodb/issues/12566#issuecomment-326540036.


Support onClick funtion on Link component

Hi @molefrog, nice plugin, i love the Hooks approace!
I have discovered that the Link component didn't accept a onClick prop to do something other then the normal behavior of navigate.
I have implemented the code to support this functionality locally, can i make a PR to merge with your package?
Thanks in advance!

@Alex-Ferreli Sounds great, sure send the PR, would be glad to review it. And thanks for the kind words!
Do I get it right that what you're trying to achieve is to be able to attach an event handler, which will be fired after the transition happens? Sort of:
const afterNavigate = () => { console.log('navigated to /foo') }

<Link onClick={afterNavigate} href="/foo">Click</Link>

Closed via #12


Introduce bytes & vendor support

Introduce:

Working Bytes module with test support to match the go-upstream (there
is slight deviation from the specification, albeit minor).
Mostly complete Codecs module.
Introduce a LarkyObject interface to mimic Python-object like protocols. This
begins the process for "object" like support in Larky.
Allow running of individual standard library tests by using JUnit5's
TestFactory and DynamicTest classes.
Introduce the asserts.assert_fails construct which tests that something
fails.

🎆 🥳


No 'Access-Control-Allow-Origin' header is present

Getting following error in fetching twitter feeds from yesterday.
Failed to load https://api.jublo.net/codebird/oauth2/token: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'your website url' is therefore not allowed access. The response had HTTP status code 403.
I think this error is also coming on the demo page.

same here

Yes I'm having this problem again and it's been going on for I think for about a week now.  Before it would clear up pretty quickly, but now I'm wondering if this codebird api is just dead.

Yeah, still dead.


Project refactoring

The project need a refactoring to improve its maintainability. Configuration blocks can be extracted to another domain, also utility functions.
Each consumer must have its own module and its responsibility needs to be better delimited.

Solved in commit 260f179


Pantheon: prepare for vala 0.56

Description of changes
So they won't fail to build when vala 0.56 becomes default (which will likely happen in the GNOME 42 pull request).
Things done

On x86_64-linux, use Vala 0.55.91 to build Pantheon packages:

[x] Built nixosTests.pantheon
[x] Built pantheon.appcenter pantheon.sideload pantheon-tweaks wingpanel-indicator-ayatana

[x] Fits CONTRIBUTING.md.

@ofborg test pantheon

Result of nixpkgs-review pr 164085 run on x86_64-linux 1

2 packages built:

pantheon.elementary-mail
pantheon.elementary-photos


Change Front-matter text color and font for individual post

Hey
Wondering how to successfully change text color and font of particular words in the Front-matter of individual posts (excerpt and title mainly). Is this possible with Minima ?
Best

@AleksanderTidemann I don't understand the question actually.
Could you explain with some examples..?

@ashmaroli sure! In the image below; I want a couple of words in the title to have a different color than the rest. Same with the excerpt section. Is it possible to use HTML to change the text color and font color of these sections without changing it globally?

Styling is only possible with HTML markup.
What you can do therefore is use HTML directly and style the individual phrases using CSS.
The above front matter can be changed into:
title: 'This is the Title'
excerpt: 'Hello there.'

Now use CSS to style the generated markup:
.red {
  color: red
}
.bold-text {
  color: green;
  font-weight: bold;
}

Thanks, @ashmaroli ! Although, there seems to be a difference between using HTML markup in the title and excerpt. I only get the Excerpt to work, as the image below shows. The CSS code is at the bottom to ensure it overrides any other title CSS config. Any thoughts?


try.gitea.io is not sending emails ?

I've tried using the forgot password feature on try.gitea.io but haven't received any email, nor the system told me about the service being disabled. Is it supposed to work ? Can you check why it isn't working ?
My username is strkhub, created by logging in via GitHub

Note that I get a message like this: A new confirmation email has been sent to xxx@xxx. Please check your inbox within the next 3 hours to complete the registration process. so everything makes me think it's supposed to work. Will wait these 3 hours and report you back if I still don't get the mail.

\cc @tboerger

3 hours are gone (4, at this time) but I hadn't received any mail yet.
@go-gitea/owners can you take a look at the logs (according to @tboerger on Gitter, the server is configured to send those mails)

As try.gitea.io sends emails now, can this be closed?


Refinement of comments and documentation

Solving plan:

[x] Refine whole Evaluation folder to allow tinkering of the function
[ ] Provide general comments and docs for multi-purpose headers, to give general overview, inluding Move.h header
[ ] Process Thread Management and Interface directories to allow full UCI implementation
[ ] Prepare comments for Search Directory
[ ] Prepare full docs of serch algorithm
[ ] Possibly do docs and comments for Move gen?

If there is need to have earlier Access to details and 'docs' simply mention that here


Crashing opening settings

Microsoft PowerToys version
0.58.0
Running as admin

[X] Yes

Area(s) with issue?
Settings
Steps to reproduce
Right-click on PowerToys icon and select Settings. It crashes.
Or
Double click on PowerToys icon
✔ Expected Behavior
Settings opens
❌ Actual Behavior
It crashes.
Other Software
No response

Duplicate of #18457

Do you use some optimizer software like RivaTuner?
Also could you check this ?
/needinfo

@stefansjfw Not using RivaTuner explorer.exe shows as Elevated Yes

In that case, try enabling UAC? That should fix your issue ..

@stefansjfw how do I do that?
Type UAC in windows search and open User Account Control Settings window. Move scroller to up/away from the "Never notify" (bottom option).

As far as I know, nothing has changed in that area. I have been using PowerToys for quite a while without any issues.

PowerToys has changed :) We started using WinAppSDK in the latest release. WinAppSDK  version we're using has limitation if you try to run app elevated.

@stefansjfw Changed to "notify me only when apps try to make changes to my computer (do not dim my desktop" and rebooted. Now the settings are working.

@BillDenton Thanks for the feedback. We'll be switching to the newer version of the WinAppSDK which should support running apps elevated soon


zopen port of the_silver_searcher

It would be nice if the_silver_searcher could be ported to z/OS.  It's a really fast search tool for files in the zFS, and I've used it on Linux before to dig through piles of source code for text.  It's like a supercharged grep.

@IgorTodorovskiIBM i think you have a discussion in meta on things people want ported - would it make sense to move this request there to get more eyes on it?

@AnthonyGiorgio , if you could also post this under https://github.com/orgs/ZOSOpenTools/discussions/23 .
@MikeFultonDev It would be nice if we can consolidate them and allow users to vote on the packages they want to see ported next.

@IgorTodorovskiIBM Done.


🐛 firebase_auth found internal error

While adding package i found this kind of errors which are from internal package code as attached here with
Because firebase_auth >=3.6.1 <3.6.2 depends on firebase_auth_platform_interface ^6.5.1 and firebase_auth >=3.6.2 <3.6.3 depends on firebase_auth_platform_interface ^6.5.2, firebase_auth >=3.6.1 <3.6.3 requires firebase_auth_platform_interface ^6.5.1.
And because firebase_auth >=3.6.3 <3.6.4 depends on firebase_auth_platform_interface ^6.5.3, firebase_auth >=3.6.1 <3.6.4 requires firebase_auth_platform_interface ^6.5.1.
And because firebase_auth >=3.6.4 <3.7.0 depends on firebase_auth_platform_interface ^6.5.4 and firebase_auth >=3.7.0 depends on firebase_auth_platform_interface ^6.6.0, firebase_auth >=3.6.1 requires firebase_auth_platform_interface ^6.5.1.
Because every version of flutter from sdk depends on collection 1.15.0 and firebase_auth_platform_interface >=6.4.0 depends on collection ^1.16.0, flutter from sdk is incompatible with firebase_auth_platform_interface >=6.4.0.
Thus, flutter from sdk is incompatible with firebase_auth >=3.6.1.
So, because freetimeproject_codes depends on both flutter from sdk and firebase_auth ^3.6.1, version solving failed.
pub get failed (1; So, because freetimeproject_codes depends on both flutter from sdk and firebase_auth ^3.6.1, version solving failed.)

@RjMakwana2660
Can you provide flutter doctor -v, pubspec.yaml and adding which package throws this error ?

pubspec.txt
@darshankawar here is attached pubspec.yaml
currently i have managed it with downgrading the firebase_auth version to firebase_auth: ^3.3.19

Thanks for the update. From pubspec.yaml file you shared, I see that it has flutter_test dependency which uses or depends on  collection: ^1.15.0, but firebase_auth version you were trying to add requires collection package to be ^1.16.0 and hence due to this conflict, you are getting the error. (Because every version of flutter from sdk depends on collection 1.15.0)
You will need to use latest flutter_test dependency which is dependent on your sdk version. Currently you are using sdk: ">=2.16.2 <3.0.0" which you need to upgrade to latest available which will also update your flutter_test. This should resolve your issue.

currently i have managed it with downgrading the firebase_auth version to firebase_auth: ^3.3.19

You don't need to do this. Upgrading your sdk constraint should resolve the issue.
Closing as above should work.


Add quirk for TZ3000

Adds support for UseeLink 302EU-Zigbee, based on https://github.com/zigpy/zha-device-handlers/issues/1632#issuecomment-1627072807

Hi!
Based on the description "UseeLink 302EU-Zigbee" I guess this is the device in question.
Wouldn't it be better to name the class "Plug_4AC_4USB_TZ3000_cfnprab5" to make it clear, that it has 4 USB (3 x USB-A, 1 x USB-C) ports?
I got this SM-302Ez-PA20 which looks nearly the same. But I think mine has the ability to set "child_lock" and at least to control the "power_outage_memory" (or "power_on_state") as mentioned here. Initially I integrated this power strip with Z2M on HAOS although not listed as a supported device. Pairing went well but surprisingly it showed up as SM-0306E-2W which caused the wrong picture of the device on the Z2M UI. I was able to toggle the outlets and USB ports like in ZHA and additionally could set the "power_outage_memory" to it's expected values "on", "off", "restore" from the Z2M UI and it worked as expected although I got an error message for an "UNSUPPORTED_ATTRIBUTE". The same goes for the "child_lock". Setting from the UI to both values worked as expected despite an "UNSUPPORTED_ATTRIBUTE" message. I think the behaviour matches the comment from @javicalle as the "power_outage_memory" was only visible  for endpoint 1 (and as general attribute which may be forced the error message).
To figure out what was the problem I tried the other way with ZHA, which I haven't used until then.
I got it working with your quirk, thanks for that. Toggling the 4 AC outlets and the USB ports as a group from the ZHA UI and / or from the developer tools works without issues.

Further configuration for "power_on_state", "power_outage_memory" or "start_up_on_off", or whatever you would call it, doesn't work so I extended your quirk with "quirk_id = TUYA_PLUG_ONOFF" in the hope of getting "child_lock" and "power_outage_memory" / "power_on_state" working. I got additional attributes in the "TuyaZBOnOffAttributeCluster"

but haven't been able to get it working. I couldn't set the value to "on", "off" or "restore". Initially it has the value "None".
The same is for

Any idea how to get it working, for at least endpoint 1?


Adding the prefix and version in the OpenApi Server

Here is the setup I am using. What I am trying to do is have the Server have the prefixes
https://localhost:5001/api/v1
And the endpoints be just /linkedin/person
Is there a way to do that? Attached screenshot of current OpenApi json.
 #region API
        // Add your API key authentication scheme
        services.AddAuthentication()
            .AddScheme<AuthenticationSchemeOptions, ApiKeyAuth>(ApiKeyAuth.SchemeName, null);
        services.AddEndpointsApiExplorer();
        services.AddFastEndpoints();
        services.SwaggerDocument(o =>
        {
            o.MaxEndpointVersion = 1;
            o.EnableJWTBearerAuth = false;
            o.ShortSchemaNames = true;
            o.ExcludeNonFastEndpoints = true;
            o.DocumentSettings = d =>
            {
                d.Title = $"{Configurations.Settings.SiteName} API";
                d.DocumentName = "v1";
                d.Description = SiteApi.ApiDescription;
                d.AddAuth(ApiKeyAuth.SchemeName, new()
                {
                    Name = ApiKeyAuth.HeaderName,
                    In = OpenApiSecurityApiKeyLocation.Header,
                    Type = OpenApiSecuritySchemeType.ApiKey
                });
                // d.PostProcess = p =>
                // {
                //     p.Servers.Clear();
                //     p.Servers.Add(new OpenApiServer { Url = $"https://{Configurations.Settings.ApiHostName}/api/v1" });
                // };
            };
        });
        #endregion

  app.UseEndpoints(endpoints =>
        {
            endpoints.MapFastEndpoints(o =>
            {
                o.Versioning.DefaultVersion = 1;
                o.Versioning.Prefix = "v";
                o.Versioning.PrependToRoute = true;
                o.Endpoints.RoutePrefix = "api";
                o.Endpoints.Configurator = c =>
                {
                    c.AuthSchemes(ApiKeyAuth.SchemeName);
                    c.Throttle(3, 1, ApiKeyAuth.HeaderName);
                    c.Description(b =>
                    {
                        b.ProducesProblemFE(403);
                    });
                };
            });

        app.UseOpenApi(o =>
        {
            o.Path = "/openapi/{documentName}.json";
            o.PostProcess = (document, request) =>
            {
                document.Servers.Clear();
                document.Servers.Add(new OpenApiServer { Url = $"https://{Configurations.Settings.ApiHostName}" });
            };
        });

I have tried adding it to the following. But it just duplicates /api/v1 in both the server and the path.
        app.UseOpenApi(o =>
        {
            o.Path = "/openapi/{documentName}.json";
            o.PostProcess = (document, request) =>
            {
                document.Servers.Clear();
                document.Servers.Add(new OpenApiServer { Url = $"https://{Configurations.Settings.ApiHostName}/api/v1" });
            };
        });

your startup config is incorrect.
you should not be using AddEndpointsApiExplorer() and UseOpenApi() calls.
please see doc page for the correct setup.
you just need to do this:

as for customizing the swagger paths, have a look at this gist.


Advanced Constraints

Create constraints that allow more freedom of placement.
Implementation: non-linear Optimisation
scipy offers a range of optimisation methods.
https://docs.scipy.org/doc/scipy-0.14.0/reference/generated/scipy.optimize.minimize.html
Optimisation methods could be defaulted, then optionally fine-tuned by advanced users, all through one common interface (scipy).

Inspiration for constraints... naming observed from official OnShape and Solidworks tutorials:
Solidworks

angle, coincident, concentric, distance, lock, parallel, perpendicular, and tangent mates
limit, linear/linear coupler, path, symmetry, and width
cam-follower, gear, hinge, rack and pinion, screw, and universal joint

OnShape

Fastened
Revolute
Slider
Planar
Cylindrical
Pin Slot
Ball

Todo List
cleanup tasks

[ ] remove todo box from Constraints

Clarification from @dcowden (paraphrased slightly)

Fixed: orient one mate to a fixed position and orientation in space ( DONE: LockConstraint) DOF: 6
Coincident: orient two mates, matching both orientation and position ( DONE: RelativeLockConstraint ) DOF: 6
Cylindrical: orient the origin of two mates, and fix an axis of rotation on the z axis ( DOF: 4 )
Revolve: orient two mates to share an axis of rotation ( cylindrical, but without matching the origin) (DOF: 1 )
Parallel: orient two mates so that their x-y planes are parallel, with some fixed offset. if offset is zero, the planes are coplanar, otherwise, they are offset by some amount ( DOF: 2 )
Coplanar: a Parallel constraint with a zero offset
Slide: orient two mates so that their z axes align, and can only slide along that direction (DOF: 1 )
Spherical: align the origin of two mates, but allow them to rotate freely otherwise ( DOF: 3 )
Slot: orient two mates so that the origin of one mate must lie on one selected axis of another, and must not rotate ( DOF: 5 )

I think this is as good a list as any, it makes perfect sense to me on first reading, and it comes from an experienced modeller & programmer!
@dcowden: you mentioned potentially allowing a change of axis.
For example, the Cylindrical could revolve around the y-axis instead of the z-axis.
I think I like the simplicity of restricting how the constraints are implemented. It also makes documenting them much easier, and therefore more likely to be understood by laypeople.
But that wouldn't mean the creations themselves are limited, authors can create a new mate by getting an existing one and rotating the mate itself.
With that in mind, I'll write the code now so you can add a CoordSystem to a Mate... for example
mate = Mate(component, CoordSystem((0, 0, 10)))
new_mate = mate + CoordSystem(xDir=(0,1,0), normal=(1,0,0))

What I'll do before initial release:
I want to remove "Constraint" from the end of each constraint, it's unnecessary, and a bit oldschool.
If authors want that to be clear they can write from cqparts import constraint and constraint.Fixed
I'll also rename LockConstraint to Fixed, and RelativeLockConstraint to Coincident

@fragmuffin sounds good.
This is a picture of the assembly toolbar for Onshape-- its very similar to Solidworks. You can see the icons for the kinds of mates.

The first icon to the right of "Insert" is the icon they use for a mate.

Are you also thinking about constraints between parts/assemblies? I'd love to have some way to constrain parts to be separated by lengths or angles.

@SeanDS : Yes! absolutely. constraints will be between any 2 Components, which includes both Part and Assembly

Hi, I have a proof of concept for a flexible solver using the scipy least square root solver. It is working quite nicely and I can apply multiple constriants to any part.
For now I have to following constraints working:

Origin: constrains origin and direction of a part to a CoordinateSystem()
OriginDir: constrains only the orientation of a part to a CoordinateSystem()
Position: constrains position and direction of two mates
Direction: constrains only the direction of two mates
Plane: constrains one mates position to be located in a plane parall to the seconds mate plane

Next steps would be to clean it up and integrate it into the cqparts code. This would mean to replace the current solver.solve method the Assembly.build function and to add new constraint classes.
My questions:

is it desired that I work on this?
is anyone else currently working on this?
can I modify the Assembly.build function or is anyone else currently working on this?
which branch should I fork to minimize merge conflicts?

This is so awesome!

@everlanes: I don't have time right now to check this out, but it sounds awesome! There seem to be very few Pythonic CAD libraries that support constraints, so having these in cqparts would be great. Thanks for working on this, and I'll let you know if I have any feedback when I try it out :-)

I have uploaded a first draft for a numeric solver with flexible constraints in #158
All tests are passing in my development environment, but sadly not with AppVeyor - OCC dependency is missing and scipy dependency should be added. I'm just not sure, where to add these dependencies.


.../tensorNet.h(183): error: expected an identifier

Hi,
I'm trying to integrate the segNet example in another project and I get this error I don't manage to understand :
".../tensorNet.h(183): error: expected an identifier".
The corresponding code is :
struct outputLayer
{
std::string name;
Dims3 dims;
uint32_t size;
float* CPU;
float* CUDA;          <--- line 183
};
I managed to build and execute the example alone and I haven't changed anything in tensorNet.h/.cpp, neither in segNet.h/.cpp during the implementation so I have no idea what could have brought this issue.
Any idea ?
Thanks a lot !

Hello, if you pull from master using the latest JetPack on your TX1/TX2, you should now get it to build ok.


AxiStreamMonitor / AxiStreamSink: idle() not working as expected

While working with the AxiStreamMonitor / AxiStreamSink classes, I had the impression that the function 'def idle(self)' of these classes does not work as expected. I want to use it, to check if the bus is idle, all packets have arrived and the test can be terminated.
The function is based on the self.active variable, but I have not found a situation when it would be set to true.
Did I miss something?
Thank you

Hmm, yes, it looks like that's not implemented properly.  I pretty much exclusively use send and recv so that code path has basically never been tested.  I'll take a look at getting that implemented properly.

Try the current version and see if it does what you need.

Great! Works as expected!
Thanks a lot


Document the ABS-based finality gadget

Document the current status of the Block Relay and its API.

Closed by #47


setState on afterValidate hooks

Description
We have many questions about using setState in hooks afterValidate - users want to change  state after validation. We need investigations why it doesn't work. And fix it if it's a mistake. If it's not a mistake, it's enough to create a working demo.
Steps to reproduce

https://github.com/handsontable/support/issues/116#issuecomment-497224262

Your environment

React wrapper version:
Handsontable version:
Browser Name and version:
Operating System:

I have a workaround for this issue.
You can see the jsfiddle here
The key is to store the invalid cells using row and prop/col into this.state.invalidCells as a dictionary.
At componentDidUpdate, re-mark the cells as invalid based on this.state.invalidCells
That will solve the issue

Thank you for sharing @simkimsia

Hi @simkimsia,
We merged wrapper for React into our main repository, handsontable/handsontable. Now library and wrapper are available in monorepo.
We also transfer all issues into one common issue board, where you can follow work progress and request issues.


Feature request: support refreshing when an endpoint returns 200

What version of Livereload are you using?
0.9.1
What OS are you using?
MacOS
Details
I often use livereload along with nodemon to cause refreshes on server / client whenever a backend or frontend file changes.
My particular use case: I have a hapi server that serves static files and handlebars-rendered HTML. Webpack compiles SASS and minifies a JS file. When I change a JS file, nodemon refreshes Hapi. When I change CSS or dist/app.js, livereload refreshes the browser.
The issue is that a delay option will vary computer to computer, and needs to increase as the app restart time increases from added complexity. Instead, could there maybe be an option to livereload after, say, http://localhost:4000/ returns 200? EG:
livereload --endpoint "http://localhost:4000/" --backoff 1000 [...options]

I think this is getting far too out of scope for what this program is designed to do, so I think I'm going to decline this.
You can integrate livereload into your dev server if you want more specific behavior.
Alternatively, there are tools higher-up the stack that provide what you're looking to do, like browser-sync.


New version: AntibodySoftware.WizTree version 4.20

Created by 🥟 Dumplings in workflow run #6940.

Log
Updated: 4.19 → 4.20
Submitting WinGet manifests
Creating manifests
WinGet exits with exitcode -1978335231
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run


[QUERY] Workflow Execution Authority for Guest Users

Hi
When a multi-channel-guest user in a workspace runs an app (shortcut trigger) created in the next-gen platform, they get the message This workflow can't be run by external teams".
I checked the command reference but couldn't find a solution. Is there any way to work around this and have the guest user start and run it?

May be relevant to standalone workspaces: assuming you have Admin access to your workspace, under App Management Settings (.slack.com/apps/manage/settings), there is a "Multi-Channel Guests can only use default slash commands and shortcuts" option. Can you validate that this option is OFF?

I'm trying to solve this problem in a workspace that is part of a larger enterprise / org / grid but I tried subscribing to the pro plan on my own slack account to see if the same problem with a Single, standalone workspace.
When I execute a workflow deployed with the Next-Gen Platform as a guest user, I get the message "This workflow can't be run by external teams".(※1) The option "Multi-Channel Guests can only use default slash commands and shortcuts" is OFF.
The workflow created by the Workflow Builder runs fine.(※2)
Please note that I'm trying this with single channel guests due to the price of the slack pro plan.
※1 execute workfrow create by Next-Gen Platform App

※2 execute workfrow create by Workflow Builder

I did some digging internally and indeed, this is something that we are currently limiting due to feedback we received from larger enterprise administrators. Initially, guests had access to run workflows, but admins got nervous, so we felt it was safer for our customers to turn this off.
We are working on a 'trusted team' / 'trusted user' feature that should address this issue.

Thanks for looking into this.
I understand the background. I look forward to the day when that feature is implemented!

@sukekyo26 can you provide details as to what steps are in each workflow? You said the same workflow, but one built with code vs. the other built in workflow builder, have different runtime permission behaviour. That seems odd to me. Can you tell me what steps are used in each? And if a custom function is used in either, what scopes that function requires? I want to be 100% sure that the workflow-builder-built vs. coded workflows are identical.
FYI we also released a tweak to behaviour here for guest users running coded workflows last week, to align workflow builder-built and coded workflows better. Perhaps give this another shot in case that fixes things for your specific use case?
Note that this particular area - the specific runtime permissions for guest and away users for workflows using 'risky' scopes - is in active development, as we've identified potential areas of abuse in this space. I would expect further future communications in this area as I believe we can expect changes incoming for this general use case.

@filmaj

can you provide details as to what steps are in each workflow? You said the same workflow, but one built with code vs. the other built in workflow builder, have different runtime permission behaviour. That seems odd to me. Can you tell me what steps are used in each? And if a custom function is used in either, what scopes that function requires? I want to be 100% sure that the workflow-builder-built vs. coded workflows are identical.

Sorry for the misunderstanding.
the workflow-builder-built vs. coded workflows is not the same thing. I indicated that workflow-builder-built workflows can be executed by guest users as opposed to coded workflows. However, I now feel that this contrast is misleading. My apologies.
Also, at the time I opened this issue, I have confirmed that coded workflows without custom workflow steps can be executed by guest users.
FMI, My coded workflow steps are as follows.
Trigger: Link trigger
Workflowstep1: custom function(Modal operation)
Client in use: views.open, views.update
Workflowstep2: custom function(Send message)
Client in use: chat.postMessage
Workflowstep3: custom function(Send thread message and modal operation)
Client in use: chat.postMessage, views.open, chat.update

FYI we also released a tweak to behaviour here for guest users running coded workflows last week, to align workflow builder-built and coded workflows better. Perhaps give this another shot in case that fixes things for your specific use case?

Thank you very much! I will try it.

Hello ! Update here: guests should have the ability to execute workflows using 'risky' steps once more. We are still working on solidifying which exact scopes we consider risky for Guest users, so we have not documented specifically which scopes would be considered risky at this time. However, I can say that your workflow as listed above should now be executable by guest users.
I will close this issue but if you have further questions or problems feel free to re-open or file a new issue.


Add equivalences from CR<Pauli> to R<Paulis>

Summary
Add equivalences from controlled Pauli rotations to 2-Pauli rotations. This is particularly interesting e.g. for QPD applications where cuts are usually defined for 2-Pauli rotations or for compiling to the RZX gate and using a pulse-efficient decomposition. This could e.g. be used to compile a CRY gate to an efficient pulse sequence instead of 2 CX gates.

It's not really related to this PR, as a standalone these equivalences are fine on their own. But one thing we are missing in the equivalence lib (it came up on slack) is targeting a CR<Pauli> basis. I think to fix it we can just define a rules from cx to the crx, etc to fix that.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

64 of 64 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 85.244%

Totals

Change from base Build<PHONE_NUMBER>:
0%

Covered Lines:
67097

Relevant Lines:
78712

💛  - Coveralls

@mtreinish I opened an issue to keep track of these missing equivalences: https://github.com/Qiskit/qiskit-terra/issues/9511


Doesn’t work in new iOS versions

I have my iPad at iOS 13.3.1 and it never loads now. It doesn’t load for iOS 13.3 either

This appears to be a duplicate of #123
It is also the same issue that I had on my fork: https://github.com/brandosha/gameplaycolor/issues/4

Thanks @brandosha. It is indeed a duplicate of #123. Closing as such.


How to work with RisingEdge in functions?

Hello!
I have my own test example and it's working nicely with my Verilog code when all timing control lies in main @cocotb.test() function. This code section
for cycle in range(10):
	yield posedge(dut.clock)
	print "* test=", int(dut.test)

works fine and I see tick message in the console, Verilog code also producing output. Now I want to split this test to run in functions with parameters, however neither of test1() or test2() functions calling at all! I see the following output:
# !try test1
# !end test1
...
# !try test2
# !end test2

Why I do not see any tick and functions test1 & test2 just not called at all?
I know that multiple @cocotb.test() for different tests - but how to parametrize them? I want to run single test with different parameters - that's why I want to see them in functions...
Full code:
import cocotb
from cocotb.triggers import Timer as ps
from cocotb.triggers import RisingEdge as posedge
from cocotb.result import TestFailure as failed

@cocotb.coroutine
def clock_gen(signal, period = 1000):
	while True:
		signal <= 0
		yield ps(period / 2)
		signal <= 1
		yield ps(period / 2)
		print "tick"

@cocotb.test()
def test_cocotb(dut):
	dut.log.info("test_cocotb start")
	cocotb.fork(clock_gen(dut.clock))

	print "!try test1"
	test1(dut)
	print "!end test1"

	for cycle in range(10):
		yield posedge(dut.clock)
		print "* test=", int(dut.test)

	print "!try test2"
	test2(dut)
	print "!end test2"

	raise failed("test failed " + str(int(dut.test)))
	dut.log.info("test_cocotb finished")

def test1(dut):
	print "----- 111"
	for cycle in range(15):
		dut.log.info("tick1")
		yield posedge(dut.clock)
		print "1 test=", int(dut.test), "cycle=", int(cycle)

@cocotb.coroutine
def test2(dut):
	print "----- 222"
	for cycle in range(10):
		dut.log.info("tick2")
		yield posedge(dut.clock)
		print "2 test=", int(dut.test), "cycle=", int(cycle)

I found workaround:
print "!try test1"
yield test1(dut)
print "!end test1"

for cycle in range(10):
yield posedge(dut.clock)
print "* test=", int(dut.test)

print "!try test2"
yield test2(dut)
print "!end test2"

Why this yield required? Anyway, problem solved. Hope this helps to other newbies like me :)


Compatibility with 1.5

Confirm all functionality works with FVTT 0.7.10 and FFG SW 1.5.
Creating as a result of #48

[ ] Settings
[x] Intro Crawl
[x] Datapad creator
[x] Bounty creator
[ ] Shop
[ ] Attack animations
[ ] Talent checker
[ ] Combat renamer
[ ] Dice helper
[ ] Strain remainder

The vendor default size appears to be too small for the shop content:

Looks like there are no breaking changes. Easier than expected. :)

Hi there! I'm looking to try out your module on FVTT v.0.7.10, FFG v. 1.5; however, one of the modules which is a dependency of yours (foundryvtt-fxmaster) throws an error requiring a core FVTT update to 0.8.6 to function. Naturally, since FFG isn't compatible with 0.8.6 yet, this makes installing your module more difficult. I'm able to install your module by installing fxmaster v1.0.4. Is there something in the installation instructions that I have missed to avoid having to downgrade to this version? Are there other solutions for this problem?
Thanks in advance for your time, and my apologies if this problem has been addressed already or if I have posted this in the wrong section :)

I'll write up a guide on installation steps in the wiki to try and make this less painful - beyond that, there isn't much that can be done - I can't pin specific versions of modules as dependencies and I'm not super happy about the idea of investing a lot of time into working around that limitation when support for 0.8.x should be forthcoming in the near future.
Thank you for the comment and it's no trouble at all. Feel free to DM me on Discord if you have more thoughts on the matter.


MacOS file type filter in native file dialog

What does the pull request do?

Adds file type chooser to the native file dialog
Fixes support for file extensions
Ensures file formats priority to be "File extension > Apple Type UID > MimeType".
Default file extension is also respected as default selected file format + macOS automatically appends this extension to the file name.

What is the current behavior?
It's not possible for the user to select file type filter.
File extension file filters don't work.
How was the solution implemented (if it's not obvious)?
https://github.com/chromium/chromium/blob/49f63f17ac50384e9f367622dd5baf9701d4da3f/components/remote_cocoa/app_shim/select_file_dialog_bridge.mm#L65
Fixed issues
Fixes #5216

https://github.com/AvaloniaUI/Avalonia/assets/3163374/fb8121d1-c2f2-4046-af9e-535889135dfa

You can test this PR using the following package version. 11.0.999-cibuild0039550-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0040072-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0040084-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0040146-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]


don't load raw comments for invalid serialized ranges

When the symbol graph tool is run over a framework with source info that points to nonexistent files, it will eventually try to load RawComments from the file module, even though all its ranges don't have source files to load from. This causes a situation where it hits an assertion in the RawComment initializer and crashes the tool. This PR aims to sidestep the error by only calling the RawComment initializer for valid ranges.
I couldn't figure out how to write a test for this, so i could only verify it by running the compiled tool over the framework that previously caused the crash. While that worked out, it doesn't mean that this can continue to be verified in the future. Like i mentioned earlier, i'm not entirely sure what's going on; the frameworks i was given had .swiftsourceinfo files in them, but something about them meant that they couldn't get loaded. The person who opened the issue (linked below) was handed the frameworks by the team that manages the library, and didn't know how they were created. The source library is open-source (CareKit), so if we need to, we can ask them directly. If you know more about how to mess with these source-info files so we can put something into a SymbolGraph test (or can pass it off to someone who does), let me know! I've been able to reproduce this in a test, which is now part of this PR. The issue isn't about scrubbed source info, it's about source info that refers to files that don't exist on the current system.
Fixes rdar://73833411

I've been able to reproduce this in a test, which is now part of the PR. The issue is about source information that points to files that don't exist on the system. By moving the source file between compiling the module and running the symbol graph tool, the issue can be reproduced.

@swift-ci Please smoke test

@swift-ci Please smoke test

Windows CI error seems to be a failed clone, let's see if i can get it to try again...
@swift-ci Please smoke test Windows platform

Windows CI looks like it passed all the tests.


Did anything changed in popover syntax since last release?

Hi again,
After a bit of struggle to build a UMD from the latest sources (after discovering that the docs are outdated, i followed what's indicated in #359 ), i now have a problem with popovers.
Popovers using data api are working well, however those who were declared through JS no more work. I previously declared my popovers using this exact syntax (the syntax used in bootstrap native docs for v3):
new Popover(inputs[i],{       container: "body",       placement: "top",       trigger: "manual",       title: "Attention!",       content: "Verrouillage majuscule actif"     });
It worked like a charm when i was using the latest available release, but now firefox developer throws me this error:
ReferenceError: Popover is not defined
The error location corresponds to "Popover" in the previously mentioned code.
I think i'm probably issuing a lack of documentation following a syntax change. Am i doing something wrong?

If you're doing a custom build make sure you have everything you need, but before that, make sure it works with the default build. Do some console.log before, after and in between.

Your options values are pretty off the line, you should check the docs (the options part of the documentation is the same basically).

After checking the docs, it seems i used some twbs options. I removed them but it's still not working. The error is not about a wrong parameter or something like that, it clearly says that the Popover class does not exist by the time i try to make a new popover object out of it.
However this exact same script works pretty well with the latest standard bsn release.
Actually, it is not really a custom build: i just tried to do a UMD build with what's inside the current master branch with the stock index.js file.
npm run custom INPUTFILE:src/index.js,OUTPUTFILE:dist/bootstrap-native-custom.js,MIN:false,FORMAT:umd
Except this, the build runs fine and so far all the functionalities i have tested are working well. No errors occured during the build process. As i mentionned, popovers initialized through DATA API are working well, i only can't initialize popovers through JS.
The javascript files are loading in the correct order (BSN then my custom JS), i don't understand why this class isn't defined at this point.

I found the solution (by instinct, nothing scientific here)!
I needed to declare my popover this way:
new BSN.Popover(inputs[i],{ container: "body", placement: "top", title: "Attention!", content: "Verrouillage majuscule actif" });
instead of:
new Popover(inputs[i],{ container: "body", placement: "top", title: "Attention!", content: "Verrouillage majuscule actif" });
Probably something has changed since the last release concerning this. I don't know if this behavior is intended.

No, it's the way our build script handles ES6 source and export everything to global, something I will document carefully.

Ok, then i guess all classes should now be contained inside the BSN object.
I'll close the issue.


Allow setting initial viewport before displaying image

Fixes #33.
This change allows setViewport to set viewport on enabled elements with undefined images. Glad to make any changes you'd like, just let me know.

Noticed the code climate issues. Please let me know what you think about fixing those.

Coverage decreased (-0.04%) to 43.97% when pulling 89551a92759c3df604dfec791fcfbe5b5cb4be7c on ComputerComps:new_api_endpoint into ba393a18a0367760ded0247df5c0f2c3cea9da4b on cornerstonejs:master.

Thanks for the PR! I'll try to get to this ASAP.

friendly ping @swederik

Could you add tiny test for the new functionality?

setViewport should be able to set a viewport before an image is displayed

Then I guess it's good to merge.

Thanks!


Create a table in the database for Movie details

Table Name : Movie
Fields:
movie_id (Primary Key)
title
genre
director
release_date
CREATE TABLE Movie (
movie_id INT PRIMARY KEY AUTO_INCREMENT,
title VARCHAR(255) NOT NULL,
genre VARCHAR(255) NOT NULL,
director VARCHAR(255) NOT NULL,
release_date DATE NOT NULL
);

This implementation gives the user about the clear information of the movie details like release date, director, genre.

With this implementation, developer can extra information about the movie so makes the users easy to know about the cast and crew details.