Deleted ExternalStructure subclasses reappear as zombies in external callbacks arguments

Describe the bug
As far as I can tell, deleted definitions of external structure classes can get partially resurrected if a new class has been created with the same name. This results in weird behavior like different class definitions being used in different contexts. And in each case, the deleted class appears as a hollowed out version of the real class with an empty method dictionary.
I noticed this after uninstalling and reinstalling a package with many ExternalStructure definitions. New objects created with the reinstalled ExternalStructure classes were throwing #doesNotUnderstand: errors even though the method definitions were visible, both in the class browser and in the method dictionary of the class. Upon closer inspection, I noticed that #instanceSpec returned a different value for the ExternalStructure class when I ran it in a workspace vs. on an external structure passed as an argument to a MessageCallback.
To Reproduce
Steps to reproduce the behavior:

Open fresh image
Create a subclass of ExternalStructure named ZombieStructure
Add the following method to the ZombieStructure class:

ZombieStructure class>>defineFields
	self defineField: #test type: LPVOIDField new

Run ZombieStructure compileDefinition
Delete the ZombieStructure class
Run Metaclass allInstances select: [ :each | each instanceClass name = #ZombieStructure ]
#(ZombieStructure class) is returned rather than an empty array

Expected behavior
Deleted definitions of ExternalStructure classes should not be used for new instances.
Please complete the following information):

Dolphin v7.1.24
Windows 10

