When there are multiple gateways, Discord messages don't get sent to Mattermost

Describe the bug
When the matterbridge is running and there are multiple gateways defined in the toml-file,
discord-messages don't show up at the other end (Mattermost for me).
[[samechannelgateway]] still works both ways, as long as there are no more than
one ordinary gateway. Otherwise, those channels all have the same issue.
Also, there's no way to use [[samechannelgateway]] with channels that reside in categories
in Discord.
Environment (please complete the following information):

OS: Debian 10.3
Matterbridge version: 1.16.5 4be00bbe

Additional context
matterbridgeconfig.txt (download)
[discord.mydiscord]

# You can get your token by following the instructions on
# https://github.com/42wim/matterbridge/wiki/Discord-bot-setup
# If you want roles/groups mentions to be shown with names instead of ID,
# you'll need to give your bot the "Manage Roles" permission.
Token="###########################################################"
Server="##########" # picked from guilds the bot is connected to
# UseUserName=true
RemoteNickFormat="*{NICK}*:\t"

[mattermost.mymattermost]
#The mattermost hostname. (do not prefix it with http or https)
Server="localhost:8065"

#the team name as can be seen in the mattermost webinterface URL
#in lowercase, without spaces
Team="######"

#login/pass of your bot.
#Use a dedicated user for this and not your own!
Login="###########"
Password="####################"

RemoteNickFormat="*{NICK}*:\t"
PrefixMessagesWithNick=true

NoTLS=true

# this must always be [[gateway]], don't use anything else
[[gateway]]
name="gateway1"
enable=true

[[gateway.inout]]
account="discord.mydiscord"
channel="###########/###"

[[gateway.inout]]
account="mattermost.mymattermost"
channel="##############"

# this must always be [[gateway]], don't use anything else
[[gateway]]
name="gateway2"
enable=true

[[gateway.inout]]
account="discord.mydiscord"
channel="###############"

[[gateway.inout]]
account="mattermost.mymattermost"
channel="##############"

[[samechannelgateway]] still works both ways, as long as there are no more than
one ordinary gateway. Otherwise, those channels all have the same issue.

What do you mean by "[[samechannelgateway]]"? I can't find anything like that in your config.

Also, there's no way to use [[samechannelgateway]] with channels that reside in categories in Discord.

I notice for the discord gateway you're doing this: "###########/###"
What exactly are you putting here (do you need to censor it? we can't access your guild and it's just an id)

To refer to the channel above you need to put "development", not "mta development/development".
If you have multiple channels with the same name, you can enable Discord Developer Mode, right click the channel you want, click "Copy ID", and as the string put: "ID:<paste_id_here>". So that line would look like channel="ID:366384007535001612" for us.

It's something like this. The string 'art' wouldn't uniquely identify the channel.
Project 2  is a category
[[gateway.inout]]
account="discord.mydiscord"
channel="Project 2/art"

I'll try that id thing tomorrow, that might work. Thanks!

about [[samechannelgateway]]
I tried to do the same thing as the sample config does.
It was the only way to get multiple channels to match
eachother without hitting the bug where Discord channels
aren't reflected to Mattermost.
You can try with or without it, it does not trigger the bug on its own,
as far as I can tell.

If you have multiple channels with the same name, you can enable Discord Developer Mode, right click the channel you want, click "Copy ID", and as the string put: "ID:<paste_id_here>". So that line would look like channel="ID:366384007535001612" for us.

I'll try that id thing tomorrow, that might work. Thanks!

Hmm, no that doesn't help. There's no problem connecting to a Discord channel right now, but... I can not create a Mattermost channel that has a matching name, slashes are not allowed.

@steinbitglis

Project 2 is a category
When using categories you should be able to do channel="Project 2/art"

What are the mattermost channels?
You stripped too much needed information, tokens and passwords are fine to strip but channel names are needed to debug.
Your config with "normal" channel names should work fine.
Also run your setup with -debug flag, type the message on discord and paste the debug here (or pastebin)

I've nailed it down to a name collision between 'general' and 'Team 2/general'.
Putting IDs like you said fixed it.

imo. though, there shouldn't be a name collision, because the strings are different

The solution here would be to resolve the category ones first (deleting from the list), then resolving the non-category names.
And/or supporting /general which would mean "no parent"


MM-36262 Disable buttons on older or published retrospective reminders.

Summary

When a new reminder happens, the buttons on the previous reminder message are disabled. So at any given time, there is only one reminder in-channel for retros that has active buttons.
When someone clicks no retro, it disables the buttons on the original message and creates an entry in the timeline.
When someone clicks start retro, it takes them to that page and nothing happens to the message.
The reminder will repeat at the configured interval according to the playbook until either retro is published or someone chooses to skip it.
When the retro is published, the buttons on the previous reminder messages are disabled.

Disabled messages above and active message below (ignore the <1m)

@abhijit-singh I used the disabled styling from the existing buttons. The hover effect still fires when the buttons are disabled (consistent with other buttons around Playbooks) does that make sense?
Ticket Link
https://mattermost.atlassian.net/browse/MM-36262
Checklist
- [ ] Telemetry updated

[x] Gated by experimental feature flag
- [ ] Unit tests updated

I used the disabled styling from the existing buttons. The hover effect still fires when the buttons are disabled (consistent with other buttons around Playbooks) does that make sense?

@crspeller Disabled buttons should be greyed out according to the design system. I've also seen them work that way in other parts of the product, not with that exact styling always though. And ideally, there shouldn't be a hover effect apart from triggering a tooltip if any.

I remember the UI platform team working on a common button component. Can we start using this?

Fixed so it is grey and no hover effect. Common components are not ready yet. I asked UI Platform and they say they are close.


