An assetBundle problem

Hi, I have the code like this:
class BootBoxAsset extends AssetBundle
{
    public $sourcePath = '@vendor/almasaeed2010/adminlte';

    public $css = [
    ];

    public $js = [
        'https://cdn.bootcss.com/bootbox.js/4.4.0/bootbox.js'
    ];

    public $depends = [
        'plugins\jQuery\jquery-2.2.3.min.js',
        'bootstrap\js\bootstrap.min.js',
    ];

    public $jsOptions = ['position' => \yii\web\View::POS_HEAD];
}

As the code shows above, I want to use bootbox.js which requires the related jquery and bootstrap dependency that located in my $sourcePath variable combined with the directories in $depends array,
when I register this asset in my view, there's an error tells me that the Class plugins\jQuery\jquery-2.2.3.min.js does not exist, so what should I do to register the dependency not start from yii directory?
thanks!
Additional info

Q
A

Yii version
2.0

PHP version
7.0

Operating system
ubuntu

You need a create a separate asset for those two files. And then give the path to those bundles in $depends.

class NewAsset extends AssetBundle
{
 //if the files in the web directory
  $baseUrl = '@web';

    public $css = [
    ];

    public $js = [
        'plugins\jQuery\jquery-2.2.3.min.js',
'bootstrap\js\bootstrap.min.js',
    ];

    public $depends = [

    ];

    public $jsOptions = ['position' => \yii\web\View::POS_HEAD];
}

Then use it like this.
class BootBoxAsset extends AssetBundle
{
    public $sourcePath = '@vendor/almasaeed2010/adminlte';

    public $css = [
    ];

    public $js = [
        'https://cdn.bootcss.com/bootbox.js/4.4.0/bootbox.js'
    ];

    public $depends = [
        'yii/web/NewAsset
    ];

    public $jsOptions = ['position' => \yii\web\View::POS_HEAD];
}


[Bugbash] the qnamaker tutorial should reference the qna doc that creates a bot out of an existing qnamaker service

this doc
https://review.docs.microsoft.com/en-us/azure/bot-service/bot-builder-tutorial-add-qna?view=azure-bot-service-4.0&branch=pr-en-us-1325&tabs=csharp
should link this doc, maybe at the beginning?
https://docs.microsoft.com/en-us/azure/cognitive-services/qnamaker/tutorials/create-qna-bot

created and added link https://aka.ms/azure-create-qna to article prerequisites


Changed test method name in MovieTest

In #208, I named my test method as testName instead of testQuote. This pr fixes it.

Codecov Report

Merging #209 (59870d7) into master (7568c4a) will decrease coverage by 0.10%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #209      +/-   ##
============================================
- Coverage     94.63%   94.53%   -0.11%
+ Complexity     1833     1832       -1
============================================
  Files           186      186
  Lines          3859     3859
  Branches        387      387
============================================
- Hits           3652     3648       -4
- Misses          105      108       +3
- Partials        102      103       +1

Impacted Files
Coverage Δ

src/main/java/net/datafaker/Commerce.java
88.46% <0.00%> (-7.70%)
:arrow_down:

...ker/idnumbers/pt/br/IdNumberGeneratorPtBrUtil.java
92.59% <0.00%> (-3.71%)
:arrow_down:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Ywt/optimize op print info log

close print op log info by default
optimize fetching environment variable
performance of yolov5: 73.4% -> 74.0%

Close this issue since we have the same one https://github.com/DeepLink-org/deeplink.framework/pull/524


Revise parameters for Kaldi mfcc compatibility test

Similar to #679
We should also revise the parameters for mfcc test.
See also #681

I would like to work on this.

I would like to work on this.

Hi @engineerchuan
Thanks. Do you know what are good parameters for mfcc? I am not expert but we can consult with our collaborators.

Not off top of my head.  Let me study it first for a day and come up with a proposal.

Hi @mthrok,
I would like to follow this approach with some questions:

For testing compute-fbank-feats and compute-mfcc-feats, first extract the default argument values.
As suggested in https://github.com/pytorch/audio/issues/679, use example datasets to augment more examples of "valid" kaldi parameters both for fbank and mfcc.

Question 1: How should we store and keep the default values for fbank and mfcc up to date?
Recommendation - cache the default fbank values and the override values in json.  In future, revise manually if kaldi default argument values or example datasets default argument values change.
Question 2: How should we handle when some datasets don't have fbank config or don't have mfcc config?
Recommendation - we should only use configs from datasets for testing fbank or mfcc if they have the respective config.
Example: Switchboard has both fbank and mfcc config, thus we will use both for testing.
Example: librispeech only stores mfcc config, thus we will not use librispeech for testing fbank
Question 3: What should we do with generate_fbank_data.py?
Currently generate_fbank_data.py generates random parameters, which may be invalid.  We could have it make network wget calls to the relevant repositories if possible to retrieve and parse the values.  It could inspect Kaldi source code directory or execute the executable path with --help to parse out default values.  This sounds hacky and maybe we should skip it for now.

Question 1: How should we store and keep the default values for fbank and mfcc up to date?
Recommendation - cache the default fbank values and the override values in json. In future, revise manually if kaldi default argument values or example datasets default argument values change.

I am not quite sure what you mean by cache, but in terms of JSON data, I think providing empty arguments {}, would result in default parameters in both Kaldi CLI and torchaudio's implementation. That way if Kaldi changes default values, I think we can notice. Then we can add arguments with the current default values {"allow_downsample": false, "allow_upsample": false, ... }. I think the later is what you mean by caching.
BTW: Currently kaldi used in test CI is updated manually and I do it from time to time by building a new Docker file and pushing it. Although we plan to update it automatically, we do not know when that will happen.
Also, note that there are some parameter discrepancies on parameters due to inconsistent design. Kaldi expects full range wave form where as typical torchaudio functional expects normalized waveform, yet torchaudio.compliance.kaldi module expects full range values, which confuse users. https://github.com/pytorch/audio/issues/371#issuecomment-625613872, https://github.com/pytorch/audio/issues/328 I think for this test case, we use load_wav with normalize=False, but you might hit something. We have an idea of making kaldi module consistent with the rest of the code base but we have not planned work items yet.

Question 2: How should we handle when some datasets don't have fbank config or don't have mfcc config?
Recommendation - we should only use configs from datasets for testing fbank or mfcc if they have the respective config.
Example: Switchboard has both fbank and mfcc config, thus we will use both for testing.
Example: librispeech only stores mfcc config, thus we will not use librispeech for testing fbank

Yes, that makes sense.

Question 3: What should we do with generate_fbank_data.py?
Currently generate_fbank_data.py generates random parameters, which may be invalid. We could have it make network wget calls to the relevant repositories if possible to retrieve and parse the values. It could inspect Kaldi source code directory or execute the executable path with --help to parse out default values. This sounds hacky and maybe we should skip it for now.

generate_fbank_data.py is obsolete and provides no value. so we can simply delete it. If our tests can incorporate the latest changes on Kaldi side automatically, it would be nice, but at this moment, the priority is to have a good coverage of valid use cases. That itself is a great improvement.
Also making tests depend on external resource (networking, files stored elsewhere) increase maintenance cost, so we would like to refrain from doing it. Parsing help message of executables is plausible because it's available but let's defer on that one. We can discuss the extra value of doing that once we have a good set of values to test.

Hi @engineerchuan
I'm working on refactoring legacy code in our project: we have 40 Mb of Kaldi MFCC binary which we would like to replace with torchaudio.compliance.kaldi.mfcc
I managed to get nearly identical results between the call of Kaldi binary file and torchaudio.compliance.kaldi.mfcc.
I don't know what my Kaldi binary version is, but it and the Torch implementation have several different default params, see table below:

parameter
torch_value
kaldi_value

blackman_coeff
0.42
0.42

cepstral_lifter
22.0
22

channel
-1
-1

dither
0.0
1

energy_floor
1.0
0

frame_length
25.0
25

frame_shift
10.0
10

high_freq
0.0
0

htk_compat
False
False

low_freq
20.0
20

num_ceps
13
13

min_duration
0.0
0

num_mel_bins
23
23

preemphasis_coefficient
0.97
0.97

raw_energy
True
True

remove_dc_offset
True
True

round_to_power_of_two
True
True

sample_frequency
16000.0
16000

snip_edges
True
True

subtract_mean
False
False

use_energy
False
True

vtln_high
-500.0
-500

vtln_low
100.0
100

vtln_warp
1.0
1

window_type
povey
povey

allow_downsample

False

allow_upsample

False

debug_mel

False

max_feature_vectors

-1

output_format

kaldi

utt2spk

""

vtln_map

""

