MR docs issue burndown #2

Docs Build status updates of commit 79a03c4:
:clock10: Pending: waiting for processors (68 builds ahead of you)
:warning: Docs Build is busy, currently there are 68 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 79a03c4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/scene-understanding-SDK.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit a656da7:
:clock10: Pending: waiting for processors (17 builds ahead of you)
:warning: Docs Build is busy, currently there are 17 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit a656da7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/install-the-tools.md
:white_check_mark:Succeeded
View

mixed-reality-docs/scene-understanding-SDK.md
:white_check_mark:Succeeded
View

mixed-reality-docs/text-in-unity.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 0dd95b5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/install-the-tools.md
:white_check_mark:Succeeded
View

mixed-reality-docs/scene-understanding-SDK.md
:white_check_mark:Succeeded
View

mixed-reality-docs/text-in-unity.md
:white_check_mark:Succeeded
View

mixed-reality-docs/voice-input.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Add latest news to the nav

CVI 8092016.pdf

Just a thought....to get the site live it might make sense to hide all blog content until CVI are ready to post


ERROR: Could not install packages due to an EnvironmentError: [Errno 30] Read-only file system: '/cvmfs/soft.computecanada.ca/easybuild/software/2017/Core/python/3.7.4/lib/python3.7/site-packages/humanfriendly-8.2.dist-info'

On compute canada
Full output
[jcohen@gra-login3 spine-generic]$ module load python/3.7
[jcohen@gra-login3 spine-generic]$ pip install -e .
Ignoring pip: markers 'python_version < "3"' don't match your environment
Looking in links: /cvmfs/soft.computecanada.ca/custom/python/wheelhouse/nix/avx2, /cvmfs/soft.computecanada.ca/custom/python/wheelhouse/nix/generic, /cvmfs/soft.computecanada.ca/custom/python/wheelhouse/generic
Obtaining file:///home/jcohen/spine-generic
Collecting coloredlogs (from spinegeneric==2.2.1)
  Using cached https://files.pythonhosted.org/packages/5c/2f/12747be360d6dea432e7b5dfae3419132cb008535cfe614af73b9ce2643b/coloredlogs-14.0-py2.py3-none-any.whl
Collecting matplotlib (from spinegeneric==2.2.1)
Collecting nibabel (from spinegeneric==2.2.1)
  Using cached https://files.pythonhosted.org/packages/8b/8c/cf676b9b3cf69164ba0703a9dcb86ed895ab172e09bece4480db4f03fcce/nibabel-3.1.1-py3-none-any.whl
Collecting pandas (from spinegeneric==2.2.1)
Collecting scipy (from spinegeneric==2.2.1)
Collecting tqdm (from spinegeneric==2.2.1)
  Using cached https://files.pythonhosted.org/packages/af/88/7b0ea5fa8192d1733dea459a9e3059afc87819cb4072c43263f2ec7ab768/tqdm-4.48.0-py2.py3-none-any.whl
Collecting sklearn (from spinegeneric==2.2.1)
Collecting pyyaml (from spinegeneric==2.2.1)
Collecting pybids (from spinegeneric==2.2.1)
  Using cached https://files.pythonhosted.org/packages/71/26/084e5a866bbe118f344638500dedf7b9b26a2b9ddab25a78fafbad9ae9b1/pybids-0.11.1-py2.py3-none-any.whl
Collecting humanfriendly>=7.1 (from coloredlogs->spinegeneric==2.2.1)
  Using cached https://files.pythonhosted.org/packages/8e/2d/2f1b0a780b8c948c06c74c8c80e68ac354da52397ba432a1c5ac1923c3af/humanfriendly-8.2-py2.py3-none-any.whl
Collecting numpy>=1.11 (from matplotlib->spinegeneric==2.2.1)
Collecting python-dateutil>=2.1 (from matplotlib->spinegeneric==2.2.1)
Collecting cycler>=0.10 (from matplotlib->spinegeneric==2.2.1)
Collecting pyparsing!=2.0.4,!=2.1.2,!=2.1.6,>=2.0.1 (from matplotlib->spinegeneric==2.2.1)
Collecting kiwisolver>=1.0.1 (from matplotlib->spinegeneric==2.2.1)
Collecting packaging>=14.3 (from nibabel->spinegeneric==2.2.1)
Collecting pytz>=2017.2 (from pandas->spinegeneric==2.2.1)
Collecting scikit-learn (from sklearn->spinegeneric==2.2.1)
Collecting bids-validator (from pybids->spinegeneric==2.2.1)
  Using cached https://files.pythonhosted.org/packages/3a/f5/247f0f077fe1cd9f52d8716af6fbf7d712bd1edd63cf776d8cf645371904/bids_validator-1.5.4-py2.py3-none-any.whl
Collecting num2words (from pybids->spinegeneric==2.2.1)
  Using cached https://files.pythonhosted.org/packages/eb/a2/ea800689730732e27711c41beed4b2a129b34974435bdc450377ec407738/num2words-0.5.10-py3-none-any.whl
Collecting patsy (from pybids->spinegeneric==2.2.1)
Collecting sqlalchemy (from pybids->spinegeneric==2.2.1)
Collecting six>=1.5 (from python-dateutil>=2.1->matplotlib->spinegeneric==2.2.1)
Requirement already satisfied: setuptools in /cvmfs/soft.computecanada.ca/easybuild/software/2017/Core/python/3.7.4/lib/python3.7/site-packages (from kiwisolver>=1.0.1->matplotlib->spinegeneric==2.2.1) (40.8.0)
Collecting threadpoolctl>=2.0.0 (from scikit-learn->sklearn->spinegeneric==2.2.1)
Collecting joblib>=0.11 (from scikit-learn->sklearn->spinegeneric==2.2.1)
Collecting docopt>=0.6.2 (from num2words->pybids->spinegeneric==2.2.1)
Installing collected packages: humanfriendly, coloredlogs, numpy, six, python-dateutil, cycler, pyparsing, kiwisolver, matplotlib, packaging, nibabel, pytz, pandas, scipy, tqdm, threadpoolctl, joblib, scikit-learn, sklearn, pyyaml, bids-validator, docopt, num2words, patsy, sqlalchemy, pybids, spinegeneric
ERROR: Could not install packages due to an EnvironmentError: [Errno 30] Read-only file system: '/cvmfs/soft.computecanada.ca/easybuild/software/2017/Core/python/3.7.4/lib/python3.7/site-packages/humanfriendly-8.2.dist-info'

Might be that a venv is necessary

I can try to look into it. Could you please let me know on what system did you get this error message ?

fixed with using venv


adjusting add_serialized_chains behaviour

Hi @grst,
I'm adding some updates to dandelion's parser and came across a small issue and wondering if you could give me a hand.
I've made a small snippet of the all_contig_annotations.json file from 10x's resource to use as a fixture (attached as 'data.txt' here because github doesn't allow me to attach with .json extension):
data.txt
The following round tripping works:
import dandelion as ddl
import scirpy as ir

adata = ir.io.read_10x_vdj('data.json')
vdj = ddl.read_10x_vdj('data.json') # I've wrote a new parser for dandelion

vdj_1 = ir.io.to_dandelion(adata)
adata_1 = ddl.to_scirpy(vdj) # this basically calls ir.io.from_dandelion

adata_2 = ir.io.from_dandelion(vdj)
vdj_2 = ddl.from_scirpy(adata) # this basically calls ir.io.to_dandelion

And while this works:
adata_3 = ddl.to_scirpy(vdj_2)

the following doesn't:
vdj_3 = ddl.from_scirpy(adata_3)
adata_4 = ir.io.to_dandelion(adata_2)

Both come up with the same issue:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-15-00d75a1a5f48> in <module>
     26     try:
     27         print(row['extra_chains'])
---> 28         tmp_ir_cell.add_serialized_chains(row["extra_chains"])
     29     except KeyError:
     30         print(row['extra_chains'])

~/Documents/Github/scirpy/scirpy/io/_datastructures.py in add_serialized_chains(self, serialized_chains)
    164             tmp_chains = json.loads(serialized_chains)
    165             for chain in tmp_chains:
--> 166                 self.add_chain(chain)
    167
    168     def _split_chains(self) -> Tuple[bool, dict]:

~/Documents/Github/scirpy/scirpy/io/_datastructures.py in add_chain(self, chain)
    144             self._chain_fields = list(chain.keys())
    145         elif self._chain_fields != list(chain.keys()):
--> 146             raise ValueError("All chains must have the same fields!")
    147
    148         if "locus" not in chain:

ValueError: All chains must have the same fields!

My proposed adjustments solves the round tripping issue but I wasn't sure if it was fitting with how the original class was supposed to function in situations like this were there's a mismatch in the type of fields present in the main chains vs extra chains.

Hi Kelvin,
thanks for that detailed report.

how the original class was supposed to function in situations like this were there's a mismatch in the type of fields present in the main chains vs extra chains.

The class expects no differences in the types of fields to be present. Not because this would be strictly required, but rather because it's usually a sign that something went wrong at some point.
Do you have any idea where these additional np1_length fields come from all of a sudden?
(Or rather: why do they get removed in the 'normal' chains?)

Do you have any idea where these additional np1_length fields come from all of a sudden?
(Or rather: why do they get removed in the 'normal' chains?)

So I think the issue lies in:
adata_3 = ddl.to_scirpy(vdj_2) # or adata_3 = ir.io.from_dandelion(vdj_2)

Which does this:
def from_dandelion(dandelion, transfer=False) -> AnnData:
   # skipping
    adata = read_airr(dandelion_df)
   # skipping

I suspect the issue might be with the include_fields option that trims the transferred items to whatever's in DEFAULT_AIRR_FIELDS, which doesn't include the np1_length etc. keys.
def read_airr(
    path: Union[str, Sequence[str], Path, Sequence[Path]],
    use_umi_count_col: Union[bool, Literal["auto"]] = "auto",
    infer_locus: bool = True,
    cell_attributes: Collection[str] = DEFAULT_AIRR_CELL_ATTRIBUTES,
    include_fields: Optional[Collection[str]] = DEFAULT_AIRR_FIELDS,
) -> AnnData:
# rest of code
    return from_airr_cells(airr_cells.values(), include_fields=include_fields)

Will test a bit and report back.

That's probably it!
However, shouldn't the extra chains get converted back to "normal chains" during to_dandelion? And then, during from_dandelion they should be treated and trimmed like any other chain.
I'm missing something :thinking:

Yes i think that solved it.

However, shouldn't the extra chains get converted back to "normal chains" during to_dandelion? And then, during from_dandelion they should be treated and trimmed like any other chain.

Yea i thought so too. What it looks like is the extra chains and normal chains were extracted differently during the  from_airr_cells` bit.

Ok, I looked into it using your example data. The np1_length should never end up in extra_chains when using from_dandelion (which calls read_airr internally).
It turns out I forgot to filter extra_chains for include_fields entirely. I think the solution is to pass include_fields to the AirrCell._serialize_chains function:
https://github.com/icbi-lab/scirpy/blob/f25858842993b7ce039b00ccaef055ad78ee9746/scirpy/io/_datastructures.py#L295
Do you want to give it a go?

No dice. I got:
_serialize_chains() got an unexpected keyword argument 'include_fields'

Ah, I meant implementing support for include_fields in _serialize_chains -- should be trivial. Should have been more specific.

Ah ok sure.
I'm trying to work this out, unfortunately with much difficulty.
The AirrCell.add_chain is called by _serialize_chains and this works cleanly when dealing with extra or normal chains.
from_airr_cells's .to_scirpy_record() works perfectly too. Only thing is it calls this in the first step:
https://github.com/icbi-lab/scirpy/blob/f25858842993b7ce039b00ccaef055ad78ee9746/scirpy/io/_convert_anndata.py#L76-L78
I wonder if this is where it needs to be tweaked?
https://github.com/icbi-lab/scirpy/blob/f25858842993b7ce039b00ccaef055ad78ee9746/scirpy/io/_datastructures.py#L303-L314

_to_scirpy_records first decides which chains will be extra and which will be "normal":
https://github.com/icbi-lab/scirpy/blob/f25858842993b7ce039b00ccaef055ad78ee9746/scirpy/io/_datastructures.py#L294
Then the 'extra' chains get serialized and added:
https://github.com/icbi-lab/scirpy/blob/f25858842993b7ce039b00ccaef055ad78ee9746/scirpy/io/_datastructures.py#L295
So ultimately, _serialize_chains would need to be modified to accept an include_fields parameters:
https://github.com/icbi-lab/scirpy/blob/f25858842993b7ce039b00ccaef055ad78ee9746/scirpy/io/_datastructures.py#L236-L248
The section you mentioned above already handles include_fields correctly for 'normal' chains. However, by then the extra_chains already got added including all fields.

The filtering for extra chains still doesn't work... I'll run it through with the debugger

I completely misread how https://github.com/icbi-lab/scirpy/blob/f25858842993b7ce039b00ccaef055ad78ee9746/scirpy/io/_datastructures.py#L240-L247 works during review. Actually, it wasn't filtering at all.
I think it should be working now.

Does this work for you now?

Yes i think so! Let me do a bit of testing.
I also need to add a function to sanitize the transferred dataframe on dandelion's side.

ok, cool!
I'll take a few days off next week, and will be back to review & merge on Thursday ;)

Looks like it's all working with dandelion v0.1.4! Although it took 2.5 hours to finish the tests on the macos3.8 test check.
Ready to merge when you're back.

Although it took 2.5 hours to finish the tests on the macos3.8 test check

Let's hope that was a random hiccup :roll_eyes:
Thanks again for the details bug report and your contribution! Will release v0.8 later.


3 brackets on settings elements

It seems this is in error?
{{{extraCode}}}
https://github.com/TelescopeJS/Telescope/blob/master/client/views/common/layout.html#L18
{{{footerCode}}}
https://github.com/TelescopeJS/Telescope/blob/master/client/views/common/footer.html#L4

@cloudunicorn I was hoping to use Olark and ran into the same issue--did you ever find a way to use JS scripts?


langflow 1.0.15 import problem

Bug Description
Hello,
Installed and started langflow in dev mode -
but when I am importing flows, even the ones that i create in 1.0.15 I get error in js console
"index-DnQl3Zy0.js:1 Uncaught (in promise) DOMException: Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node.
at o.onchange (http://<IP_ADDRESS>:7860/assets/index-DnQl3Zy0.js:1:4767)"
and the importing flow is not imported
Reproduction

installing all the required libs on windows wls, setting up right
running poetry install
running make init
the welcome message shows
going to http://<IP_ADDRESS>:7860
importing flow
getting error in js console
import failed, no uploaded flow in the lib

A. same goes for environment for make frontend / make backend
Expected behavior
it should add the flow
Who can help?
No response
Operating System
Windows 11 pro with WLS
Langflow Version
1.0.15
Python Version
3.11
Screenshot
No response
Flow File
Memory chatbot from starting template or others

Hey, @severfire !
Good catch! Thank god we didn’t release it yet haha
Should be fixed asap.

@ogabrielluiz was it bug or was it something wrong with my env? :-/

We've released it yesterday. Could you try again? We have some big changes yesterday

@ogabrielluiz
Tested it further.
Seems like it is the problem with Brave browser.
Tested with:

newly built dev 1.0.15 (make init)
pipx version of langflow 1.0.15 (langflow run)
with brave and firefox.

Seems like Brave browser had problems.
Looks then it is not critical.
Next time I will test with both browsers.

hey @severfire, just tested on brave and did not saw this warning on console, I will close the issue for now, if it happens again we can re-open :)


Add option to CLI deploy command to register extensions in ExtensionRegistry

Adds the option --register-extension to the deploy command:
thirdweb deploy --register-extension

Compiles the contracts in the smart contract project directory and prompts the user to select extension contracts to deploy and register in an ExtensionRegistry.

/release-pr


Feature  upgrading twemoji and emojilib

Upgrading Twemoji Parser
Upgrading Emojilib
Upgrading Unicode Emoji Json
Updating emoji list data

⚠ WARNING: This PR is large.
Make sure your code has tests and you have done the necessary QA.


masthead color.

Hey Michael!
I'm wondering how to change a color of masthead? I add background color to masthead.scss, but unfortunately still have little white block after animated menu. Can you point out how to resolve it?

What exact CSS changes did you make so I can get a feel for what you tried.
Also a screenshot of what it looked like would be helpful.

here is screenshot of white block:

this is a code (I only added 3 background variables to 3 masthead elements + changed position to fixed for sticky header)
/* ==========================================================================
   MASTHEAD
   ========================================================================== */

.masthead {
  position: fixed;
  top: 0;
  border-bottom: 0px solid $border-color;
  -webkit-animation: intro 0.3s both;
          animation: intro 0.3s both;
  -webkit-animation-delay: 0.15s;
          animation-delay: 0.15s;
  z-index: 20;
  width: 100%;
  background: $lighter-gray;
  height: 55px;

  &__inner-wrap {
    @include container;
    @include clearfix;
    padding: 0 1em 0;
    font-family: $serif;
    font-size: $type-size-5;

    @include breakpoint($x-large) {
      max-width: $x-large;
    }

    nav {
      z-index: 10;
    }

    a {
      text-decoration: none;
    }
  }
}

.masthead__menu {
  background: $lighter-gray;
  @include breakpoint($large) {
    @include prefix(2.5 of 12);
  }

  ul {
    margin: 0;
    padding: 0;
    clear: both;
    list-style-type: none;
  }
}

.masthead__menu-item {
  background: $lighter-gray;
  display: block;
  list-style-type: none;
  white-space: nowrap;

  &--lg {
    padding-right: 0;
    font-weight: 700;
  }
}

I'm not at my computer to fully dig into the code but I'd suggest right clicking the white box and hit "inspect" to fire up your browser's web dev tools. It'll tell you exactly what the element or class is so you can search for it in the Sass partials to edit. That's my go to method for debugging CSS.
I suspect it's this line in _sass/_navigations.scss related to the "greedy nav" used in the masthead.
https://github.com/mmistakes/minimal-mistakes/blob/master/_sass/_navigation.scss#L178

I suspect it's this line in _sass/_navigations.scss related to the "greedy nav" used in the masthead.

bingo! thanks a lot @mmistakes


Grid view support

does it support grid view out of the box, or does it require major tweaks for the grid view to work.

It's a list view, you'd have to make another class extending the grid view I suspect but could copy the approach. However I stopped using Unity a few years ago so I don't know the current status of their class structure, that's a task for others to pick up if they want.


fix(pacman): enclose directory paths in PKGBUILD in double quotes

This ensures that paths with spaces on the target machine are properly preserved during pacman's installation of the package.
Context: some users of our Moxin app that was packaged for Arch Linux using cargo-packager reported that their pacman install command was failing due to paths containing whitespace.
Thanks to @cassaundra for proposing this fix and confirming that it works.

I tried pushing the change file myself but I couldn't 😢 lacking permissions on your repo

oh shoot, sorry guys I completely forgot about this PR in the hecticness of so much travel. Will push shortly!

done! thanks for the reminder, and my apologies for the delay.

Normally we require that commits are signed, but in the interest of approving your contribution I will relax the requirements this time.

thanks, will keep that in mind for my next contribution.


Saves the high quality gif

Q
A

Bug?
no

New Feature?
yes

Version Used
N/A

FFmpeg Version
FFmpeg 3.4.1

OS
Debian 9

Save Gif in high Quality
FFMpeg\Media\Gif.php
public function save_with_high_quality($pathfile)
{
    /**
     * @see http://blog.pkh.me/p/21-high-quality-gif-with-ffmpeg.html
     */

	$filters = "fps=30,scale=".$this->dimension->getWidth().":-1:flags=lanczos";
	$palette = "/tmp/palette.png";

    $commands = $commands_2 = array(
        '-ss', (string)$this->timecode
    );

    if(null !== $this->duration) {
        $commands[] = $commands_2[] = '-t';
        $commands[] = $commands_2[] = (string)$this->duration;
    }

    $commands[] = $commands_2[] = '-i';
    $commands[] = $commands_2[] = $this->pathfile;
    $commands[] = '-vf';
    $commands[] = $filters.',palettegen';
    $commands[] = '-y';
    $commands[] = $palette;

    $commands_2[] = '-i';
    $commands_2[] = $palette;
    $commands_2[] = '-lavfi';
    $commands_2[] = $filters.' [x]; [x][1:v] paletteuse';
    $commands_2[] = '-y';

    $commands_2 = array_merge($commands_2, array($pathfile));

    try {
        $this->driver->command($commands);
    } catch (ExecutionFailureException $e) {
        $this->cleanupTemporaryFile($pathfile);
        throw new RuntimeException('Unable to generate palette', $e->getCode(), $e);
    }

    try {
        $this->driver->command($commands_2);
    } catch (ExecutionFailureException $e) {
        $this->cleanupTemporaryFile($pathfile);
        throw new RuntimeException('Unable to save gif', $e->getCode(), $e);
    }

    return $this;
}

I'd like to have this in a pr for the 1.x branch. So in case somebody wants to commit this change and write tests for it, I'm happy to merge it.
If you need help, do not hesitate to ask.

Sorry, i'm not experienced with github yet to create a PR in the right way :-(

No worries :)

Ping? ;-) PR exists already


[DPAS]: Improve the Triton GEMM/flash attention/sparse attention kernel performance to get 60% tensor core peak performance.

As Phil's comments that the backend need to get 60% tensor core peak performance for flash attention and dense matmul, the Triton would package it as part of Triton 3.0 release.

All right. I think the steps here are to get tt.dot to use the DPAS instruction on PVC GPUs correctly first. Then use 2D block loads for the operands and then analyze what the performance for these workloads is at that point.

related to issue #772 and #773

Linked issues:

#772
#773

Closing this issue. The reminder will be tracked in

#773


Test batch begin/invoke/end routines via NPI dispatch tables

Currently batch APIs are not tested via NPI dispatch tables. We should add tests for these using the sample ext driver.
Created per comment in #2329

Maybe also update netebpfext to invoke batch APIs for XDP hook.

The routines and the NPI contract also need to be documented.


Can't get ID of Activity by using activities = client.get_activities()

Hi,
I was wondering how I can get the id of an activity by using the client.get_activities() function.
When I look into this function and also the activity model behind it, I can't see the field ID.
My intention was, that I can download all activities from an athlete and then enable the possibility to get in addition the details (streams) of an activity by pressing an button.
When I look into the developer api from strata https://developers.strava.com/docs/reference/#api-Activities-getLoggedInAthleteActivities. I can see that there is an id field which is provided by the api.
Did somebody have an idea how to get the id for collecting the stream data from an activity by having the list of activities?
BR

Does this not work for you ... ?

activities = client.get_activities()
for activity in activities:
  print(f"Activity ID: {activity.id}")

Hi, the result method to_dict() not include id field

I'm running into the same issue

the id in the Activity entity is named upload_id rather than id.

the id in the Activity entity is named upload_id rather than id.

No, this is not correct : for instance i retrieve an activity.upload_id = '3125668582' while the activity.id =<PHONE_NUMBER>
btw upload_id is also a field of the Activity before converting it to dict.

Same issue. Upload ID does not match the id
Great library thank you!


Add protobuf support to qdk

NOTE: Introducing a breaking change: In problem class' deserialize method, the parameter problem_as_json of type string
is changed to input_problem of type Union[str, list] to accomodate both json strings and protobuf messages.
This feature is to support protobuf serialization of problems.
When a user specifies a problem, they can now create Problem(name=<>, problem_type=<>, serialization_type="application/x-protobuf") which is an optional value. If not supplied, json is assumed and no change to the current flow occurs except the following:

when calling problem.serialize the serialize function calls serialize_to_json and
problem.deserialize calls deserialize_from_json
When uploading the problem, blob's metadata is set to "{SerializtionType:application/json}"

If set to application/x-protobuf the following things occur:

problem.serialize calls serialize_to_protobuf. This method, constructs the problem as a list of protobuf messages, each message containing 1000 terms and returns a list. The name field in the metadata is added in this step. The problem type and the name are attached to the first message in the list.
problem.deserilize forwards to deserialize_from_json: This method reads the list of messages and constructs the Problem object
When uploading, the method sets the blob's metadata property to {"SerializationType": application/x-protobuf}
When downloading, the method checks for the SerializationType metadata set on the blob and if it's set to application/x-protobuf, it calls the  deserialize_from_proto method to complete it's operation
When submitting a job with Solver.submit(), there is a check for if the specified solver supports protobuf. If not, a value error is returned to the user

The following solvers are supported:

SubstochasticMonteCarlo
PopulationAnnealing
Both these solver classes now have protobuff_supported method that returns true.

In addition, a problem.proto file is added, which is the schema for the format and a problem_pb2.py which is generated from this schema.
The following test scenarios are added:

serializing to proto
deserialing from proto
downloading a proto
Submitting a solver with valid solver type for proto
Submitting a solver with invalid sovler type for proto and get user error.

The following diagram explains how the protobuf messages are sent from the problem as a series of smaller messages with
1000 terms and the serialize_to_proto methd in problem.py does this construction

Left some comments. Please rename azure.quantum.problem_pb2 to something else, like azure.quantum.serialization.protobuf.

Hi, I moved the file to serialization, but not renaming it since the name is generated based on the protobuf format schema

/azp run


attiny424 not programmed using pyupdi.

Hello all.
I have been trying to use attiny424 (avr-2 series). When i upload it using pyupdi, it shows that attiny424 is notwrong argument. I checked datasheet and it clearly shows that UPDI is available.

I recommend to use pymcuprog instead of pyupdi:
https://pypi.org/project/pymcuprog/

I've installed it. when i check in device folder, attiny424 is not present
got any tutorials to use ??

Good point - I was not aware it was missing.
I would just copy the attiny414.py in the device scripts folder and edit the device id to match the 424.
Start by checking connectivity: pymcuprog ping -d attiny424 -t uart -u
then you could for example write flash using: pymcuprog write -m flash -f hexfile.hex

Using UPDI, I can burn code if I select any microcontroller which is listed, I can upload the code this way. Maybe i can select attiny414 and write code..might work..will update.


vkCreateShaderModule() returns VK_ERROR_OUT_OF_HOST_MEMORY due to bad shader magic

Describe the bug
A clear and concise description of what the bug is.
Frequency of occurance. Command line output. Stack trace.
The attached Vulkan application runs on a Raspberry Pi 4, although the Pi4 uses a completely different Vulkan driver.
I was hoping to run it on a 3B+ using rpi-vk-driver, however when vkCreateShaderModule() is called, rpi-vk-driver returns VK_ERROR_OUT_OF_HOST_MEMORY.
Further inspection reveals that VK_ERROR_OUT_OF_HOST_MEMORY is returned because the shader magic for the shader does not add up, as seen here: https://github.com/Yours3lf/rpi-vk-driver/blob/master/driver/shader.c#L25
These same SPIR-V shaders run fine on a Raspberry Pi 4 (v3dv) as well as on mesa's RADV and NVIDIA's proprietary drivers for GTX graphics cards, so I suspect there is a driver issue.
Is there anything that can be done about this?
To Reproduce
Steps to reproduce the behavior:

Extract the attached zip file to some directory- the executable "micro0" and "data" should be in the same directory.
In a terminal, cd to the directory containing "micro0" and "data", then run micro0: ./micro0
The program will error and close due to vkCreateShaderModule() failing, and if you add a printf() statement to the if(magic != 0x14E45250) if statement block in the driver, you will find the driver is returning VK_ERROR_OUT_OF_HOST_MEMORY due to bad shader magic.

Expected behavior
At the very least, vkCreateShaderModule() should complete successfully, because I know these are good/valid SPIR-V shaders.
Given vkCreateShaderModule() completes successfully, the program should probably run fine then-
there could be other non-driver issues running this application on the 3B+ that I have not discovered yet, but probably not since it runs fine on the Pi4 and other Vulkan drivers.
Device information (please complete the following information):

Device info (eg. Raspberry Pi 3 Model A+)
OS information (cat /proc/version, lsb_release -a)
Kernel module information: lsmod
Driver version (git commit ID or release info)
Tested on a Raspberry Pi 3B+ running Raspbian.
Driver version: git-c95f714

$ cat /proc/version
Linux version 5.4.51-v7+ (dom@buildbot) (gcc version 4.9.3 (crosstool-NG crosstool-ng-1.22.0-88-g8460611)) #1333 SMP Mon Aug 10 16:45:19 BST 2020

$ lsb_release -a
No LSB modules are available.
Distributor ID:	Raspbian
Description:	Raspbian GNU/Linux 10 (buster)
Release:	10
Codename:	buster

$ lsmod
Module                  Size  Used by
cmac                   16384  1
rfcomm                 45056  4
bnep                   20480  2
hci_uart               40960  1
btbcm                  16384  1 hci_uart
bluetooth             376832  29 hci_uart,bnep,btbcm,rfcomm
ecdh_generic           16384  2 bluetooth
ecc                    32768  1 ecdh_generic
8021q                  32768  0
garp                   16384  1 8021q
stp                    16384  1 garp
llc                    16384  2 garp,stp
evdev                  24576  3
vc4                   233472  3
cec                    49152  1 vc4
hid_apple              16384  0
brcmfmac              319488  0
drm_kms_helper        184320  3 vc4
brcmutil               16384  1 brcmfmac
drm                   446464  3 vc4,drm_kms_helper
sha256_generic         16384  0
libsha256              20480  1 sha256_generic
drm_panel_orientation_quirks    16384  1 drm
snd_soc_core          200704  1 vc4
snd_compress           20480  1 snd_soc_core
cfg80211              679936  1 brcmfmac
snd_pcm_dmaengine      16384  1 snd_soc_core
syscopyarea            16384  1 drm_kms_helper
sysfillrect            16384  1 drm_kms_helper
sysimgblt              16384  1 drm_kms_helper
fb_sys_fops            16384  1 drm_kms_helper
rfkill                 28672  6 bluetooth,cfg80211
raspberrypi_hwmon      16384  0
bcm2835_codec          36864  0
i2c_bcm2835            16384  0
bcm2835_isp            32768  0
bcm2835_v4l2           45056  0
v4l2_mem2mem           32768  1 bcm2835_codec
bcm2835_mmal_vchiq     28672  3 bcm2835_isp,bcm2835_codec,bcm2835_v4l2
videobuf2_dma_contig    20480  2 bcm2835_isp,bcm2835_codec
videobuf2_vmalloc      16384  1 bcm2835_v4l2
videobuf2_memops       16384  2 videobuf2_dma_contig,videobuf2_vmalloc
videobuf2_v4l2         28672  4 bcm2835_isp,bcm2835_codec,bcm2835_v4l2,v4l2_mem2mem
videobuf2_common       57344  5 bcm2835_isp,bcm2835_codec,bcm2835_v4l2,v4l2_mem2mem,videobuf2_v4l2
snd_bcm2835            24576  1
snd_pcm                94208  4 vc4,snd_pcm_dmaengine,snd_bcm2835,snd_soc_core
videodev              237568  6 bcm2835_isp,bcm2835_codec,videobuf2_common,bcm2835_v4l2,v4l2_mem2mem,videobuf2_v4l2
snd_timer              32768  1 snd_pcm
mc                     45056  6 bcm2835_isp,bcm2835_codec,videobuf2_common,videodev,v4l2_mem2mem,videobuf2_v4l2
snd                    69632  9 snd_compress,snd_timer,snd_bcm2835,snd_soc_core,snd_pcm
vc_sm_cma              32768  2 bcm2835_isp,bcm2835_mmal_vchiq
uio_pdrv_genirq        16384  0
uio                    20480  1 uio_pdrv_genirq
fixed                  16384  0
i2c_dev                16384  0
ip_tables              28672  0
x_tables               32768  1 ip_tables
ipv6                  450560  24
nf_defrag_ipv6         20480  1 ipv6

micro0.zip

This driver does not support SPIR-V, you need to use assembly, please see the numerous examples :)
If you feel like that, feel free to add SPIR-V support, I'd gladly merge it.


Reach out to mentors

We have discussed looking for mentors to support us in a variety of ways, depending on what the mentor may be best suited for / interested in.

Individual mentor for our roles
Group mentor to speak to us in our team meeting and where we could ask questions

We have discussed mentors meeting with us just one time, or on an ongoing basis. We are taking a "More the merrier" attitude :D If you have someone in mind, see what they would feel most comfortable committing to.
Technical mentors:

C#/.NET

I found a group of new C#.net programmes.. they all seem pretty cool.. will see how this goes..


TypeError: cacheDir is not a function

I got this error when testing out the API module on our reference guide: https://docs.readme.com/developers/reference/projects
const sdk = require('api')('https://docs.readme.com/developers/openapi/5e1a4f6284248b0018149174');

sdk.getProject()
  .then(res => res.json())
  .then(res => {
    console.log(res);
  });

I saved it to /tmp/node-simple.js and ran npm install api, then ran node node-simple.js and got the following error:
$ node node-simple.js
/private/tmp/node_modules/api/src/cache.js:17
    this.dir = cacheDir();
               ^

TypeError: cacheDir is not a function
    at new SdkCache (/private/tmp/node_modules/api/src/cache.js:17:16)
    at Sdk.load (/private/tmp/node_modules/api/src/index.js:31:19)
    at module.exports.uri (/private/tmp/node_modules/api/src/index.js:141:23)
    at Object.<anonymous> (/private/tmp/node-simple.js:1:27)
    at Module._compile (internal/modules/cjs/loader.js:776:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:787:10)
    at Module.load (internal/modules/cjs/loader.js:653:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
    at Function.Module._load (internal/modules/cjs/loader.js:585:3)
    at Function.Module.runMain (internal/modules/cjs/loader.js:829:12)

Confusingly when it's in another directory it works as intended? Could this be an error in running from /tmp? I use it exclusively for messing around with stuff like this.

Interesting, it looks like the find-cache-dir module we're using for locating the cache dir returns nothing if it can't find the modules dir: https://github.com/avajs/find-cache-dir/blob/master/index.js#L44-L67

I definitely have the module installed:
$ npm ls api
/private/tmp
└──<EMAIL_ADDRESS>
And there's a /tmp/node_modules dir too.

Will look into it, maybe its a bug in find-cache-dir.

Able to replicate this when not running the library out of /tmp.


feat: Add Footer Section

Your checklist for this pull request
🚨Please review the guidelines for contributing to this repository.

[x] Make sure you are requesting to pull a topic/feature/bugfix branch (right side). Don't request your default branch!
[x] Make sure you are making a pull request against the default branch (left side). Also you should start your branch off default branch.
[x] Check the commit's or even all commits' message styles matches our requested structure.
[x] Check your code additions will fail neither code linting checks nor unit test.
[x] I have added necessary documentation (if appropriate)

Description
Please describe your pull request.
❤Thank you!
Post merge checklist

[ ] Follow steps from the guidelines for contributing to this repository.
[ ] If you are a new contributor, ping in the thread and one of the maintainers will add you to all-contributors list.

Hey @ORKO06. There are quite a few changes required.

Why are you using section container. Section container takes up the full height of the screen which we don't want for containers.
Why are you placing the footer inside a container? It should expand throughout the screen (in terms of width). What is to be done here is, the light grey background part should extend throughout the screen and the contents of the footer should be within the container. So remove the footer from the container (in pages/index.js) and use container in the footer component.
The design of the footer is of 3 column layout. You are using a 2 column thinge.
The colour of the icons is greyish (secondary colour), you are using white.
The 2 cards are the exact same component, so why 2 different components?


samples: nffs: New NFFS sample

This is a simple application demonstrating how to read & write persistence data on
flash of the nRF52 SoC, using NFFS file system.
Signed-off-by: vikrant8051 (Vikrant More<EMAIL_ADDRESS>new file:   samples/subsys/fs/nffs/CMakeLists.txt
new file:   samples/subsys/fs/nffs/README.rst
new file:   samples/subsys/fs/nffs/prj.conf
new file:   samples/subsys/fs/nffs/sample.yaml
new file:   samples/subsys/fs/nffs/src/main.c

Hi Vikrant. Thanks for submitting this again. You will need to make  some modifications, but it is important that you do not close the Pull Request.  Instead you just modify your commits and the push them with git push -f.

@vikrant8051 let's start by fixing the commit message.
 Now I'll copy some reviewers and we can do a code review.

@vikrant8051 now that you have review comments you need to follow this workflow:
<edit all the files you need to make changes>
$ git add <changed files>
$ git commit --amend
$ git push -f

Done !!

@vikrant8051 great! so now we need to wait for more reviewers and also see whether the checks succeed or fail. If they fail you can click on the link to see what went wrong.

Hmmmm...done !!

Codecov Report

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

@@           Coverage Diff           @@
##           master    #6156   +/-   ##
=======================================
  Coverage   52.88%   52.88%
=======================================
  Files         412      412
  Lines       40274    40274
  Branches     7801     7801
=======================================
  Hits        21297    21297
  Misses      15761    15761
  Partials     3216     3216

Continue to review full report at Codecov.

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

@vikrant8051 as you can see there are some test failures. I believe it's  because you are including nrf.h and it's trying to build for other architectures. https://app.shippable.com/github/zephyrproject-rtos/zephyr/runs/10695/summary/console

hence I was added in samples/boards/nrf52.

@vikrant8051 ok but then you need to edit the whitelist in sample.yaml  to restrict the build to the platforms that are supported, like this: https://github.com/zephyrproject-rtos/zephyr/blob/master/samples/boards/nrf52/power_mgr/sample.yaml

But I was able to build it even in samples/subsys/fs/nffs.

Ok, done !!

done !!

Done !!
If anything else is remaining then don't hesitate to edit. I've already enable that feature so that reviewer can edit as per his requirement.

Done !!

Why did you close this PR and delete the branch? Please reopen it with those changes when you are ready for more review. As Carles mentioned earlier, it is important that you not close the PR after updating it.

If anything else is remaining then don't hesitate to edit it. I've already enable feature so that reviewer can edit as per his requirement.

This isn't really how Zephyr works, I'm afraid -- the idea is that reviewers describe changes they think should be made, and the person proposing the PR can respond and update -- and re-test -- the PR as needed.

BTW, here is the workflow document: http://docs.zephyrproject.org/contribute/contribute_guidelines.html


added LinearSVR_Normalize

Move the scaling operation above the model description.
Add a section title has Data Rescaling or Data Scaling.
Provide description / detail about normalize in 2-3 lines

Hey shreepad,
You would fork repo again. we have changed some folder names

The main SVM folder has been renamed to Support Vector Machine

Your code for normalize has got converted into markdown text instead of code


Add json liquid filter to serialize data to a JSON string

Example: {{ data | json}} takes the object of the payloads data key and converts it to JSON

Nice, good idea!  Please add to the wiki too.

Done, thanks for the reminder. :)


dtgePreview is always null?

Your example
@inherits Umbraco.Web.Mvc.UmbracoViewPage
@if (ViewContext.RouteData.Values["dtgePreview"])
{
	// Render preview view
}
else
{
	// Render front end view
}

But ViewContext.RouteData.Values["dtgePreview"] is always null
If I render in front its null.
If I open content in backend its null.
If I preview the content using Umbraco Preview (in case I totally misunderstood)

Thanks for raising this @jesperordrup - odd that it's null. I'll investigate where the bug is ... unless if someone gets there before me? 😃

Hello Mr @leekelleher,
Think I found it. You transfer route values correct to actions but not to partials.
Looks great:
return helper.Action(editorAlias, controllerName, routeValues)
Looks less great:
return helper.Partial(content.DocumentTypeAlias, content);


[Feature]: job management style finish

Purpose
redesign & job management style finish

Tests

API and Format

Documentation

@zhangmo8, the frontend style of job management is contrary to offline discussion. Please confirm the prototype of the job management with @labbomb.

The fields already conform to job management PTAL @SteNicholas


ActiveRecord::InvalidForeignKey: Mysql2::Error: Cannot delete or update a parent row: a foreign key constraint fails
/govuk/collections-publisher/db/schema.rb:25:in `block in <top (required)>'
/govuk/collections-publisher/db/schema.rb:13:in `<top (required)>'
Caused by:
Mysql2::Error: Cannot delete or update a parent row: a foreign key constraint fails
/govuk/collections-publisher/db/schema.rb:25:in `block in <top (required)>'
/govuk/collections-publisher/db/schema.rb:13:in `<top (required)>'
Tasks: TOP => db:schema:load
(See full trace by running task with --trace)

I've worked around it using @kevindew 's suggestion in this Slack thread -

run govuk-docker run collections-publisher-lite bin/rails db:drop
comment out the second db:setup invocation for the test environment on line 3 of the collections-publisher Makefile
re-run make collections-publisher

but it would be good to fix the issue for the next new starter

Thanks for raising this @aldavidson and welcome to GOV.UK 🎉
For further context, there seems to be a couple of issues at play. At the root of this there is a problem where there is an expectation that rake db:setup is consistently idempotent - whereas it only seems to be partially (it'll break on db:schema:load for MySQL) - @benthorner any thoughts on if we should reconsider returning to the conditionals of https://github.com/alphagov/govuk-docker/commit/1ec59d4543c30103a7b36129289e0135c77a61ca ?
Then there's an issue that this actually fails on a first execution due to Collections Publisher being a Rails 6.0 app and not benefiting from the 6.1 fix of https://github.com/alphagov/govuk-docker/blob/2fe4ee96cfd4fdae9a042727cefb79b85889b8f1/projects/collections-publisher/Makefile#L3 - this seems a minor concern since it wouldn't be a problem if idempotency was resolved and the app will (hopefully) be upgraded to 6.1 in the coming weeks.

Thanks for investigating a bit @kevindew ⭐. I've had a look and agree MySQL does have an annoyingly different behaviour. Since we have so few apps using MySQL, it's likely I didn't test them in 1ec59d4.
All of our MySQL apps still seem to be on Rails 6.0. I tried a quick upgrade of Collections Publisher, and the initial setup issue was indeed resolved by the behaviour change, since the test DB isn't setup with the dev one.
I can think of a few options to get it working again:

Revert 1ec59d4. Tempting, but not ideal, as it confuses the purpose again.
Remove the second db:setup until each app upgrades to 6.1 (maybe just comment it out).
Catch the error and warn about it (optionally remove the catch after upgrading).
Switch to db:drop db:setup, to prevent the error (could be unexpected).
Switch to db:create db:migrate db:seed (doesn't work as old migration files are broken).

I dislike the second option, as while people shouldn't be constantly using make bootstrap, it's confusing if you do run it again and it errors: it's unclear if the app is broken, GOV.UK Docker is broken, or that you can't run it twice.
I'm tempted to go with option 3:

We're not hiding the error, or doing something surprising.
The error is a good prompt you shouldn't rerun make bootstrap.
It'll work before and after the upgrade to 6.1.

What do you think?

Thanks for taking a look Ben.

I can think of a few options to get it working again:

Revert 1ec59d4. Tempting, but not ideal, as it confuses the purpose again.
Remove the second db:setup until each app upgrades to 6.1 (maybe just comment it out).
Catch the error and warn about it (optionally remove the catch after upgrading).
Switch to db:drop db:setup, to prevent the error (could be unexpected).
~Switch to db:create db:migrate db:seed~ (doesn't work as old migration files are broken).

I agree with your reasonings for not wanting option 2 or 4. 👍
I'm not quite convinced by a warning though as this isn't necessarily a mistake (even if all apps are 6.1) given make has dependencies you could easily re-run make on an all that is already run without having a set-up step.
I find myself leaning more towards option 1, however I believe there is a short hand Rails task we can use instead,  db:prepare which does setup for no db or migrate if one exists, it arrived in Rails 6.

I'm not quite convinced by a warning though as this isn't necessarily a mistake

Yeah, there are several reasons someone might re-run the Makefile for an app. "warning" was a bit vague. It would just be some message to try and explain the error that preceded it, so the user wouldn't be alarmed.
However...

I believe there is a short hand Rails task we can use instead, db:prepare

That's a much a better outcome! Let's do that: https://github.com/alphagov/govuk-docker/pull/494


PCL and Eigen Reference Console Application

Hi,
I have installed PCL using vcpkg:
.\vcpkg install pcl:x64-windows and .\vcpkg integrate install
And I am trying just to reference any pcl header file for C++ 64 bit Console Application.
But the compiler cannot find Eigen.
The VCPKG shows that I installed the 64 bit version of Eigen.
I just want to see if I can build a console application by referencing any PCL header file and that it is recognized by Visual Studio.
I attached the screenshot below:

How I can properly reference Eigen?

Thanks for posting this issue. If you want use header files of VCPKG in C++ project, you also need run command .\vcpkg integrate project on VCPKG. About use header files of Eigen3 will be fixed in PR #13739.

Thanks @LilyWangL for your answer. Unfortunatelly, even by introducing modification commented at PR#13739 the problem persists. Any idea why? Thank you again,

@9and3 I tried to install the pcl:x64-windows and eigen:x64-windows with latest vcpkg 2022-02-11-dac008bf9b55ef48e10bcc81e273b97a82bbf72d and after run .\vcpkg integrate install , I could find and build the header files of pcl and eigen in a console application with x64 platform.

I also tested following codes and it built successfully.
#include <pcl/common/distances.h>    //distances.h include <Eigen/Core>
int main()
{
    std::cout << "Hello World!\n";
}

Hello @Cheney-W , thank you for taking the time. Indeed, with the latest build it does work. Thank you again!


Emscripten SDK: python2 doesn't exist on a freshly installed Mac

emcc does not run out of the box on a new Mac, since /usr/bin/python2 doesn't exist. Probably something for emscripten or the emscripten SDK to fix.

PS: the fix should be a check for 'python2' in 'fips diag tools', and tell the user to install 'brew python' on OSX, since this has a proper python2 link

Fixed.


Pressing F8 when already viewing the history screen crashes the app

Before Proceeding, please acknowledge:

[x] I have consulted the Troubleshooting guide.
[x] I have searched Issues and Discussions in this repo.
[x] Feature requests should be initiated as Discussions. This is a bug report.

Describe the bug
Pressing F8 when already viewing history crashes the app.
To Reproduce

Launch harlequin
Press F8 to view history screen
Press F8 again

Expected behavior
One of: (a) nothing, or (b) exit history screen, as when esc is pressed.
Actual behavior
The app crashes with a stack trace (hard to paste rich-rendered stack traces here) that concludes:
DuplicateIds: Tried to insert a widget with ID 'history_screen', but a widget HistoryScreen(id='history_screen') already exists with that ID in this list of children. The children of a widget must have unique IDs.

Contributing
Are you interested in contributing a fix?

[ ] Yes
[x] Maybe
[ ] No

Additional context
Please provide as much as you can, but don't waste your time if it's definitely not relevant or you don't know.
What is the output of harlequin --version?
$ rye run harlequin --version
harlequin, version 1.16.0

Installed Adapters:
  - duckdb, version 1.16.0
  - sqlite, version 1.16.0

What database adapter are you using with Harlequin? (Default is duckdb)
duckdb
What other options are you using when invoking Harlequin? (If you are using a profile, please add relevant items from your profile here).
Can you tell us more about your system?

Shell: zsh
Terminal: iTerm2
OS of the shell: macOS
OS of the terminal (if different from the shell):
Default locale / language for your OS/Terminal/Shell: en_US.UTF-8

[!TIP]
For example, for my system, these are:

Bash
Windows Terminal
Ubuntu 22.04 / WSL2
Windows 11
en_US/UTF-8

How did you install Harlequin?

[ ] pipx
[ ] conda/mamba
[ ] pip (global install)
[ ] pip + venv/virtualenv or pipenv
[ ] Poetry
[x] uv/Rye
[ ] other: _____________

Thanks for the report!


Add description field to Cargo.toml

Part of review of missing fields as part of #59, needs other authors also though.

Fixes #59


pull request for e2e test at 01/03/2017 14:45:08

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Is it possible to apply CSS style for the input box?

Hi @riophae ,
Thanks for the features provided already.
But it would be great if I had more control on the inputbox , so that I could apply some validation on the selected content and change its style for wrongly/not selected values
(Something like applying VeeValidate?)

(Wrong selected value - red border for input and red text)

Great suggestion! Actually this is a feature already in plan, just not implemented yet...

@riophae , Glad to know.
But do you have a quick solution for the same now :smile: ? Probably a workaround?
like you mentioned about text styling by
.vue-treeselect__menu {
  color: black;
}

Yeah, you can add your own CSS class and then apply new styles.
<template>
  <treeselect :class="treeselectClass" ... />
</template>

<script>
  export default {
    data() {
      fieldValid: false,
    },

    computed: {
      treeselectClass() {
        return {
          'treeselect-invalid': !this.fieldValid,
        }
      },
    },
  }
</script>

/* Note that, you better don't change the border color when the menu is open. */
.vue-treeselect.field-invalid:not(.vue-treeselect--open) .vue-treeselect__control {
  border-color: red;
}

.vue-treeselect.field-invalid .vue-treeselect__single-value {
  color: red;
}

Hi @riophae ,
Thanks :smile:

tip:

It would be cool to have a prop for this feature, because I have v-for for objects and each object has treeselect, so the solution proposed above makes invalid all treeselects.

tip:

This feature is not yet implemented?

Yeah, you can add your own CSS class and then apply new styles.
<template>
  <treeselect :class="treeselectClass" ... />
</template>

<script>
  export default {
    data() {
      fieldValid: false,
    },

    computed: {
      treeselectClass() {
        return {
          'treeselect-invalid': !this.fieldValid,
        }
      },
    },
  }
</script>

/* Note that, you better don't change the border color when the menu is open. */
.vue-treeselect.treeselect-invalid:not(.vue-treeselect--open) .vue-treeselect__control {
  border-color: red;
}

.vue-treeselect.treeselect-invalid .vue-treeselect__single-value {
  color: red;
}

I used this solution, while a better solution it's implemented


NextJS 14 app fails to build with - Cannot find module 'crypto'

Describe the bug
When running npm run build on a brand new NextJS app with a simple import from the @hashgraph/hedera-wallet-connect, the app fails to build.
To Reproduce
Here is a reproducible example of the bug on the build-issue branch:
https://github.com/justynspooner/hedera-wallet-connect-test/tree/build-issue
node: v22.3.0
NextJS: 14.2.4
@hashgraph/hedera-wallet-connect: 1.2.0

Run npm run build and see the console error with:
> Build error occurred
Error: Failed to collect page data for /
    at /dev/hedera-wallet-connect-test/node_modules/next/dist/build/utils.js:1268:15
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  type: 'Error'
}

This error is useless mainly because NextJS isn't surfacing the underlying issue 🤦  We discover the underlying issue when we run in dev mode as described below.
Expected behavior
The app should build successfully within a new NextJS app.
Additional Context
When running npm run dev the app does run, however there are lots of warnings in the console:
○ Compiling / ...
(node:71683) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.
(Use `node --trace-warnings ...` to show where the warning was created)
 ⚠<EMAIL_ADDRESS>Critical dependency: require function is used in a way in which dependencies cannot be statically extracted

Import trace for requested module:
<EMAIL_ADDRESS>./components/HederaWalletConnect.tsx

<EMAIL_ADDRESS>Critical dependency: require function is used in a way in which dependencies cannot be statically extracted

Import trace for requested module:
<EMAIL_ADDRESS>./components/HederaWalletConnect.tsx
 ⨯ Error: Cannot find module 'crypto'
    at webpackEmptyContext (/Users/justyn/dev/hedera-wallet-connect-test/.next/server/app/page.js:22:10)
    at __webpack_require__ (/Users/justyn/dev/hedera-wallet-connect-test/.next/server/webpack-runtime.js:33:42)
    at eval (./components/HederaWalletConnect.tsx:7:90)
    at (ssr)/./components/HederaWalletConnect.tsx (/Users/justyn/dev/hedera-wallet-connect-test/.next/server/app/page.js:279:1)
    at Object.__webpack_require__ [as require] (/Users/justyn/dev/hedera-wallet-connect-test/.next/server/webpack-runtime.js:33:42)
digest: "2480459522"
 GET / 500 in 1405ms
 ⚠<EMAIL_ADDRESS>Critical dependency: require function is used in a way in which dependencies cannot be statically extracted

Import trace for requested module:
<EMAIL_ADDRESS>./components/HederaWalletConnect.tsx

<EMAIL_ADDRESS>Critical dependency: require function is used in a way in which dependencies cannot be statically extracted

Import trace for requested module:
<EMAIL_ADDRESS>./components/HederaWalletConnect.tsx
 ⚠<EMAIL_ADDRESS>Critical dependency: require function is used in a way in which dependencies cannot be statically extracted

Import trace for requested module:
<EMAIL_ADDRESS>./components/HederaWalletConnect.tsx

<EMAIL_ADDRESS>Critical dependency: require function is used in a way in which dependencies cannot be statically extracted

Import trace for requested module:
<EMAIL_ADDRESS>./components/HederaWalletConnect.tsx
 ⚠<EMAIL_ADDRESS>Critical dependency: require function is used in a way in which dependencies cannot be statically extracted

Import trace for requested module:
<EMAIL_ADDRESS>./components/HederaWalletConnect.tsx

<EMAIL_ADDRESS>Critical dependency: require function is used in a way in which dependencies cannot be statically extracted

The Error: Cannot find module 'crypto' issue looks like it might stem from this code in the @hashgraph/sdk but haven't had time to look into this properly yet.
https://github.com/hashgraph/hedera-sdk-js/blob/482fc3b364bc744dd1b1dd3822baa08251a0130c/packages/cryptography/src/primitive/aes.js#L1
Current Work Around
To work around the issue, we can dynamically import the @hashgraph/hedera-wallet-connect as seen here:
https://github.com/justynspooner/hedera-wallet-connect-test/blob/main/components/HederaWalletConnect.tsx

thanks @justynspooner, with nextjs 14, I believe this error has to do with the change to server components and the complexity involved with that

Here's what our team has found as well:

The problem is caused by the attempt of Nextjs 14 to use the node version of the package on the frontend side.
There is no good way to solve this, but you can move the package import to an asynchronous function and call it like that.
Also in nextConfig you should add swcMinify: false
Example of use - init method can be imported into a component and used when pressing a button / initializing a component.

import { LedgerId } from "@hashgraph/sdk";

const init = async () => {
  const { HederaJsonRpcMethod, DAppConnector, HederaChainId } = await import(
    "@hashgraph/hedera-wallet-connect"
  );

  const projectId = "your wallet connect project id";
  const metadata: any = {
    name: "App",
    description: "My app",
    url: "https://example.com/",
    icons: ["https://example/logo.png"],
  };

  const dAppConnector = new DAppConnector(
    metadata,
    LedgerId.fromString("testnet"),
    projectId,
    Object.values(HederaJsonRpcMethod),
    [],
    [HederaChainId.Testnet]
  );
  try {
    await dAppConnector.init({ logger: "error" });
    await dAppConnector.openModal();

    //the rest of the logic goes here
    //...
    //...
  } catch (e) {
    await dAppConnector.disconnectAll();
    console.log(e);
  }
};
export default init;

@justynspooner @franfernandez20
from our call, does this problem go away if we just disable minification in esbuild?
If so, I think it makes perfect sense to disable minification as the consumer of the library should be tasked with minification

@justynspooner @franfernandez20
from our call, does this problem go away if we just disable minification in esbuild?
If so, I think it makes perfect sense to disable minification as the consumer of the library should be tasked with minification

Yes, in our case, we address the problems by avoiding minification. Disabling it makes sense to me.

This solution does work although, since the issue is with the minification of the @hashgraph/sdk another option might be to keep the minification but add the @hashgraph/sdk as a peer dependency instead:
package.json
…
"peerDependencies": {
    "@hashgraph/sdk": "^2.40.0",
},
…

This would have the added benefit of letting the consumer pick the version they would like to use rather than HWC enforcing it.
It would also mean that projects that already use the @hashgraph/sdk for their own services wouldn't have to bundle two different versions.

should be complete in v1.3.2

marking as closed, please feel free to re-open if something was missed

Looks like with 1.3.2 you need to npm i @walletconnect/modal (as well as go swcMinify: false) to get this to build correctly

Also looks like bumping up to 1.3.2 and then installing walletconnect/modal fixes all the above issues...


TR_Data

I don't have that mod, but it's generated as dependency.
Is there a way to remove one dependency from it?

same issue TR_Data.esm dosent exist its TR_Data.bsa

the work around i figured out is to
rename the the file to .esp
then use TESDTK
open the file
remove TR_Data.esm as a dependancy
the updated file should be in the TESDTK file location
move the esm out and reconvert to .omwaddon
and replace the old file in Documents\My Games\OpenMW\data


37 add expressions support

Turns out original solution that was using the javascript engine is out of date. Will have to find a different library to use.


update links to other Risteys versions to direct to the corresponding…

… page in selected version, unless endpoint page is not found.

Looks great, thanks! Just did a rebase to tidy up the commit messages before putting into master.


lamini apis broke on JAn 26, 2024

Hello,
I was testing a small RAG bot using lamini APIs and this morning the working code stopped working. Once I updated to lamini python lib version 2.0.8. I am getting API error 502
Bad Gateway for url: https://api.lamini.ai/v1/inference/embedding
Did the endpoint change? I dont see any document
site-packages\lamini\api\rest_requests.py", line 176, in make_web_request
raise APIError(f"API error {description}")
lamini.error.error.APIError: API error 502
with a larger txt file I was getting model name error.
Any guidance please?

Now I am getting
requests\models.py", line 1021, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 404 Client Error: Not Found for url: https://api.lamini.ai/v1/inference/embedding

Hello, thank you for reaching out! Could you try using version 2.0.7?
We are actively debugging some issues with 2.0.8 right now.

Following up, the endpoint is correct. Could you try again now?

Alright, endpoint is now working and the 2.0.8 is working now.

Great! We'd like to offer some Lamini credits as thanks for reaching out! Please email us at<EMAIL_ADDRESS>and we can send them over 🦙


Shift-JIS file encoding support

Hello,
I'm trying to use flutter desktop (windows), to open a text file encoded with Shift-JIS, but I didn't find any plugins or flutter feature to do it. Can you put more encoders inside default dart language?
The text file is a log file that contains Chinese and Japanese symbols.
Thank you

Please file Dart feature requests against the Dart project, which has its own repositories and issue trackers.
Closing as this is not within the Flutter project.


Articles publishing too early

https://kubernetes.dev/blog/2022/01/10/meet-our-contributors-india-ep-01/ was intended to publish next week; however, it is already live.
I assume this is a bug.
/kind bug

/remove-lifecycle rotten
This is still a bug

/assign

According to https://github.com/kubernetes/contributor-site/blob/master/.github/workflows/netlify-periodic-build.yml runs at 3:30 UTC daily (thus the blog uses UTC time). You have to use [**Hugo front matter**](https://gohugo.io/content-management/front-matter/) to establish the date you want to publish your article and set the time to 0330 UTC or earlier, then it should get published.
What's happening *now* that's outside of that current setup?

The config is set to build future, so it ignores the date and builds everything. The makefile just has to be updated to remove that command
https://github.com/kubernetes/contributor-site/blob/master/Makefile#L128

If anyone wants to work on this, I'm happy to explain in detail what the fix could look like.

Hi, Can I work on this issue?

Yes, feel free. You don’t need permission. Make sure you follow the steps in the contributor guide to get yourself ready to commit: https://www.kubernetes.dev/docs/guide/

Hi Folks,
I have figured out the solution.
Addressing the actual issue by @sftim  with article publishing early than the intended date. My assumption is there is no such feature to schedule posts - I looked up the GH workflow as pointed by @chris-short and it runs daily and just builds them, no special job or stage to schedule posts.
The only option is to understand how Hugo handles scheduling future posts. @mrbobbytables gave the right direction to check the hugo --buildFuture - which Hugo enforces to build the future posts for the enabled PublishDate blog posts (see here: https://gohugo.io/content-management/front-matter/)
So, the solution is to:

Disable the --build future flag in the production-build stage in the makefile &
Enable the PublishDate option for any later dated posts - needs support to change this process in the Markdown template
--build future could be untouched for other stages to preview the post.

It works. I tested locally for posts with the PublishDate (set to today+1) option for a test blog post, based on PublishDate Hugo will not publish and waits until the publishDate is today and the netlify build job runs to publish them. So this could be the option for us to schedule the post for a future date after the PR merge.
Please let me know about this solution. We can discuss this for further clarifications. Once all is good. I will provide the PR for the same.
(Thanks, @chris-short  for giving hints during the weekly SIG Marketing meeting)

Thanks @imnmo
Please open that PR with a proposed fix. I'll be happy to review it (might not be the same day; sorry about that).

@imnmo, could we help you get working on a fix? It's OK to ask for help.

@sftim unfortunately, I didn't find time past week to provide the PR. I will work on this issue today and create PR for discussions.

/assign @imnmo

/close

Can anyone help me close this issue? I can't, but the PR is merged and can be closed.

/close


Adjustable polling frequency as env variable

Reason: When using the client, longer queries will exit with a 429 due to a high number of failed attempts to get a result. Allowing the user to define the poll frequency is beneficial if he's aware that he's running a heavier query.
Still defaults to 1 as it currently is.

This is already possible with functions like run_query by passing in your "ping_frequency"
https://github.com/duneanalytics/dune-client/blob/main/dune_client/api/extensions.py
Not sure if it's a great idea to have library constants dynamically loaded. Would love to hear others thoughts...

Oh my bad, you're right. I completely missed this somehow.
Closing it, thanks 👍

I still think it would be cool to have this polling dynamically adjusted using some estimate mechanism on the query to avoid these 429's, but probably not worth the effort though

I think dune was having performance issues last week causing many queries to run a very long time. This seems to be resolved now. So you shouldn't even need to adjust the polling anymore.


Fails on windows 11, simply hangs, does nothing.

Your tool fails massively on Windows 11, I tried it, here is a live stream:
It simply seems to hang:
https://www.youtube.com/live/hfBqzlVsUmI?si=V7UxHNL9pzovCU3e

It does seem to be working, but it is weird as fuck, command line parameters not working, must use the keyboard to type in the secret and then press enter ?! Was it too much for you to document this properly ?!?!?!?!?!?!?!?!?:
https://www.youtube.com/live/MyyBbhV00rs?si=Mq4chy-ig6xJWYSa
I suspect it's done this way to make copy & paste hard ?!
By the way when trying to copy & paste into windows 11 terminal window it will reveal the contents of the text !
Don't do this on a live stream with your bitcoin private keys !

Laterrrrrr

I see a big down side to this 2FA...:

It's hard to remember the secret.
It's need special software.

So it's very "non-portable/convenient".
If I go to my laptop and went to login in there instead of my main PC where I setup it up I am screwed... and at the very least I need to "transfer" the secret and setup the software again !
It's easy to see how this can lead to leaks of secrets in case the laptop is "dumped" into the garbage.
A rememberable password is more secure in that regards... this is definetly a turn for the worse...

Also some further notes:

Caps lock must be on for the this application to work, this should be documented somewhere.
Enter must be pressed to acquire the number.
*************** should be shown when typing.... that would help to see the application has focus and is responding to key presses.

Hopefully, version 1.2 (along with its updated associated README) will provide better information regarding how to use the program. Let me know if it doesn't and I will try to address your complaint. Thanks.


Bot Infrastructure

Bots need work and testing to cover

bridgers
relayers
unhappy path resolution

only the watcher bots are currently implemented, with little testing

Let's track this under #39


Theme modernization

I'll try to make the website look more modern. Currently implemented:

[x] Make light the default theme. It's a lot more friendly and looks more modern. Come and fight me
[x] If the visitor has a dark theme preference set, dark theme will be chosen as their default theme
[x] Add theme switcher
[x] Make the theme switcher remember your choice after reloads
[x] Remove the ugly shadow from the header
[x] Custom scrollbar
[x] Replace the ugly gray backgrounds with some more friendly blue-ish gray
[x] Fix the missing un-themed black parts in the light theme
[x] Replace the filtered logo with a real, light-theme logo
[x] Make the header responsive for mobile users

How the light theme currently looks:

How the light theme currently looks after getting electronicboy came. It should look more like the new papermc forums:

On Mobile:

And the black theme:

Oh, and the navbar is also sticky now

I pretty much made most changes I wanted to make, especially in terms of the light mode & making Hangar look more like the Paper forums.
I think this should be merged, so further, smaller changes which improve some details can be made in smaller PRs. It's for sure not perfect at this state, but that way other people can base their changes pr these changes.


No state parameter specified in Linkedin Oauth

Hi, I'm struggling with an IE issue (oauth doesn't pass code from popup back to parent window) and trying to go back to basics for testing purposes.
I've setup your example app using Node on the backend. Google oauth works fine, but Linkedin fails with this error:
?error=invalid_request&error_description=You+need+to+pass+the+%22state%22+parameter
I believe everything is setup properly (not much to configure, really) - you can see it here: http://jg-sat-test.herokuapp.com/#/login
And the code is here:
https://ide.c9.io/jgentes/sat

I just confirmed that now happens with my production install after upgrading the bower install of satellizer to .9.0. It was working fine before the upgrade.
The good news is that the IE window now closes, and both Chrome/IE have the same issue of not passing the STATE. I'll keep looking into it.

Ok, I know this is a complete hack, but I needed a fix today and I am very happy to say that it has fixed the IE problem I was having, which was primarily due to having an old version of satellite in my bower.json file (vs. the new version in my package.json file for node)
Here's what I hacked to fix it, I hope it helps you figure out the problem:
I forced + '&state=STATE' into the querystring:
var url = defaults.authorizationEndpoint + '?' + oauth2.buildQueryString() + '&state=STATE';
Then I commented out the check for matching state in localstorage:
            return popup.open(url, defaults.popupOptions)               .then(function(oauthData) {                 if (defaults.responseType === 'token') {                   return oauthData;                 }                 /*                 if (oauthData.state && oauthData.state !== $window.localStorage[stateName]) {                   return $q.reject('OAuth 2.0 state parameter mismatch.');                 }                 */                 return oauth2.exchangeForToken(oauthData, userData);               });

I will take a look at it today. Although, I already know what is the issue. In v0.9 I have added support for OAuth 2.0 state. Previously, LinkedIn had a hardcoded state string by default which is probably no longer being sent because it is overriden by the generic state function for OAuth 2.0 providers.

@jgentes I just double checked the code and it looks good to me.
It turns out that for LinkedIn as you as you specified requiredUrlParams: ['state'], which it is by default, it does not matter whether you set state or not, it defaults to STATE (even if you remove state: 'STATE' from satellizer.js.

I have only tested this with Firefox and Chrome, but I would imagine IE would be no different.

Thank you for testing that.
The Heroku install I setup was using the example code you provided with no customization. Any idea why that's not working? Does it work for you when you test
http://jg-sat-test.herokuapp.com/#/
-James

On Jan 31, 2015, at 5:19 PM, Sahat Yalkabov<EMAIL_ADDRESS>wrote:
@jgentes I just double checked the code and it looks good to me.
It turns out that for LinkedIn as you as you specified requiredUrlParams: ['state'], which it is by default, it does not matter whether you set state or not, it defaults to STATE (even if you remove state: 'STATE' from satellizer.js.
I have only tested this with Firefox and Chrome, but I would imagine IE would be no different.
—
Reply to this email directly or view it on GitHub.

What would I do without Google Chrome debugger. I was able to debug the problem on the spot:

Find this line in this OAuth 2.0 service:
if (angular.isFunction(defaults.state)) {
  $window.localStorage[stateName] = defaults.state();
}

and change it to:
if (angular.isFunction(defaults.state)) {
  $window.localStorage[stateName] = defaults.state();
} else if (angular.isString(defaults.state)) {
  $window.localStorage[stateName] = defaults.state;
}

I have already made that change, but will push it out and bump the version tomorrow.

That's great, thank you - I appreciate all the time you put into this. It
has saved me a ton!
On Sat, Jan 31, 2015 at 9:18 PM, Sahat Yalkabov<EMAIL_ADDRESS>wrote:

What would I do without Google Chrome debugger. I was able to debug the
problem on the spot:
[image: screenshot 2015-01-31 21 16 07]
https://cloud.githubusercontent.com/assets/544954/5991121/7f1688c8-a98e-11e4-8210-9f623fa49867.png
Find this line in this OAuth 2.0 service:
if (angular.isFunction(defaults.state)) {
$window.localStorage[stateName] = defaults.state();
}
and change it to:
if (angular.isFunction(defaults.state)) {
$window.localStorage[stateName] = defaults.state();
} else if (angular.isString(defaults.state)) {
$window.localStorage[stateName] = defaults.state;
}
I have already made that change, but will push it out and bump the version
tomorrow.
—
Reply to this email directly or view it on GitHub
https://github.com/sahat/satellizer/issues/267#issuecomment-72352188.

Sure. Fixed in 0.9.2.


Ctrl+Shift+T for new info for VSCode

When I open after some time Code, there is some welcome tab with new data about VS Code. It after closing can't be opened, i.e. Ctrl+Shift+T does not work on those.

Make that possible to open in Reopen Closed Editor option.

/needsMoreInfo

Here it is: https://www.youtube.com/watch?v=-Hl4TBu0vY0
After closing this type of tab, it won't open with Ctrl+Shift+T, or via File menu.

Use the Help -> Welcome command to reopen the welcome screen.
The welcome page is not an editor so the reopen previous editor command is not expected to work


Security: Vulnerabilites - 2 High, 3 Moderate

Version
5.0.8
Reproduction link

minimatch ReDoS vulnerability
Exposure of Sensitive Information to an Unauthorized Actor in nanoid
Regular Expression Denial of Service in postcss
PostCSS line return parsing error

Environment info
 System:
    OS: Windows 10 10.0.19045
    CPU: (16) x64 12th Gen Intel(R) Core(TM) i7-1260P
  Binaries:
    Node: 14.21.3 - C:\Program Files\nodejs\node.EXE
  npmPackages:
    @vue/cli-plugin-unit-mocha: 5.0.8 => 5.0.8
    @vue/cli-service: 5.0.8 => 5.0.8
    vue: 2.7.14 => 2.7.14

Steps to reproduce
Run npm audit on any application using @vue/cli-plugin-unit-mocha and @vue/cli-service - Version 5.0.8
Output:
High minimatch ReDoS vulnerability
Package         minimatch
Dependency of   @vue/cli-plugin-unit-mocha [dev]
Path            @vue/cli-plugin-unit-mocha > mocha > glob > minimatch
More info https://github.com/advisories/GHSA-f8q6-p94x-37v3
High            minimatch ReDoS vulnerability
Package         minimatch
Patched in      >=3.0.5
Dependency of   @vue/cli-plugin-unit-mocha [dev]
Path            @vue/cli-plugin-unit-mocha > mocha > minimatch
More info https://github.com/advisories/GHSA-f8q6-p94x-37v3
Moderate        Exposure of Sensitive Information to an Unauthorized Actor in nanoid
Package         nanoid
Patched in      >=3.1.31
Dependency of   @vue/cli-plugin-unit-mocha [dev]
Path            @vue/cli-plugin-unit-mocha > mocha > nanoid
More info https://github.com/advisories/GHSA-qrpm-p2h7-hrv2
Moderate        PostCSS line return parsing error
Package         postcss
Patched in      >=8.4.31
Dependency of   @vue/cli-service [dev]
Path            @vue/cli-service > @vue/component-compiler-utils > postcss
More info https://github.com/advisories/GHSA-7fh5-64p2-3v2j
Moderate        PostCSS line return parsing error
Package         postcss
Patched in      >=8.4.31
Dependency of   @vue/cli-service [dev]
Path            @vue/cli-service > @vue/vue-loader-v15 >@vue/component-compiler-utils > postcss
More info https://github.com/advisories/GHSA-7fh5-64p2-3v2j
What is expected?
There should not be any vulnerabilities
What is actually happening?
There are existing vulnerabilities

aight cuh, you gotta switch the moderators with the crypto currency so that its 42 High and 3 moderate

At this moment last pr that was accepted is:
https://github.com/vuejs/vue-cli/pull/7324
merged by @sodatea into dev from dependabot/npm_and_yarn/loader-utils-1.4.1 on Nov 9, 2022
In README you can read that Vue CLI is now in maintenance mode, so you should migrate and remove this package.

is there a solution for this yet?


Initial etcd leader-election implementation.

Hi,
I've created a etcd implementation for spring-cloud-cluster leader-election along with autoconfiguration. I've also updated the tests and the leaderelection doc. Please pull.

/cc @pperalta

EtcdTests contains a test which needs a real server to execute and hence causing the CI check to fail. Do you have any suggestions as how to go about fixing that? I'm planning to write a mock client.

In spring-cloud-etcd I install etcd on travis: https://github.com/spring-cloud-incubator/spring-cloud-etcd/blob/master/.travis.yml

@spencergibb thank you! That made my life easy :)

Hi Venil,
Thank you for your contribution! Have you signed the Individual Contributor Agreement? This is a requirement for us to accept pull requests from contributors:
https://support.springsource.com/spring_committer_signup
Thanks,
Patrick

Hi Patrick,
Thanks for the quick review. I've just signed the individual contributor agreement.
I'll try implementing the candidate.onGranted mechanism as you've suggested; will keep you updated on it.

Hi @pperalta,
Apart from the changes you had mentioned, I'm also spawning a separate thread for publishing revocation events. This is to avoid publishing of those revocation events before the GrantNotification execution actually finishes.

I've added 2 more tests.

leader yielding
revocation of leadership in blocking candidate implementations

I've added 2 more tests.

leader yielding
revocation of leadership in blocking candidate implementations

Hi Venil,
I had imagined a design with two threads:
etcd leadership thread

attempts to acquire leadership
issues heartbeats
manages worker thread
raises leader events

worker thread

started by etcd leadership thread when leadership granted
interrupted by etcd leadership thread when leadership revoked

What do you think?

Hi Patrick,
In the design you've specified I would like to know how would leader yielding work?
As the worker thread itself would be the interrupting thread, we might not clearly understand when the worker thread would exit, if its a blocking candidate implementation. This is the reason I've kept another thread for interrupting the grantor and awaiting its termination.

I'll go through the designed you've proposed once again and let me see if I manage without the revoker thread.
Thanks,
Venil

Hi Venil,
I put together a quick prototype of what I was thinking and pushed it to here:
https://github.com/pperalta/spring-cloud-cluster/commit/55d1128e4b87d6ff5a8a44190bc1b20f85dc1551
Please have a look and let me know what you think.
Thanks,
Patrick

Hi Patrick,
Thank you so much for the example code. I've implemented it accordingly now.
Regards,
Venil

Hi Patrick,
Were you able to verify the final implementation?

Hi Venil,
I am still working on reviewing your changes. We had a long holiday weekend in the US so I haven't looked in a few days. I'll update the PR with comments soon!

OK.
Thanks Patrick.

Hi Patrick,
Could you please go through these commits; I've tried to fix the problems you've highlighted.
Thanks,
Venil

I pushed some suggestions to https://github.com/pperalta/spring-cloud-cluster/commit/bb6676c95e6b870570439b171a62dbfd7c0569e5 for your consideration. It includes changes based on my latest feedback, and it attempts to simplify exception handling.
I was also thinking that this project would be an interesting candidate to use the new Spring State Machine project. Since we are tracking multiple states with various if then else statements, using a state machine could simplify the implementation. This is your PR so there is no obligation, but I thought that I'd make the suggestion in case you're interested. 😊

Hi Patrick,
I've updated the implementation as per your changes. Also, I've modified the etcd configuration properties to accept connect uris in a comma separated fashion instead of a YAML list one.
I'm interested in implementing this feature using Spring StateMachine that you mentioned; I'm planning to create a separate PR for it. Should that be fine?

Yes, if you want to try with the state machine that would be great! In the meantime do you want me to merge this PR or do you want me to wait until you get a chance to try the state machine?

I would request you to please merge the PR while I work on the state machine base implementation.
Thanks Patrick for bearing with me :)
Regards,
Venil

Rebased, squashed, and merged as ed17e15dc823d37059800022fa843c251ca902c5. Thanks Venil!


Beta tech debt

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: # (issue, if applicable)
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[x] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

Summary by Sourcery
Refactor the component registration to utilize RuntimeUtils, update documentation to align with the new registration method, and update stylesheet links to newer versions in the README.
Enhancements:

Refactor the component registration process to use RuntimeUtils for registering custom elements, improving modularity and reducing code duplication.

Documentation:

Update documentation to reflect changes in the component registration process, ensuring consistency with the new implementation.

Chores:

Update stylesheet links in README.md to use newer versions of design tokens and web core stylesheets, ensuring compatibility with the latest design standards.

:tada: This PR is included in version 4.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Using curl calls on cloud servers

[x] Using curl calls on cloud servers by sub process
[x] Corporate Info API fix
[x] Added test cases for useSubProcess
[x] README updated

This PR no longer required since this PR(https://github.com/hi-imcodeman/stock-nse-india/pull/77) in place.


ENH: add MCS as another derivative-free optimization method

this code aim at providing a relevant first step in implementing the algoritm MCS (multilevel coordinate search)
Reference issue
relevant first step for #14750
What does this implement/fix?
implements the initialisation step  of the algorithm as written in the main paper:
https://www.mat.univie.ac.at/~neum/ms/mcs.pdf (pages 1 to 8)
Additional information
see the attached report for additional information about the algorithm and the implementation.
I'm planning to update this pull request in the following days
mcs.pdf

I think a good first step is defining __minimize_mcs(fun, x0) in your file, then adding it like Nelder-Mead here:
https://github.com/scipy/scipy/blob/3da3fb3de8beffc79797b7b62ea3c98cc8075d2e/scipy/optimize/_minimize.py#L668-L670
And adding a simple test like:
https://github.com/scipy/scipy/blob/2fc2458c2f46dd59d9900d500a8dddd5066a4d71/scipy/optimize/tests/test_optimize.py#L319-L348

@mnasini thanks for the contribution. Would you like to finish this up so it can be merged? There is a review waiting for you.

@AtsushiSakai that is a good suggestion for local minimizers, but since this is a global optimizer, there is precedent for having its own function. I think that's a good place to start. minimize has gotten somewhat cluttered, so I think it's OK for this to have the API that is best for this optimizer.
@mnasini thanks, and please let us know if you're still interested.

Did this get closed because of lack of activity? It's a shame if so as MCS would be great to have in scipy (or accessible in Python at all).


CMake updates for SDL3_image

No more 'd' debug suffix
Use the same so/dylib versioning scheme as used by SDL3
Install sdl3-image.pc
Add .editorconfig
Update ci to SDL3 -> fails because SDL3 is not available

Linux prefix:
prefix
├── include
│   └── SDL3
│       └── SDL_image.h
├── lib64
│   ├── cmake
│   │   └── SDL3
│   │       ├── Findlibjxl.cmake
│   │       ├── Findwebp.cmake
│   │       ├── SDL3_imageConfig.cmake
│   │       ├── SDL3_imageConfigVersion.cmake
│   │       ├── SDL3_image-shared-targets.cmake
│   │       └── SDL3_image-shared-targets-debug.cmake
│   ├── libSDL3_image.so -> libSDL3_image.so.0
│   ├── libSDL3_image.so.0 -> libSDL3_image.so.0.0.0
│   ├── libSDL3_image.so.0.0.0
│   └── pkgconfig
│       └── sdl3-image.pc
└── share
    └── licenses
        └── SDL3_image
            └── LICENSE.txt

MinGW prefix:
prefix
├── include
│   └── SDL3
│       └── SDL_image.h
├── lib64
│   ├── cmake
│   │   └── SDL3
│   │       ├── Findlibjxl.cmake
│   │       ├── Findwebp.cmake
│   │       ├── SDL3_imageConfig.cmake
│   │       ├── SDL3_imageConfigVersion.cmake
│   │       ├── SDL3_image-shared-targets.cmake
│   │       └── SDL3_image-shared-targets-debug.cmake
│   ├── libSDL3_image.so -> libSDL3_image.so.0
│   ├── libSDL3_image.so.0 -> libSDL3_image.so.0.0.0
│   ├── libSDL3_image.so.0.0.0
│   └── pkgconfig
│       └── sdl3-image.pc
└── share
    └── licenses
        └── SDL3_image
            └── LICENSE.txt

Looks good to me!

tga.c and tif.c changes should go in directly, independent of this pull request.

tga.c and tif.c changes should go in directly, independent of this pull request.

... and test/main.c changes too.

I think this is good to merge.
testimage fails (see https://github.com/libsdl-org/SDL_image/issues/324), but it does too for SDL2_image.

Great. Can you make similar changes for SDL_ttf and SDL_mixer?
Thanks!


Error creating an object with a connect input

Describe the bug
Trying to create a new object with a connection type fails.
To Reproduce
My datamodel is here:
datamodel.prisma.txt
My summarized database model here:
DDBB_script.txt
I have customertype entities from 1 to 4, (4 inserts included in the above script). When I try to send a mutation to create a new customer, it fails.
Mutation
mutation CreateCustomer($customer: CustomerCreateInput!) {
  createCustomer(data: $customer) {
    code
  }
}

Variables
{
  "customer" : {
    "code": "TEST",
    "name": "Test Test",
    "type": {
      "connect": {
        "id": 3,
        "code": "PBUY"
      }
    },
    "sapCode":"TESTSAP"
  }
}

I recieve this error from docker.. It's trying to insert null in type field..
org.postgresql.util.PSQLException: ERROR: null value in column "type" violates not-null constraint
  Detail: Failing row contains (4, TEST, TESTSAP, Test TEst, null, 2019-03-05 11:32:15.134+00, 2019-03-05 11:32:15.134+00).
at org.postgresql.core.v3.QueryExecutorImpl.receiveErrorResponse(QueryExecutorImpl.java:2433)
at org.postgresql.core.v3.QueryExecutorImpl.processResults(QueryExecutorImpl.java:2178)
at org.postgresql.core.v3.QueryExecutorImpl.execute(QueryExecutorImpl.java:306)
at org.postgresql.jdbc.PgStatement.executeInternal(PgStatement.java:441)
at org.postgresql.jdbc.PgStatement.execute(PgStatement.java:365)
at org.postgresql.jdbc.PgPreparedStatement.executeWithFlags(PgPreparedStatement.java:155)
at org.postgresql.jdbc.PgPreparedStatement.execute(PgPreparedStatement.java:144)
at com.zaxxer.hikari.pool.ProxyPreparedStatement.execute(ProxyPreparedStatement.java:44)
at com.zaxxer.hikari.pool.HikariProxyPreparedStatement.execute(HikariProxyPreparedStatement.java)
at com.prisma.connector.shared.jdbc.SharedSlickExtensions.$anonfun$insertReturningGeneratedKeysToDBIO$1(SharedSlickExtensions.scala:28)
at com.prisma.connector.shared.jdbc.SharedSlickExtensions.$anonfun$jooqToDBIO$1(SharedSlickExtensions.scala:62)
at slick.jdbc.SimpleJdbcAction.run(StreamingInvokerAction.scala:70)
at slick.jdbc.SimpleJdbcAction.run(StreamingInvokerAction.scala:69)
at slick.basic.BasicBackend$DatabaseDef$$anon$2.liftedTree1$1(BasicBackend.scala:275)
at slick.basic.BasicBackend$DatabaseDef$$anon$2.run(BasicBackend.scala:275)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at java.lang.Thread.run(Thread.java:748)

Expected behavior
Create a customer succesfully linking its type to an existing type.
Versions (please complete the following information):

Connector: Postgres
Prisma Server: 1.27.4
prisma CLI: prisma/1.29.0-alpha.7 (windows-x64) node-v8.9.4
OS: Windows 10
Prisma client: 1.27.4

Same issues here https://github.com/prisma/prisma/issues/3319

I came ended up using the same "fix" as mentioned here https://github.com/prisma/prisma/issues/3319#issuecomment-443395291
While not ideal, it may help you in the interim.

But if I have to remove the non null contrainst I will get inconsistent data, for sure! Is it a bug from october 2018 and is not fixed yet? 🤔
Thanks!

Hi @marticrespi
I believe this is something that the new datamodel 1.1 will fix so I think that is why it is not being updated. The new datamodel is under a prototype flag in the current release. Can you try that with that? https://github.com/prisma/prisma/issues/3233#issuecomment-465513624
For now I am going to close this issue as a duplicate of #3319

@pantharshit00 I have already rawAccess = true. Only works if you remove the FK from database.

Having the same problem


React v15 complains about props being passed to native  element

As soon as I upgraded to React v15 I started getting this error:

Which I can only assume is because of this line:
https://github.com/glittershark/reactable/blob/6da047162b8bfe37dde79ebe91190d974ccb79c5/src/reactable/table.jsx#L503

This was a new warning added in v15.2 if I'm not mistaken.

This is new at v15.2:
3 days ago: https://gist.github.com/jimfb/d99e0678e9da715ccf6454961ef04d1b

Yea, I should have been more explicit in my initial post: I think it is a bit risky to pass the rest of the props to the native table element, especially only being filtered by a small blacklist.  I think a whitelist would be more appropriate, taken from here: http://www.w3schools.com/tags/tag_table.asp .  The table doesn't even have that many unique attributes.

@dcurletti The blacklist in filterPropsFrom  is recycled between all the other tags as well, so to replace it with a whitelist you'd probably need to create different whitelists per tag.
I took a shot at expanding the blacklist with as many props as I could find in #293

Just for everyone's attention, still getting the same errors with React v15.3, too

@hassankhan Are you using master? Because the fix hasn't been put into a release yet.

Yep, I noticed that so I opened #305 😄

Gracias.


Revamp business profiles

Business profiles, like mine for example, are pretty bare bones. They list the company name, the person associated, a sentence or two, and a photo.
Even though these profiles aren't publicly linked to, they can be accessed when a developer receives a message. So I'd like to add a bit more info to make it easier for the business to share details about their company and contact.
As a first pass, I'd love to see the following fields added:

Bio - along with hero, a bit more info about the company
URL - where can I find more information about the company?
Contact role - what does the person do at the company?

There's a big design decision to make before adding these fields. I don't want the business profile to look just like the developer one. But I also don't want to design an entirely new screen.
I'm open to ideas!

After thinking on this a bit more I think we can get a v1 of this out without much more design requirements.
A PR addressing this issue should:

[ ] Add a website field to the business form and public view
[ ] Add a role field to the business form and public view

We will not be addressing bio at this time. If you are unsure of where/how to put the fields then toss them at the bottom and submit a PR. I can take over for the design!

First pass PR here: https://github.com/joemasilotti/railsdevs.com/pull/397

Awesome - all yours! I'll comment on the PR.

Closed, for now, via #397.


Remove local_run in Experiment

@martinwicke this changes the API by removing "local_eval_frequency", can you review, please?

This has been deprecated for months so I think it would be safe to remove it. It would be better if there's internal deprecation tracking for removal dates and the person who's responsible for removal on time.

Yes, the dates were a mistake.

@martinwicke does this mean you approve of this change or not :-)

Taking the train. Will update when I've arrived.

@tensorflow-jenkins Test this please

Ping! Could this be merged?

 We're planning some reorganization of this anyway, so let's leave it as is. Removing things is fiendishly hard. Better not to try without access to the internal tests. :(