Is RUN_POLAR redondant with RUN_POSTALL in .ini files ?

Run the polar recipes
RUN_POLAR = True
Run the postprocessing recipes
RUN_POSTALL = True

These are completely different.
You can find the sequences here:
http://apero.exoplanets.ca/auto/recipe_sequences/spirou/sequences.html
Note that the names (POLAR and POSTALL) depend on the sequence being used in the run.ini files
But I think you are looking at a run.ini will the full_seq thus you look at this page:
http://apero.exoplanets.ca/auto/recipe_sequences/spirou/sequence_full_seq.html
ORDER | RECIPE | SHORTNAME | RECIPE KIND | REF RECIPE | FIBER | FILTERS | ARGS | KWARGS

35 | apero_pol_spirou.py | POLAR | polar-tcorr | No | AB | KW_DPRTYPE: POLAR_FP, POLAR_DARK |   | –exposures=[TELLU_OBJ]

36 | apero_postprocess_spirou.py | POSTALL | post-all | No | – | KW_DPRTYPE: OBJ_FP, OBJ_DARK, POLAR_FP, POLAR_DARK | {files}=[DRS_PP] |

So

RUN_POLAR runs the apero_pol_spirou.py recipe
RUN_POSTALL runs the apero_postprocessing_spirou.py recipe

I reminder that apero_postprocessing just produces the CADC outputs from the rest of the reduction (so just the packaging of the e.fits, t.fits, s.fits, v.fits)  it does not run any code to run the CCF, extraction, telluric correction etc

Thanks Neil, Indeed I'm using the full_seq


How do you plan to use the library?

Hello everyone,
I know most of the use cases of this library will be quite classical: conversation between users. However, some of you may have ideas of what they need that we didn't think of.
Please don't hesitate to propose ideas about this library, about how you would like it to work.

i think its a good idea to formulate a mission statement and also be
strict about it in the sense that you reject features that try to make
the library do something completely different than the mission
statement. it should be focussed on some tasks. if you end up doing all
imaginable kind of messaging, it can become hard to maintain and very
hard to get into as a new user of the library. the more flexibility, the
more complexity and the steeper learning curve.
if there is some small detail that gives a lot of flexibility for third
party code to do other things than intended, that makes sense. but
adding whole new layers of functionality is probably not.

I completely agree, we have to keep the library clear about what it does: simple messaging between users.
On the other hand, I reviewed the issues of the bundle and some ideas were interesting while staying in the scope of the library. For instance, the tagging system is a generic and simple way to provide several features (directories, search, conversations status, etc.) and it does fit in the library.

Hi @tgalopin just don't forget this statement "...simple messaging between users." after 1k commits ;)
And about other hand - provide file attachment ability in core objects (not full functionality implementation, just possibility to implement it in your own project if you will need it).

One more thing I think we should keep in mind: often the SocialNetwork (or Friends) functionality is the reason when the project need the messaging system. So there have to be smart validation layer where developers can put theirs validators such  as "isFriend, "isBlocked" etc.

@ionafan2 I don't think we need anything special about the validation. People would just register additional constraint for the Symfony validator for the classes (assuming we rely on the Symfony validator). this way, they can add whatever validation rule they want without any change in the library.
@tgalopin regarding tagging, the question is whether this needs to be in the library, or whether the library should just have enough extension points to allow having it built on top of it. In some cases, it is more maintainable to provide an extension point than to build the feature directly (and the extension point can be used to build several features generally)

@stof yes, you are right. I'm just notice you that the way when adding new behavior is better than extending or changing existing in example on validations, it can be filters or something else

For the validation I think @stof is right, adding validation rules is just adding custom methods in the models child classes. We don't want to take care of validation in the library.
I'm not totally sure right now if the tagging system should be in the core library or not. A dilemma I have is how to deal with read/not read conversations status. As discussed in https://github.com/FriendsOfSymfony/FOSMessageBundle/issues/112, I think we should store this state in the entity linking a conversation and a user (ConversationPerson) and not on the messages (like in the bundle). I think tagging could be a way to store this status.
On the other hand, we probably want more precision than just a tag, like what is the last message seen (or better, what is the last visit date, to avoid a link to a message). Storing the last visit date let the developer redirect the user to the right page in the conversation if the conversation is paginated for instance.
In the end, I think the best would be to have in the ConversationPerson object a field $lastSeenDate, the repository could have a method to retrieve the last message seen by a given person in a given conversation and the tagging system could be externalized (something optionnal) as @stof proposed. I'm thinking of something like:
$conversation = $repository->getConversation($id);
$lastMessageSeen =  $repository->getLastMessageSeen($conversation, $person);

// We can find the right page to redirect the user to if we paginate conversations

What do you think of this?

I started an issue for the missions of the library to summarize the different discussions we had in the different topics: https://github.com/FriendsOfSymfony/FOSMessage/issues/5.
If you could give your input, that would be great!


nx-remotecache-custom dependency version is declared as ^17.0.0, should be ^5.0.0

Hi, as in the title, the dependency to nx-remotecache-custom is currently wrong
Change got introduced here, probably because of automatic migration:
https://github.com/robinpellegrims/pellegrims/pull/249/files

Actually, nx-remotecache-custom switched to mirroring the version of nx, so they went from v5 to v17 when upgrading to nx v17.
See their changelog on  https://github.com/NiklasPor/nx-remotecache-custom/blob/main/CHANGELOG.md


feat: add GPU operation

Tested on g4dn.xlarge with AMI ami-05bd609552ea2b875 (i.e. generated by https://github.com/sustainable-computing-io/kepler-metal-ci/actions/runs/11746812304/job/32727360505)


Support podman based distributions

Distributions like Fedora, Red Hat Enterprise Linux provide podman instead of docker.
podman has high compatibility with docker but the implementation is different. podman doesn't have a daemon that runs as root. This allows regular users to launch containers without elevating their privileges.
It would be nice if tye supports podman based distributions and is able to work in a rootless mode. (The network creation currently performed by tye requires root.)
I can help implement this.
cc @davidfowl

I think this makes sense to do. We need to figure out what to do about some of the options in the tye.yaml. They have the name docker 😄. I'm also interested to see how compatible the APIs are (I know what podman is but I haven't looked deeply enough to understand the differences). Maybe throw up a draft PR with the implementation so we can learn more and then we can iterate from there.


Add support to run launchpad commands from an Azure AD application

Launchpad initialization is only supported at the moment when executed from Azure user. This issue adds the support to deploy the launchpad from an Azure AD Application

@LaurentLesle Any estimate when this would be done?


Order of announcements

#Summary
The announcments in the bank dashboard are currently showing with the ones with an expiry date first, then the ones without.   They should show in reverse creation date order.
Details
Example
This is what it currently looks like (on prod)

In this case,  the order should be May 7,  April 30, April 16
Recreation

as superuser, create 2 announcments - one with an expiry date, then one without.
sign in as a bank
The one without an expiry date (which was created most recently)  should be on top.

Criteria for completion

[ ] functionality as above
[ ] test to support

May I please take this one?

For clairification: you would like the announcements without an expiry date shown first and all in descending created at order?

I want them displayed in reverse creation date order whether or not they have an expiry.


Required qual in where only works when the same field is in the column list

See in the example below how removing the id column from the columns causes the qual in the where clause to stop working.
Interesting to me that adding id to the column list causes a qual on the user_id field to work...
> select id, username from twitter_user_follower where user_id in (select id from twitter_user where username = 'steampipeio')
+------------+---------------+
| id         | username      |
+------------+---------------+
| 853536468  | daleburncock  |
| 130100271  | NLNils        |
|<PHONE_NUMBER> | _cloudkate    |
| 14372877   | dlatkins      |
| 825766640  | HamelHusain   |
| 160129663  | vatsparth     |
|<PHONE_NUMBER> | LtB1991       |
| 115454948  | raj_mohanty   |
| 348925658  | ibm2100       |
|<PHONE_NUMBER> | turbothq      |
| 8092452    | nathanwallace |
|<PHONE_NUMBER> | TheFatJohnny  |
+------------+---------------+
>
> select uf.id, uf.username from twitter_user_follower as uf where uf.user_id in (select id from twitter_user where username = 'steampipeio')
+------------+---------------+
| id         | username      |
+------------+---------------+
| 853536468  | daleburncock  |
| 130100271  | NLNils        |
|<PHONE_NUMBER> | _cloudkate    |
| 14372877   | dlatkins      |
| 825766640  | HamelHusain   |
| 160129663  | vatsparth     |
|<PHONE_NUMBER> | LtB1991       |
| 115454948  | raj_mohanty   |
| 348925658  | ibm2100       |
|<PHONE_NUMBER> | turbothq      |
| 8092452    | nathanwallace |
|<PHONE_NUMBER> | TheFatJohnny  |
+------------+---------------+
>
> select uf.username from twitter_user_follower as uf where uf.user_id in (select id from twitter_user where username = 'steampipeio')
Error: 'List' call requires an '=' qual for column: user_id
>

the problem is the query plan generated for each query
if id is included in the selected coulmns,  the planner selects a Nested loop, first doing a foreign scan on twitter_user, grouping, then a forieng scan on twitter_user_follower with the filter (user_id = twitter_user.id)
Therefore it provides the qual value of user_id which we can pass to the plugin

+-------------------------------------------------------------------------------------------------+
| QUERY PLAN                                                                                |
+-------------------------------------------------------------------------------------------------+
| Nested Loop  (cost=20000000002500.00..20000000002800.02 rows=500000 width=64)                   |
|   ->  HashAggregate  (cost=20000000002500.00..20000000002500.01 rows=1 width=200)               |
|         Group Key: twitter_user.id                                                              |
|         ->  Foreign Scan on twitter_user  (cost=0.00..20000000000000.00 rows=1000000 width=200) |
|               Filter: (username = 'steampipeio'::text)                                          |
|   ->  Foreign Scan on twitter_user_follower  (cost=0.00..300.00 rows=1 width=300)               |
|         Filter: (user_id = twitter_user.id)                                                     |
+-------------------------------------------------------------------------------------------------+

if id is not included in the columns, the planner selects a Nested Loop Semi Join
This means that within each loop iteration, it first does a foreign scan of the outer table, twitter_user_follower, with no quals and then a foreign scan on the inner table (twitter_user) to do the filtering
Thus, no quals are passed to the FDW, or the plugin.
explain select username from twitter_user_follower where user_id in (select id from twitter_user where username = 'steampipeio')
+----------------------------------------------------------------------------------------------------+
| QUERY PLAN                                                                                |
+----------------------------------------------------------------------------------------------------+
| Nested Loop Semi Join  (cost=0.00..20000100024999.00 rows=500000 width=32)                         |
|   ->  Foreign Scan on twitter_user_follower  (cost=0.00..20000000000000.00 rows=1000000 width=200) |
|   ->  Foreign Scan on twitter_user  (cost=0.00..200.00 rows=1 width=200)                           |
|         Filter: ((username = 'steampipeio'::text) AND (twitter_user_follower.user_id = id))        |
+----------------------------------------------------------------------------------------------------+

solution is to add in optional key columns to the path keys we generate

we need to better understand how we can influence the query plan from the fdw


Fix implicit integer conversion reported by Clang 8

