Cannot run tests

I'm not sure if there is anything wrong with my local setup or if this is a bug: when I run "tox -e py27" on master I get the following error:

ERROR: invocation failed, logfile: /home/chiniforooshan/github/chiniforooshan/cloudbridge/.tox/py27/log/py27-1.log
ERROR: actionid=py27
msg=getenv
cmdargs=[local('/home/chiniforooshan/github/chiniforooshan/cloudbridge/.tox/py27/bin/pip'), 'install', '--pre', '-rrequirements.txt', 'coverage']
env={...}
Could not open requirements file: [Errno 2] No such file or directory: 'requirements.txt'
Storing debug log for failure in /home/chiniforooshan/github/chiniforooshan/cloudbridge/.tox/py27/tmp/pseudo-home/.pip/pip.log
ERROR: could not install deps [-rrequirements.txt, coverage]
___________________________________________________________________________________ summary ____________________________________________________________________________________
ERROR:   py27: could not install deps [-rrequirements.txt, coverage]

cat .tox/py27/tmp/pseudo-home/.pip/pip.log

Could not open requirements file: [Errno 2] No such file or directory: 'requirements.txt'
Exception information:
Traceback (most recent call last):
File "/home/chiniforooshan/github/chiniforooshan/cloudbridge/.tox/py27/local/lib/python2.7/site-packages/pip/basecommand.py", line 122, in main
status = self.run(options, args)
File "/home/chiniforooshan/github/chiniforooshan/cloudbridge/.tox/py27/local/lib/python2.7/site-packages/pip/commands/install.py", line 262, in run
for req in parse_requirements(filename, finder=finder, options=options, session=session):
File "/home/chiniforooshan/github/chiniforooshan/cloudbridge/.tox/py27/local/lib/python2.7/site-packages/pip/req.py", line 1546, in parse_requirements
session=session,
File "/home/chiniforooshan/github/chiniforooshan/cloudbridge/.tox/py27/local/lib/python2.7/site-packages/pip/download.py", line 278, in get_file_content
raise InstallationError('Could not open requirements file: %s' % str(e))
InstallationError: Could not open requirements file: [Errno 2] No such file or directory: 'requirements.txt'

ls

CHANGELOG.rst  cloudbridge  cloudbridge.egg-info  docs  LICENSE  README.rst  requirements.txt setup.py  temp  test  tox.ini

Hmm - I just did a fresh clone of the repo and tests got under way, so it might be something with the setup. What system are you on?
Here's the exact set of commands I ran:
mkdir cb && cd cb
virtualenv -p /usr/bin/python2.7 .vnv
source .vnv/bin/activate
git clone git@github.com:gvlproject/cloudbridge.git && cd cloudbridge
pip install -e .[dev]
tox -e py27

At this point tests started running (but without additional credentials setup, most of them failed).

Running in a virtual environment + installing python2.7-dev fixed it. Thanks!

Great! Let us know if you have any more questions.


[move-only] Make sure that we insert compensating destroys for phi values

rdar://104726000

@swift-ci smoke test

Windows test failure was unrelated


Add APIs for Design Reviews

This PR adds API functions for the Design Review (Pull Requests in Gitea-speak) endpoints for AllSpice Hub. Note: I haven't implemented the endpoints to get/create reviews etc, as I don't think that's high enough priority for CI and automation scripts.

Do we have any docs regarding pagination for the API and how users should consume it?

Right now, there is no pagination - if you use get_design_reviews you will get all design reviews, which is why there's an optional state filter available. This can stress the server if there's a lot of design reviews, and adding more filter options and more control over which pages you fetch is being tracked in #21 because it's more general.


Display facts for aliases in *BSD

interfaces.rb - aliases on *BSD are iterated over up to their actual count - slightly modified Michelle Sullivan's version
util/ip.rb - added get_alias_no method to return alias count; method get_interface_value modified by Michelle Sullivan to allow obtaining values for specific aliases

Originated here:
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=190796

Did this ever end up in the main code base?

I would say no


Describe table endpoint

Method: GET
Endpoint: /describe/DATABASE/SCHEMA/TABLE
SQL to Postgres adapter:
SELECT *
FROM information_schema.columns
WHERE table_schema = 'SCHEMA'
AND table_name   = 'TABLE'

Return in JSON (by example):
{
  "table_catalog":"DATABASE",
  "table_schema":"SCHEMA",
  "table_name":"TABLE",
  "column_name":"code",
  "ordinal_position":1,
  "column_default":null,
  "is_nullable":"NO",
  "data_type":"integer",
  "character_maximum_length":null,
  "character_octet_length":null,
  "numeric_precision":32,
  "numeric_precision_radix":2,
  "numeric_scale":0,
  "datetime_precision":null,
  "interval_type":null,
  "interval_precision":null,
  "character_set_catalog":null,
  "character_set_schema":null,
  "character_set_name":null,
  "collation_catalog":null,
  "collation_schema":null,
  "collation_name":null,
  "domain_catalog":null,
  "domain_schema":null,
  "domain_name":null,
  "udt_catalog":"DATABASE",
  "udt_schema":"pg_catalog",
  "udt_name":"int4",
  "scope_catalog":null,
  "scope_schema":null,
  "scope_name":null,
  "maximum_cardinality":null,
  "dtd_identifier":"1",
  "is_self_referencing":"NO",
  "is_identity":"NO",
  "identity_generation":null,
  "identity_start":null,
  "identity_increment":null,
  "identity_maximum":null,
  "identity_minimum":null,
  "identity_cycle":"NO",
  "is_generated":"NEVER",
  "generation_expression":null,
  "is_updatable":"YES"
}

+1 for idea and -1 do use "information_schema".
I worry because "information_schema" implementation in PostgreSQL have performance issues mostly in database and with a high number of objects.
And I'm not totally convinced if we need return all this information. Some database have their own implementation of DESCRIBE command (i.e MySQL) an de maybe we need to define our return interfaces for our describe endpoint instead of simple return a query result.

MySQL describe fields:

Field
Type
Null
Key
Default
Extra

fixed in #410


[feature request] add technical info of added video

how about allow to include quality, languages, subtitles, size and location to message? idk if possible.

It is possible but why would you need all that extra details? I'm using radarr and sonarr so all that extra info is almost always the same.

to have it on telegram without need aditional things :)

These features isn't necessary in my opinion but I could look into making them optional.


Add support for semantic manifest files

It would be great if this library supported semantic_manifest parsing.

@MiConnell Please feel free to re-open the issue. It would be hard to support the semantic manifest artifact in dbt-artifacts-parser as of now, since the JSON schema isn't publicly available.
https://github.com/yu-iskw/dbt-artifacts-parser/pull/106#issuecomment-2327930002


[Errno 13] Permission denied: '/usr/local/lib/python2.7/dist-packages/test-easy-install-3065.write-test'

Hello,
Can anybody help me with this procedure?
Create a new Google Compute Engine:
gcloud compute instances create datastore-test --zone europe-west1-b --scopes compute-rw,datastore,userinfo-email
Use the web console
Open the instance using "Open in browser window".
Run this command:
easy_install --upgrade google-api-python-client
Results with Error:
error: can't create or remove files in install directory
The following error occurred while trying to add or remove files in the
installation directory:
[Errno 13] Permission denied: '/usr/local/lib/python2.7/dist-packages/test-easy-install-3065.write-test'
The installation directory you specified (via --install-dir, --prefix, or
the distutils default setting) was:
/usr/local/lib/python2.7/dist-packages/
Perhaps your account does not have write access to this directory?  If the
installation directory is a system-owned directory, you may need to sign in
as the administrator or "root" account.  If you do not have administrative
access to this machine, you may wish to choose a different installation
directory, preferably one that is listed in your PYTHONPATH environment
variable.
For information on other options, you may wish to consult the
documentation at:
https://pythonhosted.org/setuptools/easy_install.html
Please make the appropriate changes for your system and try again.

I think you may want to use sudo as follows:
sudo easy_install --upgrade google-api-python-client
or use pip and create a virtualenv so that you can install packages as a normal user.

sudo python setup.py install

After 'sudo python setup.py install', Where to get the build package to run?

I have meet the same problem, and I have resolved it by uinsg python setup.py install --user, you can try it.


chore: footer missalignment on mobile screens

Description
Removed the padding-x so as to have greater space on mobile screens (updated version #501 ) :

Additional Information

[X] I have read the contributing docs (if this is your first contribution)
[X] This is not a duplicate of any existing pull request

Related Issues
fixes #497
Your ENS/address: shivbhonde.eth

Pushed some minor tweaks to solve the footer on very small devices :
https://github.com/scaffold-eth/scaffold-eth-2/assets/80153681/ac970e3b-9a9d-421b-a592-c03c74b5e9db
Also solved the packages/nextjs/pages/index.tsx on Home pages was not wrapping on smaller devices and was breaking out screen


Use C++17 style static variable syntax

This is the recommended way to do this since C++17. It also fixes a puzzling linker error if building with something other than SCons.

if building with something other than SCons.

Which is it really something to consider? The code is built for SCons

There's many cases like this, for example in ObjectDB

There's many cases like this, for example in ObjectDB

I wonder if his linker error crops up for the other cases as well. :/

If this should be done I think it should be done properly

If this should be done I think it should be done properly

I think I agree. I was hastily under the impression that our code style involved using the newer syntax.

I think the linker error would be a sign of the buildsystem not having properly rebuilt all the objects that need to be rebuilt. Rebuilding from scratch might solve it. I don't see why the current syntax wouldn't work, and it's indeed used in several other places.
This specific one was added in #81844 merged recently.

I fixed the ones I could find with grepping. There were quite lot more of them than one would have expected.
This change is useful just on its own, because shows the default value in the header file rather than you having to go look it up in the corresponding cpp file.

I think the linker error would be a sign of the buildsystem not having properly rebuilt all the objects that need to be rebuilt.

No, that is not it. It is a complicated way in which compiler settings work with PIE in a way that I don't properly understand myself.

This change is useful just on its own, because shows the default value in the header file rather than you having to go look it up in the corresponding cpp file.

On one hand yes, on another no, as it now makes changing the value affect the inheritance chain

You're using spaces for indentation everywhere, you need to use tabs

Still not complete, several of the touched files still have entries left in the previous style

This took so long that at some point I got fed up. I'll fix the rest only after there is a decision to actually merge this. I don't want to waste hours of my time on something that might be rejected outright.

Understandable, but the change would need to be verified to actually work correctly so it's a bit complicated that way

There are comments in this thread that this should not be merged even if it works, because the old one is good enough. First a decision needs to be made that this will be merged if it works. Until that is decided, putting more effort into this is pointless.

I still don't understand how the original code would break compilation.
There's a lot of misunderstanding around buildsystems when treating them as black boxes, when at the end of the day all they do is call the compiler for each file with a list of options.
What option are you using in your Meson PR that differs from the SCons setup?
I'm not against modernizing usage (and indeed be careful around thread local stuff, that's touchy), but I need to know what we're fixing.

