Service inject example is not correct for TypeScript

At https://angular.io/docs/js/latest/guide/displaying-data.html#Create-a-class, there is the example for inject the Service in the components.
ES5 example is:
//ES5
function FriendsService() {
  this.names = ["Alice", "Aarav", "Martín", "Shannon", "Ariana", "Kai"];
}
function DisplayComponent(friends) {
  this.myName = "Alice";
  this.names = friends.names;
}
DisplayComponent.annotations = [
  new angular.Component({
    selector: "display",
    injectables: [FriendsService]
  }),
  new angular.View({
    template: '{{ myName }}

{{ name }}

',
    directives: [angular.For, angular.If]
  })
];
DisplayComponent.parameters = [[FriendsService]];
document.addEventListener("DOMContentLoaded", function() {
  angular.bootstrap(DisplayComponent);
});

But TypeScript example is:
//TypeScript
import {Component, View, bootstrap, For} from
...
  directives: [For]

I think TypeScript example should be:
import {Component, View, bootstrap, For, If} from 'angular2/angular2';

class FriendsService {
  constructor() {
    this.names = ["Alice", "Aarav", "Martín", "Shannon", "Ariana", "Kai"];
  }
}

@Component({
  selector: 'display',
  injectables: [FriendsService]
})

@View({
  template: `

My name: {{ myName }}

Friends:

          {{ name }}

 3">You have many friends!

`,
  directives: [For, If]
})

class DisplayComponent {
  myName: string;
  name: Array;

  constructor(friends:FriendsService) {
      this.myName = "Alice";
      this.names = friends.names;
  }
}

bootstrap(DisplayComponent);

The Commit bdf28bc02645c963eb50cd34b2fe15e0910ff7e4 resolved that.

Thanks for following up on this issue.

:+1:

In the Typescript-version (bdf28bc), you changed the injectables: [FriendsService] to injectables: [DisplayComponent]. Is that right? Seems like a mismatch vs. the ES5 code.

The commit was made by @rkirov. But it seems that the DisplayComponent in Typescript doesn't have the myName property.


Fixed #30892 -- Fixed slugify() doesn't return a valid slug for "İ"

This eliminates the validation error when validating a string
containing an "İ" that has been run trough slugify with
allow_unicode=true.
Thanks Luis Nell for solution idea as well as a very detailed bug report.
I am new to writing tests in Django so any feedback about that or anything else would be appreciated.
See the issue for the discussion up until this point. https://code.djangoproject.com/ticket/30892

Thank you for fixing this so fast! Looking forward to the release this is included in, hopefully.

@Sjbrgsn Thanks for this patch :+1: I checked characters with unicode Case Mapping Collisions and
others "I" letters with diacritic marks (ÍíÌìĬĭÎîǏǐÏïḮḯĨĩĮįĪīỈỉȈȉȊȋỊịḬḭƗɨᵻᶖİiIı) and it seems that only İ was affected. I also removed redundant test from tests/model_fields/test_slugfield.py.

I know that we have a JS code that powers the ModelAdmin.prepopulated_fields feature for SlugField. Does this code require adjustments as well?

nvm, it looks we're already handling this character https://github.com/django/django/blob/cf5d4701dc12ad69d51042b0d7e81e4a54de4bd7/django/contrib/admin/static/admin/js/urlify.js#L33

@charettes Good catch :+1: It looks that it doesn't work properly:

@felixxm how do I reproduce that error? If I can reproduce it I might be able to fix that as well

@charettes @Sjbrgsn We use this map to remove Unicode character when they're not allowed, so we will not use it in this case. We should convert to a lowercase:
s = s.toLowerCase(); // convert to lowercase

before we remove all not Unicode characters:
if (allowUnicode) {
    // Keep Unicode letters including both lowercase and uppercase
    // characters, whitespace, and dash; remove other characters.
    s = XRegExp.replace(s, XRegExp('[^-_\\p{L}\\p{N}\\s]', 'g'), '');

so we should add the same fix to the urlify.js.
@Sjbrgsn You can reproduce this issue by creating a model with two fields models.SlugField(..., allow_unicode=True) and using prepopulated_fields = {"slug_1": ("slug_2",)} on an admin page.
I can push fix if you want because I have it ready :smile:

Ok I managed to reproduce it. Since you already have a solution it seems unnecessary for me to do the  work again so go ahead :)

I added fix and selenium tests.

buildbot, test on selenium.


optimise ReadBytes

Avoids integer expansion, bit shifting, and bit ORing operations. Instead simply reads into an aligned buffer, transmutes, then swaps bytes as necessary.
My original attempted solution was much neater, using a macro to auto-create the type and impl for each integer size, but rust macros aren't up to that yet :/
Feel free to tweak the patch as necessary

Ah, I see from the travis result that this repr attribute was not stable yet ~1.24
I had wondered why this solution had been overlooked, since the quality of code generally looks pretty good. A Patch for a later time?

Thank you for taking the time to open this pull request.
I benchmarked this change against master, and I found no strong evidence that it is faster:
	Welch Two Sample t-test

data:  before and after
t = 2.0356, df = 36359, p-value = 0.0418
alternative hypothesis: true difference in means is not equal to 0
95 percent confidence interval:
 0.0003841598 0.0203188116
sample estimates:
mean of x mean of y
 13.84424  13.83389

These are the mean times per sample in nanoseconds, measured over 18217 blocks (5 songs) that were each decoded 140 times, keeping the minimum time for each block to eliminate noise.
It might be that there is a speedup, and more measurements would unveil it, but from my measurements so far, the speedup would be a 0.07% decrease in decode time, which I don't think is worth the extra complexity.
This result makes sense to me, even if the generated code looks better: the functions you modified are cold, they are only used when reading file metadata and the frame headers, and the cost of this is negligible in comparison to reading the frames themselves.
Did you observe a speedup on your system?

Note that starting from Rust 1.32 this can be done safely via u16::from_le_bytes(). If you choose to proceed with this PR, please add a TODO to switch to that implementation once Rust version 1.32 can be assumed.


Emoticon Not Rendering as Expected?

So currently in my source I have

but it currently renders as

Can you share the string in this example?

&format!("生活,삶,जिंदगी 😀 FPS: {}", fps),
copy pasted this exactly how it is in the source.

I can not replicate this error. If you save this string into a file and open it in editor-libcosmic, it looks OK.

So i was debugging this and noticed something weird.
First off here is the code itself doing to offsets.
for run in buffer.layout_runs() {
            let line_y = run.line_y;

            for glyph in run.glyphs.iter() {
                let (allocation, is_color) = if let Some(allocation) =
                    text_atlas.atlas.get(&glyph.cache_key)
                {
                    (allocation, false)
                } else if let Some(allocation) =
                    emoji_atlas.atlas.get(&glyph.cache_key)
                {
                    (allocation, true)
                } else {
                    continue;
                };

                let (u, v, width, height) = allocation.rect();
                let (u, v, width, height) =
                    (u as i32, v as i32, width as f32, height as f32);

                let (x, y) = (
                    (pos[0] + glyph.x_int) as f32,
                    (pos[1] + line_y + glyph.y_int) as f32,
                );

                let (u1, v1) = (u as f32, v as f32);

                let color = if is_color {
                    Color::rgba(255, 255, 255, 255)
                } else {
                    match glyph.color_opt {
                        Some(color) => color,
                        None => self.color,
                    }
                };

                let default = TextVertex {
                    position: [x, y, pos[2] as f32],
                    hw: [width, height],
                    tex_coord: [u1, v1],
                    layer: allocation.layer as u32,
                    color: color.0,
                    is_color: is_color as u32,
                };

                text_buf.push(default);
            }
        }

Secondly When it gets too गी the width becomes 0

I would expect this to be the size of the last glyph? Then when moved to the next glyph which apparently is the Smiley face

I will see if maybe my version is just behind using cargo update and test again but its acting strange around that part.

this was fixed by adding the image.placement.left to the X position.


Russian Language not showing up in app

I can try fixing this. assign this to me.

assign to me.

assign to me

assign this to me


Add links to network info scripts

Add relative symlinks to networkinfo scripts on bin directory.
Signed-off-by: Daniel Finimundi<EMAIL_ADDRESS>

That's great, Daniel. Thank you.


Short variable names style guide

📖 Documentation
It's not obvious how all those xyz names are generated. Is ma Monad type of A?
A little guide would be helpful.

@steida can you provide a list of examples that are confusing? I can try to add them to https://github.com/gcanti/fp-ts/blob/master/docs/introduction/code-conventions.md in a new "variable naming conventions" section, although I believe there will be some exceptions around the codebase

To answer your question up to now

Is ma Monad type of A?

yes. If I'm not wrong the "rule" is in general fa if the value is a Functor (or similarly fab for Functor<(a: A) => B>), ma for any other typeclass, that typically will be indeed a Monad

And what is as?

if a: A is usually a generic parameter of type A, as is an Array

Yeah, but why as? Why not a?
export function init(as: Array): Option<Array> {
  const len = as.length
  return len === 0 ? none : some(as.slice(0, len - 1))
}

Btw, is there any guide when to write function declaration and when arrow function?

I think function is preferred and used for all (at least vast majority) of the exported functions. Also I think it looks nicer in the docs. const is sometimes used for internal functions and/or preferred for brevity. I'm not sure whether there are more meaningful conventions in the codebase other than these

I have this rule: "Use function declaration for all functions with name whenever possible. Otherwise, use arrow function."


Modify and create some forms

Modified:

PlantForm -> PlantCRUD
UserForm -> UserCRUD

Created:

Profile: to save detail information of user or admin.
UserForm and AdminForm: were created by basing on MainForm

Deleted: MainForm

All good! Keep it up


🐛 Specifying a relative icon path for a new BrowserWindow breaks Windows build with Electron 9.0.0

Preflight Checklist

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Issue Details

Electron Packager Version:

14.2.1

Electron Version:

12.16.1

Operating System:

win32 10.0.18363 (x64)

Last Known Working Electron Packager version::

14.2.1 (with Electron 8.3.0)

Expected Behavior
Packaged app should still be able to open browser windows with a relative icon path specified.
Actual Behavior
Packaged app can't open BrowserWindows with a relative path icon field specified. Instead of rendering the page, nothing is written to the DOM. This started today when I upgraded Electron to 9.0.0.
Workarounds:

Specify an absolute icon path (ex: __dirname + '/icon.png')
Remove the icon field

To Reproduce

