Add Third-party Native PHP Client to the Doc.

Motivation
I have developed a php client.
Modifications
Add Third-party Native PHP Client to the Doc.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation
Check the box below or label this PR directly.
Need to update docs?

[x] doc
(Your PR contains doc changes)

@ikilobyte Thanks for your contribution.

Thanks for your contribution!
I've renamed your PR title to make it more descriptive.
The PR title will be the content of the release note and the commit message when it gets merged.
Please follow the rules stated in the Pulsar Pull Request Naming Guideline. Thanks! 😊

Thanks! 😊

@dave2wave does this client need to be renamed? (context: https://lists.apache.org/thread/c54wmgq9o7q5wg85wxxbgqb8rwk1v8nh)

@dave2wave does this client need to be renamed? (context: https://lists.apache.org/thread/c54wmgq9o7q5wg85wxxbgqb8rwk1v8nh)

I think that this can proceed. I feel that the PMC will allow these links. Further cleanup should be done. We can discuss on the dev@ list which of these 3rd party client projects are defunct.


added utility snippets

#141
Deep Copy of an Object
This snippet creates a deep copy of an object, which can be useful when you want to create a copy that does not reference the original object.
Caching Function Results
This snippet uses a decorator to cache results of expensive function calls, which can significantly improve performance.

@UTSAVS26 can you merge it i want to raise 1 more pr. Thank you


error when running run.py for prediction

Hey,
at the outset: beautiful effort. It really saved me a lot of time from recoding the whole spancat pipeline.
However, i get the following error when running the run all command. Can you tell me what am I missing? am running your code as is, only difference is to save time/get to prediction faster, am running the training only for 2 epochs. Do you think that might be causing this problem?
(toxic) mitch@D21ML-MMITHUN toxic_spans % python run.py "Enter your toxic sentence here, my apologies for the example, you idiot"
Traceback (most recent call last):
  File "/Users/mitch/research/piranha/toxic_spans/run.py", line 58, in <module>
    run(args.input, spans_key=spans_key, model_reference=args.model_reference)
  File "/Users/mitch/research/piranha/toxic_spans/run.py", line 17, in run
    nlp = spacy.load(model_reference)
  File "/Users/mitch/opt/miniconda3/envs/toxic/lib/python3.10/site-packages/spacy/__init__.py", line 54, in load
    return util.load_model(
  File "/Users/mitch/opt/miniconda3/envs/toxic/lib/python3.10/site-packages/spacy/util.py", line 434, in load_model
    return load_model_from_path(Path(name), **kwargs)  # type: ignore[arg-type]
  File "/Users/mitch/opt/miniconda3/envs/toxic/lib/python3.10/site-packages/spacy/util.py", line 506, in load_model_from_path
    nlp = load_model_from_config(
  File "/Users/mitch/opt/miniconda3/envs/toxic/lib/python3.10/site-packages/spacy/util.py", line 554, in load_model_from_config
    nlp = lang_cls.from_config(
  File "/Users/mitch/opt/miniconda3/envs/toxic/lib/python3.10/site-packages/spacy/language.py", line 1818, in from_config
    nlp.add_pipe(
  File "/Users/mitch/opt/miniconda3/envs/toxic/lib/python3.10/site-packages/spacy/language.py", line 801, in add_pipe
    pipe_component = self.create_pipe(
  File "/Users/mitch/opt/miniconda3/envs/toxic/lib/python3.10/site-packages/spacy/language.py", line 680, in create_pipe
    resolved = registry.resolve(cfg, validate=validate)
  File "/Users/mitch/opt/miniconda3/envs/toxic/lib/python3.10/site-packages/confection/__init__.py", line 728, in resolve
    resolved, _ = cls._make(
  File "/Users/mitch/opt/miniconda3/envs/toxic/lib/python3.10/site-packages/confection/__init__.py", line 777, in _make
    filled, _, resolved = cls._fill(
  File "/Users/mitch/opt/miniconda3/envs/toxic/lib/python3.10/site-packages/confection/__init__.py", line 832, in _fill
    filled[key], validation[v_key], final[key] = cls._fill(
  File "/Users/mitch/opt/miniconda3/envs/toxic/lib/python3.10/site-packages/confection/__init__.py", line 831, in _fill
    promise_schema = cls.make_promise_schema(value, resolve=resolve)
  File "/Users/mitch/opt/miniconda3/envs/toxic/lib/python3.10/site-packages/confection/__init__.py", line 1023, in make_promise_schema
    func = cls.get(reg_name, func_name)
  File "/Users/mitch/opt/miniconda3/envs/toxic/lib/python3.10/site-packages/spacy/util.py", line 139, in get
    raise RegistryError(
catalogue.RegistryError: [E893] Could not find function 'spancat_char_scorer.v1' in function registry 'scorers'. If you're using a custom function, make sure the code is available. If the function is provided by a third-party package, e.g. spacy-transformers, make sure the package is installed in your environment.

Available names: spacy.attribute_ruler_scorer.v1, spacy.entity_linker_scorer.v1, spacy.entity_ruler_scorer.v1, spacy.lemmatizer_scorer.v1, spacy.morphologizer_scorer.v1, spacy.ner_scorer.v1, spacy.overlapping_labeled_spans_scorer.v1, spacy.parser_scorer.v1, spacy.senter_scorer.v1, spacy.spancat_scorer.v1, spacy.tagger_scorer.v1, spacy.textcat_multilabel_scorer.v1, spacy.textcat_scorer.v1```

never mind, i just had to do import scripts.evaluate_char in run.py


dart2native fails

I'm trying to compile a dart script under windows:
dart2native -v log.dart
Generating AOT kernel dill.
Generating AOT snapshot.
Unexpected object (Class with illegal cid, full-aot): 0x1f79ed954b1 Library:'dart:collection' Class: _CompactLinkedHashSet@3220832
Failed to generate native files:
Generating AOT snapshot failed!
The following script is sufficient to generate the error:
Create the script:
log.dart

import 'package:logger/logger.dart';

void main() {
 Log.e();
}

class Log extends Logger {
  Log();

  factory Log.e() {
    var self = Log();
    self.e('a');
    return self;
  }
}

Then run:
dart2native log.dart

OS                       : windows
OS Version               : "Windows 10 Enterprise Evaluation" 10.0 (Build 18363)
Path separator           : \
dart version             : 2.7.1
dart exe path            : c:\program files\dart\dart-sdk\bin\dart.exe
dart path                : c:\program files\dart\dart-sdk\bin\dart.exe : c:\program files\dart\dart-sdk\bin\dart.exe
dart2Native path         : c:\program files\dart\dart-sdk\bin\dart2native.bat : c:\program files\dart\dart-sdk\bin\dart2native.bat
pub get path             : c:\program files\dart\dart-sdk\bin\pub.bat : c:\program files\dart\dart-sdk\bin\pub.bat
Pub cache                : c:\users\user\appdata\roaming\pub\cache
Package Config           : c:\users\user\file:\c:\users\user\dshell.packages

Again, I think you may have intended to file this against https://github.com/dart-lang/sdk.


@orta => iPhone settings

[x] Added scrollview
[x] Custom navigation bar
[x] Transparent overlay for iPad
[x] Tests

@sarahscott you should comment that you've made changes, I only found out about this because I saw it in CI when I was doing something else

updates!

Code is good, just comments now

comments added :+1:

OK, so, enough nit-picking from me, but in the future, can you please make comments multi-line at somewhere around 120 chars ( there's a setting in Xcode for showing a line ) - it makes it hard to read outside of an IDE.

vs

https://github.com/artsy/eigen/blob/master/Artsy/App/ARAppDelegate.h#L7-L13
:+1:


sc-7295 gds user ui: refact addresses component

Scope of changes
sc-7295 gds user ui: refact addresses component
Type of change

[ ] bug fix
[ ] new feature
[ ] documentation
[x] refact

Acceptance criteria
Refactoring of the addresses component
Author checklist

[x] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[ ]  I have recompiled and included new protocol buffers to reflect changes I made
[ ]  I have added new test fixtures as needed to support added tests
[ ]   Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[x]  I have moved the associated Shortcut story to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.

Codecov Report

Merging #645 (2461261) into main (0af2cba) will increase coverage by 0.01%.
The diff coverage is 92.30%.

:exclamation: Current head 2461261 differs from pull request most recent head 45a60bd. Consider uploading reports for the commit 45a60bd to get more accurate results

@@            Coverage Diff             @@
##             main     #645      +/-   ##
==========================================
+ Coverage   46.00%   46.01%   +0.01%
==========================================
  Files         550      552       +2
  Lines       16172    16176       +4
  Branches     1201     1194       -7
==========================================
+ Hits         7440     7444       +4
  Misses       7599     7599
  Partials     1133     1133

Impacted Files
Coverage Δ

web/gds-user-ui/src/components/Addresses/index.tsx
100.00% <ø> (+9.09%)
:arrow_up:

...s-user-ui/src/components/Addresses/AddressList.tsx
88.88% <88.88%> (ø)

...b/gds-user-ui/src/components/Addresses/Address.tsx
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Does ramda work with fluture-js?

R.chain doesn't resolve the Future. It just passes the future back.
import { Future } from 'fluture';
import * as R from 'ramda';
import Result from 'folktale/result';

console.log(
  R.chain(R.identity)
  (
    Future(
      (reject, resolve) => {
        resolve('future');
      }
    ).fold(Result.Error, Result.Ok)
  )
);

Output

Future {

_spawn:

Future {

_computation: [Function],
context: { head: null, tail: [Circular] } },

_actions: {

head: {

mapper: [Function: f1],
context: [Object] },
tail: { head: [Object], tail: [Object] } } }

Expected Output

Result.Ok('future')

Hi @nsinghal90, thanks for the question. Ramda works great with Fluture. The issue with your example is due to how Futures work.
One difference between Futures and Promises is that Futures are lazy. In practical terms, nothing happens with a Future until one calls fork. So what you end up with is still an unexecuted Future, which is the reason for the weird output above.
You can take a look at the documentation for fold and notice that it still returns a Future, and thus does not run the computations.
Also, remember that due to the async nature of Futures, you can't just console.log the result. The result never gets returned to the calling code, but instead gets passed to one of the callbacks passed to fork. To get this code to log the result, you'll need to fork the future, passing in console.log as the callbacks.  You can read about consuming Futures here.
R.pipe(
  R.chain(R.identity),
  R.fold(Result.Error, Result.Ok),
  F.fork(console.error, console.log)
)(F.resolve('future'));


Docker Log Drivers and ELK Stack

Docker supports various types of log drivers as a way of gathering logs from your Docker containers.  One common scenario for docker-elk might be to extend the docker-compose file to add an app to and have the log drivers build into Docker pipe the logs to the ELK stack.  For example you can easily configure logstash with a syslog endpoint and then use the Docker syslog driver to pipe logs to logstash.
The problem appears to be that Docker tries to make a connection to the syslog endpoint before starting the containers for the ELK stack so the whole docker-compose up command fails.  I am wondering if there is any way around this or other alternatives to solving this problem.

Sorry, not so much time actually to work on this project, I'll try to think about an evolution for this.

If I understood correctly, this can be achieved with an environment variable in the compose file. Set up the varibale with something like :
export $IP_LOGSTASH=$(docker-machine ip <elk machine name>)

I have written the docker-elk-deployment project to supports the Elastic Stack 5.2.0+ on swarm mode cluster, and use gelf logging driver to gathering logs from docker containers, anyone who meet the requirement can go to have a try.

@ryanjbaxter as you said the target (syslog in your case) must be up before the other containers can even start. It's a pretty bad design but for prototyping with Compose it should be enough to make your syslog service a dependency of all its consumers with depends_on:.


NTR 5/18/28S_rRNA_gene

Hi,
I would like to request the terms:

5S_rRNA_gene
18S_rRNA_gene
28S_rRNA_gene
is_a rRNA_gene

in PMID:31294478 it is shown that pombe Sen1 locates to 5s rRNA genes, but not to other rRNA genes.

could the S please be capitalized to 5S_rRNA__gene?  The S refers to 'sedimentation coefficient' and these are always uppercase S instead of lowercase.

Yes of course, thanks

updated term request based on community curator input

Hi Stacia, do you need any more information for this? Cheers.

Hi @Antonialock i'm not the one who makes SO changes.  that would be @mikebada or @keilbeck or ...?

Hi @Antonialock,
I apologize that it has taken me so long to get back to looking at term requests.
I see how these terms would be important. I have a little bit of a question about putting all three of these under rRNA_gene(SO:0001637). The other three terms each have something that seems like a parent term under small_subunit_rRNA(SO:0000650) or under large_subunit_rRNA(SO:0000651).
rRNA_5S(SO:0000652)
rRNA_18S(SO:0000407)
rRNA_28S(SO:0000653)
I am thinking that I could perhaps put the three new terms under rRNA_gene and mark them as derived from the three terms that I have listed here. Does that sound reasonable?
Also, could you please comment on my proposed definitions for these terms? I have taken them from the definitions of the terms they will be placed under as derived_from. I would prefer that you check them for accuracy before I add them.
5S_rRNA_gene: A gene which codes for 5S_rRNA, which is a portion of the large subunit of the ribosome in both eukaryotes and prokaryotes.
18S_rRNA_gene: A gene which codes for 18S_rRNA, which functions as the small subunit of the ribosome in eukaryotes.
28S_rRNA_gene: A gene which codes for 28S_rRNA, which functions as a component of the large subunit of the ribosome in eukaryotes.
Thanks,
Dave

Hi Dave,
I'm not an expert on ribosomal components but the defs and additional parentage sound fine to me.
Many thanks.

Hi Dave,
From the 2005 SO publication:
https://genomebiology.biomedcentral.com/articles/10.1186/gb-2005-6-5-r44
"If the commonly used name begins with a number, such as 28S RNA, the stem is moved to the front - for example, RNA_28S."
So the gene terms should be rRNA_28S_gene, similar to the rRNA terms, unless that's no longer a rule. I do see some other terms that start with numbers, so maybe that rule is defunct.
Also, while you're at it, how about adding gene terms for the rest of the specific rRNA types:
rRNA_5_8S_gene
rRNA_21S_gene
rRNA_25S_gene
rRNA_23S_gene
rRNA_16S_gene
-Terence

Thank you, Terrence, for pointing out that I was not using the correct name convention. The new terms are all is_a: SO:0001637 ! rRNA_gene. Here are the terms that were added:
id: SO:0002236
name: rRNA_18S_gene
def: "A gene which codes for 18S_rRNA, which functions as the small subunit of the ribosome in eukaryotes." []
relationship: derives_from SO:0000407 ! rRNA_18S
id: SO:0002237
name: rRNA_16S_gene
def: "A gene which codes for 16S_rRNA, which functions as the small subunit of the ribosome in prokaryotes." []
relationship: derives_from SO:0001000 ! rRNA_16S
id: SO:0002238
name: rRNA_5S_gene
def: "A gene which codes for 5S_rRNA, which is a portion of the large subunit of the ribosome in both eukaryotes and prokaryotes." []
relationship: derives_from SO:0000652 ! rRNA_5S
id: SO:0002239
name: rRNA_28S_gene
def: "A gene which codes for 28S_rRNA, which functions as a component of the large subunit of the ribosome in eukaryotes." []
relationship: derives_from SO:0000653 ! rRNA_28S
id: SO:0002240
name: rRNA_5_8S_gene
def: "A gene which codes for 5_8S_rRNA (5.8S rRNA), which functions as a component of the large subunit of the ribosome in eukaryotes." []
relationship: derives_from SO:0000375 ! rRNA_5_8S
id: SO:0002241
name: rRNA_21S_gene
def: "A gene which codes for 21S_rRNA, which functions as a component of the large subunit of the ribosome in mitochondria." []
relationship: derives_from SO:0001171 ! rRNA_21S
id: SO:0002242
name: rRNA_25S_gene
def: "A gene which codes for 25S_rRNA, which functions as a component of the large subunit of the ribosome in some eukaryotes." []
relationship: derives_from SO:0001002 ! rRNA_25S
id: SO:0002243
name: rRNA_23S_gene
def: "A gene which codes for 23S_rRNA, which functions as a component of the large subunit of the ribosome in prokaryotes." []
relationship: derives_from SO:0001001 ! rRNA_23S
The changes have been pushed to GitHub. The SO Browser should be updated within about 24 hours.
Thanks,
Dave


0xMakka/1089 build retirement receipt page based on new designs

Description
Left to complete:

[x] View Carbonmark Profile links
[x] Create your own retirement link on desktop
[x] Provided by Carbonmark.com link
[x] Get started today link
[x] Icon on Download PDF button

Related Ticket
Resolves #1089
Changes

Checklist

[x] I have run npm run build-all without errors
[x] I have formatted JS and TS files with npm run format-all

@0xMakka I made a mistake on this requirement now that I see it in implementation:

VIEW CARBONMARK PROFILE link CTA takes user to https://www.carbonmark.com/users/login or https://www.carbonmark.com/users/[user] if logged in

Please change so VIEW CARBONMARK PROFILE link CTA takes user to https://www.carbonmark.com/users/[user] always.

hey @0xMakka if you can address the change requested from atmos then I can re-test from UI perspective. thanks!

hey @0xMakka if you can address the change requested from atmos then I can re-test from UI perspective. thanks!

Hey @jabby09 the changes requested by atmos for the MCO2 links are in there now so think it should be good to retest. 👍

"MUTABLE" should be "IMMUTABLE". This is also an error in the Figma design.

"Carbonmark.com" has a broken link.

I think "ASSET RETIRED" is a bit inaccurate here. We are not retiring BCT, but we are redeeming a tco2 from BCT and then retiring the tco2.

MCO2 "Project Name" is just "MCO2", which looks too bare. I think we should add a brief explanation as a "Project Description", and perhaps remove "Project Name" instead.
Something along the lines of "To learn more about the projects behind the MCO2 token, please visit ..."

The LinkedIn share button doesn't work for me.

When the user doesn't have a CM profile (example), the button "VIEW CARBONMARK PROFILE" points to "/users/undefined", which is a broken link.

6. example

Thanks @ShimonD-KlimaDAO for the feedback :)

This should now be updated.
I've Updated to always point to carbonmark.com and not the url based on the environment.
Should this then be changed to something like "Asset Redeemed"? @jabby09 any thoughts?
Yeah, I agree with this - can update this text to whatever the recommendation is from you guys. 👍
Linkedin share button works for me, are you seeing an error message or something? (See attached screenshot)
What is the recommendation here, should we be hiding the "View carbonmark profile" link if the beneficiary doesn't have a carbonmark profile or hiding the full beneficiary card information. I know in the designs it says if a beneficiary/beneficiary address doesn't exist to hide the card - but maybe this falls under that requirement also? @jabby09

Thanks @ShimonD-KlimaDAO for the feedback :)

This should now be updated.
I've Updated to always point to carbonmark.com and not the url based on the environment.
Should this then be changed to something like "Asset Redeemed"? @jabby09 any thoughts?
Yeah, I agree with this - can update this text to whatever the recommendation is from you guys. 👍
Linkedin share button works for me, are you seeing an error message or something? (See attached screenshot)
What is the recommendation here, should we be hiding the "View carbonmark profile" link if the beneficiary doesn't have a carbonmark profile or hiding the full beneficiary card information. I know in the designs it says if a beneficiary/beneficiary address doesn't exist to hide the card - but maybe this falls under that requirement also? @jabby09

thanks @ShimonD-KlimaDAO and @0xMakka I'm working on getting answers for 3, 4, and 6
LinkedIn also works for me

A note on 3 "asset retired": Just beware that it's also possible to retire an asset you own directly, in this case this field should be empty (hidden?). If you need me to find an example of a retirement that didn't come from a pool let me know and I'll find one for you. I personally would be OK with removing this field altogether.

Code looks good, I only see 1 change worth mentioning. We can do it in a follow-up PR. I'm going to run format-all on staging in a separate PR and merge it first, to remove all the unrelated formatting changes that snuck in this PR.

@Atmosfearful thanks for the review - yeah I realised this after I ran format-all on the project I got all the carbonmark-api changes because it wasn't formatted on staging.

A note on 3 "asset retired": Just beware that it's also possible to retire an asset you own directly, in this case this field should be empty (hidden?). If you need me to find an example of a retirement that didn't come from a pool let me know and I'll find one for you. I personally would be OK with removing this field altogether.

Yeah if you had an example that would be great so I can actually cater for this case. I'm happy to remove this field if that's what you guys agree on.

There is no need to lazy load this anymore since we aren't loading the PDF library anymore. Just render the button directly

Yeah good shout, will address this as part of this PR as I'm waiting on answers from jabby on 3, 4, and 6 so will most likely have to make some changes here regardless. 👍

@0xMakka It looks like it correctly says "TCO2" already 🔥
https://carbonmark-git-makka-1089-new-retirement-receipt-page-klimadao.vercel.app/retirements/atmosfearful.klima/6

As for 6 - "view carbonmark profile" button - the carbonmark application is able to resolve any 0x address to a skeleton profile page, for example here is markcuban.klima - https://carbonmark-git-makka-1089-new-retirement-receipt-page-klimadao.vercel.app/users/0x293ed38530005620e4b28600f196a97e1125daac
So as a simple fix I will sneak a commit in this PR that always passes the beneficiary address to the URL, that way we can always resolve a link.
But I think we should revisit this based on discussions we had in our product sync meeting today-- if profiles are only for sellers, it doesn't make much sense to show a seller profile picture and profile on a retirement page...

I've also added a commit to address this @jabby09 https://github.com/KlimaDAO/klimadao/pull/1140#pullrequestreview-1442491203


Changed the structure of io/files and io/folder

this is mainly to tidy up the reading of granules and their sidecars.
I here mimic the approach in STAREMaster_py.
Adding further products should be very simple now, and with this approach we can even create a catalogue of a folder of a mix of different granules.

there seems to be a mistake in the requirements.txt. Closing this and opening a new one.


error when launching my project

i am trying to recreate your project but i am getting this error when entering carPlay mode:
Application does not implement CarPlay template application lifecycle methods in its scene delegate.
How can i solve it? Your project dont have a sceneDelegate and it dont crash

I added the entitlement com.apple.developer.carplay-maps and it dont crash. I am not even using maps but maybe it need one kind of entitlement


Add discardableResult to XMLParser.parse

On macOS you don't get a warning when discarding the result of this
function. It isn't written in Swift so it doesn't have this attribute,
so I'm assuming it's Objective-C import semantics that imply allowing
discarding it without warnings.

@swift-ci please test

Please select the main branch as base instead of master. This is the new required check for PRs.

changed, thanks

@swift-ci please test

The CI requirement for main is stuck for some reason 😢

should I force push something?

IDK, I'd let the reviewers decide, they can probably force the check if anything.

There was a discussion about this in slack, the tl;dr is that Apple folks don't want this unless there's a plan to annotate all APIs that suffer from this issue.

There was a discussion about this in slack, the tl;dr is that Apple folks don't want this unless there's a plan to annotate all APIs that suffer from this issue.


iOS error ？

dropdown_button2: 1.1.1  #下拉框

You're using old flutter version "version 2.2.3" and the package depends on some core flutter fields which has been added after that version like "DropdownMenuItem".
Please update your flutter version to some newer version. You can upgrade to latest version by entering "flutter upgrade" in the terminal.


Echarts not reflow

When used Gmap in a container,  after Gmap is loaded and clicked full screen button ,the Canvas not reflow.
Canvas is not the same size as windows .

How I solved it in project.

<template>

</template>

// Idle
this.gmap = this.chart.getModel().getComponent('gmap').getGoogleMap()
this.gmap.addListener('idle', () => {
  const domGmstyle = document.getElementsByClassName('gm-style')[0] as HTMLDivElement
  const domGmwrap = document.getElementById('gMap') as HTMLDivElement
  if(domGmstyle.offsetHeight === window.innerHeight
    && domGmstyle.offsetWidth === window.innerWidth
  ){ // Fullscreen
    domGmwrap.style.position = 'fixed'
    this.chart.resize()
  }
})

Could it be fixed by #15?

Could it be fixed by #15?

Yes, it fixed.

Could it be fixed by #15?

It fixed.


Broken page

Summary of problem or feature request
when I tried to access this page
http://datatables.yajrabox.com/html/columns
I get this
FatalErrorException in HtmlBuilderController.php line 53: Call to undefined method Yajra\Datatables\Facades\Datatables::getRequest()

@vipindasks, thanks for reporting. Demo fixed.

@yajra Thank you so much. :-)


Add cron jobs for all apps that require them

https://trello.com/c/ppfM0Sw8/1025-ensure-scheduled-tasks-by-the-whenever-gem-can-run

Do any of these jobs assume that they're writing to the same filesystem as the webserver/worker process like in EC2?

They all appear to use external caches or databases, except this report creation task. I'm not sure what this report is needed for, so I'll do some investigation.

Have you checked that they run successfully? (no need to add monitoring though as they're currently unmonitored)

Not all of them. I was planning on merging this in and letting them run at their usual times, then check them in the morning

Do you have a plan to keep these cronjobs in sync with the applications in the meantime? i.e. if a developer adds/changes/removes a cronjob, causing the cronjobs here to differ from what the app is requesting via the whenever gem.

Once EC2 is switched off, this repo should become the source of truth for these cron jobs. We shouldn't need to use the whenever gem anymore, since we won't be using it anywhere for crontab generation. Until then, we could definitely get into a situation where more jobs are added and we don't notice, but we can just do another check before switching EC2 off (sourcegraph makes this very easy)

Cool, all SGTM 👍


App crashes when trying to display notifications

Notifications don't work on my S22 so I left the app open while waiting for one and the result is a crash every time. I already tried clearing the cache and reinstalling.

Duplicate of #62, I just forgot to release the fix.


New Fearture Needed: an alias name for hightlight color.

As you can see, the name for "Alternative Highlight Colors"  is fixed while it was created. I now have to create duplicate colors for those colors which refer to same meaning.So an alias name, sometimes, can be convenient.

I disagree. Why not consider first the name of the color and then create it? Why would you need multiple names for the same color? This feature won't be added.


Use Box<&dyn Error> in Error::DataIntegrityError type

Does not trivially work with Eq and other derives


Could not load file or assembly 'Microsoft.Practices.Unity.Configuration' When unityContainer.LoadConfiguration() is used.

Hi,
I have created a very simple console application to test the Unity IoC by using unity container configuration from the app.config.
During the application initialization (unityContainer.LoadConfiguration();) the unity throws an exception: "Could not load file or assembly 'Microsoft.Practices.Unity.Configuration'"
The same application works with Unity 3.5 but not with Unity 5.5.0 with .NET 4.6.2.
Applicaton structure:
namespace ConsoleApp1
{
    public interface ILogger    {   }
    public class EventLogLogger : ILogger    {   }
}

App.config
<?xml version="1.0" encoding="utf-8" ?>
<configuration>
  <configSections>
    <section name="unity" type="Microsoft.Practices.Unity.Configuration.UnityConfigurationSection, Microsoft.Practices.Unity.Configuration" />
  </configSections>
  <unity xmlns="http://schemas.microsoft.com/practices/2010/unity">
    <namespace name="ConsoleApp1" />
    <container>
      <register type="ILogger" mapTo="EventLogLogger" />
    </container>
  </unity>
</configuration>

Unity_IoC_With_App.Config.zip

DLL name has changed. You should use this:

``````

Thanks, indeed it solved the problem.


New version: NeXLMatrixCorrection v0.3.5-dev

Registering package: NeXLMatrixCorrection
Repository: https://github.com/usnistgov/NeXLMatrixCorrection.jl
Created by: @NicholasWMRitchie
Version: v0.3.5-dev
Commit: 9124faa61f367e6194705fce2cd0af39a773ad8b
Git reference: v0.3.5
Description: EPMA matrix correction algorithms
Release notes:

Optimize Hyperspectrum quantify(...)

Please block


Katie/continuing work

Review status: 0 of 14 files reviewed at latest revision, 12 unresolved discussions, some commit checks failed.

arcs/calc_iaa.py, line 10 [r3] (raw file):
I'm inclined to use Pandas here rather than writing this logic explicitly. My sense is that it makes the code less error-prone, more concise, and easier to maintain. Thoughts?

arcs/calc_iaa.py, line 31 [r3] (raw file):
Use Pandas, as suggested above?

arcs/collection.py, line 130 [r3] (raw file):
Why is this gross?

arcs/collection.py, line 213 [r3] (raw file):
Should we do this at the module-level?

arcs/collection.py, line 295 [r3] (raw file):
Remove?

arcs/download_crowdflower.py, line 36 [r3] (raw file):
Remove?

arcs/download_crowdflower.py, line 37 [r3] (raw file):
Remove?

arcs/download_crowdflower.py, line 55 [r3] (raw file):
Remove?

arcs/download_crowdflower.py, line 63 [r3] (raw file):
Maybe we should report these in the output of this script?

arcs/measure_ndcg.py, line 12 [r3] (raw file):
Any reason not to make this a top-level function? It'll make it more testable.

arcs/measure_ndcg.py, line 15 [r3] (raw file):
I'm inclined to use the variant described here: https://en.wikipedia.org/wiki/Discounted_cumulative_gain#Discounted_Cumulative_Gain, since (according to Wikipedia) it's the industry standard and it simplifies this function somewhat since it can be done using the built-in sum:
def dcg(judgments):
    return sum([(2**j - 1) / log(i + 1, 2) for (i, j) in enumerate(judgments)])

arcs/measure_ndcg.py, line 31 [r3] (raw file):
We should (at some point) move this out into a unit test.

Comments from the review on Reviewable.io

Review status: 0 of 15 files reviewed at latest revision, 12 unresolved discussions, some commit checks failed.

arcs/calc_iaa.py, line 10 [r3] (raw file):
Agreed.

arcs/calc_iaa.py, line 31 [r3] (raw file):
Ditto

arcs/collection.py, line 130 [r3] (raw file):
Oh, I really don't like using a list comprehension that doesn't output anything, but I suppose a little part of me finds it satisfying, so I do it.

arcs/collection.py, line 213 [r3] (raw file):
probably.

arcs/collection.py, line 295 [r3] (raw file):
Sure. Though more logging doesn't ever hurt.

arcs/download_crowdflower.py, line 36 [r3] (raw file):
certainly.

arcs/download_crowdflower.py, line 37 [r3] (raw file):
indubitably.

arcs/download_crowdflower.py, line 55 [r3] (raw file):
Instead of commenting out one line and uncommenting another, this should be a flag (like -c --cache to load from json)

arcs/download_crowdflower.py, line 63 [r3] (raw file):
This, and throwing away the 1-values at the bottom here. This whole thing ought to also perhaps be in a function named for what it does (get the ndcg for each domain, query)

arcs/measure_ndcg.py, line 15 [r3] (raw file):
That...doesn't quite work, does it? At i = 1, you're dividing by log(1) = 0. Ah, I see, the mathematical notation is 1-indexed. I think if we just change it to
sum([(2**j - 1) / log(i + 2, 2) for (i, j) in enumerate(judgments)])

that'll work. I'm okay with using that version. I didn't notice that section I suppose :-D

arcs/measure_ndcg.py, line 31 [r3] (raw file):
We should (at some point) have unit tests :-)

Comments from the review on Reviewable.io

arcs/collection.py, line 130 [r3] (raw file):
Oh, the comment applies to this?
    [dom_counts[d].add(q) for d, q, rl in filtered]

If so, I agree with you.

Comments from the review on Reviewable.io

arcs/collection.py, line 295 [r3] (raw file):
No, but it can be confusing. I'm not clear what prefacing args w/ "Namespace:" is telling us. Maybe it should be "Parameters:"?

Comments from the review on Reviewable.io

arcs/measure_ndcg.py, line 15 [r3] (raw file):
One additional thought. We might actually want to take two iterables as input. One for the positions of each judged result, and the other for the actual judgments. This seems more precise than simply using enumerate, since it's possible that a particular a result will be filtered out (for being in a foreign language, for example).

Comments from the review on Reviewable.io

Review status: 0 of 15 files reviewed at latest revision, 12 unresolved discussions, some commit checks failed.

arcs/measure_ndcg.py, line 15 [r3] (raw file):
How about an indexed dict?
{1:2, 2:3, 3:4}

that's what I ended up with.

Comments from the review on Reviewable.io

arcs/measure_ndcg.py, line 15 [r3] (raw file):
The indexed dict approach is totally reasonable, but I'm partial to taking simple array-like iterables for integration w/ pandas and numpy.

Comments from the review on Reviewable.io

Review status: 0 of 15 files reviewed at latest revision, 12 unresolved discussions, some commit checks failed.

arcs/collection.py, line 130 [r3] (raw file):
left in for now

Comments from the review on Reviewable.io


updateSchemas should be called after all schemas are defined, not before them!

I recently joined a project that uses papr. It's a great tool. But the first time I looked at the example in the "about papr" section in https://plexinc.github.io/papr, my first reaction was, why are they calling updateSchemas before defining the schema? how can it update the schemas if no schemas are defined yet? Maybe I'm missing something?!
Anyway, I tested the sample code in both cases, calling updateSchemas before and after defining the schema and it seems that I'm right! When I call updateSchemas before defining the schema, no validation happens when I try to insert a document. Although I should say the type system is pretty robust and acts as a kind of validation but without it, you can insert a document that does not match the schema.
Only when I call updateSchemas after defining the schema, the validation actually works.
I should say that it's hard for me to believe that I'm the first one who noticed there might be a problem here so I'm not quite sure if I'm doing things right but I can't find any problem with what I did because I'm just using the sample code.
Is there really a problem or am I missing something?
Thanks

You're absolutely correct! The sample code is a bit misleading. If you look elsewhere, the docs clearly state that:

Whenever a change is made to a papr schema, updateSchema will need to be called in order to update the JSON Schema validator on the collection.

But I agree, the sample code should be updated to reflect this (or at least we could add a comment)


Make sure the backup object store is empty when bootstrapping a new cluster

When bootstrapping a new cluster from a recovery object store, and the target cluster has a backup object store configured, invoke barman-cloud-check-wal-archive to make sure that the destination backup object store is empty.
This prevents users from erroneously overwrite an existing object store. The same concept can be probably extended to any bootstrap phase.

@gbartolini I found an issue, we can't check the "future" object store from the operator since we don't have the barman-cloud-check-wal-archive command, this can only run inside a cluster already created, meaning that we can check during the init job only, and that will create at least one pod, it's that solution ok with you? It's the only solution I can see by now since only those container will have the Barman tools we need.

@gbartolini After talking with @mnencia I have a cleared idea and I'll be moving this soon

I believe https://github.com/cloudnative-pg/cloudnative-pg/issues/62#issuecomment-1113878444 applies to this issue as well?

Taking the issue back - I think I can add a quick E2E

The existing suite is not testing cluster restoring from another and having its own backup location.
We might want to add E2E for this case...

I have added an issue to add E2E tests for the new backup + restore scenarios.
For the moment this PR could be tested manually. Moving to second review.


iOS support?

pydantic and pydantic-core do not appear to build on iOS. Do you plan to add support for it?
Or if it is supported, where can I find documentation for it?
I am using the Beeware mobile-forge and Briefcase and can't generate a binary wheel for pydantic-core, which means I can't install it on iOS.
Many libraries use Pydantic, which blocks them from being used on iOS. For example, spaCy, Presidio, thinc, and others.

Here's my mobile-forge recipe:
package:
  name: pydantic-core
  version: 2.14.6

Here's the output from the forge build:
$ forge iphoneos:12.0:arm64 pydantic-core
================================================================================
Building pydantic-core 2.14.6 for ios_12_0_iphoneos_arm64
================================================================================

[venv3.10-ios_12_0_iphoneos_arm64] Unpack sources
Unpacking downloads/pydantic-core-2.14.6.tar.gz...

[venv3.10-ios_12_0_iphoneos_arm64] Apply patches
No patches to apply.

[venv3.10-ios_12_0_iphoneos_arm64] Create clean build environment
Creating venv3.10-ios_12_0_iphoneos_arm64...

Verifying cross-platform environment...
done.

Cross platform-environment venv3.10-ios_12_0_iphoneos_arm64 created.

[venv3.10-ios_12_0_iphoneos_arm64] Install forge host requirements
No host requirements.

[venv3.10-ios_12_0_iphoneos_arm64] Install forge build requirements
No build requirements.

[venv3.10-ios_12_0_iphoneos_arm64] Install pyproject.toml build requirements

Looking in links: /Users/adam/src/mobile-forge/dist
Collecting build
Using cached build-1.0.3-py3-none-any.whl (18 kB)
Collecting wheel
Using cached wheel-0.42.0-py3-none-any.whl (65 kB)
Processing ./dist/maturin-1.4.0-cp310-cp310-ios_12_0_iphoneos_arm64.whl
Collecting typing-extensions!=4.7.0,>=4.6.0
Using cached typing_extensions-4.9.0-py3-none-any.whl (32 kB)
Collecting pyproject_hooks
Using cached pyproject_hooks-1.0.0-py3-none-any.whl (9.3 kB)
Collecting tomli>=1.1.0
Using cached tomli-2.0.1-py3-none-any.whl (12 kB)
Collecting packaging>=19.0
Using cached packaging-23.2-py3-none-any.whl (53 kB)
Installing collected packages: wheel, typing-extensions, tomli, packaging, pyproject_hooks, maturin, build
Successfully installed build-1.0.3 maturin-1.4.0 packaging-23.2 pyproject_hooks-1.0.0 tomli-2.0.1 typing-extensions-4.9.0 wheel-0.42.0

Looking in links: /Users/adam/src/mobile-forge/dist
Collecting build
Using cached build-1.0.3-py3-none-any.whl (18 kB)
Collecting wheel
Using cached wheel-0.42.0-py3-none-any.whl (65 kB)
Collecting maturin<2,>=1
Downloading maturin-1.4.0-py3-none-macosx_10_12_x86_64.macosx_11_0_arm64.macosx_10_12_universal2.whl (15.7 MB)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 15.7/15.7 MB 33.5 MB/s eta 0:00:00
Collecting typing-extensions!=4.7.0,>=4.6.0
Using cached typing_extensions-4.9.0-py3-none-any.whl (32 kB)
Collecting pyproject_hooks
Using cached pyproject_hooks-1.0.0-py3-none-any.whl (9.3 kB)
Collecting tomli>=1.1.0
Using cached tomli-2.0.1-py3-none-any.whl (12 kB)
Collecting packaging>=19.0
Using cached packaging-23.2-py3-none-any.whl (53 kB)
Installing collected packages: wheel, typing-extensions, tomli, packaging, pyproject_hooks, maturin, build
Successfully installed build-1.0.3 maturin-1.4.0 packaging-23.2 pyproject_hooks-1.0.0 tomli-2.0.1 typing-extensions-4.9.0 wheel-0.42.0

* Getting build dependencies for wheel...
* Building wheel...
Running `maturin pep517 build-wheel -i /Users/adam/src/mobile-forge/build/cp310/pydantic-core/2.14.6/venv3.10-ios_12_0_iphoneos_arm64/venv3.10-ios_12_0_iphoneos_arm64/bin/python --compatibility off`
💥 maturin failed
Caused by: Cargo metadata failed. Do you have cargo in your PATH?
Caused by: No such file or directory (os error 2)
Error: command ['maturin', 'pep517', 'build-wheel', '-i', '/Users/adam/src/mobile-forge/build/cp310/pydantic-core/2.14.6/venv3.10-ios_12_0_iphoneos_arm64/venv3.10-ios_12_0_iphoneos_arm64/bin/python', '--compatibility', 'off'] returned non-zero exit status 1

ERROR Backend subprocess exited when trying to invoke build_wheel
********************************************************************************
Failed build: pydantic-core 2.14.6 for iphoneos 12.0 on arm64
********************************************************************************
Traceback (most recent call last):
  File "/Users/adam/src/mobile-forge/src/forge/build.py", line 267, in build
    self._build()
  File "/Users/adam/src/mobile-forge/src/forge/build.py", line 518, in _build
    self.cross_venv.run(
  File "/Users/adam/src/mobile-forge/src/forge/cross.py", line 356, in run
    return subprocess.run(logfile, *args, **self.cross_kwargs(kwargs))
  File "/Users/adam/src/mobile-forge/src/forge/subprocess.py", line 49, in run
    raise stdlib_subprocess.CalledProcessError(return_code, args)
subprocess.CalledProcessError: Command '(['python', '-m', 'build', '--no-isolation', '--wheel', '--outdir', '/Users/adam/src/mobile-forge/dist'],)' returned non-zero exit status 1.
Failed builds for:
 * pydantic-core (default version) (ios_12_0_iphoneos_arm64)

It appears that this is because maturin does not support iOS. If I run the command that the bdist_wheel script is trying to run, I get this:
$ maturin pep517 build-wheel -i /Users/adam/src/mobile-forge/build/cp310/pydantic-core/2.14.6/venv3.10-ios_12_0_iphoneos_arm64/venv3.10-ios_12_0_iphoneos_arm64/bin/python --compatibility off --target aarch64-apple-ios
📦 Including license file "/Users/adam/src/mobile-forge/build/cp310/pydantic-core/2.14.6/LICENSE"
🍹 Building a mixed python/rust project
🔗 Found pyo3 bindings
💥 maturin failed
  Caused by: The operating system Ios is not supported

I looked at trying to add iOS support to maturin, but my changes don't seem to work. Here's the branch with my iOS changes:
https://github.com/adamfeuer/maturin/tree/feature/ios-support
When I use my changed maturin, I get the following output:
$ ~/src/maturin/target/release/maturin pep517 build-wheel -i python3.10 --compatibility off --target aarch64-apple-ios
📦 Including license file "/Users/adam/src/mobile-forge/build/cp310/pydantic-core/2.14.6/LICENSE"
🍹 Building a mixed python/rust project
🔗 Found pyo3 bindings
🐍 Found CPython 3.10 at /Users/adam/src/mobile-forge/venv3.10/bin/python3.10
📡 Using build options features, bindings from pyproject.toml
   Compiling autocfg v1.1.0
   Compiling proc-macro2 v1.0.69
   Compiling quote v1.0.29
   Compiling unicode-ident v1.0.10
   Compiling target-lexicon v0.12.9
   Compiling python3-dll-a v0.2.9
   Compiling once_cell v1.18.0
   Compiling libc v0.2.147
   Compiling static_assertions v1.1.0
   Compiling heck v0.4.1
   Compiling version_check v0.9.4
   Compiling cfg-if v1.0.0
   Compiling lexical-util v0.8.5
   Compiling rustversion v1.0.13
   Compiling parking_lot_core v0.9.8
   Compiling tinyvec_macros v0.1.1
   Compiling smallvec v1.11.1
   Compiling scopeguard v1.1.0
   Compiling tinyvec v1.6.0
   Compiling ahash v0.8.6
   Compiling num-traits v0.2.16
   Compiling lock_api v0.4.10
   Compiling num-integer v0.1.45
   Compiling memoffset v0.9.0
   Compiling num-bigint v0.4.4
   Compiling lexical-parse-integer v0.8.6
   Compiling lexical-write-integer v0.8.5
   Compiling memchr v2.6.3
   Compiling pyo3-build-config v0.20.0
   Compiling serde v1.0.190
   Compiling syn v2.0.38
   Compiling aho-corasick v1.0.2
   Compiling lexical-write-float v0.8.5
   Compiling lexical-parse-float v0.8.5
   Compiling unicode-normalization v0.1.22
   Compiling getrandom v0.2.10
   Compiling parking_lot v0.12.1
   Compiling equivalent v1.0.1
   Compiling percent-encoding v2.3.0
   Compiling serde_json v1.0.108
   Compiling hashbrown v0.14.0
   Compiling indoc v2.0.4
   Compiling unicode-bidi v0.3.13
   Compiling regex-syntax v0.8.2
   Compiling zerocopy v0.7.20
   Compiling unindent v0.2.3
   Compiling idna v0.4.0
   Compiling indexmap v2.0.0
   Compiling form_urlencoded v1.2.0
   Compiling lexical-core v0.8.5
   Compiling ryu v1.0.14
   Compiling itoa v1.0.8
   Compiling base64 v0.21.5
   Compiling url v2.4.1
   Compiling uuid v1.5.0
   Compiling pyo3-ffi v0.20.0
   Compiling pyo3 v0.20.0
   Compiling pydantic-core v2.14.6 (/Users/adam/src/mobile-forge/build/cp310/pydantic-core/2.14.6)
   Compiling pyo3-macros-backend v0.20.0
error: failed to run custom build command for `pyo3-ffi v0.20.0`

Caused by:
  process didn't exit successfully: `/Users/adam/src/mobile-forge/build/cp310/pydantic-core/2.14.6/target/release/build/pyo3-ffi-b9fbb289a993548c/build-script-build` (exit status: 1)
  --- stdout
  cargo:rerun-if-env-changed=PYO3_CROSS
  cargo:rerun-if-env-changed=PYO3_CROSS_LIB_DIR
  cargo:rerun-if-env-changed=PYO3_CROSS_PYTHON_VERSION
  cargo:rerun-if-env-changed=PYO3_CROSS_PYTHON_IMPLEMENTATION
  cargo:rerun-if-env-changed=PYO3_NO_PYTHON

  --- stderr
  error: PYO3_CROSS_PYTHON_VERSION or an abi3-py3* feature must be specified when cross-compiling and PYO3_CROSS_LIB_DIR is not set.
warning: build failed, waiting for other jobs to finish...
💥 maturin failed
  Caused by: Failed to build a native library through cargo
  Caused by: Cargo build finished with "exit status: 101": `env -u CARGO PYO3_ENVIRONMENT_SIGNATURE="cpython-3.10-64bit" PYO3_PYTHON="/Users/adam/src/mobile-forge/venv3.10/bin/python3.10" PYTHON_SYS_EXECUTABLE="/Users/adam/src/mobile-forge/venv3.10/bin/python3.10" "cargo" "rustc" "--features" "pyo3/extension-module" "--target" "aarch64-apple-ios" "--message-format" "json-render-diagnostics" "--manifest-path" "/Users/adam/src/mobile-forge/build/cp310/pydantic-core/2.14.6/Cargo.toml" "--release" "--lib" "--crate-type" "cdylib"

Do you know where the problem lies? In maturin, in pydantic-core, or both? I'm happy to work on this, I'm just not sure where to go from here.

If rust/maturin/pyo3 can build for iOS, we would absolutely support it. Until then there's nothing we can do.
When I checked last, there were virtually no (10s out of 250m) of installs of pydantic v1 on iOS. So it's very rarely used.
You might try the warm build which should work.

@adamfeuer I think the best course of action is to start with an issue / PR in maturin to understand how it should detect the mobile-forge build configuration, and then we can proceed from there to land changes in PyO3 also if needed.

@davidhewitt Ah! Thank you! That is great! If I need to see the source code, is it on the v1 branch?
Do you know how the other iOS users installed pydantic v1?

v1 is pure python. (It was optimized by Cython at build time, but you can skip that if needed.)
I believe the maintenance branch is at https://github.com/pydantic/pydantic/tree/1.10.X-fixes
Note that we don't intend to support v1 much longer, and we only merge occasional bugfixes now.

V1 is on the 1.10.X-fixes branch. Pydantic-core didn't exist until V2.
I've updated my comment above to say.

You might try the wasm build which should work.

As you'll see here we already have a preview of running pydantic-core with webassembly, if that runs for you in iOS, it might be the easiest solution.

@davidhewitt Pure python, perfect. I just built it for iOS using the mobile-forge, so it looks like that will work. I have to rebuild the other stack of software that uses it now.
Re: the 1.10.x branch not being supported soon, I understand. If I can get this working with 1.10.x, it will solve my immediate problem, and then I can devote some effort to fixing maturin for iOS.
@samuelcolvin I'll try the wasm if the pure python 1.10.x doesn't work.
Thank you both so much for your fast help! I'll post here if I get this working, and let you know what's happening with maturin.

@adamfeuer See https://github.com/PyO3/maturin/issues/1742

@messense Thank you for letting me know about this! Wonderful news!


Can't install documentation plugin via admin dashboard

Describe the bug
When tried to install documentation plugin from administration dashboard its occurs with the error.
Steps to reproduce the behavior

Follow documentation to install Strapi and mongoDB with docker-compose Installing using Docker
Choose Polish Language as a dashboard language (It was automatically selected in my case)
Using English Language installation was successful, only when Polish language (and maybe others) is selected its occurs with errors
Go to Marketplace in Administration Dashboard and try to install Documentation Plugin

Expected behavior
Installed documentation without errors.
Code snippets
Code wasn't modified. Fresh installation.
System

Node.js version: v12.18.2
YARN_VERSION 1.22.4
Strapi version: v3.1.1
Database: MongoDB v4.2.8
Operating system: MacOS Catalina (Strapi and MongoDB installed in docker)

Additional context

Hi @oskar-gmerek thanks for reporting. This one is a tricky One. I would recommend you do it manualy for now as we need to review how we can install a plugin when we need to restart the process and rebuild the admin. Quite a difficult task :)

I installed the documentation plugin without problems after changed language to english. Just reporting something is wrong there. Im not familiar enough yet to find the bug by myself.


Promise rejection type.

Search Terms
Promise Reject Type

Suggestion

Add ability to type Promise rejections.
// Current Promise Constructor Implementation:
new <T>(executor: (resolve: (value?: T | PromiseLike<T>) => void, reject: (reason?: any) => void) => void): Promise<T>;

// Proposed Change:
new <T, E = any>(executor: (resolve: (value?: T | PromiseLike<T>) => void, reject: (reason?: E) => void) => void): Promise<T, E>;

Use Cases

When handling promise rejections, the type any isn't very useful. I would be useful to have the rejection have an actual type without the need to cast or type guard.
Examples

Promise.reject<never, string>('hello world')
  .catch(reason => {
    console.error(reason.length); // `reason` is of type string.
  });

class MyError extends Error {
  // ...
}

Promise.reject<never, MyError>(new MyError(/* ... */))
  .catch(reason => {
    // `reason` is of type MyError.
    const info = reason.getMoreInfo();
    // ...
  });

Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

Looks like duplicate of #6283 and other similar.
See for more details https://github.com/microsoft/TypeScript/issues/6283#issuecomment-240804072

I don't feel like they rejection type always has to be of type any/unknown.
It is obviously possible to detect what the rejection type is when Promise.reject is used. The hard bit would be detecting rejections caused by a throw statement inside an async function. But if we approach it from the perspective of a throw statement just being an alternative type of return statement, it seems possible to me (but probably quite difficult).
I've been having a bit of a play with this to see what it would be like add support. This is what I've got so far just by playing with type definitions.

p2                                         // p2 is of type Promise<number, string>
  .then((v2) =>  void cosnole.log(v2))     // v2 is of type number
  .catch((e2) => void console.error(e2));  // e2 is actually type string | ReferenceError, not string

Why would the type be string | ReferenceError? Isn't TypeScript already be reporting ReferenceError when they happen (2304)? (Or is that a strict mode only thing? If so then this proposal could just be for strict mode.)
playground

Yes, TS will report 2304 in this obvious case however emitted JS code will still have a change to receive both string and ReferenceError. Idea is to show that if there is something between resolve and catch - there are no guarantees that code between them wont throw something else (e.g. failed network request, incorrect normalize function etc.)
Looks like string will be only true for case p2.catch(...) but only if you believe that there are no other errors before resolve/reject.

If myPromise.then(...).catch(...) can't be handled due to this but myPromise.catch(...) could be, imo that's still worth pursuing.

In unsound cases or unknown external code (e.g. computedBoolean) there is always a chance to have behavior that will throw
declare const computedBoolean: (params: any) => boolean; // but in general case might throw
const list = [{id: 111}, {id: 222}];

function firstThreeIds(data: Array<{id: number}>): number[] {
    return [
      data[0].id,
      data[1].id,
      data[2].id // no compile time error, but will throw
    ];
}

const p = new Promise<number[], string>((resolve, reject) => {
  if (computedBoolean(list)) { // can throw before resolve
    resolve(firstThreeIds(list)); // wil throw before resolve
  } else {
    reject("foo");
  }
});

p
  .catch((e) => { /* ... */}); // actually string | TypeError

So it looks like closest thing e can be typed is E | Error in definition (string | Error in current example).
And again if computedBoolean won't throw anything else (not sure which case will be encountered more often).

Could this proposal solve this issue?
https://github.com/microsoft/TypeScript/issues/39846

I just opened #45869.  I think maybe the best-case outcome would be implementing #13219 (throws-clause) and this issue, in which case it would become possible to document Promises that have a rejects type of never, i.e. rejection is not possible.  (Think return Promise.resolve(1);, etc.)
It would be a lot of work for most projects to migrate to throws-clause compliance but once done, the type checker would be able to reason about exception / rejection paths -- I would love to automate that reasoning, rather than getting bitten by orphaned async-function calls and having to hunt them down to fix on my own.

It's not. The rejection type is always unknown

Just encountered working with reason as an any type.  I understand why it doesn't have a specific type, but I guess I'm wondering why it's not instead typed as unknown?  There's a similar situation for the error property of ErrorEvent as it's completely non-standard.
In both situations, it seems that the appropriate type should be unknown, which by definition forces the developer into safe type guards.  Having it typed as any just takes TS out of the picture altogether.

Since this change isn't breaking you could always monkey patch the appropriate .d.ts file with the proposed change.  I think it would be nice if this made it into the official typings.  There still other changes that would need to be made to the compiler to get a good developer experience.  For instance, if you throw inside of a then() block, it would be nice if TS could infer that update the rejection type appropriately.  There was a ticket about adding support for tracking the exception types thrown by functions, but it was closed earlier this year, see https://github.com/microsoft/TypeScript/issues/13219#issuecomment-1515037604.  I'm guessing this proposal is dead-in-the-water for similar reasons.

I don't understand the argument that the rejection type can't be guaranteed.
Nothing can be guaranteed by typescript at runtime. Don't see why rejection type is different.
If I'm telling typescript rejection type is a certain type is because I'm supposed to know what I'm doing. Typescript code is usually full of castings assuming some stuff is defined, etc.. Don't see why this should be any different.

If you read the issues I linked from my previous post (2 years ago!), you'll see some of the reasoning. I think one of the most compelling arguments against allowing manual declaration of a throws or rejects type is that a novice who reads your code is unlikely to realize that you're making an unsafe assertion, along the lines of a typecast.
I'd still really like to see a way to annotate Promises that cannot reject, though. I actually write a lot of async functions with top level try/catch - it's a good pattern to follow when implementing an Express router callback, since the server library will not handle a rejection for you.

I think one of the most compelling arguments against allowing manual declaration of a throws or rejects type is that a novice who reads your code is unlikely to realize that you're making an unsafe assertion, along the lines of a typecast.

In this case is the developer that set the return type the one that needs to guaranteed the thrown exception is the correct type. I'm telling typescript I will only reject with that kind of type.
In typescript is perfectly valid to write
function sum(a: number, b: number): number {
  return 'hello world' and any as number;
}

This is totally valid TS although would fail at runtime, don't see why the developer could not explicitly tells typescript he will be make sure  the rejected value is a certain type. I'm not asking typescript to check thrown types etc. just so the consumers of the promises know the rejected type.

Sorry I wasn't more specific, but I wrote the previous comment from a phone.  I was referring to https://github.com/microsoft/TypeScript/issues/45869#issuecomment-920252246 :

it’s really not obvious that there is an assertion. Experienced TS developers are attuned to look out for unsoundness when they see as type assertions; this proposal makes the unsoundness very easy to overlook.

The point was that, if rejection-types were implemented, I could write
async function lookup(key: string): Promise<string, never> {
  try {
    return maybeGetValueEventually(key);
  } catch {
    logThatMightThrow(key);
    return "fallback";
  }
}

but without comprehensive checked exceptions (a feature that has already been rejected) the type-checker couldn't prove that the function never rejects.
In this example, the first generic argument in the returned-value Promise type (string) is type-checked -- if I wrote return 0 in the catch block instead of return "fallback", it would get flagged as an error -- but the second generic argument (never) is an unsafe assertion that I'm making.  There's no way for the typechecker to determine if the Promise never actually rejects, but there's no cast or as keyword to act as a red flag.  You'd need a pretty deep expertise in TypeScript to actually understand that difference.
I do think this could be handled by a linter rule ("all explicit Promise rejection types require a comment"), and I would still argue that some construct for asserting at its source that a Promise cannot reject is useful enough to merit a certain amount of novice-developer hazard, but I wanted to point out that there is a totally reasonable counterargument here.


Add support for boundaryless regex

ToDo:

[x] Add Filter class
[x] Write tests for Filter class
[x] Add suport for boundaryless regex using Filter class
[x] Write some more tests
[ ] Update README
[x] Update CLI
[x] Add flag that disables boundaryless
[x] Replace re.search()

So far, I have added a Filter class. It behaves just like the Distribution class, although it does not store regexes.
Now, you can initialize Distribution using either another Distribution, a Filter or a filters_dict. The same applies to the Filter class.

This is huge!!!!!!!!!!!!!

forgot to say maybe, but this should integrate with filter system so you can filter say:
only have boundaries on regex below 1.0 rarity etc :))

forgot to say maybe, but this should integrate with filter system so you can filter say:
only have boundaries on regex below 1.0 rarity etc :))

This one is already done.

README left 😱

Breaking CLI changes btw

API docs left🚀

Finally, API docs are updated

Let me review and then merge :)))


sep 2022 iotm - Jurassic Parka - database entries

Add mafia-generated database lines for Sep. 2022 IotM.

Sorry...didn't see gausie's commit until immediately after submitting this.

Sorry...didn't see gausie's commit until immediately after submitting this.

No problem.  Would you consider closing or reconciling the conflicts?

Doh...I meant to do that earlier.


"%" symbol in attributes does not work

This is slightly weird bug. I have a use case where i want to set up some data-* attributes. I am doing it this way:
div#foo data-label="Remaining %"

and this works great. The problem starts when I wrap this string in a function call, for example:
div#foo data-label=gettext("Remaining %")

or
div#foo data-label=escape_javascript("Remaining %")

I am getting the following error:

== Compilation error on file lib/user_interface/views/projects_view.ex ==
** (TokenMissingError) lib/user_interface/templates/projects/show.html.slim:1: missing terminator: " (for string starting at line 1)
(eex) lib/eex/compiler.ex:36: EEx.Compiler.generate_buffer/4
(phoenix) lib/phoenix/template.ex:378: Phoenix.Template.compile/2
(phoenix) lib/phoenix/template.ex:186: Phoenix.Template."-MACRO-before_compile/2-fun-0-"/3
(elixir) lib/enum.ex:1755: Enum."-reduce/3-lists^foldl/2-0-"/3
(phoenix) expanding macro: Phoenix.Template.before_compile/1
lib/user_interface/views/projects_view.ex:1: FooApp.UserInterface.ProjectsView (module)
(elixir) lib/kernel/parallel_compiler.ex:117: anonymous fn/4 in Kernel.ParallelCompiler.spawn_compilers/1

The workaround I am using is to wrap the gettext("Remaining %") into one more function, put that function in a view and call it in the template such as:
div#foo data-label=remaining_label()

where in my view:
def remaining_label do
  gettext("Remaining %")
end

Hi, @hubertlepicki. Thanks for the bug report!
I would suggest to try my fork https://github.com/Rakoth/slime until #113 is merged.

closed by #113


Compatibility to Magento Commerce 2.2.5

Currently the module is not fully compatible to Magento Commerce. If we are installing Magento with this module installed (i.e. during integration tests), we get the following error:
Starting Magento installation:
File permissions check...
[Progress: 1 / 721]
Required extensions check...
[Progress: 2 / 721]
Enabling Maintenance Mode...
[Progress: 3 / 721]
Installing deployment configuration...
[Progress: 4 / 721]
Installing database schema:

  [Zend_Db_Statement_Exception]
  SQLSTATE[42S02]: Base table or view not found: 1146 Table 'xyz.flag' doesn't exist, query was: SELECT `flag`.* FROM `flag` WHERE (`flag`.`flag_code`='staging')

After I removed the module, it worked.
My assumption is that it could be because there is no sequence tag in the module.xml, the module should probably run after the Magento_UrlRewrite module.

Well, for me it was... clearly reproducable, and I reinstalled Magento from an empty database about 20 times, until I found that this extension was causing the error. After that, I reinstalled twice with and twice without this extension to be absolutely sure. Note it's the Commerce edition, not OpenSource.

Hello @avstudnitz.
Yes, sorry. The issue was on extension side. I've fixed issue and released a new version "1.2.0".
Please, update and use this latest version.
Best regards,
Oleg Koval.

Great, thanks a lot!

@avstudnitz Please, upgrade to v1.2.1 - I've fixed compilation issues.

This worked for me on Adobe commerce 2.3
php bin/magento setup:uninstall
rm -rf var/cache/ var/di/ var/page_cache/  generated/  pub/static/
php bin/magento setup:install --cleanup-database ...other options...


MSC3051: A scalable relation format

Signed-off-by: Nicolas Werner<EMAIL_ADDRESS>Rendered
Nheko's implementation for edits uses this released with version 0.8.2 and newer. This falls back transparently to the current format and adds both to events. mtxclient PR (some bugfixes happened at a later date).

Since this has "needs-implementation", I believe I can go ahead and add experimental support to Synapse/Element?

It has an implementation, where do you see "needs-implementation"? :D

oh, i only quickly glanced at this paper and thought it reused m.relates_to. In that case, no special changes in particular are needed indeed.

to add to the above - i just want to know how the implementations should act when e.g. both the array and the relates_to specifies the fact the message is an edit, and whether editing a message should set relates_to for fallback

Since relates_to is the older format, I would use the array, when both are present, since clients will be sending the old format as fallback.

Another thing worth thinking about is encryption (though it could alternatively be discussed in MSC2674). Relation aggregations seem useful, but other than anything needed for that, the server doesn't seem to need to know anything (as discussed in https://github.com/matrix-org/matrix-doc/issues/2678)

Best case you open a thread for every concern. Makes it easier to keep track of what is still open. For that just leave a comment on one of the lines in the proposal.


[react] Drop unsuffixed components or setup suffixed-only components in a dedicated export to prevent auto-import pollution.

Package

[ ] lucide
[ ] lucide-angular
[ ] lucide-flutter
[ ] lucide-preact
[X] lucide-react
[ ] lucide-react-native
[ ] lucide-solid
[ ] lucide-svelte
[ ] lucide-vue
[ ] lucide-vue-next
[ ] Figma plugin
[ ] all JS packages
[ ] site

Description
I've started to use lucide-react recently and the unsuffixed icons (eg. File instead of FileIcon, Link vs. LinkIcon) do mess quite a bit with my workflow as it tends to pollute auto-complete/auto-import for similarly named objects (eg. Typescript type File, or the common Link component from routers), it's confusing and I'm wasting (arguably little time) on this every time I encounter a conflict.
Ideally I think it would be wise to drop the unsuffixed exports before tagging 1.0 as it leads to duplication/confusion but I guess people might not want such breaking change.
So I'm wondering if we could have an optional export without the unsuffixed components:

import { FileIcon, LinkIcon } from "lucide-react/icons";

Then use something like vscode typescript.preferences.autoImportFileExcludePatterns to ignore the base path with every export from polluting autocomplete.
Use cases
I would lead to a better developer experience when importing icons.
Checklist

[X] I have searched the existing issues to make sure this bug has not already been reported.

I'm totally in favour of dropping non-suffixed/prefixed names from our exports starting from v1.0, preferably only keeping either Lucide<IconName> or <IconName>Icon.

Hmm, I don't know about dropping it. It's just a matter of taste I think.
Also, that means that everyone who updates to v1 should refactor their project and rename all their imports and icon components. Nobody will like that..
But I get your point. When using the library I think I ran into this a few times.
So the solution I want to propose is the ability to change this by overriding the type declarations.
In your project, you could create a typing file like lucide-react.d.ts or whatever you want to name it.
And include this content:
// lucide-react.d.ts
declare module "lucide-react" {
  export * from "lucide-react/dist/lucide-react.prefixed";
  // This will re-export all exports but only includes the icons with prefixes and other library types
}

In every package, we can create a new entry file that includes different import styles.

lucide-react/dist/lucide-react.prefixed, only includes export like LucideHome, LucideX, LucideMenu
lucide-react/dist/lucide-react.suffixed, only includes export like HomeIcon, XIcon, MenuIcon

I've already tested it, and it works. But need to rewrite some things to generate multiple aliases files.

Thanks for writing all this up! The workaround by @ericfennis would be the MVP and perhaps solve already the issue.
Regarding: "Nobody will like that.." I think this is a "do it once and get over it"-migration. In contrast, the "pollution" when using multiple libraries will stay an ongoing issue. And with the drive towards configless project setups, I'd argue that it would be a huge win to just drop unsuffxed/unprefixed icons.

Thanks Eric for guiding me to this issue page.
I am only every using the suffixed version of the icons, like "LinkIcon", to keep them distinguishable from other naming convention like 'Link' from the next/link package.
As you pointed out, I created the file lucide-react.d.ts in the main dir of my NextJS project. As I intent to use the suffixed version, I included the following content:
declare module 'lucide-react' {
	export * from 'lucide-react/dist/lucide-react.suffixed';
	// This will re-export all exports but only includes the icons with prefixes and other library types
}

Typescript is immediately throwing errors, like:
Module '"lucide-react"' has no exported member 'AlertCircleIcon'.

I tried to check node_modules/lucide-react/dist and it seems there is no lucide-react.suffixed folder in there (I have to say though, that I am not well aversed with handling node-modules).
Can you point me in the right direction? Thanks!

@tillka yeah sorry maybe I confused you a bit. This is a proposal for a fix. This is still not implemented yet.

@tillka yeah sorry maybe I confused you a bit. This is a proposal for a fix. This is still not implemented yet.

Oh, thought it was an MVP. Then, you can put me on the wait list for this feature to be implemented. ;)

As React libs like shadcn ui  becoming more popular, component composition will be the default for many devs. I lost embarrassing amount of time on icons auto import.

This auto-import pollution is THE most annoying thing I run into on a daily basis.         "typescript.preferences.autoImportFileExcludePatterns": ["lucide-react"],            just stop exporting icons as Calendar, Section, Container. What should devs call their components?

"typescript.preferences.autoImportFileExcludePatterns": ["lucide-react"],  then  create Icons.tsx , manually    import { Check, ChevronDown } from 'lucide-react'   export const ChevronDownIcon = ChevronDown                ..       Adding a config file for a icon package to thousands of config files frontend project is not ideal.   I already lost enough time because of this.

So I also struggle with this, permanently. We solved with a small vite plugin that reexports only the right icons:
function LucideIconPlugin(): Plugin {
  return {
    name: "lucide-icon-plugin",
    buildStart: async () => {
      const declarations = await readFile(
        "node_modules/lucide-react/dist/lucide-react.d.ts",
        "utf-8",
      );

      const iconNames = new Set(declarations.match(/(?<=as )[a-zA-Z0-9]+Icon/g));
      const jsFile = `export * from "lucide-react/dist/esm/lucide-react";\n`;
      const declarationFile =
        'import { Icon } from "lucide-react";\n' +
        [...iconNames]!
          .map((icon) => `declare const ${icon}: import("lucide-react").LucideIcon;\n`)
          .join("");

      writeFileSync("libs/components/icons.gen.d.ts", declarationFile);
      writeFileSync("libs/components/icons.gen.js", jsFile);
    },
  };
}

This will create:
// icons.gen.js
export * from "lucide-react/dist/esm/lucide-react";

// icons.gen.d.ts
declare const AArrowDownIcon: import("lucide-react").LucideIcon;
declare const AArrowUpIcon: import("lucide-react").LucideIcon;
declare const ALargeSmallIcon: import("lucide-react").LucideIcon;
declare const AccessibilityIcon: import("lucide-react").LucideIcon;
declare const ActivityIcon: import("lucide-react").LucideIcon;
...

Now just ignore the lucide-react imports in VSCode:
{
  "typescript.preferences.autoImportFileExcludePatterns": [
    "lucide-react"
  ]
}

And now TS suggestions only show the icons with suffix:


⚠ St. Lawrence Global Observatory CKAN has degraded performance

In 30fc6df, St. Lawrence Global Observatory CKAN (https://catalogue.ogsl.ca/en/) experienced degraded performance:

HTTP code: 200
Response time: 10651 ms

Resolved: St. Lawrence Global Observatory CKAN performance has improved in ed5673e.


Duplicate (already submitted)

What does "duplicate" mean here?
If the transaction has a duplicate nonce, it will be rejected with an invalid nonce.
If the nonce is different, there's no reason to reject the transaction (assuming there's still sufficient funds).

Valid nonce but duplicate of a transaction that already exists in the pool

Integration test


i18n(ko-KR): update drupal.mdx

Description (required)

update drupal.mdx
#9682

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ko
guides/cms/drupal.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


:ambulance: Product quantities representation according to the set lo…

…cation

There is more simple way at least in another version https://github.com/it-projects-llc/pos-addons/commit/b83570361569d6072cc5c827bbdc745dd186f315#diff-98128c6f5bc800511052fa25a4bfbbfe


move vehicle-purchase from concept to practice exercise.

It turns out having 2 concept exercises for the same concept breaks the syllabus. We have cars-assemble as the concept exercise for conditionals, so vehicle-purchase needs to be a practice exercise.
I've left it as wip, to stop configlet lint complaining that there is no student-facing concept for the booleans prerequisite.

Hey all.
So we don't support this. Once an exercise is in a config.json, it's in for good. You can't delete it or change its type). It has to be deprecated (with the appropriate flag/status).
In this case I've managed to change data in the database manually to put things in an ok state, but this involved deleting Colin's solution. This wouldn't be a doable fix if we had more solutions, but it's fine for this situation.
So no harm done, but probably worth just double checking with Erik when doing anything "unusual" like this :)

@ErikSchierboom I wonder if configlet or some CI should catch this? Both removing exercises and changing the type should be banned.

Ah ok that makes sense. Apologies, and thank you for the manual DB fixing and for clarifying @iHiD!

Oops, sorry! I knew you don't like re-purposing practice exercises as concept, but I (wrongly) thought it would be OK to move a wip concept exercise that had been wrongly classified in the first place, and that almost no students had even seen. I stand corrected...
Thank you to you and Erik for untangling this mess!


This is a good plugin,But I hope I Can Customize my own shortcut key.

You can totally do that already, just add your own shortcuts in System Preferences > Keyboard > Shortcuts.

Related to: https://github.com/zenangst/MarvinXcode/issues/15

That's great,haha,COOL! Thank you very much...and I think you'd better put this picture in readme.md,Do you think so?

For the newbie doesn't know how to use this plugin like me...

Very true, thanks @shenguanjiejie, I'll make a task to update the README :+1:

https://github.com/zenangst/MarvinXcode/issues/18

@shenguanjiejie Fixed now

Well done! Best wishes for you and your plugin.   :)