merge from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>chore(scripts): cover@#overwrite c8 -x 'tests/**/*.mjs' --temp-directory build/tmp ava && c8 report -r lcov -o build/coverage --temp-directory build/tmp
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/*-test.mjs' as in template
chore(package): set $.ava.extensions='mjs' as in template

:tada: This PR is included in version 8.1.82 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[IMP] Make max_sequence_length non optional for HF-Model Class

We should max_sequence_length non optional for HF-Model Class non optional as its needed  here:
https://github.com/rapidsai/crossfit/blob/1ee3de4af3aa543ded1041c2231ad01476b33103/crossfit/backend/torch/hf/model.py#L82-L118
https://github.com/rapidsai/crossfit/blob/1ee3de4af3aa543ded1041c2231ad01476b33103/crossfit/backend/torch/hf/model.py#L82

Another datapoint for this is  issue max_sequence_length for the facebook-opt-123m model, where the model_max_length is set to an incorrect value<PHONE_NUMBER>000000019884624838656) according to the HuggingFace PR. This seems to be a configuration error on HuggingFace's end, not an issue with our code.
We couldn't find a standardized method to determine max_sequence_length after some research. As a workaround, I suggest implementing an optional configuration for max_sequence_length in our semdedup code, defaulting to 512.
For more context:
Discussion on determining max model length on Stack Overflow
Related discussion on HuggingFace

Fixed by:https://github.com/rapidsai/crossfit/commit/0cc29937115f9858272c51e8faffe31155915901


Bug Fix: Always index into squence-like outputs

Some operators are intended to produce 1 or more outputs and will always return a Sequence[torch.Tensor]. Therefore, when len(output_values) == 1, the code breaks since it treated it as a single output tensor. I fix this issue by modifying the condition to index into output.
Include op types are: ['If', 'Loop', 'Scan', 'SequenceMap', 'Split']

@senysenyseny16 the workflows should now all pass. Thanks!


Accessor cache using CeslumAsync::SqliteCache

Addresses #650.
This builds on #634, which needs to be merged first.

I did some benchmarking with this USDA
load_test.zip
results:
First load, clear cache:  74 seconds, 116mb downloaded
Second load, cached: 44 seconds, 0mb downloaded
Restart app, load cached: 44 seconds, 0mb downloaded
Appears to be working as expected.
I do notice a rather large hang (~10-20 seconds) when I hit Reload all Tilesets in Cesium Debugging UI.  This isn't unique to this branch, and I suspect it has something to do with unloading all of the tileset resources before it begins reloaded.
Point is, if we ignore the hang, the actual loading of the cached data goes down to ~24 seconds.

Experimented here with the clear cache functionality, it appears to be working well!
See the results below - times with the red dash were after clearing cache, the rest are a reload from cached data.  Looking quite consistent.

Similar tests with Google Photorealistic 3D tiles

@timoore could you update CHANGES.md?

I added this to CHANGES.md in https://github.com/CesiumGS/cesium-omniverse/pull/696. So no need to update the change log in this PR.


NPM login not possible, if NPM is behind a (apache) reverse proxy with authentication

Checklist

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?

Yes

Are you sure you're not using someone else's docker image?

Yes

Have you searched for similar issues (both open and closed)?

Yes

Describe the bug
I've put the NPM administration (port 81) behind an Apache reverse proxy, with this configuration:
ProxyPass "/" "http://<IP_ADDRESS>:81/" retry=0
ProxyPassReverse "/" "http://<IP_ADDRESS>:81/"
RewriteCond %{HTTP:Upgrade} websocket [NC]
RewriteCond %{HTTP:Connection} upgrade [NC]
RewriteRule ^/(.*) "ws://<IP_ADDRESS>:81/$1" [P,L]

This works fine. I can see the NPM login screen and successfully login after providing my credentials.
However, if I enable authentication in the Apache configuration, it's no longer possible to login:
<Location />
        AuthType Digest
        AuthName "Please login"
        AuthDigestProvider file
        AuthUserFile /some/path/.htdigest
        Require valid-user
</Location>

When I access the site, I have to HTTP (digest) authenticate myself. After that, I see the NPM login screen as expected. However, I am not able to login. When I enter my credentials, NPM just states "unauthorized".
If I remove the Apache authentication, it's possible to login again.
Nginx Proxy Manager Version

v2.9.18
Expected behavior

NPM login should be possible.

I guess it's not possible the way I want. NPM uses the Authorization: Bearer .. HTTP-Header for authorization.
At the same time, Apache uses Authorization: digest .. HTTP-Header.


Can't get position in OnClickListener ?

I am using Draggable & Swipeable RecyclerView in my App. I can't get the position of the item in Onclick. Do you have any examples for it.
This is my current Implementation. Currently i am hardcoding the position value.

onBindViewHolder
holder.mContainer.setOnClickListener(mItemDeleteListener);

 public ExerciseRVItemAdapter(AbstractDataProvider dataProvider) {
        mProvider = dataProvider;

        mItemDeleteListener = new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                mProvider.removeItem(0);
            }
        };

        setHasStableIds(true);
    }

Hi. This library does not provide a special way to handling click events, but you can use RecyclerView.getChildAdapterPosition().
ref.) http://developer.android.com/reference/android/support/v7/widget/RecyclerView.html#getChildAdapterPosition(android.view.View)

Does any of your examples handle on click events? So that i can follow them.

Hi. I do not include simple click event handling in most of demos, because adding more lines of code make sample code more complicated.
I found a good article how to handle item click in RecyclerView.

RecyclerView Part 3: Android RecyclerView OnClick

FYI: This library have a helper class to handle click event of the children views of itemViews more easier.
  void onClickItemViewChildButton(View button) {
      View itemView = RecyclerViewAdapterUtils.getParentViewHolderItemView(button);
      RecyclerView recyclerView = RecyclerViewAdapterUtils.getParentRecyclerView(itemView);
      int position = recyclerView.getChildAdapterPosition(itemView);

      if (position != RecyclerView.NO_POSITION) {
          // your code here.
      }
  }

Actual usage in demo app:

https://github.com/h6ah4i/android-advancedrecyclerview/blob/0.8.7/example/src/main/java/com/h6ah4i/android/example/advrecyclerview/demo_s_button/SwipeableWIthButtonExampleAdapter.java#L110

There have a solution to get position when onitemclick:

implements View.OnClickListener in your adapter

    @Override
      public void onClick(View v) {
          if (mItemClickListener != null) {
              mItemClickListener.onItemClick(v, groupPosition);
          }
      }

2    . In onBindGroupViewHolder
 holder.v.setOnClickListener(this);
3      .In your adapter
```
   public void SetOnItemClickListener(final OnItemClickListener mItemClickListener) {
            this.mItemClickListener = mItemClickListener;
        }

    public interface OnItemClickListener {
        void onItemClick(View view, int position);
    }

initialization

MyAdapter myAdapter = new MyAdapter();
        myAdapter.SetOnItemClickListener(new MyAdapter.OnItemClickListener() {
            @Override
            public void onItemClick(View view, int position) {
               Log.d("testposition",position+"");

            }
        });


feat: add rollup options.

Hello.
Our team requested to use an automatic import feature like Nuxt.
To use unplugin-auto-import ,  unplugin-vue-components   and alias settings were also required.
I can set rollupOptions, so please check it.
Created with reference to @11ty/eleventy-plugin-vue.

Thank you for submitting this pull request.
Unfortunately, I don’t have the time anymore to maintain this project and I would've archived it in the near future anyways.
My suggestion would be that you make the necessary changes in your fork and publish that to your npm account. So you can control what and when you want to release on your own.

Thank you for explaining.


cargo-recursive-clean panics when encountering unwalkable directory

Failed to read entry: WalkError { depth: 8, kind: Io { path: Some("< ... snip ... >"), error: Os { code: 13, kind: PermissionDenied, message: "Permission denied" } } }

This should display a warning and continue execution instead of panicking

Thanks for reporting the issue.
It should be fixed in version 0.2.310, when encountering a error during walk it now prints the error, not panicking.


Any way of capturing higher resolution data?

AFAIK, the Zipcode query will return the zone at the center (centroid?) of the Zipcode area. For Zipcodes that cover areas with steep slopes (or have other features that create microclimate variations), this ends up not being very useful.
Is there any way to query for specific lat/long coordinates instead, and construct a higher resolution dataset?

Not that I know about. The USDA's web query system only permits queries by ZIP, and that's the API that's being used to extract this data.

I've been digging in to see if I could find a workaround, but:

http://planthardiness.ars.usda.gov/ArcGIS/rest/services/uspz/MapServer/generatekml only returns a low-res image layer
However, the interactive map makes other calls to populate the popup for a particular point: http://planthardiness.ars.usda.gov/PHZMWeb/InteractiveMap.aspx

So, after some URL simplification, what do you make of this URL?:
http://planthardiness.ars.usda.gov/ArcGIS/rest/services/uspz/MapServer/identify?geometryType=esriGeometryEnvelope&geometry={"xmin"%3A-13917255.505217%2C"ymin"%3A2760258.9675211%2C+"xmax"%3A+-7399655.505217%2C+"ymax"%3A+6435458.9675211%2C+"spatialReference"%3A{"wkid"%3A102113}}&sr=102113&layers=&time=&layerTimeOptions=&layerdefs=&tolerance=3&mapExtent={"xmin"%3A-13917255.505217%2C"ymin"%3A2760258.9675211%2C+"xmax"%3A+-7399655.505217%2C+"ymax"%3A+6435458.9675211%2C+"spatialReference"%3A{"wkid"%3A102113}}&imageDisplay=1%2C1%2C1&returnGeometry=true&f=json

Oh, that's very interesting. Let's shop this around to some folks who know Esri's RESTful API well enough to weigh in!

(It's funny—I went one level deeper than the website to use their internal API, but it never occurred to me to look one level deeper still. I felt smug enough my initial, minimal accomplishment that I stopped thinking critically.)

I mentioned this to @tmcw and, being @tmcw, he just went ahead and whipped up this solution. :) The next challenge is to either integrate his code (which he's kindly provided under an MIT license) or rewrite it in Python, using Mercantile.

:100: :clap:
Now the code just needs to step through the bounding box at set intervals.
Not sure what the resolution should be if we want to reconstruct shapefiles rather than end up with a huge x,y,zone table.

Yeah, the nice thing about using ZIPs is that I didn't have to consider density, how to limit queries to land, or really anything other than getting a list of ZIPs. I might—God help me—have to actually learn something about working with geodata.

:scream:

Due to #7, we have to use a different method now: #8.


docs: replace weird emoji

It is actually correct. The "weird" one suggest "limited support"

i intend to

i can use this one ✔ instead

fixed


Host Identity server in the same application with a client

In Identity server 3 there is an option to host the server on the same application with a client. According to the official example which is here we can add a piece of code that looks like this
app.Map("/identity", idsrvApp =>
    {
        idsrvApp.UseIdentityServer(new IdentityServerOptions
            {
                SiteName = "Embedded IdentityServer",
                SigningCertificate = LoadCertificate(),
                Factory = new IdentityServerServiceFactory()
                    .UseInMemoryUsers(Users.Get())
                    .UseInMemoryClients(Clients.Get())
                    .UseInMemoryScopes(StandardScopes.All)
        });
    });

And then when we navigate to the _http://myapplication.com/identity_ we could see some info about the Identity server.
Is there something similar in Identity Server 4? I guess that this must be a configuration in IdentityServerOptions class as now we just have to do
app.UseIdentityServer()

inside Configure method.

I'm not sure I follow -- are you just asking about the "welcome" page?

The scenario I am interesting in is to host the server and the client under the same application and not have the server in a separate one. In that case I am not sure how the configuration in the server should look like. In case of Identity server 3 the code on the first comment does the trick.
Is it a bit more clear now?

So this is technically possible -- i've never tried it. But you won't see anything by default in IdSvr4 if you navigate to the base path where you have IdSvr. You could inspect ~/.well-known/openid-configuration to see if the metadata is there.

Yes that makes the trick. I am getting back a json response that looks like
{
  "issuer":"http://localhost:5000",
  "jwks_uri":"http://localhost:5000/.well-known/openid-configuration/jwks",
  "authorization_endpoint":"http://localhost:5000/connect/authorize",
  "token_endpoint":"http://localhost:5000/connect/token",
  "userinfo_endpoint":"http://localhost:5000/connect/userinfo"
 ...
}

So I guess everything is in place. Thanks a lot

np

How did you solved this issue?

Where did you specify "/identity"?


[BUG] Types do not resolve properly in an ESM TypeScript project

What is the bug?
The type declarations do not work when using ESM. I am getting the following error
Could not find a declaration file for module '@opensearch-project/opensearch'. '/Users/samgoodwin/workspaces/eventual-live/node_modules/.pnpm/@opensearch-project+opensearch<EMAIL_ADDRESS>implicitly has an 'any' type.
  There are types at '/Users/samgoodwin/workspaces/eventual-live/packages/<EMAIL_ADDRESS>but this result could not be resolved when respecting package.json "exports". The '@opensearch-project/opensearch' library may need to update its package.json or typings.ts(7016)

Looks like there is a bug with the exports configuration for ESM support. This is the most important part:
There are types at /(my-workspace)/packages/<EMAIL_ADDRESS>but this result could not be resolved when respecting package.json "exports". The '@opensearch-project/opensearch library may need to update its package.json or typings.

How can one reproduce the bug?
Consume this package from a typescript project with the following tsconfig settings:
"module": "esnext",
"moduleResolution": "NodeNext",

What is the expected behavior?
There should be no error when importing this package in a TypeScript project using ESM.
What is your host/environment?
macOS 13.1 (22C65)

Typescript docs say that the "types" export condition should be before all others and that ESM and CJS should have their own .d.(?)ts files.
Current
"exports": {
    ".": {
      "require": "./index.js",
      "types": "./index.d.ts",
      "import": "./index.mjs"
    },
    "./aws": "./lib/aws/index.js",
    "./*": "./*"
  },

Expected
"exports": {
    ".": {
      "require": {
          types: "./index.d.js",
          default: "./index.d.js"
      },
      "import": {
          types: "./index.d.mjs",
          default: "./index.mjs"
      },
    },
    "./aws": "./lib/aws/index.js",
    "./*": "./*"
  },

My issue is caused by the broken latest tag in NPM https://github.com/opensearch-project/opensearch-js/issues/521


helm values for a Developer licensed cluster

Hi,
I'm trying to setup a Confluent cluster in a non-enterprise configuration.
I've tried different options, changing both the number of kafka broker replicas to 1 and the kafka image itself, but everything seems to end in a crashLoopBackoff of the pods.
What I did here was to apply the helm values taken from this repo, and

disable the control center
set kafka image on cp-kafka with 1 replica

Applied with this command:
    helm install -f confluent-cp-helm-values.yaml confluent confluentinc/cp-helm-charts --namespace confluent
## ------------------------------------------------------
## Zookeeper
## ------------------------------------------------------
cp-zookeeper:
  enabled: true
  servers: 3
  image: confluentinc/cp-zookeeper
  imageTag: 6.2.0
  heapOptions: "-Xms512M -Xmx512M"
  persistence:
    enabled: true
    dataDirSize: 10Gi
    dataDirStorageClass: "oci-bv"
snaphots.
    dataLogDirSize: 10Gi
    dataLogDirStorageClass: "oci-bv"
  securityContext:
    runAsUser: 0
  resources: {}
## ------------------------------------------------------
## Kafka
## ------------------------------------------------------
cp-kafka:
  enabled: true
  brokers: 1                        # NO MORE THAN ONE REPLICAS IS ALLOWED FOR DEVELOPER LICENSE
  image: confluentinc/cp-kafka
  imageTag: 6.2.0
  heapOptions: "-Xms512M -Xmx512M"
  persistence:
    enabled: true
    storageClass: "oci-bv"
    size: 5Gi
    disksPerBroker: 1
  resources: {}
  securityContext:
    runAsUser: 0

## ------------------------------------------------------
## Schema Registry
## ------------------------------------------------------
cp-schema-registry:
  enabled: true
  image: confluentinc/cp-schema-registry
  imageTag: 6.2.0
  imagePullSecrets:
  #  - name: "regcred"
  heapOptions: "-Xms512M -Xmx512M"
  resources: {}

## ------------------------------------------------------
## REST Proxy
## ------------------------------------------------------
cp-kafka-rest:
  enabled: true
  image: confluentinc/cp-kafka-rest
  imageTag: 6.2.0
  heapOptions: "-Xms512M -Xmx512M"
  resources: {}

## ------------------------------------------------------
## Kafka Connect
## ------------------------------------------------------
cp-kafka-connect:
  enabled: true
  image: confluentinc/cp-kafka-connect
  imageTag: 6.2.0
  heapOptions: "-Xms512M -Xmx512M"
  resources: {}

## ------------------------------------------------------
## KSQL Server
## ------------------------------------------------------
cp-ksql-server:
  enabled: true
  image: confluentinc/cp-ksqldb-server
  imageTag: 6.2.0
  heapOptions: "-Xms512M -Xmx512M"
  ksql:
    headless: false

## ------------------------------------------------------
## Control Center
## ------------------------------------------------------
cp-control-center:
  enabled: false
  image: confluentinc/cp-enterprise-control-center
  imageTag: 6.2.0
  heapOptions: "-Xms512M -Xmx512M"
  resources: {}

It seems they can't see each other, for example trying to reach "confluent-cp-kafka-0.confluent-cp-kafka-headless.confluent:9092" the kafka broker fails.

Instead using cp-enterprise-kafka, the pods restarts seem lees frequent and due to timeouts:
""
ERROR Failed to start KSQL (io.confluent.ksql.rest.server.KsqlServerMain)
org.apache.kafka.common.errors.TimeoutException: Failed to get offsets by times in 30000ms
""

cp-helm-charts are not designed to be used with Kafka nodes less than 3

So how are we supposed to use the community licensed cluster?
Il giorno lun 26 lug 2021 alle ore 15:26 Viktor Gamov <
@.***> ha scritto:

cp-helm-charts are not designed to be used with Kafka nodes less than 3
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/confluentinc/cp-helm-charts/issues/545#issuecomment-886702438,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB72DNNXPBYQA33LNBXO7OTTZVO77ANCNFSM5AC7ASSQ
.

@tuxerrante don;t use cp-enterprise-kafka, cp-server and cp-control-center images. for Kafka use cp-kafka. you don't need a license to use the confluent community. hope this helps.

@tuxerrante don;t use cp-enterprise-kafka, cp-server and cp-control-center images. for Kafka use cp-kafka. you don't need a license to use the confluent community. hope this helps.

Sorry not sure I've understood, you've mentioned all the possible alternatives.
How should be built a complete Community Confluent cluster?
This seems to work:
confluentinc/cp-zookeeper: as many replicas as the nodes
confluentinc/cp-enterprise-kafka: 1 broker with replication factor ovveride to 1
confluentinc/cp-schema-registry: enabled
confluentinc/cp-kafka-rest: enabled
confluentinc/cp-kafka-connect: with replication.factor = 1
confluentinc/cp-ksqldb-server: 1 replica
confluentinc/cp-enterprise-control-center: disabled

confluentinc/cp-enterprise-kafka: 1 broker with replication factor ovveride to 1

Note: This is still an enterprise image, as mentioned


No range on type error

assume type ident
assume val text_of_id  : ident -> string

noeq
type t = { tm : ident * int }

let rec f e : bool =
    match e.tm with
    | (id, _) when text_of_id = "blah" -> false (* forgot to apply to id *)
    | _ -> true

Gives:
(Error 54) Prims.string is not equal to the expected type _: FStar.Ident.ident -> FStar.All.ML Prims.string

without any location.

Related, string literals seem to only carry the position of their ending ":
$ cat Bug1988.fst
module Bug1988

let x : int = "string literal"
$ ./bin/fstar.exe Bug1988.fst
Bug1988.fst(3,29-3,30): (Error 189) Expected expression of type "Prims.int"; got expression ""string literal"" of type "Prims.string"


⚠ Bofrost.at has degraded performance

In d190f56, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1471 ms

Resolved: Bofrost.at performance has improved in 176677c.


scikit-learn compatible API

Is there a reason why the library doesn't offer a scikit-learn compatbile API? A class that can work via the fit_transform() API?

Hi! Thanks for raising this issue, and sorry for the delay in my response.
I'm happy to consider adding an API that's compatible with scikit-learn.
I'm assuming your talking about scikit-learn's estimator and transform APIs (fit, transform, and fit_transform).
Off the top of my head:
We could have versions of preserve_neighbors and preserve_distances that implemented this API. That makes sense to me, because these functions take raw vector data and preprocess it (conceptually, fit). The transform method would actually compute the embedding.
Would that be helpful?

I'm assuming you're talking about scikit-learn's estimator and transform APIs (fit, transform, and fit_transform).

Yep! That's the one! I'm interested in such an API because it might help users in my bulk labelling interface.
In terms of implementation, maybe the neatest way is to add a class, maybe something like:
import pymde
from pymde import PyMDE

component = PyMDE(method="preserve_neighbors", constraint=pymde.Standardized())

If you want to go the extra mile, I may even go as far as having a constraint-parameter as a string and allowing keyword arguments to pass through. That way, if folks want to use GridSearchCV they can still get nice output. Strings/numbers work a bit better in summary tables than Python objects. But I think just having a scikit-learn compatible class, even if it's just using standard parameters will also go a long way.

ps. I'm also a huge fan of cvxpy by the way!

Okay, great! I'd love for PyMDE to be useful for bulk, which looks awesome, by the way.
Thanks for the code snippet --- something like that could definitely work. I'll put something together in the coming weeks.


Composer problem

Hi!
How can install with composer?
Thanks!

I add
   "repositories": [         {             "type": "vcs",             "url": "https://github.com/alexwight/php-signalr-client"         }     ],
to composer.json and I get to work
Just a message about

Package react/socket-client is abandoned, you should avoid using it. Use react/socket instead.

Thanks!

Ah, I hadn't added it to packagist as I just used this for my own project and didn't get around to writing any documentation yet.  You can now just
composer require alexwight/php-signalr-client


Integrated CCu3 WebServer does not redirect requests to react router

for nginx you need to do this:
https://www.barrydobson.com/post/react-router-nginx/

if you reload any other additional path then /addons/mui/ you will get at 404
like if you refresh the browser on this path => /addons/mui/rooms/


Keeping version information in table.md.

In #7, I added some version information to table.md, which could be useful later.
Is there any sane way to add this automatically in report_to_table.py? I pulled the versions by doing a bash loop over all the envs, source activating, calling python -V and pip list, and scanning over the output**. Putting a bunch of subprocess calls in the script is probably going to provide more headache than benefit though. May as well just do it manually by that point.
Is there any way to extract version info from a virtualenv without actually being in it? I  guess we could os.walk() and get extract it from the directory names under site-packages, but that's arguably worse than subprocessing bash.
** Incidentally it looked like a handful of environments had broken pips after running all the nox sessions, but I got to look into that more later.

@swolebro since the nox envs all install latest setuptools and pip, it's probably okay to just spit out the latest version numbers into the table.


Created environment is not shown in UI

Problem: after successful creation of environment from mongo template with one container, environment does not appear in UI, both in Env-t list and Console

karaf.log.txt

cannot reproduce

Fixed


Clarify documentation of pattern formatting when finding single/multiple git config repositories

The part of the documentation about 'Git backend' gives example of single and multiple git repos:

In the above example, if {application} does not match to any of the patterns, it will use the default uri defined under "spring.cloud.config.server.git.uri". For the "simple" repository, the pattern is "simple" (i.e. it only matches one application). The pattern format is a comma-separated list of application names with wildcards.
The paragraph above created some confusion and I'd like to see a bit more clarification would help future users:

For the "simple" repository, the pattern is "simple" (i.e. it only matches one application). is very confusing. My gotcha was that "simple" isn't a repository as the doc says, but an application (as specified in spring.application.name of config client).
The second thing that threw me off was /pattern1 in the uri. I thought there is supposed to be some patterning involved, when it was none. The right hand side is just an address of git repo where config for "simple" application live. Maybe you could rephrase it as: To define config repo for exactly one application use one-liner specification "{application}: https://github.com/anyname-repo". To define config repo for one or more applications use multi-line specification involving wildcards "pattern: pattern*,*pattern1*".
Third gotcha is that you cannot list anything below simple one-liner specification. E.g.

But then the question is - how should I provide github credentials and where? The doc doesn't talk about that.
Last is specification of wildcard locations within patterns. it wasn't clear to me that setting a pattern as pattern: *data or pattern:* wouldn't work unless i wrap it in double quotes. E.g. this doesn't work:

But this does work:

What is it about the unquoted "pattern" that doesn't work? I'd expect those last two samples are identical.

@dsyer It doesn't like the first star (*) if not provided inside of the quotes. Below is the error I get if I don't wrap it in double quotes:

Fine, so it's just invalid YAML. I consider the error message documentation enough but I added a line in the user guide FWIW anyway.

Thanks for doing that!


Error in comment

In javascript (and many others), this is a common way to declare some documentation for a certain piece of code:
/**
 * Comment
 */

