[Release-1.24] Bump various dependencies for CVEs

Proposed Changes
NOTE: Each dependency change has applied in separate commits to review easier. Will be squashed on merge.
Fix Trivy CVEs that come from the go.mod dependencies:

Update the terratest library used in E2E terraform tests, was pulling in a large number of CVE from indirect hashicorp dependencies. Removed gax-go replace, it is a indirect dependency and needs the newest version.
golang.org/x/net is used by flannel. Updated the "replace" version with the version used by flannel v0.21.1-k3s1.23
golang.org/x/sys is used by containerd. Updated the "replace" version with the version used by containerd v1.5.16-k3s1
Update wrangler to latest version

Before:
$ trivy fs . --ignore-unfixed
Total: 12 (UNKNOWN: 0, LOW: 1, MEDIUM: 5, HIGH: 5, CRITICAL: 1)

After:
$ trivy fs . --ignore-unfixed
Total: 1 (UNKNOWN: 0, LOW: 1, MEDIUM: 0, HIGH: 0, CRITICAL: 0)

The last cve is related to /x/net, we need to update it in flannel first.

Types of Changes
Dependency updates

Verification
make passes

Testing
N/A

Linked Issues

User-Facing Change

Further Comments

Codecov Report
Patch and project coverage have no change.

Comparison is base (cabeae0) 9.63% compared to head (2d75fbd) 9.63%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff              @@
##           release-1.24   #7042   +/-   ##
============================================
  Coverage          9.63%   9.63%
============================================
  Files               148     148
  Lines             10777   10777
============================================
  Hits               1038    1038
  Misses             9521    9521
  Partials            218     218

Flag
Coverage Δ

unittests
9.63% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Exported keys deleted from download dir when uninstalling

When Uninstalling. Why? Typo in title.

https://stackoverflow.com/questions/35209375/android-6-0-external-storage-files-being-deleted-upon-app-uninstall/35257122

The wording of the popup has been updated to warn the user about this annoying side effect of uninstalling Riot
Available in Riot 0.8.15


Display desktop notifications

Related: https://github.com/sindresorhus/notifier-for-github-chrome/issues/12

Since the extension currently only gets the number of notifications, it'll be needed to switch to the notifications api, like on the chrome version.

Apple has deprecated JS-based Safari extensions, so I'm deprecating this project.


Beachball on iMac Intel 2019

My environment:
Enchanted runs as a client on an iMac 2019, Intel Core i9, 64 GB RAM, macOS 14.5
The LLM runs on a Windows 11 PC (server) with a Nvidia RTX 3070 under Ollama, which is really fast.
Problem:
Typing a prompt in Enchanted gives a beachball cursor for several seconds. Input freezes during this.
Assumed reason:
On the server side, I see some kind of ping from Enchanted which seems to check if Ollama is reachable.
I assume that these pings do not run async on the client side, or the ping rate is too high.
Maybe the server's presence only should be checked once, before a new prompt is sent.

Ouch. just realized that this can be fine-tuned in the settings. Prob solved!


ConfigurationError: pipeline_name must be provided if stateful ingestion is enabled.

Describe the bug
I recently installed datahub (running v0.9.3). Did try to upgrade to 0.9.5 before i read it was broken, might have something to do with this error. Running the sync with snowflake causes this error (see logs below).
I'm running in K8s and installed using the helm charts.
Desktop (please complete the following information):

Version v0.9.3

RUN_INGEST - {'errors': [],
 'exec_id': '907f9947-0ba6-4acd-955d-2210c2bbec51',
 'infos': ['2023-01-04 13:55:26.996503 [exec_id=907f9947-0ba6-4acd-955d-2210c2bbec51] INFO: Starting execution for task with name=RUN_INGEST',
           '2023-01-04 13:55:33.118663 [exec_id=907f9947-0ba6-4acd-955d-2210c2bbec51] INFO: stdout=venv setup time = 0\n'
           'This version of datahub supports report-to functionality\n'
           'datahub  ingest run -c /tmp/datahub/ingest/907f9947-0ba6-4acd-955d-2210c2bbec51/recipe.yml --report-to '
           '/tmp/datahub/ingest/907f9947-0ba6-4acd-955d-2210c2bbec51/ingestion_report.json\n'
           '[2023-01-04 13:55:29,050] INFO     {datahub.cli.ingest_cli:165} - DataHub CLI version: 0.9.3\n'
           '[2023-01-04 13:55:29,068] INFO     {datahub.ingestion.run.pipeline:174} - Sink configured successfully. DataHubRestEmitter: configured '
           'to talk to http://datahub-datahub-gms:8080\n'
           '[2023-01-04 13:55:30,892] INFO     {datahub.ingestion.source_config.sql.snowflake:231} - using authenticator type '
           "'DEFAULT_AUTHENTICATOR'\n"
           '[2023-01-04 13:55:31,150] ERROR    {datahub.entrypoints:207} - Command failed: Failed to configure source (snowflake): pipeline_name '
           'must be provided if stateful ingestion is enabled.\n'
           'Traceback (most recent call last):\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/datahub/ingestion/run/pipeline.py", line 193, in __init__\n'
           '    self.source: Source = source_class.create(\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/datahub/ingestion/source/snowflake/snowflake_v2.py", line '
           '247, in create\n'
           '    return cls(ctx, config)\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/datahub/ingestion/source/snowflake/snowflake_v2.py", line '
           '187, in __init__\n'
           '    super().__init__(config, ctx)\n'
           '  File '
           '"/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/datahub/ingestion/source/state/stateful_ingestion_base.py", line '
           '124, in __init__\n'
           '    self._initialize_checkpointing_state_provider()\n'
           '  File '
           '"/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/datahub/ingestion/source/state/stateful_ingestion_base.py", line '
           '149, in _initialize_checkpointing_state_provider\n'
           '    raise ConfigurationError(\n'
           'datahub.configuration.common.ConfigurationError: pipeline_name must be provided if stateful ingestion is enabled.\n'
           '\n'
           'The above exception was the direct cause of the following exception:\n'
           '\n'
           'Traceback (most recent call last):\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/datahub/entrypoints.py", line 165, in main\n'
           '    sys.exit(datahub(standalone_mode=False, **kwargs))\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/click/core.py", line 1130, in __call__\n'
           '    return self.main(*args, **kwargs)\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/click/core.py", line 1055, in main\n'
           '    rv = self.invoke(ctx)\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/click/core.py", line 1657, in invoke\n'
           '    return _process_result(sub_ctx.command.invoke(sub_ctx))\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/click/core.py", line 1657, in invoke\n'
           '    return _process_result(sub_ctx.command.invoke(sub_ctx))\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/click/core.py", line 1404, in invoke\n'
           '    return ctx.invoke(self.callback, **ctx.params)\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/click/core.py", line 760, in invoke\n'
           '    return __callback(*args, **kwargs)\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/click/decorators.py", line 26, in new_func\n'
           '    return f(get_current_context(), *args, **kwargs)\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/datahub/telemetry/telemetry.py", line 347, in wrapper\n'
           '    raise e\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/datahub/telemetry/telemetry.py", line 299, in wrapper\n'
           '    res = func(*args, **kwargs)\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/datahub/utilities/memory_leak_detector.py", line 95, in '
           'wrapper\n'
           '    return func(ctx, *args, **kwargs)\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/datahub/cli/ingest_cli.py", line 179, in run\n'
           '    pipeline = Pipeline.create(\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/datahub/ingestion/run/pipeline.py", line 313, in create\n'
           '    return cls(\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/datahub/ingestion/run/pipeline.py", line 199, in __init__\n'
           '    self._raise_initialization_error(\n'
           '  File "/tmp/datahub/ingest/venv-snowflake-0.9.3/lib/python3.10/site-packages/datahub/ingestion/run/pipeline.py", line 129, in '
           '_raise_initialization_error\n'
           '    raise PipelineInitError(f"{msg}: {e}") from e\n'
           'datahub.ingestion.run.pipeline.PipelineInitError: Failed to configure source (snowflake): pipeline_name must be provided if stateful '
           'ingestion is enabled.\n',
           "2023-01-04 13:55:33.118926 [exec_id=907f9947-0ba6-4acd-955d-2210c2bbec51] INFO: Failed to execute 'datahub ingest'",
           '2023-01-04 13:55:33.120275 [exec_id=907f9947-0ba6-4acd-955d-2210c2bbec51] INFO: Caught exception EXECUTING '
           'task_id=907f9947-0ba6-4acd-955d-2210c2bbec51, name=RUN_INGEST, stacktrace=Traceback (most recent call last):\n'
           '  File "/usr/local/lib/python3.10/site-packages/acryl/executor/execution/default_executor.py", line 123, in execute_task\n'
           '    task_event_loop.run_until_complete(task_future)\n'
           '  File "/usr/local/lib/python3.10/asyncio/base_events.py", line 646, in run_until_complete\n'
           '    return future.result()\n'
           '  File "/usr/local/lib/python3.10/site-packages/acryl/executor/execution/sub_process_ingestion_task.py", line 168, in execute\n'
           '    raise TaskError("Failed to execute \'datahub ingest\'")\n'
           "acryl.executor.execution.task.TaskError: Failed to execute 'datahub ingest'\n"]}
Execution finished with errors.

hey @ActivateConsulting ! are you running ingestion through the UI with managed ingestion?
I believe I know the issue causing this, and it is indeed fixed in the most recent version, but like you said there is another issue with the current version we are currently in the process of fixing and rolling out a new update.
In order to fix this in the short term, you can do one of two things in order to get your snowflake source running again.

change your recipe so that:

        stateful_ingestion:
            enabled: false

be either directly editing the YAML or checking "Enable Stateful Ingestion" off from the advanced section of your UI form.

add a pipeline_name to your recipe if you truly do want stateful ingestion to be enabled. This must be unique across all ingestion sources, though, and remain constant for this ingestion source in order to have stateful_ingestion running properly. The safest bet would be to simply turn it off for now.