Cache resiliency improvements

Cache lock improvements for Unix locking scenarios under stress as implemented by MSAL extensions for java here

Not only we finished #63, we also implemented #100 based on the recent findings from Az CLI. Closing this issue now.


[BEAM-13563] Restructure Kinesis Source for AWS 2

This just prepares the Kinesis Source for use with a generalized ClientBuilderFactory.
Instead of passing on all Read parameters individually, pass on Read itself.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @aromanenko-dev

Run Java PostCommit


[Snyk] Security upgrade org.springframework:spring-webmvc from 5.3.18 to 6.0.0

Snyk has created this PR to fix 1 vulnerabilities in the maven dependencies of this project.
Snyk changed the following file(s):

master/pom.xml

Vulnerabilities that will be fixed with an upgrade:

Issue
Score
Upgrade

Denial of Service (DoS) SNYK-JAVA-ORGSPRINGFRAMEWORK-8384234
  631
org.springframework:spring-webmvc:  5.3.18 -> 6.0.0 Major version upgrade No Known Exploit

[!IMPORTANT]

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

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Denial of Service (DoS)

:tada: Snyk checks have passed. No issues have been found so far.
:white_check_mark: code/snyk check is complete. No issues have been found. (View Details)


[Bug] Graph sometimes displays empty node