This exemple uses one space indentation, so if you set into your .editorconfig
indent_size = 2
and then you run eclint check on that exemple, it will give you this error:
invalid indent size: 1, expected: 2
I think it should be a way to ignore comments

This very same issue has been reported before in https://github.com/jedmao/eclint/issues/39 and perhaps others.


added cloning of frameBufferScale

Fix for issue #134

Thanks!
Just one thing: use put instead =, otherwise the same instance will be shared by two different variables

ah... cool (not really a kotlin coder :P) Anyway I guess the cmdLists.addAll on cmdList also shares instances.

Thanks!
That's true, I added a TODO to keep an eye on it.. nice catch


Update de_DE.trn

Changed many translations and hopefully improved them. Half of them were spelling mistakes, the other ones were mostly grammar mistakes. I changed "Klon" to "Kopie", because it was named like this already at some translations, I hope I didn't messed it up. Changed "Bewohner" to "Dorfbewohner" because this is more correct (http://minecraft-de.gamepedia.com/Dorfbewohner). Changed "Kennzeichen" to "Tags". Changed to formal speech (were it wasn't) because the first half of the translation file was like this ("du" -> "Sie").
In line 1580 (t737) is a space "{0: n}", I am not quite sure if this is intended. And there are some translation missing, for example the update text and may I suggest using different translation files for the inbuild filters?