I'll leave this issue open for a few days or until I hear back from you to ensure things are alright!

Hi Chris,
I did add the ingestion through the UI, and this happened after a day or so of running the environment. I did try running it without the stateful_ingestion enabled, with the same result. Unfortunately I'm unable to reproduce the error as I ended up re-intalling the whole solution and starting from scratch.

@ActivateConsulting gotcha. well I'm sorry for the difficulty but glad it's working as desired now! Since we pushed a fix for this some time ago and you aren't seeing the issue anymore, I think it's safe to close this issue.
If you see anything like this again please feel free to re-open and/or let us know.


Date pullfix

Great code!  Noticed that it was having trouble pulling data from before 2006 for a few stations of interest.  I added some functionality to fix data pulling from pre-2006.
Also - I'm fairly new to git so apologies if I messed up!

Interesting.  I re-purposed this code from an earlier implementation in MATLAB and I was able to pull all data from 1985 to 2014 for station NDBC 46042.  However, if you've found a case where this isn't the case let's take a look!
BTW, this is still definitely a work in progress (see project/issues).  I've got quite a few plans for this module and I'll update the PyPI version once I have the current ideas implemented.

Was taking a look at NDBC station DPIA1 and was having some trouble pulling the older data.  Seemed to work fine for that and some surrounding stations with the edits, but maybe mine was just a unique case.
Either way - super useful tool, thanks for sharing!

I appreciate the update but I changed the approach for the following reasons:

It is reasonable to assume there may be more URL patterns for other stations
It is undesirable to have to modify function code for each new pattern
Increasing the number of if..else operations increases code complexity

Therefore I changed the approach to use lists of URL patterns for both years and months.  The __checkurl__  function was revised and __buildurls__ function was added to facilitate correct URL determination and usage.  These modifications have been merged into master branch.
I'm glad you're finding this useful!  I have a number of improvements in mind to facilitate both exploratory data analysis as well as integration into an automated data workflow.  I hope to have most of the improvements committed to master by the end of the year.


Caching strategies for multiple anndata files

