Support issue for Python template

Expected Behaviour
A way to configure how many threads of the underline WSGI server to be used for function execuiton
Current Behaviour
Default threads count is 4 and thus only 4 parallel functions are executed at the same time
Possible Solution
Providing an env variable inside templates
USER app

ENV fprocess="python index.py"
ENV cgi_headers="true"
ENV mode="http"
ENV upstream_url="http://<IP_ADDRESS>:5000"

Steps to Reproduce
Function
version: 1.0
provider:
  name: openfaas
  gateway: http://<IP_ADDRESS>:31112
functions:
  hello-openfaas-of-watchdog:
    lang: python3-http
    handler: ./hello-openfaas-of-watchdog
    image: afilipov/hello-openfaas-of-watchdog:latest
    labels:
      com.openfaas.scale.min: 1
      com.openfaas.scale.max: 1
    environment:
      write_debug: true
      read_timeout: 60s
      write_timeout: 60s
      exec_timeout: 180s

import time

def handle(event, context):
    # some_str = ' ' * 1024*300

    print("Start : %s" % time.ctime())
    time.sleep(15)
    print("End : %s" % time.ctime())

    return "Response from of-watchdog"

Testing results with 50 parallel invokations
hey -n=50 -c 50 -m POST -d=Test  http://<IP_ADDRESS>:31112/function/hello-openfaas-of-watchdog
2020/09/03 10:05:45 OperationalMode: http
2020/09/03 10:05:45 Timeouts: read: 1m0s, write: 1m0s hard: 3m0s.
2020/09/03 10:05:45 Listening on port: 8080
2020/09/03 10:05:45 Writing lock-file to: /tmp/.lock
2020/09/03 10:05:45 Metrics listening on port: 8081
2020/09/03 10:06:24 POST / - 200 OK - ContentLength: 0
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 1
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 2
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 3
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 4
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 5
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 6
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 7
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 8
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 9
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 10
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 11
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 12
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 13
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 14
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 15
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 16
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 17
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 18
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 19
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 20
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 21
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 22
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 23
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 24
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 25
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 26
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 27
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 28
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 29
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 30
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 31
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 32
2020/09/03 10:08:00 stderr: WARNING:waitress.queue:Task queue depth is 33
2020/09/03 10:08:01 stderr: WARNING:waitress.queue:Task queue depth is 34
2020/09/03 10:08:01 stderr: WARNING:waitress.queue:Task queue depth is 35
2020/09/03 10:08:01 stderr: WARNING:waitress.queue:Task queue depth is 36
2020/09/03 10:08:01 stderr: WARNING:waitress.queue:Task queue depth is 37
2020/09/03 10:08:01 stderr: WARNING:waitress.queue:Task queue depth is 38
2020/09/03 10:08:02 stderr: WARNING:waitress.queue:Task queue depth is 39
2020/09/03 10:08:02 stderr: WARNING:waitress.queue:Task queue depth is 40
2020/09/03 10:08:02 stderr: WARNING:waitress.queue:Task queue depth is 41
2020/09/03 10:08:02 stderr: WARNING:waitress.queue:Task queue depth is 42
2020/09/03 10:08:02 stderr: WARNING:waitress.queue:Task queue depth is 43
2020/09/03 10:08:03 stderr: WARNING:waitress.queue:Task queue depth is 44
2020/09/03 10:08:03 stderr: WARNING:waitress.queue:Task queue depth is 45
2020/09/03 10:08:03 stderr: WARNING:waitress.queue:Task queue depth is 46

2020/09/03 10:08:15 POST / - 200 OK - ContentLength: 0
2020/09/03 10:08:15 POST / - 200 OK - ContentLength: 0
2020/09/03 10:08:15 POST / - 200 OK - ContentLength: 0
2020/09/03 10:08:15 POST / - 200 OK - ContentLength: 0

2020/09/03 10:08:30 POST / - 200 OK - ContentLength: 0
2020/09/03 10:08:30 POST / - 200 OK - ContentLength: 0
2020/09/03 10:08:30 POST / - 200 OK - ContentLength: 0
2020/09/03 10:08:30 POST / - 200 OK - ContentLength: 0

From the last 2 blocks of logs we can see that only 4 requests are processed at the same time. Tested the same scenario with the classic watchdog and the execution time is much lower because of the multiple parallel processes.
   We are looking for high concurrency and throughput with the new watchdog. With simple function without sleep the tests shows significant improvements with the new of-watchdog.
Your Environment

Docker version 19.03.1

Kubernetes version v1.15.4

Hi @AntonioFilipov
I'm not sure this is anything to do with the watchdog, it sounds like an issue with the python3-http template that you're using and the behaviour of its HTTP server called waitress.
I'd suggest you spend a bit of time with their documentation and browsing the template repo below:
You can find the repo here: https://github.com/openfaas-incubator/python-flask-template
For now I'll close this issue since it's attributing the behaviour to the of-watchdog which I believe to be incorrect (happy to be proven wrong of course).
Alex

/set title: Support issue for Python template

It seems like you may want the setting in the Waitress docs:
--threads=INT

https://docs.pylonsproject.org/projects/waitress/en/stable/runner.html#runner
If it can configured via ENV-var, then that'd be your easiest option, otherwise feel free to experiment by forking the templates repo, changing them and testing them out.
faas-cli template pull https://github.com/your-repo/templates

You should also configure some minimum level of availability for the function, like a min scale value, and then consider using the endpoint loadbalancing feature in openfaas, to make sure the replicas are getting an even balance and to prevent any keep-alive code in your testing app.
Endpoint load-balancing in the docs:
https://docs.openfaas.com/architecture/production/#endpoint-load-balancing


Questions regarding strong and em

Hi,
I'm trying to wrap my head around an issue I'm encountering with **strong** and *emphasised* texts: they are just rendered as vanilla paragraph text on my system, instead of the expected strong and italic text.
I've noticed that the css rules specifically reset the font property of the strong and em tags (in css/base.css, and also later on for strong), and use bold and italic fonts instead (Open Sans Bold, and Open Sans Italic). While this works well on the Mapbox API page because the aforementioned fonts are served to the visitor, I'm afraid most developers would not want to do the same.
I'm not sure I understand the reason behind this choice, rather than just using vanilla properties of the tags.
Several tweaks may do the trick:

don't reset rules for strong and em (or maybe do font-family: inherit instead of font: inherit),
remove the font-weight: normal for .strong.

What do you think ?
BR

Yeah, sorry about this. This is the result of an still-present bug in base.css, where it used web fonts incorrectly, in a systematic way that spread the problem around.
I've just implemented #24, which should fix this issue - can you check it out and see if it resolves the problem?

Done ! LGTM !


Undelivered Mail Returned to Sender

☺ Please check the troubleshooting page before opening a new issue. ☺
https://github.com/jeboehm/docker-mailserver/wiki/Troubleshooting
Describe the bug
The IP address sending this message does not have a 550-5.7.25 PTR record setup, or the corresponding forward DNS entry does not 550-5.7.25 point to the sending IP. As a policy, Gmail does not accept messages 550-5.7.25 from IPs with missing PTR records. Please visit 550-5.7.25  https://support.google.com/mail/answer/81126#ip-practices
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Send email success
Docker environment (please complete the following information):

Output of docker info
Output of docker-compose version

Hi Quang,
your bug description contains the solution.
I'm closing this. Please put more effort into creating a new issue next time.


ASCII 232 for recognizing GS1 not read on Android but on iOS

iOS recognizes the GS1 initial char 232 correctly (this line: https://github.com/phonegap/phonegap-plugin-barcodescanner/blob/master/src/ios/zxing-all-in-one.cpp#L4697)
However on Android nothing is returned and the corresponding line is never hit. Meaning the byte stream of raw bytes does not include it.
Anyone around knowing in detail about the ZXing codec for Android?

Why was this cloaed? Is there a solution?
I have implemented the fix in my fork. If this is not solved I will open a PR.

@bitflower, see #222 and please re-validate that issue is still actual after the latest changes in the plugin. If yes, feel free to reopen this.
And your PR is welcome :)


[Rust] Arrow/Parquet/Datafusion interop not possible?

Hello, I have a just played with the new 3.0 crates of the arrow family, and keep hitting a wall for hours now.

I started with arrow itself, and built a large StructArray with my data. This was the easy part.

To cache this bigStructArray, I'd like to cache it as parquet file to disk. Seems like I can not use the parquet crate to achieve this, and the arrow crate has nothing useful, either. (CSV/JSON is not an option, this would en in a custom binary format again)

Now the most disappointing part: It is absolutely unclear for me how I would query the StructArray (like: give me all records where column_X = Y, and so on) there a compute kernels in the docs but how to use it is completely unclear for me. Then I saw the datafusion crate which looks exactly like what I want, but... again no hint how to use my StructArray from memory.

I had the impression that the point of arrow, as a portable memory structure, is exactly this. And these crates do use arrow internally... Did I get something totally wrong? May someone give me a few hints what I'm missing (save/load/query)? It would be sad to go back to my large rust Vec blobs and manually iter()/filter() stuff.
Thanks in Advance!

Hi @Anonyfox, thanks for reaching out to the project. I can understand your frustration at hitting walls. What you're trying to do isn't as trivial as it should be, mainly because we don't yet have a lot of comprehensive guides on how the various crates in the ecosystem interop.
@elferherrera has started https://elferherrera.github.io/arrow_guide/reading_parquet.html, and perhaps a guide that could help unblock you, could live there.
Do you mind sharing the schema of your StructArray? It sounds like you should be creating a RecordBatch from the StructArray, where it'll be easier to save it to Parquet (or the Arrow IPC format, which would address the CSV/JSON concern).
Based on the schema that you share, we can then look at how to help out.
Thanks

Maybe I just took some wrong directions, therefore I'll outline what I want to achieve with some snippets. So, this resembles my basic struct:
pub struct Book {
    /// NOT unique, but known to be ascii with fixed length of 10
    pub code: String,
    pub authors: u8,
    pub reviewers: u8,
    pub title: String,
    pub keywords: Vec<String>,
    pub price: f32,
    pub is_english: bool,
   // ... more fields of mostly Strings
}

And I have a few millions of them, as `Vec<Book>`. Now to illustrate the use case a bit more, I have a Library like this:

pub struct Library {
// will be updated when purchasing/... happens
local_books: Vec,
// will be updated periodically from a remote source
external_books: Vec,
}

In my application there exists a lazy static `RwLock<Library>`, which receives many read queries per second (_"how many books written by 3 authors have you?"_) where I have to lookup in `local_books` and if nothing is found there perform the same lookup in `external_books`, returning the final result.

