Added Edit Habit Function

This PR adds the functionality that allows users to edit existing habits on the list. This closes #7.
To edit, the user swipes the habit in question towards the left to spawn an Edit and Delete Button as shown below. To despawn these buttons, swipe towards the right.

The user can click the Edit Button of the habit they wish to Edit, in which they will be prompted with a dialog box which the same one use in the Add Habit function, but with the text fields prefilled with the Habit's values. The user can edit these values and commit them by pressing ok, as shown below.

This PR should now also solve #68. There are some UI issues, like the Edit and Delete buttons not despawning correctly, and when a habit is added, it sometimes isn't appended to the end of the list (this is due to _snapshots mimicking the order of habits in the database, which is based on document title, and not the order in which it is added). However, this does correctly edit the selected Habit both locally and on the database.

@jaskim9824 The order of insertion to the list isn't that big of a deal, but have you tracked/documented the bugs? When do you plan on working on a fix for them?


COPC / attachment merge error handing updates

Correct Reassembly failure handling.

Send COPC ACK after attempting to merge.

The COPC needs to be ACK'd before creating the bundle to ensure MIGRATION_COMPLETE is the most recent migration status


Add docker-compose.yml for local testing

Use httbin as simple backend

Please add a note to the README regarding this docker-compose setup


Appstudio update test-component-pac-elzq

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-elzq-on-pull-request has failed.

StatusDurationName

✅ Succeeded
43 seconds
appstudio-init

✅ Succeeded
26 seconds
clone-repository

✅ Succeeded
29 seconds
sast-snyk-check

✅ Succeeded
23 seconds
appstudio-configure-build

❌ Failed
1 minute
build-container

✅ Succeeded
4 seconds
show-summary

Failure reasontask build-container has the status "Failed":
2023/02/07 09:54:54 Skipping step because a previous step failed

Pipelines as Code CI/test-component-pac-elzq-on-pull-request has failed.

StatusDurationName

✅ Succeeded
43 seconds
appstudio-init

✅ Succeeded
26 seconds
clone-repository

✅ Succeeded
29 seconds
sast-snyk-check

✅ Succeeded
23 seconds
appstudio-configure-build

❌ Failed
1 minute
build-container

✅ Succeeded
4 seconds
show-summary

Failure reasontask build-container has the status "Failed":
2023/02/07 09:54:54 Skipping step because a previous step failed


Center align canvas

Canvas already scales to fit as per .scaleMode, but the canvas is aligned to top-left.
Change this line, and center by default.
https://github.com/NewChromantics/PopLottie.UnityPackage/blob/main/LottieParser.cs#L1045

Don't implement any other modes, as fitting will likely to be covered by https://github.com/NewChromantics/PopLottie.UnityPackage/issues/4
If not though... we may want leading/top/center alignment options.... again, match what options the "background" image has

8ac05cfc3016da6033b0984f9ab64af5774ff6a0
Done in v0.0.2


⚠ Bofrost.com has degraded performance

In bf02551, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 10063 ms

Resolved: Bofrost.com performance has improved in 8d785dd after 13 minutes.


Add method to dump database to CSV files

Is your feature request related to a problem? Please describe.
It would be good to be able to dump database backends into CSV format. This would enable for some flexibility
in how results can be persisted and shared.
Describe the solution you'd like
Dumping the database could take the following form:

a CLI command like pacsanini db dump where each table is dumped to a csv file.
a -o/--output option could be specified to indicate the output directory where these files should be stored.
optional flags corresponding to table names could be set to indicate that only specific tables should be dumped. The default would be all tables.

Describe alternatives you've considered
Having users develop their custom solution.
Additional context

This could be done in the same branch that tackles #62.

Closed with #64.


TLSv1.3 exporter secret is not correct

Current code always passes empty data to tls13_hkdf_expand.
But according to https://tlswg.github.io/tls13-spec/draft-ietf-tls-tls13.html#rfc.section.7.1, it should be a hash value of empty data.

That is not the way I read the spec:
Note that in some cases a zero-length HashValue (indicated by “”) is passed to HKDF-Expand-Label.

So "" mean a zero-length hash value. Not a full length hash value of nothing.

Note: this is not the only instance of this. Derive-Secret(x, x, "")  is used in a number of other places. Interoperability testing with other implementations suggest they have the same interpretation.

I'm not export in reading TLSv1.3 draft, and could be wrong, but the zero-length HashValue is specifically pointing to the following construct:
finished_key =
    HKDF-Expand-Label(BaseKey, "finished", "", Hash.length)

See "" is directly passed to HKDF-Expand-Label.
On the other hand, we are talking about Derive-Secret(Secret, label, "").
By definition:
    Derive-Secret(Secret, Label, "") =
         HKDF-Expand-Label(Secret, Label,
                           Transcript-Hash(""), Hash.length)

This is not the same as HKDF-Expand-Label(Secret, Label, "", Hash.length).

Actually, with the current openssl master, I have interop failure with TLS exporter from mint https://github.com/ekr/mint during IETF QUIC hackathon.

Yes. On reading it again, I think you are correct.

Fixed by c8b93876f1.


Migrate passes to Pass Infra

Migrate current relax passes to the Pass infra.

Merged, thank you @altanh @hypercubestart @ZihengJiang @tqchen for the review! :)


Karpenter consolidation replaces the node with exact same node (EC2 instance) type

Description
Observed Behavior:
Karpenter consolidation replaces the node with exact same node (EC2 instance) type when spec.disruption.consolidationPolicy: WhenUnderutilized is set.  Also, eks-node-viewer doesn't show the node to be deleted/replaced as "Cordoned" - this was atleast the behaviour observed during consolidation with earlier versions of Karpenter.
Expected Behavior:
My understanding is, consolidation should kick-in during below situations for OnDemand instance types:

Delete a node – When pods can run on free capacity of other nodes in the cluster
Deletes a node – When node is empty
Replaces a node – When pods can run on a combination of free capacity of other nodes in the cluster + more efficient replacement node

However, I noticed the "Replace node" happens whenever Karpenter finds the node is underutilized - the node is replaced on continuous basis and with the exact same node type. In my case t4g.nano was replaced with a t4g.nano, the replacement node was not efficient then the original node in any way, rather exactly same. This behaviour made me think the replacement is happening based on utilization only.
Also, the node to be deleted/replaced should be Cordoned first, drained and deleted only after pods are placed onto the new node.
Reproduction Steps:
NodeClass.yaml (karpenter-demo is my Cluster name):
apiVersion: karpenter.k8s.aws/v1beta1
kind: EC2NodeClass
metadata:
  creationTimestamp: null
  name: default
spec:
  amiFamily: AL2
  role: KarpenterNodeRole-karpenter-demo
  securityGroupSelectorTerms:
  - tags:
      karpenter.sh/discovery: karpenter-demo
  subnetSelectorTerms:
  - tags:
      karpenter.sh/discovery: karpenter-demo
status: {}

Nodepool.yaml
apiVersion: karpenter.sh/v1beta1
kind: NodePool
metadata:
  creationTimestamp: null
  name: default
spec:
  disruption:
    consolidationPolicy: WhenUnderutilized
    expireAfter: Never
  limits:
    cpu: 10k
  template:
    spec:
      nodeClassRef:
        name: default
      requirements:
      - key: karpenter.sh/capacity-type
        operator: In
        values:
        - on-demand
      resources: {}
status: {}

Deployment.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  name: inflate
spec:
  replicas: 1
  selector:
    matchLabels:
      app: inflate
  template:
    metadata:
      labels:
        app: inflate
    spec:
      terminationGracePeriodSeconds: 0
      containers:
        - name: inflate
          image: public.ecr.aws/eks-distro/kubernetes/pause:3.7
          resources:
            requests:
              cpu: 1

Screens from eks-node-viewer:

ip-192-168-151-16.us-west-2.compute.internal (t4g.nano) is being consolidated (because it is underutilized?) and replaced with ip-192-168-24-218.us-west-2.compute.internal (again, t4g.nano).

After sometime, ip-192-168-24-218.us-west-2.compute.internal will be again replaced with another t4g.nano instance and the cycle repeats continuously.
Additionally, unlike earlier versions of Karpenter eks-node-viewer doesn't show the node to be replaced as "Cordoned".  Since the logs were rotating fast, it was hard to check if pods were being graciously moved to the new node.
Do I have something misconfigured in NodePool or NodeClass or Deployment manifest? or is this the expected Consolidation behaviour in v1beta1 that needs additional configuration to make it work as expected? If there are no misconfigurations or additional configurations to control this, then this is a potential bug that needs attention.
Versions:

Chart Version:
Kubernetes Version (kubectl version):

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

As far as cordening and draining of selected nodes for disruption are concerned , it will be well handled once the issue https://github.com/aws/karpenter-core/issues/624 is solved

You're using an unreleased version of the beta?

Can you provide some logs?

@badrish-s does your t4g.nano node have any node memory pressure taints?

Apologies. I was on vacation and didn't get a chance to respond to this issue earlier. I am back now, and I did try to set up the Karpenter v1beta1 freshly (using instructions)and applied the same set of nodepools, nodeclass and deployments. I am unable to reproduce the issue now, i.e. the node is NOT being replaced on continuous basis with exact same node type.
I'd like to mention that, the KARPENTER VERSION I am currently using is the latest i.e. v0.32.2 - this was different when I originally tested and reported this issue during October 2023 -  I was testing with an internal only image - v0-2012cf98c2e2e9625e858842c9f2d177efb0c364. I believe I did something incorrect earlier or the issue is been addressed now with latest version v0.32.2. git-hub actions has closed this issue due to inactivity and I will let it remain that way until I see this again (hopefully never). Thanks for looking into this!

Sounds good @badrish-s. Glad to hear that the issue appears to be resolved on the latest version!


Add parameter in executable

Description
That can be add options for executable program parameter
Use Case
Launcher set to directly running game, that could be set some options
Proposed Solution
Add submenu in executable and launcher setting

I'll add the option to include arguments in running programs by appending sub-menus to the path options.

I just came up with something suddenly, but can't we just add an argument to the shortcut?


Add SpannersBoomsticksContinued

Closes https://github.com/KSP-CKAN/NetKAN/pull/2380

LGTM! Merging with Thanks! :D


nodejs: import path resolution

