Multiple AZ-agnostic persistent volumes

Just ran into an interesting problem while testing #393. If persistence is enabled, the chart provisions two volumes - one for wordpress data, and another for apache data. But if the specified (or default) StorageClass doesn't specify a GKE/AWS zone in which to create volumes, one is selected at random (ref). This could (and in my case, did) result in the two volumes being provisioned in different zones, preventing the cluster from scheduling the pod.
AFAIK there's nothing we can technically do at the chart level to avoid this scenario, other than mandating one pod-one volume. But it's probably something we want to mention somewhere, as it complicates our goal of easy out-of-the-box persistence.

this one should probably be reported to kubernetes, as it is something which should be addressed there

@redbaron I don't think of it as a bug, though. Zone can be part of the StorageClass definition, but it's not (and, IMHO, shouldn't be) required.

it can be,but it doesn't help if you want to run multi-AZ StatefulSet

I ran into this as well and created a PR for it, as I had not searched deep enough for issues to come across this one. In this case both volumes are mounted under the same parent folder, so it's an easy fix to just combine it into one volume.
Issue is discussed in the context of petsets here in kubernetes core: https://github.com/kubernetes/kubernetes/issues/35695

@mgoodness given that we have addressed this in provisioner can we close this issue. Also, FWIW we have opened a larger issue to deal with scheduling and provisioning zonal resources. https://github.com/kubernetes/kubernetes/issues/41598

@kow3ns as I remarked in the issue you linked, the talk is all about StatefulSets but this issue can just as wel hit Deployment such as in the Wordpress chart.


epiphany: 42.3 -> 42.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for epiphany is: "WebKit based web browser for GNOME"
meta.homepage for epiphany is: "https://wiki.gnome.org/Apps/Epiphany"
meta.changelog for epiphany is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 42.4 with grep in /nix/store/044sg59r055p8xxz7zwc5znpvir1jjzl-epiphany-42.4
found 42.4 in filename of file in /nix/store/044sg59r055p8xxz7zwc5znpvir1jjzl-epiphany-42.4

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath
epiphany
gnome.epiphany
pantheon.elementary-gsettings-schemas
pantheon.epiphany

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/044sg59r055p8xxz7zwc5znpvir1jjzl-epiphany-42.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A epiphany https://github.com/r-ryantm/nixpkgs/archive/19fd6dde3b0981cc547413eb0527b6cc48126c9c.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/044sg59r055p8xxz7zwc5znpvir1jjzl-epiphany-42.4
ls -la /nix/store/044sg59r055p8xxz7zwc5znpvir1jjzl-epiphany-42.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

epiphany
pantheon.elementary-gsettings-schemas
pantheon.epiphany

Maintainer pings
cc @bobby285271 @hedning @jtojnar @dasj19 @maxeaubrey @davidak @bobby285271 for testing.

Closing in favour of #182618.


[BUG] Freezing a plotly plot stores two copies of the image in the notebook output

When running the two cells below in a Jupyter notebook:
import fiftyone as fo
import fiftyone.zoo as foz
from fiftyone import ViewField as F

dataset = foz.load_zoo_dataset("quickstart-geo")

locations = dataset.values("location.point.coordinates")
num_objects = dataset.values(F("ground_truth.detections").length())

plot = fo.location_scatterplot(locations=locations, sizes=num_objects)
plot.show()

plot.freeze()

the intention is that plot.freeze() replaces the interactive plot with a screenshot, so that when the notebook is closed and reopened, the static image version of the plot will be there.
Currently, plot.freeze() appears to work in Jupyter notebooks (jupyter notebook), but, upon reopening the notebook, one sees that two copies of the image were actually stored:

When running the same code in JupyterLab (jupyter lab), the duplicate images immediately appear in the cell output upon calling plot.freeze().
The relevant code that will need to be tweaked in some way is here:
https://github.com/voxel51/fiftyone/blob/9ee5a680519f274c52a52816a7bd23c775a992f2/fiftyone/core/plots/plotly.py#L1842-L1885
As a side note, it would be preferable if FigureWidget(...) also did not appear in the cell output when the notebook is reopened. That is likely there because self._widget.close() makes the plot invisible in the active notebook, but doesn't actually delete it from the cell output.

I also see this in interactive plot workflows (using jupyter lab) with something like
plot.show()
session.plots.attach(plot,...)
session.show()

Maybe expected given the original bug.


⚠ Bofrost.nl has degraded performance

In c504674, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 963 ms

Resolved: Bofrost.nl performance has improved in 63f61f6 after 6 minutes.


updated Example in Class:LoraModel

The previous example is no longer functional, but I have made slight modifications to ensure its compatibility with the latest version, making it executable once again.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Hi @TianyiPeng Thanks for the PR, looking at other model cards docstrings I think this is intended:
https://github.com/huggingface/peft/blob/c46d76ae3a323fa208bd777b9eb60a383e094d49/src/peft/tuners/p_tuning.py#L91
However there is indeed a typo in the current docstring of LoraModel, the argument adapter_name is missing.
https://github.com/huggingface/peft/blob/c46d76ae3a323fa208bd777b9eb60a383e094d49/src/peft/tuners/lora.py#L174
Can you add that in the docstring instead? Thanks!

I have a question. Seems that not only the adapter_name is missing, we also need that config[adapter_name] to be the real config (see below). Should we still just change the docstring or maybe we should change the code to add a case for if adapter_name is None? Any suggestion?
https://github.com/huggingface/peft/blob/5a0e19dda1048ff8caaa12970ba7574f9cdfbf76/src/peft/tuners/lora.py#L175-L181
I find the same issue for docstrings in other adapters such as:
https://github.com/huggingface/peft/blob/5a0e19dda1048ff8caaa12970ba7574f9cdfbf76/src/peft/tuners/adalora.py#L78-L84

Thanks for iterating, I think you are right, I think we should first update the docstring to reflect the current situation then we can think of a potential fix; would you be happy to address these changes on docstrings?

Hi, I updated the docstring accordingly. Happy to help more if needed.

Reference

Done!


Feature - Equip items from bags with Right Click

Right Click items to equip them

Must be in your bags.
Will equip to an empty slot when available.
Equip Priority: Primary, Secondary, Range, Chest, Legs, Head, Arms, Hands, Feet, Shoulders, Back, Neck, Face, Waist, WristLeft, WristRight, EarLeft, EarRight, RingLeft, RingRight, Ammo
Can hold Shift (2nd) / Ctrl (3rd) / Shift+Ctrl (4th) to equip the item to alternate slots if it can be equipped in several slots in the list.

Demo
Options

Converted the manual ini usage to ZealSetting, thanks for suggestion that, since I wasn't aware of it.
Also did another round of attempting to break/desync it, both on a local sever (~30ms latency) and then on Quarm (~100ms latency), and I could not break it. One of either scenarios happened, but I never got kicked or desynced:
Scenario 1

Say "read" to PoD (summons Tome of Order and Discord to the cursor)
Simultaneously press RMB to swap immediately after.
Items are swapped, and Book is left on the cursor.

Scenario 2

Say "read" to PoD (summons Tome of Order and Discord to the cursor)
Simultaneously press RMB to swap immediately after.
Book appears on cursor, and nothing else happens.


watt-toolkit: init at 2.8.4

Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg eval

@wegank

@wegank

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/639

My machine is down


error when deal big file

when I use this package to md5 a  buffer  of a  400+MB  file,   the error comes and says:
FATAL ERROR: invalid table size Allocation failed - process out of memory

md5 is not the right library for hashing files of that size. Have a look at Node's Crypto API, where you can create hashes of streams.

I'm having the same problem with smaller files, approx. 60MB in this case.
I don't think this library is very useful if it can't handle even <100MB files.

Sure. This library is not helpful at all.

what's reason cause this?
I'm really curious

@pvorb


Upgrade to Kotlin 1.3.70 and latest rules_kotlin

I just moved the maven_install call up.  rules_kotlin now depends on rules_jvm_external, and you get a cycle error complaint if it is declared after rules_kotlin. Ditto for rules_proto.

/gcbrun

Looking at the history of Kotlin versioning internally it appears to be ~2 weeks after release, so we're about 6 days away from internal updating to 1.3.70 if history holds.

I can leave this PR for testing and we can land when they switch. I'm interested in the affects it has on GCB performance. Also, I need to test re-enabling JS builds.

Weird that tools/local-presubmit works, but on GCB it fails
ERROR: /usr/src/app/java/arcs/sdk/android/storage/service/testutil/BUILD:7:1: //java/arcs/sdk/android/storage/service/testutil:testutil_kt depends on @com_github_jetbrains_kotlin//:kotlin-stdlib in repository @com_github_jetbrains_kotlin which failed to fetch. no such package '@com_github_jetbrains_kotlin//': java.io.IOException: Error downloading [https://github.com/JetBrains/kotlin/releases/download/v1.3.70/kotlin-compiler-1.3.70.zip] to /root/.cache/bazel/_bazel_root/a14564dce24fc232216f1aef117728d1/external/com_github_jetbrains_kotlin/kotlin-compiler-1.3.70.zip: Checksum was 709d782ff707a633278bac4c63bab3026b768e717f8aaf62de1036c994bc89c7 but wanted 12f97cff23ff8116904cb97a7ef4e3af5c3b8e5df9d9e63baa251d9a73b42fbb
Step #3 - "local-presubmit": ERROR: Analysis of target '//java/arcs/sdk/android/storage/service/testutil:testutil' failed; build aborted: no such package '@com_github_jetbrains_kotlin//': java.io.IOException: Error downloading [https://github.com/JetBrains/kotlin/releases/download/v1.3.70/kotlin-compiler-1.3.70.zip] to /root/.cache/bazel/_bazel_root/a14564dce24fc232216f1aef117728d1/external/com_github_jetbrains_kotlin/kotlin-compiler-1.3.70.zip: Checksum was 709d782ff707a633278bac4c63bab3026b768e717f8aaf62de1036c994bc89c7 but wanted 12f97cff23ff8116904cb97a7ef4e3af5c3b8e5df9d9e63baa251d9a73b42fbb
Step #3 - "local-presubmit":

It seems GCB can't download from this GitHub url. I'm gonna try 1.3.60 to see if it is related to the new rules_kotlin implementation using a different downloader.

709d782ff707a633278bac4c63bab3026b768e717f8aaf62de1036c994bc89c7

oops, never mind, it' the SHA

I'm super excited about 1.3.70, but internal hasn't switched to 1.3.70 yet. If we upgrade in OSS before that it could easily lead to nasty build breaks.
See cs query f:third_party/kotlin/kotlin/BUILD for the current authoritative google3 version.
Talk internally sounds like they will upgrade fairly soon.

1.3.70 is now live on G3 as the default.  Let's goooooooo. :)


userGestureInProgress always be true when using CupertinoPageRoute.

Description
navigator.userGestureInProgress always be the true when using CupertinoPageRoute.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] My PR includes tests for all changed/updated/fixed behaviors (See [Test Coverage]).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read [Handling breaking changes]).
[x] No, this is not a breaking change.

