Brighterscript on the horizon at all?

Curious if you had any thoughts on brighterscript integration at all in the future or if so, would you be interested in myself helping to make it happen?
I've gotten really used to the nice syntax of brighterscript and it makes things feel very clean and integrated so wasn't sure if you had considered it at all?

Hey @jfrux , thanks for your interest and potential support!
We already passed a phase of swearing for the BrightScript syntax looking for other solutions and BrighterScript was really tempting to use. We ended with a feeling that in such a small community of BS language creating a transpileable language and maintaining it can be tricky so we do have not enough trust in such movement.
We had an attempt to use ROPM as the importing system, even created a few PRs but we found too many difficulties and required changes to achieve our goals (which we didn't fully achieve yet). I'm mentioning this because it would close us to use brighterscript in the project by default.
Also, as the whole Kopytko Ecosystem is kinda opinionated, we didn't want to enforce its users to use BrighterScript. However, if we did it, we would do it as a plugin thanks to the packager's plugins mechanism. So if this is something you need or find helpful for other users, we encourage you to create a public plugin for the packager with a BrighterScript transpilation action. If you decide to do it, please let us know so we can help you by making a code review and link to your plugin in the docs :)
Sorry for a delayed response

Wow! Thank you so much for the detailed response.
I really appreciate it!
No worries on not buying into it.  I fully understand!
I tried to get started using this on a project but I did find that I had trouble understanding the best places to put some of my logic.  It had a quick on-boarding experience but felt limited quickly.  Is that normal? Are there any examples you know of that make use of more advanced UI techniques?

Thank you for your opinion. I get your point - even if it's quite opinionated, it still has not enough real-life examples. I think my example app could help you: https://github.com/pawelhertman/kopytko-example-app. It shows a lot of possibilities and advantages of Kopytko Framework, Utils, and Packager. Please keep in mind it's not an official example app. The official one is in our backlog.

No worries at all! Thank you for sharing this! Will take a look.
I’m happy you’re working on this.  It is a much needed effort in this community to have something more rapid to develop on.  Really do appreciate your efforts!
Keep it up!
From: Paweł Hertman @.>
Date: Thursday, March 24, 2022 at 9:07 AM
To: getndazn/kopytko-framework @.>
Cc: jfrux @.>, Mention @.>
Subject: Re: [getndazn/kopytko-framework] Brighterscript on the horizon at all? (Issue #23)
Thank you for your opinion. I get your point - even if it's quite opinionated, it still has not enough real-life examples. I think my example app could help you: https://github.com/pawelhertman/kopytko-example-app. It shows a lot of possibilities and advantages of Kopytko Framework, Utils, and Packager. Please keep in mind it's not an official example app. The official one is in our backlog.
—
Reply to this email directly, view it on GitHubhttps://github.com/getndazn/kopytko-framework/issues/23#issuecomment-1077608364, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AABJYRLIAMP5FDOSXUENG53VBRSJ5ANCNFSM5QGTVBFQ.
You are receiving this because you were mentioned.Message ID: @.***>

Thanks :) Is there anything else we could help you with in terms of the topic of this "issue"? If not, I'm gonna close it. Feel free to join our slack when seeking help.


Please check ioBroker.unifi with js-controller 2.0

Hi,
the new js-controller 2.0 will come into latest repository in the next days and we want to make sure that all adapters are working well. We already did a 2 weeks Beta test and so some adapter were aleady checked and some needed slight adjustments.
You can find more information in https://github.com/ioBroker/ioBroker.js-controller/issues/482 and in the ioBroker Forum. If you have more technical questions please write in the referenced issue or in the Developer thread please. General questions are best in the genral thread.
Please update your systems to js-controller 2.0 and check your adapter.
Please close this issue once you have checked your adapter or received successfull reports from users.
Thank you very much for your support. Please contact us in the other Threads or Forum on any question.

Fixed with e9ba4cd.

In the ioBroker forum it was confirmed, that the adapter works with js-controller v2.2.9.


podspec needs to be updated to point to latest JXLS updates

The current pod spec points to tag 1.0 in the repository.  The pod spec should be updated to point to the latest changes.  I'm not an expert at podfile creation.  If a 1.1 (or 1.0.1) tag was created, the pod spec could be pointed to that tag instead.  There should be a way to create the pod spec, so that you could use 1.0 or 1.0.1 depending on your Podfile.  However, I don't understand exactly how that is done.  I thought about creating this as a pull request, but I'm not sure how I would do that since you would need to additional git tags, etc.
Thanks!

+1

bump

I don’t use pods. PRs welcome. :)

+1


What does the "desync_flag" parameter do?

Hi,
We are having an issue synthesising the configuration block as I have an inconsistency in the setting of the "desync_flag" paramater.
What should this be set to?
For information, the demo has it set to 20 - which is the same as MaxFramesPerCol=20 and in the BRAM block "WRITE_ENABLE_FROM_DATA = 20".
Many thanks,
Rob.

I don't know what that does, either. I have forwarded it to @dirk-koch and am still waiting for a reply. He is quite busy recently, so might take a while.

This can be handy for various situations, including adding software binaries into a single file.
However, in order to encode when we should start with interpreting the bitstream, we mandate that the header is padded to 32-bit (4 bytes) and that the first word is FAB0 FAB1.
We configure frame by frame and each frame starts with an 32-bit address and then one extra 32-bit word per fabric row (CLB row or elementary block). This is slightly different to Xilinx who do a frame autoincrement and in FABulous you may write frames in arbitrary order and even multiple times, if required.
This is used to initialize BRAMs and the register-file tiles.
We had deliberately not foreseen an extra port for configuration (nobody would do that); and instead of multiplexing an existing port, we can directly use the fabric to initialize anything (takes a little more time, if needed, but is cheaper).
Anyway, in order to tell the FPGA (i.e. the configuration state machine in ConfigFSM.v) that we are done with configuration (a time out is not a clean way to do that), we simply use one bit in the address field of a frame to indicate that. You can then even send meta data again and config data.
By using different FAB0 FABx keys, we could configure multiple paralleled fabrics from the same bistream
One thing that we will change in the bitstream format is that the we will also use a binary format to address frames within a column (so a binary format for both major and minor addresses.
Right now, we have a 20-bit bitmask for frames (minor addresses) which, well limits the number of frames to 20 (without further encoding). And with this, we have 32x20 config bits, which is likely too little for LUT-6 fabrics.


Event when clicking on the icon

Is it possible to have an event that listens for a click on the icon itself? Instead of waiting for a click on one of the menu items below it.

hey @mdings!
First, sorry for the late reply. My github notifications are a mess.

Is it possible to have an event that listens for a click on the icon itself?

The OS integration and  event creation happens in https://github.com/ssbc/systrayhelper so the feature needs to be added there first.
Since this is essentially the menu opened event, I don't think it would be that hard to add.
Is this still relevant for you?


SASS files are missing in NPM package

According to the documentation 'src' folder should be available in npm but it is missing.

Enjoy :) I added it in 3.0.0-beta.5


Doesn't work with deno compile

This doesn't work with deno compile because Deno.execPath returns the path to the compiled executable instead of deno

Use the portable loader (loader: "portable") in the plugin options.


Improve Sharp Aquos TV component - Fixes #4973

Description:
Improve Sharp Aquos TV component - Fixes #4973
Related issue (if applicable): fixes #4973
Checklist:
If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

Thanks! 🐬


Key Update Malfunction

What happened?
When attempting to enter Carte Key, field does not retain number and the number is not saved.
What did you expect to happen?
Key was not saved.
Version
3.5.x
What browser are you seeing the problem on?
Firefox, Chrome, Safari, Microsoft Edge
Relevant log output
Uncaught TypeError: (intermediate value)(...) is not a function

This issue will be closed and archived in 3 days, as there has been no activity in the last 60 days. If this issue is still relevant or you would like to see it actioned, please respond and we will re-open this issue. If this issue is critical to your business, consider the Dedicated Support Service where a Service Level Agreement is offered.


Add integration test for Harvester network controller

Test case

Enable VLAN network, set default physical NIC
Change physical NIC of a node

empty NIC
valid NIC
invalid NIC

Disable VLAN network

close as network e2e tests was added in https://github.com/harvester/tests


Build: Rebuild when libhermit-rs is modified

Uses thecargo:rerun-if-changed=PATHdirective to inform cargo which files in libhermit-rs should be watched and trigger a rebuild of hermit-sys. We don't need to do this for the files in hermit-sys, since cargo already watches those.
I used this pattern as a template.

This helps a lot. Thanks!


[python-package] How to refit a classifier?

I have a timeseries data that I have to fit a classifier, I would like to re-train it every month with new data coming in. I would like to keep some consistency, so I prefer to pre_warm the tree with the previous boosting trees and refit the data, or just iterate few times to get the results I need. In other words I want similar tree structure more or less.
Seems like the refit function is the best way to do it but unfortunately It doesnt seem to be an option for scikit API only boosters. What is the best way I can approach this? I have done the following thus far:
{'objective': 'multiclassova',
 'num_class': 6,
 'boosting_type': 'gbdt',
 'reg_alpha': 0.0,
 'reg_lambda': 0.0,
 'num_leaves': 103,
 'feature_fraction': 0.9311573062675359,
 'bagging_fraction': 0.9568372729883741,
 'bagging_freq': 1,
 'min_child_samples': 54,
 'learning_rate': 0.056834238901176865,
 'max_depth': 99,
 'min_data_in_leaf': 2,
 'min_gain_to_split': 2.272224629201629,
 'drop_rate': 0.6143619198733702,
 'n_estimators': 232,
 'force_col_wise': True,
 'class_weight': {1: 1.2466666666666666,
  5: 0.29088888888888886,
  3: 0.3177184466019417,
  0: 0.6233333333333333,
  4: 1.3357142857142859,
  2: 3.85},
 'seed': 42,
 'random_state': 42,
 'verbose': -1,
 'eval_metric': 'precision'}

