UT - Onboarding

The following items, generally in order, must be completed to successfully onboard a receiver to ReportStream:

[ ] Added to organization.yml
[ ] Added to organizations-local.yml
[ ] Added to staging settings table
[ ] Added to prod settings table
[ ] Added at least one receiver
[ ] Added OKTA group to download site
[ ] Added customers to OKTA group for download site
[ ] Is customer contact made?
What doc format are we supporting for the receiver (HLV, CSV, other)? (Add comment below)
What transport(s) are we supporting for the receiver (SFTP, other)? (Add comment below)
[ ]  Got credentials?
[ ] Tested ELR connection locally?
[ ] Tested ELR connection from Azure?
[ ] Has the receiver accessed the download site?
[ ] Has the receiver been sent at least one test file?
[ ] Has the receiver signed off on file formatting?
[ ] Are there labs sending data?
[ ] Has the receiver confirmed receipt of the data?
[ ] Have the files processed successfully?
[ ] Are we in production?
[ ] Update customerStatus flag in production settings to active

2022-09-30.  Sarah sent an email requesting information to connect them up with ReportStream.

Email sent to Sararh on 10/7/2022:
Sarah
I apologize for the delay in response. Jason went out on paternity leave almost immediately after the call and we haven't had time to debrief Rachelle about what was discussed. I can answer some of these questions but not all of them.
We do have an implimentation guide we can send. It's a little bit out of date but I'm currently working on updating it. If you're willing to wait a week or two I can send that out.
We don't require a specific name format
We do accept v2.5.1, in fact we prefer it!
Our usual protocol for accepting files is SFTP. When we begin onboarding we will request your static IP addresses in order to whitelist them and we will create the SFTP for you.  Our SFTP connections do not have a rotation or expiration.
Our system does accept batched messages and we are not aware of any limit to that. Our system functions better with batched reporting and every 2 hours will work great for us.
We are happy with any identifier in MSH-5 and -6. Our usual identifiers are:
MSH.5 = NEDSS
MSH.6 = usually UDOH but can be any indicator
We do require an identifier in MSH.4.1. The remainder of the MSH.4 segment is nice to have but not neccesary
We would like to receive the AOE questions
I'm unsure about the QST flag - I'll ask Jason when he returns.
Our onboarding form provides most of the required elements. The official information from UDHHS is located at https://epi.health.utah.gov/disease-reporting/. however
We don't need you to suppress the QST flag, as far as we know.
We do have a series of fields we strongly recommend but we will make do without them. Unless there is no patient demographic data or no test result we really don't throw away any chance for data. The fields we prefer to be included are in the implementation guide we can send out in about a week.
We don't require a specific value in MSH.21.1
We only want to receive OTC results if they're ready by an app/computer or by qualified personnell (i.e. nurse, lab tech). If they are home performed and read without an app we do not want to receive them. The OBX. field should be listed as "At-Home Test" for any OTC tests performed and read by an app.
I apologize again for the delay in this email and in the start of this process. Since our kickoff meeting we've had quite a few changes within our electronic data exchange team. I have accepted the position of EDX team lead and can be your point of contact. If you email here and don't get a response feel free to reach out directly to my email at<EMAIL_ADDRESS>Let us know what our next steps are so we can get things started!
Teri

2022-10-14: Emailed Teri for SFTP credential and gave IP for whitelist.
"===
Good After noon, Teri,
My Name is OTT SATHNGAM.  I am responsible for onboarding your organization with ReportStream.  I completed ReportStream configuration setting for your organization.
I ready to send you the sample hl7 file for validation.  Please provide me your SFTP information such as credential and path to your staging and production either by KeyBase
(my KeyBase Id: oslynn1) or by secured email.  Below is our ReportStream's IP address and my test system for white listing.
ReportStream IPs:  <IP_ADDRESS>/24 (i.e <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, … , <IP_ADDRESS>)
Test system IP:  <IP_ADDRESS>
Please notify me when I can start SFTP the sample hl7 for evaluation.
Very best regards,
-Ott Sathngam
CDC SWE

Any communication from UT on the whitelisting?

@brick-green no we have not. I can follow-up with them.

2022-10-31: Followed up with Teri Bill.
Teri,
How are you?  Would you please give me the status of your Setting Up your SFTP site?  Do you have the IP for us to whitelist it?
Very best regards,
-Ott Sathngam
CDC SWE

11/4/2022: BX email to Rachelle Bouton:

Hi Rachelle,
I received an auto-reply email that you were the correct person to contact while Teri is out of the office. Would you happen to have a status update on Utah's SFTP connection set-up progress?
Thanks,
Berni

11/4/2022: Teri auto-reply message to BX:

I will be out of the office until November 14, 2022. I will respond as quickly as I am able when I return.  For general questions please contact<EMAIL_ADDRESS>For any emergency questions please contact Rachelle Boulton at<EMAIL_ADDRESS>Thank you!
Teri Bills

11/4/2022: BX email to Teri Bills:

Hi Teri,

I wanted to quickly make a few introductions in this follow-up email to you post-kickoff call with the CDC PRIME project members, since we've not yet had the chance to meet. My name is Berni Xiong, and I'm the new delivery manager on ReportStream's Onboarding and Operations teams. I work closely with Brandon Canniff (copied in this email as well) who has recently joined our team (replacing Sarah White). Together, with the support of our amazing engineering team, we'll be able to assist you with any questions you have during your onboarding experience with ReportStream.
In reviewing the previous email threads, it sounds as though we're waiting to get the green light from your team that you've set up the SFTP connection, is that correct?
Our engineers are ready to assist when your team is, so please let us know if there's anything we can do to help. We look forwarding to continuing on with the onboarding process with your team!
Thanks,
Berni

11/4/2022: Waiting on Rachelle Boulton to hopefully provide a status update in Teri Bills' absence.

BX sent email to Teri and Rachelle on 11/21:

Hi Teri and Rachelle,
Brick is currently out on PTO this week, so I wanted to make sure I reached out in the event either of you had emailed him to follow up on continuing with the onboarding process.
I realize that it's a holiday week; so if you decide it's better to pick back up where we left off either next week or December, our team is happy to assist. Let us know what works best for you.
Thanks,
Berni

I just followed up with them.  And, I still haven't receive any answer, yet.

@oslynn requested a firewall to talk to their system. No response from CDC yet. Requested @jeremy-page for access