We use least recently used (LRU) strategy to cache anndata files. This is optimal when one anndata file is being fetched at a time. But when the indices of multiple anndata files that are being fetched known in advance there is room for optimization (as implemented in the materialize method of DistributedAnnDataCollection. One of the questions is when to call this method. It can be done when DistributedAnnDataCollection is indexed as currently implemented or at a later stage allowing DistributedAnnDataCollection to be lazily indexed multiple times.

Closed via #127


add hello world code to project

Step 4 of the interview process...

Second pull request... tests fixed.


Require entity-type in profile status get to avoid permafail

Summary
***Provide a brief overview of the changes and the issue being addressed.
This commit marks entity-type as required in
minder profile status get to avoide sending
ENTITY_UNSPECIFIED to the API (which always fails).
Change Type
Mark the type of change your PR introduces:

[x] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 53.077%. remained the same
when pulling a3b802ef269a836e78bd54e5577936b4c6cf62be on puerco:require-type
into 9877dba1b0602c4bb38f1ed407b0e67740a20663 on stacklok:main.


[RESOLVED] No SNMP data collection

Problem started at 01:43:59 on 2023.05.16
Problem name: No SNMP data collection
Host: S100-Santiago
Severity: Warning
Operational data: Current state: not available (0)
Original problem ID: 20681
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23525&eventid=20681

Problem has been resolved in 1m 0s at 01:44:59 on 2023.05.16
Problem name: No SNMP data collection
Host: S100-Santiago
Severity: Warning
Original problem ID: 20681


Fix structs inputs of functions

Current code generates wrong signature of function for functions accepting structs.
example:
IBar function set(T memory t) external; gives: set(tuple) instead of set((uint256,address))
I replaced signature/name calculation with function from ethers.js
Also: added artifacts/ and cache/ to .gitignore

@cytadela8 Awesome !


link containing  <> brackets breaks the library

parsing this will throw an error:

Hey, thanks for reporting. Can you be more specific about what the error is?

if a notion page has a link surrounded with angular brackets, the library will throw error. screenshot of Notion page:

I tested it locally and the generated markdown looks like so:

however, it won't render the link which is a problem with the markdown parser and now this library.


API Server Erroring

@coderanger 👋 I know it's been a while, but I've been playing with this project this afternoon.
I've got it most of the way there, but the API server seems to always return 400 which is causing the waiter containers to crash.
Any ideas? I'm reading through the code trying to figure it out - using the main image as well.
EDIT
Ok, its obviously because my curl code has no payload.
However, the waiter container is having
"2024/08/13 23:17:14 Error while polling: Post "http://migrations-operator-api-service.migrations-operator-system.svc:5000/api/ready": dial tcp <IP_ADDRESS>:5000: connect: connection refused"

I presume this is related to the fact that we also inject the istio proxy in our pods, but am not sure

If it's redirecting the network traffic through Istio then it does sound likely that is causing the issue. I know Istio has it's own internal ACL layer for authorizing inter-service traffic. I don't use it myself so I can't really help more than pointing you to their docs.

I have partially resolved my problems. Istio injects an initContainer itself which does iptables stuff.
I was able to use an annotation to allow waiter traffic on port 5000.
There are still some problems with the migration Pod itself, which seems to never Terminate and instead stay in a NotReady state after completing - likely also related to the istio proxy sidecar but I'm not 100% sure how.
This can also be mitigated in the short-term by adding an annotation to disable injection.

Daemon sidecars on Jobs is a long-standing problem, https://github.com/kubernetes/enhancements/issues/753 should address at least this kind of simple case with it but is still in beta AFAIK.

thanks! yep, been waiting for that spec to be finalized for many years


feat: refactor SQLConnector connection handling

This is a draft PR that proposes a solution to issue #1393.
Broad overview:

We are no longer storing an open connection on this object. Instead, we store an engine. Wherever we were previously making a new engine, we instead access the connector's existing engine. When a connection is needed, we open one at that time.
When connections are opened, they're opened using self._connect() as a context manager. This rule is put into place where it's possible to do so without making a breaking change.
Several methods are prepared for deprecation, but their function is preserved.

(Breaking change) _connection is removed altogether. The tap no longer accepts that argument on init, and we don't store a single connection at all

This is a breaking change for any subclasses that were using this property
This is a breaking change for any subclasses that counted on a single connection being reused throughout the lifetime of the instance
This is a breaking change for any cases where an external method call counted on a single connection being returned each time that create_sqlalchemy_connection was called (or for anyone who didn't respect the protectedness of _connection).

(Slightly breaking) connection still exists, it just calls create_sqlalchemy_connection. So this is a slightly breaking change—- it returns a new connection instead of the _connection that prviously had been preserved on the instance.
create_sqlalchemy_engine now just returns self._engine, since we have an engine on this object anyway.
create_sqlalchemy_connection now opens a connection using self._engine and returns the open connection.

Detailed changes in functionality:

self._connect() is added. This is the correct way to open connections now—- you use it as a context manager, exactly as you would sqlalchemy.engine.connect()... in fact currently it just directly wraps that.
We now create and cache a single engine for this instance. It's accessed as a protected property under _engine.
create_sqlalchemy_connection calls self._engine now instead of self.create_sqlalchemy_engine, since the connector now has an engine. This and the next item help extricate and disconnect these soon-to-be-deprecated methods.
create_sqlalchemy_engine now calls self._engine instead of making the engine itself.
Everywhere else in the SQLConnector that connection.execute was called now uses the correct/new _connect strategy.
Internal calls to create_sqlalchemy_engine now just call _engine (this is just a change to discover_catalog_entries).

:books: Documentation preview :books:: https://meltano-sdk--1394.org.readthedocs.build/en/1394/

I used an lru_cache decorator at first, but there doesn't seem to be any real way to handle typing for a lru_cache-wrapped function that returns something— see here. So instead I just used another property to cache it. Maybe not the most pythonic(?) but it works fine. Another option is to type: ignore that line. I haven't worked much with type checkers, so I'm not sure what the right move is here. Ofc open to suggestions if anyone has an opinion.

@qbatten this is great, thanks for all your work on connection handling 🙏 As we are pre-1.0 for the SDK, and especially since there are far fewer users of the SQLConnector at this point, I think the breaking changes are OK. However, it would be good to proactively reach out to the Taps/Targets we know about that would potentially be impacted (using Hub data), to make sure they are pinning their SDK version and are aware of the changes 🙂 I can help with that 👍

Test coverage! Woohoo.

@qbatten this is looking great 😄


Error returned by the controller is not the right one for the semantic

Signed-off-by: Juan Manuel Parrilla Madrid<EMAIL_ADDRESS>Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #HOSTEDCP-569
Checklist

[X] Subject and description added to both, commit and PR.
[X] Relevant issues have been referenced.
[ ] This change includes docs.
[X] This change includes unit tests.

/hold


Calendar for no graze window for plant community action includes a year but the field is only mm and dd

REQUIREMENTS

[ ] calendar label is month only ex. "October"
[ ] only month and day are captured in field

Related to #137

@calebissharp Noticed today that the no-graze window field is showing year, month, day and time.  Just want month and day for these two fields.  See requirements above.  Looked good before (or I didn’t notice it) so not sure what happened.


Bug: can not publish message

Describe the bug

How to reproduce
Include source code:
import json

from faststream import FastStream
from faststream.rabbit import ExchangeType, RabbitBroker, RabbitExchange, RabbitQueue
import asyncio

broker = RabbitBroker(f"amqp://{config.MQ_USER}:{config.MQ_PWD}@{config.MQ_HOST}:{config.MQ_PORT}/")
app = FastStream(broker)

exch = RabbitExchange(config.MQ_INSERT_ES_EXCHANGE,  type=ExchangeType.TOPIC)

queue_1 = RabbitQueue("algo.test-queue-2",  routing_key=config.MQ_INSERT_ES_ROUTING_KEY,durable=True)

if __name__ == '__main__':
    msg ={}
    msg["id"]=1
    asyncio.run(broker.publish(message=json.dumps(msg),exchange=exch,routing_key=config.MQ_INSERT_ES_ROUTING_KEY))
    print(111)
...

check this: https://faststream.airt.ai/latest/rabbit/publishing/#rabbit-publishing

I close it as a copy of #1972


feat: add child state reasons via v0 grpc

Signed-off-by: Tiago Castro<EMAIL_ADDRESS>

bors merge


Missing pagination for some bulk queries

Summary
While looking at some pagination issues in hermes queries (https://github.com/informalsystems/ibc-rs/issues/811)  found that there is no pagination field for the following queries:
QueryClientConnectionsRequest, QueryUnreceivedPacketsRequest , QueryUnreceivedAcksRequest.

Problem Definition

Proposal

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

Actually probably not required for QueryUnreceivedPacketsRequest  and QueryUnreceivedAcksRequest as we query for specific sequences.

@ancazamfir the lack of pagination in QueryClientConnectionsRequest, is still an issue for hermes?
I ask because I was looking a bit into it and I think that it will not be so straightforward to add pagination to it. I will try to explain...
I think the regular way of adding pagination to the gRPC endpoint using SDK's machinery would be something like this:
clientConnectionPaths := []string{}
store := prefix.NewStore(ctx.KVStore(q.storeKey), []byte(host.ClientConnectionsKey(req.ClientId)))
pageRes, err := query.Paginate(store, req.Pagination, func(key, value []byte) error {
	var result types.ClientPaths
	if err := q.cdc.Unmarshal(value, &result); err != nil {
		return err
	}

	clientConnectionPaths = append(clientConnectionPaths, result.Paths...)
	return nil
})
if err != nil {
	return nil, err
}

The problem is that the connection paths for a given client ID are all stored as a []string in the key clients/<client-id>/connections, so when doing the query.Paginate there's always just one page returned that contains all the connection paths for that client, since there's nothing to paginate over on the provided prefix ([]byte(host.ClientConnectionsKey(req.ClientId))).
Alternatively we could get all the connection paths as it is done right now and do the pagination ourselves, but before building something so specific for this query, I would like to know if there's really a strong need for pagination in this query.
cc @colin-axner to double check my reasoning here when he has some time or provide alternatives ideas.

It's been a long time and can't recall the details. I was doing local testing with high scale numbers but can't recall if anything was breaking and how it was breaking. We can probably do more tests.
Are you asking in order to close the issue?

Are you asking in order to close the issue?

Yes, that's basically the reason. I can understand this could be a problem in stress-testing scenarios if you add lots of connections on top of the client, but if this is not an issue that would likely be hit in real scenarios, then I would consider closing the issue. It can be open anyway again if needed!

Closing for now since there's no action to take at the moment. We will reopen if needed.


Prisma client: mutation ignored when callback not provided

I am implementing an mutation resolver.
I make and await mutations that are essential to fulfilling this query.
Then, I want to initiate an additional mutation, immediately return and complete this mutation in background, while client already got his response.
Something along lines:
async mutationResolver() {
  const updated = await prisma.updateEntityX({ ... });
  prisma.updateEntityY({ ... }); // < no await or any "then" callback
  return updated;
}

Then, I inspect prisma database and it seems like call to prisma.updateEntityY() is totally ignored, until I provide some callback, e.g.:
prisma.updateEntityY({ ... }).then(() => {}); // just an empty handler

Is this really the case? Seems weird.
Thanks.

Hi @dodas,
We internally batch thing up using data loader so this is expected behavior in my opinion. Let me know any solid use case of your setup if there is any.

I already described my exact use case :)
I need to perform a mutation that's result is not needed for resolver's return value.
I just want to return value from resolver and finish this additional mutation in background.

Related: https://github.com/prisma/prisma/issues/4179
cc @divyenduz

Thanks for raising this issue @dodas
Currently, you have to consume the Prisma client promise for it to actually fire, it is designed to be lazy right now. But in your case, you are calling .then and the mutation still does not fire, which sounds like a bug.
This was a related discussion with some workarounds but it is closed by the author, so, I would keep this issue open to track this: https://github.com/prisma/prisma/issues/3827


Quel nom pour l'app EIMIS ?

This issue to track the dicussions about our app name and also prod domain and subdomains

Some suggestions

Tchap Santé as people already call it
Chap would make me very happy 🤭
Santement Santé + Element 🤷🏻

As we will suggest users to use android and apple Element app our deployed web app should also be a standard, un-customized Element.

About Matrix server name, like it's suggested here it would be wise to keep the domain as simple as possible as it will be part of the matrix ID. So just eimis.beta.gouv.fr ?


Launch rockstar

Traceback (most recent call last):
File "/usr/local/bin/rockstar", line 9, in
load_entry_point('rockstar==0.5', 'console_scripts', 'rockstar')()
File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/pkg_resources.py", line 357, in load_entry_point
return get_distribution(dist).load_entry_point(group, name)
File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/pkg_resources.py", line 2394, in load_entry_point
return ep.load()
File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/pkg_resources.py", line 2108, in load
entry = import(self.module_name, globals(),globals(), ['name'])
File "/Library/Python/2.7/site-packages/rockstar/init.py", line 1, in
from .RockStar import RockStar
File "/Library/Python/2.7/site-packages/rockstar/RockStar.py", line 11, in
import git
File "/Library/Python/2.7/site-packages/git/init.py", line 6, in
from repository import Repository, InvalidRepositoryError
File "/Library/Python/2.7/site-packages/git/repository.py", line 11, in
from objects import Commit
File "/Library/Python/2.7/site-packages/git/objects/init.py", line 10, in
from .submodule import util as smutil
File "/Library/Python/2.7/site-packages/git/objects/submodule/util.py", line 3, in
from git.config import GitConfigParser
ImportError: cannot import name GitConfigParser
Hello,
I've just install rockstar, but when I try to launch rockstar, i've got an error about a GitConfigParser ?
Any ideas ?
Max.

Did you try creating a repository using git init?
It seems like that's the issue.

Yes.
» rockstar --days=300
And after that I've got this error... :(

Ok i've just reinstall gitpython with pip and reinstall it... fixed.
Thanks


Can't create newline in Hero Card text [MS Teams]?

I want to have multiple lines in my Hero Card text and I followed this post by doing the following in my code
        var card = new HeroCard {
            Title = buildName + " " + buildType + " on " + buildDate,
            Subtitle = "Build Evaluation: " + buildEvalResult,
            Text = "Hello \n\n World \n\r Hello! \r\r New Line?",
            Images = new List<CardImage> ()
            {
                new CardImage(url: await getHeroCardPic("HAPPY"))
            },
            Buttons = new List<CardAction> ()
            {
                // card Actions...
             }
        };

When I try this out in MS Teams I see a card with no line breaks in the text. Am I doing this wrong, or does MS Teams not support this yet? Thanks for the help.

I got this working using  like so:
Text = "Hello \nWorld \nHello! \r\r New Line? <br> Word Here \n Not working"


Edit template editors are weird sizes

When I click to edit an existing template, the phase and questions all open with a weird assortment of editors and contents contain html markup (screenshot). Desired behavior is to open the main editing page with existing question content as read only (related issue #1502 ). Then if I click to edit a question, I should see a normal layout w/editors of same size and no html.

should be fixed by changes made in #1486


Create an action that fails if the user has committed package-lock

Users are committing package-lock and pnpm-lock
We're restricting the repo to yarn, so any PR that adds either of these should be rejected
Let's add a workflow that checks this

I am adding the workflow


How do you actually move or rename an area or iteration?

The example bodies contain only an ID or name field, which does not provide the obviously necessary parameter of where to move that item to.

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

ID: ccea58b5-0d3d-4964-da9c-f122fce3d2e9
Version Independent ID: 7fbb19c2-dc05-ed12-b4e6-4b2b3c849988
Content: Classification Nodes (Azure DevOps Work Item Tracking)
Content Source: docs-ref-autogen/5.0/wit/Classification%20Nodes.yml
Product: devops
Technology: devops-ref
GitHub Login: @wnjenkin
Microsoft Alias: whjenkin

I'm having this problem as well.  I've tried everything I can think of to move an existing node under another node, but none of the obvious options work.  My node structure is like this:

Kitchen

Program
Project

And I want to move Project under Program.

Kitchen

Program

Project

I'm using the Python SDK.  Here are the patterns and code I am using:
`
import azure.devops.v5_1.work_item_tracking.models as Models
personal_access_token = '*********************************'
organization_name = 'JustinCodes'
project_name = 'Kitchen'
connection = connect(organzation_name=organization_name,
personal_access_token=personal_access_token)
client = connection.clients.get_work_item_tracking_client()
patterns = [
'\Kitchen\Area\Program\Project',
'\Kitchen\Program\Area'
'\Kitchen\Area\Program',
'\Kitchen\Program\Project'
'\Kitchen\Program',
'\Program\Project',
'\Area\Program',
'\Program',
'Program',
'/Kitchen/Area/Program/Project',
'/Kitchen/Area/Program',
'/Kitchen/Program/Project'
'/Kitchen/Program',
'/Program/Project',
'/Area/Program',
'/Program',
]
for pattern in patterns:
posted_node = Models.WorkItemClassificationNode(
    id=89,
    path=pattern
)

output = client.create_or_update_classification_node(
    posted_node, project_name, 'areas')

`
None of these work.  They always move the node back to root under Kitchen.


Multiple Layouts in Keyboard Manager

Summary of the new feature/enhancement
Add Support for multiple saved layouts in Keyboard Manager. E. g. I have a mapping for my Apple Keyboard where my Alt and Meta keys are swapped and I have a different mapping for my normal keyboard where I for example have Media Keys on F13 and up.
Proposed technical implementation details (optional)
I would like to have like a list of different mappings. If I create a new mapping, I can choose to save it. The option to rename mappings would be great to. If it is possible to automatically swap to ta specified mapping if I connect a special keyboard, I would find that great to. But that is optional if it is to hard / time consuming to implement.
Conclusion

Add option to save mappings
Saved mappings are available in list
(Optional) Automatically switch to mapping XY if special Keyboard is plugged in.

I'm sorry for my bad English, I'm not a native Speaker.

hi we're tracking profiles here: #1881 and exporting/saving presets here: #4452. In regards to your 3rd issue, that's not currently in our v1 plan to switch keyboards, but do please submit separate issue.
closing this as a duplicate, please follow them for updates.


Update auth indicator on home page

Looking to update the indicator on the home page after discussion with @Tobbe and @dthyresson over on #41
Here's an example of an updated version to get the ball rolling on brainstorming

In that example, "Authenticated" or "Not Authenticated" would be hidden for screen readers only

It's still not clear to me (as an imaginary first time visitor to the site) what "green" means. Installed? Enabled? Selected?
What about going back to the (pulsating) dot, but add a visible text next to also?
Or place some kind of legend above or below all the buttons?
~Or a hover tooltip?~ Doesn't work very well on mobile...
Or maybe a locked/unlocked symbol would be more clear? Instead of the dot or checkmark I mean.
Just throwing out ideas :)

What about something like this? A bit more subtle than the badge but still gets the messaged across.

@Tobbe what do you think? I like it as well.

I also have hidden text appended to the message where the screen reader would read "Signed In to Auth0" or "Signed Out of Auth0"

Very nice @morganmspencer! Maybe just remove it from the providers that aren't implemented yet. Or replace with some kind of message saying they're not implemented yet


Eng documentation

We need to write it. The wiki is a good place.

Can I close this since we have started doing this? we can create individual tasks for specific documentation (document this api, document data workflow, etc.)


Runtime patch

Arrays (aka. "vecof" / "vectorof") may have been implemented incorrectly in runtime.c.
The identification bit should be at position 62, not 63.
validate_vector should not exit with an error when receiving an array pointer.
Even if print_vector is never called, the case for arrays was still missing.
The patches are copied from the cases of tuples (aka. "vec").
No guarantee for correctness.

Thanks for the patch! There are compile time warnings due to using format "%lld" on int64_t.
This should be "%" PRId64 IMHO.


For Later - Cleaning up CSS and formatting for pending outbox page

The formatting of the navbar and buttons are off currently

This has been accomplished with Less and CSS edits along with removing the bootstrap CDN (and making the appropriate changes to not need it anymore)


Consider moving StartAsync() to Main()

With C# 7.1, Main() can now be asynchronous, so StartAsync() may no longer be necessary.

Completed in https://github.com/the-mighty-mo/BotBase/commit/dfb2f859198dfa69c4410c2d8f2b4b90bd5655ca


How about color change while incrementing?

Its a feature request - Could you possibly add something like show specific color for increment and different color for decrement?

Sounds good! Do you want to go ahead and make a PR?


[Snyk] Upgrade react-router from 5.3.3 to 5.3.4

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade react-router from 5.3.3 to 5.3.4.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released a year ago, on 2022-10-02.

The recommended version fixes:

Severity
Issue
PriorityScore (*)
Exploit Maturity

Incomplete List of Disallowed Inputs SNYK-JS-BABELTRAVERSE-5962462
572/1000  Why? Proof of Concept exploit, CVSS 9.3
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.

Release notes

Package name: react-router

5.3.4 - 2022-10-02

5.3.3 - 2022-05-18

    from react-router GitHub release notes

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.536%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
637

Relevant Lines:
640

💛  - Coveralls


maint: Adds optional HTTP headers example to values.yaml

Which problem is this PR solving?
Updates the example helm-chart to show how to configure additional HTTP headers to be extracted.

Implemented in https://github.com/honeycombio/honeycomb-network-agent/pull/277

Short description of the changes

Update example values.yaml with commented out config that shows how to set additional HTTP headers to extract

How to verify that this has the expected result
The values.yaml show how to set additional headers to extract.

Added blocked label until the agent PR has merged.


AssertionError: The length of palette should not be less than num_classes.

I want to display bounding boxes with test.py but facing this issue "The length of the palette should not be less than num_classes."
Below are the details of the dataset and installation environment.
My command line: CUDA_VISIBLE_DEVICES=0 python tools/test.py configs/yolact/yolact_r50_1xb8-55e_coco.py work_dirs/yolact_r50_1xb8-55e_coco/epoch_55.pth --show-dir work_dirs
I have setup mmdetection with this guide https://mmdetection.readthedocs.io/en/latest/get_started.html
I think I have the latest version of mmdetection with mmengine, we can see the installation
pip install -U openmim
mim install mmengine
mim install "mmcv>=2.0.0"
I am using a dataset with 121 classes, I followed the guidelines given at #8003  , however, I did not find any functional code to replace PALETTE = None.
I also have an old version of mmdetection on another PC and the guide at #8003  helped me successfully to replace PALETTE = None.
Now I have a bigger GPU and I installed a newer version of mmdetection,

Is there any possibility to solve this problem.
Thank you

Hello,
I just encountered this problem, but I solved it.
Please check your config file yolact_r50_1xb8-55e_coco.py
You mentioned that your dataset has 121 classes, this may be due to the fact that the number of elements of palettes in the metainfo you specified in config is less than 121.
Let me give an example to explain my explanation above:
If I am using a dataset with 3 classes, I will redefine metainfo  in my config file as follows:
metainfo = dict(
    classes=class_name,
    palette=[**(119, 20, 60), (20, 80, 60), (19, 202, 60)**]  # This represents the color when drawing, just set it freely
)

It is obvious that there are 3 elements in the list of palettes, which are consistent with the number of classes in the dataset I am using.
I hope I can help you.

You need to expand the list of colors

您需要扩展颜色列表
修改哪个位置可以扩展颜色列表


Minor fixes and added data-validation support

@jhoneill for the behavior changes, seems like they are breaking, the module is at v5.4.4. EPPlus is not fully ported, so bump it to v5.5.0?

They shouldn't break anything which is live - the one thing which I found was different - and impacted the tests - was if I had one column and row 1 was null the spreadsheet was only considered to start at row 2.
The rule used to go

If it's a date
If it's a timespan
If it's a value
If it starts with =
5  if it look likes a URI
6  otherwise can we make it a number ? No ? Insert a string

Then between 3 & 4 I put is it of type URI, and  is it not of type string  - that tried to call the toString method if it was null. Which caused and error. So before 1 I put "is it null, if so do nothing" .
And I've changed 6
Does it have digits ?  does no-number-conversion apply ? OK now try to make it a number.

I've put in a fix for #555 as well :-)


pull request for e2e test at 01/12/2017 07:45:11

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


ui: fix june track on sidebar

What does this pull request do? Explain your changes. (required)

Specific updates (required)

How did you test each of these updates (required)

Does this pull request close any open issues?

Screenshots (optional)

Checklist

[ ] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

@suhailkakar is this ready for review?


Issue running Cellpose

This is the error message I get when trying to run cellpose from the base environment. Is it due to an installation error?

Yes, it is. Try installing in a separate environment, and if you're on Mac, take a look at the special Mac instructions in the readme.

closing this issue please comment if you are still having issues with this


Put the expanded details of feature, diseases, etc in a panel to make things clearer

When expanding a feature or a disease or interpretation, or any other phenopacket sub element,  the details are not clearly separated from the next item in the list of feature, disease, interpretation etc. We could put them in a Panel like the Vital Status for instance.

Working as described


Minor changes to shelley.cddl for ease of use

In order for ease of code generation these changes, which don't impact
the CBOR representation, were made. These certificate/multisig changes
are the same ones that were done to relay before, as otherwise there is
not enough information for code generation tooling in each group choice
variant to get proper names from, as comments are discarded.
amount changing to coin was done to improve code generated and to make
it more clear for readers.

CI is failing on this transient issue - https://github.com/input-output-hk/cardano-ledger-specs/issues/1638 - if you -re-trigger the tests (with an empty commit, say) you should be ok

somehow I missed @uroboros 's comments, making mine redundant. :) I also notice that the commit isn't signed, so I'll go ahead and sign at and force push, two birds.

oops, it's from a fork, I can't sign it

I didn't have permissions to push a branch directly to this repo so I used a fork. I re-did it with --signoff, will it work now?

I didn't have permissions to push a branch directly to this repo so I used a fork. I re-did it with --signoff, will it work now?

it's still not show up on github as verified. you could try git commit --amend -S and then force pushing. but I am also happy to put up a PR myself and grab #1648 while I'm at it

@rooooooooob you can close this PR if you are happy with #1660

Closing, I had tried --amend -s before. I think it was using a cached build that was failing which wasn't signed off since the code was the same and the failure pointed to the first one it failed on. But I'll just close this since that other PR covers the changes.


Add tests and peek function for emission measure

Related to comments in #110, this PR adds a method for quickly plotting the emission measure and also adds a few tests.

@jwreep with this PR, you should now be able to do something like,
import pydrad.parse
s = pydrad.parse.Strand('/path/to/my/hydrad')
s[10].peek_emission_measure()

to preview the column emission measure at the 10th timestep.


ENH: Improve robustness of Python code running in the console

When a complete string of command is executed (either copy-pasting or running from file) then we no longer split the string to lines and execute line by line because that often leads to errors. For example, parsing of this Python code line by line would fail due to the empty line in the function definition:
def test1():
    print("one")

    print("two")
    print("three")

def test2():
    print("ok")

Also make the console remember the path of the last run Python file to make it easier to run it again.
Before this fix (failure while trying to specify test1() function):

After the fix (everything works as expected):

Thanks! What you have described is something I’ve dealt with over the years when copy/pasting some functions.

Thanks for working on this 🙏
It is a great improvement that will greatly improve productivity.
Since I am reviewing on my phone, I couldn't test. That said, I spotted few typos in the comments (more more, muiltiline)

The improvements to the  runfile function make sense, especially the saving and restoring of the current command.
If systematically displaying the entire file in the console turns out to be a problem, it could be revisited to only display  the first N lines of (or first N and last N).

Thanks for the feedback. I've fixed the typos and changed the behavior when running a file to print just a message (Execute: file/to/script.py) instead of the file content.

I've addressed the review comments. I'll merge this now to give a chance for everyone to try this.

Thanks @jcfr, I've fixed one more more typo but apparently there were two occurrences.


Import the initial arcade bits

Pull the eng/common stuff from https://github.com/dotnet/arcade
Set the project to build and version info
Add .dotnet to .gitignore
Use dotnet 5.0.102 to build

This is enough to get eng/common/build.sh --restore --build to build the project.

@akoeplinger does this seem like a good start?

Ok, using Versions props now. I'll merge this and then start on darc once I read the docs.


Add spring-cloud-function-stream-source

As a developer, I'd like to add spring-cloud-function-stream-source as an OOTB source.
Acceptance:

Decide whether to bundle this app with all the supported binders vs. continue to generate apps for each binder
Reference guide updated with supported properties
App coordinate included in application metadata and aggregated http-endpoint artifacts
Test coverage included

This should probably be called spring-cloud-function-stream-source (consistent with spring-cloud-function-stream-processor where the last term is aligned with the role it plays within SCDF).

Thanks! Makes sense. I updated the title and description.

See https://github.com/spring-cloud-stream-app-starters/function/pull/1


Don't Bury "CSV Download" Function in a Deep Location

I just explored new InfluxDB 2.0 Dashboard, and found that the download CSV function is removed from dashboard, though I can still found it in its explore menu.
Please keep download CSV in the front dashboard as shown in the chronograf, bury that function is Ok for developer, but it will be a disaster for my end-user.
This is super important for my end-user.
InfluxDB 2.0

Chronograf

@xufangda thanks for the issue. It wasn't our intention to bury it, i think it just got missed in an update of the cell menu. We will take a look at adding it back.
I'm not sure why it's behind a featureFlag at the moment but i'll track that down as well.
https://github.com/influxdata/influxdb/blob/master/ui/src/shared/components/cells/CellContext.tsx#L123

Hi Russorat,
Thanks for reply, but it seems no follow up till now. Could you please raise the priority of this issue. Since you have already made this feature, it is relatively simple to resolve this issue. It is more like kind/bug, not a kind/feature-request. Because you don't spend much effort to create this feature, it seems you can simply remove the  tag and you can resolve this problem.
From our team side, we wish not to create and maintain a new fork. However, this feature is profoundly important for our end-user.
Many thanks.

@xufangda thanks! we will work this into an upcoming release. If you have a fix in a fork, please open a PR and we will review. That's the fastest way to get this fixed.


don't show icon in swift2

i have v0.6 and import FontAwesome_swift or import FontAwesome
When i use this lib for show icon:
self.title = String.fontAwesomeIconWithName(.Android)
i have the next result:

Any idea?

Did you set the FontAwesome font for the navigation bar title?

yes!! i need set FontAwesome Font ;)
Thanks


Vulnerability roundup 85: nghttp2-1.40.0: 1 advisory [7.5]

search, files

[ ] CVE-2020-11080 CVSSv3=7.5 (nixos-20.03, nixos-unstable)

Scanned versions: nixos-20.03: a84b797b28e; nixos-unstable: 22c98819ccd. May contain false positives.

20.03 is frozen and EOL.
master at 1.41.0
Closing.


include rows when outer join key is undefined

Includes rows when outer join key is undefined. This changes the results of outer joins.
Left, right and full outer joins give the desired results now.
Full outer joins were originally multiplying rows with undefined keys, but are now changed to be summing them instead.

[ch1965]

verified [ch875] : result looks ok now


Invalid Credentials

I have an Cornerstone user with full admin privileges, however when I test the connection I'm still getting an Invalid Credentials error. Are there specific security permissions this user needs to have? (Even though the user I'm using is System Admin and should already have all permissions.)

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

