Burrow Install - undefined: sync.Map

Running go get github.com/linkedin/Burrow resulted in the following:
# github.com/linkedin/Burrow/core/internal/cluster src/github.com/linkedin/Burrow/core/internal/cluster/kafka_cluster.go:224: undefined: sync.Map
Any ideas?

Same problem.

sync.Map is only available in go 1.9 and higher. My bet is you're trying to compile using go 1.8.*

Yeah, make sure you're using golang 1.9


Update Set-SafeLinksPolicy.md

https://github.com/MicrosoftDocs/office-docs-powershell/issues/4321

Docs Build status updates of commit 9223a2c:
:clock10: Full build: the average full build time is 11 min(s) 47 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 9223a2c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/advanced-threat-protection/Set-SafeLinksPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Doc | pre-requisites missing in the README.md

What?

📚 Detail the necessary leverage cli Python (>= 3.8) and packages requirements at the README.md

╭─    ~/Binbash/repos/Leverage/ref-architecture/le-dev-makefiles  on   master ················································································································ 127 ✘  at 16:43:35 
╰─ pip3 install leverage
Requirement already satisfied: leverage in /usr/local/lib/python3.9/site-packages (0.0.10)
Requirement already satisfied: yaenv==1.4.1 in /usr/local/lib/python3.9/site-packages (from leverage) (1.4.1)
Requirement already satisfied: GitPython==3.1.11 in /usr/local/lib/python3.9/site-packages (from leverage) (3.1.11)
Requirement already satisfied: gitdb<5,>=4.0.1 in /usr/local/lib/python3.9/site-packages (from GitPython==3.1.11->leverage) (4.0.5)
Requirement already satisfied: smmap<4,>=3.0.1 in /usr/local/lib/python3.9/site-packages (from gitdb<5,>=4.0.1->GitPython==3.1.11->leverage) (3.0.5)

Why?

✅ Improve the documentation in order to assure that new users can install and use the leverage cli without problems in their workstations envs.

@diego-ojeda-binbash OJ I've seen that we already have -> Supports python >= 3.6 in the README.md.
Could with close this issue?
CC: @angelofenoglio @lgallard

@exequielrafaela Sure. Let's do that. :)

Covered at https://github.com/binbashar/leverage/blob/master/README.md


Error: Uncaught (in promise): invalid views to insert with 3.9.2

Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
Please do not submit support requests or "How to" questions here. Instead, please use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
Seems similar to #11873 and #11374, which are marked as resolved.
I'm using the latest<EMAIL_ADDRESS>with<EMAIL_ADDRESS>and<EMAIL_ADDRESS>I have to specific issue when navigating through the app, except when I start using the back and next buttons of the navigation, I get this error in the console:
ERROR Error: Uncaught (in promise): invalid views to insert
    at resolvePromise (zone.js:824)
    at Object.eval [as reject] (zone.js:746)
    at NavControllerBase._fireError (nav-controller-base.js:322)
    at NavControllerBase._failed (nav-controller-base.js:310)
    at eval (nav-controller-base.js:365)
    at ZoneDelegate.invoke (zone.js:392)
    at Object.onInvoke (core.js:4629)
    at ZoneDelegate.invoke (zone.js:391)
    at Zone.run (zone.js:142)
    at eval (zone.js:873)

Expected behavior:
No error in the console.
Steps to reproduce:

Clone this repo: https://github.com/ngx-rocket/starter-kit
git checkout cordova/ionic
npm install
npm start to run the app (the build warning can be safely ignored)
Enter any login pass, then navigate a bit between home and about views
Click back/next on your navigator to trigger the error (tested on Chrome)

Related code:
See https://github.com/ngx-rocket/starter-kit, branch cordova/ionic
Navigation is tied to angular-router only for the route/events, navigation events are catched and then the Ionic router is used to navigate using setRoot like this:
  private updateNav(route: ActivatedRoute) {
    if (route.component === IonicApp) {
      if (!route.firstChild) {
        return;
      }
      route = route.firstChild;
      if (!this.nav.getActive() || this.nav.getActive().component !== route.component) {
        this.nav.setRoot(route.component, route.params, { animate: true, direction: 'forward' });
      }
    }
  }

Nothing too strange here, ultimately it is the Ionic router used, only reacting to URL changes.
Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
cli packages: (/usr/local/lib/node_modules)

    @ionic/cli-utils  : 1.19.0
    ionic (Ionic CLI) : 3.19.0

System:

    Node : v8.9.1
    npm  : 5.5.1
    OS   : macOS Sierra

Misc:

    backend : pro

Incredible. I did the same.


Fix unsafe state access in value entity proxy

The 'handler' parameter to 'performAction' was
called from a future outside of message processing.
While it 'maybe never lead to problems in practice',
accessing actor state like that is generally not allowed,
and could have lead to confusing situations (e.g.
misleading 'sender' parameters or actual problems when
we'd refactor things later).
For this reason I didn't find a way to introduce a test
that covers the problem.

LGTM!


crash error in discriminator training

after many OOM epochs get the following from the reporting

Traceback (most recent call last):
  File "/scratch/mk8347/mcrygan/main.py", line 31, in <module>
    predictor.train_crystal_models()
  File "/scratch/mk8347/mcrygan/crystal_modeller.py", line 528, in train_crystal_models
    self.logger.log_epoch_analysis(test_loader)
  File "/scratch/mk8347/mcrygan/reporting/logger.py", line 174, in log_epoch_analysis
    detailed_reporting(self.config, self.dataDims, test_loader, self.test_stats, extra_test_dict=self.extra_stats)
  File "/scratch/mk8347/mcrygan/reporting/online.py", line 1349, in detailed_reporting
    discriminator_BT_reporting(dataDims, wandb, test_epoch_stats_dict, extra_test_dict)
  File "/scratch/mk8347/mcrygan/reporting/online.py", line 917, in discriminator_BT_reporting
    make_and_plot_BT_figs(crystals_for_targets, target_identifiers_inds, identifiers_list,
  File "/scratch/mk8347/mcrygan/reporting/online.py", line 847, in make_and_plot_BT_figs
    fig = make_correlates_plot(tracking_features, scores_dict['CSD'], dataDims)
  File "/scratch/mk8347/mcrygan/reporting/online.py", line 1321, in make_correlates_plot
    fig.update_xaxes(range=[np.amin(list(g_loss_dict.values())), np.amax(list(g_loss_dict.values()))])
  File "<__array_function__ internals>", line 200, in amin
  File "/scratch/mk8347/crystal_learning/lib/python3.9/site-packages/numpy/core/fromnumeric.py", line 2946, in amin
    return _wrapreduction(a, np.minimum, 'min', axis, None, out,
  File "/scratch/mk8347/crystal_learning/lib/python3.9/site-packages/numpy/core/fromnumeric.py", line 86, in _wrapreduction
    return ufunc.reduce(obj, axis, dtype, out, **passkwargs)
ValueError: zero-size array to reduction operation minimum which has no identity

distinct from the issue of the OOM cascade, it seems like it is missing some reporting from an earlier epoch

actually the g_loss_dict is just empty

fixed


README should make it clear that sliding-sync proxy should run behind a reverse proxy

Currently, the README offers a snippet of nginx config for those wanting to run sliding-sync proxy on the same FQDN as their base Matrix domain, but it doesn't make explicit in any way that in any case, sliding-sync proxy should run behind a reverse proxy like nginx or Caddy.
More problematic, sentences like "Optionally also set SYNCV3_TLS_CERT=path/to/cert.pem and SYNCV3_TLS_KEY=path/to/key.pem to listen on HTTPS instead of HTTP" can let users think that it's ok to run sliding-sync proxy in plain HTTP with clients directly connect to it.
The README should be updated to make it clear that sliding-sync proxy should be run behind a reverse proxy, and ideally provide the configuration for how to do so for nginx and Caddy.
It will probably make sense to also make clearer the two cases described in the readme: running sliding-sync proxy on its own, separate FQDN (cue reverse proxy instructions) and running sliding-sync proxy on the same FQDN as the Matrix base domain (cue reverse proxy instructions).

It's not really our place to say how a server admin should run their proxy.
Whilst I would agree the "typical" configurations would be A) behind a reverse proxy in HTTP mode, B) internet-facing in HTTPS mode, all 4 permutations are valid depending on your use case. You may want to keep using HTTPS inside your DC for security reasons (so C) behind a reverse proxy in HTTPS mode) or you may already be on a custom trusted secure network / testing for evaluation purposes (so D) intranet-facing in HTTP mode).
It's also not at all clear why we would have nginx and Caddy instructions but not Apache, HAProxy, traefik, or countless others. I'd preferably have no reverse proxy instructions at all, as we do not test the snippets we provide so they risk bitrotting, but there is value in having one clear-ish example which can be tweaked depending on your reverse proxy of choice.
This is just unfortunately the cost to running your own servers: you need to know this and cannot expect every piece of server software you run to have your specific requirements exactly met. If we wanted to lower the barrier here, we should be owning more of the networking stack e.g provide a helm chart with an Ingress we know about so can configure correctly for the user.
I don't see anything actionable here, though I am open to PRs which make the documentation clearer for server admins, provided A) it doesn't make the README a choose-your-own-adventure game of if apache then elif nginx then ... as that obfuscates things, B) doesn't rule out valid use cases which the proxy supports, C) isn't a pain to maintain.

