Allow resizing width of SQL Lab left bar / editor

CATEGORY
Choose one

[ ] Bug Fix
[X] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY

This PR adds a gutter for resizing the width of SQL Lab left bar and editor. Currently, the setting is not persisted, since in the cases we observed the use was more circumstantial.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Note that the gutter is not vertically centered, which I found more intuitive. I can move it to the center.
TEST PLAN

See video above.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[X] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS
@etr2460 @graceguo-supercat

Codecov Report

Merging #8099 into master will decrease coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #8099      +/-   ##
=========================================
- Coverage   65.93%   65.9%   -0.04%
=========================================
  Files         485     485
  Lines       22887   22925      +38
  Branches     2521    2538      +17
=========================================
+ Hits        15090   15108      +18
- Misses       7666    7685      +19
- Partials      131     132       +1

Impacted Files
Coverage Δ

superset/assets/src/SqlLab/constants.js
100% <100%> (ø)
:arrow_up:

...uperset/assets/src/SqlLab/components/SqlEditor.jsx
57.66% <100%> (+2.27%)
:arrow_up:

.../src/explore/components/ControlPanelsContainer.jsx
68.35% <0%> (-11.65%)
:arrow_down:

...uperset/assets/src/SqlLab/components/SouthPane.jsx
85.71% <0%> (-5.72%)
:arrow_down:

superset/assets/src/explore/controlUtils.js
94.8% <0%> (-2.21%)
:arrow_down:

superset/assets/src/explore/store.js
92.3% <0%> (-1.45%)
:arrow_down:

superset/sql_lab.py
76.84% <0%> (-0.17%)
:arrow_down:

superset/assets/src/explore/controls.jsx
42.85% <0%> (ø)
:arrow_up:

...rset/assets/src/explore/reducers/exploreReducer.js
33.33% <0%> (+1.75%)
:arrow_up:

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 279307c...e8fe5b9. Read the comment docs.

I am strugging to get this to work in version 35.2.  Any ideas why that could be?


Archive as a Service for on prem desktop files

For legal compliance, we have a need for :
Archive as a service so each machine – there will be hundreds – needs to be backed up one time – some need to be kept indefinitely and some 180 days (or maybe 90 days – depends on policy decisions).  Do you have the ability to configure backup profiles with:
Run one time backup.
Have the choice at backup time – keep indefinitely or keep for 180 days with automatic agind – expiration.
Also need the ability to remove backups (archives) once the indefinite are no longer needed.
And this data will be accessed infrequently if ever before it ages out so the Archive tier of storage would be ideal.

@stevekeyres
Since this channel is for driving improvements towards MS Docs, could you provide the URL of the specific documentation that your feedback is applicable to? That way, we are able to make sure your feedback is addressed by the right team.


spans added but transaction never sent

Hi,
I'm trying to test RUM with a third party web application. I insert js library and initialize it via nginx filter, all seems to be loaded properly within the page
Library version 1.0.0: elastic-apm-js-base.umd.min.js
Initialization
<script src="/static_ext/elastic-apm-js-base.umd.min.js"></script>
<script>elasticApm.init({
	logLevel: "debug",
	serviceName: "Dev_CelonisAS",
	serverUrl: "https://sapelk00.bsh.corp.bshg.com:8200"
	})
</script>

page-load tranasaction is being sent to apm-server whenever the page is loaded/reloaded. But for subsequent odata/ajax (ext.HttpRequest spans) i see in the console that thse spans are being started but nothing is sent
When i load the page i see the spans being added...
TransactionService.startSpan POST api/analysis/169/selections ext.HttpRequest
TransactionService.startSpan PUT api/analysis/169 ext.HttpRequest
TransactionService.startSpan GET api/folders/ed5a56a9-b1d9-4687-a025-e1dc924c205d/breadcrumbs ext.HttpRequest

.....
Then wen the page is loaded I see a call to
TransactionService.add
and with it all the spans contained in the transaction are sent to apm-server
If i continue in the page, clicking here and there (several odata calls are triggered with these interactions but the page is stil the same) i see all the spans being added again, but as the page is not reloaded all these spans are not sent
Do i miss something? How could we proceed to send these spans to apm-server?
As far as i can see HttpRequests are properly instrumented as all xhr calls are triggered by apm agent library.
Shoud i include some additional event listener to create/send transactions with all these spans?
Any help/example would be apreciated.
BR,
Raul

Hi @raulmartinezr ,
Thanks for using Elastic APM.
Currently, only the page load transactions are sent to the APM Server by default. However, you can use our Custom Transaction API to create transactions after the page load. One common approach is to create a new transaction with each route change therefore you would register an event listener with your routing library and create a transaction when a route change happens. please note that all of the XHR spans are automatically added to your custom transaction once you create one.
We are also considering to provide integration with commonly used routing libraries but there are no timelines for that yet.
Can I ask which routing library you currently use?
Cheers,
Hamid

Hi @jahtalab,
The webapp is developed with spring boot +   angular and odata services.
But as i do not want to touch this third party app, i installed nginx in front and there I inject javascript in html.
What in did so far is...

add click events in angular links, not with angular but with jquery (filtering by angular attributes)

when the event is triggered previous transaction is ended and new one is created. In this way transactions can be there without being sent to apm-server forever, thats why i want to implement
1.- define max timeout for transactions
2.- modal popup to control transactions when RUM mode is enabled. I'll configure nginx to enable this mode via parameter

Confogured in this way i had an issue; some odata requests triggered by the click were sent faster than transaction closing an creation, so some spans were lost. That's why i had to change the sequence of click event handlers in links, prioritizing mine.

Only a few of these functionalities are pending, hope all of them work fine
By the way, one question. I use javascript agent and java agent(server side transactions and spans).... do you know if apm-server correlates transacions belonging to the same request and merge them in the same transaction before sending it to elasticsearch?
I tried to locate some correlation id in request headers (sent by javascript agent somewhere) but no luck.
Maybe this correlation is done in a different way. If its not done, It would be a nice feature... browser.. network.... server and database times,  all spans together in the same transaction :)
BR,
Raul

@raulmartinezr ,
Thanks for sharing your progress.
We are considering ways to improve sending custom transactions. One of the approaches is to automatically detect when a transaction is finished by keeping track of all the "tasks" (such as XHRs) that happens while the transaction is active.
Do you mind sharing parts of your instrumentation in this issue? Having your feedback would help us a lot in improving our custom transaction feature.
Regarding correlating multiple transactions, we're working on a feature called distributed tracing which would enable you to see the correlation of transactions generated by different services.

@raulmartinezr will be great if you could share the code!


[CODE IMPROVEMENT] Utility code for pushing to HF

We currently only directly support the push to HF for the GUI via:
https://github.com/h2oai/h2o-llmstudio/blob/main/app_utils/sections/experiment.py#L1596
When using the CLI, there is no dedicated util function. We should prepare a notebook and/or dedicated util function to do the transform and push to the hub also for CLI usage.

Hey @psinger is this open for contribution? I would like to explore this, first-timer here. I have experience with HF Hub and downloading models using requests from the hub, locally.

Yes sure, but keep in mind that we have the code already ready for the GUI as per what I linked. So would just need to be ported for cli.

Thank you, I will take a look into the link you sent

closed with #300


Bro how to download the code filles

how to downlload codefiles

Hey! I just added this. While the project is running, go to Files and click the download button.


Merge upstream master

Hi @aminya, I think it's time to merge in a couple miscellaneous changes from upstream. Putting this up so you can take a look at the changes if you want first.
Note: Rather than actually hitting merge on this, please allow me to push the cleaner merge I did here to our master:  https://github.com/atom-ide-community/atom/commits/merge-upstream-master I don't like "merge atom/master into upstream_master" then "merge upstream_master into master". It's a more confusing commit history than actually doing a real merge in one direction.

Also if you have some other stuff you want merged before this, then feel free to let me know and I'll try to work around that.
The actual diff here seems pretty small, so I don't think it will make merge conflicts for our open PRs, but I'm only 90% sure about that.


⚠ Bofrost.at has degraded performance

In 2c670d9, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2265 ms

Resolved: Bofrost.at performance has improved in 8b6cd5b after 7 minutes.


Is there support for ID Token validation?

Implementations should validate the ID Token and its authenticity (that it is indeed signed by the expected Provider).
I couldn't find any explicit methods that might assist in ID Token validation – do they exist in this lib? If not, do you have suggestions for handling this?

Check the OidcClient library.

I think I had initially tried that lib and ran into some trouble (before I had the Provider properly configured). I should try again.
The readme suggests it is built for native app environments – will it work in a .NET WebForms app?

Yes, have a look at the samples: https://github.com/IdentityModel/IdentityModel.OidcClient.Samples

For WebForms you would use the OpenID Connect MW in Katana.

I can't support owin or katana in this app.

well - then you need to do it by hand.
similar to:
https://github.com/IdentityServer/IdentityServer4.Samples/tree/release/Clients/src/MvcManual
But this is only a sample - there is some stuff missing.

@leastprivilege cool thanks for the reference.


A letter not working

Type: Performance Issue
VS Code was running all good when I was doing HTML,later I closed it and open it after sometime,at that time I saw that in vs code twhile doinh HTML the letter 'P' AND 'p' is not working only.Every letters ought their is working except for p.How to resolve this?Need a reoly as soon as possible please
VS Code version: Code 1.87.2 (863d2581ecda6849923a2118d93a088b0745d9d6, 2024-03-08T15:20:17.278Z)
OS version: Windows_NT x64 10.0.22631
Modes:

System Info

Item
Value

CPUs
12th Gen Intel(R) Core(TM) i5-1235U (12 x 2496)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
7.69GB (1.05GB free)