Works okayish, but consumes quite a bit of RAM and I have to build "queries" manually using chains of `iter()`/`filter()`.

My thought now was that I can leverage Arrow and it's friends to make this thing more efficient, at least not slower, and have dynamic SQL-ish queries for easier development.

Now after playing around even more, especially with datafusion, I got a working state of this (leveraging RecordBatches as you hinted):

pub struct Library {
local_books: MemTable,
external_books: MemTable,
}

where the table schema looks like this for both MemTables:

vec![
Field::new("code", DataType::Utf8, false), // fixed size binary seems to not work yet in datafusion
Field::new("authors", DataType::Int8, false), // u8 -> i8 cast is safe in my case
Field::new("reviewers", DataType::Int8, false), // u8 -> i8 cast is safe in my case
Field::new("title", DataType::Utf8, false),
Field::new("keywords", DataType::Utf8, true), // joined the Vec with ',', couldn't get ListTable to work
// ...
]

Now the hard wall I faced: actually querying the table! I tried something like this:

impl Library {
pub async fn count(&self) -> Result {
let mut ctx = ExecutionContext::new();
ctx.register_table("local_books", Box::new(self.local)); // doesn't work, attempting move
let sql = "SELECT COUNT(*) FROM local_books";
let results = ctx.sql(&sql)?.collect().await?;
// ... returning the count
todo!()
}
}

- the `ExecutionContext` needs to be mutable, so I must generate a new one for each call (or: must do once I have dynamically parameterized calls)
- `register_table()` only accepts `dyn TableProvider`, not a reference or an `Arc` or anything else that would not need deep cloning
- my tables are quite large, so I really want to reuse them once generated, and they might be replaced (outer RwLock) while the application is running
- caching things on disk is trivial with serde for `Vec<Book>` but requires much disk space, and while _writing_ is allowed to take a few seconds without issue, _reading_ the dump must be really fast
- I found literally zero hints how to serialize/deserialize the MemTable from/to parquet in the docs/examples

The insight here is that cloning a MemTable is actually cheap. MemTables hold RecordBatches, which themselves are built from ArrayRefs (i.e., Arc<dyn Array>).
This seems to be an anti-pattern though, so much so that MemTable doesn't implement Clone. You'd probably want to store RecordBatches in your Library instead, and build MemTables from them whenever you want to add them to the context (RecordBatches are Clone).
This also means that adding record batches to the context is cheap, and that the context doesn't own those batches. If you want to refresh data in the context, however, you'll need to unregister the table and reregister it with the new data. If you don't do this, when you hold your batches behind an RwLock and replace them, the context will actually keep an Arc pointed to the old data and prevent it from being deallocated.

Hi, your sequence no true, is false

Seems like this issue is resolved.  Going to close it.


[Bounty] Training RNN-T

Hi @geohot,
I want to take up this task. Please let me know what dataset do you intend me to train the model with.
Best,
Rakesh

Bounties can be locked by submitting a draft PR that shows some considerable work in progress.

I see some "bounty-locked" tags at https://github.com/tinygrad/tinygrad/issues?q=RNN-T ... Is this issue available for the take ?

seems locked #933, tho not in the bounty spreadsheet. @geohot may have to update that.


When using elyra dev tag error is main.js: SyntaxError: expected expression, got '?'

Hi, using kubernetes and the elyra image I get the following error:
SyntaxError: expected expression, got '?'main.e41ae1cf1107c2b063a2.js:46:31
It seems to be this line: _CONFIG_DATA = configData ?? Object.create(null); that fail.
when launching with helm  my chart for jupyterhub I get this warning:
coalesce.go:196: warning: cannot overwrite table with non table for extraConfig (map[])
My config:
hub:
  db:
    type: sqlite-memory

  extraConfig: |-
    from kubespawner import KubeSpawner
    from tornado import gen
    import yaml

    class CustomKubeSpawner(KubeSpawner):
        def get_env(self):
            env = super().get_env()

            env['KG_HTTP_USER'] =  self.user.name
            env['KERNEL_USERNAME'] =  self.user.name

            return env
    c.JupyterHub.spawner_class = CustomKubeSpawner
    config = '/etc/jupyter/jupyter_notebook_config.py'
    c.Spawner.start_timeout = 500
ingress:
  enabled: true
  annotations:
    kubernetes.io/ingress.class: traefik
  hosts:
    - ""
singleuser:
  # force using new UI from jupyterlab
  defaultUrl: "/lab"
  image:
    name: elyra/elyra
    tag: dev
    # disable this in a production environment
    pullPolicy: "Always"
  storage:
    dynamic:
      storageClass: nfs-dynamic
  extraEnv:
    JUPYTER_GATEWAY_URL: http://myurl/gateway
    KG_REQUEST_TIMEOUT: "120"

This is good timing, as I am just about updating my jupyterhub scripts/environment.
By taking a quick look at your config, I noticed it has a couple of differences:
https://github.com/lresende/ansible-kubernetes-cluster/blob/master/roles/jupyterhub/templates/jupyterhub-config.yaml.j2
Let me make progress on my side and I will let you know.
BTW, what version of Hub or Z2JH are you using?

Hello,
first I tried the 1.2.1 but then I get warning because I'm not using 1.1.0, then I downgraded to 1.1.0

A couple of updates here:

coalesce.go:196: warning: cannot overwrite table with non table for extraConfig (map[])

I got this warning, but AFAIK it is not affecting the configuration at all, and I moved this to a lower priority

But if I understand correctly, normally when I go to /gateway, traefik should redirect the query to / on the gateway right ? Not /gateway

Yes, you will need to configure stripPrefix, and with the new traefik, it should look like this or the old way.
Current status:
I am now tweaking the Elyra image to properly work with the volume mounts, as the current one is configuring things on the jovyan user home directory that gets overridden by the nfs storage configured here.

ok to be honest I'm fed up debugging this traefik things, so much time I'm on it and it didn't work. I always get a "404 ERROR for /gateway" It's like the PathPrefixStrip is not working nor used by this F**** traefik....

Yes, these are some of the issues I have fixed either on the ansible-scripts and/or the new image quay.io/elyra/elyra:dev. Once you pass these, the one extra remaining thing is that in order to have this working with lab3, we need to set JUPYTERHUB_SINGLEUSER_APP: "jupyter_server.serverapp.ServerApp" on the hub config.
With these changes, I seem to have all working now, let me push these changes to the respective repositories and as you to do a final test.
BTW, Thank you for the patience, I know how it is, as I was also very frustrated with ingress and some of the K8s configuration stuff. But I am glad we made good progress.

I have pushed the latest JupyterHub config and you could use the command below to update your hub deployment:
helm upgrade hub jupyterhub/jupyterhub --namespace hub --version 0.10.6 --values /tmp/ansible-install/jupyterhub-config.yaml

Note that the config is now using the elyra image from quay.io, which I have been updating with the fixes to some of the permission issues you described above.
@dummys Please let me know if that works for you.

And last but not least, is it possible to make it work with ubuntu ? It would be nice to deploy everything from scratch.

In ubuntu you could use microk8s which will give you a working k8s environment with storage and ingress,  and you just need to cherry-pick the commands to deploy gateway and hub from the ansible scripts.

The changes to Elyra docker image have been released as part of Elyra 1.4.2, and configuration updates have been updated to the ansible scripts. Note that, Elyra 1.4.2 is still based on JupyterLab 2.x while master (and the Eyra image dev tag) is based on JupyterLab 3.0 release candidates.

ok but then changing the version of the hub means changing the version of the gateway ? I have made a small fix to fix the permissions issue:

The hub and gateway versions are independent.

ok I see. For now its not working
 https://github.com/jupyter/enterprise_gateway/releases/download/v2.3.0/jupyter_enterprise_gateway_helm-2.3.0.tgz
I use this gateway

Find me or others on the Elyra team at: https://gitter.im/elyra-ai/community
We also have dev meetings every Thursday, see project readme for details (at the bottom of the readme)

ok but I didn't received any response on the gitter.

@dummys please reopen if we still need to discuss this issue.


Truncating script names

Hello Viktor. I was running into an issue with the sidebar being too wide and realized that is it has to do with the script names not truncating...would it be possible to add this feature?
Seems like this is what should be happening based on the plugin images but it's not working on my version for some reason. I'm on 3.0.4 (3092)

https://forum.glyphsapp.com/t/feature-request-narrower-sidebar/19723/6


expose PCI VPD info in resources

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

Thanks a lot, that's very useful!
I'll do some commit re-shuffle and add the API extension bits and swagger updates.

Documentation preview available at: https://linuxcontainers.org/lxd/docs/pr.10387/


Add zappi phase setting select

Add support for zappi phase select, fixed #509 , depends on https://github.com/CJNE/pymyenergi/pull/35

Simplified ZappiChargeModeSelect a little bit, can probably be simplified even more by moving the mapping of internal id to readable string to the translations.

Added test extension

@CJNE Any remarks on this PR?

Lint errors are generic, see https://github.com/CJNE/ha-myenergi/pull/554

Sorry for the delay! It's looking good, thank you!


Expand Pypika and string literal tests for escaped characters

Expands the test coverage for Pypika queries and string literal parsing to include new lines, tabs, and arbitrary unicode characters.

Pypika Query Tests: Adds new test cases in tests/test_sql_encoding.py to verify the correct encoding of string literals containing new lines (\n), tabs (\t), and arbitrary unicode characters (e.g., \u1234) in Pypika queries. Each test constructs a query involving these characters and asserts the generated SQL matches the expected output.
String Literal Parsing Tests: Updates tests/test_string_literals.py to include test cases for parsing string literals with new lines, tabs, and arbitrary unicode characters in both single and double-quoted strings. These tests ensure the parser correctly unescapes these characters and returns the expected string.

For more details, open the Copilot Workspace session.

Updates tests to support direct usage of unicode characters in SQL queries without escaping.

Modifies the test_pypika_encoding_with_arbitrary_unicode in tests/test_sql_encoding.py to expect unicode characters (\u1234) to be used directly in the SQL query string without escaping. This aligns with the task's objective to support utf-8 characters in SQL queries natively.
Keeps tests for encoding string literals with single and double quotes, new lines, and tabs unchanged, focusing the update solely on the handling of unicode characters.

For more details, open the Copilot Workspace session.


Docs for events

After reading through the docs I found I didn’t know enough about events to successfully use the framework, and I needed to read the source to really grasp how to utilize them, I think I’ll write a specific section of the docs about events to help others that have that problem

Sounds good. There is an example in the examples section of the repo that I originally intended as an interactive way to explain events. Perhaps you can embed that in the page you are writing as well.

