LineEndingInContentFixer

HI!
as original fixer stop changing the content (very good!) after merge of #2209, we need a second, risky fixer that will fix the line ending in content

@gharlan , it does not look like a big one to make, and you already claimed that you would use it.
Could you send a proposal please ?

This would be a copy-paste of the linefeed fixer as on 1.11 with a new name?
Or do you want a configuration option on the current (patched) version of the fixer?

It should work on tokens level and touch non-whitespaces only (to not duplicate fixer).
I wouldn't made it one fixer, as this fixer will be risky, while existing one is not.


Add license scan badge

The CNCF requires that we adopt either Snyk or Fossa for license scanning. We tried Snyk, but it wasn't ideal - it bundles security and license scanning, and both were high-noise.
Fossa is much lower-noise. This project is already onboarded through the Fossa UI, with a clean bill of health. This PR contains some finishing touches:

Adding a badge to the README
Adding a license check to PRs

The Fossa action is currently failing with permissions problems. I've reached out to our CNCF contacts to understand what we're supposed to do here.

Correction: we are using Fossa (not Snyk), but decided that we don't need the badges.


kubeadm HA guide: fix indentation and use EOF instead of EOL

This includes two changes to the kubeadm HA guide:

Use EOF instead of EOL for heredocs. This is purely for consistency with the rest of the docs/website (I couldn't find any other instances of EOL being used).
Fix indentation to be 3 spaces instead of 4. This is important because having indentation be 4 spaces for code (shell) snippets results in a leading space in the snippet. This means that a copy/paste of a heredoc will not work directly. It also means that commands may not appear in your shell history (depending on the shell, of course).

docs/admin/authorization/rbac.md is a good reference for an existing file that has proper formatting.
I ran make stage and verified that I can copy/paste heredocs and they work without modifications now.
cc @jamiehannaford

This change is

/assign @jamiehannaford

Fixed conflicts.

/approve


I'm having troubles with Pyodbc on my macbook m1

File "/Users/paulogentil/django-maker/BackendCreator/MapFramework/MapDb.py", line 3, in
import pyodbc as dtb
ImportError: dlopen(/Users/paulogentil/patovenv/lib/python3.7/site-packages/pyodbc.cpython-37m-darwin.so, 2): Library not loaded: /usr/local/opt/unixodbc/lib/libodbc.2.dylib
Referenced from: /Users/paulogentil/patovenv/lib/python3.7/site-packages/pyodbc.cpython-37m-darwin.so
Reason: image not found
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
I tried to install unixodbc and configure the odbc.ini file but nothing happens
Someone can help me

Make sure you are consistently using ARM or x86 binaries throughout. Your Python, pyODBC, unixODBC, and ODBC driver all have to be of the same architecture. Mixing them has been the cause of a few other issues here. https://github.com/mkleehammer/pyodbc/issues?q=M1

I follow that steps:
1-I install the pyodbc package manually (https://pypi.org/project/pyodbc/#files)
2-I install unixodbc
3-I run these commands:
pip uninstall pyodbc
export CPPFLAGS="-I/opt/homebrew/Cellar/unixodbc/2.3.11/include"
export LDFLAGS="-L/opt/homebrew/Cellar/unixodbc/2.3.11/lib -liodbc -liodbcinst"
cd /Users/paulogentil/
pip install pyodbc-4.0.32.tar.gz
After all those steps i'm having a new error:
zsh: abort      python main.py


Update project name in api/README.md.

Coverage remained the same at 81.315% when pulling a12419151e072df03fc1de5f5fba6926c5005579 on sebright:instrumentation into 6e1ab48620aef90ba897fb5a7e93185a4a015103 on census-instrumentation:master.

Codecov Report

Merging #866 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #866      +/-   ##
============================================
+ Coverage     79.34%   79.37%   +0.02%
  Complexity      908      908
============================================
  Files           156      156
  Lines          4014     4014
  Branches        378      378
============================================
+ Hits           3185     3186       +1
  Misses          750      750
+ Partials         79       78       -1

Impacted Files
Coverage Δ
Complexity Δ

...census/implcore/trace/export/SpanExporterImpl.java
98.33% <0%> (+1.66%)
7% <0%> (ø)
: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 6e1ab48...a124191. Read the comment docs.


add reset_all option to hsctl script

Add an option to fully reset the environment back to a clean state

Remove any files created by the application when it was running
Stop and remove any running containers
Remove any hydroshare images (hydroshare, defaultworker, dockerworker)

merged


Cannot seem to get a "data connection" to a github repo

Trying to create a data connection, will not accept the git clone URL and returns "Unable to fetch repository files: 404 {"message":"Not Found","documentation_url":"https://docs.github.com/rest/repos/contents#get-repository-content"}"
If I just paste in the url from the address bar then it will import 2 files only, .gitignore and readme.md

https://github.com/Mintplex-Labs/anything-llm/issues/1296
With no PAT the repo grabber will only get the top-level files and will not deep-read the files because of API rate-limits


feature rename JSONB type name and function name to JSON

Proposed changes
Issue Number: close #xxx
Problem summary
To be more compatible with MySQL, rename JSONB type name and function name to JSON.
The old JSONB type name and jsonb_xx function can still be used for backward compatibility.
There is a function jsonb_extract remained since json_extract is used by json string function and more work need to change it. It will be changed further.
Checklist(Required)

[ ] Does it affect the original behavior
[x] Has unit tests been added
[x] Has document been added or modified
[ ] Does it need to update dependencies
[x] Is this PR support rollback (If NO, please explain WHY)

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

run buildall

run buildall


user list inside tabpane

hi,
i have admin lte 2.3.8 and i have this code (user list) inside a tab pane  but the images dont format well
code here ( because github code wont show html code) https://jsfiddle.net/a37u81y8/
it lookes strange http://prntscr.com/h8u4a6
Thanks for your help

Images should be the same size and have square dimensions (equal height and width. e.g, 100X100)

Cool that works
But i have another litle problem wich is when i have the window in full size the space between 2 images next to other is too much but when i resize the windows seems fine


Global custom reactions are not enabled by the custom reactions module.

If you have all server modules disabled, then enable the customreactions module for a specific channel, global reactions will still be blocked.
Steps to reproduce:
Setup your permissions like this:
3. cm customreactions enable #testchannel
2. asm disable
1. asm enable
add a global reaction
try to use that reaction in #testchannel.
Result: if verbose is on, NadekoBot will say that 2. asm disable is preventing the command.
Workaround: enable all modules on the channel then manually disable all the other modules.
aum and arm also allow the global reactions to be used.

Try using .cm actualcustomreactions enable
This one is like a hidden Module for reactions.

^


Allow removing uninitialized mounts

Previously we were overly finicky when removing mounts.
If a mount isn't mounted or initialized we shouldn't care.
If it's in the config remove it, if not don't.
Fixes #1746
Signed-off-by: Dominik Schulz<EMAIL_ADDRESS>

Yes, reproduced and tested locally.

Yes, reproduced and tested locally.


add compare runs tests

closes: https://issues.redhat.com/browse/RHOAIENG-8500
Description

add cypress tests for compare runs

markdown
roc cure
confusion matrix

How Has This Been Tested?

tests pass
Test Impact

coverage to metrics section of compare runs
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

/approve


New Command System

@Gregyyy src/main/java/tech/ypsilon/bbbot/discord/command/LegacyCommand.java: Which use has the last EmbedBuilder in the last lines without any message with this builder sent?

I forget to add 'e.getChannel().sendMessage(embedBuilder.build()).queue();'. I will push to this branch later to fix it.

Don't push to the branch.. Got merged into dev branch pls use the dev instead


Connect Directly to Kafka Brokers

My company has provided a Kafka environment in AWS, but our DevOps team insists that we should be connecting directly to the Kafka Broker (e.g. 9092). I am able to connect with this approach using kafkacat, but cannot with node-kafka.
The following works...
kafkacat -b my-amazon-remote-url:9092 -L

But, when I run my code using kafka-node, it does not work.
const kafkaNode = require('kafka-node');

const client = new kafkaNode.Client(
 'my-amazon-remote-url:9092',
  'test-client'
);

const producer = new kafkaNode.Producer(client);

producer.on('error', (err) => {
  console.log('NEVER GETS HERE', err);
});

producer.on('ready', () => {
  console.log('NEVER GETS HERE');
});

Does kafka-node support connecting to the 9092 ports?
kafkacat CAN connect this way, so is this just something that has not been implemented in kafka-node or is it simply the wrong way to connect to Kafka? Why is my DevOps team insisting that we use the 9092 port type connection? Would there be a reason?
Even when I attempt to connect in the above example using the zookeeper port, I still cannot connect to the remote version.

const client = new kafkaNode.Client(
 'my-amazon-remote-url:2181',
  'test-client'
);
...

Still no luck, although the above does work on my local Kafka instance (installed on Mac OSX following standard instructions on Kafka.org.)
I appreciate any help on this issue? Thank you!
Kevin

Hi @kevinswarner Kafka is turning away from zookeeper so I've been working on a local branch with a kafka only Client. Unfortunately, it's not complete yet.  I'll try to get some more time to work on it.

Hello,
It looks like I might have a similar requirement, might I ask how you're getting on with your branch?
Would you need some help?

Thanks for the offer @vidhill.
I've been making good progress and pushing updates to kafka-client. It's usable and it passes the tests I've written but I haven't tested in any deployed environments yet.  I would like to get more testing and feedback from others before submitting a PR.
To test

Point kafka-node in package.json to SOHU-Co/kafka-node#kafka-client
If using a ConsumerGroup change host to kafkaHost

Ok, I will give that a go, it may be a few days before I get back around to this task, but I will report back when I have tried it.
Many thanks

I will also test this during the week and add some feedback on friday.

Anybody had a chance to run the new Kafka Client?

Hi ,
I am also facing problem in connecting to client.
But I am able to connect to zookeeper port 2181
But devops team suggest to connect to kafka 9092
How to achieve it with this module.
I have tried the above solution but it didn't work

if you are instantiating your own client use KafkaClient instead of the Client. If using the GroupConsumer specify a kafkaHost instead of host. This should be covered in the README.

Below is the code I am using through kafkaClient , Still no success , I get an error saying connection refused
import kafka from 'kafka-node';
const Producer = kafka.Producer;
const client = new kafka.KafkaClient('<IP_ADDRESS>:9092');
const producer = new Producer(client);
producer.on('ready', () => {
producer.send(payloads, (err) => {
if (err) {
console.log(err);
}
});
});
producer.on('error', (err) => {
console.log(err);
});
});

