remove last enzyme tests

Overview

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: [ABC-123]
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change
[ ] This PR includes testing instructions tests for the code change
[ ] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Don't make me tap the sign.
PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

Portal
Portal Link
Portal Env

Another Repo
Another Link
Another Env

📬Published Alpha Packages:@codecademy/gamut-icons@7.9.7-alpha.bedaf1.0@codecademy/gamut-illustrations@0.32.9-alpha.bedaf1.0@codecademy/gamut-kit@0.6.262-alpha.bedaf1.0@codecademy/gamut-labs@34.0.9-alpha.bedaf1.0@codecademy/gamut-patterns@0.8.22-alpha.bedaf1.0@codecademy/gamut-styles@14.2.5-alpha.bedaf1.0@codecademy/gamut-tests@4.0.22-alpha.bedaf1.0@codecademy/gamut@51.7.2-alpha.bedaf1.0@codecademy/markdown-overrides@0.9.19-alpha.bedaf1.0@codecademy/monaco@0.6.119-alpha.bedaf1.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://63b897db9e5d250057531942--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@7.9.7-alpha.24963a.0@codecademy/gamut-illustrations@0.32.9-alpha.24963a.0@codecademy/gamut-kit@0.6.262-alpha.24963a.0@codecademy/gamut-labs@34.0.9-alpha.24963a.0@codecademy/gamut-patterns@0.8.22-alpha.24963a.0@codecademy/gamut-styles@14.2.5-alpha.24963a.0@codecademy/gamut-tests@4.0.22-alpha.24963a.0@codecademy/gamut@51.7.2-alpha.24963a.0@codecademy/markdown-overrides@0.9.19-alpha.24963a.0@codecademy/monaco@0.6.119-alpha.24963a.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://63b89becec314a009498684e--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:eslint-plugin-gamut@2.0.3-alpha.7d0c0b.0@codecademy/gamut-icons@7.9.7-alpha.7d0c0b.0@codecademy/gamut-illustrations@0.32.9-alpha.7d0c0b.0@codecademy/gamut-kit@0.6.262-alpha.7d0c0b.0@codecademy/gamut-labs@34.0.9-alpha.7d0c0b.0@codecademy/gamut-patterns@0.8.22-alpha.7d0c0b.0@codecademy/gamut-styles@14.2.5-alpha.7d0c0b.0@codecademy/gamut-tests@4.0.22-alpha.7d0c0b.0@codecademy/gamut@51.7.2-alpha.7d0c0b.0@codecademy/markdown-overrides@0.9.19-alpha.7d0c0b.0@codecademy/monaco@0.6.119-alpha.7d0c0b.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://63b8ce1a0e674d26235459f4--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:eslint-plugin-gamut@2.0.3-alpha.6b6c40.0@codecademy/gamut-icons@7.9.7-alpha.6b6c40.0@codecademy/gamut-illustrations@0.32.9-alpha.6b6c40.0@codecademy/gamut-kit@0.6.262-alpha.6b6c40.0@codecademy/gamut-labs@34.0.9-alpha.6b6c40.0@codecademy/gamut-patterns@0.8.22-alpha.6b6c40.0@codecademy/gamut-styles@14.2.5-alpha.6b6c40.0@codecademy/gamut-tests@4.0.22-alpha.6b6c40.0@codecademy/gamut@51.7.2-alpha.6b6c40.0@codecademy/markdown-overrides@0.9.19-alpha.6b6c40.0@codecademy/monaco@0.6.119-alpha.6b6c40.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://63b997700e674d33f6545b2d--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:eslint-plugin-gamut@2.0.3-alpha.4aa3ec.0@codecademy/gamut-icons@7.9.7-alpha.4aa3ec.0@codecademy/gamut-illustrations@0.32.9-alpha.4aa3ec.0@codecademy/gamut-kit@0.6.262-alpha.4aa3ec.0@codecademy/gamut-labs@34.0.9-alpha.4aa3ec.0@codecademy/gamut-patterns@0.8.22-alpha.4aa3ec.0@codecademy/gamut-styles@14.2.5-alpha.4aa3ec.0@codecademy/gamut-tests@4.0.22-alpha.4aa3ec.0@codecademy/gamut@51.7.2-alpha.4aa3ec.0@codecademy/markdown-overrides@0.9.19-alpha.4aa3ec.0@codecademy/monaco@0.6.119-alpha.4aa3ec.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://63bc59c41f34dc3af2224a89--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:eslint-plugin-gamut@2.0.3-alpha.54f1d7.0@codecademy/gamut-icons@7.9.7-alpha.54f1d7.0@codecademy/gamut-illustrations@0.32.9-alpha.54f1d7.0@codecademy/gamut-kit@0.6.263-alpha.54f1d7.0@codecademy/gamut-labs@34.0.10-alpha.54f1d7.0@codecademy/gamut-patterns@0.8.22-alpha.54f1d7.0@codecademy/gamut-styles@14.2.5-alpha.54f1d7.0@codecademy/gamut-tests@4.0.22-alpha.54f1d7.0@codecademy/gamut@51.7.2-alpha.54f1d7.0@codecademy/markdown-overrides@0.9.19-alpha.54f1d7.0@codecademy/monaco@0.6.119-alpha.54f1d7.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://63bc69ef745e5c0a70dcc96d--gamut-preview.netlify.appDeploy Logs


Add docs folder to structure

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
Clear concise documentation
Describe alternatives you've considered
Left as is
Additional context
Thinking along the lines of: https://github.com/dotnet/machinelearning/tree/master/docs

@aslotte agree? Input needed.

We can certainly do that. I was thinking we could build on the Wiki section instead and add a link from our Readme file to it (in addition to some simple examples there) https://github.com/aslotte/MLOps.NET/wiki
What do you think?

@aslotte sounds good to me :) I'll try and add things there. There are other tickets covering documentation, this one was only around the folder structure. I'll close this one.


No hint text on Code tab

There should be text on this page. Is it invisible or was it deleted?

I think Jake fixed this. Moving to validate


Build error

Hello,
When I compile Quasimodo with the command "invoke build-quasimodo" I get the error :
/usr/bin/ld: final link edit failed: wrong value
collect2: error: ld returned 1 exit status
make : *** [Makefile:43 : libquasimodo.so] Error 1
Could you help me?

Did all the previous steps in README/install.sh work as expected?

Sorry for the late reply. Here's the DOI - 10.5281/zenodo.7922448. However, a lot has changed since then. If things didn't work out still, can you please send an email to the email-id listed on the paper - https://arxiv.org/pdf/2302.04349.pdf. Thanks!


[BugFix] Support random distribution in dynamic partition scheduler

Fixes #22401

Support random distribution for aggregate key table;
Support random distribution in dynamic partition scheduler;

What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

