Create Continuous Deployment setup

Research, then create a CD setup for the weather tracker Lambda function. It could be all done in GitHub Actions or it could involve AWS Services like CodePipeline and/or CodeDeploy. Research (and document) the pros/cons and final decision here.

I have initially set up this repository with GHA CI/CD workflows. After doing some research, I have decided that GHA is the better way to go and will not be doing further investigation into AWS CodePipeline. The article linked below contains the best reasoning for staying with GHA that I found in my research:
https://serverlessfirst.com/switch-codepipeline-to-github-actions/


Parse XML with namespace

Hello,
First, thanks for this promising tool set. And I hope I send the question on the good canal and repository.
I try to map an xml with namespaces for nodes (an xsd type file).
When I remove the namespaces from my source file, the test triples are generated.
But when I restaure namespace in the xml file and add xsd: ns to xpath, I get an empty set of triples.
As I find no example of "xml with namespace" parsing, I ask myself how I can do that.
Here is the example I try to tackle, this can be added to mattey.
Thanks for you help,
regards

source xml file with ns

<?xml version="1.0" encoding="utf-8"?>
<xsd:schema xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:hropen="https://hropenstandards.org/schema/xml/" targetNamespace="https://hropenstandards.org/schema/xml/" version="4.3.0" id="PersonType" xmlns:xsd="http://www.w3.org/2001/XMLSchema">
  <xsd:include schemaLocation="PersonPhysicalInclusion.xsd" />
  <xsd:include schemaLocation="PersonLegalInclusion.xsd" />
  <xsd:include schemaLocation="../profile/PersonProfileInclusion.xsd" />
  <xsd:include schemaLocation="PersonBaseType.xsd" />
  <xsd:complexType name="PersonType">
    <xsd:annotation>
      <xsd:documentation>A schema that represents all of the information of a person.</xsd:documentation>
    </xsd:annotation>
    <xsd:all>
      <xsd:element minOccurs="0" maxOccurs="1" name="legalId" type="hropen:IdentifierType">
        <xsd:annotation>
          <xsd:documentation>The legal identifier of a person. The issuer is most likely a country or state.</xsd:documentation>
        </xsd:annotation>
      </xsd:element>
    </xsd:all>
  </xsd:complexType>
</xsd:schema>

yarrrml configuration

prefixes:
  rml: 'http://semweb.mmlab.be/ns/rml#'
  rr: 'http://www.w3.org/ns/r2rml#'
  ql: 'http://semweb.mmlab.be/ns/ql#'
  rdf: 'http://www.w3.org/1999/02/22-rdf-syntax-ns#'
  "": 'http://example.org/rules/'
  schema: 'http://schema.org/'
  dbo: 'http://dbpedia.org/ontology/'
  test: 'https://test.fr/mm'
  fno: 'https://w3id.org/function/ontology#'
  fnoi: 'https://w3id.org/function/vocabulary/implementation#'
  fnom: 'https://w3id.org/function/vocabulary/mapping#'
  fnml: 'http://semweb.mmlab.be/ns/fnml#'
  grel: 'http://users.ugent.be/~bjdmeest/function/grel.ttl#'
  xsd: 'http://www.w3.org/2001/XMLSchema'
mappings:
  mapping0:
    sources:
      - [Supergirl.xml~xpath, /xsd:schema/xsd:complexType/xsd:all/xsd:element]
    s: 'http://example.org/character/$(@name)'
    po:
      - [a, 'schema:PersonTEST~iri']
      - ['schema:TARGET', $(@type)]
      - ['schema:DESCR', $(xsd:annotation/xsd:documentation)]
      - ['dbo:SHACL', $(@minOccurs)]

Humm... maybe extract the source's xmlns and reuse them in xpath call ?
This require a well formated xml. But it's the minimum...
I don't know how the xpath interpreter is configurable, but passing the source's xmlns should be doable.
I think it's better "mapping man" experience than the declarative way of the Carmel implementation seems to do this :
carml:declaresNamespace [
        carml:namespacePrefix "edxl-cap" ;
        carml:namespaceName "http://release.niem.gov/niem/adapters/edxl-cap/3.0/" ;

Linked to #9

@DylanVanAssche please find a PR for solving namespaced xpath.
Please note, that it fix work for full namespaced tree.
If the xml mix namespaced and not, this should be explored. See this document for detail about this: "even the default namespace is a namespace, and thus matching names have to be prefixed in XPath".
Another remark:
What do you think about creating an xPathSingleton to provide the xPath object and not create multiple instances of it in XMLRecord and XMLRecordFactory:
XPath xPath = XPathFactory.newInstance().newXPath();
xPath.setNamespaceContext(new NamespaceResolver(document));

@florent-andre Thanks for the PR! I will have a look next week :)

If the xml mix namespaced and not, this should be explored. See this document for detail about this: "even the default namespace is a namespace, and thus matching names have to be prefixed in XPath".

I'm not that familiar with XML namespaces, but I think this is PR is a good start in general, we can just mention it with a TODO comment that this case is not explored.

What do you think about creating an xPathSingleton to provide the xPath object and not create multiple instances of it in XMLRecord and XMLRecordFactory:

That would actually be better I think... Feel free to try it :)
As long as the test cases still pass after this change it is fine.

As this PR was merged, I close this issue.
Thanks guys for building and maintaining this lib !


Usage on web for dev

I am trying to use while on web localhost but whichever Stripe method i call i get this error :
Stripe does not have web implementation.
I expected to be functional on web for dev purpose, am i missing something or i would have to build app to test it?

OS: Linux
Browser Chrome
Version 1.0.0-beta.4

I am having the same issue. Did you find any solution for this?

@HardikDG Not with this lib. Since i was using Ionic with React, react-stripe lib worked form me on web and mobile.

Thanks for your quick response.
I am also using an ionic-react application. I will check the above library. @NikolaStanisavljevic  This library works well in IOS/Android apps also?
In my past experience, they were not allowing js payment library in the mobile apps

Your welcome.
I am not sure for android apps, because i build this app only for apple devices. But can confirm it will work on IOS/web


[Bug Report] Tap-Pane Loading Issue

Element UI version
2.5.4
OS/Browsers version
Windows 10 / Chrome 72.0.3626.109
Vue version
2.6.7
Reproduction Link
https://jsfiddle.net/64hbf1js/
Steps to reproduce
https://jsfiddle.net/64hbf1js/
What is Expected?
Tab User should display a loading spinner
What is actually happening?
Tab User doesnt display a loading spinner

Duplicate of #14351


Only implement RTCPin for actual RTCIO pins

Thank you!
Thank you for your contribution.
Please make sure that your submission includes the following:
Must

[x] The code compiles without errors or warnings.
[x] All examples work.
[x] cargo fmt was run.
[ ] Your changes were added to the CHANGELOG.md in the proper section.
[x] You updated existing examples or added examples (if applicable).
[x] Added examples are checked in CI

Nice to have

[x] You add a description of your work to this PR.
[x] You added proper docs for your newly added features and code.

Previously, RTCPin was implemented for all GpioPin: IsAnalogPin. This is somewhat incorrect as, for example, the S3 has RTCIO pins that are not analog, or, at least they don't have analog functions (RTC_GPIO0, RTC_GPIO21). Either way, relying on IsAnalogPin seems more fragile than it should be, so let's implement RTCPin for the pins that are listed in the rtc_pins! macro invocation.
One bonus of this PR is that implementing pullup/pulldown control will become simpler as I will not need to write unnecessary matches and unreachable branches, I think :)

I'm not sure without researching it but I think I've touched code that was introduced in the deep sleep API PR. Hopefully more to come here, too :)


Update sbt-scalafmt to 2.5.2

About this PR
📦 Updates org.scalameta:sbt-scalafmt from 2.3.0 to 2.5.2
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "sbt-scalafmt" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta", artifactId = "sbt-scalafmt" }
}]

labels: sbt-plugin-update, early-semver-minor, semver-spec-minor, commit-count:1

Codecov Report
Patch and project coverage have no change.

Comparison is base (1454702) 100.00% compared to head (f82bdf0) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##            master      #234   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           48        48
  Branches         5         5
=========================================
  Hits            48        48

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add unit test for metadata reader and writer

Whipped up a simple sanity test for metadata readers and writers. This is mostly a sanity test. We have plenty of runtime coverage for this, but it's always nice to have a unit test.
This will be useful to give people an idea how to use the metadata reader and writer APIs since this is a very self-contained sample.
There's a separate commit in it to disable the test from compiling because of issue tracked in WIP pull request #489.

LGTM


Docs whitespace : Updated the docs with the difference between the whitespace and \s

Resolves #{{TODO: add issue number}}.
Description

What is the purpose of this pull request?

This pull request:

Updated the docs with the difference between the whitespace and \s

Related Issues

Does this pull request have any related issues?

This pull request:

This pull request resolve issue #3107

Questions

Any questions for reviewers of this pull request?

@kgryte i think this time i had done the correct work and i had even undo the changes in the .gitignore file that i had previously done by mistake as i was exploring the repository..Hope sir i had done the right work this time...Please give me the feedback sir
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

ready for initial review. @kgryte


enable dataset, dataflow in data preparation

Description

enable dataset, dataflow in data preparation
Related Issue :

How Has This Been Tested?

Check Lnb in datapreparation.
Go to dataset, dataflow, data snaphot and check the tab menu at the top.

Need additional checks?
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

deploy to 1


Failure to extend the password flow with an id_token

Describe the bug
I've attempted to extend the password flow so it also returns a JWT, instead of returning a token the server returns with an error.
Error Stacks
Traceback (most recent call last):
  File "/home/****/Dev/example-oidc-server/venv/lib/python3.8/site-packages/flask/app.py", line 2464, in __call__
    return self.wsgi_app(environ, start_response)
  File "/home/****/Dev/example-oidc-server/venv/lib/python3.8/site-packages/flask/app.py", line 2450, in wsgi_app
    response = self.handle_exception(e)
  File "/home/****/Dev/example-oidc-server/venv/lib/python3.8/site-packages/flask/app.py", line 1867, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/home/****/Dev/example-oidc-server/venv/lib/python3.8/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/home/****/Dev/example-oidc-server/venv/lib/python3.8/site-packages/flask/app.py", line 2447, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/****/Dev/example-oidc-server/venv/lib/python3.8/site-packages/flask/app.py", line 1952, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/****/Dev/example-oidc-server/venv/lib/python3.8/site-packages/flask/app.py", line 1821, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/home/****/Dev/example-oidc-server/venv/lib/python3.8/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/home/****/Dev/example-oidc-server/venv/lib/python3.8/site-packages/flask/app.py", line 1950, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/****/Dev/example-oidc-server/venv/lib/python3.8/site-packages/flask/app.py", line 1936, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/home/****/Dev/example-oidc-server/website/routes.py", line 97, in issue_token
    return authorization.create_token_response()
  File "/home/****/Dev/example-oidc-server/venv/lib/python3.8/site-packages/authlib/oauth2/rfc6749/authorization_server.py", line 216, in create_token_response
    args = grant.create_token_response()
  File "/home/****/Dev/example-oidc-server/venv/lib/python3.8/site-packages/authlib/oauth2/rfc6749/grants/resource_owner_password_credentials.py", line 145, in create_token_response
    self.execute_hook('process_token', token=token)
  File "/home/****/Dev/example-oidc-server/venv/lib/python3.8/site-packages/authlib/oauth2/rfc6749/grants/base.py", line 88, in execute_hook
    hook(self, *args, **kwargs)
  File "/home/****/Dev/example-oidc-server/venv/lib/python3.8/site-packages/authlib/oidc/core/grants/code.py", line 116, in process_token
    config['nonce'] = credential.get_nonce()
