chore(dao): Replace save/overwrite with create/update respectively

SUMMARY
Per  SIP-92 Proposal for restructuring the Python code base https://github.com/apache/superset/pull/24331 organized all the DAOs to be housed within a shared folder—the result of which highlighted numerous inconsistencies, repetition, and inefficiencies.
This PR (one of many smaller bitesized PRs) replaces the save and overwrite methods with create and update methods respectively which removes a bunch of copypasta.
Note this PR required a fair amount of yak shaving as the create and update methods needed to be updated to support a combination of a instance and/or the attributes representing the instance.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[x] Has associated issue: https://github.com/apache/superset/issues/20630
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@michael-s-molina I've hopefully addressed/answered your questions? Would it be possible to (re)review the change? Once merged I was hoping to make some inroads in terms of how we commit/rollback changes as well as grouping all the commands together. Ideally these tasks will be easier to manage after once this PR is merged otherwise the merge conflict could be rather convoluted.


Windows compatible?

Sorry to ask here. I loved what I saw on two minutes paper and wanted to try, but I have almost no experience with c++
And I saw that gc++ is needed, so I thoungh it was aimed for linux distribuitions...
And what an amazing job you did @art049 incredible!!

Mingw is gcc/g++ and some extra's for windows. I'm currently working on building this project for windows and I'll come back when/if I get it to work. (Direct link to Mingw for windows)

Hello @Creativity-is-Lacking
After 3 full days trying, I was able to build in windows using VS2019
Here is the fork I've done
https://github.com/carlosedubarreto/InteractiveWoodCombustion
But a sad thing.... The only thing that was working was the combustion part of the project 😥
After those 3 days compiling, I just got a smoke simulation like this one

Well, at least it compiled... I'll take a look and see if what I can do. :)

To be honest..... the only thing that it helped me was to have courage to dive into C++.
The CUDA SDK example (that comes when you install the SDK could be a much better test 😊
If you want to test what I did. I used the CUDA 10.1 to compile.

Unfortunately, my cuda is still acting up. On the bright side, however, I was able to run the binaries you compiled! the first major thing that jumps out at me is the shader path. If your shader files aren't located at "D:/CPP/int_wood v3 (CUDA)/x64/Debug/src/shaders/shader.vert" than update the shader location in main.cpp line 124. Other than that it seems odd that you can see a camera box upon zooming out, but I'm long out of time for today...

Oh yeah, I had coded the shader location for me.. but, it didn't make much difference (setting up the correct location for the files)..
Lol... I got that same feeling when I saw only a camera and a gray ball 😅

Today marks the day when I have given up on fixing my cuda install. Using smoke, magic, mirrors, and virtual machines I have finally been able to compile it! I should be able to be more helpful now 😄

Yey, congrats buddy 🎉
I got confused with the message, but in the end I think it was a good final result 😅


Permission denied (publickey)

Hello, looks like I need some help with initial configuration :)
What ssh key is used by default?
And where these configs are stored?
gerrit config -a
name = default
host = [ 'localhost' ]
user = [ 'jenkins' ]
port = [ '29418' ]

gerrit clone playground

Cloning project playground from default config into folder playground...
Cloning into 'playground'...
Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
(node:70212) [DEP0097] DeprecationWarning: Using a domain property in MakeCallback is deprecated. Use the async_context variant of MakeCallback or the AsyncResource class instead.
gerrit ERROR There was an error running a git command
gerrit ERROR   command: 'clone' '--progress' 'ssh://[ 'jenkins' ]@[ 'localhost' ]:29418/playground.git' 'playground'
gerrit ERROR   output:
gerrit ERROR   error: Cloning into 'playground'...
Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
gerrit ERROR   code: 128

cat ~/.ssh/config

host *localhost*
hostname <IP_ADDRESS>
User jenkins
Port 29418
TCPKeepAlive yes
IdentityFile ~/.ssh/id_rsa

git clone --progress ssh://jenkins@localhost:29418/playground.git

Cloning into 'playground'...
remote: Counting objects: 65, done
remote: Finding sources: 100% (65/65)
remote: Total 68 (delta 22), reused 68 (delta 22)
Receiving objects: 100% (68/68), 7.44 KiB | 0 bytes/s, done.
Resolving deltas: 100% (22/22), done.
Checking connectivity... done.

Simply seems like your key pair wasn't registered with the server (which appears to be localhost?)
I wouldn't think that this would be a problem with gerrit-cli, are you able to performa a regular git clone of the project?

Yeah i can do regular git clone, but gerrit clone is failing.

Same problem here:
gerrit --loglevel 10 --verbose --debug ssh git-receive-pack
gerrit ERROR UNKNOWN<EMAIL_ADDRESS>Permission denied (publickey).

I have met the same problem and may I know how u solve it?


Convert to babel 6 plugin?

This looks really awesome! Do you have any plans to convert this to work under Babel 6?

Very good idea @sirbrillig! Any chance you could lend us a hand on this? Drafting out what would need to change would help, too.

Happy to try, although I've never written a babel plugin so this will be a learning experience!
I've given it a (naive) start over at https://github.com/sirbrillig/babel-plugin-jsx-to-handlebars/tree/update-to-babel-6
I've updated the dependencies, converted the build process, and extended the JSX syntax plugin, but now there's a lot of details.
I'm getting a little stuck on the transformation API changes. Currently I've got it to throw this error:
Error: source/test.jsx: Unknown element during JSX processing:  JSXText

I'm not sure on how the babel APIs have changed but I think there is probably a problem in one of the JSX transformations in the plugin. Debugging this kind is not that easy, so I would recommend you to add try-catch blocks inside each method and check if there are any errors thrown. That will give you at least some hint and hopefully a stack trace.

Thanks a lot at @sirbrillig! I took the freedom to take your branch and go some steps further: https://github.com/sinnerschrader/babel-plugin-jsx-to-handlebars/tree/sirbrillig/update-to-babel-6 . This does not crash right away when executing the tests but still needs some work around function closures. Could you have a look @KnisterPeter ?


feat(handlers): add group DM function

Please describe the changes this PR makes and why it should be merged:
Status

[ ] Code changes have been tested against the Discord API, or there are no code changes

Semantic versioning classification:

[x] This PR changes the library's interface (methods or parameters added)

[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)

[ ] This PR only includes non-code changes, like changes to documentation, README, etc.

closing this due to oauth2 not being implemented in rn.

closing this due to oauth2 not being implemented in rn.


MLP: Multilayer Perceptron

JIRA: MADLIB-413
Add train and predict for multilayer perceptron.

Jenkins, ok to build.

Can one of the admins verify this patch?

jenkins ok to build

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/109/