ID: 478935d3-ee72-f272-2c7f-c2d687a12cc9
Version Independent ID: 1401622a-3bfd-a4fc-5dd4-63bcec18f52f
Content: Tutorial: Configure Cornerstone OnDemand for automatic user provisioning with Azure Active Directory
Content Source: articles/active-directory/active-directory-saas-cornerstone-ondemand-provisioning-tutorial.md
Service: active-directory
GitHub Login: @zhchia
Microsoft Alias: v-ant

@russ4stall Thanks for your feedback! We will investigate and update as appropriate.

@russ4stall Thanks for reporting! I have assigned the issue to the author to investigate and update as appropriate.

Hi @russ4stall , could you email us at<EMAIL_ADDRESS>I'd like to dive in deeper into this issue with you and would need some information from you that I am not comfortable sharing in a public forum.

@MohitGargMSFT - do you happen to know if this issue has been resolved? I just tried this as well with our Cornerstone portal but received the same error message. Thanks.

Hi @tdobw: This issue has been fixed a few months ago. Could you check and see if:

In the Admin Username field, try entering the username in the “domain\user” format instead of “user”. The domain should be your Cornerstone tenant domain. The user account should have permissions to access the Cornerstone user management API.
In a web browser, try browsing to the Cornerstone User management API endpoint URL that you submitted in the Azure portal. You should get a page like the one below. If you don’t, the URL probably isn’t correct.

