zsh command hash table is stale with system ruby

I have a zsh script to setup dev environments that create tmux sessions and windows. (see http://pastie.org/2643820)
Occasionally, a tmux window's zsh's command hash would still be pointing to the system ruby.
[master][~/code/ezpz] hash | grep ruby=
jruby=/Users/brice/.rbenv/shims/jruby
ruby=/usr/bin/ruby

[master][~/code/ezpz] env | grep PATH
PATH=/Users/brice/.rbenv/shims:/Users/brice/.rbenv/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin:/usr/X11/bin:/Users/brice/.rbenv/shims:/Users/brice/.rbenv/bin:/opt/local/bin:/usr/local/git/bin

[master][~/code/ezpz] ruby -v
ruby 1.8.7 (2010-01-10 patchlevel 249) [universal-darwin11.0]

[master][~/code/ezpz] which ruby
/usr/bin/ruby

[master][~/code/ezpz] /Users/brice/.rbenv/shims/ruby -v
ruby 1.9.2p290 (2011-07-09 revision 32553) [x86_64-darwin11.0.0]

Would it be a good idea to maybe include a call the builtin rehash after rbenv-init has been run? This seems to have solved my problem. I just do the following in .zshrc
export PATH="$HOME/.rbenv/bin:$PATH"
eval "$(rbenv init -)"
builtin rehash

Here is some background information I found on this issue from rvm's history: https://github.com/wayneeseguin/rvm/commit/f6473f29458d60201c47cf65ae7dde465ef0adfc

Seems like zsh really needs rehash after any calls to rbenv rehash.
Theres a few issues. rehash is specific to zsh so we just can't add a call to our scripts. We'd need to detect zsh which we're already doing a bad job already. See #59.
Also rbenv rehash spawns a subprocess so running rehash there won't affect your current shell.
So I don't really see a good way to approach this issue.
I am an unsophisticated shell user so I'm afraid I have little more to add other than these observations.

Josh, seems like we could force this by calling hash -r every time our rbenv function runs.

I think I'm having the same problem with zsh. I can't seem to use rbenv's ruby no matter what I try.
rbenv global
=> 1.9.3-p0

rbenv local
=> 1.9.3-p0

which ruby
=> /usr/bin/ruby

hash | grep ruby=
ruby=/usr/bin/ruby

I have the following in my .zshrc
export PATH="$HOME/.rbenv/bin:$PATH"
eval "$(rbenv init -)"
builtin hash -r

I don't actually know what the "hash" is or what rehash does but I'm sure I've typed it at least 20 times at various stages! Nothing seems to work.


Cannot save in google colab

This is my google colab file
https://colab.research.google.com/drive/1dnROnz1kDQsHI4ReTjjF79EnHc3GlvKd
For some reason I cannot seem to save any trained model in google colab.
It is giving some kind of segmentation error. The initial part of the file is the same as the template google colab file and the version of stable baselines is the latest as I am updating the stable baselines version.
However on trying to save it is giving this error.
!python train.py --algo ppo2 --env BipedalWalkerHardcore-v2 -f logs -n 10000
========== BipedalWalkerHardcore-v2 ==========
OrderedDict([('cliprange', 'lin_0.2'),
('ent_coef', 0.001),
('gamma', 0.99),
('lam', 0.95),
('learning_rate', 'lin_2.5e-4'),
('n_envs', 16),
('n_steps', 2048),
('n_timesteps', 100000000.0),
('nminibatches', 32),
('noptepochs', 10),
('normalize', True),
('policy', 'MlpPolicy')])
Using 16 environments
Normalizing input and return
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/stable_baselines/common/policies.py:436: flatten (from tensorflow.python.layers.core) is deprecated and will be removed in a future version.
Instructions for updating:
Use keras.layers.flatten instead.
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/op_def_library.py:263: colocate_with (from tensorflow.python.framework.ops) is deprecated and will be removed in a future version.
Instructions for updating:
Colocations handled automatically by placer.
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/math_ops.py:3066: to_int32 (from tensorflow.python.ops.math_ops) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.cast instead.
Saving to logs/ppo2/BipedalWalkerHardcore-v2_2
[e88f4df2d52c:03067] *** Process received signal ***
[e88f4df2d52c:03067] Signal: Segmentation fault (11)
[e88f4df2d52c:03067] Signal code: Address not mapped (1)
[e88f4df2d52c:03067] Failing at address: 0x7fee1aba120d
[e88f4df2d52c:03067] [ 0] /lib/x86_64-linux-gnu/libpthread.so.0(+0x12890)[0x7fee1de55890]
[e88f4df2d52c:03067] [ 1] /lib/x86_64-linux-gnu/libc.so.6(getenv+0xa5)[0x7fee1da94785]
[e88f4df2d52c:03067] [ 2] /usr/lib/x86_64-linux-gnu/libtcmalloc.so.4(_ZN13TCMallocGuardD1Ev+0x34)[0x7fee1e2ffe44]
[e88f4df2d52c:03067] [ 3] /lib/x86_64-linux-gnu/libc.so.6(__cxa_finalize+0xf5)[0x7fee1da95615]
[e88f4df2d52c:03067] [ 4] /usr/lib/x86_64-linux-gnu/libtcmalloc.so.4(+0x13cb3)[0x7fee1e2fdcb3]
[e88f4df2d52c:03067] *** End of error message ***
I want to save the logs folder on to my google drive or at least find a way to download them into my computer. But in this case I cannot even seem to save the file onto log it seems.

Hello,

Saving to logs/ppo2/BipedalWalkerHardcore-v2_2

looking at your stack trace, the model was saved in that folder.
I should add a warning, even though you get a seg fault, saving is working.


Shimmering Skies

It's me again!
I've been obsessively checking the API for photos of set 5 cards, and it looks like all the Shimmering Skies cards are on the API!
Hoping it's a smooth update this time! I still need to update my app for the new data formatting.

Yeah, I had been checking for the set 5 release every few hours since the start of the week too :D
I've been verifying the cards since Wednesday (local) evening. It's going decently quickly, though I know the Enchanted cards and especially the promo cards they added might pose an issue.
But I'm pretty confident that I can get at least the English datafile update out this weekend at the latest. The French update will come a few days after that.
And hopefully adapting to the V2 format changes will go smoothly for you!

I've just published a new version of the LorcanaJSON dataset with all the Set 5 cards, plus some extra promo and special cards, and a few tiny changes. As usual the details are in the changelog.
I hope they don't keep adding new card designs, if they didn't I would've been done a day ago :p


Nakamura test

Proof of concept np.loadtxt-based support for Nakamura corresp files. I guess the same could be done for the Nakamura match format.

The new np.loadtxt-based loader for Nakamura's match format was added, including documentation and tests.


getPixelColor returns no hex

Expected Behavior
return the hex value of a green pixel. in this example from: https://upload.wikimedia.org/wikipedia/commons/d/de/Color-Green.JPG
following code is used:
Jimp.read('https://upload.wikimedia.org/wikipedia/commons/d/de/Color-Green.JPG', (err, green) => {
   if (err) throw err;
   var color = green.getPixelColor(20, 20);
   console.log('getpixelcolor ?: ' + color);
   console.log(Jimp.intToRGBA(color));
   console.log('getpixelcolor hex:' + color.toString(16));
});

Current Behavior
I get a Pixel Color in decimal as a return value. which is some kind of magenta.
The RGB Conversion seems fine, just the "a" value cannot be correct as far as I know
this is the log:
21:50:23.023 | info | javascript.0 (4257) script.js.common.Garagentor: getpixelcolor ?: 16712191
21:50:23.024 | info | javascript.0 (4257) script.js.common.Garagentor: {'r':0,'g':255,'b':1,'a':255}
21:50:23.025 | info | javascript.0 (4257) script.js.common.Garagentor: getpixelcolor hex:ff01ff

Failure Information (for bugs)

Steps to Reproduce
described above

Screenshots

Context

Jimp Version: newest
Operating System: Linux
Node version: 14.21.0

Failure Logs
i don't have anything more

If anyone wants to submit a PR to fix I'm able to review and merge!

I needed that library for exact that reason :D i have the same bug.
The images look like that:
Top left small image is the original, the big one is the same but crafted via getPixelColor:

Is this actually an issue? I just wrote a script and everything seems to work correctly
import { Jimp, intToRGBA } from "jimp";
import { promises as fs } from "fs";

const buffer = await fs.readFile("test.png");
const image = new Jimp({
  width: 100,
  height: 100,
  color: 0x01ff00ff,
});

const copy = new Jimp({
  width: image.bitmap.width,
  height: image.bitmap.height,
  color: 0xffffffff,
});

image.scan((x, y) => {
  const color = image.getPixelColor(x, y);
  console.log(intToRGBA(color)); // prints: { r: 1, g: 255, b: 0, a: 255 }
  copy.setPixelColor(color, x, y);
});

await copy.write("./output.png");

The resulting image


Runtime exception for rasa run/rasa shell

Rasa version: 1.5.3
Rasa SDK version (if used & relevant): 1.5.2
Rasa X version (if used & relevant): 0.23.5
Python version: 3.7.1
Operating system (windows, osx, ...): Ubuntu 18.04.1 LTS
Issue:
abhishek@abhishek-laptop:~/Projects/python/rasa/chatbot$ rasa run
2020-01-04 11:55:48 INFO     root  - Starting Rasa server on http://localhost:5005
2020-01-04 11:55:51 INFO     absl  - Entry Point [tensor2tensor.envs.tic_tac_toe_env:TicTacToeEnv] registered with id [T2TEnv-TicTacToeEnv-v0]
2020-01-04 11:55:51.596725: E tensorflow/stream_executor/cuda/cuda_driver.cc:318] failed call to cuInit: UNKNOWN ERROR (303)
Error (including full traceback):

Command or request that led to error:

Content of configuration file (config.yml) (if relevant):

Content of domain file (domain.yml) (if relevant):

Thanks for raising this issue, @amn41 will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗


Notify the user of unsaved work when closing tab

Is your feature request related to a problem? Please describe.
-
Describe the solution you'd like
In case the user made changes to his file but didn't save and is closing the browser tab notify the user of unsaved work when closing tab
Describe alternatives you've considered
-
Additional context
-

:tada: This issue has been resolved in version 2.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Spark Roadmap 2018

Next year Spark is going to be faster, stronger, and better.
We are also going to rewrite everything in Rust.
Please add here all the ideas you had and would like to see happening in Spark 2018, starting from technical issues like 'better tests', 'finally getting to fixing that bug' and 'rewrite everything in Rust' up to products and systems that can be implemented on top of Spark.
Be specific where possible!

"rewrite everything in Rust" --- Why? What's wrong with JavaScript? Rewriting everything sounds bad for me.

Hi!
Rewrite everything in Rust is a joke.., albeit not a good one.
On Tue, 25 Apr 2017, 21:42 Roy Zahor<EMAIL_ADDRESS>wrote:

