Please add the example with CPU-intensive or blocking operations

Feature Request
Crates
tonic
Motivation
It would be nice to know if there is anything special needs to be done to run CPU instensive/blocking tasks in Tonic service.
Proposal
Please add a simple hello_world or echo example where server is correctly performing plain blocking operation, for example blocking sleep or blocking file read. For example it would also nice to see how to control the size of the underlying thread pool and how to combine purely async and CPU-intensive tasks in one service if possible.
There are examples like helloworld/server_blocking.rs, but it is not quite clear what are they demonstrating, they use different runtime. Is it the runtime for CPU intensive tasks?
Would simple code like this do the job:
let reply = tokio::task::spawn_blocking(move || {
    ...
}).await;

Any details that are worth noting?
Alternatives
If existing examples do already demonstrate that, we can make comments/readme more explicits explaining it.
P.S. Great job with the project, thanks!

Thanks for the kind words! And, tokio will soon have more docs for this (and I would totally accept a PR with an example) type of workload. In general, I would suggest using tokio::task::spawn_blocking since that will provide better consistent latency.

Here are some docs on how to use blocking https://docs.rs/tokio/0.2.22/tokio/task/index.html#blocking-and-yielding nothing here is specific to tonic, so I don't know if it makes sense to include an example for it. Going to close this issue, feel free to re-open if you have more questions.


Enable ZSTD buffer pool by default

Reporter: Dongjoon Hyun / @dongjoon-hyun
Assignee: Dongjoon Hyun / @dongjoon-hyun
Note: This issue was originally created as PARQUET-2044. Please see the migration documentation for further details.

Dongjoon Hyun / @dongjoon-hyun:
https://github.com/apache/parquet-mr/pull/903


Simplify sdk

Description
This PR simplifies the SDK, by using the context.Context to pass
data between Endpoints in the chain.
By doing this, and a few other minor changes, the Endpoints used
in CompositeEndpoint just need to be normal implementers of
networkservice.NetworkServiceServer
Among other things, this means that CompositeEndpoints can be nested
within each other.
Note: Updates to sdk/README.md will be forthcoming.
Work is related to #1324 #1508
Motivation and Context

How Has This Been Tested?

[x] Covered by existing integration testing
[ ] Added integration testing to cover
[x] Tested locally
[ ] Have not tested

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

@haiodo This looks good to go to me once it passes CI.

@edwarnicke @nickolaev @lobkovilya It seems all issues related to SDK are fixed, Please take a look. Thanks.

rebased on master


home screen import modal off center in beta

Describe the bug
import modal is off center

To Reproduce
Steps to reproduce the behavior:

Go to home screen import dialog
modal off center

Expected behavior
not off center

also gear wheel -> report abuse on homescreen

and 'reset' dialog, about, etc; looks like just all on homescreen


ISPN-7391 & ISPN-7392 Hot Rod authentication improvements for both client and server

https://issues.jboss.org/browse/ISPN-7391
https://issues.jboss.org/browse/ISPN-7392

Add simple authentication configuration to the Hot Rod clients
Ensure the server applies SASL policies correctly
ship a more secure example authentication configuration
enhance the server docs with more details about securing a Hot Rod server

@wburns I've cleaned up the docs and added a paragraph to the upgrading guide as suggested

Pulling...

Integrated into master, thanks @tristantarrant !


Just a friendly reminder of the new materialize repo

I've waited any real update of this repo for a quite long time, but nothing except the donation kind of update. You may have the same question will me.
Now there is a new fork to this repo, which I believe it could be call real materialize css repo:
https://github.com/materializecss/materialize
And also I wanna thanks those guys who create and maintain that repo.

By the way, there will be a new release in recent days according with the discussions.
What a wonderful day when I got this info.

yes, the forked project is dusting off the dirt and getting the project back to 2020. more progress there than in the last year in this repo.
the maintainers here, we thank you for the hard work. and now that you are gone, please hand over the project properly.


Get cloud monitor supported namespaces and metrics from SDK.

Hi Team.
When building an app that syncs cloud monitor metrics via cloud API. It is necessary to provide namespace and metric names so that the metrics data can be pulled.
Now, one implementation is to read documents at https://cloud.tencent.com/document/product/248/6843 and manage the metric names in config file.
It is possible to get the supported metric names via SDK? So the code can be nice and clean without managing a long list of metric names by developer themselves.

Sounds reasonable, but those documents are not synced by my team, so I cannot generate the namespaces & metrics in the SDK for now.
I will discuss it with the Cloud Monitor team later.
Thanks for the feedback.


Mac Aarch64 test results never arrived in TRSS for Java 17 release

https://ci.adoptopenjdk.net/job/build-scripts/job/openjdk17-pipeline/188/ is the pipeline that kicked off the build
https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk17/job/jdk17-mac-aarch64-hotspot/46/console is the build job that then triggers the test job (which all completed)
NOTE: The test runs were delayed due to an offline machine, but once switched on it went to completion.

The output of https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk17/job/jdk17-mac-aarch64-hotspot/46/ is too big (~170M+). It exceeds the TRSS limit.


Wrong couchbase ticket registry configuration

spring.factories in cas-server-support-couchbase-ticket-registry loads absolutely wrong configuration file - file from MongoDB module.
Was it tested?

Thanks for the find. You should submit a pull request that fixes the problem and then start testing.

@FeyFre Due to bad edit in 6ae24ed.
It was reverted in 52f83fb but hasn't been published out to the repositories yet. I'm still getting the same error at CAS 5.1.1 (Gradle)

Just noticed the build config is configured to publish to sonatype. Should be published soon.


feat(storage): enable client's bulk query read from remote storage

Context
See #11597. This is an improvement by slightly changing the interface to accept a query slice instead of a pointer, and yields multiple query results. The effort was just refactoring storage calls that must pass a slice instead, and already existing function calls that support a single query result are not affected.
Changes

Change client's Read function to receive a slice of *prompb.Query, and return a slice of *prompb.QueryResult.

Additional information
N/A.

I don't understand.  You changed an internal interface?  No user can do anything more after this PR than before?
Also, CI tests are failing.

@bboreham Nobody could pass more than one query through the client's read function, even though we can send more than one query in the proto's. If you could however pass a slice to it, you can send one or more queries to it, being more versatile than just restricting to a single query param.


Update link to the spec

Update link to the spec to go to gtfs.org (contains the latest spec) instead of Google's documentation.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


logging: Prevent multiple arguments evaluation

Logging v2 is utilizing complex preprocessing operations to
prepare message at compile time. Multiple operations are peformed
on log message arguments. However, it is expected that argument
will be evaluated only once (e.g. it can be a call to a function
with side effects). Adding additional layer which creates copies
of user arguments on stack and passes them to further processing.
Fixes #42050.

@carlescufi @nashif it's usually hard to find second reviewer for logging. This is medium bug so it would be nice to close it for v3.0.0. Any idea who could take a look at that?

Fix is relatively simple. It's about creating local variables from input arguments before passing them further. The only trick is that local variable has to be initialized as arg + 0 to promote and avoid errors if argument is a bit field. But that leads to warnings when argument is void * so warning needs to be suppressed.


Nested hierarchy levels?

I try to understand the YAML syntax. In your demo ur'e using nested Links inside a sub-menu (Icons->Font Awesome). Can you provide the YAML syntax or the source for that?
Regards

Nevermind, I just found it in the docs...

@thle Do you have some suggestions on how to improve the YAML syntax documentation? Was it hard to understand? I know it's not straightforward, because of the limitations of YAML and my wish of supporting a concise syntax for bookmarks with and without notes.


#7292: Add repeat ttnn sweeps

Add YAML and generation function for TTNN repeat op

+@rtawfik01
are changes in llk modules tt-llk-* intentional or accidental?
Could you please check?


Mobile product ads aka Advantage+ Catalog Ads not receiving params

Hi,
I'm on  "react-native-fbsdk-next": "^13.0.0" and "react-native": "0.71.6".
I'm using the following hook in AppStackNavigator:
export default function () {
  useEffect(() => {
    async function manageiOSTracking() {
      const dispatch = store.dispatch;
      let status = await check(PERMISSIONS.IOS.APP_TRACKING_TRANSPARENCY);
      Settings.initializeSDK();
      Settings.setAppID(FACEBOOK_APP_ID as string);
      Settings.setClientToken(FACEBOOK_APP_CLIENT_TOKEN as string);
      if (status === RESULTS.UNAVAILABLE || status === RESULTS.DENIED) {
        status = await request(PERMISSIONS.IOS.APP_TRACKING_TRANSPARENCY);
      }

      if (
        status === RESULTS.GRANTED ||
        status === RESULTS.LIMITED ||
        status === RESULTS.UNAVAILABLE
      ) {
        console.log('iOS tracking granted');
        Settings.setAdvertiserIDCollectionEnabled(true);
        await Settings.setAdvertiserTrackingEnabled(true);
        dispatch(setUserAdsTracking(true));
      } else {
        dispatch(setUserAdsTracking(false));
        console.log('iOS tracking NOT granted');
      }
    }

    manageiOSTracking();
  }, []);
}

Then, for example at first checkout step of my app, I track the INITIATED_CHECKOUT event like this:
 AppEventsLogger.logEvent(
        AppEventsLogger.AppEvents
    .InitiatedCheckout,
        100,
        {
        content_type: 'product',
        ids:  product.ids?.length > 1
        ? `[${product.ids.map((id: number) => `"${id}"`).join(',')}]`
        : product?.ids[0];
        currency: '€',
        num_items: 3,
      );

Now, I can see both from network tab of developer tools and Meta Developer Events Manager that events are successfully sent, but I have two major problems:

I'm using this library and I'm sending events with parameters in order to make work the Advantage+ Catalog Ads set by my customer, but the relative "flags" don't turn "green", as you can see in this image below:

If I try to use the "Test events" function in FB Events manager, even triggering many events from my app doesn't show anything on the realtime dashboard side...

Any hint?

Ok, for the posterity, solution to problem 1): obviously you have to log in in the app with the same Facebook Account you're using as Meta Developer.


Recursive CTEs not allowing direct selection into struct

Currently there is no way to explicitly use the recursive selection and select directly in a struct.
The only workaround I could accomplish is to inner join with the same table and extract the values from that join:
category_tree_initial_query =
  Category
  |> where([c], c.id == 1)

category_tree_recursion_query =
  Category
  |> join(:inner, [c], ct in "category_tree", on: c.parent_id == ct.id)

