CSCFAIRMETA-10: Add cumulative datasets to Qvain light

Add support for setting cumulative_state for new datasets.
For existing datasets, use change_cumulative_state RPC to toggle the state. Any changes to the dataset must be saved first. Added "changed" observable to store to enforce this.
Files cannot be removed from cumulative datasets.
Side effects:

On dataset save, "selected files" are moved to "previously selected files" like they would be if you reloaded the dataset after saving.
The onChange handler for title/description mutated store state directly instead of using the store actions that already existed. Not sure why. Fixed so the store can notice when title/description is changed.

Needs this in Metax for cumulated datasets to work properly: https://github.com/CSCfi/metax-api/pull/510


Add romanian language

Add romanian language and reordered language menu.

Looks good, but can you keep the languages on index.html in the original order and add Romanian after Magyar? They're roughly ordered by geography so similar languages can be grouped together.

Done. :)

Thanks!


Magick rock not found

I am using POP OS, kitty, latest neovim nightly appimage
I installed magick with luarocks --local --lua-version=5.1 install magick, but when I try to run nvim --clean -c ":luafile minimal-setup.lua", I get
Error detected while processing User Autocommands for "VeryLazy":
image.nvim: magick rock not found, please install it and restart your editor

I'm not sure why, I checked and magick is indeed in ~/.luarocks/share/lua/5.1, maybe it's because the default installed lua is 5.3?

I vaguely remember getting this error myself, but can't recreate it now. I do remember having issues with the latest nvim (but is just working for me now so don't know).
The error in my case was related to nvim 9.2 moving to LuaJIT 2.1.1692716794, which was having issues with hererocks. You can try nvim 9.1, but I can't explain why 9.4 is suddenly working now for me (or exactly why it didn't work).

I have the same issue under arch and neovim 0.10.0-dev-9afbfb4 (nightly). ImageMagick is installed via paru from the aur and is accesible (which magick reveals /usr/bin/magick). I saw #18 and figured it's probably a related issue, but being a MacOS the solution didn't help me too much. I will say that I didn't spent too much time looking into this though as I was just interested in trying image.nvim and when it didn't run and I couldn't figure out how to solve in ~20 minutes I figured I'd wait until a patch emerges to try again, but I am happy to test any suggestions or provide any feedback.

Could you test on stable just so we can rule that out? I don't have the issue on nightly on macos though so I'm not hopeful

You could edit this file https://github.com/benlubas/image.nvim/blob/1355ea1dbc059136c6d6bd3fbefb0838ebe4ec85/lua/image/magick.lua#L1 so that it prints the full error (I should submit a PR to make this the default tbh). The variable magick will contain the error, so just print that with vim.inspect() b/c it might be a table.
If that doesn't give you anything useful you can go into the luarock itself and poke around (this is what I did) and go add some print statements.
There's also one thing that you can do to verify that it's a neovim path issue that's mentioned in the other issue. This comment.

Ah, if the pcall fails here magick has the error?
local has_magick, magick = pcall(require, "magick")

Yup

Okay, so the error (for me) occurs on line 55 of init.lua, not in magick.lua. When this error is tripped, here's the contents of magick.
{
  has_magick = false,
  load_image = <function 1>,
  magick = "module 'magick' not found:\n\tno field package.preload['magick']\ncache_loader: module magick not found\ncache_loader_lib: module magick not found\n\tno file './magick.lua'\n\tno file '/home/runner/work/neovim/neovim/.de
ps/usr/share/luajit-2.1/magick.lua'\n\tno file '/usr/local/share/lua/5.1/magick.lua'\n\tno file '/usr/local/share/lua/5.1/magick/init.lua'\n\tno file '/home/runner/work/neovim/neovim/.deps/usr/share/lua/5.1/magick.lua'\n\tno file '/
home/runner/work/neovim/neovim/.deps/usr/share/lua/5.1/magick/init.lua'\n\tno file './magick.so'\n\tno file '/usr/local/lib/lua/5.1/magick.so'\n\tno file '/home/runner/work/neovim/neovim/.deps/usr/lib/lua/5.1/magick.so'\n\tno file '
/usr/local/lib/lua/5.1/loadall.so'"
}

Interestingly if I'm inside of the ~/.local/share/nvim/lazy/image.nvim/lua/image directory at the time then the error doesn't trigger due to the presence of magick.lua there (I can also touch magick.lua in any directory to stop this error occurring on nvim load).
From what I understand, this is looking in the listed locations for magick.lua, but a quick fd magick.lua --hidden from my home directory reveals that .local/share/nvim/lazy/image.nvim/lua/image/magick.lua is the only place this exists.
For me, running luarocks --local --lua-version 5.1 install magick installs magick to ~/.luarocks/share/lua/5.1/magick, but I have no magick.lua in this directory or sub directories after install anyway. The files in this directory are base_image.lua gmwand.lua version.lua enum.lua init.lua wand gmwand thumb.lua wand.lua, where gmwand and wand are subdirectories, each containing the files data.lua image.lua lib.lua.

Ah, just tested this with neovim v0.9.4 and the error doesn't occur, so it seems to be some problem with image.nvim and neovim 0.10.

I followed the steps from https://github.com/leafo/magick, and ran
sudo apt-get install luajit
sudo apt-get install libmagickwand-dev
sudo apt-get install libgraphicsmagick1-dev

Now, when I run the nvim --clean -c ":luafile minimal-setup.lua", I don't get any errors, but the image does not load, There's just an empty space in between the lines, but no image loads.
I'm running kitty without tmux, if that's the issue maybe.

@HawkinsT it's ok that there's no magick.lua, it should load magick/init.lua.
 Apt has an old version of Kitty that has caused issues in the past. So installing the latest kitty version could fix it

Thanks! I installed kitty through the curl command and it works now.

@bananaboy139 awesome!
could you close this issue now that everyone is resolved?


Custom ORDER BY clause with ModelCriteria

I would use NULLS LAST option in the ORDER BY clause. However, no function of the class ModelCriteria allows this.
Is there a solution without rewriting the query into native SQL language ?
Thanks.

Not built-in, but you can write your own in your Query class, that modifies $this->orderByColumns

Obviously... thanks, it's perfect !


HelmDeploy@0 support multiple valueFiles

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type:  Feature unless it's supposed to already do it
Enter Task Name: HelmDeploy@0
list here (V# not needed):
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
Environment

Server - Azure Pipelines or TFS on-premises? Azure Pipelines

Issue Description
I would like to be able to pass multiple files to the valueFile input of the task. Helm itself supports this with using --values multiple times but the task only allows 1 to be specified.

@shigupt202 this seems like a good enhacement. Perhaps we could use the same input with \n as a delimiter.
Then people can supply something like
valuesFile: |
   values1.yml
   values2.yml
   values3.yml

and we can apply them in the same order. This would ensure back-compat as well.
Thoughts?

Agreed. I'll add the support.

Any idea why this change was reverted in #13292?
This would be a really useful feature as Helm supports multiple values files but the task does not


How can I change it to stream output mode

Has the previous stream output mode been modified? The current overall response output has a long waiting time. Would you please tell me what parameters to modify to revert it back?

If you are using Nginx as a reverse proxy, you need to add the parameter proxy_buffering off; to the configuration

If you are using Nginx as a reverse proxy, you need to add the parameter proxy_buffering off; to the configuration

In which file do I add this?

There is also still an issue with some browsers (eg Firefox) where streaming is delayed due to an absent server response header when the chatbot answers.  Described here:
https://github.com/mckaywrigley/chatbot-ui/issues/343

In which file do I add this?

In the nginx configuration file containing the server block that controls the app.   The main nginx config is frequently /etc/nginx/nginx.conf

There is also still an issue with some browsers (eg Firefox) where streaming is delayed due to an absent server response header when the chatbot answers. Described here:

Ah, I noticed this, as I use LibreWolf which is a fork of Firefox. Had me so confused.

In the nginx configuration file containing the server block that controls the app. The main nginx config is frequently /etc/nginx/nginx.conf

Oh, does it have to be a system-wide configuration, rather than having the stream output only for the chatbotui app?

There is also still an issue with some browsers (eg Firefox) where streaming is delayed due to an absent server response header when the chatbot answers. Described here:

Ah, I noticed this, as I use LibreWolf which is a fork of Firefox. Had me so confused.

In the nginx configuration file containing the server block that controls the app. The main nginx config is frequently /etc/nginx/nginx.conf

Oh, does it have to be a system-wide configuration, rather than having the stream output only for the chatbotui app? My apologies as I haven't been exposed to Nginx much before these last few days.

Well, there are two separate issues.  Nginx buffering will prevent any streaming, while the missing server response header (for me anyway) caused Firefox to buffer a certain amount of text and then begin to stream.
I'm a total JS and nginx newb, so not the best source of advice.   Ask the bot!   With GPT assist I was able to fix both issues.  The code I posted on the other thread works to fix Firefox.  For nginx I think I ended up disabling streaming globally on my server.   Didn't notice any bad effects.

I think it necessary to put a stream toggle options, since more comfortable and less buggy (especially when outputting code) with stream turned false.


Support extra Heroku apps:create options - region, space, team

It would be useful to document how to pass Heroku CLI options on datasette publish, e.g. --region eu.

Now that I've split the heroku command out into a separate default plugin this is a much easier thing to add: https://github.com/simonw/datasette/blob/master/datasette/publish/heroku.py

In case it's useful to anyone, as datasette publish heroku doesn't yet support Heroku's cli tool --team flag: If you are using a Heroku personal account but need to use a team/organization account instead, set a default team for your account to use. Commands run through datasette publish heroku will then be actioned using that Heroku team. Kudos to the Heroku support team for pointing this out to me 🤦 .


I add a new feature : Export  Top-Group as Layer

this feature will make the group which is level 1 (the top level) group to a single image.
I think this feature is useful for somebody.

https://github.com/finscn/Photoshop-Export-Layers-as-Images

:+1:

+1

+1


Enable wheels CI scripts to run locally

Taking inspiration from #35 and #62
This PR adds / updates:

rapids-configure-sccache (new) : configures sccache for CI/local, with the intention being to remove this information from CI containers
rapids-env-update (modified) : calls the rapids-configure-sccache
rapids-prompt-local-repo-config (new) : consolidates prompts for users to configure repo information locally
rapids-download-wheels/conda-from-s3 (updated) : uses rapids-prompt-local-repo-config
rapids-upload-wheels/conda-to-s3 (modifed) : guards to ensure local builds don't try to upload, also removes exit 0 for local builds

Is there a way to avoid duplicating this logic from the conda builds? Maybe putting it into a separate tool? I agree that we don't want to (nor can we really) lump the logic into rapids-package-name, but I also would like to share conda/wheel logic on this.

Also you may want to make sure that whatever you do supports https://github.com/rapidsai/gha-tools/pull/62/ (another reason to avoid duplicating if possible).

I already copied over #62 here.
I'll think on how to minimize duplication, maybe we can combine the prompts as part of another tool.

Yeah another tool seems like the lowest hanging fruit.

@ajschmidt8 as we move towards local builds, @vyasr and I discussed that some steps may be entirely skippable locally and the build would still be able to succeed. Uploading to S3 is one such step, and I don't think we should exit early.


Count photo views

Probably through https://github.com/charlotte-ruby/impressionist

Fixed by #486


Invalid doc or implementation for  "Preload functions" - case with has_many :through

https://github.com/elixir-ecto/ecto/blob/761d8a3ac2a3e2d309d69b10046252d1a9cd9ea3/lib/ecto/query.ex#L1925-L1934
In terms of the schemas in docs, running this
Repo.all(from p in Post, preload: [comments_authors: ^&IO.inspect/1])

raises an exception, of course, but also prints ids of the... authors. Am I missing something?

What is the exception that you get?

What, the function that you give must execute a query. So if you are only inspecting and not return the query results, then it won’t work.

yes, but documentation says that function will receive the IDs of the comments and not of the         posts. But I got ids of the users.

Can you please fill in the issues template then and provide an app or a test that reproduces The error? Thanks!


Enable STS as an option to pull credentials

Use AWS STS to create short lived user credentials to use for ECR credentials.

// cc #7


Consistent information to generated files

CMAKE adds the following information to generated files:
# CMAKE generated file: DO NOT EDIT!
# Generated by "Ninja" Generator, CMake Version 3.23

Based on that I would suggest to add to all *.h files that are generated by csolution/cbuild the following text:
# CSOLUTION generated file: DO NOT EDIT!
# Generated by: csolution version 1.4.0

We should align this also with legacy products like uVision, CMSIS-Pack Eclipse.
For *.yml, i.e. cbuild.yml the proposal is to add:
  generated-by: csolution version 1.4.0

We should agree that these generated files do not get updated on disk if only the version of the tool is different.

The files get generated in RTE Model. It  does not know the tool that is using it.
We need to pass the tool name and its version to RTE Model.
I would rather stick to use neutral "Run Time Environment", other wise every tool will overwrite the content because it will differ.

Just FYI, The current header of the generated *.h files generates header info something like
/*
 * Auto generated Run-Time-Environment Configuration File
 *      *** Do not modify ! ***
 *
 * Project: 'Project'
 * Target:  'Target 1'
 */

The files get generated in RTE Model. It does not know the tool that is using it. We need to pass the tool name and its version to RTE Model. I would rather stick to use neutral "Run Time Environment", otherwise every tool will overwrite the content because it will differ.

In order to avoid the overwriting, We can avoid updating the files if the difference is only in the tool name or version.

To be included in cmsis-toolbox 1.7.0

Please review implementation in CMSIS-Toolbox 1.7.0 and raise a new issue if there is anything missing.


Installation Difficulty

Many of the dependencies of MpGlue have now switched to PY 3.x, which makes MpGlue difficult to install outside of a Docker instance, which isn't always possible.  What would it take for MpGlue (and spfeas) be modified to allow dependencies with later py versions?

Dear Jordan,
Thank you for all the work.  Just a heads up that there seems to be an issue with some of the dependencies around GDAL.   The installation works just fine, but running python -c "import mpglue;mpglue.test()" results in execptions saying "GDAL Python must be installed", when it is, ver 2.3.3
During the handling of that exception there is also an ImportError from importing the Vector Tools.
Thanks for checking it out!

Hi Walker,
The first thing that I can think of is the relative location where you are executing the test. Where are you running the test line from? Inside the Git directory?
Are you using Python 2.x or 3.x?

Does this issue solve your problem?

Hi Jordan,
I was running it from a local directory (downloads), but upon further reflection with Ben it seems to be an install issue with MS Visual C++.  I was running py3.7 but  I'm up and running now on the 2.7 version using docker.  At some point we'll go through and see about updating the docker instance to run the py3.x mpglue and spfeas.


Switch to folder IDs in Exchange

Store all folders from Exchange by folder ID
in kopia. Remove the old duplicate folder
name stop-gap measure as well since it's no
longer required
Update tests to check LocationPath instead
of FullPath since LocationPath still has
display name info

Does this PR need a docs update or release note?

[x] :white_check_mark: Yes, it's included
[ ] :clock1: Yes, but in a later PR
[ ] :no_entry: No

Type of change

[x] :sunflower: Feature
[x] :bug: Bugfix
[ ] :world_map: Documentation
[ ] :robot: Supportability/Tests
[ ] :computer: CI/Deployment
[ ] :broom: Tech Debt/Cleanup

Issue(s)

closes #3197

Test Plan

[x] :muscle: Manual
[ ] :zap: Unit test
[x] :green_heart: E2E

Manually tested making an incremental backup from an old backup without LocationRef info for Exchange and an old OneDrive backup and results were as expected


--line-ending does not seem to do anything

Version information
PS C:\Projects\DigitalCore> sf version --verbose --json
{
  "architecture": "win32-x64",
  "cliVersion"<EMAIL_ADDRESS>  "nodeVersion": "node-v20.8.1",
  "osVersion": "Windows_NT 10.0.19045",
  "rootPath": "C:\\Users\\JanvadH\\AppData\\Roaming\\npm\\node_modules\\@salesforce\\cli",
  "shell": "powershell",
  "pluginVersions": [
    "@dxatscale/sfpowerscripts 25.5.3 (user) published 356 days ago (Fri Nov 24 2023)",
    "@jayree/sfdx-plugin-org 1.2.56 (user) published 145 days ago (Sun Jun 23 2024) (latest is 1.2.81)",
    "@oclif/plugin-autocomplete 3.2.7 (core)",
    "@oclif/plugin-commands 3.3.4 (user) published 188 days ago (Sat May 11 2024) (latest is 4.1.8)",
    "@oclif/plugin-help 5.2.17 (user) published 461 days ago (Sat Aug 12 2023) (latest is 6.2.16)",
    "@oclif/plugin-not-found 3.2.24 (core)",
    "@oclif/plugin-plugins 5.4.15 (core)",
    "@oclif/plugin-search 1.2.13 (core)",
    "@oclif/plugin-update 4.6.8 (core)",
    "@oclif/plugin-version 2.2.15 (core)",
    "@oclif/plugin-warn-if-update-available 3.1.20 (core)",
    "@oclif/plugin-which 3.2.16 (core)",
    "@salesforce/cli 2.65.8 (core)",
    "apex 3.5.5 (core)",
    "api 1.3.1 (core)",
    "auth 3.6.70 (core)",
    "data 3.9.0 (core)",
    "deploy-retrieve 3.15.4 (core)",
    "info 3.4.15 (core)",
    "limits 3.3.37 (core)",
    "marketplace 1.3.2 (core)",
    "org 5.0.2 (core)",
    "packaging 2.4.6 (user) published 161 days ago (Thu Jun 06 2024) (latest is 2.9.0)",
    "schema 3.3.39 (core)",
    "settings 2.4.2 (core)",
    "signups 2.3.12 (user) published 153 days ago (Sat Jun 15 2024) (latest is 2.6.3)",
    "sobject 1.4.44 (core)",
    "telemetry 3.6.18 (core)",
    "templates 56.3.26 (core)",
    "trust 3.7.38 (core)",
    "user 3.6.0 (core)",
    "sfdmu 4.33.17 (user) published 197 days ago (Thu May 02 2024) (latest is 4.36.7)",
    "sfdx-git-delta 5.40.2 (user) published 155 days ago (Thu Jun 13 2024) (latest is 5.49.4)"
  ]
}

Goal
I want to insert records using sf data import bulk from a CSV file.
Issue
We are running Windows. The CSV file uses LF line endings. The CLI does not recognize this, and attempts to upload the file using CRLF endings, which causes a mismatch issue. Rerunning the import command using the --line-ending LF flag does not affect the behavior of the CLI, and we receive the same error telling us that it's trying to use CRLF endings during upload, which causes an error.
Error message:
PS C:\Projects\DigitalCore> sf data import bulk -o orgName -f fileWithLfLineEndings.csv --line-ending LF
───────────────── Importing data ─────────────────

✔ Creating ingest job 3.06s
✘ Processing the job 5.58s
   ▸ Processed records: ✘
   ▸ Successful records: ✘
   ▸ Failed records: ✘
/lightning/setup/AsyncApiJobStatus/page?address=%2F750QI00000BCEbxYAH)
Elapsed Time: 9.04s

Error (JobFailedError): Job failed to be processed due to:

ClientInputError : LineEnding is invalid on user data. Current LineEnding setting is CRLF

To review the details of this job, run this command:

sf org open --target-org<EMAIL_ADDRESS>--path "/lightning/setup/AsyncApiJobStatus/page?address=%2F750QI00000BCEbxY

Workaround
We are able to work around this issue by changing the line endings in the CSV file to CRLF using a text editor or shell utility.
Expected behaviour
The --line-ending LF flag causes the CLI to perform an upload using LF line endings, instead of ignoring it and using CRLF all the time.
Additional info
The help info for the command contains a typo:
--line-ending=<option>  Line ending used in the CSV file. Default value on Windows is `CRLF`; on macOS and Linux it's `LR`.
                              <options: CRLF|LF>

Presumably 'LR' should be 'LF'.

Hey @OscarTheGrinch - we're aware, and already working on the fix!

Hi @WillieRuemmele , thanks for the lightning fast response! (Pun somewhat intended) In that case I'll close this ticket, and watch the PR.


[WIP] Fix work with promise chains

Our goals:

[x] Fix https://github.com/Codeception/CodeceptJS/issues/543 - After block not properly executed if Scenario fails
[x] Expected behavior in promise chains: _beforeSuite hooks from helpers -> BeforeSuite from test -> _before hooks from helpers -> Before from test - > Test steps -> _failed hooks from helpers (if test failed) -> After from test -> _after hooks from helpers -> AfterSuite from test -> _afterSuite hook from helpers.
[x] if during test we got errors from any hook (in test or in helper) - stop complete this suite and go to another
[ ] if during test we got error from Selenium server - stop complete this suite and go to another
[ ] if restart option is false - close all tabs expect one in _after (ready for webdriverIO)
[ ] Don't close browser between suites, when restart option is false. We should start browser only one time and close it only after all tests.

Please fix test/runner/interface_test.js, it tests the order of events

Wow, awesome work. We are getting closer and closer to 1.0 release!

Please try to use interactive shell to check that nothing was broken for it.
Also, execute one test with within block to be sure this changes to recorder doesn't break them.

@DavertMik
During these changes I found one interaction that may be not correct. I want to ask your advice about it.
Now we have clear promises chain as described in topic. It will break only after fail. Also if we get error in BeforeSuite, Before or After hooks we will skip all tests in this suite (looks it's ok, because there is no sense to try repeat BeforeSuite/Before/After if it's not working). But there is an interesting ways with After and AfterSuite hooks.
If After hook from test will fail, then _after hooks from helpers will not be executed.
The same thing with AfterSuite - if it fails, then _afterSuite hooks from helpers will not be executed. It looks normal, but this can cause an issue, that after fails in these hooks, browser will not close or we will not clear environment for next tests.
I think, that maybe we can track this and force close/clear browser in this cases.
What do you think? Should we track this, or it should be resolved from user side?

If After hook from test will fail, then _after hooks from helpers will not be executed.

Yes, this makes a big problem for closing browser sessions...
I have 2 ideas about this:

run _after hook for Helper even test has failed in After. Same for AfterSuite.
add new finalize hook which will be executed no matter what happened.

But this is important to solve it in one way or another.

@DavertMik please review the changes. pause() and within block works as expected


Config seperation from code

per the well known guidance for 12 factor apps separate configurations from Code
see django docs on settings
configs to include

[x] Database connections
[x] Django Secret key
[x] RCRAinfo URL (prod or preprod)

package options

Python standard library (Great, no dependencies but something to load vars into env would be good)

[x] python-dotenv

python-decouple
django-environ like dotenv but django specific

#36

#36


Allow for customizable background

The extension is great as it takes inspiration from the great inlay hints IntelliJ and other JetBrains products offer, but it would be great to see a setting that would look like this:
"inline-parameters.backgroundColor": "#123456",
You could pass hex code values or rgb/rgba values along with false if you wanted a transparent background as well!
This would alter the background color behind the text of the inlay hints. I'm currently using a One Dark Pro theme and this is what the hinting looks like:

The idea would be to alter the coloring behind url:

Does inlineparameters.annotationBackground not already do this as you want?

Here is what I am getting when I try to implement the setting as you described:

That's not what the only difference should be. As this extension's readme says, it is not a setting, it is a themeable colour customisation and needs to go within a workbench.ColorCustomizations object. For example:

  "workbench.colorCustomizations": {
    "inlineparameters.annotationBackground": "#ff0000"
  }

I've just tested that still works on the latest version myself and it does, so I'm going to mark this as closed. Let me know if you have any problems getting that working.


Update Plexamp and platform runtime versions

Everything seems to work fine here, except that when I try to open this via the launcher, it fails silently. If I do make run it works perfectly though.
I'm new to this area of Linux, but the changes are straightforward enough. Let me know what you think!

Thanks for the contribution!
As for your question, make install installs the flatpak as a user-specific one. On my Linux distro (openSUSE with Gnome), the launcher doesn't even show the icon of Plexamp.
I think the right solution would be to submit this flatpak to the flathub, and then install it as a system-wide one.
Unfortunately I didn't have time to do the submission due to a lack of time and, most important of all, due to some legal questions I have. I don't know if I'm entitled to distribute an app done by someone else on flathub...


Remove code that prints "<SPACE/NULS>" in hexdumps

when the data block ends with SPACEs or NULs.
The problem is, you can't see if the data ends
with SPACE or NUL or a combination of both.
This can happen for instance with
openssl rsautl -decrypt -hexdump

I would like to fix this in 1.1.0 and 1.0.2 as well.
What do you think?

Hmmm. Is this really a bug fix? If not then this is just a change in behaviour which wouldn't be allowed in 1.1.0 or 1.0.2. It its not allowed there then maybe it shouldn't be allowed in 1.1.1 either?

Yeah, I ran into it when I multiplied a pkcs padded cipher text message with 256^e mod N
and was surprised that the openssl rsautl -decode -raw -hexdump
did not show the value of the last byte.
When I looked further I realized that I have no way of knowing if the last byte
is 0x20 or 0x00.

you know it is 20th anniversary of the bleichenbacher attack....

Yeah, I ran into it when I multiplied a pkcs padded cipher text message with 256^e mod N
and was surprised that the openssl rsautl -decode -raw -hexdump
did not show the value of the last byte.
When I looked further I realized that I have no way of knowing if the last byte
is 0x20 or 0x00.

But does that actually make it a bug fix? Or just highlight a deficiency in our functionality? It appears to work as designed. I guess the main thing is whether this change would be likely to break anything (e.g. automated scripts etc). Perhaps that's unlikely? I'm not sure.

It is clearly a bugfix.  The manpage says that hexdump "outputs the hex data"
The BIO_dump routine is not documented.
Yes this fix should be ported to all releases.

Thanks,
I will fix the commit message before I check in (s.SPACE/NULS.SPACES/NULS.)

Pushed to master.
Cherry-pick did not work, so I will do a new PR for release branches.


Support 100 continue responses

It seems that HTTP isn't following the 100-continue responses:
HTTP.headers('Content-Type' => 'application/json', 'Accept' => 'application/json').post('https://foo.bar/api', body: '{...}')
=> #<HTTP::Response/1.1 100 Continue {}>

when cURL does:
curl -v -X POST \
>   https://foo.bar/api \
>   -H 'Accept: application/json' \
>   -H 'Content-Type: application/json' \
>   -d '{...}'
...
> POST /api HTTP/1.1
> Host: foo.bar
> Accept: application/json
> Content-Type: application/json
> Content-Length: 294
>
* upload completely sent off: 294 out of 294 bytes
< HTTP/1.1 100 Continue
< HTTP/1.1 200 OK
...

I would expect the HTTP response to be <HTTP::Response/1.1 200 OK {}>.
Is it something what the HTTP gem should support?

Any feedback on this one, please?
This is currently blocking us from building out an integration as the partner API is returning these 1xx codes.
@ixti
Thanks already.

If I understand correctly, yes, that's something we should support. As far as I understand second response comes immediately after the 100 Continue - in other words 1 request, but 2 responses. Am I right?

Indeed, it seems that we are receiving 2 responses from one request. I'll see to prepare a PR to support this 100 continue code.

I've tried to add support for reading the second response and the only solution which I've found is a change in readpartial method:
return unless @pending_response
   chunk = @parser.read(size)
   return chunk if chunk
   finished = (read_more(size) == :eof) || @parser.finished?
+
+  if finished && @buffer.include?("HTTP/1.1 100 Continue")
+    @parser.reset
+    finished = false
+  end
+
   chunk    = @parser.read(size)
   finish_response if finished

It seems to be very hacky for me, and the response code will be always 100.
I've spent few hours more to find anything better. Experimented with @state.final_chunk? in parser.rb but it seems to always return false for me(even when the response isn't in chunks at all).
Anyone is having any idea where we can put the logic to support this kind of scenario?
For more context - in Net::Http it's implemented in this way:  (ruby-2.5.1/lib/ruby/2.5.0/net/http.rb)

https://github.com/httprb/http/commit/eb7b9529aa74b75823c9d6321ef4c9741031601f -- that's half-working patch :D


Sound level control in Pushover messages?

Would it be feasible to get a volume level indication in the Pushover configuration. Does Pushover even allow?

double post


Port pricing

kept header transparent on pricing, terms, privacy for continuity with home page, also liked how it looked a bit more. split up footer into own component.

Check travis for linting errors that need fixing. Also consider setting up the eslint plugin for your text editor to make your life easier


Database boilerplate and errors on first run

Hello,
can you provide a pre-filled basic database like agencyos did?
I imported your schema to sqlite db without issue, followed your basic hints for creating home, main and footer, but there are not detailed infos about what to fill in Globals.
Is that ok to have only "en" language or i need to insert also "zh"?
I just filled the * fields in Globals but i have these errors on home:

and a console log with:
⚠ metadata.metadataBase is not set for resolving social open graph or twitter images, using "http://localhost:3000". See https://nextjs.org/docs/app/api-reference/functions/generate-metadata#metadatabase
Warning: Each child in a list should have a unique "key" prop.

