tasks.list not working

Hi.
I am trying to call the tasks.list but it isn't working. I was debbuging the code and it isn't even calling the list function from the tasks helper.
The code that I have written is:
exports.extractTasks = () => {
    let promise = new Promise((resolve, reject) => {
        harvest.tasks.list({}, (error, res, body) => {
            resolve(body);
        });
    });

    return promise;
}

Can anyone help me?

I can't promise to resolve your issue, but I won't reject your ask for help. Right... think that want cost me my joking privileges for a week.
Just ran the tests and was able to log a list of tasks successfully to screen. Here are a couple of thoughts:

It is a good idea to a catch setup for possible error handling. Might save you some debug in the long run.
The promise variable ins't truly needed in this case as you are immediately returning it. Might be better to think of this as a generator function.
How are you calling your function? Could it be something there?

Here is a modified version of your code that would also handle errors:
exports.extractTasks = (options = {}) => {
  return new Promise((resolve, reject) => {
    harvest.tasks.list(options, (error, res, body) => {
      if (error) {
        reject(error);
      }
      resolve(body);
    });
  });
}

The above also would allow you to pass in parameters or options should you need, but has a default of an empty object. Then for your call:
extractTasks()
  .then((results) => {
    // do some stuff
  })
  .catch((error) => {
    // handle your error
  })

Does any of that help?

@simplyspoke Unfortunately no.
I added the error handling at the function but it still isn't working. I tried to add some simple console logs in the harvest original module, at the list function, just to output any info if the tasks.list is called, but It is outputting the info at every other resource but not in the tasks resource.
More details:
I changed the harvest > lib > mixins > api-base.js list function to:
  list(options, cb) {
    let uri = this.baseUri;

    console.log(uri) //simple debug
    this.harvest.request('GET', uri, options, cb);
  },

When I run the code, the order of the execution are: extract clients - extract tasks extract users - extract projects
I got these outputs:
/clients/
and nothing more

If I remove the extract tasks from the queu I got:
/clients/
/people/
/projects/

It seems like it is getting stucked while trying to list all the tasks

I presume you have upgraded to v1.0.9? In the process of testing out promises structures like your example above.

O, nice it worked now! Thanks


Options not tested

We have options -iothreads, -ncycles, -maxthreads, -nophasefounders, -caling_threshold, -call_phase not tested. We may want to design some tests for them.

Discussed with GG and we don't need to worry about -iothreads, -ncycles, -maxthreads and we can revisit other options later


memory leak in nc_open for non-existant file

nctest is showing a memory leak. Seems that what is happening is the path is not freed in nc_open when the file does not exist.
I will fix tomorrow, but first let my outstanding PR be merged, since it contains some changes in nc_open in dfile.c, and I don't want to get too confused. ;-)
=================================================================
==14790==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 45 byte(s) in 3 object(s) allocated from:
    #0 0x7f2a6f44c1c0 in __interceptor_strdup (/usr/lib/x86_64-linux-gnu/libasan.so.3+0x5a1c0)
    #1 0x7f2a6bf5787e in NC_open /home/ed/tmp/netcdf-c/libdispatch/dfile.c:1854
    #2 0x7f2a6bf5565f in nc_open /home/ed/tmp/netcdf-c/libdispatch/dfile.c:652
    #3 0x7f2a6bf4faff in ncopen /home/ed/tmp/netcdf-c/libdispatch/dv2i.c:420
    #4 0x5633fb2498c1 in test_ncopen /home/ed/tmp/netcdf-c/nctest/cdftests.c:117
    #5 0x5633fb24e5c5 in main /home/ed/tmp/netcdf-c/nctest/driver.c:71
    #6 0x7f2a6b9263f0 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x203f0)

Direct leak of 27 byte(s) in 3 object(s) allocated from:
    #0 0x7f2a6f44c1c0 in __interceptor_strdup (/usr/lib/x86_64-linux-gnu/libasan.so.3+0x5a1c0)
    #1 0x7f2a6bf5787e in NC_open /home/ed/tmp/netcdf-c/libdispatch/dfile.c:1854
    #2 0x7f2a6bf5565f in nc_open /home/ed/tmp/netcdf-c/libdispatch/dfile.c:652
    #3 0x7f2a6bf4faff in ncopen /home/ed/tmp/netcdf-c/libdispatch/dv2i.c:420
    #4 0x5633fb249a26 in test_ncopen /home/ed/tmp/netcdf-c/nctest/cdftests.c:151
    #5 0x5633fb24e5c5 in main /home/ed/tmp/netcdf-c/nctest/driver.c:71
    #6 0x7f2a6b9263f0 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x203f0)

Direct leak of 21 byte(s) in 3 object(s) allocated from:
    #0 0x7f2a6f44c1c0 in __interceptor_strdup (/usr/lib/x86_64-linux-gnu/libasan.so.3+0x5a1c0)
    #1 0x7f2a6bf5787e in NC_open /home/ed/tmp/netcdf-c/libdispatch/dfile.c:1854
    #2 0x7f2a6bf5565f in nc_open /home/ed/tmp/netcdf-c/libdispatch/dfile.c:652
    #3 0x7f2a6bf4faff in ncopen /home/ed/tmp/netcdf-c/libdispatch/dv2i.c:420
    #4 0x5633fb24c35c in test_ncabort /home/ed/tmp/netcdf-c/nctest/cdftests.c:840
    #5 0x5633fb24e643 in main /home/ed/tmp/netcdf-c/nctest/driver.c:77
    #6 0x7f2a6b9263f0 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x203f0)

The warning messages on travis are not fatal; the issue appears to be pulling the docker image from dockerhub.  I'm trying to track it down but for now I can restart the failed ones by hand and they should work.  It will just take a little bit longer to work through. Going through them all now, thanks for all the help, Ed! :).

Awesome. Once you get all those ones merged, I will see if I can clear up the other memory problems in the tests.
For the PIO work I depend on the address sanitizer to keep me honest, so I really need all of netCDF and its tests to be memory clean.

FYI this issue can also be seen in nc_test/tst_names.c.
Looking forward to seeing the PR merged so these errors will go away.
 ./tst_names

*** testing names with file tst_names.nc...
*** switching to netCDF classic format...Error at line 282 (loop=131): No error
Error at line 282 (loop=132): No error
ok.
*** switching to netCDF 64-bit offset format...Error at line 282 (loop=131): No error
Error at line 282 (loop=132): No error
ok.
*** switching to netCDF 64-bit data format...Error at line 282 (loop=131): No error
Error at line 282 (loop=132): No error
Error at line 304: NetCDF: Unknown file format
*** switching to netCDF netCDF-4/HDF5 format...Error at line 282 (loop=131): No error
Error at line 282 (loop=132): No error
ok.
*** switching to netCDF netCDF-4 classic model format...Error at line 282 (loop=131): No error
Error at line 282 (loop=132): No error
ok.
*** Tests successful!

=================================================================
==9359==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 13 byte(s) in 1 object(s) allocated from:
    #0 0x7fed2fe8d538 in strdup (/usr/lib/x86_64-linux-gnu/libasan.so.4+0x77538)
    #1 0x7fed2c86a2c7 in NC_open /home/ed/n2/netcdf-c/libdispatch/dfile.c:1867
    #2 0x7fed2c867b5a in nc_open /home/ed/n2/netcdf-c/libdispatch/dfile.c:652
    #3 0x5640203c0961 in main /home/ed/n2/netcdf-c/nc_test/tst_names.c:302
    #4 0x7fed2c1fd1c0 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x211c0)

SUMMARY: AddressSanitizer: 13 byte(s) leaked in 1 allocation(s).


Using "ruff format" with stdin

I've started using the Zed editor and want to have ruff format my code (as I'm switching from black). I can set up Zed to format-on-save using black as follows
  "language_overrides": {
    "Python": {
      "preferred_line_length": 88,
      "formatter": {
        "external": {
          "command": "black",
          "arguments": ["-"]
        }
      }
    }
  }

but when I try to the do the same with ruff, no such luck. Trying either of two below fails to work.
          "command": "ruff format",
          "arguments": ["-"]

          "command": "ruff",
          "arguments": ["format", "-"]

I would expect the latter to work -- any idea what error you're running into? Any trace you could share?

Can you share your entire config? Could it be that there's a "format_on_save": "off" somewhere in the global setting? Zed also allows folder specific settings so I would check those as well. Can you also share a code snippet which you're facing this problem?

Closing now in the absence of more info, but can always re-open.


Add a Bitdeli Badge to README

Pull request made by @intrepion at https://bitdeli.com

Current coverage is 100.00%

Merging #2 into master will not affect coverage as of 312a7c7

@@            master      #2   diff @@
======================================
  Files            1       1
  Stmts            7       7
  Branches         0       0
  Methods          2       2
======================================
  Hit              7       7
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 312a7c7

Powered by Codecov. Updated on successful CI builds.

Adding this manually


Escape Key freeze While Selenium test is running

Hi Dear florentbr!
Just noticed that when SeleniumBasic 2.0.6 tests are run the escape key of keyboard does not work. While trying pressing escape too many times selenium gives error such like KeyError.
It seems Selenium blocks Escape key or keep in its use and for other applications Escape key does not work. Please check this issue.

Duplicate of #43


Fixed bug which showed "No results" just before showing results

I've discovered that when I use an external api (which could be slower than a local json) and I write slow, the dropdown shows me "No results found" just before I get the result.
To replicate:

Your json must have something which matches "butterflies"
Slowly type "butterflies"
This happens because the $timeout sometimes can't be stopped in time and the last calling still sets scope.searching to false.
In my opinion it makes more sense to call directly to the function and at the same time set a $timeout just in case it couldn't find anything.

staled


python310Packages.pytest-ansible: disable tests that fail in darwin sandbox

Description of changes
These tests do some funky networking that are not resolved just by enabling local networking.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 244091 run on x86_64-darwin 1

4 packages built:

python310Packages.pytest-ansible
python310Packages.pytest-ansible.dist
python311Packages.pytest-ansible
python311Packages.pytest-ansible.dist


Show doc in Colab

First, thanks for the tools !
I tried to use show_docs in Colab and this is what I got printed
<IPython.core.display.Markdown object>, however it works on jupyter notebook.
Is this a known issue? Is there a workaround?

I'm not sure it has been tested before. If they don't want to display Markdown, then it can't work, no. You can always pass disp=False that will return you the Markdown code, that you can then copy and paste in a cell.
Another possibility is that you have to trust the notebook before seeing those results interpreted (you have to do this in jupyter for instance).

I'm not sure it has been tested before. If they don't want to display Markdown, then it can't work, no. You can always pass disp=False that will return you the Markdown code, that you can then copy and paste in a cell.

Thanks for your answer, that works somehow but there is some code which is not rendered correctly (both in Colab and jupyter), linebreaks (\n in the output) for example.


Performance alert when deploying to iOS 10.1