Also, take note of #141 and #142, which are going to change the API.

#159

This is now done. Feel free to leave feedback about the docs here.


Broken link in Use Cases

The link "A Rest API using Django and authentication with OAuth2 AND third parties!" in docs/uses_cases.rst is broken.

PR merged.


actually take path for FileStorage::from_path

Otherwise; if one has a userland pathbuf an unsafe conversion back to String has to be made:
let mut user_config_directory = dirs::config()?;
user_config_directory.push("foo");
user_config_directory.push("bar.txt");

let egui_filepath = user_config_directory.to_str()?.to_owned(); // fallible but both sides want to use a path.
FileStorage::from_path(egui_filepath);

Tests (cargo test) unfortunately seem to be broken on main already; but example_glium works as expected with the removal of .into() below.

@emilk Updated per for your feedback, thanks!


Added ScanIntoItems and ScanIntoItem functions

Which issue is fixed by this PR? If not related to an existing issue, what does this PR do?
First I created a ScanIntoItems function that basically unpacks a list
into sql's scan function.
Then I updated the generic cruder to use the function, although it is meant
more for endpoints that don't already have a generic function. I updated
the generic case since it could be used as a model and so that the error
messages won't look too different from implementors that use ScanIntoItems.
ScanIntoItems is especially convienent when returning a number of items.
It could potentially be used for: #2494
Here is an example that is applicable for cachegroups:
items := []interface{}{
    &cg.ID,
    &cg.ParentName,
    &cg.SecondaryParentName,
}
if err = api.ScanIntoItems(resultRows, items); err != nil {
    if err == sql.ErrNoRows {
	return errors.New("no cachegroup found with this id"), nil, http.StatusNotFound
    }
    return nil, err, http.StatusInternalServerError
}

Mayber later down the line, ScanIntoItems would be prefered over
ScanStruct since ScanStruct belongs to sqlx. I don't think we use sqlx
for anything other than ScanStruct, so we could potentially get rid of
a dependency.
Which TC components are affected by this PR?

Traffic Ops

What is the best way to verify this PR? Please include manual steps or automated tests.
(If no tests are part of this PR, please provide explanation as to why no tests are included.)
Run the traffic ops integration tests.
This PR doesn't introduce new tests, but probably should. The function advises function callers that they don't need to close sql.Rows. The ability of the function to close the rows properly should be tested, but I'm not sure how it should be done yet. I found the mockdb, but I haven't found an example of a query running against it yet.
Check all that apply

[ ] This PR includes tests
[x] This PR includes documentation updates
[ ] This PR includes an update to CHANGELOG.md
[x] This PR includes all required license headers
[x] This PR does not include a database migration
[x] This PR does not fix a serious security flaw. Read more: www.apache.org/security

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3398/
Test PASSed.

Update:
As I've written it now, ScanIntoItems uses sql.Rows, but an update or create should use QueryRow, instead of NamedQuery or Query.
The generic create uses NamedQuery only because it allows us to map fields from an interface using their names. Note: is possible to have a simple NamedQueryRow function (https://github.com/jmoiron/sqlx/issues/102), or to make a function that uses reflection to read from the struct in a linear fashion (instead of using names). Both of these solutions would eliminate the need for ScanIntoIItems
TL;DR:
I don't think ScanIntoItems should be used as it currently exists, so I'm closing this PR.


Improvement almost right feedback

Adds a second try after you got something almost right. Moreover it provides the option to supply a hint together with that second try. It is also now checked if you made mistakes due to case sensitivity.

By writing a new method: render_almost_right(current_document, content_textfield, hint = "")

By adding an if clause to the check_current_document() function

By adapting verify_document() and giving the verificationObject another attribute

render_almost_right(current_document, content_textfield, hint = "") needs adaptiion so the hint is actually featured in the HTML link.


Remove trashcollector functionality

Currently there is this trashcollector id hard coded in the frontend.
Just remove that and all functionality that comes with that. Also in the backend (see).
We dont need that in my opinion.
On the routing page we can maybe just have a button, that says "hide trashbins of this route" instead of "assign route" and another button "show all trashbins" instead of "unassign all bins".

Also remove the "assigned" key in the trashbin table.

also remove in the backend
https://github.com/tinyaiot-ps/backend/issues/37


CORS

Anything I need to do to setup CORS? Having a bit of trouble.

I tried the following in order to satisfy an CORS error:
const headers = {
'Access-Control-Allow-Credentials': 'true'
};
export const lokkaClient = new Lokka({transport: new Transport(API_BASE_URL, {headers : headers})});
But the above header setting does not work.   it showed up in the out-going request as this:  Access-Control-Request-Headers:access-control-allow-credentials, content-type.   As result the request comes back with the same CORS error.

On the backend, you need to do something like this:
app.use(cors({
   origin: 'https://myurl.com',
   credentials: true,
}));

@arunoda, you are suggesting the change on our API server side.  Actually I was able to get thru using jQuery.  Seems the call will work as long as the header  "Access-Control-Allow-Credentials=true" is present in the call.  The fact that I can get the result via jQuery seems to show that our server setup is OK.   But your http transport simply does not honor the setting I explicitly set as shown above.   Which was the reason of the failing...    Appreciate it if you could fix... thanks!
@reggi , looping you in on this

@arunoda , any chance this has been fixed?   what I reported above is this:
I set the header as this:
const headers = {
'Access-Control-Allow-Credentials': 'true'
};
But the actual header set by your code is this:
Access-Control-Request-Headers:access-control-allow-credentials, content-type
If I explicitly set 'Access-Control-Allow-Credentials': 'true', why can't the actual header to set as "Access-Control-Allow-Credentials:true" when the http request goes out?
Appreciate if you could take a look?   thanks!

The problem is fixed for me if the fetch mode option is used and set to 'cors'. @arunoda  please merge: https://github.com/kadirahq/lokka-transport-http/pull/4
I am using a personal fork in the mean-time.

I am also facing the issue of not being able to set Access-Control-Allow-Credentials.  Is there a way to fix this?

So is this project just dead now ...?


feat(evm): migrate to alloy-genesis, rm ethers-core

Motivation

Migrates to using alloy genesis instead of ethers core types for genesis utils.
Solution

Removes ethers core, migrates to alloy genesis.
The loadAllocs cheatcode was modified to try reading from a json file which only contains allocs, if the allocs read from the genesis file is empty. This is because now Genesis can be deserialized as default, and will not fail if the alloc field is missing.

failing tests look related

Yeah tests are failing because it seems storage is not being set properly—need to take a look

bump here

@mattsse for the issue, see: https://github.com/alloy-rs/alloy/pull/151#issuecomment-1908605365

thanks, no this makes sense to me 🙏


Site Title Doesn't Include "Ludum Dare"

Issue:
Typing "Ludum Dare" into urlbar doesn't return ldjam.com main site because the site's title doesn't actually include the name.
Steps to reproduce:

Begin typing "Ludum Dare"
Get offered LD Twitter, and previous LD events, but not the main site.
Regret.
Type "ldjam" instead.
Repeat every time you want to enter the site.

Use-case:
So far I for one have failed to enter the site on first attempt every time.
Typing "Ludum..." and browser autofilling would be a great boost in autocomplete usability.
Things attempted:
I tried searching for "ldjam.com" in the repo, but on a quick non-webdev glance couldn't find where the title is set, so I failed to create pull request directly.
Suggested Fix:
Add "Ludum Dare" anywhere in the title, e.g.
"LDJam.com - Ludum Dare"
"Home of Ludum Dare - LDJam.com"
"Ludum Dare"

I think this has been resolved, the https://ldjam.com/ title is currently 'ldjam.com | Ludum Dare game jam' for me.


improve whatwg-url compatibility

Using web-platform-tests
Ultimately it will be probably impossible to match behavior to fully compatible but it is a perfect case to find potential bugs
Some changes in PR are WIP
Current status: 112/206 passing (ignoring invalid cases)

I guess finally we should use the global URL for parsing. Focusing on such compatibility is tricky.


Add schema to temp table, fixes #3909

@juanignaciosl please one-liner review
thanks for your help btw

I've made a comment about the quoting. But, anyway, has this ever worked for organizations? :-\

Frontend tests were OK :+1: (details)

yes, it worked for sure, I demoed it and several people used it with org users for sure.

I meant with users with dashes in the name and so on, but if you've tested, :+1:

Frontend tests were OK :+1: (details)


Typing into URL bar brings up search box and removes focus from URL bar, which prevent you from typing!

The history search feature appears as soon as you try and type into the URL bar. This isn't bad, but the problem is that it removes the focus from the URL bar, so typing more letters no longer appears in the URL bar. This makes it essentially impossible to try and type a URL or hostname in. See this animation:

Consider keeping the text cursor focus in the URL bar, so a user can properly type in a URL

Thank you!

Reopening because this still occurs if typing a url that start with gemini://


For web frameworks

https://github.com/crocodilejs/crocodile-node-mvc-framework

NOT RELEASED YET

Try again when it's more mature.
Also please read the contributing guidelines next time.

Okay I understand, but no need to come off as snarky and rude haha, was
just trying to contribute since all the other frameworks listed are trash +
outdated.
On Nov 4, 2016 11:44 AM, "Sindre Sorhus"<EMAIL_ADDRESS>wrote:

NOT RELEASED YET
Try again when it's more mature.
Also please read the contributing guidelines
http://../blob/master/contributing.md next time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sindresorhus/awesome-nodejs/issues/613#issuecomment-258467205,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAf7hSjNuxI_mLROzayDiRvOQJuWVjqiks5q61LegaJpZM4Kpo6D
.


Add aws authenticator

Also moved aws_exports configure line because I believe we had it in the wrong place initially. Haven't tested out yet though

nope i have a better one. this one broke things somehow


Transaction status in loop not reset

Consider the following code
<?php

foreach($users as $user){
	$this->db->trans_begin();

	$this->db->query('some sql 1');
	$this->db->query('some sql 2');
	$this->db->query('some sql 3');

	if($this->db->trans_status() === FALSE){
		$this->db->trans_rollback();
	}
	else{
		$this->db->trans_commit();
	}
}

I am using this code to loop an user array to create users.
Each loop need to use 3 query to create a single user eg. insert user info, insert permission, ...
My expectation:
When some users failed, skip that user and rollback. Then continue to create users after that.
But the thing I got:
$this->db->trans_status() always return FALSE when something failed in the middle, all users got rolled back after one user failed.
I think $this->db->trans_status() never reset when doing commit or rollback.
CI version: 3.1.5
PHP version: 5.6.31
Server: Windows Server 2012 R2

#4026
duplicated

This is not a duplicate of #4026.
That was about failed queries outside of a transaction causing future rollbacks, and was fixed in 3.0.1. If it was the same issue, you wouldn't be experiencing it on 3.1.5.

Try with the commit linked above.

seems good

Great, closing then.


using the controller for ppm and fanspeed

So I see the output in documentation is in same unit as sensor. But , i want to control the fanspeed of a ventilation unit to keep the Co2 ppm on a specific level.
Should min and max output then be in PPM or in RPM (or % of max speed which is easier)
so sensor.co2 = ppm
output.min= 5 (% of max rpm)
output.max = 95 (% of max rpm)
OR
output.min = 400ppm
output.max=1400ppm
but then i dont know what to put the fan speed at.
Could you please enlighten me before i plunge into a wrong direction?

Mostly I use the output from 0-100% as @Cor1984 stated, then I calculate the   percentage accordingly. But if you prefer, you could set the minimum to 400 and the max to 1400, that that would be the rpm to pass to the fan.

yeah.. thx both.. Did some more reading on PID controllers in general. I did go with the 100% approach. Ventilator has hard time (capacity wise) to pull ppm levels down when more people are present, so i have no good way to adjust the parameters.. will play for a few days from here (though setpoint changes are disrupting the controller i have the feeling)


feat: Create spacy notebook example

This PR create an example notebook for using Unstructured with Spacy.
Not sure why there are 8 files on this PR. Github automatically added them. I do not have these changes locally

Notebook updates look good! LGTM once you add the requirements.txt for the example notebook.

Forgot to add them to the commit 🤦♀.


metrics_utils.cc:136] Check failed: shard < num_shards (254 vs. 4)

