[bug] resource manifest is not validated properly

~I have noticed that some fields from pod template spec do not make it to the actual pods created by the controller. In my case, the environment and workdir fields are dropped.~
Resource manifest is not validated properly. In the following manifest fields workdir and environment are incorrect:
apiVersion: "kubeflow.org/v1"
kind: "TFJob"
metadata:
  name: "mnist"
spec:
  cleanPodPolicy: None
  tfReplicaSpecs:
    Worker:
      replicas: 1
      restartPolicy: Never
      template:
        metadata:
          annotations:
            sidecar.istio.io/inject: "false"
        spec:
          nodeSelector:
            Nodelabel: sandbox-gpu
          containers:
            - name: tensorflow
              image: gcr.io/kubeflow-ci/tf-mnist-with-summaries:1.0
              command:
                - "python"
                - "/var/tf_mnist/mnist_with_summaries.py"
                - "--log_dir=/train/logs"
                - "--learning_rate=0.01"
                - "--batch_size=150"
              workdir: /hello/world
              environment:
                hello: world
              resources:
                limits:
                  nvidia.com/gpu: 1
                  cpu: 1
                  memory: 2Gi

Actual result:
TFJob is saved as is without any error messages. While the actual pods created by the controller lack the invalid fields (which is expected):
spec:
  containers:
  - command:
    - python
    - /var/tf_mnist/mnist_with_summaries.py
    - --log_dir=/train/logs
    - --learning_rate=0.01
    - --batch_size=150
    image: gcr.io/kubeflow-ci/tf-mnist-with-summaries:1.0
    imagePullPolicy: IfNotPresent
    name: tensorflow
    ports:
    - containerPort: 2222
      name: tfjob-port
      protocol: TCP
    resources:
      limits:
        cpu: "1"
        memory: 2Gi
        nvidia.com/gpu: "1"
      requests:
        cpu: "1"
        memory: 2Gi
        nvidia.com/gpu: "1"
    terminationMessagePath: /dev/termination-log
    terminationMessagePolicy: File
    volumeMounts:
    - mountPath: /var/run/secrets/kubernetes.io/serviceaccount
      name: default-token-xscv6
      readOnly: true

As you can see, the environment and workdir fields are missing.
Expected result:
~- all fields from pod template spec are properly propagated to the actual pods~

Resource validation error is thrown
The TFJob resource is not created

If you want to define env vars in TFJob, please refer to this https://kubernetes.io/docs/tasks/inject-data-application/define-environment-variable-container/
envs:
  - name: hello
    value: world

If you want to define env vars in TFJob, please refer to this https://kubernetes.io/docs/tasks/inject-data-application/define-environment-variable-container/
envs:
  - name: hello
    value: world

For work dir, you need to use workingDir.

For work dir, you need to use workingDir.

Oh, my bad. I wasn't paying enough attention to the key names. I was expecting an incorrect manifest would fail validation. Should I file another issue about the validation?

Oh, my bad. I wasn't paying enough attention to the key names. I was expecting an incorrect manifest would fail validation. Should I file another issue about the validation?

You can update the issue title, thanks!

You can update the issue title, thanks!

I have updated the issue title and description accordingly

I have updated the issue title and description accordingly


nginx-config-formatter: Init at 2016-06-16

Motivation for this change
I'd like to automatically format the nginx config file, when generated using structured config. At present the result is quite unreadable.
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@Baughn BTW, I really like this idea. I agree the nginx config files are rough.

The commit log currently reflects the PR's history. Want me to squash it, or will you do that?

yes, squash it

Done.


Access underlying socket object

In an application I'm working on I'd like to check that the socket has flushed data. This can be done using socket.bufferedAmount according to this SO post
Could this library support a function to check this, or simply expose the underlying socket object for these use cases, e.g:
const ws = new Sockette('ws://localhost:3000', {
  timeout: 5e3,
  maxAttempts: 10,
  onopen: e => console.log('Connected!', e),
  onmessage: e => console.log('Received:', e),
  onreconnect: e => console.log('Reconnecting...', e),
  onmaximum: e => console.log('Stop Attempting!', e),
  onclose: e => console.log('Closed!', e),
  onerror: e => console.log('Error:', e)
});

// access underlying socket
ws.socket.bufferedAmount

Hey, you were so close 😉
The underlying socket is available to any & all event handlers. The WebSocket is an event emitter, so the current, up-to-date WebSocket instance is always available as ev.target. You can save that reference and do whatever you'd like with it!
There are a few examples scattered across the issues; https://github.com/lukeed/sockette/issues/12#issuecomment-365170978, https://github.com/lukeed/sockette/issues/31#issuecomment-406741545, and https://github.com/lukeed/sockette/issues/18
Hope that helps!

Man I need to RTFM next time and not just the Example code. Thanks! 👍

No worries!


feat: add resource type

This finally fixes the implementation of the resource type.
Details
Take the following wit file:
interface has_resource {
   func constructor_a () -> a

   resource a {
      func f1()
   }
}

It translates into this host code:
use tauri_bindgen_host::{ResourceTable, ResourceId};

// resources are represented by their own structs, allowing them to keep their own state.
struct ResourceA;

impl has_resource::A for ResourceA {
    fn f1(&self) {
        println!("called resource method!!")
    }
}

struct Ctx {
    // there is no centrally managed resource table as with wasmtime or deno
    // instead callers manage their own resource tables.
    // While this is more verbose it's both much simpler and reduces lock contention (by splitting one big lock into multiple small ones)
    a: ResourceTable,
}

impl has_resource::HasResource for Ctx {
    type A = ResourceA;

    // this is required under the hood so the dispatcher can get the resource
    fn get_a(&self, id: ResourceId) -> ::tauri_bindgen_host::Result<Arc<Self::A>> {
        self.a.get::<Self::A>(id)
    }

    // this is the regular interface function
    fn construct_a(&self) -> ResourceId {
        self.a.push(Arc::new(ResourceA)).unwrap()
    }
}

TODO list

[x] Host implementation
[x] Guest JS implementation
[x] Guest TS implementation
[x] Guest Rust implementation
[x] Markdown implementation

I think @tauri-apps/wg-tauri is better suited for this, however I have not been following the tauri-bindgen closely so I am not sure what I should be looking at to review.
I should not that I needed to have a resource table in https://github.com/tauri-apps/tauri/pull/7709 so I used deno's implementation (but with Arc instead of Rc because of tauri's Send + Sync requirements) so this PR would replace that implementation, right?

struct Ctx {
    // there is no centrally managed resource table as with wasmtime or deno
    // instead callers manage their own resource tables.
    // While this is more verbose it's both much simpler and reduces lock contention (by splitting one big lock into multiple small ones)
    a: ResourceTable,
}

How would this struct look like if I have multiple resources? would it look like this?
struct Ctx {
    a: ResourceTable,
    b: ResourceTable,
    c: ResourceTable,
}

If so, it seems unnecessary, if applying to the menu PR I linked above, where I usually need to use 2~4 different menu types (aka resources), I will have to lock 2~4, instead of just one lock with a central resource table.

Yeah so I slapped this ResourceTable impl together just to have something that works. But you're not forced to use it at all, the idea is that the generated trait forces you to implement a method called get_<resource ident>(&self, id: ResourceId) and that will be used under the hook to determine which resource's mehthods will be called.
It doesn't really matter what structure you use to store the resources as long as you can produce a reference to it when asked.


Update the output message for FreeCommand

Previous message of "xxx was successfully freed from memory" is not accurate. Change this to Alluxio space

Jenkins, test this please

Merged build finished. Test PASSed.

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

@LuQQiu PTAL

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.0 please


[4.5.0] api docs and Jenkins CI fixes

Updated API docs spec-url to work with nginx cos changes.
Fixed cleanup stage to correctly delete the integration test flyway migration file after execution or failure.
Fixed report collection order for integTest stage. So that Newman report is collected even if ZAP has failures

Same failed tests due to cat API change - can be ignored.


Added review model

created review model with id, date, user_id, detail, rating, club_id

@nthungdev I already checked, it's good, you can merge after you check it


Why is DOCKER_BUILDKIT=0 explicitly set?

I would like to build my own packages and need BuildKit. Why is BuildKit explicitly disabled?
https://github.com/dusty-nv/jetson-containers/blob/6f0d65ed3884430d099bf3ffb7777318a6277f9e/jetson_containers/container.py#L105
Thanks for your help.

Apparently BuildKit doesn't allow access to the NVIDIA container runtime which is needed for CUDA libraries. I don't believe there's a workaround, but if there is one it would be nice to know.

Yes, unlike vanilla docker build it does not honor the default docker runtime during container builds.  If you find a workaround or that it is fixed, let me know too!


Super Bare Bones create/open drawer

Adding the NewGalaxyModal.js which will be the panel that opens for the Galaxy configurations (aka create or delete).
this is WIP for create logic and is mostly for getting this file out there to add the NewGalaxyModal which will be used for the delete logic too.

Clicking the create button here will open the side panel - the 'create' button there is disabled as I haven't finished adding the logic for actual creation


[Feature Request] SMTP - Configurable 'from' display name

As an adminstrator, I would like to be able to configure the SMTP 'from' diplay name in config for usability purposes.

@steffendx thoughts?

I've merged your pull request.


New version: Thwani.Uhm version 0.1.4

Pull request has been created with WinGet Releaser v2 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Packages\Thwani.Uhm__DefaultSource\uhm.exe returned exit code: 1
Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Links\uhm.exe returned exit code: 1

(Automated response - build 886.)

https://github.com/microsoft/winget-pkgs/issues/142854


⚠ ClamAV has degraded performance

In efeb9b6, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 67 ms

Resolved: ClamAV performance has improved in 6338784.


SPM error