Awwww
I just started today to add more strings and edit bad Translations.
I also didnt like some strings.

Oh I am sorry. I hope I changed the translations in a good way

They are in a pretty good way!
Thanks!

Thank you for the update!
Having per filter/brush translations mean have more files to maintain in more folders.
Some strings in the actual translations may not be used anymore. A clean up will come.
In the string 737 '{0:n}' is used (no space), I'll change this.
Which strings are not translated?

At least the "New version of MCEdit ..." text has no translation, even though the buttons from this message have one

If you talk about the popup which warns people about a new available version, it is translated...
But there's maybe an error in the German resource.

I didn't find the update translation, is there a way to trigger this message?
Apparently the "Ctrl-S" in the text in the left corner when you changed something is not translated too, but I don't know really why, the translation exists as "Ctrl" = "Strg" and "Ctrl-" = "Strg-", but maybe it is actually "Ctrl-{0}"

'Ctrl-{0}' does not exists in MCEdit.
I looked at the strings which need the 'Ctrl-' prefix, and found they are hardcoded.
We'll have a look at this.
The translations are now updated. (It happens that Internet can have some lag...)

The update notification is translated I just checked today.
They are all single Strings like:
CTRL+S
CTRL+D
CTRL+B
...
Thats why they are not there.
If the system would be changed to "CTRL+{0}" it would be added.

Github is getting ddosed since 2 days :/
status.github.com

Using formated strings is a possible solution.
The fact is we'll have to track this type of strings, and make sure changing them dosn't break anything...

List of missing translations (addition to the already mentioned ones):

Click the mouse button again to place the other selection corner. (Box selection)
Stop recording
Currently recording a macro
Add macro
Edit NBT (change player data)
Enter a Player Name: (create new player)

Note that the text in brackets only says where you will find the text, it is not part of the actual text


Add docs and examples

What would you like to be added:
Easy to understand documentation and code examples
Why is this needed:
Because I and 2 pepole I asked cant understand this docs.
We can't even understand how to connect to API.
Goog docs examle PyTelegramBotApi

Pydora is just a python wrapper around the Pandora JSON v5 API, which is mostly documented. I had intended the pydora package to be a little example of how to use the library. Specifically PlayerApp is wirth a read.
I'll keep this open as a to-do to add more docs. But to get you started here's an overview:
The APIClient is how you talk to the Pandora API. It needs some configuration and other stuff, you can make these yourself, but the easiest way is to use a client builder of which the most straightforward to use is SettingsDict which just takes a bit of configuration and returns the configured APIClient. Once you have that you need to call the login method with the user's credentials and you're ready to go.
The APIClient methods all return Python models that map pretty much directly to the underlying responses from Pandora. Some models have a little bit of glue to do things like parse dates into python datetime objects and to call other APIClient methods (for example: a PlaylistItem can add thumbs up and down feedback for the track).
So with all that being said, here's a super minimal worked example to load a track (you'll need to pass the track URL, which points to an mp3 or aac file, off to something that can play it, like VLC; this API doesn't actually handle turning audio data into sounds):
from pandora.clientbuilder import SettingsDictBuilder

def play_music(url):
    # You'll need to implement this to do something like call VLC with the track to play it
    pass

# You will need to read the README, it has a link to a page that contains the values
# required for the below dictionary. You can copy/paste them to replace see_link_above
client = SettingsDictBuilder({
    "DECRYPTION_KEY": "see_link_above",
    "ENCRYPTION_KEY": "see_link_above",
    "PARTNER_USER": "see_link_above",
    "PARTNER_PASSWORD": "see_link_above",
    "DEVICE": "see_link_above",
}).build()

client.login("your-pandora-username", "your-pandora-password")

stations = client.get_station_list()
first_station_playlist = stations[0].get_playlist()
first_track = first_station_playlist[0]

print(f"Now playing {first_track.song_name} by {first_track.artist_name} on album {first_track.album_name}")

play_music(first_track.audio_url)

So that's the most simple application you can build with Pydora but you really should read the code over in the pydora player app since it uses a lot of the Pandora playback API and builds out a command-line player. There are also examples of how to control an audio output sync like VLC.
Until I have time for more docs, feel free to ask questions here.

Do you know something about "music.getTrack", it is bot in library and there is no description on rest api docs.
How to get more accurate search results via client.search?

1. Do you know something about "music.getTrack", it is not in library and there is no description on rest api docs.

I don't know what that API is or what it does. How did you find out about it? Why do you think you need to use it?
Not all of the API has been discovered. The calls in the documentation are what the players up to this point have needed. In the past when I've discovered new APIs or gained a better understanding of them I submitted pull requests to the docs and implemented the API call in Pydora. The maintainer of that repo is pretty responsive.

2. How to get more accurate search results via client.search?

As far as I know this is completely under the control of Pandora's server side code. There's nothing in the client you can do for better search results. Do you have reason to believe that the search results from Pydora are not the same as the ones returned from the official apps?


CRYP-130: System tests for transaction list page

I added the system tests for the transaction list page

@jason-gerard  should I change my system test to make sure the first 10 transactions of a wallet are present and then test for 3 transactions (1 for each wallet) test the actual information inside of it?

@jason-gerard  should I change my system test to make sure the first 10 transactions of a wallet are present and then test for 3 transactions (1 for each wallet) test the actual information inside of it?

Probably would be better, but up to you, because we are making sure what we expect is there in a lower level not just that our for loop printed the right number of things if that makes sense.

Looking back on it, I think testing the transactions details in the transaction details page makes more sense. For this test I think we can leave it as is for now and refactor in future sprints.


HBASE-24618 Backport HBASE-21204 (NPE when scan raw DELETE_FAMILY_VERSION and codec is not set) to branch-1

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
7m  2s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
prototool
0m  1s
prototool was not available.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ branch-1 Compile Tests _

+0 :ok:
mvndep
2m 33s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
8m 15s
branch-1 passed

+1 :green_heart:
compile
1m  5s
branch-1 passed with JDK v1.8.0_252

+1 :green_heart:
compile
1m 13s
branch-1 passed with JDK v1.7.0_262

+1 :green_heart:
checkstyle
4m  5s
branch-1 passed

+1 :green_heart:
shadedjars
3m  0s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 54s
branch-1 passed with JDK v1.8.0_252

+1 :green_heart:
javadoc
0m 51s
branch-1 passed with JDK v1.7.0_262

+0 :ok:
spotbugs
2m 42s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
5m  7s
branch-1 passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 18s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m 53s
the patch passed

+1 :green_heart:
compile
1m  5s
the patch passed with JDK v1.8.0_252

+1 :green_heart:
cc
1m  5s
the patch passed

+1 :green_heart:
javac
1m  5s
the patch passed

+1 :green_heart:
compile
1m 13s
the patch passed with JDK v1.7.0_262

+1 :green_heart:
cc
1m 13s
the patch passed

+1 :green_heart:
javac
1m 13s
the patch passed

+1 :green_heart:
checkstyle
4m  1s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
2m 46s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
4m 38s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2.

+1 :green_heart:
hbaseprotoc
1m 26s
the patch passed

+1 :green_heart:
javadoc
0m 43s
the patch passed with JDK v1.8.0_252

+1 :green_heart:
javadoc
0m 53s
the patch passed with JDK v1.7.0_262

+1 :green_heart:
findbugs
5m  7s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 28s
hbase-protocol in the patch passed.

+1 :green_heart:
unit
128m  3s
hbase-server in the patch passed.

+1 :green_heart:
asflicense
0m 59s
The patch does not generate ASF License warnings.

191m 38s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1960/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1960

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc prototool

uname
Linux 84a7451b8061 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1960/out/precommit/personality/provided.sh

git revision
branch-1 / d144923

Default Java
1.7.0_262

Multi-JDK versions
/usr/lib/jvm/zulu-8-amd64:1.8.0_252 /usr/lib/jvm/zulu-7-amd64:1.7.0_262

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1960/1/testReport/

Max. process+thread count
4261 (vs. ulimit of 10000)

modules
C: hbase-protocol hbase-server U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1960/1/console

versions
git=1.9.1 maven=3.0.5 findbugs=3.0.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Thanks for the reviews @apurtell @bharathv @virajjasani


added new tests. all pass

Codecov Report

Merging #10 into master will increase coverage by 6.05%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
+ Coverage   81.63%   87.68%   +6.05%
==========================================
  Files           6        6
  Lines         196      203       +7
  Branches       29       30       +1
==========================================
+ Hits          160      178      +18
+ Misses         36       25      -11

Impacted Files
Coverage Δ

contracts/TestOracle.sol
100% <100%> (+50%)
:arrow_up:

contracts/UjoPatronageBadges.sol
96.49% <0%> (+15.78%)
: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 8f8bcd5...81b820d. Read the comment docs.

Coverage worked. Please lint the tests. :)

Tests look good otherwise. Just check some of the spelling typos too. Will merge once linted.

Hey. The js wasn't properly linted in the tests. How are you linting atm? Usually it's the easiest if you have an IDE plugin that shows the issues.
I'm going to merge this for now so that we can deploy a version to unblock others. Please then submit a subsequent PR to fix the linting! :)

Hey @nihar97. Have you fixed the linting on the tests yet? Is it in some other repo/branch?


Offline configuration value

config.GetAsync should work when the IPFS daemon is not running.  It should access the configuration file on the localhost.

This can not work with multple servers on the local host. For example, go-ipfs uses ~/.ipfs and js-ipfs uses ~/.jsipfs.


Further docs clarifications

@HarHarLinks let me know how this reads

maybe you can also incorporate #152 in this PR

yet my suggestion to expand GitLab capabilities stands

I'm going to leave that issue open as a reminder to have split out secrets work.


Attempt to prevent duplicate feedback

This PR attempts to prevent duplicate feedback by adding an additional check after the database transaction is committed.