I am making a Docker image for spinning up a dynamic nodejs-server based on a proto file, and it's not finding things like google/api/annotations.proto. If I comment out the import, it works fine. If I download googleapis it resolves it fine, but the include paths get weird after the first import. I have this issue if it's not in Docker, but I thought it might be easiest to reproduce that way.
I'd prefer to not do anything hacky that would mess with protoc (like an empty file or commenting the import, because it is actually being used elsewhere.)
You can download a complete docker project here
I get this error:
Error: failed to import '/usr/src/app/google/api/google/api/http.proto' in '/usr/src/app/google/api/annotations.proto': file not found

Note the repeated directory google/api.
Possible fixes for my usecase:

ability to set include dirs like protoc with -I
proper resolution of paths (which I think will be hard without the includes, but maybe not)
ability to just ignore annotations import, and inline the Well Known Types

I'm a grpc n00b but try,....
  const proto = grpc.load({root: __dirname, file: 'test.proto'});

@pollen8 that still doesn't resolve the issue that I can't include my system-wide copy of googleapis, but if I put it in the same dir as __dirname & test.proto, it does work, which is a great workaround, even if it is a bit of a compromise.
Is it not desired for it to work like protoc? For reference, to generate javascript with protoc, I can do this without issue:
protoc -I/protobuf -I$(pwd) $(pwd)/test.proto --js_out=import_style=commonjs,binary:generated

For now, I can just have a single include dir that has all the google stuff in it, too, and use that as root. I guess it's not a huge deal, and it will work in protoc and grpc.

yes, that's what I ended up doing as well.
It looks like in protobuf6 you can do this
const protoFile = path.join(__dirname, '/protos/myproto.proto');
const root = new protobuf.Root();

root.resolvePath = function(origin, target) {
    return path/to/protofile
};

protobuf.load(protoFile, root, function(err, root) {
....
})

but grpc is currently on protobuf5 and I'm not sure if the grpc.load method will be updated to allow the injection of the root object.

Yeh, we explored some of that here. SO maybe the better question is "How can I help upgrade the dependency version of protobuffjs?"

Actually, I spoke too soon. I made a minimal example that better illustrates what I am trying to accomplish here.
I want to keep my definitions in a namespaced directory, and import them from each other.
When I run it, I get this:
><EMAIL_ADDRESS>start /Users/konsumer/Desktop/grpc_issue
> node index.js

/Users/konsumer/Desktop/grpc_issue/node_modules/grpc/src/node/index.js:124
  return loadObject(builder.ns, options);
                           ^

TypeError: Cannot read property 'ns' of null
    at Object.load (/Users/konsumer/Desktop/grpc_issue/node_modules/grpc/src/node/index.js:124:28)
    at Object.<anonymous> (/Users/konsumer/Desktop/grpc_issue/index.js:5:20)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.runMain (module.js:605:10)
    at run (bootstrap_node.js:420:7)
    at startup (bootstrap_node.js:139:9)

Which is grpc's way of saying "couldn't find a proto file" (see #9652)
No probs if I run protoc like this:
protoc -I$(pwd)/proto/ $(pwd)/proto/demo/rpc/helloworld.proto --js_out=import_style=commonjs,binary:generated

Did you try the above suggestion with {root: 'path/to/proto/', file: 'demo/rpc/helloworld.proto'}? I think that should be functionally equivalent to that protoc command. Alternatively, you could just use protoc. You can get a copy of protoc that comes with the JavaScript gRPC code generator by installing the npm package grpc-tools, and that just requires an extra parameter --grpc_out=out_path to get service files for gRPC.

Aha!
In the example I had this:
const proto = grpc.load({
  file: resolve(__dirname, 'proto/demo/rpc/helloworld.proto'),
  root: resolve(__dirname, 'proto')
})

If I strip off the path for file, it actually works fine. Sort of confusing, because it's not exactly equivalent to protoc command (with full path.) So I ended up with this:
const proto = grpc.load({
  file: 'demo/rpc/helloworld.proto',
  root: resolve(__dirname, 'proto')
})

@murgatroid99 Re: using protoc, I am trying to make dynamic tools that use the proto files directly, so I can generate a client & server from a minimal server implementation + proto file. I want to not have to keep generated stuff in sync with the proto file, and just use the definition directly.

This is a very similar problem to #7524

The @grpc/proto-loader package now supports an includeDirs parameter that solves this problem.


Export value of sample_limit

Proposal
This is a feature request.
Export value of sample_limit so that alerts can be set for scrapes that are approaching the limit without hard-coding the limit in the alert.
I checked the exported metrics on Prometheus 2.3.2 and didn't see this currently exported.

This would be a new additional sample for every single target, which is not something to add lightly from a resource usage standpoint. It's expected that you'd know the values you're using, and not have many of them, so this could be handled fairly sanely via configuration management.

That's a good point. I had forgotten that the limit was set per scape config, not globally.


CLI missing --features flag

First of all, thanks for the dataset!
On your website, you mention a --features flag for the CLI. However, there is no such flag implemented in config.py. Seems like one should use --dataset slowfast8x8_r101_k400. So I guess you should either add the --features flag as a shortcut for this or adjust the website.

Hi Leonard, Exactly right.  We'll update the docs, and please use --dataset with the particular feature in its place.


import_birdflow missing expected contents

Colab run for Chimney Swift:
> bf <- import_birdflow(model_file)
Error in import_birdflow_v2(hdf5 = hdf5, ...) :
  hdf5 file:./chiswi_2020_124km_trained.hdf5 is missing expected contents 'metadata/sparse_stats'

Link to model file (expires in a week):
https://www.dropbox.com/t/DtotFDn3wrliiInE

For future reference the above was preprocessed with:
bf <- preprocess_species("chiswi", gb = 1, skip_quality_checks = TRUE)