The package product 'ConstraintKit' requires minimum platform version 12.0 for the iOS platform, but this target supports 9.0
please update package :(

@tobi404 hey 👋
Please use version 1.2.0 of Stack Kit to resolve this problem.


want to support delimiter / syntax for oracle on syntax parsing

System information:

Operating system: macOS 12.3.1 x86_64
DBeaver version:  22.0.4
Additional extensions

Connection specification:

Describe the problem you're observing:

Steps to reproduce, if exist:
CREATE VIEW a1 AS SELECT * FROM dual
/
CREATE VIEW a2 AS SELECT * FROM dual
/

Include any warning/errors/backtraces from the logs

Originally posted by @travelliu in https://github.com/dbeaver/dbeaver/discussions/16448

How to support delimiter / syntax. Does not affect division operations ？
delimiter Syntax /. commonly used in create statements. Such as

Please, try latest DBeaver version. I can't reproduce it against DBeaver 22.2.1

The issue is about the conflict between division operator and delimiter. It's reproducible.

Hopefully, delimiter is supported directly during syntax parsing, and users can easily forget to set delimiters, and conflict between division operator and delimiter


Support Partial Responses

All fields are currently being returned by default.  Supporting partial responses would be helpful:
https://developers.pinterest.com/docs/api/overview/?#partial-responses

I can't think of a good way to handle this without wrecking the interface up.  Right now everything is returning everything always.  Going to close this out until someone perhaps runs into a specific need.


Enhances SpringListener to work correctly with all Spring's Listeners

More information at https://github.com/kotlintest/kotlintest/issues/950
Spring executes transactions (and this include anything inside @DataJPATest) through a Listener called TransactionalTestExecutionListener. This comes pre-registered with a SpringBootTest, and KotlinTest was executing this listener correctly as per #887.
However, only executing the Listener wasn't enough, as it couldn't autowire the EntityManager. Initially, this was happening because our SpringListener had a fake "test method" that was a pointer to java.lang.Object.hashcode method. We made this hack to allow us to execute code inside Spring Test Framework, and it seemed fine on our tests.
Due to hashcode being a java.lang.Object method, Spring couldn't find the transaction attribute, as it verified directly that the method wasn't from Java Object.
With this in mind, we must use an object from another class. My first attempt was to use a dummy method, declared in the SpringListener itself. But this also wouldn't work, as the listener would never find the TransactionAttribute, as the method doesn't (and shouldn't) declare a @Transaction, and SpringListener also didn't declare that annotation. The only way to have a method with that annotation would be inside user's Spec definition, where the user would choose it.
We don't have methods, as we define everything through a DSL. But that didn't seem an issue, as Spring's Listener would get the Transaction Attribute from the method's class if it doesn't declare it. @DataJPATest already declare transaction attributes, so if we managed to get a method inside it, it would work without having to declare @Transaction.
If we could guarantee that our users were overriding a method, we could get its reference, but we couldn't force that, and referecing a method from Spec wouldn't work (as Spec doesn't declare the attributes).
The only way to solve this issue that I found was to inject a dummy method into the class, and get it through reflection, so that Spring Listener would correctly find the Transaction Attributes, and that's what I've done.
Fixes #950

So it tries to find a method in the class, can't find any (because we don't have any) and then returns null, so the entity manager is never autowired in? Shouldn't it / wouldn't it throw some error if it couldn't autowire the class ?
I'm also confused why it's looking for a method in the first place and why having an arbitrary method is enough for it to work. If you have 10 methods in the spec, which one would it pick, would it matter, and if not, why not.

It throws an error in runtime when it can't resolve the entity manager.
It doesn't try to find a method in class, Spring expects us to pass a method to it (as it's used to JUnit, that uses methods). Initially we decided to use any method (hashcode), and most of the Listeners worked.
However, some listeners such as DataJpaTest tries to find specific information in the method description (@Transactional). If it doesn't find it, it tries to search for the class' annotations that provides the information.
It also won't accept any java.lang.Object methods.
The way we had previously was:
method = java.lang.Object.hashcode -> Not working
An alternative would be:
method = Spec.anyMethod -> No @Transaction in it -> Search for it in class -> find nothing -> fails
My fix does the following:
method = Injected method in MySpec -> No @Transaction in it -> Search for it in class -> finds @Transaction or DataJpaTest -> works

So, we need a method with @Transaction, or a method in a class that has @Transaction. The only way to do it is trying to read that from the User class, but it can be done by class, only by method.
It's kinda weird

Thanks thats a great explanation.
And amazing job by the way.

Merging so this doesn't get out of sync.


Allow to supply user object when sending via cloud code

Right now locale callbacks are not working for me, I assume due to the fact that user is always undefined when sending vie Cloud code.
This change would allow to supply the email adapter with the user object when sending an email via cloud code (I hope, haven't tested it yet, but it's a very small change)
happy about any feedback

Thanks for submitting this PR.
Would you please also create an issue and reference it in this PR? It helps us to better understand the issue you discovered and your use case.

sure, done.
I've also testet this change now and it's working for me.
This change allows setting the user when calling the sendEmail function
Parse.Cloud.sendEmail({
templateName: "custom_email",
recipient<EMAIL_ADDRESS>placeholders: request.params.placeholders,
user:  request.user
});

This is missing a test case and some docs, can you give me access to your branch and I add this real quick?

yeah sorry, wasn't sure how to do that stuff :D
You should have access to the fork now

Thanks, I added the tests and did some minor changes.
Can you try out if this branch solves the issue for you?
It is now possible to pass the user and a recipient in sendMail which can seem like a contradiction, because also the user may have an email address:
Parse.Cloud.send({ recipient<EMAIL_ADDRESS>user: aUserWithEmailAddress });

To solve his I added the logic that if recipient is not set, the email address of the user user.get('email') is used as recipient by default.


Crate is incompatible with custon Results

In code that uses custom Result types this crate creates a conflict and cannot be used: instead of returning a Result this crate should consider returning a "std::result::Result".
Example: in user code we might see use std::io::Result which will break this crate.

@Yuri6037 Thanks for raising this issue. Could you try https://crates.io/crates/enum-display-derive/0.1.1 and let me know whether it fixes your issue?


Optimize the selection of pixel format in WGL to early exit if found

Another startup saving on Windows - around ~20ms unless you select a crazy multisampling number that cannot be found.
Also removes a temp buffer allocation.
Same change could be applied to GLX looking at code. (not familiar enough to test myself)

Apologies for not responding yet. I'll need to focus on getting the WebGPU backend finally merged, but after that I'll give the PR a whirl.

PS: I'm not bothering with GLX fixes too much at the moment, since it will probably be dropped with the alternative EGL initialization in place (first step would be to make EGL the default, and after a while drop GLX completely).

Btw... I would suggest creating a dedicated branch for each PR on your side in the future :)
(makes it a bit easier for me to test those changes on my side, e.g. I already have branch called dtrebilco-master here)

Ok merged! I did a couple of small coding style fixes (also one indentation I missed in the last PR):
https://github.com/floooh/sokol/commit/1db04318439b18fa775ce58eee3a75cb4341c763

PS: many thanks for the PR!

OK, I'll try - I am not really up on how I should use github - I just create a fork for each new pull request I do to projects here.
I don't have any other pending changes.
FYI: This was all done/found while doing a line by line conversion to Rust of Windows startup code - I don't plan on taking it further, was just a leaning exercise to see how easy it was or what issues I would have.
https://github.com/dtrebilco/TestLoad/blob/main/src/sapp.rs

Basically, if you are on the master branch of your sokol fork, just create a new branch with a somewhat descriptive name:
git checkout -b wgl-optimize-select-pixelformat

...or I think the new-fangled method is:
git branch wgl-optimize-select-pixelformat

...then implement the change in there and create a pull request from that branch.
Then when the pull request gets merged you can delete that branch and sync your fork from the upstream sokol repo:
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

FYI: I'm seeing a pretty serious problem on my old Asus laptop with integrated Intel GPU with this fix (GL programs are stuck at startup with a white screen). I'll try to investigate, but maybe I need to revert this PR.

The difference between your code and the old code seems to be that the old code returns 28 from _sapp_wgl_find_pixel_format(), and the new code returns 27.
Some sort of off-by-one error?

Ok, the difference between pixel format 27 (which is broken) and format 28 (which works) is the doublebuffer flag.
Format 27 has doublebuffer=false (which doesn't work), and format 28 has doublebuffer=true.

Ah found the issue, the u.doublebuffer here will only ever be set to true, but never reset to false:
https://github.com/floooh/sokol/blob/4eb208b2d9a0eb87567f0817551ecea2934d7bcb/sokol_app.h#L6825-L6827

Ok, fixed in https://github.com/floooh/sokol/commit/85d65418c145b2b809498122d0d2bb75e4a96573

Opps - sorry - initially I had that struct initialized in the loop like the old code when testing.
Pulling it out of the loop I thought was fine.

tbf the original code with the if (...) was a bit wonky too ;)


Fix: Remove useless User\Mapper\UserHydrator

This PR

[x] removes the - what I believe to be useless (unless I'm missing something) - User\Mapper\UserHydrator

Merged, thanks!


Clustering based on tile distance

Hello! I was wondering if it's possible today to create clusters based on anything else other than the pixel distance? Mapbox has a good example where they use clusterRadius. The difference is that clustering based on radius remains consistent irrespective of the projection and zoom level. As a result, clusters are more predictable and nicer looking. When I use the 3d projection in Cesium, clusters shift uncontrollably as I rotate the globe. This is not a desired behaviour since what was previously a cluster of 10 might become a cluster of 6, then 13 as I slowly spin the globe. Here is a video of the desired final effect. If this is not a current feature and not on the roadmap, would you be open to a pull request?

Hello @mrlubos, I don't believe we currently have support for this.  I'll leave this issue open so we can plan to look into it in the future if enough other users request this functionality.  If you have time to look into it and open a pull request, we'd be happy to review it!


Not detecting MATE on Arch

Description
Using MATE 1.16.1 (GTK2, the default install) from Arch. It's not detecting the DE, therefore, not detecting themes, icons, etc.
Neofetch version
2.0.2
Screenshot
Neofetch: https://u.teknik.io/frZB7.png
Mate appearance settings: https://u.teknik.io/e27kH.png
Verbose log
https://p.teknik.io/4DC91

I'm assuming the output of echo $XDG_CURRENT_DESKTOP is empty?
What's the output of xprop -root?

I'm assuming the output of echo $XDG_CURRENT_DESKTOP is empty?
What's the output of xprop -root?

Also, how are you starting MATE?

Also, how are you starting MATE?

@konimex yeah, XDG_CURRENT_DESKTOP is empty.
output of xprop -root: https://p.teknik.io/i1kB3
@dylanaraps starting from lightdm. are you using mate or mate-gtk3? I think on gtk3 it works properly.

@konimex yeah, XDG_CURRENT_DESKTOP is empty.
output of xprop -root: https://p.teknik.io/i1kB3
@dylanaraps starting from lightdm. are you using mate or mate-gtk3? I think on gtk3 it works properly.

screenfetch is detecting correctly, so the information is out there somehow :D
https://u.teknik.io/IEqBm.png

screenfetch is detecting correctly, so the information is out there somehow :D
https://u.teknik.io/IEqBm.png

I'll install lightdm and see if I can reproduce this. :)

I'll install lightdm and see if I can reproduce this. :)

On my Arch system MATE detection (and also KDE) doesn't work, too.
$XDG_CURRENT_DESKTOP is not empty,
but it turned out that in line 539 get_wm is not executed. I have to change it to either
[[ "$wm_run" -ne 1 ]] && get_wm
or
((wm_run != 1)) && get_wm
Then the second problem is line 542. I think it should be de="${XDG_CURRENT_DESKTOP/'X-'}"
Without these two changes $de and $wm are both empty and the function is left in line 559

Thanks @mstraube!
Can you test out #575 and let me know if it fixes the issues?

#583 fix for me! awesome, thanks guys!!

Thanks for testing it. :)