@ArtOfCode- Are you sure this should be merged? The failures in CI appear to be in the area directly affected by this change and, at least how I'm reading the test output, indicate that the code is not doing the correct thing in all four failing tests. It appears to be doing the same wrong thing in all four tests.
At least if I'm reading the CI test output correctly, the new code is destroying all of the user's feedback (at least in the last 24 hours) when it sees a duplicate, rather than just destroying all the feedback given by the user in the last 24 hours, except the most recent one (i.e. it's not leaving the most recent one).
It doesn't look like we have a test which checks for what the code does when the existing feedback is > 24 hours old, but it would probably be good to add such a test in order to double-check that the code handles that case properly and doesn't have a similar issue.
[Note: The "should not ignore unidentical feedback from the same user" test should be looking for a change of 1, not 2, but it still shows the same problem as the other two tests (i.e. that we end with 0 feedbacks from the user, when there should be 1 feedback remaining).]

@ArtOfCode- There is a bug introduced in this PR.

Cheers both - fix merged


Inject generated precache-manifest directly inside the service worker instead of creating a precache-manifest file.

Feature Request Description:
I'm using workbox-webpack-plugin's GenerateSW. But it creates a precache manifest file resulting to the following code:
/**
 * Welcome to your Workbox-powered service worker!
 *
 * You'll need to register this file in your web app and you should
 * disable HTTP caching for this file too.
 * See https://goo.gl/nhQhGp
 *
 * The rest of the code is auto-generated. Please don't update this file
 * directly; instead, make changes to your Workbox build configuration
 * and re-run your build process.
 * See https://goo.gl/2aRDsh
 */

importScripts("workbox-v3.6.3/workbox-sw.js");
workbox.setConfig({modulePathPrefix: "workbox-v3.6.3"});

importScripts(
  "precache-manifest.4d330e4e1240ee5970dbef2d1c1a1fd7.js"
);

/**
 * The workboxSW.precacheAndRoute() method efficiently caches and responds to
 * requests for URLs in the manifest.
 * See https://goo.gl/S9QRab
 */
self.__precacheManifest = [].concat(self.__precacheManifest || []);
workbox.precaching.suppressWarnings();
workbox.precaching.precacheAndRoute(self.__precacheManifest, {});

What I want is to remove importScripts("precache-manifest.4147d0eae38244e105572db91f23fa97.js"); and totally inject the contents of precache-manifest inside of the service worker resulting to the following code:
/**
 * Welcome to your Workbox-powered service worker!
 *
 * You'll need to register this file in your web app and you should
 * disable HTTP caching for this file too.
 * See https://goo.gl/nhQhGp
 *
 * The rest of the code is auto-generated. Please don't update this file
 * directly; instead, make changes to your Workbox build configuration
 * and re-run your build process.
 * See https://goo.gl/2aRDsh
 */

importScripts("workbox-v3.6.3/workbox-sw.js");
workbox.setConfig({modulePathPrefix: "workbox-v3.6.3"});

self.__precacheManifest = [
  {
    "url": "css/file2.d39cd1945dcec8bba59a.css"
  },
  {
    "url": "js/file.d39cd1945dcec8bba59a.js"
  }
];

/**
 * The workboxSW.precacheAndRoute() method efficiently caches and responds to
 * requests for URLs in the manifest.
 * See https://goo.gl/S9QRab
 */
workbox.precaching.suppressWarnings();
workbox.precaching.precacheAndRoute(self.__precacheManifest, {});

I'm not sure if it already exists, I'm looking at the issues to see if someone else has opened a similar issue but I also couldn't search for one.
Here's my config btw:
new workboxWebpackPlugin.GenerateSW({
  swDest: 'serviceWorker.js',
  importWorkboxFrom: 'local',
  excludeChunks: ['runtime', 'main.css']
})

Splitting up the manifest into a separate asset was a deliberate design decision made when we rolled out workbox-webpack-plugin, and is a different approach from what earlier tools, like sw-precache-webpack-plugin used.
So, I mean, it's conceptually possible, but folks from the community using webpack seemed to want it the other way around, which motivated the choice in Workbox.
Is the current approach breaking anything for you? Or is this just a matter of personal preference?

It's breaking updates. When I release new updates, the updates does reflect on the site immediately. In fact, most of the time I have to clear my cache manually.

That's the expected behavior of using precaching—the new resources won't be shown immediately, since the old resources are loaded in a cache-first manner.
I think what you're seeing is independent from the question of whether or not the precache manifest is inlined vs. imported.
There's some discussion of the benefits of precaching here, and UI patterns for prompting users to update can be found here.
If you're not satisfied with the cache-first behavior that comes with precaching, using a runtime caching strategy for your assets is another option.
I'm inclined to close this issue—hopefully the above links give you the context you need to decide whether or not to use precaching. If you're seeing buggy behavior or something along those lines, we can reopen.

Maybe it should do cache-first when the files required are actually up-to-date on the cache. I am already prompting the user when a new version is available, so he can reload the page. Even that takes some time to appear and after reloading the page after that appears, the update still does not reflect immediately. Especially on installed PWAs. I literally have to uninstall and add to homescreen again just to get the updated version of the site. Can't workbox precache route and do staleWhileRevalidate? I assume it already knows how to do that since the precache-manifest would give it the new files which would have different hash names.

There are a number of trade-offs. Since it's directly relevant to your feedback, and since folks might not click through to the Stack Overflow answer, here's what I've previously written about precaching:

Using stale-while-revalidate for all of your assets, as well as for your HTML, is definitely a legitimate approach. It means that you don't have to do anything special as part of your build process, for instance, which could be nice in some scenarios.
Whenever you're using a strategy that reads from the cache, whether it's via precaching or stale-while-revalidate, there's going to be some sort of revalidation step to ensure that you don't end up serving out of date responses indefinitely.
If you use Workbox's precaching, that revalidation is efficient, in that the browser only needs to make a single request for your generated service-worker.js file, and that response serves as the source of truth for whether anything precached actually changed. Assuming your precached assets don't change that frequently, the majority of the time your service-worker.js will be identical to the last time it was retrieved, and there won't be any further bandwidth or CPU cycles used on updating.
If you use runtime caching with a stale-while-revalidate policy for everything, then that "while-revalidate" step happens for each and every response. You'll get the "stale" response back to the page almost immediately, so your overall performance should still be good, but you're incurring extra requests made by your service worker "in the background" to re-fetch each URL, and update the cache. There's an increase in bandwidth and CPU cycles used in this approach.
Apart from using additional resources, another reason you might prefer precaching to stale-while-revalidate is that you can populate your full cache ahead of time, without having to wait for the first time they're accessed. If there are certain assets that are only used on a subsection of your web app, and you'd like those assets to be cached ahead of time, that would be trickier to do if you're only doing runtime caching.
And one more advantage offered by precaching is that it will update your cache en masse. This helps avoid scenarios where, e.g., one JavaScript file was updated by virtue of being requested on a previous page, but then when you navigate to the next page, the newer JavaScript isn't compatible with the DOM provided by your stale HTML. Precaching everything reduces the chances of these versioning mismatches from happening. (Especially if you do not enable skipWaiting.)

You write:

Especially on installed PWAs. I literally have to uninstall and add to homescreen again just to get the updated version of the site.

That's definitely not the expected behavior. Can you share a URL for your deployed web app and we can take a look as to whether anything stands out? (Again, I don't believe that this has anything to do with the question of an inlined precache manifest vs. a separate imported file.)

Sadly not yet, it's not in production environment yet but only in pre-production. That means only office network can access it. My guess is that the new service worker which is suppose to serve the new precache manifest is not being registered. Maybe on this line register('serviceWorker.js'), the serviceWorker.js may be being served from cache which means the new service worker wouldn't be registered. I'm not sure yet since it's quite hard to investigate on mobile phones.
However, I'll try to create a repo that reproduces the observed effect I told you about.

I also think that option for injecting precache manifest inline in service worker would be a good option.
In my case on production with every deployment the javascript bundles are changing and so is changing precache manifest file, because new bundles are showing up etc.
The problem is that users have cached service worker in their browsers and their registered service workers are requesting previous precache manifest files wich doesn't exist anymore on the server and it ends up with lot of 404 responses from server.
So inline option for me would be a great thing. Also maybe there is a way to generate inject manifest file without manifestHash.
We serve assets in folders like that (assets/1544088797202/client.js) and also precache manifest is served from assets/1544088797202/precache-manifest.[manifestHash].js so I don't see the need of double hashing this asset (in folder and in filename). Unfortunately workbox-webpack-plugin is not allowing to generate precache manifest file without manifestHash in filename.

@wariatroku7 using skipWaiting: true and clientsClaim: true should fix your problem.

I am seeing this issue with InjectManifest class in<EMAIL_ADDRESS>Despite providing a swSrc, empty array in .precacheAndRoute([]) is not being replaced.
            new InjectManifest({
                swDest: './sw.js',
                swSrc: './src/serviceworker/sw.js'
              })

definitely dont want to make another network call to get precache-manifest.

I have a scenario which makes me very inconvenient. My precache-manifest.js gets deployed to a different domain. Basically, I am unable to transform the arrayself.__precacheManifest at runtime with workbox-webpack-plugin for a different domain.
There are 2 workarounds

Use the CLI instead of workbox-webpack-plugin
Or put precache-manifest.js next to sw.js.

I also think that option for injecting precache manifest inline in service worker would be a good option.
In my case on production with every deployment the javascript bundles are changing and so is changing precache manifest file, because new bundles are showing up etc.
The problem is that users have cached service worker in their browsers and their registered service workers are requesting previous precache manifest files wich doesn't exist anymore on the server and it ends up with lot of 404 responses from server.
So inline option for me would be a great thing. Also maybe there is a way to generate inject manifest file without manifestHash.
We serve assets in folders like that (assets/1544088797202/client.js) and also precache manifest is served from assets/1544088797202/precache-manifest.[manifestHash].js so I don't see the need of double hashing this asset (in folder and in filename). Unfortunately workbox-webpack-plugin is not allowing to generate precache manifest file without manifestHash in filename.

In Workbox v5's GenerateSW mode, the precache manifest is inlined into the top-level service worker by default.
However, by default in v5, the Workbox runtime code is split up into a separate chunk and imported via importScripts(). So you will still end up with two files.
But there is an option in v5 to change this if you really want everything in a single file: inlineWorkboxRuntime: true.

This should be addressed by the current Workbox v5.0.0 alpha.


Charge cell data with SQLite Android

How to code for communicating tableview with SQLite?

Example:
the user save the parameter of cell for later charge in a table with SQLite to continue the work.

Hi @bbf-developer
As far as I can understand your request, you want to show your SQLite data on a TableView. Am I right?
If I am, there is no any direct connection by SQLite DB to TableView. Unfortunately, you should build your own.

Yes, i develop the method directly into a App, only i need where starting, thanks a lot @evrencoskun

@bbf-developer , can we close this issue?


refactor: fs manager

https://github.com/apache/incubator-pegasus/issues/1383

Support IO_ERROR status for disk
remove get_disk_tag and replica::init_disk_tag(), _disk_status and _disk_tag of replica
move codes from replica_disk_migrator.cpp to fs_manager.cpp
remove some commented code

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e4350d4). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1473   +/-   ##
=========================================
  Coverage          ?   53.74%
=========================================
  Files             ?       27
  Lines             ?     2538
  Branches          ?        0
=========================================
  Hits              ?     1364
  Misses            ?     1128
  Partials          ?       46

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Write a JavaScript program to print ascii value of a character

