Change units doesn't query for the new units

When I select change units, I'm not being prompted to enter the new units.

I don't think it is implemented yet. See #14.

Implemented in 37956fef9abb3d76d60ccc90217bcc687fa992e0.


V9/canvas groups

This should only be merged after the config PR has been merged.

I just realized, I still need to add a properties interface to this, so that people who add additional layers can access them via the layer groups.

I just realized, I still need to add a properties interface to this, so that people who add additional layers can access them via the layer groups.

This will be done in a separate issue/PR.


parser helper with multiple fallbacks

related to : https://github.com/uselagoon/lagoon-ui/issues/120
parseGitUrl builds on top of git-url-parse and  if giturlparse fails:

tries to extract certain fragments of the url if it has a similar structure as a git url.
returns the raw string if the above failed.

@mattswann I'd like your input on this

@tobybellwood @rocketeerbkw I totally didn't forget about this PR 😅
as mentioned in the review and in recent discussion, there's no need to guess a url if the parser fails, instead we hide the source section in the sidebar and show giturl on the environment page


Comment folding broken - "Hide Comments" links/button not showing up

Just recently comment folding has broken for me, and the usual "Hide comments" (don't remember exact wording) are not showing up. I'm using Chrome Version 41.0.2272.118 (64-bit).
Screenshot: http://goo.gl/MecJb.

Same here. Using Chrome on OSX Version 41.0.2272.118 (64-bit)

It seems the issue is present with
// Skip this row if we're on a comment permalink page and it's the comment at the top
if (row.nextSibling && row.nextSibling.getElementsByClassName("yclinks").length) return;

row.nextSibling returns text element. This should be replaced by row.nextElementSibling which correctly returns the <tr> element
Example:
// Skip this row if we're on a comment permalink page and it's the comment at the top
if (row.nextElementSibling && row.nextElementSibling.getElementsByClassName("yclinks").length) return;

Thanks for the fix. Unless I'm imagining how it used to work, folding would also fold up all the childred/reply comments. It's only folding the one comment after this fix.


WSL Ubuntu stop work

Since version 0.12.2 (the first I noticed the error) when I try to start in Visual Code Insiders with Remote Ubuntu (WSL) the build tasks successfully, but the server give the following error:

version 0.12.3  , mac platform  don't Goto Definition

I have installed WSL and ubuntu and it works now, maybe.

It went back to normal, thanks!


Include query source in activerecord logs

This is done by default since rails 5.2 in the main rails logsubscriber. Implementation is stolen from
current implementation of rails default logsubscriber:
https://github.com/rails/rails/blob/c675783eb4fb42a357ec068d4cfd83bf08f78aea/activerecord/lib/active_record/log_subscriber.rb
Issue # (if available)
Description of changes
ActiveRecord query log payload now contains query_source - reference to the line where query was made from.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Bump, @reidmorrison

Rails Semantic Logger already logs the source filename and line number of the query and does not require Rails 5.2 to do it.
Look for config.semantic_logger.backtrace_level = :info in the docs: https://rocketjob.github.io/semantic_logger/rails.html
What we could do is additionally check the ::ActiveRecord::Base.verbose_query_logs flag when the SemanticLogger.backtrace_level_index is not met. Something like:
        # Log the location of the query itself.
        if (logger.send(:level_index) >= SemanticLogger.backtrace_level_index) || log_query_source?
          log[:backtrace] = SemanticLogger::Utils.strip_backtrace
        end

No activity for 1 year


Non-Babel project adds babel files in new project

Version
3.11.0
Environment info
System:
    OS: Windows 10
    CPU: (8) x64 Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz
  Binaries:
    Node: 10.16.3 - C:\Program Files\nodejs\node.EXE
    Yarn: Not Found
    npm: 6.11.3 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: 44.18362.329.0
  npmGlobalPackages:
    @vue/cli: Not Found

Steps to reproduce

vue create test-app
select manual presets
select presets: PWA, Router, Vuex, Linter, Unit Testing
History mode: Yes
select: ESLint   Prettier
select: Lint On Save
select: Jest
select: In dedicated config files

What is expected?
The resulting new project should contain only what's used by the selected presets and mandatory Vue files, nothing extra.
Upon viewing the new project's files, I know what everything is and everything is there because I specifically asked Vue to put it there.
What is actually happening?
Extra files which seem to be files used by packages or libraries not installed, appear in the new project's directory.
Although I'm mostly confident I don't need them, I'm not 100% certain of hidden consequences for deleting them.
i.e.

what is .babelrc? babel bla bla, don't need it - delete.
what is .browserlistrc? don't know, don't need it - delete.
what is postcss.config.js? don't know, don't need it - delete.
why does my package.json have babel-eslint devDependency? don't know, wasted space, don't need it - npm un babel-eslint

This is not that important relative to the rest of the work the Vue team is doing, but it does mean every new project (according to the reproduction steps above) needs to have these files deleted and the babel-eslint package uninstalled if the goal is to keep the project minimal.

what is .babelrc?

Are you sure? We would generate babel.config.js maybe, but not babelrc?

what is postcss.config.js?

We always add postcss for applying autoprefixer. It's documented and it's not an option you can un-select right now. If you think it should be, open a featue request.

what is .browserlistrc?

autoprefixer needs this file to know what your targets browsers are.

why does my package.json have babel-eslint devDependency?

That can indeed be fixed in the eslint plugin.

Are you sure? We would generate babel.config.js maybe, but not babelrc?
Positive. Below is the contents of the generated babelrc:
{   "plugins": [     "transform-es2015-modules-commonjs"   ] }

We always add postcss for applying autoprefixer.
Since Vue CLI uses this internally, it's fine. Being that it is a default that cannot be turned off, it would be nice if that it was tucked away from my project files, but again it's not a big deal.

If the babel-eslint package was fixed in v4 that's enough for me, I'll just keep deleting it until I can upgrade to v4.

Positive. Below is the contents of the generated babelrc:
{ "plugins": [ "transform-es2015-modules-commonjs" ] }

Working on that. Maybe we can hide the details in a jest preset. For now, it's required for jest & babel-jest to properly process es module syntax.

Since Vue CLI uses this internally, it's fine. Being that it is a default that cannot be turned off, it would be nice if it was tucked away from my project files, but again it's not a big deal.

When scaffolding the project, the last prompt is "Where do you prefer placing config for Babel, PostCSS, ESLint, etc.?" If "In package.json" is chosen, browserslist & postcss config will be placed in package.json.

Parsing error: Unexpected token import

The dynamic import syntax wasn't supported in ESLint until very recently with the release of espree v6.1.0 last month. Before that, babel-eslint is necessary for any project that uses import(), so we added it by default in case users are confused by this issue.
We still need time to fully update to ESLint 6 for all our config options. And contributions are welcome.

@sodatea thanks for the clarification on those requirements, it seems like all those "extra" files are in fact not extra after all... yet.

If "In package.json" is chosen, browserslist & postcss config will be placed in package.json.

True, but I should have specified that by "tucked away from my project files" I meant to not exist at all on the surface, if it's a default and must exist somewhere it'd be nice if it was on a lower level like in a Vue CLI preset or config somewhere in its node_modules folder. I'm not sure what the pros/cons of that would be though.

This issue has been fixed as of v4.2 😄
babel-jest is hidden behind a Jest preset, and babel-eslint is no longer mandatory with the new ecmaVersion: 2020 parserOption of ESLint.


Declarative: Fix Cycle detection issues with partial graph knowledge.

Maintaining the 2 pass algorithm, except that integrating with the CallStack information. This method is more robust for scenarios where we don't get to know the full graph but partial parts (since Json.NET handles the rest of the graph).
Fixing bug with JsonReader disparity with Json.Net when loading references
Simplifying cycle detection logic
Deprecating obsolete classes
Creating new class to support legacy code.

Fixes https://github.com/microsoft/botbuilder-dotnet/issues/3970

Tested with Calendar and Todo skill, they all work well.


add clips_vendor to index of current distros

Please Add This Package to be indexed in the rosdistro.
humble jazzy and rolling
The source is here:
https://github.com/carologistics/clips_vendor
Checks

[] All packages have a declared license in the package.xml
[] This repository has a LICENSE file
[] This package is expected to build on the submitted rosdistro

Explanation
I would like to vendor CLIPS (https://clipsrules.net/) as a preparation for releasing a high-level knowledge-based reasoning framework the CLIPS-Executive for ROS 2 using it. We have been using CLIPS for many years now, but never went through with actually making it available to the ROS ecosystem, which I now want to correct.
A very old version of CLIPS is available in Ubuntu already (https://packages.ubuntu.com/noble/clips) (Version 6.30-4.1build1) as well as on fedora (https://packages.fedoraproject.org/pkgs/clips/clips/) with version 6.31-10.
However, Version 6.4x has been around for years now, offering a more modern API  and version 7 is currently being developed as well.
Packaging CLIPS is a bit of a hassle as it is plain C code and makefiles (and the build system in general is minimalistic, not compiling shared libraries).
The packaged versions for Ubuntu and Fedora use plain makefiles and provide pkg-config files for projects.
To make it broadly available in the ROS ecosystem I instead  wrote a new CMake based buildsys that is used by the proposed clips_vendor package, which allows for a way easier integration with other ROS projects and offers some additional QoL things such as optionally building the java-based examples and IDE.
The vendor package exposes 3 ways of leveraging CLIPS, as plain C library, as C++ library or as C++ library with added namespaces (by automatically wrapping every code block with a namespace). The latter variant is what I would recommend for usage to minimize the amount of unintentional interference with other libraries (which is sadly still unavoidable to some extend due to the large number of preprocessor macros used throughout CLIPS).
The package is building fine on Ubuntu and Fedora (tested on humble and jazzy and saw no breaking changes in rolling), I have no Windows setup to test it on at the moment, but CLIPS itself is compatible with Windows and therefore I only expect some tweaking to be necessary (if at all) to make it available on Windows.
There will be two warnings when building (and I already oppressed many more):
/home/tviehmann/ros2/clips_ws/build/clips_vendor/clips_vendor-prefix/src/clips_vendor/core/textpro.c:714:25: warning: invalid conversion from ‘void*’ to ‘char*’ [-fpermissive]
  714 |     theString = genalloc(theEnv,length+1);
      |                 ~~~~~~~~^~~~~~~~~~~~~~~~~
      |                         |
      |                         void*
At global scope:
cc1plus: note: unrecognized command-line option ‘-Wno-invalid-cast’ may have been intended to silence earlier diagnostics
/home/tviehmann/ros2/clips_ws/build/clips_vendor/clips_vendor-prefix/src/clips_vendor-build/ns_core/textpro.c: In function ‘clips::TextProError clips::ParseLine(Environment*, const char*, int*, char*, char**)’:
/home/tviehmann/ros2/clips_ws/build/clips_vendor/clips_vendor-prefix/src/clips_vendor-build/ns_core/textpro.c:723:25: warning: invalid conversion from ‘void*’ to ‘char*’ [-fpermissive]
  723 |     theString = genalloc(theEnv,length+1);
      |                 ~~~~~~~~^~~~~~~~~~~~~~~~~
      |                         |
      |                         void*

I could patch them out in principle, but for maintainability I so far tried to stay away from patching individual things in CLIPS (I am not the maintainer of CLIPS, just an enthusiastic user).
Once the package is indexed I intend to start the release procedure, following the documentation of releasing a package.
So feedback is very welcome :)

Can you point to a license file for CLIPS? I see this on the website (https://clipsrules.net/):

Since 1996, CLIPS has been available as public domain software.

but I can't find a license file here: https://svn.code.sf.net/p/clipsrules/code/branches/64x (or by searching the files after cloning it locally using git svn clone https://svn.code.sf.net/p/clipsrules/code/branches/64x).

The license is documented in the basic programming guide https://clipsrules.net/documentation/v641/bpg641.pdf

general comment: I think the license of the vendor package doesn't need to match that of the vendor, but I could be wrong
Maybe @nuclearsandwich or @cottsay can say what we do in most of our vendor packages.

It would be a lot cleaner if we had a definitive license/statement we could point to, but maybe others think this is enough.

general comment: I think the license of the vendor package doesn't need to match that of the vendor, but I could be wrong

I think we usually have both the license of the vendored software and the license of the vendor package itself (e.g., Apache 2.0) in the vendor package's package.xml. Some examples:

https://github.com/ros2/libyaml_vendor/blob/5545d39cf02ea3678288d0fa235036f3851d9fb2/package.xml#L12-L13
https://github.com/ros2/mimick_vendor/blob/aad6edc1cfdc0dcca18fcd2ea3d11a585662f408/package.xml#L14-L15
https://github.com/ros2/yaml_cpp_vendor/blob/cb9b3a0cfd15a27e8362ce6deb183e486796f8ec/package.xml#L14-L15
https://github.com/ros2/pybind11_vendor/blob/e8b3edb280efc6eea36b7218e0b71cdc8c85b7e4/package.xml#L14-L15
https://github.com/ros2/spdlog_vendor/blob/ddbe8fe82f868e1418590e1dd0b64d6b863de508/package.xml#L16-L17

I think we usually have both the license of the vendored software and the license of the vendor package itself (e.g., Apache 2.0) in the vendor package's package.xml. Some examples:

ah nice, I was not aware of that! I would like to keep the vendor package under MIT-O aswell, but should probably at least add a comment in the package.xml file to reflect that it applies to both the vendor and  the vendored software.
I also contacted the author to request adding a LICENSE file (and if possible, update the license tag in sourceforge):
https://sourceforge.net/p/clipsrules/discussion/776945/thread/f3176d3efe/

Per the author, the license is now part of the source code https://sourceforge.net/p/clipsrules/discussion/776945/thread/f3176d3efe/#e913
I updated the vendor package to fetch the latest version to reflect that and distinguished between vendor and source package license via a comment:
https://github.com/carologistics/clips_vendor/blob/edd029d04e7753589e082aa40f240df58732abfa/package.xml#L8


Save button in content editor downloads php files instead of saving

Hi Diego,
I've updated my bludit installation to the 3.11.0 Pro version lately.
Today I wanted to change published content. After I did the change and clicked on "Save" the site was starting the download of a PHP file instead of saving the content.
More irritating: This is only true for one specific article. All other contents can be changed without problems.
Any idea what could cause such a behavior?
Regards,
Christian

Hi,
it's really strange behavior.
Maybe there is some code inside the page? or some configuration in your webserver, some regex ?
Which web server are you using and PHP version ?
I can recommend following these threads.

https://stackoverflow.com/questions/5121495/php-code-is-not-being-executed-instead-code-shows-on-the-page
https://stackoverflow.com/questions/12142172/apache-shows-php-code-instead-of-executing

Hi Diego,
thanks a lot for the quick reply!
Yes, it is a strange behaviour indeed. There is no special stuff with this specific content. Just an article with some images like other ones in my bludit installation.
I'm using a webspace package provided by the german hoster Strato and I played a little bit around with the PHP versions. The behavior persists from version 7.2 to 7.4.
I will have a look to the links you provided later. Thank you for these hints.
I could also provide you the content file if this helps you investigating.
Regards,
Christian

Can you try to reproduce it on the demo (https://demo.bludit.com), I mean with the same content and URL name, but I'm pretty sure is from the webserver the issue.

Hi Diego,
today I wanted to dig a little bit deeper into this issue.
What should I say... It isn't reproduceable anymore. I can change and the article without any problems. I didn't change anything. No update of Bludit or the plugins since I reported this issue.
We are living in a strange world sometimes ;o)
I will close this issue.
Thanks for your tips and time.
Stay safe and healthy with your family!


fix security issue Uncaught DOMException when click to share

If this does what it does please merge it for alpha


Updated DNS article to Control Panel

Removed Service Panel specific information
Added how to manage DNS in the Control Panel

docs/hypernode-platform/dns/dns-for-control-panel-users.md:166: WARNING: 'myst' reference target not found: /services/control-panel/how-to-link-your-domain-to-your-hypernode.html#link-and-sync-your-domain-to-your-hypernode


tests fail on Scala version that will soon become 2.13.0-M3

in recent 2.13 community build runs such as https://scala-ci.typesafe.com/view/scala-2.13.x/job/scala-2.13.x-integrate-community-build/919/consoleFull we see:
----------------------------------- Failures -----------------------------------
X test.utest.examples.TestPathTests.'testPath.'foo 10ms
  utest.AssertionError: assert(implicitly[utest.framework.TestPath].value == Seq("testPath", "foo"))
    utest.asserts.Asserts$.assertImpl(Asserts.scala:114)
    test.utest.examples.TestPathTests$.$anonfun$tests$3(TestPathTests.scala:9)
X test.utest.FrameworkTests.'catchCastError 0ms
  utest.NoSuchTestException: [ah]
    utest.TestRunner$.runAsync(TestRunner.scala:60)
    utest.TestRunner$.run(TestRunner.scala:124)
    test.utest.FrameworkTests$.$anonfun$tests$79(FrameworkTests.scala:232)
X test.utest.FrameworkTests.'testSelection 0ms
  utest.NoSuchTestException: [A]
    utest.TestRunner$.runAsync(TestRunner.scala:60)
    utest.TestRunner$.run(TestRunner.scala:124)
    test.utest.FrameworkTests$.$anonfun$tests$84(FrameworkTests.scala:249)
X test.utest.FrameworkTests.'testPath.'foo 1ms
  utest.AssertionError: assert(implicitly[utest.framework.TestPath] == TestPath(Seq("testPath", "foo
  ")))
    utest.asserts.Asserts$.assertImpl(Asserts.scala:114)
    test.utest.FrameworkTests$.$anonfun$tests$107(FrameworkTests.scala:294)

the problem is reproducible outside of dbuild at https://github.com/scalacommunitybuild/utest/tree/community-build-2.13 with ++2.13.0-pre-79e5101
(perhaps related, perhaps not: the eta-expansions I had to add in scalacommunitybuild/utest@99e30c579d9c38a956e7ddcc36a5fc5d14b29324 seem suspicious to me; I'll talk to the team about whether this is intentional or not) (never mind, it's correct, nullary eta expansion was deprecated in 2.12.0 and is an error in 2.13)

2.13.0-M3 has been published

it's green again in recent 2.13 community build runs


OffsetDateTime + Duration behavior

It would be nice to have this behavior documented somewhere.
Can this operation overflow? If so, will it panic? Will it wrap or saturate value?
It would also be nice to have checked_*, saturating_* and wrapping_* families of methods (with duration operand).

I have started working on this.

Would you like to do take on the saturating methods as well? The actual implementation should be trivial; it's the documentation and tests that'll take a short bit.

Sure. Actually, I almost finished implementation, docs and tests. This is how I found #384 (:

Awesome. I'll hold off on changing the internals of OffsetDateTime until then.

Also, what do you think about adding MIN/MAX constants for PrimitiveDateTime, MAX constant for Time and min_for_offset/max_for_offset inherent const functions for OffsetDateTime, at least with pub(crate) visibility? They are not strictly required, but I would like to have them in saturating_* methods rather than literal values (via macros):
pub const fn saturating_add(self, duration: Duration) -> Self {
    if let Some(datetime) = self.checked_add(duration) {
        datetime
    } else if duration.is_negative() {
        Self::MIN
    } else {
        Self::MAX
    }
}

For PrimitiveDateTime pub const {MIN,MAX} is fine. For Time, let's keep it pub(crate) (you can add MIN for symmetry if you'd prefer).
Given that I'm going to at least attempt, and likely migrate towards, an implementation that stores the value in the local offset and not UTC, I don't think those methods are necessary for OffsetDateTime. Honestly it'd be fine if you just stubbed them out — the documentation and tests are where the real work is at, with the latter being nearly identical to PrimitiveDateTime. If you go that route, just put #[ignore] on the relevant tests for the time being so CI doesn't fail.

Got it. Actually I can just do PrimitiveDateTime::MIN.assume_utc().replace_offset(self.offset) for now. With current ranges, this will do the same thing as min_for_offset.

Closing as implemented. Thanks again.


javascriptGlobal, javascriptMessage should be removed from javascript.vim

The keywords they define are not standard js, and they are in syntax/yajs/web-window.vim anyway, which is where they belong. Looks like they might be left over from some old code?

If you are willing to help test.
I start to refactor this part in webidl branch.
https://github.com/othree/yajs.vim/tree/webidl

I will try out that branch, thanks.
On Jan 8, 2016 2:29 AM, "othree"<EMAIL_ADDRESS>wrote:

If you are willing to help test.
I start to refactor this part in webidl branch.
https://github.com/othree/yajs.vim/tree/webidl
—
Reply to this email directly or view it on GitHub
https://github.com/othree/yajs.vim/issues/62#issuecomment-169957294.


Add missing "name" argument to init()

Fixes #648.

Coverage decreased (-0.05%) to 41.808% when pulling 16c1db93c9bf6185a835245792c2fca314d18580 on techhat:nameerror into 3e80c20b0008ba6004753004be1462f0295c4b1b on Azure:master.

Coverage decreased (-0.05%) to 41.808% when pulling d47ebd79727e806a8b408fe8da8acc2ac12120ad on techhat:nameerror into 3e80c20b0008ba6004753004be1462f0295c4b1b on Azure:master.

Answered why closed in the #648 issue.


poetry build builds wheel without compression

[x] I am on the latest Poetry version.
[x] I have searched the issues of this repo and believe that this is not a duplicate.
[x] If an exception occurs when executing a command, I executed it again in debug mode (-vvv option).

OS version and name: Windows 7
Poetry version: 0.12.10 (Python 3.7.0 x64)
Link of a Gist with the contents of your pyproject.toml file: https://gist.github.com/outcatcher/fdcc2c4f69236cc561476504ade6ba19

Issue
This is not exactly an error, rather just unexpected behaviour, maybe should be a feature request?
Poetry builds wheel with bigger file size than setuptools setup.py for same source and same wheel content
Reproduced on repository: https://github.com/outcatcher/poetry_not_compressing_build
Versions of wheels are be different: 0.1.0 for poetry and 0.1.1 for setup.py

Closed by #838

It has been fixed in the latest 0.12.12 version


Module 4 - Code Review

Module 4: Adding User and API features with Amazon API Gateway and AWS Cognito
Code: https://github.com/DavidChristiansen/aws-modern-application-workshop/tree/cdk-typescript/module-4

All of the same comments as in module 1 here are true in this module - 2 vs. 4 spaces, class names, lowercase properties and locals, readonly properties, Stack props, etc.
Other than that, looks good.

Done, please re-open if you need more info from me.


feat: doc for audiomonophoner crafter

Doc for audiomonophoner crafter

@deepankarm thanks for the review, I've incorporated the changes in this PR


Frontend / Client should be inside a separate folder

@EugeneGohh Front / Client should be inside a separate folder not nested inside of the backend / server folder.

there should be one front-end (or client) and one back-end (or server) project directories. I guess they would probably share one parent directory folder i.e. "project-folder"
This separation is necessary because for some reasons I am aware of but the biggest reason is the package.json and "build" (for production)  will not work correctly or have conflicts. Not a great explanation but that's the gist of it.

Example from Stackoverflow:
Client Folder
- /client
-- /src
...
-- index.js
-- registerServiceWorker.js
- .gitignore
- package-lock.json
- package.json

Server Folder
- /server
- /models
-- item.js
- /node.modules
-- ...
- server.js
- package-lock.json
- package.json

Got it thanks. It's this thread from StackOverflow right?

closed first request
We're closing the first pull request and trying again from Eugene's local.


Make test for creating a webhook more robust

We should create a new repo, put a webhook in it, and delete the entire repo; this will:

Not bind this test to a dependency upon a source repository ( a remote resource)
Ensure we don't accidentally remove webhooks that we didn't create (as we are currently doing by removing all webhooks for a repo)

.  Means we need to add SPI support to delete a repository.
Depends on #61

Depends on #58

Currently:
@Test
    public void createGithubWebHook() throws Exception {
        // given
        final URL webhookUrl = new URL("https://<IP_ADDRESS>");
        final GitHubRepository targetRepo = getGitHubService().fork(NAME_GITHUB_SOURCE_REPO);
        // when
        GitHubWebhook webhook = getGitHubService().createWebhook(targetRepo, webhookUrl, GitHubWebhookEvent.ALL);
        // then
        Assert.assertNotNull(webhook);
        Assert.assertEquals(webhookUrl.toString(), webhook.getUrl());
        // After the test remove the webhook we created
        ((GitHubServiceSpi) getGitHubService()).deleteWebhook(targetRepo, webhook);
    }

Let's change this to instead of forking, make a new repo, and instead of removing a webhook, delete the repo.

1aa7df0e834e5b25e05b9957e1895c82a01e3bc2


Update utils.py

Fix the bug when testing the model , the input noise is a zero vector, actually the input noise is Gaussian distribution.
And change the dir test picture from "samlpes" to "test_samples" to make it more clear.

This is fixed in #233. I think it's better to keep them all into samples because the name, test_samples, is vague.


Removed whitespace that messed up formatting

@mgnslndh : Thanks for your contribution! The author(s) have been notified to review your proposed change.


🐞[Bug]:  There is a simple navbar below the main navbar

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
There are two navbars one below the other. The small , simpler one is unnecessary and should be removed
Expected behavior
Only one main navbar should exist
Add ScreenShots

What browsers are you seeing the problem on?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

Please assign this issue to me


Phred Upgraded HardHat is big purple block

The PHRED Upgraded hardhat has an problem with the texture, and renders as a big purple blockhead


property indicating "root" part

We currently infer the root part of a work (ex: W22084_1088:

:workPartOf W22084_S0013
has root part W22084

We currently use tmp:workRootWork, but then we have the problem of defining a skos:prefLabel, etc. for this new property...
Maybe :workPartOfRoot?

resolved by adding :workHasRoot


Updating roadmap - Android 13

Hey @hamza-usmani do you have any idea when 2211 will be ready for general availability?  We are tracking an issue that appears to be resolved in Android 13.
Looks like releases have stopped since October 20th


Please update

This page looks obsolete. Powershell commands look old, please update the contents using Az powershell module.

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

ID: 60d899e8-cf6e-a99a-91aa-2d812118854a
Version Independent ID: a6b53c2f-078d-4c74-0d3d-49118f1ab122
Content: Configure Load Balancer TCP idle timeout in Azure
Content Source: articles/load-balancer/load-balancer-tcp-idle-timeout.md
Service: load-balancer
GitHub Login: @KumudD
Microsoft Alias: kumud

@kikawash , Thank you for your feedback. We will investigate and update the doc accordingly.

@KumudD , Looks like this doc has commands of Classic model. Can you change the commands/ Samples to new Az commends ?

#reassign:@asudbring

Updated article to reflect new az commands and removed legacy content
#please-close


WSL instances won't display the tab title set in SUI

Windows Terminal version
main fbe0416
Windows build number
10.0.19044.0
Other Software
WSL Ubuntu, Arch & Fedora
Steps to reproduce

set Tab title in SUI

Expected Behavior

new WSL tab to have defined Tab title

Actual Behavior

WSL instances won't show Tab title from settings.json, instead it shows user@host
it does show correct tab title for Powershell and Command Prompt

Sample setting for WSL Ubuntu:
           {
                "guid": "{2c4de342-38b7-51cf-b940-2309a097f518}",
                "hidden": false,
                "name": "Ubuntu",
                "source": "Windows.Terminal.Wsl",
                "startingDirectory": "~",
                "tabTitle": "Ubuntu"
            },

and terminal:

you probably want this setting too:


Create WorldModel class

Contains the following methods:

[x] AddLandmark - method for adding a landmark to the world model
[x] AddRigidBody - method for adding a rigid body to the world model
[x] AddParametersToProblem - Adds all landmarks and rigid bodies to a ceres problem as either free parameters or constants

Resolved with commit b5745d77cb25e0bb54af9f1bce263b41afb263f3


Update build.gradle, fixed error: resource android:attr/fontVariationSettings not found, error: resource android:attr/ttcIndex not found.

Changed compileSdkVersion to  28, minSdkVersion to 19

This will fix #2


Feature: Allow setting scalar lists without set

This works:
  const resultJsonUpdateWithoutSet = await prisma.post.update({
    where: {
      id: resultJsonArray.id,
    },
    data: {
      coinflips: {
        set: [true, true, true, false, true],
      },
    },
  })

This does not:
   const resultJsonUpdateWithoutSet = await prisma.post.update({
    where: {
      id: resultJsonArray.id,
    },
    data: {
      coinflips: [true, true, true, false, true]
    },
  })

(Note the missing {  set: ... } after coinflips)
Error message:
...
Unknown arg `0` in data.coinflips.0 for type PostUpdatecoinflipsInput. Available args:
type PostUpdatecoinflipsInput {
  set?: List<Boolean>
}
...

Possible TODO:

[ ] fix dmmf transformation (should work with and without an explicit set:)
[ ] un-comment runtime test https://github.com/prisma/prisma/blob/104db729f96f8959cd7e02750baebc7e8c947e25/src/packages/client/src/tests/runtime-tests/blog-env-postgresql/index.js#L295
[ ] update snapshot https://github.com/prisma/prisma/blob/104db729f96f8959cd7e02750baebc7e8c947e25/src/packages/client/src/tests/update.test.ts#L263

This was never intended to work and therefore is not a regression.
It applies to all scalar lists and is something we should discuss if it's something we want, as this sounds like a valuable addition to the API to me.

Note: Probably covered by https://github.com/prisma/prisma-engines/issues/1088 - so no implementation work in Typescript necessary for now.


v10.1.0-rc.1

Release Notes - Counterparty Core v10.1.0-rc.1
Note: This update requires a reparse from block 835,500. (If you are upgrading from v9.x.y, then you need a full database rebuild.)
ChangeLog
Bugfixes

Validate non-empty block_indexes in call to api.get_blocks (fix for #1621)
Reproduce order expiration bug in v9.61 (fix for #1631)
Fix get_blocks call when several block indexes are provided (fix for #1629)

Codebase

Split out counterparty-cli package into counterparty-core and counterparty-wallet packages
Implement light / heavy healthz probes
Automatic code checking and correction with Ruff
Refactor transaction file singleton to class
Run PRAGMA optimize on shutting down

Command-Line Interface

Rename counterpary-client to counterparty-wallet

Credits

Ouziel Slama
Adam Krellenstein
Warren Puffett
Matt Marcello

With https://github.com/CounterpartyXCP/counterparty-core/pull/1642 and https://github.com/CounterpartyXCP/counterparty-core/pull/1644 I think we are good.

Awesome! Great job making this so fast.


Render to texture only shows clear color

I'm trying to get basic render to texture working but so far the only thing I see in the image copied out of the texture is the clear color.  If I comment out setViewFrameBuffer() https://github.com/lucasw/bgfx_ros/blob/ac6d1862635b1e990fac5c9b63f65fb740097978/src/bgfx_ros.cpp#L365 the cubes appear fine back on the regular display window.
I've looked at the hdr example and some of the other issues here about rendering-to-texture, but I think I'm missing something basic about the process that the more advanced  examples are obscuring.  Do I need to modify my fragment shader to render to the texture?  Do I need a setTexture, and if so what is the UniformHandle for?  I've tried adding that, but I notice in all the examples the UniformHandle is being used within the corresponding shaders.
Eventually I'd like to eliminate the window (currently provided by SDL) and only use the rendered image elsewhere in my application.
This is using the latest bgfx/bx source, and the cubes and shaders are almost exactly like in the 01 cubes example https://github.com/lucasw/bgfx_ros/tree/master/src

You're writing and reading from the same texture at the same time.
Draw something in frame buffer view 0, then in view 1 use that texture to render in back buffer.


Opening a message while sort-by unread moves message from previous view

Expected behavior
When using sort-by unread, opening a message and pressing next/prev will use the same ordering I saw before clicking on the message.
Actual behavior
Opening the message causes the order to shift, and next/prev no longer make much sense. The message moves as it is now read. Unfortunately this makes viewing a few messages in the middle of a thread very difficult.
Steps to reproduce

Change ordering to sort-by unread
Open Message
Click next/previous arrow doesn't go to where it was previously

Environment
K-9 Mail version: 5.208
Android version: 7.0
Account type (IMAP, POP3, WebDAV/Exchange): IMAP/SMTP

Opening the message marks it as read, which changes the position of the current message in the message list.

Fixed by #6271


Add --custom-pass-list and --list-passes

Adds a CLI option to cpg-neo4j that allows a user to choose specific passes to run. Example: Use --custom-pass-list=DFGPass to run only DFGPass. This is especially useful for debugging.
This feature requires TranslationConfiguration to keep a list of all choosable passes. Currently, the list is stored in a global variable of KClass objects (that reflection is used on in order to get their names and create instances of them). For now, this variable only contains passes that are part of defaultPasses(). This could be easily changed by adding other passes to the variable.

Note: Many passes depend on another pass, so what you specify here and what you will actually observe will differ

Hi everyone, thanks for your feedback! I believe I resolved most issues with the last commit. Also wrote some more notes on auto-generating the list and pass dependencies above.

Sorry for dragging this along (and be somewhat negative towards this), but somehow this doesn't feel right to me. It introduces three list of passes - which are completely static, so I am not quite sure what the point of returning them is via the config. You could just return the list directly in the neo4j application if you really need this information in the CLI.
With regards to registering custom passes, I would suggest following the same as we have with the languages:
https://github.com/Fraunhofer-AISEC/cpg/blob/747d435d700df04ca38bbb28637fa3cf034c2140/cpg-core/src/main/kotlin/de/fraunhofer/aisec/cpg/TranslationConfiguration.kt#L390-L411
This has the slight disadvantage that you need to specify the complete FQN, but it has the advantage that you can register any pass that is somehow on the class path.
If we really want to have a list of all possible passes, we could use https://github.com/ronmamo/reflections#usage (JVM only) or possibly move towards a dependency injection framework like https://github.com/InsertKoinIO/koin (in the future)

So I will try to sum up and find tasks to solve those concerns and find some middle ground to the current implementation:

I agree that we should try to keep as much static pass configuration out of the cpg-core and move it to the cpg-neo4j, expecialy because the neo4j is the only component that needs to register passes by name. I suggest moving these functions to cpg-neo4j as suggested.
When it comes to the name based registration: I like having the ability to register all existing passes, on the other hand it is unhandy to specify all passes over their FQN. Suggestion: allow both, create passes from their name, if that fails, e.g. because the pass name was a simple name and not a FQN, prepend out default pass package name.

The last part will allow to register passes not shown among the available passes list, but that is a feature then. Inclusion of an additional framework is something that we should look into but I think it would drag this PR on even longer. Lets investigate that in the future if we see the need to have a complete list of specifiable passes.
How does that sound @oxisto @shenniger?

Thanks all for the good ideas!
I have pushed two major changes to this:

Everything about the custom pass list was moved to cpg-neo4j and out of the main module.
As suggested, FQDNs can now be used in addition to the passes in the list.


Only encode spaces as + in query params and values

As per RFC 3986, spaces in the path portion of the URL must be encoded as %20 and never +. + must be left as-is (or optionally encoded as %2b).
Spaces can be encoded as + or %20 in content with the type application/x-www-form-urlencoded, which as per RFC 1866 is the content encoding for the query part of the URL only - it is not applicable for the path part of the URL.
There's no specification detailing how spaces in fragments should be encoded, but I checked how Chromium and Firefox handles it - both browsers encode spaces as %20 and treat + as-is. That means, if a DOM element has an ID of test 1, then the URL fragment #test 1 scrolls to it, but the fragment #test+1 does not. #test+1 only scrolls to the element if it has the literal id value test+1. Surprisingly, the id value test%201 also works for with the URL fragment #test 1 in both browsers.
Firefox displays fragments with spaces in them literally in the URL bar (http://domain.tld/#test 1), while Chromium transforms all entered spaces to %20 (http://domain.tld/#test%201).
This PR introduces new procedures encodeQueryComponent and decodeQueryComponent, which behave as encodeUrlComponent and decodeUrlComponent accordingly do currently. encodeUrlComponent and decodeUrlComponent have been modified to unconditionally encode spaces as %20, and leave + as-is (not encoded). Procedures which handle encoding and decoding full URLs have been updated to use the correct procedures when encoding or decoding URL parts. Tests have also been updated to reflect this "new" behaviour.
This PR fixes issue #5 and subsequently https://github.com/treeform/puppy/issues/80.

Great PR! Thanks.


defaultView

Can I use 'timelineDay' as my defaultView?

Yes you can, however, you will need to purchase a license key for Schedule from FullCalendar and provide the key in the options otherwise you will be using the trial.

What about adding day-week-month, how to do it, only month and listWeek works with me ?


Add Icon to a header cell and use Pipe in rows

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[x ] support request => Please do not submit support request here

Current behavior
Hello, I want to put icon in column header cell, and use a Pipe in row cell.
Pipe will count a length of row contain and add photo, when lenght is less then 4.
I can put icon to a header cell this way:

columns = [
 {
    // Prop represents the name of the property that holds this cell's value
    prop: 'description',
    // Name represents the label for the column header.
    name: 'Description<i class="fa fa-warning"></i>'
}
]

I found a few solutions but all of them using ng-template like this:
   <ngx-datatable-column name="facility" >
   	 <template let-columns="columns" ngx-datatable-cell-template>
            	{{columns}}
            </template>
       </ngx-datatable-column>
       <ngx-datatable-column name="system" >
            <template let-row="row" let-value="value" ngx-datatable-cell-template>            		{{value}}
            </template>
    </ngx-datatable-column>

In that way me columns declared in component:
	public columns=[
		{x},
		{y} ...
	]

were changed to a value from ngx-datable-column and icon disappears.
Expected behavior

Reproduction of the problem

What is the motivation / use case for changing the behavior?

Please tell us about your environment:
Windows 10

Table version: 6.3.0

Angular version: 2.4.10

Browser: [all ]

Language: [all | TypeScript X.X | ES6/7 | ES5] Typescript 2.0.3

I don't think I understand what you're saying here. Could you provide some screenshots?
What is this supposed to do?
<template let-columns="columns" ngx-datatable-cell-template>{{columns}} </template>
You declare your columns on the outer ngx-datatable component. I don't think columns will populate here; I don't think Swimlane provides in in their ngx-datatable-cell-template OutletContext.

Thank You vary much for hints, and sorry that I can't well describe my problem, also Your hint was very helpful.


Inquiring Kapow

Hey, I know that you configured the Ethminer for the GPU. But it is generating a 0 mh/s hashrate compared to tRex generating 20 mh/s per computer. The only problem previously that you mentioned was that it was not open source. Is it possible for you to configure any open-source miner for Raven Coin as an option to mine because it is one of the most profitable ones out there for mining on pc? It would really mean a lot because XMR does not generate as much money as Ravencoin does.  I don't trust anyone else except for you. Usually, people who have had the XMR mining software that you made (the silent xmrig) text me that their computer was slow and it was not really undetectable. If I wanted the computer to be faster, it would generate less of a profit, but on the plus side, Ravencoin only uses GPUs, which is fine with some process killers. So if possible can you add a ravencoin gpu miner that can mine with NVIDIA? Thanks for your time.

Oh one more thing if possible could you also add a feature like "high performance mode" so the computer can run at high performance mode instead of just Balanced, or Power Saver? Thanks!

configured the Ethminer for the GPU. But it is generating a 0 mh/s hashrate compared to tRex generating 20 mh/s per computer. The only problem previously that you mentioned was that it was not open source. Is it possible for you to configure any open-source miner for Raven Coin as an option to mine because it is one of the most profitable ones out there for mining on pc? It would really mean a lot because XMR does not generate as much money as Ravencoin does.  I don't trust anyone else except for you. Usually, people who have had the XMR mining software that you made (the silent xmrig) text me that their computer was slow and it was not really undetectable. If I wanted the computer to be faster, it would generate less of a profit, but on the plus side, Ravencoin only uses GPUs, which is fine with some process killers. So if possible can you add a ravencoin gpu miner that can mine with NVIDIA? Thanks for your time.

so i know you need cuda for navida device to work but from what i know you have to do it manually to each computer with navida devices, i asked in #389 at the very bottom if there was a way to have the silent miner be incorporated with cuda so there won't be any need to have the user download it and load

Hey, I know that you configured the Ethminer for the GPU. But it is generating a 0 mh/s hashrate compared to tRex generating 20 mh/s per computer. The only problem previously that you mentioned was that it was not open source. Is it possible for you to configure any open-source miner for Raven Coin as an option to mine because it is one of the most profitable ones out there for mining on pc? It would really mean a lot because XMR does not generate as much money as Ravencoin does.  I don't trust anyone else except for you. Usually, people who have had the XMR mining software that you made (the silent xmrig) text me that their computer was slow and it was not really undetectable. If I wanted the computer to be faster, it would generate less of a profit, but on the plus side, Ravencoin only uses GPUs, which is fine with some process killers. So if possible can you add a ravencoin gpu miner that can mine with NVIDIA? Thanks for your time.

How are you checking the hashrate? Using the web panel? And there aren't any open-source miners for Ravencoin that work. The GPU miner in the silent miner should work just fine as long as all the settings are correct. Most people mine KawPow and RandomX using the miner.

Oh one more thing if possible could you also add a feature like "high performance mode" so the computer can run at high performance mode instead of just Balanced, or Power Saver? Thanks!

Yes maybe, though the user might notice that.


Make ensurepath also ensure pip-user-installed pipx script is in path

Fixes #427
This PR changes pipx ensurepath to test to see if pipx has been installed using pipx install --user and if it has, then ensure that the path to the pipx script itself is in the user's PATH along with pipx's packages bin directory.
It reports on a per-path basis the status ("successfully added", "already added", "needs shell restart").  It also prints summary messages after all paths have been handled similar to the previous ensurepath.

BTW I've tested this on Windows and macOS, using as many different install possibilities as I could (user, non-user, rebooted shell, non-rebooted shell)


Please add license

In case someone would like to continue your work onward, this project needs a clear license file. Otherwise for a legal perspective such work might be unusable in formal contexts. Would it be possible to add a license to the project?

I specified the license in the package.json but I forgot to add the license file.
Thanks for letting me know


Fixing an incorrect constant so we will report emulation failures app…

…ropriately
This PR is against the v1.1.0 branch for the upcoming release.

:+1:

Looks good.

LGTM!


Update BingoGoalPack1

Changing Peaks to Peak in a few places.
This should probably wait to be merged until BingoSync and BingoSyncExtension are both ready with similar changes

Alpha's extension provides a sufficient fix, so this can safely be merged now.


Faster and more compact bracket matching function

Need a better replacement for the matching_bracket() in loadjson. The current function is very slow in octave (matlab 8.x is ok, despite that profiling still returns high run-time)
https://github.com/fangq/jsonlab/blob/master/loadjson.m#L459-L496

It would be helpful to have an example to test against.


Feat resolve#6

#6

Man on description of your PR you must have to write "resolve #issuId" for example "resolve #6"
About your branch name, it must have a short description of your task like "feat-card" or "feat-movie-filters"... do you understand?

about colorsDarkMode why do you use .txt instead .json, i think it would better than .txt

Man the API_KEY file you must have to DELETE!
Keep it in .env file


Adding buy me a coffee or sponsor button

What feature?
will be adding this button
Add screenshots
Add screenshots
Code of Conduct

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

Wud love to work on it

Not Required now.

Not Required now.

alright


[skrifa] NamedInstance.postscript_name_id "not present" case

In an instance record, the postscript name ID is optional. postscript_name_id is therefore typed as Option<NameId>. All good.
However, in the binary, the "not present" case is represented as binary 0xFFFF. When the postscript name ID is not present, both skrifa NamedInstance.postscript_name_id and read-fonts InstanceRecord.post_script_name_id do not return None but instead return Some(65535) - so in fact this Option always returns Some.
I feel that certainly in skrifa as a "mid-level" library, and possibly also in read-fonts, the "not-present" case should be interpreted from the binary value and returned as a None.

Are you using an older version of read-fonts? This was changed a couple months ago: https://github.com/googlefonts/fontations/blob/d52fd0c7258701cdd52dd82be7937be9bc41684f/read-fonts/src/tables/instance_record.rs#L56

Sorry, yes, I was. Confirmed working OK now.


Add multipart actions

Hello,
During multipart process , it will be very interesting for us to make action when part upload is finished.
For example, a simple progress bar in file ..
How can add a new function to do that in goofys code ?
Regards,
Nicolas Prochazka

goofys cannot support this since it works at the file system level, underneath the application.  You can use something like pv, or pipe viewer:
http://linux.die.net/man/1/pv

goofys knows when it sends new parts of multipart content, is it possible to call a function at this time,
a go fonction, for exemple that calls a lambda function.
Regards,
Nicolas

goofys has no way to interact with the terminal; it works at the file system level.

I do not interact with terminal, I want to execute a Go funtion when a part
is finishing to upload.
Regards,
Nicolas Prochazka
2016-09-06 18:51 GMT+02:00 Andrew Gaul<EMAIL_ADDRESS>
goofys has no way to interact with the terminal; it works at the file
system level.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kahing/goofys/issues/104#issuecomment-245014550, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABgm0PJu3guQ1a_dfbX1mYTPMb13yNh1ks5qnZn2gaJpZM4J1HjN
.

 Goofys isn't quite designed for that, any reason why you don't use the aws sdk directly?

hello,
before using goofys, we are using s3cmd . We have modify s3cmd code, so
when a part of multipart upload finish, s3cmd send some command ( example:
an sns message, a sqs message ) , so we can log the progress of uploading
file with an other application.
Now we like do the same thing with goofys, when a copy is progressing, when
a part of multipart is finished, send a new message,
so we think that a "trigger function" when a part ip finished , in goofys
is a good thing for us :)
Regards,
2016-09-06 23:16 GMT+02:00 Ka-Hing Cheung<EMAIL_ADDRESS>
Are you embedding goofys into your own go process? Goofys isn't quite
designed for that, any reason why you don't use the aws sdk directly?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kahing/goofys/issues/104#issuecomment-245094452, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABgm0I5M_GBTaPSQyt6T93f11ocjsNrrks5qndg4gaJpZM4J1HjN
.

this sounds like a very niche use case of the already pretty niche goofys :-P could you describe more about what you do on these notifications on every part upload? I need to understand this better before I can figure out what to do with this.
also, could you describe what's the motivation behind moving from s3cmd to goofys? This is more for my curiousity

When a part of a multipart upload is finished, is interesting to send this information.
The idea is to create a "callback" function in goofys , so dev can  easily implement this callback,
In our case, we send a sns notification, so our product can be informe that file is in progress.
S3cmd does not implement filesystem mount, and we need it because our software open file.
Regards,
Nicolas

If you need to modify goofys for the callbacks anyway, adding the callbacks at the right place isn't that hard. Doesn't sound like it's something that I want to do anything about


[Error] NameError: name 'math' is not defined in compressive_transformer_pytorch.py

hello, I run code "examples/enwik8_simple" now, and I got error as follows:
train.py:65: DeprecationWarning: The binary mode of fromstring is deprecated, as it behaves surprisingly on unicode inputs. Use frombuffer instead   X = np.fromstring(file.read(int(95e6)), dtype=np.uint8) training:   0%|                                                                                | 0/100000 [00:00<?, ?it/s] Traceback (most recent call last):   File "train.py", line 101, in <module>     for mlm_loss, aux_loss, is_last in model(next(train_loader), max_batch_size = MAX_BATCH_SIZE, return_loss = True):   File "/home/donghyun/donghyun/anaconda3/envs/pytorch/lib/python3.7/site-packages/compressive_transformer_pytorch/autoregressive_wrapper.py", line 151, in forward     logits, new_mem, aux_loss = self.net(xi_seg_b, mask = mask_seg_b, memories = mem, **kwargs)   File "/home/donghyun/donghyun/anaconda3/envs/pytorch/lib/python3.7/site-packages/torch/nn/modules/module.py", line 547, in __call__     result = self.forward(*input, **kwargs)   File "/home/donghyun/donghyun/anaconda3/envs/pytorch/lib/python3.7/site-packages/compressive_transformer_pytorch/compressive_transformer_pytorch.py", line 338, in f orward     x, = ff(x)   File "/home/donghyun/donghyun/anaconda3/envs/pytorch/lib/python3.7/site-packages/torch/nn/modules/module.py", line 547, in __call__     result = self.forward(*input, **kwargs)   File "/home/donghyun/donghyun/anaconda3/envs/pytorch/lib/python3.7/site-packages/compressive_transformer_pytorch/compressive_transformer_pytorch.py", line 84, in fo rward     out = self.fn(x, **kwargs)   File "/home/donghyun/donghyun/anaconda3/envs/pytorch/lib/python3.7/site-packages/torch/nn/modules/module.py", line 547, in __call__     result = self.forward(*input, **kwargs)   File "/home/donghyun/donghyun/anaconda3/envs/pytorch/lib/python3.7/site-packages/compressive_transformer_pytorch/compressive_transformer_pytorch.py", line 106, in f orward     return self.fn(x, **kwargs)   File "/home/donghyun/donghyun/anaconda3/envs/pytorch/lib/python3.7/site-packages/torch/nn/modules/module.py", line 547, in __call__     result = self.forward(*input, **kwargs)   File "/home/donghyun/donghyun/anaconda3/envs/pytorch/lib/python3.7/site-packages/compressive_transformer_pytorch/compressive_transformer_pytorch.py", line 140, in f orward     x = self.act(x)   File "/home/donghyun/donghyun/anaconda3/envs/pytorch/lib/python3.7/site-packages/torch/nn/modules/module.py", line 547, in __call__     result = self.forward(*input, **kwargs)   File "/home/donghyun/donghyun/anaconda3/envs/pytorch/lib/python3.7/site-packages/compressive_transformer_pytorch/compressive_transformer_pytorch.py", line 122, in f orward     return 0.5 * x * (1 + torch.tanh(math.sqrt(2 / math.pi) * (x + 0.044715 * torch.pow(x, 3)))) NameError: name 'math' is not defined
so I inserted "import math" into compressive_transformer_pytorch.py file and it work well. I hope you modify compressive_transformer_pytorch.py code.

@DarkAssult thank you for your interest and in testing this :) fixing now!

done!

thank you, I close this issue :)


Multiprocess

More work on multiprocess scenarios. Not quite there yet. Quorums can be specified, they find each other, but of course they do not acquire consensus or proceed. Just stall out. More to do.

Thanks!  I'll do some review tonight, but I don't have a ton of time available so I probably won't finish.  I'll try to finish it up tomorrow after the end of my class.

Looks like we've got a merge conflict with the latest update to stellar-core, but other than that :+1:

:+1:


Dark mode image fix

Resolves #12
Hi there,
I really enjoyed using your app and appreciate its clean UI and efficient use of Riverpod. In this PR, I've made a small enhancement by replacing the empty_dark.png and empty_light.png files with a single PNG image. Additionally, I've introduced an Assets class to manage assets more efficiently, allowing for easier maintenance and future additions. Feel free to leverage this class for any upcoming asset management needs.
Thanks!

Screenshots:

Thank you.


Share Extension won't launch after upgrading to SDK 51

Everything was working on SDK 50. I upgraded to SDK 51 using Expo's instructions. The app itself works fine. When I try to launch the share extension, I now get this error:

I have tried:

clearing all caches (metro, pods, etc)
deleting all node_modules + yarn.lock and reinstalling everything
new builds

Note that I am in a monorepo but this has never caused issues before. I simply upgraded from SDK 50 to SDK 51, updating package versions and nothing else, and the issue started occurring.
I have tried launching the share extension on both the iOS simulator as well as my physical device, same issue occurs (debug mode and release mode fail).
Here are relevant files:
metro.config.js
// Learn more https://docs.expo.io/guides/customizing-metro
/**
 * @type {import('expo/metro-config')}
 */
// Sentry replacement
// const { getDefaultConfig } = require('expo/metro-config');
const { getSentryExpoConfig } = require('@sentry/react-native/metro');

const path = require('path');

const projectRoot = __dirname;
const workspaceRoot = path.resolve(__dirname, '../..');

// Sentry replacement
// const config = getDefaultConfig(projectRoot);
const config = getSentryExpoConfig(projectRoot);

config.watchFolders = [workspaceRoot];
config.resolver.nodeModulesPaths = [
  path.resolve(projectRoot, 'node_modules'),
  path.resolve(workspaceRoot, 'node_modules'),
];

config.transformer = { ...config.transformer, unstable_allowRequireContext: true };
// ! HAVE TO USE metro-minify-esbuild BC TERSER WAS BREAKING THE BUILD ENTIRELY!
config.transformer.minifierPath = require.resolve('metro-minify-esbuild');
config.transformer.minifierConfig = {
  drop: ['console'],
};

/**
 * Add support for share.js as a recognized extension to the Metro config.
 * This allows creating an index.share.js entry point for our iOS share extension
 */
config.transformer.getTransformOptions = () => ({
  resolver: {
    sourceExts: [...config.resolver.sourceExts, 'share.js'], // Add 'share.js' as a recognized extension
  },
});

module.exports = config;

index.share.js
// could be any component you want to use as the root component of your share extension's bundle
import { ShareExtension } from 'app/shareExtension';
import { AppRegistry } from 'react-native';

// IMPORTANT: the first argument to registerComponent, must be "shareExtension"
AppRegistry.registerComponent('shareExtension', () => ShareExtension);

index.js
// import 'expo-router/entry';

import { registerRootComponent } from 'expo';
import { ExpoRoot } from 'expo-router';

// Must be exported or Fast Refresh won't update the context
export function App() {
  const ctx = require.context('./app');
  return <ExpoRoot context={ctx} />;
}

registerRootComponent(App);

app.json expo share extension plugin entry
[
        "expo-share-extension",
        {
          "excludedPackages": [
            "expo-asset",
            "expo-av",
            "expo-build-properties",
            "expo-contacts",
            "expo-dev-client",
            "expo-device",
            "expo-font",
            "expo-linear-gradient",
            "expo-location",
            "expo-notifications",
            "expo-router",
            "expo-status-bar",
            "expo-system-ui",
            "expo-updates"
          ],
          "backgroundColor": {
            "red": 255,
            "green": 255,
            "blue": 255,
            "alpha": 0
          }
        }
      ]

package.json for the expo app
{
  "name": "expo-app",
  "version": "1.0.0",
  "main": "index.js",
  "private": true,
  "scripts": {
    ...
  },
  "dependencies": {
    "@babel/runtime": "^7.18.9",
    "@benable/ui": "0.0.1",
    "@fullstory/babel-plugin-annotate-react": "^2.3.0",
    "@fullstory/react-native": "^1.4.2",
    "@gorhom/portal": "^1.0.14",
    "@intercom/intercom-react-native": "^7.2.1",
    "@react-native-async-storage/async-storage": "1.23.1",
    "@react-native-google-signin/google-signin": "^11.0.0",
    "@react-native-masked-view/masked-view": "0.3.1",
    "@react-navigation/elements": "^1.3.22",
    "@react-navigation/native": "^6.1.6",
    "@react-navigation/native-stack": "^6.10.0",
    "@sentry/react-native": "~5.22.0",
    "@shopify/flash-list": "1.7.1",
    "app": "0.0.1",
    "babel-plugin-module-resolver": "^4.1.0",
    "expo": "^51.0.0",
    "expo-apple-authentication": "~6.4.2",
    "expo-application": "~5.9.1",
    "expo-asset": "~10.0.10",
    "expo-av": "~14.0.7",
    "expo-blur": "~13.0.2",
    "expo-build-properties": "~0.12.5",
    "expo-clipboard": "~6.0.3",
    "expo-constants": "~16.0.2",
    "expo-contacts": "~13.0.5",
    "expo-dev-client": "~4.0.26",
    "expo-device": "~6.0.2",
    "expo-font": "~12.0.10",
    "expo-haptics": "~13.0.1",
    "expo-image": "~1.12.15",
    "expo-image-picker": "~15.0.7",
    "expo-linear-gradient": "~13.0.2",
    "expo-linking": "~6.3.1",
    "expo-location": "~17.0.1",
    "expo-notifications": "~0.28.17",
    "expo-router": "~3.5.23",
    "expo-secure-store": "~13.0.2",
    "expo-share-extension": "^1.10.4",
    "expo-splash-screen": "~0.27.5",
    "expo-status-bar": "~1.12.1",
    "expo-system-ui": "~3.0.7",
    "expo-updates": "~0.25.24",
    "expo-web-browser": "~13.0.3",
    "mixpanel-react-native": "^3.0.2",
    "promise": "^8.3.0",
    "qs": "^6.13.0",
    "react": "~18.2.0",
    "react-dom": "~18.2.0",
    "react-native": "0.74.5",
    "react-native-draggable-flatlist": "^4.0.1",
    "react-native-gesture-handler": "~2.16.1",
    "react-native-pager-view": "6.3.0",
    "react-native-reanimated": "~3.10.1",
    "react-native-reanimated-carousel": "^3.5.1",
    "react-native-safe-area-context": "4.10.5",
    "react-native-screens": "3.31.1",
    "react-native-shared-group-preferences": "^1.1.24",
    "react-native-svg": "15.2.0",
    "react-native-tab-view": "^3.5.2",
    "react-native-toast-message": "^2.2.0",
    "react-native-uuid": "^2.0.1",
    "react-native-view-shot": "3.8.0",
    "react-native-web": "~0.19.6",
    "react-native-webview": "13.8.6",
    "react-native-youtube-iframe": "^2.3.0"
  },
  "devDependencies": {
    "@babel/core": "^7.20.0",
    "@tamagui/babel-plugin": "1.100.0",
    "babel-plugin-transform-inline-environment-variables": "^0.4.4",
    "esbuild": "^0.20.2",
    "metro-minify-esbuild": "^0.2.0",
    "typescript": "~5.3.3"
  }
}

it should work fine, I'm using it with v51. Have you done a npx expo prebuild --platform ios --clean?

@nhuesmann any luck?

@MaxAst so sorry I didn't see your reply until now. I was able to get it working, but it still doesn't totally make sense why. I had a config file that both the app and share extension use, and that config file imported the expo-updates library. What is strange is that this wasn't causing any issues for the share extension when running on SDK 50. However, in SDK 51, if I changed things around and got rid of all expo-updates related imports, the share extension runs again. Nothing changed in my code, and I do run a clean prebuild before every build I create. So I think there's a potential solution if I rework some code to not have any shared expo-updates imports, but it's just still a little confusing why this would only be a problem on SDK 51.

The pod for expo-updates is automatically removed by expo-share-extension, because the setup for enabling EAS updates would have been a lot of work. So I guess it makes sense that JS that depends on expo-updates would fail when run in the share extension

relevant section in the repo: https://github.com/MaxAst/expo-share-extension/blob/main/plugin/src/withPodfile.ts#L39-L43

will close this for now, lmk if you need further help with this


:wrench: issue #42 fix

What was changed:

Removed #preview-text html code along with associated styles.
Added #canvas styling to achieve the checkered background commonly associated with transparent.
Added a .dark version of the checkered background as well.

Things to know:

The colours of the checkered grade must remain fairly close to each other in terms of shade. When given too much of a contrast, the checkered grade takes on a warped look and is difficult on the eyes.
I am a new FED and this is my first attempt at contributing to open source code, please provide any feedback pertaining to this pr (apologies for not including the commit emoji 🤦🏻).

What was changed:

Restored #preview-text html code along with associated styles in cubfan135
Removed #preview-text html code along with associated styles from Source/cubfan135 (index and style)
Added #canvas styling to achieve the checkered background commonly associated with transparent in Source/cubfan135.style.css
Added a .dark version of the checkered background as well in Source/cubfan135.style.css
Removed previously implemented box-shadow for .dark canvas

What was changed:

Restored #preview-text html code along with associated styles in cubfan135
Removed #preview-text html code along with associated styles from Source/cubfan135 (index and style)
Added #canvas styling to achieve the checkered background commonly associated with transparent in Source/cubfan135.style.css
Added a .dark version of the checkered background as well in Source/cubfan135.style.css
Removed previously implemented box-shadow for .dark canvas


Release-1.2.1-alpha.7

Prepare alpha release

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 70.326%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1157

Relevant Lines:
1550

💛  - Coveralls


Add docs directory

@amityadav0 All the stup steps have been moved to setup.md. That's why I have removed them from template docs.
I'll mention this in the docs too.


Show version number

This would be in the plugin menu and include the version number of the chrome package, so bug reporters do not have to go into the chrome://extensions/ page to find the version.

This is now done! Thanks @ronwnor


Continue fine tuning a model e.g. text-davinci-002

I am only able to define the base model for fine-tuning as ada, babbage, curie or davinci. I would like to be able to (1) fine tune on the official fine tuned models, e.g. text-davinci-002 to allow me to compare my fine tuning to the default latest and greatest. And of lesser importance: (2) continue a fine-tuning on my own models.

Thanks for writing in @eyaler! I'll forward this along to the fine tuning group

This has now been implemented! https://beta.openai.com/docs/guides/fine-tuning/continue-fine-tuning-from-a-fine-tuned-model

Well, (2) has.  Newer models take awhile for us to deriksk, though we roll them out when we can

@hallacy thanks! should i open a new issue for (1)?

I would probably suggest no, but there are reasons for yes:
No: Frankly this repo doesn't have the ability to support this request.  The ability to finetune comes from our API and this repo serves mostly as a wrapper to the API.  I wouldn't be able to take any action on this repo to enable the feature in the future.
Yes: There isn't a publicly facing place for developers to vote on OpenAI feature requests.  Arguably, the best places for making your wishes known to openai are either (and this is the currently preferred method) writing in to<EMAIL_ADDRESS>with your request, in a repo like this (and then having other developers emoji react or +1 or something), or @ing us on social media.
Hopefully that helps.  Let me know if it doesn't


chore: improve "please subscribe" wording

Suggestion: improved the wording from "please subscribe" to "sign up" in the Data Proxy section of the docs.

@ruheni I like that wording change, thanks... do you want to re-commit; it looks like many of the checks failed.

I've redeployed the site. I'm not familiar with Percy, and I'm not sure how to resolve that error.


correct spelling mistake

Coverage remained the same at 92.073% when pulling 01f85ac1d3e677182710fbc5fb411d6ef326e38f on EdwardBetts:spelling into a3688bdfb3b7e945bcd37a929add60ce5f89e5d6 on chaseadamsio:master.


Require tls

Add some static TLS certs for use with testing/local runs

Tests are no longer working -- ginkgo eats up lots of CPU but doesn't do anything

possibly due to TLS not being enabled for http server in test?


Update Ansible Galaxy to 1.2.2

The role on Ansible Galaxy is still set to 1.1.0 and it would be helpful to update it

Hey there,
Ansible Galaxy was recently updated and I was unable to get it to even recognize my projects when it even worked at all. This was something that used to work well, but I will try to get it updated again.

Fixed in version 2.0.6 which installs Consul v1.2.2


nodeid map: provide map via hive cell

This commit introduces a hive cell for the nodeid map.
The datapath cell has been refactored to depend on the nodeid map - which introduces explicit dependencies with a proper init order.
This way, the map is properly initialized when accessing it for nodeid restoration. Currently this is only working due to the fact, that the map gets implicitly opened when iterating over its entries.
Follow up of https://github.com/cilium/cilium/pull/25497

/test

/test

rebased to main and resolved conflicts with signalmap registration

rebased to main and resolved conflicts
@jrajahalme & @nathanjsweet: ready for review

next rebase to main to resolve conflicts :pray:

/test

rebased to main and resolved conflicts.
@nathanjsweet : ready for review

/test


Esp32 not detecting gyro

I have flashed the esp32 firmware to esp32 nodemcu and wired up mpy6050 as per the image on the docs . But in betaflight there is no gyro .
I think the image is for spi connection and mpu6050 is i2c . Where do I connect mpu6050 on esp32 ?

mpu6050 use i2c only..
esp32   | mpu6050
D22      | SCL
D21      | SDA

Thanks it works . I'm using esp 32 wroom . Pid loop is fixed at 1K how to make it 8K

use mpu6500 or mpu9250 support spi

Thanks for the information .
Awesome project . I'm trying to build a brushed flight controller .
On Thu, 27 Jul, 2023, 2:41 pm Auto Plant Bali, @.***>
wrote:

use mpu6500 or mpu9250 support spi
—
Reply to this email directly, view it on GitHub
https://github.com/rtlopez/esp-fc/issues/45#issuecomment-1653213641, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BAOHSJY25WSW4ACVIH7SJQ3XSIWCTANCNFSM6AAAAAA2ZGHXP4
.
You are receiving this because you authored the thread.Message ID:
@.***>

Issues closed

Hi @Shally1986 @AutoPlantBali,
actually not quite, I checked it and I2C does not work on esp32. So I prepared fix and just merged to master branch. If you want to test it, flash device directly from this branch, I'm going to prepare new release soon.
MPU6050 works only on I2C bus.
Also keep in mind that wiring for esp32 might not be up to date, werify your pinout with CLI output command get pin, especially motor output. On ESP32 you can remap most pins according to your needs too.

Post connecting mpu6050 to i2c I'm able to detect gyro and accelerometer in
betaflight .
How do I flash directly from branch ? Do i need to compile ? Could you help
me with the firmware or details on compilation and pin out for pi pico .
Thanks
Kushal Sahota
On Thu, 27 Jul, 2023, 3:34 pm rtlopez, @.***> wrote:

Hi @Shally1986 https://github.com/Shally1986 @AutoPlantBali
https://github.com/AutoPlantBali,
actually not quite, I checked it and I2C does not work on esp32. So I
prepared fix and just merged to master branch. If you want to test it,
flash device directly from this branch, I'm going to prepare new release
soon.
MPU6050 works only on I2C bus.
Also keep in mind that wiring for esp32 might not be up to date, werify
your pinout with CLI output command get pin, especially motor output. On
ESP32 you can remap most pins according to your needs too.
—
Reply to this email directly, view it on GitHub
https://github.com/rtlopez/esp-fc/issues/45#issuecomment-1653304175, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BAOHSJ6JDUXA5S3OY5X4IVLXSI4LFANCNFSM6AAAAAA2ZGHXP4
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Shally1986 I forgot, you can try this build https://github.com/rtlopez/esp-fc/actions/runs/5679139675
RP2040 is experimental, and because of lack FPU, do not perform fast. I have not yet prepared the upload process without the development environment.


Use resolve instead of query

Using resolve instead of query lowers the chance of false-positives due to misconfigured local resolvers and search lists.

This will not work with DNSPython 1.16
>>> import dns.resolver
>>> resolv = dns.resolver.Resolver()
>>> resolv.resolve('github.com')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: 'Resolver' object has no attribute 'resolve'

However, as a workaround, I will configure stub resolver object with empty search list:
resolv.search = []

It will work however with the most up-to-date version of DNSPython (2.0.0):
>>> import dns.resolver
>>> resolv = dns.resolver.Resolver()
>>> resolv.resolve("github.com")
<dns.resolver.Answer object at 0x7f94a94974a8>
>>> print (dns.__version__)
2.0.0
>>>

I'm going to switch once 2.x is widely adopted.

Just tried your workaround and it does not fix my problem with getting false-positives (line 3):
>>> resolv.search = []
>>> resolv.query("doesnotexist.aaa")
<dns.resolver.Answer object at 0x7f94a9497780>
>>> resolv.resolve("doesnotexist.aaa")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.6/dist-packages/dns/resolver.py", line 1030, in resolve
    (request, answer) = resolution.next_request()
  File "/usr/local/lib/python3.6/dist-packages/dns/resolver.py", line 585, in next_request
    responses=self.nxdomain_responses)
dns.resolver.NXDOMAIN: The DNS query name does not exist: doesnotexist.aaa.

Added support with backward compatibility in 05aa0655e25d2e0566b1630195c44a8182a3c273


Link does not open on Chrome

Add target="_blank" and it opens, while the popup closes. This isn't terribly useful because the user might want to open multiple at the same time. Firefox naturally does it well: clicking a link opens in a new tab even if it has no target, and keeps the tab open, until user focus moves out.
According to this SO answer, keeping popup open can be achieved with:
e.preventDefault()
chrome.tabs.create({ url, active: false })

However, this may require the tabs permission.

Fixed in 3e6daecdb1c83dbf1c17a9f5d2eaba4d599bc2b5, uploading to Chrome Web Store.


Provide guidance om how inferno validator works

We need a text narrative that describes how the validator could be used.
Also, we need to preload the IG package into the validator.
Now validator sends a request to localhost that is incorrect, it should use the base URL of the deployed server:

It looks like we don't need to preload IGs into the validator because it already downloads all IGs from "https://packages2.fhir.org/packages/catalog". In this case, AU core profiles are available from "https://packages2.fhir.org/packages/catalog?op=find&name=hl7.fhir.au.core&pkgcanonical=&canonical=&fhirversion=".

It's a guide for users
https://github.com/hl7au/au-fhir-core-inferno/blob/master/docs/validator_instructions.md


Added Google Analytics code in head layout

Added Google Tag Manager script
Added pushing a (Custom) searchTerm Event dataLayer with the search term. This is "sent to" Google Tag Manager when the conditions are met and handled by Google Analytics.

PR closed -> transfered to https://github.com/CoherentLabs/GameUIComponents/pull/93


generativeMountainArt-hephaestusHex

This project generates perspective mountain art. I've always liked drawing perspective art with roads, and with mountains close by to me, I made this art generator. Almost everything is user inputed in this art, except for the mountains, representative of the fact that mountains are works of nature, and humans cannot control it. My experience with coding is limited, as I only know to program in Python , C#, and C++, and my experience with JS is minimal. I enjoyed reading the blot docs to make this art work.

I like these perspective mountain scenes. Nice work using clipping.
 You can order it here.


added KubeCon NA 24 blog post

Added a roundup blog post for KubeCon NA 2024.

http://www-testing-pulumi-docs-origin-pr-13298-79ad64af.s3-website.us-west-2.amazonaws.com/blog/kubecon-na-2024-roundup/

http://www-testing-pulumi-docs-origin-pr-13298-25eeaeb4.s3-website.us-west-2.amazonaws.com/blog/kubecon-na-2024-roundup/

http://www-testing-pulumi-docs-origin-pr-13298-31ccd7e3.s3-website.us-west-2.amazonaws.com/blog/kubecon-na-2024-roundup/

http://www-testing-pulumi-docs-origin-pr-13298-6a16637e.s3-website.us-west-2.amazonaws.com/blog/kubecon-na-2024-roundup/


Unhandled promise rejection

I think this is an issue with unfurl rather than micro-unfurl's service need to double check later so making this so i don't forget.

non issue


Refactor Activators common code and improve its testability

I've created the BaseActivator that encapsulates activator logic.
It solves a use case when the service needs to wait for N other services.
This pattern is used in the config-store and Kafka-CDC-producer.
I think that many client's modules may follow the same pattern.
Let me know wdyt.

@dimas-b @dougwettlaufer I've changed the design to use ServiceTracker - it looks better now.
I've addressed all comments and added missing features. I also migrated GrapqlActivator to a new mechanism. Let me know wdyt.

@dimas-b I've migrated the API to type-safe according to your suggestion. It looks pretty solid now. Please take a look.
Once we merge this PR, I will create a follow-up ticket to migrate all existing Activators to this new API.
It should be done separately because all other activators have no unit or integration tests, so migrating them will require a substantial amount of manual testing. I've migrated only HealthCheckerActivator and GraphqlActivator because they were simple to test, and I wanted to have N use-cases for this new BaseActivator class in this PR.
Once we merge this PR, all new activators should use this new mechanism (I will adapt the CDC and Kafka CDC producer activators)


Add Ania Brajtkopf to Contributors list

@anya92, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?


Use config() instead of env() to prevent caching issues.

It's discouraged to use the env() helper directly in your blade templates. Use config('app.env') == 'production' or app()->environment('production') instead.
When you run php artisan config:cache your env() call will return null.
laravel.com/docs/5.6/configuration#configuration-caching

I actually covered config cache in an article and totally forgot about it. 😂
Thanks for this!


chore(release): v0.8.0 🎉

:robot: I have created a release *beep* *boop*
0.8.0 (2022-11-23)
Features

hooks: allow users to specify an onSet hook, which gets triggered exactly after set is confirmed (be77bf6)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ehmpathy/with-simple-caching/releases/tag/v0.8.0 :sunflower:


fix #159755

Remove views service dependency in composite part
Create ProgressScope for composites by depending on the visibility events from the composite
Create ProgressScope for views by depending on the visibility of the views
ViewService still has to be eager because it registers viewlets and panels in the constructor.

Nice 🚀


Usage with react-router

I'm curious as to the preferred usage with react-router or if a different routing approach is more favorable with baobab.
If I run 0.4.x I can get baobab setup easily using the mixin included with the tree:
//...
var state = require('../state');

module.exports = React.createClass({
  mixins: [state.mixin],

  cursors: {
    users: ['models', 'users']
  },
  //...

When upgrading to 1.0.x I'm unable to get things working with the provided docs. I realize that 1.0.x was just released so I'm not sure if getting this working with react-router has been figured out yet.
Is there an idiomatic way to use baobab-react with react-router?

Hello @danawoodman. Could you describe what is making the repo's mixins fail in your case? The repo is indeed not totally stable right now but will be soon.
I guess the way to use baobab-react and react-router depends highly on whether you want to use mixins or higher order components. If you are using mixins I think you can just use both mixins: baobab-react ones and the router's ones.

Right now I'm getting:
ERROR in ./~/baobab-react/mixins.js
Module not found: Error: Cannot resolve 'file' or 'directory' ./dist-modules/mixins.js in /path/to/code/node_modules/baobab-react

I'm passing the tree down the main <RouteHandler> in react-router

I'm using io.js FWIW

// other imports...
var mixin = require('baobab-react/mixins').branch;

module.exports = React.createClass({
  mixins: [mixin],

  cursors: {
    users: ['models', 'users']
  },

  render() {
    // use `this.state.users`
  }
});

Seems to relate to there being no dist-modules folder which seems to be created as a pre-publish step. Is this on npm yet?

No, the package is not on npm yet but will be very soon (tomorrow or friday at most). The fact that you cannot require the files must indeed come from the package not being built when installing it from git with npm. Can you try to run npm prepublish in the package's folder (probably node_modules/baobab-react) and see whether this can fix your import issue?

Read my mind. Just did that and got a bit further. It seems the issue is related to react-router integration as if I use it with a plain component it works but if I pass down the tree in a <RouteHandler tree={tree} /> it doesn't seem to pass the tree down properly even though I'm passing down {...this.props} in the handler to the child components.

I don't remember whether the RouteHandler component propagates its props to the component it actually refers. But shouldn't you attach the baobab mixin to a component higher than the RouteHandler?

I'll check to see if I can pass them in when the routes are intialized and report back
On Thursday, Apr 23, 2015 at 6:56 AM, Guillaume Plique<EMAIL_ADDRESS>wrote:
I don't remember whether the RouteHandler component propagates its props to the component it actually refers. But shouldn't you attach the baobab mixin to a component higher than the RouteHandler?
—
Reply to this email directly or view it on GitHub.

Sweet, figured it out! I was being silly and needed to pass the tree into the route handler:
var tree = require('./tree');

Router.run(Routes, function (Handler, state) {
  React.render(<Handler tree={tree} />, document.body);
});

Hope this is helpful for others using baobab + react-router. Maybe it warrants an addition into the README @Yomguithereal?
Cheers!

What I did is, in the root element which is rendering the Router.RouteHandler, I set the tree property of the context to my stateTree
import PropTypes from "baobab-react/prop-types";
let stateTree = window.stateTree = require("./stateTree.js");

var App = React.createClass({
    childContextTypes: {
         tree: PropTypes.baobab
    },
    getChildContext() {
        return {
            tree: stateTree
        };
    },
    render() {
        return (

                <Router.RouteHandler/>
                //...

Would someone be kind to post a complete example of how to use baobab-react with react-router? I'm trying to piece these bits together but it's not working for me - really hard to find out what piece is missing in this puzzle... Pretty please? :)

var stateTree = new Baobab({
  stores: {
    whatevers: []
  }
},
{
  shiftReferences: true
});

let App = React.createClass({
    childContextTypes: {
         tree: PropTypes.baobab
    },
    getChildContext() {
        return {
            tree: stateTree //reference to your baobab
        };
    },
    render() {
        //your main component
        return (

                    <FancyComponent />

                <Router.RouteHandler/>

        );
    }
});

let routes = (
    <Route name="app" path="/app" handler={App}>
        <Route name="whatever" path="/app/whatever" handler={WhateverComponent}/>
    </Route>
);

Router.run(
    routes,
    Router.HistoryLocation,
    Handler => {
        React.render(<Handler />, document.body); //or whatever
    }
);

//WhateverComponent.jsx
import {branch} from "baobab-react/higher-order";

let WhateverComponent = React.createClass({
    contextTypes: {
        router: React.PropTypes.func
    },
    render() {
        let list = this.props.whatevers.map((we, i) => {we.foobar});
        return (
{list}
);
    }
});

export default branch(WhateverComponent, {
    cursors: { whatevers: ["stores", "whatevers"] }
});

OMG @egeozcan! It Works! Yay! :) Thank you very much!! Should that be added to the readme file? I'm sure it'd be of great help for people first arriving at this project!

You're welcome. I think this is wiki material but I'm not the one to decide^^

@egeozcan why not pass in the tree to the <Handler/> in Router.run like I had posted? Seems more explicit that way but I guess I'm not sure if the differences, if any.

@danawoodman because it did't work on my machine© :smile: Seriously, the routing got broken after the first successful navigation (clicking on any link worked once, then page url kept on responding but route context not), for some strange reason. Didn't investigate too much, will post an update if I find out why.

Nice one @egeozcan saved me a load of time! This is definitely wiki material.

Will probably have time to rewrite the docs this week end. I'll include this example while scaffolding a little wiki.

I have used the mixin approach mentioned earlier without a hitch.  But now I am wondering how to manage the case where you change tree instances?
If I release the tree, reassign it to a new baobab, and attach new update handlers, the baobab side is fine.  but on the react side, despite the attempt of the root mixin to reassign the childContext it got from the route handler tree prop, the child still seems to hold a ref to the original context tree.  this causes the cursor listeners to be lost in all the child components.
so if you do this, where tree is a baobab:
Global.tree = require('./tree');

Router.run(Routes, function (Handler, state) {
  React.render(<Handler tree={Global.tree} />, document.body);
});

and Routes:
<Route handler={Page}>
  <Route name="home" path="/" handler={Home} />
  <Route name="users" path="users/:id" handler={Users} />
  <NotFoundRoute handler={Oops} />
</Route>

with Page:
var mixin = require('baobab-react/mixins').root;
module.exports = React.createClass({
  contextTypes: {
    router: React.PropTypes.func.isRequired
  },
  mixins: [mixin],
  render() {
      return (
        <RouteHandler />
      );
  }
});

and Users:
var mixin = require('baobab-react/mixins').branch;
module.exports = React.createClass({
  mixins: [mixin],
  cursors: {
    users: ['models', 'users']
  },
  render() {
    // use `this.state.users`
  }
});

then say,
Global.tree.release();
Global.tree = new Baobab(defaults, options);
and then route back to Users
./users/123
...then the Users component nextState remains the same, not changed via the cursor.  And if you try to look at the tree context you get an error:

which I suspect is holding a ref to the old tree/cursors/facets when the component was mounted.
i don't know if there is an issue with the way the child component is setting the tree context, or how the listeners are referenced when the component is updated...after getting a different tree context.
or maybe I'm messing up the way react-router should work with baobab?
thoughts?

Hello @larryhengl. I need to investigate a little bit to see whence this problem comes. I'll open a dedicated issue for this one (#67).
Out of curiosity, why do you need to swap the tree at runtime?

@Yomguithereal  thanks for looking into this!
the idea of swapping trees at runtime comes from an app i'm working on that does db searches, where i want to serve search results as separate trees (or a reset of the single tree used in the app).  this allows for various ui interactions to facet/filter the results and manage active filter state in the ui for the given results. i didn't want to persist the app state across searches nor cache the search data per se - but if i wanted to do that then modeling the tree differently may obviate the need to reset the tree.

I guess that a workaround for now would be to completely change the data of your tree likewise:
tree.set(newData);

right, that does the trick.
somehow i had it in my head that that would bork the facets i had defined, but they were fine (correctly updated).  however, since there is no "silent" set option, all the emitters fired.  so the challenge then was to either turn off the update event handlers (and add them back after the set) or do better null data handling. i did the latter. :-)
thanks for the help, and the lib!

With the new version of react-router I can't pass props to the compononet.
I can't do anymore <Handler tree={Global.tree} />
Now I have
React.render(
  <Router>
    <Route path="/" component={Layout}>
      <IndexRoute component={Landing}/>
    </Route>
  </Router>
  , document.getElementById('content')
);

How can I pass the tree down to the components ?

@manuelmazzuola, with wrappers component you might try something like:
var Root = require('baobab-react/wrappers');

React.render(
  <Root tree={myTree}>
    <Router>
      <Route path="/" component={Layout}>
        <IndexRoute component={Landing}/>
      </Route>
    </Router>
  </Root>
  , document.getElementById('content')
);

Or create a component wrapping the router or some part of it and root it with mixins or higher-order components.

@manuelmazzuola : i tried your method however i'm getting the following error:
Warning: React.createElement: type should not be null, undefined, boolean, or number. It should be a string (for DOM elements) or a ReactClass (for composite components).warning @ bundle.js:27495ReactElementValidator.createElement @ bundle.js:181482.baobab @ bundle.js:168s @ bundle.js:1e @ bundle.js:1(anonymous function) @ bundle.js:1
bundle.js:27051 Uncaught Error: Invariant Violation: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: object.

I've resolved with
const createBaobabComponent = (Component, props) => {
  return <Component {...props} tree={tree} />
};

React.render(
  <Router createElement={createBaobabComponent}>
    <Route path="/" component={Layout}>
    </Route>
  </Router>
  , document.getElementById('content')
);

@manuelmazzuola you are a beautiful person, thank you so! :+1:


resize page to fit svg image

issue #344
Signed-off-by: splig<EMAIL_ADDRESS>

@scary4cat Thank you!


Dockerfile: download URL for maven is outdated

In Dockerfile, the maven version is 3.6.0. But in https://apache.osuosl.org/maven/maven-3 , it only has 3.6.1/3.6.2/3.6.3. How about use version 3.6.3 instead?

@vagetablechicken
Could you please verify it and bump it up to 3.6.3 if it works fun?

OK，I'll fix it


Add to README how to avoid abuse

Hi again,
Would you be willing to add to the readme what would be the best way to avoid abuse ?
I think it would be a great addition for anyone how figuring things out (including myself)
Thank you

I have added a Security section to README which gives somes options how to avoid abuse of the lambda function.