Process Argv
--crash-reporter-id 00a44b6d-786a-4193-91ce-a78769113dd0

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    0	   127	  7864	code main
    0	    84	  7620	fileWatcher [1]
    0	   200	  9868	window [1] (Untitled-1 - .vscode - Visual Studio Code)
    0	   108	 10104	extensionHost [1]
    0	   100	  3528	     "C:\Users\DELL\AppData\Local\Programs\Microsoft VS Code\Code.exe" "c:\Users\DELL\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\html-language-features\server\dist\node\htmlServerMain" --node-ipc --clientProcessId=10104
    5	   185	 10688	   gpu-process
    0	    28	 13820	   crashpad-handler
    0	    79	 14172	shared-process
    0	    48	 15024	   utility-network-service
    0	    94	 15228	window [2] (Issue Reporter)
    0	    71	 17632	ptyHost
    0	   172	 19628	   "C:\Program Files\Google\Chrome\Application\chrome.exe" --single-argument https://aka.ms/vscode-youtube
    0	   136	  8224	     window
    0	    65	  8448	     window
    0	   281	  9344	     gpu-process
    0	   363	  9620	     window
    0	     4	 10084	     C:\Windows\system32\cmd.exe /d /s /c ""C:\Program Files\McAfee\WebAdvisor\BrowserHost.exe" chrome-extension://fheoggkfdfchfphceeifdbepaooicaho/ --parent-window=0" < \\.\pipe\chrome.nativeMessaging.in.e3ffe7483211fe50 > \\.\pipe\chrome.nativeMessaging.out.e3ffe7483211fe50
    0	    14	  4892	       "C:\Program Files\McAfee\WebAdvisor\BrowserHost.exe"  chrome-extension://fheoggkfdfchfphceeifdbepaooicaho/ --parent-window=0
    0	    12	 12560	       C:\Windows\system32\conhost.exe 0x4
    0	    28	 10452	     window
    0	    23	 13992	     utility-process
    0	    43	 14104	     utility-network-service
    0	    22	 16072	     utility-process
    0	     8	 16720	     crashpad-handler
    0	    44	 20044	     window
    0	    60	 20864	     window

Workspace Info
|  Window (Untitled-1 - .vscode - Visual Studio Code)
|    Folder (.vscode): 4 files
|      File types: json(2)
|      Conf files:;

Extensions: none
A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes516:30244333
vscoreces:30445986
vscod805:30301674
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:30899288
c4g48928:30535728
azure-dev_surveyone:30548225
962ge761:30959799
pythongtdpath:30769146
welcomedialogc:30910334
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
h48ei257:30992801
pythontbext0:30879054
accentitlementst:30995554
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
d6fi2104:30959263
cppperfcontrol:30979541
d34g3935:30971562
fegfb526:30981948
bg6jg535:30979843
ccp2r3:30993541
dsvsc020:30976470
jchc7451:30973076

What do you mean a letter is not working?
Do you maybe define a keyboard shortcut for p ?

Nooooooooo it's that while I was typing code in VS code for HTML everything
was perfect but the problem occurred when I typed the world P/p i.e the
letter P/p is not working when I am typing it in VS Code except that each
and every leter is showing but not the letter P/p. Please resolve the issue
as soon as possible
On Thu, 21 Mar 2024, 5:26 pm Martin Aeschlimann, @.***>
wrote:

What do you mean a letter is not working?
Do you maybe define a keyboard shortcut for p ?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/208273#issuecomment-2012075596,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHFPJPZYOPTKF3TSJN5NWITYZLDGRAVCNFSM6AAAAABFAWBCMWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJSGA3TKNJZGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Here's a guide how to troubleshoot key issues: https://github.com/microsoft/vscode/wiki/Keybinding-Issues

Hi :-)
Thanks for filing this issue!

Could you provide your keybindings.json file contents? (invoke command "Preferences: Open Keyboard Shortcuts (JSON)" to open it)

Could you please try the following:

2.1. Run the command "Developer: Toggle Keyboard Shortcuts Troubleshooting" (you can, for example, run it from the Command Palette).
This will activate logging of dispatched keyboard shortcuts and will open an output panel with the corresponding log file.
2.2. Press exactly the keybinding you desire
2.3. Check what VS Code detects
If you are having trouble understanding the output, please feel free to paste here and we'll be happy to help ☺

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Releasing v0.5.0

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (cdbaf62) 98.97% compared to head (2a71190) 98.87%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #29      +/-   ##
==========================================
- Coverage   98.97%   98.87%   -0.10%
==========================================
  Files          35       24      -11
  Lines         781      535     -246
==========================================
- Hits          773      529     -244
+ Misses          8        6       -2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Remove initTooltips include & add upgrade considerations

Overview

Fully remove initTooltips util and replace with Controller usage, add release notes and backwards compatible support for data-tippy-* (on initial load only) to be removed fully in a future release.
Adds more unit tests to the tooltip & dropdown controller code, specifically for the hide on esc behaviour.
Fixes #10668 (last part of the issue), built on #10869

Checklist

[x] Do the tests still pass?
[x] Does the code comply with the style guide?
[x] For front-end changes: Did you test on all of Wagtail’s supported environments?
[x] For new features: Has the documentation been updated accordingly? upgrade considerations only

How to test this change

In the code changes, the plugins are now moved to the controller files with one being exported (hide on esc) as it's used by other code. These functions are 100% the same.
A key change however, is that we no longer need to dynamically init tooltips in the chooser modal or when dynamic search results get patched into the DOM.

To validate this, do some testing as follows;

Load the pages search http://localhost:8000/admin/pages/search/?q=breads and then confirm, on initial load, that the time ago style tooltips show.
Now, do a search for a different page, when the HTML gets updated with the new results, confirm these new results have functional tooltips.
Now, edit the home page (or any page with a page chooser in use), within the chooser modal, confirm the tooltips work correctly in the initial modal results.
When, do a search within the modal to confirm that the tooltips still work.

This is now ready for review and is the last bit of removal for the data-tippy-* attribute usage, except for a small support for these attributes on initial load. I am happy to add a mutation observer to support this approach for more use cases if we feel that's needed.
Would love to get your eyes on this either @thibaudcolas or @laymonage  - while #10668 is closed, this PR will do the final clean up from that issue.

Adding to 5.2 as this is a key part of finishing off our full migration of the tooltips with a deprecation plan in place.

Note to self. Bump upgrade considerations to 5.3

Rebased, changed to 5.3

Rebased, fixed a typo and changed the deprecated comment to be @deprecated RemovedInWagtail70. This is in TooltipController - where we will keep supporting the old data attributes on load for some time.


python3Packages.pikepdf: 1.1.0 -> 1.6.2, pybind11: 2.2.4 -> 2.3.0

Motivation for this change
This fixes #67850.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I tested ocrmypdf on NixOS x86_64.
Notify maintainers
cc @yuriaisaka @Kiwi @FRidh

@GrahamcOfBorg build python3Packages.pikepdf ocrmypdf
pybind11 users:
@GrahamcOfBorg build python3Packages.accupy
@GrahamcOfBorg build pythonPackages.pyopencl python3Packages.pyopencl
@GrahamcOfBorg build pythonPackages.fasttext python3Packages.fasttext
@GrahamcOfBorg build pythonPackages.pyfma python3Packages.pyfma
@GrahamcOfBorg build pythonPackages.pyamg python3Packages.pyamg
@GrahamcOfBorg build caffe2


linux-sgx: implement POSIX calls based on getsockname and set/getbooloption.

Hi there,
I continue implementing some of the missing POSIX OCALLs that I use in my apps.
The ones I propose in this PR are the OCALLs that rely on the POSIX function getbooloption, setbooloption and getsockname .
Cheers

Hello @wenyongh,
Thanks for your review! I have applied the changes. I also noticed it was the case for TCP_FASTOPEN_CONNECT, so I have removed the #if/#endif clauses related to it.
Cheers


NIFI-7708: Add Azure SDK for Java (and transitive dependencies) to LI…

…CENCE and NOTICE files
https://issues.apache.org/jira/browse/NIFI-7708
Description of PR
Adds dependencies to the LICENSE and NOTICE files in the Azure bundle and in the Assembly
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[x] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[x] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

@pgyori LGTM. I did a check of the included Jars and this looks correct. Re-building the rebase and then will merge.

Thank you, @jfrazee !


Allow dub language selection for Funimation

Checklist

[x] I'm reporting a feature request
[x] I've verified that I'm running youtube-dl version 2021.04.01
[x] I've searched the bugtracker for similar feature requests including closed ones

Description