AttributeError: 'NoneType' object has no attribute 'get_nonce'

To Reproduce
Using a combination of the example-oauth2-server and example-oidc-server extend the authorisation server so that the PasswordGrant uses the OpenIDCode extension.
authorization.register_grant(PasswordGrant, [
    OpenIDCode(require_nonce=False),
])

Expected behavior
When using the password flow you should also get an id_token
Environment:

OS: Linux pop-os 5.8.0-7630-generic #32~1605108806~20.10~7e52b13-Ubuntu SMP Wed Nov 11 19:10:30 UTC  x86_64 GNU/Linux
Python Version: 3.8.6
Authlib Version: 0.13

@jackburridge currently OpenIDCode is only available for authorization code flow. I'm working on a generic OpenIDToken extension: https://github.com/lepture/authlib/blob/master/authlib/oidc/core/grants/code.py#L22
For now, it doesn't support password flow either. I'll make it available in v1.


Update to new version of apollo

Hi, are you going to update to the new version of apollo server?
It would be cool to make a screencast and update the blog post.
thanks..

I will try to update the project this weekend. Thanks for the reminder!

Will you update the blog post? or make a screencast would be way more cool..
Thanks


Unexpected add using declaration behavior on type aliases.

struct Y {};
namespace a {
using X = Y;
}  // namespace a

void foo() { a::X^ x; }

Trigger code actions at ^. First of all the message says, Add using declaration for Y, rather than a::X. After executing the action it inserts using a::Y at the top of the file, which is wrong as the symbol's unqual name is X (insertion point is also wrong, this needs to be inserted after type declaration, but a separate bug) but replaces a::X with X, which works as expected.

Fixed in  https://reviews.llvm.org/D91966


chore: bdd-gh-1583471406 to 0.0.1

chore: Promote bdd-gh-1583471406 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


FPS Unlock Feature

Would be nice of you could add the FPS unlocker to this. 60 fps is hurting my eyes on my high refresh rate monitor.
Thanks!
https://github.com/34736384/genshin-fps-unlock

implemented in https://github.com/3Shain/yet-another-anime-game-launcher/commit/bc91d90d0e89f408a7b28946668b45f06c3a4b64

Ty for adding this. Respect!
On Thu, Mar 30, 2023, 2:19 PM 3Shain @.***> wrote:

implemented in bc91d90
https://github.com/3Shain/yet-another-anime-game-launcher/commit/bc91d90d0e89f408a7b28946668b45f06c3a4b64
—
Reply to this email directly, view it on GitHub
https://github.com/3Shain/yet-another-anime-game-launcher/issues/8#issuecomment-1490732029,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEGIMSHZMV63Z3I5UWHGBCDW6XFELANCNFSM6AAAAAAWKYTFB4
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hey in the game settings, i cant change the FPS settings in game to 120 fps, the max is 60fps, even though I have used the FPS unlocker
I have the 16inch M1 Pro

Can your system reach high frames? What resolution are you running the game at?
I can easily maintain 120 at 1440p on Medium/High settings.
M2 Mac Mini Pro 12core cpu/19core gpu

no my system can reach 120 fps. my fps counter does reach 120 FPS, but the in-game settings on genshin impact only has the option of going to 60 fps

no my system can reach 120 fps. my fps counter does reach 120 FPS, but the in-game settings on genshin impact only has the option of going to 60 fps

The in game one is bypassed if you have the fps unlocked turned on, hence why I ask if you can reach 120. If the DXVK Hud is saying 120 in game, you are hitting 120.


Add CONTRIBUTING.md file

Can we please add or refer to the URL containing the guidelines for future contributors, I see there's nothing mentioned about it in the readme or docs for this repo.