With regards to your repro, the class object is still referenced from the history list of the browser. If you close the browser, it will go away. Or at least it did when I tried it. If you you can run the following to see why it is still referenced:
ReferenceFinder findAllPathsTo: (Metaclass allInstances select: [ :each | each instanceClass name = #ZombieStructure ]) first

It would be better if the history list stored a symbolic reference rather than a hard reference, but it isn't related to your issue. However, I do remember encountering the problem.
Your issue is almost certainly caused by one or more of the following:

Pre-existing callback instances might still be in scope after you deleted the structure classes. These hold a reference to the class they were originally instantiated against. You can't expect that they will update to a new version of the class if you delete and recreate it (although they would if you changed the definition of the class). You'll need to make sure that uninstalling the package tears down any callback instances somehow, either from class uninitialize methods, or a package uninstall script as a last resort (since these are not visible to the tooling).
You are "referencing" the structure classes from strings. This is quite likely because unfortunately the ExternalDescriptor class encourages this by providing a parser that takes a string as input. These references embedded in strings are not visible to the package system or other tooling, so it does not recognise them as a prerequisite dependency. As a result you can uninstall the package with the structures without the system requiring that you also uninstall the package with the methods that create or define the callbacks, leaving them in an invalid state (referring to obsolete classes). Also, these references will not be visible to the application deployment image stripper either, so there is a risk it will remove classes that are required for callbacks.
Even more unfortunately, ExternalDescriptor in 7.1 (and earlier) has a cache of shared instances. This is a weak collection, but in the circumstance of (1) above, the old instances would prevent the cache entry referring to the old class going away. This would mean that when you ask for an ExternalDescriptor with the same string, the old definition that refers to the old class will get returned. It shouldn't happen if the code with the dependent callbacks is uninstalled with the structures, but as mentioned the design that encourages hidden class references in strings makes it easy to fall foul of this problem.

If you ensure that there are no callback instances existing that refer to the old class definition, and you re-execute whatever code parses the callback descriptor, then you should not see the problem. You can make it much less likely by ensuring that you construct any callback descriptor strings you parse using an actual reference to the class, as this will mean the referring code will have to be uninstalled at the same time you uninstall the structures, so the shared descriptor instances should go away. I would also recommend disabling the ExternalDescriptor shared instance cache. This can be done by setting the class variable ExternalDescriptor.Shared to nil.
An example of what not to do is in this DisplayMonitor class>>forCanvas:intersecting:do: method from Dolphin 7.1
https://github.com/dolphinsmalltalk/Dolphin/blob/48f8321cb370664717dc5f6657368f707df9c163/Core/Object Arts/Dolphin/MVP/Base/DisplayMonitor.cls#L194
In Dolphin 8 the hidden class reference is avoided like this
https://github.com/dolphinsmalltalk/Dolphin/blob/ca05ce6fd96df3729f38e114caf4970204498f1a/Core/Object Arts/Dolphin/MVP/Base/UI.DisplayMonitor.cls#L197
This means that the RECTL class appears in the literal frame of the method, and so is visible to reference searches and package prerequisites tracing. The ##() expression makes it a constant to avoid the cost of rebuilding the string every time, but importantly the literal reference to RECTL is retained. Doing that (or making the descriptor itself a constant, or storing a shared instance in a class variable) with the class name in a string would just recreate the stale reference problem of ExternalDescriptor.Shared.
I'll send a a fix to remove the shared instance cache in 7.1. I removed it from D8 a couple of years ago now in 619d810404b136b83b327bc1e6064d6c8501cf30.

If you you can run the following to see why it is still referenced:
ReferenceFinder findAllPathsTo: (Metaclass allInstances select: [ :each | each instanceClass name = #ZombieStructure ]) first

Very handy. I did try using #inspectReferences but got lost in the rabbit hole. This is much better.

However, I do remember encountering the problem, and your issue is almost certainly caused by one or more of the following:

I checked and it was indeed a combination of 1 and 3. Thanks!


The model text-davinci-003 has been deprecated

{
"error": {
"message": "The model text-davinci-003 has been deprecated, learn more here: https://platform.openai.com/docs/deprecations",
"type": "invalid_request_error",
"param": null,
"code": "model_not_found"
}
}
It gives this error. Any solution?

Try gpt-3.5-turbo-instruct


api-management-howto-integrate-internal-vnet-appgateway: how to set password for pfx files?

[Enter feedback here]
I appreciate it if you could clarify a little how to specify the passwords.

$gatewayCertPfxPassword = "certificatePassword123"   # Password for api.contoso.net pfx certificate
$portalCertPfxPassword = "certificatePassword123"    # Password for portal.contoso.net pfx certificate
$managementCertPfxPassword = "certificatePassword123"    # Password for management.contoso.net pfx certificate

The pfx file does not have a password when downloaded from Key Vault.
https://docs.microsoft.com/en-us/azure/key-vault/certificates/quick-create-portal#export-certificate-from-key-vault
I tried to set passwords by the following steps.
https://mcilis.medium.com/how-to-change-the-password-of-a-pfx-file-ba46cb193dfe
However, the following error occured.

Is this what is supposed to do?

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

ID: 37c1f253-44c2-bd07-4c74-9d4c77c45e6e
Version Independent ID: 281ce98c-115b-3c39-52b7-3df95dfd8b52
Content: Use API Management in a virtual network with Azure Application Gateway - Azure API Management
Content Source: articles/api-management/api-management-howto-integrate-internal-vnet-appgateway.md
Service: api-management
GitHub Login: @dlepow
Microsoft Alias: danlep

@HiroakiMachida Thank you for your feedback! We will review and update as appropriate.

@HiroakiMachida Thanks for reaching out
From the given details I understand that you're trying to integrate App Gateway with Api Management according to this  document.
currently you're trying to setup the custom domain names for your APIM endpoints where you need to provide .pfx file. I assume that you generated a self signed certificate using key vault and trying to download that cert in .pfx format from key vault correct me if am wrong here.
when you try to assign a .pfx file from key vault to your custom domain you need not to provide a password please refer this  document.
Also refer this blog which explains step by step process of integrating App Gateway with API Management.
let me know incase of further queries, I would be glad to help you.

@HiroakiMachida Just following up here. Hope my previous comment clears things up.

@HiroakiMachida Hope the above helps and clears things up. We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


PR: Update core dependencies for 4.2.2

Description of Changes
This changes the following requirements in preparation for 4.2.2:

spyder-kernels -> >=1.10.2,<1.11.0
pyls-spyder -> >=0.3.2

Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @ccordoba12

no possible upgrade for jedi ?


Include configuration value need_patient_banner in launch params

Pass need_patient_banner to fEMR launch apps.
done: Add config var for an environment to control need_patient_banner setting.
done: Pass need_patient_banner adjacent to other launch query string parameters.

@achen2401 this looked like it slipped through the cracks, but when I rebase with develop to sort out the merge conflict, I see the changes have already been added to the work you did WRT getAppLaunchURL?  I think we can safely close this PR, unless I'm missing something?  Thanks.

Thanks @pbugni .  Yeah, I believe we can close this PR.

@pbugni Please ignore my previous comment. I realize that we should merge this because we still need config var for an environment to control need_patient_banner setting.  We probably should merge open PR's related to this:
For COSRI environment: https://github.com/uwcirg/cosri-environments/pull/83
For DCW environment: https://github.com/uwcirg/dcw-environments/pull/27
I did resolve the conflict related to PatientListTable.js that was blocking the merge of this.

Thank you @achen2401


workspace focus change doesn't work with Sonoma 14.2

after the upgrade yabai switch workspaces stopped working. Desipite having sudo yabai --load-sa, yabai -m space --focus (prev|next|[0-9]) is not working.
I have tried to restart the service or gain insight into what's going on with running yabai in verbose mode, but no dice. I have yabai installed from brew and it's currently at yabai-v6.0.1 version.
For some reason it seems that yabai --load-sa doesn't work:
% sudo yabai --load-sa; echo $?
1
%

There was the exact same issue in some 14.1.x which was only addressed by a yabai update, so may require another update?

Guys, please read/search the issues. After every update it's expected to have some problems: https://github.com/koekeishiya/yabai/issues/2007

closing in favor of https://github.com/koekeishiya/yabai/issues/2007 and now i know what sa stands for in --load-sa. Still a bit unclear on how the connection between workspace focus and script-addition should have been made, but thanks for pointing it out @thiagomeireless


[Package Issue]: Python.Python override causes installer to ignore scope

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Installation issue.
Brief description of your issue
Using overrides makes it impossible to install Python system-wide.
Steps to reproduce
winget install --id Python.Python.3.11 --scope machine --override '/quiet'

Actual behavior
Python is installed in %LOCALAPPDATA%\Programs.
Expected behavior
Python is installed in %PROGRAMFILES% .
Environment
Windows Package Manager v1.7.11261
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19045.4412
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.22.11261.0

Winget Directories
-------------------------------------------------------------------------------------------------------------------------------
Logs                               %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir
User Settings                      %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\settings.json
Portable Links Directory (User)    %LOCALAPPDATA%\Microsoft\WinGet\Links
Portable Links Directory (Machine) C:\Program Files\WinGet\Links
Portable Package Root (User)       %LOCALAPPDATA%\Microsoft\WinGet\Packages
Portable Package Root              C:\Program Files\WinGet\Packages
Portable Package Root (x86)        C:\Program Files (x86)\WinGet\Packages
Installer Downloads                %USERPROFILE%\Downloads

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Admin Setting                             State
--------------------------------------------------
LocalManifestFiles                        Disabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Disabled
LocalArchiveMalwareScanOverride           Disabled

Screenshots and Logs
No response

The scope is controlled by the InstallAllUsers switch from the installer, which would be overrode by your own installer switches by using the --override switch from WinGet.
https://github.com/SpecterShell/winget-pkgs/blob/713c2747ac9bde2ef1bb0dec0f30aad06be6ac30/manifests/p/Python/Python/3/11/3.11.9/Python.Python.3.11.installer.yaml#L42
It is recommended to use the --silent switch from WinGet for silent (without progress) installation.

\quiet was just for a minimum reproducible example. The full command is as follows:
/quiet AssociateFiles=0 CompileAll=0 PrependPath=1 Shortcuts=1 Include_doc=0 Include_debug=0 Include_dev=0 Include_exe=0 Include_launcher=0 Include_lib=1 Include_pip=1 Include_symbols=0 Include_tcltk=0 Include_test=0 Include_tools=0'

--silent alone simply isn't enough. What is the reason it is recommended?

No I should be sorry, I wasn't clear in explaining my intent in the initial example.
Thanks very much for the explanation, this makes much more sense now. Is there a technical reason winget discards Custom switches when --overrides is used? Can't they be appended? At the very least, a warning before proceeding with the install that the --scope machine of my install command will be ignored would be helpful.

No I should be sorry, I wasn't clear in explaining my intent in the initial example.
Thanks very much for the explanation, this makes much more sense now. Is there a technical reason winget discards Custom switches when --overrides is used? Can't they be appended? At the very least, a dialog before proceeding with the install informing the user that the --scope machine part of the install command will be discarded would be helpful.

It not only discards the Custom switch, but also all other switches, including the InstallLocation switch above, the pre-defined Silent switch (/quiet /norestart in this case), the pre-defined SilentWithProgress switch (/passive /norestart) and the pre-defined Log switch (/log "<LOGPATH>"). All of them will be overrode by --override.
The --custom WinGet switch will only override the custom switch. Since this switch has already been used for forcing the scope for the installer and will be overrode by --custom, you need to manually include InstallAllUsers=1 in your switches.
So the final command line would be like
winget install --id Python.Python.3.11 --scope machine --silent --custom 'InstallAllUsers=1 AssociateFiles=0 CompileAll=0 PrependPath=1 Shortcuts=1 Include_doc=0 Include_debug=0 Include_dev=0 Include_exe=0 Include_launcher=0 Include_lib=1 Include_pip=1 Include_symbols=0 Include_tcltk=0 Include_test=0 Include_tools=0'

The --custom WinGet switch will only override the custom switch.

Actually, it doesn't! --custom will only append the value
2024-05-17 10:07:01.168 [CLI ] WinGet invoked with arguments: 'install' '-m' 'C:\Users\WDAGUtilityAccount\Desktop\SandboxTest\3.12.3' '--verbose-logs' '--ignore-local-archive-malware-scan' '--dependency-source' 'winget' '--custom' '/quiet'
|
| Truncated for readability
|
2024-05-17 10:07:04.732 [CLI ] Starting: 'C:\Users\WDAGUtilityAccount\AppData\Local\Temp\WinGet\Python.Python.<IP_ADDRESS>.3\python-3.12.3-amd64.exe' with arguments '/passive /norestart /log "C:\Users\WDAGUtilityAccount\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir\WinGet-Python.Python.<IP_ADDRESS>.3-2024-05-17-10-07-04.731.log" InstallAllUsers=0 PrependPath=1 /quiet'

@github-account1111 - try using --custom instead of --override to add parameters to the install without erasing any of the ones defined in the manifest

Hi guys,
Sorry, maybe this is not related directly, but hope someone can advise. I'm trying to add a number of Python versions under winget remediation. All tests are under system user. Started to test the install first. When I run: .\winget.exe install -e --id Python.Python.3.11 --scope machine it does install ok and regular user can open it. But when I try to check it with .\winget.exe list -e --id Python.Python.3.11 --scope machine it's not found. List works when I remove --scope machine. When I try to uninstall it without --scope it does fail with An unexpected error 0x800401f5: App not found. Can someone please advise what I'm doing wrong? How can I upgrade those 10-12 versions we have in our environment?


GH-35775: [Go][Parquet] Allow key value file metadata to be written after writing row groups

Rationale for this change
The key value file metadata may include information generated while writing row groups.  Currently, it is not possible to set the key value file metadata after creating a writer.  With the changes in this branch, key value pairs may be added any time before closing the writer.
What changes are included in this PR?
This branch adds a writer.AppendKeyValueMetadata(key, value) method to the parquet file.Writer and to the pqarrow.FileWriter.
Are these changes tested?
Tests are added for the new functionality.
Are there any user-facing changes?
The KeyValueMetadata field on the parquet file.Writer has been renamed to initialKeyValueMetadata.  This is a breaking change.  Although the field was exported, setting it did not result in new key value metadata being written.  Instead, it represented the initial key value metadata if the writer was passed the WithWriteMetadata write option.
The WithWriteMetadata option can still be used to provide the initial key value metadata values.  In addition, the AppendKeyValueMetadata method can be called to add key value pairs after creating a writer.
The FileMetadata field on the parquet file.Writer has been removed.  Previously, setting this field value had no effect.

This PR includes breaking changes to public APIs.
The KeyValueMetadata field is no longer exported from the parquet file.Writer struct.  Use the WithWriteMetadata writer option to set key value metadata when creating a writer or use the AppendKeyValueMetadata method to add key value metadata after creating a writer.
The FileMetadata field on the parquet file.Writer has been removed.

Closes: #35775

Thanks, @zeroshade


Configure and updated Runtime HOST settings using in CLI

Why make this change?
Resolved issue #2420
This change is made to enable dab users to update/change values of Rest and Cache runtime settings from CLI commands.

What is this change?

Made changes in ConfigureOptions.cs to include the additional parameters for Host Runtime settings.
Changes in ConfigureGenerator.cs to include the function to update the newly incoming values in the parameters of Host runtime settings
Changes in Tests to add individual tests for all Host individual runtime settings like Host: Mode, Cors.Origins, Cors.AllowCredentials, Authentication.Provider, Authentication.Jwt.Audience, Authentication.Jwt.Issuer.

How was this tested?


Memory leak running with Jest+Sentry

🐛 Bug Report
We use @sentry/node in our unit tests with Jest.
We discovered that just importing @sentry/node (as const Sentry = require('@sentry/node')) leaks memory when running in Jest tests.
I've created a minimal repro here to showcase the issue (listed further).
Good to note that Jest has memory leak issues with many libraries, not just this one. Some popular libraries, e.g graceful-fs were "fixed" to not leak memory with Jest.
Why this issue is important - it blocks us to run our test suite in our CI environment (CircleCI), which has 4Gb memory constraint. It runs out-of-memory already with ~150 test files that we have in our project. See steps to reproduce further.
Some related issues in Jest:
https://github.com/getsentry/sentry-javascript/issues/1966
https://github.com/facebook/jest/issues/8247
facebook/jest#6814
facebook/jest#6738
facebook/jest#7311
facebook/jest#6399
To Reproduce
Steps to reproduce the behavior:
Use minimal repro repository: https://github.com/kirillgroshkov/sentry-jest-leak-repro
Run npm run test-leaking or npm run test-leaking-detectLeaks to see the issue.
Expected behavior
Expected to not increase memory usage (garbage collect all memory). Similar to what happens when you run npm run test-working.
Link to repl or repo (highly encouraged)
https://github.com/kirillgroshkov/sentry-jest-leak-repro
Run npx envinfo --preset jest
Paste the results here:
  System:
    OS: macOS 10.14.4
    CPU: (8) x64 Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz
  Binaries:
    Node: 10.15.0 - /usr/local/bin/node
    Yarn: 1.13.0 - /usr/local/bin/yarn
    npm: 6.9.0 - /usr/local/bin/npm
  npmPackages:
    jest: ^24.5.0 => 24.5.0

This isn't related directly to Sentry, but instead:
@sentry/node -> https-proxy-agent -> agent-base
This file is the culprit:
https://github.com/TooTallNate/node-agent-base/blob/master/patch-core.js
It's importing and then patching the https module in a way that creates a real memory leak when executed repeatedly (which is a test runner-specific thing but not a Jest-only problem). If you want to fix it, a PR to that repository is probably most appropriate.
Since this leak isn't related to Jest and I don't think there's a reasonable way Jest can actually fix it, I'm likely to close this issue. If you can think of something Jest could do to solve this issue, feel free to suggest.

Ahaa, that's a good discovery! I've read about agent-base before, here: https://github.com/facebook/jest/issues/6814#issuecomment-441787087
Then I can try to monkey-patch that module and see what happens...
If that will help - then indeed, probably this issue can be closed.

@scotthovestadt any chance you can share some quick tips on how I can do such discoveries myself? I believe I have many more leaking libs in my big project... How do you debug it and see which module the leak is coming from?

Sure, it's a relatively manual process (could probably be automated). I just open the entry point for the import causing issues, and comment everything out. I verify no leak. I enable it's require calls one by one until the memory leak appears (according to Jest's detection), then go into that entry point. Repeat...
Jest memory leak detector is really amazing for this type of investigation. :)

Makes sense!
What would you do if certain combination of commented and uncommented requires fail "sometimes" (1 of 5 times)?.. I see no randomness, hanging promises, timeouts around..

Usually memory leak won't be created by a combination of different requires. Sometimes more than one may have a leak, or more than one may eventually import the same source module that creates the leak. I usually just keep going down the rabbit hole until I find the source.
If it was flakey, it's likely not caused by just importing it and has to do with certain code paths, so it gets a bit harder. Before trying to narrow down the root cause I'd definitely isolate it to a single easily reproduced test file (like you did for the issues you posted here).


Call the next() function at the right place, remove the redundant 'utf8' encoding.

Hi, thanks for doing this, could you give me some context about this PR?

no longer an issue since it's fixed as part of #847


is there any way to flash it with mac OS?

is there any way to flash it with mac OS?
I do not have any windows OS so do not have putty

Use the terminal of MacOS and find the serial port the use the following command.
'''
screen /dev/cu.serial 115200
'''
There are also others serial port utilities, try to google them.


Matching redesign

This is a redesign of the matching interface. It's not everything we want yet, but should be enough for user testing (we should probably wait for multiple dataset support and/or breaking the matching interface out as an independent component before doing a deeper redesign).
Partially addresses #59 and #165

[Current coverage][cc-pull] is 51.54%

Merging [#196][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #196   diff @@
==========================================
  Files            21         21
  Lines           520        520
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            268        268
  Misses          252        252
  Partials          0          0

Powered by Codecov. Last updated by [63bb9e8...118c325][cc-compare]
[cc-base-branch]: https://codecov.io/gh/Kitware/candela/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/Kitware/candela/compare/63bb9e87ea12b39a4091b4a60dc2008fe17eca0b...118c325771c3191bd10d19a32f3a6c1a20bde2f5
[cc-pull]: https://codecov.io/gh/Kitware/candela/pull/196?src=pr

This looks/feels awesome! I just have two style suggestions:

The visual behavior for deselecting a matching is a little unintuitive. I think it should basically be the same behavior as selecting a matching, but in reverse. When making a match, you see potential links as light blue, dotted lines; hovering over a potential target turns it into a dark blue, solid line. Clicking to select turns it light green, solid, and removes the dotted lines. When deselecting I think the green line should turn light green and dotted, and upon clicking to deselect it should disappear (as it does currently).

Is there a less bulky icon concept we can use for the Boolean type? The Venn diagram icon is "inky" and it connotes sets more directly than Booleans.

Both of these could be handled in separate topic branches (if they are deemed valid suggestions), so let's discuss how we want to handle them.

I totally agree with both suggestions... actually, it looks like something with the line hovers / highlights is misbehaving (more software bug, less design flaw), so I'll get on fixing that (changing to WIP until that gets fixed). And I'll try to find a better boolean icon.

Looks like now when deselecting it turns orange and dotted to indicate the link will be removed. This seems fine to me. Boolean icon is much better - I just think it needs to be compressed more so it looks more like one icon and doesn't run into other icons as much:

Other than that one icon thing LGTM, so if you fix it and ping me for final validation, you should be good to merge.

If this LGTY, I'll go ahead and merge:

LGTM. Bonus points for extra padding all around.


Turkish language added.

Turkish language added and the URL on the Ethereum's website has been changed to Turkish.

Sorry that this took incredibly long to review, we were waiting for our translation team to have time to go over this and the ticket got lost over time.
Thank you for the translations @egemenkus!


Add expect_violation from rubocop-rspec

A while back I implemented a helper for rubocop-rspec that builds on top of inspect_source. I think it makes cops really easy to test, it makes the specs very clear even to someone who is new to the codebase, and it actually generates much more strict test cases. Here is an example from the rubocop-rspec tests:
it 'detects :each for hooks' do
  expect_violation(<<-RUBY)
    before(:each) { true }
    ^^^^^^^^^^^^^ Omit the default `:each` argument for RSpec hooks.
    after(:each)  { true }
    ^^^^^^^^^^^^ Omit the default `:each` argument for RSpec hooks.
    around(:each) { true }
    ^^^^^^^^^^^^^ Omit the default `:each` argument for RSpec hooks.
  RUBY
end

An equivalently thorough test with inspect_source would look like this:
it 'detects :each for hooks' do
  inspect_source(cop, ['before(:each) { true }',
                       'after(:each)  { true }',
                       'around(:each) { true }'])

  expect(cop.messages)
    .to all(eq('Omit the default `:each` argument for RSpec hooks.'))
  expect(cop.offenses.map(&:line)).to eq([1, 2, 3])
  expect(cop.highlights).to eq([
                                 'before(:example)',
                                 'after(:example)',
                                 'around(:example)'
                               ])
end

In practice I think people end up only writing something like the inspect_source followed by an assertion that three offenses were detected.
I think writing the tests like this is a big win. Let me know if you'd like me to upstream this spec support.

Definitely like how this looks. 😀 The test suite sure could use some work.
We have a lot of alignment cops and whatnot, checking leading space. Do you figure they will work properly with the HEREDOC?

That's quite interesting indeed! Quite compact!
Apart from what @Drenmi mentioned:

violations -> offenses
it took me a moment to understand the format of the heredoc - I wonder if there isn't something we can do to make it even easier to comprehend
what happens when you're dealing with long code highlights and long messages? This won't look nearly as pretty...

what happens when you're dealing with long code highlights and long messages? This won't look nearly as pretty...

Can't do much about long code highlights (at least off the top of my head). For long messages we could possibly adopt a rule like this
let(:message) { 'Omit the default `:each` argument for RSpec hooks.' }

# No message in the annotation. Looks to see if `let(:message) { ... }` is defined

it 'detects :each for hooks' do
  expect_violation(<<-RUBY)
    before(:each) { true }
    ^^^^^^^^^^^^^
    after(:each)  { true }
    ^^^^^^^^^^^^
    around(:each) { true }
    ^^^^^^^^^^^^^
  RUBY
end

The nice thing about building it on top of inspect_source is that you don't have to use it in cases where it is too noisy


Re-layout the distro so that mvn is not in bin

The new invoker test is passing, our mvn is hidden when mvnd is installed via SDK man. All known problems solved from my PoV.
Any more scenarios we should test come to your mind, @gnodet ?


Need to remove log file mentions from docs since we have removed log files from generating

Description:
Need to remove log file mentions from docs since we have removed log files from generating

https://apk.docs.wso2.com/en/latest/administration/configurations/configure-logs-enforcer/


Use a manual Future impl for methods

I'm wrapping Receiver::recv and Sender::send in structs. Because they're async fn instead of manual Future impls they require boxing when stored. It'd be nice if we could expose two new types:

RecvFuture
SendFuture

These would not require boxing, which means they can be inlined in implementations and would increase overall perf when wrapping async-channel. Given the simplicity of send and recv I suspect the implementation to be rather straight forward.
Thanks!

Ran into this again today; still stand by that this is a change we should implement.

This would probably require similarly exposed Future impls in event-listener.

Filed a PR here: https://github.com/smol-rs/async-channel/pull/33


Post Ideas: Programar Atmel SAM4S

Hacer un 0 to 100% sobre como programar la SAM4S Xplained Pro.

https://boundarydevices.com/debugging-using-segger-j-link-jtag/

openOCD from source,
git clone git://git.code.sf.net/p/openocd/code openocd

edbg
git clone https://github.com/ataradov/edbg
make all
No dependencias para arch linux.

Progress per https://wiki.segger.com/IMX6SX
1.) working at /opt/SEGGER/JLink_V614b/
2.) create /opt/SEGGER/JLink_V614b/JLinkDevices.xml and paste JLinkDevices.xml from https://wiki.segger.com/IMX6SX
3.) change JLinkScriptFile entry to IMX6SoloX_Connect_CortexA9.JLinkScript
4.) copy Samples/JLink/Scripts/IMX6SoloX_Connect_CortexA9.JLinkScript Devices/NXP/.
5.) get flashing algorithm; wget https://wiki.segger.com/images/e/ec/NXP_iMX6SX_SABRE_Board_QSPI_ReferenceFlashAlgorithm.zip
6.) unzip NXP_iMX6SX_SABRE_Board_QSPI_ReferenceFlashAlgorithm.zip
7.) run; JLinkExe -device MCIMX6X1_A9 -if JTAG -speed 1000 -JTAGConf -1,-1
…
Cortex-A9 identified.

