Document SSCA Posture Management Features

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6696909481f546cddf1677b5--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6696bce11373c78384438475--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6696bd526a89c90dba04c1b0--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6696bd75f619af7bc74dcec5--harness-developer.netlify.app


Added memory ballast e2e behavior test

Added a new test that verifies that the ballast behaves only counts to
virtual memory and is not actually allocated.
Also added tests/results/BASELINE.md to act as the baseline test
results file. The TESTRESULTS.md file was modified on every run and it
would change in every PR from every contributor. It added additional
work when one didn't want to include it in a PR which was most of the
time. Only time it should be updated is when someone adds or updates a
new perf test.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@21339fb). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #85   +/-   ##
=========================================
  Coverage          ?   71.33%
=========================================
  Files             ?       91
  Lines             ?     6025
  Branches          ?        0
=========================================
  Hits              ?     4298
  Misses            ?     1503
  Partials          ?      224

Continue to review full report at Codecov.

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


echo server example will only respond once the send stream is closed

I don't actually want to merge this PR, i want to use it to illustrate a possible issue.
The rust client example currently fails with a timeout.  The server accepts a bi-directional stream from the client, but hangs waiting to read from the receive stream.
In this PR, in the client example, we call .finish() on the send stream before waiting for a response on the receive stream. This causes the server to actually receive the message, and it then responds echoing the message.
The web client example also closes the send stream before waiting to see the echoed response: https://github.com/kixelated/webtransport-rs/blob/3153f826549bfa04a220e89c8798f37b822dc39f/webtransport-quinn/web/client.js#L57 which explains why the web example works fine.
However, this is not the normal behaviour of Quinn - usually bi-directional streams allow both parties to send and receive data when the stream is open.
I have no idea why this is happening here, it could be something to do with the session mutex.

Yeah I encountered this bug too, but didn't bother to investigate it.
I think the issue is that the send stream is FIN only once dropped. The Rust compiler could drop send where you added the line, since it's no longer used, but it's probably getting dropped at the end of the function.
drop(send) will probably work as will creating a new scope. I think the finish solution is nice too.


ripple issue when setting drawable

On Samsung j7 android 6.1 when changing background drawable with ripple effect, ripple plays without touching.
even setting clickable to false wont fix the problem, but fixed problem for my other devices
this.setClickable(false);
this.setBackground(offRipple);
this.setClickable(true);

Could you please provide a runnable project which can reproduce the issue?

https://github.com/meysamhadigheh/HivaTecTools
HivaTecTools/app/src/main/java/hivatec/ir/hivatectoolstest/activities/ButtonActivity.java
there is a sample of multiselect buttons.


Task Styles is very slow

Submit a feature request or bug report

[x] This is a bug report

What is the current behavior?
Task Styles (in build or watch very very very slow)
[17:15:59] Using gulpfile /private/var/www/loooping2017/wp-content/themes/loooping2017/gulpfile.js
[17:15:59] Starting 'clean'...
[17:15:59] Finished 'clean' after 19 ms
[17:15:59] Starting 'default'...
[17:15:59] Starting 'build'...
[17:15:59] Starting 'wiredep'...
[17:16:00] Finished 'default' after 1.42 s
[17:16:00] Finished 'wiredep' after 1.52 s
[17:16:00] Starting 'styles'...
[17:16:07] Finished 'styles' after 6.39 s
[17:16:07] Starting 'jshint'...
[17:16:07] Finished 'jshint' after 309 ms
[17:16:07] Starting 'scripts'...
[17:16:13] Finished 'scripts' after 5.44 s
[17:16:13] Starting 'fonts'...
[17:16:13] Starting 'images'...
[17:16:13] gulp-imagemin: Minified 0 images
[17:16:13] Finished 'images' after 58 ms
[17:16:13] Finished 'fonts' after 185 ms
[17:16:13] Finished 'build' after 14 s
What is the expected or desired behavior?
Need much more velocity

Please describe your local environment:
WordPress version: 4.7.2
OS: Mac Os Sierra (10.12.3)
NPM/Node version: Node v7.5.0 | NPM 4.1.2

Need much more velocity

haha, yes! see https://discourse.roots.io/t/gulp-watch-taking-way-too-long/6124/12

Thanks brooo :P


Crash on Start possibility

I just see crash possibility:

if module: is missing in module configuration object
module configuration object is empty

crash return:
[2024-05-07 12:54:10.972] [ERROR] (node:18916) UnhandledPromiseRejectionWarning: TypeError: Cannot read properties of undefined (reading 'split')
    at loadModule (/home/bugsounet/MagicMirror/js/app.js:156:27)
    at loadModules (/home/bugsounet/MagicMirror/js/app.js:213:10)
    at async App.start (/home/bugsounet/MagicMirror/js/app.js:261:3)
(Use `electron --trace-warnings ...` to show where the warning was created)

I will see to patch it

but a few modules in defaults don't have config object.. why now?

oh..
I thought you meant
config:{}

should we correct this issue?

yes.. never crash

never crash, no issue about this,  so it's not needed ;)

I font understand.  we should never crash from users mistakes.
so anytime we find one we should fix it. like this
yes it's needed to.prevent crash


Cuda flavor not working

I made an empty env with just python=3.10 and pip:
$ micromamba create -n openmm82-pypi-cuda pip python=3.10
I then activated the environment and installed the cuda package variant:
$ pip install --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple/ openmm[cuda12]
Running test installation didn't show the cuda platform:
$ python -m openmm.testInstallation

OpenMM Version: 8.2
Git Revision: ffb3082fe7a2cec102acefb30c373db92d035968

There are 3 Platforms available:

1 Reference - Successfully computed forces
2 CPU - Successfully computed forces
3 OpenCL - Successfully computed forces

Median difference in forces between platforms:

Reference vs. CPU: 6.3083e-06
Reference vs. OpenCL: 6.75018e-06
CPU vs. OpenCL: 7.64527e-07

All differences are within tolerance.

So I checked for plugin loading failures:
$ python -c "import openmm; print(openmm.Platform.getPluginLoadFailures())"
# reformatted this slightly so it is easier to read
('Error loading library /home/mmh/micromamba/envs/openmm82-pypi-cuda/lib/python3.10/site-packages/OpenMM.libs/lib/plugins/libOpenMMHIP.so: libhiprtc.so.6: cannot open shared object file: No such file or directory',
'Error loading library /home/mmh/micromamba/envs/openmm82-pypi-cuda/lib/python3.10/site-packages/OpenMM.libs/lib/plugins/libOpenMMCUDA.so: libcufft.so.11: cannot open shared object file: No such file or directory',
'Error loading library /home/mmh/micromamba/envs/openmm82-pypi-cuda/lib/python3.10/site-packages/OpenMM.libs/lib/plugins/libOpenMMRPMDHIP.so: libOpenMMHIP.so: cannot open shared object file: No such file or directory',
'Error loading library /home/mmh/micromamba/envs/openmm82-pypi-cuda/lib/python3.10/site-packages/OpenMM.libs/lib/plugins/libOpenMMRPMDCUDA.so: libOpenMMCUDA.so: cannot open shared object file: No such file or directory',
'Error loading library /home/mmh/micromamba/envs/openmm82-pypi-cuda/lib/python3.10/site-packages/OpenMM.libs/lib/plugins/libOpenMMDrudeHIP.so: libOpenMMHIP.so: cannot open shared object file: No such file or directory',
'Error loading library /home/mmh/micromamba/envs/openmm82-pypi-cuda/lib/python3.10/site-packages/OpenMM.libs/lib/plugins/libOpenMMAmoebaHIP.so: libOpenMMHIP.so: cannot open shared object file: No such file or directory',
'Error loading library /home/mmh/micromamba/envs/openmm82-pypi-cuda/lib/python3.10/site-packages/OpenMM.libs/lib/plugins/libOpenMMDrudeCUDA.so: libOpenMMCUDA.so: cannot open shared object file: No such file or directory',
'Error loading library /home/mmh/micromamba/envs/openmm82-pypi-cuda/lib/python3.10/site-packages/OpenMM.libs/lib/plugins/libOpenMMAmoebaCUDA.so: libOpenMMCUDA.so: cannot open shared object file: No such file or directory')

The file is there, but there seems to be a linking issue?
$ ldd /home/mmh/micromamba/envs/openmm82-pypi-cuda/lib/python3.10/site-packages/OpenMM.libs/lib/plugins/libOpenMMCUDA.so
	linux-vdso.so.1 (0x00007cadf6ba9000)
	librt.so.1 => /lib/x86_64-linux-gnu/librt.so.1 (0x00007cadf6b7d000)
	libOpenMM.so => not found
	libcuda.so.1 => /lib/x86_64-linux-gnu/libcuda.so.1 (0x00007cadf4600000)
	libcufft.so.11 => not found
	libnvrtc.so.12 => not found
	libpthread.so.0 => /lib/x86_64-linux-gnu/libpthread.so.0 (0x00007cadf6b76000)
	libdl.so.2 => /lib/x86_64-linux-gnu/libdl.so.2 (0x00007cadf6b71000)
	libstdc++.so.6 => /lib/x86_64-linux-gnu/libstdc++.so.6 (0x00007cadf4200000)
	libm.so.6 => /lib/x86_64-linux-gnu/libm.so.6 (0x00007cadf6a8a000)
	libgcc_s.so.1 => /lib/x86_64-linux-gnu/libgcc_s.so.1 (0x00007cadf6a6a000)
	libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007cadf3e00000)
	/lib64/ld-linux-x86-64.so.2 (0x00007cadf6bab000)

Also sorry @peastman I didn't realize you will still debugging this!

Feel free to help debug it if you want!  At the moment I'm looking into the Mac builds, which fail with a linker error.  otool -l _openmm.cpython-310-darwin.so reveals
Load command 16
          cmd LC_RPATH
      cmdsize 48
         path /Users/runner/openmm-install/lib (offset 12)

That was the path to the libraries on the build machine.  delocate should have replaced it...