Upon deploying an app (with both ionic 1 and 2) the first time to iOS 10.1 I see the following:

From what I can tell online, this is because iOS 10.1 only wants 64-bit apps and the app I am trying to deploy is 32-bit.  According to Cordova, you can build 64-bit apps and have been able to for some time.  Thus, I assume it is also possible with Ionic.  However, there is no documentation on how to do this.
I am doing something wrong or is this truly a bug?
Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.1
Ionic CLI Version: 2.1.4
Ionic App Lib Version: 2.1.2
Ionic App Scripts Version: 0.0.38
ios-deploy version: 1.9.0
ios-sim version: 5.0.10
OS: Mac OS X Sierra
Node Version: v6.9.0
Xcode version: Xcode 8.1 Build version 8B62

that happened to me too with ionic package build.

Hello @devknack, are you also using ionic package build ?

Check your version of cordova-ios - I think that's the source of the issue.

I've got rid of this warning after I updated my cordova-ios version to 4.3.0 and specified that on package.json. It seems that ionic cli adds iOS by default with an outdated version of cordova-ios.

I think platform versions are fixed according to the installed version of Cordova cli. If you have the latest cli (6.4) then iOS 4.3 is automatically installed when adding iOS platform - at least for me.

@biesbjerg I also have Cordova cli 6.4 and it adds Cordova iOS 4.1.1 to my projects.

This states otherwise: https://cordova.apache.org/news/2016/10/25/tools-release.html

Updated Platforms CB-12039: updated pinned Android to 6.0.0 and iOS to 4.3.0.

Maybe you have the platform pinned to an older version in your config.xml? http://cordova.apache.org/docs/en/latest/platform_plugin_versioning_ref/index.html#platform-versioning

Hello! Thanks for opening an issue with us! As this seems like more of a support question i will urge that you ask this question on our forum)forum.ionicframework.com) or on our slack channel. Thanks for using Ionic!


Footer is not displayed anymore with recent Chromiums

Footer templates set via ChromicPDF.Template are not displayed anymore with recent Chromiums.
ChromicPDF.Template sets page dimensions and header/footer dimensions via rules in @page { ... } and sets preferCSSPageSize: true in the printToPDF call. Some time after Chromium 117, footers set via footerTemplate have disappeared (margin is there but blank).
Analysis so far

Somewhere in between Chromium 117.0.5938.62 and 119.0.6045.159
setting the page margins via printToPDF options (in inches) fixes it
but even if only margin-top is present in @page, it breaks again

even with preferCSSPageSize: false

Minified example of what is broken
@page {
  size: 210mm 297mm;
  /* or any other variation of specifying margin-top and margin-bottom */
  margin: 40mm 0 40mm;
}

See stylesheet for actual code.
Things we've tried

size vs. width/height in @page has no effect
removing the zoom rule from #header, #footer has no effect
keeping a margin-inline: ... is ok, but specifying top/bottom in the CSS immediately breaks it

We encountered the same here! It was introduced in one of projects on 5 October when we deployed a new release of our software with the latest images of our build OS.

@janwillemvd can you pinpoint the chromium version where it broke? We currently only have 117.0.5938.62 as last known good and 119.0.6045.159 as broken.

script to reproduce this https://gist.github.com/maltoe/104d192430a059f3672cb7a8c7a3b1bf (assumes local chrome is 120 or so and chrome 115 is run via docker)

potential culprits:

anything on this page after Aug 4 https://github.com/chromium/chromium/commits/main/components/printing

analysis so far:

zoom level has definitely changed and we need to reconsider the default #header, #footer { zoom: 0.75; } in ChromicPDF.Template, but it seems this is unrelated to the disappearance of the footer
setting the page margins in the CSS breaks the footer, setting it in the options for printToPDF makes it work again.
hence, potential fix would be to take the header_height and footer_height values and convert them to inches and set them in the options instead of the CSS, but sucks a bit

@janwillemvd We have worked around this in our apps now by not using ChromicPDF.Template anymore, but passing in the CSS manually (without any margin rule in @page) and with marginTop etc. passed as print_to_pdf option.

@janwillemvd can you pinpoint the chromium version where it broke? We currently only have 117.0.5938.62 as last known good and 119.0.6045.159 as broken.

Will check this afternoon and let you know!

@janwillemvd We have worked around this in our apps now by not using ChromicPDF.Template anymore, but passing in the CSS manually (without any margin rule in @page) and with marginTop etc. passed as print_to_pdf option.

Thanks, @maltoe, will try that!

Maybe a bit bold, but perhaps it may help: @mstensho - you've recently worked on related code in Chromium (e.g. this commit) -> Do you have any idea what has changed and what we can do to mitigate? Context in the issue description.

Don't know why the footer would disappear, but the font size change is explained here: https://bugs.chromium.org/p/chromium/issues/detail?id=1509917#c3

@mstensho thank you for responding! :purple_heart: Unfortunately just dropping the zoom rule doesn't fix it :/
Do you think it makes sense to file a bug at bugs.chromium.org? Is what we've been doing here even "officially" supported? I've found lots of examples about @page { margin: 0 } on the internet, but nobody seems to be using @page { margin-bottom: ... } to give space to the footer.

@page { margin:0; } means no room for headers and footers, so that's definitely not going to help. Or is this a separate "print job" for headers and footers? In that case margin:0 seems right.
There is code in Chromium that checks whether the headers / footers overlap with the page area box [1], and omit any such headers / footers.
In the CSS at https://github.com/bitcrowd/chromic_pdf/blob/ed525c2400922cc83ebdd41eaa8907de68cb0c55/lib/chromic_pdf/template.ex#L245-L275 :
 @page {
      width: <%= @width %>;
      height: <%= @height %>;
      margin: <%= @header_height %> 0 <%= @footer_height %>;
    }

Setting the 'width' and 'height' properties for the page context isn't supported (at least not yet), and it probably wouldn't do what you think, anyway. I would expect the 'size' descriptor to be used instead, i.e. something like this:
      size: <%= @width %> <%= @height %>;

Unless this is for a separate "print job" for headers / footers. Then you should probably just remove width / height and set margins to 0. But I don't know this framework, so hard for me to tell.
In the test, I don't think width and height are specified, are they? We have guards in Chromium against invalid page area sizes, e.g. if the total size of the margins are larger than the page box [1], which would result in a zero or negative page area. In such cases we fall back to the default page size and margins (ignore CSS completely). If you try to squeeze in a footer taller than the default bottom margin, it will be discarded, since it would overlap with the page area. See above. Are the headers and footers really 40mm? Looks smaller to me.
[1] https://drafts.csswg.org/css-page-3/#page-model

I was able to reproduce the behaviour using puppeteer, if that helps.
<!DOCTYPE html>

    <meta charset="UTF-8">

hello