Works for me too.
Sorry for the delay, it was too late yesterday ;)


can mainApplicationFile python file can be fetched from volume mount or directly from S3?

can mainApplicationFile python file can be fetched from volume mount/S3 checkout instead of local in docker images?
Im using spark opertor in kubernetes, instead of spark submit..
if it is local://, then we need to build images with the python files in it, any changes in file we have rebuild the image?
Could you please suggest a way?
S3:
apiVersion: "sparkoperator.k8s.io/v1beta2" kind: SparkApplication metadata:   name: pyspark-pi   namespace: default spec:   type: Python   pythonVersion: "2"   mode: cluster   image: "gcr.io/spark-operator/spark-py:v2.4.5"   imagePullPolicy: Always   **mainApplicationFile: s3:///<bucket>/src/main/python/pi.py**   sparkVersion: "2.4.5"
Or from mounted volume

You can point mainApplicationFile to a file in S3. When you say "from volume mount", what do you mean by that?

mainApplicationFile can be accessed from S3? Can u show some example?
when i tried i m getting the below error
Exception in thread "main" java.io.IOException: No FileSystem for scheme: s3
May be im missing something.. please do provide some example to use S3 in main applicationfile.
If i can use that then i dont. need volume mount...

Yes, you can use mainApplicationFile that is remote in S3 or GCS. If the error message was in the operator logs, it means the operator image is missing necessary jars to handle S3 dependencies. You would need to build a custom operator image with the necessary dependent jars and config.

@ssvinoth22 - I believe this issue of S3 jars can be resolved by adding sparkConf in spec section as per this documentation
Below is an example of my YAML file where I am not seeing the s3 error anymore:
apiVersion: "sparkoperator.k8s.io/v1beta2" kind: SparkApplication metadata:   name: pyspark-pi   namespace: default spec:   type: Python   pythonVersion: "3"   mode: cluster   image: "gcr.io/spark-operator/spark-py:v3.0.0"   imagePullPolicy: Always   mainApplicationFile: s3a://myBucket/input/appFile.py   sparkVersion: "3.0.0"   sparkConf:     "spark.jars.packages": "com.amazonaws:aws-java-sdk-pom:1.11.271,org.apache.hadoop:hadoop-aws:3.1.0"     "spark.hadoop.fs.s3a.impl": "org.apache.hadoop.fs.s3a.S3AFileSystem"     "spark.hadoop.fs.s3a.access.key": "<access-key>"     "spark.hadoop.fs.s3a.secret.key": "<secret-key>"
@liyinan926 - However after deploying this, I am seeing another error as below:
Exception in thread "main" java.io.FileNotFoundException: /opt/spark/.ivy2/cache/resolved-org.apache.spark-spark-submit-parent-d3d506ae-d79f-45f6-b459-cfa5dc649610-1.0.xml (No such file or directory)         at java.io.FileOutputStream.open0(Native Method)         at java.io.FileOutputStream.open(FileOutputStream.java:270)         at java.io.FileOutputStream.<init>(FileOutputStream.java:213)         at java.io.FileOutputStream.<init>(FileOutputStream.java:162)         at org.apache.ivy.plugins.parser.xml.XmlModuleDescriptorWriter.write(XmlModuleDescriptorWriter.java:70)         at org.apache.ivy.plugins.parser.xml.XmlModuleDescriptorWriter.write(XmlModuleDescriptorWriter.java:62)         at org.apache.ivy.core.module.descriptor.DefaultModuleDescriptor.toIvyFile(DefaultModuleDescriptor.java:563)         at org.apache.ivy.core.cache.DefaultResolutionCacheManager.saveResolvedModuleDescriptor(DefaultResolutionCacheManager.java:176)         at org.apache.ivy.core.resolve.ResolveEngine.resolve(ResolveEngine.java:245)         at org.apache.ivy.Ivy.resolve(Ivy.java:523)         at org.apache.spark.deploy.SparkSubmitUtils$.resolveMavenCoordinates(SparkSubmit.scala:1387)         at org.apache.spark.deploy.DependencyUtils$.resolveMavenDependencies(DependencyUtils.scala:54)         at org.apache.spark.deploy.SparkSubmit.prepareSubmitEnvironment(SparkSubmit.scala:308)         at org.apache.spark.deploy.SparkSubmit.org$apache$spark$deploy$SparkSubmit$$runMain(SparkSubmit.scala:871)         at org.apache.spark.deploy.SparkSubmit.doRunMain$1(SparkSubmit.scala:180)         at org.apache.spark.deploy.SparkSubmit.submit(SparkSubmit.scala:203)         at org.apache.spark.deploy.SparkSubmit.doSubmit(SparkSubmit.scala:90)         at org.apache.spark.deploy.SparkSubmit$$anon$2.doSubmit(SparkSubmit.scala:1007)         at org.apache.spark.deploy.SparkSubmit$.main(SparkSubmit.scala:1016)         at org.apache.spark.deploy.SparkSubmit.main(SparkSubmit.scala)
Could you please suggest if I'm missing something ?
Thanks in advance! :)

We got the same error.
sparkConf:
"spark.jars.packages": "org.apache.hadoop:hadoop-aws:3.3.0"

Ivy Default Cache set to: /opt/spark/.ivy2/cache
The jars for the packages stored in: /opt/spark/.ivy2/jars
:: loading settings :: url = jar:file:/opt/spark/jars/ivy-2.4.0.jar!/org/apache/ivy/core/settings/ivysettings.xml
org.apache.hadoop#hadoop-aws added as a dependency
:: resolving dependencies :: org.apache.spark#spark-submit-parent-643244df-1233-45fc-b34c-b7c5259e62db;1.0
confs: [default]
Exception in thread "main" java.io.FileNotFoundException: /opt/spark/.ivy2/cache/resolved-org.apache.spark-spark-submit-parent-643244df-1233-45fc-b34c-b7c5259e62db-1.0.xml (No such file or directory)
at java.io.FileOutputStream.open0(Native Method)
at java.io.FileOutputStream.open(FileOutputStream.java:270)
at java.io.FileOutputStream.(FileOutputStream.java:213)
at java.io.FileOutputStream.(FileOutputStream.java:162)
at org.apache.ivy.plugins.parser.xml.XmlModuleDescriptorWriter.write(XmlModuleDescriptorWriter.java:70)
at org.apache.ivy.plugins.parser.xml.XmlModuleDescriptorWriter.write(XmlModuleDescriptorWriter.java:62)
at org.apache.ivy.core.module.descriptor.DefaultModuleDescriptor.toIvyFile(DefaultModuleDescriptor.java:563)
at org.apache.ivy.core.cache.DefaultResolutionCacheManager.saveResolvedModuleDescriptor(DefaultResolutionCacheManager.java:176)
at org.apache.ivy.core.resolve.ResolveEngine.resolve(ResolveEngine.java:245)
at org.apache.ivy.Ivy.resolve(Ivy.java:523)
at org.apache.spark.deploy.SparkSubmitUtils$.resolveMavenCoordinates(SparkSubmit.scala:1387)
at org.apache.spark.deploy.DependencyUtils$.resolveMavenDependencies(DependencyUtils.scala:54)
at org.apache.spark.deploy.SparkSubmit.prepareSubmitEnvironment(SparkSubmit.scala:308)
at org.apache.spark.deploy.SparkSubmit.org$apache$spark$deploy$SparkSubmit$$runMain(SparkSubmit.scala:871)
at org.apache.spark.deploy.SparkSubmit.doRunMain$1(SparkSubmit.scala:180)
at org.apache.spark.deploy.SparkSubmit.submit(SparkSubmit.scala:203)
at org.apache.spark.deploy.SparkSubmit.doSubmit(SparkSubmit.scala:90)
at org.apache.spark.deploy.SparkSubmit$$anon$2.doSubmit(SparkSubmit.scala:1007)
at org.apache.spark.deploy.SparkSubmit$.main(SparkSubmit.scala:1016)
at org.apache.spark.deploy.SparkSubmit.main(SparkSubmit.scala)
stream closed

Same problem here. I'm really struggling with this. re:

You would need to build a custom operator image with the necessary dependent jars and config.

I tried going down this route, but I'm utterly stumped as to how to add the necessary jars. I can build spark from source, but how do I add in the dependency of hadoop-aws while ensuring transitive dependencies also get installed?

Did anyone get a handle on this problem? I am also having the same issue

@plathrop , can try this link : https://github.com/aws-samples/eks-spark-benchmark/tree/master/docker
It shows how to add the dependency of hadoop-aws  to an already built image that has spark.

Hi all,
Apologies for not being able to reply earlier. You can try out this Dockerfile shared by @bbenzikry which has s3 packages installed in it. By using this Dockerfile, you won't need to add S3 packages using YAML file, which will avoid the Ivy Default Cache set to: /opt/spark/.ivy2/cache errors. Attaching sample Dockerfile and YAML file that I used for reference.
app_yaml.txt
Dockerfile.txt

Thanks for the reply. I did end up building my own images, with slightly different takes on the above solutions.

I had a similar problem (Exception in thread "main" java.io.IOException: No FileSystem for scheme: gs) getting mainApplicationFile and dependencies from google cloud storage.
End up building a new spark-operater image with necessary dependencies (gcs-connector-hadoop3-2.2.0-shaded.jar) and modified the helm chart by adding a secret holding the service account key to access gcs to charts\spark-operator-chart\templates\deployment.yaml