Can you describe the issue this is fixing a bit? It would also need tests (this makes sure we don't accidentally break this behavior in the future)


Forgot Email id, not able to recover git hub account by username

Hi, my username is anu3rag and i forgot the email id associated with it, could you please help me to recover my account? Thanks

Looks like is still valid, but not sure if we can provide that information with just a username.


Require a Stream that could parse Json from a std::io::Read

Stream initiates with a &mut std::io::Read.
Stream::next will return one Json if succeed.
Differ from json::Builder:

json::Builder will check the tailing characters, and Stream won't
json::Builder accepts an Iterator<Item=char> while Stream accepts a std::io::Read.

May need to refactor the json::Parser for working with std::io::Chars.

In Serde you would use a StreamDeserializer which you can construct from an iterator of bytes or an io::Read and it gives you an iterator of deserialized values of type T of your choice.

I'm going to close this now that this crate is deprecated in favor of serde. We're discontinuing feature development in rustc-serialize but will still continue to merge bug fixes if they arise.


fix(carousel-focus): carousel scroll behaviour when interacted with cta

Stop carousel when any button inside carousel item has been clicked and fix broken view on tab
switch
Description
If a CTA on one of the advertising tiles has focus, and the animation / scrolling is on, it breaks.
Tabbing between the controls on the ads DCI-17572
Fixes # (issue)
#622
Requirements
Please briefly outline any requirements for the work which may aid the testing and review process.
Storybook link: (once netlify has deployed link provide a link to the component)
Design link: DCI-17572
Checklist:

[x] The commit messages follow the convention for this project
[x] I have provided an adequate amount of test coverage
[x] I have added the functionality to the test app
[x] I have provided a story in storybook to document the changes
[x] I have provided documentation in the notes section of the story
[x] I have added any new public feature modules to public-api.ts

@Nupur04 would you mind updating the commit message context from carousel-focus to carousel please?


Wrong location showing?

Hi guys,
I've got a problem implementing the maps into my app. I've installed react-native-maps and was able to get a working map on the screen, but it's showing the wrong location...
This is my code for the mapview (it's nested inside of a scrollview, perhaps that helps):
                   <MapView                         style={styles.map}                         scrollEnabled={true}                         zoomEnabled={true}                         pitchEnabled={false}                         rotateEnabled={false}                         initialRegion={{                             latitude: this.props.data.latitude,                             longitude: this.props.data.longitude,                             latitudeDelta: 2,                             longitudeDelta: 2,                         }}                     >                     <MapView.Marker                       title={this.props.data.location}                       coordinate={{                           latitude: this.props.data.latitude,                           longitude: this.props.data.longitude,                           latitudeDelta: 2,                           longitudeDelta: 2,                       }}                     />                     </MapView>
The coordinates in "this.props.data" are for latitude: 33.8847361 and longitude: -118.4109089. This should be somewhere in California according to Google Maps, yet my map is showing a location somewhere in Asia. Does anyone have an idea on what's going wrong?

I'm seeing the same problem. Using coordinates in Delaware, and it's showing someplace in China.

Let us know if the issue still persists

I'm facing the same issue, the only difference is that it sometimes loads the correct location. The problem is on iOS, I haven't had the sam on android.


[Feature Request] Add KV list method

KV namespaces now support the list() method which returns a paginated list of keys: https://developers.cloudflare.com/workers/reference/storage/listing-keys/.
This method is missing from the KeyValueStore class.

I've personally being using this shim:
// namespace.test.ts

/* eslint-disable @typescript-eslint/camelcase */
import Cloudworker from '@dollarshaveclub/cloudworker'

const LIST_MAX_LIMIT = 1000

const namespace = new (Cloudworker as any).KeyValueStore()
namespace.list = async function(
  this,
  {
    limit = 1000,
    prefix = ``,
    cursor = `-1`,
  }: {
    prefix?: string
    limit?: number
    cursor?: string
  }
): Promise<{
  keys: {
    name: string
    expiration?: number
  }[]
  list_complete: boolean
  cursor: string
}> {
  if (limit > LIST_MAX_LIMIT) limit = LIST_MAX_LIMIT
  if (limit < 0) limit = 0

  const { store } = this

  let keys = Array.from(store.keys() as string[]).filter((key: string) =>
    key.startsWith(prefix)
  )

  keys = keys.slice(+cursor + 1)

  const list_complete = keys.length <= limit

  keys = keys.slice(0, limit)

  const nextCursor = (
    +cursor + (list_complete ? keys.length : limit)
  ).toString()

  return Promise.resolve({
    keys: keys.map(key => ({
      name: key,
    })),
    list_complete,
    cursor: nextCursor,
  })
}

export { namespace }

If there is interest, I'm more than happy to write up a PR. The only concern I have is with the expiration part of the response. This hasn't been implemented in KeyValueStore yet, so I've just been dropping it from the response.

This looks solid!

I also don't know how Cloudflare are generating the cursors they use (e.g. 6Ck1la0VxJ0djhidm1MdX2FyD), so I've been using the index of the (ordered) map, which seems to work for testing.

The main thing to verify here would be the behavior of the cursors when there's lots of insertions in between list operations. I slightly suspect that Cloudflare's cursors use the last key of the previous "page" as a starting point for the next page rather than its global order, but I haven't verified this experimentally.


Upgrade Errai snapshot version.

Related:
https://github.com/errai/errai/pull/301
https://github.com/AppFormer/uberfire/pull/906
https://github.com/dashbuilder/dashbuilder/pull/401
https://github.com/kiegroup/droolsjbpm-build-bootstrap/pull/605

@dgutierr Could you please approve this? Just a version upgrade of Errai.


Fix running tests on Windows and exclude corrupted characters from character-to-language mapping for Latvian

On Windows, file.encoding value depends on regional settings of a user and is never set to UTF-8 by default. Therefore, tests that include strings with non-ASCII characters fail on Windows. The solution is to add file.encoding as a parameter for JVM in gradle.properties.
When writing a text file, newline characters depend on OS - Windows uses a carriage return + a line feed (\r\n), however, Linux and Mac OS use just a line feed (\n). Newline characters used in a developer's source code depend on IDE and/or Git settings. Therefore, tests with multiline strings may fail if newline characters are not normalized.
A text corpus or multiple corpora used for building a language model for Latvian were not clean and contained corrupted characters (presumably, due to encoding errors). Weird characters were removed from character-to-language mappings and tests also fixed for words that contain such non-native characters.

Hello Jānis,
thank you very much for this pull request. I develop on macOS, so I wasn't aware of the Windows problems you have mentioned. I have merged your changes and will release Lingua 1.1.0 soon which will include them. :)


The Problem with Metrics is a Fundamental Problem

It is partially connected to https://github.com/chaoss/wg-common/issues/164 and I think that it could also involve WG-risk.
This paper is a little bit oriented to AI but has still a nice general overview on what kind of mitigation we could adopt when we are going to rely on metrics as targets:
https://arxiv.org/abs/2002.08512

Just to add few other resources other then in the original ticket:
https://www.holistics.io/blog/four-types-goodharts-law/
https://mpra.ub.uni-muenchen.de/90649/1/MPRA_paper_90649.pdf

https://www.ribbonfarm.com/2016/06/09/goodharts-law-and-why-measurement-is-hard/

https://openai.com/blog/measuring-goodharts-law/

How can we work together to bring some of your ideas forward as an actionable metrics model?

I'm not familiar with the way the group works as I'm pretty new.
I think that on one side we should try to involve some experts on this subject such as the author or related works. Is there any major conference on this issue?
In parallel, when we interact with the OSPOs, we should begin to understand how they direct resources, especially in "enterprise" opensource projects, since resources are often allocated to achieve objectives that for companies must be measurable.
What do the OSPOs do? Do they transform our metrics into measurable goals? etc..

/cc @ davidmanheim I think we have mentioned two of your works in this thread https://github.com/chaoss/wg-metrics-models/issues/17#issuecomment-1100697795. Are you still involved in this topic?

My proposal is to evaluate some section in the model template using 4.1/5.1 in https://mpra.ub.uni-muenchen.de/90649/1/MPRA_paper_90649.pdf:

Can we add for every single included metric a Pre-gaming analysts of the risk?
Coherence: what is the coherence of our metric model?

In the model (template) we could add a model card section where we define:

Diversification: how we have diversified our metrics portfolio to limit gaming?

Because the different metrics typically
require different behaviors, and they will be to some extent in tension with one another,
they are likely to make gaming harder.

Randomization. What kind of randomization strategy we suggest?

the weights on compo-
nents or the relative rewards are uncertain, gaming the metric may become less worth-
while.

What kind of soft-metrics we suggest to integrate when an Org is adopting a specific model?

Human judgment, peer evaluation, and other techniques may be
able to reduce gaming specific to metrics. Metrics are often seen as a way to avoid
subjectivity, but a combination of metrics and human judgment may be able to capture
the best of both worlds.

What limit strategy we suggest?

Limiting Metrics. Failures are often the result of too much pressure on the opti-
mization. By using metrics to set a standard or provide a limited incentive instead of a
presenting value to maximize, the overoptimization pressure can sometimes be mitigated

Abandoning Measurement. How we are going to review the model after applied on the field?

Sometimes, the value of better incentivising partic-
ipants and the potential for perverse incentives issues make it worthwhile to be wary
of what Muller refers to as metric fixation.[Mul18] As he suggest, sometimes the best
solution is to do nothing - or at least nothing involving measurement.

/cc @dynamicwebpaige (if interested)

Closing as these issues are more corporately related and less about open source community health.

Maybe create a blog post or a Discourse thread with respect to this issue.

In communities where we have an heavy presence of corporate members, developers and  repo gatekeepers it is going to impact a lot as all our metrics and our models are at risk to be deformed.


Lifecycle hooks

Hi, my question it's about Lifecycle. In this gist https://gist.github.com/matedemorphy/94c6ad13b6aebd89862b5b14120c98df you could see the implementation of a cursor spinner wich should hide in afterReflex(), but sometimes this callback doesn't works.
I have this line in config/initializers/session_store.rb
Rails.application.config.session_store :cookie_store, key: _my_app, domain: lvh.me, but then in development.rb
config.session_store :cache_store
config.cache_store = :file_store, Rails.root.join(tmp, cache).to_s, { size: 320.megabytes } it could some conflict there?

Did adding ids resolve this? Curious based on the conversation in Discord. https://discordapp.com/channels/629472241427415060/629472241427415062/731945448339931146

@hopsoft Yes, solved!


make awesome

:star: PR built and available in a preview environment liuxiaopiao-node-http-pr-1 here


Botbuilder generator env file + gitignore causes cloned repos to fail. Could cause confusion for new users.

This pertains to the botbuilder-generator yeoman template
Confusing behavior
Currently the yeoman template generates a .env file and includes the botFilePath in the env file. However, the .env file is in .gitignore (correctly so!) and that could cause confusion when checked into github.  A second user could clone the repo and run into an error of not being able to run the bot.
To Reproduce
Steps to reproduce the behavior:

Create a new bot using botbuilder-generator.
Push code to github
As a new user (or in a new folder) clone the bot.
Notice bot won't start due to missing botFilePath value.

Expected behavior
The expectation is that everything needed to run the bot should be pushed to github. While not checking in the .env is correct and good practice, it does cause friction because the botFilePath is in there and results in a non functioning bot.
Perhaps including the expected local botFilePath in the Readme and making it clear that when cloning the repo, users need to add a .env file and include the default values for the botFilePath.
[bug]

.bot files have been removed code generators and samples.  you can see this in the samples-work-in-progress branch, which we plan to merge into master and publish to our various registries (NuGet, npmjs.org) within the very near future, about a week.


Use python:slim as the base docker image and install JRE instead of JDK

The current size of the docker image is about 1G, it's a little bit big for production. I have two suggestions:

Change the base docker image from centos to python:3-slim. Slim is smaller than Centos and many systems use it in the production environment.
Use JRE instead of JDK.

I have tested locally, the size of image will be 760M after these two changes.
By the way, we can use docker multi-stage to build images instead of running a command to install packages:
FROM python:3-slim

COPY --from=openjdk:11-jre-slim /usr/local/openjdk-11 /usr/local/openjdk-11

ENV JAVA_HOME /usr/local/openjdk-11
ENV PATH $JAVA_HOME/bin:$PATH

It's easier to use  python:3-slim as the base image and copy JRE from openjdk:11-jre-slim  than vice versa.

What image are you looking at? The size of recent images is ~600 MB according to dockerhub: https://hub.docker.com/r/prestosql/presto/tags
Regarding the use of JRE, Presto needs a full JDK, so that probably won't work. Also, the JRE is being phased out -- there are no public builds of a JRE anymore (that I can find, aside from that docker image), and in the future, Oracle may even stop producing those. See https://stackoverflow.com/questions/53111921/how-to-get-java-11-run-time-environment-working-since-there-is-no-more-jre-11-fo

@martint  Thanks.
the size of images is ~600M after compression and 1.03G before compression.
foodeMacBook-Pro:docker foo$ docker images | grep prestosql | grep 333
prestosql/presto                                   333                 5adc7a4d7326        3 weeks ago         1.03GB

OpenJDK is ~300M, Presto is ~450M, and OS is ~300M. it's reasonable, I will close this issue.

If you want to build your own image, you can remove connectors you don’t use, start with a smaller OS image, etc. We use a standard CentOS image since it’s generic and makes it easier to install debug tools or modify as needed. The provided Docker image is meant to be useable by anyone, especially  for demos or getting started, not optimized for any specific use case.
Don’t use Alpine or similar, as the Hadoop native code for the Hive connector requires glibc based Linux distributions.


bug: wrong colors

Take screenshot from daisyui documentation (Shift+Windows+S with default windows Snipping Tool).. Paste into Excalidraw App (Chrome Extension on Desktop Ctrl+V).. Expected to see same colors.. but looks different..
https://daisyui.com/components/stat/#with-custom-colors-and-button

Yeah this is happening since you are exporting in dark theme in Excalidraw and we invert and hue-rotate the colors and go with whatever it gives us, this works most of the time. Though I I agree it is confusing specially for the above case and also when you want to use your own custom colors. We need to improve this so colour shades are closer to the actual colors.

Thanks for explaining.. that's right.. switching back to light theme renders correct colors.. what's surprising the issue only happens to that specific color.. didn't notice with any others..

it would be cool when inverting colors to also add it for white otherwise you have to play "hide and seek" with them

@anatoliibakhtin do you mean when the text is overlaid over an image, it should not be inverted? Not super easy to do for a general case.

@dwelle Sorry, I wasn't clear. Added to the comment above.
As far as I understand, the main function of night mode is to reduce eye strain, when working, at night, secondary is to be beautiful. For me, it is enough to simply invert the white color of the image so that the black text does not disappear into the white background of the image.
So that the colors of the ink and the colors of the image are consistent (do not disappear as they do now) with each other.

Sorry, I didn't make my comment above clear. Now Fixed*

pls solve these issues, it is quite annoying seen the my colors changed on dark mode

I love how dark mode - dark color implemented here: https://chrome.google.com/webstore/detail/dark-reader/eimadpbcbfnmbkopoojfekhnkhdbieeh

Yellow stays yellow and doesn't turn into something like chocolate :))


docs(cn):  Fixed sidebar translation for learn pages into chinese

请仔细检查翻译用语

Sorry, this is just a test. I'm trying to practice my English and Git skills. Thank you for your understanding and sorry for any inconvenience.

 I'll come back when I have free time from school and read the regulations carefully. Thank you and have a happy life.


Generate remapping weights for 0.25 ocean

type_non-CMIP6   |    by<EMAIL_ADDRESS>
Generate remapping weights files for 0.25 degree ocean to N96 atmosphere grid

Issue migrated from trac:382 at 2024-01-31 18:36:11 +1100

<EMAIL_ADDRESS>changed status from assigned to accepted

<EMAIL_ADDRESS>set owner to aph502

<EMAIL_ADDRESS>commented

Hi Martin,
Scott explained the Arakawa C-grid was the explanation for the offset in the grids.
What remapping weights files are required? All three?
Cheers
Aidan

<EMAIL_ADDRESS>uploaded file lon_diff.png (230.0 KiB)
Difference between longitudes

<EMAIL_ADDRESS>commented

For CM2 we have all three.
rmp_cice_to_um1t_CONSERV_FRACNNEI.nc
rmp_cice_to_um1u_CONSERV_FRACNNEI.nc
rmp_cice_to_um1v_CONSERV_FRACNNEI.nc
rmp_um1t_to_cice_CONSERV_DESTAREA.nc
rmp_um1t_to_cice_CONSERV_FRACNNEI.nc
rmp_um1u_to_cice_BILINEAR.nc
rmp_um1u_to_cice_CONSERV_FRACNNEI.nc
rmp_um1v_to_cice_BILINEAR.nc
rmp_um1v_to_cice_CONSERV_FRACNNEI.nc
I'm not sure if your version of the 0.25 degree grid is the same as the one used for the files in /g/data/access/access-cm2/input_O025/cpl_n216?

<EMAIL_ADDRESS>changed _comment0 which not transferred by tractive

<EMAIL_ADDRESS>commented

Martin,
Scott tells me the horizontal grid for the N216 config is not the same as that used for CM2, so I'll go ahead and regenerate the remapping weights files.
I see there is some duplication with multiple remapping weights files for atmosphere to cice:
rmp_um1t_to_cice_CONSERV_DESTAREA.nc
rmp_um1t_to_cice_CONSERV_FRACNNEI.nc
rmp_um1u_to_cice_BILINEAR.nc
rmp_um1u_to_cice_CONSERV_FRACNNEI.nc
rmp_um1v_to_cice_BILINEAR.nc
rmp_um1v_to_cice_CONSERV_FRACNNEI.nc
Can you let me know what ones are currently used for CM2 so I know what to create, i.e. conservative v bilinear, that sort of thing.
Cheers
Aidan

<EMAIL_ADDRESS>commented

Martin,
Which 1 degree CICE grid should I be using?
I found this one, but it doesn't have the v grid
/g/data/access/access-cm2/input_O1/cice_360x300/cice_grid_20101208.nc
Cheers
Aidan

<EMAIL_ADDRESS>commented

I have just checked if the CICE grid (/g/data/ik11/inputs/access-om2/input_20201022_old_topog/cice_025deg/grid.nc) is the same as the ocean grid (/g/data/ik11/inputs/access-om2/input_20201022_old_topog/mom_025deg/ocean_hgrid.nc).
They are the same within 5e-14.

<EMAIL_ADDRESS>commented

It looks like the grids are the same, with a maximum difference of 1.5e-5 in longitude, and 3.5e-6 in latitude

Is that close enough to use those remapping weights, or would you like me to generate them? It is no big deal to do so.

<EMAIL_ADDRESS>commented

Hi Martin,
Sorry for my confusion, of course I need the 1 degree atmosphere grid. Sorry.
I guess it is this file?
/g/data/access/access-cm2/input_O1/cpl_n96/oasis3_grids_N96_06032014.nc
and the um1u.lat, um1u.lon, um1v.lat, um1v.lat, um1t.lat, um1t.lon variables.
Which of the namcouple files in /g/data/access/access-cm2/input_O1/cpl_n96 is the one currently used in CM2?
Cheers
Aidan

<EMAIL_ADDRESS>commented

A first iteration of the remapping weights is documented in this repository
https://github.com/aidanheerdegen/access_cm2_025_remap_weights
I did not have time to check round-tripping of fields.
As explained here
http://climate-cms.wikis.unsw.edu.au/Interpolating_NetCDF_files_to_different_grids
that is very simple to do with the NCO tool ncks.
It is simple to regenerate these at any time if different options are required. Note for example that the option -i was used (--ignore_unmapped), which ignore unmapped destination points. In this case, the output matrix won't contain entries for the unmapped destination points.I believe this is not the same as the remapping weights used in the CM2.

<EMAIL_ADDRESS>commented

That's the correct grid file.
namcouple_112fields_360_bil_GA7.1 is the one we're using.


WD_Demo: Provide example on how to pass additional arguments to ChromeDriver

There is a difference in how additional arguments are passed to the web driver based on the browser being used.
The wd_demo provides an example of how to pass acceptInsecureCerts to the Firefox driver.
A similar example should be made available for the chrome function as well.
I will raise a PR for this soon.

I don't know for sure without seeing an example, but this almost sounds like ti would be better as a FAQ on the wiki.

I have raised a PR for this example enhancement.
https://github.com/Danp2/WebDriver/pull/72

@rajan123456 I'm not inclined to merge this PR because it doesn't improve the functionality of the script. I would suggest either  a FAQ as mentioned above or create a new document (capabilites.md or something similar) where multiple DesiredCapabilities definitions could be shown along with detailed descriptions.

Hi @Danp2 - I am looking to add an example within the Demo script which demonstrates how to add additional parameters while invoking the chrome driver. For example, in the SetupGecko block below, we have options such as acceptInsecureCerts.
Func SetupGecko()
    _WD_Option('Driver', 'geckodriver.exe')
    _WD_Option('DriverParams', '--log trace')
    _WD_Option('Port', 4444)
    $sDesiredCapabilities = '{"capabilities": {"alwaysMatch": {"browserName": "firefox", "acceptInsecureCerts":true}}}'
EndFunc   ;==>SetupGecko

For the chrome driver, the syntax is slightly different, and I do not want to disable certificate verification by default, hence added a simple option in the demo script, start-maximized. A user looking at the script would be able to easily extrapolate other arguments based on the chrome driver documentation.
Func SetupChrome()
    _WD_Option('Driver', 'chromedriver.exe')
    _WD_Option('Port', 9515)
    _WD_Option('DriverParams', '--verbose --log-path="' & @ScriptDir & '\chrome.log"')
    $sDesiredCapabilities = '{"capabilities": {"alwaysMatch": {"goog:chromeOptions": {"w3c": true, "excludeSwitches": [ "enable-automation"], "args":["start-maximized"] }}}}'EndFunc   ;==>SetupChrome

I would suggest either a FAQ as mentioned above or create a new document (capabilites.md or something similar) where multiple DesiredCapabilities definitions could be shown along with detailed descriptions.

Let me know if you have a similar FAQ or document for documenting other capabilities for browsers such as Firefox, Edge - and I would add the note to them as well.

Hi @rajan123456 -- I gave you a couple of alternatives above. Personally, I like the idea of creating a new .md file where different Desired Capabilities configurations could be documented.


Custom IAM Role for Driver and Executor Pods?

Hi all,
I wanted to check if it is possible to define an AWS IAM role that should be attached to driver and worker pods in Spark application YAML file. As per my understanding, currently these pods inherit the role from EKS node-group, but I would like to specify my own custom role in the YAML file. Can somebody please suggest on the possibility ?
I thought of passing IAM Role / OIDC Service Account in below manner but did not work. It fails saying S3 access denied (403 error).
apiVersion: "sparkoperator.k8s.io/v1beta2" kind: SparkApplication metadata:   name: test-pyspark-app   namespace: dev spec:   type: Python   pythonVersion: "3"   mode: cluster   image: "coqueirotree/spark-py"   imagePullPolicy: Always   mainApplicationFile: s3a://my-bucket/TestFile.py   sparkVersion: "3.0.0"   sparkConf:     "spark.kubernetes.driverEnv.serviceAccount": "svc-spark-account"     "spark.kubernetes.driverEnv.serviceAccountName": "svc-spark-account"     "spark.kubernetes.authenticate.driver.serviceAccount": "svc-spark-account"     "spark.kubernetes.authenticate.driver.serviceAccountName": "svc-spark-account"     "spark.kubernetes.executorEnv.serviceAccount": "svc-spark-account"     "spark.kubernetes.executorEnv.serviceAccountName": "svc-spark-account"     "spark.kubernetes.authenticate.executor.serviceAccount": "svc-spark-account"     "spark.kubernetes.authenticate.executor.serviceAccountName": "svc-spark-account"   driver:     cores: 1     coreLimit: "1200m"     memory: "512m"     labels:       version: 3.0.0     serviceAccount: svc-spark-account     serviceAccountName: svc-spark-account   executor:     cores: 1     instances: 1     memory: "512m"     labels:       version: 3.0.0     serviceAccount: svc-spark-account     serviceAccountName: svc-spark-account
Can someone please advise if I should do something differently in this case? Thanks in advance :)

IRSA requires AWS SDK support with assume-web-identity-role. I have not check  dependencies. If you have time to help, feel free to take a look. Additionaly, S3 is a little bit different, because spark use hadoop-s3 package. You'd better check mapping and find the version.

@batCoder95

I was able to run SparkApplications, setting an iAM role for the service account
dependencies

Spark: 3.0.0
Hadoop: 3.2.1
AWS SDK: 1.11.82

I did not check other combinations of versions, but I think Hadoop requires 3.2.1 or upper because hadoop-aws 3.2.0 is built by older SDK which does not support assume-web-identity-role
references

hadoop 3.2.0: https://mvnrepository.com/artifact/org.apache.hadoop/hadoop-aws/3.2.0
minimum sdk for service account with iam role: https://docs.aws.amazon.com/eks/latest/userguide/iam-roles-for-service-accounts-minimum-sdk.html


feat: port Topology.Algebra.Order.ExtendFrom

Only changes are use to exact and adding one of the implicit arguments. I've also removed the universe and types.


Unable to serialise maps and UInt8Arrays at the same time

> a = new Map()
Map(0) {size: 0}
> a.set("test", 123);
Map(1) {'test' => 123}
> msgpack.decode(msgpack.encode(a))
{}
> msgpack.decode(msgpack.encode(a, {codec: msgpack.createCodec({usemap: true})}))
{test: 123}
> msgpack.decode(msgpack.encode(enc.encode("test")))
Uint8Array(4) [116, 101, 115, 116, buffer: ArrayBuffer(4), byteLength: 4, byteOffset: 0, length: 4, Symbol(Symbol.toStringTag): 'Uint8Array']
> msgpack.decode(msgpack.encode(enc.encode("test"), {codec: msgpack.createCodec({usemap: true})}))
{0: 116, 1: 101, 2: 115, 3: 116}

The last output is wrong. Note that deserialisation is without usemap in all examples; the issue is only at the serialization side (where, ideally, you'd not have to specify usemap at all, there is no reason you can't automatically detect that an object is a Map object just like you detect that it's a UInt8Array and handle it specially).

I'm using version 0.1.26 btw

I'm also facing the same issue. Did you find a solution for it?

Nope, just worked around the issue (accepted that keys are garbled and recurse over object and ungarble them; all of mine are byte arrays, so the string representation of them are deterministic and the mapping reversable).


Temperature sensor not available?

Could have sworn this was available before.

This also doesn't look good. Overview dashboard shows temperature sensor as unavailable. I know the other sensors only show up when you configure them but pretty sure this one was available before.

I'll put in a official bug report if you need me to, but maybe you can give me some pointers here.
Thanks,

You need to enable them.


Black Litterman efficient frontier Gamma Errror

ef = pyp.EfficientFrontier(bl_return_confi, S_bl_confi, weight_bounds=(0, 0.1), gamma=1)
gamma at the end is not working and gives error. I turn it to "0" but could be solved

Can you provide a specific error type? What is the message and how I can recreate the error? Also please specify the dependency versions. That could help


[FEATURE] Skill and experience timeline widget add

Describe the feature you want 描述你的功能需求
Can you add the similiar function like this theme, so users can add they skills and experience timeline to make the blog post as an resume.
Useful reference 有价值的参考
No response

I am not particular interested in this one, but PR is always welcome~

I am closing this issue for now as not planned. We can reopen it if there are more interests in this area.


Make MLflow integration work with kubeflow and scheduled pipelines

Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[x] If my change requires a change to the documentation, I have updated the documentation accordingly.
[x] I have added tests to cover my changes.

Types of changes

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

Describe changes
This re-factoring of the MLflow integration is required to make it also function properly in the context of remotely executed pipelines (e.g. kubeflow).

instead of a pipeline decorator, enable_mlflow is now a step decorator, while retaining all of its previous functionality. This is also the only breaking change introduced in this PR. This change was required because the modification made by pipeline decorators to user defined pipelines are not reflected in the compiled pipeline that is executed remotely.
a StepEnvironment global singleton/context is used to store runtime environment information for the current step that can be consumed from within the step function implementation. The same singleton is used by the MLflow step decorator to determine the current pipeline run ID and pipeline name used in creating the MLflow experiment and run contexts.
a scheduled pipeline run example is now available for the MLflow integration, in addition to the regular run

As a general side-question before I even start the review (and feel to just comment on it - i know it is outside of the scope of this particular PR):
Currently, we are in Scenario 1 of mlflow deployment:

Will mlflow work with kubeflow  in a more production-ready cloud-setting. I.e. When we are in Scenario 4 of mlflow:

Source: https://www.mlflow.org/docs/latest/tracking.html#scenario-4-mlflow-with-remote-tracking-server-backend-and-artifact-stores
If any design decisions have been taken in this PR that affect an easy transition into scenario 4, we must ask this question now before merging. Just wanted to bring that up before reviewing (my sincere apologies for that late comment on this issue - i know its a 3rd refactor of this and i should have raised this earlier perhaps)

If any design decisions have been taken in this PR that affect an easy transition into scenario 4, we must ask this question now before merging. Just wanted to bring that up before reviewing (my sincere apologies for that late comment on this issue - i know its a 3rd refactor of this and i should have raised this earlier perhaps)

@htahir1 This PR is orthogonal with how MLflow can be set up to store artifacts and experiment tracking data. If anything, it makes switching to other MLflow deployment scenarios even more flexible, because now it should theoretically be possible to connect different steps to different MLflow tracking servers and artifact storage backends.
Having a pipeline decorator instead of a step decorator did have one advantage over the current approach: you could enable all steps in a pipeline to MLflow with a single line of code. More of a syntactic convenience, but a useful one nonetheless. Having said that, we can still have both: a step decorator and a pipeline decorator (that basically applies the step decorator to all steps). If you feel we should add that to this PR, let me know, it's not too much effort.

Personally, I really liked the @enable_mlflow decorator at the pipeline level and would advocate for implementing it if its not a big hassle.

@AlexejPenner thanks for the CRs, you're a mkdocs / API docs guru. I hope I addressed all your comments.

@AlexejPenner thanks for the CRs, you're a mkdocs / API docs guru. I hope I addressed all your comments.

@stefannica You got it all :)

@schustmi thanks, I applied all your suggestions


Adding OpenCV library to qtrpi

I have successful cross-compiled by follow instruction of 1st video, Thank you for your work, I'm very appreciate that  :+1:
Currently, I'm also working on opencv with Qt, I installed opencv on my host and also done a simple project  (load image). So, I want to add opencv to qtrpi and using raspberry to load image but I don't know how to do that. I have read the 'Tutorial of init-qtrpi-full.sh' but I could not imagine the way to go ( The truth is I do not have much experience in library installation, the installation of opencv I did before is completely depend on step by step tutorial :( .. )
So, could you please teach me how to add opencv library to qtrpi ? thank you.

The QtRpi project only provides an easy-to-use way to cross-compile Qt for the RaspberryPi. OpenCV is clearly out of the scope of QtRpi.
Your task is not easy if you lack of experience in library and cross-compilation. However here are some tips to get Qt + OpenCV on Rpi:

Keep in mind that all tasks (prepare, compilation, ..) can be done directly on your Rpi. The cross-compilation is only here to provide a convenient (and faster) way to develop and build your application.
So you can do your first experiments on the Rpi directly. You will at first, avoid the cross-compile difficulties. Use only QtRpi for the Qt part for now.
Prepare the Rpi (depedencies) and build OpenCV on the Rpi.
Build an easy C++/OpenCV on your Rpi.
Build an easy C++/Qt/OpenCV on your Rpi.
Now that you know how to do it on a Rpi, you can update the file prepare-sysroot-full.sh with all your OpenCV stuff
You should now be able to cross-compile a C++/Qt/OpenCV appliaction from your host

You should search for some generic tutorials to compile OpenCV on a raspberry pi (without the Qt or the cross-compilation part for now).

Thank you, I will test these points :)


When the outputModule of the experiments is configured as true, the method of importing the module packaged by the dll plugin is undefined

Bug report

When webpack is configured
{
  experiments: {
    outputModule: true
  }
}

Introducing the module packaged by webpack.DllPlugin, the method will be undefined.
What is the current behavior?
After a module is packaged by webpack.DllPlugin, the method of importing the module, the method is undefined, resulting in an error.
If the current behavior is a bug, please provide the steps to reproduce.

https://github.com/duan602728596/webpack-dll-bug

npm install
npm run dll
npm run start
Check index.html, the console will output the method

What is the expected behavior?

Other relevant information:
webpack version:  5.39.0
Node.js version: 16.3.0
Operating System:
Additional tools:

No problems, DLL by default create commonjs format (look at the type option), so you should use default export, you can use:
import * as react from 'react';

const { createElement } = react;

but yes, it is not good.
i.e. you need to set (unfortunately we cannot inherit outputModule, you need explicitly set type, same for library.target):
new webpack.DllPlugin({
  path: path.join(__dirname, 'dll/manifest.json'),
  name: '[name]_[hash:5]',
  type: 'module'
})

Unfortunately, external pure ECMA modules is not supported yet, close in favor https://github.com/webpack/webpack/issues/8895


boot failure after updating to 2020-04-06 image on Beaglebone Green Wireless

Hi there,
I was unable to update my BBGW board to the latest image. The existing image onboard is 2016-05-27. I tried the 2020-04-06 images with emmc flasher. Burning looked working, the LEDs kept cycling as expected and after ~5 mins all LEDs were off.
However, after removing SD card and powering the board, the booting was not successful. At the beginning those LED blinked a few times, then settled with fewer on. I couldn't ssh to it via either USB or WiFi.
What I don't understand is even when booting from SD card, the image version displayed at the terminal still shows the old one. I don't have a USB2TTL cable to see what's going on with booting. Can anyone help?
Thanks,
Dawei

@dwliu other then no usb interface, everything sounds correct..  Sounds like it booted but just failed.
What OS are you running on your Development PC, Windows (7,8,10,etc), Linux or Mac?
Do you use see a new "BeagleBone-WXYZ" ssid when you scan for wireless devices on any phone/computer?
WXYZ = random mac, so it won't match..
What was the file name of the 2020-04-06 release that you tried to use? Did use use https://www.balena.io/etcher/ to write the microSD?
Regards,

@RobertCNelson Sorry for the late response. Thank you so much for replying! I am on MacOS 10.11. However, I am not sure if it is related to my computer. I tried burning the BBGW with power supply only, and it booted failed the same way.
I had wireless working before I burned the new image. After burning, the green LED near WiFi module was not on, and the SSID was not detected either.
For the images, I tried both AM3358 Debian 10.3 2020-04-06 4GB SD IoT, and AM3358 Debian 10.3 2020-04-06 4GB eMMC IoT Flasher. They ended up the same. I didn't use BalenaEtcher to write the image though. I used Raspberry Pi Imager and manually assigned the image (thought it should work because verification succeeded).
Given what you said, I am leaning toward that either microSD or the burning has issue. I am going to try the burning tool you recommended.
Thanks again.
Dawei

BalenaEther imager didn't help either. Got the same issue. I have only one 4GB microSD card. Tried with 16GB card but the imager reported error. Unless you have any other suggestion, I guess I'd wait until I have the USB2TTL cable to check the boot process.


[CONF] add s3 urlprefix

This PR adjusts conf.py to add a url reference for static assets in generated notebooks.
Current location is s3 compare-lectures bucket.
This will need to be updated in the future.
This will require the latest sphinxcontrib-jupyter version.

If this is merged, does it mean we will be able to use the notebooks with links not in _static?  Might be work checking sooner rather than later.

hey @arnavs no update should be required as the prefix should take you to the relative root level (as per the relative link).

@jlperla what links (not in _static) do you have in mind? Can you give me an example?

I am not sure what I meant... but I think I meant testing?  Basically, take the notebook output which has images refereed, move a file so that it no longer has any local folders accessible, and see if it works.  RIght now we need to distribute the whole _static folder with the notebook to see images.

if we use the prefix then the whole contents of _static can be referred via the internet with this setup. That reminds me though we probably need to add prefix options for:

[ ] _downloads for internal links to files.

Thanks @mmcky, I think this looks good and the output works, so we should merge it. Will create a separate issue for these checkmarks/TODOs.


Check for plugin compatibility

fixes #650
Adds a notification window if plugin versions are not equal:

Todo: finalize and externalize wording.
Note: the check is owned by the cloud tools plugin. Once this is pushed out and users update their plugin, any subsequent plugin updates where the versions are mismatched will show this warning.

Looks good.
Instead of ' check for updates. Or, manage..'
How about "Please click here to ensure you've updated all your plugins."
Also, localization.

The only issue with your suggested wording (assuming 'click here' links to the platform/plugin update dialog) is that it also includes platform updates. I did not see the ability to only update installed plugins, but I'll check to make sure.
I think its a little restrictive if the user is forced to also update the platform just to get the latest version of the plugins which is why I added that "manage" link originally.

interesting. So if there's a platform update and plugin updates, the user
can't pick and choose?
On Mon, Apr 25, 2016 at 3:50 PM, Etan Shaul<EMAIL_ADDRESS>wrote:

The only issue with your suggested wording (assuming 'click here' links to
the platform/plugin update dialog) is that it also includes platform
updates. I did not see the ability to only update installed plugins, but
I'll check to make sure.
I think its a little restrictive if the user is forced to also update the
platform just to get the latest version of the plugins which is why I added
that "manage" link originally.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/gcloud-intellij/pull/651#issuecomment-214497536

Not from what I've seen.
Here's what the "update" link opens up (which is the same as choosing check for updates from the help menu). It looks like the platform update supersedes any plugin updates:

Localized strings

Actually, if you click "ignore this update" in the above screenshot, and there are outdated plugins, then only plugin updates will be shown:

So I think your wording suggestion is fine.

Updated:

:ship:


cri: Don't use rel path for image volumes

Runc 1.1 throws a warning when using rel destination paths, and runc 1.2 is planning to thow an error (i.e. won't start the container).
Let's just make this an abs path in the only place it might not be: the mounts created due to VOLUME directives in the Dockerfile.

We discussed this issue in today's containerd community meeting and decided to make this small PR that should fix it, and if this is merged without concerns, backport it to 1.6 and 1.7.
cc @mikebrow

As discussed in the community meeting, perhaps a switch is nec. in runc, deprecated/temporary, to enable ignoring/disabling the abs check, until such time as the affected container runtimes have been patched and deployed and/or are no longer in support.

/ok-to-test

@gabriel-samfira @fuweid thanks! Re-pushed so tests run again, it didn't seem related to this PR.

@gabriel-samfira @fuweid fixed the CI now, PTAL (the tests I added were quite useful!).
The issue was that when executing on Windows, filepath.Join() does a clean, which changes from slash to backslash, and didn't match the expected paths.
I've adjusted the code to work fine on windows and linux, and added a log line as @mikebrow suggested. The changes were basically this.

@rata would you update with mike's suggestion? after update, I will merge it. Thanks

@fuweid just changed the log line from info to debug, thanks!


Gridstack slow to render the grid when loading from cookie OR localStorage

Hello,
I have created a grid for my webpage. When Gridstack is loaded, it checks if a cookie exists, if it does it loads the JSON data from the cookie and loads the grid based of the data. If the cookie does not exist, it will load the "default layout" of the grid.
This enables me to support the feature of loading saved grids even after the browser/windows has closed.
This is my script so far:
$(function() {
    // Screen measurements
    var screenWidth = $(window).width();
    var screenHeight = $(window).height();
    var qscreenWidth = screenWidth / 4;
    var qscreenHeight = screenHeight / 4;

    // GridStack INIT
    var options = {
        width: 12,
        float: false,
        removable: '.trash',
        verticalMargin:0,
        cellHeight:qscreenHeight,
        removeTimeout: 200,
        acceptWidgets: '.grid-stack-item',
        animate :true,
        draggable: {
            handle: 'header.widget-header'
        }
    };
    $('.grid-stack').gridstack(options);

    // Load the grid layout
    loadGrid();

    // Save the grid layout
    $(".grid-stack").on("change", function(event, items) {
        saveGrid();
    });
});

/*
 * Save the grid layout to a cookie.
 *
 * localStorage is too slow, so we have to use a cookie as a storage solution.
 */
function saveGrid() {
    var res = _.map($('.grid-stack .grid-stack-item:visible'), function (el) {
        el = $(el);
        var node = el.data('_gridstack_node');
        return {
            id: el.attr('data-custom-id'),
            x: node.x,
            y: node.y,
            width: node.width,
            height: node.height
        };
    });
    deleteCookie("th-dashboard-layout");
    createCookie("th-dashboard-layout", JSON.stringify(res), 30);
}

/**
 * Load the grid from the cookie, if the grid does not exist, use our default grid.
 */
function loadGrid() {
    if(getCookie("th-dashboard-layout")) {
        var serialization = getCookie("th-dashboard-layout");
    } else {
        var serialization = [{"id":"1","x":0,"y":0,"width":3,"height":2},{"id":"2","x":3,"y":0,"width":6,"height":2},{"id":"3","x":9,"y":0,"width":3,"height":2},{"id":"4","x":0,"y":2,"width":3,"height":2},{"id":"5","x":3,"y":2,"width":6,"height":2},{"id":"6","x":9,"y":2,"width":3,"height":2},{"id":"7","x":0,"y":4,"width":3,"height":2},{"id":"8","x":3,"y":4,"width":6,"height":2},{"id":"9","x":9,"y":4,"width":3,"height":2}];
    }

    serialization = GridStackUI.Utils.sort(serialization);

    var grid = $('.grid-stack').data('gridstack');
    grid.removeAll();

    _.each(serialization, function (node) {
        if(node.id == "1") {
            grid.addWidget($('

Littlefish FX Pip Counter & Trading Room Hours

Widget 1 content

'),
            node.x, node.y, node.width, node.height);
        } else if(node.id == "2") {
            grid.addWidget($('

Live Chart

Widget 2 content

'),
            node.x, node.y, node.width, node.height);
        } else if(node.id == "3") {
            grid.addWidget($('

Latest News

Widget 3 content

'),
            node.x, node.y, node.width, node.height);
        } else if(node.id == "4") {
            grid.addWidget($('

Latest Analysis Articles

Widget 4 content

'),
            node.x, node.y, node.width, node.height);
        } else if(node.id = "5") {
            grid.addWidget($('

Live Trading Room

Widget 5 content

'),
            node.x, node.y, node.width, node.height);
        } else if(node.id == "6") {
            grid.addWidget($('

Latest Educational Articles

Widget 6 content

'),
            node.x, node.y, node.width, node.height);
        } else if(node.id == "7") {
            grid.addWidget($('

Live Twitter Feed

Widget 7 content

'),
            node.x, node.y, node.width, node.height);
        } else if(node.id == "8") {
            grid.addWidget($('

Order Book Data

Widget 8 content

'),
            node.x, node.y, node.width, node.height);
        } else if(node.id == "9") {
            grid.addWidget($('

Live Economic Calendar

Widget 9 content

'),
            node.x, node.y, node.width, node.height);
        }
    });
}

/**
 * Create a cookie with a value and expiration.
 */
function createCookie(name, value, days) {
    if (days) {
        var date = new Date();
        date.setTime(date.getTime() + (days * 24 * 60 * 60 * 1000));
        var expires = "; expires=" + date.toGMTString();
    }
    else var expires = "";
    document.cookie = name + "=" + value + expires + "; path=/";
}

/**
 * Get the cookie value.
 *
 * Can also be used to detect whether the cookie exists.
 */
function getCookie(name) {
    var nameEQ = name + "=";
    var ca = document.cookie.split(';');
    for (var i = 0; i < ca.length; i++) {
        var c = ca[i];
        while (c.charAt(0) == ' ') c = c.substring(1, c.length);
        if (c.indexOf(nameEQ) == 0) return c.substring(nameEQ.length, c.length);
    }
    return null;
}

/**
 * Delete or expire the cookie.
 */
function deleteCookie(name) {
    createCookie(name, "", -1);
}

And this is my HTML:
`
`
I have tried also using localStorage, however I get the same problem. You can see a working example here. If you either resize a widget, or change its position, it will automatically save. After it has saved, ensure that the cookie "th-dashboard-layout" exists, if it does refresh the page. You will then see it takes Gridstack a while to load the grid, once it does, it doesn't even load it correctly.
When I pass the JSON data directly to Gridstack rather than getting Gridstack to access the cookie or localStorage, it loads instantly and displays correctly.
I'm not wanting to try with sessionStorage as it simply does not meet my requirements (restore grid layout after the browser/window closes).
Thanks,
Kieron

The problem is because you aren't converting your JSON string stored in the cookie back to a Javascript object. You need to parse it using this:
var serialization = JSON.parse(getCookie("th-dashboard-layout"));
Also when you serialize the layout, the id doesn't get written out as it is referring to an attribute that doesn't exist 'data-custom-id', so the type of node is lost when you save the grid.

Thanks, @euang . What you're suggesting should work. Were you able to have any success, @kieronsutton00 ?

Closing due to inactivity.


doc on wiki

Cool project ! thx ! Please add this awesome binding to the list of current clients: http://wiki.ros.org/Client Libraries

Thanks for the suggestion, I just added it to the list.


Need Cancel button. Soft-Abort.

There is currently no way to STOP a process without having to go through the re-zero, re-home and likely re-perspective.
Need a clean & recoverable  STOP button.
the last time I aborted a job I had to power-cycle the K40, load up K40Whisperer, "initialize laser" close K40W and THEN a load up Meerk40T as MeerK40T can't do a full "reinitialization" when it has gotten confused.
currently I will just disable the laser, and let it continue on rather than have to re-do everything after an abort....
PS....6.17 seems MUCH a better...

A recoverable stop button. Like empty the current buffer, and then exist where you actually are without needing to kill things? Like nothing's in danger, go ahead and do that extra 2 seconds of data in the buffer, then just keep working without interruption. Like not "stop" but rather cancel.

Cancel would work.  It is for when the material isn't placed correctly, the power is set wrong, or something else that makes you want to cancel, re-position and re-start.  Currently, a "restart" is a rather painful process involving the "other" software to re-initialize the K40...
A STOP, clear the buffer, goto PHYSICAL HOME, then to MeerK40T Home, then BEEP would be ideal.
The Goto Physical Home (0,0) establishes where you are, going to MeerK40T Home is ready to re-start work, and BEEP is READY to go.

Another idea, based on the Spooler and Operations menu would be a "SKIP Current Operation".
I was working on a "4 Up" when I realized the material wasn't secured and the air-assist had moved it.  I just wanted to SKIP the 1st of the 4 items it was going to engrave after a pause to secure everything.  The FIRST of the 4 was trashed, but the next 3 would be fine...  PAUSE, SKIP, Un-PAUSE would go to the next item after clearing out the buffer and establishing where it was.  (Home 0,0, MeerK40T Home, then on to next item)
This would be a MAJOR feature upgrade, but very useful

I think you need to hit stop a few more times and home a few more times. I've never needed to disconnect and reconnect the K40 for that. But, there's a button for that controller.

My K40 Doesn't have a stop button.  Not sure if it was an earlier model or
what.  I have the ANALOG version, with a current meter, power dial, Laser
enable, and laser test.  Thats it.
The Only STOP/PAUSE button I have is the Meerk40T "PAUSE"  or the
"Controller STOP"
Please don't wait for me to respond when booking an inspection. I turn off
my devices when on appointments and someone else could get your spot.
Please CALL our service or book yourself online to save the spot as soon as
you know the time you want.
Bob Sisson ACI, BVI
Inspections by Bob
ASHI Member # 212016
MD Home inspectors License #29666*
*MAC-ASHI Chapter President 2010-2012
ASHI Mid-Atlantic Group Leader 2009-2012
ASHI National Board Of Directors 2012-2015, 2017,2018-2020
<EMAIL_ADDRESS>www.inspectionsbybob.com
(301) 208-8289 Scheduling/Pricing
On Mon, Feb 1, 2021 at 9:30 PM tatarize<EMAIL_ADDRESS>wrote:

I think you need to hit stop a few more times and home a few more times.
I've never needed to disconnect and reconnect the K40 for that. But,
there's a button for that controller.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/meerk40t/meerk40t/issues/325#issuecomment-771304379,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJ2GYYAVHI2TQTW7MSLB3DS45PV5ANCNFSM4W5OHHJQ
.

You could also try hitting the disconnect reconnect button in controller that somehow stops. I dunno why it would take resetting that elsewhere. But, resetting is part of disconnect for the program.

Whenever something is "running,"  there needs to be a big red "CANCEL"
button that stays on top.  If I see a pattern will not fit on my material,
I really don't want to take the time to go looking for a menu, especially
if that menu is minimized/hidden for some reason and doesn't want to be
seen.   My
wife wants KISS.  If something is going wrong, she is just going to turn
off the Laser and yell, "BOB, come fix this thing..."
I need HER to run it without me and deal with MOST issues that come up
without having to re-do any calibrations or worry about what order to do
things...  It needs just to work, each time, every time, and if it doesn't
there needs to be a clear "RESET/Home/Clear/whatever..." to get it back to
a known state so SHE can run something through...
Please don't wait for me to respond when booking an inspection. I turn off
my devices when on appointments, and someone else could get your spot.
Please CALL our service or book yourself online to save the spot as soon as
you know the time you want.
Bob Sisson ACI, BVI
Inspections by Bob
ASHI Member # 212016
MD Home inspectors License #29666*
*MAC-ASHI Chapter President 2010-2012
ASHI Mid-Atlantic Group Leader 2009-2012
ASHI National Board Of Directors 2012-2015, 2017,2018-2020
<EMAIL_ADDRESS>www.inspectionsbybob.com
(301) 208-8289 Scheduling/Pricing
On Mon, Feb 1, 2021 at 10:34 PM tatarize<EMAIL_ADDRESS>wrote:

You could also try hitting the disconnect reconnect button in controller
that somehow stops. I dunno why it would take resetting that elsewhere.
But, resetting is part of disconnect for the program.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/meerk40t/meerk40t/issues/325#issuecomment-771329589,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJ2GYYTBYY3BF3CBHTI2MLS45XC3ANCNFSM4W5OHHJQ
.

As above Abort on my K40 m2 nano does NOT require reinitialisation, but (as reported elsewhere) it now goes elsewhere and grinds the steppers rather than instantly stopping. Also mostly needs 2x click on Home to do an actual Home suggesting m2 nano is not left in a correct state.
But what is for sure is that when you use Abort, an unknown number of instructions already sent to the laser will be lost, and so position will be unknown. A Cancel feature that simply stops sending any more instructions should (in theory) leave a known position allowing some sort of restart as requested by @inspectionsbybob. I guess you would need to return the m2 nano to a set state - so strictly speaking some further instructions would need to be sent.


VCOCO evaluation

For V-COCO, you  use the utilities provided by Gupta et al, but the length of the test dataset is 4532 in your code，this means that the resulting cache.pkl contains only 4532 outputs. However, in the utilities provided by Gupta et al.,  the images in the v-coco/data/splits/vcoco_test.ids is 4946, which is not equal to the 4532. Will this affect the final test results? Thanks.

Hi @aqingcv,
I vaguely remember that I removed those images without valid human-object interactions, since some annotations in V-COCO only have <subject, predicate> pairs, but not the triplets.
Fred.


Formatter: Keep Tree Structure

Hi, I use this vscode extension for Hugo project not for Rust project.
A tree structure will help me to manage, but the formatter will delete all space in the start of lines. How to keep it?
Am I missing a setting for it? Or it should be?
The settings I already enabled are:
{
    "evenBetterToml.formatter.alignEntries": true,
    "evenBetterToml.formatter.indentEntries": true,
    "evenBetterToml.formatter.indentTables": true,
    "evenBetterToml.formatter.inlineTableExpand": true
}

I came here for the same thing.

How do I preserve / control indentation?
How do I ignore formatting for a file? (in my case it is a fly.io fly.toml file that is auto-generated and I would prefer not to have to format it every time).


Install default_env on all nodes

#1240  👈
main

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

Join @dongreenberg and the rest of your teammates on  Graphite

Merge activity

Sep 6, 6:47 PM EDT: @dongreenberg started a stack merge that includes this pull request via Graphite.


feat: Update Notice source code such that field name comments match documentation in RULES.md

As part of our effort to alignment documentation and source code in #1324, this PR applies an automated transform to update the comments of each Notice field in source to match the equivalent field documentation string from RULES.md.  This gets us one step closer to generating RULES.md from source code directly.
Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Run the unit tests with gradle test to make sure you didn't break anything
[x] Format the title like "feat: [new feature short description]". Title must follow the Conventional Commit Specification(https://www.conventionalcommits.org/en/v1.0.0/).
[x] Linked all relevant issues

@isabelle-dr I know this review looks a little bit scary, but it's largely an automated transformation from RULES.md to source code.  As always, curious what you think.

@bdferris-v2 in looking at this for #1289 I noticed some copy/paste mistakes where row number comment was copied for file name field, so I fixed those.

@briandonahue those are actually errors in the source RULES.md file :)  But fixing them here is appreciated, thanks!

After the conflicts are resolved, I think we're good to merge this PR!
cc @davidgamez


Improve ux for select node in all solutions

We're dealing with a single form in all solutions right now which not allowing us to disable <SelectNode /> if user entered some invalid data.
todo:

Move <SelectNode /> into it's own form and allow to disable it based on other form's validation value

verified
select node has it's own from now
automated and manual

in the automated selection load node button is enabled when the user insert new data or change specifications

testlodge:
https://app.testlodge.com/a/26076/projects/40893/suites/234374?expand_section=422336#case_4164692
https://app.testlodge.com/a/26076/projects/40893/suites/234374?expand_section=422336#case_4164673

Verified:

Devnet
2.3.0-alpha3


website stuff

*.js.org
the people at js.org provide free subdomains for your projects. I thought it might be cool to grab cell.js.org
https://github.com/js-org/dns.js.org
Proper CDN with versioning
i think it would be best to serve cell from a versioned cdn cdn.celljs.org/1.0.0/cell.min.js perhaps using cdnjs as our main cdn

*.js.org

I was just looking into this, but looks like we will have to give up on celljs.org if we want to apply for cell.js.org because github allows only one CNAME... Let me know if you know a way around

Proper CDN with versioning

Great idea, do you know a way to automate this? Or is this something we manually update everytime there's a new release? (been an "avid user" of cdnjs but never really hosted my own library there before 😅) Or even a pointer to how to get this setup would be awesome too. Thanks!

CNAME allows one host per line iirc (i might be wrong)
cdnjs works by having a friggin huge github repo with every version of every lib they host, and updates are just prs. This could be automated with travis.

@gliechtenstein could you tag a release sometime in the near future so cdnjs's auto update can run when they merge the pr

Wow totally missed that entire cdnjs thread and just saw it, sorry about that.
I just went ahead and created a release from master branch https://github.com/intercellular/cell/releases/tag/v1.0.0
This would work right?

@gliechtenstein https://cdnjs.cloudflare.com/ajax/libs/cell/1.0.0/cell.min.js 🎉

@devsnek Awesome! Thanks! 🔥🔥🔥
I'll close this since the cdnjs issue is taken care of and there's not much we can do about the CNAME thing at the moment. I guess we can revisit if something changes or there's a way to do this without giving up on the main domain :)

@gliechtenstein i was thinking perhaps the links everywhere that go to celljs.org/cell.min.js should be changed to cdnjs links and also the cell.min.js should be removed from the develop branch

@devsnek done and done and done 👍

Switched out the celljs link from the tutorial repo https://github.com/intercellular/tutorial/commit/70c90efe9f328147be075f7d4fa30081e79a73e6
Updated all existing code from play.celljs.org
Removed cell.min.js from develop: https://github.com/intercellular/cell/commit/b1e8437b0e01c4323d52f1108bb9f65171388a59


Master Branch: Kaminari  vs Will_paginate errors

Hey,
With the new master branch migrating to kamari, will_paginate is now incompatible, leading to errors such as this.  Perhaps we could support both  will_paginate or kaminari.
undefined local variable or method 'per' for #<ActiveRecord::Relation:0xa0bbc74>
relevant commit:   https://github.com/gregbell/active_admin/commit/96a0cfa25fb3bcb0be7e4b7121b627cce6f7b960#lib/active_admin/resource_controller/collection.rb
-Mario

+1 for this being an issue.

+1 for this issue

This commit could help a bit: https://github.com/amatsuda/kaminari/commit/3cf929f7d000997ceca8a25cbfa81e5bb299088e
I'm using kaminari 0.13.0 and below is my patch to paginate method:
module ActiveAdmin::ResourceController::Collection::Pagination
  def paginate(chain)
    chain.send(Kaminari.config.page_method_name, params[:page]).per(@per_page || active_admin_application.default_per_page)
  end
end if Kaminari.config.respond_to?(:page_method_name)

I have it on top of config/initializers/active_admin.rb file.
config/initializers/kaminari_config.rb:
Kaminari.configure do |config|
  config.page_method_name = :bonzo # :)
end

I have upgraded Kaminari in ActiveAdmin and changed the pagination method in this PR https://github.com/gregbell/active_admin/pull/869
With the patch changing the page_method_name in the initializer, eliminates the conflict with will_paginate

Looks like this got fixed thanks to #869! :)

#Its working for me
if defined?(WillPaginate)
module WillPaginate
module ActiveRecord
module RelationMethods
def per(value = nil) per_page(value) end
def total_count() count end
end
end
module CollectionMethods
alias_method :num_pages, :total_pages
end
end
end


Feature request: Multiple configurable new_user email messages (per incoming channel)

Context of the request
We have several email Addresses routed to helpy (like info@, support@, etc.) and everyone is getting the same new_user message. This is for some customer confusing: They contacted just the general inquiry info@ but receive the specialized message for our support requests.
Requested Function
The locales key new_user should be configurable to have several different messages. Especially the welcome message should be able to be configured per incoming email address (info@, support@, abuse@, etc.). Furthermore all self created tickets (from agents during a phone call for example) should also get different welcome messages.

This is possible with the cloud hosted or on-premise cloud version by creating trigger responses that are filtered by channel, group, etc.  Closing from here.


Fix #381 StatsD Reporting while using orchestrator

Fixes #381

Codecov Report

Merging #382 (a463caa) into develop (935a95a) will increase coverage by 0.09%.
The diff coverage is 93.33%.

@@             Coverage Diff             @@
##           develop     #382      +/-   ##
===========================================
+ Coverage    82.18%   82.27%   +0.09%
===========================================
  Files           31       32       +1
  Lines         1044     1072      +28
  Branches       183      193      +10
===========================================
+ Hits           858      882      +24
- Misses         186      190       +4

Impacted Files
Coverage Δ

packages/core/src/package/PackageDiffImpl.ts
92.30% <93.33%> (+1.39%)
:arrow_up:

.../core/src/sfpcommands/apextest/TriggerApexTests.ts
100.00% <0.00%> (ø)

packages/core/src/utils/SFPStatsSender.ts
50.00% <0.00%> (ø)

...ages/core/src/artifacts/ArtifactFilePathFetcher.ts
30.33% <0.00%> (+0.33%)
:arrow_up:

packages/core/src/project/ProjectConfig.ts
88.52% <0.00%> (+1.63%)
:arrow_up:

Continue to review full report at Codecov.

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


Interesting portal placement

Is it a mod compatibility issue?
No
If it's a mod compatibility issue, please provide the link(s) to the incompatible mod(s)
No response
Minecraft version
1.20.1
Describe the issue

Immersive Portals causes some interesting portal frame generation, this one is one of the best.
It happens also in vanilla world generation. Maybe amplified by my combination of Terralith and Tectonic.
Maybe use vanilla portal frame generation and then place portal there?

Vanilla always generate 2x3 nether portal which ignores shape difference so ImmPtl cannot use vanilla portal generation.
The screenshot seem to be non-vanilla dimension. Please provide an example of abnormal nether portal generation in vanilla dimensions.

Some parameters has changed to make this less likely to occur.


Update page.tsx

Related Issue
Closes #354
Description
Features Section:
Utilized a grid layout (grid-cols-1 md:grid-cols-3) for displaying key features (Real-time Collaboration, Intuitive Editor, Secure Sharing).
Each feature card (bg-gray-100) includes updated styling (p-8 rounded-lg shadow-lg) for consistency and readability (text-gray-800).
How to Use and FAQs Sections:
How to Use: Structured as a step-by-step guide (grid-cols-1 md:grid-cols-2) with numbered icons (bg-blue-600 text-white) and descriptive text (text-xl font-semibold).
FAQs: Presented in a grid format (grid-cols-1 md:grid-cols-2) with cards (bg-gray-100) containing common questions (text-2xl font-bold) and detailed answers (text-gray-800).
User Testimonials:
Showcased user quotes (bg-gray-100) in a consistent format (p-8 rounded-lg shadow-lg) with improved readability (text-gray-800).
Contact Information and Footer:
Enhanced the Get in Touch section (bg-blue-800 text-white) with updated styling (p-8 rounded-lg) and contact details (text-gray-800).
Footer (bg-gray-100) includes copyright information (text-gray-800) and links (text-blue-600 underline) to privacy policy and terms of service.
Overall Layout and Structure:
Maintained the existing layout (container mx-auto px-4 py-8) for responsiveness and consistency across different screen sizes.
Ensured all content sections (section) are properly structured and spaced (mb-12, mt-10) for improved readability and user experience.
Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]
Checklist:

[X] I have performed a self-review of my code
[X] I have read and followed the Contribution Guidelines.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

@mahek0620

@subhadeeproy3902 file has been modified

@mahek0620 Version has changed. git pull and then fix the issue you have raised and send me a PR again. I will get to it ASAP


[Question] Emulator and Skype will render markdown on the message's "text" element but will ignore markdown on the attachment's "text" elements

System Information (Required)

SDK Language: C#
SDK Version: <IP_ADDRESS>
Development Environment: LOCALHOST

Issue Description
As already mentioned, this is down to both the channel (client - e.g. emulator, skype, facebook, your own directline client) implementing markdown support, since the text is sent as markdown in the json response.
However, at the moment you'll probably find that both the emulator and skype will render markdown on the message's "text" element but will ignore markdown on the attachment's "text" elements, e.g.
Code Example
HeroCard heroCard = new HeroCard()
{
    Text= "Booo - no *markdown* supported here",
    Buttons = cardButtons
};

vs
var reply = activity.CreateReply("**Lovely lovely markdown**\n\n *yey!*");

var heroCard = new HeroCard()
{
    Text = "Booo - no *markdown* supported here",
    Buttons = cardButtons
};

reply.Attachments = new List<Attachment> {
  hero.ToAttachment()
};

Expected Behavior
It should support Markdown
Actual Results
Emulator and Skype will render markdown on the message's "text" element but will ignore markdown on the attachment's "text" elements

@naqushab are you requesting this as an enhancement or trying to figure out a different way?

From the testing I did it does not seem that you can use markdown in the card itself.

Yes. But I want to apply some styling, add a new line in card's text. Is there a way I can?

Can you show me an example of what you mean by adding a new line?  If you are just trying to send a line break "\n\n" was the only thing I could ever get to work, but I don't think that is your question because you showed me "\n\n" in your example.  Also you may want to look at Adaptive cards.  They have TextBlock elements that may be useful for you.  Samples here, Cool blog here. Good luck!


Keep spaces and newline characters when sanitizing the commit message

To reduce the already unlikely chance of accidentally  setting a version bump #flag in the sanitized version of the PR description even further, we now keep whitespaces and newline characters.
#patch

Yeah, I really don't like that either. It was also hard to find that this syntax actually exists and works. I still thought that this is nicer than using another tool such as tr when bash has this pattern replacement built-in.


An Array is given with 0s, 1s and 2s and you need to sort the array

The extension is missing.
use comment for the output details.


Improve the animations for the attacks

[!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 #33

PR #34   👈

This stack of pull requests is managed by Graphite.

Merge activity

Dec 21, 8:56 PM: @@kadhirvelm started a stack merge that includes this pull request via Graphite.


Blue screen problem on 19044

Win10 19044 32-bit operating system
Hooker Driver triggers unreadable errors when hooking functions。
I remember fixing this a few months ago, but the code was lost。
It seems to be because ExAllocatPoolWithTag, but I forget the exact fix.

Sorry, I forgot to post a reply on this issue.
Thank you for reporting the bug!
I'm currently busy with other tasks, but I will look into the problem when I have some time.


This Document seems wrong

There is no session timeout option in system settings for on-premises installed in local VM. Is this document applicable for on-premises installed in Azure VM? please mention it clearly on for whom does this feature applies to

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

ID: 17a34c85-e068-c5a9-6db7-ff20690ea8e8
Version Independent ID: 46997a79-b404-51bc-9b9d-8f835b1c1714
Content: Security enhancements: User session and access management
Content Source: ce/customerengagement/on-premises/admin/user-session-management.md
Service: crm-online
GitHub Login: @jimholtz
Microsoft Alias: jimholtz

@irah2008
Thanks for bringing this to our attention. You're right about the content. We've revised to cover on-premises only. Azure Active Directory does not apply to on-premises version.


[Server/Client]: Use an enum for the websocket message types

WIP!
Task Description:
Right now we are using strings for denoting what the type of a message is (a move, a request for ids, etc). We should switch to using an enumerator instead. This will allows us to much more easily change and add messages. It will also prevent the possibilities of typos and confusion about what messages are supported.
Implementation Steps:
Everywhere that sends and reads messages will need to use the same mess

Difficulties:

Prioritization:

Extra Info:

Relevant Links:

Time Estimate:

If you have any questions while working on this task, post in the Programming Channel on Discord, reply to this issue on GitHub, or ask The Software Team Lead at one of our meetings.

Already done in a branch soon to be merged


Critical dependency: the request of a dependency is an expression - /webgpu/webgpu-graphics-device.js

Title: Critical dependency error in playcanvas module with webpack
Issue description
When building my project, webpack throws a critical dependency warning associated with the playcanvas module.
The warning states
WARNING in ./node_modules/playcanvas/build/playcanvas.mjs/platform/graphics/webgpu/webgpu-graphics-device.js
Critical dependency: the request of a dependency is an expression

Steps to Reproduce

import * as pc from 'playcanvas';

export default class ViewerManager {

    constructor() {
        console.log(pc)
    }
}

Environment
Node: ^16.0.0 || ^18.0.0
npm: ^7.0.0 || ^8.0.0 || ^9.0.0
playcanvas version: 1.66.0
webpack version: ^5.76.3
Expected behavior
The project should build without any critical dependency warnings.
Actual behavior
Webpack throws a warning regarding a critical dependency issue in the playcanvas module.

Thank you for pointing it out!
Happens since https://github.com/playcanvas/engine/pull/5711
We should stick to document.createElement('script') in src/platform/graphics/webgpu/webgpu-graphics-device.js
In addition to this webpack issue, we can also no longer use relative paths because of import.

I believe it is due to those 2 lines:
https://github.com/playcanvas/engine/blob/3fc8bf0ef9b50b32335cd5c7cc0718eaaf1eb7ed/src/platform/graphics/webgpu/webgpu-graphics-device.js#L169-L172
Here the guys suggest to explicitly convert to string:
https://stackoverflow.com/questions/42908116/webpack-critical-dependency-the-request-of-a-dependency-is-an-expression
However, it sounds like a webpack issue? Not sure the engine should support it, as it has chosen rollup.

However, it sounds like a webpack issue? Not sure the engine should support it, as it has chosen rollup.

We have to be agnostic about that, the engine (ESM files in npm package) are consumed by third-party bundlers to create a user bundle

Any thoughts on this @slimbuck ?

We have a mentioned of it here: https://forum.playcanvas.com/t/engine-release-v1-66-0/33297/3

The previous method of loading glslang with a script tag was actually loading a 'loader script' which performed the dynamic import (see here).
The loader script was downloading the actual wasm glue script via CDN (unacceptable) and using a dynamic import statement anyway (though it included the webpackIgnore comment and so worked with webpack).
So it seems we have two options to address this issue:

add /* webpackIgnore: true */ to the import statement and update tooling to keep the comment (by default comments are stripped)
use the string approach found by @LeXXik above

The second option is the simplest and least invasive.
Can anyone test that the following works correctly with webpack?:
        const results = await Promise.all([
            import(`${twgslUrl}`).then(module => twgsl(twgslUrl.replace('.js', '.wasm'))),
            import(`${glslangUrl}`).then(module => module.default())
        ]);

Can anyone test that the following works correctly with webpack?:

Just tested and it definitely works (red: as it is currently, green: with your fix):

Lovely - thank you so much! I'll merge the fix then.

The issue still happens if webpack tries to use prebuilt es5 playcanvas engine (node_modules/playcanvas/build/playcanvas.js) instead the default mjs.
One possible solution is to add noParse option to the module section of your webpack config:
...
module: {
        ...
        noParse: [
            // ignore imports and requires in playcanvas build
            /playcanvas\./
        ],
        ...
}
...

Wow, what a chain of issues rewriting that dynamically added <script> to import, thank you for reporting - it makes sense, since template strings are transpiled.
Can't you use ESM version aswell if you put {type: "module"} in your package.json? Transpiling down to ESM should be easy, while you can't "up-transpile" the ES5/UMD version to ESM.

It's likely this script import will be cause this issue as well.

Can't you use ESM version aswell if you put {type: "module"} in your package.json? Transpiling down to ESM should be easy, while you can't "up-transpile" the ES5/UMD version to ESM.

@kungfooman I use Typescript so using default ESM import works, but my output target is ES5 and I import prebuilt ES5 version of the playcanvas engine to avoid needing to recompile entire playcanvas source code on each update during development and when creating production builds.

We're in the same boat as @pavle-goloskokovic . While we can use ESM in our projects, it would require us to turn on transpilation of dependencies in node_modules ensure that we can still support ES5.
In our case, webpack tries to resolve the dynamic imports at build time and spits out hundreds of errors stemming from files within the ESM-module folder of the playcanvas package.  The noParse trick works well, but we would prefer if the issue could be solved on the playcanvas side rather than requiring downstream projects to use special webpack configurations in order to get the commonjs version to work with webpack.

@marklundin is just cooking this one, any feedback?
#6090


Nothing happens when executing exe file

Everything is in the title.
Btw, windows 10 pro here. I put exe in folder with all videos. When executing .exe the only thing that happens is the brief lost of focus on the exlorer window.

Ok, try v3.1.0
Think this should fix your issue or at least give you a better screen to show me.
Re-open this if it still doesn't work


Trivial - Contributing.MD Updates (replace bitcoin reference with dogecoin) - Issue  fixes #1992

This is mostly a search and replace of the text 'BITCOIN' to 'DOGECOIN". I also removed references to 'BIP' as these are not used for this project.
I have no idea of the model specified in this CONTRIBUTING.MD is valid to Dogecoin, (eg: consensus), so if this needs change just drop a comment and I can update as well.

👎  This is a very deceptive pull request, trying to sneak in feature request #1991 under the guise of a simple readme change for #1992

👎 This is a very deceptive pull request, trying to sneak in feature request #1991 under the guise of a simple readme change for #1992

This was not intentionally deceptive, I was having a hard time understanding which branch was active. I re-based this under 1.21 thanks to @patricklodder guidance

I was going to say, it looks like this has unexpected commits included in it, rather than is intentionally doing anything strange.

@geekwisdom I've changed the base to 1.14.4-dev which where we're actively working (and there's a coming PR to address the branch strategy). Can you rebase to remove the unexpected commits, please?

Rebased and resolved conflicts

its not intuition it's a subanolomy

We've applied a wider re-write, can you try rebasing, we may have covered all your changes already, but if not please let me know. Thank you for all your work, it's appreciated.

Love the new re-write changes, very clear and easy to understand!
Okay, I think I got the 'rebase' correct. I need more practice with rebasing.
If I missed anything just let me know. Tks

Can you please do that for me the same?
On Sun, Jun 6, 2021 at 11:00 AM Geek Wisdom @.***>
wrote:

Love the new re-write changes, very clear and easy to understand!
Okay, I think I got the 'rebase' correct. I need more practice with
rebasing.
If I missed anything just let me know. Tks
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dogecoin/dogecoin/pull/1994#issuecomment-855412740,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUHGK7GXUG6OCGU46YSXGZLTROEPZANCNFSM45L3S2ZQ
.

Ok I have a son and I think he won’t worry.
On Sun, Jun 6, 2021 at 11:12 PM Shaheen Jalilyan @.***>
wrote:

Can you please do that for me the same?
On Sun, Jun 6, 2021 at 11:00 AM Geek Wisdom @.***>
wrote:

Love the new re-write changes, very clear and easy to understand!
Okay, I think I got the 'rebase' correct. I need more practice with
rebasing.
If I missed anything just let me know. Tks
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dogecoin/dogecoin/pull/1994#issuecomment-855412740,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUHGK7GXUG6OCGU46YSXGZLTROEPZANCNFSM45L3S2ZQ
.