How about adding a short paragraph akin to what Synapse's README has? That seems quite clear in intention without obfuscating things and makes sense (to me) since sliding-sync is effectively a HS extension.


Bump futures crates to 0.3.19

Motivation
Some futures crates must be merged together, so this PR combines #3264 #3266 #3267
Specifications

Designs

Solution
Combine #3264 #3266 #3267

Review
I already reviewed the updates themselves, the reviewer just need to check if this PR truly combines all of #3264 #3266 #3267
Anyone can review this, it's not particularly urgent.

Reviewer Checklist

[ ] Code implements Specs and Designs
[ ] Tests for Expected Behaviour
[ ] Tests for Errors

Follow Up Work

Google Cloud failed due to bug #3239, I'll restart that job.


problem with oakd.launch.py

Robot Model
Turtlebot4 Standard
ROS distro
Humble
Networking Configuration
I do not know
OS
Ubuntu 22.04
Built from source or installed?
Built from Source
Package version

Expected behaviour
i'm launching
ros2 launch turtlebot4_bringup oakd.launch.py
in config file oakd_pro.yaml i changed:
i_pipeline_type: RGB to RGBD to get depth image topic
i_height: 720 to 250 in order to match preview image size (rgb image topic) that is 250x250
i_width: 1280 to 250  in order to match preview image size (rgb image topic) that is 250x250
i wanted to have images of the same size because it's a requirement to run a visual odometry algorithm (rtabmap).
Actual behaviour
when i change
i_height: 720 to 250
i_width: 1280 to 250
i get an error and camera always stop working
is there a way to obtain images of the same dimensions without making the camera crash? Thank you for your help
Error messages
No response
To Reproduce
ros2 launch turtlebot4_bringup oakd.launch.py
with the changes in config file written above
Other notes
No response

@IacopoFornai Please provide the full log of the output from when you launch the node including the error lines.
That being said, also make sure that if you are launching the camera node manually, that you have already stopped the oakd node that is launched at boot. Or, instead of launching manually, source the workspace etc. such that the node is launched automatically using your custom settings.


nxp_mpu: suspicious ENDADDR_ROUND() macro

This is currently implemented as:
#define ENDADDR_ROUND(x) (x - 0x1f)
This is not a rounding operation unless the last 5 bits are always 1.
Please double-check what this is supposed to be doing.

Yeah looking at the NXP reference manual, it does appear that the lower 5 bits are all considered to be 1's.  So this works correctly as currently implemented.


idle time peaks for 1 frame roughly 30 seconds after game starts

Godot version:
3.2.1.stable
OS/device including version:
windows 10
Issue description:
idle time peaks for 1 frame roughly 30 seconds after game starts (image below is 144 fps)

the only solution ive found is to set the "force fps" debug setting to at least 1 frame lower than my monitors refresh rate (so 143 or less is fine). doing that makes it so there isnt even a little spike.
heres an (unresolved) reddit thread i found that had the same issue: https://www.reddit.com/r/godot/comments/bhq5ah/idle_time_peaks_some_seconds_after_game_starts/
Steps to reproduce:
hit play and wait 30 seconds
Minimal reproduction project:
any project in my testing (including brand new blank ones)

Looks to be the same as https://github.com/godotengine/godot/issues/33969

Duplicate of #33969.


add manual bindings for GRecMutex

@sdroege : I'll need this for https://gitlab.freedesktop.org/gstreamer/gstreamer-rs/issues/161
Thanks to @GuillaumeGomez for helping me implementing this during the hackfest. :)

The implementation looks good to me. Just a few things we should hide to users but that's it. Good work!

If we decide to add these, I'd recommend first of all adding the normal mutex. I don't think it's a good idea to provide these in the bindings as user-facing API though. If anything it should be used internally for interaction with existing C code. The mutexes/etc in the standard library are better.
@gdesmott see also how I used GMutex and GRecMutex in gstreamer-rs internally in a few places already.

The mutexes/etc in the standard library are better.

To extend on this, the Rust model is to protect data and not code. The Rust mutexes are containers around data, while the GLib ones are only protecting code.

I no longer need this so feel free to WONTFIX if you think that's not worst adding to glib-rs.
If not I'll be happy to write similar bindings for GMutex.

If we actually do want this, @GuillaumeGomez suggested another lock API taking a FnOnce which will be called right away after locking the mutex. This would match better the "mutex protecting code" semantic you described.

I don't think it would be a good idea to merge this as its behaviour is less safe and different to how Rust mutexes work.

It's up to you and @GuillaumeGomez, I don't really care as a I no longer need it for my bindings. Feel free to close.

Let's close this for now, we can easily resurrect it later if needed.


CONTINENT - FETCH population values for all the COUNTRIES in A CONTINENT from the database.

Added to github


basic linux support

Hello!
this is with regard to issue #104
The latest pyfemm supports linux, so I merged that into pyleecan.
From my tests, it supports drawing in femm properly, as well as simulation, though it is slow.
Some of the things that I noticed didnt work are :-

the get_meshsolution function didnt work, but I think the errors were only to do with the filepaths.
Can we have a global isWindows flag somewhere? Maybe in the _FemmHandler itself?

parallelization didnt work - I think thats because in Linux we need to create a file in the same directory as femm to transfer the command, so if you open multiple FEMMs its not clear which instance is opening or deleting the file. A quick though ugly hack might be to just have multiple copies of the FEMM folder.

Thank you
Harsha

Hello there !
Thank you very much for this contribution.
It is not really code related, but could you provide some details on how you managed to "install" FEMM on your Linux OS ?
I am working with Mac OS, and I want to check if your PR is working on my side as well.
Best,
Raphaël

Hello!
I just installed it through wine. If you look at openFemm function inside  _FemmHandler.py file (in the Classes subdirectory), there are some default paths for linux that the code searches for FEMM (including the wine dir).  The new function even takes a path to the exe as an argument. Maybe for MacOS you might need to change those paths, or feed the path into the function as an argument
Thanks,
Harsha

Okay thanks. For unknown reason, I was not able to use Wine. I am currently able to run femm GUI, but I still have to look how to make it work with pyfemm. I will let you know when I can test your PR.

Hello,
I checked your modifications and it looks fine to me. All the tests are passing on Windows including with parallelization.
I would only have two readability improvements to suggest:

What about renaming windowOS to is_window_OS ?
Do you think that you could split the modified methods into X_Win and X_Linux with an encapsulating method calling the correct version according to windowOS ? In particular for openfemm, callfemm and callfemm_noeval
I should have time to do both myself if needed.

@RaphaelPile When do you think you would have time to check if it works on your side ? I'm currently not using Linux so I would be a great help if you could validate these modifications :) Otherwise I will take time to install Linux but I don't know when.
By curiosity what are the methods "X_setcomment" ? Are they methods introduced by pyfemm 0.1.3 ? I think we haven't updated the _FEMMHandler for a long time and I somehow managed to forget where this file came from in the first place :/
Thanks again for your contribution,
Pierre

Hello,
Sorry for the delay. I have made the changes you suggested, it makes the code much more readable.
the new X_setcomment functions allow you to "set the comment text in the problem definition". They were adding in the latest pyfemm update
Unfortunately its still very slow in linux. You can use it to draw the machine, but very tough to use it for doing simulations and impossible for optimizations I think.
Thank you,
Harsha

@BonneelP I don't know, I have already issues to configure VSCode for python on my Mac OS. I will try my best to do it before the end of October.

Hello,
I didn't manage to try your modifications on Linux, but all the tests are still Ok on windows and I'm about to create a new release to solve issue #569 so I'm (finally) merging your modification. Sorry for the long time to merge your contribution, it will be really helpful for the community.
I will still try to find some time to use your code on Linux to see if we can solve the speed issue.
Best regards,
Pierre


[TASK] 🚀 Pre-launch audit: Nominations page

[ ] Add actual staked  / ideal staked (3%/50%)

[ ] Link to “Open Explorer” did not use the actual connected network RPC (eventually this will link to Subscan)

[ ] On network change, we need to identify how to reset state across the app, specific to nominations (i.e. local storage should maybe be reset OR stored per network)

[ ] Payouts doesn’t realize I have nominations, it should show me a different message. “There are no active payouts for your nominations currently.”

[ ] Hide validators profile page as it is currently displaying dummy data

The balance is not currently updated when changing rpc endpoint
https://github.com/webb-tools/webb-dapp/assets/69841784/0583be92-d1b6-4416-a334-c0007abe0ab0

