Subclasses of JointPrior are re-initialized wrong from dictionaries and json

I am not a member of the LVK and, thus, cannot create issues or merge requests in the main repository.
I have encountered issues with subclassing JointPrior and JointPriorDist that occur when initializing from dict and json representations, as done when loading a bilby results object. In both instances, checks for MultivariateGaussin(Dist) are hardcoded without need, as far as I understand it. Changing these checks to (Base)JointPrior(Dist) solves the problems.
For json, this block
https://github.com/lscsoft/bilby/blob/bad30b7cfd732d16b62322e45db5bb5f9ebc4f3c/bilby/core/utils/io.py#L29-L47
could be updated to
class BilbyJsonEncoder(json.JSONEncoder):
    def default(self, obj):
        from ..prior import BaseJointPriorDist, Prior, PriorDict
        from ...gw.prior import HealPixMapPriorDist
        from ...bilby_mcmc.proposals import ProposalCycle

        if isinstance(obj, np.integer):
            return int(obj)
        if isinstance(obj, np.floating):
            return float(obj)
        if isinstance(obj, PriorDict):
            return {"__prior_dict__": True, "content": obj._get_json_dict()}
        if isinstance(obj, (BaseJointPriorDist, HealPixMapPriorDist, Prior)):

For dicts,
https://github.com/lscsoft/bilby/blob/bad30b7cfd732d16b62322e45db5bb5f9ebc4f3c/bilby/core/prior/dict.py#L208-L308
could be updated to
def from_dictionary(self, dictionary):
        jpdkwargs = {}
        for key in list(dictionary.keys()):
            val = dictionary[key]
            if isinstance(val, Prior):
                continue
            elif isinstance(val, (int, float)):
                dictionary[key] = DeltaFunction(peak=val)
            elif isinstance(val, str):
                cls = val.split("(")[0]
                args = "(".join(val.split("(")[1:])[:-1]
                try:
                    dictionary[key] = DeltaFunction(peak=float(cls))
                    logger.debug("{} converted to DeltaFunction prior".format(key))
                    continue
                except ValueError:
                    pass
                if "." in cls:
                    module = ".".join(cls.split(".")[:-1])
                    cls = cls.split(".")[-1]
                else:
                    module = __name__.replace(
                        "." + os.path.basename(__file__).replace(".py", ""), ""
                    )
                try:
                    cls = getattr(import_module(module), cls, cls)
                except ModuleNotFoundError:
                    logger.error(
                        "Cannot import prior class {} for entry: {}={}".format(
                            cls, key, val
                        )
                    )
                    raise
                if key.lower() in ["conversion_function", "condition_func"]:
                    setattr(self, key, cls)
                elif isinstance(cls, str):
                    if "(" in val:
                        raise TypeError("Unable to parse prior class {}".format(cls))
                    else:
                        continue
                elif issubclass(cls, BaseJointPriorDist):
                    dictionary.pop(key)
                    if key not in jpdkwargs:
                        jpdkwargs[key] = cls.from_repr(args)
                elif issubclass(cls, JointPrior):
                    jpkwargs = {
                        item[0].strip(): cls._parse_argument_string(item[1])
                        for item in cls._split_repr(
                            ", ".join(
                                [arg for arg in args.split(",") if "dist=" not in arg]
                            )
                        ).items()
                    }
                    keymatch = re.match(r"dist=(?P<distkey>\S+),", args)
                    if keymatch is None:
                        raise ValueError(
                            "'dist' argument for JointPrior is not specified"
                        )

                    if keymatch["distkey"] not in jpdkwargs:
                        raise ValueError(
                            f"BaseJointPriorDist {keymatch['distkey']} must be defined before {cls.__name__}"
                        )

                    jpkwargs["dist"] = jpdkwargs[keymatch["distkey"]]
                    dictionary[key] = cls(**jpkwargs)
                else:
                    try:
                        dictionary[key] = cls.from_repr(args)
                    except TypeError as e:
                        raise TypeError(
                            "Unable to parse prior, bad entry: {} "
                            "= {}. Error message {}".format(key, val, e)
                        )
            elif isinstance(val, dict):
                try:
                    _class = getattr(
                        import_module(val.get("__module__", "none")),
                        val.get("__name__", "none"),
                    )
                    dictionary[key] = _class(**val.get("kwargs", dict()))
                except ImportError:
                    logger.debug(
                        "Cannot import prior module {}.{}".format(
                            val.get("__module__", "none"), val.get("__name__", "none")
                        )
                    )
                    logger.warning(
                        "Cannot convert {} into a prior object. "
                        "Leaving as dictionary.".format(key)
                    )
                    continue
            else:
                raise TypeError(
                    "Unable to parse prior, bad entry: {} "
                    "= {} of type {}".format(key, val, type(val))
                )
        self.update(dictionary)

Here, I have updated the names of some variables to better reflect what is happening as well.

Somewhat related to the above, it is currently also not possible to subclass BaseJointPriorDist more than once due to this line:
https://github.com/lscsoft/bilby/blob/bad30b7cfd732d16b62322e45db5bb5f9ebc4f3c/bilby/core/prior/joint.py#L739
Again, the fix while keeping the spirit of the original line would be
        if not issubclass(dist, BaseJointPriorDist):

However, IMO it would be even better to use
        if not isinstance(dist, BaseJointPriorDist):

because it does not make sense for dist to not be initialized.

Hi @JasperMartins, could you email this issue to<EMAIL_ADDRESS>


https://heycam.github.io/webidl/

Idea Title
Be as descriptive as you can with your idea/bug report/question/suggestion.
Issue Details
Add a detailed explanation of what is wrong or what you want implemented.


use pymssql  replace pyodbc to connect mssql in aws glue

Describe the bug
when i use datadiff in aws glue python shell,i can not connect mssql because pyodbc need odbc driver: ODBC Driver 18 for SQL Server ,
but aws glue do not support odbc driver,i hope datadiff can use pymssql to connect mssql .
since pymssql is easier for users to install, can it be turned into a configuration port option?
Make sure to include the following (minus sensitive information):
my scripts:
'''
from data_diff import connect_to_table, diff_tables, disable_tracking
mssql_table1 = connect_to_table(mssql, "table", "id") # mssql is a Connection string
redshift_table2 = connect_to_table(redshift, "table", "id")# redshift is a Connection string
for different_row in diff_tables(mssql_table1, redshift_table2):
plus_or_minus, columns = different_row
print(plus_or_minus, columns)
'''
Error: ('01000', "[01000] [unixODBC][Driver Manager]Can't open lib 'ODBC Driver 18 for SQL Server' : file not found (0) (SQLDriverConnect)")
Describe the environment
python 3.9
glue 3.0
datadiff v0.11.1

Hi @MengruXiao,
Thank you for trying out data-diff and for taking the time to open this issue. We made a hard decision to sunset the data-diff package and won't provide further development or support. Diffing functionality will continue to be available in Datafold Cloud. We have completely rewritten the diffing engine in the cloud over the past few months and have solved the fundamental issues with the original algorithm used in the data-diff package, as well as a better experience for MS SQL Server 2019+. Feel free to take it for a trial or contact us at<EMAIL_ADDRESS>if you have any questions.
-Gleb


[5.5.x] update report collector

Description

This PR introduces the following changes:

propagates --since to planet journal log collector
fixes journalctl binary location inside planet

Type of change

Bug fix (non-breaking change which fixes an issue)
Internal change (not necessarily a bug fix or a new feature)
This change has a user-facing impact

Linked tickets and other PRs

Refs https://github.com/gravitational/gravity/pull/1719

TODOs

[x] Self-review the change
[ ] Perform manual testing
[ ] Address review feedback

Implementation

Testing done

Ran various combinations of gravity report and gravity system report and verified availability of expected data points (e.g. planet journal logs).
Additional information

Also, I saw that https://github.com/gravitational/gravity/pull/1719 removed k8s logs collection using kubectl cluster-info dump - why?

@a-palchikov ah, the intention was that passing --since 0s to gravity report would collect k8s logs using kubectl cluster-info dump. Looks like I made a mistake.
A default since value was specified for both gravity report and gravity system report. So if --since 0s is passed to gravity report gravity system report will be called with unspecified --since and it will defer to the default value.
We can fix this by removing the default value for gravity system report. Or we can explicitly pass 0s to gravity system report when calling gravity report.


[WIP] Germany marriage abroad

https://trello.com/c/14Va6yhg/1230-getting-married-abroad-germany-same-sex
Updated the 9 same sex outcomes to reflect the fact that same sex marriage has now come into law.

I'm closing this so @hannahwhittaker can open another PR using these commits but in a local branch: https://github.com/alphagov/smart-answers/tree/germany-marriage-abroad


Introduce the 20 unsuccessful application cap

Context
With continuous applications,  if there’s no upper application limit in place, we allow for the possibility of an unsuccessful candidates to enter a state of unlimited application choices (i.e. rejected/withdraws adds another.. repeat).
This can cause technical and performance problems in the apply application (also in the performance of the Vendor API or in the monthly report).
To prevent this we need to limit the total number of application choices that a candidate could have during the whole cycle.
Changes proposed in this pull request

introduce limit
prevent submission if the candidate is at this limit
prevent candidate from adding a course choice on the find-to-apply journey

Hitting the limit

Landing from find

Guidance to review
served as a reminder about the inconsistency between calling things "choices" or "applications" so opted to follow any existing convention.
Link to Trello card
https://trello.com/c/qLILAMqE/447-ca-limit-and-validate-the-maximum-number-of-applications-within-a-cycle
Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[ ] Required environment variables have been updated added to the Azure KeyVault

Looks good, but will detect_submitted_applications_with_more_than_the_max_unsuccessful_choices ever return anything?

hopefully not! but i'm being cautious until we see how continuous applications plays out in terms of the different state combinations


Publish new analyzer package.