Not sure what this is. While playing around with the module, I accidentally hit a nan condition. Here is the output of the session:
madlibtest=# CREATE TABLE iris_data(
madlibtest(#     id integer,
madlibtest(#     attributes numeric[],
madlibtest(#     class_text varchar,
madlibtest(#     class integer
madlibtest(# );
CREATE TABLE
madlibtest=# INSERT INTO iris_data VALUES
madlibtest-# (1,ARRAY[5.1,3.5,1.4,0.2],'Iris-setosa',1);
INSERT 0 1
madlibtest=# SELECT madlib.mlp_classification(
madlibtest(#     'iris_data',      -- Source table
madlibtest(#     'mlp_model',      -- Destination table
madlibtest(#     'attributes',     -- Input features
madlibtest(#     'class_text',     -- Label
madlibtest(#     ARRAY[5],         -- Number of units per layer
madlibtest(#     'step_size=0.003,
madlibtest'#     n_iterations=5000,
madlibtest'#     tolerance=0',     -- Optimizer params
madlibtest(#     'tanh');          -- Activation function
WARNING:  loss: nan
ERROR:  spiexceptions.UndefinedColumn: column "nan" does not exist
LINE 4: ...7731e-05,-0.000170379656714,0.000110138191401,1.0,nan,1.0,1....
                                                             ^
QUERY:
                SELECT madlib.internal_mlp_igd_distance(
                        (NULL)::DOUBLE PRECISION[],
                        ARRAY[ 2.0,4.0,5.0,1.0,0.003,1.0,2.0,1.0,1.0,1.0,1.0,1.0,-0.000161677781273,2.34991541167e-05,-5.27816928274e-05,1.6207561377e-06,9.30167921009e-05,-2.2322484638e-06,-8.8803764219e-05,6.92299455941e-05,0.000174430298478,1.36067005437e-07,-0.000126402572755,0.00010846859258,-0.000131999662994,-0.000111939884066,0.000154233933791,9.62727960033e-05,7.92279598497e-05,3.55520197415e-05,4.28280053118e-05,-0.000136356574146,-3.07769363501e-05,-8.97131375537e-05,-0.000117176292437,-9.89281636713e-05,0.000234615355418,1.0,1.0,1.0,1.0,1.0,1.0,2.00514836476e-05,-9.37533114569e-05,7.58698692162e-05,-2.64470277731e-05,-0.000170379656714,0.000110138191401,1.0,nan,1.0,1.0,1.0,1.0,1.0,-0.000161677781273,2.34991541167e-05,-5.27816928274e-05,1.6207561377e-06,9.30167921009e-05,-2.2322484638e-06,-8.8803764219e-05,6.92299455941e-05,0.000174430298478,1.36067005437e-07,-0.000126402572755,0.00010846859258,-0.000131999662994,-0.000111939884066,0.000154233933791,9.62727960033e-05,7.92279598497e-05,3.55520197415e-05,4.28280053118e-05,-0.000136356574146,-3.07769363501e-05,-8.97131375537e-05,-0.000117176292437,-9.89281636713e-05,0.000234615355418,1.0,1.0,1.0,1.0,1.0,1.0,2.00514836476e-05,-9.37533114569e-05,7.58698692162e-05,-2.64470277731e-05,-0.000170379656714,0.000110138191401,0.0,0.0 ]::double precision[]) as state_dist

CONTEXT:  Traceback (most recent call last):
  PL/Python function "mlp_classification", line 32, in <module>
    True
  PL/Python function "mlp_classification", line 140, in mlp
PL/Python function "mlp_classification"
madlibtest=#

In regards to 'nan' issue. We have run across this in the past. It has to do with regularization, which is scheduled for phase 2. Once we implement regularization this issue should go away. However I will try recreate this bug and verify that is the true issue.

There are a few varying MLP naming references scattered about.  I believe Multilayer perceptron is correct (no space or hyphen separating multi and layer). I have seen Multi Layer perceptron and Multi-layer perceptron also used.  Can you please clean them up?
Here are those that I noticed:
-*- mode: ack; default-directory: "~/workspace/MADlib/incubator-madlib/src/" -*-
Ack started at Wed Jul 12 16:12:18

ack -i "layer perceptron"
modules/convex/mlp_igd.cpp
55: * @brief Perform the multi-layer perceptron transition step
173: * @brief Perform the multi-layer perceptron final step

modules/convex/mlp_igd.hpp
25: * @brief Multi-layer perceptron (incremental gradient): Transition function
30: * @brief Multi-layer perceptron (incremental gradient): State merge function
35: * @brief Multi-layer perceptron (incremental gradient): Final function
40: * @brief Multi-layer perceptron (incremental gradient): Difference in
46: * @brief Multi-layer perceptron (incremental gradient): Convert
52: * @brief Multi-layer perceptron (incremental gradient): Predict

modules/convex/type/state.hpp
547: *        incremental gradient descent for multi-layer perceptron

ports/postgres/modules/convex/mlp.sql_in
23: * @brief SQL functions for multi-layer perceptron
45:Multi Layer Perceptron (MLP) is a model for regression and
358:-# Generate a multilayer perceptron with a single hidden layer of 5 units.
416:-# Now train a regression model using a multilayer perceptron a single hidden layer of two nodes.

ports/postgres/modules/convex/mlp_igd.py_in
23:@brief Multilayer perceptron using IGD: Driver functions
503:    Multi Layer Perceptron (MLP) is a model for regression and
676:    Multi Layer Perceptron (MLP) is a model for regression and

Ack finished at Wed Jul 12 16:12:18

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/110/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/111/

Upon reviewing the MLP module code coverage metrics produced by running madpack -s madlib -p postgres install-check -t convex/mlp, the following is revealed:

mlp_igd_merge::run is not tested
The relu (rectified linear unit) activation is not tested
The negative case testing for an invalid activation is not tested

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/116/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/117/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/118/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/119/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/122/


Children of children / recursive validation

I want to validate the children of an element that's passed as children.
For example:
<x><y/></x>

Where y is wrapped with injectIntl and, so I want to check in x that y is either of type "A" or type "B", but because it's wrapped, I need to go one level deep.
Is there a way to do this with these proptypes?
childrenOfType doesn't help because it can only check if it's of type injectIntl.

I'm not sure what you're asking - can you provide the actual component code?

class Foo extends Component {

};
...
export default injectIntl(Foo);

export default class Bar extends Component {

  render() {
    const Foo = this.props.foo;
    return <Foo />;
  }
}

FormDropZoneWrapper.propTypes = {
  dropZoneComponent: PropTypes.func.isRequired, // I want here to validate that Foo is actually Foo and not some other component, but it's wrapped by intl so I can't
};

In that case, you'd use PropTypes.oneOf([Foo]) - iow, nobody can ever get at the unwrapped Foo, only the wrapped one, so you only have to validate that it's the wrapped one.


Ensure old authentication services are removed

Ensure old authentication services are removed from the cached map
Unify implementation of Keycloak interaction in KeycloakUserApi and RealmController

qq.  What use-case cause there to be old authentication services to be removed?  Is this an upgrade scenario?

@k-wall its for the case where you

create authservice a
create and delete address spaces using it
remove authservice a
create authservice b
create and delete address spaces

Problem is that 'a' will still have a handle in api-server for instance, so when you query for all messaging users, it will try to contact it if it is not removed. The other bug was that it could be null if it was queried before it was up, which would end up putting a null in the keycloakMap. This is also fixed in this PR.


Shader compilation error in shader editor is classified as internal error where it shouldn't

Tested versions
Godot v4.3.dev (ae51db75e)
System information
Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3060 (NVIDIA; <IP_ADDRESS>19) - 11th Gen Intel(R) Core(TM) i7-11700K @ 3.60GHz (16 Threads)
Issue description
The behaviour is script editor is : when you type in something that won't compile, the error is only shown in the bottom of the editor, and after manully save, the error will appear as internal error notification and logged to the console:

For shader editor, when you type in something that won't compile, after some short delay the error is shown  in the bottom of the editor, notification and console.

Thoughts that open to discussion :

The behaviour is different between script editor and shader editor, and to be honest, the shader editor's default behaviour is a little annoying as if I don't type fast, the error will keep jump out
IMHO these errors that caused by user's code should not be counted as "INTERNAL ERROR" because it's the user's wrong syntax or grammar that caused these error, if the script or shader is internal and it's from the engine part but compiled failed, then it's more like an "INTERNAL ERROR"

Steps to reproduce
Write some code in Shader Editor and observe the error report, compare it to how error is reported in built-in script editor
Minimal reproduction project (MRP)
N/A

in ShaderTextEditor::_validate_script it will call TextShaderEditor::apply_shaders which will set the code for shader and triggers the toast.


Avoid directional correction when the motion is downward

Fix: #58786
We make a slide on the intersection between the plane of the motion and the plane of the ground to keep the correct direction. However, as we use up_direction to determine the motion plane, when the motion is downward it will be stopped.
After:
https://user-images.githubusercontent.com/6397893/157445504-3406328d-e86b-4351-bf5d-b26045c38358.mp4

Thanks!


Allow add_categories() to accept Series/np.array and make raising on dup...

...licates optional, fixes #9927

@mortada ok, for now pls take out the raise_on_duplicates kw and leave the rest.

@jreback  it's updated

@mortada thanks!


server port

Hi, my ip camera has different port than 80...the server node use always port 80, even if i change it. I got this error with debug : " Fetch error request to http://192.168.xxx.xxx/ISAPI/Event/notification/alertStream failed, reason: connect ECONNREFUSED 192.168.xxx.xxx:80 ", my port is 81..
Thanks

Hi Mateuic
have you starred my node? Remember to do it :-)
Today a new version with that fix will be out. I'll advise you when ready.

V. 1.0.42 is out for the fix.
Thank you for check it out and let me know if it still not working,.


Share selected row IDs between data providers

Summary
When using multiple Data Grids with data providers attached to each, make it possible to share the id of a selected row on the first grid in the getRecords method of the data provider of the second.
Examples
export default createDataProvider({

  async getRecords({id, paginationModel: { start = 0, pageSize=3 } }) {
    console.log("id",id) // Selected row id of the first grid

          const response = await axios.get(`end-point/${id}?api_key=...`,{
        headers: {
          'Content-Type': 'application/json',
         }
      });

Motivation
From @VitaliiDubinin:

The idea is to have possibility to choose object in the first table, then when I select any object in the first table it will show its nested object in the second table and give me possibilities to make changes in the nested objects by dataProvider.

Search keywords: data provider

I believe this would be solved by https://github.com/mui/mui-toolpad/issues/3052.
When a data provider lives on the page the same way a query does, we could allow binding page state to its filter.

Agreed @Janpot - I'll close this as a duplicate


[Snyk] Upgrade org.apache.kafka:connect-api from 3.4.1 to 3.8.0

Snyk has created this PR to upgrade org.apache.kafka:connect-api from 3.4.1 to 3.8.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 9 versions ahead of your current version.

The recommended version was released on 3 months ago.

Issues fixed by the recommended upgrade:

Issue
Score
Exploit Maturity

Denial of Service (DoS)SNYK-JAVA-ORGXERIALSNAPPY-5710960
146
Proof of Concept

Allocation of Resources Without Limits or ThrottlingSNYK-JAVA-ORGXERIALSNAPPY-5918282
146
Proof of Concept

Integer Overflow or WraparoundSNYK-JAVA-ORGXERIALSNAPPY-5710959
146
Proof of Concept

Integer Overflow or WraparoundSNYK-JAVA-ORGXERIALSNAPPY-5710961
146
Proof of Concept

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.
Max score is 1000. Note that the real score may have changed since the PR was raised.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Fails

:no_entry_sign:

PR Title '[Snyk] Upgrade org.apache.kafka:connect-api from 3.4.1 to 3.8.0': subject may not be empty, type may not be empty

Warnings

:warning:

Commit Message 'fix: upgrade org.apache.kafka:connect-api from 3.4.1 to 3.8.0
Snyk has created this PR to upgrade org.apache.kafka:connect-api from 3.4.1 to 3.8.0.
See this package in maven:
org.apache.kafka:connect-api
See this project in Snyk:
https://app.snyk.io/org/connectors-0mk/project/958677f5-44f0-4f4e-9ed9-06f0da4959d8?utm_source=github&utm_medium=referral&page=upgrade-pr': body's lines must not be longer than 100 characters

Pull request title should follow Conventional commits.

  Generated by :no_entry_sign: dangerJS against 0965be414b1db39c52aab0f1f8a529ac0dae3bf9


fix: issue #4

the span is finished before the actual redis command is executed. so the span does not capture actual execution duration. this PR fixes this issue.

added comment. About CI, I'm good to merge this PR with failing CI.
To solve the problem this project needs to start using releases so it can be compatible with the new and the old version of go-redis. But I don't really have time for it right now.

thanks again!


CP-346: Config Standardization for FE repositories

Description
This PR follows-up on the recent changes to eslint-condif-typescript, in which the new legacy preset was added to support our existing front-end repositories (see).
Further details around this effort, referencing some Turo repositories, are in our internal Jira system (see particular comment).
Even though currently this repository includes 2 presets (default and extended), the goal is to combine them soon, after testing them with some existing packages.
The default is the one resulting from the recent effort described in the ticket; whereas the extended preset resembles the original config of this repo, adding some further react-related plugins and rules.
Fixes #CP-346
Changes

feat: adapted to our front-end repositories

🚀 PR created with fotingo

@Toolo added a note regarding the legacy extension, which I had also thought about. See: https://github.com/open-turo/eslint-config-react/pull/85/commits/99dc61df68cd97e90664e693c9088c6c95fc5dea
@brantphoto Yes, I'm planing on testing both configs on a couple of packages and if those tests are successful, I'll make the extended become the default (and only) config.


Top Left Alignment of Paper.text

It would be awesome if we had another attribute for paper.text() that could specify top-left alignment. It's difficult to put text inside a box, especially if such text is later subject to change ( for example, if it grows, then Raphael will shift it up ).
I suggest we had 'text-anchor':'leftTop' or something similar that could override the code on line 6245 of raphael.js 2.1.0 ... So
dif = a.y - bb.y;
instead of
dif = a.y - (bb.y + bb.height / 2);
if 'text-anchor': 'topLeft' is specified...
Any thoughts on that? Anybody supports this idea?
Fiddle example:
http://jsfiddle.net/patrykkabaj/NGRP9/
I know why this happens - it's because by default the anchor is middle and SVG states that middle is geometrical middle of an object. However, in case of start the vertical middle stays, it would be great to be able to change this behaviour.
Example what currently happens:

I've quickly tried modifying the code but haven't got time now to investigate it properly.
The method I am talking about and last 3 lines:
tuneText = function (el, params) {
        if (el.type != "text" || !(params[has]("text") || params[has]("font") || params[has]("font-size") || params[has]("x") || params[has]("y"))) {
            return;
        }
        var a = el.attrs,
            node = el.node,
            fontSize = node.firstChild ? toInt(R._g.doc.defaultView.getComputedStyle(node.firstChild, E).getPropertyValue("font-size"), 10) : 10;

        if (params[has]("text")) {
            a.text = params.text;
            while (node.firstChild) {
                node.removeChild(node.firstChild);
            }
            var texts = Str(params.text).split("\n"),
                tspans = [],
                tspan;
            for (var i = 0, ii = texts.length; i < ii; i++) {
                tspan = $("tspan");
                i && $(tspan, {dy: fontSize * leading, x: a.x});
                tspan.appendChild(R._g.doc.createTextNode(texts[i]));
                node.appendChild(tspan);
                tspans[i] = tspan;
            }
        } else {
            tspans = node.getElementsByTagName("tspan");
            for (i = 0, ii = tspans.length; i < ii; i++) if (i) {
                $(tspans[i], {dy: fontSize * leading, x: a.x});
            } else {
                $(tspans[0], {dy: 0});
            }
        }
        $(node, {x: a.x, y: a.y});
        el._.dirty = 1;
        var bb = el._getBBox(),
            dif = a.y - (bb.y + bb.height / 2);
        dif && R.is(dif, "finite") && $(tspans[0], {dy: dif});
    },

also the same opinion
but change to
dif = a.y - bb.y;

not works for IE8, anybody has a workaround for IE?

I think you can use the bounding size of the element to compute the top left position.
var bb1 = obj1.getBBox();
var topLeft= {
x: bb1.x - bb1.width / 2,
y: bb1.y - bb1.height /2
};


Issue with k8s.io/docs/setup/independent/high-availability/

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
Wrong markdown formatting for "bash cat >config.yaml < ..." command inside "Run kubeadm init on master0" header.
It is really hard to understand the content of this config. Fix it, please... Thank you.
Page to Update:
https://kubernetes.io/docs/setup/independent/high-availability/#kubeadm-init-master0

Configuration file in the v.1.9 documentation appears similar, and can be used as reference in the interim of v.1.10 documentation getting fixed:
https://v1-9.docs.kubernetes.io/docs/setup/independent/high-availability/

Additional reference for config.yaml, under section "Using kubeadm init with a configuration file":
https://kubernetes.io/docs/reference/setup-tools/kubeadm/kubeadm-init/

yes notice that yesterday , open the issue on wrong section
https://github.com/kubernetes/kubernetes/issues/64826#issuecomment-395175803

Fixed in PR #8663


curl: argument list too long

Just now in a chat with Claude, nvim said:
Error detected while processing function <SNR>12_SendChatMessage[23]..<SNR>12_ClaudeQueryInternal:
line   48:
E903: Process failed to start: argument list too long: "/usr/bin/curl"

Hmm, didn't hit that issue yet.
What platform are you using? (In case it's Linux, what does ulimit -s say?)


Kineticist Thermal Numbus gives enemies elemental resistance

I was checking if Termal Numbus was automated with damage each turn on enemies but discovered that they enemy got the same resistance I and my allies got.
Running 6.2.2
Tested with fire at level 4. Everybody in the area got fire resistance 4.

Greetings,
Please note that Aura AE-likes work by using the party disposition. If you add the Kineticist to the Party Folder, the effect should only apply to allies as demonstrated below:

Let us know if this works.
Thanks!

Yea it works if I put the Kineticist in the party sheet. I will closing this issue. My only complaint would be that how the party disposition works with feats/auras is not that obvious.


Fix deprecated enum-enum conversion warnings in C++20

Merge requests to Eigen have just been casting to int to fix the warning.

Thanks!


Replace EH version with its current latest release

Updates the ExpansionHunter (EH) workflow to use Illumina's current latest release, replacing an in-house built version. Specifically:

[x] Update the Dockerfile;
[x] Add a thread_count workflow option since EH's current latest release is multi-threaded;
[x] Remove timings since it the output of the in-house built;
[x] Export all the JSON outputs of EH in addition to converting them to TSV and combining the output of multiple shards;
[x] Change the default disk type from HDD to SSD since the process is I/O-bound. In a follow up PR, we can extend the RuntimeAttr so it takes the type of disk and applies accordingly.

Thanks for reviewing this @cwhelan. seeking mode is advantageous when running on a small variant catalog, and streaming mode is faster when running on a larger catalog. So, I guess they are both useful for different use-cases.

OK, sounds good to leave it in there!

Thanks for the review @cwhelan!


bootique-jdbc-test: support for loading test data from CSV

Trying to scale our test setup. It is not very convenient to define large data sets in the Java code. Will be much easier to store them in a text file like .csv and load them via Table objects.  This way the datasets can be created form spreadsheets or using DB query tools.
I guess the most labor-intensive part of this task will be proper type conversion.

Initial implementation is in. Basic datatype support is tested on Derby. Other DBs may require explicit String to value converters.


ci: on main commits and cache node_modules

What is the status of this?

I think it cannot be merged as is, need to recheck


[bug]: Drawer hides scrollbar and causes page to resize

Describe the bug
By default, when opened, the Drawer add classes to both the html and body, removing the scrollbar and adding some classes to compensate for it, but the page in the background still resizes.
https://github.com/user-attachments/assets/abaa7a6b-182b-4d4c-9788-fa3d42e80a8f
I don't know if that's an expected behavior, since the Drawer is mainly used in mobile and that doesn't seem to happen there.
Troubleshooting

I tried applying the zeroRight class from the library to both the html and body (separately), but that was ignored when the Drawer opened.
I tried scrollbar-gutter as well but that would be ignored when the Drawer was opened.
In my search for a solution, I saw many people complaining about the Radix Dialog having problems with position fixed.
I added the noBodyClasses prop on the Drawer as false, which stops the Drawer from adding the following classes to the body component:

Which allowed the following classes, added by react-remove-bar, to be active (I can't actually tell if it indeed comes from the library, I'm assuming that based on the names).

Now that the body is display relative, the padding added to the html works as well as the margin-right in the body, creating a double-compensation for the lack of scrollbar. At first I tried removing the margin, and at first glance that seemed to solve the problem, but then realized that when you move the Drawer around, the padding applied in the html is set to 0px, causing the page to resize again.
My workaround
I've added the noBodyClasses prop to false as default in the component and removed the shouldScaleBackground as it made no sense to me to leave it as true by default:
// If you want you can declare this props when using the Drawer, but in my case it made more sense to do it this way.
const Drawer = ({
  noBodyStyles = true,
  ...props
}: React.ComponentProps<typeof DrawerPrimitive.Root>) => (
  <DrawerPrimitive.Root noBodyStyles={noBodyStyles} {...props} />
);
Drawer.displayName = "Drawer";

and the following code to the global.css file:
@layer base {
  html {
    padding-right: 0px !important;
  }
}

So far I found no drawbacks in using the Drawer that way. No margin is added to the page when there's no scrollbar and the Drawer worked the same in the mobile*. I think this happens due to something in the Vaul library (as using the noBodyClasses and some CSS tweaks solve the issue), but I don't think I have the knowledge required to fix it directly in there.
*Its top is being hidden by the dynamic URL bar in the bottom of the mobile nav, but I think this is easily solvable with dvh. I haven't tried it yet.
Affected component/components
Drawer
How to reproduce
This behavior can be seen in the Shadcn's Drawer component page, although it's not that noticeable as in my example 'cus they're using the Scale Background thing.

Go to Shadcn's Drawer page
Open the first Drawer and close it paying attention to the resize that happens when the scrollbar is rendered
To make it more noticeable, you can open the dev tools and resize the page to a size where the components in the page are more cramped

This will not happen if you toggle device toolbar
Codesandbox/StackBlitz link
https://ui.shadcn.com/docs/components/drawer
Logs
No response
System Info
Mac M1 running Sonoma 14.4.1,
Arc Browser 1.51.0 (51691)

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

Same issue. Stuck for 2 days

having the same issue here but for sheet and select components

having the same issue here but for sheet and select components

@soheil0082 just try using the atribute modal=true in the component. Usually is the root component:

@soheil0082

Here is my temporary solution, u can set className w-[100vw] on  tag to resolve
 <html
      lang={lang}
      dir={dir}
      className={`${inter.variable} ${montserrat.variable} ${roboto.variable} w-[100vw]`}
      suppressHydrationWarning
    >

Try set property  preventScrollRestoration={true} on <Drawer>, it works for me

Having the same issue. Using disablePreventScroll={false} on <Drawer> fixes this for me.

I had the same issue as well. What I ended up doing was wrapping my body contents in a div and setting the padding, etc there. This gets around the issue of the drawer component overriding styles when opened.
My Root Layout:
export default function RootLayout({
  children,
}: Readonly<{
  children: React.ReactNode;
}>) {
  return (

      <body
        className={`${notoSans.className} flex flex-col items-center justify-center`}
      >

          <ThemeProvider
            attribute="class"
            defaultTheme="system"
            disableTransitionOnChange
          >
            <NavBar />
            {children}

          </ThemeProvider>

  );

Thanks for creating this thread @gabriel-mopereira
Hope this helps someone!

I had the same issue as well. What I ended up doing was wrapping my body contents in a div and setting the padding, etc there. This gets around the issue of the drawer component overriding styles when opened.
My Root Layout:
export default function RootLayout({
  children,
}: Readonly<{
  children: React.ReactNode;
}>) {
  return (

      <body
        className={`${notoSans.className} flex flex-col items-center justify-center`}
      >

          <ThemeProvider
            attribute="class"
            defaultTheme="system"
            disableTransitionOnChange
          >
            <NavBar />
            {children}

          </ThemeProvider>

  );

Thanks for creating this thread @gabriel-mopereira Hope this helps someone!

This worked for me with other components as well. Thank you!


fix: pass cache control header for static files

Description
The cache_control that is set on a router created by create_static_files_router isn't considered because the return type of the handler function is an async callable so the resolved headers are ignored here. As a workaround, the cache control header is resolved and passed directly to the ASGIFileResponse.
It may have been better for the StaticFiles.handle to return a File response so that this would be handled as per our normal flow and there would be no need for special casing here. This would also allow the users to provide any headers/cookies they might want to attach to the router. However, this would be a breaking change and this could be a change considered for the next major version.

Closes
Fixes #3129

It may have been better for the StaticFiles.handle to return a File response so that this would be handled as per our normal flow and there would be no need for special casing here. This would also allow the users to provide any headers/cookies they might want to attach to the router. However, this would be a breaking change and this could be a change considered for the next major version.

Yeah, the whole StaticFiles app thing isn't really needed anymore and just kept around for backwards compatibility.


Fix tox

These changes were supposed to have landed as part of 1ac4422.

Im looking into why this failed CI atm.

I had to add an empty placeholder unit test otherwise stestr fails if it doesn't discover any tests.

this is now good to merge. pep8 and pylint fixes will be fixed in subsequent patches.


Remove duplicates from Find usages result

I'm sure this becomes kind of obvious after some time dealing with FCS, but commenting explicitly about usage of the distinct in those cases helps highlight some the necessary hoops that are needed.
At later stage, this constitute feedback for FCS API, which you rightly hinted once, could benefit higher level constructs for easier consumption.
I know comments can go out of date (which can be picked up in code review) but having those comments can make the codebase less intimidating, and for sure can avoid mishaps on later work around that place.
It is cool to not have duplicates in find usages 👍

@smoothdeveloper done :)


EM27 webscraper

This PR adds a module for obtaining the EM27 properties (the properties displayed in the "EM27 SOH Monitor" panel on the original GUI).
The properties are provided on a web server set up by the EM27's computer, thus this module provides web-scraping functionality to obtain the data from the corresponding web page. I have also included a snapshot of the webpage for testing functionality when access to the live server is not possible.
The module contains simple methods for opening and closing the connection to the server, reading the source HTML, and parsing this to return the data relating to the monitored properties. The expectation is that these data will be transmitted using pubsub to be displayed on the GUI.
Closes #109

Hey @dc2917. I'm guessing you forgot to tag me as a reviewer for this...?

Hey @dc2917. I'm guessing you forgot to tag me as a reviewer for this...?

I was going to give it one last look over before requesting reviews, but yeah I reckon it's review-ready now

Btw you need to request a re-review from @dalonsoa because it won't let you merge this if one of the reviews is stuck at "changes requested".


Can my checkpoint be published as an example?

Hi, I think for many people spending $50 will seem like a pretty big deal. I have already spent and would like to share my checkpoint at 160it.
Is there any way to give my checkpoint as an example so people can play without spending a lot of money?
I can't spend much time working on the project myself, but it will be interesting what people can do.

Thanks for sharing.bro

I have to use vpn for openai api. I don't how to use proxy for Voyager.

Thank you so much for your efforts @daswer123! We will release our checkpoints and include yours as an additional example soon!

Cool, It will be interesting to look

Hi, @daswer123. Thank you very much for sharing your checkpoints. We included your checkpoints in Community Contributions.

Nice, thanks :)


[#710] Update color palette

Fixes #710
Changes

Updates color design tokens to the new colors, taken from the styleguide in zeplin
Color variations now use the naming format light-4, light-3, light-2, light-1, base, dark-1, dark-2, dark-3, dark-4, instead of lightest, lighter, light, base, dark, darker, darkest. Note that there’re new points on the scale — dark-4 and light-4 did not exist in the old scale.
Updates color usage throughout the codebase

How to QA
Pull this branch, and run yarn storybook.
Most changes in this PR are updating the use of u-fg, u-bg, and u-border utility classes. The core changes are in settings/_color-palette.scss.
Check the color-palette page in storybook, and click around some components.
⚠ the components may not look beautiful — updating them to properly use these colors will come in future PRs, when we update the base styles and component styles to match the new designs (see zeplin).
📸

Checks
Delete if not applicable:

[x] Storybook documentation has been updated
[x] Fixtures in test/scss/ have been updated
[x] Your changes have been added to the unreleased section of CHANGELOG.md

@ed-eva thanks for the review 🙏 I applied all your suggestions


fix: use GitHub PAT to avoid 'early EOF' errors in workflow runner

Description
Use GitHub PAT (personal access token) to avoid 'early EOF' errors in workflow runner. This essentially replaces the use of SSH in the workflows.
Resolves #437.
Required changes on GitHub for this PR:

Generate a PAT on the user that holds UEPseudo, either a "fine-grained token" or a "classic token" (more on that below), under Settings, Developer Settings, Personal access tokens.
Add the PAT to RE-UE4SS repository action secrets (under repo's Settings, Secrets and variables, Actions), with secret name being UEPSEUDO_PAT, and with the personal access token as the value.

Note: The SSH key is still used in other workflows other than experimental and release.
Personal access tokens can be either classic or fine-grained. Fine-grained tokens allows for read-only access to private repositories you specify that the user is a (direct) owner of, but has an expiration date of 1 year at most, and will need to be regenerated and replaced when it expires. Classic tokens can be specified to never expire, but has less control over access - to be able to just read private repositories, full repo access must be given to the token, i.e., read/write access to user repositories.
Configuration of the token:

Configuration for Fine-grained tokens

Everything is default unless specified
Name:  any descriptive name
Expiration:  to preference, at most 1 year
Repository access:  Only select repositories. Add UEPseudo into the selection list. RE-UE4SS does not need to be selected as long as it remains a public repository.
Permissions:  Under Repository permissions, set Contents to Access: Read-only.
Overview should now say "2 permissions for 1 of your repositories" (Contents and Metadata). Generate the token.

Configuration for Classic tokens

Everything is default unless specified
Note:  any descriptive name
Expiration:  to preference, indefinite expiration is allowed
Select scopes: repo checkbox must be checked. Will also unconditionally enable the options below it.
Generate the token.

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
All 7 test workflow runs have passed the Checkout step with no git 'early EOF' errors nor retries.
Complete workflow run 1 - using fine-grained PAT
Workflow run 2 - using fine-grained PAT - cancelled after the Checkout step succeeded to save resources.
Workflow run 3 - ^
Workflow run 4 - ^
Workflow run 5 - ^
Workflow run 6 - using classic PAT - also cancelled after the Checkout step succeeded.
Workflow run 7 - ^
Checklist

[x] Any dependent changes have been merged and published in downstream modules.

What's the status on this?

What's the status on this?

As far as I know, we're waiting on @narknon to configure the token before this is merged so that our CI doesn't break.

I've asked him to do it at least 3 times and he just ghosts me every time so I simply don't know what his problem is... Maybe he thinks it's to do with xmake which he doesn't like, but it's got nothing to do with xmake as this problem has been happening for many months.

How long have you been trying to get him to do it ? Right after this PR was created ?
Also, has he been particularly inactive during this time ?

Right after the PR was created, I've mentioned him in GitHub and in UE4SS server, both places he has been active in since. He's reviewed, commented on and merged other PRs around this one so there's no way he hasn't seen this.

Right after the PR was created, I've mentioned him in GitHub and in UE4SS server, both places he has been active in since. He's reviewed, commented on and merged other PRs around this one so there's no way he hasn't seen this.

That's very frustrating.
If he has a reason for not wanting this PR to be merged, he should tell us.
If he's just needlessly blocking this PR then what do we do ? That would be a very hostile thing to do, and I'm not sure what do to about it, but I certainly don't want to let this (or any) PR be blocked without reason.

Yeah that's what I mean I just don't get it. I can't possibly think of what reason there is to ignore it.

A semi-related question, why do we have uepseudo as a separate user, and not an organization? And another question would be why do we even need a separate user if we could host the fork in this repository?

A semi-related question, why do we have uepseudo as a separate user, and not an organization? And another question would be why do we even need a separate user if we could host the fork in this repository?

Could there be some permission problem ? The UEPseudo repo is a bit special because it's literally a fork of UE (so that everyone requires an Epic account) that's been cleaned out, perhaps it was easier to create the fork with a non-org account or something.
This is just a guess though, I really don't know at all.

From my testing, github allows us to fork unreal onto an organization account, so maybe we should switch that to this org / create a separate org just for the unreal fork, so several people can have permissions.

From my testing, github allows us to fork unreal onto an organization account, so maybe we should switch that to this org / create a separate org just for the unreal fork, so several people can have permissions.

Well if there's no problem with having it in this org, then if a move is happening it should be to this org in my opinion.
Whether a move should happen or not is another question.
How would developers (including mod developers) be affected by the repo changing ? Do they have to manually change the remote on their local repos or does git take care of this automatically ?
If people have to manually make a change, I foresee lots of problems in the future about outdated (and eventually non-compiling) UEPseudo code as people might not even realize (or otherwise won't know how) that they need to change the remote.

From my understanding, if we update the submodule in here, users will get the change automatically when doing git submodule update --init --recursive (which they should be doing each pull so that the unreal submodule gets fetched)

From my understanding, if we update the submodule in here, users will get the change automatically when doing git submodule update --init --recursive (which they should be doing each pull so that the unreal submodule gets fetched)

What about users that use the alternative way of updating by cd'ing into deps/first/Unreal and doing git pull, and also users that make forks and are constantly checking out different ones ? I guess we could assume that they know what they're doing and can solve any git problem on their own as long as we let them know about the change.
Users that don't have permission to make forks directly in UEPseudo will obviously have to re-fork and set up their stuff again as well I think.
Wouldn't it be easier to add multiple people to the UEPseudo repo with the appropriate access ?

Multiple people already have access to the UEPseudo repo.  I'm pretty sure I'm not the only one that can do this.  No, it cannot be moved into an organization account because that would violate UE's license.  As far as how active I've been, I haven't been active.  Talking in discord or merging a few simple PRs doesn't mean I'm following everything, but thanks for making a ton of assumptions about me in a public PR.

Multiple people already have access to the UEPseudo repo.

Could you check and let us know who, so that we know who to ask in the future ?
I know I don't have access.

You definitely have at least Write access.  I'm not sure if Admin or actual ownership is required for PATs.

Only the account that owns UEPseudo can make a PAT for UEPsuedo, since it's in the account settings rather than repository settings.


Update ubuntu2004.yml

User description
Description
New tool, Bug fixing, or Improvement?
Please include a summary of the change and which issue is fixed. Also include relevant motivation and context.
For new tools, please provide total size and installation time.

Related issue:
Check list

[ ] Related issue / work item is attached
[x] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[x] Changes are tested and related VM images are successfully generated

PR Type
enhancement, configuration changes

Description

Added a workflow_dispatch event to the Ubuntu 20.04 CI workflow configuration file, allowing the workflow to be manually triggered.
This enhancement provides more flexibility in testing and deploying changes.

Changes walkthrough 📝
Relevant filesConfiguration changes

ubuntu2004.ymlAdd manual trigger to Ubuntu 20.04 CI workflow

.github/workflows/ubuntu2004.yml
Added workflow_dispatch event trigger to the CI workflow.  This allows manual triggering of the workflow.

+1/-0

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

Summary by Sourcery
Enable manual triggering of the Ubuntu20.04 CI workflow by adding 'workflow_dispatch' to the event triggers.
CI:

Add manual trigger capability to the Ubuntu20.04 CI workflow by including 'workflow_dispatch' event.

PR Reviewer Guide 🔍
⏱ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No key issues to review

/review


[C#] Security issue in JSON dependency

It seems that C# integration tests have started failing due to a security issue having been reported for System.Text.Json
  /arrow/csharp/test/Apache.Arrow.IntegrationTest/Apache.Arrow.IntegrationTest.csproj : error NU1903: Warning As Error: Package 'System.Text.Json' 8.0.4 has a known high severity vulnerability, https://github.com/advisories/GHSA-8g4q-xg66-9fp4 [/arrow/csharp/Apache.Arrow.sln]

Component(s)
C#

Odd; it's been days since Dependabot has send a PR for this in the arrow-adbc repo. Maybe it's backed up for Arrow?
This is an easy enough change -- just update to 8.0.5 -- and if no one else does it I'll be able to do it later today.

@CurtHagenlocher is this required for 18.0.0?

@raulcd I take that back, the dependency in question is used only in test code so it probably doesn't need to be a blocker for 18.0.0.

Thanks, that makes sense. There's a couple more issues found that will make us create a new release candidate.
Let's leave it as 18.0.0 for now and if it's done before those I'll cherry pick it.

I should have looked more closely at this yesterday. The warning is not coming on the main branch (which was indeed fixed due to Dependabot) but probably from the release branch? In any event, the PR which addressed this in main was #44343.

Thanks @CurtHagenlocher. I've added the backport-candidate label to the PR. I'll cherry-pick manually if we end up doing a new RC. I close this as completed.

I have update the milestone on #44343 and this issue as this was indeed part of 18.0.0


[WIP] pythonPackages.elasticsearch[-dsl] 5.x and 6.x

Motivation for this change
This touches the elasticsearch and elasticsearch-dsl python modules in nixpkgs, bumping and splitting into 5.x and 6.x verisons, while providing aliases for backwards compatibility.
Different versions are required as the bindings are specific to the version of the elasticsearch they connect to…
Tests are currently disabled, as they try to connect to a local elasticsearch cluster.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-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"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

superseded by https://github.com/NixOS/nixpkgs/pull/44389 and https://github.com/NixOS/nixpkgs/pull/44339.


Problems building TF-Lite Android Benchmark tool

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Build machine is an Ubuntu VM with 10GB RAM, Oracle VirtualBox on Win10.
Ubuntu 18.04.3 LTS
kernel 5.3.0-26-generic

Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
Target device - Galaxy Tab S6, Snapdragon 855. Android 9. (and others)

TensorFlow installed from (source or binary): source

TensorFlow version: 2.0.1

Python version: 2.7

Installed using virtualenv? pip? conda?: no

Bazel version (if compiling from source): 1.2.1

GCC/Compiler version (if compiling from source): 7.4.0

CUDA/cuDNN version: none

GPU model and memory: none

Describe the problem
I have successfully converted Keras models to TF-Lite and run them in a simple Android app.
Now I want to optimise performance, I really need the Benchmark tool to see what operations are being sent to which chips and how long each takes, to see whether they way I've built my models suits the devices.
I can't get the benchmark tool to build. i can't get the camera demo to build either. i have tried various versions of the NDK & SDK and get a variety of errors that look like errors in the code but I'm sure it's just my build environment.
Provide the exact sequence of commands / steps that you executed before running into the problem
Build machine is an Ubuntu VM with 10GB RAM, Oracle VirtualBox on Win10.
Ubuntu 18.04.3 LTS
kernel 5.3.0-26-generic
Target device - Galaxy Tab S6, Snapdragon 855. Android 9.
Bazel version 1.2.1
gcc version 7.4.0
Recent install of Android Studio provides the SDK & NDK as below. (I did try NDK 14 but it didn't seem to help)
Cloned the TF repo. On master branch, commit d75abdf from 28th Jan 2020
Ran ./configure, options:
Please specify the location of python. [Default is /usr/bin/python]: - using default
('which python' shows /usr/bin/python, 'python --version' shows 2.7.17)
Found possible Python library paths:
/usr/local/lib/python2.7/dist-packages
/usr/lib/python2.7/dist-packages
Please input the desired Python library path to use.  Default is [/usr/local/lib/python2.7/dist-packages]  - using default
Do you wish to build TensorFlow with XLA JIT support? [Y/n]:  - using default
Do you wish to build TensorFlow with OpenCL SYCL support? [y/N]:  - using default
Do you wish to build TensorFlow with ROCm support? [y/N]:  - using default
Do you wish to build TensorFlow with CUDA support? [y/N]:  - using default
Do you wish to download a fresh release of clang? (Experimental) [y/N]:   - using default
Please specify optimization flags to use during compilation when bazel option "--config=opt" is specified [Default is -march=native -Wno-sign-compare]:   - using default
Would you like to interactively configure ./WORKSPACE for Android builds? [y/N]: y
Please specify the home path of the Android NDK to use. [Default is /home/medaphor/Android/Sdk/ndk-bundle]:
/home/medaphor/Android/Sdk/ndk/21.0.6113669
Please specify the (min) Android NDK API level to use. [Available levels: ['16', '17', '18', '19', '21', '22', '23', '24', '26', '27', '28', '29']] [Default is 21]:  - using default
Please specify the home path of the Android SDK to use. [Default is /home/medaphor/Android/Sdk]:  - using default
Please specify the Android SDK API level to use. [Available levels: ['24', '28', '29']] [Default is 29]: - using default
Please specify an Android build tools version to use. [Available versions: ['24.0.3', '29.0.2']] [Default is 29.0.2]:   - using default
Then following: https://github.com/tensorflow/tensorflow/tree/master/tensorflow/lite/tools/benchmark/android
(looking at the Master branch version of the doc)
1st step is:
(0) Refer to https://github.com/tensorflow/tensorflow/tree/master/tensorflow/examples/android to edit the WORKSPACE to configure the android NDK/SDK.
Well the configure script does that i believe. Doc is out of date? no action taken.
next step is:
(1) Build for your specific platform, e.g.:
bazel build -c opt
--config=android_arm64
--cxxopt='--std=c++11'
tensorflow/lite/tools/benchmark/android:benchmark_model
ran that, and get error:
ERROR: /home/medaphor/tensorflow/tensorflow/lite/experimental/delegates/hexagon/BUILD:66:1: C++ compilation of rule '//tensorflow/lite/experimental/delegates/hexagon:hexagon_delegate' failed (Exit 1)
tensorflow/lite/experimental/delegates/hexagon/hexagon_delegate.cc:52:32: error: no member named 'make_unique' in namespace 'std'
auto hexagon_kernel = std::make_unique();
~~~~~^
tensorflow/lite/experimental/delegates/hexagon/hexagon_delegate.cc:52:44: error: 'HexagonDelegateKernel' does not refer to a value
auto hexagon_kernel = std::make_unique();
^
./tensorflow/lite/experimental/delegates/hexagon/hexagon_delegate_kernel.h:40:7: note: declared here
class HexagonDelegateKernel {
^
tensorflow/lite/experimental/delegates/hexagon/hexagon_delegate.cc:52:67: error: expected expression
auto hexagon_kernel = std::make_unique();
^
UPDATE:
I missed this warning:
WARNING: The major revision of the Android NDK referenced by android_ndk_repository rule 'androidndk' is 21. The major revisions supported by Bazel are [10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20]. Bazel will attempt to treat the NDK as if it was r20. This may cause compilation and linkage problems. Please download a supported NDK version.
So installed NDK 20.0.5594570,
ran configure again,
did a bazel clean --expunge,
and tried to build. No more warnings but now get this error:
ERROR: /home/medaphor/tensorflow/tensorflow/lite/experimental/ruy/BUILD:246:1: C++ compilation of rule '//tensorflow/lite/experimental/ruy:detect_arm' failed (Exit 1)
tensorflow/lite/experimental/ruy/detect_arm.cc:59:10: error: use of undeclared identifier 'getauxval'
return getauxval(AT_HWCAP) & kLocalHwcapAsimddp;
^
UPDATE:
changing to:
--cxxopt='--std=c++14'
fixes it. Could you please update the readme's?

Documentation updated.


feat(pfInfoStatusCard): Adds Info Status Card and tests

Description
As per the issue, creates the requested component as to the spec.
Whatz it look like?

PR Checklist

[x] Unit tests are included
[x] Screenshots are attached (if there are visual changes in the UI)
[x] A Designer (@beanh66) is assigned as a reviewer (if there are visual changes in the UI)
[x] A CSS rep (@cshinn) is assigned as a reviewer (if there are visual changes in the UI)

closes #629
@cdcabrera @jeff-phillips-18 @dtaylor113  I can't add any labels to this, hoping ya can help out (enhancement, tagging those who need to review)
cc @serenamarie125 @chriskacerguis

@jeff-phillips-18 updated as per request 🙇♀

After having talked through on this with @jeff-phillips-18 ... looks like an agreement.
@AllenBW I know it's a little bit of a back track, but can you revert towards the "@" instead of the "<" (one-way) syntax. @jeff-phillips-18 brought up a good point for keeping the card components consistent especially if anyone needs to implement multiples.
However, for newer components in general "one-way" syntax for life (or until the framework alters again)!

no worries! happy to do 🤘

We need a quick visual review here as well - @jennyhaines can you take a look?

Thanks for tagging me, @serenamarie125
Hi @AllenBW - It looks like currently that .info-img is set to "margin-right: 10px". I would change this to 15px to give a bit more breathing room between the icon and content. The "No Top Border, Image Icon, No Title" example shows a small top margin between the content and the top of the card. I would increase it, matching the same height that you have between "Favorite Things" title and the top of the card on the "With HTML" example. I think that across the board, the spacing between individual lines of content could be reduced by 5px since they are a little bit too spaced apart in my opinion. The line spacing for wrapping content could be reduced slightly (maybe 2-4px less) as well.

@AllenBW I think there should also be a clear distinction between the labels and the values, so I suggest making the labels (for example, "VM Name:" or "Host Name:") semibold, which I think translates to "font-weight: 600"

@serenamarie125 oh oh! Did I request tagging of wrong individual for review? 😟
@jennyhaines Thanks for the input! will play with the pixels and post an update.
For simplicity, i'll call em example 1 2 and 3 (html being the third)  The margin you see in 1 but not in two is a result of 2 not having a title, there is no added margin.
As for the second comment this card doesn't presume to know anything about the desired display content for each examples we're just looking at strings, (there is no title vs contents), for examples 1 and 2, simple text fields (rather than html), it is not possible to to semi bold the contents.  In 3, a user can design any markup necessary.


missing const_tag2idx.json

Hi, very nice work. Thank you for sharing your code.
When I run the code, it reports an error that const_tag2idx.json is missing.
How can I get this file?
Thank you

Hi, thanks for your response. When can we have the updated data for training? or could you provide your pretrained model.
Thanks a lot

Preparing TRAINING dataset
Total number of clips 10191
Traceback (most recent call last):
File "/hy-tmp/VLCStoryGan/train_vlcgan.py", line 103, in
storydataset = data.StoryDataset(base, dir_path, video_transforms, cfg.MARTT.num_hidden_layers,
File "/hy-tmp/VLCStoryGan/vlcgan/pororo_data_const.py", line 213, in init
self.const_tag2id = json.load(open(textvec + 'const_tag2idx.json'))
FileNotFoundError: [Errno 2] No such file or directory: './data/data/const_tag2idx.json'
when i run with train_vlcgan.py , file of const_tag2idx.json is missing.

Is the file const_tag2idx.json uploaded?  I couldn't find it 😢

I found it from the Pororo dataset link attached on Readme. 😄


Logging In Feedback

Button should change to "Logging in..." and become disabled when clicked
Need feedback if login fails

there are some updates coming up to fix that on next PR.
we added a loader to the page as well.

Fixed here
https://github.com/Lambda-School-Labs/CS8-propertymgt/pull/56


A program to title case any sentence

The function titleCase() within the program takes a sentence as an argument and returns a new sentence in which the first letter of every word is uppercase/title case.
What does the program do?
It title cases any valid sentence.
In what programming language it is written?
Javascript.

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


[Request] Icon for MATLAB

Language: MATLAB
File ending: ".m"
Right now, it confuses ".m" files with ".c" file.

Hi,
please take a look at https://github.com/PKief/vscode-extension-material-icon-theme/issues/31.
For the files ending with ".m" the C-Icon is the default icon because it is widely used for C.
You should install the Matlab extension for your editor.
After that you can use the following config in your editor settings:
"files.associations": {
    "*.m": "matlab"
}

Or just use the command palette of vs code on the files for matlab:

I hope this will help you.

Problem solved. Thank you @PKief .


Uses browser even though linkedin app is installed

Hi,
I have the problem that the browser is used to login into linkedin instead of the app (the app is installed)
I would post more information but I don't know which one
Thanks,
Ben

you need to setup LSApplicationQueriesSchemes which is detailed in https://developer.linkedin.com/docs/ios-sdk (section: Whitelisting LinkedIn Custom Schemes)


Made channel URL explicit

What is the motivation for this change?  It has a very serious downside, which is that it prevents these examples from being used inside an airgap or firewall where the pyviz channel is mirrored locally but internet access is blocked.  Ideally most of these examples will soon be rewritten to depend only on defaults, which will avoid the issue, but omitting the URL was a deliberate policy that should only be changed for a reason that outweighs the problems it causes.

Read the discussion on slack.


update the link of Node.js example

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/skills/cortana-samples.md
:white_check_mark:Succeeded

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

We are revamping this page. This content is no longer shown.


tcmalloc core

#0  tcmalloc::CentralFreeList::FetchFromSpans (this=0x120f6a0) at src/central_freelist.cc:298
#1  0x00000000008ff7b8 in tcmalloc::CentralFreeList::RemoveRange (this=0x120f6a0, start=0x7f508125fa58, end=0x7f508125fa60, N=64) at src/central_freelist.cc:269
#2  0x000000000090721a in tcmalloc::ThreadCache::FetchFromCentralCache (this=0x31b7fe980, cl=Variable "cl" is not available.
) at src/static_vars.h:59
#3  0x00000000008fdd5e in (anonymous namespace)::do_malloc_no_errno (size=Variable "size" is not available.
) at src/thread_cache.h:363
#4  0x00000000009b19df in tc_new (size=256) at src/tcmalloc.cc:1423
#5  0x00000000004c0348 in std::vector<std::pair<int, double>, std::allocator<std::pair<int, double> > >::_M_insert_aux (this=0x7f5081260100, __position={_M_current = 0x4a4671f00},
__x=@0x7f50812600e0) at /usr/lib/gcc/x86_64-redhat-linux/3.4.5/../../../../include/c++/3.4.5/ext/new_allocator.h:81
#6  0x00000000004ea649 in MaxentModelPredict::run (this=0x2c76e5b60, rq_terms=@0x7f5081260500, wp_terms=@0x7f50812604d0, result=0x7f50812604b0)
at /usr/lib/gcc/x86_64-redhat-linux/3.4.5/../../../../include/c++/3.4.5/bits/stl_vector.h:330
#7  0x00000000004bf274 in QrwQueryCategoryProcessor::process (this=0x3048671e0, input=0x4903abc30, output=0x2e95c7550) at src/processor/qrw_query_category_processor.cc:381
#8  0x00000000004a3451 in qrw_query_understand_processor_t::process (this=0x2cff01b40, input=0x4903abc30, output=0x2e95c7550) at src/processor/qrw_query_understand_processor.cpp:412
#9  0x000000000048fe22 in qrw_as_agent_processor_t::process (this=0x4903abcc0, input=0x2ddd4de40, output=0x2e95c7400, term_tag=@0x7f5081260cc0)
at src/processor/qrw_as_agent_processor.cpp:403
#10 0x0000000000469b30 in qrw::process_as_agent (thd_data=@0x7f5081260d80, pb_as_agent_input=@0x49f5d86f0, pb_as_agent_output=@0x4a0469330, is_gbk=false)
at src/svr/qrw_server_callback.cpp:1162
#11 0x0000000000470e39 in qrw::server_callback_core (thd_data=Variable "thd_data" is not available.
) at ../../../../../app/search/tuangou/se/qrw-interface/output/include/qrw_interface.pb.h:7166
#12 0x0000000000471159 in qrw::server_callback () at src/svr/qrw_server_callback.cpp:1722
#13 0x0000000000737c11 in default_native () at ubserver_app.cpp:283
#14 0x0000000000732d66 in eppool_consume (pool=0x3137707c0, data=0x1d13b80) at eppool.cpp:663
#15 0x0000000000732bc1 in _eppool_workers (param=0x1d13b80) at eppool.cpp:618
#16 0x0000003f0b90610a in start_thread () from /lib64/tls/libpthread.so.0
#17 0x0000003f0b0c5ee3 in clone () from /lib64/tls/libc.so.6
#18 0x0000000000000000 in ?? ()

Hi. Issues like that are usually due to problems in the app. Have you tried running your app under address sanitizer or similar tool?

hi alk, problems like double delete a pointer? @alk

(gdb) bt
#0  (anonymous namespace)::do_malloc_no_errno (size=Variable "size" is not available.
) at src/linked_list.h:43
#1  0x00000000009b22ef in tc_new (size=256) at src/tcmalloc.cc:1423
#2  0x0000000000481035 in __gnu_cxx::new_allocatorstd::string::allocate (this=0x7fdc0220e3b0, __n=32)
at /usr/lib/gcc/x86_64-redhat-linux/3.4.5/../../../../include/c++/3.4.5/ext/new_allocator.h:81
#3  0x000000000047fd08 in std::_Vector_base<std::string, std::allocatorstd::string >::_M_allocate (this=0x7fdc0220e3b0, __n=32)
at /usr/lib/gcc/x86_64-redhat-linux/3.4.5/../../../../include/c++/3.4.5/bits/stl_vector.h:113
#4  0x000000000047e308 in std::vector<std::string, std::allocatorstd::string >::_M_insert_aux (this=0x7fdc0220e3b0, __position={_M_current = 0x3d8930300}, __x=@0x3d9295b40)
at /usr/lib/gcc/x86_64-redhat-linux/3.4.5/../../../../include/c++/3.4.5/bits/vector.tcc:244
#5  0x000000000047d3c6 in std::vector<std::string, std::allocatorstd::string >::push_back (this=0x7fdc0220e3b0, __x=@0x3d9295b40)
at /usr/lib/gcc/x86_64-redhat-linux/3.4.5/../../../../include/c++/3.4.5/bits/stl_vector.h:564
#6  0x00000000004c386d in CategoryQueryRecognizer::process (this=0x1ab09e0, query_str=@0x22ac598c0, terms=@0x22ac598e8, output=0x20b899750)
at src/processor/category_query_recognizer.cc:125
#7  0x00000000004c6d24 in QrwQueryCategoryProcessor::process (this=0x21c89eca0, input=0x22ac598c0, output=0x20b899750) at src/processor/qrw_query_category_processor.cc:180
#8  0x00000000004ad7fe in qrw_query_understand_processor_t::process (this=0x20d010320, input=0x22ac598c0, output=0x20b899750) at src/processor/qrw_query_understand_processor.cpp:570
#9  0x0000000000498e06 in qrw_as_agent_processor_t::process (this=0x2c7d493c0, input=0x220e14660, output=0x20b899600, term_tag=@0x7fdc0220ec90)
at src/processor/qrw_as_agent_processor.cpp:403
#10 0x000000000046e2cc in qrw::process_as_agent (thd_data=@0x7fdc0220ed88, pb_as_agent_input=@0x39ce91a40, pb_as_agent_output=@0x21777d2b0, is_gbk=true)
at src/svr/qrw_server_callback.cpp:1219
#11 0x0000000000470f8e in qrw::server_callback_core (thd_data=0x237cf7900, request=@0x7fdc0220ee80, response=@0x7fdc0220ee00) at src/svr/qrw_server_callback.cpp:1736
#12 0x00000000004712fe in qrw::server_callback () at src/svr/qrw_server_callback.cpp:1869
#13 0x000000000073a2a5 in default_native () at ubserver_app.cpp:283
#14 0x00000000007353fa in eppool_consume (pool=0x237cf79c0, data=0x1d4a580) at eppool.cpp:663
#15 0x0000000000735255 in _eppool_workers (param=0x1d4a580) at eppool.cpp:618
#16 0x0000003f0b90610a in start_thread () from /lib64/tls/libpthread.so.0
#17 0x0000003f0b0c5ee3 in clone () from /lib64/tls/libc.so.6
#18 0x0000000000000000 in ?? ()
core at  push_back

Yes, this could be double deletion or app continuing to mutate freed object. Again, address sanitizer and similar tool are known to be effecting in catching that kind of problems.


[SR-8262] Swift Package Manager fails at checkout when dependent repository contains Git LFS objects

Previous ID
SR-8262

Radar
rdar://problem/42246102

Original Reporter
yifei (JIRA User)

Type
Bug

Additional Detail from JIRA

Votes
2

Component/s
Package Manager

Labels
Bug

Assignee
None

Priority
Medium

md5: 2218c2c1d09b919e983b204e24d25058

Issue Description:
The error is in the format of: (XXXXX being the name of LFS-tracked file)
Error downloading object: XXXXX (hash): Smudge error: Error downloading XXXXX (hash): batch request: missing protocol: "/path/to/project/.build/repositories/dependency.git/info/lfs"

error: external filter 'git-lfs filter-process' failed
fatal: XXXXX: smudge filter lfs failed
warning: Clone succeeded, but checkout failed.
You can inspect what was checked out with 'git status'
and retry the checkout with 'git checkout -f HEAD'

This happens during the "cloneCheckout" phase i.e. invoking "git clone --shared". If I disable the LFS filter by editing the global .gitconfig, the project builds fine.

Has there been any update on this issue? There's a similar discussion at https://forums.swift.org/t/swiftpm-with-git-lfs/42396.
I am running into this issue as well with https://github.com/foxglove/mcap. This dependency fails during checkout: .package(url: "https://github.com/foxglove/mcap", branch: "main"),
Creating working copy for https://github.com/foxglove/mcap
error: Couldn’t check out revision ‘2dad37cad3c7875842b44f3fe927e2dcfdecf7d4’:
    Downloading docs/img/0398582-hero.png (669 KB)
    Error downloading object: docs/img/0398582-hero.png (266f9d3): Smudge error: Error downloading docs/img/0398582-hero.png (266f9d3dd91c1bd739e1d79ae4abd459f71e8d4c661656745504ea52913467b5): error transferring "266f9d3dd91c1bd739e1d79ae4abd459f71e8d4c661656745504ea52913467b5": [0] remote missing object 266f9d3dd91c1bd739e1d79ae4abd459f71e8d4c661656745504ea52913467b5
    Errors logged to /Users/jacob/Desktop/testpackage/.build/checkouts/mcap/.git/lfs/logs/20220923T112418.927225.log
    Use `git lfs logs last` to view the log.
    error: external filter 'git-lfs filter-process' failed
    fatal: docs/img/0398582-hero.png: smudge filter lfs failed

A simple git clone succeeds.

We're talking about an Xcode specific issue here, correct? I tried reproducing with https://github.com/foxglove/mcap and swift build and it seems to work fine for me.

Nope, it's not an Xcode specific issue. swift build within the repository works fine when you manually clone it. The problem is with swiftpm's automatic cloning.
Try a new project with this Package.swift:
// swift-tools-version:5.5
import PackageDescription

let package = Package(
  name: "demo",
  products: [],
  dependencies: [
    .package(url: "https://github.com/foxglove/mcap", branch: "main"),
  ],
  targets: []
)

Then run swift build:
$ swift build
Fetching https://github.com/foxglove/mcap from cache
Fetched https://github.com/foxglove/mcap (2.03s)
Fetching https://github.com/apple/swift-collections from cache
Fetched https://github.com/apple/swift-collections (0.76s)
Fetching https://github.com/apple/swift-algorithms from cache
Fetched https://github.com/apple/swift-algorithms (0.33s)
Computing version for https://github.com/apple/swift-algorithms
Computed https://github.com/apple/swift-algorithms at 1.0.0 (0.32s)
Fetching https://github.com/apple/swift-numerics from cache
Fetched https://github.com/apple/swift-numerics (0.60s)
Computing version for https://github.com/apple/swift-numerics
Computed https://github.com/apple/swift-numerics at 1.0.2 (0.33s)
Creating working copy for https://github.com/apple/swift-algorithms
Working copy of https://github.com/apple/swift-algorithms resolved at 1.0.0
Creating working copy for https://github.com/apple/swift-collections
Working copy of https://github.com/apple/swift-collections resolved at 418378107c87a4b312e29a51f773ce0e4e12e199
Creating working copy for https://github.com/apple/swift-numerics
Working copy of https://github.com/apple/swift-numerics resolved at 1.0.2
Creating working copy for https://github.com/foxglove/mcap
error: Couldn’t check out revision ‘14fa18afb83e504ae03397fd4e9292a6c83a7e80’:
    Downloading docs/img/0398582-hero.png (669 KB)
    Error downloading object: docs/img/0398582-hero.png (266f9d3): Smudge error: Error downloading docs/img/0398582-hero.png (266f9d3dd91c1bd739e1d79ae4abd459f71e8d4c661656745504ea52913467b5): error transferring "266f9d3dd91c1bd739e1d79ae4abd459f71e8d4c661656745504ea52913467b5": [0] remote missing object 266f9d3dd91c1bd739e1d79ae4abd459f71e8d4c661656745504ea52913467b5
    Errors logged to /Users/jacob/Desktop/swifttest/.build/checkouts/mcap/.git/lfs/logs/20221006T101212.79192.log
    Use `git lfs logs last` to view the log.
    error: external filter 'git-lfs filter-process' failed
    fatal: docs/img/0398582-hero.png: smudge filter lfs failed

Interesting, this does not happen for me. I ended up with the same commit as you, but nothing in .git/lfs:
❯ git show --oneline -s
14fa18afb (HEAD, origin/main, origin/HEAD, main) fix link in README (#636)

/tmp/foo/.build/checkouts/mcap heads/main
❯ ls -ld .git/lfs
".git/lfs": No such file or directory (os error 2)

I'm not really familiar with git-lfs, is there any global config required for it?

I'm not really familiar with git-lfs, is there any global config required for it?

Yes, normally users will run git lfs install once, as described here: https://git-lfs.github.com
It's a very common thing to have installed, since many projects use Git LFS.
What do you see when you look at /tmp/foo/.build/checkouts/mcap/docs/img/0398582-hero.png? I'm guessing you have only the LFS pointer file, which is what you'd get if you don't have LFS installed.

Also note that some folks on Discourse have dug deeper into why this happens: https://forums.swift.org/t/swiftpm-with-git-lfs/42396/13

Thanks, this helps!

Thanks for looking into this! 🙌 Just wondering if there’s been any update so far on a possible resolution?

I tried adding a .lfsconfig file to the repo, and this seems to have worked around the issue for swift build (like the small example I gave you), but it still doesn't work when adding the dependency in Xcode...
Example project that fails to build in Xcode on my machine: TestPackage.zip

Based on this thread and the forum discussion, I came to the same blocking errors.
The only work around I have is to manually clone the repo and include it as a local package—which of course negates all advantages of using a dependency manager. 😫

Not sure if it helps you, but I believe the .lfsconfig file has to be present on the repo's default branch. When SPM clones the package it starts with the default branch and then checks out whichever branch/commit you requested.

Is there any update to this?

+1

+1

One option for this I wrote up here (which only applies if the LFS files are something optional for your package, like test results): https://stackoverflow.com/questions/67457594/swift-package-manager-does-not-work-with-git-lfs/77336212#77336212

The workaround from @briancordanyoung seem to work, but the Xcode path has changed
cd /Applications/Xcode.app/Contents/Developer/usr/bin
ln -s /opt/homebrew/bin/git-lfs git-lfs

Seems the workaround conflict with SIP, it's weird to include XCode folder in SIP.

Are you sure it is SIP @yangkx1024? Wonder why the symlinking was allowed for me, I have not disabled SIP
csrutil status

System Integrity Protection status: enabled.

I disabled the SIP and the ln command success, but now I stuck with the git-lfs issue, same as this post: https://forums.swift.org/t/swiftpm-with-git-lfs/42396/13

@neonichu if you learned anything about the root cause of this issue or possible workaround, I’m sure many of us would be interested to hear it, thanks :)

I am using the following workaround:

Switch to using your system Git.

When you encounter error that says "Package Resolution Failed", dismiss with "Add Anyway". This will add a partially configured package dependency.

If Xcode Command Line Tools is never installed on your machine, install it first following instructions from Apple Developer website.

Run the following command from Terminal, from the directory where your .xcodeproj is located, as appropriate for your project. It will resolve the package through your system Git. Your system Git needs to already have Git LFS configured.
xcodebuild -scmProvider system -resolvePackageDependencies

Note: You can also add -project, -workspace, and -scheme arguments as appropriate to target your development setup. Refer to the xcodebuild (Command Line Tools) FAQ and -help section.

@iosdevben  I would love to have this fixed. But I would not let it prevent you from adopting SPM. I've been using this work around all this time and the benefits of SPM have made it completely worth it.


Optimize pooled string array joins

This PR optimizes the PoolStringArray.join() function by applying the logic used by the native StringBuilder class. The new implementation is about 9x faster in general and 4x faster for empty delimitiers.
Test A: A single test joins 1000 random strings of 32 characters (GUIDs) with a comma.
Test B: A single test joins 1000 random strings of 32 characters (GUIDs) with an empty string.
Each test ran 100 times and these are the average duration of a single run. The time contains the join() call, only.
Test machine: Windows 10 Pro 1903, i7 3.3 GHz, 16 GB, SSD
Custom Build: VC++ 2019, release_debug, x64

Build
Test A
Test B

Test A
Test B

Exported offical 3.1
347 ms
132 ms

Editor offical 3.1
368 ms
136 ms

Editor w/ old impl
423 ms
171 ms

Editor w/ new impl
47 ms
43 ms

Editor w/ old StringBuilder
1013 ms
563 ms

Editor w/ new StringBuilder
174 ms
107 ms

All join-results were tested against a static string to be correct.
The test source can be found here: https://gist.github.com/fkollmann/ee72937393fcff74fcd252cdaf2e2ce6

Hm, great. I changed the base branch and GitHub added reviewers for all delta files... Sry for that.

Moving to 4.0 milestone as this is not ready to merge yet, and we are now in release freeze for 3.2.
There are still some still issues to address, and commits should be squashed together.
CC @lawnjelly who seems to like testing optimizations :)

I won't be able to profile as I can't work on godot for the next week (and this will probably need profiling from within godot, rather than a testbed, because of the dependencies on string etc), but I've had a quick look through the code, and I can't believe I'm saying this, but I actually agree with most of the author's decisions. :+1:
The original type of method the author suggested would be the fastest, because although String etc are great for convenience, all the house keeping etc won't beat some optimized code. But I understand the desire to use string builder. The main impetus is to keep the number of allocations down, and I can tell that's the drive behind both designs, with a bulk allocation of chunks, and providing the COW thing works with the strings (I don't know the string implementation well) then that should hopefully stop all the small allocations the previous string builder suffered from.
The previous string builder also suffered from using godot Vector, which does far too many allocations, and hopefully someone will get around to working on the more traditional grow allocation vector alternative that was discussed on IRC a few weeks ago.
I also noted that if you use C style strings with the string builder, you need to keep them around until you get the result as it doesn't copy them (which is sensible, as that would kill performance). That kind of thing that should be written in a comment describing the class. In general personally I find there is far too little comments in godot, but  I don't know the general opinion on that.

@fkollmann Is this still desired? If so, it needs to be rebased on the latest master branch. Since this PR was last updated, PoolArrays have been removed and replaced with PackedArrays. Therefore, this PR may need to be redone from scratch, or may no longer be relevant.
If not, abandoned pull requests will be closed in the future as announced here.

@fkollmann While performance improvements are very welcome, this PR has been inactive for a long time and needs significant work, so I'm closing this. If this is still desired, you are welcome to open another PR.


imagenet can't work as readme says, see attached log

test.log with latest code on v6.0 Linux36.3
All kinds of prints, and I didn't think this prints are OK. Anyone knows why?

Hi @lida2003 , unfortunately your log got truncated - can you try posting the full one, along with the command-line you used to run it and with any errors you encountered?

Yes, really need to wait for a while. sometimes 40mins, even longer.


JavaScript -> .repeaterVal() doesn't return data from fields

Hi, tried to get the data form all fields in javascript using $(".repeater").repeaterVal(); and I found out that the  doesn't get returned by this.
Can you please check?
Example:
<form class="repeater">
    <div data-repeater-list="group-a">
      <div data-repeater-item>
        <input type="text" name="text-input" value="A"/>
        <input type="number" name="number-input" value="1"/>
        <input data-repeater-delete type="button" value="Delete"/>
      </div>
    </div>
    <input data-repeater-create type="button" value="Add"/>
    <input type="button" value="Check data" onclick="checkData()" />
</form>

<script src="path/to/jquery.js"></script>
<script src="path/to/jquery.repeater/jquery.repeater.js"></script>
<script>
    $(document).ready(function () {
        $('.repeater').repeater();

        function checkData(){
          let data = $(".repeater").repeaterVal();
          console.log(data);
        }
    });
</script>

Hi, I am facing the same problem. Did you find out a solution for this? Thanks!

@ahazeemi check this PR with add support for input type="number" https://github.com/DubFriend/jquery.repeater/pull/94

Hey budies you can used this
$('#').val(window.id);
$(this).slideUp(deleteElement);

console.log($('.repeater').repeaterVal());

Hi, $('.repeater-medicationtime').repeaterVal() by using this I want to get the values from fields but it didnot get values from fields of type time and date

If anyone is facing same issue, download whole repeater files, in that you should find jquery.input.js file, replace it with this one:  https://pastebin.com/ezrQ412T
It supports date and number both, I created date code from this trick: https://github.com/DubFriend/jquery.repeater/pull/94/commits/8c709d074ed5f0ad4463a51fec51e785f6dbd299

Any anwser don't work for me.
`

Repeater
jquery.repeater

Repeater

  <input type="button" id="in" value="Check data" />

`

If anyone is facing same issue, download whole repeater files, in that you should find jquery.input.js file, replace it with this one: https://pastebin.com/ezrQ412T
It supports date and number both, I created date code from this trick: 8c709d0
Make sure to include lib.js file as well. download whole repeater files and you should see this file in src folder.

Don't work for me; i can't display nor date type nor number type.
Please help me


Update api and build process

For https://github.com/CityOfZion/neon-wallet/issues/33
Updated API to be on par with neon-wallet.
Updated build process to package into node library.
Updated tests to test from packaged index.js.
Updated README.
Still can be polished further but good stop point to import into neon-wallet.

If there are any updates to neon-wallet/src/wallet, I will need to update this branch before merge

Looks great! Can we keep the quick start in the readme though?

Awesome work!


A simple tutorial to get to know Solito/ can't add or edit a new screen.

Hi all,
First of all, I really appreciate the effort the founder of Solito and the community is doing for creating a future-proof React Native + Next JS stack.
The problem I'm having is I don't know how to add a new screen or edit an existing screen correctly. What exactly solito/link component look for in the file name, URL, or in the navigator to create React Navigation's version of that route? Working with Next JS is easy, as the routes are defined on the Next JS project.
I tried to change the name of user/detail-screen.tsx to user/screen.tsx, like home/screen.tsx in the mono repo, but Solito still looks for user-detail. I looked for URL and React Navigation's setup, but there is no user-detail anywhere. I don't know where Solito pulls out the name. The folder structure is also a bit confusing to me as I don't know what is allowed and what isn't.
I tried to look out for documentation and it doesn't seem to have a section like "Create a new Route" or a simple tutorial to make something (a simple to-do list for example) out of Solito.
Here's the code:

packages/app/navigation/native:

import { createNativeStackNavigator } from '@react-navigation/native-stack'

import { HomeScreen } from '../../features/home/screen'
import { UserDetailScreen } from '../../features/user/screen'

const Stack = createNativeStackNavigator<{
  home: undefined
  user: {
    id: string
  }
}>()

export function NativeNavigation() {
  return (
    <Stack.Navigator>
      <Stack.Screen
        name="home"
        component={HomeScreen}
        options={{
          title: 'Home',
        }}
      />
      <Stack.Screen
        name="user"
        component={UserDetailScreen}
        options={{
          title: 'User',
        }}
      />
    </Stack.Navigator>
  )
}

packages/app/features/user/screen.tsx

import { View, Text } from 'dripsy'
import { createParam } from 'solito'
import { TextLink } from 'solito/link'

const { useParam } = createParam<{ id: string }>()

export function UserDetailScreen() {
  const [id] = useParam('id')

  return (
    <View sx={{ flex: 1, justifyContent: 'center', alignItems: 'center' }}>
      <Text
        sx={{ textAlign: 'center', mb: 16, fontWeight: 'bold' }}
      >{`User ID: ${id}`}</Text>

      <TextLink href="/">👈 Go Home</TextLink>
    </View>
  )
}

packages/app/featues/home/screen.tsx

import { Text, useSx, View, H1, P, Row, A } from 'dripsy'
import { TextLink } from 'solito/link'
import { MotiLink } from 'solito/moti'

export function HomeScreen() {
  const sx = useSx()

  return (
    <View
      sx={{ flex: 1, justifyContent: 'center', alignItems: 'center', p: 16 }}
    >

Welcome to Solito.

      <View sx={{ maxWidth: 600 }}>

          Here is a basic starter to show you how you can navigate from one
          screen to another. This screen uses the same code on Next.js and React
          Native.

          Solito is made by{' '}

            Fernando Rojo

          .

      </View>
      <View sx={{ height: 32 }} />
      <Row>
        <TextLink
          href="/user/fernando"
          textProps={{
            style: sx({ fontSize: 16, fontWeight: 'bold', color: 'blue' }),
          }}
        >
          Regular Link
        </TextLink>
        <View sx={{ width: 32 }} />
        <MotiLink
          href="/user/fernando"
          animate={({ hovered, pressed }) => {
            'worklet'

            return {
              scale: pressed ? 0.95 : hovered ? 1.1 : 1,
              rotateZ: pressed ? '0deg' : hovered ? '-3deg' : '0deg',
            }
          }}
          transition={{
            type: 'timing',
            duration: 150,
          }}
        >
          <Text
            selectable={false}
            sx={{ fontSize: 16, color: 'black', fontWeight: 'bold' }}
          >
            Moti Link
          </Text>
        </MotiLink>
      </Row>
    </View>
  )
}

I'd really appreciate it if there is some resource for beginners to make a simple project using Solito, the only resource I found is getting started with Solito and NativeBase (https://docs.nativebase.io/solito) but it is also an example mono repo and not exactly a tutorial.
Thanks again for helping me out!

I'd really appreciate it if there is a tutorial that goes through some of the main concepts like nested react-navigation routing (Stack -> Tabs), and how that would change into Next JS. In short some of the things which show the potential Solito has and where it shines

all fair points! video tutorials are a lot of work but i’d love if someone could make one. once you make a screen once, you’ll find it to be really easy. i’ll do my best to do this if i have time at some point

Agreed! It would be better create a new discussion for creating tutorials or move this to discussions as I've figured out to add new screens.

@theJsTsGuy DId you already figured it out? I think you have to change user-detail in packages/app/provider/navigation/index.tsx


Update the board information to reflect the new board member roles.

We should update the board page to reflect the new roles taken on by board members.
Could also use this opportunity to add a little more information about the board.

Done in #78 thanks @JoshMacSween


add nominees from Africa COVID-19 Response Toolkit

#33
Added 10 Nominees from the Africa COVID-19 Response Toolkit

@ijeomaemeruwa thanks for your batch submission of nominees. It appears the CI failed for all the files because of "no newline character at the end the files". Kindly fix this on the same branch then we can proceed to review the updated versions. Thanks.

Okay I will, thank you!
On Thu, 1 Apr 2021 at 15:42, Nathan Baleeta @.***>
wrote:

@ijeomaemeruwa https://github.com/ijeomaemeruwa thanks for your batch
submission of nominees. It appears the CI failed for all the files because
of "no newline character at the end the files". Kindly fix this on the same
branch then we can proceed to review the updated versions. Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/unicef/publicgoods-candidates/pull/386#issuecomment-811956128,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOEPWERFEFL63T3PNYHH4ZLTGSA5HANCNFSM42GU6AJQ
.

@ijeomaemeruwa, thanks for your contribution. Please note that an earlier contributor had already nominated 8 of the 10 projects listed there, and thus those 8 shall not be included in this PR. Kindly remove them, and submit the remaining 2. Thanks 🙏

Okay
On Thu, 1 Apr 2021 at 22:12, Victor @.***> wrote:

@ijeomaemeruwa https://github.com/ijeomaemeruwa, thanks for your
contribution. Please note that an earlier contributor had already nominated
8 of the 10 projects listed there, and thus those 8 shall not be included
in this PR. Kindly remove them, and submit the remaining 2. Thanks 🙏
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/unicef/publicgoods-candidates/pull/386#issuecomment-812175526,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOEPWEW6GQPSSJ6Y2RDPAPLTGTOU7ANCNFSM42GU6AJQ
.