The balance is not currently updated when changing rpc endpoint
CleanShot.2024-03-26.at.08.57.52.mp4

I think this may be due to a problem with local storage caching.

Claiming this one! @devpavan04

@devpavan04 added additional task for removing the Get tTNT button when switched to mainnet


Ignore utest and atest directories on codecov.io

I've tried to put codecov.yml to another directory (e.g. .github) and reference it from the action, but this doesn't work :-(


Guessing this has been pretty much abandoned?.

The 1.1.0-androidv2 of esptouch_flutter has been setting in pre-release for a year with the last updates to esptouch_flutter coming 18 months ago.  What package are people using with similar functionality now?

Bump from a friend ;)
@vincevargadev @max-koehler  @ducnguyenenterprise

The main concern with this no longer being maintained is:
The plugin `esptouch_flutter` uses a deprecated version of the Android embedding.

@vincevargadev @max-koehler @ducnguyenenterprise
Please, help us!!!
One big problem: the new IOS 16 dont work with this library.
can you give us a light?

@lucasromeiroTF I think you might have the same case as here. You can read more about it as Breaking change in iOS 14.6


<EMAIL_ADDRESS>decompress error

PS C:\Users\Michael T. Richter> scoop install meld
WARN  Purging previous failed installation of meld.
ERROR 'meld' isn't installed correctly.
Removing older version (3.22.0).
'meld' was uninstalled.
Installing 'meld' (3.22.0) [64bit] from extras bucket
Loading Meld-3.22.0-mingw.msi from cache
Checking hash of Meld-3.22.0-mingw.msi ... ok.
Extracting Meld-3.22.0-mingw.msi ... Invoke-ExternalCommand: C:\Users\Michael T. Richter\scoop\apps\scoop\current\lib\decompress.ps1:161
Line |
 161 |      $Status = Invoke-ExternalCommand $MsiPath $ArgList -LogPath $LogP …
     |                                       ~~~~~~~~
     | Cannot validate argument on parameter 'FilePath'. The argument is null or empty. Provide an argument that is not null or
     | empty, and then try the command again.
Failed to extract files from C:\Users\Michael T. Richter\scoop\apps\meld\3.22.0\Meld-3.22.0-mingw.msi.
Log file:
  ~\scoop\apps\meld\3.22.0\msi.log

Please try again or create a new issue by using the following link and paste your console output:
https://github.com/ScoopInstaller/Extras/issues/new?title=meld%403.22.0%3a+decompress+error
PS C:\Users\Michael T. Richter>

~
❯ scoop install meld
WARN  Scoop uses 'aria2c' for multi-connection downloads.
WARN  Should it cause issues, run 'scoop config aria2-enabled false' to disable it.
WARN  To disable this warning, run 'scoop config aria2-warning-enabled false'.
Installing 'meld' (3.22.0) [64bit] from extras bucket
Starting download with aria2 ...
Download: Download Results:
Download: gid   |stat|avg speed  |path/URI
Download: ======+====+===========+=======================================================
Download: 5480ff|OK  |   5.6MiB/s|C:/Users/lutra/scoop/cache/meld#3.22.0#https_download.gnome.org_binaries_win32_meld_3.22_Meld-3.22.0-mingw.msi
Download: Status Legend:
Download: (OK):download completed.
Checking hash of Meld-3.22.0-mingw.msi ... ok.
Extracting Meld-3.22.0-mingw.msi ... done.
Linking ~\scoop\apps\meld\current => ~\scoop\apps\meld\3.22.0
Creating shim for 'Meld'.
Creating shortcut for Meld (Meld.exe)
'meld' (3.22.0) was installed successfully!

Failed to reproduce the problem, I suggest you run scoop checkup to fix potential issues.

scoop checkup lists three packages that "aren't installed" that scoop list says are installed.
And with that I'm out.  Scoop already fails all of the important fallacies of distributed computing and now turns out to be fragile like a snowflake in a fireplace.  I'd rather install things manually than rely on a system as fundamentally broken as this one.

Temporarily closed, feel free to reopen this issue.
Reasons:

Cannot reproduce.
No response and additional information


wr.s3.read_csv() should support single-file paths more explicitly

Is your idea related to a problem? Please describe.
I ran into a weird issue trying to read CSV files generated by Athena while using the awswrangler.s3.read_csv() method. I kept getting errors about file encodings. I tested the CSV files and opened support cases and of course, they were in 'UTF-8'. However, after some debugging, I found that the awswrangler library was also finding the metadata file generated by Athena, switching into multi-file read mode, and then throwing an error upon attempting to read the binary metadata file as a unicode CSV.
Describe the solution you'd like
I think the s3.read_csv method should assume a single file path when using the path parameter and a new parameter, "prefix", should be added. Until then, documentation for the method should be updated to warn users of this potential mixup. Although the docs are literally correct, they are not intuitive.
It might make sense to add a different CSV method to the Athena module such as "wr.athena.read_results(execution_id: str) -> pd.DataFrame" to better match the semantics for users who want to retrieve query results when the query execution was executed outside of awswrangler which is often the case in distributed systems. This alone would not solve the issue of a user finding the s3.read_csv method and assuming that there are no files with a prefix including file extension ".csv" other than the data file.

@cshumaker-irb Can you clarify what you mean by "assume a single file path"? If you pass a complete object key it will read that object, If you pass a prefix, it will read all objects under that prefix - I think that's a fair intuitive expectation.
You can ignore objects (like Athena metadata that's causing issues in your case) by using path_ignore_suffix parameter and filter suffixes using path_suffix.
I do agree a method retrieving query results by an execution id makes sense, I think we can add a wrapper around existing wr.athena._fetch_csv/parquet_result to fetch query metadata by query id and then return the result.

Hi @kukushking, thanks for looking into this. Between a quick note in the documentation and the athena.read_execution_results(), I doubt anyone will run into this problem.
The reason I ran into it is that I wasn't thinking about the metadata files while starting to work with the read_csv() method and while trying to debug the error, was trying to understand why the CSV would cause a decoding error (it wasn't, it was the metadata file). I think a lot of users probably aren't going to think about the metadata file but I don't know how many are using read_csv on Athena results anyway.
WRT to "assume a single file path", perhaps that's the right solution. What I meant was that the path variable shouldn't be treated as either a prefix or a list of keys. Maybe the better solution would be to have a "keys" argument and a "prefix" argument, rather than one argument that captures both because it might prompt users to check if there's other files.
I'm not sure the path_ignore_suffix and path_suffix are the right solution. They will probably do the job but not intuitv
I've solved the problem on my side by just wrapping the key in a list so I don't really need this to be implemented a certain way, I just want to spare future users a couple hours.


Would like to delete individual pipeline runs by id [Feature Request]

Is your feature request related to a problem? Please describe.
I cannot delete pipeline runs from the az pipelines command set. A sample use case would be to delete all pipeline runs that are not run against the main or master branch. I often accumulate pipeline runs when testing a new or updated pipeline on a feature branch in my git repository. When I'm done I want to clean them out of the history - they have no value other than to clutter things up.  I have to manually clean them one at a time in the user interface.
Would like to be able refine a query against the runs, then iterate through runs that meet certain criteria and delete them.
Describe the solution you'd like
az pipelines runs delete -id xxxx

Looking for a similar function.
For the time being, I am playing with a bash script:
 - job: Cleanup
        steps:
          - checkout: none
          - bash: |
              declare -a buildUrls=$(az pipelines runs list --org <org> --project <prj> --branch <branch> --query '[?url].url | sort(@)')

              buildUrls=${buildUrls:1:-1}
              buildUrls=$(echo $buildUrls | tr -d '"')
              IFS=","
              declare -a urls=($buildUrls)

              for element in "${urls[@]}"; do
                  az rest \
                    --method delete \
                    --headers "Authorization=Bearer $AZURE_DEVOPS_EXT_PAT" \
                    --url $element\?api-version\=7.1-preview.1 \
                    --resource 499b84ac-1321-427f-aa17-267ca6975798
              done
            env:
               AZURE_DEVOPS_EXT_PAT: $(System.AccessToken)
            displayName: "Delete Build Runs"


select.getFeatures().clear() takes a very long time to complete

Describe the bug
The time taken by the select.getFeatures().Clear() function grows exponentially with the number of objects:

1000 (1s)
10000 (3s)
30000 (>120s)

If the layer is visible or not the execution time is the same. No events are attached to select or features collection.
All features shares same style instance via same styleFunction.
To Reproduce
Select 1000, 10000 and 30000 features and use select.getFeatures().Clear()
Expected behavior
select.getFeatures().Clear() should be completed within a reasonable time.

Out of curiosity, how are you selecting 30000 features with a Select interaction?
In order to help, you'd have to provide more context from your code. It is likely that a listener connected to the clear() operation takes a long time to execute.

Probably such a large selection of features is not the most appropriate so we do not consider this as an OL error,
therefore, you can close the issue.
The selection is made with the "polygon selection tool".
You can test by using VectorLayer and then use select.getFeatures().push(ftr).