Description
Write a JavaScript program to print ascii value of a character
ASCII value is the numeric representation of a character. For example, the ASCII value of the character 'A' is 65. The ASCII value of the character 'a' is 97. The ASCII value of the character '0' is 48. The ASCII value of the character ' ' is 32.
Input  : A
Output : 65

How to contribute

Save the solution in program/PrintAsciiValueOfACharacter.js file
Add PrintAsciiValueOfACharacter.js file in print-ascii-value-of-a-character folder

!assign


initReports

You also need to add the permissions to your views. For now, could do it like Yahya showed us, isOwnerOrReadOnly

good notes Suhaib, but first you need to specify the difference the Student Report you merged before and this report you added to the requirements sheet


AI-generated Sonarqube issue remediation: remediation_branch-2024-10-24_01-53 -> main

From 21 remediated issues 2 require additional actions

Issue Key
Message
Action

86c60264-d624-4584-b9fb-b9d0cd74697f
Make sure this MySQL database password gets changed and removed from the code.
Ensure that the environment variable MYSQL_PASSWORD is set in the deployment environment to avoid runtime errors.

b8d0df50-cc43-4f46-aa5b-021e668a2493
Rename field "vets"
Verify that all references to the 'vets' field in the Vets class have been updated to 'vetList' in the codebase.

@CodiumAI-Agent /review

PR Reviewer Guide 🔍
Here are some key observations to aid the review process:

⏱ Estimated effort to review: 3 🔵🔵🔵⚪⚪
🧪 PR contains tests
🔒 No security concerns identified
⚡ Recommended focus areas for review
CSS AdjustmentThe change in vertical alignment from 0.255em to 0 might affect the visual layout. Verify that this change aligns with the intended design specifications.
Logic SimplificationThe logic for checking pet ID has been simplified. Ensure that this refactoring does not alter the expected functionality, especially concerning the handling of new vs. existing pets.
Field RenamingThe field 'vets' has been renamed to 'vetList'. Confirm that all references to this field across the project have been updated to prevent runtime errors.
Environment VariableThe MYSQL_PASSWORD is now set from an environment variable instead of a hardcoded value. Ensure that this variable is correctly set in all deployment environments to avoid connection issues.


Enhancing the UI of Intership section for better user experience ✌.

Please include the image in the pull request if assigned.

I would like to work on this issue

please assign me this issue

Sure @ShravaniAK , I will be assigning this issue on 30th sep till than explore the project more and join the contributors chats on our discord from description.

Hello sir, @DeveloperAshish8
I've handled these kind of issues earlier as well thus i can come up with a more fascination solution to the same .Kindly assign me the issue so that i can start working on it,Thanyou!

Hey @ShravaniAK , please attach an image or video in the PR.


Add context

This a compromise on https://github.com/kbst/terraform-provider-kustomize/pull/27 only having the addition of the context, without any change to the fallback logic.

Thank you for moving this to a separate PR. LGTM.


HTTPClient: read_response_body_chunk() breaks for small files