Thanks,
Zhen

@tdobw: Could you share with me the webservice URL of your Cornerstone OnDemand tenant? What is the value of the portal name that you are using? If you can send them to<EMAIL_ADDRESS>that would be great.

#please-close


[BUG]: some test cases are failing.....

Description
`Summary of all failing tests
FAIL backtracking/test/generateparentheses.test.ts
● Test suite failed to run
backtracking/test/generateparentheses.test.ts:1:37 - error TS2307: Cannot find module '../generateParentheses' or its corresponding type declarations.

1 import { generateParentheses } from "../generateParentheses";
                                      ~~~~~~~~~~~~~~~~~~~~~~~~

FAIL backtracking/test/all_combinations_of_size_k.test.ts
● Test suite failed to run
backtracking/test/all_combinations_of_size_k.test.ts:1:38 - error TS2307: Cannot find module '../all-combinations-of-size-k' or its corresponding type declarations.

1 import { generateCombinations } from "../all-combinations-of-size-k";
                                       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

FAIL maths/test/ugly_numbers.test.ts
● Ugly Numbers
expect(received).toEqual(expected) // deep equality

- Expected  - 11
+ Received  + 44

  Array [
-   1,
-   2,
-   3,
-   4,
-   5,
-   6,
-   8,
-   9,
-   10,
-   12,
-   15,
+   Object {
+     "done": false,
+     "value": 1,
+   },
+   Object {
+     "done": false,
+     "value": 2,
+   },
+   Object {
+     "done": false,
+     "value": 3,
+   },
+   Object {
+     "done": false,
+     "value": 4,
+   },
+   Object {
+     "done": false,
+     "value": 5,
+   },
+   Object {
+     "done": false,
+     "value": 6,
+   },
+   Object {
+     "done": false,
+     "value": 8,
+   },
+   Object {
+     "done": false,
+     "value": 9,
+   },
+   Object {
+     "done": false,
+     "value": 10,
+   },
+   Object {
+     "done": false,
+     "value": 12,
+   },
+   Object {
+     "done": false,
+     "value": 15,
+   },
  ]

  3 | test('Ugly Numbers', () => {
  4 |   const uglyNumbers = UglyNumbers();
> 5 |   expect(Array(11).fill(undefined).map(() => uglyNumbers.next())).toEqual([1, 2, 3, 4, 5, 6, 8, 9, 10, 12, 15]);
    |                                                                   ^
  6 | })
  7 |

  at Object.<anonymous> (maths/test/ugly_numbers.test.ts:5:67)

Test Suites: 3 failed, 77 passed, 80 total
Tests:       1 failed, 691 passed, 692 total
Snapshots:   0 total
Time:        23.258 s
Ran all test suites.
Error: Process completed with exit code 1.`
Expected Behavior
All test cases should pass and all the build for each and every PRs should run successfully !
Actual Behavior
But in this case test cases are faling and build is failing due this.
need to fix it.
Steps to reproduce (if applicable)
No response
Specifications
No response

Hi @raklaptudirm @appgurueu please make it high priority.
this need to be fix......
Thank You !


[Bug]:  Incorrect results with ONNX and OpenVINO after training Patchcore with default settings – all predictions are abnormal

Describe the bug
I trained the Patchcore model using default settings.
In the PyTorch inference, the model can correctly distinguish between normal and abnormal samples.
However, after exporting the model to ONNX and OpenVINO, all inference results are classified as abnormal, meaning the model fails to correctly predict normal samples.
Dataset
Folder
Model
PatchCore
Steps to reproduce the behavior
Train Patchcore:
datamodule = Folder(
    name="crop1",
    root=r"E:\crop1",
    normal_dir="normal",
    abnormal_dir="abnormal",
    mask_dir="mask",
    task=TaskType.SEGMENTATION,
    num_workers=0,
    image_size=[640,640],
    val_split_ratio=0.5,
    train_batch_size=1,
    eval_batch_size=1,
)

# Setup the datamodule
datamodule.setup()

model = Patchcore()
engine = Engine(
    task=TaskType.SEGMENTATION,
    )
engine.train(datamodule=datamodule, model=model)

Export
engine.export(
    model= model,
    export_type=ExportType.OPENVINO,
    export_root=r'E:\results\Patchcore\crop1',
    input_size=[640,640],
    ckpt_path=r'E:\crop1\latest\weights\lightning\model.ckpt',
    )

I received the following warning during the export process:

OS information
OS information:

OS: Windows 11
Python version: 3.10.11
Anomalib version: 1.1.0
PyTorch version: 2.2.2cu118

Expected behavior
Additionally, when I commented out the center crop operation in the Patchcore model, the issue still persisted. I was still unable to get correct results from ONNX or OpenVINO.
Screenshots
No response
Pip/GitHub
pip
What version/branch did you use?
No response
Configuration YAML
N/A