"rewrite everything in Rust" --- Why? What's wrong with JavaScript?
Rewriting everything sounds bad for me.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Midburn/Spark/issues/274#issuecomment-297126817, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHHcKIuaQakxdlk2YQStHXUzkoZx4vdZks5rzj6hgaJpZM4M6eaN
.


JR renders 105 (Param not allowed) for new columns added via migration

After adding a new column to a model and successfully running the db migration, JR refuses to acknowledge it and accept it as a JSON parameter. Instead it renders the 105 Param not allowed error. Or am I doing something wrong?

Did you add the new attribute to the Resource as well?

Yes

@lgebhardt I found out which line is failing: https://github.com/cerebris/jsonapi-resources/blob/master/lib/jsonapi/request.rb#L350. The include? test should be made with the formatted key since that's what in formatted_allowed_fields.
Also a correction of the original issue: Actually it doesn't fail for newly added columns; it fails for column names separated by underscores.
Here's one way to fix it (PR #158):
params_not_allowed.push(key) unless formatted_allowed_fields.include?(format_key(key).to_sym)


Added support for kedro namespaces for parameters

Now params which defined as outer_namespace.inner_namespace.param_name are converted to the following structure before adding to the pipeline:
{
  "outer_namespace": {
      "inner_namespace": {
        "param_name": "value",
        ...
      },
      "another_inner_namespace": {
        "param_name": "value"
        ...
      }
  }
}

Description
describe the purpose of the change here
Resolves https://github.com/getindata/kedro-kubeflow/issues/219
PR Checklist

[X] Tests added
[X] Changelog updated

Looks great, @DmitriyLamzin! Thanks for your contribution

Looks like CI is broken. I don't think my changes could break it.

@DmitriyLamzin You're right, the CI is broken, I have the fix ready, and we will merge it on Monday.

@DmitriyLamzin Released as 0.7.4 :tada:


Fix compile under catalyst (iosmac)

I guess the CI failure have been fixed in https://github.com/mixpanel/mixpanel-iphone/pull/878/commits/ae2e862c6da4bc9b47c0945063e744e733c9ddcb.
Can someone make this pull request again?

Move this one to https://github.com/mixpanel/mixpanel-iphone/pull/896, thanks so much!


✨ Add all maintainers to package.yml output using this workflow [python programming]