The currently published package constrains downstream deps to an old version of cli_util.  As of https://github.com/dart-lang/sdk/commit/ba1a6e39f3d0892052e0e8cb4cdcd364d683ed45 this got bumped to the latest (^0.1.0).  It would be awesome to push a new version so the linter can play with the new logging bits in 0.1.0. 👍
Thanks!
cc @bwilkerson

Yep. I heard about this earlier today. We need to determine whether there have been any breaking changes since the last published version. If there are, I think we'll want to publish without publishing the breaking changes, given how recent the last set of breaking changes were.

Fair enough.  It'd be awfully easy to push a version w/ just that change.  Maybe we should just go that route to be safe?

That would be fine with me.

My plan is to cherry-pick this change onto https://github.com/dart-lang/sdk/commit/ba1a6e39f3d0892052e0e8cb4cdcd364d683ed45.  I wasn't planning to create a branch (we haven't consistently), but could.  Either way, how does 0.30.1 sound for a version number?
@bwilkerson @devoncarew

sgtm

Published 0.30.0+1 based on ba1a6e3.
pubspec update to HEAD up for review: https://codereview.chromium.org/2895823002/

Hmmmm.  Consuming this package from the linter is giving me fits now:
Failed to precompile test:test:
Unhandled exception:
Could not import "package:front_end/src/base/api_signature.dart" from "package:analyzer/src/generated/engine.dart": FileSystemException: Cannot open file, path = '/Users/pquitslund/.pub-cache/hosted/pub.dartlang.org/front_end-0.1.0-alpha.4/lib/src/base/api_signature.dart' (OS Error: No such file or directory, errno = 2)
null

May push a quick +2 to get us back to sanity and try again on Monday...

OK, so this'll learn me for doing things on a Saturday night.   I'm having a hard time figuring out what commit corresponds to 0.30.0...
I see https://github.com/dart-lang/sdk/issues/29516, but I'm not sure what the last commit was.

Sorry for all the Saturday night noise.  FWIW I think things are properly sorted now.
For the curious, I resorted to downloading the archive for 0.30.0 from pub, modifying it publishing from there.  (I'm a little embarrassed about this so please, if there was a better way I'm all ears.)
I've got a new PR up to get the pubspec in HEAD up to date: https://codereview.chromium.org/2897783002/.
I think once that lands, sanity is restored and clients can go ahead an enjoy the new goodies in cli_util. :)

Optimistically closing as 0.30.0+2 is up and I've confirmed the linter can consume it. 👍


Drop python 3.5 support

Version 19.6 of Sanic drops support for Python 3.5. Does anyone depend on python 3.5 or have any objections?

@erohmensing What did our customers say?
@tmbo Any objections?
When dropping the support, we can also clean up / remove stuff which we we did for Python 3.5 compatibility:

manual ordering of dicts (e.g. in Domain, TrainingData
casting Path

From Ella:

and all the tests where we had to say the result was one of a few different possible dicts (goes along with dict ordering)

No qualms from the customers yet!

Okay, please give a go in here as soon as you have all the feedback.

@wochinge I'll collect and make sure we hear from everyone on our customer board -- can you check in with Ergo and Helvitia

Scalable + Ergo are fine


Automater: detect blocked PRs

Currently some PRs are stuck. This may be test flakes or real breakages. We should have some way to manage this - github issues, slack notifications, auto /retest, etc are all ideas

After watching these jobs and helping them along, I see a few items causing blockage:

First is a flakey test failing. The fix is to rerun the test, which will hopefully pass and the PR merges.
The PR may also show rebasing needed. We can maybe update the jobs to try and prevent some of this. I suspect some jobs are doing the same thing, like update_deps, and then it's a first one wins. There are two rebase needed issues:

All the changes in the PR are already merged. Then it's imply a close the PR.
Some of the changes are merged.
I think if the Automator logic would close any active PRs IF there are no changes found, that will solve the first issue. The logic already updates the existing PR if there are changes, so hopefully pretty easy to remove the PR if there are none.
We need a way to rerun the failed automator jobs. This rerun, combined with the prior change to close PRs with no changes should either close the PR or update it with changes to unblock the rebase.


README Support lifecycle link broken.

The Link in the main README.md under Support lifecycle  to the Dynatrace support lifecycle for Kubernetes and Openshift is broken and return a 404.
It should probably direct to https://www.dynatrace.com/support/help/shortlink/support-model-k8s-ocp.

Hi, thanks for pointing out, did not notice that the link changed - fixed and updated.
Have a great day!


native_stack_traces package is preventing flutter from upgrading test deps

package:native_stack_traces needs to be updated with the latest versions of several packages (specifically args) since there have been some major releases since it was last published. This is currently one of the packages constraining the Flutter SDK
flutter/flutter#77856

ahh, this has been updated but something else is holding us back...


Add support to ctop

ISSUE TYPE

Feature request

SUMMARY

Add support to ctop tools. Top-like interface for container metrics.
Goal

ctop provides a concise and condensed overview of real-time metrics for multiple containers
as well as an single container view for inspecting a specific container.
Easy to explore containers. ALA tig for GIT

@llaville I suppose you will have to run this on the host os in order to see the running container. I guess it would not make much sense to have this tool inside the container or?

Yes you're right in standard use case.
BTW, I've discover this tool recently, and it was also an occasion to share it !

This might be true, but the Devilbox is not mounting the docker socket to allow Docker in Docker. This would also introduce in a big security risk: https://stackoverflow.com/questions/40844197/what-is-the-docker-security-risk-of-var-run-docker-sock

You right. For security reason, we cannot adopt this tool. I'll close this request.


[5.4] Add includeWhen directive

This is a directive that shortens:
@if ($user->ownsPost($post)
@include('posts.edit-controls', ['post' => $post])
@endif

to...
@includeWhen($user->ownsPost($post), 'posts.edit-controls', ['post' => $post])

This was merged. Not sure why it's not detecting.

@taylorotwell commit hash got changed when you rebased. That's probably the reason.

Not that commit authors are still kept as is. You're just now indicated as the comitter.

Found the commit that added this into the 5.4 branch, just for traceability: 5fe39aa3bac01f49ffaf5d44bf03688690106dbe


Change TMDB to use Plex instead. Thus minimizing reliance on other external systems.

Solved as of v1.0.0


[flink-cdc-pipeline-connectors] Add Implementation of DataSink In Doris

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
Apache Doris is an open Source MPP database, and it‘s being used by more and more enterprises.
we can try to add it to the first pipeline.
You may need to wait for https://github.com/ververica/flink-cdc-connectors/issues/2642 completed in order to deal with different kinds of SchemaChangeEvent..
Solution
No response
Alternatives
No response
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

I'd like to take it.

@JNSimba Hi, now you can try to add Doris pipeline connector as Sink related APIs are done.

Resolved by 4abd86a50082ed92d3f62f5cf66e1686e423205d


fix file creation, removed -n character

First of all, THANK YOU!
This should fix the -n character in the output file.

Hi Christian, thanks for your contribution!
I use echo -n to clear the file if it already exists. It doesn't work for you?
I suspect you use OSX, because I have fixed another issue with mktemp command, which you tried to solve in your code. Can you try this revision? https://github.com/zakjan/cert-chain-resolver/commit/3f4a5770c0c6e589aa8a994c362c57de76f0aebc

The fix for mktemp works on Mac, but i still get the -n character at the beginning of the file.
I guess the most cross platform way to do it the would be to cp /dev/null $CHAINED_FILENAME
Committing now, If you like you can merge it :)

One last thing :)
FILE_SRC_DIR=$(cd "$(dirname $1)"; pwd)
FILENAME=$(basename $1)

openssl x509 -in $FILE_SRC_DIR/$FILENAME -out $CURRENT_FILENAME

Is it really necessary to absolutize path to input file?

I found it didn't work if executed for example:
sudo sh ./resolve.sh ../ssl/ssl.pem output.pem

I had issues with the relative path, now it works for me

So now this PR can contain only the change echo -n -> cp /dev/null, right? Then I merge it.

No I meant that I solved the issue with the relative path with
FILE_SRC_DIR=$(cd "$(dirname $1)"; pwd)
FILENAME=$(basename $1)

openssl x509 -in $FILE_SRC_DIR/$FILENAME -out $CURRENT_FILENAME

Sorry for the confusion..

Ah, I see. Does this fix it for you? https://github.com/zakjan/cert-chain-resolver/commit/4318a6179b336dff9a345010ffbd7f322335d545

yep :)


Settings lost when doing batches

Any Pre-Releases / Beta Releases
none
version <IP_ADDRESS>
Describe the bug or Issue
I have a Template that loads at startup with my video encoder and audio encoder settings saved.  I frequently change the output files size via the size drop down menu or by typing it in.   After selecting my files for batch mode I get an unwanted "Please select a template" menu.  I select the same custom template that is loaded at startup.  This overwrites any changes made to the template that was loaded on startup.  My sizes and settings that I initially entered on the main screen are lost.
Step To Reproduce the Problem

Open staxrip with a custom template loaded at startup
Change the file size
Click on source/open/file batch/add/add files
Select your desired videos click open
Click ok
Menu appears
On jobs screen click load and you will see that the settings changed back to the default templates