When building on Ubuntu 18.04.2 LTS (Linux 4.15.0 x86_64) with Clang 8, the compilation fails:
$ CC=clang-8 ./tools/build.py --clean --debug --profile=es2015-subset --lto=off
jerry-core/ecma/operations/ecma-container-object.c:72:45: error: implicit conversion loses integer precision: 'lit_magic_string_id_t' to 'uint16_t'
      (aka 'unsigned short') [-Werror,-Wimplicit-int-conversion]
  map_obj_p->header.u.class_prop.class_id = lit_id;
                                          ~ ^~~~~~
jerry-core/ecma/operations/ecma-container-object.c:672:42: error: implicit conversion loses integer precision: 'ecma_pseudo_array_type_t' to 'uint8_t'
      (aka 'unsigned char') [-Werror,-Wimplicit-int-conversion]
                                         iterator_type,
                                         ^~~~~~~~~~~~~

This commit fixes the issue by adding explicit conversions. After the change, the build succeeds.

lgtm. In the future imho we would need to add a static assert check that both type's values can fit into its given type (uint16/uint8) range.


Needless import to has_inherited_table in Customizing -> Model Class code example

Expected Behavior
Example in flask-sqlalchemy/docs/customizing.rst Model section shouldn't contain needless imports
Actual Behavior
Example in flask-sqlalchemy/docs/customizing.rst Model section contains needless import to has_inherited_table which can cause confusion
from sqlalchemy.ext.declarative import declared_attr, has_inherited_table

Environment

Flask-SQLAlchemy version: 706982b (commit id when reported)

Also I don't know if this example is still valid:
Part of my model:
class ObjectToMeasure:
    # Just an abstract class - it will not be converted to a Table
    is_working = db.Column(db.Boolean)

class SpecificObjectToMeasure(ObjectToMeasure, db.Model):
    object_id = db.Column(db.Integer, db.ForeignKey('object.id'), nullable=False)
    declared_size = db.Column(db.Float)

    def __repr__(self):
        return f"SpecificObjectToMeasure('{self.object_id}', '{self.is_working}', '{self.declared_size}')"

class Measurement(TimestampMixin):
    # Just an abstract class - it will not be converted to a Table
    user_id = db.Column(db.Integer, db.ForeignKey('user.id'), nullable=False)
    title = db.Column(db.UnicodeText)

class SpecificMeasurement(Measurement, db.Model):
    object_to_measure_id = db.Column(db.Integer, db.ForeignKey('object_to_measure.id'), nullable=False)

    def __repr__(self):
        return f"SpecificMeasurement('{self.object_to_measure_id}', '{self.user_id}', '{self.title}')"

Where TimestampMixin is:
class TimestampMixin(object):
    timestamp = db.Column(
        db.DateTime, nullable=False, default=datetime.utcnow)

SQLAlchemy object initialisation:
class IdModel(Model):
    @declared_attr
    def id(cls):
        for base in cls.__mro__[1:-1]:
            if getattr(base, '__table__', None) is not None:
                type = sa.ForeignKey(base.id)
                break
        else:
            type = sa.Integer

        return sa.Column(type, primary_key=True)

db = SQLAlchemy(model_class=IdModel)

# then in create_app method:

db.init_app(app)

