Scully sever not exiting after rendering static files

🐞 Bug report
Description
When running scully it doesn't appear that the servers are stopping when scully is done building the static files
🔬 Minimal Reproduction
💻Your Environment
Angular Version:

13.2.3

Scully Version:

"@scullyio/init": "^2.1.0",
"@scullyio/ng-lib": "^2.1.0",
"@scullyio/scully": "^2.1.0",
"@scullyio/scully-plugin-puppeteer": "^2.1.0",

🔥 Exception or Error

Hi @sonicparke thanks for reporting this issue, do you have by example a repo for reproduction? We have seen sort of these issues in the past so we would like to look if this has to due because of configuration or any edge case.

@Jefiozie unfortunately I don't. It's running in a private nx repo. If you aren't able to reproduce it in a fresh setup I can try to setup a repo that will reproduce it.

I think I'm getting this too.
When I run scully:serve, the server immediately exits with the following errors:
hisham@mb3 frontend % npm run scully:serve

><EMAIL_ADDRESS>scully:serve
> scully serve --project website

  ✔ Starting servers for project "website"
  ✔ Scully development Servers stopped, and exited
  ✔ Scully development Servers stopped, and exited

I see there is a running process as follows:
|--- 03014 hisham Scully-dev-server-website
after I kill this process, I'm able to run scully:serve again.

I think I'm getting this too.
When I run scully:serve, the server immediately exits with the following errors:
hisham@mb3 frontend % npm run scully:serve

><EMAIL_ADDRESS>scully:serve
> scully serve --project website

  ✔ Starting servers for project "website"
  ✔ Scully development Servers stopped, and exited
  ✔ Scully development Servers stopped, and exited

I see there is a running process as follows:
|--- 03014 hisham Scully-dev-server-website
after I kill this process, I'm able to run scully:serve again.

This worked for me too. But its not very elegant.


add Chinese (zh_CN) translations to 1.14.7, full coverage

@patricklodder is this what you meant when you suggested that I just include your commits in the translation PR?

Yes, perfect! I'm just going to mark this as blocked until the string freeze gets merged.

Does the linter check the entire file at once? I fixed those issues and a few more popped up (it seems some are from previous contributions, and others are from me).

Does the linter check the entire file at once?

Yes. I just pseudo-fixed them myself locally and I get this after:
$ python3 contrib/devtools/check-translations.py
Checking all translations...
No errors detected.

@slightlyskepticalpotat for some reason, GitHub is not processing the update to 1.14.7 - it should either stop recognizing my commits in your branch or throw a fit about merge conflicts, but it does neither.
I'm not sure why.

I decided to try this before rebasing, looks like it worked.

Awesome hack 😂


Store Token.position as namedtuple (or similar)

Rather than just storing the position as a tuple (<start line>, <end line>), as is done now. It would be better to store it as a namedtuple/dataclass. This would allow it to be more extensible to including e.g. character ranges for spans, document name, or even cell index for cell type documents.

Closed by d1d7283fdce8883327b97afde8668c70040052f3


Can't add custom reactions

I set up nadekobot so I'm the owner account and invited it to my server, but I can't get it to add custom reactions. Every time I run .acr <trigger> <message>, I get the error "Insufficient permissions. Requires Bot ownership for global custom reactions, and Administrator for guild custom reactions."
I know I'm the owner, however, as I can run .die and the bot restarts. What's wrong?

And that happened in your own server or on your friends server?
Also check if you have perms on both then test if it doesn't work restart the bot.

use .acr in DMs to make global custom reactions and those require you to have bot ownership.
.acr in server channels require you to have administrator privilege. It's tested and it works. If you really are a server owner, try giving yourself a role with administration checked in order to make server-sepecific custom reactions.

I tried it on my own server, and it worked fine. I'm having the issue on a friend's server.
I've set the permrole properly (running ;permrole doesn't return an error), set myself as owner in credentials.json, but I'm not sure if there's a step I'm missing


Handilng authentication exception

I faced the problem, now realization of Client use this scenario:

make request, if error (401 - Unauthenticate, interesting for issue context)
if 401 code of previous response, invoke Authenticator for specifying credentials and follow up request.

Consider credentials is wrong (for example user specified wrong credentials).
than next request will return 401 again, and this repeats MAX_FOLLOW_UPS times. After all it throws ProtocolException("Too many follow-up requests: ").
And question is: "How I can detect wrong credentials?"
There is urge way for client uses this HTTPClient is to detect count of equal requests or something the same now. It seems as spike.
Is there any different way to handle bad credentials?

You can test your credentials against the credentials of the unauthenticated request:
      @Override public Request authenticate(Proxy proxy, Response response) {
        System.out.println("Authenticating for response: " + response);
        System.out.println("Challenges: " + response.challenges());
        String credential = Credentials.basic("jesse", "password1");

        if (credential.equals(response.request().header("Authorization"))) {
          return null; // If we already failed with these credentials, don't retry.
        }

        return response.request().newBuilder()
            .header("Authorization", credential)
            .build();
      }

I've added this example and another to our recipes doc.


ImportError: cannot import name 'PublicKey'

Circular import error
Python 3.6.1 (default, Sep  6 2017, 16:18:23)
[GCC 5.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.

>>> from steembase.account import Publickey
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/vvk/devel/golos/golos-python/steembase/account.py", line 5, in <module>
    from steem.utils import compat_bytes, compat_chr
  File "/home/vvk/devel/golos/golos-python/steem/__init__.py", line 2, in <module>
    from .steem import Steem
  File "/home/vvk/devel/golos/golos-python/steem/steem.py", line 1, in <module>
    from .commit import Commit
  File "/home/vvk/devel/golos/golos-python/steem/commit.py", line 10, in <module>
    from steembase import memo
  File "/home/vvk/devel/golos/golos-python/steembase/memo.py", line 10, in <module>
    from .operations import Memo
  File "/home/vvk/devel/golos/golos-python/steembase/operations.py", line 8, in <module>
    from .account import PublicKey
ImportError: cannot import name 'PublicKey'
>>>

workaround: import Steem first
>>> from steem import Steem
>>> from steembase.account import PublicKey


spidermonkey_68: drop

Motivation for this change
Things done

Built on platform(s)

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

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

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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

libproxy does not support spidermonkey other than 68. It builds with a different version because the cmake flag is broken and pacrunner_mozjs is not built: https://github.com/libproxy/libproxy/issues/153

Maybe it would make sense to remove js support in libproxy outside of darwin since it doesn't build anyway.

It is misleading since it doesn't do anything.
If the prediction that the next version supports esr78 is right, it is already out of esr maintenance.
If the reference is optimistically changed to esr91 it will add a lot of build dependencies for esr updates in the interim for something non-functional.
The Duktape pacrunner is already integrated so it should be evaluated and might be the better choice when the next version releases.

cc @zhaofengli who implemented the Duktape support.

I agree with @lostnet. It doesn't make sense to have a mozjs dependency that isn't actually used. For Duktape, maybe we can ask the upstream to cut a release.

I asked upstream here.


Don't print out credentials on credhub import

What version of the credhub server you are using?
Server Version: 0.0.0
What version of the credhub cli you are using?
CLI Version: 1.7.5
If you were attempting to accomplish a task, what was it you were attempting to do?
credhub import --file <some-file>
What did you expect to happen?
Credhub to say successfully import credentials without printing the credentials to stdout. That is, I expect just the following output.
Import complete.
Successfully set: 3
Failed to set: 0

We are running this command in CI and I didn't expect the CLI to print out all our credentials to concourse web output.
What was the actual behavior?
The current behavior prints out all the credentials to stdout.
http://credhub-api.cfapps.io/version/2.1/#bulk-import
id: 67fc3def-bbfb-4953-83f8-4ab0682ad675
name: /example-ssh
type: ssh
value:
  public_key: ssh-rsa AAAAB3NzaC1y...W9RWFM1
  private_key: |
    -----BEGIN RSA PRIVATE KEY-----
    ...
    -----END RSA PRIVATE KEY-----
version_created_at: 2017-01-01T04:07:18Z

id: 2ba73fbd-439e-40ef-b005-0e1db8815063
name: /example-password
type: password
value: SqFcE2c0AuRvet2YhrxdFbPtkBmjiq
version_created_at: 2017-01-01T04:07:28Z

id: 22a1e87b-ba0b-4bc9-bb26-4e5fc5fb1b2f
name: /example-value
type: value
value: sample
version_created_at: 2017-01-01T04:07:38Z

Import complete.
Successfully set: 3
Failed to set: 0

Please confirm where necessary:

[x] I have included a log output
[ ] My log includes an error message
[x] I have included steps for reproduction

If you are a PCF customer with an Operation Manager (PCF Ops Manager) please direct your questions to support (https://support.pivotal.io/)

@wfernandes Thanks for pointing this out. We'll prioritize this as necessary

@wfernandes thanks again for filing this issue. We have released a fix here: https://github.com/cloudfoundry-incubator/credhub-cli/releases/tag/2.5.0.
Please let us know if you run into anymore issues.


Fix: Desc sorting for deposit and withdrawals

@xenirio you don't need to fork to push code in branches here


[Canary] remove unnecessary tests

Remove unnecessary canaries.
mxnet Paddle doesn't have cu113, mxnet doesn't have cu112. @siddvenk

Just close this PR for now. Let's fix the documentation. https://docs.djl.ai/engines/mxnet/mxnet-engine/index.html we don't have a clear version support


temporary fix to help churn through notifications backlog

This implements option (3) of https://github.com/voltrondata-labs/arrow-benchmarks-ci/issues/148#issuecomment-1644208839.
I will pause the normal schedule_and_publish pipeline schedule, and kick off a Buildkite build of this pipeline on this branch with no global timeout. It should try to go through each item in the queue, one at a time. If it can't get a compare result after 5 minutes (probably 4 tries of 60 seconds each) it will mark that notification as "finished" without posting a message, and say so in the logs.
Hopefully there's only a few problematic notification runs, and the rest of them work.

I forgot that we don't build and push an image for branches in this repo. I think it'll be easier to merge this, kick off the backfill, and then revert it.


NoClassDefFoundError on iOS when using then

I'm getting a rather obscure error on iOS when using then (e.g., Actions.alphaTo(1f) then Actions.alphaTo(1f)):
java.lang.NoClassDefFoundError: java.util.function.Consumer
	at java.lang.VMClassLoader.findClassInClasspathForLoader(Native Method)
	at java.lang.PathClassLoader.findClass(PathClassLoader.java:193)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:515)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:475)
	at ktx.actors.ActionsKt.addUnwrapped(actions.kt:52)
	at ktx.actors.ActionsKt.then(actions.kt:45)

