Add teams notification feature - Enabling push notifications to teams channel

Integrating the notifications to Teams Channel
Description
I've expanded the notification capability within the spark_expectations library to include Microsoft Teams alongside the existing Slack notifications. This improvement allows users to receive updates on data quality job execution status across both Slack and Teams platforms.
Related Issue
Please link to the issue here: https://github.com/Nike-Inc/spark-expectations/issues/56
Motivation and Context
The motivation behind this change is to broaden the reach of data quality framework execution status notifications. While Slack is a popular communication platform, some organizations prefer or exclusively use Microsoft Teams. By incorporating Teams notifications into the spark_expectation library, we enhance its usability for a wider audience, ensuring that data quality job execution status alerts are accessible across multiple collaboration tools.
How Has This Been Tested?
To ensure the reliability and functionality of the Teams notification feature, I thoroughly tested the following aspects:
Unit Tests:- Implemented unit tests to verify the correctness of the Teams notification implementation.
Integration Testing: Conducted integration tests by simulating different data quality scenarios and validating the generation and delivery of notifications to Teams.
Environment: Tested the feature in various environments like databricks, local IDEs as well as to ensure compatibility and functionality across different setups.
Message Formatting: Checked the appearance and readability of messages sent to Teams to ensure they display correctly.
Link to Unit test doccument:-
This comprehensive testing approach aimed to validate the feature's effectiveness, compatibility, and correctness before integrating it into the spark_expectations library.
Screenshots (if appropriate):
Test Results & Proofs - SE Teams Notif Feature.zip
Please refer the attachments section
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)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

@BharatKumarEY Thanks for the contribution!
Looks like the build failed because of the linting issues. Can you please run make fmt in your local and again run make cov to make sure the tests pass and push the code again.

@asingamaneni Linting issues got resolved, I've ran the make fmt and make cov in my local and it's success.

@asingamaneni I see the Deploy failed due to type error, we fixed it and ran the mypy and now it it's success and also ran the make cov
report too. PFA of the screenshots.


LoadImagesFromPatch stopped pausing the work flow until the image changes and now works constantly

Previously, my workflow was launched for execution only after saving the PSD. And now there is no pause, the cycle is always on.
This is bad because... the video card works all the time (and is generally inconvenient)

I solved the problem by setting the following settings to execute. Now the workflow starts as before (only after saving the PSD)


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


underlying problem

Some ERC20 standard transfer functions do not revert if the transaction fails, which can result in part of the recipients receiving funds. To avoid this issue, you can consider using the safeTransfer function from the OpenZeppelin library. I think.

Thank you for your suggestion. Initially, I did consider using safeTransfer. Considering that the usdt.transfer method always returns false on the TRON chain, I chose to use the current interface form for transfer without checking the return value. This allows me to deploy the same code on multiple chains.
After all, most tokens use safeMath in the transfer, and will revert if they fail; However, I have also seen some tokens on etherscan that return false when the conditions are insufficient. We will make further improvements in the next version.


Wrong demo url

There is an error in the demo url. The following url is wrong: https://your-account.blob.core.windows.net/azure-ad-b2c/unified.html
it should be:
https://your-account.blob.core.windows.net/root/azure-ad-b2c/unified.html

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

ID: 84aabaf3-e7da-3519-995c-72d36c537843
Version Independent ID: b35afa8d-a02c-a77a-6831-2b3f6a6c1a7d
Content: Customize the user interface of your app with a custom policy - Azure AD B2C
Content Source: articles/active-directory-b2c/custom-policy-ui-customization.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@sven5 Thank you for pointing this out . We will investigate and update the document as needed.

@sven5 We checked this and you are correct . The URL is missing blob containers name as suggested in the article above in section 2.1 where we create a container . We have intiated a pull request and its under review with the content team. Once it is approved , the change should be live in the next 24-48 hours after approval. We will now close this issue.  Thank you for your contribution.


Google UI changes

Google changed to developer console UI, updated walkthrough to match new UI.

:memo:++ :star: :rocket:
Thanks for the doc updates!


Move Hal Plugin to ServiceManager to remove circular dependancy issue

This plugin is now accessed from both ControllerPlugin and ViewHelper directly via the ServiceManager,
removing the circular dependancy issue #82

I'm closing this PR, after looking further into the Plugin there needs to be more work to get this right. Currently it doesn't work at all because of the circular dependancy since the commit 3 days ago. This PR fixes that but I'm not happy with it and its impossible to set up satisfactory unit tests.


feat: Add new check to code-checker

PR Type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Context

[x] Closes #10454

Needs QA check

@kodadot/qa-guild please review

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Screenshot 📸

[x] My fix has changed something on UI;

cc @Jarsen136


configure_jwt_authorization_flow() does not honor the explicit OAuth grants via oauth URL. Throws 'Consent Required" error

Despite granting access by logging in using the URL given by oauth_login_url below, the subsequent "api_client.configure_jwt_authorization_flow" call always results in 'consent_required' error. The corresponding integrator key is set up with right redirect URI and key pair (of which I use the private key as private_key_filename below). However, using the test parameters (user_id, integrator key, etc) from the unit_test works fine. There is some other nuance or detail that is explicitly missing in the example or prerequisites. Note that the account is not associated to any Organization yet. I am not yet there. But I would expect this basic flow to work as is.
oauth_login_url = api_client.get_jwt_uri(integrator_key, redirect_uri, oauth_base_url)

print(oauth_login_url)

       https://account-d.docusign.com/oauth/auth?response_type=code&client_id=<integrator_key>&scope=signature%2Bimpersonation&redirect_uri=https%3A%2F%2Fwww.docusign.com%2Fapi

api_client.configure_jwt_authorization_flow(private_key_filename, oauth_base_url, integrator_key, user_id, 3600)

  Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/userme/projects/docudocu/lib/python2.7/site-packages/docusign_esign/api_client.py", line 118, in configure_jwt_authorization_flow
    post_params=self.sanitize_for_serialization({"assertion": assertion, "grant_type": "urn:ietf:params:oauth:grant-type:jwt-bearer"}))
  File "/Users/userme/projects/docudocu/lib/python2.7/site-packages/docusign_esign/api_client.py", line 418, in request
    body=body)
  File "/Users/userme/projects/docudocu/lib/python2.7/site-packages/docusign_esign/rest.py", line 244, in POST
    body=body)
  File "/Users/userme/projects/docudocu/lib/python2.7/site-packages/docusign_esign/rest.py", line 200, in request
    raise ApiException(http_resp=r)

    docusign_esign.rest.ApiException: (400)
    Reason: Bad Request
    HTTP response headers: HTTPHeaderDict({'X-DocuSign-Node': 'CH1DFE2', 'Content-Length': '28', 'Strict-Transport-Security': 'max-age=31536000; includeSubDomains; preload', 'X-Content-Type-Options': 'nosniff', 'Content-Type': 'application/json; charset=utf-8', 'Expires': '-1', 'Content-Security-Policy-Report-Only': "script-src 'unsafe-inline' 'unsafe-eval' 'self';style-src 'unsafe-inline' 'self';img-src https://docucdn-a.akamaihd.net/ 'self';font-src 'self';connect-src 'self';object-src 'none';media-src 'none';frame-src 'none';frame-ancestors 'none';report-uri /client-errors/csp", 'X-XSS-Protection': '1; mode=block', 'X-DocuSign-TraceToken': 'b009b0a6-19ad-4e58-844e-76fc5b509cbb', 'Pragma': 'no-cache', 'Cache-Control': 'no-cache', 'Date': 'Wed, 29 Nov 2017 22:52:10 GMT', 'X-Frame-Options': 'SAMEORIGIN', 'X-AspNetMvc-Version': '5.2'})
    HTTP response body: {"error":"consent_required"}