Issue: Occasionally, there will be a random gray node with no associated information displayed on the Knowledge graph.
Steps to reproduce:

Click on "Projects" tab
Click on "Graph" tab

This will spawn an empty node on the graph:

I'm always surprised at the different ways in which people use Knowledge... you'll have to explain to me what the bung project is all about some day...
In the meantime, thanks for the report. There may be an extra element being added to the array of Sources being input into the graph component. I'll check it out (or you can open your first MR 😉 )

This is a very strange issue.
The stray node only appears when going from any other view to the graph view for the first time. If you switch projects while staying in the graph view, it disappears.
Further, the id of the element is non-deterministic (it changes every time you switch from any view to the graph view). Even more strange, the id I get from the node never corresponds to any existing projects or sources. But somehow the id is always a valid UUID...
Further research is needed.


example won't build for linux

CMake Error at /snap/flutter/91/usr/share/cmake-3.10/Modules/FindPkgConfig.cmake:415 (message):
A required package was not found
Call Stack (most recent call first):
/snap/flutter/91/usr/share/cmake-3.10/Modules/FindPkgConfig.cmake:593 (_pkg_check_modules_internal)
flutter/ephemeral/.plugin_symlinks/system_tray/linux/CMakeLists.txt:10 (pkg_check_modules)