Update from UTAH:
Hello,
We are set up to receive through the sFTP. Please us know when file has been sent over.
Thank you,
Rocio
On Fri, Jan 13, 2023 at 1:34 PM Sathngam, Ott (CDC/DDPHSS/OD/HITSSU<EMAIL_ADDRESS>wrote:
Good after noon, Rocio,
Is UT SFTP ready to accept production test result?  If yes, I will activate ReportStream to route test result to your organization, this Monday.
Very respectfully,
-Ott Sathngam
CDC SWE

Emailed UTAH and advised them that they are LIVE.
From: Report Stream (CDC<EMAIL_ADDRESS>Sent: Thursday, January 19, 2023 3:33 PM
To: Utah DCP Informatics Program<EMAIL_ADDRESS>Rocio Ramos (DHHS<EMAIL_ADDRESS>Cc: Sathngam, Ott (CDC/DDPHSS/OD/HITSSU<EMAIL_ADDRESS>Reeves, Maurice (CDC/OD/OCS<EMAIL_ADDRESS>Green, Brick (CDC/DDPHSS/OD/HITSSU) (CTR<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: CDC’s ReportStream Kickoff - Feedback Requested
Hi Rocio,
I am happy to inform that you are now connected to ReportStream Production.  Please be sure to monitor your feed and let us know if you need anything further.
Best regards,
Sharon
The ReportStream Support Team
Contact us at<EMAIL_ADDRESS>Support hours:  8:00 am through 9:00 pm Eastern time
Monday-Friday, excluding US public holidays
From<EMAIL_ADDRESS><EMAIL_ADDRESS>on behalf of Utah DCP Informatics Program<EMAIL_ADDRESS>Sent: Wednesday, January 18, 2023 1:39 PM
To: Report Stream (CDC<EMAIL_ADDRESS>Cc: Sathngam, Ott (CDC/DDPHSS/OD/HITSSU<EMAIL_ADDRESS>Xiong, Bernadette (CDC/DDPHSS/OD/HITSSU) (CTR<EMAIL_ADDRESS>Reeves, Maurice (CDC/OD/OCS<EMAIL_ADDRESS>Green, Brick (CDC/DDPHSS/OD/HITSSU) (CTR<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Rocio Ramos (DHHS<EMAIL_ADDRESS>Subject: Re: CDC’s ReportStream Kickoff - Feedback Requested
Perfect. Thank you.
On Wed, Jan 18, 2023 at 11:00 AM Report Stream (CDC<EMAIL_ADDRESS>wrote:
Hi Rocio,
Thank you for your email.  I am confirming receipt.  Will reach out to the Engineer working on this and they will get back to you shortly.
Best regards,
Sharon
The ReportStream Support Team
Contact us at<EMAIL_ADDRESS>Support hours:  8:00 am through 9:00 pm Eastern time
Monday-Friday, excluding US public holidays
From<EMAIL_ADDRESS><EMAIL_ADDRESS>on behalf of Utah DCP Informatics Program<EMAIL_ADDRESS>Sent: Wednesday, January 18, 2023 12:14 PM
To: Sathngam, Ott (CDC/DDPHSS/OD/HITSSU<EMAIL_ADDRESS>Cc: Xiong, Bernadette (CDC/DDPHSS/OD/HITSSU) (CTR<EMAIL_ADDRESS>Report Stream (CDC<EMAIL_ADDRESS>Reeves, Maurice (CDC/OD/OCS<EMAIL_ADDRESS>Green, Brick (CDC/DDPHSS/OD/HITSSU) (CTR<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Rocio Ramos (DHHS<EMAIL_ADDRESS>Subject: Re: CDC’s ReportStream Kickoff - Feedback Requested
Hello,
We are set up to receive through the sFTP. Please us know when file has been sent over.
Thank you,
Rocio

Done.


Feature Request: Cloud Saves for Retroarch

I love your way to add ROMs, i also like the Cloudsave feature from Retroarch on Steam. If there a possibility to get them together?

We can not use Steam as a cloudsave for external apps. But we could devise a solution that is self hosted or synced among other chimeraos devices. How does it sound? Did you manage to make something like that work on your own?

I think emudeck has a solution for this. We could take a look at what they are doing.

They use rclone. We could do something like that...


--experimental_no_product_name_out_symlink

The flag was removed in version 5-0 according to https://blog.bazel.build/2022/01/19/bazel-5.0.html ("Removed flag --experimental_no_product_name_out_symlink; it is effectively always true."
As a result it complain if bazel 5.0 is installed on the system because common.mk uses this flag. I think we should use this conditionally depending on bazel version installed on the machine.

Per https://blog.bazel.build/2022/01/19/bazel-5.0.html

Bazel 5.0 is a major release and Bazel’s second LTS release. It contains new features and backwards incompatible changes from 4.x, the previous LTS release.

If there isn't a reason to use Bazel 5.0, I would stick to Bazel 4.x for now, since more issues may be occurred when building the app on other platforms with Bazel 5.0.

This is exactly what I did. Perhaps, we need to mention this on document for those who setup it up for the first time. I found if we run choco install bazel --version 4.2 then it install the specific version instead of the latest version

The tested environment was mentioned in the instruction for iOS but not for Windows. Perhaps can you create a PR and add it to the Windows instruction after a successful build?

Bazel is evolving fast. I usually use whatever bazel version used by TensorFlow. @shabanim  please add doc about required bazel version for the project.

the PR https://github.com/mlcommons/mobile_app_open/pull/248 is sent


Architecture nodes

WIP: Übersetzung der Dokumentation "Nodes"
WIP: German translation of documentation "Nodes"

/language de

This PR may need a rebase on the latest master as we've got the initial German translation merged back right now.

/lgtm
/approve


Run server doesn't work after creating a new project

Probably related to wrong pathing

Works now, but we should block user from running server until ember install is done.

For sure

Done.


Clean up learned_l2_port handling

I've tried to reproduce the failure when it's not there, and I have not
been able to -- even when there are no learned hosts it still seems to work
ok. I was going to leave it as a hard error until it shows up again or can
be reproduced, since I think the real problem might be some other
configuration issue (e.g. wrong Faucet version).
On Thu, Dec 19, 2019 at 10:26 AM Anurag Porripireddi <
<EMAIL_ADDRESS>wrote:

@anurag6 commented on this pull request.
In forch/faucet_state_collector.py
https://github.com/grafnu/forch/pull/138#discussion_r360003507:

@@ -171,17 +171,23 @@ def restore_states_from_metrics(self, metrics):
label = int(sample.labels.get(label_name, 0))
restore_method(self, current_time, switch, label, int(sample.value))
self._restore_dataplane_state_from_metrics(metrics)

   self._restore_l2_learn_state_from_samples(metrics['l2_learn'].samples)

   self._restore_l2_learn_state_from_samples(metrics['learned_l2_port'].samples)

Are we sure that 'learned_l2_port' would always be in metrics?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/grafnu/forch/pull/138?email_source=notifications&email_token=AAIEPDYGTD34ALKALM525Y3QZO4FZA5CNFSM4J5MWV62YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCP2PWLA#pullrequestreview-334822188,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIEPDYWIZC6OW2ERBNTNK3QZO4FZANCNFSM4J5MWV6Q
.


Upgrade test projects to .Net 6.0

Using supported .Net version
Improved performance (15-20%)
Fixing some .Net version discrepancies

✅ No public API change.

✅ No public API change.

Far too many nuget upgrades that PA client won't be able to absorb


TimeBucket does not allow for extra arguments allowed by timescaledb

The current implementation of TimeBucket only allows the use of the required arguments (bucket_width and ts).
It would be nice if the remaining arguments could be used as well with the package.
The available argument combinations are as follow:

time_bucket(bucket_width, ts)
time_bucket(bucket_width, ts, offset)
time_bucket(bucket_width, ts, origin)
time_bucket(bucket_width, ts, timezone, origin, offset)

Opened a PR for it: https://github.com/schlunsen/django-timescaledb/pull/44
Did not include support for timezone, because could not find a way to send a named parameter into the db function

what if make all args required? you still will be able to pass None when calling the function


Validate support for folders

Decided that folder support in the context of pico devices was low value and medium complexity to implement. Will not do.


Make 'noParse' option more flexible

Inspired by similar code in browserify. The goal is to align the treatment of the noParse option in browserify and module-deps.

+1

Really need this.  Currently unable to --noparse relative filepaths without it.

Hey @geigerzaehler (and @flintinatux), I really like the idea. A few things though:

I'm not sure where I've seen that code before but it looks very familiar. If it's a copy/paste, can you show me from where?
path.resolve should be from this.options.basedir not the cwd (though for the common case basedir is the cwd).
Can you add a test?

I'm not sure where I've seen that code before but it looks very familiar. If it's a copy/paste, can you show me from where?

Yes, I copied it from the browserify source as it should behave like browserify itself. I added the link to the PR description.

path.resolve should be from this.options.basedir not the cwd (though for the common case basedir is the cwd).

Yes, it probably should, although browserify behaves differently. Do you think this should be changed?

Can you add a test?

Can and will. Might take a couple of days until I get it done, though.

@geigerzaehler Thanks for the source link! I knew I'd seen it somewhere :stuck_out_tongue:

path.resolve should be from this.options.basedir not the cwd (though for the common case basedir is the cwd).

Yes, it probably should, although browserify behaves differently. Do you think this should be changed?

Yup. I'd never given this any thought, but it's kinda inconsistent there too. But it's easy to miss since most of the time, the basedir is the cwd. @jmm What do you think? We should fix this on both ends. Relative paths in noparse would be great. We need logic that behaves like b.require for these types of path resolutions.

Can and will. Might take a couple of days until I get it done, though.

Awesome!

There are a lot of these kinds of path matching issues and I would like to squash all of them. I think that if entry files are specified relative to the basedir, then the noParse files should be too.

We need logic that behaves like b.require for these types of path resolutions.

Yeah probably. Another issue with noParse is that it's sensitive to file extension. An entry file entry.js can currently be specified as ./entry, whereas that wouldn't work for a noParse file.
It looks like there's a lot of undocumented functionality here: the relative path resolution details, boolean value, function value, testing path segments against noParse values. That should be addressed before changing anything.
In any case, I dislike the idea of essentially the same code being duplicated in browserify and module-deps. Are they similar enough that they can be consolidated somehow? Maybe the row could be passed to browserify's global transform and just set noParse on it. But, module-deps should be able to noParse files when used independently of browserify, right? So maybe this functionality could live in module-deps or be extracted to a module that browserify and module-deps can both depend on.

@jmm I don't know what was going through my mind, but deep inside I knew noparse wasn't quite right, but didn't realize how much. Ignore what I said earlier. I went and looked at all my builds (even SC ones), and it turns out I've been using it wrong this entire time :disappointed:

It looks like there's a lot of undocumented functionality here: the relative path resolution details, boolean value, function value, testing path segments against noParse values. That should be addressed before changing anything.

Yup. Though some of that is already here https://github.com/substack/module-deps#var-d--mdepsopts.

I dislike the idea of essentially the same code being duplicated in browserify and module-deps. Are they similar enough that they can be consolidated somehow?

Yes! So what if we remove noparse on the row objects[^1] and noparse the module-deps option, and instead create a parseFilter option that works kinda like how postFilter works now. It would run after the transforms have been applied but before the deps are parsed (duh). That way module-deps doesn't have to know about what paths to parse or not - just like how it doesn't have to know about ignore or exclude. And in browserify, you could consolidate all that that path testing logic in a single place. Then noparse, ignore and exclude would be behave exactly the same.
Let me know if you like this. The module-deps side is easy - I can do it tomorrow. I could also help with the browserify side.
[^1]: you could still allow b.require('underscore', {noparse: true}); but handle it inside browserify - or we could keep it and just not run it through the parseFilter function. But that's just complicating things.

It's small but a parseFilter could even return false for json and we can remove this assumption.

Yup. Though some of that is already here https://github.com/substack/module-deps#var-d--mdepsopts.

Hmm, I just see:

opts.noParse - an array of absolute paths to not parse for dependencies.

I like this idea (as long as it makes sense for the semantics of those options):

Then noparse, ignore and exclude would be behave exactly the same.

So what if we remove...noparse the module-deps option

When using module-deps independently of browserify, does the current noparse API make sense in contrast to parseFilter(), or you'd be comfortable yanking it out and saying good riddance (with a new major version obviously)? You could keep it and either just overload it to be able to accept a function, or keep it and add opts.parseFilter, and update browserify to utilize that.

So what if we remove noparse on the row objects[^1]
[^1]: you could still allow b.require('underscore', {noparse: true}); but handle it inside browserify -

I'm not sure I like the idea of ditching that property. I'm leaning towards thinking I like the flexibility it offers.

or we could keep it and just not run it through the parseFilter function. But that's just complicating things.

It's already setup to work that way though, isn't it? Adding opts.parseFilter would just involve tweaking the implementation of parseDeps(), which already isn't run when row.noparse is truthy.

It's small but a parseFilter could even return false for json and we can remove this assumption.

I like that idea, except you'd need to keep that around in some form to accomodate using module-deps independently of browserify, right?

The discussion above is out of my scope and I don’t think moving forward with this code makes sense at this point. If you would like to merge this nevertheless, I’ll be happy to fix the remaining issues (in particular tests).

Sorry @geigerzaehler! Thank you for bringing this up though. I think the going the parseFilter route fits better with the existing patterns. The net effect should be the same. I have a few tweaks to make in https://github.com/substack/node-browserify/issues/1256 - hopefully I'll get to it this week

Sorry @geigerzaehler!

No problem. Glad I could help with pointing out this problem! Looking forward to the improved flexibility and functionality.


Override env variables when packaging the chart

Feature Request:
In most CI/CD pipelines, we update the image Tag. We use a step/task to replace the old tag with a new one. Then we create the package.
The idea would be to remove that intermediate step/task and override the tag when running the command package:
$ helm package --set image.tag=2.0
and output would be the package tgz with image tag 2.0.

duplicate request of https://github.com/helm/helm/issues/3141. Please make sure to search for open issues before opening a new one. Thanks!


Update PL translation (v1.10.2)

Thanks much.  This may sit for a few days, version 2.0 is imminent - it'll look the same, but the performance improvements should be pretty sweet.  I've been rescanning my 460 mod collection over and over for a few days, and it went from well over 2min to somewhere around 20seconds.


Bazel support: Update Imath to 3.1.8

This PR only affects the Bazel Build of OpenEXR:

Update of Imath to 3.1.8
Reformat Bazel files
Update Bazel version

Which branch is 3.1.8? releaseor RB-3.1?

RB-3.1
V3.1.8-rc2 is a tag currently at the head of RB-3.1

@cary-ilm See here: https://github.com/AcademySoftwareFoundation/openexr/pull/1437


Feature request: Optional "hover" on large screens

Brace yourself for a controversial feature request :D
I have customers who do like the clicky menu but to make them feel safe they would also like the menu to hover when on desktop screens (using a mouse).
I can easily update my CSS so that, in addition to the click functionality, the menu can be hovered over. However, if a menu was already clicked on and the dropdown is showing, when the user hovers, the open one doesn't close itself. This is especially problematic when dealing with mega menus because they usually span the width of navigations.
I'm not sure the best way to handle this, but I wish there was a way that if a user hovers over another top level clicky toggle in the menu, it would close any other menus that are open.

Thinking out loud: Perhaps an alternative would just be to have an option to disable the clicky functionality for certain breakpoints. That way I could have mobile and touch devices work with clicky, but also have CSS hovering for large screens.

@ajplopez Thanks for writing up this request and your previous feedback on the script. I really do appreciate it all.
I "braced myself" and thought over it a bit before responding. Conclusion: This is not a feature I'm going to add.
This script directly came out of my article "In Praise of the Unambiguous Click Menu", and that article is the best explanation for why I won't be adding this. Maybe that article can be helpful in educating your clients? (It has helped with some of mine.)
I simply don't think hover-triggered dropdowns are a good idea, and adding them to this script would introduce a ton of complexity that I do not want to support. It's headspinning, for instance, to think about how a sort of combination hover/click/keyboard menu should support ARIA and remain accessible.
Adding a bunch of complexity for something I don't believe in is not a road I'm willing to go down.
This project has an open source license, so you can absolutely take this and run with it to do something else. I'd happily even add a link to help people find it if this isn't what they're looking for. That said, I suspect you'd be better off building something different from scratch, rather than trying to shoehorn hover support onto this script that was built very explicitly without support for hovering.
Thanks again for the friendly request.

Thanks for your thoughtful response. I agree - it's not something that I really want to do but sometimes client just insist on doing things like this. I always try to educate them as much as possible and hopefully they'll change their minds eventually.
For what it's worth, I did get this working by using your method of "closing submenus with js" and it seems to work great.
document.addEventListener("DOMContentLoaded", () => {
  const myMenu = document.querySelector("ul.clicky-menu");
  const topLevelItems = document.querySelectorAll(
    "ul.clicky-menu > li.top-level",
  );

  topLevelItems.forEach((item) => {
    item.addEventListener("mouseenter", () => {
      if (window.innerWidth >= 1280) {
        myMenu.dispatchEvent(new Event("clickyMenusClose"));
      }
    });
  });
});

Thanks for sharing that code! I'm glad you were able to get that working. It's cool to see the new custom event being so useful across different use-cases!


update docs

please updata docs ,NodeJs is v8.9.3

Would you like to collate the changing sections that seems meaningful, which could be helpful to us to update the documents.

js const a = 1

const a = 1


Updated docs for FileCopyDetails

Added a note to the class header that using getFile() with filters are not supported. (See also https://discuss.gradle.org/t/combining-eachfile-filter-results-in-failure/14801/2)

@ysb33r Thank you for clarifying this finer point of how the filters work.
We need more people with an eye for this kind of detail,
~Pepper


chore: release v1.653.0

fix: use txid for tx link (#7484)
feat: turn portals flag on (#7471)
feat: temporary remove underlying assets for runepool (#7480)
feat: disable withdraw if runepool opportunity is not mature (#7472)
chore: prevent future myzod break parsing snapshot (#7474)
feat: implement runepool withdraw (#7449)
fix: prioritize portals market data for lp assets (#7451)
fix: make zapper akschually great again (#7470)

Smoke test
XHRs

Seeing 403s on Covalent

Savers Withdraw
https://viewblock.io/thorchain/tx/638e6b9eef283785083a4b47add009355df3c21f6c9ad46abbbf4d42e9e22c8c

Miner Fees are initially 0 while loading, we should add a proper loading state there

https://jam.dev/c/9060c57b-5290-49a1-ac5c-8347fed42887
Savers Deposits
https://jam.dev/c/7f7d414d-9d2e-4ddc-a9b5-19eed8fabe59

On the flip side, we do have a loading state here, although refresh on confirm click is no bueno and should probably be locked

Li.Fi Multi-Hop (Hyphen)
Fees estimation failing on insufficient funds for intrinsic transaction cost
https://jam.dev/c/257001b3-15d1-4e84-886b-adcf1a11a1e6
Also tested with AllBridge:
https://jam.dev/c/52c98e07-caf8-48ae-9085-bc88ac9e0faf
And CCTP:
https://jam.dev/c/e91b80f5-95fb-4fce-9ca4-095cd34c524b
This seems to be caused by wrong balances detection on Arb One

Same-chain swap (0x)
https://jam.dev/c/606740e2-219a-4a39-9055-376cdcd7d4c3


🚀 Feature: Add WhatsApp with Vonage for Deno

Appwrite Functions allow you to extend and customize your Appwrite server functionality by executing your custom code. 🤩 You can learn more at our official Appwrite Functions docs.
Your task is to implement the WhatsApp with Vonage function in Deno. You can look at the existing Appwrite Functions Templates in another coding language.
Your function should behave and be implemented similarly to existing WhatsApp with Vonage Node.js template.
Tasks summary:

[ ] Fork & clone Appwrite's templates repository
[ ] Create a branch feat-implement-whatsapp-with-vonage-deno
[ ] Write code for the function in folder deno/whatsapp-with-vonage/.
[ ] Test the function using Appwrite, provide a video.
[ ] Write a README.md explaining what the function does, how to use it, and customize it. Here is an example.
[ ] Submit a pull request to our templates repository

If you need any help, contact us on our Discord server.
Are you ready to work on this issue? 🤔 Let us know, and we will assign it to you 😊
Happy Appwriting!

Hey..I'd like to take this up

Hey..I'd like to work on this. Can you please assign this to me

Hi have assigned the issue to you! Thank you for showing interest in contributing to Appwrite! Happy Hacktoberfest 🎃
Notes:
Please update us with your progress every 3 days, so that we know that you are working on it.
Join us on Discord - https://appwrite.io/discord to chat about Hacktoberfest and Appwrite!

Hi , if it gets unassigned please assign me.


Added quicksort and playfair encryption algorithm

@Leonkoithara  please read the guidelines

@moogacs I have made the required changes, I hope you can merge it now....


Controlling Domestic Hot Water Tank through automation

Hello,
Automatons lost ability to turn on/off hot water tank after 2024.9 update. Manual controls works as before. Also floor heating can be controlled through automations as before.
When running the action, comes following error:
Error running action
Entity water_heater.domestic_hot_water_tank does not support this service.

just checked the action in developer tools
action: water_heater.set_operation_mode
target:
  entity_id: water_heater.domestic_hot_water_tank
data:
  operation_mode: "on"

It worked ok. How does your automation looks like and what you are trying to set?

Got it work by changing "then" and "else"
   then:
     - action: water_heater.set_operation_mode
       target:
         entity_id: water_heater.domestic_hot_water_tank
       metadata: {}
       data:
         operation_mode: "on"
   else:
     - action: water_heater.set_operation_mode
       target:
         entity_id: water_heater.domestic_hot_water_tank
       metadata: {}
       data:
         operation_mode: "off"

Don´t know what broke old automation, but this way works again. Thanks for help @tadasdanielius and for great Custom Component (one of the most important ones in my setup)!


C API function rav1e_status_to_str() returns a string the caller needs to free

The function should ideally be changed to return a const char* string (Like the new version API functions). If that's not possible, then the doxy should mention that the string needs to be freed, or it will leak.

It is not possible, the error messages are not static strings and contain exactly what went wrong, if you could send a patch for the documentation would be great :)

Closed by #1822.


Can't correctly read simple text

Basic Information
Using v5.3.0
I initially started with v5.1.0, which still couldn't correctly recognize them
Operating System
Ubuntu 22.04 Jammy
Current Behavior
Its either I am doing something wrong or tesseract can't correctly read some simple text using LSTM. Legacy mode isn't helpful as it intermittently misidentify some numbers. I can't seem to recognize the number 9 if its the starting character in a text/line, it is always falsely recognized as the number 0. I also can't seem to read the text 5.11x. I've been banging my on this issue for days now.
These images below can't be correctly recognized as described above:

Expected Behavior
To correctly recognize the text as is.
Suggested Fix
Is there something I ought to do to get the correct text? I really wanted to train my own custom model but that looks like overkill for these simple texts. At least the training wiki already discourages me from persuing further as it recommends training for special cases, which I doubt this is.

Please read the documentation:
>tesseract i4012_980x_preprocessed.png -
9.80x
>tesseract i4012_926x_preprocessed.png -
9.26X
>tesseract i4012_923x_preprocessed.png -
9.23x

Would be much helpful if you could write what exact preprocessing steps you performed.

Your post does not conform guidelines described in https://github.com/tesseract-ocr/tesseract/blob/main/CONTRIBUTING.md
I just did what is suggested in the documentation. It is your job to read it and to test suggestions.


let RemoteSlobrok do its own initial health check

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.
@havardpe please review

/sd/workspace/src/github.com/vespa-engine/vespa/slobrok/src/vespa/slobrok/server/remote_slobrok.cpp:336: Failed assertion: '_checkServerReq == nullptr'


Unclear rule of ED & provider count in pallet-balances

With #12951, the rule of how ED and provider is changed and this breaks some of our code. While we are updating our code to work with the new behaviour, we found the existing code are preventing us to do what we believe a valid use case that should work.
Our use case:
Acala EVM+ requires a storage deposit to deploy smart contracts. The fund is reserved on the contract account and refunded when the contract is removed. The initial storage deposit is relatively large and always higher than ED. As new bytes are written in the contract storages, additional storage deposit will be charged and reserved on the contract account. Those additional deposits can be low and less than ED. We were expecting the initial deposit provides provider ref count to allow the contract to be able to receive additional storage deposit that's below ED.
Compatibility solution (that's not working, but we expect it to work):
Reserve no longer provides provider ref count. That's fine, as we can manually increase the provider ref count in our EVM pallet before transfer the initial storage deposit and reduce it after the contract is removed.
https://github.com/AcalaNetwork/Acala/issues/2495
However, the current implementation does not allow such case.
https://github.com/paritytech/substrate/blob/569aae5341ea0c1d10426fa1ec13a36c0b64393b/frame/system/src/lib.rs#L1689
This check will prevent transfer of any amount less than ED, regardless of provider count.
Is this behaviour expected? A provider ref count cannot lift the requirement of minimal balance?
Can we customize this behaviour? How (other than fork the code)?
To my understanding, reserve/hold no longer provides ref count because they are slashable. However, many use cases of reserve/hold does not intent to slash the funds and it makes perfect sense in such case to allow those funds to provide a ref count.  The requirement of a single pallet (staking) shouldn't impact all the other pallets and other parachains.
And do we have documentations of the minimal balances rule (of pallet balances and pallet assets) and provider / consume count rule and how they work together?

And do we have documentations of the minimal balances rule (of pallet balances and pallet assets) and provider / consume count rule and how they work together?

This is documented in frame_system. Consumers cannot be created on an account without there being at least one provider. Providers cannot be removed from an account until there are zero consumers. A non-zero provider count or a non-zero sufficient count keep an account in existence.
Balances pallet gives a provider ref to accounts which contain funds. It disallows balances below the existential deposit. If a balance falls below, it is wiped out and there is no provider reference.

Can we customize this behaviour?

No.

We are explicitly adding a provider count in our evm pallet so am I correct that with an account with provider count of 1, it is valid to transfer balances less than ED to it? Because that's not the case currently.
This line does not check provider at all https://github.com/paritytech/substrate/blob/569aae5341ea0c1d10426fa1ec13a36c0b64393b/frame/system/src/lib.rs#L1689

I remember that this behavior was counterintuitive for me, too. I was expecting that a manual  increment of a provider would lift the ED requirement and pull it into existence. But you still need the ED and hence the provider of pallet-balances.

can I say this is a bug that should be fixed?

The current logic is well-defined and is as expected:

Balances pallet gives a provider ref to accounts which contain funds. It disallows balances below the existential deposit. If a balance falls below, it is wiped out and there is no provider reference.

For Balances pallet to allow free balances below the ED when providers are non-zero it would need to act as a consumer to be economically safe. It is not currently possible to do this since Balances pallet would need to switch between being a Consumer and a Provider depending on whether the balance is less than ED or not. When reducing the balance to somewhere between zero and the ED (not inclusive) it would need to determine whether other providers exist and adapt its behaviour between wiping out the account (if none do) or increment consumers (if there are some).
It would be possible to alter the logic to do this and I'd not be against including it. However, any such PR would obviously need a comprehensive review and audit before merge, release and upgrade. I'd expect the lead time to be no less than 2 months. As such I would seriously consider alternatives such as a path similar to that which the pallet-contracts team took.

I see. We would use different approach in our code to not be blocked by this.
I will leave this open as I still want to see this improved.

I think a better solution will be refactoring pallet-balances in such way to allow customization of the behaviour of providers & dusts.
For example, I would like to customize it in such a way that with provider, it is possible for account to old balances less than ED without increase a consumer. When the provider is dropped to zero, the dust amount will be dusted.

How exactly would it be dusted? There is no callback in frame_system::dec_provider on purpose due to the difficulty with weight benchmarking that such arbitrary logic brings.

I see. We would use different approach in our code to not be blocked by this.
I will leave this open as I still want to see this improved.

I'm not sure what the issue is; perhaps you want to change the name and description since I believe I've made the ED rule quite clear(?)

We already have an on account killed hook which should be enough.
Will make another issue.


Edit page link not working behind reverse proxy

I'm running Raneto behind a reverse proxy.
This mostly works fine in terms of rendering notes properly, except when you open a note (e.g. data.md), and select Actions -> Edit you get directed to https://mydomain/data/edit instead of https://mydomain/notes/data/edit because it seems to ignore the base_url property I have configured
I've had a look at the template for the Edit link and it relies on this current_url property (https://github.com/gilbitron/Raneto/blob/master/themes/default/templates/page.html#L59) in https://github.com/gilbitron/Raneto/blob/master/app/routes/wildcard.route.js#L105
If I update this to
current_url   : req.protocol + '://' + req.get('host') + config.base_url + req.originalUrl,

It fixes the problem, but I'm not sure if that's the right way to go about it seeing as people might put FQDNs in the base_url
Setup I'm running

latest version of https://github.com/linuxserver/docker-raneto/releases image
Caddy reverse proxy

Relevant caddy section:
	proxy /notes pi3:8922 {
	    transparent
	    without /notes
	}

raneto config.js
base_url: '/notes'

Latest version of Raneto from linuxserver.io works great for me with Traefik 1.7 as a reverse proxy.

@djhworld Thanks again for reporting.
Do you have any updates to share? Good, bad, etc?
I've moved this to the Roadmap so we can verify it.
Closing this issue (but you're welcome to re-open)


ISPN-7591, ISPN-7476 and small ISPN-7517 adjustment

All fixes in same PR since they're all related to Hibernate 2LC so I've been testing them all together.

https://issues.jboss.org/browse/ISPN-7591
https://issues.jboss.org/browse/ISPN-7476
Plus a little revert on ISPN-7517 to keep TestResourceTracker.setThreadTestName as public.

The ugly or-switch in ExternallyMarshallable is bloated enough - can't you convert that into static SortedMap with custom comparator

Nice to have but given that this is just an assert, I don't see an urgent need to fix this right now.

... and register org.hibernate.* packages in Hibernate's testsuite?

The filtering capabilities cannot be modified from outside Infinispan source code, and don't intend to add it as indicated in the JIRA.

@rvansa Added a new commit to switch the white classes to a list and then use anyMatch to find if a class is amongst those.

I got that you plan to get rid of that class soon anyway, and that's why you don't like to make that class public and add static method to alter that from Hibernate testsuite?
Btw., why don't you define Externalizer for StandardCacheEntryImpl and register it in LifecycleCallbacks?

I got that you plan to get rid of that class soon anyway, and that's why you don't like to make that class public and add static method to alter that from Hibernate testsuite?

Hmmm, yeah, that might be a good idea. Simple yet flexible. I'll give that a go.

Btw., why don't you define Externalizer for StandardCacheEntryImpl and register it in LifecycleCallbacks?

That's not as easy as it sounds. First, from Infinispan I can't get hold of all the instance variables. disassembledStateText is not externally available. Secondly, there's no public ctor to be able to take those variables and construct a StandardCacheEntryImpl instance. You could maybe hack something with reflection, but not really in the mood.

Added one more commit make the ExternallyMarshallable public and adding a method to append to the white list.

@galderz That class is not set in stone, I suppose - you can't make it dependent on Infinispan, but you can expose methods.
Anyway, I think that this PR is good as it is and can be integrated.

@galderz IMO org.hibernate classes shouldn't be on the blacklist at all... maybe if the blacklist was dynamic too, L2 tests could add them to the blacklist, but I really don't think it should be there by default.
And I'm not convinced about the need to support badly implemented equals() either :)

And I'm not convinced about the need to support badly implemented equals() either :)

It's not about supporting it, but about not blowing up with NPE something new with Infinispan 9. We didn't used to blow up like that before.

@galderz IMO org.hibernate classes shouldn't be on the blacklist at all... maybe if the blacklist was dynamic too, L2 tests could add them to the blacklist, but I really don't think it should be there by default.

I think it should since we have different parts within Infinispan itself that externalize Hibernate classes (e.g. query). So, Hibernate is a dependency just like JGroups and Arjuna are. I can't see why Hibernate should be treated differently.

@galderz That class is not set in stone, I suppose - you can't make it dependent on Infinispan, but you can expose methods.

In theory yeah, I could do that, but that would involve convincing Steve and that might not be easy since in his view Infinispan should be transparent in terms of the keys and values that are stored. I can see his point... but it's true that providing externalizers for it would make it more compact for sure and hence would probably give some small performance gain. Until I have some proof of how much gain this could have, I don't think it's worth the time to convince Steve.

I think it should since we have different parts within Infinispan itself that externalize Hibernate classes (e.g. query). So, Hibernate is a dependency just like JGroups and Arjuna are. I can't see why Hibernate should be treated differently.

The difference is we don't depend on all of Hibernate, only on Hibernate Search, and only in the query module, so the core shouldn't know about it. Besides, we really should be transparent in terms of the keys and values that are stored :)

The difference is we don't depend on all of Hibernate, only on Hibernate Search, and only in the query module, so the core shouldn't know about it.

Good point, I'll remove the hibernate ref.

Updated, I can squash some stuff when this is ready :)

Rebased and fixed conflict. Happy with this? I can squash commits.

@galderz I'd like to see a more descriptive name for IncorrectKey and/or a link to the JIRA in the javadoc, otherwise LGTM.

@danberindei I've renamed the key to FalseEqualsKey (and test methods) and sorted out the rebase.

@wburns Addressed your comment

LGTM. Any more comments @wburns @slaskawi ?

Integrated, thanks Galder!


BREAKING, Incorrect mobile required length

After the latest update the widget requires 9 digits for example  where a country needs 10 (Egypt) which how it was initially, i also want to remove the x/9 validator but to no avail.

Version  intl_phone_field: ^2.0.0

You can change the validator field.

I'm already trying to override it, as below. Could you please clarify how this can be done?
  validator: (value) {
                                    if (value.isEmpty || value.length < 7) {
                                       return 'Please enter a valid number';
                                     }
                                        return null;
                                      },

how did you fix this?

@remymumoh that was a while back, but i didn't fix it, i used another package instead. There are several alternatives you can find that are standard and reliable with their validations.


can't impl Drop on self referencing structs

#[ouroboros::self_referencing]
struct Foo {
    a: String,
    #[borrows(a)]
    b: &'this str,
}

impl Drop for Foo {
    fn drop(&mut self) {}
}

results in this error:
error[E0509]: cannot move out of type `Foo`, which implements the `Drop` trait
 --> src/main.rs:1:1
  |
1 | #[ouroboros::self_referencing]
  | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  | |
  | cannot move out of here
  | move occurs because `self.a` has type `AliasableBox<String>`, which does not implement the `Copy` trait
  | help: consider borrowing here: `&#[ouroboros::self_referencing]`
  |
  = note: this error originates in the attribute macro `ouroboros::self_referencing` (in Nightly builds, run with -Z macro-backtrace for more info)

I am not sure Ouroboros could/should provide a nice API for this.
As per the Destructors page in the Rustonomicon:
(Emphasis added by me)

Note that taking &mut self means that even if you could suppress recursive Drop, Rust will prevent you from e.g. moving fields out of self. For most types, this is totally fine.

For self-referencing types, it is not fine. This is the error you are encountering above.

But there is a simple workaround: Wrap the self-referencing struct in a 'normal' struct, and use the normal borrow_fieldname(), with_fieldname_mut() etc. functions in there as desired:
pub struct Foo(FooInner);

#[ouroboros::self_referencing]
struct FooInner {
    a: String,
    #[borrows(a)]
    b: &'this str,
}

impl Drop for Foo {
    fn drop(&mut self) {
        println!("Drop called with b: {:?}", self.0.borrow_b());
        // etc
    }
}

impl Foo {
    // Use whatever parameters are appropriate to construct the default FooInner here:
    pub fn new(...) -> Foo {
      let inner = FooInnerBuilder{...}.build();
      Foo(inner)
    }
}


Add lost samples back in read tracking

Due to adding some fault-tolerance in trimming steps, samples that have no reads passing are now missing in the final read tracking table.  We need to add those back if possible.

This is likely due to how we currently skip failed runs as they don't generate FASTQ; unless there is a way to pull in the missing read info later (or build in the assumption that missing samples were dropped in trimming/filtering) this may be difficult to work around.


[Feature Request] Adapting based on props

I think that we can support Adapting based on props by compiling it to style prop.
// input
const Color = styled('span')`
  color: ${(props) => props.color};
`;

// output
const Color = (props) => {
  return (
    <span className={...} style={{ color: props.color }}>
      {props.children}
    </span>
  );
}

I don't really see this as being all that useful, given how limited this would be, and the additional complexity involved. This would be much more easily accomplished with something like withProps from recompose.

I think ${(props) => props.color} is not really readable syntax. CSS should be declarative.
It is nice to have this feature. But not a high priority at least for me.

It is just a simple case. If we can read variables from props, it means that we can use context to pass theme variables.
const Button = styled('button')`
  background-color: ${(props) => props.theme.color};
`;

<ThemeProvider>
  <Button />
</ThemeProvider>

@benjycui theme properties on context is something that css-in-js libraries like emotion need to support because they don't allow for css preprocessor pipelines. Astroturf is intentionally taking a different approach, and deciding to lean on existing tooling. In 99% (in our experience) there isn't any additional value to storing theme variables in js. In most cases a theme.scss file of sass variables (or postcss values, etc) is sufficient and preferable to having to figure out how to convert the value from js to css.
In other words, we feel like variables in css are used the vast majority of the time only in css, and so it makes more sense to keep them in css, and occasionally convert to js instead of starting in a format that is the less common case.

I think that we can support Adapting based on props by compiling it to style prop.

It'll be very limited. For example, you can't use this interpolation inside pseudo-selector or media queries. This also brings specificity issues because it won't possible to override this inline style easily. IMO you're better off using inline style directly for such simple stuff.
There are some other approaches you can use, like CSS variables or currentColor, where it makes sense.


Errors logs print in green instead of red

Description
Errors logs print in green instead of red. It makes it harder for authors to find out the issues.  Could we change that?
See: https://github.com/operator-framework/community-operators/pull/4078/checks?check_run_id=2828622734#step:3:1319

c/c @mvalarh @J0zi @dmesser

low priority, but nice to have. Thank you:)

Closed as we are tracking this in the new repo: https://github.com/k8s-operatorhub/operatorhub.io/issues/4


feature: affirmatively label each Asahi created APFS volume

feature: affirmatively label each Asahi created APFS volume
this could be accomplished by touch /.asahiVolume at install time when the Volume is mounted.   This can also apply to the ESP.
Helpful for later uninstall or other housekeeping tasks.

PR forthcoming

osenum.py already has a way of identifying what is a stub and what isn't.


custom shortcut (option+c) to open maccy is not working

Before Submitting Your Bug Report

[X] I have verified that there isn't already an issue reporting the same bug to prevent duplication.
[X] I have seen the FAQ.

Maccy Version (see 'About' window)
2.0.0 (47)
macOS Version
15.0
Maccy Settings
{
    "KeyboardShortcuts_delete" = "{\\"carbonModifiers\\":2048,\\"carbonKeyCode\\":51}";
    "KeyboardShortcuts_pin" = "{\\"carbonModifiers\\":2048,\\"carbonKeyCode\\":35}";
    "KeyboardShortcuts_popup" = "{\\"carbonKeyCode\\":8,\\"carbonModifiers\\":2048}";
    "LaunchAtLogin__hasMigrated" = 1;
    "NSStatusItem Visible Item-0" = 0;
    "NSStatusItem Visible Item-1" = 0;
    "NSWindow Frame SUUpdateAlert" = "446 409 620 398 0 0 1512 944 ";
    "NSWindow Frame com.sindresorhus.Settings.FrameAutosaveName" = "501 366 510 473 0 0 1512 944 ";
    SUEnableAutomaticChecks = 1;
    SUHasLaunchedBefore = 1;
    SULastCheckTime = "2024-09-18 14:52:27 +0000";
    SUUpdateGroupIdentifier =<PHONE_NUMBER>;
    avoidTakingFocus = 1;
    enabledPasteboardTypes =     (
        "public.html",
        "public.file-url",
        "public.png",
        "public.utf8-plain-text",
        "public.tiff",
        "public.rtf"
    );
    historySize = 99;
    ignoredPasteboardTypes =     (
        "de.petermaurer.TransientPasteboardType",
        "com.agilebits.onepassword",
        "net.antelle.keeweb",
        "Pasteboard generator type",
        "com.typeit4me.clipping"
    );
    migrations =     {
        "2024-07-01-version-2" = 1;
    };
    pasteByDefault = 1;
    popupPosition = cursor;
    showFooter = 1;
    showInStatusBar = 0;
    showSearch = 1;
    showTitle = 0;
    windowPosition = "[0.37996031746031744,0.6408898305084746]";
    windowSize = "[487,503]";
}

Description
option with any letter if I select as a shortcut to open the maccy then it's not working. If I add cmd to these like cmd+option+{any letter} it works.
Steps to Reproduce

open preferences --> General
select the open and record the shortcut to option+c and try opening the maccy.

Paging @vijaymihiran    This is not a issue with Maccy; it's is a major bug in macOS Sequoia. There's a reddit thread about it here.
https://www.reddit.com/r/macapps/comments/1fjpiiw/psa_macos_15_breaks_optionkey_hotkeys/

I have the same issue, I guess we're gonna wait for a fix from apple


[feat] Report typed errors for easier integration

We should have a set of error types that the driver reports. More elaboration needed on the actual set and implementation.

@aldld FYI

Yes 👍  @andrefurlan-db
For starters, the three error messages that until recently resided on root in err.go file (i.e. not implemented; unsupported tx and query params) - I see that just yesterday they were hidden under internal 😞
I would appreciate the opposite approach: turn them straight into sentinel errors and keep them exported for users to check against.

I am not suggesting sentinel errors are the ultimate way to go (+1 for types), it just seems a more sensible and easily achievable alternative to these error messages, at least until these features get supported.

Would be nice to implement something similar like other go sql drivers do:
https://pkg.go.dev/github.com/lib/pq#Error
https://pkg.go.dev/github.com/go-sql-driver/mysql#MySQLError
https://pkg.go.dev/github.com/jackc/pgconn#PgError
https://pkg.go.dev/github.com/denisenkom/go-mssqldb#Error
https://pkg.go.dev/github.com/snowflakedb/gosnowflake#SnowflakeError
https://pkg.go.dev/github.com/vertica/vertica-sql-go#VError.SQLState


Vertex Array Object extension should be re-instantiated for new GL context

The VAO extension wrapper doesn't re-instantiate the extension when a new GL context is provided to VertexArrayObject.init(gl). This leads to undefined behavior and scrambled rendering / random GL buffer errors in console / page crash.

See #280 for background & examples.

In addition, VAOs should ideally be destroyed along with other GL resources (in VBOMesh).

See https://github.com/tangrams/tangram/compare/vao-reset

Merged in https://github.com/tangrams/tangram/commit/d47ed12971f96775291ba5e91d9f04406d04a5c0


eBPF performance tests should perform end-to-end tests

eBPF performance tests currently only focus on the platform and execution context.
It would make sense to add the ability to run more meaningful end-to-end performance tests that loaded eBPF programs + maps and measure performance in situ.

Proposal:
Build a library of cross-platform eBPF performance tests built on top of BPF_PROG_TEST_RUN and libbpf. This will permit running the same tests on Linux and Windows for cross platform performance comparisons.

Perf test for LRU and hash map
Write empty ebpf program and measure the invocation of eBPF program.

I would recommend updating bpftool to allow it to set CPU id for the bpftool prog run command. This will allow using bpftool for performance measurements without needing to write a new tool.


Movies and New Shows sensors not pulling any data

Hi I just installed your awesome integration but the sensors sensor.trakt_upcoming_movies and sensor.trakt_upcoming_new_shows are not pulling any data.
I can confirm however that sensor.trakt_upcoming_premieres, sensor.trakt_upcoming_shows and sensor.trakt_upcoming_dvd are all working.
Wasn't sure if I hit a limit on API calls or something.

Do you have upcoming movies and new shows in your trakt watchlist? You have to include them in your watchlist first since these will only show your specific ones and not everything that will be coming out

Ahh I misunderstood! Ok that works, thank you!


Add data for <input type="search">

Fixes #1011.
Testcase: http://jsfiddle.net/cvrebert/ye7Lcq9n/

I've always felt this one to be kinda pointless, as often enough people try to find ways to undo the browser-specific styling for search. But I guess it's good to know which browsers have that and which don't. Thanks!

I've always felt this one to be kinda pointless

Agreed, but it had many upvotes and was easy to test, so I figured, "eh, ok".


clean up pdclient code

This PR cleans up some duplicate code.

/run-all-tests

/run-all-tests

LGTM

/run-all-tests tikv=release-2.1 tidb=release-2.1 pd=release-2.1


Mobiledoc support

- Do you want to request a feature or report a bug?
This is a feature request!
- What is the current behavior?
Old-school content
Mobiledoc seems quite groundbreaking. The folks over at Ghost are implementing it as well into their new editor. I think it'd fit in nicely with the platform-agnostic approach of Netlify-CMS.
Link: https://github.com/bustlelabs/mobiledoc-kit

This is very interesting. At the least it's worth keeping open for discussion and seeing how interested the community is in making this happen.

It is! I believe Mobiledoc has the potential to be the de facto data format for content. Being able to store everything in an abstract way opens up all kinds of possibilities.

Here is the new Ghost editor, by the way: https://github.com/TryGhost/Ghost-Admin/tree/master/lib/gh-koenig

@maartenbusstra the CMS can currently handle JSON files and JSON frontmatter as well.

Closing as stale due to lack of activity and/or interest. Please comment if you feel this should be reopened.


Bug 1959546: Update to current master

As part of bug #1959546 and  1934180, it makes sense to merge current master branch.
Benefits:

Unit test coverage (most of the Files changed are XMLs for simulated vCenter that we use for unit tests).
Bugfixes (perhaps more than the two above).
As you can see below, the nr. of commits that we merge here is not that big.

Risks:

Newer  govc version 0.23 (another big chunk of Files changed)

WIP:
~* needs discussion with QA~

/bugzilla refresh

/bugzilla refresh

I tested this on a 4.7 cluster where I was using datastore cluster and it seems be working:
521 02:14:49.160183       1 datastore.go:56] CheckStorageClasses: dsc: datastore team-ssd-intel is part of intel-ds-cluster datastore cluster
I0521 02:14:49.160200       1 datastore.go:67] CheckStorageClasses checked 2 storage classes, 1 problems found
I0521 02:14:49.160207       1 operator.go:241] CheckStorageClasses failed: StorageClass dsc: datastore team-ssd-intel is part of intel-ds-cluster datastore cluster

/lgtm

[patch-manager] I evaluated this last week and skipped it due to the size and the fact that the bugs referenced only moved to MODIFIED on May 17th. Finally, we were constrained on capacity last week as well. We will re-evaluate when the merge window opens again later this week.

We probably want openshift/cluster-storage-operator#169 to go in first anyway, so we remain covered in one way or another, and that PR is still waiting on lgtm/approval.

@wking  if we merge https://github.com/openshift/cluster-storage-operator/pull/169 first then - it will try to alert of metric that does not exist. It may not matter in the end because both PRs needs to be picked.


It's just broken... Can't run brew upgrade brew install etc.

brew config output
HOMEBREW_VERSION: 3.5.4-91-g5400254-dirty
ORIGIN: https://github.com/Homebrew/brew
HEAD:<PHONE_NUMBER>b7e4bb68f45d2519f156e2d2529946
Last commit: 9 hours ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: bedc1492d26b3872167262459f16c87256d82926
Core tap last commit: 4 hours ago
Core tap branch: master
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_BROWSER: open-browser
HOMEBREW_CASK_OPTS: []
HOMEBREW_EDITOR: lvim
HOMEBREW_MAKE_JOBS: 8
Homebrew Ruby: 2.6.8 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: octa-core 64-bit arm_firestorm_icestorm
Clang: 13.1.6 build 1316
Git: 2.37.1 => /opt/homebrew/bin/git
Curl: 7.79.1 => /usr/bin/curl
macOS: 12.4-arm64
CLT: <IP_ADDRESS>.1.1651278267
Xcode: N/A
Rosetta 2: false

brew doctor output
Error: cannot load such file -- warning
/opt/homebrew/Library/Homebrew/warnings.rb:4:in `require'
/opt/homebrew/Library/Homebrew/warnings.rb:4:in `<top (required)>'
/opt/homebrew/Library/Homebrew/ignorable.rb:4:in `require'
/opt/homebrew/Library/Homebrew/ignorable.rb:4:in `<top (required)>'
/opt/homebrew/Library/Homebrew/formulary.rb:80:in `require'
/opt/homebrew/Library/Homebrew/formulary.rb:80:in `load_formula'
/opt/homebrew/Library/Homebrew/formulary.rb:122:in `load_formula_from_path'
/opt/homebrew/Library/Homebrew/formulary.rb:309:in `load_file'
/opt/homebrew/Library/Homebrew/formulary.rb:466:in `load_file'
/opt/homebrew/Library/Homebrew/formulary.rb:299:in `klass'
/opt/homebrew/Library/Homebrew/formulary.rb:294:in `get_formula'
/opt/homebrew/Library/Homebrew/formulary.rb:456:in `get_formula'
/opt/homebrew/Library/Homebrew/formulary.rb:540:in `factory'
/opt/homebrew/Library/Homebrew/formulary.rb:588:in `from_keg'
/opt/homebrew/Library/Homebrew/formulary.rb:561:in `from_rack'
/opt/homebrew/Library/Homebrew/formula.rb:1703:in `block in installed'
/opt/homebrew/Library/Homebrew/formula.rb:1702:in `each'
/opt/homebrew/Library/Homebrew/formula.rb:1702:in `flat_map'
/opt/homebrew/Library/Homebrew/formula.rb:1702:in `installed'
/opt/homebrew/Library/Homebrew/diagnostic.rb:700:in `check_deprecated_disabled'
/opt/homebrew/Library/Homebrew/cmd/doctor.rb:64:in `block in doctor'
/opt/homebrew/Library/Homebrew/cmd/doctor.rb:57:in `each'
/opt/homebrew/Library/Homebrew/cmd/doctor.rb:57:in `doctor'
/opt/homebrew/Library/Homebrew/brew.rb:93:in `<main>'

Verification

[X] I ran brew update and am still able to reproduce my issue.
[X] I have resolved all warnings from brew doctor and that did not fix my problem.

What were you trying to do (and why)?
Install a package
❯ brew install fnm

Error: cannot load such file -- warning
/opt/homebrew/Library/Homebrew/warnings.rb:4:in `require'
/opt/homebrew/Library/Homebrew/warnings.rb:4:in `<top (required)>'
/opt/homebrew/Library/Homebrew/ignorable.rb:4:in `require'
/opt/homebrew/Library/Homebrew/ignorable.rb:4:in `<top (required)>'
/opt/homebrew/Library/Homebrew/formulary.rb:80:in `require'
/opt/homebrew/Library/Homebrew/formulary.rb:80:in `load_formula'
/opt/homebrew/Library/Homebrew/formulary.rb:122:in `load_formula_from_path'
/opt/homebrew/Library/Homebrew/formulary.rb:309:in `load_file'
/opt/homebrew/Library/Homebrew/formulary.rb:299:in `klass'
/opt/homebrew/Library/Homebrew/formulary.rb:294:in `get_formula'
/opt/homebrew/Library/Homebrew/formulary.rb:540:in `factory'
/opt/homebrew/Library/Homebrew/cli/parser.rb:626:in `block in formulae'
/opt/homebrew/Library/Homebrew/cli/parser.rb:622:in `map'
/opt/homebrew/Library/Homebrew/cli/parser.rb:622:in `formulae'
/opt/homebrew/Library/Homebrew/cli/parser.rb:307:in `parse'
/opt/homebrew/Library/Homebrew/cmd/install.rb:141:in `install'
/opt/homebrew/Library/Homebrew/brew.rb:93:in `<main>'

What happened (include all command output)?
I just did a Control-C and that caused this issue. Even after the reboot, it stays the same.
What did you expect to happen?
I expect it to work...
Step-by-step reproduction instructions (by running brew commands)
IDK how to reproduce this...

Try doing
brew update-reset

but note this will destroy any local modifications you've made to your Homebrew installation.
Since this appears to be particular to your machine rather than a bug in Homebrew, I'm closing this. Please post in the Homebrew Discussions page if you're looking for more help to resolve this.


Powertoys Run will not detect unit conversions

Microsoft PowerToys version
0.67.1
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
Implicit unit conversions do not seem to work for me (i.e. without the direct activation command (default %%)).

✔ Expected Behavior
Unit conversions do not have to use the direct activation command:
❌ Actual Behavior
Unit conversion only happens when invoked via direct activation:

Other Software
Win10 21H2

Hi @martixy
Have you enabled the "include in global results" option for the unit converter plugin?

/needinfo

🤦♀


Gk corrections

Started tackling by backlog of corrections. Mostly link fixes, added github projects resource content.

👍


[$500] Desktop - Unread messages get marked as read without me visiting the chats

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Send the message from Account A (mobile) to Account B (Desktop app)
Check if message is Unread in Desktop app

Expected Result:
Unread messages displayed in LHN
Actual Result:
Unread messages get marked as read without me visiting the chats
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: n/a
Reproducible in production?: n/a
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
n/a
Expensify/Expensify Issue URL:
Issue reported by: @rafecolton
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1693523157400069
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0100b1d4dc36ebfcc0
Upwork Job ID:<PHONE_NUMBER>069742592
Last Price Increase: 2023-09-06
Automatic offers:

tienifr | Contributor | 26649040

Given that there are multiple reports, I'm going to go ahead and put this one out there.

cc @danieldoglas - as Rocio's theory is that this is coming from #vip-reliable-updates - do you agree?

@kbecciv @sophiepintoraetz @tienifr I wasn't able to reproduce it (sending messages from ios app to the desktop app and messages in LHN are not marked as read)... According to this thread on Slack I guess this occurs only in some special circumstances, am I right? What are these circumstances?

Just to inform: I'll be OOO on Thursday and Friday (7-8 September).

Proposal

Please re-state the problem that we are trying to solve in this issue.
Desktop: Unread message gets marked as read , without visiting the opening the message.
What is the root cause of that problem?
In the below , It is not checked  whether  optionItem  is unread or not ,It only checks  with mention.
https://github.com/Expensify/App/blob/57b76067cd3d7b59e84c4a387d11ad33e6a97d05/src/components/LHNOptionsList/OptionRowLHN.js#L105-L109
What changes do you think we should make in order to solve the problem?
We should update the line by adding optionItem.isUnread
 const shouldShowGreenDotIndicator =
        !hasBrickError &&
        (optionItem.isUnreadWithMention ||  optionItem.isUnread ||
            ReportUtils.isWaitingForIOUActionFromCurrentUser(optionItem) ||
            (optionItem.isTaskReport && optionItem.isTaskAssignee && !optionItem.isCompletedTaskReport && !optionItem.isArchivedRoom));

what alternative solutions did you explore?(Optional)
N/A

@tienifr - are you able to reproduce this?

Ok, I've finally reproduced the issue successfully! I didn't notice at first that it is marked as read after some short period of time.
Also reviewed proposals and the solution from @tienifr seem to be correct and I think we could proceed with it.
Solution proposed by @davewish is, I think incorrect - it also fixes the issue but it adds the green dot next to the unread message which shouldn't happen.
🎀 👀 🎀 C+ reviewed

please , I need clarification ,    my proposal is reviewed as incorrect   by @burczu  and   job assigned to me . then I will proceed with the job

yeah @cristipaval - I've recommended solution from @tienifr

@tienifr, you should be all set now! 🚀

Payouts due:
Issue Reporter: N/A
Contributor: $750 @tienifr
Contributor+: $750 @burczu - can request seeing as you have completed the BZ checklist, thank you!
Eligible for 50% #urgency bonus? Y

@sophiepintoraetz I'm not eligible for payment as I'm contractor from Callstack.

Oh thanks for confirming, then we are super done!


Adding support for multiple status values to fix apache issues

For OWASP CRS we need support for multiple status codes so that if Apache fails  and returns a 400 before CRS is hit we can support the check still on other webservers.

LGTM


Fix unstable MDLC Unit test  + MDLC free dataslot

fixes https://github.com/NLog/NLog/issues/1237

Current coverage is 74.38%

Merging #1239 into master will not affect coverage as of dbfd27b

@@            master   #1239   diff @@
======================================
  Files          266     266
  Stmts        15352   15355     +3
  Branches      1613    1612     -1
  Methods          0       0
======================================
+ Hit          11419   11422     +3
  Partial        380     380
  Missed        3553    3553

Review entire Coverage Diff as of dbfd27b

Uncovered Suggestions

+0.09% via ...gingConfiguration.cs#282...295
+0.09% via ...gingConfiguration.cs#248...261
+0.09% via ...argets/FileTarget.cs#1704...1716
See 7 more...

Powered by Codecov. Updated on successful CI builds.


Spam Domain Submission

Duplicate Domains

[X] Yes, I checked that the domain is not currently in the list
[X] I read the wiki explaining each different categories and examples before submitting

(Required) Spam Comment Direct Link
I have no idea how to get a direct link.
The following video only has 4 comments (including the spam comments):
https://www.youtube.com/watch?v=OZ9O921L2os
Please inform me as to how you can get a direct link
(Optional) Screenshot of comments using the domain

(Required) List of the spam domains

dhisex.uno
casual.centre.baby

Added

@s1072489 You can get a direct link by copying the link from the comment published time.


Can't download bilibili av2587092

[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'http://www.bilibili.com/video/av2587092/']
[debug] Encodings: locale cp950, fs mbcs, out cp950, pref cp950
[debug] youtube-dl version 2015.07.07
[debug] Python version 2.7.8 - Windows-XP-5.1.2600-SP3
[debug] exe versions: avconv v12_dev0-1569-g2cb34f8, ffmpeg N-73667-g99b0cad
[debug] Proxy map: {}
[BiliBili] 2587092: Downloading webpage
ERROR: Unable to extract cid; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type
  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "youtube_dl\YoutubeDL.pyo", line 654, in extract_info
  File "youtube_dl\extractor\common.pyo", line 273, in extract
  File "youtube_dl\extractor\bilibili.pyo", line 68, in _real_extract
  File "youtube_dl\extractor\common.pyo", line 555, in _search_regex
RegexNotFoundError: Unable to extract cid; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest
version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Can you watch it in browser?

@dstftw It's a China-restricted video.

@yan12125 I've already figured that out.

Sorry for not noticing that. I think this issue can be closed as --proxy works for me.

Well, let's confirm that OP is unable to play it in browser either and close then.

yes I CANNOT watch this in browser.
maybe the "error" message should point out that to users?

It will in the next version.


Add a simple CI

The CI system should:

Build the exporter, and run tests
Build in docker, just to test
Run a mixin linter
Maybe start a docker-compose and call /metrics just to test

https://github.com/thepalbi/ethereum-prometheus-exporter/pull/24


Make DCS API more consistent

While reviewing #91 I noticed that there is some inconsistency in the dcs module, which we might want to fix as a part of a future breaking change. SetPixelFormat uses a private tuple field and a new constructor, while other types with one parameter use a public tuple field and don't provide a constructor. IMO we should use a private fields and a constructor for all command types with parameters to be consistent.

I agree, but I'm quite busy atm. I can have a look later. Happy to review a PR though.
This should preferably be addressed before e-h 1.0 support so we can have it all in one breaking change.


Add "Go to Top" Button Component

Hello,
It would be incredibly helpful to have a "Go to Top" button integrated into the SQLPage. Given the extensive reading involved in my use case, such a feature would significantly enhance user experience by allowing users to quickly navigate back to the top of the page.
The button should be clearly visible but not intrusive. It could be similar to the example from the Tabler.io website, as shown in the attached screenshot.

Maybe we could add a "position" attribute to the button component that could be set to inline (the default) or "(top|bottom) (right|left)" ?
And for scrolling to the top, we would just make a button that targets #sqlpage_header.
Pull request welcome !

I’m not sure if a standard button is sufficient. The concept of a ‘go to the top’ button is more complex. It remains hidden even if you scroll down slightly, only appearing when it’s truly needed.


Reset communication will result into resetting everything

I want to implement the reset communication for my platform.
What I have done is to re-initialize the CANopen stack with CO_CANopenInit();.
The problem is that this will result in deleting all configuration of the callback functions and I think more like that. I have configured these things from within my application, on several places.
I expected that there was an method to only re-initialize things like state-machines, buffers, hardware, (static configuration). And let the dynamic configuration unchanged.

I agree. Currently all those "CO_xyz_init()" functions initializes or set to zero everything. And that is actually not necessary, because memory for all those objects is allocated with "calloc()", which sets them to zero anyway.
I think, we can change that paradigm and leave some variables or callbacks unchanged by "CO_xyz_init()" functions. I think, it is only necessary to remove some "set-to-null" lines from "init" functions and document that.
Can you prepare a pull request for what you think it is necessary?

Current approach in CO_CANopenInit() is to reconfigure everything, it clears also all callback functions. If there is no strong reason, I have no plans to change this behavior.


Warm transfer how to differentiate 2 calls connected

After warm transfer completed we are unable to identify the which call is hold.getActiveinitialConnection.hold will hold the active initial connection.
If we want to get the connection state in OnRefresh how we can identify which call is hold. First call or second call how do we differentiate the both calls.
Contact events having the contact I'd which common for all. Connection I'd is unique for 3 connections.
But we want to know onrefresh of contact state which connection is hold or resume or connecting or connected.

We can use onRefresh to identify all the calls connected to the agent with the connection id.

Hi @sivac47123,
you can identify which connection is on hold by calling connection.isOnHold() API.
For example,
contact.onRefresh((_contact) => {
  const connections = _contact.getConnections();
  connections.forEach((con) => console.log(con, con.isOnHold()));
});

Hope this helps. Please reopen the ticket if you have more questions regarding this issue.


Made the hero section responsive to all screens

🛠 Fixes Issue
Closes #10
👨💻 Changes proposed
feat: make navbar sticky
import: Open Sans font (font given in mockup)
fix: fixed some build errors
fix: navbar blocking page on mobile
fix: made hero section responsive + refactor code
Merge branch 'dscvitc:main' into main
🔧 Check List (Check the type of issue it fixes)

[x] Bug Fix
[ ] New Extension
[ ] Enhancement
[ ] Documentation

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] Any dependent changes have been merged and published in downstream modules
[x] I have done git pull before adding my new changes
[x] The title of my pull request is a short description of the requested changes.

This is similar to #37 could you please collaborate

Further there is a similar PR merged on dev branch #33 . Please work on the dev branch instead of the main

Further there is a similar PR merged on dev branch #33 . Please work on the dev branch instead of the main

Changes base to "dev"

Cool🚀


[api-draft][connector] Add SeaTunnel jdbc source

Purpose of this pull request

related to #1946
use ParallelSource.
Currently supports mysql, supports multiple split
Check list

[x] Code changed are covered with tests, or it does not need tests for reason:
[x] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[x] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

PTAL @ruanwenjun @CalvinKirs @Hisoka-X thanks.


Use authentication_keys for resource lookup instead of static email field

The Devise gem allows to use different authentication_keys instead of the "email" field.
This change (zhaknafein/devise-api@bb8057994632c93db74ab5ba006e5b81f56479b3) will make devise-api follow the same configuration as Devise.
Sorry for the missing translations and tests, i accept suggestions on how to replace the "email" word everywhere :)

Hello, I'm hitting the same issue and this would fix it. 👍 for the fix, those parameters if I understood correctly are already sanitized by the method sign_in_params, could simply pass params.


[Feature]: Interaction event matching

What happened?
I have a scenario where you can do a search to display an image of one of 100s of cards. Sometimes there are multiple matches (there are even similar versions of the card with the exact same name), and so I dynamically display buttons so that you can choose the best match.
The problem I'm facing, is I don't have a way to know which card was selected in this scenario. My old version would set the buttons custom ID to the unique ID for the card, and use that to display the correct one when they created that. But I can't viably register @ButtonComponent for every single card to handle updating the card.
If possible, it'd be nice to have something like glob matching for button IDs. For example:
  @ButtonComponent('card-btn_*')
  async pickCard(interaction: ButtonInteraction) {
    const [, cardId] = interaction.customId.split('_');

    await interaction.editReply(getCardMessageById(cardId));
  }

Reproduction
Feature request, N/A
Version
Stable
Relevant log output
No response
Code of Conduct

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

you should prefer @On wrapper for this task. since customid is not fixed.

however, will see if it's possible to add regex support 😉

Added regex support for button/selectmenu decorators, this would resolve many common case for developers.

Added regex support for button/selectmenu decorators, this would resolve many common case for developers.

Thats what i'm also asked for once in discord, thank you!


Ajax with sync

We are trying to add sync to an existing rails app that does a lot of ajax requests that render rails partials. When we tried to add sync to one of those partials, we noticed that jquery strips out the sync script tags, which seems to break sync. Is there a workaround for this? If not, would it be possible to patch sync so it could use html comments or some other tag instead of the script tags?

@ajb was there any thoughts into using comments instead of script tags? We are using pjax to load our main contents areas and jquery is stripping out the script tags. Sync is working on initial page load but once someone clicks a link to load the page via pjax Sync does not work anymore.

No, I only closed the thread due to age/inactivity. I'd be open to a PR or more discussion around this, but at first glance, I'm a little concerned about the performance implications of this, from XRay: https://github.com/brentd/xray-rails/blob/master/app/assets/javascripts/xray.js.coffee#L44-L45

Yeah, I was able to get it to work with using comments. Trying to see if there is a better way to handle getting the comments on load instead during each partial.

After playing with the comment implementation for a bit I decided to go a different route and wanted to see what you thought. Since each sync partial must start with a containing element, I inserted data-sync-item-id="<authToken" before the end of the first element end bracket (>). Then in the coffee script I say:
if !@$el or @$el.length == 0
@$el = $("[data-sync-item-id='#{@authToken}']")
Its working well so far but I wanted to see what you thought @ajb.

I can't tell you off-hand, since I'm not the author of the original code.
I'm more than happy to review a PR though.
On Mon, Apr 13, 2015 at 2:49 PM, Matt Ledom<EMAIL_ADDRESS>wrote:

After playing with the comment implementation for a bit I decided to go a
different route and wanted to see what you thought. Since each sync partial
must start with a containing element, I inserted data-sync-item-id="). Then
in the coffee script I say:
if !@$el or @$el.length == 0
@$el = $("[data-sync-item-id='#{@authToken https://github.com/authToken
}']")
Its working well so far but I wanted to see what you thought @ajb
https://github.com/ajb.
—
Reply to this email directly or view it on GitHub
https://github.com/chrismccord/sync/issues/121#issuecomment-92458110.

--
Adam Becker
(951) 9-BECKER
@AdamJacobBecker

What also comes to mind is how would we implement this for sync_new?
On Mon, Apr 13, 2015 at 2:49 PM, Matt Ledom<EMAIL_ADDRESS>wrote:

After playing with the comment implementation for a bit I decided to go a
different route and wanted to see what you thought. Since each sync partial
must start with a containing element, I inserted data-sync-item-id="). Then
in the coffee script I say:
if !@$el or @$el.length == 0
@$el = $("[data-sync-item-id='#{@authToken https://github.com/authToken
}']")
Its working well so far but I wanted to see what you thought @ajb
https://github.com/ajb.
—
Reply to this email directly or view it on GitHub
https://github.com/chrismccord/sync/issues/121#issuecomment-92458110.

--
Adam Becker
(951) 9-BECKER
@AdamJacobBecker

FWIW, I recently implemented a feature in an app where a partial that contains a sync'd element is rendered via AJAX:

I remembered this issue while I was implementing the feature, but it didn't give me any trouble. @RobinClowers, if you've still got time for this ticket, I'm curious to know if I'm misunderstanding your original request, or if there's something else going on here?


[5869] Rename early years routes

Context
Rename early years routes
Changes proposed in this pull request
Amended Early years (undergrad) to Early years undergraduate
Amended Early years (salaried) to Early years graduate employment based
Amended Early years (postgrad) to Early years graduate entry
Amended Early years (assessment only) to Early years assessment only
Guidance to review
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 Rename Early years routes


BREAKING CHANGE: Migrate vuetify2

DO NOT MERGE YET
This is a breaking change update where we upgrade to vuetify 2. Another PR will fix a bunch of issues that arise with this commit.
You can see the current status at: https://paraview-glance-dev.netlify.com/

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Tour java improvements

Incorporated Till's feedback (also fixing pre-release workflow)
Cleaned up code Java snippets (mainly the calls)
Improved the JavaTour text
Added state introspection via psql instead of printing

I recently discovered the curl --json flag - it means we can shorten example requests to something like:
curl http://localhost:8080/example.UserSession/ExpireTicket --json '{"user_id": "123", "ticket_id":"456"}'

Up to you if you want to replace all the occurrences; either way it's a copy-paste operation for the reader.

@pcholakov I remember there was some concern with --json: https://github.com/restatedev/documentation/issues/31


🐛 [firebase_messaging] java.io.IOException: java.util.concurrent.ExecutionException: java.io.IOException: MISSING_INSTANCEID_SERVICE

Exception
java.io.IOException: java.util.concurrent.ExecutionException: java.io.IOException: MISSING_INSTANCEID_SERVICE

Check attached SS for details as there is no stack-trace we found.
Environment
Flutter 2.5.3 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 18116933e7 (3 months ago) • 2021-10-15 10:46:35 -0700
Engine • revision d3ea636dc5
Tools • Dart 2.14.4
Android AVD
Galaxy Tab A Lite
Code which is causing exception
this._firebaseToken = await FirebaseMessaging.instance.getToken();

File inside pub causing issue:
/Users/user/.pub-cache/hosted/pub.dartlang.org/firebase_messaging_platform_interface-3.0.9/lib/src/method_channel/method_channel_messaging.dart

@Purvik
The error you are getting MISSING_INSTANCEID_SERVICE is due to token not getting generated and that's because the devices you are using may no support Google Play Service. Only devices with Google Play are supported.
See this link for more details.
You can try on Google Play Service enabled device / tablet or emulator and see if you still get exception or not.
For now, I am closing this as a non-issue.


Tabular: Early stopping is too aggressive

There seems to be some kind of global early stopping setting that I'm having difficulty disabling. Is there early stopping setting in task.fit that is agnostic of the algorithm being used? If so, is there any way I can disable it so I can run the full iterations specified?
I am trying to train two CatBoost configurations with 5 bagging folds and 1 stacked ensemble without time limits. I am able to train a full 3,000 iterations using a Catboost classifier (not from Autogluon) with the same configuration, but when I employ this config in Autogluon, it early stops around 1,000 iterations consistently. Furthermore, the validation loss does not increase at any point, so I also am not sure why the early stopping is being triggered (and it does not seem to be Catboost triggering it).
Here are the Catboost configurations:
cat1_params = {
    'iterations': 3000,
  'learning_rate': 0.4,
  'depth': 7,
  'l2_leaf_reg': 3,
  'model_size_reg': 0.5,
  'use_best_model': True,
  'od_type': 'Iter',
  'od_wait': 2000,
  'task_type': 'GPU'
}

cat2_params = {
  'iterations': 3000,
  'learning_rate': 0.3,
  'depth': 6,
  'l2_leaf_reg': 3,
  'use_best_model': True,
  'od_type': 'Iter',
  'od_wait': 2000,
  'task_type': 'GPU'
}

And here is the task.fit configuration:
predictor = task.fit(train_data=train_data, label=label_column,
                     output_directory=dbfs_dir, eval_metric='log_loss', verbosity=3,
                     num_bagging_folds=5, stack_ensemble_levels=1,
                     hyperparameters = { 'CAT':[cat1_params, cat2_params]},
                     presets='ignore_text')

As you can see from this image here, both training and validation loss decreases monotonically throughout the entire run, but something is triggering early stopping before it is able to run the 3000 iterations specified.

Thanks for submitting an issue!
I suspect that the model actually did have the validation loss increase which triggered the early stopping. For large enough datasets the early stopping rounds is 10, so you would need to set verbosity=4 to see the per-iteration losses rather than every 20 iterations to check for sure.
In 0.0.14, unfortunately there is no option to disable or change the early stopping rounds of the CatBoost and LightGBM models.
It is currently set by this code:
        if X_val is not None:
            X_val = self.preprocess(X_val)
            X_val = Pool(data=X_val, label=y_val, cat_features=cat_features)
            eval_set = X_val
            if num_rows_train <= 10000:
                modifier = 1
            else:
                modifier = 10000/num_rows_train
            early_stopping_rounds = max(round(modifier*150), 10)
            num_sample_iter_max = max(round(modifier*50), 2)
        else:
            eval_set = None
            early_stopping_rounds = None
            num_sample_iter_max = 50

To get around this, you can edit the code in autogluon/utils/tabular/ml/models/catboost/catboost_model.py
I will plan to make this overridable in v0.1 release. This way you should get roughly identical results training CatBoost alone compared to with AutoGluon, and you will be able to extend/turn off early stopping.

How to turn off early stopping.


Testing for key functions

Goal is just to have some amount of unit tests of the functions in place. In this case, because I am planning on making some significant revisions in #49 , this PR is focused on testing the expected behavior of the pre-processing, the input data validation, and then the immediate validation/tests of the inputs being passed to stan. It is not intending to be comprehensive. I have added to the description in #49 to specify that that PR requires that all those functions be tested.
This will close #27, but also some scope creep/clean up will close #48, #17, #51, and #21.

@dylanhmorris Think I addressed all of your questions here, ready for review

Ok I am a bit stuck on this particular CI issue, seems to have randomly cropped up. Otherwise ready for your continued review @dylanhmorris

Looks great, @kaitejohnson! A few minor comments. Since you are adding testing, I wonder: Do you have the coverage information? That should be handy now.

Yeah, let me look into adding that into CI?

Ok @dylanhmorris I think I covered all of your suggestions/additional tests. I think any other tests should be new issues, this was mostly just to have some amount of testing infrastructure, not to be fully comprehensive and I am sure we will find bugs/more things to write tests for as we go.
@gvegayon I am going to look into including testing coverage in CI

LGTM. Feel free to merge. Did you want to move the forecast_date out of the model_spec in this PR or keep it for a separate one, @kaitejohnson.

I think it would be good to do in this PR! Sorry I must have missed a comment, do you think it makes sense to just make it a separate argument?

I think it would be good to do in this PR! Sorry I must have missed a comment, do you think it makes sense to just make it a separate argument?

Yes I do.


Boolean applicator keywords must be non-empty

Nuget Package
JsonSchema.Net
Package Version
6.0.4
Operating System
Windows
.Net Target (if relevant, please specify the version in the description)
None
A clear and concise description of the bug.
The meta-schema https://json-schema.org/draft/2020-12/meta/applicator has minItems in schemArray: i.e, for allOf, anyOf.. any array. But schema works for thsi:
{
  "$id":"http://example.com",
  "allOf":[]
}

And validates anything
What did you expect?
Invalid schema
Please add test code or steps to reproduce the behavior.
execute
 {
  "$id":"http://example.com",
  "allOf":[]
}

in json-everything
Is there any other information you'd like to share regarding this bug?
No
Code of Conduct & Contributor Guidelines

[X] I agree to follow this project's Code of Conduct and Contribution Guidelines.

Thanks for reporting this.  I'll update this in the next version.


chore(main): release 0.8.0

:robot: I have created a release beep boop
0.8.0 (2024-11-18)
Features

add websocket broadcast functionality on post reading (65dd526), closes #47

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.8.0
:sunflower:


Kong Environment Variable Override

Summary
Kong will override the default .conf file parameters when environment variables KONG_* are created. I have noticed when a number of these environment variables exist but are empty/blank Kong still attempts to use them and it breaks application start up.
Steps To Reproduce

Create a bare bones environment. Set every environment variable as blank besides your db connection     required values.
Watch as Kong attempts to reference blank environment variables during startup and will go bananas(crash).

A few you can try it out on to see, I have not exhaustive tested each one individually yet:
-KONG_CASSANDRA_CONSISTENCY
-KONG_ERROR_DEFAULT_TYPE
After getting errors with the above 2 blank I rage quit and manually edited the existing deploy yaml to wipe all non used env variables to alleviate the problem.
Additional Details & Logs

Kong version 0.11.2

Let me explain why this is important, I am currently using Jenkins Jobs that reference an OpenShift template file that contains every parameter variable as an environment variable Kong supports. I wrap each parameter in "" 's since sometimes special characters and such in a db password will cause issues with how my Jenkinsfile handles the inbound parameter as well as some OSE templates in earlier versions have trouble parsing with unwrapped special characters. This means when my Kong launches every environment variable does exist but many are empty. I was hoping Kong's logic when choosing to override the default configurable environment variables check if the env var is empty or nil before using it, and in those cases  keep the default .conf files values.
Would you agree this would make how Kong enables overrides a bit safer and more tolerant?
And yes I could probably find a workaround in my JenkinsFile and OSE template but I think apps generally should make blank/null checks anytime they allow overriding of default application parameters.

Hello there,
Thanks a lot for writing such a lengthy issue, and I am sorry this has caused you trouble.
Unfortunately I'm afraid we'll have to decline this request.
While you argued for it very well, it comes down to a personal opinion: you think empty values should be ignored automatically by kong. We simply don't agree with that.
We see kong as a UNIX tool, and make it behave in consequence. The same way bash doesn't "set a default value" to $PATH if you set it to "" in your .bashrc file, kong will not "automatically discard blanks".
The only way I see around this is by altering your Jenkins configuration. I hope you get it working in little time, now that you have found the problem. Again, apologies for the difficulties this whole situation have caused you

Sure, the simple workaround for me is to provide my Jenkins defaults for blanks anyways. I will just set all my defaults to the kong.conf default that way there will be no blanks :) . Thanks for looking into this though.


Add ability to specify the attached disk type

When attaching the disk you have the ability to specify traditional or ssd, can you please make this an option. One costs way more than the other

Closed with commit 981d1e9820ba547fce7c777e186ba86416758071. Options are Default (Premium for VMs that support Premium Storage, Standard for those that don't) and Standard.


Emit "changed" signal when value is changed in ranges.

Emit "changed" signal when value is changed in ranges.
Also included the correct property info to the changed signal, it seemed to be missing.
Fixes #6354

This is intended, you have value_changed for this, but this class is a mess and will be rewritten after 3.1 is out to make more sense.


Adds an example entry

Thanks for the crate! Just a minor tweak to easily run example with  cargo run --example minimal.

Thanks! But, does cargo run --example minimal not work for you without this?

Weird, I believe it did not work until I updated to the latest toolchain, but now I'm unable to reproduce even with downgrading. Sorry for distraction, I believe I've goofed up somewhere!


Use SKBitmap-backed bitmap for RenderTargetBitmap

This removes the need to Snapshot the image for saving and enables CopyPixels

I have tested this PR and it works as intended.

@llfab @kekekeks is there an issue for this PR? Feels like there was one. Just to close it.

@llfab @kekekeks is there an issue for this PR? Feels like there was one. Just to close it.

No, I don't think so. There was my PR which is now obsolete: https://github.com/AvaloniaUI/Avalonia/pull/10286