category_tree_query =
  category_tree_initial_query
  |> union_all(^category_tree_recursion_query)

"custom_cte"
|> recursive_ctes(true)
|> with_cte("custom_cte", as: ^category_tree_query)
|> join(:inner, [c], c1 in Category, on: c.id == c1.id, as: :cat)
|> select([cat: cat], cat)

The issue here is that you are not starting the query with a schema, you are using "custom_cte". You can attach a schema to it by using a tuple {"custom_cte", Category} |> with_cte(...) |> ....

Thank you! The example in documentation is a little misleading, I thought the syntax is for selecting a touple.


feature request: field expressions in preview labels (and text alignment)

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[x] Mac OS X
[x] Windows
[x] Etc.

Shell

[x] bash
[x] zsh
[x] fish

Problem / Steps to reproduce
preview labels are a great feature, it would also be great to have it parsed for field expressions, eg
--preview='cat {+}' --preview-label='cat {+}'

separately (and forgive me for not filing a separate issue) it would also be nice to optionally left/right align the preview label, eg --preview-label-pos='top,right' or --preview-label-pos='bottom,left'

fzf --multi --preview 'cat {}' --bind 'focus:transform-preview-label:echo {+}'

left/right align the preview label

--preview-label-pos[=N[:top|bottom]]
Position of the border label on the border line of the preview window. Specify a positive integer as the column position from the left. Specify a negative integer to right-align the label. Label is printed on the top border line by default, add :bottom to put it on the border line on the bottom. The default value 0 (or center) will put the label at the center of the border line.

fzf --multi --preview 'cat {}' --bind 'focus:transform-preview-label:echo {+}' --preview-label-pos top,2

fzf --multi --preview 'cat {}' --bind 'focus:transform-preview-label:echo {+}' --preview-label-pos top,-2


Automatic install on Android misses permissions setup

RN 50.4
RNIP- latest
As per topic - linking misses adding
===================================
uses-permission android:name="android.permission.CAMERA"
uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE"
===================================
necessary strings to AndroidManifest.xml, they still should be added manually.
This probably should be reflected in docs.

There is a part titled IMPORTANT NOTE in https://github.com/react-community/react-native-image-picker#automatic-installation which says You'll still need to perform step 4 for iOS and steps 2 and 5 for Android of the manual instructions below. From the Readme file on automatic installation


postgres part of rdkit with out installing postgres also

the installation docs http://rdkit.org/docs/Install.html are missing instructions on how to install the postgres part of rdkit with out installing postgres also.
for example, we already have postgres 11 installed on the machine, and would like to install rdkit in such a way that it uses that vs installing postgres 9.
I tried to get it to stop installing postgres also. but, the several things I tried seemed to be of no avail.
it would seem to me that this would be a somewhat common issue and the docs should address this.

Yeah, the docs for the postgresql install are definitely overly reliant on conda; sorry about that.
The "short" form of the answer is that you need to do an RDKit build with the cartridge build enabled via the -DRDK_BUILD_PGSQL=ON argument.
Here's a command I just did from a builds/build_pgsql directory inside the source tree (if you have a recent version of boost installed on your system, you can skip the -DBOOST_ROOT part):
cmake -DBOOST_ROOT=/opt/boost_1_67_0_cxx14/ -DRDK_BUILD_PYTHON_WRAPPERS=OFF -DRDK_BUILD_CPP_TESTS=OFF -DRDK_BUILD_PGSQL=ON --PostgreSQL_CONFIG=/usr/bin/pg_config -DRDK_BUILD_INCHI_SUPPORT=ON ../..
After that finishes, you build with (replace 4 with however many threads you want to use):
make -j4

And then install and test the binaries with:
sh ./Code/PgSQL/rdkit/pgsql_install.sh
sh ./Code/PgSQL/rdkit/pgsql_regress.sh

At that point users should be able to install the extension in their own databases using create extension rdkit from the psql prompt.
Here's a quick demo of that:
(base) glandrum@otter:/scratch/RDKit_git/builds/build_psql$ psql my_test_db
psql (10.9 (Ubuntu 10.9-0ubuntu<IP_ADDRESS>))
Type "help" for help.

my_test_db=# create extension rdkit;
CREATE EXTENSION
my_test_db=# select 'c1ccccc1O'::mol;
    mol
-----------
 Oc1ccccc1
(1 row)

my_test_db=#


Verdaccio UI not visible

I deployed the vardaccio/verdaccio:5 image using deployment in the EKS cluster and served on the ingress load balancer. I can't show the UI page in my domain. When I opened my domain in the web browser the service was done to the pod IP of the server. why did this happen?
this is deployment.yaml file
apiVersion: v1
kind: Pod
metadata:
  labels:
    app: test-registry
  namespace: default
spec:
  containers:
  - env:
    - name: HOSTNAME
      value: localhost
    image: verdaccio/verdaccio:5
    imagePullPolicy: IfNotPresent
    livenessProbe:
      failureThreshold: 3
      httpGet:
        path: /-/ping
        port: port
        scheme: HTTP
      initialDelaySeconds: 5
      periodSeconds: 10
      successThreshold: 1
      timeoutSeconds: 1
    name: test-registry
    ports:
    - containerPort: 4873
      name: port
      protocol: TCP
    readinessProbe:
      failureThreshold: 3
      httpGet:
        path: /-/ping
        port: port
        scheme: HTTP
      periodSeconds: 10
      successThreshold: 1
      timeoutSeconds: 1
    resources: {}
    terminationMessagePath: /dev/termination-log
    terminationMessagePolicy: File
    volumeMounts:
    - mountPath: /verdaccio/conf
      name: config
      readOnly: true
    - mountPath: /verdaccio/storage
      name: verdaccio-data
    - mountPath: /var/run/secrets/kubernetes.io/serviceaccount
      name: kube-api-access-6sv7r
      readOnly: true
  dnsPolicy: ClusterFirst
  enableServiceLinks: true
  preemptionPolicy: PreemptLowerPriority
  priority: 0
  restartPolicy: Always
  schedulerName: default-scheduler
  securityContext:
    fsGroup: 100
    runAsUser: 10001
  serviceAccount: default
  serviceAccountName: default
  terminationGracePeriodSeconds: 30
  volumes:
  - configMap:
      defaultMode: 420
      name: test-registry
    name: config
  - name: verdaccio-data
    persistentVolumeClaim:
      claimName: verdaccio-data

I can see your application uses IP address instead the domain, the application can either use only one domain, see https://verdaccio.org/docs/env#public-url
ref:

https://github.com/verdaccio/verdaccio/discussions/3055
(other tickets also discuss this, please search a bit)

Hey @juanpicado, When I edited my config.yaml file with this configuration. then my pod was gone on CrashLoopBackOff status.
VERDACCIO_PUBLIC_URL='https://verdaccio.zurutech.online/';
url_prefix: '/'

Then I removed this configuration and ran it again. then the pod was running but I got an error.

vim config/config.yaml and set html_cache false ,restart your server


Implement IntervalMonthDayNano scalar conversion in DataFusion Unparser

Describe the bug
The unparser code in DataFusion (responsible for turning LogicalPlans back into SQL statements) does not handle IntervalMonthDayNano scalar values, which causes several of the TPC-H queries to fail with the following error:
Error: "query tpch_q4 to results: External error: This feature is not implemented: Unsupported scalar: IntervalMonthDayNano(\"237684487542793012780631851008\")"
The relevant code that needs to be implemented:
https://github.com/apache/datafusion/blob/1db3263497532fda5167386781755463f53c00a4/datafusion/sql/src/unparser/expr.rs#L802
This affects the following benchmark queries:

tpch_q4

Filed https://github.com/apache/datafusion/issues/10795 to track on the DataFusion side.

The upstream fix is merged, however, the issue is still not fixed for affected bench tests, as the str unparsed from this is interval '1 mons' which is not supported by dremio connector.

Closed this to open a more specific one for failing bench in dremio.


Check following files

If needed:

sun-ejb-jar.xml - all of them
requiredGlassfishResources.properties - all of them.

Check ess version.

Fixed with  597dc95

Check also all beans.xml files if they have proper descriptor version.

Fixed with 570dd0c


[Feature] Provide a way to replace the default 'Page' with a custom instance

Hi,
It would be great if there is a way to change the way nativescript-angular's bootstrap creates the Page instances used though the application life circle. I looked though the code and noticed there is a provider (defaultPageProvider) for the Page elements (not exactly sure how it is utilized as removing it in my fork did not break any navigation) and couple of places where the new Page is being created (for example here and here)
Having the possibility of placing our own class which inherits the Page class in an Angular 2 {N} application will be very useful and as it is already possible in vanilla {N} I think it will make migrating to Angular even easier and better.

Some thoughts on that:
The Page provider gives you access to the current page instance that is hosting the component and there is no way to replace it.
Maybe the way to go is to use PageFactoryProvider, that you can replace with custom factory for pages in your app.

Another way of achieving this behavior would be (not sure if possible probably not :)) to use the same approach like the ActionBar in Angular 2, by simply declaring a <Page> in the Component itself. This way we may have different HTML declared Page instances rather than having nativescript-angular create them for us.

Hi guys, any progress on this?, has someone found a workaround for android?.


terraform provider for netbird REST API

Is your feature request related to a problem? Please describe.
It would be great to be able to manage netbird server with terraform (a lot of people already use terraform to manage their infrastructure)
Describe the solution you'd like
a terraform provider to interact with the netbird REST API

Hello @theblop, thanks for the request. We are looking at a terraform provider initial version for this quarter. We will keep this updated as we progress with it.

@mlsmaycon
Hey, do you have any updates on the topic? Which resources are you going to support?

Any update on this ? Would be super convenient !

looks like speakeasy make terraform providers for projects so it looks like the start, hopefully theres a private branch just about to be committed! 🙏

Is it safe (and legal) to use that codebase as a foundation @speakeasy-sdks / @TristanSpeakEasy?

We have a community member working on a provider. We are using the Hashicorp OpenAPI Provider Spec Generator for the foundation.
We should have an initial version of the provider earlier in Q4 this year.

Hey @tafaust that repository is meant to be an example of our generation capabilities at https://github.com/speakeasy-api and may not be up to date with the latest Netbird API
@mlsmaycon would be happy to help you all generate and maintain a really nice terraform provider. Here are a few we maintain in real time.

https://github.com/opalsecurity/terraform-provider-opal
https://github.com/StyraInc/terraform-provider-styra
https://github.com/airbytehq/terraform-provider-airbyte