Note the file size 1500 has changed back to 203, the initial setting of the template.
If possible, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: [Windows 10 Build 1803
Version <IP_ADDRESS>

Additional context
Add any other Information or additional context that can be Helpful.
I am running this by double clicking the staxrip.exe file.
I don't run any special commands on it that I know of.
this problem repeated on a clean install on a new computer.
I think the online documentation talks about this in 6.2.2 -AddJob:showConfirmation,templateName
templateName: Name of the template to be loaded after the job was added. Empty to load no template.
Adds a job to the job list.
I am not sure if this is related or not.

I would take a look at your settings folder and make sure the .dat and Template folder are named correctly.
You are using a Beta Release, so bugs are expected. It's a risky using Beta Releases. 2.0 Final has not been Released yet.

  Will it say final for final releases instead of stable?

 Will it say final for final releases instead of stable?

Info Screen is Based on odd or Even Numbers. odd = UnStable, Even = Stable.
Take Whats Posted as Fact, Allot of the info Screens Need to be Updated.


Controlled Vocabulary Fields Show Explicit Splitter-Separated Hierarchy and Does Not Automatically Assign Terms

Describe the bug
Apologies for the gory title, I just couldn't think of a better way to explain the issue I'm having.

We have uploaded (multiple) controlled vocabularies for our DSpace instance, but this issue is preserved across the different fields.

The first issue is that when we go into the submission form and select an entry from the controlled vocabulary for a field (in this case, it's dc.subject, the result on the simple item view is a discovery.xml splitter-separated hierarchy for the term, the last term being the term of interest. We would like for the last term to be the only one to appear on the page. However, we do want the full, splitter-separated path of that term to be browseable when clicked on (currently set to valueList).

The second issue is that when we bulk upload these items, we do so only with the final term of the term hierarchy/path attached. So in this example, "Clinical Follow-Up" is in dc.subject and shows up as linked in the onebox Submission Form. However, when we click on the item, it doesn't expand into its full path; it only browses-by the final term. I suspect this is because these entries simply don't contain the full item path, but is there a way to have them automatically link?

These items are being generated by ds-generic-item-page-field:
      <ds-generic-item-page-field [item]="object" [fields]="['dc.subject']" [separator]="', '"
        [label]="'item.preview.dc.subject'"></ds-generic-item-page-field>

And are being made browseable in local.cfg like so:
webui.browse.index.5 = tag:metadata:dc.subject:valueList
webui.browse.link.6 = tag:dc.subject

To Reproduce

Upload/select/enable a controlled vocabulary
Chose a non-top-level vocabulary term for that item
Configure local.cfg like above
Access them on the Simple Item page

Expected behavior

I want the Controlled Vocab terms to be automatically assigned to their place in the Controlled Vocab hierarchy in the items
I want to preserve the "browse link" behavior.

Related work
Link to any related tickets or PRs here.

I generated a quick Python script that will traverse a given XML, assemble the path for each term, and then update the corresponding term in metadatavalue.
https://gist.github.com/aseyedia/bc8754f0144a949618eb6cdf11aaae84


Angular support for the menu-button table column

Pull Request
🤨 Rationale
Angular support for the menu-button table column
👩💻 Implementation

Updated the dependency on @ni/eslint-config-angular to pull in a change to not require the menu-slot attribute on the nimble-table-column-menu-button to be localized in angular
Created Angular module & directive for the table column (@ni/nimble-angular/table-column/menu-button entry point)
Re-exported the menu-button page object from angular (@ni/nimble-angular/table-column/menu-button/testing entry point)
Updated the table in the example app to have a "Color" column that is a menu button

🧪 Testing
Ran auto tests
Manually tested example app changes
✅ Checklist

[ ] I have updated the project documentation to reflect my changes or determined no changes are needed.

@m-akinc, will you buddy this PR for me?


404 Not Found

Browsing to http://mubix.github.io/post-exploitation-wiki/, I receive the following error
404
There isn't a GitHub Pages site here.
:(

Try https://github.com/pwnwiki/pwnwiki.github.io/


FYI on hash reliance

Just wanting to note that changes like https://github.com/benbalter/licensee/pull/171 can be painful if a user is relying on the hash coming out of licensee to be stable :)
I was supporting custom licenses by having a separate mapping of hash to license name, and spent a while scratching my head wondering why my hashes were all incorrect when I migrated to a different machine (and later version of Licensee).
Understood that y'all were fixing bugs, but just wanted to note that the hash is part of the API so that's a consideration for future bugs.

@hyandell Great point, and sorry for the hassle. Yes, changes to how we normalize licenses will change the resulting hash. I was thinking of the change as a bug fix, but see now how that could be seen as a breaking change. I'll be sure to at the very least, call out any potential hash changes in the changelog.

Thanks. Will close as this was just an FYI.
I should probably not rely on the hash directly - I have the file, so after learning it's not a standard hash, I can then record at the repo/file= rather than hash=.


Bug(?) in "schemer_export" Line 179

Line 179:
if ~isnumeric(fname)
I guess it should be
if ~isnumeric(flag_mode)
Regards,

You are correct! Thanks for pointing this out.
After I created this bug, I only tested the code by exporting through the GUI before releasing it...
It's now fixed.

Also, thanks for creating a GitHub account just to open this issue. I appreciate it!


Version and no admin styles fixes

This pull request fixes the following issues:

BEFORE
AFTER

needs a comment about bootstrap thing, but overall LGTM

LGTM


Remove 0 showing next to my positions

Remove 0 showing next to my positions

Changes 👷♀

Fixed issue with condition that was rendering 0 when user has no open vaults next to "My positions" link in navigation.

How to test 🧪
Use two different wallets to test it. One that already has vaults and one that doesn't. When connected, check if number of vaults is displayed correctly and there's no 0 when you don't have any.

there is no longer loading indicator when position number is being loaded, is that expected?

Yes, loading indicator is something specific that I did for Ajna, it was never a part of main LP. This PR is fixing a small issue that @marcinciarka pointed out - due to a way JSX is working, there was a random zero rendering next to the link in some configurations.


URL Cleanup

This commit updates URLs to prefer the https protocol. Redirects are not followed to avoid accidentally expanding intentionally shortened URLs (i.e. if using a URL shortener).
Fixed URLs
Fixed Success
These URLs were switched to an https URL with a 2xx status. While the status was successful, your review is still recommended.

[ ] http://www.apache.org/licenses/ with 1 occurrences migrated to:
https://www.apache.org/licenses/ (https result 200).
[ ] http://www.apache.org/licenses/LICENSE-2.0 with 16 occurrences migrated to:
https://www.apache.org/licenses/LICENSE-2.0 (https result 200).

https://issues.apache.org/jira/browse/LEGAL-457
This is not possible until the ASF elects to alter their license; this may even be accompanied by a 2.1 revision change to avoid confusion. Follow the issue at the link above.


how to use the finetuned mistral model for inference with sglang

how to use the finetuned mistral model for inference with sglang.
Please share the code for this

For the official model, you can just use
python -m sglang.launch_server --model-path mistralai/Mistral-7B-Instruct-v0.2 --port 30000

and follow the usage https://github.com/sgl-project/sglang?tab=readme-ov-file#usage.
For your fine-tuned models, just replace mistralai/Mistral-7B-Instruct-v0.2 with your model path

Tried above  method to load a finetuned mistral model. but it raises the given error
ImportError: /tmp/pip_packages/vllm/_C.cpython-38-x86_64-linux-gnu.so: undefined symbol: _ZN2at4_ops15to_dtype_layout4callERKNS_6TensorEN3c108optionalINS5_10ScalarTypeEEENS6_INS5_6LayoutEEENS6_INS5_6DeviceEEENS6_IbEEbbNS6_INS5_12MemoryFormatEEE


feat: Initial state problem

Let's assume that we have contract like this:
handle(...)

function first(state, action) {
    if(action.caller == ALICE) {
       return {state: [...state, `${ALICE} was here`]}
    }
}

function second(state ,action) {
    if(action.caller == BOB && state.includes(`${ALICE} was here`) {
       return {state: [...state, `${BOB} was here`]}
    }
}

How I can be sure, that ALICE really called first function, if it can be just initialised in this way with initState option.
deploy(src, {initState: [`${ALICE} was here}`] })

This two ^^ examples are NOT differentiable from smart contract perspective.
To easiest solution would be to allow disabling initialState for some contract IMHO

Maybe there is solution, however i don't see at the time of writing

Let's look at this from another perspective.
Let's say that initial state is empty, but ALICE is putting into state sth like:
if(action.caller == ALICE) {
       return {state: [...state, `${BOB} was here`]}
    }

what now? :-)
How can you tell that it wasn't BOB who had set this value?

I believe the only way in this case would be to sign each new change with the caller wallet?
And then, each time you need to use this data in the contract - first validate the signature.

Every one who has read code, and know that it can't be changed (assumptions on blockchain trust).

Well, the same way you can verify the initial state...(as it is kept in the contract deployment transaction).
But...I guess I understand your point of view...we should probably define sth like 'init/constructor' function in the contract code itself - and then the SDK - when it 'knows' the it it evaluating the state from the scratch - calls the init/'constructor' function first - and then moves to evaluating the interactions.
What do you think?

Well, the same way you can verify the initial state...(as it is kept in the contract deployment transaction).
Can I access it within smart-contract code?

Exactly, and we should to run constructor always once, the should be no other way to init contract state.
I'm currently writing smart contract which Offer which can have several stages PENDING->ACCEPTED_BY_BUYER -> ... ->FINALIZED, now I'm not able to assert (from within smart contract) that OFFER passed all this stages, and was not initialised with state appropriate to  ACCEPTED_BY_BUYER stage for example. Of course I can check it by analysing all interactions, however it is not good DX(developer experience) and is less secure in my opinion.

If contract has a 'special' init function (name TBA), the SDK will be responsible for calling such function before evaluating interactions with the state defined in the deployment transaction)

how about WOOT_HACK_WOOT_init ;-)
now, being serious - I would use a convention that good-old angular.js was using for storing its internal stuff - so sth. like

$$init(initialState)
$$_init(initialState)
or maybe __init(initalState)

Not sure how to secure against manual init calls from the contract code....from this perspective it would probably the best to keep the init function in a separate arweave transaction - so that it couldn't be called from the contract itself.
So here's an alternative: consider storing the init function code in the 'data' field of the contract transaction (instead of the initial state json, that is usually kept there). This would nicely separate the standard contract code from the initialization code (and I believe would be more secure - the user would have no means of calling the init function manually - either from the contract code or by creating an interaction)

For me 3. looks the best, this is convention from python I think, pretty popular lang :) Maybe even __init__(initialState).
Maybe we could ask community :)

