Bug: Infracost provider fails to find resources when resource name contains “module.***”

Need to reproduce this first


Error 404: The page you were trying to access doesn't exist or has been removed.

I have upgraded from 3.4.7-1 to 4.0.3-1 and everything works well except users page. when i try to edit user getting 404 error with below error message. could you please help me to solve this.
"The page you were trying to access doesn't exist or has been removed."

Missing template, responding with 404: Missing partial users/_user_color_form with {:locale=>[:en], :formats=>[:html], :variants=>[], :handlers=>[:raw, :erb, :html, :builder, :ruby, :rsb]}. Searched in:

"/home/redmine/redmine/plugins/redmineup_tags/app/views"
"/home/redmine/redmine/plugins/redmine_work_time/app/views"
"/home/redmine/redmine/plugins/redmine_percent_done/app/views"
"/home/redmine/redmine/plugins/redmine_mentions/app/views"
"/home/redmine/redmine/plugins/redmine_hearts/app/views"
"/home/redmine/redmine/plugins/redmine_checklists/app/views"
"/home/redmine/redmine/plugins/redmine_agile/app/views"
"/home/redmine/redmine/plugins/easy_wbs/app/views"
"/home/redmine/redmine/plugins/easy_mindmup/app/views"
"/home/redmine/redmine/app/views"
"/home/redmine/data/tmp/bundle/ruby/2.4.0/gems/redmine_extensions-0.3.00/app/views"

Rendering common/error.html.erb within layouts/base
Rendered common/error.html.erb within layouts/base (0.5ms)
Rendered plugins/redmineup_tags/app/views/tags/_additional_assets.html.erb (0.5ms)
Rendered plugins/redmineup_tags/app/views/tags/_select2_transformation_rules.html.erb (0.1ms)
Completed 404 Not Found in 205ms (Views: 13.4ms | ActiveRecord: 17.9ms)

This is likely a redmine plugin issue with the newer version of redmine.  When upgrading redmine versions, make sure the new version is supported by all your plugins.
https://www.redmineup.com/questions/216-agile-plugin-not-working

yes, you are right. it was caused by agile plugin. everything normal after agile plugin removed. thank you.


Airflow Submission Example via API (Documentation)

Add documentation for submitting a job to airflow via APIs to Gitbook
Acceptance criteria required to implement the epic

[ ] Gitbook page with explanation and example of submitting a job to Airflow via OGC API is available for Users
[ ] Add information to MDPS Tutorial

No gherkin validation is needed for this feature. Ticket should point to the documentation when complete.

System Validation Criteria: a non-SPS member (Godwin, Brian, Mike or Jimmie) is able to follow the documentation to submit jobs via the OGC (and Airflow?) API.

@rtapella please provide a status when you get back

I think we have this but I don't know if it's linked. Will look as I update in January


Provide frees-puretest module which includes support for testing @free and @tagless algebras

See #388 and https://github.com/hablapps/freestyle/pull/1

This module is going to be provided in the puretest hablapps org as puretest-frees or in puretest-cats if it can be abstracted away from freestyle per comment https://github.com/hablapps/freestyle/pull/1#issuecomment-347151326


[ENG-4187] Test file detail view section

Ticket: https://openscience.atlassian.net/browse/ENG-4187
Feature flag: feature/file-metadata-mirage-tests

Purpose
The purpose of these changes is to add tests for the ember-osf-web portion of the File Metadata project.
Summary of Changes
Tests were added to the registration-file-detail-test.ts file to verify the functionality of metadata label and div elements, the edit metadata form and its save and cancel buttons, and the mobile default view for the application.
Screenshot(s)

Side Effects
There should be no unintended side effects.
QA Notes
-Are all added components and functionalities properly tested?
-Are there any suggested tests not already covered?
-Does the default view change for tablet view? From tablet to mobile, mobile to desktop and vice versa?
-Are all elements properly rendered on the screen? Does the edit metadata button remain hidden when permissions change?

Aside from the two broken tests because of a missing data-test selector, I think the big thing that's missing is testing the functionality of changing the metadata and verifying that the new metadata is populating the view. Right now, you are, for the most part, just testing that the labels are there. It might be worth looking through the ticket description again to see if there are other suggested tests to include.

Ok. I had originally written a test that changed the title textarea value that had required the canUserEdit boolean field for the edit metadata button. I had not written any more until that passed, but can follow up with all the fields.

Aside from the two broken tests because of a missing data-test selector, I think the big thing that's missing is testing the functionality of changing the metadata and verifying that the new metadata is populating the view. Right now, you are, for the most part, just testing that the labels are there. It might be worth looking through the ticket description again to see if there are other suggested tests to include.

Ok. I had originally written a test that changed the title textarea value that had required the canUserEdit boolean field for the edit metadata button. I had not written any more until that passed, but can follow up with all the fields.

Aside from the two broken tests because of a missing data-test selector, I think the big thing that's missing is testing the functionality of changing the metadata and verifying that the new metadata is populating the view. Right now, you are, for the most part, just testing that the labels are there. It might be worth looking through the ticket description again to see if there are other suggested tests to include.

Ok. I had originally written a test that changed the title textarea value that had required the canUserEdit boolean field for the edit metadata button. I had not written any more until that passed, but can follow up with all the fields.

Tests have now been added that include updating the title change for the metadata edit form and adding verifications for changing the description, resource type and language. I have also sectioned off A11y tests as well as the moved the cancel and edit buttons’.

There's a few specific items, such as not to use random values in tests. In general, you'll want to make sure you're verifying all of the fields are what they say they are in read-only and write modes. There's a test file referenced in the review (tests/acceptance/guid-node/metadata-test.ts); take a look at that, what it's testing and how, and model your tests on that.

Thank you Brian. Tests are now updated, including the selectChoose over the Math random selector. I have also recently taken care of merge conflicts. When available, I would appreciate your second review.

Hi, @chth0n1x. You'll need to make a new pr with this branch pointed to develop. It was automatically closed when we merged the feature branch.


Endless SAML redirect

If you get an endless SAML redirect,

try running docker stop sunet-nc2; docker start sunet-nc2 on the host system,
check docker logs -f sunet-nc2 to look for errors.
Maybe you forgot to do the second login in a private browsing tab,
Did you type http:// instead of https://?

With these checks the problem can be avoided.


Persist and load embedding configuration changes using DynamoDB

Background
Link to issue
https://biomage.atlassian.net/browse/BIOMAGE-450
Link to staging deployment URL
TBD
Links to any Pull Requests related to this
InfraMock PR (tbd)
Anything else the reviewers should know about the changes here
N/A
Changes
Code changes
Added functionality for loading and saving experiment processing settings to/from DynamoDB.
Definition of DONE
Your changes will be ready for merging after each of the steps below have been completed:
Testing

[ ] Unit tests written
[x] Tested locally with Inframock
[ ] Deployed to staging

To set up easy local testing with inframock, follow the instructions here: https://github.com/biomage-ltd/inframock
To deploy to the staging environment, follow the instructions here: https://github.com/biomage-ltd/biomage-utils
Documentation updates
Is all relevant documentation updated to reflect the proposed changes in this PR?

[x] Relevant Github READMEs updated
[x] Relevant wiki pages created/updated

Approvers

[ ] Approved by a member of the core engineering team
[ ] (UX changes) Approved by vickymorrison (this is her username, tag her if you need approval)

Just before merging:

[ ] After the PR is approved, the unstage script in here: https://github.com/biomage-ltd/biomage-utils is executed. This script cleans up your deployment to staging

Optional

[x] Photo of a cute animal attached to this PR

looks good. I like the 1.5s debounce stuff that, if I understand correctly, stops every user input/keypress from trying to save immediately

looks good. I like the 1.5s debounce stuff that, if I understand correctly, stops every user input/keypress from trying to save immediately

looks good. I like the 1.5s debounce stuff that, if I understand correctly, stops every user input/keypress from trying to save immediately

Correct. The debounce makes sure that the function wrapped in the debounce is not called until a certain amount of time has elapsed since the last time the debounce was hit.

looks good. I like the 1.5s debounce stuff that, if I understand correctly, stops every user input/keypress from trying to save immediately

Correct. The debounce makes sure that the function wrapped in the debounce is not called until a certain amount of time has elapsed since the last time the debounce was hit.

Staging available at https://ui-tup1zojf7s7jzginx8tzmtq1zo.scp-staging.biomage.net/.

Staging available at https://ui-tup1zojf7s7jzginx8tzmtq1zo.scp-staging.biomage.net/.


dataEmptyMessageFontSize doesn't exist

Hello,
I'm using fusionchart for a view project and I followed the fusioncharts documentation for the properties you can add and the "dataEmptyMessageFontSize" property doesn't work, so I go to your project and when I see in the configuration file, the dataEmptyMessageFontSize property is not present.
Is this normal?

@tibdev78 You can add the config here  : https://github.com/fusioncharts/vue-fusioncharts/blob/master/src/config.js
If you install the component locate vue-fusioncharts/src/config.js in node modules and add the option

@AyanBhadury Do you want me to clone the project, add options to the configuration file and make a merge request?

Ah finally what I was looking at was the chart attributes and not the chart object properties
dataEmptyMessageFontSize not exist in the object properties .
Sorry.


Add climate projections to dashboard

The new files from ncar are in nc format. Do we need to convert to csv?
Files are here: https://github.com/dotis/FGBNMS_DBv3/tree/main/data

Example loading of .nc files into R

You can also read from NetCDF directly using the raster or preferred newer terra R package.

4.6 Working with netCDFs | R as GIS for Economists

This is because it uses the underlying GDAL drive for NetCDF:

NetCDF: Network Common Data Form — GDAL documentation


Fix np.fmod to match numpy behavior

Fix for np.fmod so that np.fmod with negative numbers matches numpy's behavior

Need to update to use icmp_signed (was initially tested on numba 0.55)