[FE PR Coverage Check]
:heart_eyes: pass : 12 / 14 (85.71%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/clone/DynamicPartitionScheduler.java
10
12
83.33%
[269, 270]

:large_blue_circle:
com/starrocks/sql/analyzer/CreateTableAnalyzer.java
2
2
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 16 / 18 (88.89%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/clone/DynamicPartitionScheduler.java
10
12
83.33%
[269, 270]

:large_blue_circle:
com/starrocks/sql/analyzer/CreateTableAnalyzer.java
6
6
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 16 / 18 (88.89%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/clone/DynamicPartitionScheduler.java
10
12
83.33%
[269, 270]

:large_blue_circle:
com/starrocks/sql/analyzer/CreateTableAnalyzer.java
6
6
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 18 / 20 (90.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/clone/DynamicPartitionScheduler.java
10
12
83.33%
[269, 270]

:large_blue_circle:
com/starrocks/sql/analyzer/CreateTableAnalyzer.java
8
8
100.00%
[]

https://github.com/Mergifyio backport branch-3.1


Installer for Arch Linux

Add installer for Arch Linux, mycroft-core package in aur is flagged out-of-date.
Some kind of official support would be nice.
Thank you

Please! It affects many people who want to use it! Just modify the .sh script to be compatible with Arch Linux!

Desktop Linux installation is currently performed via out "Github" installation mechanism.  This works for Arch as well as Debian, Suse and Fedora.  See:  https://github.com/MycroftAI/mycroft-core#getting-started
I could create a general "linux_desktop.sh" script to include here -- would that be/have been useful and clearer for you?

The dev_setup.sh script already supports Arch. Kudos for that. But the package names seem to be incorrect, and the installation fails. I did the Github installation, has no problems on a Neon VM, but fails on Manjaro VM. Both systems freshly installed. It also fails on my host computer, also Manjaro, but many things installed already. It seems to be unable to install a specific Python library, seemingly related to venv and pip. I opened a forum post for that and even sent logs.
One suggestion of mine is Flatpak/Snap. An universal package format, running in a container which talks to the actual hardware and interacts like "normal" software. But without the hassle of dependencies which are bundled together with the software. If desktop Mycroft was a Flatpak/Snap, using venv would also be obsolete. Both platforms are really easy to use. And they get updated instantly and hassle-free, automatically so you push an update and all users get it in minutes. Just a suggestion, maybe worth considering. It is possibe that it is impossible.
Best wishes!


Add tasks to disable service

In our case, this is needed to remove duplicate nodes from a Jenkins controller.

@ala-ableton Please let me know when I should rebase/squash.

@ala-ableton I noticed a problem with this change, which is that you can't just deploy a Mac host which previously disabled its service (you'll get an error from launchctl). I've fixed this in 8712b13, but also added another cleanup-ish commit in ae9d005 to remove an unneeded changed_when: false task. Just FYI!


Ingredients amount change

I know this is all about static generation, so this might be a tough call.
Anyway, most online cookbooks provide a feature to specify the serving size and calculate the ingredients up/down accordingly.
I understand that this might be hard with a static side.
But maybe by using multiple files (one for each serving size) such a thing could be specified (even though it would be a ton of work administrative wise)

You could probably do this with JavaScript – provide a text field where the user can input a multiplier, then look for any numbers in the list of ingredients for each step and multiply them accordingly. But off the top of my head, here's a bunch of way this is tricky or can turn misleading quickly:

Quantities like "½" would need to be normalized first.
Some people write "1½" for "1.5", which is another case that requires normalization.
Another special case would be required for things like "2-3 tbsp".
Pluralization – if you multiply a recipe with 1.5, "1 cup" might turn into "1.5 cup", but should be "1.5 cups". While this might be doable for English, remember that this tool is intended to be language-agnostic.
If there's only one of a thing, some recipe writers might omit the number, writing "egg" instead of "1 egg". You could assume a missing number to be 1 and multiply accordingly, but...
...things like "salt" and "pepper" are frequently just written as-is, which shouldn't be turned into "2 salt(s)" when a recipe is multiplied. Also, in this case, the instruction might say to "add a pinch of salt", which won't be nearly enough if multiplying a recipe by 5! Hard to detect.
There's probably much more I didn't think of.

That's why I think adding a feature like this might do more harm than good – ingredient amounts might be off if specified in a non-standard way, potentially ruining a recipe. Then again, multiplying in your head on the fly is also error-prone. I assume proper recipe sites use well-tested, content-language-specific libraries for this, which is well outside the scope of this project.
You could always manually maintain two variants of a recipe – titled, say, "Lasagna (small)" and "Lasagna (large)" – with differing ingredient amounts. That already works fine and corresponds with your multiple-file idea, but as you say, it's a lot of work to maintain if you rework your recipes frequently.


Use github step names as span names

Looks like the API is lagging behind, so accessing the current step name of the one job that is in progress is often the last step name

temporarily changed to the action name as fallback, which makes this not a bug, but an enhancement

https://stackoverflow.com/questions/78430075/how-to-access-my-own-step-name-in-any-github-action


Automatic bulk flashing

There should be a special mode that can be enabled that flashes as soon as a the DFU bootloader is detected.
Suggested by IBNobody over on Reddit.

Over in #16 @NoahAndrews and I have been talking about how to best implement this. One of the (until now) undocumented intents of this issue was to make bulk flashing easier. We should be able to enable this, plug in a keyboard, have it enter DFU mode, automatically flash, unplug, plug in another keyboard, enter dfu mode, automatically flash, etc.

An interesting edge case I ran into with the new button: I had a keyboard already in DFU mode plugged in, I started the app, and then loaded the flash. Realizing I wanted to try "Flash When Ready" I unplugged the keyboard. After Flash Keyboard became disabled the Flash When Ready button did not show up. I had to Choose .hex again to have that appear.

I'll work on that edge case. Should be easy.
As I explained in the other thread, I don't like the idea of showing the FWR button before the hex file is loaded to our typical user. Your use-case, while valid, is certainly very exotic, and the typical user experience shouldn't be compromised to make it work. If we can agree on that, the question becomes how to present a bulk-flashing option to the user. Maybe a "Flash this file to every keyboard I plug in" button, worded better.

I agree with hiding complexity from the typical user. But what value is that button adding now? You don't save any clicks, you simply change whether you click a button before or after pressing reset.

From the other thread:

Also, IMO the main reason we need a flash-when-ready button is because it can take a while for Windows to detect the device, install the driver, and then for our app to detect the device. We don't want to make the user hold their finger over the button while all of that happens.

If that's the only use case you want to support why not fold it into the flash button?
In any case, that means we still need to build a mechanism to support the bulk loading use case. What's your suggestion to handle that?

That's a fair question. We should probably do that.
I'm thinking an "Advanced" button, that pops up either another (smaller) window with the bulk loading option or a menu with the option. Any thoughts on that?

That's one possibility. I'm not sure if that fits with the design paradigm we're using so far. I was considering the possibility of introducing bootstrap tabs and having an "Advanced" tab the user could flip over to.
I'm a little hesitant to implement this feature in a way that hides the fact it's enabled. That's part of why I like how the teensy loader has implemented it. I think having a toolbar across the top with one of the buttons being "Flash when ready" (teensy calls this "Auto" which is nice and concise) may be a good direction to move in. This is part of what I hinted at in the other thread when I said it may require waiting for more features to be implemented. Having a toolbar with one button seems somehow sub-optimal.

My thought was that the advanced button would go in the bottom left corner, in line with the others. ~Would that fix your concern?~ I like the tabs idea too, though I think it would perhaps add too much visual bulk.
Edit: I mistakenly thought your "having a toolbar with one button" statement applied to my advanced button at first. In what way do you think an advanced button doesn't fit with our design? It's less intrusive than adding a tab bar.

I'm a little hesitant to implement this feature in a way that hides the fact it's enabled.

You're referring to the bulk-flash feature here, right? I definitely agree. Maybe a slim banner across the top after each flash, saying something like "To flash another keyboard with this hex file, put it in dfu mode now."
The toolbar across the top of an advanced tab sounds good to me, aside from feeling mixed about a tab bar. What other buttons do you think belong there?

Haha, whoops. Let's reopen this issue, and rename it to "Bulk flashing mode"

Go github! :)

Just in cased you missed it with the merge stuff, I did respond to your last discussion comment

Yeah, I think I agree with that. I think a dropdown menu from an advanced button (rather like Chrome's menu) would be a decent design though.
I think the advanced functionality like what you're saying would be in the toolbar should be hidden, so I'm not liking the idea of putting that toolbar on the main page. On the other hand, we could have a "show advanced toolbar" option in our advanced menu dropdown. ;)
I don't like the idea of a Fuses tab and an Advanced tab. If there's an advanced tab, that kind of implies that the other tabs aren't advanced, which a Fuses tab certainly would be.
I'm kind of starting to lean toward having the advanced menu with an "enable advanced mode" option.  The bulk flash could also live in the menu, since it could be useful to someone who shouldn't be messing with all of the advanced options and doesn't want the full advanced mode. The advanced menu could also have an option to show the full status window, like we have now. I like the idea of the full advanced mode having tabs, but I'm undecided.

The particular collection of tabs can be changed. Likewise the particular collection of toolbar buttons. I think the bigger question to answer right now is whether one of those is the direction we want to move in.
I think figuring out what kind of user we're targeting will help a lot. The least technical user will never flash their keyboard. I hope they'll find this tool simple enough to use, but the chances they will even try are small. The most technical user will probably be using "make dfu" to flash their keyboard. That means we're targeting a goldilox zone of power users who are technical but probably aren't comfortable with the CLI.
I'm a big fan of discoverability. I'm also a fan of not using platform specific techniques if I can avoid it. When those two intersect, as they do here, I'm inclined to find the path that works for all platforms. Right now I'm leaning towards tabs as the right balance to satisfy both. We can change what these tabs are, even in the future, but here's another stab at what they might be:

Flash - The default tab, what users see and the natural tab to click on if they accidentally click on another tab.
Fuse - I think getting and setting fuses requires its own UI
Expert - Auto mode, status box, etc, all available here

The particular collection of tabs can be changed...  I think the bigger question to answer right now is...

Yeah, sorry. I was on a robotics team in high school, and I heard similar phrases fairly often. :P

I think figuring out what kind of user we're targeting will help a lot.

Agreed.

The most technical user will probably be using "make dfu" to flash their keyboard.

At least at this point in time, there are actually a lot of command-line-comfortable users using this tool because of the build and flashing situation on Windows. I'm one of them. The readme currently recommends to Windows 10 users that you build the hex file using the Windows Subsystem for Linux command line, and flash it using this tool.

That means we're targeting a goldilox zone of power users who are technical but probably aren't comfortable with the CLI.

In the future, this zone will presumably expand to include less technical users, if we add Teensy support (allows us to support Ergodox EZ users who are using this tool) and whenever you finish your own configuration tool. If you and Jack want to sell more keyboards, I suggest we focus on making keyboard customization as easy as possible, and that includes flashing. That's why I'm so resistant to making the advanced stuff too easy to get to. And it's not like a big button that says "Advanced" is hard to discover. We can also make it so that when the user switches to expert mode, that preference gets saved, eliminating the extra step in the future.
I'm still pushing for a simple mode that doesn't have tabs, but I do like tabs for showing the advanced stuff.

One of the reasons I started working on this tool was so that users of my (in progress) web GUI could flash their keyboards. I'm very sympathetic to the "keep it simple" argument. I also think that giving users discoverable hooks into your more advanced features, so that they know they exist when they're ready to try them, is important.
That's why I think it's important that it be easy to get back to the simple mode if you've accidentally clicked into the advanced mode. Tabs are a well known and cross-platform UI method that non-technical users are highly likely to be familiar with. If you accidentally click on Advanced, there's the Basic tab right there that they can find. You will always have some confused users, but I submit that tabs will confuse the fewest.

I hadn't brought it up, but the whole time I was thinking of a confirmation popup before the user would be allowed into expert mode. It could also inform them of an easily-visible button to bring them back to the standard interface.
Your arguments do make sense, and I'll accept tabs if I have to. It's just that I love the elegance of the current design, and I feel tabs would do a lot to make it uglier. Plus it reduces the chance of messing with dangerous options.

I just created #24 as a demonstration of the tabs interface. I think you'll find it's really quite simple, and that non-technical users shouldn't find it too difficult to figure out.

I do like that better than what I was envisioning (more like a browser tab). :)
I do have another suggestion though. If we're going with the tabs-up-front approach, I think we should try to keep it to just simple and expert. Maybe instead of a fuses tab, we have a "Configure fuses" button, that brings up a configuration interface, maybe in a modal. Any thoughts there?

This has been implemented in #36.


Add thousands separators for block height

Fixes https://github.com/helium/explorer/issues/304
Add thousands separator for block height in two locations:

On transactions page

In the block height column of the activity list

I also searched the codebase for other instances of block.height and height and couldn't find another instance that didn't have .toLocaleString().

Hmm I don't have the ability to add a reviewer but tagging @danielcolinjames <- thanks for pointing me in the right direction!

awesome, thanks @rootulp ! will review this today 😄


Creating OPU or IPU in BV returns a 422 Unprocessable Entity in FEAGI

resolved due to URL typo

confirmed


How to delete wrong model from nuclio

I created my own model through nuclio, but I found a problem with the name. Now I want to delete the wrong model and the wrong model will appear.Function name doesn't conform to k8s naming convention and cannot be deleted, how should I solve this problem

We do not develop or support nuclio, please contact the nuclio team.


Love the plugin!

Hey just wanna say I love this plugin! Adding some cosmetic feature into this game is a nice change once in a while. Is there by any chance that you can add more teleportation transmogs into the plugin? So far there are only 3 and they are very similar in nature. Would love to see a new variety. Something like the desert amulet teleport would be aweome

I have added 6 new teleport effects with this update https://github.com/runelite/plugin-hub/pull/3262
Once it's merged in you'll be able to use them.
Thanks!
Nick


Not working on x64 - WIN7

C:\Users\sharrva>nodist
(x64)
No builds installed, yet.
Installed 0.9.10
C:\Users\sharrva>nodist + 0.9.10
0.9.10
C:\Users\sharrva>nodist
(x64)
0.9.10
C:\Users\sharrva>nodist 0.9.10
0.9.10
ERROR -
Unsupported 16-Bit Application
The program or feature "??\C:\dev\softwares\marcelklehr-nodist-v0.5.2-6-g037a98e\marcelklehr-nodist-037a98e\v-x64\0.9.10\node.exe" cannot start or run due to incompatibity with 64-bit versions of Windows. Please contact the software vendor to ask if a 64-bit Windows compatible version is available.

OK

I can't reproduce this. The beginning of the above path looks weird, though (\??\C:\).
Can you run the v0.9 executable directly? Can you run the x64 executables from http://nodejs.org/dist ?

My preference was to not install node from exe. Node otherwise works fine on this x64 machine. I will retry and let you know. Thanks for coming back.

Since nodist gets the node executables directly from nodejs.org it would help to know whether those work on your machine at all.

Added a comment at issue- #46. Need a way/fix to download the 64 bit versions.

FYI, you should consider uninstalling your current version of Nodist (follow the instructions in the README) and installing via the latest [and new] official installer (a "setup.exe"-style installer).  As of Nodist version 0.7.1, I am able to have it manage x64 versions of Node just fine.  :+1:

Closing this for now.


Add Image Badges

fix: #1934

Added Interactive Badges: Introduced badges for the project version, license, Python version, documentation, support, and demos to improve accessibility.
Enhanced README Header: Organized the badges in a two-row table format for a cleaner and more structured layout.
Improved User Engagement: Provided direct links to important resources (documentation, Discord support, and examples), improving navigation and user experience.
Highlighted Project Requirements: Clearly displayed the project version (v1.1.2) and Python version (3.8) to make requirements more visible.

@kaiyumdev, could you please remove the table and center everything?

@RymMichaut Hello, I have resolved the issue with this PR. If there are any further changes or updates needed, please let me know, and I’ll address them right away. Otherwise, please proceed with merging the pull request. Thank you!


Remove go.mod replace directive, use json fork directly

Remove go.mod replace directive, use json fork directly.
(Replace directives don't work beyond your active repo, which means they don't work for users of this library.)

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

Project coverage is 93.49%. Comparing base (c952a9c) to head (5baa06b).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main       #4   +/-   ##
=======================================
  Coverage   93.49%   93.49%
=======================================
  Files           1        1
  Lines         338      338
=======================================
  Hits          316      316
  Misses         16       16
  Partials        6        6

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


bug: build failed

build failed after pnpm create
error log:
> npm run build:ssg && npm run rss

><EMAIL_ADDRESS>build:ssg
> valaxy build --ssg

  🌌 Valaxy  v0.10.3

  🪐 theme   > yun
  📁 C:\Users\Samarium\Documents\projects\valaxy-blog

i use vite-ssg to do ssg build...                                                                                13:22:39

[vite-ssg] Build for client...
vite v2.9.14 building for production...
✓ 160 modules transformed.
[vite]: Rollup failed to resolve import "@waline/client/dist/waline.css?used" from<EMAIL_ADDRESS>This is most likely unintended because it can break your application at runtime.
If you do want to externalize this module explicitly add it to
`build.rollupOptions.external`

 ERROR  [vite-ssg] An internal error occurred.                                                                                13:22:41

Error: [vite]: Rollup failed to resolve import "@waline/client/dist/waline.css?used" from<EMAIL_ADDRESS>This is most likely unintended because it can break your application at runtime.
If you do want to externalize this module explicitly add it to
`build.rollupOptions.external`
    at onRollupWarning (C:\Users\Samarium\Documents\projects\valaxy-blog\node_modules\.pnpm\vite@2.9.14_sass@1.54.0\node_modules\vite\dist\node\chunks\dep-c9998dc6.js:41797:19)
    at onwarn (C:\Users\Samarium\Documents\projects\valaxy-blog\node_modules\.pnpm\vite@2.9.14_sass@1.54.0\node_modules\vite\dist\node\chunks\dep-c9998dc6.js:41613:13)
    at Object.onwarn (C:\Users\Samarium\Documents\projects\valaxy-blog\node_modules\.pnpm\rollup@2.77.0\node_modules\rollup\dist\shared\rollup.js:23215:13)
    at ModuleLoader.handleResolveId (C:\Users\Samarium\Documents\projects\valaxy-blog\node_modules\.pnpm\rollup@2.77.0\node_modules\rollup\dist\shared\rollup.js:22465:26)
    at ModuleLoader.resolveDynamicImport (C:\Users\Samarium\Documents\projects\valaxy-blog\node_modules\.pnpm\rollup@2.77.0\node_modules\rollup\dist\shared\rollup.js:22517:120)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async C:\Users\Samarium\Documents\projects\valaxy-blog\node_modules\.pnpm\rollup@2.77.0\node_modules\rollup\dist\shared\rollup.js:22412:32

You can see #77, which can temporarily solve your problem

You can see #77, which temporarily solves your problem, and I think this question is a duplicate

Thanks, solved by adding "@waline/client": "^2.6.2" to dependencies. I hope this can be fixed in a later version.

After that, I'll separate it.


Show parents/ancestors option

Hi, thank you for the great plugin!
This is a feature request/idea: how about offering an option to show resource parents/ancestors (i.e.: reverse hierarchy).
It's extremely useful when you have many CRDs and are confused which CR has others.

This is by design not supported so that the chosen object can be the root of the tree. Otherwise it might be part of a very large tree that might be hard to spot. ￼￼

Similarly an object might have more than one owner, in that case we don't wanna show multiple trees. ￼


Running multi-tests with DartOptions

Multitests seem to preclude the use of DartOptions. When combined, test runs fail like this:
Test configuration: dart2js_d8_release_x64_checked
Unhandled exception:
'file:///usr/local/google/home/eernst/devel/dart/work/sdk/tools/testing/dart/test_suite.dart': Failed assertion: line 1550 pos 12: '!isMultitest || dartOptions == null' is not true.
#0      _rootHandleUncaughtError.<anonymous closure> (dart:async/zone.dart:883)
#1      _microtaskLoop (dart:async/schedule_microtask.dart:41)
#2      _startMicrotaskLoop (dart:async/schedule_microtask.dart:50)
#3      _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:96)
#4      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:149)

To reproduce: Add a new test tests/language/multi_options_test.dart which contains the following:
// DartOptions=

library multi_option_test;

main() {
  int i = "42"; /// 1: static type warning, dynamic type error
}

Then run any one of these commands:
tools/test.py -mrelease --checked -cdart2js -rd8 language/multi_options
tools/test.py -mrelease -cdart2js -rd8 language/multi_options
tools/test.py -mrelease -cnone -rvm language/multi_options

Many other commands behave similarly, presumably because this has nothing to do with -c, -r, etc.
I first noticed this for a non-empty DartOptions (// DartOptions=--generic-method-syntax), so the assertion violation is not caused by its emptiness.

Yes, vm and dart options in comments inside the test are incompatible with multitests.  There are two separate systems of parsing a test to handle special comments in them, the multitest system and the options system, and so a test has to decide which system to use.


Fix #418 Added MySQL as DB storage layer.

Added MySQL driver to IronFunctions

It looks like you may have cloned off an out dated version of the postgres package.
I wonder if most of this code could be shared between the two.  Are there any other differences besides the table schema?  It would be nice to have a base sql package with specific behavior overridden or plugged in.

That is a good idea, since there aren't differences besides the sql schema. I will start refactoring the code and do a new commit.

@martinpinto according to test logs, there's problem accessing mysql inside test environment in CircleCI. So, in order to fix that, it is recommended:
At this point, you’ll want to create your database, load it with your schema, and (possibly) preload it with data. If you use MySQL or Postgres, you can use the circle_test database and the ubuntu user rather than creating the database yourself. No password is required for any database.
See https://circleci.com/docs/1.0/manually/#databases

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.4 out of 5 committers have signed the CLA.:white_check_mark: jmank88:white_check_mark: martinpinto:white_check_mark: seiflotfy:white_check_mark: denismakogon:x: CircleCICircleCI seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

I think CLA is broken on this one

Need docs for this: #596


New experimental feed

The experimental builds are now hosted on GitLab herein. This is because it has a 250GB limit instead of the "2GB" one we have with Azure Artifacts (though we were using ~7.5GB somehow?)
The feed URL is https://dotnet.github.io/Silk.NET/nuget/experimental/index.json - this is https://gitlab.com/api/v4/projects/47661606/packages/nuget/index.json but mirrored to our site so we can give a URL to our users that we have control over, meaning future switches between experimental feed hosts are seamless.
This also adds back the documentation page on the experimental feed that was never ported over from docs.ultz.co.uk


[BUG][WEBJS] -

Hi! I can't generate a QR CODE because instance stays STARTING forever, tried on different devices, and it repeats the same behaviour... Am i'm doing something wrong? Or is this a bug? I'm using the lastest version!

I have the same problem. Any solution?

Yea, same problem here. Im waiting they fix it

same， tried different version, seems communication with WhatsApp issues, old version also can not load QR code when start new session.

me too, please fix this.

Same issue. Latest version available, all sessions "Starting" forever:

same here, sometimes it starts but after scanning qr code nothing happens and whatsapp says after a minute that connection was not successful

I have the same problem here, it keeps showing the message "Whatsapp is out of date. Updating now ...."
No QR code is shown!
I think the problem is from Whatsapp web!

docker name changed, after change name it works

 docker name changed, after change name it works

it work for me also, thankyou

I have the same problem here, it keeps showing the message "Whatsapp is out of date. Updating now ...."
No QR code is shown! Whats wrong...

same here, using devlikeapro/waha-plus

I also have pro image from May and "Whatsapp is out of date. Updating now ...." problem. I could renew my subscription, but problem is still not solved.

For those who are still experiencing this issue, try pulling the image again and rerunning it. This resolved the problem for me!

For those who are still experiencing this issue, try pulling the image again and rerunning it. This resolved the problem for me!

please share the commands

For those who are still experiencing this issue, try pulling the image again and rerunning it. This resolved the problem for me!

please share the commands

I am using waha-plus
if you're like me:
You can download the image using:
docker login -u devlikeapro -p {KEY}
docker pull devlikeapro/waha-plus
docker logout
and then run it using:
docker run -it -p 3000:3000/tcp devlikeapro/waha-plus
If you're using normal version it's the same think just instead of waha-plus change it to waha. Also you won't need to login!
You can check the doc for more info:
https://waha.devlike.pro/docs/overview/quick-start/

thanks for sharing.
i had a script that was still calling the old docker name,
I confirm that now I get the qr code

:warning: Please make sure you're using the new docker image name devlikeapro/waha and devlikeapro/waha-plus in both docker pull command AND docker run commands.
Stop and remove the previous container if you have any (you can check it by docker ps)
Check GET /api/version response after that it must return 2024.6.8 or later (NOT 2024.6.1 or 2024.6.2 - it 100% means you're running old image, devlikeapro/whatsapp-http-api)
If after update you still have issues - kindle create SEPARATE issue and provide GET /api/version response with details.
https://waha.devlike.pro/blog/docker-image-has-been-renamed-to-devlikeapro/waha/


Method signature has incorrect const parameter

The parameter index should not be const and I also believe that parameter T is a const reference. The signature should read: bool IndexOf(T const& value, uint32_t& index) const;

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

ID: d8d728b7-7156-0b95-87aa-ad91cf5819cd
Version Independent ID: 820479aa-3aa5-710b-b877-7d1bfb2eb6a9
Content: IVectorView<T>.IndexOf(T, UInt32) Method (Windows.Foundation.Collections) - Windows UWP applications
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.foundation.collections.ivectorview_t.yml&version=GBlive&_a=contents
Product: uwp
Technology: winrt-reference
Git Login: hickeys
Microsoft Alias: hickeys

Sorry for the delayed response to this issue. The syntax block in our API reference are generated dynamically using reflection over the source files. It seems that since you opened this issue, the syntax generation has been fixed:
bool IndexOf(T const& value, [Out] uint32_t & index);
Thank you for contributing to the accuracy of our documentation!


Mutable borrow api implemented for sections

closes #2

Name section would be nice to get as well (for reading also)
with the note that it might be unavailable even if present until you use parse_names

why not to try parse_names in the fn names_section in case of it's absence? It'll be less confusing API I guess.

Maybe, if you can make a sound implementation of this


Avoid TypeError if container has already disappear

Summary
(In case of using Turbolinks or Turbo) When user fires render_async, and user move another page before async complete, then it raise TypeError:
TypeError: null is not an object (evaluating 'container.outerHTML = request.response')

This commit is for avoid this error, Thanks.

@nikolalsvk  Would you merge this pull request, and bump render_async version, please. Our app faces so many TypeError in our Sentry. Of course, if you have a time to do it :pray:

Hey, @yhirano55, I want to test this out and write an integration test during the weekend. If this is making a lot of problems, you can try to install render_async from your fork until I make a release. How does that sound?

I want to test this out and write an integration test during the weekend.

Sorry, I understood it. I’ll wait your great work. Thanks a lot.

I released these changes in https://rubygems.org/gems/render_async/versions/2.1.10, thanks for contributing


When prompting for assignment (after family member) mention the assignment

From @cougar2009
I found myself confused a bit on setup  "who has this assignment"
I think you could setup by saying "who would you like to say the opening prayer?"  and then "who would you like to give the lesson?
at the end of the well defined assignments you could offer the chance for a custom assignment...but I doubt it would be needed.

More context copied from slack.
[9:12]
So you think having the assignment prompted for first and then asking for the family member with the assignment name would be best, correct?  Just confirming the recommendation.  I hadn't thought about asking for assignments first yet.
James Hurst
[9:30 AM]
yeah I don't think you need to prompt for the assignment...because they are pretty well known and defined  (prayer, treat, lesson, song, etc)   so you can just skip right to it in the setup by saying  "who would you like to give the lesson"
Paul Eden [10:03 AM]
Interesting.  My thought was there are some families that do some assignments and others not.  For example, in my family, which only has 4 members we don't assign the prayer because its too small (there aren't enough of us for one of us to do just that).  We always rotate between conducting, lesson, treat and activity.  And the person conducting decides who prays and who picks the song.  Due to that I'm wondering if I should keep the assignment prompt there. (edited)
James Hurst
[10:06 AM]
valid points....
[10:08]
lots of ways you could go about it
[10:08]
when it said "who has this assignment"  I found myself asking myself "which one"  wondering what this was referrring to
Paul Eden
[10:27 AM]
Gotcha I'll either remove that prompt or add the name of the assignment/person into the prompt so it's more clear.


Make user supplied input more obvious

The command example given does not make it obvious that name is to be supplied by the user. Add square bracket notation around it to indicate the need to replace it.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "patch-1" git@github.com:elliot/machine.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Hi @elliot, you should sign your commits for the PR to be merged

Closing. Fixed in https://github.com/docker/machine/pull/2389


Fix missing icons on FF on general screen

Fixes #1311
Didn't create change log entry for such a small fix

I don't think so, it's only a visual bug fix


[AVM Module Issue]: Microsoft.Web/connections, need parameterValueSet for deploying service bus type API Connection

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Bug
Module Name
avm/res/web/connection
(Optional) Module Version
0.1.0
Description
Using Chrome developer tool to see the payload sent in JSON when creating an API connection that's service bus type. Similar to this Q&A https://learn.microsoft.com/en-us/answers/questions/1427460/how-to-paramatize-service-bus-managed-identity-in
We need either an additional properties parameter or a parmeterValueSet parmeter. Currently there are customParameterValues, nonSecretParameterValues and parameterValues non of them fulfils that parameterValueSet under properties.
{
  "type": "Microsoft.Web/connections",
  "apiVersion": "2016-06-01",
  "name": "[parameters('connections_servicebus_name')]",
  "location": "[parameters('location')]",
  "kind": "V2",
  "properties": {
      "api": {
          "id": "[concat('/subscriptions/', parameters('subscriptionId'), '/providers/Microsoft.Web/locations/', parameters('location'), '/managedApis/servicebus')]"
      },
      "parameterValueSet": {
          "name": "managedIdentityAuth",
          "values": {
              "namespaceEndpoint": {
              "value": "[concat('sb://', parameters('servicebus_namespace'), '.servicebus.windows.net/')]"
              }
          }
      },
      "displayName": "[parameters('connections_servicebus_name')]"
  }
}

(Optional) Correlation Id
No response

This module is currently orphaned: https://github.com/Azure/Azure-Verified-Modules/issues/767

Hi @DJHaose,
if it is just the missing parameter, we can implement it. I'd like to add an example of a Service Bus connection to the documentation. Would you mind providing a complete example of your scenario?

A request to add the property has already been created here: https://github.com/Azure/bicep-types-az/issues/1812


ANCM: warning message which happens when failing to create log directory is ambigous

Enable logging
Create a file named logs on the root directory to make ANCM fails to create logs directory
Send a request and check the message of event warning #1004

EXPECTED:
The warning message include the information that ANCM fails to create the "logs" directory path.
ACTUAL:
The below warning message is shown. The actual reason of the failure is to fail to create log directory but the message is saying that it fails to create file, which is too general to figure out the reason of the problem.
Warning: Could not create stdoutLogFile , ErrorCode = '0x80070003'.

I believe this fixed by #602
@jhkimnew Can you confirm and close?


Offscreen "strum notes" are not detected by the engine.

Describe your bug here.
So if a "strum note" is offscreen, The notes are weird:

Missing and Hitting does nothing
Scripts with the function opponentNoteHit, noteMiss and goodNoteHit does not work.

Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
Nope, Just a M.I.L.F script

More information about your issue is needed.

offscreen in what axis? Y or X?

Both


Support for Cassandra

It looks like Cassandra connectors for Spark are maintained through a third party packages. One specifically is by Datastax.
Is support for third party integrations, specifically Cassandra connectors, on the roadmap for Mobius?
Also, would you consider a pull request where Cassandra support is optionally included as part of the build?

I didn't see that there was an example folder and I just searching the SparkCLR solution.
The CassandraDataFrame example answers a few questions.

Good that the existing example answered few questions.
Mobius does not interact with any connector directly. Mobius relies on Spark's functionality to load the data to Spark - this is true for Cassandra as well.

That makes sense, thanks Kaarthik.
I've run into another issue, along the same line. Each connector implementation can provide RDD implementations. These implementations may have extensions to the general scala RDD api to make data access more efficient for the underlying store. See Cassandra data selection, filtering and grouping and Performing efficient joins. It's my understanding that Mobius would have to interact with these RDDs. How would I go about accessing one of these methods? Would I need to implement Cassandra specific extensions to Mobius' RDD classes?

@skaarthik any word on this?

@dwnichols - your assessment is correct. For RDD (and Unstructured Streaming), extensions are needed for data sources like Cassandra. Mobius has implemented extensions for Kafka and Eventhub for unstructured streaming. As far as RDDs, Mobius does not have any extension including the one that is needed for Cassandra. It is not too hard to implement such extensions for Mobius but at the same time it is not too easy because the underlying APIs needed in Mobius are not a part of its public API (should have been part of the public dev API). HTH.

@dwnichols - closing this issue for now. Feel free to reopen if needed.


"Multifactor Init Lifetime" should default to 0 when configuring external IdPs

Preflight Checklist

[X] I could not find a solution in the existing issues, docs, nor discussions
[X] I have joined the ZITADEL chat

Describe your problem
Given I have configured an external IdP,
When I login,
Then a user is created, and
by default I get the 2FA nudge
Now I have to figure out how to disable this nudge. The setting "Multifactor Init Lifetime" is not easy to find. This makes the whole setup process harder than it should be.
Describe your ideal solution
Option 1
Set "Multifactor Init Lifetime" to 0 as default organization settings
Option 2
Given "Multifactor Init Lifetime" is > 0,
When I activate an IdP,
Then I get a pop-up asking me if I want to disable the 2FA nudge
Option 3
Given "Multifactor Init Lifetime" is > 0,
When I activate an IdP,
I get warned about the 2FA nudge with instructions how to disable it
Version
No response
Environment
ZITADEL Cloud
Additional Context
Maybe this is a a good spot to also check for other configurations that don't make much sense and propose the changes to the user.

@livio-a @muhlemmer @peintnermax Any opinions on this?
The quickest solution is probably to put the default to 0. And also works for those that do not use the console.
But I also like the 2 option, to get into a direction where we have more guided ways in the console ui

Option 1 would result in new setups without MFA nudge and new users now being aware where to enable it. Basically it's the same problem reversed, where unaware admins wouldn't know about enabling the nudge.
Option 2 or 3 sound the most reasonable and still uses the most secure default setup, where MFA is preferred when no IDPs are configured yet. We should also include updating the documentation in any case, besides updating the UI.

Yes that makes sense.
In that case I would go for option 2.
Will change the issue above to add the acceptance criteria

A customer had the same issue. Since they do not have any local users on those organizations anymore, they could disable the prompt, but if they ever had local and external users the problem would re-arise.


update join discord and claim button styles

update join discord button to type="link"
update the claim button to non primary

Closed via #1cdebee


TPM, not TMP

"Backing up TMP to Active Directory was supported only on Windows 10 version 1507 and 1511."

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

ID: 61c3977f-3b78-1d8e-ad8f-50538ad1934f
Version Independent ID: 46b9ab6d-7a30-4dab-70fa-e1c97cc3ec3e
Content: Set up MDT for BitLocker (Windows 10) - Windows Deployment
Content Source: windows/deployment/deploy-windows-mdt/set-up-mdt-for-bitlocker.md
Product: w10
Technology: windows
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

Funny typo. Thank you for pointing it out. 👍


Cannot find module solc\soljson

I installed TronBox on my windows system using
npm install -g tronbox
Project was init successfully using
tronbox init
But the compile commands like these are failing
tronbox compile
tronbox migrate

This is the error I'm getting:

Can you find the soljson_v0.4.25 in .tronbox?

Never worked for me on Windows.
Works fine on Ubuntu

may be you should try to run tronbox --download-compiler 0.4.25 again then rerun tronbox compile.

https://developers.tron.network/reference/tronbox-configuration#resolve-naming-conflicts-on-windows
You should do this. @chicket


InstallUI redirect if ardana-service requires auth (bsc#1090278)

After a deployment is successful, the ardana-service is in a mode where it requires authentication to operate. This causes the installui to get a 401 error when making most calla to the ardana-service rest service. When this happens, the UI will now redirect to the deployment summary. In the future (when the day0 UI is also used for day2 operations post-install), it will instead redirect to the login screen

obsoleted by https://github.com/ArdanaCLM/ardana-installer-ui/pull/100


Update README.md

Fix code snippits in readme.

Nope, looks like everything has been fixed.  Closing.


Issue with spread operator

I'm getting this error on Ubuntu:
# ./node_modules/.bin/electron-rebuild

/home/root/test/node_modules/electron-rebuild/lib/src/rebuild.js:73
                depKeys.push(...Object.keys(rootPackageJson.dependencies || {}));
                             ^^^

SyntaxError: Unexpected token ...
    at exports.runInThisContext (vm.js:53:16)
    at Module._compile (module.js:374:25)
    at Module._extensions..js (module.js:417:10)
    at Object.require.extensions.(anonymous function) [as .js] (/home/root/test/node_modules/babel-register/lib/node.js:152:7)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Module.require (module.js:354:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (/home/root/test/node_modules/electron-rebuild/src/cli.ts:9:1)
    at Module._compile (module.js:410:26)

Seems to be choking on the spread operator. Does electron-rebuild itself run under electron's node or the system's node? I'm guessing the system's node, which at version 4.2.6 can't handle the spread operator. Does that sound right? I can't necessarily upgrade it.
Here is my environment info:
Ubuntu: 16.04, ARMv7
system node: v4.2.6
npm: 3.5.2
electron: v1.4.16
electron node: 6.5.0

It works fine in my dev environment, which is running node 6.11.0.

FYI, the problem did seem to be the fact that my system node, which as at v4.2.6 wasn't new enough to support the spread operator. Once I upgraded to v6.5.0 the rebuild worked fine.

Thanks for filing an issue, but according to https://github.com/electron/electron-rebuild/blob/dbd5b278a7e435b7dd7ca7676388c550c1ced3a6/package.json#L30-L32 the minimum Node version for electron-rebuild is Node 6.

Checkout pull request to update docs with this info.


False reporting broken link when link is an email

🐛 Bug Report
In a .md file I have a link with an email:
Contact [support<EMAIL_ADDRESS>
and when I build the website I get the error: Broken links found!
as it tries to resolve the email link to an actual internal url, but it's not a website url.
Have you read the Contributing Guidelines on issues?
Yes
To Reproduce

Add [support<EMAIL_ADDRESS>in any md page
Run yarn build

It will yield the broken link error
Expected behavior

I expected the build process to not process the email link as an internal link.
Actual Behavior

Docusaurus build process tried to validate the email link, and failed to do so as it's not an internal url.
Your Environment

Docusaurus version used: 2.0.0-alpha.61
Environment name and version: Chrome Version 84.0.4147.135, Nodejs v12.18.2
Operating system and version: OSX 10.15.6 (19G2021) desktop

Reproducible Demo
Easily reproducible locally

Hi,
Contact [support<EMAIL_ADDRESS>
this is not a valid link and will lead to the 404 page as we try to navigate to it<EMAIL_ADDRESS>You should use the mailto: protocol:
Contact [support<EMAIL_ADDRESS>


⚠ Bofrost.lu has degraded performance

In 3420059, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2262 ms

Resolved: Bofrost.lu performance has improved in ce6a049 after 1 hour, 35 minutes.


Add --pre option to pip install

Pip by default only finds stable releases https://packaging.python.org/tutorials/installing-packages/#installing-prereleases
We can support packages with pre-releases only by adding the --pre option, we could add this as an option for the config file.
Not sure how many users could face this, or if there is a workaround and avoid the --pre option.
More background at https://github.com/rtfd/readthedocs.org/issues/5016

We decide that adding more flags is not a solution for all this cases. A better solution would be #1083


[request] internet radio/audio stream support

[request] internet radio/audio stream support
radios.m3u   --> radio playlist support

+1 for radio playlist support (m3u/audpl)

+1 for this

+1 for Icecast stream support

+1 for Icecast stream support

+1 for this

+1

+1

+1


Improve add item type UI in theme editor

In the "Manage Theme Items" dialog:

Trim the received type name, as did in the add item type dialog.
Allow the user to submit the type in line edit using enter.
Clear the type name line edit after the type is added.

Haven't looked at the code yet, but an empty type name is a valid value.

I known empty type name is valid for functions like Control::get_icon() to be used as an shortcut for current control name. But why does empty type name make sense in a theme? The hierarchy can't be displayed properly in the Inspector (or is that by design?)

I presume it makes sense for the same reason as making your own custom theme types: organization. If memory serves me right, items of an empty type are displayed directly on the theme in the Inspector instead of using a subsection. Maybe not ideal, but it should still work.
But the bottom line is, the API allows it therefore it should be allowed in the UI as well. I see no reason to remove it from the API either, as it doesn't introduce complexity or potential errors, but can be useful for someone.

I think the API does not do any validation.
It even accepts foo/bar which will add an additional level of hierarchy in the Inspector and will be rejected on deserialization because there is no type named bar (save the theme and restart the editor).
Anyway, I'll just do the trim, enter and clear-do-submit things, and make API validations in a separate PR.

Good idea. I would still keep the empty named type possible, but I welcome more validation for other values, especially if they produce bugs.

Thanks!


Auth user different from email

I cannot use another USERNAME_FIELD different from email
Issue Summary
I cannot superclass nor fork customer app in order to use another USERNAME_FIELD different from email
Steps to Reproduce
a) startapp MyUser with a model like
class MyUser(AbstractUser):
email = models.EmailField(verbose_name='indirizzo email',max_length=255,unique=True,blank=False)
credential = models.CharField(max_length=16, verbose_name='Utenza', unique=True)
first_name = models.CharField(max_length=64, verbose_name='Nome', null=True)
last_name = models.CharField(max_length=64, verbose_name='Cognome')
telefono = models.CharField(max_length=64, verbose_name='Telefono cellulare')
.......
is_active = models.BooleanField(default=True)
is_admin = models.BooleanField(default=False)
objects = MyUserManager()
USERNAME_FIELD = 'credential'

override legacy form login with
class EmailAuthenticationForm(CoreEmailAuthenticationForm):
credential = forms.CharField(label=_('Utenza'))
redirect_url = forms.CharField(
widget=forms.HiddenInput, required=False)
Done django settings but ... then login, register stop working...
even with some customization on forms of the forkep customer app it looks like should I re-write the entire oscar.apps.customer.forms because it is email hard coded  ?
What is the proper way to change login field from email to another ?
Thankyou for your time
Technical details

Python version: 3.8.12
Django version: 3.2.11
Oscar version: 3.1

No, you cannot have more than one field defined in USERNAME_FIELD.
One option would be to write your own custom login to check for both fields yourself.
You could create a custom authentication backend that tries to look up the user on the 'email' or 'username' fields
To write our own authentication backend, we need to implement at least two methods i.e. get_user(user_id) and authenticate(**credentials)
 `from django.contrib.auth import get_user_model
  from django.contrib.auth.models import check_password

  class MyEmailBackend(object):
              """
              Custom Email Backend to perform authentication via email
              """
              def authenticate(self, username=None, password=None):
                  my_user_model = get_user_model()
                  try:
                      user = my_user_model.objects.get(email=username)
                      if user.check_password(password):
                          return user # return user on valid credentials
                  except my_user_model.DoesNotExist:
                      return None # return None if custom user model does not exist
                  except:
                      return None # return None in case of other exceptions

              def get_user(self, user_id):
                  my_user_model = get_user_model()
                  try:
                      return my_user_model.objects.get(pk=user_id)
                  except my_user_model.DoesNotExist:
                      return None`

After writing the custom authentication backend, specify this authentication backend in the AUTHENTICATION_BACKENDS setting.
AUTHENTICATION_BACKENDS contains the list of authentication backends to be used. Django tries authenticating across all of its authentication backends. If the first authentication method fails, Django tries the second one, and so on, until all backends have been attempted.
      AUTHENTICATION_BACKENDS = (         'my_app.backends.MyEmailBackend', # our custom authentication backend         'django.contrib.auth.backends.ModelBackend') # fallback to default authentication backend if first fails
If authentication via MyEmailBackend fails i.e user could not be authenticated via email, then we use Django's default authentication ModelBackend which will try to authenticate via username field of MyUser model.

Thank you , I'll give a try.
btw I did'nt intend to use 2 fields:
I need just to use 'credential' instead of 'email'
Thank you again

Create the same custom authentication backend that tries to look up the user on the credential  field
You can replace this line on MyEmailBackend recently explained Here
user = my_user_model.objects.get(email=username)
to
user = my_user_model.objects.get(credential=username)
Now you can use credential instead of email

Thank you again
Very helpful :-)

It looks like you've figured this out, and that there isn't an issue in the core, so I'm going to close this. If you need further help, please use the mailing list/slack/stack overflow to continue the discussion.


Simple integration with Truffle

Edits to Ethereum::Contract.create to optionally use Truffle artifacts to initialize a contract object.

This commit adds a feature to Ethereum::Contract.create to make it possible to load ABI and bytecode from Truffle artifacts. It is probably more useful for attaching to already deployed contracts (for example contracts deployed via Truffle migrations).

Good stuff! I'll take a closer look and get back to you.

Cool Marek thanks. Just one thing: regarding the bytecode, I'm not sure which one of  'byteCode' or 'deployedBytecode' from the truffle artifacts should go in the contract, so that may need to be looked at more closely.

Hi, thanks for this PR helped me get going talking from Ruby to gEth node for truffle deployed contract.
The only issue was line #72 in contract.rb. I had to change to unlinked_binary for it to work.


Add tests for some bash edge cases

Current coverage is 80.77% (diff: 100%)

Merging #158 into master will increase coverage by 1.63%

@@             master       #158   diff @@
==========================================
  Files             5          5
  Lines           796        796
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            630        643    +13
+ Misses          166        153    -13
  Partials          0          0

Powered by Codecov. Last update bbbef66...35665c3


No Way To Get A New Salvage Magnet IC

Description

Whenever a meteor blows it up and the board's gone, they'd need an admin to spawn a new one. Maybe either make the board indestructible or salv can somehow buy/craft a new one that gets locked when one already exists?

magnet pulls are tied to the station so technically having multiple isn't really an issue. just for reference for whoever tackles this.


author_profile not working in posts

Environment

Minimal Mistakes version: last
Ruby gem or remote theme version: mmistakes/minimal-mistakes
Jekyll version: N/A
Git repository URL: https://github.com/Flodu31/Flodu31.github.io
GitHub Pages hosted (if yes provide URL to site): https://cloudyjourney.fr/
Operating system: N/A

Expected behavior
In my blog, I can't see the profile on the left, for posts.
Steps to reproduce the behavior
Here is an exemple: https://cloudyjourney.fr/2021/01/07/migration-blog-github-pages/
Other
It works for pages, in my index, but not for posts
Thanks for your help.
Florent

@Flodu31 , why did you close this? What was the end result or solution? Please always post what you did to fix something if you got something to work. It may help others later on.

@Flodu31 , why did you close this? What was the end result or solution? Please always post what you did to fix something if you got something to work. It may help others later on.

Hi,
I added an author.yml file with details, in the _data folder.

Hi,
I added an author.yml file with details, in the _data folder.


Add run_policy to ElasticTask

Why are the changes needed?
Elastic tasks currently do not expose a run policy. As such, the run policy of a PyTorchJob created via an Elastic task cannot be configured. The same functionality exists for a PyTorchJob created via a PyTorch task.
What changes were proposed in this pull request?
We are adding the same run policy configuration support from PyTorch tasks to Elastic tasks.
How was this patch tested?
Unit tests added
Check all the applicable boxes

[ ] I updated the documentation accordingly.
[X] All new and existing tests passed.
[X] All commits are signed-off.

@fg91  I messed up the DCP with the original PR, so re-opening as a new PR here


Add allowEmpty endpoint option

Component "camel-git"
Just added a "allowEmpty" option to the endpoint to manage empty commits. I initilized this option to true for backward compatibility reason whether it sounds more natural to set it to false to match native git behaviour. I also create the corresponding header that take precedence on the property.
I tried to create an issue with this but at that time the site : http://issues.apache.org/jira/browse/CAMEL were Offline.

Thanks. I will review tomorrow :-)
Can you raise a JIRA anyway just for tracking purpose?
Thanks

Please, open the same PR against master. Since this is a new option, this can only be released with a new major release. We'll wait for your PR against master.
Please also run mvn -Psourcecheck to fix codestyle errors.
Thanks.

I'll do that ! Thanks for your quick reply. I had a particular concern on the behaviour of the component. It's making use of exchange.getOut which build a total new message and then lose all headers. Is there a particular reason for that ? If you manipulate a file and need to do "git add" and "git commit" you may lose the commit message in between. I was thinking about refactoring this in a new version but since my little change cannot be a fix.... Let me know !

When I implemented the comment I based my idea on single operation that's why I used exchange.getOut. What we can do is copy the headers from IN to OUT inside the component code.

Great ! At least the CamelGit headers I would say, but may be all.

We can use the MessageHelper method
https://github.com/apache/camel/blob/master/camel-core/src/main/java/org/apache/camel/util/MessageHelper.java#L488-L490

Fine !

I was planning to add some new commands as well like RemoteAdd that I need if I start with a empty repo without cloning

Thanks. Since the other PR has been merged, can you close this one?


Added hall_of_fame attribute such that users can retrieve the k best …

…scores and parameters from a run.

if you have to add another output to maximize, please change it so that it returns an extra dict instead so we can start simply naming additional new outputs.

I felt like using the other objects to retrieve the hall of fame was a bit circuitous. I'll add the item to maximize.

This is all ready to go.

Can you fix the conflicts so I can merge this? Thanks for the pull request!

I resolved these as best I could but I think another commit occurred around the same time as mine. Perhaps we should have the other merger have a look over my resolutions.


Validate SLES 15 SP2

validate SLES 15 SP2 on 2.4 branch

The following OS & docker version combination has been validated for provisioning custom cluster in Rancher v2.4-head single-install setup.
suse-sles15-sp2-docker-19.03.5


Blue square appears at end of description of image uploaded via Nearby

With latest master, upload a picture using Nearby
Open the picture in your contributions
Notice the blue square (shown by the red arrow):

Actually the image as seen on the website does not even seem to have a description:
== {{int:filedesc}} ==
{{Information
|description=
|source={{own}}
|author=[[User:Syced|Syced]]
|date={{According to Exif data|2021-03-24}}
}}
{{Location|35.04006388888889|139.81208055555555}}
== {{int:license-header}} ==
{{self|cc-zero}}

{{Uploaded from Mobile|platform=Android|version=<IP_ADDRESS>1~7a04a2b}}

[[Category:Tomiura, Minamiboso]]

It seems to also happen sometimes for Non-Nearby uploads.

Could I be assigned this issue? I experienced the same bug today. I use the application in dark mode and it is also present there.

@kurtkaiser It is yours, thanks! 🙂

I am noticing the date and disappointed in myself. I have not given this the focus it deserves. I hope to spend more time on it but if it needs to be passed to someone else I understand.

@kurtkaiser No problem, take your time :-)

Having pulled that latest and rebuilt I am no longer seeing this issue, pictured below. Does this appear to be resolved for anyone else?

Thanks for coming back to this!
I am still seeing this issue on https://commons.wikimedia.org/wiki/File:July_2004,_Pays_Dogon_15.jpg that I uploaded with the app yesterday (GitHub master or super-close to it):

@kurtkaiser Are you still working on this? I still see the issue. Thanks! :-)

Hello @nicolas-raoul
I have also faced the similar issue today while reviewing
https://github.com/commons-app/apps-android-commons/assets/114223204/3862c507-9bed-4afa-ae7f-de495a6c577e
On going to same image through explore map gave me same blue box
https://github.com/commons-app/apps-android-commons/assets/114223204/5d292d14-9d7c-43b3-90ec-f480121240f6
Can I work on this issue ?

I have found the issue, it is with the HTMLTextView I have fixed it :-). Will be sharing the PR as soon as possible.
https://github.com/commons-app/apps-android-commons/assets/114223204/72addb0a-8a61-4431-8bf2-6065d802c6aa


Update German translations

The prisoner is indeed referred to as "Häftling", fixed grammar, their sarcophagus is referenced to as "Kammer" (chamber).
I renamed the entrances as the given words are grammatically wrong unless this was in the ship log as "given name" for them.
While we are at it, I really dislike the German name given for the dam. I could not find ingame references to it so I am skipping this one.

Maybe the dam should be renamed to Reservoir? This is the official name of the area, isn't it?


Fixed variable naming in behavior testing example

Why is this necessary?

Coverage remained the same at 100.0% when pulling 63962fe299949264f6d66fdc33ab740d89baabb7 on yohanmishkin:patch-1 into f68536fcb456b0cb18f3e419da8beac9f9335fbc on react-boilerplate:dev.

Thanks for your contribution @yohanmishkin !


[assume-mainactor] Only perform the change for items that are within the current module.

I also added a small runtime test just as a sanity check. We do not change any codegen here since the change is at the Sema level... but I thought it would be prudent to at least have a small smoke test.
rdar://140439795

@swift-ci smoke test

@swift-ci smoke test


ci: add nightly test against a matrix of Python and PyArrow versions

What feature or improvement would you like to see?
Let's make sure we test different versions of dependencies, including nightly/prerelease, if possible.
We may also want to include Polars and Pandas?

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx


Update spam_onmicrosoft.yml

Description
Negating auto replies.
Associated samples

https://platform.sublime.security/messages/08cf293cd8605b2c96b06240111281476c8eaed57af51c1fac6c47a711a8c213
https://platform.sublime.security/messages/cab25e4504848ff845adcc612fadae8ac6bd5c982ea90b34c16827df0ee6c61b

/mql-mimic-exempt 423145, 423146, 423152, 538014, 538105


docs: migrate community guide from inner source

Description

This migrates the Lightning Community Guide we had on the inner source side and adds information about the boundsMargin property.
References

Testing

Check that the "Lightning Community Guide" page appears in Storybook and works as expected.
Automation

Checklist

[ ] all commented code has been removed
[ ] any new console issues have been resolved
[ ] code linter and formatter has been run
[ ] test coverage meets repo requirements
[ ] PR name matches the expected semantic-commit syntax

Test Execution Failed.

:tada: This PR is included in version @lightningjs/ui-components-v2.21.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Not addiing data when datasource:"grid/getsupplier"

I am fetching json data from "/Grid/GetPlayers" method in controller when clicked add it doesn't add data to the grid
dataSource is "/Grid/GetPlayers" following type of error occured
Uncaught TypeError: data.dataSource.push is not a function
at Object.addRow (gijgo.js:3831)
at n.fn.init.self.addRow (gijgo.js:4342)
at HTMLButtonElement.Save (jquery-ui-grid.html:91)
at HTMLButtonElement.dispatch (jquery-2.1.4.min.js:3)
at HTMLButtonElement.r.handle (jquery-2.1.4.min.js:3)
I have also tried one of demo model given by your in "http://gijgo.com/LiveEdit/Index/jquery-ui-grid.html?component=grid" it gives the same error

We fixed that issue in github, but we are going to deploy this on gijgo.com with the next release on February. Thanks for the feedback.


Change the site promo banner to promote Chrome Dev Summit

What do you want it to say?

err. Lemme check in with Katie.  We are specifically wanting to promote the livestream at the moment.

Don't miss out on the action at this years Chrome Dev Summit, happening on Nov 10th & 11th. Register for livestream updates.


Failed to load bloomz 7Bmt

Hi thank you very much for this great library! I am really excited about it!
I tried to fine tune bloomz 7B with the 4 bit lora on alpaca by running: python qlora.py --model_name_path my_bloomz_path. However, the job was killed halfway when it was loading the base model.
It seems that some process amid the 4-bit quantization drains all the memory.Can you please give me some advice about how I can reduce the memory footprint in order to load the base model?
Thank you very much !

Hello I already solved the problem by setting a lower kill score for the process: echo -17 /proc/myprocessid/oom_score_adj . The cpu usage is 400% when loading the model and it gets killed.
Now it works as a charm!

hello!
I want to fine tune bloom 7B, which has exact same structure as bloomz.
But in my case, the model takes about 11GB with 4bit quantization.
I think it is not properly quantized because in the paper 7b model will take only 5GB.
could you share your case? how much vram bloomz 7b takes?

Hi do you set torch_dtype = tf.float16 or bfloat16 when you were loading
the model?  If yes then I think there is no problem with your code. Bloom
may take up more vram because it may have a large token embedding matrix to
support multi-lingual capability, compared with the English only focused
llama model
김재희(Kim Jaehee) @.***>于2023年7月12日 周三22:10写道：

hello!
I want to fine tune bloom 7B, which has exact same structure as bloomz.
But in my case, the model takes about 11GB with 4bit quantization.
I think it is not properly quantized because in the paper 7b model will
take only 5GB.
could you share your case? how much vram bloomz 7b takes?
—
Reply to this email directly, view it on GitHub
https://github.com/artidoro/qlora/issues/8#issuecomment-1632603204, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABN7Y6GPGPUKSBIWA6VJS2DXP2V6HANCNFSM6AAAAAAYNTY7WU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks for your reply!
I usedtype=bfloat16and double quantization as QLoRA proposed.
I think the memory usage reported in QLoRA(5GB for 7B model) is only quantized model's load.
Below is my obersvations.
all experiments use bloom-7b1, 4bit double quantization with nf4, bfloat16 dtype.
(1) Quantized Model Only

about 5.5GB

(2) Quantized Model and Copied Weight for Mixed Precision

about 10GB

So, the lowest memory requirement for training 7B model is more than just 8GB vram(which is general vram spec for commercial gpus).
+) 7B train with QLoRA with 4 batch size consumes about 24G


Bumped ccx-rules-ocp version

Codecov Report
Base: 72.68% // Head: 72.68% // No change to project coverage :thumbsup:

Coverage data is based on head (e30cd98) compared to base (efaa59c).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #412   +/-   ##
=======================================
  Coverage   72.68%   72.68%
=======================================
  Files          11       11
  Lines         582      582
=======================================
  Hits          423      423
  Misses        134      134
  Partials       25       25

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


enhancement/request-retransmission

closes #23

@oskarth, this is a rough implementation and not final yet, but opinion would be welcome already

Fun fact: this brings back race conditions

We should run simulations to see what effect this has, also should REQUEST and OFFER next epoch calculation be the same?


/quests command not work

(Required) What is your Quests version / build number (do not say "latest")?
3.4.5
(Required) Please describe the problem and how to replicate it. Any console errors?
/quests command not work
(Optional) Can you provide an example quest from quests.yml using Pastebin.com?
starter quests.yml

What happens when you enter the command? Any errors?

i not have any errors

command not exist

i am in 1.8.8

Alright, that either means the plugin is not installed or it did not load properly. Could you please upload startup log? On Windows, this is located in \logs\latest.log

https://pastebin.com/WP6y0ru5

i have the commands of PermissionsEx

/questadmin and /quest work but not /quests

In that case, try running the following command (replace yourName with your Minecraft username):
/pex user yourName add quests.*
and then run
/questadmin reload

it work thx


1password: fix hashes

Description of changes
Building on current unstable fails due to hash mismatch.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 181665 run on x86_64-linux 1

1 package built:

_1password

Definitely strange.
@ofborg build 1password

@ofborg build _1password

hey @savannidgerinel, do you have any ideas why the hashes might've changed?

hey @savannidgerinel, do you have any ideas why the hashes might've changed?

I do not. I don't work with the CLI team very much so maybe this is legit. But I will reach out to them and to our build engineer, though I don't expect a response until tomorrow.

Hey, y'all. Please hold off on this. I've talked with the CLI team and this was a mistake that they made last week. They're trying to revert it, but the caching servers are causing trouble.

Thanks for the update. So if I'm understanding you correctly, the original hashes should be valid again eventually, right?

Thanks for the update. So if I'm understanding you correctly, the original hashes should be valid again eventually, right?

Yes. In fact, they should be valid again now. The team was able to fix the problem yesterday afternoon.

Yes. In fact, they should be valid again now. The team was able to fix the problem yesterday afternoon.

I'm not seeing this, sadly. For example, https://cache.agilebits.com/dist/1P/op2/pkg/v2.5.1/op_linux_arm64_v2.5.1.zip returns a file with a SHA-256 of 91f5615f98eb4bb29124d6317a3a7a5330c3415cf0f789aaaffbe5f353ee713a (sha256-kfVhX5jrS7KRJNYxejp6UzDDQVzw94mqr/vl81PucTo=), which is equal to neither the original hash or the new one in this PR.


feat: added a clipboard-check icon

I have read the CONTRIBUTING.md file.
YES/NO
YES
I have run yarn gen-cli to generate the necessary files
YES/NO
YES
What kind of change does this PR introduce?
This change adds a new icon which wasn't existing before now

Bug fix, feature, docs update, ...
What is the new behavior?
The new behavior is that a new Icon is being added for users
Feel free to include screenshots if it includes visual changes.
Demo
Please attach a short video demo of the changes.
Additional context
Add any other context or screenshots.

it's kinda laggy, could you please fix that?
https://github.com/user-attachments/assets/2764a351-c063-48e3-922a-33d6bd127f33
also, for this animation it would be okay just draw this check icon using pathLength

right would do. Sorry for the late response.

Made changes as requested kindly check the vides below
https://github.com/user-attachments/assets/79f72c42-ac08-45a7-80fa-aefb39e36c8e

Made changes as requested kindly check the vides below
https://github.com/user-attachments/assets/79f72c42-ac08-45a7-80fa-aefb39e36c8e

normally pathLength should be 1, then the check path is visible. when mouseenter the pathLength should be 0 to 1, and with the opacity from 0 to 1.

made changes as requested. Kindly check the video below
https://github.com/user-attachments/assets/95eae173-ba65-4973-a371-7ce63fa9c902

made changes as requested. Kindly check the video below
clipboard.mp4

happy new year 🎊


Main index html banner is not mobile responsive

Is taken on a mobile Chrome on android
@rattermeyer  / @tjaeschke  - fyi ...
the code is pretty interesting to me ...

             *Installation Guide
             *Visit our repository

could we just introduce flex-wrap: wrap;  - based on https://stackoverflow.com/questions/41625258/button-group-not-wrapping-when-too-long


[fix/aut826] Refresh parameters table only when parameters change

Refresh parameters table only when user parameters or workflow change

This is very nice. I see these changes as a big step towards where we want to see the application progress towards on the UI side.
I had some very small comments prior to merging.

So as you see, I have done with the observer .. but I don't trigger for all modifications (we could have done notification in the encpasulation of workflow modification, or to the parameters modifications), for the moment, I think it should do the trick.


500 pizzas with pineapple in your door

oshi package is an open source library made for hwid / take hardware info, you can perfectly google it.
And the .so linux file is an hwid system made in Rust, if you read the code in the hwid things, you can see some native methods which works with that native library.
Take this down 😒

thanks for clarifying it.


jaeger-client package contains a Buffer Overflow vulnerability.

Explanation
The jaeger-client package contains a Buffer Overflow vulnerability. The append function in udp_sender.js does not properly limit the number of spans being appended during a flush to ensure that they fit within the _maxSpanBytes buffer. An attacker can exploit this by appending a large number of spans during a flush, causing an EMSGSIZE error and ultimately a Denial of Service (DoS).
Detection
The application is vulnerable by using this component.
Recommendation
There is no non-vulnerable version of this component/package. We recommend investigating alternative components or a potential mitigating control.
Root Cause
jaeger-client.3.19.0[3.14.3, )

We don't use Jaeger over UDP.

Thank You for prompt response. Please do provide additional details so I can working with our security team for waiver. Also I do noticed jaeger client was removed in earlier version v0.18 if so why it is reporting.

https://github.com/signalfx/splunk-otel-js/pull/445

OpenTelemetry's Jaeger export is still relying on the jaeger-client package (https://github.com/open-telemetry/opentelemetry-js/issues/2618). This should hopefully be fixed soon.
As for our repository - this is the code that creates a Jaeger exporter: https://github.com/signalfx/splunk-otel-js/blob/main/src/tracing/options.ts#L192-L213
It sets the endpoint configuration option, if this option is set OpenTelemetry switches to Jaeger HTTP sender: https://github.com/open-telemetry/opentelemetry-js/blob/main/packages/opentelemetry-exporter-jaeger/src/jaeger.ts#L134
So our distro does not hit the UDP code path.
Hopefully once the issue in OTel is resolved, vulnerability detection tools will stop reporting it. But in our case it is not a vulnerability unless users explicitly pass in a Jaeger UDP exporter via advanced config.

Is there a link to this report anywhere? Would be nice to see the details.

Jaeger exporter will be removed in an upcoming 2.0 release.

Thank You for update.Thanks & Regards,Kumar.On Oct 15, 2022, at 6:30 AM, Siim Kallas @.***> wrote:
Jaeger exporter will be removed in an upcoming 2.0 release.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Loosen package constraints

The pkg constraints right now are exceptionally hard to satisfy :)

I'm happy to loosen package constraints some what, but I'm nervous about removing them entirely. The current ones where automatically generated so they may be a bit strict.
I did just try to create a new project withe sized-grid using stack's auto resolve and it worked fine, so it doesn't seem the impossible to use this.


S3 UFS is getting much more read than needed to the UFS

Alluxio Version:
2.6.2
Describe the bug
When reading parquet files from cold cache with S3 UFS we see a huge bandwidth coming out of the store.
I believe this is because the S3 request is built with a starting offset but not with an end range. so it reads as much as it can until the HTTP request is aborted. would be better to start the http request only after the read call happens and apply some reasonable "end range" based on the read request. or at least start with some configurable "readahead" like S3A does
To Reproduce

Set up an S3 UFS with a large file
Try to read via alluxio only 100 bytes from the file
you should see the traffic from S3 is much greater than that

Expected behavior
S3 reads should be as small as required
Urgency
Blocker to use S3 UFS, but we can try other things
Additional context
Add any other context about the problem here.

@ggezer PTAL

I don't think that is the issue since we witnessed it while caching was
completely turned off.
We more understand that in certain cases impala will read all the "pages"
in the parquet files which might be many and small in case the table has
many columns (~500 for us). So in the case of alluxio over S3, this might
mean thousands of unended S3 reads over the same file.
On October 19, 2021 04:54:31 Beinan @.***> wrote:

Thank you @idanz ! I just want to better understand your use case, are you
just trying to read the footer of each Parquet file? I'm thinking if we
could cache parquet footer separately.
As @yuzhu said, alluxio is caching the data on a block level (64MB or
128MB), so reducing the block size might save you some bandwidth, you cloud
try the config below.
alluxio.user.block.size.bytes.default=16MB
Note that small block size (<=8MB) might cause some issue on the compute
engine (e.g. Presto) side, but I encourage you try 1MB block size to see
what would happen.
Thank you again for reporting this issue to us!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

If we look at the grpc request of each read,  looks like it always requests the entire file?

Ok, so I guess for each read it won't be beyond the chunk size, but it might keep looping to read the entire file.  I will take a look at how we construct the request on client side

@beinan for your reference, https://docs.aws.amazon.com/AWSJavaSDK/latest/javadoc/com/amazonaws/services/s3/model/GetObjectRequest.html#setRange-long-long- , I think we are issuing request with setRange(start) only.  see https://github.com/Alluxio/alluxio/blob/6a1876002e276c1a86d22b49f212e385f574f086/underfs/s3a/src/main/java/alluxio/underfs/s3a/S3AInputStream.java#L144

i am wondering if https://github.com/Alluxio/alluxio/issues/14215 is related. There we somehow exhausted s3 client pool, and caused reads to eventually fail. @LuQQiu @beinan


Training with Alternative Datasets

I have been struggling with training with alternative dataset. Since my dataset is in the RefCOCO format, I wanted to test by training focall_unicl_lang_v1.yaml with refcocog_val_umd. I specified the dataset mapper to be the refcoco one, but still didn't work. As this can be a big question, I would appreciate if you could highlight a few critical steps on how to make training on datasets like RefCOCO work. Thank you!

@theodore-zhao
what are the steps , you follow to train SEEM on your custom dataset


Is it possible to generate nginx.conf which the server in upstream uses K8s FQDN instead of IP Address in the ingress controller?

Hi,
With the ingress controller, in the upstream of the nginx.conf, the server is IP address as follow;
upstream xxxx-svc-8080 {
#least_conn;
sticky;
server <IP_ADDRESS>:8080 max_fails=0 fail_timeout=0;
server <IP_ADDRESS>:8080 max_fails=0 fail_timeout=0;
}
Is it possible to generate the server with FQDN, such as [pod-name].[namespace].pod.cluster.local instead of IP Address?
Thanks

@tjliupeng this is possible but requires a changes an the use of a resolver in nginx. The resolver in nginx is tricky and not good with updates and high number of queries. Also this introduce a percibible latency serving requests

Ok. Thanks. So we won't consider this requirement at present.


Add support for arbitrary image resolutions

Hi,
This PR adds support for arbitrary image resolution. Here's what I did to make it possible:

Rework Image Resizing layers as pointed out here
Rewrite Block and Unblock layers to use pure tensorflow: this was necessary because einops does not accept tensors as pattern arguments, which is necessary in some layers of maxim.
Realize that dim_u and dim_v could be substituted for the window_size squared.
Apply changes to the model itself. (edit: by that I meant substitute the layers for the TF ones, and plug in the necessary arguments for them to work)

I hope this helps, please let me know what you think.
you can test it quickly by: pytest -v maxim/tests/
Best,
Levi.

Please let me know if anything remains unclear.
Best,

@LeviVasconcelos
Things are looking good. Thanks so much for your hard work. I left a couple more questions.
I would suggest doing the following:

Including a detailed summary of changes needed to make this work in the README.
Running the conversion script for each model and verifying their outputs.
Creating PRs to each of the MAXIM model repositories (Google) here: https://huggingface.co/models?search=maxim

@LeviVasconcelos a friendly ping :)