In case there is only one interaction.select, is it really necessary to check in which interaction.select the Feature is located,
because slow operation is caused by the function interaction.getFeatures().GetArray().IndexOf (feature) !== -1.

https://github.com/openlayers/openlayers/blob/246aded8192391bbeb9034dbf260e84e0f9d58a3/src/ol/interaction/Select.js#L393
It might make sense to optimize the code so that this test is performed only when there are multiple interaction.select.

Thanks for investigating this! Would you be able to provide a pull request with such an optimization? That would be great.

I do not have the environment setup to test changes. Would be possible for someone else to make the changes.
If I relate to https://github.com/openlayers/openlayers/pull/1069, my suggestion is something in this direction (not tested):
 /**
   * @param {import("../Feature.js").default} feature Feature
   * @private
   */
  restorePreviousStyle_(feature) {
    const key = getUid(feature);
    const selectInteractions = /** @type {Array<Select>} */ this.getMap()
      .getInteractions()
      .getArray()
      .filter(function (interaction) {
        return interaction instanceof Select && interaction.getStyle();
      });

    if (selectInteractions.length > 1) {
       const selectInteractionsLenght = selectInteractions.filter(function (interaction) {
         return interaction.getFeatures().getArray().indexOf(feature) !== -1;
       }).length;

      if (selectInteractionsLenght > 0) {
        feature.setStyle(
          selectInteractions[selectInteractionsLenght - 1].getStyle()
        );

        return;
      }
    }

    feature.setStyle(originalFeatureStyles[key]);
    delete originalFeatureStyles[key];
  }


[Event Request] Table 81 Gen. Journal Line - OnAfterLookUpAppliesToDocVend

Hi Team,
In table 81 we need possibility to modify record with our own code. Having event at the end of the procedure LookUpAppliesToDocVend, would be perfect solution:

[IntegrationEvent(false, false)]
local procedure OnAfterLookUpAppliesToDocVend(var GenJournalLine: Record "Gen. Journal Line"; CustLedgEntry: Record "Cust. Ledger Entry"; VendLedgEntry: Record "Vendor Ledger Entry"; ApplyVendEntries: Page "Apply Vendor Entries")

Hi @MantasGurinas . Why do you need ApplyVendEntries page as parameter in this event? Is it ok if you get it in event above?

Hi @RomanZanizdra , after inspecting more, it can be moved to event above or dismissed.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Bug bash to find outdated content (docs & samples) to accompish tasks for Key Vault

Update from @ramya-rao-a
Review the .Net samples for Key Vault in the below locations to find the ones that do not use the latest version of the package. Update this issue with the list of samples that need updating.

Key Vault .Net samples repos under Azure-samples org
Key Vault .Net samples under docs.microsoft.com/samples

Additionally, ensure the samples that live next to the source code for Key Vault in this repo shows up in Key Vault .Net samples under docs.microsoft.com/samples. This is done by setting up the right metadata. Ensure that these samples are not outdated.

@christothes Re-opening this issue with additional instructions added to the issue description following the approach we are using across languages and services. Please take another look

Thanks for updating the issue description @AlexGhiondea
@christothes Please use https://github.com/Azure/azure-sdk-for-net/issues/16733 to update the above samples


Align Keycloak endpoints to recommendation

https://www.keycloak.org/server/reverseproxy#_exposed_path_recommendations
Lets expose only recommended endpoints for Keycloak.
Acceptance Criteria:

EDP scripts and documentation provides only recommended Keycloak endpoints out of the box for better security;

By including these parameters in your values.yaml file, you configure Keycloak to follow the recommended endpoint settings. The console.enabled: false parameter disables the Keycloak console endpoint for better security. The exposeMetrics: false and exposeHealth: false parameters disable the metrics and health check endpoints, respectively, to eliminate unnecessary attack vectors.
health:
  enabled: false
metrics:
  enabled: false
ingress:
  console:
    enabled: false

For more information and additional configuration options, please refer to the documentation on our website about the EDP project.


Way to run operations on CPU, thusly RAM instead of GPU/VRAM?

I'm limited by the Amazon instance. I can buy a server with a K80 and 12GB of VRAM but I'd also be willing to pay more for slower processing on the cloud if I can get the CPU working.
I'm going to look at the code after this but in the event that the answer is obvious to the community, I wanted to ask before I go digging.
Thanks so much! Hope everyone's results have been fun.

Just do not enable CUDNN and the script will run on a CPU. But it will take you hours if not days to get a picture.

The CPU implementation of the convolution operation is too slow for any practical use. You would have to implement something faster first.

I believe you that it's slow and I appreciate both very quick responses!
Since our needs may be slightly different, by what magnitude is it impractical? We talking days or weeks for 1MP + resolution?
Thanks!

fyi https://github.com/jcjohnson/neural-style/

I've used it, the results even with gradient smoothing are not ideal whatsoever. I've done enormous amounts of tweaking and attempted to isolate what I like about it, it just isn't as good. I will perhaps come back to it anyways because it's more fleshed out.


Introduce debug functions to pg_analytics

What
Users are getting table not found errors that are tough to debug because we don't have access to their DB. We should introduce a function that prints the DataFusion schema, i.e.
SELECT * FROM paradedb.parquet_schema();
-- Returns a table with all DataFusion tables and their respective schemas

Why
For debugging
How

Closing these as we're archiving this repository for good


Add assembly for satd

Coverage decreased (-0.02%) to 81.011% when pulling 78318491bc00757008c1fe40ea687fa10a292241 on KyleSiefring:satd_asm_review into 5079a62a69ec62578452ead6b9d5e5022b37996e on xiph:master.

Code review and replies from tim.
+  pub mod tiling { pub use crate::tiling::*; }

Can this be pub(crate)?

+pub fn get_sad

This should be pub(crate)

+pub fn get_satd

This should be pub(crate).

+pub use self::nasm::get_satd;

pub(crate)?

> All of these are copied from the surrounding lines or similiar code. I don't know why they are the way they are and I don't really want to change them in this patch.

+fn bench_get_satd

Benchmarks are nice and all, but what I actually want to see are correctness tests. SAD has a test against "known good" values, but even better would be a way to run the whole encoder where the SIMD is always checked against the "native" version (a la CHECK_ASM in Daala).

> We have tests for satd like sad already in the code. I will add a check_asm mode when changing to static dispatch.

+    let org_stride = plane_org.plane_cfg.stride as libc::ptrdiff_t;
+    let ref_stride = plane_ref.plane_cfg.stride as libc::ptrdiff_t;

I think you should probably just use isize instead.

> Yeah, I just copy and pasted from sad. I can change this. I would like to switch to isize myself since I don't see a difference.

+    ; 0, 1, 0, 1, 0, 1, 0, 1
+    ; 2, 3, 2, 3, 2, 3, 2, 3

These comments pretty much make sense to you, and no one else.
I get what you mean here, for the first one, but beyond this what you call "0", "1", etc., becomes unclear, as the inputs are a two dimensional array which you are partially transposing. I.e., it is not easy to verify where all of the values from an input column go, because (a) you name all of the values from different columns the same thing and (b) you're mixing multiple values together without changing their names. I only figured that you were transforming columns and then rows (instead of, for example, transforming 2x2 blocks and then groups of 2x2 blocks) by tracing through the instructions manually. From the comments this was completely opaque.

> Thanks for the input. I was wondering if the comments would be erudite.

+    shufps             tmp, m0, m1, q2020
+    shufps              m0, m1, q3131

Mixing floating point and integer operations on the same registers is something that Intel threatened to make slow in the past, followed through on for some processors, and then made fast again. It is the sort of thing they are likely to mess with in the future as well, so I would avoid it where we can. In this case I do not think that you need it at all, especially since you do not care about the order of the final coefficients in the registers.

> I'm not super concerned about intel breaking a floating point shuffle instruction in the future. I will write it with your method since it looks faster.

+    ; Reduce horizontally early instead of fully transposing
+    ; 2, X, 2, X

I'm not sure this is saving any work, since you are only using half of the resulting registers.

If you use the sequence bfly -> punpck[lh]wd -> bfly -> punpck[lh]dq -> bfly -> punpck[lh]qdq -> bfly+abs+add then you wind up with everything in m0 ready to be summed horizontally, and do not require shufps (to be avoided for the above reasons) or pshufd (slow on some processors, because it requires moving things on arbitrary sides of the 64-bit boundary).

Here is an example of comments decribing what happens in the above sequence that someone else might actually be able to follow. It uses the convention that the first lane is written on the left, which is in my experience the more common one.

m0  d2 c2 b2 a2 d0 c0 b0 a0
m1  d3 c3 b3 a3 d1 c1 b1 a1

bfly ->
m0  d2+d3 c2+c3 b2+b3 a2+a3 d0+d1 c0+c1 b0+b1 a0+a1
m1  d2-d3 c2-c3 b2-b3 a2-a3 d0-d1 c0-c1 b0-b1 a0-a1