And I've got:
 File ".../models.py", line 102, in <module>
    class SpecificMeasurement(Measurement, db.Model):
  File ".../venv/lib/python3.8/site-packages/flask_sqlalchemy/model.py", line 67, in __init__
    super(NameMetaMixin, cls).__init__(name, bases, d)
  File ".../venv/lib/python3.8/site-packages/flask_sqlalchemy/model.py", line 121, in __init__
    super(BindMetaMixin, cls).__init__(name, bases, d)
  File ".../venv/lib/python3.8/site-packages/sqlalchemy/ext/declarative/api.py", line 75, in __init__
    _as_declarative(cls, classname, cls.__dict__)
  File ".../venv/lib/python3.8/site-packages/sqlalchemy/ext/declarative/base.py", line 131, in _as_declarative
    _MapperConfig.setup_mapping(cls, classname, dict_)
  File ".../venv/lib/python3.8/site-packages/sqlalchemy/ext/declarative/base.py", line 160, in setup_mapping
    cfg_cls(cls_, classname, dict_)
  File ".../venv/lib/python3.8/site-packages/sqlalchemy/ext/declarative/base.py", line 180, in __init__
    self._scan_attributes()
  File ".../venv/lib/python3.8/site-packages/sqlalchemy/ext/declarative/base.py", line 232, in _scan_attributes
    self._produce_column_copies(base)
  File ".../venv/lib/python3.8/site-packages/sqlalchemy/ext/declarative/base.py", line 377, in _produce_column_copies
    raise exc.InvalidRequestError(
sqlalchemy.exc.InvalidRequestError: Columns with foreign keys to other columns must be declared as @declared_attr callables on declarative mixin classes.


ehn(api): add unregisterLanguage method

Changes

Currently, there is no way of un-do a registerLanguage operation except re-loading the whole library. Adds a unregisterLanguage method to the public API.
Checklist

[x] No markup tests, this shouldn't change anything to the markup
[x] Updated the changelog at CHANGES.md
[x] Added myself to AUTHORS.txt, under Contributors

Curious: What is your use case for needing to unregister a language?

This would also need to be documented in api.rst.

Curious: What is your use case for needing to unregister a language?

Useful for testing, in the test suite we have to remove the module from the cache to remove the test languages added by the other tests:
https://github.com/highlightjs/highlight.js/blob/742e87b4c512faaf8118323e3192aa94628e262f/test/detect/index.js#L3-L4
This could be avoided if the other tests were cleaning up their test languages. This is needed to move the test suite to ESM syntax (https://github.com/joshgoebel/highlight.js/pull/1).

This is needed to move the test suite to ESM syntax

Ah, that makes sense since we can't just delete the require cache anymore. :)

@aduh95 Thanks!


UofT-DSI | visualization - Assignment 3

What changes are you trying to make?
Completing Assignment 3: comparing good and bad data visualizations based on the data visualization ethics rules
What did you learn from the changes you have made?
I learned how to enhance user accessibility, reproducibility, and equitability
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
A reference to a related issue in your repository (if applicable)
Checklist

[ ] I can confirm that my changes are working as intended

A3 complete


Fix member borrowing under if

This change is

Current dependencies on/for this PR:

main

PR #3775   👈

This comment was auto-generated by Graphite.


Use streams for bridge methods that return string output

Made few other changes based on Ariels feedback. Changed the two bridge methods to return InputStream instead of String. You can ignore the changes made for ctool as that was mainly done to avoid errors. I might be changing those based on ctool implementation.

I don't think there is any need to use a Stream for readClusterLogs, since that is currently purely for post-processing, and just makes the code a lot clunkier. -1 there.

Ariel was more asking for InputStreams as returns from calls to nodetool, or other tools where we're blocking for the output to complete, and then returning the whole output as a String

I agree with making it clunkier.. But I thought the changes were to be made for abstract bridge methods, so changed the two.. Will discard the previous changes and make new

Most changes are relevant to CCMBridge. CtoolBridge has been changed a lot since the last push. Will be creating a PR for that tomorrow

Is this ready for merging? It looks fine.

Yeah, This can be merged. Will be creating another PR for ctool


_changes: don't stop the request if heartbeat is specified

Overview
If the heartbeat parameter is specified in the _changes endpoint,
it will usually override the timeout parameter. It also needs to override
the limit parameter.

Testing recommendations

Related Issues or Pull Requests

Checklist

[ ] Code is written and works correctly
[ ] Changes are covered by tests
[ ] Any new configurable parameters are documented in rel/overlay/etc/default.ini
[ ] Documentation changes were made in the src/docs folder
[ ] Documentation changes were backported (separated PR) to affected branches

This sounds wrong. We want a changes response with a limit=X parameter to end just after the Xth row is sent, regardless of whether heartbeat is set. What is the point in keeping the changes response open after that point?
I confirmed locally that, without this change, ?feed=continuous&limit=5 and ?feed=continuous&limit=5&heartbeat=5000 both end after the 5th row is sent, and with the patch the first one still ends but ?feed=continuous&limit=5&heartbeat=5000 continues to send heartbeats after the 5th row is sent, but will never again send anything but heartbeats. Why would we want that?

heartbeat (number) – Period in milliseconds after which an empty line is sent in the results. Only applicable for longpoll, continuous, and eventsource feeds. Overrides any timeout to keep the feed alive indefinitely. Default is 60000. May be true to use default value.

Before changing limit to override timeout, if we set heartbeat it would override the timeout, and keep the request active indefinitely. After the change, limit has a higher priority. So wondering if we want to keep the old behavior and have heartbeat have the highest priority.
@rnewson Why do we need the heartbeat behavior before? Do you know any use cases?

heartbeat is used to send periodic newlines back to the client to avoid the connection from timing out when we're waiting for more lines to be emitted.
The key is waiting for more lines. With a limit parameter, once we fetched the limit number lines there is nothing to wait for; keeping the request alive indefinitely leaks resources which I consider a bug which should have now been fixed.

Verified timeout and heartbeat locally, heartbeat always overrides timeout.
I will close this PR, thanks for the explanation, @nickva and @rnewson.


Consider switch to Chairmarks.jl

Chairmarks.jl (discourse thread here) from @LilithHafner is much faster than BenchmarkTools.jl. With BenchmarkTools.jl, it can definitely take a while to run an entire benchmark suite, especially if you need to use benchmark tuning, so I am very curious about whether we could try to switch AirspeedVelocity.jl to use this!
It would be really nice to get ultra-fast benchmarks for an entire package's suite.
cc @Zentrik @Krastanov in case of interest.

I haven't played around with ChairMarks too much but setting gctrial to false should reduce start up time for BenchmarkTools ~300ms.

Cross-posting thread on Chairmarks.jl for translating benchmarks: https://github.com/LilithHafner/Chairmarks.jl/issues/70

I haven't played around with ChairMarks too much but setting gctrial to false should reduce start up time for BenchmarkTools ~300ms.

Interesting. Would this hurt accuracy at all though?
Some benchmarks can take ~20 minutes to run with BenchmarkTools though so 300ms might not save much.

Would this hurt accuracy at all though?

I suspect it might, but if the benchmark is 20 minutes long that seems unlikely to matter. Chairmarks essentially run with gctrial=false anyways.

Some benchmarks can take ~20 minutes to run with BenchmarkTools though so 300ms might not save much.

I don't know how well BenchmarkTools and Chairmarks perform on 20 minute workloads but at that point aren't you only running the your benchmark function once, I wouldn't think either would have much overhead and you could use @time at that point.

Some benchmarks can take ~20 minutes to run with BenchmarkTools though so 300ms might not save much.

Just a guess here: a perhaps that 20 minute run includes multiple benchmarks, each of which has its own overhead. If on the other hand, it is actually a single 20 minute workload...

I don't know how well BenchmarkTools and Chairmarks perform on 20 minute workloads but at that point aren't you only running the your benchmark function once, I wouldn't think either would have much overhead and you could use @time at that point.

Chairmarks and @time are pretty much equivalent at those timescales, though BenchmarkTools will run thrice (or twice if evals is specified).
julia> @time @time sleep(10)
 10.011087 seconds (4 allocations: 112 bytes)
 10.019674 seconds (444 allocations: 19.547 KiB, 0.08% compilation time)

julia> @time @b sleep(10)
[ Info: Loading Chairmarks ...
 10.037857 seconds (114.04 k allocations: 5.883 MiB, 0.23% compilation time)
10.011 s (4 allocs: 112 bytes, without a warmup)

julia> @time @btime sleep(10)
[ Info: Loading BenchmarkTools ...
  10.011 s (4 allocations: 112 bytes)
 30.654176 seconds (1.90 M allocations: 96.328 MiB, 1.14% gc time, 0.92% compilation time)

julia> @time @btime sleep(10) evals=1
  10.011 s (4 allocations: 112 bytes)
 20.205827 seconds (32.59 k allocations: 1.615 MiB, 0.82% gc time, 0.07% compilation time)

In theory all this would take is a package extension on Chairmarks, which removes run! from whichever code extracts SUITE, and extracts statistics slightly differently...


Write unit test for file pkg/util/config_util.go

[kind/Enhancement]

Which functionality do you think we should update/improve?
UT of file pkg/util/config_util.go
Why is this needed?
Follow up issue of #1362

/close


[GLUTEN-1632][CH]Daily Update Clickhouse Version (20241213)

Auto commit by gluten daily build, please check the build status and merge it if it's green.

Run Gluten Clickhouse CI on x86


Add option to dump/show current Kong config into YAML/JSON format

Add ability to dump/show current kong config.
e.g. :
# Dump config as yaml
konfig --dump yml --host localhost:8001
# => ---
#  apis:
#    -
#      name: "mockbin"
#      attributes:
#        upstream_url: "http://mockbin.com/"
#        request_host: "mockbin.com"

# Dump config as json
konfig --dump json --host localhost:8001
# => {
#    "apis": [
#        {
#            "name": "mockbin",
#            "attributes": {
#                "upstream_url": "http://mockbin.com/",
#                "request_host": "mockbin.com"
#            }
#        }
#    ]
#}

So you would be able to dump and (re-)load a kong "kongfig" like that :
konfig --dump yml --host $KONG_PRODUCTION_HOST:8001 > config.yml
konfig --path ./config.yml --host localhost:8001

I would love to associate a PR with this issue if the team ( @CyExy , @jerizm , @eddmann , @albarin , @gavinlove ) agrees it does feet with kongfig's puspose/use-cases.

@dvxam we have now added this feature in 1.2 kongfig dump --host localhost:8001 --format json


Add the ability to use multiple processes to export to json

This is a super useful python script.
We have several hundred files from one of our collection exports and the single-process default was going to take hours/days at the rate it was going. So, added an option to set how many processes(defaults to cpu_num - 1) and then changed how the source_dir and dest_dir params are passed.
Used Black for formatting so some formatting stuff changed.

@ryaminal  thanks for the PR. could you also make sure you update the documentation in readme according to the changes please.

Ah, good call on the README changes. I've made some changes. Let me know if it's sufficient.

Was becoming cumbersome to manage the branches and changes. Closing in favor of one PR.


Failed install JaBbA package

I installed the dependencies of the package, except for the IBM academic initiative software. When i tried to install it, R turns out this

installing source package 'JaBbA' ...
** using staged installation

WARNING: this package has a configure script
It probably needs manual configuration

** R
** inst
** byte-compile and prepare package for lazy loading
Warning messages:
1: multiple methods tables found for 'seqinfo<-'
2: replacing previous import 'GenomicRanges::shift' by 'data.table::shift' when loading 'gGnome'
3: replacing previous import 'GenomicRanges::score' by 'fishHook::score' when loading 'gGnome'
4: replacing previous import 'GenomicRanges::seqinfo<-' by 'gTrack::seqinfo<-' when loading 'gGnome'
5: replacing previous import 'Matrix::%&%' by 'gUtils::%&%' when loading 'gGnome'
6: replacing previous import 'GenomicRanges::union' by 'igraph::union' when loading 'gGnome'
7: replacing previous import 'data.table::melt' by 'reshape2::melt' when loading 'gGnome'
8: multiple methods tables found for '%&%'
** help
No man pages found in package  'JaBbA'
*** installing help indices
** building package indices
** testing if installed package can be loaded from temporary location
Warning: multiple methods tables found for 'seqinfo<-'
Warning: replacing previous import 'GenomicRanges::shift' by 'data.table::shift' when loading 'gGnome'
Warning: replacing previous import 'GenomicRanges::score' by 'fishHook::score' when loading 'gGnome'
Warning: replacing previous import 'GenomicRanges::seqinfo<-' by 'gTrack::seqinfo<-' when loading 'gGnome'
Warning: replacing previous import 'Matrix::%&%' by 'gUtils::%&%' when loading 'gGnome'
Warning: replacing previous import 'GenomicRanges::union' by 'igraph::union' when loading 'gGnome'
Warning: replacing previous import 'data.table::melt' by 'reshape2::melt' when loading 'gGnome'
Warning: multiple methods tables found for '%&%'
Error: package or namespace load failed for 'JaBbA' in library.dynam(lib, package, package.lib):
DLL 'JaBbA' not found: maybe not installed for this architecture?
Error: loading failed
Execution halted
ERROR: loading failed

removing 'D:/R/R-3.6.3/library/JaBbA'

I tried the augment devtools::install_github("mskilab/JaBbA"， dependency = TRUE)， but it seems that it's not working. I tried to download the source code to local files and installed it. The same error still occurs.
My operating system is win10
This is my session info
R version 3.6.3 (2020-02-29)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 10 x64 (build 18363)
Matrix products: default
locale:
[1] LC_COLLATE=Chinese (Simplified)_China.936  LC_CTYPE=Chinese (Simplified)_China.936
[3] LC_MONETARY=Chinese (Simplified)_China.936 LC_NUMERIC=C
[5] LC_TIME=Chinese (Simplified)_China.936
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
loaded via a namespace (and not attached):
[1] XVector_0.26.0         GenomeInfoDb_1.22.1    compiler_3.6.3         BiocManager_1.30.10    git2r_0.27.1
[6] zlibbioc_1.32.0        bitops_1.0-6           prettyunits_1.1.1      remotes_2.1.1          tools_3.6.3
[11] testthat_3.0.1         digest_0.6.25          pkgbuild_1.0.8         pkgload_1.1.0          memoise_1.1.0
[16] lattice_0.20-41        pkgconfig_2.0.3        rlang_0.4.10           igraph_1.2.5           Matrix_1.2-18
[21] cli_2.2.0              rstudioapi_0.11        curl_4.3               parallel_3.6.3         GenomeInfoDbData_1.2.2
[26] withr_2.4.0            IRanges_2.20.2         desc_1.2.0             fs_1.4.1               S4Vectors_0.24.4
[31] devtools_2.3.0         stats4_3.6.3           rprojroot_1.3-2        grid_3.6.3             data.table_1.13.0
[36] glue_1.4.1             R6_2.4.1               processx_3.4.5         fansi_0.4.1            sessioninfo_1.1.1
[41] callr_3.5.1            magrittr_1.5           backports_1.1.7        ps_1.5.0               ellipsis_0.3.1
[46] usethis_1.6.1          GenomicRanges_1.38.0   BiocGenerics_0.32.0    assertthat_0.2.1       RCurl_1.98-1.2
[51] crayon_1.3.4
Please let me know if you need more information about my computer.

I installed the dependencies of the package, except for the IBM academic initiative software. When i tried to install it, R turns out this

installing source package 'JaBbA' ...

** using staged installation

WARNING: this package has a configure script
     It probably needs manual configuration

** R
** inst
** byte-compile and prepare package for lazy loading
Warning messages:
1: multiple methods tables found for 'seqinfo<-'
2: replacing previous import 'GenomicRanges::shift' by 'data.table::shift' when loading 'gGnome'
3: replacing previous import 'GenomicRanges::score' by 'fishHook::score' when loading 'gGnome'
4: replacing previous import 'GenomicRanges::seqinfo<-' by 'gTrack::seqinfo<-' when loading 'gGnome'
5: replacing previous import 'Matrix::%&%' by 'gUtils::%&%' when loading 'gGnome'
6: replacing previous import 'GenomicRanges::union' by 'igraph::union' when loading 'gGnome'
7: replacing previous import 'data.table::melt' by 'reshape2::melt' when loading 'gGnome'
8: multiple methods tables found for '%&%'
** help
No man pages found in package  'JaBbA'
*** installing help indices
** building package indices
** testing if installed package can be loaded from temporary location
Warning: multiple methods tables found for 'seqinfo<-'
Warning: replacing previous import 'GenomicRanges::shift' by 'data.table::shift' when loading 'gGnome'
Warning: replacing previous import 'GenomicRanges::score' by 'fishHook::score' when loading 'gGnome'
Warning: replacing previous import 'GenomicRanges::seqinfo<-' by 'gTrack::seqinfo<-' when loading 'gGnome'
Warning: replacing previous import 'Matrix::%&%' by 'gUtils::%&%' when loading 'gGnome'
Warning: replacing previous import 'GenomicRanges::union' by 'igraph::union' when loading 'gGnome'
Warning: replacing previous import 'data.table::melt' by 'reshape2::melt' when loading 'gGnome'
Warning: multiple methods tables found for '%&%'
Error: package or namespace load failed for 'JaBbA' in library.dynam(lib, package, package.lib):
DLL 'JaBbA' not found: maybe not installed for this architecture?
Error: loading failed
Execution halted
ERROR: loading failed

removing 'D:/R/R-3.6.3/library/JaBbA'

I tried the augment devtools::install_github("mskilab/JaBbA"， dependency = TRUE)， but it seems that it's not working. I tried to download the source code to local files and installed it. The same error still occurs.
My operating system is win10
This is my session info
R version 3.6.3 (2020-02-29)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 10 x64 (build 18363)
Matrix products: default
locale:
[1] LC_COLLATE=Chinese (Simplified)_China.936  LC_CTYPE=Chinese (Simplified)_China.936
[3] LC_MONETARY=Chinese (Simplified)_China.936 LC_NUMERIC=C
[5] LC_TIME=Chinese (Simplified)_China.936
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
loaded via a namespace (and not attached):
[1] XVector_0.26.0         GenomeInfoDb_1.22.1    compiler_3.6.3         BiocManager_1.30.10    git2r_0.27.1
[6] zlibbioc_1.32.0        bitops_1.0-6           prettyunits_1.1.1      remotes_2.1.1          tools_3.6.3
[11] testthat_3.0.1         digest_0.6.25          pkgbuild_1.0.8         pkgload_1.1.0          memoise_1.1.0
[16] lattice_0.20-41        pkgconfig_2.0.3        rlang_0.4.10           igraph_1.2.5           Matrix_1.2-18
[21] cli_2.2.0              rstudioapi_0.11        curl_4.3               parallel_3.6.3         GenomeInfoDbData_1.2.2
[26] withr_2.4.0            IRanges_2.20.2         desc_1.2.0             fs_1.4.1               S4Vectors_0.24.4
[31] devtools_2.3.0         stats4_3.6.3           rprojroot_1.3-2        grid_3.6.3             data.table_1.13.0
[36] glue_1.4.1             R6_2.4.1               processx_3.4.5         fansi_0.4.1            sessioninfo_1.1.1
[41] callr_3.5.1            magrittr_1.5           backports_1.1.7        ps_1.5.0               ellipsis_0.3.1
[46] usethis_1.6.1          GenomicRanges_1.38.0   BiocGenerics_0.32.0    assertthat_0.2.1       RCurl_1.98-1.2
[51] crayon_1.3.4
Please let me know if you need more information about my computer.

Hi,
We haven't tested on Windows yet, but if DLL is not found that likely means the C code is not compiled properly. Please try on a Linux machine if you can. We will see if there's a solution to this meanwhile.
Thank you,
Xiaotong

I installed the dependencies of the package, except for the IBM academic initiative software. When i tried to install it, R turns out this

installing source package 'JaBbA' ...

** using staged installation

WARNING: this package has a configure script
     It probably needs manual configuration

** R
** inst
** byte-compile and prepare package for lazy loading
Warning messages:
1: multiple methods tables found for 'seqinfo<-'
2: replacing previous import 'GenomicRanges::shift' by 'data.table::shift' when loading 'gGnome'
3: replacing previous import 'GenomicRanges::score' by 'fishHook::score' when loading 'gGnome'
4: replacing previous import 'GenomicRanges::seqinfo<-' by 'gTrack::seqinfo<-' when loading 'gGnome'
5: replacing previous import 'Matrix::%&%' by 'gUtils::%&%' when loading 'gGnome'
6: replacing previous import 'GenomicRanges::union' by 'igraph::union' when loading 'gGnome'
7: replacing previous import 'data.table::melt' by 'reshape2::melt' when loading 'gGnome'
8: multiple methods tables found for '%&%'
** help
No man pages found in package  'JaBbA'
*** installing help indices
** building package indices
** testing if installed package can be loaded from temporary location
Warning: multiple methods tables found for 'seqinfo<-'
Warning: replacing previous import 'GenomicRanges::shift' by 'data.table::shift' when loading 'gGnome'
Warning: replacing previous import 'GenomicRanges::score' by 'fishHook::score' when loading 'gGnome'
Warning: replacing previous import 'GenomicRanges::seqinfo<-' by 'gTrack::seqinfo<-' when loading 'gGnome'
Warning: replacing previous import 'Matrix::%&%' by 'gUtils::%&%' when loading 'gGnome'
Warning: replacing previous import 'GenomicRanges::union' by 'igraph::union' when loading 'gGnome'
Warning: replacing previous import 'data.table::melt' by 'reshape2::melt' when loading 'gGnome'
Warning: multiple methods tables found for '%&%'
Error: package or namespace load failed for 'JaBbA' in library.dynam(lib, package, package.lib):
DLL 'JaBbA' not found: maybe not installed for this architecture?
Error: loading failed
Execution halted
ERROR: loading failed

removing 'D:/R/R-3.6.3/library/JaBbA'

I tried the augment devtools::install_github("mskilab/JaBbA"， dependency = TRUE)， but it seems that it's not working. I tried to download the source code to local files and installed it. The same error still occurs.
My operating system is win10
This is my session info
R version 3.6.3 (2020-02-29)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 10 x64 (build 18363)
Matrix products: default
locale:
[1] LC_COLLATE=Chinese (Simplified)_China.936  LC_CTYPE=Chinese (Simplified)_China.936
[3] LC_MONETARY=Chinese (Simplified)_China.936 LC_NUMERIC=C
[5] LC_TIME=Chinese (Simplified)_China.936
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
loaded via a namespace (and not attached):
[1] XVector_0.26.0         GenomeInfoDb_1.22.1    compiler_3.6.3         BiocManager_1.30.10    git2r_0.27.1
[6] zlibbioc_1.32.0        bitops_1.0-6           prettyunits_1.1.1      remotes_2.1.1          tools_3.6.3
[11] testthat_3.0.1         digest_0.6.25          pkgbuild_1.0.8         pkgload_1.1.0          memoise_1.1.0
[16] lattice_0.20-41        pkgconfig_2.0.3        rlang_0.4.10           igraph_1.2.5           Matrix_1.2-18
[21] cli_2.2.0              rstudioapi_0.11        curl_4.3               parallel_3.6.3         GenomeInfoDbData_1.2.2
[26] withr_2.4.0            IRanges_2.20.2         desc_1.2.0             fs_1.4.1               S4Vectors_0.24.4
[31] devtools_2.3.0         stats4_3.6.3           rprojroot_1.3-2        grid_3.6.3             data.table_1.13.0
[36] glue_1.4.1             R6_2.4.1               processx_3.4.5         fansi_0.4.1            sessioninfo_1.1.1
[41] callr_3.5.1            magrittr_1.5           backports_1.1.7        ps_1.5.0               ellipsis_0.3.1
[46] usethis_1.6.1          GenomicRanges_1.38.0   BiocGenerics_0.32.0    assertthat_0.2.1       RCurl_1.98-1.2
[51] crayon_1.3.4
Please let me know if you need more information about my computer.

Hi,
We haven't tested on Windows yet, but if DLL is not found that likely means the C code is not compiled properly. Please try on a Linux machine if you can. We will see if there's a solution to this meanwhile.
Thank you,
Xiaotong

Hi,
I just noticed you mention that you did not install CPLEX from IBM. It is a requirement for JaBbA, so please try installing that first and follow the README to set up the correct path to your installation. I'll close this for now, please let me know if after you install CPLEX (there's a free academic license), this error still persists.
Xiaotong

Hi,
I just noticed you mention that you did not install CPLEX from IBM. It is a requirement for JaBbA, so please try installing that first and follow the README to set up the correct path to your installation. I'll close this for now, please let me know if after you install CPLEX (there's a free academic license), this error still persists.
Xiaotong


SSLException results in native crash

Hello all, I am using Jodd 3.6.6 on Android 6.0 now (using Nexus 5).
No problems before on older API versions, but when I try and run my app on API 23, it results in a native crash.
*** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
Build fingerprint: 'google/hammerhead/hammerhead:6.0/MRA58K/2256973:user/release-keys'
Revision: '0'
ABI: 'arm'
pid: 19386, tid: 19839, name: MinervaService]  >>> com.minervafeed <<<
signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x9cfb8fff
    r0 12f3ba80  r1 9cfb9000  r2 ffffffff  r3 ffffffff
    r4 0025be38  r5 708c5680  r6 9cfb9000  r7 ffffffff
    r8 00000000  r9 9f2a6b00  sl ffffffff  fp ffffffff
    ip 00000000  sp 9ce9f3a0  lr a1374463  pc a12512fe  cpsr a00e0030

backtrace:
    #00 pc 007122fe  /data/app/com.minervafeed-1/oat/arm/base.odex (offset 0x535000)
    #01 pc 00835461  /data/app/com.minervafeed-1/oat/arm/base.odex (offset 0x535000)

Debugging it gives me this error instead of a native crash;
jodd.http.HttpException: Write error: ssl=0x9e7bc580: I/O error during system call, Broken pipe; <--- javax.net.ssl.SSLException: Write error: ssl=0x9e7bc580: I/O error during system call, Broken pipe

It fails in HttpBrowser.java, on the line 125 in 3.6.6
this.httpResponse = httpRequest.send();

Any response is appreciated.

Hmmm... It sounds like this?

In other words: use Authentication via header instead of CredentielsService?

Authentication happens through cookies, so the Authentication header is not used though...

The only thing I may suggest is to try to use different way how to configure/use SSL... It looks like internal Android thing to me, according to similar problems.
Would it be possible for you to use different way to configure SSL? I will try to download the Android SDK and try it; however, it will take some time...

I don't know of any configuration possibilities regarding SSL, but the broken pipe thing might point to a request that was closed before and then later on reused...

http://stackoverflow.com/questions/3352424/httpurlconnection-openconnection-fails-second-time This seems to be a similar problem, although it does not seem to result in a native crash

Any news on this? I am not sure if I can help with this atm...

No news yet, I'm monitoring this closely, I'll let you know if anything changes

I have replaced the Jodd HTTP library with my own client, based off HttpUrlConnection and it still gives an error... The only other library I'm using atm is Lagarto, but I would not know what would cause an error like this in that library... Still debugging this

Is there any chance we can ask this on Android support forum? If HttpUrlConnection is also failing, then they should be aware of that, I guess.

Yes, I will close this for now and look some more into the issue and what the root cause is... I'll report back if I know something more...


daprd does not install on container rebuild

If you rebuild a dev container daprd fails to install into the container because the other dapr containers are already spun up (dapr_placement, dapr_redis, dapr_zipkin).

VSCode Version: 1.56.2
Local OS Version: Windows 10/WSL Ubuntu 20.04
Name of Dev Container Definition with Issue: dapr-dotnet

Steps to Reproduce:

create remote container, including building the image
edit any files in .devcontainer folder
rebuild dev container

Workaround:
run following commands in newly rebuilt container
dapr uninstall
dapr init

@Chuxel That would probably work, though if Dapr is otherwise setup it's just extra initialization time that isn't technically required.  What might be better still is a bit more nuanced logic in a real script that only attempts to do a dapr init when Dapr isn't already setup.
I had planned on moving these Dev Containers to a "Docker-in-Docker" model which simplifies other aspects of using Dapr, too; it would alleviate the problem altogether because it would always be starting from a known state (i.e. no Dapr), so the initialization logic would always succeed.  However, I got bogged down by attempts to make it work on the M1 Mac, which requires the base images be arm64 compatible; I discovered that there was no Debian arm64 build of Moby, needed as the base .NET images are based on Debian.  (Unlike the base Node.js images, which are based on Ubuntu, which can use Moby builds we (i.e. Microsoft) appear to build ourselves.  Or it might have been the inverse, I forget exactly.) In any case, my attention got drawn elsewhere and I've not made it back to updating the Dev Containers.

@philliphoff Yeah I do not have a M1 mac at the moment to cross check (I'm on a 16" MBP) - so that's good to know, but also odd.  Would it make sense to add the uninstall / init tweak near term?

Ok, closing as fixed for now since this unblocks. Happy to take an update to move to docker-in-docker as you have the cycles @philliphoff !


Mouse cursor changing rapidly when placed between lines of source control?

Type: Bug

Needs to be a retina Mac OS screen.
Place cursor between 2 lines of source control in the icons area. The area the issue occurs is only 1 (actual) pixel tall.

See video
VS Code version: Code 1.92.0 (Universal) (b1c0a14de1414fcdaa400695b4db1c0799bc3124, 2024-07-31T23:26:45.634Z)
OS version: Darwin x64 22.6.0
Modes:
Remote OS version: Linux x64 5.14.0-284.30.1.el9_2.x86_64

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 x 2600)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
2, 2, 2

Memory (System)
32.00GB (4.31GB free)

Process Argv

Screen Reader
no

VM
0%

Item
Value

Remote
SSH: orwell.revenueuniverse.com

OS
Linux x64 5.14.0-284.30.1.el9_2.x86_64

CPUs
DO-Regular (1 x 0)

Memory (System)
1.72GB (0.62GB free)

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

remote-ssh
ms-
0.113.1

remote-ssh-edit
ms-
0.86.0

remote-explorer
ms-
0.4.3

composer-php-vscode
DEV
1.49.15728

intelli-php-vscode
DEV
0.12.15062

phptools-vscode
DEV
1.49.15728

profiler-php-vscode
DEV
1.49.15728

php-debug
xde
1.35.0

https://github.com/user-attachments/assets/1bf2cdfc-1264-44c6-9457-80de5eb58cd3

It keeps saying I need to enter my password to install a helper tool and I have been pressing cancel.
Is pressing cancel preventing an update from installing?
Why does it need to ask this every time it seems?

That's what it shows. I have 500 mbit fiber here. Nothing is really downloading.

@deepak1556, have you seen this behaviour before? Thanks!

@MikeRich88, are you still able to reproduce this with the latest VS Code release? Thanks!


root_dir for Contents Manager not loaded from config file

I added the lines
c.ContentsManager.root_dir = '~'
c.FileContentsManager.root_dir = '~'

to ~/.jupyter/jupyter_notebook_config.py.
I then ran the Jupyter notebook server in debug mode, as follows (the setting of notebook_dir is deliberate):
jupyter lab --notebook-dir='/' --debug

EDIT: the exact same things happen when using the command
jupyter notebook --notebook-dir='/' --debug

and as far as I can tell, JupyterLab uses Notebook's contents manager anyway /EDIT
and it confirmed that the config file was loaded:
[D 16:50:53.349 LabApp] Loaded config file: ~/.jupyter/jupyter_notebook_config.py

My notebook server extension, which uses the .save method of the contents manager to save a notebook to the user's home directory (well in practice, to the root_dir of the contents manager), reports a 403 permissions error when trying to run, and in its log output (see code on GitHub) shows that the value of the contents manager's root directory is the value of notebook_dir, instead of the value specified in the config file.
[D 16:51:48.310 LabApp] Contents manager root_dir: /

This would be expected if no value of root_dir had been specified in the config file, since according to the code, the value of notebook_dir is to be loaded as the default if no value is specified in the configuration. However, a value was specified, and the value of root_dir is supposed to be configurable, so this seems to be a bug.
For further evidence that the value of root_dir specified in the config file is being ignored, I changed the config file to say
c.FileContentsManager.root_dir = 42

Because the value of root_dir is validated, if this configuration is being loaded, then a TraitError should be thrown as a result of this configuration setting. However, no errors are raised when starting the notebook server.
Random Thoughts:
All I can think of at the moment is that perhaps JupyterLab uses a different contents manager than the Jupyter notebook, maybe a subclass, and that for this reason the configuration is being effectively ignored? But I also changed the value of c.ContentsManager.root_dir in the config file, as well as c.LargeFileContentsManager.root_dir (since as far as I can tell the LargeFileContentsManager subclass of FileContentsManager is actually what's loaded/used), but it still does nothing. I initially thought that this was an issue with JupyterHub, but am now fairly convinced that this is not the case, especially since it's possible to reproduce the problem without running JupyterHub.
Note: I originally posted this issue in the Notebook repository, but this has nothing to do with the Notebook frontend, and is about the server backend used by both Lab and Notebook (and JupyterHub-SingleUser), so it seems like it might fit here. Please let me know if that is in fact mistaken.

fixed by https://github.com/jupyter/jupyter_server/pull/135

@krinsman Really sorry for the slow reply here. I see your PR was merged in jupyter/notebook. We're sprinting on jupyter_server (and other Jupyter-related projects) this week, so we should get some time to review this issue closely and merge #135.
Thanks for you patience!

@Zsailer Thank you for the heads-up/update!
No worries, I needed to update the PR I made to match the PR which was merged into notebook anyway. (It's done now.) Thank you for the reminder!


--keyspace by file

looking more adv feature
--keyspace to --keyf keyspacefile.txt
where keyfiles list looks like
example inside file
1:100000000
50000000:70000000
800000000:8500000000
hope this feature add soon

this is easier solved by simple bat/bash script, depending on your OS. No need to add this feature into program itself

@alkiczx i think you are not calc time, fact and figures, maybe weak in math :)
by bat/bash scripting, you always need to create it for 1 million ranges, 2nd each line script will start and stop, and load time also will waste,
time is money, and world is going on fast calc, and speedy work and automation, if you love bat and bash script, hope you must enjoy that, i am also afraid you will leav bitcrack too for brainflayer :)

since you did not explain what should it exactly do there is no wonder I did not understand you. Anyway, since I have my own modified bitcrack for what i need, i definitelly do no need brainflayer, which does something very different ;)