It looks like there's no RPATH specified in libOpenMMCUDA.so.  The CUDA libraries installed with pip get put in a bunch of folders: site-packages/nvidia/cufft/lib, site-packages/nvidia/cuda_nvrtc/lib, etc.  If we want that to work, we need to make it search all of those locations.

Can you test the latest build from #13?  It sets the RPATH, and I verified that it's present in the installed library:
$ objdump -x ~/miniconda3/envs/test/lib/python3.10/site-packages/OpenMM.libs/lib/plugins/libOpenMMCUDA.so | grep PATH
  RUNPATH              $ORIGIN/..:$ORIGIN/../../../nvidia/cufft/lib:$ORIGIN/../../../nvidia/cuda_nvrtc/lib

When I run ldd on it, it reports
	libOpenMM.so => /home/peastman/miniconda3/envs/test/lib/python3.10/site-packages/OpenMM.libs/lib/plugins/../libOpenMM.so (0x00007fb1c77c9000)
	libcuda.so.1 => /lib/x86_64-linux-gnu/libcuda.so.1 (0x00007fb1c562b000)
	libcufft.so.11 => /usr/local/cuda/lib64/libcufft.so.11 (0x00007fb1b4600000)
	libnvrtc.so.12 => /usr/local/cuda/lib64/libnvrtc.so.12 (0x00007fb1b0800000)

It's successfully finding libOpenMM.so, but it's linking to globally installed versions of libcufft.so and libnvrtc.so.  Presumably it's because that location is higher in the search path, and it would use the ones from the environment if there weren't global ones?

How do I delete the exiting wheel from the test server and replace it with the fixed one?  If I try to delete the existing one, it warns me I won't be able to upload a new file with the same name.

PyPI doesn't let you "overwrite" existing releases, so we need to change the version to something like 8.2.0rc1 or something, really whatever convention you like from https://packaging.python.org/en/latest/specifications/version-specifiers/#examples-of-compliant-version-schemes

Also #13 worked!
OpenMM Version: 8.2
Git Revision: f5fc52ffd757a86aa1d05bd35a21108deff9eda1

There are 4 Platforms available:

1 Reference - Successfully computed forces
2 CPU - Successfully computed forces
3 CUDA - Successfully computed forces
4 OpenCL - Successfully computed forces

Median difference in forces between platforms:

Reference vs. CPU: 6.29719e-06
Reference vs. CUDA: 6.74822e-06
CPU vs. CUDA: 7.47712e-07
Reference vs. OpenCL: 6.75018e-06
CPU vs. OpenCL: 7.6531e-07
CUDA vs. OpenCL: 1.78763e-07

All differences are within tolerance.


OutputCompletionSource

Adds OutputCompletionSource<T> that allows the creation of an Output<T> that can later be resolved to the final result of a different Output<T>.
This can be used to solve partial circular reference problems. For example given two components that create multiple resources and return some results from that, you would normally have to have all the inputs for one component available to call it and then afterwards call the other component. With an OutputCompletionSource<T> you can instead set up a circular reference:
var ocs = new OutputCompletionSource<int>()
var (resultA1, resultA2) = MakeComponentA(ocs.Output);
var resultB = MakeComponentB(resultA1);
ocs.Set(resultB);

As long as MakeComponentA can resolve its A1 result without needing the incoming output resolved then the above will work.
Note that it's possible to deadlock the program with this feature. If resultA1 above can't resolve until the input is resolved, and resultB can't resolve till its input is resolved then both will end up stuck waiting for the other. Users can already deadlock their programs with other dotnet constructs (for example doing a similar thing with TaskCompletionSource<T>).

Yeh let me fill in docs then we can probably merge this.

Are we settled on the API design and naming for this? What's it going to look like in the other SDKs?

Will be discussing next week but leaning towards a unified name DeferredOutput<T> for all the languages


Bring back playwright to stable versions

We will provisionally use next, and this is just a reminder.

In #332 alpha version of playwright (@playwright/test@next) has been installed to have the following bug fixed:
https://github.com/microsoft/playwright/issues/27617
We should go back to a stable version when the fix will be included in the next playwright release.


Get.offAllNamed lost their arguments.

Describe the bug
Get.toNamed(SomeRouteName, arguments: myArguments)
Get.offNamed(SomeRouteName, arguments: myArguments)
Get.offAllNamed(SomeRouteName, arguments: myArguments)
Get offAllNamed with arguments return always null.
Is it expected behavior?
To Reproduce
try to call offAllNamed method with my arguments.
Get.offAllNamed(SomeRouteName, arguments: myArguments)
it gives always null.
Expected behavior
print(Get.arguments);
'myArguments...'
Screenshots
If applicable, add screenshots to help explain your problem.
Flutter Version:
Channel stable, v1.17.1 on Mac OS X 10.15.1
Get Version:
get: ^2.5.6
Describe on which device you found the bug:
Ios 11 pro simulator
Minimal reproduce code
Provide a minimum reproduction code for the problem

Get.routeSettings.arguments is it also null?

Fix on 2.5.9 version.

Sorry for late reply and
Thank you :)


doesnt work on Firefox

TypeError: e is undefined
    NextJS 11
[framework-63157d71ad419e09.js:9:69810](https://github-guessr.vercel.app/_next/static/chunks/framework-63157d71ad419e09.js)
Uncaught TypeError: e is undefined
    NextJS 11
[main-c59a42b9e9a6f445.js:1:11213](https://github-guessr.vercel.app/_next/static/chunks/main-c59a42b9e9a6f445.js)

Additional information: Tested it with Firefox for Android and it worked.

Thank you for the feedback!
May I ask for more details about your environment? I read your comment and tried it on both Firefox for Mac and Firefox for Android, but I didn't see any errors.

what details

Found this issue on NextJS repo, it doesn't work on version 101 of Firefox.

The NextJS development team has made a fix for this issue, see file.
It is available in canary release, so maybe next stable version of NextJS will include this fix.

Indeed, I got an error when I ran it on the older version. Thank you for your help!
I'll fix it in this Pull Request https://github.com/tan-z-tan/GitHubGuessr/pull/5


Added logic in edit_buildspec_file and edit_buildspec_test to allow f…

…or the regression testing from test_edit_test and test_edit_file
.
Me and @Xiangs18 had an interesting discussion on the best way to call edit_buildspec_file and edit_buildspec_test in the regression test. I left both positional arguments editor and buildspecs null because I knew that the open=False would stop that from mattering. @Xiangs18 was rightfully concerned about this being very un standard. @shahzebsiddiqui Do you think what I did was appropriate. And if I should create builspecs for this file what would they be.

@JeffreyFG just one tip when opening PR please update the title of PR to something meaningful since github uses the commit message and that doesn't make sense.
Also if u see a PR address an issue u can also link that in PR.

@JeffreyFG i realize that the regression test will not test coverage for the entire method simply because we have it only set to open=False if we set to open=True it won't work right now since it will just hang indefinitely.
One idea i had in mind, we would need to refactor the original implementation by changing the subprocess.call to BuildTestCommand in https://github.com/buildtesters/buildtest/blob/devel/buildtest/cli/buildspec.py#L946

Note that this class execute method has an optional argument timeout which can be set to something like 1 or 2 second during the regression test.
https://github.com/buildtesters/buildtest/blob/7eba9872f0ee5ea3a71b8ee5355f3410fc72f793/buildtest/utils/command.py#L88
We would need to pass the timeout argument to the method edit_buildspec_test which by default should be timeout=None. I hope that makes sense. Please try this out and run the regression test locally and see if the coverage report captures entire method.


README - Getting started

Adding a getting started section
Quick start commands
Adding explanation for our yaml files

Couple of questions:

Why are we putting them all in the readme.md as opposed to another file/folder and linking to it?
Do we think folks are going to continue to scroll that far to see them?
How will we ensure these stay up to date as the objects change?
There are several other constraints on several of the fields in many files, that we've not indicated in these tables- are we planning to expand this more later, or just rely on validation failures to guide folks along?

Known failure in app inspect! merging this PR


fix moving file function

#253. Pass the full smb-uri as a parameter when moving files, no longer check and splite.
#205. Handle the case where the new name is actually a smb-uri, which means it should be moved to a new directory.
#208. No longer judge whether the path ends with /, the renaming logic is the same.
Has been compiled and run verification.

It's my fault that I didn't unify the quotation marks.


woah rude

turbo rude

cantfix wontfix

@tpope, congrats for the initiative.
It would be better if could silent specific gems only, like httparty since some others may have useful changelogs that we could be interested on during a bundle update for example... Maybe we could have a ~/.silenced_gems file where we could just put "httparty" there. Maybe installing gem-shut-the-fuck-up could create this file with this line by default...

This would actually be a nice gem that people used if you hadn't decided to include random and unnecessary cursing. But hey, to each his own. :smile:

Brilliant


possibility to destroy the refresh mechanism

I need this on my one-page-app to be able to have a pull-to-refresh mechanism only on some pages

Why not using hammer.destroy() instead. :)

hmmm... is it really the same?


Is the RPM binary built from opentelemetry-collector-contrib code

Hi,
I would like to ask whether the releases released in github.com/open-telemetry/opentelemetry-collector-releases/releases built from the code here https://github.com/open-telemetry/opentelemetry-collector-contrib, please?
If no, where can we get the source code for this binary?
Thank you.
Best Regards,
Erica Ooi

It is, here's the manifest of the components included in the binary: https://github.com/open-telemetry/opentelemetry-collector-releases/blob/main/distributions/otelcol-contrib/manifest.yaml


Rerouted Content and Improved UI

Transferred contents of app/mainserver/page.tsx to app/home/page.tsx. Nearly completed UI of the main chatting page for CTPChat.
app/home/page.tsx changes:
Added Settings Dialog when the Settings button is pressed. Added infoContainer button to collapse left side in the future. Changed colors for all elements. Changed some elements' opacity such as chatContainer. Began adding explicit chat functionality to chatContainer.
chatContainer's chatting functionality and buttons still have a few bugs. Chatting does not appear for the first time when "submit" button is clicked. Top left and right buttons do not collapse left and right divs yet.

Please resolve the conflicts

Resolved conflicts from /signup, /login, and homepage /app. Reverted /signup and /login pages to pre-pull request and updated homepage /app to be able to add logo clickable functionality as explained in latest commit.


FIX: Fixes #211

Allowing locale switcher to maintain correct highlighted default when using browser navigation.

I think that this improves the overall value of the module, even if the cookie key is hard-coded.
Feel free to soft-code the key in another PR if you like. :D

Feel free to soft-code the key in another PR if you like. :D
Friday :-)