punpck[lh]wd
m0  d0-d1 d0+d1 c0-c1 c0+c1 b0-b1 b0+b1 a0-a1 a0+a1
m1  d2-d3 d2+d3 c2-c3 c2+c3 b2-b3 b2+b3 a2-a3 a2+a3

bfly ->
m0  d0-d1+d2-d3 d0+d1+d2+d3 c0-c1+c2-c3 c0+c1+c2+c3 b0-b1+b2-b3 b0+b1+b2+b3 a0-a1+a2-a3 a0+a1+a2+a3
m1  d0-d2-d2+d3 d0+d1-d2-d3 c0-c1-c2+c3 c0+c1-c2-c3 b0-b1-b2+b3 b0+b1-b2-b3 a0-a1-a2-a3 a0+a1-a2-a3

->
m0  s2 s0 r2 r0 q2 q0 p2 p0
m1  s3 s1 r3 r1 q3 q1 p3 p1

punpck[lh]dq ->
m0  q3 q1 q2 q0 p3 p1 p2 p0
m1  s3 s1 s2 s0 r3 r1 r2 r0

bfly ->
m0  q3+s3 q1+s1 q2+s2 q0+s0 p3+r3 p1+r1 p2+r2 p0+r0
m1  q3-s3 q1-s1 q2-s2 q0-s0 p3-r3 p1-r1 p2-r2 p0-r0

punpck[lh]qdq ->
m0  p3-r3 p1-r1 p2-r2 p0-r0 p3+r3 p1+r1 p2+r2 p0+r0
m1  q3-s3 q1-s1 q2-s2 q0-s0 q3+s3 q1+s1 q2+s2 q0+s0

bflyabs ->
m0  p3-r3+q3-s3  p1-r1+q1-s1 p2-r2+q2-s2 p0-r0+q0-s0 p3+r3+q3+s3 p1+r1+q1+s1 p2+r2+q2+s2 p0+r0+q0+s0
m1  p3-r1-q3+s3  p1-r1-q1+s1 p2-r2-q2+s2 p0-r0-q0+s0 p3+r3-q3-s3 p1+r1-q1-s1 p2+r2-q2-s2 p0+r0-q0-s0

> Will try to implement this.

+    ; Utilize the equality (abs(a+b)+abs(a-b))/2 = max(abs(a),abs(b)) to merge
+    ;  the final butterfly stage, calculation of absolute values, and the
+    ;  first stage of accumulation.

Can't do this.

> Got it.

+    movhlps             m1, m0

Mixing float and int ops again. movhlps is also creating a false dependency here (the high 64 bits of m1 are left unmodified, but you do not care what they are). You can just use punpckhqdq.

+    movhlps            xm1, xm0

I believe this usage can be replaced by punpckhqdq as well.

> I would have not caught the false dependancy. Will avoid this instruction in the future. It's in many places in dav1d so I thought it was good.

+; Two Hadamard transforms share a lane.

I'm not sure what this means (here and elsewhere). "Lane" usually refers to a single element of a SIMD register (i.e., a 16-bit word in a 128-bit register), so it would not make any sense to share one. Do you just mean register?

> lane means an 128-bit part of an avx register. This is actually pretty common parlance. I will be more specific. Use 128-bit lane instead.

+    ; transpose

libtheora's sse2trans.h includes an implementation of an 8x8 transpose that also only uses one temporary but does not require any register renaming, incidentally. It is the sort of thing that should probably be in its own macro, as well. I'm sure we'll want to use it in more places eventually.

> This looks like something that doesn't translate well to avx2's 3 op instructions.

+    ; 0->8, 1->0, 2->1, 3->2, 4->3, 5->4, 6->5, 7->6, 8->7

I think this comment is backwards? (m0, m1) is an active output pair from the previous stage, but sending m0 -> m8 and m1 -> m0 both (a) splits it up and (b) discards the value of of m1 (since m0 is immediately overwritten in the next block).

> Oh, wow. These comments were giving me headaches when I wrote them. Will look into fixing them.

+    ; Reduce horizontally and convert to 32 bits
+    pxor                m2, m2
+    punpcklwd           m1, m0, m2
+    punpckhwd           m0, m2
+    paddd               m0, m1

Any reason not to use pmaddwd with a register of all 1's?

+    pshufd             xm1, xm0, q1111

If only the low 32 bits matter, any reason not to use pshuflw instead? It should be faster.

> Will do.

Squash and merge anytime I guess :)

I'm going to wait for Tim to respond. He may have time to review code on the trip back.

Somewhere in this the strides are getting swapped :/.
Should modify the tests to actually test for this.


⚠ Samsung has degraded performance

In e7754ee, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2627 ms

Resolved: Samsung performance has improved in dada854.


feat(commerce): add commerce category facets

Taking this over from Fred since he's on vacation! ✋

@louis-bompart @olamothe :wave: I'd love to get a review on this when you have the chance


Triggers test plan

Fixes #42

@gabo1208 great.. this is good progress
/lgtm

@lionelvillard if you know someone interested in reviewing this please let me know..  if not I will merge and we will try to review and send changes with PRs.
I want to have at least the basic sections of the spec covered for future reference.

@gabo1208 if you can create another issue to add the links to the tests from rekt that you find, that will be great.. but let's get this PR merged first

/close
This was an exercise in 1.0 preparation, but we don't need to commit these files at this point.


Made some EX-era cards look for that time's definition of "Basic Pokémon" [in play], also a similar thing but for SM-era cards and "Evolution Pokémon"

Should correct issues with "treat that card as a Basic Pokémon", namely:

Holon Fossil (HP 86)
Anorith (LM 29)
Strange Cave (LM 77)
Omanyte (PK 56)

Many cards across the EX-era files were coded using pcs.basic, while many others used !pcs.evolution. Checking around, the correct definition for these attacks would be "Basic Pokémon [in Play]: Not an evolved card". There's no need to add/remove "BASIC" or "STAGE 1" cardTypes, the bottom intent of that text in the above cards' effect is not considering them as evolved. On the opposite side of the spectrum, Baby cards using "Baby Evolution" make the Basic put on top of them count as an evolved Pokémon, even though being a Basic Pokémon card:

Q. Say I use my Pichu's "Baby Evolution" power to evolve Pichu into Pikachu, which is now an "evolved Basic Pokémon". Can I then play Rare Candy on my Pikachu to evolve it into Raichu or Raichu-EX during the same turn?
A. Since Pikachu would be considered an "Evolved Pokémon" at that point, Rare Candy cannot be used on that Pikachu. Rare Candy can only be used on "non-Evolved" Pokémon. (Apr 14, 2005 PUI Rules Team)

DP-era onwards, the term "Basic Pokémon" is now used for actual "BASIC" cards on top, so those should indeed work with pcs.basic as they already do:

Q. If I have Uxie, Azelf, and Mesprit with the "Downer Material" Poke-BODY in play, and my opponent has a Snorlax evolved from a Munchlax in play, do they have to pay more for the evolved Basic Pokemon's attacks?
A. Yes, they do. While Snorlax is evolved it is still a Basic Pokemon card, and that is what Downer Material is looking at. (May 27, 2010 PUI Rules Team)

As a reminder to myself, last commit includes changes for the EX Deoxys set in particular. There's another PR (#583) with changes for that set, so assuming both are good to merge it might be needed to rebase whichever is accepted last :P

Also, just pushed updates for SM-era sets regarding "Evolution Pokémon". This term was added around the BW/XY era, with cards like craddily placing Stage 1 and 2 Pokémon in play. More notorious example being Breakpoint's Frogadier with "Shadow Duplicates":

Glaceon-EX's "Crystal Ray" attack makes it immune to damage done by attacks from "Evolution Pokemon". This is a new term that means any Pokemon that is an evolution card. So a Stage 1 Frogadier brought to the Bench by "Water Duplicates" would be affected by this. Basic Pokemon and Restored Pokemon are not affected. (Fates Collide FAQ; May 5, 2016 TPCi Rules Team)

The cards updated all refer to "Evolution Pokémon", and so they should use the method pcs.realEvolution, instead of the current pcs.evolution. Cards from this era looking for "Basic Pokémon" all look for pcs.basic already, so no need for changes there.

I think I see. "Unevolved" didn't exist as a term in the EX era and earlier, so unevolved Pokémon were to be treated as "Basic" Pokémon, but the modern term used since the LvX era is "unevolved" instead, which frees Basic from having dual meaning? (Based off of searching "unevolved" in the different compendiums)

but the modern term used since the LvX era is "unevolved" instead, which frees Basic from having dual meaning?

Ahh, felt I was missing something. Didn't put any modern cards using "unevolved" as example, haha. But yeah, that'd be the key difference :)


Can't create cell on Safari when a previous cell is focused

Environment

