[update] Fix build problem of 'Dockerfile'

Issue Type

Bug fix

Runtime Environment

Operating system and version: Ubuntu 20.04 LTS / Ubuntu 18.04 LTS
Terminal emulator and version: XTerm
Python version: 3.6 (inside Docker)
NVML version (driver version): [e.g. 515.86 / 440.44]
nvitop version or commit: 0.11.0
python-ml-py version: -
Locale: -

Description

Adding following changes to Dockerfile to resolve the problem in running nvitop command in a Docker container.

Copying requirements.txt to /tmp/requirements.txt in the image
Running pip install -r /tmp/requirements.txt to install dependencies

Motivation and Context
When following the commands below as indicated in https://github.com/XuehaiPan/nvitop#for-docker-users, nvitop in the container throws an error:

Commandsgit clone --depth=1 https://github.com/XuehaiPan/nvitop.git && cd nvitop  # clone this repo first
docker build --tag nvitop:latest .  # build the Docker image
docker run -it --rm --runtime=nvidia --gpus=all --pid=host nvitop:latest  # run the Docker container

ErrorTraceback (most recent call last):
  File "/usr/lib/python3.6/runpy.py", line 183, in _run_module_as_main
    mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
  File "/usr/lib/python3.6/runpy.py", line 142, in _get_module_details
    return _get_module_details(pkg_main_name, error)
  File "/usr/lib/python3.6/runpy.py", line 109, in _get_module_details
    __import__(pkg_name)
  File "/nvitop/nvitop/__init__.py", line 19, in <module>
    from nvitop import core
  File "/nvitop/nvitop/core/__init__.py", line 19, in <module>
    from nvitop.core import host, libcuda, libnvml, utils
  File "/nvitop/nvitop/core/host.py", line 25, in <module>
    import psutil as _psutil
ModuleNotFoundError: No module named 'psutil'

I modified Dockerfile so that the dependencies are explicitly installed.
By the change, the error due to lack of dependencies was resolved and nvitop command was successfully run.
Testing
I tested building the image and running nvitop command in a docker container in two environments mentioned above, and confirmed successful build and execution.
The change does not affect any other areas of the codebase.
Images / Videos
N/A

By the change, the error due to lack of dependencies was resolved and nvitop command was successfully run.

@ShigemichiMatsuzaki Thanks for raising this!
The problem here is the Python 3.6 setuptools library is too old in the docker container. The maximum version of setuptools is 59.6.0. It does not support pyproject.toml based projects. The Dockerfile has not been updated for a long time and I did not aware of this while migrating to pyproject.toml.
PS: The requirements.txt is also a legacy approach to handling dependencies. I think we should move on and bypass the tech debt here.

Hi,
Thank you for the prompt response!
I really appreciate it.
And many thanks to your excellent work.


Add localization

Added

Localization Support

This localization solution has this advantages:

Generates source code based on properties files
Allows to change the language in runtime
It can auto detect languages automatically without extra configuration
It can use named variables in the language files ("My name is {{name}}")
It can also support import/override Languages from the filesystem (not implemented yet)

And here is why I didn't use compose resources


Install mricrogl without Python support

@neurolabusc you recently mentioned the package could be built without Python scripting support.
Sadly I am having trouble building the python4lazarus module (upstream does not support lazbuild, and the Lazarus forums could not help out thus far either).
In case I have to drop python4lazarus, how can I build mricrogly in its absence?
I am currently getting this error:
silenthost ~ # emerge -av mricrogl

 * IMPORTANT: 5 news items need reading for repository 'gentoo'.
 * Use eselect news read to view new items.

These are the packages that would be merged, in order:

Calculating dependencies... done!
[ebuild  N     ] sci-visualization/mricrogl-1.0.20181111::chymeric  0 KiB

Total: 1 package (1 new), Size of downloads: 0 KiB

Would you like to merge these packages? [Yes/No] y

>>> Verifying ebuild manifests

>>> Emerging (1 of 1) sci-visualization/mricrogl-1.0.20181111::chymeric
 * mricrogl-1.0.20181111.tar.gz SHA256 SHA512 WHIRLPOOL size ;-) ...                                                                 [ ok ]
>>> Unpacking source...
>>> Unpacking mricrogl-1.0.20181111.tar.gz to /var/tmp/portage/sci-visualization/mricrogl-1.0.20181111/work
>>> Source unpacked in /var/tmp/portage/sci-visualization/mricrogl-1.0.20181111/work
>>> Preparing source in /var/tmp/portage/sci-visualization/mricrogl-1.0.20181111/work/MRIcroGL-1.0.20181111 ...
>>> Source prepared.
>>> Configuring source in /var/tmp/portage/sci-visualization/mricrogl-1.0.20181111/work/MRIcroGL-1.0.20181111 ...
>>> Source configured.
>>> Compiling source in /var/tmp/portage/sci-visualization/mricrogl-1.0.20181111/work/MRIcroGL-1.0.20181111 ...
cp: cannot stat '/etc/lazarus': No such file or directory
SetPrimaryConfigPath NewValue="system-lazarus-config" -> "/var/tmp/portage/sci-visualization/mricrogl-1.0.20181111/work/MRIcroGL-1.0.20181111/system-lazarus-config"
SetupCompilerFilename: Searching compiler ...
SearchCompilerCandidates Value=fpc File=/usr/bin/fpc
Hint: (lazarus) [RunTool] "/usr/bin/fpc" "-iWTOTP"
Hint: (lazarus) [RunTool] "/usr/bin/fpc" "-va" "compilertest.pas"
SetupCompilerFilename: using /usr/bin/fpc
Hint: (lazarus) storing Lazarus directory "/usr/share/lazarus/" Compiler path "/usr/bin/fpc" in "/var/tmp/portage/sci-visualization/mricrogl-1.0.20181111/work/MRIcroGL-1.0.20181111/system-lazarus-config/environmentoptions.xml"
Error: (lazbuild) Broken dependency: python4lazarus_package
 * ERROR: sci-visualization/mricrogl-1.0.20181111::chymeric failed (compile phase):
 *   (no error message)
 *
 * Call stack:
 *     ebuild.sh, line 125:  Called src_compile
 *   environment, line 785:  Called die
 * The specific snippet of code:
 *       lazbuild -B --lazarusdir="/usr/share/lazarus/" --pcp="system-lazarus-config" simplelaz.lpi || die
 *
 * If you need support, post the output of `emerge --info '=sci-visualization/mricrogl-1.0.20181111::chymeric'`,
 * the complete build log and the output of `emerge -pqv '=sci-visualization/mricrogl-1.0.20181111::chymeric'`.
 * The complete build log is located at '/var/tmp/portage/sci-visualization/mricrogl-1.0.20181111/temp/build.log'.
 * The ebuild environment file is located at '/var/tmp/portage/sci-visualization/mricrogl-1.0.20181111/temp/environment'.
 * Working directory: '/var/tmp/portage/sci-visualization/mricrogl-1.0.20181111/work/MRIcroGL-1.0.20181111'
 * S: '/var/tmp/portage/sci-visualization/mricrogl-1.0.20181111/work/MRIcroGL-1.0.20181111'

>>> Failed to emerge sci-visualization/mricrogl-1.0.20181111, Log file:

>>>  '/var/tmp/portage/sci-visualization/mricrogl-1.0.20181111/temp/build.log'

 * Messages for package sci-visualization/mricrogl-1.0.20181111:

 * ERROR: sci-visualization/mricrogl-1.0.20181111::chymeric failed (compile phase):
 *   (no error message)
 *
 * Call stack:
 *     ebuild.sh, line 125:  Called src_compile
 *   environment, line 785:  Called die
 * The specific snippet of code:
 *       lazbuild -B --lazarusdir="/usr/share/lazarus/" --pcp="system-lazarus-config" simplelaz.lpi || die
 *
 * If you need support, post the output of `emerge --info '=sci-visualization/mricrogl-1.0.20181111::chymeric'`,
 * the complete build log and the output of `emerge -pqv '=sci-visualization/mricrogl-1.0.20181111::chymeric'`.
 * The complete build log is located at '/var/tmp/portage/sci-visualization/mricrogl-1.0.20181111/temp/build.log'.
 * The ebuild environment file is located at '/var/tmp/portage/sci-visualization/mricrogl-1.0.20181111/temp/environment'.
 * Working directory: '/var/tmp/portage/sci-visualization/mricrogl-1.0.20181111/work/MRIcroGL-1.0.20181111'
 * S: '/var/tmp/portage/sci-visualization/mricrogl-1.0.20181111/work/MRIcroGL-1.0.20181111'

What version of lazbuild are you using? lazbuild --v? I would expect the command
lazbuild --verbose-pkgsearch lazopenglcontext --verbose-pkgsearch pascalscript should install these dependencies, assuming you have a modern version of Lazarus.
Any reason you want to build MRIcroGL1.0 vs 1.2? 1.0 is clearly deprecated, I am a full time teacher, scientist and administrator, so do not have much time for old projects. I am happy if they prove stable and robust, but I would urge you to heed the warnings on the MRIcroGL1.0 web page to consider 1.2

ah, sorry, I somehow lost track of the newer versions (seeing as they are from a new repo). I'll have a look at that and comment on the respective repo if issues pop up. Thank you :)


Recovery Image not available

This page links to https://support.microsoft.com/en-us/surfacerecoveryimage?devicetype=surfacehub2s for downloading recovery image of Surface Hub 2S. After entering the serialnumber of our device this page only shows "The recovery image for your device isn't available for download through its page. For help, please contact-us"

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

ID: b6435b14-27da-d10e-4919-b3194ca03a07
Version Independent ID: b181319b-1bb4-010a-eeab-64ac199fb24c
Content: Reset and recovery for Surface Hub 2S
Content Source: devices/surface-hub/surface-hub-2s-recover-reset.md
Product: surface-hub
Technology: windows
GitHub Login: @robmazz
Microsoft Alias: robmazz

@officedocsbot assign @mypil

@fabianseither - Thank you for submitting feedback.
I will get this issue over to the Win10 ITPro writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@fabianseither - Thank you for engaging with the community here for the docs.
I think the best way forward is if you can contact support and open a service ticket for Windows 10 products  so this can get resolved ASAP. Based on outcome, let me know if it is something that can be called out in the docs.
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.

@officedocsbot close

@fabianseither Did this get fixed for you? Can you confirm the filename?


Update Playwright test and ChatMessage component

This pull request includes updates to the Playwright test in chat_io.spec.ts and the ChatMessage component in chatMessage/index.tsx. The Playwright test now includes assertions for the chat output and input, ensuring that the correct messages are displayed. The ChatMessage component now includes data-testid attributes for the chat messages, allowing for easier testing and identification of specific messages.

LGTM


Increase vm boot timeout to 500 seconds or more

May be..I added this as the tests failed with following error message:
https://github.com/fredrikhgrelland/terraform-nomad-hive/blob/133af774e0553f83f30f5bad5e554a869a90441a/Vagrantfile.default#L4
If you look above, you should be able to see the error(s) that
Vagrant had when attempting to connect to the machine. These errors
are usually good hints as to what may be wrong.

If you're using a custom box, make sure that networking is properly
working and you're able to connect to the machine. It is a common
problem that networking isn't setup properly in these boxes.
Verify that authentication configurations are also setup properly,
as well.