Currently, the workflow in this repo will create an output file called packages.yml that is a data file.
This data file drives our websites packages listing here. . The data file template for a single package is below.
This is useful however we need to add a all_maintainers list o that we can list all maintainers of the package not just the submitting author. Our new template does include a All current maintainers: so we should be able to parse that section of the issue .
Output packages.yml file currently look ssomething like this (NOTE that i've edited the categories format manually - a separate issue to be opened):
- package_name: xclim
  package_description: Climate indices computation package based on Xarray
  submitting_author:
      name: Trevor James Smith
      github_username: Zeitsperre
  repository_link: https://github.com/Ouranosinc/xclim
  version_submitted: v0.40.0
  categories:
    - data-munging
    - reproducibility
    - geospatial
  editor:
      name:
      github_username: Batalex
  reviewer_1:
      name:
      github_username: jmunroe
  reviewer_2:
      name:
      github_username: aguspesce
  archive: TBD
  version_accepted: TBD
  created_at: '2023-01-16T16:59:20Z'
  updated_at: '2023-04-12T20:21:36Z'
  closed_at:
  gh_meta:
      name: xclim
      description: 'Library of derived climate variables, ie climate indicators,
          based on xarray. '
      created_at: 07/27/2018
      stargazers_count: 225
      watchers_count: 225
      forks: 43
      open_issues_count: 49
      forks_count: 43
      documentation: https://xclim.readthedocs.io/en/stable/
      contrib_count: 26
      last_commit: 04/12/2023

We should add code that creates a section like this to the yaml file (maintainers list):
- package_name: xclim
  package_description: Climate indices computation package based on Xarray
  submitting_author:
    - name: Trevor James Smith
      github_username: Zeitsperre
  maintainers:
    - name:
      github_username: user_name_here

related TODO: leah should update all old issues to have all-maintainers!

Working on this issue (PyCon 2023) 😊👩🏻💻

@all-contributors please add @tiffanyxiao for code, review

Thank you so much @tiffanyxiao just added you as a contributor to this repo. your name & github profile will appear in our readme file here and in the next week you'll be added to our website as a contributor

hi @tiffanyxiao i wanted to mention that myself and @NickleDave will be around tomorrow. and while we did NOT sign up for a sprint, we are going to informally sprint on pyOpenSci tomorrow. If you are interested in joining us we'd love company and i'm willing to answer any and all questions! let me know if you are interested in another day sprinting on pyopensci things either here or via email leah at pyopensci.org ! 🎉  (and no worries if you are working on other things tomorrow either this is just an invite with no expectations!)

closing this as it has been completed!! thank you again @tiffanyxiao !!


Minor typo fix in index.html

Fixed the wording in the sentence and replaced the apostrophes with ASCII ones

Coverage remained the same at 100.0% when pulling c2efde0d36378508311ecfd45b2c2e9338974234 on reconrey:patch-1 into 1d3933b24dcaf94df12791134d975dccac8df79d on tiagoporto:master.


Update scala 2.12 to 2.12.16 in docs

Shouldn't this have been merged to main?

Oh, seems it's handled by https://github.com/VirtusLab/scala-cli/pull/1111.


[auto updater] white screen on updates

Getting the following errors - 20:02:46.377 › Error: Error: Cannot find latest-linux.yml in the latest release artifacts (https://github.com/smol-ai/GodMode/releases/download/v1.0.0-beta.4/latest-linux.yml): HttpError: 404
"method: GET url: https://github.com/smol-ai/GodMode/releases/download/v1.0.0-beta.4/latest-linux.yml\n\nPlease double check that your authentication token is correct. Due to security reasons, actual status maybe not reported, but 404.\n"
Headers: {
"cache-control": "no-cache",
"content-encoding": "gzip",
"content-length": "29",
"content-security-policy": "default-src 'none'; base-uri 'self'; connect-src 'self'; form-action 'self'; img-src 'self' data:; script-src 'self'; style-src 'unsafe-inline'",
"content-type": "text/plain; charset=utf-8",
"date": "Tue, 29 Aug 2023 00:02:17 GMT",
"referrer-policy": "no-referrer-when-downgrade",
"server": "GitHub.com",
"strict-transport-security": "max-age=31536000; includeSubdomains; preload",
"vary": "X-PJAX, X-PJAX-Container, Turbo-Visit, Turbo-Frame, Accept-Encoding, Accept, X-Requested-With",
"x-content-type-options": "nosniff",
"x-frame-options": "deny",
"x-github-request-id": "A657:06B4:5A1D5B:7E778F:64ED35A6",
"x-xss-protection": "0"
}
at createHttpError (/tmp/.mount_GodModgKrZiG/resources/app.asar/dist/main/webpack:/godmode/node_modules/builder-util-runtime/out/httpExecutor.js:14:12)
at o.handleResponse (/tmp/.mount_GodModgKrZiG/resources/app.asar/dist/main/webpack:/godmode/node_modules/builder-util-runtime/out/httpExecutor.js:116:20)
at ClientRequest. (/tmp/.mount_GodModgKrZiG/resources/app.asar/dist/main/webpack:/godmode/node_modules/builder-util-runtime/out/httpExecutor.js:82:26)
at ClientRequest.emit (node:events:513:28)
at SimpleURLLoaderWrapper. (node:electron/js2c/browser_init:2:49930)
at SimpleURLLoaderWrapper.emit (node:events:513:28)
at newError (/tmp/.mount_GodModgKrZiG/resources/app.asar/dist/main/webpack:/godmode/node_modules/builder-util-runtime/out/index.js:47:19)
at rawData (/tmp/.mount_GodModgKrZiG/resources/app.asar/dist/main/webpack:/godmode/node_modules/electron-updater/out/providers/GitHubProvider.js:108:54)
at t.GitHubProvider.info [as getLatestVersion] (/tmp/.mount_GodModgKrZiG/resources/app.asar/dist/main/webpack:/godmode/node_modules/electron-updater/out/providers/GitHubProvider.js:120:17)
at h.result [as getUpdateInfoAndProvider] (/tmp/.mount_GodModgKrZiG/resources/app.asar/dist/main/webpack:/godmode/node_modules/electron-updater/out/AppUpdater.js:319:13)
at h.doCheckForUpdates (/tmp/.mount_GodModgKrZiG/resources/app.asar/dist/main/webpack:/godmode/node_modules/electron-updater/out/AppUpdater.js:333:15)
(node:38495) UnhandledPromiseRejectionWarning: Error: Cannot find latest-linux.yml in the latest release artifacts (https://github.com/smol-ai/GodMode/releases/download/v1.0.0-beta.4/latest-linux.yml): HttpError: 404
"method: GET url: https://github.com/smol-ai/GodMode/releases/download/v1.0.0-beta.4/latest-linux.yml\n\nPlease double check that your authentication token is correct. Due to security reasons, actual status maybe not reported, but 404.\n"
Headers: {
"cache-control": "no-cache",
"content-encoding": "gzip",
"content-length": "29",
"content-security-policy": "default-src 'none'; base-uri 'self'; connect-src 'self'; form-action 'self'; img-src 'self' data:; script-src 'self'; style-src 'unsafe-inline'",
"content-type": "text/plain; charset=utf-8",
"date": "Tue, 29 Aug 2023 00:02:17 GMT",
"referrer-policy": "no-referrer-when-downgrade",
"server": "GitHub.com",
"strict-transport-security": "max-age=31536000; includeSubdomains; preload",
"vary": "X-PJAX, X-PJAX-Container, Turbo-Visit, Turbo-Frame, Accept-Encoding, Accept, X-Requested-With",
"x-content-type-options": "nosniff",
"x-frame-options": "deny",
"x-github-request-id": "A657:06B4:5A1D5B:7E778F:64ED35A6",
"x-xss-protection": "0"
}
at createHttpError (/tmp/.mount_GodModgKrZiG/resources/app.asar/dist/main/webpack:/godmode/node_modules/builder-util-runtime/out/httpExecutor.js:14:12)
at o.handleResponse (/tmp/.mount_GodModgKrZiG/resources/app.asar/dist/main/webpack:/godmode/node_modules/builder-util-runtime/out/httpExecutor.js:116:20)
at ClientRequest. (/tmp/.mount_GodModgKrZiG/resources/app.asar/dist/main/webpack:/godmode/node_modules/builder-util-runtime/out/httpExecutor.js:82:26)
at ClientRequest.emit (node:events:513:28)
at SimpleURLLoaderWrapper. (node:electron/js2c/browser_init:2:49930)
at SimpleURLLoaderWrapper.emit (node:events:513:28)
at newError (/tmp/.mount_GodModgKrZiG/resources/app.asar/dist/main/webpack:/godmode/node_modules/builder-util-runtime/out/index.js:47:19)
at rawData (/tmp/.mount_GodModgKrZiG/resources/app.asar/dist/main/webpack:/godmode/node_modules/electron-updater/out/providers/GitHubProvider.js:108:54)
at t.GitHubProvider.info [as getLatestVersion] (/tmp/.mount_GodModgKrZiG/resources/app.asar/dist/main/webpack:/godmode/node_modules/electron-updater/out/providers/GitHubProvider.js:120:17)
at h.result [as getUpdateInfoAndProvider] (/tmp/.mount_GodModgKrZiG/resources/app.asar/dist/main/webpack:/godmode/node_modules/electron-updater/out/AppUpdater.js:319:13)
at h.doCheckForUpdates (/tmp/.mount_GodModgKrZiG/resources/app.asar/dist/main/webpack:/godmode/node_modules/electron-updater/out/AppUpdater.js:333:15)
(Use godmode --trace-warnings ... to show where the warning was created)
(node:38495) 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)

no its not about downloading the file - when you boot up the app, it should check for the file on my hosted release instance, thats how it figures out if there has been an update. i've now uploaded the file.

still getting a white screen, getting the following error - true
14:43:41.456 › Checking for update
libva error: vaGetDriverNameByIndex() failed with unknown libva error, driver_name = (null)
IPC test: ping
false
14:43:46.504 › Update for version 1.0.0-beta.5 is not available (latest version: 1.0.0-beta.5, downgrade is disallowed).
14:43:46.506 › checkForUpdatesAndNotify called, downloadPromise is null

do you mind reading the errors a bit more and seeing what files it is looking for?

how do i do that? Any arguments I can pass to it?

not that I'm aware unfortunately. cc @seanoliver in case you can think of something

@zono50 we just pushed beta 6 today, lmk if still see same issues. still not sure how to debug.

Immediately updated, and immediately the same issue, weird thing tho version 3 works perfectly

i'd like to enable a argument flag to open up devtools for you so that you can see if there are any errors, however i dont really know how to do it. if you've got the inclination, could use some help adding that in so that we can debug this and all future userland issues that I cant replicate.
as for right now, you seem to be the only person reporting problems, so i'm wondering if it only affects linux users, or worse, linux arm64 users, in which case its probably that you are using the linux arm64 binary that i didnt continue uploading

I'm using the app image, but i'm open to any ideas

I may be wrong, but wondering if this is what's causing my white screen? libva error: vaGetDriverNameByIndex() failed with unknown libva error, driver_name = (null)

yeah i have no idea what that is. could you doublecheck on a friend’s linux setup, or maybe build locally from scratch and enable devtools?

Tried to build it locally, but to no avail. I'll use version 3 for now. If we can figure out what changed from version 3 to version 6 that could be causing this, that would be cool

ok. its more likely that beta 3 was missing the auto updating stuff which is why its not complaining.
i googled the error a bit and found this but it wasnt in english. it looks to be a linux specific issue. this https://www.reddit.com/r/archlinux/comments/nl8e0y/google_chrome_libva_error/ and https://askubuntu.com/questions/1339829/chromium-core-dumped-libva-error-vagetdrivernamebyindex-failed-with-unknown had more potential for things to try. we could perhaps disable gpu or disable compositing here for linux builds but i didnt do anything that funky in beta 3

circling back to note that it was my turn to get the white screen when encountering beta 8. i need to figure out how to make this auto updater work.

I have the same problem on 1.0.0-beta.8: white screen a few seconds after starting, with the following logs:
IPC test: ping
false
20:38:05.871 › Update for version 1.0.0-beta.8 is not available (latest version: 1.0.0-beta.8, downgrade is disallowed).
20:38:05.872 › checkForUpdatesAndNotify called, downloadPromise is null

Getting this error with GodMode-1.0.0-beta.10.AppImage for Linux. It crashes the whole app and you're left with a white screen.


Default sorting onload

Is there a default sorting on-load that is possible? Currently, it is configured to be the first column in descending order. Can one, say, change the order to be ascending as soon as the table loads?

This is solved. Use the query parameter in the controller to set the default sort.


Allow get_note to return notes that have been nullified.

Maybe return non-nullified notes by default and modify NoteGetterOptions to allow someone to specify that they want to get notes that have been nullified.
This came up when working on an airdrop contract with a feedback partner. They want to be able to prove the earliest interaction that someone had with a contract with a note. The proof of interaction is still valid for the airdrop, even if the corresponding note has been nullified.

As part of this work, it would be useful to refactor the test functions in the inclusion_proof_contract.

Shortterm, can add a todo for the nullifier inclusion proofs etc, as #4179 should make it significantly easier to check those when the headers are included properly.


Overhaul GraphQL test suite

Depends on #1618

Codecov Report

Merging #1750 (7518961) into main (ddf0ec9) will increase coverage by 4.23%.
The diff coverage is 70.73%.

@@            Coverage Diff             @@
##             main    #1750      +/-   ##
==========================================
+ Coverage   60.66%   64.89%   +4.23%
==========================================
  Files         102       98       -4
  Lines       12920    12181     -739
==========================================
+ Hits         7838     7905      +67
+ Misses       5082     4276     -806

Impacted Files
Coverage Δ

src/irmin-unix/cli.ml
20.91% <0.00%> (+0.11%)
:arrow_up:

src/irmin-unix/graphql.ml
0.00% <ø> (ø)

src/irmin-graphql/server.ml
51.42% <71.60%> (+3.87%)
:arrow_up:

src/irmin-pack/inode.ml
68.03% <0.00%> (-2.06%)
:arrow_down:

src/irmin-test/node.ml
93.18% <0.00%> (-1.82%)
:arrow_down:

src/irmin-unix/fs.ml
65.16% <0.00%> (-0.65%)
:arrow_down:

src/irmin-git/node.ml
74.72% <0.00%> (-0.56%)
:arrow_down:

bench/irmin-pack/trace_stat_summary_pp.ml
0.18% <0.00%> (-0.19%)
:arrow_down:

src/irmin/node.ml
50.61% <0.00%> (-0.05%)
:arrow_down:

... and 11 more

Continue to review full report at Codecov.

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


Support for LMDB flags

Added the support for the LMDB flags we discussed earlier.
Closes #15.
Laurent

Thank you for the contribution but isn’t a flush/sync function is necessary to force LMDB to persist data on disk if flags like NoSync/NoMetaSync is enabled?

Thanks for your review, below a description of the modifications for this new PR:

Added a force_sync method to the Env struct.
Reused the constant defined in lmdb_sys.
Removed the dependency to enumflag2 because it was not compatible with the reference to lmdb_sys constants (enumflags2 only supports literals). However I kept a C-style enum for clarity and soundness.

Thank you for this pull request. I will squash your commits.
Sorry that I didn't provide a contributing guide, next time would it be possible to rebase your commits instead of merging master into your branch? 😊


Fix March 19 on Spain

Hello,
I think this date should be removed from the Holidays because each year it randomly changes between the states.. for example this year only the following states celebrates it:
Galicia, País Vasco, Navarra, Castilla-La Mancha, Región de Murcia y Comunidad Valenciana.
It would be impossible to follow it each year as it is a "exchangable holiday" so each year only 6 states in the country gets to celebrate it. for example in the current file MADRID its on it. but MADRID does not celebrate it..
I just realized all this because it did not wake me up for work haha

Hi,
thank you for your interest and support. We have updated Spanish holidays according to this:
https://www.timeanddate.com/holidays/spain/san-jose
Thank you very much, cheers
dr-p


Change the order the different versions of API are registered

Register the API versions to the DefaultRESTMapper in the order of preferred versions first.
This makes kubectl sends requests to URLs in the format defined by the latest version of API, thus fixing #7512, #7260, and #7491

cc @bgrant0607

LGTM

Hi @smarterclayton, the shippable fails because the test-cmd.sh fails. These lines cause the problem:
if [[ "${version}" != "v1beta3" ]]; then
    kube::log::status "Testing kubectl(${version}:minions)"
    kube::test::get_object_assert minions "{{range.items}}{{$id_field}}:{{end}}" '<IP_ADDRESS>:'
...

It's called when ${version} = "". I think the nil version means the default version, which is now v1beta3. So I guess we should modify the script to skip the minions test when version is nil. Am I right? Thanks.

Yes. You can change that if condition to
  if [ "${version}" = "v1beta1" ] || [ "${version}" = "v1beta2" ]; then

Hi @smarterclayton , just out of curious, I have an irrelevant question about the test-cmd.sh. What does this line do?
[ "$(kubectl get minions -t $'{{ .apiVersion }}' "${kube_flags[@]}")" == "v1beta3" ]

Why is there a $ sign before the template "$'{{ .apiVersion }}'"? And where is the test result used? I searched for "$?" but there is none. I know very little about bash..

Correct, now that v1beta3 is the default the empty API version will not have minions as the default name.

On May 1, 2015, at 5:53 PM, Chao Xu<EMAIL_ADDRESS>wrote:
Hi @smarterclayton, the shippable fails because the test-cmd.sh fails. These lines cause the problem:
if [[ "${version}" != "v1beta3" ]]; then
kube::log::status "Testing kubectl(${version}:minions)"
kube::test::get_object_assert minions "{{range.items}}{{$id_field}}:{{end}}" '<IP_ADDRESS>:'
...
It's called when ${version} = "". I think the nil version means the default version, which is now v1beta3. So I guess we should modify the script to skip the minions test when version is nil. Am I right? Thanks.
—
Reply to this email directly or view it on GitHub.

That's a great question I believe that might be the result of a copy and paste bug - I don't think that $ should be there.

On May 1, 2015, at 5:59 PM, Chao Xu<EMAIL_ADDRESS>wrote:
Hi @smarterclayton , just out of curious, I have an irrelevant question about the test-cmd.sh. What does this line do?
[ "$(kubectl get minions -t $'{{ .apiVersion }}' "${kube_flags[@]}")" == "v1beta3" ]
Why is there a $ sign before the template "$'{{ .apiVersion }}'"? And where is the test result used? I searched for "$?" but there is none. I know very little about bash..
—
Reply to this email directly or view it on GitHub.

Hi @smarterclayton, the PR passes the shippable, could you merge it? Thanks.

Rerunning travis

LGTM

resolve #7260


microsoft-graph-docs-contrib

[Enter feedback here]

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

ID: e64b996c-9981-08f0-055a-811e58fc6018
Version Independent ID: 527fca4c-6331-febc-12e9-e5bd02c70fa2
Content: Get started with Microsoft Graph Toolkit - Microsoft Graph
Content Source: concepts/toolkit/get-started/overview.md
Product: non-product-specific
Technology: microsoft-graph
GitHub Login: @sebastienlevert
Microsoft Alias: MSGraphDocsVteam

@yueyuecompany23 can you help me understand the feedback? Or maybe this wasn't meant to be an issue? Thanks!

Closing as no feedback from author.


Updating issue8test.js

2nd pull request without merging

Code is clean and has helpful comments. Casper file is good.

Those two new HTML files make all the tests pass for me.
:+1: for merging

A couple of things on the integration test:

When you have followed a link from courses.html to another page, instead of using a link to go back to courses.html, you can us the .back() method.
You'll want to test all of the links in the header.
After you followed a link to another page, you were testing the title of the page but you used this.getTitle(). This test will never fail. When testing the page title, you should provide a string for the expected title.


This is an experiment to see if using the bazel-provided python + fusesoc caused the big step function in bitstream build times.

This doesn't appear to be the cause.


Error: Cannot find a record for UID all_lower_case_letters. Cannot download filename

There is an issue with parsing. The given uid is converted to lower case and hence can not get a file.
[ TEST4 2022-04-07 14:35 ]
username@host:~/git/eval-ksm (main*$%=) $ ksm secret download --uid=pluIUSAmO3r7oyAPYKMDOg --name=java11_db-client.jks --file-output=client.jks
Error: Cannot find a record for UID pluiusamo3r7oyapykmdog. Cannot download java11_db-client.jks

[ TEST4 2022-04-07 14:39 ]
username@host:~/git/eval-ksm (main*$%=) ksm secret get pluiusamo3r7oyapykmdog && echo success
success

[ TEST4 2022-04-07 14:40 ]
username@host:~/git/eval-ksm (main*$%=) $ ksm secret get pluIUSAmO3r7oyAPYKMDOg

 Record: pluIUSAmO3r7oyAPYKMDOg
 Title: test-mysql-credentials
 Record Type: databaseClusterCredentials

 Field                   Value
 ----------------------- ------------------------------------------------------------------------------
 Type                    mysql
 Hostname or IP Address  [{"port": "32100", "hostName": "db11"}]
 Hostname or IP Address  [{"port": "32100", "hostName": "db12"}]
 Hostname or IP Address  [{"port": "32100", "hostName": "db13"}]
 Login                   dbuser
 Password                ****
 trustStore              ["4JjLTEq6KO6Qowvwf8ZyNA", "xrE4oKfWqBd6JzTGpxGN_A", "q-q9jRPvshsHfNo2e6Ib4Q"]

 File Name              Type Size
 ---------------------- ---- ----
 client.jks                  3551
 java11_db-trusted.jks       1143
 java11_db-client.jks        3551

As you can see the lower case query also returns success ( rc=0 ), which should be another bug.
You should also note that it is possible to upload multiple files with the same name and even the same content. Does this make sense? How should I download a specific file with same filename but different content? For what are the uid-like id's in the array of the tustStore field?

There is a bug processing params that use a = to separate the param name and the value. If you remove the =, it will work fine.

Funny I just started to use that notation with equals sign because there is another bug which needs exactly that https://github.com/Keeper-Security/Commander/issues/713

As for trustStore, that looks to be some kind of custom record type. I have no idea what that is. You could always try to get it and seeing what you get.

Yes this is customRecord but is the same with any other record with file attachements. It was just as example while evaluating.


v5.4 Changelog

@amyblais Did we remove a lot of the UX/UI improvements from the changelog? I think they're still worth keeping, just re-organized like Katie mentioned.


iOS 18 runtime can not be downloaded

➜ xcodes runtimes download "iOS 18.0"
Error: Runtime com.apple.dmg.iPhoneSimulatorSDK18_0 is missing source url. Downloading of iOS 18 runtimes are not supported. Please install manually see https://developer.apple.com/documentation/xcode/installing-additional-simulator-runtime

When I click on the iOS 18 install button:

@mttlmnt and @wangwanjie: release notes states "iOS 18 runtimes are not supported downloading via the xcodes runtimes command. Apple has included a new way to download that is not supported."
you have to manually down load the stimulators you need/want

@MattKiazyk (moving to here since I'm working on this issue and don't want to spam the other one).  I started looking at this and I found the code to run the xcodebuild install process (https://github.com/XcodesOrg/XcodesApp/pull/622/files#diff-fc53334fc7d50be6adfb7a819b958d83855d0d4e8e5b4757f103bbaf53536de6R211).  From what I can tell this looks like it only downloads it.  When I had done this locally I had to use xcodebuild -importPlatform "$file" to actually install the DMG that xcodebuild found.

@MattKiazyk okay, sorry for the extra updates.  I've got a WIP approach going here that seems to be downloading and installing the runtimes.  I think it's still missing the version check stuff, but I'd love to get some feedback on a couple things when you have time:

Does the overall approach here make sense to you? (e.g. you can only install, so just let the runtime download command fail and redirect to runtime install
What would be the best way to check that the correct selected xcode version is being used for the new download style?

I think for now that is fine. I can see in a future where the download command would add onto the xcodebuild command to save the package.
See https://github.com/XcodesOrg/XcodesApp/blob/main/Xcodes/Backend/AppState%2BRuntimes.swift#L61 where I specifically am checking for Xcode 16.1. Think we can do something similar.

Thanks for the work!

Awesome, thanks so much for the response!  I saw this bit for checking, but wasn't sure what this repo's version of self.allXcodes would be.  Is there an easy way to grab that?
https://github.com/XcodesOrg/XcodesApp/blob/dd9a348298421b14f513f55ae281d940c943fd5c/Xcodes/Backend/AppState%2BRuntimes.swift#L52

You'll have to probably call Current.shell.xcodeSelectPrintPath() to get the version and parse it.

Or you might be able to check the result from the new xcodebuild command and if it errors because the option doesn't exist you can assume it's not supported.

@MattKiazyk , I think I've gotten it to a point where I felt comfortable opening up a full PR, please let me know if there's anything you'd like changed or adjusted.

Realized I forgot to actually include the PR here https://github.com/XcodesOrg/xcodes/pull/391 cc @MattKiazyk .


fix(go/adbc/driver/snowflake): Made GetObjects case insensitive

Description:
GetObjects API was inconsistent case sensitivity for patterns. getObjectsDbSchemas driver implementation used LIKE whereas getObjectsTables used ILIKE
Solution:
Based on discussion here: https://github.com/apache/arrow-adbc/issues/1314 changed GetObjects API to use ILIKE throughout instead of LIKE
Testing:
Added tests for lowercase, uppercase and _ wildcard
Fixes #1314.

Refactoring includes being able to match the exact catalog, schema, and table names when wild cards are involved in the name indentifiers of the GetObjects patterns
For example: A pattern name like FOO_BAR will match against FOO1BAR and FOOABAR. Similarly FOO%BAR will match with foohellobar, FOObyeBAR etc.. Therefore it is essential to filter out the returned matches with the expected name identifiers.
Note: Similar checks would have to be added by authors using the Driver when calling the GetObjects API.


Support the IntelliJ Platform.

https://www.jetbrains.org/intellij/sdk/docs/basics/getting_started/plugin_compatibility.html

Thank you I appreciate it :)


Error: installed gemini is not compatible with GUI

Error: installed gemini is not compatible with GUI
gemini version should be in range  >= 0.8.x < 0.13.0
Error: No compatible version
I installed "gemini" and "gemini-gui" step-by-step by https://ru.bem.info/tools/testing/gemini/

Fixed in https://github.com/gemini-testing/gemini-gui/pull/17
<EMAIL_ADDRESS>


VEGA-805 Add team switcher

Codecov Report

Merging #31 (aa8a826) into main (43b7e4f) will decrease coverage by 1.02%.
The diff coverage is 77.50%.

@@            Coverage Diff             @@
##             main      #31      +/-   ##
==========================================
- Coverage   88.26%   87.23%   -1.03%
==========================================
  Files          21       22       +1
  Lines         443      478      +35
==========================================
+ Hits          391      417      +26
- Misses         28       33       +5
- Partials       24       28       +4

Impacted Files
Coverage Δ

internal/server/central_cases.go
82.75% <0.00%> (-2.96%)
:arrow_down:

internal/sirius/teams.go
63.63% <63.63%> (ø)

internal/server/server.go
93.50% <100.00%> (ø)

internal/server/team_work_in_progress.go
100.00% <100.00%> (ø)

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 43b7e4f...aa8a826. Read the comment docs.


get generated keys from queryRunner.insertBatch

This class will be useful to get generated keys from the queryRunner.insertBatch operation.
Eg:-
   ResultSetHandler<List<Long>> rsh = new ScalarListHandler<Long>();
   Object[][] params = {{"col1row1", "col2row1"}, {"col1row2", "col2row2"}};
   List<Long> ids = queryRunner.insertBatch("INSERT INTO testtable (col1, col2) VALUES (?,?)", rsh, params);

Hi @visruth, what using the ColumnListHandler class?  These seem the same to me.  Can you highlight what I'm missing?
ColumnListHandler<Long> rsh = new ColumnListHandler<Long>();
...


Update zigbang.py

Unicode 코덱 처리 문제로, 처리할 수 없는 문자열을 try, catch문으로 예외처리.

@qkr0990: thanks for your PR. This is really nice solution for Windows support.

But we changed our base development environment from console to ipython notebook. This fuzzy-characters doesn't have problem any more. Thanks :)


65 add spell cards abm view

Closes #65.
It requires #64 to be merged first.

Codecov Report

Merging #70 (b8501ba) into release-candidate (ffc8e95) will not change coverage.
The diff coverage is 100.00%.

@@                 Coverage Diff                 @@
##           release-candidate       #70   +/-   ##
===================================================
  Coverage             100.00%   100.00%
===================================================
  Files                     53        55    +2
  Lines                   1285      1350   +65
===================================================
+ Hits                    1285      1350   +65

Impacted Files
Coverage Δ

...tests/test_booster_pack_based_management_system.py
100.00% <100.00%> (ø)

...GiOh/booster_packs/tests/test_booster_pack_card.py
100.00% <100.00%> (ø)

.../test_booster_pack_card_based_management_system.py
100.00% <100.00%> (ø)

...YugiAcademy/YuGiOh/cards/card_management_system.py
100.00% <100.00%> (ø)

source/YugiAcademy/YuGiOh/cards/spell_card.py
100.00% <100.00%> (ø)

.../YugiAcademy/YuGiOh/cards/tests/test_spell_card.py
100.00% <100.00%> (ø)

...s/tests/test_spell_card_based_management_system.py
100.00% <100.00%> (ø)

...iAcademy/YuGiOh/migrations/0003_spellcard_image.py
100.00% <100.00%> (ø)

...emy/YuGiOh/migrations/0004_alter_spellcard_type.py
100.00% <100.00%> (ø)

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


About Evaluation

Sorry, just want to know how to evaluate training? I don't know how to get the last four pictures of the evaluation.

Thank you for the comment. To get the pictures, refer to  https://chainercv.readthedocs.io/en/stable/tutorial/detection.html. I'm sorry, but this project is old, so the actual code is not available. I recommend using the current mainstream torchvision object detection method.

Thank you very much!


Use valid WAT identifiers for memories.

The motivation is that we'd like to use the disassembler in Chromium DevTools (similar to what FF DevTools uses), but make sure that what we display is valid WAT that can be fed back into wat2wasm for example.

Thank you.

Thanks!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Drop kubectl exec in favor of port-forward for envoy commands

Related: #23543
We cannot use proxy as we only listen on localhost; port-forward
bypasses this issue

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
3b193a87557ecade79c03af01172eae8a365b8a0
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-local-tests_istio
19db2d8cb814ba9da00e1cf9f096b8792914e035
link
/test integ-local-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Clear labels should remove checkmarks in popup

Description
On the "New issue" or "New Pull Request" page in the "Labels" popup, the "Clear labels" function does not remove the checkmarks in the label list.
This can easily be reproduced:

Go to the "New Issue" page
Open the "Labels" popup
Add one or more labels (checkmark appears)
[Optionally close the labels popup to "commit" the selection and reopen]
Click "Clear labels" (the popup closes)
Open the "Labels" popup again
Find the checkmarks still present

The expected behavior would be, that the checkmarks also get removed when "Clear labels" is clicked.
Screenshots

Gitea Version
1.19.0+dev-223-gea5a752ee
Can you reproduce the bug on the Gitea demo site?
Yes
Operating System
Pop!_OS 22.04 LTS 6.0.6-76060006-generic
Browser Version
Firefox 108.0 (64-bit)

Can't reproduce it on my local dev setup. After every selection the page is reloaded which fixes your issue. I have the same reload behaviour on the Gitea demo site but there I can't use "Clear labels" 🤷♂

Can't reproduce it on my local dev setup. After every selection the page is reloaded which fixes your issue. I have the same reload behaviour on the Gitea demo site but there I can't use "Clear labels" man_shrugging

I only have the try.gitea.org instance to test. But I still can reproduce it with Firefox 108.0 and also have the same behavior with Chromium Version 108.0.5359.124 (Official Build) (64-bit) (Flatpak install) and Vivaldi 5.6.2867.50 (Stable channel) stable (64-bit) tested just now on try.gitea.org running Gitea version 1.19.0+dev-226-g71ca3067b.
I also checked for JS errors or something, but none are printed out in the console.
I do not have any reloads happening, and I also do not have the case where it does not work at all.

I have the same Firefox version and get an error:

Don't know what's the problem.

The problem is highly liked caused by a server/network problem.
At the moment, the "label" works like this:

Choose labels (including Clear)
Hide the dropdown
Update the labels to server (one by one)
Reload page

If any request in step 3 fails, the page won't be reloaded, then you see the stale page.
If you could help to figure out why step 3 fails, it would be very helpful.

Tested it just now again on https://try.gitea.io.
Still almost the same behavior for me. The only difference from my initial report is that the popup stays open during step 5.
No requests are made at all when I change the labels. Never had the Promise issue @KN4CK3R describes.
Gitea Version
1.20.0+dev-48-g04347eb81
Operating System
Pop!_OS 22.04 LTS 6.1.11-76060111-generic
Browser Version
Firefox 110.0 (64-bit)

That's really  strange. I just did a new test on try.gitea.io , I can clear lables and add labels. Firefox 110.0.1 (64-bit) (it couldn't be related to browser IMO)
https://try.gitea.io/wxiaoguang/test/issues/20

Oh, sorry,  I see, you mean "New Issue"? That page seems to be broken for long time ....
I could take a look  if I get time. Thank you for your report.

The fix: https://github.com/go-gitea/gitea/pull/23224
ps: I didn't change the "clear" behavior: IMO in history, it is done by purpose to keep the list shown after clicking the "clear", just like clicking other items.


overmask kinda sucks

like the concept is really good and works GREAT in theory if i manually draw a mask over the region, but simply programmatically expanding the mask region just moves the seam instead of hiding it

this is really frustrating >:(

i probably shouldn't use the issue tracker as a sporadic blog

potentially fixed in https://github.com/zero01101/openOutpaint/commit/8002772ee6aa4b2f5b544af82cb6d545cf81368f


As a user, I get confused that I have to scan the QR codes

Remove QR codes

Yagni pinciple. If we need it again later, then we can implement it then


When inline, image alignment buttons don't work

Hey again! It sounds like this extension doesn't support inline mode yet, which makes sense given how new it is. I just wanted to flag that the alignment buttons also don't work when inline is enabled on the extension.
I ended up forking the extension to work around this, using the text-align extension's commands for alignment instead, as that works with the paragraph node when the image is inserted inline.
To do this, I ended up swapping out the position controller event handlers to be:
        $leftController.addEventListener("click", () => {
          editor.chain().focus().setTextAlign("left").run();
          dispatchNodeView();
        });
        $centerController.addEventListener("click", () => {
          editor.chain().focus().setTextAlign("center").run();
          dispatchNodeView();
        });
        $rightController.addEventListener("click", () => {
          editor.chain().focus().setTextAlign("right").run();
          dispatchNodeView();
        });

An easier change to support this, if you're interested, would be to expose the paintPositionContoller method as an extension option that can be overridden. If that is a friendly change to you, I'd be happy to make a contribution!

Hi @mason-os,
First, thank you for your interest in my extension! I've reviewed your issues and tested your code. Your solution is promising, but for this inline case, we need to consider potential side effects.
We are planning to refactor the entire logic to better support this inline feature. I apologize for any inconvenience and let you know when the feature is complete.

How exactly can i overwrite the default behavior? i have an vanilla javascript code base and all i do is adding                                              ImageResize to the tiptap extension array right now. I couldn't figure out how you used your own events for the buttons.


Screenshot is removing borders and aligning text to bottom

Please make sure you are testing with the latest release of html2canvas.
Old versions are not supported and issues reported for them will be closed.
Please follow the general troubleshooting steps first:

[x] You are using the latest version
[x] You are testing using the non-minified version of html2canvas and checked any potential issues reported in the console

For some reason text is aligning to the bottom? Also borders around divs aren't showing

@bugkitio I'm also facing same issue. Is there any update ?

same issue


Custom metrics are not registered to the same registry

I attempt to setup custom metrics in combination with express-prometheus-middleware. My custom metric is counting, but is not served by the metrics endpoint, and from what I've understood I end up with two prometheus registers.
I made a minimal runnable example which can be seen at https://github.com/haakondr/prometheus-express-custom-metrics
My server is defined as follows:
const express = require('express');
const promMid = require('express-prometheus-middleware');
const client = require('prom-client');

const startServer = () => {
  const app = express();
  const port = 8080;

  const counter = new client.Counter({
    name: 'hellocounter',
    help: 'Counting hellos',
  });

  app.use(promMid({
    metricsPath: '/metrics',
    collectDefaultMetrics: true,
  }));

  app.get('/', (req, res) => {
    counter.inc(1);
    res.send(client.register.metrics());
  });

  return app.listen(port, () => {
    console.log(`Server started`);
  });

};

module.exports = {
  startServer: startServer,
};

With this setup, the default metrics from express-prometheus-middleware are served on /metrics, but not my hellocounter. GET / responds with my hellocounter, so it is being incremented.
When I debug in https://github.com/joao-fontenele/express-prometheus-middleware/blob/master/src/index.js#L89 I don't see my hellocounter, so I suspect that the problem is that both my custom metric and express-prometheus-middleware metrics are using global prometheus registers, but these are module scoped. When I have explicitly added prom-client as a dependency, I end up with two modules of prom-client:
npm ls | grep prom-client
│ ├─┬<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>
So, my question is as follows:

what is the recommended approach for adding custom metrics?

One possible solution is to allow passing in prometheus registry as an option, and pass this along to Prometheus.collectDefaultMetrics in  https://github.com/joao-fontenele/express-prometheus-middleware/blob/master/src/index.js#L63.
Eager to hear if you have a solution to this. If you think my suggestion is a viable solution, I can submit a PR.

Hi, I also got this problem some time ago - what helped me was to force the same dependency in package.json:
"prom-client": "^11.5.3",

and then run
npm dedupe

after npm install ... I think also yarn would remove the duplicate dependencies automatically, then you have just single prom-client and it should work.
This is a workaround - I agree that a PR with passing registry would be better

hello @haakondr,
It seems the latest commit of #23 fixes the issue, could you confirm? I've tried to clone and install the update, and it seems to work.
Just had to modify https://github.com/haakondr/prometheus-express-custom-metrics/blob/master/tests/test.js#L35 to

@joao-fontenele I can confirm that #23 solves the problem. Thanks!

closed by #23


Update Zeroize Crate

Description:
The Zeroize crate current version creates dependency conflicts when used with crates such as ethers-rs. Updating the version to resolve those conflicts.

:link: zboto Link

@AmeanAsad The PR looks good. Did you see my comment ?

Hey @rllola, are you okay if you merge this?

Merged @AmeanAsad ! Thank you. I am doing a release asap.


Edit modifications in second dock

https://github.com/conveyal/scenario-editor/issues/128
Was playing around with trying to get the docks to still be resizable and resize the map on edit. I couldn't figure out a good way to do this via react. Right now they are fixed sizes but still using react-dock and the map isn't resizing when the second dock is activated. If there is a good way to get the dockWidth from application.js to the children then I think it would be relatively easy enough to figure out.
Also fixed some typos and little quirks I found as I was working on this.

Is it possible to accomplish this without an additional action? I originally thought the implementation would use the "active" modification action and status to show the dock. Collapsing  the dock would set the active modification to null.

I think the standard caret / chevron style is facing right for closed and down for open. Let's switch to that style. I know I may have messed this up in the ticket and elsewhere in the application but after using it in this particular change it's more obvious to me.

@trevorgerhardt Yeah, I should have asked about this. I was under the impression that one could make  edits via the map and might want to close the extra panel. But of course I'm not as familiar with the functionality. Let me know if I should go ahead and update that.

Yup! Please do!

@trevorgerhardt Ok, updated! I noticed in master that deleting a modification throws an error. But, I was happy to see a bug I thought was in my branch went away when I merged in master. Whew!
I also updated with the resizing functionality for the docks and map.

@catherinesnow I should really be using a development branch :)
Looks great! Just merged. Appreciate the work. I'll be following up more later this week.


Question 9 need ? marks after each of the 2 questions

Should have a ? after HIPPA etc. And at the end.

No.


Add ui-vue package.

Add Ton Connect Vue Kit.

Hi, thanks for your suggestion.
Unfortunately, we do not plan to create such a package ourselves in the near future. However, we will gladly include community developments in the SDK.

I just implemented this UI kit in Vue looking on React implementation: https://github.com/d0rich/esprit/tree/d-frontend/apps/d.d0rich.me/src/features/tonconnect
Here you can find not only tonconnect ui, but some other TON libs.
And I wanted to keep it in Pinia storage. More reliable way is to make it simple composable.

I just implemented this UI kit in Vue looking on React implementation: https://github.com/d0rich/esprit/tree/d-frontend/apps/d.d0rich.me/src/features/tonconnect
Here you can find not only tonconnect ui, but some other TON libs.
And I wanted to keep it in Pinia storage. More reliable way is to make it simple composable.

Your link is dead . but i've found this repo (not tested) https://github.com/TownSquareXYZ/tonconnect-ui-vue/tree/main

I just implemented this UI kit in Vue looking on React implementation: https://github.com/d0rich/esprit/tree/d-frontend/apps/d.d0rich.me/src/features/tonconnect
Here you can find not only tonconnect ui, but some other TON libs.
And I wanted to keep it in Pinia storage. More reliable way is to make it simple composable.

Your link is dead . but i've found this repo (not tested) https://github.com/TownSquareXYZ/tonconnect-ui-vue/tree/main

I am trying to npm install this but it would not work.

yes me too, it doesn't work with vue 3/Nuxt/TypeScript...
So i've implemented the cdn script :
onMounted(() => {     console.log('Mounted')     if (props.project.isConnected==false) {   useHead({       script: [           {               src: 'https://unpkg.com/<EMAIL_ADDRESS>              type: 'text/javascript',               onload(el) {                 //console.log('tonconnect-ui.min.js LOADED')                 initializeTonConnect();                               },           },       ],   }, { mode: 'client' }); } });
and
`const  initializeTonConnect = async ()  => {
tonConnectUI.value= new TON_CONNECT_UI.TonConnectUI({
manifestUrl: 'https://xxx/tonconnect-manifest.json',
buttonRootId: 'ton-connect'
});
const unsubscribeModal = await tonConnectUI.value.onModalStateChange(
(state) => {
// update state/reactive variables to show updates in the ui
// state.status will be 'opened' or 'closed'
// if state.status is 'closed', you can check state.closeReason to find out the reason
if( state.status=="opened"){
getWallet()
}
}
);
const unsubscribe = tonConnectUI.value.onStatusChange(
(walletAndwalletInfo) => {
//console.log('onStatusChange',walletAndwalletInfo)
if(walletAndwalletInfo.account){
props.project.isConnected=true;
tonConnexionStatus.value = Boolean(walletAndwalletInfo.account.address!='')
changeAddress(walletAndwalletInfo);
   }
}

);`

hey guys
i use this, works for first view page in nuxt 3 but after page change and back to page get this error
Uncaught (in promise) DOMException: Failed to execute 'define' on 'CustomElementRegistry': the name "tc-root" has already been used with this registry
please guide me

Ok, i think you have two buttonRootId no ?

<template>

</template>

No only have one div

https://github.com/ton-connect/sdk/issues/218
I fully explained in the above link


Fix hero section

I can fix this issue; I have 1.5 years of experience with JavaScript and I have worked on several projects in my college. So, I can fix this hero section problem. Please assign this project to me .

Assigning


Implement filter by keyword to work with Enter key

User Story
As a user, I want to see filter results when executing search with the Enter key.
Acceptance Criteria
Given a user has entered keyword criteria
When the user selects the keyboard Enter key
Then the system returns filter results.

Test Case: TC259

Test Passed


Want to correct typos in Gym Website and Add meaningful text at some places.

Describe the bug
There are typos in two headings in Gym Website and dummy text under health, cardio, body, fitness sub-headings. I want top make it more meaningful.
Screenshots
Desktop (please complete the following information):

OS: MacOS
Browser: Firefox
Version: 106.0

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.
Steps Followed✅
Check the box with [x] If you followed all the steps.

[x ] Describe the bug
[ ] To Reproduce
[ ] Expected behavior
[ ] Screenshots
[x] Additional context

Code of Conduct✅

[x] I agree to follow this project's Code of Conduct.
[x] I checked the current issues for duplicate problems.
[x] I have gone through project README.

hey can i work on this issue of correcting typos in Gym Website and Add meaningful text at some places.


Buttons don't show up until page is reloaded.

Triggering on DOMContentLoaded is insufficient.

Fixed by https://github.com/dpoeschl/EmailGitHubIssues/commit/a20cd0959c32926f9da46518eddc24f1cd581f05


Change MainWindow.cs to MauiProgram.cs

Use the correct filename that contains the static CreateMauiApp method.

Internal previews

📄 File
🔗 Preview link

aspnetcore/blazor/hybrid/security/index.md
aspnetcore/blazor/hybrid/security/index

Thanks @thomasclaudiushuber ... This is probably a recent change that I didn't hear about. Do you know when they made the change?

Hi @guardrex, in my opinion it's not a recent change, it has always been that way for .NET MAUI apps.
There was even never a MainWindow.cs file in MAUI. MAUI contains a MainPage.cs, but not a MainWindow.cs. But the code with the static CreateMauiProgram method was always in a MauiProgram.cs file, this is true for the .NET 6 and for the .NET 7 project template. I think MainWindow.cs was a copy/paste issue from something about WPF, because there that file exists.

Yes, I think you're correct. The rest of the docs use the correct filename. This is the only doc that seems to have a grimlin 😈 in it. It came in on this PR ... https://github.com/dotnet/AspNetCore.Docs/pull/25791 ... and that was derived from PU notes. I think that when I was writing it up I just flat-out had the wrong filename in my head 🙈. It was reviewed, but it wasn't caught by Jaiver at the time. That's how the sneaky ones 😈😈😈 really do well ... I put them in AND they escape review.
Stand-by for a sec ... I just need to make sure that they're all out of this before we merge it.

Thank you @guardrex, looks good now. :)👍 ✨


Fix aabb intersects ray normal being incorrectly always negative

Also, I am pretty sure that the intersection point result is also incorrect, or its in other coords that I don't know about, but the results don't even seem consistent with the other ray intersection functions... so the whole functions may need a proper review and/or rewrite.

There's already 2 PRs for this: #83138 and #86755 .

Superseded by #86755. Thanks for your contribution!


Add define gl_InstanceID of gl_InstanceIndex for webgpu

Fixes: https://github.com/playcanvas/engine/issues/7026
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

thanks @AlexAPPi


Closes #206 Clean packages

Thank you for your Pull Request! admiraldev have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the datacutr codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the devel branch until you have checked off each task.

[x] Place Closes #<insert_issue_number> into the beginning of your Pull Request Title (Use Edit button in top-right if you need to update)
[x] Code is formatted according to the tidyverse style guide. Run styler::style_file() to style R and Rmd files
[x] Updated relevant unit tests or have written new unit tests - See Unit Test Guide
[x] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[x] Update to all relevant roxygen headers and examples
[x] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[x] Address any updates needed for vignettes and/or templates
[x] Update NEWS.md if the changes pertain to a user-facing function (i.e. it has an @export tag) or documentation aimed at users (rather than developers)
[x] Build datacutr site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the "Reference" page.
[x] Address or fix all lintr warnings and errors - lintr::lint_package()
[x] Run R CMD check locally and address all errors and warnings - devtools::check()
[x] Link the issue in the Development Section on the right hand side.
[x] Address all merge conflicts and resolve appropriately
[x] Pat yourself on the back for a job well done! Much love to your accomplishment!

Deleting merge/branch and will handle in seperate branch


Day selection lost on month/year change

Hi, I noticed with some user tests it's annoying behaviour: when I click on a day then I change month or year I loose the day choice, it's possible keep the day selected?
I have another issue too:  The standard behaviour should be close the datepicker on day selection, am I right? Why I have to click on close button?
PS: I use materialize pickadate https://github.com/dogfalo/materialize/

Hi, this might be a little late and you might have come up with a solution, ut this might be useful to anyone else that runs into the same problem.
`var dp = $('.datepicker').pickadate({
min: [1940,1,1],
max: true,
selectMonths: true,
selectYears: 65,
onSet: function(e){
if(e.highlight != null){
var picker = dp.pickadate('picker');
picker.set('select', new Date(Number(e.highlight[0]), Number(e.highlight[1]), Number(e.highlight[2])));
}
				}
			  });`

it's late for that project, but I can totally reuse this code for the next one :+1:

Hi, I am using the amsul.ca/pickadate.js and have a similar problem where the year selection is lost if the user does not select the 'DAY' last. Can anyone advise of a fix? I thought the above might have helped but sadly not. Any advice would be appreciated

I tried using this fix but it breaks the selection via prev/next month selectors. I think that is due to #948.
As the highlight object seems to have changed I adjusted the callback accordingly:
if(e.highlight != null){
  picker.set('select', [e.highlight.year, e.highlight.month, e.highlight.date]);
}

Thank you @fredula the code working excelent!

@burka so right now this fix doesn't work anymore because the highlight object holds the values of the previous selection? Or how are you using this @romanlezama ?

Hello @fabyeah this is my example code https://jsfiddle.net/romanlezama/2gotaobp/ for me is working very good!!

@romanlezama thanks for the fiddle. Selecting a year or month from the dropdown works, but the arrow buttons for previous and next month don't work properly anymore. Also when you have selected the 31st and choose a month with only 30 days, it will jump to the 1st of the next month.

@fabyeah I don't remember exactly, Finally the fix was incomplete for me. Something about previos/next or pre-selection or .... . I don't remember exactly, I stopped trying after some attempts.

Still an issue. Fixed with:
var $input = jQuery( "#datepicker" ).pickadate({
	selectYears: true,
	selectMonths: true,
	clear: '',
	format: "d mmmm yyyy",
	onSet: function(e){
		var picker = $input.pickadate('picker');
		if(e.highlight!=null){
			picker.set('select', new Date(e.highlight.year+" "+e.highlight.month+" "+e.highlight.date));
		}
	},
	max: new Date()
});

@tohirsolomons  can you create a PR so we can merge this fix and test it further?


sed -i does not work on macOS

When using macOS with bash (domain-setup.sh), I receive an error: sed: -i may not be used with stdin.
Though there is no error when using Cloud Shell (Linux).

Also, env.mk is created/edited in both domain-steup.sh and bootstrap.sh scripts.

For macOS, use the gnu version of sed: brew install gnu-sed and then follow instructions for putting gnubin earlier in your path.


pkgs.openscad: cannot export to PDF

Issue description
OpenSCAD seems to be built without support for PDF exports.
(This is a very useful feature to print projections on paper to check the
dimensions of a design for example).
Version: OpenSCAD version 2021.01
On nixpkgs: master (2022-09-18)
Steps to reproduce

Open OpenSCAD
Go to File -> Export -> Export as PDF…
Save to some file
Observe the following error in the OpenSCAD console:

ERROR: Export to PDF format was not enabled when building the application.

CC package maintainers: @bjornfor @7c6f434c @gebner

Any idea what dependency they need for PDF?
(image file export has been enough for me so I never looked)

I think cairo. (I was looking into this earlier today.)

Yes, cairo was missing: https://github.com/NixOS/nixpkgs/pull/192916


feat: able to customize length limit of subject and warn when exceed

I think it's better to remind user when their subject message exceed limit and  let them rewrite their subject.

It makes totally sense to me

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Packages shipped with ESM shouldn't be prebundled

For example I'm using this package, which already support ESM module, but Vite is still going to prebundle it unless you put it in the optimizeDeps.exclude explicitly.

This is expected, Vite needs to pre-bundle dependencies by default to avoid bottlenecks when the browser requests too many files. It doesn't matter if it is ESM. If you would like to avoid pre-bundling a dependency, as you said, there is an option available.


Action for publishing on tag release

Documentation for the new release process is included.

Very nice! Especially the detailed documentation


.container within navbar undoes ability to use flexbox positioning

A stated goal of the Alpha6 .navbar is the ability to use flexbox to position items within.
This works fine for elements that are immediate children of .navbar (the flex parent). However, when a .container is placed within .navbar (a common pattern), flexbox positioning is no longer an option because .container is not display: flex.
One could manually add d-flex but that seems unintuitive when display: flex is presented as the assumed display context for navbars.

+1 I am seeing the same issue.
The left and right margins that are set to the container class are having some strange interaction with the navbar.

Mac OS 10.12.2
Chrome

Here is the issue in action:

If you go into the inspector and disable margin-left: auto and margin-right: auto on the .container class then things go back to normal on mobile, but this isn't a fix because it messes up the alignment of the container on larger screens where the container width is less than the screen width.
Here's what it looks like on mobile with the margins disabled though:

It looks like if you disable the auto margins at <576px then the problem is remedied, but I'm not sure if there would be other consequences of doing so.

@davidtmiller I believe you're experiencing this issue, which also seems related to container being the only flex item within the navbar flex parent.

@henwen Yep that's the one...thanks!

Fixes in #21605

@davidtmiller, I think this is a different issue. What @henwen  report is that .navbar is display: flex and allow to position its content with flex, while .container is not display: flex.
Is it your actual issue @henwen ?
Although,  a .container within a .navbar is display: flex, but only at a breakpoint in which the navbar is expanded (i.e. no toggle button and all item visible). See _navbar.scss#L155.
Indeed, a .container within a .navbar is not display: flex when the menu is collapsible (nav-item non visible in the top bar, toggle button visible).
Is what you trying to achieve is to position with flex the content of the container when the navbar is collapsible ?

@vanduynslagerp yep, that's my actual issue, and it's the case regardless of whether the navbar is collapsible or not.

I believe the container issue was fixed with #21722.


Water/Lava Eggs not functioning

I'm not sure if this is just me or not, but every attempt to use the water and lava eggs has failed. If anyone else has had this issue, please let me know- alternatively, if water/lava eggs are just planned, with the items as placeholders, also let me know.  I have the architecturey api and Polylib by creeperhost installed, along with of course the mod itself, all on 1.18.2. Downloaded/installed mod through curseforge launcher, if that helps.
Many thanks!

This should have been fixed in the latest 1.19 version


Add Hooks Functionality

I have finally gotten around to work on this.
I added 2 hooks initialize_environment and pre_render. The pre_render allow the user to modify the render_kwargs as necessary.
Would love to know your thoughts on this.
Related Issue
Issue #9

Hi Suhendi. I quickly skimmed through the PR and it looks really good! :+1: I'll have a busy start of the week so I may have time to properly review it in the second half.


Node env Error: Can't resolve 'fs'

This is my full code :
import juice from 'juice'
	import htmlDocx from 'html-docx-js'
	import { saveAs } from 'file-saver'

	export default {
		methods: {
			handleExport () {
				var content = htmlDocx.asBlob('hello')
				saveAs(content, 'test.docx')
			}
		}
	}

Run webpack compiler，Cmd show a error :
Module not found: Error: Can't resolve 'fs' in 'F:\gittest\js-office-demo\node_modules\html-docx-js\build

Waiting online .... @anowak @kozborn @gpurgal

@gongph that's not the problem with package itself, but rather with your bundling tool (webpack). You can either consume already bundled (html-docx-js/dist/html-docx-js) package or configure webpack to polyfil node modules (e.g. fs).
Try setting this in your webpack config file:
node: {
    fs: 'empty'
}

Now, I put html-docx.js under the dist  directory.
Very very tks!! @N1kto

Not working for me. When I add
node: {
    fs: 'empty'
}

in my webpack config the "Can't resolve fs" error goes away. After this when I invoke the following function

  exportToWord = () => {
    var converted = htmlDocx.asBlob(this.state.section.content);
    FileSaver.saveAs(converted, 'test.docx');
  }

I get this error:
Uncaught TypeError: fs.readFileSync is not a function
@gongph Is it working for you?

@abhisheknalin try consume bundled version from /dist folder. E.g.:
import htmlDocx from 'html-docx-js/dist/html-docx-js';

const converted = htmlDocx.asBlob(content);
saveAs(converted, 'test.docx');

Thanks a lot for your time.
import htmlDocx from 'html-docx-js/dist/html-docx';
this import works

import htmlDocx from 'html-docx-js/dist/html-docx';
am using this header the app is working but the below linting issue arrises.
help me how to resolve it?
Parse errors in imported module 'html-docx-js/dist/html-docx': Line 12413: 'with' in strict mode (12413:1)  import/namespace

Even my webpack gives the following warning:
WARNING in ./~/html-docx-js/dist/html-docx.js
Critical dependencies:
1:439-446 This seems to be a pre-built javascript file. Though this is possible, it's not recommended. Try to require the original source to get better results.
 @ ./~/html-docx-js/dist/html-docx.js 1:439-446

But it works. So I think you need to edit the linting rules you are using.

Great!!
I think this could be in the readme file

its not an readme file it is an linting warning for import header rules mismatch.

@abhisheknalin try consume bundled version from /dist folder. E.g.:
import htmlDocx from 'html-docx-js/dist/html-docx';

const converted = htmlDocx.asBlob(content);
saveAs(converted, 'test.docx');

Tried this but get the following:
./src/vendor/html-docx-js/html-docx.js
Syntax error: 'with' in strict mode (13103:0)
I am attempting to use this with create-react-app (as I suspect many others are), so the webpack config approach is also not great, I would rather not eject from create-react-app.


Add unit tests for Pack / Unpack / Base functions

Added Pack / Unpack / Base functions UTs
Added go modules support (from go 1.11)
Added go 1.11 to travis

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: plutov sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


recover from a checkpoint state

description
when multiple errors cascade in the network, it's very likely that network stalls to a grind. in situations where mesh data is in a bad state such that tortoise cannot make progress, we want to revert to a genesis-like state and restart tortoise.
considerations:

preserve global state (spawned accounts, balances and nonces)
accounts do not need to be re-spawned
preserve ATXs so that smeshers do not have to re-initialize the storage
prevent replay attack on transactions (via reserving nonce)
it's ok to reset vesting schedule (the only vesting account is for investors/devs)
rewards for staying in the network (?)

proposal
the spacemesh administrator will provide a json file that contains necessary data for the network to recover.
the revived network will use the same genesis ID so that miners can use existing identities and PoST data.
recovery data

reboot layer: the first layer the revived network will start running hare protocol

tortoise will tally votes starting from this layer
syncer and mesh will manage mesh and states from this layer

active set: the bootsrapping active set at reboot layer

beacon: the bootstrapping beacon at reboot epoch

snapshot of all ATXs: this is the most recent ATX of each miner. it serves two main purposes:

miners do not need to re-initialize their PoST data
used as previous/positioning ATX for building next ATXs in the revived network

each ATX should contain

coinbase (for the first epoch rewards after recovery)
space unit (numUnit) (?)
tick height (base tick height + tick count), for new ATX's base tick height
vrf nonce (for building new ATX)
commitment atx (for building new ATX)
publish layer (?)
sequence
initial post indices (?)

snapshot of accounts at a known good layer

implementation plan
a known good layer to recover state from
the spacemesh administrator manually determine a known good layer. (manner TBD)
dump recovery data
given a known good layer, issue an RPC to a node that does the following

pause syncer/hare
revert state to the specified layer
dump accounts table
dump atxs data
unpause syncer/hare

the spacemesh administrator should dump recovery data from multiple nodes to verify that
the dumped data are identical.
load recovery data
node operator download the recovery data from spacemesh website and restart go-spacemesh
binary with --recovery_data pointing to this file. the node will do the following on startup

backup the old database
fill the account table with the account snapshot
fill the atxs table with most recent ATXs from miners
set effective genesis layer to reboot layer - 1

misc

fix https://github.com/spacemeshos/go-spacemesh/issues/3774
make sure commitment atx does not to be available in the atx validation flow, only the bytes exist.
make sure effective genesis layer is used in all the necessary places

i realized that we technically don't need any revert on vm if the block that we want to target was already applied. vm stores accounts with a version (version is layer number), so to get the state at layer 10 - we need to load account state at layer 10 or below. it is better because you can do snapshot without stopping node, so testing is simpler, and you can do that just periodically.

why do you need to pause syncer/hare ?

i missed the fact that accounts table is versioned by layer! that's why i want to freeze the mesh after revert. syncer and hare are the only two components that can advance the mesh state. now it's not necessary.

read sql transaction can be open for as long as you need it, it won't affect anything else, but you will have same concurrency guarantees

yes. will update the description.

adding discussion from slack:
from kimmy
questions about checkpoint recovery

say we checkpoint at layer 200. recover at 205.
but only part of the network recovered from this checkpoint file, say partition A, partition B
continues on with the old mesh.
how should we handle gossip from partition B->A? say a layer 210 ballot from miners in B
and direct peer fetch requests from A->B? say syncing a block certificate for layer 210
currently the only way i can think of is to add some protocol id in gossip/sync requests... any ideas?

from Iddo
When we establish secure channel (sodium secretbox) with peer we should check that the peer's
protocol_id us high enough, and drop the peer if he later sends signed messages (such as ballots)
with lower protocol_id ? Even for the discovery protocol it's better to store in the address book tuples
(pubkey, ip_address, protocol_id) rather than just (pubkey, ip_address) so that we wouldn't try to
select a random peer (for establishing a new secure channel) whose protocol_id is too low.

from Dmitry
Yes a bump in the protocol will do. Technically it is a bump in what we pass to libp2p

from kimmy
thanks both. let me double-check on what i understood.
afaik we have the following 3 kinds of protocol versioning
1. peers connection: handshake protocol version
this prevents establishing peer connection btwn partition A and B
2. gossip: gossip topic version per data type
this prevents gossip msgs mixing btwn partition A and B
3. direct peer requests: protocol version per data type
this prevents fetching data btwn partition A and B
i think updating 1 is sufficient?

from Iddo
kimmy, If I understand correctly then the way that any communication is done is by discovery
(to maintain address book of potential peers) and then (1) establishing secure channel with
several peers and doing all the gossip and direct peer requests with them.
So we just check the protocol_id during (1) and refuse to talk with a peer that has a low
protocol_id, but if the peer is malicious then he might later send gossip data (2) with wrong
protocol_id so if we does it for the kind of data that the protocol_id is important (such as ballots
after the layer in which the upgrade happened because we don't want to count the ballots of
the old version after that layer) then we should drop/blacklist this peer.

I also mentioned that for the discovery it would be nice to have the protocol_id of each potential
peer, so if we can do that then that's better.

from Dmitry
> i think updating 1 is sufficient?
i would rather update all relevant protocols. handshake doesn't block negotiation of the protocol,
so if we also do what Iddo said (as we should), then we may erroneously blacklist node that was
slow with update

a followup conversation with Dmitry regarding

handshake doesn't block negotiation of the protocol

the order of peer handshake and gossip messages are not guaranteed. there was concurrency
issues observed. there maybe a setting to guarantee but it's safer all around to update all protocols.


Proxy-Wasm mega backport

This backport includes Proxy-Wasm C++ Host and SDK updates,
including Emscripten, as well as build and related reliability fixes.

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@kyessenov @bianpengyuan I'm still waiting for https://github.com/envoyproxy/envoy/pull/13753 and https://github.com/envoyproxy/envoy/pull/13836, but I'd like to get those fixes merged into 1.8 release.

I am not happy about changes in istio/envoy that are not in upstream. I'm not convince they solve the problem (e.g. the leak is still there AFAICT) and the configuration model change will have unpredictable impact on stats module.
Have you validated that they do not destabilize istio/proxy?

@kyessenov Those two unmerged changes are not included in this PR, and yeah agree the change has to be in upstream first.

@bianpengyuan Yes, I didn't read "those" correctly at first. Still many of the changes included here are a build-up to the fixes. They are not needed unless we get the actual fixes.

@kyessenov there are no changes here that are not already merged upstream. I tested with the existing tests in istio/proxy.

The only outstanding issue is the memory leak (fixed with either https://github.com/envoyproxy/envoy/pull/13836 or https://github.com/envoyproxy/envoy/pull/13941), everything else is backported here.
@bianpengyuan @kyessenov let me know if you want to merge this now, so that we can run tests with this over the weekend, or if you want to wait for the memory leak to be merged first.

Cherry-picked envoyproxy/envoy#13836, so this is now ready to go.
@bianpengyuan @kyessenov the question is whether we want to merge envoyproxy/envoy#13753 and do extended testing before 1.8.1, or should I remove it from this backport? I originally said that we can drop it, but I'm having second thoughts now.

After some testing, I am worried that wasm extensions that works with 1.8.0 could break with 1.8.1. Is it possible that we can avoid this massive update, especially tool chain and dependencies, and only cherry-pick code fixes?

Is https://github.com/envoyproxy/envoy/pull/13753 the reason why we need to make this cherry-pick so bloated? If that is the case I'd rather push that to 1.9 because I don't believe there is user who heavily reconfigures the plugin.

After some testing, I am worried that wasm extensions that works with 1.8.0 could break with 1.8.1. Is it possible that we can avoid this massive update, especially tool chain and dependencies, and only cherry-pick code fixes?

I agree that this massive update is a bit problematic for the .1 release (in retrospective, I should have just backported those changes individually 2 weeks ago instead of waiting for all fixes to be merged to Envoy, since that took way more time than I anticipated), but the issue is that we cannot cherry-pick select fixes, since we pull all other changes via proxy-wasm-cpp-host update anyway (unless we want to create a separate branch with backports there, but that seems like a lot of unnecessary work).
I can drop envoyproxy/envoy#13707 from cherry-picks, which would retain old Emscripten version for Istio 1.8.x. Everything else (other than envoyproxy/envoy#13753) is either a build or a bug fix.

Is envoyproxy#13753 the reason why we need to make this cherry-pick so bloated? If that is the case I'd rather push that to 1.9 because I don't believe there is user who heavily reconfigures the plugin.

No, the primarily reason is that there was a ton of bugfixes done over the last 2 months.
We could drop envoyproxy/envoy#13753, but then we need to create a branch in proxy-wasm-cpp-host that excludes the corresponding change there, otherwise we'd be losing a few minor fixes that were merged afterwards.

(removed Emscripten update from the backports)

I dropped a few changes from here (including reconfiguration) now that we have istio-release/v1.8 branch in Proxy-Wasm repos. PTAL.

@PiotrSikora: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test-release_envoy_release-1.8
ecd204715cfe35e10afc579a3cabd8cb5d06f7ea
link
/test test-release_envoy_release-1.8

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Test failure seems to be real.

@PiotrSikora: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test-release_envoy_release-1.8
ecd204715cfe35e10afc579a3cabd8cb5d06f7ea
link
/test test-release_envoy_release-1.8

test-asan_envoy_release-1.8
ecd204715cfe35e10afc579a3cabd8cb5d06f7ea
link
/test test-asan_envoy_release-1.8

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@PiotrSikora: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test-release_envoy_release-1.8
ecd204715cfe35e10afc579a3cabd8cb5d06f7ea
link
/test test-release_envoy_release-1.8

test-asan_envoy_release-1.8
ecd204715cfe35e10afc579a3cabd8cb5d06f7ea
link
/test test-asan_envoy_release-1.8

test-tsan_envoy_release-1.8
ecd204715cfe35e10afc579a3cabd8cb5d06f7ea
link
/test test-tsan_envoy_release-1.8

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Test failure seems to be real.

I forgot to revert one of the new tests to the old-style setup with reconfiguration. Fixed now.


Allow user defined docker image

Allow user define Docker image (from Docker Hub) and customized launch command.

This is a great idea @dclong, thanks for sharing. Right now you can customize your builds by adding requirements and env variables, but the Docker image is a great idea.

Labeled for future releases.


corrected spelling of 'list'

Corrected spelling in xgb.cv.R.
I'd be happy to rebuild the documentation... but I'm having issues.
Are roxygen2::roxygenise() or devtools::document() used? Both fail with the following:
 xgboost_R.cc:2:10:fatal error: logging.h: No such file or directory
    #include <dmlc/logging.h>
             ^~~~~~~~~~~~~~~~
   compilation terminated.
   /usr/lib/R/etc/Makeconf:175: recipe for target 'xgboost_R.o' failed
   make: *** [xgboost_R.o] Error 1
   ERROR: compilation failed for package `xgboost`

I tried the make html from the doc folder, but I get the perplexing:
sphinx-build -b html -d _build/doctrees   . _build/html
Traceback (most recent call last):
  File "/usr/bin/sphinx-build", line 14, in <module>
    from sphinx import main
ImportError: cannot import name 'main'
Makefile:55: recipe for target 'html' failed
make: *** [html] Error 1

@paulkaefer Did you do git clone --recursive https://github.com/dmlc/xgboost.git when cloning XGBoost?


Implement opa_eval_ctx_set_entrypoint

See https://github.com/open-policy-agent/npm-opa-wasm/blob/7d30ab746042dd9dffd3446f9cb3c031b62df791/src/opa.js#L250

Yes, for OPA this is every useful feature.

@christophwille any estimation for this fix ?

No plans wrt to timeline (I don't need it).

Does anyone have good concise simple example regos to use as unit tests?

This is a PR from npm-opa-wasm, I thinks regos can be extracted from there
https://github.com/open-policy-agent/npm-opa-wasm/pull/50

@ievgenii-shepeliuk @eshepelyuk Please have a look at  https://github.com/christophwille/dotnet-opa-wasm/pull/17 to see if that would be an API palatable to your use case.

Could you add a test to call entrypoint like example/one/myRule ?
Otherwise LGTM, great work.
Thank you !


feat: optimize document columns

:tada: This PR is included in version 1.65.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Added the story change emitter and on initial load as well as on story and group change.

On every story change if the it is initial load then the previous story details will be null and for others the the previous details will populate.
Issue ticket number and link - (https://github.com/Gauravdarkslayer/ngx-stories/issues/25)
Checklist before requesting a review

[ ] I have performed a self-review of my code.

[ ] Followed the repository's Contributing Guidelines.

[ ] I ran the app and tested it locally to verify that it works as expected.

Hi @Gauravdarkslayer,
I have added the new emitter. Please review the PR and tell me the changes if anything will be there.


Latest version fails

Import 'https://cdn.skypack.dev/-/asap@v2.0.6-vAazu0geAa8UqzgbsaB8/dist=es2020,mode=raw/^2.0.6' failed: 404 Not Found

Ok, found the bug. It's related with a dependency of Denjucks (the Deno version of Nunjucks) that is failing.
I've fixed it cloning the repo and fixing the bugs https://github.com/oscarotero/denjucks
Thanks for warning

I'm a bit confused here.
I found this problem when running a GitLab job, which just uses:
deno run -q --unstable --allow-read --allow-write --allow-net https://deno.land/x/lume/cli.js
so it always uses the latest version. This now works fine.
However, I then tried on my own machine. I ran
deno install -f -r --unstable --allow-read --allow-write --allow-net https://deno.land/x/lume/cli.js
to update to the latest version. I then run lume -v which shows 0.9.3. However, when I then run lume, I get the same error as before
Download https://cdn.pika.dev/asap/^2.0.6
Download https://cdn.skypack.dev/asap/^2.0.6
Download https://cdn.skypack.dev/-/asap@v2.0.6-vAazu0geAa8UqzgbsaB8/dist=es2020,mode=raw/^2.0.6
error: Uncaught (in promise) TypeError: Import 'https://cdn.skypack.dev/-/asap@v2.0.6-vAazu0geAa8UqzgbsaB8/dist=es2020,mode=raw/^2.0.6' failed: 404 Not Found
    at https://raw.githubusercontent.com/thegamingninja/denjucks/master/src/deps.js:3:0

So it looks like it's still using the old version of denjucks.

Yes, I didn't find a good way to upgrade lume (the new version has a lume --upgrade command) but it needs more work.
The best way is delete the cache of deno: Run deno info and then a rm -rf {DENO_DIR location}

seems to be an issue with Deno. Removing the entire cache does work, but it's a bit drastic ☹ There are several issues relating to install on Deno's repo, so perhaps this will be improved in future versions. I'll close this, as the Lume issue is resolved, even if re-installing doesn't seem to upgrade the cache properly.


Dispose IDisosables in HtmlTransformer

Description
There's two IDisposables being used in HtmlTransformer (a Stream and an XmlReader). We should Dispose() both of them.
I think this is mostly a hygiene issue. Assembly.GetManifestResourceStream() allocates memory only, not an some non-memory resource that has to be Dispose()d correctly. Same thing for XmlReader too. So I don't think not disposing these has any real impact.
This was flagged by a static analysis too.
Related issue
Kindly link any related issues. E.g. Fixes #xyz.   -- Shall I file one?

[ ] I have ensured that there is a previously discussed and approved issue.

/azp run

Can I find this extension on nuget? I can see Microsoft.TestPlatform.Extensions.TrxLogger, but I heed the HtmlLogger so I can override functionality to attachments


Figuring out why a problem is infeasible

Hi,
Is there a way to figure out why a problem is infeasible ?

Best way is to iteratively add constraints, to see where it starts becoming infeasible


fix: use xz compression when building deb package

Backports https://github.com/microsoft/vscode/pull/230471
Fixes https://github.com/microsoft/vscode/issues/230418

Approved in Zurich standup.