Hi,

There’s no boot requirement for kernel JTAG debugging so as you like.
Well you need to attach before the crash happens. Some add a “while(true);” loop at the beginning of kernel initialization (after MMU enablement) so you have time to connect. Then you can step out of the loop with the JTAG.
https://elinux.org/images/f/ff/JTAG_In_Linux_Driver_Debug_Anderson.pdf
Please have a look at segger and eclipse forums for app debugging:
https://forum.segger.com/
https://www.eclipse.org/forums/index.php

Regards


Apply fixes from StyleCI

This pull request applies code style fixes from an analysis carried out by StyleCI.

For more information, click here.

Pull Request Test Coverage Report for Build 16

0 of 324 (0.0%) changed or added relevant lines in 23 files are covered.
40 unchanged lines in 16 files lost coverage.
Overall coverage remained the same at 0.0%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Chiron/Stack/RequestHandlerStack.php
0
1
0.0%

src/Chiron/Middleware/ParsedBodyMiddleware.php
0
1
0.0%

src/Chiron/Handler/MaintenanceHandler.php
0
1
0.0%

src/Chiron/Exception/HttpException.php
0
1
0.0%

src/Chiron/Http/Factories/ServerRequestFactory.php
0
2
0.0%

src/Chiron/Middleware/EmitterMiddleware.php
0
2
0.0%

src/Chiron/Config/Config.php
0
2
0.0%

src/Chiron/Middleware/MethodOverrideMiddleware.php
0
3
0.0%

src/Chiron/Logger.php
0
3
0.0%

src/Chiron/Middleware/ErrorHandlerMiddleware.php
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/Chiron/Middleware/EmitterMiddleware.php
1
0.0%

src/Chiron/Middleware/ParsedBodyMiddleware.php
1
0.0%

src/Chiron/Container.php
1
0.0%

src/Chiron/Http/Message.php
1
0.0%

src/Chiron/Handler/AbstractExceptionHandler.php
1
0.0%

src/Chiron/Http/Stream.php
1
0.0%

src/Chiron/Handler/HttpExceptionHandler.php
1
0.0%

src/Chiron/Http/Uri.php
1
0.0%

src/Chiron/DeferredRequestHandler.php
1
0.0%

src/Chiron/Handler/NotAllowedHandler.php
2
0.0%

Totals

Change from base Build 15:
0.0%

Covered Lines:
0

Relevant Lines:
3957

💛  - Coveralls


Feature/orgs ui fix 2

more fixes for TN-442:  https://jira.movember.com/browse/TN-442
re-indent text,
fix order of orgs

adding one more fix, close for now


⚠ Jenkins Plugins Artifactory has degraded performance

In d84f4aa, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1219 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 6fc68b6 after 14 minutes.


Tuple length checking doesn't work with tuple() constructor

I searched for other, similar issues, but none seemed to cover this exact case.
 Environment data

Language Server version: 2022.2.1
OS and version: darwin arm64
Python version (and distribution if applicable, e.g. Anaconda): 3.10.0
python.analysis.indexing: null
python.analysis.typeCheckingMode: basic

 Expected behaviour
An error should be raised if tuple() is assigned to a tuple type of defined length
 Actual behaviour