As a workaround, changing to Actions.sequence() works with no issues.
KTX: 1.9.9-b2
libGDX: 1.9.10
Kotlin: 1.3.41
RoboVM: 2.3.7

What's your source compatibility level? It looks like Kotlin tries to compile against Java 8. Try adding these in build.gradle:
sourceCompatibility = 1.6
targetCompatibility = 1.6

compileKotlin {
    kotlinOptions {
        jvmTarget = "1.6"
    }
}

Hm, adding that to both my iOS and core build.gradle files made no difference (sourceCompatibility was previously set to 1.7 in each).

We might have to add the jvmTarget in our build files. I'll publish a snapshot release this weekend with the changes on our side (and LibGDX 1.9.10 compatibility) and let you know, I'd appreciate if you take a look.

I updated the JVM target and uploaded KTX 1.9.10-SNAPSHOT. You can use it by adding this repository in your build.gradle:
maven { url 'https://oss.sonatype.org/content/repositories/snapshots/' }

Let me know if that fixed your issue.

(Build failed due to JDK installation issues, unrelated to Kotlin JVM target.)

Yep, that did the trick. Thanks!

Thanks for testing, I'll release 1.9.10-b1 with the fix this weekend.

@thebnich Just wanted you to know that 1.9.10-b1 should be available shortly. It's pretty much the same in terms of content as the snapshot you're currently using, but you might want to move to the stable version, as 1.9.10-SNAPSHOT can be overridden and contain new API changes.


CI fix

Travis CI using pytorch-nightly has been annoying. Occasionally pytorch/audio master would be passing and then the next day the same test would be fail due to changes in pytorch.
The last passing installs for our python 3.5/3.6 tests were:
pytorch-nightly-cpu-1.1.0.dev20190516|      py3.5_cpu_0
pytorch-nightly-cpu-1.2.0.dev20190716|      py3.6_cpu_0

so want to fix it on 1.1.0.dev20190516

This is dangerous because we're now developing against an arbitrary version. If nightlies keep on breaking our build we should create an issue around that work on the process under which we pick a nightly. Also, its important feedback for pytorch to have us fail on nightlies, since we provide additional coverage. If anything we should maybe look into linking these errors back to the main repo.


Try CircleCI

As an alternative to Travis.
https://circleci.com/
Recommended in: https://github.com/travis-ci/travis-ci/issues/9815#issuecomment-411057404
May help with: https://github.com/opensistemas-hub/osbrain/pull/236

Are we really in a hurry to support 3.7 (which we already know works anyway)?
If it's up to me, I can wait for Travis to support the new version, and switching to CircleCI doesn't seem to be worth the trouble IMO.


Forms: Add TextFormElement tests

Related to issue: #apollo/809

Description:
Introduces TextFormElement tests using the common test design. Note that the test 10.3 will be implemented as an image comparison test when such a framework is available.
Summary of changes:

Added test case 10.1 and 10.2 of the common toolkit tests.
Updated sdk version to the latest since the TextFormElement API is now approved and generated.
Updated FeatureForm doc.

Pre-merge Checklist

a vTest Job for this PR has been run

[x] link: https://runtime-kotlin.esri.com/view/all/job/vtest/job/toolkit/72/

Unit and/or integration tests have been added to exercise this PR's logic, and the tests are passing:

[x] Yes
[ ] No

thanks @sorenoid, merging.


TS2488 When use object in v-for

I have component:
<template>
...
      <li
        v-for="(version, lib) in versions"
        :key="lib"
      >
        <strong>{{ lib }}</strong>: v{{ version }}
      </li>
...
</template>

<script lang="ts">
export default defineComponent({
  setup() {
    const {versions} = useElectron()
    // It makes no sense to make "versions" reactive
    return {versions} // Version is ftat object {node: '14', chrome: '86' ... }
  },
})
</script>

In Runtime it works as expected. But checking the types falls with a error:
Error: src/components/About.vue(8,34): error TS2488: Type 'Readonly<Record<string, string>>' must have a '[Symbol.iterator]()' method that returns an iterator.

This is expected, you can add  & { [Symbol.iterator](): IterableIterator<FOR_TYPE> } to your type for fix.

OK. This really fixed the problem. I wonder what caused this behavior. It doesn't seem intuitive.

For example:
declare let foo: Record<string, string>;
for (const item of foo) { } // TS2488

declare let bar: Record<string, string> & { [Symbol.iterator](): IterableIterator<string> };
for (const item of bar) { } // works

I found that for...in can better to handle it, and it was fixed in vue-tsc 0.0.14. Sorry for make the confusion.


[maintenance] dependency updates

Updated dependencies to keep everything nice and clean.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

@contra sorry but had to revert eslint from ^3.0.0 to ^2.0.0 and babel-eslint from ^7.0.0 to ^6.0.0 for backward compatibility for node v0.12.*

@gnavvy Is that needed for dev deps?


[VERSION TRACK] v2.3.0 Known Issues & Bugs

Known Bugs - v2.3.0
This issue is an umbrella issue to keep track of bugs in the current version.  You can reply directly here, but opening a new issue would make life easier for us.
Issue List

[ ] Possible Feature add #122 - Vehicle/Implement Listing in description
[ ] Possible Feature add #125  - Folder/Collection Last used tracking

2.3.0 IS BROKEN - PLEASE STAND BY FOR 2.4.0

If possible/not difficult, please add lines for translation:

"hp": "л. с.",
"kph": "км/ч",
"mph": "миля/ч",
"m": "м",
"kg": "кг",
"t": "т",

Of course, except for "hp", everything will be intuitively understandable and without translation to users with any knowledge of the English language. :)

Sure, I'll get this in today probably, it will be in the next build.  That's my fault, I am well aware those units are translatable.  Thanks!


fix(android): android gradle plugin 8 compatibility

Hi there 👋
This is the minimal change to support android gradle plugin 8 coming up with react-native 0.73, while still maintaining backwards compatibility with much older react-native that still bundles android gradle plugin <= 6
It does not attempt to do anything with the AndroidManifest.xml file, as leaving the namespace in there is only a warning, but it allows the module to compile when I use it in my work app where I'm testing AGP8+ compatibility in the modules I maintain and use
Cheers

++


Update jsDelivr link

jsDelivr switched to a fully automated system, that can serve files from npm and GitHub. This means all future releases will be available automatically, but will use a new link structure.
I updated the link now so you don't forget to do it when you release a new version.
You can find links for all files at https://www.jsdelivr.com/package/npm/angular-moment.
Feel free to ping me if you have any questions regarding this change.

Coverage remained the same at 93.243% when pulling e451420db1b6fde799d1fbe7eb3ef2c24b3d4a41 on LukasDrgon:patch-1 into 843f6fe93b7514919085be9cef021b966b270017 on urish:master.

Thanks!


NTR: renal zinc excretion

For review https://github.com/geneontology/go-annotation/issues/4346

New term
renal zinc excretion

*is_a GO:0097254 renal tubular secretion

Definition (free text)
The elimination of zinc ions from peritubular capillaries (or surrounding hemolymph in invertebrates) into the renal tubules to be incorporated subsequently into the urine.

Reference

PMID:28196538

Gene product name and ID to be annotated to this term

ZnT63C; FBgn0035432

+[Term]
+id: GO:0180002
+name: renal zinc excretion
+def: "The elimination of zinc ions from peritubular capillaries (or surrounding hemolymph in invertebrates) into the renal tubules to be incorporated subsequently into the urine." [PMID:28196538]
+synonym: "renal zinc ion excretion" EXACT []
+synonym: "renal zinc secretion" EXACT []
+is_a: GO:0097254 ! renal tubular secretion
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/24307 xsd:anyURI
+created_by: vw
+creation_date: 2022-11-01T17:35:12Z


How to connect the hyperdiscovery ?

When the hyperdiscovery have be created ,how to connect it pass dat-node app?

Hey, I suggest using the dat SDK instead which is a sort of replacement for dat-node which uses hyperdiscovery under the hood.


Can you provide arbitrary frame interpolation training code

The existing version only includes the fixed step fill frame training process, I want to learn the process of arbitrary frame interpolation, thx

Thanks for your concern! We won't provide the training code that supports arbitrary temporal interpolation in order to reduce the complexity of the code base. But it should be noticed that the configuration of the training dataset is the only thing that needs to be changed (we follow the setting of RIFE); the rest of the training process is essentially the same for both fixed- and arbitrary-training.

Thank you for your reply. In fact, while waiting for your reply, I have built arbitrary temporal interpolation training methods, also referring to RIFE's Settings. However, during the training process, my training loss is almost unchanged, and the performance of PSNR test after each epoch becomes worse and worse with the training, the interpolation frame calculated by the trained model is all black, have you ever encountered similar cases?

No, the training process was very stable. You can see if the time step is computed correctly and passed into the model.

Hello, @JRStudio-107 and @GuozhenZhang1999 I am also interested in exploring the training of arbitrary temporal interpolation. Could you plz release the relevant code in another branch or repo?

Hi I wanted to ask if the arbitrary timestep training code can be shared? How can I reproduce the results??


Migrate to Gradle

Use save-cloud project as reference
It's a stopper for

#836
#365

It looks like when we will be migrating to Gradle - we will again create /buildSrc directory and that will be a duplication again.
Probably it will be great to create a universal gradle plugin for Kotlin multiplatform plugin that will include main projects.


GCP: support VM images that come with container runtime pre-installed

VM images provided by users might have the container runtime. This issue tracks verifying if the container runtime is already pre-installed and matches the version specified by the user.

cc @kcoronado

This issue was moved to kubernetes-sigs/cluster-api#80


KeyError: 'total_count'

Hello, I used "bash run_sh/run_singlelell_norman. sh" while running the demo of the perturbation prediction, and then an error message "KeyError: 'total_count'" was displayed. I found that the demo. h5ad had a total_ Count, but in normal's perturb_ processed. hbad file does not have this column, what should I do?

Hi, you need to add the total count column to your own data, since our scFoundation model requires the total count information as input. You can check your file with the demo data demo.h5ad we provided.


Release/1.77.0

What
tbc
How to review
Describe the steps required to test the changes.
Who can review
Describe who worked on the changes, so that other people can review.

closing due to 1.78 being imminent


Custom JS files are not copied to public/js folder

Custom JS files place in assets/js/ folder are not copied into public/js folder.
See below for directory structure
assets
└── js
    ├── jquery.js
    └── node_modules
        ├── jquery
        │   ├── dist
        │   │   ├── jquery.js
        │   │   ├── jquery.min.js
        │   │   ├── jquery.min.map
        │   │   ├── jquery.slim.js
        │   │   ├── jquery.slim.min.js
        │   │   └── jquery.slim.min.map
        │   └── package.json