Yeah, I can't be too demanding can I? :P


How do you update AutoLayout constraints in IGListCollectionView? What about Nested IGListCollectionView's?

Hello All,
I am currently working on a project in which I need to create multiple levels of expanding views (see screenshot below).

I have this implemented with IGListKit following something similar to the 'NestedAdapterViewController' View Controller in the provided example project. I have a SectionSectionCollection which is returned for my objects that are sections and creates a single cell with a UICollectionView within it. The SectionSectionCollection has an adapter and is the IGListAdapterDataSource for the cell's UICollectionView. As the datasource if this section has a section within it's IGListDiffable it will return an additional SectionSectionController and thus we have nested IGListCollectionView's! Wooo!
Where I am having a problem is the fact that the 'SectionCells' have constraints that will get updated to expand or collapse. I am not sure exactly how to go about having the IGListCollectionView's update to be the height of their content as child views are expanded.
The solution I have now works for the first level that is 'Section 1' and the second level 'Section 2'. I am setting the cell's button to a target selector within the 'SectionSectionController' and calling [self.collectionContext reloadSectionController:self]; on the parent SectionSectionController.
Q: How do I expand a cell and have the IGListCollectionView respond to the updated constraint.
Q: How do I put this into practice with nested IGListCollectionView's, is the way I am currently doing it along the right track?

#pragma mark IBActions
- (void)cellButton_TouchUpInside:(id)sender {
    _isExpanded = !self.isExpanded;

    [self reloadSectionController];
}

...

#pragma mark <IGListSectionType>
- (UICollectionViewCell *)cellForItemAtIndex:(NSInteger)index {
    ExpandingCell *cell = (ExpandingCell *)[self.collectionContext dequeueReusableCellOfClass:[ExpandingCell class] forSectionController:self atIndex:index];

    [cell setIsExpanded:self.isExpanded];
    [cell setSection:self.object];
    [cell.cellButton addTarget:self action:@selector(cellButton_TouchUpInside:) forControlEvents:UIControlEventTouchUpInside];
    [self.adapter setCollectionView:cell.collectionView];

    return cell;
}

...

#pragma mark Private API
- (void)reloadSectionController {
    if(!!self.parentSectionController) {
        [self.parentSectionController reloadSectionController];
    } else {
        [self.collectionContext reloadSectionController:self];
    }
}

New issue checklist

[ YES ] I have reviewed the README and documentation
[ YES ] I have searched existing issues and this is not a duplicate

General information

IGListKit version: 2.0.0
iOS version(s): 9.0
CocoaPods/Carthage version: CocoaPods
Xcode version: 8.1
Devices/Simulators affected: N/A
Reproducible in the demo project? (Yes/No): N/A
Related issues: N/A

Oh man, this is tricky 🤔 . I think I understand your setup:

Vertical collection view + adapter
Parent section controller that returns a single SectionCell
SectionCell has an embedded vertical collection view + adapter
SectionCell height should be the content size of the embedded collection view

Right? YMMV, I've never done this before.
One idea worth trying is the parent section controller owning the UICollectionViewLayout that powers the embedded collection view. Attach the layout when creating your cell:
@interface ParentSectionController
@property IGListAdapter *adapter;
@property UICollectionViewFlowLayout *layout;
@end

- (CGSize)sizeForItemAtIndex:(NSInteger)index {
  return CGSizeMake(self.collectionContext.containerSize.width,
    self.layout. collectionViewContentSize.height);
}

- (UICollectionViewCell *)cellForItemAtIndex:(NSInteger)index {
  SectionCell *cell = [self.collectionContext deque...];
  cell.collectionView.collectionViewLayout = self.layout;
  self.adapter.collectionView = cell.collectionView;
  return cell;
}

If that doesn't work, the other option is having the parent section controller own the embedded collection view itself and just addSubview: when configuring the cell. This is probably less performant, but might work.
cc @jessesquires for other ideas?

@rnystrom  That is pretty much my setup.
Except:

the adapter for the cell's UICollecitonView is owned by the SectionSectionController that creates it.

Also right now the SectionCell creates a basic UICollectionViewFlowLayout when it creates the IGListCollectionView. I'll give your suggestions a try and let you know how they work out!

Oh wow so there are multiple layers of nesting, dang! That's pretty cool, but tough. Let me know how playing w/ who owns the layout/collectionView goes.
FWIW we do the same thing, having our cells create the layout+collectionView, but we don't really have any requirements quite like this.

I was unable to get this to work with the UICollectionViewFlowLayout owned by the SectionController. I decided to try to figure this out with a more simple problem, i.e. without expanding / collapsing the SectionCells.
I can add all the items and adjust the SectionCell's constraintCollectionViewHeight.constant on preferredLayoutAttributesFittingAttributes:. This will set the correct height of collection view. The collection view however will not update unless I start to scroll a bit.
What am I missing here?

@egarlock this is sort of a "chicken and egg" problem, since the parent will have to size itself first, then size the child. However the child is driving the size of the parent.
What about after the initial sizing of the child calling [self.collectionContext reloadSectionController:self]?

Demo
I went ahead and made a repo to show you what I'm currently doing. It's hard to explain. Everything works for the first 2 levels and I'm using [self.collectionContext reloadSectionController:self].

I decided to go with custom expandable views for now. Thanks for the help @rnystrom!

Sure thing! Closing this as it sounds like you're unblocked. If you come up w/ any interesting solutions for this, more details or a blog post or something would be really helpful for others to learn from. Pretty interesting problem!

Can I bring this thread back to life and ask a related question?
I have nested section controllers that go 3 levels deep. It's very similar to Instagram's Profile UI. I'll break it down to make sure I'm being accurate here - let's say Instagram's Profile UI was broken down as such from top to bottom:

TopSectionController (FollowersCell, UsernameCell, BioCell, ButtonsCell)
StoriesSectionController (NestedHorizontalCollectionViewCell)

NestedSingleStorySectionController (StoryCell)

ContentSectionController (NestedHorizontalCollectionViewCell + SegmentedControlHeaderView)

PhotosSectionController (NestedVerticalCollectionViewCell)

PhotoItemSectionController (PhotoItemCell)

ReelsSectionController (NestedVerticalCollectionViewCell)

ReelItemSectionController (ReelItemCell)

TaggedPhotosSectionController (NestedVerticalCollectionViewCell)

PhotoItemSectionController (PhotoItemCell)

Two things stand out to me:

When you swipe up from the top of the the screen, the deceleration of the top-level collection view is passed to the bottom-level collection view (or scrolling is disabled on the bottom-level vertical CV).
More Importantly, getting the sizing down of the nested collection views. It seems when you swipe horizontally the contentOffset of the bottom-level vertical collection view (photos, reels, tagged) is set to 0, and the size is calculated from bottom to top before the entire ContentSectionController  is reloaded with the items size?

I have a very, very similar overall architecture for a UI I'm experimenting with and I can't seem to get the second item perfectly right.


Fix incomplete pattern matches over C# enums producing the wrong warning

PR #4522 has a bug, causing it to always produce FS0104 for non-F#-defined enum. For example, the following code:
// System.DateTimeKind = | Unspecified = 0 | Utc = 1 | Local = 2
let f = function System.DateTimeKind.Unspecified -> 0

currently produces this compiler warning:
typecheck error FS0104: Enums may take values outside known cases. For example, the value 'enum<System.DateTimeKind> (1)' may indicate a case not covered by the pattern(s).

when it is supposed to produce:
typecheck error FS0025: Incomplete pattern matches on this expression. For example, the value 'System.DateTimeKind.Utc' may indicate a case not covered by the pattern(s).

This PR corrects the issue.
Thanks to @charlesroddie for identifying the bug.

@dotnet-bot test this please