I meet this problem too. Do you solve this problem ?

waymo dataset only has 4 classes, maybe you give 256 classes into evaluation code

and one box 3d has 7 attribute, while in your picture it is 4, which is weird. @fengjinyuan @Tongjiaxun

Hello, I have the same issue.
In my case, I want to use waymo eval metric on customized dataset.
Is it possible to change num_shards, which is the total number of classes for custom dataset?
I found "label.proto" file but I am not sure that only this file contains total class info.
How can I change the num_shards?
Thanks in advance


ESP Insights simultaneous HTTP request error

I've recently come across an issue where using esp insights, while also polling a separate HTTPS GET request, can cause issues on occasion.
For example, I run a GET request every 5 seconds to check the active score of a hockey game. On occasion, insights will try to open an HTTPS request to upload to send the data to the cloud at the same time the HTTPS request for retrieving the score is open (or vice versa).
This isn't a huge deal as insights will just try again later and succeed. However, on a VERY rare occasion, this will somehow create a loop of back to back http failures that flood the console with logs and render the esp useless until the program is restarted.
Sadly I don't have a minimum reproducible example as this is so rare I've only ever caught it 4 times now. If there's a way to possibly let insights know when it should not attempt to open a connection to send the logs, that would be great.

Hi @connormartin12 you might very well be running out of sockets in your scenario.
The insights requests are executed from esp_rmaker_work_queue. To avoid the scenario happening, you may simply call esp_rmaker_work_queue_add_task with the task you want to execute. This will make sure both of these requests go in serial and not parallel at any instance.
Do check esp_rmaker_work_queue.h for API details.

Hi @vikramdattu, thanks for the advice. I tried this and it successfully solved the issue I was experiencing.


modern-sysu-thesis:0.3.0

I am submitting

[ ] a new package
[x] an update for a package

Description:

add an enhencement feature of pages' header, with context of the thesis's title and chapters
fix numbering of figures, tables

[x] ensured that my package is licensed such that users can use and distribute the contents of its template directory without restriction, after modifying them through normal use.

Thank you :)


inconsistency regarding team's properties between /j/users/:userId/teams and /j/teams

For Team entity define mandatory properties and set the others as optionals.
Proposal for mandatory properties:

Id
Name
MembersCount
TournamentsCount
IsPrivate
ImageURL

optionals:

Description
Accuracy
Joined
Members
Tournaments

the array of teams should be under a Teams JSON node. Only /j/users/:userId/teams is doing it.

@santiaago Could you review this?

ok with this


Revamp Your Website: Stylish Navbar Menu for Contributing to an Open-Source Project for a Sleek and Modern Design

Effect :When We Hover on Navbar Menu Option we see the Effect
Before Changes:

After Changes:

Hi @jeevan10017
Hi @Open-Code-Crafters
I recently updated the navbar menu for your project to make it more attractive and user-friendly. Enhance readability, improve navigation, and give your site a visually appealing touch that stands out with these simple yet impactful design updates. I've attached screenshots of the changes, and I'd love to contribute to this enhancement to improve the overall design.
Please assign me to this task. It would be a great opportunity to contribute while enhancing my skills.
I'm looking forward to hearing back from you!
Best regards,
@khurshed07
https://github.com/khurshed07

@khurshed07 , please proceed


Use new memory_stats call; use pandas for formatting

This PR

Adds detailed memory stats
Handles peaks using reset_peak_memory_stats() rather than emptying the cache
Dumps the trace as a pandas dataframe, which can be used for downstream analysis
Does the formatting using pandas' to_string, so variable numbers of fields are supported

Upshot is that .records() will give you a dataframe like this

with 96 columns covering every memory stat Torch offers. Terminal output can be whatever subset you want:

I don't think you'll want to take this PR as is; I've broken a bunch of stuff getting to what I need for the task at hand. I'm more making this PR in case you want to nab parts for the main branch.

now with bonus HTML barcharts

And now it accurately reports peaks and allocated/freed when you're profiling a hierarchy of functions.

This has been really useful for me, thanks so much for making it!
I've tried to restore some of the things I know that I broke, but some other stuff is still going to block merging with main. In particular,

It only does HTML output right now, no text
It requires pandas as a dependency. This might be too heavy for your tastes.

It's great that you liked this tool. This is really a big PR, hopefully I can review these changes thoroughly this weekend.

Thanks! No need to feel rushed though, and I understand if you decide it's too far from the original tool to be justifiable to fold in.

@andyljones I'm totally ok to drop python2.7 support. We have only < 1% python2.7 downloads.
Pandas dependency should not be a problem for me, it's a pretty common package. But I'm wondering if we can bypass pandas for the command-line interface.

Ok, I've resolved lots of the smaller issues. Bigger ones:

Text output: IPython handles objects that could be either text or HTML by looking for specific methods on the object returned by the cell. This means we can definitely do text output, but it'll require returning an object from print_stats, which means print_stats won't be able to take a stream.
What might be better is to move all the rich output to a new .render() method which'll return an object, and print_stats can pipe text to whatever stream it's been passed.  That has the advantage of preserving the existing API.
Terminal pandas: I'm inclined to use pandas to format the text output too. That's what I used for the text styling at the top of the thread:

and I'm loathe to re-implement a pandas-free code path for terminal use.

Let's clean things up based on ipython and pandas

Great. I'll do a full pass. Rather than list off individual names you'd like changed, is a there a style guide you can point me at?

We followed Google's python style guide.

Right:

Now does text or html output
Added docstrings
Added a global interpreter reset cause @profile accumulates in an ugly way in an interpreter session
Got the tests working
Got travis working
Updated the readme
Did lots of small fixes

can you make the function name an action rather than a thing? e.g. _subset_line_records to _extract_line_records, _line_records_merged_with_code to merge_line_records_with_code.

I've done this, but as an aside this is an idiom from functional programming. If a function has no side-effects, it effectively is its return value, so it should be called a noun rather than a verb. Think of it as soft command-query separation.
OK. I've fulfilled most of the revision requests, and at this point the cleanup for the PR has taken substantially longer than implementing the original functionality. I respect that you've got high standards for the code in your projects, and I know you've invested time too in reviewing my code, but I'm afraid I can't put any more time towards this. If you're not happy with the PR as-is, that's okay: I'll instead publish the functionality I wrote as it's own package. I'd be disappointed, but unfortunately that's just how things work out sometimes.

Andy, thanks for your valued time to make this PR and address my suggestions. This PR has been impressive since the first proposal. I'm just trying to help you make it more pythonic and readable (for most python coders). It's a common situation that people spend more time addressing code reviews than functionality.
I can merge this PR first and then do some changes myself, but I think it's better for you to make this PR a complete one. Sadly you don't have that amount of time.

Huurah! Thanks very much


[FR] Stock History Persistence

Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

Problem statement
Currently it is possible to write notes when performing stock changes (i.e. add, remove, count, transfer). However, if stock ever reaches 0, at which point the stock item is deleted ("Delete on deplete" defaults to True when adding a new stock item), this history with notes is lost, even from within the admin panel.
While delete on deplete can be disabled (albeit it must be done manually every time a stock item is created and cannot be changed by default for all new stock items in the current implementation), it creates a lot of unnecessary residual stock items, especially if items can be present in multiple locations.
Hence, there is a need to aggregate stock change history into a centralised location where it can persist beyond the lifetime of individual stock items.
Suggested solution
On the Part interface, it may be useful to have a "History" or "Stock History" tab below the existing "Stock" tab which aggregates all stock changes across all stock items for a product (including stock items that have been deleted) in reverse chronological order.

I haven't yet looked at how history is implemented, but my initial thoughts are that the individual stock changes/the entire history should be decoupled from stock items, and should only be cascade deleted when the product itself is deleted, rather than the stock item it corresponds to.
Describe alternatives you've considered
Open to discussion for alternatives.
Examples of other systems
No response
Do you want to develop this?

[ ] I want to develop this.

@maksimstojkovic I personally do not see a value in this - intrested to see what other think of it.
Centralized stock history will get unreadable fast.

As an aside, could this be something that could be implemented as an optional plugin, rather than a core function?

@maksimstojkovic you could certainly implement it as a plugin. Might be a good way to go about it, keeps the mainline codebase clean.
For reference, there is currently work on a "stocktake" feature, which has some similarities to what you are suggesting here:
https://github.com/inventree/InvenTree/pull/4069

@maksimstojkovic I personally do not see a value in this - intrested to see what other think of it. Centralized stock history will get unreadable fast.

Hi @matmair,
For my use, we have to keep track of every 'single' items (with batch number), which/when are used in what preparation, over many years. It's a regulatory obligation.
In our process, I explicitely add the check to remove 'delete on deplete' for any new item users creates.

@Bbillyben ref: https://github.com/inventree/InvenTree/pull/4106

@Bbillyben I still do not see what benefit it brings to track every movement of stock of a part on the part itself - that will balloon quickly on a mid-size installation and be very hard to reconstruct. I think #4106 makes your use case a bit easier.