(Note: Also asked a question here https://stackoverflow.com/questions/47563725/getting-consent-required-error-despite-successful-permission-grant-using-oauth )

@rajaramsrn Looks like Amir and Larry are trying to help you S.O. please don't forget to close this issue once they solve your issue. I will make sure to post a summary of the solution here, once the issue if resolved.

@mmallis87 Please review my PR and S.O. response for this issue. Thank you.

If there are plans to fix this issue of delimiter in get_jwt_uri() utility much more comprehensively along with the CI test issues, I am fine to ignore my PR. But I seriously need this capability to generate and suggest the correct 'Grant requesting URL with 2 claims' to my app users as per the JWT OAUTH doc. Either way please let me know what your release plans are to make such fixed pip image Thanks in bunch.

The PR was merged and a new version will be released in PyPi (pip) soon. I will close this once it's shipped.

Shipped: https://pypi.python.org/pypi/docusign-esign/1.0.2 @


[WIP]: fix wrong flow order in logout process

The logout event for a delegated Auth with pac4j appear too late in the Logout process and need that the ProfileManager exist. For that the DelegatedAuthenticationSAML2ClientLogoutAction class should be called before org.apereo.cas.web.flow.logout.TerminateSessionAction.destroyApplicationSession()

Warning it's in a Work In Progress, I can't find the best suitable way so I request some help.

Ok I'm getting this error (not builded with my custom version of the lib before):
2019-07-08 10:57:12,048 ERROR [org.apereo.cas.web.flow.configurer.AbstractCasWebflowConfigurer] - <Cannot cast org.springframework.webflow.engine.EndState to org.springframework.webflow.engine.DecisionState>
java.lang.ClassCastException: Cannot cast org.springframework.webflow.engine.EndState to org.springframework.webflow.engine.DecisionState
	at java.lang.Class.cast(Class.java:3606) ~[?:?]
	at org.apereo.cas.web.flow.configurer.AbstractCasWebflowConfigurer.getState(AbstractCasWebflowConfigurer.java:728) ~[cas-server-core-webflow-api-6.1.0-RC5-SNAPSHOT.jar!/:6.1.0-RC5-SNAPSHOT]
	at org.apereo.cas.web.flow.DelegatedAuthenticationWebflowConfigurer.createSaml2ClientLogoutAction(DelegatedAuthenticationWebflowConfigurer.java:55) ~[cas-server-support-pac4j-webflow-6.1.0-RC5-SNAPSHOT.jar!/:6.1.0-RC5-SNAPSHOT]
	at org.apereo.cas.web.flow.DelegatedAuthenticationWebflowConfigurer.doInitialize(DelegatedAuthenticationWebflowConfigurer.java:49) ~[cas-server-support-pac4j-webflow-6.1.0-RC5-SNAPSHOT.jar!/:6.1.0-RC5-SNAPSHOT]
	at org.apereo.cas.web.flow.configurer.AbstractCasWebflowConfigurer.initialize(AbstractCasWebflowConfigurer.java:114) ~[cas-server-core-webflow-api-6.1.0-RC5-SNAPSHOT.jar!/:6.1.0-RC5-SNAPSHOT]

I close the PR as the problems doesn't come from here. The clientName can't be retrieved at logout because no profile can be retrieved from the sessionStore. I can't find where at the moment the credentials from delegated Auth aren't saved or removed before the logout Action.
(the webcontext constains only the redirect_url, even if I put the pac4j logout action at first place in the webflow)


OpenAL AL/ALC extensions should receive fully populated capabilities object

Environment

LWJGL version: 3.2.2
LWJGL build #: unknown
Java version: 1.8.0_261 amd64
Platform: Windows 10 64-bit
Module: openal

Description
Calling extension methods will crash the application if pointer checking is enabled and an OpenAL router is being used (instead of the OpenAL driver DLL directly).
This makes it impossible to use OpenAL Soft through a router because AL.createCapabilities calls alcGetThreadContext:
https://github.com/LWJGL/lwjgl3/blob/b07955a2ad8b66deae2b1c5093511d1eb10e7d84/modules/lwjgl/openal/src/main/java/org/lwjgl/openal/AL.java#L218
However alcGetThreadContext does not receive a fully populated ALCCapabilities, it generates a new one that does not use alcGetProcAddress with the correct function pointers:
https://github.com/LWJGL/lwjgl3/blob/5ab5b35ab0cae4116b7bd547d04002ba74d2ff77/modules/lwjgl/openal/src/generated/java/org/lwjgl/openal/EXTThreadLocalContext.java#L47
This pattern is seen in various extensions, and will cause problems when using a router. Extensions should not call getICD, but always receive the fully populated capabilities.
How to reproduce

Install OpenAL from https://www.openal.org/downloads/oalinst.zip, System32's OpenAL.dll will be the router for our application.
Download OpenAL Soft from https://openal-soft.org/ and copy to System32 as soft_oal.dll
Run the following code, setting -Dorg.lwjgl.openal.libname="C:/Windows/System32/OpenAL32.dll"

int deviceId = ALC10.alcOpenDevice("OpenAL Soft");
ALCCapabilities alcCaps = ALC.createCapabilities(deviceId);
ALCapabilities alCaps = AL.createCapabilities(alcCaps); <-- Should throw exception here on line 218 from AL.class

If wrap_oal.dll is installed, the following will work as it does not support EXT_thread_local_context:
int deviceId = ALC10.alcOpenDevice("Generic Software");
ALCCapabilities alcCaps = ALC.createCapabilities(deviceId);
ALCapabilities alCaps = AL.createCapabilities(alcCaps);

Hey @LAGonauta,
I've been working on this for a while now. Fixing the problem you described was relatively simple, but then I hit a different problem with ALC_EXT_thread_local_context. I would get a properly initialized ALCCapabilities, ALC_EXT_thread_local_context was reported as available and I could call its functions, but alcSetThreadContext always returned false. I had to build a custom OpenAL Soft and a custom router to figure it out:

When the router (OpenAL32.dll) from https://www.openal.org/ is used, the ALCcontext handle returned to the client program is actually a wrapped handle. It is not the actual context handle created by the implementation (soft_oal.dll, wrap_oal.dll, etc.). Normally, when an ALC function is called, that handle is unwrapped and forwarded to the implementation. This does not happen when ALC_EXT_thread_local_context functions are called. OpenAL Soft receives the wrapped handle, which it does not recognize, so it sets the error state to ALC_INVALID_CONTEXT and returns without doing anything.

When the router from OpenAL Soft is used... we crash on alcOpenDevice. OpenAL Soft's router is still experimental, so I had a look and there was a nested locking bug (had to replace an std::mutex with an std::recursive_mutex). With that fixed, alcSetThreadContext works fine and everything is OK (even with non-OpenAL Soft implementations).

I will push the ALCCapabilities fix when I clean it up, but my recommendation would be: don't use the router, go to the implementation directly. You could search for implementations manually if you have to (look for *oal.dll in the usual paths).

Thanks @LAGonauta!


Refactor to use state_is_tuple=True

In TensorFlow 0.11, the default behavior is for model states to be represented as tuples; forcing the old concatenated behavior using state_is_tuple=False prints a deprecation warning.
Adapting the code to work with state_is_tuple=True turned out to be easier than I expected, because the TensorFlow example script ptb_word_lm.py is structured very similarly to char-rnn-tensorflow and was refactored back in August to use tuple states. This pull request ports the changes from that example to char-rnn-tensorflow.

Thanks!


[🚀] Export/Import for settings and progress

:rocket: Feature request
Description
It would be nice to have the chance to save

hacking progress (continueCode cookie)
local settings

cookies language, cookieconsent_status and welcomebanner_status
local storage displayedChallengeCategories, showOnlyTutorialChallenges, showDisabledChallenges and displayedDifficulties

in a local file and restore from that file later on the same or a different computer.
Solution ideas
The settings/progress could be saved in a JSON object and then exported via JSON.stringify into https://www.npmjs.com/package/file-saver. Restore mechanism could be a lonal-only file upload parsing the JSON string and populating the corresponding items in cookies and local storage.
Possible alternatives
Some kind of sophisticated cloud storage and syncing like IntelliJ... 🤣

Preliminary save/restore: 00a8573030010e3d972ccb90679572bffb66bcbd

Save and Restore is implemented, I need to add some tests and also allow the whole feature to be turned off (e.g. by default in CTF mode) because it's just too convenient to cheat with this.


Packaging

Tracking

[ ] Debian
[ ] Mac (homebrew)

Yes, for ubuntu. Thank you!

Okay I'll look into this. I plan to release version 0.3 this week, and one of my goals for that release is to provide binaries in the release which should make it easier for those without a Zig compiler.

If can, please consider homebrew also.
Thanks so much.

@mathjiajia homebrew is a great idea! I develop on Arch Linux, but I know a few people who I can test zf on macs with.
Currently to my knowledge zf does not work properly on macs. The interface does not draw properly. I plan to fix this this week, it shouldn't be more than fixing a few syscalls.
I'm going to update this issue to be tracking support for various package managers

@mathjiajia homebrew is a great idea! I develop on Arch Linux, but I know a few people who I can test zf on macs with.
Currently to my knowledge zf does not work properly on macs. The interface does not draw properly. I plan to fix this this week, it shouldn't be more than fixing a few syscalls.
I'm going to update this issue to be tracking support for various package managers

I see. Thank you.
I have tried zf-native.nvim with telescope, and it is great. Matching filename (with higher priority) is what I need exactly.

@mathjiajia I have looked a bit into homebrew now. From their docs

We will reject formulae that seem too obscure, partly because they won’t get maintained and partly because we have to draw the line somewhere.
We frown on authors submitting their own work unless it is very popular.

I fear that zf might be a bit too young for homebrew to accept, and it sounds like they would prefer someone other than me make the package.
It seems that I could maintain my own tap. I'm not a user of homebrew, but I could imagine some people would prefer a package in the official core repo.
I do plan to make macos binaries available soon with future releases, but I'm not sure how to go forward with zf and homebrew. (Maybe spread the word to anyone you think might enjoy it? :smile: ) Perhaps someone with more experience with homebrew could give some advice too.

I fear that zf might be a bit too young for homebrew to accept, and it sounds like they would prefer someone other than me make the package.

I think so.

It seems that I could maintain my own tap. I'm not a user of homebrew, but I could imagine some people would prefer a package in the official core repo.

Thanks. Maintaining your tap is a good choice. When zf becoming popular, it will be easy to add into homebrew core.

I do plan to make macos binaries available soon with future releases, but I'm not sure how to go forward with zf and homebrew. (Maybe spread the word to anyone you think might enjoy it? 😄 ) Perhaps someone with more experience with homebrew could give some advice too.

Anyway, thanks so much for your efforts @natecraddock 👏

@natecraddock: I saw that you said you develop on Arch Linux. I'm assuming that you're familiar with the PKGBUILD format, in which I think makedeb would work great for creating Debian packages (prenote, it's a personal project).

@hwittenborn sorry for taking so long to reply! I think makedeb looks wonderful. I'll take a closer look soon, but from what I can tell it just takes PKGBUILD files and outputs a .deb file?

Not a problem @natecraddock :)

it just takes PKGBUILD files and outputs a .deb file?

That's correct, yes.

If can, please consider homebrew also. Thanks so much.

Not sure if you are still interested @mathjiajia, but I have just added instructions to the readme for brew installation instructions! I have tested myself and everything seems to work. I'm still new to homebrew, so let me know if things don't work

@natecraddock Thank you for reminding me about this news.
I would like to try it now.

@natecraddock Please also consider having pre-built binaries uploaded with new releases directly on GitHub. This will allow downloading zf directly off GitHub via something like: https://github.com/zyedidia/eget

@dufferzafar thanks for the reminder! I have been intending to do this for a while now, I'll try to get that done soon

@dufferzafar I cross-compiled some testing binaries to the [latest release]. I tested the cross-complied macos x86 binary with eget, but I can't verify if others work. If all works well, I'll look into adding some GitHub actions to automate these builds. Let me know if it works!

I have opened a PR for packaging this to nixpkgs - https://github.com/NixOS/nixpkgs/pull/187403

@dit7ya Thanks for doing that! I subscribed to the PR and I'll update the zf readme when it is merged.


[BUG] Curve TWAP Bot fails with Front-End Alert "Error - Please stop Bot"

Transaction: https://etherscan.io/tx/0x637290663ce90d2316e0922d585270d8136abf18f0ab60170c2c75078303e5d4
User then sees the MyBots page error to Stop bot.

We implemented an error field that results in this FE message.
Here is the code line https://github.com/VolumeFi/curve-twap-bot-script/blob/master/bot_execute.py#L308 where error is set to  1 if the bot can't be processed by the backend. If Error is 1, then the FE displays this message so the user can withdraw https://github.com/VolumeFi/palomabot/pull/231.
Here is the ticket that requested the backend change: https://github.com/VolumeFi/paloma/issues/935


A problem about running VariantQC

Hi,
I meet a problem about running VariantQC.
code:
java -jar ~/DISCVRSeq-1.19.jar VariantQC -O ./gbm39_vcf_report.html -R ~/ref-vcf-hg38/resources_broad_hg38_v0_Homo_sapiens_assembly38.fasta -V ./gbm39_hg38_sv.vcf
error log:

UNEXPECTED ERROR: Couldn't find state for chr9_KI270718v1_random at node org.broadinstitute.hellbender.tools.walkers.varianteval.stratifications.manager.StratNode@39fcbef6

Please, search for this error in our issue tracker or post a new one:
org.broadinstitute.hellbender.exceptions.GATKException: Couldn't find state for chr9_KI270718v1_random at node org.broadinstitute.hellbender.tools.walkers.varianteval.stratifications.manager.StratNode@39fcbef6
Thank so much!

@groveswallow is there any chance the set of contigs in the VCF and reference FASTA are out of sync?

emmm,I have checked the file of fa and vcf.The results as follows:
The part head of vcf:
##contig=<ID=chr2_KI270716v1_random,length=153799>
##contig=<ID=chr3_GL000221v1_random,length=155397>
##contig=<ID=chr4_GL000008v2_random,length=209709>
##contig=<ID=chr5_GL000208v1_random,length=92689>
##contig=<ID=chr9_KI270717v1_random,length=40062>
##contig=<ID=chr9_KI270718v1_random,length=38054>
##contig=<ID=chr9_KI270719v1_random,length=176845>
##contig=<ID=chr9_KI270720v1_random,length=39050>
##contig=<ID=chr11_KI270721v1_random,length=100316>
And view the chr9_KI270718v1_random in reference file seem to be also right.
my code:
samtools faidx ref.fasta chr9_KI270718v1_random > chr9_KI270718v1_random.fa
head of chr9_KI270718v1_random.fa:

chr9_KI270718v1_random
AGGATCCTCTTCCTCATCCACAGATCCAACAGCCGGTAACATTCCGGCCTACAGCACGTC
CCTACCTCAGCTGAACCGCTGGGACCAACACGCCGTACCGCAGCCGCGCACGCCCAGCAC
GCCACTCCCGGCCTGGCCGGGGCCTGACGTCATCACATCGCGACGCTAGCTGATCCCGCC
CGGATTACCTGGGCTCTGGACGAGAACTGCCTCGTCCCTTAGCCACACCCCCCGGGGAGG
TGGGCTCTCCCCGACGACAGCTGGACACGCCCCTGGGCGTGTTCTGTGAACCTGAGCGAC
GCTGCGGAGAGTCCTTAGGGCGCACTAACCACCTGTCTGAAGGTGTCACTGGCGCCTAGA
GCCTTAATAAGCTACTGGCTGTAGGTAGAGCTGAGGCCAGCCTGCTGGAAATCTGGAGTG
GGTAGCCTGACCTAACCAGTTCCCACAGGGCATAGATTTTATCAACTTGATTTTTTAGTC
TCGTTAAAAAAAAAAGTTTTAATTCATAGGGGAAAGAATGGAAGGTTATAACATTACAAT

@groveswallow can you please do a few things to help debug this:

can you please post your command and the full output from it?

can you run the command with "-L chr9_KI270718v1_random" added? This should in theory limit to that contig. Can you also pick another small contig, perhaps chr9_KI270717v1_random and try this by itself?

Thanks

@groveswallow I'm writing to follow up: did you end up resolving this? if you'd like us to look into it, can you please post the items we asked about above? thanks.

@groveswallow I'm writing to follow up: did you end up resolving this? if you'd like us to look into it, can you please post the items we asked about above? thanks.

Really sorry about that.I did not resolve this question.I will try to find the items and post.

@groveswallow OK, that's fine. If there was an easy solution I'd be curious to know (to see if we could make error reporting more informative, for example), but overall glad to hear its working.


Deselect not possible for single selection

Choosen nicely supports deselection in single select mode. You only have to ensure that the first option tag is empty. Unfortunately, this no longer works with ajax-choosen.
It would be great if that was included.

I found another problem.
It is in chosen.js at init line:
this.allow_single_deselect
I removed : (this.form_field.options[0] !== null && this.form_field.options[0] !== undefined) && this.form_field.options[0].text === ""
And then allow_single_deselect is defined from options.

@wejrowski Could you please post a full example? I also have problems, this problem is driving me nuts...


Create a release puchnlist

Ideally automation fixes most of this and our punchlist is minimal.

I'll PR some extra verification steps.

Duplicate of #409


projectile-find-tag: No applicable method: xref-backend-identifier-completion-table, nil

Here are some things you should try before filing a bug report:
Observed behavior
CTRL-] used to find a tag now raises No applicable method: xref-backend-identifier-completion-table, nil
I see CTRL-] in bound to projectille-find-tag
Expected behavior
I would have expected the same behavior as what we have with find-tag
Steps to reproduce