Without this change linking fails with this:
/usr/bin/ld: godot.p/main_main.cpp.o: warning: relocation against `_ZN22GDScriptLanguageServer13port_overrideE' in read-only section `.text'
/usr/bin/ld: godot.p/main_main.cpp.o: in function `Main::setup(char const*, int, char**, bool)':
/home/jpakkane/projects/build-godot-Desktop-debug/../godot/main/main.cpp:1534: undefined reference to `GDScriptLanguageServer::port_override'
/usr/bin/ld: warning: creating DT_TEXTREL in a PIE
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.

Guessing this has something to do with how the toolchain wants to make some things position independent by default but not others.

The toolchain here should be GCC and GNU ld, it shouldn't depend whether it's called by SCons or Meson/Ninja. So what I need to know is what you're configuring differently in Meson/Ninja that exposes this issue, so we can assess whether it's an actual bug in Godot or not.
So far to me this exposes a bug in the Meson/Ninja config which either removes or adds an option that Godot isn't meant to be compiled with.
If meson can generate a compilation database, it would be useful to compare with the one generated by SCons (with scons compiledb=yes).

I agree, you just should change Meson compiler options. It's not a Meson or source code issue, just Meson configuration.

The reason for the original failure turned out to be a bad #define setup. I have fixed that in my own branch.


Move all tests to a separate top-level folder

https://www.pyopensci.org/python-package-guide/package-structure-code/python-package-structure.html
Required for #244

Addressed in 6a0d3c2


Support miniscript

Depends on #694 Previous attempt is in #684
This is a somewhat primitive version of Miniscript, which does not support taking the xpub as its internal key representation. So it is closer to an Ivy rather than an output descriptor.
Integration with the output descriptor is not impossible, but I think we should wait for bitcoin core to support it.

Closing since the spec for Miniscript has changed.
a new attempt is in #764


Unable to connect to remote images server in 2.15

Hi,
It seems that the following commit might have broken something for me with remote images:
https://github.com/lxc/lxd/commit/efdb5b8120484e7d91c935f830bf9538bc5c8b47
I have a remote server that I typically add with the following:
lxc remote add lxc-images lxc-images.somecorp.com --public
And it's added as expected:
| lxc-images      | https://lxc-images.somecorp.com:8443 | lxd           | YES    | NO     |
Init command:
lxc init lxc-images:centos7.3-custom centos-73 -p default -p docker  -c security.privileged=true
Works fine under 2.14, but under 2.15 results in the error message as per the commit, 'The source server isn't listening on the network'. Is there some sort of new way that remotes need to be set up in 2.15?
Thanks!

Doh, no, I think the new client code is just trying to be a bit too clever with public remotes. I'll send a fix.

I'm also adding a test for this, we somehow don't have one for --public...

Fixed in my bugfix branch and has a teest to make sure it doesn't show up again.

Thanks for the quick response!

On a very slightly related note, it's apparently not possible to downgrade via the ubuntu-lxc ppa as apt-cache always only shows the most recent, even though the previous version is available in the pool. I guess I can switch to the lxd-git-master archive as a workaround instead once that gets merged and built.

If you are to downgrade, you'd need to pick up the 2.14 version of lxd-client and only downgrade that. Never downgrade LXD (lxd package) itself unless one of the developers tell you it's safe to do as database and on-disk storage changes somewhat frequently and will not cope with downgrading.
In this instance, as soon as my branch is merged, I'll be uploading cherry-pick of all the fixes to the Ubuntu package, so once automated testing is done (which can easily take 3-4 hours), a new version of the package will be available with the fix.

If you need an immediate fix. You can download https://dl.stgraber.org/lxd-3464 (sha256: ef929bf889eddbc97da615dbc1f6d75fac8e65e767b92348d95e3fe1e839b79d) which is a built of the 2.15 client with my fix applied.


Unit test attack handler

Overview
Introduces the full unit testing for attack logic that was absent before. Unit tests now ends the game similar to a real-world scenario.
This will be also useful in case of rematch testing

As soon you merge this, I'll update rematch branch and push it. After that I'll write unit tests for rematch


Automatically generate website resources during Netlify build

This automatically generates our website resources files, and removes them from the repository so that we don't need to check them in anymore. When netlify runs our build, it now runs npm install in the website directory and installs the necessary tools (such as postcss, the tools are listed in the website/package.json file) so that when hugo runs a build, that the resources can be generated.

Shouldn't the resources directory be completely gone now? Why are there still some files there in the repo?

I've removed the new resources that were added yesterday in #218.


what is the best way to manage session across multiple users ?

Bot Info

SDK Platform: Node JS
Active Channels: webchat
Deployment Environment: Azure App Service

Issue Description
i want to manage session across multiple user ( for the segregation of chat between users who are interacting with bot concurrently)
Code Example
is privateConversationData is enough to handle this ?

In your session object you have 3 databags to store state data  userData, conversationData, and privateConversationData.  These are the recommended way to store data.


Sample repo fails to build

cos-toolkit version:
doesn't matter
CPU architecture, OS, and Version:

Describe the bug
CI pipelines are red https://github.com/rancher-sandbox/cos-toolkit-sample-repo/actions/runs/1031547724, the sample repo was never updated after we removed the sample selinux policies from the tree.
To Reproduce
clone the sample repo and run cos-build
Expected behavior
To work
Logs

Additional context

I've opened https://github.com/rancher-sandbox/cos-toolkit-sample-repo/pull/10 with a fix but it fails, I'm trying to see why


Running into discord embed limit of 1024.