I'm on Debian.  I got past the first issue, but now getting
[   +1 ms] /snap/flutter/current/usr/bin/ld: cannot find -lpkgcfg_lib_APPINDICATOR_appindicator3-NOTFOUND
[        ] /snap/flutter/current/usr/bin/ld: cannot find -lpkgcfg_lib_APPINDICATOR_dbusmenu-glib-NOTFOUND
[        ] clang: error: linker command failed with exit code 1 (use -v to see invocation)

I tried everything.  I installed every single appIndicator package and nothing works.

Which Linux system are you using？
You can try cmd ldconfig -p | grep libappindicator to show what libappindicator do you have in your system.

libappindicator3.so.1 (libc6,x86-64) => /lib/x86_64-linux-gnu/libappindicator3.so.1
libappindicator3.so (libc6,x86-64) => /lib/x86_64-linux-gnu/libappindicator3.so
libappindicator.so.1 (libc6,x86-64) => /lib/x86_64-linux-gnu/libappindicator.so.1
libappindicator.so (libc6,x86-64) => /lib/x86_64-linux-gnu/libappindicator.so

Linux 5950x 4.19.0-17-amd64 #1 SMP Debian 4.19.194-3 (2021-07-18) x86_64 GNU/Linux

can you try cmd pkg-config --cflags appindicator3-0.1 or pkg-config --cflags appindicator-0.1 to find some errors?