I've followed the instructions at https://sourcethemes.com/academic/docs/customization/#add-scripts-js and added  jquery.js to params.toml
Hugo version 0.7.0, latest version Hugo-academic. Any suggestions? Much appreciated.

for your own local scripts, let’s say that you have a JS file named custom.js. We can place your file in assets/js/ (create the folders within your website root if they don’t exist). Then open your config/_default/params.toml and set plugins_js = ["custom"], where custom is your JS filename without its extension.

Can you show the relevant line of your params.toml? Did you specify just the filename without th .js extension?

In params.toml
# Load JS plugins
#   E.g. To load `/assets/js/custom.js`, set `plugins_js = ["custom"]`.
plugins_js  = ["jquery"]

I can confirm assets/js/jquery.js exists.
After ran hugo, I checked the public/js directory it generated, it has only the following files:
academic.min.01f19851374de3eb9e5c5e6b18db5b9f.js
academic.min.37431be2d92d7fb0160054761ab79602.js
mathjax-config.js
vendor

The custom script should be included in the academic.min.*.js file.

Alternatively, if you are looking to self-host the integrated assets rather than using third party CDN, there is a guide here https://sourcethemes.com/academic/docs/offline-site/


Licenses and contribution

Hey,
In short: I like to contribute. What I am missing atm:

Licenses (file headers as well as license file) are missing names an mail.
Contribution-Guide. Are contributions only welcome as add on?
Gitter/Slack something like that to get in touch without open an issue

What rules are there to contribute?

About chat, what about using matrix?
If not, I can also provide a rocket.chat hosting.

I don't have any preferences, but as project owner @xetys should take this decision IMHO.
In other projects Gitter was easy to setup and offers a badge, most important aspect! :smile:

gitter chat is there

ps: about contributions. I will do some guidelines when I find the time. In general all kind of contribution is welcome, like fixes, addons, docs etc...

thb I would like group some files into packages and write some tests. Also resolve the separates types. AFAIR is a anti pattern in GO to have a separate type/model file. Also a Makefile is needed IMHO.
Doing Fixes? -> No question about it! Of course I will also these things.
If you have no problem with that you will get some PRs soon. Long term I would like to fancy up the CLI respond . Do you have anything in the pipe yet?

Nice! I will do so. Im also not a pro, but did some small projects and eager to train my inner gopher.

@xetys This is very useful! I Imagined I will be using it to deploy kube cluster soon on Hetzner. We are using hetzner a lot. Do you have a milestone to release v1 yet? I'm not a go expert, but probably can help with the docs, example, etc.

sure @lucernae
feel free to chop through our issues and suggest them for project 1.0, I will check that out and add them to the TODO list


Fix lang-error duplicate import warning

Purpose
Jar Resolver's getJarFilePathsRequiredForTestExecution causes a warning stating duplicate entries for lang-error. This is working fine with the current release, but when creating the uber jar for the test modules, this function is used.
Fixes #41993
Approach
Inside the getJarFilePathsRequiredForTestExecution function, filter the langlib packages from packageResolution's dependencies.
Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Commit messages must be in sentence case (Starting letter must be Upper Case).

Commit messages must be in sentence case (Starting letter must be Upper Case).

Changed them to sentence case

Squashed the commits


Fails to install Poetry when running docker-compose

Step 6/8 : RUN poetry install &&     bundler install
 ---> Running in da6a35f96e3d
/bin/sh: poetry: not found
ERROR: Service 'web' failed to build: The command '/bin/sh -c poetry install &&     bundler install' returned a non-zero code: 127

This is the error I get when I run docker-compose up -d

I solved this bug, and website is running sucessfully  in my browser, but the main features like pin, follow is incomplete.

the same problem too

I solved this bug, and website is running successfully in my browser, but the main features like pin, follow is incomplete.

Can you please share your solution. I just faced the same issue.

i solved the problem with changing dockerfile to:
FROM python:2.7
ENV LIBRARY_PATH=/lib:/usr/lib
WORKDIR /stream
ADD . /stream
RUN apt update
RUN apt install -y python-pip
RUN pip install poetry==0.8.1
RUN  poetry install
EXPOSE 8000
CMD ["poetry", "run", "python", "manage.py", "runserver", "<IP_ADDRESS>:8000"]

In case it helps anyone years later 😅  I kept the same base image but amended just a couple of lines below. Looks like py2-anyjson support was dropped from alpine here and then the poetry stuff I guess was ripped off from above :D
RUN apk add --no-cache --repository http://dl-3.alpinelinux.org/alpine/edge/testing/ \
-    git zlib-dev jpeg-dev gcc musl-dev postgresql-dev py2-anyjson ruby-bundler py2-pip libjpeg curl && \
+    git py-pip zlib-dev jpeg-dev gcc musl-dev postgresql-dev py-anyjson ruby-bundler py2-pip libjpeg curl && \
     curl -sSL https://raw.githubusercontent.com/sdispater/poetry/master/get-poetry.py | python

 WORKDIR /stream

 ADD . /stream

+RUN pip install poetry==0.8.1
 RUN poetry install && \
     bundler install

 EXPOSE 8000

-CMD ["poetry", "run", "python", "manage.py", "runserver", "<IP_ADDRESS>:8000"]
+CMD ["poetry", "run", "python", "manage.py", "runserver", "<IP_ADDRESS>:8000"]


DBZ-6676 Oracle XML/RAW documentation updates