Here's a walk through of our generation as well as Hashicorp's method. Would love to help if we can !

Using Speakeasy makes it incredibly simple to create a Terraform provider for Netbird. After some testing, I was able to build a functional provider. However, there are two aspects of the Netbird API that might need addressing:

Route Creation: Currently, the Netbird API allows defining either a network or a domain when creating a route. However, if an empty network value is provided while specifying a domain, the API returns an error, as only one of the two is allowed. Could this behavior be adjusted to accept an empty network value when a domain is defined? This change would simplify Terraform provider development, though it is possible to implement a workaround.

Setup Key Values: When setup keys are created, their values are returned in plain text. However, subsequent API responses for these keys provide only obfuscated values. This causes the Terraform provider to retrieve only the obfuscated version, which can also be worked around but adds complexity.

According to Speakeasy's documentation, their service might be free for open-source projects. Would the Netbird team be interested in exploring this option?
(Text reference: [Speakeasy Pricing](https://www.speakeasy.com/pricing))
If you're working on a 'free as in speech' OSS project, you can use Speakeasy for free. Just fill out this TypeForm: https://speakeasyapi.typeform.com/oss-program


daylight savings time: wrong event positioning after update

In countries that make use of daylight savings time there is a strange behaviour when moving events around the legal time change.
Steps to reproduce:
https://codesandbox.io/p/sandbox/standard-forked-3tj9r3
Try to move (drag & drop, or edit modal) "Event 1" from Sat 30 03:00 - 04:00 to Sun 31 03:00 - 04:00.
The new event will be placed on the previous cell (02:00 - 03:00).
Also Event 2 is reporting correct dates, but wrong position.
Tested with Italy's daylight savings (31/03/2024 at 02:00 AM).

Thx for pointing this out

For the issue related to the exact hour switching from summary/standard time, I havent find a proper way to fix it. I can see it also in other calendar packages. For example in this react big calendar you cannot add an event from 1-2 AM 31/3/2024. It has to be 1-3 and more. Basiaclly there is no 2AM in that day.
In the day where summer/standard switch occur, there is kinda of missing 1h, which makes the event placed on previous cell. This is a bug, and should be fixed in #352


Steps

Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

What does this PR do?
Fixes # (issue).
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hello @williamFalcon! Thanks for opening this PR.

In the file pytorch_lightning/core/lightning.py:

Line 453:101: E501 line too long (113 > 100 characters)
Line 465:101: E501 line too long (109 > 100 characters)
Line 517:101: E501 line too long (105 > 100 characters)
Line 530:101: E501 line too long (105 > 100 characters)
Line 533:101: E501 line too long (104 > 100 characters)
Line 580:101: E501 line too long (102 > 100 characters)
Line 590:101: E501 line too long (111 > 100 characters)
Line 591:101: E501 line too long (117 > 100 characters)
Line 677:101: E501 line too long (107 > 100 characters)
Line 689:101: E501 line too long (109 > 100 characters)

Do see the Hitchhiker's guide to code style

Will validation_end be renamed to validation_epoch_end? Validation can be done multiple times per epoch by setting the flag in the trainer, so renaming that would be confusing.

Will validation_end be renamed to validation_epoch_end? Validation can be done multiple times per epoch by setting the flag in the trainer, so renaming that would be confusing.

yes, but validation_epoch_end is called at the end of a validation epoch NOT training epoch


Fix header documentation

Moved tr_info to right place in header documentation

@jupe @tommikas

Fixed

Is 5 commits enough to fix documentation?

I think that should finally be it. At least didn't notice any other instances of this in the two files...


iOS 15.1.1 is only available to iPhone 12 and later, other iPhones only get iOS 15.1 - What to do?

Your Question

Website URL Path: https://support.apple.com/en-us/HT201222
Question: According to the Apple Support page linked to above, iOS 15.1.1 is only available to the iPhone 12 & iPhone 13 series, iOS 15.1 is still the newest iOS for the other phones.
What should we do with https://github.com/corona-warn-app/cwa-website/blob/195d9ef8ec34306a16020f52ff6808016b6b3ad3/gulpfile.js#L426? Leave it on 15.1.1 or set it back to 15.1?

Obsolete with https://github.com/corona-warn-app/cwa-website/pull/2185


Add fetch id token event

I have a requirement that I need add an event before send fetch id token POST request, so that I can handle my UI view, such as showing a loader, because request the /token endpoint would probably take some long time to process.

Not sure what your question or request here is? You have all the regular tools available: http interceptors in Angular, overriding protected methods, subscribing to events in this library. For example, you can track my sample approach for a Loading overlay that seems to do exactly what you ask.

I means when I'm using code flow, there are two steps calls.  After the first call (/authorize endpoint) response back and redirect to the redirection url, angular will reload the component, so that the loader will disappear, at the same time the the second call (/token endpoint) has not completed yet. Then If I have a event before the second call, then I can emit the loader work again.

Ok, I sort off understand. But then one of the mentioned approaches should work, non? Not sure what else to add...

It's working like the following:
in library: oauth-service.ts
public tryLoginCodeFlow(): Promise<void> {
  if (code) {
    this.eventsSubject.next(new OAuthInfoEvent('fetch_token'));
    // fetching token at '/token'
  }
}

in client: authentication.service.ts
authenticating$ = new BehaviorSubject<boolean>(false);
...
this.oauthService.events.pipe(
  map((event: OAuthEvent) => event.type)
).subscribe(type => {
  if (type === 'fetch_token') {
    this.authenticating$.next(true);
  }
  if (type === 'token_received') {
    // route to protected resouce
  }
});

I believe this event is necessary between the two calls, as it's most probably has a long time to retrieve the token back.

Okay yeah I see, you'd like the library to emit an event for in between getting back to the SPA and exchanging the code for a token? That makes sense to me, could help with some types of apps.


Change from PHP 5.6 to PHP 5.5.9

Hello, thank you very much for putting together this package.
Our company is still stuck with PHP 5.5 in our production environments. When I found your package, I could not install it because composer.json dictates PHP 5.6 at minimum.
After inspecting the code I found no reason it wouldn't work on PHP 5.5. I downloaded the zip, added an entry to the autoloader and tested it throughly. It works perfectly.
Since it's so simple, I see no drawback in downgrading the requirements to PHP 5.5. Could you make the minimum requirement to be PHP 5.5.9? (same as Laravel 5.1 LTS)

Agreed. Thanks for the PR.


[FEATURE] Allow hiding LLM/Agent Output from Message History

I am currently building an agent flow, where an LLM is deciding what to do next. The LLM generates JSON for a Condition node, and the Condition will reroute the flow based on the input of the LLM. There is really no need for the output of the LLM to be added to the message history. Not only would it safe tokens to be able to hide the output of the LLM, but the unnecessary messages actually make it harder for following LLMs to 'understand' the context.
I would like to see a simple checkbox "Do not copy to history" on LLMs and Agents. If checked the output is handed over to the next node as usual, but it doesn't appear in the chat history.

if you are using embedded, you can turn it off by showAgentMessages: false: https://github.com/FlowiseAI/FlowiseChatEmbed?tab=readme-ov-file#configuration

@HenryHengZJ  I don't want to hide the output from me, but from the other LLMs down the line in the workflow. I want to decide, which messages are added to the history and which are not.

Or add a node that allows removing messages from the state.
Langgraph has a example here.

This is a very desired feature, any updates on it?
I have similar issue, where state.messages are piling up to a point where agents just get lost. Not to mention the token usage.

I have a similar same issue. I want to clear the conversation history or "disable" appending some output to the list of messages, but I'm not able to achieve that. Any solutions ?

I opened a PR to add a parameter in the Agent Node and LLM Node that disables the insertion of messages from the state:
View PR


feat: support packages for a user

I skipped compiling to dist, let me know if I shall change anything.
Needed to update @actions/github to be able to use the endpoints.
fixes #5

Any updates on this? Having this feature would be very helpful.

Will take a look during the week!

ChatGPT says:
Certainly! Here's an example curl request using the GitHub API to verify if a repository is owned by a user or an organization:
curl -H "Authorization: Bearer YOUR_GITHUB_TOKEN" \
  https://api.github.com/repos/OWNER/REPO \
  | jq '.owner.type'

Replace YOUR_GITHUB_TOKEN with your actual GitHub personal access token, OWNER with the username or organization name that owns the repository, and REPO with the name of the repository you want to check.
The jq command at the end of the request is used to parse the JSON response and extract only the type field of the owner object, which will be either "User" or "Organization". This will allow you to determine if the repository is owned by a user or an organization.
Make sure to include this curl request inside of a GitHub action or workflow file to automate the process!

@vlussenburg Isn't that similar to using ${{ github.repository_owner }} or does it cover ANY owner instead of just the owner of the repo?

@vlussenburg Isn't that similar to using ${{ github.repository_owner }} or does it cover ANY owner instead of just the owner of the repo?
Found this other action that does something similar using Python and it works: https://github.com/chipkent/action-cleanup-package

Good question. Will have to dive in later. Having said that I do like that example

Hey there, yeah the isUser is kind of messy, but I at least wanted to submit a draft to implement this.
Looking up the repository with the owner would be a good idea. Would have to look into how the SDK treats the data here @actions/github.
Could be a step to avoid hardcoding the names in yaml as well, when we would use the vars.


[UPDATE] Removed the secondary MQTT topic.

Short Description:
Fixes a bug where data gets parsed before actually checking if the data is acquired from the correct channel.
Edit: I have also removed the second topic (#pgomapcatch). The map-chat now also uses the #pgo channel to display pokemon on the map.
Fixes/Resolves/Closes (please use correct syntax):

You need change the web as well.

We use an old commit of the web without MQTT right? The map-chat is updated with the 2nd commit.

we should not release counter_id to the web, that's why I use two channel.

@solderzzc both channels send out the encounter_id. The only thing the JSON sends that the other channel does not, is the spawn_point_id. (Which we can remove from the JSON if needed).

@Jasperrr91 he did not say it does not currently send out encounter_id, he's saying it shouldn't - and I will fix that on the account of this.
We also do not need to broadcast to a second channel, it's fine how it is though what could be done is a user controlled variable to set broadcasting to a different broker.

Don't close this then lol. This can be merged without problem. As this removes the second channel and adds stability as it checks for the correct channel before parsing.

Please open a clean PR

What's not clean about it?

The channels do not reflect channels officially sponsored by default by the project, please see my post above for more information.

Yes I have read that. What does this have to do with this PR? It improves the current code as is.
If you want me to make the broker/channel variable. I'll do that in a sec. But this is fine as is.

Again, this channel "pgo" is not officially sponsored by the project - we do not use it. If you want to push this through I kindly ask that you make a clean PR without use of these channels.

@ro can you please tell me which channel is officially sponsored by the project then?
As the current dev codebase does in fact use "pgo":
self.mqttc.publish("pgomapcatch/all/catchable/"+str(data['pokemon_id']), str(data['latitude'])+","+str(data['longitude'])+","+str(data['encounter_id'])+","+str(data['pokemon_id'])+","+str(data['expiration_timestamp_ms'])+","+str(data['pokemon_name']))
json_data = json.dumps(data)
self.mqttc.publish("pgo/all/catchable/"+str(data['pokemon_id']), json_data)

Even more so, it is the channel used by the bot to receive pokemon: self._mqttc.subscribe("pgo/#", 0)

I stand corrected, must of changed very very recently, as a matter of hours. This was not the case.
@solderzzc both channels send out the encounter_id. The only thing the JSON sends that the other channel does not, is the spawn_point_id. (Which we can remove from the JSON if needed).
No, both channels do NOT send out the encounter_id.

            self.mqttc.publish("pgomapcatch/all/catchable/"+str(data['pokemon_id']), str(data['latitude'])+","+str(data['longitude'])+","+str(data['encounter_id'])+","+str(data['pokemon_id'])+","+str(data['expiration_timestamp_ms'])+","+str(data['pokemon_name']))

            json_data = json.dumps(data)
            self.mqttc.publish("pgo/all/catchable/"+str(data['pokemon_id']), json_data)

No problem. They do both send out the encounter_id 😛
First publish sends out data['encounter_id']. Then the entire data object is converted to JSON and then published to the second channel.

I'm not getting that in code, but I've been wrong in this PR before lol so please, if you can show me where the encounter_id is sent through both topics/channels I'd appreciate it.

self.mqttc.publish("pgomapcatch/all/catchable/"+str(data['pokemon_id']), str(data['latitude'])+","+str(data['longitude'])+","+str(data['encounter_id'])+","+str(data['pokemon_id'])+","+str(data['expiration_timestamp_ms'])+","+str(data['pokemon_name']))
Encounter_id in the data object is sent to the pgomapcatch/all/catchable/ channel.

json_data = json.dumps(data)
self.mqttc.publish("pgo/all/catchable/"+str(data['pokemon_id']), json_data)
Entire data object (including the encounter_id contained in it) is sent to the pgo/dall/catchable/ channel.

I'm wondering you why removed the # that indicates these lines are commented out and not compiled?

I did no such thing. Haven't changed any python comments.

hahaha
On Sat, Aug 27, 2016 at 12:27 AM, Jasperrr91<EMAIL_ADDRESS>wrote:

I did no such thing. Haven't changed any python comments.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/PokemonGoF/PokemonGo-Bot/pull/4771#issuecomment-242783160,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUR4o6pN4IMVHX-Rj6j1JvySAkCGLKXoks5qjxPfgaJpZM4JuH07
.

@Jasperrr91 you agree that since only the pokemon_id is extracted from the array, that no - the entire object is NOT sent to mqtt? If this is true, then we can also agree that again, no, the encounter_id is NOT sent to both channels.
The entire array is NOT sent.

@BreezeRo no I don't agree. The pokemon_id is extracted from the array to construct the topic. Then, after the , the json_data is sent. And the json_data contains the same attributes as the  data object.
So yes, the entire array is indeed sent.

I stand corrected, the 3 lines you pasted above are actually one line that resemble the previous two (on dev branch atleast) which are in fact commented out.
Sue me :)
Will merge after encounter_id + other ups are cleared

As proof, I've done a console.log of both channels.  For #pgo the console log is as follows:
var pokemon = JSON.parse(payload);
console.log(pokemon);

For #pgomapcatch it's as follows:
var pLoadR = payload.toString();
console.log(pLoadR);

Result:

The slight discrepancy between the encounter_ids for each channel, is caused by float -> integer correction:

Which goes back to what I said in my previous post
"he did not say it does not currently send out encounter_id, he's saying it shouldn't"

Yes, which brings me back to a point I mentioned before:

What does this have to do with this PR? It improves the current code as is.

My code is an improvement of the current codebase. I can remove the encounter_id through a future - seperate - PR if you want but you already mentioned before that you would do it.

I really don't understand the problem here :hand: , I already said
"Will merge after encounter_id + other ups are cleared"

Yea and those are cleared up aren't they? 😜

I was talking about the parts I need to do

data.pop('encounter_id', None)

We don't merge two channels into one :)
Close at the time.


Compression in saved workspace

Is the exported workspace compressed in the current version?

No, it's currently just encoded with msgpack

Thanks for confirming that, we can close this issue.
On Mon, Apr 26, 2021 at 10:56 AM Nick Hu @.***> wrote:

No, it's currently just encoded with msgpack
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/homotopy-io/homotopy-rs/issues/56#issuecomment-826694221,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACQ4OHQ4XPSQNJZDB6CEYW3TKU2FLANCNFSM43QSNK2A
.


"Ștefanca", "Tolăcești" are listed in the drop-down menu

Description:
When user writes "Ștefanca" and "Tolăcești" the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Ștefanca"

Expected results
The location appears in the drop-down menu as "Ștefanca, Alba (Bistra)"

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Tolăcești"

Expected results
The location appears in the drop-down menu as "Tolăcești, Alba (Bistra)"


KOGITO-9389: Add dashboards to operator deployment  on the cli (dashbuilder support)

On this PR:
Serverless workflow projects containing dashboards, will be deployed on the operator:
├── application.properties
├── dashboards
│   ├── age2.dash.yml
│   └── products.dash.yaml
├── specs
│   ├── workflow-service-openapi.json
│   └── workflow-service-schema.json
├── workflow-minimal.svg
└── workflow-service.sw.json
./dashboards/ is the default directory for dashboards being loaded on dev UI:

@domhanak I'm merging this because it blocks other PRs. If you want to suggest any change just let me know and I'll address it in a new PR.


Fix peer dependency issues

Using this module results in several warnings regarding peer dependencies:
warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "@cfaester/enzyme-adapter-react-18 ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>
This is due to an unnecessarily strict peer dependency range ^18.2.0 - The PR replaces that with simply >=18.
Also directly related to it, the two dependencies react-is and react-test-renderer have been moved to peer dependencies as well. It is not really recommended to list a specific version of these packages as a direct dependency (unless that specific version is required). Instead, it is better to leave it up to the integrating project to supply a compatible version of the necessary module.
In a separate commit, a few @types packages were added to dev dependencies (this could also be moved to a separate PR if preferred, please let me know).

@friederbluemle Hey Frieder. Thanks for taking the time to open a PR. I'll run a few internal tests, and if it looks good submit a version next week 🙂

Hi @cfaester - Just checking, have you had a chance to run the tests? Any questions/issues?

@cfaester ping

Hey everyone. I just wanted to chime in. I've been away for holidays for a while, and more importantly away from a comprehensive test-solution for a while. I will push a "next" version with these fixes, but bear in mind, they are untested.

Hey everyone. I just wanted to chime in. I've been away for holidays for a while, and more importantly away from a comprehensive test-solution for a while. I will push a "next" version with these fixes, but bear in mind, they are untested.

Live at<EMAIL_ADDRESS>@tmc friederbluemle

Alright guys, I am including this in 0.8.0

Thank you @cfaester :+1:

I saw the 0.8.0 version is already published to npmjs.com - It would be nice to also create and entry for it on GitHub Releases, along with a brief explanation of differences (changelog). Thanks again!


jwplatform.py is broken with  https://github.com/joedborg/youtube-dl/pull/49

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2019.04.07. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2019.04.07
youtube-dl --version
2019.04.07

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] User config: []

[debug] Custom config: []

[debug] Command-line args: [u'--verbose', u'--prefer-ffmpeg', u'--format', u'bestaudio', u'--extract-audio', u'--audio-format', u'opus', u'--audio-quality', u'0', u'--keep-video', u'--output', u'\u0421\u0432\u043e\u0439 \u0441\u043e\u0431\u0441\u0442\u0432\u0435\u043d\u043d\u044b\u0439 \u043f\u0443\u0442\u044c (\u0414\u0435\u0432\u0438\u0434 \u0414\u043e\u0440\u043d\u0430\u043d).f%(format_id)s.%(ext)s', u'https://content.jwplatform.com/manifests/NdSc5Qlg.m3u8?exp=1554635121']

[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8

[debug] youtube-dl version 2019.04.07

[debug] Git HEAD: a46d9e5

[debug] Python version 2.7.16 (CPython) - Darwin-15.6.0-x86_64-i386-64bit

[debug] exe versions: avconv 12.3, avprobe 12.3, ffmpeg 4.0, ffprobe 4.0

[debug] Proxy map: {}

[generic] NdSc5Qlg: Requesting header

WARNING: Could not send HEAD request to https://content.jwplatform.com/manifests/NdSc5Qlg.m3u8?exp=1554635121: HTTP Error 403: Forbidden

[generic] NdSc5Qlg: Downloading webpage

ERROR: Unable to download webpage: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see https://yt-dl.org/update on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

File "/Users/pavel.shevtsiv/IdeaProjects/youtube-dl/youtube_dl/extractor/common.py", line 626, in _request_webpage

return self._downloader.urlopen(url_or_request)

File "/Users/pavel.shevtsiv/IdeaProjects/youtube-dl/youtube_dl/YoutubeDL.py", line 2227, in urlopen

return self._opener.open(req, timeout=self._socket_timeout)

File "/anaconda2/lib/python2.7/urllib2.py", line 435, in open

response = meth(req, response)

File "/anaconda2/lib/python2.7/urllib2.py", line 548, in http_response

'http', request, response, code, msg, hdrs)

File "/anaconda2/lib/python2.7/urllib2.py", line 473, in error

return self._call_chain(*args)

File "/anaconda2/lib/python2.7/urllib2.py", line 407, in _call_chain

result = func(*args)

File "/anaconda2/lib/python2.7/urllib2.py", line 556, in http_error_default

raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Playlist:  https://content.jwplatform.com/manifests/NdSc5Qlg.m3u8?exp=1554638831&sig=b7f01a3ce1e275dffe6ff74599f02ac5

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
jwplatform.py is broken with
https://github.com/joedborg/youtube-dl/pull/49
locally checked
git checkout b0ead0e youtube_dl/extractor/jwplatform.py
[debug] Command-line args: [u'--verbose', u'--prefer-ffmpeg', u'--format', u'bestaudio', u'--extractaudio', u'--audio-format', u'opus', u'--audio-quality', u'0', u'--keep-video', u'--output',
u'⧵u0421⧵u0432⧵u043e⧵u0439 ⧵u0441⧵u043e⧵u0431⧵u0441⧵u0442⧵u0432⧵u0435⧵u043d⧵u043d⧵
u044b⧵u0439 ⧵u043f⧵u0443⧵u0442⧵u044c (⧵u0414⧵u0435⧵u0432⧵u0438⧵u0434
⧵u0414⧵u043e⧵u0440⧵u043d⧵u0430⧵u043d).f%(format_id)s.%(ext)s', u'https://content.jwplatform.
com/manifests/NdSc5Qlg.m3u8?exp=1554638831&sig=b7f01a3ce1e275dffe6ff74599f02ac5']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.04.07
[debug] Git HEAD: a46d9e5
[debug] Python version 2.7.16 (CPython) - Darwin-15.6.0-x86_64-i386-64bit
[debug] exe versions: avconv 12.3, avprobe 12.3, ffmpeg 4.0, ffprobe 4.0
[debug] Proxy map: {}
[generic] NdSc5Qlg: Requesting header
[generic] NdSc5Qlg: Downloading m3u8 information
[debug] Invoking downloader on u'https://jwpsrvvh.akamaihd.net/i/content/conversions/
wdjHsEgT/videos/NdSc5Qlg1174958.m4a/index_0_av.m3u8?hdnts=exp=1554645949~acl=/
NdSc5Qlg1174958.m4a/~hmac=f21b00021afd9b239203835df49484
9caf1e458daeb9c4b7fc4fc90a17b0a748'
So it is working for previous comm
command to run
python -m youtube_dl --verbose --prefer-ffmpeg --format 'bestaudio' --extract-audio --audio-format "opus"
--audio-quality 0 --keep-video --output 'Свой собственный путь (Девид Дорнан).f%(format_id)s.%(ext)s'
https://content.jwplatform.com/manifests/NdSc5Qlg.m3u8?exp=1554638831\&sig=
b7f01a3ce1e275dffe6ff74599f02ac5

Does not work in browser either.

Hi, @dstftw . This issue is valid. @remitamine knows all details and has fixed the issue. I've just checked this version 9c01725 . Now it is working as expected.


BUG - dataset loader

I tried running the following code as described in the tutorial:
dataset = load_dataset('unitxt/data', 'card=cards.wnli')
However, I encountered an assertion error:
AssertionError: Specify either template or template_card_index in card
Here is the link to the tutorial I followed: Unitxt Documentation.
Could you please advise on how to resolve this error?

You need to choose a template. We will either fix the documentation (cc: @michal-jacovi ) or set a default behaviour. Thanks for noting.

Hi. I think you are were looking at an old documentation page:
https://www.unitxt.ai/en/1.7.6/docs/no_installation_usage.html
latest documentation is up to date:
https://www.unitxt.ai/en/latest/docs/loading_datasets.html

Thanks for pointing that out.
I was probably looking at an old page due to cookies, I'll check the latest documentation.


Activity

(:
Result:

Reference:

Hi @thomaspink, could you please review this feature?
Contact me for anything. Thanks!
👍


Conflicts with latest Docker refactor and documentation

The latest round of refactoring on the Docker files has meant that all of the existing documentation for how to run Qlever with docker is out of date.
Some examples:

Previously the documentation specified "/index", but now the script seems to fail if the indexes are not found in "/data"? And this seems to be hardcoded? https://github.com/ad-freiburg/qlever/blob/master/docker-entrypoint.sh#L40
The instructions for how to set users are not at all clear. There is a comment in the docker entrypoint script that says the use of -u shouldn't be used, but then several examples that do use it.
There are no instructions on the expected form of the host system or how docker is executed. Should the docker command be run as root? Or should there be a qlever user on the host system? What privileges should this user have?

After working through these I am trying to run it with the following command, and it still is not working. It's failing with a usermod: no changes error, and just keeps restarting. The basics of this command worked prior to the latest Dockerfile update, but I've made changes to the -v argument (to point it to "/data"), added the -w argument, and the two -e arguments for UID / GID, after unsuccessfully trying to use -u (until I saw the note in the bash file).
docker run -d --restart unless-stopped -v /opt/qlever/qlever-indices/myindex:/data -w /data -e UID=$(id -u qlever) -e GID=$(id -g qlever) -p 7001:7001 -e INDEX_PREFIX=myindex -e MEMORY_FOR_QUERIES=4GB -e CACHE_MAX_SIZE_GB=30GB -e CACHE_MAX_SIZE_GB_SINGLE_ENTRY=5GB --name qlever.myindex adfreiburg/qlever
I would be grateful for any help in updating the command to work with the latest changes!

Looking through #1439  perhaps also @ludovicm67 can help understand the new setup?
If anyone else is currently struggling, for now I have reverted to the last Docker release before the change:
docker pull adfreiburg/qlever:commit-44e2ba8
I think I understand that now the ENTRYPOINT is no longer set to automatically start the server, but to run the docker entrypoint script, and then we have to now use the qlever script to start the server, i.e., `[docker command] -c "qlever start ..."
However, this gives me an error,
Invoking command `start` without a Qleverfile. You have to specify all required arguments on the command line. This is possible, but not recommended.

Command: start

docker run -d --restart=unless-stopped -u $(id -u):$(id -g) -v /etc/localtime:/etc/localtime:ro -v $(pwd):/index -p None:None -w /index --init --entrypoint bash --name qlever.server.rism docker.io/adfreiburg/qlever -c 'ServerMain -i rism -j 8 -p None -m 5G -c 2G -e 1G -k 200 -s 30s > rism.server-log.txt 2>&1'

So it looks like the qlever start command is trying to start docker within docker?

@ahankinson Sorry to hear about those problems + I have a few questions:

You mention that "previously the documentation specified". Which documentation are you referring to? For over two years now, the QLever README.md says: "Use QLever via the qlever script, following the instructions on https://github.com/ad-freiburg/qlever-control ."

The qlever script (which you can install via the respective repo or even simpler via pip install qlever) should work with the new Dockerfile just as it worked with the old Dockerfile.

You have posted a Docker command line. Where does that command line come from? There is no scenario where you should have to write such a command line yourself.

If you don't want the goodies of the new entrypoint script, you can simply call docker with --entrypoint bash and then do whatever you want in the container, including calling the QLever binaries IndexBuilderMain and ServerMain directly (which you shouldn't, but you can).

Overall, my impression is that you have been using QLever in a deprecated way for a long time and that deprecated way was broken by the recent changes to the Dockerfile.

Hi @hannahbast , and thanks for you response!
I was looking at this documentation, particularly the "Start the engine" section: https://github.com/ad-freiburg/qlever/blob/master/docs/quickstart.md
With the old Dockerfile I didn't need to use the qlever command, since the old entrypoint in the Dockerfile ran the server directly.
https://github.com/ad-freiburg/qlever/blob/80938b74f8a7e4919abd479a357bbe759896363f/Dockerfile#L51
I pasted a couple docker command lines. The one in the original post comes from the Quickstart documentation above (with some modification and re-ordering).
The docker command in in my second message  comes from the qlever script in the new setup, when I try to use the new setup. I'm obviously not doing something right.
It's not that I don't want the new "goodies" -- far from it! I like to keep our installation up-to-date to take advantage of the new developments. But this particular change could use a bit more clarification on the setup and state of the machine.
I didn't think I was using it in a deprecated way... There's even a "quickstart.md.DEPRECATED" file which I was not using... 😁


[FEATURE REQ] Investigate whether spring-security-ldap can work together with AAD

Refs:

https://docs.microsoft.com/en-us/azure/active-directory/fundamentals/auth-ldap
https://github.com/spring-projects/spring-security-samples/tree/1ad87dcaf5208837a9a2c24d65d5fb5ec87d5f38/servlet/spring-boot/java/ldap

Closing this issue. Use this one instead: https://github.com/Azure-Samples/spring-boot-application-with-azure-active-directory/issues/5


Fix a grammar mistake

@RedasP : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Issue with Hour Field Not Triggering in CronJob Registration

Hello, I am a developer using PocketBase in Korea. I am currently developing an extension in JavaScript to register CronJobs with fields for minute, hour, day, month, and week. I have confirmed that the minute and week fields are working correctly through tests, but the hour field does not seem to operate correctly even though I have input the correct numbers for the specific hours.
Pocketbase v0.22.4

I'm not able to reproduce it locally with the latest v0.22.9 and it is working correctly for me.
If you are still not able to resolve it, then please elaborate a little more by providing minimal code sample/reproducible steps and I'll try to investigate it further.
If you have other questions, please use the Q&A discussions.


Corrupted secure-properties-tool.jar file

Hi Team,
This document https://docs.mulesoft.com/mule-runtime/4.3/secure-configuration-properties leads the users to a corrupted version of secure-properties-tool.jar file.
** Corrupted. If you try to execute this one, you will get "no main manifest attribute".
https://docs.mulesoft.com/downloads/mule-runtime/4.2/secure-properties-tool.jar
** Good, but older, version.
https://docs.mulesoft.com/downloads/mule-runtime/4.1/secure-properties-tool.jar
Best Regards,

@Cristian-Venticinque https://www.mulesoft.org/jira/browse/DOCS-8602
GitHub Issue #1557: Corrupted secure-properties-tool.jar file

Closing this PR because the tool is working as intended. Created DOCS-8603 to clarify that you need to use the same syntax that is shown in the docs version from where you downloaded the tool, using v4.1 syntax with v4.2 and later secure-properties-tool.jar files causes that error.


DribbbleShot#expandImageAndFinish() and transition animation issue

I noticed that the transition animation was off when the ListView is scrolled up. Specifically, when the commentList is scrolled to bottom and I finish the activity, the ImageView seemed to expand to the original position, and it was transition back to a wrong position in the RecyclerView, where the image is cut off, and eventually the image jumped back to the correct position. An example is the gif below:

I checked the code in DribbbleShot#expandImageAndFinish(), the animation brings the ImageView back to offset = 0, but that doesn't seem to be enough to adjust the transition animation.
To work around it, I tried scrolling the commentList back to position 0 before finishing the activity, and it is working quite well. All I did was using commentsList.smoothScrollToPosition(0); instead of the ObjectAnimation. To make things smoother, you can also add a slight delay on calling finishAfterTransition().

Fixed by f98b8769f2d447bc40bf6ce7b1531ebf10da795f


[Enhancement] Push more predicates down to the analytic node

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #16348
Problem Summary(Required) ：

Predicate which contains sort columns and window function columns can not push down, otherwise we could push the predicates down to the analytic
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

[FE PR Coverage Check]
:heart_eyes: pass : 9 / 9 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/PushDownPredicateWindowRule.java
9
9
100.00%
[]


Bump ms to v2 due a ReDoS vuln

<EMAIL_ADDRESS>is vulnerable to a ReDoS attack: https://snyk.io/vuln/npm:ms:20170412
Looking at the code, the breaking change in<EMAIL_ADDRESS>that the accepted string is now limited to 100 chars.

Thanks @dominykas!


Errors after upgrade from V3 to V4

Expected Behavior
Website running correctly
Current Behavior
$ hugo server -w
Watching for changes in /Users/serverfriendsorg/projects/gitlab.com/serverfriends_org/ws_serverfriendsorg/{assets,content,data,i18n,layouts,node_modules,package.hugo.json,package.json}
Watching for config changes in /Users/serverfriendsorg/projects/gitlab.com/serverfriends_org/ws_serverfriendsorg/hugo.yaml, /Users/serverfriendsorg/projects/gitlab.com/serverfriends_org/ws_serverfriendsorg/go.mod
Start building sites …
hugo v0.126.1+extended darwin/arm64 BuildDate=2024-05-15T10:42:34Z VendorInfo=brew

Built in 958 ms
Error: error building site: render: failed to render pages: render of "home" failed: "/Users/serverfriendsorg/Library/Caches/hugo_cache/modules/filecache/modules/pkg/mod/github.com/hugo-toha/toha/v4@v4.5.0/layouts/index.html:52:18": execute of template failed: template: index.html:52:18: executing "index.html" at <partial .section.template .>: error calling partial: "/Users/serverfriendsorg/Library/Caches/hugo_cache/modules/filecache/modules/pkg/mod/github.com/hugo-toha/toha/v4@v4.5.0/layouts/partials/sections/experiences.html:21:12": execute of template failed: template: partials/sections/experiences.html:21:12: executing "partials/sections/experiences.html" at <partial "sections/experiences/positions.html" $experience>: error calling partial: "/Users/serverfriendsorg/Library/Caches/hugo_cache/modules/filecache/modules/pkg/mod/github.com/hugo-toha/toha/v4@v4.5.0/layouts/partials/sections/experiences/positions.html:14:37": execute of template failed: template: partials/sections/experiences/positions.html:14:37: executing "partials/sections/

Possible Solution

Steps to Reproduce

Screenshots

Browsers Affected

[ ] Chrome
[ ] Firefox
[ ] Edge
[ ] Safari 11
[ ] Safari 10
[ ] IE 11

Detailed Description

Hugo Version: hugo v0.126.1+extended darwin/arm64 BuildDate=2024-05-15T10:42:34Z VendorInfo=brew
node/npm: v22.2.0/10.7.0
Go: version go1.22.3 darwin/arm64

Thanks in advance for looking into this.

There are some breaking changes between v3 and v4. Please, follow this migration guide.
https://toha-guides.netlify.app/posts/update-v3-to-v4/

Thanks @hossainemruz, that worked!
I had initialized the wrong project, so I ran instead of hugo mod init  the theme path github.com/hugo-toha/toha/v4


PushData only pushed to active followers.

If a follower is lagging too far, do not flood it with data from new IOs (new rreq, new LSNs) ,  reserve the capability for catching up,  that follower can request data via FetchData.

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

Project coverage is 67.35%. Comparing base (1a0cef8) to head (07082de).
Report is 87 commits behind head on master.

Files with missing lines
Patch %
Lines

src/lib/replication/repl_dev/raft_repl_dev.cpp
80.00%
2 Missing and 3 partials :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #584       +/-   ##
===========================================
+ Coverage   56.51%   67.35%   +10.83%
===========================================
  Files         108      109        +1
  Lines       10300    10702      +402
  Branches     1402     1466       +64
===========================================
+ Hits         5821     7208     +1387
+ Misses       3894     2799     -1095
- Partials      585      695      +110

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


connect callback not called

Simple example as follows:
require('fs').readFile('/etc/mysql/mysql-ssl-ca-cert.pem','utf8',function(err,caFile){
    if(!err){
        var conn = require('mysql').createConnection({
            host     : 'localhost',
            user     : 'mysql',
            password : 'password',
            database : 'mydb',
            ssl      : { ca : caFile }
        });
        console.log('before');
        conn.connect(function(err){
            console.log('inside');
            if(!err){
                connection.query('select count(*) as found from `codes` where `id` = ?', ['aaa111a1'], function(err, rows, fields) {
                    if (!err) {
                        console.log(rows[0].found?'found':'not found');
                        connection.end(function(err) { if(err) console.error('Error On DB Close.'); });
                    }else{
                        console.log(err);
                    }
                });
            }else{
                console.log(err);
            }
        });
        console.log('after');
    }else{
        console.log(err);
    }
});

the output is simply:
before
after

The connection worked up until I updated to node 4.1.0

still busted on 4.2.1

Hi @CamJN , I don't think it was expected for this to be fixed in 4.2.1 :) It's really bad that Node.js 4.1.0 would break something that worked in 4.0.0.

Have you filed a bug with Node.js? Have you been able to dig in?
I tried the repo you gave above on 4.2.1 just now on Windows 7 x64 and MySQL 5.6.27 and it worked fine over SSL. Is there a possibility you could offer up a PR?
Does the mysql2 module work?

I've not had the time to dig into this much. And I wouldn't begin to know what node is doing that is breaking things for this module.
with mysql2 i get:
(node) crypto.createCredentials is deprecated. Use tls.createSecureContext instead.
events.js:141
      throw er; // Unhandled 'error' event
      ^

Error:<PHONE_NUMBER>72384:error:14077410:SSL routines:SSL23_GET_SERVER_HELLO:sslv3 alert handshake failure:../deps/openssl/openssl/ssl/s23_clnt.c:769:

    at Error (native)
    at SlabBuffer.use (_tls_legacy.js:42:18)
    at CleartextStream.read [as _read] (_tls_legacy.js:260:29)
    at CleartextStream.Readable.read (_stream_readable.js:328:10)
    at EncryptedStream.write [as _write] (_tls_legacy.js:174:25)
    at doWrite (_stream_writable.js:292:12)
    at writeOrBuffer (_stream_writable.js:278:5)
    at EncryptedStream.Writable.write (_stream_writable.js:207:11)
    at Socket.ondata (_stream_readable.js:528:20)
    at emitOne (events.js:77:13)

I'm on node 4.2.1 and mysql 5.6.27 both from homebrew on OS X.

Gotcha. The weirdest part is, as far as I know, you're the only user reporting this issue at all. Has all your testing been on the same computer or different ones? I'm starting to wonder if there is something up with the machine (or a configuration on that machine) you are using if the issue is isolated to the machines.

The same thing happens when I try on my other box. So if it's a problem on my end it's reproducible.

I install mysql, node, and openssl from homebrew. (latest stable versions, as binaries from bottles)
I create a set of self signed certs / keys / ca-cert with openssl. (https://github.com/CamJN/dot-files/blob/master/etc/mysql/mysql-certs.sh & https://github.com/CamJN/dot-files/blob/master/etc/openssl/openssl.cnf)
I setup mysql via my.cnf to use the self signed cert to encrypt traffic. (https://github.com/CamJN/dot-files/blob/master/etc/mysql/my.cnf)
I create my db in mysql, then populate it.
I require ssl to connect to the db. (via GRANT USAGE ON *.* TO 'root'@'localhost' REQUIRE SSL )
I point node at my ca cert when it tries to connect to the db. (as above)
The connection callback is never called.

Gotcha. Tests with SSL connections are running on Travis CI environment, and they are all successful, even on Node.js 4.2.1. I don't personally have any access to a Mac OS X machine, so unless you're able to rule out it's a Mac OS X issue (by confining it all to Windows of Linux), then we can't rule that out, either, which would also put it out of my reach to ever be able to reproduce & debug.

So travis supports OS X as a CI env. I don't know if using more than two envs costs money on travis but if not I could likely port your linux scripts/config for you if you like.

AFAIK Node.js is not supported on Max OS X on Travis CI (according to https://github.com/travis-ci/travis-ci/issues/2311).

Yeah I'd have to get node.js from homebrew, but since that's how many/most os x developers will get it that's probably a good thing.

Gotcha. It also looks like MySQL is also not a service on the Travis CI Mac OS X, so I suppose you have to get that from homebrew as well. The test suite for this module already takes a while, so hopefully installing all these things doesn't make the tests take a really long time.

Otherwise, yes, especially if you can setup Travis CI to test on Mac OS X and it reproduces this issue, though I'm not sure how much that will even help me, because I cannot attach a debugger to the Node.js that runs in Travis CI to debug anything about the issue...

Ok. I'd make you a VirtualBox VM but technically you are only supposed to run OS X on apple hardware so depending on how you feel about EULAs that might not be of much interest to you.

Let's at least see if it can reproduce on Travis CI first.

Well I couldn't get the CI to work on OSX at a reasonable speed (also lots of random npm errors). But a combination of updates to mysql 5.7.9 and node 5.1 fixed the issue.

Interesting. If you experience it again or are able to determine what the root cause was, we would love to hear it, especially if there is some version of MySQL we are not compatible with (perhaps we an fix it or at least document it somewhere).

I had an idea, when I upgraded to mysql 5.7.9, I had to stop using CAMELLIA128-SHA as my cipher which I had been using previously per (http://www.couyon.net/blog/if-all-of-your-mysql-ssl-clients-just-broke), so that might be related.


Register CloudEvent types supported by AWSCodeCommit source

By exposing the event types we support in the source's status, the Eventing duck controller automatically registers those event types in the cluster.
$ kubectl -n dev get eventtypes.eventing.knative.dev
NAME                               TYPE                                 SOURCE
        SCHEMA   BROKER    DESCRIPTION   READY   REASON
3b925f5e33af6ed5f076188abd5b0e3e   com.amazon.codecommit.pull_request   https://git-codecommit.eu-central-1.amazonaws.com/v1/repos/source-test            default                 True
f925f2b47f8fecf95ee7639cdc31564f   com.amazon.codecommit.commit         https://git-codecommit.eu-central-1.amazonaws.com/v1/repos/source-test            default                 True

Closes #63

very nice to get the event type in there


In Navbar Logo redirect to Error 404

I want to change redirection of Logo when we clicked it that's redirect to 404 error page.
please assign me this task @MonalikaPatnaik.

its been resolved!
Thanks!


[WIP] Make support multiple groups of default streams

This Pull Request aims at solving issue #6693 allowing easily adding multiple groups for default streams and an carry forward to the work of @hackerkid in the pr #6973. Using the roadmap provided by @timabbott , we have now been able to reach these checkpoints.

[X] Add a DefaultStreamGroup table with name and ID columns.
Thanks to @hackerkid  for #6973 most of the work important work has been done. 😃
[X] Change DefaultStream to have an additional DefaultStreamGroup column.
I have successfully written a migration that creates a default group that contains all the default streams
now present.
[ ] And we'd need to move the unique_together to be on DefaultStreamGroup, Stream, not Realm, Stream).
[ ] Modify the registration process to, if the realm has a non-null DefaultStreamGroup, offer a page like the above in the signup process.
Though Presently there are options shown on time of registration, but this thing is not implemented correctly yet.
[ ] Expand the default stream API endpoints to support adding/removing these.
[ ] Expand the UI for default streams to support these.

Closing as stale and failing tests.


"Changing store on the fly" error when embedding  in a stateless component

I'm working on a large app where we're slowly integrating Redux piecemeal, essentially converting small components into "embedded Redux apps" that are the top-level of the Redux app, but not the top-level of the React app. This has worked for the most part, but we ran into this issue:
var ReduxRoot = props => (
    <Provider store={props}>
        <ReactReduxConnectedComponent/>
    </Provider>
)

It works fine, but any time the parent component re-renders, a <Provider> does not support changingstoreon the fly error pops up in the console.
This is fixed by adding a shouldComponentUpdate, and returning false if this.props and nextProps are equal. But that prevents us from taking advantage of the stateless component pipeline.
I know it's not the idealized use of Redux to embed it within a React hierarchy instead of at the top level, so I'm not sure if this issue is considered worth addressing. But maybe it is.

I know it's not the idealized use of Redux to embed it within a React hierarchy instead of at the top level, so I'm not sure if this issue is considered worth addressing. But maybe it is.

It's totally fine, we should support this.

It works fine, but any time the parent component re-renders, a  does not support changingstoreon the fly error pops up in the console.

Please see the relevant check. It's comparing identities. This means that the store prop is different on every render. This is problematic. (You're effectively recreating store and destroying its state—why?)
This is also a bit weird:
var ReduxRoot = props => (
    <Provider store={props}>
        <ReactReduxConnectedComponent/>
    </Provider>
)

Shouldn't that be more like
var ReduxRoot = props => (
    <Provider store={props.store}>
        <ReactReduxConnectedComponent/>
    </Provider>
)

?

This is also a bit weird:

Aha, that was the problem. Since the initial state of the store was the only data I needed to pass to ReduxRoot, it was basically:
<ReduxRoot {...initialStore}/>

So that meant props, even though it always had the same contents, was actually a new object being passed to <Provider>.
I fixed the problem by using your correction, except with destructuring
var ReduxRoot = ({store}) => {

for an ever-so-satisfying elimination of six characters.
I guess one way to avoid this error is if <Provider> could check to see if the store it's being passed is actually identical to its previous store, but that might add some unnecessary overhead.

Glad you found the problem! Yeah, we only rely on object identity there, something more complicated seems unwarranted.

Sorry for being a bit off topic but I'm looking for a code example that demonstrates switching between different stores in react-native. This thread seems to touch the subject.
Specifically, I want to switch between users (log in/out) where each user has its own store.
Could you point me to any articles/tutorials available online.
Thanks a lot!

spot on, dan!

Sorry too for being off topic but I'm having the same issue as @beebase. I want to change between stores in a login/logout logic in react-native, and I'm not sure how can that be done. Have you found a solution @beebase ? Thanks!

You can't change the store passed to <Provider>, but you could render a new <Provider>. Give it a key and change it when you change the store.

"This means that the store prop is different on every render. This is problematic. (You're effectively recreating store and destroying its state—why?)"
I don't think we should change store  - changing store means losing current store state

interesting...
never encountered a case where I need to create 2 stores and switch between them..
can't we put the switch inside store, and read different part from store state?  nevertheless, I believe you have your own ground to have 2 stores

Throwing in an answer for those who searched for the Error message in the title: Don't call createStore in the provider attribute!
Don't do this:
ReactDOM.render(
 <Provider store={createStore(reducers)}>
   <App/>
 </Provider>,
 document.getElementById('root')
);

The correct way is this:
var store = createStore(reducers);
ReactDOM.render(
 <Provider store={store}>
   <App/>
 </Provider>,
 document.getElementById('root')
);

hey all, i have another problem relating to this but figure i shouldn't post it here. it's posted on stackOverflow

I had the same problem as given in the title but after the advise of @totymedli, i solved my problem. I am using react-native my approach for the solution was as advised;
const store = createStore(reducers);
export default class App extends React.Component {
render() {
return (
    <Provider store={store}>
      <View style={{ flex: 1 }}>
        <Header title="Tech Stack" />
        <LibraryList />
      </View>
    </Provider>

);

}
}

I solved like this;
Instead of export default configureStore()
configureStore(...);

export default configureStore();

Had the same problem, solution was so simple..as @carslan  said all you need is to define the store outside of your class component and pass it to store props of .
Hope it helps...
**const store = createStore(reducers);
export default class App extends React.Component {
render() {
return (
    <Provider store={store}>
      <View style={{ flex: 1 }}>
        <Header title="Tech Stack" />
        <LibraryList />
      </View>
    </Provider>

);

}
}**

@carslan a stophen grider student huh?..

Here is my solution:
// index.tsx
import createStore from './redux/createStore'
const store = createStore({})

const renderApp = () => {
  ReactDOM.render(
    <MuiThemeProvider theme={theme}>
      <Provider store={store}>
        <Router>
          <App />
        </Router>
      </Provider>
    </MuiThemeProvider>,
    document.getElementById('root'),
  )
}
renderApp()

if (module.hot) {
  // only need replace entry point
  module.hot.accept('./App', () => {
    renderApp()
  })
}

The one important thing is that should not replace the file which create the redux store. In my above example is the index.tsx.


Add IFP Innovacion en Formacion Profesional

Add "IFP - Innovacion en Formacion Profesional" to apply for license.
Official URL: https://www.ifp.es/
Vocational Education and Training in "Web applications development".
domain teachers: @campus.ifp.es
domain students: @alumnoifp.es

@MiaMohn I am sorry but the domain alumnoifp.es is listed among hacked domains in the internet. Also, the domain is not mentioned at the official website of the school.
Please apply for free educational license with an official document which proves that you are a student. You can do it using "OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students.
I have to close the request without merging.
Thank you for your understanding!


Add DepthError for mapping error depth, and comment out dummy gadget for feature scroll

Description

Add DepthError for mapping error depth to call and create opcodes. Otherwise it seems that ExecutionState::ErrorDepth (deleted) was run into dummy gadget before.
And comment out dummy gadget for feature scroll (only used for selfdestruct and oog-selfdestruct).

Issue Link
Related comment https://github.com/scroll-tech/zkevm-circuits/pull/463#issuecomment-1510042778
Test
Both callop_error_depth (renamed) and test_create_error_depth cases could run successfully.

@lispc Can we just remove ExecError::Depth? Since the error depth has no different to other call/create op, this error state seems meaningless. It doesn't need to be one to one mapping with geth.


Format react-native code (.js files)

VSCode Version: 1.6.1
OS Version: macOS 10.12（16A323）

Steps to Reproduce:

react-native init SomeProject
code SomeProject
open index.ios.js
CMD+SHIFT+P
format (enter)

Thank you for opening this up. I'm not seeing this with the latest release. Can you install VS Code Insiders and try again? The issue is resolved on my end.

The issue is resolved in VS Code Insiders,  Thanks.


MA generated playlists are not adding to the queue what was in the playlist

What version of Music Assistant has the issue?
2.0.0b144
What version of the Home Assistant Integration have you got installed?
2024.5.1
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
When I go to one of the MA generated playlists like "100 random tracks (from library)" I get to see a list like so:

Then I click Play --> Play Now:

The queue is replaced but NOT with the songs that were shown in the playlist (see screenshot above to compare):

I assume this is unwanted behaviour so thought I'd log it
How to reproduce
See "The problem" section where the steps are explained with screenshots
Music Providers
MA playlists seem to use the filesystem library, which are FLAC's mostly in my case
Player Providers
DLNA towards a Marantz/HEOS receiver
Full log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2024.5.1
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

This is correct, these playlists are completely dynamic all the time.

It’s an interesting question then. Should they refresh also when played or only when viewed? If both there is not much point being able to view the list as it will change when played?

My assumption would indeed be that the list you see when you click the Play button is the list added to the queue. It's great that it's a random list, I refreshed the browser a few times to get a new list and if I liked it pressed the Play button. That's when I indeed noticed that there's yet another list added to the queue, which was unexpected.

This should be fixed in b148


CDS Number and CDS Reason

identifier of a CDS advice + reason if not followed.
Is this related with ScoreResult?

Asked question on reason: https://chat.fhir.org/#narrow/stream/179159-cds-hooks/topic/What.20if.20a.20clinician.20does.20not.20follow.20CDS.20advice.3F/near/281119275

@costateixeira Please refer to PM I sent you last week: In Referral Prescription, they are talking about a CDS identifier + a reason why a CDS advice was not followed. Any resources that could capture this?

As the existing profiles to capture the reason are specific to the US (e.g. http://hl7.org/fhir/us/qicore/StructureDefinition/qicore-procedurenotdone ), the proposal is to make an extension containing a  https://www.hl7.org/fhir/guidanceresponse.html and a note (Annotation) to capture the reason.

duplicate of #93


Changed opacity of menu background map

To take the edge off of the background and so that the white text is more readable over the entire screen, like in the credits

Looks great!


Finished electron-packager for LinuxX64

Prepared and improved script setup_pro_env.bash for activating the environment if possible and launch Python server ( the environment activation is done by the app because in some run cases/contexts like launch by clicking on file browser its imposible to reach a previously activated environment ), this script also take care of the python lifecycle. Also prepared main.js for multiple log levels and diferent run configurations. At the end I modified the README file with the command for build the package.

Hello @ricardovb! Thanks for submitting the PR.

In the file src/main/python/server.py, following                are the PEP8 issues :

Line 40:37: E251 unexpected spaces around keyword / parameter equals
Line 40:39: E251 unexpected spaces around keyword / parameter equals
Line 42:38: E211 whitespace before '('
Line 43:13: E128 continuation line under-indented for visual indent
Line 56:1: E305 expected 2 blank lines after class or function definition, found 1

Hello @ricardovb! Thanks for updating the PR.

In the file src/main/python/server.py, following                are the PEP8 issues :

Line 40:37: E251 unexpected spaces around keyword / parameter equals
Line 40:39: E251 unexpected spaces around keyword / parameter equals
Line 42:38: E211 whitespace before '('
Line 43:13: E128 continuation line under-indented for visual indent
Line 56:1: E305 expected 2 blank lines after class or function definition, found 1

Hello @ricardovb! Thanks for updating the PR.

In the file src/main/python/server.py, following                are the PEP8 issues :

Line 40:37: E251 unexpected spaces around keyword / parameter equals
Line 40:39: E251 unexpected spaces around keyword / parameter equals
Line 42:38: E211 whitespace before '('
Line 43:13: E128 continuation line under-indented for visual indent
Line 56:1: E305 expected 2 blank lines after class or function definition, found 1

Hello @ricardovb! Thanks for updating the PR.

In the file src/main/python/server.py, following                are the PEP8 issues :

Line 40:37: E251 unexpected spaces around keyword / parameter equals
Line 40:39: E251 unexpected spaces around keyword / parameter equals
Line 42:38: E211 whitespace before '('
Line 43:13: E128 continuation line under-indented for visual indent
Line 56:1: E305 expected 2 blank lines after class or function definition, found 1


exploitdb: 2023-10-03 -> 2023-10-10

Description of changes
update to 2023-10-10
diff: https://gitlab.com/exploit-database/exploitdb/-/compare/refs/tags/2023-10-03...2023-10-10

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 260441 run on x86_64-linux 1

1 package built:

exploitdb


Pay transfer fees in the asset being transfered

Depends https://github.com/cryptonomex/graphene/issues/211

Depends on https://github.com/cryptonomex/graphene-ui/issues/174


User Personal Data Integration

With the introduction of a more powerful system prompt on a faster server, it becomes possible to expand the LLM's capabilities. One possible expansion involves integrating the current user's personal information onto the initial system prompt.
Currently, the streamlit-structure-haktan branch contains some placeholder information such as allergies and dietary preferences. This data being included has allowed the LLM to further personalize its output. For example, vegan users will not have meat-based dishes recommended to them.
To complete this feature, two tasks need to be done:

The User table in the current database must be updated to include the above information.

The chatbot app must be able to receive the current user's personal information and integrate it onto the system prompt.

The two tasks have been completed and the system prompt now receives the user's relevant personal information. This allows for more personalized responses from the LLM.


[Question] Tree Shaking howto?

Because code is run on browser and server, we cannot take full advantage of webpack tree shaking. The code is transformed to commonjs before anything.
How is it possible to take full advantage of tree shaking? (I've tried webpack-common-shake without success)
ESM is landing in node 8.5 with --experimental-module and also available with @std/esm package. Do you have any plan about this?

[x] I have searched the issues of this repository and believe that this is not x duplicate.

Tree Shaking si enabled by default in next as long as you're not using a babel preset that transforms imports in commonjs, also next is using babel-preset-env but without transforming the modules.

Are you sure?, because in webpack.js there is a plugin to get it back to commonjs https://github.com/arunoda/next.js/blob/68738d1c904476e9c99989ae5ee12a90c184fbc1/server/build/webpack.js#L228

Yes, I already asked something similar some days ago

We can either close this issue. Or submit a simple PR to verify the tree shaking output for the people to believe I


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1275 ms

Resolved: Bofrost.de performance has improved in f967886 after 16 minutes.


Adjust box width to let them fit on a row

Also use the 'card' class for the empty card
on the second row, to make sure all boxes align
with each others.

Thanks.  I'd prefer not to merge this because it makes the boxes different widths, which looks weird.  (Yes, the current situation also looks weird.)
What about reducing the margins on the sides?  Or not putting them in rows, and just letting them flow?  So wide screens would have 3 across, narrower 2 or 1?

I do not know why this suddenly appears, the goal was to have two lines and 3 boxes per line.
My first attempt in this commit was to make all of them 3, but then realize that we can make a few boxes wider, and I picked the one where we could expect wider content... this why it became 4, 3, 4.
I understand your concern and can fix it by making all of them 3
not sure about not putting them in rows... this could lead to unexpected layout, but maybe...
I'm going with a second attempt 3,3,3 :-)

@rspier I've applied your idea via c4975f6e and at first glance it looks great on phone, tablet and regular screen resolutions :-)

I merged c4975f6  but didn't pull in the "width-3" change because it made the boxes too narrow.
More work needs to be done, but this is a huge improvement.

I just pushed another change that tweaked things a little more to make 3 across work even better, at the cost of borders.
The inside of the boxes is still a little odd.  We might be happier targeting 2 across on most screens to have more width to play with to keep things even.


Support more databases

Today, only SQLite is supported. This is great for small-scale deployments but has its limits. Eventually, support for more scalable solutions will be added. I am considering both SQL Server and Cosmos DB.

See: https://docs.microsoft.com/en-us/ef/core/managing-schemas/migrations/providers

Added support for SQL Server. I will revisit other database providers if the need arises.


Influxdb 1.8.10 to 2.2.0 - database not upgraded

Steps to reproduce:
List the minimal actions needed to reproduce the behavior.

I've installed Influxdb 1.8.10 using apt-get and imported the database (influxd restore -portable...)
I've upgraded to InfluxDB 2.2.0 using apt-get
I've run influxd upgrade and filled in the form
After entering web interface, I was forced to finish the onboarding and fill the basic information (username, organization name) one more time
After filling it in, the database that I've imported on InfluxDB 1.8.10 was not moved to InfluxDB 2.2.0
I've tried removing /root/.influxdbv2 and running influxd upgrade again, but it still does not show my database on InfluxDB 2.2.0

According to influxd upgrade logs, the database was upgraded successfuly:

Am I doing something wrong or is there a bug that makes it impossible to upgrade from 1.8.10 to 2.2.0?

I managed to successfuly upgrade InfluxDB 1.8.9 to 2.2.0 when I was not using apt-get & systemctl.
Steps that I've done:

Downloaded and installed InfluxDB 1.8.9
Imported the database (influxd restore -portable...)
Stopped influxdb and influxd
Uninstalled InfluxDB 1.8.9 (apt-get remove influxdb)
Downloaded and installed InfluxDB 2.2.0
Stopped Influxdb and influxd
Run influxd upgrade
Started influxd (using influxd&, not systemctl start influxdb)

The issue is probably when upgrading from 1.8.10 to 2.2.0 or when using apt-get to install InfluxDB.

Now, when the database was upgraded properly, when I start influxdb2 as a service (systemctl start influxdb) it still forces me to do the onboarding on the webpage. It does not show the onboarding when starting influxdb using influxd&. Is there a way to disable the onboarding?

Hello I have the same issue. Is there any solution to this?

I believe this is a pathing issue. When you run the influxd upgrade, can you specify the config path of your 1.x instance and see if that succeeds? Docs here should go into more detail on how to do that.


Releases v1.0.4

Coverage decreased (-0.3%) to 70.87% when pulling de56d607e91169db213ae05cfa7759776934ac5a on releases into a26162dece54986be27b1cb8b8ee4265b035f450 on master.


Switch safemode to use the query param

Making this switch because using the path can lead to undefined routes and some confusion on wildcard routes.
https://github.com/rancher/dashboard/pull/9135#discussion_r1232391119

@codyrancher  What milestone is this for?

I am sorry, but the approval was wrong here @codyrancher . Richard pointed out quite well that we are missing the route param, which isn't necessarily bad, but we should remove it from the docs and delete the page that it loads (safemode.vue) if we are not to use it 🙏

I thought we wanted to leave both options. I've gone ahead and removed the safemode page and route.


Added Azure Blob Storage, Container and BLOB file features.

Added Azure Blob Storage, Container and BLOB file features.
Added tests for blob storage.

You need to add the tests that should be carried out under test/stages.
It looks like this code only needs one dependency, the resource group. As such, see the SSH-key one as an example,
https://github.com/cloud-native-toolkit/terraform-azure-ssh-key/tree/main/test/stages
I have updated the workflows, so once the you update the test stages, it should pass the verify PR testcase before it can be merged..


Bug fix for API url, overall code uplift

FWC API frequenty changes between two URLs - https://api.fwc.gov.au/api/v1/awards or https://uatapi.fwc.gov.au/api/v1/awards
In commit 8a04b6d0bcee03251ea52689799eeface26f4560, URL was switched back to https://api.fwc.gov.au/api/v1/awards which stopped working again recently, therefore prompting this bug fix.
Commits for this pull request:
25e3dfbbf7388ff6c96486828c4add1afb1a609f - implemented connection checks for both URLs during init to determine which one is currently enabled by FWC
e780cc7c41167572411e0546044ddc8f9d18a355 - cleaned up formatting by adding spacing, updating docstrings, adding type hint to methods, and following 120 line length limit. Also uplifted payload dictionary declaration under each API method.
I've incremented the version from 0.2.2 to 0.3.0.
-edit to trigger github actions

close this pr first and then open again to trigger newly created testings.

Nicely done the work! I think we can also add some unit tests. Possibly just use the examples


Support automatic task logging

@yadudoc No that was a mistake, but that commit has since been merged, so no point in pulling it out now.


create smaller package for serverless

serverless (Cloud Functions) limits the developer on their package size. Can we have a keen-serverless package or a node-only package?

@gdi2290 thanks for the note- as of v4 you can use the standalone modules that make up this package to reduce unneeded bundle weight:

keen-tracking.js
keen-analysis.js
keen-dataviz.js

Will this work for ya? Let me know!

thanks, maybe you can write something in the readme suggesting that you can simply include keen-tracking.js for serverlesss apps

That's a good idea, I'll update this issue info to track that work - thanks!