As you can see, aside from several missing params, dither, energy_floor and use_energy are set to the opposite.
(also, Kaldi has gigantic dithering by default, so I spent good porting of today trying to understand why the two sets of results don't match)


Help

I already played the game with the patch once and it worked perfectly, I was about to replay it again in Japanese "Hard" but for some reason it's not working anymore, when I start the game *Both in normal or hard" the cutscene have the audio in Japanese but the game still have the audio in English And yes the subtitles are on Can you help me? I don't know if it is a configuration on my emulator or maybe a bug in the patch, since I use a pre-patched version Cdromance

Try to redownload the game or repatch it without the safe mode enabled. This mode makes it so that some audio will be in english

Will close this due to never receiving any updates. Reopen it if you have further updates.


[Table's not updated to show that "Web app that signs in users" scenario does support platforms such as Java.]

Title

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

ID: d014c45f-5c4a-2598-ae92-89f48ae6b207
Version Independent ID: e7d95537-e03b-230a-635b-7032ff101a6b
Content: Microsoft identity platform authentication flows & app scenarios
Content Source: articles/active-directory/develop/authentication-flows-app-scenarios.md
Service: active-directory
Sub-service: develop
GitHub Login: @jmprieur
Microsoft Alias: jmprieur

Looking at the table furthest down, the only icon showing support for web app scenario is .NET. However, clicking on the detailed web app guide here shows more supported frameworks.

@osaid202 Thanks for your comment! We will review and provide an update as appropriate.


Force Directed Graph layout

https://github.com/thi-ng/geom/blob/master/geom-viz/src/core.org#force-directed-graph-layout
It looks like this project is on pause, but I'm wondering if a graph layout could be in the works.

Hi @den1k - sorry this was in the works, but never more than half-baked and it's unlikely I will continue this. I don't have the code available right now, but I used the geom.physics module to drive the layout...
Also, not directly related, but the only public demo of the physics engine is here:
http://demo.thi.ng/geom/physics/
Maybe that's useful to get going...


Add Deferred into zepto

To replace Jquery with zepto we need to have Promise based $.ajax. By default $.Deferred is not included in zepto-browserify

Please use the native fetch() call for Ajax calls. That is promise based and supported by most modern browsers. We also deliver polyfills for it.


[opt](inverted index) opt for log output when matching without an index

Proposed changes

Generates a large volume of log output when scanning large amounts of data

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


graph saved with same name as another graph overwrites the previous one without notice

click publish graph
save with the name of an already-published graph

expected one of:

graph saved with a non-conflicting name
ask for confirmation
"computer says no" (quickest to implement, I vote).

actual:
graph overwrites previous graph without notice

Yes, it would be great to have a dialog "A graph with this name already exists. Would you like to overwrite it? Yes / No"


Remove BedData in favor of DyeableData.

BedData only contains a key for dye colors. We already have that in DyeableData.
Related: SpongePowered/SpongeCommon#1611

BedData either probably needs to have a data value whether it is occupied / being slept in, or a new DataManipulator found to cover this purpose.

If this will be added in API 8, it will very likely be removed anyway, due the 1.13 blockstate changes. Beds will no longer be tile entities.

@Cybermaxke I think this is targeted for API 7 just BedData will be deprecated instead of removed.


chore: upgrade go dependencies

from: https://github.com/jenkins-x/jx3-versions

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign jstrachan
You can assign the PR to them by writing /assign @jstrachan in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr
cb54a4146a7f24ad87e58097c3feda703aaea8a4
link
/test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr
7638c9da7482cbb68546f7d03a06f1cdbb5b8f30
link
/test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


[Request for External] Page 345 InsertGLAccNetChange()

Please make InsertGLAccNetChange on Page 345 public and external

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Add Try.of method accepting void supplier.

It would be cool if Try accepted a void method. E.g.
Try.of(() -> Thread.sleep(1))
    .flatMap(nothing -> doSomethingElse());

Not sure if "void supplier" is the best name though, as it seems to be kind of self contradicting.

Current coverage is 96.47% (diff: 50.00%)

Merging #1645 into master will decrease coverage by 0.01%

@@             master      #1645   diff @@
==========================================
  Files            89         89
  Lines         10940      10944     +4
  Methods           0          0
  Messages          0          0
  Branches       1850       1850
==========================================
+ Hits          10556      10558     +2
- Misses          240        242     +2
  Partials        144        144

Powered by Codecov. Last update 60d08c9...3dc662f

I think this will raise ambiguities. Why not use Try.run?

@whirlwin I thought about your suggestion. I also like to have same factory method names, it is better than learning new API (of, run, ...).
Also I verified that of(CheckedSupplier) does not clash with of(CheckedVoidSupplier).
However, with Project Valhalla Supplier<void> will be the same as Runnable (as I understand Brian Goetz' mission statement). So having both methods named of is no good idea from the strategic viewpoint.
I hope you can live with Try.run(). I will close this PR.
Many thanks for participating! I hope I will see more PR's from you in future :-)


Pip install issues - macOS

Im running Apple clang version 12.0.5 (clang-1<IP_ADDRESS>), tried both python 3.9, 3.8, 3.6. Ive tried torch 1.4 and 1.10.
Im running into the same problem everytime;  ERROR: Failed building wheel for ctcdecode.
The error message is veeeery long, but the top of it looks like this:
ERROR: Command errored out with exit status 1:    command: /Users/magnuslyk/opt/miniconda3/envs/xtech/bin/python -u -c 'import io, os, sys, setuptools, tokenize; sys.argv[0] = '"'"'/private/var/folders/40/mvywxcyx5h5463679qv4g3780000gn/T/pip-req-build-r9pb__ba/setup.py'"'"'; __file__='"'"'/private/var/folders/40/mvywxcyx5h5463679qv4g3780000gn/T/pip-req-build-r9pb__ba/setup.py'"'"';f = getattr(tokenize, '"'"'open'"'"', open)(__file__) if os.path.exists(__file__) else io.StringIO('"'"'from setuptools import setup; setup()'"'"');code = f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' bdist_wheel -d /private/var/folders/40/mvywxcyx5h5463679qv4g3780000gn/T/pip-wheel-6rgarvb_        cwd: /private/var/folders/40/mvywxcyx5h5463679qv4g3780000gn/T/pip-req-build-r9pb__ba/   Complete output (7260 lines):   /Users/magnuslyk/opt/miniconda3/envs/xtech/lib/python3.6/distutils/extension.py:131: UserWarning: Unknown Extension options: 'package', 'with_cuda'     warnings.warn(msg)   running bdist_wheel   running build   running build_py   creating build   creating build/lib.macosx-10.9-x86_64-3.6   creating build/lib.macosx-10.9-x86_64-3.6/ctcdecode   copying ctcdecode/__init__.py -> build/lib.macosx-10.9-x86_64-3.6/ctcdecode   running build_ext ...
I have been going through all issue postings with similar problems, and it seems the issue on macOS persists.
Can anyone help with this?

Try executing
pip install git+https://github.com/parlance/ctcdecode

bump


Added Remove Job script, fixed issue with not returing jobId if job already exsists

Hi!
After your changes code in Add-DatabricksNotebookJob looks great, I really love it.
I found a minor issue - if Job already exists in Databricks, it doesn't return JobId (because of typo and Write-Output in script).
Write-Output caused returning array with jobId and WriteOutput value after running script instead of only jobId.
Please take a look on it and merge if you think it's fine.
Thanks.

Good spot - thanks. I will merge now.
Ignore the failed build alert - it's because you do not have access to my VSTS instance. I will get that sorted as well as I'm working on the tests at the moment.


radio-inline/checkbox-inline positioning issue

I'm not sure if this is on purpose or if my HTML is wrong but the radio-inline and checkbox-inline components seem to be positioned incorrectly. They are missing a container that has positioning ( position: relative) which makes them overflow and behave weirdly when used in an overflow: auto context.
Example and fix: http://jsbin.com/juvemipuli/2/edit?html,css,output
Fix:
.radio-inline, .checkbox-inline {
  position: relative;
}

Hi @OrganicPanda!
You appear to have posted a live example (http://jsbin.com/juvemipuli/1/edit), which is always a good first step. However, according to the HTML5 validator, your example has some validation errors, which might potentially be causing your issue:

line 37, column 66: Duplicate ID inlineCheckbox1.
line 40, column 66: Duplicate ID inlineCheckbox2.
line 43, column 66: Duplicate ID inlineCheckbox3.
line 49, column 86: Duplicate ID inlineRadio1.
line 52, column 86: Duplicate ID inlineRadio2.
line 55, column 86: Duplicate ID inlineRadio3.
line 61, column 66: Duplicate ID inlineCheckbox1.
line 64, column 66: Duplicate ID inlineCheckbox2.
line 67, column 66: Duplicate ID inlineCheckbox3.
line 73, column 86: Duplicate ID inlineRadio1.
line 76, column 86: Duplicate ID inlineRadio2.
line 79, column 86: Duplicate ID inlineRadio3.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

I find that incredibly obnoxious but I have fixed it anyway.

This isn't a problem with the checkboxes and radios—you're setting a fixed height and asking the browser to still overflow it. That'll always lead to some kind of weird behavior like this. I'm fine adding the relative positioning I suppose, but I don't view this as a bug on our end.

I'm not sure I understand your comment @mdo. As far as I'm aware you always need to have some kind of sizing in order to have overflow. Whether that is via absolute positioning/setting a width or height/using flexbox etc etc.. you always need some kind of size otherwise the element will expand to the size of the content and there is no overflow.
The problem is that not all types of sizing imply a position (setting a width/height or in my case using flexbox) so it would be very useful to have radios and checkboxes manage their own position more completely and not leaving it up to chance that a container will have a position or be in the normal flow.

Oh snap, yeah, I was thinking this wasn't an issue because we had something set to prevent the need for position: relative, but we probably need to set it since we have position: absolute on the inputs themselves.


[Provider] Cerebras Inference API

https://inference-docs.cerebras.ai/quickstart

This was done in #565


remove Org Admin from coverage

org admin no longer required

Hey @maxdobeck this PR is stuck in here, need any help?

Yes please @discombobulateme looks like this is live now


Update docs to include ping script tracking

Fixes: https://github.com/mgmco/bugs/issues/2169
Changes proposed in this pull request:

TSIA

cc @mgmco/developers

@mtimofiiv I wasn't sure, in the new situation where the client sets the cookie + uses the ping script. Does the name of the cookie have to be specified somewhere?
<script src="https://tracking.membergetmember.co/scripts/ping.js"></script>
<script>
  (function() {
    var MGMbuyerInfo = {
      email<EMAIL_ADDRESS>// Required
      name: 'Your new client\'s name',
      bank_account: 'Your new client\'s bank account',
      state: 'confirmed',
      customer_id: 'Your new client\'s customer ID',
      // Insert other parameters if needed
    };

    window.SocialShopSale = MGMbuyerInfo;
    window.SocialShopPing.refresh();
  })();
</script>

The \'s looks a bit strange. Why not name: "Your new client's name",?

It looks like you are still going for API tracking mainly? I would say the getting started page of tracking should have the focus on our ping script right?

Eh, if you read "callbacks" then it gives you 2 options, where the first option listed is the ping script. Right below it we state that the choice is theirs with the advantages/disadvantages listed very short. Getting started doesn't say anything about which sort of tracking, or am I missing something?

The 's looks a bit strange. Why not name: "Your new client's name",?

Cause the HTML fucks up, I'll change it to something different

This is branch, you're reading the old docs

I thought it was the Getting started section (which is not changed in this PR) but it is the Marketeers section. Never mind my comment then.

Here's how you can run this branch locally: https://github.com/mgmco/clients.membergetmember.co/blob/gh-pages/README.md

It is one page right? Could you create a .pdf of it?

docs_#7.pdf

Awesome, thanks!

Looks quite good in PDF!

@chrissvo yes, the cookie can be specified by name via the MGMbuyerInfo.cookie_name property, or alternatively they can just pass the mgm sale ID via MGMbuyerInfo.sale_id

Okay, I'll add something on the cookie_name

Can you change the marketing contact details from<EMAIL_ADDRESS>to<EMAIL_ADDRESS><EMAIL_ADDRESS>gets to the inbox of @MartineHardeveld and moi

PR waiting for that @vireshnarain: https://github.com/mgmco/clients.membergetmember.co/pull/8


Make no-unused-disable fixable

Summary:
Add a fixer for no-unused-disable that just removes the entire comment node. If the comment is disabling multiple rules at once, do not offer to fix.
Test plan:

npm run test
run the rule on a handful of files in a local project, note that it
finds many issues and successfully fixes them

Ironic, I will fix those lint errors :)

Thank you very much for this PR.
That idea sounds good to me.
However, unfortunately, I rewrote the no-unused-disable rule massively because the current no-unused-disable rule doesn't work in ESLint 5.0.0-alpha. (v3.0.0-beta.0) (see #12 for details)
If you are still interesting to implement this, you can investigate in lib/patch.js file. This file does the following actions:

It overrides the Linter.prototype.verify  public method and it does if eslint-comments/no-unused-disable rule is enabled.

It verifies the code with reportUnusedDisableDirectives option.
It convert the reportUnusedDisableDirectives errors to eslint-comments/no-unused-disable errors before it returns the result.

I'll check that out, thanks!

@ianobermiller Still interested in finishing this? :)

Perhaps at some point, but it is no longer a priority.

That should do it :)

I apologize for my delay. I had been diving into busy days.
LGTM, thank you so much!


2.0.48 bug: AttributeTypecastBehavior::resetOldAttributes() causes "class has no attribute named" InvalidArgumentException

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Fixed issues
#19856

@uaoleg please don't forget a line for CHANGELOG.

@samdark done

Thanks.


Figure generation

Which part of the code actually help to generate those figure for the results section, any help please, how to generate plot.

I'm also stuck on that.

Checkout: https://stellargraph.readthedocs.io/en/stable/demos/time-series/gcn-lstm-time-series.html
Notebook: https://colab.research.google.com/github/stellargraph/stellargraph/blob/develop/demos/time-series/gcn-lstm-time-series.ipynb


Escape error when check-googlefonts

When trying to run this command fontbakery check-googlefonts .\WD-XLLubrifontSC-Regular.otf, it gives an error after some checks:
Traceback (most recent call last):
  File "C:\users\nksl\appdata\local\programs\python\python39\lib\sre_parse.py", line 1039, in parse_template
    this = chr(ESCAPES[this][1])
KeyError: '\\W'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\users\nksl\appdata\local\programs\python\python39\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\users\nksl\appdata\local\programs\python\python39\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\NKSL\AppData\Local\Programs\Python\Python39\Scripts\fontbakery.exe\__main__.py", line 7, in <module>
  File "C:\users\nksl\appdata\local\programs\python\python39\lib\site-packages\fontbakery\cli.py", line 44, in main
    run_profile_check(subcommand_module[6:])
  File "C:\users\nksl\appdata\local\programs\python\python39\lib\site-packages\fontbakery\cli.py", line 24, in run_profile_check
    sys.exit(check_profile_main(module.profile))
  File "C:\users\nksl\appdata\local\programs\python\python39\lib\site-packages\fontbakery\commands\check_profile.py", line 342, in main
    distribute_generator(status_generator, [reporter.receive for reporter in reporters])
  File "C:\users\nksl\appdata\local\programs\python\python39\lib\site-packages\fontbakery\checkrunner.py", line 594, in distribute_generator
    target(item)
  File "C:\users\nksl\appdata\local\programs\python\python39\lib\site-packages\fontbakery\reporters\__init__.py", line 131, in receive
    self._output(event)
  File "C:\users\nksl\appdata\local\programs\python\python39\lib\site-packages\fontbakery\reporters\terminal.py", line 189, in _output
    text = self._render_event(event)
  File "C:\users\nksl\appdata\local\programs\python\python39\lib\site-packages\fontbakery\reporters\terminal.py", line 575, in _render_event
    self._render_event_async(print, event)
  File "C:\users\nksl\appdata\local\programs\python\python39\lib\site-packages\fontbakery\reporters\terminal.py", line 564, in _render_event_async
    self._render_event_sync(print, e)
  File "C:\users\nksl\appdata\local\programs\python\python39\lib\site-packages\fontbakery\reporters\terminal.py", line 463, in _render_event_sync
    formated_msg = parse_md(formated_msg)
  File "C:\users\nksl\appdata\local\programs\python\python39\lib\site-packages\fontbakery\reporters\terminal.py", line 597, in parse_md
    formated_text = re.sub('<#MD-TABLE-'+str(i)+'#>', tables[i], formated_text)
  File "C:\users\nksl\appdata\local\programs\python\python39\lib\re.py", line 210, in sub
    return _compile(pattern, flags).sub(repl, string, count)
  File "C:\users\nksl\appdata\local\programs\python\python39\lib\re.py", line 327, in _subx
    template = _compile_repl(template, pattern)
  File "C:\users\nksl\appdata\local\programs\python\python39\lib\re.py", line 318, in _compile_repl
    return sre_parse.parse_template(repl, pattern)
  File "C:\users\nksl\appdata\local\programs\python\python39\lib\sre_parse.py", line 1042, in parse_template
    raise s.error('bad escape %s' % this, len(this))
re.error: bad escape \W at position 103 (line 1, column 104)

Font: WD-XLLubrifontSC-Regular.zip

It happens again with another font:
Traceback (most recent call last):
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\sre_parse.py", line 1041, in parse_template
    this = chr(ESCAPES[this][1])
KeyError: '\\L'

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\runpy.py", line 86, in _run_code
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\Scripts\fontbakery.exe\__main__.py", line 7, in <module>
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\fontbakery\cli.py", line 45, in main
    run_profile_check(subcommand_module[6:])
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\fontbakery\cli.py", line 25, in run_profile_check
    sys.exit(check_profile_main(module.profile))
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\fontbakery\commands\check_profile.py", line 347, in main
    distribute_generator(status_generator, [reporter.receive for reporter in reporters])
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\fontbakery\checkrunner.py", line 594, in distribute_generator
    target(item)
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\fontbakery\reporters\__init__.py", line 131, in receive
    self._output(event)
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\fontbakery\reporters\terminal.py", line 189, in _output
    text = self._render_event(event)
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\fontbakery\reporters\terminal.py", line 575, in _render_event
    self._render_event_async(print, event)
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\fontbakery\reporters\terminal.py", line 564, in _render_event_async
    self._render_event_sync(print, e)
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\fontbakery\reporters\terminal.py", line 463, in _render_event_sync
    formated_msg = parse_md(formated_msg)
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\fontbakery\reporters\terminal.py", line 597, in parse_md
    formated_text = re.sub('<#MD-TABLE-'+str(i)+'#>', tables[i], formated_text)
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\re.py", line 209, in sub
    return _compile(pattern, flags).sub(repl, string, count)
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\re.py", line 326, in _subx
    template = _compile_repl(template, pattern)
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\re.py", line 317, in _compile_repl
    return sre_parse.parse_template(repl, pattern)
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\sre_parse.py", line 1044, in parse_template
    raise s.error('bad escape %s' % this, len(this))
re.error: bad escape \L at position 103 (line 1, column 104)

Font: LXGWWenKai-Regular.zip
Seems like this only affects Chinese font so far. Can it be related to using Chinese characters in name table of font? (given it stops right after entries checking related to name, e.g.:
 >> com.google.fonts/check/name/license
    Check copyright namerecords match license file.
    with .\LXGWWenKai-Regular.ttf

      Rationale:
      A known licensing description must be provided in the NameID 14 (LICENSE
      DESCRIPTION) entries of the name table.

      The source of truth for this check (to determine which license is in use)
      is a file placed side-by-side to your font project including the licensing
      terms.

      Depending on the chosen license, one of the following string snippets is
      expected to be found on the NameID 13 (LICENSE DESCRIPTION) entries of the
      name table:
      - "This Font Software is licensed under the SIL Open Font License, Version
      1.1. This license is available with a FAQ at: https://scripts.sil.org/OFL"
      - "Licensed under the Apache License, Version 2.0"
      - "Licensed under the Ubuntu Font Licence 1.0."

      Currently accepted licenses are Apache or Open Font License.
      For a small set of legacy families the Ubuntu Font License may be
      acceptable as well.

      When in doubt, please choose OFL for new font projects.

     FAIL License file LICENSE.txt exists but NameID 13 (LICENSE
          DESCRIPTION) value on platform 1 (MACINTOSH) is not specified for
          that. Value was: "This Font Software is licensed under the SIL Open
          Font License, Version 1.1. This license is available with a FAQ at:
          https://scripts.sil.org/OFL" Must be changed to "Licensed under the
          Apache License, Version 2.0" [code: wrong]

     FAIL License file LICENSE.txt exists but NameID 13 (LICENSE
          DESCRIPTION) value on platform 3 (WINDOWS) is not specified for that.
          Value was: "This Font Software is licensed under the SIL Open Font
          License, Version 1.1. This license is available with a FAQ at:
          https://scripts.sil.org/OFL" Must be changed to "Licensed under the
          Apache License, Version 2.0" [code: wrong]

    Result: FAIL

)

Related to #3750.

We now render Markdown tables directly, so this regular expression is no more.


[Issue 5775][Doc] Description of broker-tool added

Fixes https://github.com/apache/pulsar/issues/5775
Motivation
Add the description for broker-tool accordingly.
Refer to: https://github.com/apache/pulsar/pull/5768
Modifications
broker-tool section added.
Verifying this change
This change needs to be reviewed.
Does this pull request potentially affect one of the following parts:
Dependencies (does it add or upgrade a dependency): (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
Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? ( docs )
If a feature is not applicable for documentation, explain why?


Special Decks fail to load

among others, none of the decks in the special decks folder load. might be good to do a pass on the old ones to update/remove broken ones

yeah I need to redo most of those decks due to the GUID changes.  Dunno how to approach it yet.
The planes and schemes should work again though.  Turns out they were just missing altogether.


Hermes Chain Configuration Unclear

Crate
ibc-relayer(-cli)
Summary
Please close this issue if a duplicate.
Relayers are still a relatively new thing in the Cosmos/IBC landscape and there aren't many operators that run them or know how to run and configure them effectively.
The current configuration, specifically the [[chain]], doesn't really document what chains the values correspond to.
Specifically, I imagine a relayer will listen for events/txs from a source chain and relay them to a destination chain and this relayer will typically exist on the same host of the destination chain. It's not clear, for example, if the id is the source chain in this context or the chain being relayed (txs sent) to (I assume the destination chain). The same applies for the gRPC, websocket, and RPC endpoints. I think the rest of the configurations are pretty self-explanatory (they're for the destination chain).
Proposal
Document in the configuration if the relevant values correspond to the source or destination chain.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Based on the example config, https://hermes.informal.systems/config.html#example-configuration-file, it seems Hermes expects an operator to run a full-node for both  chains? If so, feel free to close this issue :)

Hi Alexandr,
Thanks for opening this issue, I realize our documentation is not very clear in the points you raised! I'll try to clarify here, and please follow-up if I'm missing anything.

Based on the example config, https://hermes.informal.systems/config.html#example-configuration-file, it seems Hermes expects an operator to run a full-node for both chains, typically on the same host? If so, feel free to close this issue :)

Hermes does expect connectivity to all endpoints (namely, rpc, grpc, and ws) for all chains specified in config.toml. But it depends what functionality do you want to use Hermes for concretely. Suppose you configure two chains cosmoshub-4 and osmosis-1 in config.toml, but all you ever use Hermes is to
hermes query clients cosmoshub-4

Then at runtime Hermes will not try to establish connection to any of the endpoints configured for chain osmosis-1.

typically on the same host?

Not necessarily on the same host.

The current configuration, specifically the [[chain]], doesn't really document what chains the values correspond to.

Each [[chain]] section has a an id tag, which specifies the chain identifier for that whole section.

Specifically, I imagine a relayer will listen for events/txs from a source chain and relay them to a destination chain and this relayer will typically exist on the same host of the destination chain.

If you run hermes start, then Hermes will relay on all channels available between all the configured chains. In this way, every configured chain will act as a source (in the sense that Hermes listens for events) and as a destination (to relay packets that others chains have sent). To restrict relaying on specific channels, you can use packet_filter policies.

Thanks for the excellent explanation @adizere!
Quick clarification,

In this way, every configured chain will act as a source (in the sense that Hermes listens for events) and as a destination (to relay packets that others chains have sent). To restrict relaying on specific channels, or uni-directionally, you can use packet_filter policies.

Every chain will act as source of events, but what does it do with those events? I imagine it constructs messages to sign and relay to other chains, right? So it'll need configuration for the destination chain(s) as well?
Every chain will acts as a destination, but it only for txs that the relayer signs and constructs from another source chain, right?

Every chain will act as source of events, but what does it do with those events? I imagine it constructs messages to sign and relay to other chains, right? So it'll need configuration for the destination chain(s) as well?
Every chain will acts as a destination, but it only for txs that the relayer signs and constructs from another source chain, right?

That's correct. If there are only two chains configured, then only packets between those two can Hermes relay, i.e., the two chains will serve as a source for each other, and likewise as a destination for each other's relevant events. Hermes will ignore all events that pertain to chains which are unknown (unspecified in config.toml).

Glad to help!
Is there a specific use-case you might have in mind that you'd like Hermes to cover (e.g., uni-directional relaying)? Also, I leave it up to you to close this issue if we managed to clarify it.

This clarifies it for me. Thank you so much.

Let's actually re-open this until we clarify the points above directly in the configuration/guide.


fix: [#6825] Update xunit and fix compatibility issues in unit tests

Addresses #6825
#minor
Description
This PR updates xunit package version from 2.4.1 to 2.9.0 and applies the respective changes to meet the new version requirements.
Specific Changes

Changed async void methods to async Task.
Swapped assert equal parameters to leave the expected value in the correct argument position.
Changed assert equal comparison of size collection 0 and 1 to Empty and Single methods.
Added async operators instead of the use of Wait, GetAwaiter, and Result methods.

Testing
The following image shows the unit tests working in Net 6.0 and 8.0.

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Testing.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Streaming.dll

@JhontSouth Conflicts

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Testing.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Streaming.dll


QueryStateAsync returns "Not Implemented" error when querying cosmos db via dotnet sdk + Azure container app, but works locally.

Hi I'm running into an issue when querying cosmos db with the dotnet sdk in an Azure container app. The query works locally, but not when deployed via bicep to a container app.
The error from Dapr returned is the following:
Status(StatusCode=\"Unimplemented\",` Detail=\"Not Implemented\")

Having checked with the container app documentation, they have now upgraded them to support 1.7.3, which I thought would support querying: https://docs.microsoft.com/en-gb/azure/container-apps/dapr-overview?WT.mc_id=DP-MVP-5001699&tabs=bicep1%2Cyaml
So I think I must be doing some wrong. Are you able to help me spot my error? Here's the info:
We just do a simple search for a custom property 'InstanceId' on the record. See below the code used to call the QueryStateAsync function.
private async Task<StateQueryResponse<SomeRecordObject>> QueryAsync(SomeSearchQueryObject query, Dictionary<string, string> metadata)
    {
        var stateStoreQuery = new StateStoreQuery()
        {
            Filter = new Filter
            {
                EQ = new Eq()
                {
                    InstanceId = query.InstanceId
                }
            }
        };
        var result = await _daprClient.QueryStateAsync<SomeRecordObject>(
            "cosmos-statestore",
            JsonConvert.SerializeObject(stateStoreQuery),
            metadata);
        return result;
    }

Here's the bicep that deploys the container app:
resource containerApp 'Microsoft.App/containerApps@2022-01-01-preview' = {
  name: name
  location: location
  properties: {
    managedEnvironmentId: containerAppEnvironmentId
    configuration: {
      secrets: [
        {
          name: 'container-registry-password'
          value: registryPassword
        }
      ]
      registries: [
        {
          server: registry
          username: registryUsername
          passwordSecretRef: 'container-registry-password'
        }
      ]
      ingress: {
        external: false
        targetPort: 5050
        transport: 'auto'
      }
      dapr: {
        enabled: true
        appId: name
        appProtocol: 'http'
        appPort: 5050
      }
    }
    template: {
      containers: [
        {
          image: containerImage
          name: name
          env: envVars
          resources: {
            cpu: json('0.5')
            memory: '1.0Gi'
          }
        }
      ]
      scale: {
        minReplicas: 1
        maxReplicas: 1
      }
    }
  }
}

Here's the bicep deploying cosmos db statestore dapr component (with the container app environment - which is omitted)
  resource cosmosStatestorDaprComponent 'daprComponents@2022-01-01-preview' = {
    name: cosmosName
    properties: {
      componentType: 'state.azure.cosmosdb'
      version: 'v1'
      ignoreErrors: false
      initTimeout: '5s'
      metadata: [
        {
          name: 'url'
          value: cosmosUrl
        }
        {
          name: 'masterKey'
          value: cosmosKey
        }
        {
          name: 'database'
          value: cosmosDb
        }
        {
          name: 'collection'
          value: cosmosCollection
        }
        {
          name: 'keyPrefix'
          value: cosmosKeyprefix
        }
      ]
      scopes: [
        cosmosScope
        cosmosSecondaryScope
      ]
    }
  }

The docker-compose file I use (which works locally) is below (I have omitted everything except the app, the dapr sidecar and the dapr placement). Not the only difference I can see between the local deployment and the container app deployment is local it uses grpc and container app it uses http - which I think shouldn't make a difference.
 myservicename:
    image: myserviceimagename
    build:
      context: .
      dockerfile: dockerfilepath
    env_file:
      - ./dev.env
    ports:
      - "50003:50001"
      - "5050:5050"
    depends_on:
      - placement
    networks:
      - myservicenetwork
  sidecar-dapr:
    image: "daprio/daprd:latest"
    command: [
      "./daprd",
      "-app-id", "myservicename",
      "-app-port", "5050",
      "-components-path","/components",
      "-dapr-grpc-port", "50001",
      "-placement-host-address", "placement:50000"
    ]
    volumes:
      - "./dapr/components/:/components"
    depends_on:
      - myservicename
    network_mode: "myservicenetwork"

  placement:
    image: "daprio/dapr"
    command: ["./placement", "-port", "50000"]
    ports:
      - "50000:50000"
    networks:
      - myservicenetwork

Finally, here is the dapr cosmos statestore yaml component:
apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: cosmos-statestore
  namespace: default
spec:
  type: state.azure.cosmosdb
  version: v1
  metadata:
    - name: url
      value: https://somecosmosurllol.documents.azure.com:443/
    - name: masterKey
      value: somemasterkey
    - name: database
      value: somedatabasename
    - name: collection
      value: somecollection
    - name: keyPrefix
      value: none

Another thing to add is that the function "GetStateAsync" DOES work via the container app, so it can connect and get data from the statestore, just not QueryStateAsync.
Thanks for your help in advance.

I am facing the same issue, any solutions

@k33lan - Did you find a solution on this


producer failover behavior when node in cluster failed

Description
Hi, I am trying to understand how the fail over would work when node in cluster failed. Say I have 3 nodes in cluster, and the producer have 3 of them configured:
BootstrapServers = "kakfa1:9092,kafka2:9092,kafka3:9092",
create a topic that have 3 replicas, single partition.  with all 3 nodes up and running. producer working fine. Then shutdown kafka3.  Should the producer automatically use the working one? or we have to rebuild the producer with kafka1 and kafk2?
my observations was it always hit the last one in the list first. so if I shutdown kafka3, above config will always fail the produce. But if I reverse the order to kakfa3:9092,kafka2:9092,kafka1:9092
it will always work.
What is the correct way to handle node failure with the library?
Thanks,
Xu
How to reproduce
Checklist
Please provide the following information:

[ ] A complete (i.e. we can run it), minimal program demonstrating the problem. No need to supply a project file.
[ ] Confluent.Kafka nuget version.
[ ] Apache Kafka version.
[ ] Client configuration.
[ ] Operating system.
[ ] Provide logs (with "debug" : "..." as necessary in configuration).
[ ] Provide broker log excerpts.
[ ] Critical issue.

the producer is designed to automatically handle/recover from all errors where possible. Note that you don't need to specify all the brokers in the bootstrap.servers list - the client will discover those you don't specify and order shouldn't matter.
when you say 'fail to produce', what is the error?

I got an aggregated exception, the inner one is a timeout.
I changed the timeout to 10 sec, or default. makes no difference.
The reason I put all 3 in config was that I was hoping to give the library more connection points. If I just give one, e.g. kafka1, and that one happened to be down, there will be no chance for the producer to reach other two working ones.

when I use another java producer client, the order doesn't matter, it produced fine. so I don't think this was due to the cluster setup. Actually, there seems to be a background thread printed some error like:
%3|1557971151.275|FAIL|kafkactipsclient#producer-1| [thrd:kafka3:9092/bootstrap]: kafka3:9092/bootstrap: Connect to ipv4#<IP_ADDRESS>:9092 failed: No connection could be made because the target machine actively refused it... (after 2009ms in state CONNECT)
%3|1557971151.275|ERROR|kafkactipsclient#producer-1| [thrd:kafka3:9092/bootstrap]: kafka3:9092/bootstrap: Connect to ipv4#<IP_ADDRESS>:9092 failed: No connection could be made because the target machine actively refused it... (after 2009ms in state CONNECT)
kafka3 was down.
I have updated the library to the latest 1.0.0

i don't know without more information - you can get verbose client logs by setting the Debug config property to all. That error message is consistent with the broker being down (note that you can redirect it by setting a log handler if you don't want messages output to stderr).

sure, it was a test program so I can get more debugging info.
thanks.

ha, looks like the kafka2 is indeed having trouble. as kafka1 shows 2 brokers, while kafka2 shows  0 in the metadata returned. when kafkas3 in the last position, the connect went to kafka2 first, so failed.
I think this is still a setup problem. Thanks for pointing me to the right verbose flag.

Found my problem: the kafka1 has a newer version than kafka2 and kafka3.

Matt, just let you know after upgrade the 2 and 3 to the latest, the failover is working properly.
Thanks,
Xu


Add invoice settings to customers

Customers were missing invoice_prefix and invoice_settings.
These changes enables a PaymentMethod to be set as the default for invoice or subscription payments, by setting invoice_settings.default_payment_method on the Customer, to the PaymentMethod’s ID, as described here:
https://stripe.com/docs/api/payment_methods/attach

Good PR. I had to monkeypatch StripeMock::Data to make my tests pass but I would rather not having to do that :s

Could you change this as well?
https://github.com/rebelidealist/stripe-ruby-mock/blob/697f8c0ae1f485c10a4d3af9b7599a144f9d91f6/lib/stripe_mock/request_handlers/subscriptions.rb#L258-L264
Currently if the customer has a default payment method but no default source (ie, only using the new SCA-compatible API), creating a subscription will fail.

Fixed :)


(#65) Support new option unorderedObjects

See #65.

@addaleax I'll whip up some tests. :)

@addaleax Done!

Released in v1.3.0. Cheers!


Introduce HeaderButton Component and Containers

This PR introduces a HeaderButton functional component to render all buttons to be used in the header. I am using Font Awesome again because grommet-icons is lacking and I the package allows me to rotate icons with a prop (needed for the layout button).
All buttons are visible in the storybook yarn storybook

Is the design actually using font awesome icons or can we grab the svg path and make a custom one? Grommet icons support customization and I can link to an example of us doing it in the monorepo.

The design uses font awesome for items like the save button and icons in the layout button

No, this is supposed to be based on the inVision design, which is a bit different from the basic Grommet checkbox. https://projects.invisionapp.com/d/?origin=v7?origin=v7#/console/17925240/378388615/preview

I don't have access to that inVision file unfortunately, but if the design is as intended, then cool! 👍

The oddly-sized toggle box should now be taken care of.


Desktop app is not consistently switching focus back to teams

I often have a team's call open while also working in Azure Dev Ops (in Chrome) or addressing communication on other apps like Slack or Skype. I've noticed that the desktop app doesn't always set the focus back to teams for the keypad macros to be correctly processed. In Chrome it opens the bookmarks manager ('Ctrl'+'Shift'+'O')  so it doesn't seem like that 'Alt'+'0' is being recognized by the GlobalKeys app. I tried to increase the 'sleep' in the Pico code from .4 to .5 speculating that maybe the desktop app isn't able to process everything quickly enough but that didn't seem to make any difference. I'm curious if you have any ideas as to why it might not be working before I try my hand at debugging the project.

Hey,
Annoying isn't it? I thought I cracked it to be fair, but obviously not.
I'll be having another mess with it at some point.
What I was struggling with is basically identifying what the 'Call' window is, since you can't get it from the window handle title. I've used an automation UI API to drill into the window and identify components, such as the 'Leave' button to determine if it's the call window...
Trouble is, if the call window is out of focus or worse minimised, the UI api won't bring anything useful back. Therefore, I've implemented a shell window event handler, which registers every new teams related window in a cache, so at some point it has to be in focus and will identify the 'leave' button (amongst a couple of other) and then refers back to it by the handle id when the global key event handler sees a 'Alt + 0'.
When you close the call window, that handler ID is removed from the cache, and there's a timer that periodically checks if the handler is still valid, and if not, removes from the cache. The reason being, if more than one 'call window' exists in the cache, then the program doesn't know which one to activate.
If no call window is present, is should activate the 'main' Teams window... Then the 'Answer Call' buttons can work...
This technique seems to be great for me. It seems to work all the time...
I wonder if. Is your teams call window getting focus, then your original window taking over it again or does it appear the call window isn't activating at all?
Do you have multiple 'chat' windows open (i usually have just the main teams window open, plus a call window if i'm in a call). I've noticed the automation UI API takes an ages to iterate over control when multiple chat windows are open - In which case, the program might not have enough time to identify the call window, and so the 'Alt + 0' is actually lost.
Also, what version of the app are you using? the 1.0.2 is the latest.
Regards,
Jamie


Metadata

Hello, thanks for your work on this great project.  I have been experimenting with it and am impressed!
My question is regarding metadata.  Specifically, is there a way I can associate metadata (e.g., a url or file path) with a particular piece of textual content I have added via the insert function?

I´m waiting as well to an answer about this. Check it out. maybe we can consolidate rather than opening multiple threads!
#422
https://github.com/HKUDS/LightRAG/issues/422


Update purifycss.js

Just added one line comment about the option "rejected" currently working.

You should also add in purifycss.js, -at least- line 108:
        case 'combinator':
            ruleString += rulePart[1];
            break;
        case 'nthselector':
            ruleString += ':' + rulePart[1][1];
            break;


Add Kitematic documentation to docs.master.docker.com

Signed-off-by: Sven Dowideit<EMAIL_ADDRESS>

oh, no, this is a public repo, i don't need to do it this way :)


feat: Adding Location to ease out filtering

What feature?
Adding proper location (state, district, pincode etc.) in Register Donor Form
Adding location in Find Blood page to filter out locations
Add screenshots

Record

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue

Interesting, can you explain more about where we can add this? @msanchariii

Eg. we can add pincode field and a City/villege field while registering Donors. And In Find Blood we can add a search bar to filter out the donors. As we all know in emergency cases one must find appropiate blood donors as near as possible.
we may need to modify prisma schema.

Great Idea, Go ahead @msanchariii

Hey @Saurav-Pant I've Implemented the feature.

Added 'pincode' field in Prisma Schema
Modified the api/DonorForm
Added City/town/Village field in DonorForm
Added a search bar to filter by location in FindBlood Page

Also wanted to mention, I was facing a GET request error as there wern't any pincode field in existing data. It got fixed when I made sure every Donor had pincode.
Now, May I create a PR ??


Test for idempotency of Ansible calls

Proposed Changes

Add an ensure_idempotency func that performs 2 calls and check for 'changed' result
Make sure minimal_api_version kw is passed to call_kafka_topic func
Just in case, remove usage of mutable objects as default kw value to avoid possible side effects

Awesome! LGMT!


fix archiving to postgres

changes

ensure the version string matches the sql table name (which is automatically lowercase) after it's been written to the edm-data.recipes DB

Shouldn't this only have the change here? This is going to change output folder paths in DO. Not sure if we currently have any that output versions with capital letters, but still think it's best to only do this specifically for the table name and nowhere else

What specific dataset is this needed for?

@fvankrieken
good call that this only has to apply to the sql statements
and I don't think there'd be any issue with inconsistent casing between csv and postgres (other than we'd probably prefer consistency)
this came up when archiving new adult use input data and I wrote capital letters in the version (20230426_noM2M3)

Why were the noM2M3 making it in to the table? I'm assuming you just added them by mistake when specifying the version and this created the issue? We should keep the convention where we pass the version just as a YYYYMMDD without any additional characters

I did it on purpose since there are indeed multiple versions of the source data that don't only vary by the date they were acquired, they were generated with different tax lot exclusion criteria (20230425 vs 20230426_nom2m3). the other two options seemed to be:

archive the new data as 20230426

since outputs based on 20230425 and 20230426 source data were both needed by GIS, the script had to be run on both of them. so the answer to the question "which is the nom2m3 output?" would have to be stored somewhere
seemed better to store the answer as the data version (and the output folder) rather than in comments, human minds, or re-investigating later lol

create a different template yml file

doesn't seem worth creating an entirely different database schema (e.g. recipes.dcp_proximity_establishments_nom2m3) when they have the same columns/value types and are gonna be processed in the same way

in data library, if a version isn't explicitly declared when archiving, it does still use the date. so we still have that feature. seems like that's how convention is implemented, via the default behavior of not passing a version

@damonmcc that makes sense, I think that the way you went about it was good. Don't think it would have made sense to have two different templates


NextJS router.push causes runtime exception

Summary
Introduced in https://github.com/jbx-protocol/juice-interface/pull/1555
Last good commit: a31623ce01e3b1a0f2a2429e57c69492a5e5f8a6
Fixed temporarily with https://github.com/jbx-protocol/juice-interface/pull/1631.
Discussion: https://discord.com/channels/775859454780244028/866040669712678942
Steps to reproduce

git checkout 99d2fdc40643d8ad2ff567e073f9d7969ecca816 (revert the revert)
Create a project
Wait for the tx success
Observe page redirection. A runtime exception should occur (see screenshot)

Relevant logs and/or screenshots

I have discovered that this is actually an issue as to where the network provider sits:
If the network provider is embedded in each page individually, upon switching pages (while a wallet is connected) it will fail. By adding it to the app level (_app.page.tsx), this solves it. See https://github.com/jbx-protocol/juice-interface/pull/1673


ocamlPackages.lame: 0.3.6 -> 0.3.7

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/ocaml:lame/versions.
meta.description for ocamlPackages.lame is: Bindings for the lame library which provides functions for encoding mp3 files
meta.homepage for ocamlPackages.lame is: https://github.com/savonet/ocaml-lame
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.3.7 with grep in /nix/store/i50lklhf30p3b6pb0sj887425sdy69zz-ocaml4.14.1-lame-0.3.7
found 0.3.7 in filename of file in /nix/store/i50lklhf30p3b6pb0sj887425sdy69zz-ocaml4.14.1-lame-0.3.7

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
liquidsoap

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/i50lklhf30p3b6pb0sj887425sdy69zz-ocaml4.14.1-lame-0.3.7 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ocamlPackages.lame https://github.com/r-ryantm/nixpkgs/archive/acf53d31d64c1a556563376a46cdb95827ac3b78.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/i50lklhf30p3b6pb0sj887425sdy69zz-ocaml4.14.1-lame-0.3.7
ls -la /nix/store/i50lklhf30p3b6pb0sj887425sdy69zz-ocaml4.14.1-lame-0.3.7/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

liquidsoap
ocamlPackages.lame

Maintainer pings
cc @dali99 for testing.

@ofborg build ocamlPackages.lame ocamlPackages.lame.passthru.tests


Cannot run Daemon Example: driver not supported

Hi All,
I am trying to get the example on the readme working and am bumping into some problems.
Running docker run --name daemon --privileged -it rancher/docker yields the following:
INFO[0000] Listening for HTTP on unix (/var/run/docker.sock)
ERRO[0000] 'overlay' not found as a supported filesystem on this host. Please ensure kernel is new enough and has overlay support loaded.
FATA[0000] Error starting daemon: error initializing graphdriver: driver not supported

Trying to run the AUFS container (docker run --name daemon --privileged -d rancher/docker -d -s aufs) yields:
FATA[0000] Error starting daemon: error initializing graphdriver: driver not supported

docker version
Client:
Version:      1.9.0-dev
API version:  1.21
Go version:   go1.4.2
Git commit:   eb7b2ed
Built:        Fri Aug 21 16:15:36 UTC 2015
OS/Arch:      linux/amd64

Server:
Version:      1.9.0-dev
API version:  1.21
Go version:   go1.4.2
Git commit:   eb7b2ed
Built:        Fri Aug 21 16:15:36 UTC 2015
OS/Arch:      linux/amd64

Docker info
Containers: 2
Images: 7
Storage Driver: devicemapper
 Pool Name: docker-253:2-1731808-pool
 Pool Blocksize: 65.54 kB
 Backing Filesystem: extfs
 Data file: /dev/loop0
 Metadata file: /dev/loop1
 Data Space Used: 1.851 GB
 Data Space Total: 107.4 GB
 Data Space Available: 20.02 GB
 Metadata Space Used: 1.749 MB
 Metadata Space Total: 2.147 GB
 Metadata Space Available: 2.146 GB
 Udev Sync Supported: true
 Deferred Removal Enabled: false
 Data loop file: /var/lib/docker/devicemapper/devicemapper/data
 Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
 Library Version: 1.02.93 (2015-01-30)
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.19.7-200.fc21.x86_64
Operating System: Fedora 21 (Twenty One)
CPUs: 8
Total Memory: 15.37 GiB

Running on Fedora 21 3.19.7-200.fc21.x86_64
I suspect that this has to do with the fact that I'm not running on Rancher, but It would still be great to be able to get a "From Scratch" version of Docker in a container. Wondering if this is possible on Fedora?
Any help would be greatly appreciated.

Fedora 21 should have overlay.  Does modprobe overlay work?  I can launch a F21 machine and try it out.

Aha. That was silly of me. modprobe overlay got things working again. Thanks a bunch!


Support for Meteor

This is more of an idea and/or feature request, if at all possible. It would be useful to have this data while developing a Meteor application.
Thanks for the tool!

I work with Meteor too, would be really nice to see it there

I was able to get this module working with Meteor. All it needs to do is to modify the Meteor's execution script:

The script is: /Users/username/.meteor/meteor (if you are on MacOS)
Replace the last line used to start node server at the bottom with these lines:

function npm_package_is_installed {
  # set to 1 initially
  local return_=1
  # set to 0 if not found
  ls node_modules | grep $1 >/dev/null 2>&1 || { local return_=0; }
  # return value
  echo "$return_"
}

NODEJS_DASHBOARD="$(which nodejs-dashboard)"
ARR=("" "run" "--settings")

if [[ $1 == "--dashboard" ]] && [[ " ${ARR[*]} " == *" $2 "* ]]; then
  if [[ ! -n "${NODEJS_DASHBOARD}" ]]; then
    echo "nodejs-dashboard is not installed globally"
    exit 1
  fi
  if [[ "$(npm_package_is_installed nodejs-dashboard)" == "0" ]]; then
    echo "Please install nodejs-dashboard to you project"
    exit 1
  fi
  shift
  exec "$NODEJS_DASHBOARD" "--" "$DEV_BUNDLE/bin/node" "-r" "nodejs-dashboard" ${TOOL_NODE_FLAGS} "$METEOR" "$@"
else
  exec "$DEV_BUNDLE/bin/node" ${TOOL_NODE_FLAGS} "$METEOR" "$@"
fi

Then you will need to add --dashboard option to meteor command to run with nodejs-dashboard. Without --dashboard, it will works normally.
meteor --dashboard
meteor --dashboard --settings settings.json
meteor --dashboard run
meteor --dashboard run --settings settings.json

Also, your Meteor project should have version 1.4., otherwise the measurement for cpu and memory may not work. This is because Meteor 1.3 and older run with Node 0.10. which seems uncompatible with this module.

@nlhuykhang That would be a pretty nice addition to Meteor. Do you intend to push a PR on their repo?


test/common/quic/platform/quic_platform_test times out and fails

example failure:
INFO: 29581 processes: 17191 remote cache hit, 11058 internal, 1 local, 1331 remote.
//test/common/quic/platform:quic_platform_test                          TIMEOUT in 2 out of 2 in 305.0s
  Stats over 2 runs: max = 305.0s, min = 301.8s, avg = 303.4s, dev = 1.6s
  /build/tmp/_bazel_envoybuild/b570b5ccd0454dc9af9f65ab1833764d/execroot/envoy/bazel-out/k8-opt/testlogs/test/common/quic/platform/quic_platform_test/test.log
  /build/tmp/_bazel_envoybuild/b570b5ccd0454dc9af9f65ab1833764d/execroot/envoy/bazel-out/k8-opt/testlogs/test/common/quic/platform/quic_platform_test/test_attempts/attempt_1.log

https://dev.azure.com/cncf/envoy/_build/results?buildId=113584&view=logs&j=8c169225-0ae8-53bd-947f-07cb81846cb5&t=6b0ace90-28b2-51d9-2951-b1fd35e67dec&l=1025
according to azp analytics this test has failed 1/1 times in the last 14 days
https://dev.azure.com/cncf/envoy/_test/analytics?definitionId=11&contextType=build

cc @RyanTheOptimist

Hm. I don't think I understand that data here:
https://dev.azure.com/cncf/envoy/_test/analytics?definitionId=11&contextType=build
This seems to show it only failed once in the last 30 days but shows 0 other runs. Do you understand how to interpret that data? (In particular, I don't think anything has changed with this recently so I don't think anything has gotten worse, but who knows :>)

im pretty sure i have seen this failing previously so i also dont think its new - but the azp evidence suggests that it never passes
not sure why it shows it as only having run once - i have tried putting all filters on (ie include passing etc) and searching for the failing test - but it never seems to show anything different
it could be that its not a direct target so it shows if it fails but not if it passes - but that is just conjecture
either way  - i think this is test is at least flakey if not permafailing so would be good to address

also re azp fail analytics - you can post a link with filters or subpages etc - it always just points back to the main page - otherwise i would have posted what i was actually looking at

Hm, I don't think the azp evidence suggests that it never passes, does it? I think it says that it's failed once in the past 30 days. It could have also failed other times, but there is no data either way on that front, annoyingly. If it never passed then it would block PRs, right?
The log shows a timeout in the QuicThread test so a deadlock in that test is certainly plausible. I'll do some digging.
(Sorry about the useless URL. I totally missed that. :<)

Hm, I don't think the azp evidence suggests that it never passes, does it?

it has failed 1/1 times - 0% pass rate

Heh, sure. But my point is that I'm confident this test has run WAY more than 1 time over the last 30 days.

I'm confident this test has run WAY more than 1 time over the last 30 days.

i would have thought so too - but that is what im getting from azp - not sure how to understand that


ARROW-9106: [Python] Allow specifying CSV file encoding

Also add a C++ InputStream wrapper that transforms data from the stream according to an arbitrary callable.

Should the user guide also be updated? (the changes in docs/source/cpp/csv.rst  and docs/source/python/csv.rst from https://github.com/apache/arrow/commit/1325d139b7301df40ec0b22a74afd675aebe0d94)

Ah, right, thank you.

I'll add a test that encoding errors are propagated correctly (hopefully :-)).

Ok, I added some tests for error propagation. I'm going to merge if CI stays green.

Looks good here. Merging


Consider deprecating the 'Simple Transaction Builder' in the long run

I suggest we consider deprecating 'simple transaction builder' in the long run.
More practically, we should first have to get proper debug tooling for the advanced tx builder but afterwards we can mark the 'simple tx builder' as deprecated (but still available). Then in the V1 release it could be fully removed.
The Simple Transaction Builder
The simple transaction builder is the original way to spend from Contracts using the CashScript SDK. It has the following features:

Automatic UTXO selection
Automatic change output
Call transaction building directly on the contract object

Together they try to abstract away 'transaction building' in favor of a wallet-like API where you can "just spend from a balance":
const transferDetails = await contract.functions
  .transfer(new SignatureTemplate(bobPriv))
  .to('bitcoincash:qrhea03074073ff3zv9whh0nggxc7k03ssh8jv9mkx', 10000n)
  .send();

This method breaks down fast: what if you want to send the contract.getBalance() amount, how to spend this full contractBalance when taking into account fees without knowing the number of inputs? What if you want to spend from covenants, where having multiple inputs requires the shape of the transaction to change?
Bad Abstraction
The simple transaction builder is a bad abstraction, it hides the fact that transactions are built with in- and outputs and assumes that a similar to a wallet smart contracts usage involves 'spending some amount from a total balance, returning the change".
The 'simple transaction builder' has a lot of extra settings to override this default fee/change behavior making the API less simple and more complex than the 'advanced tx builder'.
In practice most usage of the simple transaction builder involves .from() and .withoutChange(), at which point you'd be better just using the advanced builder. It defeats the point of the simple transaction builder and is a good indication it is a bad abstraction.
const outputAmount = contractUtxo.value - 1000n
const transferDetails = await new TransactionBuilder({ provider })
  .addInput(contractUtxo, contract.unlock.spend(aliceTemplate, 1000n))
  .addOutput({ to: aliceAddress, amount: 1000n})
  .send()

Anti-features
I argue the 3 features of the simple tx builder are actually anti-features:

This assumes the selected inputs don't affect the transaction shape besides the changeAmount, but in many cases the inputs DO define the full tx shape (including outputs) like when sending the max balance or working with (self replicating) covenants where the inputs decide the structure of the outputs.

Sending change back to the contract is a bad, weird default that only makes sense in this weird abstraction level where contracts are seen as a 'wallet' where to spend a balance from. However this is rarely how contracts really work.

calling a function on an object seems like a nice abstraction but it hides the core of the UTXO model: transactions have in and outputs. This causes new developers to have a wrong mental model from the start and causes them to have to learn about the 'simple tx builder' workarounds like  .from() and .withoutChange(), when they'd be better just using the advanced builder

2 Ways of doing things
I consider it very bad that there are two ways of doing things, both with different abstraction levels and their own API documentation.
It introduces the questions what the difference is between the two and causes people to understand both at some basic level.
The advanced ts builder is also just accessible to beginner devs, but it starts them of with the correct mental model instead of confusing abstractions. Instead of calling 'functions on contracts' the mental model becomes 'creating a transaction with in- and outputs'
There would just be one 'transaction builder' and one documentation page. This definitely feels like the correct direction long term

I would agree, having two different ways to do the same thing can cause some confusion.
Also agree that abstracting away the change output management in the simple builder can lead to some potentially bad situations if someone learns to ignore change outputs then applies that logic to the advanced builder.
I think it also does help with learning the UTXO mental model, and I would keep .addInput/.addOutput of the advanced builder to reinforce this over .to/.from of the simple builder.
I myself immediately defaulted to always using .withoutChange() and .withoutTokenChange() on the simple builder.
All assuming the advanced builder gets debugging 😁


Highlighting (copying) from left edge of window with cursor in full screen fails

Environment
10.0.18362.0 Terminal: 0.4.2382.0
Steps to reproduce
Maximize the terminal window.  Move cursor all the way to the left of the screen and start dragging to highlight the text on that row.
Expected behavior
It starts highlighting the row you are on (the text).  This is similar to putty, cmd, and powershell outside of terminal.
Actual behavior
The row you are on is not highlighted.  You must click on the first character of the row, not to the left of it.
This requires for more precise highlighting to avoid missing what you want to copy and is far slower.

Does this happen also in non-maximized window and not happen if you disable padding?
If yes, then this is #1367

@mcpiroman indeed, good catch lets hope #1672 or a similar fix gets merged soon.  For now padding off works to solve:)  Closing as dupe.


Is jsconsole.com still a thing ?

Hi,
As far as I understand with https://github.com/remy/jsconsole/issues/119, https://github.com/remy/jsconsole/issues/115, https://github.com/remy/jsconsole/issues/105, https://github.com/remy/jsconsole/issues/104 and https://github.com/remy/jsconsole/issues/103, jsconsole.com cost you money and you no longer wan't to support it. I understand that.
However, jsconsole is a great tool, that can help many of people debugging website and more. I don't understand why you would keep the jsconsole.com domain alive, without a clear message telling it just no longer work because of "money".
There is so many tutorials around the web that tell to use jsconsole but, well when you try it you get a 404 on the remote.js script ? If you no longer want to support this project, maybe write in the README it's deprecated or opened for donation, money won't comes by itself.
Also, could you at least add documentation to self-host this project, or alternative to jsconsole ?
Thank you.

jsconsole.com is a thing. See latest commits for support of maintenance.
Currently hosted at…you guessed it, https://jsconsole.com
Remote is not supported.

Or maybe create a small simple solution that users can self host locally?

Having just spent an hour debugging because I didn't know this tool was deprecated, trying to figure out why the remote script call was failing: Thanks. A lot.


Temporarily removing promise handling in response to customer report.

Closing due to customer report that their integration is behaving as expected now, and due to the issue emerging after an update of core from 1.13 to 1.19.  (1.19 itself might not have been the cause of the issue.)


Add WatchStreamExt::reflect to allow chaining on a reflector

Thought this might be nicer than the separate function call now that we allow doing everything on the stream chain. It also makes it clearer to see what parts get reflected (particularly pre or post modify calls) because it follows from the chain order:
watcher(api, watcher::Config::default().any_semantic())
    .default_backoff()
    .modify(|pod| {
        pod.managed_fields_mut().clear();
    })
    .reflect(writer)
    .applied_objects();

NB: I did some rename experiments on this branch but it was a bad idea and force pushed it away. Sorry about the noise.

Just some small nits.
Appreciate it!

Should the kube::runtime::reflector be removed in favor of this?

Maybe. I wouldn't want to remove the fn immediately on the introduction here, but if this doesn't have any downsides, then we can probably put a longer lasting deprecation marker on reflector() in the future. For now have only tried to make some reasonable set of docs on the new method and changed over some internal usage.


Cleaned up the header

Added the notifications icon, did some rearranging, bumped up the size of the icons and improved the transitions. The home icon is now back to the right so that the header has balance (3 icons to the left of the search bar and 3 to the right).

I also bumped up the icon size in my last commit. I'll de-conflict them in my next commit.


remove the temporary version.jackson-databind in the pom.xml

It was introduced due to an unmatched release from Jackson core and it is not necessary now.

 remove the temporary version.jackson-databind in the pom.xml light-oauth2 master

 remove the temporary version.jackson-databind in the pom.xml light-oauth2 1.6.x


Turn off teams button

@toolmantim I kicked off this PR for you (to make sure that the the disabling worked in the backend). I wasn't sure what styling you had in mind so I just did something simple for now.
You can put it next to the "There are no teams in this organization" message if you want, but you'll just need to make sure that it doesn't show when you search for a team with no results (cause I think they're the same message)

Just updated it to this:

The intro/enable screen needs updating too to match, but that's a separate PR.


Missing Content-Security-Policy header

Hi,
i noticed that Mosparo currently does not set Content-Security-Policy (CSP) headers, which are important security headers.
Therefore I have created a first draft with which these are set via the ".htaccess" file, but in my draft "script-src" and "style-src" are set with "unsafe-inline".
Pull request: https://github.com/mosparo/mosparo/pull/44

Hi @Digi92
Thank you very much for bringing this issue to our attention.
I will try to add a fix without using the unsafe-inline. It should be possible, but I have to make some adjustments.
We will reply as soon as we've fixed this issue.
Kind regards,
zepich

Hi @Digi92
Thank you again for your issue and the pull request.
We've investigated the issue and added the headers you added in your pull request. We're using a Symfony module to provide these headers. We can hash our inline JavaScript codes with the module and ensure we do not need the unsafe-inline flags. By default, only the default-src option is set since mosparo is using resources from the mosparo host itself.
If a page has an inline script, the bundle automatically adds the hash in the script-src option.
I've removed possible inline styles, URLs starting with data:, and other things that cannot work with the CSP headers. There is one exception: on the design settings page, we cannot provide a clean CSP header. The color picker uses inline styles and data: URLs. Therefore, we must now use the unsafe-inline option on this page. We will fix that with the next major version (v0.4) since we're planning a refactoring of the design settings page anyways (see #46).
I hope this solution fits your expectations too. I will close your pull request without merging (because changing the .htaccess file would only work for Apache servers, not for nginx).
The new version, v0.3.15, with the security headers and other fixes, will be released tomorrow.
Thank you very much for making mosparo better!
Kind regards,
zepich


QUESTION: Mouse vs touch movement X Y calibration to viewport px is different. How do you calibrate 1:1 to px?

useDrag produces different x y calibration ratios with the viewport depending upon the input source.
For example, a movement mx result of 1 is not equal to 1px when input is from a mouse in a browser. However, it is correct 1:1 when input is from a touch device.
How do you calibrate this?

What do you mean? mouse events are in pixel coordinates as far as I know. Can you show a sandbox illustrating the problem?

Thanks @dbismut.
Upon further investigation, this was an css related issue and unrelated. It was odd that touch worked perfectly while mouse input would produce different behavior.
Closing.

touch-action: none, am I right?

Indeed, plus I had overly complex flexbox and perspective css that adjusted offsets further complicating the situation. It's working perfectly now, but made me scratch my head for a bit.


[NETBEANS-54] Module Review api.search

The changes look to only touch files, that I would have seen under "central problems". The change itself looks good though.

Right. I thought, basically, "let's just get it done".

Hope someone can do the merges of my PRs, such as this one, haven't got that working yet.

Done.


Hierarchy modify on entity

entities are associated to a layer at the creation phase and no further modification is allowed (i.e. to replace the owning layer with another one).

Maybe this should be possible but only if there is no risk session for this entity.

New feature of change the layer of an entity (with hierarchy control to avoid inconsistent data) was added


Issue with Linear Drift Correction

And when I try to perform linear drift correction:

When I start a new editing session, the drift correction seems to work fine.

Seems to be addressed. Closing.


Set a basic autoresponder for noreply emails to the Zulip server

Since Zulip sends outgoing emails (with a noreply@ address), we should probably configure Zulip to make that address bounce in some reasonable way rather than timing out.
See https://chat.zulip.org/#narrow/stream/production.20help/subject/Bouncing.20Emails/near/233987 for some context and useful ideas for how to do this easily.

@zulipbot claim

abandon


Updated automated install instructions and install script

I updated the automated install instructions to where the user wouldn't have to clone the repo
(since the script already does this)

I found a way to use for do loops for the path to steam and used them to replace the mess that was the install-theme function

If you're using curl already, why not download the theme itself with curl? Main benefit is that it reduces the amount of stuff you need to download (bc we can just make it download the specific flavor), but it also would remove the git dependency, and maybe even the wget and unzip deps

If you're using curl already, why not download the theme itself with curl? Main benefit is that it reduces the amount of stuff you need to download (bc we can just make it download the specific flavor), but it also would remove the git dependency, and maybe even the wget and unzip deps

because you can't download certain directories of git repos (at least not in a script), only the full repositories

If you're using curl already, why not download the theme itself with curl? Main benefit is that it reduces the amount of stuff you need to download (bc we can just make it download the specific flavor), but it also would remove the git dependency, and maybe even the wget and unzip deps

I could use curl to download the theme, but using it for metro and the metro patch would just make the script WAY longer than it needs to be. Also, I honestly don't even know why the unzip dependency is still in there considering that it was only needed for SFP

You could make a call to the gh content API, I.E. https://api.github.com/repos/catppuccin/steam/contents/themes/macchiato-- it gives a list of files, which you could grep and download.
Also, wym we can't test it, i tested the updated script on my machine like 20 minutes ago and it worked without any problems whatsoever?

using gh would add yet another dependency and the user would also have to log into the GitHub CLI before being able to use the script.
? it's an api call, you just curl it
Also, wdym we can't test it, i tested the updated script on my machine like 20 minutes ago and it worked without any problems whatsoever?
github just had a ton of SSL certificates expire

using gh would add yet another dependency and the user would also have to log into the GitHub CLI before being able to use the script.
? it's an api call, you just curl it
Also, wdym we can't test it, i tested the updated script on my machine like 20 minutes ago and it worked without any problems whatsoever?
github just had a ton of SSL certificates expire

bruh, I'm stupid, i saw gh and assumed you meant the GitHub cli ☠
I'll update it when i get back home lmao

also looks like the SSL just got renewed. No pressure to do it btw, it's just a tiny time saver

also looks like the SSL just got renewed. No pressure to do it btw, it's just a tiny time saver

nevermind, just found out that the GitHub mobile app allows making edits to files in the pull request ui

also looks like the SSL just got renewed. No pressure to do it btw, it's just a tiny time saver

Nevermind, apparently the only resources you can use to download a certain directory from a repository are either subversion or a website (which would mean either another dependency would have to be added, or it wouldn't be nearly as quick as it currently is)

@Oman395 good to merge?

Should be good to merge if the install is working, can't test rn but it it's functional we can merge prob

Should be good to merge if the install is working, can't test rn but it it's functional we can merge prob

just tested it with my main steam install, then tried it after renaming the steam folder (to test the part that happens if the script can't find an installation), both of them work just fine

@Oman395 good to merge?

Merged, sorry, I forgor

Merged, sorry, I forgor

it's fine lol


Feature/gdocs paragraphs linebreaks

https://cnissues.atlassian.net/browse/COPILOT-7832
Convert newlines in gdocs paste to LineBreak and Paragraph annotations.
In the gdocs source converter, we divide the document into regions defined by each boundary of a block annotation. Within each block boundary, look for single newlines to convert to a LineBreak and multiple newlines to divide the block boundary into Paragraphs, as Paragraphs must not cross the boundary of a BlockAnnotation. Thus something like
--  Blockquote
    Some text
    More text separated by a single newline

    More text separated by multiple newlines
    --  List
        --  List Item
            Some text
            More text separated by a single newline

            More text separated by multiple newlines
        --  List Item
            More text

would be divided into LineBreaks and Paragraphs accordingly:
--  Blockquote
    --  Paragraph
        Some text
        --  LineBreak
        More text separated by a single newline

    --  Paragraph
        More text separated by multiple newlines
    --  List
        --  List Item
            --  Paragraph
                Some text
                --  LineBreak
                More text separated by a single newline

            --  Paragraph
                More text separated by multiple newlines
        --  List Item
            More text

Some convenience methods have been added to @atjson/document:
Document.match(regex, start?, end?) will return an array of {start, end} for each regex in a given region of a document
NamedCollection.outerJoin and Join.outerJoin mimics a left outer join in SQL, which keeps all items on the LHS of a join when the RHS may be empty. This allows queries such as finding all LineBreaks contained in a List but not in any ListItems.
Convenience methods like Collection.reduce, Collection.forEach, Join.forEach, and Join.where have been added.

I will be adding more tests to the gdocs source converter test, but in the meantime please take a look if y'all have time!


VARCHAR fields are padded with additional characters until field's maximum length is reached when trimming is disabled

First of all, thank you for all the effort that you've put in releasing this library and sharing it. It looks great.
We've been testing it and we've detected that when we process ebcdic files with a varchar field at the end of the record, when it gets translated into ascii(utf-8) it gets padded with additional characters at the end of the field until it reaches the maximum length defined for that field in the copybook. Obviously this happens only when the varchar field in the ebcdic file uses less bytes than the maximum specified in the copybook and cobrix-trimming-option is set to "none".
For instance, given the following copybook (taken fom za.co.absa.cobrix.spark.cobol.source.regression.Test04VarcharFields):
01  R.
  03 N     PIC X(1).
  03 V     PIC X(10).

If the input ebcdic record was:
------------------------------------
| RDW                | N    | V    |
------------------------------------
|0x00 0x00 0x02 0x00 | 0xF4 | 0xF1 |
------------------------------------

The expected output when trimming is set to "none" should be just 2 bytes:

N -> the ebcdic2ascii translation for 0xF4
V -> the ebcdic2ascii translation for 0xF1

But instead of that the actual output is 11 bytes:

N -> the ebcdic2ascii translation for 0xF4 -> OK
V -> the ebcdic2ascii translation for 0xF1 plus 9 additional characters (whitespaces or non-printable, depending on the codepage used) -> Not OK

Would it be possible to fix this so that the translated field doesn't get those extra characters that don't come from the source field ?
Also, are there any plans to support multiple varchar fields in the copybook ?
Thank you!

Thanks for the detailed description of the feature request. This makes perfect sense, we are going to implement it.
Please, elaborate more on the multiple varchar fields idea. I'm not sure how it can look in a copybook and what would be the layout of a data file in this case.
Please, raise a separate issue for multiple varchar fields discussion.

Thank you @yruslan for the prompt reply and for accepting the feature request. Do you have any rough estimate on when it would be available?
On the other hand, we're trying to find documentation on the multiple varchar fields specification to include it in a separate issue.

This feature will be implemented in about a week.

The varchar trimming is implemented in the snapshot version. Please, try this version and let me know if it works for you:
<dependency>
      <groupId>za.co.absa.cobrix</groupId>
      <artifactId>spark-cobol</artifactId>
      <version>1.0.2-SNAPSHOT</version>
</dependency>

Thank you @yruslan !
We're testing it and from the functional point of view it looks good 👍
On the other hand, performance wise it's not going as we expected, but we need to do further testing to understand what's happening. If we find something worth mentioning we'll publish it here.

Cool, thanks! Please, let me know if the snapshot version is slower than 1.0.1.
If no performance degradation is observed in 1.0.2-SNAPSHOT, we will release 1.0.2 on Monday.

Snapshot version is not slower that 1.0.1. Whatever our performance issue is, it's not caused by the snapshot version.