Elixir & Erlang/OTP versions (elixir --version): 1.14.4 / 25
Operating system: Mac OS Ventura
How have you started Livebook (mix phx.server, livebook CLI, Docker, etc): livebook cli
Livebook version (use git rev-parse HEAD if running with mix): 0.9.3
Browsers that reproduce this bug (the more the merrier): Safari (worked fine in Chrome)
Include what is logged in the browser console: Nothing logged
Include what is logged to the server console: Nothing logged

Current behavior
If the focus is on a cell not visible anymore, due to scrolling, when clicking on  + Code, instead of creating a new cell, it simply goes back to the focused cell and doesn't create a new cell.
Here's a recording of the problem:
https://github.com/livebook-dev/livebook/assets/128147/26540397-e2d0-436d-8b02-6e343709ca7f
In this video I'm reproducing the error with some empty cells, but I've got this error already a few times in more real notebooks 😅
Expected behavior
Expected to behave the same in Chrome, to create a new cell and refocus on this new cell 💪
Thanks!

Seeing this one as well, on basically the same environment. It does not seem to matter how Livebook is started, it occurs when running the app or via mix phx.server.
One data point to hopefully help track this down - the upstream cell is not scrolled back to if you first click just outside the insert buttons (e.g. just below, between).

@awerment thanks for pinpointing the relevant vscode code :)
We can't do the workaround you suggested, because it effectively annihilates the if. The reason we need it in the first place, is that if a markdown cell is focused and you mouse-down on "+ Code", we want to keep editor focus (which includes re-focusing the Monaco editor). Otherwise (that is, if we exit focus mode) the markdown preview would collapse, the page would shift and the mouse-up would happen outside the button, so the click would be ignored.
Hopefully we can address this upstream, I opened an issue https://github.com/microsoft/monaco-editor/issues/4042 with a possible fix.

@jonatanklosko Thanks for looking into this and the explanation! So the "fix" would cause the same issue (clicks not registering because of the page shifting) in other cases, I see.


how do i reference my own data in train.py script?

i see the example_entry.py, and ive populated it with my own data.
how do i train on that file? im running python train.py but not sure what is next. Sorry if this is in the readme im not the best reader.

hi @basiclaser  if your own dataset is the same format described on the example_entry.py, you can just give the path of the dataset here PERSONACHAT_URL = "https://s3.amazonaws.com/datasets.huggingface.co/personachat/personachat_self_original.json"
instead of the persona dataset


[REEF-533] Add missing Javadoc comments in Java code: reef-utils

JIRA:
REEF-553
Pull Request:
Closes #

Second commit fixes link in commit message (JIRA number)

I made a pass with one comment.

I've updated PR

I'll test and merge.


Some Peer Feedback from Hold 11

The most elegant solution in the project was:

I think that if you attempt to make the code in Q4 part of the model class then it could become quite elegant. If you did this, the you should be able to just call the result from the class and then you would only need maybe 3 lines of code for Q4 in the notebook. But of course, if you have trouble loading, then maybe ignore this suggestion.

The hardest section of code in the project to understand was:

I just think in general you should clean up the project folder, don't think you need more than one notebook(.ipynb) and one py file(.py).

This part of the project could be better documented:

Overall I think you should only make comments and explanations in English and not in Danish. I don't think its permited to hand in in Danish as this course is in English. Luckily I speak Danish, otherwise some of the comments would not be understandable.
Q2 and Q4 could use more clarification.

An idea for an improvement/clarification could be:

I see that you are having you issues with loading, try calling the model like someting like this: "model = HouseholdSpecializationModel.HouseholdSpecializationModelClass()", i hope it works. otherwise its not a big problem, just not too nice to look at in the notebook.

An idea for an extension could be:

The output in Q1 could be added to a pandas dataframe, feel free to look at out code for Q1. Adding a dataframe should make it easier to see what is happening.

Hope you get the loading to work
Best wishes
Hold 11

A little follow up from Hold 11:
An idea for an extension before the exam could be to plot the results of Question 3, as the question ask you to do so ;))


Update SDK and Flutter Version Dependencies

Hi!
Would it be possible to update the dependencies of sdk and flutter version?
I updated and my project broke because of it.
Thank you
Sorry my bad english

same issue here.

It should be fixed now. Please let me know if you run into any issues.


Discriminator training and loss

Hello!
Thanks for your approach, in advance. I have some questions about your implementation.

Why are you training the discriminator since the very beginning? In MATLAB code it seems to begin discriminator training from epoch 15 on 256x256.
Have you trying to use nn.BCEWithLogitsLoss() loss for the adversarial, discriminator, real and fake loss?

I carefully looked at the training process of the source code, and it is exactly as you said. The reason I did the discriminator training at the beginning was that, in my experience, discriminator pretraining makes the discrimination loss more stable. But I haven't tried training with the discriminator loss, you can try to adjust the discriminator training start time later.

I have another question. Is there any reason you initialize weight in that way? I cannot find the way they initialize them in the original implementation. Thank you in advance.

In the supplementary material of the original paper, it is mentioned in the Additional Training Details chapter: We use He et al.’s method [15] to initialize the weights of our encoder and decoder conv layers, while the bias terms are initialized to zero.
[15] Kaiming He, Xiangyu Zhang, Shaoqing Ren, and Jian Sun. Delving deep into rectifiers: Surpassing human-level performance on ImageNet classification. In ICCV, 2015. 3


WIP: ESP32 AT WiFi Modem driver: modem: esp32 (Argon)