`;

(async () => {
  const browser = await puppeteer.launch();
  const page = await browser.newPage();
  await page.setContent(html);
  await page.pdf({
    path: 'output.pdf',
    displayHeaderFooter: true,
    preferCSSPageSize: true,
    headerTemplate: html,
    footerTemplate: html,
  });
  await browser.close();
})();

run with:
npm init -y
npm install puppeteer
node test.js

Also @mstensho thanks again for your invaluable help! :purple_heart: much appreciated

@janwillemvd can you pinpoint the chromium version where it broke? We currently only have 117.0.5938.62 as last known good and 119.0.6045.159 as broken.

I can confirm that it's broken in our app which uses 117.0.5938.132 (running on Debian 12).

@janwillemvd :exclamation: you're right, 117 was broken for us, too. I thought it was the version we used before 119, but in fact that was only for a few days (and PDFs from the period are broken too)... So our last known good version is in fact 112.
@mstensho Would you like me to open a bug on bugs.chromium.org?

A bug report sounds fine.
Also, I really meant to point to the code that performs the header / footer overlap testing yesterday, but it looks like I forgot. Sorry.
Here it is:
https://source.chromium.org/chromium/chromium/src/+/main:components/printing/resources/print_header_footer_template_page.html;l=124-127;drc=f6529c7990744370869e4ab2794caae6c46ba044

Thanks, I'll open that bug report in a second then.
The code you've linked I had found before and found it suspicious, but it should only apply to elements with a .text class, right? So the default header/footer elements match, but not my custom footerTemplate.

Oh right, it needs to be .text, indeed.

:tada: I think I've figured this out now. The problem is that we've been prepending the entire CSS snippet above to both the content HTML and the headerTemplate and footerTemplate options. The @page { } directives used to be ignored within headerTemplate and footerTemplate, but apparently the page margins are now applied to the headers and footers as well, which makes their content disappear (pushed outside their content boxes basically).
@mstensho This is certainly a behaviour change in Chromium since after 115, but probably a good one. Thanks again for your time.
For ChromicPDF, I'm going to simply split this CSS apart and not include any @page rules in the header and footer templates anymore.

Thank you so much @maltoe, really saving the day here <3

Glad you figured it out! I'm not quite sure why page margins were ignored previously, but it certainly makes more sense to honor margins on a general basis.


Add slovak/SK language support

Add slovak (SK) language translations
Add support for SK URL routes

Fantastic! :)

@all-contributors please add @sbocinec for translating the site into slavic

You could also add yourself as codeowner of the sk translations: https://github.com/BustByte/coronastatus/blob/master/CODEOWNERS#L5

One more thing - added Slovak wordlist https://github.com/BustByte/coronastatus/pull/231/commits/70726ae62f096ecd675b320a7f52ca8a1c4fad7e - Could not find any good one, so took the english BTC, translated using Google translate and did some normalizations (diacritic removal/transliteration, lowercase,,...). Looks better than nothing

@sbocinec is this ready to merge?

@sbocinec is this ready to merge?

It will require few more fixes + adding the privacy policy, however, will create a new PR for those.


Update fluent-terminal-np.json to <IP_ADDRESS> and new manifest

Seems like everything regarding the installation of Fluent Terminal has been changed. You just need to run the appxbundle file now.

On another note can you take a look at another UWP app manifest? Its a modern fluent design tabbed Notepad called Notepads. Everything works fine but the installation script has been designed to shut down your terminal after installation
https://gist.github.com/Restia666Ashdoll/61fd66c2e153462becac877249903d06
Also, for some reason a manifest is not created in installation directory. If I manually put the manifest in installation directory, uninstallation script works just fine.

Also, added a shim for Fluent Terminal

Thanks!


Layer Rendering: order / depth buffer

Issue #1286

Looks like this item is also fixed by this PR.

So the tile masking and opaquePassCutoff check don't need to be done at all?

So the tile masking and opaquePassCutoff check don't need to be done at all?

I'm not entirely sure, but apparently the visuals are correct if we're not filtering out drawables. Most likely it has to do with the opaque pass, where depth is written.

So the tile masking and opaquePassCutoff check don't need to be done at all?

I'm not entirely sure, but apparently the visuals are correct if we're not filtering out drawables. Most likely it has to do with the opaque pass, where depth is written.

tile masking remains for TileLayerGroupGL. the LayerGroupGL doesn't need it since it's not tile oriented, its drawables should not have tileID set

Oh, I see, yeah, I removed the tile masking stuff from LayerGroupGL in my PR as well.

it would be also interesting to check whether the legacy path draws multiple times. I think it does, and where it doesn't it returns early from the RenderLayer.render method. Like it does for line:
void RenderLineLayer::render(PaintParameters& parameters) {
  assert(renderTiles);
  if (parameters.pass == RenderPass::Opaque) {
    return;
  }

for RenderFillLayer it does not early return. and there's some opaquePassCutoff logic in the Renderer::Impl::render as well


Many additional FAIL/ERROR with build/test using '-g' flag, (MacOS 10.12, MacBookPro)

Lots of additional errors appeared when using '-g' during runtests.py testing, as did one python malloc error.
[H/W MacBookPro (late 2016) running MacOS 10.12.5]
I observed lots of errors when running tests with '-g'.  There are 4 possible combinations of building and running tests with this flag, and 3 are described below.
setup.py build with/without '-g'.
runtests.py with/without -g.

No '-g' flag in either build or test.
FAILED (KNOWNFAIL=90, SKIP=1895, errors=8, failures=45)
Build with '-g', runtests.py with '-g'.
FAILED (KNOWNFAIL=90, SKIP=1895, errors=175, failures=124)
Build with '-g', runtests.py without it.
runtests.py dies during test_arpack after reporting a malloc error

Details:

No '-g' flag in either build or test.
FAILED (KNOWNFAIL=90, SKIP=1895, errors=8, failures=45)
Errors/Failures by type:
1 test_basic.TestMatrixNorms.test_matrix_norms
1 test_basic.TestVectorNorms.test_overflow
1 test_asum
1 test_dot
1 test_nrm2
1 test_slamch
1 test_lange
5 test_arpack.test_symmetric_modes
3 test_arpack.test_real_nonsymmetric_modes
38 test_iterative.test_convergence

Build with '-g', runtests.py with '-g'.
FAILED (KNOWNFAIL=90, SKIP=1895, errors=175, failures=124)
Errors/Failures by type:
1 test_basic.TestMatrixNorms.test_matrix_norms
1 test_basic.TestVectorNorms.test_overflow
1 test_asum
1 test_dot
1 test_nrm2
1 test_complex_args
1 test_lange
76 test_arpack.test_symmetric_modes
48 test_arpack.test_hermitian_modes
5 test_arpack.test_symmetric_starting_vector
53 test_arpack.test_real_nonsymmetric_modes
54 test_arpack.test_complex_nonsymmetric_modes
8 test_arpack.test_standard_nonsymmetric_starting_vector
8 test_arpack.test_general_nonsymmetric_starting_vector
1 test_arpack.test_svd_linop
38 test_iterative.test_convergence

The bulk of the additional '-g' errors/failures involved test_arpack.

Build with '-g", runtests.py without it.
runtests.py dies during test_arpack after reporting a malloc error
python(47504,0x7fffef7623c0) malloc: *** error for object 0x7ff1bb99c9e8: incorrect checksum for freed object - object was probably modified after being freed.

Expected behavior?
Reproducing shell example:
python setup.py build -v -f
pip install -v .
python runtests.py -n -v -j3 > runtests.log 2>&1
pip uninstall scipy
python setup.py build -v -g -f
pip install -v .
python runtests.py -n -v -j3 > runtests.g.log 2>&1
python runtests.py -n -g -v -j3 > runtests.g2.log 2>&1
Scipy/Numpy/Python version information:
1.0.0.dev0+3c1b095 1.13.0 sys.version_info(major=3, minor=6, micro=1, releaselevel='final', serial=0)

Not expected on osx. Looks like the BLAS/LAPACK is broken in your build.
.
Note that setup.py build and pip install should not be necessary,
runtests.py should be fine by itself.
.
Try to see if it works in a clean rebuild; do rm -rf build etc. If
not, attach the build log (for a clean rebuild) or post it at
gist.github.com.

It appears you are using intel MKL on OSX. The test failures look like
it's linking in a version of this library with the wrong fortran ABI.

Thanks.  I'm using Anaconda as a base for the Python environment.  All the .so files built by SciPy seem to use libgfortran.4.dylib.
$ find  build/lib.macosx-10.7-x86_64-3.6/scipy -regex ".*\.so" | xargs otool -L | fgrep fortran
    /usr/local/opt/gcc/lib/gcc/7/libgfortran.4.dylib (compatibility version 5.0.0, current version 5.0.0)
    ...

The fortran compiler appears to be  /usr/local/bin/gfortran, installed by homebrew.
$ /usr/local/bin/gfortran --version
GNU Fortran (Homebrew GCC 7.1.0) 7.1.0`.

Env variables are
export CC=clang
export CXX=clang++
export LDFLAGS="$LDFLAGS -undefined dynamic_lookup -bundle"
export FFLAGS=-ff2c

(set after reading https://www.scipy.org/scipylib/building/macosx.html) The first 3 seem necessary. I rebuilt just now without the FFLAGS and it didn't change any of the errors.
Is it possible to specify which version of the library to link in?
Or specify how to build scipy in a manner compatible with the installed libraries?

Those instructions are probably out of date. AFAIK, it should build out of the box, but on the other hand I don't use osx. FFLAGS should not be necessary (although it likely doesn't affect anything).
Read site.cfg.example if you want to build with MKL. Information about which library has which fortran ABI is probably found in intel MKL documentation. You want the g77 one. However, I'm not sure if building against intel MKL obtained from anaconda is actually possible, or whether you need openblas etc.

Those instructions are probably out of date.

They are. Things on OS X build fine for current master with no env vars set.