classifier_obj = lgb.LGBMClassifier(**params)
classifier_obj.fit(X_train, y_train, categorical_feature=categorical_data + binary_data)

booster = lgb.train(params={}, train_set=data_set['train_data'], categorical_feature=categorical_data + binary_data, init_model=classifier_obj, keep_training_booster=True, num_boost_round=1)

booster.refit(data=X_train, label=y_train)

I'm sure this is wrong by just looking at the outputs can anyone point to the right direction? Thanks

Thanks for using LightGBM.
Is it absolutely necessary to "refit" (modify the values of the leaf nodes without changing the total number of trees)? Or would it bee acceptable to add more trees, trained on the newly-arrived data. If you clarify that precisely, it would help us to offer some advice.
Please also see this explanation: https://stackoverflow.com/questions/73664093/lightgbm-train-vs-update-vs-refit/73669068#73669068

Also...I see that you double-posted this here and on Stack Overflow (link. Please do not do that.
Maintainers here also monitor the [lightgbm] tag on Stack Overflow. I could have been spending time preparing an answer here while another maintainer was spending time answering your Stack Overflow post, which would have been a waste of maintainers' limited attention that could otherwise have been spent improving this project. Double-posting also makes it less likely that others with a similar question will find the relevant discussion and answer.

Hi James, thanks for the response. I will take a look.
also, I literally posted on StackOverflow about 30 min ago. I waited few days, in-case this got backlog and would take longer than expected to get a respond. Out of respect to you I will link this to StackOverflow. Thanks in advance.

It would be best if we can use the same exact "refit", if not I can use the update function?

What is preventing you from using Booster.refit()? You showed an example using that and said "this is wrong by just looking at the outputs", but didn't share those outputs or explain what is "wrong" about them.
We would be happy to help but need your help to understand what specifically you are looking for.


refactor: clean up form, use fetch instead of XHR, use catppuccin theme

also added a basic favicon and the Inter font :p

also it added the sharex commit cuz dummy me forgot to pull new changes (oops), apologies


Added return statements for addNote and addBody

This will allow bridge to auto-select a body or note when they are created by the user.

Closes #85

[Current coverage][cc-pull] is 93.92%

Merging [#86][cc-pull] into [master][cc-base-branch] will increase coverage by -6.07%

@@             master        #86   diff @@
==========================================
  Files             6          6
  Lines           849        395   -454
  Methods          67         67
  Messages          0          0
  Branches         88         88
==========================================
- Hits            849        371   -478
- Misses            0         24    +24
  Partials          0          0

5 files (not in diff) in src were modified. more

Misses +17
Hits -327

File src/engine.js was modified. more

Misses +7
Partials 0
Hits -150

Powered by Codecov. Last updated by 00a0287
[cc-base-branch]: https://codecov.io/gh/orbitable/engine/branch/master?src=pr
[cc-pull]: https://codecov.io/gh/orbitable/engine/pull/86?src=pr


[question] how to register a "traditional" Lua-C function into a Sol2 sol::state?

I have inherited a code that uses Lua's C-API to expose a C function int some_function(lua_State *L). I am trying to refactor the code to use Sol2 but I would like to avoid touching the content of this function. How can I register it using a sol::state object?

You should be able to bind it like you would any other function:
lua["some_function"] = some_function;

If the function signature is int (lua_State*) should get automatically bound like a normal lua_CFunction (see here).


Quickstart guide does not work with hostpath on bare metal

/kind bug

What versions of software are you using?
Operating System: Linux
Output of odo version: odo v2.5.0 (724f16e68)
How did you run odo exactly?
Take the example from the quick start guide, and using a hostpath-created-PV, try to write to the folder when you do odo push:
git clone https://github.com/odo-devfiles/odo-quickstart
cd odo-quickstart/backend
odo create java backend
odo push

Actual behavior
You'll get an error when pushing:
~/odo-quickstart/backend  main ✗                                                                                119d ◒
▶ odo push

Validation
 ✓  Validating the devfile [21427ns]

Creating Services for component java
 ✓  Created service "PostgresCluster/hippo" on the cluster; refer "odo link -h" to know how to link it to the component

Creating Kubernetes resources for component java
 ✓  Added storage m2 to java
 ✓  Waiting for component to start [10s]
 ✓  Links are in sync with the cluster, no changes are required
 ✓  Waiting for component to start [16ms]
 ⚠  Unable to create ingress, missing host information for Endpoint 8080-tcp, please check instructions on URL creation (refer `odo url create --help`)

Applying URL changes
 ✓  URL java: http://java.k8s.land/ created

Syncing to component java
 ✓  Checking files for pushing [5ms]
 ✓  Syncing files to the component [242ms]

Executing devfile commands for component java
 ✗  Executing build command "mvn clean -Dmaven.repo.local=/home/user/.m2/repository package -Dmaven.test.skip=true" [2s]
 ✗  Failed to start component with name "java". Error: Failed to create the component: command execution failed: unable to execute the run command: unable to exec command [/bin/sh -c mvn clean -Dmaven.repo.local=/home/user/.m2/repository package -Dmaven.test.skip=true]:
[ERROR] Could not create local repository at /home/user/.m2/repository -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/LocalRepositoryNotAccessibleException
: error while streaming command: command terminated with exit code 1

If you check the PVC and PV, you can see that it's been provisioned properly, but the directory is not writeable:
~/odo-quickstart/backend  main ✗                                                                                119d ◒  ⍉
▶ k get pvc
NAME                          STATUS   VOLUME                                     CAPACITY   ACCESS MODES   STORAGECLASS   AGE
hippo-instance1-lvwd-pgdata   Bound    pvc-4f8f5163-08bc-49c6-b7fd-d1ff5b3c60c3   1Gi        RWO            hostpath       4m59s
hippo-repo1                   Bound    pvc-f683a3cb-5a58-46db-9ec1-9ed147bb2535   1Gi        RWO            hostpath       4m57s
hippo-repo2                   Bound    pvc-307e5b94-f704-470c-ab95-937008a4384a   1Gi        RWO            hostpath       4m56s
m2-java-app                   Bound    pvc-a615534d-7294-4426-bdc4-5a9f9dff6154   3Gi        RWO            hostpath       5m1s

~/odo-quickstart/backend  main ✗                                                                                119d ◒
▶ k get pv
NAME                                       CAPACITY   ACCESS MODES   RECLAIM POLICY   STATUS   CLAIM                                    STORAGECLASS   REASON   AGE
pvc-2cdfd4ce-61da-4ffd-ae23-458e11369165   500Gi      ROX            Delete           Bound    default/duplicati-backup-files           hostpath                14d
pvc-307e5b94-f704-470c-ab95-937008a4384a   1Gi        RWO            Delete           Bound    quickstart/hippo-repo2                   hostpath                4m57s
pvc-4f8f5163-08bc-49c6-b7fd-d1ff5b3c60c3   1Gi        RWO            Delete           Bound    quickstart/hippo-instance1-lvwd-pgdata   hostpath                5m1s
pvc-57edc694-e04d-4e7e-a405-5bbab054a109   1Gi        RWO            Delete           Bound    default/duplicati-config                 hostpath                14d
pvc-94c70009-d686-4abf-8780-52accc66658f   10Gi       RWO            Delete           Bound    default/duplicati-backups                hostpath                14d
pvc-a615534d-7294-4426-bdc4-5a9f9dff6154   3Gi        RWO            Delete           Bound    quickstart/m2-java-app                   hostpath                5m3s
pvc-c87e52dd-20d5-4beb-9c43-815662b9e893   1Gi        RWO            Delete           Bound    default/duplicati-source                 hostpath                14d
pvc-f683a3cb-5a58-46db-9ec1-9ed147bb2535   1Gi        RWO            Delete           Bound    quickstart/hippo-repo1                   hostpath                4m59s

And when you check folder permissions, it's using root:
server@chewie:~/k8s-volumes$ ls -lah
total 52K
drwxr-xr-x 13 root   root   4.0K Jan 25 15:46 .
drwxr-xr-x  8 server server 4.0K Jan 11 09:45 ..
drwxr-xr-x  2 root   root   4.0K Jan 25 14:57 pvc-13a3cec8-69be-4ccb-a680-fba7f85fda37
drwxr-xr-x  2 root   root   4.0K Jan 25 15:46 pvc-307e5b94-f704-470c-ab95-937008a4384a
drwxr-xr-x  2 root   root   4.0K Jan 25 14:59 pvc-3534a38c-e29b-48db-b89c-0b9d20ee412f
drwxr-xr-x  2 root   root   4.0K Jan 25 15:45 pvc-4f8f5163-08bc-49c6-b7fd-d1ff5b3c60c3
drwxr-xr-x  7 server server 4.0K Jan 25 09:53 pvc-57edc694-e04d-4e7e-a405-5bbab054a109
drwxrwxrwx  3 root   root   4.0K Jan 25 15:10 pvc-607af353-3317-4000-92b5-13e501764547
drwxr-xr-x  2 root   root   4.0K Jan 11 09:51 pvc-94c70009-d686-4abf-8780-52accc66658f
drwxr-xr-x  2 root   root   4.0K Jan 25 15:03 pvc-9e86e2a4-dde8-4f02-ac24-eb1a598e7829
drwxrwxrwx  3 root   root   4.0K Jan 25 15:44 pvc-a615534d-7294-4426-bdc4-5a9f9dff6154
drwxr-xr-x  2 root   root   4.0K Jan 11 09:51 pvc-c87e52dd-20d5-4beb-9c43-815662b9e893
drwxr-xr-x  2 root   root   4.0K Jan 25 15:46 pvc-f683a3cb-5a58-46db-9ec1-9ed147bb2535

Unfortunately, it's because the container is trying to use the user user and not root causing you to be unable to write to the folder since the created PV is owned by root.
Expected behavior
Be able to write to the folder
Any logs, error output, etc?
See above

I don't think that there is anything that odo can do in this case. This is a matter of proper PV setup, and it heavily depends on underlying storage provisioned. This is out of odo's scope.

@kadel I feel as though we may need some documentation that details the caveats that developers may encounter with their bare-metal deployment. I'll make sure to create documentation as I go regarding this.

@cdrage are you trying on a real k8s cluster? That is, neither minikube nor CRC?

Yup. Real k8s cluster deployed via k0s. Which imitates hybrid + bare metal environments at data centers.


python312Packages.python-utils: 3.8.2 -> 3.9.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.python-utils is: Module with some convenient utilities
meta.homepage for python312Packages.python-utils is: https://github.com/WoLpH/python-utils
meta.changelog for python312Packages.python-utils is: https://github.com/wolph/python-utils/releases/tag/v3.9.0
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.9.0 with grep in /nix/store/6s9zfm8ric28nw1j81h1rd4k99fjp5xf-python3.12-python-utils-3.9.0
found 3.9.0 in filename of file in /nix/store/6s9zfm8ric28nw1j81h1rd4k99fjp5xf-python3.12-python-utils-3.9.0

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

24 total rebuild path(s)

24 package rebuild(s)

First fifty rebuilds by attrpath

cura
portmod
python311Packages.angr
python311Packages.angrcli
python311Packages.angrop
python311Packages.awswrangler
python311Packages.baselines
python311Packages.gradient
python311Packages.numpy-stl
python311Packages.progressbar2
python311Packages.python-utils
python311Packages.xnatpy
python312Packages.angr
python312Packages.angrcli
python312Packages.angrop
python312Packages.awswrangler
python312Packages.gradient
python312Packages.numpy-stl
python312Packages.progressbar2
python312Packages.python-utils
python312Packages.xnatpy
trueseeing
twitch-chat-downloader

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6s9zfm8ric28nw1j81h1rd4k99fjp5xf-python3.12-python-utils-3.9.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/918c171f93c7dfa663315d9a502683ca48a34d4e#python312Packages.python-utils

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6s9zfm8ric28nw1j81h1rd4k99fjp5xf-python3.12-python-utils-3.9.0
ls -la /nix/store/6s9zfm8ric28nw1j81h1rd4k99fjp5xf-python3.12-python-utils-3.9.0/bin

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

3 packages failed to build:

cura
portmod
portmod.dist

42 packages built:

python311Packages.angr
python311Packages.angr.dist
python311Packages.angrcli
python311Packages.angrcli.dist
python311Packages.angrop
python311Packages.angrop.dist
python311Packages.awswrangler
python311Packages.awswrangler.dist
python311Packages.baselines
python311Packages.baselines.dist
python311Packages.gradient
python311Packages.gradient.dist
python311Packages.numpy-stl
python311Packages.numpy-stl.dist
python311Packages.progressbar2
python311Packages.progressbar2.dist
python311Packages.python-utils
python311Packages.python-utils.dist
python311Packages.xnatpy
python311Packages.xnatpy.dist
python312Packages.angr
python312Packages.angr.dist
python312Packages.angrcli
python312Packages.angrcli.dist
python312Packages.angrop
python312Packages.angrop.dist
python312Packages.awswrangler
python312Packages.awswrangler.dist
python312Packages.gradient
python312Packages.gradient.dist
python312Packages.numpy-stl
python312Packages.numpy-stl.dist
python312Packages.progressbar2
python312Packages.progressbar2.dist
python312Packages.python-utils
python312Packages.python-utils.dist
python312Packages.xnatpy
python312Packages.xnatpy.dist
trueseeing
trueseeing.dist
twitch-chat-downloader
twitch-chat-downloader.dist

Maintainer pings

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 344351 at 1351ea87 run on aarch64-darwin 1

:fast_forward: 7 packages marked as broken and skipped:

python311Packages.angr
python311Packages.angrcli
python311Packages.angrop
python311Packages.baselines
python312Packages.angr
python312Packages.angrcli
python312Packages.angrop

:white_check_mark: 14 packages built:

python311Packages.awswrangler
python311Packages.gradient
python311Packages.numpy-stl
python311Packages.progressbar2
python311Packages.python-utils
python311Packages.xnatpy
python312Packages.awswrangler
python312Packages.gradient
python312Packages.numpy-stl
python312Packages.progressbar2
python312Packages.python-utils
python312Packages.xnatpy
trueseeing
twitch-chat-downloader


$bits function returns 0 when struct is located outside of current package.

package b;
  typedef struct packed {
    logic                one;
    logic [10:0]         vc;
    logic [10:0]         vb;
  } foo_bar_t;

endpackage : b

package aa;

  function automatic b::foo_bar_t foo_bar;

    input b::foo_bar_t a;

    b::foo_bar_t y;
    reg [$bits(a)-1:0] a_vec;

    begin
      foo_bar = y; // return the whole struct
    end

  endfunction : foo_bar
endpackage : aa

$bits(a) is returned incorrectly as 0 when it should return 23.

@hzeller One problem I found in our internal project

Fixed by #2539


Error on opening Pull request

Gitea version:
Git version: 345bc06
Operating system: docker container
Database:

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Log gist:

Description
Opening a new pull request for a freshly pushed branch sometimes gives an error with an unformated html file and the following message in a <pre> tag:
template: repo/diff/box:109:21: executing "repo/diff/box" at <call $.IsImageFile $file.Name>: error calling call: runtime error: invalid memory address or nil pointer dereference

url for this is gitea.xyz/user/repo/compare/master...branch
Not sure if this helps but this is the last line of the gitea.log:
_issues`, `05/14 12:03:42 ^[[0m^[[32m...xorm/session_find.go:195:^[[32mnoCacheFind()^[[0m ^[[1;32m[I]^[[0m [SQL] ^[[1mSELECT `id`, `repo_id`, `name`, `content`, `is_closed`, `num_issues`, `05/14 12:03:42 ^[[0m^[[32m...xorm/session_find.go:195:^[[32mnoCacheFind()^[[0m ^[[1;32m[I]^[[0m [SQL] ^[[1mSELECT `id`, `user_id`, `repo_id`, `mode` FROM `access` WHERE (repo_id 05/14 12:03:42 ^[[0m^[[32m...s/context/context.go:138:^[[32mHTML()^[[0m ^[[1;34m[D]^[[0m Template: ^[[1mrepo/pulls/compare^[[0m

Could it be that it is for added or deleted files?

I did the following to fix the error:

reset one file that was staged for deletion
rebased the branch on current master
took everything to a new branch

Now the pull request works but it feels like a workaround.

I just stumbled upon the same issue. In my case it happens when the diff contains a submodule.
Like here: https://try.gitea.io/saitho/test2/compare/noimage...submodule2

I also have submodules activated for that branch but I also have git lfs active. Is there a possibility to add a bit more debug output for what is happening here?

So the problem is when your PR has a submodule. @ukos-git Could you change the title?


BIG-IP v<IP_ADDRESS>.0.1645 image in Liberty fails test_health_monitor_basic test

Title: BIG-IP v<IP_ADDRESS>.0.1645 image in Liberty fails test_health_monitor_basic test
Attachments:

test_health_monitor_basic.txt

Details: For bugs, copy and paste the following template into your new issue and fill it out.

Agent Version
8.3.1
Operating System
Linux host-136.int.lineratesystems.com 3.10.0-327.10.1.el7.x86_64 #1 SMP Tue Feb 16 17:03:50 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
OpenStack Release
Liberty
Bug Severity
3
Severity: 3
Description
This test failed.  In comparison to previous trtl results, this appears to be a new failure as compared to 6/15's results.
Deployment
Standard undercloud deployment in test infra comparable to nightly.

@dflanigan @mattgreene This issue can be closed, This test is a scenario test and we can't run scenario tests against overcloud deployment. We just need to add them to the exclude file for BIGIP13.

@ssorenso confirm that this test is excluded.


Sort token list by balance and symbol

Thank you for submitting this pull request :)
Resolves #1424
Short description
Definition of Done

[x] Steps to manually test the change have been documented
[ ] Acceptance criteria are met
[x] Change has been manually tested by the reviewer (dApp)

Steps to manually test the change (dApp)

Serve dApp
Connect
Click on token on transfer screen
Click connect new token
Check sorting in the token list

Force merging this as coverage was at 100%


(PDB-885) Correct hostname for beaker fallback test instances

This corrects a typo in the ec2 instance configuration designed to test the
terminus fallback scenario.
Signed-off-by: Ken Barber<EMAIL_ADDRESS>

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/397/


Chart.yaml version for image tag

Changing template for controller/cronjob to use chart.yaml version value to create image url
Addresses issue where image tag doesn't exist. Currently release process updates the version in chart.yaml. This PR changes the chart to use that version to build the image url.
This should be a short-term fix until a different solution is developed to handle pulling by tag and promotion.

/hold
Up for discussion if we want to go this path

/lgtm
/approve

/unhold


Settings - User roles - missing translations for the permission icons

Q
A

Bug?
no

New Feature?
no

Sulu Version
1.3.0

Actual Behavior
When I hover over the permission icons I see the corresponding translation key.  The actual translation is missing.
Expected Behavior
There should be a translation for the translation keys.
Steps to Reproduce

Settings
User roles
Details
Hover over permission icons

Already fixed in https://github.com/sulu/sulu/pull/2774, should be fine with 1.4.0.


Enhance fanout

@iaroslav-ciupin. no. I wrote my own tests to make sure it work in different scenarios (resharding, service restart etc).
But that makes sense 👍 , I'll deploy and test it on proto.

tested on proto0, eec-admin with Enhanced Fan-Out  https://jenkins-build.ellationeng.cx-mgmt.com/job/eec-automated-test-proto0/147/
Look good.


Provide value setting API based on Array index

Description:
Provide value setting API based on Array index.
doc.update((root) => {
  root.list = [0,1,2];
  root[0] = 1; // value setting
})

Unlike JavaScript behavior, indexes that exceed the length of the array are treated as errors.
Why:

Users can change the value of an element in Array.

Close this issue for now as there are currently no cases requested.

I think it would be good to provide array.set since it's a basic feature.
It could be used in array-based to-do lists or in other examples.


Enhancement: Add pyproject.toml

CONTRIBUTING.md suggests poetry for development.
How then does it come that no pyprojct.toml is included?
Shouldn't we add one? Should we maybe replace setup.py entirely?

Been there, done that.
Some things about poetry I didn't like last time I tried it:

Even by using the install method with curl, poetry dependencies clashes with my own dependencies, at least in pyenv. It's probably a bug... I think poetry dependencies should be embedded into the library to avoid the problem completely... but its repo has tons of pending pull requests right now so I'm not very motivated to code it and send one more.

Compared with pip install . is really slow (I know, pip doesn't generate a lock file automatically, but still it feels that takes forever).

(And before you ask, I despise pipenv and everything around it, so is also not an option for me)
So, thank you, but I want to wait a year for the packaging environment to stabilize a bit more.
I'm experimenting with using setup.cfg instead, like this guy: https://github.com/jaraco/skeleton

Thanks for your insight. Indeed I have not been able to make a dev setup working with poetry either.
It was the first time I actually used poetry -> thought it would live up to people considering it superior to pipenv. At least not for me.
pipenv works for my setup but then I am not developing packages on a regular basis.
(I heard its dependency resolution is really inept.)


Can you have a look?


Error: Command failed:

Hello,
I am getting below error and I am not understanding what I am doing wrong. I will really appreciate if someone can help me out with this.
Error: Command failed:
    at ChildProcess.onExit (E:\APPS\temp\FCBatchManagerTs\node_modules\gm\lib\command.js:301:17)
    at ChildProcess.emit (node:events:394:28)
    at ChildProcess.cp.emit (E:\APPS\temp\FCBatchManagerTs\node_modules\gm\node_modules\cross-spawn\lib\enoent.js:40:29)
    at maybeClose (node:internal/child_process:1064:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:301:5)

Same issue here


Implemented workaround for browsers where overrideMimeType is not sup…

The library uses overrideMimeType to specify how the content should be treated in the response of the XHR request. This property is not supported on older versions of IE, so I have implemented an alternate method for accomplishing the same thing.

Nice, thanks!


should ignore some dirs' codecov

Ignore them can improve our codecov from  62% to 68.5%.
Signed-off-by: chen hui<EMAIL_ADDRESS>

Ignore them can improve our codecov from 62% to 68.5%.

@huchen2021 , We can ignore these dirs to be explicit but I do not think this will increase code coverage as currently only the packages with test files are considered i.e. agent, controllers. You could also see the percentage increase in code coverage with your PR here.

Ignore them can improve our codecov from 62% to 68.5%.

@huchen2021 , We can ignore these dirs to be explicit but I do not think this will increase code coverage as currently only the packages with test files are considered i.e. agent, controllers. You could also see the percentage increase in code coverage with your PR here.

This PR is just for studying purpose, and will not be merged.

This PR is just for studying purpose, and will not be merged.

@huchen2021, Is it ok to close this PR.

This PR is just for studying purpose, and will not be merged.

@huchen2021, Is it ok to close this PR.

ok


Fix code scanning alert no. 1: Unsafe HTML constructed from library input

Fixes https://github.com/shimakaze09/Blog/security/code-scanning/1
To fix the problem, we need to ensure that any user-supplied data in the options object is sanitized before being used to construct HTML. This can be achieved by escaping any potentially unsafe characters in the options properties before constructing the style string.
The best way to fix this without changing existing functionality is to create a helper function that escapes unsafe characters and use this function in the buildStyle method. This will ensure that any data used in constructing the style string is safe.
Suggested fixes powered by Copilot Autofix. Review carefully before merging.

@coderabbitai full review


Integrated terminal typing issue

VSCode Version: 1.47.0
OS Version: Microsoft Windows [Version 10.0.17763.1282]

Steps to Reproduce:

Open new integrated terminal window. New window will appeared scrolled and trying to type will just scroll window but text does not appear.
Maximize terminal window and all is fine.
Minimize again and we back to the same issue.

See animated gif

Does this issue occur when all extensions are disabled?: Yes

This looks like an issue with our old terminal backend "winpty", going forward we are focusing our efforts on the new backend "conpty" which is build by the Console team at Microsoft and should fix many issues with the terminal under Windows. You can read more about getting ConPTY here.
/duplicate https://github.com/Microsoft/vscode/issues/45693


Check proto cache in CI

WHAT
Empty commit to check proto cache in CI.
WHY
After merging https://github.com/google/horologist/pull/810 which changed the proto definition, builds are failing in CI with:
> Task :media-sample:kaptDebugKotlin
e: /home/runner/work/horologist/horologist/sample/src/main/java/com/google/android/horologist/datalayer/DataLayerNodesScreen.kt: (47, 15): Type mismatch: inferred type is List<Node> but Int was expected

e: /home/runner/work/horologist/horologist/sample/src/main/java/com/google/android/horologist/datalayer/DataLayerNodesScreen.kt: (51, 32): Unresolved reference: displayName
> Task :sample:compileDebugKotlin FAILED
e: /home/runner/work/horologist/horologist/sample/src/main/java/com/google/android/horologist/datalayer/DataLayerNodesScreen.kt: (51, 50): Unresolved reference: id
e: /home/runner/work/horologist/horologist/sample/src/main/java/com/google/android/horologist/datalayer/DataLayerNodesScreen.kt: (51, 64): Unresolved reference: isNearby
e: /home/runner/work/horologist/horologist/sample/src/main/java/com/google/android/horologist/datalayer/DataLayerNodesScreen.kt: (72, 15): Type mismatch: inferred type is List<Map.Entry<String, SampleProto.Data>> but Int was expected
e: /home/runner/work/horologist/horologist/sample/src/main/java/com/google/android/horologist/datalayer/DataLayerNodesScreen.kt: (72, 51): Destructuring declaration initializer of type Int must have a 'component1()' function
e: /home/runner/work/horologist/horologist/sample/src/main/java/com/google/android/horologist/datalayer/DataLayerNodesScreen.kt: (72, 55): Destructuring declaration initializer of type Int must have a 'component2()' function

Checklist :clipboard:

[N/A] Add explicit visibility modifier and explicit return types for public declarations
[N/A] Run spotless check
[N/A] Run tests
[N/A] Update metalava's signature text files

false alarm, issue was on import that was removed somehow automatically


Disabled edges and nodes physics

I disabled the terrible edges and nodes physics on Lateral Movement view by setting to false the physics property on options variable as documented here.

Good idea. Thank you!


$state.reload not working.

I have a user with role "admin" and it has access to index page. When I delete a record from index page, I call $state.reload to refresh the data which is not working.

any update on this...Its been over a month...

Have the same issue here
event.preventDefault();
          toState = angular.extend({'$$finishAuthorize': true}, toState);

          if ($rootScope.$broadcast('$stateChangePermissionStart', toState, toParams).defaultPrevented) {
            return;
          }

          Permission.authorize(permissions, toParams).then(function () {
            // If authorized, use call state.go without triggering the event.
            // Then trigger $stateChangeSuccess manually to resume the rest of the process
            // Note: This is a pseudo-hacky fix which should be fixed in future ui-router versions
            if (!$rootScope.$broadcast('$stateChangeStart', toState, toParams, fromState, fromParams).defaultPrevented) {
              $rootScope.$broadcast('$stateChangePermissionAccepted', toState, toParams);

              $state.go(toState.name, toParams, {notify: false}).then(function() {
                $rootScope
                  .$broadcast('$stateChangeSuccess', toState, toParams, fromState, fromParams);
              });
            }
          }

particularly $state.go(toState.name, toParams, {notify: false})
permission prevents original $stateChangeStart event and then manually invokes $state.go but the problem here is that {reload: true} flag, which should be presented in order to force state to reload is missing. I'm afraid I have to fork the library at my job to fix this behaviour. My personal solution is to replace the line mentioned above with this:
$state.go(toState.name, toParams, {notify: false, reload: true})

Hmm, I think this can be fixed. Newer versions of ui-router now pass options to the $stateChangeStart event. I think we could add a reload: true option or something there.
I'm sorry about the lack of responsiveness lately, as you can see I'm alone maintaining this repo right now and currently getting a crazy ton of work in my full-time job.
If anyone PRs I'm willing to review the PR that fixes this issue in a timely manner.
Thanks!

+1

Waiting for: https://github.com/angular-ui/ui-router/milestones/0.2.16
El mar., 27 oct. 2015 a las 20:20, Cameron Alexander (<
<EMAIL_ADDRESS>escribió:

+1
—
Reply to this email directly or view it on GitHub
https://github.com/Narzerus/angular-permission/issues/78#issuecomment-151673159
.

Could you give some more details on how this is a "commonly known problem with angular-ui"? As @beshanoe pointed out the issue is that angular-permission doesn't pass the reload option from the original $state.go call.
$state.go(toState.name, toParams, {notify: false, reload: true})

... okay, should have looked at the current situation first. It seems that ui-router doesn't pass that option with the event
$rootScope.$broadcast('$stateChangeStart', to.self, toParams, from.self, fromParams


Named type arguments

When a class has multiple type arguments, it's not always obvious what they each are for. It would be nice if we could name them, the same way we name regular arguments. If we could simultaneously make the named ones optional and give them default values (separate from their lower bound), it would make it significantly easier to then add type arguments without breaking clients of the original API.

This is interesting, but I wonder if the motivation is more important.
With type inference, you should rarely (?) need to type the argument. For example:
class Tuple<A, B> {
  final A a;
  final B b;
  Tuple(this.a, this.b);
}

void main() {
  // Automatically infers Tuple<String, int>
  var tuple = new Tuple('Hello', 5);
}

What I would like to see, instead, is a default argument. For example:
class Tuple<A, B, C = Null> { ... }

void main() {
  // Automatically infers Tuple<String, int, Null>.
  var tuple = new Tuple<String, int>();
}

Adding a wildcard ? could also help (similar to _):
class Tuple<A, B = Null, C = Null> { ... }

void main() {
  // Automatically infers Tuple<String, Null, int>.
  var tuple = new Tuple<String, ?, int>();
}

... I realize that doesn't solve the problem of if you want to invoke it manually (without inference), or specifying what the arguments are for without looking at the API - but I'd be curious what languages with generics don't use positionals.

I was mostly thinking of generic methods, where the type arguments are often actual arguments (e.g. "search for objects of this type", or "create objects of type Foo with this type as its type argument").

Ah, that's a good point. To be clear, something like:
abstract class Context {
  T findWidgetOfType<T>();
}

And you'd like to, for example, specify:
abstract class Context {
  /// NEW: Specify 'P' to filter by widgets that have 'P' as a parent.
  ///   findWidgetOfType<Foo, Bar>();
  ///   ... will find widgets of type 'Foo' where 'Bar' is the direct parent.
  T findWidgetOfType<T, P>();
}

... but that would be a breaking change, forcing you to write:
T findWidgetOfTypeWithParent<T, P>();

... correct?

Yeah, that's a great example. Currently, we either have to make a new API or force any users of this API to add a , dynamic to the end of any such call. Ideally we'd be able to say findWidgetOfType<Foo, parent: Bar> and have findWidgetOfType<Foo> imply parent: dynamic.

Another example which uses class type arguments would be a class that represents a function call, whether one type is the type of the arguments and one type is the type of the return value. One might imagine wanting to default both to void and only requiring that they be specified for instances that actually have an argument or have a return value, respectively. (Specifically, I'm thinking of ModalRoutes in Flutter, where having arguments and having return values are not common, and having both is even less common, but where when you do have them, it's nice to be able to get some extra type checking.)

(Type inference doesn't typically help in those cases, especially for the return value, since the value is provided much later. For the argument it might sometimes help. In any case, right now you always have to specify both if you specify either, which is awkward.)

Sounds like asking for type parameters to have the full complexity of function parameters (optional, named, default values).
While clearly possible, I'd loathe to have T Function<T, [S extends num]>(T, [S]) be assignable to T Function<T>(T).
Maybe it'd be easier to allow normal parameters to carry types, and not have a separate type parameter syntax (probably not, though, since that would likely mean using Type objects as types, unless we define Type parameters to be special, like T foo(T x, {type T}) => x;. That's probably too complex too.

I guess the point of this issue could be to remember that this particular language design space has a simple end (the current design) and a complex end (@lrhn's full generalization of type parameters); and maybe @Hixie's proposal (allowing some type parameters to have a name and a default value) would be a useful sweet spot between expressive power and counter-productive complexity.
One thing comes to mind, though; with class A<X, y: Y extends num = int, Z, w: W = void> {...}, what does A<double, double, double> mean, A<double, int, double, double> or A<double, double, double, void>? ;-)

If we have named type parameters, they should probably require a name when applied.
We could omit optional positional type parameters, so only named type parameters can be optional.
It seems like a lack of generality, though.

Being able to use Type values as types would be amazing.
I agree that if you name the type parameter, the name should be required on use, just as with regular parameters.


Fix typo in demo name

Not sure if this will fix the issue, but it looks like drawing was misspelled.

Oops!  Probably means drawimg as in draw image.  Taking it back.

Hi @AbbyRead
Yes, the correct name is drawimg. It correspond with DrawImage demo app.


Fix flaky tests

test_get_split_content_splits_longer_content is a bit flaky and sometimes splits == 1 resulting in
    def test_get_split_content_splits_longer_content():
        content_length = 340
        split_length = 200
        body = fake.text(max_nb_chars=content_length)[:content_length]
        instance = ExamplePageFactory.create(title="a", body=body)
        splits = instance._get_split_content(split_length=split_length, split_overlap=0)
>       assert len(splits) == 2
E       AssertionError: assert 1 == 2
E        +  where 1 = len(['a\nPainting right various sort. Western medical cut third of song hotel social. Early young with from form dog hand.\nStyle though rest home shoulder agency. Own use look blue film loss enough.'])

most prononunce in CI, but had it locally a few times

I believe this was fixed by #24
/cc @tm-kn

Haven't seen evidence of tests failing randomly in a while. Closing.


Configuration Migration fails with warn log: [CONFIG MIGRATION] While migration the configuration, no protocol factory for protocolId 'file_polling_protocol' was found. This adapter will be skipped and must be migrated by hand.

Expected behavior
Automatic Configuration Migration works on Edge Startup
Actual behavior
The automatic confguration migration fails with logs like:
[CONFIG MIGRATION] While migration the configuration, no protocol factory for protocolId 'file_polling_protocol' was found. This adapter will be skipped and must be migrated by hand.
To Reproduce
Start HiveMQ Edge with an older config.xml that includes the old adapter protocol ids.
Details
This will be fixed with the next bugfix release. The problem results from a change in the protocol ids which leads HiveMQ Edge to not find the adapter factories for old protocol ids and ultimately to a failed config migration.

Until tthe fix is released, this can be fixed by editing the config.xml by hand:
In the protocol adapter section, the tags that indicate the protocol ids must be updated:
file_polling_protocol becomes file
file_input becomes file
opc-ua-client becomes opcua

The issue should be resolved with the new version 2024.9


feat(bitbucket): support self-hosted instances

Atlassian has different APIs for the Cloud and Server (self-hosted)
version of Bitbucket. Therefore, a dedicated remote is needed to handle
the self-hosted version.
Description

Motivation and Context
closes #762

How Has This Been Tested?

Screenshots / Logs (if applicable)
Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (no code change)
[ ] Refactor (refactoring production code)
[ ] Other

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly.
[x] I have formatted the code with rustfmt.
[x] I checked the lints with clippy.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 24.62687% with 101 lines in your changes missing coverage. Please review.

Project coverage is 35.41%. Comparing base (227a307) to head (0e4fb24).

Files
Patch %
Lines

git-cliff-core/src/remote/bitbucket_server.rs
0.00%
44 Missing :warning:

git-cliff-core/src/changelog.rs
28.58%
25 Missing :warning:

git-cliff/src/logger.rs
0.00%
9 Missing :warning:

git-cliff-core/src/remote/bitbucket.rs
0.00%
8 Missing :warning:

git-cliff-core/src/remote/gitlab.rs
55.56%
4 Missing :warning:

git-cliff-core/src/remote/gitea.rs
62.50%
3 Missing :warning:

git-cliff-core/src/remote/github.rs
62.50%
3 Missing :warning:

git-cliff-core/src/config.rs
50.00%
2 Missing :warning:

git-cliff-core/src/remote/mod.rs
71.43%
2 Missing :warning:

git-cliff-core/src/commit.rs
50.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #763      +/-   ##
==========================================
- Coverage   36.71%   35.41%   -1.30%
==========================================
  Files          19       20       +1
  Lines        1501     1596      +95
==========================================
+ Hits          551      565      +14
- Misses        950     1031      +81

Flag
Coverage Δ

unit-tests
35.41% <24.63%> (-1.30%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@orhun absolutely I'll continue on it. Just haven't have much time lately to continue on where I started.
I really like the idea of switching the implementation based on whether a custom URI was used. Felt really odd to have these two configs despite they pretty much achieving the same.
A few things are obviously still missing here and there. Especially testing is an issue I guess.
To have a self-hosted Bitbucket server running is easy as you can just grab the latest Docker image and run it, but then you need to get a 30-day license and so on. Way too many steps to verify the implementation.
As far as I understand the API isn't really evolving much (maybe the server variant gets the update to API v2 at some point eventually). So my idea is to record a few queries and save them in the repo, anonymized of course. Then we can see if we use a full blown HTTP mocking crate or simply deserialize the files into expected responses.
That way we can at least verify the implementation to some degree without having to run a full server every time. Might be a good solution for the other backends as well.

absolutely I'll continue on it. Just haven't have much time lately to continue on where I started.

Thanks for showing interest! Just an idea, maybe it makes sense to start this PR over since we won't have two separate functionalities for this :)

To have a self-hosted Bitbucket server running is easy as you can just grab the latest Docker image and run it, but then you need to get a 30-day license and so on. Way too many steps to verify the implementation.

Wow, that's really bad ;/

So my idea is to record a few queries and save them in the repo, anonymized of course. Then we can see if we use a full blown HTTP mocking crate or simply deserialize the files into expected responses. That way we can at least verify the implementation to some degree without having to run a full server every time. Might be a good solution for the other backends as well.

Loved this idea! This will also save us from spurious CI errors due to networking issues that happen occasionally with GitLab etc.
Would love to have this in!

No need to start it over, we can use the power of rebase and force pushes :wink:. But it's basically somewhat of a start over, haha.
First step done, now the Bitbucket implementation picks the Cloud or Server variant based on whether a custom API URL is set.
Will get to add in some tests and finish up some missing docs (clippy will probably fail) next.

Perfect!

Just lmk when this is ready for review :)

Definitely! The basics are ready, just didn't get a chance to collect some API calls for some automated tests yet.
Once I feel it's ready I'll move it from a draft into a regular PR.

Just a small heads up, I didn't forget about the PR. Priorities are quite somewhere else at the moment, so I simply didn't get to pull over and sanitize those request/response samples.
But I'm sure I'll get to it sometime.

No worries, sounds good :)

Hey, lost interest? :) (it's totally fine)

Hey, sorry for that. I should have probably written a comment first and then closed the PR (actually auto-closed due to deleting the branch).
This was mostly for work and the priority on it was pretty low. Plus the latest changes made a rebase pretty hard, and I forgot half of my PR changes as well because I haven't touched it in a long time.
So I figured it's best to close this for now and then rather make a fresh PR from scratch with the missing bits included, that we discussed here so far.


HDFS-16641. [SBN read] Add a metric to quantify the number of re-enter rpcs

Description of PR
In our prod environment, Observer NameNode also have high QueueTime, but with low processingTime.
After tracing and found the root cause maybe is that calls from clients frequently re-enter the Queue because the client call's state id is bigger than the server state id.
So I think maybe we should add a metric to quantify the number of re-enter rpcs to help us locate this anomaly easily.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 57s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m 34s

Maven dependency ordering for branch

-1 :x:
mvninstall
16m 11s
/branch-mvninstall-root.txt
root in trunk failed.

+1 :green_heart:
compile
29m 24s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
compile
22m 46s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
4m 33s

trunk passed

+1 :green_heart:
mvnsite
4m  3s

trunk passed

+1 :green_heart:
javadoc
3m  4s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
3m  5s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
6m 54s

trunk passed

+1 :green_heart:
shadedclient
26m 52s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 25s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 29s

the patch passed

+1 :green_heart:
compile
24m  9s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javac
24m  9s

the patch passed

+1 :green_heart:
compile
21m 33s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
21m 33s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
4m 26s
/results-checkstyle-root.txt
root: The patch generated 2 new + 195 unchanged - 0 fixed = 197 total (was 195)

+1 :green_heart:
mvnsite
3m 57s

the patch passed

+1 :green_heart:
javadoc
2m 55s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
3m 11s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
7m  8s

the patch passed

+1 :green_heart:
shadedclient
27m 15s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
18m 22s

hadoop-common in the patch passed.

-1 :x:
unit
365m 40s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
1m 35s

The patch does not generate ASF License warnings.

615m 14s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.namenode.ha.TestObserverNode

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4496/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4496

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 48fb58339e14 4.15.0-166-generic #174-Ubuntu SMP Wed Dec 8 19:07:44 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 4b9573b2a0c70cbe4609812e26ff4387a04ca2ee

Default Java
Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS> /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4496/1/testReport/

Max. process+thread count
2355 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4496/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 54s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
15m 12s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
28m 35s

trunk passed

+1 :green_heart:
compile
25m  9s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
compile
21m 43s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
4m 36s

trunk passed

+1 :green_heart:
mvnsite
3m 57s

trunk passed

+1 :green_heart:
javadoc
3m  5s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
3m 10s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
6m 58s

trunk passed

+1 :green_heart:
shadedclient
26m 50s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 27s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 30s

the patch passed

+1 :green_heart:
compile
24m 32s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javac
24m 32s

the patch passed

+1 :green_heart:
compile
21m 47s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
21m 47s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
4m 30s
/results-checkstyle-root.txt
root: The patch generated 5 new + 306 unchanged - 0 fixed = 311 total (was 306)

+1 :green_heart:
mvnsite
3m 59s

the patch passed

+1 :green_heart:
javadoc
2m 57s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
3m  9s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
7m  4s

the patch passed

+1 :green_heart:
shadedclient
27m 21s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
18m 26s

hadoop-common in the patch passed.

+1 :green_heart:
unit
363m 36s

hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
1m 36s

The patch does not generate ASF License warnings.

621m 54s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4496/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4496

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux a298d1709cd4 4.15.0-166-generic #174-Ubuntu SMP Wed Dec 8 19:07:44 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / acf0e38b8b6ab2b6741bd3b71512b452e62385eb

Default Java
Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS> /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4496/2/testReport/

Max. process+thread count
2125 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4496/2/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m  1s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
15m  1s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
29m 38s

trunk passed

+1 :green_heart:
compile
25m 22s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
compile
21m 47s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
4m 37s

trunk passed

+1 :green_heart:
mvnsite
4m  2s

trunk passed

+1 :green_heart:
javadoc
3m  3s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
3m  8s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
6m 56s

trunk passed

+1 :green_heart:
shadedclient
27m  2s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 27s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 31s

the patch passed

+1 :green_heart:
compile
24m 28s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javac
24m 28s

the patch passed

+1 :green_heart:
compile
21m 51s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
21m 51s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
4m 30s
/results-checkstyle-root.txt
root: The patch generated 2 new + 306 unchanged - 0 fixed = 308 total (was 306)

+1 :green_heart:
mvnsite
3m 56s

the patch passed

+1 :green_heart:
javadoc
2m 55s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
3m  7s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
7m  4s

the patch passed

+1 :green_heart:
shadedclient
27m 20s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
18m 29s

hadoop-common in the patch passed.

+1 :green_heart:
unit
388m 20s

hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
1m 41s

The patch does not generate ASF License warnings.

647m 49s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4496/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4496

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux e3b40cda7e4f 4.15.0-166-generic #174-Ubuntu SMP Wed Dec 8 19:07:44 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 963a1bb7296c2497cd71e1201549f0dfd0d02f16

Default Java
Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS> /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4496/3/testReport/

Max. process+thread count
2196 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4496/3/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

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

This message was automatically generated.


1 rest persist in memory

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

ok

ok


Question/Error : No job functions found. Try making your job classes and methods public.

Hello,
I tried to follow your tutorial unfortunately, when I am running the azure function locally, I have this error in verbose mode:

[2022-02-20T18:36:52.519Z] Reading functions metadata
[2022-02-20T18:36:52.519Z] 0 functions found
[2022-02-20T18:36:52.538Z] 0 functions loaded
[2022-02-20T18:36:52.554Z] Generating 0 job function(s)
[2022-02-20T18:36:52.568Z] No job functions found. Try making your job classes and methods public. If you're using binding extensions (e.g. Azure Storage, ServiceBus, Timers, etc.) make sure you've called the registration method for the extension(s) in your startup code (e.g. builder.AddAzureStorage(), builder.AddServiceBus(), builder.AddTimers(), etc.).

I don't understand why the app isn't loaded correctly while my configuration looks like this :
function.json

{
"scriptFile": "app.py",
"bindings": [
{
"authLevel": "anonymous",
"type": "httpTrigger",
"direction": "in",
"name": "req",
"methods": [
"get",
"post",
"put",
"patch",
"delete"
],
"route": "/{*route}"
},
{
"type": "http",
"direction": "out",
"name": "$return"
}
]

}
app.py

import azure.functions as func
from http_asgi import AsgiMiddleware
from app import app
def main(req: func.HttpRequest, context: func.Context) -> func.HttpResponse:
return AsgiMiddleware(app).handle(req, context)

app/init.py

from fastapi import FastAPI, Depends
from app.auth.jwt_bearer import JWTBearer
from app.routes.device import router as device_router
from app.routes.auth import router as admin_router
from app.routes.user import router as user_router
from app.routes.transceiver import router as transceiver_router
app = FastAPI()
token_listener = JWTBearer()
app.include_router(admin_router, tags=["Auth"], prefix="/auth")
app.include_router(user_router, tags=["Users"], prefix="/users", dependencies=[Depends(token_listener)])
app.include_router(device_router, tags=["Devices"], prefix="/devices", dependencies=[Depends(token_listener)])
app.include_router(user_router, tags=["Transceivers"], prefix="/transceivers", dependencies=[Depends(token_listener)])

Could you please have a look and help me on this one?
Thank you,
Romain

After cloning your project, adding my files to it, the error was not there anymore. I advise it to any who would meet the same issue


BUG: fix syevr series segfault by explicitly specifying operator priority

With numpy 1.22.x f2py, dimension of isuppz in syevr is mistranslated.
Avoid this by explicitly specifying operator priority with parenthesis.
Fixes #16527

Reference issue

What does this implement/fix?

Additional information

Hi @mtasaka. Thank you for a very detailed analysis. The problem started with f2py changing its behavior handling ternary operators and we had some early warnings of this (see https://github.com/scipy/scipy/issues/15386#issuecomment-1008246141)
Anyways, just for another data point, I am not able to reproduce with main branch of numpy and latest OpenBLAS on Windows. But since the change is not affecting the code I think this can go in as soon as the CI complete.

Failure is not relevant. Thanks @mtasaka

@tylerjereddy maybe this one for 1.9.1 too?


Is Objaverse 1.0 a subset of Objaverse-XL?

If so, how could one filter those objects that are in Objaverse-XL but not in 1.0?

Yes, see here: https://colab.research.google.com/drive/1NpR2MLs6ydXm4XWT_ovpM16CtzvGJVC7?usp=sharing
Basically:
from objaverse.xl import get_annotations
df = get_annotations()
new_df = df[df["source"] != "sketchfab"]


[BUG] Update Profile Name

Describe the bug
When trying to change the profile name, it returns the following error:
{
    "statusCode": 404,
    "error": "Not Found",
    "message": "myAppStateKey (\"AAAAALVm\") not present"
}

To Reproduce
Steps to reproduce the behavior:

Created a new connection
Use instance.socket?.updateProfileName("New Name")

I have not been able to recreate in two whatsapp accounts each one different business and normal
Whatsapp BUSSINES

Whatsapp Not Bussines

It's working normally here too.

Please, reopen if this still happening

Describe the bug When trying to change the profile name, it returns the following error:
{
    "statusCode": 404,
    "error": "Not Found",
    "message": "myAppStateKey (\"AAAAALVm\") not present"
}

To Reproduce Steps to reproduce the behavior:

Created a new connection
Use instance.socket?.updateProfileName("New Name")

Same issue for me:
{
"status": 500,
"error": "Internal Server Error",
"response": {
"message": [
"Error updating profile name",
"Error: myAppStateKey ("AAAAAKNl") not present"
]
}
}

Acontece comigo tambem
{
"status": 500,
"error": "Internal Server Error",
"response": {
"message": [
"Error updating profile name",
"Error: myAppStateKey ("AAAAADoC") not present"
]
}
}

I had problems here too, apparently it's only on WhatsApp Business, on Messenger it updates normally.
See the return below:
{
    "status": 500,
    "error": "Internal Server Error",
    "response": {
        "message": [
            "Error updating profile name",
            "Error: myAppStateKey (\"AAAAAGdq\") not present"
        ]
    }
}


Add support for file include.

It seems that one of the likeable features of TypeScript, compared to Babel lets say, is that it emits readable code.
Consider the following code:
X.ts:
namespace a.b.x {
	class X {

	}
}

Y.ts:
namespace a.b.y {
	class y {

	}
}

This emits:
var a;
(function (a) {
    var b;
    (function (b) {
        var x;
        (function (x) {
            var X = (function () {
                function X() {
                }
                return X;
            })();
        })(x = b.x || (b.x = {}));
    })(b = a.b || (a.b = {}));
})(a || (a = {}));

var a;
(function (a) {
    var b;
    (function (b) {
        var y;
        (function (y_1) {
            var y = (function () {
                function y() {
                }
                return y;
            })();
        })(y = b.y || (b.y = {}));
    })(b = a.b || (a.b = {}));
})(a || (a = {}));

There's a lot of code redundancy, which can really clutter the emitted output for projects that rely heavily on namespaces.
Wouldn't it be nice if we could just do something like this:
namespace a.b {
    export namespace x {
        /// include X.ts
    }
    export namespace y {
        /// include Y.ts
    }
}

The include statement simply places the included file into the script during pre-compilation.
My guess is that this is already partly implemented with /// reference, only that ///reference doesn't seem to work unless it's either the first line, or one succeeding another /// reference.

there are scoping issues at play here that would make tooling for this fairly hard. it also makes external and internal module syntax ambiguous.
The original issue is tracked by #447.

@mhegazy,  I don't really see how this issue is a duplicate of #447. #447 is about compiler optimisation and I agree with everything you say in your comment here about it.
But we are talking here of a simple include, so any scoping issues are the developer problems and not that of the compiler.
I mean, you get the same feature in C++, and one could easily do this with libraries like [grunt-include-source] (https://github.com/jwvdiermen/grunt-include-source).
Something like this could also help with the redeclaration overhead of the current mixins guide.

C++ template, processor directives, includes are all nice features, but they make tooling very hard and adds significant complexity to the language. I do not think we would be open to supporting such features.
As for this proposal here are some of the issues we would run into:

to identify whether a namespace is exposed on its parent, it needs to be exported. so in the example above, you need to have x.ts look like export namespace x {..}. this is ambiguous from the get go, as now we do not know if this is an external module or just a nested namespace. if not then this changes the errors you get in a file based on how another file includes it, which is inconsistent at best.
single file compilation is completely busted, as in isolation, we have no way how to emit this file
opening a file in your IDE does not tell the IDE what is the lexical scope. a lot of the implementation of lexical scope features in LS today (e.g. navigationbar items) depend on knowing the lexical context of a module.
using comments to change scoping is just not right. We already have /// references and do not think we need to add more.

I think the issue is the shape of the emit. if you agree, this should be an emitter change, that "optimizes" the output by compiling namespace declarations if you use --out. which i believe is covered by #447.


test: Check for custom spans in Istio tracing test

Description

Changes proposed in this pull request:

Add e2e test for istio-proxy trace, test ensure istio-proxy traces are collected with telemetry trace collector

Related Issues and Documents

Closes:
Related issues:
https://github.com/kyma-project/telemetry-manager/issues/317
Traceability

[x] The PR is linked to a GitHub Issue.
[ ] New features have a milestone label set.
[ ] New features have defined acceptance criteria in a corresponding GitHub Issue, and all criteria are satisfied with this PR.
[ ] The corresponding GitHub Issue has a respective area label.
[ ] The follow-up issues (if any) are linked in the Related Issues section.

Testability
The feature is unit-tested:

[ ] Yes.
[ ] No, because unit tests are not needed.
[ ] No, because of ...

The feature is e2e-tested:

[ ] Yes.
[ ] No, because e2e-tests are not needed.
[ ] No, because of ...

Tests conducted for the PR:

Codebase

[ ] My code follows the Effective Go style guidelines.
[ ] The code was planned and designed following the defined architecture and the separation of concerns.
[ ] The code has sufficient comments, particularly for all hard-to-understand areas.
[ ] This PR adds value and shows no feature creep.
[ ] I have augmented the test suite that proves my fix is effective or that my feature works.
[ ] Adjusted the documentation if the change is user-facing.

To do:

Validate connection between Istio spans and custom spans
Validate other span attributes (e.g., k8s resource attribtues)
Replace sample workload image to use Kyma registry

To do:

Validate connection between Istio spans and custom spans
Validate other span attributes (e.g., k8s resource attribtues)

/retest

/retest

/retest


Stream parser article html and text to Snowflake

Goal
What changed? What is the business/product goal?

Change 1
Change 2

Implementation Decisions
Deployment steps

[ ] Database migrations?
[ ] Secrets?
[ ] FOR POCKET DEVELOPERS ONLY - Post in #changelog:

Write a #changelog message using our best practices if this PR (potentially) impacts users. Describe the 'why' and 'what'. @mention relevant teams. Link to this PR.

References
JIRA ticket:

Link to JIRA ticket

Documentation:

Project doc

:rocket: I like the way this is going.


Handling Windows PreShutdown events

I have need to respond to PreShutdown events.  These are events that were added to Windows services starting with Vista.
https://docs.microsoft.com/en-us/windows/win32/services/service-changes-for-windows-vista
https://scottdorman.blog/2006/06/18/windows-vista-kernel-changes-can-i-get-some-service/
https://www.atomia.com/2011/11/16/how-to-process-the-preshutdown-event-in-a-managed-windows-service/
https://www.sivachandran.in/2012/03/handling-pre-shutdown-notification-in-c.html
See also TopShelf Issue:  Support SERVICE_CONTROL_PRESHUTDOWN #371
I've been using this functionality for 8+ years in my own library ServiceBaseEx that I built based on a decompile of ServiceBase, but TopShelf makes so many things easier I'd prefer to use it.  Alas, not supported here yet.  I'm at a loss as to why Microsoft hasn't added support to ServiceBase, but clearly they have no intention.
This week, being on furlough for Covid, I spent some time seeing what it would take.  It turns out to not be too bad to hack in, and I have done it, but I'm not sure you guys would accept it.  The solution I came up with was to modify WindowsServiceHost to add a CanPreShutdown property.  If set, the constructor calls a function that sets the properties and callbacks required.  This uses reflection against ServiceBase and is therefore somewhat dependent on the MS ServiceBase implementation details.  Maybe a bit brittle.
Alternatively, could we consider using and modifying the ServiceBase from MS on GitHub which is apparently MIT licensed.  The changes are very minor and would leave TopShelf fully in control of the implementation.
https://github.com/microsoft/referencesource/blob/master/System.ServiceProcess/ServiceBase.cs

The files in this repository are licensed under the MIT license unless otherwise specified in the file header. If the file header only contains a copyright header (e.g., "Copyright (c) Microsoft Corporation. All rights reserved.", you can assume the associated file to be MIT-licensed.

I'm happy to help implement this and share the code if the owners here are interested.

I was hoping there might be some interest, or at least feedback.  The preshutdown events are part of the standard Windows Service API, so I'm not proposing anything that doesn't further coverage of core Windows service capabilities.  I guess the oddity here is that the .Net API doesn't cover this functionality.
I've already built a version of TopShelf for my own use that implements this support, I was thinking that contributing it back would be good manners.  I only thought I'd take feedback on how the maintainers might prefer it be implemented before creating a PR.
Regardless, I want to say that this is a great tool and I appreciate your sharing it with the community.

I am closing all issues that haven't seen any activity. You're welcome to submit a PR to resolve the issue or provide more information.


Warn users when perf is not working as expected

I don't recall if there's a way to do perf test for just the calls required for what's used in Cirron, but maybe there's a way to test that the flags necessary are enabled (i.e. either the user is running as root, or they have done something like sudo sysctl kernel.perf_event_paranoid=3 on Ubuntu, and changed other flags).
An alternative could be offering a Dockerfile so users can docker build .... and docker run #args-to-run-privileged? etc.
Cheers

I don't recall if there's a way to do perf test for just the calls required for what's used in Cirron, but maybe there's a way to test that the flags necessary are enabled (i.e. either the user is running as root, or they have done something like sudo sysctl kernel.perf_event_paranoid=3 on Ubuntu, and changed other flags).

I think it's OK to just let perf_event_open fail in that case? We should probably mention the possible fixes in the error message though, as we do when the lib compilation fails.

An alternative could be offering a Dockerfile so users can docker build .... and docker run #args-to-run-privileged? etc.

Great idea! Probably not something I'll add myself in the foreseeable future though.


Dashboard layout

update dashboard layout, including title, tab name, plot text annotation, etc.

thank you for updating the layout and the details!


Webui tests

Type: feature / unit test
The following has been addressed in the PR:

There is a related issue
Unit or Functional tests are included in the PR

Description:
Resolves #415

@ayusharma we have a conflict here jest.config.js

resolving conflict.

transform-class-properties plugin is not compatible with our Functional testing. If you remove that test will work again. We have a fancy https://github.com/verdaccio/verdaccio/blob/feature-jest-test/test/functional/lib/request.js class that extends from Promise and transform-class-properties just does not allow such behavior.  I had to deal with that weeks ago https://github.com/juanpicado/babel-promise-test

Issues:
The first one is on .babelrc configuration file.

"es2015" preset is not needed it. "es2015-node4" does already include all you need.
"transform-class-properties" does break the functional test. You don't need it.

     "test": {
-      "presets": [ "es2015-node4", "es2015", "flow", "react"],
+      "presets": [ "es2015-node4", "flow", "react"],
       "plugins": [
         "transform-class-properties",
         "transform-object-rest-spread",
-        "transform-class-properties",
         "syntax-async-functions"
       ]
     },

The second issue is on jest.config.js.
If you map storage
    'storage': '<rootDir>/test/webui/components/__mocks__/storage.js',

causes
    TypeError: Storage is not a constructor

I think you have to use something less generic than storage.  I'd rename all those modules as eg: uiStorage to avoid future conflics.
There are more issues, this is what I found by far.

If you update the circleci.yaml removing Node4, you are done here 👍


Ripme only getting the first 12 or so images on Instagram again

@cyian-1756 * Ripme version: 1.7.41

Java version: Latest
Operating system: Win7 sp1 64bit
Exact URL you were trying to rip when the problem occurred: https://www.instagram.com/taylorswift
Please include any additional information about how to reproduce the problem:
Only getting the first 12 images again

Expected Behavior
To get the whole album
Detail the expected behavior here.
Get the whole album
Actual Behavior
Only getting the first page or 12 images
Detail the actual (incorrect) behavior here. You can post log snippets or attach log files to your issue report.

1.7.43 was working fine for me last night, but now getting this behaviour for any album I try. Default settings. Album pages loading fine in browser, so IP hasn't been rate limited.


Cache existing generator data to improve speed

Most of the time for every scenario is spent on existing generators. Caching results would help speed things up dramatically when evaluating lots of different policy/technology scenarios. Something like functools.lru_cache requires all function inputs to be hashable. That's difficult here since we're dealing with dataframes, sql database connections, etc.
But the settings file is just a dictionary, which can be hashed. So I could do the hashing myself based on relevant sections of the settings file. And cached data can be permanently stored in parquet files for use in different sessions.

Or maybe use something like the joblib Memory class. https://joblib.readthedocs.io/en/latest/memory.html


Introduce mssql connection property 'host name in certificate'

The PR introduces support for specifying host name in certificate, available starting with MDS 5.0 for all encrypted connections.
Host name in certificate
Indicates the host name specified in TLS certificate that will be used by SQL Server to validate server certificate. When not specified, the server name is used by default for certificate validation.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.962%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
28916

Relevant Lines:
64014

💛  - Coveralls


How to turn off Asp.Net Core Integration on Isolated runtime on v4 with .NET 8

Description
I'm trying to use the default template created by vs code on Mac. this is the code in my Program.cs
var host = new HostBuilder()
    // .ConfigureFunctionsWebApplication(app => {
    //     app.UseMiddleware<MyMiddleware>();
    // })
    .ConfigureFunctionsWorkerDefaults((app) => {
        app.UseMiddleware<MyMiddleware>();
    })
    .ConfigureServices(services => {
        services.AddApplicationInsightsTelemetryWorkerService();
        services.ConfigureFunctionsApplicationInsights();
    })
    .Build();

With the above code, it throws the following error:
/Users/test/Projects/dotnetserver/IsolatedFunction/Program.cs(10,6): error AZFW0014: The registration for method 'ConfigureFunctionsWebApplication' is expected for ASP.NET Core Integration. (https://aka.ms/azfw-rules?ruleid=AZFW0014) [/Users/test/Projects/dotnetserver/IsolatedFunction/IsolatedFunction.csproj]

I understand this is because its trying to build an Asp.Net Core based function, but how to turn that behavior off?
Steps to reproduce
Use the default template provided by vs code for HTTP Trigger functions on .NET 8 isolated runtime. Update the program.cs code as mentioned above

If you choose not to use ASP.NET Core integration, remove the package reference to Microsoft.Azure.Functions.Worker.Extensions.Http.AspNetCore.
If this package is referenced in your dotnet-isolated function app, you should use ConfigureFunctionsWebApplication.
https://learn.microsoft.com/en-us/azure/azure-functions/dotnet-isolated-process-guide?tabs=windows#aspnet-core-integration

Thanks let me try that

Yes. it worked, closing this issue, thanks. But I think it should be a more straightforward configuration than removing a project reference. Also I couldn't find this in any docs, that also needs an update


Ruby dependency package builders available

@larskanis
I've got the packagers up at MSP-Greg/ruby-makepkg-mingw.
Builds gdbm, libyaml, and openssl.  Provided some scripts also.
Since I've been working with RI, I've only done 32 bit builds.  Of the three packages, only the openssl package has x64 appended to dll file names.  Not sure if that's normal, or if I'm missing a build setting.
Anyway, see the readme.
Of course, I'm very interested in what you (and others?) find as far as building and testing...
Reminder, that one line in PKGBUILD.
Also, wondering if you and others can give the native readline some testing.  Seems to work on my builds, but there are some test issues.  They might be caused by the testing trying to mimic console input, which may be an issue with Windows.

@larskanis Update
Finally dumped RI and now building from local (Ruby trunk) with makepkg-mingw and PKGBUILD.
test-all has no failures/errors in any gdbm/dbm, libyaml, or OpenSSL tests, both on i686 & X86_64.

@larskanis
I added a release, so I've got xz files for i686 and x86_64 packages at MSP-Greg/ruby-makepkg-mingw Release v0.1
Hence, no one needs to go to the trouble of building them.

@larskanis
Yesterday I finished code to allow `relatively easy' building of Ruby trunk from a local repo.  It was essentially two files:
build_utils.rb - run in a windows cmd window before build (or makepkg-mingw).  It used git to update the ruby repo, and also RI2 repo.  Three 'required' things it did to the src directory - bundled gems, revision.h, and cleaning previous build artifacts.  Since (originally) I thought I'd need git commands, I left it out of PKGBUILD, since I didn't want the potential conflicts between GitHub windows and similar commands in MSYS2, nor the seemingly large number of packages needed to run git in MSYS2.
You may have come across it already, but all the items added to src for and during build are contained in the ruby .gitignore file.
[package_utils]((https://github.com/MSP-Greg/ruby-makepkg-mingw/blob/master/mingw-w64-ruby/package_utils.rb) - this ran from within PKGBUILD, two tasks - add dll's, and add RI2.
Now the moving to 'why all the info'.  With the gem branch, I can add RI2 to the list of bundled gems and have ruby's package system add it to the package files.
Hence, I would no longer need git, and I might be able to move build-utils.rb into PKGBUILD, thereby allowing ruby version detection (of a local repo) to be handled properly.
So now the question - is the RI2 gem branch far enough along so that I could build it and test it in a build?

@MSP-Greg I roughly - but not fully - understand what you're doing.
Regarding the gem branch: This is the preparation of something called "RubyInstaller for Applications" or similar. It shall be possible to build customized Installer.exe or 7z files for particular Ruby applications or other Ruby use cases. It can be integrated per rubyinstaller-build.gem into Ruby applications and allows building of specialized installers. It shall also be possible to fine tune appearance and integration of the application by overloading almost all parts of the rubyinstaller-build.gem as needed. And of course it shall be possible to add arbitrary gems or msys2 packages to the installer package, so that everything works out of the box.
So, that's the vision. Now that it runs green again and I hope to have most of the required restructuring in place, I'll probably merge to the master branch soon. However there are still a lot more changes to do for the above feature set.

@larskanis
Wow.  That's quite a jump in scope.  Thank you.

I roughly - but not fully - understand what you're doing.

I probably could have been clearer.  Simple goal, to build ruby based on a local repo of trunk, along with integrating your work to allow gem extension installation and use.
First issue was getting source to match what is contained in the cache.ruby-lang.org bz2 files.  This was done by build_utils.rb (which probably should be named prepare_utils.rb).
Second issue was adding all the things (*.dlls, RI2 files) to the pkg directory, so makepkg-mingw would add them to the  xz file.  I don't have need for the xz file, but if someone else wanted it, it would contain what was needed.
When I saw the gem branch and gemspec file, I thought having an RI2 gem would make adding RI2 to a build much easier.
Two things --

Part of the reason I prefer to build from local source is that things go wrong.  I built with svn 57782, and all went as expected.  This morning I built with svn 57789, and check stopped (during the 'prep' stage) with a segfault.  With local, it's a bit easier to move around.

From a few messages, and the posting of my logs, the ruby core team has already made changes/fixes related to issues with mingw/MSYS2 builds.  A few have to do with readline.

It's not often that one wakes up, checks the recent ruby/ruby commits, and finds several referencing MinGW or Windows.  Thanks to @nobu and the core team for the work.
My local build code is now in MSP-Greg/ruby-loco.  I tried to make it run 'as close to windows' as possible, and it installs the RI2 runtime code (thanks to @larskanis).
If you try it, please review the README, as it is definitely pre-release.  For v2.3, use the MinGW OpenSSL package, for v2.4+, I've used the 1.1.0f package in the release.
Finally, I'm doing daily trunk builds, with results at Ruby MinGW Test Results.  Build and test results have been stable, although there are several patches...

As discussed in https://github.com/oneclick/rubyinstaller2/issues/55 , the official RI2 package repository is now https://github.com/oneclick/rubyinstaller2-packages .


adding a GAN

In https://github.com/facebookresearch/nevergrad/blob/5374254868e7689357f2bc61afecff5982cdb324/nevergrad/benchmark/experiments.py#L1100
there is an experiment using Nevergrad for optimizing a loss L:Image --> L(Image)
We need something based on latent variables! E.g.
given latent variables z, a GAN G, we could have z--> L(G(z)) so that we would work in the latent space and not directly on images.
A second step would be to also have the gradient, i.e. z --> L(G(z)), Gradient L(G(z)), so that we could compare to LBFGS or Adam or whatever.

Done by @brozi .


Add quiet option and change progress contents

What's
Add quiet option to suppress progress messages.
And change the progress display.
Related pull request #1
TODO

[x] Add quiet option
[x] Change progress contents

New progress spinner


RHIROS-426 | Upgrade ROS api routes from v0 to v1

Changes:

Replaced "v0" with "v1" on all routes
Modified usages references of all routes accordingly
Renamed v0 directory in ros.api to v1

Reference links:

consoledot
semver spec

Additional:
Added tests for api endpoints
Note:
Wrote test for the status endpoint, working on tests for other endpoints as well, hence MR is on WIP
Tried my best to update all references to "v1", in case I missed any please let me know

Need to change v0 on frontend repo also, here also https://github.com/RedHatInsights/ros-frontend/blob/main/src/constants.js#L2

@r14chandra Good point!
@PreetiW Does front-end need any more changes in respect to v1?

@saltgen can you resolve the conflicts before we merge this?

@saltgen can you resolve the conflicts before we merge this?

@apuntamb Ah I think these conflicts came up, cause of the earlier merges, on it.

@saltgen can you resolve the conflicts before we merge this?

@apuntamb Conflicts resolved :+1:


⚠ Bofrost.lu has degraded performance

In 8993b8d, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2257 ms

Resolved: Bofrost.lu performance has improved in 8dc8bee after 6 minutes.