Hey, I decided to write my own plugin.  I didn't want the dojowindow.  And I couldn't get past those compile errors.
I got it working in a private repo app, but the plugin is open source.  https://github.com/sgehrman/status_bar/tree/master
I need help with Windows if you happen to be a Windows guy.  I only have mac and linux.


Most fallible methods should return Result<T, failure::Error> instead of Result<T, wasm_pack::error::Error>

A wasm_pack::error::Error will Into a failure::Error so existing ? locations shouldn't need any changes.
Basically, only "leaf" errors should be wasm_pack::error::Error and everything else should be failure::Error.
Then we should put .context("..") (or .with_context(|e| ...)) onto all non-leaf ? sites, providing better error messages and cause-and-effect chains.

I'd be happy to take this one!

@fitzgen wanted to check my understanding on this. The idea is I'll convert a fn like
step_check_crate_config to Result<(), failure::Error> and add a context / with_context to its call to check_crate_config
What I'm not seeing -- and apologies if I'm overlooking something obvious -- is the best way to convert that failure:Error to a wasm_pack::error::Error in a fn like run
Thanks!

step_check_crate_config to Result<(), failure::Error> and add a context / with_context to its call to check_crate_config

Correct.

What I'm not seeing -- and apologies if I'm overlooking something obvious -- is the best way to convert that failure:Error to a wasm_pack::error::Error in a fn like run