I renamed metadata$sparse_stats to metadata$sparse within BirdFlow objects a few weeks ago ( https://github.com/birdflow-science/BirdFlowR/commit/78ce61308cec8bf3e9a8646cfa84dc591db874ad ).   Is there any way you could have preprocessed on an older version of the package and then attempted to import on a newer version?    And out of curiosity if you rerun the above preprocessing and then follow up with:
bf$metadata$sparse
bf$metadata$sparse_stats

What do you get?  With the current package the first should be NA and the second should be NULL.
To do: Update the import function to be less picky about the meta data.  It's not needed to run the models and  they are still in flux.

I was able to import your file with the current (same as demo) BirdFlowR package.   I still think there was something funky going on with package versions for you.  Maybe you preprocessed with a current version of the package and then tried to import with an old? It's hard to imagine that could happen though.   In any case the distributions that the models were trained on were zero for some steps at the end and beginning of the year, which caused the model to break in other ways even after importing.  Those problems can be attributed to skipping the quality checks though.   I'm adding another feature request to allow preprocessing for part of a year.  This is the second time that a species has come up where that seems like it might be a reasonable work around the poor data in the Amazon.


Why does the job configuration of the plugin need to contain the name of the job?

We are generating Jenkins jobs and were wondering why the configuration snippet of this plugin contains the job name:
<org.jenkinsci.plugins.ghprb.GhprbTrigger plugin="ghprb@@1.16-5">
    ...
    <project>JOB_NAME</project>
</org.jenkinsci.plugins.ghprb.GhprbTrigger>

Can't the plugin get the job name in a different way (rather then embedding it into the config snippet)?

I am not sure why this is saved there. There is a project reference in Ghprb.java, and this may be jenkins way of saving a reference to it when it restarts/shuts down.


Insecure network warning

I dont know what is link for but it redirected to page with http causing HTTPS insecure error
https://reporoster.com/stars/alexbelgium/hassio-addons
redirected to
http://bytecrank.com/nastyox/reporoster/php/stargazersSVG.php?user=alexbelgium&repo=hassio-addons
https://github.com/alexbelgium/hassio-addons/blame/master/flexget/README.md#L17

Inspect html shows it loading the img src file

Hi, thanks for noticing this. That's the code to show the number of stars that is hosted on a separate site to run the code... Is that an issue for a readme file?

Not sure why u still need this but this site is already down
http://bytecrank.com/nastyox/reporoster/php/stargazersSVG.php?user=alexbelgium&repo=hassio-addons
If you using https, any link using http will result insecure page warning.

Thanks, I'll check how to solve this

I'm using this site to generate it : https://www.reporoster.com/ probably it has some difficulties for the moment as indeed even though the code generates the links it doesn't show images

Hi, this was reported on their repo here : https://github.com/nastyox/Repo-Roster/issues/5

I've changed the way to reference people having starred and will remove the invalid code. Thanks!

Hi, I also noticed this. It is intentional that the old code is still in there? Because of that there is still mixed-content, which leads to unsafe pages.
https://github.com/alexbelgium/hassio-addons/blame/master/plex/README.md#L17

Well I had left both once it worked again, but I'll remove it then


Import of Armory wallets

Would it be possible to add the ability to import Armory wallets?
There are a number of different ways they can be exported but for starters importing from single-sheet unencrypted paper backups in version 1.35c would probably be the most valuable.
Those consist of a root key printed as two lines with 9 groups of 4 letters each. I've found the creation routine here: https://github.com/goatpig/BitcoinArmory/blob/d1521b476cc14a4174bdc739cc0995eeb7e7e656/extras/PromoKit.py#L107-L113

Armoury is now so outdated that I believe very few will still rely on it to store any funds, so I'm not confident this is a good use of time. If the key can be converted into a WIF, Sparrow can sweep it. Otherwise, closing this off unless there is significant interest.


Can we avoid generating different types for requests, results, and sub-objects?

The swagger-to-api-description script that I wrote used to generate different shapes for each operation, even if two operations involved the same schema in the swagger file.  As a result, there were places in my CPP code where I had to convert between two Model classes that were exactly the same except in name. I went back and improved the script to generate a single shape for each schema in the swagger file because this code to convert between classes was becoming unmaintainable. Much to my chagrin, I learned that the code-generator would still create multiple Model classes for the same shape--a class for when the shape was a request, a class for when the shape was a result, and a class for when the shape was inside another shape.
Is there a way that we can eliminate these separate classes and generate only one class per shape? Alternatively, could we merge just the result and sub-shape classes? (Having separate classes for requests hasn't actually caused any problems.) The only notable difference that I see between JsonResultHeader.vm and JsonSubObjectHeader.vm is what they do with "useRequiredField." I see that this switch controls whether the class includes "HasBeenSet" functions. Does it do anything else? If not, I imagine that we could simply include those functions in the merged class.
An alternative solution to merging the classes might be to give all of them methods that allow conversion to/from JSON. Presently, only the sub-shape class includes the Jsonize() function and the from-JSON constructor. If All of the classes had these methods, then conversions between the classes would be maintainable.

yes, use composition. It will only generate one shape. see aws-cpp-sdk-dynamodb TableDescription and AttributeValue.
Define the structure then put a reference to the structure in your request and result shapes.
Sent from my iPhone

On Feb 1, 2017, at 7:24 PM, Chris Simmons<EMAIL_ADDRESS>wrote:
The swagger-to-api-description script that I wrote used to generate different shapes for each operation, even if two operations involved the same schema in the swagger file. As a result, there were places in my CPP code where I had to convert between two Model classes that were exactly the same except in name. I went back and improved the script to generate a single shape for each schema in the swagger file because this code to convert between classes was becoming unmaintainable. Much to my chagrin, I learned that the code-generator would still create multiple Model classes for the same shape--a class for when the shape is a request, a class for when the shape is a result, and a class for when the shape is inside another shape.
Is there a way that we can eliminate these separate classes and generate only one class per shape? Alternatively, could we merge just the result and sub-shape classes? (Having separate classes for requests hasn't actually caused any problems.) The only notable difference that I see between JsonResultHeader.vm and JsonSubObjectHeader.vm is what they do with "useRequiredField." I see that this switch controls whether the class includes "HasBeenSet" functions. Does it do anything else? If not, I imagine that we could simply include those functions in the merged class.
An alternative solution to merging the classes might be to give all of them methods that allow conversion to/from JSON. Presently, only the sub-shape class includes the Jsonize() function and the from-JSON constructor. If All of the classes had these methods, then conversions between the classes would be maintainable.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I see. I could make that work, but it's not ideal. What's the cause for separate classes for requests and results? I know that an ObjC library generated from an API Gateway doesn't have this quirk.


Aesthetic polish

followed design feedback as best as I could... my text box still doesn't match the figma not sure if that's worth prioritizing

great work!


Migrate to new Sensu core api and types packages

Pending https://github.com/sensu/sensu-go/issues/4875, we will need to rename our api/core/vX and types imports.
Likely warrants a major version bump.

Look like this was done!


Allow Diaper Change without Any Contents

See https://github.com/babybuddy/babybuddy/pull/431.

Closed by 740e929fd711916a10ee90c1abe1b1b8d002557f.


upgrade to JUnit 4.13 RC1

this is the latest release of JUnit4

Hi, thanks for the PR, we are going to keep using the stable version.

obsolete.

what about 4.12.x?

JUnit 4.13.x is expected to be released very soon
https://github.com/junit-team/junit4/issues/1496

ok no problem then, we'll wait.


Revidium incompatability

Mod Name
Revidium
Mod Homepage
https://github.com/TavstalDev/Revidium
Minecraft version
1.20.1
Describe the bug
fatal crash via noref mixin error
Steps to reproduce

build the mod
drop it in the mods folder
start the game
crash

Logs
https://paste.ubuntu.com/p/bwX8r2wqFR/
Additional context
build command used was "bash gradlew build"

normal nvidium works with connector. I added it to my modpack and it worked well

What is the difference between this and Nvidium? If you're looking for a Forge port, try out ascendium.


Error when trying to delete certain values

In my Solid Pod, I have the value:
n:note "First\n\nSecond" ;
This is to demonstrate adding multi-line string values. It works the first time we try to save the data, but not update. Upon some investigation it seems that when updating the value, it skips the DELETE step and just does the INSERT, leading to two values. This happens when we use the replace function.
If I try to explicitly use the delete function instead, I am seeing an error:
Error submitting form TypeError: Cannot read property 'match' of undefined
    at SparqlHandler.createVar (SparqlHandler.js:157)
    at SparqlHandler.pathExpressionToQuery (SparqlHandler.js:95)
    at SparqlHandler._callee$ (SparqlHandler.js:74)
    at tryCatch (runtime.js:63)
    at Generator.invoke [as _invoke] (runtime.js:282)
    at Generator.prototype.(:3000/anonymous function) [as next] (http://localhost:3000/static/js/1.chunk.js:56208:21)
    at asyncGeneratorStep (asyncToGenerator.js:3)
    at _next (asyncToGenerator.js:25)

Tracing this back, I think this is from line 75 of the SparqlHander.js file. It is trying to .match on something called suggestion, which on a delete is hit once with the proper predicate for note, and is hit a second time with undefined.

Root cause: no newline escaping in https://github.com/RubenVerborgh/LDflex/blob/d9893526d39fb16de21bc39d56212708a58befa9/src/SparqlHandler.js#L97.
Secondary cause: assuming that suggestion cannot be undefined


Can a ts_library depend on a .js file?

I have been using a hacky tsconfig.json to combine several .ts and .js files. I'm not sure how to achieve the same result with Bazel TypeScript rules. Is it possible?
{
  "compilerOptions": {
    "target": "es5",
    "module": "system",
    "lib": ["es6"],
    "declaration": false,
    "outFile": "./compiled_typescript.js",
    "strict": true,
    "downlevelIteration": true,
    "allowJs": true
},
  "files": [
    "system-production.src.js",
    "resize.ts",
    "tilespec.ts",
    "app.js"
  ]
}

The rollup_bundle rule mostly works, thanks.
I ended up also adding a concat rule because of a need to export some global definitions. I'll inquire about how to pass more options to rollup_bundle on the other project because my concat rule is trying to accomplish the same thing as passing --intro or --globals to rollup. (Rollup API)
If anyone is curious about the working project, see the BUILD.bazel file for the project.

Good to hear!
I'll certainly add more configurability for rollup and uglify, I'm just waiting to understand the use cases so we expose the minimal configuration affordances. For --globals we already snuck one of them in:
https://github.com/bazelbuild/rules_nodejs/blob/master/internal/rollup/uglify.config.json#L7
and could conceivably hard-code more of them ("global" in the real sense that every Bazel user would get these) but a map-typed attribute seems reasonable too.
--intro I am not sure, as you observe you can easily have another action prepend some text, and a packaging rule is inevitably downstream of the rollup bundle.
Seems like your OP here is resolved, please open issues on rules_nodejs to add the configuration abilities you need.


Doesn't work with Touch Devices

The plugin can not be used with touch devices such as smart phones and tablets. Tapping the EditInOSM  icon has no effect, it doesn't expand the menu.
Testcase:
Open the demo page (http://yohanboniface.github.com/Leaflet.EditInOSM/) with web browser on mobile phone.
Tested on samsung S2 jellybean with Firefox nightly
Clue: Click events?

@yohanboniface any news on this? I am going to use this plugin in a project, If I come up with a solution I will send a pull request with the fix.


[core] refactor updatequeuejobs thread

updatequeuejobs thread should only update running AWs that are not candidates for deletion to move to either completed or RunningHoldCompletion state

PR has been merged and updatequeuejobs thread has been refactored.


New tuning for gazebo model with beter system ID based parameters

Should partially solve #60  and have even better tuning for the velocity controller (which a bit slow now).

I've reverted some of the system ID since we still have to do more research on what are the proper values. but at least it flies more stabily now and reacts better to external twists! Before it was much slower in reaction


exiting failed: no such branch

With a VSCode branch, I created a PR and then tried to exit and am seeing exiting failed: no such branch
https://user-images.githubusercontent.com/29464607/176953171-a92d2c16-8beb-4bf4-9ce7-33e8eeda5788.mov

the result when I tried to checkout main in the terminal


Sell Potions Fix

Selling all potions with one action as it is in the game code.

@SerVamP Thank you for finding this bug. This has been addressed in commit 991dc3f.


Allow dumping any schedule from the CLI.

Previously, we could only dump the Update schedule from the CLI. Now we can do any schedule.
To make this happen, I also redesigned the CLI slightly. Instead of having flags for each command (--dump-render, --dump-update-schedule), I switched them to look like commands. So now users can do cargo r -- dump-render render.dot or cargo r -- dump-schedule Update update.dot. However this does mean we can't do both at the same time. This seems like a reasonable approach though and makes it easier to extend. For example we can have flags that are only available in specific commands (e.g., filter by system set should only be a valid flag if you're running the dump-schedule command).
In a future PR, I think we can also extend this to support sub-apps with an additional flag to the dump-schedule command.

@jakobhellermann is there anything other changes I should make here?

@jakobhellermann is there anything other changes I should make here?

sorry no, it was pretty much good to go, I just forgot about it. I've merged it to main now, with one change to the cli:
It's now
cargo run -- dump-schedule Update | dot -Tsvg out.svg
cargo run --dump-schedule Update -o out.dot

i.e. by default it prints to stdout, and you can pass -o or --output to write to a file.


Fixed issue with Windows Path in tests

When adding xcTestPath to the testEnv path it is possible for testEnv.Path to be undefined. In that case we need to drop back to using process.env.Path

@stevapple @compnerd can one of you verify this.

Hey @0xTim apparently the approval from @compnerd isn't enough to unblock merging. Can you approve?


3DViewer - Learning Path and Documentation are outdated

The Mendix tutorial"Build a 3D Model Inspection App" and the documentation about the 3D Model Viewer are both not up too date.
Build a 3D Model Inspection App
https://academy.mendix.com/link/paths/115/Build-a-3D-Model-Inspection-App
Doc
https://docs.mendix.com/appstore/app-services/3d-viewer
Both instructions state, that in order to get the model viewer to run, you need an on StartUp MicroFlow.
"Add VisServerAction java action as AfterStartup microflow"
But this VisServer Action is outdated:

The current VisServerAction looks like this

So it is unclear how I can setup the 3D Model viewer.
It is okay, if the How To is not up to date (as it recommends using an older version of 3D Model viewer) but the documentation should be up to date / have versions

The issue in the doc is fixed.
I also forwarded the issue in the learning path to academy and connected them with the app service developer. They are working on resolving this issue.


Possibility to update context files

Author Name: OpenNebula Systems Support Team (OpenNebula Systems Support Team)
Original Redmine Issue: 4639, https://dev.opennebula.org/issues/4639
Original Date: 2016-07-13

Right now the only way to change a file in the files datastore is deleting it and adding a new one with the same name. This causes problems with already created VMs. There should be a way to upload new versions of the same file.

Should work on current versions


[Android] Make unit tests pass

Connect-disconnect-connect issue: "Client" is the one initiating the connection, "server" is the one who gets the incoming connection. The issue occurs when client establishes connection, then disconnects and tries to connect again. The BluetoothSocket.connect() succeeds, but the server is completely oblivious to this. The client can write data to socket for some amount, but nothing actually gets through.
The issue is fixed once the server restarts the application. Moreover, recreating Bluetooth Connector Library instances does not help at all.

Now that the reconnect failure has been fixed, I'm seeing the tests stop at info: Running on android test: 94. network changes are ignored while stopping
This doesn't seem to be a problem in Java code. The last thing in logcat is 15921-16275/com.test.thalitest I/jxcore-log﹕ Reconnected to the test server


The (model) and (source) links should support showing line numbers like in a gist

By having the link show line numbers, you can more easily find errors and warnings

I would add these could be new pages that maintain the application instead of using the "raw" output.
Additionally, this should probably have proper back-links and stuff. This could all be achieved via JS on the main page without loading new pages. Prism.js seems like a good start.
Just thinking aloud.

Small concern, the source links now point to their sources rather than stored locally. The model however could use highlighting.


Provide Prometheus retention value

Without a retention value, Prometheus will fill up the disk.
Prometheus expose this value with -storage.local.retention parameter.
The actual value depends on the use case and disk

I think 1 month is a sensible default.

We use a 20GB instance and had to reduce the retention time to 7 days.
for 8 nodes and 7 days our disk usage 5.6GB.
It is something which needed to be taken into consideration...

All I can say is, the number of metrics is going to increase all the time.

@tzach please close this issue or merge the pull request

fix by #250
retention is avilable as a command line option
-b storage.local.retention=1000h


proposal: math: use generics.

While going through math code, I see many functions that's usage could be simplified by using generics.
this will simplify code like this:
var a, b int = 5, 10
var max int
max = int(math.Max(float64(a), float64(b)))

to be just
var a, b int = 5, 10
var max int
max = math.Max(a, b)

because Max declaration could become
func Max[T number](x, y T) T {...}

what the number should be needs to be discussed. My proposal is to make it
type number interface {
   ~int | ~uint | ~float32 | ~float64
}

That would be a backwards incompatible change.
In general, see https://github.com/golang/go/discussions/48287


Fix isBrowser detection

Taking inspiration from https://github.com/akiran/can-use-dom I integrated a better way to handle environment detection.
It fixes an issue I opened yesterday #1551

Can we create a new utils file to prevent the duplication of the logic?

@alltom there it is. You can close the issue I opened once you merge the PR

Hey @oliviertassinari is there still something wrong?

I can't merge, @hai-cea can. Looks like he is on holidays :/

Done! Thanks @izziaraffaele

@shaurya947 I do what I can :)

@shaurya947 Could you please update the npm version? Somehow in 0.12.1 I don't have this PR

@izziaraffaele might have to wait a couple more days before another 0.12.x release.


Implement advanced services

This PR introduces a new interface for services.
We re-implemented the service interface completely. The reloadResources() method is now considered deprecated and should be not used anymore.
Consider the services as a resource for objects of a certain type T. You can now add resource items to the service during the application runtime and make it available to other components in the app.
Components can also subscribe to service update events, in case they are supposed the be updated, once the resource content changes during the app runtime.

 Service component overhaul

I just noticed that the list of available offers is not updated after a new offer was created.


validate tx in mempool in parallel

Describe the feature
during my work on https://github.com/paradigmxyz/reth/pull/6546, i find the tx validation here https://github.com/paradigmxyz/reth/blob/cd0a2f34bcdd0e4ce86c915d3299848c3ba18ac7/crates/transaction-pool/src/validate/eth.rs#L60 is done in serial but it feels like a trivially parallelizable task, @emhane asks me to open an issue to ask for more opinion, if there might be some performance gain, i can open a pr on this.
Additional context
No response

under the hood they are already parallelized via a validation task
currently, we're using 2, but we should actually consider bumping this
https://github.com/paradigmxyz/reth/blob/d7a2123181df49ffda4ff497d6b7fb66e55dc822/crates/transaction-pool/src/validate/task.rs#L48-L54
closing this because we already have a way to parallelize these


Edge documentation says that extension has been moved to V3, but it hasn't

This was originally filed here: MicrosoftDocs/edge-developer/issues/2264
See this documentation: https://learn.microsoft.com/en-us/microsoft-edge/extensions-chromium/getting-started/part2-content-scripts?tabs=v3
It starts with:

To see the completed extension package source for this part of the tutorial, go to MicrosoftEdge-Extensions repo > extension-getting-started-part2.
The source code has been updated from Manifest V2 to Manifest V3.

But that's not true, if you follow that link, it leads to this repo, which contains a manifest v2, not v3.

Thank you @captainbrosset for bringing it here.
Apologies for the inconvenience faced by the developer. We will look into it & make the necessary changes in the docs.

Hi @captainbrosset, we have updated it. You can find the Manifest Version 3 sample code [here].(https://github.com/microsoft/MicrosoftEdge-Extensions/tree/main/Extension samples)


Query order presignature status

This PR adds a SignaturePending status to order for pre-sign orders in the OrderMetadata for signalling that a pre-sign order is waiting for a pre-signature in the GPv2 settlement contract.
Please note that my SQL skillz are :-1:. So please scrutinize the query modification that was made.
Test Plan
Added new unit tests for the added order status computation. Added an ignored test for the Postgres query for presignature status that can be manually run:
$ cargo test -p orderbook -- --ignored postgres_presignature_status --test-threads 1
    Finished test [unoptimized + debuginfo] target(s) in 0.14s
     Running unittests (target/debug/deps/orderbook-68be6fafd7ab0abc)

running 1 test
test database::orders::tests::postgres_presignature_status ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 59 filtered out; finished in 0.06s

Could we rebase this set of branches on main so that it includes the change to SQL query to make reviewing easier?

Could we rebase this set of branches on main so that it includes the change to SQL query to make reviewing easier?

@vkgnosis should be rebased now.


Unable to zoom or translate when using in angular project

I followed the steps as per your sample angular stackblitz example. I get the following error
core.js:4197 ERROR TypeError: Cannot read property 'transform' of undefined
    at TreeChart.zoomed (d3-org-chart.js:1185)
    at SVGSVGElement.<anonymous> (d3-org-chart.js:167)
    at Dispatch.call (dispatch.js:57)
    at Gesture.emit (zoom.js:219)
    at Gesture.zoom (zoom.js:207)
    at SVGSVGElement.wheeled (zoom.js:262)
    at SVGSVGElement.<anonymous> (on.js:3)
    at ZoneDelegate.invokeTask (zone-evergreen.js:399)
    at Object.onInvokeTask (core.js:27424)
    at ZoneDelegate.invokeTask (zone-evergreen.js:398)

I am using this particular version
Angular CLI: 10.0.7
Node: 10.22.0
OS: win32 x64

Angular: 10.0.11
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router
Ivy Workspace: Yes

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.1000.7
@angular-devkit/build-angular     0.1000.7
@angular-devkit/build-optimizer   0.1000.7
@angular-devkit/build-webpack     0.1000.7
@angular-devkit/core              10.0.7
@angular-devkit/schematics        10.0.7
@angular/cli                      10.0.7
@ngtools/webpack                  10.0.7
@schematics/angular               10.0.7
@schematics/update                0.1000.7
rxjs                              6.5.5
typescript                        3.9.7
webpack                           4.43.0

I tried to import index.js into my project and I still face the same error. Is there any quick workaround to fix the issue?

As a work around I tried to download downlaod d3.v5.min.js and index.js from your jsfiddle example and used it in my project. I get the below error, I see this in angular stackblitz example as well.
core.js:4197 ERROR TypeError: Cannot read property 'length' of null
    at Object.T [as max] (d3.v5.min.js:2)
    at Chart.render (index.js:138)
    at D3SampleChartComponent.updateChart (d3-sample-chart.component.ts:52)
    at D3SampleChartComponent.ngAfterViewInit (d3-sample-chart.component.ts:33)
    at callHook (core.js:3038)
    at callHooks (core.js:3008)
    at executeInitAndCheckHooks (core.js:2960)
    at refreshView (core.js:7242)
    at refreshComponent (core.js:8325)
    at refreshChildComponents (core.js:6964)

Ok, Investigating now

Ok, found an issue.
It was trying to initialize the org chart without passing data first.
Here is updated example - https://stackblitz.com/edit/d3-org-chart-angular-integration
I made changes in d3-org-chart.compontent .ts
Please use same d3 version as in stackblitz example

I don't see the issue in your stackblitz example but when I download the example and try to build it on local I get error with (same thing happened yesterday when trying to build)
node_modules/@angular/forms/forms.d.ts(4352,9): error TS1086: An accessor cannot be declared in an ambient context.

i ｢wdm｣: Failed to compile.
Error from chokidar (C:\): Error: EBUSY: resource busy or locked, lstat 'C:\hiberfil.sys'
Error from chokidar (C:\): Error: EBUSY: resource busy or locked, lstat 'C:\pagefile.sys'
Error from chokidar (C:\): Error: EBUSY: resource busy or locked, lstat 'C:\swapfile.sys'

@sai-github quick googling shows that,   that looks like angular versions mismatch.
https://stackoverflow.com/questions/60131331/error-ts1086-an-accessor-cannot-be-declared-in-an-ambient-context-in-angular-9

Yes, I tried changing angular-devkit but no luck. I am surprised on why it is working in stackblitz but but fails on my local?

Probably because @angular/cli,  @angular/core
I will download it once I find a time and will test it if it works locally, will post results here

@bumbeishvili the fix to this problem is to change dependency
From
"dependencies": {
    "d3": "*"
  }

To
"dependencies": {
    "d3": "5.16.0"
  }

d3 has released v6 and if you put "d3": "*" in package.json it'll try to use v6 which is not compatible with<EMAIL_ADDRESS>

@aizaiz I am not sure if whether that's connected to the problem or not. In the stackblitz example, the d3 version is already pinned to 5.15.1 version.
Just for the note: v2 version of this tree  will be based on v6 - https://github.com/bumbeishvili/d3-organization-chart/issues/14

@bumbeishvili tried to reinstall the package on stackblitz example, it'll produce the error

Ah, I thought you were referring to Stackblitz examples only. Thanks for the PR, I will merge it tomorrow and will update the npm version as well

Nice!, will be waiting

Merged and updated npm version


Fix package names for JAL/LUI

For some reason two files have no package. This PR makes things consistent with other instructions.

Thanks, merged!


DynamoDB label attributes inconsistent due to null label module

Describe the Bug
Label created for DynamoDB is different depending on how attributes was passed to this module: as variable or in context
Moving attributes to context results in this plan (unrelated changes snipped):
-/+ resource "aws_dynamodb_table" "with_server_side_encryption" {
      ~ id               = "eg-terraform-state-lock" -> (known after apply)
      ~ name             = "eg-terraform-state-lock" -> "eg-terraform-lock-state" # forces replacementd
      ~ tags             = {
          ~ "Attributes" = "state-lock" -> "lock-state"
          ~ "Name"       = "eg-terraform-state-lock" -> "eg-terraform-lock-state"
            # (1 unchanged element hidden)
        }
      ~ tags_all         = {
          ~ "Attributes" = "state-lock" -> "lock-state"
          ~ "Name"       = "eg-terraform-state-lock" -> "eg-terraform-lock-state"
            # (1 unchanged element hidden)
        }

Seems to be related to cloudposse/terraform-null-label#114 which was released in 0.22.1 while this module pins  to 0.22.0
Expected Behavior
dynamodb_table_label should have same order regardless of how attributes were passed.
Assuming suggested README usage producing desirable result (concat(context, var, ["state"]), label module can be bumped to 0.22.1 or newer
Steps to Reproduce
Steps to reproduce the behavior:

Setup module as suggested in README
Run plan and make note of DynamoDB table resource, particularly id, name and tags
Move attributes variable to context object
Run plan again and compare DynamoDB values

Environment (please complete the following information):
$ terraform version
Terraform v0.15.4
on linux_amd64
+ provider registry.terraform.io/hashicorp/aws v3.42.0
+ provider registry.terraform.io/hashicorp/local v2.1.0
+ provider registry.terraform.io/hashicorp/template v2.2.0

$ terraform get -update
Downloading cloudposse/tfstate-backend/aws 0.33.0 for terraform_state_backend...
- terraform_state_backend in .terraform/modules/terraform_state_backend
Downloading cloudposse/label/null 0.22.0 for terraform_state_backend.dynamodb_table_label...
- terraform_state_backend.dynamodb_table_label in .terraform/modules/terraform_state_backend.dynamodb_table_label
Downloading cloudposse/label/null 0.24.1 for terraform_state_backend.this...
- terraform_state_backend.this in .terraform/modules/terraform_state_backend.this

Additional Context
This module is likely to be used in root module and combined with README instructions this issue is unlikely affect many

@Xerkus the latest version of this module pins v0.24.1 of terraform-null-label. Could your issue be that you're pinned to an older version? Latest is 0.33.0.

terraform get in the log above shows 0.33.0
It still uses 0.22.0 in default branch, here:
https://github.com/cloudposse/terraform-aws-tfstate-backend/blob/793d3f90c25d9f17f4a299be7b13ae5141795345/main.tf#L164-L169


Fix builds, bump to aya 0.12.0 and some cleanups

Stop using aya:main and use the recent 0.12.0
release.
Bump some other crate deps
other small fixups too make this all work.
run cargo fmt

/lgtm
/approve


Support error references in cell ranges

This PR adds support for decoding references with an error. It does this by using a regex to find the sheetname, if any, and handle the optional quotes. I wish JS regex had named groups, in which case this would be a bit nicer.

Might be worth using non-capturing groups to clean up the regex a little and make it obvious which parts are important - /(?:...)/

I added the non-capturing groups, and added a little bit of comments.

Anything I can do to improve this PR to get it merged?

@holm - sorry to take so long - my head was buried in the image branch. Just published this today

Thanks a lot!


Is this normal?

VBoxContainer: 136,100
ERROR: get_edited_scene_root: Index current_edited_scene out of size (edited_scene.size()).
At: tools/editor/editor_data.cpp:500.

Duplicate issue reported here #2654

Closing as duplicate of #2654.


does powa support PG14?

We are planning to implement PG14 wondering if it is supported?

Yes, powa and all the supported extension are compatible with pg 14.

Thank you
Any plans to add it to the yum repository?
On Fri, Oct 8, 2021, 19:14 Julien Rouhaud @.***> wrote:

Yes, powa and all the supported extension are compatible with pg 14.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/powa-team/powa-archivist/issues/44#issuecomment-939206432,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIHWEYGI465ZTZ2ZMVLXNHLUF6QQ7ANCNFSM5FUOMIWA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Oh, I didn't know that the RPM weren't available for pg 14.  I don't see any issue with either powa-archivist or powa-web either in the repo or the yum pgdg redmine, and quick testing shows that it's working as expected.
Maybe there's some thinko on the packaging side.  @devrimgunduz do you have any clue?

@banlex73 I see for instance at https://download.postgresql.org/pub/repos/yum/14/redhat/rhel-8-x86_64/ that the powa packages are present, apparently added on Oct. 19th.  Can you confirm that it works as expected?

I am writing to confirm that those 2 packages available via yum and
working as expected.
Thank you
пт, 11 лют. 2022 р. о 20:37 Julien Rouhaud @.***> пише:

@banlex73 https://github.com/banlex73 I see for instance at
https://download.postgresql.org/pub/repos/yum/14/redhat/rhel-8-x86_64/
that the powa packages are present, apparently added on Oct. 19th. Can you
confirm that it works as expected?
—
Reply to this email directly, view it on GitHub
https://github.com/powa-team/powa-archivist/issues/44#issuecomment-1036989130,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIHWEYDVWCJXVQTLN6DUMZLU2XPZ3ANCNFSM5FUOMIWA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Excellent news, thanks a lot for the confirmation!


Update Static files doc

Fixes
Internal review URL
To review the doc build

download https://www.dropbox.com/sh/yy994551cje0kap/AAB0tRnm7gze2kYrSYPSFHIda?dl=0
unzip
view HTML file in FF (Fire Fox). It best to view the doc with the widest window that has no left or right TOC (Table  of contents)

@serpent5  here's another one when you have time, no hurry.

@Rick-Anderson I should be able to review this over the weekend.

Note the two UseStaticFiles calls in the following example. The first call enables the serving of static files in the wwwroot folder. The second call enables directory browsing of the wwwroot/images folder using the URL https://<hostname>/MyImages.

I commented that out becauses you don't need 2  UseStaticFiles calls  - I removed one., Just wanted to make sure you don't really need two. I tested and it works fine,
Thanks for all your awesome work. Anything else can be done in a follow up PR. I'll have this live in the next hour.


Make transform inspector more compact

Godot 3.1
I find the inspector for transforms is taking a lot of space.
Spatial, Rigidbody, shader params, everytime there is one, it is composed of 3 vertical fields for position, then 3 vertical fields for rotation, then again 3 vertical fields for rotation, AND also a raw 3x4 matrix which is not used often (see #20419).
See the comparison between Godot and Unity, to scale:

We can fit the other one 5 times.
I think a proper transform editor should more look like the Unity one, with the 3 properties fields inline, and eventually have a toggle button to switch to raw mode where the full control represents the raw matrix.
More like:

Or:

I definetely like the proposal. Seems better to me.

I agree that these look good, although I prefer the first one as it doesn't
waste much space.
On Fri, Aug 31, 2018, 5:20 PM Webster Sheets<EMAIL_ADDRESS>wrote:

Both of the proposed layouts look very good indeed. If possible, it would
be even better if the widgets (and the inspector in general) would change
between these two layouts based upon available horizontal space. When there
is large amounts of horizontal space, it would place the Translation/etc.
text on the same line as the vector input, and when there is not enough
space, it would split onto two lines.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/20423#issuecomment-417803649,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEJES7B3BmBFZ9N3HbiMAnTGwpo4AUQKks5uWba0gaJpZM4Ve8Yb
.

It's in master now, if you want to have a look ;)

closing, as this is indeed in Master now.


mkcloud: workaround host grabbing guest devices

so we free up the grabbed mappings
before trying to free the underlying drbd LV

:+1:

Cool, I seem this error the last two days: +1


Allow 'lograge' to be disabled from an env variable

This alters the 'lograge' configuration to enable it to be turned off by setting a LOGRAGE_ENABLED environment variable to false. It remains enabled by default if no environment variable is present.
Lograge is a Ruby gem which changes the default Rails logging format to make it less verbose, instead having a single line of information. Whilst this may be useful for production environments, I've found it to be frustrating in development, as the default Rails logging contains useful information such as the parameters, and each view and partial being rendered.
With lograge:

Without lograge (Rails default):

@slorek I was guessing that the logs were being parsed in this format by some service or another.
I was tempted to default Lograge to be disabled in development for everyone, but didn't want to make this decision without consulting all the other developers first. Probably makes sense though - I can't see any reason why anyone would prefer the lograge format in development vs the more verbose Rails default format.

👌 As we split shared-web apart I want to copy development dependencies into each app so we can avoid this cross-dependency.


PLT-1539: Use foldr for Foldable

Suggested by @effectfully in https://github.com/input-output-hk/plutus/pull/5168#issuecomment-1445268943
This doesn't make any and friends short-circuit, but does reduce the cost (sometimes dramatically) across the board, even more so than this suggestion for improving foldMap.

Seems good to me. I doubt anyone is defining their own Foldable instances, and if they are it's an easy fix.

I'll capture these prettyprinter improvements in a ticket.
I also noticed
data (Monoid :: * -> *) a | Monoid_match where
    CConsMonoid : (\a -> a -> a -> a) a -> a -> Monoid a

That seems messed up.

I just went ahead and did it: https://github.com/input-output-hk/plutus/pull/5184
Not sure what's up with the CConsMonoid thing though!


DBUILD_SHARED_LIBS in Windows is broken for master

When I use this
cmake .. -Dgflags_DIR=/c/devel/gflags/tmp/ -DBUILD_SHARED_LIBS=1

To build glog shared library on Windows I get this error when I try to build the solution:
1>c:\devel\glog\src\stacktrace_windows-inl.h(42): error C2375: 'google::GetStackTrace': redefinition; different linkage

That line is this:
    GOOGLE_GLOG_DLL_DECL
    int GetStackTrace(void** result, int max_depth, int skip_count) {

During build it must be export, during usage it must be import. I think GOOGLE_GLOG_DLL_DECL definition is not complete.

This error is the current master. The release v0.3.5 compiles fine. Issue #103 is related.

That's odd. I was fairly certain I'd tested this scenario (and hence added that GOOGLE_GLOG_DLL_DECL to fix it), must have messed it up though somehow.

@andschwa The GOOGLE_GLOG_DLL_DECL for GetStackTrace had to be added in stacktrace.h instead of the windows implementation header. Also, GOOGLE_GLOG_DLL_DECL is missing in a few places.
I have a patch ready which I will submit soon.

OH! Thank you @sergiud.

@andschwa You're welcome.
I've submitted #227.


Missing parameter scope in the PolicyAssignmentCollection.Get(String, CancellationToken) Method

I'm using this PolicyAssignmentCollection.Get(String, CancellationToken) Method to get an assignment
the document states  Path /{scope}/providers/Microsoft.Authorization/policyAssignments/{policyAssignmentName}, but this method accepts only one argument policyAssignmentName. Thus Scope is not assigned and I get this error promoting missing parameter 'Scope'
When I switched to REST API and feed with both argument, it works. so I think the SDK defintion is missing a parameter Scope

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

ID: ebf8772a-b946-d248-c07e-52a6c324889b
Version Independent ID: 5d810fbd-f147-1f5b-b3a7-e79ec483f2c9
Content: PolicyAssignmentCollection.Get(String, CancellationToken) Method (Azure.ResourceManager.Resources) - Azure for .NET Developers
Content Source: xml/Azure.ResourceManager.Resources/PolicyAssignmentCollection.xml
Service: azure
GitHub Login: @azure-sdk
Microsoft Alias: azsdkdocs

Hi @chenjeyok , Thank you for using Azure SDK for .NET.
Not sure how you get PolicyAssignmentCollection class, but the scope is auto defined from where you get it.
Like when you get the collection from resource group, the scope is the resource group id.

Thanks @HarveyLink this has solved the confusion, thank you very much!


UI: edit node modal requires 1300px width

It'd be nice if the 'edit node' modal degraded gracefully. Anything less than 1300px means horizontal scrolling is required, and under about 1180px the "add disk" and "save" buttons are past the fold, which makes the UI very difficult to use.
The 'node' screen degrades poorly under 1020px, it seems the modal should at work at that pixel width.
I'm on Chrome, Ubuntu, HiDPI, using the v0.7.0 release of Longhorn. Some quick testing gives this matrix:
browser, OS, main 'node' width minimum, node modal width minimum
Chrome, Linux: 1020px, 1300px
Firefox, Linux: 1700px, 1300px
Chrome, MacOS: 1020px, 1300px
Firefox, MacOS: 1520px, 1180px
Safari, MacOS: 1040px, 1315px
Obviously the Firefox experience is less than ideal. In fact on the MacOS version it is wider than my laptop (Mac Air 13", retina) can display.
I didn't see an issue for this though it is tangentially related to #197.

@smallteeths can you check? I am not sure what would be the implications.

Validation: Partially Failed
OS: Ubuntu 16.04
Steps to test:

Adjust screen resolution to 1280*800
In node page, Click on Edit nodes and disks
Edit nodes and disks modal  width has been adjusted to 850px.

Browser: Firefox v71.0 (Failed)

Browser: Google Chrome v79.0.3945.88 (Official Build) (PASS)

Validation: PASSED


qa_crowbarsetup: print return code of failed function

otherwise it can be rather tricky to tell apart failed runs
that all just do exit 30

ok.. although I found the previous if ! .. structure easier to read. anyway, +1

+1


Improve JSON test comparisons

it's really annoying that minor changes cause flips in json values, leading to false failures... find a tool to do a more intelligent json comparison than raw text

maybe kotest-json??

cuz this sucks https://github.com/bkbnio/kompendium/pull/69


Multi-touch motionEvent drag-to-pan interferes with pinch-to-zoom in messageView

Expected behaviour
Both drag-to-pan and pinch-to-move action should be able to operate simultaneously. See Gmail message view, web view on Android or earlier releases of k9
Actual behaviour
With 2-finger touch action, messageView support only drag-to-pan even when user does a pin-to-zoom action
Steps to reproduce

In messageView, touch display with two fingers
Perform pin-to-zoom
messageView perform a drag-to-pan action instead of text zooming

To perform zoom, user must touch the screen with only one finger first to scroll locked the screen. Then with second finger touch and drag, then zoom can only happen. Both then the drag-to-pan is being disabled.
Environment
K-9 Mail version: version 5 but probably earlier. Earlier version with database version 43 is working ok.
Android version: KitKat
Account type (IMAP, POP3, WebDAV/Exchange): All

I believe this is no longer an issue.

yes, the problem has been resolved.


Could not find MsBuild on your machine. Please set the MSBUILD variable to the location of MSBuild.exe and run razzle again.

Steps to reproduce
From A 2019 Dev Prompt Run: in the terminal root
.\tools\razzle.cmd
Expected behavior
environment variables set
Actual behavior

The Powershell commands work it's razzle that does not and I don't know the format or -MSBuildPath.

Honestly, your best bet is going to be to just build it inside VS. We use vswhere to find msbuild, and if your VS install isn't showing up that will be an interesting discussion to have.

Honestly, your best bet is going to be to just build it inside VS. We use vswhere to find msbuild, and if your VS install isn't showing up that will be an interesting discussion to have.

I got it to build using 'nuget restore *.sln' and then msbuild. Razzle  just does not work right now

I got it to build using 'nuget restore *.sln' and then msbuild. Razzle  just does not work right now

set the build config to x64

set the build config to x64

\OpenConsole.sln  /m /p:Configuration=Debug /p:Platform=x64 works there are 8 errors in the master branch.

\OpenConsole.sln  /m /p:Configuration=Debug /p:Platform=x64 works there are 8 errors in the master branch.

For the record, I don't run razzle from a VS dev prompt. I use that from a plain old cmd.exe.

For the record, I don't run razzle from a VS dev prompt. I use that from a plain old cmd.exe.


Enable Azure update management for a VM

Hello,
Azure CLI to deploy the Log Analytics Agent VM extension to an existing virtual machine is not enabling the azure update management so everytime after installation we have to enable in portal
az vm extension set
--resource-group myResourceGroup
--vm-name myVM
--name OmsAgentForLinux
--publisher Microsoft.EnterpriseCloud.Monitoring
--version 1.10.1 --protected-settings '{"workspaceKey":"myWorkspaceKey"}'
--settings '{"workspaceId":"myWorkspaceId"}'
I am not sure if there is already a "az cli" command to enable azure update management but if there isn't one then its good to have a command

@qwordy please take a look

You can do this with PowerShell Az: New-AzAutomationSoftwareUpdateConfiguration, but I don't think there is an azure cli equivalent to that.

It's not supported by Azure CLI now.
https://docs.microsoft.com/en-us/azure/automation/automation-update-management
We have planned it in Azure CLI's breadth coverage plan. Current status is spec ready.
I will update here if there is any update.

@qwordy ， May I know whether the Azure CLI for azure automation is ready now?
if ready, could share the related azcli command or official document about azure update management?
Thanks.

@yonzhan ,
I have noticed you have helped added this into backlog and can I think it means we can expect this in the near future and currently we cannot support manage update management via cli or API?
Thanks~

You can do this with PowerShell Az: New-AzAutomationSoftwareUpdateConfiguration, but I don't think there is an azure cli equivalent to that.

@Icybiubiubiu Now, CLI supports the equivalent commands of PowerShell command New-AzAutomationSoftwareUpdateConfiguration, please try this command automation software-update-configuration


Support NuGet Central Package Management

For solutions using the new NuGet Central Packagement feature, trying the --upgrade option will cause dotnet-outdated to add the package versions to the csproj package references, rather than updating the package versions in Directory.Packages.props, and consequently break the build. A simplified example of how this manifests is shown below.
Updating the tool to support this NuGet feature would allow such projects to leverage the --upgrade option, rather than just needing to manually update using the console's output as a guide on what to update.
Happy to help-out with getting this supported!
Before
Directory.Packages.Props
<Project>
  <ItemGroup>
    <PackageVersion Include="Newtonsoft.Json" Version="12.0.2" />
  </ItemGroup>
</Project>

Project.csproj
<Project Sdk="Microsoft.NET.Sdk">
  <ItemGroup>
    <PackageReference Include="Newtonsoft.Json" />
  </ItemGroup>
</Project>

After
Directory.Packages.Props
<Project>
  <ItemGroup>
    <PackageVersion Include="Newtonsoft.Json" Version="12.0.2" />
  </ItemGroup>
</Project>

Project.csproj
<Project Sdk="Microsoft.NET.Sdk">
  <ItemGroup>
    <PackageReference Include="Newtonsoft.Json">
      <Version>12.0.3</Version>
    <PackageReference>
  </ItemGroup>
</Project>

Yeah, I was going to say that I thought this would "just work" because we shell out to the sdk. NuKeeper for example needed explicit support added, but we should get it here for free.
It's good we have the issue to track it with.

Would it be outrageous to edit the Directory.Packages.target from within the tool and then shell out for a restore?

@steveoh I don't think it would be outrageous. It could be a while before the standard dotnet CLI tools support central package management. If you you are interested in raising a PR in the meantime it would be most welcome.

Now that NuKeeper has deprecated itself and is no longer maintained, This became something of a thorn in our workflow, as we don't use any kind of bots of auto-PR creation tools right now, just developers updating their NuGets in their feature branches as needed.
In our case, we have the "half-way" CPVM system - With csprojs having packages with Include and no Version tags and simply using a Directory.Build.Targets file with packages that have Update and Version tags.
I'm not sure if that's entirely supported either way (Though it did work with NuKeeper's explicit support, as mentioned).
So i raise this again for consideration. As for being able to PR - Can we start a discussion of how this would be implemented, theoratically? So anyone who might be able to contribute will have more of a stepping to start from.

I went ahead and opened PR #243 to add some of the basic "placeholder' support for upgrading CPVM controlled packages as expected by the CPVM preview spec, in lieu of SDK CLI support.

Hello all.
Just wanted to hop in and say that I'd really like this feature to be implemented and would be happy to help any way I can.
I use central package management in all my projects, since it simplifies managing versions on >1 project solutions.
For dotnet-affected, we implemented a feature where we can detect which projects are affected by adding/deleting/updating a nuget package. I think that works great together with dotnet-outdated: you update all your packages, and then in CI or locally you only build/test/publish the projects that were changed/affected by the package changes.
I wanted to share our implementation for reading Directory.Packages.props, which uses MSBuild in order to do so. This supports conditions as well, which is quite nice to have (we use conditions in props files for multi framework targeting)
We also have tests where we create Directory.Packages.props files, see here
Particularly, we ensure conditions are interpreted
This is the code we use for updating packages, I wouldn't use that in production (its for testing only) but perhaps we can get some inspiration for dotnet-outdated.
Should we create a package to share this code between these two tools?
Regards,
Leo.

Is there a PR or something to see where this was implemented?

#243, linked in the comment about which closed the issue.

Hey all. I was taking a very quick look at the PR and I was thinking:

Are multiple Directory.Packages.props supported?
Does it work when packages have conditions in Directory.Packages.props? i.e multiple target frameworks.
Are VersionOverride taken into account?
Is ManagePackageVersionsCentrally respected? May have a Directory.Packages.props but projects ManagePackageVersionsCentrally=false.

We've made several changes at dotnet affected to improve how we evaluate Directory.Packages.props files in order to support these "edge cases".
Regards,
Leo.

Hierarchiel ones you mean? as long as Import tags are used, it should be fine.
Probably not, unless the support for that was outside the area i updated.
Nope - They werne't a thing until after i submitted the PR and the tools weren't updated yet.
The nuget restore outputs give us if a version is centrally managed or not; if thats respected there, it will apply here.

All in all, as the OP says, this was done to add rudimentry suppport until the underlying dotnet CLI fully supports those scenarios.
For myself/the usage we have of this tool internally, we currently don't need the edge cases.
I'm sure other PR contributions will be happily accepted :)

hi @ransagy thanks for the prompt reply. I think this is a great addition to this project.
For the first point I meant this:

What this means is that if you had multiple Directory.Packages.props files in your repository, the file that is closest to your project’s directory will be evaluated for it. This allows you extra control at various levels of your repository.

source
Thanks for clarifying the rest of the points :+1:


Lint crago.lock file when updated

When a PR updates Cargo.lock file, we should ensure that the lock file was generated by cargo command and not tampered with. This could entail a CI that

Gets the current SHA of Cargo.lock generated by PR/user.
Deleting the current Cargo.lock
Generating a new Cargo.lock, possibly calling cargo check command.
Getting the SHA of the new lock file
Ensure the SHA generated by user PR is same as that of CI

Can I try this?

@elliotwutingfeng that would be great! let us know if you have questions along the way.

Thanks! I believe this can be closed now.


:bug: Fix #178

@Schlepptop The tests are failing due to the recent switch from Travis CI to Github Actions. The integration tests should be disabled for PRs but aren't. I'll look into it.

Thanks for the PR!

Closing in favor of #181


Hide generated address and show "Click here to join meeting", or similar text

Hi,
how can we change the meeting link to show another text, ex "Click here to join meeting", instead of the long URL?
I guess I have to change something in jitsi_url_generator.js.
/Pelle

Anything else I can assist with before I close this issue?

Thanks, exactly what I wanted. It solved my question!
const generator = makeUrlGenerator(function (url) {
  $('#url').attr('href', url).text("Click here to join meeting");
}, DEFAULT_DOMAIN, DEFAULT_ROOM_NAME);
/Pelle

Awesome. Glad I could help. :)


Add tests for ApolloInterceptorChain

Closes #442

@sav007 Have made the requested set of changes. If everything else looks fine, we can merge it.


TypeError: can't pickle pyodbc.Connection objects

Environment
Python: 2.7
pyodbc: 4.0.23
OS: UBUNTU (version16.4)
DB: SQL Azure(12.0.2000.8)
driver: Microsoft ODBC Driver 17 for SQL Server
issue:
am getting below error while using multiprocessing.pool with pyodbc connection object.
File "/usr/lib/python2.7/multiprocessing/pool.py", line 567, in get
raise self._value
TypeError: can't pickle pyodbc.Connection objects

Do you have repro code? I want to try and see if I can reproduce this

I'm not sure it makes sense to pickle a connection.  It isn't possible for the actual socket, pipe, etc. to be pickled (the driver has these, not pyodbc) and sent to the child process, so the only option would be to save the connection string and make a new connection when unpickled.
I don't want to keep the original connection string in memory any longer than I have to since it may have sensitive information like a password.
I think it would be better if you pickled the connection string instead and had the called function create a new connection.


Updating to use latest kafka messages artifact

This PR was successfully deployed to production by CircleCI build 67 at 2016-11-30 13:19 GMT.


Allow for verifying when all you have is the root hash and encoding

In some cases it may be necessary to validate a leaf when you only have the root and encoding information. Currently this is not possible. With this new verifyWithRoot static method it is now possible.

Thanks, this is a good suggestion. Could you try to update the docs in the readme with this new function?

@Amxx @frangio I have refactored the method into a single function and made your recommended changes.

I moved the docs section to the top. We should also move StandardMerkleTree.load to the top so all static methods are together but I'll do that after we merge this.
As a few last points, I would rename the function to simplify verify. And the order of arguments seems weird to me. I would change to:
StandardMerkleTree.verify(root, leafEncoding, leaf, proof)

@frangio Makes sense! I have made changes and added a test for invalid proof on the static verify method,


Could not determine the dependencies of task ':flutter_braintree:compileDebugAidl'.

When i am running a flutter project it is giving following error
FAILURE: Build failed with an exception.

What went wrong:
Could not determine the dependencies of task ':flutter_braintree:compileDebugAidl'.

Could not resolve all task dependencies for configuration ':flutter_braintree:debugCompileClasspath'.
Could not find org.jfrog.cardinalcommerce.gradle:cardinalmobilesdk:2.2.4-1.
Required by:
project :flutter_braintree > com.braintreepayments.api:three-d-secure:3.14.2

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 19s
Exception: Gradle task assembleDebug failed with exit code 1
Pubspec.yaml:
flutter_braintree: ^1.1.0+1

THANKS

This is hopefully fixed in version 1.2.1. Let me know if it works!

Issue is back on scene https://github.com/pikaju/flutter-braintree/issues/74

This worked https://developer.paypal.com/braintree/docs/guides/3d-secure/client-side#generate-a-client-token


Allow using a Cross.toml beside another workspace member Cargo.toml

Checklist

[X] I've looked through the issues and pull requests for similar request
[ ] This feature could be solved with a custom docker image (optional)

Describe your request
Currently, if you have a workspace laid out like
.
Cargo.toml
Cross.toml
member/
member/Cargo.toml
member/Cross.toml

and we're currently in member, cross would not use member/Cross.toml, this should be fixed
Describe why this would be a good inclusion for cross
No response

This is closely related to #1158 but in reverse


[WIP] changeview/manage members dialog to use multiselect component consist…

Summary
Change View/Manage members dialog to be consistent with 'Add members to team' dialog
Ticket Link
Issue
Jira
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] Added or updated unit tests (required for all new features)
[ ] Has server changes (please link)
[ ] Has redux changes (please link)
[x] Has UI changes
[ ] Includes text changes and localization file (.../i18n/en.json and .../webapp/i18n/en.json) updates
[ ] Touches critical sections of the codebase (auth, posting, etc.)

Thanks @xjlim for the pull request!
Per the CONTRIBUTING.md file displayed when you created this pull request, we need to add you to the list of approved contributors for the Mattermost project.
Please help complete the Mattermost contribution license agreement?
This is a standard procedure for many open source projects. Your form should be processed within 24 hours and reviewers for your pull request will be able to proceed.
Please let us know if you have any questions.
We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

Right now for both view/manage dialog, user can search to filter the list and navigate up/down on the list. What is the expected behavior when user presses the enter key on the multiselect component?

^ @jasonblais

@xjlim ENTER should open the dropdown menu for the user that's in focus in the Manage Members modal, as it does now.

In View Members modal, ENTER shouldn't take any action.

the up/down navigation might conflict with the dropdown up/down. should up/down not work for the dropdown then

@xjlim Yeah I think the up/down arrows should work on the dropdown.
But you raise a good point as it's not obvious how you can get back to scrolling through the users list. Hmm. Do you have any thoughts on that?
I'll think about it too, I hadn't thought about that case earlier.

i was thinking that the user can 'esc'ape out of the dropdown, but 'esc' by default will dismiss the dialog.

@hmhealey I don't think we'll use multiselect in the modal the same way.
Functionally the only change should be that you can navigate through the users list with up/down arrow keys to change the selected user in the list. My understanding was that this was simplest to achieve by changing the "View/Manage Members" dialog to use the same UI component as "Add Members to Team".
Is there a better way to achieve this?
@xjlim Thanks for your patience here,

@xjlim Terribly sorry for the late response here.
Would it be feasible to have the escape key close the dropdown, without dismissing the dialog? I think that would be most natural. The second escape key would then dismiss the dialog.
If that is not tehcnically feasible, we could consider other options like the left arrow key going back to the user list.
Again, really apologize for the delay here @xjlim. I'll make sure to be more responsive.

@xjlim Let me know if you have any questions about the above comment?

@jasonblais I believe it won't be easy to do the first case as that involves changing bootstrap modal default escape behaviour. I will try the second option.

Hey @xjlim , let me know if there's anything we can help with?

@jasonblais Sorry, I have gotten busy. I will try to pick it up this weekend. I believed I was stuck trying to figure out how to toggle the dropdown of each row within the dialog and found that there are some inconsistency in the design. Some dialogs use the react bootstrap modal wrapper with props to show/hide while others use ref to directly call show/hide the modal instance.

@xjlim No worries - if you have any further questions, don't hesitate to ask in the Contributors or Developers channel at https://pre-release.mattermost.com


fix replacement index,length for terminal completions

adds more tests cases and:
fixes #236256

fixes https://github.com/microsoft/vscode/issues/236257

fixes https://github.com/microsoft/vscode/issues/236254

Can this one complete from after the last /? Otherwise it won't scale it eventually the completion in the widget will get truncated:

like this:
.npmrc
bat/
...

It continues completing if that's what you mean

@meganrogge it should be this:

seems like that's a feature request?  with extension completions disabled, in pwsh, it works like this:

I'm not sure how to do that without breaking things.


YOURLS is accessing the long URL

Technical details

YOURLS version: 1.7.1
PHP version: php5-fpm (php5.5.9)
ShareX version: 11.0.1
Server version: Ubuntu 14.04.4 LTS

Reproducible Bug Summary

Have plugin listed below.

Use API to shorten a URL containing GET variables (i.e. /index.php?var=example)

YOURLS API does not return shortened URL.

Shorten URL with YOURLS

YOURLS accesses the destination of the long URL, leaving a mark as though YOURLS was a user accessing it.

An explanation of my problem. I have YOURLS setup on a home server. I have this plugin (redirect with get) setup with my installation.
I'm currently using ShareX to upload a picture, of which I attempt to shorten a URL containing a GET variable at the end of it. The URL attempting to be shortened works properly. The shortened URL of the long URL works properly.
And at this point are the two problems I'm experiencing, first, ShareX is using the YOURLS API on my server, however, upon shortening the URL while utilizing the above mentioned plugin, it does not return the shortened URL, and thus, ShareX is not able to copy the shortened URL to my clipboard. Instead, I have to access the YOURLS webpage and copy it manually..
And the second problem is that upon shortening a URL, YOURLS accesses the destination of the long URL, and it shows a request similar to that of a user's in the access logs. Is there a way to avoid this?

YOURLS fetches the long URL to retrieve the page title if no title is specified in the API request. It appears there is no setting to override this behavior. However a plugin can short circuit the title retrieval by returning true from a filter on shunt_get_remote_title.
The API issue is beyond my current ability to look into things.

Regarding YOURLS requesting remote URL, see @dgw's comment.
Regarding the other problem, I don't know ShareX and how it works, sorry

I've been looking over the source for both ShareX and YOURLS today. I believe the issue is within the API or the GET plugin.
ShareX has a pretty simple interface with YOURLS, relevant code below, can post more if you need:
(ShareX/ShareX.UploadersLib/URLShorteners/YourlsURLShortener.cs Line 85-95)
                arguments.Add("action", "shorturl");
                arguments.Add("url", url);
                //arguments.Add("keyword", "");
                arguments.Add("title", url);
                arguments.Add("format", "simple");

                result.Response = SendRequest(HttpMethod.POST, APIURL, arguments);
                result.ShortenedURL = result.Response;
            }

            return result;

As for the API, I feel that it has something to do with (includes/functions-api.php Line 17-26 or 102-106) with the yourls_api_action_shorturl() function, or the yourls_api_output() function, given that it's simply not returning a shorturl. As for the plugin's code, it's not very complicated, and it simply changes what's passed as the $url variable to include the query string at the end of the long URL.
Maybe I'm hugely misunderstanding the problem and it does actually lie within ShareX, but at first glance it doesn't seem that way.

Yep, ShareX uses pretty simple code there. And it looks like the title parameter isn't even sent.
It could be instructive, if you have access to the webserver log files, to have a look at the log lines from ShareX's requests to see exactly what arguments it sends. You might need to alter the logging settings to capture POST parameters, but it probably would be useful information to have for debugging. Failing that, I'm not sure how one would go about debugging YOURLS more in-depth (more of a Python guy these days than PHP), but maybe @ozh can provide some pointers if server logs don't help.

In the end, my fault. Not YOURLS or ShareX.
While troubleshooting something else I had changed "ClipboardContentFormat" to "$shorturl" instead of "$result" within ShareX.


Jetbrains IDE [Goland] completely crashes/freezes upon attempt to open new project

Hyprland Version
v0.27.2
Bug or Regression?
Bug
Description
Hello,
First of all, thank you a lot for this nice product! I've been using it for a couple of days and thought that doing some bug-reports could be an initial way of contributing.
Here is my running version:
❯ hyprctl version
Hyprland, built from branch  at commit b08b72358ad549fd066e5be0fc3aa4c9df367607 dirty ().
Tag: v0.27.2

GoLand 2023.2
Build #GO-232.8660.185, built on July 26, 2023
Runtime version: 17.0.7+7-b1000.6 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Linux 6.4.6-arch1-1
GC: G1 Young Generation, G1 Old Generation
Memory: 3970M
Cores: 12
Registry:
    suggest.all.run.configurations.from.context=true
    ide.completion.variant.limit=500
    editor.distraction.free.mode=true
    ide.experimental.ui=true

Non-Bundled Plugins:
    org.toml.lang (232.8660.88)
    IdeaVIM (2.4.1)
    com.intellij.ml.llm (232.9353)

Current Desktop: Hyprland

When working on a project on goland, if I attempt to open a new project, the whole IDE crashes and no longer responds. I cannot even close the window using killactive.  My only option is to open a shell and issue a kill command on the goland process.
I'm not sure if this issue is a hyprland specific or Goland but I would be glad to provide more troubleshoot-information if instructed how.
How to reproduce

open goland;
file -> Open...

Crash reports, logs, images, videos
I have inspected those files and I cannot spot crash reports or interesting things in the /tmp log.  Everything seems to be pretty business as usual, e.g.:
[LOG] Window 55ce998bf480 set title to New Issue · hyprwm/Hyprland - Brave
[LOG] Searching for matching rules for Brave-browser (title: New Issue · hyprwm/Hyprland - Brave)
[LOG] Window 55ce998bf480 set title to New Issue · hyprwm/Hyprland - Brave
[LOG] Keybind triggered, calling dispatcher (64, , 52)
[LOG] Set keyboard focus to surface 55ce99358e50, with window name: tmux
[LOG] Activate request for surface at 55ce99358e50

Some more details:
The application does not seem to actually crash, but the pop-up window that is supposed to ask on whether the user wants to open the project in a new window does not appear: the way I realized that was by using mod+tab to cycle between my workspace windows and realizing that the cycle size was 3 instead of 2 (1 window was hidden and 2 were correctly receiving focus).

The issue also happens when invoking the IDE clone function as at the end of the process a popup window should appear.

jetbrains IDEs are generally terrible on linux due to java's cancerous interactions with X11. No wonder something's up.
I am also not going to pay for an IDE to test.

Hey @vaxerski! thank you for the response.
I would love to contribute as a bridge and report back to Jetbrains what exactly is wrong with their window systems when looked at from a wayland perspective. My only problem is that I'm way ignorant of the whole stack and therefore lack the knowledge and terminology to report the issues in their bug system. If you ever have time to take a look, Itellije IDEA and Pycharm both have a community edition for which there is no pay wall and both of them also have this issue.
I am happy to contribute in any way I can.

I have not looked into jetbrains' stuff for a while now and am not planning to, so I cant give any concrete examples, but wrong atoms being applied to windows, obscure input sniffing methods and more exploitation of Xorg's quitks are often in java gui apps, especially jetbrains.

https://youtu.be/bX5FODgduFs
works for me on a recent git build.
I had a crash on the first start.
but after starting it again, it worked perfectly.

https://youtu.be/bX5FODgduFs
works for me on a recent git build.
I had a crash on the first start.
but after starting it again, it worked perfectly.
maybe it is your tmux?
try just as clean as possible, default config, no gimmicks and add step-by-step more

Thx! I realized that it is now actually also working for me after applying a rule specified in one of the other jetbrain related issues.
I am still fighting with the ctrl-tab switcher which don't lose focus as expected at first. Did you find a workaround for that?

I don't use that functionality at all (I just installed the tool, normally I use PHPStorm),
I use the new UI and have al symbols on the right side and if I want to use shortcuts I use ALT+1.....

They argue it is better but I constantly report errors that happen on their java version but doesn't happen on openJDK.

JetBrains IDE's now have experimental Wayland support
Please try the versions in their Early Acess Program and see if they fix this issue.
Relevant Download links:
https://www.jetbrains.com/idea/nextversion/
https://www.jetbrains.com/pycharm/nextversion/
https://www.jetbrains.com/go/nextversion/

I think we can close this, do crashes in Xwayland and Wayland version,
the ctrl-tab switcher also works on both versions


xActiveDirectory: Add common modules

Pull Request (PR) description

Changes to xActiveDirectory

Added localization module DscResource.LocalizationHelper containing
the helper functions Get-LocalizedData, New-InvalidArgumentException,
New-InvalidOperationException, New-ObjectNotFoundException, and
New-InvalidResultException (issue #257).
For more information around these helper functions and localization
in resources, see Localization section in the Style Guideline.
Added common module DscResource.Common containing the helper function
Test-DscParameterState. The goal is that all resource common functions
are moved to this module (functions that are or can be used by more
than one resource) (issue #257).

Changes to xADReplicationSiteLink

Make use of the new localization helper functions.

This Pull Request (PR) fixes the following issues

Fixes #257

Task list

[x] Added an entry under the Unreleased section of the change log in the README.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[x] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[x] Unit tests added/updated. See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[x] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

Codecov Report

Merging #258 into dev will increase coverage by <1%.
The diff coverage is 100%.

@@         Coverage Diff          @@
##            dev   #258    +/-   ##
====================================
+ Coverage    86%    87%   +<1%
====================================
  Files        19     20     +1
  Lines      1741   1852   +111
  Branches     11     11
====================================
+ Hits       1513   1624   +111
  Misses      217    217
  Partials     11     11


Comments on Python

I see some of the comments regarding Python in the table. I wanted to suggest maybe taking another look at it.

The flask web framework is an extremely popular library in Python. Development has not stalled. The three main Python web frameworks are flask, django, and pyramid.
I may be confused but one of the comments appears to indicate that Python has no standard library. This is not true, Python is known for its "batteries included" philosophy. Additionally, it has thousands of libraries easily installed from the python package index, pypi.org.

Good one! That is something we need to keep in mind during discussions in the future 👍

Closed because we have chosen to use Java and Java frameworks


White rectangles on top menus when clicked using dark mode.

VSCode Version: 1.46.1
OS Version: Linux Mint 19

Steps to Reproduce:

Choose color theme Dark+
Click on any of the top menu items (File, Edit, Selection, etc)
The menus appear first showing a white rectangle then the right content appears maybe 100 ms after the white rectangle, but the white generates a very annoying effect, it's just too bright and breaks the dark mode theme.

Does this issue occur when all extensions are disabled?: Yes

/duplicate of #76850


Add Google VertexAI support.

Support VertexAI framework.
Support detection and error handling.
This pull request closes issue #33

Thanks to your PR, I thought we should abstract out an interface that different api platforms just need to implement instead of changing the code when adding a new platform

I addressed all of your comments with my latest check-ins.  I added an integration test, which requires setting the VERTEX_API_KEY as well as the VERTEX_LOCATION and VERTEX_PROJECT, which would need to be created through GCloud.  One small note, the integration test requires a dependency on "golang.org/x/oauth2/google".
)

Thanks to your PR, I thought we should abstract out an interface that different api platforms just need to implement instead of changing the code when adding a new platform

I feel that the current solution is more invasive to the code, if we later add a new api provider (such as aws), it may also add a lot of if else judgment, the better solution is to abstract a common interface, what do you think? @WillMatthews

Ok, I just refactored everything to use an adapter.  The adapter can contain provider-specific code, while keeping the existing Client implementation handling most of the common logic.  The base code is much cleaner now, and should be fairly extensible to AWS or other providers.

Thank you very much!

Please modify the golines format check error

Done.  I also added some unit tests for Vertex specific code to fix the code coverage.

Thanks to your PR, I thought we should abstract out an interface that different api platforms just need to implement instead of changing the code when adding a new platform

I feel that the current solution is more invasive to the code, if we later add a new api provider (such as aws), it may also add a lot of if else judgment, the better solution is to abstract a common interface, what do you think? @WillMatthews

Whoops - sorry for the slow reply. I agree that an interface is the right approach here. Thank you very much for your work on this PR @steveheyman! It's exciting to now have VertexAI support built in :smiley: