Do not render < 1/1 > in Ghost Text hover when there is only one suggestion

Where there is only one ghost text suggestion there is no need to render the < 1/1 > part.
Could we not render it in this case? Or you only find out async, and you think it might cause too much flicker?

Or you only find out async, and you think it might cause too much flicker?

If you want to remove the first 3 UI elements if there is only one suggestion, then there might be significant flicker.
If you only want to hide the label, there might be a lot of empty space.

I think only this option makes sense:

Unless we show the hover only after the provider sent us all the results, but then it might take a while until the hover shows.

The hover already shows with a hover delay. If it needs an additional delay it might not be noticeable. Can we configure how fast the hover is triggered for ghost text, if yes we could decrease this time and with the delay we introduce the experience might be the same.

if yes we could decrease this time

This would increase the number of accidental hovers though! (And thus increasing the number of requesting all suggestions)

Good idea.
Is it worth it - not sure. I think the other issue I filed is more important as it is clearly a bug https://github.com/microsoft/vscode/issues/189058
I leave this one up to you to decide. Feel free to close as designed.

If the providers answer in that time, I don't show the label if there is only one suggestion. Otherwise, I show the label.

Thinking about it, this could be perceived as buggy (if it only sometimes shows).

Feel free to close as designed.

🫡


Adding .ipynb input support

like
rubydown -i hoge.ipynb

This will

nbconvert hoge.ipynb to hoge.md
run hoge.md and generate hoge.html

I started this in ipynb-support branch


WIP: Component logger debug level setting

Client side implementation for setting component log levels for planning and vision.

@ziyan Okay, this branch should be ready to be merged.


Breakpoints no longer work in VSCode 1.73.x

Breakpoints (both regular breakpoints and break-on-exceptions) fail to work in Visual Studio Code 1.73.0 and 1.73.1
Not sure if this is the cause of the break, but there do appear to be changes to the debug adapter protocol in 1.73? Not sure if this is something that needs adjustment in this extension, or something needing updating in the simulator itself.
https://code.visualstudio.com/updates/v1_73#_debug-adapter-protocol
For now, workaround is to roll back VSCode to 1.72 (September 2022) or earlier.

Thanks for reporting this! I am looking into it.
FWIW It looks like breakpoints do still work in 1.73.x, but only if you set them after the debug adapter session has been launched. It looks like VSCode is not sending the initial breakpoints to the Playdate Simulator for some reason.

Fixed by a6e1eaebc3bcd8906ef4c7268950d8b92b1b5abf

Confirmed working for me in 1.73.1, thanks so much! 💜


Add support of point data

Support point data according to CF convention. Only works once the pull request TUW-GEO/pygeobase/pull/25 of pygeobase has been merged (otherwise lon, lat information is missing).

Could you fix Python 3 support?

yes, I wasn't aware of the fact that iteritems was removed in python 3

Also itervalues was removed.

I see, learned something today.

That looks cool. Could you add it to the changelog?

done. I have to say that we are not really consistent about the change log style and file type.


