bower_components not generated

The folder bower_components is not generated with mvn command. Include a task inside maven to execute bower install.

Irrelevant. bower_components will be installed with prod profile. Update README.md to include instruction to generate the folder.


add a method to RootEntity to get all the data

Is your feature request related to a problem? Please describe.
If we need to ask for all the RootEntities that are stored, wich kind of method we use? at the moment the two queriies method are get and find but the get_all() method, for example is not present
Describe the solution you'd like
add a method get_all() that return the entire RootEntities set
Describe alternatives you've considered
at the momento have no alternatives

Hi @andrea-mucci!
Currently, we have a way to do that through the find method and Condition.TRUE, which is satisfied by all the root entities stored on the database.
But I agree with you, adding a get_all method will provide the same functionality in a more intuitive way.


There's no way to clean up node_modules after building

Some users may want to have their node_modules removed after generating some set of static assets akin to the source removal concept in the Go buildpack. Perhaps we should consider allowing users to opt-in to this workflow for the Node.js buildpack family as well.

See a user discussion regarding this: https://paketobuildpacks.slack.com/archives/CUD6R3CPL/p1612338123000800?thread_ts=1608826963.002200&cid=CUD6R3CPL
This feature should probably include the ability to clean up user supplied directories (e.g. src)

A common scenario: trying to build and serve a frontend application (i.e. React, Angular).
Ideally the build should use the nodejs buildpack as well as nginx to serve the app.
When the application is built it will need npm installed dependencies, but for it's run image we only need the static content served by nginx.
The current images built with both buildpacks now have the huge disadvantage of being heavy for absolutely no benefit. In my current project we have a 1.2GB image, where 1GB is just unused node_modules.
It would be nice if the layer persistence could be switched off.

I noticed the suggest to use the source-removal buildpack in https://github.com/paketo-buildpacks/nodejs/issues/650#issuecomment-1285799375
Just though I'd mention in case that would work for this use case as well.

I think this specific case is really better served by the Web Servers buildpack. Its designed to handle frontend applications like this and only has node_modules available during the build phase.
For any other case, using the source-removal buildpack would probably be the best route.
I'm gonna close this as we don't plan to do anything about this in the Node.js buildpack at this time.


Added italic type

so sorry I basically noticed this only today, and it could've been a goo dhint ... unfortunately, the idea was to have echomd 100% compatible input, and in shells/terminals italic is not usually available (Linux is fine though)


Add Energy Consumption Calculator

An energy consumption calculator is a tool that helps determine how much electricity is used by a household or business over a specified period of time. It usually requires information such as the appliances used, their power usage, and the number of hours they are used. This information can then be inputted into the calculator to determine total energy consumption. Calculators can also help identify areas where energy is being wasted and suggest ways to reduce consumption. Please assign it to me under gssoc

Closing this as exists #761


migrate setup.py to pyproject.toml

Actual Behavior:
testing/dev dependencies are baked into one install requirements in setup.py. at the same time we want to migrate setup.py to pyproject.toml to modernize and to make the packaging consistent with delphi_utils module
Context
https://github.com/cmu-delphi/covidcast-indicators/issues/1973#issuecomment-2192669283
https://github.com/cmu-delphi/covidcast-indicators/pull/2014#pullrequestreview-2228651795

duplicate of https://github.com/cmu-delphi/covidcast-indicators/issues/2016


Scroll does not move on some sites

It works with default/Chrome's scroll, but fail to move with MinimalScrollbar

https://nightlies.videolan.org/build/win64/
add more here

Though we can scroll with scroll wheel or touchpad, the scrollbar does not move.


The config error message should mention brunch-config, not config

I think this already was addressed previously


WIP: Workaround undefined depth formats

I have absolutely no intention on trying to check this in as-is. It was a first pass, I am just trying to get feedback on what are the "RIGHT" facilities from spirv-opt to use.
I will add specific questions
Problems that I already intend to fix, but wanted to get early feedback

Names. Nothing is consistent.
Nesting is a bit silly right now, obviously needs fixing.

Is this change still in progress?

This is technically still in progress. I have not had a lot of time recently to come back and finish it. I will try to get to it soon

Thanks @s-perron . I think I have responded to all of your comments.
I still have to squash, and write tests. But I have a question:
Now that I use ReplaceAllUses, the new type ends up at the end of the type-manager. This is a problem since there are other types that depend on it (E.g. an OpTypePointer).
Is there any way to change the location that it will be inserted? Also am I possibly just doing this wrong.

@AWoloszyn I this still something you want to add?  If you are not actively working on it, I would like to close the PR.  You will still have your branch if you want to resubmit later.

I will close. I want to get around to this, but things keep getting in the way. I will re-submit if I get a chance. I have found a different workaround for the problem on the specific devices anyway.


Add option to Limit maximum number of go routines launched from serve()

In Container with limited amount of RAM, heavy HTTP load causes to many go routines to be created and the call back handler routines some time later do their memory allocations that cause container to be killed.
This simple extension whose use is optional allows the setting of maximum go routines launched from serve().
It is up to the programmer to determine what this limit should be through load testing their web site.
Printing out information from calling runtime.ReadMemStats() at a reduced rate to a log file for plotting and analysis
can help to determine this limit.
This PR will be imported into Gerrit with the title and first
comment (this text) used to generate the subject and body of
the Gerrit change.
Please ensure you adhere to every item in this list.
More info can be found at https://github.com/golang/go/wiki/CommitMessage

The PR title is formatted as follows: net/http: frob the quux before blarfing

The package name goes before the colon
The part after the colon uses the verb tense + phrase that completes the blank in,
"This change modifies Go to ___________"
Lowercase verb after the colon
No trailing period
Keep the title as short as possible. ideally under 76 characters or shorter

No Markdown
The first PR comment (this one) is wrapped at 76 characters, unless it's
really needed (ASCII art, table, or long link)
If there is a corresponding issue, add either Fixes #1234 or Updates #1234
(the latter if this is not a complete fix) to this comment
If referring to a repo other than golang/go you can use the
owner/repo#issue_number syntax: Fixes golang/tools#1234
We do not use Signed-off-by lines in Go. Please don't add them.
Our Gerrit server & GitHub bots enforce CLA compliance instead.
Delete these instructions once you have read and applied them

@googlebot I signed it!

Please see PDF's in:
https://github.com/redhug1/go-ListenAndServeLimits
that show the effectiveness of limiting the number of go routines on the memory allocations.


reformat table

Sort the table by code and replace the sections with tags.
Inspired by https://github.com/multiformats/multicodec/pull/116 but rebased on master. This version also puts the description last as it's optional.
This PR also includes an interactive table viewer (which we can improve over time).

@Zolmeister apologies for commandeering your PR. Does this look good to you?

Note: The "viewer" is currently viewing master.

@Stebalien the sort order is different (numeric value vs alphabetical, e.g. 0x11 is not next to 0x1100), but probably more correct given https://github.com/multiformats/multicodec/issues/115 so yes, looks good

Ah, yeah, I sorted numerically. Emacs FTW!

@Stebalien what do you think about re-adding some tabs so csv is more readable by itself?

Done.

Thanks a lot @Stebalien. It's been on my TODO list, but you made it happen!


Typo in user interface on Settings page

In the "Generate Example Orchestration File" label , Orchestration is spelled "Orcherstration" on the user interface. Adding this issue to remember to fix the typo on the next pull request

@lawen4cer Can I work on the issue?

Go right ahead!

It would be good to get that resolved before we wrap up #14, which is happening soon.

It would be good to get that resolved before we wrap up #14, which is happening soon.

Done :).


[Visual Assets] Revisit & revise all graphics to reflect 'solution'-centric view

Ensure all graphics currently used in documentation moves from a Nox.Lib or Nox.Cli emphasis to a Nox Solution view

Migrated to Jira


Renaming TransferInfo::Ether variant to Native

Braking change not adding value.
Already discussed and agreed on: https://github.com/gnosis/safe-client-gateway/issues/179#issuecomment-720475505


Document design for RPC Client

How are responses arriving at the shared callback mapped to the correct promise? How will promises be broken when it is time to shut down?
Chris recommends that this be assigned to Uzi

For now, we want the simplest implementation of RPC Client / Server. So long as we can maintain backwards compatibility on the interface, we can release enhancements down the road (e.g. adding a thread pool)


JDBC Data source timing related load failure

Imagine the JDBC db server is booting up  -  while the cowj is booting up.
Now, cowj knowingly throws an error at boot - because it fails to connect to the db in the load time in the Cowj running thread.
Given the "architecture" is about getting dedicated Connection per thread based on lazy loading,
this  at best should be configurable.

no-crash-boot is a new config property that we can now use in JDBCWrapper to ensure things gets booted even if connection failure happens.
It is suggested to use it rarely in prod.

Works


Hello there, I still have the same issue.

Hello, I was wondering if there's any way you can send me the updated main.py file? Since I'm kinda new at this. I tried looking at other issues and seeing if I could resolve them by seeing what you said, but none of them worked. I don't know if I did it correctly or if it stopped working.

Nvm, Managed to fix it.


[docs] How to Model Schemas in typescript

Documentation suggests that we should avoid having to defined schema in the models to reduce the burden on managing schema in migrations and also in the models.
But the examples listed seem to always have schema defined in the model also.

What is best practice?

If I am defining schema in the model to help with typscript, then how should I be using "?" or "!" to represent nullable fields?

You should define tupescript types for the fields and relations. The docs talk about objection not defining the database schema in the model. As in not generating database tables and columns based on models.

@koskimas is there a preferred way to define typescript models?
Or do I just use how the model is defined in Person.ts in the koa example?
examples/koa-ts/models/Person.ts

i write the model properties like:
// Imports we'll need to flesh out our class definition.
import { Model, QueryBuilder, RelationMapping } from 'objection'
// Import related model type
import Whatever from './whatever'

export default class Something extends Model {
  static get tableName(): string {
    return 'something'
  }

  declare somethingKey: number
  static get idColumn(): string {
    return 'somethingKey'
  }

  // plain old not null number field
  declare something: number
  // key for a related Whatever - see |whatever| defined next, and |relationMappings| below
  declare whateverKey: number
  /** Fetched using relation 'whatever'. */
  declare whatever?: Whatever

  static get modifiers(): {
    filterSomethingIs(query: QueryBuilder<Something>, value: number): void
  } {
    return {
      filterSomethingIs(query: QueryBuilder<Something>, value: number) {
        void query.where(`${query.tableNameFor(Something)}.something`, value)
      },
    }
  }

  static get relationMappings(): {
    whatever: RelationMapping<Whatever>
  } {
    return {
      whatever: {
        relation: Model.BelongsToOne,
        from: `${Something.tableName}.whateverKey`,
        to: `${Whatever.tableName}.${Whatever.idColumn}`,
      },
    }
  }
}