I was in vacation this last week, thus the late reply.
What do you think of a quick call this or next week? I have a couple questions that I think would be quickly answered in a ~10 mins call.
Let me know what you think...

Would prefer chatting via email as I will be busy next week.

Just a friendly heads up: i will be very busy until jan 20th. Afterward i should start working on it.
Best,

Great to see the update. I'm eager to test out this code. My hope is to get it working in TFJS. I'm guessing that may require the implementation of a few operations, similar to what was done to get it working for arbitrary resolutions? Any tips or suggestions appreciated.

Great to know it however I don't about TFJS :(
Maybe reach out to Jason Meyes?

Hi,
sorry for the delay, life got in the way.

@LeviVasconcelos
Things are looking good. Thanks so much for your hard work. I left a couple more questions.
I would suggest doing the following:
* Including a detailed summary of changes needed to make this work in the README.

I uploaded the README file explaining in details the changes done.

* Running the [conversion script](https://github.com/sayakpaul/maxim-tf/blob/main/convert_to_tf.py) for each model and verifying their outputs.

I ran convert_to_tf for 5 different models. The results of the models can be found here.
I also modified run_eval by removing the dynamic_resize flag.

* Creating PRs to each of the MAXIM model repositories (Google) here: https://huggingface.co/models?search=maxim

Should I create the PRs right away? Or should we merge this first?

@sayakpaul friendly ping.

Hey thanks for your hardwork!
Could you be so kind to remind me about this again in maybe 1 week? A little busy right now.

@sayakpaul pinging as requested ;)