when I write something in the TextField,the page  will reload and the keyboard just  appear for a second and disapear.

  @override
  Widget build(BuildContext context) {
    super.build(context);

    return Scaffold(
        appBar: AppBar(
          title: Text(' '),
        ),
        body: FutureBuilder(
          builder: (context, snapshot) {
            switch (snapshot.connectionState) {
              case ConnectionState.none:
              case ConnectionState.active:
              case ConnectionState.waiting:
                print('Has error: ${snapshot.hasError}');
                print('Has data: ${snapshot.hasData}');
                print('Snapshot data: ${snapshot.data}');
                return Center();
              case ConnectionState.done:
                if (snapshot.hasError) {
                  return Center(
                    child: Text('something wrong),
                  );
                }
                return Container(
                  color: Colors.grey[200],
                  child: Column(
                    children: <Widget>[
                      Expanded(
                        child: ListView(
                          children: <Widget>[

                        Container(
      color: Colors.white,
      margin: EdgeInsets.only(top: 5),
      child: ListTile(
        leading: Text('sonthing remark'),
        title: TextField(
          controller: _controller,
          decoration: InputDecoration.collapsed(
            hintText: 'enter somthing',
          ),
        ),
      ),
    )

Beside, when I delete the ConnectionState, the problem won't appear. Could anyone help me fix this problem?

Please provide the output of flutter doctor so that it's clear what version of Flutter you're running and what device you're targeting.

Closing since without an answer
@Cyurainya Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow.
Thank you


Add license header to top of all code files

See https://opensource.google/docs/releasing/licenses/
We can either use;
Copyright (C) 2020  The Project X-Ray Authors.

Use of this source code is governed by a ISC-style
license that can be found in the LICENSE file or at
https://opensource.org/licenses/ISC

Or put the full license at the top;
Copyright (C) 2020  The Project X-Ray Authors.

Permission to use, copy, modify, and/or distribute this software for any
purpose with or without fee is hereby granted, provided that the above
copyright notice and this permission notice appear in all copies.

 IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.

Full process -> https://github.com/SymbiFlow/ideas/issues/47


way-shell: init package and module

Description of changes

way-shell: init at 0.0.7
nixos/way-shell: init

Continuation of #329677 under the express permission of @eclairevoyant. I have
chosen to adapt this PR (and I will a few others) in correlation to my own
workflows. I will also be taking over the mantle of maintainer for PRs that init
new packages.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See
Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits
CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

👋 Thank you for this, if you need any assistance or modification to code to aide this process just LMK.

I appreciate the offer, will make sure to let you know :)

Hello, just a heads up, A new release was just cut: https://github.com/ldelossa/way-shell/releases/tag/v0.0.8. I am not sure if this affects this packaging at all, but wanted to make it visible.

Thanks for letting  me know, I'll bump the package version shortly.

I have been using this PR with the latest release (0.0.10 at the moment) and it works great c:


Duplicated registrations with Custom Sql Storage

When you register a webhook receiver with sql storage it allows duplicated registers of the same receiver, causing it to receive multiple messages when the event he was listening to occurs.
I use this example:
https://blogs.msdn.microsoft.com/webdev/2015/11/07/updates-to-microsoft-asp-net-webhooks-preview/
This is a bug or I have made somthing wrong?

You mean that users can register the same WebHook URI multiple times? Yes, that's correct, we don't prevent the same URI to be registered multiple times.


Unable to move or install chroot on SD Card

Please paste the output of the following command here: sudo edit-chroot -all
chronos@localhost ~/Downloads $ sudo edit-chroot -all
No chroots found in /mnt/stateful_partition/crouton/chroots

Please describe your issue:
I moved my chroot from my internal drive to my SD Card and the task completed successfully.  When I run sudo startxfce4, I receive the same error as above "No chroots...."
When I attempt to update the chroot, I receive the error "Too many symbolic links".  See output below
Also, when I follow this guide, How-To-Install-To-External-Drive, I receive the same error during a fresh installation for "Too many symbolic links"
sudo crouton -n kali-rolling -p /media/removable/SD\ Card -u
/media/removable/SD Card/chroots/kali-rolling already exists; updating it...
Preparing chroot environment...
Installing brightness into the chroot...
Installing croutonpowerd into the chroot...
Installing croutonversion into the chroot...
Installing host-dbus into the chroot...
Installing host-wayland into the chroot...
Installing volume into the chroot...
Installing pulseaudio-default.pa into the chroot...
Installing croutoncycle into the chroot...
Installing croutontriggerd into the chroot...
Installing croutonxinitrc-wrapper into the chroot...
Installing setres into the chroot...
Installing xinit into the chroot...
Installing xbindkeysrc.scm into the chroot...
Installing xorg-intel-sna.conf into the chroot...
Installing xserverrc into the chroot...
Installing xserverrc-xorg into the chroot...
Installing xserverrc-local.example into the chroot...
Installing startxfce4 into the host...
Installing crouton-noroot into the chroot...
Installing startxfce4 into the chroot...
cp: failed to access '/var/run/crouton/media/removable/SD Card/chroots/kali-rolling/usr/local/bin/startxfce4': Too many levels of symbolic links

@duck955,

I moved my chroot from my internal drive to my SD Card and the task completed successfully.

If you are just having trouble launching your desktop from external media them take a look at this comment, it may help with your issue:

sudo: startxfce4: command not found from SD CARD HP 11A G6 · Issue #4358 · dnschneid/crouton

Hope this helps,
-DennisLfromGA

@DennisLfromGA  thank you for the reply
I ran the start command based on the issue you linked and with the help menu for startxfce4.  Below is my output.
chronos@localhost /media/removable/SD Card $ sudo startxfce4 -c /media/removable/SD\ Card/chroots
Entering /media/removable/SD Card/chroots/kali-rolling...
chroot: failed to run command 'su': Too many levels of symbolic `links`
WARNING: starting chroot system dbus daemon failed with code 126
chroot: failed to run command 'su': Too many levels of symbolic links
Unmounting /media/removable/SD Card/chroots/kali-rolling...

Additionally, when following the xfce4 wiki, the setup will not complete.  It also fails with too many symlinks.

sudo enter-chroot /media/remove/sd card/chroot/chrootname?

I just ran into this same problem with a chroot on SD card.  Spent far too long troubleshooting, but it looks like the problem (at least for me) is that the SD card is being mounted with the nosymfollow option:
# mount|grep 'SD Card'
/dev/mmcblk1p1 on /media/removable/SD Card type ext4 (rw,nosuid,nodev,relatime,nosymfollow,dirsync,seclabel,data=ordered)

If I remove that with mount -o remount,symfollow /media/removable/SD\ Card/ everything works again.  The nosymfollow option prevents following of symlinks, which triggers a confusing message with chroot.  I was already remounting the SD card to remove the noexec flag so I just added this to my first-run-after-reboot ritual.  Minor inconvenience now, but I wasted a lot of time hunting down the problem thanks to useless error messages.
This seems to have started for me after a recent ChromeOS update, so I guess Google started mounting the SD cards differently and broke everything?  Not sure when this started but I'm currently on beta channel, running 88.0.

+1

This solution worked for me, thanks!


Index page linking correctly

Navbar logo linking to the index page.
Trending page and index page navbar home option linking to the index page

hey @Ankur-lohiya, please consider raising an issue regarding your changes and after getting assigned you can work on it and create a PR

hey saloni, Thanks for accepting my previous PR and from now on i will keep
this point in my mind.
On Sat, Oct 2, 2021 at 11:08 AM Saloni Subedi @.***>
wrote:

hey @Ankur-lohiya https://github.com/Ankur-lohiya, please consider
raising an issue regarding your changes and after getting assigned you can
work on it and create a PR
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/saloni-15/Heel2Toe/pull/50#issuecomment-932686964,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKDLZDAWLSUA676WZYTXBDDUE2LGDANCNFSM5FFIYF6Q
.
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.


Update httpcore5 to 5.2.2

About this PR
📦 Updates org.apache.httpcomponents.core5:httpcore5 from 5.2 to 5.2.2
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (5.2).
You might want to review and update them manually.
build.sbt
src/main/resources/logback.xml

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.apache.httpcomponents.core5", artifactId = "httpcore5" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.apache.httpcomponents.core5", artifactId = "httpcore5" }
}]

labels: library-update, old-version-remains, commit-count:1

Superseded by #668.


Math button shortcuts for people who don't know TeX

I was thinking how convenient would it be for people who don't know TeX (yet) to have visual buttons to input mathematical notations and equations. There is a trade off between minimalism and features, I know. But a full GUI based Math editor is required for a full featured text editor.
@benweet Would you like to add that feature, or else, will you accept pull requests that implements these features?

@akerbos , @benweet , @juzerali
How about an extension that provides a list of matchig of latex commands, when when user starts typing  a particular latex command. Ex: $\left...$ (then a popup appears and suggests \leftarror, \leftbracket etc). Then user can navigate with arrow down and select by enter or space.
I think this could not only help beginners, but speed the work of advanced users too.

My idea for opening this ticket was to assist users who have absolutely no
experience with latex. What you are suggesting is helpful to experienced
users.
On Sun 1 Mar, 2015 6:42 am victorel-petrovich<EMAIL_ADDRESS>wrote:

@akerbos https://github.com/akerbos , @benweet
https://github.com/benweet , @juzerali https://github.com/juzerali
How about an extension that provides a list of matchig of latex commands,
when when user starts typing a particular latex command. Ex: $\left...$
(then a popup appears and suggests \leftarror, \leftbracket etc). Then user
can navigate with arrow down and select by enter or space.
I think this could not only help beginners, but speed the work of advanced
users too.
—
Reply to this email directly or view it on GitHub
https://github.com/benweet/stackedit/issues/185#issuecomment-76562253.

@victorel-petrovich That's a non-trivial task in itself. The list of available commands depends on which packages are loaded, what you have defined above, conflict resolution, and so on.

Yeah, I agree, it's advanced feature. I saw it in LYX.

Closing as too old.


Add RKNN object detection docs

probably just want a little getting started section saying "to get started change the pipeline type to Object Detection" with an admonition about how this won't work on non Opi for parity with the other pipelines

good call done

Still a few more left of Photon over PhotonVision but lgtm.

Note that the yolov5 model might have to be trained by the same rknn fork version of yolov5. We trained a yolov5 using ultralytics and the conversion didn't work because of pickling/unpickling issues

Yeah it's apparently quite particular; alex_idk is working on docs


ci: linux: specify target triple for neon pack-build (vectordb)

fixes that all neon pack-build packs are named vectordb-linux-x64-musl-*.tgz even when cross-compiling
adds 2nd param:
TARGET_TRIPLE=${2:-x86_64-unknown-linux-gnu}
npm run pack-build -- -t $TARGET_TRIPLE

sorry, @wjones127, there's a bug,
it should be npm run pack-build -- -t $TARGET_TRIPLE instead of npm run pack-build -t $TARGET_TRIPLE


The TreeView class should expose an Indentation property

The TreeView class should expose an Indentation property that sets the multiple used when indenting TreeViewItems.
Currently, the calculation is hard-coded as depth * 16, which does not visually suit many re-stylings/re-templatings of the TreeViewItem.

Good suggestion indeed, but awaiting higher upvotes before pitching. Labeling it for WinUI vNext so that it stays on our radar for the next version.


Make reference kernel and result check optional

Done in development branch, see 8e71804.

Done in development branch, see 8e71804.


Check postal codes are strings

You can pass integer postal code instead of strings. The config check should either reject the postal code and clearly warn the user, either stringify it.

Closed by https://github.com/Phyks/Flatisfy/commit/571b42f5b15eba27d6e91d3e8739cab280f2a0e5. Thanks!


Adding Bengali Language Support

en.json5 translated to bn.json5 for (bengali language)
DSpace is one of the most used software for Institutional Repository in Bangladesh, adding 'bengali' support will have a huge impact on user interaction and satisfaction.
I have tested my changes on my local computer every thing seems to work fine on my local machine. I have just added bn.json5 file to src/assets/i18n and language code to environment.test.ts file
{
"code": "bn",
"label": "বাংলা",
"active": true
}
This is my first contribution to DSpace, if there are any mistake and need to update kindly let me know.
Regards
Md. Topu Raihan
Graduate Dept. of Information Science & Library Management

@raihantopu : It looks like this PR has accidentally changed too many files.
Ideally, to add Bengali, all we'd really need to the new bn.json5 file.  We can then review it briefly (using Google Translate) and then we'll help enable it in the configuration by default.
If you could just submit us a PR with only the bn.json5 file, that's good enough. We can take it from there.


modin process bar

mdf['title'].apply(func)
modin can add a modin-process-bar

Hi @Jie-Yuan, thanks for the request.
Is this something you'd be interested in contributing? Typically a progress bar requires extra compute and will decrease overall performance. Some components of Modin are asynchronous, so we would have to change that to make it work properly.
Would a progress bar help with using Modin even though it will decrease performance?

@devin-petersohn  a progress bar will be a useful function if it does for me.  I think a progress bar can be an option for users,  there may be two methods I can suggest:

built-in function
from modin.pandas import tqdm mdf['title'].tqdm(apply(func))
function params - (apply, map)
from modin.pandas import tqdm mdf['title'].tqdm(func, tqdm=True)

i'm interested in contributing this new feature ~

@sunnyjiechao Thanks! For starters it should probably go in modin.experimental.pandas.
I think the best course of action would be to register a tqdm option to modin.experimental.pandas.options. That way you can enable globally like this:
import modin.experimental.pandas as pd
pd.options.tqdm = True  # enable progress bar
...
pd.options.tqdm = False  # disable progress bar
...

I think it feels more pandas-like to add as an option rather than add a new API in that way. Plus if you want progress bar for multiple lines in a row it's just one addition.
I propose this get broken up into 2 pull requests. The first pull request is to add an option for tqdm. The second pull request can implement the progress bar. Do you think that makes sense @sunnyjiechao ?
Link to pandas options: https://github.com/pandas-dev/pandas/blob/master/pandas/_config/config.py

Yes, I can't agree with you more, Your thoughts are very good that it looks more pandas-like.

I think all it will take is adding pandas.core.config.register_option("tqdm", False) to the init.py in modin.experimental.pandas (Link to file) to add the option.
It would be good to test it first though. After that PR, then it can be a check for the option flag to use the progress bar functionality.

Got u, thanks

@devin-petersohn So sorry for that, after working on it long time, i found it is very diffcult to figure it out and also not useful enough for wasting too compute resource, i have to say git it up.
I am willing try to contribute  more useful rather than fancy function of modin

No problem @Jie-Yuan, if you want to take a look at #658, there is a list of things there that can be implemented. Let me know!

I was just going to ask about this when I found https://github.com/modin-project/modin/pull/1589

Now that we have a progress bar for some queries, I'm going to close this issue in favor of more specific progress bar feature requests, e.g. #2775 and #4167.


higher resolution

I assume if I retrain the sketch network on higher resolution samples and then train both the structure and texture models on ~same sized samples I can achieve a higher resolution output? Should I also modify the --subimg_size option?
Have you tried this or seen any issues with network size maxing out the GPU?

I think you're right. To obtain HR results, HR images should be used for training. (I think another way might be to obtain LR results and then use SuperResolution methods to obtain HR images)
I think the --subimg_size mainly depends on the size of structure/texture patterns. If those are quite large in your HR images, then the --subimg_size should be increased correspondingly to make the network fully perceive the key patterns.
Sorry, I'm not sure if there are issues with network size maxing out the GPU. I haven't tried it before.

no worries, this is really helpful. thanks! I think the subimg_size param will actually determine the GPU usage, and I think it can be brought up a bit based on my previous work. Will give it a shot and let you know!

Looking forward to it :)


Added Parsijoo support

Fixes #293
Checklist

[x] I have read the Contribution & Best practices Guide and my PR follows them.
[x] My branch is up-to-date with the Upstream master branch.
[x] I have added necessary documentation (if appropriate)

Changes proposed in this pull request:

Added Parsijoo support
@gabru-md @vaibhavsingh97 Please review. Providing the heroku deployment at https://shielded-castle-17515.herokuapp.com/
Thanks !!

@mariobehling @niranjan94 please review !!


Make types like Box<dyn DynAccess> implement Access

by generalizing the P: Deref-implementation to ?Sized,
and implementing Access for some dyn DynAccess trait object types.
Not yet documented or polished, this PRs main purpose for now is to allow CI testing.

The failing clippy lints are unrelated to this PR.

Older compiler version apparently fails in one of the dependencies (half).

Codecov Report

Merging #78 (a113002) into master (f7f192d) will increase coverage by 0.60%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master      #78      +/-   ##
==========================================
+ Coverage   83.37%   83.97%   +0.60%
==========================================
  Files          18       18
  Lines        1149     1161      +12
==========================================
+ Hits          958      975      +17
+ Misses        191      186       -5

Impacted Files
Coverage Δ

src/access.rs
86.02% <33.33%> (-7.81%)
:arrow_down:

src/lib.rs
87.56% <0.00%> (+0.51%)
:arrow_up:

src/strategy/hybrid.rs
93.84% <0.00%> (+6.15%)
:arrow_up:

src/debt/helping.rs
89.06% <0.00%> (+10.93%)
:arrow_up:

@vorner I’ve addressed the unrelated CI failures (I don’t know if that’s the "proper" way to do this with the dependencies for 1.45 compilation, but it seems like it works); feel free to re-run CI.

are there any cases left where it is necessary?

as already noted over in #77:

the actual AccessConvert is somewhat more general, as it allows – say – Box<dyn Foo> for a trait Foo: DynAccess, too.

Similarly, you can use it for things like Box<dyn DynAccess + Unpin> or Box<dyn DynAccess + Sync> or other more rare variantions on the trait object type that I didn't cover with any explicit implementation. (I chose the same selection three of auto-trait combinations (none, only send, and send+sync) that e. g. dyn Any in the standard library supports for downcasting; if there are others with practical use, they can be added later.)

Hello again. Just wanted to check, are you still working on the docs, or did that one get lost in the rest of the discussion? 😇 Just that we both don't wait on something from the other side.

Thanks for the ping. I didn’t misunderstand, but I haven’t written the docs yet either; I’ll try to allocate some time to this soon :-)

Hello.
I've discovered this still sits here. Not that I would have too much free time myself, but should I just take it over and write the docs, so it moves forward?

I forgot this PR even exists, so ehm... sure, feel free to take over, thank you.

I've taken the main commit, put it onto current (which already had the CI issues mostly solved) master and made some docs for it and merged in #85.
I'll try to collect few more outstanding PRs and release a new version soon. If I don't get around to it by the end of the year, feel free to ping me.
Thank you for discovering the way to do this :-)


[TASK] Unit & integration Testing for Frontend Web

Please implement frontend testing for the web implementation. Please use the testing framework, Jest unless you find better testing frameworks and can justify why we should use those testing frameworks.

[x] Research and learn how Jest works and how to write test code
[x] Implement testing to frontend code.
[ ] Look into integration testing when you are complete with this task.

@Tinashe-Austin , please tick off as you complete each task by clicking on the task you are done with in this comment
Additional info
Read through the jest documentation.
Look into integration testing for when we need to do integration testing you have an idea of how it works

there is a weird import causing errors but im basically done


chore: add download-whisper-model bin

#17  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Feature: Custom repository actions & keyboard shortcuts

It would be super lovely if there keyboard shortcuts for the "open" actions, so I could open a project in Windows Terminal or Windows Explorer without using the mouse when a repository is selected. Perhaps something as simple as 1, 2, 3?
Happy to submit a PR for this if it's something that you see in the vision for your project.

I wanted to refactor this for a while now tbh. At the moment, there are two action providers which return the available menu items to click in dependency of the user selection. One for Windows one for macOS.
Currently, these actions are not configurable. The providers decide what's possible (depending on the installed tools and the selection) and decide the order of the actions. The first action is also available per [Return] key, the secondary action can be triggered with [Ctrl]+[Return].
There are already GitHub issues to make these actions configurable (#105 & #108). So I really want this to be better for my users but at the same time I want to keep intelligent defaults to keep RepoZ as low-conf as possible. My credo is that even though this is a developer tool I don't want it to look like one 😊
I will look at this once I find some time. If you are willing to step in ... well ... I'd be very grateful for each task you'd grab.

@awaescher Absolutely, I understand completely. I'll wait a bit so you'll have some time figuring out how you want it to work. Once you know and if you don't have any spare time, ping me and I'll happily donate some time to implement it 😄

Oh, I already have an idea. But in my eyes it is not obligatory - I am open to any suggestion. Since this is a developer tool, I would actually have thought of a configuration file that can be opened easily via RepoZ. As I said, I want to keep the UI as simple as possible, but if someone wants to make adjustments, he should have as many options as possible.
What do you think about the idea that RepoZ ships with such a file by default and lets the developer make the necessary adjustments. The problem: If you can change all kinds of things yourself, it's a bit more difficult to validate and parse (e.g. the keyboard shortcuts) and keep supported in future versions.
# Define your context actions for RepoZ.
# To enable path probing, you can define multiple paths for the application executable to run.
# Try to avoid absolute paths if possible by using environment variables like %PROGRAMFILES% or %APPDATA%, for example.
# Use the placeholder {RepositoryPath} to build command line arguments. RepoZ will replace it with the given repository path.

{
  "name": "Open in Visual Studio Code",
  "paths":
  [
    "%LocalAppData%/Microsoft VS Code/code.exe",
    "%ProgramW6432%/Microsoft VS Code/code.exe"
  ],
  "arguments": "{RepositoryPath}",
  "keys": "Ctrl+1",
  "sort": 1,
  "visible": "true"
},
{
  "name": "Open in Windows Terminal",
  "paths":
  [
    "%LocalAppData%\Microsoft\WindowsApps\wt.exe"
  ],
  "arguments": "-d {RepositoryPath}",
  "keys": "Ctrl+2",
  "sort": 2,
  "visible": "true"
},
{
  "name": "Open in PowerShell",
  "paths":
  [
    "%WINDIR%\System32\WindowsPowerShell\v1.0\powershell.exe"
  ],
  "arguments": "-executionpolicy bypass -noexit -command 'Set-Location '{RepositoryPath}''",
  "keys": "Ctrl+P",
  "sort": 3,
  "visible": "false"
}

@awaescher I think that's a great idea. Perhaps go the Windows Terminal way and specify a schema for the settings file as well. I would say, validate the best you can and if there's an error then show it and fall back to the default settings.
I would probably remove the sort property as well and just use the order that things are specified in the settings file.
Would be nice to have a Settings... menu that opens the configuration file in the default editor for JSON files.
(A lot of inspiration from Windows Terminal here, but it's worked well for them, and once you've stabilized the schema for the settings, you could even add a UI for it)

Good point with the sort order, developer blindness 😄 I was inspired by WT as well as I thought about that file. VS Code was very successful with this approach as well (now they built an UI around it).
There's already a service to read/write local settings which puts files to %AppData%\Roaming\RepoZ. I think this would be a good place for the new context action settings file.

Would be nice to have a Settings... menu that opens the configuration file in the default editor for JSON files.

That's what I thought as well. We could place it either in the "main" menu like in the first image or in the context menu itself like in the second one.

The main menu may fit better for this menu item as it's a global thing to define these actions (thinking of "Settings ...") but the more I think about it, the more I like the second one because there you don't need to define the context "Repository Actions" (I even don't like that term very much 😊) because you're already in the correct context.
What do you think?

I don't know about the settings file. I my mind, having it like that limits the context menu to simple "open this directory in this program" things. Of course, I might be biased, but I still think the context menu has the potential to be more powerful than that.
One idea might be to add internal commands to the context menu in addition to external apps? These commands could be picked up by code in the app to do more complex things.

@henriksen The sole reason for this is to get a settings file for hardcoded stuff. Adding more types of commands or expanding the schema with new functionality can be done incrementally later on when the need arise and we know how we (primarily @awaescher) want stuff to work.

I thought of this issue in here as I merged the PR from @henriksen as well. Just to make sure we're talking about the same ... I'd thought that the config file to define the "Open in ..." actions won't replace the whole context menu. We should keep the newly merged "Open Solution" area intact. As well as the basic git commands and the possibility to ignore repositories.
The stuff we were talking in here is indeed "Open the directory of a given repository with ...". In the sample file I added the placeholder for the {RepositoryPath}. But we could add more of these repository properties if they are useful. I did just not think of it yet.
Another thing I thought after I merged the PR: For us Windows Devs, *.sln is the most used file in our repositories. But we could also make this part dynamic by adding the possibilities to define the extensions one wants to be included into the menu.
This could be
{
  "name": "Open Visual Studio solutions",
  "extensions":  [  "*.sln"  ]
},
{
  "name": "Open Xcode projects",
  "extensions":  [  "*.xcodeproj"  ]
}

... to have a menu with submenus for the matching files:

We're going away from the initial simplicity, that's why I struggle. However, I think the features could be very useful if the defaults are chosen wisely, like 2-3 default "Open in ..." actions and the "Open Visual Studio solutions" sub menu.

As long as it's not a "throw the baby out with the bathwater" kinda thing 😊
One good thing about having a settings file is that each user can reduce their own complexity by removing things they don't need. For instance, I'll never use the Command Prompt, Git Bash or Powershell options so I can just disable those in the settings. Another developer might use VS Code exclusively and prefer to drop the Visual Studio entry. Or someone might want to rename "Visual Studio" to "Rider".
It does increase the flexibility for those things that can be generalized!

The "Open Whatever solutions/projects" concept is pretty generalizable and can benefit from an open customization. For instance, what file pattern to look for, which folders to in (in addition to root), how to start them (just open the file or a custom command line?), etc. Personally, I don't think it complicates things significantly. The UI stays pretty much the same (or less), some things get a submenu, but with keyboard shortcuts that's pretty fast to navigate.

That's what I thought. I would like to remove some of the current actions as well. As you said, Windows Command Prompty & Git Bash are two candidates I'd add to the default config with visible=false so the user can add it easily if he or she wants.

If I wanted to start on a config file and the structure needed to read the settings, would extending FileAppSettingsService and AppSettings be the place to do it, or would you prefer a separate structure? (or are you already working on this?)

Hm, at the moment I'd say I'd go for an additional file. If the devs change that file manually and they mess up it any way, we have no problems with parsing the app settings.

May I suggest a "smaller fix" that would go a long way as a temporary help for the originally posted problem?. Maybe just adding a simple hotkey that opens the right-click menu, and thus allows for the already existing keyboard navigation (up/down) would at least allow us to not have to reach for the mouse for that one click?
Just opening the context menu instead of having fixed "1, 2, 3, ..." shortcuts would at least alleviate the issue of the dynamic nature of the action providers.
It would have to be something that isn't "valid input" for the typing field, (so unfortunately I can't suggest to use the same . as e.g. my favorite GitUp client for mac), but maybe something as simple as Right arrow could work? Assuming that our cursor is at the end of the typing field anyway, then right arrow might just work?
While typing this, I just realized, that if/when I happen to be stuck on a Windows machine with a full "windows style" external keyboard, then it actually has a hardware "menu" button that works :-)
But it would be really nice to have a keyboard shortcut that works on keyboards without this dedicated menu key, and on mac as well.

it actually has a hardware "menu" button that works :-)

That's what I use all the time 😃. There's not other shortcut I remember for context menus ... but there's F4 (on Windows) to open drop down controls like ComboBoxes ... which could be a bit like a sub menu. I'm not sure if I'd like that.

There's not other shortcut I remember for context menus

Shift-F10 opens context menus. Same keycode as the "Menu" button.

Just a note to myself that Cmder could be a good addition for the "Open with ..."-gallery.

I started to implement this feature a few days ago. Here's the task list I'm working with.

[x] Define json files → see RepoZ-RepositoryActions
[ ] Deploy default configuration
[x] Build the context menu from json file
[ ] Show the customization possibilities in the UI
[ ] Handle & show json parser errors
[ ] Implement keyboard shortcuts
[ ] (Translations)

(Please note that I only work on it from time to time, so progress can be slow)

Pull-Request #153 has just been merged - however keyboard shortcuts are not implemented yet.
Sorry for the delay, I am pretty busy atm 😬

Just build master to try out the new features and wow! Congrats on an awesome job, completely in love with the new editable actions.
Inspired me to do a quick tool to open a solution in Visual Studio: https://github.com/sboulema/VSLauncher

I've just installed v5.5 from the Windows installer and am looking for the feature to define my own "Open in ..." app.  The repo actions look to be exactly what I need but I see they aren't released as part of a build yet.
Do you have an estimate for when this feature will be placed into a release?


[DevTools Docs Feedback]

I do not work for anyone i want back my Administrator rights & my UAC I have never purchased any type of OFFICE this is my home computer all my rights have been taken away by some Administrator & password & taken over UAC no matter what I try my updates are blocked Troubleshooting The Wizard all blocked on 10/03/2022 I got a new motherboard Windows 10 22H2 version OS I had Malwarebytes installed but windows firewall security infiltrated evidently put my Lenovo &  WIN-QM1NE1H1PGP to sleep & returned my pc back to 2019 it says i have S version but microsoft store says i do not have S Malwarebytes blocks everything windows tells them I think a trogon has taken over & by-passed my security what happened to microsoft regular security it was great with my other security I do not want to do a clean install i'm 78 & not to savvy but i do know my pc is not fun anymore WiFi is blocked so is my att router I only have one domain yours no file & printer sharing no public it says everything is controlled by you administrator who ever the hell that is I do not know I tried blocking javascript that does not work eithor just said i could not sign-in to MicrosoftEdge!

Sorry about the inconvenience, but this repository is specific to documentation issues related to Edge web development technologies. Your issues seem unrelated to that. So I'm going to close this. Thanks for your understanding.


[CPR] update to 1.9

Library name: CPR
New version number: 1.9
Other information that may be useful (release notes, etc...)
vcpkg has cpr already updated for version 1.9 but there was an update to the library 2 days ago which adds things i really need and vcpkg isn't updated for it

Fixed by https://github.com/microsoft/vcpkg/pull/25244

@JonLiu1993 hello, i dont think you have read the "more information" text i have shared, vcpkg IS updated for 1.9 but not for the latest commit: https://github.com/libcpr/cpr/commit/5ae5fdeadfbbb784192c034be0222016123c94ad which adds some stuff which are very helpful

@bleriaf ,the frequency of cpr Upstream releases the official version is very high, we do not recommend updating the port to the unofficial version of commit, you can wait for a while and I will update to the latest version immediately after the latest version of cpr is released.


Supported offline

Description
Added the support for offline so that app works even the internet is not available. Also fixes little issues.

I appreciate your pull request @kapilip . Everything work well. I haven't focused on security issues for this project much, however there is a point that redux offline save redux store data in local storage  which cause security issues. Do you have any idea about that?

@Qolzam Yes, I have read the security issues with local storage and we shouldn't be storing the sensitive information like the user's credentials or payment details in it. So IMHO we aren't storing any of these in the local storage and storing the public information which shall be available to other users. Let me know your thoughts.
BTW you are doing a wonderful job, we have used this structure for our internal app and later added offline support for our app. We would happy to contribute to react-social-network as we add more things in our app :)

@kapilip I am happy to hear that It's helpful to your project. And I do appreciate your contribution. You are right, not sure about next:branch and if personal data which comes to redux store is important for user or not, however we need be aware of that. How is your idea that we make a pluggable feature of offline-support? Kind of option for developer!
By the way I am currently working on v0.7.0:branch NOT READY (I will release backend as soon as possible) in my spare time and it's going so slow, but I am trying to make this social more independent to Firebase (#109).
Indeed I am trying to make this project closer to production. But as you know there are 531,664 lines of code (in src folder) need to be maintained and developed.
Maybe you are more interested in social network with K8S for real world production which will come after (#109).
Thanks again!

@kapilip I am happy to hear that It's helpful to your project. And I do appreciate your contribution. You are right, not sure about next:branch and if personal data which comes to redux store is important for user or not, however we need be aware of that. How is your idea that we make a pluggable feature of offline-support? Kind of option for developer!
By the way I am currently working on v0.7.0:branch NOT READY (I will release backend as soon as possible) in my spare time and it's going so slow, but I am trying to make this social more independent to Firebase (#109).
Indeed I am trying to make this project closer to production. But as you know there are 531,664 lines of code (in src folder) need to be maintained and developed.
Maybe you are more interested in social network with K8S for real-world production which will come after (#109).
Thanks again!

That's a good idea to make offline support as optional. Could you suggest how can I make it a plugin?
Ok, I shall try to contribute to v0.7.0 branch.  Could you tell which features you would like me to work on? I would be happy to contribute it to the project.

@Qolzam I tried to run v.0.7.0 and looks like I would also need to the source code for the API. Could you also push the code in the git repository so that I can run v0.7.0 locally?

@kapilip Thanks for checking, you are great 👍 . Can you tell why you can not start? This branch is not ready yet, It's a little bit buggy :). After completing OAuth I will push complete backed. Until next week I will organize backend too.
About plugin let's discuss here #97

@Qolzam We shall be waiting for the branch to be ready :-)

Hello @kapilip. Here is the back-end. Please let me know if you face with any bugs.
Thanks!

@Qolzam Thanks a lot :-) We shall take a look and let you know if we have any question


Composite types are not understood by sqlc

Version
1.17.2
What happened?
Given
Migration file
create type myarg as (
    id integer,
    name text
);

When
Query file
-- name: BasicQuery :one
select $1::myarg;

Then
const basicQuery = `-- name: BasicQuery :one
select $1::myarg
`

func (q *Queries) BasicQuery(ctx context.Context, dollar_1 string) (string, error) {
	row := q.db.QueryRow(ctx, basicQuery, dollar_1)
	var column_1 string
	err := row.Scan(&column_1)
	return column_1, err
}

Note that the second input parameter (dollar_1 string) is of type string, which is not what I expected. The same goes for the output parameter which I expected to a struct but is instead a string.
When
Query file
-- name: LessBasicQuery :many
select id, text from unnest(@args::myarg[]);

Then
When you generate the Go code the following error shows up in the terminal.
queries/queries.sql:28:8: column "id" does not exist
sql/generate.go:3: running "sqlc": exit status 1

When
Query file
-- name: LessBasicQuery :many
select * from unnest(@args::myarg[]);

Then
const lessBasicQuery = `-- name: LessBasicQuery :many
select  from unnest($1::myarg[])
`

type LessBasicQueryRow struct {
}

func (q *Queries) LessBasicQuery(ctx context.Context, args []string) ([]*LessBasicQueryRow, error) {
	rows, err := q.db.Query(ctx, lessBasicQuery, args)
	if err != nil {
		return nil, err
	}
	defer rows.Close()
	var items []*LessBasicQueryRow
	for rows.Next() {
		var i LessBasicQueryRow
		if err := rows.Scan(); err != nil {
			return nil, err
		}
		items = append(items, &i)
	}
	if err := rows.Err(); err != nil {
		return nil, err
	}
	return items, nil
}

Notice that the input argument (args []string) is a slice of string instead of an expected struct. The output parameter LessBasicQueryRow is struct, but its implementation is empty i.e. no fields have been generated.
Relevant log output
No response
Database schema
create type myarg as (
    id integer,
    name text
);

SQL queries
No response
Configuration
version: '2'
sql:
  - schema: "migrations/"
    queries: "queries/"
    engine: "postgresql"
    gen:
      go:
        package: "store"
        out: "./store"
        sql_package: "pgx/v4"
        emit_result_struct_pointers: true
        emit_params_struct_pointers: true
        emit_interface: false
        output_db_file_name: "db.go"

Playground URL
No response
What operating system are you using?
macOS
What database engines are you using?
PostgreSQL
What type of code are you generating?
Go

Seeing the same thing -- tested in a small repo as well with pgx/v4, pgx/v5 and sql, and it all yields the same result

Or perhaps it's intended, not sure how it worked before:
https://github.com/kyleconroy/sqlc/pull/311
I'll give it a try by using the overrides in the config file


Merge GitOps 'promotion' Pull Requests automatically

This issue is part of https://bigblue.aha.io/features/ICPAROAD-326 for those with Aha! access.
The current solution for the "Build and Promote" pipeline builds a microservice and leaves a Pull Request (PR) open, ready to be manually promoted to the GitOps dev repo. This is what's currently going to happen in the pipeline when building from a merge to master of a microservice project:

Build the microservice project
Move the updated app-deploy.yaml into the /config directory (creating if needed)
Create a kustomization.yaml to reference the app-deploy.yaml and any other config files in the /config directory
Raise a Pull Request against the GitOps repo to check-in the /config directory into /environments/<envName>/services/<project name>/base

This leaves a PR open in GitHub that has to be merged, meaning manual intervention is required to flow changes into the Dev environment.
A working prototype has been developed in https://github.com/rhd-gitops-example/services/tree/master/automerge-example. Use this issue to add automerge support into the appropriate ICPA pipelines as part of Aha-326.

@kvijai82 @stephenkinder  is this one in plan as well?

This goes with parent epic https://github.ibm.com/IBMCloudPak4Apps/icpa-roadmap/issues/272 and Aha! Link: https://bigblue.aha.io/features/ICPAROAD-326


Could

Thank you very much for providing such excellent work, I recently tried to reproduce DiscoNet+SyncNet mentioned in your paper, could you please provide a more detailed Readme or other guidelines? Thank you very much.

Sorry for inconvenient, Our paper is based on an early version of V2X-Sim(Version 1.0 in 2021). We are working on the new version of V2X-Sim 2.0 dataset, including V2V and V2I. We will release the full version of code and guidelines after we transfer our whole pipeline to the new version of V2X-Sim.


Add release notes for everything between 0.87.2 and 0.88.0 inclusive

Purpose
Add some release notes
Changes
Release notes added to CHANGELOG
Side effects
People will learn what's happened recently if they look in the CHANGELOG

👍  Cherry-picked into release/0.88.0


Restore #757

#757 was merged but it caused errors downstream. Maybe we need some more testing on this class anyway.

@zhut19 can it be that the map wasn't updated in privevat nt files repo?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 7 (71.43%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.02%) to 85.47%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

straxen/itp_map.py
5
7
71.43%

Files with Coverage Reduction
New Missed Lines
%

straxen/itp_map.py
2
77.04%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
6059

Relevant Lines:
7089

💛  - Coveralls

Thanks for the checks!


Nsdf branch

PR from @scrgiorgio and @pascucci to preload data. Thanks so much for adding pre-loader that helps with loading the data.

Resolving merge conflict, this seems to be working fine!


Prevent errors in gasPrice

For transactions of too little gasPrice , the code is not defined and therefore terminated with an error.

web3.trace.transaction is a parity extension, only supported by the Parity.

Issues already handled.


Bluetooth: Shell: Set err to 0 before shell_strtoul

The value of err was not initialized to 0 prior
to calling shell_strtoul, which only sets err
on actual error.

Do you know why this not caught by CI testing?

Do you know why this not caught by CI testing?

We don't have checks for uninitialized values AFAIK, and we don't run the shell in CI

Do you know why this not caught by CI testing?

We don't have checks for uninitialized values AFAIK, and we don't run the shell in CI

I was guessing that the compiler could catch this

Do you know why this not caught by CI testing?

We don't have checks for uninitialized values AFAIK, and we don't run the shell in CI

I was guessing that the compiler could catch this

We could also consider setting *err to 0 in shell_strtoul - but didn't want to rock the boat so close to deadline in case somewhere else expects the current behavior.

We could also consider setting *err to 0 in shell_strtoul - but didn't want to rock the boat so close to deadline in case somewhere else expects the current behavior.

Yeah, this is a common issue when people use these shell functions. There was a reason why it was designed like that, but I don't recall it

We could also consider setting *err to 0 in shell_strtoul - but didn't want to rock the boat so close to deadline in case somewhere else expects the current behavior.

Yeah, this is a common issue when people use these shell functions. There was a reason why it was designed like that, but I don't recall it

I guess you can chain them and just do one error check in the end.  Still not optimal, imo ;)

We could also consider setting *err to 0 in shell_strtoul - but didn't want to rock the boat so close to deadline in case somewhere else expects the current behavior.

Yeah, this is a common issue when people use these shell functions. There was a reason why it was designed like that, but I don't recall it

I guess you can chain them and just do one error check in the end. Still not optimal, imo ;)

Right, that was the reason. I agree that it's not optimal, and it's hardly ever used for that was that does not give good error message to the user. It is far more typical to parse something, and return a specific error for that parsing to the user to show what value failed


Bug: iOS Input shows ugly box on click

Bug Report
Ionic version:

[x] 4.x
Current behavior:

A random input box appears in an input field when clicking it. Same for iOS on phone and in chrome.
Bug Video
Expected behavior:

There should be no box appearing
Steps to reproduce:

Just click the input field and it appears.
Related code:

If you need some I can provide it.
Other information:

Appears on the phone (capacitor) and in the browser (chrome)
Ionic info:

@ionic/core 4.0.2 in combination with Vue and @ionic/vue

Hi there,
Thanks for opening an issue with us! Can you provide a repository with the code required to reproduce this issue? Thanks!

Repo
If you need more assistance let me know! Happy to help.

Run npm run serve and click in the login field


Add docs and issue / PR templates

I've moved the content of the README to the osmox documentation site that will be built and released to https://arup-group.github.io/osmox after this PR is merged.
@val-ismaili would be worth checking out the content locally by installing the osmox dev environment (using the new install instructions) and running mkdocs serve in the command line. Then you can browse a local build of the docs.
I've tried splitting up the existing README into separate pages, but am not the best person to decide on the parcelling of content into different sections!

Codecov Report
Attention: Patch coverage is 0% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 63.75%. Comparing base (1afe3bf) to head (7105159).

Files
Patch %
Lines

src/osmox/cli.py
0.00%
2 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #40   +/-   ##
=======================================
  Coverage   63.75%   63.75%
=======================================
  Files           5        5
  Lines         480      480
  Branches      116      116
=======================================
  Hits          306      306
  Misses        154      154
  Partials       20       20

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


website isn't high enough in google search

Tim has made it extremely clear to me he wants MatterFi to be higher in the native search of Google.com, Bing, and DuckDuckGo.
All of these systems use tags (keywords) to be listed higher AFAIK, search into "increasing SEO" is required

I've done similar things in the past, it also helps to have the site linked from many other site locations to boost the results position.

You have my carte blanch approval to break and try anything that does not cost us money to improve this.
Good luck, please let me know what you end up doing.

I've updated the Keywords used in search indexing. updated description for url sharing preview link.

Is a good read https://neilpatel.com/blog/google-index/
We can also submit the URL to google. https://www.google.com/webmasters/tools/submit-url
Tried: https://pingomatic.com/
Using search engine "robots.txt" https://www.matterfi.com/robots.txt

I think most the free stuff is done, we can review to see if it was enough for the search engine boost. Should also double check to make sure that the linked social media accounts have url links back to matterfi.com
That could be the next step is spreading the web url to other sites with 'MatterFi" as the hyper link text if available.

ok no real rush on it, but will help with SEO
On Fri, Dec 4, 2020 at 11:12 AM Robert Douglas<EMAIL_ADDRESS>wrote:

I'm swamped today with time-sensitive work, but when I have a moment,
AFTER the logo is complete, I will sign into all of our social media
accounts listed in #1
https://github.com/matterfi/matterfi.github.io/issues/1 and update the
logo as well as add links, this issue can stay open until that is complete
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/matterfi/matterfi.github.io/issues/3#issuecomment-738898588,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKZOMI6V25OQGCOLY6K6RVTSTEJ6BANCNFSM4UK4IDLQ
.


Add logging to attendance importer

Who is this PR for?
developers trying to confirm that log flushing isn't working right

selfie


Documentation fix: Correct description from 'ChatGPT' to 'Google Bard'

This pull request resolves an issue in the README file where line #11 incorrectly mentioned "ChatGPT" instead of "Google Bard." Thank you!

Thank you


Commit history overflow in issue page

Description

Gitea Version
latest
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
build
Database
None

Maybe https://github.com/go-gitea/gitea/pull/30484 has fixed it.

That PR fixed another event type, but chances are this one here was fixed as well with it.

Tested in 1.23.0+dev-104-gf9fdac9809 which contains #30484, it is not fixed.


bug: flickering of completion menu w/ cssls

Make sure you have done the following

[X] I have updated to the latest version of blink.cmp
[X] I have read the README

Bug Description
sample specifying the bug:
https://github.com/user-attachments/assets/62875d36-fa9b-4849-b4c3-dfa8c8fdd67d
As the recording suggests, blinking (no pun intended) occurs with the menu.
This only occured to me when I am writing property for a CSS selector with using the LSP (cssls and emmet-language-server)
Relevant configuration
--- Not sure what could be the relevant config for this matter, pasting the one for the menu in here:
{
  completion = {
    list = {
      max_items = 50,
      selection = 'manual',
    },
    menu = {
      max_height = 100,
      scrolloff = 999,
    },
  },
}

neovim version
NVIM v0.11.0-dev-1388+g39781be14b
blink.cmp version: branch, tag, or commit
v0.*

I’m experiencing the same issue as well, but with emmet-language-server.
https://github.com/user-attachments/assets/8123858e-df09-4519-9f68-9ed174531c6d

For anyone looking to fix this, we show the menu as soon as any LSP returns a response, with the motivation being to prevent slow LSPs from blocking all the LSPs. We should instead make this configurable since it shouldn't be necessary in most cases. In the future, it would be ideal if we could detect slow LSPs explicitly


UI: Fix SVG exporting to remove black regions

Is your feature request related to a problem? Please describe.
At the moment I export svg and try to use other applications to convert it to pdf. But the svg file has some black parts.
Describe the solution you'd like
It would be great to be able to export a high-quality image of plots. At the moment, the report and png exports are not good enough to put it on paper.

Hey @kargarisaac , sorry to hear that the current resolution is not satisfactory.
I'm afraid the only option to control the resolution of your exported charts is this menu:

I'm bringing this issue up with the team. What resolution would you prefer to have?

Putting PNG format in PDF doesn't sound good, especially when zooming. By using the way you suggested, I think the text size would be small and not readable in picture. I export svg and use Figma to remove black parts and customize everything there. In Figma there is a black color that by setting its value to 0% it removes the black parts (Under the curve from start to end) I mentioned in the question.

Hey @kargarisaac
Can you confirm this behaviour still exists?
I seem to have downloaded a loss plot from my experiment and did not visualise any black spots on it.
Could you also screenshot and let us know about the black spots in the SVG that you are seeing?

Hey @kargarisaac
Can you confirm this behaviour still exists?
I seem to have downloaded a loss plot from my experiment and did not visualise any black spots on it.
Could you also screenshot and let us know about the black spots in the SVG that you are seeing?

Can you explain how you visualize the downloaded svg? what is the software you use? I tested again to visualize it in diagram.net in google drive but it has the black parts again:

Could you share the link to the chart here?

The project is ongoing, so I cannot share it, but the downloaded file from wand is attached.
W&B Chart 4_21_2021, 1_47_06 PM.zip

Hi,
Any update on this?

Hi @kargarisaac, are you still experiencing this issue? If so, can you send another example? The last file you sent looks to be correct.

Hi @sydholl, Yes, The problem exists. I didn't get a clear answer. How can I use the SVG file? how to open and visualize it? any recommended software on ubuntu? I used to use https://app.diagrams.net/ in google drive but after a while, it dodn't work anymore and I see black parts.

test.zip

Hi @kargarisaac so the black regions are a result of not having a standardized SVG format across all different toolset. I know it's not your first choice but your best bet to avoid this issue is to export your chart as a PNG.

@kargarisaac we're using the browsers native SVG rendering functionality.  Have you tried exporting in exporting from a different browser?  What browser generated the attached SVG?  As @aidanjdonohue mentioned we export a high resolution PNG which should render nicely in embedded documents.

@aidanjdonohue @vanpelt
thank you. I use chrome. The browser can open it successfully.
I have to use png then. the problem with png is that when I save it, it's not similar to what is shown in the browser. Legends size and style change.

Hey, any updates on this issue?

Hi @MohamedAteya, no movement on standardization. If you're having issues please either try a different browser or export a PNG instead.


How to install spyder plugins when installed from dmg/.app?

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[x] Could not reproduce inside jupyter qtconsole (if console-related)
[x] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder
[x] Reset preferences with spyder --reset
[x] Reinstalled the latest version of Anaconda
[x] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
I installed Spyder with the MacOS dmg/app. Then I wanted to try some plugins, but even when starting the shipped python and corresponding pip via ./python -m pip or from within spyder via cellmagic, it seems either unable to install (from within spyder) or installs where it apparently is not used/found (from outside)
What steps reproduce the problem?

Start spyder via dmg/app
Try to install plugin in one of the ways above

What is the expected output? What do you see instead?
I expected to see plugin tabs in Spyder. Instead I do not see it.
Paste Traceback/Error Below (if applicable)

%%bash
pip3 install spyder-terminal

Python path configuration:
  PYTHONHOME = '/Applications/Spyder.app/Contents/Resources'
  PYTHONPATH = '/Applications/Spyder.app/Contents/Resources'
  program name = '/Applications/Xcode.app/Contents/Developer/usr/bin/python3'
  isolated = 0
  environment = 1
  user site = 1
  import site = 1
  sys._base_executable = '/Applications/Xcode.app/Contents/Developer/usr/bin/python3'
  sys.base_prefix = '/Applications/Spyder.app/Contents/Resources'
  sys.base_exec_prefix = '/Applications/Spyder.app/Contents/Resources'
  sys.executable = '/Applications/Xcode.app/Contents/Developer/usr/bin/python3'
  sys.prefix = '/Applications/Spyder.app/Contents/Resources'
  sys.exec_prefix = '/Applications/Spyder.app/Contents/Resources'
  sys.path = [
    '/Applications/Spyder.app/Contents/Resources',
    '/Applications/Spyder.app/Contents/Resources/lib/python38.zip',
    '/Applications/Spyder.app/Contents/Resources/lib/python3.8',
    '/Applications/Spyder.app/Contents/Resources/lib/python3.8/lib-dynload',
  ]
Fatal Python error: init_fs_encoding: failed to get the Python codec of the filesystem encoding
Python runtime state: core initialized
ModuleNotFoundError: No module named 'encodings'

Current thread 0x000000011b379dc0 (most recent call first):
<no Python frame>
Traceback (most recent call last):

  File "<ipython-input-9-51830e68d79d>", line 1, in <module>
    get_ipython().run_cell_magic('bash', '', 'pip3 install spyder-terminal\n\n')

  File "/Applications/Spyder.app/Contents/Resources/lib/python3.9/IPython/core/interactiveshell.py", line 2403, in run_cell_magic
    result = fn(*args, **kwargs)

  File "/Applications/Spyder.app/Contents/Resources/lib/python3.9/IPython/core/magics/script.py", line 142, in named_script_magic
    return self.shebang(line, cell)

  File "decorator.pyc", line 232, in fun

  File "/Applications/Spyder.app/Contents/Resources/lib/python3.9/IPython/core/magic.py", line 187, in <lambda>
    call = lambda f, *a, **k: f(*a, **k)

  File "/Applications/Spyder.app/Contents/Resources/lib/python3.9/IPython/core/magics/script.py", line 245, in shebang
    raise CalledProcessError(p.returncode, cell, output=out, stderr=err)

CalledProcessError: Command 'b'pip3 install spyder-terminal\n\n'' returned non-zero exit status 1.

 There are seveval things related to your question:

As you discovered, we don't provide pip in our installers to avoid people messing with them by installing other packages and breaking Spyder. The main purpose of our installers is to be very robust and stable, so that people don't suffer by incompatible package versions or broken installations (which is quite common with pip and conda).
None of our plugins have been updated to work with Spyder 5 yet.
We're investigating how to create a mechanism to install plugins in our installers. That's probably something that won't be available until next year.

I'm going to close this issue because there's nothing we can do about it for now.

Ah ok, thanks for clarifying. That makes sense, also I was not aware that plugins are not compatible with spyder 5.

Yeah, the first one will be spyder-terminal, followed by spyder-notebook.


:bug: Add lock to child record

We are adding a lock to child record due to odd
race-condition-like behaviors seen when collections are forming relations. Reindexing would fail because #ordered_works is not returning anything.

Issue: https://github.com/scientist-softserv/utk-hyku/issues/500
Related to: https://github.com/samvera-labs/bulkrax/issues/804

Closing this PR because this turned out to not be related to the issue as expected. When tested using this branch the errors still occurred.