using declare says "this property should be there" without generating any defineProperty calls. see https://www.typescriptlang.org/docs/handbook/release-notes/typescript-3-7.html#the-usedefineforclassfields-flag-and-the-declare-property-modifier this i see as an improvement over the example's use of properties defined on the class itself. also note that i eschew use of !.
whether written my way or as shown in the koa-ts example, i'm running into issues around nullable related objects and graph operations. the current typings for PartialModelGraph exclude undefined but not null in testing for relatedPropertyType extends Model, so if you have declare nullablePet?: Pet | null, it thinks the nullablePet property is not a Model, and instead expects a full Expression - so a full Pet or null (roughly), rather than the usual PartialModelObject<Pet>. i'm not yet clear whether we are expected to "just treat undefined as null", or if this is an overlooked case in the typings. but at runtime, a null (you fetched the relation, but it's set to null rather than a key for this - a Pet with no Person owner, say) vs undefined (you haven't even fetched that relation yet, though if it's a BelongsTo relation, you already know what you'll find when you do fetch by checking the key value on your current object) distinction does exist, so i'm leaning towards "overlooked case". 😬

@jeremy-w about PartialModelGraph and nullable fields like nullablePet?: Pet | null, this might be interesting for you:
https://github.com/ovos/objection.js/pull/16
since we haven't published our fork to npm (might do soon, but for now you might simply point your package.json entry to github url + tag or branch) but here is a patch file at hand:
diff --git a/node_modules/objection/typings/objection/index.d.ts b/node_modules/objection/typings/objection/index.d.ts
index c7efd99..ce8b322 100644
--- a/node_modules/objection/typings/objection/index.d.ts
+++ b/node_modules/objection/typings/objection/index.d.ts
@@ -222,15 +222,19 @@ declare namespace Objection {
    * Just like PartialModelObject but this is applied recursively to relations.
    */
   type PartialModelGraph<M, T = M & GraphParameters> = {
-    [K in DataPropertyNames<T>]?: Defined<T[K]> extends Model
-      ? PartialModelGraph<Defined<T[K]>>
-      : Defined<T[K]> extends Array<infer I>
+    [K in DataPropertyNames<T>]?: null extends T[K]
+      ? PartialModelGraphField<NonNullable<T[K]>> | null // handle nullable BelongsToOneRelations
+      : PartialModelGraphField<T[K]>;
+  };
+
+  type PartialModelGraphField<F> = Defined<F> extends Model
+    ? PartialModelGraph<Defined<F>>
+    : Defined<F> extends Array<infer I>
       ? I extends Model
         ? PartialModelGraph<I>[]
-        : Expression<T[K]>
-      : Expression<T[K]>;
-  };
+        : Expression<F>
+      : Expression<F>;

   /**
    * Extracts the property names (excluding relations) of a model class.
    */


add coffeescript syntax

Since there's no official solarized dark syntax, I used textmate coffeescript as a base.
before:

after:

this looks good to me. Need to review a few other issues pulls, but this will work fine


caption didn't work on my gif

Command that caused the error
&caption my reaction to that information
Image that caused the error
https://tenor.com/view/richardwatterson-richard-watterson-alemak-sexy-gif-20740276
Steps to reproduce