@cartermp What is the cause of the CI failure? It reports:
Exception: Yaaf.FSharp.Scripting.FsiEvaluationException: Error while compiling or executing fsharp snippet. ---> System.Exception: Operation failed. The error text has been printed in the error stream. To return the corresponding FSharpErrorInfo use the EvalInteractionNonThrowing, EvalScriptNonThrowing or EvalExpressionNonThrowing
... etc etc
System.Net.WebException: The remote server returned an error: (404) Not Found.
   at System.Net.WebClient.DownloadFile(Uri address, String fileName

cc @matthid  - sounds like a FAKE issue on CI?

@forki Is this the broken .NET SDK installer problem? Where is the other issue about that?

I only know of Twitter discussion. And the urls worked for us again. /cc
@davkean
Matthias Dittrich<EMAIL_ADDRESS>schrieb am Mi., 20. Juni 2018,
21:56:

@forki https://github.com/forki Is this the broken .NET SDK installer
problem? Where is the other issue about that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/visualfsharp/pull/5206#issuecomment-398876573,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNAo-bN_BV37JUiU-mP1B73mDGQkGks5t-qkHgaJpZM4Uqo0M
.

test Ubuntu16.04 Release_fcs Build Windows_NT Release_fcs Build

test Windows_NT Release_fcs Build

Thanks @cartermp! Now that CI is working... Requesting reviews 🙂 @dsyme @forki

So typical try again until it works error?

@matthid jawohl!

There's no more for me to do at the moment. Could this be considered for merge? @dsyme


use conduit 0.9.0 for 3.4.1

Conduit release is coming, use it for VisIt 3.4.1 that way folks can access new Silo features via VisIt's CLI

#18962

We will use conduit 0.9.1 due to address a C++ standard setting in mfem 4.6


Docker Toolbox - local host refused to connect

Hi all, I went to office hours today with Arjun to catch up on the lecture with docker set up, since i have windows and didn't follow along on my computer then. we successfully got docker toolbox to work, but i am now having issues with seeing the image -

does anything stick out here? do you have any lecture notes for this class? I didn't see anything in the repo but I could have missed it
Thanks!
Sonali

If you swing by my office, I can take a look w/ you.

ok walking over

Hey folks, did this end up getting resolved? If not, let me know and I can continue to help out!

@suyashkumar - sorry for missing your comment!
We decided to install docker on my vm instead, which worked but I am not able to access Ouwen's stuff

am i supposed to be inputting my vm ip address somewhere in that command? @mlp6 or @suyashkumar

Try running that same command with sudo at the beginning.

ya i just noticed that and did so:

You don't want to use the URL: localhost:8888; you want to replace localhost with the IP address of your VM.  You should be able to get that using the command /sbin/ifconfig, which should yield something that starts with 152.16.XXX.XXX or 152.3.XXX.XXX.

For example: <IP_ADDRESS>:8888

ah ok yes that is what i was thinking but wasn't sure of if i was right / how. will try that now

Here is what my command output for the IP address yields:
mlp6@vcm-299:~$ /sbin/ifconfig
eth0      Link encap:Ethernet  HWaddr 00:50:56:a1:08:cc
          inet addr:<IP_ADDRESS>  Bcast:<IP_ADDRESS>  Mask:<IP_ADDRESS>

You want the address entry that shows up associated with inet addr.

ya i got it. phew. back on track

🕺


Inconsistent results among multiple runs

I am using twitterscraper to get the replies to some twitter accounts.
I am running the following queries as a test:
to%3Amatteorenzi%20since%3A2017-08-21%20until%3A2017-08-27
to%3Amatteosalvinimi%20since%3A2017-08-21%20until%3A2017-08-27

When performing multiple runs I get a different amount of results each time as below, with left number being the result of first query and right one for the second. Each line is a different run.
544, 4216
386, 4121
295, 4180
Why does this happen? Any way I can prevent it?

Have you checked the keyword thorugh https://twitter.com/search-advanced
Recently I checked there is something wrong with Twitter, because when I run the keyword Telkomsel, it doesn't work well. I think your keyword got the same problem from Twitter.
But it can work well if we add another keyword after the main keyword as example : Amatteorenzi cool

What I am doing is collecting replies to a given twitter user. Given the amount of results (thousands) I can't check by hand what is the correct result. I am going to investigate this further and provide updates if I find some more useful information.

Thank you @Fragoel2 . At the moment I am too busy with work to have a look at it myself.

Hi @Fragoel2 ,
For three runs of the first query, I get the following 1087 tweets at each run. That is, there is no inconsistent run...
With you It seems to be exiting earlier for some reason. Can you find something in the log file or could it have something to do with your internet speed?


Update auto-apply-retention-labels-in-office-365-using-content-types-…

…and-metadata.md
Please use this template to ensure easier processing of your pull request
Category

[ ] Content fix
[ ] New article
[x] Example checked item (delete this line)

For the above list, an empty checkbox is [ ] as in [SPACE]. A checked checkbox is [x] with no space between the brackets. Use the PREVIEW tab at the top right to preview the rendering before submitting your issue.
(DELETE THIS PARAGRAPH AFTER READING)

Related issues:

fixes #issuenumber
partially #issuenumber
mentioned in #issuenumber

If this fixes (aka: closes) or references an issue, please reference it here. This helps maintaining the issue list as it will (1) link the PR to the issue & (2) automatically close the issue when this PR is merged in.
(DELETE THIS PARAGRAPH AFTER READING)

What's in this Pull Request?

Please describe the changes in this PR. Sample description or details around bugs which are being fixed.
(DELETE THIS PARAGRAPH AFTER READING)

Guidance

Please update this PR information accordingly. We'll use this as part of our release notes in monthly communications.
Please target your PR to 'master' branch. Released documents are in live branch.
(DELETE THIS PARAGRAPH AFTER READING)

Docs Build status updates of commit 8770d34:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Community/auto-apply-retention-labels-in-office-365-using-content-types-and-metadata.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Testing with @joannecklein


Feature: Module selection should highlight whole dependency tree, and call out any dependencies unique to the selected module.

One question I often have about a dependency graph is, "If I add/remove [module X], what dependencies actually get added / removed from the graph."  I don't know of any tools that answer this particularly well, but I think we're well positioned to do so.
For example, we could enhance the current module selection to highlight the whole descendent dependency tree, rather than just immediate child dependencies.  And, further highlight which of those dependencies are exclusive to the selected module. (i.e. which ones would actually get removed if the selected module is removed.)
#67 sort of does this in that when you collapse a module, any dependencies it shares with other modules will remain in the graph - but it's hard to see due to how the graph gets re-laid out.

I think questions like this are difficult to show in a graph, generally speaking.
In this case specifically I think it would make the most sense to show an additional, separate graph for the new module and its dependencies, dimming any dependencies that overlap with the existing ones.


Simplify component imports

Description

move pipeline_components to pipelines/kfp_components
update Makefile
update cloud build actions

This removes the need to compile each component individually before compiling & running a pipeline. As a result, the dev workflow has one less step and the overall compilation time locally and in cloud build reduces.
How has this been tested?

[x] make setup
[x] make setup-components
[x] make setup-all-components
[x] make test-trigger
[x] make compile-pipeline
[x] make test-components
[x] make test-all-components
[x] make run
[x] make e2e-tests

Checklist

[x] I have commented my code, particularly in hard-to-understand areas
[x] I have successfully run the E2E tests, and have included the links to the pipeline runs below
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] I have updated any relevant documentation to reflect my changes
[ ] I have assigned a reviewer and messaged them

Pipeline run links:

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun


Finally need a group noun

Proper nouns Group and Entity are currently handled by the generic Noun class, but we need a way to handle groups with sorting so group names like The Judds and The Rolling Stones can be properly sorted.

Commit:
https://github.com/cjcodeproj/medialibrary/commit/8355f9871029eb05b5f83a7f5018f9fccadb8036

Closing Notes

Covered both Entity and Group since the code is nearly identical
Support for Entity acronyms.


Insufficient color contrast on highlighted code block in Dark Mode

Affected page
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures/object-oriented-programming/define-a-constructor-function
To Reproduce
Steps to reproduce the behavior:

Go to https://www.freecodecamp.org/learn/. Login & switch to Dark mode from Menu.
Click on Java Script Algorithm and Data Structure Certificationcourse
Scroll down to 'Object Oriented Programming' module & open 'Define Constructor Function' page.
Highlight the code block in the theory section by selecting & dragging with the mouse.

Expected behavior
On highlighting, the code becomes unreadable due to failing color contrast. This is a WCAG violation. There should be sufficient color contrast ratio (4.5:1) between text and background even when highlighted.
Screenshots

System:

Device: Laptop
OS: Windows 10
Browser: Edge (chromium based)
Version: 91.0.864.41 (Official build) (64-bit)

Hi @pkrc267 Can I work on this issue?

Hey @aknouche
Issues are typically not assigned to anyone other than long-time contributors. You can read more about this at https://contribute.freecodecamp.org/#/FAQ?id=how-can-i-get-an-issue-assigned-to-me.
Issues labelled with help wanted or first timers only are open for contributions.
Please make sure you read our guidelines for contributing. Join us in our chat room or the forum if you need help contributing, our moderators will guide you through this.

Thanks for the reply @gikf, This issue is marked with the label "help wanted" so can I work on it and make a PR without being assigned to it then?

Yep, @aknouche, that's exactly right. You're very welcome to work on this.

Ok will do thank you : )

The lesson doesn't matter here as the issue is present in every challenge which shows a code snippet. Basically a global css style has to be set just for the selection of the code snippet part.

Hi, Ok thanks. I can see that this will be fixed through another issue so I
will let it be
Den ons 9 juni 2021 kl 06:23 skrev Niraj Nandish @.***>:

The lesson doesn't matter here as the issue is present in every challenge
which shows a code snippet. Basically a global css style has to be set just
for the selection of the code snippet part.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/issues/42405#issuecomment-857361472,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKGLCGJ52L4UAV6IYDLYVXDTR3UEZANCNFSM46JSSCKQ
.

Hey, I want to work this issue, Can you assign me this?

@prsmahajan, it looks like there is already a PR open for this issue.


[vcs] Autogenerated VCS Makefiles not listing C++ include dirs

Hi all,
VCS builds fail due to incorrect use of filelist +incdirs+ and needed args on the VCS command line.
Details:
After running, e.g.
fusesoc --cores-root . run --flag=fileset_top --target=sim --tool=vcs --setup --build lowrisc:systems:top_earlgrey
the Makefile written to
$REPO_TOP/build/lowrisc_systems_top_earlgrey_0.1/sim-vcs/Makefile
does not contain the include dirs for the C++ files listed in the filelist:
$REPO_TOP/build/lowrisc_systems_top_earlgrey_0.1/sim-vcs/lowrisc_systems_top_earlgrey_0.1.scr
Instead, the C++ include dirs are listed as +incdir+ plusargs in the filelist. This doesn't work, and the build fails (after a few other HDL fixes - out of scope for this issue).
+incdir+ plusargs only tell VCS where to look for includes in HDL files. In order to specify passthrough arguments for the C++ compiler, the VCS command can use -CC and -LDFLAGS (among others detailed in the VCS manual) at the end of the line, each followed by the appropriate passthroughs for the C++ compiler.
This is true as of VCS P-2019.06-1.

The behavior you're describing is expected. What are you trying to achieve? We don't run top-level simulations through fusesoc directly, but use dvsim instead, where we pass additional options to VCS (or other simulators) as needed. You can find a bit of documentation at https://docs.opentitan.org/doc/ug/getting_started_dv/#building-and-running-tests.