No error is raised
 Code Snippet / Additional information
from typing import Tuple
Point3D = Tuple[int, int, int]

t1: Point3D = 1, 2, 3  # correct no error
t2: Point3D = 1, 2, 3, 4  # correct error; Element size mismatch; expected 3 but received 4
t3: Point3D = () # correct error; Element size mismatch; expected 3 but received 0
t4: Point3D = tuple() # expected error, none thrown

Given that tuple() returns an empty tuple, I'd expect t3 and t4 to behave identically. But, only one throws an error.

This is expected behavior, although I admit that it's less than ideal.
The type returned by the constructor call tuple() is dictated by the typeshed stub file builtins.pyi. Here's the specific line that declares the __new__ method. You'll see that the parameter __iterable has a default value. If you call the constructor and don't provide an argument for this parameter, the type variable _T_co is not populated, so the resulting type is tuple[_T_co@tuple], which is effectively tuple[Any].
A potential fix is for the typeshed stub to provide an overload that specifies a return type of tuple[()] if the __iterable argument isn't provided. You're welcome to file a bug (or a PR) in the typeshed repo. Pylance regularly picks up new versions of the typeshed stubs.
In the meantime, I recommend using the () form rather than tuple(). Not only is it more terse, but it also avoids this problem from a type checking standpoint.

Fair enough, thank you!

I'm happy to file a PR. While I was playing with it, I noticed that mypy does handle this as expected. t4 above throws main.py:7: error: Incompatible types in assignment (expression has type "Tuple[<nothing>, ...]", variable has type "Tuple[int, int, int]").
I think the typeshed PR makes sense in this case, but I wanted to point that out in case it was useful.

Yes, I noticed that mypy handles this case. It probably overrides the type stub in this particular case. I'm reluctant to ignore / override the information in the type stub because that often leads to other problems as the type stubs evolve or improve. It's preferable to solve this problem in the stubs.


Refactor: change handler to transport

Changed handlers name to be called transports as it more accurate.
Made the default timestamps option to be true.
Add some more unit tests.

:tada: This PR is included in version 1.6.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


How to add Icon and one other label

Hello Developer,
I want this type of drop down list. I really confused how to add image in left side.  I trying to add imageview in dropdown Xib in  podfile and when i using the imageview then show me error
Initializer is inaccessible due to 'internal' protection level.

Screen Shot I want  this type of drop down

I would also like to insert a second label but it gives me an error:
'[<NSObject 0x600002678600> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key longText.'

lass is not key value coding-comp

create custom cell , please follow instructions given in example


Test Data

Hi,
Thank you for sharing your great work. If you do me a favor, I would like to ask you the test data, cube.avi and obj files.
Thanks,
Bobby

Hi,
could you please share the cube.avi, bunny.avi test videos please?
Best regards,
Joao Teixeira


long press trigger

Hello,
Apparently when using the long press on buttons, the action is triggered after the button is released.
Is is possible to trigger the action 1 or 2 seconds after starting to press the button rather than after release? This way the user would see the action happen and instinctively know that he can release the button (as it is the case for the Ikea shortcut buttons).
Thanks in advance.

No.  Unlike most real buttons, the GPIO pin doesn’t report hold, only press and release.  Homebridge RPi uses the duration between these to determine single press vs long press.

Thanks for the answer.


add ability to pass back key and value in the same longest prefix cal…

…l; really bad way to do it, but do not want to track byte sizes across boundary
Didn't want to use the radix string/split mechanism but didn't have a lot of time to devote and allocating memory from the ruby process would be harder because we would have to track maximum byte sizes to alloc memory on the ruby side and then pass it to C; it's easier to use the pointer and read of the pointer already being returned

vectors are guaranteed to have contiguous memory, and memcpying onto arrays of chars is fine also. But come to think of it, I'm positive the compiler would optimize this to a memcpy anyway.


Enable binary crates as dependencies

This is useful in a variety of situations such as:

Embedding one binary into another (i.e. BIOS, kernel)
Testing using a third-party utility (i.e. nc)

Fixes #4316.

@ehuss

It also doesn't handle if the package has both a library and a binary.