So here's an alternative: consider storing the init function code in the 'data' field of the contract transaction (instead of the initial state json, that is usually kept there). This would nicely separate the standard contract code from the initialization code (and I believe would be more secure - the user would have no means of calling the init function manually - either from the contract code or by creating an interaction)

Sound perfect. So:

The interaction {function: '__init__'} would check if it is first interaction, if so it will pass deploySourceTx.data as args?
Then everything works normally
If user call as first interaction for example {function: 'blabal} he will get error('call __init__ first), because he haven't called {function: '__init__'} before?

On balance:

Support calling __init (or something like that) function from SDK
__init function is stored within contract
If contract doesn't expose __init function, just don't call it
Throw error on interactions trying to call __init
Calling constructor is possible many times, but only from within contract code
if __init function is exposed initialState is passed as argument
if __init function is NOT exposed use initialState as it is working now
if __init function fails don't deploy contract (i'm not sure is this good idea, but logical)

Approved :-)

I'm thinking about 3 - How we should deduce if smart-contract is using constructor?

add field to manifest {constructor: true}
(+) works with old contracts
(+) the simplest
(-) dev has to provide value on deployment

parse ast of contract
(-) may take too much time if contract is bloated
(+) works in all cases

Be naive and try to call __init if function doesn't exist don't fail. if function exists and fails then fail.
(+) simple
(-) doesnt work on old contracts which, already has method __init to surpass we could event use more sophisticated name.

I would choose 1.
What is your opinion @ppedziwiatr

Let's be explicit here - option numer 1!

released in https://github.com/warp-contracts/warp/releases/tag/1.2.56


Update DataFlow.md