@matmair,
Yep your right! thanks @SchrodingersGat !
That's just an example's situation where data persistent -for specific items- is mandatory! We have to keep track where, who, when for any audit by regulatory administration.

Interested to see what solutions get recommended / how they would be implemented.
Not sure if it will go into core/upstream but I would be willing to help move functioning code into a plugin if it gets rejected. That is a general offer for all things that won't go into core.

@Bbillyben is the addition of optional configuration for "delete on deplete" enough to close out this issue for you?

Hi @SchrodingersGat, from my side, it's quite enough !
But I don't know if it's fit with @maksimstojkovic request.

@SchrodingersGat that solution works for me, so this issue can be closed.
I may look into the plugin alternative some time in the future.

@maksimstojkovic feel free to ping us if you want any plugin pointers.


ValueError: Type mismatch (<type 'tuple'> vs. <type 'str'>) with values (() vs. icdar2015_test) for config key: TEST.DATASETS

I try python tools/test_net.py --cfg configs/text/mask_textspotter.yaml
but i can not import datasets
the problem is as follows:
make: Entering directory '/home/xie/MaskTextSpotter/lanms'
make: 'adaptor.so' is up to date.
make: Leaving directory '/home/xie/MaskTextSpotter/lanms'
[E init_intrinsics_check.cc:43] CPU feature avx is present on your machine, but the Caffe2 binary is not compiled with it. It means you may not get the full speed of your CPU.
[E init_intrinsics_check.cc:43] CPU feature avx2 is present on your machine, but the Caffe2 binary is not compiled with it. It means you may not get the full speed of your CPU.
[E init_intrinsics_check.cc:43] CPU feature fma is present on your machine, but the Caffe2 binary is not compiled with it. It means you may not get the full speed of your CPU.
INFO test_net.py: 142: Called with args:
INFO test_net.py: 143: Namespace(cfg_file='configs/text/mask_textspotter.yaml', multi_gpu_testing=False, opts=[], range=None, vis=False, wait=True)
Traceback (most recent call last):
File "tools/test_net.py", line 145, in
merge_cfg_from_file(args.cfg_file)
File "/home/MaskTextSpotter/lib/core/config.py", line 1095, in merge_cfg_from_file
_merge_a_into_b(yaml_cfg, __C)
File "/home/MaskTextSpotter/lib/core/config.py", line 1153, in _merge_a_into_b
_merge_a_into_b(v, b[k], stack=stack_push)
File "/home/MaskTextSpotter/lib/core/config.py", line 1147, in _merge_a_into_b
v = _check_and_coerce_cfg_value_type(v, b[k], k, full_key)
File "/home/MaskTextSpotter/lib/core/config.py", line 1242, in _check_and_coerce_cfg_value_type
'key: {}'.format(type_b, type_a, value_b, value_a, full_key)
ValueError: Type mismatch (<type 'tuple'> vs. <type 'str'>) with values (() vs. icdar2015_test) for config key: TEST.DATASETS

It seems value of TEST.DATASETS in mask_textspotter.yaml is not correct.


Add default properties to telemetry events

Ensure telemetry events have the default set of properties set:

[ ] MachineId
[ ] Frontend
[ ] Verb

This is complete.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 925 ms

Resolved: Bofrost.at performance has improved in cc9b38e.


[Question]: Can the bundled NodeJS be updated to patch vulnerabilities?

Describe your question
Similar to #4075, security scanning tools have reported that the version of NodeJS 16 installed with the agent is vulnerable to several CVEs. Could the bundled version of NodeJS 16 please be updated to at least 16.19.1?
EDIT1: I will attempt to submit a PR later today.
Versions
pipelines-agent-linux-x64-2.218.1 running on Ubuntu 18.04
Environment type (Please select at least one enviroment where you face this issue)

[X] Self-Hosted
[ ] Microsoft Hosted
[ ] VMSS Pool
[ ] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Operation system
Ubuntu 18.04
Version controll system
Git
Azure DevOps Server Version (if applicable)
N/A

Hi @easkay-castlight thanks for the reporting! We'll take a look at it
For future security issues you find, please check our policy: https://github.com/microsoft/azure-pipelines-agent/security/policy
thank you

The fix comes with new agent release.
Once it will be rolled out for all customers, we'll close this issue as resolved.
Thanks!

@easkay-castlight agent should be available for all customers, please let us know if the issue is not resolved for you.


zoom

Hi!
The zoom function in st does not work for me, when I press Alt+Shift+Up it scrolls. I've tried to rebuild st but with no success. Someone alsa suggested that I may have a another binding that interferes with this binding, but I have larbs... Please help...

It could be that is your window manager that grabs this key shortcuts.
Try to either change the shortcut in st, with something that is not used by your WM, or the other way around, check Yokut WM settings.

BTW, with up, you mean Alt+Shift+PgUp , right? ;-)

Yes PgUp, I use dwm... just installed larbs

Which alt are you holding down (left or right)? Are you using a special keyboard layout?

Left one, yes I use a swedish keyboard, but it has worked before... But I changed the shortcuts and now it works even though the shortcut is a bit more uncomfortable.


Add monitor

Adds monitor.py with preliminary logging support for validators. Contains a Sink class to dump logs into, a basic sink for logging to json lines and a log function that can be used to log any data dict to the sink.
Adds a .log method to the Validator class that can be used inside validators. Includes an example for the ReadingTime validator.
Adds a basic test for the json lines sink.

No longer needed. We're using OTEL now for logs, and metadata for specific info needed to be passed back through the ValidationResult


Concat is slow

Hi, here is my code. I want to concatenate 5100 dataframes into one big dataframe. The memory usage of these 5100 dataframes is about 160G. However, the processing speed is really slow. I have been waiting for about 40 minutes, but it still hasn't finished. Here is my code:
import modin.pandas as pd
import os
import ray
from tqdm import tqdm

ray.init()

def merge_feather_files(folder_path, suffix):
    # 获取所有符合条件的文件名
    files = [f for f in os.listdir(folder_path) if f.endswith(suffix + '.feather')]

    # 读取所有文件并合并
    df_list = [pd.read_feather(os.path.join(folder_path, file)) for file in tqdm(files)]
    merged_df = pd.concat(df_list)

    # 按照日期列排序
    # merged_df.sort_values(by='date', inplace=True)
    merged_df.to_feather('factors_date/factors'+suffix+'.feather')

# 使用方法示例
folder_path = 'factors_batch'  # 设置文件夹路径
for i in range(1,25):
    merged_df = merge_feather_files(folder_path, f'_{i}')

My machine has 128 cores, I am certain that they are not all utilized for processing.

The reason seems to be the same as in #6865 (to_feather that defaults to pandas)


Use relay as extension

We have to wait until the next release of relay (June 15th) until we can merge this and test it.

New relay is now released, so this can go live!


Suggestion: use WebpackHTMLPlugin

Not sure if it would fit into this framework, but it might make things a little simpler.

I've tried it before. I use it for all of my web projects but I ran into limitations with resolving native / external modules when the HTML page was served by webpack as opposed to loaded directly from the file system. That said, it's been a while since I tried.

@AlexFrazer I havent used this plugin before. Can you elaborate on how it could improve the project? I'm somewhat hesitant on adding new dependencies because we already have too many 😢

Closing do to no response 😢


Adding a method to create an entity naturally

API | Common

@rexbut: Can you rebase this against bleeding?

To fix conflicts, I recreated a new branch : https://github.com/SpongePowered/SpongeCommon/pull/923


Feature/home page text

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


Update version from 0.0.129 to 0.0.130

This is an automated version-update on pyproject.toml.
The workflow was triggered by a succesfull pull request from branch:
With push event SHA of: 792650901be0d50b2f55ec9ec661c8559cd0f705
This action is performed by the Github Action: publish.yml after every accepted pull request. When finished,
this will have published the latest patch of the Extractable codebase to Pypi so it can be pip installed.

updated version in pyproject.toml from 0.0.129 to 0.0.130


sendMessageReaction function's remove parameter get wrong value

    sendMessageReaction({roomId, messageId, reaction, remove}) {                 console.log('<sendMsgReaction>')                 console.log('roomId', roomId)                 console.log('messageId', messageId)                 console.log('reaction', reaction)                 console.log('remove', remove)             },
When I remove a reaction
<sendMsgReaction> roomId 5fc298782963b427d2047c03  messageId 5fc90ff6176568191a178f62  reaction {name: "joy"} remove false
When i add a reaction(emoj)
<sendMsgReaction> roomId 5fc298782963b427d2047c03 messageId 5fc90ff6176568191a178f62  reaction {icon: "😂", name: "joy"}  remove undefined
you can see, when remove emoji, the remove parameter get false, while i add an emoji, the remove is undefined.
also, when add a reaction emoji, the
reaction {icon: "😂", name: "joy"} app.js:2725 remove undefined
I means the icon parameter seems has no use, as in database side, we don't need to store it, we store like this
`
"reactions":{
"joy":[2]
}
`
so the icon is useless.

The icon param is more for information, but it does no harm to keep it.
I cannot reproduce your issue, when I remove an emoji by clicking it I get remove: true.
Please provide minimal component example where I can reproduce this issue.

Closing until I get a reproducible example.


support add tos

support add tos to ping connection

if this feature can be added, i can do it

See previous discussion: https://github.com/go-ping/ping/pull/49


support resque 2.0?

A resque 2.0 was finally released on November 6 2018
The latest release of resque-pool is 0.6.0, with a gemspec that says resque ~> 1.22 -- so 1.x, not 2.0.
It's possible resque-pool would just work with resque 2.0 if the gemspec allowed it, there aren't many changes listed in the resque changes file.
Resque 2.0 does support redis 4, and is presumably needed in order to support redis 4.
Is this project still supported? I'm a bit confused as to the whole resque eco-system these days!

This can be closed after #171 is merged. (but we are missing the release...will create separate issue for that)

Please try v0.7.0, which includes #171. Let me know how it works for you, thanks!


Idea/META: Save and Restore Local and Cloud Backups

What happened (or feature request):

Feature request