The behavior you're describing is expected. What are you trying to achieve? We don't run top-level simulations through fusesoc directly, but use dvsim instead, where we pass additional options to VCS (or other simulators) as needed. You can find a bit of documentation at https://docs.opentitan.org/doc/ug/getting_started_dv/#building-and-running-tests.

@imphil - thanks for the info and quick response.
My question then is: Is there a proper method to do the same thing with VCS as is done in the Verilator flow described here?
What I'm trying to do: generate the top-level design files and firmware, and simulate system-level use cases with VCS in a manner similar to the Verilator flow.
I did this before (last year) with previous iterations of the design.
So I'm not referring to steps needed to run the simulation, or (necessarily) UVM for DV, only building the simulation binary using VCS with the eventual goal of using DPI to interface with the sim. From what I can tell from the (collective) docs, a way to do this would be akin to fusesoc --cores-root . run --flag=fileset_top --target=sim --setup --build lowrisc:systems:top_earlgrey_verilator.
There is no "getting started with VCS" page yet, as far as I could find, so I extrapolated that adding --tool=vcs and altering the target would do the job. And it (kind of) does! ...with the exception of the incorrect +incdir+ lines in the filelist and missing -CC -I and -LDFLAGS -lstdc++ args in the Makefile (modulo independent (afaiks) HDL issues).

@imphil - thanks for the info and quick response.
My question then is: Is there a proper method to do the same thing with VCS as is done in the Verilator flow described here?
What I'm trying to do: generate the top-level design files and firmware, and simulate system-level use cases with VCS in a manner similar to the Verilator flow.
I did this before (last year) with previous iterations of the design.
So I'm not referring to steps needed to run the simulation, or (necessarily) UVM for DV, only building the simulation binary using VCS with the eventual goal of using DPI to interface with the sim. From what I can tell from the (collective) docs, a way to do this would be akin to fusesoc --cores-root . run --flag=fileset_top --target=sim --setup --build lowrisc:systems:top_earlgrey_verilator.
There is no "getting started with VCS" page yet, as far as I could find, so I extrapolated that adding --tool=vcs and altering the target would do the job. And it (kind of) does! ...with the exception of the incorrect +incdir+ lines in the filelist and missing -CC -I and -LDFLAGS -lstdc++ args in the Makefile (modulo independent (afaiks) HDL issues).

What you are trying to do is something we have never tried. We run DV simulations with VCS using util/dvsim/dvsim.py, a regression tool that ships with the repo.
I am not unsure what you are trying to achieve. But IIUC you want to stop at the simv generation step? You can still do that using dvsim. Try running the following:
  ./util/dvsim/dvsim.py hw/top_earlgrey/dv/chip_sim_cfg.hjson -i chip_uart_tx_rx --build-only

This will not run any tests, only generate the simulation executable. Note that the top level design used is slightly different (lowrisc:system:top_earlgrey_asic) than the one used by Verilator. The UVM testbench around it does not have the same DPI-C constructs as Verilator either, because we have UVM agents in place instead of those.
To run the full regression:
  ./util/dvsim/dvsim.py hw/top_earlgrey/dv/chip_sim_cfg.hjson -i all

To find individual tests to run, try passing --list tests.
Note that even though it is a UVM based testbench, for the most part, we are doing integration testing (system level usecases) natively with Ibex running a bare-metal SW image (same as Verilator). We run most of the same tests as Verilator. A small subset of DV tests bypass the CPU and directly run UVM test sequences instead. We use those UVM agents to send and receive data over the SoC interfaces (UART, SPI, GPIO etc) instead of the DPI-C constructs.
The HJSon file supplied to dvsim describes the test entries (most of them require SW image to be compiled). The SW image to be compiled for a test is sequenced by dvsim itself, so meson / ninja does not need to be invoked separately to build the firmware. Each SW image may be paired with a UVM test sequence which sends stimulus / performs checks from the testbench side, based on the SW test's needs.
Let me know if this works for you.

What you are trying to do is something we have never tried. We run DV simulations with VCS using util/dvsim/dvsim.py, a regression tool that ships with the repo.
I am not unsure what you are trying to achieve. But IIUC you want to stop at the simv generation step? You can still do that using dvsim. Try running the following:
  ./util/dvsim/dvsim.py hw/top_earlgrey/dv/chip_sim_cfg.hjson -i chip_uart_tx_rx --build-only

This will not run any tests, only generate the simulation executable. Note that the top level design used is slightly different (lowrisc:system:top_earlgrey_asic) than the one used by Verilator. The UVM testbench around it does not have the same DPI-C constructs as Verilator either, because we have UVM agents in place instead of those.
To run the full regression:
  ./util/dvsim/dvsim.py hw/top_earlgrey/dv/chip_sim_cfg.hjson -i all

To find individual tests to run, try passing --list tests.
Note that even though it is a UVM based testbench, for the most part, we are doing integration testing (system level usecases) natively with Ibex running a bare-metal SW image (same as Verilator). We run most of the same tests as Verilator. A small subset of DV tests bypass the CPU and directly run UVM test sequences instead. We use those UVM agents to send and receive data over the SoC interfaces (UART, SPI, GPIO etc) instead of the DPI-C constructs.
The HJSon file supplied to dvsim describes the test entries (most of them require SW image to be compiled). The SW image to be compiled for a test is sequenced by dvsim itself, so meson / ninja does not need to be invoked separately to build the firmware. Each SW image may be paired with a UVM test sequence which sends stimulus / performs checks from the testbench side, based on the SW test's needs.
Let me know if this works for you.


Razor light bulb - Fully qualify "generic" components does the right thing

Ideally we probably shouldn't be showing this at all or we should be stripping out the genericness.

In TypeAccessibilityCodeActionProvider we're taking the Roslyn suggestion for fully qualifying the type name, but the rules for Razor and C# are different, and we're not dealing with that. We need to strip generic arguments at the least.

We're taking the suggestion here in ComponentAccessibilityCodeActionProvider


sample: usb: shell: add keybord harness string

Add keybord harness string to sample.yaml file.

How do you run the test with this fix? When I run ./scripts/twister -T samples/subsys/usb/shell -p nrf52840dk_nrf52840 --device-testing --device-serial /dev/ttyACM0 -v --inline-logs it simply skips the test.

How do you run the test with this fix? When I run ./scripts/twister -T samples/subsys/usb/shell -p nrf52840dk_nrf52840 --device-testing --device-serial /dev/ttyACM0 -v --inline-logs it simply skips the test.

It is not a test, it is a sample, but yes, see also https://docs.zephyrproject.org/latest/develop/test/twister.html#test-runner-twister

How do you run the test with this fix? When I run ./scripts/twister -T samples/subsys/usb/shell -p nrf52840dk_nrf52840 --device-testing --device-serial /dev/ttyACM0 -v --inline-logs it simply skips the test.

It is not a test, it is a sample, but yes, see also https://docs.zephyrproject.org/latest/develop/test/twister.html#test-runner-twister

Could you please be a bit more specific? The documentation mentions The test configuration has defined a harness but you don’t have it or haven’t set it up., but I cannot see how to setup the kyeboard harness and https://github.com/zephyrproject-rtos/zephyr/discussions/46553 is unanswered.

This simply skips the test instead of actually solving the issue. The problem seems to be in twister not in the shell sample.
If I add ser.write(b'\n') before
https://github.com/zephyrproject-rtos/zephyr/blob/da2305081242b61067784a9f0c034f0105bc31a8/scripts/pylib/twister/twisterlib/handlers.py#L376
then twister still fails despite the log containing a lot of lines.

Where did I claim it is solving an issue? Whatever you found out about twister, string harness keyboard belongs in the sample.yml just like in any other shell sample.


Deprecate strategy "pagination"

The strategy code should be on the Score API cuz we still need pagination there, but we should remove it as a strategy.

[ ] Make it impossible to select this strategy in space settings on UI and hub
as mentioned here https://github.com/snapshot-labs/snapshot-strategies/issues/688
[ ] Once we do the announcement about this, we can remove this strategy

This issue can be assigned to me. I'm currently working on a fix.

PR snapshot-labs/snapshot#3229 removes the pagination strategy from the UI. Technically someone can still update their space strategy setting by calling the hub api, but I feel like removing the strategy from the UI is sufficient enough. However, I can still make changes to the snapshot-hub if you disagree with my assessment.

@bonustrack just wanted to confirm with you that this bounty has been completed.

I need to use the "pagination" strategy.  If I cannot use it, then what do I do instead?

Hey @bonustrack, can we close this issue?

@zzuziak We should have a way to announce "pagination" and "multichain" strategies will be deprecated. to give some time for everyone to move to normal strategies then we can remove them


fix memory leak when gquic50 decrypt packet

same memory leak issue in gquic50 with #426 , which may cause continuous memory growth.
another suspicious memory leak when packet_in->pi_data is own.

@litespeedtech I think this PR can be merged, please check it again, thanks!


move type imports to sanity instead of @sanity/types

Just finishing a big ol' refactor to update our CMS to v3 and I noticed that all sanity types seem to be able to come from the sanity package directly. And further in my investigation I noticed that they are even extending their own @sanity/types definitions in order to avoid cyclical dependancies? The effect is that the @sanity/types ends up incomplete if used directly.
I was going to put up a PR in sanity to fix their types, but in my hunting, I found where they are doing their declaration merging. So, I went back to this repo, and replaced all the imports  from @sanity/types with sanity and it started picking up the correctly augmented types!
The tests pass, and my code is informed 🤷 Is there anything I'm missing or any other way to help out?

I thought I might be doing this from the wrong place... can you move it from dependencies into devDependencies and peerDependencies? I don't want to be installing sanity for anyone and creating any duplicate installations.

Oh, good idea! change made

Oh, i added sanity to the readme install, too, since it is now a peer dep and won't work without it. (might cause confusion if someone is trying to use this in a helper lib or something but forgot to install sanity)


A non necessary notebook

new_notebook.ipynb is not necessary.

@hombit @rachel3834 @jsobeck  What do you think about deleting this file?