All those should become failure::Errors too, so no need to convert to wasm_pack::error::Error ;)

Thanks!

Thank you!

Ah, gotcha. I think I'd misread your intent with leaf / non-leaf.
Am I right in thinking that when you say "leaf" you mean something like a function that reads a config from disk? And a non-leaf fn is one that builds on top of it, e.g. initiating a build that depends on that config?

Sorry about the inactivity -- life has crept up in a major way and I haven't had a chance to get to this yet. If anyone wants to take a stab at it, please do!

I would like to give this a shot. I will try to start working on this soon and create a PR when I have. In the meantime if someone else would like to grab this, please do and let me know if that's the case. :)

@fitzgen: Thank you for the information, I will branch out from your PR.


Remove mvn-repo branch

The mvn-repo branch is outdated and therefore ready to be removed.  It was for 8.x early access releases.  It contains large binaries which could impact clones.
We will likely recreate an mvn-repo branch when we begin the Early Access releases for 9.x.

Charles, can you confirm sample stack and anything else that you know of does not currently depend on the mvn-repo branch?

Only development snapshots and EA relied on it.  I think its acceptable to remove.  @laurelnaiad may have an opinion too.

I think you're good to go with this removal-- however will you also purge it somehow?  It might require a push -f somewhere to actually prune the .git directory