Request: TypeScript support (~C# parity)

Hi - we have a project using Next.JS and would love to leverage SK on the backend.
A full TypeScript port (ideally 1:1 parity with C#) would be truly valuable and appreciated.
As part of this, a NPM distribution of the TS SDK (i.e. https://registry.npmjs.org).
Happy to discuss/explain the scenarios more.
Thanks!

@jsegeren , parity would be great for C#.  If you want to help with that effort we would love to get PRs from you in this branch - https://github.com/microsoft/semantic-kernel/tree/experimental-typescript

+1 on this feature, really interesting! I'll try to participate in the typescript branch

+1

Closing since we currently do not have plans to add a TypeScript version of the SDK. This may change in the future.


Bug report: CSV columns not written if it is not in the header row

Priority
(Medium) I'm annoyed but I'll live
Description
For example, if I call m365 aad user list -p id,accountEnabled,department -o csv and the department field is only set for some users and not the first user returned, the CSV writer discards the column entirely.
So if I run it with --debug, I can see that it correctly retrieves the property:
...
  {
    "id": "some-uuid-here",
    "accountEnabled": true,
    "department": null
  },
  {
    "id": "some-other-uuid",
    "accountEnabled": false,
    "department": "Department Name"
  }
...

Then it only writes the following CSV:
id,accountEnabled
some-uuid-here,true
some-other-uuid,false

And the department is stripped because the field isn't written to the header row. I can't set this ahead in the CSV as a workaround either, since it writes to the file instead of appending.
Steps to reproduce
Call the m365 aad user list command with properties that only exists for some users but not the others.
Expected results
A full CSV with all the requested fields in the output.
Actual results
The CSV only partially writes without some fields that aren't populated for all users.
Diagnostics
No response
CLI for Microsoft 365 version
v7.1.0
nodejs version
v21.1.0
Operating system (environment)
macOS
Shell
bash
cli doctor
No response
Additional Info
No response

@jackylamhk thanks for reporting this.
@Adam-it, I feel like we have a similar problem here like we had a while back with tenant people profilecardproperty list. If we have a list of objects and the first one has an empty property, in this case, department, it wouldn't be added as a column in the CSV output. What do you think?

I have a feeling this might be an issue with the CSV writer - what if we change the default behavior to print out fields that are null too?

I believe you are correct @jackylamhk!. It seems to be csv-stringify indeed.
However, I think we can only provide the columns ourselves, or allow it to autogenerate. There don't seem to be extra options to influence the autogeneration of headers.
We could of course pass in the headers ourselves. We do have the object at our disposal when calling csv-stringify.

Correction, I've just looked further than my nose is long, and it turns out that our removal of object-properties, removes null-properties as well:
    if (typeof l[x] === 'object') {
        delete l[x];
    }

null appears to be an object in JavaScript. We can adapt the check there to fix this problem.

It's going to be fun to update all CSV output samples ☺

We don't necessarily have to do that, this issue only occurs in situations where a property of the first object on the array is null.
Also, I learned from Vesa: "be improvenist, not perfectionist" 😄  the idea is clear...

Good observation, and it's indeed caused by the first object in the array that we use to determine properties to include in the output. We could update this logic and only look at their properties and not at their values, excluding complex objects which we can't render in a csv output in a meaningful way

Already fixed it @waldekmastykarz, it was caused by the complex object check...


Elm 0.17

Elm 0.17 came with some syntax changes.

See #1172

Thanks. I searched the issues, but not the PRs.


Replace vl-hog with a modern C++ solution

vl-hog has really good performance and it's hard to replace. But I don't like to include C headers and I would really like to replace it with a header-only, modern C++ solution. It's just hard to find one, and rewriting vl-hog would be a lot of work.
I managed to include vl-hog as "header-only" by #include'ing vl-hog's .c file, and I had to make some casts in its code explicit in the process to get it to compile. I hope what I did is not problematic.

Did you try the opencv HOG implementation ? since you already use opencv...

I've had quite bad performance with OpenCV's HOG implementation when I tried it back then. It was either accuracy, speed, or both.


The role selector is not precise enough

The role selector is not precise enough.
I want to activate the "Contributor" role and get the "Azure Kubernetes Service Contributor Role".
$ az-pim-cli list
== Service - prod ==
	 - Azure Kubernetes Service Cluster Admin Role
	 - Azure Kubernetes Service Cluster User Role
	 - Azure Kubernetes Service Contributor Role
	 - Contributor
	 - Owner
	 - Reader
	 - Virtual Machine Administrator Login
	 - Virtual Machine User Login

$ az-pim-cli activate -s "Service - prod" -r "Contributor" --reason "test" -d 240
2024/06/11 12:01:13 Activating role 'Azure Kubernetes Service Contributor Role' in subscription 'Service - prod' with reason 'test'
...

Thanks for the report! This should be resolved as of #12


[Fix-3077][ui] Fix the edit duplicated name verify

What is the purpose of the pull request
colse(#3077 )
Fix the edit duplicated name verify
Brief change log

Change the verify for edit and create process definition

Verify this pull request

Manually verified the change by testing locally.

Your pr caused the workflow to be uneditable, please refer to the screenshot below to fix


Some question about RANSAC

Hello,in PFNet ,we should use ransac to sample point from PF, how u implement it , i use the kornia the cuda out of memory

Hi @Acmenwangtuo ,
exactly as you say, For training, we don't really need the real value of homography, so I only return delta_hat as a prediction of corner points of the image: https://github.com/NeurAI-Lab/biHomE/blob/main/src/heads/NoOpHead.py#L34
However, we need to use some method to find the homography out of the perspective field for testing, and here comes the RANSAC algorithm: https://github.com/NeurAI-Lab/biHomE/blob/main/src/heads/NoOpHead.py#L76
In general, the Zeng method takes much more GPU memory than other models implemented in this repo.
Best,
Daniel

Hi @Acmenwangtuo ,
I am closing the issue for now. Feel free to reopen if you'd like.
Best,
Daniel


Fluid Pipes fill both Drums with Liquid.

I sent a Gas Collecting Mission to get some Oxygen. When the rocket returned the oxygen was unloaded with the Rocket Fluid Unloader and then the gas was piped using Liquid Pipe into a drum containing Oxygen also connected to the pipe was drum containing Nitrogen. After unloading both drums contains 32 buckets more gas.
AdvancedRocketry-1.10.2-1.2.0c
LibVulpes-1.10.2-0.2.3-universal

What mod was the drum from?  And was it AR pipes or pipes from another mod that you used?

The drums are from Extra Utilities 2 and I was using Liquid Pipes from AR. I used Iron Drums and the hold 256 buckets, so fairly good for a space station. Normally you can't mix liquids in drums like this but I don't use them very often.

Alright, will try and replicate on my end

should be fixed in the latest release


Adds vagrant runner

This PR is not ready for review.

Hello @JayjeetAtGithub! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file cli/popper/gha.py:

Line 546:1: W293 blank line contains whitespace
Line 554:51: E225 missing whitespace around operator
Line 554:80: E501 line too long (81 > 79 characters)
Line 568:1: W293 blank line contains whitespace
Line 580:1: W293 blank line contains whitespace
Line 610:26: E203 whitespace before ','
Line 610:27: E231 missing whitespace after ','

@ivotron While adding VagrantRunner,  in implementing the --reuse feature, we need to check the existence of a VM. For this, we need a vagrant cli functionality vagrant global-status which is absent in python-vagrant. I forked python-vagrant and implemented the feature and then used it to build and test VagrantRunner temporarily. Also, this is due to which the builds are failing. So, what shall we do now, continue with the forked lib or is there any other way around? Please have a look. Thanks.

@ivotron virtualbox installation is failing in Travis CI even after several different attempts. Some answers say that we can't run virtualbox in Travis CI. Can you please help me out with this?

hey @JayjeetAtGithub, thanks a lot for working on this. We have also tried to install virtualbox on Travis in the past without success. If this gets too much of your time, then I'd say we leave it out for now.
Regarding the changes done to the python-vagrant package, we can open a PR on the upstream repo. We can point to the fork in the setup.py file in the meantime (hopefully they'll merge the PR).

@ivotron Does that mean we won't have a test vagrant in the codebase and VagrantRunner would depend on local machine tests?

@JayjeetAtGithub , yes, that would mean we have to test manually. Alternatively, we could also see if we can use the docker provider for vagrant. In this way we test the popper/vagrant interaction.

@ivotron Can we clone the remote actions within the runners ?. This is needed to reuse the environment in case of vagrant which otherwise gets destroyed during erasing the prev clone and recloning. We would like to check whether the prev clone is present and if present we would skip cloning.

@ivotron Can you also give me some idea how to set the env vars inside the vagrant VM's ?
I was doing vagrant ssh --command "echo export VAR=value >> ~/.bashrc", but this was'nt working.

@ivotron Can you also give me some idea how to set the env vars inside the vagrant VM's ?
I was doing vagrant ssh --command "echo export VAR=value >> ~/.bashrc", but this was'nt working.

I believe this can be done by modifying the Vagrantfile, so that we add a shell provisioner and define the environment variables there.

@ivotron Can we clone the remote actions within the runners ? This is needed to reuse the environment in case of vagrant which otherwise gets destroyed during erasing the prev clone and recloning. We would like to check whether the prev clone is present and if present we would skip cloning when the --reuse flag is passed. In this way, we can preserve the vagrant environment for reusing.

This is related to #550 and #652, in the sense that if those two were already solved, the problem you described would not arise.

@ivotron Please review this. The proper functioning of this feature depends on #653. Thanks.


fix jobsub_q -total command, which currently isn't working

Herb reported in INC000001152470 that jobsub_q -totals was not working for him. I confirmed that this is the case. Steve Timm brought up that the flag should really be -total, since condor_q removed the "s" from the flag a few years ago. I also tried jobsub_q -total and that did not work.
Shreyas confirms:
jobsub_q -G uboone -total
Attempting to get token from https://htvaultprod.fnal.gov:8200/ ... failed
Attempting kerberos auth with https://htvaultprod.fnal.gov:8200/ ... succeeded
Attempting to get token from https://htvaultprod.fnal.gov:8200/ ... succeeded
Storing vault token in /tmp/vt_u10610
Storing bearer token in /tmp/bt_token_uboone_Analysis_10610
JOBSUBJOBID                             OWNER           SUBMITTED     RUNTIME   ST PRIO   SIZE  COMMAND

[sbhat@fifeutilgpvm01 ~]$ condor_q -G uboone -total

-- Schedd: jobsub04.fnal.gov : <<IP_ADDRESS>:9615?... @ 06/01/23 09:17:41
Total for query: 11890 jobs; 1 completed, 0 removed, 5277 idle, 5643 running, 969 held, 0 suspended
Total for all users: 11890 jobs; 1 completed, 0 removed, 5277 idle, 5643 running, 969 held, 0 suspended

-- Schedd: jobsub05.fnal.gov : <<IP_ADDRESS>:9615?... @ 06/01/23 09:17:42
Total for query: 8144 jobs; 2 completed, 0 removed, 3182 idle, 4400 running, 560 held, 0 suspended
Total for all users: 8144 jobs; 2 completed, 0 removed, 3182 idle, 4400 running, 560 held, 0 suspended
...

I'm trying to decide what to do about jobsub_q --totals... If we just flag it as a formatting option, it's a Very Small Change and we get totals for query and totals for all separately for all the schedds, which sort-of meets the definition, but  I suspect is not what people want.  Otherwise we could catch it at the jobsub level, and after we run the regular condor_q  calls without -totals, we could run them again with -totals and collect the output and combine them, which could give us inconsistent results if jobs change status while we're running; or third we could make --totals run the condor_q's into a pipe and count the job statuses ourselves on the way past...

So even though we had #433, I realize that one of the new unit tests requires a data file that was never checked in, tests/data/jq_out.txt.  Could you please push that to the repo, @marcmengel ?  I'm OK if you do a direct push to master.

File added directly on master.   Sorry for missing that...

Closing this out.


ensure that home directories of users inherit User::Home from /etc/skel

Whatever tools copy the files must also copy extended attributes.

Noted.  Thanks.

Invalid: we removed support for the "User" domain from the layer.


Custom 404 page

Looks like currently there isn't a solution for global 404 pages in app-dir, will keep an eye on the documentation for this becoming available and implement when it does


Runnable trait for Systems and Schedules

Consider adding a Runnable trait implemented on Systems, ParSystems, and Schedules. Then, World::run() could operate on a Runnable generic type, allowing the method to be reused for more than just Schedules.
This will also allow Systems to be usable without the parallel feature enabled.

Not sure if this will actually be possible currently, due to the possibility of a type implementing both System and ParSystem. For now, there may just have to be a separate method for systems, another for par_systems, and a third for schedules.


fix: Retain "None" Option in Technical User Setup After Submission

Description

Context: App Release Process -> Step 4: Technical Integration -> Technical User Setup.
Change: Added an if condition in useEffect to verify if the user has assigned roles. If no roles are found, the "None" option is automatically selected.

Why

Issue: The "None" option in the Technical User Setup was not retaining its selection after submission.
Solution: This fix ensures that the "None" option remains selected when the user revisits the "Technical Integration" step, providing a more consistent and user-friendly experience.

Issue
Github Issue: #1036
Checklist

[x] I have followed the contributing guidelines
[ ] I have performed IP checks for added or updated 3rd party libraries
[x] I have created and linked IP issues or requested their creation by a committer
[x] I have performed a self-review of my own code
[x] I have successfully tested my changes locally
[ ] I have added tests that prove my changes work
[x] I have checked that new and existing tests pass locally with my changes
[x] I have commented my code, particularly in hard-to-understand areas

https://github.com/user-attachments/assets/acfee86c-72a7-430e-9325-8956adbbd7e9

Hi @oyo & @manojava-gk,
I have covered your feedback, Can you review it again? 🙂
Thanks!


[ENH] Adding an interface of ngboost to skpro

Reference Issues/PRs

Enhances #135
What does this implement/fix? Explain your changes.

This PR has added a way to use ngboost regressor in skpro.
Tested it using the following piece of code
import pandas as pd
import numpy as np
from sklearn.model_selection import train_test_split
from sklearn.metrics import mean_squared_error

#Load Boston housing dataset
data_url = "http://lib.stat.cmu.edu/datasets/boston"
raw_df = pd.read_csv(data_url, sep="\s+", skiprows=22, header=None)
X = np.hstack([raw_df.values[::2, :], raw_df.values[1::2, :2]])
Y = raw_df.values[1::2, 2]

X_train, X_test, Y_train, Y_test = train_test_split(X, Y, test_size=0.2)

ngb = NGBoostRegressor()._fit(X_train, Y_train)
Y_preds = ngb._predict(X_test)
Y_dists = ngb._pred_dist(X_test)

# test Mean Squared Error
test_MSE = mean_squared_error(Y_preds, Y_test)
print('Test MSE', test_MSE)

# test Negative Log Likelihood
test_NLL = -Y_dists.logpdf(Y_test).mean()
print('Test NLL', test_NLL)

Does your contribution introduce a new dependency? If yes, which one?

NGBRegressor has been added from the ngboost package.
What should a reviewer concentrate their feedback on?

What modifications need to be done and some help on how to add get_test_params function and the parameters to be sent via FITTED_PARAMS_TO_FORWARD.
Did you add any tests for the change?

No
Any other comments?

PR checklist

For all contributions

[x] I've added myself to the list of contributors with any new badges I've earned :-)
How to: add yourself to the all-contributors file in the skpro root directory (not the CONTRIBUTORS.md). Common badges: code - fixing a bug, or adding code logic. doc - writing or improving documentation or docstrings. bug - reporting or diagnosing a bug (get this plus code if you also fixed the bug in the PR).maintenance - CI, test framework, release.
See here for full badge reference
[x] The PR title starts with either [ENH], [MNT], [DOC], or [BUG]. [BUG] - bugfix, [MNT] - CI, test framework, [ENH] - adding or improving code, [DOC] - writing or improving documentation or docstrings.

For new estimators

[ ] I've added the estimator to the API reference - in docs/source/api_reference/taskname.rst, follow the pattern.
[ ] I've added one or more illustrative usage examples to the docstring, in a pydocstyle compliant Examples section.
[ ] If the estimator relies on a soft dependency, I've set the python_dependencies tag and ensured
dependency isolation, see the estimator dependencies guide.

I have implemented the _predict_proba after converting the return type of ngboost's normal distribution to skpro Normal distribution it must now be extended to other distributions that ngboost supports and supported by skpro. I will also work on implementing predict_interval and predict_quantile next. In the meantime I will also try to make the conversion of the return types into a function.
The current output upon calling
Y_pred_proba = ngb._predict_proba(X_test)
print(Y_pred_proba) is
[iter 0] loss=3.6479 val_loss=0.0000 scale=1.0000 norm=6.7178
[iter 100] loss=2.6977 val_loss=0.0000 scale=2.0000 norm=4.9421
[iter 200] loss=2.1456 val_loss=0.0000 scale=2.0000 norm=3.3556
[iter 300] loss=1.8761 val_loss=0.0000 scale=1.0000 norm=1.4400
[iter 400] loss=1.7477 val_loss=0.0000 scale=1.0000 norm=1.3337
<ngboost.distns.normal.Normal object at 0x736de3d40f50>
Normal(columns=RangeIndex(start=0, stop=1, step=1),
       index=RangeIndex(start=0, stop=102, step=1),
       mu=             0
0    16.962767
1    21.572000
2    16.014035
3    30.217625
4    19.818455
..         ...
97   11.209679
98   25.100160
99    7.723768
100  19.720101
101  24.292480

[102 rows x 1 columns],
       sigma=            0
0    1.206229
1    1.079737
2    1.241355
3    1.223454
4    1.187693
..        ...
97   1.292326
98   1.216184
99   1.304442
100  1.106799
101  1.176687

[102 rows x 1 columns])
Test MSE 16.048500328511995
Test NLL 4.412747994731287

Please let me know if it looks good or something needs to be changed.

Also, you should add ngboost as a soft dependency to pyproject.toml, and via the python_dependeices tag to the estimator.

@fkiraly could you please review the updates I have made to the code and if anything else needs to be modified.

Nice, thanks! Really great addition!
I've started the CI tests to see whether they pass - I assume you have checked with check_estimator already.

Could you please help me with the check_estimator. Upon running
from sklearn.utils.estimator_checks import check_estimator
check_estimator(NGBoostRegressor())

It is throwing a TypeError: input must be a pd.DataFrame.

I have resolved the shape mismatch error using  ravel() which the check_estimator suggested to apply on y. I have also changed the kwargs["columns"] to be y.columns now .
But some tests are failing in the check_estimator although it is running it without any errors.
FAILED: test_constructor[NGBoostRegressor]
FAILED: test_set_params_sklearn[NGBoostRegressor]
FAILED: test_fit_does_not_overwrite_hyper_params[NGBoostRegressor-ProbaRegressorBasic]
FAILED: test_fit_does_not_overwrite_hyper_params[NGBoostRegressor-ProbaRegressorXcolMixIxYnp]
FAILED: test_fit_does_not_overwrite_hyper_params[NGBoostRegressor-ProbaRegressorSurvival]
FAILED: test_input_output_contract[NGBoostRegressor]

Could you please help out as to why these are failing.

Could you please help out as to why these are failing.

Of course, will look at it later today.
Can you also describe what you have found out in debugging these failures? You can use raise_exceptions=True to get an error raise for debugging.

Can you also describe what you have found out in debugging these failures? You can use raise_exceptions=True to get an error raise for debugging.

I have now resolved all the errors except test_fit_does_not_overwrite_hyper_params.
And I have found it is being caused due to initialisation of the base learner/estimator to None in init but  i am replacing it with DecisionTreeRegressor later on if estimator is default None.
Can you please suggest what i can do to solve this issue. I tried doing the initialisation to DecisionTreeRegressor which is being done in _fit in init itself if it is None but it then gives test_constructor and test_sklearn_params asssertion error.

See above - look at the error messages, they should tell you precisely what the problem is.
The parameters written to self must always remain unmutated and identical to what was passed in __init__.

See above - look at the error messages, they should tell you precisely what the problem is. The parameters written to self must always remain unmutated and identical to what was passed in __init__.

Yes I have now made it self.estimator_  to be passed to NGBRegressor so it is keeping the __init__ params intact and check_estimator is giving all tests passed!

Great! Looks excellent, I will give it a review once the tests pass. From superficial inspection, looks ready to merge!

Hm, it looks like check_estimator tests are indeed passing! Very nice!
Well, with exception of the sixth (index 5) parameter set - do you have an idea why that parameter set specifically is failing? Seems nans are being produced.

Nice, thanks!
This is ready to merge, could you open the issue on the skpro tracker to investigate the TDistribution failure? With code if you have it, otherwise just a quick note.

I have now opened #291 along with the code causing the issue.


[MISC] Move section on sessions.tsv file: longitudinal files -> modality agnostic files

fixes #796

[x]  moves info about the sessions.tsv to the modality agnostic page.
[x]  moves info about session entity to common principle page.
[x]  restyle some of the filename templates in the modality agnostis page to make them match that of those that macro generated.

For reviewers
These edits are simple copy paste and might affect the "flow" of the text.
Especially for the common principle page, I tried to add it in the place that made most sense ("filename structure") but some of the information added there relates to folder structure, so this is not ideal.
Before moving on further on this I would be curious to hear your thoughts.

Thanks @Remi-Gau


Tutorial; expand wildcard types section

We should also discuss what happens when you return something with a wildcard type.

There's a newer issue for this, #5038, so I'm closing this one in favor of that one.


Nick: Watch cloudiffic.com

Nick requests the watch of the watch_keyword cloudiffic\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

cloudiffic\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by cigien in Charcoal HQ


Upgrade Redpanda image to v21.11.1

This PR updates the image to Redpanda v21.11.1, which brings the rpk group command:
Usage:
  rpk group [command]

Aliases:
  group, g

Available Commands:
  delete      Delete groups from brokers.
  describe    Describe group offset status & lag.
  list        List all groups.
  seek        Modify a group's current offsets.

Flags:
      --brokers strings         Comma-separated list of broker ip:port pairs (e.g. --brokers '<IP_ADDRESS>:9092,<IP_ADDRESS>:9092,<IP_ADDRESS>:9092' ). Alternatively, you may set the REDPANDA_BROKERS environment variable with the comma-separated list of broker addresses.
      --config string           Redpanda config file, if not set the file will be searched for in the default locations
  -h, --help                    help for group
      --password string         SASL password to be used for authentication.
      --sasl-mechanism string   The authentication mechanism to use. Supported values: SCRAM-SHA-256, SCRAM-SHA-512.
      --tls-cert string         The certificate to be used for TLS authentication with the broker.
      --tls-enabled             Enable TLS for the Kafka API (not necessary if specifying custom certs).
      --tls-key string          The certificate key to be used for TLS authentication with the broker.
      --tls-truststore string   The truststore to be used for TLS communication with the broker.
      --user string             SASL user to be used for authentication.

Global Flags:
  -v, --verbose   Enable verbose logging (default: false).

Use "rpk group [command] --help" for more information about a command.

@jamesza FYI, this is the functionality we were looking for wrt https://bet-project.atlassian.net/wiki/spaces/BE/pages/2260828148/Risk+Register 007.


doc: gsg: Include instructions to set SDK env vars

The basic environment variables need to be set to be able to build
blinky (at least ZEPHYR_TOOLCHAIN_VARIANT) but were not inlcuded in the
GSG.
Signed-off-by: Carles Cufi<EMAIL_ADDRESS>

Almost there from my side. Just a few more things.

Thank you very much! I've now applied all your comments.

@gmarull and @stephanosio, this has been reworked extensively, I'd be grateful if you could take a look.


port depends on nothing when BUILD_TESTING but fails to build test dependency?!

Describe the bug
My port doesn't depend on anything https://github.com/offscale/vcpkg/tree/c-str-span/ports/c-str-span
  "dependencies": [
    {
      "name": "vcpkg-cmake",
      "host": true
    }
  ]

But building depending on it gives:
CMake Error at vcpkg/scripts/buildsystems/vcpkg.cmake:843 (_find_package):
  Could not find a package configuration file provided by "cmocka" with any
  of the following names:

    cmockaConfig.cmake
    cmocka-config.cmake

However my CMakeLists.txt for c-str-span has:
include(CTest)
if (BUILD_TESTING)
    add_subdirectory("tests")
endif (BUILD_TESTING)

Environment

OS: [e.g. Windows/Linux etc...]
Compiler: revision

To Reproduce
Steps to reproduce the behavior:

./vcpkg install c-str-span works but doing it from a dependent project fails
E.g., git clone https://github.com/SamuelMarks/cdd-c
cd cdd-c
cmake -DCMAKE_TOOLCHAIN_FILE="..\vcpkg\scripts\buildsystems\vcpkg.cmake" -S . -B build

Expected behavior
It shouldn't try and depend on the linked repos vcpkg.json. https://github.com/SamuelMarks/c-str-span/blob/master/vcpkg.json has this currently:
  "dependencies": [
    "cmocka"
  ]

Failure logs
-(please attached failure logs)
Additional context
Unless I'm misusing vcpkg and it's not meant to list cmocka as a dependency like that but instead as some sort of feature?

Because https://github.com/SamuelMarks/c-str-span/blob/3edcc6e747160cf7404ef5a371844d2f39717093/tests/CMakeLists.txt#L49-L51 calls find_package(cmocka CONFIG REQUIRED).

To Reproduce Steps to reproduce the behavior:

./vcpkg install c-str-span works but doing it from a dependent project fails

You shuold also run command ./vcpkg install cmocka.

Because https://github.com/SamuelMarks/c-str-span/blob/3edcc6e747160cf7404ef5a371844d2f39717093/tests/CMakeLists.txt#L49-L51 calls find_package(cmocka CONFIG REQUIRED).

@JackBoosY Yes, but that CMakeLists.txt should never be read, due to https://github.com/SamuelMarks/c-str-span/blob/3edcc6e/CMakeLists.txt#L50-L53
include(CTest)
if (BUILD_TESTING)
    add_subdirectory("tests")
endif (BUILD_TESTING)

According to the official doc: https://cmake.org/cmake/help/latest/module/CTest.html
The module automatically creates a BUILD_TESTING option that selects whether to enable testing support (ON by default).

Did you pass -DBUILD_TESTING=OFF?

We don't have enough information to solve this issue, unfortunately.  if it is still an issue, please ping me and I will reopen this issue.

According to the official doc: https://cmake.org/cmake/help/latest/module/CTest.html
The module automatically creates a BUILD_TESTING option that selects whether to enable testing support (ON by default).

Also you didn't make an option to set BUILD_TESTING to OFF by default. And did you pass -DBUILD_TESTING=OFF? I didn't see it in the cmake command line.

@JackBoosY  Yes see here: https://github.com/offscale/vcpkg/blob/7fc89b6/ports/c-str-span/portfile.cmake
vcpkg_cmake_configure(
    SOURCE_PATH "${SOURCE_PATH}"
    OPTIONS
        "-DBUILD_TESTING=OFF"
)

@FrankXie05 This port should be buildable without requiring the test dependencies to be built with it.

@JackBoosY Yes see here: https://github.com/offscale/vcpkg/blob/7fc89b6/ports/c-str-span/portfile.cmake
vcpkg_cmake_configure(
    SOURCE_PATH "${SOURCE_PATH}"
    OPTIONS
        "-DBUILD_TESTING=OFF"
)

@FrankXie05 This port should be buildable without requiring the test dependencies to be built with it.

I can't reproduce this failure when using you provided port c-str-span:
PS J:\vcpkg-test\vcpkg> .\vcpkg.exe install c-str-span
Computing installation plan...
The following packages will be built and installed:
    c-str-span[core]:x86-windows -> 2023-02-11
Detecting compiler hash for triplet x86-windows...
Restored 0 package(s) from C:\Users\JackBoos\AppData\Local\vcpkg\archives in 150.7 us. Use --debug to see more details.
Installing 1/1 c-str-span:x86-windows...
Building c-str-span[core]:x86-windows...
-- Downloading https://github.com/SamuelMarks/c-str-span/archive/5ca092771880914062494c14f72c64594b0d97e5.tar.gz -> SamuelMarks-c-str-span-5ca092771880914062494c14f72c64594b0d97e5.tar.gz...
-- Extracting source J:/vcpkg-test/vcpkg/downloads/SamuelMarks-c-str-span-5ca092771880914062494c14f72c64594b0d97e5.tar.gz
-- Using source at J:/vcpkg-test/vcpkg/buildtrees/c-str-span/src/594b0d97e5-3e15528cd8.clean
-- Found external ninja('1.10.2').
-- Configuring x86-windows
-- Building x86-windows-dbg
-- Building x86-windows-rel
-- Installing: J:/vcpkg-test/vcpkg/packages/c-str-span_x86-windows/share/c-str-span/copyright
-- Performing post-build validation
Stored binary cache: "C:\Users\JackBoos\AppData\Local\vcpkg\archives\dc\dc7239e564680f57a448eda7d5dad5224a5a85e0305dff3d80f01a77a4622623.zip"
Elapsed time to handle c-str-span:x86-windows: 7.831 s
Total install time: 16.23 s
c-str-span provides CMake targets:

    # this is heuristically generated, and may not be correct
    find_package(c_str_span CONFIG REQUIRED)
    target_link_libraries(main PRIVATE c_str_span c_str_span_compiler_flags)

PS J:\vcpkg-test\vcpkg>

@JackBoosY To replicate this bug, you'll want to revert to one before this commit https://github.com/offscale/vcpkg/commit/19dfd786932588262faef84d1b860145c6d627c1 ; i.e., 3a4009f2a064c8aa6d293c84fbaecf9d34ed1c84
(I explicitly removed cmocka from my repos vcpkg.json here https://github.com/SamuelMarks/c-str-span/commit/5ca092771880914062494c14f72c64594b0d97e5 )

@JackBoosY To replicate this bug, you'll want to revert to one before this commit offscale@19dfd78 ; i.e., 3a4009f2a064c8aa6d293c84fbaecf9d34ed1c84
(I explicitly removed cmocka from my repos vcpkg.json here SamuelMarks/c-str-span@5ca0927 )

As I said, if BUILD_TESTING is ON or not defined BUILD_TESTING, this port will find cmoka. It's by design.

CMake Error at vcpkg/scripts/buildsystems/vcpkg.cmake:843 (_find_package):
  Could not find a package configuration file provided by "cmocka" with any
  of the following names:

    cmockaConfig.cmake
    cmocka-config.cmake

It would have been good to add the CMake stack trace following these line.

@JackBoosY Yes but BUILD_TESTING is set to OFF in the vcpkg.json. Replicate by checking out at the aforementioned commit hash.

Add the CMake stack trace...


[3.0] Throwing methods must return an object type in the @objc thunk to be @objc

A throwing method can only be exposed to Objective-C if we can map the
return type convention, either because the return type is Void or
because it is something that is bridged to an object type (and can
therefore be nil to indicate error). Our predicate for checking
"bridged to an object type" didn't account for value types that are
exposed to (or come from) C, and therefore aren't actually bridged to
object types in the Objective-C thunk, meaning they cannot be
optional. An existing hack dealt with the largest class of
these---types like Int and Bool that can be dynamically bridged
to NSNumber---but generalize this by checking exactly how the result
type is going to be represented in Objective-C, rejecting @objc for
cases where the result type won't be an object type.

Resolves rdar://problem/28035614.

@swift-ci please test macOS

@jrose-apple , mind taking a look?

@rudkx , you might be interested in this

Huh. Are there any other places left over where we're not using the centralized checking? (LGTM.)

This was the last use of use of getBridgedToObjC() in the @objc validation logic, at least.

:+1:


Remove redundant setParentFolderId.

Part of a series of changes removing redundant inserts and updates.
The change in testModifiedDocumentsNoCheckpointObjId to expect the
folder as well as the aaa and bbb documents restores the original
behavior, which was modified in commit dea56c3a. Apparently, the
length of the object ID has some impact on the query results from H2.

LGTM


Use Angula2 grid in Angular2 project using Javascript

Hi @BTMorton ,
I will use angular2 grid for my project, but my project is using Javascript instead of TypeScript. How can I apply the Grid directive to my component? Do you know any tutorials guide me to using that way? Help me please.
Many thanks.

I'm not sure, I'm afraid having only used it using Typescript.
https://angular.io/docs/js/latest/quickstart.html might help
Basically, in the ng.core.Component object, add directives: [NgGrid, NgGridItem], but first you'll need to include those two via system.js. Check out this file for an example: https://github.com/BTMorton/angular2-grid/blob/master/demo/app/app.component.js
Hope that helps!

Thanks for your help. :+1:

Sorry I can't offer you more advice!


Replication Snapshot and Compact

Reviewers please focus on resource mgr, home raft repl log, raft repl dev and JournalVDev.
Not much logic change in other layer/files.
Changes:

add snapshot creation to listener (HomeObject PR is in progress) and compact handling (called by raft and in repl dev it will persist compact_lsn as a barrier for truncation).
Note: this is not a generic snapshot that we will implement for other use cases (e.g. HomeBlks). This snapshot creation is bear minimum for nuraft snapshot and HomeObject use case.
add home raft log store truncation. The truncation will be triggered by nuraft server after a snapshot is created.
in HomeStore it remembers the compact_lsn and later resource manager will trigger all logstore truncation in the system.
HomeStore doesn't reserve any extra log items. Reservation is done through dynamic config which used to initialize raft server on startup.
add resource manager resource timer to trigger raft log store truncation as needed.
Add JournalVDev critical high watermark, in its callback it will trigger raft log store truncation immediately so that it doesn't need to want for next timer audit.

Testing:
snapshot creation and truncation can be triggered from the logs with below command line.
./bin/test_raft_repl_dev --gtest_filter=Snapshot_and_Compact --log_mods replication:debug --num_io=999999 --snapshot_distance=200 --num_raft_logs_resv=20000 --res_mgr_audit_timer_ms=120000
it also needs to be added to replication long duration test.

Codecov Report
Attention: Patch coverage is 36.90476% with 53 lines in your changes are missing coverage. Please review.

Project coverage is 57.84%. Comparing base (60808d0) to head (deb52b8).

Files
Patch %
Lines

src/lib/replication/repl_dev/raft_repl_dev.cpp
0.00%
15 Missing :warning:

src/lib/common/resource_mgr.cpp
62.85%
10 Missing and 3 partials :warning:

.../lib/replication/log_store/home_raft_log_store.cpp
0.00%
13 Missing :warning:

src/lib/replication/log_store/repl_log_store.cpp
0.00%
4 Missing :warning:

src/lib/replication/repl_dev/raft_repl_dev.h
0.00%
4 Missing :warning:

...rc/lib/replication/repl_dev/raft_state_machine.cpp
0.00%
2 Missing :warning:

src/lib/checkpoint/cp_mgr.cpp
0.00%
1 Missing :warning:

src/lib/replication/service/raft_repl_service.cpp
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #368      +/-   ##
==========================================
+ Coverage   57.76%   57.84%   +0.07%
==========================================
  Files         108      107       -1
  Lines        9519     9563      +44
  Branches     1233     1231       -2
==========================================
+ Hits         5499     5532      +33
- Misses       3481     3493      +12
+ Partials      539      538       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


refactor(alpha): migrate to Lua API

Migrate alpha to use Lua callback
Signed-off-by: Aaron Pham<EMAIL_ADDRESS>

LGTM


🐛: Misalignment of logo shadow on footer

Title
Fix Misalignment of Logo Shadow in Footer
Bug
Misalignment of Logo Shadow in Footer
Changes
Corrected the alignment of the logo shadow in the footer to ensure consistent appearance and positioning.
Updated CSS styling for the logo shadow to improve visual coherence and aesthetics across all screen sizes.
Screenshots 📷

Full Name
Priyanshi Bhargava
Participant Role
GSSoC-ext :Contributor

Title
Fix Misalignment of Logo Shadow in Footer
Bug
Misalignment of Logo Shadow in Footer
Changes
Corrected the alignment of the logo shadow in the footer to ensure consistent appearance and positioning. Updated CSS styling for the logo shadow to improve visual coherence and aesthetics across all screen sizes.
Screenshots 📷

Full Name
Priyanshi Bhargava
Participant Role
GSSoC-ext :Contributor

Oky do it


Errors while running geometry examples

I have been trying to use the Polyhedron class but unfortunately none of the examples in cvxpy/examples/geometry/ work, not sure if i'm looking at the right branch(master) for recent updates.
example-
`from convex_sets import ConvexHull, Polyhedron, intersect, is_empty, dist, proj, contains
import numpy as np
n = 2
m = 2n
A1 = np.matrix("1 1; 1 -1; -1 1; -1 -1")
A2 = np.matrix("1 0; -1 0; 0 1; 0 -1")
b1 = 2np.ones((m,1))
b2 = np.matrix("5; -3; 4; -2")
poly1 = Polyhedron(A1, b1)
poly2 = Polyhedron(A2, b2)
assert contains(poly1, [-1,-1,1])`

The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()
File "\cvxpy\examples\geometry\convex_sets.py", line 48, in sep_hyp

The geometry example is 5 years old and uses outdated features of cvxpy. I'm not sure it can be recreated, at least in any easy way, with the current cvxpy. I have removed the example. If you're able to fix it, that would be a major contribution!


Update webapi-differences.md

Fixes #34020

Thank you so much @Rick-Anderson for raising this PR.


Include headers in authentication methods

used for lets-chat-reverseproxy
there are some organizations that use Kerberos or some authentication via SSO headers (REMOTE_USER, X-Remote-User and so on).
the passport-reverseproxy came to the rescue, but the req object in app/auth/index.js only sends the user and password. for reverseproxy it should also send the headers so we could authenticate the user.
plus, if we will make it happen, users won't have to actively sign in in organizations with SSO headers. auto login for the win.
but the auto login is not implemented here yet.
maybe the app should let the plugins to authenticate automatically the users?

This is great @Schniz ! I'll see if I can get this working on my install.
Other possible names might be 'headerauth' or 'trustedauth'.
In the future if we support more plugins I think we'll want to clarify the names. I.e. this might become 'lets-chat-auth-reverseproxy'. Given the current code your naming scheme is right

@geoffwhittington seems legit. when you guys would like to change the name you can tell me and I do so or you can just fork it.
Its based on the kerberos plugin :stuck_out_tongue:

+1

This can't be merged as-is, it'll break XMPP.

We should update the wiki if reverse proxy is a new auth thing we can support

@Schniz can you try the latest code on master? Could you also amend your README with the following?

adding that reverse proxy is not supported for XMPP
and removing the Node 0.10 part

:+1:  I'll test it this week
thanks for the inputs!
XMPP does not have headers? which XMPP client you guys use to test the app?


Add --outfile to wrangler deploy for generating a worker bundle

What this PR solves / how to test
Step one of landing https://github.com/cloudflare/workers-sdk/pull/5580. This PR adds an --outfile option to wrangler deploy that generates a worker bundle instead of a directory of files. Follow up PRs will:

Add support for deploying a worker bundle directly, skipping the Wrangler build process (for both Workers and Pages)

Author has addressed the following

Tests

[ ] TODO (before merge)
[x] Included
[ ] Not necessary because:

E2E Tests CI Job required? (Use "e2e" label or ask maintainer to run separately)

[ ] I don't know
[ ] Required / Maybe required
[x] Not required because:

Changeset (Changeset guidelines)

[ ] TODO (before merge)
[x] Included
[ ] Not necessary because:

Public documentation

[ ] TODO (before merge)
[ ] Cloudflare docs PR(s): https://github.com/cloudflare/cloudflare-docs/pull/...
[x] Not necessary because: This set of features should be documented together once it's all landed and ready

Closing in favour of a new approach from @petebacondarwin


Catch up with initial API changes

Now we're at the point where I can start filling these in, but there's a little bit of catch-up to do with API changes that have already landed in SDL3.
I filled in the removed SDL_RWFromFP as a start, but there are several others already to catch up on.
After this catches up, this bug can close, and then we can just sort of keep this in sync as we go.

Pushed https://github.com/libsdl-org/sdl2-compat/commit/842c8c9b28d7d5345cf684b5972351a2d5ca6aae to implement SDL_GetTicks and SDL_GetTicks64 after SDL3
timer changes. @slouken, @icculus: Please check.

The library is broken at present:

The SDL_Event structures have changed timestamp from 32 bits to 64 bits.

GameControllerGetSensorDataWithTimestamp and SensorGetDataWithTimestamp
are removed from SDL3: they are still in sdl3 syms.h as passthrough and
load attempts will fail only because I didn't want builds to fail: Even
if you add stubs for them the library will segfault miserably, because
the event changes aren't handled properly yet.

Yeah, there's a lot to do...I'm going to catch up sdl2-compat early this week.

Okay, we're back in business again. I spent way more time with this than I expected, but here's what I decided for SDL_Event handling:

sdl12-compat implements its own event queue, just pulling in events from SDL2 when pumping, but the SDL2 queue is way more complex (locking, allocation, the poll sentinel, etc), so I ended up abandoning this.
Copying the SDL2 code verbatim into sdl2-compat was an option, but it's a lot of code, and you still have SDL3 doing all that same work anyhow, so I ended up abandoning this, too.
What I ended up doing instead: we use the SDL3 event queue! Whenever we need an SDL2 event (SDL_PollEvent, SDL_PushEvent, etc), we convert at that point, which right now means just adjusting for the timestamp field. We always run an SDL3 event filter (which doesn't do anything atm if there aren't any SDL2 event filters or event watchers to run)...nothing else has changed yet, but when, for example, SDL_WINDOWEVENT gets split into multiple toplevel events, this filter will insert an equivalent SDL3 event into the queue with the appropriate type, and then leave the SDL3 event alone. When the original event gets to the app, the SDL3 version will just have an event type it can't recognize, which might have been something added in a later SDL2 for all it knows, or perhaps an user-registered event, and ignore it.

Maybe this blows up in fun ways, but for now, we're back to a working state. I've also fixed up the other things that changed (SDL_assert data type names, joystick balls, sensor timestamps). There are probably other things that have changed since that I still need to fix up, but we aren't totally blocked now.

Mouse coords becoming floats in SDL3 needs dealing with.

This should be resolved now with your PR (thanks!).
I think we're caught up now in general (but there are still some bugs to work out with device indices vs instance ids, etc), and we should probably close this and start opening separate issues for future things, as more things will be changing and it'll be good to see clearly what work is pending.


Update free-courses-en.md with Learn Blockchain & Solidity Development Full Course 2024

What does this PR do?
Add resource
For resources
Description
Learn Blockchain & Solidity Development Full Course 2024
Why is this valuable (or not)?
This course covers blockchain and Solidity development, up to date with multiple projects for hands-on learning.
How do we know it's really free?
It's on youtube.
For book lists, is it a book? For course lists, is it a course? etc.
course
Checklist:

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

Follow-up

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

Thanks, and welcome to free-programming-books!


set class attribute instead of using classname

Fixes #2407

Can you add a test for the case of doing appendChild (not using Polymer.dom) of an SVG tree and then calling scopeSubtree on it and verifying the SVG gets correctly scoped?

@kevinpschaaf Added!

Thanks!  LGTM.


non latin symbols in a muc roster

in a muc roster any non latin character is printed bold. You can check this on any Cyrillic or "UTF" symbols like ħ̈

Something wrong with web fonts/fonts substitution in system/browser? Cannot reproduce on windows/osx.

Lato 2.0 font with cyrillic letters - http://www.latofonts.com/download/Lato2OFLWeb.zip

Do you think we should use this font in kaiwa?

We already use it from Google Fonts, but there is 1.0 version without cyrillic


On switch account screen show STX balance rather than dollar value

Can we revert this change? Showing dollar value here (alone) makes no sense, first of all this is the STX wallet, We want to see the STX balance ;-). And the dollar value doesn't take into account all assets. So the value is usually off.
I think this was done to save space, make it look cleaner, save space by leaving the dollar value out instead?
Users need the STX amount because it helps them keep track of specific accounts (when they do not have BNS names). dollar value varies so is not helpful in this regard.
For reference the old view (prior to wallet version 3.34.0).

Assigning @mica000 so she can take a fresh look at our approach here not that we're dealing with BTC and STX. The idea indeed was to simplify this space with just the dollar value across both balance types, though I agree we could probably do better here by not removing so much data re: the specific balances.

A recent request from a user was to showing the dollar equivalent value in the balances tab. I guess this would require having endpoints to check the dollar value of such tokens and show no dollar value if it is unknown:

@314159265359879 @markmhx What are your thoughts on this solution?

I would suggest we show the total balance in dollars on the main header, as a space for your "total portifolio", since we also have BTC, at least for now.
Later, the header needs a design update. Currently, the account name takes too much space and not all users have .btc name, leaving the header to be used by a generic name for account 1, 2, 3... But to tackle this we need to define what are the main actions the user want to do and to see on the home screen. We can discuss this soon.
For the balance, Indeed we can show the dollar under each account balance.

Works for me! Presumably we'd still show the raw BNS value with a copy button in the header if available? Want to include a version for that as well?
Also, how does this approach map to how we display balances under "Choose account"?

@mica000 @markmhx is this ready for development?

Yep for the home screen changes, though the account selection ones will require some more design guidance. They can be tackled separately though 🙏

@kyranjamie could you help here please? how it better to count total user balance here?


How many images(collection mode) can we load in the browser using openseadragon

Ex:
I have WidthxHeight = 1280x960 each image and converted each image with tiles with "256".
how many images can we load in collection mode (to extent) ?
Till how many images it supports to load in collection mode ?

That's quite a few! So, your biggest challenge is going to be all of the file transfers. You definitely don't want to have to load a separate DZI file for each image. Fortunately you can use the "inline configuration" option mentioned here:
http://openseadragon.github.io/examples/tilesource-dzi/
That way you can send all of the DZI information as a single file and then break it up into tilesource info on the client side. Assuming you're using DZI and you have it in the standard XML, this CodePen gives some code for converting into the "inline" style:
https://codepen.io/iangilman/pen/gpejpJ
That should help a good deal. Of course you'll still need to load tiles for all the images, which is also a lot! There is an optimization for that in the "Deep Zoom Collection" format, but OSD doesn't yet support that feature... Here's the issue: https://github.com/openseadragon/openseadragon/issues/67 Of course we'd love to support that feature but nobody has stepped up to implement it.

BTW, do all of these images need to be able to move independently of each other, or be rearranged dynamically? If not, another approach is to build them all into a single giant image and DZI that. Until we have the DZC optimization, that's the most efficient way to do this sort of thing at the kind of scale you're talking about.


Add tests to ensure Unicode string comparison is by codepoint - Issue #570

Commit motivated by discussion with @Julian here

I'm pretty sure we already have these kinds of tests. See minLength and maxLength.
These tests are checking the JSON parser, not JSON Schema.
I can provide links in the morning.

thanks for your comment @gregsdennis
as far as I can tell, the test cases you mention seem different
to me this highlights that the way length is calculated for strings is different than the spec's directive on establishing identity/equality for strings
the reason I thought to put these cases in the tests for const is that it's the place where it makes the most sense to speak the schema validator evaluating equality for two strings
the issue of length might just be separate - and out of scope for the original issue
that is, it may be an issue (for this test suite) that a schema with a string field with maxLength 1 essentially accepts unbounded zalgo input since that may count as a single grapheme - but that seems like it's a consideration that is out of scope for the issue as it was originally raises - and doesn't affect the (in)correctness of this particular change
but maybe I've misunderstood


Contextual conditions

Motivation
As per #18, using EasyPeasy along with size classes is not trivial
Goals
Facilitate activation/deactivation of NSLayoutConstraints based on size classes and device.
The final result should allow the following:
Width(50).when { $0.isVerticalCompact }
Width(50).when { $0.isiPad }
Width(50).when { $0.isCompact }

In addition, if the compiler flag EASY_RELOAD is defined, easy_reload will be automatically triggered when the trait collections change.
Tasks

[x] ContextualCondition implementation
[x] Compiler flag to enable swizzling of traitCollectionDidChange(_:)
[ ] Write more unit test
[ ] Update documentation

I'd be great to count on @ashfurrow opinion here :)

Awesome, this is exactly the kind of thing I had in mind 💯

Thank you very much for your precious feedback! I'm glad you like this approach 🍻


Using playwright helper to run test throw exception at the end of test

I'm using playwright helper to run automation test
Throwing exception at the end of the test
(node:52183) UnhandledPromiseRejectionWarning: TypeError: this.browser.disconnect is not a function
    at Playwright._stopBrowser (/Users/harveyl/git/bose-ui-automation/node_modules/codeceptjs/lib/helper/playwright.js:562:26)
    at Playwright._finishTest (/Users/harveyl/git/bose-ui-automation/node_modules/codeceptjs/lib/helper/playwright.js:360:62)
    at /Users/harveyl/git/bose-ui-automation/node_modules/codeceptjs/lib/listener/helpers.js:27:69
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
(node:52183) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 2)
(node:52183) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Below is my playwright help configuration:
let playwright = {
  url: "http://localhost",
  browser: 'chromium',
  chromium: {
    // args: [
    //   `--remote-allow-origins=*`
    // ],
    browserWSEndpoint: "ws://<IP_ADDRESS>:8083/wd/hub",
    wsEndpoint: "ws://<IP_ADDRESS>:8083/wd/hub"
  }
}

Below is the source code of playwright helper, seems the 'await this.browser.disconnect();' is not working as expected
  async _stopBrowser() {
    this.withinLocator = null;
    this._setPage(null);
    this.context = null;
    popupStore.clear();

    if (this.isRemoteBrowser) {
      await this.browser.disconnect();
    } else {
      await this.browser.close();
    }
  }

according to https://github.com/codeceptjs/CodeceptJS/pull/2629
there is no reference for method disconnect() on Playwright's doc.
You should be good if you are using codecept 3.x or above


Issue 8547

https://github.com/zulip/zulip/pull/9047
QA: ran Casper frontend tests in dev mode

Closing as https://github.com/zulip/zulip/pull/9047 that this references was also closed as resolved.


name2unicode() does not conform to the Adobe Glyph List Specification

Earlier issues (#183, #229) mention that the name2unicode() function does not handle hexadecimal literals well. These issues are solved by changing the regex that matches the hexadecimal literals.
However, the functionality of name2unicode() still does not match the mapping in the spec.
The spec also lists a couple of examples that could serve as unittests and would fail now.
I suggest we follow the spec, with the exception of specific mapping for Zapf Dingbats.

This issue is fixed by #263


Updated versions of linters and other dependencies

Issues:
Issue#
Description:
Updates are needed to fix bugs in linters ALT-10537,  ALT-10549

@polischuks The build shouldn't fail, it needs to be fixed

@polischuks @meanmail Hi! You can't just update the linters version, because

New versions may include new inspections that may be enabled by default but are not appropriate for students
Some inspections may be removed
As far as I remember we decided not to upgrade detekt because we had problems with its new static analysis system, which uses classpaths

So to properly update the linters you should check all their changelogs and update the corresponding configuration files. It would also be more convenient to do it in separate PRs because there will be a lot of changes.


Cant't sign in leetcode.cn, raise timeout

I use windows10, I sign  with cookie after installed raise this error:

the cookie I use is from this place, it seems no problem:

I use cn= leetcom.cn , config as
opts = {
  cn = {
    enabled = true,
  },
},

the detail error info is below:
Error executing vim.schedule lua callback: ...ta/Local/nvim-data/lazy/plenary.nvim/lua/plenary/job.lua:500: 'curl -sSL -D C:\Users\Administrator\AppData\Local\Temp\plenary_curl_905e4572.headers -X POST -H Origin: https://leetcode.cn/ -H Referer: https://leetcode.cn -H Content-Type: application/json -H Accept: application/json -H X-Requested-With: XMLHttpRequest -H User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:123.0) Gecko/20100101 Firefox/123.0 -H X-Csrftoken: fuRpYTaszdT30wek4v8AF4LNNln0tbK5U89VDnGXfdo8tbF25pFFwFJsfES86UmU -H Host: leetcode.cn -H Cookie: {MY_COOKIE} --data-raw {"variables":[],"query":"        query globalData {\n            userStatus {\n                slug: userSlug\n                name: username\n                real_name: realName\n                is_signed_in: isSignedIn\n                is_premium: isPremium\n                is_verified: isVerified\n            }\n        }\n    "} --http2 https://leetcode.cn/graphql/' was unable to complete in 10000 ms
stack traceback:
	[C]: in function 'error'
	...ta/Local/nvim-data/lazy/plenary.nvim/lua/plenary/job.lua:500: in function 'wait'
	...ta/Local/nvim-data/lazy/plenary.nvim/lua/plenary/job.lua:454: in function 'sync'
	...a/Local/nvim-data/lazy/plenary.nvim/lua/plenary/curl.lua:317: in function <...a/Local/nvim-data/lazy/plenary.nvim/lua/plenary/curl.lua:268>
	.../nvim-data/lazy/leetcode.nvim/lua/leetcode/api/utils.lua:92: in function 'query'
	...l/nvim-data/lazy/leetcode.nvim/lua/leetcode/api/auth.lua:22: in function 'user'
	...im-data/lazy/leetcode.nvim/lua/leetcode/cache/cookie.lua:29: in function 'set'
	...im-data/lazy/leetcode.nvim/lua/leetcode/command/init.lua:59: in function 'on_submit'
	...ata/Local/nvim-data/lazy/nui.nvim/lua/nui/input/init.lua:162: in function <...ata/Local/nvim-data/lazy/nui.nvim/lua/nui/input/init.lua:149>

similar issue

I have the same questions

same on windows

I have the same
problem as well in windows 10. Investigating right now.

I resolved the problem by updating the curl to 8.10.1. It seems that the older version of libcurl is not compatible with (curl) --http2 argument.

Great!

Please use cygwin if you're an windows user

same problem, i use termux in android. my curl version is 8.11.0

What error are you getting?

same problem, i use termux in android. my curl version is 8.11.0

What error are you getting?

https://github.com/nvim-lua/plenary.nvim/issues/536 could you check if any of the tips here fix your issue?

same problem, i use termux in android. my curl version is 8.11.0

What error are you getting?

nvim-lua/plenary.nvim#536 could you check if any of the tips here fix your issue?

i am so happy that it work，thanks


How to get multi-view images

Hi! Thanks for your great work! I am curious about how to get multi-view object images in the "Object Caption" step of your annotation pipeline. It seems that only a 3D point cloud and object bounding box is needed? But, how to decide the camera pose of each view, and how to render the images to make them look realistic?
I am also wondering will the code implementation for the entire SceneVerse annotation pipeline be released :)

Thanks for the interest, for most of the datasets considered (e.g. ScanNet) it does contain RGBD videos for the original capture, you can use those for projecting 3D bounding boxes to 2D for VLMs to generate.
As for the second question, we are wrapping up the code release, stay tuned.

Hi, as HM3D originally did not provide multi-view images, we did not generate RGBD videos, we instead tried synthesizing viewpoints for objects for captioning.

Can you please share the synthesizing viewpoints?


cacheDirectory outside of output.path

how can I set the cacheDirectory outside of output.path? I remove the dist folder each time I build which conflicts with having a cacheDirectory

@gdi2290 You can specify an absolute path with __dirname.
new HardSourceWebpackPlugin({
  cacheDirectory: path.join(__dirname, 'tmp/hard-source/[confighash]'),
  recordsPath: path.join(__dirname, 'tmp/hard-source/[confighash]/records.json'),
  // ... other stuff as normal
}),


Move all deadline cancellations from components to transactions/provider

Also add configuration parameter for the timeout

Fixes #473

Codecov Report

:exclamation: No coverage uploaded for pull request base (rusk-ipc@34acd3c). Click here to learn what that means.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             rusk-ipc     #539   +/-   ##
===========================================
  Coverage            ?   54.71%
===========================================
  Files               ?      155
  Lines               ?     9165
  Branches            ?        0
===========================================
  Hits                ?     5015
  Misses              ?     3535
  Partials            ?      615

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 34acd3c...1ccff16. Read the comment docs.


Automate versioning tag for about page

It's sometimes hard to verify if developers are on the same version. Adding some automated thing that goes into the about page would be great. If we can figure it out.

/move to satyrnjs/satyrn


[iOS] Make MLCEngine input to take in structured data

This PR modifies the MLCEngine chatCompletion to take in structured data.

cc @YiyanZhai


Predictions are wrong.

I am training docformer, 6 epochs completed. The predictions are not good.
Can you please tell what is minimum number of epochs required to get better prediction on document classification?
Also, in hugging face demo, why the id2label is in this sequence?
id2label = ['scientific_report',
            'resume',
            'memo',
            'file_folder',
            'specification',
            'news_article',
            'letter',
            'form',
            'budget',
            'handwritten',
            'email',
            'invoice',
            'presentation',
            'scientific_publication',
            'questionnaire',
            'advertisement']

Hi there,
For Q1. I am really not sure, how many epochs would be required to get a good result since I have not experimented with it. But as soon as I something related to it, I would immediately get back to you.
Q2. The id2label is in that specific order because while I was training the model on the Kaggle platform, this was the exact order for label encoding, i.e the 0th class was 'scientific_report' and so on.
Regards,
Akarsh


fix(gcs): Fix startup error when bucketLocation is not specified

Fixes spinnaker/spinnaker#6000.

test(gcs): Add some implementations to FakeStorageRpc
In an upcoming commit, we'll need to also support getting a bucket; to prepare for that let's update the FakeStorageRpc to:

Keep track of what bucket objects are in (rather than now where it assumes they are all in the same bucket)
Handle creating and getting a bucket

We'll add an additional Map around the blobs data structure that maps the bucket to a Map of blobs.  (This is a bit of a complex data structure that in production code might be better expressed as classes, but for this single fake in a test it's porbably fine.)
Creating a bucket initializes its blobs to an empty map; we know if a bucket exists by whether it has an entry in the map.
I used the documentation on the interface we're mocking to decide how to handle a missing bucket; in some cases we just return null
(when getting an object or a bucket) while in others we propagate the 404.
This required one small change to the existing tests, which is that we'll need to create the bucket before we use them in the tests.

test(gcs): Add test to demonstrate NPE on missing bucketLocation
This commit adds a broken test to demostrate the NPE that occurs on startup when the bucketLocation is not specified.

fix(gcs): Fix startup error when bucketLocation is not specified
There was a regression in 1.22 where omitting bucketLocation from one's GCS config now causes an error on startup. Prior to the rewrite of GcsStorageService, we accepted either null or empty string as the bucket location; now that GcsStorageService is in kotlin and does not use a String? for the field, an error occurs on trying to create the GcsStorageService.
The fix is just to default the string to "" in the config properties.

Added the backport-candidate label as this was a regression in 1.22 caused by #911.  I've written tests demonstrating the bug and the fix, so this should satisfy the release branch patch criteria.

test(gcs): Replace nested map with classes
To simplify the nested map, define a Buckets class and a BucketContents class, with operation names that make it more clear what is happening.
Also, I realized that before my changes we were appending the bucket name to the object name before putting it into the map, but now we don't need to do that anymore because we have a separate map for each bucket, so remove some unecessary calls to fullPath (which is now only used in printing error messages).

Found one tiny thing I thought I changed while looking at this before merging.  (I made a private constructor that takes a map, then a public constructor that always passes an empty map...but the way to do this in kotlin is just make a val that's not part of the constructor at all.). Fixed that and added ready-to-merge so I can't change this any more 😀

@Mergifyio refresh

@Mergifyio backport release-1.22.x


Only check inexact match when no exact match is found.

Fix dotnet/runtime#39620
The same as dotnet/runtime#39961
Avoid the exact match result be overwrite by later inexact ones.
This may not happen due to limited callsites, just make the code more clear.

Is there a test case to verify the fix?

Infact, I don't know how to create an test for this internal api. And maybe it's not a problem due to it's internal and there is no dupe tables with different case actually used by public apis.
https://github.com/dotnet/sqlclient/search?q=filename%3AMetaData.xml&unscoped_q=filename%3AMetaData.xml

@yyjdelete it may be worth looking into this a little further and finding an actual user impact - since the current implementation does seem buggy.

If it isn't testable but is clearly a fix I'm fine with merging it but testing does always need to be considered een though writing tests sucks.

This bug would only surface if the embedded resource, Microsoft.Data.SqlClient.SqlMetaData.xml, contained multiple MetaDataCollections with a CollectionName differing only by case and the desired entry (case sensitive match) appeared before the undesired entry (case insensitive match). Testing the bug would be awkward. I'm fine with simply fixing the bug. Bonus points if a couple lines are added to TestGetSchema to test conn.GetSchema("Databases") in addition to conn.GetSchema("DATABASES") to cover both blocks of the if statement in question.

This bug would only surface if the embedded resource, Microsoft.Data.SqlClient.SqlMetaData.xml, contained multiple MetaDataCollections with a CollectionName differing only by case and the desired entry (case sensitive match) appeared before the undesired entry (case insensitive match). Testing the bug would be awkward. I'm fine with simply fixing the bug. Bonus points if a couple lines are added to TestGetSchema to test conn.GetSchema("Databases") in addition to conn.GetSchema("DATABASES") to cover both blocks of the if statement in question.

@David-Engel
There is already another test with conn.GetSchema("Databases")(case sensitive match) in ConnectionSchemaTest.
https://github.com/dotnet/SqlClient/blob/4258e4b44220c5f96997717a9b3441968108abd4/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/DataBaseSchemaTest/ConnectionSchemaTest.cs#L32-L36


incompatible with expected float_ref.

Traceback (most recent call last):
File "convert_TFLite.py", line 90, in
convert_from_savedModel(savedModelDir, TFLiteFile, quan=True, integerOnly=True)
File "convert_TFLite.py", line 50, in convert_from_savedModel
TFLiteModel = converter.convert()
File "/usr/local/lib/python3.5/dist-packages/tensorflow/lite/python/lite.py", line 459, in convert
self._funcs[0], lower_control_flow=False))
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/convert_to_constants.py", line 707, in convert_variables_to_constants_v2_as_graph
frozen_func = _construct_concrete_function(func, graph_def, converted_inputs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/convert_to_constants.py", line 406, in _construct_concrete_function
new_output_names)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/eager/wrap_function.py", line 633, in function_from_graph_def
wrapped_import = wrap_function(_imports_graph_def, [])
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/eager/wrap_function.py", line 611, in wrap_function
collections={}),
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/func_graph.py", line 981, in func_graph_from_py_func
func_outputs = python_func(*func_args, **func_kwargs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/eager/wrap_function.py", line 86, in call
return self.call_with_variable_creator_scope(self._fn)(*args, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/eager/wrap_function.py", line 92, in wrapped
return fn(*args, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/eager/wrap_function.py", line 631, in _imports_graph_def
importer.import_graph_def(graph_def, name="")
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/util/deprecation.py", line 507, in new_func
return func(*args, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/importer.py", line 405, in import_graph_def
producer_op_list=producer_op_list)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/importer.py", line 501, in _import_graph_def_internal
raise ValueError(str(e))
ValueError: Input 0 of node conv21/pointwise/BatchNorm/cond/Assign/Switch was passed float from conv21/pointwise/BatchNorm/moving_mean:0 incompatible with expected float_ref.
when I use "tf.lite.TFLiteConverter.from_saved_model()" to convert checkpoint to tflite，I got above err about BatchNorm, please help me, ths.
Originally posted by @ZhouKai90 in https://github.com/tensorflow/tensorflow/issues/3628#issuecomment-663382551

Switch is not supported by TFLite. Please use the v2 control flow ops while creating a TF graph. https://www.tensorflow.org/api_docs/python/tf/compat/v1/enable_control_flow_v2

@ZhouKai90
Can you please provide with simple stand alone code and tf version for us to replicate the issue or if possible share colab gist with the error.


Automatically delete CMakeCache.txt when new Visual Studio toolchain installed

Upon upgrading from Visual Studio 2019 to 2022, we need to delete CMakeCache.txt in application builds, or we'll encounter a build failure.
Repro steps:

Create a boilerplate flutter app.
Invoke flutter run with Visual Studio 2019 installed.
Install Visual Studio 2022.
Invoke flutter run in the same app.

Expected result: the app builds and runs
Actual result:
Launching lib\main.dart on Windows in debug mode...
CMake Error: Error: generator : Visual Studio 17 2022
Does not match the generator used previously: Visual Studio 16 2019
Either remove the CMakeCache.txt file and CMakeFiles directory or choose a different binary directory.
Building Windows application...
Exception: Unable to generate build files

In this case, we should automatically delete CMakeCache.txt and re-run the failing step. If it fails again, then we should bail out.
Related: https://github.com/flutter/flutter/issues/85922

In case anyone finds this via searching for the error message: flutter clean should resolve this.

Go  flutter/packages/flutter_toops/lib/src/windows and open build_windows.dart file and on line 24 change const int kCurrentUwpTemplateVersion = Visual Studio 17 2022;
Save the file and run flutter run and select windows device it must run fine by now without any errors 🎉

I'm experiencing this issue since I upgraded to the latest version of Flutter

[√] Flutter (Channel stable, 2.10.2, on Microsoft Windows [Version 10.0.19044.1526], locale en-AU)
• Flutter version 2.10.2 at C:\tools\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 097d3313d8 (3 days ago), 2022-02-18 19:33:08 -0600
• Engine revision a83ed0e5e3
• Dart version 2.16.1
• DevTools version 2.9.2

The problem you described is after running flutter doctor command if so
then try upgrading flutter to the latest version by flutter upgrade and see
if the problem persists or is solved . And in the second case if you
experienced something else then please elaborate your problem a bit more.
On Sun, 6 Mar, 2022, 10:18 pm HACKER NEXT, @.***> wrote:

Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/93680#issuecomment-1047314649,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS5MFQC3QUHAYHFB7QBZUODU4LI7ZANCNFSM5ICSZHVQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you wMessage ID:
@.***>

In case anyone finds this via searching for the error message: flutter clean should resolve this.

DO NOT DO THIS! i did the command flutter clean.... and i need create another project =´[


cleanup downstream

remove weldx
change abacusutils tests per suggestion: https://github.com/asdf-format/asdf/pull/1845#issuecomment-2407650943

Description

Tasks

[ ] run pre-commit on your machine
[ ] run pytest on your machine
[ ] Does this PR add new features and / or change user-facing code / API? (if not, label with no-changelog-entry-needed)

[ ] write news fragment(s) in changes/: echo "changed something" > changes/<PR#>.<changetype>.rst (see below for change types)
[ ] update relevant docstrings and / or docs/ page
[ ] for any new features, add unit tests

news fragment change types...

changes/<PR#>.feature.rst: new feature
changes/<PR#>.bugfix.rst: bug fix
changes/<PR#>.doc.rst: documentation change
changes/<PR#>.removal.rst: deprecation or removal of public API
changes/<PR#>.general.rst: infrastructure or miscellaneous change

Fixes the CI so merging.


BUG: FILES Preview <IP_ADDRESS> Spanish translation.

Several interface texts are not translated into Spanish. Their corresponding strings are not found in Crowdin
Contextual menu

Tags

Crowdin is having some issues with syncing, I'm doing it manually for now but it's not going to always be a 1:1 match.

Crowdin support pointed me toward another setting so hopefully this is resolved in the next 24 hours.

Agreed. Thanks


file.UploadIntoStream parameters ignored

Following parameters are ignored in Dialog of file.UploadIntoStream function:

FromFolder
FromFilter

FromFilter is quite important for user experience. Now anything I set, ends up with default filter.
See example below:

See the msdn documentation: https://msdn.microsoft.com/en-us/library/dd301099(v=nav.90).aspx
Those parameters are not supported by the web client.


--player-continuous-http

Is it possible to tell the "player" for example vlc to try to reconnect X number of times instead of an infinite loop?

I got it with  --retry-open '4'


LocalVariableHidesMemberAnalyzer doesn't format the description message

https://github.com/icsharpcode/RefactoringEssentials/blob/master/RefactoringEssentials/CSharp/Diagnostics/Synced/CodeQuality/LocalVariableHidesMemberAnalyzer.cs#L16
This is coming out as just Local variable '{0}' hides {1} '{2}' instead of having variable names.

Thanks for reporting!


OTA: Allow for updating SPIFFS

For Homie projects using SPIFFS, it would be beneficial to also allow for updating the filesystem remotely.
esp8266/Arduino Updater provides support for SPIFFS updates. We just need a Homie convention for OTA/SPIFFS. Maybe use topics $implementation/ota/firmware and $implementation/ota/filesystem instead of $implementation/ota/payload?
Depending on the topic, BootNormal::_onMqttMessage() would specify U_FLASH or U_SPIFFS to the command argument when calling Update.begin().

The problem is the config.json and some state file (e.g. BYPASS_STANDALONE are actually stored in the SPIFFS. So if we overwrite it, the device would basically be "factory resetted".

Oops. I didn't think about config.json and didn't even know about the state file. I guess it's too risky (e.g. power fail during update) to temporarily keep those in RAM while updating SPIFFS and then copying them back into the new SPIFFS.
Let me review the SPIFFS code and the Updater code. Maybe we can "mount" the Updater memory range as a temporary SPIFFS partition and copy the two files into this partition after (or right before) calling Update.end(). That way, even if the SPIFFS upgrade fails, there would always be a good SPIFFS partition with a valid configuration.

Working on #172 I learned that the esp8266/Arduino Updater class updates SPIFFS "in-place". That is, it writes the update directly into SPIFFS's flash sectors. So there is no second copy of SPIFFS available that we could have used for mounting a "shadow SPIFFS" to backup Homie's SPIFFS files before committing the SPIFFS update to flash.
I think it would be too much hassle to mimic for SPIFFS inside Homie what the Updater class is doing for firmware. If at all, this feature should be implemented inside Updater. Maybe one day I will submit a PR to the esp8266/Arduino Updater class.
Looks like this wasn't the greatest feature proposal ;-). Would you mind if I closed this issue?

Thanks for your time! Go ahead and close the issue :) we can reopen it if it's ever available in Updater.


Kafka example fails with Zookeeper panic "Unable to create data directory"

MacOSX 11.3.1
Node 12.x
I copied/pasted the first "built-in Zookeeper" example from https://github.com/testcontainers/testcontainers-node/tree/master/src/modules/kafka. The docker instances are created, but the cp-zookeeper instance fails with the following:
===> User
uid=1000(appuser) gid=1000(appuser) groups=1000(appuser)
===> Configuring ...
===> Running preflight checks ...
===> Check if /var/lib/zookeeper/data is writable ...
===> Check if /var/lib/zookeeper/log is writable ...
===> Launching ...
===> Launching zookeeper ...
[2021-05-10 15:33:21,279] INFO Reading configuration from: /etc/kafka/zookeeper.properties (org.apache.zookeeper.server.quorum.QuorumPeerConfig)
[2021-05-10 15:33:21,301] INFO clientPortAddress is <IP_ADDRESS>:48471 (org.apache.zookeeper.server.quorum.QuorumPeerConfig)
[2021-05-10 15:33:21,301] INFO secureClientPort is not set (org.apache.zookeeper.server.quorum.QuorumPeerConfig)
[2021-05-10 15:33:21,306] INFO autopurge.snapRetainCount set to 3 (org.apache.zookeeper.server.DatadirCleanupManager)
[2021-05-10 15:33:21,306] INFO autopurge.purgeInterval set to 0 (org.apache.zookeeper.server.DatadirCleanupManager)
[2021-05-10 15:33:21,307] INFO Purge task is not scheduled. (org.apache.zookeeper.server.DatadirCleanupManager)
[2021-05-10 15:33:21,307] WARN Either no config or no quorum defined in config, running  in standalone mode (org.apache.zookeeper.server.quorum.QuorumPeerMain)
[2021-05-10 15:33:21,317] INFO Log4j 1.2 jmx support found and enabled. (org.apache.zookeeper.jmx.ManagedUtil)
[2021-05-10 15:33:21,336] INFO Reading configuration from: /etc/kafka/zookeeper.properties (org.apache.zookeeper.server.quorum.QuorumPeerConfig)
[2021-05-10 15:33:21,337] INFO clientPortAddress is <IP_ADDRESS>:48471 (org.apache.zookeeper.server.quorum.QuorumPeerConfig)
[2021-05-10 15:33:21,337] INFO secureClientPort is not set (org.apache.zookeeper.server.quorum.QuorumPeerConfig)
[2021-05-10 15:33:21,338] INFO Starting server (org.apache.zookeeper.server.ZooKeeperServerMain)
[2021-05-10 15:33:21,343] INFO zookeeper.snapshot.trust.empty : false (org.apache.zookeeper.server.persistence.FileTxnSnapLog)
[2021-05-10 15:33:21,344] ERROR Unable to access datadir, exiting abnormally (org.apache.zookeeper.server.ZooKeeperServerMain)
org.apache.zookeeper.server.persistence.FileTxnSnapLog$DatadirException: Unable to create data directory /var/lib/zookeeper/log/version-2
	at org.apache.zookeeper.server.persistence.FileTxnSnapLog.<init>(FileTxnSnapLog.java:127)
	at org.apache.zookeeper.server.ZooKeeperServerMain.runFromConfig(ZooKeeperServerMain.java:125)
	at org.apache.zookeeper.server.ZooKeeperServerMain.initializeAndRun(ZooKeeperServerMain.java:107)
	at org.apache.zookeeper.server.ZooKeeperServerMain.main(ZooKeeperServerMain.java:65)
	at org.apache.zookeeper.server.quorum.QuorumPeerMain.initializeAndRun(QuorumPeerMain.java:128)
	at org.apache.zookeeper.server.quorum.QuorumPeerMain.main(QuorumPeerMain.java:82)
Unable to access datadir, exiting abnormally

Looks like there was a major version change from 5.4.4 to 6.1.1, could you try 5.4.4? If that works then we can hardcode the version in the example

I got a Error: (HTTP code 404) no such container - No such image: confluentinc/cp-zookeeper:5.5.4 when I tried
const { KafkaContainer } = require("testcontainers");

        const container = await new KafkaContainer(
            "confluentinc/cp-kafka:5.5.4", // Kafka image
            undefined, // Host
            "confluentinc/cp-zookeeper:5.5.4" // ZK image
        )
        .withExposedPorts(9093)
        .start();

Nevermind, I tried to pull the image manually and I was out of docker disk space. ☝ works I think. Verifying I can connect...

Hardcoding v5.5.4 of the images resolves the issue. I would recommend hardcoding that version in your lib for the built-ins @cristianrgreco

I've raised a PR which sets the version to 5.5.4 by default, thanks @jonathankeebler. Is there anything else for this issue or can it be closed?


Add table name validator

Closes #78

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 4 (75.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.006%) to 64.068%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

daiquiri/query/validators.py
3
4
75.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.006%

Covered Lines:
4700

Relevant Lines:
7336

💛  - Coveralls


Frontier silicon

Description:
new component - frontier silicon, 'depends on latest' netdisco
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#1966
Example entry for configuration.yaml (if applicable):
media_player:
  - platform: frontier_silicon
    host: IP_ADDRESS

I need some help here (my first python project) - the travis requirements check won't pass.
I have a single requirement REQUIREMENTS = ['fsapi==0.0.7'] (https://pypi.python.org/pypi/fsapi/0.0.7).
I guess I did sth. wrong in the fsapi project: https://github.com/zhelev/python-fsapi.
The project requires lxml and requests itself.
The components works as a custom component. Any ideas what I did wrong?

hey @zhelev
you need to add something like
# homeassistant.components.media_player.frontier_silicon
fsapi==0.0.7
at the right place of requirements_all.txt in home-assistant root directory

@zhelev:
I just had a look at your library and I guess it would be better to include is as a non asyncio platform, as it does not look async friendly.
Maybe someone with good experience in asynchronous programming like @balloob can confirm or rebut my impression.

@scarface-4711 Thanks for the feedback, appreciate it. I've reverted to a sync implementation for now and am still trying to get the requirements to work.

@zhelev In the root of your repo, run script/gen_requirements_all.py. That will update the requirements file (then commit the resulting changes).

It's too bad that you removed the checklist from your API because besides the requirements instruction that you missed, you also missed the coverage task. Please do that one too.

@balloob I am pretty new here and it is my first python project. Can you give me a hint what do you mean with removing the 'checklist'?  So far I just followed the tips from the other guys and updated my branch.
The coverage fails because I do not have a unit test or there is any another reason?

I guess I've reached a dead end here, closing the PR and will retry, will revisit all steps and hope this time it works out


Build site in another folder?

I know this might be a silly question, but is there a way to build the site in a different folder, rather than the same folder that contains all the source files? Something like a serve(target="/path/to/somewhere/") ?

A bit cheeky but you could just do serve(...); cp(...)
More to the point though, no there isn't. I think it should be easy to support though because everything is done via this path thing which is set up at the start and could take config.md in consideration for an alternative path for __site
Is this just a preference thing or do you have a more specific use case in mind?

Well, what happened was that git/Github/GithubPages eventually got fed up with my site's repo (when it got over 3Gb - I can't blame them), and all my attempts to fix it also failed (and might even have made it worse), with things taking hours to fail rather than minutes, so eventually I just nuked the entire thing. and started again. And my current idea is to maintain two separate repos, one with the site, the other with the source.
It is probably just a preference - an easier set-up to think about -  but I wondered if it might be useful as a general feature... 🤔

is it 3Gb because the whole git history is kept on the gh-pages branch? or because you actually have 3Gb of assets?
in the first case there may be a setting of the github-pages-deploy-action to avoid that, in the latter, maybe it'd make sense to store assets on a cloud bucket somewhere.
Anyway, can you test Franklin#tl/buildpath setting site_path = joinpath("your", "desired", "output", "path") in your config.md and see if that works for you? Note that the minification won't work anymore (it expects the __site path) so set minify=false in the final call to optimize, it's irrelevant anyway.

can you test Franklin#tl/buildpath setting site_path

It works great! If you think it would be useful to others as well, please add it to one of your forthcoming *ranklin releases...

store assets on a cloud bucket

Haha, well if it was important stuff I might do that, but this is just idle amusement, so I'll stop when it starts to get out of hand... :)
Thanks so much for your work!


We should update Docker-PowerShell module name to match.

Currently the Docker-PowerShell module is named Docker. It seems more reasonable to call it Docker-PowerShell right?
The only changes would be:
> Install-Module Docker-PowerShell
> Import-Module Docker-PowerShell

What other PowerShell modules have PowerShell in the name?

I'm in favor of leaving this as is... the module name is descriptive enough, because the fact that is a powershell module is implied by the fact you are importing it into powershell using Install-Module.  But the repository is named Docker-PowerShell to differentiate from the actual docker product fork.  I don't feel like the names should have to match, because it is named reasonable for identifying it.

Yea my fear is that Install-Package Docker will conflict someday. But I guess then you would just have to add -Source DockerPS-Dev or whatever feed you meant for Docker PS.


Missing 6.0.31 and 8.0.6 tags

URL(s)

https://github.com/dotnet/core/blob/main/release-notes/6.0/6.0.31/6.0.31.md
https://github.com/dotnet/core/blob/main/release-notes/8.0/8.0.6/8.0.6.md

Description
Hello team,
I see new release notes were added to the repository regarding the latest 6.0.31 and 8.0.6 releases. However I don't see any tag at /tags pointing to the source code. Do you have any ETA on when will they be added? Thank you in advance.

Related issue from dotnet-docker repo: https://github.com/dotnet/dotnet-docker/issues/5512

Tags added and release pages added.


Update to rush 5.97 and pnpm 7

The latest major version of pnpm drop Node 12 support.
Latest Rush version is needed to move up to latest pnpm version

edit: Technically the above is now a lie given pnpm 8 has just been released two days ago.

@calebmshafer @aruniverse do we want to move to rush 5.97 and pnpm 8.3.1 (those are now the latest)?

I'll update this PR to the latest versions.

I can work on it.

Fighting with various issues, might stick with pnpm 7 for now.

hopefully this pr can be replaced with https://github.com/iTwin/itwinjs-core/pull/5418, a different approach to updating the audit


Bug

Hi, I've encountered this bug:
(base) [my computer's name]:server Haram$ cd SentimentArcs_WebApp
(base) [my computer's name]:SentimentArcs_WebApp Haram$ python -m venv venv
(base) [my computer's name]:SentimentArcs_WebApp Haram$ source venv/bin/activate
(venv) (base) [my computer's name]:SentimentArcs_WebApp Haram$ pip install -r server/requirements.txt
Processing /home/decaf/Projects/sentiment_ui/SentimentArcsPackage
ERROR: Could not install packages due to an OSError: [Errno 2] No such file or directory: '/System/Volumes/Data/home/decaf/Projects/sentiment_ui/SentimentArcsPackage
Could you help?
Best, Haram

Thanks for the quick response. I changed the requirements.txt to reflect a correct path. And it worked!
Yet, I encountered another error message after running the requirements.txt:
ERROR: Ignored the following versions that require a different python version: 1.6.2 Requires-Python >=3.7,<3.10; 1.6.3 Requires-Python >=3.7,<3.10; 1.7.0 Requires-Python >=3.7,<3.10; 1.7.1 Requires-Python >=3.7,<3.10
ERROR: Could not find a version that satisfies the requirement sentarcs (unavailable) (from versions: none)
ERROR: No matching distribution found for sentarcs (unavailable)
And I tried to run "python server/server.py", and encountered another message: Traceback (most recent call last):
File "/Users/Haram/SentimentArcs_WebApp/server/server.py", line 1, in
from flask import Flask, request, send_file, send_from_directory
ModuleNotFoundError: No module named 'flask'
Can you help? Thanks in advance!


Staking and mining code cleanup, improvements

Read the commit messages.
Is it good like this or do I have to split it by commits?
Interest and Research subsidy visible in getmininginfo.
Check out -debug=true -printtoconsole

Hey, cool. It looks like this one still tries to sign the same block over and over again, is that correct?

@denravonska performed the changes, please verify

Excellent, I will try it out first thing tomorrow morning.

Trying this on testnet now and I noticed that it started receiving blocks from the network while still loading the blocks from disk. I'm also seeing a lot of these in the log, but I guess that is to be expected with 0 mag and a semi-broken testnet:
ERROR: CheckStakeKernelHashV3: Magnitude too low for POR reward:<PHONE_NUMBER>.000

I would like to point you, @ravon, to this wiki page. I am confused about what outputs is which flag meant to enable.

I will close this request not. This one is only half done and I feel that more refactoring is needed.

Are you sure? It's still an improvement over the current implementation.

I kind of misunderstood what a Close really means so I open this again with a push :)
Remember: this code is experimental and not complete.

New status of this PR:
New stake code is pretty stable and stakes blocks just right. Less complex code and unnoticeably lower cpu usage.
Some info is not displayed in status. Other info added.
#317 is still an issue: watch out for 1GRC research reward and report there.
Sightly more debug output when enabled.
Only one set of coins (UTXO) is ever added to stake transaction, thus sightly slower interest.
All stake coins and reward is placed in single output: no more 1 cent outputs but may not split outputs as expected.
And due to the PoW bug/feature (#332) minersleep option does affect staking frequency.
I would appreciate some testing.

We should try to include this in the next staging merge if possible. Regarding the minersleep option I'd prefer if we removed it. When debugging it's trivial to alter the code.
I can test it on testnet if you want to.

Running this in testnet now. What's missing from the status?
Getting -ERROR: CheckStakeKernelHashV3:  Coin Age < RSA_Weight: 49104; 106280 :(

Those indents thou in debug print for RSA_WEIGHT ;o) looks nice thou to have that added.


Updated Code for 2024-03

Updated deprecated code, added documentation, Organized imports alphabetically.

Looks good. Thanks!


[BUG] JSON-RPC result error

Describe the bug
Please provide a short description of the bug.
When I request a call for RPC node, it prints out result+%.
Please refer to the screenshot attached.
To Reproduce
Steps to reproduce the behavior, e.g.:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Request the call for RPC node
Please refer to the screenshot attached.
Expected behavior
A clear and concise description of what you expected to happen.
No %
Screenshots
If applicable, add screenshots to help explain your problem.

Technical Details (please complete relevant fields):

RPC Public/Private

Additional context
Add any other context about the problem here.

ty