Hey guys, any update on this?

Gently pinging @sayakpaul here.

Hi! Any news on this PR?


Issues connecting to wi-fi

Trying to connect photon to wifi, however the wizard seems to be getting stuck on this step:

Fixed by setting Particle on listening mode before starting particle serial wifi wizard.


Pick up system reserved threshold 500ms because of config missing

Please answer these questions before submitting your issue.

Why do you submit this issue?
[x] Question or discussion
[ ] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?
oap server keeps showing up this log Pick up system reserved threshold 500ms because of config missing

I use docker-oap-es7

Requirement or improvement

Please describe about your requirements or improvement suggestions.

As you can see, it's just a waring log message. You can ignore or config threshold  through dynamic config service in OAP.

but, server print it very fast, who to stop it?

How can I solve it?


Disable shutdownTimeLimit when the debugger is attached

Similar to how startupTimeLimit is ignored while the debugger is attached to enable debugging startup, shutdownTimeLimit should also be ignored to help debug shutdown.

@Tratcher What's the impact with the debugging experience?
cc @shirhatti

It's difficult to debug your shutdown code because the process gets terminated.

Same as #173


kubernetesVersion worked does not work with kind v0.11.1

kind.ctlptl.yaml:
apiVersion: ctlptl.dev/v1alpha1
kind: Cluster
product: kind
registry: ctlptl-registry
kubernetesVersion: "v1.21.1"
---
apiVersion: ctlptl.dev/v1alpha1
kind: Registry
name: ctlptl-registry

ctlptl apply -f ./k8s/kind.ctlptl.yaml
Creating registry "ctlptl-registry"...
registry.ctlptl.dev/ctlptl-registry created
creating cluster: No available kindest/node versions for kind version v0.11.1.
Please file an issue: https://github.com/tilt-dev/ctlptl/issues/new
make: *** [cluster-up] Error 1

It worked with kind 0.9.0. If I remove the version, kind starts just fine:
ctlptl apply -f ./k8s/kind.ctlptl.yaml
registry.ctlptl.dev/ctlptl-registry created
Creating cluster "kind" ...
 ✓ Ensuring node image (kindest/node:v1.21.1) 🖼
 ✓ Preparing nodes 📦
 ✓ Writing configuration 📜
 ✓ Starting control-plane 🕹
 ✓ Installing CNI 🔌
 ✓ Installing StorageClass 💾
Set kubectl context to "kind-kind"

Oh, I see new kind versions require the kindK8sNodeTable to be updated. I'll submit a PR shortly.

thanks for the fix! This should be fixed in v0.5.4