Clone this reprository https://github.com/ChrisAntaki/electron-packager-bug-repro
Install dependencies with yarn
Test the app locally with yarn start, it should work fine
Package the app with yarn package, the packaged app should just show a white screen

It's not recommended to use relative paths in Electron apps, see the FAQ entry for more details.
That being said, this sounds like a regression in Electron itself, not Electron. Electron Packager has no control over the icon option of the BrowserWindow constructor.


Random chrashes

I have several "bugs"
First the most important one:
Your request to Open AI API failed: server_error
The server had an error while processing your request. Sorry about that!
This happens about every 10 th time.
Second. Image creation

I changed the line nr 50 from "open" to $BROWSER
even so my terminal can do imcat it does not start because it is not a iTerm.app (i guess). So the it would be nicer if possible to change the if statement

I noticed several server errors yesterday as well. I assume they are from OpenAI's servers working at capacity and failing to complete all the requests, because at the same time I couldn't log in to the ChatGPT site. Sadly, as long as the requests the script sends are properly formatted, with correct headers etc, which they are, there is nothing we can do about OpenAI's servers.
About the image generation, the command open works in MacOS by opening the image in your default browser. I am guessing it's not working properly in Linux. In that case you can use xdg-open as you can see @sourav-majumdar-math did here. You can also open the image with an image viewer, or a specific browser but whatever you choose it has to be included in your $PATH. Others have used different packages/solutions to show images in their terminal:
@sourav-majumdar-math used tiv code
@pfr-dev used Kitty terminal
   if [[ "$TERM" == "xterm-kitty" ]]; then
     curl -sS $image_url -o temp_image.png
       kitty +kitten icat temp_image.png

Let me know if that helps!

Excellent.
Thanks for the fast reply.
Yes I thought it might be an overload server. Last night I did use script for an hour and experienced no crashes
Browse is working and image in terminal also
Thanks


feat: add support for new resources

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Rename project to drupal-vm

I just registered the domain drupalvm.com, and am going to submit this VM into the fray as a decent option for local Drupal development, once I finish a couple other tweaks. So part of this process is renaming this project to 'drupal-vm'.
Anyone with an existing repo will need to update his or her remote, but that's a simple process like:
git remote rm origin
git remote add origin git@github.com:geerlingguy/drupal-vm.git

Actually, as long as I don't create another project named drupal-dev-vm, redirects should continue working indefinitely.
But still, if you want to update your repo, that's how you do it :)


[BUG] vulkan fails, CMake issue?

$ make
Consolidate compiler generated dependencies of target libfastfetch
[  1%] Building C object CMakeFiles/libfastfetch.dir/src/detection/vulkan/vulkan.c.o
/home/stefan/g/fastfetch/src/detection/vulkan/vulkan.c:21:29: error: unknown type name 'VkPhysicalDeviceDriverProperties'; did you mean 'VkPhysicalDeviceDriverPropertiesKHR'?
 static void applyDriverName(VkPhysicalDeviceDriverProperties* properties, FFstrbuf* result)
                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                             VkPhysicalDeviceDriverPropertiesKHR