$ /usr/local/bin/gfortran --version
GNU Fortran (Homebrew GCC 7.1.0) 7.1.0`.

Not sure that has been tested before, it's brand new. I've used GNU Fortran (Homebrew gcc 4.9.1) 4.9.1 for a long time.

I'm doing a survey of the open issues in support of the CZ grant opportunity, and I'm trying to understand what would be required to close this issue. If I were to successfully build with MKL on a Mac (and update the build instructions if any notes were needed) with a recent version of Homebrew's GCC, would that close the issue?

This was quite some time ago.  I'm running again to see if the failures still exist.

Thank you!

Currently in my environment I see the same 15 failures with or without -g, in build/or test.
TestVectorNorms.test_overflow
TestMatrixNorms.test_matrix_norms
TestFBLAS1Simple.test_asum
TestFBLAS1Simple.test_dot
TestFBLAS1Simple.test_nrm2
TestWfuncPointers.test_complex_args
TestFlapackSimple.test_lange
test_symmetric_modes
test_hermitian_modes
test_symmetric_starting_vector
test_real_nonsymmetric_modes
test_complex_nonsymmetric_modes
test_standard_nonsymmetric_starting_vector
test_general_nonsymmetric_starting_vector
test_svd_linop

I'm presuming it is something to do with whatever BLAS/LAPACK is installed on my machine, but certainly the differences due to use of -g are gone, so I'll close out this Issue.

How does one build scipy so that Fortran is called without -g flag, is it configurable somehow?
(I'd like to use flang, which has problems with COMMON blocks if -g is used)


Update free-courses-en.md

Added new Course On Digital Marketing
What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Added New Course on Digital Marketing.

I don't think that digital marketing fits into free-programming-books. I'll defer to @eshellman but I would close this one.

I agree with @Thenlie


litte qustion.

See how to fix this error on discord server in errors and tips section. (fbgemm.dll theme)


Page Number-Align is broken or unintuitive

I have only the following code in a .typ file.
#set page(
    number-align: right + top,
    numbering: "1",
)

And it creates the following PDF with the page numbering on the bottom right:

Is this feature broken, or am I not understanding how the number-align works?

Currently, number-align determines the alignment of the number within the footer. But I agree that this is confusing, maybe we should put it into the header instead when an explicit top alignment is given.

At this time is there a way to get the page number in the header?

At this time is there a way to get the page number in the header?

#set page(header: [
  #align(right)[#counter(page).display()]
])


test/beast/websocket failed against 32 bits compiler (gcc/clang)

With 64 mod.

boost_1_75_0/b2 toolset=clang address-model=64 --abbreviate-paths --disable-icu  --reconfigure -d+1 -d+2 cflags="-std=c++14  "  write

Pass.
With 32 mode.

boost_1_75_0/b2 toolset=clang address-model=32 --abbreviate-paths --disable-icu  --reconfigure -d+1 -d+2 cflags="-std=c++14  "  write
/bin/sh: line 9: 4003719 Segmentation fault      (core dumped) "../../../../../bin.v2/libs/beast/test/beast/websocket/write.test/clng-lnx-14.0./dbg/adrs-mdl-32/thrdp-pthrd/thrd-mlt/vsblt-hdn/write" > "../../../../../bin.v2/libs/beast/test/beast/websocket/write.test/clng-lnx-14.0./dbg/adrs-mdl-32/thrdp-pthrd/thrd-mlt/vsblt-hdn/write.output" 2>&1 < /dev/null

Failed.

$cat write.output
beast.websocket.write
EXIT STATUS: 139

Thank you. I will investigate.

@xing1xu Can you share some more details about the issue?


Eiffel protocol MVP

I have a question regarding the scope of the protocol. There are quite many event types listed in the README but does it make sense to start with that many? I mean perhaps it would be better to narrow down scope so we can get the first usable version of protocol faster?
For instance, take EiffelActivityQueuedEvent. I have a hard time imagining use case and most importantly a good way of implementing a good way to send those correctly. Are they even used?
Perhaps it makes sense to start with the few core event and skip corner cases?
I mean it must be 20% of events that cover 80% of use cases, isn't it? Shall we define those first and then, if there is a need, look into more complicated cases?

It's absolutely possible to start with a small "core" set of events and only use those: a key principle is freedom in which events to use, but not in how you use them. We're going to focus on such a subset of events for the first "drop" or milestone: all Artifact events, all Activity events and ConfidenceLevelModified.
So to answer your question: absolutely, but EiffelActivityQueuedEvent is a poor example. If you're going to use Activity events, you're going to need ActivityQueued. But it is of course possible to forego that.

Proposal: Change names of ActivityQueued and ActivityDequeued to ActivityTriggered and ActivityCanceled. Triggering is always relevant, even if you don't queue. If you do queue, however, it becomes implicitly clear from the fact that there's a time delta between Triggered and Started.

Yep, that's what we'll do.


InitHandle and NativeClass::new_instance() now refer to a global class registry to check if the class has been added.

This is related to issue #602 where and it allows the InitHandle struct and NativeClass trait to properly track whether a class has been added to the godot API.
This is just my first pass at a naming scheme, so if there is a preferred naming convention to go by or modifying the error messages, please let me know.
For the "class_registry", I decided to use the RwLock as there shouldn't be any need for locking after initialization. I thought this would be a performance be a minimal number of writes after initialization. This should lower the performance cost for reads after that point. The registry consists of a simple hash set as outlined in the Issue
For the NativeClass::new_instance() method, I chose to use a debug_assert!(), to give a more concrete error message when using a debug-build, but remove the additional check during instancing from Release builds.
If there are any other requests regarding this, I would be happy to make the necessary changes.

Could this also make it easier to support inheriting from another Rust NativeClass type? The engine will check if you pass the name of a class owned by the same gdnative library: https://github.com/godotengine/godot/blob/3.x/modules/gdnative/nativescript/godot_nativescript.cpp#L66


Chat/completions endpoint using Xef DSL

The objective of this issue is to update the chat/completions endpoint to use the Xef Library depending on the JSON that comes in the request.

@Montagon Can we close this task?


fix fail to start pod

if pod is stopped automatically, we should reset pod otherwise
start pod will fail with error:
./hyper ERROR: Error from daemon's response: pod pod-JfRVQkHSZc is already running
Signed-off-by: Gao feng<EMAIL_ADDRESS>

LGTM


make check-migrations exits cleanly when migration directory is corrupt

Commit: 0a1a59bf339cb6a498c350c6f9043e82a163794f
Node version: v14.19.3
I'd expect a target called check-migrations should be exiting with a non-zero code if it fails, but in this instance that doesn't seem to be the case.
Steps to recreate
rm lib/model/migrations/20170920-01-initial.js
make check-migrations

Example
$ make check-migrations; echo "Exited with status: $?"
node -e 'const { withDatabase, checkMigrations } = require("./lib/model/migrate"); withDatabase(require("config").get("default.database"))(checkMigrations);'
(node:769870) UnhandledPromiseRejectionWarning: Error: The migration directory is corrupt, the following files are missing: 20220627-01-add-random-counter.js
    at validateMigrationList (/x/odk/central-backend/node_modules/knex/lib/migrate/Migrator.js:575:11)
    at Migrator.list (/x/odk/central-backend/node_modules/knex/lib/migrate/Migrator.js:290:7)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:769870) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:769870) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
Exited with status: 0

check-migrations is supposed to exit with a nonzero code if the migrations aren't right! You can see it trying to do that here:
https://github.com/getodk/central-backend/blob/0a1a59bf339cb6a498c350c6f9043e82a163794f/lib/model/migrate.js#L28-L34
If I'm remembering correctly, we don't use the makefile anywhere in production. Instead, the server is started using this script:
https://github.com/getodk/central/blob/master/files/service/scripts/start-odk.sh
Given how that script works, my instinct is that if we think checkMigrations() might fail without setting a nonzero code, we should modify checkMigrations() rather than the makefile. Specifically, we could catch an error thrown by db.migrate.list(). That said, it also seems pretty unlikely that db.migrate.list() would throw an error in production: something else would have to go wrong I think.
Given that we don't use the makefile in production, I'm not sure that it's super important that those commands set a nonzero code. But maybe in some workflows it'd be helpful. (It certainly wouldn't hurt!)

Given that we don't use the makefile in production, I'm not sure that it's super important that all commands set a nonzero code.

Great point!  I'll dig in and see if I can fix the underlying problem in the script.

Issue updated to include details of make migrations behaviour.


Semver Versioning

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe the problem.
Can you make it so it conforms to SemVer 2.0
Describe the solution you'd like
Calling new APIVersion. Will result in something like this:-
using System.Diagnostics.CodeAnalysis;
using static System.String;
using static System.Text.RegularExpressions.Regex;
using static System.Text.RegularExpressions.RegexOptions;
using Microsoft.AspNetCore.Mvc.Versioning;
namespace Microsoft.AspNetCore.Mvc;
///
/// Represents the application programming interface (API) version of a service using Semantic Versioning (SemVer).
///
public partial class ApiVersion : IEquatable, IComparable, IFormattable
{
const int Prime = 397;
const string ParsePattern = @"^(\d+).(\d+).(\d+)(?:-(.*))?$";
int hashCode;
ApiVersion()
{
    MajorVersion = 1;
    MinorVersion = 0;
    PatchVersion = 0;
    PreRelease = null;

    hashCode = GetHashCodeInternal();
}

/// <summary>
/// Initializes a new instance of the <see cref="ApiVersion"/> class.
/// </summary>
/// <param name="majorVersion">The major version.</param>
/// <param name="minorVersion">The minor version.</param>
/// <param name="patchVersion">The patch version.</param>
/// <param name="preRelease">The pre-release version.</param>
public ApiVersion(int majorVersion, int minorVersion, int patchVersion, string? preRelease = null)
{
    if (majorVersion < 0)
    {
        throw new ArgumentOutOfRangeException(nameof(majorVersion));
    }

    if (minorVersion < 0)
    {
        throw new ArgumentOutOfRangeException(nameof(minorVersion));
    }

    if (patchVersion < 0)
    {
        throw new ArgumentOutOfRangeException(nameof(patchVersion));
    }

    MajorVersion = majorVersion;
    MinorVersion = minorVersion;
    PatchVersion = patchVersion;
    PreRelease = preRelease;

    hashCode = GetHashCodeInternal();
}

/// <summary>
/// Gets the default API version.
/// </summary>
/// <value>The default <see cref="ApiVersion">API version</see>, which is always "1.0.0".</value>
public static ApiVersion Default { get; } = new ApiVersion();

/// <summary>
/// Gets the major version number.
/// </summary>
/// <value>The major version number.</value>
public int MajorVersion { get; }

/// <summary>
/// Gets the minor version number.
/// </summary>
/// <value>The minor version number.</value>
public int MinorVersion { get; }

/// <summary>
/// Gets the patch version number.
/// </summary>
/// <value>The patch version number.</value>
public int PatchVersion { get; }

/// <summary>
/// Gets the optional pre-release version.
/// </summary>
/// <value>The pre-release version.</value>
/// <remarks>The pre-release version allows services to indicate versions that are not release quality or guaranteed to be backward compatible. Example values might include "alpha", "beta", "rc", etc.</remarks>
public string? PreRelease { get; }

/// <summary>
/// Parses the specified text into an API version.
/// </summary>
/// <param name="text">The text to parse.</param>
/// <returns>The parsed <see cref="ApiVersion">API version</see>.</returns>
/// <exception cref="FormatException">The specified text is not a valid SemVer API version.</exception>
[SuppressMessage("Microsoft.Design", "CA1011:ConsiderPassingBaseTypesAsParameters", Justification = "This is the only type supported.")]
public static ApiVersion Parse(string text)
{
    if (TryParse(text, out var result))
    {
        return result;
    }

    throw new FormatException($"The provided version '{text}' is not a valid SemVer API version.");
}

/// <summary>
/// Tries to parse the specified text into an API version.
/// </summary>
/// <param name="text">The text to parse.</param>
/// <param name="version">The parsed <see cref="ApiVersion">API version</see> if successful, or <c>null</c> if parsing fails.</param>
/// <returns><c>true</c> if the parsing is successful; otherwise, <c>false</c>.</returns>
[SuppressMessage("Microsoft.Design", "CA1011:ConsiderPassingBaseTypesAsParameters", Justification = "This is the only type supported.")]
public static bool TryParse(string text, out ApiVersion? version)
{
    var match = Match(text, ParsePattern, Compiled | IgnoreCase);
    if (match.Success)
    {
        var major = int.Parse(match.Groups[1].Value);
        var minor = int.Parse(match.Groups[2].Value);
        var patch = int.Parse(match.Groups[3].Value);
        var preRelease = match.Groups[4].Success ? match.Groups[4].Value : null;

        version = new ApiVersion(major,
                                 minor,
                                 patch,
                                 preRelease);
        return true;
    }

    version = null;
    return false;
}

/// <inheritdoc/>
public override bool Equals(object? obj) => obj is ApiVersion other && Equals(other);

/// <inheritdoc/>
public bool Equals(ApiVersion? other)
{
    if (ReferenceEquals(this, other))
    {
        return true;
    }

    return other != null &&
           MajorVersion == other.MajorVersion &&
           MinorVersion == other.MinorVersion &&
           PatchVersion == other.PatchVersion &&
           Equals(PreRelease, other.PreRelease);
}

/// <inheritdoc/>
public override int GetHashCode() => hashCode;

int GetHashCodeInternal()
{
    var hash = MajorVersion;
    hash = (hash * Prime) ^ MinorVersion;
    hash = (hash * Prime) ^ PatchVersion;

    if (!IsNullOrEmpty(PreRelease))
    {
        hash = (hash * Prime) ^ PreRelease!.GetHashCode();
    }

    return hash;
}

/// <inheritdoc/>
public override string ToString() => PreRelease == null
    ? $"{MajorVersion}.{MinorVersion}.{PatchVersion}"
    : $"{MajorVersion}.{MinorVersion}.{PatchVersion}-{PreRelease}";

/// <inheritdoc/>
public string ToString(string? format, IFormatProvider? formatProvider) => ToString();

/// <inheritdoc/>
public int CompareTo(ApiVersion? other)
{
    if (ReferenceEquals(this, other))
    {
        return 0;
    }

    if (other is null)
    {
        return 1;
    }

    var majorComparison = MajorVersion.CompareTo(other.MajorVersion);
    if (majorComparison != 0)
    {
        return majorComparison;
    }

    var minorComparison = MinorVersion.CompareTo(other.MinorVersion);
    if (minorComparison != 0)
    {
        return minorComparison;
    }

    var patchComparison = PatchVersion.CompareTo(other.PatchVersion);
    if (patchComparison != 0)
    {
        return patchComparison;
    }

    if (IsNullOrEmpty(PreRelease) && IsNullOrEmpty(other.PreRelease))
    {
        return 0;
    }

    if (IsNullOrEmpty(PreRelease))
    {
        return 1;
    }

    if (IsNullOrEmpty(other.PreRelease))
    {
        return -1;
    }

    return StringComparer.OrdinalIgnoreCase.Compare(PreRelease, other.PreRelease);
}

/// <summary>
/// Determines whether two specified API versions are equal.
/// </summary>
/// <param name="version1">The first API version to compare, or <c>null</c>.</param>
/// <param name="version2">The second API version to compare, or <c>null</c>.</param>
/// <returns><c>true</c> if the two versions are equal; otherwise, <c>false</c>.</returns>
public static bool operator ==(ApiVersion? version1, ApiVersion? version2)
{
    if (ReferenceEquals(version1, version2))
    {
        return true;
    }

    if (version1 is null || version2 is null)
    {
        return false;
    }

    return version1.Equals(version2);
}

/// <summary>
/// Determines whether two specified API versions are not equal.
/// </summary>
/// <param name="version1">The first API version to compare, or <c>null</c>.</param>
/// <param name="version2">The second API version to compare, or <c>null</c>.</param>
/// <returns><c>true</c> if the two versions are not equal; otherwise, <c>false</c>.</returns>
public static bool operator !=(ApiVersion? version1, ApiVersion? version2) => !(version1 == version2);

/// <summary>
/// Determines whether the first specified API version is less than the second specified API version.
/// </summary>
/// <param name="version1">The first API version to compare, or <c>null</c>.</param>
/// <param name="version2">The second API version to compare, or <c>null</c>.</param>
/// <returns><c>true</c> if <paramref name="version1"/> is less than <paramref name="version2"/>; otherwise, <c>false</c>.</returns>
public static bool operator <(ApiVersion? version1, ApiVersion? version2)
{
    if (version1 is null)
    {
        return version2 is not null;
    }

    return version1.CompareTo(version2) < 0;
}

/// <summary>
/// Determines whether the first specified API version is greater than the second specified API version.
/// </summary>
/// <param name="version1">The first API version to compare, or <c>null</c>.</param>
/// <param name="version2">The second API version to compare, or <c>null</c>.</param>
/// <returns><c>true</c> if <paramref name="version1"/> is greater than <paramref name="version2"/>; otherwise, <c>false</c>.</returns>
public static bool operator >(ApiVersion? version1, ApiVersion? version2)
{
    if (version1 is null)
    {
        return false;
    }

    return version1.CompareTo(version2) > 0;
}

/// <summary>
/// Determines whether the first specified API version is less than or equal to the second specified API version.
/// </summary>
/// <param name="version1">The first API version to compare, or <c>null</c>.</param>
/// <param name="version2">The second API version to compare, or <c>null</c>.</param>
/// <returns><c>true</c> if <paramref name="version1"/> is less than or equal to <paramref name="version2"/>; otherwise, <c>false</c>.</returns>
public static bool operator <=(ApiVersion? version1, ApiVersion? version2)
{
    if (version1 is null)
    {
        return true;
    }

    return version1.CompareTo(version2) <= 0;
}

/// <summary>
/// Determines whether the first specified API version is greater than or equal to the second specified API version.
/// </summary>
/// <param name="version1">The first API version to compare, or <c>null</c>.</param>
/// <param name="version2">The second API version to compare, or <c>null</c>.</param>
/// <returns><c>true</c> if <paramref name="version1"/> is greater than or equal to <paramref name="version2"/>; otherwise, <c>false</c>.</returns>
public static bool operator >=(ApiVersion? version1, ApiVersion? version2)
{
    if (version1 is null)
    {
        return version2 is null;
    }

    return version1.CompareTo(version2) >= 0;
}

}
Additional context
No response

In short, this feature has been asked for in the past and it is unlikely to ever happen. An API version is not like a Semantic Version. In fact, one could argue that there's no point in even having support for a minor version, but it's very well-established. A patch version definitely has no place in an API version. An API version is not like a binary version. In fact, it shouldn't be thought of as a version at all (though that's the established vernacular). It is much more like a media type. API versions must be discrete and are aligned to wire contracts. Patching is useful and interesting for binary components, but is meaningless to a client over HTTP. Patches are not to have visible changes. If there is no visible change to a client, then why advertise it and make a client know about it?
Based on your example, it would seem to based off older versions. IApiVersionParser was introduced in 6.0 and provides a common abstraction for parsing API versions. That separates parsing from the ApiVersion type. This makes it much easier to subclass ApiVersion and change it to suiting your needs, such as adding a PatchVersion property. Parsing, however, is only part of the picture. You still have to consider hashing, comparison, formatting, and use in attributes. Very few people have been willing to put in the effort to achieve a comprehensive solution.
The only reason that ApiVersion is a class instead of a struct or interface is that you cannot extend structures and interfaces could not have operator overloads until C# 11. Ideally, some interface IApiVersion would implement IFormattable, IEquatable, IComparable, and have no other members. The reasons ApiVersion stayed largely unchanged was for backward compatibility and limitations in the language. I've considered improving things more, but it I have weigh what could result in a significant change.
#990 discusses how to create a custom ApiVersion. There is also a link to a Gist that I created that provides a bare minimum working implementation end-to-end. If you really want to support Semantic Versioning, that should be enough to get you started.

Could an option be added to the action?

I'm not sure I follow your question. What would the option be? How would it be applied to an action? What would it look like?

Last call. Was there any follow up discussion or elaboration that you wanted to provide? Even if there were a way to match a Semantic Version, it probably isn't that valuable. If you have a sound version policy, say N-2, what's the value in matching /value?api-version=1.*? There would never be more than 3 possibilities. This approach is pretty nonsensical for a date such /value?api-version=2023-08-01. It's not clear how you envision this working.


Add the ability to generate Tls certificate with QED command tool

Improve our generate command in QED command tool and enable it to generate self-signed Tls certificate.
This feature allow us to remove openssl (external|sys tools) dependency in our test scripts.

I would love the commands be like:
qed generate self-signed-cert -> this is the command as it is now
qed generate csr -> generates a  csr to be signed by an external CA

I would love the commands be like:
qed generate self-signed-cert -> this is the command as it is now
qed generate csr -> generates a csr to be signed by an external CA

Renaming done and you can also check my proposal for CSR generation  in https://github.com/BBVA/qed/pull/138/commits/206e8711900c0b5b5b5e4dbac2763a53afbd50d7


plotting from command line

Does anyone else have the problem of iplot not working well from the command line?  When I want to plot something from IPython for example, I run something like df.iplot().  This executes, but nothing shows up.  The way around it has been:
# ensure plotly/cufflinks plots to browser
# cufflinks must be imported AFTER this
import plotly.io as pio
pio.renderers.default = "browser"
import cufflinks as cf

df.iplot()

Ok, there are a few ways to do this (although some things seem buggy).
# opens as a plot.ly plot on their site, unless you do cf.go_offline()
df.iplot(asPlot=True)

# this should show the image, but doesn't
df.iplot(asImage=True, display_image=True)

@nateGeorge when you say "plotting from the command line" you mean in a regular terminal with a regular python shell, or in ipython in a terminal, or in VSCode or PyCharm or Spyder or... ?

I used IPython actually.

OK so here's whats happening:

IPython in a terminal can't display images inline in general: it always needs to defer to some external system to open a window to render an image, so asImage isn't going to do anything there. This isn't a bug so much as a limitation of the terminal :)
Cufflinks wraps plotly and chart_studio as "offline" and "online" modes
In "offline" mode this means you'll use the renderer framework from plotly, and setting the default renderer to browser will get you a browser window with a plot in it, all locally-hosted
In "online" mode you'll upload your chart to Chart Studio Cloud and pop open a browser window that displays that.

If you don't want to set the default renderer in offline mode, you can use something like df.iplot(asFigure=True).show("browser") or df.figure().show("browser")

Awesome, thanks, much appreciated.


Event - use api version

Short description:
More details:
What this PR does / why we need it:
Which issue(s) this PR fixes:
Special notes for reviewer:
Bug:

/verified

/approve


[Bug Report]: EventHub role assignment name not unique across EventHub namespaces and EventHub

Describe the bug
Hello,
if you apply the same kind of role assignments to 2 EventHubs from the same namespace, you'll bump into an error such as:
"Azure Deployment Error “Tenant ID, application ID, principal ID, and scope are not allowed to be updated".
It comes from the fact there's a bug when the name of the azure role assignment is forged in https://github.com/Azure/ResourceModules/blob/main/modules/event-hub/namespaces/eventhubs/.bicep/nested_roleAssignments.bicep
resource eventHub 'Microsoft.EventHub/namespaces/eventhubs@2022-01-01-preview' existing = {
  name: '${split(resourceId, '/')[8]}/${split(resourceId, '/')[10]}'
}

resource roleAssignment 'Microsoft.Authorization/roleAssignments@2022-04-01' = [for principalId in principalIds: {
  name: guid(split(resourceId, '/')[0], split(resourceId, '/')[1], principalId, roleDefinitionIdOrName)

That should be aligned as:
resource eventHub 'Microsoft.EventHub/namespaces/eventhubs@2022-01-01-preview' existing = {
  name: '${split(resourceId, '/')[8]}/${split(resourceId, '/')[10]}'
}

resource roleAssignment 'Microsoft.Authorization/roleAssignments@2022-04-01' = [for principalId in principalIds: {
  name: guid(split(resourceId, '/')[8], split(resourceId, '/')[10], principalId, roleDefinitionIdOrName)

To reproduce
Apply the same kind of role assignments to 2 EventHubs from the same namespace
Code snippet
No response
Relevant log output
"Azure Deployment Error “Tenant ID, application ID, principal ID, and scope are not allowed to be updated".

Hey @sebbrochet,
good catch. I'd suggest to apply the approach we're also using for, for example, the Key Vault Secrets Role Assignments:
resource secret 'Microsoft.KeyVault/vaults/secrets@2022-07-01' existing = {
  name: '${split(resourceId, '/')[8]}/${split(resourceId, '/')[10]}'
}

resource roleAssignment 'Microsoft.Authorization/roleAssignments@2022-04-01' = [for principalId in principalIds: {
  name: guid(secret.id, principalId, roleDefinitionIdOrName)
  (..)
}

In other words: No split whatsoever in the Role Assignment name, but instead the full resource ID.


Meta data for DataFrame and Column

In order to not lose information that is encoded in DataFrames and Columns that is not covered by our API, we may want to provide extra metadata slots for these.
One may argue that this should be covered in the API, and this defeats the purpose of a standard, but I think it's a very pragmatic approach to guarantee lossless roundtripping for information outside of this standard and help adoption (because there is an escape hatch).
Example metadata for a dataframe

path: for when it's backed by a file or remote
description: metadata describing the dataframe
license: CC0, MIT
history: log of how the data was produced

Example metadata for a column:

unit: string that describes the unit ('km/s', 'parsec', 'furlong')
description: metadata describing the column
expression: in vaex, this is the expression in string form
is_index: an indicator that this column is the index in Pandas.

This could also help to round trip Arrow extension types: https://arrow.apache.org/docs/python/extending_types.html and I guess the same holds for Pandas.
An implementation could be a def get_metadata(self) -> dict[str, Any] where we recommend prefixing keys with implementation specific names, like 'arrow.extention_type', 'vaex.unit', 'pandas.extension_type_name' etc.
Commonly used keys could be upgraded to be part of the API in the future (non-prefixed keys) that we formalize and document.
FYI: metadata is a first-class citizen in the Clojure language https://clojure.org/reference/metadata

This was implemented in gh-43, so closing this issue.


Python3.10 not recoginized and source not recognized?

im having an issue for installation atm, it worked the first time i installed it. but after deleting it.
im trying to install again, and im getting these error

Hm. Not really sure what's going on there but it sounds like an issue with your venv config? Given that it's been a while I'll close this, but if you're still having problems (and it's not venv related) feel free to reopen.


microk8s.helm3 not able to follow "helm.sh/hook-weight"

@Pradumn-18 i am not too familiar with helm, is the hook-weight already available on helm v3.0.2?  Since the helm3 addon is kindda old though.
But if you are keen to try the latest/stable or latest/edge just to see if it works, helm3 addon was upgraded to 3.5.0.

I am closing this issue for now.


DSD-500: Carousel hook

Fixes JIRA ticket DSD-500
This PR does the following:

This adds a useCarouselStyles custom hook that returns two functions nextSlide and prevSlide used for sliding programmatically, and a carouselStyle JS object used for styling the carousel wrapper. All three must be used together to be able to "slide" within a container wrapper for slide elements.
This is implemented in the Tabs component for the mobile view.
This wraps around the container, so if you reach the last element and invoke the nextSlide function, it'll go to the first slide. Likewise, if you are on the first slide and invoke the prevSlide function, it'll go to the last slide.
@nypl-wluisi I think this should work well for the Slideshow component but if it needs adjuments, that's fine. Let me know or you can modify it once the Slideshow work begins. Either way, this should help start the implementation for that component.
@bigfishdesign13 I need some help with the focus styles for the buttons. I can't get the left or right focus outline to display correctly but maybe it's not a big deal:

How has this been tested?
Through Storybook's responsive viewport feature.
Checklist:

[ ] I have updated the Storybook documentation accordingly.
[ ] All new and existing tests passed.

Front End Review:

[ ] View the example in Storybook

Tugboat has finished building the preview for this pull request!
Link:

https://pr699-dkzycddnjdj3vkwnvibrvhb5jdo7fece.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/615347247207f285e32cb7e2

Tugboat has finished building the preview for this pull request!
Link:

https://pr699-dkzycddnjdj3vkwnvibrvhb5jdo7fece.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/615347247207f285e32cb7e2

Tugboat has finished building the preview for this pull request!
Link:

https://pr699-dkzycddnjdj3vkwnvibrvhb5jdo7fece.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/615347247207f285e32cb7e2

Tugboat has finished building the preview for this pull request!
Link:

https://pr699-dkzycddnjdj3vkwnvibrvhb5jdo7fece.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/615347247207f285e32cb7e2

Tugboat has finished building the preview for this pull request!
Link:

https://pr699-dkzycddnjdj3vkwnvibrvhb5jdo7fece.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/615347247207f285e32cb7e2

Tugboat has finished building the preview for this pull request!
Link:

https://pr699-dkzycddnjdj3vkwnvibrvhb5jdo7fece.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/615347247207f285e32cb7e2

Tugboat has finished building the preview for this pull request!
Link:

https://pr699-dkzycddnjdj3vkwnvibrvhb5jdo7fece.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/615347247207f285e32cb7e2

Ready for review @bigfishdesign13  @nypl-wluisi

Tugboat has finished building the preview for this pull request!
Link:

https://pr699-dkzycddnjdj3vkwnvibrvhb5jdo7fece.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/615347247207f285e32cb7e2

Tugboat has finished building the preview for this pull request!
Link:

https://pr699-dkzycddnjdj3vkwnvibrvhb5jdo7fece.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/615347247207f285e32cb7e2

Tugboat has finished building the preview for this pull request!
Link:

https://pr699-dkzycddnjdj3vkwnvibrvhb5jdo7fece.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/615347247207f285e32cb7e2


Setup pipeline for qpid-proton-j-extensions

We need to setup a pipeline for the Maven package at:
https://github.com/Azure/qpid-proton-j-extensions
This package is used by both ServiceBus and EventHubs in the Azure SDK for Java. We would like to have an end-to-end pipeline for this like we do the other Java libraries, but for now this package should remain outside the Java mono-repo.

/cc @conniey


IS [5.9.0- 5.10.0] Add template mapping for  configs exists in the identity.xml

Currently in the IS 5.9.0. and 5.10.0 , There is no template mapping  for the ResourceAccessControl  in the identity.xml
Due to that it is unable to configure and maintain the configs via deployement.toml.
Shall we take a step to fix this.

Can be done by adding following config in the deployment.toml
[[resource.access_control]]
context=""
secure="<true/false>"
http_method=""
permissions=[""]
hence closing the issue


Future features

Do you've some plan to future features of project? I would like to contribute.

Probably more RFC compliance and testing against real or debugging servers. The emphasis of the library is not so much on "throw everything inside" but more about a very polished set of tools, I.e. stability over heaps of features.
That being said feel free to work on these things I have in mind:

Internationalised headers
Perhaps a SMTP and POP3 adapter so that transports are more swappable, we'll see how that goes

So I've made a head start on internalisation, you can check out the internationalised-headers branch and perhaps clone from that. Or submit your own version, I'm okay either way and am open to any suggestions.


Handling path parameters before calling the path handler.

From the petstore example:
/pets/{pet}:
    get:
      description: Returns a user based on a single ID, if the user does not have access to the pet
      operationId: findPetById
      parameters:
        - name: pet
          in: path
          description: ID of pet to fetch
          required: true
          type: integer
          format: int64

Using Express' router.param(), you could do something like this:
router.param('pet', function(req, res, next, id) {

  // try to get the pet details from the Pet model and attach it to the request object
  Pet.find(id, function(err, pet) {
    if (err) {
      next(err);
    } else if (pet) {
      req.pet = pet;
      next();
    } else {
      next(new Error('failed to load pet'));
    }
  });
});

This is convenient because your path handlers do not have to do any logic to fetch the pet, either the handler is called and req.pet is already set, or it is not called at all.
petController.findPetById = (req, res, next) => {
  res.json(req.pet); // req.pet is already set by the router.param above
};

Can swagger-tools' router do something similar?

Not that I'm aware.  swagger-metadata handles parsing all parameters and sets req.swagger.params accordingly.  I'm not sure when router.param is called but I bet it runs before any of the swagger-tools middlewares.  I'm not sure I can do more so I'm going to close this but I'd love to keep talking about it.

Fair enough. It's only for convenience, so that if you have multiple routes with {pet} it in, you would avoid having to fetch the pet in each corresponding route handler.

Yeah, I've used it in the past but swagger-tools only tries to solve the problems related to Swagger and does not try to get involved in business logic, like looking up things.  You could just as easily do this though by using router.param and updating req.swagger... appropriately.  I'm open to suggestions as long as they are related to Swagger or making the middleware better.

I just ran into it while moving our entire API over to swagger. We have a piece of code that does "app.param(...)" (see [http://expressjs.com/en/api.html#app.param]). This middleware runs before any handler of any route, but the key point is that it is local only to the router that it is called on. I have about 20 endpoints that all make use of this so I'm currently looking for a solution - apart from manually calling the existing function...


How to output log to file?

How to configure log output to file?I want to check some errors through the log。
【The program was running for a while and it was not recorded abnormally. Restarting IIS was normal again】

This is bit outside of the scope of this project as we now run inside of docker containers and not IIS. I'd recommend taking a look at this: https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/iis/logging-and-diagnostics?view=aspnetcore-5.0 If you could post your solution as comment below that might help others who are also running under IIS. Also, I'm curious what version you are running.

这有点超出该项目的范围，因为我们现在在docker容器而不是IIS中运行。我建议您看一下：https : //docs.microsoft.com/zh-cn/aspnet/core/host-and-deploy/iis/logging-and-diagnostics?view=aspnetcore-5.0如果可以的话在下面将您的解决方案作为注释发布，这可能会对在IIS下也运行的其他人有所帮助。另外，我很好奇您正在运行哪个版本。

IIS can output the log to disk when stdoutlogenabled = true is turned on. The above is the error message. It's too difficult~

I solved this problem because I didn't turn on [storage: provider = folder; Path=..\storage] configuration. If you do not have this configuration, it seems that the program can run normally. It will not work after a certain time. Because job can not work normally, thank God for your reply. Thank you!

That error is due to the storage path not being the same in the api and the jobs. You'd need to make sure you attached volume and it's pointing to the same folder. I'm closing due to the original issue being resolved by the stdoutlogenabled = true


Does it support arm board?

My arm board is :
aarch aarch64 GUN/Liunx, Ubuntu
but can't install raisim, found the introduction for raisim:
######################################
System Requirements
Linux
We recommend ubuntu 20.04 or higher but RaiSim might work on other distributions. Works only on a X86 CPU with an AVX2 instruction set (Intel Haswell or higher).
#######################################
If I want to install the env for the leg, I need to install ocs2, and ocs2 needs raisim, so it can't run on arm board, right?


DBeaver Code Completion shows no proposals when using dblink

Description
when I use dblink in dbeaver, it doesn’t work properly.
Example code)
Select *
from inst1.mydatabase@dblink2 a
Where a. -> sql assist or code completion is not working here.
DBeaver Version
Community 23.1.0
Operating System
windows 10 1607
Database and driver
oracle 19c
oracle jdbc driver 11
Steps to reproduce
1.connect to oracle
2.input query like
"Select *
from inst1.mydatabase@dblink2 a
Where a."
3.sql assist or code completion is not working (no popup to provice proposals)
Additional context
No response

Thank you or bug report.

I'm having same issue as @ksside request #20375   .  It would be very practical if implemented.
Thanks!


transfer problem

How to transfer model.ckpt-152000.meta into .pb file, we will apply to graph_transfer?
Thank you!!

This question is better asked on  StackOverflow since it is not a  bug or feature request. There is also a larger community that reads questions there. Thanks!


website: text block too narrow

🐛 Bug
If I search something on the search bar, the website will look like this:

CC @neuromusic @codemonkey800
Last time I moved an issue to napari-sphinx-theme I was totally wrong so I'll wait on this one until I get a few more eyeballs on this. 😂

I just checked the dev site and everything seems fine there... So I think we can close this :)


Fix | Static Analyzers

Hello!

Type: bug fix | code quality
Link to issue:

In raising this pull request, I confirm the following:

[X] I have checked that another pull request for this purpose does not exist
[ ] I wrote some tests for this PR
[ ] I updated the CHANGELOG

Small description of change:

Improved github actions to run all checks even if some check failed

Thanks

Thank you


Add support for readers and writers to Tar

This fixes #20 for the Tar Archiver implementation.

Accidentily opened this pull request to my own repo.


Release Google.Cloud.Datastream.V1 version 1.0.0-beta02

Changes in this release:
New features

Add Location mixin to Google.Cloud.Datastream.V1 (commit 26dccaf)

Created release for Google.Cloud.Datastream.V1-1.0.0-beta02


Use errors.Is to check for a specific error

Signed-off-by: Fish-pro<EMAIL_ADDRESS>
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
comparing with != will fail on wrapped errors. Use errors.Is to check for a specific error
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/lgtm
/approve
Thanks!


Exam mode security guard

This MR introduces a simple exam mode security guard.
If we are running in exam mode, we replace the current ctx's RunConfiguration to a default one that has no meta tests or code checks inside.
We also delete the Configuration.java and the Configuration*.class compiled version. This means any attempt to read the file from the disk or to load it via reflection should fail.

Out of curiosity because I don't remember the pipeline too well -- is there a chance that the old Configuration.class containing the metatests gets classloaded before it is deleted? Is it possible that a representation of this class still exists in memory?

It's definitely possible, @Arraying, that's what my follow-up commit handles. I reset the classloader now!

Another comment: we remove the RunConfiguration, but would it perhaps be possible for someone to retrieve references to the previously instantiated MetaTest / CodeCheck objects?

Good catch, @martinmladenov! I'll copy the other parameters from the RunConfiguration.
No, for multiple reasons. The attacker can have access to the MetaTest class/interface but never on the concrete one we have that's produced by the call to metaTests() or whatever in our concrete Configuration implementation. Given that their code can't get access to our Configuration class anymore, there's no chance.

Done, @martinmladenov! Please, take a look at it again and merge it if ready. I think we need better Selenium integration tests. I'll open a ticket for that.


Change in smoothing quality (e.g. Rounded Corners) @v2.5.2

Description
In this commit https://github.com/JimBobSquarePants/ImageProcessor/commit/d3aaab4f36d58949b03766576eb9b26987c18fc1#diff-9dc0ac7d60ebb80477dfc99ca385c8faL111
The image quality settings were centralized but it looks like smoothing was not carried over (perhaps intentionally?). This results in reduced quality for some processors. Not sure if this was intended or not.
With Smoothing (High Quality)

After change (No smoothing)

Steps to Reproduce

Run ImageFactory.RoundedCorners() at latest version

Will have a release out soon to fix this. Thanks for raising it.


[Feature]: Exportable configuration

Feature description
It would be a nice to have to export my configuration and associated folders into a singular folder so that I can back up any backgrounds, themes, settings, and presets to GitHub so I can use it in the future if I need to reinstall my operating system, without worrying about anything being lot
Implementation Suggestion
N/a, just an "export" command with WSH to a file path would be great
Anything else?
n/a

As a workaround, all your configs are stored in ~/.config/waveterm, should be easy enough to add to your dotfiles sync script.

Hi, can I work on this?

Sure, I'll assign to you. What's your planned implementation? I was thinking it could be cool to zip up the config dir and put it at a user-provided directory

Feature description
It would be a nice to have to export my configuration and associated folders into a singular folder so that I can back up any backgrounds, themes, settings, and presets to GitHub so I can use it in the future if I need to reinstall my operating system, without worrying about anything being lot
Implementation Suggestion
N/a, just an "export" command with WSH to a file path would be great

Yeah, I think this would be cool. I'll go with this approach then.

If the PR looks good I'll update the documentation for this command.

The PR looks good to me, made one small suggestion based on the CodeRabbit comment, but otherwise I think it's good to go!

Wouldn't it be a good idea to associate this Feature with the PR itself?

Alright, now we need another to import configs from a path. lmfao

The logic there is a bit trickier, need to merge with existing configs and determine the override logic there


Test doesn't start after cancel on iOS

Hello. Suddenly got such problem -> after cancel test, new one doesn't start. isTestInProgress() returns true.

Hello. Suddenly got such problem -> after cancel test, new one doesn't start. isTestInProgress() returns true.

I am currently working on it. Thank you for reporting the issue!!

Hello, any updates on this issue?

Hello. Any update on this issue ? Thanks


Export certificate error

Getting error trying to export cert using any of the plugins. My guess is that is because there is no chain. I can't see chain information on newly created certificate from CFSSL CA, its blank.
Error:
 [2019-03-28 13:58:17,566] ERROR in schema:  Traceback (most recent call last):   File "/usr/local/src/lemur/lemur/common/schema.py", line 161, in decorated_function     resp = f(*args, **kwargs)   File "/usr/local/src/lemur/lemur/certificates/views.py", line 1013, in post     extension, passphrase, data = plugin.export(cert.body, cert.chain, cert.private_key, options)   File "/usr/local/src/lemur/lemur/plugins/lemur_csr/plugin.py", line 93, in export     create_csr(body, chain, output_tmp, key)   File "/usr/local/src/lemur/lemur/plugins/lemur_csr/plugin.py", line 42, in create_csr     assert isinstance(chain, str) AssertionError

Hello, unfortunately we do not use CFSSL internally and thus are unable to support this plugin.
The primary code contributors appear to be @chadhendrie , @johanneslanger and @alwaysjolley who may be able to investigate.

you need to include your CA certificates in the lemur configuration. See this document for more details
https://www.howtoforge.com/tutorial/integration-of-cfssl-with-the-lemur-certificate-manager/


Getting Search + Buttons in navigation bar

I'm trying to find a way to make the navigation bar entirely a search box except for 1-2 buttons on the very right. Right now I can get the icon buttons to show up on the same row but the search box is really tiny (I want to fill all space except for the space occupied by the right button(s)).
Is there a built-in way to do this in materialize that I'm missing or do I need to do some CSS overriding? Which the latter is fine but I wanted to ask to see if I'm missing anything obvious.
Here is some code I'm playing with:

        <form class="left">

                <input id="search" type="search" />

        </form>

maybe

        <form>

            <input id="search" type="search" />
            search

        </form>

close

link

Oh. Not sure why I didn't think of using the grid. Ugh, thanks. I'll try that tonight.

There is a slight bit of padding but otherwise this is good. I can go from here; thanks!

yeah this is also my trouble

This solution is not working on
Google Chrome	52.0.2743.82

Yep, I can confirm this solution breaks on chrome

Solution ->
input[type="search"] {
height: 64px !important; /* or height of nav */
}

Thanks for the code snippet, I used this and also added close icon to my search-form but close icon never shows up.. Any work arounds?

I had this same problem of stylizing the search bar in the navalize of materialize, I got the code snippet from friend @ahlechandre ahlechandre and I modified it to stay as I wanted, follow the code.
HTML
`