@khushbookatta in the README for KafkaClient states:

Constructor accepts an single options object (see below)

So try
new kafka.KafkaClient({kafkaHost: '<IP_ADDRESS>:9092'});

Can you add an example to the KafkaClient README? For me it wasn't obvious that arguments have to be passed as an object (altough now that I saw your example above, I also noticed that it says in the Notable Differences Constructor accepts an single options object (see below)). I spent way to much time on making this connection to the broker work :)

Thanks for your help . It worked for me . It was mentioned clearly in the readMe file but still I missed it some how, in the hussel of other things.
I can add an example in the README file .
Thanks
Khushboo Katta
Get Outlook for Androidhttps://aka.ms/ghei36
From: Zala Herga
Sent: Friday, November 10, 7:55 PM
Subject: Re: [SOHU-Co/kafka-node] Connect Directly to Kafka Brokers (#666)
To: SOHU-Co/kafka-node
Cc: Khushboo Katta, Mention
Can you add an example to the KafkaClient README? For me it wasn't obvious that arguments have to be passed as an object (altough now that I saw your example above, I also noticed that it says in the Notable Differences Constructor accepts an single options object (see below)). I spent way to much time on making this connection to the broker work :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/SOHU-Co/kafka-node/issues/666#issuecomment-343486448, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATIWesreKOLr8wKm1kT5hCTzrwGxgijlks5s1FzDgaJpZM4NRPhu.

this code worked for me
Broker brokerForPartition1 = new Broker("ip",9092);
GlobalPartitionInformation partitionInfo = new GlobalPartitionInformation("Topic name");
partitionInfo.addPartition(0, brokerForPartition1);
BrokerHosts temp = new StaticHosts(partitionInfo);


Protected domains separator ambiguous

Comma vs. pipe
From the webpage:
Protected domains
Specify the domains used for identities in your environment. All traffic to the fully-qualified domains appearing in this list will be protected. Separate multiple domains with the "," delimiter.

From the Intune built-in help (tooltip):
Specify the DNS names that form your corporate network. These are used in conjunction with the IP ranges that you specify to define your corporate network boundary. Multiple values can be specified by separating individual entries with a '|'.

This setting is required to have Windows Information Protection enabled.

For example: corp.contoso.com|region.contoso.com

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

ID: c1ad3f92-24de-5eb5-8454-c72d85737183
Version Independent ID: 900125eb-31ea-faf3-ac30-7c3d7b30f611
Content: Create a Windows Information Protection (WIP) policy with MDM using the Azure portal for Microsoft Intune (Windows 10) - Microsoft 365 Security
Content Source: windows/security/information-protection/windows-information-protection/create-wip-policy-using-intune-azure.md
Product: w10
Technology: windows
GitHub Login: @DulceMontemayor
Microsoft Alias: dansimp

@velden
With the help of your feedback, the content has been updated with relevant changes accordingly. Thanks.


request help: Mac os can't start apisix

start apisix
./bin/apisix start
nginx: [warn] could not build optimal variables_hash, you should increase either variables_hash_max_size: 1024 or variables_hash_bucket_size: 64; ignoring variables_hash_bucket_size
how can I fix it?
Environment

apisix version (cmd: apisix version):1.4
OS:macos Mojave  10.14.6 (18G103)

This bug has been around since OpenResty version 1.17 and is myabe a issue of OpenResty or Nginx under mac

BTW,  is APISIX still can wroks well? I think warn level will not block start.

I think you make a try with older openresty

BTW, is APISIX still can wroks well? I think warn level will not block start.

apisix  is not work, i reinstall openresty <IP_ADDRESS>  ,apisix is working now.

ok, we can close this issue


fix: http connection type

Super seeding @kylemay-gridstone PR

Support for HTTP connection type was added in #228 but the AgentConfigBuilder doesn't set the connection type from useConnectionType(_:) in the build() method.
Fixes issue with OpenTelemetryHelper getURL method by removing extraneous "("
Fixes issue in OpenTelemetryInitializer initializeWithHttp method which does not configure the correct endpoints using "/v1/" for grpc compatibility

CLA reviewed and signed

run docs-build

run docs-build


Add 32bit package guard for non-32bit installs

This patch is with view of preventing the installation of 32bit
libraries on test combinations that do not need them.

Codecov Report

Merging #3394 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3394      +/-   ##
==========================================
- Coverage   80.65%   80.65%   -0.01%
==========================================
  Files         390      390
  Lines       78159    78159
  Branches     8823     8823
==========================================
- Hits        63043    63040       -3
- Misses      13736    13738       +2
- Partials     1380     1381       +1

Is this ready for merge?  The CI runs fine.

Think so, for a 32bit config this branch https://travis-ci.org/numba/numba/jobs/439067675#L461 runs and for a 64bit, it does not https://travis-ci.org/numba/numba/jobs/439067665#L460. Wall time is 48 minutes max (32bit, py27).


[Development] Control flow

[ ] Lambda lift pass
[ ] Support closure
[ ] Add relax.while_loop similar to the current relay.while_loop (functional through recursion)
[ ] TensorArray support: enable opaque Object type and implement TensorArrayRead/TensorArrayWrite PackedFuncs.
[ ] E2E LSTM demo
[ ] Consider adding while and foreach intrinsics (imperative through iteration)

One question, why do we need to support TensorArray for control flow? in order to store the results in the while loop?

One question, why do we need to support TensorArray for control flow? in order to store the results in the while loop?

TensorArray is commonly used with control flow, for example in the TensorFlow while_loop, TensorArray is utilized to store the results, there are tons of TensorArrayRead/TensorArrayWrite operations in the loop. TensorArray support is necessary for end2end model support.


test

Insert description of issue here

✅ By submitting this issue, I have verified the following

[x] Checked to see if the issue has already been discussed before 🤔
[ ] If proposing new content to be added, made sure enough details were provided 🔍

tested


linuxPackages: drop latest_hardened, introduce versioned attributes

Motivation for this change
See https://github.com/NixOS/nixpkgs/pull/133587 for context. Further details are in the commit messages.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Rebased to resolve the merge conflict.

We had more conflict... you might like to re-check the state after PR #134594.


RLError: (SSLError(1, u'[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:581)'),)

Getting always the same issue... what am i doing wrong?
root@retropie:~# wget https://raw.githubusercontent.com/notro/rpi-source/master/rpi-source -O /usr/bin/rpi-source && sudo chmod +x /usr/bin/rpi-source && /usr/bin/rpi-source -q --tag-update
--2016-04-26 05:14:58--  https://raw.githubusercontent.com/notro/rpi-source/master/rpi-source
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... <IP_ADDRESS>
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 12471 (12K) [text/plain]
Saving to: ‘/usr/bin/rpi-source’

/usr/bin/rpi-source           100%[=================================================>]  12.18K  --.-KB/s   in 0.001s

2016-04-26 05:14:59 (19.8 MB/s) - ‘/usr/bin/rpi-source’ saved [12471/12471]

ERROR:
Couldn't download https://api.github.com/repos/notro/rpi-source/git/refs/heads/master, URLError: (SSLError(1, u'[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:581)'),)

Help: https://github.com/notro/rpi-source/wiki
root@retropie:~#

I haven't seen that problem with rpi-source.

Just my two cents... check system time. If your raspberry is not time synced, certs are not ( yet ) valid

add agrs {verify=False } to avoid  verification
requests.post(url,params=values,verify=False)

This one might help as well before the script:
export PYTHONHTTPSVERIFY=0


fix: cast action message regression

Change Summary
This PR adds new Message component which is rendered in case there is an message and no sourceFrame. It also fixes the regression.
Merge Checklist

[x] PR has a Changeset
[x] PR includes documentation if necessary
[x] PR updates the boilerplates if necessary

@stephancill it is possible to do it. But I think that probably onMessage handler would be better and we could remove the Message component and just render null. Or perhaps we can provide both in case you want something different.
What is your opinion?

@stephancill I added onMessage() handler so you can optionally do something completely different in response to message. Also I allowed returning null from MessageTooltip and Message so you can turn these off.


NIFI-13690 Backport of updated dependencies as prudent

Summary
NIFI-13690 This PR updates as many of the same libraries as prudent from #9206 . Some are major versions ahead so those were not updated.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the support/nifi-1.x branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[x] Build completed using mvn clean install -P contrib-check

[x] JDK 8

Licensing

[x] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[x] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Merged in https://github.com/apache/nifi/commit/8bdafb58c5ac930e795158b3cacabcceffba34c6


Remove usage of deprecated file

The class alias is not made so Composer can recognize it without checking it on runtime, so users with --classmap-authoritative cannot use the plugin currently :)

BTW, I am not sure if you are aware, but it does not seem like tests etc run properly in Travis @markstory

I didn't see that. I'll get that fixed.


Renaming chats is not working correctly

Upon renaming a chat, the actual name in the sidebar goes blank.

@danemadsen wonderful!


Reject second stream attempting to overtake existing stream

Closes #13

Awesome, thank you!

No problem!


Feature request: Add a generic model parameter to Cursive.

The callbacks as they are implemented now (using static lifetimes) limit what can be captured in closures.
Adding a generic data member called model to the Cursive struct may be a good way of avoiding the aforementioned limitation. This model can then be accessed from callbacks using a method called model() to get a shared reference to the model and a method called mut_model() to get a mutable reference to the model. Note: all callbacks are passed a reference to the Cursive struct by default.
Thoughts?

Hi!
It's a very interesting idea!
One solution would be to make have Cursive<T> with a model: T, but there's a problem with this: it means every place where we use a Cursive, we have to take this generic parameter into account.
In practice, this means most views would need to be T-generic themselves, even views that don't access the model. This ends up being quite messy, and a sensible loss in convenience.
Instead, we could store this information in runtime instead of compile-time. This means storing something like Box<Any>, and offering methods like Cursive::model<T>(&self) -> Option<&T> - this would let users get the type they stored, without adding a type parameter everywhere.

Awesome idea @gyscos. I am still learning Rust and it seems a lot of it from you :) thanks!

The master branch now brings Cursive::set_user_data, Cursive::user_data and Cursive::with_user_data to cover this use-case. Tell me if it works for you! :)

Looks great! Thanks! :)