I am running into this error occasionally when the bot writes a lot of text. There should be a way to either limit the output to max or create a new message when max is reached.
Traceback (most recent call last):
File "C:\Users\John\miniconda3\envs\textgen\lib\site-packages\discord\app_commands\commands.py", line 842, in _do_call
return await self.callback(interaction, **params)  # type: ignore
File "C:\Users\John\Downloads\one-click-installers-oobabooga-windows\text-generation-webui\bot.py", line 173, in reply
await ll_gen(ctx, queues)
File "C:\Users\John\Downloads\one-click-installers-oobabooga-windows\text-generation-webui\bot.py", line 116, in ll_gen
await msg.edit(embed=reply_embed)
File "C:\Users\John\miniconda3\envs\textgen\lib\site-packages\discord\webhook\async.py", line 841, in edit
return await self.state.webhook.edit_message(
File "C:\Users\John\miniconda3\envs\textgen\lib\site-packages\discord\webhook\async.py", line 1974, in edit_message
data = await adapter.edit_webhook_message(
File "C:\Users\John\miniconda3\envs\textgen\lib\site-packages\discord\webhook\async.py", line 221, in request
raise HTTPException(response, data)
discord.errors.HTTPException: 400 Bad Request (error code: 50035): Invalid Form Body
In embeds.0.fields.1.value: Must be 1024 or fewer in length.
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\Users\John\miniconda3\envs\textgen\lib\site-packages\discord\ext\commands\hybrid.py", line 438, in _invoke_with_namespace
value = await self._do_call(ctx, ctx.kwargs)  # type: ignore
File "C:\Users\John\miniconda3\envs\textgen\lib\site-packages\discord\app_commands\commands.py", line 860, in _do_call
raise CommandInvokeError(self, e) from e
discord.app_commands.errors.CommandInvokeError: Command 'reply' raised an exception: HTTPException: 400 Bad Request (error code: 50035): Invalid Form Body
In embeds.0.fields.1.value: Must be 1024 or fewer in length.
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\Users\John\miniconda3\envs\textgen\lib\site-packages\discord\ext\commands\hybrid.py", line 438, in _invoke_with_namespace
value = await self._do_call(ctx, ctx.kwargs)  # type: ignore
File "C:\Users\John\miniconda3\envs\textgen\lib\site-packages\discord\app_commands\commands.py", line 860, in _do_call
raise CommandInvokeError(self, e) from e
discord.ext.commands.errors.HybridCommandError: Hybrid command raised an error: Command 'reply' raised an exception: HTTPException: 400 Bad Request (error code: 50035): Invalid Form Body
In embeds.0.fields.1.value: Must be 1024 or fewer in length.

@keninishna I was able to reproduce and it should be fixed now. Thanks for the issue!


Model improvements

Codecov Report

Merging #638 into metadata-editor will decrease coverage by 0.97%.
The diff coverage is 100%.

@@                 Coverage Diff                 @@
##           metadata-editor     #638      +/-   ##
===================================================
- Coverage            52.94%   51.96%   -0.98%
===================================================
  Files                  258      272      +14
  Lines                 7106     7337     +231
===================================================
+ Hits                  3762     3813      +51
- Misses                3344     3524     +180

Impacted Files
Coverage Δ

src/article/metadata/EntityEditor.js
75% <ø> (+73.33%)
:arrow_up:

src/article/metadata/MetadataPackage.js
100% <ø> (ø)
:arrow_up:

src/dar/PersistedDocumentArchive.js
54.2% <100%> (-3.21%)
:arrow_down:

src/article/ArticlePanel.js
76.47% <100%> (+2.94%)
:arrow_up:

src/article/converter/r2t/ConvertArticleMeta.js
88.29% <100%> (+0.65%)
:arrow_up:

src/article/converter/r2t/EntityConverters.js
95.57% <100%> (ø)
:arrow_up:

src/article/editor/FigComponent.js
0% <0%> (-100%)
:arrow_down:

src/article/editor/CaptionComponent.js
0% <0%> (-100%)
:arrow_down:

src/shared/ToolPrompt.js
0% <0%> (-100%)
:arrow_down:

src/article/editor/TitleGroupComponent.js
0% <0%> (-100%)
:arrow_down:

... and 96 more

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 13c0a30...9397d03. Read the comment docs.


sway: fix "Mapping is not an object" warning

Fixes #3763.
Also a little code simplications while we're at it.

Thanks!

thanks @c4rlo !


How I can make page button navigation with the url instead of setState and change history?

Have you saw this on the readme? Perhaps could help.
[as](https://github.com/thijssmudde/react-headless-pagination#as)
Type: React.ReactElement Default:

Provide a custom React Element, such as a next/link component.


Implemented default voters controller

Added default voters API endpoints in /api/v1/accelerator/voters

#1780  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tommylau523 and the rest of your teammates on  Graphite

I'm not sure we need this. If we only expose the default voters list via the admin UI, nothing will consume these endpoints, right?

I was working with the assumptions that the admin UI will consume these APIs. Maybe that means that I have a gap of understanding of what the admin UI actually is.

I'm not sure we need this. If we only expose the default voters list via the admin UI, nothing will consume these endpoints, right?

I was working with the assumptions that the admin UI will consume these APIs. Maybe that means that I have a gap of understanding of what the admin UI actually is.

Another consideration is that when we update the list of default voters, we may want to propagate those changes to existing projects and their cohort phases. That is reflected in the downstream PR.


Bad markdown formatting in documentation (datepicker props table)

Pipe character for the union & enum types requires escaping.
https://github.com/Hacker0x01/react-datepicker/blob/master/docs/datepicker.md

Fixed in https://github.com/Hacker0x01/react-datepicker/pull/902

My bad, master branch is still showing the old docs though =/


feat: product details page

Hi,
Here's a PR for a product detail page. This is a simple product detail page, I'm planning to add another one with a slider image in the future.

Cheers

@all-contributors please add @populationtwo for code

@all-contributors please add @populationtwo for code


dynamic icon ?

is it possible to display a dynamic icon like a loader for example?

Or something like scrolling text?

You can use md.tray.setImage to update the tray icon. I'd imagine you can combine that with some nifty setTimeout logic to do something interesting.

It's been a while but I was also looking for the solution and haven't found anything, so here's something I did:
setIcon (type) {
  if (type == 'active'){
    this.tray.setImage(settings.get('icons.tray.active'))
  } else if (type == 'default'){
    this.tray.setImage(settings.get('icons.tray.default'))
  }
}

var trayAnimation = setInterval(() => {
  setTimeout(() => {
    this.parent.setIcon('active')
  }, 300)
  setTimeout(() => {
    this.parent.setIcon('default')
  }, 600)
}, 600)

...and when you want to stop the animation:
clearInterval(trayAnimation)

Is anyone aware of a way (or workaround) to display some dynamic text?

Does someone want to add a small example app in the examples/ folder?

I'm looking to be able to use an svg, and change the colors. Does anyone know if that is possible?

Have a look at this example @andriyor https://github.com/maxogden/menubar/tree/master/examples/icon-animation


Avoid changing file modified time if unmodified

The mogrify command line tool is meant to overwrite existing files 1. As such, even if you do mogrify -verbose image.jpg it will touch image.jpg and update its modification time. Would be ideal if this could be avoided.

Potential solution: use option -write /dev/null for verbose/1
But what about Windows?

Urgh it's to much hustle as usual with Windows :)

Haha yep. I tested on Windows and NUL works, thankfully.


Add cwd option

Fixes #5
Usage:
// resolve with ['folder/**'] if root/path/folder is a directory
await glob('folder', { cwd: 'root/path' })

Thanks, and sorry for the late response!


Fix #1429: Adjust logic to handle coordinates in decreasing order

... As well as longitudes in the 0-360 range.

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #1429

[ ] Tests for the changes have been added (for bug fixes / features)

[ ] (If applicable) Documentation has been added / updated (for bug fixes / features)

[x] CHANGES.rst has been updated (with summary of main changes)

[x] Link to issue (:issue:number) and pull request (:pull:number) has been added

What kind of change does this PR introduce?

Handle decreasing pressure level coordinates, longitudes in the 0-360 range.

Does this PR introduce a breaking change?
no
Other information:

@Thomasjkeel  Fixes to broaden the support for different input grids.


Support cc.xml

Generate cc.xml feeds like Jenkins, Travis, CruiseControl, etc. This will allow tools like http://ccmenu.org/ to function.

Implemented in 448ca9cc2a9ee21304fd0f27544e5769fa825fa0 available at {baseApiUrl}/branches/state/cc.xml


Only import hcsshim package in windows files

Reduce packages that are compiled in linux binaries.

Do you worry about the size of the binaries, the security implication of the unneeded dependencies or something else?

Do you worry about the size of the binaries, the security implication of the unneeded dependencies or something else?

No. It's just about code clean up, and it let me maintain less dependency in downstream distribution.
If it's not preferred, I'm fine too. (I haven't investigated the failed tests yet..)

SGTM


Python Relative Path

VSCode Version: 1.47.3
OS Version: macOS 10.15.5

Steps to Reproduce:

Run a Python script and it cannot find files in same directory when called with relative path.
Able to fix by using 'cd' in VSCode's terminal to navigate to the same working directory as my .py file.

Essentially when you hit run in VSCode, it runs all paths relative to where the Terminal thinks it is, not where the .py file is.

Does this issue occur when all extensions are disabled?: Yes/No
Yes

Please provide a screenshot or a gif illustrating the problem (attaching images won't work via email, you need to go to github.com). See this wiki page for more general info on how to report actionable terminal issues.
/label "needs more info" integrated-terminal


RenderComponent does not wait for all threads to complete before returning.

Describe the bug or question
When a component under test has async methods, the RenderComponent method does not wait for all threads to complete before returning.
Example:
BlazorApp1.zip
Component under test
FetchData
With this test:
FetchDataRendersCorrectly
Results in this output:
SaveAsync is executed after the test method completes.
Expected behavior:
RenderComponent should wait until all threads have completed before returning control to the test method.
Version info:

Beta 5.1
.NET Core/Blazor 3.1:

Since it is the Submit() call that triggers an async operation, which causes a render, your best option is currently to use the WaitForNextRender helper method in the framework. E.g.
WaitForNextRender(() => cut.Find("form").Submit());

Check out WikiSearch example or the FetchData example in the Wiki for more examples of how to use WaitForNextRender:

https://github.com/egil/razor-components-testing-library/wiki/C%23-test-examples#testing-components-that-use-on-ijsruntime
https://github.com/egil/razor-components-testing-library/wiki/C%23-test-examples#testing-components-with-injected-dependencies

Note, that the WaitForNextRender will only wait for one render. If your async logic causes multiple renders before the expected changes are visible, it will not be enough. I am actively working on a solution for that, see this PR if you are interested in following along: https://github.com/egil/razor-components-testing-library/pull/59

The PR you are working on might fix this.
I tried WaitForNextRender.  It did not work.  I think the Test method is blocking the async task from running.

Tests run in the xunit synchronization context, and the renderer, and thus the async code you have, run in the renderer's dispatcher's context. That said, when you do Task.Run, it is running in an entirely new thread, and I am not sure which sync context it gets attached to. I generally do not recommend using Task.Run like that in your components.
Take a look at the samples I liked to above to see if these do not match your scenario. Otherwise, yes, you would have to wait for the next release of the library, or roll your own custom async waiter.

I only used Task.Run to simulate what happens with a SaveAsync OnValidSubmit="SaveAsync".
Any way you make SaveAsync an aync Task will recreate the problem.
The xunit syncronization context has to complete before the SaveAsync will be dispatched.
I think the RenderComponent needs to be awaited as to not block the SaveAsync from being dispatched.
Will the issue you are working on the the next version address this?
Thanks,
Jeff

You can see the issue by placing a break point in SaveAsync and on the assert so you can see which order methods are called.

Most likely. From your small sample, there are certainly some things I would do differently. Like not injecting a concrete class in your FetchData, but instead an IWeatherForcastService, that you can mock. That way you can control when and during the lifecylce of the test, the forecasts are returned to the component.
Take a look at the sample TodoList test in the Samples folder, it shows this approach.

I don't thing my question is about what I can do different.  It is about the threading model being used in the library.  Please keep in mind I cannot share the actual code being tested so I put together small examples that demonstrate an issue.
When a form is submitted and OnValidSubmit calls an aysnc Task, the task will not be dispatched until the xunit thread completes.  I beleive this is because the thread is being blocked by not having an await on the calling thread.  If there was a RenderComponentAsync that did an await, the child threads would run and the RenderComponent would wait until the child thread completes.
This is demonstrated by setting a break point in the SaveAsync method that is called OnValidSubmit and another break point on the assert so that you can see the SaveAsync method isn't called until the test thread is complete.

Hmm I an not sure I can replicate the issue. RenderComponent should block until the first render of the component under test is completed. When I download your sample and just run the test, everything is green. What missing?
C:\Users\egh\Downloads\BlazorApp1 (2)\BlazorApp1\BlazorApp1> ls

    Directory: C:\Users\egh\Downloads\BlazorApp1 (2)\BlazorApp1\BlazorApp1

Mode                LastWriteTime         Length Name
----                -------------         ------ ----
d-----       18-02-2020     17:18                BlazorApp1
d-----       18-02-2020     17:18                ComponentTests
-a----       18-02-2020     17:18           1632 BlazorApp1.sln

C:\Users\egh\Downloads\BlazorApp1 (2)\BlazorApp1\BlazorApp1> dotnet test
Test run for C:\Users\egh\Downloads\BlazorApp1 (2)\BlazorApp1\BlazorApp1\ComponentTests\bin\Debug\netcoreapp3.1\ComponentTests.dll(.NETCoreApp,Version=v3.1)
Microsoft (R) Test Execution Command Line Tool Version 16.3.0
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...

A total of 1 test files matched the specified pattern.

Test Run Successful.
Total tests: 1
     Passed: 1
 Total time: 3,3801 Seconds

If you put break points where I said, you will see that the SaveAsync is not called until after the Assert and the test method is completed.  The SaveAsync should be called after the submit is performed.

When I run your test in your sample (on a dual core, four thread laptop), the first breakpoint it hits, is the one in line 21 in FetchDataTest.cs, then it hits the breakpoint on line 23 in the same class, and then the breakpoint in FetchData on line 81. Is that also what you are seeing?
If I change the test to this, which will be possible in the next version of the library, the breakpoint on Assert.True(cut.Instance.saved) isnt hit before after SaveAsync has completed:
[Fact]
public void FetchDataRendersCorrectly()
{
    Services.AddSingleton<WeatherForecastService>();

    var cut = RenderComponent<FetchData>();

    cut.Find("#Summary_textbox").Change("test");
    cut.Find("form").Submit();
    cut.WaitForState(() => cut.Instance.saved);

    Assert.True(cut.Instance.saved);
}

This still doesnt have anything to do with the RenderComponent method though, as far as I can see.

Your description above of the actions is correct.  I the WaitForState will allow the component thread to run as you stated, then yes that will address the issue.
Any idea when the next version with the WaitForState will be pushed?

Your description above of the actions is correct. I the WaitForState will allow the component thread to run as you stated, then yes that will address the issue.

WaitForState will wait (block) the test thread until the predicate you pass to it returns true (or a timeout is passed). So that may include zero or more renders in the render thread.
WaitForNextRender only waits (block) the test thread for one render on the render thread, or until a timeout is passed. So if your logic requires you to wait for multiple renders, then you cannot use that.

Any idea when the next version with the WaitForState will be pushed?

Hopefully within a week, maybe two.


Running plutus playground

Area
[] Plutus Foundation Related to the GHC plugin, Haskell-to-Plutus compiler, on-chain code
[] Plutus Application Framework Related to the Plutus application backend (PAB), emulator, Plutus libraries
[] Marlowe Related to Marlowe
[x] Other Any other topic (Playgrounds, etc.)
Summary
Trying to run Plutus playground on my Ubuntu 21.04 laptop, I installed nix sh <(curl -L https://nixos.org/nix/install) , cloned the repository ( tested both master and 3746610e536), started the server successfully, however when trying to run the client npm start, it does not find the modules, all of them it seems. I suspect something wrong with path, but I am not sure.
where are the packages located?

Note the server is running without any problem

Is this also happening with npm start? the screenshot it's showing a different command being invoked
I tested 2ae7a0293f0cfcbfb1d1a35ac84097239639a3fe and after entering a nix-shell and running npm start plutus playground is running fine. Make sure the directory is clean

Yes I tested npm start and all modules not found.
I started from scratch last version and I got the following error  (commit fd3a906b2bf5c2932d9d3d0e9366e3decf83a83a)

It does not work  because I am connecting to the machine remotely, running the browser on the machine it self it works.
I guess the client listens on <IP_ADDRESS> not <IP_ADDRESS>.
https://<IP_ADDRESS>:8009/  this works
https://<IP_ADDRESS>:8009/   this does not work (server in the same LAN)

To access the playground client remotely I added  --host <IP_ADDRESS>  line 6 on package.json
"webpack:server": "webpack-cli serve --progress --inline --hot --mode=development --node-env=development --host <IP_ADDRESS>"


dummy commit do not review.

Description

change me!
User-facing documentation

[ ] CHANGELOG is updated OR update is not needed
[ ] documentation PR is created and is linked above OR is not needed

Testing and quality

[ ] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[ ] CI results are inspected

Automated testing

[ ] added unit tests
[ ] added e2e tests
[ ] added regression tests
[ ] added compatibility tests
[ ] modified existing tests

How I validated my change

change me!

/test gke-qa-e2e-tests


Add sites table for PCA

Add a table of representative sites to conduct PCA on datasets, e.g., within the ancestry stage of the large cohort pipeline.

@KatalinaBobowik is this something we're copying once, or regenerating frequently?
I would have thought this section of the references script would be a better model - you can add the HGDP bucket as a source and get CI to copy it over, rather than adding a separate script to do the copy
https://github.com/populationgenomics/references/blob/main/references.py#L197

The service-account here probably does not have read access to the hgdp bucket - because it's not public.

@MattWellie this is just something that will be copied once.
So to clarify, under src I would add where the sites table is copied from (i.e., gs://cpg-hgdp-1kg-main/sites_table/v1-0/pruned_variants.ht/), is that right?

see that it has an attached gsutil cp -r command

Do I need to add gsutil cp -r somewhere? Or add in transfer_cmd=gcs_cp_r?

@illusional I hadn't spotted that - in addition to this change, we need to add references to here in order to run this transfer script?
@KatalinaBobowik based on Michael's comment I'd ignore my suggestion and go with your current approach. Just for reference if it were an accessible bucket this:
Source(
    name='ancestry',
    src='gs://cpg-hgdp-1kg-main/sites_table/v1-0',
    dst='ancestry',
    transfer_cmd=gcs_cp_r,
    files=dict(sites_table='pruned_variants.ht'),
)

would result in the new config section:
[references.ancestry]
sites_table=gs://cpg-common-main/references/ancestry/pruned_variants.ht

Good shout @MattWellie - need that tokens/repository-map change to run this.

Thanks for that suggestion @MattWellie , I've updated it now 👌


fix detection of web workers as node.js runtime

When trying to use a wasm module and its javascript wrapper inside a web worker, the wrapper will call require( "fs" ) and try to load the wasm module from a local file instead of loading it from the server. The wrapper wrongly assumes that the js code is being executed in a node.js runtime.
That is because when executing the wrapper in a web worker in the browser window is undefined.
Please merge this PR to enable using cargo web with web workers.

LGTM! Thanks!


Plugin organization

A couple questions/suggestions related to plugin organization:

Categories or tags?
Some of the plugins don't fit easily into single categories. For instance, a plugin that is both a generator and a tag to interact with that generator. I think tags would work well here.
Two tag "types", plugin type and purpose?
Having a hard time thinking up good names for these, but I think it would be good to have two types of tags, one for the plugin type (e.g., tag, generator, other), and one to group the plugin by purpose (e.g., images, code, asset pipeline, performance).
Collections?
I think where people have more than one plugin in a repo it would be good to list each plugin separately but point to the same url. Thoughts?

Would you mind providing an example of why they might be bundled together and released as one plugin?

jekyll-feed generates a feed, and provides the {% feed_meta %} for linking to that feed.

I'd use categories for the type and tags for the purpose. Is that too inflexible?

:+1:


KOGITO-1355 - Update slave label for jenkins build

@mbiarnes @spolti @ricardozanini kogito-cloud-s2i-images-master build is not there, should we remove this step?

No, it doesn't, looks like the job was removed from jenkins, do you know why it was removed?

@mbiarnes seems like we missed this job from the kogito scripts: https://github.com/kiegroup/kie-jenkins-scripts/blob/master/job-dsls/jobs/kogito.groovy#L312

@mbiarnes updates?

Plus, this node needs to have cekit and its dependencies/configurations in place.

@cristianonicolai apparently we'll not need it anymore, can we close this PR?


AnimatorComponent

@TheCodeCrafter
What do you think of this Pull Request? Have you seen this or tested it, would it help us in anyway ?

Oooh let me check.. I have not seen this before but I'm quite intrigued now...

Closing since this functionality has been re-implemented by @slimbuck


Set up Signet-like miner for warnet

For war-games, we don't want teams to be able to mine their own blocks, which means we need an admin controlled miner. Signet is likely the easiest way to do this, so we will need to:

Run warnet in signet mode
Have a signet miner controlled by the war-games admin
Mine the blocks into each team's namespace (randomly? round robin?)

The idea of randomly mining into each teams namespace is to ensure blocks will come from enemy teams, thus each teams need to maintain connections to malicious peers to hear about new blocks. Otherwise, you could just firewall your nodes from any attacker.

Signet was introduced in 0.21.0 meaning wargames including nodes prior to this version will not support signet at all. I think that is an issue since many exciting vulnerabilities were present before signet.
So, options?

Keep regtest

make a new set of bitcoin node images for every version with rpc generatetoaddress removed
admin has control over a v27 node, the only one in network, with the CPU miner still working

Use Testnet 3

bring a BitAxe or small ASIC to tabconf, use actual PoW to prevent players from mining
means connecting meatspace hardware into the cluster somehow
block 546 is checkpointed, will have to config all tanks to invalidate block after that, start our warnet chain there

😭
Agree we should keep the older images. Keep regtest seems like the easiest path forward in the short term. Instead of modifying the images themselves, can't we just use -rpcwhitelist to allow everything but generatetoaddress? Sure, this looks ugly, but it's a config change which means we can do it just for Tabconf and not need to modify our images.
Using testnet 3 seems cool, but this seems really hard to get ready in the next month.
A third (unholy) option: we back port https://github.com/bitcoin/bitcoin/pull/18267/ to the pre-0.21 images 😬 I took a look at the PR and the bulk of the logic is adding signet.{cpp,h} and then lots of test files. I think this is the best approach long-term and would be happy to take a stab at it. Essentially, we just add the signet files and the necessary consensus changes and leave out the tests to keep the patches small. These old images will never change, so there isn't any risk of needing to maintain the patch (unlike our addrman patch that is applied to new images).
TLDR; try to back port signet to the old images, and if that doesn't work we use the rpc whitelist method with regtest?

This is only needed for this single tabconf exercise. IMO it doesn't seem worth much effort. Just ask folks not to do it.
Another option is to rebuild (all images ZZZZZ) with an additional (bypassable) prohibition of bitcoin-cli *generate* commands, like this: https://github.com/bitcoin-dev-project/warnet/compare/main...willcl-ark:warnet:no-generate

rpcwhitelist was added in 20 or 21 I think.

In that case, seems like our list of "we'll just ask them not to do it" is getting pretty large. If we are going to rebuild old images, I'd prefer just adding signet to the older vulnerable images. @pinheadmz can you keep working on adding a signet miner and only test with images that support signet? I'll work on back porting signet to the vulnerable images since I've already got some other work related to old patched images I'd like to do (e.g., #508 ).
I'm fairly certain backporting signet for the old images is possible, but if you're concerned about it not being possible before Tabconf, I can time box this work to this week and you can start on the signet miner next week after I've patched the images.

It's a bona fide strategy!

Another option is to wrap all the player tanks in a proxy: https://github.com/pinheadmz/rpc-auth-proxy
This proxy has a rpc blocklist. Maybe we can set it up as a service, then it can be applied in wargames but not in other simulations.

or we could block any mining RPCs in the scenario framework?


Fix Iceberg partition tables

This commit includes two fixes:

Use a table scan which loads column stats with each data file.
Otherwise the column stats returned by Iceberg are null.
Make minValues and maxValues in class Partition mutable so that
Partition::updateStats doesn't throw an error.

cc: @electrum @Parth-Brahmbhatt

Umbrella issue: #1324

The code is simple and looks good, but it's hard to tell what was broken/fixed. Can you add a test that fails without this change?

The code is simple and looks good, but it's hard to tell what was broken/fixed. Can you add a test that fails without this change? An integration test in TestIcebergSmoke is probably best.

Sure. I've added such a test in TestIcebergSmoke. I also have to add the support for RowType conversion in TestingPrestoClient so that the test case can query the partitions table which has row type columns.

There were 2 check-style violations before so I updated the PR. I also rebased it upon the latest master so that a failure in Travis is fixed. Now the Travis passes.

Merged, thanks!


js-is-sticky is applied when sibling reaches top of page, not when sticky element actually sticks

I've thrown some of my code into a codepen (sorry if it's a little messy):
https://codepen.io/anon/pen/eVZZEZ
What's happening is that .headnav gets applied with the .js-is-sticky class as soon as its sibling, header, reaches the top of the viewport. The intended result is that the class is applied when .headnav itself reaches the top of the viewport and is sticking.
Any clue why this happens? Thanks.

Used version: 3.0.1

@w-biggs did @DanielRuf's not help? 😊

@w-biggs did @DanielRuf's not help? 😊

I just added the information about which version the issue reporter uses so we can better track this. I thought we have an issue template but it seems we have none (might make debugging easier).

Yeah, I'm using 3.0.1 already - sorry for not specifying.

@w-biggs @DanielRuf  I'm not saying that there is not a bug here but there is an issue with the setup in the codepen. Please review this—mainly position: sticky.
Sorry I won't be able to help here more for the next week/2 weeks.
~Thanks

@yowainwright I see no issue with how the codepen is set up based on the specification found there - what am I missing?

@w-biggs here you go. Sorry for the copy/paste job. This is right from this same link again. 😊 .

"containing block" is the key word. That is a confusing but important difference between position: fixed and position: sticky.

Ah okay, thanks. The spec is confusing - further down the page, it says that it switches when the element itself reaches the threshold (which is the behavior I'm seeing in-browser, but not by this plugin). I wonder if every browser is misinterpreting the spec in this case? Hm.

@w-biggs thanks for your thoughts!
This issue has traversed to a non-stickybits issue so I'm closing it. 😊

For further demos check out other closed issues or this page from the Stickybits wiki.


Skip writing the vmdb.yml.db in test.

Follow up to #6675.
Apparently, there were other places in the miq_server_spec.rb that we also were
saving the configuration.  Instead of also changing these locations to not save,
we should blindly skip saving the vmdb.yml.db in test mode.

Thanks @Fryguy @bdunne for the ideas and help finding the problem.

:+1: LGTM


Exclude admin pages from Browsersync

Hello all,
is it possible to prevent admin pages from auto-reloading?

Just don't access the admin pages via the browsersync server?


Create Docker Hub repo for Maesh adapter

Description
Create Docker Hub repo for Maesh adapter --> https://cloud.docker.com/u/layer5

A new public repository has been created and is currently set to autobuild based on merge to master --> https://cloud.docker.com/u/layer5/repository/docker/layer5/meshery-maesh.
A new issue will need to be opened to track moving away from autobuild by Docker Hub and moving to building by GitHub Actions.


view-backend-exportable-private: Add wl_client_add_destroy_listener in the ViewBackend

Add callback for wl_client_add_destroy_listener in the ViewBackend.This callback set the m_client to null.
Guard the wl_client_flush() calls when m_client is null.

This MR is attempts to address this issue: https://bugs.webkit.org/show_bug.cgi?id=218612
Signed-off-by: Pablo Saavedra<EMAIL_ADDRESS>

This seems to have merge conflicts, so at least a rebase will be needed. I have written a couple of comments as well.

Rebased.

I've cherry-picked the cleanups suggested by @zdobersek. Also, I've verified the unregisterViewBackend() is called when the  WebView is destroyed (ex: g_object_unref()) so It looks like the right place to put the call for the  wl_client_destroy().

Should we squashed all together in just one single commit before merge?

@psaavedra: Ping! Could you please rebase this MR, if it is still relevant? Thanks!


Code intelligence doesn't work for included and referenced files

It seems the code intelligence doesn't work for files using include 'file.as' and also for <mx:Script source="file.as"/>. Some of our projects are using Flex 3 so it's mx:Script.
Is this a known issue? I noticed there was a similar case https://github.com/BowlerHatLLC/vscode-as3mxml/issues/331.

It's a known issue. See #324.
It's better than it used to be, but as you've discovered, it's not perfect. There are probably sections of your included files where code intelligence actually works. It may depend on the size of the file, though. I had some trouble figuring out how certain offsets were supposed to work, so I gave up for now. I may revisit it in the future.

I see, thank you for the explanation! Looking forward to the future!


darwin.stdenv: format with nixfmt-rfc-style

Description of changes
I’m going to be doing a lot of work on the Darwin stdenv. Rather than risk merge conflicts, I’d like to get it reformatted before I start working. I confirmed with nix-instantiate that the resulting hash for the stdenv is the same, so it should not cause rebuilds. This is targeting staging-next because staging-next will merge in a few days. Currently, staging and staging-next have the same base commit.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Commit message typo needs fixing. (And darwin.stdenv: update .git-blame-ignore-revs feels backwards to me, though apparently other people are using that style…)
LGTM otherwise, given that ofborg says 0 rebuilds.

I fixed the typo in the commit message. Once checks are green, I’ll merge.


grsecurity: add GRKERNSEC_DENYUSB option (disabled by default)

This option tells the kernel to ignore plug-in events of USB devices. Useful to protect against attacks with malicious hardware. Currently disabled by default, though.

OK, I see everyone being in favour and the failure doesn't depend on the PR.
If I understand correctly, linuxPackages_grsec_*.klibc is broken...
Merged.


allow configuring the dev host

Allow overriding of localhost for dev host, for use cases where we're running in docker-compose

closing in favour of https://github.com/grafana/pdc-agent/pull/108


Laser frame_id fix

Fix for all CER robots: lidar frame_id was not correct.
Removed / from all frame_id parameters

Merged thanks @randaz81 👍🏻
@violadelbono I've seen you're working out the CER04 branch.
Could you please include these additions as well in your branch?


Sign In & Sign Up Page UI refactoring

I'd like to make the sign in & sign up pages more visually appealing to the users by working on the UI part.

Hi @MonalikaPatnaik can I get issue assigned under GSSoc'23? thanks!

Heyy @tayyab-ilyas you can improve the UI of signup page in accordance to the current UI of sign-in page!
are u willing to work on this? lmk if u cann't!

thanks for assigning! I'll start working

Heyy pls check my comment on your PR!

Heyy pls check my comment on your PR!

I've committed the changes!


Can't push new branch

Hey!
I've created an es6 refactor branch and I'd be happy to share...
Regards,
Peter

Okay.. my bad! I think I was too tired.. I was trying to push directly to your repo.


🐛 Bug Report: Content is not fully visible and will also add up the hamburger icon for managing the state.

📜 Description
while i am opening the default deployed page,
there is the issue with the background text that gets hidden in the left side at 100% resolution and want to align it so that it gets fully visible to the users and also helps in better understanding with the project .

will add up the hamburger also for managing and merging the sidebar into left side .
👟 Reproduction steps
1.Will add up the hamburger icon for merging the sidebar.
2.Will also align up the text in some more lines so that it gets wide and fully visible.
👍 Expected behavior
it should align up the text and will make text looks more good .
👎 Actual Behavior with Screenshots
after aligning up the text it should be looking like .

and will also add up the hamburger for collasping up the navbar to the left side .
💻 Operating system
Windows
What browsers are you seeing the problem on?
Chrome
🤖 What development environment are you experiencing this bug on?
Local dev server
🔒 Did you set the correct environment variables in the right path? List the environment variable names (not values please!)
No response
📃 Provide any additional context for the Bug.
No response
📖 Relevant log output
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🔗 Are you willing to submit PR?
Yes, I am willing to submit a PR!
🧑⚖ Code of Conduct

[X] I agree to follow this project's Code of Conduct

@larinam pls assign me the issuse so that I start working on it and raise up the PR as soon as possible.

@dartpain If this bug needs fixing can u assign me this?

@dartpain As I see, the test environment already has an arrpw icon to close the sidebar, which transforms into the hamburger.

@tenokami okay sure sure so i will update the text style that causes , half text hide behind the sidebar and will upgrade into its good version.


removed duplicated docs in NetworkManager.js

@Meir017 thanks. Can you please sign CLA to make bot happy?

I signed it!


Little suggestion: rename build.py to polly

How about rename build.py to polly?

polly --reconfig --install --config RelWithDebInfo --toolchain osx-10-11-dep-10-10

How about rename build.py to polly?

Yes, I think it make sense. Does it conflicts with some other tools? For example if there will be two scripts build.py and polly.py will it solve problem? Also I can't run script without extension on Windows, so it should be polly.py instead of just polly.
@headupinclouds What do you think?

:+1: I like the suggestion.  I'm also in favor of maintaining both build.py and polly.py initially to allow some time for transition.

Maybe just shell scripts polly.bat and polly which runs polly.py (or
build.py).
David Hirvonen<EMAIL_ADDRESS>周一22:55写道：

[image: :+1:] I like the suggestion. I'm also in favor of maintaining
both build.py and polly.py initially to allow some time for transition.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/ruslo/polly/issues/73#issuecomment-208384812

Maybe just shell scripts polly.bat and polly which runs polly.py (or build.py)

Okay, let it be polly.bat calling polly.py on Windows; polly shell script for calling polly.py on *nix; and build.py call polly.py for backward compatibility.
@xpol Feel free to work on this one and send me pull request.

#75 added.
The maybe we don't needs polly.py. just move it to polly ?

The maybe we don't needs polly.py. just move it to polly ?

Symmetry looks nicely :)


FancyZones forgetting the layout

It's been happening multiple times. I set up a custom layout for my single monitor. Some days I go to put my windows in the zones and I changed it to the default priority grid. Upon opening FancyZones, my custom layouts are gone.

Are you using version 0.35 when you create these?  were the missing layouts created in 0.35?


capture the whole parsed string along side with captured substrings.

(a.! ~ b.! ~c.!).! is a Parser[String], ignoring the inner captures. There should be a way to capture all 4 things like in a regex ((...)(...)(...)).
Apparently "it should be trivial to implement by taking the source code for Capture and making it append to a tuple instead of replacing the innards".
Just recording it here, hopefully I can contribute this some time.

Could you please elaborate on a use-case where this feature is required.

@MartinSenne Sure. We parse natural language, detecting certain expressions within a larger body of text. We do that by making sure the expression adheres to a certain grammar. We then want to extract the text of the whole expressions as well as structured data for it's parts. E.g. "Nov 9th" and "9th of November" should lead to the same structured data, but we need the original text as well.

This is actually pretty easy to do yourself; if someone wants to implement it and paste the 5 lines here that would be great =D

a) This would be good to have as an option.
b) A related option that would be very helpful: store the name of the rule/parser along with the captured data.  This would solve 2 problems:

for a non-trivial parser, it can be difficult to identify which rules/parsers caused which data to be captured.  Having the name of the rule/parser would help you know how to handle the captured data.
currently, the structure is lost for something like the above example, (a.! ~ b.! ~c.!).!, as the entire result is concatenated into a string, and you don't know which parts were a, b, or c.  If instead you try (a.! ~ b.! ~c.!) (without the group .!), when combined with a parent rule, the structure is sometimes lost, for example giving you (a,b,c,d) instead of ((a,b,c),(d))

closing due to inactivity


EmptyDataError: No columns to parse from file

Hi, I am unable to use the following code for my adls gen2 blob file.
import dask.dataframe as dd
from pkg_resources import parse_version
import fsspec
if parse_version(fsspec.version) < parse_version('0.6.2'):
from fsspec.registry import known_implementations
known_implementations['abfs'] = {'class': 'adlfs.AzureBlobFileSystem'}
STORAGE_OPTIONS={'account_name': 'account_key', 'account_key': 'account_key'}
ddf = dd.read_csv('abfs://{container_name}/test.csv', storage_options=STORAGE_OPTIONS)
Below is the error I am getting
EmptyDataError: No columns to parse from file

Hi, I already have a adls gen 2 and i am able to connect to it using
block_blob_service = BlockBlobService(account_name=account_name, account_key=account_key)
block_blob_service.get_blob_to_text(container_name, blob_location)

see also https://github.com/dask/adlfs/issues/48#issuecomment-615545666

@GautamKumarSkyPointCloud try now

Closing - no additional feedback from @GautamKumarSkyPointCloud.  Issues reported as resolved by @raybellwaves.


added fare sort

While booking a bus you would like to see the cheapest options first, so sorting the buses by fare. I have added a sort function in main.py

resolved issue : #209

@DyuthiVivek You need to check the code you are adding, you are writing wrong syntax in python and you are trying to even run the project, so I remove all the labels of this PR.
never do again.


Support Kotlin 1.8.21

Kotlin 1.8.21 is not present on the compatibility chart, and using this kotlin version produces a warning or error.

Thanks. We will fix it.
While you waiting, you can check the RC version of our compiler plugin.
To do this, add these lines to all your gradle modules:
compose {
    kotlinCompilerPlugin.set("org.jetbrains.compose.compiler:compiler:1.4.7-rc01")
    kotlinCompilerPluginArgs.add("suppressKotlinVersionCompatibilityCheck=1.8.21")
}

Would be great to have your release support .22 as well since it's such an exceedingly minor bump (with no impact on a compiler plugins).

JB Compose compiler 1.4.7 out yesterday supports 1.8.21 but does NOT support 1.8.22 😬.

JB Compose compiler 1.4.7 out yesterday supports 1.8.21 but does NOT support 1.8.22 😬.

Yeah, we are waiting for Google JetPack Compose compiler with support for 1.8.22 for now

It's out today as 1.4.8. Would be amazing to get a fast-follow from you all so we can all be up-to-date before 1.9.

@eymar Do we plan to publish new Compose compiler with Kotlin 1.8.22 before next release of Compose Multiplatform?

Do we plan to publish new Compose compiler with Kotlin 1.8.22 before next release of Compose Multiplatform?

It was not planned. Anyway, here it is: https://maven.pkg.jetbrains.space/public/p/compose/dev/org/jetbrains/compose/compiler/compiler/1.4.8/
it requires:
compose {
    kotlinCompilerPlugin.set("1.4.8")
}

cc: @JakeWharton

kotlin 1.8.21 is currently supported with:
compose {
    kotlinCompilerPlugin.set("1.4.7")
}

closing this issue.

Thanks! Will that get synchronized to Maven Central so we can rely on it in places other than Compose UI multiplatform?

Just got the notification. Many thanks!

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


[Feature Request] Support signed integer index for moreh_getitem

moreh_getitem takes a signed integer tensor as the index tensor.

@adifrancescoTT There is no assignee regarding this issue.

@adifrancescoTT There is no assignee regarding this issue.

My mistake. I'm handling this issue. (https://github.com/tenstorrent/tt-metal/pull/8978)
@sangwon-chae @adifrancescoTT


make readiness probe optional

For Teztnets, it's useful to disable this probe: when I start a new
testnet, sometimes I am waiting for other participants to join for the
chain to kick off.
When recreating dailynet or mondaynet, I need to wait for other peers to
connect to me to download the most recent blocks: with a probe, that's
not possible.
Must be explicitly set to false, otherwise by default the probe will be
present. It can only be set at node type level. Since it's processed by
helm and not config-generator, I didn't see a way to make it
configurable at global or instance level.
I will need this to migrate teztnets to the new aws org without disruption.

It is good to add this option. Just a few questions:

when I start a new
testnet, sometimes I am waiting for other participants to join for the
chain to kick off.

If it doesn't have blocks and so is on 0, doesn't it return success?

When recreating dailynet or mondaynet, I need to wait for other peers to
connect to me to download the most recent blocks: with a probe, that's
not possible.

Does this mean that the chain already exists and has blocks? So basically blocks stopped being produced and therefore the chain gets older and probe returns false?

Since it's processed by helm and not config-generator, I didn't see a way to make it
configurable at global or instance level.

Can set option in those places and then mount tezos-config configMap? Then probe looks at NODE_GLOBALS and it's own node options. Can return success msg saying why skipping validation.


Use published artifact for IoT

I know gguuss has signed the CLA - not sure why CLABot is unhappy.


Pointer data resampler (#41118)

PTAL

nit: Can you please add a short description to the PR describing what this does and what it is for? Also, are there any issues associated with this? If so, please link them from the PR description.

I don't mean any review comment or suggestion for modification, but I want to point out the thing I wrote in https://github.com/flutter/flutter/pull/60796#issuecomment-656472296 .
This resampling happens in the PointerData level, while our unit test framework flutter_test injects the driving input in the PointerEvent level. So if this is integrated into the framework in the future in someway (like become a on/off flag in the framework), rather than the current use case as replacing the binding in the app, the instance of PointerDataResampler in the framework wouldn't be able to be tested using WidgetTester for integration test because anything from PointerDataResampler would look like device input rather than test input and will be ignored.

(PR triage) @dreveman @liyuqian Are there still plans for this PR? It has been in-active for some weeks now...

(PR triage) @dreveman @liyuqian Are there still plans for this PR? It has been in-active for some weeks now...

@dreveman told us this Monday that he's working on an updated resampler for PointerEvent instead of PointerData. The new work should be available soon.

PTAL and let me know if I missed any feedback. I'm hoping to have this approved and landed today unless some significant changes are requested. Thanks!


(PUP-3618) Add supporting facts for pkgng provider

To support the Puppet package provider for PkgNG, these facts help
determine if pkg is supported and enabled on a given system.
This was a community effort:
Alex Hornung<EMAIL_ADDRESS>Brice Lopez<EMAIL_ADDRESS>Dennis Weewer<EMAIL_ADDRESS>Jiří Kubíček<EMAIL_ADDRESS>Reinier Schoof<EMAIL_ADDRESS>Ryan Langseth<EMAIL_ADDRESS>Ryan Melcer<EMAIL_ADDRESS>Tim Bishop<EMAIL_ADDRESS>clinta<EMAIL_ADDRESS>
Thanks all!

CLA signed by all contributors.

@xaque208 If this is primarily to enable default provider behavior, I really think we should be putting this logic in the provider and not having it as a fact, especially a default-shipping fact.  Is there something I'm missing here (short of perhaps having to implement additional functionality in provider confines to support your needs) that requires this to be a fact?

Well, I suppose when the provider gets merged into core, some of this will change, but I've been able to do:
if $pkg_enabled and $kernel == 'FreeBSD' {
  Package {
    provider => 'pkgng',
  }
}

The base system, while supporting pkgng, does not enable it by default.  You must still bootstrap the system to get the package manager fully functional on a new system.
I'm fine with moving the logic of detecting enablement in the provider if thats the preferred way to do this.  I didn't really get feedback on how to handle the facts, and I am just now catching up on your messages in the provider PR.

Triage notes: this PR depends on Puppet PR-3276

Yes, this depends on the resolution of Puppet PR 3276 (where I think the recommendation was to not rely on facter extensions).
However, if there is consensus around a facter component to the solution, two bits of logistics:

this would need a FACT ticket for this PR (rather than a PUP ticket)
it would need to be re-targeted at the facter/2.x branch?

Backstory on the latter: https://groups.google.com/forum/#!msg/puppet-dev/tBqHZB9M308/QqdaldMsJlkJ
Thanks for the contribution!

With a slight change I've made this morning to Puppet PR 3276, this should no longer be needed.

Okay thanks for the update @xaque208. Closing.


ActiveJet AMY-208 not being detected.

Information

ratbagd version (ratbagd --version): 0.15
Distribution: openSUSE Leap 15.2
Kernel version (ex. uname -srmo): 5.3.18

Device info (if applicable): N/A
Describe the bug
Device cannot be configured. Piper (0.5.1) GUI says that no devices were detected. Sorry, I don't know what else I could put here.
To Reproduce
Steps to reproduce the behavior:

Connect the mouse via USB.
Open the Piper (0.5.1) GUI.
You will see a message that no devices were found.

Logs
$ ratbagd --verbose=raw
Initializing libratbag
ratbag debug: New device: USB OPTICAL MOUSE
ratbag debug: Using data directory '/usr/share/libratbag'
ratbag debug: No data file found for 18f8:0f99
ratbag debug: New device: USB OPTICAL MOUSE
ratbag debug: Using data directory '/usr/share/libratbag'
ratbag debug: No data file found for 18f8:0f99
ratbag debug: New device: SYN1B7F:00 06CB:CD41
ratbag debug: Using data directory '/usr/share/libratbag'
ratbag debug: No data file found for 06cb:cd41
DBus server ready

Additional context
~ ❯❯❯ lsusb
Bus 004 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub
Bus 003 Device 004: ID 04ca:3016 Lite-On Technology Corp.
Bus 003 Device 003: ID 1c7a:0570 LighTuning Technology Inc.
Bus 003 Device 002: ID 05e3:0608 Genesys Logic, Inc. Hub
Bus 003 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub
Bus 002 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub
Bus 001 Device 003: ID 1bcf:2cb0 Sunplus Innovation Technology Inc.
Bus 001 Device 002: ID 18f8:0f99 [Maxxter] Optical gaming mouse
Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub

lsusb shows the mouse as Maxxter for some reason. Also, the RGB lights on the mouse are constantly cycling through 4 colors (red, blue, pink, purple) and I can't control the via any button on the mouse. It looks like a disco-ball. Sorry, I don't know what kind of info I could give.
Device specification provided by manufacturer: https://productz.com/en/activejet-amy-208/p/3Lbo6
Anyway, thanks for all Your work.

duplicate of #615


Example for usage with nx monorepo workspace (question)

Is there any example how to use sheriff with an nx monorepo workspace? I'd like to use it for only one library. Where have I place the config-file and the eslint rule? Currently I only get errors.
Thank's for your help!

Hi @richardliebmann , I hope that you could make use of the example. I'm going to close this issue. If there are some further questions, create a new issue. Thanks


Unused Parameter when using preinstalled poco-lib

This happens in:
ClickHouse/dbms/src/IO/InterserverWriteBuffer.cpp:
In constructor
‘DB::InterserverWriteBuffer::InterserverWriteBuffer(...)’:
27:28: error: unused parameter ‘send_timeout’ [-Werror=unused-parameter]
const Poco::Timespan & send_timeout,
^~~~~~~~~~~~
28:28: error: unused parameter ‘receive_timeout’ [-Werror=unused-parameter]
const Poco::Timespan & receive_timeout)
^~~~~~~~~~~~~~~
I've copied the macro a little bit below, to remove the parameters from function declaration, and it works then,
I should assume if there are going to be invokes to these function, they should be changed as well. Yet that would not stand as an optimal solution, especially for future implementations.

Maybe there can be some kind of trivial use of these two parameters, to circumvent the state of not being used.

I've changed it to a void cast and it works fine.
#if POCO_CLICKHOUSE_PATCH || POCO_VERSION >= 0x02000000
session.setTimeout(connection_timeout, send_timeout, receive_timeout);
#else
session.setTimeout(connection_timeout);
static_cast  (send_timeout);
static_cast  (receive_timeout);
#endif

Yes, this is correct solution.
Do you want to make a pull request? (Or we can commit it.)

I'll make one. Thx.


Use standard base image and remove unused dependencies in install-cni Dockerfile

Fixes #25693.

@bcheung: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
1e022b146236121a34869deb77e1e0a11d7c5e8a
link
/test release-notes_istio

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

@bcheung: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
1e022b146236121a34869deb77e1e0a11d7c5e8a
link
/test release-notes_istio

lint_istio
1e022b146236121a34869deb77e1e0a11d7c5e8a
link
/test lint_istio

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

Cherrypick to release-1.7?

Thanks @bcheung

/retest

In response to a cherrypick label: new pull request created: #25756


CON-474: repro test and bug fix

Added logic to only add an empty snapshot to the database's context
if that particular snapshot is the last that the database knows
about

@RaghavBabu @javierlores @as577 FYI


Added msg when calling node.error()

Added msg when calling node.error function to be able to capture the error with Catch node

Please merge it!


Fixes file path formatting

Description
Replaces some linux file separators with the system separator
Fixes a typo where the quotes that go around a filename in the // line comments was put inside of the getRelativePath function

I am going to merge this PR and fix Appveyor afterwards.


CLI client doesn't work on Windows

Fixed in 0.1.2: setup.py now uses entry_points for cross-platform compatibility


[BUG]: Using showNotificationControls - title is referenced from mp3 file metadata, not source.metadata.title

Version
6.3.0
What platforms are you having the problem on?
iOS
System Version
react native 0.73.6. iOS 17.5.1
On what device are you experiencing the issue?
Real device
Architecture
Old architecture
What happened?
When playing an mp3 file whilst using showNotificationControls, the controls display the "title" value stored in the mp3 file metadata. The value provided in the source prop is ignored.
If the mp3 file does not contain "title" metadata it works correctly.
iOS has the issue. Android works as expected.
Reproduction
No response
Reproduction
Add a value to the "title" metadata field for a given mp3 file (Apple Music is one app where you can do this).
Then specify the metadata.title value in source.
In the notification controls, notice that the value encoded in the mp3 takes precedence.
const source: ReactVideoSourceProperties = {
      uri: 'file:///example.mp3',
      metadata: {
        title: 'Love Me Do', // if the mp3 file contains "title" metadata, this property is ignored
        artist: 'The Beatles',
      },
    }

return (
  <Video
    source={source}
    showNotificationControls
  />
)

I will look into this next week 👀

Hey @nickmcmillan can you test https://github.com/TheWidlarzGroup/react-native-video/pull/4050 ?

@KrzysztofMoch perfect, thank you 🙏


[ISSUE] Statement Execute API - Get Result Chunk By Index

Description
There is a missing query parameter when calling this API method using the SDK. It is looking for the row_offset in the query parameters.
Reproduction
from databricks.sdk import WorkspaceClient
from databricks.sdk.service.sql import Disposition, Format

w = WorkspaceClient()

warehouse_id = "XXXXXX"

statement = w.statement_execution.execute_statement(
    disposition=Disposition.EXTERNAL_LINKS,
    format=Format.CSV,
    statement=(
        "SELECT * "
        "FROM table"
    ),
    warehouse_id=warehouse_id,
    wait_timeout="50s",
)

for chunk in statement.manifest.chunks:
    result = w.statement_execution.get_statement_result_chunk_n(
        statement.statement_id, chunk.chunk_index
    )

Expected behavior
I should be able to get the details of the next chunk of  results.
Debug Logs
Traceback (most recent call last):
  File "XXXXXXXXXXXXXXX/lib/python3.8/site-packages/starlette/routing.py", line 677, in lifespan
    async with self.lifespan_context(app) as maybe_state:
  File "XXXXXXXXXXXXXXX/lib/python3.8/contextlib.py", line 171, in __aenter__
    return await self.gen.__anext__()
  File "XXXXXXXXXXXXXXXcore/lifespan.py", line 27, in lifespan
    file_path = download_data_from_databricks(config.DATA_WAREHOUSE_NAME)
  File "XXXXXXXXXXXXXXXcore/databricks.py", line 49, in download_data_from_databricks
    result = w.statement_execution.get_statement_result_chunk_n(statement.statement_id, chunk.chunk_index)
  File "XXXXXXXXXXXXXXX/lib/python3.8/site-packages/databricks/sdk/service/sql.py", line 3457, in get_statement_result_chunk_n
    json = self._api.do(
  File "XXXXXXXXXXXXXXX/lib/python3.8/site-packages/databricks/sdk/core.py", line 939, in do
    raise self._make_nicer_error(status_code=response.status_code, **payload) from None
databricks.sdk.core.DatabricksError: Required field is not specified: row_offset.

Other Information
databricks-sdk==0.2.1

Thanks for raising this. I've raised a ticket with the team responsible for this endpoint to correct their API definition.

Same issue here with databricks-sdk==0.3.0
It would be nice to have a fix as only the 1st chunk of data can be retrieved at the moment.
Thanks

workaround:
`import requests
from databricks.sdk import WorkspaceClient
from databricks.sdk.service.sql import Disposition
DBRX_WORKSPACE = 'XXXXX'
DBRX_TOKEN = 'XXXXX'
HEADERS = {"Authorization": f'Bearer {DBRX_TOKEN}'}
warehouse_id = 'XXXXX'
sql_str = "SELECT * FROM table"
w = WorkspaceClient()
r = w.statement_execution.execute_statement(warehouse_id=warehouse_id, statement=sql_str, disposition=Disposition.INLINE)
for chunk in r.manifest.chunks:
r_chunk = requests.get(f'{DBRX_WORKSPACE}/api/2.0/sql/statements/{r.statement_id}/result/chunks/{chunk.chunk_index}?row_offset={str(chunk.row_offset)}', headers=HEADERS)`


Clarify read labels in coverage viz.

Add "loose" to read label and updated on-hover tooltip to clarify that only
unassembled reads were included.
Description

Maybe individual reads?

I checked with Katrina, and loose reads is not an official science term. But she felt that the unknown term will nudge people to look at the docs, whereas aligned reads won't trigger any alarms.
"individual" reads feels somewhat more vague than "loose". "Loose" captures the notion that the read should be connected to something bigger but isn't.


Added Sort an Array folder with README and .C file in it

Description
A comprehensive description about the Pull Request
Reference Issue number #346
346 must refer to the issue related to this PR, visit here
Type of change
Choosing one or more options from the following as per the nature of your Pull request.
NOTE: Check boxes using [x]

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Documentation Update

Checklist:

[x] I have named my files and folder, according to this project's guidelines.
[x] My code follows the style guidelines of this project.
[x] My Pull Request has a descriptive title. (not a vague title like Update index.md)
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] I have created a helpful and easy to understand README.md, with problem description and my name.
[ ] I have included a requirements.txt file (if external libraries are required.)
[x] My changes do not produce any warnings.
[x] I have starred this repository.
[ ] I have added a working sample/screenshot of the script.
[x] I have checked for trailing spaces in file names and none have them.

@shree-create Your code filename has spaces in it. Rename it.

@shree-create Your code filename has spaces in it. Rename it.
yeah I renamed it now


Search History

The search history currently only accepts 6 searches and then stops updating. Would like to see the most recent search replace the oldest search.

Resolved by adding conditional statement at line 53 on script.js file. When the length of the array exceeds more than 6, replace the oldest search using .splice() method and replace with newest search in the array.


Combination sum using Recursion in C++ and Python

Is your feature request related to a problem? Please describe.
The Combination Sum problem is a classic algorithmic problem that involves finding all unique combinations of numbers in a given array that sum up to a specific target value. This problem can be solved using recursion
Describe the solution you'd like
To solve this problem recursively, we can follow these steps:
Define a helper function that performs the recursive backtracking process.
The helper function should take the candidates, target sum, current index, current combination, and a result vector as parameters.
If the target sum becomes 0, add the current combination to the result vector and return.
If the target sum becomes negative or the current index is out of bounds, return.
Otherwise, there are two choices:

Include the current candidate in the combination. Recur with the same index, decrement the target sum by the current candidate,
and add the current candidate to the combination.
Exclude the current candidate from the combination. Recur with the next index and the same target sum.

In the main function, initialize an empty result vector and call the helper function with the appropriate parameters.
Return the result vector containing all the valid combinations.
.

Please assign me this issue in C++ and Python  @Kumar-laxmi

This is an easy issue, so can you implement this in C and Java too

yes i can implement it in c and java as well @Kumar-laxmi

please assign @Kumar-laxmi


Alembic not recognizing models altogether

Hello,
when I put metadata.create_all(engine) in the startup function, it works fine, but when I try to migrate using Alembic, alembic drops the tables created earlier and never recognizes them at all from thereon.
project tree:
├── __init__.py
├── alembic.ini
├── migrations
│   ├── README
│   ├── env.py
│   ├── script.py.mako
│   └── versions
├── pr.db
├── requirements.txt
├── src
│   ├── __init__.py
│   ├── api
│   │   ├── Auth.py
│   │   ├── __init__.py
│   ├── app.py
│   ├── config
│   │   ├── __init__.py
│   │   └── settings.py
│   ├── models
│   │   ├── User.py
│   │   ├── __init__.py
│   ├── pr.db
│   ├── routers.py
│   └── utils
│       ├── Auth.py
│       ├── Models.py
│       ├── __init__.py

Alembic env.py file:
import os
import sys
from logging.config import fileConfig

from alembic import context
from sqlalchemy import create_engine

# add app folder to system path (alternative is running it from parent folder with python -m ...)
myPath = os.path.dirname(os.path.abspath(__file__))
sys.path.insert(0, myPath + '/../../')

# this is the Alembic Config object, which provides
# access to the values within the .ini file in use.
config = context.config

# Interpret the config file for Python logging.
# This line sets up loggers basically.
fileConfig(config.config_file_name)

# add your model's MetaData object here
# for 'autogenerate' support
# from myapp import mymodel
# target_metadata = mymodel.Base.metadata
from fastshop.src.config.settings import metadata

target_metadata = metadata

# other values from the config, defined by the needs of env.py,
# can be acquired:
# my_important_option = config.get_main_option("my_important_option")
# ... etc.

URL = "sqlite:///pr.db"

def run_migrations_offline():
    """Run migrations in 'offline' mode.

    This configures the context with just a URL
    and not an Engine, though an Engine is acceptable
    here as well.  By skipping the Engine creation
    we don't even need a DBAPI to be available.

    Calls to context.execute() here emit the given string to the
    script output.

    """
    context.configure(
        url=URL,
        target_metadata=target_metadata,
        literal_binds=True,
        dialect_opts={"paramstyle": "named"},
        # if you use UUID field set also this param
        # the prefix has to match sqlalchemy import name in alembic
        # that can be set by sqlalchemy_module_prefix option (default 'sa.')
        user_module_prefix='sa.'
    )

    with context.begin_transaction():
        context.run_migrations()

def run_migrations_online():
    """Run migrations in 'online' mode.

    In this scenario we need to create an Engine
    and associate a connection with the context.

    """
    connectable = create_engine(URL)

    with connectable.connect() as connection:
        context.configure(
            connection=connection,
            target_metadata=target_metadata,
            # if you use UUID field set also this param
            # the prefix has to match sqlalchemy import name in alembic
            # that can be set by sqlalchemy_module_prefix option (default 'sa.')
            user_module_prefix='sa.'
        )

        with context.begin_transaction():
            context.run_migrations()

if context.is_offline_mode():
    run_migrations_offline()
else:
    run_migrations_online()

I know that alembic is properly configured because as I said, it does recognize the migrated models and drops the tables that were created with metadata.create_all(engine)!
future migrations don't work, it just keeps dropping after the initial migration and never recognizes future models created!
Any help, please!

I resolved the issue by importing models inside env.py and restructure the directory tree.
Regards 🙂

I resolved the issue by importing models inside env.py and restructure the directory tree.
Regards 🙂


Add fixes for installing from git

This removes the galaxy.yml templating, and flips some of the
templated values to the namespace used on galaxy.ansible.com.

This could impact the release process as git and the release use different namespaces. @SteveHNH thoughts? is there another place downstream that galaxy.yml could be updated for the appropriate namespace?

I can't test the release process, but it should do the right thing by modifying the galaxy.yml that's made as part of it?


No history imported w/ import_historical_data service

Added this integration today, able to see current values. However, when I call the import_historical_data service, no historical data is added as far as I can tell. I've tried both high values and as low as 1 day. I don't see any errors reported in the logs looking at HA Core.
Happy to try to troubleshoot / look further if pointed in the right direction!

Hi,
We have 2 different sensors per meter, so we are working around one HA core issue.
Please double-check that you are looking at a sensor with a "statistic" suffix.

Hi, I originally checked both sensors and there was no data (even after 15 minutes). I took a look today though, and now the statistics sensor has 1 month of data.
Does the sensor support long term statistics in HA? I tried to import a little over 1,000 days.

Sorry for the delay. Please take a look at the readme of the project, there is an explanation of how to import statistics via Developer tools -> Services

Right, that's the service I called. What I mean by statistics is "long term statistics" vs "recorder" (shorter term) in Home Assistant. The most history I've seen on the "statistics" entity is 31 days, which is what I have the "recorder" days set to in HA. So, just wondering if that is intended, or if it is supposed to be able to import long term statistics as well.
When I set to longer periods, like 90 days, I don't see any of those statistics being pulled in.

@some-guy-23 , well, it's weird. Could you please check home assistant logs?

@kdeyev sure, here is from the debug logs, for 186 Days (~6 Months) value for importing -
2024-01-27 10:42:46.660 INFO (MainThread) [custom_components.eyeonwater.coordinator] 4439 data points will be imported
So, seems like it is fetching the data correctly at least, but then failing to actually convert it / add it to Home Assistant statistics?

@some-guy-23 if you don't see any error messages, so probably it was delivered to HA.
I just successfully imported 900 days of water consumption history and it took like 5 minutes. Maybe it worth to try it again (after checking that the integration is up to date).

Updated to latest version and attempted a couple more times, but still no joy. I just see that initial line in logging that it finds the data points, but something seems to be failing after that. Not sure if there's other logging I could enabled to help track it down?

The best suggestion I could give - please try to use Python package pyonwater directly and see what historical data is fetched, I hope it will give us better understanding of what's going on.
There is an example in the repo.


Resource busy

When running pyOCD using the pyUSB backend (as root) on Arch Linux I get the a usb.core.USBError: [Errno 16] Resource busy error during startup.
... which seems to indicate that the HID device is already claimed by something else. However, the is_kernel_driver_active() carried out in pyusb backend in getAllConnectedInterface() reports that no kernel driver is active.
Manually trying to claim the interface immediately after it has been found results in the same error.
I am pretty sure there are no other applications trying to access the device. Running OpenOCD v0.8 against the same CMSIS-DAP interface works correctly.
If pyOCD is run once (failing with Resource busy error), subsequent invokations of opencd will fail until I power-cycle my mbed-device.
Traceback (most recent call last):
File "gdbserver.py", line 26, in
board_selected = MbedBoard.chooseBoard()
File "/usr/lib/python2.7/site-packages/pyOCD/board/mbed_board.py", line 113, in chooseBoard
all_mbeds = MbedBoard.getAllConnectedBoards(transport, False, blocking)
File "/usr/lib/python2.7/site-packages/pyOCD/board/mbed_board.py", line 85, in getAllConnectedBoards
mbed.write([0x80])
File "/usr/lib/python2.7/site-packages/pyOCD/interface/pyusb_backend.py", line 129, in write
self.ep_out.write(data)
File "/usr/lib/python2.7/site-packages/usb/core.py", line 283, in write
return self.device.write(self, data, timeout)
File "/usr/lib/python2.7/site-packages/usb/core.py", line 650, in write
intf, ep = self._ctx.setup_request(self, endpoint)
File "/usr/lib/python2.7/site-packages/usb/core.py", line 154, in setup_request
self.managed_claim_interface(device, intf)
File "/usr/lib/python2.7/site-packages/usb/core.py", line 110, in managed_claim_interface
self.backend.claim_interface(self.handle, i)
File "/usr/lib/python2.7/site-packages/usb/backend/libusb1.py", line 741, in claim_interface
_check(self.lib.libusb_claim_interface(dev_handle.handle, intf))
File "/usr/lib/python2.7/site-packages/usb/backend/libusb1.py", line 554, in _check
raise USBError(_str_error[ret], ret, _libusb_errno[ret])
usb.core.USBError: [Errno 16] Resource busy

Has anyone seen this lately?  This sounds like it might be the same issue as #112.  If is is then it has been fixed in #110 and can be closed.  Could anyone confirm / rule out this?

Closing this issue since it is likely fixed.  Please reopen if you see it again.


Proposal: remove Prelude dependency

Although I like what Prelude is trying to accomplish, it seems out-of-place as a dependency for Result. Every added dependency is an additional burden for consumers, and Prelude specifically doesn't seem to add enough value here to justify it being a requirement.
c.f. the conversation from #reactivecocoa
/cc @nachosoto @saniul @tony

Related to #28.

Also, correct me if I'm wrong, but we want this framework to be the standardized Result type across every Swift project (you know, dream big or go home).
If that's the case, we need to win the hearts and minds of the Swift programmers who are turned off by long dependency lists, and who just want to solve the problem immediately in front of themselves. Making the framework appear smaller will garner broader appeal.

Prelude’s required by Either. That can be changed, of course.

I agree that Result should be as stand-alone as possible. I get the Box dependency, but Prelude is too big a lift IMO, particularly for how little it is needed by this code.
Related discussion: http://stackoverflow.com/a/29540037/97337 (see the comments)
I'd much rather a few lines of code duplication than a 4-framework bundle.
As an alternative, I've been thinking about how to automate combining all the dependencies into a one-file bundle.

@rnapier: I don’t follow how the comments at that link relate to this?

I'd much rather a few lines of code duplication than a 4-framework bundle.

There are two things at play here:

are the reasons to eschew dependencies well-founded, and
will people come around?

I’ve yet to hear a well-founded argument against µframework dependencies (modulo issues with xcodebuild, grumble grumble), but the second one is more interesting. We familiarize users with smaller factoring by use of Result.framework; is that enough?
I think it will be, in time. I don’t know, though. Maybe not.

Prelude’s required by Either.

Ah, my bad—I thought Result brought it in independently for some reason.
We can focus this discussion on #28, then.


WIP

Fixes #

[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[ ] Title summarizes what is changing.
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for feature-requests.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).

Codecov Report

Merging #12460 (0771673) into main (458f63b) will decrease coverage by 15%.
The diff coverage is n/a.

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

@@            Coverage Diff            @@
##            main   #12460      +/-   ##
=========================================
- Coverage     69%      54%     -16%
=========================================
  Files        495      317     -178
  Lines      31132    20573   -10559
  Branches    4905     3558    -1347
=========================================
- Hits       21764    11174   -10590
- Misses      7527     8291     +764
+ Partials    1841     1108     -733

Impacted Files
Coverage Δ

src/kernels/kernelController.ts
16% <0%> (-84%)
:arrow_down:

src/platform/common/utils/stopWatch.ts
16% <0%> (-84%)
:arrow_down:

src/kernels/kernelSettings.ts
12% <0%> (-75%)
:arrow_down:

...c/kernels/execution/cellExecutionMessageHandler.ts
4% <0%> (-74%)
:arrow_down:

src/kernels/execution/cellExecutionQueue.ts
6% <0%> (-74%)
:arrow_down:

src/kernels/execution/cellExecution.ts
11% <0%> (-72%)
:arrow_down:

src/kernels/common/kernelConnectionWrapper.ts
4% <0%> (-70%)
:arrow_down:

...ls/raw/session/rawNotebookSupportedService.node.ts
17% <0%> (-70%)
:arrow_down:

...ls/jupyter/session/jupyterSessionManagerFactory.ts
33% <0%> (-67%)
:arrow_down:

src/platform/webviews/webviewPanelProvider.ts
12% <0%> (-64%)
:arrow_down:

... and 332 more


Fixes 4740: Contacts permission

Description
Fixes #4740
What changes did you make and why?

Removed Contacts Permission.
Implemented Android Autofill Framework.

App Permissions

Tests performed (required)
Tested on Samsung A20 (API Level 30)

@nicolas-raoul I've tested this on API level 30 and it's working perfectly.

Yeah, It's working fine.

Hi @vivek-gite , is this PR still active:)

Closing since it does not seemed active anymore :)


Document flashing internal HDD on NUC-type device

[xginn8] undefined

[xginn8] This issue has attached support thread https://jel.ly.fish/#/support-thread~e77e0151-c247-441a-afb4-28d7ec08519e