/home/stefan/g/fastfetch/src/detection/vulkan/vulkan.c: In function 'detectVulkan':
/home/stefan/g/fastfetch/src/detection/vulkan/vulkan.c:130:9: error: unknown type name 'VkPhysicalDeviceDriverProperties'; did you mean 'VkPhysicalDeviceDriverPropertiesKHR'?
         VkPhysicalDeviceDriverProperties driverProperties = {
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
         VkPhysicalDeviceDriverPropertiesKHR
/home/stefan/g/fastfetch/src/detection/vulkan/vulkan.c:131:13: error: field name not in record or union initializer
             .sType = VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_DRIVER_PROPERTIES,
             ^
/home/stefan/g/fastfetch/src/detection/vulkan/vulkan.c:131:13: note: (near initialization for 'driverProperties')
/home/stefan/g/fastfetch/src/detection/vulkan/vulkan.c:131:22: error: 'VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_DRIVER_PROPERTIES' undeclared (first use in this function); did you mean 'VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_DRIVER_PROPERTIES_KHR'?
             .sType = VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_DRIVER_PROPERTIES,
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                      VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_DRIVER_PROPERTIES_KHR
/home/stefan/g/fastfetch/src/detection/vulkan/vulkan.c:131:22: note: each undeclared identifier is reported only once for each function it appears in
/home/stefan/g/fastfetch/src/detection/vulkan/vulkan.c:156:13: error: implicit declaration of function applyDriverName' [-Werror=implicit-function-declaration]
             applyDriverName(&driverProperties, &result->driver);
             ^~~~~~~~~~~~~~~
/home/stefan/g/fastfetch/src/detection/vulkan/vulkan.c:163:42: error: request for member 'conformanceVersion' in something not a structure or union
                 .major = driverProperties.conformanceVersion.major,
                                          ^
/home/stefan/g/fastfetch/src/detection/vulkan/vulkan.c:164:42: error: request for member 'conformanceVersion' in something not a structure or union
                 .minor = driverProperties.conformanceVersion.minor,
                                          ^
/home/stefan/g/fastfetch/src/detection/vulkan/vulkan.c:165:42: error: request for member 'conformanceVersion' in something not a structure or union
                 .patch = driverProperties.conformanceVersion.patch,
                                          ^
/home/stefan/g/fastfetch/src/detection/vulkan/vulkan.c:190:53: error: request for member 'driverInfo' in something not a structure or union
         ffStrbufInitS(&gpu->driver, driverProperties.driverInfo);
                                                     ^
cc1: some warnings being treated as errors
make[2]: *** [CMakeFiles/libfastfetch.dir/build.make:538: CMakeFiles/libfastfetch.dir/src/detection/vulkan/vulkan.c.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:87: CMakeFiles/libfastfetch.dir/all] Error 2
make: *** [Makefile:156: all] Error 2
$ uname -a
Linux Stefan-PC 4.4.0-19041-Microsoft #3996-Microsoft Thu Jan 18 16:36:00 PST 2024 x86_64 GNU/Linux

You may specify -DENABLE_VULKAN=OFF to disable it.

Please test if https://github.com/fastfetch-cli/fastfetch/commit/6274ef591b3882e25667a6f8190035195acb0401 resolve your issue.

Try again please

compiles, thx


non-linear steps

I was trying to figure out how to add this in a PR, but couldn't see where is best
If the min,max is 1 and 100,000,000 and I want to pick a range for 1 to 100, it is hard.
if the steps are not linear or exponential, smaller selected values get a finer resolution than the higher values
similarly, might be also good to input an array of not even steps like [1,12,100,400,1000,10000,123545, 100000] for possible options to choose instead of linear steps or step by 1

I do understand times when this would be needed, but for now I dont think I'll be implementing it as it would add too much complexity to a control that I'm trying to keep fairly simple to implement and understand.
Sorry!


fix: Update xpu manager release

PR Checklist

[x] Added label to the Pull Request for easier discoverability and search
[x] Commit Message meets guidelines as indicated in the URL https://github.com/intel-retail/automated-self-checkout/blob/main/CONTRIBUTING.md
[x] Every commit is a single defect fix and does not mix feature addition or changes
[ ] Unit Tests have been added for new changes
[ ] Updated Documentation as relevant to the changes
[x] All commented code has been removed
[x] If you've added a dependency, you've ensured license is compatible with repository license and clearly outlined the added dependency.
[ ] PR change contains code related to security
[ ] PR introduces changes that breaks compatibility with other modules (If YES, please provide details below)

What are you changing?

Updating XPU manager as a result of the previous version being removed from github
Issue this PR will close
close: #394
Anything the reviewer should know when reviewing this PR?
Test Instructions if applicable

cd benchmark-scripts
make build-xpu or make build-all
See failure

If the there are associated PRs in other repositories, please link them here (i.e. intel-retail/automated-self-checkout )

why this is a breaking change?

Technically it's a fixing change since the current version is broken but I see your point. I'll remove that tag since it's not really the intent.


[sct_extract_metric] simulate registration of the template to the subject space

Template is modified when registering to the template space (interpolation) --> Partial volume might be affected. Does it affect the extraction of metrics?
I would try two things:
1- compare extracted metrics in the template space VS in the subject space.
2- Create a synthetic phantom as in LEvy 2015 but not registered with the template with a different shape (e.g. larger spinal cord.. then smaller spinal cord). Test the extraction of the metrics in both case (large and small spinal cord) and do a pairwise ttest (overestimation or underestimation?) compared to the Ground Truth.
Post the results in this thread!

Please Simon, send me the link to the existing synthetic phantom!

You can start from the script validate_atlas.py available under dev/atlas/validate_atlas.


feat(curriculum): add all english block 1 animations

This adds the videos to all of the block 1.1 challenges. I added videos in many places where there wasn't an audio reference in the markdown - they seemed to fit. When reviewing, we should make sure that we have the videos where we want them, and not where we don't.
It makes some slight tweaks in a couple other files, as well.
To test this:
-Download the english assets here: https://github.com/moT01/english-assets
-Serve the assets folder on port 8080 - I run http-server while in the english-assets folder
-Let me know if you need help with that
Edit: Alternatively, change the domain variable in scene-assets.tsx to const domain = 'https://cdn.freecodecamp.org/curriculum/english/animation-assets';
Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Thanks for the reviews 🎉


[taglib] build failure

Host Environment

OS: Windows 10 Pro 21H2 x64
Compiler: Visual Studio 2022, MSVC v143

To Reproduce
Steps to reproduce the behavior:
./vcpkg install taglib --triplet=x64-windows
Failure logs
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:158 (message):
    Command failed: "C:/Program Files/CMake/bin/cmake.exe" --build . --config Debug --target install -- -v -j5
    Working Directory: C:/Users/cstob/Documents/rsgain/vcpkg/buildtrees/taglib/x64-windows-dbg
    See logs for more information:
      C:\Users\cstob\Documents\rsgain\vcpkg\buildtrees\taglib\install-x64-windows-dbg-out.log
      C:\Users\cstob\Documents\rsgain\vcpkg\buildtrees\taglib\install-x64-windows-dbg-err.log

Call Stack (most recent call first):
  installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_build.cmake:112 (vcpkg_execute_build_process)
  installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_install.cmake:41 (vcpkg_cmake_build)
  ports/taglib/portfile.cmake:19 (vcpkg_cmake_install)
  scripts/ports.cmake:145 (include)

error: building taglib:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `git pull` and `.\vcpkg update`.
Then check for known issues at:
  https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+taglib
You can submit a new issue at:
  https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[taglib]+Build+error
including:
  package: taglib[core]:x64-windows -> 1.12.0#2
    vcpkg-tool version: 2022-03-30-692785ac944e81417840c6de244fb3e18a4b35eb
    vcpkg-scripts version: a5d6d1451 2022-05-06 (2 days ago)

Additional context
I built taglib a couple weeks ago with no issue. I'm trying to build again today on a different PC and I get the build failure as shown above. I did some investigation and there haven't been any recent commits on the taglib port, but there have been several for its dependency vcpkg-cmake. This one in particular looks suspicious as it changed the vcpkg_execute_build_process function which is the most recent failure in the stack trace. Not sure how to confirm this is causing the issue.
@Neumann-A
@BillyONeal
Workaround
For a workaround, I started a new vcpkg root based on the most recent git tag, 2022.04.12.
git clone https://github.com/microsoft/vcpkg.git
cd vcpkg
git checkout tags/2022.04.12

Then bootstrap and run the vcpkg install command, and it successfully builds.

Logs? Be aware that all those changes are tested by CI.
It also helps to compare CMake configure lines from the Logs before and after that change.
For me it works as expected from a clean vcpkg.
Starting package 3/3: taglib:x64-windows
Building package taglib[core]:x64-windows...
-- Using cached taglib-taglib-v1.12.tar.gz.
-- Extracting source G:/vcpkg_common/downloads/taglib-taglib-v1.12.tar.gz
-- Applying patch msvc-disable-deprecated-warnings.patch
-- Using source at E:/vcpkg_folders/master2/buildtrees/taglib/src/v1.12-8b13cd6052.clean
CMake Warning at installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_configure.cmake:115 (message):
  PREFER_NINJA has been deprecated in vcpkg_cmake_configure.  Please remove
  it from the portfile!
Call Stack (most recent call first):
  ports/taglib/portfile.cmake:14 (vcpkg_cmake_configure)
  scripts/ports.cmake:145 (include)
-- Found external ninja('1.10.2').
-- Configuring x64-windows
-- Building x64-windows-dbg
-- Building x64-windows-rel
-- Fixing pkgconfig file: E:/vcpkg_folders/master2/packages/taglib_x64-windows/lib/pkgconfig/taglib.pc
-- Fixing pkgconfig file: E:/vcpkg_folders/master2/packages/taglib_x64-windows/lib/pkgconfig/taglib_c.pc
-- Using cached msys-mingw-w64-i686-pkg-config-0.29.2-3-any.pkg.tar.zst.
-- Using cached msys-mingw-w64-i686-libwinpthread-git-<IP_ADDRESS>73.5be8fcd83-1-any.pkg.tar.zst.
-- Using msys root at G:/vcpkg_common/downloads/tools/msys2/9a1ec3f33446b195
-- Fixing pkgconfig file: E:/vcpkg_folders/master2/packages/taglib_x64-windows/debug/lib/pkgconfig/taglib.pc
-- Fixing pkgconfig file: E:/vcpkg_folders/master2/packages/taglib_x64-windows/debug/lib/pkgconfig/taglib_c.pc
-- Performing post-build validation
-- Performing post-build validation done
Stored binary cache: G:\vcpkg_common\binary\e1\e119d2f7c113b5d245085e17db517d4b929660506bff78a93e097791016aac05.zip
Installing package taglib[core]:x64-windows...
Elapsed time for package taglib:x64-windows: 1.231 min

Sorry, I didn't reproduce your problem locally, can you provide your error log? Could you please use the latest vcpkg and try again?
      C:\Users\cstob\Documents\rsgain\vcpkg\buildtrees\taglib\install-x64-windows-dbg-out.log
      C:\Users\cstob\Documents\rsgain\vcpkg\buildtrees\taglib\install-x64-windows-dbg-err.log

Try steps:
1. git clone https://github.com/microsoft/vcpkg
2. cd .\vcpkg\
3. .\bootstrap-vcpkg.bat
4. .\vcpkg.exe install taglib --triplet=x64-windows

  C:\Users\cstob\Documents\rsgain\vcpkg\buildtrees\taglib\install-x64-windows-dbg-out.log

also the config-x64-windows-dbg-out.log to compare how cmake was configured.

I looked at the logs, the relevant error message was:
ninja: error: manifest 'build.ninja' still dirty after 100 tries

After some research I came across this comment. It's an issue related to the system clock being out of sync. Apparently, Ninja has a problem with this for some reason.
I dual boot with Linux which uses UTC instead of local time for the system clock, and I also recently traveled to a different timezone but didn't update the settings. Not sure which of those was the culprit, but I went into the Windows settings updated the timezone and synced the time and now it builds fine. Not sure why it built successfully with the git tag 2022.04.12 with the clock out of sync. Perhaps some change to the Ninja config?
@Neumann-A I apologize for falsely assuming that the previously mentioned commit was causing this issue and tagging you here.

It's an issue related to the system clock being out of sync.  Apparently, Ninja has a problem with this for some reason.

Ninja is designed for speed and simplicity. It is "relying on simple dependencies between file timestamps" i.e. it expects that generated files' timestamps are newer than their sources' timestamps.


Complete rework of the codebase + added some new features

Moved whole codebase into isolated scope (other from global)
Moved all the functionality to the classes to logically group the code
Added friendly user interface to interact with the new API
Added dynamic cell titles (e.g. 12:23, čas na dôkaz)
Added better cookies manipulation
Added more scalable and memory efficient way to store data in LocalStorage + added security checks for potential invalid data
Added ability to change the play area size (dynamic play field generation)
Made use of HTML templates to prevent heavy code repetition

Wow, this is something. Good job man.

Kolego, doporučuji zhlédnout: https://www.youtube.com/watch?v=-AQfQFcXac8

Wow, this is something. Good job man.

Nie, nie je to dobrá práca.


/var/log/microsoft_mdatp_.log should be /var/log/microsoft/mdatp/.log

RE: https://docs.microsoft.com/en-us/microsoft-365/security/defender-endpoint/linux-privacy?view=o365-worldwide
This:
Product installation and uninstallation logs under /var/log/microsoft_mdatp_*.log
Should be:
Product installation and uninstallation logs under /var/log/microsoft/mdatp/*.log
Thx,
Yong

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

ID: b6e7ec33-6824-41e4-7a9c-56a46598955f
Version Independent ID: 643d0a7f-d84b-10e7-455d-d79b4da50dba
Content: Privacy for Microsoft Defender for Endpoint on Linux
Content Source: microsoft-365/security/defender-endpoint/linux-privacy.md
Product: m365-security
Technology: mde
GitHub Login: @Dansimp
Microsoft Alias: dansimp

@msbemba Thank you very much for the contribution and updating the documentation with PR. @YongRhee-MSFT Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Referendum low med high impact settings #215

Added referendums consider the setting's impact to determine the unity and quorum thresholds
Modified tests

Deploy referendums, settings, update settings


Enquiry about the NiftyMIC commands

Hi, I have included two reconstructed images here from our highly motion corrupted 2D T2-HASTE slices. I compared the images with BTK generated ones. I reconstructed those outputs in two steps:

niftymic_reconstruct_volume
--filenames path-to-stack1.nii.gz ... path-to-stackN.nii.gz
--filenames-masks path-to-stack1_mask.nii.gz ... path-to-stackN_mask.nii.gz
--output path-to-srr.nii.gz \

niftymic_reconstruct_volume_from_slices
--filenames path-to-stack1.nii.gz ... path-to-stackN.nii.gz
--dir-input-mc dir-to-motion_correction
--output path-to-srr.nii.gz
--reconstruction-type HuberL2
--alpha 0.003
I used the brain+badslice mask for the first step.

For both of the data, by qualitative observation the reconstruction is not optimal. Do you have any suggestion or explanation for that? We actually intend to publish our study soon.

Hi,
Difficult to tell without more information on the input data. To me, it seems that the default assumption of slice thickness equals pixel spacing is incorrect. Do you acquire your stacks with overlap? If that's the case, please provide the slice thickness using --slice-thicknesses.
Best,
Michael

@sahas111, has this solved the issue for you?


Craft CMS 5.* Update

This plugin blocks me from updating to Craft CMS 5.0.
Are any updates planned?

Hi @JohnathonJellyPcs
Thank you for your feedback. Yes, I plan to update the plugin in the next two weeks.


[Functional, Like a Comment] System doesn't recalculate total likes number in real time

Environment:

macOS Catalina 10.15.6
Google Chrome Version 87.0.4280.88 (Official Build) (x86_64) + Safari Version 13.1.2 (156<IP_ADDRESS>.3)
Windows 10 Pro version 2004 (OS Build 19041.630), Firefox 83.0 (x64) + Opera (v. 72.0.3815.400)

Reproducible: always
Build found: 17/12/20
Preconditions

Go to https://ita-social-projects.github.io/GreenCityClient/#/welcome in one browser (e.g. Google Chrome)
Sign in with valid data as User.
Go to https://ita-social-projects.github.io/GreenCityClient/#/welcome in another browser (e.g. Safari)
Sign in with valid data as another User.
Open to browsers to observe actions in real time on both.

Steps to reproduce

Click on 'Eco News' tab in one browser. There is at least one News with comment.
Like not your comment.

Actual result
System doesn't recalculate total likes number in real time in another browser
(See video: https://drive.google.com/file/d/1A3A4hv6kgzLq1qthtr2ZxjtSImggIF6I/view?usp=sharing)
Expected result
System should recalculate total likes number in real time on both browsers.
User story and test case links
User story #766
Labels to be added
"Bug", Priority ("pri: "), Severity ("severity:"), Type ("UI, "Functional")

The bug is still reproduce on environments:
macOS Big Sur 11.4
Chrome 91.0.4472.77
Safari 14.1.1 (166<IP_ADDRESS>.4)


Credentails exposed

https://github.com/sona3Admin/sona3-dev/blob/8c71b1e25a2fa28fb11eb6a3c17d224ed7aef237/.env#L1-L4
This would be the 3rd time in three days. Your Bucketeer account has been suspended at the moment - please open a support ticket so we can get an acknolwedgement that they keys cannot be public or you might have your data deleted or account otherwise compromised. Apologies for the inconvenience.

Sorry, I will remove the credentials from the .env file and redeploy the app,
but if I put the credentials in the config var section in settings tab, would the credentials still be public in this case too?


Bug: Conflicting peer dependency when upgrading to Vite 5

issue encountered while upgrading a Vite 4 project to Vite 5. When running npm i, the installation fails due to a conflicting peer dependency on<EMAIL_ADDRESS>enforce: "pre" should be moved outside the tranfromIndexHtml hook also using transfrom function is deprecated. now you can access the HTML directly from the hook: https://vitejs.dev/guide/api-plugin.html#transformindexhtml

Hi @hamedbaatour, Thanks for your feedback, vite@5 is now supported in<EMAIL_ADDRESS>


remove Assertions from chai.d.ts

this interface is insufficiently typed and leads to implicit-any
errors. I propose removing this entirely as it actually types the
jquery-chai plugin, not the core chai library. If anyone needs it
in the future, a jquery-chai.d.ts with the correct typings can be
created.

close the PR because this is too old.


Add Photo to About Me Page

except a photo my about me page is compatible with template, add a photo as well

It's a small detail but your image should be between your name and the first introduction about you


AccessDeniedException: User is not authorized to access the web ACL

Looks like there was some error in the IAM policy example. Whenever, I set a WebACL annotation on one of the ingresses, I get the following errors.
AccessDeniedException: User is not authorized to access the web ACL

The issue is that policy doesn't have access to the waf-regional resources.
ALB Ingress Controller Version: on both v1.0-beta4 and v1.0-beta5.
Issue got resolved after making some changes to the policy. Will create a PR for the same.

@kumudt  I think the example policy had it already:
https://github.com/kubernetes-sigs/aws-alb-ingress-controller/blob/master/examples/iam-policy.json#L70
https://github.com/kubernetes-sigs/aws-alb-ingress-controller/blob/master/examples/iam-policy.json#L81

@anhpham1509 It needs
"waf-regional:GetWebACLForResource",
"waf-regional:GetWebACL",
"waf-regional:AssociateWebACL",
"waf-regional:DisassociateWebACL",
"waf:GetWebACL"

These are invalid IAM Policy Rules
"waf:AssociateWebACL",
"waf:DisassociateWebACL"


feat: add force change option

Details

Added an option for force change

Screenshots

Choose one of these(put a 'x' in the bracket):

[x] The change doesn't require a change to the documentation.
[ ] The documentation is updated accordingly.

Implementation note, gotchas and Future TODOs

Changed label text to the following


Make TravisCI notify Slack

piping everything into #cap channel

Current coverage is 89.53%

Merging #105 into master will not affect coverage as of 01f74de

@@            master    #105   diff @@
======================================
  Files           60      60
  Stmts         3918    3918
  Branches         0       0
  Methods        227     227
======================================
  Hit           3508    3508
  Partial          0       0
  Missed         410     410

Review entire Coverage Diff as of 01f74de

Powered by Codecov. Updated on successful CI builds.

It did work


UnboundLocalError: local variable 'error_rate' referenced before assignment

Hello Developers,
I am following the short tutorial from here for ancestry classification. It was working fine when I had earlier versions of the gnomad and hail. I did a fresh install yetsreday with updated versions and now I have the following error.
It appears to be a bug.
ht, rf_model = assign_population_pcs(
...     ht,
...     pc_cols=ht.scores,
...     fit=fit,
... )

2022-09-29 14:55:46 Hail: INFO: Coerced sorted dataset              (0 + 1) / 1]
INFO (gnomad.sample_qc.ancestry 224): Found the following sample count after population assignment: sas: 1
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/bioinfoRD/ARCdata/Projects_AMT/conda_envs/hail/lib/python3.10/site-packages/gnomad/sample_qc/ancestry.py", line 235, in assign_population_pcs
    min_assignment_prob=min_prob, error_rate=error_rate
UnboundLocalError: local variable 'error_rate' referenced before assignment

Edit:
Just as debugging, I added error_rate: float  = 0.0  at line 163
The above error was gone but the new error popped up:
File "/bioinfoRD/ARCdata/Projects_AMT/conda_envs/hail/lib/python3.10/site-packages/gnomad/sample_qc/ancestry.py", line 241, in assign_population_pcs
    evaluation_sample=hl.literal(list(evaluate_fit.s)).contains(pops_ht.s),
UnboundLocalError: local variable 'evaluate_fit' referenced before assignment

Then I ended up commenting out the annotate function at line 250 and it worked.
But this needs to be done properly think.

Hi,
I also just received this error following the gnomad ancestry inference vignette. I am using the terra-jupyter-hail docker us.gcr.io/broad-dsp-gcr-public/terra-jupyter-hail:1.0.20. The release notes mentions updating hail to 0.2.98.

Thank you @PoisonAlien and @sabrinacamp2 It looks like this error was fixed with in this commit, so this should be resolved in the newest gnomad_methods release. However, as described in this other issue. The code in the blog post won't work as is.
I have put in a PR to modify the new code so the blog post example can work.
If you try this again with the newest version of gnomad_methods (v0.6.3), you would need to modify the code in the following way:
This:
htt, rf_model = assign_population_pcs(
    ht,
    pc_cols=ht.scores,
    fit=fit,
)

should change to:
htt, rf_model = assign_population_pcs(
    ht,
    pc_cols=range(num_pcs),
    fit=fit,
)

This should be fixed in the next release so the original blog post code can work as expected.
Thank you,
Julia

The newest release v0.6.4 has fixed the assign_population_pcs function so that it still works with the code in the blog post. Closing the issue since I think this is now fixed, but please reach out if it's still a problem.


Tests need to be rewritten/randomized in Project Euler: Problem 69: Totient maximum

Describe your problem and how to reproduce it:
I can just pass the challenge by typing
function totientMaximum() {
  // Good luck!
  return 510510;
}

Add a Link to the page with the problem:
https://www.freecodecamp.org/learn/coding-interview-prep/project-euler/problem-69-totient-maximum

I agree with this issue, but am wondering if we should open a todo list for the Euler Project challenges instead?

Maybe considering this are in the Interview Prep and not in the curriculum we can just chuck it at people wanting to practice and do not go crazy in all this "if I write the expected output it passes", because I think this is pretty common in the Euler or even Rosetta challenges

I pointed this out before, but this is something that is difficult to test for and I'm not sure it's worth trying to stop. People in the interview prep section are there for a reason. If you try to regex a stop to return 510510, or maybe just check for 510510, people can still use return 510511-1 or something like that. It's just too easy to get around I think.
What about a check against the length of their code? You really can't pass any of these without a decent size chunk of code. What about making sure that it's >100 characters or something?
What about isolating the return ... part of the code, and checking that the ... is not a number or an equation or something?

@moT01 You are right; I just wanted to get a quick-fix out there.

What about a check against the length of their code? You really can't pass any of these without a decent size chunk of code. What about making sure that it's >100 characters or something?

The issue with this is, unless the test text does not give away the fact that it is looking at code size, because I could add any amount of garbage code to the end of my script. However, this in combination with another test might be beneficial.

What about isolating the return ... part of the code, and checking that the ... is not a number or an equation or something?

This probably will not work: 1) Many of my return statements are equations of some-kind; 2) Passing the variable that is the solution is just as easy.
Perhaps you are right -  we should just leave them as is, and let the satisfaction of legitimately passing be the tests.

I don't really like this, but what about removing the number from the tests?
Instead of should return 510510, it says should return the correct number. I don't like it because I use that number to try and figure out how write an algorithm sometimes. It doesn't really stop the direct return of a result, but users wouldn't know what the number is supposed to be anyway.

You know, I've been thinking, and now I don't think we need to touch any problem (in this section) to check if the user is really completing it.
Since this section is not backed by an fCC Certificate anyway, if the users just use workarounds, they're cheating themselves.
Thoughts @QuincyLarson ?

Thanks for reporting this @thecodingaviator. This is definitely something we want to improve over time.
I do agree that anyone who cheats is just cheating themselves, and a lot of the solutions are freely available online. Still, we want to avoid letting people return a single number and passing a challenge if possible.
One thing we've done is adapt the functions to take arguments, adjust the description, and add more tests. This one would be pretty easy to adapt because there's a table with the answers in the description.
We could change the last sentence of the description to "Find the value of n ≤ limit for which n/φ(n) is a maximum." Then we could add or adjust the tests so we're checking for totientMaximum(10) and totientMaximum(1000000), which should return 6 and 510510, respectively.

@scissorsneedfoodtoo , Yes, that sounds like a good plan.
In terms of the solution, are we wanting a solution that is as concise/performative as possible so campers can see how to improve? Or, are we wanting a solution that is simple, so campers can refer to workflow, if/when they get stuck?

@Sky020, awesome, thank you for all of your help with this.
We have a good mix of those two types of solutions spread across the coding interview prep section. Any solution is perfectly fine as long as it can pass all the test cases before the timeout. We'll add them to the Guide posts on the forum as we go and readjust as necessary if other solutions are submitted to the challenges here or on the Guide posts themselves.

Thanks for reporting this @thecodingaviator. This is definitely something we want to improve over time.
I do agree that anyone who cheats is just cheating themselves, and a lot of the solutions are freely available online. Still, we want to avoid letting people return a single number and passing a challenge if possible.
One thing we've done is adapt the functions to take arguments, adjust the description, and add more tests. This one would be pretty easy to adapt because there's a table with the answers in the description.
We could change the last sentence of the description to "Find the value of n ≤ limit for which n/φ(n) is a maximum." Then we could add or adjust the tests so we're checking for totientMaximum(10) and totientMaximum(1000000), which should return 6 and 510510, respectively.
Also we could add a solution like
function totientMaximum(limit) {
  let result = 1;
  const primes = [];
  let i = 0;

  const isPrime = num => {
    for (let i = 2; i <= Math.sqrt(num); i++) {
      if (num % i === 0) return false;
    }
    return num !== 0 && num !== 1;
  }

  for (let i = 2; i < 200; i++) {
    if (isPrime(i)) primes.push(i);
  }

  while(result * primes[i] < limit) {
    result *= primes[i];
    i++;
  }

  return result;
}

But that's almost directly from this blog, so if there's a more concise original solution, we should go with that.
What do you think @thecodingaviator and @Sky020?

Sounds awesome!

I think we are safe to close this in favor of the issue linked above. Thanks for all the input on this everyone. If anyone thinks I am wrong in closing this, feel free to leave a comment and we can reopen it. Thanks and happy coding 🎉


🔧 refactor: renamed folders (#686)

1. Description

1.1. Checklist

[ ] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[ ] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[ ] I have updated/added tests for ALL new/updated/fixed functionality.
[ ] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[ ] I have updated/added relevant examples in examples.

1.2. Breaking Change

[ ] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

1.3. Related Issues

Codecov Report

Merging #689 (96db841) into main (4b483e7) will increase coverage by 1.88%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #689      +/-   ##
==========================================
+ Coverage   80.38%   82.27%   +1.88%
==========================================
  Files         224        5     -219
  Lines        2723       79    -2644
==========================================
- Hits         2189       65    -2124
+ Misses        534       14     -520

see 219 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


List holochain ref+date used by each project, based on default.nix in each project

Strategy:

cloning the repos and searching locally

Fixed in https://github.com/holochain-ci/matrix/pull/20


Bert

WIP BERT training and eval

@Aceticia training script completed and tested.

@Aceticia Ready for review. Please take a look. Thank you!

@Aceticia Ready for merge


Tuesday 18-10-2023 roll call

Roll Call!
Leave a comment with these two things:

A summary of your study using only emojis
Something you'd like to share, anything goes! (within respect)

emoji cheat sheet

Good morning 🐈

Good morning 🌞

present

Good morning!

Good morning

Good Morning

Good morning...

hi

hello

present

present

🤝

Good morning 🌞

hello
:innocent:


Show build time in about page

It would be nice for the "about" page to show build time next to the version.
This would be done by storing the current date to a file within the container in the app's Dockerfile build script and reading that file in.

Implemented by #118


Fail to pull foreign layers when not supported

- What I did
Provide a better error message when a Unix daemon attempts to pull a foreign layer.
fixes #28892
- How I did it
Check if the layer being pulled is foreign, and fail on Unix daemons if so.
- How to verify it
Added test to verify.
Signed-off-by: Darren Stahl<EMAIL_ADDRESS>/cc @jhowardmsft

/cc @stevvooe @aaronlehmann

Hmmm. I would have though a better fix would be to do what we currently do on Windows introduced in https://github.com/docker/docker/pull/24819/files, although with the caveat of this comment: https://github.com/docker/docker/blob/master/distribution/pull_v2.go#L549-L556 whereby there would be a slight perf degredation on pull in general.

I agree with @jhowardmsft, couldn't the approach in #24819 be generalized? Ideally (and I don't know whether that info is there), it would also check CPU architecture etc.

LGTM
I think this is the right approach. If you somehow already have the foreign layer, the pull shouldn't be blocked. If we inspected the manifest for foreign layers instead, we would have to be stricter. But I don't have particularly strong feelings. If people think it's cleaner to look at the descriptors in the manifest, that's also an option, and wouldn't hurt performance because the manifest is always downloaded before anything else.

@friism @jhowardmsft IIRC there was concerns about the perf impact on Linux of pulling the metadata before the layer rather than doing so in parallel. I'm fine to do it that way instead, but I avoided doing so due to the perf hit.

This looks okay but I am worried if it is too narrow.
Typically, the mediaType is only used to enforce the policy on push. pull is gated by the existence of urls/availability. If an unavailable layer is causing retries, we may have another bug.

@stevvooe I changed it to check for existence of URLs instead of relying on the media type. My understanding is that there should be no URLs unless the layer is foreign. Is that correct?

@stevvooe I changed it to check for existence of URLs instead of relying on the media type. My understanding is that there should be no URLs unless the layer is foreign. Is that correct?

It is "correct" but too aggressive. It will make the upgrade path for foreign layers more complex than necessary.
The right fix is to error out in a timely fashion when the layer is not available remotely. From #28892, the "unknown blob" error should be terminal. No retries should be attempted.

@stevvooe See https://github.com/docker/docker/pull/28954 for making "unknown blob" a terminal error during the layer download.

@stevvooe See #28954 for making "unknown blob" a terminal error during the layer download.

Thanks!
Does that solve the usability problem from your perspective?

@stevvooe No, I still think that "unknown blob" is still a poor error message to convey that the platform is incompatible with the image being downloaded. I'm going to open a new PR with an alternative approach.

I've opened https://github.com/docker/docker/pull/29001 as an alternative solution to this problem. It blocks Windows images rather than those with foreign layers.

@darrenstahlmsft
Can we close this PR?
If we concluded to adopt #29001 or other idea else, we can reopen this PR again in that time.

Yes, I'll close this PR, as it seems like the other one is closer to an agreeable solution.


Add CPOs for for_loop algorithms

Can go in 1.5.0 or 1.6.0. This should be fairly straightforward (i.e. no big surprises with builds and tests), but let's see.

I like the doc fixes. @hkaiser Shouldn't the doxygen comments regarding the no-exec-policy overloads be added in the other algorithms that have already been adapted, too? Plus, should we link this to #4822?

@gonidelis does this address your comment about other algorithms needing updating: https://github.com/STEllAR-GROUP/hpx/pull/4908?

@msimberg  Yeah. That's great! Plus, we need to fix these doxygen comments that concern the overloads that do not take an exec-policy argument like you did on this PR.

Do you plan to still add range-based versions here? We can add those in a separate PR as well.

Do you plan to still add range-based versions here? We can add those in a separate PR as well.

I was not planning on doing that here, but can definitely do it in a separate PR. I suppose there's no document for those, but they would just replace first and last with range?

I suppose there's no document for those, but they would just replace first and last with range?

Yes, that's what we've done for the other algorithms before C++20. This also might entail adding sentinel support to the machinery. But even that can be done separately.

@hkaiser anything against this still going in 1.5.0?

@hkaiser anything against this still going in 1.5.0?

No objections here. Please go ahead.


Issue112 - Support Sage Pay Form

Support Sage Pay Form
A little refactoring included to help support this payment method with a little less code duplication.
Also includes issue #114 so that the [mandatory] delivery address can be set to copy the billing address automatically.
TOO: More parameters to be supported, along with tests, so this PR is mot final yet, but it will remain stable, so is here for visibility if you would like to give it a go.

I think this is ready to merge now...


User interface

User interface of cratesfyi
Web based user interface. It is under heavy development and this branch is
usually live at: https://beta.docs.rs/ build is in progress, search and most starred crates may not work.
Database is frozen and cratesfyi is not building any more crates during development.
Development
You can download subset of database dump (~270MB uncompressed) and help development of user interface without the need of containers (only user interface will work). You can run cratesfyi's web interface with:
# first grab database dump
wget http://cratesfyi.mooo.com/cratesfyi-web-subset-20160722.sql.gz
# restore database dump (you need to create a database and user first)
gzip -dc cratesfyi-web-subset-20160722.sql.gz | psql cratesfyi -U cratesfyi
# get cratesfyi-prefix tarball crated for only web interface
wget http://cratesfyi.mooo.com/cratesfyi-web-prefix-20160722.tar.gz
# extract cratesfyi-prefix
tar xzfv cratesfyi-web-prefix-20160722.tar.gz

# enter project directory, clone repository and checkout web branch
cd cratesfyi
git clone https://github.com/onur/cratesfyi.git cratesfyi
cd cratesfyi
git checkout web

# you also need to compile style.scss with scss
# make sure style.css link is correct in cratesfyi-prefix/public_html directory
scss templates/style.scss > templates/style.css

# compile and start web interface
CRATESFYI_PREFIX=../cratesfyi-prefix CRATESFYI_DATABASE_URL=postgresql://cratesfyi@localhost cargo run -- start-web-server

TODO

[x] Proper error handling among handlers (this is partially done, idea is to get rid of all unwraps).
[x] Proper 404 page.
[ ] Releases:

[x] Recent releases
[ ] Pagination in search results
[x] Bring crate if I am feeling lucky button pressed in home page
[x] Proper search result page when we can't find any create (currently returning 404)
[x] Releases by authors
[x] Most starred crates (github)
Author list and search (maybe in future)
[x] Release activity with highcharts (similar to metacpan).

[x] Crate details:

[x] Need to show more data about crate, such as:

[x] Homepage, original rustdoc link (original documentation link is not saved and not available in database :/)
License (looks like I forgot to save license fields)
[x] Github fields (stars, forks, issue count, last commit date)

[x] Proper padding for crate README.

Dependency chain with graphviz. (maybe in future)

[ ] Source browser: /source/{crate}/{version}

[x] highlightjs based syntax highlighting for rust source.
[ ] Show relative path of the file when user is browsing a file.

[x] Proper colorization of build icon in builds tab.
[x] Add favicon.
[x] Add Rust menu to navigation to point users to official Rust channels.

src/web/releases.rs:        // TODO: Currently, there is no way to know we are on the last page.
src/web/mod.rs:        // TODO: Use DocBuilderOptions for paths
src/web/mod.rs:        // TODO: Add a custom 404 page
src/web/crate_details.rs:// TODO: Add target name and versions

Issues

[x] Wrong grid size calculation for sm grid in builds.hbs
[x] Padding of search input in navigation.
[x] Link colors in crate README.

src/web/releases.rs:                // FIXME: There is no pagination
src/web/builds.rs:    // FIXME: getting builds.output may cause performance issues when release have tons of builds
src/web/mod.rs:        // FIXME: URGENT use correct path
src/web/mod.rs:        // FIXME: Need to sort versions with semver, database is not keeping them sorted
src/web/mod.rs:        // FIXME: This test is doing nothing
src/web/rustdoc.rs:    // FIXME: This is a bit inefficient but allowing us to use less code in general

Fixes: #1, #4
Successor of: #10

This is so cool!

This amazing work @onur, thanks a ton for the effort you put into this, it'll greatly help the Rust community :)

I am glad you liked it guys. It will be more awesome when I add stuff in TODO.

This looks fantastic!
And thank you for choosing handlebars(-iron). I'm the author of handlebars-iron, feel free to ping me (here or in handlebars/iron repo) if there is any issue.

Are you interested in PRs against the web branch with fixes for the TODOs you list above? Or would you like to finish building all the basics yourself first?

Also, it'd be really useful if you could release a database dump that is a subset of the full dataset. Access to all the crates isn't needed to build the UI :)

Thanks for your interest @jonhoo
Sure you can send PR's. A subset of database makes sense, I will prepare one.

I just prepared subset of database (~270 MB I believe this is best I can do) and a simple cratesfyi-prefix directory for this branch.

Great, thank you!
Is there a particular reason why you don't just include cratesfyi-prefix in the main repository?

The reason is there is only style.css in my cratesfyi-prefix/public_html directory (along with LXC containers and crates.io-index repository). Rest of the files are automatically stored and served from database (mini dump didn't have them). And cratesfyi-prefix must be outside of the project directory, cargo is looking to find crates in container. I will try to automate compilation of style.css.

Unless I'm mistaken, it should be fine to include a .scss file inside src/ (or inside some subdirectory of src/), no? Of course, there isn't really a good convention for where to keep static assets in Cargo (like your font files for example), but you could probably get away with having a src/html directory or something. It's just unfortunate to need an entirely separate directory in order to set up the project.

Wrote up a PR for the first two TODOs in #17.

Thanks again @jonhoo. BTW I just realized since there isn't any crate in cratesfyi_prefix, you can actually store it in project directory. Just make sure style.css compiled from style.scss exists in public_html (along with rustdoc css' and fonts).

Just finished setting up new server and moved to new domain: https://beta.docs.rs/
Initial build is started and it may take a week. It's building documentations for only tier 1 platforms. Search and most starred crates may not work at this time. I think we can go live when it finishes builds. Just need to write a small daemon to run various utils and add a menu to access different platforms of documentations.

I think we can go live when it finishes builds.

And one other thing: it's still called crates.fyi everywhere

Yes and that @steveklabnik

Alright we are docs.rs now @steveklabnik!

looking good!
On Sun, Jul 31, 2016 at 5:40 AM, Onur Aslan<EMAIL_ADDRESS>wrote:

Alright we are docs.rs now @steveklabnik https://github.com/steveklabnik
!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/onur/docs.rs/pull/12#issuecomment-236420680, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AABsis00kDpo8Q7scXEh4C2OVyTiY0MNks5qbG2rgaJpZM4I80i-
.

Docs.rs finally finished building crates. I'll write a simple daemon tomorrow, and then I think we are ready to go.

Awesome!!!!
On Aug 22, 2016, 19:45 -0400, Onur Aslan<EMAIL_ADDRESS>wrote:

Docs.rs finally finished building crates. I'll write a simple daemon tomorrow, and then I think we are ready to go.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (https://github.com/onur/docs.rs/pull/12#issuecomment-241585167), or mute the thread (https://github.com/notifications/unsubscribe-auth/AABsirUp6ImjUVgL56bXXR3Q-AfT_kTyks5qijSfgaJpZM4I80i-).

I've been monitoring daemon and it's working fine. I only saw one error; failed to pull recent changes from crates-io.index for 10 minutes. Docs.rs is currently pulling changes twice in 3 minute margin, IDK if github have some limits, maybe its better idea to increase delay between pulls.

@alexcrichton might have some opinions/comments about ^

AFAIK Cargo's load on the index would likely be far higher than docs.rs's, so that rate of polling shouldn't be too bad in theory?

@alexcrichton sounds reasonable! Just wanted to make sure you knew!

@onur :confetti_ball:
does this mean we can talk about this publicly now?

Yes @steveklabnik docs.rs is finally live!


[3pt] Create a schema.sql - Student A

Create a schema.sql file with tables that will be analogical to the structure of the classes that you created:

books table (add all properties and associations from the parent Item class as table columns)
labels table

https://github.com/jsug9/Catalog-Capstone/pull/64


Feature/file menu

Add the file menu with Open and Exit options. Make the emulator properly shutdown when window is closed. Implement CPU delay to slow down execution to something reasonable.

Coverage decreased (-3.5%) to 74.64% when pulling 315fb8023335f491eaf408d640b2c40b5b9ddc48 on feature/file-menu into<PHONE_NUMBER>72cb7bf629b8ba63be12d1479cc4aa on master.


getTokensForOwner throws invalid BigNumber exception for some tokens

[REQUIRED] Environment

Browser version:
Alchemy SDK version: 2.8.2

[REQUIRED] Describe the problem
A simple call to getTokensForOwner() for some users throws an error. I've noticed this type of exception myself in the past when working with ethers.js if trying to parse a response from a contract call that's expected to return a value but instead returns 0x. Note that the call works fine for other accounts.
Exception:
Error: invalid BigNumber string (argument="value", value="0x", code=INVALID_ARGUMENT, version=bignumber/5.7.0)
    at Logger.makeError (/node_modules/@ethersproject/logger/lib/index.js:238:21)
    at Logger.throwError (/node_modules/@ethersproject/logger/lib/index.js:247:20)
    at Logger.throwArgumentError (/node_modules/@ethersproject/logger/lib/index.js:250:21)
    at BigNumber.from (/node_modules/@ethersproject/bignumber/lib/bignumber.js:199:27)
    at /node_modules/alchemy-sdk/dist/cjs/index-25500d2e.js:1817:49
    at Array.map (<anonymous>)
    at CoreNamespace.<anonymous> (/node_modules/alchemy-sdk/dist/cjs/index-25500d2e.js:1815:62)
    at Generator.next (<anonymous>)
    at fulfilled (/node_modules/alchemy-sdk/dist/cjs/index-25500d2e.js:399:58)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  reason: 'invalid BigNumber string',
  code: 'INVALID_ARGUMENT',
  argument: 'value',
  value: '0x'
}

How to reproduce:
Sample script:
import {
  Alchemy,
  Network
} from 'alchemy-sdk'

const settings = {
  network: Network.Mainnet,
  apiKey: '<my_api_key>'
}

const sdk = new Alchemy(settings)

sdk.core.getTokensForOwner('<address>').then(console.log)

Relevant code or sample repro:

@mholtzman Thanks for the report! Can you share a sample address for me to repro this on my end?


split sbom (icm.json) to arch specific files

STONEBLD-1158

Maintainers will complete the following section

[x] Commit messages are descriptive enough
[x] Code coverage from testing does not decrease and new code is covered
[x] Python type annotations added to new code
[n/a] JSON/YAML configuration changes are updated in the relevant schema
[n/a] Changes to metadata also update the documentation for the metadata
[n/a] Pull request has a link to an osbs-docs PR for user documentation updates
[n/a] New feature can be disabled from a configuration file

LGTM with one minor comment. Seems like the pipeline is broken, even in master. I'll try to see what's going on.

rn updated


Add  diag method to CsMatBase

A method to add fetching the diagonal of a matrix.
I'm quite new to Rust, so I'm not sure if this useful. It's intended to spark a discussion around the implementation. I really wanted the a diag_view method, but I'm not sure how to implement it, so ideas are welcome.

Hello, and thanks for contributing this feature! I'll try and review it soon.

Should I bump the package version and add a changelog btw?

Should I bump the package version and add a changelog btw?

Not on this branch, this branch has breaking changes preparing for a 0.10 version. I will backport your changes to the 0.9 branch and issue a minor release.

@matzhaugen I've published version 0.9.3 on crates.io
There is now a v0.9 branch that can be used to backport these kind of changes to 0.9 while we continue iterating on 0.10.

Sounds good!


fix blob GC test

This was caused by #10621, which added support for blob handles using local IDs while offline. Due to a race condition, the container would not always be connected and uploadBlob() would return a local ID handle instead of one using the storage ID as the test expected, which would cause it to fail. I hit similar issues in other tests in the PR, but this seems to only happen against ODSP, not when run locally.
https://github.com/microsoft/FluidFramework/pull/10621/files#diff-aa6ab18dfe191259e467ea83ba348405ff244fbc66f8d5dfe2fc0d7ec62dcbf7

This was caused by #10621, which added support for blob handles using local IDs while offline. Due to a race condition, the container would not always be connected and uploadBlob() would return a local ID handle instead of one using the storage ID as the test expected, which would cause it to fail. I hit similar issues in other tests in the PR, but this seems to only happen against ODSP, not when run locally.
https://github.com/microsoft/FluidFramework/pull/10621/files#diff-aa6ab18dfe191259e467ea83ba348405ff244fbc66f8d5dfe2fc0d7ec62dcbf7

Should the test be updated to handle the scenario added by #10621? Do we have tests that cover them?

Should the test be updated to handle the scenario added by https://github.com/microsoft/FluidFramework/pull/10621? Do we have tests that cover them?

There are no GC tests, no. It might be good to add some. The functionality is similar to behavior in detached containers.


Adding cursor following animation

@MitulSonagara
I would like to add a cursor following animation and make it align with the website styling
Kindly assign this to me

@AswaniBolisetti
not needed for me


SelectOrInsert method?

In pg-go there was a method SelectOrInsert. The migration guide does not contain any info about deletion of this method. Does it exist in Bun?
If not - any chance you will add it in near future?

It does not exist in Bun, because it seems to be obsoleted since PostgreSQL added support for upserts. Nowadays you  should just use INSERT ON CONFLICT DO UPDATE. Let me know if that works for you and I will update the migration guide.
We may add something like SelectOrInsert in future but we need to understand

what task does it solve that you can't achieve with upserts
it should work for mysql and sqlite

@vmihailenco thanks for the fast response!
If no update needed I thing workaround for SelectOrInsert method will be:
res, err := conn.NewInsert().Model(entity).Where("name = ?", name).On("CONFLICT DO NOTHING").Returning("*").Exec(ctx)

So actually no need in separate SelectOrInsert method since it will break backward compatibility with other database drivers.

:+1: I will mention this in the migration guide.

@vmihailenco such solution is not working, because it is illegal to specify WHERE clause in INSERT queries. The error will be: ERROR #42601 syntax error at or near "WHERE".
Also Bun automatically adds WHERE table.deleted_at IS NULL to insert query for models with soft_delete field if On("CONFLICT DO NOTHING") specified.
By this I want to say that seems like currently it is impossible to make SelectOrInsert query with On method.
Probably On("CONFLICT UPDATE") will also not work for models with soft_delete field.

It is hard to guess why you need where, but you can use a second SELECT query if needed.

Bun will add where clause automatically for models with "soft_delete" field.

But if I want to select by using non-PK, such insert won't be conflicting.

Also no chance to select sth with Relation in case if record exists.

But if I want to select by using non-PK, such insert won't be conflicting.

It is not safe to insert by a non-unique index. You will have duplicate rows in the database.

Also no chance to select sth with Relation in case if record exists.

Inserts don't support Relation so SelectOrInsert will produce very different results depending on db state.


Fix dmdt description

in the description of the dmdt parameters, I think it would be useful to specify the units of the dt, so that one can easily compute the delta he needs to look for.

This was a request from a user. Here is the current description:

Its mags per day.  Often there are two detections with very small deltaT between them, which means dmdt is a big random number with a massive error. I am working with Stephen to make a much better estimate of rise rate and colour change that will actually be useful.

I'll fix the description for now and warn users to check the error.


KSF-83 Rollback Avro-UI version in the KSF project

Set avro-ui version to 0.2.1

/cc @ashvayka

LGTM


Year/Weekly overview error

Hello g1eb!
Since we last spoke I began to adapt this project of yours to show the data I need, and also make some code adjustments needed for the calendar to fit my project. Maybe my suggestion of simplifying the data format was a bit overkill, since I could feed the calendar the correct format. So no rushes here 😄
As you can see in the following image:

I'm using the calendar to show stays in places instead of projects, so in my case (and even in the project context) I have some stays that are of little seconds. In those cases D3 cannot create the rectangles in the week view, hence those erros. Maybe some scaling here to fix this problem with small data?
Other issue in this image:

You see the tooltip? That circle is related to March, however it is drawn on September/October. The labels are a bit messed up. They should be aligned with the data I am passing and not with the real world data. Thank you!

First one seems like a scaling/rounding issue indeed, let me take a look at that for ya :)
As for the second one, calendar should start drawing circles at the appropriate position during the year. Maybe even scale the labels to fit the data? Say if there is no data for first couple months then there is no need to show it? What do you think?

What do you mean with scaling the labels? Maybe the second option is better: if there is no data for certain days/months, no need to show it. If you could exemplify it, it would be awesome!

Yeah, basically hiding labels/circles if there is no data. With scaling I mean that if you have two or three months of data available that you use available width for that range. I did not mean to really scale circles to ovals, poor choice of words :)
Let me get back to you when I fix the first bug.

Yeah, that's it. I agree with what you said! I'll be waiting for new.

Heya Pedro! Sorry that took a while, I wanted to make more time to be sure I had the right fix 😸
Let me know what you think, I hope new release fixed your issue!

Hello,
Sorry for the time I took to answer, I was on vacation with little to no Internet access. I already tried the new release, it's all working good!!
Regarding the data format as we talked once, I'm having no problems with it, so I think it is not that hard to format.
About the option to observe multiple years, any idea?

Hey :) no worries, holidays are important! If this is solved, I'm going to close the issue 🍒
Glad it worked out with the format, it is aimed to be very generic indeed.
As for multiple years I have not made up my mind yet.. I like your suggestion to make a year selector and changing the year view on click. I need some more time to implement it though.
Oh something else I wanted to ask you, are you using this inside gridster? How is that working out?  😺

Yes you can close it, it's all good.
Let me know when you have a solution for the multiple years issue!
As for the gridster, yes, I'm using it. I was having some resizing issues with the calendar (the circles were becoming too big and not aligned with the labels). But since this calendar can be sort of "fixed" (no need to resize), I can say it is working very good. Check my angular repo or ask me anything if you need. Cheers 😄

Cool! Thanks, I'll keep you posted :)


Resolve errors arising from new list update

Description

This PR resolves all the error in the simple-lang modules that arise after updating the list core behaviour.

Efficient check in the Windows-Build.bat to determine the C/C++ build toolchain correctly
Add more examples for the new OOP Features
Create unit test files in each folder in examples folder

Type Of Change

Please delete options that are not relevant and mark the ones that are relevant.

[x] Bug Fix: Non breaking change which fixes an issue.
[ ] New feature: Non breaking change which adds a new functionality ( module, environment etc).
[ ] Breaking change - Fixes a feature that would cause existing functionality to not work as expected.
[ ] Documentation - Module or general documentation.

Briefly Explain Why This PR Should Be Merged.
This PR should be merge ASAP as the modules is presumably unusable due to changes in the list behavior
How Has This Been Tested ?

This has been thoroughly tested by running a general unit test on the entire sample files in th examples folder using the following files

unit_test.sim
algorithms/unit_test.sim
basic/unit_test.sim
gui/unit_test.sim
intermediate/unit_test.sim
internetandparser/unit_test.sim
modules/unit_test.sim
syntax/unit_test.sim

Checklist:

[ ] I have created a supporting issue for this PR.
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests passed locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

I'll be reviewing this unfailingly tomorrow.

First, I'm surprised you were trying to overwrite final variables well knowing they can't be overwritten. That blocked the installation because the DynamicModule list variable in DynamicLoader.sim file is a final var.
final __mainDynaicModulesCount = 0

final __mainDynaicModules = [
		["systemic.dll", "string_savant.dll"],		#Windows
		["systemic.so", "string_savant.so"],		#Linux
		["systemic.dylib", "string_savant.dylib"] 	#MAC
	]

Here again:

final dynamicModules = []

Then, you tried to overwrite the variable :
__mainDynaicModulesCount++

Secondly, you didn't change the names of the String module blocks in bake.sim. You renamed a few blocks but didn't change them in bake.sim.
Thirdly, I don't know why bake isn't building, perhaps check it. Also, I had to comment out the png_write line in fulltick module until a better solution is given off.

I ll check it

I fixed them already so you've gotta check why bake doesn't work.

You can commit leave bake for the next two PR

You can commit leave bake for the next two PR

Okay then..


Badges

Add badges for:

Build: https://github.com/aureliano/caravela/actions/workflows/go.yml/badge.svg
Reference: https://pkg.go.dev/badge/github.com/
Coverage: converalls?
Go report card
Quality gate status

Maybe OpenSSF Best Practices too.

Only build, coverage and reference will be added. Thus, reference will be done after issue #17.

Implemented by 85fdcbce65c8ccf04a52ac294dab1c42f4808102.


Sidecar: Shipper bytes uploaded metric

Describe the solution you'd like
I can't see a metric for the number of bytes the shipper has uploaded under:

thanos_objstore*
thanos_shipper*

This might be useful for helping debug replicas monitoring the same targets which should roughly be shipping the same amount of data.

Hi,
I am interested in implementing this, if you do not mind 😄

@ritaCanavarro Assigned to you.

Hi @jpds, I was looking at the shipper.go file and I see this metric thanos_shipper_uploads_total I wanted to know how is this metric different from the expected shipper bytes uploaded

Hi @yeya24 😊
If you could review the PR: https://github.com/thanos-io/thanos/pull/6500?
Thanks!

Hi 😊
@yeya24 could you please code review the PR?
PR: https://github.com/thanos-io/thanos/pull/6544
Best regards,
Rita Canavarro

Hi @yeya24,
This change is being applied here: https://github.com/thanos-io/objstore/pull/66.
After some discussions, it was decided to move the implementation there.

Hi @yeya24,
The metrics were implemented here:  https://github.com/thanos-io/objstore/pull/66 and are already merged to master.

Closing as #6544 got merged.
Thank you, @ritaCanavarro 🙇


Decide where to store config and chain data

Currently py-evm doesn't have a standard place to store its configs and chain data. We need to agree on one and start storing stuff there to avoid having to hard-code or regenerate values on every run
I believe we should use $XDG_DATA_HOME and $XDG_CONFIG_HOME as per https://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html

@gsalgado I took a quick gander at that spec.  Would you mind expanding this issue to contain the following.

What data do we need to store that is global
What data do we need to store for each chain
What folder structure we might use for this data.
What base directories we would use according the the XDG spec from above.

I cannot think of any global data at this point. Can you think of anything?
Chain specific:

block/state (one for lightchain and one for full chain)
private keys for accounts
P2P node's private key (used to generate the node's unique ID and for authentication)
peer statistics

Since all of that is actually data (and not user-maintained config) generated/updated by py-evm, I believe it should go under $XDG_DATA_HOME (which defaults to $HOME/.local/share).
As for the directory structure, I think the following would be a good start:
├── py-evm
│   └── mainnet
│       ├── chain         # LevelDB
│       ├── keys          # KeyStore, whatever format we chose
│       ├── lightchain    # LevelDB
│       ├── nodekey       # a text file containing the node's P2P private key
│       └── peers         # LevelDB

That also leaves room for any global data to be stored directly under $XDG_DATA_HOME/py-evm

I like it and happy to learn about $XDG_DATA_HOME which is a standard I wasn't aware of.

Oh, I think we might benefit from not implementing a keystore.  I don't want to manage user's private keys (at least not using py-evm).  I'm imagining this as something that moves into the web3.py layer with the web3.account API.  We can still read from keystore files and whatnot, I just don't want to do it at the py-evm level.


feat: update to protobufjs 6

Current coverage is 94.31% (diff: 95.88%)

No coverage report found for master at a03cb3d.

Powered by Codecov. Last update a03cb3d...cb49168


golden-cheetah: 3.7-DEV2408 -> 3.7-DEV2412

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Drafted because currently failing to build
golden-cheetah> Gui/Pages.cpp: In member function 'void ProcessorPage::processorSelected(QTreeWidgetItem*)':
golden-cheetah> Gui/Pages.cpp:2713:17: error: 'actionButtons' was not declared in this scope; did you mean 'ActionButtonBox'?
golden-cheetah>  2713 |             if (actionButtons != nullptr) {
golden-cheetah>       |                 ^~~~~~~~~~~~~
golden-cheetah>       |                 ActionButtonBox


Wake Lock Update

What's changed, or what was fixed?

adds details to Wake Lock post

Target Live Date: ASAP

[ ] This has been reviewed and approved by (NAME)
[x] I have run npm test locally and all tests pass.
[x] I have added the appropriate type-something label.
[x] I've staged the site and manually verified that my content displays correctly.

:+1:


Implement SEO and path handling on web

We need to figure out a good way to handle URLs on web and have good SEO for sites. #701 is related.

Also, we still need to implement scrolling on mobile web.

And zooming.

I implemented scrolling on mobile web a while ago, but zooming is still not implemented.

Path handling has already been improved on web.

Here is a key potential innovation: once we implement a reasonable HTML generator for SEO purposes, we could also easily use the same HTML as a preview for apps before they load. It would only take a small amount of simple CSS generation to make this viable, especially for text content. I think that would significantly ameliorate one of the biggest issues with web, which is the long load times, especially for static content like blogs. If you can start reading the blog while it loads with a decent experience and then get the full experience after it loads, I think that could potentially be very good and not too difficult to implement.


Add RS3 Music

Replaces the ingame music with its RS3 variant when available. When a soundtrack is not available it will fallback to its OSRS variant.
Note: the music is fetched from an external source

Looks fine besides the warning message.

Thank you for taking the time to review it. I've applied your suggested changes.


Simplify the requests API

Folllowing https://github.com/wellcomecollection/catalogue-api/pull/224
Refactors the requests API to remove some potential bugs.

I think this could do with some more tests around the failure cases on lookupRequests, but might be for a future PR - depending on review.


Check if CommandLineTools are installed before trying to execute inst…

…all_name_tool
Motivation:
We should check if CommandLineTools are installed before trying to execute install_name_tool as otherwise it might open a prompt to ask users to install it which is unexpected.
Modifications:
Check if the folder /Library/Developer/CommandLineTools exists before trying to execute install_name_tool
Result:
Better user experience. Related to https://github.com/netty/netty/issues/13125

Lets just merge this one for now


Add a UI to select the aggregation function of a field

As discussed in #10.
The list of selectable functions is set using the aggregates option of the config.
The default value is:
[
      {name: 'count', caption: 'Count', aggregateFunc: Aggregations.count},
      {name: 'sum', caption: 'Sum', aggregateFunc: Aggregations.sum},
      {name: 'avg', caption: 'Average', aggregateFunc: Aggregations.avg},
      {name: 'min', caption: 'Minimum', aggregateFunc: Aggregations.min},
      {name: 'max', caption: 'Maximum', aggregateFunc: Aggregations.max},
      {name: 'prod', caption: 'Product', aggregateFunc: Aggregations.prod},
      {name: 'stdev', caption: 'Standard Deviation', aggregateFunc: Aggregations.stdev},
      {name: 'stdevp', caption: 'Standard Deviation P.', aggregateFunc: Aggregations.stdevp},
      {name: 'var', caption: 'Variance', aggregateFunc: Aggregations.var},
      {name: 'varp', caption: 'Variance P.', aggregateFunc: Aggregations.varp}
]

name is used to be displayed in the field like: "Field Caption (avg)"
caption is used to be displayed in the select field.
aggregateFunc works like the aggregateFunc option of the Pivot Grid Field: it can either be a constant such as 'avg' or 'max' or a function.

Hi Valentin,
Really good start. But, imo, it's not complete. My observations:

the code for this popup and the filter popup can be combined (using mixins)
config aggregates should extend default ones, not replace them.
no need to rebuild the entire pivot grid, just recompute that field values and refresh the data area
aggregate name should be highlighted when hovered
popup is not aligned with anything and its style is different from the filter popup (all popups should have the same style)

Thanks,
--
nnajm.

the code for this popup and the filter popup can be combined (using mixins)

I think we should diretly merge filters and aggregates in a common settings popup (like you mention in #10)

config aggregates should extend default ones, not replace them.

I am not sure to understand what you mean. I added an default array in the aggregate module to add caption field for the aggregates (for the select widget).

no need to rebuild the entire pivot grid, just recompute that field values and refresh the data area.

About the rebuilding part, I was not sure how to handle it. I simply copied the way it was done in the applyFilter functions... Any advice would be welcomed ;)

aggregate name should be highlighted when hovered

It will be handle in the settings popup.

popup is not aligned with anything and its style is different from the filter popup (all popups should have the same style)

idem.

@ValentinH Can you share aggregateFunc code that is Change the aggregateFunc of a data from the UI ?

@ValentinH Can you share aggregateFunc code that is Change the aggregateFunc of a data from the UI ?

The code is the one in this pull request

@ValentinH  thanks for response,  just now I am  download orb.js but I cant find new changes.  can you send to my email<EMAIL_ADDRESS>

As I said my changes are in this PR, look in the file tab: https://github.com/nnajm/orb/pull/15/files

@ValentinH  Can you send me workable solution, please help me.

No

Sorry I am asked demo orb.js with select the aggregation function from UI.

I have one more question, did you implement lazy loading on orb on  ??

The code is in this PR. If you want me to do your job, then I'm available as a freelance and can invoice you. ;)


Call services from routes to not invoke at registration

Also breaks on 3.3.
A controller's connect() is called during registration, and the addition of the routes will invoke the constructors on the services you're using.

Working like a charm! 💖


Find quick wins in recently-shipped BCD keys

@Elchi3 and I had a call earlier and picked out a few recent items that are likely to be additions to existing features. Anyone is welcome to take parts of this and get it into real PRs.

I think we can close this. We now have a neat way to look at the burndown list and more PRs have been (and will be) created.


add BitColorPicker documentation (#674)

This closes #674

this component is under heavy refactoring.
please wait until the refactoring is done

since the required PR is merged, please continue work on this PR


scene.setBackgroundColor( ) doesn't round trip to blocks

Repo:

add a scene.setBackgroundColor(11) block to on start
change view to javascript, edit new value  scene.setBackgroundColor(13)
return to blocks view

Result:

displays as a grey text block

Fixed


adapt updated schemagen api and improve design

rest-schemagen 0.18.0 has to be released in order to make this work

Coverage decreased (-3.5%) to 86.782% when pulling d17bbffa77ee9cdf93b933c1a2a4117c7efeefed on updated_schemagen_api into fe47d0a144346c64f1f5486b82c6c2106d1787ab on master.

Coverage decreased (-3.5%) to 86.782% when pulling 4fe70558ce6b8b121ea35ac3cc6291284c2ec243 on updated_schemagen_api into fe47d0a144346c64f1f5486b82c6c2106d1787ab on master.

Coverage decreased (-0.7%) to 89.625% when pulling 52aa619541241ab8e5f9840ffd2a6bfe2bebb18c on updated_schemagen_api into fe47d0a144346c64f1f5486b82c6c2106d1787ab on master.

Coverage increased (+1.0%) to 91.291% when pulling 0bec93931401535ed6ec53ccb00ad8054ad335f3 on updated_schemagen_api into fe47d0a144346c64f1f5486b82c6c2106d1787ab on master.


Open files, sometimes open in the same tab as current instead of another

VSCode Version: Code 1.12.2 (19222cdc84ce72202478ba1cec5cb557b71163de, 2017-05-10T13:16:25.899Z)
OS Version: Linux x64 4.9.5-040905-generic
Extensions:

Extension
Author
Version

gherkin-indent
AravindKumar
2.0.0

genuuid
espresso3389
0.3.3

ruby-rubocop
misogi
0.3.5

mssql
ms-mssql
1.0.0

Ruby
rebornix
0.12.1

cucumber
stevejpurves
0.9.6

Steps to Reproduce:

Open a file
Open another one
It should open in a next-to-right tab but it open in the same as the first one

This is a feature, read up about preview mode https://code.visualstudio.com/docs/getstarted/userinterface#_preview-mode


Add automatic pypi release for evaluate

This GH actions workflow will do an automatic pypi release as soon as you push a tag in GitHub that begins with v*. This requires setting the PYPI_TOKEN secret in GitHub. Example from huggingface_hub: https://github.com/huggingface/huggingface_hub/tags (one tag per release is also a good idea since that helps draft automatic release notes).

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Thanks @osanseviero! That looks great. Wdyt @LysandreJik?

Looks great, but be very careful when pushing tags then :)


Isolate domains info per tab/window

The extension currently gives the domains from most recently opened page. This means it gives a wrong list when you visit a site in one tab, switches back to other tab, and click on the extension.

Fixed.


Display mod actions in forum topics

I've added actionid in user log, but existing events need to be converted.

Added them, still need to convert existing events whenever we push to prod.

Perhaps replies should display delete event, instead of hiding it. Would also make sense adding a reason field.

Make a select box with reasons like "you're a poophead, RJ" and "duck off, Steam account seller".

:nail_care:


onInput event firing twice

Issue report
Expected Behavior

onInput event should fire once
onChange does not fire an event

Current Behavior

onInput event fires twice
onChange should fire an event

Possible Solution

Talked with Geoff, and he mentioned that he could have something to do with how ods-inputoptions is surfacing the events

Steps to Reproduce (for bugs)
https://github.com/AlaskaAirlines/OrionReactDemo/blob/master/src/Components.js#L62

Link above was to the demo site github. Should be able to clone and run the site locally with a custom console log onInput, and you should see two events fired twice.

Context

Trying to omniture tag how our users are changing their mind when preordering

Your Environment

Browser Name and version: Chrome Version 78.0.3904.108 (Official Build) (64-bit)
Operating System and version (desktop or mobile): Desktop Windows

@aaronluannguyen is this causing a functional issue with your current application?
If not, I am inclined to not address this as we are working on a full replacement for this component. If you are able, a pull request with a suggested change would be very helpful.


Update banner styles

Fixes

Related to #2015 by @zackkrida
Description

Updates the banner styles to have padding and rounded edges.
Testing Instructions

Check out some localhost page(s) with banners, like:
localhost:8443/ar?referrer=creativecommons.org
localhost:8443/search-help?referrer=creativecommons.org

Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[x] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Here is an idea: One alert banner with four variants, and one customized banner.

The alert banner has four variants (Info, Warning, Success, Error) and the customized banner is in dark charcoal and allows changing the icon as in the former types.
These two banners are due to having yellow and white backgrounds, and since Openverse is mostly white, I prioritized a style without border color and rely on icon taxonomy to convey the banner's purpose. In that vein, the color contrast requirement is met between icon and banner background color.
Our current case shows banners in the homepage, but I think we can address cases like Openverse is not working correctly. See the right side of the image below.

Reaching consistency is hard as the component needs to work over two different backgrounds, but if we use the customized dark banner for some cases, we can handle the alerts confidently.

What do you think of this idea? Pinging @dhruvkb for visibility

@fcoveram this PR was only focused on the spacing and rounded corners you mentioned previously, but I love your designs for an alert system! That would also require very little work—it is only a matter of small styling adjustments. Could you move those designs to their own issue for additional discussion?

I like the alerts, but that would increase the scope of the PR considerably. It would be best to spin it out into its own issue.

Sorry for extending the PR scope, last comments in #2015 mention style specs and that's what the design shared is about.
Putting aside the alert variants, the style specs for the dark banner are:
Small (for xs, sm, md)

Height: auto
Text: "Caption regular" style
Info icon in 24px size
Close icon in 24px size
Padding: 12px for top and bottom; 8px for left and right
Margin: 8px for top, left, and right. Margin bottom is 0.
Gap between icons and texts: 8px.
Gap between banners: 8px.
Close action and link do not have hover style

Large (for lg, xl, 2xl)

Height: auto
Text: "Paragraph small" style
Info icon in 24px size
Close icon in 24px size
Padding: 16px for all sides.
Margin: 8px for top, left, and right. Margin bottom is 0.
Gap between icons and texts: 8px.
Gap between banners: 8px.
Close action and link do not have hover style

Here is the Figma file where mockups for all breakpoints are. The banner doesn't exist in the Design Library yet.

Thanks, @fcoveram! This PR now uses 8px for the banner gap and top/side margins and is ready for review. We'll implement the dark banner styles and/or the alert styles in a separate PR.