I agree and would recommend deletion.

Thank you for your comment. I deleted the notebook and closing the isseu.


[Enhancement] In ORC reader, add partition and non-exist column after eval conjuncts

Why I'm doing:
Add partition column in a heavy operation in data lake, because most partition columns are string.

If the user's predicates have greater selectivity, the cost of adding the partition column is unnecessary.
What I'm doing:
Add partition column after eval conjuncts.
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

@Mergify backport branch-3.2 branch-3.1


Add TypeScript extensions to EXTENSION_RE

Closes #13

@fardjad LGTM

@rochdev Great! Thanks.
So I think this issue can be closed by releasing a new version of this library and updating the dependencies of dd-trace-js?


Issue #90: Hack to force restart docker after salt call.

This is fundaentally bad bootstrap script from kube-up. It starts docker/kubelet in the same time when docker graph storage is configured and all salt configurations are updated. There is always
race among these initialization steps.
Kube-up calls salt and doesn't restart docker afterwards on AWS and instead let docker health checker to detect and restart. Because of race, docker health checker sometimes misses this too. Docker daemon ended up with old generic options rather than ones provided by salt. Docker daemon responded to docker info and docker ps call but wouldn't start any containers as it's told to use device mapper as graph storage but underlying storage is overlay2.
The correct fix would be using systemd in right way to define dependencies among OS booting, network config, device mounting, graph storage, all salt configurations, and finally docker and kubelet startup.
In the mean time, this seems to be a simple hack to guarantee docker options specified in salt does take effect.

Updated pull request.
This is because we use old kube up logic to bootstrap 1.6 kubernetes. With 1.4 kubernetes code, docker is stopped after graph storage is configured. It is restarted by salt call or docker health checker. There was still a race if docker health checker beats salt call but that was unlikely in practice. With 1.6 kubernetes, kube
There are two possible options:

Force restart docker after salt call. This is less reliable in theory as we couldn't detect it if this bounce fails. But it's a bit cleaner as it doesn't require parsing docker info output. And it always bounce docker daemon which is likely more predictable. And service bounce failure should be a rare event.

Let docker health checker search for docker daemon arguments and restart if it's incorrect. This works better as it'll be called repeatedly. But I don't like the parsing part.

The correct fix would be using systemd in right way to define dependencies among OS booting, network config, device mounting, graph storage, all salt configurations, and finally docker and kubelet startup.
In the mean time, this seems to be a simple hack to guarantee docker options specified in salt does take effect.


Phuc thanh

Expected Behavior

Current Behavior

Screenshots (if appropriate):
Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Franz Version used:
Operating System and version:

Please use the issue template to describe your problem with all the necessary information.
https://raw.githubusercontent.com/meetfranz/franz/master/.github/ISSUE_TEMPLATE.md


Use tempfile to create new files

Using uuid to generate random file name is not a good idea. Use mkstmep instead.

The same applied to the work directory creation


Recursive field in decodable struct

I want to deserialize json to struct which has a field with any json instance:
#[derive(RustcDecodable, RustcEncodable)]
pub struct TestStruct  {
    data_int: u8,
    data_str: String,
    data_map: BTreeMap<String, Json>,
}

But compiler fails with:
src\worker.rs:16:10: 16:24 error: the trait `rustc_serialize::serialize::Decodable` is not implemented for the type `rustc_serialize::json::Json` [E0277]

If I use definite type like String instead of Json program compiles well.
Why is it impossible to decode to a resursive struct? I see the Decodable not implemented, but maybe there is meaning to make a PR to fix it? I can to spend some time to do it.

There error here is actually quite illuminating! The Decodable trait is indeed not defined for Json, and Decodable is only implemented for BTreeMap if the value also implements Decodable. This implementation unfortunately can't exist due to the design of this library, but using a concrete struct should work!

Thank you for explanation! This is a useful feature for the next reimplementation of this lib.


allow for helm check to always pass.

default will be to use helm if available but the override env var will allow the check to pass if helm is not present by not running helm

wait... is helm override supposed to be "true" to allow it to continue.. and "false" if you care about the version?


Move "Searching for..." python install logging to verbose

I think we should just show the progress bars here personally (by default).

I'm referring to the top part here:
❯ uv python install
warning: `uv python install` is experimental and may change without warning
Searching for Python versions matching: Python 3.12.1
Searching for Python versions matching: Python 3.11.7
Searching for Python versions matching: Python 3.10.13
Searching for Python versions matching: Python 3.9.18
Searching for Python versions matching: Python 3.8.18
Searching for Python versions matching: Python 3.8.12
Installed 6 versions in 4.06s
 + cpython-3.8.12-macos-aarch64-none
 + cpython-3.8.18-macos-aarch64-none
 + cpython-3.9.18-macos-aarch64-none
 + cpython-3.10.13-macos-aarch64-none
 + cpython-3.11.7-macos-aarch64-none
 + cpython-3.12.1-macos-aarch64-none

I guess we print

Searching for Python versions matching: Python 3.8.12
Found existing installation for Python 3.8.12: cpython-3.8.12-macos-aarch64-none

I think we could just show the latter one, though I think both of these are basically debug logs and we should have some other indicator more consistent with the rest of our CLI that we found an existing version?

Hmm, we generally don't really show anything (e.g. in uv pip install) if a package is already satisfied.

I think the "what satisfied this request" is a bit more complicated with Python versions than packages, but that is a fair point.
Also, it's a bit different in that we might read the versions from a .python-version(s) file and if we only display what we need to install how will you know which versions were checked?


Get also return keys values

@GDownes @chrisbarbour  any chance to review this PR ?

@GDownes @chrisbarbour any chance to review this PR ?

Sorry for the delay, thats merged now. Thank you for the push


[FLINK-4604] Add support for standard deviation/variance

add rule for reduce standard deviation/variance functions
Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

[x] General

The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message (including the JIRA id)

[x] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[x] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

Hello, I updated PR for calcite 1.12.

Thanks for the update @ex00. I will go through your changes and merge this tomorrow.

Thanks @twalthr!


Document update - examples directory

The installation instructions regarding finding the kivy-examples directory is incorrect. For linux it is under /usr/share, for Windows under the python installation directory/share.
I don't know if this applies to all OS, pkg mgrs, python versions. It does apply to the latest Ubuntu and python 2.7, and windows 10, python 2.7, 3.4.
The windows install instructions tell us to open a command line. When I do that I am in c:\users\bgailer, and proceed with installation. When i come to
python share\kivy-examples\demo\showcase\main.py there is an assumption that I am in a specific directory. It takes a bit of experimentation to discover that it is the python install directory. That needs to be explicit.
I will take a stab at updating the documentation. This is my first time doing anything with a code repository, so patience please.

Related: #5121


Avoid displaying numbers with hyperlink at bibliography section

At each reference in the section bibliography, there are numbers with hyperlink, which when clicked, will lead readers to where the reference is cited in the thesis.
However, I do not want these numbers to appear in my thesis. May I know what changes should I made?

Please delete "backref=page" at line 171 of 'Thesis.cls'.

Thanks, that solves my problem.


Verification of user's group fails in LDAP backend.

Hi Folks, @m4dcoder
after setting up LDAP auth backend, I found, that ldap binding works fine, it finds the user to be authenticated, but, when it comes to the group validation, it fails. After some debugging,
I get the following result:
2016-05-30 17:09:24,507 41870672 INFO app [-] st2auth.app app created. 2016-05-30 17:09:39,828 56682928 ERROR ldap_backend [-] Query"(|(&(objectClass=*)(|(member=uid=laszlo,ou=people,dc=office,dc=ams-ix,dc=net)(uniqueMember=uid=laszlo,ou=people,dc=office,dc=ams-ix,dc=net))))" None 2016-05-30 17:09:39,828 56682928 ERROR ldap_backend [-] Unable to verify membership for user "laszlo". None 2016-05-30 17:09:39,829 56682928 AUDIT auth [-] Invalid credentials provided (remote_addr='<IP_ADDRESS>',auth_backend='LDAPAuthenticationBackend')
The "Query" in the output is added by me.
However, the following ldapsearch command works fine:
ldapsearch -H ldaps://ldap.ams-ix.net -x -b ou=groups,dc=office,dc=ams-ix,dc=net -D uid=st2_auth,ou=applications,dc=office,dc=ams-ix,dc=net -W '(cn=noc)'
and it returns the members of 'noc' group.
The related part of st2.conf is:
backend_kwargs = {"bind_dn": "uid=st2_auth,ou=applications,dc=office,dc=ams-ix,dc=net", "bind_password": "********", "base_ou": "dc=office,dc=ams-ix,dc=net", "group_dns": ["cn=noc,ou=groups,dc=office,dc=ams-ix,dc=net"], "host": "ldap.ams-ix.net", "port": 636, "use_ssl": true, "cacert": "/etc/ssl/st2/ldap.crt"}
Additionally, if I comment out the related try - except block from this:
/opt/stackstorm/st2/lib/python2.7/site-packages/st2auth_enterprise_ldap_backend/ldap_backend.py file, everything works fine, and the user gets authenticated.
Please let me know, if you need any more information.
TIA,
Cheers

Thanks for the report.
I'm not exactly sure why it's not working (it looks like a invalid query, but I would need to dig in), but @m4dcoder might have more clue :)
For one, we need to add more debug logging to the backend so it will make things easier to debug:

Log actual connection URL (that's important to see if SSL / TLS is used)
Log information which was retrieved by the server (user groups, etc.)

On a related note, @m4dcoder does it make sense to make membership check optional (the thing which is causing issues here)? I'm not too sure, since I think it's usually a good idea to verify that user is part of some specific group which should have access to StackStorm.

Hey @Kami,
here is what you've asked for:
2016-05-31 14:08:10,965 41907536 INFO __init__ [-] Connecting to database "st2" @ "<IP_ADDRESS>:27017" as user "None". 2016-05-31 14:08:11,020 41907536 INFO app [-] st2auth.app app created. 2016-05-31 14:08:19,620 56826288 ERROR ldap_backend [-] Endpoint: "ldaps://ldap.ams-ix.net:636" None 2016-05-31 14:08:19,620 56826288 ERROR ldap_backend [-] Group DNs: "[u'CN=noc,ou=groups,dc=office,dc=ams-ix,dc=net']" None 2016-05-31 14:08:19,621 56826288 ERROR ldap_backend [-] Executed Query: "(|(&(objectClass=*)(|(member=uid=laszlo,ou=people,dc=office,dc=ams-ix,dc=net)(uniqueMember=uid=laszlo,ou=people,dc=office,dc=ams-ix,dc=net))))" None 2016-05-31 14:08:19,621 56826288 ERROR ldap_backend [-] Result of Query: "[]" None 2016-05-31 14:08:19,621 56826288 ERROR ldap_backend [-] Unable to verify membership for user "laszlo". None 2016-05-31 14:08:19,622 56826288 AUDIT auth [-] Invalid credentials provided (remote_addr='<IP_ADDRESS>',auth_backend='LDAPAuthenticationBackend')
And the code which produced this:
( earlier in the code, I stored endpoint into self, so I can print it later )
`
# Setup connection and options.
protocol = 'ldaps' if self._use_ssl else 'ldap'
endpoint = '%s://%s:%d' % (protocol, self._host, int(self._port))
    # store endpoint in self
    self._endpoint = endpoint

    ...

        # Search if user is member of pre-defined groups.
        try:
            query = '(|(&(objectClass=*)(|(member={0})(uniqueMember={0}))))'.format(user_dn)
            result = connection.search_s(self._base_ou, self._scope, query, [])
            entries = [entry[0] for entry in result if entry[0] is not None] if result else []

            if not list(set(self._group_dns) & set(entries)):
                LOG.exception('Endpoint: "%s"' % self._endpoint )
                LOG.exception('Group DNs: "%s"' % self._group_dns )
                LOG.exception('Executed Query: "%s"' % query )
                LOG.exception('Result of Query: "%s"' % result )
                LOG.exception('Unable to verify membership for user "%s".' % username)
                return False
        except Exception:
            LOG.exception('Unexpected error when querying membership for user "%s".' % username)
            return False

`
Or, if you prefer, just give me some code I should execute with some debug statements.
I think this is my first python code, I ever touched. :)
Cheers

Sorry for butting in.
I've fixed this before for CheckMk, see http://lists.mathias-kettner.de/pipermail/omd-devel/2013-July/000450.html.
It looks like the code is expecting group membership to be implemented in objects with parent classes groupOfNames or groupOfUniqueNames, using attributes member or uniqueMember. We use posixGroup with the memberUid attribute. The memberUid attribute should contain only the username (uid attribute), not a complete DN.
In this case, I think the fix would be:
query = '(|(&(objectClass=*)(|(member={0})(uniqueMember={0})(memberUid={1})))'.format(user_dn, username)

@lbogardi Thanks for providing additional details.
@sbakker Thanks for the pointers - we will look into it (and also amend and improve our tests :)).
/cc @m4dcoder

Hi @m4dcoder, @Kami
a possible (working in our case) solution for the problem:
            # Search if user is member of pre-defined groups.
            try:
                query = '(|(&(objectClass=*)(|(memberUid={0})(uniqueMember={0}))))'.format(username)
                result = connection.search_s(self._base_ou, self._scope, query, [])

                entries = [entry[0] for entry in result if entry[0] is not None] if result else []
                members = entry[1]['memberUid']

                if not list(set(members) & set([username])):
                    LOG.exception('Unable to verify membership for user "%s".' % username)
                    return False
            except Exception:
                LOG.exception('Unexpected error when querying membership for user "%s".' % username)
                return False

as it was pointed out by @sbakker, the uniqueMember filter might be unneccessary, too.
Hope it helps,
Cheers

Fwiw, I'm in favor of not requiring but supporting group membership restriction. When I first got started with St2 Enterprise, we didn't yet have an A/D group for it created. Since I had to specify a group, I ended up just piggybacking on another group for a while.
PS hi @sbakker! surprised to see a fellow check_mk user here! I've got a super-basic check_mk PR pending right now if you're interested. :)

Ooh, check_mk to St2, nice! But yeah, we went from a badly maintained Nagios setup to Zabbix (shudder), before giving check_mk a try. It's pretty cool and it's not too difficult to write custom checks.


Missing Localization Strings

Hi Dave, Unintended(Birdwing) found a few strings that weren't in the localization file.
Critical Injury and Old Age in serverSapien.lua are hard-coded.

fixed b21


Removes hammer and hammer_plug deps and their code references

Resolves Issue https://github.com/firezone/firezone/issues/1216

@ssaunderss thank you! Can you please remove mentions of rate limits from the docs?
This is the best kind of PRs where we just remove stuff :).

Pull Request Test Coverage Report for Build 7deb3a927fe3d6534a208bff5b9d85dad37cb98f-PR-1230

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 66.092%

Totals

Change from base Build 38c41feb67c270efbe511db271ec3d9a44f79ea8:
0.0%

Covered Lines:
1189

Relevant Lines:
1799

💛  - Coveralls

@ssaunderss thank you! Can you please remove mentions of rate limits from the docs?

@AndrewDryga I only found one place in the docs that mentioned the in house rate limiter, everything else looks like its nginx related to me. Let me know if I missed anything!


Update team section in readme

Codecov Report

Merging #128 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #128   +/-   ##
=======================================
  Coverage   53.39%   53.39%
=======================================
  Files          55       55
  Lines        2358     2358
  Branches      253      253
=======================================
  Hits         1259     1259
  Misses       1025     1025
  Partials       74       74

Continue to review full report at Codecov.

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

@Shekharrajak @lunayach Can you please approve this?


profile views stats

Description
The stats/graphs of profile views are not displaying properly when hovering over them.
Screenshots
https://user-images.githubusercontent.com/100037271/220819264-f0f96649-fdbc-430d-b066-ad6efdaf0a90.mp4
Additional information
No response

Done 🎉

Thank you @abdulsamad


Add contrastLevel parameter to ColorScheme.fromSeed

This PR is to add a parameter contrastLevel to ColorScheme.fromSeed so that we can construct high contrast ColorScheme.
https://github.com/flutter/flutter/assets/36861262/c609c996-5dfe-4c6c-800c-349a99de4256
Related to https://github.com/flutter/flutter/issues/149683
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] I followed the [breaking change policy] and added [Data Driven Fixes] where supported.
[x] All existing and new tests are passing.

Once this is land, will add the high-contrast ColorScheme to the a11y-assessments app:) @chunhtai

LGTM, is there another pr to update the a11y app as well?

Yes! Will create another PR to add the high-contrast color scheme to the a11y assessment app.


Add flexbox to div enclosing circuit boxes

Describe the bug
Currently the image and header of boxes are separate( as shown by border) which makes shadow effect cover only the image part. Also, this makes it difficult to apply any common styling to the box.
Expected behavior
Enclose image and header under a common div so that any changes made to the box will apply to the whole box. Eg: Shadow effect on hover.
Solution
Use flex-box property as it aligns every box perfectly and helps in responsiveness too.
Screenshots

Are you working on this issue? (Yes/No)
For GCI students under medium issue.

The image and header are enclosed inside a div. The box-shadow can be given to the outside div.

Though the circuits section is already pretty responsive, rewriting it using flexbox might be a good beginner task. Is it possible to edit the issue for the same?

Sure

Can I work on this?

Sure

Can You please explain me what needs to be done here @ShreyaPrasad1209 ?
Like we should add display : flex ; in the css or anything extra?

@ShreyaPrasad1209 any updates?
Like we should add display : flex ; in the css or anything else?


[BUG] Test that query result shapes are correct in invariants

Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

Discovered that there is an issue with PersistentLocalHnswSegment where query_vectors can return results with lengths different than n_results. This PR implements a test both displays and catches this issue.
Fixes off by one issue in hnsw/BF merge logic in both single node and distributed

Test plan
How are these changes tested?

Added a test for the breaking case that fails on main, fix shows test passes
[x] Tests pass locally with pytest for python, yarn test for js, cargo test for rust

LGTM! Counting is the hardest problem...


Feedback after first test

Disclaimers:

I only tested until the ./ink run spec command, which is failing.**
I was not able to test ink in docker as you did not push the image to the registry and I did not take the time to build the image myself

Overall feedback

I love how the tool is packaged.
I find that the outputs of the commands are very verbose, in the philosophy of simplifying connector development, I think we should reduce the amount of output sent to sdtout and keep the bare minimum to understand what's going on at a high level.
The tool relies a lot on subprocesses called via python (docker/git/gpg): I'm prone to make "ink in docker" the default runtime to avoid a dependency management hell and control the versions of the required tooling.
I think this is a great prototype to iterate on if we want to make it our default interface for building connectors. The only drawback I find in this approach is, by moving away from the monorepo, user might lose the feeling of being part of an open source project they forked. It might reduce the incentive to contribute. I think we can mitigate this by providing guidance / encouraging messages (or even dedicated a command) to make the connector open source by opening a PR on the monorepo.

Installation

I’d suggest installing the CLI (./ink) globally. Do you find it more convenient to have the wrapper ./ink script within the connector project?
The packaging / install logic is very neat.  I think we could take inspiration from it for octavia-cli.
Especially for picking python/docker execution environment. Is it a common script packaging pattern I could read more about, or is it something you came up with?
I realize that calling ./ink init actually installs the tool 🤣

./ink init

Why is initialize_project calling patch_connector?  Shouldn't this function be called only by generate_connector?
The output is verbose due to git clone output being sent to stdout. Wdyt about reducing the amount of output to the strict minimum for understanding what's going on at a high level?

./ink generate