I was thinking the constructor for SqliteConnection could be made more idiomatic by passing it a &Path instead of a &str. An added benefit would be that non-utf8 file paths could be opened as well.
A different constructor would have to be added for an in-memory connection.
What do you think?

I think you're right. It looks like std::path is currently being reworked (https://github.com/rust-lang/rust/issues/20034, https://github.com/rust-lang/rfcs/pull/474#issuecomment-68608439). If you want to do the work to make the changes for the current std::path, I won't stop you. :) If not (and I don't, personally), we can leave this issue open and make this change when std::path is reformed.

Oh, and I'm definitely not feeling intruded upon. Glad to see someone using this!

@marcusklaas How does #19 look? Path reform landed a couple days ago.

Good stuff!


Webapp is broken?

Dear Jiang lab,
It looks like your CTF simulator web app is broken -- The link from https://jiang.bio.purdue.edu/ctfsimulation/ is taking me to an error page.

it is back


xcorr: Calculate power-of-2 sized FFTs

First of all, thank you very much for your code, and for licensing it MIT :)
In the xcorr function of your libxcorr fork, you calculate three FFTs of size 2 * N - 1, which is always an odd number. Most FFT libraries, including fftw3, are optimized for FFT sizes that are powers of 2. When implementing MPM pitch estimation based on your code, but exchanging the FFT library for the non-GPL licensed KissFFT, this was very noticeable to me.
I changed the xcorr and normalized_square_difference functions to set N2 = 2 * N, and the first xcorr zero-padding memcpy call to
std::memcpy(signala_ext.data() + N, signala.data(), sizeof(std::complex<double>) * N);

, effectively duplicating the center sample. Since my N was a power of 2, this change resulted in a FFT calculation time of less than 1/10th the original time.
This change did not affect the results of the pitch estimation, when comparing the results of MPM pitch estimation over 4096 samples of data at a precision of 0.001.
I'm not sure if the change leads to mathematically equivalent results, but the resulting increase in performance is definitely worth it to me. Perhaps you have a better understanding of the algorithms and can prove whether it makes a difference.

First of all, thank you very much for your code, and for licensing it MIT :)
No prob. Actually somebody e-mailed me to ask for that (unless that was you) so thanks to him as well :+1:
Most FFT libraries, including fftw3, are optimized for FFT sizes that are powers of 2.
That's very interesting, I didn't know that. Do you want to make a PR?

I also wonder if I should add assert (N % 2 == 0) to libxcorr to enforce performant calls to FFTW3.

n is the size of the transform. It can be any positive integer. FFTW is best at handling sizes of the form 2a 3b 5c 7d 11e 13f, where e+f is either 0 or 1, and the other exponents are arbitrary. Other sizes are computed by means of a slow, general-purpose routine (which nevertheless retains O(n lg n) performance, even for prime sizes). (It is possible to customize FFTW for different array sizes. See Section Installation and Customization, for more information.) Transforms whose sizes are powers of 2 are especially fast.
http://www.fftw.org/fftw2_doc/fftw_3.html

I've written my own codebase based on your implementation, so I haven't actually modified your code, which is why I didn't create a PR in the first place. While the assertion you're suggesting makes sense, it's obviously only effective if the actual size of the FFT, N2, which is based on N, is even as well.
Regarding the License: I'm not sure whether you are actually allowed to license your code under MIT, given that it uses FFTW3, which is licensed under GPL, which spreads like cancer and requires you to license your work under GPL as well. Therefore, I propose switching to a different FFT library with a license compatible to MIT, for example Kiss FFT.
Note that Kiss FFT is less performant than FFTW3, however it's still suitable for real-time pitch estimation of audio when working with even FFT sizes.
Here's my autocorrelation function based on xcorr, which you're free to use however you wish. Note that I changed the input to only a single vector of samples, as we're only calculating autocorrelation for MPM anyway. If that doesn't suit your use case, you can easily revert that change. Note that I also changed the calculation of N2, which is why you need to change your size2 variable inside the normalized_square_difference function to 2 * size as well. Again, I'm not sure whether the modification of N2 leads to mathematically equivalent results, however my tests did not show a noticeable difference in the pitch estimation.
Also, I changed the FFT library to Kiss FFT, so it is suitable for use with the MIT license. However, Kiss FFT uses float values instead of doubles by default, although according to their docs, it should be possible to change that. I just haven't been able to do so since I'm pretty new to C++ and its non-existent build and dependency system.
#include <kiss_fft.h> // in the header file

void autocorrelation(const std::vector<float> &data, std::complex<float> *result, int n) {
	int n2 = 2 * n;

	std::complex<float> signala_ext[n2];
	std::complex<float> signalb_ext[n2];

	// zero padding of input data
	for (int i = 0; i < n; i++) {
		signala_ext[n + i] = data[i];
		signalb_ext[i] = data[i];
	}

	std::complex<float> outa[n2];
	std::complex<float> outb[n2];
	std::complex<float> out[n2];

	auto fftCfgForward = kiss_fft_alloc(n2, false, nullptr, nullptr);
	auto fftCfgBackward = kiss_fft_alloc(n2, true, nullptr, nullptr);
	if (!fftCfgForward || !fftCfgBackward) {
		throw std::bad_alloc();
	}

	kiss_fft(fftCfgForward, (kiss_fft_cpx *) signala_ext, (kiss_fft_cpx *) outa);
	kiss_fft(fftCfgForward, (kiss_fft_cpx *) signalb_ext, (kiss_fft_cpx *) outb);

	for (int i = 0; i < n2; ++i) {
		out[i] = outa[i] * conj(outb[i]);
	}

	kiss_fft(fftCfgBackward, (kiss_fft_cpx *) out, (kiss_fft_cpx *) result);

	free(fftCfgForward);
	free(fftCfgBackward);
}
``

You could optimize this even more by making a class for correlation and frequency calculation, and creating the `fftCfgForward` and `fftCfgBackward` in the constructor and freeing it in the destructor, however the overhead of the allocation should be minimal enough so it doesn't matter.

Lots of good info here. I will explore well-licensed, simpler, more concise correlation and/or FFT libraries and stop depending on libxcorr in this project.

FYI if you're interested, my Python code for MPM is much more solid (using scipy, numpy, and numba) and with those well-optimized libraries it's probably pretty dang performant: https://github.com/sevagh/transcribe/blob/master/transcribe/pitch.py#L9

I finally got some numbers to prove it. I simplified things by taking the xcorr code from libxcorr and embedding it directly inside this repo.
When N2 = 2 * N - 1, which is the minimum for auto-correlation to work correctly, but not necessarily a power of 2:
sevagh:pitch-detection $ hyperfine --min-runs 10 './bin/sinewave --freq 1337 --algo mpm --size 64000'
Benchmark #1: ./bin/sinewave --freq 1337 --algo mpm --size 64000

  Time (mean ± σ):      66.8 ms ±  11.3 ms    [User: 57.8 ms, System: 8.1 ms]

  Range (min … max):    50.3 ms …  85.2 ms

With this bit of code to find the next power of 2:
	/* https://stackoverflow.com/questions/466204/rounding-up-to-next-power-of-2 */
	if (N2 & (N2 - 1)) {
		N2--;
		N2 |= N2 >> 1;
		N2 |= N2 >> 2;
		N2 |= N2 >> 4;
		N2 |= N2 >> 8;
		N2 |= N2 >> 16;
		N2++;
	}

	assert(!(N2 & (N2 - 1)));

results are faster:
sevagh:pitch-detection $ hyperfine --min-runs 10 './bin/sinewave --freq 1337 --
algo mpm --size 64000'
Benchmark #1: ./bin/sinewave --freq 1337 --algo mpm --size 64000

  Time (mean ± σ):      24.7 ms ±   2.6 ms    [User: 18.0 ms, System: 6.4 ms]

  Range (min … max):    21.0 ms …  34.9 ms

sevagh:pitch-detection $

Yes they are faster, but do they also result in the same pitch estimation for the same input files?

This has been addressed in a few ways.

FFTW has been switched to FFTS to fix the license issue
FFTS performs better

Since it's not really my job to make the input arrays perform better, I decided against doing power-of-2 handling. It should be a pleasant surprise to the user.
I'll put a bit in the README.


Amplify push failed with GraphQLSchema schema update and Resource is not in the state stackUpdateComplete.

Amplify CLI Version
8.5.0
Question
I am trying to update our existing schema with some new fields but ended up with The following resource(s) failed to update: [GraphQLSchema] and Resource is not in the state stackUpdateComplete error.
I am using cloudformation to update existing backend. This is the process which I followed -
1 - Pull existing amplify backend using - amplify pull --appId ************ --envName testapp
2- Make changes for adding new fields in one of our models.
3- amplify api gql-compile
4- amplify push
I am using SSO user and aws_access_key_id, aws_secret_access_key and aws_session_token.
Every time I tried, It failed and status of cloudformation stack change in to UPDATE_ROLLBACK_FAILED state. In order to make subsequent push we needed to perform continuous roll back which breaks our environment and we are not able to use it.

I'm also having this issue on my deployment as well, though mine seems to be related to @searchable directive (turns into a 50/50 chance of deploy). I'd be curious to know if you're using Open Search integration? Current suspicion is that some service was updated to a new api and that's causing the issue since I'm using CLI v10.2.3 and from what I can gather this issue is across CLI versions.

Hi @mattcreaser, Thanks. Yes our schema is using @searchable directive and not sure about Open Search integration. Before some one else used to do all schema update related things and now due to some reason I need to do the same and we don’t know how the other person used to do it without any issue.
Is it possible that this issue will only occur with CLI / CloudFormation and not with Cloud9?
I suspect the other person used to do it using Cloud9.

Hey! I've been doing some digging, even deploying a new project using the latest version of the CLI the @searchable direction seems to block any deployments. I see version 7.6.12 has been depreciated , current minimum appears to be 7.6.15 maybe try running npm i -g<EMAIL_ADDRESS>I've never done any deploys using Cloud9 so it could be possible, though I'm getting at least the same error message stopping my deployments as of Dec 23rd when attempting to deploy any new models using the @searchable directive. No stack drifts on CloudFormation from what I can see.

I tried almost everything from stackoverflow and GitHub issues but nothing works for me.

@rahulnainwal107 I suspect that we're having different issues, but I was able to debug my issue to it's root cause, in my case I have too many models using @searchable which was causing Maximum policy size of 10240 bytes exceeded for role on the OpenSearchStreamingLambdaIAMRoleDefaultPolicy I created an issue for it here #11690.
Since the error Resource is not in the state stackUpdateComplete I believe to basically mean "something happened" Figured I'd drop the steps I used to debug the issue here just in cause you're able to get a bit closer to understanding the issue.
In the AWS Console go to CloudFormation -> Stacks. There you'll want to search the resources that are throwing the errors (those would be the bits you censored in your above screenshot). In my case it was failing on the SearchableStack.
Click on the resource stack in question and navigate to the Events tab, from there it should basically read a log for everything deployed in that stack, scroll through the events until you see the UPDATE_FAILED status, there should be a status reason along side providing you the appropriate error.
Hopefully that'll get you a bit closer to your issue, and be sure to report back any findings as it'll help give more context to the developers, and others having issues.

Thanks for insightful info @MtthwBrwng. Yes we are also having @searchable at multiple places. Error looks like this from CloudFormation -> Stacks

Internal Failure while saving the schema.

For more insights - Output of amplify diagnose --send-report

Finally we are able to successfully push schema changes using #2335

When we are trying to add new fields for some models and trying to repeat same process then again we are getting same error.
We are trying to update schema and following these commands in sequence -
1- amplify api gql-compile --minify
2- amplify push
Is there any other step we need to follow?


Update testcontainers-scala-postgresql, ... to 0.40.9

Updates

com.dimafeng:testcontainers-scala-postgresql
com.dimafeng:testcontainers-scala-scalatest

from 0.40.0 to 0.40.9.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.dimafeng" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.dimafeng" }
}]

labels: test-library-update, early-semver-minor, semver-spec-patch, commit-count:1

Superseded by #88.


add Chinese (Simplified), Japanese, Korean and Portuguese localizations.

Attempt #3000032. It's been years we've been trying to get this right, but the arcane rules of Git that were written down generations ago and altered by faulty humans are working against us. I haven't eaten in weeks. This is our last hope, my dying breath is pressing the submit button. Godspeed.
Tested the autobuild, and there are the results. Will run these by the folks who can translate.

https://github.com/macadmins/nudge/pull/272/checks?check_run_id=3829174063 artifact now has the new languages in the nudge.app

For anyone watching, I now have to add the safe to test tag each time I want to kick off a build. This is to prevent rogue PRs from accessing the signing certificates as the PR check now creates a new test build for authors to play with.


Redeem offer

@0xneves I need your help to specify this task... The main questions I have are:

How to create a list of offered trades from a given user?
Which is the contract address of Swaplace SC?
What is the function to call in order to redeem a trade?

With a single EthereumAddress (the address of the authenticated user) am I capable of doing the request of every params needed for this function call?
Thanks!

Duplicated: https://github.com/blockful-io/swaplace-dapp/issues/15


Updates to use MAPL_GetResource in Surf

NOTE: This PR requires MAPL 2.6.7 (soon to be released) because it needs https://github.com/GEOS-ESM/MAPL/pull/834 to enable the MAPL_GetResource call with an ESMF_Config
This PR is due to a request by @wmputman to have GEOSgcm output the configuration values that GEOS_SurfaceGridComp.rc sets a la how we see outputs in the log from AGCM.rc. To do this takes a few changes:

You need support in MAPL for calling MAPL_GetResource with an ESMF_Config (see https://github.com/GEOS-ESM/MAPL/pull/834)
You need to set PRINTRC: 1 in GEOS_SurfaceGridComp.rc so that it will output even defaults (this echoes AGCM.rc.tmpl
Obviously, we'll also want the detemplating code in GEOSgcm_App that @sdrabenh is working on.

I also converted the calls to "match" how other MAPL_GetResource calls look like (config, val, label, default, rc).
This PR will be a Draft until MAPL 2.6.7 is released an in GEOSgcm and GEOSldas. Until and once MAPL 2.6.7 is in, I will work with @biljanaorescanin and @gmao-rreichle to get this "just right".

@gmao-rreichle Since GEOSgcm has now moved to using Icarus-NLv3 by 'default' (i.e., it's what you get by pressing "Enter" for the Land BCs question in gcm_setup), it does make sense to change this. So, I've updated the defaults per your comments.
Please review.


Inpatient users can cause havoc in Surveyor.

Using Rails 0.20.0, Rails 3 branch.
The model of saving of responses as they are answered is really cool but if a user is inpatient and does not wait for the ajax request to finish, allowing the calllback to update the id of a newly created response, then:

if the user clicks the next section button, then a duplicate response can be created on the server
if the user unchecks a checkbox, then the response is not removed on the server

The UI needs to be blocked while the ajax request is submitted, so the user cannot reak havoc.  Possible solution:  http://jquery.malsup.com/block/

Closed by 673da85f658309aac3bb84e1486f76d440d54c06
And then further refined by c543ebf26831c13ae3ad22fd815c973107ef49b1


Add socket_keepalive options

fixed #99
Description
Add support to set socket keepalive options

@stj Nice code, thanks~


aks-engine 0.35.0

See https://github.com/Azure/aks-engine/releases/tag/v0.35.0

Thanks @bacongobbler! 🍺


Qt and jetgpio_example.cpp problem

hi, first of all thank you for pair for code..
my problem that, i am using Qt and i make copy code in jetgpio_example.cpp to mainwindow.cpp but editor show error
undefined reference to gpioInitialise'  undefined reference to gpioSetMode'
and other function, i did add in header #include <jetgpio.h> and read installation doc. make and make install, by the way , if i run jetgpio_example.cpp in with this way
Compile with: g++ -Wall -o jetgpio_example jetgpio_example.cpp -ljetgpio

Execute with: sudo ./jetgpio_example
everything is okey. but in Qt doesnt work.
have you got any idea..or solition

#include <stdint.h>
#include <stdlib.h>
#include <jetgpio.h>
MainWindow::MainWindow(QWidget *parent)
: QMainWindow(parent)
, ui(new Ui::MainWindow)
{
ui->setupUi(this);
int Init;
Init = gpioInitialise();
if (Init < 0)
{
   /* jetgpio initialisation failed */
   printf("Jetgpio initialisation failed. Error code:  %d\n", Init);
   exit(Init);
}
else
{
   /* jetgpio initialised okay*/
   printf("Jetgpio initialisation OK. Return code:  %d\n", Init);
}

// Setting up pin 3 as OUTPUT and 7 as INPUT

int stat1 = gpioSetMode(3, JET_OUTPUT);
if (stat1 < 0)
{
   /* gpio setting up failed */
   printf("gpio setting up failed. Error code:  %d\n", stat1);
   exit(Init);
}
else
{
   /* gpio setting up okay*/
   printf("gpio setting up okay. Return code:  %d\n", stat1);
}

int stat2 = gpioSetMode(7, JET_INPUT);
if (stat2 < 0)
{
   /* gpio setting up failed */
   printf("gpio setting up failed. Error code:  %d\n", stat2);
   exit(Init);
}
else
{
   /* gpio setting up okay*/
   printf("gpio setting up okay. Return code:  %d\n", stat2);

MainWindow::MainWindow(QWidget *parent)
: QMainWindow(parent)
, ui(new Ui::MainWindow)
{
ui->setupUi(this);
int Init;

Init = gpioInitialise();
if (Init < 0)
{
   /* jetgpio initialisation failed */
   printf("Jetgpio initialisation failed. Error code:  %d\n", Init);
   exit(Init);
}
else
{
   /* jetgpio initialised okay*/
   printf("Jetgpio initialisation OK. Return code:  %d\n", Init);
}

// Setting up pin 3 as OUTPUT and 7 as INPUT

int stat1 = gpioSetMode(3, JET_OUTPUT);
if (stat1 < 0)
{
   /* gpio setting up failed */
   printf("gpio setting up failed. Error code:  %d\n", stat1);
   exit(Init);
}
else
{
   /* gpio setting up okay*/
   printf("gpio setting up okay. Return code:  %d\n", stat1);
}

int stat2 = gpioSetMode(7, JET_INPUT);
if (stat2 < 0)
{
   /* gpio setting up failed */
   printf("gpio setting up failed. Error code:  %d\n", stat2);
   exit(Init);
}
else
{
   /* gpio setting up okay*/
   printf("gpio setting up okay. Return code:  %d\n", stat2);
}

MainWindow::MainWindow(QWidget *parent)
: QMainWindow(parent)
, ui(new Ui::MainWindow)
{
ui->setupUi(this);
int Init;
Init = gpioInitialise(); //**_<---- problem this line,**_
if (Init < 0)
{
   /* jetgpio initialisation failed */
   printf("Jetgpio initialisation failed. Error code:  %d\n", Init);
   exit(Init);
}
else
{
   /* jetgpio initialised okay*/
   printf("Jetgpio initialisation OK. Return code:  %d\n", Init);
}

// Setting up pin 3 as OUTPUT and 7 as INPUT

int stat1 = gpioSetMode(3, JET_OUTPUT);
if (stat1 < 0)
{
   /* gpio setting up failed */
   printf("gpio setting up failed. Error code:  %d\n", stat1);
   exit(Init);
}
else
{
   /* gpio setting up okay*/
   printf("gpio setting up okay. Return code:  %d\n", stat1);
}

int stat2 = gpioSetMode(7, JET_INPUT);
if (stat2 < 0)
{
   /* gpio setting up failed */
   printf("gpio setting up failed. Error code:  %d\n", stat2);
   exit(Init);
}
else
{
   /* gpio setting up okay*/
   printf("gpio setting up okay. Return code:  %d\n", stat2);
}

///-------------------------------------------------------------------///
thank you...
assyhgit

Hi there
That "undefined reference" when you try to compile looks like my library is not being linked, if you look at my example:
g++ -Wall -o jetgpio_example jetgpio_example.cpp -ljetgpio
You notice that I have to tell that the jetgpio library has to be linked, not sure what your development environment is or if you are using make, cmake or something like that, but you have to make sure that my library is being linked, so for instance in the following Makefile (not a real project just to give you an example):
CC=g++
CFLAGS=-I. -I/usr/include/qt4/QtGui -I/usr/include/qt4/QtCore -I/usr/include/qt4
LIBS=-lQtGui -lQtCore -ljetgpio
app: main.cpp
$(CC) $(CFLAGS) $(LIBS) $< -o $@
You notice how I put -ljetgpio within LIBS to make sure that is going to be linked

Related to how to do makefiles just google it, it is going to be easier for you and I am not an expert in the QT framework anyways.
Just to give you a brief explanation CFLAGS is a variable  the same as CC so for instance: CC CFLAGS -o jetgpio_example jetgpio_example.c -ljetgpio where CC = gcc and CFLAGS =  -Wall it is the same as writing: gcc -Wall -o jetgpio_example jetgpio_example.c -ljetgpio within the makefile.
Related to using sudo to execute any application created with my lib, there is nothing you can do about it, you have to use it. The error message is telling you why,  by using my lib you are reading/writing /dev/mem so you have to be root or use sudo, it is a protected file. A potential solution for not having to use sudo every time or log as root, would be to change the access rights/ownership of the /dev/mem file, I am going to leave it for you to google the stuff if you are interested!

thank you for explain.you are right.  i will find a solition... may be run command on startup to myApp through .sh file , sudo eco etc. or uploader app
thank you again..


Fix incorrect parser error assumption in semicolon handling leading to incremental parser brokenness

#43005 added special handling for missing semicolons to attempt to produce better errors. Unfortunately, it made an incorrect assumption about when the tokenizer would have already reported an error. This led to a class node with a syntax error being created without the flag noting that it contained an error (there's an initializer, but there's not a brace after it), which then led to the incremental parser trying to use that node rather than deciding it was unsafe.
Just remove this assumption; this fixes the new test case, and only reverts one error change introduced in #43005 (which doesn't seem correct to me anyway).
Fixes #47895

@typescript-bot cherry-pick this to release-4.6


 You should also verify that your $GOPATH is behaving correctly for cached packages.

Closing since there was no follow up, and it looks to be a local setup issue.


Abandoned Project?

Has this project been abandoned?

@tonytonyistony thanks for the question.
The project is in maintenance mode. I'll fix more significant issues, update ESI endpoints when they change, but I don't have any plans for new features.
Is there anything specific you are looking for?

I just hadn't seen any activity in the issue board in quite a while, I wanted to see how long it would take to get a response.
I actually just finished setting up your reconbot and Im having a hell of time getting it to post things in my discord. Whats even more frustrating is Im getting no errors in the logs or anything. So Im not quite sure what is breaking.
Do you have a slack or discord I could join to ask some questions about this? Or should I stick to this issue board for support?

Sorry to hear the tool is causing silent issues.
There is no message board for the project, but you can reach me at The Random One#5205 on Discord.

Ill hit you up tomorrow. Its getting late here and didn't think you would message back so fast. I've been messing with this most of the day and I need to get some sleep.

@flakas I sent you a friend request in discord

I'll close this ticket as we kind of figured it out.
Only a subset of notifications was whitelisted, so triggered ones during testing didn't show up.
As a future improvement, I'll configure the default run.py to allow all supported notification types.
Thanks for your help!


Sticky movement

The movement when I try to play is sticking to meshes and the ground it is also glitchy and teleporting me everywhere please help

what version

i cant even import it

2022.3.23f1

2022.3.17f1

I’ve seen other people with the same error and they all use the same version so try mine
2022.3.17f1


Refactor: make AppendEntriesResponse be an enum

It completes #197.

This change is

@mergify update

Thanks man!

CI failed in  t90_issue_216_stale_last_log_id::stale_last_log_id.
It seems main branch has set the timeout from 500ms to 1000ms, let me merge, and retry here.

CI failed in  t90_issue_216_stale_last_log_id::stale_last_log_id. It seems main branch has set the timeout from 500ms to 1000ms, let me merge, and retry here.

That's case that fails very easily... Maybe it should be removed...


Backport logic to DuckDB 1.0.0

These changes should allow to build the extension against DuckDB v1.0.0, that should mean easier access / distribution right away without having to wait September.
I tested it like:
carlo@ScroogeMcDuck sql-only-duckdb-extension % duckdb -unsigned
v1.0.0 1f98600c2c
Enter ".help" for usage hints.
D FORCE INSTALL dynamic_sql_examples FROM 'build/release/repository';
D LOAD dynamic_sql_examples;
D SELECT times_two(32);
┌───────────────┐
│ times_two(32) │
│     int32     │
├───────────────┤
│            64 │
└───────────────┘

using brew installed duckdb 1.0.0, and seems to work as expected.

You should strongly consider adding this to community extensions obviously.

Moved to draft since while looking at this I noticed a problem in the manylinux action that needs fixing.

I think the table functions are what were not 1.0.0 compatible. If you still have the build handy, mind testing times_two_table? Otherwise I can!

I think it's still works, like:
LOAD dynamic_sql_examples;
FROM times_two_table(100);
┌───────────────┐
│ output_column │
│     int32     │
├───────────────┤
│           200 │
└───────────────┘
FROM times_two_table(123.0);
┌───────────────┐
│ output_column │
│ decimal(14,1) │
├───────────────┤
│         246.0 │
└───────────────┘

Idea (if this makes sense) is that the implementation has been basically inlined.

Sounds great to me! Thanks Carlo!!

I think this is ready to be looked up.

Thanks Carlo! I really appreciate how much cleaner this fix is than my workaround.


[Bug]: When importing the app into the Git-connected app on a specific branch, breaks the application

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When importing the app into the Git-connected app on a specific branch, the app may break.
https://www.loom.com/share/74da40ad25984ab599a8bf73c1dc03e8
Steps To Reproduce

go to https://appsmith-huxvaxjvc-get-appsmith.vercel.app/
open any git connect app
checkout from master to new branch
now import the app
the imported app doesnt displayed and checkout to master break the application

Public Sample App
No response
Environment
Deploy Preview
Issue video log
No response
Version
deploy preview

We are going ahead with disabling update app via JSON import for git connected application.

@berzerkeer - Please disable the import button for Git-connected apps with the below message.
"This feature is not supported for apps connected to Git. Please use Git Pull to update and sync your app."


[Concurrency] Flip the implicitly async select in a concurrency error.

This diagnostic was presenting "implicitly async" for explicitly async calls, and vice versa, because the select was incorrect. The caller is passing apply->isImplicitlyAsync(), and true selects the second option.
Resolves https://github.com/swiftlang/swift/issues/74472

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test Linux


Add a way to provide the index settings from the extension configuration panel

Description
Currently, the extension indexes the documents add, update or delete to Firestore in a Meilisearch instance. It could be interesting to add the possibility to make settings. There are several that we could implement like searchableAttributes.
Basic example
We could add this setting option in the extension configuration panel:

To avoid overwhelming the user with to many questions, we prefer not implementing this for the moment. If asked for, we may reconsider

To avoid overwhelming the user with too many questions, we prefer not implementing this for the moment.


Cant run

hey Zackary, im looking to use this and expand on it, i was wondering if you could reach out and give me a hand with setup, discord: @mrdeadkingz

hi there,
may i know what errors you run into?
currently, it's available in pypi as an alpha release and `pip install reldplayer[all]. It should be able to set up on all dependencies

been trying to run it in pycharm, i just keep getting exit code 0's


Neo4JSession session leakages

Version used - 0.2.27
We are observing that a database with 50000 nodes ends up leaking sessions on subsequent add/update/read operations.
At first we thought it is because we were using a single instance of graph and creating multiple transactions. After the database is populated, we restarted our app and invoked just 1 call - read/write doesn't matter and it ends up in this error.
Could it be memory related? I think not because the node is either written or read properly. Please advise on the possible next steps.
Finalizing Neo4JSession [227030447] without explicit call to close(), the code is leaking sessions!

Please update issue with sample code, I am using this library on three applications on production for more than a year without any problems (more than 50K nodes).  Make sure you are using it like this:
    // create graph instance
    try (Graph graph = new Neo4JGraph(driver, vertexIdProvider, edgeIdProvider)) {
        // begin transaction
        try (Transaction transaction = graph.tx()) {
            // use Graph API to create, update and delete Vertices and Edges

            // commit transaction
            transaction.commit();
        }
    }

The scope of the graph instance is limited to a single transaction (short lived) (unit of work)

Thanks @rjbaucells. I see no more leaks, but my application that used to run at 45 (RPS) requests per second is now running at < 5. If you may share info, can you let know whats the throughput possible with this library and any suggestions to improvise?
fyi - We use the default Neo4JNativeElementIdProvider both vertex and edge.

ok, closing this.

The leaks observed have gone away by making graph instance as short lived unit.
One of the dependent services had some issues that caused slow turn around time. So highly likely that the library is not inducing the slowness.

Good to hear you got it fixed


Require cycle on react-native-sdk

[Fri Dec 03 2021 14:58:38.951]  WARN     Require cycle: node_modules/react-native-avo-inspector/dist-native/AvoInspector.js -> node_modules/react-native-avo-inspector/dist-native/AvoSessionTracker.js -> node_modules/react-native-avo-inspector/dist-native/AvoInspector.js

Require cycles are allowed, but can result in uninitialized values. Consider refactoring to remove the need for a cycle.
[Fri Dec 03 2021 14:58:38.952]  WARN     Require cycle: node_modules/react-native-avo-inspector/dist-native/AvoInspector.js -> node_modules/react-native-avo-inspector/dist-native/AvoBatcher.js -> node_modules/react-native-avo-inspector/dist-native/AvoInspector.js

Require cycles are allowed, but can result in uninitialized values. Consider refactoring to remove the need for a cycle.
[Fri Dec 03 2021 14:58:38.954]  WARN     Require cycle: node_modules/react-native-avo-inspector/dist-native/AvoInspector.js -> node_modules/react-native-avo-inspector/dist-native/AvoNetworkCallsHandler.js -> node_modules/react-native-avo-inspector/dist-native/AvoInspector.js

Require cycles are allowed, but can result in uninitialized values. Consider refactoring to remove the need for a cycle.
[Fri Dec 03 2021 14:58:38.955]  WARN     Require cycle: node_modules/react-native-avo-inspector/dist-native/AvoInspector.js -> node_modules/react-native-avo-inspector/dist-native/AvoNetworkCallsHandler.js -> node_modules/react-native-avo-inspector/dist-native/AvoInstallationId.js -> node_modules/react-native-avo-inspector/dist-native/AvoInspector.js

Hi,
We found some scary warnings when setting up react-native-avo-inspector
<EMAIL_ADDRESS><EMAIL_ADDRESS>Assistance would be greatly appreciated!
Cheers,
Sam

Thank you for reporting @sbycrosz. We'll take a look at this and get back to you!


Correct for the size of the tabs when calculating our initial window size

Summary of the Pull Request
This fixes our calculation for the initial size of the window. WE weren't accounting for the height of the tabs, so the initialRows was consistently wrong.
PR Checklist

[x] Closes #2061
[x] I work here
[ ] Tests added/passed
[n/a] Requires documentation to be updated

Detailed Description of the Pull Request / Additional comments
For the tabs below the titlebar case, there's 6px (unscaled) of space that I cannot account for. I seriously have no idea where it's coming from. When we end up creating the first TermControl after startup, there's an inexplicable 6*scale difference between the height of the tabContent and the SwapChainPanel's size.
Validation Steps Performed
Checked all six of the following cases:

1.0 DPI scaling, Tabs in Titlebar
1.25 DPI scaling, Tabs in Titlebar
1.0 DPI scaling, Tabs NOT in Titlebar, always show tabs
1.0 DPI scaling, Tabs NOT in Titlebar, DON'T always show tabs
1.25 DPI scaling, Tabs NOT in Titlebar, always show tabs
1.25 DPI scaling, Tabs NOT in Titlebar, DON'T always show tabs

Er did we ever figure out if greg904 was right? Probably we should have blocked this

Oh sorry, never replied to that - no, we're shockingly not. Hence the comment there. In startup, _titlebar is nullptr, so we don't add anything for it there.


[Drawer] breaks DOM scrolling when shown while hiding dialog

If you have a scrolling page with a dockable LeftNav and docked={false}, and that LeftNav is the parent of a React component, it creates a giant div that breaks scrolling if you attempt to show it upon closing another modal dialog (using states as switches).
Currently using:
ReactJS : 0.14.7
Material-UI : HEAD@1d4c4a8
Google Chrome Version 47.0.2526.106

Thanks for the clear report! :+1:
An example would be good too.
Do you want to have a go at fixing it?

I haven't had a chance to figure it out since unfortunately (well fortunately) I can still test the rest of my app for the time being (its a CMS) and I had a lot of other priorities/deadlines with my current sprint.
I did look around earlier this week to get these details and couldn't out my finger on what it changed in the DOM to cause this spefically... but I noticed it happened as soon as I created a LeftNav component that was self contained within its own React Component render (and this issue doesn't occur when docked=true, however for a modal this is not too great). Sorry for not being more elaborate, but if I can find some time to make a jist next week I will definitely do that.

Well, this thing seems beyond me. Since it seems a pretty serious UI issue, can one of the project devs look into it? I can also confirm that this issue seems to be global (as in all browsers) as it also breaks my Firefox (v42.0)

Without an example to reproduce the issue with, there's really very little we can do to help, sorry.

Sorry about the delayed response; I had an issue and found the issue more specifically! It basically happens if you have any dialog pop up, close it via a state flag during an action while simultaneously pulling out the LeftNav (also via setState). For now, I am using a workaround of setting few ms timeout before updating state on the callback, but it's still an issue. I will also test a few alternative fixes as workaround now that are not as hacky.

We have been porting the component on the v1-beta branch. We reimplemented it from the ground-up. While we haven't tested it, I think that the issue is most likely fixed on that branch. Hence, I'm closing it.
Still, we will accept PR fixes until v1-beta takes over the master branch.


refactor: make it more clear where to start and what repository to use

Make it clear, that a Student have to clone the repo and write the code in the cloned repo.
(this PR was born because of complaints from other students, who didn't understand what to do. So let's minimize confusing)

another example when people are confused by not clear instructions: https://discordapp.com/channels/755676888680366081/1057556451540877384/1066031631971926026


Domain activation is Failed - [BRIEF DESCRIPTION]

App Id: d1fvrxwvjqvoa8
Region: ap-southeast-1
Step: Domain activation
Status: Failed

Note: Do not include information that is sensitive in nature such as your domain name, company etc.

Issue/question
A clear and concise description of what the issue/question is.
Error message
If there is an error message, please include it here.
Note: Be sure to check the message for sensitive information.
Additional information
Please add any other relevant information. Please feel free to include screenshots.

I created amplify app once  no problem with custom domain and deleted
after that I create new amplify and have error on 'domain activation' step

@itsGno What exactly is the error message you are seeing?
From what I can tell your domain domain is AVAILABLE and active with respect to this appId: d1fvrxwvjqvoa8.
Are you still having issues?

Resolving due to age - re-open if you are still facing the issue!


Update doc with bundled Rouge styles

I updated documentation with themes that are bundled with Rouge.

Thanks for this addition. Great suggestion!


rustflags overwritten not added

Problem
When both
[build]
rustflags =

and
[target.'cfg(...)']
rustflags =

are used, the target flags overwrite the build flags instead of adding on top of them.
This seems like a bug to me since I would expect [build] rust flags to be globals that are added upon.
Steps
The following is a failing test case I would expect to succeed:
#[cargo_test]
fn build_flags_and_target_flags() {
    let config = format!(
        r#"
            [build]
            rustflags = ["-Wunused"]
            [target.'cfg(target_arch = "{}")']
            rustflags = ["-Wunsafe-code"]
        "#,
        std::env::consts::ARCH
    );

    let p = project()
        .file(".cargo/config.toml", config.as_str())
        .file("src/lib.rs", r#""#)
        .build();

    p.cargo("build -vv")
        .with_stderr_contains("[..]-Wunsafe-code[..]")
        .with_stderr_contains("[..]-Wunused[..]")
        .run();
}

Possible Solution(s)
A possible solution would be changing core/compiler/build_context/target_info.rs:env_args
end of function to:
    let mut all_rustflags = Vec::new();
    if let Some(rustflags) = rustflags_from_env(flags) {
        all_rustflags.extend(rustflags);
    }
    if let Some(rustflags) = rustflags_from_target(config, host_triple, target_cfg, kind, flags)? {
        all_rustflags.extend(rustflags);
    }
    if let Some(rustflags) = rustflags_from_build(config, flags)? {
        all_rustflags.extend(rustflags);
    }
    Ok(all_rustflags)

Notes
No response
Version
Stable and Nightly both reproduce
Nightly:
cargo 1.66.0-nightly (082503982 2022-09-13)
release: 1.66.0-nightly
commit-hash: 082503982ea0fb7a8fd72210427d43a2e2128a63
commit-date: 2022-09-13
host: aarch64-apple-darwin
libgit2: 1.5.0 (sys:0.15.0 vendored)
libcurl: 7.79.1 (sys:0.4.55+curl-7.83.1 system ssl:(SecureTransport) LibreSSL/3.3.6)
os: Mac OS 12.5.1 [64-bit]

Stable:
cargo 1.63.0 (fd9c4297c 2022-07-01)
release: 1.63.0
commit-hash: fd9c4297ccbee36d39e9a79067edab0b614edb5a
commit-date: 2022-07-01
host: aarch64-apple-darwin
libgit2: 1.4.2 (sys:0.14.2 vendored)
libcurl: 7.79.1 (sys:0.4.55+curl-7.83.1 system ssl:(SecureTransport) LibreSSL/3.3.6)
os: Mac OS 12.5.1 [64-bit]

Thank you for this comprehensive report, and even with tests! I really appreciate that.
I believe this is a duplicate of #5376, so I'll close this in favour of that, as more discussions happened there. Thank you.


[BUG] Python cannot find the module after manually changing python version

Describe the bug
After changing the python version to 3.11.6 (Due to the shipped version being quite old and 3.11.6 having performance improvements) using this guide my python script which uses the picamera2 module python cannot find the module anymore.
To Reproduce

Install Python 3.11.6 using this guide
Install the picamera2 module using the apt method in the README
Run a python script that uses the picamera2 module

Expected behaviour
The script would execute without any errors and work as it would with the Python version that is installed by default
Console Output, Screenshots
pi@raspberrypi:~/Desktop/Overengineering-squared/new Programm/Python $ python cam.py Traceback (most recent call last):   File "/home/pi/Desktop/Overengineering-squared/new Programm/Python/cam.py", line 6, in <module>     from picamera2 import Picamera2 ModuleNotFoundError: No module named 'picamera2'
Hardware :
Raspberry Pi 4b with 4GB of ram
Camera Module V3
Additional Information
Installing the module using the pip method does not work due to it freezing while installing PyQt5 at the Preparing metadata stage

Hi, unfortunately the libcamera and kms++ packages (these are not Raspberry Pi packages) do not have Python bindings that are installable with different versions of Python, so this is not possible. Actually we are looking at the problem but the main issue is that the libcamera library does not have sufficient ABI stability for it to be easy to create Python bindings for different versions of Python, and different versions of libcamera, that will actually work.

Thanks for the answer.
Guess il have to use python 3.9.x until the Pi 5 releases which hopefully ships with a newer version of python as its based on a newer debian where the issues are hopefully fixed.

FYI Debian Bookworm uses Python 3.11.x


Use attribute((unused)) for clang too

Otherwise, results in warnings like
.../STC/include\stx/../stc/cvec.h(280,1): warning: unused function 'vec_int_drop' [-Wunused-function]

At least for clang on Windows, __clang__ is defined, but __GNUC__ is not (as confirmed by clang -E -dm -x c nul).

Hi, thanks. I use the mingw distrubution on winlibs.com, where clang defines GNUC, so I assumed that was always the case.


Mobile - Toggle Re design for Staff / Client Area

Mobile - Toggle Re design for Staff / Client Area
"Go to Area" is not good, need button like structure