Work In Progress (Is functional)
Initial ESP32 AT WiFi Modem using WIFI_OFFLOAD
Including particle Argon config
Particle Argon does require ESP32 firmware rest instructions are available using Circuit Python [https://learn.adafruit.com/adding-a-wifi-co-processor-to-circuitpython-esp8266-esp32/program-particle-argon]
Has not been tested with IPv6, may need work to include

@mike-scott @pabigot
Thoughts or input as to improvements?
Do we know others working with Argon boards?

@WilliamGFish thanks for the ping.  I have Argon HW and give this a look.

In general, I'd like to see the driver migrated to MODEM_CONTEXT. That API should provide much better interfaces which:

reduce code size and allow for more re-usability
community can improve the generic parts over time
possibly use part of the socket API there
eventually power management hooks

my guess is that your ~1900 line driver file might be able to get cut in 1/2.

Mike,
I was just happy that it worked to start, I'll have a look at the MODEM_CONTEXT to see what I can do re. reducing code size and portability.
Any and all suggestions appreciated.

@WilliamGFish  Is it posssible for this driver to be compatible with esp8266? As esp32/esp8266 are from the same company. I don't have esp32 at hand, but lots of esp8266. If possible, we'd merge #18919 with this PR.

@WilliamGFish Is it posssible for this driver to be compatible with esp8266? As esp32/esp8266 are from the same company. I don't have esp32 at hand, but lots of esp8266. If possible, we'd merge #18919 with this PR.

There are a lot of similarities so it should be easy enough.
You should be able to change the AT+ commands as that is the only significant difference I can see (without making any detailed investigation).

@tsvehagen
Have you been working on the ESP8266?
I am about free up some time later this week, no need for both of us working on the same thing. The esp32 and 8266 are super similar with a few changes to AT commands.
Tell me know...
Thanks,
Billy..

@tsvehagen
Have you been working on the ESP8266?
I am about free up some time later this week, no need for both of us working on the same thing. The esp32 and 8266 are super similar with a few changes to AT commands.
Tell me know...
Thanks,
Billy..

Hey!
Yea I did some work to try and get something working with the MODEM_CONTEXT stuff that @mike-scott did. It's quite different from the original PR but I'll create a new PR for it tomorrow to see what people think. Maybe you can try it out with the esp32 as well.
Cheers

Hey!
Yea I did some work to try and get something working with the MODEM_CONTEXT stuff that @mike-scott did. It's quite different from the original PR but I'll create a new PR for it tomorrow to see what people think. Maybe you can try it out with the esp32 as well.
Cheers

I'll keep my eyes open for any updates.
Happy to help where I can; team effort hey...

Going to work with @tsvehagen using his rebuild driver to recreate the esp32 (Argon) version.
#21492
Billy..

Is this handled by drivers/wifi/esp

Is this handled by drivers/wifi/esp

Yes it is. Closing


Dynamic import of Astronomy

First I'd like to say that Astronomy is great! Thanks for the hard work.
I'm trying to get Astronomy loaded dynamically. Read the closed issue #642 and thought it would be possible.
Even a new meteor project loads astronomy to the client.
My test:
meteor create testAst
cd testAst
meteor add jagi:astronomy
meteor --extra-packages bundle-visualizer --production

Hmm in deed it bundles it no matter what.
I've made some tests and even with the new package it still bundles it, so probably there is no way I can leave Meteor packages out of bundle. It's probably only reserved for the NPM packages. However, if you know about any Atmosphere package that doesn't get bundled, then please let me know.

Ok I found out that apparently you have to add {lazy: true} to your main
module export. I will fix that on Monday
On Sat, Sep 15, 2018, 8:21 PM arggh<EMAIL_ADDRESS>wrote:

If I remember correctly, you could just have a bunch of js files inside
the Meteor package, which can be imported by the app code. If they're not
imported explicitly, they don't get bundled.
At least that's how I'm dynamically importing Blaze templates from a
package when needed.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jagi/meteor-astronomy/issues/683#issuecomment-421611267,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABAKXQ9cRq1X0iNr7DWs9KvFhOE5CttVks5ubUUagaJpZM4WEKVG
.

Oh good! Somehow I had completely missed that one.

Fixed in 2.6.0

Oh! This is great! Thank you for taking the time!
Sorry I missed the conversation. (Apparently I had disabled all github notifications...)


Update dataset_prepare.md

fix path & add an alternative way for downloading sync.zip because utils/download_from_gdrive.py is not working

:D Thanks a lot.


feat: add composition library for Go

This PR adds a composition library as a Go package, by running our already existing TypeScript composition inside
a JavaScript runtime. The process is entirely opaque for callers of this package, which accepts and returns Go
types.
Benchmark indicates that when running with the v8 backend, performance is as good as running the composition
algorithm in Node.

Motivation and Context
The composition library in TypeScript is not convention to run for clients that have all their tooling in Go. This wrapper
provides a turnkey solution that enables them to compose federated graphs from Go.

[x] Tests or benchmark included
[x] Documentation is changed or added on https://app.gitbook.com/
[x] PR title must follow conventional-commit-standard

@Aenimus This PR implies that every feature we develop in composition has to be tested with the go package and we can't use Node.js specific things anymore. Are you aware of this?


affected/package: html/template

What version of Go are you using (go version)?

$ go version
go version go1.21.0 darwin/arm64

Does this issue reproduce with the latest release?
Yes it does.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE='on'
GOARCH='arm64'
GOBIN=''
GOCACHE='/Users/jadegopher/Library/Caches/go-build'
GOENV='/Users/jadegopher/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='arm64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/jadegopher/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='darwin'
GOPATH='/Users/jadegopher/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/Users/jadegopher/sdk/go1.21.0'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/Users/jadegopher/sdk/go1.21.0/pkg/tool/darwin_arm64'
GOVCS=''
GOVERSION='go1.21.0'
GCCGO='gccgo'
AR='ar'
CC='clang'
CXX='clang++'
CGO_ENABLED='1'
GOMOD='/Users/jadegopher/GolandProjects/invoicer/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/kg/mc8vcsg52t58_bwffgndt1j80000gn/T/go-build2617316567=/tmp/go-build -gno-record-gcc-switches -fno-common'

What did you do?
When iI call pointer receiver function in template I get an error. If you make non pointer receiver function everything works fine
https://go.dev/play/p/fpnUIKIb0jh
What did you expect to see?
String "This is Bob" in stdout
What did you see instead?
2009/11/10 23:00:00 template: :1:2: executing "" at <.String>: can't evaluate field String in type main.Person

String uses a pointer receiver.
Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions


Hi there, this book is really great, but the site to read it is unavailable currently

What happen?

Hi @SeasonLeee  is it still the case ? The site looks up https://www.reactnative.guide/ at the moment.

@a7ul Hey, thanks for the response, I just checked, it works now!
I will close the issue~
And thanks for the great knowledge sharing project~


[Hufi-apikey] feat : add apikey to hufi job launcher

Description
Follow the same iteration for adding api-key to JobLauncher and adapt to Campaigns.
Summary of changes

Added endpoint to generate api-key.

 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.Mehdi seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hufi is going to have its own repo


Make perform_advanced_search filter by judgment collection

Changes in this PR:

Made perform_advanced_search filter by judgment collection
Added a test for perform_advanced_search

Trello card / Rollbar error (etc)
https://trello.com/c/T6x6GXmc/803-ensure-judgments-dont-display-press-summaries-erroneously
Note: I need to publish a new version of the api_client before merging this in. And before that I want to change the api_client to accept collections as a list os strings instead of a comma separated list.
Testing:
Have tested locally by changing collections=["judgment"] to collections="judgments" temporarily, pointing to staging with the current comma separated version of the search-v2.xqy that I copied onto staging (as marklogic repo not yet deployed) and my local copy of main of the apiclient which has the collections parameter as comma separated input too. "judgments" as thats the collection name I added a couple of test docs on staging to, but in practice when we deploy, I will migrate all current documents (judgments) to be added to "judgment" collection. Before that, we will make sure that the ingester adds  all new judgments to the judgment collection.
Once I have updated downstream I will update this PR appropriately

Closed in favour of: https://github.com/nationalarchives/ds-caselaw-public-ui/pull/738


Widget cartesian  update line  bugfix

fixes #61
and fixes misalignment of graph to axis.
there was a factor missing..

@FoamyGuy i just questioned myself if it makes more sens to change the nameing to line_add() and lines_clear()
this way it feels a little bit better ordered....
what you are thinking?

@s-light I do like the idea of adding "plot" to the names to make it distinct from the axis lines. How about add_plot_line() and clear_plot_lines()?

so you prefer the action first naming?
(i personally prefer object subobject action naming... this also help for simple autocomplete - all the function/attribut names start with the object name the belong to...)
i am fine with this. :-)


Docs: Remove useless random hex thingy

idk if this was meant to be here...

yes, it wasn't, thanks


[fealib] set AST element's location=None by default

I want to change the ufo2ft featureWriters API so that they will manipulate feaLib.ast.FeatureFile objects, instead of outputting strings.
Currently, the location is a required positional argument, and it's always the first; in addition, all the constructor arguments are implemented as required positional arguments.
This makes it cumbersome to create new ast elements from code, as one has to provide a None location as first argument, and then one has to pass all the default values for the subsequent positional arguments. The call becomes a very long, with lots of unnecessary Nones. For example, compare these two (old and proposed) methods to create a new ValueRecord object (e.g. for use in a PairPosStatement or kerning):
- value = ast.ValueRecord(None, False,
-                         None, None, -40, None
-                         None, None, None, None)
+ value = ast.ValueRecord(xAdvance=-40)

With this PR, I changed all the ast elements' constructors so that the location argument becomes optional and defaults to None.
Also, we can now make other arguments as kwargs with a default value, now that the first parameter is no longer the location.
This may break other's existing tools that rely on the implicit order of the positional arguments.
I believe it's a necessary break, to move things forward.
cc @mhosken @brawer @behdad

Nice.

refactoring is easy when you a have an extensive test suite (thanks Sascha! :+1:)

This breaks my open PR 😭 Time to get it in before it gets worse.


Render the segmentation as mesh

Impact: Render the segmentation layers as 3d surface mesh to make them more performant.
The current strategy for rendering the segmentation in neuroglancer is to render it as a image volume, as it is a binary mask, we will be able to optimize it by rendering it as a surface mesh.
Note: There are some known issues with neuroglancer rendering multiscale surface mesh. We should precompute a multiscale mesh to handle the issues.

The support for single-resolution mesh in neuroglancer is a legacy feature.
We would have to navigate some known pitfalls for the multiresolution mesh. There are code snippets available, that will help us with that, but might require a little poking around.


testlapack: use residualOrthogonal helper in Dorg2lTest and DgesvdTest

Please take a look.

Yes, I'll add it. There is still isOrthogonal that I want to replace.


Bug(scope): inlineLang (ru|static) doesn't work properly when numbers of messageFormat arguments differ for two langs

Is there an existing issue for this?

[X] I have searched the existing issues

Which Transloco package(s) are the source of the bug?
MessageFormat
Is this a regression?
No
Current behavior
When defaultLang is EN locale I'm trying to switch to the RU locale and get an error in the console because of a number of messageFormat arguments. If I delete 'few' and 'many' for the RU locale then there is no error
ERROR
Error: Invalid key `few` for argument `count`. Valid plural keys for this locale are `one`, `other`, and explicit keys like `=0`.

Expected behavior
We rely on inlineLang locale and get a proper combination of messageFormat arguments for validation
Please provide a link to a minimal reproduction of the bug
https://stackblitz.com/edit/transloco-example-nqxlnz?file=src/index.html
Transloco Config
prodMode: environment.production,
        availableLangs: [
          { id: 'en', label: 'English' },
          { id: 'ru', label: 'Russian' },
        ],
        reRenderOnLangChange: true,
        fallbackLang: 'ru',
        defaultLang: 'en',
        missingHandler: {
          useFallbackTranslation: true,
        },
        scopeMapping: {
          'transpilers/messageformat': 'mf',
        },

Please provide the environment you discovered this bug in
Transloco: 2.20.0
Angular: 12.1.3

Browser
Chrome Desktop Version 95.0.4638.69 (Official Build) (x86_64)

Additional context
No response
I would like to make a pull request for this bug
No