Logs
N/A

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Which inference method do you use for Torch? TorchInferencer or Enginer.predict?
It could possibly be a normalization issue? @ashwinvaidya17, any thoughts on this?

Thanks for sharing. @djdameln adding you here as well since you are working on OpenVINO inference refactor.

@watertianyi, you don't need to use OpenVINO inference if you have NVIDIA GPU. You could just use Lightning or TorchInferencer, which should work fine.
here, you could find more information about OpenVINO GPU support
https://docs.openvino.ai/2024/notebooks/gpu-device-with-output.html

@samet-akcay
The above problem has been solved. The main reason is that input_size cannot be set when converting OpenVINO because the error message is dynamic size. Is this correct? I would like to inquire whether OpenVINO supports NVIDIA GPU. The second question is if I want to improve the patchcore inference speed. What should be done? Because I have tried TorchInferencer and batchsize batch testing, the inference time is the same, and there is no change.

engine.export(
model= model,
export_type=ExportType.OPENVINO,
export_root=r'E:\results\Patchcore\crop1',
input_size=[640,640],
ckpt_path=r'E:\crop1\latest\weights\lightning\model.ckpt',
) When you define this, where is ExportType class:? in which packet it define ?


Crawler stopped working

https://github.com/7onn/politicians-contacts/actions/runs/4849366037/jobs/8641284966#step:6:12

Hopefully fixed by: https://github.com/7onn/politicians-contacts/commit/e7070af84ceb02e60232a15221708161c95dda67#diff-db382220333bd3822b782c823612a6f79d998f3365e594b534e3641aed82d4b7R39


Qobj header from a job doesn't return the info passed to execute function

Information

Qiskit Terra version: 0.18.3
Python version: 3.8.10
Operating system: Linux (IBM Quantum Lab)

What is the current behavior?
Qobj header from a job submitted to cloud simulator / local Aer simulator / real quantum system doesn't return the same info passed to qobj_header argument in the execute function, even though the documentation states the following:

qobj_header (QobjHeader or dict) – User input that will be inserted in Qobj header, and will also be copied to the corresponding qiskit.result.Result header. Headers do not affect the run.

What could be the issues?
Steps to reproduce the problem
from qiskit import IBMQ, QuantumCircuit, execute
from qiskit.circuit.random import random_circuit

provider = IBMQ.load_account()
backend = provider.get_backend('ibmq_qasm_simulator')
qc = random_circuit(3, 3)

## https://qiskit.org/documentation/apidoc/execute.html

test_job = execute(
    qc,
    qobj_header={
        'test_header': 'test'
    },
    backend=backend
)

test_job.result().header.to_dict()

returns
{'backend_name': 'ibmq_qasm_simulator', 'backend_version': '0.1.547'}

Same behaviour for Aer QasmSimulator.
backend = QasmSimulator()
backend = provider.get_backend('ibmq_belem')

What is the expected behavior?
The code above should return info passed to the execute function.
{'backend_name': 'ibmq_qasm_simulator', 'backend_version': '0.1.547', 'test_header': 'test'}

Suggested solutions

@mtreinish Thanks for the reply! That's really helpful. We will use the circuit metadata.


Move stubs on server/index.js to proper routers and do other several changes

Add more fields on .env

_VERSION
A version of server-side API.
PORT
Port to listen to requests.
NODE_ENV
The current state of the project. Currently, it is set to dev. In this state, all the authentication will not work and worked as production mode, this value must be changed as production. If any programmer needs to test any method or function which must be worked only with authentication (or not authenticated state) in production mode but is required to be worked on any state in development, then the following code snippet can be used (I used this property on server/auth.js file):

require('dotenv').config();
// codes ...

if (process.env.NODE_ENV == 'dev') {
  // something to do on development mode such as `next()` without authentication
} else {
  // something to do on production mode
}

For development, nodemon is added to the project module as devDependencies. This module restarts the executing server when there is any change in the project.

Move stubs on server/index.js to proper routers
For testing, stubs were previously written in server/index.js. However, since there are routers for the stubs, these must be removed to the right place.
--- Moved endpoints ---

GET /api/v1/instructor/dashboard -> GET /api/v1/dashboard (updated)
This endpoint was not matched with the endpoint document, therefore, it is updated to the correct endpoint.
GET /api/v1/simulation/:simulation_id/introduction
POST /api/v1/simulation/:simulation_id/introduction -> PUT /api/v1/simulation/:simulation_id/introduction
Updated for the same reason with the first endpoint.

When moving stubs, status codes are removed from JSON response. I think including status codes to JSON response is not the correct way in Express.js. Express.js has a method status in the response object, which is a method to send a status code as HTTP/HTTPS format. Therefore, all the status codes such as success (200 or 202), not found (404), etc. must be sent by this method. But if we need additional error codes to describe details of errors, I think that can be included in JSON response.

Great stuff young!


tab text should be visible on mobile

to reproduce, view any lesson in a screen narrower than 750px and hover on a tab

i'd like to take this issue please

@marwicha please go ahead.

Just a friendly reminder that the first submitted pr will be considered for the review process regardless of assignment.

Hello I am new to open source can I work on this?

@wincentrtz, please go ahead with submitting your solution as a pr.

Hi @ahmadabdolsaheb, I can't reproduce this on my local dev and https://freecodecamp.dev. The tab menu is in light gray with black text. Hover state make it slightly darker. Both state are easily readable.
Is it already fixed? or Am I missing something obvious?

@rickysyamsi, seems like you're in day mode which doesn't have this problem. Log in to freecodecamp.dev then go to Portfolio, Update my Settings, and switch to night mode. Then when you hover over these tabs in the mobile view you'll see the light colored tabs and text like in the OP.

Thank you for your interest @rickysyamsi,
So a:not(.fcc_suggestion_item):hover { on global.css is a very specific selector that overrides any class selector.
I would double check this issue after #37208 goes through to see if this issue can still be reproduced.

@scissorsneedfoodtoo @ahmadabdolsaheb Thank you for the responses guys.
I can reproduce this issue with night mode enabled.
I also still able to reproduce this issue with changes from PR #37208 copied to my local dev.
I can solve the issue with a specific selector that already exists in classic.css, I just add the necessary hover state.
#challenge-page-tabs .nav-tabs > li > a:hover {
  color: var(--secondary-background);
}

var(--secondary-background) is derived from hover state color of footer tab menu (Help, Video, Hint, Reset, Run) so it looks consistent.
I think that using a specific selector is a good approach to make sure that there are no unintended side effects. What do you guys think about this?

@rickysyamsi thanks for investigating that. yes that would be a good approach.
I look forward to your the PR.

@ahmadabdolsaheb You're welcome. I'm glad to help :)


Feat: Authorino UI basic test

New tests for the Authorino (Kserve Authentication operator) feature.

Tests Included:

Verify User Can Serve And Query A Token Protected Model Using The UI
Verify User Can Serve But Can't Query A Token Protected Model Without The Token
Verify User Can Serve And Query A Model Using The UI Protected With Multiple Tokens
Verify User Can not Query A Token Protected Model With A Disabled Token Using The UI
Verify User Can Query A Token Protected Model Using The UI Without Token After Disabling Token Protection

Jira: https://issues.redhat.com/browse/RHOAIENG-4603

@FedeAlonso Can you provide a successful run of these Test Cases on the rhods-ci-pr-test job?

@FedeAlonso Can you provide a successful run of these Test Cases on the rhods-ci-pr-test job? (build number)

@asanzgom Sure
I've created a run and it was a Success, but It has been already deleted (it was the /rhods-ci-pr-test/3024/ and has finished less than 15 mins ago.)
This an screenshot from the received mail. I can send it personally to you if needed.

Test results after last modifications:


Use grateful