Yeah, the project is quite new and we need to create proper documentation (see open issue #27 ).
Meanwhile, it would be enough to follow the generic rules stated for another project handled by us, IntelOwl. See https://intelowl.readthedocs.io/en/latest/Contribute.html

added in develop branch


ChefDK support

I was curious if there is a trick to getting this to work with ChefDK.  I'm running into conflicts with Hashie:
$ chef --version
Chef Development Kit Version: 0.9.0
chef-client version: 12.5.1
berks version: 4.0.1
kitchen version: 1.4.2
$ gem install chef-vault-testfixtures
Fetching: chef-vault-testfixtures-0.5.0.gem (100%)
Successfully installed chef-vault-testfixtures-0.5.0
1 gem installed
$ rspec
/opt/chefdk/embedded/lib/ruby/site_ruby/2.1.0/rubygems/specification.rb:2112:in `raise_if_conflicts': Unable to activate chef-vault-testfixtures-0.5.0, because hashie-3.4.2 conflicts with hashie (~> 2.1) (Gem::ConflictError)
	from /opt/chefdk/embedded/lib/ruby/site_ruby/2.1.0/rubygems/specification.rb:1280:in `activate'
	from /opt/chefdk/embedded/lib/ruby/site_ruby/2.1.0/rubygems.rb:198:in `rescue in try_activate'
	from /opt/chefdk/embedded/lib/ruby/site_ruby/2.1.0/rubygems.rb:195:in `try_activate'
...

I realize I could maintain a Gemfile then bundle install and bundle exec to address this but have noticed a trend of using ChefDK to avoid having to maintain gem dependencies and was wondering if there was a way to have it work here.
If not are there plans have this work 'natively' with ChefDK, which to me would mean having the gem work using the core gems included in ChefDK (as in Hashie 3.4.2 which is included in ChefDK 0.9.0 to address the issue above)?
Or maybe it'd be better to ask the maintainers of ChefDK if they could pull in this additional gem into their product?
Thanks!

@jmccann Have you tried installing into the chefdk gem space?
chef gem install chef-vault-testfixtures

Typically chef-vault-testfixtures installs via bundler and a Gemfile, which keeps its dependencies isolated.  Mark is correct that if you haven't run chef shell-init bash prior to running gem install ... then you might be trying to install into some other ruby install, but we use it all the time with bundler.
The idea is that if I create a cookbook that uses vault and -testfixtures, and six months later ChefDK has been upgraded, or ChefSpec or Test-Kitchen have been released with new features, you can grab my cookbook from source control, set up dependencies with bundler and be ensured of getting the same results I did no matter how long it's been since I last ran tests.

Yes, I have my env set to use ChefDK ENV by default.  I also tried with gem install and chef gem install to make sure.
I'll continue to use Gemfiles in the case where I need to use this gem then.  Was just wondering if I was missing something obvious.
Thanks for the responses and the gem! :smile:


Fix the case of remote pull from smart connector

The fix to SNAP-2101 corrected the full scan to use PK lookups for smart connector
column batch reads. However, it fails to fetch a value if it is on a remote node
(e.g. if that node does not have a thrift server running or bucket was just moved).
Changes proposed in this pull request

GenericStatement.java: now allow for primary-key convertible queries to use the
store distribution plan rather than local derby plan
GemFireResultSet.java, GemFireContainer.java: fixed get/getAll to handle raw objects like DVD[]
added koloboke generated ObjectObjectHashMap to replace THashMap
corrected logging flags to reduce transaction logging with QueryDistribution
and add some local bucket related logs with it (was only for single hop)

Changes to reduce round-trips from connector

replace explicit procedure calls with properties in StatementAttrs that
will be applied by server as part of the statement execution itself:

retainBucketIds: for SET_BUCKETS_FOR_LOCAL_EXECUTION to retain the bucketIds
set on connection across multiple statements (args of the procedure were added previously)
metadataVersion: for check the schema version and throwing exception on mismatch
(to refresh catalog cache on smart connector)
snapshotTransactionId: the ID to replace USE_SNAPSHOT_TXID
This should be further extended to all of SNAPSHOT procedures to instead
set properties on connection and use regular connection commit/rollback/setIsolation
as required (not done here since driver-initiated transactions will change to be on driver)

allow setting common set of StatementAttrs on connection to be inherited by all
statements (use by prepareStatement that needs it before statement creation)
updated BlobChunk.java.tmpl with current BlobChunk.java

Patch testing
store precheckin
ReleaseNotes changes
NA
Other PRs
https://github.com/SnappyDataInc/snappydata/pull/896

Merging now. Will take care of any review comments later.


stacked bar chart doesn't support negative values

In which chart does the bug happen?

Stacked bar chart

RAWGraphs Version
2.0 beta: https://app.rawgraphs.io/

Steps to reproduce this
Aiming to reproduce the stacked bar chart @ https://observablehq.com/@declann/some-cashflows?collection=@declann/calculang
which is created with vega-lite.
There are negative values, but we can see that this works neatly in the vega-lite visualisation.
To attempt to reproduce the visual, I did:

Hit 'csv download (unpivoted)' in https://observablehq.com/@declann/some-cashflows?collection=@declann/calculang
At https://app.rawgraphs.io/ load the csv file and set separator to comma
Configured mappings:

But it results in an error message "Values cannot be negative".
It would be useful to support negative values, as in vega-lite.

The vega-lite stacked chart (includes a line overlay which we do not need):

I would love to work on this issue @giorgiocaviglia


Syntax error when running under Python3.5

It looks like the f"..." syntax is not supported in Python3.5.
Running mkdocs build gives:
Traceback (most recent call last):
File "/usr/local/bin/mkdocs", line 11, in
sys.exit(cli())
File "/usr/local/lib/python3.5/dist-packages/click/core.py", line 829, in call
return self.main(*args, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/click/core.py", line 782, in main
rv = self.invoke(ctx)
File "/usr/local/lib/python3.5/dist-packages/click/core.py", line 1259, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/lib/python3.5/dist-packages/click/core.py", line 1066, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/local/lib/python3.5/dist-packages/click/core.py", line 610, in invoke
return callback(*args, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/mkdocs/main.py", line 152, in build_command
build.build(config.load_config(**kwargs), dirty=not clean)
File "/usr/local/lib/python3.5/dist-packages/mkdocs/config/base.py", line 197, in load_config
errors, warnings = cfg.validate()
File "/usr/local/lib/python3.5/dist-packages/mkdocs/config/base.py", line 107, in validate
run_failed, run_warnings = self._validate()
File "/usr/local/lib/python3.5/dist-packages/mkdocs/config/base.py", line 62, in _validate
self[key] = config_option.validate(value)
File "/usr/local/lib/python3.5/dist-packages/mkdocs/config/config_options.py", line 131, in validate
return self.run_validation(value)
File "/usr/local/lib/python3.5/dist-packages/mkdocs/config/config_options.py", line 609, in run_validation
plgins[item] = self.load_plugin(item, cfg)
File "/usr/local/lib/python3.5/dist-packages/mkdocs/config/config_options.py", line 617, in load_plugin
Plugin = self.installed_plugins[name].load()
File "/usr/local/lib/python3.5/dist-packages/pkg_resources/init.py", line 2447, in load
return self.resolve()
File "/usr/local/lib/python3.5/dist-packages/pkg_resources/init.py", line 2453, in resolve
module = import(self.module_name, fromlist=['name'], level=0)
File "/usr/local/lib/python3.5/dist-packages/mkdocs_with_pdf/plugin.py", line 68
f'(set environment variable {env_name} to 1 to enable)'
^
SyntaxError: invalid syntax

Thank you for your report.
Sorry, I did never run it on 3.5.
As you pointed out, f-strings is not supported in 3.5.
https://www.python.org/dev/peps/pep-0498/
Can I drop support for Python 3.5?
https://www.python.org/downloads/release/python-3510/

Thank you for your report.
Sorry, I did never run it on 3.5.
As you pointed out, f-strings is not supported in 3.5.
https://www.python.org/dev/peps/pep-0498/
Can I drop support for Python 3.5?
https://www.python.org/downloads/release/python-3510/

This issue was fixed in v0.8.3. Please check it.
However, support for Python 3.5 will end in the near future.

This issue was fixed in v0.8.3. Please check it.
However, support for Python 3.5 will end in the near future.

It works now. Thanks for fixing!

It works now. Thanks for fixing!


Document Asset Handling

It looks like the way to use custom fonts is to add a link in the index.html. It would be nice if there were a mechanism to link and load them as first-class resources like you can on iOS and Android. I'm not sure how other asset types are handled here.

When I think about how fonts are loaded in iOS and Android, it's essentially leveraging a configuration option provided by the platform itself, and in the case of the web adding those link tags to the html file is the closest equivalent.
I'm all for a more first-class way to support font loading but unfortunately that isn't really a part of React Native's core and in the early stages of this project I want to focus most on supporting the entirely of the RN core before we move on to creating builtin, web-specific features.
In most cases, if you're itching to get a more streamlined experience you can still implement your own native module which provides that platform specific behavior. For example, like you I wanted to streamline font loading for the demos I was building so I wrote a small native module to allow for loading them from the "react thread": https://gist.github.com/vincentriemer/b8e5e97a92a44638a118016801d378cd


RE2022-276: add display name to PMI biolog  heatmap

LGTM. This is the last one for display name updates right?

yes. After that, I am planning to reload updated data.


Fixes to doc workflow

Local tests failed to catch a problem with the new runner env.

Looks good to me.


Implement the proc_for_pid() in a more stable way.

It seems that jailbreakd will call forkfix at runtime, which then invokes proc_iterate() to iterate through the allproc list. If a process dies or is created simultaneously, it may lead to a kernel panic.
A more stable approach is to only traverse the allproc list when jailbreakd initializes its own proc address for the first time. Then, for subsequent PIDs, task_for_pid() is used to obtain the task port, and the address of the task port is obtained through itk_space of the task self, finally bsd_info is used to obtain the proc address of the target process. Since kernel r/w operations use resources of the current process in kernel, accessing invalid addresses due to other random events is almost non-existent. The unsafe stage only occurs during the first traversal.
By the way, launchd does not have the task_for_pid-allow entitlement, so it can only use the proc_iterate method. (I'm confused as to why userspace reboot gets stuck when launchd calls task_for_pid().)
Tested on iPad Pro A12Z running iOS 15.1 based on tag 1.0.0-beta.8.

Hmm if this is actually an issue then it might be a better idea to just kcall proc_find

@opa334 However, proc_find() increases the reference count of the proc, which means proc_rele() needs to be kcall-ed at the appropriate position in the code. Moreover, if jailbreakd crashes, is killed, or exits (userspace reboot) during this time, the proc kernel resource will leak permanently. Of course, leaking is better than crashing, but it still has side effects.
As for the frequency of crashes, it should depend on the frequency of fork and the busyness of other processes. I used a similar method to implement a daemon a few years ago using proc_iterate, it would panic almost every day due to my frequent usage. However, in recent years, there have been no observable panics since switching to using itk_space.

You see, proc_for_pid is not only called by forkfix. It is called every single time a process starts and gets wx_allowed. I don't think using proc_find / proc_rele should be a big problem honestly.

Btw, what does this panic look like? Spinlock? While itk_space is definitely a solution too, I would try to avoid it due to launchd also needing this stability fix. I guess past jailbreaks were able to give launchd task-for-pid_allow using entitlement injection.

The panic could manifest as a Kernel data abort or element modified after free error in the proc zone.
I'm thinking that since launchd also needs proc_for_pid, kcall(proc_find) is indeed the most suitable option, provided that the kcall primitive is stable enough. On the one hand, the kernel and launchd have the same lifecycle, and if launchd dies, the entire system will restart. On the other hand, using entitlement injection to grant launchd the task-for-pid_allow permission doesn't seem to be an easy task. So as long as the injected code does not have any leakage logic, it should be fine.
To minimize the amount of code changes, I believe that the implementation of proc_for_pid should be as follows:
proc_for_pid_kcall:
    proc_ptr = kcall(proc_find)
    kcall(proc_rele, proc_ptr)
    return proc_ptr

This way, the returned proc_ptr still retains the same semantics as the original proc_for_pid, and there is no need to check if there is any possibility of leakage in other code. Moreover, it is easy to fallback to the original proc_for_pid.
Both proc_for_pid_kcall and proc_for_pid_itkspace only have one unstable point, as you mentioned, the target process may die after proc_for_pid but before memory is patched. Can this be guaranteed by SIGSTOP of the new process? If so, there should be no UAF risk unless the user or another process sends SIGKILL.

However, it looks like my device hasn't experienced any spinlock panics yet. I'll see if there's anything else I can help with. Thank you so much for your genius work!👍


httpd22 2.2.32

Created with brew bump-formula-pr.

Can you fix this audit error ?  * C: 106: col 3: Please set plist_options when using a formula-defined plist.

Fixed it, thanks for the feedback

Looking at the CI it seems that the test is failing. It looks like its using sbin/httpd but I couldn't find that the Formula put the httpd binary in sbin - can you double check that and change it to bin if appropriate ?

I fixed the test path. I'm not sure what to do about these remaining errors.

For the non-executables put in /usr/local/opt/httpd22/bin/, should we add execution privileges to them? Not sure of the repercussions.
About the Headers I would say it should be a keg-only package that user needs to link after install.
What do you think?

httpd24 isn't keg only and that doesn't show the error so it shouldn't be needed. As far as the envvars files go there seems to be some code in the httpd24 formula that mitigates that issue.

I can surely remove keg_only option, but then it will not pass the audit.
$ brew audit httpd22
homebrew/apache/httpd22:
  * Header files that shadow system header files were installed to "/usr/local/opt/httpd22/include"
    The offending files are:
      /usr/local/opt/httpd22/include/apache2/ap_compat.h
      /usr/local/opt/httpd22/include/apache2/ap_config.h
      /usr/local/opt/httpd22/include/apache2/ap_config_auto.h
      /usr/local/opt/httpd22/include/apache2/ap_config_layout.h
      /usr/local/opt/httpd22/include/apache2/ap_listen.h
      /usr/local/opt/httpd22/include/apache2/ap_mmn.h
      /usr/local/opt/httpd22/include/apache2/ap_mpm.h
      /usr/local/opt/httpd22/include/apache2/ap_provider.h
      /usr/local/opt/httpd22/include/apache2/ap_regex.h
      /usr/local/opt/httpd22/include/apache2/ap_regkey.h
      /usr/local/opt/httpd22/include/apache2/ap_release.h
      /usr/local/opt/httpd22/include/apache2/http_config.h
      /usr/local/opt/httpd22/include/apache2/http_connection.h
      /usr/local/opt/httpd22/include/apache2/http_core.h
      /usr/local/opt/httpd22/include/apache2/http_log.h
      /usr/local/opt/httpd22/include/apache2/http_main.h
      /usr/local/opt/httpd22/include/apache2/http_protocol.h
      /usr/local/opt/httpd22/include/apache2/http_request.h
      /usr/local/opt/httpd22/include/apache2/http_vhost.h
      /usr/local/opt/httpd22/include/apache2/httpd.h
      /usr/local/opt/httpd22/include/apache2/mod_auth.h
      /usr/local/opt/httpd22/include/apache2/mod_cgi.h
      /usr/local/opt/httpd22/include/apache2/mod_core.h
      /usr/local/opt/httpd22/include/apache2/mod_dav.h
      /usr/local/opt/httpd22/include/apache2/mod_dbd.h
      /usr/local/opt/httpd22/include/apache2/mod_include.h
      /usr/local/opt/httpd22/include/apache2/mod_log_config.h
      /usr/local/opt/httpd22/include/apache2/mod_proxy.h
      /usr/local/opt/httpd22/include/apache2/mod_rewrite.h
      /usr/local/opt/httpd22/include/apache2/mod_ssl.h
      /usr/local/opt/httpd22/include/apache2/mod_status.h
      /usr/local/opt/httpd22/include/apache2/mpm_common.h
      /usr/local/opt/httpd22/include/apache2/os.h
      /usr/local/opt/httpd22/include/apache2/scoreboard.h
      /usr/local/opt/httpd22/include/apache2/unixd.h
      /usr/local/opt/httpd22/include/apache2/util_cfgtree.h
      /usr/local/opt/httpd22/include/apache2/util_charset.h
      /usr/local/opt/httpd22/include/apache2/util_ebcdic.h
      /usr/local/opt/httpd22/include/apache2/util_filter.h
      /usr/local/opt/httpd22/include/apache2/util_ldap.h
      /usr/local/opt/httpd22/include/apache2/util_md5.h
      /usr/local/opt/httpd22/include/apache2/util_script.h
      /usr/local/opt/httpd22/include/apache2/util_time.h
      /usr/local/opt/httpd22/include/apache2/util_xml.h
Error: 1 problem in 1 formula

The httpd24 formula has a different includedir:    ${prefix}/include/apache2 . Can you try replacing apache2 with httpd ? macOS uses the apache2 directory name and hence the clash in the audit. Not certain what kind of effects this will have though but at least the cause has been identified.

Thanks for the effort @capripot !


Add License to the site and repository

closes #79
As @bast will already be busy with the privacy statement, I am adding the LICENSE stuff here. I suppose we publish the page under the CC BY 4.0 license as discussed during the OC meeting on Wednesday. This PR

adds a LICENSE file
adds a copyright disclaimer to the README
adds a license file to the website (where I also copied the LICENSE information of minimal mistakes, as we incorporate these packages): https://155-267395254-gh.circle-artifacts.com/0/SORSE20/license/index.html
adds the license to the footer

Did I forget anything @bast and @vsoch?

Thanks a lot for taking this! I think this is good to go. One could argue that CC-BY would not really apply to the website source code (but I could also be wrong) and we could say that the website sources are covered by MIT but the website content by CC-BY but in practice the content might be more important.

Thanks for the feedback @bast. I understand your point but on the other hand, about 90% of the words in this source code are content. So it would be strange to give the website a rather strict CC BY license, but then use a relaxed MIT license for the source code. It's difficult to handle though :sweat_smile: I'd just use CC BY for both (or maybe GPL for the source, and CC BY for the website?), but we can also change this later, I think
@vsoch are there any comments from your side?

I agree Philipp. The CC-BY part is the really important and relevant part.

My 0.02 is that more permissive is better, but in terms of separating software vs. content, I think for the software I would just reference the original author's license and provide a link to it, and then use CC 4 for the content. In summary:

content is CC by 4 (and this seems appropriate - websites is listed as one of the general things it covers)
original software we reference a link to the minimal mistakes repository (MIT).

I don't think we've done any new software development here, so I don't think it makes sense to try and put another license for that.

Hmm, maybe my understanding of a LICENSE in the source code repo is then wrong. If I use an MIT license for the LICENSE file, for instance, wouldn't that mean that the content of a file, such as _pages/home.md falls under this MIT license, too? Where it should rather use the CC BY license?
I agree @vsoch that we do not implement much new software development here, which is exactly why I thought that CC BY fits better to the source code repository

I think it's good as it is (LICENSE file is CC-BY) if we add:

original software we reference a link to the minimal mistakes repository (MIT).

alright, thank you both for the feedback! this is also a new lesson to me :sweat_smile:
I included a disclaimer in the LICENSE file that also links to the MIT license of minimal mistakes, and I also included that the _includes and _layouts folders (this is the part where we do have software development) fall under the MIT license. 0052586fd2afcc5083cc20722c989da332a94a93
This is also stated in the README e50b5891e89a139693224f08d03b911b797b7f20
@vsoch and @bast: if you agree with that, please approve the review and I'll merge this PR

It might be, that Github has a problem as it may not recognize the license anymore. This is what I get with licensee detect SORSE20:
License:        NOASSERTION
Matched files:  LICENSE, README.md
LICENSE:
  Content hash:  c25d7e8e69b3284695bd9012bd6ff430e9445b89
  License:       NOASSERTION
  Closest non-matching licenses:
    CC-BY-4.0 similarity:     95.03%
    CC-BY-SA-4.0 similarity:  89.83%
    MPL-2.0 similarity:       36.34%
README.md:
  Content hash:  e857b27db16916db0cad7297a3491b4d8adb8a8c
  Confidence:    90.00%
  Matcher:       Licensee::Matchers::Reference
  License:       CC-BY-4.0
  Closest non-matching licenses:
    WTFPL similarity:  14.81%
    ISC similarity:    13.33%
    0BSD similarity:   13.33%

I would add the notice about MIT into the README but not into the LICENSE. For this reason so that GitHub recognizes it as CC-BY. But I can also do it as follow-up PR.


Needed a little help.

Hey!
Firstly, I found this NN amazing and this whole repo very informative. But I'm also kind of just starting off with Neural Networks. I didn't know how I could implement it after training it. I don't even know if my NN is being trained or not. When i put the main.py module through my terminal, it just says that its using the TensorFlow backend and then just stops running the code.
I'm running the same modules you are, I just went through them a couple of times to understand what you were doing in the code, but aside from that the code is untouched. I was really hoping you could help me out with this.
Thanks in advanced!

Hello @umbracide! There are two places where you can see some more useful output. For one, as the program runs, it logs to log.txt. So after you start main.py, you could tail -f log.txt in another terminal to see the output there.
The other thing you can do is change verbose=0 to verbose=1 in the train.py file in both the fit() and evaluate functions. This will show you the neural net training progress as it's training.
Hope that helps!


[DEV] Often "Server seems to be busy or offline - try again"

I am getting really often the Server seems to be busy or offline - try again message, resulting in a really slow bot:

I'm using follow cluster strategy, so most  of the pokemons catched are lured.

because server seems to be busy or offline...

@arnogues it doesn't look like so, because the behaviour would be more erratic then
In this case, it almost always block at the same action

@pokecunt this is Github, not 9Gag

@pokecunt I know they added emojis and smileys, but this is still for coding

@arnogues I confirm this doesn't seems to happen when using MoveToFort

Have you tried the solution at https://github.com/tejado/pgoapi/commit/e6db98b9a931731a9f449365a31dc1a9b794f5a0 ? Worked for me.


Use venv for scripts

Previously, my README included instructions for users to set up a virtual environment or use a dev container. However, it's easy to miss that instruction, and then the scripts unexpectedly install packages into the main Python requirement.
This PR changes the scripts to create a venv in the root and use that venv's Python for all the commands. This is similar to what's done in https://github.com/pamelafox/azure-search-openai-demo/ except that repo creates multiple venvs, one for backend and one for scripts. Given the overlap between them, I think it's more straightforward to create

@sarah-widder My colleague did some Windows testing and made suggestions for improving the workflow, this PR is the result.

Thanks for approval! Please merge on my behalf :)


[Enhancement] STI relations should choose the appropriate model on instantiation

STI works great the API is clear, so thanks guys! :clap:
The only problem is with instantiation (new MyModel({ /* ... */ })). If MyModel has a relation to a STI model, it ignores the type.
So, if I have a model Employee extended by classes like HR, CEO etc., and a model Firm that has a haveMany relation to employees and the following data:
const firmData = {
  id: 1,
  name: 'VUEX ORM Ltd.',
  employees: [
    { id: 1, name: 'John', type: 'HR' },
    { id: 2, name: 'Jane', type: 'CEO' }
  ]
}

passing this data to any mutations will choose the right class for John and Jane, instantiating a new Firm, however, with this data will not, and both will be instances of the Employee class.
await Firm.create({ data: firmData })
Firm.query().with('employees').find(1).employees
// => [ HR { id: 1, name: 'John' }, CEO { id: 2, name: 'Jane' } ]

// but
new Firm(firmData).employees
// => [ Employee { id: 1, name: 'John' }, Employee { id: 2, name: 'Jane' } ]

Now, sometimes I need to clone the model instances with relations (and, so far, it is done by re-instantiating them, see: #440) and it would be just awesome if the related records were of the right class.
What do you think, guys?
Does this issue need a codepen or something?
@kiaking if it can be implemented, may I offer my assistance?
Thanks!

@SzNagyMisu Ahhh... thanks for the report. This is because when instantiating model, it's in fact not looking at inheritance. Yap, we fix this one.

Would you like me to submit a PR?

@SzNagyMisu Oh yes if you can that would be awesome! 🤝

This is now fixed at Vuex ORM 0.32.4! Thanks @SzNagyMisu for the awesome fix! 🎉
https://github.com/vuex-orm/vuex-orm/releases/tag/v0.32.4


feat: dynamic clientId Initialization

Hi :)

Added folders for methods, constants, and states (for global ref) under src, and restructured the folder architecture considering scalability for future method development.
Introduced a new method named setGoogleClientId, which operates by changing the global ref value.
The clientId can be initialized in the following ways:

Statically during middleware installation.
Dynamically in any component.
Initially set and later updated dynamically.

Fixed an issue where the style height value was not being applied to the element before the renderButton of GoogleSignInButton is called.
Updated the documentation, including a caution block in the section about initializing nuxt-vue3-google-signin.

I am very happy to contribute to this project. Please let me know if there are any suggestions for improvements in the folder structure or functionality.

Thanks @zuhno
this is a brilliant work, and im really happy that you are really keen to be a contributor
i will review the code and let you know my thoughts

@kasvith Please let me know if there are any parts that need to be revised.

Sure, i didnt have much time lately to focus on the project

hi @zuhno made a comment

im adding this to the v3 milestone

I have checked this. I'll check the support for this feature in nuxt.
thank you for reviewing the PR :)

i have checked the related repository and will follow up on the merge status. thanks.

Now all looks great, i will check this out locally and then we are good to go


Uncaught ReferenceError: wanakana is not defined

Thanks for the great tool!
For the past couple of days, I've been seeing this error in the console with quizzes. Clicking on the quiz button doesn't do anything as a result. Getting the same result on Firefox, Safari, and Chrome:
Uncaught ReferenceError: exports is not defined
    <anonymous> https://unpkg.com/wanakana:3
[wanakana:3:1](https://unpkg.com/wanakana)
Uncaught ReferenceError: wanakana is not defined
    get_questions https://sakubun.herokuapp.com/scripts/quiz.js:122
    jQuery 4
        c
        fireWith
        l
        o
[quiz.js:122:17](https://sakubun.herokuapp.com/scripts/quiz.js)
    get_questions https://sakubun.herokuapp.com/scripts/quiz.js:122
    jQuery 4
Source map error: Error: request failed with status 404
Resource URL: https://unpkg.com/wanakana
Source Map URL: index.js.map

Thanks for reporting! Can you check if it's fixed now?

That worked, thank you!!


FRR v1.19 Routing: Diagnostics: General:IPv4 Routes

https://github.com/opnsense/plugins/blob/8a72865e007cd157f0cbf6eace34a67971525c46/net/frr/src/opnsense/scripts/quagga/quagga.rb#L121
"ruby -d /usr/local/opnsense/scripts/quagga/quagga.rb --general-routes"
"Exception NoMethodError at /usr/local/opnsense/scripts/quagga/quagga.rb:121 - undefined method split for nil:NilClass"
Results in blank output in the Web-UI

I can't reproduce here, for me it works fine.
What happens when you revert back to 1.17?

I got the same error. frr v1.17 opnsense 20.7.5
ruby -d /usr/local/opnsense/scripts/quagga/quagga.rb --general-routes
Exception `LoadError' at /usr/local/lib/ruby/2.6/rubygems.rb:1375 - cannot load such file -- rubygems/defaults/operating_system
Exception `LoadError' at /usr/local/lib/ruby/2.6/rubygems.rb:1384 - cannot load such file -- rubygems/defaults/ruby
Exception `Gem::MissingSpecError' at /usr/local/lib/ruby/2.6/rubygems/dependency.rb:311 - Gem::MissingSpecError
/usr/local/opnsense/scripts/quagga/quagga.rb:565: warning: assigned but unused variable - mode
Using Ext extension for JSON.
Exception `NoMethodError' at /usr/local/opnsense/scripts/quagga/quagga.rb:121 - undefined method `split' for nil:NilClass
{"general_routes":"error"}```

i'm also seeing this in two independent test environments.
#2143 should fix a lot of these issues by using FRR's JSON output directly. it also generally overhauls/improves the diagnostics UI. i expect to add the missing pieces of the new python based parser pretty soon


Fix tests and address linting/typing issues

I didn't want to add a bunch of noise to the base PR, so I moved this effeort into a separate PR. Should be pretty uncontroversial, but would love at least a quick check from someone.

This should probably be against the master branch. And you can later rebase auth_ext on top of the updated master

This should probably be against the master branch. And you can later rebase auth_ext on top of the updated master

There is only one change that actually exists on master which is the SCRAM change and it requires the update to the testbase, which I can definitely move.


[SPARK-41741][SQL] Encode the string using the UTF_8 charset in ParquetFilters

What changes were proposed in this pull request?
This PR makes it encode the string using the UTF_8 charset in ParquetFilters.
Why are the changes needed?
Fix data issue where the default charset is not UTF_8.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Manual test.

Merged to master, branch-3.4 and branch-3.3.


Revamp Ballerina/salesforce connector

Description:
This is the main task to revamp the existing Ballerina Salesforce connector.

[ ] Go through the existing APIs and come up with draft design #4913

[x] REST API
[x] APEX REST API
[x] Bulk API
[x] SOAP API
[x] Stream API
[x] Metadata API

[ ] Go through the connector and identify issues and possible improvements

Due to the large nuber of APIs and design complexities, we have decided to implement only the REST parts of these APIs,  including REST API, BULK API V2, and APEX REST API for the first iteration


feat: implement ArtifactRollbackReboot retry

If ArtifactRollbackVerifyReboot fails, ArtifactRollbackReboot will be retried CONFIG_MENDER_ROLLBACK_REBOOT_LOOP_RETRIES times.
Ticket: MEN-7496
Changelog: Title

Note that this might cause an infinite loop with the code in its current stage, however there is already a ticket for this (MEN-7516). The state data counter will give an error if it's exceeded, but we don't check what set_and_store_state returns atm


[pyclaragenomics] [cudapoa] Issus-356 - Update recent changes to python binding

Refect BatchSize contructor change in python binding.

@rilango thanks for making these changes! in another PR we can make the changes for the python API to accept the other options exposed as part of the new BatchSize struct


feat: Added actionable

Part of https://github.com/lenra-io/lenra_ui_runner/issues/33

:tada: This PR is included in version 1.0.0-fix-doc-release.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Dockerise usage

Could you provide some more information for this issue. I would want to work on it.

Basically it would be to wrap babelish to a docker container, so people can use it via docker without having ruby installed on their machine.
So needs to be done:

create dockerfile
documentation how to run it: i.e. docker run netbe/babelish csv2strings... or something like that


Can't parse remote let property

type A {

  remote let b: Int

}

fails to parse with:
x.hylo:3.3: error: expected declaration
  remote let b: Int
  ^

The syntax is let b: remote let Int


Include the available role sizes from locations

Previously the locations client did not return information about the available role sizes in each location. This commit adds two additional properties to the LocationList type to expose a list of Web/Worker role sizes and VM sizes available per region. Since the data is already returned by the API endpoint no API call changes are required.

Ignore this one sorry, branched from master instead of dev.


feat: trigger batch job to update dataset artifact citations upon DOI update

Reason for Change

#5880

Changes

Added BatchJobProvider and made it parameter for BusinessLogic
Added method to BatchJobProvider to kickoff dataset artifact metadata update batch job for a given dataset + map of metadata update changes to persist
Added static method to BusinessLogic for generating dataset citations (+ refactored for DRYness)
Trigger dataset artifact metadata update batch job to update the 'citation' for each dataset in a collection upon a detected DOI update in the business logic method that handles collection metadata updates
Renamed instances of existing_dataset_version --> current_dataset_version in business logic for consistency

Testing steps

TBD: unit test + rdev

@nayib-jose-gloria you're going to need to generate a new business logic class in each subtask, and pass into the update_h5ad call. Right now that function is referencing a (shared) business logic as an instance attribute.


Better cert-rotation support in istio

Certificate rotation is needed in case istiod is restarted, because a new plug-in CA might have been inserted.
This is the first step towards https://github.com/istio/istio/issues/30947 better cert-rotation support.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[X] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

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

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

I am not fully following the requirement here may @howardjohn is. But the implementation relying on getting all stats and comparing LDS version seems too fragile.

@ramaraochavali  Thanks for your comment. Please check out our intention to this changes from here.
I am also pinging @shankgan (there is some feature recently pushed to upstream https://github.com/istio/istio/pull/30871) to take a look.

I agree with Rama. Does https://github.com/istio/istio/pull/30871 satisfy this completely?

I agree with Rama. Does #30871 satisfy this completely? - not completely. Have documented the pending requirements on https://github.com/istio/istio/issues/30947

Does it work with istiod restart ?

I think this PR may be quite useful. My only issue with it is that it requires a restart of Istiod for proxies to get certs from the new CA. If we ever implement a fix to start using new ca certs on the fly, then requiring istiod to be restarted is unnecessary IMO.

I think this PR may be quite useful. My only issue with it is that it requires a restart of Istiod for proxies to get certs from the new CA. If we ever implement a fix to start using new ca certs on the fly, then requiring istiod to be restarted is unnecessary IMO.

Is it Ok to continue with this PR until istio has support for restarting istiod not required to pick new CA ?
Otherwise can I take any part in the RFC ?

I think this PR may be quite useful. My only issue with it is that it requires a restart of Istiod for proxies to get certs from the new CA. If we ever implement a fix to start using new ca certs on the fly, then requiring istiod to be restarted is unnecessary IMO.

Is it Ok to continue with this PR until istio has support for restarting istiod not required to pick new CA ?
Otherwise can I take any part in the RFC ?

Would it be possible for you to look into this feature (i.e not having to restart Istiod when we change the content of "cacerts")?
More specifically - In createIstioCA, we don't have a file-watcher over the contents of cacerts. This makes it difficult to do root-rotation when using plugin certs without restarting Istiod. Ideally, when we get an event-callback mentioning that contents of file/folder are changed, we need to alter the CA parameters (certificate/cert-chain). This will ensure that all new workloads are signed with the new cert parameters.
@myidpt - What do you think?

Would it be possible for you to look into this feature (i.e not having to restart Istiod when we change the content of "cacerts")?
Sure, I will look into this.
More specifically - In createIstioCA, we don't have a file-watcher over the contents of cacerts. This makes it difficult to do root-rotation when using plugin certs without restarting Istiod. Ideally, when we get an event-callback mentioning that contents of file/folder are changed, we need to alter the CA parameters (certificate/cert-chain). Some of that code can be borrowed from our SELF-SIGNED root-rotation library I think. This will ensure that all new workloads are signed with the new CA certs.

I think those are good pointers to start looking into this feature.

I am closing this, as we are trying to solve this with different approach.


Add original stacktrace to exceptions with original exceptions

Closes #296
This adds the original stacktrace to exceptions. It improves the debugging experience, and it's especially helpful if those exceptions are reported via crash reporting tools.
Technically, this is a breaking change.
This would be really nice to have for https://github.com/ueman/sentry-dart-tools/tree/main/sentry_link

LGTM


Content-ID header over 64 characters gets an extra newline; breaks batch requests

I'm working with batch requests, and adding custom request_id values. With a request_id value of length 28, the total length of the Content-ID value inside the < and > characters is 64.
Somehow, for >=64 characters, "\n " (including trailing space) gets added before the Content-ID value, but after the "Content-ID: ", resulting in a malformed Content-ID value which makes the round trip back (response takes the form Content-ID: response- <uuid+request_id> instead of Content-ID: <response-uuid+request_id>) and throws an exception (a BatchError for a malformed Content-ID value).
I've narrowed this down to this patch of googleapiclient/http.py starting at line 1234:
# encode the body: note that we can't use `as_string`, because
# it plays games with `From ` lines.
print(message) #no newline here
fp = StringIO()
g = Generator(fp, mangle_from_=False)
g.flatten(message, unixfrom=False)
body = fp.getvalue()
print(body,[body]) #newline appears here, printing a list shows newline as \n

This can be reproduced without requiring an authorised call, using the following test code:
from httplib2 import Http
from apiclient.discovery import build
directory = build('admin', 'directory_v1')

def errorTest(id_length):
    req_id = "abcdefghijklmnopqrstuvwxyz0123456789"
    try:
        batch_dir = directory.new_batch_http_request()
        batch_dir.add(directory.groups().list(), request_id=req_id[0:id_length])
        batch_dir.execute(http=Http()) #BatchError, unprintable (separate issue)
    except Exception as e:
        print(e.reason)
        raise

for i in [27,28]:
    print(i,"############################################",i) #readability
    try:
        errorTest(i)
    except AttributeError:
        #only occurs if call not authorised; BatchError occurs first and isn't caught
        pass

Thank you for the very clear report!

By Design: Line Folding (RFC2822) (Unrecognised by server)
I've found that this line-folding is by design (for lines with length >78), thanks to this StackOverflow Question.
This ordinarily would not be an issue, but for the Content-ID header, it contains no spaces. As such, when the total length is over 78, line folding will occur at the first available whitespace character, which happens to be immediately after the colon, and the server does not recognise this as folding.
(I am unsure if folding is supposed to occur at the beginning of a header, as that does not appear to be explicitly described by RFC2822, but I would assume the implication is that folding may occur at the beginning of a header value.)
If I insert a space within the header value at an appropriate location, it folds there instead, and is unfolded correctly by the server, however the server does not re-fold it in the response (given an absolute maximum 998 character limit in RFC2822, this is irrelevant).
"Content-ID: <>" is 14 characters, leaving a maximum of 64 within the greater than/less than symbols (angle brackets) before folding will occur. The greater than/less than symbols have no effect on the folding (apart from altering the line length).
The folding only occurs in the MIMEMultipart object, not the MIMENonMultipart object - and only in Python3, not Python2.
Possible Workaround
My proposal, if the server-side issue cannot be fixed, is to store the request_id values separately, and uniquely identify them with a base 10 integer in the actual request, as follows:
request_id_lookup = { 0:"my_request_id", 1:"my_second_request" }
#alternatively, a list, because they are sorted and integer-indexed
request_id_lookup = ["my_request_id", "my_second_request"]

Then, the header would look something like Content-ID: <uuid+1234> - 2^64 is only 20 digits in base 10, which only pushes the line to 71 characters. As a matter of fact, the order list already behaves like request_id_lookup, so a simple enumerate call in creating the request is all that would be necessary (plus a lookup to decode it).
I am willing to write this workaround, but would like confirmation that this is the way to proceed, as it is a workaround and does not address the actual issue.

I'm currently reviewing ways to avoid the folding.

This appears to be the very simplest fix:
https://github.com/google/google-api-python-client/compare/master...mcdonc:content-id-linefolding-164
(once I finish tests, I'll submit a PR)
Thanks so much for the precise bug report (and apologies for the interminable wait for a response).


⚠ TorrentLeech has degraded performance

In 081c465, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 5150 ms

Resolved: TorrentLeech performance has improved in f09ac77 after 24 minutes.


Pull replicator filter not working in 1.3.0-27 against CouchDB

Setting a pull replicator filter does not seem to be working in any of the 1.3.0 pre-releases (tested 1.3.0-12, 23 and 27). So until now when for example setting a server-side filter function like:
pullReplication.setFilter("sync/pull");
Then the request query string would contain &filter=sync%2Fpull to allow for filtering. However, with 1.3.0 the filter seems to no longer be added to the query string.

Is the replication running with Sync Gateway or CouchDB? (the version might be helpful to have as well).

It is running with CouchDB 1.6.

Hi @moritz-scandit,
I have some questions:

Are you using CBL Native API? or using CBL  from PhoneGap?
How did you set filter method? could you please provide code snippet?

Thanks!

We are using the Native Android API in our app. The filter is set through just a call to .setFilter(string), removing a few detail this is how the whole thing more or less looks like:
manager = new Manager(new AndroidContext(context), managerOptions);
database = manager.getDatabase(DATABASE_NAME);

BasicAuthenticator authenticator = new BasicAuthenticator(user, password);
pullReplication = database.createPullReplication(remoteDatabaseUrl);
pushReplication = database.createPushReplication(remoteDatabaseUrl);
pullReplication.setAuthenticator(authenticator);
pushReplication.setAuthenticator(authenticator);
pullReplication.setContinuous(true);
pushReplication.setContinuous(true);

pullReplication.setFilter("sync/pull");

pullReplication.start();
pushReplication.start();

It works perfectly like this in 1.1.0 and 1.2.1 but does no longer add the filter to the request's query string in 3.0.0.

Hi @moritz-scandit,
Thank you for code snippet. Could you please also provide your filter function which is set to database?
http://developer.couchbase.com/documentation/mobile/current/develop/guides/couchbase-lite/native-api/replication/index.html#filters

Hi @moritz-scandit,
Sorry, I misunderstand your issue. You'd like to use CouchDB server side filter function? We will look into the issue.
http://developer.couchbase.com/documentation/mobile/current/develop/guides/couchbase-lite/native-api/replication/index.html

Yes, exactly. The filter is server-side which is why I expect it to be part of the query string of change requests as it was until 3.0.0.

I tested this with the new 1.4.0 release of the Android Couchbase Lite, and it's still not working.
I have looked at the client code in couchbase-lite-java-core, and I see where it's dropping the filter params from the query string (the filter name gets placed in the query string in all cases, but the filter params get skipped for a POST). I think the right fix may be to set the request to a POST only if docIDs is set in ChangeTracker, but still do GET for all other filtered _changes queries. I'll look into this.

@philmay, CouchDB only handle docIDs as request body of POST /_changes REST.  CouchDB requires all other parameters in the HTTP query parameter. I guess Cloudant requires everything in HTTP query parameter.
If you can not modify server side codes, you can modify CBL code to use GET instead of POST, and build CBL by yourself.

@hideki, Yep, that's what I plan to do. I have what I think is a quick and simple fix to keep the POST around for docIDs, but use GET for all others. This should be compatible with CouchDB/Cloudant.

simplest solution could be set usePOST variable in ChangeTracker false.
https://github.com/couchbase/couchbase-lite-java-core/blob/master/src/main/java/com/couchbase/lite/replicator/ChangeTracker.java#L68

I was looking at that. If I make the default for usePOST false in the ChangeTracker constructor, and then have it set to true if setDocIDs() is called, then it should work for all cases. You really do need to use the POST if you have a list of docIDs, and that's still compatible with CouchDB/Cloudant as well.
Thanks!

@hideki, I made the following change to ChangeTracker.java

Default to GET for _changes queries, except for the following two cases.
Use POST if the filter is on Sync Gateway channels (filter=sync_gateway/bychannel).
Use POST if the filter is on doc_ids (filter=_doc_ids).

I think this should cover the long URL string issue with using GET always.
Would you be open to accepting this as a pull request? It's about 25 lines of new code in ChangeTracker.java, and 6 lines in the unit tests. If so, I'll submit it.

Hi @philmay
We can not accept this change at this point. There are two reasons:

We have discussed incompatibility with CouchDB and Cloudant for a while. As Couchbase Lite development team, we decided to implement current way. We hope CouchDB and Cloudant fix their POST /_changes REST API.
Couchbase Lite has other implementation CBL iOS and .NET. We can not accept this change only for CBL Android/Java.

Hope you can understand this.
Regards,

Hi @hideki,
Thanks for the quick reply. I do have followup questions:

If we make the same changes to the iOS and .NET versions, would you be open to accepting the change?
In the CBL docs, it is stated that filtered pull replication is supported against Cloudant/CouchDB (http://developer.couchbase.com/documentation/mobile/current/guides/couchbase-lite/native-api/replication/index.html#filtered-pull-from-couchdb-pouchdb-or-cloudant). Have you decided to no longer support compatibility?
Is Couchbase currently engaging with the CouchDB/Cloudant community in any way to come to a consensus on how to remain compatible with replication?

I'm just trying to get a better idea of what our options are going forward.
Thanks for your help! I really appreciate it!

@philmay,

My comment 1 is a primary reason. So we might not be able to accept changes.

CBL started to call /_changes REST API with POST. If its request fails, try to use GET. As far as I know, CBL should work with CouchDB 1.6.1 that I confirmed.
https://github.com/couchbase/couchbase-lite-java-core/blob/3ed5056ee4edfe27e67f816f08d6e342fd7f8b73/src/main/java/com/couchbase/lite/replicator/ChangeTracker.java#L341

Couchbase founders started CouchDB. But currently, we do not actively involve with CouchDB. Honestly, we don't discuss the compatibility issues with them

@hideki, Thanks for the info!
In response to #2 above – CBL is putting the filter name (set via setFilter() call) into the query string for a POST, but not the filter parameters (set via setFilterParams() call). In response to this POST, CouchDB does not generate an error. Rather, the filter function is happily called with null for all the filter parameters and we get a 200 OK, but the results are not what we expect (because all the filter parameters resolve to null rather than what the programmer set them to). I have tested this in both CouchDB 1.6 and 2.0, and both behave the same way (no error, but bad filter logic).
This bug is particularly difficult to detect, precisely because it does not produce an obvious error. Instead it produces odd behavior with results that don't match the intent of the app logic. My fix gets around this by using GET for all _changes queries except for filtering by Couchbase channels or CouchDB doc_ids. It's also pretty simple  ;)
Thanks again!

Sorry I know it is not the best place for this questions, but I can not find info about.  I have to change also the usePOST variable in ChangeTracker to false, but the ChangeTracker  code is bloqueck because is compiled by gradle, so please how Can I import couchbase Lite 1.4 (android) not using gradle ? I have tried a lot of things but I am absolutely  stuck

@philmay
You could download 1.4 jar files from download site: https://www.couchbase.com/downloads .
And you could add all required jar files into your project.

@llisas, I have an advantage of having an internal maven repo here on-site, so I configured my build to use that, and pushed my fixed CBL 1.4 artifacts there. What @hideki suggested also works, which is how I did it before moving to our internal maven repo. I do recall it being tricky to get the build to work with just the jars.
It's been a while, but here's the part of my old build.gradle for including the locally built jars in my Android build:
// Configs for including CBL explicitly via local aar/jar files
    compile(name: 'couchbase-lite-android-release', ext: 'aar')
    compile(name: 'couchbase-lite-java-core', ext: 'jar')
    compile(name: 'couchbase-lite-java-forestdb-release', ext: 'aar')
    compile(name: 'couchbase-lite-android-sqlite-custom', ext: 'aar')
    compile(name: 'couchbase-lite-java-native-sqlcipher', ext: 'aar')

// Configs for the other libs (required for both of above CBL configs
    compile 'com.squareup.okhttp3:okhttp:3.3.1'
    compile 'com.fasterxml.jackson.core:jackson-core:2.5.0'
    compile 'com.fasterxml.jackson.core:jackson-annotations:2.5.0'
    compile 'com.fasterxml.jackson.core:jackson-databind:2.5.0'
    compile 'com.github.oxo42:stateless4j:2.4.0'

Hope this helps.

@philmay  does not work on my site : (
¿Could  anyone, please,please, describe the steps I have to do in order to import CBL to my proyect? . I have tried all that I have read without success

Is it failing to compile or failing to run? Do you have the CBL .jar and .aar files in lib directories in your project that are being searched during build? I have the following in my top-level build.gradle file, specifying that dependencies may be found in my app libs directory and the top-level libs directory for my project. The .jar and .aar files go in these dirs. Again, we've been on internal maven for so long now, I don't do it this way anymore, and I doubt I could reproduce it. It did work at one time, however.
allprojects {
    repositories {
        jcenter()
        flatDir {
            dirs '../libs'
            dirs 'libs'
        }
    }
}

Usign filter params in query string is not a correct way to implement at query string has its own limitation. Instead one should use selector instead of filter if you are using CouchDB Server 2.0 and above. Only thing you need to make sure at your client side is, set the filter name = _selector and in filter params, pass params in following format -
{
  "selector": {
    "key_name_to_filter": "key_value_to_filter",
    "$or": [
      {
        "$and": [
          {
            "key_1": "value_1"
          },
          {
            "key_2": "value2"
          }
        ]
      },
      {
        "$and": [
          {
            "key_3": "value_3"
          },
          {
            "key_4": "value_4"
          }
        ]
      }
    ]
  }
}

Above format is just an example illustrating you can create complex queries in selector filter.

If filter parameters are set and sent via POST, then it will trigger a CouchDB timeout bug https://github.com/apache/couchdb/issues/1606.
With respect to Couchbase team, I suggest we starting a folk (from @philmay https://github.com/couchbase/couchbase-lite-android/issues/942#issuecomment-295879313 ) of this library, until CouchDB would support filter parameters in POST in their future release.
@philmay Will you be willing publish a fork, and other android developers can use your fork as depedency in gradle? Or, can you please make a pull request against latest 1.4.4 release? I can help publish a fork.


Question about 3d keypoint data.

When I using keypoints_3d in RGBto3D_train.json, I found that x or y value may be -1400 or less(body part, 0-16).  Is the unit of measurement for the data millimeters? Where is the origin of the coordinates? I seem to not have found a clear answer in the data description.
If anyone can help me, thank you very much.

We recorded the value in millimeter. You can justify easily the unit of our data by: if it is generally <10, then its in meter; if it is generally >100, then its in millimeter
As for origin, if you want to project 3D coordinates into 2D, please check the closed issue 5 and 6, and I also past the answer here too: The 3d keypoints in RGBto3D_train.json are actually R_cam * X_global, so you don't need R_cam but you need T_cam to project correctly into 2D.
The 2d keypoints are X=K_cam(R_cam * X_global + T_cam) then X[:, :, :2] /= X[:, :, 2:3] is the 2D coordinates
Otherwise, if you just want to do training and evaluation without 2D, then the position of the origin is not important, since our proposed evaluation are always pelvis aligned (global translation ignored)

Alternatively, you could also download H3WB dataset in a format commonly employed for 3D pose estimation tasks here.
We provide an accompanying data preparation class for this format. We also recommend this format for your experiments.
In this format all 3D keypoints are in global coordinate system.

@Zhuyue0324 I want to use 3D_global, so I just need to figure out R_cam and then use it to divide x,y,z data. Am I correct?

Yes, you are right. Or you just use the data which @nerminsamet has provided, which directly has 3D global coordinates


Aux POW missing chain merkle root in parent coinbase

Hi,
I've installed latest wallet but have a some problem with merged mining.
I try to merge with LTC and can find a block but when I try to submit it on Doge node it return
2021-11-12 12:20:02 ERROR: Aux POW missing chain merkle root in parent coinbase
2021-11-12 12:20:02 ERROR: CheckAuxPowProofOfWork : AUX POW is not valid
2021-11-12 12:20:02 ERROR: ProcessNewBlock: AcceptBlock FAILED
can you help me to main LTC+DOGE not only LTC?))

Did it work on the previous version or is this a new installation?

this is a new installation

Alright! Converting to Discussion->Q&A.


didWriteValueForCharacteristic not trigger short time after connection in IOS

After connection, didWriteValueForCharacteristic trigger correctly after calling writeByteArray() function, but after some random time, it does not trigger, but it successfully write data to peripheral.
Steps to reproduce the behavior:

Connect a peripheral from an IOS device.
Set Notify Value as true to read characteristic after connection
Send data to write characteristic and see didWriteValueForCharacteristic trigger in PeripheralDelegate.kt file
Wait a little bit, approximately 5-10 seconds and try again, peripheral can get the data but didWriteValueForCharacteristic not trigger anymore.
Disconnect device and reconnect. See same behaviour, the function trigger at the beginning and then it brokes.

didWriteValueForCharacteristic must be triggered after every writeByteArray() function's call.
All Apple devices show same behaviour. (Iphone 7 (IOS 15.6.1), Iphone 8 (IOS 15.6.1), Ipad 9 (IOS 16.1.1), Mac M1 (Ventura 13.0.1))
There is no any problem with Android devices.

Interesting, maybe its getting deallocated on ios.
Happy for you to put a pull request in with the fix, otherwise i will have to look at it when i get time


npm install --production error！

npm WARN cannot run in wd<EMAIL_ADDRESS>node core/server/utils/npm/preinstall.js (wd=/data/node/ghost)
my nodejs version is v4.4.4

Hi @dlmufei, sorry to hear you're having trouble. As mentioned in the readme & contrib guide, if you have questions about Ghost or need help, please head on over to slack - https://ghost.org/slack/ - and post in the #help channel. There you can get help & support from the Ghost community, which we're not able to do here in the GitHub bug tracker.


Fix linebreak after microphone icon in speakers overlay.

@normanjaeckel Please review later. I have to merge it today. Thanks.

Ok.


Filter not finding an artist

I have some music by a rapper named G-Eazy and when I search for his tracks in the filter box they are not showing up.
Here's a link to a flac file you can test with on your end.
https://filebin.pw/9rgkY/

So for this particular tag, the first hyphen (the one after G) isn't a standard hyphen. It's this unicode character: https://www.fileformat.info/info/unicode/char/2010/index.htm, which is why regular text search isn't matching it.

Looks like we may be able to get around this by using SQLITE FTS tables. Will require some more investigation. http://www.sqlite.org/fts3.html#tokenizer

I'm not positive why musicbrainz is injecting a utf8 -. I'll sort that out on my end for now.

Already have another issue tracking fuzzy search. I believe we can close this now.


[REVIEW] Linkedin Redesign Server

Project link
https://github.com/symaticvisuals/linkedin-redesigned-server

Reviewed on video for my YouTube channel
some notes

use releases
hide packages
improve read me
use gh actions
add repo description


Manage User Details

When a new user joins the room there should be a notification sent about the new user joined.

Completed


Rings wont go in trinket slot

I've opened a few worlds to test this out and in all of them I can place the rings in any trinket slots. Shift click doesn't do anything and manually trying to place it wont work either.

Hello, i'm not able to replicate the issue. Can you replicate this with only Trinkets, Rings of Ascension and Fabric API installed?
I think it is a mod incompatibility


Doc Enhancement request : create activity using a 3rd party go package

Hi,
I am a newbie at golang and I am struggling managing dependencies and imports correctly.
Using a 3rd party package shall be a very common practice when developing actvities. I followed the createactivity.md doc which is a good step-by-step manual. But, at the end, I fail make it work as my 3rd party package dependencies are clearly not well managed.
Can you please enhance the doc to clearly explain how to manage creating an activity using 3rd party go packages ?
Thanks
Eric

@ecarlier-tibco Can you please explain a little bit better what do you mean by 3rd party go packages? And add a step to step explanation on how to reproduce the issue if possible?
Thank you very much!

Hi @ecarlier-tibco do you still feel this is needed? 3rd party packages don't really require any special management, you import them in your activity and thats pretty much it...

This can be closed. It was long time ago, and felt the need at that time because of an issue that was solved since


Add  Component

Resolves #32 and #33.  This also changes the card width to be consistent with the figma, and changes the cursor to pointer when hovering a button.

Border on regular textfield is actually 'light' grey (on the figma) rather than 'lighter' as specified on the mock, so I made it 'light' in the code as well
I only listed options for the type prop for stuff that doesn't look weird
This expects a React Element to be passed into the icon prop, rather than just putting in an SVG as a child - since there can be an SVG as a result of setting the error prop
The user can specify if they want a textarea by setting the as prop to "textarea", and optionally setting the number of lines they want with the lines prop

The red outlined text area shows the lines set to 2, while the disabled text area is the default

I have no idea why there seems to be weird spaces here ^ as a side note...

I added some basic documentation in terms of defining props, which will hopefully be helpful later down the road.  I can add some more in a separate PR!


Add support for spell variants from consumable items

The Select Other Variant button on chat cards from consumable items deletes the original message instead of updating it because I didn't want make the whole thing even more janky. I probably should've renamed isFromConsumable but the name is technically still correct because it points at the consumable the spell is from. 😄
Closes #3382

I just realized this can't handle Trick Magic Item...  Back to the drawing board.


Modularize write_file

Hey, I just discovered your lib, read through the code and noticed that all (?) write_file methods follow the same pattern. What do you think about a modularization where the general structural work is done by abstract_formatter and the derived classes only perform the format specific customization? Something like the visitor pattern. The abstract formatter would iterate through the sections and items, pick eligible rows, retrieve the translations, maybe even write the entries and the specific formatters only do the stuff that's really special about a format.
Would you accept a PR?

You can take a look at where I'm going in my fork.

Opened #87


[Python][CI] Compatibility with newer minio

Describe the bug, including details regarding any error messages, version, and platform.
Conda-forge very recently added minio, which means we could now use that, at least in the conda(-forge) jobs. I'd be interested in doing that because I see a bunch of skips in the test suite of the following kind (not exhaustive):
SKIPPED [2] pyarrow/tests/test_dataset.py:2623: `minio` command cannot be located
SKIPPED [1] pyarrow/tests/test_dataset.py:2638: `minio` command cannot be located
SKIPPED [1] pyarrow/tests/test_dataset.py:2664: `minio` command cannot be located
SKIPPED [1] pyarrow/tests/test_dataset.py:4754: `minio` command cannot be located
SKIPPED [1] pyarrow/tests/test_dataset.py:4821: `minio` command cannot be located
SKIPPED [1] pyarrow/tests/test_fs.py:471: `minio` command cannot be located
SKIPPED [2] pyarrow/tests/test_fs.py:562: `minio` command cannot be located

so if all it takes is adding the test dependency - why not?
As it turns out, this blows up pretty badly on main (through a proxy PR on our infra), which is at least partly, because arrow currently pins to a relatively outdated version
https://github.com/apache/arrow/blob/4fac528e2ee9c4efce453420275cbaf0b3b6adb0/ci/scripts/install_minio.sh#L54-L56
while the oldest available version in conda-forge is 2<IP_ADDRESS>.07.06.
Aside from 2-3 exceptions, the failures are all from the teardown of the s3fs and py_fsspec_s3fs fixtures, where the code unconditionally does fs.delete_dir(bucket), even though the test presumably scribbled stuff in there.

I'm quite out of my depths here, but I think a alternative would be to somehow pipe through ForceBucketDelete or --force. A cheap alternative is the following patch (also doesn't cover 100%), which just doesn't care about failed bucket deletes:
--- a/python/pyarrow/tests/test_fs.py
+++ b/python/pyarrow/tests/test_fs.py
@@ -256,7 +256,10 @@ def s3fs(request, s3_server):
         allow_move_dir=False,
         allow_append_to_file=False,
     )
-    fs.delete_dir(bucket)
+    try:
+        fs.delete_dir(bucket)
+    except OSError:
+        pass

 @pytest.fixture
@@ -358,7 +361,10 @@ def py_fsspec_s3fs(request, s3_server):
         allow_move_dir=False,
         allow_append_to_file=True,
     )
-    fs.delete_dir(bucket)
+    try:
+        fs.delete_dir(bucket)
+    except OSError:
+        pass

 @pytest.fixture(params=[

After that patch, the only remaining errors are then:
FAILED pyarrow/tests/test_fs.py::test_get_file_info[PyFileSystem(FSSpecHandler(s3fs.S3FileSystem()))] - AssertionError: assert <FileType.Directory: 3> == <FileType.NotFound: 0>
 +  where <FileType.Directory: 3> = <FileInfo for 'pyarrow-filesystem/a/aa/aaa/': type=FileType.Directory>.type
 +  and   <FileType.NotFound: 0> = FileType.NotFound
FAILED pyarrow/tests/test_fs.py::test_delete_dir[S3FileSystem] - Failed: DID NOT RAISE <class 'OSError'>
FAILED pyarrow/tests/test_fs.py::test_delete_dir_contents[S3FileSystem] - Failed: DID NOT RAISE <class 'OSError'>
FAILED pyarrow/tests/test_fs.py::test_move_directory[PyFileSystem(FSSpecHandler(s3fs.S3FileSystem()))] - Failed: DID NOT RAISE <class 'OSError'>

This issue does not happen on 13.0.0?!
The final kicker is that all this is passing with arrow 13 - I even checked that the tests didn't get skipped. So it appears there are at least two things at work here: a change in minio behaviour & a change in pyarrow somewhere.
Component(s)
Continuous Integration, Python

@h-vetinari Can you try running the Arrow C++ tests with this new Minio version and report the results?

I don't understand - that's exactly what I did in https://github.com/conda-forge/arrow-cpp-feedstock/pull/1170. You can look at one of the resulting CI runs here

Ok, but if you don't report the results on the Arrow issue tracker, then nobody notices :-)

Wait, no, the CI run you're pointing to is showing the PyArrow tests. I'm talking about the Arrow C++ tests.

I'm talking about the Arrow C++ tests.

Ah, sorry for the misunderstanding. I've been trying to enable the C++ tests in https://github.com/conda-forge/arrow-cpp-feedstock/pull/1058, but it's not passing yet (completely aside from this issue here).

Ok, but if you don't report the results on the Arrow issue tracker, then nobody notices :-)

I had opened https://github.com/apache/arrow/issues/35587 for that. 🙃

@pitrou in case it's helpful, I ran the test suite from a recent C++ build with minio RELEASE.2024-09-09T16-59-28Z on macOS aarch64 and got a few failures. Here's one as an example:
[ RUN      ] TestS3FS.GetFileInfoSelectorRecursive
/Users/bryce/src/apache/arrow/cpp/src/arrow/filesystem/s3fs_test.cc:826: Failure
Failed
'_error_or_value66.status()' failed with IOError: When listing objects under key '' in bucket 'empty-bucket': AWS Error UNKNOWN (HTTP status 429) during ListObjectsV2 operation: Unable to parse ExceptionName: TooManyRequests Message: Please reduce your request rate

Key part there being "Please reduce your request rate".
I then set export MINIO_API_REQUESTS_MAX=1600 and then only got two failures which should be easily resolved. It looks like some behavior changed in minio:
[ RUN      ] TestS3FS.CreateDir
/Users/bryce/src/apache/arrow/cpp/src/arrow/filesystem/s3fs_test.cc:1011: Failure
Failed
Expected 'fs_->CreateDir("bucket/somefile")' to fail with IOError, but got OK

@amoeba Thanks for the report. It should be useful when someone tries to update Minio on CI, I guess.

For the purposes of testing the python-bits, this issue has been solved in 18.0.0rc0 (presumably by https://github.com/apache/arrow/pull/44225) - thank you! I've been waiting for arrow v18 to enable testing the C++ library as well, will report back in #35587.


API style fixes

Coverage increased (+0.002%) to 32.828% when pulling 8b1b771764f7b63d37551418b52ff56a86d16c1f on api_wiki_4 into 7404531aed339b233154d1b6eb03dc34bb6e550d on master.

Coverage decreased (-0.06%) to 32.762% when pulling 71f1438c1586d860b2f73a3f94305f0819873e72 on api_wiki_4 into 7404531aed339b233154d1b6eb03dc34bb6e550d on master.


Change tide repo url to https

I suggest changing tide submodule url to https protocol, as ssh enforces users to be registered at github and have ssh keys set up

thank you


react-slick and ES6

Does this component work with ES6 is the first question I have? I am getting some strange behavior like some of the properties are stating that they are functions and not what they are defined in you documentation, for example speed when I define it as React.PropTypes.int as per your documentation I get the following error
csfWidgets-external.js:33520Warning: Failed prop type: Carousel: prop type speed is invalid; it must be a function, usually from React.PropTypes.
in Carousel
Also I can never get the carousel to only show one slide:
render() {
    return(

            <Slider {...this.props}>

            </Slider>

     );
}

Carousel.defaultProps = {
id: 'carouselContainer',
content: '',
dots: true,
infinite: true,
adaptiveHeight: false,
arrows: true,
slidesToShow: 1
};
I believe that I have defined things correctly and I have also included both of the CSS files that I need.
Please let me know what I might be missing here or if there is a issue with this component and ES6.
Thank you in advance for looking into this matter.

If I remove all the property definitions from my class it seems to fix the issue

The error messages go away but the system still does not show only 1 image and the arrows do not show either

I have the same problem.


Automated install

Make an automated install and update for the user.
WIP: https://gist.github.com/ThomasTJdev/2110b891b4620c5f3c9635ccc52db5ad
Goal: curl -sSL https://nimwc.org/install.sh | bash

So it does not install Nim, so the user must have Nim already installed previously,
...then why not use NimScript instead of Bash :grey_question:
:thinking:

Yeah, otherwise user will be prompted for installing using choosenim.
I just feel pretty limited using NimScript.. would it be possible to achieve the same using NimScript?

We can steal ideas and inspiration from https://github.com/kaushalmodi/nim_config#list-available-tasks
nim_config does complex stuff.
Besides Bash while being kinda universal, its features are not,
Centos and debian stable tend to have very old bash with missing features and commands compared to newer ones.
We can try doing both too.
It should live on the DevOps folder.
:thinking:


[BUG] UniGetUI won't install on Windows 10 ARM64

Please confirm these before moving forward

[X] I have searched for my issue and have not found a work-in-progress/duplicate/resolved issue.
[X] I have tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions.
[X] This issue is about a bug (if it is not, please use the correct template).

UniGetUI Version
3.1.1 / 3.1.2b4
Windows version, edition, and architecture
Windows 10 Home - ARM64
Describe your issue
When trying to Install UniGetUI on AMR64 Window 10 Home the install works but starting the application fails.
This does however work on Windows 11 of the same hardware.
With other software you need to pick the x86 version for Win10 ARM64 and x64 for Win11 ARM64 perhaps the installer drops the x64 build on Win10 ARM64?
Steps to reproduce the issue
Install UniGetUI either via installer or WinGet CLI.
Installer works but once the installer tries to spawn the applicaton it fails, WinGet bails with "no applicable installer found".

UniGetUI Log
N/A

Package Managers Logs
N/A

Relevant information
No response
Screenshots and videos
No response

Windows 10 Arm64 is not supported.
Supported OS:

Windows 10 x64
Windows 11 x64
Windows 11 arm64 (through x64 emulation)


Core dumped

Hi,
I tried to run it, using the container image, but only got core dumps. Here's a transcript:
./elfconv.sh /bin/echo
[INFO] Building libelfconv.a ...
In file included from Entry.cpp:3:
/root/elfconv/bin/..//backend/remill/include/remill/Arch/Runtime/Intrinsics.h:70:44: warning: '__remill_undefined_f80' has C-linkage specified, but returns user-defined type 'float80_t' which is incompatible with C [-Wreturn-type-c-linkage]
   70 | [[gnu::used, gnu::const]] extern float80_t __remill_undefined_f80(void);
      |                                            ^
1 warning generated.
In file included from VmIntrinsics.cpp:5:
/root/elfconv/bin/..//backend/remill/include/remill/Arch/Runtime/Intrinsics.h:70:44: warning: '__remill_undefined_f80' has C-linkage specified, but returns user-defined type 'float80_t' which is incompatible with C [-Wreturn-type-c-linkage]
   70 | [[gnu::used, gnu::const]] extern float80_t __remill_undefined_f80(void);
      |                                            ^
1 warning generated.
[INFO] Generate libelfconv.a.
[INFO] Converting ELF to LLVM bitcode ...
[BUG] GetFuncEntry is called but symbols is empty
./elfconv.sh: line 53: 10235 Aborted                 (core dumped) ./elflift --arch aarch64 --bc_out lift.bc --target_elf "$ELFPATH" --dbg_fun_cfg "$2"

(gdb) bt
#0  __pthread_kill_implementation (threadid=281473331847200, signo=signo@entry=6, no_tid=no_tid@entry=0) at ./nptl/pthread_kill.c:44
#1  0x0000ffff9ddff254 in __pthread_kill_internal (signo=6, threadid=<optimized out>) at ./nptl/pthread_kill.c:78
#2  0x0000ffff9ddba67c in __GI_raise (sig=sig@entry=6) at ../sysdeps/posix/raise.c:26
#3  0x0000ffff9dda7130 in __GI_abort () at ./stdlib/abort.c:79
#4  0x0000aaaac78959c4 in BinaryLoader::ELFObject::GetFuncEntry (this=0xfffffb34ceb0) at /root/elfconv/front/Binary/Loader.cpp:245
#5  0x0000aaaac78a4e10 in AArch64TraceManager::SetELFData (this=0xfffffb34ce48) at /root/elfconv/front/TraceManager.cpp:81
#6  0x0000aaaac78adba0 in main (argc=1, argv=0xfffffb34d298) at /root/elfconv/front/Lift.cpp:42

Is /bin/echo something that can't be converted to WebAssembly?

With the hello example:
./elfconv.sh ../examples/hello/a.out
[INFO] Building libelfconv.a ...
In file included from Entry.cpp:3:
/root/elfconv/bin/..//backend/remill/include/remill/Arch/Runtime/Intrinsics.h:70:44: warning: '__remill_undefined_f80' has C-linkage specified, but returns user-defined type 'float80_t' which is incompatible with C [-Wreturn-type-c-linkage]
   70 | [[gnu::used, gnu::const]] extern float80_t __remill_undefined_f80(void);
      |                                            ^
1 warning generated.
In file included from VmIntrinsics.cpp:5:
/root/elfconv/bin/..//backend/remill/include/remill/Arch/Runtime/Intrinsics.h:70:44: warning: '__remill_undefined_f80' has C-linkage specified, but returns user-defined type 'float80_t' which is incompatible with C [-Wreturn-type-c-linkage]
   70 | [[gnu::used, gnu::const]] extern float80_t __remill_undefined_f80(void);
      |                                            ^
1 warning generated.
[INFO] Generate libelfconv.a.
[INFO] Converting ELF to LLVM bitcode ...
./elflift --arch aarch64 --bc_out lift.bc --target_elf /root/elfconv/examples/hello/a.out --dbg_fun_cfg
terminate called after throwing an instance of 'std::runtime_error'
  what():  [ERROR]<EMAIL_ADDRESS>is included in multiple sections.

./elfconv.sh: line 60: 11199 Aborted                 (core dumped) ./elflift --arch aarch64 --bc_out lift.bc --target_elf "$ELFPATH" --dbg_fun_cfg "$2"
[INFO] Generate lift.bc.
[INFO] Converting LLVM bitcode to WASM binary and glue code (for browser) ...
emcc: error: lift.bc: No such file or directory ("lift.bc" was expected to be an input file, based on the commandline arguments provided)
[INFO] Generate WASM binary.

Did I miss something?

@jedisct1
Thank you for trying!
First, elfconv cannot compile the dynamically linked or stripped binaries now, so unfortunately, I think you may fail to compile many ELF binaries placed in the directories of Ubuntu from the beginning. However, I will implement them in the future because those functions are important.
Second, to prevent the failing of ./elfconv.sh ../examples/hello/a.out
I placed the Makefile in the every examples/* directory, and please try again after pulling the latest commit.
Thanks!

Thank you!


Bump standard to 3.11.1

This fix looks awesome.
https://github.com/feross/standard/issues/144

Indeed I will merge it to standard-engine and update semistandard once I get online in an hour or so.

Released semistandard 4.2.1 which includes that fix!


why do scripts run on npm i (let's find out)

I don't understand why scripts run on npm i...
must... know... why...

cannot have a script called "prepare"...
https://docs.npmjs.com/cli/v6/using-npm/scripts#life-cycle-scripts


Update gluten & tls

A gently PR to update gluten and use the last version of tls.0.15.0.

Thanks! This looks like a variant of https://github.com/anmonteiro/gluten/pull/24. @anmonteiro, what do you think about merging that PR? We could then point Dream to the merged commit. Alternatively, we could point Dream to @hannesm's fork that the PR originates from, or 0c9341a64ee7432c7a3e1a5e97b4012fee2775c2 as proposed by @dinosaure here in this PR, but I'm pretty wary of generating even more stack forks and mystery vendored branches than Dream already is using :)

To unblock this, I pulled the Gluten commit into aantron/gluten, branch fork. I'll repoint the submodule to whatever gets merged into upstream Gluten later.


project-wide source indexing

I seemly misunderstand the the 'source.request.indexsource' request, which just returns the annotated structure of sources. However, now it is not important for current basic functionalities.
I close this issue now, but maybe re-open it when we need it.