If the box appears to be booting properly, you may want to increase
the timeout ("config.vm.boot_timeout") value.
make: *** [up] Error 1
Error: Process completed with exit code 2.

The default timeout value is 300 seconds : https://www.vagrantup.com/docs/vagrantfile/machine_settings#config-vm-boot_timeout
I will create an issue so we can keep a track of this.
Originally posted by @Neha-Sinha2305 in https://github.com/fredrikhgrelland/terraform-nomad-hive/pull/22#discussion_r502345544

Was fixed here https://github.com/fredrikhgrelland/terraform-nomad-hive/pull/22


Update description and Nexusmods page to have RED4ext as a dependency from now on

README
Github is NOT here for support and general questions, it's designed for developers to discuss development and code. If your issue is unrelated to anything technical, do not make an issue, you will be blocked and your issue closed without an answer.
Bug
Operating system: Windows 11 Pro 23H2
Game version: 2.12
CET version: 1.31.1-yawn
GPU: AMD RX 6800XT
Description
The new version of CET that supports Cyberpunk 2.12 now requires RED4ext to be installed. Without it the mod won't run at all. No overlay, no logs. Despite being on a clean install, with absolutely no leftover files from a previous CET version.
Previous versions ran fine without any other dependencies and since there was no information on the readme, Nexusmods description & popup for "Additional files required" for this version, I had no idea it was now required for it to work.
Expected behaviour
Add an info in the description for this release that RED4ext is required for the mod to run. Also add RED4ext as a required file on the Nexusmods page.
Also update the Cyber Engine Tweaks wiki to list it as a dependency.
Reproduction
Fresh install of Cyberpunk 2077 2.12.
Download latest CET release, 1.31.1-yawn
Drag n drop to game directory. CET will now not run despite not having any files from the older versions, like cyber_engine_tweaks.asi

Thanks for posting this. The new release made it seem like I cant read. But nowhere did it say we needed to delete a file. Couldnt find it in the wiki either. Anyways deleting that file (or any leftover files for that matter) still didnt work like you mentioned. This has always been drag and drop for me, which the wiki implies as well. Let me see if installing RED4ext solves it for me.

Can confirm it works. Downloaded latest red4ext here (non-pdbs version): https://github.com/WopsS/RED4ext/releases . Extract that first to cyberpunk root installation before extracting cyber engine tweaks to same cyberpunk root. Everything should work now

It's fixed :)


Workspace Factory #9: Import Toolbox XML

Allows user to choose a local file containing toolbox XML and load it into the workspace factory editing space to continue editing it. If the user does not choose a valid XML file, it prints an error message.
Also adds a "Clear" button to entirely clear the workspace factory editing space. This was trivial to add because it is necessary to clear before loading a toolbox from a file.

This change is

+@picklesrus +@vicng +@quachtina96 +@rachel-fenichel

Review status: 0 of 5 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: 0 of 5 files reviewed at latest revision, 2 unresolved discussions.

demos/workspacefactory/factory_controller.js, line 521 [r1] (raw file):

      if (item.tagName == 'category') {
        this.createCategory(item.getAttribute('name'), false);
        this.switchElement(this.model.getElementByIndex(i).id);

maybe you could provide an alias for
"this.model.getElementByIndex(i)"  at the start of the for loop--might make the code more readable

demos/workspacefactory/factory_view.js, line 67 [r1] (raw file):

};

FactoryView.prototype.addEmptyCategoryMessage = function() {

comment this function

Comments from Reviewable

Review status: 0 of 5 files reviewed at latest revision, 15 unresolved discussions.

demos/workspacefactory/factory_controller.js, line 476 [r1] (raw file):

 * Should contain valid toolbox XML.
 */
FactoryController.prototype.import = function(file) {

Rename to importFile

demos/workspacefactory/factory_controller.js, line 504 [r1] (raw file):

 * @param {!Element} tree XML tree to be loaded to toolbox editing area.
 */
FactoryController.prototype.importFromTree = function(tree) {

Mark as private.
Also, I think parts of this method could be broken up into the Model class. This method would then just update the View after the Model has been updated.

demos/workspacefactory/factory_controller.js, line 508 [r1] (raw file):

  this.model.clearToolboxList();
  this.view.clearToolboxTabs();
  // No categories present.

Move the "No categories present" inside the if block.
Add line break above this line. It's hard to read a method if it's written without line breaks separating "paragraphs" of code.

demos/workspacefactory/factory_controller.js, line 515 [r1] (raw file):

    // Add message to denote empty category.
    this.view.addEmptyCategoryMessage();
  // Categories/separators present.

Move this comment inside the else block.

demos/workspacefactory/factory_controller.js, line 518 [r1] (raw file):

  } else {
    for (var i = 0, item; item = tree.children[i]; i++) {
      // If the element is a category, create a new category and switch to it

Move this inside the if statement

demos/workspacefactory/factory_controller.js, line 523 [r1] (raw file):

        this.switchElement(this.model.getElementByIndex(i).id);
        var category = this.model.getElementByIndex(i);
        // Add all blocks to the category.

Add line break above this line.

demos/workspacefactory/factory_controller.js, line 525 [r1] (raw file):

        // Add all blocks to the category.
        for (var j = 0, blockXml; blockXml = item.children[j]; j++) {
          var block = Blockly.Xml.domToBlock(blockXml, this.toolboxWorkspace);

Is block supposed to be assigned to the category?

demos/workspacefactory/factory_controller.js, line 527 [r1] (raw file):

          var block = Blockly.Xml.domToBlock(blockXml, this.toolboxWorkspace);
        }
        // Evenly space the blocks.

Add line break above this line.

demos/workspacefactory/factory_controller.js, line 533 [r1] (raw file):

          category.changeColor(item.color);
        }
      // If the element is a separator, add the separator and switch to it.

Move this inside the else block

demos/workspacefactory/factory_controller.js, line 540 [r1] (raw file):

    }
  }
  this.updatePreview();

Add line break above this line.

demos/workspacefactory/factory_view.js, line 62 [r1] (raw file):

  var count = table.rows.length;
  table.deleteRow(index);
  // If last category removed, add category help text and disable category

Add line break above this line.

demos/workspacefactory/factory_view.js, line 267 [r1] (raw file):

 * be moved. Must be rendered.
 */
FactoryView.prototype.distributeBlocks = function(blocks) {

Can it be changed so the blocks are spaced relative to each other (not by absolute value relative to the workspace)? This can be a TODO. Might want to check with someone else to see if there's a method that exists in the library for this already too.

demos/workspacefactory/index.html, line 36 [r1] (raw file):

        <button id="button_print">Print</button>
        <input type="file" id="input_import" class="inputfile"></input>
        <label for="input_import">Import</label>

Move Import before Export?

Comments from Reviewable

Review status: 0 of 7 files reviewed at latest revision, 15 unresolved discussions, some commit checks failed.

demos/workspacefactory/factory_controller.js, line 476 [r1] (raw file):
Previously, vicng wrote…
> Rename to importFile

Done.

demos/workspacefactory/factory_controller.js, line 504 [r1] (raw file):
Previously, vicng wrote…
> Mark as private.
>
> Also, I think parts of this method could be broken up into the Model class. This method would then just update the View after the Model has been updated.

Marked as private, and we talked about leaving all of this in the controller.

demos/workspacefactory/factory_controller.js, line 508 [r1] (raw file):
Previously, vicng wrote…
> Move the "No categories present" inside the if block.
>
> Add line break above this line. It's hard to read a method if it's written without line breaks separating "paragraphs" of code.

Done.

demos/workspacefactory/factory_controller.js, line 515 [r1] (raw file):
Previously, vicng wrote…
> Move this comment inside the else block.

Done.

demos/workspacefactory/factory_controller.js, line 518 [r1] (raw file):
Previously, vicng wrote…
> Move this inside the if statement

Done.

demos/workspacefactory/factory_controller.js, line 521 [r1] (raw file):
Previously, quachtina96 (Tina Quach) wrote…
> maybe you could provide an alias for
> "this.model.getElementByIndex(i)"  at the start of the for loop--might make the code more readable

Done by reordering.

demos/workspacefactory/factory_controller.js, line 523 [r1] (raw file):
Previously, vicng wrote…
> Add line break above this line.

Done.

demos/workspacefactory/factory_controller.js, line 525 [r1] (raw file):
Previously, vicng wrote…
> Is block supposed to be assigned to the category?

It's being loaded into the workspace, properly spaced, and then saved. Clarified comment.

demos/workspacefactory/factory_controller.js, line 527 [r1] (raw file):
Previously, vicng wrote…
> Add line break above this line.

Done.

demos/workspacefactory/factory_controller.js, line 533 [r1] (raw file):
Previously, vicng wrote…
> Move this inside the else block

Done.

demos/workspacefactory/factory_controller.js, line 540 [r1] (raw file):
Previously, vicng wrote…
> Add line break above this line.

Done.

demos/workspacefactory/factory_view.js, line 62 [r1] (raw file):
Previously, vicng wrote…
> Add line break above this line.

Done.

demos/workspacefactory/factory_view.js, line 67 [r1] (raw file):
Previously, quachtina96 (Tina Quach) wrote…
> comment this function

Oops thanks!

demos/workspacefactory/factory_view.js, line 267 [r1] (raw file):
Previously, vicng wrote…
> Can it be changed so the blocks are spaced relative to each other (not by absolute value relative to the workspace)? This can be a TODO. Might want to check with someone else to see if there's a method that exists in the library for this already too.

Added as a TODO for now, will check to see if this exists.

demos/workspacefactory/index.html, line 36 [r1] (raw file):
Previously, vicng wrote…
> Move Import before Export?

Done.

Comments from Reviewable

Review status: 0 of 7 files reviewed at latest revision, 15 unresolved discussions, some commit checks failed.

demos/workspacefactory/factory_view.js, line 267 [r1] (raw file):
Previously, evd2014 (Emma Dauterman) wrote…
> Added as a TODO for now, will check to see if this exists.

Just found WorkspaceSvg.cleanUp_() which does exactly that, so removed distributeBlocks() and replaced it with that. Submitted a pull request to Blockly develop for making cleanUp_() public, and once that gets changed on master, I'll change the call to clean up the blocks to be cleanUp() (added TODO).
I'm also planning on taking out XY attributes in standardCategories and using cleanUp_() for layout there as well (future CL).

Comments from Reviewable

Review status: 0 of 7 files reviewed at latest revision, 16 unresolved discussions, some commit checks failed.

demos/workspacefactory/factory_controller.js, line 525 [r1] (raw file):
Previously, evd2014 (Emma Dauterman) wrote…
> It's being loaded into the workspace, properly spaced, and then saved. Clarified comment.

Remove `var block =`?

demos/workspacefactory/factory_controller.js, line 502 [r3] (raw file):

 * XML format.
 *
 * @package

@private?

Comments from Reviewable

Review status: 0 of 7 files reviewed at latest revision, 16 unresolved discussions, some commit checks failed.

demos/workspacefactory/factory_controller.js, line 525 [r1] (raw file):
Previously, vicng wrote…
> Remove `var block =`?

Done.

demos/workspacefactory/factory_controller.js, line 502 [r3] (raw file):
Previously, vicng wrote…
> `@private`?

I'll change that. Saw @package in the Closure documentation and thought I was supposed to use that, but I'll use @private.

Comments from Reviewable

 When vicng is happy.

Review status: 0 of 6 files reviewed at latest revision, 16 unresolved discussions.

demos/workspacefactory/factory_controller.js, line 502 [r3] (raw file):
Previously, evd2014 (Emma Dauterman) wrote…
> I'll change that. Saw @package in the Closure documentation and thought I was supposed to use that, but I'll use @private.

if its only called from this class then yeah, you want private.  package is used in blocky in places where we want to be able use stuff inside of blockly, but not have it visible to developers (e.g. the workspace tells the flyout to do something that we don't )

Comments from Reviewable

 after small formatting

Review status: 0 of 6 files reviewed at latest revision, 21 unresolved discussions.

demos/workspacefactory/factory_controller.js, line 499 [r6] (raw file):

    return;
  }
  var reader = new FileReader();

Nit: Add line above this

demos/workspacefactory/factory_controller.js, line 511 [r6] (raw file):

    }
  }
  // Read the file.