Create a tag file (ctags -e)
start emacs
go over a name an type CTRL-]

System information

emacsclient 25.2.2
console mode

System information
DOOM Doctor
Running Emacs v25.2.2, commit 7bfd7c1ade93ac16788eae90d72a182ddcd32987

shell: /bin/bash
Compiled with:
  SOUND NOTIFY LIBSELINUX ZLIB
uname -a:
  PST 2018 i686 i686 i386 GNU/Linux

Attempt to load DOOM: success! Loaded v2.0.9
Revision 7bfd7c1ade93ac16788eae90d72a182ddcd32987

----

test-emacs
test-windows
test-fonts
Warning: couldn’t find material-design-icons.ttf font in /home/amoriello/.local/share/fonts/
  You can install it by running ‘M-x all-the-icons-install-fonts’ within
  Emacs.

  This could also mean you’ve installed them in non-standard locations,
  in which case, ignore this warning.
Warning: couldn’t find weathericons.ttf font in /home/amoriello/.local/share/fonts/
  You can install it by running ‘M-x all-the-icons-install-fonts’ within
  Emacs.

  This could also mean you’ve installed them in non-standard locations,
  in which case, ignore this warning.
Warning: couldn’t find octicons.ttf font in /home/amoriello/.local/share/fonts/
  You can install it by running ‘M-x all-the-icons-install-fonts’ within
  Emacs.

  This could also mean you’ve installed them in non-standard locations,
  in which case, ignore this warning.
Warning: couldn’t find fontawesome.ttf font in /home/amoriello/.local/share/fonts/
  You can install it by running ‘M-x all-the-icons-install-fonts’ within
  Emacs.

  This could also mean you’ve installed them in non-standard locations,
  in which case, ignore this warning.
Warning: couldn’t find file-icons.ttf font in /home/amoriello/.local/share/fonts/
  You can install it by running ‘M-x all-the-icons-install-fonts’ within
  Emacs.

  This could also mean you’ve installed them in non-standard locations,
  in which case, ignore this warning.
Warning: couldn’t find all-the-icons.ttf font in /home/amoriello/.local/share/fonts/
  You can install it by running ‘M-x all-the-icons-install-fonts’ within
  Emacs.

  This could also mean you’ve installed them in non-standard locations,
  in which case, ignore this warning.
test-gnutls
test-tls
Warning: You didn’t install Emacs with gnutls support
  This may cause ’pecular error’ errors with the Doom doctor, and is
  likely to interfere with package management. Your mileage may vary.
test-tar
Everything seems fine, happy Emacs’ing!