Logo

search

Produtos
Sobre
Contato

`
CSS
`input[id="search"] {
height: 45px !important;
  margin-top: 10px;
   border-radius: 5px;
}
.search-input{
      background-color: #82b1ff ;
  height: 45px !important;
  margin-top: 10px;
  border-radius: 5px;
}
nav .input-field label {
    top: -8px;
    left: 0;
}`

I had this same problem of stylizing the search bar in the navalize of materialize, I got the code snippet from friend @ahlechandre ahlechandre and I modified it to stay as I wanted, follow the code.


Avoid returning None

Having been stung by the billion dollar mistake a couple of times now (issue #1) it might be a good swap None for a sensible default where possible. For instance, return key number -1 instead in MenuSelectionHandler. In that case we're still going to get an exception down the line, KeyError probably, but it's a good practice to develop.

Menu and MenuSelectionHandler use Menu.NO_SELECTION rather than None.
Hardware still uses it, but we can deal with that when we remove the entire Hardware class as per issue #5.

And it's cropped up again

Menu.fill_missing_options() should solve the problem.


Update Developer's Guide for v1.4

Fixes #212

This should be merged before https://github.com/AY2122S1-CS2103-W14-2/tp/pull/223, which will update ModelClassDiagram.puml.


crash, exit code -2

Environment

LWJGL version: 3.1.0
LWJGL build #: 0
Java version: 8
Platform: Windows
Module: core

Description
I copied hello world example from the website and it works good, but when I close window it crashes and I get error
ERROR: JDWP Unable to get JNI 1.2 environment, jvm->GetEnv() return code = -2
       JDWP exit error AGENT_ERROR_NO_JNI_ENV(183): [util.c:840]

Is this a lwjgl issue, or is something wrong with my java binaries?

LWJGL version: 3.1.0

Could you please try with 3.1.6 instead? Also, could you post the full output of java -version?

Ah sorry, my version is actually 3.1.6, build 14, I was looking at wrong numbers. And here is output of java version
java version "1.8.0_162"
Java SE Runtime Environment (build 1.8.0_162-b12)
Java HotSpot 64-Bit Server VM (build 25.162-b12, mixed mode)

BTW, I am using Eclipse to build project

Are you running your application in a debugger? Does it happen if not?

It is hard to tell, since the error dosn't display always. I had to test it multiple times and here are the results (numbers represent number of times tested):

No crash
Crash

Debugger
92
3

No debugger
88
0

Apparently it's a known low-priority issue marked as won't fix:
JDK-6476706
There's a call to GetEnv somewhere, probably a race condition when exiting the debugging session makes it return NULL.
IMHO it's harmless and can be ignored. It's not related to LWJGL and won't happen in non-debugging runs.

Ok, I'm closing this issue.


Homebridge unreachable after iPhone config

My Homebridge, since 22nd June has been unreachable from my iPhone. (previously no problems)
I quit homebridge, cleared my HomeKit config, persist folder, and re-started the homebridge.
I was then able to discover the HomeBridge on the iPhone, and add to a room.
Shortly thereafter, HomeBridge disconnected, and won't reconnect unless I repeat the above steps :-(
Any thoughts?

sigh... it came back on its' own


Fix glam matrices

Hi!
I noticed that matrix types with glam stopped working in 0.6.0. A condition got inadvertently flipped here: https://github.com/LucentFlux/naga-to-tokenstream/commit/90c32ee6095056be4f1018210b31a733d4023ca1#diff-ed12f5ea605f23bb4d26cc65778e3daff9db3eec40e2abfc82beafca130a99a0L98-R100. This PR fixes that.

Ah amazing, thanks!


Add retry to GpuHashJoin.filterNulls

close https://github.com/NVIDIA/spark-rapids/issues/8347
This PR adds the retry support to GpuHashJoin.filterNulls.

build


Light command sink on Metal

Our Deferred command sink does much less work at recording time than Immediate, but it was still designed to do as much work as possible without having an actual metal command buffer underneath. For latency-limited application, we may need a command sink that really tries to minimize that work.
A Light sink would, for example, store the hashmap keys of render passes and depth-stencil states without bothering to actually request those states. This would be done at submit time, which has the benefit of being (largely) single-threaded, so at least no lock contention is expected there. We could even dispatch an async task to create a state once we get an unknown key.

I tried this approach and it doesn't appear to be giving us anything. Code will stay at https://github.com/kvark/gfx/tree/mtl-defer-storage-access


Add profile XMLs for PHIN Specs 2.0 and 3.0

Should the folder names match verbiage that comes on actual HL7 messages so we can look up those profiles dynamically?

Yes that is a great idea... but I think we should do another issue for it @marcia-schulman . We'll need exact names for 2.0, 3.0, 3.1, etc.. coming from the message, such as NOTF_ORU_V3.0 for 3.0 and so on...

New Ticket was created #114 to rename folders


Add strict config duplicate handling, fixes

Fix output for configuration loader messages
Strict mode for configuration ids enforces clean directories as no
duplicate ids are allowed anymore.

Clean here means, that there must not be multiple configuration files using the same configuration ids!

So it has nothing to do with directories?


Set up VersionVigilante on the GitCommand.jl repository

This pull requests sets up VersionVigilante on the GitCommand.jl repository. cc: @DilumAluthge

Codecov Report

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

@@          Coverage Diff          @@
##           master    #33   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           7      7
  Lines          47     37   -10
=====================================
- Hits           47     37   -10

Impacted Files
Coverage Δ

src/util.jl
100% <0%> (ø)
:arrow_up:

src/env_mapping.jl
100% <0%> (ø)
:arrow_up:

src/public.jl
100% <0%> (ø)
:arrow_up:

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 392ca9b...8c8b4bb. Read the comment docs.


Revamped graphics

-revamped warehouses and storehouses entities graphics (hr and lr)
-custom icons for each warehouse
-3 new tech icons
-new mod thumbnail
-changed requirement for warehouse-logistics-research-1 from "robotics" to "logistic-robotics" (proposition)
-bots aim for the opening on the warehouse
-wire connector on warehouse appears only when connected
-updated changelog and version

Looks good so far, I want to do a bit of testing in game, specifically to do with the research, I think you're right needing logistics science for the logistics warehouse

Would you be willing to share the layered source files with us? They clearly don't belong in the repo, but if I never have to pixel-edit another sprite like I did to add the Buffer variants a while back, it'll still be too soon! 😂

I mean, why not, if he's willing to share the source files, then maybe it'll help keeping them around in the repo if we need to edit the sprites, then we have the source layers to help us? (maybe if they add different types of chests or something)

Would you be willing to share the layered source files with us? They clearly don't belong in the repo, but if I never have to pixel-edit another sprite like I did to add the Buffer variants a while back, it'll still be too soon! 😂

I mean, why not, if he's willing to share the source files, then maybe it'll help keeping them around in the repo if we need to edit the sprites, then we have the source layers to help us? (maybe if they add different types of chests or something)

I can, but it's 103MB (including source for icons etc.), so where should I transfer it? I can give it to you directly on google drive or something if you'd like. I trust you with it. It's in PSD (CS6 minimum).
For the shadow thing, I can correct this, I just thought it wasn't so important.
I could add a wire connector to the storehouse if you wish, that's a bit more work. Once this PR is resolved, I'll merge and do these small stuffs.

You'll find in line-notes a few things I'd rather be left to a separate PR or the release process. I have updated the checklist in #51 with all the tasks that I think this PR takes care of. Happy to have input from you on the remaining stuff, like multiple icon sizes, LUTs, and animated warehouse roofs!
Animated roofs (you mean open close the chest part?) I can do, but that'll require a bit of work (I need to make an animation of just the opening based on existing chests and place it exactly right on top of the screen, and for each one),
LUTs, I have no idea what it implies, but I'm curious. I've read the blog post, but I'm curious about it, as it could also improve BioIndustries. For multiple icon sizes (mipmaps), no idea if it's really necessary. Maybe there's a script out there to make them automatically? In bioindustries, I removed them all, it's easier to work with, and I found no visible difference.

Well I'm doing some of these corrections right now, adding the connector image to the storehouse too if possible. I'll push these in this PR.

I'm making the opening chest animation, so you'll be able to close all points in #51 except LUT (as I said, I don't know what it is)

I was very busy yesterday and today, so it took more time than it should, but here it is, chest animations, and corresponding sounds.
I think it's ready now, unless you still have something !

BTW, do you need a white version ?
https://github.com/Warehousing/Warehousing/pull/87
I don't know what the white logistic chest is used for, but it'd be pretty quick to export that too.

I merged the linked-chest variants, so let me know if you need assistance with a rebase! Replacing the white sprites added by #87 would be much appreciated, yes!
Vanilla Factorio defines a "linked chest" that's only usable in the map editor. I guess scenario authors use them for making the same items available in multiple locations? Whatever the purpose, someone asked for warehouse versions and it was an easy code change, so I did it.

so let me know if you need assistance with a rebase

Ok I'm stuck, it won't let me use the resolve conflict tool on github on my pull request, only command line, but I use github desktop.
https://github.com/snouz/Warehousing/pull/1
Could you help please?

@snouz Rebased and cleaned up the whitespace errors etc. while I was at it.
Knowing that you use GitHub Desktop, I'm not sure the best way for you to start using the revised history… GHD is very limited in terms of what it will let you do. The simplest approach might be deleting your local repo and re-cloning. You can verify what changed in the force-push summary (also linked above in the event message).

Would you be willing to share the layered source files with us? They clearly don't belong in the repo, but if I never have to pixel-edit another sprite like I did to add the Buffer variants a while back, it'll still be too soon! 😂

I mean, why not, if he's willing to share the source files, then maybe it'll help keeping them around in the repo if we need to edit the sprites, then we have the source layers to help us? (maybe if they add different types of chests or something)

@Anoyomouse It already takes way too long to clone this repo because of the source-assets I added before thinking about long-term impact 😅 and that's only ~25MB of map + high-quality screenshot. A 100MB+ PSD would make life very annoying indeed. I'm honestly tempted to move those source-assets into a separate repo and scrub the history here, but that would be disruptive. For now it's just best to avoid adding any more unnecessary large files to this repo. We can keep the final PSD in its own repo, though? Maybe alongside the tools needed to generate some stuff like the multi-size icons.

@snouz Welp, I had to force-push two more times because I missed some whitespace errors… Definitely done now though!

Thanks, I think it's now all good !

Have you had the chance to review it? The update shouldn't break anything.

Have you had the chance to review it? The update shouldn't break anything.

It's not breakage that I'd worry about; the quick tests I've done already showed that things still work as expected. What I want to test more is just the experience of playing with the new graphics, to see if there are any obvious tweaks left to make while we still have your attention! My weekend was unexpectedly busy, and left me very little time for anything on the computer (including "real" work), but I promise that I'll take the first chance I have to finish putting these through their paces.
I did have a sec to clean up the history just now though. If I don't find anything to tweak, I'll just merge as soon as I'm done playing around with it. 😸

| to see if there are any obvious tweaks left to make while we still have your attention!
I'm not going anywhere : ), I can do modifications after, I'm still contactable everywhere and 2 updates means a lot more downloads, so the mod goes up in the list!

Finally had time to play around! This was ready right before a trip, and I had no idea how little free time I'd have while traveling. Returned home earlier this week.
I realized that no one updated the mod's Readme with credit for the revamp, so that's now done. Will merge.

@snouz @Anoyomouse Let's figure out where to keep the original working files, in case we need to make tweaks!

@snouz Oh, while I'm thinking about it, would it make sense at all to split the graphics up into (more) layers and use the game's ability to tint sprites? I'm wondering if that would save enough file size to be worthwhile, or even just separating the common pieces of the logistic variants from the uniquely colored parts. Future enhancement?

@snouz would you be willing to make me a thumbnail for my warehouse compression plugin so it looks basically the same as the base mod? I was thinking just changing the word "MOD" to "COMPRESSION"

@snouz would you be willing to make me a thumbnail for my warehouse compression plugin so it looks basically the same as the base mod? I was thinking just changing the word "MOD" to "COMPRESSION"

Sure !
Here's a fully customized thumbnail, tell me if it works for you
https://i.imgur.com/LyVeVXg.png

Perfect! Thanks man :D


updates envs

[x] Fix #941
[x] updates DLC example dataset from GIN

@bendichter @rly this fixes the import errors happening on tests. It also fixes an error due to a test DLC data that has been removed from GIN

LGTM!

PR tests and daily tests pass on this branch. Looks great. Thank you @luiztauffer !


fix: include message when assert.*() fails

Thanks @alxndrsn
It's published in<EMAIL_ADDRESS>


[DOC] add missing parameters for scala doc

Merged build triggered.

Merged build started.

Test build #41515 has started for   PR 8412 at commit 58e9d2e.

I think that's OK, but if you're going to add some params, it would be better to put effort into fully adding missing params for a whole file or package.

Test build #41515 timed out     for PR 8412 at commit 58e9d2e     after a configured wait of 175m.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41515/
Test FAILed.

Test build #1689 has started for   PR 8412 at commit 58e9d2e.

Merged build triggered.

Merged build started.

Test build #41530 has started for   PR 8412 at commit 8b41ddf.

@srowen , I added the missing params for APIs that intending to adding params in the whole file. I'm wondering whether those missing @param are on purpose or not. Since in current spark code, there are many places missing @param and @return, if we want to make the doc better, I think someone need to open a JIRA to solve it.
And I submit this PR because SparkContext.scala is a very import file in spark core, and It's constructor APIs should be with complete scala docs.

No problem, I'll merge it. I don't think these were omitted on purpose. You might look over this one file and see if anything else needs a touch up.

Test build #1689 has finished for   PR 8412 at commit 58e9d2e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #41530 timed out     for PR 8412 at commit 8b41ddf     after a configured wait of 175m.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41530/
Test FAILed.

Merged build finished. Test FAILed.


[question] How can I create a component using saga and export it as a package

How can I create a component (like a react-redux-saga-messenger) using redux and redux-saga and create a npm package that can be installed by other apps?
in the main app I want something like this
npm install react-redux-saga-messenger --save
import Messenger from 'react-redux-saga-messenger';

class App extends Component {
   render() {
       <Provider store={store}>
             <Messenger />
       </Provider>
   }
}

In your 'react-redux-saga-messenger', use default export to <Messenger /> component and named export to saga. In the host application, you also need fork messenger's saga in the root saga like this.
import { messenger } from 'react-redux-saga-messenger';

export default function rootSaga() {
  yield fork(yourSaga);
  yield fork(messenger);
}

@sibelius We recommend you to ask a question on StackOverflow with redux-saga tag.

@kuy what happens if I want to add two of these messengers components ? Their actions will conflict with each other right?

@sibelius please note that questions should be posted on SO. this is to keep the issues repo. focused on bugs & proposals.

ok, I will post on SO

I've found this package that solves this problem: https://github.com/threepointone/react-redux-saga


New: add infrastructure and business logic

This too-big pull request adds all of the pieces for an initial release.

Self-approved 🤫


Please include a license file.

Some companies won't let you incorporate your code into their project without a license file explicitly stating what rights others have to use your work.

+1 @emsifa, I was about to open an issue for this - is this possible?


[DOC-2554] Added security FAQs

DOC-2554
Added security FAQs
Signed-off-by: Smriti S<EMAIL_ADDRESS>

Preview environment: https://ci.pr.harness.io/pr-580

Preview environment: https://ci.pr.harness.io/pr-580

Preview environment: https://ci.pr.harness.io/pr-580

Preview environment: https://ci.pr.harness.io/pr-580

Merging this after verifying the local build. Error with 'publish docker image' is a known issue after migrating off of CI’s Cloud Infrastructure.