Hi! Have a similar issue with the fallback tool. When active lang and fallback lang have different plural forms, messageformat lib tries to compile plural string with active language config.
It happens, because, as I understand, plural config sets only while changing language (e.g. via setActiveLang method). And when we try to compile fallback string, lib tries to use active lang's config.
For example:
We have active korean lang, and then, because we don't have translation for some string, we need to use english fallback. But when we try to compile english fallback string, lib uses korean plural config, where there is no "one" form. Because of it, we have an error and then in some cases fallbacks don't work, because we have unhandled error and isResolvedMissingOnce flag always equals "true" (I cannot reproduce this, but have this problem in my app).
Reproduced behaviour with plural forms on stackblitz.


Does not honor /etc/hosts

There's currently no way to connect to "localhost" or any other host names listed in the local /etc/hosts file. We could provide a hard-coded default for some common host names (like "localhost"), but this is bound to break.
In particular, this also affects usage with linked Docker containers where one container references another container by its host name. This means that there's currently no way to connect to other containers by their name.
Opening this ticket as a reminder - this actually depends on https://github.com/reactphp/dns/issues/10

Isn't it possible to make the DNS dependency optional? I have a custom DNS server, and I don't want my application to be bound by its IP – this would mean headache for the maintainers of my code. :)

Isn't it possible to make the DNS dependency optional?

I agree with your suggestion, but it's probably unrelated to this ticket. Have you seen #12 yet? :)

Ah. What I meant was, making that dependency optional will fix this issue, as well. If I either create a non-resolving connector or pass in a NullResolver, then things in /etc/hosts would be resolved with no problems, right?

Also see #7. If you want to finish that PR to have a NullResolver as commented that could move this along.

Thanks a lot. Will see if I can wrap my head around it and finish it :-)

If I either create a non-resolving connector or pass in a NullResolver, then things in /etc/hosts would be resolved with no problems, right?

Not quite :) #12 (and #7 and #46) only deal with making the DNS resolver optional. This means that NO DNS lookups whatsoever will be performed and as such only IPs can be used.
This ticket here discusses introducing a new feature to take the hosts file into account so that the host names listed there will actually be resolved. This should probably still be part of the DNS component, so this depends on https://github.com/reactphp/dns/issues/10.

Closed in favor of https://github.com/reactphp/socket/issues/88 (Migrating over to Socket now that #86 is in)


[QUESTION] Are there plans to add content for new Stream on SharePoint?

WEB PART NOT LOADING: If you are getting the error: "Microsoft 365 learning pathways has a configuration issue. Ask your administrator for assistance". Please review the Diagnosing Pre-Check guidance before submitting your bug.

Submit your question
What is the context of your question

[ ] Installation
[x] Learning Pathways Content
[ ] SharePoint Success Content
[ ] Administration
[ ] Custom Content
[ ] Viewer Web Part

What is your question

Are there plans to add content for new Stream on SharePoint?

@jhwang-ms here is a request for additional content on Stream.

Hi @jhwang-ms , @dcashpeterson
We have implemented this feature for our client and we too are in need for MS Stream content. Request to push these changes at the earliest.

Hi @jhwang-ms
Do we have a date when it will be pushed? Request you to please add as a part of M365 Learning Pathways asap.


Check for block creation transactions fails in case of a fork

Since we enfore a minerfee except for the block creation txes, we must check for every transaction if it is indeed a block creation tx for the validation context. Part of this check is verifyinig if a blockstake respent is indeed as expected according to the header. However if there is a fork in the network, and the bs output which is being used is in the innactive fork (for this node), the check fails, since the header points to a block height (thus the active fork)

@LeeSmet What is the functional consequence of this?

A daemon which has the mentioned fix will reject all block creation transactions which reference a blockstakeoutput in an inactive fork, meaning as soon as there is a fork of more than one block, it might be rejected outright, causing network separation

Fixed in 352f22c25b371bc62fdec90cfcd0821e65cf30cf


Add Signin/Signup

Should have firebase auth signin and connect to firebase backend.

I will work on this!


Callback for when the presenter is available for use in the activity?

I am trying to call my presenter method when the activity is created. But it fails because the presenter is not initialised.
How do I handle situations like this? Is it a good idea to have a callback from PresenterActivity when the presenter is loaded?
Thanks
EDIT: Turns out due to the Presenter Loader concept, the presenter will be available onResume() of the activity.

For activities it is available in onStart() and in fragments it is available in onResume()
The callback you are looking for is onPresenterProvided() https://github.com/Plastix/Kotlin-Android-Boilerplate/blob/master/app/src/main/kotlin/io/github/plastix/kotlinboilerplate/ui/base/PresenterActivity.kt#L25


merge dev to main

feat: add support for comparing fields in the same model
fix: duplicated zod schema imported when there're multiple fields with the same enum type
fix: conditions hoisted from nested read overwrites top-level where conditions

Fixes #476 #632 #634


fix lint issues

There were some lint issues on my fork that I hadn't fixed up sorry.

@mbhall88 were you added to this repo? Seems like actions are only enabled to member of the repo?

(you might also want to sign up for github actions beta, if you haven't done so yet: https://github.com/features/actions)

(you might also want to sign up for github actions beta, if you haven't done so yet: https://github.com/features/actions)

Ah, I hadn't signed up for it. So maybe that was the issue?

I wonder if it has to do with the GitHub Actions beta, since everyone should have access:


Feature erase

Note: I have commented undo command from mousemove event because it was creating error.

Old erase.


How to reconnect properly?

I have made a hardware simulator script in Windows environment which should survive ADS errors and switching TwinCAT between config mode and run mode. Reconnection does not work by calling .open() and .close() or deleting the whole instance of Connection class and recreating it. Only way to get data moving again is to kill the whole script, but I don't want to have any external loop which restarts the process when the script exits. Is there any known bulletproof way to reconnect ADS?

Are you closing the connection on the side of the pyads client before you disconnect your TwinCAT host?

I investigated this a bit more and seems like the problem actually is that I stop getting callbacks after a while. Running the test code below with a local ADS server on my PC does not have any problems, but with an actual PLC pyads just stops calling the callback function after few seconds and I haven't found any way to restart it. Do you know what could cause this kind of behavior and how can I fix it?
import pyads
import time

AMS_ADDRESS         = 'x.xx.x.xxx.x.x'
AMS_PORT            = 851
VARIABLE_PATH       = 'some.variable'

plc = pyads.Connection(AMS_ADDRESS, AMS_PORT)
plc.open()

counter = 1
def callback(notification, data_name):
    global counter
    print('Callback called %d times' % counter)
    counter += 1

attr = pyads.NotificationAttrib(2,
                                trans_mode=pyads.ADSTRANS_SERVERCYCLE,
                                cycle_time=100 )

plc.add_device_notification(VARIABLE_PATH, attr, callback)

while(1):
    time.sleep(1)

I have the same issue. From my Linux PC I am reading and writing data on the PLC with the read_write command.
The problem is that if the connection is lost I am first getting "ADSError: timeout elapsed (1861)." and when the PLC is back on the network I can not reconnect to the PLC.
I am getting the error "ADSError: Unknown Error (-1).".
I tried to close the connection and reopen the connection, but this is not solving the issue.
Is there any other command that should be issued beside close() so that the connection is truly closed or reset?
Do I need to clear some other things?
Did somebody managed to reconnect successfully?
P.S.: If I simply restart the program everything is working, but I think that there should be a way to reconnect.

I managed to reconnect after a PLC restart/config mode. However without routing. So in case of a route to the plc some more steps are needed.
I think the point is to wait until ADS is started again (port open etc.).
What i do is calling the open() methode and check if the connection is up again by calling read_state(). If the read state fails i just try again with open() followed by read_state() after a few seconds an so on.
Once connected again remember to reasing the notifications since they are not valid anymore after a PLC restart.
Try to close all handles before the PLC shut's down if possible (state)
Below the code that worked in my case (without reasinging the notifications):
def __try_to_connect(self, trials, sleep_time_sec, callback, event):

        reconnected = False

        with tqdm(total=trials) as pbar:
            for trial in range(trials):
                if event.is_set():
                    print("close down event is set")
                    return
                time.sleep(sleep_time_sec)

                try:
                    self.plc.open()
                    state = self.plc.read_state()
                    if state[0] != 0:
                        reconnected = True
                        break
                except Exception as e:
                    pass

                pbar.update(1)

        callback(reconnected)

Maybe it would make sense to handle reconnection within plc.open() method by adding a retries parameter.

I think that would be a nice option. Anyway it would be nice to get a feedback whether the plc.open() fuction successfully opened the connection or not. This would make cleare that this function call maybe can't open the connection and makes it easy to react upon the outcome.
I thought instead of a read_state() an easier way of checking the connection could be checking the returned port number but that seems not to be a valid indicator since as long as a message router is present it will return a valid port nummer.


Fix the CI for eslint changes

Re: #39210 - perhaps the order of the args is important.

Come to the conclusion it's probably not my PR's fault - likely due to this update to typescript-eslint https://github.com/typescript-eslint/typescript-eslint/releases/tag/v3.6.0