Post gif (https://tenor.com/view/richardwatterson-richard-watterson-alemak-sexy-gif-20740276)
&caption my reaction to that information
See error

Self-hosted instance?
No
Error file
Message: Service undefined cannot handle commands!
Stack Trace: undefined
Additional context
No response

Seems like you might have been caught up in a phase during restart, where none of the services are running. Should hopefully be a temporary thing.


Randomly missing rows when calling the same ticker n number of times in a row

Hey here's a reproducible example. A different number of rows are returned when making multiple API requests back to back. I've verified that this happens across all time intervals except 1d (at least I haven't seen this problem with setting interval = '1d').
import yfinance as yf
import pandas as pd

# today is 2022-11-21 so I can go as far back as 2922-09-24 for 2m data.
start_date = '2022-09-24'

df_nvda_2m1 = yf.Ticker('NVDA').history(start=start_date, prepost=True, threads=False, interval='2m')
df_nvda_2m1.shape # 14671 rows

df_nvda_2m2 = yf.Ticker('NVDA').history(start=start_date, prepost=True, threads=False, interval='2m')
df_nvda_2m2.shape # 14671 rows

df_nvda_2m3 = yf.Ticker('NVDA').history(start=start_date, prepost=True, threads=False, interval='2m')
df_nvda_2m3.shape # 14593 rows

This isn't a problem for a single ticker since I won't call the same ticker 3 times in a row within a few seconds, but since the parallelized version of yf.download and running yf.Tickers also return missing rows, I decided to pull multiple tickers in a for loop for that reason. However, even when downloading multiple tickers in a for loop I still experience missing rows. Is this expected? If so, any suggestions on how to avoid this? If I run time.sleep(60) before making a request to the next ticker this will take over 3 days to download all tickers just for a single interval.

I cannot reproduce using that code, and I am reluctant to run enough times to reproduce because could be Yahoo spam detection system limiting requests (I don't want to trigger myself).
Can you reproduce your end but write those different tables to file then upload to a fork branch?
Btw the solution to spam detection is to stop spamming with caching or pay for Plus.

I wasn't aware yfinance library had a premium version. Also not sure how to avoid the spamming or if I want to migrate to the caching library.
I ran the below code 3 times and each time the dfs had 3 rows missing in the second df.
>>> df1 = t.history(prepost=True, start='2022-11-18', interval='1m', threads=False)
>>> df1.shape
(2745, 7)
>>> df2 = t.history(prepost=True, start='2022-11-18', interval='1m', threads=False)
>>> df2.shape
(2748, 7)

Now check this out
>>> df1.index.min()
Timestamp('2022-11-18 04:00:00-0500', tz='America/New_York')
>>> df1.index.max()
Timestamp('2022-11-22 19:59:00-0500', tz='America/New_York')
>>> df2.index.min()
Timestamp('2022-11-18 04:00:00-0500', tz='America/New_York')
>>> df2.index.max()
Timestamp('2022-11-22 19:59:00-0500', tz='America/New_York')

>>> [i for i in df1.index if i not in df2.index]
[]
>>> [i for i in df2.index if i not in df1.index]
[Timestamp('2022-11-18 05:26:00-0500', tz='America/New_York'), Timestamp('2022-11-21 18:14:00-0500', tz='America/New_York'), Timestamp('2022-11-22 17:05:00-0500', tz='America/New_York')]

>>> df2[df2.index.isin(['2022-11-18 05:26:00-0500', '2022-11-21 18:14:00-0500', '2022-11-22 17:05:00-0500'])]
                              Open    High     Low    Close  Volume  Dividends  Stock Splits
Datetime
2022-11-18 05:26:00-05:00  158.190  158.19  158.18  158.180       0          0             0
2022-11-21 18:14:00-05:00  153.110  153.11  153.01  153.070       0          0             0
2022-11-22 17:05:00-05:00  160.295  160.34  160.25  160.295       0          0             0

I meant Yahoo Plus. I've never used only heard, I suppose they whitelist IP addresses.
Your issue is very difficult to debug because only you experience it. #689 raised similar problem but appears not solved. Is that code ever 100% successful? E.g. after avoiding Yahoo for few hours or even a night?

Yep it appears I'm not the only one having this issue. I experience it much more frequently with intraday data. Oh well, I guess this is what happens when you use a free service. I guess there's no fix on yfinance python libraries end is there?

I've quickly coded up a potential fix - a simple "cache" that combines multiple fetches. Call Ticker.history() twice (or more) and eventually should return all the data.
It's in Git branch dev/simple-prices-cache. You'll need to PIP install dateutils

@melgazar9 @paulmcq Any feedback on the branch?

Hey @ValueRaider I don't seem to have this issue anymore after upgrading. I'll close and reach out if I notice it again. Thanks!


Init containers not updated on upgrade

Hi there!
I am using spec.initContainers on a deployment and during the first upgrade to add the initContainers, everything worked fine.
Second upgrade, I only changed things on the initContainers and the deployment didn't even get upgraded.
I tested changing something else and the deployment got updated but upon inspection, there were two annotations: pod.alpha.kubernetes.io/init-containers and pod.beta.kubernetes.io/init-containers, no spec.initContainers and the values on the annotations were still the ones from the first upgrade.
After that I attempted to use the annotations directly instead of the new spec field, and got the same results: the annotations never got updated
After several tests I've come to the conclusion that something is excluding initContainers from the upgrade diffs.
Am I doing something wrong here or is this a bug?
I have used initContainers on annotations before and they worked, but this was a few versions ago.
My helm is:
Client: &version.Version{SemVer:"v2.5.0", GitCommit:"012cb0ac1a1b2f888144ef5a67b8dab6c2d45be6", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.5.0", GitCommit:"012cb0ac1a1b2f888144ef5a67b8dab6c2d45be6", GitTreeState:"clean"}

And Kubernetes:
Client Version: version.Info{Major:"1", Minor:"5", GitVersion:"v1.5.6", GitCommit:"114f8911f9597be669a747ab72787e0bd74c9359", GitTreeState:"clean", BuildDate:"2017-03-28T13:54:20Z", GoVersion:"go1.7.4", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.4", GitCommit:"d6f433224538d4f9ca2f7ae19b252e6fcb66a3ae", GitTreeState:"clean", BuildDate:"2017-05-19T18:33:17Z", GoVersion:"go1.7.5", Compiler:"gc", Platform:"linux/amd64"}

This was posted in #helm-users and I made the comment that I have run into my initContainer details not updating before. I changed my templating to update the initspec and the annotations and it works after that.
...
  template:
    metadata:
      annotations:
        pod.alpha.kubernetes.io/init-containers: '[{"name":"code-copy","image":"example_image:latest","command":["cp","-r","/var/www/html/","/html"],"resources":{},"volumeMounts":[{"name":"app-volume","mountPath":"/html"}],"terminationMessagePath":"/dev/termination-log","terminationMessagePolicy":"File","imagePullPolicy":"IfNotPresent"}]'
        pod.beta.kubernetes.io/init-containers: '[{"name":"code-copy","image":"example_image:latest","command":["cp","-r","/var/www/html/","/html"],"resources":{},"volumeMounts":[{"name":"app-volume","mountPath":"/html"}],"terminationMessagePath":"/dev/termination-log","terminationMessagePolicy":"File","imagePullPolicy":"IfNotPresent"}]'
      labels:
        app: hip-new
        hip-new: www-{{ $k }}
      name: www-{{ $k }}
      namespace: engineering
    spec:
      initContainers:
        - name: code-copy
          image: example_image:latest
          command: ["cp", "-r", "/var/www/html/", "/html"]
          volumeMounts:
            - name: app-volume
              mountPath: /html
      containers:
        - name: fpm
...

sort of idea, so if you want to update the image, since you have a later version.. change the latest in all 3 places to what you need.
Then it works, I did note you saying you knew about this and it is still not working. I dunno.

I use a variable {{ .Values.image.tag }} matching on the init container and regular containers for the docker image tag. When upgrading, the regular containers get updated and the initContainers do not.
...
spec:
  template:
    spec:
      initContainers:
        - name: assets
          image: "{{ .Values.assets.repository }}:{{ .Values.image.tag }}"
      containers
        - name: other-container
          image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"
...

I will try to replicate the init container on both annotations and the spec to see what happens...

Just to clarify what I tried so far:

Using only alpha init container annotation
Using only spec.initContainers
I have yet to try:
Use only the beta annotation
Use both the beta and alpha annotations
Use both the beta and alpha annotations and the spec.initContainers

I managed to replicate the issue on a new chart:
helm create init-bug
echo '      initContainers: [ { name: "sleeper", image: "alpine:{{ .Values.alpineVersion }}", args: ["/bin/ash", "-c", "sleep 10"] } ]'
helm install init-bug --name buggy --set alpineVersion=3.5
helm upgrade buggy init-bug --set alpineVersion=3.6
kubectl get deployment buggy-init-bug -o jsonpath="{.spec.template.spec.initContainers[0].image}"

Running this will output
alpine:3.5

If I run the upgrade on --dry-run and --debug the init container looks correct, but when actually running the code, the deployment is not even changed.

Same here. Init containers are not changed on kubectl apply -f to the same resource file

This is probably related to kubernetes/kubernetes#47264
I'll try changing the beta annotation. Hopefully it'll work

Same here when using spec.initContainers in an CronJob. Changes are not reflected in Kubernetes when upgrading. Requesting to change the label of this issue to Bug.
Example:
apiVersion: batch/v2alpha1
kind: CronJob
spec:
  jobTemplate:
    spec:
      template:
        spec:
          initContainers:
          …

Same problem here.
I got an init-container and at least two containers ,
both the init-container and container #1 are set to use the same image defined by parameters:
image: {{ .Values.Image }}:{{ .Values.ImageVersion }}
So when I do update ".Values.ImageVersion" with a new version, the container #1 does use a new version but not the init-container!!
That's a real problem and bug!!
The init-container parameters should be updated on "upgrade".

Having the init container definition in double with the beta annotation is a work around that works for me to wait Kubernetes 1.8:
https://github.com/kubernetes/kubernetes/issues/47264#issuecomment-311584187

Yes, this is what worked for me too. I have the same init container as a beta annotation and as initContainer on the spec and it works. As stated it's a workaround but I don't feel this will get fixed so quickly

Sounds like this is an upstream issue which we have no control over, unfortunately. I'd suggest following up at kubernetes/kubernetes if there are further issues with updating init containers. Sorry we can't help you with this!

As a newer comment on the thread, it looks like explicitly setting the annotations to null will also have the desired effect of only looking at the spec.


Request support for pH-W3988

Log Message

Please paste logs here

Information about DPS mappings

{
  "result": {
    "category": "dgnbj",
    "functions": [],
    "status": [
      {
        "code": "temp_current",
        "type": "Integer",
        "values": "{\"unit\":\"℃\",\"min\":-100,\"max\":1100,\"scale\":1,\"step\":1}"
      }
    ]
  },
  "success": true,
  "t":<PHONE_NUMBER>692,
  "tid": "ca2d5f83c89111edbae0c235483784fc"
}

Current Temperature: 8
pH value: 102
EC value: 110
Salinity value: 113
Proportion value: 116
ORP value: 119
Product ID

Information about how the device functions
This is a Pool testing box for measuring the pH value and other values that is used to know how good the pool water is.

This appears to be almost the same as #555.  As that issue also has logs attached, I will close this one.


upgrade artifact-download action to v4 to maintain compatibility with…

… the snyk-test action wich uses artifact-upload v4

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.02%. Comparing base (56fd6d7) to head (c65ed4f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #50   +/-   ##
=======================================
  Coverage   79.02%   79.02%
=======================================
  Files          14       14
  Lines        1006     1006
=======================================
  Hits          795      795
  Misses        143      143
  Partials       68       68

Flag
Coverage Δ

?

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

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


Use Log.displayStat to the entry times in the journal detail view

Now, the stat configuration affects to the entry time in the journal detail view. This closes #60
stat human

stat decimal

Thanks. I'm busy with summer classes so I won't be doing much work on this for the next couple of months

yw @joshavanier. I'll try to create PRs based on some ideas I have. I'm using your app daily and I really like it. Good luck with your classes!


Rename the project

What is the current behavior?
We don't have a name to help with the releases #86 #1
What is the new behavior?
We have the jsonapi-serializer name which is an alias for the old fast_jsonapi.
Checklist
Please make sure the following requirements are complete:

[ ] Make an intermediary release with the warning to upgrade to the new gem
[ ] Automate the gem releases
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes /
features)
[x] All automated checks pass (CI/CD)

In regards to:

Make an intermediary release with the warning to upgrade to the new gem

Should we release 1.7.3 with a warning that 2.0.0 will begin using the new project name?

Should we release 1.7.3 with a warning that 2.0.0 will begin using the new project name?

Yep, once we have everything on rubygems, let's do that!

v1.7.3 is out...

Thanks to @eclairedelune this is now unforked as well :bow:


Update sbt-scoverage to 1.9.3

Updates org.scoverage:sbt-scoverage from 1.9.2 to 1.9.3.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scoverage", artifactId = "sbt-scoverage" } ]

labels: sbt-plugin-update, early-semver-patch, semver-spec-patch, commit-count:1

Codecov Report

Merging #206 (0132b34) into master (4aeec82) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #206   +/-   ##
=======================================
  Coverage   84.04%   84.04%
=======================================
  Files          19       19
  Lines         282      282
  Branches        6        6
=======================================
  Hits          237      237
  Misses         45       45

Continue to review full report at Codecov.

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


Update to Get-AdminPowerAppOperationStatus Example 1

Leaving this here as the edit link on the MSFT Doc Page for it appears to be linked to an old/missing repo name.
https://docs.microsoft.com/en-us/powershell/module/microsoft.powerapps.administration.powershell/get-adminpowerappoperationstatus?view=pa-ps-latest#example-1
Please change "-OperationUrl" parameter to "OperationStatusUrl"

@MichaelHolmesWP
Thanks for the correction. Updated.


wasm-bindgen’s std feature is non-additive, breaking some build configurations

Steps to Reproduce
cargo init --lib
cargo add<EMAIL_ADDRESS>cargo add<EMAIL_ADDRESS>--no-default-features
cargo check --target=wasm32-unknown-unknown

Expected Behavior
The project should build
Actual Behavior
error[E0432]: unresolved import `wasm_bindgen::__rt::LazyCell`
    --> /Users/kpreid/.cargo/registry/src/index.crates.io-6f17d22bba15001f/js-sys-0.3.73/src/lib.rs:6041:13
     |
6041 |         use wasm_bindgen::__rt::LazyCell;
     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ no `LazyCell` in `__rt`
     |
note: found an item that was configured out
    --> /Users/kpreid/.cargo/registry/src/index.crates.io-6f17d22bba15001f/wasm-bindgen-0.2.96/src/lib.rs:1598:16
     |
1598 |     pub struct LazyCell<T, F = fn() -> T>(::once_cell::unsync::Lazy<T, F>);
     |                ^^^^^^^^
note: the item is gated here
    --> /Users/kpreid/.cargo/registry/src/index.crates.io-6f17d22bba15001f/wasm-bindgen-0.2.96/src/lib.rs:1597:5
     |
1597 |     #[cfg(not(feature = "std"))]
     |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Additional Context
This problem is occurring because wasm-bindgen is exporting the item LazyCell only if the feature wasm-bindgen/std feature is not enabled, but wasm-bindgen-futures uses LazyCell if wasm-bindgen-futures/std feature is not enabled. But it is not true that wasm-bindgen/std implies wasm-bindgen-futures/std, so this can fail to compile if wasm-bindgen/std is enabled and wasm-bindgen-futures/std is not.
Like all Cargo packages, wasm-bindgen must ensure that its std feature is additive — only adds items, without removing any — or if necessary, use some other technique like exporting a macro by which wasm-bindgen-futures can make its code conditional on whether wasm-bindgen/std is enabled rather than whether wasm-bindgen-futures/std is enabled.

Thank you! I will cut a new release with the fix in #4305 ASAP.


Feature request: Highlight instructions in the playground.

Feature:
Highlight certain opcodes (especially jump instructions) in different colors for better reading & understanding of the bytecode.

Implementation:
Maybe a menu or list where all opcodes are listed and you can pick a custom color for each one.
Additional freature:
The left column in the playground should keep track of the current instruction in the right column via a highlighted line.

Thanks so much for this suggestion, sorry I didn't answer earlier, but I'm not sure this is needed and worth adding more code for.


Darksky: Expose missing conditions for day 0 forecast

Description:
See #18205 for a detailed background on this issue. This PR implements proposal 1 form there.
Dark Sky Sensor didn't expose conditions for day 0 (today forecast) that had the same name as current conditions. With this change all conditions form day 0 (today) forecast are exposed the same way as conditions for the rest of the days (1..7): as dark_sky_<condition>_<day>. As a consequence, conditions for day 0 that were already exposed now have _0 suffix. This actually improves the code by removing most of special handling, based on condition name.
To get today forecast the user now has to add - 0 to forecast configuration parameter.
Conditions, for which suffix _0 appeared: precip_accumulation, temperature_high, temperature_low, apparent_temperature_high, apparent_temperature_low, precip_intensity_max, moon_phase.
This is a breaking change!
Related issue (if applicable): fixes #18205
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#7424
Example entry for configuration.yaml (if applicable):
sensor:
  - platform: darksky
    api_key: YOUR_API_KEY
    forecast:
      - 0
    monitored_conditions:
      - <any>

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

I've tested this in 83.b0
It seems that also humidity and temperature are affected as I have those sensors now with and without _0

my config:

- platform: darksky
  api_key: !secret darksky_secret
  language: es
  forecast:
    - 0
  monitored_conditions:
    - temperature
    - humidity
    - temperature_low
    - temperature_high
    - hourly_summary
  update_interval:
    minutes: 5

@luca-angemi, take a look at table in #18205. The logic is:

conditions that had X in column 1 didn't change name.
conditions that had ? in column 2 were not available before. Now they're exposed as <condition>_0.
values that had - in column 1 and X in column 2 changed their name to <condition>_0.

So what you see now is: sensor.dark_sky_temperature is current air temperature, sensor.dark_sky_temperature_0 is forecasted air temperature for today. Same for humidity.

By the way, regarding unknown value: API documentation for temperature states "optional, not on minutely". So, while they state that they provide it on all forecasts except minutely, the value is optional and they don't seem to provide it on daily forecast either.


[FIX] missing line

Artifacts available for download:

Artifact Name
Download Link

clang.versions
https://github.com/seqan/actions/actions/runs/11998667606/artifacts/2230076628

gcc.versions
https://github.com/seqan/actions/actions/runs/11998667606/artifacts/2230077009

intel.versions
https://github.com/seqan/actions/actions/runs/11998667606/artifacts/2230075939


Tests are installed alongside library files

Observed behaviour
ls fontbakery/specifications in its venv shows:
-a----       2018-03-15     10:43              0 __init__.py
-a----       2018-03-26     10:55            207 __init__.pyc
-a----       2018-03-26     10:16         203779 googlefonts.py
-a----       2018-03-26     10:55         181694 googlefonts.pyc
-a----       2018-03-19     11:36         104219 googlefonts_test.py
-a----       2018-03-26     10:55          83874 googlefonts_test.pyc
-a----       2018-03-23     08:56           6041 ufo_sources.py
-a----       2018-03-26     10:55           7215 ufo_sources.pyc
-a----       2018-03-23     08:56           3018 ufo_sources_test.py
-a----       2018-03-26     10:55           4423 ufo_sources_test.pyc

Expected behaviour
Tests shouldn't be installed.
Is there a reason tests aren't in their own $root/tests directory? Why are they spread out inside Lib/?
https://blog.ionelmc.ro/2014/05/25/python-packaging/#the-structure

One solution would be to move all tests to tests/. How about it?

@felipesanches Ping :)


Add end-to-end testing workflow

In this PR I add end-to-end tests for the Miden node.
closes: #222

Should we merge this?

Should we merge this?

I just want to go over, cleanup and should be ready for review & merge.

Thank you! Looks great! I did a very quick review and left a couple of nits/questions inline. I defer a more detailed review to @Dominik1999 or @hackaugusto.
One note for the future: these end-to-end tests will work only when node and client versions are in sync. But we won't be able to use them when the node has some breaking changes which haven't been propagated to the client yet. To address this, we'll need to create a separate "test runner" which will not rely on miden-client. Let's create an issue for this.

Created here: https://github.com/0xPolygonMiden/miden-node/issues/291

This PR does not encompass what we need for end-to-end testing on the Miden node. Some of these tests are already covered by the Miden client. We will implement more robust end-to-end testing when the Miden node and general Miden crates are ready.


Setting up Travis CI

👍 For fixing the string metrics test
Can you set to test with more then one hawkular version ?
We have a PR to support legacy API #20 and It will be nice if legacy versions also get tested.

Can you rebase?

Thanks 👍

@burmanm This is ready for another review could you please review it when you get a chance? Thank you!, @yaacov I added tests for version 0.15, I'm not sure if that is enough.

@rubenvp8510 it's just what I wanted ( ver < 0.16 )
p.s.
I wanted this because the QE team is now using very old 0.8, I hope they will start using 0.21 soon, with the alerts, and I will not need this :-)


Separate sources which depend on UnityEngine.dll from independents

This comit includes 2 csprojs

UniRx.Library.csproj: independent on UnityEngine.dll, can be used from non-Unity projects
UniRx.Library.Unity.csproj: dependent on UnityEngine.dll

Some classes and namespaces in UniRx.Library.Unity.csproj are conditionally renamed because a partial class can not be defined over 2 separate projects, if the UniRxLibrary symbol defined.

Thank you, accept your request.
I'll move ReactiveProperty, ReactiveCollection, ReactiveDictionary from UniRx.Library.Unity to UniRx.Library.


Cloned the repository, worked out how it worked, added some sentences

It is a bit confusing that you have two different readme-files in two different formats. We would suggest to sum up all the info in just one readme-file in markdown format so that it is more user-friendly.

Thanks for the feedback! We merged the files to make the repository more user-friendly and also updated it to be more readable. We hope you are satisfied with the changes!


the  cloudshell addon cannot work.

Describe the bug
When I open cloudshell popup,there are error page .

the page will show the error massage as follow:
Prepare The Cloud Shell
Failed:no kind is registered for the type v1alpha1.CloudShell in scheme "pkg/runtime/scheme.go:100"

KubeVela Version
1.9.3
Cluster information
kubernetes :  1.24.10
** The addon version  **
CloudShell 0.0.9
** The browser version  **
firefox(115.0.2 (64 位) and edge(版本 114.0.1823.67 (正式版本) (64 位))

This is a velaux issue, and fixed https://github.com/kubevela/velaux/pull/856 . We'll release velaux addon as soon as possible.


chore: improve recordings, API, tests, and GitHub actions

Add dependency github.com/appleboy/com v0.1.7
Import github.com/appleboy/com/random
Change default value of Value from "deploy" to an empty string
Set plugin.Config.RemoteName to a random string if it's empty

fix https://github.com/appleboy/drone-git-push/issues/65

thanks :)


Test Suite aborts on NoSuchMethodError

I just upgraded our complete codebase to use 3.0.0 and after fixing all the deprecations, I still see tests sporadically abort with this message. This seems to happen in Jenkins more than local builds, though I'm still trying to nail down the pattern (if there is one).
Of course, when it happens on Jenkins, we don't always notice, because of Issue #297 which doesn't seem to actually be fixed.
Cause: java.lang.NoSuchMethodError: org.scalatest.FunSpecLike$ItWord.apply(Ljava/lang/String;Lscala/collection/Seq;Lscala/Function0;)V
at com.truefit.utils.misc.MessageBundleSpec$$anonfun$checkAllPropertiesExist$1.apply(MessageBundleSpec.scala:128)
at com.truefit.utils.misc.MessageBundleSpec$$anonfun$checkAllPropertiesExist$1.apply(MessageBundleSpec.scala:126)
at scala.collection.Iterator$class.foreach(Iterator.scala:727)
at scala.collection.AbstractIterator.foreach(Iterator.scala:1157)
at scala.collection.IterableLike$class.foreach(IterableLike.scala:72)
at scala.collection.AbstractIterable.foreach(Iterable.scala:54)
at com.truefit.utils.misc.MessageBundleSpec$class.checkAllPropertiesExist(MessageBundleSpec.scala:126)
at com.truefit.consumer.model.LocalePropertySpec.checkAllPropertiesExist(LocalePropertySpec.scala:8)at com.truefit.consumer.model.LocalePropertySpec$$anonfun$1.apply$mcV$sp(LocalePropertySpec.scala:10)
at org.scalatest.SuperEngine.registerNestedBranch(Engine.scala:595)
...

I'm getting the same here. Definitely a lot more on our Jenkins builds

This turned out to be user error. We had a test suite that depended on code in another repo and I hadn't pushed the version update to that repo, yet. Getting both repos (and therefore code artifacts) on 3.0.0 fixed the problem.

Hey, sometimes when you ignore an issue it just goes away! Sorry for the delay in looking into this. Just haven't been able to catch up yet. Glad to hear you got it working.

The fix for me was a bit different. The scala gradle plugin gets the zinc classpaths mixed up.
adding
            if (it.name == "zinc") {
                force 'org.scala-lang:scala-library:2.10.5'
            }

to
configurations.all {
        resolutionStrategy {

fixed the issue


Python validation and tests

There are some easy validations and tests we could run on this repo:

Make sure all json schemas are valid
Make sure no two json schemas have the same name
Make sure all vertices have _key required and all edges have _key, _from, and _to required
Validate AQL syntax on views

This could be run with some python scripts on make test

Closed in https://github.com/kbase/relation_engine_spec/pull/9


Return string type for JSONText.Value()

In order to support binary_parameters flag in lib/pq, we need to explicitly return string type to signal lib/pq to send the field via text format (pointed out by cbandy at https://github.com/lib/pq/issues/528). This change shouldn't break backwards compatibility if binary_parameters isn't turned on.
We've been testing this fork in production for a few days with pgbouncer transaction pooling mode, and so far things are working as expected.

Hello, @ardan-bkennedy, and I recently stepped in to help maintain this project.
We are sorting the opened issues and pull requests and would like to know if you still NEED this merged.
Thank you for your contribution.


Make bundle size smaller

use ramda instead of lodash
move prop-types to peerDependencies

How does using ramda utilities make the bundle size smaller? Can you show a comparison table, compressed and uncompressed, etc?

@blord-fullscreen Because lodash/find and lodash/sortby will use other methods of lodash, looks like you just use these two methods , but others will be referenced.
// lodash/find
var createFind = require('./_createFind'),
    findIndex = require('./findIndex');

// lodash/_createFind
var baseIteratee = require('./_baseIteratee'),
    isArrayLike = require('./isArrayLike'),
    keys = require('./keys');
// ...

After we used ramda instead,  the uncompressed bundle size reduced from 151kb to 46kb.

bundle size(uncompressed)
finally size of find & sortBy
percent

lodash
151.38kb
104.7kb
69.16%

ramda
46.42kb
7.96kb
17.14%

Closing in favour of #374


[issue-589] Use consul /v1/health/service/ for service discovery

My scala is pretty rusty, but this is how I approached a solution for issue-589.  I chose to use /v1/health/service/ which returns a json payload like ...
[
  {
    "Node": {
      "Node": "foobar",
      "Address": "<IP_ADDRESS>",
      "TaggedAddresses": {
        "lan": "<IP_ADDRESS>",
        "wan": "<IP_ADDRESS>"
      }
    },
    "Service": {
      "ID": "redis",
      "Service": "redis",
      "Tags": null,
      "Address": "<IP_ADDRESS>",
      "Port": 8000
    },
    "Checks": [
      {
        "Node": "foobar",
        "CheckID": "service:redis",
        "Name": "Service 'redis' check",
        "Status": "passing",
        "Notes": "",
        "Output": "",
        "ServiceID": "redis",
        "ServiceName": "redis"
      },
      {
        "Node": "foobar",
        "CheckID": "serfHealth",
        "Name": "Serf Health Status",
        "Status": "passing",
        "Notes": "",
        "Output": "",
        "ServiceID": "",
        "ServiceName": ""
      }
    ]
  }
]

Tests on my machine.
> consul/test
[info] AgentApiTest:
[info] - localAgent returns LocalAgent instance
Aug 23, 2016 10:40:10 AM io.buoyant.consul.v1.BaseApi$$anonfun$1 applyOrElse
ERROR: retrying consul catalog request on error java.lang.Exception: I have no idea who to talk to
Aug 23, 2016 10:40:10 AM io.buoyant.consul.v1.BaseApi$$anonfun$1 applyOrElse
ERROR: retrying consul catalog request on error java.lang.Exception: I have no idea who to talk to
[info] CatalogApiTest:
[info] - datacenters endpoint returns a seq of datacenter names
[info] - serviceNodes endpoint returns a seq of ServiceNodes
[info] - serviceMap endpoint returns a map of serviceNames to tags
[info] - blocking index returned from one call can be used to set index on subsequent calls
[info] - propagates client failures
[info] - makes infinite retry attempts on retry = true
[info] - reports invalid datacenter as an unexpected response
[info] KvApiTest:
[info] - list returns an indexed seq of key names
[info] - list reports wrong paths with an error
[info] - get returns an indexed value
[info] - get uses raw values
[info] - get reports wrong paths with an error
[info] - multiGet returns an indexed seq of values
[info] - multiGet by default is non-recurse
[info] - multiGet with recurse set to true adds a recurse parameter
[info] - multiGet reports wrong paths with an error
[info] - put returns true on success
[info] - put returns false on failure
[info] - put cas flag
[info] - delete returns true on success
[info] - delete returns false on failure
[info] - delete cas flag
[info] - delete by default is non-recurse
[info] - delete with recurse set to true adds a recurse parameter
[info] - blocking index returned from one call can be used to set index on subsequent calls
[info] - propagates client failures
[info] - makes infinite retry attempts on retry = true
[info] - reports invalid datacenter as an unexpected response
[info] - get reports permission errors
[info] Run completed in 799 milliseconds.
[info] Total number of tests run: 30
[info] Suites: completed 3, aborted 0
[info] Tests: succeeded 30, failed 0, canceled 0, ignored 0, pending 0
[info] All tests passed.
[success] Total time: 3 s, completed Aug 23, 2016 10:40:11 AM

As discussed in https://github.com/BuoyantIO/linkerd/issues/589, I think we'd like this to be a separate namer from the CatalogNamer (though it looks like it will be almost identical) so that we can get a health-based namer merged before validating that it is a suitable replacement for the catalog namer in all cases.
Also, I think the passing of passing should be configurable so that we can choose to respect consul's health checks or not.

As luck would have it '?passing=false' does not have the effect one would hope for.  Even so, why would anyone want down services (that are failing health checks) to be considered as valid endpoints?

I think if you omit the ?passing parameter entirely, the response won't be filtered to passing services.  As for why you would want that, you may not want to treat consul as the authority on which services are healthy.

I see your reasoning with '?passing', and in that case I think it makes sense to use the CatalogAPI in that case.  How do you feel about adding a paramenter useHeathCheck: Option[Boolean] = Some(False) to case class ConsulConfig so I can just switch out CatalogApi for HealthApi based on user configured preference.  I ask b/c I feel implementing an entire namer for such a small change is not worth the effort.

This is what I'm considering for HealthApi.
diff --git a/consul/src/main/scala/io/buoyant/consul/v1/HealthApi.scala b/consul/src/main/scala/io/buoyant/consul/v1/HealthApi.scala
new file mode 100644
index 0000000..2235433
--- /dev/null
+++ b/consul/src/main/scala/io/buoyant/consul/v1/HealthApi.scala
@@ -0,0 +1,79 @@
+package io.buoyant.consul.v1
+
+import com.twitter.conversions.time._
+import com.twitter.finagle.http
+import com.twitter.finagle.service.Backoff
+import com.twitter.finagle.stats.{DefaultStatsReceiver, StatsReceiver}
+import com.twitter.util.{Duration, Future}
+
+object HealthApi {
+  def apply(c: Client): HealthApi = new HealthApi(c, s"/$versionString")
+}
+
+class HealthApi(
+  override val client: Client,
+  override val uriPrefix: String,
+  override val backoffs: Stream[Duration] = Backoff.exponentialJittered(1.milliseconds, 5.seconds),
+  override val stats: StatsReceiver = DefaultStatsReceiver
+) extends CatalogApi(
+  client,
+  uriPrefix,
+  backoffs,
+  stats
+) {
+
+  val healthPrefix = s"$uriPrefix/health"
+
+  // https://www.consul.io/docs/agent/http/health.html#health_service
+  override def serviceNodes(
+    serviceName: String,
+    datacenter: Option[String] = None,
+    tag: Option[String] = None,
+    blockingIndex: Option[String] = None,
+    retry: Boolean = false
+  ): Future[Indexed[Seq[ServiceNode]]] = {
+    val req = mkreq(
+      http.Method.Get,
+      s"$healthPrefix/service/$serviceName",
+      "index" -> blockingIndex,
+      "dc" -> datacenter,
+      "tag" -> tag,
+      "passing" -> Some("true")
+    )
+    executeJson[Seq[ServiceHealth]](req, retry).map(
+      indexed => {
+        val service = indexed.value.flatten(_.Service).head
+        val node = indexed.value.flatten(_.Node).head
+        val result = List(
+          ServiceNode(
+            node.Node,
+            service.Address,
+            service.ID,
+            service.Service,
+            service.Tags,
+            service.Address,
+            service.Port
+          )
+        )
+        Indexed[Seq[ServiceNode]](result, indexed.index)
+      }
+    )
+  }
+}
+
+case class Node(
+  Node: Option[String]
+)
+
+case class Service_(
+  ID: Option[String],
+  Service: Option[String],
+  Tags: Option[Seq[String]],
+  Address: Option[String],
+  Port: Option[Int]
+)
+
+case class ServiceHealth(
+  Node: Option[Node],
+  Service: Option[Service_]
+)
diff --git a/consul/src/test/scala/io/buoyant/consul/v1/HealthApiTest.scala b/consul/src/test/scala/io/buoyant/consul/v1/HealthApiTest.scala
new file mode 100644
index 0000000..e4c8829
--- /dev/null
+++ b/consul/src/test/scala/io/buoyant/consul/v1/HealthApiTest.scala
@@ -0,0 +1,33 @@
+package io.buoyant.consul.v1
+
+import com.twitter.finagle.Service
+import com.twitter.finagle.http.{Request, Response}
+import com.twitter.io.Buf
+import com.twitter.util.Future
+import io.buoyant.test.Awaits
+import org.scalatest.FunSuite
+
+class HealthApiTest extends FunSuite with Awaits {
+  val nodesBuf = Buf.Utf8("""[{"Node":{"Node":"Sarahs-MBP-2","Address":"<IP_ADDRESS>"}},{"Service": {"Service":"hosted_web","Tags":["master"],"Port":8084, "Address":""}}]""")
+  var lastUri = ""
+
+  def stubService(buf: Buf) = Service.mk[Request, Response] { req =>
+    val rsp = Response()
+    rsp.setContentTypeJson()
+    rsp.content = buf
+    rsp.headerMap.set("X-Consul-Index", "4")
+    lastUri = req.uri
+    Future.value(rsp)
+  }
+
+  test("serviceNodes endpoint returns a seq of ServiceNodes") {
+    val service = stubService(nodesBuf)
+
+    val response = await(HealthApi(service).serviceNodes("hosted_web")).value
+    assert(response.size == 1)
+    assert(response.head.ServiceName == Some("hosted_web"))
+    assert(response.head.Node == Some("Sarahs-MBP-2"))
+    assert(response.head.ServiceAddress == Some(""))
+    assert(response.head.ServicePort == Some(8084))
+  }
+}

@JustinVenus that sounds reasonable to me.  In that case I think we would rename CatalogNamer to ConsulNamer since it can be backed by either the CatalogApi or the HealthApi.

@adleong I think this meets the requirements that we discussed.  Please advise if i should add any other members to trait ConsulApi extends BaseApi.  Also sorry git chose to treat the rename as delete + add.

@adleong @Ashald I've uploaded linkerd-0.7.3-SNAPSHOT-exe to dropbox if you'd like to verify the same binary sha256 below.  I'm deploying this version to my test environment.
shasum -a 256 /Users/jvenus/Projects/linkerd/linkerd/target/scala-2.11/linkerd-0.7.3-SNAPSHOT-exec
4a5cffe054b3e988bbb173c356ca13d2915d3d1334d6a96456e054ffa0eec5ca  /Users/jvenus/Projects/linkerd/linkerd/target/scala-2.11/linkerd-0.7.3-SNAPSHOT-exec

@JustinVenus I'll give it a try later today

Hm, just noticed 1 issue: looks like it's not possible to rely on catalog api but do respect the maintenance health check, right? Or in other words, filter out only those services that are in maintenance mode.

@Ashald  I do not see a way to use the consul catalog api to filter out anything for us.  From my very quick reading of the consul health api, it seems we can get the entireor the passing status of a service group.  It appears that we'd have to filter unhealthy out ourselves.
Fwiw, I am happy to accept deferring the knowledge of what is in maintenance to something else.  I personally don't feel like linkerd should care about the health of consul provided services when useHealthCheck is explicitly enabled.

To follow up on the maintenance checking I think it would be relatively easy to implement a case Class Checks and do something with the result when we are mapping the other case classes to ServiceNodes.
However, I think a larger discussion about what do do with the result would need to be had.
I'm mostly interested in getting failing nodes out of linkerd's consideration.  Zipkin in my environment shows me significant time spent, retrying to find working services (when consul returns unhealthy services).

This looks really good to me, aside from a couple of small style nits.  Could you also update linkerd/docs/namer.md and CHANGES.md?

@adleong thanks for the feedback.  I will make the changes and update this pull request.

@adleong next steps?

Hi @JustinVenus, sorry I missed your response to my comment.  I've replied above.

@JustinVenus I verified your custom build and it works as expected

@Ashald awesome, thanks for trying it out.

@adleong I think this is ready to go.

Thanks for making those fixes, @JustinVenus.  I just had a couple of requests about the docs and then I think we can merge.

@adleong documentation updates have been made.

:star: Awesome!  Thanks for all your hard work on this, @JustinVenus

:star: this looks really good. Very happy to merge this ;)


Feature Request: Setting to automatically toggle website's own dark mode

It would be really nice if there was a setting that enabled Midnight Lizard to automatically toggle on and use any website's own dark mode when available (though i'm not sure of this can be implemented). Because any website's own dark mode is the best performance wise & also appearance wise.

Hey I agree with this, I made a similar issue https://github.com/Midnight-Lizard/Midnight-Lizard/issues/376
I'm going to look into the code and see if I can come up with some solution in the next few days, There should be a pretty easy way to detect if a website is already dark mode.


TPv2 Styling and Compression Changes

This PR does many things:

Styles across forms/tables are consolidated to remove duplicates and ensure a consistent experience.
Removes FontAwesome in favor of Material Icons.
Enables SSR Hydration
Adds animations to the sidebar as well as adding a button/change collapse.
Adds a custom webpack configuration to compress large files using gzip and brotli.
Changes to the server to provide the compressed files if available and supported.
Added linter rule to disallow mixed spaces-tabs.
Implemented double clicking on a table row to navigate to that detail page.
Adds missing buttons/links from TPv1 parity.
Fixes an issue where the sidebar would "flash" on the screen momentarily when at the login screen.

Which Traffic Control components are affected by this PR?

Traffic Portal v2

What is the best way to verify this PR?
Ensure the build/lint/tests pass. Confirm FontAwesome is no longer downloaded. Confirm sidebar works and animations are appropriate. Verify that js/css/svg/ttf files are compressed/served compressed if accepted. Navigate through the app and ensure the tables/details all look the same at various resolutions. If detail pages feature multiple fields per column then the card should be responsive based on screen width. Every tables page should support double clicking to open details.
Each page has also been updated to match what is found in TPv1 with respect to View ... buttons and View Details hints on detail forms. For instance, the Cache Group detail page has Queue Updates (disabled), View ASNs, and View Servers. The View Detail has been implemented by adding a hint under the relevant field with a chain link icon that when clicked should take the user to the relevant table/detail.

PR submission checklist

[x] This PR has tests
[ ] This PR has documentation
[ ] This PR has a CHANGELOG.md entry
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Code looks fine, needs a rebase then I'll do some manual testing


157 dropdown

By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---
CONTRIBUTING.md also tells you what to expect in the PR process.
Description
'Choices' added as a web component based on ChoicesJs library.
Components.d.ts file added to gitignore

📦 Published PR as canary version: 18.2.1--canary.170.83587c2ca778b0a53f8dc2fd4fd559912809b056.1

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

we will not use choicesjs for our implementation


[1.30.8] sap.m.MessagePopover openBy design issue using inside toolbar

Inside the methdo "openBy" there is a comment:
// If MessagePopover is opened from an instance of sap.m.Toolbar
// and is instance of sap.m.Popover remove the Arrow
I would like to use the MessagePopover as a popover inside my Panel Toolbar (the same way i expect from Popovers). Why the hell am i not allowed to do this ;-)
OK, from SemanticPage point of view this is cool to have a default behavior inside the footer, but i need more control outside the fiori space.
Maybe this would be a general nice feature for popups to use it this way (remove anchor), but please make it controllable!
BTW: MP is a child of control, but using class="sapUiSizeCompact" inside xml fragment also does not work in this context.
Best regards,
Holger

HI Holger,
We will ask the opinion of the responsible developer about setting the showArrow by default.
BTW, where do you set the sapUiSizeCompact, since popovers are in StaticUIArea they cannot inherit styles from the view.
Regards,
Cahit

Hi Cahit,
generall we do something like
// toggle compact style
jQuery.sap.syncStyleClass(
"sapUiSizeCompact", oView, oCmp._mFragments[sFragmentId]
);
at the end of our getXmlFragment helper, which works quite well if using ex. class="sapUiSizeCompact" on fragment root. In this case using the class on the button, the size will be reused inside fragment!
But also using directly
oCmp._mFragments[sFragmentId].addStyleClass("sapUiSizeCompact");
does not work (did not test it for xmlview instead of xmlfragment)!
In our Design, we mostly use Cozy design, but some elements like m.table will be Compact on none Touch devices for layout reasons.
Regards,
Holger

Hi @hschaefer123 ,
Sorry for the late response. The control is implemented by design and removing the arrow is intended effect which will not be changed. Regarding sapUiSizeCompact issue, the version 1.30 is not supported anymore. If the issue is still available on supported version please raise an issue.
Regards,
Diana


Add Mechanism to Refetch in createMutatorStore Without a Loading State

Hi I have a use case like this:
const queryKey = "things"

export const $things = createFetcherStore(queryKey, {
      fetcher: () =>
        getThings().then((response) => response.data),
    })

export const $createThing = createMutatorStore<Payload>(async ({ data, invalidate }) => {
  invalidate(queryKey);
  return createThing(payload);
});

I would really like a way for createMutatorStore to be able to call invalidate to trigger a refetch but without setting $things.loading back to true. I want it to replace the old data with the new data but do it without clearing everything out before because it's fine to show stale data until the new stuff comes back
Either that, or expose a refetch callback on the createFetcherStore return value, so that we can call it from the react component
This seems pretty standard in libraries like react query

ah nvm I guess you can just say if (!data && loading) as the condition for first load

@martinmckenna Yep, you guessed it right! I suppose, it'd be better to update the README to reflect a better approach then if (loading) return <>loading</>. I already use a different approach in all my SB demos.


Make source maps work in plug code

When something errors out, you get generated source code line numbers. It would be good to figure out to enable sourcemaps for this as well  for a better developer experience.

This now works by using silverbullet plug:compile --debug


Create Operation Module

Reorganized Operation related code to a separate module, like with Assertions. Also revisited some of the logic in extracting a path from an Operation. Revisited unit tests, too.

@jhlee-mitre good idea! it's only for spec fixtures though, so fixtures used in unit testing, so I think it's fine to keep in one. If I was updating for our sample fixtures for the driver, I would definitely take your suggestion.


move required asterisk to css file

@freedayko would you mind rebase the Pull Request for remove the conflicts, so it could be rewiewed for merge?

@freedayko I gonna reject and close the PR since already was integrated in master. If you consider that still is needed the change, please reopen this PR or open a new issue. Thanks for your contribution!


Translation Updates

Translation Process Automation generated string (DO NOT EDIT): [app/translations/webrtc-devices/de.json]

Fixed translation issue. Looks good to me.

thanks @kiyoshiiwase


Two lists with task lists causes strange formatting

Why?
## Supported Builtins

- [ ] assert

- [x] node globals _partly_

Formats as:
## Supported Builtins

- [ ]

  assert
- [x]

  node globals _partly_

See:

https://github.com/raphlinus/pulldown-cmark/issues/455 (not an issue)
https://spec.commonmark.org/0.29/#loose


Malfunction after core_2024.1.0 update

hi,
after core_2024.1.0 update the integration does not update sonoff states, but the current measurements work

I'm having the same issue.

Duplicate of #1288, please close this issue and track #1288

Same issue with my POWR320D, after latest core update no measurement data is available, also the device is no longer in the debug log

Yes. I am facing the same issue. Please help. Thank you so much.

People, stop posting here and head over to #1288

There is a fix in #1288. Thank you!


Updated the test task with the changes to reduce the calls to gettestrun

Updated the test task with the changes to reduce the calls to gettestrun

@cltshivash You didn't rename the package folders to reflect sprint/patch version?

@cltshivash You didn't rename the package folders to reflect sprint/patch version?

Yes. I didn't. Will pick up next.


Requirements to accelerate the installation process of Dapr components

We need to consider accelerating the Dapr installation process, especially in scenarios where access to the dapr image is blocked.
This will cause the installation process to fail.

how to accelerate the dapr installation process?

Dapr installation  failed :
✗ Dapr - Failed!
 ⇲ Error reports:
Failed to init Dapr:
{"time":"2022-07-10T08:42:04.795768Z","status":"failure","msg":"looks like \"https://dapr.github.io/helm-charts\" is not a valid chart repository or cannot be reached: Get \"https://dapr.github.io/helm-charts/index.yaml\": read tcp <IP_ADDRESS>:60333-\u003e<IP_ADDRESS>:443: read: connection reset by peer"}
: exit status 1

@ZoeShaw101 now the helm is the recommended way to install OpenFunction.
You can find guide here https://openfunction.dev/docs/getting-started/installation/#Install-OpenFunction


[Feature request] Senders name & email alternative option

Instead of including another column for the name, it would be much simpler if the senders name and email were together, for example:
John Smith (johnsmith@email.com),

I'd rather have addon do one thing and be as simple as possible. Shouldn't be that hard to fork and customize it, though - all the relevant code is in customcol.js, in getCellText function.

That's awesome, thank you!
Here's the fork btw - https://github.com/Savasdotexe/namenaddress-column/ I've also added you as an author, I hope that is appropriate and you do not mind? Seeing as you've done pretty much everything.
There is just one issue that I've noticed. Some names are coming out skewed, like for example - https://i.imgur.com/oWoznw3.png
The default 'From' column is just a reference to compare the sender name. It does not seem to be due to special characters as I noticed a few others with normal characters were similar issue even though the email address header itself reads it fine. Some names are also in quotes ( like "Name").

This looks like the same issue as #4 (third paragraph). Usually Thunderbird automatically decodes MIME Encoded-Word back to readable form, but in some rare cases like here it doesn't. So far I didn't manage to reproduce the issue on any of my mails, so I can't really test and fix it.
You can try using decodeWords method from libmime.js before passing a result from your getAddress method. It should handle such cases.


Can't acquire online players correctly

Hi.Yesterday I tried to use mcstatus to acquire server online players.But it reported an error.To be precise, it should be the server side problem.I use the CatServer side as the server,and I have already reported the problem to the developers of CatServer.And I've given some specific information in this issue.
And I will also put some core code here as well.

By the way, I have three Minecraft Servers, one is CatServer and the others are pure forge server.Only the CatServer raise exception.

What error? Which version of mcstatus?

What error? Which version of mcstatus?

9.3.0

What error? Which version of mcstatus?

Traceback

What error? Which version of mcstatus?

And another Traceback

Could you try again with 9.3.2? There was a fix for #383 that fixed BungeeCord status, maybe CatServer expects the same behavior?

Could you try again with 9.3.2? There was a fix for #383 that fixed BungeeCord status, maybe CatServer expects the same behavior?

I have tried version 9.4.1 last night.But unfortunately, the problem still exists.

Does the multiplayer screen in a vanilla client show information about a CatServer? Typically ConnectionRefusedError is due to a firewall interfering.

Show information?I

Yes, when you open the multiplayer list on a client, does the CatServer show information such as MOTD and the server icon?

Does the multiplayer screen in a vanilla client show information about a CatServer? Typically ConnectionRefusedError is due to a firewall interfering.

I misunderstood your meaning.Multiplayer screen in a vanilla client can show information about a CatServer correctly.

Show information?I

Yes, when you open the multiplayer list on a client, does the CatServer show information such as MOTD and the server icon?

I got a new client.

But unfortunately, althought I fix the mistake,it still didn't work.

Do you still get the same error though? Or do you now see something different?

Closing this as stale, if you're still experiencing this problem and want the issue re-opened, just add another comment explaining the second problem you were having. @half-nothing


Out of memory caused by partition write operations

Issue
The issue arises when we are writing the dask dataframe to parquet files. The process involves creating a separate Parquet file for each partition and column. We construct an execution graph where the final steps involve these write operations.
However, dask does not execute the graph partitions sequentially, leading to inefficient memory usage and out-of-memory errors (related to issue 10480). The write step in the dask computation graph has dependencies on the columns of every partition.

We need to optimize and declutter the write operations for better memory management and stable execution of fondant components.
Option 1: Declutter execution graph
We could consider using delayed objects to simplify the execution graph. This involves wrapping each column write process of each partition into a delayed object, which could potentially enable sequential execution.

Pseudo code for wrapping the write process within a delayed object:
def write_partition_to_parquet(partition, partition_index, output_dir):
        partition.to_parquet(os.path.join(output_dir, f'partition_{partition_index}.parquet'))

# this needs to be done for each column, e.g. COLUMN_df.to_delayed()
for partition_index, partition in enumerate(index_df.to_delayed()):
        delayed_task = delayed(write_partition_to_parquet)(partition, partition_index, output_dir + "/id")
        delayed_tasks.append(delayed_task)

However, I didn't find a practical solution for this yet. The execution of the graph is still not handled sequentially by dask.
Also the graph looks still not optimal, cause we have number_column x number_of_partitions computations steps at the end.
Maybe this could be a short-term solution if we can successfully execute the subgraphs sequentially.
Option 2: Scalable workaround
I have been exploring a potentially more stable solution by leveraging the dask client in combination with a LocalCluster.
Instead of using the dataframe.compute() method, I have been experimenting with dataframe.persist().
This approach triggers asynchronous partition-wise computation and immediately returns a list of pointers containing dask futures.
A dask future is a representation of an asynchronous computation. It can have different statuses, such as "pending" when waiting to complete and "finished" when the computation has successfully finished.
To utilise this approach, I would iterate over the pointers of finished futures, retrieve the computed pandas dataframe, and then write it column-wise to a single parquet file.
In my experiments, this method has proven to be quite stable. It effectively releases memory and can spill data to disk when I'm close to out-of-memory limits.
I have included the source code at the end of the issue for running the experiment locally. You have the flexibility to adjust the number of threads, workers, and the size of the data generated for the simulation.
This approach seems to be the most stable solution, but it does require some foundational changes to the fondant core.
We would have to incorporate the usage of the dask client.
Test code
Requirements:
dask==2023.5.0
distributed==2023.5.0
pandas==2.0.2
dask[diagnostics] # optional
pyarrow==12.0.1
Diagnostics dashboard If you run the code locally, you can access localhost:8787/status to get additional insights into the execution.
import os
import sys
import time
import pandas as pd
from dask.distributed import LocalCluster, wait
import dask.dataframe as dd
from dask.distributed import futures_of
from concurrent.futures import ThreadPoolExecutor
from distributed import Client

# Arguments
output_dir = './tmp'
num_workers = 1
thread_per_worker = 1
memory_limit_per_worker = 2e9 # 2Gb
num_thread_saving_parquets = 1

# Helper functions
def artificial_delay(milliseconds):
    end_time = time.time() + milliseconds / 1000
    while time.time() < end_time:
        _ = 2 + 2  # Perform a simple computation

def process_partition(df, gb_to_produce = 1):
    """Dummy method for processing partition. Large objects are created to simulate oom."""
    print(df.head())
    print("process partition")
    artificial_delay(10000)
    counter = 0
    large_string = "t" * ((1024 * 1024 * 128)) # ~130Mb
    print(sys.getsizeof(large_string))

    to_append = []
    while counter < (10) * gb_to_produce: # x Gb in Total
        counter += 1
        to_append.append([counter, large_string])

    return pd.DataFrame(data=to_append, columns=["id", "url"])

if __name__ == "__main__":
    # Record the start time
    start_time = time.time()

    worker_kwargs = {
        "n_workers": num_workers,
        "memory_limit": memory_limit_per_worker,
        "threads_per_worker": thread_per_worker
    }

    cluster = LocalCluster(**worker_kwargs)
    client = Client(cluster)

    dataframe = dd.from_dict({"id": [0, 1, 2, 3], "url": ["a", "b", "c", "d"]}, npartitions=4)

    dataframe = dataframe.map_partitions(process_partition, meta=dataframe)

    # Getting back a list of futures. Futures are submitted to the local client and will be executed
    # asynchronously. The list contains pointers to futures.
    dataframe = dataframe.persist()

    # List of Dask futures
    futures_list = futures_of(dataframe)

    # Create a dictionary to map futures to partition indices
    # TODO: if partition size is larger as memory, and partition gets assigned to new worker the
    # the futures identifier might be changed
    future_to_partition = {}
    for partition_idx, future in enumerate(futures_list):
        future_to_partition[future] = partition_idx

    with ThreadPoolExecutor(max_workers=num_thread_saving_parquets) as executor:
        while futures_list:
            # Check the status of all futures
            completed, not_completed = wait(futures_list, return_when="FIRST_COMPLETED")

            # Process completed futures
            for future in completed:

                partition_idx = future_to_partition[future]

                # Pandas result for partition is available - and can be accessed immediately
                pandas_df = future.result()

                # Save the pandas dataframe column wise to parquet
                for column_name in dataframe.columns:
                    # Extract the column data as a Pandas Series
                    column_series = pandas_df[column_name]

                    # Create a Pandas DataFrame with the single column
                    column_df = column_series.to_frame(name=column_name)

                    # TODO check filepath is aligned with fondant
                    # Store to parquet file
                    # Create a subdirectory for each partition
                    partition_directory = os.path.join(output_dir, f"{column_name}")
                    os.makedirs(partition_directory, exist_ok=True)

                    output_filename = f"{partition_idx}.parquet"
                    output_path = os.path.join(partition_directory, output_filename)

                    # Save the Pandas DataFrame as a Parquet file
                    column_df.to_parquet(output_path)

                # Remove the completed future from the list
                futures_list.remove(future)

            # TODO:
            # Sleep or perform other work while waiting for more futures to complete
            # Check what could be useful here!

    end_time = time.time()
    elapsed_time = end_time - start_time
    print(f"Elapsed time after saving to parquets: {elapsed_time} seconds")

    # Shutdown client
    client.close()

Seems to be that the local cluster solves it for now. @RobbeSneyders should we close the issue?


Editor window - bounce with tree

I really don't like the 'bounce' when minimising / restoring the tree. Would like transitions to be instant please. Similar, the transition isn't consistent with what happens when you minimise the optional properties panel
By Julian

Done


Update pagintor to support newer Django versions

It appears there is no longer a _count in the Pagination class that RawQuerySetPaginator inherits from. This adds a default _count to the class.

Thank you for your generous contribution.


Test::Quattor::Critic: add Perl::Critic whitelist testing

Requires #114

@jrha and you thought #112 would cause lots of work 😁

also, please don't make any comments on what we should add and what not. if you want certain tests, open a PR, set a milestone and check your agenda when you have time fix the fallout

:see_no_evil:

The perltidy test is way too difficult to tune, and perltidy has no support for whitelisting. You are advised to look at the output and we should discuss final parameters and possible impact in followup PRs


Windows 11 get() returns undefined for same working query on Linux

Hi all,
First of all much appreciation for a great project! I've used it in a few small projects now without issue.
However, I sent some code over to a Windows 11 machine and am experiencing some strange behaviour with getting results. I've been simplifying the code over the last couple of hours and think it may be either a bug or a version issue?
I've simplified my query to the below:

let result = db.prepare(`select * from user`).get();

On my ubuntu machine this correctly returns a user. On my Windows 11 machine this returns undefined
run is working fine and I'm able to add new rows and do updates, just not able to get results on this machine.
Project and machine specs:
windows 11
node v16.14.0
"better-sqlite3": "^7.5.0",

Many thanks for any help or pointers on this!
Side-note:
I've ran the command manually on both machines in sqlitebrowser and they both return results. The db on both machines is the same with the same user account (it's sitting in the project at the the moment). I expect if it could not get a result it would return an empty array or object as opposed to undefined anyway.

Are you using fileMustExist: true? Just making sure the basics are right, because you might just create a new db on Windows without realizing.

Hi Prinzhorn, thanks for taking a look at this. I just booted up the laptop and added the above option. The DB was already there with data though as all other commands were working and I was looking at the DB locally with a sqlite browser.
However, upon rebooting the laptop it's actually working normally now! Not sure what that was about.. I blame Windows 11.


.from(int) and .to(int) in csv feeder (and tsv, ssv) ?

Hi, I have a use case where I have a csv record of around 5k data in a single file and needed to push a certain part of data to each separate scenario, is it possible to have a .from(int rowNumber) and .to(int rowNumber) to quickly push the data ?
My pseudo-code, I think:
val csvFill = csv("my/dir/to/file.csv)
                    .circular //can be looped in range defined
                    .from(rowNumber = 1)
                    .to(rowNumber = 1000)

Currently workaround is to separate csv into smaller files and feed to each scenario, but it's not really good in time aspect.

What you’ve explained is not a use case, it’s a solution you can think of. Why do you need this exactly?
If your actual goal is just to make sure the virtual users of the different scenarios don’t use the same data, you can simply share a single feeder instance.
I’m not fond of the feature as I have a feeling it won’t play nicely with Gatling Enterprise’s shard.

Hi @slandelle
The reason of needing this is to reduce the time to prepare test data, so instead of separating into smaller files for each scenario, user can decide right on one single csv file at which part of data can be passed.
Take my case as an example with sample csvFile content:
column1, column2, conditionData
data1, data_1, condition1
data2, data_2, condition2
...
data5000, data_5000, condition

I have 5 scenarios, one single csv file with 5000 data rows, each scenario I have to pass 1000 data directly into it based on the conditionData column.
So to fulfill my need to pass 1000 data into each scenario, I have to take that 5000 data file, manually split it into 5 csv files, each has 1000 data with satisfy condition, it would be taking some time to do it.

If your actual goal is just to make sure the virtual users of the different scenarios don’t use the same data, you can simply share a single feeder instance.
It was not actually about unique data, but rather affected by a condition that will put V.U into different scenario.

I hope this explanation is cleared enough

The easiest way, as long as your data is not humongous and can fit in memory, is to load all the records and filter them based on your conditionData.
See https://docs.gatling.io/reference/script/core/session/feeders/#read-records


it crashed

Microsoft PowerToys version
10.0.19045.0
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.19045.0
IntPtr Length: 8
x64: True
Date: 12-02-2023 11.16.28 PM
Exception:
System.InvalidOperationException: Dispatcher processing has been suspended, but messages are still being processed.
at System.Windows.Threading.Dispatcher.WndProcHook(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
✔ Expected Behavior
searching for a file
❌ Actual Behavior
it crashed
Other Software
No response

/dup https://github.com/microsoft/PowerToys/issues/18166


Permission denied After mount

Hi, I am getting Permission denied as soon as mount.
I have root, and another user - let's say userA
on root,

create /s3 folder with permission 777 and owner to userA(1001)
checkout to userA  (login as - sudo su userA)
cd /s3 - can access

checkout to root again

mount /s3 folder to amazon s3 with command below

goofys -o allow_other --cache="--free:1%:/tmp" --uid=1001 --debug_fuse --debug_s3 --file-mode=0777 --dir-mode=0777 s3_bucket /s3

manually put file under /s3 -> works all well
checkout to userA
cd /s3 - permission denied
cd /  && ls -la

d?????????   ? ?    ?        ?            ? s3

checkout root again

cd / &&  ls -la

drwxrwxrwx   2 userA root  4096 Nov 12 01:54 s3

umount /s3
checkout userA again
cd / && ls -la

drwxrwxrwx   2 userA root  4096 Nov 12 00:16 s3

conclusion
userA has all permission for s3 folder but lost them as soon as I mount it with s3 - I can see ??? for permissions instead of rwx for s3 folder
Did I miss something..?
All works fine when manually put file under /s3 in root user
but I can access /s3 folder with userA after mount..

Probably, replace --cache="--free:1%:/tmp" with --cache="-oallow_other:--free:1%:/tmp" to fix this situation.

I believe @ikuradon 's suggestion would work


Added smart_path option for file name getters.

https://github.com/tom-anders/telescope-vim-bookmarks.nvim/pull/2
Hey!
So I've done this after the discussion with @Conni2461 and how this would overall benefit the project.
Placing myself at mercy, I've also decided to change the enum logic as well. So that a user doesn't have to deal with importing/exporting/creating enums but configure the options as it is (ie. smart_path = true)
Just to have the preview here as well:

Let me know what you think!

I think the naming is reasonable. as you said, it's smart relative paths that are calculated based on other files names in the context. I wanted to keep the <>_path  naming convention.
Well, this was just a suggestion. Continue as you were
I can comment out the code line by line here to make it more understandable. But basically it saves all files within the context and for each new file function looks at the difference between saved filepaths and current file. then according to their difference, it takes the biggest difference it can take (as in if a/b/c/d, a/b/d differs at index 1) and then adjusts the filenames according to the biggest index it can find, taking the one before it to give context. (../b/c/d/, ../b/d)

I thought it should be a separate implementation as well but I was directed to this by one of the main contributors - which makes it safe to merge in my mind. The function itself can benefit from open source community as well.

I updated my pull request in the main repo to not use enums: nvim-telescope/telescope.nvim#839
I think these changes would benefit Telescope, but I think these should be merged as a separate PR in the main repo in favor of smaller PRs. @Conni2461 would you prefer combining into one PR, or try to merge this separately after the current PR is merged? I can work with either way.

I don't really prefer one over the other. I am fine with both, tbh.
Note: we are squashing PRs so if you want full commit credit you probably want to make a separate PR to telescope :)

Hey!
Wanted to say I don’t really care about that when it’s improving the product I’m using day to day. Thanks for y’all’s effort.
I can do the PR creation after this PR is merged if the owner prefers that way because I feel like that’s the case. Also maybe I can improve it in the meanwhile.
I’ve been using it locally for a while now and it’s working accordingly. there is a bug that happens every once in a while - the second file in the results shows up as full path sometimes when there are 4+ results. (i’m suspecting that there is a bug with initial loop logic) but if you hover over it, it corrects itself (i’m assuming because of rerender).
i can make the logic more solid to prevent such a thing and make it “production ready”. but we are going to go in a release cycle at work in a month or so — but i can make time here and there.
if you guys can give me an approximate merge time that’d be perfect for my planning.
also if things comes down to it @Conni2461 you can copy my code and improve it and push to the main repo. you have my explicit permission.

Hey!
Wanted to say I don’t really care about that when it’s improving the product I’m using day to day. Thanks for y’all’s effort.
I can do the PR creation after this PR is merged if the owner prefers that way because I feel like that’s the case. Also maybe I can improve it in the meanwhile.
I’ve been using it locally for a while now and it’s working accordingly. there is a bug that happens every once in a while - the second file in the results shows up as full path sometimes when there are 4+ results. (i’m suspecting that there is a bug with initial loop logic) but if you hover over it, it corrects itself (i’m assuming because of rerender).
i can make the logic more solid to prevent such a thing and make it “production ready”. but we are going to go in a release cycle at work in a month or so — but i can make time here and there.
if you guys can give me an approximate merge time that’d be perfect for my planning.
also if things comes down to it @Conni2461 you can copy my code and improve it and push to the main repo. you have my explicit permission.

Maybe can merge in a week-ish.
I've linked to this pull request as a TODO from this Telescope wiki, to make sure this is on our radars.

Hey, this is my fork that I use for my own pull requests. You might want to check https://github.com/nvim-telescope/telescope.nvim/pull/914, although seems that pull request is not active.


Don't depend on pkg_resources

Expected Behavior
I should be able to run a command like python manage.py shell without any errors.
Current Behavior
[ERROR] management_commands File "/var/www/example.com/manage.py", line 33: Management command error: manage.py shell
Traceback (most recent call last):
  File "/var/www/example.com/manage.py", line 31, in main
    execute_from_command_line(sys.argv)
  File "/var/www/example.com/venv/lib/python3.12/site-packages/django/core/management/__init__.py", line 442, in execute_from_command_line
    utility.execute()
  File "/var/www/example.com/venv/lib/python3.12/site-packages/django/core/management/__init__.py", line 416, in execute
    django.setup()
  File "/var/www/example.com/venv/lib/python3.12/site-packages/django/__init__.py", line 24, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/var/www/example.com/venv/lib/python3.12/site-packages/django/apps/registry.py", line 91, in populate
    app_config = AppConfig.create(entry)
                 ^^^^^^^^^^^^^^^^^^^^^^^
  File "/var/www/example.com/venv/lib/python3.12/site-packages/django/apps/config.py", line 193, in create
    import_module(entry)
  File "/usr/lib/python3.12/importlib/__init__.py", line 90, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1387, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 995, in exec_module
  File "<frozen importlib._bootstrap>", line 488, in _call_with_frames_removed
  File "/var/www/example.com/venv/lib/python3.12/site-packages/user_sessions/__init__.py", line 1, in <module>
    from pkg_resources import DistributionNotFound, get_distribution
ModuleNotFoundError: No module named 'pkg_resources'

Your Environment

Python version: 3.12.3
Django version: 4.2.11
django-user-sessions version: 2.0.0

Relevant: https://github.com/python/cpython/issues/95299

pkg_resources is used to read the current package version.
We should probably write the version into a file at build time instead with something like:
[build-system]
requires = ["setuptools>=45", "wheel", "setuptools_scm>=6.2"]

[tool.setuptools_scm]
write_to = "user_sessions/version.py"
version_scheme = "no-guess-dev"

The error you're encountering, ModuleNotFoundError: No module named 'pkg_resources', typically occurs when the setuptools package is not installed or is not properly configured in your Python environment. Here are some steps to resolve this issue:
Steps to Fix the Error

Install or Upgrade setuptools:
Ensure that setuptools is installed and up-to-date. You can do this by running the following commands:
pip install --upgrade setuptools

Uninstall and Reinstall setuptools:
If the above step doesn't work, try uninstalling and then reinstalling setuptools:
pip uninstall -y setuptools
pip install setuptools

Upgrade pip, setuptools, and wheel:
Sometimes, upgrading pip, setuptools, and wheel can resolve the issue:
# For Unix/macOS
python3 -m pip install --upgrade pip setuptools wheel

# For Windows
py -m pip install --upgrade pip setuptools wheel

Create a Virtual Environment:
If the issue persists, try creating a new virtual environment and installing the necessary packages within it:
# For Unix/macOS
python3 -m venv my_env
source my_env/bin/activate

# For Windows
py -m venv my_env
my_env\Scripts\activate

# Then install the required packages
pip install django django-user-sessions

Example Configuration for setuptools_scm
If you need to write the version into a file at build time, you can configure setuptools_scm as follows:

Add setuptools_scm to your pyproject.toml:
[build-system]
requires = ["setuptools>=45", "wheel", "setuptools_scm>=6.2"]

[tool.setuptools_scm]
write_to = "user_sessions/version.py"
version_scheme = "no-guess-dev"

Modify __init__.py:
In your __init__.py, use the following code to import the version:
from . import version

__version__ = version.version

By following these steps, you should be able to resolve the ModuleNotFoundError: No module named 'pkg_resources' and run python manage.py shell without any errors.


IPv6 not working for DTLS enabled non debug builds

IPv6 works well for non DTLS builds. But for non debug builds with DTLS enabled, IPv6 fails to register with leshan device management server. Commands used
cmake -DDTLS=1 ../client/ -DCMAKE_BUILD_TYPE=Release
make

Logs attached
Note: IPv6 works for DTLS debug builds.
Code is in sync with wakaama/master.
Compiler: Apple LLVM version 7.3.0 (clang-703.0.29)

Hi,
It is actually any build without LOGS that fails with the same error. Even Debug IPv4 non-DTLS...
I'm looking at this.

It seems #193 fixed this issue.
Could you try on your set-up ?
Regards


[Merged by Bors] - chore(CategoryTheory/Limits/Shapes/Terminal): remove some unnecessary HasLimit and HasColimit hypotheses

Most of the HasLimit F and HasColimit F in the bottom part of the file can be inferred from the rest of the hypotheses.

Thanks!
bors merge

Why was the performance impact of this so bad?


Don't use Time.now in Ruby

The Ruby benchmarks are using Time.now to measure time. The problem with this is that this clock is not monotonic - that means it may unpredictably move both forward and backward in time while the program is running. I'd recommend you always use Process.clock_gettime(Process::CLOCK_MONOTONIC) as a drop-in replacement for Time.now for this purpose. CLOCK_MONOTONIC always moves steadily in the same direction.

Script time already measure by CLOCK_MONOTONIC, Time - internal measure (which teoretically exclude load env, and others). May be add monotonic to internal measure too.

fixed, all things measured by external ruby process with CLOCK_MONOTONIC


center of rotation not restored from settings

Start VisItGo to Controls>View, Advanced tabSet Center of rotation to say "1 1 0" Save SettingsExit VisItvi ~/.config and grep for 'centerOfRotation'. I shows "1 1 0" Restart VisItGo to Controls>View advanced tabCenter of rotation is "0 0 0" It seems like center of rotation is better suited for session content than settings content anyways.
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 1614
Status: Pending
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: center of rotation not restored from settings
Assigned to: -
Category: -
Target version: -
Author: Mark Miller
Start: 10/08/2013
Due date:
% Done: 0%
Estimated time:
Created: 10/08/2013 04:27 pm
Updated: 10/29/2013 04:54 pm
Likelihood: 2 - Rare
Severity: 2 - Minor Irritation
Found in version: 2.6.3
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
Start VisItGo to Controls>View, Advanced tabSet Center of rotation to say "1 1 0" Save SettingsExit VisItvi ~/.config and grep for 'centerOfRotation'. I shows "1 1 0" Restart VisItGo to Controls>View advanced tabCenter of rotation is "0 0 0" It seems like center of rotation is better suited for session content than settings content anyways.
Comments:

Loading the session file works, but I don't see a save option in the View section (maybe it was removed?). @markcmiller86 Is this sufficient to close the ticket?