Nit: Add line above this

demos/workspacefactory/factory_controller.js, line 539 [r6] (raw file):

    for (var i = 0, item; item = tree.children[i]; i++) {
      if (item.tagName == 'category') {
      // If the element is a category, create a new category and switch to it.

Nit: Indentation

demos/workspacefactory/factory_controller.js, line 559 [r6] (raw file):

        }
      } else {
      // If the element is a separator, add the separator and switch to it.

Nit: Indentation

demos/workspacefactory/factory_model.js, line 255 [r6] (raw file):

FactoryModel.prototype.clearToolboxList = function() {
  this.toolboxList = [];
  // TODO: When merge changes, also clear shadowList.

Nit: Add your name to the TODO

Comments from Reviewable

Review status: 0 of 6 files reviewed at latest revision, 21 unresolved discussions.

demos/workspacefactory/factory_controller.js, line 499 [r6] (raw file):
Previously, vicng wrote…
> Nit: Add line above this

Done.

demos/workspacefactory/factory_controller.js, line 511 [r6] (raw file):
Previously, vicng wrote…
> Nit: Add line above this

Done.

demos/workspacefactory/factory_controller.js, line 539 [r6] (raw file):
Previously, vicng wrote…
> Nit: Indentation

Done.

demos/workspacefactory/factory_controller.js, line 559 [r6] (raw file):
Previously, vicng wrote…
> Nit: Indentation

Done.

demos/workspacefactory/factory_model.js, line 255 [r6] (raw file):
Previously, vicng wrote…
> Nit: Add your name to the TODO

Done.

Comments from Reviewable


UI Plugin: move rows to element attribute

It makes more sense to put this on element rather than in default payload.

Won't do for we will upgrade to the new UI plugin.


Exchanges the word engagement for community event across the mircosite.

I think I needed to delete some of the commenting structure to get it to not

conflict 🤔


[wip] Implement Intercity Runner

This is heavily a WIP PR, not even sure where this is heading.
So this is the best description I can give you now:
IT MIGHT BE AWESOME :trollface:

Closing this for now.


how to get filename of embedded resource of an dll(assembly

Issue Title
I'm working on a project where i'm trying to involve a plugin system on aspnetcore2.0 , i want to load views and controllers in seperated assemblies dynamically, thus, load static files in assembly using relative path.  I implemented a EmbeddedResourceFileProvider based on IFileProvider, but failed to load resource file names.
General
I use code like below to find all the resource files embedded in assembly, which translates resourceName like "App.NYMC.UI.Views.Account.Register.js" into relative path like ''Views/Account/Register.js" which can be used in views while accessing that javascript file;
`        internal virtual void AddResources(Dictionary<string, EmbeddedResourceItem> resources)
{
foreach (var resourceName in Assembly.GetManifestResourceNames())
{
if (!resourceName.StartsWith(ResourceNamespace))
{
continue;
}
            using (var stream = Assembly.GetManifestResourceStream(resourceName))
            {
                var filePath = RootPath + ConvertToRelativePath(resourceName);

                resources[filePath] = new EmbeddedResourceItem(
                    CalculateFileName(filePath),
                    stream.GetAllBytes(),
                    Assembly
                );
            }
        }
    }

`
Then i can reference file like this in .cshtml:
<script src="/Views/Account/Register.js"></script>
The problem is that the filePath is invalid when the file name include ".". For example, file "Views/Content/js/site.min.js" got a resource name as 'Views.Content.js.site.min.js'. So i was not able to discover its orginal file name via resourceName.
I tried to get file name like this, but it returned null.
var resourceInfo = Assembly.GetManifestResourceInfo(resourceName);
It seems like that the "." symbol in file name is used to be convert to "_" in .net framework, will we able to  do something like or beyond that so we can identify the file name part in resourceName.

@natemcmaster @danmosemsft

Issue moved to dotnet/corefx #30287 via ZenHub

This is a known limitation for ASP.NET Core 2.0 and older. In ASP.NET Core 2.1, @javiercn wrote a new file provider which will better map embedded resources back to file names. Microsoft.Extensions.FileProviders.ManifestEmbeddedFileProvider.
cc @javiercn - do we have docs for how to enable this?

@natemcmaster I'm not sure if this is covered as part of the razor in class libraries topic.

Ok, let's use https://github.com/aspnet/Docs/issues/6988 to get this feature document.

Should this remain closed then (and the corefx one)?

Yes, I don't think this is an actual issue for corefx. IIUC the problem @tukzer is asking about is how to map from embedded resource name back to the original file path that produced this resource. This is not possible using vanilla .NET runtime and SDK bits. You have to capture this mapping at compile time, which is what @javiercn implemented in the new ManifestEmbeddedFileProvider API.


Add randomness to the spikes behaviour

Motivation
All the spikes were moving together, we needed to add some randomness.
Summary of changes
The next status depends on the value set on the ConfiguratorApp and a random value also depending on that value.
How to test it?
See if the spikes are spawning and hiding separately.
Checklist

[x] Tested the changes locally.
[x] Reviewed the changes on GitHub, line by line.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

Nico you are right, let's merge it and create the issue please.


Clearing an input element makes it not changeable.

I use this line of code to clear the text input field generated by your library around a file input field:
inputFields.value = ''
And after applying this code the input field doesn't change anymore when I choose a file. Its value changes, but not visually.
How to correctly clear this field using your library?

Hi,
sorry for the very late reply. I made some update in the package and added support to reset the file upload field:

Now the default Form reset button (event) is supported.
You can use this code (experimental): componentRegister.getComponentById('myFileInputElement').resetInput_();

Please update your package to 1.7.0 or download it from the https://github.com/Gixx/mdl-components-ext/tree/master/package folder.
Note: the demo website will be updated later...

Thank you very much for the update!


For a 0.18.18 release

#611

Codecov Report

Merging #612 (f7185dd) into master (44d15dc) will decrease coverage by 6.33%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master     #612      +/-   ##
==========================================
- Coverage   86.02%   79.69%   -6.34%
==========================================
  Files          37       37
  Lines        3192     2502     -690
==========================================
- Hits         2746     1994     -752
- Misses        446      508      +62

Impacted Files
Coverage Δ

src/machines.jl
76.25% <46.15%> (-8.27%)
:arrow_down:

src/composition/learning_networks/machines.jl
94.56% <100.00%> (+1.65%)
:arrow_up:

src/composition/learning_networks/arrows.jl
0.00% <0.00%> (-73.34%)
:arrow_down:

src/measures/loss_functions_interface.jl
54.71% <0.00%> (-22.48%)
:arrow_down:

src/interface/data_utils.jl
75.51% <0.00%> (-16.43%)
:arrow_down:

src/utilities.jl
63.15% <0.00%> (-15.28%)
:arrow_down:

src/resampling.jl
76.64% <0.00%> (-12.90%)
:arrow_down:

src/composition/models/_wrapped_function.jl
22.22% <0.00%> (-11.12%)
:arrow_down:

src/data/datasets.jl
89.47% <0.00%> (-10.53%)
:arrow_down:

src/composition/learning_networks/inspection.jl
76.31% <0.00%> (-9.96%)
:arrow_down:

... and 26 more

Continue to review full report at Codecov.

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

@DilumAluthge Wonder if you can help me out here.
Can you explain the documentation error logged here? I've checked General.jl and ScientificTypes version 2.1.0 and 2.1.1 both exist (indeed they are used successfully in the code tests) and yet:
ERROR: Unsatisfiable requirements detected for package ScientificTypes [321657f4]:
 ScientificTypes [321657f4] log:
 ├─possible versions are: 0.1.0-2.0.1 or uninstalled


Cannot find version of gen/packed.%(version)s.js

I'm using Django 1.11 with Whitenoise, and build my assets with:
./manage.py collectstatic
./manage.py assets build --manifest django

Built assets are correctly added to the staticfiles.json manifest.
IF I use the version of the file in the query string, the file is served but Whitenoise does not add far into the future cache headers:
✗ curl -I "http://localhost:8000/static/gen/packed.js?84fed0ca" 2>/dev/null |grep -i cache
Cache-Control: max-age=60, public

When I use the version placeholder:
output='gen/packed.%(version)s.js',

I get an error when trying to render the template:
webassets.exceptions.BundleError: Cannot find version of <Bundle output=gen/packed.%(version)s.js, filters=[<webassets.filter.jsmin.JSMin object at 0x106d45cf8>], contents=('js/vendor/jquery/jquery.min.js', 'js/vendor/bootstrap/bootstrap.min.js', 'js/main.js')>. There is no manifest which knows the version, and it cannot be determined dynamically, because: output target has a placeholder

Versions:
django-assets==0.12
webassets==0.12.1

The reason for not having the correct cache headers is because Whitenoise considers a file immutable when it has the hash in the filename, not the query string. So I really need to make django_assets correctly render the asset URL when the version placeholder is used with a Django manifest.

If I configure the manifest like this in the settings file:
ASSETS_MANIFEST = DjangoManifest()

The templates start to be rendered, however, there's a period added that's wrong:
<link rel="stylesheet" href="/static/gen/packed..f9a66270.css">
<script type="text/javascript" src="/static/gen/packed..84fed0ca.js"></script>


Service Log Aggregation

This may already exist within the Spring Cloud project.  Are there any existing implementations for aggregated logging for services?
Preferably something utilizing the existing Bus.  Using Logstash seems to be popular. The problem I have is installing the Logstash agent takes almost as much RAM on the service as the service itself.

Cloudfoundry already has aggregated logging, so it's not going to be a high priority for Spring Cloud. If you have an implementation in mind feel free to suggest something though (Spring AMQP has log4j and logback support already, so you can probably just use that).

What are your thoughts on using something like example as a basis?
https://github.com/spring-projects/spring-amqp-samples/tree/master/log4j

That's what I meant in my last comment (but we would prefer to use autoconfig to detect the logging system in use and switch to logback if it is available).


bug: update docker image glibc error.

➜  ~ k -n community logs goplus-community-dev-v1-54d99bf645-85ncz
Error relocating /community/cmd/gopcomm/community: fcntl64: symbol not found
➜  ~

Ref: https://nijialin.com/2022/08/15/flagr-debug-fcntl64-error/

Maybe need to update dockerfile base image.

#318

#319


Problems with disabled (xit) tests

We have a spec file with a single xit test in it and it turns into a failed test after applying the plugin:

Sorry, spoke to soon.


Proposed changes to the EVF repository interfaces

renamed the repositories
made the repositories only deal in entities, connections is a Janus Concern, not a EVF concern.

This ready for review?

yesh

@rjkip @arothuis

:+1:


chore: remove doc/tags

This fixes the error when updating Neogit through an auto-indexing plugin manager such as lazy. See https://github.com/NeogitOrg/neogit/discussions/628#discussioncomment-6492754
For the users they may need to once more do the git restore doc/tags dance as docs/tags would be in the state of modified by us, deleted by them or UD

:Lazy sync

Caused by going from master to the remove-tags branch in plugin spec and updating (after removing local dev overrides)

This will be a breaking change, so I added the requested tracking issue for those

@CKolkey what do you think about this breaking change?
auto-session had to do a similar breaking change a few months ago

I'm totally fine with this, but I'm not sure how this is a breaking change. What will users need to do?

The breaking commit is that users may need to go and explicitely restore the repo inside ~/.local/share/nvim/lazy/neogit as lazy will fail to pull as tags may be changed locally due to the aforementioned auto-indexing


Fix back button not working as expected with recent workflows

Description & motivation 💭
When paginating through pages and/or adding search queries on Recent Workflows

clicking "back" in the browser will take you to the previous page or query
clicking Back to Workflows on the workflow detail page will take you back to the most recent workflows page with the most recent queries

This makes it easier to navigate between workflows or investigate multiple workflows.
Screenshots (if applicable) 📸
n/a
Design Considerations 🎨
n/a
Testing 🧪
How was this tested 👻

[x] Manual testing
[ ] E2E tests added
[ ] Unit tests added

Steps for others to test: 🚶🏽♂🚶🏽♀

Go to Namespaces > then Recent Workflows > then enter a query

[ ] Verify clicking the back btn in the browser takes you back to Recent Workflows without the query

Go to Recent Workflows > then enter a query and go to another page > go to the detail page for a workflow

[ ] Verify clicking the Back to Workflows returns to the current Recent Workflows page with the query applied

Go to Recent Workflows > then enter a query and change the number per page > go to the detail page for a workflow

[ ] Verify clicking the Back to Workflows returns to the current Recent Workflows page with the query applied and the same number per page

Go to Recent Workflows > then enter a query > click on Workflows (it should clear the query) > go to the detail page for a workflow

[ ] Verify clicking the Back to Workflows returns to the current Recent Workflows page with no query applied

Checklists
Draft Checklist
n/a
Merge Checklist
n/a
Issue(s) closed

DT-3
DT-291

Docs
Any docs updates needed?
n/a

I think the thing missing here is clearing the page param. When I do a query, page to page=2, then click an execution, then go back, page=2 is perserved. However when i clear the query or click the workflow button, then go to an execution and then go back, page=2 is still perserved.

The one issue that I still see if if you apply some filters and click the Workflow icon in the nav, the query is cleared but the UI still shows the filters applied.


split out the model generation code

We have some extra types like JSONB and ARRAY(NUMERIC) we can use for Postgres, let's take advantage of that.

@pmcnr-hx
So I've split out model generation into seprate modules. Using a common 'default' module for everything other than Postgres right now, as my optimisations are only for Postgres. For others, it's the same.

@championswimmer could this also take care of #53 as you are splitting out by dialect?

@aledalgrande this fixes #53 by making [type] a virtual field

also, @aledalgrande do you think I can target postgres >= 9.4 ??
Will allow using JSONB type
Are you using < v9.4 ?

@championswimmer using latest stable, so it's fine!

Continues to #75


SCSS at-rules are reodered after declarations

SCSS @mixin, @include and @function at-rules shouldn't be grouped with other at-rules as some of them are order-dependent (@function/@mixin) and may lead to properties being overwritten (@include), so I suggest putting them between variables and declarations, like it was done in stylelint-config-clean-order for example.
scss.ts seems doesn't affect order of these as at-rules are matched by order/order rather order/properties-order.

The original idea was actually "not to handle" them at all, leaving them in space (via unspecified), but something went off.
Explicitly supporting these constructs sounds like a good idea.


Skipping tmp_dir2 isn't working if you specify anything after it

i tried ./chia_plot pool-key farmer-key /mnt/970evo 4 7 and this is the error I get:
Invalid <tmp_dir2>: 4 (needs trailing '/' or '\')
If I don't add 4 7 it works fine. Do I specify the same directory or put a / in place of tmp_dir2?

run this

./chia_plot pool-key farmer-key /mnt/970evo/ /mnt/970evo/ 4 7

yeah it was trying to use "4" as tmp_dir2


Python 3 import error

When attempting to import satyr in a python 3.5 environment the following error is returned:
import satyr
 File "<path>/lib/python3.5/site-packages/satyr/scheduler.py", line 58
    raise exc_type, exc_value, traceback
                  ^
SyntaxError: invalid syntax

Satyr isn't tested against python 3 due to mesos incompatibility, see https://issues.apache.org/jira/browse/MESOS-355

Ah, ok I was unaware of this. I had assumed it was compatible as I am using python 3 and the Apache Spark framework on a Mesos cluster

Spark's python support is independent from Mesos module. Spark as a mesos framework is written in Scala.

I'm closing this issue now.


fix(infrastructure): update package-lock file

Codecov Report

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

@@           Coverage Diff           @@
##           master     #254   +/-   ##
=======================================
  Coverage   98.86%   98.86%
=======================================
  Files          23       23
  Lines         969      969
  Branches       98       98
=======================================
  Hits          958      958
  Misses         11       11

Continue to review full report at Codecov.

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

this is stale...closing


Dictionary browser - Hierarchy tab view lost

Open dictionary browser
Select class
Hit hierarchy tab. Nothing

Hierarchy tab hooks were lost in recent commits. Restored them, although the wiring may be slightly different than before.

Hierarchy tab causes hierarchy to be visible in 3.0.6.


Print version

@swilly22 can you please review and merge this?

I'm closing this in favor of https://github.com/RedisLabs/RAMP/pull/20 so that our build scripts are not left in the cold (they point to this branch).


Some modifications to data_dir and Dockerfile

Is your pull request related to a problem? Please describe.
Yes, directories in XDG_DATA_DIR must not start with a dot
Description
USE PERMASIGNER_HOME variable if it's set to use as data directory.
if on linux use XDG_DATA_DIR if it's set, otherwise use ~/.local/share
if on macOS or iOS use ~/Library/Application Support/
cloned repo and docker container will keep defaulting to local output/ folder.
Include only essential files (use .dockerignore) in the docker image, to reduce it's size.
Describe alternatives you've considered
None.
Additional context
Add any other context about the pull request here.
Checklist

[x] The new code was tested.
[x] The code isn't overly messy.

Is this ready to merge?

Manually merged in b13beaa


Update Persian translation

Translation of new strings + some fixes
Please merge it...
Thanks!

LGTM, thanks!


Presentation: Add an enum value required for presentation-components 4.0

@mergifyio backport release/3.7.x

@grigasp is this needed for 3.6 or can it just go into 3.7?

fyi @skirby1996

@grigasp is this needed for 3.6 or can it just go into 3.7?

I'd like it to get to 3.6 too. Is there a reason we can't make a 3.6 patch?

@grigasp is this needed for 3.6 or can it just go into 3.7?

I'd like it to get to 3.6 too. Is there a reason we can't make a 3.6 patch?

we can get in

/azp run


Add callback function support

Adds an argument to MPIPool.map() that mimics the behavior of the stdlib multiprocessing.Pool.map_async(). This is a trivial addition, but I think this will allow for thread-safe file-writing from MPI -- if only the master process does the writing, there is no risk of concurrent writes.

PING

@dfm


Need Clarification on Part 3 Section IV

Awesome guide and was able to finish.  The only step that wasn't clear to me, so I skipped it, was:

Update your CFW SysNAND to the latest version using system settings (if it is not already)

If I were to do this now, I read this as boot up my device and go to Settings and perform a regular system update.  Is that correct?

Correct, It is good practice to do a nand backup before the upgrade though.

Hmmm...so did the upgrade and when I go to System Settings, it shows Sys 11.0.0-33U.  When booting up with R held down, I see the exact same thing. Is this expected?
Followed the installation instructions...am I missing something?

I am unsure what you mean by holding R. The objective of this tutorial is to overwrite your system nand with a modifiable system nand. So since your on 11.0.0-33U. When holding select on boot do you boot into a9lh and do you still have fbi installed on your home menu? If so you have succeeded

Thanks....closing this issue.


reset_camera=Camera.KEEP should preserve clipping planes

Also, leaving the clipping planes tab disables it. This seems like a bug to me but this may or may not be indented so I thought I'd mention it.

not by intention

Hmm, I showed some objects after set_defaults(reset_camera=Camera.KEEP).
Went into clipping and played around.
Use show to show more objects.
For me positions are kept.
Are you doing anything else?

Understood.
Will change from "bug" to "enhancement"

fixed in 2.2.2


New Feature: add LocalStorage module

Does this not break with the current purity of fp-ts to be platform independent? Currently all features can be used either in the browser or node. IMHO this seems to be an extension like browser-ts for me?

Yes, I wasn't sure about this. All modules are optional though, you can import just what you need.

All modules are optional though, you can import just what you need.

Where would you draw the line? This can be interpreted as to collapse the whole fp-ts ecosystem into one package? I am just curious as a (possible) contributor where to contribute which feature.

Where would you draw the line?

Not sure, let's decide together. I'd say
Criteria

no dependencies (for example fp-ts-rxjs is an external package because of the rxjs dependency)
non opinionated (for example monocle-ts is an external package because is somehow opinionated)
the new module is just a (type-safe) wrapper of native APIs (for example LocalStorage)
there's prior art (a Haskell / PureScript / Scala established library)
other?

It is really hard to argue. It is just what I see how most of the libraries are organized. You have some core (what I would see is fp-ts) and an ecosystem around it. Some examples would be react and react-dom, reflex and reflex-dom and so on. It feels like lodash or rambda would add localStorage support. Which seems to make the library as I said "impure" and destroys this "core" aspect.
So an argument I would like to add is "runs in every actual js engine (v8)".

@mlegenhausen

"runs in every actual js engine (v8)"

I'd say  "every part of the library runs in every js engine"

Ok, I'll put up a fp-ts-local-storage package, thanks for the discussion.

@gcanti I like your critera list should we add a CONTRIBUTOR.md file?

@mlegenhausen sure, would you like to send a PR?

@gcanti what would be the prefix/suffix we should use for new packages? Some developers already used the -ts suffix for custom packages like you do for io-ts, monocle-ts and elm-ts. Which seems sometimes misleading when we think about node-ts which has nothing todo with the project.
It would be nice if could define one, so packages can be found easier even when they are not referenced in the readme.

@mlegenhausen Personally I use the suffix -ts to mean an existing library ported to TypeScript and the prefix fp-ts- if is a kind of add-on of fp-ts (for example some bindings)


"Uniforms were never supplied for set (3) at the time of drawing, which are required by the pipeline" after freeing a GPUParticle2D

Godot version
4.1.1.stable.mono
System information
Godot v4.1.1.stable.mono - macOS 13.6.0 - Vulkan (Forward+) - integrated Apple M1 - Apple M1 (8 Threads)
Issue description
I have a Node2D which is responsible for emitting 2 children (GPUParticles2D).
I also added the timer to the parent node and connected its Timeout signal to free the parent.
The error I get is:
E 0:00:07:0555   compute_list_dispatch: Uniforms were never supplied for set (3) at the time of drawing, which are required by the pipeline
  <C++ Error>    Method/function failed.
  <C++ Source>   drivers/vulkan/rendering_device_vulkan.cpp:8026 @ compute_list_dispatch()

I remember having this problem around 6 months ago and don't remember finding an adequate solution. What fixed it back then was changing the GPUParticle2D node type to CPUParticle2D.
Steps to reproduce

Create a scene and add Node2D to it

Add a few GPUParticle2D children (duplicate one a few times favorably)

Add a timer child

Add a script to the parent node

Set the timer WaitTime to the particles' LifeTime

In the _Ready() method, add the listener to the Timeout signal of a timer, which frees the node

In the _Ready() method, emit the particles

Minimal reproduction project
N/A

Maybe related #82257

I can't seem to reproduce this, but the instructions seem a bit vague.
Could you attach a MRP test project? It will give a common base to all testers to see if this only happens on specific systems for example. And yes, there is a good chanche that this is a duplicate of https://github.com/godotengine/godot/issues/82257, but testing this to make sure would still be useful.


[PLUGIN] Create a plugin for Draw Diagramas with Markdown

Load this js on the plugin method siteBodyEnd()
http://support.typora.io/Draw-Diagrams-With-Markdown/

Implemented.
https://plugins.bludit.com/plugin/sequence-diagrams


Caching should be disabled in debug mode

As the LinearTestRunner is not able to record the used files reliably, caching should be disabled.

Resolved in de2e00ad3d3ad334559e91b60140c8add915dd33


(PDB-841) Retire the legacy storeconfigs tool

This commit removes support for the legacy storeconfigs export tool as
we no longer support Puppet <4 in PuppetDB.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-master/135/
Test PASSed.

@ajroetker looking good, I think we just have removed too many tests :-).

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-master/138/
Test PASSed.

transient travis failure ^


Generalize simple_merkle

It would be good to make simple merkle a trait (and/or use generics) to make it take any parameter that can be hashable or to make it easy to swap the underlying hash algo. See comment in: https://github.com/interchainio/tendermint-rs/pull/59#pullrequestreview-319176856

Some precedent:

my objecthash trait: https://docs.rs/objecthash/0.4.1/objecthash/trait.ObjectHash.html
libra crypto: https://github.com/libra/libra/blob/master/crypto/crypto/src/hash.rs#L405

If someone wants to work on this in the meantime, I would suggest the following API for a start:
trait MerkelizedHasher {
    /// Adds data to be merkelized.
    fn add<D: Into<Vec<u8>>>(&self, d: D);
    /// Takes all data and returns the hash.
    fn merkelize<H: Digest>(&self) -> Hash; // the return elem also needs to be sth, generic
}

I'm not saying this needs to be a trait for now. Instead this could be a struct. It would be cool if the (inner/leaf node) Hasher (here called Digest) would be replaceable, and we can collect the data to be merkelized with add (or a smarter name) to make this look more understandable:
https://github.com/interchainio/tendermint-rs/blob/24955d18a99f3f3a846f519b770de5887159a0fa/tendermint/src/block/header.rs#L104-L128

Something else you might want to look at for inspiration is the Rust Hash and Hasher traits:
https://doc.rust-lang.org/std/hash/trait.Hash.html
pub trait Hash {
    fn hash<H>(&self, state: &mut H)
    where
        H: Hasher;

    fn hash_slice<H>(data: &[Self], state: &mut H)
    where
        H: Hasher;
}

https://doc.rust-lang.org/std/hash/trait.Hasher.html
pub trait Hasher {
    fn finish(&self) -> u64;
    fn write(&mut self, bytes: &[u8]);
    // [...]
}


How to implement Configurability via CQL and also conduct performance  testing

Related and being tracked in issue #15

Related to #15

@f-odhiambo is there more information on this issue?

I think we can track this on #81

these look like 2 separate issues?


Add 'Size' field to 'Properties' dialog of ADLS Gen2 blobs

Storage Explorer Version: 1.7.0/20190227.3
Platform/OS Version: Windows 10/ MacOS High Sierra/ Linux Ubuntu 16.04
Architecture: ia32
Commit: 1f111467
Regression From: Not a regression
Steps to Reproduce:

Choose one ADLS Gen2 account -> Open one blob container.
Upload a blob file to it -> Select the uploaded blob then check the system properties on 'Properties' dialog.

Expected Experience:
The 'Size' field shows on the 'Properties' dialog.
Actual Experience:
No 'Size' field shows on the 'Properties' dialog.

Closing as won't fix. Size is available in the editor.


There should be something like that for Crystal, too!
The technical part should not be too complicated, the playground already has most of the functionality for interactive coding and it could easily be integrated into an HTML site with the tutorial texts and example code. A static site would probably be sufficient for that.
Creating a great tour guide will be much more challenging (as everything with documentation... 😄 ). The Gitbook offers a good starting point, and the Go tour could be a guide, as well as similar resources (I've found https://www.learnrubyonline.org/ so far, which also exists for some other languages).

http://tryruby.org/ is another great one that I have shown to multiple people to get them started with Ruby (and even programming in general).

I think it's a great idea and I'd say go for it! Count on us to link it from crystal-lang.org once it begins shaping up, setting a subdomain for it, etc

I just stumbled upon https://github.com/crystal-lang/crystal-presents which has the technical foundation already covered! And looks pretty nice, too, @bcardiff

a very good "tour" for me was the ruby koans these could probably relatively easily be transferred to crystal although I do not know how much they depend on rake specifics (nothing prevents crystal from using ruby rake though)

@GloverDonovan - For a beginning outline, the talk that @asterite gave (maybe a year or two ago) would be a good starting point. I have lost the link to the video, but I'm sure someone has it or maybe @asterite can provide the slides or code used for the presentation.

@marksiemers Thanks for the reply. I plan to have one main idea covered in each section, such as a section on modules and another on classes, each with their own individual subsections / exercises. A link to the video would be great; I'd appreciate some help on an outline.
I want to try test-driven learning (i.e. run a spec file every time someone submits their code) so that end-users can see exactly how their code behaves when given a few test cases.
One roadblock for me though is that the crystal playground isn't sandboxed by default (i.e. you can access system files, etc.). I'm not sure how I'd fix this for a production server.

Did some testing and it looks like the Crystal play website uses playpen to sandbox things. How reliable has this been so far, and is there any reason to prefer playpen over firejail?

Here is the video: https://vimeo.com/191351066

I want to try test-driven learning (i.e. run a spec file every time someone submits their code) so that end-users can see exactly how their code behaves when given a few test cases.

we have carc.in

The source for that is at https://github.com/jhass/carc.in and it uses playpen, as @GloverDonovan already mentioned.
I'm sure we can find a way to either run the Playground in a sandboxed environment as well - or just simply use carc.in as a backend for a publicly hosted playground.
In fact, the technical implementation is really not important right now. The main issue is putting together great content.
For now, we can just use the playground as is and only run it locally.

I agree with @straight-shoota, getting the content together should be the highest priority - even if that means only running locally (similar to running go's tour locally).
In the future, I like some of the features that a local Crystal play server provides over carc.in

Showing the variable values and types for each line of code
Having a modal window for loops, so you can see what happened for each iteration

I also like the workbook functionality, but I think that is not as applicable to an online version unless crystal wants to build their own repl.it

The content should largely be written long-form tutorials.
If there is a walkthrough interactive thingy it should come after the "book-style" thing. Long-form text is one thing everyone can agree on, I know that I would skip any interactive tutorials I found and search for an introductory text. Others are different but the basics please first.

I've done a lot of programming tutorials online, but just want to say I really enjoyed Haskell's.  The words they use make the experience tens time better and increase the developer's confidence (which inherently increases the developer's learning ability).
Examples:

First Time's a Charm
Well done, you typed it perfect!
Hi there, chris! That's a pretty name. Honest. You're getting the hang of this!
Great, you made a list of numbers! If you win we'll split the winnings, right?

I'm not saying we need to have this exact level of happiness during an interactive tour, but I just want to re-iterate that words of encouragement can really make a difference.

I would love to work on this!

Great, feel free to check out forum.crystal-lang.org/t/interactive-tutorial/3019 and crystal-lang/crystal-book#484


Requested modifications to notifications

It would be nice if the notifications included:

Links to the jenkins web UI.
The name of the slave that builds are running on.

It already includes links to the Jenkins web UI.  It's likely you do not have it configured correctly in your settings.  I'm not sure Jenkins shares what build slave a build is running on.  I've always gotten around this by adding hostname to my build scripts.

@charlesmchen were you able to figure out the configuration so that Jenkins links show up in the slack post?

@samrocketman I did not see any way to include a clickable link to jenkins UI in the slack message, can you please give a pointer to docs or a screen shot or instructions?
I was trying to set it up and got notifications like this:
blah-dev - #751 Failure after 26 min (</job/blah-dev/751/|Open>)

with nothing clickable

Oh I was close, lol.
Jenkins > Configuration > Global Slack Notifier Settings > Build Server URL
Working fine now :-)

Great.  That should probably be updated from / to something like https://yourbuildserver.com/ or something like that as an example.

Jenkins > Configuration > Global Slack Notifier Settings > Build Server URL seems to have been removed
closing


"No compatible code running on thread" error after "CameraView_OnDetected" finishes

Hi, I've been having this issue, and it seems to happen at the end of the method CameraView_OnDetected.
The error says "No compatible code running on thread", then it says "System.NullReferenceException: 'Object reference not set to an instance of an object.'"

it errors out after I step after this breakpoint.

Any ideas on what it could be?

Hi @Scott-VentsUS ,
I searched from Google and it looks like some bugs from Visual Studio and Android.
Which version are you using and does it work when you are not debugging?

Hi @Scott-VentsUS ,
I searched from Google and it looks like some bugs from Visual Studio and Android.
Which version are you using and does it work when you are not debugging?

Microsoft Visual Studio Community 2019
Version 16.9.3
VisualStudio.16.Release/16.9.3+31129.286
Microsoft .NET Framework
Version 4.8.04084
Installed Version: Community
Visual C++ 2019   00435-60000-00000-AA632
Microsoft Visual C++ 2019
ADL Tools Service Provider   1.0
This package contains services used by Data Lake tools
ASA Service Provider   1.0
ASP.NET and Web Tools 2019   16.9.693.2781
ASP.NET and Web Tools 2019
ASP.NET Core Razor Language Services   <IP_ADDRESS>2521+5741df381174d72f08e3632bb99f52e8635b6a1a
Provides languages services for ASP.NET Core Razor.
ASP.NET Web Frameworks and Tools 2019   16.9.693.2781
For additional information, visit https://www.asp.net/
Azure App Service Tools v3.0.0   16.9.693.2781
Azure App Service Tools v3.0.0
Azure Data Lake Node   1.0
This package contains the Data Lake integration nodes for Server Explorer.
Azure Data Lake Tools for Visual Studio   2.6.1000.0
Microsoft Azure Data Lake Tools for Visual Studio
Azure Functions and Web Jobs Tools   16.9.693.2781
Azure Functions and Web Jobs Tools
Azure Stream Analytics Tools for Visual Studio   2.6.1000.0
Microsoft Azure Stream Analytics Tools for Visual Studio
C# Tools   3.9.0-6.21160.10+59eedc33d35754759994155ea2f4e1012a9951e3
C# components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.
Common Azure Tools   1.10
Provides common services for use by Azure Mobile Services and Microsoft Azure Tools.
Cookiecutter   16.9.21026.1
Provides tools for finding, instantiating and customizing templates in cookiecutter format.
Extensibility Message Bus   1.2.6 (master@34d6af2)
Provides common messaging-based MEF services for loosely coupled Visual Studio extension components communication and integration.
IntelliCode Extension   1.0
IntelliCode Visual Studio Extension Detailed Info
Microsoft Azure HDInsight Azure Node   2.6.1000.0
HDInsight Node under Azure Node
Microsoft Azure Hive Query Language Service   2.6.1000.0
Language service for Hive query
Microsoft Azure Stream Analytics Language Service   2.6.1000.0
Language service for Azure Stream Analytics
Microsoft Azure Stream Analytics Node   1.0
Azure Stream Analytics Node under Azure Node
Microsoft Azure Tools   2.9
Microsoft Azure Tools for Microsoft Visual Studio 2019 - v2.9.40218.1
Microsoft Continuous Delivery Tools for Visual Studio   0.4
Simplifying the configuration of Azure DevOps pipelines from within the Visual Studio IDE.
Microsoft JVM Debugger   1.0
Provides support for connecting the Visual Studio debugger to JDWP compatible Java Virtual Machines
Microsoft Library Manager   2.1.113+g422d40002e.RR
Install client-side libraries easily to any web project
Microsoft MI-Based Debugger   1.0
Provides support for connecting Visual Studio to MI compatible debuggers
Microsoft Visual C++ Wizards   1.0
Microsoft Visual C++ Wizards
Microsoft Visual Studio Tools for Containers   1.1
Develop, run, validate your ASP.NET Core applications in the target environment. F5 your application directly into a container with debugging, or CTRL + F5 to edit & refresh your app without having to rebuild the container.
Microsoft Visual Studio VC Package   1.0
Microsoft Visual Studio VC Package
Mono Debugging for Visual Studio   16.9.7 (df23ba6)
Support for debugging Mono processes with Visual Studio.
Node.js Tools   1.5.21130.2 Commit Hash:dacc0e71a5e060cdcab466a301cacea359c1b6da
Adds support for developing and debugging Node.js apps in Visual Studio
NuGet Package Manager   5.9.0
NuGet Package Manager in Visual Studio. For more information about NuGet, visit https://docs.nuget.org/
Office Developer Tools for Visual Studio   16.0.30502.00
Microsoft Office Developer Tools for Visual Studio
ProjectServicesPackage Extension   1.0
ProjectServicesPackage Visual Studio Extension Detailed Info
Python   16.9.21026.1
Provides IntelliSense, projects, templates, debugging, interactive windows, and other support for Python developers.
Python - Conda support   16.9.21026.1
Conda support for Python projects.
Python - Django support   16.9.21026.1
Provides templates and integration for the Django web framework.
Python - IronPython support   16.9.21026.1
Provides templates and integration for IronPython-based projects.
Python - Profiling support   16.9.21026.1
Profiling support for Python projects.
SQL Server Data Tools   16.0.62103.10080
Microsoft SQL Server Data Tools
Test Adapter for Boost.Test   1.0
Enables Visual Studio's testing tools with unit tests written for Boost.Test.  The use terms and Third Party Notices are available in the extension installation directory.
Test Adapter for Google Test   1.0
Enables Visual Studio's testing tools with unit tests written for Google Test.  The use terms and Third Party Notices are available in the extension installation directory.
ToolWindowHostedEditor   1.0
Hosting json editor into a tool window
TypeScript Tools   16.0.30201.2001
TypeScript Tools for Microsoft Visual Studio
Visual Basic Tools   3.9.0-6.21160.10+59eedc33d35754759994155ea2f4e1012a9951e3
Visual Basic components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.
Visual C++ for Cross Platform Mobile Development (Android)   16.0.31025.194
Visual C++ for Cross Platform Mobile Development (Android)
Visual F# Tools   16.9.0-beta.21102.9+7ce7132f1459095e635194d09d6f73265352029a
Microsoft Visual F# Tools
Visual Studio Code Debug Adapter Host Package   1.0
Interop layer for hosting Visual Studio Code debug adapters in Visual Studio
Visual Studio Container Tools Extensions   1.0
View, manage, and diagnose containers within Visual Studio.
Visual Studio Tools for CMake   1.0
Visual Studio Tools for CMake
Visual Studio Tools for Containers   1.0
Visual Studio Tools for Containers
VisualStudio.DeviceLog   1.0
Information about my package
VisualStudio.Foo   1.0
Information about my package
VisualStudio.Mac   1.0
Mac Extension for Visual Studio
Workflow Manager Tools 1.0   1.0
This package contains the necessary Visual Studio integration components for Workflow Manager.
Xamarin   <IP_ADDRESS>3 (d16-9@1bba9e0)
Visual Studio extension to enable development for Xamarin.iOS and Xamarin.Android.
Xamarin Designer   <IP_ADDRESS>6 (remotes/origin/d16-9@fdbf64026)
Visual Studio extension to enable Xamarin Designer tools in Visual Studio.
Xamarin Templates   16.9.68 (8e9b569)
Templates for building iOS, Android, and Windows apps with Xamarin and Xamarin.Forms.
Xamarin.Android SDK   <IP_ADDRESS> (d16-9/877f572)
Xamarin.Android Reference Assemblies and MSBuild support.
Mono: 5e9cb6d
Java.Interop: xamarin/java.interop/d16-9@54f8c24
ProGuard: Guardsquare/proguard/v7.0.1@912d149
SQLite: xamarin/sqlite/3.34.1@daff8f4
Xamarin.Android Tools: xamarin/xamarin-android-tools/d16-9@d210f11
Xamarin.iOS and Xamarin.Mac SDK   <IP_ADDRESS> (3836759d4)
Xamarin.iOS and Xamarin.Mac Reference Assemblies and MSBuild support.
Android version 11
And I'll be honest, I don't know how to put the app on the phone without debugging...

Okay, I was able to figure it out. Nothing wrong with this code, it's my own code. I double checked the output and found that there was an exception in my code that crashed the app. If anyone else is finding this from google, look at your output code very carefully. It should tell you what line of code that broke, use a try catch, then you should be able to find it.
Thanks!


fix typos

"Equitable" => "Equatable" ? 🤔
Checklist

[x] All tests are passed.
[ ] Added tests.
[ ] Documented the code using Xcode markup.
[x] Searched existing pull requests for ensure not duplicated.

Description
Thank you all for creating such an awesome library!
I fixed some typos on document.

Thanks @HAKASHUN  👍


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 873 ms

Resolved: Bofrost.lu performance has improved in ca5537e after 6 minutes.


MultiPlatform: why using NSUrlConnection instead of NSUrlSession for iOS?

Concerning the MultiPlatform project:
https://www.apollographql.com/docs/android/essentials/get-started-multiplatform/
It says that the iOS client uses NSUrlConnection
which has been deprecated by Apple for NSUrlSession
I am writing here, as in the URL, it appears as part of the Android project.
I had written in the Apollo-iOS project:
https://github.com/apollographql/apollo-ios/discussions/1613#discussioncomment-266190
They told me they are using NSUrlSession there
is there any reason why you use NSUrlConnection for the MultiPlatform project?
is there a plan to use NSUrlSession soon?

Yikes, it's most likely a leftover of ancient times. The multiplatform code uses NSUrlSession, see https://github.com/apollographql/apollo-android/blob/main/apollo-runtime-kotlin/src/iosMain/kotlin/com/apollographql/apollo/network/http/ApolloHttpNetworkTransport.kt#L84
I'll fix the doc in 2 min! Thanks for reporting this!

Yikes, it's most likely a leftover of ancient times. The multiplatform code uses NSUrlSession, see https://github.com/apollographql/apollo-android/blob/main/apollo-runtime-kotlin/src/iosMain/kotlin/com/apollographql/apollo/network/http/ApolloHttpNetworkTransport.kt#L84
I'll fix the doc in 2 min! Thanks for reporting this!

Closed by https://github.com/apollographql/apollo-android/pull/2847

Closed by https://github.com/apollographql/apollo-android/pull/2847

Great, that was quick!
But something strange happens in the doc page now.
I tested with both Chrome and Safari on MacOS.
When I load this page:
https://www.apollographql.com/docs/android/essentials/get-started-multiplatform/
the link text is "NSUrlSession" for 1 second, and then it becomes "NSUrlConnection"
while the link url is always correct: https://developer.apple.com/documentation/foundation/nsurlsession

Great, that was quick!
But something strange happens in the doc page now.
I tested with both Chrome and Safari on MacOS.
When I load this page:
https://www.apollographql.com/docs/android/essentials/get-started-multiplatform/
the link text is "NSUrlSession" for 1 second, and then it becomes "NSUrlConnection"
while the link url is always correct: https://developer.apple.com/documentation/foundation/nsurlsession

the link text is "NSUrlSession" for 1 second, and then it becomes "NSUrlConnection"

Interesting, I have the same thing... Let's keep this ticket open as a reminder to test this again in a couple of hours. Maybe Gatsby/our doc engine has a cache somewhere that needs to be updated. Let's try again a bit later.

the link text is "NSUrlSession" for 1 second, and then it becomes "NSUrlConnection"

Interesting, I have the same thing... Let's keep this ticket open as a reminder to test this again in a couple of hours. Maybe Gatsby/our doc engine has a cache somewhere that needs to be updated. Let's try again a bit later.

I'm only getting the NSUrlSession, not NSUrlConnection.

I'm only getting the NSUrlSession, not NSUrlConnection.

Looks good for me now.

Looks good for me now.


Improve naming of internal transaction methods

Improve naming of internal transaction methods
Description
This PR improves how we name internal transaction names. The names used were confusing.
Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

Codecov Report

Merging #714 (7617f93) into main (5361f71) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #714   +/-   ##
=======================================
  Coverage   92.36%   92.36%
=======================================
  Files          52       52
  Lines       11792    11779   -13
=======================================
- Hits        10892    10880   -12
+ Misses        900      899    -1

Impacted Files
Coverage Δ

src/execution/execution_entry_point.rs
97.35% <ø> (ø)

src/syscalls/business_logic_syscall_handler.rs
77.56% <ø> (ø)

src/syscalls/syscall_response.rs
74.28% <ø> (ø)

src/execution/mod.rs
97.55% <100.00%> (-0.03%)
:arrow_down:

src/transaction/declare.rs
98.63% <100.00%> (-0.02%)
:arrow_down:

src/transaction/declare_v2.rs
86.18% <100.00%> (+0.11%)
:arrow_up:

src/transaction/deploy.rs
99.02% <100.00%> (-0.03%)
:arrow_down:

src/transaction/deploy_account.rs
96.87% <100.00%> (-0.05%)
:arrow_down:

src/transaction/invoke_function.rs
99.31% <100.00%> (-0.01%)
:arrow_down:

src/transaction/l1_handler.rs
96.94% <100.00%> (-0.03%)
:arrow_down:

... and 2 files with indirect coverage changes


new version since july cant run

I cant use the command line "python run ...." because since July, maybe numpy 2.0 or opencv-python seem  cause problem.
I shot a image of 650 by 920 (https://www.wallsauce.com/uploads/wallsauce-com/images/blogs/Dark-Kitchen.jpg)
error: OpenCV(4.10.0) :-1: error: (-5:Bad argument) in function 'resize'

Overload resolution failed:

Can't parse 'dsize'. Sequence item with index 0 has a wrong type
Can't parse 'dsize'. Sequence item with index 0 has a wrong type

[113](file:///F:/projets/poc_cv_2024/test/depthAnything/Depth_Anything_V2/depth_anything_v2/util/transform.py:113) sample["image"] = cv2.resize(sample["image"], (width, height), interpolation=self.__image_interpolation_method)
I already use and that work  with the code before July

Did you slove it？


Fixing Bug: Delete Menu Item

Delete menu item is not changing the form action url.

Fixes #2503

Not working for me.
It doesn't replace anything and thus redirects to /admin/menus/X/item/__id

Ya that's fine too.. :-)


Etcd api-gateway watch error

Currently, watching using Node Webhooks has an issue when using Etcd's api-gateway, specifically with watching. Using the watch api request used to work but stopped randomly after working with the node error:

MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 connection listeners added. Use emitter.setMaxListeners() to increase limit

Change the following code in /client/server.js to that of how Etcd's doc uses watching to see that nothing is returned:
This: webHooks.add('etcd', http://${etcd}/v3beta/kv/range)
To this: webHooks.add('etcd', http://${etcd}/v3beta/watch)

To understand how the client works a little better, refer to the docs


feat: Improving the arrangement of the Gallery part in the main page.

The Gallery part is having space to the left. Which is not looking good.
It can be improved by displaying three images overthere.
Screenshot

@kunalkeshan , can i work on this feature.

Hey @Sahil-K123 sure, thanks!
I thought it was a one time issue and didn't reload and check because it was not showing up in gallery page.
Thanks for brining this up, appreciate the help.


JavaScript heap out of memory

Issue Name
JavaScript heap out of memory
Summary
Our docker container restarted due to this issue. Are there anyone who run into the same issue. Could increasing nodejs memory help or there are underlying problems?
Steps to Reproduce
I don't know
Current Behaviour
Container restarts when this happens.
Expected Behaviour
Extra Details

[1:0x55f635c23a80]  1350946 ms: Mark-sweep 1394.4 (1426.3) -> 1393.9 (1426.3) MB, 530.0 / 0.0 ms  (average mu = 0.093, current mu = 0.005) allocation failure scavenge might not succeed
[1:0x55f635c23a80]  1351741 ms: Mark-sweep 1394.6 (1426.3) -> 1394.2 (1426.8) MB, 793.0 / 0.0 ms  (average mu = 0.044, current mu = 0.002) allocation failure scavenge might not succeed
FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory

<--- JS stacktrace --->

==== JS stack trace =========================================

    0: ExitFrame [pc: 0x1976a78dbe1d]
Security context: 0x04710871e6e9 <JSObject>
    1: deserializeObject(aka deserializeObject) [0x3b25d93ed411] [/usr/src/app/node_modules/bson/lib/bson/parser/deserializer.js:~43] [pc=0x1976a80a29b6](this=0x0d999a6026f1 <undefined>,buffer=0x3091f6c694f9 <Uint8Array map = 0x396de8b569a1>,index=13580717,options=0x3091f6c694c9 <Object map = 0x1a08c5a7b8e1>,isArray=0x0d999a6028c9 <true>)
    2: deserializeO...

Yes, that can happen, when a user (i.e. an API call) requires a lot of data. At PSI we therefore start the API server with the following flag: "node --max-old-space-size=4096 ."  This will make the problem appear less likely. However this is not a "real" solution. Probably we would need to have a default "limit" parameter on the API server side to avoid extensive data queries to the underlying Mongo instance.

I see. I will increase the size and I agree that maybe we should prevent the users from such queries. I have also noticed that for example the queries for metadataKeys goes through all the datasets to gather unique metadataKeys. I am not sure how the search is performed (if is is performed incrementally) but maybe we could save these unique metadataKeys in database and updating them regularly instead of searching in real time.

Yes, I agree. In fact for this purpose there is startup parameter that limits this number in config.local.js:   metadataKeysReturnLimit: 100,  metadataParentInstancesReturnLimit: 10,


Use old git command

Some versions of git don't have git-url (case in point on Debian 8, no
get-url on packaged git).
Use old command in getting URL.

Yup, that was the stackoverflow link I used :+1:

LGTM!


failed to allocate a guard page

Hello,
I'm trying to use this crate but it give me this error:
thread '<unnamed>' panicked at 'failed to allocate a guard page', src/libstd/sys/unix/thread.rs:336:17
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace.
fatal runtime error: failed to initiate panic, error 94375168
error: valgrind command failed

I'm using:

FreeBSD 11.2
rustc 1.39.0
valgrind 3.10.1

Hello,
can you please supply the following information:

is this reproducable
which crate are you analyzing
the whole command line used
The output of the run with RUST_BACKTRACE=1
Then I hope to fix that bug.

Hello, sorry for the huge delay!

Yes it happens with everything
I can generate the bug with the hello world provided by cargo when you run cargo new [name]
cargo valgrind

% export RUST_BACKTRACE=1
% cargo new hello
    Created binary (application) `hello` package
% cd hello
% cargo run
   Compiling hello v0.1.0 (/home/irevoire/hello)
    Finished dev [unoptimized + debuginfo] target(s) in 0.62s
     Running `target/debug/hello`
Hello, world!

% cargo valgrind
    Finished dev [unoptimized + debuginfo] target(s) in 0.02s
   Analyzing `target/debug/hello`
thread '<unnamed>' panicked at 'failed to allocate a guard page', src/libstd/sys/unix/thread.rs:336:17
stack backtrace:
   0: <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt
   1: core::fmt::write
   2: std::io::stdio::_print
   3: rust_oom
   4: rust_oom
   5: std::panicking::rust_panic_with_hook
   6: std::panicking::begin_panic_fmt
   7: std::rt::lang_start_internal
   8:
   9:
  10:
  11: <unknown>
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
fatal runtime error: failed to initiate panic, error 94376704
error: valgrind command failed

This bug also happens with only valgrind so I’ll move this issue to the rust-lang repository.


Cannot get commit history of a special file with git log --color | emojify

For example when I run git log --color | emojify README.md I should get the commit history of the README.md but it only output README.md.

Does git log --color README.md | emojify work as you'd like to?

It worked, thanks !!


Gameplay

Here some ideas of mine:
Goal/Story
We could use a weather related story, like:

Earth is getting colder because all the greenhouse gasses got avoided by all governments and now there is only one cat that is able to get [some kind of fancy scientific device here] from the future to use it in the past to save the world!

Goal: travel to the future to bring sth to the past and apply it.
With this we could go from warmer natural forces (dust/sand storm) in the past to colder forces (snow/blizzard in the future) or we just use the exactly inverted story and make it more realistically with what is actually going on (more meaningful maybe, if we want that?).
What natural forces should we use?

dust/sand
wind /leaves
rain
water/waves
snow
ice cubes

Should there be some that represent rather the past or the future?
Which one is easier/harder to avoid as a cat? (I.e. which one should be for lower or higher "levels"?)
Game Controls

tapping/clicking gives the cat energy
natural forces appear randomly while walking
the forces steal energy from the cat if they are not avoided by jumping or ducking to avoid/weaken collisions
if the cat runs out of energy it is send to the past (some years/decades?)
to travel to the future the cat must have enough energy
in order to collect energy the cat could have e.g. 3 batteries in her body (because it’s a cute little cyber cat)
each battery stores a certain amount of energy that could be moved from your cat energy to this storage (which will remove the energy from your cat of course and make it harder to stay in the present after storing energy to get to the future... wow, sounds great doesn’t it? ;))
as soon as you filled all batteries you travel to the future and fight against the next/stronger natural forces
if you reached the future you will pick up the device and bring it to the past by just being a lazy cat :)

@kennethlarsen What do you think? What are your ideas regarding the gameplay and goal?


fix: refactor ibc-derive to handle context with generic types and projects dependent on ibc-core

Closes: #910
Closes: #999

PR author checklist:

[X] Added changelog entry, using unclog.
[ ] Added tests.
[X] Linked to GitHub issue.
[ ] Updated code comments and documentation (e.g., docs/).
[ ] Tagged one reviewer who will be the one responsible for shepherding this PR.

Reviewer checklist:

[ ] Reviewed Files changed in the GitHub PR explorer.
[ ] Manually tested (in case integration/unit/mock tests are absent).

If I have a context with trait-bounded generics as following:
struct MyIbcContext<T: MyTrait> { ... }

How do I the trait-bound its generics while deriving IbcClientState? The following doesn't work.
#[derive(IbcClientState)]
#[validation(MyIbcContext<T: MyTrait>)]
#[execution(MyIbcContext<T: MyTrait>)]
enum AnyClientState { ... }

How do I the trait-bound its generics while deriving IbcClientState? The following doesn't work.

What about this?
#[derive(IbcClientState)]
 #[validation(MyIbcContext<T>)]
 #[execution(MyIbcContext<T>)]
 enum AnyClientState { ... }

The trait bounds are needed when we are deriving the ClientState{Validation, Execution} :sweat_smile:

The trait bounds are needed when we are deriving the ClientState{Validation, Execution} 😅

Ahh, that's right. That's a new case!


Changes are not saved in "Account Settings" page if accessed from My Profile -> Manage user

Steps to reproduce:

Log in to AMO-dev page https://addons-dev.allizom.org/en-US/firefox
Go to "Username"->My Profile->Manage user
Make a change and click "Update Account" blue button

Expected results:
The changes are saved.
Actual results:
The changes are not saved.
Notes/Issues:
If the "Account Settings" page is accessed directly from the menu, the changes are saved and a "Profile updated" message is displayed on top of the page.
Verified on FF43(Win 7). Issue is reproducing on AMO-dev
Screencast for this issue: http://screencast.com/t/J9jquTLwexvj

https://sentry.prod.mozaws.net/operations/olympia-dev/group/161890/

Since its on -dev, possibly related to FxA?

This is actually the admin edit user form but it is being used to edit the currently logged in user. I'm not sure if that's a valid use case or not but it is currently allowed. The form isn't actually submitted because the "Admin log" field hasn't been filled out so this is working as intended. That being said the other required fields are marked as required so it is clear that they should be filled in.
Once you fill in the Admin log field you get the Sentry error but that is unrelated to the screencast.

I submitted https://github.com/mozilla/olympia/pull/1171 to mark the admin_log field as required.
I'm not sure what's up with the error. I filed https://github.com/mozilla/olympia/issues/1172 to fix it.

Looks good, thanks! Do we want a test for this specific use case?
r+wc


CRA and Airbnb require conflicting versions of eslint-plugin-jsx-a11y

This makes it difficult to use CRA with eslint-config-airbnb@16, which is very popular with React development.
Is this a bug report?
Yes.

Can you also reproduce the problem with npm 4.x?

Yes.
Which terms did you search for in User Guide?

Airbnb
accessibility
a11y

Environment

node -v: 8.9.1
npm -v: 5.6.0
npm ls react-scripts (if you haven’t ejected): 1.0.17

Then, specify:

Operating system: macOS 10.13.1

Steps to Reproduce

Install create-react-app.
Install eslint-config-airbnb@16.

Expected Behavior

CRA should support eslint-plugin-jsx-a11y@16.
Actual Behavior

These errors are mutually exclusive but I'm including them both for convenience and debugging purposes.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.

This might be related to #3418.

We don’t officially support using ESLint configs other than the default one without ejecting. Depending on whether our versions are in sync, it may or may not work, but we won’t commit to always supporting it. In fact I’d say we disagree with some decisions there and think they’re not great universal rules for people developing React apps.
Are there particular reasons you need another config? We’ve tried to include rules that would be valuable for everyone, but exclude any styling rules (so you can use tools like Prettier to automatic format your code).

If there are specific rules that you feel are missing in our default config, please raise a separate issue and we will discuss adding those.

(That said you can eject and use any config if that’s your strong preference.)

Crossposting my comment on #3418:

I need eslint in all my projects for CI purposes and I would prefer not to be able to run a different version of it or at least have a different config using the same version of eslint. In my opinion, a boilerplate should not limit my ability to use a popular tool, and if there's anything I can do to help I'd be happy to contribute.

My issue is not with CRA's default eslint config (in fact I think it's great, I just want stricter configs for my projects.) I would prefer to be able to any eslint config and use CRA only as a build tool, not as a linter, if possible, since I personally see it as a build tool that shouldn't limit my ability to use other JS ecosystem tools (except for Jest since it's still pretty configurable in CRA and it is sort of a build tool).
I know this involves more of a maintenance burden, but do you think it would be worth it to use some sort of workaround for eslint users (alternatively I would fork/eject my own shared config based off CRA or use a different tool)? For example, eslint could simply be inlined as a build-time dependency of CRA, which IMO would make the problem vanish (it would not be in the package.json, which shouldn't be an issue anyway since the eslint config is meant to be used internally, not with separate instances of eslint).
Also I appreciate the advocation of Prettier and while it does make sense for many projects to only use Prettier for styling and only use eslint to catch bugs (like with CRA's default config), there are many useful eslint rules that are outside of Prettier's domain of simply reformatting from the AST while being way too opinionated to be moved into CRA's default configs. For example, the eslint-plugin-import/import rule with absolute-first enforces that absolute imports are always before relative imports. This is something that's really helpful for making larger codebases consistent, but it would be too restrictive to add this to the default eslint config and it's not strictly a syntax issue so it can't be handled by Prettier (at least as far as I'm aware).

a boilerplate should not limit my ability to use a popular tool

We're not a boilerplate. 😄
If you'd like to treat react-scripts as a "boilerplate", that involves ejecting -- at which point you can tune things however you please.

I can't use eslint with CRA even if I execute it separately with a different config?

Sure you can! Due to eslint limitations, you simply can't install it side-by-side to react-scripts; try a directory higher.

I know this involves more of a maintenance burden, but do you think it would be worth it to use some sort of workaround for eslint users (alternatively I would fork/eject my own shared config based off CRA or use a different tool)?
-snip-

We're open to making this work by default [installing eslint next to react-scripts], but we don't have the time to work on that right now. I'd be open to a pull request for that.
To be clear: in no way should any custom eslint configuration or rules have an effect on our development or build process; it simply shouldn't break it / prevent the user from having their own lint step.

For example, the eslint-plugin-import/import rule with absolute-first enforces that absolute imports are always before relative imports. This is something that's really helpful for making larger codebases consistent, but it would be too restrictive to add this to the default eslint config and it's not strictly a syntax issue so it can't be handled by Prettier (at least as far as I'm aware).

You can add such individual rules to your .eslintrc and then run eslint yourself manually. Of course, this wouldn't work if any plugin versions are incompatible with the ones we use, but I don't see a solution here—we can support only one major version at a time. This is why including other presets (like Airbnb) is more challenging because now there are many plugins which all have to match in versions.


CI: exclude all vendor/*.md's from pullaprove

We now have multiple sub-projects that have their own vendor
trees with .md files in them we do not want docs review to happen
for (as they are out of our control). Thus, exclude all vendor dirs
through the whole tree for .md file review.
Fixes: #209
Signed-off-by: Graham Whaley<EMAIL_ADDRESS>

Related to https://github.com/clearcontainers/tests/pull/204#issuecomment-316402388
As far as I can find pullapprove uses '*nix style matching', from this library:
https://docs.python.org/2/library/fnmatch.html
so, I think we need both those vendor exclusion lines, and believe the second double * line should work.
@devimc what do you think??
It's slightly difficult to test this without creating some python around that library or meddling with some .md files to check it out. We might just have to slide it in and see how it goes over time??

lgtm

lgtm

re-pushed with comments for the delectation of @jodh-intel

lgtm


feat: Moving Databricks health checks to geh-core

Description

References

#0000

Codecov Report

Merging #244 (0ae837a) into main (7429e93) will increase coverage by 0.35%.
Report is 9 commits behind head on main.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##             main     #244      +/-   ##
==========================================
+ Coverage   71.22%   71.58%   +0.35%
==========================================
  Files         199      199
  Lines        3566     3575       +9
  Branches      441      441
==========================================
+ Hits         2540     2559      +19
+ Misses        881      871      -10
  Partials      145      145

Flag
Coverage Δ

databricks
86.73% <87.50%> (+5.98%)
:arrow_up:

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

Files Changed
Coverage Δ

...lStatementExecution/Internal/SqlStatementClient.cs
82.43% <ø> (-0.47%)
:arrow_down:

...ndencyInjection/SqlStatementExecutionExtensions.cs
86.36% <86.36%> (ø)

...cks/source/Common/AppSettings/DatabricksOptions.cs
100.00% <100.00%> (ø)

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


[Request] - Maintain a changelog

Hi there.
Love the project. You've probably saved me a collective year of time/headache, so thank you so much for that.
I'm wondering if you might be willing to maintain a changelog for this project? It would be nice to be able to come to the repo when new releases are made and quickly see what's changed without diving into the commit history.
Thanks so much either way

Sure, sounds doable. I think it'll (more or less) be a mirror of commit history, but if it's helpful, it shouldn't be too much effort.
I'll add a changelog on the Wiki and add a link to it from the main repo.
Though this is probably not going to happen for a month or so cause I'm pretty busy. I'll come back to this ~ Dec 20.

Woo, done! https://github.com/anishathalye/dotbot/wiki/Changelog


Need to replace Plant.id graphic and copyright date

The logo for 'Flower Checker' needs to be removed and replaced with the Plant.ID logo. Make the following text edits too:

Here is logo to replace

https://github.com/TreeMama/iSeaTree-React-Prototype/pull/560


fixed issue "Remove-FinOpsHub fails when there's only one resource"

🛠 Description

Get-HubIdentifier.ps1:
I optimized the logic to directly extract a valid 13-character alphanumeric string and improve efficiency. The current method generates substrings unnecessarily (tested it with Verbose Output), slowing down performance and producing irrelevant results.
Regex Matching: Uses a regular expression to directly extract 13-character alphanumeric strings, removing the need for substring generation.
Excludes Special Characters: Filters out matches with hyphens or underscores to ensure a clean identifier.
Efficiency: Processes each string once, significantly improving performance for large collections.
Remove-FinOpsHub.ps1
This PR simplifies the Remove-FinOpsHub logic by replacing the .Reduce() method with a more sequential foreach loop for deleting resources. It improves error handling, ensuring each resource deletion attempt is properly logged and handled individually. The script now includes clearer verbose messages for filtered resources, deletion attempts, and success statuses. These changes provide better transparency and avoid potential issues caused by parallel deletion, making the script more predictable and easier to debug.
Fixes #
#554
📋 Checklist

🔬 How did you test this change?

[X] 🤏 Lint tests
[ ] 🤞 PS -WhatIf / az validate
[X] 👍 Manually deployed + verified
[X] 💪 Unit tests
[ ] 🙌 Integration tests

🙋♀ Do any of the following that apply?

[ ] 🚨 This is a breaking change.
[ ] 🤏 The change is less than 20 lines of code.

📑 Did you update docs/changelog.md?

[ ] ✅ Updated changelog (required for dev PRs)
[ ] ➡ Will add log in a future PR (feature branch PRs only)
[ ] ❎ Log not needed (small/internal change)

📖 Did you update documentation?

[ ] ✅ Public docs in docs (required for dev)
[ ] ✅ Internal dev docs in src (required for dev)
[ ] ➡ Will add docs in a future PR (feature branch PRs only)
[ ] ❎ Docs not needed (small/internal change)

@flanakin , do you want me to squash and merge this?

@jamelachahbar I looked at the integration tests we have here, which weren't working before. I was able to get something working for now, so we have an automated test for this. I updated that in the branch, so we're in a good state.
Thanks for getting this in!


Adding/Removing classes in codebehind crashes on PriorityLevel.cs

try out RenderDemo on https://github.com/AvaloniaUI/Avalonia/tree/repro-bindingprio-bug and repeatedly click the Pause/Resume Animations button.
cc: @MarchingCube
I tried fixing it by adding the following check on PriorityLevel.cs and it seem to work but im not sure if that's the correct fix.
Line 204:
                    if (_bindings.Count > 0)
                        _bindings.Remove(_binding);

Looks like we are unsubscribing twice, will investigate.

Fixed in https://github.com/AvaloniaUI/Avalonia/pull/3062.


Added mandatory labels

@mfojtik, @brenton PTAL

Looks good to me.

I'll let @mfojtik handle the merging.

@brenton want to merge https://github.com/openshift/sti-base/issues/38 first, will merge this after that beast gets in.
@eliskasl can you please make PR's for other repos?


Set default buffer_chunk_limit

According to Loggly's documentation:

If you get the error “413 Request Entity Too Large”, it means your file size is larger than the limit of 5MB. Please split your file into chunks less than 5MB, or curl the file one line at a time.

So let's overwrite the buffer_chunk_limit param, which is default to 8MB by fluentd.

Unfortunately still seeing error message after setting this. Anyone would like to have a look on this?

less than 5MB

Did you try 4MB? :D

No response. Will close this one.


Integrate OceanSense data and charts

We would like to integrate new datasets coming from OceanSense and display it in new charts.

The data will come directly from an external API
Only a very small sample of reefs will be affected

Here are the 5 icons for acidity, conductivity, pressure, dissolved oxygen, and oxidation reduction potential.
icons.zip


call start() while already started

Tengo este error call start() while already started

I also encountered this problem, very strange.
https://github.com/juliansteenbakker/mobile_scanner/assets/17878528/9edc4317-464a-4866-93c3-5a7ff323c1a8

Como  me sujieres apagar y encender el scaner