(Related to #30) include https://github.com/Pakillo/grateful in the html template.

Might benefit from using multiple bibs: https://www.andrewheiss.com/blog/2023/12/11/separate-bibliographies-quarto/#how-to-get-quarto-to-work-with-multiple-bibliographies


Docs:

What is the improvement or update you wish to see?
Update the copy command for .env.template files so that when running the project it actually works.
Is there any context that might help us understand?
The package/commercejs README shows:

Despite several tries that command did not help. Only modifying the .env file at /site seemed to work.

The command works, but yes, it depends on where you run it, if you run it from the root folder of the project, use:
cp packages/commercejs/.env.template site/.env.local
or if you run it from the site folder, use the following:
cp ../packages/commercejs/.env.template .env.local


merge 0.4.0 to develop

What this PR does:
remove unused *.jar
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #291 (544d101) into develop (74afc4e) will increase coverage by 0.71%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #291      +/-   ##
===========================================
+ Coverage    33.13%   33.84%   +0.71%
===========================================
  Files           62       61       -1
  Lines         3323     3203     -120
===========================================
- Hits          1101     1084      -17
+ Misses        2108     2007     -101
+ Partials       114      112       -2

Impacted Files
Coverage Δ

...adapter/dubboregistry/remoting/zookeeper/client.go

pkg/client/dubbo/dubbo.go
20.85% <0.00%> (+0.71%)
:arrow_up:

Continue to review full report at Codecov.

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


Headers in pasted URLs (external links) do not load the page at the specified header

Posted about this before but the issue was closed immediately after someone said to update (which did not solve the issue).
I did ask for it to be reopened 4 days ago, but it has not been so I am making another.
🐛 Bug Report
When trying to load a page at a specific header via an external link or pasting in a URL with a # header argument, the page instead loads at the beginning rather than at the header specified.
Have you read the Contributing Guidelines on issues?
Yes
To Reproduce

Paste in this link in your browser and hit enter (https://docs.lunalabs.io/docs/playable/getting-started/step-by-step#2-create-your-playable-scene-in-unity)
Notice that the URL removes the header argument and instead loads at the beginning of the page

Expected behaviour
I expected the page to load with # header still in the URL and at that specified header location on the page.
Note: Links that are on pages that link to headers (both for headers on the same page, and to headers on other pages) work fine.
Actual Behaviour
The page instead loads at the beginning rather than at the header specified, and you can see the URL change before the page loads to remove the # header argument from it.
A gif of this happening

(A gif of the full page view of the problem occurring if you click here)
Your Environment

Public source code: Not available publicly
Public site url: https://docs.lunalabs.io/
Docusaurus version used: 2.0.0-beta.0
Environment name and version: Chrome 90.0.4430.212, Babel 7.10.4, Node 14.0.19
Operating system and version: Windows 10 Pro, Version 20H2

Thanks for upgrading.
I didn't re-open just because I don't work on weekends :)
Also was pretty sure this issue was a duplicate of something else. Just found it again, so let's close this one: https://github.com/facebook/docusaurus/issues/3081

hmmm not 100% sure it's the same bug, but looks related, so re-opening

Yeah there issue seems to be links that are within their docs, whereas for us it is external links or pasted in URLs that don't function properly.

Just following up on this, do you have any suggestions for a workaround?
Or if this has turned out to be a known bug or not?

It seems to be related to hosting provider of your site, for some reason it makes a redirect, which is why hash in URL seems to be lost after redirection. Are you using GitHub Pages?

We are indeed using GitHub Pages

And this is strange since I have deployed demo site on GitHub Pages and I cannot reproduce this issue. Perhaps you added redirects or something like that?

Turns out we had a script that we made a while back that fixed a similar problem previously, but had stopped working.
An adjustment was made and things work fine now


f4 / t4 address support not implemented

I was trying to use Filecoin.js to sign a message to a f4* address on the FVM Wallaby testnet, and I was getting an exception with
the message "Unknown protocol indicator byte."
I traced it down the code here:
[Unknown protoc](https://github.com/Zondax/filecoin-signing-tools/blob/945ed77a36ca1f45f075b0e15b1e3c1ae97b5af8/signer-npm/js/src/utils.ts#L109)
There isn't yet a ProtocolIndicator entry for f4 / t4 addresses:
https://github.com/Zondax/filecoin-signing-tools/blob/945ed77a36ca1f45f075b0e15b1e3c1ae97b5af8/signer-npm/js/src/constants.ts
Here is the official FIP for the f4 address class:
https://github.com/filecoin-project/FIPs/blob/master/FIPS/fip-0048.md

:link: zboto Link

Hi @jimpick! Sorry for making you wait for so long. The package is out. Closing the PR.
https://github.com/Zondax/filecoin-signing-tools/releases/tag/v2.0.0


Livewire 3 requests are not identified as Livewire in transansaction

How do you use Sentry?
Sentry SaaS (sentry.io)
SDK version
3.8.2
Steps to reproduce

Use Laravel with Livewire 3 (regular Livewire Components, not Volt)
Make livewire component update request, using the default POST /livewire/update endpoint
See the transaction in Sentry as /livewire/update

Expected result
I expect to see the component name in the transaction to differentiate between different component updates
Actual result
I see the default /livewire/update as transaction name

I guess this is being addressed in #736

Yes, correct, unfortunately Livewire team has made some drastic changes and no guide on how to migrate our current setup to their new structure. So we didn't complete the work yet to get our integration working for Livewire 3. If you have insights I'd be glad to accept some help on the PR otherwise we will eventually get to it!

Thanks for the answer!
I'm pretty new to Livewire so I don't have any insight (yet) that could help 😄

Did some research today
We can still listen to some events via LivewireManager::listen

component.booted -> ???
component.boot -> ???
component.dehydrate -> dehydrate
component.mount -> mount

There are zero docs, zero upgrade guides nor any meaningful PRs to look at, so this is very tedious.
To check for a livewire request, they now perform request()->hasHeader('X-Livewire').

Support for Livewire 3 was added in v4.0.0.

Support for Livewire 3 was added in v4.0.0.

Is support for livewire2?

Yes, both Livewire 2 and 3 are supported.

@stayallive Can you help me?
I have issue with livewire2 and sentry. Each action of livewire, it will make transaction like " livewire?component=pos.pos-transaction". But my url is pos/pos-transaction. How can I group it in transaction pos/pos-transaction and identify each action of livewire (Example: search, goToPage...)

Please open a new issue.


Failed to submit trail with http code 500

nni Environment:

nni version: 1.3
nni mode(local|pai|remote): pai 0.14
OS: ubuntu  16.04
python version: 3.6
is conda or virtualenv used?: anaconda
is running in docker?: no

log
[2/8/2020, 2:48:24 PM] ERROR [ 'Submit trial JHmXB failed, http code:500, http body: {"message":"Request failed with status code 400"}' ]
[2/8/2020, 2:48:27 PM] DEBUG [ 'ipcInterface command type: [PI], content:[]' ]
[2/8/2020, 2:48:28 PM] DEBUG [ 'GET: /experiment: body:\n{}' ]
[2/8/2020, 2:48:28 PM] DEBUG [ 'GET: /check-status: body:\n{}' ]
[2/8/2020, 2:48:28 PM] DEBUG [ 'GET: /trial-jobs: body:\n{}' ]
[2/8/2020, 2:48:28 PM] DEBUG [ 'getTrialJobsByReplayEvents begin' ]
[2/8/2020, 2:48:28 PM] DEBUG [ 'getTrialJobsByReplayEvents done' ]
[2/8/2020, 2:48:28 PM] DEBUG [ 'GET: /metric-data: body:\n{}' ]
[2/8/2020, 2:48:28 PM] DEBUG [ 'GET: /experiment: body:\n{}' ]
[2/8/2020, 2:48:28 PM] DEBUG [ 'GET: /trial-jobs: body:\n{}' ]
[2/8/2020, 2:48:28 PM] DEBUG [ 'GET: /metric-data: body:\n{}' ]
[2/8/2020, 2:48:28 PM] DEBUG [ 'getTrialJobsByReplayEvents begin' ]
[2/8/2020, 2:48:28 PM] DEBUG [ 'getTrialJobsByReplayEvents done' ]
[2/8/2020, 2:48:28 PM] DEBUG [ 'GET: /version: body:\n{}' ]

more information

trail num: 5
nni <= 1.2 works fine with the same configurations.

Hi @ICEORY, what is your running mode?

config.yml

authorName: iceory
logDir: /media/nni
experimentName: debug_nni
trialConcurrency: 5
maxExecDuration: 10h
maxTrialNum: 5
trainingServicePlatform: pai
useAnnotation: true
nniManagerIp: 10.10.xxx.xxx
tuner:
  builtinTunerName: TPE
  classArgs:
    optimize_mode: maximize
trial:
  command: echo hostname
  codeDir: ./
  gpuNum: 1
  cpuNum: 4
  memoryMB: 8000
  virtualCluster: default
  image: my_nni_docker
  containerNFSMountPath: /ssd/iceory/nni_container # nni >=1.3
  nniManagerNFSMountPath: /media/nni # nni >= 1.3
  paiStoragePlugin: team_wise # nni >= 1.3

paiConfig:
  userName: ******
  passWord: ******
  host: 192.168.xxx.xxx

Does the PAI's cluster you are using support password login?  Some clusters of PAI only support token authentication, this issue might caused by the authentication problem. The default host prefix NNI used is http, could you please check if your host support http prefix, or only support https prefix?

our PAI cluster support password login
the host prefix is http, not https
nni <= 1.2 is ok, so I guess the issue is caused by the update of nni 1.3

In nni v1.3, the original PAI mode is modified to PAIYarn mode, https://github.com/microsoft/nni/blob/master/docs/en_US/TrainingService/PaiYarnMode.md. The new PAI mode in NNI is for PAI's pure K8S version. Is your PAI cluster a pure K8S version or a Yarn based version? Is your PAI's storage HDFS or NFS? Have you tried using paiYarn mode in NNI?

Our PAI cluster is Yarn based. I change the config file and the issue is solved. Thanks!


fix: changed to post request from get request

@AravindRam Please review and merge the changes. I have also fixed a variable casting which was not done right.

@AravindRam I have to remove the test file since it wont allow tika to build without the grobid server running. Will write a proper test with all use cases soon.

Hi, sorry to chip in but I saw this request from Tika and wanted to help :) ,
Post request should work fine if you update grobid-quantities from the repo(not grobid)
Thanks

@cmenekse
Thanks a lot. That worked!
I did not notice that there was an update to the grobid quantities repo.

@yashtanna93
I have merged that fix now. But, let me know what was the issue with the test code.

@AravindRam The issue is it is not ignoring the test incase of server not running. I really don't know how it works without the rest server running.
@chrismattmann could you give us some insight how does nltk parser does the test code when the rest server is not running

hi @yashtanna93 @AravindRam it checks to see if something is running on the port, and if not, it just uses assumeTrue (if it's not true it still goes on).


[Backport] [1.3] Discrepancy in result from _validate/query API and actual query validity (#2416) #2466

Signed-off-by: Andriy Redko<EMAIL_ADDRESS>Description
Backport of https://github.com/opensearch-project/OpenSearch/pull/2416 to 1.x
Issues Resolved
[List any issues this PR will resolve]
Check List

[X] New functionality includes testing.

[X] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dblock that's sad, but may be for 1.3.1 ?

@dblock that's sad, but may be for 1.3.1 ?

Yes, let's wait a bit till 1.3 does get released to merge, or some automation will pickup the change as 1.3.

@peterzhuamazon @dblock the 1.3 is on 1.3.1 already, worth merging? thank you!

@peterzhuamazon Can you please approve this one now that 1.3.1 has been incremented?

Closing, this didn't make it into 1.3, see you in 2.0!


Update committers.md

Working on the document updates for this page
Purpose / why
As per comment, updates the committers list to add Annie, but also implements the recently discussed changes to the committers program.
What changes were made?
As above
Verification
You could review the google doc that's linked in the comment below if you want to check the other changes.

As per @rosiel 's advice, I'm asking @aOelschlager to commit this change as our newest committer (Hi Annie! This PR updates our committers language to the latest copy in this document and adds you to our list of committers!

Annie already merged the committer list: #2202
The rest looks good.

whoops. Thanks @rosiel


fix: only log after cordon succeeds

Fixes #
Description
If the cordon in a termination reconcile failed, we would still log as cordoned.
How was this change tested?
make presubmit
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
19 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.02%) to 81.435%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/termination/terminator/terminator.go
1
92.59%

pkg/controllers/deprovisioning/helpers.go
2
83.64%

pkg/scheduling/requirements.go
2
98.5%

pkg/controllers/node/controller.go
7
73.53%

pkg/controllers/provisioning/scheduling/preferences.go
7
86.67%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
6764

Relevant Lines:
8306

💛  - Coveralls


Result Preview Panel, duplicate field "URL" when retrieving RSS indexed news

Dear Sirs,
as You can notice in the attached image, "reference" and "URL" metafields have the same content, and I am not able anyway to hide "URL". Also, I tried to "send" it in the "Advanced" hidden portion of the page (advanced=true in config.json) but that field is still there over the "reference" field. look like it is a standard field (like reference, title, summary, database)
Is there a way to move/hide such a field?

I attach my config.json, too
thanks v.m.
config.json.txt

no replies since 1 month?


Final checks for july implementation

These are all pretty minor tidy-up edits. @johnarban, feel free to take a look if you have time. If you don't it should be fine to merge before morning.

these seem alright to me


Assignment inside iterator scope

This:
for x in myList
  x ?= 1

throws Error: the variable "x" can't be assigned with ?= because it has not been defined.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/18023140-assignment-inside-iterator-scope?utm_campaign=plugin&utm_content=tracker%2F33145&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F33145&utm_medium=issues&utm_source=github).

https://github.com/michaelficarra/CoffeeScriptRedux/issues/299 looks like the same bug, although this example is broader -- no destructuring is needed to trigger the bug.


mpv integration

suggestion, so that we can see videos inline

I think actually playing videos inline inside the Lagrange window would add too many dependencies to the build.
At this point I’m more inclined to implement an external handler feature so you can easily hand the video URL to a video viewer app (e.g., mpv).

I think actually playing videos inline inside the Lagrange window would add too many dependencies to the build.
At this point I’m more inclined to implement an external handler feature so you can easily hand the video URL to a video viewer app (e.g., mpv).

I'd like to see this, especially if macOS/Linux allowed the use of the standard ~/.mailcap.


Running the rife model, it always fills up the memory and gets killed

Even with 120G memory, how to solve

Haven't had a similar problem.

Videos with dynamic frame rates and dynamic bit rates from acfun.com . typically range in size from 100MB to 300MB. I've tested interpolation on domestic cloud GPUs,
Videos with fixed frame rates and dynamic bit rates also deplete memory during testing.

Splitting the video into multiple short videos, interpolating them and then merging them together is one solution. Alternatively, you can modify the video inference code (e.g. “models/DI-RIFE/inference_img_plus_sdi.py”), which now does this by extracting all frames and interpolating them.


PROPOSAL: download-product can download from external blobstores (ie s3) with same configuration file

Problem
For Ops Manager users in air-gapped environments, downloading products via the concourse s3-resource can be difficult. For example, the regex to match the *.pivotal file can match newer versions than what they'd like to install on the current OpsManager.
The download-product config file is an external configuration file, which allows explicit configuration of product, a version or version range, and the latest stemcell to be downloaded. This works well when the environment can connect to Pivnet.
We'd like to use the same config file to support downloading assets from an s3 blobstore.
Proposed Change
Pivnet provides an API to download a product and associated stemcell. This same API cannot be replicated in s3. When the product files (as-is) are uploaded to s3, there is no metadata associating with the product slug and exact Pivnet version number.
When download-product is used to download a file from Pivnet, the outputted file would be prepended with the Pivnet product version specific information.
For example, with the product configuration:
---
pivnet-api-token: some-token
pivnet-file-glob: "*vsphere*ova"
pivnet-product-slug: ops-manager
product-version: 2.4.3

The outputted *.ova file would like ops-manager-2.4.3-pcf-vsphere-2.4-build.145.ova instead of pcf-vsphere-2.4-build.145.ova.
This file would be stored in s3, via a concourse put or s3 CLI.
Within the air-gapped environment, additional configuration is required to connect to an s3 bucket.
For example,
pivnet-api-token: some-token
pivnet-file-glob: "*vsphere*ova"
pivnet-product-slug: ops-manager
product-version: 2.4.3
s3:
  bucket: my-bucket
  access-key-id: some-token
  secret-access-key: some-token
  region-name: region
  endpoint: https://example.com
  disable-ssl: false
  skip-ssl-verification: false
  use-v2-signing: true

When invoking download-product the flag for --air-gapped will be provided to force usage of s3 instead of pivnet.
Then download-product would look for all files prepended with ops-manager-2.4.3 and then use the pivnet-file-glob to narrow down selection.
Notes

The outputted file would be a different format from current download-product. This is a breaking change for anyone doing a put in a pipeline. We could only do this action we know this will also be used for an air-gapped environment -- perhaps if s3 and pivnet credentials are detected.
Stemcell support is coming. tl;dr: we'd look at the tile metadata, and search s3 for the latest tile. It doesn't require the same prepending.
We'd follow a similar pattern for Azure and GCS. We are just doing s3 first.

why --air-gapped instead of something like --blobstore?
"Air Gapped" has certain meanings for some customers, which differ based on who is using the term. It can also have certain security connotations that may or may not be correct.
Thinking ahead, is the pattern for azure/gcs going to be discerned by different config key names, or are we thinking different flags?

This is feature is being worked on. We have been able to achieve everything from above.
We've also added support for v2 signing, so that it may with older s3-compatible blob stores.

This update has been updated and released in https://github.com/pivotal-cf/om/releases/tag/0.52.0

We're making substantial changes to this from its initial release, so we're re-opening this issue. I've updated the original description to reflect our working notes.

Download-product has been updated and released!:https://github.com/pivotal-cf/om/releases/tag/0.54.0
The biggest changes are around the prefixing that we were using. Instead of {slug-version}-restoffilename.tgz, where if the file somewhat matched we didn't prefix, we are always prefixing with [slug, version]originalfilename.tgz. This was done to make the prefix more obvious. This prefixing method also allowed for this command to be more stable and fixed a lot of bugs from 0.52.0.


Updating URL Schema - "turtles.docs.rancher.com"

Adding the updated URL schema for Rancher Turtles documentation in the config and the CNAME file. Additionally updated links that were throwing errors while attempting yarn start.

I think this is superseded by https://github.com/rancher/turtles-docs/pull/74

I think this also configures the docs to be hosted on a *rancher.com domain so I don't know if it's completely superseded by #74.

@sunilarjun Can you please provide reasoning for CNAME change? We are hosting this currently under https://docs.rancher-turtles.com/. Who owns the CNAME?

Hello @Danil-Grigorev! The CNAME change was approved in the site standardization meeting to align with SUSE docs sites, and other changes are listed in the main issue as well. The CNAME is currently maintained by EIO and there is a ticket to update the subdomain here: https://github.com/rancherlabs/eio/issues/2141. I believe the ticket must be completed first, and then this PR can take effect.

CNAME PR https://github.com/SUSE/suse-it-infra/pull/4140 is merged, this should be ready


RTM

Hi, ive just seen this repo and ive got very excited about it, as im a huge fan of C# instead of JS so i was wondering when you guys are gonna release an RTM stable version of Blazor?, and also is there any documentation like the one for the ASP.net core in docs.microsoft.com or any where else??
tnx

You can check this site for documentation :https://blazor.net/

This is a hub for Blazor related stuff.
https://github.com/AdrienTorris/awesome-blazor

We don't have a specific announcement about release plans yet, but are working on it and will let you know as soon as we do have a specific plan.


Native extension is downloaded to readonly location on NixOs

OS: nixos
Nixos has a very read-only approach to filesystems. Currently the location that the native extension is written to is readonly and fails. Rust-Analyzer interestingly also downloades a native extension but doesn't fail. They write their extension to globalStoragePath (which is ~/.config/Code/User/globalStorage, so it's writable by the user running Code.)
If it's not a large change, would that be a better location to download the native extension to than where its currently being downloaded to?

The download is saved at the path returned  by os.tmpdir() (most likely /tmp).  Are you saying it isn't writable?

Interesting - thanks - let me do some digging...
Sent with GitHawk

I can't quite figure out what the issue is, but it does seem to be downloading ok, just doesn't seem to be a valid exe Then again when I compile the adapter2 from source it doesn't seem happy either despite compiling fine.


Unify version check TC

Signed-off-by: Luca Giorgi<EMAIL_ADDRESS>

Tested locally and works, Jenkins currently unavailable to launch a PR test

PR job: https://opendatascience-jenkins-csb-rhods.apps.ocp-c1.prod.psi.redhat.com/job/rhods-ci-pr-test/365/

PR job: https://opendatascience-jenkins-csb-rhods.apps.ocp-c1.prod.psi.redhat.com/job/rhods-ci-pr-test/365/

the Verify Libraries in Tensorflow Image is failing because of a missing element. Is that expected?

It failed because it could not find Element with locator '//input[contains(@id, "tensorflow")]/../span' not found., seems like a timing issue or similar

Additional PR test: https://opendatascience-jenkins-csb-rhods.apps.ocp-c1.prod.psi.redhat.com/job/rhods-ci-pr-test/371

lgtm

/test image=qe-catalog-source:v160-8;testcluster=qeaisrhods-tr5;testrepo=https://github.com/lugi0/ods-ci;testbranch=feature/automatic-version;testarg=--test-case tests/Tests/500__jupyterhub/test-versions.robot;


[SMALLFIX] Fix typo in error message

post -> port

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/7111/
Test PASSed.

LGTM