https://issues.redhat.com/browse/DBZ-6676
@roldanbob This is a follow-up from DBZ-3605 (PR #4615).

@roldanbob I addressed everything but this one discussion referenced here:
https://github.com/debezium/debezium/pull/4615#discussion_r1245794743


Change text in Chrome

Type of issue

[ ] Bug
[ ] New feature
[ ] Enhancement

Description
Replace app title with Microsoft brand adherent: "Microsoft Azure IoT Suite Remote Monitoring Preview"

if this is the browser title, i.e. the browser Tab title, I'd prefer a "Remote Monitoring - Microsoft Azure IoT Suite", so when the browser truncates it, the most contextual information remains visible.

@dluc I will check in with the Content designer about it, but I think your logic makes sense


Update usage.md

纠正 cancel execution 一节的示例代码。

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

cancel execution 一节的示例代码重复了一行。


Upgrade Elasticsearch to 5.4.x

This PR upgrades Spandex to Elasticsearch version 5.4.x. The largest changes to make this happen involved switching from the always in-memory completion suggester to using edge_ngrams + term queries, in conjunction with an aggregation (https://github.com/elastic/elasticsearch/issues/22912) to dedupe results. This approach trades off some performance for a more scalable and simpler architecture.
In addition to these architectural changes, we cleaned up the repo a bit to simplify index maintenance, and for consistency with ES client code in cetera.
Unit testing via a local node is no longer supported in Elasticsearch. As a separate chunk of work, we will use the new Elasticsearch integration test framework to increase our test coverage. We may also just continue to add tests to Cheetah for tests that make more sense as integration tests.

This build looks good.


recognise and grant maintainer role

or module-author?

Maybe the amount of modules maintained too !

As well yes

@Atinux this issue can be closed by https://github.com/nuxt/nuxters/pull/53


Add option 'ignoreItemOffsetOnMirrorPosition'

When the user drag an item -> the mirror DOM item get positionned relative to the mouse position with OR without correcting it with its offsets.

@Metroslim Can you explain this further? Is this in reference to an issue?

@dcantatore I didn't look at the issues before doing this PR but it seems that it can answer the issue https://github.com/bevacqua/dragula/issues/616.
If the option is set to true then the mirror DOM will be position on the pointer. To be precise, the left top corner of the mirror DOM will be just under the pointer. To fullfill the need of the related issue you just have to add transform: translate(-50%) to the .gu-mirror css.
If you want I can take some screen capture to demonstrate.

@Metroslim -  Screen capture would be great. Can you add the css change as well as the information to the readme?

Hey, screen capture wasn't that easy (confidential project) but I just push updates on Readme and Examples with the information relative to this new option. I'm not an native english speaker so maybe you will find better way to explain the feature. BTW, I'm using ng2-dragula and if you merge this PR you will have to update the @types so that the DragulaOption will be in sync. Thank for your work !

Hi there, I haven't heard from you since two weeks. Did you look at my new commit that include a new section in the example to demonstrate what this new option offer ?
(BTW I may be using swivel for handling communication between pages and service worker so thanks again for the gret work !)


nginx: MacOS build

As #17 and #23 discussed, the patched nginx needs to be linked with Apple's Security framework when compiling on macOS. https://github.com/cloudflare/quiche/commit/e1f9d135cae6f8aedaaebb31457b1234f115bc60 should fix that.

Merged, thanks @hguandl!


Cut release v0.22.7

What's Changed
Most importantly the syntax for tags is now $myTag to declare a tag and myTag to then use it later. If you format your code via Alt+Shift+F or the three dot menu on the code panel it will fix this for you. We will keep backwards compatibility with string tags for a few releases but you should update your code now.
The nice thing about this is now the code editor autocompletes tag names for you and now you can get better errors when you abuse tags.
So for example this code:
const sketch001 = startSketchOn('XZ')
  |> startProfileAt([65.87, 230.28], %)
  |> line([248.25, -79.48], %)
  |> line([-74.04, -283.09], %, 'myTag')
  |> line([-192.72, 235.18], %)
  |> lineTo([profileStartX(%), profileStartY(%)], %)
  |> close(%)
  |> extrude(100, %)

const sketch002 = startSketchOn(sketch001, 'myTag')
  |> startProfileAt([185.8, 131.39], %)
  |> line([62.76, -12.99], %)
  |> line([-27.05, -47.61], %)
  |> line([-43.65, 9.37], %)
  |> lineTo([profileStartX(%), profileStartY(%)], %)
  |> close(%)

Becomes:
const sketch001 = startSketchOn('XZ')
  |> startProfileAt([65.87, 230.28], %)
  |> line([248.25, -79.48], %)
  |> line([-74.04, -283.09], %, $myTag)
  |> line([-192.72, 235.18], %)
  |> lineTo([profileStartX(%), profileStartY(%)], %)
  |> close(%)
  |> extrude(100, %)

const sketch002 = startSketchOn(sketch001, myTag)
  |> startProfileAt([185.8, 131.39], %)
  |> line([62.76, -12.99], %)
  |> line([-27.05, -47.61], %)
  |> line([-43.65, 9.37], %)
  |> lineTo([profileStartX(%), profileStartY(%)], %)
  |> close(%)

More semantic tokens modifiers (#2823)
Move walk handlers out of lint (#2822)
Update onboarding KCL (#2820)
Only show one error at once (#2801)
More expressive semantic tokens (#2814)
Add more tests for various scenarios (#2812)
Allow lifetime refs in KCL stdlib parameters (#2802)
Hide grid (#2777)
Fix some recast bugs (#2781)
Tag as top level construct  (#2769)
Tags are globals (#2795)
Add sketch tool events to command bar (#2708)
Allow for sketching on the face of a chamfer in kcl (#2760)
Execute chamfers and fillets early if in a pattern (#2759)
Semantic tokens used for highlighting (#2806)
Delete key works for click and point (#2752)
Rework zoom (https://github.com/KittyCAD/modeling-app/pull/2798)
Add setting for grid visibility (https://github.com/KittyCAD/modeling-app/pull/2838)
Fallback colors for different browsers (https://github.com/KittyCAD/modeling-app/pull/2770)

Full Changelog: https://github.com/KittyCAD/modeling-app/compare/v0.22.6...v0.22.7

Testing on macOS:

sign in
open project
close project
create project
create sketch
create rectangle
create var
resize with var
extrude with var
export to STL: failed
export to OBJ: failed

Issue created #2840

between ubuntu and macos they playwright arent failing on the same tests so i think we are okay.

the tests work locally for me and i pushed a fix to one to make more reliable

Did my usual test round on macOS, looks good. Will try Windows in a minute

Windows looking good too


fix(tasks/coverage): when providing filter, do not run just submodules

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #2309

PR #2310   👈

This stack of pull requests is managed by Graphite.


Iterate 1st tranche of IA guidance

Iterate 1st tranche of IA guidance after user testing.

[ ] Take IA out of headings/page titles

About IA

[ ] Make What IA is say "What is IA?"
[ ] Headings found to be “jarring” by dyslexic user- H1s, H2s, H3s with very little content in between - need more info
[ ] Most users want to see more examples, and “see examples of 'not how to do it' vs ' how to do it', to see the value of a good IA and how to do IA”
[ ] Suggestion of using NHS.uk as a case study for examples throughout the guidance
[ ] Make examples visual (note: only one user saw visuals on Why IA is important)
[ ] Iterate images on Why IA is important
[ ] Users tried to click blog posts – would be good examples, but need more context: what could I learn from it? Where is the link taking me?”
[ ] IA team contact info – add what they can support with, and how to contact- crosslink to other page
[ ] IA principles - One user still wanting an example on this page, or linking off to examples on other pages

Your topic in the wider IA

[ ] “Topic” not fully explained – users didn’t find a definition, one equated to content.
[ ] User confused by word “wider” in title
[ ] IA team contact- add another form of contact for those not on slack
[ ] No definition of canons, unclear
[ ] Opportunity to flesh out importance of synonyms using NHS.uk as example

Measure the success of your IA

[ ] Agile terms – some users want reference to when this is done (e.g discovery, beta) while others state agile isn’t used by all teams so should be used.
[ ] Need to include info on what to do if no access to analytics, or how to gain access
[ ] Examples – suggestion of examples needed for internal search, user needs
[ ] Definition of information scent from Why IA is important needed here, or cross linked
[ ] “content switches between general and NHS specific.. Could the difference be made clearer?”

Overall

[ ] More examples, more visual e.g use diagrams
[ ] How does this fit into a wider design process?


fix the issue failed to get infra in ocp311

Signed-off-by: Zhiwei Yin<EMAIL_ADDRESS>fix issue: https://github.com/open-cluster-management/backlog/issues/18477

/cherry-pick release-2.4

/assign @elgnay

/lgtm


cleanup some TODOs in SDS

https://github.com/istio/istio/issues/5903
some clean up which includes -

use workload's IPAddress as certOptions.Host to generate key/cert.
update secretName used in SDS reponse to envoy.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: quanjielin
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: incfly
Assign the PR to them by writing /assign @incfly in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

security/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @JimmyCYJ

as synced offline, @JimmyCYJ 's prototype shows client could pass secret name in DiscoveryRequest.ResourceNames to server, so that server side can parse secretName and use it to

construct response to clients
construct csr template

so close this PR.


Variable paginator.pages not found in context while rendering 'index.html'

New project, theme cloned to the themes dir and set in config.
>zola serve
Building site...
Checking all internal links with anchors.
> Successfully checked 0 internal link(s) with anchors.
-> Creating 0 pages (0 orphan) and 0 sections
Error: Failed to render section 'D:\Workspaces\StPauls\stpauls-www\content\_index.md'
Reason: Failed to render 'index.html'
Reason: Variable `paginator.pages` not found in context while rendering 'index.html'

zola 0.15.3

I will take a look and give you a feedback.
If you don't want to wait you can take a look at this repo:
https://github.com/marcodpt/marcodpt.github.io
It is a sample that I use myself.

same problem here

New project, theme cloned to the themes dir and set in config.
>zola serve
Building site...
Checking all internal links with anchors.
> Successfully checked 0 internal link(s) with anchors.
-> Creating 0 pages (0 orphan) and 0 sections
Error: Failed to render section 'D:\Workspaces\StPauls\stpauls-www\content\_index.md'
Reason: Failed to render 'index.html'
Reason: Variable `paginator.pages` not found in context while rendering 'index.html'

zola 0.15.3

cover  all the files from themes/templates including config.toml and content dir


fix bug with "Next" pager button

"Next" button did not work correctly when you have more then 10 pages.

Seems to work okay here: http://live.datatables.net/huseqesu/1/edit .
Can you link to a test case showing the issue please? My guess is that something is being given as a string rather than a number.
Also, this isn't the repo to submit changes for DataTables. This is a generated repo only. The DataTablesSrc repo one is where to submit changes.

Sorry, there was bug in my settings.


Link Color Issues

Thanks for the wonderful library. I am using Version 7 of your library that was published through xamarin. My issue is related to the link color of the cell.UITextView. By default it is always white. I can change the foreground color of the message bubble text by setting the cell.TextView.TextColor which works great. But the link color always appears as White. I tried changing the TintColor of the UITextView so that the link color will change, but that did not work. Here is the code that I am using. Could someone please help?
public override UICollectionViewCell GetCell (UICollectionView collectionView, NSIndexPath indexPath)
{
    var cell =  base.GetCell (collectionView, indexPath) as MessagesCollectionViewCell;
    cell.TextView.DataDetectorTypes = UIDataDetectorType.All;

    var message = messages [indexPath.Row];

    if (message.SenderId == SenderId) {

        // My color
        cell.TextView.TextColor = UIColor.White;
        cell.TextView.TintColor = UIColor.Yellow;

    } else {

        // Other party color
        cell.TextView.TextColor = UIColor.Black;
        cell.TextView.TintColor = UIColor.Blue;
    }

    return cell;
}

Link highlight colors are controlled by UITextView's linkTextAttributes property.
This is what we do in Swift (not sure the exact C# syntax)
cell.textView.linkTextAttributes = [NSForegroundColorAttributeName: linkColor, NSUnderlineStyleAttributeName: 1]

Thanks @enouveau / @harlanhaskins !

Thanks HarlanHaskins
Here is the C# code if anyone else would need it in the future:
var key1 = new NSMutableString ();
key1.SetString (new NSString ("NSColor"));
var value1 = UIColor.Blue;

var key2 = new NSMutableString ();
key2.SetString (new NSString ("NSUnderline"));
var value2 = new NSNumber(1);

var dict = new NSDictionary (key1, value1, key2, value2);

cell.TextView.WeakLinkTextAttributes = dict;

Does Xamarin not provide the NSString constants? Those strings aren't guaranteed to be "NSColor" and "NSUnderline"...
Also do you need to use an NSMutableString there?

Good questions. I put a breakpoint in the debugger to check what type was the key and it showed a NSMutableString. So I went with that. I am not aware of any NSColor or NSUnderline constants in xamarin. Or I might not be looking at the right place.

I am using UIStringAttributeKey.ForegroundColor and UIStringAttributeKey.UnderlineStyle

I know this is deprecated, but if anyone is looking for this on swift 4 the values have changed.
cell.textView?.linkTextAttributes = [NSAttributedStringKey.foregroundColor.rawValue : UIColor.blue , NSAttributedStringKey.underlineStyle.rawValue : NSUnderlineStyle.styleSingle.rawValue]

I want to remove underline but this is not working
var key2 = new NSMutableString ();
key2.SetString (new NSString ("NSUnderline"));
var value2 = new NSNumber(1);

Thanks HarlanHaskins
Here is the C# code if anyone else would need it in the future:
var key1 = new NSMutableString ();
key1.SetString (new NSString ("NSColor"));
var value1 = UIColor.Blue;

var key2 = new NSMutableString ();
key2.SetString (new NSString ("NSUnderline"));
var value2 = new NSNumber(1);

var dict = new NSDictionary (key1, value1, key2, value2);

cell.TextView.WeakLinkTextAttributes = dict;

We are trying this code but underline cant not removed.
var value2 = new NSNumber(0);  //its not working

Link highlight colors are controlled by UITextView's linkTextAttributes property.
This is what we do in Swift (not sure the exact C# syntax)
cell.textView.linkTextAttributes = [
    NSForegroundColorAttributeName: linkColor,
    NSUnderlineStyleAttributeName: NSUnderlineStyle.StyleSingle.rawValue
]

Simple C# port:
var linkTextAttributes = NSDictionary.FromObjectsAndKeys(
    new NSObject[]
    {
        NSNumber.FromInt32((int)NSUnderlineStyle.Single),
        UIColor.Red
    },
    new []
    {
        UIStringAttributeKey.UnderlineStyle,
        UIStringAttributeKey.ForegroundColor
    });

_textView.WeakLinkTextAttributes = linkTextAttributes;


Docstrings for enum bevy_app::CoreStage are not consistent

How can Bevy's documentation be improved?
All the other docstrings for the variants use the phrasing "Name of app stage X", while the docstring for CoreStage::Startup simply describes the stage.
While I personally am not a fan of the "Name of app stage" phrasing, if it is being used it should be used consistently.
Phrasing of either the Startup docstring or the other docstrings should be adjusted to match.  Probably Startup should be changed in order to match both the other variants of CoreStage and the docstrings for StartupStage.
Relevant source code
https://github.com/bevyengine/bevy/blob/51a5070cd2d3d7262616b2efc8a6cd3b1991d7e6/crates/bevy_app/src/lib.rs#L23-L39

I did not realize this while I was writing this issue but PR #2434 renders this issue meaningless by removing the enum variant in question.


Mark MapGetWithNotNullAssertionOperator rule to require type resolution.

This relates to #2994
This rule should have always declared to require type resolution. I cannot find violation without a binding context.

Warnings

:warning:
It looks like this PR contains functional changes without a corresponding test.

  Generated by :no_entry_sign: dangerJS against dc7b0e1103902456414139d8031b443d86e87479


Added a Type "Comment" that has to start with "#"

made a solution for issue #64
Just write # as first char of your ip and it wont do anything. Use it as a placeholder or if you know about an error, restart vmping often and do not want to start this to ping automatically
Only these two files should be changed, the others were accidentially commited to that branch and the pull request. So just update:
vmPing/Classes/Probe.cs
vmPing/Classes/Probe-Util.cs

missing feedback -> close.


Check question content

Fixes #108
Depends on #143

Lel conflicts everywhere

Made requested changes @indocomsoft

alright done

Let's wait until #147 is merged. I'll fix the tests to use the new helper.

Alright, I just merged from master. Just make sure you git pull --rebase before pushing

@indocomsoft Refactored tests to use new ChangesetCase, should be good to go


Fixup styling and documentation

CC @Geod24

Edit: updated with missing this.

Added another commit.

Fixed.

Rebased.


Changing swiping direction?

Question: I want to implement a stack-of-cards type of swiper, but I want user to be able to swipe right instead of left, so stack itself would be on the left side. How's it possible?
Thank you.

Hi @pukhalski,
The good news is: I've already done that!
Take a look at the source code for the stack effect and you'll see that the effect has been reverted between iOS and Android — you can preview the difference here.
One word of caution though: make sure to read about custom interpolation's caveats and to take a look at #262.

Thank you, @bd-arc. That was quick ;)

Is there a reason why the effect is reversed for iOS and Android? Just curious :)


Camera with Environment gets smeared on the right side

If a viewport camera has an environment attached the result is a smearing of the right side of the image.

On what platform do you experience this issue? Does it happen from the start, or only after resizing the viewport? What are your project resolution and stretch settings?
Cool graphics btw :D

The platform is Windows 8.1, Godot 2.0 Alpha. I am previewing the project on a 1200x1920 screen (in vertical position). The stretching does NOT appear if the preview is not stretched (in my case shrinked) to fit my monitor.
My Settings are:

Thanks.

Might be related to #1571 and #2377. There is a rebasing issue in the PR currently but maybe you could try to apply the patch directly to see if it fixes your issue?
See: https://github.com/rippledodger/godot/commit/f15bd9c8aa6ff62ed3358f4809b735a56110f3fb.patch
This requires being able to compile Godot from git of course.

I applied the patch (well one who knows programming did) and this was the result. Not an improvement unfortunately. The black areas is where there used to be image. Ah well, rolling back Godot.

Does it happen if you change stretch mode from viewport to 2D?

you most likely don't want this kind of stretching if you are doing 3D, as it is meant mostly for 2D games

This on screen smearing still happens when stretch mode is set to 2D.

Can I get a test project for this?

a very simple test project would be apreciated

this was fixed by a recent commit (tested on screenspace shaders project) , repoen if it still fails in different cases


If table has id, changeColumn() throws error when reverting

After adding a new table to my database, and generating the migrate script for it, everything seemed well. the actual migration went flawlessly, and worked every time.
The error happens when I try to revert those migrations. The generated script checks whether a column named 'id' already exists, and modifies it if it does. The problem is that the change() method cannot handle changeColumn(), and crashes showing the following error.
Fatal error: Uncaught Phinx\Migration\IrreversibleMigrationException: Cannot reverse a "changeColumn" command in [PATH]\robmorgan\phinx\src\Phinx\Db\Adapter\ProxyAdapter.php:228
This is the generated migrate script:
    <?php

    use Phinx\Migration\AbstractMigration;
    use Phinx\Db\Adapter\MysqlAdapter;

    class AddNewsTable extends AbstractMigration
    {
        public function change()
        {
          $table = $this->table("news", ['engine' => "InnoDB", 'encoding' => "utf8", 'collation' => "utf8_general_ci", 'comment' => ""]);
      $table->save();
      if ($this->table('news')->hasColumn('id')) {
          $this->table("news")->changeColumn('id', 'integer', ['null' => false, 'limit' => MysqlAdapter::INT_REGULAR, 'precision' => 10, 'identity' => 'enable'])->update();
      } else {
          $this->table("news")->addColumn('id', 'integer', ['null' => false, 'limit' => MysqlAdapter::INT_REGULAR, 'precision' => 10, 'identity' => 'enable'])->update();
      }
      $table->addColumn('title', 'string', ['null' => false, 'limit' => 255, 'collation' => "utf8_general_ci", 'encoding' => "utf8", 'after' => 'id'])->update();
      $table->addColumn('body', 'string', ['null' => false, 'limit' => 255, 'collation' => "utf8_general_ci", 'encoding' => "utf8", 'after' => 'title'])->update();
      $table->addColumn('created_at', 'datetime', ['null' => false, 'default' => 'CURRENT_TIMESTAMP', 'after' => 'body'])->update();
      $table->save();
    }
    }

Is deleting and recreating the column a possible solution?
Great project and thanks!

Hi @bramkorsten Well, reverting a migration is possible (in the theory), but not possible in every case and not 100% supported by Phinx itself. Even simple reverse operations will fail with a error message like this. That's why I decided to not implement a reverse migration feature.

Is deleting and recreating the column a possible solution?

In the development environment this may still be ok, but in production I wouldn't do it. I would recommend never reverting a migration, as it is only too dangerous and error-prone. A backup before migration would be the safer method.

Ah I see... But isn't that the whole 'shablam' about migrations? Being able to rollback to say a previous version or something like that? It seems that for a simple migration like creating a simple table, rolling back shouldn't be an issue.
I know that I could always go back to writing the up() and down() scripts myself, but your tool seemed like the perfect way to handle this for me.
Anyway, thanks for the help!

But isn't that the whole 'shablam' about migrations?

For me, migrations are an important part of continuous integration (CI) and continuous deployment (CD). The primary goal is to automate all these deployment steps. Today it is simply too complex and error-prone to manually migrate all servers to a defined database schema version. Even if you manage to migrate the schema back, this does not mean that this is also possible with your data.
If you really need a down() method, of course you still have the possibility to implement it yourself.
So far I have seen no need to generate a down() method. If there are more requests in this regard, I could imagine implementing it.


args.rs:28-30: Add --v option for verbose output. Let's...

The puzzle 79-d0a80208 from #79 has to be resolved:
https://github.com/h1alexbel/fakehub/blob/6932d8bc8d0c334bc7dba4106a820a77e305f622/cli/src/args.rs#L28-L30
The puzzle was created by @rultor on 18-Jul-24.
Estimate: 25 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd thanks for the report, here is a feedback: * The todo message is informative but it lacks spec...

I see you're talking to me, but I can't reply since I'm not a chat bot.

@0pdd the puzzle #105 is still not solved.


Landing page: Cosmetic & responsive design updates

The HRA Dashboards landing page has been adjusted to be more dynamic and cosmetic.
This is a low priority development request.
Please see my annotated image of the updates and new section of the MVP1 canvas for all adjustments/breakpoints:

@LibbyUX fixed in PR #477 . Thanks

@LibbyUX can you review and close if finished?

@bherr2 yes, thank you for the reminder ping!
Off the top, it does need to be closer to spec:

Currently, the font on the visual cards does not match the spec. The live production cards use Nunito Sans, however, this was specified using our Label font, Metropolis 500/Medium.
Diversity & Inclusion needs to be reordered still in navigation

Let me review this more thoroughly tomorrow morning. I think the footers may need to match the design system footer as well. Adding to my task sheet. Thank you so much!

@LibbyUX please review from this URL for now https://cdn.humanatlas.io/ui/dashboard-ui/

Compared with notes. This is looking so fabulous!

Display font & overall notes
Visual cards: Questions & Requests
Footer: Requests

Also, two super exciting notes to document in GitHub as well as the Figma canvas:

The Display font weight in this preview looks AMAZING. This production font weight be applied globally to all interfaces using it.
The footer is so close. After these updates, it too can be applied globally.

Thank you! Go team!!

Hi @axdanbol, wanted to provide a status update on this one:
We can close this one out when the visual cards match the spec.
I'm looking for:

The specified interaction states
The bottom of the card has a white container for the label space
The alignment text alignment is left center
There is no wrapping of the text when the components resize

Please let me know if there is anything else I can do for this! Thanks so much!

@LibbyUX I fixed the styles for the card. Please review here


Extract sources from .egg files for code analysis

Rather than ignoring .egg files, we should extract the source files from inside them so we can analyze the package.

(See #215 for an example)

Not a high enough priority, and the Python community is moving away from setuptools.


Issues when compiled as SPM dependency (Xcode 11)

Some missing imports of UIKit let SPM fail to compile Parchment.
I added all imports to be able to use Parchment as a dependency.

Awesome! Thanks for fixing this 🙌


Allow passing a color to default

In some circumstances it is nice to be able to bypass the palette and choose a specific color for the avatar directly. We should allow a color parameter that will override the default.

I assume this is for the background color? And this would be a similar implementation to the size parameter?

@ClemMakesApps that's correct - the background color and similar implementation to the size parameter yep.
We'd also need to update the README to reflect the new param once implemented!

Sounds good. What are your thoughts on a parameter to change the text color?

Closed by 83a151467808f4c9f0dfa1b1b9b49b2b7c3d794d


Add STEM Academy

https://github.com/tapyu/awesome-stem-academy
An awesome list of academic resources which is tailored to STEM, that is, Science, Technology, Engineering, Mathematics (no Arts). Specially focused on theoretical aspects.
By submitting this pull request I confirm I've read and complied with the below requirements 🖖
Please read it multiple times. I spent a lot of time on these guidelines and most people miss a lot.
Requirements for your pull request

[x] Don't open a Draft / WIP pull request while you work on the guidelines. A pull request should be 100% ready and should adhere to all the guidelines when you open it. Instead use #2242 for incubation visibility.
[x] Don't waste my time. Do a good job, adhere to all the guidelines, and be responsive.
[x] You have to review at least 2 other open pull requests.
Try to prioritize unreviewed PRs, but you can also add more comments to reviewed PRs. Go through the below list when reviewing. This requirement is meant to help make the Awesome project self-sustaining. Comment here which PRs you reviewed. You're expected to put a good effort into this and to be thorough. Look at previous PR reviews for inspiration. Just commenting “looks good” or simply marking the pull request as approved does not count! You have to actually point out mistakes or improvement suggestions. Comments pointing out lint violation are allowed, but does not count as a review.
https://github.com/sindresorhus/awesome/pull/2893#pullrequestreview-1799927209 and https://github.com/sindresorhus/awesome/pull/2888#pullrequestreview-1799928333
[x] You have read and understood the instructions for creating a list.
[x] This pull request has a title in the format Add Name of List. It should not contain the word Awesome.

✅ Add Swift
✅ Add Software Architecture
❌ Update readme.md
❌ Add Awesome Swift
❌ Add swift
❌ add Swift
❌ Adding Swift
❌ Added Swift

[x] Your entry here should include a short description of the project/theme of the list. It should not describe the list itself. The first character should be uppercase and the description should end in a dot. It should be an objective description and not a tagline or marketing blurb. It should not contain the name of the list.

✅ - [iOS](…) - Mobile operating system for Apple phones and tablets.
✅ - [Framer](…) - Prototyping interactive UI designs.
❌ - [iOS](…) - Resources and tools for iOS development.
❌ - [Framer](…)
❌ - [Framer](…) - prototyping interactive UI designs

[x] Your entry should be added at the bottom of the appropriate category.
[x] The title of your entry should be title-cased and the URL to your list should end in #readme.

Example: - [Software Architecture](https://github.com/simskij/awesome-software-architecture#readme) - The discipline of designing and building software.

[x] No blockchain-related lists.
[x] The suggested Awesome list complies with the below requirements.

Requirements for your Awesome list

[x] Has been around for at least 30 days.That means 30 days from either the first real commit or when it was open-sourced. Whatever is most recent.
[x] Run awesome-lint on your list and fix the reported issues. If there are false-positives or things that cannot/shouldn't be fixed, please report it.
[x] The default branch should be named main, not master.
[x] Includes a succinct description of the project/theme at the top of the readme. (Example)

✅ Mobile operating system for Apple phones and tablets.
✅ Prototyping interactive UI designs.
❌ Resources and tools for iOS development.
❌ Awesome Framer packages and tools.

[x] It's the result of hard work and the best I could possibly produce.
If you have not put in considerable effort into your list, your pull request will be immediately closed.
[x] The repo name of your list should be in lowercase slug format: awesome-name-of-list.

✅ awesome-swift
✅ awesome-web-typography
❌ awesome-Swift
❌ AwesomeWebTypography

[x] The heading title of your list should be in title case format: # Awesome Name of List.

✅ # Awesome Swift
✅ # Awesome Web Typography
❌ # awesome-swift
❌ # AwesomeSwift

[x] Non-generated Markdown file in a GitHub repo.
[x] The repo should have awesome-list & awesome as GitHub topics. I encourage you to add more relevant topics.
[x] Not a duplicate. Please search for existing submissions.
[x] Only has awesome items. Awesome lists are curations of the best, not everything.
[x] Does not contain items that are unmaintained, has archived repo, deprecated, or missing docs. If you really need to include such items, they should be in a separate Markdown file.
[x] Includes a project logo/illustration whenever possible.

Either centered, fullwidth, or placed at the top-right of the readme. (Example)
The image should link to the project website or any relevant website.
The image should be high-DPI. Set it to a maximum of half the width of the original image.
Don't include both a title saying Awesome X and a logo with Awesome X. You can put the header image in a # (Markdown header) or <h1>.

[x] Entries have a description, unless the title is descriptive enough by itself. It rarely is though.
[x] Includes the Awesome badge.

Should be placed on the right side of the readme heading.

Can be placed centered if the list has a centered graphics header.

Should link back to this list.

[x] Has a Table of Contents section.

Should be named Contents, not Table of Contents.
Should be the first section in the list.
Should only have one level of nested lists, preferably none.
Must not feature Contributing or Footnotes sections.

[x] Has an appropriate license.

We strongly recommend the CC0 license, but any Creative Commons license will work.

Tip: You can quickly add it to your repo by going to this URL: https://github.com/<user>/<repo>/community/license/new?branch=main&template=cc0-1.0 (replace <user> and <repo> accordingly).

A code license like MIT, BSD, Apache, GPL, etc, is not acceptable. Neither are WTFPL and Unlicense.
Place a file named license or LICENSE in the repo root with the license text.
Do not add the license name, text, or a Licence section to the readme. GitHub already shows the license name and link to the full text at the top of the repo.
To verify that you've read all the guidelines, please comment on your pull request with just the word unicorn.

[x] Has contribution guidelines.

The file should be named contributing.md. The casing is up to you.
It can optionally be linked from the readme in a dedicated section titled Contributing, positioned at the top or bottom of the main content.
The section should not appear in the Table of Contents.

[x] All non-important but necessary content (like extra copyright notices, hyperlinks to sources, pointers to expansive content, etc) should be grouped in a Footnotes section at the bottom of the readme. The section should not be present in the Table of Contents.
[x] Has consistent formatting and proper spelling/grammar.

The link and description are separated by a dash. Example: - [AVA](…) - JavaScript test runner.
The description starts with an uppercase character and ends with a period.
Consistent and correct naming. For example, Node.js, not NodeJS or node.js.

[x] Does not use hard-wrapping.
[x] Does not include a CI (e.g. GitHub Actions) badge.You can still use a CI for linting, but the badge has no value in the readme.
[x] Does not include an Inspired by awesome-foo or Inspired by the Awesome project kinda link at the top of the readme. The Awesome badge is enough.

Go to the top and read it again.

Unicorn

I am really sorry, the PR https://github.com/sindresorhus/awesome/pull/2924 was closed because I wrongly delete the forked repo. I tried to recover it many and many times, but without success, so I needed to started it over in a new PR. I apologize for that.

Comments pointing out lint violation are allowed, but does not count as a review.

It looks very messy to have the links before the item title.

@sindresorhus That is the whole point of this awesome list: multiple links of a single topic/course. I cannot change that. Otherwise, the awesome will get too messy.

Plus, you can see that I strictly followed every guidelines you stated.

FYI, although at first glance it might seem different, it is not messy, at all. Every tag link, as well its positioning and occurrence, is strictly defined in CONTRIBUTING.md. I invite you to read it through to see how well organized the tags are in this awesome list.

Just commenting “looks good” or simply marking the pull request as approved does not count! You have to actually point out mistakes or improvement suggestions. Comments pointing out lint violation are allowed, but does not count as a review.

The links could be on a second line or something. It makes no sense to have to scan the link titles with your eyes before you actually see the title.

The links could be on a second line or something. It makes no sense to have to scan the link titles with your eyes before you actually see the title. The list will not be accepted in the form.

Agreed that the tag links preceding the short title might not be the best approach, but I do think creating a second line to put them is a good solution either. I would make the list necessarily long. Instead, I put the tag links in the end of the short title so that the reader's eyes read first the item title, and then the links. Let me see if this solution suits for you.

Just commenting “looks good” or simply marking the pull request as approved does not count! You have to actually point out mistakes or improvement suggestions. Comments pointing out lint violation are allowed, but does not count as a review.

That is not my case. In PR I've contributed, I point out mistakes if any was found.


Crash on start

> ~/.local/bin/lsd outline /Users/antony/Downloads/italy\ trip\ august.md
Traceback (most recent call last):
  File "/Users/antony/.local/bin/lsd", line 8, in <module>
    sys.exit(main())
  File "/Users/antony/.local/pipx/venvs/logseq-doctor/lib/python3.10/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/Users/antony/.local/pipx/venvs/logseq-doctor/lib/python3.10/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/Users/antony/.local/pipx/venvs/logseq-doctor/lib/python3.10/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/antony/.local/pipx/venvs/logseq-doctor/lib/python3.10/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/antony/.local/pipx/venvs/logseq-doctor/lib/python3.10/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/Users/antony/.local/pipx/venvs/logseq-doctor/lib/python3.10/site-packages/logseq_doctor/cli.py", line 34, in outline
    print(flat_markdown_to_outline(file.read()))
  File "/Users/antony/.local/pipx/venvs/logseq-doctor/lib/python3.10/site-packages/logseq_doctor/__init__.py", line 98, in flat_markdown_to_outline
    return mistletoe.markdown(markdown_contents, LogseqRenderer)
  File "/Users/antony/.local/pipx/venvs/logseq-doctor/lib/python3.10/site-packages/mistletoe/__init__.py", line 21, in markdown
    return renderer.render(Document(iterable))
  File "/Users/antony/.local/pipx/venvs/logseq-doctor/lib/python3.10/site-packages/mistletoe/base_renderer.py", line 94, in render
    return self.render_map[token.__class__.__name__](token)
  File "/Users/antony/.local/pipx/venvs/logseq-doctor/lib/python3.10/site-packages/mistletoe/base_renderer.py", line 204, in render_document
    return self.render_inner(token)
  File "/Users/antony/.local/pipx/venvs/logseq-doctor/lib/python3.10/site-packages/mistletoe/base_renderer.py", line 109, in render_inner
    return ''.join(map(self.render, token.children))
  File "/Users/antony/.local/pipx/venvs/logseq-doctor/lib/python3.10/site-packages/mistletoe/base_renderer.py", line 94, in render
    return self.render_map[token.__class__.__name__](token)
  File "/Users/antony/.local/pipx/venvs/logseq-doctor/lib/python3.10/site-packages/mistletoe/base_renderer.py", line 201, in render_thematic_break
    return self.render_inner(token)
  File "/Users/antony/.local/pipx/venvs/logseq-doctor/lib/python3.10/site-packages/mistletoe/base_renderer.py", line 109, in render_inner
    return ''.join(map(self.render, token.children))
AttributeError: 'ThematicBreak' object has no attribute 'children'

Probably a duplicate of https://github.com/andreoliwa/logseq-doctor/issues/55.
Can you check if it's the same case?
Or paste some example of Markdown that causes the crash.

I just released a new version v0.2.1.
I'm closing this bug, please reopen it if the problem persists.


Added support for small objc methods

As described in #2719, this should allow the parsing of objc small methods, where fields in method_t are offsets instead of pointers.

Oops used the wrong target for the pull request. I will create another one pulling master onto master.


Curvature

Hi,  Is it possible to create a curvature in the canvas between two nodes or it's allowed only through JSON points connections Key?
Thank you!

Never mind.Double clicking the connection creates pointer.


Operator Api | Add ride inquiry endpoint

This PR will add the ride inquiry endpoint for the operator api.

Same here - i'm with robin, no commented out code.. otherwise fine!


Sign in - Send new code triggers an error

Description
A clear and concise description of what the problem is.
Steps to reproduce

Go to 'Security code verification page'
Click on 'Send new code'
See error "We've encountered an error with your request. Please try again later."
Click on 'Sign in'
No action occurs

Details

Browser: Chrome
Operating system: Mac
Environment: AWS app router
Language: English
Viewport: Desktop

Expected behaviour
Clicking send new code, should trigger a process to send the new code. Clicking on the Sign in link should take the user to the start of the process again.
Screenshots or videos
Visual proof to help explain the problem.

Private Zenhub Video
Additional context
Any other information about the problem here.

Resolved in https://github.com/cds-snc/platform-forms-client/pull/3203


Use R functions in Sparklyr

I am researching how to use R function on line but still have hard time figuring out. Please help.
My initial code looks like:
whatever %>%
group_by(a) %>%
summarize(count=n()) %>%
collect() %>%
ggplot(aes(x=a, y=count)) +
geom_point()
I want to repeat this multiple times since there are other columns I want to check with the same function.
So I wrote:
point_dist <- function(dta, vari) {
dta %>%
group_by(vari) %>%
summarize(count=n()) %>%
collect() %>%
ggplot(aes(x=vari, y=count)) +
gemo_point()
}
point_dist(whatever, a)
but keep telling me:
Error in eval_bare(sym, env) : object 'a' not found
Don't know why.
I either don't know if this is the right direction I shall go.
Thanks again.

You can get help on R and tidyverse questions here: http://community.rstudio.com -- Closing this one since seems not Spark + R related.

@abrahamdu Specifically look for RStudio community forum topics and issues related to "tidy evaluation" for the explanation behind your particular error message.


[Bug] Background will always change to running game despite being disabled

Unchecking 'Set StartPage background to background image of the currently running game' does not disable it.
The background will still change to the running game's background.

fixed in v2.10.2


Resources related to Data science

Add resources related to data science.

I'd like to fork this repo and add a few resources related to data science, if that's alright?

I would like to add a few resources for Data Science in this list.

#25 I added a few data science resources.
@abdullah-K looking forward to seeing what resources you add.


resync 20240507

resync with the tip of the upstream master branch
we can now drop our KNI-specific dep bump, they are done by u/s

depends on https://github.com/openshift-kni/numaresources-operator/pull/933

/hold

/cc @shajmakh

/hold cancel
dependency merged


x/website:

#
What version of Go are you using (go version)?

$ go version

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?

What did you expect to see?
What did you see instead?

Empty issue


Build(deps-dev): Bump @typescript-eslint/parser from 5.9.1 to 5.10.0 (#2415)

Pull request opened by the merge tool on behalf of #2415

Merged in 13cdcda6c945a6b3d2b2688424dad284bdb6e23a


⚠ AsianCinema has degraded performance

In 55924fe, AsianCinema ($ACM) experienced degraded performance:

HTTP code: 200
Response time: 3453 ms

Resolved: AsianCinema performance has improved in 01145af after 10 minutes.


ReferenceError: JSONEditor is not defined

I just include only single json editor file in my app manually. In chrome it's working fine, but in Firefox it's thrown an error. And if we add firebug extension and open the developer tool and reload the page, then it's working fine and if we use the default developer tool then it's thrown an error. Please find the attachment below and help me out.

I have the same error in Chrome Version 64.0.3282.119

python manage.py collectstatic seems to fix it.


Auto download of ACSM file

Awesome tool!
A great addition would be the automatic download of the *.acsm file of borrowed media into a given folder to open it with external readers.

Thanks and yes i have the same idea on my todo list, but this needs a lot new development. My ideal solution would be something like this:

Get a list of all borrowed media
Compare with already known (and downloaded) media
Integrate libgourou and give it a list of new acsm urls to download
Download actual media (not acsm files) to disk into the user specified folder

Maybe someone wants to contribute this. That would be a really nice enhancement


Add requirements.txt

Good suggestion!

@harshanas is this issue set just to add a text file named "requirements" in the repository? I'll do it.

Can you pass the requirements? I can do it too.

@thiagopaiva99 It'd be the package in app.py.

@aewens done!

To clarify, this issue is referring to the requirements.txt file that pip uses for installing packages not in python's standard library. context

In light of  changes made in #1, there's another package to include for anyone working on this.

Since I added a dependency in PR#8, I figured I might as well document that dependency via requirements.txt so I opened PR#9.


Moving scripts folder inside package folder

Verified that the scripts exposed through entry_points work as intended in a new/fresh virtual environment. Moved every script that relied on the runner into the package folder. I'm not sure we need them all.
Just noticed that the ini config files are not installed. They probably have to be handled with package_data if we need them.
Left sphinx docs scripts and other items in the root scripts folder.

Sounds good/run_in_human_input_mode works!
Dunno if we want to split the scripts up later, but I don't think its a big deal right now. I guess the only other thing is if we directly reference the scripts/ folder in any docs, we want to update those paths.

Most importantly, does cache_addressables work?

Re cache_addressables, seems like it (I deleted what was in my Application.persistentDataPath and ran that script).

Thanks for the prompt feedback. I'll merge as soon as GH Actions finishes with this last push.


pull request for e2e test at 11/27/2016 01:26:05

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


deno serve --parallel

Is it possible to use the deno serve with fresh? When I try to run deno serve --parallel dev.ts it spun 8 different ports. This is not ideal behavior. deno serve automatically handles the multi worker listening on single port. Is there any way to leverage that for fresh?

The deno serve command is currently not supported with Fresh, but the plan is to move over to that. Currently, Fresh spawns its own server via the Deno.serve JS API.

The deno serve command is currently not supported with Fresh, but the plan is to move over to that. Currently, Fresh spawns its own server via the Deno.serve JS API.

A workaround (in fresh 2) is to use the handler right?

yup, that can be used with deno serve

here's a simple serve.ts I created to make this work

It's clear that some of fresh config, like https://fresh.deno.dev/docs/concepts/server-configuration#-server, can't be applied here
@marvinhagemeister can I do this? Any suggestions?


Some errors not fit into one line

ruff 0.0.255
When testing files, I see a lot of oneline error info
error: Failed to parse Desktop/33/SetPortInfoFF5.py: invalid syntax. Got unexpected token '__init__' at line 13 column 10
error: Failed to parse Desktop/33/PpmImagePlugin4.py: Got unexpected token 󤄳 at line 133 column 35
error: Failed to parse Desktop/33/DTDValidateError4.py: Got unexpected token 𓸾 at line 4 column 1
error: Failed to parse Desktop/33/_test_fortran3.py: unexpected EOF while parsing at line 38 column 1
error: Failed to parse Desktop/33/netr_DELTA_ID_UNION8.py: Got unexpected token 򍾖 at line 14 column 31
error: Failed to parse Desktop/33/winapi7.py: Got unexpected token 􋉐 at line 33 column 1
error: Failed to parse Desktop/33/rtf3.py: invalid syntax. Got unexpected token Newline at line 90 column 45
error: Failed to parse Desktop/33/NETLOGON_SAM_LOGON_RESPONSE6.py: Got unexpected token 񰅣 at line 57 column 65
error: Failed to parse Desktop/33/decoders7.py: Got unexpected nesting at line 2 column 1
error: Failed to parse Desktop/33/comparisons5.py: Got unexpected token 󯻁 at line 1 column 13
error: Failed to parse Desktop/33/abag1.py: unexpected EOF while parsing at line 17 column 1
error: Failed to parse Desktop/33/ed255194.py: invalid syntax. Got unexpected token '|' at line 52 column 2
error: Failed to parse Desktop/33/Content3.py: unexpected EOF while parsing at line 21 column 0
error: Failed to parse Desktop/33/vesti8.py: Got unexpected token 򻲈 at line 3 column 29
error: Failed to parse Desktop/33/libglib-2.0.so.0.7200.3-gdb0.py: Got unexpected token 󵟷 at line 1 column 1

but some files produce multiline errors, which is inconsistent and looks bad:
error: Failed to parse fix_except0.py: invalid syntax. Got unexpected token """
    try_stmt< 'try' ':' (simple_stmt | suite)
                  cleanup=(except_clause ':' (simple_stmt | suite))+
                  tail=(['except' ':' (simple_stmt | suite)]
                        ['else' ':' (simple_stmt | suite)]
                        ['finally' ':' (simple_stmt | suite)]) >
    """ at line 39 column 14
fix_except0.py:39:15: E999 SyntaxError: invalid syntax. Got unexpected token """
    try_stmt< 'try' ':' (simple_stmt | suite)
                  cleanup=(except_clause ':' (simple_stmt | suite))+
                  tail=(['except' ':' (simple_stmt | suite)]
                        ['else' ':' (simple_stmt | suite)]
                        ['finally' ':' (simple_stmt | suite)]) >
    """
Found 1 error.

Example file - fix_except0.py.zip

Not much we can do here as we don't control those errors (they come from LALRPOP, from RustPython). We could omit them entirely, but that seems even worse to me.

Maybe omitting them (just showing SyntaxError) is correct?

We could potentially truncate after the first line.
One thing that's on my mind is to re-work Diagnostics in general and, as part of that, I would have pushed towards printing diagnostics with more context by default (multiline diagnostics). Ideally, the system would support a "verbose" mode that prints extra context information that may be too verbose by default.
Why I'm bringing this up. Has it been an intentional decision only to print one-line diagnostics when running the check command and if so, what was its motivation?

@MichaReiser - It's mostly born out of a "consistent-with-Flake8-by-default" attitude. We do support printing Clippy-like context via the --show-source flag (which can also be set in pyproject.toml):

But, clearly, it's not the default. That's not based on a principled stance other than that Flake8 also supports a --show-source flag and also doesn't default to it. We could definitely consider making that the default.

We could definitely consider making that the default.

Please do :pray:

We could definitely consider making that the default.

Please do :pray:

Hy @jfmengels nice to see you here. I'm reading many of your blog posts to get inspiration for ruff's configuration and behaviour. Thanks for taking to write down your knowledge.

I'm reading many of your blog posts to get inspiration for ruff's configuration and behaviour. Thanks for taking to write down your knowledge.

Happy to help! Static analysis is a field that I care about a lot, and I want better tools for everyone. And it's not like there is any competition between linters, since we tend to target different languages 😄
I'm also trying to learn things from Ruff (performance mostly), though since the implementation of elm-review is done in a very different language, I don't know how much I will be able to put to use in practice.
But let's not side-track the initial topic anymore 😅, always happy to discuss static analysis in other places.

Is this came from ParseErrorType::UnrecognizedToken? The multiline output is totally unexpected. It must be a bug of RustPython.

but some files produce multiline errors, which is inconsistent and looks bad

but that specific file just happens to have a multi-line unexpected token (a triple quoted string literal in this case):
    PATTERN  """
    try_stmt< 'try' ':' (simple_stmt | suite)
                  cleanup=(except_clause ':' (simple_stmt | suite))+
                  tail=(['except' ':' (simple_stmt | suite)]
                        ['else' ':' (simple_stmt | suite)]
                        ['finally' ':' (simple_stmt | suite)]) >
    """

hence why the output is multi-line.

Ah, that was confusing due to contents of the string. Still it looks like the error message need to be escaped (and prorably shortened too?) when being formatted, is it?

Ah, that was confusing due to contents of the string

Exactly! That's why I wanted to run locally and check and realized its just part of the source code.
Not sure if we shorten/trim here, we can choose either really. I think it makes more sense to show the token that is invalid in its entirety considering that large-ish tokens like this one are not really the norm.

Maybe omitting them (just showing SyntaxError) is correct?

This seems reasonable to me. #3931 adds a new DisplayParseError that implements our custom rendering for SyntaxErrors. We can customize it to change the rendering of UnrecognizedTokento only show the message, or render the first line only (truncate with...`). I can take a stab at it

@DimitrisJim That's reasonable. Probably escaping will be enough.


iOS 17: Migrating to the new Calendar access levels

Hi,
Thank you for having a great repository.
I am working on the new Xcode beta 15 with iOS 17 and I am experiencing some issues with calendar access.
This is due to Apple introducing new levels of access to calendars.
Taken from one of Apples tech notes (https://developer.apple.com/documentation/technotes/tn3152-migrating-to-the-latest-calendar-access-levels):
Xcode 15 includes SDKs for iOS 17, macOS 14, and watchOS 10 that introduce notable changes in EventKit such as new access levels: the ability to add events using EventKitUI without requesting access to Calendar, write-only access, and full access. Apps that don’t request access to calendar data use EKEventEditViewController to present UI for editing an event, which the person using your app can save. See Accessing the Event Store for details. If the user grants your app write-only access, your app can create and write calendar events. However, your app can’t read or delete any events from the user’s calendars, including the ones it creates. If the user grants your app full access, your app can read, write, create, and delete calendar events as in iOS 16 and earlier, for example. To learn how to update your app to use these new features, see TN3152: Migrating to the latest Calendar access levels.
The current PermissionKit cannot access the calendar and cannot view events from the calendar.
The button has the "active"/"pressed" styling and on tap nothing will happen.

Hey! Actual version hasn't UI, so it's related to old versions


should toggle-visibility have contain:paint or at least its z-ordering effects (stacking context) all of the time?

Right now toggle-visibility not being normal acts like content-visibility: auto is present only when the toggle is in the inactive state.  This implies contain: paint, which has z-ordering effects, which means that an element with toggle-visibility != normal has a higher default z-order when its contents are hidden than when its contents are visible.
This seems like it might be undesirable.
Perhaps it would make sense to imply some of the containment, or at least the z-ordering effects of contain:paint (i.e., creating a stacking context), whenever toggle-visibility is not normal rather than only when the toggle is inactive.

One possibility is that toggle-visibility != normal should work like content-visibility: auto, except connected to the toggle instead of connected to being on-screen.  (Right now the spec says that it's only like content-visibility: auto when the toggle is in an inactive state.)  This definition does seem to be easy to implement in an implementation that already implements content-visibility.
That said, it seems less necessary, and perhaps undesirable, to have all the containment effects when the toggle is active.  However, I think the z-ordering effects are needed, and there may be other aspects of containment that I'm not thinking of that either are useful for the visible state or useful to switch states efficiently.

That said, contain affects a lot of CSS properties, and having whether contain is present depend on toggle state means a ton of CSS property computed values would suddenly depend on toggle state, which is probably undesirable.  So maybe it should just work like content-visibility: auto with all the containment effects.

Yeah, I think that's just badly written on my part. The intention is definitely what you suggested, where it acts like content-visiblity: auto but tied to toggle state instead.

(Like, I'm still not entirely certain I shouldn't just be using content-visibility with a new value.)

"like a new value of content-visibility" was sort of how I was thinking about the implementation.

(An interesting question for the case of keeping it a distinct property is what to do when both content-visibility and toggle-visibility have non-initial values.  I'm inclined to follow content-visibility since most of its effects are stronger.)

Yeah, I think the behavior we want is that toggle-visibility can apply hidden-ness in addition to things like content-visibility.


RNReactNativeHapticFeedback is not available

Hey! I want to use haptic vibration once button tap but I have faced weird issue.
When click button I am getting warning and haptic vibration is not working.
"react-native": "0.70.1",
"react-native-haptic-feedback": "^1.14.0",
Usage:
const onHeptic = () => ReactNativeHapticFeedback.trigger('impactMedium', options);
Is this package has any problem?

Not known at least. Did you run pod install after adding the package?

Same issue here (after pod install)
"react-native": "0.70.1"
"react-native-haptic-feedback": "2.0.0"

Same problem on the old architecture

Ok, could reproduce. Will fix the issue and involve an example project to avoid further mistakes

Thanks, @mkuczera, iOS is working, however Android emulator still throws this error.

Got this build error on iOS:
error Failed to build the app: No package name found. We couldn't parse the namespace from neither your build.gradle[.kts] file at */node_modules/react-native-haptic-feedback/android/build.gradle nor your package in the AndroidManifest at null.

@mkuczera Everything works now. Thanks and have a productive day!

Having the same issue again - "RNReactNativeHapticFeedback is not available"

I got this issue when I forgot to
cd ios; pod install

still getting RNReactNativeHapticFeedback is not available when using 2.0.2, 2.0.1, 2.0.3

@ayoubechc where you able to fix it ?

@ayoubechc where you able to fix it ?

I use  version 1.13.0 with react native version :0.66.0, and it work fine

Same issue here, using the latest version.
"react-native-haptic-feedback": "^2.0.3",

The same issue on 2.0.3(Latest version).

I noticed that the issue you said has been fixed in 2.1.0 on conversion: pull/108.  So when will it be published on 2.1.0?  I really appreciate it if you can publish it soon.

me too, is ther any solution

I'm still getting an error RNReactNativeHapticFeedback is not available

I'm still getting an error RNReactNativeHapticFeedback is not available 2.2.0

are you rebuild completely the app after install the package ? I rebuild with xcode and the error disapear ;)

I'm still getting an error RNReactNativeHapticFeedback is not available

Hello! Im getting a 'WARN' when I try to run:
     if (Platform.OS === 'ios') {       trigger('impactMedium', optionsButtonFeedback);     }
It says:
WARN  RNReactNativeHapticFeedback is not available
Is there a fix for this? Im using version 2.2.0

In the "react-native-haptic-feedback" version: "^2.3.1", this problem is present


Zen30 firmware 4.10

Firmware 4.10 added some parameters that aren't defined in the driver currently.
Is there anyway to add those parameters?
VER. 800LR (released 11/2023)
Hardware:
800 Series Z-Wave chip

Firmware: 4.10
Long Range Ready
Separated ramp rate settings into 4 new parameters: ramp rate ON physical (13), ramp rate OFF physical (31), ramp rate ON Z-Wave (32), and ramp rate OFF Z-Wave (33). [See all settings here.](https://www.support.getzooz.com/kb/article/468-zen30-double-switch-advanced-settings/)
Adjust Double Tap (parameter 17) and Single Tap (parameter 18) behavior to match ZEN72 and ZEN77 dimmers.

I have a fix posted on a separate link temporarily, as a Beta version while I work on overhauling all 3 switch drivers.
https://community.hubitat.com/t/driver-zooz-zen-switches-advanced-and-dimmers/58649/448?u=jtp10181

I am going to keep this as an open issue until I post it as a final version, in case anyone else finds this in here it will give them the info they need.

This is pushed to main branch now: https://github.com/jtp10181/Hubitat/commit/6e9c599fa944df355b3f99d3653e99921c2d2a4f


Publish 04/26/2022 3:30 PM PT

Docs Build status updates of commit 5fd230f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Export-ExchangeCertificate.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Import-ExchangeCertificate.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Implement Pagination

[x] I checked to make sure that this issue has not already been filed.

[x] I'm reporting the issue to the correct repository (for multi-repository projects) (optional)

Expected Behavior:
Users should be able to move from one page to another

Current Behavior:
All the results are shown. There is no pagination.
Solution:
The GET API can take 2 query params - page and limit. Make use of them while calling APIs and retrieving results. Default value is page=1 and limit = 10.
API Documentation

@Manvityagi I would like to work on this Plz assign

@Manvityagi Is this issue still open? If yes then I would like to work on this!

@Vishal1003 It has been assigned to you.
I think @ritik307 is not available as of now.

@Manvityagi if this issue becomes available in future, I would like to work on it.

@Vishal1003 Please respond if you're working on this issue, otherwise I'll have to unassign you from this issue.

@vaishali614 I'm really sorry. I won't be able to work on this issue as of now.

@Manvityagi if this issue becomes available in future, I would like to work on it.

Awesome @akshitakanojia. I have assigned to you, Feel free to comment here if you face any difficulties in implementation.

@Manvityagi is it possible to get total number of pages as well in the api response, as it will be required to show the page numbers. Otherwise I can show only prev current next.

@Manvityagi is it possible to get total number of pages as well in the api response, as it will be required to show the page numbers. Otherwise I can show only prev current next.

Hey, yeah. We can do that.
Would you mind creating that issue in backend?

okay I'll create.


ambient color is not applied

My model is not displayed in the viewer below.
https://gltf-viewer.donmccurdy.com/
Because ambient color is not applied?
How can I solve it?

Currently we only convert the diffuse and transparency channels from non-PBR materials.
PBR materials even don't have the concept of ambient, so I cannot convert it.
I


message_edit: Show modal if user cannot resolve topics.

We show a modal if user is not allowed to resolve or unresolve topics due to time limit.
The modal just contains the text mentioning user cannot resolve topic without stating the count
of messages that are withing the time limit as we do not recommend partial resolving of topics.

Screenshots and screen captures:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Unresolving the topic from warning banner in compose box -

The implementation lgtm aside from polishing the modal heading/content, which I'll leave for @alya to review.

Updated wording discussed on CZO:

You do not have permission to resolve topics with messages older than X in this organization.

(where X is relative times, based on what the org setting is ("2 hours", "1 week", etc.))

Updated the PR -

Changed the text in the modal as discussed.
We now do not show separate modal when resolving or unresolving topic using "Move topic"/"Move message" modal and consider it as a simple message move.

Merged, thanks @sahil839!


Add support for reading/writing gzipped messages

Re comment on #552, I thought we decided not to gzip for simplicity? Did I misinterpret? I suppose supporting doesn't hurt either way.

Yes, that's what we decided before I started looking at the Git LFS bandwidth requirements---we get "charged" for every download/clone of the repo, so reducing the size on Git LFS is actually pretty important. Hope that's ok to change?

Gotcha, SGTM