Add link to open linkding web site

The goal is create shortcut to open configured linkding web site from plug-in

Hey @tomamplius, I've added an icon for this in #46, please take a look and let me know what you think.


Javadoc for generated sources

Differences were noticed when I built java api source files for linux and mac operating systems. Fully documented sources are only generated with linux builds.
How it looks after build:

for linux(javadoc exists on source code - very comfortable for working in IDE):

    /**
     * Contains information about the period of inactivity after which the current user's account will automatically be deleted.
     */
    public static class AccountTtl extends Object {
        /**
         * Number of days of inactivity before the account will be flagged for deletion; 30-366 days.
         */
        public int days;

        /**
         * Contains information about the period of inactivity after which the current user's account will automatically be deleted.
         */
        public AccountTtl() {
        }

        /**
         * Contains information about the period of inactivity after which the current user's account will automatically be deleted.
         *
         * @param days Number of days of inactivity before the account will be flagged for deletion; 30-366 days.
         */
        public AccountTtl(int days) {
            this.days = days;
        }

        /**
         * Identifier uniquely determining type of the object.
         */
        public static final int CONSTRUCTOR =<PHONE_NUMBER>;

        /**
         * @return this.CONSTRUCTOR
         */
        @Override
        public int getConstructor() {
            return CONSTRUCTOR;
        }
    }

for macos:

    public static class AccountTtl extends Object {

        public int days;

        public AccountTtl() {
        }

        public AccountTtl(int days) {
            this.days = days;
        }

        public static final int CONSTRUCTOR =<PHONE_NUMBER>;

        @Override
        public int getConstructor() {
            return CONSTRUCTOR;
        }
    }

Is it possible to include documented sources for macos builds? It's very convenient to have everything in one place and read docs directly from the IDE. Thanks for your time.

PHP is required for documentation generation, so you need to check that PHP is installed.

PHP is required for documentation generation, so you need to check that PHP is installed.

Thanks for help!


Metabase add ingress labels

@pmint93
What this PR does / why we need it:
This adds support for custom labels to attach to the ingress object. We in particular use this feature as we have multiple ingress controllers to handle services that are internal or public and use Ingress labels to determine which controller binds a particular Ingress object. We've tested this in our own k8s cluster and it works as expected.
Special notes for your reviewer:
Also updated the base version of Metabase to v0.29.3. Let me know if you want to split that out to a separate PR.

@pmint93
We use multiple clusters of Traefik as our Ingress Controller, with each cluster serving a particular kind of traffic (currently internal or external, depending on whether the endpoint is public or not). Traefik does support using kubernetes.io/ingress.class to specify that Ingress objects should be served by Traefik but does not support using this to split Ingresses between different Traefik controllers.
Traefik does support using labels for this purpose (presumably with the justification that labels are intended to be used as "selectors", while annotations are supposed to contain metadata about the object). Hence the PR.

@ReillyProcentive
Ah, I see. Then I think we're good to go merging this 👍
Thanks for your PR.

/approve

/assign @mgoodness

/ok-to-test
/lgtm

This one might need to be manually merged due to the lack of reporting from circleci when that was broken.


Create a Design for building Transcelerator as an extension

Determine what functionality of papi may be missing to implement T-XL extension.  Determine what functionality would be beneficial and which functionality can be ignored for the tech demo
https://docs.google.com/document/d/1wwq0sFpr9A_rs1dTjcKWEfca1S92RKApikFbNleO7kM/edit#heading=h.wx6zis9tcqi2 Notes on the C# decision. Click the "Detailed" hyperlink for information on why the decision went to dlls in the dotnet provider

Some high-level thoughts:

First step (now underway) will be a fairly significant refactor of the existing Transcelerator code to separate the UI from the business logic and create .Net Standard DLL(s) that can be loaded in platform.Bible.
If it is feasible, I think it would be best to maintain a single Github repo with the Transcelerator code. Some research will need to be done to see if there is an easy-ish way to use the https://github.com/paranext/paranext-extension-template to generate the needed files and structure, and then copy everything into the existing Transceleratror repo.

Thanks for the hard work on this! Looks great. Just a few thoughts:

I would prefer to make the names more generic than specifically mentioning "DataProvider" everywhere. It seems to me like there is nothing specifically tying these dotnet dlls down to making only data providers. Maybe the only thing we implement right now is data providers, but the parameters IPapi and IPluginHost don't seem to be constraining the extension to registering only data providers. I envision a future where there is some possibility that dotnet dlls would want to do other things (like registering commands or something), so I'd prefer to genericize the names. Then later we can add other functionality as we want to do so, and it wouldn't be weird that people are adding other functionality to their dotnet dlls because they're not called data provider dlls. I suggest the following adjustments:

Static class name ExtensionDataProviders -> Extension or DotNetExtension or something similar
Register function -> Activate function (to match the JS side)
Manifest field name dataProviderDll -> dotnetDll

Maybe add a Deactivate function to the static class interface?

I just added a little blurb about adding a Deactivate command as well.


[Community] [New extension] Add poki games sdk extension

JS only extension.
The actions names match the ones given in the Poki SDK documentation.

I think the last error "JavaScript disallowed properties" is not relevant here since it's the advised way in https://wiki.gdevelop.io/gdevelop5/extensions/best-practices/#javascript-usage

It should disable audio while an ad is running, and bring it back once the ad is finished

I felt like that's something the creator should handle: Do they want to pause or stop the music or do they want to lower the volume? I don't think this extension should handle this complexity.

The "Shareable URL" API is missing

That was not requested by Wesley but I can add it

Ok so, here are the changes from my latest commit:

SDK is now loaded in the onFirstSceneLoaded life cycle event
There's now a condition to check if the sdk is ready
Each instruction calling the sdk returns early if the sdk is not defined
I have not added the create shareable url since the poki sdk method returns a promise, I'm not sure how we can handle promises in expressions. Otherwise, it could be an asynchronous action that stores the url into a variable but I'm not sure it's doable.

After a little test, I realized that if the game is run on a no-poki url, the sdk initialization does not fail so the condition isSdkReady will be true.
I could add a second condition is Poki SDK ready to display ads returning a member of the js class PokiSDK.SDK

@arthuro555 I must admit that I'm a bit skeptical about all those security and internet condition considerations that should be taken for GDevelop games in this extension even though I only applied what Poki themselves consider what is a correct implementation of their SDK.
Could it help to name this extension "Poki games SDK HTML"?


Request to add Huber loss function

Request to add a Huber loss function similar to the tf.keras.losses.Huber class (TF 2.0 beta API docs, source).
Potential implementation - as per @ageron notebook - but with needed amendments:
def huber_fn(y_true, y_pred):
  error = y_true - y_pred
  is_small_error = tf.abs(error) < 1 # replace `tf` with `K` where `K = keras.backend`
  squared_loss = tf.square(error) / 2 # replace `tf` with `K`
  linear_loss = tf.abs(error) - 0.5 # replace `tf` with `K`
  return tf.where(is_small_error, squared_loss, linear_loss) # replace `tf.where`

Huber loss was requested here #6410 but Log-Cosh was introduced instead (2x differentiable). tf.keras.losses.LogCosh also exists.

Thanks @jvishnuvardhan - looks like tf.keras.losses.huber is now available in tf-nightly cc @dynamicwebpaige

Thank you all - looks like tf.keras.losses.huber is part of TensorFlow 2.3 Core. Closing the issue @jvishnuvardhan @dynamicwebpaige


DataArray.sum is slower than applying numpy's .sum and re-building DataArray

What happened:
I had a DataArray with shape (72, 262, 33, 7, 2, 27) and applied .sum() over three dimensions. I noticed a big spike in memory usage, so I tried converting it to a numpy ndarray, apply .sum() and re-build the DataArray. It outperformed it in time and it didn't have a memory spike.
What you expected to happen:
It should be faster and not have a memory spike.
Minimal Complete Verifiable Example:
import pandas as pd
import numpy as np
import xarray as xr

index_a = pd.Index([f'A-{i}' for i in range(72)], name='index_a')
index_b = pd.Index([f'B-{i}' for i in range(262)], name='index_b')
index_c = pd.Index([f'C-{i}' for i in range(33)], name='index_c')
index_d = pd.Index([f'D-{i}' for i in range(7)], name='index_d')
index_e = pd.Index([f'E-{i}' for i in range(2)], name='index_e')
index_f = pd.Index([f'F-{i}' for i in range(27)], name='index_f')

indexes = [index_a, index_b, index_c, index_d, index_e, index_f]

np_array = np.ones(tuple([len(ind) for ind in indexes]))
# Build base DataArray
da = xr.DataArray(np_array, indexes)

dims_to_sum = ['index_b', 'index_c', 'index_d', 'index_e']

# DataArray sum()
%timeit da.sum(dims_to_sum)

def sum_with_np(dataarray, sum_dim_names):

    sum_dims_indexes = dataarray.get_axis_num(sum_dim_names)
    np_array_2 = dataarray.values.sum(axis=sum_dims_indexes)

    remaining_dims = [pd.Index(dataarray.coords[dim].values, name=dim)
                          for dim in dataarray.dims if dim not in sum_dim_names]

    return xr.DataArray(np_array_2, remaining_dims)

# Applying numpy sum() and re-building DataArray
%timeit sum_with_np(da, dims_to_sum)

Anything else we need to know?:
I provided that dimensionality because it performs worse as dimentionality goes up.
Performance:

DataArray.sum(): 1.37 s ± 107 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)
numpy.sum() and re-building DataArray: 359 ms ± 28.6 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

I apologize if the example is not short enough. This is my first time creating an issue here.
I'm not sure if this behaviour is expected because DataArray sum() must do something else I'm not taking account of.
We use xarray at work a lot and we love it (we build financial-economic models and data analytics solutions for companies in Python).
 For a direct comparison, use .sum(skipna=False).

Thanks! That is what I needed.


build error with c lib

gcc -shared -O3 -o libblake3.so blake3.c blake3_dispatch.c blake3_portable.c \
>     blake3_sse41_x86-64_unix.S blake3_avx2_x86-64_unix.S blake3_avx512_x86-64_unix.S
/usr/bin/ld: /tmp/ccoz9HaM.o: relocation R_X86_64_PC32 against symbol `blake3_simd_degree' can not be used when making a shared object; recompile with -fPIC
/usr/bin/ld: final link failed: Bad value
collect2: error: ld returned 1 exit status

I'm getting the error above when trying to compile the share library. including -fPIC solves it, but not sure if that will result in intended behavior. a small clarification on this would be good

You'll need to specify the particular system (Linux (?) distribution, GCC version, at least) this is happening on. I cannot reproduce this.

Sorry about that
NAME="Ubuntu"
VERSION="16.04.6 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.6 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial
UBUNTU_CODENAME=xenial

GCC is
ost=x86_64-linux-gnu --target=x86_64-linux-gnu
Thread model: posix
gcc version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.12)

also now that i have your attention
x86_64-w64-mingw32-gcc -I/usr/lib/jvm/default-java/include -I/usr/lib/jvm/default-java/include/linux -shared
-o ../blake3.dll blake3.c blake3_dispatch.c blake3_portable.c blake3_sse41_x86-64_windows_gnu.S \
blake3_avx512_x86-64_windows_gnu.S

this should build a working dll for windows 64 right? I'm working on JNI bindings for the c api. so thats why i need to build the project in different platforms while appending the Java bindings
Thanks!

Could mere Windows users without compilers try this bleeding-edge stuff? Give us a binary, please.

It would appear that on this GCC version, either -fPIC is not a default or -shared does not activate it automatically for the selected input files, even though it's necessary for linking to work. Adding -fPIC manually is an appropriate solution.
Regarding Windows, I am not too familiar with the MinGW toolchain, but you might need dllexport and dllimport annotations, like shown here. A DLL library was not something we explicitly targeted.

@sneves, what library? I talk about .exe, generated as less dependent as possible. Like @Cyan4973 does with xxHash.  Wider audience uses, as you know, Windows. And we’re still patiently waiting for a  SHA2/3 replacement, because BLAKE2 never achieved it (that is, screwed because of being too diverse).

@sneves i just tried it with gcc 7.4 and it resulted in the same error btw. strange

@sneves i got it to compile a working dll with the following
x86_64-w64-mingw32-gcc -I/usr/lib/jvm/default-java/include -static -shared -o ../../../natives/windows_64/blake3.dll blake3.c blake3_dispatch.c blake3_portable.c blake3_sse41_x86-64_win
dows_gnu.S blake3_avx2_x86-64_windows_gnu.S blake3_avx512_x86-64_windows_gnu.S org_scash_NativeBLAKE3.c

it works with either -static or -static-libgcc without either of these the dll is trying to find libgcc_s_seh-1.dll which is the dll in charge of handling and throwing exceptions. I'm not very familiar with c. I found this by exploring the dll dependencies with the Dependency Checker for dlls.
It works now as in the library is successfully loaded into jvm. Testing the actually bindings is next.
https://github.com/sken77/BLAKE3jni
here is a link of the project

@sergeevabc, this issue is discussing a specific set of compiler errors. If you'd like to discuss pre-compiled binaries, please open a separate issue. (If you want to try out BLAKE3, I recommend taking a look at the b3sum binary, by installing the Rust toolchain and running cargo install b3sum. But if that doesn't work for you, again, please open a separate issue, to avoid taking this one off-topic.)

@sken77 tweeted :)

It looks like the questions here have been answered, so I'm going to close this one. I've opened https://github.com/BLAKE3-team/BLAKE3/issues/78 about pre-compiled binaries.


[GLUTEN-8244][CORE] Softaffinity use consistent hash schedule

What changes were proposed in this pull request?
https://github.com/apache/incubator-gluten/issues/8244, This PR aims to make softaffinity use consistent hash schedule.
How was this patch tested?
unit tests and GA.

@zzcclp @jackylee-ch @zhztheplayer @PHILO-HE Could you please take a look, thanks!

Naive question: why we need consistent hash here? Is it for avoiding cache misses to the max extent when cluster gets changed?

Yes, especially when using the spark dynamic executor.

What does vanilla spark do here? Is it an enhancement of Spark scheduling?

AFAIK, the consistent hashing cannot solve the cache miss problem for velox backend, but can bring a small improvement. For the ch backend, can consistent hashing significantly improve the performance and resove the cache miss? @zzcclp @loneylee

AFAIK, the consistent hashing cannot solve the cache miss problem for velox backend, but can bring a small improvement.

Thank you for helping confirm. Also, is it possible that the local cache can be benefited?

Yes, especially when using the spark dynamic executor.

Yes I assume this is another key point. Dynamic Allocation ON + Local Cache ON, could this be a typical target scenario of the change? @yikf are you using local cache so far?

Also, is it possible that the local cache can be benefited?

Yes. With consistent hashing, we can schedule the cache location in advance, which is better than knowing the cache location after execution. And consistent hashing is useful for cluster changes, especially when executors are rescheduled due to executor deaths.

AFAIK, the consistent hashing cannot solve the cache miss problem for velox backend, but can bring a small improvement.

Thank you for helping confirm. Also, is it possible that the local cache can be benefited?

Yes, especially when using the spark dynamic executor.

Yes I assume this is another key point. Dynamic Allocation ON + Local Cache ON, could this be a typical target scenario of the change? @yikf are you using local cache so far?

As @jackylee-ch said, pure soft affinity scheduling using consistent hashing would be better than the current logic in scenarios where executors change, which is the charm of consistent hashing. for local cache, I assume that there will be some benefits from the scheduling optimization as well.
We have used it in the TPC-DS, and we are also looking for scenarios in our online environment. Additionally, we would like to use soft affinity scheduling to optimize other scheduling within our organization.

We have used it in the TPC-DS, and we are also looking for scenarios in our online environment. Additionally, we would like to use soft affinity scheduling to optimize other scheduling within our organization.

Is it an enhancement of vanilla Spark as well? or Gluten only?

Is it an enhancement of vanilla Spark as well? or Gluten only?

AFAIK, it is only used for Gluten, vanilla Spark doesn't need this.

We have used it in the TPC-DS, and we are also looking for scenarios in our online environment. Additionally, we would like to use soft affinity scheduling to optimize other scheduling within our organization.

Is it an enhancement of vanilla Spark as well? or Gluten only?

It is currently only using gluten, but the soft affinity scheduling is relatively independent, and we can optimize it internally with the internal Spark version.

@jackylee-ch @zhztheplayer @FelixYBW Hi, Could you please take a look another?

Basically look good to me, left a few comments.
BTW, have you test this pr with duplicateReadingDetect?

Also cc @zzcclp @zhli1142015 , any more question about this pr?

Basically look good to me, left a few comments. BTW, have you test this pr with duplicateReadingDetect?

The UTs contains the duplicateReadingDetect  suite, and it passed the UTs validation.

LGTM.
BTW, I saw you also use SA with local cache, which cache do you use?

LGTM. BTW, I saw you also use SA with local cache, which cache do you use?

We use Velox cache on TPC-DS, mainly with SSDs and RAM, and we are currently exploring scenarios for the online environment.

AFAIK, the consistent hashing cannot solve the cache miss problem for velox backend, but can bring a small improvement. For the ch backend, can consistent hashing significantly improve the performance and resove the cache miss? @zzcclp @loneylee

For the ch backend, the consistent hashing will reduce the cache missing, I don't know what about the performance.


extraModules is not iterable error

@nuxtjs/tailwindcss: 6.5.0
nuxt: 3.2.3
I'm very randomly getting the below error and it's a little hard to replicate. I will keep trying to provide a reproduction. But it happens usually after tailwind.config.js (configPath) / tailwind.css (cssPath) are updated and when I see the "Please restart the Nuxt server to apply changes" warning message.

What is actually happening?

Thanks for looking into this.

What's the file preview in the screenshot? Is it @nuxtjs/tailwindcss/dist/module.mjs?
Does this continue with the latest versions of Nuxt and this module?

Fix coming in #672

Thanks bunch 🙏🏾


TypeError in GET /b/large/GYLR2JCJ.png

Error in ClashLeaders
TypeError in GET /b/large/GYLR2JCJ.png
a bytes-like object is required, not 'tuple'
View on Bugsnag
Stacktrace
clashleaders/views/badge.py:31 - clan_badge_png

View full stacktrace
Created automatically via Bugsnag

An error linked to this issue has been marked as fixed in Bugsnag
TypeError in GET /b/large/GYLR2JCJ.png


The nextTab and prevTab actions do not work correctly when initiated via wt.exe or the command palette

In short, wt focus-tab --next and --previous don't do anything at all on my system, while Next tab and Previous tab via the command palette behave oddly, although I can't imagine that the latter is often used by anyone.
Windows Terminal version (or Windows build number)
1.8.1032.0
Steps to reproduce
Case 1:
Run wt -w new new-tab ; new-tab ; focus-tab --next (or --previous).
Case 2:
Run wt -w new new-tab ; new-tab.
In the new window, run wt -w 0 focus-tab --next (or --previous).
Case 3:
Run wt -w new new-tab ; new-tab ; new-tab.
In the new window, open the command palette (by default using Ctrl+Shift+P) and type in Next tab (or Previous tab) Enter.
Expected Behavior
Case 1:
The first tab is focused.
Case 2:
The first tab is focused.
Case 3:
The desired tab (according to the tabSwitcherMode) is focused.
Actual Behavior
Case 1:
The focus-tab subcommand is ignored; the second tab remains focused.
Case 2:
The focus-tab subcommand is ignored; the second tab remains focused.
Note that in the two cases above, wt focus-tab --target 0 works correctly.
Case 3:
This depends on the tabSwitcherMode:

mru — the desired tab is focused for a split second, and then the originally-focused tab regains focus.
inOrder — the desired tab is focused for a split second, and then tab index 0 gains focus.
disabled — the desired tab is focused (there is no bug in this case).

My apologies if I should have separated this into two issues.
Thanks!

Discussion during triage: We probably shouldn't even be using MRU during focus-tab.

Yeah, MRU during startup makes little sense. I will issue a PR (it is few lines of code). However, I thought we need to update this in documentation and found no documentation for these flags. Am I missing something?


Allow to select year less than 1900 for start and end dates in temporal coverage metadata

Verified on dev and it is working as expected. closing.


fix redaction error on README.md file

This commit fix two redaction errors on README.md file

thanks!


Extract metadata file/URL with python3-saml

I have a Python-Django project and I'm trying to integrate with a C#/.NET existing website SAML authentication, using their idp.
I absolutely need to generate a metadata file to give to whoever manages the IdP (C#/.NET website) to implement SSO.
How can I implement it?

The Django Demo contains a metadata method that contains an example on how generate the XML file
Here you have another example:
https://github.com/SAML-Toolkits/python3-saml#the-sp-endpoints


Featurepatch

Added a Github action for labelling issues as well as greetings.
Features of this PR:
Labels will be generated based on the content which is written in the issue description. like if its enhancement type, then labeller will add enhancement label, similarly for bug and other.
We can mention the keywords for a particular label in the issue_labeller.yml file inside the .github folder.
The greeting action will add a comment on the issue and PR for the first time contributors.
The message present in the greeting action (like congrats 🎉 on creating your first PR, etc) can cheer someone that his/her contribution is being acknowledged and encourage them to contribute more.
@prateek3255  @apollonian and other maintainers of the repository do provide reviews/suggestions to this pull request as per your convenience and  merge it into the repository.

Are the suggested changes done in this PR working or should I try something else @prateek3255  @apollonian and other maintainers of this repository?


Update azure-stack-storage-dev.md

Correct some characters in sample of node.js for incorrect code format.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@xiaofmao : Thanks for your contribution to the Azure documentation! The author, @xiaofmao, has been notified to review your proposed change.

@xiaofmao : Thanks for your contribution to the Azure documentation! The author, @xiaofmao, has been notified to review your proposed change.

I cannot approve my own PR. Could you please help to review and approve @vhorne ?

@xiaofmao - Thanks for the submission, but azure-docs-pr is the appropriate repo for this submission.
Article authors working in the public repo says...
If you are the listed author for an Azure technical article, you need to use the private repository (azure-docs-pr) for all content updates and new articles.
Closing the pull request so the updates can be made in the private repo.
Thanks


Add examples of AllowSafeAssignment option

This is only document change.
This commit adds examples of AllowSafeAssignment option for Lint/AssignmentInCondition cop, Style/ParenthesesAroundCondition cop, and Style/TernaryParentheses cop.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

I think we can also explain the concept of safe assignment in the top-level description of the cop for extra clarity.

Sure! I updated the top-level description of cops based on the following.
https://github.com/rubocop-hq/rubocop/blob/v0.74.0/lib/rubocop/cop/mixin/safe_assignment.rb#L5-L7

Thanks!


Alpaca-Electron-mac-arm64-v1.0.5 app is "damaged".

Describe the bug
MacOS reports: “Alpaca Electron” is damaged and can’t be opened. You should move it to the Trash.
To Reproduce
Steps to reproduce the behavior:

Download https://github.com/ItsPi3141/alpaca-electron/releases/download/v1.0.5/Alpaca-Electron-mac-arm64-v1.0.5.dmg
Click on the downloaded dmg
Move Alpaca Electron to Applications
Click on Alpaca Electron in Applications
See error message.
Try control-click instead.
Still see error message.

Expected behavior
Alpaca Electron at least opens.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: MacOS arm64
Version Ventura 13.4 (22F66)

Additional context
I've been trying to find some way to run an LLM locally on a beefy laptop, and thus far nothing anyone has recommended has worked, so this lengthens the amusing tale of fail.

Describe the bug MacOS reports: “Alpaca Electron” is damaged and can’t be opened. You should move it to the Trash.
To Reproduce Steps to reproduce the behavior:

Download https://github.com/ItsPi3141/alpaca-electron/releases/download/v1.0.5/Alpaca-Electron-mac-arm64-v1.0.5.dmg
Click on the downloaded dmg
Move Alpaca Electron to Applications
Click on Alpaca Electron in Applications
See error message.
Try control-click instead.
Still see error message.

Expected behavior Alpaca Electron at least opens.
Screenshots If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: MacOS arm64
Version Ventura 13.4 (22F66)

Additional context I've been trying to find some way to run an LLM locally on a beefy laptop, and thus far nothing anyone has recommended has worked, so this lengthens the amusing tale of fail.

Try this in terminal
xattr -cr /Applications/Alpaca\ Electron.app/

Success! (it starts and whines at me about wanting an Alpaca model).  Not sure if this counts as closable or not.

FWIW I'm experiencing exactly the same

Please look at the Troubleshooting section in the readme file.
https://github.com/ItsPi3141/alpaca-electron#macos


Completely remade!

Since this is a remake, there will be a pretty lengthy comment.
I have completely remade the DC24 Bot in JavaScript! This is because the old bot was no longer being maintained, and was hard to edit. This new bot contains the same features as the old one, but more efficient.
This includes:
--Error Lookup (Although the database needs contributions)
--Mail Code Sharing
--DNS lookup
--Patchers lookup
And potentially more coming in the future! I hope this can serve as the new DC24 bot, with many updates to come in the future.

JavaScript
why does nobody make a bot in another language these days

i knew someone was gonna say that

indeed
When C# edition

¯_(ツ)_/¯

Doesn't look bad tho :P

Seconded on the above - I think this was via GitHub web, using GitHub Desktop or the like will help a ton.

The only reason I made this was because it was an obsolete version of RiiConnect24 bot. Its more efficient because theres config.json which allows easy setting changes. Also, most of the commands on the old DC24 bot just said soon:tm: so this was needed regardless

In terms of seeing it in action, you can do so in this server: https://discord.gg/aGWrJXK


Updating the  v1.1 = updated short description

Description
v1.1 = updated short description
Issues Resolved
Updated Description in UI
Contribution Check List

[X ] All tests pass.
[X ] New functionality includes testing.
[X ] New functionality has been documented in the README if applicable
[X ] New functionality has been documented in CHANGELOG.MD

          2 Errors

:no_entry_sign:
Please add a changelog

:no_entry_sign:
Please add labels to this Pull Request

          1 Warning

:warning:
Should this include readme changes

  Generated by :no_entry_sign: Danger


SW-585: Use ? as a value placeholder in PostgreSQL backend

When using PostgreSQL backend, we cannot use ? as a place holder fro value parameters, but we can only specify them by name, or by position. This patch add support for ? placeholder and it replaces it by position placeholder.

Is this tested and works?


Issue with CompositionApi

Describe the bug
After upgrading to vue-demi "^0.12.1",
I have "[vue-composition-api] must call Vue.use(VueCompositionAPI) before using any function." all over the place.
I tried to solve with https://stackoverflow.com/questions/69846717/vue2-and-composition-api-cannot-import-store-error-vue-composition-api-must, but with no success.
This upgrade was triggered by the upgrade "@vue/apollo-composable", from "^4.0.0-alpha.15" to "^4.0.0-alpha.16"
My current setup:
"@vue/apollo-composable": "^4.0.0-alpha.15",
vue-demi "^0.11.4"
is not having any issues
Also published here : https://github.com/vueuse/vue-demi/issues/127

Finally I used the Option 1 of  https://stackoverflow.com/questions/69846717/vue2-and-composition-api-cannot-import-store-error-vue-composition-api-must, everywhere I had used ref outside a setup function, and so far everything is ok with the latest version.


Simulator Frontend- Improve the input of constructor parameters

The user should be able to change from "form" and "json".
If the mode is JSON:

we need to parse it somehow and notify the errors in the syntax.

If the mode is form:

For every parameter add an individual form input.
If the parameter is a boolean, show a dropdown with true and false, otherwise show a text-input

closed because we will working on #116


Update release note 1.4.0

add a release information,milestones and version updates

merged to develop


[V2V] Use podman with UCI image instead of virt-v2v-wrapper directly

To be able to integrate Universal Conversion Image (UCI) in the migration flow, we need to update ConversionHost class so that it uses podman with UCI container image instead of call virt-v2v-wrapper directly.
This pull request updates the ConversionHost.run_conversion method to:

Prepare the folder structure on the conversion host:
a. Create the folders in /var/lib/uci and /var/log/uci based on task id for uniqueness
b. Generate the input file for the container under the lib folder, as we can pass the options as stdin
Build the podman command in a separate method to have unit test, even though it's fairly simple
Call the command on the conversion host, still over SSH, to create a pod named after the task id

Most of the paths are now related to the task id and not to random values returned by virt-v2v-wrapper, so it's more deterministic. It implies passing the task id to the conversion host methods and some rewrite and updates to the specs.
The ConversionHost.kill_conversion method has also been renamed and modified to kill the pod based on the task id.
The methods to retrieve the conversion state and logs have also been updated.
The pull request also creates a private utility method filter_options as filtered options are used in more that one place.
RHBZ: https://bugzilla.redhat.com/show_bug.cgi?id=1788988

@miq-bot add-label transformation, enhancement, ivanchuk/yes, wip
@miq-bot add-reviewer @agrare
@miq-bot add-reviewer @djberg96

@agrare, @djberg96, I still need to wait for virt-v2v-wrapper devs to confirm the image name and the lib and log paths, but I'd appreciate a review now, so that it's just a quick update when they confirm.

@Fryguy, @agrare, the colland built by build_podman_command is run on the conversion host, not on CloudForms appliance. So we know where VDDK is extracted. I'd like to understand better the command line injection.
For the image registry, that's a fair point. How would you parameterize it? In the settings? The image would be available on Docker Hub for ManageIQ and registry.access.redhat.com for CloudForms. I guess.

I'd like to understand better the command line injection.

As a rule of thumb, if I see "command #{variable}" I automatically flag it as a possible command line injection because even though variable might now be controlled by us, that doesn't mean changes in the future couldn't introduce a string that comes from a user params (think something like in the future we allow the path locations to be a setting).  Interpolating user data into the command line could allow for something like:
# (Some hypothetical future change)
"podman --volume /opt/vmware-vix-disklib-distrib:{Settings.v2v.vddk_volume}"

# Some bad user puts the following in Settings:
v2v:
  vddk_volume: "/opt/vmware-vix-disklib-distrib && rm -rf /"

# We end up running
podman --volume /opt/vmware-vix-disklib-distrib:/opt/vmware-vix-disklib-distrib && rm -rf /

@fdupont-redhat looks like you pulled in a bunch of other commits, maybe pulled master without rebasing?

Finally got a UCI-based conversion host to run end-to-end tests, including cancellation. Looks good.
@miq-bot remove-label wip
@agrare @Fryguy would you mind reviewing officially?

I'd like to understand better the command line injection.

As a rule of thumb, if I see "command #{variable}" I automatically flag it as a possible command line injection because even though variable might now be controlled by us, that doesn't mean changes in the future couldn't introduce a string that comes from a user params (think something like in the future we allow the path locations to be a setting). Interpolating user data into the command line could allow for something like:
# (Some hypothetical future change)
"podman --volume /opt/vmware-vix-disklib-distrib:{Settings.v2v.vddk_volume}"

# Some bad user puts the following in Settings:
v2v:
  vddk_volume: "/opt/vmware-vix-disklib-distrib && rm -rf /"

# We end up running
podman --volume /opt/vmware-vix-disklib-distrib:/opt/vmware-vix-disklib-distrib && rm -rf /

Using a command line builder that deals with escaping parameters completely prevents this...
AwesomeSpawn.build_command_line("podman", :volume => "/opt/vmware-vix-disklib-distrib:{Settings.v2v.vddk_volume}")
# => "podman --volume /opt/vmware-vix-disklib-distrib:/opt/vmware-vix-disklib-distrib\\ \\&\\&\\ rm\\ -rf\\ /"

I completely missed this comment. I could have fixed it earlier. Sorry.


Reverse parser

Hello,
I need to do this for a project I'm working on and I just thought I would ask here as I know you guys are helpful and in the chance that there might be a way.
I would like to take an HTML string and convert that string to Markdown.
For eg:

Heading
 to ### Heading
Or more complex:

123

abc

to
* 123
* abc

Can you think of a way to achieve this?
I was going to do it manually using string manipulation.
Thanks
Luke

This is effectively a dupe of #91, see Alexandre's response there https://github.com/lunet-io/markdig/issues/91#issuecomment-277191539

Thanks,
I never heard of AngleSharp, I am using HtmlAgilityPack inn my project... I will see what I can do


[Tree] Materialized Path

Hi all,
I'm in the middle of a project which needs this, so I've started to work on the Materialized Path strategy for the Tree extension yesterday.
https://github.com/comfortablynumb/DoctrineExtensions/tree/mongodb-tree-materialized-path
This is just an experiment yet. It's far from being complete. There are a lot of work to do yet, including:

Add some more mapping and runtime validations
Add more methods to the repository (so far I've included only one just to make it work)
Check and optimize the internals of the strategy
My need was only for MongoDB, but create the ORM equivalent I think it's trivial. I've abstracted as much as possible all the functionalities in AbstractMaterializedPath class. So far, I need to implement only two methods in the strategy, and create a repository for the ORM.
Add more tests (There are already some)

There are some drawbacks though:

In MongoDB, updates to multiple documents are not atomic, so this could lead in tree inconsistencies. What would be the best approach to minimize this?

And, finally, some questions:

Should we make the "level" property mandatory in this strategy? I think it's worth it, and some internal queries could benefit from this instead of using a regex.
Should we avoid in the PathSource the usage of the selected "separator", throwing a RuntimeException when trying to set the PathSource with it? I think yes, but I ask because maybe someone has a better alternative.

Well, that's all for now. I'd like to hear your opinions guys.
Thanks.

could you create a pull request with your branch ? it would make the review easier as we could comment on it.

Of course. I didn't do it because it's not complete yet. But I didn't know you couldn't comment. I'll do it now.

well, with a branch, I can comment on each commit, but not on a general diff. As it is not complete yet, simply tag is a WIP in the title


ICS27 SubmitTx rpc should account for timeout height

Summary
The new SubmitTx rpc endpoint calls directly into the existing SendTx API.
As a result the MsgSubmitTx field TimeoutHeight is unused as the existing code hardcodes a zero height.
We should refactor the existing code and private/unexported function to be able to make use of the TimeoutHeight passed via the new SubmitTx rpc.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

this is no longer an issue as we are removing the TimeoutHeight field here


Move For() to IValitRules explicit implementation

Since For(TObject @object) is static initialize method, we have to pass the object even if we just want to execute set of rules.
Tasks:

Move For() to IValitRules<TObject>
Create static initialize method like Create with optional parameter -
IEnumerable<IValitRule<TObject>>

PR merged #19


Fix flower docker image

The current image (based on flower 1.0.1) has changed the default cmd,
meaning the docker compose command fails. Fixing to the previous
0.9.7 gets things working again.
Fixes https://github.com/tiangolo/full-stack-fastapi-postgresql/issues/398

Just realised this repo has been abandoned... with many issues fixed multiple times... sniff.


remove dependency from "Ckmeans.1d.dp"

Hi, in order to plot the feature importance plot the dependency from Ckmeans.1d.dp could be removed and the function rewritten internally...

Already done.


crack-attack: enable custom sounds

Motivation for this change
Closes #22905
crack-attack will search for user-provided musics and sounds
under ~/.crack-attack/
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build crack_attack

Success on aarch64-linux (full log)
Attempted: crack_attack
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
moving /nix/store/ia500nl9gxdnvj9nqp2szmmqp8djn69c-crack-attack-1.1.14/man to /nix/store/ia500nl9gxdnvj9nqp2szmmqp8djn69c-crack-attack-1.1.14/share/man
shrinking RPATHs of ELF executables and libraries in /nix/store/ia500nl9gxdnvj9nqp2szmmqp8djn69c-crack-attack-1.1.14
shrinking /nix/store/ia500nl9gxdnvj9nqp2szmmqp8djn69c-crack-attack-1.1.14/bin/crack-attack
gzipping man pages under /nix/store/ia500nl9gxdnvj9nqp2szmmqp8djn69c-crack-attack-1.1.14/share/man/
strip is /nix/store/6dpnd5aniypn8124mmy8f88s4mq2zl07-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/ia500nl9gxdnvj9nqp2szmmqp8djn69c-crack-attack-1.1.14/bin
patching script interpreter paths in /nix/store/ia500nl9gxdnvj9nqp2szmmqp8djn69c-crack-attack-1.1.14
checking for references to /build in /nix/store/ia500nl9gxdnvj9nqp2szmmqp8djn69c-crack-attack-1.1.14...

Success on x86_64-linux (full log)
Attempted: crack_attack
Partial log (click to expand)
post-installation fixup
moving /nix/store/yvkcp6c9k4hgrpdipwp966bq5ll0ry9w-crack-attack-1.1.14/man to /nix/store/yvkcp6c9k4hgrpdipwp966bq5ll0ry9w-crack-attack-1.1.14/share/man
shrinking RPATHs of ELF executables and libraries in /nix/store/yvkcp6c9k4hgrpdipwp966bq5ll0ry9w-crack-attack-1.1.14
shrinking /nix/store/yvkcp6c9k4hgrpdipwp966bq5ll0ry9w-crack-attack-1.1.14/bin/crack-attack
gzipping man pages under /nix/store/yvkcp6c9k4hgrpdipwp966bq5ll0ry9w-crack-attack-1.1.14/share/man/
strip is /nix/store/rpbg8gmqxhz8g61p1plz5d2srs84pvmv-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/yvkcp6c9k4hgrpdipwp966bq5ll0ry9w-crack-attack-1.1.14/bin
patching script interpreter paths in /nix/store/yvkcp6c9k4hgrpdipwp966bq5ll0ry9w-crack-attack-1.1.14
checking for references to /build in /nix/store/yvkcp6c9k4hgrpdipwp966bq5ll0ry9w-crack-attack-1.1.14...
cannot link '/nix/store/.links/1qahg2vjahwq4wsswdaljwh39am5ddgpcwjrrrk2sxg0mg9pi0y6' to '/nix/store/yvkcp6c9k4hgrpdipwp966bq5ll0ry9w-crack-attack-1.1.14/share/crack-attack/sign_x10.tga': No space left on device


Feature request: Entering own promo codes and promos codes like lyft

Can we enter our own promo codes and as well as pre-generated ones.
Secondly was wondering if you can create the promo codes like LYFT. They give a 50$ coupon, where 5$ is applied per ride till 50 is done so 10 rides.
Or any way you think I can use your packaged to do that ?
And can you please change "Amount" to "number" - I got super confused as to what the diff between amount and reward was

Another thing what if a user is using JWTauth, the auth trait wont work when you check if the promo code has already been applied to user?

Can we enter our own promo codes and as well as pre-generated ones.

use Promocode  model to create custom models

Secondly was wondering if you can create the promo codes like LYFT. They give a 50$ coupon, where 5$ is applied per ride till 50 is done so 10 rides.Or any way you think I can use your packaged to do that ?

this package doesnt allow  coupon to be used multiple times . you should manage the logic separeately for it

And can you please change "Amount" to "number" - I got super confused as to what the diff between amount and reward was

haha you will get used to it

Another thing what if a user is using JWTauth, the auth trait wont work when you check if the promo code has already been applied to user?

yeah i noticed that ... and ended up using Promocodes::apply ensuring there is authenticated user

We have added functionality to allow you to use codes multiple times. So I think this issue can be closed. But we can continue conversation


Add documentation about creating custom focal operation, not part of LUE

Add example of this to lue_consume project
Document procedure of developing a new C++ library that is consuming LUE
Document procedure of developing a custom focal operation
Document how to add Python language wrappers for a custom function which consumes LUE

Mention that nanobind and pybind11 don't play well together.


Fix transparent_window example on wayland

Wayland only supports pre-multiplied alpha. Behavior on X11 seems unchanged.
Objective

Fix #10929 on wayland.

Solution

Request pre-multiplied alpha.

Testing

Ran the example locally.

Seems to break running with opengl which only supports opaque.

I'd prefer to add a toggle between the transparency modes, with a comment about which modes work on which platforms (ideally a link to docs where this information already exists).

Seems to break running with opengl which only supports opaque.

If the opengl bavkend doesnt support transparency, then this example not working at all is probably OK.

I'd prefer to add a toggle between the transparency modes, with a comment about which modes work on which platforms

Sounds like a task for a different PR.

Looks good to me!


[Examples Browser] Debug build

Adds scripts to build and watch a debug version of each example:

npm run build:debug:directory
npm run watch:debug:directory (rebuilds each time an example is edited)

Serving the examples build directory using npm run serve will then make the localhost:5000/debug directory available which contains static HTML versions of each example for debugging purposes.
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

This is fantastic, thanks for doing it. Please update README.md with new commands as well.


Subspace enable accounts

Draft PR for now...
The UI still needs some work and probably some error checking
But the idea is to be able to set some subspace credentials against an account and create/delete a sip teleport. Ideally then merge these UI changes over to the hosted platform and then do some DNS jiggery pokery to make it super simple to use Subspace enabled Jambonz

could we have a .env var (ENABLE_SUBSPACE, or something) that determines whether or not to show the subspace stuff)?

could we have a .env var (ENABLE_SUBSPACE, or something) that determines whether or not to show the subspace stuff)?

Fantastic idea 👍

I should probably tidy up the Id and secret fields following what you already have for HTTP auth

I should probably tidy up the Id and secret fields following what you already have for HTTP auth

The fields should be cleaned up visually, but I wouldn't use the same format as the basic auth fields. Those fields are toggled on/off and are optionally associated with the main webhooks fields. Just some proper spacing between the fields as you have them should suffice seeing as Subspace is a main form field in this UI.
I also agree with @davehorton about using en ENV var for enabling this functionality. That should be used to determine whether the Subspace input group is shown on the account form and if the column for Subspace is shown on the main account list table.
Other than that I think this generally looks okay. I know the backend isn't set up to support this yet so I cannot fully test the flow of actually saving these new fields etc.

@kitajchuk you can check out https://youtu.be/pvy1mNDWiSI?t=94 for how it interacts... I'm not 100% happy with it like I say in the video... (or did I because I was limited to 5 minutes?) - you have to save the details before you go back in to enable the teleport etc... so it def needs work

@danjenkins @kitajchuk I'm coming back to this now as I want to get this feature into the next release.  It looks like I need to add a corresponding API on the back-end so I will do that, then update here.  Dan, if you don't have time let me know and I will ask brandon to see if he can finish it off from here..Or maybe even I can figure it out...

I'll pick this up once the little one is asleep! 👍👍

I haven't been able to sort out the styling as i can't get the server running on my m1 mac.... will investigate how to get around that when I get a moment

@kitajchuk would you be able to give your review?

I'm trying to work out the API that I need to create on the back end....it looks like there is a POST and a DELETE...is the former to basically create a sip teleport on subspace, or something else?

Yup that’s it, there’s one to create a jwt. There’s one to create a sip
teleport and then there’s another to delete the sip teleport
On Thu, 4 Nov 2021 at 00:02, Dave Horton @.***> wrote:

I'm trying to work out the API that I need to create on the back end....it
looks like there is a POST and a DELETE...is the former to basically create
a sip teleport on subspace, or something else?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jambonz/jambonz-webapp/pull/25#issuecomment-960306168,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB3LLOPET6RV4VQDFRNMF3UKHLTHANCNFSM5EYTPWZQ
.

@davehorton theres already a PR for the api server?

I can take a look at it later today 👍
On Wed, Nov 3, 2021 at 4:28 PM Dave Horton @.***> wrote:

@kitajchuk https://github.com/kitajchuk would you be able to give your
review?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jambonz/jambonz-webapp/pull/25#issuecomment-960292131,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADLDNYVBTQFBV2CNBPJRI3UKHHSJANCNFSM5EYTPWZQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--

Brandon Lee Kitajchuk*
kitajchuk.com https://kitajchuk.com

I get an error when I try to run this branch.  It happens directly after logging in successfully, and I can't quite figure out the issue.  Any ideas @danjenkins ?
I am running this branch against your PR on the jambonz-api-server.  To recreate, if you have that branch checked out you can simply run the backend
# in jambonz-api-server
npm run integration-test

once it comes up, you can then run the webapp
# in jambonz-webapp
npm start

You should be able to log in, change password, and then see this error

Its something in that final commit, because if I test the version before that I do not get this error

It’s the ternary “ : null “ that’s causing the problem.
On Sun, 7 Nov 2021 at 19:34, Dave Horton @.***> wrote:

@.**** commented on this pull request.
In src/components/pages/internal/AccountsList.js
https://github.com/jambonz/jambonz-webapp/pull/25#discussion_r744305094:

@@ -222,7 +222,7 @@ const AccountsList = () => {
{ header: 'SIP Realm',            key: 'sip_realm' },
{ header: 'Registration Webhook', key: 'url_reg'       },
{ header: 'Queue Event Webhook',  key: 'url_queue'       },

     { header: 'Subspace',             key: 'subspace_enabled' },

     process.env.ENABLE_SUBSPACE ? { header: 'Subspace', key: 'subspace_enabled' } : null,

Looks like this line is causing the error, due to null being added to the
array
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jambonz/jambonz-webapp/pull/25#pullrequestreview-799576213,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB3LLLUPGJZBJD33Y5GN7TUK3IC7ANCNFSM5EYTPWZQ
.

another issue I am struggling with is that even though I have "ENABLE_SUBSPACE=1" in both my .env and .env.local files, I do not see the subspace entry fields in the account form when I open it.  Any ideas on that?

Odd. I couldn’t test the enable_subspace changes due to moving over to my
m1 mac… now I think about it I think the env vars get REACT_ prefixed
during the build process - will see if I can find the docs on that
On Sun, 7 Nov 2021 at 19:40, Dave Horton @.***> wrote:

another issue I am struggling with is that even though I have
"ENABLE_SUBSPACE=1" in both my .env and .env.local files, I do not see the
subspace entry fields in the account form when I open it. Any ideas on that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jambonz/jambonz-webapp/pull/25#issuecomment-962668640,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB3LLO5NZPGDZSMGLX5XUDUK3IZJANCNFSM5EYTPWZQ
.

https://create-react-app.dev/docs/adding-custom-environment-variables/
So the env var needs to start with REACT_APP_
On Sun, 7 Nov 2021 at 19:42, Dan Jenkins @.***> wrote:

Odd. I couldn’t test the enable_subspace changes due to moving over to my
m1 mac… now I think about it I think the env vars get REACT_ prefixed
during the build process - will see if I can find the docs on that
On Sun, 7 Nov 2021 at 19:40, Dave Horton @.***> wrote:

another issue I am struggling with is that even though I have
"ENABLE_SUBSPACE=1" in both my .env and .env.local files, I do not see the
subspace entry fields in the account form when I open it. Any ideas on that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jambonz/jambonz-webapp/pull/25#issuecomment-962668640,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB3LLO5NZPGDZSMGLX5XUDUK3IZJANCNFSM5EYTPWZQ
.

OK, thanks, that explains that.
I now have a question on functionality.  I tested putting in my credentials and enabling sip teleport, and the API worked fine as I can see in the webapp, I did in fact create a sip teleport:

However, if I go over to the subspace console I can see the teleport hasn't been fully configured -- it doesnt have the info as to where to send me the traffic:

This makes sense, I suppose, since when I enabled the subspace teleport in jambonz I was not prompted for the SIP IP:port of jambonz.  So this is a feature that I should need to add on this branch, correct?  Just want to make sure I am not missing something

This is great.  I have a few more small changes to make so I am going to push a new branch that is rebased on the latest main and work them there.  Will open a new PR for that and close this one.  Many thanks @danjenkins !


PT-1633 | Suggested translations update

Description :sparkles:
Note: translations from screenshots are not found, or fixed earlier
Issues :bug:
Closes :no_good_woman:
PT-1633:
Related :handshake:
Testing :alembic:
Automated tests :gear:
Manual testing :construction_worker_man:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

kultus-admin-ui branch is deployed to platta: https://kultus-admin-ui-pr357.dev.hel.ninja :rocket::rocket::rocket:


Error in diamond makedb related to taxonmap file

Hello,
I've followed the tutorial to build a diamond database hgtector 2.0b2:
taxon_map="hgtector_db/taxon.map.gz
echo $'accession\taccession.version\ttaxid' > prot.accession2taxid
zcat $taxon_map | awk -v OFS='\t' '{split($1, a, "."); print a[1], $1, $2}' >> prot.accession2taxid

diamond makedb --threads 24 --in  db.faa --taxonmap prot.accession2taxid \
--taxonnodes taxdump/nodes.dmp --taxonnames taxdump/names.dmp --db diamond/db

and I found this error at the end of the process (after diamond has already created a 30GB plus file):
Loading sequences...  [2.659s]
Masking sequences...  [3.314s]
Writing sequences...  [0.993s]
Writing accessions...  [1.747s]
Hashing sequences...  [0.21s]
Loading sequences...  [0s]
Writing trailer...  [2.748s]
Loading taxonomy...  [0.258s]
Error: Tokenizer Exception
Error: Tokenizer Exception

I didn't find much information about this, so I tried a few things and finally using the prot.accession2taxid.gz (6.1GB) from the NCBI solved the problem.
The original prot.accession2taxid (created following the instructions) is 500MB gzipped  (still can share it if you want). If that's too much, maybe some hints about what could be causing this problem will be helpful. The file looks OK to me, so I'm not sure to  what to look at.
Cheers!

I had the same problem, but I solved it with a specific version of Diamond. I tryed with many different versions and the version 0.9.26 was the only one that worked properly in your steps

Thanks for letting me know @zakisaati .

I tried the last version of diamond prior v2, 0.9.36, and  it worked. Maybe that information can be added to the README

I tried the last version of diamond prior v2, 0.9.36, and  it worked. Maybe that information can be added to the README

This issue was eventually solved in #97 . Sorry for being so late... Now any version of DIAMOND should work.


how to use fifter box,i saw but i can't find it

can't see fifter box

You can click on the little triangles inside the draggable bubbles to open
the filter box.
On Thu, Nov 29, 2018 at 23:00 huangjiaxiang<EMAIL_ADDRESS>wrote:

can't see fifter box
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nicolaskruchten/pivottable/issues/1055#issuecomment-443083444,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMbA65Id_S5aS59IDAUmk1QRyavdFf_ks5u0K1jgaJpZM4Y66nD
.

my means is search fileds,my fileds are too mucn

If anyone stumbles on this, this issue may have been made because the filters are on the column-side instead of horizontal above the rows. My experimentation showed that depending on the length (and number) of the headers, the filter panel will dynamically move between the two positions.


[FEATURE REQUEST] Nix packaging

I don't think this will be easy, because of the way that Nix works but it would be a nice to have.

I don't know how to use Nix, but I'm not opposed to this, would you be interested in getting this done?

I can look into it, but my time is pretty scarce. If I don't do it myself I will put some resources here for anyone to do.


Remove request_id from inviter connection record

Signed-off-by: Ian Costanzo<EMAIL_ADDRESS>Addresses issue https://github.com/hyperledger/aries-cloudagent-python/issues/1541
Tested in alice/faber, there are some error messages on the faber side however the connection (didexchange) is established and can be used for issuing credentials and requesting proofs.
Not sure what is the issue of removing request_id from the inviter connection record, or why it's required for didexchange but not connections protocol.

@ianco -- what do we do with this one?  Thanks

Replaced by PR #1670


[logging] Avoid logging cancellation as warnings/errors

Cancellation shows up as an error code, but it is not a real error, so
don't log it like one.
https://bugs.swift.org/browse/SR-13344

@swift-ci please test


Empty Subform Pages

Dear Base-App-Team,
there are a few empty pages (IDs 7172..7176 and 7182..7188) which seem to be legacy pages from back when doing page-Transformation from Classic Client forms to RTC pages.

Maybe now, when we're all staying home, is the perfect time for some clean up? ;-)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Have a way to clear cache

I'm writing tests for a component using useSWR, also with { suspense: true }, and if I render a component reading the same key (e.g. the same component with different keys) I'm getting the old cache in my second test.
Not sure if this will be addressed with #158, but having a way to tell SWR to clear the cache of a certain key would be awesome for this use case.
import { clearCache } from "swr";

clearCache(); // clear all cache
clearCache("/api/data"); // clear only a certain cache key

☝ something like that.
What I have tried:

mutate(key, null, false)
mutate(key, {}, false)

Both doesn't work right since useSWR will still read that null or {} from the cache instead of triggering an immediate fetch and suspending the component.
I will try to create a CodeSandbox to replicate this soon.

For testing purposes what I made to solve this is to ensure I'm always using different keys, since they are based on a prop, but still would be nice the be able to do:
import { clearCache } from "swr";
afterEach(() => clearCache);

And ensure all my tests have zero cached data.

What do you think about the idea of exporting the cache object directly?

Another thing I've been thinking is #92, which will only remove the inactive keys.
If we clear/change the cache of an active SWR hook, it might cause some unexpected behavior.

So far I haven't needed, until now, direct access to the internal cache, with it I could have done what I needed creating the clearCache function myself.
I agree that cleaning the cache it's not something you should want in production, I only need it for testing purposes.
Does SWR listen to cache changes? Maybe this way if an active SWR hook is using a cleared key it could automatically trigger a new fetch as if it was new? 🤔 This way if you still have a rendered component using the cleared key it wouldn't break, just change to loading state as if it was a new key? Just thinking out loud.

Agree to expose a way to clear the cache, it's a common scenario.
For example, with a simple component that fetches data
it('should fetch on mount', async () => {
    await act(async () => {
      <Foo id="1" />
    });

    expect(fetch).toHaveBeenCalledWith('1');
  });

  it('should show an error when fetch fails', async () => {
    (fetch as jest.Mock).mockRejectedValue(new Error('An error occured'));

    await act(async () => {
      mount(
          <Foo id="1" />
      );
    });

    expect(fetch).toHaveBeenCalledWith('1');
   // other assertions that the error message is rendered
  });

The second expectation fails.
The only way to test this behavior is to change the id propr, but it's not always feasible, depending on the test scenario

Awesome!

@quietshu something is weird here. When testing, my second test stills renders with the result provided by a previous test.
I tried a cache.clear() in a beforeEach and before rendering the component within the test, both fails.

@andreoav check my answer here https://github.com/zeit/swr/pull/231#issuecomment-591614747

@quietshu something is weird here. When testing, my second test stills renders with the result provided by a previous test.
I tried a cache.clear() in a beforeEach and before rendering the component within the test, both fails.
If I run each test at a time they pass.

I'm having the same issue. I think the cause is the cache being global instead of being contextualised.

@sergiodxa
I am using a similar idea on my SWR vue library. In fact, i find this statement wrong:

I agree that cleaning the cache it's not something you should want in production, I only need it for testing purposes.

I've written some applications where the cache TTL is about 1 minute. So, if I visit the dashboard and it fetches some data with a TTL 1 min, then logout and login with another user (with less than a minute apart) then that cache still shows the old user values and does not revalidate yet due TTL < 1 min.
The quickest solution is to delete all cached keys or the ones specified at logout, this ensures fresh data upon login.
This is the clear function I did: https://github.com/ConsoleTVs/vswr/blob/6a84e923d9db31784316b975f98db1f5eaea3140/src/swr.ts#L219

@ConsoleTVs what i usually do is to reload the page in a logout so cache is cleared, also most users don't login, see data, logout and see new data in less than 1 minutes IMO.
However, with the current cache you can do it with cache.clear() if you don't reload on logout.

Just keep in mind, that the (mobile) application we're building probably have a safe cache of Infinity, since we know exactly when we need to reload / mutate it via events, 1 min was just for testing :)
Nevertheless, is it possible with cache.clear() on react's SWR? How do you access the cache instance then?

You can import the cache and clear it with:
import { cache } from "swr"
cache.clear()

This has been available since #231 was merged

Is this actually documented on the docs page? Very interesting indeed

It's not documented yet

Ha! Been looking for this, can someone please document this, very important feature.

what i usually do is to reload the page in a logout so cache is cleared

@sergiodxa  Page reload is not possible in a React Native app.  😀
I was surprised the documentation didn't cover this.  I would have expected it to be at the bottom of the Mutation page which deals with cache updates anyway.
This also simplifies handling authentication, as you can just clear the cache every time authentication changes. In this case the fetcher method can obtain access tokens itself instead of passing them in the key every time (see https://github.com/vercel/swr/issues/325#issuecomment-817772759).

This seems to have been updated at some point, but couldn't find the exact commit or PR when.
I was able to create a new cache on each test with this:
import { SWRConfig } 'swr';

<SWRConfig value={{ provider: () => new Map() }}>
  {/* SWRConfig is given a new provider on each test so that it's caching is emptied between each test */}
  {children} {/* or the component you want to test that uses `useSWR` */}
</SWRConfig>

You can import the cache and clear it with:
import { cache } from "swr"
cache.clear()

This has been available since #231 was merged

What version of swr is this?

Going back @tobslob 's question. I am using 1.1.2 version of swr and clear method is not available on cache, only cache.get cache.set cache.delete, at least, it's not available according to typescript

Doesn't seem like the cache is being exported any more as of v1.1.2. However @andersnylund's solution of providing the wrapper works great.
Also, for those of you who want use <SWRConfig /> with renderHook(...) the way to to that would be:
const wrapper = ({ children }) => <SWRConfig value={{ provider: () => new Map() }}>{children}</SWRConfig>

test('test something', () => {
  // provide a "clean" wrapper for `swr`
  const { result } = renderHook(useFetchWithSWR, { wrapper })

  // etc...
})

@andersnylund method of using a wrapper with a cache provider is the way to do this as of 1.0.0 per the docs: https://swr.vercel.app/docs/advanced/cache#reset-cache-between-test-cases

@JamieKudla they listerally lidt a clear method at the end of thst page, is that not working?

There is no function defined to delete all caches. Is the documentation wrong?
https://swr.vercel.app/docs/advanced/cache#reset-cache-between-test-cases

@andersnylund Looks to do the trick, same solution for Storybook using a decorator:
import React from 'react';
import { DecoratorFn } from '@storybook/react';
import {
  SWRConfig,
} from 'swr'

export const StoryDecorator: DecoratorFn = (Story: any, {}) => {
  return (
    <SWRConfig value={{ provider: () => new Map() }}>
      <Story />
    </SWRConfig>
  );
};

export default null;

@Ajido
Type definition of cache is not up to date but the code actually works. If you want to fix the type issue, you can override the Cache type by adding this:

declare module "swr/dist/types" {
    export interface Cache {
        clear(): void;
    }
}

This is super awkward to implement.  Would love a clear way to clear cache for all tests.

Please checkout https://github.com/vercel/swr/issues/1887#issuecomment-1171269211 for the new cache clear approach introduced in v2 beta


FancyZones sometimes does not detect taskbar height on windows startup and snaps windows 'under' the taskbar

Microsoft PowerToys version
0.59.1
Running as admin

[ ] Yes

Area(s) with issue?
FancyZones
Steps to reproduce

Re-start Windows 10 or Windows 11 PC.
Snap a window (e.g. Edge browser) to a zone that goes full height of the desktop (e.g. the shipped 'Priority Grid' layout).
Observe where window snaps-to

✔ Expected Behavior
The bottom of the window aligns with the top of the taskbar.
❌ Actual Behavior
The bottom of the window aligns with the bottom of the screen and part of the window is hidden behind the TaskBar.
Disabling and re-enabling FancyZones, then re-snapping the window results in the correct behaviour.
It appears that during windows startup, FancyZones might be establishing the available window height before the taskbar has been fully initialised. If FancyZones happens to establish the available window height after the taskbar has been initialised on startup, the behaviour is not observed.
Other Software
No response

Same issue here, after quit and reload PowerToys, FancyZones runs correctly

Can confirm that this happens on Windows 11 as well.

Fancy zones has been driving me crazy by clipping off the bottom of the window and the only way around it was to either manually size the window or hide the taskbar and un-hide it again (this works until you move the window again)
I'm running on a laptop with an ultra wide monitor, I like splitting this in two when I'm not working with wide apps.

As mentioned the bottom of the app would always be clipped, when using Excel this is especially a pain because you cannot switch sheets easily.
On checking my Windows settings after reading #5935, I could see the windows are clipped on the top edge:

Changing this so the monitor is clipped on the lower edge fixed the issue immediately (on Windows 10 x64)

Reloading Fancy zones does work as a workaround but this seems to fix it permanently so maybe that helps pinpoint the bug in working out the window heights.

I do have the same issue intermittently, using 2 simple Full HD (1920x1080) displays without any fancy scaling. Restarting PT does fix it as well here. An unconfirmed guess is that it is related to display configuration changes, as I'm too using a notebook, but on a docking station and with the internal display disabled, just using the two displays via the dock. Right now, the system had rebooted after windows update and started the session in the background (as Windows 10 does these days). Maybe this is related to it, I could imagine that the taskbar is not yet rendered if the user session launches in the background? Could anyone here on the issue confirm this guess?

Same issue here, after quit and reload PowerToys, FancyZones runs correctly

I don't have to fully restart PowerToys, just turning off and back on FancyZones from the PT settings panel seems to do the trick for me.

Still an issue today

Still an issue on 0.64.0, running as admin.
Toggling FancyZones off and back on works.
I haven't fully verified, but it does seem to happen most often on reboot or possibly when waking from sleep. I'd also suspect that zones might be jumping ahead of the taskbar getting initialized.

/dup #10903