--type should be an argument and not an option as its required
We could use a git python client instead of calling git via a subprocess to clone the monorepo
Any strong reason to not run patch_connector before moving its file to the root of the project dir?
Could the generate command also call install?
What do you think of running a git init at the end of generate and creating a first commit with the generated code?
I think that generate does not check that init was run properly. I run once generate before init and got an error saying airbyte.yaml file was missing.
While patching connector you force the python version to be 3.9.0. I do not have this one installed which is making ./ink run fail. Did you patch this temporarily for development convenience on your side?

./ink install

The output of generate is really verbose (but this is ./generate.sh fault)
At first sight, the naming of the command looked a bit awkward to me as I was not sure to what install refers to. This is why I suggest running the install of deps on generate .But we'd still need to expose a command to install requirements if the developer changes them.

./ink run spec
I could not run this command:
FileNotFoundError: [Errno 2] No such file or directory: '/Users/augustin/workspace/source-imr/source_imr/./source_imr/imr.yaml'
I think there is a path manipulation problem somewhere, but I'm not sure if it's related to ink or the declarative source project.

I find that the outputs of the commands are very verbose, in the philosophy of simplifying connector development, I think we should reduce the amount of output sent to sdtout and keep the bare minimum to understand what's going on at a high level.
making it quiet

The tool relies a lot on subprocesses called via python (docker/git/gpg): I'm prone to make "ink in docker" the default runtime to avoid a dependency management hell and control the versions of the required tooling.
agreed but ultimately people will develop on a local env and not on docker so they still need to have a proper version of python installed. We can make the tool docker only, at least for now we can pick one or the other.

I think this is a great prototype to iterate on if we want to make it our default interface for building connectors. The only drawback I find in this approach is, by moving away from the monorepo, user might lose the feeling of being part of an open source project they forked. It might reduce the incentive to contribute. I think we can mitigate this by providing guidance / encouraging messages (or even dedicated a command) to make the connector open source by opening a PR on the monorepo.
Great idea. Monorep was great initially but righ now, especially with how many connectors we have, it is really becoming a bottle neck to contributions. Now I have a "publish" command that is not doing anything, I left it as a next step to decide what publish should mean in the context of Airbyte.

Installation

I’d suggest installing the CLI (./ink) globally. Do you find it more convenient to have the wrapper ./ink script within the connector project?
Much faster onboarding, no dependency on anything or a package manager.  (should we use brew, or others?) It makes it very self-contained with a time to implementation very short.

The packaging / install logic is very neat. I think we could take inspiration from it for octavia-cli. Especially for picking python/docker execution environment. Is it a common script packaging pattern I could read more about, or is it something you came up with?
Come up with it :)

I realize that calling ./ink init actually installs the tool 🤣
./ink init

Why is initialize_project calling patch_connector? Shouldn't this function be called only by generate_connector?
I was using it to play with migration. Removing it.

The output is verbose due to git clone output being sent to stdout. Wdyt about reducing the amount of output to the strict minimum for understanding what's going on at a high level?
Made it quiet

./ink generate

--type should be an argument and not an option as its required
I can't because I want to limit the choice of generators. At least it fails so you know which options you're missing

We could use a git python client instead of calling git via a subprocess to clone the monorepo
What value do we get ?

Any strong reason to not run patch_connector before moving its file to the root of the project dir?
No sure I understand.

Could the generate command also call install?
I can :) done

What do you think of running a git init at the end of generate and creating a first commit with the generated code?
I want to keep the code simple, let wait and see if we should do it

I think that generate does not check that init was run properly. I run once generate before init and got an error saying airbyte.yaml file was missing.
Fixed.

While patching connector you force the python version to be 3.9.0. I do not have this one installed which is making ./ink run fail. Did you patch this temporarily for development convenience on your side?
Yes, to be clear that is one of the weakness of ink right now. I don't know how to manage python.

./ink install

The output of generate is really verbose (but this is ./generate.sh fault)
I can do it at a later time

At first sight, the naming of the command looked a bit awkward to me as I was not sure to what install refers to. This is why I suggest running the install of deps on generate .But we'd still need to expose a command to install requirements if the developer changes them.
You need to have install for when you clone an existing repo. It should be the first command your run after you clone so you have all your deps installed.

./ink run spec

I could not run this command:
FileNotFoundError: [Errno 2] No such file or directory: '/Users/augustin/workspace/source-imr/source_imr/./source_imr/imr.yaml'
I think there is a path manipulation problem somewhere, but I'm not sure if it's related to ink or the declarative source project.
current problem with the cdk. just remove one of the source_imr, unrelated to ink

for the image version, i used airbyte instead of airbytehq


Add pytest marker(s) for regridding (to include/exclude) tests

clisops version: *
Python version: *
Operating System: *

Description
Could use pytest markers for dealing with tests that use xesmf. Maybe it would look like:

At top of module:import pytest
pytestmark = pytest.mark.regrid

See: https://docs.pytest.org/en/6.2.x/example/markers.html#marking-whole-classes-or-modules
Then the test rules could be:

pyXY: python -m pytest -m "not regrid" tests
conda-xesmf: python -m pytest tests (i.e. all)

This approach would avoid needing to individually add decorators...but, running locally would require remembering whether to run with the tag, or not...???

@agstephens Is this still relevant?
At the moment we use in the module
# Try importing xesmf and set to None if not found at correct version
# If set to None, the `require_module` decorator will throw an exception
XESMF_MINIMUM_VERSION = "0.6.2"
try:
    import xesmf as xe

    if parse_version(xe.__version__) < parse_version(XESMF_MINIMUM_VERSION):
        raise ValueError()
except Exception:
    xe = None

and in the tests
from clisops.core.regrid import XESMF_MINIMUM_VERSION, xe

XESMF_IMPORT_MSG = (
    f"xESMF >= {XESMF_MINIMUM_VERSION} is needed for regridding functionalities."
)

@pytest.mark.skipif(xe is None, reason=XESMF_IMPORT_MSG)
def test123():
  [...]

Hi @sol1105 , I will close this. Thanks


[BUG] Fix the href anchor tags in the footer section to link to the correct pages & sections.

Description
(Description about the issue)

In the footer section of the home page, when clicked on the "Home" link currently not direct users to the intended "Home" section. Similarly with the "Contact" link.

In the footer section of the privacy-policy page, all the links except "Terms & Conditions" and "privacy-policy" link not directs users to the indented sections & page.

In the footer section of the Terms & Conditions page, all the links except "privacy-policy" and "Terms & Conditions" link not directs users to the indented sections & page.

In the footer section of the Licensing page, all the links except "privacy-policy" and "Terms & Conditions" link not directs users to the indented sections & page.

Use Case
(Use cases and benefits)
Fixing link issues across pages ensures users can easily navigate to all relevant sections and pages, providing a seamless and consistent browsing experience.
Screenshots
(Screenshots of the issue or reference)
https://github.com/user-attachments/assets/8438d62a-bfa8-48c6-a2a7-30a2c561514b

@krittika019 assigned


Allow for installing to target OS/CPU without autodetecting current architecture

Feature request
What are you trying to achieve?
This somewhat relates to the issue described here https://github.com/lovell/sharp/issues/3904. We are building sharp to deploy in an AWS lambda environment with some limitations around the size of the deployment artifact.
Installing sharp per the AWS lambda docs works fine with the new prebuilt installations. The problem is that it also appears to autodetect the architecture where the install command is being run and install that. So e.g. installing on Mac OS but targeting linux x64 will get the VIPS for both linux and darwin. So then we end up with a huge artifact which messes with our deployment limits.
For now this is okay, we can manually remove the extra darwin files from @img for local development. In CI it's not an issue since we build on a linux architecture, so the files are the same as our target (ie AWS Lambda)
In brief, ideally there would be a way to suppress the check for the current archiecture and simply build to exactly the platform I specify.  So npm install --cpu=x64 --os=linux sharp build to linux 64 only and includes only those binaries, regardless of where I run it.
NOTE: we're on npm 9 so I my installation command looks like:
npm i @img/sharp-linux-x64 @img/sharp-libvips-linux-x64 sharp -f

I'm not sure if using that new platform targeting syntax for npm v10 would change things, but just wanted to be clear about what we're using.
When you searched for similar feature requests, what did you find that might be related?
Checked PRs, issues and docs.
What would you expect the API to look like?
In all honestly I'd expect the default behavior to not install extra files when I specify an architecture.
What alternatives have you considered?

Please provide sample image(s) that help explain this feature

Did you see https://sharp.pixelplumbing.com/install#cross-platform ?

Thanks @lovell . I did see this and this is what I was following.
To be clear this works in that it successfully installs the binary for the architecture I select.
But it also installs the binary for the architecture of the machine I'm running the install on, and there seems to be no way to suppress this. We have to manually clean up node_modules/@img to remove the unneeded binaries.

Thanks, I think I need to update the AWS Lambda section to simply point mostly to the cross-platform section as I've realised directly installing the @img/* scoped packages is probably a bad long-term idea e.g. versions may not be pinned.
As you've seen, from sharp v0.33.x onwards, your choice of package manager and its features/bugs is now even more important. From what I can tell of the state of npm, I am currently recommending people use pnpm or yarn v3+ if they know that an installation tree and/or lockfile must support multiple platforms.

Updated via commit https://github.com/lovell/sharp/commit/cd5cf7ce2d7294faeacd0f9536e874bbea950ee7

I hope this information helped. Please feel free to re-open with more details if further assistance is required.
For anyone else coming here, please see https://sharp.pixelplumbing.com/install#cross-platform for more information specific to your choice of package manager.


#142 Add username already taken prompt

The user is still able to refresh the page and get the previous username he had, overcoming this whole check.
Also, the requests to the api are direct and not throttled, so easy to spam many requests to the server.

No longer needed.


Support chaincode deployment with private data for chaincode_ops

This PR supports chaincode deployment with private data for chaincode_ops.

The current implementation internally uses some protobuf handlers based on source code in Fabric Operations Console (FOC) to encode/decode private data collections definitions.
NOTE: This PR contains minimal integration tests of the feature, but more strict tests should be added for various formats of private collections in the future.

@shimos I tried to fix alerts found by LGTM.com and pushed the revised PR. Could you please review the PR?