new scripts for NetBox

[!NOTE]
We are meticulous when it comes to merging code into the main branch, so please understand that we may reject pull requests that do not meet the project's standards. It's never personal. Also, game-related scripts have a lower chance of being merged.

Description
Adding script for NetBox as requested in discussion (#50) and (#1327) in tteck repository.
Tested on my Proxmox installation and have it running for a few days. Update script is also tested by installing older version from netbox on install (changing line 46 in install/netbox-install.sh to RELEASE="4.1.0") and after install running the update script.
Type of change
Please check the relevant option(s):

[ ] Bug fix (non-breaking change that resolves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (a fix or feature that would cause existing functionality to change unexpectedly)
[X] New script (a fully functional and thoroughly tested script or set of scripts.)

Prerequisites
The following efforts must be made for the PR to be considered. Please check when completed:

[X] Self-review performed (I have reviewed my code, ensuring it follows established patterns and conventions)
[X] Testing performed (I have tested my changes, ensuring everything works as expected)
[X] Documentation updated (I have updated any relevant documentation)

Additional Information (optional)
Installed NetBox LXC:

Running update script:

Running update script on latest version:

Running NetBox:

@havardthom @MickLesk I changed the code based on your feedback. Can you check again?


using options in example

I thought you might want to show how to pass the defined options to plato?

Seems that it may clear some confusion, @jsoverson any issues with it?


Added a test on SFH

Added a test on the SFH parameters- tau, constant, sf_start, sf_trunc, tburst, fburst.

Thanks!  I'll try and finish review and merge this early next week. cc @dweisz


[Suggestion][Question]

Platform

[x] Forge

What you want to be added?
-is ther a way to display admin commands, and if not can there be

You mean, admin commands like an help command in discord? For that just use the help command with admin role, it should show them.
Or do you mean an channel logging use of admin commands. Use the command log feature, even tho it logs all commands it it also logs use of admin commands

You have to set the channel ID of command log.channelID in your config
(Variable in code:
https://github.com/ErdbeerbaerLP/DiscordIntegration-Core/blob/f8762f79a70b95c33032fce419e0e386edf3dd40/src/main/java/de/erdbeerbaerlp/dcintegration/common/storage/Configuration.java#L508 )

How do you do this in 1.12?

You have to set the channel ID of command log.channelID in your config
(Variable in code:
https://github.com/ErdbeerbaerLP/DiscordIntegration-Core/blob/f8762f79a70b95c33032fce419e0e386edf3dd40/src/main/java/de/erdbeerbaerlp/dcintegration/common/storage/Configuration.java#L508 )

How do you do this in 1.12?

You can't yet

Latest 1.12 update should allow it now


Investigate feasibility of creating a Quarkus extension for DataNucleus

Current Behavior
Having to deal with multiple tech stacks (Alpine for API server, Quarkus for all other services) poses some challenges. It would be great if we could eventually converge them all to at least use the same framework.
Having all of the services use the same tech stack could potentially enable us to structure the project as modular monolith, such that users can decide if they prefer to deploy a single application, or multiple.
The API server largely uses standard Java EE concepts and as such would be relatively easy to port to Quarkus. The only outlier being the persistence framework, DataNucleus. There is no Quarkus extension for it (https://github.com/quarkusio/quarkus/issues/1908). However, it might be feasible to create one.
Proposed Behavior
Investigate how feasible it is to create a Quarkus extension for DataNucleus: https://quarkus.io/guides/writing-extensions
Checklist

[X] I have read and understand the contributing guidelines
[X] I have checked the existing issues for whether this enhancement was already requested

An alternative approach would be to migrate the persistence code to Hibernate instead. Looking at how much code is involved in the hibernate-orm extension, I'm not sure doing the same for DataNucleus is something we can realistically do and maintain on our own.


Inetum -  Inetum Careers button does not open company website

ENVIROMENT: production
URL: https://scrapers.peviitor.ro/src/inetum/index.html
Browser: Chrome
Device: laptop
OS: Windows 10
STEPS TO REPRODUCE:

Open URL in browser
Click on the 'Inetum Careers' button
Check the new tab opened by the' Inetum Careers" button

EXPECTED RESULTS:
Inetum company  website opens in a new tab
ACTUAL RESULTS:
URL: https://scrapers.peviitor.ro/src/inetum/index.html# opens in the new tab

it is normal to open in a new tab


npx webpack --watch

npx webpack --watch
webpack is watching the files…
⚠ ｢copy-webpack-plugin｣: unable to locate 'src/popup.html' at '/home/xky4k/Desktop/WhatsAllApp-Mod/src/popup.html'
Insufficient number of arguments or no entry found.
Alternatively, run 'webpack(-cli) --help' for usage info.
Hash: 15bd95ead4fa0099a071
Version: webpack 4.41.2
Time: 723ms
Built at: 01/22/2020 6:49:07 PM
Asset       Size      Chunks                    Chunk Names
background.js   4.33 KiB  background  [emitted]         background
bundle.js   1.09 MiB      bundle  [emitted]  [big]  bundle
images/icon_128x.png   3.69 KiB              [emitted]
images/icon_16x.png  491 bytes              [emitted]
images/icon_32x.png  881 bytes              [emitted]
images/icon_48x.png   3.34 KiB              [emitted]
injector.js   4.38 KiB    injector  [emitted]         injector
manifest.json   1.01 KiB              [emitted]
Entrypoint bundle [big] = bundle.js
Entrypoint injector = injector.js
Entrypoint background = background.js
[./src/background.js] 509 bytes {background} [built]
[./src/index.js] 1.57 KiB {bundle} [built]
[./src/injector.js] 567 bytes {injector} [built]
[./src/pack/AppContainer.js] 2.49 KiB {bundle} [built]
+ 11 hidden modules
WARNING in unable to locate 'src/popup.html' at '/home/xky4k/Desktop/WhatsAllApp-Mod/src/popup.html'
ERROR in Entry module not found: Error: Can't resolve './src/popup.js' in '/home/xky4k/Desktop/WhatsAllApp-Mod'
ERROR in ./src/pack/AppContainer.js
Module not found: Error: Can't resolve './Components/OverlayContainer' in '/home/xky4k/Desktop/WhatsAllApp-Mod/src/pack'
@ ./src/pack/AppContainer.js 13:24-64
@ ./src/index.js
ERROR in ./src/index.js
Module not found: Error: Can't resolve './pack/utils/Log' in '/home/xky4k/Desktop/WhatsAllApp-Mod/src'
@ ./src/index.js 3:11-38
ERROR in ./src/index.js
Module not found: Error: Can't resolve './pack/utils/WWapApi' in '/home/xky4k/Desktop/WhatsAllApp-Mod/src'
@ ./src/index.js 19:15-46

I did not see anywhere orange sidebar Already reloaded RIZWAN AHMAD

Hi,
also at my place.
But when you switch to 'disable' (site loads again), and then again to 'enable' (site loads again).. then it shows the orange bar. It have to write first time of use the settings.

@rizwansoaib now you solved this?


MEPC written on mret instructions

Hi all,
I noticed that on an mret instruction, mepc will get overwritten with the contents of mstack_epc as shown below:

This is causing some test failures, as this update is not reflected in the logs, resulting in a register write mismatch since the Ibex mepc now holds 0, while the mepc in the ISS will still contain the correct value that was explicitly written at an earlier point in the simulation (0x80000144).
mepc will still get written correctly whenever the core takes a trap, it is just this extra write that occurs on mret that modifies its state. According to the spec, mepc should never be written by the RTL apart from taking traps, so how does this align with the nonstandard mstack CSRs that Ibex implements?
Best,
Udi

I think the issue is the csr_restore_mret_i signal. It's asserted on any mret but should only be asserted when we're in nmi mode (as per https://github.com/riscv/riscv-isa-manual/issues/261):
https://github.com/lowRISC/ibex/blob/bb7acbdb7a1c1240cf93787396ea1f4812ca3d5e/rtl/ibex_cs_registers.sv#L547-L557
https://github.com/lowRISC/ibex/blob/bb7acbdb7a1c1240cf93787396ea1f4812ca3d5e/rtl/ibex_controller.sv#L549-L556
@vogelpi as you implemented this initially do you agree? I'd be happy to do the PR to fix.

Thanks @udinator for opening the issue and @GregAC for your suggestion. This makes sense.
But I think csr_restore_mret_i should not be changed as priv_lvl_d, mstatus need to be changed independently of nmi_mode_q. We should probably feed the nmi_mode_q to ibex_cs_registers.sv as we do for debug_mode_q and then only use this to decide what should happen with mepc_d. WDYT?

You're right can't just alter the behavior of the csr_restore_mret_id_o signal. I'll do a PR with what you suggest, feeding nmi_mode_q into ibex_cs_registers and ensure the mret behaviour is altered appropriately.


Refactor Run intro section for Qiskit Patterns

Closes #924

Is this change safe to go live to production now, or does it need to wait until the bigger change where we land all of Patterns? I'll be talking in our Monday meeting about the strategy for version control: https://github.com/Qiskit/documentation/issues/996
Tl;dr from content writing perspective:

Don't add any changes to existing folders like start/ that you are not comfortable going live to production
Don't add new folders to open-source-sync-config.json
It is safe to add new folders like analyze

I'm assuming this would wait till we have everything in place -- but I will defer to @javabster on that!  Can I merely leave this change unmerged, or should I hide it some other way till we're ready to publish? (I can wait till Monday to hear the gritty details 😆 )


Update README.md to include app_name and color requirements

This adds some information that has been discussed in #696, #815 and others.

I just want to say I love seeing a PR where I am neither the author nor the reviewer!  I'll merge this once you two have worked through the comments.

Hey, do we still require the Fabric Maven repo in the build.gradle for FirebaseUI Auth?

@TheCraftKid we only require it if you're using Twitter auth, since you need it to resolve the twitter dependency.


Getting correct output axis name when input name in parentheses

Reported by Ansley B Manke on 28 Nov 2007 21:14 UTC
We had a dataset created as a subset of another dataset, for instance:
use ocean_atlas_monthly
set axis/stride=5 temp,return=xaxis
set axis/stride=5 temp,return=yaxis
save/file=ocean_subset.nc/clobber temp[the LAS script get_datavar.jnl is going to write a subset of that data
(for reference the call is this, but we can see the bug with a simple command, below
GO get_datavar "out.nc" "osubset.nc" temp IJKL 1 10 1 1 10 1 1 1 1 1 1 1
)
In a separate Ferret session from the one that wrote the subset initially,
yes? use ocean_subset.nc
yes? LIST/FORMAT=CDF/FILE="out.nc"/CLOBBER temp[i=1:10:1,j=1:10:1,k=1:1:1,l=1:1:1](k=1:2,L=1:2]
Then)
LISTing to file out.nc
**TMAP ERR: attempt to redefine line
axis AX0 doesnt match CDF file
What's happening is that the routine cd_axis_outname.F is going to remove the
parentheses on the axis names from the input file. It applies this removal to
the wrong string variable, cutting off the ends of the axis names which should
be AX008 and AX007 but are both being returned with the shortened name AX0.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1556

Comment by<EMAIL_ADDRESS>on 2 Jan 2008 19:47 UTC
Fixed in cd_axis_outname.F


Basic warm reset test

HW-model support for warm reset.
Set ready_for_runtime in RT FW to check if boot is complete without parsing log.

@korran I am open to ideas to make warm_reset_flow and warm_reset more clear.

I think there was a merge issue. The commit list contains a lot of commits that I think are already in main.


Analytics opt-out by default or ask rather than auto opt-in

We know that analytic helps to make homebrew better, I get it. However, I don not recommend auto opt-in. Your end users should have a choice. I was very sad to see a message presented to me today stating that I was auto volunteered to send this.
==> Homebrew has enabled anonymous aggregate user behaviour analytics.
Read the analytics documentation (and how to opt-out) here:
https://docs.brew.sh/Analytics
Yes its anonymous and yes I am Uber paranoid regardless. especially in the time of privacy related concerns, I thinks it very important to ensure that a great tool such as homebrew follows strict guidelines. Therefore, I advise against auto opt-in but rather keeping it as a choice for the end user.

Thanks for your input but, as previously discussed, we will be leaving this as-is.


Change rpc-release to r11.1.0rc1

This PR creates the r11.1.0rc1 release candidate for the kilo branch.

:+1:

:hand:
:microphone:

closing in favor of the #459 rebase


don't test MCCP2 example code

#6 seems to fail in the tests so:
Added ignore to MCCP2 example code, as it shouldn't be tested without the feature on.
(It is possible to have the same doc twice, and only ignore without the feature on, I don't think it is worth it)

I just published the new version (0.1.4). You can now see this version on crate.io.
It is my fault that I didn't check the testing result before merging the previous request. Thanks for all your effort.


with HTML minify settings enabled, debug info disappear

without HTML minify settings enabled in the html code i see
<!-- Performance optimized by W3 Total Cache. Learn more: https://www.w3-edge.com/products/

Page Caching using disk: enhanced (Requested URI contains query)
Minified using disk

 Served from: sndsgn.altervista.org @ 2017-02-12 02:23:24 by W3 Total Cache -->

with HTML minify settings enabled those debug info disappear.

if i have a good memory we have fixed it in the 0.9.4.x but i don't found the PR....

Yea we got rid of it in v0.9.4.x somewhere.  In 0.9.5.x if i recall frederick shows this intentionally at all times, unless one activates Edge mode.
So one is forced to deactivate it via adding this to a themes functions.php file:
add_filter('w3tc_can_print_comment',function($w3tc_setting){ return false; },10,1);

But i want it :D
My point is: when enable html minify this info disappear
This info is useful for me for understand the state of the cache

Oh sorry..i misunderstood.  I believe you can keep it via adding: Performance optimized under the Ignored comment stems field.

Yep it works!  Always thanks!

Maybe in the Frederick logic this comment stems should became a default value
; )


Dev mromano 0.5.0

Current coverage is 91.45% (diff: 100%)

Merging #32 into development will not change coverage

@@           development        #32   diff @@
=============================================
  Files                1          1
  Lines              117        117
  Methods             17         17
  Messages             0          0
  Branches            13         13
=============================================
  Hits               107        107
  Misses              10         10
  Partials             0          0

Powered by Codecov. Last update f5d625c...aba5b9b


Add support for other platforms beyond wordpress

I am investigating the possibility of other platforms like Drupal, Laravel etc. They each have similar command line managers and I wanted to check if there was a community need or others who would like it.
I am fond of your json system for storing site information and would like to add this through the generator you have. I don't think it would be far from what you have, and could work alongside existing configurations with a "drupal" or "laravel" key in the json.

VVV in general tends to be WordPress centric, which has been our use case for it. I'm not sure how successful it would be in running other project types, since this is making direct use of the auto-site-setup mechanism in VVV itself to trigger most of the behavior. This is not to say the pieces couldn't be used successfully for other project type, however for now I don't see those use cases being particularly relevant for our company.
If there is enough demand here, I would be happy explore it further.

Completely understand. I have already done a bunch of the work to get them working. I am thinking a custom dashboard may be needed. There is at least one other similar generator to yours which uses json also.
If you want to see how a Drupal site folder would look I have that repo public already https://github.com/EHLOVader/vvv-drupal it installs drush and a stable copy of Drupal for dev.
Basic PHP sites or even ones that use Composer like Laravel also work well with the auto-site-setup.


Add BCDD::Result::Transitions

Small orthographic fix: the correct is: different from zero instead of different of zero

Hi @kupkovski, thank you for your comment. But I didn't find this message/string in the PR 😅. Maybe I changed or removed it! 🤷♂


addChapters missing from component.

  function addChaptersToPlayer(playerEl) {
    playerEl.addChapters(chapters);
  }

<Video
       onLoadedMetadata={({ target }) => {
      addChaptersToPlayer(target);
      }}
      onChapterChange={({ target }) => {
        setActiveChapter((target).activeChapter);
      }}
      ...
/>

Target Player is missing addChapters which should exist on mux-player-react or am I missing something?

this will be available in the latest release, when we get that published :s
thanks for reporting!

Works perfectly now! Thanks 😄


Use extern "C" for dynamic link pointers

Tell me if I'm wrong, but I think that the pointers in the ExternFuncs needs to be extern "C", since they point to C functions.

Tell me if I'm wrong,

:shrug:
I couldn't find any definitive answer in the docs about extern and fn pointers.


Apt should install quietly

Even with apt set to quiet (-qq), there's still stdout being written by dpkg.
Try this command:
apt-get install -qq -o=Dpkg::Use-Pty=0 <packages>

Or maybe this variant:
apt-get -qq install -o=Dpkg::Use-Pty=0 <packages>

Gotta use -o and redirect stdout
sudo apt-get -qq install -y -o=Dpkg::Use-Pty=0 git > /dev/null

Closed


Add confirmation of up to 500 document limits

batch has write restrictions. I added this code because this mock passes the test without the batch limitation.
https://firebase.google.com/docs/firestore/manage-data/transactions

Actually in order to add the tests to this pull request, you should add the unit tests to the same branch (output-1), not a new branch.

Here's how you can bring the commit into output-1, and update this pull request:
# move to branch output-1 locally
git checkout output-1

# merge the commit that contains your unit tests into output-1
git pull origin master

# update output-1 on GitHub
git push origin HEAD

Thank you for waiting.
and Thank you for teaching me how to use it.
I think I was able to add the file, can you confirm it?
※exceptions are notified in my null safety environment.
I changed my testing expectations.
Exception occurred → Document count.

And to test that your unit tests work, you can run flutter test.

Thank you for your polite advice.
I'm sorry for the trouble.
For me as a non-engineer, your advice was very informative.
I passed the modified test.

Thank you for the confirmation. sorry for making you wait.
I have committed the modified code.
I looked at the test results again from the link and it didn't seem to be fixed. I'm afraid I was able to complete the fix.

I just published your code at https://pub.dev/packages/fake_cloud_firestore/changelog#113. Thanks again!

Thank you for your polite support.
I am happy to be able to contribute.


Improved seo docs page

Description

Added Other Pages Meta Tags & Structured data and Schema markup.

Contributor Checklist

Make sure to do the following steps if they are applicable to your PR:

[x] Update e2e tests: If you changed the /template/app, then make sure to do any neccessary updates to /template/e2e-tests also.
[x] Update demo app: If you changed the /template/app, then make sure to do any neccessary updates to /opensaas-sh/app_diff also. Check /opensaas-sh/README.md for details.
[x] Update docs: If needed, update the /opensaas-sh/blog/src/content/docs.

@Martinsos I finished updating the document with your suggested changes. If you like, check it one last time.
I've also created a robots.tsx default for the template and fixed the missing alts on the images to make the Landing Page 100% Seo Score.

Seo Guide

Removed the extra <>.
Removed redundant HelmetProvider.
Shortened the section on structured data.

OpenSaaS template

Removed /api/ &  /auth/ from robots.txt

@Martinsos I don't think I've missed anything now, right?


KAFKA-6354 Update KStream JavaDoc using new State Store API

As a newbie to Kafka, I make this docs update PR and in the meanwhile to get familiar with kafka and apis :)

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-test-coverage/224/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-test-coverage/256/

Tests passed locally. Merged to trunk and cherry-picked to 1.1.
Thanks for the patch @PnPie !


BibMerge:  added CFG_SITE_RECORD as script data

Signed-off-by: Øystein Blixhavn<EMAIL_ADDRESS>

See #2508 and PR #2237 merged as 44ba77ec589c995026f19707f8925e2eba36844a.


script does not terminate - how to properly shut down?

Hi, I'm using this client in a node.js script and my script never terminates because the refresh-scheduler has an open handle. (the setTimeout in @dfuse/client/dist/lib/client/refresh-scheduler.js:27)
Is there a proper way to tell the client to cleanup / disconnect / release resources?

For now this hack does the trick:
dfuseClient.apiTokenManager.refreshScheduler.clearRefreshTimeout()

Sure thing, definitely something we can fix up easily. I'll implement something shortly and check if there is not other handle leaks at the same time.

0.2.6 has been released with the fix. You can now call client.release() which should clear out all hold resources.


The series-custom.renderItem.return_rect.info look like can't work!

Version
5.2.1
Steps to reproduce
const myChart = echarts.init(document.getElementById('main'));
        const option = {
            xAxis: {
                type: 'value',
            },
            yAxis: {
                type: 'value',
            },
            series: [
                {
                    type: 'custom',
                    data: [[1, 2]],
                    renderItem(params, api) {
                        return {
                            type: 'rect',
                            shape: {
                                x: 100,
                                y: 100,
                                width: 100,
                                height: 100,
                            },
                            info: { a: '23' },
                            style: {
                                fill: 'red',
                            },
                        }
                    }
                }
            ]
        }
        myChart.setOption(option);
        myChart.on('click', params => {
            console.log(params.info);//When i click the rect,i get undefined.
        })

What is expected?
series-custom.renderItem.return_rect.info;
console:{ a: '23' }
What is actually happening?
params.info is undefined;
The same as others.(group,circle and so on.)

It also is undefined in tooltip.formatter.

Seems to be a bug.

Seems to be a bug.

Unfortunately it seems this is still not working? Is there another way of passing data to events?

Unfortunately it is still not work at version 5.3.3

codePen is here .

There will still be this problem in 5.4, when can it be fixed


Drop python 2 support

Instead of doing this in the v1.* version, why not deprecate python in the v2 branch? Then it's a clean break opportunity.

@alex4200 do we still use python2 in the BSP use cases ?
Usually we install latest package so this would break our use cases if it is still the case.

There are still notebooks using python2, but most use cases should be using python3. I am trying to remove all python2 usecases by the time of the review.

Usually we install latest package so this would break our use cases if it is still the case.
It should not. The existing NeuroM version with python 2 support will still be available in Pypi. It will be pick up when someone tries to install NeuroM with python 2.

Instead of doing this in the v1.* version, why not deprecate python in the v2 branch? Then it's a clean break opportunity.

Yes. That's a good point. I'll try to build the Windows MorphIO wheels today or next weeks so that we can finally go forward on this.

@mgeplf It's ready to be reviewed

They say we only live once...


Gatling Jenkins Pipeline - Users executes 1 iteration of request during whole test duration

We are running a Gatling test using maven in Jenkins pipeline for the test duration of 5 minutes with 5 users with 1 request and pacing of 1 second.
The issue which we are facing is, each user fires single request and does not do anything. i.e Totally we could see only 5 requests were fired. Ideally it should execute as many times within the test duration of 5 minutes.
This happens only if we run this in the Jenkins pipeline with Jenkins agent container (with nodejs 10).
If the test is executed locally outside of Jenkins, the execution happens as expected.
Below is the maven command we use:
mvn clean gatling:execute -Dtestduration=5 -Dtestpack=Contentservicev1.json
Gatling version: 2.0.0-RC1
Scala version: 2.10.4

This gatling version wasn't even a stable release (RC = Release Candidate) and generally speaking, Gatling 2 has reached end of life last year. Please upgrade to latest Gatling 3 (3.2.0 is about to be released).


remove gitter

Let's consolidate the chat to this issue: https://github.com/a-synchronous/rubico/issues/29

Codecov Report
Patch and project coverage have no change.

Comparison is base (3130969) 99.60% compared to head (5c00b76) 99.60%.

:exclamation: Current head 5c00b76 differs from pull request most recent head e684f19. Consider uploading reports for the commit e684f19 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #266   +/-   ##
=======================================
  Coverage   99.60%   99.60%
=======================================
  Files         226      226
  Lines        3323     3323
=======================================
  Hits         3310     3310
  Misses         13       13

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


python3Packages.xknx: 0.18.12 -> 0.18.13

Motivation for this change
Update to latest upstream release 0.18.13
Change log: https://github.com/XKNX/xknx/releases/tag/0.18.13
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 145916 run on x86_64-linux 1

3 packages built:

home-assistant
python38Packages.xknx
python39Packages.xknx


feat: support for mypy by adding py.typed file

Before rebuilding your library locally with this change I was getting the following error in my project:
project/configurator.py:23: error: Class cannot subclass "BaseYamlSettings" (has type "Any")  [misc]
I have verified that all you tests still pass and that the error I was getting is resolved.
This feature is very similar to the fix/py-typed branch you have not yet merged. The difference is that I added a classifier to indicate it is Typed, and I did not add:
[tool.setuptools.package-data]
"yaml_settings_pydantic" = ["py.typed"]
I have published several typed packages successfully with that setting.

Hey @qs5779,
I thought this had been fixed, my bad. Thank you for your contribution! It has been merged.


Files won't upload (apart from Excel)

Uploading files allows me to select a file, but no data uploaded. Tried for CSV and JSON, and drag-and-drop has no effect.
Excel files, strangely, do upload for me.
Attempted in both Chrome and IE, same results.

I'm actually having the same issue. Trying to upload just a .CSV and according to the console it says:
code 404, message File Not Found

Mozzilla Firefox 53.0.2 (32 bit) :
Error: file is null
$scope.uploadFile@http://app.rawgraphs.io/js/controllers.js:58:1
@http://app.rawgraphs.io/js/controllers.js:255:7
$digest@http://app.rawgraphs.io/bower_components/angular/angular.min.js:145:373
$apply@http://app.rawgraphs.io/bower_components/angular/angular.min.js:148:339
e/q<@http://app.rawgraphs.io/bower_components/angular/angular.min.js:161:273
f@http://app.rawgraphs.io/bower_components/angular/angular.min.js:47:146
lg/k.defer/c<@http://app.rawgraphs.io/bower_components/angular/angular.min.js:50:68

Microsoft Edge 38.14393.1066.0 / Microsoft EdgeHTML 14.14393 :
TypeError: Unable to get property 'size' of undefined or null reference
at $scope.uploadFile (http://app.rawgraphs.io/js/controllers.js:58:7)
at Anonymous function (http://app.rawgraphs.io/js/controllers.js:255:7)
at m.prototype.$digest (http://app.rawgraphs.io/bower_components/angular/angular.min.js:145:373)
at m.prototype.$apply (http://app.rawgraphs.io/bower_components/angular/angular.min.js:148:339)
at Anonymous function (http://app.rawgraphs.io/bower_components/angular/angular.min.js:161:270)
at f (http://app.rawgraphs.io/bower_components/angular/angular.min.js:47:146)
at Anonymous function (http://app.rawgraphs.io/bower_components/angular/angular.min.js:50:68)

Google Chrome (Versione 58.0.3029.110 (64-bit):
TypeError: Cannot read property 'size' of null
    at m.$scope.uploadFile (controllers.js:58)
    at controllers.js:255
    at m.$digest (angular.js:17855)
    at m.$apply (angular.js:18125)
    at angular.js:19972
    at f (angular.js:6131)
    at angular.js:6411

It's getting this error on the size check:
https://github.com/densitydesign/raw/blob/master/js/controllers.js#L58
This means something is happening between when angular is getting the file from the user and passing it to this function. I don't know anything about Angular so we're now out of the scope of what I can work on.

I just tried uploading a file to the live site and it works; going to load up a local server and see if the same file works there.

It works for this file now so maybe it's a file format thing?
@BobHarper1 can you upload the file you were trying to test?
crime_homicide_subset.zip

Sorry for the lateness of my reply. Afraid that I'm still getting the same error with that datatset (but it works fine copy and paste).
Same message here
Chrome Version 58.0.3029.110 (64-bit)
TypeError: Cannot read property 'size' of null                    (angular.js:14362)
    at m.$scope.uploadFile (controllers.js:58)
    at controllers.js:255
    at m.$digest (angular.js:17855)
    at m.$apply (angular.js:18125)
    at angular.js:19972
    at f (angular.js:6131)
    at angular.js:6411

@BobHarper1 are you trying a local server or are you using the actual website the designers have up right now?

Hi same problem on app.rawgraphs.io csv, tsv doesn't work only excel
cheers

I think the problem lies with the file type validation.
When I attempt to upload .csv files, file.type as  read from the console is
type: "application/vnd.ms-excel", which is the standard MIME type for Excel files
Probably because I have excel configured as my default for opening csv files??? (like many others I presume)
There is also a problem for .json files which I attempt upload (file.type is simply "", so it also fails to parse).
I think the most simple fix to this that will be an improvement for the majority of users is to change the file load to search for .csv, .txt etc. in the file.name, as in the PR

@BobHarper1 please see if the changes i made are correct

Similar problem in Firefox 62.0b7 where a .csv file won't be uploaded by any method (no drag nor manual upload).

This problem still persists today.

no more an issue on v2


The targetVersion of CommandFactory.create(Message, int) is not documented properly

Currently it is documented as follows.
 @param targetVersion the ID of the entity for applying commands if {@code null}
                      the commands can be applied to any entity

When clearly it isn't what we say about it.
Also, for some reason, we check this int-typed parameter for not being null.
In scope of this issue we need to fix the documentation and address the nullability check.

@dmdashenkov please take a look, as you are the latest author.


About ROI pooling

I got a problem that is 'gcc: error: /home/harlan/WorkSpace/roi_pooling/src/roi_pooling.cu.o: No such file or directory' when I compile the roi_pooling package of you project ,and I cannot find the file named roi_pooling.cu.o.  I think I need some help

@harlanhong it seems that you did not compile roi_pooling.cu successfully. check whether you had some errors during the compilation.


Move Alignment in Chess Activity

Hey @llaske , I noticed a ui issue in chess activity on server dashboard.
The moves are not aligned in a straight line.
Let me know what you think.

You're right, it could be better.

Fixed in https://github.com/llaske/sugarizer/pull/1329

Great!


Unable to create a dictionary with lambdas that contains match statements in GDScript

Godot version
4.1.3-stable, 4.2-rc1
System information
Ubuntu
Issue description
When trying to create a dictionary of lambdas that contains match statements, GDScript prints out the following error: Expected expression for match pattern. It shows the error in the editor and doesn't run the project.
Steps to reproduce
Code example that produces the error:
var x := 1
var matchDic := {
	first = func ():
		match x:
			1:
				return,
	second = func ():
		match x:
			2:
				return,
}

If anyone needs a quick fix, this solves the issue:
var x := 1
var matchDic := {
	first = func ():
		match x:
			1:
				return
		pass,
	second = func ():
		match x:
			2:
				return
		pass,
}

It works as expected with if statements:
var ifDic := {
	first = func ():
		if true:
			return,
	second = func ():
		if true:
			return,
}

Minimal reproduction project
N/A

This is my first time reading the GDScript implementation, so sorry if I'm wrong, but maybe the issue could be this line: https://github.com/godotengine/godot/blob/1ba920fada9efc8c4476ded50fe673b8db541366/modules/gdscript/gdscript_parser.cpp#L1984
It expects DEDENT at the end of every match statement, so when it is COMMA like in this case, it fails. The same error occurs when calling a method with two lambdas with match statements:
var matchFuncCall = test(
	func ():
		match x:
			1:
				return,
	func ():
		match x:
			1:
				return
)


Add support for CertPoll messages and PENDING responses

Name of feature:
Add support for the parsing of CertPoll messages, and the issuing of PENDING responses.
Pain or issue this feature alleviates:
According to the SCEP protocol spec, if a request requires manual verification the SCEP server should issue a PENDING response while this verification takes place. The client should then periodically send CertPoll messages to check on the status of its original request, until it receives either a SUCCESS or a FAILURE result.
Currently, the library does not provide means to create PENDING responses, and rejects CertPoll messages with a not-yet-implemented error. This essentially limits its use to scenarios where synchronous request verification and certificate issuing is possible, which is not always the case.
Is there documentation on how to use this feature? If so, where?
No, but I'm happy to write some up if you can point me towards the best place to do it. The usage is essentially the same as the existing SUCCESS and FAILURE responses however.
In what environments or workflows is this feature supported?
This has been tested to work with the original micromdm/scep client*, and with a macOS device issued with a SCEP profile. However, this should work in any environment that can utilise CertPoll messages and/or PENDING responses.
*(The original client (unlike macOS) doesn't actually respond to PENDING messages with a CertPoll as per the spec, instead opting to resend an identical PKCSReq.)
If you've got any comments/concerns please let me know - thanks!

Hey @l4rkin,
Thank you for opening the PR. I'll take a better look soon, but from a quick skim looks reasonable 🙂


manifest: update mcuboot

Update mcuboot to the latest revision. Amongst the list of updates: removed usage of power/reboot.h.
NOTE: It looks like tfm-mcuboot uses a pinned version, can it be upgraded?

@gmarull Please update reference to proper SHA.
buildkite errors are not relevant for this patch.

@nvlsianpu done. It looks like tfm-mcuboot uses a pinned version, can it be upgraded?

It looks like tfm-mcuboot uses a pinned version, can it be upgraded?

@gmarull No, it should be left as is. It follows the version TFM specifies.


any kind of form crash with xdebug on laravel 5.2

if you enable php's xdebug extension with laravel 5.2 the it'll generate this error (in case of any kind of form):
FatalErrorException in AliasLoader.php line 63:
Maximum function nesting level of '100' reached, aborting!

in AliasLoader.php line 63
at FatalErrorException->__construct() in HandleExceptions.php line 133
at HandleExceptions->fatalExceptionFromError() in HandleExceptions.php line 118
at HandleExceptions->handleShutdown() in HandleExceptions.php line 0
at spl_autoload_call() in Container.php line 36
at BootFormsServiceProvider->AdamWathan\BootForms\{closure}() in Container.php line 289
at Container->Illuminate\Container\{closure}() in Container.php line 735
at Container->build() in Container.php line 633
at Container->make() in Application.php line 697
at Application->make() in Container.php line 1178
at Container->offsetGet() in Container.php line 51
at BootFormsServiceProvider->AdamWathan\BootForms\{closure}() in Container.php line 289

You can do:

vagrant up
vagrant ssh

Following steps in you vagrant homestead installation.

sudo nano /etc/php5/mods-available/xdebug.ini
set xdebug.max_nesting_level=100 to xdebug.max_nesting_level=500

thanks!
anyway i'm using linux:-)

Yeah this is causing me a headache. I've tried to add xdebug.max_nesting_level=500 within php.ini (WAMP) though doesn't seem to work. On php.info it still specifies that this is still set to 100. Still none the wiser on this.


Hide tabbar on scroll, not desired result

I've built a tabbar that dissapears when scrolling down and reappears when scrolling up. The problem is that when I reach the end of the scroll, on IOS my view bounces back and my function thinks that I'm scrolling up...
Is there a way to ignore this bounce back so that when I reach the end, my tabbar still stays hidden? I know I could put bounce={false} property to disable the bounce. But I would like to keep that bounce...
Here is the function :
const onScroll = (event) => {
    const currentOffset = event.nativeEvent.contentOffset.y;

    offset < currentOffset ? setShowTabBar(false) : setShowTabBar(true);
    offset = currentOffset;
  };

Closed as invalid (not following the issue template)


 I think we're going to have to hide some previously public members, and try this again in a major version release.

Coverage decreased (-0.01%) to 86.311% when pulling 7df139fecdaf8db941131b3ec4dff656fd3d2b0c on shmax:revert-optimize into 09469dae9f05d5840703897d89e61fce0b4f9246 on webonyx:master.

Can you add link on broken test/build of graphql-laravel?

https://github.com/rebing/graphql-laravel/pull/645/checks?check_run_id=848330436

There was some errors in the test suite (shame on me 😅 ) but one real break was that, for lazy types, you're not supposed to access the public property ->type directly, instead you have to use the ->getType() method.

I'm now putting on my practical hat: it doesn't matter for graphql-laravel, because there was release yet with ^14 of graphql-php. Insofar, the revert is not strictly necessary.

Taking off my practical hat: in strict semver, this is obviously a break in a minor version. But depending on how many already did you use, graphql-php might just get away with it and document that ->getType() is generally required for ^14

🤷♀

@vladar What do you think? Is this revert necessary? Please advise.

Can we just add __get, __isset and __set fallback for type property strictly for backwards compatibility (with a deprecation notice)?

Can we just add __get, __isset and __set fallback for type property strictly for backwards compatibility (with a deprecation notice)?

You mean as an interim fix until the next major version? Well, maybe, but I opened this PR 12 days ago as an emergency fix for people in the same situation as @mfn. No doubt you were busy with other things, but no one else has come forward since then, so do you think this PR is really necessary?

Agreed. Let's move on with deprecation in #702


It does not work in a repository that contains dots in the path/name.

In a repository such as https://github.com/wraith13/wraith13.github.io, it should be recognized as wraith13/wraith13.github.io, in fact it recognizes it as wraith13/wraith13 and access failed To do.

Probably the cause of the problem is here.
https://github.com/Microsoft/vscode-github-issues-prs/blob/master/src/github-issues-prs.ts#L213


-G disables --lang

Applying the -G flag causes ag to ignore --lang flags. As a contrived example, performing ag a_string --java -G '\.rb$' results ag searching Ruby files, where it should only search for Java files that end in ".rb" (i.e., this contrived, nonsense example shouldn't produce any results).
Please correct this behavior to have -G be applied only to files matching the specified languages.

Today, I tried ag -l --go -G _test redacted: "search for Go files with _test in the name that contain the string 'redacted'".
It appeared to completely ignore the -G flag.

For what it's worth, I also stumbled into the same issue today when trying to find all Python test files with ag --python -g 'test_'. I initially assumed that this has to be a bug in Ag, but after reading the conversation I realised that in this case an OR condition is used instead of AND which I expected. In other words this seems to be a usability problem not a simple bug.


Collapse Bar Tooltip does not go away on mobile browsers

I created the project with template.

This Click to expand/collapsde sidebar tooltip goes away on desktop browsers after mouse leaves the are of icon.
But on mobile browsers, i tested with safari and edge, after expand or collapse sidebar, tooltip stays there  till i press another area in the website.
Is there a parameter to hide the tooltip, like a parameter to never show the tooltip?

@ahmetkocadogan hi, sorry for the delay response. there is a parameter on Layout named TooltipText. can you have a try set the value is string.Empty?


Add ino token

[x] I will not ping the Discord about this pull request.

Please provide the following information for your token.
Please include change to the src/lib/.json and in the PR
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo
Link to the official homepage of token:

Please include website and logo for the token


NOTICK - Add duplicate key check in list to map merge

Do we still need this PR? (it has been a while)

Do we still need this PR? (it has been a while)

It would be good to get in. @fenryka do you want our team to pick this up and fix the failing tests/rebase?


Added custom tooltip to ActionIcon

Checklist

[x] Build process is done without errors. All tests pass in the /lib directory.
[x] Self-reviewed the code before submitting.
[x] Meets accessibility standards.
[x] Added/updated documentation to /website as needed.
[x] Added/updated tests as needed.

Description
After adding the new Tooltip component, it is required to use it instead of title in the ActionIcon component.
Related to #1968

Closed until Tooltip component lifecycle status is New.


Specialize unchecked_oneto

On upcoming julia versions, Base occasionally uses unchecked_oneto where it used to previously use oneto. This requires us to specialize unchecked_oneto as well to get tests to pass.
See https://github.com/JuliaLang/julia/issues/52481

This method can't be tested on current versions, as it is only added for an upcoming version. It should be tested already on the upcoming versions, as the tests on nightly fail without this.


removed unused git workflow config. renamed existing config files to …

…make their name, so i can more easily understand it in the gui

renamed action files to be more clear


Add Yard docblocks

This PR changes documentation slightly:

remove default RDoc args - this broke builds for people on RPM-built thing #44
add some whitespace
break some long lines

It's Rick & Morty time and then sleeping. Look forward to seeing your progress tomorrow!


Copy of Rule blocked: CORERULES-254/1186/3701

see #557
https://jira.cdisc.org/browse/CORERULES-254
https://jira.cdisc.org/browse/CORERULES-1186
https://jira.cdisc.org/browse/CORERULES-3701
Describe the bug

Target_is_sorted_by needs to be validated to work with Dask, currently the operator is rooted in Pandas in its present implementation
this needs to be made more robust and able to handle partial dates, date and time comparisons for sorting

this should convert to the broadest contiguous date/time part for comparison

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

see https://github.com/cdisc-org/cdisc-rules-engine/pull/830 as this works for dask and pandas

Review on #830 is already complete. The PR has implementation for this Issue AC too.


updatecli: 0.82.0 -> 0.88.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for updatecli is: Declarative Dependency Management tool
meta.homepage for updatecli is: https://www.updatecli.io
meta.changelog for updatecli is: https://github.com/updatecli/updatecli/releases/tag/v0.88.0
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.88.0 with grep in /nix/store/d9bvxfq5m3jh9kf8bmi4bl92lfvc128m-updatecli-0.88.0
found 0.88.0 in filename of file in /nix/store/d9bvxfq5m3jh9kf8bmi4bl92lfvc128m-updatecli-0.88.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

updatecli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/d9bvxfq5m3jh9kf8bmi4bl92lfvc128m-updatecli-0.88.0 \
  --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 updatecli https://github.com/r-ryantm/nixpkgs/archive/5395596316216030152e807d9a189ede58dc2930.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/5395596316216030152e807d9a189ede58dc2930#updatecli

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/d9bvxfq5m3jh9kf8bmi4bl92lfvc128m-updatecli-0.88.0
ls -la /nix/store/d9bvxfq5m3jh9kf8bmi4bl92lfvc128m-updatecli-0.88.0/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.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

updatecli

Maintainer pings
cc @Croissong for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


DLL files

Hi,
When I installed this package with Nuget package manager in VS2013 for C# project, I couldn't find dll files added in references neither in package directory to create manual references.
Can you please help me through if I am doing something incorrect.
Thanks,
R.

Hi @rahulmaxi
Which package did you install, as you may need the full toolkit or to download other modules?

Hi @Wagnerp ,
Thanks for your response. I have installed package KryptonToolkitSuite5470 -Version 5.470.1231.
This what I have done

Could you please suggest what else I am missing or should install to get it working.
Thanks,
Vikrant.

Hi @rahulmaxi
If it compiles correctly, then you should be able to use the toolkit as normal.

The Krypton components are not shoing up in the project, but you have installed them at the solution level.
What version of the .Net framework have you set your project to ? (i.e. 5.470 requires .Net 4.70)

Hi @Smurf-IV ,
Thanks for pointing out the version issue. I created another one with 4.7 but still no luck. I am still missing .dll files for it.
I can not use KryptonForm yet.

To be honest I am confused how should I install this package (Krypton-NET-5.470) via nuget to get the dll files.
Thanks,
Vikrant.

Hi @rahulmaxi
It looks as if you haven't put in using ComponentFactory.Krypton.Toolkit; yet.

@Wagnerp ,
using ComponentFactory.Krypton.Toolkit; goes unrecognized due to missing dll files. I am expecting dll files should be added in References as soon as package is installed from nuget package manager.

Yes they should all be in the References section. Do you have a folder named packages? Also, what version of Visual Studio are you using?

I am using VS2013. I do find folder name packages with following.

Hi @rahulmaxi
It seems to be a weird VS 2013 issue, all other VS versions seem to be ok.

Hi @Wagnerp ,
Looks like but only with Krypton-NET-5.470. When I attempt to install Krypton-Toolkit-Suite-Extended-NET-5.470 via nuget it adds references automatically(though I did not use it in any form). When I installed log4net from nuget it also worked well adding references automatically.

Hi @rahulmaxi
I can upload the binaries to here if you want?

Hi @Wagnerp ,
Thanks for your response. I installed the Krypton.Toolkit.Suite.for.NET.4.7.exe and manually added the required DLLs.


Note are not saved - Github auth

Using Gitihub auth, I am not able to save the notes.

Tyring to sync all note, didn't sovle the problem and there is no error displayed.

the error is due to the body size for the request which exceed the limit. when I delete notes it start syncing.


Pretrained weights

Hello,
Is it possible for me to use some other pretrained model in place of the T5 model to generate the embeddings? Or is the code written in such a way that the pretrained model has to be the T5 model?
Thank you!

Hello,
I think this is feasible because our modeling method based on text matching is similar to dense retrieval, and of course it can be replaced with other pre-trained models.
But my model code is only for T5, and this long text modeling method is also based on such an encoder-decoder architecture. Therefore I suggest you rewrite the model based on this framework. As for other parts, I think it can be adapted without modification.


[testsuite] Re-enable TestLLVMStyle.

The recent inject-local-variables fixes made sure this was
passing again, but it hasn't been marked as such.
rdar://problem/31811802

@swift-ci test and merge


Custom heading row formatter can use column index.

1⃣ Why should it be added? What are the benefits of this change?
If the Excel header contains long multi-byte text, it was inconvenient to not be able to use the header column index to use WithValidation while skipping the header.
2⃣ Does it contain multiple, unrelated changes? Please separate the PRs out.
It's OK.
I only edited 1 file and added 4 "$key".
3⃣ Does it include tests, if possible?
Yes. I added 2 test case.
I compared the test results before and after the change.
I confirmed that the only difference was whether the test cases I added were cleared or not.
4⃣ Any drawbacks? Possible breaking changes?
Probably not.
5⃣ Thanks for contributing! 🙌
Thank you for developing a great library.
The code was clean and very easy to read. I would like to learn from you.

Great addition, thank you!

Yes. I added 2 test case

Can you commit those unit tests, please. Thanks!

@patrickbrouwers
Thanks for checking.

Can you commit those unit tests, please. Thanks!
Sorry, I forgot to push it.

@ayutaya Thanks! Can you also make an update to the docs? https://docs.laravel-excel.com/3.1/imports/heading-row.html#custom-formatter On the bottom of the page you can click on Help us improve this page! Including the key in the already existing example is fine.


JSON Hyper Schema "schema" property of "links" array renamed to "submissionSchema" in 2019-09 draft

Current version of code is using /_options/links/schema JSON pointer.
See release notes: https://json-schema.org/draft/2019-09/json-schema-hypermedia.html#rfc.appendix.B

We are currently using a previous draft so we will not support he latest one.


feat(recipes): customizable layer

📝 Description
TODO changeset + docs
💣 Is this a breaking change (Yes/No):
no

closed in favor of https://github.com/chakra-ui/panda/pull/2080


XYZ help

I'm arranging the models in my software using this
m_player = ref new Sphere(XMFLOAT3(0.0f, -1.3f, 4.0f), 0.2f);
Is the left X, the middle Y and the right Z?

Yes the order is x, y, z, but what that actually means depends on your world and view transformations. For example, if you are using left-handed viewing coordinates, then +Z is farther away in the front, while with left-handed viewing coordinates, then -Z is further away in the front.
See this blog post for more information, or really any text on 3D graphics will cover the basics of viewing coordinates.


Enable instrumentation at runtime triggers exception

Describe the bug
Reported intially in our forums:
https://discuss.elastic.co/t/unable-to-dynamically-update-instrumentation-true/232645
Reproducible with tomcat9 and same java version (11.0.5), agent version is 1.15.0.
java.lang.IllegalStateException: The Byte Buddy agent is not installed or not accessible
	at co.elastic.apm.agent.shaded.bytebuddy.agent.builder.AgentBuilder$Default.patchOnByteBuddyAgent(AgentBuilder.java:9435)
	at co.elastic.apm.agent.shaded.bytebuddy.agent.builder.AgentBuilder$Default$Delegator.patchOnByteBuddyAgent(AgentBuilder.java:11007)
	at co.elastic.apm.agent.bci.ElasticApmAgent.doReInitInstrumentation(ElasticApmAgent.java:196)
	at co.elastic.apm.agent.bci.ElasticApmAgent$3.run(ElasticApmAgent.java:183)
	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.base/java.lang.Thread.run(Thread.java:834)
Caused by: java.lang.ClassNotFoundException: co.elastic.apm.agent.shaded.bytebuddy.agent.Installer
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:581)
	at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
	at co.elastic.apm.agent.shaded.bytebuddy.agent.builder.AgentBuilder$Default.patchOnByteBuddyAgent(AgentBuilder.java:9429)
	... 8 more

Steps to reproduce

start with recording=false and instrument=false within elasticapm.properties
switch those properties to true and wait for configuration reload

Seems likely to be only related to the instrument property, as it's the one that controls bytecode modification through Byte Buddy.
Expected behavior

no exception thrown, should work as when the value is set to true on startup (which is the default).

After further investigation:

exception is also thrown when trying to change from true to false.
is not reproducible with latest agent version 1.16.0
transition from false to true with both properties works as expected with 1.16.0 (transactions properly instrumented, metrics properly sent)

Thus maybe the changes done around configuration might have fixed this by accident.
As there is little we can do to properly test this scenario, I'll just close this issue for now.


Blacklist confusing global variables

Blacklist confusing global variables set by browser env.
This solves the problem we were trying to solve in #1840 in a better way.
See the list of globals that are blacklisted here
Suggest if there are any additions/deletions to be made in the blacklist.

Instead of polluting the main config with the huge list of globals, it would be better to export it from an eslint env plugin as @satya164 suggested or export it from a different file.

Something is messed up with the branch.

Reverted last commit

Do you think we could instead import globals and then copy that object and explicitly delete a few things from it? And then use that as our list.

D you mean we should import globals from a separate file without removing any variables and then blacklist variables in the main config?

I mean something like this:
var globals = require('globals');

var customGlobals = Object.assign({}, globals.browser);
delete customGlobals.name;
delete customGlobals.status;
delete customGlobals.top;
// etc

and then using globals: customGlobals.

That looks good. I will update the code

I tweaked the list, and changed to use no-restricted-globals (turns out there's already a rule for this). Thanks!

Please help beta test the new version that includes this change!
https://github.com/facebookincubator/create-react-app/issues/2172

@doshisid if the list of globals was in a separate package, i'd love a PR to eslint-config-airbnb-base that did the same thing :-)

Why list of globals from a separate package? We can use no-restricted-globals as used here. What am I missing?

@doshisid I'm referring to the long array of strings in this PR, that I'd prefer not to duplicate.

There isn't any package that I know of that exports confusing globals. I can publish a package and then submit a PR if that's what you would like.

That would be awesome! (and we could PR use of that package into create-react-app also, so we'd be sharing the same data source)

If you don't mind you can create a new package right in this monorepo. So we can quickly update if needed.

I created the package and submitted PR. 😄
Though I have no problem in creating a package in this monorepo.
I will submit a PR.

If I submit a PR using the package I have already published and the transfer the ownership of that package, Would that work?

Yep!

see #2286


Add the "client_credentials" grant type to the default admin client

See issue https://github.com/gravitee-io/issues/issues/1281

Hi @pletessier, don't you think we should also update existing admin client ?
Your code only apply if the admin client doesn't exist.

Hi @tcompiegne ,
From my point of view, setting the grant_type at init is enough. I just need that to be able to create an access_token the first time. If user changes are made after, I suppose that Gravitee shouldn't overwrite them. It would be confusing.

All right, according to our contributing guide, can you sign the CLA, and I will accept your PR.
Contributor License Agreement

Please sign our Contributor License Agreement (CLA) and send it to<EMAIL_ADDRESS>For any code changes to be accepted, the CLA must be signed.

Thanks.
Regards.

👍


[zh] Localize blog: 2022-09-14-pod-has-network-condition.md

Localized this new blog:
content/zh-cn/blog/_posts/2022-09-14-pod-has-network-condition.md

Preview: https://deploy-preview-36621--kubernetes-io-main-staging.netlify.app/zh-cn/blog/2022/09/14/pod-has-network-condition/

/approve

/lgtm


♻(frontend) use cunningham in richi forms

Purpose
Implement cunningham forms in Richie.

The responsive is broken. Maybe take a global tour

I know it's known, but verify that the bug is gone once Cunningham is fixed


Fix interpolation with Objects and Arrays

Description
Fixed: #1910 & #1635
Update how variable interpolation works, so the behavior with objects and arrays is more as expected.

Interpolation of objects and arrays now works
Try to use the default toString-Method if available

Except for the default Object and arrays

Test collection: https://cdn.discordapp.com/attachments/693228572286124085/1221951301404065873/json-testing_2.zip?ex=661471d5&is=6601fcd5&hm=277420a68a493567de437403f133622d186642f689e761d0438306b7579497ad&
Before:

After:

Hmm... I'm a bit confused that for string interpolation, you have to put double-quotes into the string yourself ("{{my_string}}"), but for arrays and objects, you don't have to put [ ] and { } around them.
Of course, fixing this by removing the need to put " around the string would break backwards-compatibility in a really bad way, and it would break otherwise needed functionality where you just want to plop a raw string into some body. (I sometimes need to send invalid JSON as part of my tests).
The more I think about this: Trying to solve serialization for the user is really tricky. Whatever we do, it should be as simple as possible, so that the user can have a good mental-model of what is going on.

Hmm... I'm a bit confused that for string interpolation, you have to put double-quotes into the string yourself ("{{my_string}}"), but for arrays and objects, you don't have to put [ ] and { } around them.
Of course, fixing this by removing the need to put " around the string would break backwards-compatibility in a really bad way, and it would break otherwise needed functionality where you just want to plop a raw string into some body. (I sometimes need to send invalid JSON as part of my tests).
The more I think about this: Trying to solve serialization for the user is really tricky. Whatever we do, it should be as simple as possible, so that the user can have a good mental-model of what is going on.

Yeah, I also thought about this and the first implementation was like this (Nothing must be quoted). But then I saw dw-0 and other wrapping strings with double quotes.

Any update here? I think the issue is still present?


[bug] Sidecar is not signed when signing the app

It seems sidecar is not signed while packaging the app which results in being blocked in mac
Fatal error in , line 0
Check failed: reservation_.SetPermissions(unprotect_start, unprotect_size, FLAG_write_code_using_rwx ? PageAllocator::kReadWriteExecute : PageAllocator::kReadWrite).
FailureMessage Object: 0x3132e8ce0
1: 0x1023f01a2 node::PerIsolatePlatformData::RunForegroundTask(std::__1::unique_ptr<v8::Task, std::__1::default_deletev8::Task>) [/app/src-tauri/target/debug/bundle/macos/makeshift.app/Contents/MacOS/sidecar]
2: 0x103682a7b V8_Fatal(char const*, ...) [/app/src-tauri/target/debug/bundle/macos/makeshift.app/Contents/MacOS/sidecar]
3: 0x1027c75eb v8::internal::MemoryChunk::SetReadAndWritable() [//app/src-tauri/target/debug/bundle/macos/Sprint.app/Contents/MacOS/sidecar]
4: 0x1027491a1 v8::internal::Heap::UnprotectAndRegisterMemoryChunk(v8::internal::MemoryChunk*, v8::internal::UnprotectMemoryOrigin) [/app/src-tauri/target/debug/bundle/macos/makeshift.app/Contents/MacOS/sidecar]
5: 0x1027113d8 v8::internal::Factory::TryNewFixedArray(int, v8::internal::AllocationType) [/app/src-tauri/target/debug/bundle/macos/makeshift.app/Contents/MacOS/sidecar]
6: 0x102751701 v8::internal::Heap::AllocateRawWithLightRetrySlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/app/src-tauri/target/debug/bundle/macos/makeshift.app/Contents/MacOS/sidecar]
7: 0x10270f79e v8::internal::Factory::CodeBuilder::AllocateCode(bool) [/app/src-tauri/target/debug/bundle/macos/Sprint.app/Contents/MacOS/sidecar]
8: 0x10270ee02 v8::internal::Factory::CodeBuilder::BuildInternal(bool) [/app/src-tauri/target/debug/bundle/macos/Sprint.app/Contents/MacOS/sidecar]
9: 0x1025b44c8 v8::internal::GenerateBaselineCode(v8::internal::Isolate*, v8::internal::Handlev8::internal::SharedFunctionInfo) [/app/src-tauri/target/debug/bundle/macos/makeshift.app/Contents/MacOS/sidecar]
10: 0x102600ee8 v8::internal::Compiler::CompileSharedWithBaseline(v8::internal::Isolate*, v8::internal::Handlev8::internal::SharedFunctionInfo, v8::internal::Compiler::ClearExceptionFlag, v8::internal::IsCompiledScope*) [/app/src-tauri/target/debug/bundle/macos/makeshift.app/Contents/MacOS/sidecar]
11: 0x1026042b0 v8::internal::Compiler::CompileBaseline(v8::internal::Isolate*, v8::internal::Handlev8::internal::JSFunction, v8::internal::Compiler::ClearExceptionFlag, v8::internal::IsCompiledScope*) [/app/src-tauri/target/debug/bundle/macos/Sprint.app/Contents/MacOS/sidecar]
12: 0x10258ec9b v8::internal::baseline::BaselineBatchCompiler::CompileBatch(v8::internal::Handlev8::internal::JSFunction) [//app/src-tauri/target/debug/bundle/macos/Sprint.app/Contents/MacOS/sidecar]
13: 0x10258ea50 v8::internal::baseline::BaselineBatchCompiler::EnqueueFunction(v8::internal::Handlev8::internal::JSFunction) [//app/src-tauri/target/debug/bundle/macos/Sprint.app/Contents/MacOS/sidecar]
14: 0x102b50780 v8::internal::Runtime_BytecodeBudgetInterruptWithStackCheckFromBytecode(int, unsigned long*, v8::internal::Isolate*) [/app/src-tauri/target/debug/bundle/macos/makeshift.app/Contents/MacOS/sidecar]
15: 0x102b5099c v8::internal::Runtime_BytecodeBudgetInterruptFromBytecode(int, unsigned long*, v8::internal::Isolate*) [/app/src-tauri/target/debug/bundle/macos/makeshift.app/Contents/MacOS/sidecar]
16: 0x1020fe359  [/app/src-tauri/target/debug/bundle/macos/makeshift.app/Contents/MacOS/sidecar]
zsh: trace trap
[✔] Environment
- OS: Mac OS 13.4.1 X64
✔ Xcode Command Line Tools: installed
✔ rustc: 1.67.1 (d5a82bbd2 2023-02-07)
✔ Cargo: 1.67.1 (8ecd4f20a 2023-01-10)
✔ rustup: 1.25.2 (fae52a197 2023-02-01)
✔ Rust toolchain: stable-x86_64-apple-darwin (default)
- node: 16.15.1
- yarn: 1.22.19
- npm: 8.11.0
[-] Packages
- tauri [RUST]: 1.2.5
- tauri-build [RUST]: 1.2.1
- wry [RUST]: 0.23.4
- tao [RUST]: 0.15.8
- @tauri-apps/api [NPM]: 1.4.0
- @tauri-apps/cli [NPM]: 1.4.0
[-] App
- build-type: bundle
- CSP: unset
- distDir: ../dist
- devPath: ../dist
- framework: Vue.js
- bundler: Rollup

Duplicate of https://github.com/tauri-apps/tauri/issues/6888 - thanks for the report :)


Don't change the value for altBoundary option

Closes #33463 and
Closes #33579
Since the bootstrap is not changing the default value of the elementContext option, changing the value of the altBoundary option is not needed for any modifier in real. By default, the value for elementContext is 'popper' which means the boundary context will be checked for the popper element by default. Setting the altBoundary to true in the flip modifier will change this behavior and then boundary context will be checked for reference element (means button element).
Doing so will require setting the value for the boundary option manually. That is not needed in real (at least for the tooltips when these are appended to the body).
See the discussion here: https://github.com/twbs/bootstrap/issues/33463#issuecomment-822352640
/CC @XhmikosR to add this in the next release if seems good.

I had a feeling this would be the reason but I wasn't sure still. 😅 I got a bit confused by what all those options really do and how they influence each other and what if you change it on one module but not on the other.
Thanks for explaining this in more detail @rohit2sharma95 :+1:.


Pre-processing and training data

Hello!
Thank you so much for the great work you've shared, would it possible for you to share the pre-processing methodology and the training data you worked with as a demo.
It would be great help.
Thank you!

Hi,
I'm just leaving this repo up as a very simple snippet for the models alone.
If you'd like more info on the preprocessing and the whole pipeline, check out https://github.com/cvqluu/dropclass_speaker which has a bit more information on how to run this completely, and it includes information on data processing.
Specifically you would make a config file with the model = FTDNN set.
If you have any further questions there feel free to bring them up on that repo.
Hope that helps,
Chau


fix: remove connection-id from metrics attributes

The OpenTelemetry Attributes for metrics included a unique identifier for each connection. This can potentially create a very large number of time series, as each connection will be a time serie. Applications that continously create and drop connections will then produce a very large number of time series, which again can result in RESOURCE_EXHAUSTED error being returned from the monitoring backend.
Fixes #2377

Is this used in the backend metrics in any way?

No (unless I misunderstand what you mean). This is for metrics that are collected using OpenTelemetry in PGAdapter and that users can choose to export to any metrics collector that has an OpenTelemetry integration (e.g. Google Cloud Monitoring, but also many other monitoring systems).

Do we need to update any documentation as well?

Do we need to update any documentation as well?

Not specifically for this change. Our OpenTelemetry documentation does not include any specifics on these metrics. But we should add a section on metrics to that documentation page. I've added #2392 for that.


Remove mistral related specs, composers, tests, and fixtures

Remove mistral related specs, composers, tests, and fixtures because they were used as references to support multiple specs or types of workflow definition in orquesta. Mistral is deprecated in StackStorm and there's no intention to support it here. A mistral to orquesta workflow conversion tool is available at https://github.com/StackStorm/orquestaconvert.

@nmaludy Here's the PR to migrate the mistral models to orquestaconvert: https://github.com/StackStorm/orquestaconvert/pull/31


[fix](partial update) Fix NPE when the query statement of an update statement is a point query in OriginPlanner

Proposed changes
close #26882
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

run buildall

run buildall


Remote Citrix windows

Description of the new feature / enhancement
Enable FancyZones to work with remote Citrix windows and applications
Scenario when this would be used?
When working with multiple remote Citrix windows and applications
Supporting information
DisplayFusion works with remote Citrix windows and apps so should be technically possible. However, I don't like DisplayFusion's interface as I find it too busy for my simple needs hence this feature will be very welcome here.

Hey all, is there an update on my request?

Hey all, is there an update on my request?

Cc: @SeraphimaZ

@abdulhaquemasum can you please specify a little bit more? As far as I know, Citrix provides many products. Which of them do you mean?

@SeraphimaZ  Sure, please review the screenshots attached.
FancyZones Citrix Window Issue.docx

Thank you for clarification.
I'm not sure if that it would be possible. I tested similar thing, tried seamless mode on the virtual machine. PowerToys is required to be installed on the virtual machine where the window was originally created to make it possible to zone and track the window by FancyZones.

Thanks for testing. I'm not sure even how that will be possible then, as it'll be like having FancyZones installed on PC A and PC B, PC B being the remote system. Each will have a different instance of FancyZones running, so a window part of PC B displaying on PC A will not work? I have since tested running the Citrix apps as a remote desktop window, so it has all the apps in the window, this remote desktop window does snap to the zones, but it's not what I'm looking for. I have shared the same screenshots with DisplayFusion, I'll see what they say.

Snapping the citrix windows does work with windows native snapping functionality, so somewhere in the corporate empire (MS), someone's found the solution. :)

does this issue still happen with v0.73.0? /needinfo

Still an issue in 73

Issue still present on
Edition	Windows 10 Pro
Version	22H2
Installed on	12/09/2020
OS build	19045.3448
Experience	Windows Feature Experience Pack 1000.19044.1000.0
Screen recording of issue can be seen here. Running setup of seamless remote desktop app as explained in above post.

Are you still experiencing this issue? /needinfo

Yes still same issue


Build.py now creates ../bin folder by its own

The build script should definitely create the ../bin/ directory if it doesn't exist, that's a good point.
However, the script in its current state would delete all of the directory's existing contents before each build. This should not be the default action, as it causes potential loss of data. Additionally, if the process fails midway, the output directory would be left empty, with no files from the last successful build.
You should change the behavior to only check if the directory exists and create it if necessary; otherwise just let it be if it's already there. I'll be happy to merge this PR once this is done.

Teorically the second version of the script delete ../bin only if it exists and it's not a directory (that means it's a file).
I'll check again everything on my server, give me some minutes and I'll be there with the results :D

I can confirm: everything is work as intended and it don't delete them whole ../bin folder each build:
no bin folder at all -> creates bin folder and then compile
bin is a folder -> directly compile without data loss
bin is a file -> removes bin file, creates bin folder and then compile
Anyway, wait that I add a cool printf when it removes the bin file pls


Added new third-party tools - Airbyte

Overview Section: Added Airbyte as a new third-party tool in the documentation under the Data Ingestion and Streaming section. The overview includes a description of how Airbyte can be leveraged with QuestDB for seamless data ingestion.
User Guide: Detailed step-by-step guide on configuring and using Airbyte to transfer data into QuestDB. The guide includes code snippets, best practices, and troubleshooting tips for smooth integration.
Fixes #59

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@goodroot I request you to check this and merge it

@goodroot Can you review and if all's ok then merge this please.


Update go.mod to remove vulnerabilities

Apply the updates suggested by govulncheck:

go 1.22.7
<EMAIL_ADDRESS>
Also run go mod tidy.

I see tests are failing with this:
go: go.mod requires go >= 1.22.7 (running go 1.21.13; GOTOOLCHAIN=local)

We are referencing a golang builder image in Dockerfile and Makefile that is still 1.21
You probably need to updated it to rhel-9-golang-1.22-openshift-4.17 instead

We also need to backport this to release-1.8 after merge.

We also need to backport this to release-1.8 after merge.

We'll have several items to backport. I'll regroup that in a single PR when everything is merged into devel.

https://github.com/openshift/release/pull/59339 got merged at last !
/retest
We're now expecting ci/prow/check to pass and ci/prow/sandboxed-containers-operator-e2e to fail (see https://github.com/openshift/release/pull/59339#issuecomment-2507814251 for details).

Fixes: https://issues.redhat.com/browse/OCPBUGS-36878
Fixes: https://issues.redhat.com/browse/OCPBUGS-41472
Fixes: https://issues.redhat.com/browse/OCPBUGS-41410


link to the meetings calendar?

I'm not sure if I didn't just miss it, but it looks like we don't link to the meetings calendar in the meetings page.
Should we change that?

I think this was just an oversight. I can open a PR to add it, or could review someone else's


Authentification

This PR is mainly for setting up the self-authentification from the user side. Basically, there is a ENV variable called "ALLOW_SIGNUP" in the docker-compose.yml file. It's a boolean:

True : users can create their account on the plateform
False : only the admin can create accounts for the users

The second feature is for resetting the password

@Hironsan Sorry to bother you, but I don't understand how to fix this codacy issue
Unexpected keyword argument 'backend' in function call from the file app/authentification/utils.py
If I remove the backend argument (as suggested by codacy), then I have a python error, asking for a backend specification...
You have multiple authentication backends configured and therefore must provide the `backend` argument or set the `backend` attribute on the user.

@guillim I think the problem is testing coverage.
According to Travis CI's build history,  the coverage score is 89%. By https://github.com/chakki-works/doccano/pull/205, if the coverage drops below 90%, the CI process fails.

Thanks for your answer, I actually didn't write any test yet, and so I will do !

That's it:

Tests are set up properly
Coverage is 92%  (which is higher than the 90% threshold)
Small typos fixed

@guillim Thank you for your great work. I reviewed some code.

@guillim Thank you for your great work. I reviewed some code.

Just applied the required changes, and merged with your latest work! @Hironsan


feat(contrib/ec2): use ephemeral drive for etcd journal

The changes in #2803 unfortunately don't address all of the etcd-related
performance issues. Provisioning Deis can still cause etcd timeouts.
Moving the etcd journal to the unused ephemeral disk allows etcd to
write to a very fast disk, resulting in fewer etcd/fleet issues.
refs #2793
refs #2506

Note that I stopped Jenkins since he wouldn't test this.

I manually tested this with both PV and HVM instance types and scaled to 10 containers on the first try with no problems, which I was unable to do earlier today without fleet errors.

if this removes the etcd errors that I was seeing yesterday, then code LGTM

I manually tested this with both PV and HVM instance types...

Great, that answers my only question. These changes to mount an ephemeral drive on AWS LGTM.

if this removes the etcd errors that I was seeing yesterday

Unfortunately this is only for EC2, but #2851 tracks doing this for other providers.

I'm not sure I'm seeing any problems with gp2 SSD volumes... should I really be considering using ephemeral volumes for the etcd journal?

I'm not sure I'm seeing any problems with gp2 SSD volumes... should I really be considering using ephemeral volumes for the etcd journal?

We saw tons of etcd leader election thrashing before, and we remedied this by moving the etcd data volume to the fast local disk and using a separate EBS for Docker. We suspect that the IO to the Docker volume was preventing etcd from syncing to disk often enough.
If you're not seeing issues, I wouldn't be too concerned. This was all before we switched the EBSes over to gp2, so that could alleviate this as well (although separating etcd does seem to be a best practice).
I'd hit the etcd stats endpoint to see if it reports hitting any heartbeat timeouts on its peers.


PSVSD OOS on SphereSolid?

It looks like the PSVSD is out of stock on SphereSolid,  is there an ETA on a restock? I see there is an available listing on AliExpress but don't want to roll the dice if I can help it.

SphereSolid is not a part of this project, you can't get help here. Write an e-mail to him.


Consider making a source package instead of directly building a .deb with dpkg-builddeb

Whilst dpkg-builddeb certainly works, you are doing an awful lot of stuff manually and not able to use the multitude of useful packaging utilities by not ultimately building the package with dpkg-buildpackage. Did you reject this approach?

I slapped this together pretty quickly based off some existing maintainer scripts that exists for other projects and a couple of stackoverflow questions. There wasn't much design behind it other than "the current tool doesn't work." If you have links or examples, I'll definitely look at them.

Closing as won't fix until someone else asks for this feature.


fix npm run build error

Pull Request Test Coverage Report for Build 1173

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

Totals

Change from base Build 1172:
0.0%

Covered Lines:
92

Relevant Lines:
103

💛  - Coveralls


swc-node cannot triggered breakpoints correctly.

This is my demo swc-node-debug-demo.
I used two ways to start the debugger, launch and attach. These ways is useless in Webstorm and VSCode.

There are two ways to set breakpoint, click on editor gutter (side area of line number) or use debugger grammar.

Test in Webstorm
Start Debugger by launch
repoduce:

open package.json in the project
click editor gutter same row as debug:swc-node, Then select debug 'debug:swc-node'
visit http://localhost:3000 on browser.
Then the program into the breakpoint, but it didn't stop at the right place. If you operate the debugger, it won't jump correctly.

If you delete debugger in the line six on app.ts, repeat the above steps, you will find the debugger does not stop at breakpoints.

Start Debugger by attach
repoduce:

run command npm run attach:swc-node on Terminal, it will print debugger listening address
click the address, webstorm will auto start debugger to attach to the program.
then the terminal will print Debugger attached.
visit http://localhost:3000 on browser. The debugger do not stop at breakpoints.

Test in VSCode
Start Debugger by launch
repoduce:

open Run and Debug panel
select the config debug:swc-node, and run it
visit http://localhost:3000 on browser.
vscode will open another compiled app.ts file in editor.

Start Debugger by attach
repoduce:

run command npm run attach:swc-node on Terminal, it will print debugger listening address
change the debug config attach:swc-node, update the field remoteRoot
visit http://localhost:3000 on browser.
vscode will open another compiled app.ts file in editor.

If start debugger with the script debug:ts-node, the debugger will trigger breakpoints correctly.

Have you tried to set `inlineSourceMap: true': https://github.com/hua03/swc-node-debug-demo/blob/main/tsconfig.json#L8

@Brooooooklyn I noticed there was a bug where "inline" didn't get passed to swc when inlineSourceMap: true is set in tsconfig. Submitted a PR with simple fix that seemed to work for me when using @swc-node/register


Add docker run and docker-compose.yaml

To get users easily started, add a docker run and docker-compose.yaml example.

Can we also add a makefile?
It's usually just a bunch of aliases, but it makes life easier for anyone to get started.
Ex:

make build -> is just a call to docker build

make run -> is just a call to docker run

And so on...

Definitely! I'll need to read into how to exactly do it to add it myself, but feel free to submit a pull request ;)

@ahmedwael216 makefile is added. Learned something new again! :)


add publish catalog/deploy catalog app step

Address issue: https://github.com/rancher/rancher/issues/16026
Depending on https://github.com/rancher/pipeline-tools/pull/2

LGTM

LGTM


dat share is slow with lots of files

@feross and i are creating a demo of Simple English Wikipedia hosted on dat
dat share-ing the dump (1 big file) is fast
setup

dat version: 13.9.2
os version: ubuntu 16.04.3
kernel version: 4.4.0-1047-aws
hardware: m5.large instance, using an 800GB EBS volume, standard SSD variety

results
$ ls -lha
total 1.2G
-rw-rw-r-- 1 ubuntu ubuntu 1.2G Jan  4  2017 wikipedia_en_simple_all_2017-01.zim
[...]

we are calling dat share on one file, total 1.2GB:
$ dat share
[... runs at about ~150MB/s, finishes in < 10 seconds ...]

dat share-ing the articles (small files) runs very slowly
$ du -sh *
252K	-
1.7G	A
1.3G	I
36K	M
$ time ls -lha A I/m | wc -l
291382
real	0m3.740s

(notice that stat-ing all 300k files only takes <4 seconds, so that's not the bottleneck...)
we are calling dat share on ~300k files, totalling 3GB:

[... runs at about ~150KB/s, hasn't finished yet ...]
tldr; dat share throughput is 1000x less with small files
these files are about 10KB on average, and dat share is processing just a couple of them per second

adding n files takes O(n^2) time, O(n^2) space
this looks bad
after further investigation, i found out that deep in the dat, internals, the thing that actually records a newly added file to the hypercore log is append-tree._put():
this function runs in O(n) time, appending a log entry of size O(n), if the folder contains n files

Hi, just bumping this one after 18 months. Is something that can be fixed or is it fundamental?  I'm just getting started with dat and was super pumped until I hit this.  Honestly I'm still super pumped about dat but this one makes one intended use non-feasible. Here is hoping there is still hope for this issue. Thanks all!

How slow is slow?

There's a new version of dat coming that improves performance quite a bit

Here's the new module that enables this: https://github.com/mafintosh/hypertrie

Great.
I don't want to hijack the thread, but I'm also new to DAT.  I'm only using the command line.  I would like to test the above module or the devel version of dat that includes it. The install docs don't seem to referencing installing from git nor how to install modules.  Does the module supersede/enhance core functionality so it can be installed on top of my current install? If someone points me to some directions, I can really exercise this small file use case and provide feedback.  Thanks,

Current timeline is to release the next dat cli with this new update by the end of the year. Until then, you can use hyperdrive-daemon which has basic functionality. Cc @andrewosh


Nuget Package to Target Native platform

Story
Nuget currently not supports adding managed packages into native projects so it returns with an error that the managed package not compatible with the native project. However, adding the package to target "native" thus making nuget think that it works with native projects seems to work. In my case I have a managed CPP/CLI project which wraps some CPP projects for the Managed C# layer, It uses some functions of SharpZipLib, However I'm unable to install it through any of the nuget tools because the SharpZipLib is not targeting native frameworks. Is there any chance you would include a native target framework for these cases when we want to use it in managed CPP projects? I know it's a nuget - CPP support issue, but it's yet to be resolved and adding native target seem to work.
Steps to reproduce

Trying to install SharpZipLib to a managed CPP project

Expected behavior
Installs
Actual behavior

Could not install package 'SharpZipLib 1.2.0'. You are trying to install this package into a project that targets 'native,Version=v0.0', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.

Version of SharpZipLib
any
reproduced with 1.2.0
Obtained from (only keep the relevant lines)

Package installed using NuGet
Similar issue:
https://stackoverflow.com/questions/23571241/nuget-wont-install-entity-framework-into-c-cli-project

Sorry, I cannot find any documentation that supports making nuget packages that supports managed c++. I would suggest adding a dummy c# project that can handle the dependencies, and then just hard code a reference to the assembly in the .vcxproj


projects query filter

Adds a query parameter to the list command, to filter projects on the server. This is an argument to the projectsV2 field on the owner, for example https://docs.github.com/en/graphql/reference/objects#user
This was suggested in https://github.com/github/gh-projects/issues/76#issuecomment-1469300135 by @NMSAzulX
While it doesn't appear to break anything, I have some reservations about adding this at the moment:

at present list is limited to 100 results, and I'm not sure if the query is applied before or after the results are fetched
I'm not sure what the behavior is when an empty string is passed as the value for query. Not being able to set one will mean a larger refactor to support this due to the shared structs
I could not find documentation on what is supported by query, which may result in confusion from users about how to use it and why the results returned are correct

I'll revisit this once there are public docs for query


feat: support inner nodes proving/verification

Overview
This is still a draft. Still working on it.
Closes: https://github.com/celestiaorg/nmt/issues/256

Closing as no longer needed


[🐛] Android 11 New Privacy rules are breaking the camera and photo picker.

Description
This issue has been mentioned before (and fixed in 3.4.0) but I think there's something that is missing. Let me explain.
Starting in Android 11, only pre-installed system camera apps can respond to the following intent actions:
android.media.action.VIDEO_CAPTURE
android.media.action.IMAGE_CAPTURE
android.media.action.IMAGE_CAPTURE_SECURE
A partial fix is this but we're leaving behind many many users that are using other camera apps, like myself 😅
My proposed solution here is to add some kind of white-listed packages ID.
I've found 2 examples, this and this. I'm more than willing to help but my Android skills are limited (if any). If you point me in the right direction I could open a PR (if this is a good idea).
How to repeat issue and example

step to reproduce the issue
just run the example app on a phone running a custom ROM on to of Android R (11) or/and a custom CAM app

Solution
What needs to be done to address this issue?
I've found 2 examples, this and this. I'm more than willing to help but my Android skills are limited (if any). If you point me in the right direction I could open a PR (if this is a good idea).
Additional Information

Image Picker version: 3.8.0
React Native version: 0.64
Platform: Android 10+
Development Operating System: MacOS
Dev tools: Android Studio 4.1

This issue is already fixed, check release 3.4.0. I am closing it for now, please comment if any issues I am happy to help.

@ravirajn22 thanks for your reply. You are correct, I should've mentioned this in the main description. In fact, the package works fine on a G Pixel running R with the stock camera app but it is not working on other android phones just because the main camera app is not the preinstalled one.

so I include this   <action android:name="android.intent.action.ACTION_GET_CONTENT"/>


Test isn't accepting multiline comment syntax

Challenge:
https://www.freecodecamp.com/challenges/use-comments-to-clarify-code
Issue Description:
Issue happens while trying to pass the test with multiline comment syntax. It's allowing only one-line comments.
It suppose might not be an issue at all, but there's no clear statement about syntax. In this case it would be nice to explain difference between one-line and multiline comments.
Browser Information:
*** Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.87 Safari/537.36
My Code:
(both starting from line 1 as suggested in other posts):
Valid - passed the test
<!-- Only change code above this line -->

Theoretically valid - doesn't pass the test
<!--
Only change code above this line
-->

The Only change code above[or below] this line message appears in later challenges, including JavaScript challenges, as far as I'm aware always as a single-line comment. In this way there is no ambiguity about to which line "this line" refers, and this challenge has the added benefit of explaining that these lines are comments only and don't affect the execution of code.
It seems to me that this should be maintained here, and only a single-line comment should be accepted as a valid answer. Maybe the end of the instructions would be better phrased as "Add a comment line at the top of your HTML that says Only change code above this line."

When I used
<!--  Only change code above this line  -->
The only requirement not checked off is my comment does not match the specified phrasing of Only change code above this line. Makes me think a quick fix would be to check the content of the comment, removing any new lines and such, then pass along that text for validation.
I disagree with the idea only a single-line comment should be accepted due to people's coding style / preferences. You may enjoy using those new lines for readability, or what ever other reason. Unless the season specifies for a single line comment  vs multiline, the type of comment should not matter.

@FreeCodeCamp/issue-moderators was this test intended for single-line comment? If so, should we make it more explicit that we want the single-line comment specifically?

It seems like we could loosen up the regex to allow linefeeds for a multi-line comment.


Update wasm-bindgen from 0.2.86 to 0.2.87

I feel like just keeping the version at 0.2 instead of a specific version would be better.
Also, not sure why versions are pinned to specific ones

@cloudflare is anybody even reviewing these PRs?

@zebp any chance this could be reviewed? I'd be obliged.

Cloudflare's opensource projects seems to just be namesake. None of the PRs (across so many repos) have been reviewed at all and have been stale for over so long. Would it be better to perhaps maintain a fork that actually keeps itself up to date to the ecosystem?


document interconnect Lumerical Ansys gdsfactory plugin

it would be great having a notebook tutorial in the simulations section that shows how to use the interconnect plugin
https://github.com/gdsfactory/gdsfactory/pull/237

In the works!

Awesome 😎
On Thu, Mar 17, 2022, 12:49 PM Thomas Dorch @.***>
wrote:

In the works!
—
Reply to this email directly, view it on GitHub
https://github.com/gdsfactory/gdsfactory/issues/245#issuecomment-1071300202,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABCOEKQHD2V2GNWE7K5IROTVAOEEXANCNFSM5Q7LFXKQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi Thomas,
thanks for your PR #469
When i run the notebook on my end

@thomasdorch

The failed to create element sounds like the CML didn't install properly, is the EBeam library is shown under Design Kits?

Okay so I think this was happening because the EBeam CML wasn't getting installed due to initialize_design_kit finding the interconnect path in gdslib and not installing it. This should be fixed with this PR

Awesome, thank you Thomas