I deleted it on github with
git push origin --delete mvn-repo

I can't find any indication that there's an additional purge needed.  But everyone who's pulled this  project will still have the remote tracking branch until they do:
git fetch --all --prune

Which will clean their local repo of branches we've removed from github.


can this giskard be used for codellama?

Hi
team,
Does it make sense to use giskard where the input is a text and output is a code?

Hello @pearlmary
You can indeed use Giskard with LLMs for code evaluation. For example by using the LLM-as-a-judge test you can add a new test asking it to create a function and have the requirement check that the function appears to be correct.
This however leaves a caveat: It relies on how good the LLM is at evaluating code.
While there currently aren't tests in our catalog for this, it should be possible to create a new test that will:

Generate text based on your prompt
Receive code as output
Start evaluating the code

And you c

Thank you for the clarifications,
One more basic question,
Can we replace openAI with any other open source LLM-as-a-judge. Do you have any examples or the syntax followed if it is possible to use other LLMs

You can use any LLM. Giskard define a "standard" library of tests, but all of them are open source. For example, the LLM-as-a-judge tests can be found here: https://github.com/Giskard-AI/giskard/blob/main/giskard/testing/tests/llm/output_requirements.py#L77-L131
From this link, you'll see a basic example. You can then write a similar function in your own environment and use the decorator so that it can be plugged into the Giskard python library or the Giskard Hub seamlessly !
Other references:
https://github.com/Giskard-AI/giskard/blob/main/giskard/llm/evaluators/base.py