Fixed JS code block formatting (was displaying ``` and not highlighting syntax).

Thanks!


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


No easy possibility in drawing the plot on an existing ax

Matplotlib works in mysterious ways, it would be great if you could pass an ax as an argument in plot_spectrum so that it would plot the bruker data on that same ax.

Hi @kulakarla,
I did a quick fix and the plot_spectrummethod has an optional existing_axargument. I hope it helps!


Allow third party repository locations

Hi there,
I'm using polymer-element-catalog for generating our own project documentation, but there seems that source code's link always go to github, and we're using stash. I'd modified this file in order to allow full http links, instead of github relative repos.
I hope you like it, I think it'll help people like me to use their own repos.
Regards

I signed it!

I signed it!

+1

heh, sent in this pull request two months ago https://github.com/Polymer/polymer-element-catalog/pull/156

+1

+1

Pinging @frankiefu -- we have two PRs on this, can you PTAL?

+1


Don't build netifaces wheel on pip parse

Adds https://github.com/ros2/ros2cli/pull/875 as a patch.
netifaces is unmaintained: https://github.com/al45tair/netifaces/issues/78.
It does not provide official prebuilt wheels for Python > 3.9: https://pypi.org/project/netifaces/#files
We don't have a non-bazel host compiler so building that Python wheel is difficult in our CI.

btw, psutil binaries are only for amd64. I guess you don't need e.g. arm64 at the moment?


Introducing Threads in Java Socket Programming

Article checklist
Attention
Our Peer Reviewer and Content Moderator teams do NOT provide any revisions services. All revisions and edits should be completed on your own forked repo (as to not take up room in the queue) in order for our team to review them all in a timely manner.
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.

As a rule of thumb - please be sure to only submit articles (pull requests) that are fully polished and ready to be published. Be sure to go through our resources documents for extra and 3rd party (vetted) resources to help improve overall technical writing.

Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels, under 300KB in size, and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[ ] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

This closes #issue-number

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

@paulodhiambo Here is the new PR.

Duplicate of #4937

@ahmadmardeni1 I think plagiarism was checked in the previous PR. Thanks

Hello @SteveadeNick
Upon running your article through our 3rd party plagiarism checker it seemed to raise a few flags and the % was higher than we typically accept.
Please see attached PDF - and revisit the article to ensure we are contributing wholly unique and original content.
Be sure to see our resources page to see more info on plagiarism and what is considered as such.
Introducing Threads in Java Socket Programming #5487 Copyleaks Report.pdf

@Kayere thanks. On them right away.

@SteveadeNick please add explanations for your code blocks, at least add inline comments to explain your code.

@paulodhiambo adding right away. Thanks

@paulodhiambo done

Hello @SteveadeNick , thank you for a great article. The article is ready to be published. The headshot pic is missing in your profile. Can you add that and let me know.
Thank you.

@lalith1403 Done. Thanks.

I'll publish this (now that the image has been added)
cc @lalith1403 👍


Solutions page: CTA section is not aligned and text is overlapping

Detailed description
Issue :- CTA Section is not aligned properly and the text is overlapping
Additional information

Issue moved to carbon-design-system/ibm-dotcom-library #3227 via ZenHub


Request skip vote for music cog.

Could we have a skip vote for the music cog please

Could we have a skip vote for the music cog please

Good. I will add this opportunity, but I can’t say when I’ll start working on it.


https://django-userena.org/ gives an ssl error

Unfortunately, in Google Chrome I get an SSL error when I visit any secure page of django-userena.

You should make it clear on django-userena.org that you are in the process of moving. It always gives a bad first impression to hit an error page when expecting a showcase. Keep up the good work anyway ;-)

The DNS has been updated to ep.io. I have also updated the README to non-secure URI's since I don't have a SSL certificate anymore.


[flink] Add CompactManifestProcedure to enable compact manifest manually

Purpose

Come up with a flink procedure, which could compact manifest manually.

Tests

API and Format

Documentation

+1

+1


Log correct message for Found/NotFound views

As per #3502 making sure we use the right messages for when we do or don't find a view.

Hi @ryanbrandenburg, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

:shipit:

:shipit:


Fix VM Retirement Error Message

Issue error log message only if there are invalid src_ids.
The following error log message is unconditional. It should only be issued when src_ids contains at least one invalid vm id.
[----] E, [2020-01-13T12:50:10.880878 #10833:46b5e3c] ERROR -- : MIQ(Vm.make_retire_request) Retirement of [Vm] IDs: [] skipped - target(s) does not exist
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1789806

Backported to hammer via https://github.com/ManageIQ/manageiq/pull/19821

Backported to ivanchuk via https://github.com/ManageIQ/manageiq/pull/19820


[Feature]: new page under devops for heroku

Is your feature request related to a problem? Please describe.
https://www.nsccbpit.tech/devops
create the page for the subject, connect it to the card.
On the page provide all information and resources related to the concerned topic such as tutorials, documentation, youtube channels, articles, courses etc.
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

Hello, i'd like to work on this

Hello i created a PR for this

resolved


List Containerd images and containers

Thanks for helping us to improve k3s! We welcome all bug reports. At this stage, we are also looking for help in testing/QAing fixes. Once we've fixed you're issue, we'll ping you in the comments to see if you can verify the fix. We'll give you details on what version can be used to test the fix. Additionally, if you are interested in testing fixes that you didn't report, look for the issues with the status/to-test label. You can pick any of these up for verification.  You can delete this message portion of the bug report.
Describe the bug
Using the ctr binary and pointing it to the k3s containerd.sock it doesn't show any resources.
Is there any other way to list any running containers / downloaded images?
To Reproduce
Steps to reproduce the behavior:

Run the K3S server and agent.
sudo ctr -a /run/k3s/containerd/containerd.sock images ls

Expected behavior
Shows a list of all the downloaded images.
Screenshots
If applicable, add screenshots to help explain your problem.

As it can be seen, there are pods running, but no images are showing at the containerd.
Additional context
Add any other context about the problem here.

K3S installed with the curl | sh script
K3S being run in an AWS EC2 server.

You can list the running containers with:
k3s crictl ps
And list the downloaded images with
k3s crictl images

As @tcassaert said, the cri-tools (crictl) is included by default, you can run "k3s crictl xxx", here are some examples.
But this issue reminds that we need to add the description of the crictl command in README.md.
$ k3s crictl images
IMAGE                                   TAG                 IMAGE ID            SIZE
docker.io/coredns/coredns               1.3.0               2ee68ed074c6e       12.3MB
docker.io/katacoda/docker-http-server   latest              0d6ee549ae131       2.25MB
docker.io/library/traefik               1.7.9               98768a8bf3fed       19.9MB
docker.io/rancher/klipper-helm          v0.1.2              b8b351c31b0ab       27.1MB
docker.io/rancher/klipper-lb            v0.1.1              4a065d8dfa588       2.71MB
k8s.gcr.io/pause                        3.1                 da86e6ba6ca19       317kB
$
$ k3s crictl ps -a
CONTAINER ID        IMAGE               CREATED              STATE               NAME                 ATTEMPT             POD ID
4f97f3077efa1       4a065d8dfa588       About a minute ago   Running             https                0                   744af145686d7
399ab75962fd1       4a065d8dfa588       About a minute ago   Running             http                 0                   744af145686d7
08ca46f92f977       98768a8bf3fed       About a minute ago   Running             traefik              0                   3a7f5bcdece05
bb4aff46007a5       b8b351c31b0ab       About a minute ago   Exited              helm                 0                   07e8ec42dda57
8a41a9fd2bab8       2ee68ed074c6e       About a minute ago   Running             coredns              0                   ad9c30825e45a
4c9c6fb8b4695       0d6ee549ae131       14 minutes ago       Running             docker-http-server   0                   ae6b8c1748850
$
$ k3s crictl pods
POD ID              CREATED             STATE               NAME                             NAMESPACE          ATTEMPT
744af145686d7       2 minutes ago       Ready               svclb-traefik-54cbdc9d99-wxr8r   kube-system         0
3a7f5bcdece05       2 minutes ago       Ready               traefik-6876857645-r8fx6         kube-system         0
07e8ec42dda57       3 minutes ago       NotReady            helm-install-traefik-5kc9l       kube-system         0
ad9c30825e45a       3 minutes ago       Ready               coredns-7748f7f6df-x29ls         kube-system         0
ae6b8c1748850       15 minutes ago      Ready               http-8654dd748-lhjf5             default          0
$
$ k3s crictl stats
CONTAINER           CPU %               MEM                 DISK                INODES
08ca46f92f977       0.29                11.15MB             53.25kB             15
399ab75962fd1       0.00                905.2kB             24.58kB             9
4c9c6fb8b4695       0.01                1.38MB              40.96kB             12
4f97f3077efa1       0.00                831.5kB             24.58kB             9
8a41a9fd2bab8       0.21                7.979MB             45.06kB             13
$

Any idea what could cause this:
➜  ~ k3s crictl ps
FATA[2019-03-12T10:49:04.198638576-03:00] failed to connect: failed to connect: context deadline exceeded

Any idea what could cause this error ?
➜  ~ k3s crictl ps
FATA[2019-03-12T10:49:04.198638576-03:00] failed to connect: failed to connect: context deadline exceeded

Ops, worked with sudo.

Looks like the question has been answered, so closing

I'll put this here just in case anyone's looking for it like I was.
k3s crictl is great but if you really want to use the ctr binary, try to specify the namespace like this:
sudo ctr -n k8s.io -a /run/k3s/containerd/containerd.sock images ls

I was also able to import a local image this way.

@uudens This is exactly what I'm looking for.
Can you please ellaborate a little bit more on how the local images appearing in
docker images
be accessed using sudo k3s crictl images?

To import a local docker image in k3s, do the following
Export the image to a tar file
sudo docker save custom-image > custom-image.tar
Import image using containerd runtime
sudo ctr -n k8s.io -a /run/k3s/containerd/containerd.sock image import custom-image.tar

Bumping an old issue here, but the last comment about reducing an image to a tarball and then importing seems super unpleasant.
Surely manifests for custom deployments/statefulsets/daemonsets/jobs/etc can use an imagePullSecret and the image names can then be prefixed with the repository of choice?
Eg for a single-host k3s setup images could refer to localhost explicitly or via hosts file mapping of  to <IP_ADDRESS> and it should then be able to obtain images from the local plain docker registry just fine.
Is it possible to use k3s with the local dockerd stack such that all images are already in the same place? This seems like a sensible approach, too.
And finally, off topic, but can the images referred to by the base distribution be specified before launch such that cached/retagged and-or modified images with extra layer(s) could be referred to thus mitigating the risk of someone pulling the pin on an upstream image by changing host repos or renaming their stuff and outright deleting the old names etc. These sorts of occurrences are very regular in the docker world.

Just to point out an easily overlooked point, containerd images are different on different nodes, which means that if you do something manually with containerd, you may need to do the same for multiple nodes.

# Show all images on the k3s
k3s crictl images

# Delete some image/s on the k3s
k3s crictl rmi NAME:TAG

# Show process
k3s crictl ps -a
k3s crictl ps

# See all pods
k3s crictl pods

# See stats
k3s crictl stats

# Import data
podman/docker save NAME:TAG | k3s ctr images import -

how to export image from k3s?

how to export image from k3s?

root@william:~# ./k3s ctr images --help
NAME:
   ctr images - Manage images

USAGE:
   ctr images command [command options] [arguments...]

COMMANDS:
   check                    Check existing images to ensure all content is available locally
   export                   Export images
   import                   Import images
   list, ls                 List images known to containerd
   mount                    Mount an image to a target path
   unmount                  Unmount the image from the target
   pull                     Pull an image from a remote
   push                     Push an image to a remote
   prune                    Remove unused images
   delete, del, remove, rm  Remove one or more images by reference
   tag                      Tag an image
   label                    Set and clear labels for an image
   convert                  Convert an image
   usage                    Display usage of snapshots for a given image ref

OPTIONS:
   --help, -h  show help

root@william:~#


feature: experimentation with MCAD

Feature Description
I would like to use MCAD as one of the options for the job queuing and scheduling interface for KubeStellar.
Proposed Solution
It may be possible to use the appWrapper resource type to complete this task.
Want to contribute?

[ ] I would like to work on this issue.

Additional Context
No response

@dumb0002 has MCAD agent deployed, and example of appWrapper working.

@dumb0002 is working on video to demonstrate MCAD deployment using KubeStellar

keep going in next sprint

MCAD and KubeStellar integration work has been completed. We are now working on the v2 of this integration and we have another issue tracking this work: https://github.com/kubestellar/kubestellar/issues/1684. Hence, this issue can be closed


Prepare separated module for jUnit tests

Because jMock integration (#63) the jUnit tests requires the JDK6+. There are workarounds how to separate target versions for library and tests but from my point of view, that is not the best choise. Because of that the jUnit tests of all modules will be moved into separated jop-tests module.

The solution is described in: https://thomassundberg.wordpress.com/2012/08/21/separating-tests-in-maven/

Resolved


PROD : Visualize dispay Value = "0"  instead of ‘no value’.

When I export the Visualize csv file for following Visualization https://www.visualize.admin.ch/de/v/-u-BR-bpgybi,
I notice that Visualize exports a Zero value for the LF1 time reference, whereas there is no value for the LFI1 period.
This is correct in Lindas.

Open https://www.visualize.admin.ch/de/v/-u-BR-bpgybi
Export csv of the Chart
Open SPARQL-Query
See error : Value 0 is not the same as "No value". It must be fixed in Visualize (for download chart AND dataset) as soon as possible.

Visualize environment and version: PROD : v4.9.3 ([4f0db71]
Browser and version : edge

@Rdataflow : fyi

Thank you for reporting the issue @ortnever we will look into it as soon as possible.
cc: @bprusinowski @adintegra


[New game]: Want to add new game Tether!

🎮 Game Request
Game logic and basic description
there is a ball through which we have to kill all the circles
Point down the features
Game points
Select program in which you are contributing
No response
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

/assign

Hey @nishant0708 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

hey @kunjgit can you add gssoc Label to this

Hey @nishant0708 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


FR-72 - Allow to ignore unknown db colums

this PR should resolve the issue mentioned here -
https://github.com/georgysavva/scany/issues/72
allowing the user to pass in select * statements and ignoring the columns that doesnt exist in the destination struct

@georgysavva heya .. thanks !
i added a test case for it , hope it does it.

@georgysavva can you please create a tag for this release?

Here is the new release containing this feature: https://github.com/georgysavva/scany/releases/tag/v1.1.0
cc: @br3w0r, @talbSE


Add a security policy

Hey there!
I belong to an open source security research community, and a member (@michaellrowley) has found an issue, but doesn’t know the best way to disclose it.
If not a hassle, might you kindly add a SECURITY.md file with an email, or another contact method? GitHub recommends this best practice to ensure security issues are responsibly disclosed, and it would serve as a simple instruction for security researchers in the future.
Thank you for your consideration, and I look forward to hearing from you!
(cc @huntr-helper)

https://github.com/networknt/light-4j/blob/master/SECURITY.md


Decide on offset when listing posts

Let's vote on the offset that we want when listing posts on archive and tag layouts.
#Offset 1

#Offset 2

Offset 1

I think that offset 2 is much. I prefer 1.

Offset 1

@njaremko voted for offset 1 on gitter.
Offset 1 it is then :smile: I will track it under a new issue.


Java client for Vespa

Is your feature request related to a problem? Please describe.
I'd want to use Vespa in a Java application, but I can't find any java client for Vespa (except vespa-feed-client).
Describe the solution you'd like
I want to have a Java client for using Vespa, at least for searching... Of course it would be great if a full-featured java client (like Elasticsearch's java client) is available, but a search client would suffice for now.
Describe alternatives you've considered
It seems that the only alternative to using a Java client, is having to call REST APIs and manually converting the result to Java objects.
Additional context
None

There is a Java client for writing documents: https://docs.vespa.ai/en/vespa-feed-client.html
There is also a standalone library for building queries: https://javadoc.io/doc/com.yahoo.vespa/client/latest/ai/vespa/client/dsl/package-summary.html
however, the most common, and efficient way to use Vespa is to just send HTTP requests containing your data as-is and then use Searchers inside Vespa to build the query from it programmatically, and then the need for a client is somewhat diminished because your request is just a simple http request.

There is a Java client for writing documents: https://docs.vespa.ai/en/vespa-feed-client.html

Yes I saw that, but it lacks some functionalities such as bulk indexing. How should I do that efficiently?

There is also a standalone library for building queries: https://javadoc.io/doc/com.yahoo.vespa/client/latest/ai/vespa/client/dsl/package-summary.html

Thanks, I'll check it out!

however, the most common, and efficient way to use Vespa is to just send HTTP requests containing your data as-is and then use Searchers inside Vespa to build the query from it programmatically, and then the need for a client is somewhat diminished because your request is just a simple http request.

I see. What HTTP client do you suggest for Java? I saw some VespaAsyncHttpClientBuilder & VespaHttpClientBuilder classes in http-utils maven package, are these the correct entry point for sending HTTP requests to Vespa?
And after all, thanks for your great work on Vespa!

Yes I saw that, but it lacks some functionalities such as bulk indexing. How should I do that efficiently?

You can do bulk indexing maximally efficiently with this client since it is based on HTTP2 streaming. Details in https://blog.vespa.ai/http2/
Or, if you mean on the receiving end, all writes to Vespa are applied one by one in real time and has been so for a long time. Special bulk operations aren't really more efficient when this is done right.
On HTTP clients, we don't want to recommend any since we haven't done any systematic evaluation. We have most experience with the Apache HTTP client. That surely works well but so does most others probably. See also this on HTTP best practices (for cloud, but most of it generally applicable): https://cloud.vespa.ai/en/http-best-practices

You can do bulk indexing maximally efficiently with this client since it is based on HTTP2 streaming. Details in https://blog.vespa.ai/http2/

Thanks!
One other question: do you recommend enabling compression on HTTP/2? I found nothing about it in the docs.

Yes, you should generally use compression if the queries or writes are large.

Also compression is not supported in the vespa-feed-client.
That is correctly observed, and that is something we plan to address. Ideally, we want to be able to support zstd compression, see https://github.com/vespa-engine/vespa/issues/18644.

Thanks for great questions! We have made a few updates to the documentation, this is useful - closing the ticket :-)


[stable/airflow] Allow users to specify a secret containing the git url

Signed-off-by: Edward Young<EMAIL_ADDRESS>What this PR does / why we need it:
Allow users to specify a secret containing the git url used for git sync and git clone.  This is useful if the git url contains sensitive information such as a token.
If dags.git.repoSecret is defined, the key-value pairs will be added to the git-sync and git-clone containers as environment variables.  The secret must contain a key named DAG_GIT_REPO with the git url as its value.
If both dags.git.repoSecret and dags.git.url are provided, the url in the dags.git.repoSecret will be used.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @maver1ck

/ok-to-test


quetsion for download packages

Hello, this is new user for your notebook, I am trying to using your notebook to feed my data, but I had seen the tensorflow version is not mentioned, which version you download your local machine? beside that, this is colab notebook? or notebook from locals?

Hi, it is a colab notebook and the last time I worked on this was in July 2023 so I'm not sure of the TF version :/


Logout happening periodically despite passing disableIdle: true in idleOptions

Describe the bug
During local development I noticed that I was getting logged out after iterating on something for a while that requires being logged in.
I started doing AuthClient.create({ idleOptions: { disableIdle: true }) to try and prevent this but it's still happening. I stepped through the code and I can confirm that an idleManager isn't being created.
Is this expected behavior, or a known issue?
To Reproduce
Steps to reproduce the behavior:

Create an auth client as described above
Authenticate using local internet identity
Leave a browser window open for a while
Refresh the page after a while
Confirm that you are logged out

Expected behavior
You remain logged in until you explicitly log out
Desktop (please complete the following information):

macOS Monterey Version 12.2.1 (21D62)
Brave Version 1.36.119 Chromium: 99.0.4844.83 (Official Build) (arm64)

Additional context
$ cat package-lock.json | grep dfinity
    "@dfinity/agent": {
      "resolved"<EMAIL_ADDRESS>    "@dfinity/auth-client": {
      "resolved"<EMAIL_ADDRESS>    "@dfinity/authentication": {
      "resolved"<EMAIL_ADDRESS>    "@dfinity/candid": {
      "resolved"<EMAIL_ADDRESS>    "@dfinity/identity": {
      "resolved"<EMAIL_ADDRESS>    "@dfinity/principal": {
      "resolved"<EMAIL_ADDRESS>

I took a look at local storage and can see that delegations under ic-delegation have an expiration field so I assume that's related to what I'm experiencing.

I just experienced this again. Confirming that local storage doesn't contain the ic-identity and ic-delegation keys anymore.

I suspect that maxTimeToLive is relevant here.
https://forum.dfinity.org/t/clarity-on-the-maxtimetolive-passed-in-authclient-for-internet-identity/13493?u=paulyoung

Do you think that maxTimeToLive is not being set correctly? The new default has been lowered to 8 hours, and came in conjunction with the idleManager changes. DisableIdle is not meant to have an affect on the maxTimeToLive setting

Just for posterity it seems that passing both  disableIdle: true, // set to true to disable idle timeout,         disableDefaultIdleCallback: true, does the trick

Hmm, disableIdle intuitively ought to cover the default callback too, so I should make sure that is correct


Migrate JSkat base module to Kotlin

Kotlin enables multiplatform re-use of JSkat base code not limited to Android.
Kotlin leads to less boiler plate code that is easier to read.
Kotlin is also more fun.

One good start would be migration of all events and command to Kotlin data classes.
Another starting point is the big mutable SkatGameData class. This one should be converted into an immutable data class with copy() operations to create the next iteration after a player did a move.


Fix typo in shape_editor.rs

targetting -> targeting

Can you please roll this into another PR that makes other feature/bug-fix changes? We generally don't encourage tiny isolated comment typo fixes except when they're included in other existing PRs.


Changes to lists of course users and groups

DynamicTable and BootstrapButton classes now allow custom class names
table columns in group and user lists are now aligned
actions in group list are now displayed as buttons, instead of a dropdown menu

Coverage remained the same at 34.806% when pulling 68e17efa84ead4ccdb25f03643a32b8bdc844ed3 on ui_list_buttons into cea3e8b98206e3f4c635044f11add2aaf9637876 on master.


Constants and expressions support

Does RgGen support constants and expressions? For example, I would like to define two module constants X and Y, and then I would like to calculate different values utilizing these constants in the expressions.

Can you tell me more details?
I'd like to know register map snippet which you want to write.

Pseudo description:
register_blocks:
  - constants: {X: 1, Y: 2}
  - name: block_0
    byte_size: 256
    registers:
    - name: register_0
      bit_fields:
      - { name: bit_field_0, bit_assignment: { width: 4 }, type: rw  , initial_value: X ** Y }

You can do this by writing your register map by Ruby format.
x = 1
y = 2
register_block {
  name 'block_0
  byte_size 256

  register {
    name 'register_0'
    bit_field { name 'bit_field_0'; bit_assignment { width: 4 }; type :rw; initial_value x**y }
  }
}

But other formats such as YAML, Excel, etc, do not support.


Adaptation UI: Enter key hides keyboard, selects next field on iOS

Adapt It Mobile Version: 1.0 beta 9
Operating System: iOS
Severity: 3 - losing on-screen keyboard; interrupts input flow
Description: When adapting on iOS, if the user clicks the on-screen Return key, the keyboard is dismissed and the input focus moves to the next target edit field. This interrupts the input flow; instead, the keyboard should stay visible when the focus moves to the next field.
Steps to Reproduce:

On iOS, launch AIM and start adapting
With the focus in a target edit field, click on the Return key (on-screen keyboard).
--> Note that the focus properly moves to the next target edit field, but the keyboard goes away.

Fixed, but the fix is in the Cordova framework and NOT in the checked-in code (ugh). Details:
The cause of this issue is an attribute of mobile safari -- keyboardDisplayRequiresUserAction. This boolean value on a web view controls whether mobile safari can suppress programmatic shown / hiding the keyboard (e.g., by focusing a field in Javascript). The comments I found online assumed that this was to get rid of bad javascript behavior by internet devs.
For our Cordova app -- which wraps a mobile safari web view -- this property is in the framework itself and needs to be changed in the file CDVUIWebViewNavigationDelegate.m::(void)webViewDidFinishLoad:(UIWebView*)theWebView. We need to add the line:
    theWebView.keyboardDisplayRequiresUserAction = NO;

That will cause mobile safari to behave like other browsers, and display the keyboard when we focus on the next target phrase. So... this needs to be added to our release checklist.


Refactor serialization to converter.

This follows the patterns of the Go and Java SDKs and allows
interoperability between Ruby and the other SDKs by converting arguments
to/from payloads in the same way.
Serialisation of protobufs has been left as-is because it is not
affected by interoperability issues.

I don't see how we would know which payloads are legacy and which are not, they both set encoding as json/plain but one has a nested array which should be unpacked and one does not. Testing for an array feels very brittle.

The composite encoder is to allow people to match the encoding stacks they are using from other SDKs. It's the same system used in Go/Java SDKs.

Added a Legacy converter as discussed on Slack. The idea would be to roll the gem out with Temporal.configure { |config| config.converter = Temporal::Client::Converters::Legacy.new }. Once all workflows have completed or been restarted all payloads should be marked with json/legacy. At this point it should be safe to remove the converter configuration, the default stack will encode new payloads with the correct semantics but be able to handle legacy payloads when reading.

The composite converter provides the same API as the others, they are interchangable.

@robholland I think you're mixing 2 separate interfaces together making them broader than they should be. This is how I see it — composite interface deals with Payloads, while all other converters deal with a Payload with a known "encoding". This way whoever is using the gem can easily extend it:
configuration.add_converter_for("my_encoding", MyConverter)
configuration.add_converter_for("my_encoding_2", MyConverter2)

without having to redefine the whole composite. It is also much clearer which converters are being override this way (if any)

@robholland I actually think that Base class for converters was a good idea. If anything it'll demonstrate how people can implement their own converters. The problem was that Composite does not fit that interface — it doesn't have encoding and has extra methods.
Please let me know if you want to get on a call to resolve this, otherwise I feel like we're getting stuck here

The composite converter provides the same API as the others, they are interchangable.

Clearly they are not because you can't do:
Converter::Composite.new(converters: [Converter::Composite.new])

Please can you just make whichever changes you would like to get this merged?

@robholland another thing I've noticed — we are not yet using new converters for converting activity or workflow headers. Is this intentional or just accidentally missing?

It was missed, the current code doesn't serialize them, I don't know if the current code for headers actually works?

@robholland it should work, yes. It's also pretty straightforward, I believe each header is serialized using a single payload

The Go and Java SDKs would produce the same Payloads structure given those workflow inputs. As a caller of to_payloads directly you see nested arrays, but note that a user does not. They would just pass inputs as an option to start_workflow or similar. inputs becomes the outer array and is special cased because we need to differentiate between individual inputs so that they go into their own payload. The Java and Go SDKs would produce the same payloads structure when given those same inputs (translated to their appropriate primitive types).

Hi @robholland and @antstorm, thanks so much for putting this together guys!  We would really love to use this over at Instacart where we're deploying temporal.  We've currently implemented e2e workflow/activity encryption using the standard SDK DataConverter configurations.  We were going to work around the lack of DataConverter in the coinbase ruby SDK but saw this issue!  Is work on this paused or will this resume?  We can wait a bit on delivery if it's near completion and can even help contribute if needed.  Thanks!

Hey @etherops, this is still active. I'll be working on creating an example that's interoperable with the cryptconverter in https://github.com/temporalio/samples-go/tree/f9c899850c39338df85df156a958ceb9e035cc5a/cryptconverter

@robholland this is fantastic, thank you! is there anything I can do to help get the feature merged?

@etherops testing alongside the go code is great if you have the opportunity, but we plan to get this reviewed, polished and merged soon.


[W5][W09-2] Rohan Dev

Sorts all entries in the addressbook in alphabetical order, and then lists the entries out to display.

It seems like your way of implementing the Sort command is slightly different from mine, I wonder if there are any significant differences that we can learn from. Nice command to have tho!

The sort command is certainly a useful one.
Perhaps, the sort method can be implemented in the unique person list class, to remove the need of a getter in the uniquepersonlist class.


Pass on opt_fast to parent clear function in ol.source.ServerVector

Currently the opt_fast argument is not available for ol.source.ServerVector but it should be since this class calls the parent method.

PR updated @elemoine

@bartvde, thanks for finding this.


[New API] :  Gemini api implementation

Brief Description 🤓
This is an. api which we could use to add image to get descriptive explanation of that image and as well as generate some content or rewrite some content
How will you develop it?🤩
create 2-3 post apis for tht and they need a gemini API key i have implemented it
Deployment
Yes
Select the open source program you are contributing to
Other
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINES of this project.

Greetings @Sahilll15🎀 ! , We are excited to have you dive into another issue with us!🚀
Your involvement in our project has been invaluable, and we're confident that your skills and insights will help us conquer this challenge😇.  Use command /gssoc24 to self assign the issue.  This issue represents an exciting opportunity for us to improve and enhance our project, and we are thrilled to have you on board.
We request you to follow CONTRIBUTING GUIDELINES.
Looking for your PR soon!
In case of any issues, you can contact us on Discord.❣

/gssoc24


Remaining work for v1.0

[ ] If two notes have the same name, then do not overwrite and record error
[ ] Logger: it would be good to have a good logging system. I believe we should also write the log for the list of files copied.
[ ] End to end testing: the end to end testing is quite flaky, add a few paths
[ ] Cli refactor: commands should go in the /commands file
[ ] Readme + Documentation
[ ] Refactor all the import inside on file (as per Deno's documentation) to avoid having to reimport all the times
[ ] Handle images (note: they should simply be a file, and handled by the link manager)
[ ] Copy the files across
[ ] Add command to clear old backup
[ ] Add command to change the configuration

Keep up to good work, you're doing great Alo. This is a solid CLI.

Closing the issue as it has been broken into single, atomic issues.


Reduce stack trace option

Currently I am finding using Jasmine2 if a test fails I am getting excessive stack traces resulting in pages of scrolling simply to find the failing test. Is there any way to configure the stack traces at all?

No, currently there is none.

@johanneswuerbach ok thanks. Editing https://github.com/airportyh/testem/blob/master/public/testem/jasmine2_adapter.js is the easiest option to achieve what I want, however, if I create a new custom adaptor based on this file Jasmine seems to struggle finding window. Any tips?

I never worked with jasmine 2, could you provide a repo to reproduce this? There are a bunch of files required for jasmine2, maybe you need to load jasmine2-boot.js manually? https://github.com/airportyh/testem/search?utf8=✓&q=jasmine2

Good point, I shall have a tinker. Thanks


Distinguish between different VDDK validation errors

There are multiple cases that can lead to a "VDDK Init image is invalid" error message for a migration plan. They are currently handled with a single VDDKInvalid condition.
This patch adds a new error condition VDDKImageNotFound (and a new advisory condition VDDKImageNotReady) to help diagnose an issue of a missing image or incorrect image url.
Resolves: https://issues.redhat.com/browse/MTV-1150

I just realized that this is probably not a full fix.
Example scenario:

I enter a URL A for the vddk image and create a migration plan with this provider.

A validation job is started for this vddk init image
the image cannot be pulled (for whatever reason)
with the patch above, the validation job will indicate a VDDKImageNotFound error.

I change the vddk image in the provider to a different url B

Due to the vddk url change, we kick off another validation job
Results the same as above

Now I change the URL back to A

due to the vddk url change, we try to validate the vddk init image again, so we try to ensure a validation job exists
we find the cached job from step 1 that has the same labels, we don't create a new one and thus no validator pod is started and we don't try to re-pull the init image
since no new job is started, there are no pods to query about InitContainerStatuses, and the only thing we know is that the job failed
we now report VDDKInvalid instead of VDDKImageNotFound

The new patch I just pushed does not yet address the issue I discussed in this comment: https://github.com/kubev2v/forklift/pull/969#issuecomment-2248506437

@jonner yeah and there's even more basic scenario that we don't handle so well -

we try to validate URL A
the job fails on pulling the image
we sort out the issue with pulling the image from URL A
-> the job needs to be recreated in order to validate A again

honestly, we were not really sure whether it makes sense to add this job back then because the only case we've heard of about the VDDK image was when someone created it on a filesystem that doesn't support symbolic links.. but apparently, we see that this job actually detects issues in the field and in that case it makes less sense to expect from users to clone the plan or archive-and-recreate the plan in order to initiate another job...
but more specifically about what you wrote above, since no new job is started, there are no pods to query about InitContainerStatuses - wouldn't we still have the pod that ran before?

@jonner yeah and there's even more basic scenario that we don't handle so well -
1. we try to validate URL `A`

2. the job fails on pulling the image

3. we sort out the issue with pulling the image from URL `A`
   -> the job needs to be recreated in order to validate A again

honestly, we were not really sure whether it makes sense to add this job back then because the only case we've heard of about the VDDK image was when someone created it on a filesystem that doesn't support symbolic links.. but apparently, we see that this job actually detects issues in the field and in that case it makes less sense to expect from users to clone the plan or archive-and-recreate the plan in order to initiate another job...

It's true that the more basic scenario has those issues, but at least it was consistent before: It failed with VDDKInvalid on the first run, and any subsequent changes didn't change that. But in the scenario I described, it would fail with VDDKImageNotFound on the first run and then any subsequent checks would report VDDKInvalid, which adds confusion, I think.

specifically about what you wrote above, since no new job is started, there are no pods to query about InitContainerStatuses - wouldn't we still have the pod that ran before?

Well, you know the code better than I do, but in my testing, the pod related to the job doesn't seem to be retained after the job completes. Specifically, after the job completes, the call to cxt.Destination.Client.List() returns no pods. Should the pod still be available somewhere?

@jonner yeah and there's even more basic scenario that we don't handle so well -
1. we try to validate URL `A`

2. the job fails on pulling the image

3. we sort out the issue with pulling the image from URL `A`
   -> the job needs to be recreated in order to validate A again

honestly, we were not really sure whether it makes sense to add this job back then because the only case we've heard of about the VDDK image was when someone created it on a filesystem that doesn't support symbolic links.. but apparently, we see that this job actually detects issues in the field and in that case it makes less sense to expect from users to clone the plan or archive-and-recreate the plan in order to initiate another job...

It's true that the more basic scenario has those issues, but at least it was consistent before: It failed with VDDKInvalid on the first run, and any subsequent changes didn't change that. But in the scenario I described, it would fail with VDDKImageNotFound on the first run and then any subsequent checks would report VDDKInvalid, which adds confusion, I think.

yeah, I agree

specifically about what you wrote above, since no new job is started, there are no pods to query about InitContainerStatuses - wouldn't we still have the pod that ran before?

Well, you know the code better than I do, but in my testing, the pod related to the job doesn't seem to be retained after the job completes. Specifically, after the job completes, the call to cxt.Destination.Client.List() returns no pods. Should the pod still be available somewhere

unless I'm missing something, I read the changes in this PR as introducing logic that says: "once the job completes, we check the status of the pod that was triggered for the job to figure out the reason for a failure", so it assumes the pod exists also when the job is completed with a failure, right?
it could be that there's cleanup of the job's completed pods at some point (not by us but by k8s/OpenShift), I think I also saw it happening (having a completed job without associates pods), but I was puzzled by that part that suggests it's always the case that completed jobs have no associated pods

unless I'm missing something, I read the changes in this PR as introducing logic that says: "once the job completes, we check the status of the pod that was triggered for the job to figure out the reason for a failure", so it assumes the pod exists also when the job is completed with a failure, right?

Not quite. What happens is that while the job is running, validateVddkImage() checks the vddk-validator-* pod associated with the job and sets the condition to VDDKImageNotReady if the pod is waiting.
At some point, the image will either be pulled (and the pod will start running), or the pod will time out waiting for the image. In the first scenario, the job should run to completion and either succeed or fail based on the validation result. In the second scenario, the job will fail.
Immediately after the job finishes (either successfully or due to timing out), the next call to validateVddkImage() will not find a pod associated with that job (in my  testing cxt.Destination.Client.List() returns no pods at this point). If the job has a JobFailed condition, the only indication about why the job failed is checking whether I have set a VDDKImageNotReady status condition on the plan during the previous reconcile (this is the reason that I made the VDDKImageNotReady  condition durable: so after the pod disappears, the next reconcile still has a hint about what might have caused the job to fail).
But the above description does not apply to a cached job. In my experience, there are no pods returned for a cached job, so there's no way to determine the failure reason in this case.
But maybe I am simply doing something wrong and there is a way to get pods for a completed job that I don't know about?

@jonner I've been investigating the issue with pod deletion, and it turns out that it happens only when the pod is not fully created. If the VDDK image is invalid (the pod is unable to pull the image) and the job reaches the VDDK_JOB_ACTIVE_DEADLINE limit, the job fails, and the pod is deleted. However, if the job is successful, the pod remains as it is. Adding additional configuration to the job spec won't assist in this case.
After discussing this with @ahadas , we believe the best approach is to remove the VDDK_JOB_ACTIVE_DEADLINE limit from the job and handle it at the plan level. We'll add a condition to the plan controller to check if the job is still running and if the time exceeds the VDDK_JOB_ACTIVE_DEADLINE parameter. If it does, we will propagate the pod status to the plan without failing the job and notify the user of the issue. This approach also addresses another issue: since the job remains running, if the user updates the VDDK URL with a correct one, the job should successfully complete under the same job and plan.
What do you think of this solution?

Interesting idea. There are some details of your proposed solution that aren't yet 100% clear in my mind, but I will investigate. It sounds promising.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 101 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@650d73d). Learn more about missing BASE report.
Report is 37 commits behind head on main.

Files with missing lines
Patch %
Lines

pkg/controller/plan/validation.go
0.00%
101 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #969   +/-   ##
=======================================
  Coverage        ?   16.22%
=======================================
  Files           ?      112
  Lines           ?    19841
  Branches        ?        0
=======================================
  Hits            ?     3220
  Misses          ?    16337
  Partials        ?      284

Flag
Coverage Δ

unittests
16.22% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


tcmu-runner.spec: fix the rpm build errors

fix the duplicate files error by providing the exact file list
fix debugsource file error by marking debug_info as nil.

Fixes: #491
Signed-off-by: Amar Tumballi<EMAIL_ADDRESS>

@lxbsz @pkalever @mikechristie please take a look and review!


[MODEL] convert gpt2 model

Description
convert gpt2 model
Checklist
Essentials

[x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[x] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage
[ ] Code is well-documented

cc @dmlc/gluon-nlp-team

Add some test cases in test_models_gpt2.py

One test you may consider is to check that these two cases will generate the same states:

feed the input tokens one by one
feed K tokens at a time

One test you may consider is to check that these two cases will generate the same states:

feed the input tokens one by one
feed K tokens at a time

solved


How to publish a string with length bigger than 255

I have a idl file ,which define a msg with only one string.
then , i tried to publish this kind of msg with a string longer than 255 bytes, the publisher work seem to be ok , but the subscriber cannot receive the msg.
Is there anyway i can fix this?
Except i convert the string to vector

I have solved the problem
just modify the idl file like this :
struct UpgoingMsg
{
string<10000> upgoing_msg;
};
then the maximum size of the string will be 10000


UI enhancement: Add forward history navigation

The Issue
Steps to reproduce

Open LBRY
Do a few searches or view a few videos, then push the back arrow
See if you can go the other way in your history (forward). I dare you. ;)

Expected behaviour
Tell us what should happen
There should be a -> arrow on the interface next to the <- arrow so you can go back and forth in your browsing history.
Actual behaviour
Tell us what happens instead
You can go backwards in history. That's it.
System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

Fixed in #481


Completion and chat completion functions

import aiohttp

base_urls = ['https://a.z-pt.com', 'http://chat.darkflow.top']

async def generate_response(prompt):
    endpoint = 'https://gpt4.gravityengine.cc/api/openai/v1/chat/completions'
    headers = {
        'Content-Type': 'application/json',
    }
    data = {
        'model': 'gpt-3.5-turbo-16k-0613',
        'temperature': 0.7,
        'messages': [
            {"role": "system", "content": "INSTRUCTIONS HEREEEEEEEEEEEE"},
            {"role": "user", "content": prompt},
        ]
    }
    for attempt in range(2):
        try:
            async with aiohttp.ClientSession() as session:
                async with session.post(endpoint, headers=headers, json=data) as response:
                    response_data = await response.json()
                    choices = response_data['choices']
                    if choices:
                        return choices[0]['message']['content']
        except aiohttp.ClientError as error:
            print(f'Error making the request with {endpoint}: {error}')
            if attempt < 1:
                print('Retrying with a different base URL.')
                break

    print('All base URLs failed to provide a response.')
    return None

async def generate_completion(prompt, max_token=None, temp=None):
    endpoint = '/api/openai/v1/engines/text-davinci-003/completions'
    headers = {'Content-Type': 'application/json'}

    async with aiohttp.ClientSession() as session:
        for base_url in base_urls:
            url = base_url + endpoint
            async with session.post(url, headers=headers, json={'prompt': prompt, 'max_tokens': max_token or 2048, 'temperature': temp or 0.7}) as response:
                if response.status != 200:
                    continue
                response_data = await response.json()
                response = response_data['choices'][0]['text']
                return response

    return None

Closing since it's broken


card listview not displayed as expected on android mobile browser

hi
i'm created one app framework7 card view component,but card view image not displayed as expected,only half image showed

Do you have a screenshot?

it solved yesterday that involved css property ,i changed css property then looks like nice!


Modify pdb_rslice.py to NOT print PDB headers

@CunliangGeng I like consistency. I'm OK with this change, provided all scripts behave the same. So, if you want to go ahead and change this for everyone, I'll gladly accept!

But sometimes you might want to keep the header... may-be a flag for this option then?
=================================
Prof. A.M.J.J. Bonvin | Faculty of Science
Utrecht University | bonvinlab.org
<EMAIL_ADDRESS>| T: +31 30 2533859

On 22 Mar 2017, at 03:35, João Rodrigues<EMAIL_ADDRESS>wrote:
@CunliangGeng I like consistency. I'm OK with this change, provided all scripts behave the same. So, if you want to go ahead and change this for everyone, I'll gladly accept!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@amjjbonvin Would a strip_header tool be more helpful? One more tool, but easier to write than flags everywhere.

Good idea!
=================================
Prof. A.M.J.J. Bonvin | Faculty of Science
Utrecht University | bonvinlab.org
<EMAIL_ADDRESS>| T: +31 30 2533859

On 22 Mar 2017, at 07:20, João Rodrigues<EMAIL_ADDRESS>wrote:
@amjjbonvin Would a strip_header tool be more helpful? One more tool, but easier to write than flags everywhere.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Commit 66242cd solves this.


Abao delete gear test failing after related change/fix

https://travis-ci.org/scitran/core/builds/211166055
@ambrussimon-invenshure This test may get skipped to resolve the CI failure. Final resolution would be to get gear deletion working at a minimum in abao or python integration tests.

InvalidId: 'undefined' is not a valid ObjectId, it must be a 12-byte input or a 24-character hex string

Looks like this test never worked, and making the endpoint function exposed that. I'll push a skip.

This is odd, I remember running into and fixing this once - maybe I somehow skipped that commit, but then I don't get how CI seemed green on the freshly rebased branch before the merge to master. Anyhow, sorry and thanks!

Ah, that was because I ran into this when fixing gear deletion on the increase-coverage-of-api-jobs-handlers branch which isn't merged yet. @ryansanford gear deletion coverage was added to py int tests there, too.


Added rel="noopener noreferrer" in anchor and alt in image tag

What:

1. Added rel="noopener noreferrer"
When a link opens a URL in a new tab with target="_blank", it is very simple for the opened page to change the location of the original page because the JavaScript variable window.opener is not null and thus "window.opener.location can be set by the opened page. This exposes the user to very simple phishing attacks.
To prevent pages from abusing window.opener, use rel=noopener on <a href=> to force its value to be null on the opened pages. With this in place, window.opener is null in Chrome 49+, Opera 36+, Firefox 52+, Desktop Safari 10.1+, and iOS Safari 10.3+. For older browsers, use "noreferrer". Cumulatively, rel="noopener noreferrer" is the safest way to mitigate this vulnerability.
2. Added alt attribute to the image tag
3. Changed all-contributer-html.js to generate safe anchor tag and image element

Issue number: N/A

@Zaky7 thank you very much for your contribution!

@Zaky7 thank you very much for your contribution!

@tomastrajan thank you 😄  for your humble reply. Nice project for beginners starting with ngrx and angular material :)


⚠ Bofrost.de has degraded performance

In 7c9c302, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1892 ms

Resolved: Bofrost.de performance has improved in a9d861f after 13 minutes.


AWS Production Checklist Clock Doc Update

Re: Production Checklist Clock
Issue Description
Verify whether Amazon Time Sync Service is sufficient
Suggested Resolution
if additional steps are required, possibly suggest a note

Default RHEL AMIs from red hat point chrony to red hat ntp servers with the following lines:
server 0.rhel.pool.ntp.org iburst
server 1.rhel.pool.ntp.org iburst
server 2.rhel.pool.ntp.org iburst
server 3.rhel.pool.ntp.org iburst
we should advise redhat users to convert to the Amazon Time Sync service server when solely on AWS

basically, we want them to replace those lines with:
Open the /etc/chrony.conf file using a text editor (such as vim or nano). Verify that the file contains the following line:
server <IP_ADDRESS> prefer iburst
If this line is not present, add it. Comment out any other server or pool lines. Restart the chrony service.

customers can confirm the change by running the command:
chronyc sources -v
the top return line should look similar to:
^* <IP_ADDRESS>               3   6    17    43    -30us[ -226us] +/-  287us
The * indicated that it's the preferred time server.  If they properly commented out the other ntp servers they shouldn't see any other sources

@keith-mcclellan Looking at the Amazon doc for this and a few additional parameters are included in the required line:
server <IP_ADDRESS> prefer iburst minpoll 4 maxpoll 4
Are these necessary?

Necessary, no - they just control how frequently chrony polls the time server.  I think they should be fine to add but I haven't tested with those options set.

OK, thanks. I will keep those parameters in our doc, just to be consistent with the Amazon guidelines that we link.