What you expected to happen:
The use case here would be someone that is primarily a soloist developing locally and not yet tied to a hosting provider or ready to push to a hosting provider for said project. While they are working, they may want to save backups (specifically the database and/or files) so that they can test something and then quickly revert if it breaks. Additionally, there is often the need for additional insurance to keep backup(s) in the cloud. An example would be a service like Node Squirrel (https://www.nodesquirrel.com/).
The solution here would be a bit more agnostic. The goal here is not a WordPress plugin/Drupal module but rather a way for ddev to accept an API key in order to encrypt/transmit backups to and from something like an S3 bucket (of course we don't have to tie ourselves to just one provider).

This continues to be a request. I think it's useful, but we need to be able to prioritize it against other features. See https://github.com/drud/ddev/issues/767 for a related request.

I like this idea, but I don't see us moving on it for some time.


Option field to ENUM in table CostCenter - was announced in March 2023 as coming asap ...?!

Describe the request
Hello @JesperSchulz,
May I kindly ask what happend to our Option2Enum request?
https://github.com/microsoft/ALAppExtensions/issues/22556
Additional context
We do really need this.
Best Regards
Thomas Hoesel

That is a good question. I just looked into the changes made, and can see that enum 1114 "Cost Center Subtype" was introduced, while the "Cost Center" table has remained unchanged. I will ask the responsible team to look into this!

Yes, this is clearly overlooked, we will fix this mistake asap

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 13980.

Availability update: We will publish a fix for this issue in the next update for release 23.
Build ID to track: 13981.


how to get the bin_file used in extract_feat.py？

I have thousands of pictures,i want to use extract_feat.py to extract features,I want to know how to get the bin file?
THX.

@yl-1993 I have the same query as @Note-Liu. Can you please suggest a way to do it ?

@Note-Liu Did you find a way to get the bin_file ?

Hi @rohan-mistry , I think you may find some clues from bin_loader. The bin file is constructed as a way to store images and labels, where each image is saved to bytes.
Alternatively, using FileListDataset might be more straightforward to read images and labels.

Hi @rohan-mistry , I think you may find some clues from bin_loader. The bin file is constructed as a way to store images and labels, where each image is saved to bytes.
Alternatively, using FileListDataset might be more straightforward to read images and labels.

is there any resnet50 pretrained model for testing?


Feature request: add in alternative references

Hi,
It would be great to have (detailed) description of what is needed to get this package to work with alternative references. For instance, in-house references, or HRC, or UKBiobank, etc.
Thanks and best,
Sander

I totally agree, and I think you might find this notebook useful regarding using other VCF files: https://github.com/slowkow/proxysnps/issues/12#issuecomment-450423002


Events

Hi,
after the update from 2.1.2 Beta to 2.3.0 Beta:
Event have no "then" or "else" and delete one conditon all condition deleted.

I released version 2.3.1-beta with this bug fixed. Thank you!

really nice and a big thanks 👍


Adding image files to build.

Hi
I'm trying to include a logo for use with material-mkdocs in my documentation. I had assumed that as long as I put it in docs/src/assets/images (because I know Lapidary puts such a directory in the build (i.e. docs/build/assets/images) lapidary would include it. However Lapidary does not do this, it simply creates docs/build/assets/two favicon files here. Can I direct Lapidary to include files in the build? Or do I have to manually do this in the make.jl file after calling makedocs()?
Thanks,
Ben.

The build/assets directory is only for use by Lapidary itself, so if you have a src/assets folder with some contents it will get copied over to build/assets but then replaced by Lapidary's own assets folder. This should be made more obvious in the output and docs.
What you should be able to do is call your "assets" directory something like src/resources instead.

What is the reason for having separate assets for Lapidary and the other assets? In mkdocs I don't see anything where it needs the assets folder except for the extra_css where it uses assets/Lapidary.css. As long as people don't overwrite or call something the same name as Lapidary.css or the favicon files, shouldn't it be ok to put stuff in assets? I've seen many web design people use an assets folder for their sites, hence why I tripped up here. Anything that absolutely should be only for Lapidary and not touched might be better in a more specifically named folder?

Yeah, there's no particular reason for it. I'll sort it out later tonight.

82aa5d50719115b899f944db376eecd79764c4f2 allows for content in src/assets to be copied to build/assets without being overwritten. @Ward9250, does that suitably resolve this issue for you?


do not recieve and create a status div

cc @canavandl
This PR no longer creates the status div. If one exists, it will get updated, otherwise, ignored.

Because of the way the autoload template works, and because we kind of make element id do double duty, JS console currently prints
Bokeh: ERROR: autoload.js configured with elementid 'dc3a9bf1-b03f-4608-adc2-59ee05cb00e3' but no matching script tag was found.

when the banner is hidden (but still works fine)


Add checking for arrow function duplicate parameters

This one isn't as clean as my last PR, so feel free to tell that this isn't the right way to do things.
I split out parameter checking into its own function so I could use it in a couple of places. The tricky part for arrow functions is that it should check parameters outside of strict mode.
Fixes #325

Any thoughts on this? @marijnh

Updated.

Updated once more.

Looks great. Merged as c9c215a


No option to specify the map in NavigationLink2D/3D

Tested versions
4.3-stable
System information
Windows 11
Issue description
NavigationLink2D/3D are constrained to a single map in the navigation server. However the NavigationLink2D/3D nodes do not expose any option to specify for which map they are valid. Instead they are just always added to the default navigation map.
Steps to reproduce

Minimal reproduction project (MRP)

Please open a proposal instead, this is very much not a bug but a potentially missing feature and is not handled well by the bug report template (as you can see when you need to ignore the steps listed in it)
So please open a proposal instead


Can we have a loader, or a bridge that extract the name of the model to string?

Like this? Or simple a universal string transfer tool, transfer the name of model to string, then we can feed that information to the Image Saver node. so we can upload our image to civitai the model information will be there?
I tired this method, still not working. The output image reject my custom prefix!

Very like you make this ancient node back to alive! Thank you for maintaining this!

Adding extra nodes just for extracting names as strings is not a solution that scales well.
If a more general approach is offered by KJNodes then please just use that. It seems to be a well maintained node set.
ComfyUI moves fast and there are breaking changes, which is not a bad thing by itself. Duplicating the functionality will only make it break in two places on the next update ;).
Ideally such a functionality should be offered by ComfyUI (frontend) itself - use any node widget values as strings in other strings.

I added a UNetLoaderWithName node.

Welcome to the world of Flux @alexopus I also see you got the good model for start. You'll be sticking with it for a long time. :P
Btw, in case of Flux, the Image Saver node doesn't require the Negative field as Flux gets all the info from a single blob of text.
It's just a gross field taking up space on the node. Currently using a primitive to just pass nothing to it to hide the field.
Also, what is that snake (🐍) node from? I see everyone using it, but Idk what do I have to install. I see it has value and I know it may come in handy.

It's from here: https://github.com/pythongosssss/ComfyUI-Custom-Scripts/


About the batchnorm in CoordAttention

Hello, Thank you for your excellent job about the attention. I am a little puzzled about the code.  compared to the Senet,  there is a  batchnorm operation  in the CoordAttention. Is it necessary for the attention mechanism?   In addition,  Is it necessary that  I replace the ReLU operation  (the self.relu(x + 3) / 6 ) with the ordinary ReLU, when the input are normalized between -1 and 1 .

In mobile network training, it would be better to use ReLU6 or Swich, which is smooth. MobileNetV3 has demonstrated this.


[Bigtable] Table and column family admin APIs

Table create, delete and list APIs
Column families create, update, delete and bulk modification
Table and Column families mock unit and acceptance test cases
Refactored longrunning job, remove redundant job methods and move into base job class LongrunningJob

@blowmage, I'm ok with this being merged.  Once all of the pull requests are done, I'll do a thorough review before we mark this client as alpha


Bug: Files won't activate

Description
I'm trying to activate Files using the Win+(Number) combination in Taskbar but it won't work.
The only workaround for me was to click the icon, but even when clicking the icon still needs two clicks because of the beep sound at the first click. (a beep will trigger when Files is minimized and you clicked the icon)
Steps To Reproduce

Focus on another window to make the Files app in the background
Press Win+(the index where Files is in the Taskbar, in my case Win+3

A default Windows beep sound will play but the app will not activate

Steps to fix

Prevent the Windows beep sound
Activate the app immediately after clicking or using the hotkey

Files Version
<IP_ADDRESS>
Windows Version
10.0.22623.875
Relevant Assets/Logs
N/A

Is Files the default file manager? What happens if you toggle it off and on again?

Is Files the default file manager? What happens if you toggle it off and on again?

Yes, it is already set as default. Toggling off and on seems to have no effect

I forgot to write that the problem only occurs when the app is minimized at first and then you hit that Win+ hotkey or click the icon. Both will produce a beep sound.
The only difference is that using the hotkey would not activate the app at all while clicking it would need you to hit the icon twice before it actually restores from being minimized.

What happens if you toggle it off and on again?

I just encountered a new one. This happens after toggling the default file manager off and ON and hitting Win+E.

I had to install the stable version and toggle the setting there to get it working again. Also, the import settings won't work in the latest preview as I have failed to restore a 2.3.57.zip backup
Will post a separate issue. :)

That's a known issue 👍

That's a known issue 👍

How about the import settings?

Also, the import settings won't work in the latest preview as I have failed to restore a 2.3.57.zip backup

Can you reproduce this if you export and import a new backup? Please open a bug report if it's still an issue.

#10194. Yair said it would be a WinAppSdk bug or Windows. would this still be the case?

Seems Files specific but hard to know


saving subscriptions across pubsub connections

I am using python 3.8.2, Redis server version=6.0.5 and redis-py 3.5.2.
I am trying to persist subscriptions even after the pubsub object was closed. I am afraid it is not very clear to me, the scope of the PubSub objects, in one hand it shouldn't be shared among several threads, on the other I can't persist changes to it.
>>> import redis
>>> r = redis.Redis(host='localhost', port=6379,db =0)
>>> ps =r.pubsub()
>>> ps.subscribe('ch1')
>>> r.pubsub_numsub('ch1')
[(b'ch1', 1)]
>>> r.bgsave()
True
>>> r.save()
>>> ps.close()
>>> ps = pubsub_numsub()
>>> r.pubsub_numsub('ch1')
[(b'ch1', 0)]

Can published messages also be persisted/loaded?

pubsub.close() closes the underlying connection to the Redis server and unsubscribes from any channels and/or patterns that you've subscribed to.
I'm confused as to why you'd want to close the pubsub object yet stay subscribed to channels and/or patterns. Can you explain what you're trying to do?

I would like to use redis-py in a webserver, to create a notification system between users; I am trying to persist subscriptions to disk. So in case the server goes down, I am in another thread, or any other case in which I don't have access to the PubSub object; I am still able to fetch those subscriptions.
pubsub.close() was just a way to simulate, this lack of access to the pubsub object.
If this is not possible on the redis-py level, I can still store the subscription relationship between users in a relational database and subscribe then back again, if I am in another thread. But this seemed like an overkill.

Correct, you need to keep a pubsub object open to continue to receive messages.
If these messages are important I would suggest using a persisted data structure like a list or stream. In the past I’ve created a list key per user. When the web server fetches the message list for a user, you can truncate the user’s list. I’ve found this far more effective and easier to work with.
Pubsub is unreliable by design.

Streams definitely seem like a better fit here. See the docs for more: https://redis.io/topics/streams-intro
There's also a free course from Redis Labs on how to use Streams: https://university.redislabs.com/courses/ru202/

Thanks @andymccurdy and @abrookins, for the clarification and the quick reply. I guess this issue can be closed.


Unexpected behavior on fast interpreter mode

Subject of the issue
The test case is expected to lead to an exception "unreachable". However, the warm on fast interpreter mode fails to raise the exception. Is it caused by the optimization in the fast interpreter?
Test case
tt_unreachable.zip
Your environment

Host OS: Ubuntu 20.04
Commit Id: f378edc408382b7bcf517b463d4a54948739075b

Steps to reproduce

build WAMR on fast interpreter mode
execute the command

iwasm --heap-size=0 -f "to_test" <test_case>

Expected behavior
Exception: unreachable

Actual behavior
-1.861157e+19:f32

@erxiaozhou thanks for spotting the issue! I submitted PR #3482 to fix it, please try again.

Close the issue as it was fixed.


DirectSessionRegistrar not found in libraries

tensorflow branch r0.7    android-ndk-r10e    bazel-0.2.1
I build the android:tensorflow_demo with the command below:
=> bazel build //tensorflow/examples/android:tensorflow_demo --verbose_failures
The build is successful, and it generate some files:
=> ??????????????.apk
=> tensorflow_demo.so
=> libandroid_tensorflow_lib.a
=> libre2.a and so on
I want to use Windows Android SDK/NDK to develop Android APP with these static libraries.
When I initialize the tensorflow session, it prompt that direct session not found.
Then I browse the code tensorflow\core\common_runtime\direct_session.cc.
At the tail, I find:
"""
class DirectSessionFactory : public SessionFactory {
public:
DirectSessionFactory() {}
Session* NewSession(const SessionOptions& options) override {
std::vector<Device*> devices;
DeviceFactory::AddDevices(options, "/job:localhost/replica:0/task:0",
&devices);
return new DirectSession(options, new DeviceMgr(devices));
}
};
class DirectSessionRegistrar {
public:
DirectSessionRegistrar() {
SessionFactory::Register("DIRECT_SESSION", new DirectSessionFactory());
}
};
static DirectSessionRegistrar registrar;
"""
nm libandroid_tensorflow_lib.a | grep "???"
nm direct_session.o | grep "???"
I can grep DirectSessionFactory from libandroid_tensorflow_lib.a/direct_session.o, but not DirectSessionRegistrar.
It seems that libandroid_tensorflow_lib.a/direct_session.o does not contain DirectSessionRegistrar.
What can i do?

@mrry, any idea here?

Hi, did you try the alwayslinks=1 and linkopt workarounds suggested in https://github.com/tensorflow/tensorflow/issues/2047?
Another potential solution is to add an explicit reference to DirectSession in the code to make sure it's not getting stripped out.
It'd be nice to know the root cause of why this is happening to people occasionally, of course, but I've never been able to reproduce.

Another thing worth checking: how much free disk space and memory does your system have? I've seen cases where Bazel "forgets" that it has successfully built entire .so libraries when running low on resources -- I wouldn't be terribly surprised if the same thing happened for .o files.
Can you do a blaze clean, then try building with --jobs=1 -s and attach the entire build log here please?

I'm not a linker expert, but it looks like the problem might be discussed in this SO question: Static initialization and destruction of a static library's globals not happening with g++. Perhaps there are some linker options that you need (e.g. -Wl,--whole-archive as suggested in this comment)?

A colleague suggests adding the following option to your build command for the code that uses libandroid_tensorflow_lib.a: linkopts=["-Wl,-no-as-needed"].

@zheng-xq @andrewharp @mrry
It my fault, now it's ok.
I develop under Windows, and I use Android.mk to build.
I must use LOCAL_WHOLE_STATIC_LIBRARIES instead of LOCAL_STATIC_LIBRARIES to link these libraries. And it also depend on android-ndk-r10e, just like bazel build.
LOCAL_WHOLE_STATIC_LIBRARIES :=
android_tensorflow_lib
protobuf re2
protobuf_lite
protos_all_cc
Now I can develop APP under Windows with Eclipse and Android.mk.
thanks.


[FLINK-6164] Make ProcessWindowFunction a RichFunction

This PR modifies the ProcessWindowFunction and ProcessAllWindowFunction classes to extend AbstractRichtFunction.
The existing Rich* versions were deprecated, and all usages of them replaced.

I've removed Serializable, adjusted the @Deprecated annotations like in #3816 and fixed the unused imports (there were also some in ProcessAllWindowFunction.scala).

Ha! I didn't think of the annotations.
This looks good to merge now!
Thanks for fixing these so quickly! 😃

merging.


Upgrade to datahub cli v0.14.1

We should ugrade to the latest version of the cli to match the version of datahub we are now using
Before doing this we should establish our ingestions still work as expected using this version in the dev environment.
upgrade PR https://github.com/ministryofjustice/data-catalogue/pull/268

v0.14.1 ingested all but 1 failure to dev - this is the same as the latest preprod run on v<IP_ADDRESS> cli https://github.com/ministryofjustice/data-catalogue/actions/runs/11067746924/job/30751759649
and is related to cadet data rather than cli version so will merged upgrade


Update backup handle generation mechanism

Issue/Feature Description:
In current kahu design, volume service uses snapshot handle as the handle to keep track of backups and its corresponding snapshots. In case of csi snapshots, since the snapshot handle mechanism can vary across different providers, having a common csi snapshotter may not work seemlessly with different storage providers
Why this issue to fixed / feature is needed(give scenarios or use cases):
So new proposal is to use snapshot content name as the backup handle to keep track of backup and snapshot mapping.
This can be consistently interpreted across backup and restore flows
How to reproduce, in case of a bug:
Backup across different csi drivers
Other Notes /  Environment Information: (Please give the env information, log link or any useful information for this issue)

Resolved through PR https://github.com/soda-cdm/kahu/pull/186


Missing calibration_handeye_result.pkl

Hi,
Thanks for your great work!
I've encountered a few issues running the training and data processing script, where calibration_handeye_result.pkl is missing from your dataset release. Is there any plan to release those calibration files?
Thank you.

I got the same issue.

The calibration data was not included in the previous release. You can access it via calibration. Thank you!


Support defines and other settings that come through dump.json

Example: https://github.com/apple/swift-crypto/blob/main/Package.swift#L82-L89
WIN32_LEAN_AND_MEAN define comes through as:
    {
      "dependencies" : [

      ],
      "exclude" : [
        "hash.txt",
        "include/boringssl_prefix_symbols_nasm.inc",
        "CMakeLists.txt"
      ],
      "name" : "CCryptoBoringSSL",
      "resources" : [

      ],
      "settings" : [
        {
          "kind" : {
            "define" : {
              "_0" : "WIN32_LEAN_AND_MEAN"
            }
          },
          "tool" : "c"
        }
      ],
      "type" : "regular"
    },

Fixed by #92.


Doesn't have a minified JS version

When is next release planned. I'm very interested in using this minified version :+1:

Sorry, I didn't realize you were waiting for a release when I committed the changes.
I'll pack a release today.

Done. As well as updated npm and bower repos. Tell me if there's a problem still.

I was putting this into my app by using bower. So the new version works great now. Thank you very much! :+1:


Error: No value accessor for ''

Hi guys,
I am trying to get ng2-tag-input working but is giving me the following error:
platform-browser.umd.js:2306 Error: No value accessor for ''
I have done exactly step by step as explained in github.
this is my code:

Input tag that allows max 3 tags

        <tag-input [ngModel]="['Javascript', 'Typescript']"
                   [maxItems]="3"
                   placeholder="Insert a new item"></tag-input>

@Component({
    moduleId: module.id,
    selector: 'jobform',
    directives: [FILE_UPLOAD_DIRECTIVES, TagInput],
    templateUrl: 'jobform.component.html'
})

Any ideas and help will be much appreciated, thanks.
Kind Regards

Hi @simoazzo!
A few questions:

which Angular version are you running?
did you configure Angular like this https://github.com/Gbuomprisco/ng2-tag-input#angular-2-configuration-rc3?

Thanks

Hi @simoazzo, did you solve this?


[BUG] Poppins is wrong! Flutter is using an old(?) version of the font.

Poppins has wrong ‘?’ (possibly a previous font version?!). Debugged by intercepting the call.

DartPad:
https://dartpad.dev/?id=3640b6eb43e760fd88d10fb22aea90e5
Codepen:
https://codepen.io/bernardoferrari/pen/LYzyXKW?editors=1100
Diff (exactly same spacing, everything is the same, code copied from Flutter web renderer):

This sounds similar to an issue I created a while ago.
Have a look at #163.
In found out that the font weight is different when you use the flutter plugin compared to downloading the font manually from Google fonts and import it locally.

In this case it is not only the FontWeight but the font itself. See how "?" changes between Flutter and the official Google Fonts font.

Internal bug is b/212679312


Memory corruption via Array.copy_to

There's bugs in Array's copy to where it doesn't validate input
for example:
actor Main
  new create(e: Env) =>
    var src: Array[U8] = [1]
    var dest: Array[U8] = [11; 1; 2; 3; 4; 5; 6]
    try
      e.out.print(dest(0)?.string())
    end
    src.copy_to(dest, 11, 0, 10)
    try
      e.out.print(dest(0)?.string())
    end

prints 11 and then prints 0. copying from an out of index src_index shouldn't be changing the destination array, it should be an error.
To fix, copy_to needs to become partial.
As part of this change, the slice method that use copy_to be switched to using _copy_to so that it isn't required to become partial UNLESS, slice actually needs to be partial. We should validate that it doesn't.
Some things that we should be adding copy_to tests methods for:

src index is outside of range (should be a bug now)
dest index is outside of valid range (should be handled correctly now)
length to copy goes outside of valid range (should be a bug now)
array was never initialized (so _ptr is null) (see example-ish test in #4173)
array was initialized but is now empty (_ptr isn't null, but size is 0) (see example-ish test in #4173)

Here's some example code that shows the length bug:
actor Main
  new create(e: Env) =>
    var src: Array[U8] = [1]
    var dest: Array[U8] = [11; 1; 2; 3; 4; 5; 6]
    try
      e.out.print(dest(3)?.string())
    end
    src.copy_to(dest, 0, 0, 10)
    try
      e.out.print(dest(3)?.string())
    end

I don't think it is necessary for copy_to to become partial. Instead, a check is needed that the len parameter  doesn't exceed _size

I don't think silently not doing this is a good API here. Most of Array if you overshoot an index are partial...
see

read_U8
read_U16
etc for read's
update_U8
etc for update's
insert
delete

Note, it appears that copy_from has similar issues.

I don't think silently not doing this is a good API here. Most of Array if you overshoot an index are partial...
see

read_U8
read_U16
etc for read's
update_U8
etc for update's
insert
delete

This is exactly how for example remove operates. If you ask for too many to be removed it silently only removes as many as it can. I am not sure that I would agree that doing it so would be bad.

Looks like this has been a bug since pretty much the very beginning (8 years ago).
I would take the position that silently truncating the range is the most consistent thing to do here, rather than making the method partial.
The general precedent in the String and Array methods seems to be:

If dealing with a single element index that is out of range, raise an error
If dealing with a span of element indexes, part or all of which are out of range, silently truncate the span to be in range (possibly making it empty, if all of it was out of range)

If you were going to take the position that the latter class of cases should also be partial, you're going to need to make a lot more of the methods partial than just copy_to and copy_from.

Re: "dest index is outside of valid range (should be handled correctly now)"
It's worth noting that requesting a destination index out of bounds, in a way that leaves "gaps" in between, we might also encounter memory corruption:
class Counter
  var value: U32 = 0
  new create(v': U32) =>
    value = v'

actor Main
  new create(e: Env) =>
    let src = [Counter(0)]
    let dest = [Counter(1)]
    src.copy_to(dest, 0, 2, 1) // past the range
    try
      e.out.print(dest(0)?.value.string()) // valid memory
      e.out.print(dest(1)?.value.string()) // invalid memory
    end

Yeah, we definitely don't want to leave gap in between in whatever the desired behavior for this method is.
We discussed this a lot on the sync call and my basic thought is:
I don't mind removing copy_to and/or copy_from, and I also don't mind making them partial, as long as we also provide a new method (e.g. something like append_array) because I think the dst_idx is where the main need for partial-ness comes from, and there's no example I could find on GitHub out in the wild that needed to copy to any destination index besides just the end of the destination buffer. These use cases can't currently use append because it's non-optimal due to not requiring the source to be contiguous memory. If we add a new method like append that uses a contiguous source buffer, I think it serves all current use cases of copy_to and copy_from in the wild.

After discussing this topic with Sean today, here's a path forward that at least he and I are in agreement on (others here can chime in as well).
This mostly matches what @ergl and I discussed in the sync call 10 days ago, so to hear more about the rationale, listen to that recording.
Make the copy_to method partial, but only one case would result in an error being raised:

When some part of the source slice is out of range, the number of elements copied is silently truncated so that it only copies in-range bytes from the source.
Instead of returning None, return USize indicating the actual number of elements that were copied (after the aforementioned truncation)
When the destination slice is fully within range, the copied elements replace the elements currently in that destination slice
When the destination index is equal to the current size of the destination array, the copied elements are appended, increasing the array's size by the copied amount
When the destination slice is only partly within range, the in-range part of the copied elements replace existing elements and the rest are appended, increasing the destination size
When the destination index is beyond the range, such that if we did expand the size and do the copy there would be a gap of "undefined" elements before the copied elements, then copy_to will raise an error (this is the only case that makes it partial)

Add a new copy_append method which does not allow specifying destination index, and thus would be a non-partial alternative.
Make copy_from partial, remove the artificial U8 restriction, and implement it in terms of copy_to so that the semantics are the same as copy_to (but mirrored)
Add a new copy_from_append method, which mirrors copy_append in the same way.

This sounds like a good way forward -- I closed #4173.

@stefandd would you be interested in implementing the above? You already did a decent amount of work towards the tests and implementation.

@jemc @SeanTAllen I am happy to, for a start, work on the new copy_to and copy_from functions. The final version of copy_to I had in my PR met the first 5 out of 6 description points above already.

Sounds great, if you're willing to continue on that path. Thanks!

@jemc

Add a new copy_append method which does not allow specifying destination index, and thus would be a non-partial alternative.

Does copy_append allow you to specify a source index and a length? I guess so, but I wanted to be sure.
If so, and with the addition of copy_append, I feel like keeping the "appending" cases in copy_to / copy_from is a bit redundant.
If we're making a breaking change, I'd prefer to specify that any out of range scenario in copy_to / copy_from results in truncation, and leave any appending operations to the new methods being introduced. That way, we avoid marking copy_to partial. For example:

When the destination index is equal to the current size of the destination array, the copied elements are appended, increasing the array's size by the copied amount

When the destination slice is only partly within range, the in-range part of the copied elements replace existing elements and the rest are appended, increasing the destination size

When the destination index is beyond the range, such that if we did expand the size and do the copy there would be a gap of "undefined" elements before the copied elements, then copy_to will raise an error (this is the only case that makes it partial)

These would be modified to the following:

When the destination slice is only partly within range, the in-range part of the copied elements replace existing elements and the rest are truncated.

When the destination index is equal to the current size of the destination array, or the index is beyond the range, nothing is copied, and the return value is zero.

That's an interesting point, and I definitely like the reduced conceptual complexity of that approach (especially that it avoids the mental overhead of remembering which cases truncate and which cases raise an error), though I think it has two downsides to mention:

There wouldn't be any method that serves the use case of copying into a place "near" the end of the destination, using some elements to overwrite, and the rest of the elements being appended to extend the destination. There would be no way to do this in just one copy - you'd have to do first one copy_to for the overwrite part and then one copy_append for the following append part.

This would become a silently breaking change rather than a "fail at compile time" breaking change. That is, in the approach I outlined above, we'd be making copy_to and copy_from partial, which would force everyone to either handle the error case or, more likely, switch to copy_append and copy_from_append (because all the real-world cases I know of are doing the append case). In contrast, the approach you outlined would not force any compile-time failures, and would instead just start failing at runtime for everyone who didn't pay attention to the release notes and update their program.

If we want to take your approach then we may want to consider at least changing the function names for force a compile-time failure so that we don't make a silently breaking behavioral change to an existing function.

@ergl @jemc
Please indicate if/when a consensus has been reached among the core developers on how to proceed..

@jemc let's you and i discuss this issue and come to a resolution so we can unblock this and hopefully @stefandd is still interested in fixing it.

@SeanTAllen I am happy to take a stab at implementing a proposed change to copy_to and copy_from

We discussed this in sync today. @SeanTAllen and I still want to go with the partial approach that I outlined in this comment: https://github.com/ponylang/ponyc/issues/4174#issuecomment-1228815465
@ergl - what do you think? Do you think you can align with this same consensus? Or do you still prefer the alternative you outlined in this comment: https://github.com/ponylang/ponyc/issues/4174#issuecomment-1229906875

@jemc Your proposal sounds good. I've come around on my idea since I posted it, your point about having a silent breaking change was a good one.

I believe we have a quorum of agreement now. @stefandd are you still up for working on this?


Integrate Biometric Authentication in Settings

[ ] Implement Biometric Authentication: Integrate biometric authentication in the settings, similar to the implementation in the Keeper app, for enhanced security.

Android -
Done
https://github.com/bithyve/bitcointribe/assets/61345820/f672c566-4c31-4e92-9ad8-aa2b9f1186db
https://github.com/bithyve/bitcointribe/assets/61345820/ce058598-7a7c-42d5-8edd-1a1f8c3c2125

Need to work little more not smooth as compared to keeper
https://github.com/bithyve/bitcointribe/assets/107177634/27f69103-af0f-48e6-adb0-1e707cafa87f

@Deveshshankar ,
As we had discussion - We've identified a UI issue in this feature, but we'll hold off on making changes until the UX design is finalised. Once we have the final UX, we'll proceed with addressing the UI issue accordingly.


Unexpected error running Liquibase: Backup Server: <IP_ADDRESS>: The value of 'allocated pages threshold' has been set to 40%.

Hi,
I am Currently using Liquibase Community 3.10.2 by Datical on local window machine . Currently i am using Sybase ASE 16.0 server. It's succesfuly connected to sybase but throws error  during sybase backup.
Error:
Unexpected error running Liquibase: Backup Server: <IP_ADDRESS>: The value of 'allocated pages threshold' has been set to 40%.
[Failed SQL: (417201) dump database EQRMS to '/home/ec2-user/EQRMS.dump']
I am following below command :
liquibase --classpath=.\lib\jtds-1.3.1.jar --changeLogFile=.\examples\sql\samplechangelog.h2.sql --url="jdbc:jtds:sybase://<IP_ADDRESS>:5002/"  --username="sa" --password="sybase123"  update
My sql cmd
--liquibase formatted sql
--changeset bob:3
dump database EQRMS to '/home/ec2-user/EQRMS.dump'
go
please give solution.
I have attached output snip , please find
thanks in advanced.

Any solution ..

Hi @sukhdevkhade Thanks for writing up this issue.  This may belong in the SAP HANA extension.  Have you tried using the extension https://github.com/liquibase/liquibase-hanadb
If not, I think the issue may be better suited there.
Is this a command that can be executed via a jdbc connection or needs a sql type client connection?

Hi @molivasdat
Currently I am using Sybase ASE 16.0.
I have successfully connected to Sybase  DB using current jar.
Some operations are also successful, such as creation tables, databases etc.
But it throwing an error during backup sql statement.
My sql statement is :
--liquibase formatted sql
--changeset bob:3
dump database EQRMS to '/home/ec2-user/EQRMS.dump'
go
please let me know anything else..
Thanks.

Hi @sukhdevkhade, sorry for the late response.
This issue is actually coming from SyBase, not from Liquibase. The SQL code in your changeset isn't being recognised as valid by SyBase.
I'd recommend checking the database logs in order to figure out the source of the problem.
Thank you
Tatiana


Beta - Declare the Doctype of an HTML Document - Guide Page 404 Not Found

Challenge Name

https://beta.freecodecamp.org/en/challenges/basic-html-and-html5/declare-the-doctype-of-an-html-document
Issue Description

When selecting the 'Get Hint' button, it re-directs you to the guides but there is no guide page for this challenge.
Returns Page not found.
https://guide.freecodecamp.org/certificates/declare-the-doctype-of-an-html-document
Browser Information

Browser Name, Version: Chrome 64.0.3282.186 64-bit
Operating System: Windows 10 Pro 64-bit
Mobile, Desktop, or Tablet: Laptop
Your Code

Screenshot

Reason for close, see  https://github.com/freeCodeCamp/freeCodeCamp/issues/16817