When fetching files smaller than length 4096, a file of length 4096 is returned any way. The resultant file is corrupt.
	} else {
		ByteArray ret;
        /*
        BUG: For small files body_left will be less than tmp_read.size();
        which defaults to 4096.
        */
		ret.resize(MAX(body_left,tmp_read.size()));
		ByteArray::Write w = ret.write();
		int _offset = 0;
		while (body_left > 0) {
			ByteArray::Write r = tmp_read.write();
			int rec=0;
			err = connection->get_partial_data(r.ptr(),MIN(body_left,tmp_read.size()),rec);
			if (rec>0) {
				copymem(w.ptr()+_offset,r.ptr(),rec);
				body_left-=rec;
				_offset += rec;
			}
		}
		if (body_left==0) {
			status=STATUS_CONNECTED;
		}
		return ret

Here is a quick repro of the issue.
Repro script (http_repro.gd) embedded below.
Try:
% godot.x11.tools.64 -s http_repro.gd
opening pack /home/kwesi/Desktop/godot/scripts/data.pck, -1, -1
URL info: www.goodellgroup.com, /tutorial/index.html
Connecting..
Requesting..
END OF RESPONSE? :
HTTP/1.1 200 OK
Content-Length: 2555
Content-Type: text/html
Last-Modified: Mon, 15 Sep 2014 16:52:55 GMT
Accept-Ranges: bytes
ETag: "22b2517f5d1cf1:ddff8"
Server: Microsoft-IIS/6.0
X-Powered-By: ASP.NET
Date: Sat, 10 Jan 2015 18:45:31 GMT
------
Requesting..
response? True
code: 200
**headers:
(Accept-Ranges:bytes), (Content-Length:2555), (Content-Type:text/html), (Date:Sat, 10 Jan 2015 18:45:31 GMT), (ETag:"22b2517f5d1cf1:ddff8"), (Last-Modified:Mon, 15 Sep 2014 16:52:55 GMT), (Server:Microsoft-IIS/6.0), (X-Powered-By:ASP.NET)
Response Length: 2555
bytes got: 4096
...
% tail --lines=5 index.html
<!-- jQuery (necessary for Bootstrap's JavaScript plugins) --><script src="https://ajax.googleapis.com/ajax/libs/jquery/1.11.0/jquery.min.js"></script><!-- Include all compiled plugins (below), or include individual files as needed --><script src="js/bootstrap.min.js"></script></div>
</div>
</body>
</html>
�`��� `����`���`Щ�a0��g��`���a���g�``p���`��������v� `Ы��`0���`���aP��g��`0��a���g��

http_repro.gd:
extends SceneTree

func _init():
    var url = "http://www.goodellgroup.com/tutorial/index.html"
    var host_path = get_host_path_by_url(url)
    print( "URL info: " + str(host_path ))
    get_buffer_by_http(host_path[0], host_path[1])
    quit()

func get_host_path_by_url(url):
    # url: http://myhost.com/path/to/file.scn
    var retval = []
    var http_prefix = "http://"
    if url.begins_with(http_prefix):
        var url_sans_prefix = url.right(http_prefix.length())
        var tokens = url_sans_prefix.split("/")
        if tokens.size() > 1:
            var host = tokens[0]
            var path = url_sans_prefix.right(host.length())
            retval = [host, path]
            return retval
    return retval

func get_buffer_by_http(host, path):
    # COPIED FROM: https://github.com/okamstudio/godot/wiki/http_client

    var err=0
    var http = HTTPClient.new() # Create the Client

    var err = http.connect(host, 80) # Connect to host/port
    assert(err==OK) # Make sure connection was OK

    #Wait until resolved and connected
    while( http.get_status()==HTTPClient.STATUS_CONNECTING or http.get_status()==HTTPClient.STATUS_RESOLVING):
        http.poll()
        print("Connecting..")
        OS.delay_msec(500)

    assert( http.get_status() == HTTPClient.STATUS_CONNECTED ) # Could not connect

    # Some headers
    var headers=[
        "User-Agent: Pirulo/1.0 (Godot)",
        "Accept: */*"
    ]

    err = http.request(HTTPClient.METHOD_GET, path, headers) # Request a page from the site (this one was chunked..)

    assert( err == OK ) # Make sure all is OK

    while (http.get_status() == HTTPClient.STATUS_REQUESTING):
        # Keep polling until the request is going on
        http.poll()
        print("Requesting..")
        OS.delay_msec(500)

    assert( http.get_status() == HTTPClient.STATUS_BODY or http.get_status() == HTTPClient.STATUS_CONNECTED ) # Make sure request finished well.

    print("response? ",http.has_response()) # Site might not have a response.

    if (http.has_response()):
        #If there is a response..

        var headers = http.get_response_headers_as_dictionary() # Get response headers
        print("code: ",http.get_response_code()) # Show response code
        print("**headers:\n",headers) # Show headers

        #Getting the HTTP Body
        var bl = http.get_response_body_length()
        if (http.is_response_chunked()):
            #Does it use chunks?
            print("Respose is Chunked!")
        else:
            #Or just plain Content-Length
            print("Response Length: ",bl)

        #This method works for both anyway

        var rb = RawArray() #array that will hold the data
        var byte_count = 0
        while(http.get_status()==HTTPClient.STATUS_BODY):
            #While there is body left to be read
            http.poll()
            var chunk = http.read_response_body_chunk() # Get a chunk
            if (chunk.size()==0):
                #got nothing, wait for buffers to fill a bit
                OS.delay_usec(1000)
            else:
                rb = rb + chunk # append to read bufer

        #done!

        print("bytes got: ",rb.size())
        var text = rb.get_string_from_ascii()
        print("Text: ",text)

        var file = File.new()
        var WRITE = 2
        file.open(path.get_file(), WRITE)
        file.store_buffer(rb)
        file.close()

This is fixed:
okamstudio@78a268c


Merge pull request #4210 from leather-wallet/release/judas-priest

Try out this version of the Leather — download extension builds.

Release/judas priest

i merged manually


Enable SHORTFIN_ENABLE_TOKENIZERS in Linux package builds

This will allow us to replace the current loosely coupled tokenizers Python package (https://pypi.org/project/tokenizers/) with a source dependency on https://github.com/mlc-ai/tokenizers-cpp that provides bindings to the underlying rust library from https://github.com/huggingface/tokenizers and C++ library from https://github.com/google/sentencepiece.
See also:

https://github.com/nod-ai/shark-ai/issues/130
https://github.com/nod-ai/shark-ai/pull/610

Note the future work items listed there
We'll still need to plumb through a Python API and update usage in files like https://github.com/nod-ai/shark-ai/blob/main/shortfin/python/shortfin_apps/llm/components/tokenizer.py

https://github.com/nod-ai/shark-ai/issues/620

Main tasks for this issue:

[ ] Add rust/cargo to a dockerfile in https://github.com/nod-ai/base-docker-images/
[ ] Switch this script to use that dockerfile: https://github.com/nod-ai/shark-ai/blob/1e26b205287d9a7153639ca0f66e0fcd51c5138e/shortfin/build_tools/build_linux_package.sh#L40-L41
[ ] Set SHORTFIN_ENABLE_TOKENIZERS in that script
[ ] Ensure that https://github.com/nod-ai/shark-ai/blob/main/.github/workflows/build_packages.yml picks up the changes

We also can't use a github action to set up rust on the host machine like the shortfin CI workflow: https://github.com/nod-ai/shark-ai/blob/7e62c25a3f307a9a0e3191fc9ebdd875aacab1fc/.github/workflows/ci-libshortfin.yml#L91-L97
since the Linux package builds are run in a manylinux docker container: https://github.com/nod-ai/shark-ai/blob/7e62c25a3f307a9a0e3191fc9ebdd875aacab1fc/shortfin/build_tools/build_linux_package.sh#L175-L180


Introspection examples script.

This adds a script with some examples of how to do basic micro-manager introspection using pycro-manager.

Great, thanks!


 Thanks for your contribution as always.
I see it emits a lot of lint errors. Perhaps we’d merge it and start submitting PRs to fix each lint error, or start with disabling failing sub-linters, and submit follow-up PRs to fix each group of errors while enabling the linter one-by-one?
My personal preference is the latter, but I'm also interested in your thought.

@mumoshu I agree with you. I will do this laster. Thanks very much.

@mumoshu ping

@mumoshu I fix for the new code.

@yxxhero Thanks 🙏 Really looking forward to get this merged!

@mumoshu ready to review.

@yxxhero You can merge this once CI passes!

@mumoshu yes. Thanks very much.


feat: support mieru protocol

update https://github.com/MetaCubeX/mihomo/pull/1697

I found a problem.
Can "github.com/enfein/mieru/v3/apis/client" not rely on the top-level package "github.com/enfein/mieru/v3/pkg/appctl"? In fact, client only calls appctl's FlatPortBindings and ValidateClientConfigSingleProfile, but this will force downstream projects to rely on the complete "google.golang.org/grpc" and increase the binary size by about 7mb on mainstream platforms.
In my personal test, copying these two functions to a separate subpackage and only importing the subpackage in the client can make mihomo no longer rely on grpc after executing go mod tidy, and greatly reduce the compilation size.


⚠ Bofrost.lu has degraded performance

In 001e471, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2212 ms

Resolved: Bofrost.lu performance has improved in 0cb16ed after 6 minutes.


Added initial catmath tests.

Coverage increased (+2.3%) to 39.08% when pulling e30bd03a329c12e31101cb5e2d4a8530e93f33f0 on apalm112:pycon2016 into c8a14475337df801f7188962a09a7ccd56f30f52 on keeppythonweird:master.

Coverage increased (+2.3%) to 39.08% when pulling bd2d00f8dfa506287be4fbbaee23a99d99178812 on apalm112:pycon2016 into c8a14475337df801f7188962a09a7ccd56f30f52 on keeppythonweird:master.

Coverage increased (+14.9%) to 51.724% when pulling 46203990ac3b3f94ba556140ff69f4b2b8683db4 on apalm112:pycon2016 into c8a14475337df801f7188962a09a7ccd56f30f52 on keeppythonweird:master.


Add Travis

Description

Motivation and Context

How Has This Been Tested?

Checklist:

[ ] I have submitted a CLA form
[ ] If applicable, I have updated the documentation accordingly.
[ ] If applicable, I have added tests to cover my changes.

@youming-lin Looks like it's fails trying to get file permissions on Linux https://travis-ci.org/IBM-Swift/Kitura/builds/265572987


[Bug]: icon is not displayed

Describe the bug
Many icons cannot be displayed

To Reproduce
Displayed immediately in the folder
Expected behavior
I need it to show up
Screenshots

Computer information

OS edition:
Extension version:
VSCode version:

Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I uninstalled the extension, deleted the folder in .vscode/extension and reinstalled it, that solved the problem

I had the same thing happen to me, so I downgraded to version 5.6.0 and the icon showed up again.

Closing as duplicate of #2428


Betterfolder sidebar misplaced

When i use BetterFolders plugin, the sidebar goes under chat/server screen.
 Midnight theme
 Vanilla

I think this is because of the new style where the server list has a background panel

I have pushed a temporary fix. Just to verify, this is on vencord right?

I have pushed a temporary fix. Just to verify, this is on vencord right?

sorry for the late reply. Yes, Vencord and after your push was fixed :D

last issue!


Should package-lock.json be include?

Shouldn't we remove package-lock.json from the commited files?
@mattpap

Though it may be tempting due to its usually large size, package-lock.json is a useful tool, which allows to have reproducible environments. Otherwise npm install would result in always installing most recent packages, up to allowed dependency specification. Note however npm install can still change packge-lock.json and upgrade dependencies to some extent. To force npm to only use the lock file, one should use npm ci. This is the command that should be used in CI systems, as it name actually suggests. To fully update dependencies, the approach that works the best for me, is to remove the lock file and run npm install (rm package-lock.json; npm install; git add package-lock.json). I suppose there may be a more idiomatic way that I'm not aware of.

We should add some of that to the developer docs.


Force 755 permissions on files copied to /usr

If this is the cp -r command,wouldn't it be better to do cp -rp?

Closing in favor of  https://github.com/containers/ai-lab-recipes/pull/462


DUI: Receiving without internet displays no error, just keeps waiting

If you try to receive anything in DUI while having no internet, the stream card will go into its "loading state" but never comes back.

Expected
Ideally, we should retry several times #1688 and if that fails we should cancel and display a warning.

Additions in fix for #1712 should be applicable here too.
We now have a way in Core to check if a user has internet connection by pinging a (mostly always online) website not owned by us.

An error message is now shown if not connected before sending/receiving. Closing this for now!


Bug/none env @function

Issue #, if available:
Updating for comment here https://github.com/aws-amplify/amplify-cli/pull/1346#discussion_r283961308
Description of changes:
Handling NONE env in @function directive.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This was created from the wrong branch. Closing and reopening correct branch.


cloneElement is not working properly

cloneElement is not working for extended classes or stateless react components
Both examples are working in React but only second is working in Preact.
working only in React
class Test extends Component {
}

cloneElement(Test, { param: 111});

working in both React and Preact
class Test extends Component {
}

cloneElement(<Test />, { param: 111});

I know that standard is to return  but this behaviour is inconsistent with react. Maybe it is related to preact-compat

Hmm - AFAIK React is just passthrough for that:
cloneElement(Test) === Test  // true

but preact cloneElement will try to do h(Test.nodeName, ... and nodeName is undefined

Right, I'm not saying Preact is correct here - I think we need to make preact's cloneElement() passthrough like React's is. It also affects Text nodes.  I just wanted to make the point that cloneElement() isn't doing anything in the example.

👍  this is an issue for me as well.
@developit is there another way you suggest adding a prop to a child component? All the examples I'm finding use React.Children and React.cloneElement, e.g.:
https://jaketrent.com/post/send-props-to-children-react/

@paul-craciunoiu-ck Preact supports cloneElement() like React, just use that.  This issue is just tracking support for cloneElement() ignoring (passing through) things it doesn't recognize.

Just an update @seeden - it looks like more recent versions of React actually behave like Preact:

https://jsfiddle.net/developit/xo3Loy8t/
FWIW even with a fix in place, cloneElement(Ctor, { foo: 'bar' }) would just return Ctor.

I fixed my code after this issue with preact. I replaced my code with
return <Test params={111} />

instead of
cloneElement(Test, { params: 111 })
```.

You can close it if you want

@developit ah I figured this out. It's actually the lack of support for synthetic events that was the issue.

Gotch.

is this somewhere related to the fact that below code does not render anything
<TransitionGroup>
  {cloneElement(props.children)}
</TransitionGroup>

@prateekbh cloneElement only works on single elements, and the children prop is always of type array. In your case you likely don't need cloneElement at all.
I just compared our implementation with react's and we both do the same thing, minus the difference with synthetic events.


Duplicate id

There are currently some duplicate ids (e.g. H3371) in the database.
After cleaning, I would suggest adding a unique constraint in the metadata.

Thanks for spotting this, I will update the database soon. I will also add that constraint in the metadata.

Dataset updated!


req.url stripping fragments?

I'm trying to build an application that uses fragments, but I can't get a handle on them in my handler. Is this intentional? I can't see any call/method that would let me get access to fragments as part of a handler.
A really basic example:
use iron::prelude::*;

fn hello_world(req: &mut Request) -> IronResult<Response> {
    println!("got url: {}", req.url);
    Ok(Response::with((iron::status::Ok, "Hello World")))
}

fn main() {
    let chain = Chain::new(hello_world);
    Iron::new(chain).http("localhost:3000").unwrap();
}

When calling GET /#tag=bar or even GET /#?tag=#token=foo, req.url is always /. Is this a limitation of Iron?

Fragment identifiers are not sent to the server.

Figured this out after posting - should have updated the issue :+1:


Security questions

Hi, I have questions gameobject.ngtb must be with the exe file? if so isn't it dangerous? in my opinion, it should be included directly in the release exe, and it should be encrypted using aes or something like that, and I would initialize the aes myself when starting the game or in some other safe way, because at the moment it looks like this, if I saw the game like this, people would they could read both my data and file location

I don't understand how this would create anything more than a slight annoyance for anyone trying to decrypt your software

in this file is for example path to your folder contents of
initialized strings and some other data
2024-01-11 9:01 GMT+01:00, DJ-Wolfy @.***>:

I don't understand how this would create anything more than a slight
annoyance for anyone trying to decrypt your software
--
Reply to this email directly or view it on GitHub:
https://github.com/m1maker/NGT/issues/13#issuecomment-1886571187
You are receiving this because you authored the thread.
Message ID: @.***>


Web Essentials combined RTVS no longer displays markdown preview

This is because top level view is now 'projection' content type and not 'markdown'. I am taking this with Mads, so this is for tracking in case any changes are needed on the RTVS side.

Red herring, markdown editor is now a separate extension


How to find out the polling frequency in version 1.0.2?

Hello. How can I find out the polling frequency of my keyboard using your software? In the early versions of your program, everything was very clear, but in the later versions you introduced graphs, according to which I do not understand what the polling frequency is on my device. Can you improve the interface, add for example a field where a figure would be displayed, at least an approximate or average, of the polling frequency of the USB device???

As development of Keyboard Inspector continued, a lot was learnt about the behaviour of not only keyboards but USB devices in general. It became evident that getting the program to spit out a number as the performance based on some heuristic was impossible since device behaviour is much more complex. Therefore Keyboard Inspector elects to instead present a few different analyses to the user, letting the user understand their device far better than any heuristic ever could. This has downsides, foremost that it raises the bar of entry to using Keyboard Inspector.
That said, a more detailed guide than that currently on github is being written.


[ Question ]

i keep getting these errors

i dont even get chance to add my api and stuff skips that

You are using node. version 19, but we recommend node 16 in our wiki, please consider downgrading it since newer versions of node break the code.

Wich node version are you using now?

16.7 as per the wiki

Could you post the error messages you see after downgrading to 16.7?

It looks like some files are missing. Is it possible you downloaded the source code instead of a release ?

Still same issue, Can someone hop into a dc call and help me?
Jazy#9710

We do not provide support through discord, sorry.
I am unable to reproduce the issue on a fresh install.
I can however reproduce the exact error message by downloading the source code and running the installer there. Please double-check if you have downloaded the release containing the necessary files and not the source code.
If you are unsure what you have downloaded, post a screenshot of the toolkit directory contents :)

this is what is in the file

This looks like the source code; not the release which is ready to run. Please download the latest release and follow the installation instructions.

I had the same issue for ages
Turns out I downloaded a clone of the repository instead of the download.
Click to download the latest version of the program not the repository
https://github.com/RCVolus/league-prod-toolkit/releases/tag/v1.3.7


How to use each chart.options ?

Hello, I'm curious about this library. Thanks for making react version of billboardjs.
I tried to apply pie.options so, I don't apply option.
https://naver.github.io/billboard.js/release/latest/doc/Options.html#.pie
How to apply options to chart component? pie options add in data object and then,
pass on the data object to billboard charts component?
example)

...

pie = {
   innerRadius: 20,
   padAngle: 0.2
}

render() {
	return <BillboardChart data={this.chartData} size={this.SIZE} pie={this.pie} />
}

Pie props don't working, maybe I did mistake ... ?

As mentioned in the README, all top-level properties for options are passable as props to the component. Therefore, to pass pie options, you would pass the pie prop:
import BillboardChart from 'react-billboardjs';

const PIE_OPTIONS = {
  label: {
    threshold: 0.1,
    ratio: 0.5,
  },
  padAngle: 0.25,
};

export const MyComponent = ({json}) => {
  const data = {
    json,
    type: 'pie',
  };

  return (
    <BillboardChart
      data={data}
      pie={PIE_OPTIONS}
    />
  );
};

Rather than passing a giant options object, you pass each property of that options object as a prop.

@planttheidea Thanks a lot!!!
I try to your advice, so passed pie prop to bb.
 PIE_OPTIONS = {
    innerRadius: 20,
  }

  render() {
    return <BillboardChart data={this.chartData} size={this.SIZE} pie={this.PIE_OPTIONS} />
  }

but, this case is don't work.
pie = {
   innerRadius: 20,
}

render() {
	return <BillboardChart data={this.chartData} size={this.SIZE} pie={this.pie} />
}

What difference between first case and second case ?

I mean it looks fine, but you've provided an incomplete representation.
If you look at the examples section in the repo, you'll see one for a donut chart. If you just change the word donut to pie you skills have an example you can play with.

@planttheidea Umm, I'm sorry I missed out the information too much.
I try to apply options of Pie chart but doesn't work.
I did what I advised and it worked normally.
Perhaps it was my mistake.
Thank you.


Improve worker

Remove resources manager
Improve fetch memory limit
Enhance logs

e2e tests: :+1:

@inalgnu would you have some time to review this ?


books

[The Joy of Google Search - Daniel M. Russel] (https://www.amazon.com/Joy-Search-Google-Insiders-Beyond/dp/0262042878) - extra exercises for sourcers, chance to understand even deeper the Google search operators and some possible tricks.

Awesome addition! Thanks 🎉

Cool! Happy to contribute :)


setup-rack.sh failure with RACK-v13.0 and Python 3.8.10 on Ubuntu 20.04

I attempted to upgrade from RACK-v12.0 to RACK-v13.0, but ran into a problem while executing the setup-rack.sh script. I believe this is a new issue as of this version. The BooleanOptionalAction for argparse was newly introduced into Python 3.9, and thus does not work in Python 3.8.10, which I believe is the current default version for Ubuntu 20.04.
I must have missed any specific notification that a later Python version was required. It's probably useful at least to not that, following the tutorial steps, this is probably the first place that the compatibility issue will show up for a user.
$ ./setup-rack.sh
Traceback (most recent call last):
  File "/home/gnelson/venv/RACK-v13.0/bin/rack", line 8, in <module>
    sys.exit(main())
  File "/home/gnelson/venv/RACK-v13.0/lib/python3.8/site-packages/rack/rack_cmd.py", line 48, in main
    args = get_argument_parser().parse_args()
  File "/home/gnelson/venv/RACK-v13.0/lib/python3.8/site-packages/rack/__init__.py", line 970, in get_argument_parser
    manifest_import_parser.add_argument('--optimize', default=True, action=argparse.BooleanOptionalAction, help='Enable RACK UI optimization when available')
AttributeError: module 'argparse' has no attribute 'BooleanOptionalAction'

We do all of our testing on the current Ubuntu LTS release. The rack CLI commands are also available inside the RACK docker image via docker exec. Support outside of that version of Ubuntu is accidental.


Make Device inherit Resources

The Resources trait only contains associated types and is used to tell what types is associated with a Device. Instead of having an associated Resources type, make Device inherit it. This will simplify the API quite a bit.

Seems that some part of the API depends on Resources being Copy.

:+1: if it's possible

This may conflict with #570, which has our strategic interest. I suppose let's not shuffle stuff too much until it's clear what the requirements for async resourcing are.

@bvssvni will it simplify the API? I briefly thought about it and didn't find any particular case that would be  helped. I believe <D as Resources> is not better than D::Resources.

I think we can close this, because the separation of device and resources fits better with the new architecture.


Property is not retained on custom view controller

I have made a UIViewController separately on UIStoryBoard as a MenuViewController, but it throws bad exception every time when I am creating any reference IBOutlet of controller objects like UITableView or UILabel etc. I tried to make both strong and weak type.
But Its working when I am creating these type of controller objects programatically.

Hi Evnaz,
Any comments on this?


Update vpn-about.adoc

mentioned PFS requirement as per AWS guidelines

So @danieldiaz-mule, it sounds like this PR is unnecessary and I should close?

So @danieldiaz-mule, it sounds like this PR is unnecessary and I should close?

To me, yes... @dtufino do you have any specific context as to why we need this?

@danieldiaz-mule  For some customers it may not be apparent that PFS is a requirement so we should probably clarify as it is stated in AWS documentation. I agree it should be added in the requirement section instead.

Closing per Daniel


OC-254 messaging config

please retest this

SonarQube analysis reported 6 issues

 1 critical
 2 major
 3 minor

Watch the comments in this conversation to review them.

All is well.

SonarQube analysis reported 1 issue

 1 major

Watch the comments in this conversation to review them.

All is well.

All is well.

All is well.


RPI-#2. Add framework of CLI

What is this PR for?

Connecting to editor every time and handling command with if/else is troublesome. Using a CLI via SSH can directly control RPi.
What type of PR is it?
[Feature]
Todos
What is the Github issue?

https://github.com/NTUEELightDance/LightDance-RPi/issues/2
How should this be tested?

Install cmd2:
pip install -U cmd2
Run:
python ./cli/cli.py
Screenshots (if appropriate)

Questions:

Do the license files need updating? No
Are there breaking changes for older versions? Yes
Does this need new documentation? Yes

LGTM.


Fix propagation bug related to missing s

Fixes #367.
There was a mistype between session and sessions in the propagation query for acquisitions.
I also fixed the propagation tests that should have caught this. The tests were showing a false positive due to all new container objects having public set to true as default.

Will merge when CI is green


Review Nervos DAO demo

https://github.com/zhb666/Nervos-DAO

[ ] tested locally, according to the documentation
[ ] code
[ ] testnet

some comments on Docs： https://github.com/ckb-js/nexus/issues/125#issuecomment-1476225859

code review: https://github.com/zhb666/Nervos-DAO/issues/3


[indexer] Add full_objects_history

Description
We add a separate table that stores full object history that we do not plan to prune.
Test plan
How did you test the new or updated feature?

Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[ ] CLI:
[ ] Rust SDK:
[ ] REST API:

@lxfind the change looks good but let's not merge to main yet, b/c this is breaking change and will be in production in the next release if merged, we usually merge it to a breaking change park and then merge it to main when backfill / DB ops are complete.

Got it. Is there a doc that describes the process? For instance, does the oncall usually manually trigger a backfill/migration from the park branch from time to time? How does it identify gaps if the indexer isn't paused?

Added ingestion change.
Removed SQL insert.
Also removed the object_status column since that can be derived from the serialized_object field.

removed the object_status column since that can be derived from the serialized_object field.

does this require deserialization of the bcs contents ?

does this require deserialization of the bcs contents ?

No, it only need to check if that column is null.


[🐞] async useVisibleTask$ blocks all other task execution and UI updates

Which component is affected?
Qwik Runtime
Describe the bug
I have been using visible tasks like useEffect in React, where I may have an async function doing lots of things. I noticed long periods of time where my app was "frozen" - the UI not responsive, other tasks not running, etc
It turns out that an async visible task blocks everything  from executing (other tasks, state changes, UI updates, etc) until done.
This bit me fairly severely, and if it is intentional it was unintuitive and (from what I can tell) undocumented
For instance, in the following code:
import { component$, useSignal, useVisibleTask$ } from<EMAIL_ADDRESS>
const delay = (ms: number) => new Promise((resolve) => setTimeout(resolve, ms));

export default component$(() => {
  const count = useSignal(0);

  useVisibleTask$(async () => {
    await delay(6000);
  });

  useVisibleTask$(async () => {
    alert('I ran!');
  });

  return (
    <>
      <button
        onClick$={() => {
          count.value++;
        }}
      >
        Count: {count.value}
      </button>
    </>
  );
});

The UI is completely unresponsive for 6 seconds. "I ran" doesn't show until 6 seconds, and the button is completely unresponsive when clicked
Reproduction
https://stackblitz.com/edit/qwik-starter-dj8iuy?file=src%2Froutes%2Findex.tsx
Steps to reproduce
Open the above the [stackblitz link])(https://stackblitz.com/edit/qwik-starter-dj8iuy?file=src%2Froutes%2Findex.tsx).
Note that for the first 6 seconds the app is completely unresponsive - clicking the button doesn't increment the count, "I ran" doesn't display, etc.
System Info
System:
    OS: macOS 13.0
    CPU: (12) arm64 Apple M2 Pro
    Memory: 150.36 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 18.15.0 - ~/.nvm/versions/node/v18.15.0/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v18.15.0/bin/yarn
    npm: 9.5.0 - ~/.nvm/versions/node/v18.15.0/bin/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 113.0.5672.126
    Edge: 113.0.1774.35
    Firefox: 112.0.2
    Safari: 16.1
  npmPackages:
    @builder.io/qwik: ^1.0.0 => 1.0.0
    @builder.io/qwik-city: ^1.0.0 => 1.0.0
    undici: 5.21.0 => 5.21.0
    vite: 4.3.0-beta.3 => 4.3.0-beta.3

Additional Information
No response

Additional context here: https://github.com/BuilderIO/qwik/issues/2672#issuecomment-1396702190

in a worst case scenario we could always make a lint rule saying don't make async visible tasks. I can't see any scenario where we want the UI to completely lock in a visible task. the workaround I use and the linter could suggest is an async iife. it's ugly but a lot less severe than a frozen page
useVisibleTask$(() => {
  (async () => {
    // Do async stuff
  })()
})