Thank you for the quick reply.
Trying to understand your point though.
LLM scan requires an OpenAI API key. Can I replace it with any other llms such as llama 2.
Have you any example tutorial notebooks, that has used an open source LLMs for performing LLM scan?
Thanks in Advance.

Hello @pearlmary
The scan currently use different evaluation technics and some are coupled with using OpenAI (specifically GPT-4).
However your model can use llama2, the workflow is the following:

We generate text using your model (llama2, GPT, Cohere, ...)
We evaluate the output of your model using GPT-4

You can find detailed information of our evaluator in the documentation, you'll see that we have LLM assisted evaluators that only works with GPT-4.
We are aware of the issues (require API key, send info to Open AI, ...) and are working on custom evaluators that aren't coupled with GPT-4.


Shared Preferences - sharing data stored between two different apps

Hi, I have two different apps that the user could have installed simultaneously. I would like to be able to read data stored while using one app by the other app.
Is this now possible?
If so, how?
If not, it would be a great feature to have.
I need it just in iOS for the time being.
Thanks,
Daniele Folatelli

Hi @danielefolatelli,
This issue doesn't seem to describe a bug or a feature request. Please see https://flutter.dev/community#community-grid for resources and asking questions like this. You may also get some help if you post it on Stack Overflow with a minimal code sample that clearly describes your problem.
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and we will reopen it.
Thank you


Missing defaults

Some of the DICOM tags which are part of the pack path are now allowed to be undefined. Specifically, PatientName, PatientBirthDate, StudyDescription, StudyDate, SeriesDescription may be undefined.
The name of the DICOM tag is used in the pack path. The string "Not defined" is used in JSON data.

Codecov Report

Merging #2 (61b7ea9) into master (34f5b82) will increase coverage by 0.26%.
The diff coverage is 86.95%.

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
+ Coverage   79.56%   79.82%   +0.26%
==========================================
  Files          10       10
  Lines         455      461       +6
==========================================
+ Hits          362      368       +6
  Misses         93       93

Files Changed
Coverage Δ

src/main.rs
0.00% <0.00%> (ø)

src/dicom_data.rs
93.44% <100.00%> (+0.10%)
:arrow_up:

src/log_models.rs
100.00% <100.00%> (ø)

src/pack_path.rs
100.00% <100.00%> (ø)


Message board dates

attempting to add manual posts to the message board with a function using serv.time or serv.rtime results in the incorrect date being displayed.

Was this fixed?

No this still seems to be an issue

This is resolved. closing


Unable to Load Tasks

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
Discussed in discord forum with @claremacrae to reproduce the error and log a bug report. Query tasks (query show below) should display the list of tasks.
not done
happens before today
sort by due
limit 10

Current behaviour
Discussed in discord forum with @claremacrae to reproduce the error and log a bug report. Query tasks (query show below) are not loading and it keep showing "Loading Tasks..." message. Notable behaviour:

The same query works in another vault
The same query and the same vault works in another computer (working in Mac Mini but not working in MacBook Pro, both machines are running macOS Sonoma 14.3.1).

[ ] Android
[ ] iPhone/iPad
[ ] Linux
[X] macOS
[ ] Windows

Obsidian Version
v1.5.3
Tasks Plugin Version
6.0.0
Checks

[X] I have tried it with all other plugins disabled and the error still occurs

Possible solution
No response

@kennygokh, please uninstall the Tasks plugin and then reinstall and enable it. Thanks.

@kennygokh, please uninstall the Tasks plugin and then reinstall and enable it. Thanks.

This worked for someone else yesterday:
https://github.com/obsidian-tasks-group/obsidian-tasks/issues/2669#issuecomment-1959038442

@kennygokh Please zip up and attach the problem vault here. Thanks.

@claremacrae no luck. still the same problem after following the steps in #2669. Unfortunately, I can't zip my vault as it has a lot of notes that are work related 🙏. Any other ways for you to collect the essential information only?

@kennygokh Please zip up and attach the problem vault here. Thanks.

Unfortunately, I can't zip my vault as it has a lot of notes that are work related

Apologies I missed that bit...

Any other ways for you to collect the essential information only?

Well, standard troubleshooting techniques apply. For example, prune a copy of your vault back to the smallest thing that reproduces that problem and that you can share.
For example: You can create a local copy of your vault, and delete all the notes, then recreate a new file, confirm the issue is reproducible, then zip that and attach it here.

In the process, you may even find that the issue is a specific note.

Hi @kennygokh, I'm going to close this for now, as there is currently nothing we can do about it.
If you still have a problem and are able to attach a vault that demos the problem, I will re-open and see if I can reproduce it, and if so, see if there's anything we can do.

Noted @claremacrae. Just to share that I have recently move this vault from Obsidian Sync to OneDrive. Obsidian Sync is disabled since then. It has been working fine for the past few weeks, only a few days ago this problem occur.  What I did this morning is to do a manual copy of the vault with a new name, and it works 😐 I have no idea what had gone wrong. I am now renaming back to original vault name (My Work) and monitor it for awhile 🤕. Hopefully it works 🙏

Hello, commenting here since I have been facing this same issue (explained here).
To summarize: my Tasks queries on my Mac are stuck on "Loading Tasks ...".

The same queries work on my mobile devices (iPad, Android).
They also work on MacOS when I open a copy of my Vault as a new Vault (with all files, settings, and plug-ins the same except Obsidian Sync).

Transferring my work to a copy like @kennygokh did is one option, but I am still planning to use Obsidian Sync, so it's not quite as simple. Any further suggestions for troubleshooting or fixing this issue?

Hello, commenting here since I have been facing this same issue (explained here).

Are you the author of that Forum post?

Any further suggestions for troubleshooting or fixing this issue?

I have replied in the Forum post.
Personally, I don't find it effective or efficient to track topics across multiple platforms, and also not in closed topics.
So if you still have problems, you can create a new Bug report here in the Tasks, and complete the template with all the requested information.
And then add a link to that ticket in the Forum post.


Fix typos

Hi @JohnnyDeuss Thanks so much for taking this on!
There were at least 3 functional changes I found -- 1 correct functional change and 2 changes that would lead to a breaking change. I'm worried that there might be more changes that we do not want to make -- any chance you could split the PR a bit? At the very least to separate docs from changes in code?

Sure, I've taken out the changes in libs/.


Raspberry pi: I cannot install python versions

Regardless of which version of python I try to install, I get the same build error (See photo for details).
Device: Rasberry pi B+

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/12879537-raspberry-pi-i-cannot-install-python-versions?utm_campaign=plugin&utm_content=tracker%2F282009&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F282009&utm_medium=issues&utm_source=github).

Please check if you have connectivity to the internet .

I got everything to work. I am at a hackathon, so the internet is so-so.
Anyway, after I got connected to the internet and installed all the packages in your common build issues page, everything starting to work.
Thanks again!


Fix for Skills endorsement check

#minor

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll


ci: add labels to PRs submitted by a bot

See https://github.com/autowarefoundation/autoware/pull/167.

Worked. https://github.com/tier4/autoware_launch/pull/280


Monoscopic mode seems to be broken

Change gvr.xml to the following:
        <mono-mode-parms
            monoFullScreen="true"
            monoMode="true" />

Fix - https://github.com/Samsung/GearVRf/pull/742


How to tune the parameters of detection algorithms?

I use roerich on my own dataset, but the result is not so good.

There are comments in the class ChangePointDetection:
    Parameters
    ----------
    scaler: A scaler object is used to scale an input data. The default one is `SmaScalerCache`
    metric: A loss function during optimize step of NN. Can be one of the following KL_sym, KL, JSD, PE, PE_sym, Wasserstein
    window_size: A size of a window when splitting input data into train and test arrays
    periods: A number of previous data-points used when constructing autoregressive matrix
    lag_size: A distance between train- and test- windows
    step: Each `step`-th data-point is used when creating the input dataset
    n_epochs: A number of epochs during training NN
    lr: A learning rate at each step of optimizer
    lam: A regularization rate
    optimizer: One of Adam, SGD, RMSprop or ASGD optimizers
    debug: default zero

How to tune the parameters of the change point algorithms, like periods, window_size , lag_size or step?

@JR-Jared, hi!
The key parameters here are window_size and lag_size. The general recommendation is to choose values such that only one change point is within 2 * (window_size + lag_size).