Add this to your private module (config.el):
(defun my*projectile-find-tag ()
  (interactive)
  (projectile-visit-project-tags-table)
  (let ((xref-backend-functions '(etags--xref-backend t)))
    (let ((find-tag-fn (projectile-determine-find-tag-fn)))
      (call-interactively find-tag-fn))))
(advice-add #'projectile-find-tag :override #'my*projectile-find-tag)

And let me know if that fixes your issue.

Yup, this fixes the issue. Thanks!


feat: Ignoring isDefault field for branch in diff

Jira: PSGO-738
Changes:

ignoring isDefault field for branch in diff
added test

https://keboola.atlassian.net/browse/CFT-3011


[Question] Handling array of structs with bus drivers

Hey @alexforencich,
thanks for the repository, I still didn't have time to implement it but I went through it shortly and it's really amazing. Sorry for raising this issue but didn't know a better way to ask you this question but do you have an idea of how to connect your AXIMaster bus drivers for an AXI slave that's an array of structs and it has different names for miso/mosi signals?

SV Structs:

simengine: verilator-master
Thanks!

I just added new objects that will hopefully be able to make this sort of thing easier.  Right now they are just wrappers around the Bus object, but I'm planning on doing a more complete rewrite.  I'm not sure exactly what it's going to look like at this point, but I'm definitely going to keep this particular use-case in mind, perhaps by supporting matching signals from multiple source entities.

oh great, thanks for the heads up! Please keep as an example too if possible....

btw, if you're willing to have something in SV, use Verilator 4.106 once the latest is hanging due to VPI changes


Vue week view

I am forced to use 1.38.0 for @schedule-x/vue, because the latest version does not show the week view by default. The only workaround is to trigger a window resize event.

Do you set defaultView to viewWeek.name?
Please always include more context data when you open issues. I'd very much like to help you, but without code for setup, version numbers, or best case even a minimal reproduction environment, it's hard to be of any help.
When I try setting the week view on page load it works.

Do you set defaultView to viewWeek.name?
Yes

Please always include more context data when you open issues.
This is what I am currently using:

npm ls
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>
node --version
v18.17.1

So, I had this issue in the past, even this morning, but somehow now, I cannot reproduce it anymore. I will close it for now.


Support extra data and files in a more general way

This allows specifying more complex workloads without special casing the template code.
metadata:
  name: myworkload
extra_files:
  - file1.yaml
  - file2.yaml
template_data:
  shared:
    ...shared_data
  run_type:
     perf_ci:
       ... perf_ci data
     default:
       ... data if no other run type matches
  kind:
    vm:
      run_type:
        perf_ci:
        default:
      default:
  extra:  # Do we need sub-structuring here?
    mariadb:
    default:

Currently a draft PR; this is for discussion rather than immediate integration.

@ebattat @jeniferh please review

Passed e2e and unit tests on my local cluster.  Most of the integration tests passed except for the ones that always fail on my cluster.

@RobertKrawitz,
I think that we need to change the name of the file name to more meaning name:
benchmark_runner/common/template_operations/update_data_template_yaml_with_environment_variables.py
jinja_render_environmant_variable.py => what do you think on this name ?

@RobertKrawitz, I think that we need to change the name of the file name to more meaning name: benchmark_runner/common/template_operations/update_data_template_yaml_with_environment_variables.py
jinja_render_environmant_variable.py => what do you think on this name ?

Perhaps simply "render_yaml_from_template.py"?


docs: optimize quick start doc

What type of PR is this?

docs
Check the PR title.

[x] This PR title match the format: <type>(optional scope): <description>
[x] The description of this PR title is user-oriented and clear enough for others to understand.

(Optional) Translate the PR title into Chinese.
(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:
zh(optional):
(Optional) Which issue(s) this PR fixes:

扩充子文件内容


Example page totally broken due to mixed content

Mixed Content: The page at 'https://thednp.github.io/bootstrap.native/' was loaded over HTTPS, but requested an insecure stylesheet 'http://maxcdn.bootstrapcdn.com/bootstrap/3.3.5/css/bootstrap.min.css'. This request has been blocked; the content must be served over HTTPS.

I suggest switching to using https:// or just // in the URLs for external resources. Cheers!

Should be https://. @thednp I don't have time to fix this ATM, can you?

Thanks for reporting.


Had cascade delete flag to core metadata and fluent API

Maintain fluent API from old stack; enum flag in core metatdata to allow for additional states in the future.

:shipit:


Build failed on MacOS with C#

What version of gRPC and what language are you using?
Language: C#
Google.Protobuf: 3.8.0
Grpc.Tools: 1.21.0
What operating system (Linux, Windows,...) and version?
OSX 10.14.5
What runtime / compiler are you using (e.g. python version or version of gcc)
dotnet 3.0.100-preview5-011568
What did you do?
Built a proto file using the C# command
<Protobuf Include="file.proto" Link="targetFile.proto" GrpcServices="Server" />
Build the solution
What did you expect to see?
Build succeeds.
What did you see instead?
obj/Debug/netcoreapp3.0/ServiceGrpc.cs(42,12,42,29): error CS0234: The type or namespace name 'BindServiceMethodAttribute' does not exist in the namespace 'Grpc.Core' (are you missing an assembly reference?)
obj/Debug/netcoreapp3.0/ServiceGrpc.cs(42,12,42,29): error CS0234: The type or namespace name 'BindServiceMethod' does not exist in the namespace 'Grpc.Core' (are you missing an assembly reference?)

Anything else we should know about your project / environment?
The same code builds if Grpc.Tools nuget version is 1.20.0

Is Grpc.Core.Api nuget at version 1.21.0 too? https://github.com/grpc/grpc/blob/master/src/csharp/Grpc.Core.Api/BindServiceMethodAttribute.cs
was added in v1.21.0.

Looks like not. I use Grpc.AspNetCore.Server and its latest version(0.1.20-pre1) has only required	Grpc.Core.Api (>= 1.20.0-pre3). That looks like the culprit. Can that package be updated so the dependency still works as expected?

Does this help? https://github.com/grpc/grpc-dotnet/releases/tag/v0.1.21-pre1
Btw, it seems that this issue belongs to the grpc-dotnet repository. https://github.com/grpc/grpc-dotnet


Dagger Hilt support KSP

Dagger Hilt support KSP
Upgrade API

Please raise the PR against develop not master

Please raise the PR against develop not master

Okay 👍


Support to Google Analytics 4 gtag.js

Hi, I want to link my page to Google Analytics, but their latest updates do not seem to be compatible with the current options.
<!-- Google tag (gtag.js) -->
<script async src="https://www.googletagmanager.com/gtag/js?id=G-XXXXXXXXXXXC"></script>
<script>
  window.dataLayer = window.dataLayer || [];
  function gtag(){dataLayer.push(arguments);}
  gtag('js', new Date());

  gtag('config', 'G-XXXXXXXXXXC');
</script>

This has been addressed with pull request #1183


fix: avoid panic when use idc vpc

这个 PR 实现什么功能/修复什么问题:

[x] 功能、bugfix描述
[x] 冒烟测试
[ ] 单元测试编写

是否需要 backport 到之前的 release 分支:

release/3.6

/area region
/cc @zexi

Codecov Report

Merging #10452 (e723b24) into master (40644f7) will increase coverage by 1.97%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master   #10452       +/-   ##
===========================================
+ Coverage    5.90%    7.88%    +1.97%
===========================================
  Files         866      110      -756
  Lines      134935    13373   -121562
===========================================
- Hits         7974     1054     -6920
+ Misses     126256    12199   -114057
+ Partials      705      120      -585

Flag
Coverage Δ

aFlag
7.88% <ø> (+1.97%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/compute/regiondrivers/base.go

pkg/cloudcommon/db/domain.go

pkg/compute/regiondrivers/openstack.go

pkg/cloudcommon/db/orderby.go

pkg/keystone/models/user_group_memberships.go

pkg/multicloud/ucloud/host.go

pkg/compute/guestdrivers/apsara.go

pkg/keystone/models/groups.go

pkg/cloudcommon/db/modelbase.go

pkg/mcclient/options/k8s/cluster_resource.go

... and 745 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 40644f7...e723b24. Read the comment docs.


Tuner starts from validation even though num_sanity_val_steps is set to 0

Bug description
In PL 1.9.0:
tuner = pl.Trainer(auto_lr_find=True, accelerator="cuda", num_sanity_val_steps=0)
tuner.tune(model_module)

The model_module is Lightning module that has a field called lr.
Starts validation even though  num_sanity_val_steps=0.
Calling fit() works fine
Also works fine in 1.7.7
How to reproduce the bug
No response
Error messages and logs
# Error messages and logs here please

Environment

Current environment
#- Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow):
#- PyTorch Lightning Version (e.g., 1.5.0):
#- Lightning App Version (e.g., 0.5.2):
#- PyTorch Version (e.g., 1.10):
#- Python version (e.g., 3.9):
#- OS (e.g., Linux):
#- CUDA/cuDNN version:
#- GPU models and configuration:
#- How you installed Lightning(`conda`, `pip`, source):
#- Running environment of LightningApp (e.g. local, cloud):

More info
No response
cc @borda

Repro:
import os

import torch
from torch.utils.data import DataLoader, Dataset

from pytorch_lightning import LightningModule, Trainer

class RandomDataset(Dataset):
    def __init__(self, size, length):
        self.len = length
        self.data = torch.randn(length, size)

    def __getitem__(self, index):
        return self.data[index]

    def __len__(self):
        return self.len

class BoringModel(LightningModule):
    def __init__(self, lr: float = 0.1):
        super().__init__()
        self.lr = lr
        self.layer = torch.nn.Linear(32, 2)

    def forward(self, x):
        return self.layer(x)

    def training_step(self, batch, batch_idx):
        print("run training!", batch_idx)
        return self(batch).sum()

    def validation_step(self, batch, batch_idx):
        if self.trainer.sanity_checking:
            raise ValueError
        print("run validation!", batch_idx)

    def configure_optimizers(self):
        return torch.optim.SGD(self.layer.parameters(), lr=self.lr)

def run():
    train_data = DataLoader(RandomDataset(32, 64), batch_size=2)
    val_data = DataLoader(RandomDataset(32, 64), batch_size=2)

    model = BoringModel()
    trainer = Trainer(
        default_root_dir=os.getcwd(),
        limit_train_batches=1,
        limit_val_batches=1,
        num_sanity_val_steps=0,
        max_epochs=1,
        enable_model_summary=False,
        auto_lr_find=True
    )
    trainer.tune(model, train_dataloaders=train_data, val_dataloaders=val_data)

if __name__ == "__main__":
    run()

I've investigated it, the initial validation is not a sanity check step. In the training_epoch_loop, the initial training step is skipped and validation is run in on_advance_end.
    def advance(self, data_fetcher: AbstractDataFetcher) -> None:
        """Runs a single training batch.

        Raises:
            StopIteration: When the epoch is canceled by the user returning -1
        """
        if self.restarting and self._should_check_val_fx():
            # skip training and run validation in `on_advance_end`
            return

_should_check_val_fx return True is a bit unfortunate.
    is_val_check_batch = (current_iteration + 1) % self.trainer.val_check_batch == 0

Current_iteration is -1, and 0 mod any number is always 0, so it returns True.
The real bug is in the self.restarting part, the loop wasn't able to reset the restarting property to False.
https://github.com/Lightning-AI/lightning/blob/fc195b95405e9e2629466e5b28c6a9243209d596/src/pytorch_lightning/tuner/lr_finder.py#L477-L481
load_state_dict set restarting to True, the code trying to set it back to False. Updating the value of Fit loop restarting property will also update its children loop's restarting property, i.e. the training_epoch_loop
However, a bug in the fit_loop property setter method prevented it.
https://github.com/Lightning-AI/lightning/blob/fc195b95405e9e2629466e5b28c6a9243209d596/src/pytorch_lightning/loops/fit_loop.py#L117-L123
I think the code is trying to convery, is we cannot set restarting to True when both

Epoch is finished
It is not iteration based training (Otherwise, epoch is irrelevant).

There isn't any restriction on setting restarting to False.
It should be written as
restarting = restarting and (epoch_unfinished or self._iteration_based_training())

When we set restarting to False, it always return False.
When we set restarting to True, either epoch_unfinished or _iteration_based_training has to be True also.

What happened in the tuning bug is
restarting = False
epoch_unfinished = True
self._iteration_based_training() = True

False and True or True == True
False and (True or True) == False

So restarting remain True, self._should_check_val_fx() also return True, the training is skipped, and validation is run.
@awaelchli , happy to open a PR if you agreed on the diagnosis and solution.


fix(router): navigation unresponsiveness

closes https://github.com/NativeScript/nativescript-angular/issues/1484

test

Hi @NathanWalker, Thank you for the PR!
Caching the current frame, only if there is no already cached frame with the same name, will disrupt the logic that handles back navigation. It also breaks some of our tests that ensure back navigation inside modal view.
Can you share some info (sample project, playground or just steps) about the scenario that is fixed with your changes? We certainly have not covered all the scenarios with the latest router changes and your case could help a lot.
Note: We have some e2e test apps that could be helpful when contributing to nativescript-angular.
modal-navigation-ng app holds tests for modal views and your changes breaks some of them. You can run end-to-end tests locally by following these instructions.

Closing in favor of this.


Drop XML config support

I've never seen anyone using xml config for a rust project config. I'd like to drop support. We can deprecate the use to start.

Closed by #137


Updated text to help define the sMSA and gMSA acronyms

Updated text to define and more consistently use the sMSA and gMSA acronyms

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

Thanks, Jason! I've asked someone over here to do a quick review on your changes. Thanks - Liza


Ignore ctrlreg in deletion when computing required ctrlinsts

How to categorize this PR?

/area quality
/kind bug
/priority normal
What this PR does / why we need it:
Ignore ControllerRegistration in deletion when computing required ControllerInstalations.
To properly apply https://github.com/gardener/gardener-extension-os-suse-chost/pull/20 on existing landscapes, it is necessary to manually delete the existing controller installations. However, on large landscapes GCM creates new controller installations before the last old one to be deleted. With this change, once the deletion of the controller registration is triggered, no new installations will be made out of it.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

`ControllerInstallation`s are no longer created for `ControllerRegistration`s that are in deletion.

I have amend it to make it easier for testing.

/status author-action

An admission plugin has been implemented, in this way it is guaranteed that creation of a controller installation for controller registration in deletion will not succeed.
The change in the controller is re-worked again and it now silently skips the creation of controller installation, however it still can update existing installations. I tried to implement some unit test about this change, something similar to https://github.com/gardener/gardener/blob/f2f47cc860e20523d279945d721e8ce506ba086d/pkg/controllermanager/controller/controllerregistration/controllerregistration_seed_control_test.go#L772-L823 but I failed to understand how these tests works and whether it is possible such a test to be implemented.

An admission plugin has been implemented, in this way it is guaranteed that creation of a controller installation for controller registration in deletion will not succeed.

Hm, I can't follow - why is such an admission plugin needed? I mean, what's the reason this is so important that we have to forbid it API-wise?

but I failed to understand how these tests works and whether it is possible such a test to be implemented.

What exactly didn't you understand?  As it's not expected that it's going to be created you don't need EXPECT() calls for the k8sClient.
As a second test case, you could also add a ControllerRegistration with a deletionTimestamp which already has an existing ControllerInstallation. In this case you would expect that an Update() call is sent.

... and if a user creates a ControllerInstallation manually then why should we forbid this?

 Basically, applying the same approach by k8s community with namespaces - if it is in deletion, no new resources can be created in the namespace.

What exactly didn't you understand?

Exactly these .EXPECT(). calls are not clear to me what are they doing and how are they supposed to be used.
Looking through the generated code also does not help. I tried to find some documentation or guides or to look through https://github.com/kubernetes-sigs/controller-runtime/tree/master/pkg/client but failed here as well. Could you point me to some guides/documentation about this framework?
What I really tried to do was to call deployNeededInstallations with a ControllerRegistration with deletionTimestamp set and then list the ControllerInstallations with the provided fake client expecting empty list as result - this failed on multiple places, so I have decided that I am doing it completely wrong.

 Basically, applying the same approach by k8s community with namespaces - if it is in deletion, no new resources can be created in the namespace.

Not sure if this comparison is fair. Personally, I don't see the necessity to have this admission plugin (because, even if the ControllerRegistration is being deleted you might need to install it to some seed for whatever reason; i.e., I don't think we need to forbid it API-wise). Other opinions maybe? @gardener/gardener-maintainers

Could you point me to some guides/documentation about this framework?

Hm, I'm confused again, we are using gomock since years not only for Gardener but also heavily in the extensions. Do you mean something different?

Ok, I was thinking that this is coming from k8s.io, while it is actually from gomock - I will spend some time with this package.
Together with @ialidzhikov we have taken a look through the change and he helped me a lot to understand more about the tests, especially for mocks and fake clients, and to implement one (thanks for that, @ialidzhikov !).
In parallel, there is a validation that disallow ControllerRigistration to be updated once it is in deletion, thus we have decided to also not update existing installations because it should not possible due to this validation.

/invite @timuthy @danielfoehrKn @BeckerMax

/assign @ialidzhikov

The admission plugin has been removed.


rename ftd to ftd-sys

Being implemented as part of https://github.com/FifthTry/python.ftd.dev/pull/1


FOLO import/export API for sealed records

Folo export/import APIs and migration action added. I add quite a few stuff to test those code.

I once thought to ask OSE admin to clean the data/folo/* manually before new Indy is deployed. But it turns out not convenient (need sysops), especially on devel the PR build will be auto-deployed. So I update the migrate code to rename the data file to .bak, then import the zip. This can avoid manual operation during migration.


Fix the contributors list so that the names doesn't appear on two lines (at least on iPhone)

Great!


Added a connector to Visier Data Exports API

Tableau can't download directly the data exposed by the Visier Data Exports API, so this connector bridges the two.

I've updated the url to use heroku, and the API parameter to use v_2.1, also overwriting a duplicate entry (DCTM Connector by thakarevb) with mine. I also formatted the file using JSON standards.

Added


Callback function

if ("COMPLETE" == a.callbackEvent)
try {
var c = "return " + a.callbackFnc.replace(/;+$/, "") + ";"
, d = (new Function(c))();
That comma before d is a typo preventing the callback....

Hi @sehgalp
I can not reproduce your issue. Even the sample app uses this callback function with Event "On Complete".
You pasted the minified code (optimized with Google Closure Compiler). The original code looks like that:
if (vOptions.callbackEvent == 'COMPLETE') {
    try {
        var fncString = "return " + vOptions.callbackFnc.replace(/;+$/, '') + ";";
        var customFnc = new Function(fncString)();
        apexDropzone.doCallback(customFnc());
    } catch (err) {
        console.log(err);
    }
}

But this doesn´t matter, you only have to insert a valid Javascript function to this field, like:
function alertOnComplete() {
    alert("All files uploaded successfully!");
}

Does this work?
But you also have the possibility to use native APEX Dynamic Action Events instead, e.g Event "Dropzone upload completed [Dropzone]" from the event select list...


Get Craft CMS working with Node 14

Forge now install node 14 by default.  The node deps aren't working out of the box with it (at least on P&B).  This task is to support both 12 and 14.

@brokenhd when you were standing up Owlet, did you run into this issue?  Wondering if it persists.

@weotch No I had no issues with the node versions

Ok cool, gonna close this then.

Actually, going to re-open, I bet you didn't run into issues because one of my deploy steps was to install node 12.

Huh, can't recreate anymore as I work on #69, closing this again.


Implement Pager for auto-paging requests

Here's a first draft based on @RussTheAerialist's code in pagination_sample.py.
Currently it:

Takes an endpoint as its argument instead of a get function, this is because these calls only apply to get anyway, so let's keep it clean.
Takes an options object and seeds the filters/sorts with it, as well as the initial page size
Can be used as an iterator

I would like it to:

Not do everything in the __call__ method. I think we can move that to a helper or to __init__

We would make the first page call on instantiation, which populates some properties that could be useful (like __len__)

Since we know the total_avaliable we might be able to implement __len__. This gives users the ability to check the length to see if they really want to iterate over it or not.
I need to read more to see if I need to worry about __next__ or not.
Add tests

This script works (tested manually):
import tableauserverclient as TSC

auth_stuff = TSC.TableauAuth(username='USERNAME', password='PASSWORD')

server = TSC.Server("http://SERVER")
server.auth.sign_in(auth_stuff)

options = TSC.RequestOptions()
options.filter.add(TSC.Filter(TSC.RequestOptions.Field.Name,
                                         TSC.RequestOptions.Operator.Equals,
                                         "WORKBOOKNAME"))

# Call with a filter
wbs = TSC.server.Pager(server.workbooks, options)
print(len([i for i in wbs[)) # prints 1

# Get them all!
wbs = TSC.server.Pager(server.workbooks)
print(len([i for i in wbs])) # prints 1086 on my test server, that's all of em!
server.auth.sign_out()

Open Questions:

Where should it live? I'm not sure Server is the right place
Thoughts on implementing __len__?
Thoughts on implementing the primary logic somewhere other than __call__? It might just be me who thinks it's weird.

/cc @RussTheAerialist @LGraber

Oh yeah ... why did you put this in the server.py file and not its own file?

Alright! Addressed feedback (in person, and on here) tests added, and tested manually with this code:
import tableauserverclient as TSC

auth_stuff = TSC.TableauAuth(username='USER', password='password')

server = TSC.Server("http://server")

server.auth.sign_in(auth_stuff)

print(server.auth_token)

options = TSC.RequestOptions(pagenumber=2, pagesize=300)

wbs = TSC.Pager(server.workbooks, options)
print("GOT WBS")
print(len({i.id for i in wbs}))
server.auth.sign_out()

I am confused because the tests don't represent how we expect the users to use this class (ie you keep casting to list and doing len) and you don't have a sample of how to use it. Can you please add a sample


Mismatched views

The following blocks have mismatched views in the repo

gpio_defaults_block_007: Has no GDS view. DEF and LEF do not match the MAG view.
gpio_defaults_block_009: Has no GDS view. DEF and LEF do not match the MAG view.
gpio_defaults_block_087: Has no GDS view. DEF and LEF do not match the MAG view.
user_id_programming: GDS, DEF, and LEF do not match the MAG view.
chip_io: GDS, DEF, and LEF do not match the MAG view.

DEF is not really an issue for me, GDS and LEF views are needed for PnR.

Fixed by PR #44


[Enhancement] Display of images within lightbox (preload and use of low-res)

Explain in a few words which functionality or improvements you would like to see in Lychee.
When having a slow connection, the display of images within the lightbox is not optimal. This can for example happen on your mobile if 4G is not available or the connection is consumed by some other application.
Observed behavior:

Lightbox is opened, but image is missing
Image gets loaded and builds up slowly
(move the next image)
Image is missing, gets loaded and builds up slowly

I'd like to propose two enhancements to improve the UX:
(1) Use low-res to start with

Use the already cached low resolution version of the album view to start with
Once the actual image for the lightbox is loaded, change to this version

(2) Pre-load of images

Preload the neighboring (i.e. predecessor and successor) images of the album when a new image is display in the lightbox

I'd like to propose two enhancements to improve the UX:
(1) Use low-res to start with

Use the already cached low resolution version of the album view to start with
Once the actual image for the lightbox is loaded, change to this version

No problem with that.

(2) Pre-load of images

Preload the neighboring (i.e. predecessor and successor) images of the album when a new image is display in the lightbox

This is actually already implemented (but only one picture each side: previous and next).

(1) Use low-res to start with

Use the already cached low resolution version of the album view to start with
Once the actual image for the lightbox is loaded, change to this version
No problem with that.

I agree it's a good suggestion. Do you have an idea how to implement it though? Things get complicated with @2x images: we don't actually know which one is in the cache (for small) or which one the browser is planning to load (for medium). Is there a way to query the browser cache content from JS? A quick web search didn't return any obvious answers...

(2) Pre-load of images
This is actually already implemented (but only one picture each side: previous and next).

Actually, I believe we only do it for the next (photo.preloadNext). We could of course expand it to cover both previous and next but I'm not sure if that would make the matters better or worse (i.e., parallel loading of two images will only make things slower, although in the case of somebody browsing from one image to the next, the previous one is likely to be cached already).

(1) Use low-res to start with

Use the already cached low resolution version of the album view to start with
Once the actual image for the lightbox is loaded, change to this version
No problem with that.

I agree it's a good suggestion. Do you have an idea how to implement it though? Things get complicated with @2x images: we don't actually know which one is in the cache (for small) or which one the browser is planning to load (for medium). Is there a way to query the browser cache content from JS? A quick web search didn't return any obvious answers...

You're right, the implementation seems to be much more complicated than I thought in the first place. What about the following idea (don't know it if works in the end):
You can access the current source of an image using currentSrc (https://developer.mozilla.org/en-US/docs/Web/API/HTMLImageElement). We look at the thumbnail in the album view and have 3 cases:

currentSrc = small: -> use the small version
currentSrc = @2x -> use the @2x version
currentSrc = placeholder -> (for discussion) use the small version
What do you think?

(2) Pre-load of images
This is actually already implemented (but only one picture each side: previous and next).

Actually, I believe we only do it for the next (photo.preloadNext). We could of course expand it to cover both previous and next but I'm not sure if that would make the matters better or worse (i.e., parallel loading of two images will only make things slower, although in the case of somebody browsing from one image to the next, the previous one is likely to be cached already).

Does not work consistently
Did some more testing - it does not seem to work consistently.
Safari (on Mac): Next image is not preloaded. Looking at the traffic, it seems like Safari is loading the thumb version of the next image, but not the medium one. Without looking at the traffic, it seems the iPhone behaves on the same way.
Firefox: Is working
Loading previous image
This should only make the first image view slower. When iterating forwards, the previous image is already cached. When iterating backwards, preloading of previous image is helpful (next image is already cached)

What about the following idea (don't know it if works in the end):
You can access the current source of an image using currentSrc (https://developer.mozilla.org/en-US/docs/Web/API/HTMLImageElement).

Yes, we actually use currentSrc already, in the preload code. I was thinking about it and came up with pretty much the same idea as you have.

We look at the thumbnail in the album view and have 3 cases:

currentSrc = small: -> use the small version
currentSrc = @2x -> use the @2x version
currentSrc = placeholder -> (for discussion) use the small version

What do you think?

Yeah, exactly. When switching from the album view to the photo view (by clicking on a thumbnail) the thumbnails aren't actually removed from the screen; they just get covered up. So it should be quite simple to look up the thumbnail and initially display whatever it is displaying (since that's obviously cached). Regarding the 3rd scenario above, I would simply do nothing (i.e., fall back to what we currently do) as that's not a common case.

Does not work consistently
Did some more testing - it does not seem to work consistently.
Safari (on Mac): Next image is not preloaded. Looking at the traffic, it seems like Safari is loading the thumb version of the next image, but not the medium one. Without looking at the traffic, it seems the iPhone behaves on the same way.
Firefox: Is working

The thumb loading you're seeing is due to the background images behind the "prev" and "next" arrows of the currently displayed photo.
We are using standard HTML5 link prefetching. Indeed, according to https://caniuse.com/#feat=link-rel-prefetch you are correct that Safari doesn't support it. Shame. Given that pretty much every other browser supports it, I'm not exactly motivated to do anything about it. But if somebody submitted a clean PR we would probably consider it (preloading can be achieved in JS without HTML5 by creating an Image object and setting its source to the desired URL)...

Loading previous image
This should only make the first image view slower. When iterating forwards, the previous image is already cached. When iterating backwards, preloading of previous image is helpful (next image is already cached)

Yes, exactly.

Given that pretty much every other browser supports it, I'm not exactly motivated to do anything about it.

:+1:

This discussion reminded me of something that bothered me about Lychee from the beginning: the fact that the last picture wraps around to first when you click on the next arrow. I never liked that so I now fixed it (well, I made it configurable) and in the process I also started working on your list. :smiley:
@tmp-hallenser, can you test the branch nextprev-improvements of Lychee-front and see if prefetching now works on Safari?

@tmp-hallenser I completed the implementation of this request in #130. It would be great if you could test it and provide feedback, especially with respect to the prefetching on Safari, which is currently untested.

Gave it a brief test - Safari now preloads the medium size image -> that's good. Many thanks!
I did some further testing regarding page speed and realized, that calling photo:get is now not the most time consuming call. In my case, the network protocol shows roughly 1000ms waiting before the transfer starts. (this is of course not related to the network speed but rather to the large library combined with low power of server). Hence another idea to improve the speed:
Can we also preload the call of photo:get?
There is the function preload which might be useful: https://developer.mozilla.org/en-US/docs/Web/HTML/Preloading_content
Unfortunately, the feature is disabled for Firefox by default.
https://caniuse.com/#search=preload

I have what I think is a fairly low-end server (a cloud instance with 512 MB RAM) that's over 1000km from me and Photo::get takes under 200ms on average, which I find quite acceptable.
DevTools in Chrome include a snazzy "Server Timing" breakdown under Network (I had no idea such info is even sent...) -- can you see if there's anything interesting in there that would shed some light into why it's much slower for you?

Here, same, my server is 1000km away and I have more than 6000 pictures on it. I still get responses over Photo::get within 100ms.

My server (Docker on a raspberry) is in the same wifi and I've got approx. 30k photos.
The Server Timing is not shown for my installation and I found quite little resources on how to get it working. My timings for photo:get are:
Queueing: 1.34 ms
Stalled: 0.41 ms
Request sent: 0.22 ms
Waiting (TTFB): 1.19 s (<- that's the issue)
Content download: 1.52 ms
I've logged into the mysql server. A "SELECT * from photos WHERE ID=(some ID)" returns the result in 2ms -> that's also not the bottle neck.
I'll do some more testing if I find some more free time.

Did some further testing and the issue must be somewhere in my config. When using the provided Dockerfile based on Debian, I'm down to 180ms waiting compared to >1s (ngninx on an alpine base image)

I'm closing this issue since the originally requested functionality has been implemented.


How to configurate Postgresql, so that I can access Postgresql database on HUE?

dear all, I'v got a problem during connect my postgresql in Centos 7.
I have a few databases on<EMAIL_ADDRESS>5432, namely db1, db2, db3, db4.......dbN.
I am trying to build it on hue with db1, it works with the following code
[Notebook]
[[interpreters]]
[[[postgresql]]]
name = postgresql
interface=sqlalchemy
options='{"url": "postgresql://postgres:[me@<IP_ADDRESS>:5432/db1]"}'
[librdbms]
[[databases]]
[[[postgresql]]]
nice_name="My pgsql1"
name=db1
engine=postgresql
host=<IP_ADDRESS>
port=5432
user=me
password=pws
#options={"init_command":"SET NAMES 'utf8'"}
But How can I connect all the databses (db1, .....,dbN)?
Thanks and I am looking forward to get your feedback soon.
PS. It is sucessfully connecting to all databases of MySQL, however it doesn't work for Postgresql.

Same question, i've tried to config multiple postgresql databases in the
[librdbms]
[[databases]]
[[[postgresql]]]
[[[postgresql2]]]
[[[postgresql3]]]...

but it didn't work out. Neither did I config the "name=" as null or mark as "#name="
i'm using postgresql version 9.2.21 and Hue version 4.10.0
I would be grateful if you have any suggestions.
Thanks


mcuboot gets the wrong value of DT_FIXED_PARTITION_ID

project：zephyrproject/bootloader/mcuboot/samples/zephyr
branch：main
mcuboot commit：70bfbd21cdf5f6d1402bc8d0031e197222ed2ec0
zephyr commit：81595ed4db2081e227d6425acb25b6eec8191787
board & cpu：A microcontroller based on ARM Cortex-m0 designed in China
The dts file is modified as follows：
&flash0 {
	status = "okay";
	partitions {
		compatible = "fixed-partitions";
		#address-cells = <1>;
		#size-cells = <1>;
		status = "okay";
		boot_partition: partition@0 {
			label = "mcuboot";
			reg = <0x000000000 0x00010000>;
			read-only;
			status = "okay";
		};
		slot0_partition: partition@10000 {
			label = "image-0";
			reg = <0x000010000 0x000016400>;
			status = "okay";
		};
		slot1_partition: partition@26400 {
			label = "image-1";
			reg = <0x00026400 0x000016400>;
			status = "okay";
		};
		scratch_partition: partition@3c800 {
			label = "image-scratch";
			reg = <0x0003c800 0x2000>;
			status = "okay";
		};
		storage_partition: partition@3e800 {
			label = "storage";
			reg = <0x0003e800 0x00000800>;
			status = "okay";
		};
	};
};

The kconfig file is modified as follows：
CONFIG_FLASH=y
CONFIG_IMG_MANAGER=y
CONFIG_FLASH_PAGE_LAYOUT=y

Compile and burn and run
zephyrproject/bootloader/mcuboot/samples/zephyr$ make BOARD=wch_eth_ch579 full.bin
After running, the value of flash_map[i].fa_id output by the code in the file zephyrproject/zephyr/subsys/storage/flash_map/flash_map.c is respectively: 32, 43, 66, 43, 104 not 1，2，3，4，5
zephyrproject/zephyr/subsys/storage/flash_map/flash_map.c

static struct flash_area const *get_flash_area_from_id(int idx)
{
	printk("%s, %d, %d, %d\r\n", __func__, __LINE__, idx, flash_map_entries);
	for (int i = 0; i < flash_map_entries; i++) {
		if (flash_map[i].fa_id == idx) {
			return &flash_map[i];
		}else {
			printk("%s, %d, %d, %d\r\n", __func__, __LINE__, idx, flash_map[i].fa_id);
		}
	}

	return NULL;
}

zephyrproject/zephyr/subsys/storage/flash_map/flash_map_default.c

#define FLASH_AREA_FOO(part)						\
	{.fa_id = DT_FIXED_PARTITION_ID(part),				\
	 .fa_off = DT_REG_ADDR(part),					\
	 .fa_dev_name = DT_LABEL(DT_MTD_FROM_FIXED_PARTITION(part)),	\
	 .fa_size = DT_REG_SIZE(part),},

#define FOREACH_PARTITION(n) DT_FOREACH_CHILD(DT_DRV_INST(n), FLASH_AREA_FOO)

/* We iterate over all compatible 'fixed-partions' nodes and
 * use DT_FOREACH_CHILD to iterate over all the partitions for that
 * 'fixed-partions' node.  This way we build a global partition map
 */
const struct flash_area default_flash_map[] = {
	DT_INST_FOREACH_STATUS_OKAY(FOREACH_PARTITION)
};

const int flash_map_entries = ARRAY_SIZE(default_flash_map);
const struct flash_area *flash_map = default_flash_map;

The fa_id is taken from generated file devicetree_unfixed.h and depends on number of defined partitions and which partitions area enabled.
Do you happen to have DTS with lots of "fixed-partitions' compatible nodes, with many of them disabled?
The "issue" has been described here: https://github.com/zephyrproject-rtos/zephyr/issues/36517

The zephyr.dts partition definition in the build directory is like this：
		flash: flash-controller@40001800 {
			compatible = "wch,ch57x-flash-controller";
			label = "FLASH_CTRL";
			reg = < 0x40001800 0x200 >;
			#address-cells = < 0x1 >;
			#size-cells = < 0x1 >;
			flash0: flash@0 {
				compatible = "soc-nv-flash";
				label = "FLASH_WCH";
				reg = < 0x0 0x3f000 >;
				erase-block-size = < 0x200 >;
				write-block-size = < 0x4 >;
				status = "okay";
				partitions {
					compatible = "fixed-partitions";
					#address-cells = < 0x1 >;
					#size-cells = < 0x1 >;
					status = "okay";
					boot_partition: partition@0 {
						label = "mcuboot";
						reg = < 0x0 0x10000 >;
						read-only;
						status = "okay";
					};
					slot0_partition: partition@10000 {
						label = "image-0";
						reg = < 0x10000 0x16400 >;
						status = "okay";
					};
					slot1_partition: partition@26400 {
						label = "image-1";
						reg = < 0x26400 0x16400 >;
						status = "okay";
					};
					scratch_partition: partition@3c800 {
						label = "image-scratch";
						reg = < 0x3c800 0x2000 >;
						status = "okay";
					};
					storage_partition: partition@3e800 {
						label = "storage";
						reg = < 0x3e800 0x800 >;
						status = "okay";
					};
				};
			};
		};

@maweicoltsmart Can you share list of PARTITION_ID from your devicetree header file?
grep PARTITION_ID zephyr/include/generated/devicetree_unfixed.h

@maweicoltsmart Do you use FLASH_AREA_ID to obtain area ID, for example FLASH_AREA_ID(storage) ?

@de-nordic
The value of flash_map[i].fa_id comes from the initialization in the file zephyr/subsys/storage/flash_map/flash_map_default.c:
#define FLASH_AREA_FOO(part)						\
	{.fa_id = DT_FIXED_PARTITION_ID(part),				\
	 .fa_off = DT_REG_ADDR(part),					\
	 .fa_dev_name = DT_LABEL(DT_MTD_FROM_FIXED_PARTITION(part)),	\
	 .fa_size = DT_REG_SIZE(part),},

Yes, I know where it comes from. The DT_FIXED_PARTITION_ID that is used to initialize the fa_id uses DT macros to get to the ..._PARTITION_ID for each known partition, this is the only way.
The flash_map_entries may be different from the last ID because it counts number of entries that are in the array; the ID is generated for each partition but only enabled partitions (status=okay) are included. This means that if you have for example seven partitions and the first two are disabled, then the flash_map_entries will be five, and IDs does not have to be neither continuous nor ordered.
For the  DT_FIXED_PARITION_ID to get strange values that  do not match these from the devicetree_unfixed.h does not seem to be possible.

Unable to reproduce, please reopen if still an issue.


How to feed multiple objects from the same image

Hello, when I use one image containing many of the same objects (see below), I get an error message. This doesn't happen when I supply the training model with one object .xml with one image.

Can you post your xml file for the image?

Can you post your xml file for the image?

images
image1.jpg
/Users/ImageAI/custom/swimmers/train/images/image1.jpg

Unknown

910
512
3

0

swimmers
Unspecified
0
0

324
332
389
370

swimmers
Unspecified
0
0

555
352
603
429

swimmers
Unspecified
0
0

96
170
167
202

swimmers
Unspecified
0
0

129
77
193
113

swimmers
Unspecified
0
0

273
206
337
237

swimmers
Unspecified
0
0

648
165
718
199

swimmers
Unspecified
0
0

30
31
94
86

plz put a screenshot from the error here bc I  have an idea, but the output would help me much.


Tool for visualizing a model

Similar to the View Entity Data Model (Read-only) command in the EF Power Tools.

This would be a great feature. You once made that super difficult edmx with a nice designer so this should be easy ;) But please do not forget to support views better this time.

Future @bricelam, don't forget about the super early prototype you started in bricelam/EFDesigner. You didn't get very far, but it might be a good starting point. You at least got it to do this:

Future @bricelam, it would be sweet if we could do something that worked with Visual Studio Code, although at present we don't know how that would look like :smile:

Is there any progress on the Model Visualizer? I would love to utilize this with my students who I teach EF basics to.

@itanex No significant progress. LLBLGen Pro and DevArt Entity Developer are the two EFCore-compatible modelling tools I that know about. (This is not an endorsement. 😇)

@bricelam Thanks for getting back. I was not impressed with DevArt tools. I will check out LLBLGen

@itanex I had the opposite experience, I wasn't impressed with LLBLGen (not intuitive) but liked Entity Developer.  I guess we may have different requirements, but I thought that DevArt provides a similar experience to the old EDMX designer.
I'd recommend anybody that still needs database first take a look at the express edition which although limited to 10 tables does give you a flavor.  Ideally "future bricelam" will get to revisit his early prototype :)

FYI: I am getting quite close to a implementation of this in my "EF Core Power Tools" https://github.com/ErikEJ/SqlCeToolbox/issues/462

This is a good idea and we are fully supportive of community efforts in this area, but we don't intend at this time to implement or maintain this in the EF team.

It's a very bad decision to close this issue. This is a high priority feature on  your roadmap. If the costs are too high to ship the function with Visual Studio, it can be provided as an extension. I don't understand why the world's largest software company (Microsoft) cannot recruit enough staff for this project (EF Core). I hope that as many developers as possible will protest against it.

@MarcoLoetscher Have you tried my free Visual Studio extension (EF Core Power Tools)?
I would be happy to work with you on anything that you miss.

@ErikEJ We use EF 6. x in an extensive WPF business application. Unfortunately, there are still some features missing in EF Core for us, so we haven't used EF Core Power Tools yet. Thank you for your work on this extension. The "DbContext Model Diagram" looks very rudimentary for me in comparison to the "Entity Data Model (Read-only)" view in EF Power Tools. I hope that this diagram can be displayed in the future like the old one.

What are you missing - maybe I could add it?
Do you want to create a model from a diagram - this is not this "feature" and not supported/mentioned by the team (EDMX no longer exists) - you can look at the 2 third party tools listed here to help with that: https://docs.microsoft.com/en-us/ef/core/extensions/

No, I just want to display a read-only model of the DbContext. The Entity Data Model of the old EF Power Tools was much clearer than that of the new one.

@MarcoLoetscher IMHO your best bet would be to use Entity Developer as it's the closest replacement I could find for the old EDMX designer.  I'd encourage you to try the Express Edition (which is limited to 10 tables) to see if it meets your needs.

I agree with @MarcoLoetscher . It's a tool I cannot build models without and we need it for cross-platform work. I'm happy to use @ErikEJ tool when I'm on windows in vs2017. But that's not my most common scenario. Future @bricelam :

:)

Would SVG format work for you, @Julielerman ?

@ErikEJ The format is not the issue but where I can run it from. If you could take the work you did to make it happen in VS 2017 without depending on the class designer and have an extension for VS Code, that would be amazaballs!

Here's another project which can generate diagrams from existing code. It can also create code from diagrams, similar to "Model First" in EF Classic.
https://github.com/msawczyn/EFDesigner


Add Synchronization Authorization Information API

Synchronization API requirements
Synchronization API requires special authentication endpoint to be hit in order to actually run a connection, from a technical point, we could only check this before running the connection, however, as we allow uploading multiple files to mutiple iModels at the same time, ensuring that we are only redirecting when no other upload operations are going on require more logic than we intend to put in this demonstration at this time, so we will always check if the token is valid whenever the user connects to the portal, and refresh if it is needed.
Different behavior can be implemented if we need to, but let start with the simplest one.

We will not go with the popup solution as we expect changes in the Synchronization portal, so this solution will be enough for now.


Track syphilis transmission sources

Track syphilis transmitters

Might replace this later with an infection-log based approach, but for the moment this works, and new transmissions match new acquisitions


Enable RTL input in admin

https://uktrade.atlassian.net/browse/CMS-18

Codecov Report

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

@@           Coverage Diff           @@
##           master      #84   +/-   ##
=======================================
  Coverage   92.11%   92.11%
=======================================
  Files          20       20
  Lines         469      469
=======================================
  Hits          432      432
  Misses         37       37

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 b67fcc6...9a35cca. Read the comment docs.


Tests failing because ENOENT ./test_files/apachepoi_SimpleWithComments.xls

I've cloned master and installed the NPM packages. When I run make test I get the following failure:
  4705 passing (1m)
  5 pending
  1 failing

  1) corner cases CFB:
     Error: ENOENT: no such file or directory, open './test_files/apachepoi_SimpleWithComments.xls'
      at Error (native)
      at Object.fs.openSync (fs.js:641:18)
      at Object.fs.readFileSync (fs.js:509:33)
      at readFileSync (xlsx.js:1306:18)
      at Object.readSync [as read] (xlsx.js:1311:23)
      at Context.<anonymous> (test.js:1782:19)

Makefile:95: recipe for target 'test' failed
make: *** [test] Error 1

I'm not sure how to diagnose this issue: the file isn't there, but I can't figure out how it's supposed to have gotten there either.

@wolfgang42 that test is a holdover from how XLS used to be structured.  It can be safely removed.  Give me one minute, we'll remove that along with a few other test changes.
For testing: on OSX/Linux you need to run make init to pull in the test files.  In windows you need to download the last release from https://github.com/SheetJS/test_files/releases ,

@SheetJSDev Where does make init come in? It looks like it's trying to use SVN to get test data from somewhere (which doesn't work on my system because I don't have Subversion installed), but I ran npm run pretest and after your update make test is working fine.

@wolfgang42 don't worry about it :) make init pulls a bunch of external test files for a larger test that we run before cutting releases.  If the base tests work with npm run pretest we'll update the docs

i see this issue just with basic .readFile function usage


[DP0] save eval_dataset for subsequent calls

fixes #1107
credit: @Sanster

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Added Cheetsheet's

Have you read the Contributing Guidelines?

[✓ ] YES
[ ] NO

Project URL

https://github.com/thefool76/The-Complete-FAANG-Preparation.git

What is the current behavior?

Issue Number: #149 Added Cheat-sheet by code with harry

Category

[ ] 𝑨𝑺𝑲 𝒀𝒐𝒖𝒓 𝑫𝒐𝒖𝒃𝒕𝒔
[ ] 𝑴𝒊𝒔𝒄𝒆𝒍𝒍𝒂𝒏𝒆𝒐𝒖𝒔 𝑺𝒕𝒖𝒇𝒇.
[ ] 𝑫𝑺𝑨
[ ] 𝑪𝒐𝒎𝒑𝒆𝒕𝒊𝒕𝒊𝒗𝒆 𝑷𝒓𝒐𝒈𝒓𝒂𝒎𝒎𝒊𝒏𝒈
[ ] 𝑻𝒆𝒄𝒉𝒏𝒊𝒄𝒂𝒍 𝑺𝒖𝒃𝒋𝒆𝒄𝒕
[ ] 𝑳𝒐𝒘 𝑳𝒆𝒗𝒆𝒍 𝑫𝒆𝒔𝒊𝒈𝒏
[ ] 𝑷𝒓𝒐𝒋𝒆𝒄𝒕𝒔
[✔ ] 𝑰𝒎𝒑𝒐𝒓𝒕𝒂𝒏𝒕 𝑩𝒐𝒐𝒌𝒔 𝒂𝒏𝒅 𝑹𝒆𝒔𝒐𝒖𝒓𝒄𝒆𝒔
[ ] 𝑩𝒆𝒉𝒂𝒗𝒊𝒐𝒖𝒓𝒂𝒍 𝑰𝒏𝒕𝒆𝒓𝒗𝒊𝒆𝒘 𝑸𝒖𝒆𝒔𝒕𝒊𝒐𝒏𝒔

Changes

Description of changes
I have added Cheat-sheet of various languages and linux's command its by codewithharry all credit goes to him i just added his resourse's

Screenshots

(Add Runnig Work Demo Image)

Checklist

[ ✔] I've read the Contributing Guidelines.
[✔ ] I've checked the issue list before deciding what to submit.
[✔ ] I've follow Tree of Index.
[ ✔] I've add work in proper folder
[ ✔] I've not delete/modify previous codes.

thanks you! i made my first commit to open source

Add file in Important Books & Resources with new file Cheat Sheet
Not outside, change path

ok sure

Accepted

Thank you !


Feature/mat 1168

MAT-1168 Address missing scenarios in mongoid model generation
Fixed following things:

Original typeName for each primitivetype
Making all model types available in their own generator
Mongoid export for all models(file containing all model export)
Remove primitive type extensions for mongoid
Documentation for each mongoid model- this to address the ruby lints

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- src/utils.ts  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Complexity increasing per file
==============================
- src/utils.ts  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Coverage increased per file
===========================
+ src/templates/rubymongoid/classTemplate.ts  100
+ src/generators/MongoidTypeGenerator.ts  96
+ src/templates/rubymongoid/complexMemberTemplate.ts  100
+ src/templates/rubymongoid/registerPartials.ts  100
+ src/FileWriter.ts  87

Complexity increasing per file
==============================
- src/utils.ts  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- src/utils.ts  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- src/utils.ts  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Coverage increased per file
===========================
+ src/templates/rubymongoid/complexMemberTemplate.ts  100
+ src/templates/rubymongoid/registerPartials.ts  100
+ src/generators/MongoidTypeGenerator.ts  96
+ src/templates/rubymongoid/classTemplate.ts  100
+ src/FileWriter.ts  87

Complexity increasing per file
==============================
- src/utils.ts  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:
Complexity increasing per file
==============================
- src/utils.ts  2

See the complete overview on Codacy

@AndrewBird81  why is this complaining, any idea? I just deleted one line

 Here is an overview of what got changed by this pull request:

Coverage increased per file
===========================
+ src/templates/rubymongoid/classTemplate.ts  100
+ src/generators/MongoidTypeGenerator.ts  96
+ src/templates/rubymongoid/registerPartials.ts  100
+ src/FileWriter.ts  87
+ src/templates/rubymongoid/complexMemberTemplate.ts  100

Complexity increasing per file
==============================
- src/utils.ts  2

See the complete overview on Codacy


Update logo.

UPD:
We already found a guy who help us. His name Payton Burdette and he is cool designer also he will work on our logo.
Great examples:

@beardedpayton Hi! Here we can control our process.

Hey, if you need help, let me know.

@tobiasmarx Email me, please<EMAIL_ADDRESS>


⚠ Account Settings API has degraded performance

In dfdced2, Account Settings API (https://accountsettings.roblox.com) experienced degraded performance:

HTTP code: 503
Response time: 3613 ms

Resolved: Account Settings API performance has improved in e9c5e9e after 17 minutes.


Language selection in schema-building tool when using custom Wikibase extension

I am not able to select the language for label/ description/alias while setting up a schema for new items to be added to a custom Wikibase. It is working for one custom Wikibase I am using, but not for a second one.
To Reproduce
Steps to reproduce the behavior:

First, connect to selected custom Wikibase via reconciliation service and manifest.
Then, add some data to a new project and attempt building a schema.
Finally, add new items and attempt to add values for description / label etc. for new items to be created in the custom Wikibase.

Current Results
When attempting to add label/ description/alias the language selection doesn't work (shows no available matches) and therefore the schema is broken and workflow cannot continue through preview and upload. This issue happens only on this manifest: https://git.vvvvvvaria.org/jules/DAAP
Expected Behavior
A language dropdown should pop up and a language from the list should be possible to be selected. This is what happens on e.g. this working manifest: https://raw.githubusercontent.com/OpenRefine/wikibase-manifests/master/openrefine-wikibase-test-manifest.json
Screenshots

Versions

Operating System: MacOS Catalina 10.15.6
Browser Version: Google Chrome Version 84.0.4147.135 (Official Build) (64-bit)
JRE or JDK Version: JRE 1.8.0_261
OpenRefine: OpenRefine master branch as of 11.09.2020

@lozanaross Thanks again for using & testing the arbitrary Wikibase instances support. I'll look into this issue tomorrow.

OpenRefine uses https://${wikibase url}/w/api.php?action=languagesearch&search=${the string to search}&format=json to search language codes with the given string.
For Wikidata, the api can be found at https://www.wikidata.org/w/api.php?action=help&modules=languagesearch.
But for daap, by visiting https://daap.bannerrepeater.org/w/api.php?action=help&modules=languagesearch, we can see that the 'languagesearch' action is not provided, that's why the language list above is empty.
It is worth mentioning that https://wikibase-registry.wmflabs.org/ doesn't support this action, either: https://wikibase-registry.wmflabs.org/w/api.php?action=help&modules=languagesearch. But Wikidata, https://artbase.rhizome.org and https://or-wikibase-test.wiki.opencura.com do support this action.
@lozanaross Do you know why the 'languagesearch' action is not available on daap, did you disable it on purpose?

If I remember correctly this API action was introduced relatively recently, so it could be down to a difference between the versions of Wikibase deployed on these instances. That being said at a first glance I cannot find any difference between these two:

https://artbase.rhizome.org/w/api.php?action=query&meta=siteinfo&siprop=extensions
https://daap.bannerrepeater.org/w/api.php?action=query&meta=siteinfo&siprop=extensions

Thank you both for your thoughts.
I have to wait to get definite answer from the DAAP developer, but as far as I remember we did disable a global language setting because it was arbitrarily defaulting users to different versions of English which was causing issues with consistency and search. (To see what I mean by global – check screenshots below from DAAP vs Wikidata respectively).

However we didn't disable the option to pick language during item creation or when assigning "monolingual" string values to properties, so there should be a full list of languages still somewhere in the API. See screenshot from DAAP:

Once again I have no idea if the API is the problem, as Antonin says, maybe not. But would it be an idea for OpenRefine to search language codes in a different place? Because I assume we may not be the only ones who may want to disable the global language setting (e.g. Wikibase Registry have done it too).

I cannot figure out what MediaWiki action API was called when rendering the select list above, maybe that API is what we need. @wetneb Do you know what that API is?

It looks like this is not calling any API from the frontend directly, but instead doing the auto-complete locally (with the entire list of languages already loaded).
That could be an approach we fall back on. We could fetch all the languages available on the instance with:
https://www.mediawiki.org/w/api.php?action=query&format=json&meta=languageinfo&liprop=name&uselang=fr
(the uselang=fr lets us offer translated language names).
We would then have to have our own "search" system to return candidates given a user input.
I do not know if this is something worth doing on our side or if we should declare that we require language search on the target Wikibase instance… What do you think?

@wetneb I think it makes sense for Wikibase operators to be required to have language search (even if they are only serving 1 language that gets returned).

OK, we will try to see if it works if we switch on the language search again, although we'll have to see if possible to actually serve just one language. Hopefully that's possible and will fix this issue. Will send an update here as soon as tested and have some results to report.

Hello, an update here – once the global language search for the Wiki was re-enabled again, I don't get this issue anymore. Over the next few days, we'll try to see if it's possible to leave search enabled, yet limit the language we serve to just one (english). Not sure if this issue is worth keeping open for now, I can report back again once we have this sorted, just in case someone else might come up against the same thing...

Hello, an update here – once the global language search for the Wiki was re-enabled again, I don't get this issue anymore.

Glad to hear that and thanks for the testing!


Need open-rpc schema-action to sync with release version (when merged)

Summary
To keep up-to-date with released versions, the schema needs to be updated alongside the release-bot merge. We can probably run a job as part of the release run that creates the bot PR.

Closed by #594


Error server

I've installed the docker image JDBC_FDW and I've run the setup postgres sql Script. It gives me "ERROR:  remote server returned an error". I've already copied the jar file in a directory :drivers/postgresql....jar

In the tests/jdbc folder there are two *.jar files are present. When you do repo checkout please make sure that you have git lfs installed locally. If not, you will checkout only hashes of the files.
$ git lfs ls-files
a221c4106b * tests/jdbc/mysql-connector-j-9.0.0.jar
a2644cbfba * tests/jdbc/postgresql-42.7.3.jar

$ ls -lh tests/jdbc/*.jar
-rw-r--r-- 1 user group 2,5M чер  9 04:15 tests/jdbc/mysql-connector-j-9.0.0.jar
-rw-rw-r-- 1 user group 1,1M бер 14 19:41 tests/jdbc/postgresql-42.7.3.jar

If everything is ok with that, then you just execute docker-compose -f tests/postgres_jdbc_compose.yml up -d from the repo root and you should end up with postgres_jdbc_fdw container up and running. You could start psql from within the container and check that 2 foreign servers have been created using jdbc_fdw extension. And for each of these servers there is a foreign table has been created as well.
$ docker exec -it postgres_jdbc_fdw psql postgres postgres
psql (16.3 (Debian 16.3-1.pgdg120+1))
Type "help" for help.

postgres=# \des
           List of foreign servers
    Name    |  Owner   | Foreign-data wrapper
------------+----------+----------------------
 rfam       | postgres | jdbc_fdw
 rnacentral | postgres | jdbc_fdw
(2 rows)

postgres=# \det
       List of foreign tables
 Schema |     Table     |   Server
--------+---------------+------------
 public | rfam_taxonomy | rfam
 public | rnacen_xref   | rnacentral
(2 rows)

These are publicibly available postgres and mysql readonly databases. Because they are available over public internet they have some throttling mechanism implemented. At the time of my tests, one of the queries works without problem, the other one returns an error.
This works
postgres=# select * from rfam_taxonomy limit 10;

But this returns mentioned by you error
postgres=# select * from rnacen_xref limit 10;
ERROR:  remote server returned an error

But if you will have a look onto the container logs, you will see
Exception in thread "Thread-2" org.postgresql.util.PSQLException: FATAL: too many connections for role "reader"
...
2024-07-29 22:11:32.635 UTC [137] ERROR:  remote server returned an error
2024-07-29 22:11:32.635 UTC [137] STATEMENT:  select * from rnacen_xref limit 10;

I.e. there is public access throttling is happening.
But you are right that jdbc_fdw very often returns this generic remote server returned an error message. Which is not too much useful. Current jdbc_fdw version is 0.4.0. I.e. it's in very raw state. And I was observing a few other situations, when it was returning such error. But in all cases, it's not related to this docker image. It's just this extension current code state.
This is one of the main reasons, why I didn't include it yet into the all-inclusive datero engine image.


Integration with Gitpod

The Problem
Setting up local dev env is time consuming and not everyone's pc meet minimum system requirements for setup.
The Solution
That's where Gitpod comes into picture and helps us in configuring project on cloud without need of installing anything on local pc.
It would be helpful for users of Swirl to easily setup dev env without any hustle(Without even entering single command to run).

Thank you! Reviewing now

Completed! Thank you @s-vamshi


Sponsored messages: Various questions

Hello,
Some questions re sponsored messages:

according to https://corefork.telegram.org/api/sponsored-messages we need to support sponsored messages fully by Jan 1st. 2022 - However I also found a date talking about Dec 15th - which is the correct one?
tdlib as per today in v 1.7.0 does not seem to support all functions for sponsored messages, at least I do not find code behind getSponsoredMessages, only the entry in the TL schema file. Is that correct?

BR Florian

The official documentation mandates Jan 1st 2022.
Yes, only the latest TDLib from master (1.7.9, currently) supports sponsored messages.

Will there be a 1.8.0 release before the deadline?

1.8.0 is likely to be released in 2021, but there is no way to guarantee this. If it will be released in hurry, it could be less stable than the current master version.

1.8.0 is good to have. Same worries about sponsored messages.

Are there some test channels around to validate that an updated app works correctly?

@Wunderfitz Yes. You can use https://t.me/SecretAdTestChannel for testing.

Tagged TDLib 1.8.0 version has been released. API for sponsored message can be found in the Change Log and the documentation.

@levlam as we are finally implementing this 1 year later :P - so is it expected that when opening a channel, we call getChatSponsoredMessage at the end of the message loading, and if its not 404 we add this message to the chat - should this be repeated periodically, or only when entering the chat?
And if it was read (viewMessages), will it 404 so that users do not see it again?

If its not 404, you add the message below the last message. You need to call the method only once per chat opening.
It is unspecified, how many times a sponsored message will be shown to the user.

Thanks and happy new year, all the best for 2023 for you and your team. You are doing amazing work, keep on!


ci(.github): removing dependabot.yml

Description
removes dependabot config for auto-updates because we don't get much out of it
re #614
Compatibility

OS
Implemented

iOS
N/A

Android
N/A

Windows
N/A

Checklist

[ ] I have tested this on a device/simulator for each compatible OS
[ ] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[ ] I updated the typings files (privateTypes.ts, types.ts)
[ ] I added a sample use of the API (example/App.js)

Same


Ocasionally getting No job was found with identifier...

Happens once in a blue moon. Logs zip is empty. If you have access to our project, here are a few of those.

We identified it as a service side issue.  We think we understand it and will try to fix in the current or next sprint rollout.

:+1:

@bryanmacfarlane we're still getting these, although rarely (once every ~3 days)

We are also getting this error on a regular basis:
No job was found with identifier 247.

we're investigating.  if you want to send me you account url etc... to me.  my email alias is bryanmac and that's at microsoft dot com


Add missing curly braces in type annotations

thanks for the catch @fredj LGTM

Thanks for the review

coveralls seems down, meging anyway