Check the top-level render call using <div>. See https://reactjs.org/link/warning-keys for more information.
    at HeroBlock (webpack-internal:///(rsc)/./components/blocks/HeroBlock.tsx:20:22)

Maybe i miss something.
Thanks

The main issue you encountered is that in the backend (Directus), many fields are optional and can be left blank. However, the frontend is built with TypeScript, and when rendering, it throws a null error for those empty fields. Since i haven't implemented much error handling on the frontend, the suggested solution is "if you need to fill in content, try to fill in all the relevant fields." In future development, i plan to make many fields in the backend mandatory during creation.

By the way, only en language is ok. just make sure NEXT_PUBLIC_LOCALE_DEFAULT is 'en'.
Also, Be aware of this project is currently under heavy development(see other branch). things may break a lot.

pre-filled basic database is on plan.

Thanks for the answer, i'll try to replicate as i can your model, BTW i ask you what are the goals to have project_setting and blog_setting in Globals, what are the scopes of having that?
BTW sometimes i have this error

⚠ metadata.metadataBase is not set for resolving social open graph or twitter images, using "http://localhost:3000". See https://nextjs.org/docs/app/api-reference/functions/generate-metadata#metadatabase
Check the top-level render call using . See https://reactjs.org/link/warning-keys for more information.
at HeroBlock (webpack-internal:///(rsc)/./components/blocks/HeroBlock.tsx:20:22)

Is it related to missing values on Globals or social empty?

pre-filled basic database is on plan.

maybe you can also provide something like "create in vercel" or "deploy on netlify" buttons.

Thanks for the answer, i'll try to replicate as i can your model, BTW i ask you what are the goals to have project_setting and blog_setting in Globals, what are the scopes of having that?
BTW sometimes i have this error

⚠ metadata.metadataBase is not set for resolving social open graph or twitter images, using "http://localhost:3000". See https://nextjs.org/docs/app/api-reference/functions/generate-metadata#metadatabase
Check the top-level render call using . See https://reactjs.org/link/warning-keys for more information.
at HeroBlock (webpack-internal:///(rsc)/./components/blocks/HeroBlock.tsx:20:22)

Is it related to missing values on Globals or social empty?

The warning you can ignore for now. Then, you can go to https://directusapi.com/admin/login to see my content setup. account<EMAIL_ADDRESS>pass: yuewu.dev
project_setting controls the heading content of "Projects page",like this page blog_setting allows you set featured post, posts per page setting and some other texts in the front-end, like this page

Thanks, i'll compare your backend with mine on self-hosted setup.
Appreciate your effort.


chore: upgrade setfit and remove huggingface_hub restriction

Description
This PR upgrades setfit to 0.7.0 or higher, as there was an incompatibility issue with huggingface_hub 0.12 or higher, which was preventing us to install the latest huggingface_hub version within the integrations extra requirements. So on, this PR akso removes the huggingface_hub restriction (while keeping the previous one >=0.5.0 for backwards compatibility).
From now on, everyone should be able to pip install argilla[integrations] without a dependency version mismatch issue that was being triggered before due to the incompatibility of datasets and huggingface_hub, which was unrelated to setfit, but since the previous restriction on huggingface_hub was due to setfit, patching setfit fixes the rest of the issues.
Type of change

[x] Dependency upgrade / fix (upgrades/downgrades/fixes a dependency related issue)

How Has This Been Tested
(Please describe the tests that you ran to verify your changes. And ideally, reference tests)

[x] Run pip install -e ".[integrations]" with a Python matrix from 3.8 to 3.11 and then running pytest -s tests/integration to re-run the integration tests to ensure that the fix does not affect any

Checklist

[ ] I added relevant documentation
[x] follows the style guidelines of this project
[ ] I did a self-review of my code
[ ] I made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I filled out the contributor form (see text above)
[ ] I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

Running pytest -s tests/integration works great besides the integration test reported at https://github.com/argilla-io/argilla/issues/3940, which should have been fixed already but it's still failing!


Suggestion: Custom Pipe Node

The 'pipe' nodes already exist, but they are predefined. I rarely use them because they don't allow me to transmit all the data I want.
The idea would be to connect any node to the 'custom pipe' node in order to truly have a workflow with as few lines as possible.

Do you think such a node makes sense, and does it appeal to you?

So that's exactly what I've made Context (and, Context - Big) for. It's essentially a more robust "Pipe" from other nodes, but much more flexible since the node itself can merge in new data, and output data from a single note (without needing "from pipe" nodes, etc).

So, they can be used to transmit large contextual data across a workflow, but also merge in. Of course, the cleanliness comes from collapsing them:

Oh 😮 I'm sorry I didn't see the point of this node sooner, I blame myself 😄
Thanks for your reply, next time I'll be more vigilant before asking a question!


Fix: can't query data from the MongoDB

Signed-off-by: barnettZQG<EMAIL_ADDRESS>Description of your changes

Fixes #3094
I have:

[x] Read and followed KubeVela's contribution process.
[x] Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
[x] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Special notes for your reviewer

有两个问题导致mongo不工作：

默认情况mongo存储的数据key都是全小写，但是我们代码里面写的驼峰，导致无法筛选查询。
之前实现GET查询时用的主键默认是name，但是有很多模型主键不是name，所以需要统一一个主键存储。


cerevisiae with radiation simulation is possible?

Description of the issue:
Hi everyone, I have a question. The question may sound strange because I'm not a biology major. Is the result of the radiation irradiation on the cerevisiae and the model results are on caffeine? If not, can the metabolisms of cerevisiae be randomly deactivated due to radiation and the result there of be represented by code?
Expected feature/value/output:
Current feature/value/output:
nope
Reproducing these results:
nope
I hereby confirm that I have:

[ ] Tested my code with all requirements for running the model
[ ] Done this analysis in the main branch of the repository
[x] Checked that a similar issue does not exist already
[ ] If needed, asked first in the Gitter chat room about the issue

No, such effects are not captured by the model, unless one would manually expand the model to somehow consider it.


proxy

Hi, I use private proxies, 10 sites, in the config 2 proxy addresses, the request time is 15000, but after connecting to all sites I see ban

So strange, I need more information about your setup to help you

So strange, I need more informations about your setup to help you

{
    "mongodb_uri": "mymogourl",
    "webhook_url": [
        "myhook"
    ],
    "discord_message_settings": {
        "botImage": "https://i.ibb.co/rG68QRy/logo123.jpg",
        "botName": "Shopify monitor",
        "footerDescription": "R.W",
        "footerImage": "https://i.ibb.co/rG68QRy/logo123.jpg",
        "timeOfNotification": true
    },
    "requestTiming": 15000,

    "__KEYWORDS_EXPLAINED" : "In this default example that I've created the monitor will notify you if find a Nike product in hanon or sensinko site, and it'll notify if find a New Balance or Adidas Yeezy product only in hanon site",
    "keywords": [["Nike","Dunk"],["Yeezy"],["Sacai"],["Nike","Travis"],["Nike","Cactus"],["Nike","Jordan","Retro"],["Nike","Stussy"],["Nike","Jordan","Low"],["Nike","SB"]],
    "sites": [
 	{"url": "kith.com"},
        {"url": "undefeated.com"},
        {"url": "noirfonce.eu"},
        {"url": "deadstock.ca"},
        {"url": "packershoes.com"},
        {"url": "laceupnyc.com"},
        {"url": "onenessboutique.com"},
        {"url": "shop.atlasskateboarding.com"},
        {"url": "crossoverconceptstore.com"},
        {"url": "dope-factory.com"}

             ],
    "proxiesList": ["http://myproxy","http://myproxy"]
}

Distributor ID: Debian
Description:    Debian GNU/Linux 10 (buster)
Release:        10
NodeJS v14.4.0


Change html tag type for canonical links from meta to link

What kind of changes does this PR include?

Minor content fixes (broken links, typos, etc.)

Description

Closes #

What does this PR change? Give us a brief description.
Change html tag type for canonical links from meta to link in examples provided for the markdown/mdx sections. I'm taking part in HACKTOBERFEST.

Did you change something visual? A before/after screenshot can be helpful.

Hello @lushiyun! Thank you for your PR.
This is a well-spotted typo in our code blocks, however, I would like to follow our translation guidelines and only update the English docs for the page markdown-content. This will trigger an update for our translators to update their respective languages.
For integrations-guide/mdx this page is autogenerated from the mdx integration README. We would greatly appreciate it if you can make a separate PR to astro.
Again thank you very much for your contribution, we are very happy to have you be part of our community.

Hi @kevinzunigacuellar. Thanks for the response! I'll make a separate PR to astro for the MDX section. I enjoyed reading astro's docs and happy to be part of the community!

Hi @lushiyun!  Ooops... I thought I could make a change and remove the changes from the other files that shouldn't be in this PR. But, when I tried, I might have deleted the page entirely...
So, I'm going to close this PR, and ask you to please make a new one that only fixes the link in markdown-content in the English docs!
Sorry about that, but I think you might get Hacktoberfest credit for this PR, too, even if it's closed because I'm not removing the label. So, more credit! Yay!  🥳


More error message errors

Another little bug I found when I was messing around with the maxLength setting.

Releasing as part of a new release now - thanks!


CLI exits with 0 even when graph deploy has failed

Which packages are impacted by your issue?
@graphprotocol/graph-cli
Describe the issue
If graph deploy fails, it should exit with 1, so that the error can be reported or handled in a CI context (e.g. Github workflow), but it exits with 0 currently.
Reproduction
https://github.com/kleros/kleros-v2/actions/runs/7578013905/job/20639879509#step:13:120
Steps to Reproduce the Bug or Issue

Run graph deploy --studio -l existingLabel
Run echo $?

Expected behavior
If graph deploy fails, it should exit with 1, so that the error can be reported or handled in a CI context (e.g. Github workflow)
Screenshots or Videos
No response
Platform

OS: macOS
NodeJS: 16.20.2
@graphprotocol/graph-cli version(s): [0.64.1]

Subgraph Manifest
No response
Subgraph GraphQL Schema
No response
Additional context
I have only looked at graph deploy with Studio, but other products or sub-commands might be affected too.

One more report on Discord https://discord.com/channels/438038660412342282/438070183794573313/1207010623297290320

One more report on Discord https://discord.com/channels/438038660412342282/438070183794573313/1207010623297290320

This was me, so I'm adding more information here.
Calling to
$ yarn graph deploy --studio <slug> --deploy-key <key> --version-label <label>

using a <label> that already exists will fail the deploy process
Build completed: QmcyuNXAXyBsFyk2buzzDxLUc8NTsws5SYAxRbGX8hDPmK

✖ Failed to deploy to Graph node https://api.studio.thegraph.com/deploy/: Version label already exists.
UNCAUGHT EXCEPTION: Error: EEXIT: 1
Done in 17.07s.

but when I call to check the exit code of that, it shows 0
$ echo $?
0

Currently I'm using
$ yarn -v
1.22.19

with the graph packages in my package.json as
    "@graphprotocol/graph-cli": "^0.61.0",
    "@graphprotocol/graph-ts": "0.30.0",

+1 This is also affecting our CI/CD pipelines. Did you find a workaround?

+1 same issue here

+1, same issue


[Feature Request] Backoff period after first alert is send

It would be nice if we could setup a backoff period after the first alert is send in order to avoid sending the same alert multiple times and flooding the recipient with messages.
example
       stream
           .groupBy('service')
       |alert()
           ...
           .backoff(10m)

which would backoff the alert for the next 10 minutes.

Hi @mantzas , does the .stateChangesOnly node work for your use case? (https://docs.influxdata.com/kapacitor/v1.2/nodes/alert_node/#statechangesonly)

hi @goller almost. it would be nice to have another alert after x minutes just as a reminder. The main purpose is not to silence the alert but to throttle the alerts so that it does not flood the recipients!

@mantzas you can, .stateChangesOnly(15m) will resend after 15 minutes if it's still in an alert state.

perfect. thanks for that.


awslabs/tough is incompatible with new root-signing

#1164 starts using "ecdsa" as the keytype as the specification now suggests.
This seems to break awslabs/tough: https://github.com/awslabs/tough/issues/754
Failed to load repository: Failed to parse root metadata: Invalid SubjectPublicKeyInfo document at line 18 column 4
@kommendorkapten plans to disable the tough test for now: let's re-enable when possible

This is now done.


Issue 267/loading documents spinner

This PR closes #267
This PR:

Pulls the "loading clients" feature out of Clients and into it's own component Loading in the Notifications folder.
Imports that into Clients to display when the client table is loading
Imports that into Forms to display it when the document table is loading

To successfully use Loading:
Make sure to use the loadingItem prop with a string.   i.e. <Loading loadingItem="clients" />

I now finally see how I requested PRs into Master instead of Development all those previous times.
I always change it to Development branch first, but when I finally type everything up and go to the bottom, the button says "Draft PR".  When I switch to "PR" to create an actual real PR, it changes the branch to Master.

Seems like a simple enough PR. I like it.
The one thing is I'd maybe rename the file from Loading (and therefore the import) to something more specific, like LoadingAnimation. That way its purpose is clearer.
Also, while looking through more MUI options, I found what they call a skeleton loading animation:
https://mui.com/material-ui/react-skeleton/
The linear animation looks good and I have no problem keeping that. I just wonder if a skeleton would be better suited to large tables of data. It's also customizable, so we can change the size and specifics if we want to (e.g. have it use one animation per row of existing data).
I think it would suit both clients and documents, so there wouldn't be a need to create a separate animation.


[Docs] How to using iPhone or Mac's neural engine on chip?

Just wonder, does there APIs or capabilities make tvm support call iPhone's neural engine as CoreML does?

No, you need to use coreml for that.

@masahi is that able to using CoreML with tvm?


Usage does not auto indent newlines

my urfave/cli version is
v2.3.0

Checklist

[x] Are you running the latest v2 release? The list of releases is here.
[x] Did you check the manual for your release? The v2 manual is here
[x] Did you perform a search about this problem? Here's the Github guide about searching.

Dependency Management

[x] My project is using go modules.
[ ] My project is using vendoring.
[ ] My project is automatically downloading the latest version.
[ ] I am unsure of what my dependency management setup is.

Describe the bug
Usage does not auto indent newlines in usage.
Expected output:
$ ./foo
NAME:
   foo - sample usage

USAGE:
   foo [global options] command [command options] [arguments...]

COMMANDS:
   run  this is a long help output for the run command, long usage
        output, long usage output, long usage output, long usage output  <-- notice the indent
   help, h  Shows a list of commands or help for one command

GLOBAL OPTIONS:
   --help, -h  show help (default: false)

Actual output:
$ ./foo
NAME:
   foo - sample usage

USAGE:
   foo [global options] command [command options] [arguments...]

COMMANDS:
   run  this is a long help output for the run command, long usage
output, long usage output, long usage output, long usage output  <-- not auto indenting
   help, h  Shows a list of commands or help for one command

GLOBAL OPTIONS:
   --help, -h  show help (default: false)

To reproduce

Example

package main

import (
	"fmt"
	"os"

	"github.com/urfave/cli/v2"
)

func main() {
	app := &cli.App{
		Usage: "sample usage",
		Commands: []*cli.Command{
			{
				Name:  "run",
				Usage: "this is a long help output for the run command, long usage\noutput, long usage output, long usage output, long usage output",
				Action: func(c *cli.Context) error {
					return fmt.Errorf("nothing")
				},
			},
		},
	}

	err := app.Run(os.Args)
	if err != nil {
		fmt.Fprintf(os.Stdout, "%s: %s\n", os.Args[0], err)
		os.Exit(1)
	}
}

Additional context
Related to: #1278
Run go version and paste its output here
go version go1.16.5 linux/amd64

Run go env and paste its output here
not relevant

@WestleyK I have a new PR #1460 which fixes this. Can you test and see ?

PR #1460 fixes this.


options of including Materialize

There are several options on how to include Materialize in this bridge.
a) which part should import the library?

the bridge
the catalog (and therefore the user's application)

I'd really like the catalog itself to show a circular preloader instead of the font-awesome spinner so the catalog would still use Materialize css because this obviously has to be available before Aurelia starts. What do you think?
b) there are some variations available..

jspm install materialize=github:github:Dogfalo/materialize
jspm install materialize=npm:materialize-css
use a vendor solution like the kendo-ui bridge does

Where...

Pulls the bundled and minified version of Materialize into the target and no sources. We cannot alter f.i. SASS files or just import single features with this variant (although I don't believe Materialize is made for "feaure picking" at all tbh). On the other hand this solves a few issues I've come across while experimenting with Materialize. For example toasts needing both a global Materialize variable, hammerjs and velocityjs to be present. Keep in mind that Materialize is not (yet?) built around any module system.

Pulls Materialize source files and dist builds. We can read and process SASS files and pick features to import. The SASS files could f.i. be used to create primary and accent colors which Materialize has no notion of, but Material does. I've already been missing that with the first Waves sample. Also we could perhaps use the "original" Waves which includes disposing functionality.
But, as stated above, there are some issues to this approach.

I'm really not sure if this makes any sense at all since there are no real "editions" of Materialize and the variations noted above only make a difference from a bridge developer's point of view.

Btw: I've read the Git Commit Guidelines but this issue would fit neither (chore) or (doc) while it also does not discuss a feature I think (or does it?). Which <type> would fit here?

Having written that (still not finished btw) I realize that perhaps the NPM version is best because it delivers source files and still contains the bundled version.
Please note though that this NPM version had issues in the past, like not containing anything useful at all (most files where missing). Of course, that need not happen again, but I'm mentioning it anyway.

@Thanood - I will address only a subset of the issues you raised here, leaving space to @JeroenVinke as well

which part should import the library?

Both - each side with a slightly different point of view. The bridge of course needs to do it as its only task is to convert third party controls into Aurelia components. The catalog has to import "parts" of the library (like CSS files, as the catalog ought to be in full control of how to render its UI. I will clearly indicate it once I get the chance to add all the code for the skeleton bridge task that evades me this whole day :-)

I'd really like the catalog itself to show a circular preloader instead of the font-awesome spinner so the catalog would still use Materialize css because this obviously has to be available before Aurelia starts. What do you think?

This is a clear problem of bootstraping the project and the reason for my insistence to start with the "green box" shown below

that has only the UI components that are used in Aurelia Skeleton Navigation app (where the Aurelia team provided enough to create any basic Aurelia app without having to have a bridge to something else). Once you have this working, you can add the "circular preloader" as the very first Aurelia Materialize component -- and only then use it in the catalog. Any other way you will remain wrapped yourself in an unneeded battle with dependencies - and that is the kind of wrapping that is not a part of the bridge development :-)
leaving the rest of my comment for later ...

Once you have this working, you can add the "circular preloader" as the very first Aurelia Materialize component -- and only then use it in the catalog.

I could do that. But then, as an Aurelia component, it would not be displayed since the point of that "spinner" animation is to show the user something before Aurelia takes control. So this would have to be added before Aurelia takes control.
Of course I could do both: add an Aurelia Materialize component but then not use it (as it makes no sense at that time) but use simple css/html and no components.
But it's all not that important. If it doesn't fit in, I'll continue to use the font-awesome spinner. Just thought it'd be nice as a kind of "corporate identity" to show some material even before Aurelia is started.

But it's all not that important.

I disagree - as I want you to not be constraint by my possible lack of knowledge about Materialize. I am referring to the case where we have not decided what to do "styles only" section of Materialize :

This is really the subject of our current discussion, which (subject) was settled for Aurelia Navigation Skeleton, the grand-father of all stuff in the "orange box" by including the styles needed to render the <i class="fa fa-spinner fa-spin"></i>
into the styles.css file.
Thus, your desire to use the circular preloader is a perfect sign of your dedication to ensure the native look and feel, and you should solve it to add the needed css data (which you commented out in the index.html file, into styles.css
Making sense?

Making sense?

I'm not sure since the Aurelia Navigation Skeleton also loads font-awesome before anything else. That file contains the base css for the classes defined in styles.css including f.i. the fa-spin keyframes.
On the other hand, when you say both parts should import Materialize, then this is not a problem at all because it wll be there anyway.

I'm not sure since the Aurelia Navigation Skeleton also loads font-awesome before anything else

Let me assure you then :-) -- check this line where font-awesome gets loaded.

I'm trying to use aurelia materialize-css using the package npm:materialize-css, because I want to use the sass files.
I added this in config.js to use my sass variables:
packages: {
"materialize-css": {
"main": "dist/js/materialize.js",
"map": {
"../css/materialize.css": "../../../src/styles/styles.scss"
}
}
},
this works using gulp watch, but with gulp export it fails:
Error: CSS Plugin:
Ignoring local @import of "dist/src/styles/styles.scss" as resource is missing.
at file:///C:/Users/marvin/Documents/Sistema Restaurant/restaurant-frontend/jspm_packages/github/systemjs/plugin-css@0.1.20/css-builder.js:76:13

I already added 'materialize-css' to bundles file.

Any ideas?

@mandia1204 I'm moving this question to a new issue: https://github.com/aurelia-ui-toolkits/aurelia-materialize-bridge/issues/403


Nancy selfhosting error on ubuntu with Mono

I am using Nancyfx as my restful api server. On windows I use Nancyfx in ASP.NET HOSTING MODE, all works fine on my App. But when I try to use a linux server, the app crushes as I use this code below
HttpClient client = new HttpClient();
HttpResponseMessage response = await client.GetAsync("http://ipaddress:1234/api/info");
response.EnsureSuccessStatusCode();
string responseBody = await response.Content.ReadAsStringAsync();
the exception here as below
An exception of type 'System.Net.Http.HttpRequestException' occurred in System.Net.Http.dll but was not handled in user code
Additional information: 500 (Internal Server Error) but when I use chrome to recieve the content, seems to work fine. Is there something wrong with my code or Nancyfx selfhosting mode?
Thanks!

Depending on your applications, this might be a problem of HttpClient-Headers.
I had the same problem and it could be fixed by explicitely adding a header for the "Accept"-header of your request.
Talking of code, you may try adding something like
client.DefaultRequestHeaders.Accept.TryParseAdd("application/json");
to your HttpClient request
Hopefully this helps
P.S.: There are other ways of adding Request Headers, but I think this is the best way doing it system-independently (tested with UWP, WP, Android, iOS and ASP.Net)

thanks , this problem solved by adding the header. thanks for the reply


Discover ZfsDatasets across HA pairs instead of double import during device scan

DESCRIPTION
With ZFS MMP enabled, force import on a ZfsPool that is already import on another host fails. The implication of that in IML is that ZfsDatasets created on a ZfsPool imported on one host will not be reported by agents running on other hosts.
This is significant because in order for a secondary mount to be identified as available for failover of a target, the agent on the secondary host of an HA pair must report that it can see the same volume (the newly created ZfsDataset created on the primary host when formatting the lustre target). With MMP enabled the force read-only import during device scan that enabled the capacity to peek the newly created ZfsDataset is no longer possible.
SOLUTION
After investigation and experimentation the conclusion is that because the resource manager processes storage resources dynamically, the required VolumeNodes (which identify a mapping between a host and a shared volume) are cleared as has the relevant storage resource is not reported in an agent scan.
As experience has shown, any shortcuts or hacks in the resource manager tend to have knock-on effects and generally are not with it, introducing VolumeNodes that exist without storage resources or not removing VolumeNodes in certain circumstances both result in undesired effects where objects are not cleaned up as expected.
The solution should centre around the ability to infer existence of a ZfsDataset (dataset) through its discovery on a HA pair. Meaning that the secondary host can assume a dataset exists even if the host cannot 'see' it (the host can see the ZfsPool (pool), but not the new dataset, because the pool cannot be imported) because the primary host is reporting its existence.
The resource manager could be modified to make these assumptions, that modification be a significant piece of work.
One plausible solution which may not take a large amount of effort involves adding state to the agent (not ideal) to enable it to remember datasets that it has 'seen'. This state could be stored persistently in the agent 'config store'. The result of this would be that as long as the pool had been previously imported and dataset discovered, when a pool is subsequently not importable, the datasets would continue to be reported by the agent and the relevant VolumeNodes would continue to exist. This state would be updated the next time the pool is imported.
A variation on that theme would be to share the details of datasets existing on a given pool between HA pairs by writing this information to a distributed keyvalue store (https://github.com/coreos/etcd) which can be used to store arbitrary data. Every time a pool is imported on one of the HA pair hosts, the datasets that exist are written to the store (accessible by every host in the HA cluster). When a pool is unavailable for import (during device scan by agent's linux device plugin), the agent reports the datasets detailed in the store.
This last variation is the recommended implementation because there are no manager changes (and therefore less undesired consequences) and only small agent changes. The distributed store will be important for future enhancements.
An interim implementation will use local storage to store last known state of zpools/datasets on an agent to enable progress whilst a suitable distributed keyvalue store is decided upon. This interim solution will work as long as pools can be exported temporarily after being formatted (datasets being created) to enable the secondary host of the HA pair to be a to 'peek' at the new dataset and therefore remember it.

zfs-etcd-schema.json.txt
suggested schema for zfs data to be held in the distributed store

still seeing pools imported between tests (which seems to be causing VolumeNodes intermittently not being created where expected and as a consequence, test failures)

now experiencing issue where node is not responding when issued with zfs/zpool commands at some point during test runs

now experiencing issue where node is not responding

Is the host oops/panicing perhaps?
You'd be able to determine that if we collected console output at test failure but we (a) don't collect console output at all and (b) don't collect anything (i.e. chroma-diagnostics/sosreport) at test failure.
Both of those are very worthy tickets in and of themselves.
For now, you can probably _fetch_help() on the failure and examine the state when it triggers.  Let me know if you need any help examining the state when it triggers.

cleaned up and landed as part of #314


Fixed teardown function of private-registry-test

Failing test: https://github.com/kubewarden/kubewarden-controller/actions/runs/4877745524/jobs/8702731582
Fixed run: https://github.com/kravciak/kubewarden-controller/actions/runs/4885861780/jobs/8720531929
Bugs reported:

https://github.com/kubewarden/kubewarden-controller/issues/443

https://github.com/kubewarden/policy-server/issues/459

Updated test module versions

Added readme about cosign required by some tests

I changed the visibility of psp-user-group:v0.1.1 to private, will delete it next week if nothing brakes.


[NETBEANS-1074] Module Review php.atoum

Change the license header of Bundle.properties
Exclude AtoumTest.php from rat because it is a template file
Fix a unit test because the license headers are changed, then the line counts are changed

In general looks good - I would however ask you not to add rat excludes, but use the "licenseinfo.xml" facitlity to declare the license for the file without modifying it. Please have a look at:
xsl/licenseinfo.xml
I think that should cover your usecase. The benefit is, that in the end the file can be traced to the license info and the rat exclusion is modular.

@matthiasblaesing Thank you for it :) I'll fix it. Should we also add image files there?

@junichi11 that would be great - it was for example done in: /languages/licenseinfo.xml

@matthiasblaesing OK, I see. I'll also do that :) Thanks!

@matthiasblaesing I've added the below to 2nd Donation: List of Modules to Review
"Please add a licenseinfo.xml instead of excluding from the rat-exclusion.txt. (e.g. see xsl/licenseinfo.xml)"

Nice conversation to understand the relicensing process :+1:


Preview button start video on target image bug with multi-target.

With multi-target, positioning the preview start video button above the target image, it is not possible to press other clickable objects but only the start video button.
Specifically, the DOM object begins with the a-mouse-cursor-hover class even if the cursor is not positioned on a clickable object and remains so until a second target is framed.
In the attached photo you can see the mouse not positioned on a clickable object, but clicking triggers the event anyway, without the possibility of clicking on other clickable objects.

Do you have a testable app online?

I have this issue too, I've worked around it in the short term but it would be good to understand how to fix it

I've been looking at this today and the behaviour is very strange.

sometimes I can interact with preview button but this is not 100% and is very hit and miss
I think this may be more of an issue with conflicts with raycasting

stale. feel free to re-open if there is new insight.


Remember me

See: https://symfony.com/doc/current/security/remember_me.html
Not integrated now, possibly something for the future

Closed by PR #46


Update training documentation to reflect changes to the Data Catalog (Part 2)

Description

We just released a new dataset system mentioned here: https://kedro.readthedocs.io/en/latest/04_user_guide/04_data_catalog.html#using-the-data-catalog-with-the-yaml-api
This means that we've done away with things like CSVLocalDataSet and CSVS3Dataset and just have one CSVDataSet as an example
This is a documentation update to reflect those changes

Context

We use these docs for client training and meet-ups, you would be helping us stay up to date

Possible Implementation

These changes are already implemented in the main documentation here so you can search for the section headers there
You need to look out for datasets that we're deprecating in future releases all mentioned here and replace them with the new ones, more information about these datasets can be found here and here
This is Part 2 of the change and you will be modifying these files:

https://github.com/quantumblacklabs/kedro-examples/blob/master/kedro-training/docs/09_versioning.md
https://github.com/quantumblacklabs/kedro-examples/blob/master/kedro-training/docs/11_configuration.md
https://github.com/quantumblacklabs/kedro-examples/blob/master/kedro-training/docs/12_transcoding.md
https://github.com/quantumblacklabs/kedro-examples/blob/master/kedro-training/docs/13_custom-datasets.md

Hi, can I work on this?

This was already implemented in https://github.com/quantumblacklabs/kedro-examples/pull/35.


Added response info middleware

What changed?
Added outbound middleware for filling rpc.ResponseInfo structure.

Why?
This can be useful for a general way to obtain information about the response before it gets deserialized and returned to the client. One such example - obtaining serialized size of the message.
Intended usage example:
ctx, responseInfo := rpc.ContextWithResponseInfo(ctx)
resp, err := client.GetReplicationMessages(ctx, request, opts...)
messageSize := responseInfo.Size
...

How did you test it?
Added unit test, verified locally with print statement that message size can be obtained that way.

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build eac1cd08-b8cd-41ea-97f3-d5ccfb24b00d

15 of 15 (100.0%)  changed or added relevant lines in 2 files are covered.
87 unchanged lines in 10 files lost coverage.
Overall coverage decreased (-0.03%) to 56.491%

Files with Coverage Reduction
New Missed Lines
%

common/types/mapper/thrift/shared.go
2
64.65%

common/util.go
2
52.47%

service/matching/taskListManager.go
2
74.09%

common/cache/lru.go
3
90.73%

common/persistence/dataManagerInterfaces.go
3
60.91%

common/task/fifoTaskScheduler.go
3
84.54%

service/history/queue/timer_queue_processor_base.go
4
78.6%

common/persistence/nosql/nosqlplugin/cassandra/workflow.go
6
49.32%

common/persistence/nosql/nosqlplugin/cassandra/workflowUtils.go
23
74.98%

common/persistence/nosql/nosqlplugin/cassandra/workflowParsingUtils.go
39
74.68%

Totals

Change from base Build 17dc8fa9-9bea-491d-a466-a062d1821eb5:
-0.03%

Covered Lines:
80341

Relevant Lines:
142218

💛  - Coveralls


'todo' const removal from reducers/todos.js

Hi there,
What I am proposing is removal of the todo  const and usage of a simpler (IMHO) and still readable and understandable code. Purpose of this change is to make learning Redux and how to connect Redux to React a bit easier.
While learning awesome Redux and expecially how to connect Redux with React app, I have found that  const todo was one of the confusing pieces of code - at least it was for me. It looks exactly like a reducer - but it is not. It is helper function which has two arguments: state  and action (just like reducer), and switch statement also sugest to me that it is a reducer (I was just looking for the code patterns at the time of learning). I was thinking (at the time of learning) that todo is reducer which are used in another reducer - todos. Yep, it's crazy - I know..
Best,
Danko

Yeah, that's a little bit easier to read. It would make sense if todo was exported, but it's an entirely internal concern. Thanks!

@timdorr What do you think about updating a docs with a same code as in this commit? If approved, I could do it as well. Thanks.

Sure, just edit the markdown file. As simple as that.


Lens UI no space for the used-by pie

Describe the Bug
Using the Lens UI, dependency view, when clicking on a service, I can see on the right I can see information about used-by and using services.
When I have a lot (40) services using one of my services, the tags take a lot of space and there is no space for the pie chart which gets crunched and almost invisible.
Steps to Reproduce
Have a crazy architecture, lots of services connected to one particular service, go to dependencies, click on the service which is used by everyone.
Expected Behaviour
I would expect the pie charts to have the same size and not depend on the size taken by the tags.

Thank you for your feedback!! I'll take a look.


Add pre-commit hooks

Add suitable TF pre-commit hooks.

@t-young31 Sort of done already?

in this repo? not really. can push some here

Yes please. Whatever you would consider good practice for TF.


Pagination

Since we should return a plain array with the items, where would we include the pagination metadata ?
The practice I know is do something like this:
{ "items" :
    [
        {
            "like_count": 0,
            "id": "string",
            "media_type": "comment",
            "owner": "string"
        },
        {
            "like_count": 0,
            "id": "string",
            "media_type": "comment",
            "owner": "string"
        },
        {
            "like_count": 0,
            "id": "string",
            "media_type": "comment",
            "owner": "string"
        }
    ],
  "meta":
    {
        "current_page": 1,
        "next_page": null,
        "total_pages": 1,
        "prev_page": null
    }
}

I'll have to check with @elbrujohalcon  how we usually do this, but I think it depends on the project. Sometimes we use page numbers, sometimes we use a cursor. As far as I know, the paginated resources are the only ones that should get enveloped items, as you wrote above @thiagogabriel . I'm gonna check if there's no item on the momentary index for this and, if there isn't, add it .

Pagination can be in the headers, right @volbap ?

In any case, we can totally add pagination as an exception to that rule, if needed.

Yeah, I would use the link headers as explained here. It seems to be a fair solution.
Also, let's try to avoid envelopes as well.

What Jayme currently supports is a non-cursor based pagination, using these headers:
X-Total, X-Per-Page, X-Page
This kind of pagination, however, may not be quite useful for certain common scenarios, like a scroll feed. So, I would not make it a standard.


IoU Prediction Post Processing

Hi, I noticed in the code you have a post-processing step for the IoU predictions:
iou_preds = (iou_preds.squeeze() + 1) * 0.5

I'm wondering why do we add 1 and then multiple by 0.5. I did not notice this step in the paper. Did you find this to work better than using something like a sigmoid function? Thanks.
Reference:
https://github.com/Vegeta2020/CIA-SSD/blob/97ec5dbf4075881cf932bce1558732d0cde82e3f/det3d/models/bbox_heads/mg_head_v4_release.py#L722

Ah I understand now sorry. It's because the prediction is always between [-1, 1]

@adpon  How to ensure the prediction is always between [-1, 1]? I saw no activation after IoU branch to regulate the range of output.


feat: add ETH-FOX V3 staking opportunity

this adds the ETH-FOX V3 staking opportunity but it currently points to the v2 opportunity as v3 does not exist yet and this was suggested in the issue concerning this pr (#90)
NOTE: FEATURE_FLAGS.ethFoxStakingV3 in src/lib/constants needs to be set to true to test this out

I have now set the contact address to 0xc770eefad204b5180df6a14ee197d99d808ee52d which is the ETH-FOX V3 address according to: https://canary.discord.com/channels/554694662431178782/937810323853934693/938201099259744326

I have now set the contact address to 0xc770eefad204b5180df6a14ee197d99d808ee52d which is the ETH-FOX V3 address according to: https://canary.discord.com/channels/554694662431178782/937810323853934693/938201099259744326

I believe it should be 0x212ebf9fd3c10f371557b08e993eaab385c3932b (0xean just replied to your message 😊)

I have now set the contact address to 0xc770eefad204b5180df6a14ee197d99d808ee52d which is the ETH-FOX V3 address according to: https://canary.discord.com/channels/554694662431178782/937810323853934693/938201099259744326

I believe it should be 0x212ebf9fd3c10f371557b08e993eaab385c3932b (0xean just replied to your message 😊)

updated!

Nice, looks good.
We'll likely need to update the periodFinish, though I don't know what to. I'd wager @0xean knows.

Nice, looks good.
We'll likely need to update the periodFinish, though I don't know what to. I'd wager @0xean knows.

Yep, that's why I marked it as TODO for now

We should get this directly from the contract.
https://etherscan.io/address/0x212ebf9fd3c10f371557b08e993eaab385c3932b#readContract
Currently, its at 0 but once we update the contract to start the rewards, it should be set to the correct unix timestamp.

so what should be done now?

for now we can modify it to be 4.5 Months from 2/22, but eventually we can update it to be more accurate.
1.) On February 22nd at 9am MST (48 hours before the second round of FOX/ETH liquidity mining ends), deploy a third round of liquidity mining rewards for the Uniswap v2 FOX/ETH Pool of 13.5M FOX rewarded over 4.5 months (targeting 75% or higher APR).

@0xean I have set the periodFinish to<PHONE_NUMBER> which is roughly 4.5 months from 2/22

1655268529 is sometime on Jun 15.
I think we are a month off or so.
Lets take the timestamp for Feb 22 2022 at 9AM MT =<PHONE_NUMBER>
135 DAYS in the current duration of the contracts rewards (~4.5 months)
135 Days * 24 Hours / Day * 60 Min / Hours * 60 Sec / min = 11664000
1645545600 + 11664000 =<PHONE_NUMBER> (Jul 7 2022)
Lets go with<PHONE_NUMBER>

@0xean ok I have set<PHONE_NUMBER> as periodFinish


Allow changing responsive prop after mount

Helpful when you render server side with something like react-snap. It will create the slider with a certain breakpoint configuration which needs to be hydrated (react 16 won't patch diff). This way we can always render in a predetermined breakpoint setting and apply the rest on the client on a second render pass.

ping @akiran

not stale

@maddhruv Yes, this issue need to addressed.


add encrypteddb GetSecretBoxKeyWithUID

For patching this up https://github.com/keybase/client/pull/20754#discussion_r342157134
Also get rid of unused secret UIs.
Not used yet. The plan is to merge this and then add a call chat/stora ge.GetSecretBoxKeyWithUID by journeycard to https://github.com/keybase/client/pull/20754

this seems like a flake. Passes locally.
panic: Log in goroutine after TestDeprovision has completed```


tfrobot: no need to validate resources during cancellation

Only MNEMONIC, NETWORK and node groups' names (project names) are validated now


Web folder update and required configuration script updates

Short Description:

Updates web to newest commits.
Updates config files for windows and posix based os to handle new FriendlyName/DisplayName on map.

👍


In Memory Contract Store, related #230

Initial implementation of the In-Memory Contract Store.
Branch is based on #241. Please review/merge this PR first.
Dominik Pinsel<EMAIL_ADDRESS>Daimler TSS GmbH, legal info/Impressum

Can we hold off on this until we have aligned on #239 and a technical design for the message flow? Otherwise, we are risking doing this piecemeal.

Can we hold off on this until we have aligned on #239 and a technical design for the message flow? Otherwise, we are risking doing this piecemeal.

I'm currently working on the data transfer in parallel, and the Contract and the corresponding ContractStore are the two things that sit between the negotiation and the data transfer.
I can merge from this branch into my transfer branch and work like this for now, but it would be nice if this branch isn't on hold for more than a week :-)

@DominikPinsel can we close this now that the ContractNegotiationStore is in place which handles this?

@DominikPinsel can we close this now that the ContractNegotiationStore is in place which handles this?

yea sure, I will close this PR and the corresponding issue.


mobile app example

should use offline-mode plugin
should be compatible with mobile specific DB (sqlite for instance)

I think these examples will greatly help. I'm trying to figure out how I can use mesh with Aurelia but I'm not sure how to yet...

Are you trying to use mongoose on the front-end?


add testing start

has a todo i.e. find why merge dropped out of sequence

OK, history is rewritten to be linear. Everyone needs to pull with rebase, then please also approve this


Make nonrecursive_access_policies the only thing

We'll also need to update the documentation to reflect this, but
updating it now would change the live site too early, right?

but updating it now would change the live site too early, right?

Yes, let's not do it just now. You can create a separate issue so that we don't forget.


Databases UI Changes

A potential reason why the graphql tests failed
$ NODE_ENV=test jest --maxWorkers=8 --coverage --maxWorkers=1

yarn run v1.12.3

$ NODE_ENV=test jest --maxWorkers=8 --coverage --maxWorkers=1

2020-06-15 14:29:35,844 - error: Image with id=c9a0f64177ef0b6ec23b9a0af796e9ef does not exist

 PASS  src/modules/webServer/storageServer.test.ts (8.091s)

 FAIL  src/modules/user/controller.spec.ts

  ● Test suite failed to run

    ENOMEM: not enough memory, read

The webapp tests also failed due to mismatches in snapshots.

The webapp tests also failed due to mismatches in snapshots.

Yes, the diagnostic snapshots still don't want to work. Don't think they are related to these changes though.

@richardgoater

Yes, the diagnostic snapshots still don't want to work. Don't think they are related to these changes though.

Your fix fixed master, but it hasn't made its way into this branch yet.

Just seen that the "Colocalised annotations" and "Other adducts" tabs are broken, will need some more work tomorrow.


Can't Login, Error 401

Newest version and last 2022 version won't let me login. Keeps giving me a 401 error and saying my "Username is required, Password is required" even though I've put them in. Tried my Current username and my old username, Email, New password old password. Nothing is working.

are you able to login via vrchat.com?

I am able to login to the site yes. So sorry for the super delayed reponse. I was attending a convention.
I'm able to login normally on the site just not the vrcx program.

it could be caused by your name if it has odd symbols, I remember that happening in the past not sure if its been fixed.

It doesn't. I wish that were the case lol
My name is all standard letters. Not even any numbers

Then im unsure.

Ah okie.

Does your password have any special characters in it(this feels so sketchy to ask, lol)? I'm not sure if VRCX escapes those properly, but mine does and I don't have issues so I'm not sure.

Does your password have any special characters in it(this feels so sketchy to ask, lol)? I'm not sure if VRCX escapes those properly, but mine does and I don't have issues so I'm not sure.

i remember someone having issues with this too its why i brought it up with the username aswell

Just took a look and the login func does handle special chars properly. Did this only start happening to you after you changed your password? You could try to rename your VRCX.sqlite3 to something else and restart to see if something that's been cached is messing up your login somehow.
The default location is in %AppData%\VRCX

Okie dokie. I'll do the fresh install. A fresh reboot would help as well correct?

You should be fine to just move the appdata folder while the program is closed and then re-opening VRCX. This'll reset everything back to defaults and help narrow down the issue.

Okie dokie. I'll letcha know what happens

What do I do about tis file?

uhh you dont have to reinstall vrcx, only move the folder listed above that will clear vrcxs settings and cache and then try to login again to see if your database has been corrupted.

Ah alright. I did move and rename the folder listed.
Same results

uh huh, can you open up the developer console and check the console for any errors? can open it by pressing ctrl shift I, or right click the vrcx icon in the tray and select devtools.

uh huh, that looks like the SQL library logic is completly failing so its unable to continue

ah
What do I do?

well what it looks like is vrcx is failing to either insert data into the database or failing to create the database, can you verify that the vrcx folder located int %appdata%\..\VRCX is gone

lemme check
I thought I moved it. Might not

make sure vrcx is completly closed from the tray icon before doing it or else it will not work

Okie

Okie I made a new fresh installation on my secondary drive. It said I had 0kb left on my main which is false sine I checked earlier and I had 7gb so ye. I'll see if this installation runs properly

then im not sure seems like an unusual issue with SQL Lite

Ah
What can I do on my end?
I'm not educated in code or anything ^^"
I only know just enough to nab pics and fonts from sites through inspect element ^^"

you could attempt to run older versions of vrcx, or atleast the nightly builds, vrcx can assist with auto installing them from the top left download icon.

The SQLite error might be unrelated and perhaps it was a buggy call trying to insert something before the new, fresh database was fully initialized.
Did this only start happening after you changed your password?
If so, the only thing I can think of is VRCX not being able to handle a specific character in your password. Try changing your password to something simple with standard character and see if you're able to log in. Then change it back to your normal pass and try again.
If it does seem to be a problem with your pass, there's probably a certain special character in it causing issues with the encoding.

I tried running the last version from 2022 and had the same issue.
I changed my password sometime late last year but I hadn't used vrcx since June so I'm not sure.
My password is all standard, Capitals, no special symbols, and numbers

The Latest change I've made is my username. But It's also just one word all caps, no specials or numbers.
I can't change it for another 77 days tho

I'm quite unsure why you're having this issue, unfortunately. Somebody else might be able to help.

Somthing to see if it makes change,

Try nightly build
use e-mail address to login
check your security software not blocking app

Okie, which versions are the nightly builds?

at the top left of vrcx click the download icon and then select nightly

then click download

Ahh okie dokie, Ill see if they work

They keep failing on installation

failure reason my be related to lack of space on storage volume, clear some space windows tends to run like garbage with less than 15gb remaining

I installed it to my secondary HDD drive which has over 100gb left on it

try fresh install using exe. maybe something broken. https://github.com/vrcx-team/VRCX/releases

I already did a couple times ^^"

Uninstall once, restart machine, install again. how would this sounds?
and are you able to login on vrchat.com using same credidential??

I shall attempt this

No change
I've no clue what to do at this point. It used to work fine in june but not now. I couldn't even login on my older version from june cuz it gave me the same error.

Same results. Sorry for the delay. I thought I already updated the thread.
Not sure why it doesn't work but oh well

I'm having this problem too

Did you try all of the above troubleshooting steps? can you list the things you've tried and send the full error in the console you're getting.


Ecto.Changeset.convert_params/1 bugfix or behaviour change

If the keys in the params are mixed atoms and strings, validations fail and the params with an atom key are marked as missing (if required). For example:
defmodule MyModule do
  def changeset(instance, params \\ nil) do
    params |> cast(instance, ~w(f1 f2))
  end
end

If I call changeset/2 like this:
MyModule.changeset(%MyModule{}, %{:f1 => "v1", "f2" => "v2"})

it will fail saying that one of the fields is required but not present. I added a failing test in the first commit, and the fix in the second commit.
I'm not 300% sure this is a real bug (I kind of smelled something strange when I saw throw/catch in the current implementation of Ecto.Changeset.convert_params/1), but I figured that a bug report with a PR for an eventual fix (if this turns out to be a bug) is better than a bug report alone :smile:

I don't think we should support mixed. The previous behaviour was on purpose and the throw/catch was to avoid traversing everything if we had binary keys. Maybe we can raise an error if you have mixed keys, instead of ignoring the atom ones, but I think supporting both puts us in a road I don't really think we should go.

@josevalim I strongly agree on the fact that we shouldn't become like Ruby/Rails, where the distinction between atoms (well, symbols) and strings is practically none (unless it bites you :imp:).
That said, I think the current behaviour is slightly confusing since

you expect strings to work (since parameters usually contain string keys since they come from the URL query)
you also expect atoms to work, because you manipulate pretty much everything else with atoms in Ecto (schema fields, struct keys and so on); this is even more true since with only atom keys it works

Furthermore, the arrays of fields you pass to the cast/4 function can pretty much be everything (as far as I managed to test): only atoms, only strings, atoms and strings. They're all converted to atoms. Probably that's another source of confusion: Ecto lets me use cast/4 with atoms/string/atoms+strings but then it "fails" when I mix atoms and strings in the parameters.
If we (well, you :smile:) decide to keep the current behaviour, I'd be glad to at least raise some kind of error so that users know what to do to fix it.
What do you think?

Fixed in master. I am just quickly raising!

Ok, that's progress :smile: Thanks!


Run unattended-upgrades on the dev VM as part of the puppet run

When we run govuk_puppet on a dev VM we expect that it will update all
the managed software on the machine, however for package updates it
relies on unattended-upgrades to actually perform the updates.  These
are set to run at some small hour in the morning, but most dev VMs are
not powered on at this time, so the updates don't actually run.
We noticed this when postgres 9.3.14 came out which introduced a breaking
change to the \connect line in sql dumps.  The servers were upgraded
but most dev VMs weren't and so replicating data generated on the servers
running 9.3.14 would silently fail when run on the dev VMs running 9.3.12.
See https://github.gds/gds/development/pull/334 for some extra context on the change.
I fully expect that this isn't the correct way to invoke the run, or that some other approach would be better.  Please chip in with better ideas.

I think this is a decent approach as it'll always be caught in a puppet run. You might want to also add a notify => 'Updating packages' or something because otherwise it'll silently sit there for a while and people will start to get twitchy.

Forgot to pay attention to the linter.  Thanks for pointing it out.

I'm getting this error from the notify:
Error: Parameter notify failed on Exec[unattended_upgrades]: No title provided and "Updating apt packages" is not a valid resource reference at /var/govuk/govuk-puppet/modules/govuk/manifests/node/s_development.pp:158

I tried this instead:
  exec { 'unattended_upgrades':
    command => '/usr/bin/unattended-upgrades',
    require => Notify['unattended_upgrades'],
  }

  notify { 'unattended_upgrades':
    message  => 'Updating apt packages',
  }

...and it looks like it's working (I presume it's now doing a lot of upgrades):
Notice: Compiled catalog for development.development in environment development in 30.70 seconds
Info: Applying configuration version '1472216804'
Notice: Updating apt packages
Notice: /Stage[main]/Govuk::Node::S_development/Notify[unattended_upgrades]/message: defined 'message' as 'Updating apt packages'

...but then I got a bunch of errors, and my postgres version is still the same as before (I had 9.3.13):
Notice: Updating apt packages
Notice: /Stage[main]/Govuk::Node::S_development/Notify[unattended_upgrades]/message: defined 'message' as 'Updating apt packages'
Notice: /Stage[main]/Govuk::Node::S_development/Exec[unattended_upgrades]/returns: executed successfully
Notice: /Stage[main]/Harden/File[/usr/bin/rlogin]/ensure: removed
Notice: /Stage[main]/Harden/File[/usr/bin/rcp]/ensure: removed
Error: Cannot create /etc/unattended-reboot/no-reboot; parent directory /etc/unattended-reboot does not exist
Error: /Stage[main]/Govuk_unattended_reboot/File[/etc/unattended-reboot/no-reboot]/ensure: change from absent to directory failed: Cannot create /etc/unattended-reboot/no-reboot; parent directory /etc/unattended-reboot does not exist
Error: Could not set 'file' on ensure: No such file or directory - /etc/unattended-reboot/check/03_no_reboot20160826-20903-ft13xs.lock at 85:/var/govuk/govuk-puppet/modules/govuk_unattended_reboot/manifests/init.pp
Error: Could not set 'file' on ensure: No such file or directory - /etc/unattended-reboot/check/03_no_reboot20160826-20903-ft13xs.lock at 85:/var/govuk/govuk-puppet/modules/govuk_unattended_reboot/manifests/init.pp
Wrapped exception:
No such file or directory - /etc/unattended-reboot/check/03_no_reboot20160826-20903-ft13xs.lock
Error: /Stage[main]/Govuk_unattended_reboot/File[03_no_reboot]/ensure: change from absent to file failed: Could not set 'file' on ensure: No such file or directory - /etc/unattended-reboot/check/03_no_reboot20160826-20903-ft13xs.lock at 85:/var/govuk/govuk-puppet/modules/govuk_unattended_reboot/manifests/init.pp

Thanks @jennyd, made that change.

#4815 seems to fix the other errors.  For the postgres not actually upgrading, I think this might be an order thing where unattended upgrades gets run early in the puppet process, but telling unattended upgrades to ask for a new version of postgres (or other packages) happens later in the process.  Running govuk_puppet a second time seems to achieve what we want which backs up that theory.  Is it possible to make this exec run as one of the last things that the puppet run does?

As an alternative can we do https://github.com/alphagov/govuk-puppet/pull/4839 instead? We'd like to avoid a puppet run suddenly taking a lot longer with no visible output.

Data point of 1: I mostly run govuk_puppet on my VM as part of the update-all.sh script, so any output in there is lost halfway through, between the git and bundler parts.  I'm not sure I'd spot the output in #4839 to prompt me, I wonder if others would be the same.  Is there a way to run untattended-upgrades so that we get some output, so a long period of nothing is less likely?
As an aside: putting the unattended-upgrades run where you've put the warning message might be a good idea as it would guarantee that it happens at the end of the puppet run.

I'm happy to add the command where the warning is. I was just a little worried about people suddenly thinking puppet runs take an extra few minutes when nothing changes. I'll update the other PR to suit.

@deanwilson that sounds like a good idea.  I'll close this PR in preference to yours.


Vectorization by dispatch vs. Hydra

I just found this highly interesting package, and would like to learn from your handling of conditionals to enhance what is essentially a vectorization package of mine.
Background: The package I'm talking about is MonteCarloMeasurements.jl, which aims at propagating sampled distributions through functions. The strategy employed by MCM is to implement a type, Particles, that behaves like a number, but internally contains a vector of samples (particles). Each scalar operation on this type is then translated easily through dispatch to a map over the inner vector. This works wonderfully well, and is much faster than manually calling a function the corresponding number of times, due SPMD properties (some benchmarks here).
The weakness of the type-based approach is that control flow based on the type Particles can currently not be handled. Particles{Bool} appearing in a boolean context should ideally cause each particle to flow through the branching code independently. For this, I imagine a compiler pass to be one viable approach, something that Hydra already seems to have in place. Now, I'm very new to compiler passes, IR and all this, but am willing to spend some time to learn it if it seems like a reasonable approach.
An example of the transformation I would like to do is from
code = Meta.@lower if x > 0
    return x^2
else
    return -x^2
end

to
code2 = Meta.@lower map(x.particles) do x
    if x > 0
        return x^2
    else
        return -x^2
    end
end

where the translation should only be done if x is of a special type defined below
struct Particles{T} <: Real
    particles::Vector{T}
end

Without this transformation, code like the following fails predictably
Base.:(^)(p::Particles,r) = Particles(p.particles.^r)
Base.:(>)(p::Particles, r) = Particles(map(>, p.particles, r))
p = Particles(randn(10))
function negsquare(x)
    if x > 0
        return x^2
    else
        return -x^2
    end
end
julia> negsquare(p)
ERROR: TypeError: non-boolean (Particles) used in boolean context

If the code was translated to
function negsquare(x)
    Particles(map(x.particles) do x
        if x > 0
            return x^2
        else
            return -x^2
        end
    end)
end
julia> negsquare(p)
Particles([0.404953, 0.210984, -1.00176, 0.253796, -0.00620389, 0.831144, -0.0240916, -1.90169, 0.875192, 1.4788])

I would get the desired result.
Do you think I can learn from the following code in Hydra to make this happen?
function fix_stmt(ssavalue, stmt, ir, old_to_new_ssavalue)
  replacement(x::SSAValue) = old_to_new_ssavalue[x]
  replacement(x) = x
  if stmt.expr isa GotoIfNot
    new_stmt = GotoIfNot(old_to_new_ssavalue[stmt.expr.cond], stmt.expr.dest)
    push!(ir, new_stmt)
  elseif stmt.expr isa GotoNode
    push!(ir, stmt)

Thankful for any input or insight on this!

What you're describing isn't just similar to the Hydra transform, it is the Hydra transform, so we should be able to make this easy for you. Taking an example from your readme, our goal is basically that you can write something close to the "naive" code foreach(_->qr($B), 1:1000) and get the best performance (even in the presence of control flow).
The details differ a bit from what you've outline here, we have a slightly more complex masking approach which basically ensures that if-statements-within-if-statements -- and loops and so on -- still exploit as much parallelism as possible.
If you want this I would recommend against trying to replicate it yourself, it's quite tricky to do well. But we'd be happy to help you dig into the code and theory here if you want to get involved.

our goal is basically that you can write something close to the "naive" code foreach(_->qr($B), 1:1000) and get the best performance

If this was realized in general, it would be truly amazing!

I would recommend against trying to replicate it yourself

Yes I agree, I would at most be able to make it work for very simple cases. making use of your implementation in Hydra is a much better approach.
I would be very happy to dig deeper into this. Having seen the speedup I can gain by my simple dispatch-based SPMD-approach on the CPU only makes me feel like this is a huge opportunity for making Julia hughely more performant. For MonteCarloMeasurements.jl, I'm happy to keep the type Particles as it is very convenient to work with when thinking about distributions, but adopt the Hydra pass wherever needed to make it fully general or performant.


Implemented some of the basic APIs

How do I know if the player has lost if func player_status() does not get the second player's symbol? Or I should check every time if board[i][j] not 0 and not player["symbol"]?

Please merge master using git merge master.

Please merge master using git merge master.

Nahum?

Great Job!!
You can push to master, and we'll continue from there to implement the agents.
When we do - we might encounter some minor changes to the Env4InRow class, but the most important things are already there :)