@benwilliamgraham thanks for the patch. Please could open an issue that demonstrates the bug that this is fixing (assuming there isn't one already open, if there is, please could you reference it)? Many thanks!

Sounds good, here's a new issue discussing it: https://github.com/numba/numba/issues/8574

Hi @benwilliamgraham, thanks for providing a test case. Can you update your patch to include this test case?

I think this needs addressing:
from numba import njit
import numpy as np

@njit
def foo(x, y):
    return np.fmod(x, y)

a = np.uint32(20)
b = np.uint32(-1)
print(f"a = {a}, b = {b}")
print("got", foo(a, b))
print("exp", foo.py_func(a, b))

which currently gives:
a = 20, b =<PHONE_NUMBER>
got 0
exp 20

The ufunc_db[np.fmod] table on numba/np/ufunc_db.py needs update as the uint* should be mapped now to npyfuncs.np_int_urem_impl

The ufunc_db[np.fmod] table on numba/np/ufunc_db.py needs update as the uint* should be mapped now to npyfuncs.np_int_urem_impl

Given the tests didn't catch this, do they need an update?

@benwilliamgraham, can you add a couple of tests for the test case reported by Stuart?

Hi @benwilliamgraham, I'm happy to continue this work if needed. Just let me know.


Support for custom antenna scopes

@ShotgunNinja This PR allows mods and users to declare custom, fixed-range, scopes for antennae.
I really appreciate the fact that Kerbalism will automatically adapt to any given system and that works very well, but in my case I want to play a 6.4x rescale with OPM with Ven's Stock Revamp antennae. Between Ven and Stock there are 8 antennae that I want to give very specific ranges to.
Basically this allows you can declare a MM patch like so:
KERBALISM
{
    ANTENNA
    {
        SCOPE
        {
            name = kilo
            value = 1e3
        }

        SCOPE
        {
            name = mega
            value = 1e6
        }

        SCOPE
        {
            name = giga
            value = 1e9
        }

        SCOPE
        {
            name = tera
            value = 1e12
        }

        SCOPE
        {
            name = peta
            value = 1e15
        }
    }
}

@PART[shortAntenna]:NEEDS[Kerbalism]:AFTER[Kerbalism]
{
    @MODULE[Antenna]
    {
        %scope = kilo
    }
}

@PART[longAntenna]:NEEDS[Kerbalism]:AFTER[Kerbalism]
{
    @MODULE[Antenna]
    {
        %scope = mega
    }
}

@PART[mediumDishAntenna]:NEEDS[Kerbalism]:AFTER[Kerbalism]
{
    @MODULE[Antenna]
    {
        %scope = giga
    }
}

@PART[commDish]:NEEDS[Kerbalism]:AFTER[Kerbalism]
{
    @MODULE[Antenna]
    {
        %scope = tera
    }
}

@PART[HighGainAntenna]:NEEDS[Kerbalism]:AFTER[Kerbalism]
{
    @MODULE[Antenna]
    {
        %scope = peta
    }
}

I've chosen to use GameDatabase rather than parsing the Settings file directly so that other mods could potentially add scopes. (You may consider using GameDatabase for general settings parsing so that mods can alter settings, or users can write MM patches against it without having to edit Settings.cfg and potentially forgetting to merge their changes with every update).
The one "downside" to this is that the ranges have to be fixed expressions and can't dynamically scale with the system.
As a bonus, I've modified the Range() method so that if the scope isn't found in range_values it returns 0 rather than throwing an exception.
On a related note I noticed that the code in the repository as it exists now contains the following in Lib.cs:
static System.Random rng = new System.Random((int)DateTime.Now.Ticks);

That throws an immediate OverflowException since DateTime.Now.Ticks is well over the maximum value for Int32. It's unnecessary too since just calling Random() uses a seed based on the current time anyway. I assume the released builds of Kerbalism don't contain this code, otherwise nothing would work. But I can easily see this frustrating potentially contributors; I know it took me a bit to figure out what the problem trying to get Kerbalism to run with my builds.

This is good. I'm going to merge manually, as you may have figured out I'm using github only as an archive...
I'm already using GameDatabase for the settings, it is somehow buried in the Lib.ParseConfig function.
For the random seed... I'll use the default constructor (and you are absolutely right) The strange thing is that there is the same code in production but it is not throwing an overflow exception.
Finally I want to take the chance to thank you for splitting the CKAN package in two, that was very appreciated and will open the road to third party profiles, not to mention those that only want a backround simulation of resources without the rest.

I'm already using GameDatabase for the settings, it is somehow buried in the Lib.ParseConfig function.

Ah, didn't notice, I guess if I want to write a MM patch against it I would have to target the Settings node, i.e.
@Settings
{
    %MessageLength = 1000
}

At least until MM has some way of targeting a specific path/file. You may want to consider "namespacing" the Settings node under a Kerbalism or KERBALISM node (I went with KERBALISM in the PR to basically keep with Squad's convention for node names).

For the random seed... I'll use the default constructor (and you are absolutely right) The strange thing is that there is the same code in production but it is not throwing an overflow exception.

I suppose the releases are built in Release mode, not Debug mode? OverflowException probably isn't thrown in Release mode.

Finally I want to take the chance to thank you for splitting the CKAN package in two, that was very appreciated and will open the road to third party profiles, not to mention those that only want a backround simulation of resources without the rest.

No problem, and when KSP-CKAN/CKAN#1728 is merged we should be able to support the other profiles that are shipped in the archive.

Good point on Settings, I'll rename it to 'Kerbalism'.
I merged this (in my local version), with some small changes and an addition.
The scopes are definited like this, in the global namespace:
AntennaScope
{
  name = my_custom_scope
  range = 1e13
}

And it is also possible to redefine the default scopes, like this:
AntennaScope
{
  name = orbit
  range = 1e6
}

About CKAN packages for the other profiles... man that is just great! Thanks a lot.

This is in the new release now. Thanks again.


Added LTV & DSR rules to example project

Added lendersCanServiceLoanBasedOnLTV and applicantSatisfiesMonthlyLoanCommitments rules to example project

All good.


Documentation for Response.asgi_send(send) method

I found myself wanting to use this method for https://github.com/simonw/datasette-auth-passwords/issues/15 - but it's not documented. It should be documented.
https://github.com/simonw/datasette/blob/8e18c7943181f228ce5ebcea48deb59ce50bee1f/datasette/utils/asgi.py#L320-L340

https://docs.datasette.io/en/latest/internals.html#returning-a-response-with-asgi-send-send


add Apache 2.0 license to jindo.go

Ⅰ. Describe what this PR does
This PR is to add Apache 2.0 license
Ⅱ. Does this pull request fix one issue?

fixes #1191
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

This PR is to add Apache 2.0 license to pkg/common/jindo.go

This PR is to add Apache 2.0 license to pkg/common/jindo.go

This PR is to add Apache 2.0 license to pkg/common/jindo.go

Please fix the DCO warning.

My guess is that because I made my email private, DCO cannot know about<EMAIL_ADDRESS>


Conversion(Chebyshev(),Ultraspherical{2}()) should return a ConversionWrapper not a TimesOperator

I guess this is somehow related:
julia> using ApproxFun

julia> u = Fun(exp)
Fun([1.26607,1.13032,0.271495,0.0443368,0.00547424,0.000542926,4.49773e-5,3.19844e-6,1.99212e-7,1.10368e-8,5.5059e-10,2.49796e-11,1.03908e-12,3.98354e-14],Chebyshev(【-1.0,1.0】))

julia> D = Derivative()
Derivative{UnsetSpace,Int64,UnsetNumber}(UnsetSpace(),1)

julia> norm(D^3*u)
ERROR: assertion failed: domainspace(ops[k]) == AnySpace() || rangespace(ops[k + 1]) == AnySpace() || domainspace(ops[k]) == rangespace(ops[k + 1])
 in TimesOperator at /Users/Mikael/.julia/v0.3/Approxfun/src/Operators/algebra.jl:231
 in TimesOperator at /Users/Mikael/.julia/v0.3/Approxfun/src/Operators/algebra.jl:260
 in Conversion at /Users/Mikael/.julia/v0.3/Approxfun/src/Spaces/Ultraspherical/UltrasphericalOperators.jl:251
 in defaultcoefficients at /Users/Mikael/.julia/v0.3/Approxfun/src/Fun/FunctionSpace.jl:277
 in coefficients at /Users/Mikael/.julia/v0.3/Approxfun/src/Fun/Fun.jl:32
 in first at /Users/Mikael/.julia/v0.3/Approxfun/src/Spaces/Ultraspherical/Ultraspherical.jl:37
 in cumsum at /Users/Mikael/.julia/v0.3/Approxfun/src/Fun/Fun.jl:253
 in sum at /Users/Mikael/.julia/v0.3/Approxfun/src/Fun/Fun.jl:305
 in norm at /Users/Mikael/.julia/v0.3/Approxfun/src/Fun/Fun.jl:235

This second bug should be fixed now, it was a bug caused by coefficients(f,Chebyshev) not choosing the domain correctly

This has been changed.  Multiplying two ConversionWrappers that Wrap a TimesOperator does the right thing and unraps before multiplying.


Status by repo_url

Every time we add a new project to https://landscape.cncf.io/ we check if there is a best practices badge. That currently requires fetching 55 pages of JSON to see if the project exists.
If we instead added a new API point to allow lookups by repo_url we could fetch the info in a single update.
We do a nightly update where we check whether any of the projects we're tracking have changed. For that, we probably want to download all 55 pages of JSON. But for adding a new project, a new API point would be helpful.
Cc @zeusthetruegod

We already support that, just use pq as the query parameter. For example:
https://bestpractices.coreinfrastructure.org/en/projects?pq=https%3A%2F%2Fgithub.com%2Fcoreinfrastructure%2Fbest-practices-badge
This is documented in the API documentation, so perhaps it could be clearer:
https://github.com/coreinfrastructure/best-practices-badge/blob/master/doc/api.md

OK, looks good, thanks.


Add support for deleting annotations

One can create annotations within the build using buildkite-agent annotate but there is no way to delete a annotation.
Use case:
I make use of this to show a summary of failed tests but I'd like to delete the annotation once I retry that step and it passes with success.

Thanks for this @queimadus, we're tracking feature requests for Buildkite over at https://forum.buildkite.community/c/feature-requests, if this is something you still run into, could I trouble you to create a thread there?


[RFC] - Release process changes for Minor / Patch release

Changes:

Minor releases have a branch cut from "master", ie: 0.9.
Patch releases are applied to the minor release branch.
Release_Process.md describes both processes separately.

https://github.com/bigchaindb/bigchaindb/blob/release-process-updates/Release_Process.md

Current coverage is 96.65% (diff: 100%)

Merging #1110 into master will not change coverage

@@             master      #1110   diff @@
==========================================
  Files            49         49
  Lines          2184       2184
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           2111       2111
  Misses           73         73
  Partials          0          0

Powered by Codecov. Last update 509b590...328a8a8


Red box is jarring and lacks context

As I'm typing code, I often end up in a state where it doesn't run correctly. But the experience is really jarring as the entire window moves, I lose context on the visual thing I was changing.
Also, the redbox is very uninformative, it says the error message but doesn't have a stack trace nor line number.

Suggestion:

Overlay the error on-top of the ui, the same way the redbox works on react native
Display the stack trace, not only the error message
Display the error in the console as well

For it to work great, I would recommend removing run on edit and instead add a reload button. Otherwise you're going to keep having red boxes as you are typing code.

Thanks @vjeux, I agree about the lack of context. We do show the line numbers for "build" errors that we get from Babel, but the runtime frame errors need to be handled better. What version of Chrome are you using? The error bar for me is under the toolbar.

https://github.com/RinconStrategies/react-run/pull/19 Addresses this finally and covers the output similar to react native.


[Details to Grid View Converter] Conversion enabled for theme but the right panel stays very narrow

Extension name
Details to Grid View Converter
Bug Description
I installed Details to Grid View Converter with DH Dawn, Helium, Nova X and enabled them in the addon settings. However, after I restarted the right panel in Grid view remained really narrow.
To Reproduce
No response
Logs
playnite.log
extensions.log
Screenshots

This is not controlled by the extension, you need to increase its width in Playnite settings here:


Extend audio tets

Detect sound card and audio sink separately.
Currently our detection methods might be different for Linux and Windows. If there is no actual oputput connected, we may have PASS for Linux (we check for sound card presence only), while at the same time we get FAIL for Windows (maybe we check just for sink, or maybe Windows is different in this matter?)
This may be often confusing, and suggesting windows drivers might be missing, while in fact there is no audio device connected.
Detecting sound card and audio sink in two different test cases, would help to isolate that.

Related: https://github.com/Dasharo/open-source-firmware-validation/issues/593
Commit: 15560f19cde19b4a2d841a26fd196018ff3272ac changes detection on windows side to detect wider spectrum of devices, sources, and sinks, which could be used as a base for separated detection (filtering speakers from audio cards or virtual sound devices)


misleading config variable names

URLs, and even URL References, are very different from Posix/OS Paths. Paths can contain space characters while URI paths can't, if this is a not a URL it should not have url in the variable name:
https://github.com/coopnorge/go-datadog-lib/blob/2e629313203952e41f1b0644099936fa740638b7/config/config.go#L16-L17

For previous discussion:

https://github.com/coopnorge/go-datadog-lib/issues/17

@aucampia That request affects general configuration inside Coop, this repository is just affected by this setup but in the scope of pckg not belong to that refactor. Consumers of pkg can re-implement the configuration.
Move this issue where the cloud platform configured Agent for Datadog otherwise it can be closed.

Where is the documentation on the platform side. I could not find it in the Observability platform onboarding guide: https://inventory.internal.coop/docs/default/component/coop-playbook/platforms/cloud_platform/observability/guide_how_to_onboard_datadog/
As @aucampia states the config variable name is miss leading, the issue should stay open in this library even if the library is the cause.
@coopnorge/cloud-platform do you know anything about dd_trace_agent_url?

@nhhagen https://inventory.internal.coop/docs/default/component/coop-playbook/platforms/cloud_platform/observability/guide_how_to_onboard_datadog/#endpoints-for-metrics-and-traces

DD_TRACE_AGENT_URL is originates from gopkg.in/DataDog/dd-trace-go.v1 and required to be set for proper operations.


About HiveType and its typeId

I felt ambiguous with the explanation provided on the docs, should the typeId of @HiveType be unique per model? Or I could do it like this, for instance:
@HiveType(typeId: 0)
class ClassA extends HiveObject {}

@HiveType(typeId: 0)
class ClassB extends HiveObject {}

@HiveType(typeId: 0)
class ClassC extends HiveObject {}

Or it should be like this?
@HiveType(typeId: 0)
class ClassA extends HiveObject {}

@HiveType(typeId: 1)
class ClassB extends HiveObject {}

@HiveType(typeId: 2)
class ClassC extends HiveObject {}

Version

Platform: iOS, Android
Flutter version: 1.22.5
Hive version: ^1.4.4+1

It must be unique! 😁

Hi @mbaumgartenbr ! Thanks for answering, so the properway is like the 2nd snippet I defined above?

Yep 🙂

I see.. Thanks for fast response, have a nice day! I'm closing it. 🐱🏍

Link -> https://docs.hivedb.dev/#/custom-objects/type_adapters?id=register-adapter
"Every type has a unique typeId which is used to find the correct adapter when a value is brought back from disk. All typeIds between 0 and 223 are allowed."

This means I can have a maximum of 223 dart model classes ?
I have to remember what's the last Type ID I used in my app, and increment accordingly ?
If my app need more than 223 Model Classes then what should I do ?

I assume it's because it generates code like this:
  @override
  void write(BinaryWriter writer, NewsModel obj) {
    writer
      ..writeByte(5)
      ..writeByte(0)
      ..write(obj.id)
      ..writeByte(1)
      ..write(obj.title)
      ..writeByte(2)
      ..write(obj.content)
      ..writeByte(3)
      ..write(obj.categories)
      ..writeByte(4)
      ..write(obj.date);
  }

This allows the plugin to regenerate the same order of writes, no matter the order in the source code.


Ionic generate tabs generates wrong boilerplate

What version of the CLI are you using? Output of ionic --version
3.0.0-beta.4
Short description of the problem:
After generating tabs I found that the name of the TabPages was wrong in the Tab root page. The tabs are named NameTabPage, where in the Tab root page they are being imported as NameTab.
What behavior are you expecting?
Either the tabs should be called NameTab or NameTabPage and so accordingly in the boilerplate code.
Steps to reproduce:
When generating tabs, using ionic generate, then choosing a name, using 2 tabs the "Generating tabs name here" is displayed over the questions on how you want to name the tabs. "Enter the first tab name:"
Post the output of ionic info below please
Cordova CLI: 6.5.0
Ionic Framework Version: 2.2.0
Ionic CLI Version: 3.0.0-beta.4
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Linux 4.4
Node Version: v7.3.0
Xcode version: Not installed

There have been significant changes to how generate works.  This is now only available for Ionic-angular 3+ projects. I am going to close this ticket because of these changes.


Support colorful output

We need to try this solution:
https://github.com/Lakr233/XTerminalUI

In addition to colorful output, we will also get input support.


Issue on page /notebooks/01-Unix.html

For grep exercise
wget https://raw.githubusercontent.com/mhasoba/TheMulQuaBio/master/data/spawannxs.txt #Cool!
gives
--2020-10-05 13:47:39--  https://raw.githubusercontent.com/mhasoba/TheMulQuaBio/master/data/spawannxs.txt Resolving raw.githubusercontent.com (raw.githubusercontent.com)... <IP_ADDRESS> Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|<IP_ADDRESS>|:443... connected. HTTP request sent, awaiting response... 404 Not Found 2020-10-05 13:47:39 ERROR 404: Not Found.

Needs to be changed to https://raw.githubusercontent.com/mhasoba/TheMulQuaBio/master/content/data/spawannxs.txt


The confirmation macro always reports an error if the schema is defined with type_specs

Example without and with type specs:
schema1 = Dry::Validation.Form do
  required(:pin).filled(:str?).confirmation
end

schema1.call(pin: "1234").errors
#=> {:pin_confirmation=>["must be equal to 1234"]}

schema1.call(pin: "1234", pin_confirmation: "1234").errors
#=> {}

schema2 = Dry::Validation.Form do
  configure { config.type_specs = true }
  required(:pin, :string).filled.confirmation
end

schema2.call({pin: "1234"}).errors
#=> {:pin_confirmation=>["must be equal to 1234"]}

schema2.call(pin: "1234", pin_confirmation: "1234").errors
#=> {:pin_confirmation=>["must be equal to 1234"]}

Closing because confirmation is going away in the 1.0.0 rewrite


Drop pointer for poller id fields

What changed?
Drop pointer for poller id fields within internal types.

Why?
This is one of PRs planned to drop pointers for always required fields in internal types. It will make them more idiomatic and easier to use. These pointer are legacy of Thrift types that were used previously through our codebase. This will also allow safer migration to grpc, as proto primitive types don't have nils by default.

How did you test it?
Passing build and existing tests.

Potential risks

Coverage increased (+0.06%) to 61.566% when pulling 9240ee5dbde2eef05f91eea1682fa63fa58014f4 on vytautas-karpavicius:drop-ptr-poller-id into 79b8acf23cee35c7b0a610084b3eb00346fc1e96 on uber:master.

Coverage increased (+0.06%) to 61.566% when pulling 9240ee5dbde2eef05f91eea1682fa63fa58014f4 on vytautas-karpavicius:drop-ptr-poller-id into 79b8acf23cee35c7b0a610084b3eb00346fc1e96 on uber:master.


feat: process plant data to create data frame

Pull Request doc
Purpose
Parse HIFLD data into the beginning of a plant data frame, for use with a Grid object.
What the code is doing
build_plant is the entry-point function, which will eventually return a data frame with all information, and all lower-level functions are accessed from this function. build_plant effectively replaces the old plant_agg function, with different structure. The lower level functions are:

map_generator_to_sub_by_location: look up substations within that interconnect and ZIP code, or if there are none, look within that interconnect and 'nearby' ZIP codes (+- 100). The logic of 'nearby' could probably be improved, but this is what was implemented in the collaborators' code.
map_generator_to_bus_by_sub: pick the lowest voltage bus within a specified substation.
add_locations_to_generators: use the locations from Power_Plants.csv, map them to the generating units as best as possible.
map_generators_to_interconnects: look up the county for each substation using the Power Plant mapping, then determine the interconnect.
add_pmin_to_generators: look up Pmin values from EIA Form 860.
helpers.map_state_and_county_to_interconnect: look up information from const.py to map a state/county to an interconnection. We also call this from data_process.transmission so that we can access this information for substations from within map_generator_to_sub_by_location.

There are also placeholders for:

estimate_heat_rate_from_ampd
add_heat_rate_to_generators

These will need further research into the best way to implement, but they are independent and so can wait for a follow-up PR.
Usage Example
>>> from prereise.gather.griddata.hifld.data_process.transmission import (
...     build_transmission,
...     create_buses,
... )
>>> from prereise.gather.griddata.hifld.data_process.generators import build_plant
>>> lines, substations = build_transmission()
>>> bus = create_buses(lines)
>>> plant = build_plant(bus, substations)
>>> plant.isna().sum()
ID                0
NAME              0
CITY              0
STATE             0
ZIP               0
TYPE              0
STATUS            0
COUNTRY           0
NAICS_CODE        0
NAICS_DESC        0
SOURCE            0
SOURCEDATE        0
PLANT             0
PLANT_CODE       86
OPERATOR          0
OPERAT_ID         0
NAMPLT_CAP        0
SUMMER_CAP        0
WINTER_CAP        0
PRIM_FUEL         0
FUEL1_TRAN        0
SEC_FUEL          0
FUEL2_TRAN        0
PRIM_MVR          0
RUN_DATE          0
RET_DATE          0
lat             402
lon             402
interconnect    402
sub_id          687
bus_id          687
Pmin              0
dtype: int64

Not all plant codes can be mapped successfully, and same with lats/lons, and therefore sub_id & bus_id, but we're only missing about 3%. Suggestions for improvement welcome.
Time to review
1 hour.

This branch is now ready for a full review. The original post has been updated with more details.

Following some discussion in #222, I'm going to refactor this to use all EIA source data for generators rather than the current HIFLD/EIA mix.


Remove and replace boost usage

This pr removes all uses of boost and replaces them with either c++11 alternatives.
In the case of boost::filesystem, it is replaced with standard library calls to std::filesystem when it is available, or it emulates the subset of std::filesystem required when it is not.
For now this pr includes some of the style commits that are in currently open pr's. I will rebase it when those are merged.
There are still a few things I need to test out before I put this in review, but I'm opening it to get CI going and for visibility.
Also, this removes the version of the function that returns a boost::shared_ptr without deprecating it first. I'm not sure if we can do that, but in other places in ROS we've replaced this and just made a typedef for people to use to ease the migration, though it would cause everyone to update their code. I guess it's also possible to retroactively deprecate it in Lunar to give some warning that it will be gone in Melodic, though I'm open to suggestion about how to do that.

There looks like a special so is require to be linked for experimental filesystem to work on Linux. I'll look into it soon.

For a breaking change like this it might be worth posting the idea to discourse for feedback from the community. Also if we break API like this it might make sense to consider doing that in a few packages at once rather then have similar breaking changes over the next few ROS distros.

For a breaking change like this it might be worth posting the idea to discourse for feedback from the community. Also if we break API like this it might make sense to consider doing that in a few packages at once rather then have similar breaking changes over the next few ROS distros.

Yes this PR is a proof of concept, we are mostly waiting on the REP3 PR to be out for a more global discussion of allowing getting rid of boost or not in the core packages. I think it will be better to gather feedback rather than package-specific threads on discourse. Based on the outcome of REP3 for melodic, we would either use this only on the ros2 branch, use a typedef and update downstream packages accordingly or actually switch to std::shared_ptr everywhere. In the meantime this work is valuable for the ROS2 packages that will need to use pluginlib.
Another alternative if we decide not to change API would be to not get rid of boost only for internal implementation

I have a more complete proposal for how to make boost "optional" and to also rename the headers but leave the current name too, but deprecated. It's a bit more scope than "remove boost", so we can either decide to take this and fixup melodic-devel to be API stable (with new deprecations) in follow up pr's, or I can roll this change into a large pr which removes boost and implements the other changes and backward compatibility solutions.
I'll make the "visionary" proposal for moving forward with backwards compatibility in a separate issue and then we can decide what to do with this pr after that.
Either way, this pr is not working at the moment.

Ok, I've reworked this pr now:

it's rebased on melodic-devel
it does not remove the boost::shared_ptr<T> createInstance(...) method, but it does deprecate it
it adds a new method std::shared_ptr<T> createSharedInstance(...) meant to replace the boost one

It still replaces the boost filesystem and other functions with c++ alternatives.
I still need to work on linking and usage of the experimental/filesystem stuff on Linux.

@wjwwood I'll do an initial release of pluginlib in melodic without this and will do another release during the alpha period with these changes once rebased and reworked.
@clalancette FYI

Ok, just a reminder that the filesystem replacement stuff (for systems which don't have it) needs to be updated with fixes from the ros2 branch.

just a reminder that the filesystem replacement stuff (for systems which don't have it) needs to be updated with fixes from the ros2 branch.

:+1:
The things I have in mind for the "rework" part are the filesystem fixes and the removal of the checking of __cplusplus as windows doesn't report c++11 compliance even if it supports it. Do you know of anything else I should keep in mind when reworking this ?

How this pr are going?

Given that we don't have another ROS 1 vehicle to ship this in, and given that we have already removed Boost for ROS 2, I'm inclined to close this PR.  @wjwwood do you agree?

That's fine with me.


Modal behavior when the app is in the background.

App hangs when a modal is open and we go to background.
If any modal is open and we keep the app in background and open it back then that modal is disappearing.

If any modal is open and we keep the app in background and open it back then that modal is disappearing.Following are some modals:
1.F&F request accept/decline modal.
2.Keeper request accept/decline modal.
3.Notifications.

If any modal is open and we keep the app in background and open it back then that modal is disappearing.Following are some modals:
1.F&F request accept/decline modal.
2.Keeper request accept/decline modal.
3.Notifications.
If we are on any account page and we keep the app in background and open it back then it shows a "Complete Level 2" modal  everytime on any account that we show it for savings account before level 2 completion

This is fixed now @cakesoft-swati

The following are some of the models that disappear when it is opened and we put the app in the background and open it again:
1.F&F request model accept / reject.

Accept / reject the keeper request model.

Tested on dev app v2.0.75(469)

I have tested it in 80, It is working fine on my side
https://user-images.githubusercontent.com/28229449/167799810-b0772863-ec0a-4d71-a013-bc6695b79f86.mov


jsx-indent behaviour with ternaries

Upgrading from v6.3.0 to v6.4.0 made a perfectly sensible way of doing ternary operators in JSX fail linting due to indentation (did not throw error in 6.3.0).
This:
this.props.asd.length > 0 ?
    <Button className='bacon-yay'>{this.props.asd.length}</Button> :
    0

is supposedly incorrect.
Instead, the linter suggests you should use this, which doesn't make sense:
this.props.asd.length > 0 ?
    <Button className='bacon-yay'>{this.props.asd.length}</Button> :
        0

Yep, it's really critical https://github.com/yannickcr/eslint-plugin-react/issues/901
Can't use 6.4.x with it

@yannickcr This issue is anything but fixed. Using 6.10.0, I'm getting this:
43:7  error  Expected indentation of 4 space characters but found 6  react/jsx-indent

in this code:
          {!condition === 0 ? '' :
            {t('export_btn')}
          }

which I find indented reasonably. Linter expects it to look like this:
          {!condition === 0 ? '' :
          {t('export_btn')}
          }

I have the same issue, our code still heavily affected by this, and we can't update from 6.3.0 for the reason that this gives a bunch of errors.
Can we at least reopen that?
I'm willing to contribute with additional info/tests failing, if necessary.

Actually NVM, the problem was related to a different issue, seems like jsx-indent is actually working fine. Sorry for direct ping.


Update .gitignore

Adding Gemfile.lock to the list of files to ignore. The lock file is not needed on GitHub, and it gets generated when you run bundle jekyll server/build locally. Removing it ensures that we have the same dependencies that GitHub uses.

Removing Gemfile.lock from source control and updating .gitignore to ignore the file in the future.


Forbiddenfruit optional

Description
Implements making forbiddenfruit optional at issue #27
Fixes #27
Also need to:

[X] ~Change setup.py to make forbiddenfruit optional~

Type of change

[X] New feature (non-breaking change which adds functionality)

Checklist

[X] My code follows the style guidelines of this project
[x] I have made corresponding changes to the documentation
[X] I have added tests that prove my fix is effective or that my feature works

Thanks for the PR! I think you still have some issues with imports, the tests fail.

I think it would also be good to specify in the Usage section of the README (and in the docs) that pyfuncol can also be used in this way (maybe at the end with a simple example?)

I think it would also be good to specify in the Usage section of the README (and in the docs) that pyfuncol can also be used in this way (maybe at the end with a simple example?)

You're right, I think it'd be important to document how to use it without extending builtins.

I'll do a final review when the suggestions will be applied 👍

Thanks a lot for your PR @m-doescode!


String size constraint in array triggers new migration with every migrate

Bug description
I have a Prisma schema where I create a table with a column set to be an array of strings with a max size constraint. The first time I run prisma migrate dev to sync the DB schema with my Prisma schema, the operation is successful. If I run prisma migrate dev another time, however, Prisma mistakenly detects that the DB schema is out of sync and prompts to create a new migration. All the migration does it dropping the column and re-adding it again with the same constraint.
How to reproduce

Change the Prisma schema as shown below:

model Test {
  id          String   @id @unique @default(dbgenerated("gen_random_ulid()"))
  testStrings String[] @db.String(16)

  @@schema("translation")
}

Run prisma migrate dev once
Run ``prisma migrate dev` a second time

Expected behavior
Running prisma migrate dev a second time does not prompt to create a new migration
Prisma information

generator client {
  provider        = "prisma-client-js"
  previewFeatures = ["multiSchema"]
}

datasource db {
  provider     = "cockroachdb"
  url          = env("DATABASE_URL")
  relationMode = "prisma"
  schemas      = ["translation"]
}

model Test {
  id          String   @id @unique @default(dbgenerated("gen_random_ulid()"))
  testStrings String[] @db.String(16)

  @@schema("translation")
}

Environment & setup

OS: macOS
Database: CockroachDB
Node.js version: v20.0.0

Prisma Version
prisma                  : 5.3.1
@prisma/client          : 5.3.1
Current platform        : darwin-arm64
Query Engine (Node-API) : libquery-engine 61e140623197a131c2a6189271ffee05a7aa9a59 (at<EMAIL_ADDRESS>Schema Engine           : schema-engine-cli 61e140623197a131c2a6189271ffee05a7aa9a59 (at ../../../node_modules/@prisma/engines/schema-engine-darwin-arm64)
Schema Wasm             : @prisma/prisma-schema-wasm 5.3.1-2.61e140623197a131c2a6189271ffee05a7aa9a59
Default Engines Hash    : 61e140623197a131c2a6189271ffee05a7aa9a59
Studio                  : 0.494.0
Preview Features        : multiSchema

Can you share the migration it creates for you?
Does you schema change when you run db pull?
If so, does that fix the problem?

Note: related feature request for supporting ULID IDs in the Prisma schema https://github.com/prisma/prisma/issues/13679

This is the migration it generates:
/*
  Warnings:

  - The `testStrings` column on the `Test` table would be dropped and recreated. This will lead to data loss if there is data in the column.

*/
-- AlterTable
ALTER TABLE "translation"."Test" DROP COLUMN "testStrings";
ALTER TABLE "translation"."Test" ADD COLUMN     "testStrings" STRING(16)[];

If I try to run prisma db pull, I see that the size constraint is removed from the Prisma schema:
model Test {
  id          String   @id @unique @default(dbgenerated("gen_random_ulid()"))
  testStrings String[]

  @@schema("translation")
}

So I would say that db pull doesn't fix the issue, because in the end it just drops the constraint.

Yes, that would be a bug. Can you open an issue for it if it does not exist yet @dariacm? Thanks.

@janpio https://github.com/prisma/prisma/issues/21227


remove duplicated content

Fixes #1136
Fixes #1135

OPS Build status updates of commit ba87e6a:
:clock10: Preparing: average preparing time is 46 sec(s)

OPS Build status updates of commit ba87e6a:
:clock10: Incremental building: average incremental building time is 22 sec(s)

OPS Build status updates of commit ba87e6a:
:warning: Validation status: warnings

File
Status
Preview URL
Details

bread/toc.json
:warning:Warning

Details

toc.json
:warning:Warning

Details

building-blocks/extending-templates/toc.json
:warning:Warning

Details

docs/vdc/networking-virtual-datacenter.md
:white_check_mark:Succeeded
View

bread/toc.json

[Warning] Error happen when converting bread/toc.json to Pdf. Details: Could not find a part of the path 'T:\jm43\bread\toc.json'.

toc.json

[Warning] Error happen when converting toc.json to Pdf. Details: Could not find file 'T:\jm43\toc.json'.

building-blocks/extending-templates/toc.json

[Warning] Error happen when converting building-blocks/extending-templates/toc.json to Pdf. Details: Could not find a part of the path 'T:\jm43\building-blocks\extending-templates\toc.json'.

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


Deleting multiple matches at once

Thanks for this cool plugin!
I'm not sure, but I think I found a bug (or I am mis-using a feature). From what I can tell, dd on a search result toggles it (turns it active/inactive). If I make a visual selection block, then press d, the lines are actually deleted (I think this is my possible mis-use of the feature). If I press dd again after doing that, the current line is replaced with a different line (I think this is the bug).
Should I be able to do visual selection blocks and delete that way?
Thanks.

yes it is missing feature .dd is only work on normal mode now
i will update it later

I fixed it

Awesome, it is working great! Thank you :-)


IDatabases.List() hangs

I'm trying to manage azure databases from our own application targeting .NET Framework 4.6.2 using Microsoft.Azure.Management.Sql.Fluent nuget package version 1.0.0-beta50. When trying to access the databases to check if a database already exists, it just hangs.
var sqlServer = azure.SqlServers.List().FirstOrDefault(.....); // works
var databases = sqlServer.Databases.List(); // hangs
await sqlServer.Databases.Define(databaseName).CreateAsync(); // works
await sqlServer.Databases.DeleteAsync(databaseName); // works
For logging into Azure I've added an auth-file with a service-principle which is (for now) Owner to the subscription. Any idea what's going on here?

after enhancing the unit test for CRUDing Databases, I don't see a problem with .List() so either this got fixed in the latest v1.0.0 (not beta) release, or we need a more specific repro.


kde remember screen setups for aditional screens, but doesnt allow to change them

Issue description
plasma5 remembered my main screen should be disable when my external monitor is attached,
now that i had a cable issue and the other monitor did not work as expected, it was impossible to alter that setting because whenever i attached the screen not working correctly the one that was working got turned off
i presume this should be an upstream issue?
Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...)
Nix version: (run nix-env --version)
Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion)

i reported https://bugs.kde.org/show_bug.cgi?id=378205

Closing as this most probably is not a bug on our side and we have too much issue bloat anyway.


struct.error: unpack_from requires a buffer of at least 5 bytes for unpacking 5 bytes at offset 0

Driver version

redshift-connector==2.0.909
Redshift version

PostgreSQL 8.0.2 on i686-pc-linux-gnu, compiled by GCC gcc (GCC) 3.4.2 20041017 (Red Hat 3.4.2-6.fc3), Redshift 1.0.52943
Using Redshift Serverless
Client Operating System

Python version

python3.10
Table schema

Problem description

Expected behaviour: query should drop table
Actual behaviour: exception is raised
Error message/stack trace:

Traceback (most recent call last):
  File "/home/airflow/.local/lib/python3.10/site-packages/airflow/providers/amazon/aws/operators/redshift_sql.py", line 82, in execute
    hook.run(self.sql, autocommit=self.autocommit, parameters=self.parameters)
  File "/home/airflow/.local/lib/python3.10/site-packages/airflow/providers/common/sql/hooks/sql.py", line 288, in run
    with closing(self.get_conn()) as conn:
  File "/home/airflow/.local/lib/python3.10/site-packages/airflow/providers/amazon/aws/hooks/redshift_sql.py", line 127, in get_conn
    return redshift_connector.connect(**conn_kwargs)
  File "/home/airflow/.local/lib/python3.10/site-packages/redshift_connector/__init__.py", line 340, in connect
    return Connection(
  File "/home/airflow/.local/lib/python3.10/site-packages/redshift_connector/core.py", line 707, in __init__
    code, data_len = ci_unpack(self._read(5))
struct.error: unpack_from requires a buffer of at least 5 bytes for unpacking 5 bytes at offset 0 (actual buffer size is 0)

Any other details that can be helpful:
Called through RedshiftSQLOperator in apache-airflow==2.4.3

Python Driver trace logs

Reproduction code

from airflow.models import DAG
from airflow.providers.amazon.aws.operators.redshift_sql import RedshiftSQLOperator

with DAG(...) as dag:
    drop_if_exists = RedshiftSQLOperator(
       task_id="...",
       sql="DROP TABLE IF EXISTS {{ params.schema }}.{{ params.table }}",
       params={"schema": "...", "table": "..."},
    )
...

Hi @bradleybonitatibus , does this issue occur consistently or only sometimes?

Hey @Brooke-white! Very infrequently. Raised this issue as it impacted one of our production serverless clusters.

Thanks for that information, @bradleybonitatibus. There are two cases I'm aware of where this error presents. My guess is you are hitting case 1 since you're using Redshift serverless and this issue presents very infrequently.

If "cold", redshift serverless can take some time to be ready to accept connections and queries. Retry logic is the work around in this case
#157 tl;dr there is a redshift server-side issue with select version() which leads to this query failing. clients directly or indirectly issuing this statement experience more consistent failures. SQLAlchemy is one such client.

Got it, will go down the suggested path of adding retry logic to handle this scenario. Thanks for your guidance!

This error message looks to me like the database connection was lost.

Improvements to error messages for this case were made in 2.0.914


Add blog post - Automating SONiC switch provisioning and management with NetBox as a central source

Relates to https://github.com/SovereignCloudStack/issues/issues/689

I merged the current state to the "staging" branch, to have a preview at https://staging.scs.community/

I merged the current state to the "staging" branch, to have a preview at https://staging.scs.community/

I added some minor fixes https://github.com/SovereignCloudStack/website/pull/1045/commits/add902be648f9fb57db1b2786b3f64120656c2bc and merged it to the staging branch again.

Any reason not to merge this @scoopex , @OgarOgarovic ?

Not from my view.

@maxwolfs : I wanted to wait for the preview at https://staging.scs.community/website/de/news/ but it does not appear there (the blogpost ist merged to staging). Any idea why that is the case?

@maxwolfs : I wanted to wait for the preview at https://staging.scs.community/website/de/news/ but it does not appear there (the blogpost ist merged to staging). Any idea why that is the case?

Maybe the staging branch has been deleted and another branch has been merged to staging? I have just done it again with this branch to staging. Should work once completed.

The build process creates the html page out of it (see here), but it seems that the content of gh-pages-staging does not get properly deployed on the server? At least that would be my guess.


Errors running makefile

I'm trying to setup a PIP service, via the cookbook documentaiton, but running into issues with this repo. I'm unable to build wof-pgis-index needed to index WOF records.
Running make or make bin, I see the following errors:
if test -d pkg; then rm -rf pkg; fi
if test -d src/github.com/whosonfirst/go-whosonfirst-pgis; then rm -rf src/github.com/whosonfirst/go-whosonfirst-pgis; fi
mkdir -p src/github.com/whosonfirst/go-whosonfirst-pgis
cp -r client src/github.com/whosonfirst/go-whosonfirst-pgis/client
cp -r vendor/* src/
# github.com/whosonfirst/go-whosonfirst-placetypes
src/github.com/whosonfirst/go-whosonfirst-placetypes/specification.go:13:6: WOFPlacetypeSpecification redeclared in this block
	previous declaration at src/github.com/whosonfirst/go-whosonfirst-placetypes/placetypes.go:27:6
# github.com/whosonfirst/go-whosonfirst-uri
src/github.com/whosonfirst/go-whosonfirst-uri/parse.go:50:3: unknown field 'IsAlternate' in struct literal of type URIArgs
src/github.com/whosonfirst/go-whosonfirst-uri/parse.go:57:16: undefined: AltGeom
src/github.com/whosonfirst/go-whosonfirst-uri/parse.go:71:7: args.AltGeom undefined (type *URIArgs has no field or method AltGeom)
src/github.com/whosonfirst/go-whosonfirst-uri/parse.go:72:7: args.IsAlternate undefined (type *URIArgs has no field or method IsAlternate)
src/github.com/whosonfirst/go-whosonfirst-uri/parse.go:105:20: uri_args.IsAlternate undefined (type *URIArgs has no field or method IsAlternate)
src/github.com/whosonfirst/go-whosonfirst-uri/parse.go:109:37: undefined: AltGeom
src/github.com/whosonfirst/go-whosonfirst-uri/uri.go:162:6: IsWOFFile redeclared in this block
	previous declaration at src/github.com/whosonfirst/go-whosonfirst-uri/parse.go:80:36
src/github.com/whosonfirst/go-whosonfirst-uri/uri.go:183:6: IsAltFile redeclared in this block
	previous declaration at src/github.com/whosonfirst/go-whosonfirst-uri/parse.go:97:36
src/github.com/whosonfirst/go-whosonfirst-uri/uri.go:204:6: IdFromPath redeclared in this block
	previous declaration at src/github.com/whosonfirst/go-whosonfirst-uri/parse.go:124:38
src/github.com/whosonfirst/go-whosonfirst-uri/uri.go:245:6: RepoFromPath redeclared in this block
	previous declaration at src/github.com/whosonfirst/go-whosonfirst-uri/parse.go:130:41
src/github.com/whosonfirst/go-whosonfirst-uri/parse.go:105:20: too many errors
# github.com/lib/pq
src/github.com/lib/pq/connector.go:27:2: not enough arguments to return
src/github.com/lib/pq/connector.go:27:10: c.open undefined (type *Connector has no field or method open)
src/github.com/lib/pq/notice.go:14:18: c.(*conn).noticeHandler undefined (type *conn has no field or method noticeHandler)
src/github.com/lib/pq/notice.go:25:11: c.(*conn).noticeHandler undefined (type *conn has no field or method noticeHandler)
src/github.com/lib/pq/rows.go:46:9: undefined: oid.TypeName
src/github.com/lib/pq/rows.go:74:26: rs.colTyps[index].Type undefined (type oid.Oid has no field or method Type)
src/github.com/lib/pq/rows.go:79:26: rs.colTyps[index].Name undefined (type oid.Oid has no field or method Name)
src/github.com/lib/pq/rows.go:86:2: not enough arguments to return
src/github.com/lib/pq/rows.go:86:26: rs.colTyps[index].Length undefined (type oid.Oid has no field or method Length)
src/github.com/lib/pq/rows.go:92:26: rs.colTyps[index].PrecisionScale undefined (type oid.Oid has no field or method PrecisionScale)
src/github.com/lib/pq/rows.go:92:26: too many errors
make: *** [bin] Error 2

Hrm, it looks like the schema needs to be updated. I will try to investigate shortly.

@stepps00 It is not a schema thing, I just read the issue quickly in between a number of other things.
The problem is caused by the fact that this package has not been updated to account for Go modules (which is a better way, overall, to handle dependencies). Can you please try the following:

Start from a fresh clone of the repo
Checkout the gomod branch
Run make cli-tools (which is the new make bin)
Try running the tool(s) that you've used in the past

Assuming that works I'll merged to master and publish a new release.

From a fresh clone of the repo, I ran the commands:
$ git checkout gomod
    Branch 'gomod' set up to track remote branch 'gomod' from 'origin'.
    Switched to a new branch 'gomod'
$ make cli-tools
    go build -mod vendor -o bin/wof-pgis-connect cmd/wof-pgis-connect/main.go
    go build -mod vendor -o bin/wof-pgis-dump cmd/wof-pgis-dump/main.go
    go build -mod vendor -o bin/wof-pgis-index cmd/wof-pgis-index/main.go
    go build -mod vendor -o bin/wof-pgis-prune cmd/wof-pgis-prune/main.go

This seemed to install without issue, but when I run the command to index a single repository I get the following error:
$ ./bin/wof-pgis-index -verbose -mode directory /path/to/repo/whosonfirst-data-admin-ro
    18:31:53.849023 [wof-pgis-index] FATAL Failed to create new indexer because Unknown driver

Ah, yes. More (past) changes to the go-whosonfirst-index interfaces. I will investigate shortyl.

Try it now? BTW, you should specify -mode repo.

$ git clone https://github.com/whosonfirst/go-whosonfirst-pgis.git
$ cd go-whosonfirst-pgis
$ git checkout gomod
    Branch 'gomod' set up to track remote branch 'gomod' from 'origin'.
    Switched to a new branch 'gomod'
$ make cli-tools
    go build -mod vendor -o bin/wof-pgis-connect cmd/wof-pgis-connect/main.go
    go build -mod vendor -o bin/wof-pgis-dump cmd/wof-pgis-dump/main.go
    go build -mod vendor -o bin/wof-pgis-index cmd/wof-pgis-index/main.go
    go build -mod vendor -o bin/wof-pgis-prune cmd/wof-pgis-prune/main.go

This seems to work as expected, I no longer see the same errors. The next step would be to index the repo I'm interested in working with, as follows:
$ ./bin/wof-pgis-index -verbose -mode repo my/path/to/whosonfirst-data-admin-ro

This also seems to work, I don't see any issues. However, when I try to use the wof-hierarchy-rebuild tool, I now see postgis errors:
$ wof-hierarchy-rebuild -C postgis my/path/to/whosonfirst-data-admin-ro/data/101/751/953/101751953.geojson -U -v
    INFO:root:rebuild hierarchy for my/path/to/whosonfirst-data-admin-ro/data/101/751/953/101751953.geojson
    DEBUG:root:[hierarchy][101751953][Slatina] rebuild feature
    DEBUG:root:[hierarchy][101751953][Slatina] controlled is []
    DEBUG:root:wof:parent_id NOT IN in controlled

    ...

    DEBUG:root:[spatial][postgis][pip] SELECT id, parent_id, placetype_id, meta, ST_AsGeoJSON(geom), ST_AsGeoJSON(centroid) FROM whosonfirst WHERE ST_Intersects(geom, ST_GeomFromGeoJSON(%s)) AND placetype_id=%s AND is_superseded=%s AND is_deprecated=%s with args: ['{"type": "Point", "coordinates": [24.36667, 44.43333]}', '102312307', 0, 0]
    DEBUG:root:[spatial][postgis][pip] time to execute query (point-in-polygon for 44.43333,24.36667) : 0.03873920440673828
    DEBUG:root:[spatial][postgis][inflate_row] {'filters': {'wof:placetype_id': '102312307', 'wof:is_superseded': 0, 'wof:is_deprecated': 0, 'wof:is_ceased': 0}, 'as_feature': True}
    ERROR:root:False
    ERROR:root:[spatial][postgis][inflate_row] BANDAID is wof:is_ceased == 0: True ('0')
    ERROR:root:[spatial][postgis][inflate_row] failed to convert row to feature
    DEBUG:root:[spatial][postgis][pip] failed to inflate row
    DEBUG:root:ensure hierarchy for 101751953 with placetype country : 0 possible
    DEBUG:root:[hierarchy][101751953][Slatina] append 0 possible hierarchies
    DEBUG:root:[hierarchy][101751953][Slatina] no append but ensure hierarchy - matches: False
    DEBUG:root:[hierarchy][101751953][Slatina] ensure common ancestors (is a locality) : region;continent;country
    INFO:root:nothing has changed when rebuilding the hierarchy for 101751953
    INFO:root:Slatina (101751953) has no changes (parent ID is still -1)
    DEBUG:root:[{'locality_id': 101751953}]
    DEBUG:root:writing /my/path/to/whosonfirst-data-admin-ro/data/101/751/953/101751953.geojson

With the hierarchy now only containing the feature itself:
    "wof:hierarchy":[
        {
            "locality_id":101751953
        }
    ],

I've passed both -mode repo and -mode directory, both give the same result. So this is probably an issue with the way I'm indexing the repo in postgres on my machine, I need to poke around.
Either way, I'm able to now run the makefile, so I think issue is fixed - thank you @thisisaaronland!


Add arg

This adds arg which is similar to commander and yargs but super tiny.

very cool. whats the selling point of "tiny" in a node app tho?

The same selling point in any app

less disk space => happy users
less require() calls => better performance
zero dependencies => faster npm install
less LOC => easier to audit and understand

I could go on but this is off the top of my head 🤔

yea the other stuff is more appealing to me than less disk space :) disk space is cheap.


Add clang-13 to the CI build

Clang has different checks and can be more pedantic than gcc. We should be including clang-13 in our CI process to make sure we actually support the versions we claim in our documentation.

Just as a quick check, I tried compiling up-cpp with clang++-18 and got several errors:
/home/user/.conan2/p/b/up-cp85320fffd98f5/b/src/datamodel/serializer/Uuid.cpp:109:20: error: implicit conversion changes signedness: 'int' to 'uint64_t' (aka 'unsigned long') [-Werror,-Wsign-conversion]
  108 |         uuid.set_msb((unix_ts_ms << UUID_TIMESTAMP_SHIFT) |
      |                                                           ~
  109 |                      (ver << UUID_VERSION_SHIFT) | rand_a);
      |                       ~~~~^~~~~~~~~~~~~~~~~~~~~
1 error generated.
gmake[2]: *** [CMakeFiles/up-cpp.dir/build.make:216: CMakeFiles/up-cpp.dir/src/datamodel/serializer/Uuid.cpp.o] Error 1
gmake[2]: *** Waiting for unfinished jobs....
/home/user/.conan2/p/b/up-cp85320fffd98f5/b/src/communication/RpcClient.cpp:135:18: error: captured structured bindings are a C++20 extension [-Werror,-Wc++20-extensions]
  135 |         auto wrapper = [callable, reqid = std::move(reqid),
      |                         ^
/home/user/.conan2/p/b/up-cp85320fffd98f5/b/src/communication/RpcClient.cpp:127:25: note: 'callable' declared here
  127 |         auto [callback_handle, callable] =
      |                                ^
/home/user/.conan2/p/b/up-cp85320fffd98f5/b/src/communication/RpcClient.cpp:140:38: error: captured structured bindings are a C++20 extension [-Werror,-Wc++20-extensions]
  140 |                                 std::call_once(*callback_once, [&callable, &m]() {
      |                                                                  ^
/home/user/.conan2/p/b/up-cp85320fffd98f5/b/src/communication/RpcClient.cpp:127:25: note: 'callable' declared here
  127 |         auto [callback_handle, callable] =
      |                                ^
/home/user/.conan2/p/b/up-cp85320fffd98f5/b/src/communication/RpcClient.cpp:148:38: error: captured structured bindings are a C++20 extension [-Werror,-Wc++20-extensions]
  148 |                                 std::call_once(*callback_once, [&callable,
      |                                                                  ^
/home/user/.conan2/p/b/up-cp85320fffd98f5/b/src/communication/RpcClient.cpp:127:25: note: 'callable' declared here
  127 |         auto [callback_handle, callable] =
      |                                ^
/home/user/.conan2/p/b/up-cp85320fffd98f5/b/src/communication/RpcClient.cpp:160:17: error: captured structured bindings are a C++20 extension [-Werror,-Wc++20-extensions]
  160 |         auto expire = [callable, callback_once](v1::UStatus&& reason) mutable {
      |                        ^
/home/user/.conan2/p/b/up-cp85320fffd98f5/b/src/communication/RpcClient.cpp:127:25: note: 'callable' declared here
  127 |         auto [callback_handle, callable] =
      |                                ^
/home/user/.conan2/p/b/up-cp85320fffd98f5/b/src/communication/RpcClient.cpp:162:25: error: captured structured bindings are a C++20 extension [-Werror,-Wc++20-extensions]
  162 |                     *callback_once, [&callable, reason = std::move(reason)]() {
      |                                       ^
/home/user/.conan2/p/b/up-cp85320fffd98f5/b/src/communication/RpcClient.cpp:127:25: note: 'callable' declared here
  127 |         auto [callback_handle, callable] =
      |                                ^
5 errors generated.
gmake[2]: *** [CMakeFiles/up-cpp.dir/build.make:118: CMakeFiles/up-cpp.dir/src/communication/RpcClient.cpp.o] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:140: CMakeFiles/up-cpp.dir/all] Error 2
gmake: *** [Makefile:146: all] Error 2

It is possible that some of our strange false-positives with clang-tidy trace back to unreported clang compilation errors.


cloudspokes/2732

This software is deprecated. Developers who wish to develop with SOAP should generate their own stubs using the wsdl. Thank you.


Made the readme more clear about how to use inside sails

Oh yep sweet, fixed that now :)

@penguinpowernz Cool thanks!


Add DinkToPdf to pdf section

@Khatib-64 feel free to raise a PR, though I suggest you take note of what @romanimm and @totpero have posted


Fix vertical scroll behavior for weekFormat calendar view

Fix handling vertical scroll animation depended on customGridViewPhysics widget param.

Thanks for your contribution! It is already ready for release :100:


TestParseGeneralizedTime fails on 32bit architectures

I'm the maintainer of the fedora package of this go library, and with version 1.5.0 I get this output from go test on i686 and armv7hl architectures:
      testing: gopkg.in/asn1-ber.v1
gopkg.in/asn1-ber.v1
--- FAIL: TestParseGeneralizedTime (0.00s)
    generalizedTime_test.go:81: test got unexpected result: wanted=2017-02-22 19:15:00 +0000 UTC, got=2017-02-22 18:59:57.852516352 +0000 UTC
    generalizedTime_test.go:81: test got unexpected result: wanted=2017-02-22 19:05:15 +0000 UTC, got=2017-02-22 19:04:57.852516352 +0000 UTC
FAIL

This looks like a loss of precision on 32bit architectures?
I'm not sure if this is a problem in the date parsing code itself or in this specific test.

Thanks @decathorpe

Thanks! That was fast :rocket:


Cannot start SIP Proxy

How can we use the routr as SIP Proxy?
Using this, we tried to start the SIP Proxy. No port is spawn and browser connect is not possible. If you can provide us a config (no need to use rctl) to run a SIP Proxy, this would be great.
Here are the steps we ran.
sipuser@loc:~$ rctl version
[System info]
Server version: v1.0
API version: v1beta1
API path: /api/v1beta1
[Env]

sipuser@loc:~$ rctl proxy -p 9000

Serving Routr Console

You can now view the console in the browser.

  URL	http://localhost:9000

This is currently a beta software
All feedback is greatly appreciated!
Exception in thread "main" java.awt.AWTError: Can't connect to X11 window server using ':0' as the value of the DISPLAY variable.
	at sun.awt.X11GraphicsEnvironment.initDisplay(Native Method)
	at sun.awt.X11GraphicsEnvironment.access$200(X11GraphicsEnvironment.java:65)
	at sun.awt.X11GraphicsEnvironment$1.run(X11GraphicsEnvironment.java:115)
	at java.security.AccessController.doPrivileged(Native Method)
	at sun.awt.X11GraphicsEnvironment.<clinit>(X11GraphicsEnvironment.java:74)
	at java.lang.Class.forName0(Native Method)
	at java.lang.Class.forName(Class.java:264)
	at java.awt.GraphicsEnvironment.createGE(GraphicsEnvironment.java:103)
	at java.awt.GraphicsEnvironment.getLocalGraphicsEnvironment(GraphicsEnvironment.java:82)
	at sun.awt.X11.XToolkit.<clinit>(XToolkit.java:132)
	at java.lang.Class.forName0(Native Method)
	at java.lang.Class.forName(Class.java:264)
	at java.awt.Toolkit$2.run(Toolkit.java:860)
	at java.awt.Toolkit$2.run(Toolkit.java:855)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.awt.Toolkit.getDefaultToolkit(Toolkit.java:854)
	at java.awt.Desktop.isDesktopSupported(Desktop.java:169)
	at java.awt.Desktop.getDesktop(Desktop.java:143)
	at io.routr.ctl.CmdProxy.openBrowser(CmdProxy.java:138)
	at io.routr.ctl.CmdProxy.run(CmdProxy.java:123)
	at io.routr.ctl.Main.main(Main.java:143)
^C

Without X11 it does not work either.
sipuser@loc:~$ unset DISPLAY
sipuser@loc:~$ rctl proxy -p 9000

Serving Routr Console

You can now view the console in the browser.

  URL	http://localhost:9000

This is currently a beta software
All feedback is greatly appreciated!
java.net.SocketException: Operation not permitted
	at sun.nio.ch.Net.listen(Native Method)
	at sun.nio.ch.ServerSocketChannelImpl.bind(ServerSocketChannelImpl.java:221)
	at sun.nio.ch.ServerSocketAdaptor.bind(ServerSocketAdaptor.java:85)
	at org.eclipse.jetty.server.ServerConnector.openAcceptChannel(ServerConnector.java:342)
	at org.eclipse.jetty.server.ServerConnector.open(ServerConnector.java:307)
	at org.eclipse.jetty.server.AbstractNetworkConnector.doStart(AbstractNetworkConnector.java:80)
	at org.eclipse.jetty.server.ServerConnector.doStart(ServerConnector.java:231)
	at org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:72)
	at org.eclipse.jetty.server.Server.doStart(Server.java:385)
	at org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:72)
	at io.routr.ctl.CmdProxy.run(CmdProxy.java:124)
	at io.routr.ctl.Main.main(Main.java:143)
^C

Any help is highly appreciated.

@github4tiri From your logs I see you are running rctl (Routr Control). You need Routr Server. This two tools ship separately.

Hello,
we were running routr server.
sudo snap install --edge rctl
wget https://github.com/fonoster/routr/releases/download/1.2.4/routr-1.2.4_linux-x64_bin.tar.gz -O - | tar xzf
cd routr*x64_bin
perl -pi -e "s|level: info|level: debug|g" config/log4j2.yml
./routr

And then we used the rctl as mentioned above. Still the issue exist, so which might be wrong?
We need to proxy SIP.
Thanks for your help in advance!

I see. Do you get logs from running ./routr? Also, which Linux distro are you using?

Hi Pedro,
I use Linux Mint / Ubuntu 20.04,
Here is console output when starting routr.
root@t480s:~/routr-1.2.4_linux-x64_bin# ./routr
[DEBUG] core.server_properties [server properties => {javax.net.ssl.trustStore=etc/certs/domains-cert.jks, gov.nist.javax.sip.MESSAGE_PROCESSOR_FACTORY=gov.nist.javax.sip.stack.NioMessageProcessorFactory, gov.nist.javax.sip.THREAD_POOL_SIZE=16, gov.nist.javax.sip.TLS_CLIENT_AUTH_TYPE=Want, gov.nist.javax.sip.TLS_CLIENT_PROTOCOLS=TLSv1.2, gov.nist.javax.sip.CACHE_CLIENT_CONNECTIONS=true, javax.net.ssl.keyStoreType=jks, gov.nist.javax.sip.MAX_TX_LIFETIME_NON_INVITE=60, gov.nist.javax.sip.REENTRANT_LISTENER=false, gov.nist.javax.sip.CANCEL_CLIENT_TRANSACTION_CHECKED=false, gov.nist.javax.sip.LOG_MESSAGE_CONTENT=false, javax.net.ssl.keyStorePassword=changeit, javax.sip.AUTOMATIC_DIALOG_SUPPORT=OFF, javax.sip.STACK_NAME=routr, javax.sip.IP_ADDRESS=<IP_ADDRESS>, gov.nist.javax.sip.NIO_BLOCKING_MODE=NONBLOCKING, gov.nist.javax.sip.MAX_TX_LIFETIME_INVITE=60, gov.nist.javax.sip.PATCH_SIP_WEBSOCKETS_HEADERS=false, javax.net.ssl.keyStore=etc/certs/domains-cert.jks, gov.nist.javax.sip.MAX_MESSAGE_SIZE=1048576}]
[INFO ] Starting Routr
[INFO ] Listening on <IP_ADDRESS>:5060 [tcp]
[INFO ] Listening on <IP_ADDRESS>:5060 [udp]
[INFO ] Listening on <IP_ADDRESS>:5061 [tls]
[INFO ] Listening on <IP_ADDRESS>:5062 [ws]
[INFO ] Listening on <IP_ADDRESS>:5063 [wss]
[DEBUG] Starting Restful service server
[DEBUG] location.RouteLoader.loadStaticRoutes [loading static routes]
[DEBUG] loading route for key =><EMAIL_ADDRESS>[INFO ] Restful service server listening on port 4567
[DEBUG] Restful service server configuration: Config(
  apiPath = /api/v1beta1
  bindAddr = <IP_ADDRESS>
  port = 4567
  unsecured = false
  keyStorePassword = /REDACTED/
  keyStore = etc/certs/api-cert.jks
  trustStore = etc/certs/api-cert.jks
  trustStorePassword = /REDACTED/
  maxThreads = 200
  minThreads = 8
  timeoutMillis = 5000
)
[DEBUG] Starting gRPC service, listening on 50099
[DEBUG] registry.Registry.registerAll [sending gateways registration]

Running these commands are fine:
root@t480s:~# rctl login https://<IP_ADDRESS>:4567/api/v1beta1 -u admin -p changeit
Done
root@t480s:~# rctl ping
up

Running the proxy fails:
# rctl proxy -p 15060

Serving Routr Console

You can now view the console in the browser.

  URL	http://localhost:15060

This is currently a beta software
All feedback is greatly appreciated!
Exception in thread "main" java.awt.AWTError: Can't connect to X11 window server using ':0' as the value of the DISPLAY variable.
	at sun.awt.X11GraphicsEnvironment.initDisplay(Native Method)
	at sun.awt.X11GraphicsEnvironment.access$200(X11GraphicsEnvironment.java:65)
	at sun.awt.X11GraphicsEnvironment$1.run(X11GraphicsEnvironment.java:115)
	at java.security.AccessController.doPrivileged(Native Method)
	at sun.awt.X11GraphicsEnvironment.<clinit>(X11GraphicsEnvironment.java:74)
	at java.lang.Class.forName0(Native Method)
	at java.lang.Class.forName(Class.java:264)
	at java.awt.GraphicsEnvironment.createGE(GraphicsEnvironment.java:103)
	at java.awt.GraphicsEnvironment.getLocalGraphicsEnvironment(GraphicsEnvironment.java:82)
	at sun.awt.X11.XToolkit.<clinit>(XToolkit.java:132)
	at java.lang.Class.forName0(Native Method)
	at java.lang.Class.forName(Class.java:264)
	at java.awt.Toolkit$2.run(Toolkit.java:860)
	at java.awt.Toolkit$2.run(Toolkit.java:855)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.awt.Toolkit.getDefaultToolkit(Toolkit.java:854)
	at java.awt.Desktop.isDesktopSupported(Desktop.java:169)
	at java.awt.Desktop.getDesktop(Desktop.java:143)
	at io.routr.ctl.CmdProxy.openBrowser(CmdProxy.java:138)
	at io.routr.ctl.CmdProxy.run(CmdProxy.java:123)
	at io.routr.ctl.Main.main(Main.java:143)

^C

And the workaround for mitigating the X11 error did not help (see above).
Besides that - maybe I am completely wrong with the approach and I have a wrong understanding of "proxy":
I am going to pass all SIP requests from my internal SIP devices over routr. Only the system which runs the routr Software should be able to talk to the internet. On my clients I want to configure the routr as the SIP Provider aka Proxy.

@github4tiri
I believe I know now what's going on.
Few things. The purpose of rctl proxy is to launch the web app, not to proxy sip requests. You see an X11 error because the environment has no graphical interface. Run the rctl command on your local station. Not on the server.
Using rctl is not at all a requirement. And, it only makes sense if you are using routr + redis. Otherwise, you want to use the yaml files.
I want to encourage you to use GH discussion and to post your use case so the community can help you.


Abiity to create a chart from an existing dataset

When a user goes to /chart/new?dataset=<datasetIri>, the cube metadata is fetched
and we go directly to the second step. If the cube metadata cannot be fetched, we
stay in first step where the user has to select a dataset.
This could be useful to entice users to create charts from an existing dataset or
for tests.
Closes https://github.com/visualize-admin/visualization-tool/issues/87.

@ptbrowne Currently, the Cube Creator links to /create/new?cube=<versionHistoryIri> (as described in #87). So the param and the logic don't quite match yet 😅


Optional Selecte(d) items label

I needed to create an autocomplete without both Select and Selected items label, but there is no option to take out the message and ion-item from them. would be nice to have this option in a future version.
Thanks for the ion-autocomplete! Very nice!

Hi @rodrigobnn,
thanks for taking that up. I will try to add it to the 0.3.3 release such that you are able to set empty values.
Thanks and regards,
Guy

Hi @rodrigobnn
the easiest way is to create a own custom template where you do not have these two ion-item defined. You can have a look at this here: https://github.com/guylabs/ion-autocomplete#template-url
Therefore I will close this issue as this is the way to go. Adding a custom functionality just blows up the codebase.
Thanks and regards,
Guy


Add Readme

Would really appreciate more introdution about the tool! Thanks!

I wrote some documentation: https://github.com/Tamriel/logseq2pandoc


Where to put course materials

Currently we have the following in the schedule, with hyperlinks to non-existent locations:

Source: https://itsleeds.github.io/TDStests/schedule.html
Eventually there should I think be 2 (maybe 3) links for each session:

Slides
Materials (including reading material)
(Possibly) homework or something else

In terms of Quarto website design I think it would make most sense to put the content in:
/slides/01.qmd

or the name of the slides, e.g.
slides/intro.qmd

Having both number of the session and the name is unhelpful I think. We can do the same for materials.
Thoughts welcome, cc @rogerbeecham, @mem48, @yyuanxuan, @eugenividal what do you think would work best?

Were are required to put course material on Minerva, I seems confusing to put things in other places too

Going for something like this: https://www.roger-beecham.com/comp-sds/
If we use GitHub for sure we're going to have at least 2 places, and removing GitHub isn't on the table.
It's more about making the content already on GitHub a bit more organised.

p1, p2, p3 etc in course folder is simple and good, going with that for now.


.player property undefined on ios

Playing a video (youtube) using $('video')[0].player.play() works on chrome, safari, android, but won't work on ios.
$('video')[0].player is undefined on ios
Firefox is giving me another error related to youtube, but .player is defined in Firefox.
MEJS v2.17.0
Any thoughts?

This is definitely an error. In iOS we basically don't initiate the player
and this should be changed.
On Thu, Jul 30, 2015 at 7:52 AM, Neil Lambert<EMAIL_ADDRESS>wrote:

Playing a video (youtube) using $('video')[0].player.play() works on
chrome, safari, android, but won't work on ios.
$('video')[0].player is undefined on ios
Firefox is giving me another error related to youtube, but .player is
defined in Firefox.
MEJS v2.17.0
Any thoughts?
—
Reply to this email directly or view it on GitHub
https://github.com/johndyer/mediaelement/issues/1548.

--
John Dyer - http://j.hn/

Found this to do this trick and solve the "you can't move nodes in iOS" problem too:
// move the <video/video> tag into the right spot
$('.mejs-mediaelement', t.container).append(t.$media);

but I still can't get the youtube video to play initially. I have to click on the iframe twice, then it plays, then I can play/pause with the mejs controls or with play().

Ok, so in the end
// move the <video/video> tag into the right spot
$('.mejs-mediaelement', t.container).append(t.$media);

isn't at all necessary since the user has to click on the youtube play button anyways.
what I'm trying to figure out now is now is how to we know the player has been "initialized" (i.e. the user has clicked the youtube play button). Because from that point on play() and pause() work well.

I ended up using
if (mejs.players[id].media.paused && mejs.players[id].media.currentTime > 0) {
  mejs.players[id].play();
}

I don't know of a better way to verify if the user has initiated playback.

I'll upload a patch to fix this that will make into the next release.


Api call in controller not found after loading the assembly at runtime.

Hi Nate,
Thanks for sharing this project, I'd really like to get a plugin architecture working in our projects. Also, congrats on your newborn! (yes I read the other issues ;-)).
I have tried using your library to perform a really simple task: Load an assembly at runtime containing a controller to include the calls from that controller, without having to reference that assembly at compile time.
I have reproduced the issue in a simple sample project here: https://github.com/Chris197/applicationparttest
Any idea what's wrong?
Regards,
Chris
UPDATE:
I have just found that replacing this code:
PluginLoader.CreateFromAssemblyFile(path).LoadDefaultAssembly();
with this:
AssemblyLoadContext.Default.LoadFromAssemblyPath(path);
does work. So the problem does seem to be related to the PluginLoader.

congrats on your newborn!

Thank you!
You're repro is basically doing what I had in my sample: https://github.com/natemcmaster/DotNetCorePlugins/blob/master/samples/aspnetcore-mvc/MvcApp/Startup.cs#L23-L33. Last time I checked, the sample still worked.
If AssemblyLoadContext.Default.LoadFromAssemblyPath is working and PluginLoader is not, then the most likely cause is that your plugin has a private version of the MVC dependency, which means types may not unify between the hosting app's version of MVC and the plugin. Try changing your code to PluginLoader.CreateFromAssemblyFile(path, PluginLoaderOptions.PreferSharedTypes)

Adding 'PluginLoaderOptions.PreferSharedTypes' solves the problem, thanks!
Though, I'm not quite sure I understand what you mean by:

the most likely cause is that your plugin has a private version of the MVC dependency

I'm comparing the code to your sample (on which it was based), but I don't see a difference.

There are multiple versions of Microsoft.AspNetCore.Mvc.dll. Version 2.1.1, 2.1.2, etc. Normally (when not doing plugin type stuff), this isn't an issue because there is only one folder of dll's and you can only have one file named Microsoft.AspNetCore.Mvc.dll in that folder. The .NET SDK will figure out for you which version to use.
Plugins introduce a new quirk. You might have multiple copies of Microsoft.AspNetCore.Mvc.dll in various plugin folders. Which one should be used? PluginLoaderOptions.PreferSharedTypes tells the plugin system to use the same version of Mvc.dll between all plugins and the app host, if possible. Sharing types is the default behavior of AssemblyLoadContext.Default.LoadFromAssemblyPath, but not PluginLoader.
Glad we figured it out. I'll update the MVC sample in this repo.


Add pixels for geoswitching

Task/Issue URL: https://app.asana.com/0/0/1205657190386366/f
Description
See attached task description
Steps to test this PR
https://app.asana.com/0/0/1205802524286065/f

Current dependencies on/for this PR:

develop

PR #3650

PR #3754   👈
PR #3752
PR #3673

PR #3714

PR #3719

PR #3674

PR #3723

This comment was auto-generated by Graphite.


Formatters

self-document the available formatter strings using TypedDict
run flake8 extensions pre-commit (for those who use the tool)

Codecov Report

Merging #107 into master will increase coverage by 0.15%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #107      +/-   ##
==========================================
+ Coverage   97.08%   97.23%   +0.15%
==========================================
  Files           2        2
  Lines         274      289      +15
==========================================
+ Hits          266      281      +15
  Misses          8        8

Impacted Files
Coverage Δ

loggo/loggo.py
97.21% <100%> (+0.15%)
:arrow_up:

Continue to review full report at Codecov.

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


Gir files zesty

Final version of #141
Part of https://github.com/gtk-rs/gir-files/issues/1

Ready to merge, but not builds.

Fine by me.
Thanks!


LMC not working like expecting with morph relationship

I updated LMC from 0.3.6 to 0.4.14, and morph relationships broke.
Explanation:

I call php artisan cache:clear to clear cache.
I get products and their prices using eager-load for the first time, everything is fine.

I get other products and their prices, the query to the database is cached and not displayed in the console, of course cache doesn't have this data - relationships is not returned.

Expecting missed query:

Product model:

Price model:

In 0.3.6 all worked fine.

@smapod Thanks for reporting this. I will look into this over the next few days.

@smapod Can you provide your eloquent query that uses the morph? This will help me create a test case. Thanks!

It's not original, but the same (clear custom scopes).
$products = Product::with(['color', 'glass', 'price', 'icons'])
            ->where('category_id', $category->id)
            ->where('active', 1)
            ->orderBy('position')
            ->paginate(60);

Corrected the answer.

@smapod awesome, thanks for the additional details. I will try to look into this over the next few days.

Likely related to #225

@smapod can you test this again with version 0.4.15? I have a feeling the whereIn fix might also have fixed this. Please let me know how it works. Thanks!

@smapod Actually, if you could test with 0.4.17, just released, which addresses an issue with where clause generation for the cache key.

Problem still exists.
I created repo https://github.com/smapod/lmc-bug, hope it will.

@smapod Thanks for the repo. I will test and see if I can reproduce.

@smapod  This is going to take a while to figure out, if it's at all possible. In the mean time, I would recommend removing any models that implement polymorphic relationships from being cachable. You should still be able to leave the other models cachable.
If anyone has any ideas on how to fix this, I certainly would welcome a PR. Thanks :)

@smapod  Update: actually got it working correctly in your example repo, thanks so much for that. Now testing the fix and creating a test case for it. Hope to have this fix out this evening or tomorrow.

@smapod Please test with release 0.4.18 or above and let me know how it works. Thanks for your patience and support on getting this resolved. Your repo did the trick! :)

@smapod Awesome, that's great to hear! :)


NTR: [Urate:anion antiporter activity]

Suggested term label:
Urate:anion antiporter activity
Definition (free text)
Enables the transfer of a solute or solutes from one side of a membrane to the other according to the reaction: urate(out) + anion (in) = urate (in) + anion (out).
Reference, in format PMID:#######
PubMed=18411268 Fig8b, 9b;
PubMed=12024214 Fig3
Gene product name and ID to be annotated to this term
URAT1/SLC22A12 (Q96S37), OAT10/SLC22A13 (Q9Y226)
Synonyms (please specify, EXACT, BROAD, NARROW or RELATED)
urate: organic anion antiporter activity | narrow
Parent term(s)
GO:0015301 anion:anion antiporter activity; GO:0015143 urate transmembrane transporter activity; GO:0008514 organic anion transmembrane transporter activity

+[Term]
+id: GO:0140813
+name: urate:anion antiporter activity
+namespace: molecular_function
+def: "Enables the transfer of a solute or solutes from one side of a membrane to the other according to the reaction: urate(out) + anion (in) = urate (in) + anion (out)." [PMID:12024214, PMID:18411268]
+synonym: "urate:organic anion antiporter activity" NARROW []
+is_a: GO:0015301 ! anion:anion antiporter activity
+intersection_of: GO:0015297 ! antiporter activity
+intersection_of: has_primary_input CHEBI:22563 ! anion
+intersection_of: has_primary_input CHEBI:30848 ! urate(1-)
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/23306 xsd:anyURI
+created_by: pg
+creation_date: 2022-05-12T05:57:12Z
+

@LucillePo Can you confirm that CHEBI:30848 ! urate(1-) is correct ?
Thanks, Pascale

From @LucillePo and Nevila Nouspikel -
urate has no charge at pH7.3.
CHEBI should be CHEBI:17775 ! 7,9-dihydro-1H-purine-2,6,8(3H)-trione
and NOT CHEBI:30848 ! urate(1-)
Also, this is then not an anion antiporter, but just an antiporter.

I will change all terms using CHEBI:30848 ! urate(1-) to CHEBI:17775 ! 7,9-dihydro-1H-purine-2,6,8(3H)-trione:

GO:0015747 urate transport
GO:0046415 urate metabolic process
GO:0034418 urate biosynthetic process
GO:0019628 urate catabolic process


Undefined array key "honey_time" again

Sometimes I get Undefined array key "honey_time" errors on production.
It happens only some times when it's a spammer that tries to submit our form. Not all spammers are triggering the error. Non-spammer submissions are working fine.
Seems to me that some spammers found a way to remove 'honey_time' key from form payload.
I am using:

Laravel 9.8.1
php 8.1.5
Honey 0.4.0 (without any custom configuration and no livewire)

# My route:
Route::post('contact-us', 'ContactFormController@post')
    ->middleware(['honey'])
    ->name('contact.us');

# My form
<form action="{{ route('contact.us') }}" method="post">
    @csrf
    <x-honey/>
    ...

The same thing is happening to me. When a spammer submits the form without the honey_time field, I get a 500 error instead of denying the request.
I have created a PR to try to solve it: #60

Hi @encodiaweb, technically the PR is ready to be merged, it's just pending of approval 😉 in the meantime, in our team, we are using the version of my fork until the merge is completed.

Thanks for the fix @AntonioJABV. And sorry for the very long delay 😛


HTTP Error 403 Lab1

I'm attempting to follow along the Lab 1 lecture using Google Colab.
Unfortunately, I'm getting a 403 error when I get to the point of running the first script.
My colab session has a P100 and is running Python 3.7.10
Complete Error Message:
/usr/local/lib/python3.7/dist-packages/pytorch_lightning/utilities/distributed.py:49: UserWarning: You have set progress_bar_refresh_rate < 20 on Google Colab. This may crash. Consider using progress_bar_refresh_rate >= 20 in Trainer.
  warnings.warn(*args, **kwargs)
GPU available: True, used: False
TPU available: None, using: 0 TPU cores
/usr/local/lib/python3.7/dist-packages/pytorch_lightning/utilities/distributed.py:49: UserWarning: GPU available but not used. Set the --gpus flag when calling the script.
  warnings.warn(*args, **kwargs)
Using downloaded and verified file: /content/fsdl-text-recognizer-2021-labs/data/downloaded/MNIST/raw/train-images-idx3-ubyte.gz
Extracting /content/fsdl-text-recognizer-2021-labs/data/downloaded/MNIST/raw/train-images-idx3-ubyte.gz to /content/fsdl-text-recognizer-2021-labs/data/downloaded/MNIST/raw
Downloading http://yann.lecun.com/exdb/mnist/train-labels-idx1-ubyte.gz to /content/fsdl-text-recognizer-2021-labs/data/downloaded/MNIST/raw/train-labels-idx1-ubyte.gz
0it [00:00, ?it/s]Traceback (most recent call last):
  File "training/run_experiment.py", line 96, in <module>
    main()
  File "training/run_experiment.py", line 89, in main
    trainer.tune(lit_model, datamodule=data)  # If passing --auto_lr_find, this will set learning rate
  File "/usr/local/lib/python3.7/dist-packages/pytorch_lightning/trainer/trainer.py", line 851, in tune
    self.tuner.tune(model, train_dataloader, val_dataloaders, datamodule)
  File "/usr/local/lib/python3.7/dist-packages/pytorch_lightning/tuner/tuning.py", line 40, in tune
    self.trainer.data_connector.prepare_data(model)
  File "/usr/local/lib/python3.7/dist-packages/pytorch_lightning/trainer/connectors/data_connector.py", line 59, in prepare_data
    self.trainer.datamodule.prepare_data()
  File "/usr/local/lib/python3.7/dist-packages/pytorch_lightning/core/datamodule.py", line 92, in wrapped_fn
    return fn(*args, **kwargs)
  File "/usr/local/lib/python3.7/dist-packages/pytorch_lightning/utilities/distributed.py", line 39, in wrapped_fn
    return fn(*args, **kwargs)
  File "/content/fsdl-text-recognizer-2021-labs/lab1/text_recognizer/data/mnist.py", line 29, in prepare_data
    TorchMNIST(self.data_dir, train=True, download=True)
  File "/usr/local/lib/python3.7/dist-packages/torchvision/datasets/mnist.py", line 79, in __init__
    self.download()
  File "/usr/local/lib/python3.7/dist-packages/torchvision/datasets/mnist.py", line 146, in download
    download_and_extract_archive(url, download_root=self.raw_folder, filename=filename, md5=md5)
  File "/usr/local/lib/python3.7/dist-packages/torchvision/datasets/utils.py", line 256, in download_and_extract_archive
    download_url(url, download_root, filename, md5)
  File "/usr/local/lib/python3.7/dist-packages/torchvision/datasets/utils.py", line 84, in download_url
    raise e
  File "/usr/local/lib/python3.7/dist-packages/torchvision/datasets/utils.py", line 72, in download_url
    reporthook=gen_bar_updater()
  File "/usr/lib/python3.7/urllib/request.py", line 247, in urlretrieve
    with contextlib.closing(urlopen(url, data)) as fp:
  File "/usr/lib/python3.7/urllib/request.py", line 222, in urlopen
    return opener.open(url, data, timeout)
  File "/usr/lib/python3.7/urllib/request.py", line 531, in open
    response = meth(req, response)
  File "/usr/lib/python3.7/urllib/request.py", line 641, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python3.7/urllib/request.py", line 569, in error
    return self._call_chain(*args)
  File "/usr/lib/python3.7/urllib/request.py", line 503, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.7/urllib/request.py", line 649, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 403: Forbidden
0it [00:00, ?it/s]

This is a known issue in torchvision which occurs due to web host change of yann lecun website where the MNIST data is being pulled from. This will be fixed in the next release of torchvision but in the meantime you can follow the solution posted on this issue's github
https://github.com/pytorch/vision/issues/3497

@impiyush I jumped the gun on closing this issue. I didn't have a chance to try the solution until now. Based on the commentary, I assumed it would work.
Unfortunately, running
from six.moves import urllib
opener = urllib.request.build_opener()
opener.addheaders = [('User-agent', 'Mozilla/5.0')]
urllib.request.install_opener(opener)

Still resulted in the same error.


Develop

v1.3.15

Added RAID 6 and RAID 10 (thanks Raj)
Added support for an unlimited number of 32 M.2 drives for RAID 0, 5, 6 and 10.
Now shows how long the resync took.
Now the script automatically reloads itself after updating.

Added RAID 6 and 10 support


Email notification BUG: not reacting on Front End

Email still being sent but it seems like the button doesn't go green. Should be just a FE issue.

Done.


Fix DateTimeFormatter and Ui referesh issues.

Fix java.time.temporal.UnsupportedTemporalTypeException: Unsupported field: YearOfEra exception.
Change initial patient sync paramters to address-city=NAIROBI
Pass a new instance of patient list to refresh RecyclerView
Remove "About Fire Engine" menu entry.

Fixes #110 and #128

@jingtang10 the submitList() related 1 line change has been removed from this PR. This should unblock this PR for go ahead.

@jingtang10 the submitList() related 1 line change has been removed from this PR. This should unblock this PR for go ahead.