I think it's also important to be forward compatible with [binary-dependencies] in Cargo.toml, because sometimes crates have both libraries and binaries, and you might only want to use one of the two but not the other. If there is no opt in to depend only on the library component of a dependency, [binary-dependencies] won't make any sense in the future (issue for binary dependencies: #1982).

:umbrella: The latest upstream changes (presumably #7820) made this pull request unmergeable. Please resolve the merge conflicts.

I'm going to close this since this PR has gotten a little old, and this feature will likely need an RFC. Once it goes through the RFC process, we can revisit the implementation.


Fix circular dependency in Settings initialization

Description
There was a weird circular class-loading dependency between Settings and Settings.Builder that could result in ClassLoader deadlock if multiple threads trigger class-loading of Settings at the same time.
Related Issues
Resolves #10065
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

LGTM, want to finish it?

I was able to reproduce the deadlock with the following simple class:
public class SettingsMain {
    public static void main(String[] args) {
        new Thread(Settings.Builder.EMPTY_SETTINGS::getSecureSettings).start();
        Settings.builder();
    }
}

When I run that, the process just hangs. (It's essentially the idea discussed in https://www.farside.org.uk/201510/deadlocks_in_java_class_initialisation.)
With the code change from this PR, it doesn't hang -- it just completes immediately.

Unfortunately, since Settings are used throughout the codebase, I can't add it as a unit test. Maybe a dedicated Gradle task?
My preference (partly driven by laziness) is just apply the fix as is -- these circular static initializer dependencies are tricky to catch in automated way.

My preference (partly driven by laziness) is just apply the fix as is -- these circular static initializer dependencies are tricky to catch in automated way.

This is tricky to test, if JVM hangs we should have a way to terminate it properly by failing the test, not very straightforward.

@dblock good with that? (skipping the test part)

YOLO I hit merged. Thanks @msfroh and @reta.


Change the label of "select lineages" to a more generic label

Given that we now have different ways of grouping data, time, location, wastewater, and lineages we should change the label "select lineages" to a more generic label. Something like "select/unselect" or select group"

There are also references to lineages in the help box, and the axis label "Variants".
Furthermore, there are references to lineages throughout the codebase. Need to think of a consistent generic vocabulary, and apply.


[Feature Request] Option to switch off menu bar icon/album art icon, option to disable popover player

As the titles says, basically to be able to use the app for displaying text and nothing else.

+1, @martinfekete10 could you consider this?

Hey, feature to switch of icon/album art will be included in the next release! Expecting to release it in the next few days

v1.6.0 was just released with this feature added, alongside several other new features and fixes 🙌

Amazing, thank you very much, the new features are great! One small thing I noticed: currently the text stops moving only if the icon is visible, even if it fits to the pre-defined pixel width. An option to enable movement only when it's needed (when it doesn't fit), would refine things further.

Also, if the pop-up player could be disabled as well, so only the text was displayed, and clicking on it was a shortcut to Spotify. This is also part of the original request.


📃:  Animated Tea Cup (CSS Art)

:red_circle: Title :  Tea Cup
:red_circle: Aim :  Project is for beginners who are interested in CSS art to get started with it.
:red_circle: Tech Stack Used :  HTML CSS
Screenshots 📷

N/A

:white_check_mark: To be Mentioned while taking the issue :

Full name : Amruta Jayanti
What is your participant role? (Mention the Open Source Program name. Eg. GSSOC, SSOC, JWOC, etc.) GSSOC

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@AmrutaJayanti go ahead.


Update overrides-v2.md

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

adding some clarification

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

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

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


typedef_test_9

Surelog issue: https://github.com/chipsalliance/Surelog/issues/2556

PR with fix: https://github.com/SymbiFlow/yosys-f4pga-plugins/pull/259


Request body mime type is not detected properly when it is given inside component schema

Description:
When mime type is given in the component schema, that mime type is not considered in the client generation
Expected
    remote isolated function postIcons(PostIconsRequest payload) returns CreatedItem|error {

Actual
    remote isolated function postIcons(json payload) returns CreatedItem|error {

Steps to reproduce:
/api/task/web-font/icons:
    post:
      description: ""
      operationId: PostIcons
      requestBody:
        $ref: "#/components/requestBodies/From_a_CollectionBody"

From icon8 openapi
Affected Versions:
Beta6

This issue is also present in the service generation.
Sample OpenAPI
paths:
  /pets:
    post:
      summary: Create a pet
      operationId: createPet
      tags:
        - pets
      requestBody:
        description: "Request to add a pet"
        $ref: "#/components/requestBodies/CreatedPet"
      responses:
        '200':
          description: Successful operation

components:
  requestBodies:
    CreatedPet:
      description: Return from creating a pet
      content:
        application/json:
          schema:
            type: object
            properties:
              petId:
                type: string
              createdDate:
                type: string

Generated resource
    resource function post pets(@http:Payload object payload) returns object {
    }

Expected resource
    resource function post pets(@http:Payload CreatedPet payload) returns object {
    }

Closing with https://github.com/ballerina-platform/openapi-tools/pull/1124


Lightstreamer SYNC ERROR

Hello,
I randomly get the following error:
Server response error:
SYNC ERROR
Exception in thread STREAM-CONN-THREAD-1:
Traceback (most recent call last):
  File "C:\Python36-32\Lib\threading.py", line 916, in _bootstrap_inner
    self.run()
  File "C:\Python36-32\Lib\threading.py", line 864, in run
    self._target(*self._args, **self._kwargs)
  File "C:\Python36-32\lib\site-packages\trading_ig-0.0.6-py3.6.egg\trading_ig\lightstreamer.py", line 386, in _receive
    self.bind()
  File "C:\Python36-32\lib\site-packages\trading_ig-0.0.6-py3.6.egg\trading_ig\lightstreamer.py", line 210, in bind
    self._handle_stream(stream_line)
  File "C:\Python36-32\lib\site-packages\trading_ig-0.0.6-py3.6.egg\trading_ig\lightstreamer.py", line 238, in _handle_stream
    raise IOError()
OSError

Maybe we could add rebind = True in lightstreamer.py as for the "LOOP ERROR" ?

Hi, did you manage to solve the problem. I have exactly the same issue. Searching the net i found that rebind is required after this error. The root of problem is that you've subscribed too many items. As I reduced the number of subscribed items the error is gone. But how should I rebind to server?

Hi,
@TraderVK or @MohammadMoradi did you manage to solve the problem? I have the same issue :)
Thanks

This is really odd. Looking at the latest code, rebind is set to True: https://github.com/ig-python/ig-markets-api-python-library/blob/master/trading_ig/lightstreamer.py#L352
But lower down there is an if statement if not rebind: which it runs as if rebind==False. i.e. it doesn't call self.bind()
So the code "looks" right but it's not running as expected. I'm fairly new to Python so can anyone understand why it's running the code in the if not rebind: block instead of in the else ?

I've just stepped through the code locally and it got to the else block as expected. I then got the error above. Not sure if that's a result of running the debugger. I'm also confused as to why my server seems to be behaving differently. I ran the debugger again and I can now see it's got a LOOP and successfully rebinded and carried on.


Widget API extension: Always-on-screen

Documentation: https://docs.google.com/document/d/1_HEq5skPp1Yp559hYp2FSO2ecw8mJQR8lOvgsmYOEro/edit?usp=sharing
Implementation(s):

matrix-widget-api

An extension to the widgets API to allow widgets to remain on screen whatever room the user is viewing. Current use case is video calls.

I'm not sure there'd need to be any changes to the widget API from a spec perspective. What did you have in mind?

Hopefully the proposal explains all :)

Ah, I see. It's a capability being requested by the widget. That makes a lot more sense :D

note: this is built upon the currently undocumented widget API (#1236)
Looks good to me :)

Looks good to me, @dbkr. Thanks!

In the interest of getting this into the integration spec:
@mscbot fcp merge

i want to use matrix widget can you help me for that.

@sunnyharsh123 I don't think this is a good place to ask questions. For that, I would recommend the Matrix HQ room

I think this is now implemented right? (the example in the matrix-widget-api is using a sticky widget)

It's not linked in the description here.

I was thinking about this line: https://github.com/matrix-org/matrix-widget-api/blob/b571e1860903b3ef712b62380f042b6ad9b5611f/examples/widget/index.html#L96
I though this would be the implementation of the Widget Api extension. Probably I got the meaning of needs-implementation wrong then.

That looks like a valid implementation to me. I believe @turt2live meant that it needs to be specified in the description of the original comment in order for the needs-implementation label to be removed.
I've added it, and am removing the label. Feel free to re-add if the implementation isn't satisfactory.


feat: adding otel in consent

still erroring @siddhart1o1

still erroring @siddhart1o1

Not sure what's wrong as I am not getting any errors locally. Also, I haven't made changes in the dashboard until commit b0c924d, and am getting an error in this commit whereas the commit before this was passing.
and getting this error
 [2023-10-25T09:55:36.331+00:00] /home/runner/work/galoy/galoy/buck-out/v2/gen/root/524f8da68ea2a374/apps/dashboard/__check-lint__/__workspace/apps/dashboard/services/graphql/index.ts
[2023-10-25T09:55:36.331+00:00]   4:1  error  There should be at least one empty line between import groups  import/order


Fix TLS connection problem

This is commit is to fix connection error when connecting to Elasticsearch endpoint that uses TLS. Previous container did not have proper config file for TLS connection and it returned "Connection aborted, Bad StatusLine" error. That was because use_ssl and verify_certs config variables were not handled by start-elastalert.sh script.

Sorry, this repository is no longer maintained. While I could accept your pull request it would not result in any modifications of the Docker image on DockerHub. I advise you to fork this repository and create your own Docker image. Thanks for effort though.

No worries. Thanks for the update anyway.


Add exam commands and storage for exams

Fix #91
Exams now have -ae and -de commands for adding and removing respectively. It is also stored in the local files now.
Take note: Since I realised that EditPersonCommand, PersonAddLessonCommand, PersonRemoveLessonCommand had some similar code, I have merged them all to EditPersonCommand. Thus I have removed my previous code for Lesson Commands.
To facilitate the above, EditPersonDescriptor now contains a way to update lessons and exams of a person.
TODO: Test cases and User Guide for exams

Codecov Report

Merging #92 (6ccbf56) into master (7daf60a) will decrease coverage by 3.88%.
The diff coverage is 29.44%.

@@             Coverage Diff              @@
##             master      #92      +/-   ##
============================================
- Coverage     64.98%   61.09%   -3.89%
+ Complexity      604      584      -20
============================================
  Files           111      113       +2
  Lines          2156     2239      +83
  Branches        246      248       +2
============================================
- Hits           1401     1368      -33
- Misses          673      789     +116
  Partials         82       82

Impacted Files
Coverage Δ

...in/java/seedu/address/logic/parser/ParserUtil.java
58.33% <0.00%> (-4.67%)
:arrow_down:

...ress/logic/parser/persons/PersonAddExamParser.java
0.00% <0.00%> (ø)

...ess/logic/parser/persons/PersonCommandsParser.java
50.00% <0.00%> (-6.25%)
:arrow_down:

...s/logic/parser/persons/PersonRemoveExamParser.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/model/person/Exam.java
0.00% <0.00%> (ø)

...c/main/java/seedu/address/model/person/Person.java
75.55% <0.00%> (-17.78%)
:arrow_down:

...model/person/exceptions/CannotAttendException.java
0.00% <0.00%> (ø)

...in/java/seedu/address/storage/JsonAdaptedExam.java
0.00% <0.00%> (ø)

...c/main/java/seedu/address/model/lesson/Lesson.java
45.23% <20.00%> (-31.91%)
:arrow_down:

...ress/logic/commands/persons/EditPersonCommand.java
74.19% <50.00%> (-18.31%)
:arrow_down:

... and 10 more

Continue to review full report at Codecov.

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


[DO NOT REVIEW] add API to wait for a tenant to appear in the tenant map

I developed this before I pivoted to
https://github.com/neondatabase/neon/pull/4299
for the problem I had at hand.
Yet, it might be useful to have an API to wait for a tenant to appear.
Includes a demo use case inside the tenant tasks.
Problem
Summary of changes
Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

Stale


Add environment variables on container specs

Why is this pull request needed and what does it do?
Finish work started in #90 (rebase + bump version)

@Ripolin the DCO check is failing, please fix the problematic commits.
Also, please update the changes section in chart.yaml

@Ripolin the DCO check is failing, please fix the problematic commits.
Also, please update the changes section in chart.yaml

Should be OK now

Please modify the changes under annotations (L21)

Done

Thanks for the PR!


Feature request: Add flags for providing owners and permission

Hey mate,
First of all. BLAZINGLY FASTTTT
Into my request. It would be great if we could add some feature to view who owns the file and what permission it was granted. So basically showing rwx permission

Sure thing. Adding to my todo

Small update: this has just been merged into my work-in-progress v2.0 branch. If all goes well it should be out this Sunday or early next week: https://github.com/solidiquis/erdtree/pull/131

https://github.com/solidiquis/erdtree/pull/131

Maybe im missing something here, but what are those flags?


Intellisense service not starting for Javascript project (React Native)

Bug Report
🔎 Search Terms
Intellisense not starting, language server not starting javascript,
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about language server settings, javascript extensions, and troubleshooting intellisense. No luck, all settings are enabled.

💻 Code

Intellisense should work. Specifically, 'go to definition' is not working.
jsconfig.json:
    "compilerOptions": {
      "module": "commonjs",
      "target": "es6"
    },
    "exclude": ["node_modules"]
  }

I've also tried disabling all extensions and it still didn't work.
🙁 Actual behavior
Intellisense will not work with ANY javascript file.
Can provide logs if needed

found an answer after 2 hours of searching (how did it happen right after I opened this issue?) problem was 'javascript and typescript language features' was auto-disabled.

found an answer after 2 hours of searching (how did it happen right after I opened this issue?) problem was 'javascript and typescript language features' was auto-disabled.


cli: vocabulary convert

Implement a command to convert/process the content of a vocabulary. This would be useful to process big files, that are in other formats, contain many unused fields, etc.
invenio vocabularies convert <vocabulary> <orig_file_path> <dst_file_path>

in names branch, merge to master https://github.com/inveniosoftware/invenio-rdm-records/issues/831


Sync Fork from Upstream Repo

Pull Request Test Coverage Report for Build 2ba8666d55fcd876777853ceaaa4484114e95049-PR-11

19 of 19 (100.0%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 98.73%

Totals

Change from base Build 132f69ddb368c04e6d258c201c885c972812b254:
0.01%

Covered Lines:
508

Relevant Lines:
508

💛  - Coveralls


"Optimizing Stock Price Prediction with LightGBM and CatBoost: Choosing the Right Model for Efficiency and Accuracy"

For stock price prediction, you might choose LightGBM or CatBoost depending on the nature of your dataset and specific requirements. If your dataset includes many categorical features, CatBoost might be more suitable. If you need to handle a very large dataset efficiently, LightGBM could be the better choice.
The main Reason using This model LightGBM or CatBoost because   It has built-in support for categorical features, which can improve performance and accuracy.
And These model will reduce time complexity

Hi,
I have reviewed the changes in this pull request. If any changes need to be done or if there are any issues, please make a comment so that it will be beneficial.
Additionally, could you please add the labels "gssoc", "level" and "hacktober"
Thank you!

@praveenarjun
Thank you for your valuable contribution.
Please ensure that you star this repository, your PR will be soon reviewed by the mentors.

Reasons for Using LightGBM and CatBoost
LightGBM:
Efficiency: LightGBM is designed to be highly efficient and can handle large datasets with faster training times.
Accuracy: It often provides better accuracy compared to other gradient boosting algorithms.
Scalability: LightGBM can handle large-scale data and high-dimensional features.
Support for Categorical Features: It can directly handle categorical features without the need for one-hot encoding.
CatBoost:
Handling Categorical Features: CatBoost is specifically designed to handle categorical features effectively, reducing the need for extensive preprocessing.
Robustness: It is less prone to overfitting and provides robust performance on various datasets.
Ease of Use: CatBoost requires minimal parameter tuning and is easy to use.
Efficiency: It is optimized for fast training and prediction times.
These models are particularly useful for tasks involving structured data and can provide significant improvements in performance and efficiency.
Import necessary libraries
import lightgbm as lgb
from catboost import CatBoostRegressor
from sklearn.metrics import mean_squared_error, mean_absolute_error, mean_absolute_percentage_error, accuracy_score, precision_score, confusion_matrix, recall_score, f1_score
Function to train and evaluate a model
def train_and_evaluate_model(model, X_train, X_test, y_train, y_test):
model.fit(X_train, y_train)
pred = model.predict(X_test)
rmse = np.sqrt(mean_squared_error(y_test, pred))
mae = mean_absolute_error(y_test, pred)
mape = mean_absolute_percentage_error(y_test, pred)
accuracy = accuracy_score(y_test > pred, y_test > pred.round())
precision = precision_score(y_test > pred, y_test > pred.round())
confusion = confusion_matrix(y_test > pred, y_test > pred.round())
recall = recall_score(y_test > pred, y_test > pred.round())
f1 = f1_score(y_test > pred, y_test > pred.round())
return rmse, mae, mape, accuracy, precision, confusion, recall, f1
Train and evaluate LightGBM model
model_lightgbm = lgb.LGBMRegressor()
metrics_lightgbm = train_and_evaluate_model(model_lightgbm, X_train, X_test, y_train, y_test)
print("LightGBM Metrics:", metrics_lightgbm)
Train and evaluate CatBoost model
model_catboost = CatBoostRegressor(verbose=0)
metrics_catboost = train_and_evaluate_model(model_catboost, X_train, X_test, y_train, y_test)
print("CatBoost Metrics:", metrics_catboost)

I am fasing some issue to merge and pull request

To Improve Model fastness we can use LightGBM and CatBoost
#15
This is my Issue

I used anpther branch for this so u can merge it fro there

how much time it take to review the code and merge


docs Add table property min_load_replica_num doc

Proposed changes
Issue Number: close #xxx

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

run buildall


Harmonize the shape of the button (the Edit product is the right shape)

What

[ ] Harmonize the shape of the button (the "Edit product" button is the right shape)

Screenshot

Hi @teolemon ,
Please assign this to me.


Computing $L_i$ instead of $L$

A first modification we could do, in the long refactor of Dual+iterate could be the following.
If $T$ satisfies the following assumption there exists a partition ${P_i}$ of $M$ of full measure such that

$T_i: = T|_{P_i}$ is injective
$|DT_i|>0$

then we can write
$$
L f(x) =\sum_{i=1}^n L_i f
$$
where
$$
L_i f(x) = \frac{f(T_i^{-1}(x))}{|DT(T_i^{-1}(x))|}\chi_{T_i(P_i)}(x)
$$
to each branch is associated an operator, and the discretized operator is the sum of these operators.

Maybe I am missing some motivation; what would be the advantage of doing this?
Here $n$ is the number of branches, right? And that expression is for the $C^0$ assembler?
I think we already do something very similar: the preimages are computed branch-by-branch, so we compute those terms separately, store them in the I, J, nzvals vectors and add them up when we create the sparse matrix in assemble.

Oh OK maybe I get it now; your plan is having this part when we split the assembly over branches as a "common sub-factor" of assemble for all basis?
For the future my suggestion would be instead having completely separated assemble methods for each basis, dispatched over the base type: especially looking at future Chebyshev / FFT strategies, they seem different enough that introducing a common interface is just a complication. Indeed all that iterate business with the duals makes the current code really hard to read.
And in a sense preimages is already a common part of all of them that we have identified and abstracted out. Note that calling preimages on a dynamic already concatenates the preimages over each branch in a single vector, so it already plays that role, more or less.

I am thinking about the fact that each one of the branches can be computed in parallel, mostly, but I need to think more about it...

Note that at least for ComposedDynamics, one should not split into branches; see https://github.com/JuliaDynamics/RigorousInvariantMeasures.jl/issues/153#issuecomment-1435947115 for an explanation.
Apart from that, I agree that preimages along different branches could be computed in parallel, and then afterwards also $L_i$ as you suggest; that is something to study.


There is still no MAI token in MetaMask Chrome plugin

The required info for MAI has been merged by this pull request https://github.com/MetaMask/contract-metadata/pull/839.
But unfortunately in the MetaMask Google Chome plugin MAI symbol and it's logo had not appeared. It's still necessary to add the token manually by adding MAI's smart contract as custom token.
Please we would really appreciate if you take a look at this issue.

Hi @yell0w4x if should be available soon with release 9.8.1

Could you do a new release of contract-metadata so that more tokens are picked up on the upcoming metamask 9.8.1 release?

Released


Pass MyPy Checks

Refactor all code in pyfixest/estimation to pass mypy checks
add a mypy.ini file
add mypy to pre-commit

Introduced error for CRV estimation:
>               if len(self._ssc) == 0:
E               TypeError: object of type 'float' has no len()

Three errors remaining:
pyfixest\utils\dev_utils.py:12: error: Cannot assign multiple types to name "DataFrameType" without an explicit "Type[...]" annotation  [misc]
pyfixest\estimation\feols_.py:1754: error: Incompatible return value type (got "tuple[ndarray[Any, Any], Any, Any | None, Any | None]", expected "tuple[ndarray[Any, Any], list[str], list[str], list[int]]")  [return-value]
pyfixest\estimation\FixestMulti_.py:394: error: Argument "_ssc_dict" to "add_fixest_multi_context" of "Feols" has incompatible type "dict[str, str] | None"; expected "dict[str, str | bool]"  [arg-type]

@s3alfisc Fantastic work with the MyPy integration! I know is hard but is better do it sooner than later :D

Without the help of my LLM coworker, I'd probably have another week ahead of me =) But there are still some fixes to apply for the report and did modules, which is also something for separate PRs. Thanks for tipping me over to get this started, the code quality has increased already (and I even found some proper bugs as well!).


fix(helm): pass os.stdin when executing a plugin

It fixes the problem with an editor opened by a plugin (Vim returns Input is not from a terminal warning and the TTY is broken).

We had to fix this in draft as well.
Relevant fix: https://github.com/Azure/draft/pull/208


Differential Ability Score Changes - e.g. type "-3" on ability scores like you do with hp

In addition to having 'max' values for each attribute, we should consider being able to type "-3" on ability scores like you do with hit points to do instant math on them, since scores move around so much during the game.

We can detect entries with a leading - or + in the actor sheet's _updateObject and apply the deltas. We already hook item updates here.

I built this locally and then decided I didn't like it, so I am going to close this one. :)


VS Code extension with regols

Hi, I create a VS Code extension Rego with regols.
@kitagry  thanks for create this rego language server ~~

Thank you! Great work!


[Bug report]: Firefly 2.0.1 for Windows does not start - white screen appears after Firefly splash screen

Platform
Desktop
Version
2.0.1
Description
Hi,
I installed the latest version of Firefly for IOTA (2.0.1). Once installed, Firefly began to start. At first it looked like it was working as it came up with the Splash screen saying Firefly. But then the Firefly writing disappeared and I was left with a blank white screen where it just hung constantly. Eventually I just closed the window by killing the process. I tried starting Firefly again a few times but with the same results.
As I couldn't even get to the page that shows all my Profiles, I decided to uninstall Firefly and re-install version 1.7.4, which is the one I was on before. However, after installation, I do get the Profiles window, but after I have entered my 6 digit passcode, yet again the screen goes white and just hangs again!
Please can you help.
Expected behaviour
When I start the application I expect to see the Splash window, followed by the screen showing my list of Profiles.
Actual behaviour
Once installed, Firefly began to start. At first it looked like it was working as it came up with the Splash screen saying Firefly. But then the Firefly writing disappeared and I was left with a blank white screen where it just hung constantly. Eventually I just closed the window by killing the process. I tried starting Firefly again a few times but with the same results.
Can the issue reliably be reproduced?
Yes
Steps to reproduce the issue
No response
Operating System
Windows
Diagnostics
It doesn't even get into Firefly so I can't get to the Diagnostics.

Developer Profile
No
Profile Type
Software (Stronghold)
Staking
Previously staked
Crash Status
White screen
Error Messages
None
Error Logs
No response
Duplicate declaration

[X] I have searched the issues tracker this issue and there is none

Code of Conduct

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

I'm geting the same issue.

Tried uninstalling and installing older 1.7.0 version. It gets as far as allowing me to input PIN No then goes white. Mon, 21 Feb 2022 16:54:26 GMT
ClientError: Response error with status code 404: 404 page not found
Tue, 08 Feb 2022 14:38:32 GMT
ClientError: Response error with status code 404: 404 page not found
Tue, 08 Feb 2022 14:24:53 GMT
ClientError: Response error with status code 404: 404 page not found
Tue, 08 Feb 2022 14:18:59 GMT
ClientError: Response error with status code 404: 404 page not found
Tue, 08 Feb 2022 13:45:43 GMT
ClientError: Response error with status code 404: 404 page not found
Tue, 08 Feb 2022 13:45:33 GMT
ClientError: Response error with status code 404: 404 page not found
Tue, 08 Feb 2022 13:45:22 GMT
ClientError: Response error with status code 404: 404 page not found
Tue, 08 Feb 2022 13:18:19 GMT
ClientError: Response error with status code 404: 404 page not found
Tue, 08 Feb 2022 13:17:49 GMT
ClientError: Response error with status code 404: 404 page not found
Sun, 06 Feb 2022 17:28:51 GMT
ClientError: Response error with status code 404: 404 page not found
Fri, 04 Feb 2022 16:18:12 GMT
ClientError: Response error with status code 404: 404 page not found
Fri, 04 Feb 2022 16:15:32 GMT
ClientError: Response error with status code 404: 404 page not found
Fri, 04 Feb 2022 16:13:32 GMT
ClientError: Response error with status code 404: 404 page not found
Fri, 04 Feb 2022 16:11:34 GMT
ClientError: Response error with status code 404: 404 page not found
Fri, 31 Dec 2021 16:41:06 GMT
ClientError: Response error with status code 404: 404 page not found
Fri, 31 Dec 2021 16:33:03 GMT
StrongholdError: failed to perform action: Engine Error: Crypto Error: error in algorithm XCHACHA20-POLY1305, Unable to read snapshot. Please try another password.``
Thu, 30 Dec 2021 21:20:00 GMT
ClientError: error sending request for url (https://chrysalis-nodes.iota.cafe/api/v1/info): operation timed out
Tue, 28 Dec 2021 13:53:30 GMT
StrongholdError: failed to perform action: Engine Error: Crypto Error: error in algorithm XCHACHA20-POLY1305, Unable to read snapshot. Please try another password.``
Tue, 28 Dec 2021 13:53:30 GMT
StrongholdError: failed to perform action: Engine Error: Crypto Error: error in algorithm XCHACHA20-POLY1305, Unable to read snapshot. Please try another password.``
Mon, 27 Dec 2021 15:52:32 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:52:22 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:51:32 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:51:22 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:50:42 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:49:42 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:49:32 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:45:52 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:45:42 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:44:32 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:41:32 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:41:22 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:40:44 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:39:44 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:39:04 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:36:43 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:36:05 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:32:58 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:31:23 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:25:39 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:23:59 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:22:29 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:20:29 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:19:29 GMT
ClientError: Response error with status code 404: 404 page not found
Mon, 27 Dec 2021 15:19:19 GMT
ClientError: Response error with status code 404: 404 page not found
Tue, 21 Dec 2021 19:22:46 GMT
ClientError: error sending request for url (https://chrysalis-nodes.iota.cafe/api/v1/info): operation timed out
Tue, 21 Dec 2021 19:22:11 GMT
ClientError: error sending request for url (https://chrysalis-nodes.iota.cafe/api/v1/info): operation timed out
Tue, 21 Dec 2021 19:21:55 GMT
ReqwestError: error sending request for url (https://chrysalis-nodes.iota.cafe/api/plugins/participation/addresses/iota1qz6vwp7nx8m6dewpe70pthz55hvs45l8pkrkuwna9xe8n47mlsnssaw53ns): error trying to connect: tcp connect error: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond. (os error 10060)
Tue, 21 Dec 2021 19:21:46 GMT
ClientError: error sending request for url (https://chrysalis-nodes.iota.cafe/api/v1/info): operation timed out
Tue, 21 Dec 2021 19:21:15 GMT
ReqwestError: error sending request for url (https://chrysalis-nodes.iota.cafe/api/plugins/participation/addresses/iota1qz6vwp7nx8m6dewpe70pthz55hvs45l8pkrkuwna9xe8n47mlsnssaw53ns): error trying to connect: tcp connect error: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond. (os error 10060)
Tue, 21 Dec 2021 19:19:56 GMT
ClientError: error sending request for url (https://chrysalis-nodes.iota.cafe/api/v1/info): operation timed out
Tue, 21 Dec 2021 19:15:20 GMT
ReqwestError: error sending request for url (https://chrysalis-nodes.iota.cafe/api/plugins/participation/addresses/iota1qz82cr7lutztfg0jzlhv2mg9858hct82pv3cxwgjhduc03y6zxc2qf9r903): error trying to connect: tcp connect error: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond. (os error 10060)
Tue, 21 Dec 2021 19:14:54 GMT
StrongholdError: failed to perform action: Engine Error: Crypto Error: error in algorithm XCHACHA20-POLY1305, Unable to read snapshot. Please try another password.``
Tue, 21 Dec 2021 19:14:53 GMT
ReqwestError: error sending request for url (https://chrysalis-nodes.iota.cafe/api/plugins/participation/addresses/iota1qz75wl8w9m8v57gwnwm2h9ky72862c2dpppq5n2uxrw4rtnxryxwuw3j2rr): error trying to connect: tcp connect error: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond. (os error 10060)
Tue, 21 Dec 2021 19:14:46 GMT
StrongholdError: failed to perform action: Engine Error: Crypto Error: error in algorithm XCHACHA20-POLY1305, Unable to read snapshot. Please try another password.``
Tue, 21 Dec 2021 18:45:37 GMT
ClientError: error sending request for url (https://chrysalis-nodes.iota.cafe/api/v1/info): operation timed out
Tue, 21 Dec 2021 18:44:56 GMT
ClientError: error sending request for url (https://chrysalis-nodes.iota.cafe/api/v1/info): operation timed out
Sun, 12 Dec 2021 11:46:21 GMT
ClientError: {}
Sun, 12 Dec 2021 11:37:33 GMT
LedgerMiscError: ledger transport error
Sun, 12 Dec 2021 11:01:41 GMT
LedgerMiscError: ledger transport error
Sun, 12 Dec 2021 11:01:30 GMT
ClientError: {}
Sun, 05 Dec 2021 18:01:49 GMT
ClientError: {}
Sun, 05 Dec 2021 17:08:31 GMT
ClientError: {}
Sun, 05 Dec 2021 16:13:47 GMT
ClientError: {}
Sun, 05 Dec 2021 15:33:45 GMT
StrongholdError: failed to perform action: Engine Error: Crypto Error: error in algorithm XCHACHA20-POLY1305, Unable to read snapshot. Please try another password.``
Sun, 05 Dec 2021 15:33:44 GMT
StrongholdError: failed to perform action: Engine Error: Crypto Error: error in algorithm XCHACHA20-POLY1305, Unable to read snapshot. Please try another password.``
Sun, 05 Dec 2021 14:30:37 GMT
ClientError: {}
Mon, 04 Oct 2021 16:06:10 GMT
ClientError: {}
Sun, 01 Aug 2021 14:20:42 GMT
ClientError: {}
Sun, 01 Aug 2021 13:52:27 GMT
ClientError: {}
Wed, 02 Jun 2021 22:29:33 GMT
ClientError: {}
Wed, 02 Jun 2021 22:23:46 GMT
ClientError: {}
Sat, 29 May 2021 13:04:20 GMT
StrongholdError: failed to perform action: Engine Error: Crypto Error: error in algorithm XCHACHA20-POLY1305, Unable to read snapshot. Please try another password.``
Sat, 29 May 2021 12:56:05 GMT
ClientError: {}
Sat, 29 May 2021 12:31:52 GMT
ClientError: {}
Sun, 09 May 2021 22:21:53 GMT
ClientError: {}
Sun, 09 May 2021 20:29:24 GMT
ClientError: {}
Sun, 09 May 2021 20:01:02 GMT
ClientError: {}
Sun, 09 May 2021 14:46:43 GMT
ClientError: {}

We are working on a fix, will release a new version soon

We released a new version which fixed the blank screen issue
Announcement message
We just released Firefly IOTA 2.0.2 hotfixing the blank screen and accessing profiles due to error Chrysalis database migration failed: migration failed no chrysalis data to migrate :pray:
:arrow_down:  Download:
https://firefly.iota.org/
:bug:  Bug Reporting:
https://github.com/iotaledger/firefly/issues/new?assignees=&labels=bug+report&template=bug_report.yml&title=[Bug+report]%3A
Happy week to everyone :tulip: and apologies for the troubles :pray:

That's great. I have downloaded the latest version and everything is working fine. Thanks very much!


Make the latest doi-ui work with JPL AWS SSO

💡 Description

@tloubrieu-jpl @ramesh-maddegoda I believe this has been completed? Or does something additional need to be updated here?

Hi @jordanpadams , @ramesh-maddegoda made the DOI-UI work with a test cognito service deployed in NGAP but disconnected from JPL directory.
This ticket is about using the JPL AWS Cognito managed by the SA's. That might sounds a bit like going backward but that was the intention of the ticket. We can discuss that if needed.

Actually we will use the NGAP Cognito which needs to include the JPL directory connection.

A Jira ticket has been created to get support from ICAM (Identity, Credential, and Access Management) team.
https://jira.jpl.nasa.gov/browse/ICAM-8551

Had a meeting with ICAM team .The doi-ui Cognito integration with JPL SSO DEV was configured successfully during the meeting and tested.
Next steps are: ICAM team to setup same configs in JPL SSO INT and JPL SSO PROD environments.

Thanks @ramesh-maddegoda , if i am not mistaken, that would be our first service in production on NGAP, how will that be best to document that ? Any ideas on that @jimmie ? It sounds like a cognito user pool can be configured in terraform.
That would be good I think to have all our deployments coded/documented in terraform scripts and maintained in a github repository.

@tloubrieu-jpl, I have deployed Cognito user pools using terraform for another project. The open source code is available at https://github.com/unity-sds/unity-cs-infra/tree/main/terraform-api-gateway-cognito/terraform-modules/cognito-user-pool. We can do something very similar.

The setup for JPL SSO PROD is completed. Tested with JPL SSO prod with  doi gamma and it is working fine.

@ramesh-maddegoda will add the email of the PDS admins in the message for first login.
The service on gamma will use the test user pool.
The service in production will use the PDS admin group user pool.

Added the PDS Admin email address to the message which is displayed when user a does not have required Cognito groups.
@tloubrieu-jpl and @eddiesarevalo, please help with reviewing https://github.com/NASA-PDS/doi-ui/pull/176

@ramesh-maddegoda was not able to test, so the persons reviewing the PR need to test it

Blocked by ticket https://github.com/NASA-PDS/doi-ui/issues/178

@tloubrieu-jpl and @jordanpadams, this ticket is not blocked anymore and doi-ui is working as expected I ws able to test this with JPL SSO. We can close this now.


There is no support to directly import libraries (official and git)

The Arduino-CLI supports importing official libraries or libraries directly from a GitHub repository. However, this feature is not implemented yet in the chonos-firmwaremng.

#New Feature --importLibrary
Options
# importing from a .ZIP file
sudo chonosFirmwareManager --importLibrary -f /tmp/Adafruit_Zero_PDM_Library-1.2.3.zip

# importing from a URL
sudo chonosFirmwareManager --importLibrary -u https://downloads.arduino.cc/libraries/github.com/acrobotic/ACROBOTIC_SSD1306-1.0.1.zip

# importing from a GitHub repository
sudo chonosFirmwareManager --importLibrary -g chon-group/javino2arduino


No acces to necessary OnPrem Functionality "GetEmailBodyTextForCust"

Describe the request
Previous request: https://github.com/microsoft/ALAppExtensions/issues/26061
For context please see this previous ticket -> https://github.com/microsoft/ALAppExtensions/issues/25795
[Scope('OnPrem')]
procedure GetEmailBodyTextForCust(var ServerEmailBodyFilePath: Text[250]; ReportUsage: Enum "Report Selection Usage"; RecordVariant: Variant; CustNo: Code[20]; var CustEmailAddress: Text[250]; EmailBodyText: Text) Result: Boolean

This procedure on the "report selection" (77) table needs to have it's attribute onprem removed so that it can be called by us. Then we can easily replace the existing "ServerEmailBodyFilePath" With another one we require. without a very conveluted solution.
Additional context
As seen in the ticket https://github.com/microsoft/ALAppExtensions/issues/25795
local procedure OnSendEmailDirectlyOnBeforeSendFileLoop(ReportUsage: Enum "Report Selection Usage"; RecordVariant: Variant; DocNo: Code[20]; DocName: Text[150]; var DefaultEmailAddress: Text[250]; ShowDialog: Boolean; var TempAttachReportSelections: Record "Report Selections" temporary; var CustomReportSelection: Record "Custom Report Selection"; var IsHandled: Boolean; var ServerEmailBodyFilePath: Text[250])
Now this function has the variable "on email body filepath". This allows us to change the input of the body per email as we see fit which is very helpfull.
Problem being the way this filepath is normally obtained is by using the method. and then further down the file is used when sending the email.
table 77 report selection, line 1378
  FoundBody := GetEmailBodyTextForCust(ServerEmailBodyFilePath, ReportUsage, RecordVariant, CustNo, EmailAddress, EmailBodyText);
This method has the attribute OnPrem meaning we can not use it for custom implementation in our SAAS environment. Maybe there is a solid reason for this, but we currently cannot see why this would be the case.
If this cannot be changed then could you please offer a different solution to this problem? How should we obtain our "ServerEmailBodyFilePath"?
Internal work item: AB#504529

Internal work item is still active and being processed. Follow original issue for progress.


[mobile]expose onNetworkTypeChanged API to Engine

Commit Message: expose onNetworkTypeChanged API to Engine
Additional Description: This prevents applications from directly depending on InternalEngine.
Risk Level: low
Testing: n/a
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: mobile only

/retest


Add code formatting instructions to CONTRIBUTING.md

This removes the now obsolete idea-code-style.xml and adds some hints on how to set up local environment.

Codecov Report

Merging #1542 into master will not change coverage.
The diff coverage is 42.85%.

@@            Coverage Diff            @@
##             master    #1542   +/-   ##
=========================================
  Coverage     74.72%   74.72%
  Complexity     1737     1737
=========================================
  Files           328      328
  Lines          5281     5281
  Branches        962      962
=========================================
  Hits           3946     3946
  Misses          810      810
  Partials        525      525

Impacted Files
Coverage Δ
Complexity Δ

...n/io/gitlab/arturbosch/detekt/cli/out/XmlEscape.kt
64.02% <42.85%> (ø)
3 <0> (ø)
:arrow_down:

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 956fb89...9d209dc. Read the comment docs.


Directives ordering lint flags import directives in examples in doc comments

See https://github.com/flutter/flutter/issues/81689
This is currently blocking the flutter/engine to flutter/flutter roll.
@pq

This got resolved in the framework and landed.


Amend existing component: Notification

Component to amend
Notification
Exists or New
Exists
Visual
N/A
Context
from slack:
this is something small, but I think the dismissDirection prop is confusingly named, I think I would prefer for this prop to be named something like slideDirection since this is being used by the Slide animation for Notification onExit and onEnter? Additionally, when I dismiss the notification it just disappears, there is no associated animation.

@chanzuckerberg/sds-design what do you think here?

https://app.shortcut.com/sci-design-system/story/204673/updates-to-dismissdirection-prop-name-and-verify-slide-animation-working-properly


Support executing Pulley in Wasmtime

This commit is the initial implementation of executing the Pulley
interpreter from the wasmtime crate. This gives access to all of the
wasmtime crate's runtime APIs backed by execution of bytecode in
Pulley. This builds on the previous PRs I've been making for support in
Pulley to culminate in testing on CI in this PR. This PR handles some
final tidbits related to producing a runnable image that can be
interpreted by the wasmtime crate such as:

Pulley compilation artifacts are no longer marked as natively
executable, just read-only.
Pulley compilation artifacts include wasm-to-array trampolines like
normal platforms (removes a pulley special-case).
Dispatch of host calls from Pulley to the Wasmtime runtime are
implemented.
Pulley's list of panicking wasm features is slimmed down as most are
covered by "this lowering isn't supported" errors.
Execution of wasm code now has an if to see whether Pulley is
enabled within a Store or not.
Traps and basic "unwinding" of the pulley stack are implemented (e.g.
a "pulley version" of setjmp and longjmp, sort of)
Halting the interpreter has been refactored to help shrink the size of
ControlFlow<Done> and handle metadata with each done state.

Some minor refactorings are also included here and there along with a
few fixes here and there necessary to get tests passing.
The next major part of this commit is updates to our wast test suite
and executing all *.wast files. Pulley is now executed by default for
all files as a new execution engine. This means that all platforms in CI
will start executing Pulley tests. At this time almost all tests are
flagged as "expected to fail" but there are a small handful of
allow-listed tests which are expected to pass. This exact list will
change over time as CLIF lowerings are implemented and the interpreter
is extended.
Follow-up PRs will extend the testing strategy further such as:

Extending #[wasmtime_test] to include Pulley in addition to Winch.
Getting testing set up on CI for 32-bit platforms.

I'll note that this is in draft form as it currently is based on https://github.com/bytecodealliance/wasmtime/pull/9743. I'm going ahead and opening this up to see what CI has to say about this.

@fitzgen ok should be rebased and updated, I believe s390x CI is failing but I think most of the rest should be passing. Once results come in I'll figure out how to best ignore s390x for now.


Update base image in Docker file

What this PR does / why we need it:
upgrade base image for docker build
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

/lgtm
/approve


Apple Silicon Arm builds

Currently linux_arm builds are available, but Apple Silicon aarch64 builds are not.

To elaborate on the above, we use MailHog and mhsendmail in VVV. We actually don't use an arm64 build currently, but may well in the future. We currently use an aarch64 build (which is also not provided). I'm currently providing downloads for both MailHog and mhsendmail on both arm64 and aarch64 oh my forks.
https://github.com/evertiro/mhsendmail/releases/tag/v0.2.0-M1
https://github.com/evertiro/MailHog/releases/tag/v1.0.0-M1

@evertiro what about docker though?

@abbluiz are you suggesting we use docker + QEMU x86 emulation? What about Raspberry Pis, arm servers, or native Apple Silicon?

@tomjn no, I was just asking really... because there's no arm docker images for mailhog and that would be good, and it would on apple m1 & other arm-based docker hosts

@abbluiz Exactly. I came here from Docker Hub, and I found this is the only ticket related to AArch64/Apple Silicon. Will be subscribing to notifications on this dicussion.

Would be great to have an official arm64 build for M1 based machines.
👍

For those looking for an arm64/aarch64 Docker image, I built one here:
https://hub.docker.com/repository/docker/jcalonso/mailhog
There is also an arm64 binary here:
https://github.com/jcalonso/MailHog/releases/tag/v1.0.1-arm
The arm build was tested on AWS Graviton t4g and Raspberry pi, I don't have a mac to test there but it should work™.
I would be great to have this on the official repo, let me know if the releases work fine and I can open an PR.

@jcalonso your binary works perfectly inside a Debian 11 arm64 VM on an Apple Silicon Mac

@jcalonso Thanks for making that version!  Working on Ubuntu 22 on M1 Mac.  Any chance you could also make an arm64 version of tkotosz/mhsendmail and tkotosz/mhsendmail-wrapper?

I discovered it's a piece of cake to compile these:

Install golang-go
Clone MailHog or mhsendmail
cd to the directory and run make and run the command it suggests to fix the errors, and run make again
The binaries should be in ~/go/bin

I LITERALLY just forked the image and ran
docker buildx build --push --platform linux/arm64,linux/amd64 -t arjenz/mailhog:latest .

to build and push it to Docker Hub. You're free to use it: https://hub.docker.com/r/arjenz/mailhog
If you want to try out a new project, created by a colleague of mine (in Rust), here's MailCrab: https://github.com/tweedegolf/mailcrab ! Also available on Docker Hub: https://hub.docker.com/r/marlonb/mailcrab


PrimaryKey data type is ignored in the migration ("serial" is always used)

When I generate the migration code related to the following model:
@serializable
@orm
abstract class _User {
  @primaryKey
  String get email;
  String get name;
  String get password;
}

The primary key is generated as a number, whereas it should be a string.
  @override
  up(Schema schema) {
    schema.create('users', (table) {
      table.declare('email', ColumnType('serial'))..primaryKey();
      table.varChar('name');
      table.varChar('password');
    });
  }

Forcing the type to varchar, is workaround to the problem :
@serializable
@orm
abstract class _User {
  @PrimaryKey(columnType: ColumnType.varChar)
  String get email;
  String get name;
  String get password;
}

I have removed the default column type in the @PrimaryKey() annotation, which will allow the generator to assume the type by default.

https://pub.dev/packages/angel_orm_generator/versions/2.0.5


Fix N+1 problem in recent reports

[x] Fix N+1 problem in recent reports (layout/application and layout/admin)

Before

After

[x] Fix page_classes. There were weird class names in some views.

# Before
<body class="user_sessions user_sessions-new no-headerno-footerno-recent-reportsno-global-nav" id="body">

# After
<body class="user_sessions user_sessions-new no-header no-footer no-recent-reports no-global-nav" id="body">

@komagata Could you please review? Thanks!

@rono23 😃


Creating the inspec_waiver_file resource for managing and formatting a waiver file

Signed-off-by: Davin Taddeo<EMAIL_ADDRESS>
Description

Chef recently introduced the use of waiver files by InSpec, as well as use via the audit cookbook.  There isn't a corresponding chef-client resource to manage such a file through chef automation if a user wanted to.  This inspec_waiver_file resource will provide that functionality.
Add an InSpec waiver entry to a given waiver file:
  inspec_waiver_file 'Add waiver entry for control' do
    file 'C:\\chef\\inspec_waiver.yml'
    control 'my_inspec_control_01'
    run_test false
    justification "The subject of this control is not managed by Chef on the systems in policy group \#{node['policy_group']}"
    expiration '2021-01-01'
    action :add
  end

Add an InSpec waiver entry to a given waiver file using the 'name' property to identify the control:
  inspec_waiver_file 'my_inspec_control_01' do
    file 'C:\\chef\\inspec_waiver.yml'
    justification "The subject of this control is not managed by Chef on the systems in policy group \#{node['policy_group']}"
    action :add
  end

Remove an InSpec waiver entry to a given waiver file:
  inspec_waiver_file "my_inspec_control_01" do
    file '/etc/chef/inspec_waiver.yml'
    action :remove
  end

Related Issue

N/A
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] All commits have been signed-off for the Developer Certificate of Origin.

The example provided needed the expiration to be encapsulated with single quotes in order to not throw an error: expiration '2021-01-01'. Also, I've been using my own recipe for a few days now that takes a different approach, it uses a template and allows you to control the entire waiver file with attributes dynamically. Just wanted to bring it up as another possible route to go down, or maybe combine the dynamic nature of a template + .each loop for attributes with a nice resource like this that takes an attribute hash and iterates over it (kinda like how the profiles attribute is set in the audit cookbook).

expeditor/cleanup_orphaned_test_resources.sh seems unrelated and uncalled by anything...
other than that, this looks fine though.

expeditor/cleanup_orphaned_test_resources.sh seems unrelated and uncalled by anything...
other than that, this looks fine though.

Not sure how that got in there, but it's been removed.

From our Slack conversation on this:

rename to inspec_waiver_file_entry
rename file to file_path and make it default to /etc/chef/inspec_waivers.yml and C:\chef\inspec_waivers.yml