Funimation offers multiple dubs, usually English and Japanese.
By default English is selected and downloaded but I only want the Japanese dub (or both if that's possible).
AFAICT there is no option in youtube-dl to select the dub language it should download.
You can force a certain language on the website using ?lang=japanese and such URLs work in youtube-dl for singular episodes but appending it to playlists doesn't work as youtube-dl interprets ?lang=japanese as an episode path and that obviously doesn't exist.

@vishnunair6 and I would like to begin work on this issue!

Amazing, thank you!
I have just realised that this is probably less of a priority for Funimation because it doesn't even support series extraction which means the hack works for all supported use-cases but work being being done in this direction is probably still beneficial for other sites and future Funimation series support.


ExponentialMovingAverage.variables_to_restore() treats all vars like they have averages

Use:
ema = tf.train.ExponentialMovingAverage(0.9)
test = tf.Variable(0.1)
print(ema.variables_to_restore([test])

outputs {u'Variable/ExponentialMovingAverage': <tensorflow.python.ops.variables.Variable object at 0x1139d1bd0>}
I expect variables_to_restore not to append /ExponentialMovingAverage to the names of variables that do not have averages tracked.
I checked that ema._averages and tf.get_collection(tf.GraphKeys.MOVING_AVERAGE_VARIABLES) were indeed both empty.  Calling ema.average_name(test) produces the modified name and this appears to be done specifically (perhaps for another reason).  So variables_to_restore incorrectly modifies the name because it uses average_name.

@sherrym Is this a bug?  The behavior of variables_to_restore seems somewhat elaborate.

Closing due to inactivity. Feel free to open a new github issue if the problem still persists in recent versions.

Not sure if this is the best place to put this piece of feedback, but I'm finding the behavior of variables_to_restore() to a bit ... bizarre and not intuitive.  Currently variables_to_restore() includes all trainable variables in the graph.  Being an instance method I would have expected variables_to_restore() to return something related to the specific instance of ExponentialMovingAverages.  But if you look at the code it has no dependency whatsoever on the instance.  Why isn't this a class method?
At a minimum I would expect the documentation to specify that even though you're calling this method on a specific instance you're going to get all variables in the graph.  I had to look through the source code to get through my misunderstanding.

Tensorflow 1.2 has another way of saving moving average and using them for inference: tf.contrib.opt.MovingAverageOptimizer.
For me that has been a very good work around to this issue.


cmake: linker_script: Fix cmake linker for some networking

Iterable sections for CoAP services and network management events were missing.

Rebased
Used the Kconfig variable introduced in #72373


replace mongodb with a F/OSS database

mongodb is no long F/OSS as of October 2018 https://en.wikipedia.org/wiki/MongoDB#MongoDB_Community_Server_2
Preferably something already packaged for Debian
Candidates include

PostgreSQL
The Guardian migrated from MongoDB to PostgreSQL in 2018 using the JSONB type

Agree that we should move away. For now we are still using mongodb 3.4 which is AGPL.
root@16776f26e336:/usr/share/doc/mongodb-org-server# ls
GNU-AGPL-3.0.gz  MPL-2.gz  README  THIRD-PARTY-NOTICES.gotools.gz  THIRD-PARTY-NOTICES.gz  changelog.gz  copyright

That said I am not fully certain if we are fully complying with Affero GPL at the moment - although we are providing links to this source code we don't provide links to the MongoDB source code?

Is Cassandra also a viable choice for the replacement?
Its license is the Apache License 2.0.
Also, moving nested schema would be straightforward without the need for foreign keys and separate tables for workflows that are in the queued object.

a viable choice for the replacement?

We should make a list of characteristics we want in a replacement database and then evaluate our options. Could you compile a list of characteristics?
Looks like https://www.google.com/search?q=how+to+decide+on+a+new+database+engine returns some useful results to consider.

@obasekiosa would you be able to look at the code and write down how cwlviewer uses the database right now?
I would strongly learn towards Postgres but we should consider all the options.

Oh great.
While currently after much digging, I am all for Postgres.
This is what I was able to compile on both options.
Postgres VS Cassandra

While still on the code coverage and making progress.
I was also able to extract the Workflow and Queued Workflow schema as is in the database.
Currently modelling the Workflow on PostgreSQL - a lot of nested objects.
I also noticed that these two collections are in a database named test. Is that the name used in the production db also?
I was also able to look into the spring data implementation for PostgreSQL and how to use it.
Haven't been able to understand how to perform queries on a JSON dump in PostgreSQL hence why I began modelling just in case.

Will start working on it this week (probably, otherwise in the beginning of next week). What I think are the requirements for this task:

[ ] update pom.xml dependencies, removing Mongo, adding Postgres + Hibernate (or data/jdbc)
[ ] replace the Spring configuration used by Mongo, probably add one for Postgres (pool settings, etc)
[ ] update the repositories to use Hibernate or JDBC instead of Data+Mongo
[ ] update entities and other annotations & classes as needed
[ ] add database migration, with initial DDL, but try to include some instructions for future DML's too
[ ] update tests
[ ] manual tests
[ ] send to review and write notes about anything important to observe when migrating, any docs, etc

I think having PostgreSQL makes things easier to have cwlviewer distributed and used in multiple places. I am not familiar with Debian guidelines, but I worked in companies that avoided Elastic or Mongo just to not have any risk of more license changes affecting their business (even if OK for a bank for example to use it, managers are required to give detailed explanation yo upper management when choosing OSS software instead of proprietary, and these license issues/changes are not weel regarded, even when for the best)
While I think the Spring Boot integration was not factored in the decision to move to another DB, it is much easier to keep SpringBoot up to date by using JPA Data instead of the MongoDB Data library (more updates, more users, more devs familiar with.)
Bruno

@fmigneault I'm a Debian Developer, so when I said "put cwlviewer in Debian" I meant "have Debian packages for cwlviewer in the main official Debian archive" :-)


deploy a warnet as a helm chart

not sure the best way to do this but one idea is convert a docker-compose.yml to a helm chart using https://kubernetes.io/docs/tasks/configure-pod-container/translate-compose-kubernetes/
Theres also https://github.com/scaling-lightning/

Closing as we don't have any immediate plans to implement a help chart for now.
LMK if you want it re-opened :)


Ability to ingest OTEL data using proto format

Is it possible to add ability to ingest to OTEL endpoint using proto format?
This is what I am currently using for one of my services to send logs:
req := plogotlp.NewExportRequestFromLogs(logs)
		data, err := req.MarshalProto()
		if err != nil {
			log.Println(err)
			return err
		}

		httpReq, err := http.NewRequest("POST", axiomUrl+"/v1/logs", bytes.NewBuffer(data))
		if err != nil {
			log.Println(err)
			return err
		}

		httpReq.Header.Set("X-Axiom-Dataset", axiomDatasetName)
		httpReq.Header.Set("X-Axiom-OrgId", axiomOrgId)
		httpReq.Header.Set("Authorization", "Bearer "+ axiomToken)
		httpReq.Header.Set("Content-Type", "application/x-protobuf")

		_, err = http.DefaultClient.Do(httpReq)
		if err != nil {
			log.Println(err)
			return err
		}

I think this is best left to the OTel SDK for Go. AFAIK, the latest release has some APIs for logs?

just checked, seems to still be in development. But I agree, OTel SDK makes more sense.


Input / output system

In gitlab by @Koensw on May 9, 2017, 14:12
Logic should be added to allow redirecting input / output. We will use a single parameter input and output for that and all messages should be automatically redirected using the same name (unless a special message flag is added to ignore those names).
The message system should be updated to support this.

In gitlab by @Koensw on May 9, 2017, 14:12
added ~2158 label

In gitlab by @Koensw on May 24, 2017, 17:10
closed via commit f0c935ddbfa1792392ad445ac559f907eeb18ae9


Symfony 6 support?

I am unable to upgrade a Symfony 5.4 project to Symfony 6.0 due to it using sentry/sentry-symfony
Checking for deprecation notices gives the following notice:

2x: The "Sentry\SentryBundle\Tracing\Doctrine\DBAL\TracingDriverForV3" class implements "Doctrine\DBAL\VersionAwarePlatformDriver" that is deprecated All drivers will have to be aware of the server version in the next major release.
2x in DeprecationErrorHandler::shutdown from Symfony\Bridge\PhpUnit

You can disregard those deprecations, they are not related to Symfony but to DBAL. In any case, we are unable to solve them, since we need to be compatible with both DBAL v2 and v3. See also #483


Fixing the Coordinators list issue

Fixing the Coordinators list going offscreen on small screen sizes issue

Please do not forget to give tehe labes of hacktoberfest


email authors of upcoming restricted release and autorelease eligible restricted works

Fixes #757
Adds two rake tasks:
Task 1- Emails authors when their restricted submission is one month from release. The email includes a link that will extend the release date by a year. There's a guard on the extension to only update if it is a restricted work and there are less than 3 years from when metadata was released to publication release date. This was done to prevent someone from using the link more than once (getting multiple extensions). Also, I decided for confirmation to just do a flash notice on the submissions page vs a separate confirmation page as previously discussed. It felt more efficient, but if that's not ideal for any reason, I'm happy to change it.
Task 2- Automatically releases submissions that have reached their release date.
This is only for graduate and honors instances. I assume we can just run the rake task on those partners.

Should only be for 'restricted for Penn State only'


Using dynamic logger for GFX-H264 backend.

Use preallocated logger for GFX-H264 backend.
Enabled OpenH264 traces on log level TRACE

conflicts need to be resolved

@MartinHaimberger done

@freerdp-bot test
@mfleisz ok, fixed (but the bot is still not happy)

Ok, rebased again, busy day today ^^

@mfleisz Ok, fixed and rebased the pull


Dynamically Changing Unit of Experiment

Experiment is getting initialized only once on the server instead of on every request.

can you post some code and a bit more detail? You should be using your experiment class as a singleton so it makes sense that it's getting initialized only once on the server.

How to change the Unit of Experiment Dynamically ?
I want to get unit from url params through react-router
I tried
ExperimentSetup.registerExperimentInput('diet', routerProps.params.diet);
But

Problem is parameterize() function takes Experiment instance which has to be created at File level instead of component level and at File Level you don't know current user id.
Signup.js
let experiment = new MyNameSpace();

export default parameterize(experiment, ["signupText"], Component)

Home.js
import Signup from './Signup'

function Home(){
   return <Signup />
}

Login.js
loginSuccess(function(userId){
   ExperimentSetup.registerExperimentInput('userId', routerProps.params.id);
});

This will give an error that userId is not there which can be avoided by setting default value but when userId is set using ExperimentSetup.registerExperimentInput('userId', routerProps.params.id); It does not Changes anything since instance of Namespace was already created earlier.

Let me create this bug on react-experiments I think its more related to that than Planout.js

I think the issue is that the component mounts before the user id is passed through since there's basically a race condition there. This is an unfortunate challenge to doing async user id passing. Perhaps there needs to be something in react-experiments that will make the component remount when a change in the user id is detected or something. Don't have time to look into this until the 22nd due to exams, but maybe @mattrheault can be of some more assistance in the meantime


Fix end-of-file

some of those files, specifically the json files in the fixtures directory are generated by the tests themselves.
specifically here: https://github.com/ansible/ansible-navigator/blob/main/tests/integration/_common.py#L45
triggered by a temporary update to the test or the use on an env var
https://github.com/ansible/ansible-navigator/blob/main/tests/integration/actions/config/base.py#L73
So........  if we update them we should probably have a followup PR/issue to not reintroduce them when fixtures are updated or created

I was expecting that and to have to make other changes. Thanks for pointing me the right place to change.
I will update the PR to do it right from start.


chore(app): alphabetize styles all scss files

Resolves: #42
Summary

Make all styles in all scss files in alphabetical order
I used this extension to sort

Checklist

[ ] Branch should be in format TYPE/scope/description
[ ] PR title is in format TYPE(scope): description

Expected TYPES, scopes are listed here

[ ] Commits are in semantic format

it looks like build failed due to import no longer exists from file deletion
https://github.com/starter-code/res-gen/blob/master/src/scss/Components/index.scss#L1


Catching a required Pokémon doesn't change the black UI element into the Pokémon's sprite

Fixed in latest


Initial Work-Server to Client Migration

Moves the exceptions file to the client, which now containers all the exceptions for both packages.

Mongo no go. Merging


feat: keyboard navigation for new headers

keys to go between header cells, from header to body and vice versa
tabbing to header elements

not in this PR:

totals support
logic for keyboard.enabled

losing coverage due to skipping totals and not running a couple of functions in accessibility-utils (that contain no logic)

I'm getting an error thrown when I use the keys to navigate the header and do the following:

I have a table with 6 columns
Focus the first column header
Navigate using the keys the last colum and then continue by pressing right key, so that focus is on the first cell (0,0 in body)
Press the left key

EDIT: Does not happen if I remove the totals row

yeah it is broken when you have a totals row at this point


Support for creating & listing webhooks

I don't have too much knowledge working with cats, but I'm developing a pull request for creating webhooks.

Any progress on this @technolaji? Do you need help?


Ability to return a value from middleware

It adds ability to return a value from middleware

i.m.o this would conflict with us ever being able to handle promises, since the promise needs to be returned. It also would have a lot of gotchas, from using middlewares that don't currently pass through the return value, to the fact that many middleware will call next() async.

The core of Express routing is maintained at https://github.com/pillarjs/router if you want to open a discussion over there regarding your idea. The PR https://github.com/pillarjs/router/pull/32 is an example of how it seems like your idea may conflict with the direction of where Express middleware is heading.

And looking at your test, it basically looks like you may want to use Koa: http://koajs.com/ which implement the type of "upstream" support that your test seems to be describing.


Check whole claiming process

We want to verify that everything works, starting from wallet address generation, over snapshot generation, running the network, claiming the funds, and sending with shimmer addresses.
4 different scenarios:
iota ledger => smr ledger
smr ledger => smr ledger
iota stronghold => smr stronghold
smr stronghold => smr stronghold

Everything worked fine.


Cricket Game

I have added a cricket game using html , css , javascript .
Game Description :
Score: You accumulate runs by clicking the "Hit the Ball!" button.
Outcome: Each click randomly generates a score between 0 to 6. If the score is 0, a message will indicate that you've been "Out".

@rajnil9 mention the issue no

@iamrahulmahato issue no. #2322

I have added the issue no. please check github pull request comment section
On Sat, Nov 9, 2024 at 12:38 PM Rahul Mahato @.***>
wrote:

@rajnil9 https://github.com/rajnil9 mention the issue no
—
Reply to this email directly, view it on GitHub
https://github.com/iamrahulmahato/master-web-development/pull/2310#issuecomment-2466088911,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BLJ52FRGK5RFELFG6JGIAXDZ7WYGLAVCNFSM6AAAAABROF5ZTKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRWGA4DQOJRGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

assign me this please @iamrahulmahato


new sessions/amalthea cleanup

There's several smaller issues with the new amalthea and sessions in data services code. This issue serves as a tracking issue for them.

ImagePullPolicy is set to IfNotPresent for custom images, so something like :latest won't update
Editing some settings on the environment doesn't propagate when unhibernating sessions and there's no warning that a full restart is needed to apply the changes. E.g. users id, entrypoint, image tag.
All the k8s environment variables are available in sessions, we should probably limit that to only necessary ones:
On session start there's sometimes a warning about the gateway being misconfigured. after refreshing, the session works.

Editing some settings on the environment doesn't propagate when unhibernating sessions and there's no warning that a full restart is needed to apply the changes. E.g. users id, entrypoint, image tag.

This is most likely good by design. Any of these changes could render the session un-resumable, so it is best to freeze the launch settings.

ImagePullPolicy is set to IfNotPresent for custom images, so something like :latest won't update

We should probably open a full issue to implement image de-referencing at session launch.

To give an more tricky example, I'm playing with buildpack images right now and those run as user 1002, but we default to user 1000. The session will actually start and run for a second, and then fail due to a permission error, which I see in the logs. So I change UID to 1002 and click launch again, and it'll still fail because it's still UID 1000.

Wait, why does this happen? The launch process should use the current settings from postgres.

one is not informing users that the session needs to be fully stopped for changes to apply (or make it a conscious choice with a button that allows force-applying it?)

This is a UI issue and needs to be tracked there.


Store exception messages when using %%gremlin --store-to

Issue #, if available: N/A
Description of changes:

Fixed %%gremlin magic command to return error messages along with query results when using --store-to.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Seems like a reasonable change - thanks @drakearch !

Great! I'm glad to help


Added getItems method

Tested and works fine. Code borrowed from /seishun/node-steam-trade

Thank you! Landed as https://github.com/Alex7Kom/node-steam-tradeoffers/commit/79440c1edc010ad1516c5c26a81446de2850a547 and https://github.com/Alex7Kom/node-steam-tradeoffers/commit/d5e5b5b695d846d9d70576b986e6473f327dc0b2.


Rename @_predatesConcurrency to @preconcurrency.

Introduce the @preconcurrency attribute name for @_predatesConcurrency,
which has been the favored name in the pitch thread so far. Retain the
old name for now to help smooth migration.

@swift-ci please test

@swift-ci please test macOS

@swift-ci please nominate


How can I prevent the completion of a task?

hi @nikku,
Thank you for sharing this excellent lib.
Though, going through your samples, is not clear to me how can I "skip" a task for being completed:
workers.registerWorker('work:A', [ 'numberVar' ], async function(context) {

  // let say, my business logic tells here that the current task is not ready
  // to be completed and I would like to postpone it for the next iteration
  // It is possible to do that?

  // complete with update variables
  return {
    variables: {
      numberVar: newNumber
    }
  };
});

Thanks,
Ionut

You're essentially asking for the unlock feature.
What exactly is the reason you'd like to wait? Polling the database or a message queue whether work was done?
An alternative would be to make your task long-running in these cases:
workers.registerWorker('work:a', async function(context) {

  let completed = false;

  while (true) {
    completed = await didWorkComplete();

    if (completed) {
      break;
    }

    // extend the lock for another five seconds
    await extendLock(5000);

    // wait for arbitrary delay
    await delay(5000);
  }

  return {
    variables: { ... }
  }
});

Throwing an error would work, too:
workers.registerWorker('work:a', async function(context) {
throw new Error('not completed yet');
});

Well in my scenario, is a "message" which can be expected to be ready up to several hours/days since the process was started,  so throwing an error can be an approach.
Thanks for your quick replay!

Closing this as things are sorted out I suppose.


initial todo list

No issue to close
Description
introduce v1 of todo list in database
Is it a feature that change user experience?
no
Client output
no changes

Pull Request Test Coverage Report for Build 274209997

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

Totals

Change from base Build 271218212:
0.0%

Covered Lines:
7496

Relevant Lines:
8576

💛  - Coveralls


Replace references to the Toulouse edition with Lyon

Applicable Issues
Fixes #1
Description of the Change
It was quite a while since Toulouse was replaced by Agen, Agen-1,    and Paris and several months since the Lyon edition was released    to it's about time we update the references in Sepia.
Since Toulouse the ID event has been added and it was included in the "Test Outcome Analysis" pipeline actor role in overview.svg. That might not be a perfect match but it's probably the best we've got at this point.
Alternate Designs
None.
Benefits
We're no longer linking to an obsolete protocol edition.
Possible Drawbacks
None.
Sign-off

Developer's Certificate of Origin 1.1
By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I    have the right to submit it under the open source license    indicated in the file; or
(b) The contribution is based upon previous work that, to the best    of my knowledge, is covered under an appropriate open source    license and I have the right under that license to submit that    work with modifications, whether created in whole or in part    by me, under the same open source license (unless I am    permitted to submit under a different license), as indicated    in the file; or
(c) The contribution was provided directly to me by some other    person who certified (a), (b) or (c) and I have not modified    it.
(d) I understand and agree that this project and the contribution    are public and that a record of the contribution (including all    personal information I submit with it, including my sign-off) is    maintained indefinitely and may be redistributed consistent with    this project or the open source license(s) involved.
Signed-off-by: Magnus Bäck<EMAIL_ADDRESS>

Since Toulouse we have got a new event type - EiffelIssueDefinedEvent. It should be added to the architecture somewhere.

Indeed, good catch:
$ git diff-tree --name-only --diff-filter=A -r edition-toulouse..edition-paris -- eiffel-vocabulary
eiffel-vocabulary/EiffelIssueDefinedEvent.md

I don't quite see how ID would fit into an existing actor. IV is in its own "Test Outcome Analysis" box but ID doesn't fit in there. How about a new "Issue Management" actor that includes ID?

I don't quite see how ID would fit into an existing actor. IV is in its own "Test Outcome Analysis" box but ID doesn't fit in there. How about a new "Issue Management" actor that includes ID?

Maybe. Should we then move IV into that box as well? I have a feeling that the boxes are too finegrained today to be able to add more events to them. We risk having one box for each new event coming. Maybe we need to revisit/define the meaning of the boxes - why are they there, how are they supposed to be divided, etc.

Should we then move IV into that box as well?

That wasn't my intention, but I guess it depends on the definition of an Issue Management role. If that role is concerned with issue tracker integration I suppose both ID and IV would fit reasonably well, although determination of whether IV should be sent is a form of test outcome analysis.

I have a feeling that the boxes are too finegrained today to be able to add more events to them. We risk having one box for each new event coming.

Yes, I also feel that they're quite narrow and specialized and it's not clear how they're supposed to be used. I wonder if it would be useful to focus on actors that actually consume events. SCC, SCS, and ID (and possibly other *DefinedEvent) are often used to model reality and produced by actors that translate reality to Eiffel events. I'm almost inclined to group them into a larger "reality translator" bucket.

Maybe we need to revisit/define the meaning of the boxes - why are they there, how are they supposed to be divided, etc.

Yes. But I think it would be unfortunate to hold back this PR. Should we attempt to find an acceptable supposedly short-term way of getting the ID event into overview.svg so we can get rid of the Toulouse references and then attack the larger discussion?

Agree with you @magnusbaeck maybe put it somewhere just to get this out of the door.

There, PR rebased and updated with references to Lyon instead of Paris. Since the branch needed to be updated from the upstream I did a non-fast forward update even though that goes against the recommendations.
Added ID to the Test Outcome Analysis role. See https://raw.githubusercontent.com/magnusbaeck/eiffel-sepia/9b8c110f9dfedff54e62e54ed7e3a5fdca110f5c/svg/overview.svg.

I changed the title of the PR to Lyon

Thanks! I remembered to change the PR description but forgot about the title.


feat(apim): Introduce gateway resource

Introduce gateway resource for Azure API Management for upcoming new API version, this is not going in to main.

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

[!NOTE]
As of January 2024 there is no PR assignee. This is expected. See https://aka.ms/azsdk/pr-arm-review.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Click here to see the details of Step 1, Breaking Changes review
If you are in purview of Step 1 of the diagram, follow the Breaking Changes review process.
IMPORTANT! This applies even if you believe your PR was mislabeled, for any reason, including tool failure.

Click here to see the details of Step 2, ARM review
See https://aka.ms/azsdk/pr-arm-review.

 Click here to see the diagram footnotes
Diagram footnotes
[1] See ARM review queue (for PR merge queues, see [2]).
[2] public repo merge queue, private repo merge queue (for ARM review queue, [1])
The ARM reviewer on-call engineer visits the merge queue twice a day, so the approximate ETA for merges is 12 - 24 hours.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[ ] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been generated using OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[x] Other, please clarify:

Update API version that has not been released yet and is being prepared (see base branch)

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with PR workflow diagram Step 2 (ARM review), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

Pl fix the errors flagged here or add a suppresion with an explanation for why they cant be fixed.
https://github.com/Azure/azure-rest-api-specs/pull/27757/checks?check_run_id=21655284311
Suppresions - https://github.com/Azure/autorest/blob/main/docs/generate/suppress-warnings.md

Note to next reviewer: Rest of the changes look good

@rkmanda Thanks for reviewing! These are known issues for which we've had exception in the past (https://github.com/Azure/azure-rest-api-specs/pull/26547#issuecomment-1839170944) and are tracked respectively in:

https://github.com/Azure/azure-rest-api-specs/issues/27012
https://github.com/Azure/azure-rest-api-specs/issues/23702

Given this PR is targeting release-ApiManagement-2023-09-01-preview and not main, is it OK to not have them as blocker?
However, I have made an update for the common types for errors given these are new resources which should be OK.

@rkmanda Thanks for reviewing! These are known issues for which we've had exception in the past (#26547 (comment)) and are tracked respectively in:

Move to common Model for Error Response  #27012
APIM : No booleans only Enums  #23702

Given this PR is targeting release-ApiManagement-2023-09-01-preview and not main, is it OK to not have them as blocker?
However, I have made an update for the common types for errors given these are new resources which should be OK.

Note to reviewer: I removed the ARMCHangesRequested label because the author mentioned offline that they wanted this response to be looked at.

Looks like there are a couple Lint errors still which were not discussed above , specifically about havving the name and capacity as required fields in the PATCH body. Are you planning to fix those ?
:

@raosuhas I have missed those, sorry. Fixing these now:

I have updated this PR and we should be good to proceed.
Things to point out:

We have an active issue open to move to common error response model, but this new resource already supports it (https://github.com/Azure/azure-rest-api-specs/issues/27012)
Any further issues related to this migration will be handled in a separate PR to the release-ApiManagement-2023-09-01-preview directly and not as part of this PR.

/r RequestMerge

/pr RequestMerge


Using multiple queueRef in the same cron?

I would like to use firebase-cron to push multiple types of tasks but it seems like one cron is limited to one task.
Can I use the same cron to push task with differents names?
Thanks for making firebase-cron. It's great!

Currently, only a single queue is used, but you can use firebase-queue specs to make something like this work.
Basically create a job with that has a start_state on the data object for your specific spec:
cron.addJob('foo', '00 00 00 * * *', {start_state: 'custom-spec-start-state', foo: 'bar'}, function(err) {
  if (err) return console.error(err);
  console.log('job added');
});

I'm also open to refactoring to only passing in the root firebase ref and having more metadata around a job. The metadata could specify the queue the job data should be push to, then that can be spread out among multiple queues and specs if necessary. So instead of doing this.queue.child('tasks').push(data), it would be something like this.ref.child(job.queue).child('tasks').push(data).
I like this idea and if anyone is interested in doing a PR, let me know. This will make using dynamically created queues and scaling easier.

You are right. It's working thanks!


.NET generated SDK

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

@jsquire
Me and @mitsha-microsoft are currently working on this project. We will definitely go through all the documentation and guidelines before finally raising a PR. Please give us some time for that. Rest assured, it will be done on our end.

Hi please keep this PR active as we are working on correcting some issues


Has anyone looked at making an Expo plugin?

Hi, I was wondering if anyone had looked at creating an Expo plugin for this library. Doing so would (ideally) make it easier to create a cross-platform app (with Expo) that incorporated this library.
I am looking to add share target functionality to an Expo-based app, and so I was wondering if anyone had already looked at this, and potentially identified any difficulties in doing it.
Thanks!

There are demands for that -- 450+ votes from expo community on a plugin as you proposed: https://expo.canny.io/feature-requests/p/share-extension-ios-share-intent-android
However, there is no such expo plugin / library to my knowledge.

I'm looking into it, but I need help getting the iOS side to work correctly to continue. If you have this working and can share some knowledge, would be happy to help.

I ended up just going with a bare React Native app that includes this library. (https://github.com/Howdju/react-native-share-menu)

@carlgieringer your own fork you made some improvements to?

I did a refactor to try and clean up the code (my opinion; I'm not much of a Swift developer). Functionality-wise, I added the ability to share multiple items, but this feature was also added recently in this repository (https://github.com/meedan/react-native-share-menu/commit/a8ab1ad4f71db29c169b9effdb937ff63dd4a59d).

Hi guys, share my expo config plugin to auto-configure iOS Share Extension.
https://github.com/langtube/expo-config-plugin-ios-share-extension

You can check out an implementation here https://github.com/frankcalise/expo-share-ios/tree/expo-plugin

Hi guys, share my expo config plugin to auto-configure iOS Share Extension.
https://github.com/langtube/expo-config-plugin-ios-share-extension

Is there a expo plugin for Android too?

any updates on this?

Hi guys, share my expo config plugin to auto-configure iOS Share Extension. https://github.com/langtube/expo-config-plugin-ios-share-extension

Thanks @timedtext for the plugin! I was able to get it working on iOS pretty easily with EAS. I'm wondering if anyone knows of an Android plugin that's similar? I can help create one if someone can help with the steps.

bumping this

See https://github.com/achorein/expo-share-intent-demo from https://expo.canny.io/feature-requests/p/share-extension-ios-share-intent-android


Ability to change the "content" folder in the config file

Hey @mrvautin,
Just found out about squido and already love it, thank you for making such an awesome product.
I have a few requests though (which I'll split up into multiple issues to help you with tracking).
The first one is if you could add the ability to change the content folder in the config file.
I usually call this folder assets but there is no way to change it.
I checked your lib/build.js file and the folder name is hardcoded there (lines 466 and 468):
// Copy the content. Images, Stylesheets, JS etc
const copyContent = async () => {
    const contentDir = path.join(process.config.sourceDir, 'content');
    const contentFiles = await glob(`${winPath(contentDir)}/*/**`);
    const buildContent = path.join(process.config.buildDir, 'content');

    ...

There are many other variables set up in the config file, maybe this could also be one of them, with content as the default value.
Also I don't know if you prefer to make the change yourself or let contributors submit PRs, let me know.
Best regards,
Ricardo

Great idea. Added in 641cf52. Let me know if you have any issues or feedback.


drivers: eswifi: Remove unused variable in ioctl routine

Variable "sd" is no longer referenced after refactoring in 2ed6b6a8ed.
It caused warning-as-error when building using sanitycheck, e.g. in CI.
This is 2nd fixed of the above refactor, after eb9a18432.
Signed-off-by: Paul Sokolovsky<EMAIL_ADDRESS>

Caught by Linaro CI, e.g. https://ci.linaro.org/view/lite-iot-ci/job/zephyr-upstream/5470/PLATFORM=disco_l475_iot1,ZEPHYR_TOOLCHAIN_VARIANT=zephyr,label=docker-xenial-amd64-13/console :
00:31:11.359 /home/buildslave/workspace/zephyr-upstream/zephyr/drivers/wifi/eswifi/eswifi_socket_offload.c:505:6: error: unused variable 'sd' [-Werror=unused-variable]
00:31:11.359   505 |  int sd = OBJ_TO_SD(obj);
00:31:11.359       |      ^~
00:31:11.359 cc1: all warnings being treated as errors
00:31:11.359 make[2]: *** [zephyr/CMakeFiles/zephyr.dir/drivers/wifi/eswifi/eswifi_socket_offload.c.obj] Error 1
00:31:11.359 make[1]: *** [zephyr/CMakeFiles/zephyr.dir/all] Error 2
00:31:11.359 make: *** [all] Error 2
00:31:11.359
00:31:11.360 INFO    - /home/buildslave/srv/zephyr/master/zephyr/disco_l475_iot1/disco_l475_iot1/samples/net/wifi/sample.net.wifi/build.log


tsserverlibrary.d.ts contains implicitAny error

TypeScript Version:  v3.7.0-dev.20191009

Search Terms:
Code
tsc --noEmit --strict node_modules/typescript/lib/tsserverlibrary.d.ts

Expected behavior:
No error
Actual behavior:
node_modules/typescript/lib/tsserverlibrary.d.ts:8559:21 - error TS7033: Property 'typingsCache' implicitly has type 'any', because its get accessor lacks a return type annotation.

8559         private get typingsCache();
                         ~~~~~~~~~~~~

Found 1 error.

On second thought this is probably not a problem with tsserverlibrary.d.ts. It seems the check for implicitAny getters should exclude private ones.

Related to https://github.com/microsoft/TypeScript/issues/33877

Nice catch @ajafff


Improve Manage Files ext: allow going to parent and sub folders by shortcuts

Description
This PR will add a new feature to Manage Files ext.
There are 2 new actions:

Go to Parent Directory has a shortcut Cmd + Shift + <-
Go to sub Directory  has a shortcut Cmd + Shift + ->

These actions will not push the new view into the stack, so next time users trigger the Manage Files ext,  users do not need to press Esc many times to go to the root.
If these actions push the new view into the stack, I believe, most of the time, users will lose their minds and be confused after navigating too many folders.

When the selected item is a file, subDirectoryPath is passed as null,
Go to sub directory action does not show,
Only Go to Sub Directory shows.

In this PR, I also upgrade code @raycast/api lib and migrate deprecated Raycast APIs to new Raycast APIs
Screencast
https://user-images.githubusercontent.com/138803/218255871-8e382a8f-a745-4bd4-a30c-1b250eb19468.mp4
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi @es183923 👋
Do you have time to give this a check?

LGTM

With the new update, I can get longer back than my start directory, which can cause a crash. Can you look into this?
Should we be able to go further back at all?
https://user-images.githubusercontent.com/38033107/220555684-2fce9568-4b41-4a33-b31b-1d710a8ece4c.mp4


Batched updates

@WebReflection would you consider batched updates, the way somilar to icaro?

As a PR? Possibily!
In general though, this project does its thing and I've never needed much else, so that if there are other libraries inspired by this project doing more already, I'm not sure there's a need to compete.


Unable to upgrade helm plugin

I forked helm-diff plugin and trying to upgrade it gives following error.
maverick@CARBON:~/.helm/plugins/helm-diff$ helm plugin install https://github.com/maver1ck/helm-diff/
Downloading https://github.com/databus23/helm-diff/releases/download/v2.9.0%2B2/helm-diff-linux.tgz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   609    0   609    0     0    396      0 --:--:--  0:00:01 --:--:--   397
100 4776k  100 4776k    0     0   530k      0  0:00:09  0:00:09 --:--:-- 1056k
Preparing to install into /home/maverick/.helm/plugins/helm-diff
helm-diff installed into /home/maverick/.helm/plugins/helm-diff/helm-diff
***
maverick@CARBON:~/.helm/plugins/helm-diff$ helm plugin update diff --debug
[debug] loading installed plugins from /home/maverick/.helm/plugins
Error: Failed to update plugin diff, got error (plugin does not exist)

Output of helm version:
Server: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}

Output of kubectl version:
Server Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.6", GitCommit:"9f8ebd171479bec0ada837d7ee641dec2f8c6dd1", GitTreeState:"clean", BuildDate:"2018-03-21T15:13:31Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):
AWS

I think the problem lies here:
maverick@CARBON:~/.helm/cache/plugins/https-github.com-maver1ck-helm-diff-$ git config --get remote.origin.url
https://github.com/maver1ck/helm-diff.git

The folder name is helm-diff.git and then helm is looking for this directory
[pid  2227] newfstatat(AT_FDCWD, "/home/maverick/.helm/plugins/helm-diff.git", 0xc4209cc788, 0) = -1 ENOENT (No such file or directory)
[pid  2227] write(2, "Error: Failed to update plugin d"..., 71Error: Failed to update plugin diff, got error (plugin does not exist)

OK. Found the reason of this problem.
PR is ready.


Extract properties from OData responses for better error reporting

Observation reveals that OData responses (responses for Table operations) have similar structures. We can present cleaner error details in the error details dialog if we can extract the useful properties and exclude the miscellaneous information.

Observation reveals that OData responses (responses for Table operations) have similar structures. We can present cleaner error details in the error details dialog if we can extract the useful properties and exclude the miscellaneous information.

If we are going to do this, we should make sure that the table APIs contractually guarantee the shape of the OData responses.


Button paypal does not show when setup called time second

The error from your example github
https://github.com/paypal/android-checkout-sdk/blob/main/quickstart-kotlin/src/main/java/com/paypal/checkoutsamples/paymentbutton/PaymentButtonQuickStartActivity.kt
This video: https://drive.google.com/file/d/1KfvUy5U011JMngmXf7-V-_8yH_PgBMfW/view?usp=sharing

@jcnoriega

I was having the same issue until i downlgraded to 0.20 version
implementation'com.paypal.checkout:android-sdk:0.2.0'

But when I downlgraded to 0.20 version, all the http requests can not receive response, so strange @faiz106

Hi! Thank you so much for reporting. A pull request to fix this has been created and it will be included on the next release next week.
The bug happens when PayPalCheckout.setConfig() is called in the Application class. A quick workaround this, is to call setConfig() on the same activity that renders the PaymentButton.

Hello, @jcnoriega ，I remove setConfig() in the Application  and call setConfig() on the same activity that renders the PaymentButton, but it stiil can't work. If I pay the bill, next time I enter the activity and error happens. if not, things go well.

Version 0.4.5 is deployed with the fix. Please confirm if issue persists

Problem solved !

resolved, closing issue


Assignment best practices?

Apologies for so many issues, this one is more of a discussion on best practices tho.
I keep running into issues like this when trying to model my game using Keystone:
import { model, Model, prop, modelAction } from "mobx-keystone";
import { Point2D } from "../src/model/geometry/point2D";

interface Position {
  x: number;
  y: number;
}

@model("TestModel")
class TestModel extends Model({
  position: prop<Position>(),
}) {
  @modelAction
  applyEvent(event: { position: Point2D }) {
    this.position = event.position;
  }
}

test(``, () => {
  const modelA = new TestModel({ position: { x: 3, y: 3 } });
  const modelB = new TestModel({ position: { x: 0, y: 0 } });

  // The below throws an error "an object cannot be assigned a new parent when it already has one"
  modelB.applyEvent({ position: modelA.position });
});

Now this error is thrown because position has been converted into an observable and has been parented under modelA and so errors when it assigned in modelB.
I keep running into issues like this, and im not 100% sure what to do about it.
I guess this is a mutable / immutable issue? Maybe I could somehow flag postion as immutable and so the above would work?

Basically what happens is this:
const modelA = new TestModel({ position: { x: 3, y: 3 } });
modelA tree now "owns" the object (now a tree node)  PA { x: 3, y: 3}
 const modelB = new TestModel({ position: { x: 0, y: 0 } });
modlB tree now "owns" the object (now tree node) PB { x: 0, y: 0 }
modelB.applyEvent({ position: modelA.position });
now you try to move into the modelA tree the object (tree node) PB, but that node is already attached to model B and a same tree node cannot be attached to two different trees at the same time (therefore the error).
There are some ways to avoid this:

cloning the object before passing it to the action (either using clone (only for tree nodes)) from mobx-keystone or the usual js shallow clone { ...point })
cloning inside the action itself

Yep pretty much what I thought.
Okay thanks!

There are some ways to avoid this:
1. cloning the object before passing it to the action (either using clone (only for tree nodes)) from mobx-keystone or the usual js shallow clone `{ ...point }`)

2. cloning inside the action itself

I'm also running into this, just to gain some clarity into the suggestion you made @xaviergonz;

are you suggesting this because PA and PB are not models?
if they were models would you instead suggest that references be used?


作者你好，我来提一个需求：

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the feature you'd like
A clear and concise description of what you want to happen.
Additional context
Add any other context or screenshots about the feature request here.

mistake

close


How to pass header attributes?

You can send one using 'AVURLAssetHTTPHeaderFieldsKey' in AVPlayer. Does VersaPlayer support. If yes, how? Please guide with example.
Cheers.

Hi @khunshan ,
Sure!, AVPlayerItem can be initialized with a AVAsset, so you can pass a AVURLAsset configured with the proper headers.
let headers: [String:String] = [:]
let asset = AVURLAsset(url: url, options: ["AVURLAssetHTTPHeaderFieldsKey": headers])
let item = VersaPlayerItem(asset: asset)
playerView.set(item: item)


Madhava/fix torch 1.6.0

Description
This PR lowers the torch requirements to torch==1.6.0.
Affected Dependencies
None
How has this been tested?
Tests were failing on python 3.6 and now they pass.
CI for all three python versions has been added.
Checklist

[x] I have followed the Contribution Guidelines and Code of Conduct
[x] I have commented my code following the OpenMined Styleguide
[x] I have labeled this PR with the relevant Type labels
[x] My changes are covered by tests

I think python 3.6 is possible: https://idolstarastronomer.com/two-futures.html
Code would look something like this but I don't have much asyncio experience so I will park this for the moment.
if sys.version_info >= (3, 7):
    ex_kwargs["initializer"] = initializer
    ex_kwargs["initargs"] = (loop,)

    with Executor(**ex_kwargs) as executor:
        for i in range(nr_parties):
            _args = args[i]
            _kwargs = kwargs.get(i, {})
            futures.append(executor.submit(funcs[i], *_args, **_kwargs))

else:
    with Executor(**ex_kwargs) as executor:
        for i in range(nr_parties):
            _args = args[i]
            _kwargs = kwargs.get(i, {})
            futures.append(
                await loop.run_in_executor(executor, funcs[i], *_args, *_kwargs)
            )

I think python 3.6 is possible: https://idolstarastronomer.com/two-futures.html
Code would look something like this but I don't have much asyncio experience so I will park this for the moment.
if sys.version_info >= (3, 7):
    ex_kwargs["initializer"] = initializer
    ex_kwargs["initargs"] = (loop,)

    with Executor(**ex_kwargs) as executor:
        for i in range(nr_parties):
            _args = args[i]
            _kwargs = kwargs.get(i, {})
            futures.append(executor.submit(funcs[i], *_args, **_kwargs))

else:
    with Executor(**ex_kwargs) as executor:
        for i in range(nr_parties):
            _args = args[i]
            _kwargs = kwargs.get(i, {})
            futures.append(
                await loop.run_in_executor(executor, funcs[i], *_args, *_kwargs)
            )

@madhavajay does this work? (it should be tested with the notebooks from the SMPC folder from Syft)

@madhavajay does this work? (it should be tested with the notebooks from the SMPC folder from Syft)

Yes, the latest 0.4 branch with this branch and torch 1.6.0 works on the examples/secure-multi-party-computation/Duet/1-DS-1-DO notebooks.
$ pip freeze | grep torch
torch==1.6.0
torchcsprng==0.1.2
torchvision==0.7.0

Yes, the latest 0.4 branch with this branch and torch 1.6.0 works on the examples/secure-multi-party-computation/Duet/1-DS-1-DO notebooks.
$ pip freeze | grep torch
torch==1.6.0
torchcsprng==0.1.2
torchvision==0.7.0


chore(readme.md): updated docs for docker images generation.

What type of PR is this?
/kind documentation

Any specific area of the project related to this PR?
/area ci

What this PR does / why we need it:
Documentation adjustment.
Which issue(s) this PR fixes:

NONE
Special notes for your reviewer:
NONE

@cappellinsamuele: The label(s) kind/documentation cannot be applied, because the repository doesn't have them.

In response to this:

What type of PR is this?
/kind documentation
Any specific area of the project related to this PR?
/area ci
What this PR does / why we need it:
Documentation adjustment.
Which issue(s) this PR fixes:
NONE
Special notes for your reviewer:
NONE

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.

LGTM label has been added.  Git tree hash: d07a2c7825e1d9db10c0cff1da1a7c128aa896ad


chore: Add use strict to all except fixtures

Coverage remained the same at 100.0% when pulling 284aa3b9ffcf26b744dd02f2de9b32ccc7837d02 on coreyfarrell:use-strict into bcbe1dfa1febcb7d5c53c065a16e8caf861fd4bd on istanbuljs:master.


Website not loading?

Hey,
Wing is one of the best framework in the market... But the website isn't working for me anymore. Do you know why?

Hello,
I agree, and experience the same here – website doesn't load. It'd be nice to keep it alive!

Hey!
The domain (along with domains for lots of my other projects) got taken over by advertisers since the name was free. The new website should be available at kbrsh.github.io/wing.

No problem! Sorry for the inconvenience.


Increase pull request pipeline size

Closes #763
Signed-off-by: Chmouel Boudjnah<EMAIL_ADDRESS>
Changes

Submitter Checklist

[ ] ♽  Run make test lint before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI
[ ] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[ ] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[ ] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[ ] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

Codecov Report

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

@@           Coverage Diff           @@
##             main     #764   +/-   ##
=======================================
  Coverage        ?   65.52%
=======================================
  Files           ?       75
  Lines           ?     4487
  Branches        ?        0
=======================================
  Hits            ?     2940
  Misses          ?     1267
  Partials        ?      280

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 2eec244...588c170. Read the comment docs.


LDAP error in views connecting

trying to auth using ldap results in
lemur[13127]: [2020-02-27 14:20:04,394] ERROR in views: ldap error: list indices must be integers or slices, not str

this is python 3.6...perhaps an issue?

What can be done?


Feature request: Search in the scrollback output?

I could not find any way to do this. I use search all the time and would really appreciate having this functionality. I would ideally like Ctrl-Shift-F to initiate this.

This is fairly high on the features TODO list.

the lack of this feature is currently a deal-breaker, when looking at long logs files, which pushes me to use some-other terminal emulator

@ChicK00o thanks. It is useful for me to have an idea as to what people need the most just so that I can figure out some reasonable priorities.

"Search in text" is pretty high on the TODO list now. I'm trying to clear out these kinds of deal-breakers. See #30

Thank you so much @sedwards2009. How can we help support you?

@WillPlatnick
Here is a quick list of ways to support this project:

fixing bugs
improving docs
promoting Extraterm
blogging about how you use is and the unique features, explaining interesting workflows.
Good old fashioned programming is useful too.
Answering questions which appear as issues on GitHub is directly useful. It saves me time.
Clearly reporting bugs.
Giving feedback about which (missing) features you feel are important and also which ones are not important. I find it hard to prioritise features at times. For example. does anyone care about cursor shapes?
General constructive and "actionable" criticism. (macOS specific stuff is especially useful because that is a platform I don't regularly use.)

@sedwards2009 Thanks. Would you welcome any financial support?

I also can't wait for this feature :) @sedwards2009 Do you have an estimate on when the release involving this feature is going to be?

@kdziadowiec  Since the last release I've spent the time improving the internal structure of the app and specifically introducing a 'command' concept. This stuff is exposed in the Extensions API which is growing like a fetus inside Extraterm. :-) This will allow me to implement Find correctly(as an extension) in a way which won't need to changed in the future and won't cause the code base to become a mess. It will also allow other functionality to be created much easier, and once the Extension API is born, then others will be able to build on that foundation work.
That said, I am now directly working on Find.

This will be in 0.41.

thanks a lot. Back to using this :D


cannot execute eks-managed-node-group module when using sub modules

Description
I am trying to create eks-managed-nodes by calling the eks-managed node modules inside the node submodule main.tf file. This node module gets called in the module eks main.tf file.  The eks-managed-node module call inside nodes module main.tf has a for_each loop in it to create multiple node groups for the same cluster . The node_groups are supplied from root vars.tf with variable type map(map(dynamic)) with each sub map containing configuration for a single node group hence generating multiple node_groups
i am creating the node role in iam module and setting the create_iam_role to false .  The entire eks module is called in the root main.tf file.
below is the folder structure.

├── main.tf
├── modules
│   ├── eks
│   │   ├── main.tf(where node module is called)
│   │   ├── modules
│   │   │   ├── cluster
│   │   │   │   ├── main.tf
│   │   │   │   ├── output.tf
│   │   │   │   └── vars.tf
│   │   │   ├── iam
│   │   │   │   ├── main.tf
│   │   │   │   ├── outputs.tf
│   │   │   │   └── vars.tf
│   │   │   ├── nodes
│   │   │   │   ├── main.tf (where eks-managed-node module is set)
│   │   │   │   └── vars.tf
│   │   ├── outputs.tf
│   │   └── vars.tf
├── outputs.tf
├── providers.tf

I am getting the following error on plan and apply

If i am making the same call from root main.tf module which changing any code inside the node module then the execution works perfectly .
Versions

Module version [Required]:

Terraform version:
1.5.2

Provider version(s):
aws: 5.10.0
kubernetes: 2.22.0

Reproduction Code [Required]

Steps to reproduce the behavior:
No
yes

Destruction if a node group in eks module lead to destruction of addons too hence created cluster myself and used the eks-managed-nodes module for node creation .
Created an eks cluster in cluster module without nay nodes.
Used nodes module for creation of nodes .
calling nodes module in following path.(./modules/eks/main.tf) alog with cluster module.

Expected behavior
clean run with terraform plan or apply
Additional context
there are multiple node groups to be created so for_each is being used in the eks-managed-node module call with iam role being passed as a variable and create_iam set to false.
How do i get the node call working from inside ./modules/eks/main.tf

if you have a depends_on, you will need to remove it


RND-40: Rename pom.xml properties, project packages and document take…

The following PR contains changes related to the initial stage of developing skeleton-based application.

Closing this PR as it isn't supposed to be merged to master.


Fix(esbuild): add options to plugins wrapper

Follow up to https://github.com/macaron-css/macaron/pull/68. Overlooked the crucial moment.
We need to provide options to plugins wrapper macaronEsbuildPlugins, not only plugin macaronEsbuildPlugin alone. Because macaron also needs to load vanilla-extract plugin by default. So one can use in esbuild config:
{
  plugins: [
    ...macaronEsbuildPlugins({
      includeNodeModulesPattern: /.../
    }),
  }
}

For now workaround is:
const macaronPlugins = () => ([
  macaronEsbuildPlugin({
    includeNodeModulesPattern: /\.tsx?$/,
  }),
  macaronEsbuildPlugins().at(-1), // vanilla-extract
]);

thanks! should be published on npm too


Textual definition update: retinal binding

Guidelines
See http://wiki.geneontology.org/index.php/Guidelines_for_GO_textual_definitions#Use_genus-differentia_patterns_for_definitions
More details at https://douroucouli.wordpress.com/2019/07/08/ontotip-write-simple-concise-clear-operational-textual-definitions/

GO term ID and label for which you request a definition update
GO:0016918 retinal binding

**New proposed definition (genus-differentia pattern, plus additional information if needed) **
Binding to retinal, one of the forms of vitamin A.

Reference(s)
PMID:15774036

Additional information
Binding (GO:0005488) by definition should be non-covalent interactions. But opsin-retinal interaction is covalent. Thus this term should exclude opsins in the definition and synonyms. Note opsin-retinal interaction in the literature is often referred to as binding.

This term retinal binding has been incorrectly used for annotations. The LD is binding and ('has input' some retinal) but there is mentioning of opsin, which is covalently bonded to retinal, in the text definition and in synonyms. Any advice on what is the best course forward to disentangle? @pgaudet Thanks.

This paper talks about covalent binding of Rhodopsin and its ligand
https://pubmed.ncbi.nlm.nih.gov/20042594/

obsolete with 'consider'  GO:0008020 G protein-coupled photoreceptor activity
and modify the definition of GO:0008020 to describe that the chromophore binds covalently (and maybe some mention of the rhodopsin cycle?)
And please replace the definition by more specific one(s)

A ref for noncovalent binding: PMID:11301033 Role of noncovalent binding of 11-cis-retinal to opsin in dark adaptation of rod and cone photoreceptors

There are other retinal binding proteins such as Interphotoreceptor retinoid-binding protein (IRBP) PMID:17222825.
In sum, I think "retinal binding" is a valid MF, despite the fact that in some instances (opsins) non-covalent binding precedes covalent attachment.


Update install.py to add .exe for windows

Update install.py to use '.exe' extension on windows so that deno can run successfully.  Addresses issue https://github.com/denoland/deno/issues/1429

Did this patch not work?

Worked, but tests failed.   Figured that I don't know enough about why it is deno vs deno.exe - i.e. what was the rationale for originally not using .exe for windows in the first place.  So instead of changing the tests for something I have minimal knowledge about, I've abandoned this PR instead.


Sensitive key data is now hidden

This is in reference to fix #6529.
Set no_log: true for all the updated files , so that sensitive information are not logged out in the ansible output, thereby increasing security and privacy.

There seems to be an issue with pull request regarding Signing of commit.
Could you please guide me as to what should be my next step, since I have rectified it later by creating a signed commit.

First you need to make sure all your commits are "verified". I'd say the easiest way is to just is to redo the commits as a single verified commit.
run git rebase -i HEAD~10, squash only your commits into a previous commits, to basically combine them
https://www.internalpointers.com/post/squash-commits-into-one-git
and do a new verified commit, then push into your fork with git push -f
Once test pass, you need to wait for it to be approved/commented on by a maintainer.


Importing Steam games

Gem for the Steam API: https://github.com/bhaberer/steam-api
Gem for Steam Omniauth: https://github.com/reu/omniauth-steam
We need to figure out how to link between GiantBomb and Steam, Wikidata is a potential option.
Unfortunately it doesn't look like the GiantBomb Wiki includes the Steam URL or ID anywhere, so this may be pretty difficult :/

GetOwnedGames is going to be the endpoint I want to use: https://partner.steamgames.com/doc/webapi/IPlayerService#GetOwnedGames

Still not 100% sure this works, but it's worked on the two accounts I tested 🤷♂ Need to test it with someone that isn't Steam friends with me.


icons on custom legends

close https://github.com/CartoDB/cartodb/issues/10726
steps to test this:

create a map, add a custom legend, and pick an icon in any of the items
check the icon (and the color) changes both in Builder and the map

this needs to be reviewed and merged first:

[x] cartodb.js support https://github.com/CartoDB/cartodb.js/pull/1545

Let's fix the conflicts.

YAY

Frontend test passing 🚀


DefaultToSelect

Currently an empty list is returned when select is not specified in the query, because no properties are selected. It would be great to specify a default value for select when select query parameter is not given.
[AutoQueryable(DefaultToSelect = "_")]
[AutoQueryable(DefaultToSelect = "*")]
[AutoQueryable(DefaultToSelect = "id,name")]

Hi @henkosh,
It will be in the next release :).


feat: lwseverity package

Summary
As a Lacework developer I would like a package to make working with Lacework severities easy as pie (or tacos).

Normalizing
Sorting
Filtering

How did you test this change?
Unit testing
Issue
https://lacework.atlassian.net/browse/ALLY-1263


I was wondering whether the pageIcon can be nil or not? Because informationImage is enough for me

PageIcon is not optional

But i did it by using this method UIImage()


dataset

could you share the dataset *_ct/_pt.nii.gz' files?

You should directly contact the challenge organizers to access the dataset. See README.md.


Synth-Liz Jaw

About the PR
Tiny PR to add a jaw section to the synthlizard snout
Why / Balance
I got pinged over this
Technical details
+1 RSI, YAML line, FTL line, JSON line,
Media

Requirements

[ ] I have read and I am following the Pull Request Guidelines.


refactor: deprecate admit tactic

The admit tactic is a synonym of sorry and can be deprecated.
Thanks to eric-wieser for suggesting the implementation of the deprecation warning.
Closes #4901

Closed for now. RFC will continue.


sqlite3.OperationalError: unable to open database file

Hi,
The Fuck 3.29 using Python 3.9.7 and ZSH 5.8
Traceback (most recent call last):
File "/usr/bin/thefuck", line 13, in
sys.exit(main())
File "/usr/share/thefuck/thefuck/entrypoints/main.py", line 26, in main
fix_command(known_args)
File "/usr/share/thefuck/thefuck/entrypoints/fix_command.py", line 41, in fix_command
corrected_commands = get_corrected_commands(command)
File "/usr/share/thefuck/thefuck/corrector.py", line 89, in get_corrected_commands
corrected for rule in get_rules()
File "/usr/share/thefuck/thefuck/corrector.py", line 48, in get_rules
return sorted(get_loaded_rules(paths),
File "/usr/share/thefuck/thefuck/corrector.py", line 17, in get_loaded_rules
rule = Rule.from_path(path)
File "/usr/share/thefuck/thefuck/types.py", line 140, in from_path
rule_module = load_source(name, str(path))
File "/usr/lib/python3.9/imp.py", line 171, in load_source
module = _load(spec)
File "", line 711, in _load
File "", line 680, in _load_unlocked
File "", line 850, in exec_module
File "", line 228, in _call_with_frames_removed
File "/usr/share/thefuck/thefuck/rules/apt_get.py", line 13, in
_get_packages = CommandNotFound.CommandNotFound().get_packages
File "/usr/lib/python3/dist-packages/CommandNotFound/CommandNotFound.py", line 79, in init
self.db = SqliteDatabase(dbpath)
File "/usr/lib/python3/dist-packages/CommandNotFound/db/db.py", line 12, in init
self.con = sqlite3.connect(filename)
sqlite3.OperationalError: unable to open database file
Did I miss something or is there a bug?
I tried with apt and new version from pip. The same story.
This issue is only on a normal user account, on root there isn't a problem.

Correct me if I'm wrong, but I don't think this is a bug in thefuck. It seems to be a known bug in CommandNotFound instead: https://bugs.launchpad.net/command-not-found/+bug/1824000
A temporary workaround is to fix the db file permissions manually:
sudo chmod ugo+r /var/lib/command-not-found/commands.db*

With no participation since nearly 2 years, and good directions to go further, maybe this issue can be closed ?


Remove usage of the flag experimental_allow_proto3_optional

Optional fields in the proto3 syntax are now stabilised. This flag has been removed in newer version of protoc and now causes issues building tests.

--experimental_allow_proto3_optional is required for supporting  protoc version 3.12. That version is still in use in Ubuntu 22.04.
Modern versions of  protoc ignore the argument for backwards compatibility.
So I think we should leave this in place


Pattern for clean AMI image based upgrading of firewall manager, upgrade LZ0

Describe the issue
CheckPoint has announced a new version of the Firewall manager.  We want to upgrade but we don't like the "reuse the existing VM and click upgrade in the UI" approach.  We prefer to IaC the upgrade and roll out a fresh AMI VM.  The issue is we need to figure this out.
Additional context

Ensure that the configuration is not lost
The AMIs are configured by the SEA, although we may want to consider terraform

Definition of done

Talk to Ryan Jaeger and see if there was any consideration for AMI upgrades.  We will likely need to provide feedback upstream.
Back up the firewall and firewall manager configurations, for reapplication on the new instances (include policies in the backup)
Update the SEA config, re-run the state machine
Create a pattern for re-running some automation to get a fresh updated VM AMI firewall manager appliance
Consider this pattern for the firewalls too, and think forward to how this could affect policies updates that we have consumed from secops (we don't want to lose them)
Use AWS ECF Dev for developing this pattern
Upgrade the Check Point Firewall Manager in LZ0
Document the pattern in private cloudops-internal in markdown and review with Carles
Look at linked ticket, create extra well formed tickets if needed
Timebox to a 3

Still relevant


style of the sharing table; link to the group/{group_id} page

Clean up the sharing sub table
Link group to the /groups/{id}

This looks good. I just see an error for codegen, but aside from that this is good to go.


Modal needs to update aria-hidden attribute after modal is shown

Hello,
V4 Alpha:
It appears that the aria-hidden="true" boolean should be updated to true once modal fires. This is holding some Assistive Technology back from being able to access the content.
http://v4-alpha.getbootstrap.com/components/modal/
Other than that this modal is really well done! Hats off :)

@joe-watkins Did you mean "to false" instead?
CC: @patrickhlauke

Need to remove the aria-hidden attribute completely because the dialog content is a child of that aria-hidden element.
http://pauljadam.com/demos/dialog-DOM.html Need to set aria-hidden on a container that holds all the main content and disable focusability of the main content or some other method of trapping focus in the dialog. Thanks!

tnx @cvrebert updated :)
@pauljadam Thanks for finding this issue and helping! ;)

The aria-hidden attribute issue is now fixed by https://github.com/twbs/bootstrap/pull/19879
Basic focus trapping is already implemented in the modal. The bigger aspect of having all content outside of the modal wrapped in an aria-hidden=true container when modal is visible (already discussed at various points in other related issues) will be handled separately at a later stage (as this will require some more careful refactoring of current markup, CSS and JS)


unvanquished: fix sdl event overflow

Fixes https://github.com/DaemonEngine/Daemon/issues/600
Things done

Built on platform(s)

[x] x86_64-linux (the only supported platform)

[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

can someone add the backport release-22.05 tag?


In windows react-native run-android not starts node

React Native version:0.60.rc

Steps To Reproduce
1.react-native init RN060
2.react-native run-android

Describe what you expected to happen:
Open node and install aplication in emulator

You need to have the emulator open before you run react-native run-android

Uhm, so you are not seeing any error?
Also, this seems to be related to the CLI, which now lives in its own repo, can you post this issue there?
https://github.com/react-native-community/cli

Same problem in this vesion and crash node after start build gradlew
he open node app but close after start build gradlew  and I need run 'react-native start'

react-native-cli: 2.0.1
react-native: 0.60.0


Fill .gitignore; stop tracking Cargo.lock

Hello,
A little contribution, while I'm trying to see what I can do to help. Good idea for a crate !

@jacquetc Thank you for your interest in AccessKit. However, I am tracking Cargo.lock on purpose. The AccessKit project will eventually include bindings for other languages. When it does, we'll offer prebuilt binaries, and tracking Cargo.lock will make sense. Tracking Cargo.lock also lets us avoid surprises in CI. So please restore Cargo.lock and remove that entry from your .gitignore.

Done :-) I'll not be the first to point that out. It may be useful to add this bit of information somewhere in the doc ?