Scraping Ettiquette

After some back and forth with support, it seems that we can only make on request at a time. No parallelism. Also no more than 300 requests a minute.
After meeting these constraints, the script is somewhat slower, but does not seem to get blocked.
We can now make requests until the quota runs out without getting banned. It does take a few minutes, but that seems to be unavoidable if we want to play by GitHub's rules.

Conflicts?

Rebased.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


First commit for Retini App v1.0.5

Retini is a super simple retina converter, converts your @3x and @2x retina files to @2x and 1x files.

Thank you. Could you please squash your commits?

Thank you. Merged in https://github.com/caskroom/homebrew-cask/commit/dffa4be8a180023eaca47da988938c9e34614117 with fixes.


Update Composer dependencies (2020-11-19-00-05)

Visual regression test passed!
View the visual regression test report


How to export preferences?

Description
Not a bug, but more a question on how to.
I have an DBeaver installation where I configured the preferences the way I prefer.
Question:
How can I export my preferences and import the into a different DBeaver installation.
In my specific case I have configured DBeaver on Mac and want the same settings on a Linux workstation.
DBeaver Version
24.,1.

Here is an article about it https://github.com/dbeaver/dbeaver/wiki/Admin-Manage-Preferences#importing-and-exporting-workspace-configuration

I would like to mention that I recently moved from one laptop to another, and I had issues with the mentioned preference export and import method. I did not research what was wrong in my case, but highly recommend to manually export the whole workspace if you can. I had to manually update couple of preference files like UI colors, templates and saved data format profiles.

@TB-HunBoka
So do I. Some preferences has been lost in my case.

@E1izabeth  It is also unclear for a user what to choose (if user has not yet read your link):
General -> Preferences
or
DBeaver Preferences

And this issue #34804 with non-reproducible bug with my exported settings shows it eloquently.

Thank you for feedback regarding this feature.
We need to identify which preferences don't come during import-export and discuss the confusion between General -> Preferences
and DBeaver -> Preferences.

@E1izabeth

set these options (all false), non standard font size (14) and color (green):

export preferences to zip-file (File - Export - Preferences)

on another computer - import preferences from this zip file

Expected: options changed as on screenshot. the font changed to green and size is bigger.
Actual: options remain unchanged. The font remains default.

@ask9 thank you for investigation!

We need to ... discuss the confusion between General -> Preferences
and DBeaver -> Preferences.

Here are a few questions to start for the further discussion when the time comes:

Should user export both options or only one is enough?
What's the difference between them? Which is preferable?
...


Define the VPC of the serverless functions deployed as part of Next js deployment

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting questions.
[X] I have removed any sensitive information from my code snippets and submission.

App Id
NEW_APP
AWS Region
us-east-2
Amplify Hosting feature
SSR
Question
I want to define the vpc of the serverless functions created as part of the Next.js deployment so that they access other resources defined in a particular vpc.
I couldn't find any documentation on how to define the functions created as part of the next js deployment.
As Next.js is moving towards a database integrated in the framework approach, I think this feature is much needed.

Hi @mekapotulas thank you for opening this issue. I've marked it as a feature request and will follow up with any additional information.

+1. I have seen many comments on Discord regarding the same issue. Setting up NextJS API routes is much easier than using AWS Amplify APIs.


profileCardResponse handling issues

Since it seems that the profileCardResponse code was copy pasted from profileResponse, it technically handles it improperly.  It still works since it passes the result along with the undefined error value, but there is no result field, and the debug messages weren't even changed.   You can request any account_id up to 2^32-1 but if it's not a valid profile the result will always be:
{
  "account_id": (input),
  "background_def_index": null,
  "slots": [],
  "badge_points": null,
  "event_points": null,
  "event_id": null
}

Can you submit a patch to fix?  Otherwise @Crazy-Duck might be able to, or I might take a look over the weekend

I can try but i'll have to figure out exactly how to determine what a "bad profile card response" is supposed to be first.

Hmm, actually it's quite possible that this is the intended behavior.  We just decode the response with the appropriate protobuf definition.  I assume if the given account_id was invalid the protobuf will not crash, but the fields will not be populated with valid data.  The client application is probably supposed to detect and handle the error (ignoring the fields).
Are the debug messages you're referring to part of node-dota2?

Upon requesting a non-number account_id I was presented with this:
Illegal value for Message.Field .CMsgClientToGCGetProfileCard.account_id: NaN (TypeError: Illegal value: NaN (not an integer))
The debug messages are part of the onProfileCardResponse handlker in community.js
Unrelated, the tab spacing is inconsistent.  Are you using 2 or 4 spaces for tabs?

I would just implement a NaN check at your application level rather than embed that into node-dota2.  I'm pretty sure that error is thrown at the protobufjs level and not at node-dota2, so I don't think it's very clean to implement error handling for that (to be consistent we would have to implement it for every single method that takes an id).
Yeah, looks like @Crazy-Duck didn't update the debug strings, but that's pretty easily fixed.
I have no idea about the spacing.  Probably different people are using different values and that's leading to inconsistency.

Here we go #139

I pretty much wrote this method in 5 min just so we'd have a fix as soon as possible when Valve nuked the regular profileRequest. It doesn't surprise me I forgot something :)

It is your code that was implemented as the fix, so If there are further issues feel free to re-open.


Controllers action not working as expected

With last npm version of footprint (beta 3) and router (beta 4) there is a problem :

Whatever footprint.controllers value, there are created (it's not visible if footprint.prefix is empty because routes can't be duplicate)

Example: prefix: '', controllers:false  => routes.length = 18
prefix: '', controllers:true  => routes.length = 18
prefix: '/api/v1', controllers:false  => routes.length = 24
prefix: '/api/v1', controllers:true  => routes.length = 24

I just see the code of the trailpack and footprint config has changed now config.footprint.controllers is an object. My PR add the ability to set config.footprint.controllers : false

Also for security reason the default behavior should be false, if you are ok I can update my PR this way. If not don't hesitate to merge it ^^


[Bug]: ValueError: x and y must have same first dimension edge case in global time_series

What happened?
See https://github.com/E3SM-Project/zppy/pull/400#issuecomment-2048589364 -- there appears to be an edge case where dimensions differ, causing global_time_series plots to not display.
What machine were you running on?
Chrysalis
Environment
zppy dev environment, while working on #400
What command did you run?
zppy -c tests/integration/post.v3.LR.piControl.cfg

Copy your cfg file
[default]
input = /lcrc/group/e3sm2/ac.golaz/E3SMv3/v3.LR.piControl
output = /lcrc/group/e3sm/ac.forsyth2/zppy_test_v3_output/v3/v3.LR.piControl
case = v3.LR.piControl
www = /lcrc/group/e3sm/public_html/diagnostic_output/ac.forsyth2/zppy_test_v3_www/v3
partition = compute
environment_commands = "source /lcrc/soft/climate/e3sm-unified/load_latest_e3sm_unified_chrysalis.sh"
campaign = "water_cycle"

[ts]
active = True
environment_commands = "source /home/ac.forsyth2/miniconda3/etc/profile.d/conda.sh; conda activate zppy_dev_with_nco_20240405"
years = "0001:0050:10",
walltime = "00:50:00"

  [[ atm_monthly_glb ]]
  input_subdir = "archive/atm/hist"
  input_files = "eam.h0"
  frequency = "monthly"
  mapping_file = "glb"

  [[ lnd_monthly_glb ]]
  input_subdir = "archive/lnd/hist"
  input_files = "elm.h0"
  frequency = "monthly"
  mapping_file = "glb"
  vars = "FSH,RH2M,LAISHA,LAISUN,QINTR,QOVER,QRUNOFF,QSOIL,QVEGE,QVEGT,SOILWATER_10CM,TSA,H2OSNO,TOTLITC,CWDC,SOIL1C,SOIL2C,SOIL3C,SOIL4C,WOOD_HARVESTC,TOTVEGC,NBP,GPP,AR,HR"

[global_time_series]
active = True
experiment_name = "v3.LR.piControl"
figstr = "v3.LR.piControl"
atmosphere_only = True
#plots_original = "net_toa_flux_restom,global_surface_air_temperature,toa_radiation,net_atm_energy_imbalance,change_ohc,max_moc,change_sea_level,net_atm_water_imbalance"
plots_lnd = "FSH,RH2M,LAISHA,LAISUN,QINTR,QOVER,QRUNOFF,QSOIL,QVEGE,QVEGT,SOILWATER_10CM,TSA,H2OSNO,TOTLITC,CWDC,SOIL1C,SOIL2C,SOIL3C,SOIL4C,WOOD_HARVESTC,TOTVEGC,NBP,GPP,AR,HR"
ts_num_years = 10
walltime = "00:30:00"
years = "1-50",
climo_years ="1-50",
ts_years ="1-50",

What jobs are failing?
No response
What stack trace are you encountering?
ValueError: x and y must have same first dimension, but have shapes (50,) and (51,)

Hi @chengzhuzhang I'm on CEST time so I just now read that there's a time axis issue. It would be helpful if you pointed me to the location of the ELM raw input file(s) that have time_bounds = [-0.0208333333333333, 31] and of the regional average timeseries files that NCO generates from that raw input file. I'll verify NCO whether just copies the time bounds and passes it through, or whether NCO creates the problem. Are you also saying that NCO creates a bad global average from that file? Or is the problem that something downstream does not know how to interpret the negative time bounds? Or...?

OK, I found the first ELM file in the PI control simulation. It has a negative time_bounds. That is legal. Below I show the bounds and the dates to which it corresponds:
(e3sm_unified_1.9.3_login) ac.zender@chrlogin1:/lcrc/group/e3sm2/ac.golaz/E3SMv3/v3.LR.piControl/archive/lnd/hist$ ncks -v time v3.LR.piControl.elm.h0.0001-01.nc | m
netcdf  v3.LR.piControl.elm.h0.0001-01 {
 dimensions:
  hist_interval = 2 ;
  time = UNLIMITED ; // (1 currently)

 variables:
  float time(time) ;
   time:long_name = "time" ;
   time:units = "days since 0001-01-01 00:00:00" ;
   time:calendar = "noleap" ;
   time:bounds = "time_bounds" ;

  double time_bounds(time,hist_interval) ;
   time_bounds:long_name = "history time interval endpoints" ;

 data:
  time = 31 ;

  time_bounds =
  -0.0208333333333333, 31 ;

} // group /
(e3sm_unified_1.9.3_login) ac.zender@chrlogin1:/lcrc/group/e3sm2/ac.golaz/E3SMv3/v3.LR.piControl/archive/lnd/hist$ ncks -v time --cal v3.LR.piControl.elm.h0.0001-01.nc
netcdf v3.LR.piControl.elm.h0.0001-01 {
  dimensions:
    hist_interval = 2 ;
    time = UNLIMITED ; // (1 currently)

  variables:
    float time(time) ;
      time:long_name = "time" ;
      time:units = "days since 0001-01-01 00:00:00" ;
      time:calendar = "noleap" ;
      time:bounds = "time_bounds" ;

    double time_bounds(time,hist_interval) ;
      time_bounds:long_name = "history time interval endpoints" ;

  data:
    time = "0001-01-31 23:32:16" ;

    time_bounds =
    "0000-12-31 23:30:00", "0001-02-01" ;

} // group /

So this appears to be a corner case that is due to the model. There is no reason I can think of why a monthly mean file should be written with an initial time corresponding to 11:30 PM of the last day of the previous month. Is there something that you want ncclimo to do when it creates timeseries of files that include such behavior? LMK. One option is to continue to treat this file as a valid January monthly mean and ignore the 30 minutes that apparently belong in the previous month. Before treating it some other way, land model people should first confirm that the 30 minutes are "real", and not some typo/bug in the model that wrote the file. Pinging @thorntonpe for comment.

I ran into the same issue when testing zppy on BGCv2 coupled simulation.
@thorntonpe , could you comment on this issue? The script I'm testing will be passed along to Xiaoying for V3 evaluations.

@czender @forsyth2 @chengzhuzhang  , I'm a little bit puzzled by this error. In my case, global_time_series was able to plot atm variables but not land variables. According to my investigation, both atm and lnd ts has the same length in terms of time dimension but with some minor differences such as the name of attributes and the variables time_bounds time_bnds. For example,
netcdf TSA_185001_201412 {
dimensions:
        rgn = 3 ;
        time = UNLIMITED ; // (1980 currently)
        lat = 360 ;
        lon = 720 ;
        rgn_len = 19 ;
        hist_interval = 2 ;
variables:
        char region_name(rgn, rgn_len) ;
                region_name:long_name = "TSA timeseries array contains area-weighted averages over these regions" ;
        float TSA(time, rgn) ;
                TSA:cell_methods = "time: mean area: mean" ;
                TSA:coordinates = "region_name" ;
                TSA:long_name = "2m air temperature" ;
                TSA:missing_value = 1.e+36f ;
                TSA:units = "K" ;

netcdf FLNS_185001_201412 {
dimensions:
        rgn = 3 ;
        time = UNLIMITED ; // (1980 currently)
        ncol = 21600 ;
        rgn_len = 19 ;
        nbnd = 2 ;
variables:
        char region_name(rgn, rgn_len) ;
                region_name:long_name = "FLNS timeseries array contains area-weighted averages over these regions" ;
        float FLNS(time, rgn) ;
                FLNS:Sampling_Sequence = "rad_lwsw" ;
                FLNS:cell_methods = "time: mean area: mean" ;
                FLNS:coordinates = "region_name" ;
                FLNS:long_name = "Net longwave flux at surface" ;
                FLNS:missing_value = 1.e+20f ;
                FLNS:units = "W/m2" ;

Since global_time_series produced atm plots, my initial thoughts is to manipulate those minor differences in land variables to match atm variables.  However, the same x, y errors appear....
lnd ts files were generated and placed in /lcrc/group/e3sm2/ac.sfeng1/E3SM_Simulations/20240315.v2.LR.BGC-LNDATM.CONTRL.ne30pg2_r05_EC30to60E2r2.chrysalis/zppy/post/lnd/glb/ts/monthly/165yr.
atm ts files are in
/lcrc/group/e3sm2/ac.sfeng1/E3SM_Simulations/20240315.v2.LR.BGC-LNDATM.CONTRL.ne30pg2_r05_EC30to60E2r2.chrysalis/zppy/post/atm/glb/ts/monthly/165yr
the test script is /lcrc/group/e3sm2/ac.sfeng1/E3SM_Simulations/20240315.v2.LR.BGC-LNDATM.CONTRL.ne30pg2_r05_EC30to60E2r2.chrysalis/zppy/post/scripts/global_time_series_1850-2014.bash

The attribute names do not make a difference. My impression is that the plotting routine either does not like the -'ve value of the first timestep in ELM time_bounds variable, or it does not like the fact that it translates to 11:30 PM the evening before the first day of the month. Not sure which.


style: Grammar mistakes in README file

While reading the README file, i noticed som grammar mistakes, which if corrected, would enhance the readability of the README.md file.
It would also help others to better understand and read the procedures to joining our community. Screenshots below

This should be "a small" or "a little"

Punctuation marks are missing in this sections. And a few word choices, could be corrected.

-- kindly work on this issue and fix this

Alright. Will do

my pull fixes most of the english, except for the camo image

Alright, I'll review for you


build server

Hi,
Is there a third party build that you can setup for this? We have been using TravisCI for the other RSocket projects.

Hi,
In original repo, it used TravisCI for CI/DI. But I removed it because lack of unit test cases.
I've added a basic .travis.yml into repo. Please open TravisCI for it. Thanks!
// Jeffsky // ʕ•̫͡•ʔ-̫͡-ʕ•͓͡•ʔ-̫͡-ʕ•̫͡•ʔ
Wechat: jeffsky1024
Email<EMAIL_ADDRESS>On Apr 17, 2019, 04:24 +0800, Robert Roeser<EMAIL_ADDRESS>wrote:
Hi,
Is there a third party build that you can setup for this? We have been using TravisCI for the other RSocket projects.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/rsocket/rsocket-go/issues/18, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABObHFxxLOaWpse9DhmIJOMXjL1BaTK_ks5vhjFHgaJpZM4czok-.


Adapt tests to current .eslintrc

Just downloaded it to explore a bit, and tests failed out of the box, so I fixed the eslint warnings first:

Un-nesting the "done" callback was trivial. :-)
Closing the proxy when its upstream server goes away seems sensible independent of whether the GET request succeeded.
Hungarianized some var names.

What may be next:

The "use" and "ignore" proxy tests look mostly the same; should I factor out the shared code in another PR?
I needed to run npm test with http_proxy=  in front (to unset the env var) or it would fail with Uncaught Error: listen EADDRINUSE :::8080. Shall I make a PR that checks for this scenario and advises to unset the proxy config?
Before I discovered the proxy reset requirement, I thought I had my own proxy on port 8080, because some of my machines do use that port. While debugging it, I found lots of hard-coded port numbers in the tests. Shall I replace them with variables, keeping the old ports as default but making it easy to override test ports by env vars? (e.g. TEST_PORTS=4401,4402,4403 npm test)

 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.

Thank you very much for your work @mk-pmb 👍
I also think that the tests could be written in a better way (e.g. find a free port instead of the hardcoded ones), but imo it is quite okay right now.


added description of Blue/Green and Canary

@theetrain updated!

@p0pr0ck5 comments addressed

@p0pr0ck5 updated!


[SPARK-39859][SQL] Support v2 DESCRIBE TABLE EXTENDED for columns

What changes were proposed in this pull request?
Support v2 DESCRIBE TABLE EXTENDED for columns
Why are the changes needed?
DS v1/v2 command parity
Does this PR introduce any user-facing change?
No
How was this patch tested?
UT

Merged to master/3.4 for Apache Spark 3.4.0.
cc @MaxGekk since he filed SPARK-39859 originally.

Thanks @dongjoon-hyun @viirya


Filter Handler and Filter Interface

Right now there is a CsvFilter and MapFilter handler.
I've got an idea that this could be removed completly by a Standalone Filter handler.
The FilterHandler would accept Payloads just as a regular Handler.
Then check if that Payload belongs to Interface Filterable.
If it does, then run Filter on it.
Filter would need 3 items

FilterGroup (naming is bad, but sort of like....context or so, )
A filter list or Directory of filters that have topic->Regexp

This is now done
As part of the payload package there is a Filter and Filterable interface.
This is usable by any PayloadType that has a ApplyFilter(f Filter) bool function.
See CsvPayload for example.


Add slither setup to the project

I have set up slither to the project, and also updated the README accordingly.

Hi @haltakov, I have completed the requests, please have a look :)

@asep-fajar-firmansyah - thanks! I fixed two small things and will merge it now :) You can take a look at my commits to understand what I changed.


Add a button for the first deliberation stage

Add a flag for submit button that lets us turn on or off the stage advance behavior.

Addressed with https://github.com/Watts-Lab/deliberation-empirica/issues/374


How to select models in Multi-AI Ensemble for best processing results

This is my current option, but I find that the audio quality after processing and merging is not as good as the audio quality of these different models when processed separately

So I want to know how these models should be matched to maximize the quality in the usual songs

The key is to find the ensemble that works best for the tracks being processed. It's hard to recommend a best ensemble overall. Personally, I don't ensemble with more than 2 models unless I really need to.
Check out the new update! New ensembling options have been added.

okey, thx

This is my current option, but I find that the audio quality after processing and merging is not as good as the audio quality of these different models when processed separately

So I want to know how these models should be matched to maximize the quality in the usual songs

Demucs v3 (md_extra model) seems to provide the best results. Sometimes, however, the vocals are not properly processed with it. When that happens, I use Ensemble mode with the following settings for better vocals extraction:


test deployed contract code against runtimeBinary

Ugh, tests are failing for some ungodly reason. I'm betting a dependency got bumped.

Otherwise looks great, and is a good addition.

Merged master in, tests pass. Thanks!


hackney 1.12.1

for adjust version with bamboo
https://github.com/thoughtbot/bamboo/blob/6f10c5887a58ec7c3ace8b1fa8fa6fb3d9bf5c1a/mix.exs#L64

close
because 0.9.2 use ~> 1.7


Can't access dns server with 'host' command after launching

Hi, after running the container following the tutorial, the host command fails to resolve 'www.google.com'. Also I get an ominous
WARNING: Localhost DNS setting (--dns=<IP_ADDRESS>) may fail in containers.
...warning which is not mentioned in the tutorial.
This is how I run the container:
docker run -d --name=bind --dns=<IP_ADDRESS> \
  --publish=<IP_ADDRESS>:53:53/udp --publish=<IP_ADDRESS>:10000:10000 \
  --volume=/srv/docker/bind:/root/data/bind \
  --env='ROOT_PASSWORD=SecretPassword' \
  sameersbn/bind:latest

This is my version and info for docker:
root@photon-rps-client [ ~ ]# docker version
Client:
 Version:      1.11.0
 API version:  1.23
 Go version:   go1.5.4
 Git commit:   4dc5990
 Built:        Wed Apr 13 19:36:04 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.11.0
 API version:  1.23
 Go version:   go1.5.4
 Git commit:   4dc5990
 Built:        Wed Apr 13 19:36:04 2016
 OS/Arch:      linux/amd64

root@photon-rps-client [ ~ ]# docker info
Containers: 1
 Running: 1
 Paused: 0
 Stopped: 0
Images: 1
Server Version: 1.11.0
Storage Driver: overlay
 Backing Filesystem: extfs
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge null host
Kernel Version: 4.4.8
Operating System: VMware Photon/Linux
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 974.9 MiB
Name: photon-rps-client
ID: CEG3:K7XA:UMDZ:HXR4:YEMV:OOIJ:6HIT:4VEJ:JDMM:YLP4:E3EO:KH5V
Docker Root Dir: /var/lib/docker
Debug mode (client): false
Debug mode (server): false
Registry: https://index.docker.io/v1/

I am running this on Photon OS on Fusion on my Mac:
VMware Photon Linux 1.0
PHOTON_BUILD_NUMBER=13c08b6

@necro351
Can you test that the DNS server is working by SSH'ing into the VM and running docker exec -it bind host www.google.com <IP_ADDRESS>. If it works, then it probably means that you need to open the 53/UDP port on the VM machine in fusion and then use the VM ip address to query the DNS server from the OSX host.

WARNING: Localhost DNS setting (--dns=<IP_ADDRESS>) may fail in containers.

this message is new, never noticed it before. but you can ignore it as we know it'll work if the BIND server running inside the container works :wink:

There is a similar container that does not have these issues for me at least, perhaps there are some changes that you can cross-patch? It's here:
https://hub.docker.com/r/cosmicq/docker-bind/
...closing this. Thanks.


Regression bug: bookshelf, publication info => crash (white window background)

[2]   readium-desktop:sync ### action type API_REQUEST +5s
[2]   readium-desktop:main#redux/sagas/api publication get [] +5s
[2]   readium-desktop:main#redux/sagas/api NotFoundError { message: 'document not found' } +1ms
[2]   readium-desktop:sync ### action type API_ERROR +1ms
[2]   readium-desktop:renderer:bookshelf_ The above error occurred in the <PublicationInfo> component:
[2]   readium-desktop:renderer:bookshelf_     in PublicationInfo (created by withApi(PublicationInfo))
[2]   readium-desktop:renderer:bookshelf_     in withApi(PublicationInfo) (created by ConnectFunction)
[2]   readium-desktop:renderer:bookshelf_     in ConnectFunction (created by withTranslator(Connect(withApi(PublicationInfo))))
[2]   readium-desktop:renderer:bookshelf_     in withTranslator(Connect(withApi(PublicationInfo))) (created by DialogManager)
[2]   readium-desktop:renderer:bookshelf_     in div (created by Dialog)
[2]   readium-desktop:renderer:bookshelf_     in div (created by Dialog)
[2]   readium-desktop:renderer:bookshelf_     in div (created by FocusLock)
[2]   readium-desktop:renderer:bookshelf_     in FocusLock (created by FocusLockCombination)
[2]   readium-desktop:renderer:bookshelf_     in FocusLockCombination (created by Dialog)
[2]   readium-desktop:renderer:bookshelf_     in Dialog (created by withTranslator(Dialog))
[2]   readium-desktop:renderer:bookshelf_     in withTranslator(Dialog) (created by DialogManager)
[2]   readium-desktop:renderer:bookshelf_     in DialogManager (created by ConnectFunction)
[2]   readium-desktop:renderer:bookshelf_     in ConnectFunction (created by Dropzone)
[2]   readium-desktop:renderer:bookshelf_     in div (created by Dropzone)
[2]   readium-desktop:renderer:bookshelf_     in Dropzone (created by App)
[2]   readium-desktop:renderer:bookshelf_     in div (created by App)
[2]   readium-desktop:renderer:bookshelf_     in Router (created by ConnectedRouter)
[2]   readium-desktop:renderer:bookshelf_     in ConnectedRouter (created by Context.Consumer)
[2]   readium-desktop:renderer:bookshelf_     in ConnectedRouterWithContext (created by ConnectFunction)
[2]   readium-desktop:renderer:bookshelf_     in ConnectFunction (created by App)
[2]   readium-desktop:renderer:bookshelf_     in Provider (created by App)
[2]   readium-desktop:renderer:bookshelf_     in App

The bug disappears if I remove the spread operator in:
https://github.com/readium/readium-desktop/blob/3c2229015ab50e3926dfaae998fba27e100dfe01/src/renderer/components/utils/hoc/api.tsx#L59-L61
See:
https://github.com/readium/readium-desktop/commit/3c2229015ab50e3926dfaae998fba27e100dfe01#diff-5eb764dcd42b75d25b25c22a7aa1bc66
PR:
https://github.com/readium/readium-desktop/pull/710

PS: same in reader view(s) (not just bookshelf / library view)


Tiny Icons on Mobile, and Suggestion to Hide Side Bar.

As the title says, the icons in the side bar appears tiny on mobile (except Containers are normal). And please add an option to hide/collapse the side bar and title bar, since they take up too much real estate on a small screen or small window. Also, in the horizontal orientation on mobile manually scrolling the page below the title doesn't show more of the map, but instead shows a grey background, the same happens in a small PC window when scrolling right, though the map portion of the screen can still be scrolled. There's also an issue of only being able to highlight a region while the mouse is hovered over it, mobile is able to tap it and it stays highlighted, which I think would be preferable. And, I assume you're working on re-adding the Green Containers; I'm not sure, but I think Gold are just RNG from previous Green ones. BTW, besides these issues, I like the new redesign.

As the title says, the icons in the side bar appears tiny on mobile (except Containers are normal)
Unfortunately, the map does not have an adaptive layout yet.

And please add an option to hide/collapse the side bar and title bar, since they take up too much real estate on a small screen or small window.
I have plans to do it, but only for side bar.

Also, in the horizontal orientation on mobile manually scrolling the page below the title doesn't show more of the map, but instead shows a grey background, the same happens in a small PC window when scrolling right, though the map portion of the screen can still be scrolled.
It will be fixed today or tomorrow. Thanks for report.

There's also an issue of only being able to highlight a region while the mouse is hovered over it, mobile is able to tap it and it stays highlighted, which I think would be preferable.
I was planning to have the region section highlighted when pointing and when clicking on the section an information window would pop up with information about the section (like as Urgent Quests and Towers)

And, I assume you're working on re-adding the Green Containers; I'm not sure, but I think Gold are just RNG from previous Green ones.
Yeap. Later i'll add back green (white) and gold containers. I already have a system for automatic change of markers depending on the day of the week.

BTW, besides these issues, I like the new redesign.
Thanks you very much =)


[BUG]: Drizzle MySql crashes if excess properties are present in the insert for a single value

What version of drizzle-orm are you using?
0.28.2
What version of drizzle-kit are you using?
0.19.12
Describe the Bug
Probably related to #362
The problem is quite simple - they way Drizzle maps value keys to columns in MySQL crashes if there's no corresponding column in the provided mapping.
const TEST_TABLE = mysqlTable("TEST", { value: int("value") });
// ERRORS because TS does excess property checking on object literals.
void db.insert(TEST_TABLE).values({ value: 5, rootOfAllEvil: -1 });
const EvilValue = { value: 5, rootOfAllEvil: -1 };
// DOESN'T because TS doesn't do it on anything but object literals.
void db.insert(TEST_TABLE).values(EvilValue);

This is what causes the problem. colEntries elements will have a value of [string, undefined] if object has an excess property. And this will crash because of undefined.name. I believe it should just be
const colEntries: [string, AnyMySqlColumn][] =  Object.entries(columns);

It looks like this problem was removed for SQLite and ps-core but not for MySQL.
I would be very glad to make a PR.
Expected behavior
Excess properties should just be ignored.
Environment & setup
No response

Closed in favor of #572


Fix ValueError: zero length field name in format on Python 2.6

This small fix make python-etcd master branch compatible with Python 2.6.

Coverage decreased (-0.3%) to 87.963% when pulling 6243cacd34903c16e2c61dc25e48889492f53c02 on victorpoluceno:fix-value-error-on-python2.6 into 3d061310094062a479ec0bfe69ca04471332e2d9 on jplana:master.

Looks good to me! 👍

Python 2.6 is not supported even for security fixes at this time.  Maybe it's time to move on?


Wording on Secure Webhook

"As any azure customer can access all object IDs through portal, without checking the owner, anyone can add the secure webhook to their own action group for azure monitor alert notification which violate security."
seems to imply that anyone can call the secure webhook randomly without providing any token whatsoever
quite the opposite is true and intended and the wording may need to be adjusted

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6b4942cb-cb22-7ef0-d78c-c5967a879d2d
Version Independent ID: 6eb84072-fa26-7434-5376-9c5a901cedff
Content: Create and manage action groups in the Azure portal - Azure Monitor
Content Source: articles/azure-monitor/alerts/action-groups.md
Service: azure-monitor
Sub-service: alerts
GitHub Login: @dkamstra
Microsoft Alias: dukek

@woutervanranst Thanks for the feedback.
Assigning content owner @dkamstra for visibility, review and document enhancement consideration as appropriate.

#please-close


ImportError: cannot import name 'weakref'

any comment?
➜  baselines git:(master) python baselines/pposgd/run_atari.py
Traceback (most recent call last):
  File "baselines/pposgd/run_atari.py", line 54, in <module>
    main()
  File "baselines/pposgd/run_atari.py", line 51, in main
    train('PongNoFrameskip-v4', num_timesteps=40e6, seed=0, num_cpu=8)
  File "baselines/pposgd/run_atari.py", line 18, in train
    from baselines.pposgd import pposgd_simple, cnn_policy
  File "/Users/Tiger/projects/baselines/baselines/pposgd/pposgd_simple.py", line 3, in <module>
    import baselines.common.tf_util as U
  File "/Users/Tiger/projects/baselines/baselines/common/tf_util.py", line 2, in <module>
    import tensorflow as tf  # pylint: ignore-module
  File "/Users/Tiger/anaconda/lib/python3.5/site-packages/tensorflow/__init__.py", line 24, in <module>
    from tensorflow.python import *
  File "/Users/Tiger/anaconda/lib/python3.5/site-packages/tensorflow/python/__init__.py", line 63, in <module>
    from tensorflow.python.framework.framework_lib import *
  File "/Users/Tiger/anaconda/lib/python3.5/site-packages/tensorflow/python/framework/framework_lib.py", line 100, in <module>
    from tensorflow.python.framework.subscribe import subscribe
  File "/Users/Tiger/anaconda/lib/python3.5/site-packages/tensorflow/python/framework/subscribe.py", line 26, in <module>
    from tensorflow.python.ops import variables
  File "/Users/Tiger/anaconda/lib/python3.5/site-packages/tensorflow/python/ops/variables.py", line 26, in <module>
    from tensorflow.python.ops import control_flow_ops
  File "/Users/Tiger/anaconda/lib/python3.5/site-packages/tensorflow/python/ops/control_flow_ops.py", line 70, in <module>
    from tensorflow.python.ops import tensor_array_ops
  File "/Users/Tiger/anaconda/lib/python3.5/site-packages/tensorflow/python/ops/tensor_array_ops.py", line 33, in <module>
    from tensorflow.python.util import tf_should_use
  File "/Users/Tiger/anaconda/lib/python3.5/site-packages/tensorflow/python/util/tf_should_use.py", line 28, in <module>
    from backports import weakref  # pylint: disable=g-bad-import-order
ImportError: cannot import name 'weakref'

➜  baselines git:(master) python
Python 3.5.3 |Anaconda 4.2.0 (x86_64)| (default, Feb 22 2017, 20:51:01)
[GCC 4.2.1 Compatible Apple LLVM 6.0 (clang-600.0.57)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import backports
>>> import backports as bp
>>> bp.weakref
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: module 'backports' has no attribute 'weakref'
>>> exit()
➜  baselines git:(master) pip install backports.weakref
Requirement already satisfied: backports.weakref in /Users/Tiger/anaconda/envs/rllab3/lib/python3.5/site-packages
➜  baselines git:(master) pip install -U backports.weakref
Requirement already up-to-date: backports.weakref in /Users/Tiger/anaconda/envs/rllab3/lib/python3.5/site-packages
➜  baselines git:(master)

-U flag didn't force reinstall, try uninstall and then install. Anyway, it looks like a problem in TF, so might be worth checking their issues (or ask if no issue found).

@pnlawlor
Thanks, your solution is also useful for me!

Locate the path here for mac user
/usr/local/lib/python2.7/site-packages/tensorflow/python/util/tf_should_use.py
then continue with @pnlawlor solution


Fix chunk with identity as chunking function

I'm very new to Lua but it seems like chunk is somewhat broken when using the identity function for chunking. Specifically it writes elements to ch[0] when instead they should be written to ch[1].
I added a failing test case for this particular situation but don't fully understand how the chunk code works (new to Lua) so haven't tried to fix it yet.

This test is failing:
https://github.com/Yonaba/Moses/blob/e1fbe125870585f8bdccfddf8df9f5edb4734904/spec/array_spec.lua#L374-L382
Could be I'm totally misreading the Lua implementation but in my local testing the change in this PR was necessary to make chunk not write to 0.

Ok so actually this seems to work in some situations but not in others...
local M = require("moses")

local function identity(x)
  return x
end

M.chunk({1, 1, 1, 2, 3, 4, 5}, identity)

-- returns:
-- {
--   1 [ 2 ]
--   2 [ 3 ]
--   3 [ 4 ]
--   4 [ 5 ]
--   0 [ 1 1 1 ]
-- }

local is_even(x)
  return (0 == (x % 2))
end

M.chunk({1, 1, 1, 2, 3, 3, 4, 4, 4, 5}, is_even))

-- returns:
-- [ [ 1 1 1 ] [ 2 ] [ 3 3 ] [ 4 4 4 ] [ 5 ] ]

Added a test to the PR and updated the description.

Good catch, @martinklepsch.
Much thanks for the awesome contribution. It has been merged.


Liveliness detection

Is your feature request related to a problem? Please describe.
Currently, the facial recognition library only verifies the identity of a person but does not provide any means to distinguish between a live person and a spoof (such as a photograph or a video). This poses a security risk in applications requiring robust authentication mechanisms.
Describe the solution you'd like
Integrate a liveliness detection feature into the facial recognition library. This feature should be able to distinguish between live persons and spoof attacks (e.g., photos, videos, masks) by analyzing facial movements, skin texture, and other biometric indicators. Ideally, the solution should work in real-time and be compatible with existing functionality of the library.
Additional context
Liveliness detection is becoming a standard requirement in many facial recognition applications, such as mobile banking, secure access control, and identity verification systems. Implementing this feature will enhance the security and reliability of the library, making it more competitive and suitable for a broader range of applications.

As an expertise in machine CNN and OpenCv I can contribute and resolve your issue . we had made same app for attendance system with accuracy of 85 % so please assign this issue to me

I would like to work on this feature @akshitagupta15june , can you please assign it to me ?

@akshitagupta15june please look into it.


Reconsider handling of out-of-order ops on delta connection

I wiped out changes of https://github.com/microsoft/FluidFramework/pull/4749/files, in particular these items below.
Reason why it's important - ops may take a long time to get to storage, and they may show up on socket sooner than
getDeltas() - this one is gone in master and needs follow up before 0.38 is cut:
            if (to !== undefined && this.lastQueuedSequenceNumber >= to) {
                // the client caught up while we were trying to fetch ops from storage
                // bail out since we no longer need to request these ops
                telemetryEvent.end({
                    deltasRetrievedTotal,
                    requests,
                    lastQueuedSequenceNumber: this.lastQueuedSequenceNumber,
                });
                return;
            }

And this one in enqueueMessages() - this one is gone in PR https://github.com/microsoft/FluidFramework/pull/5687
                if (this.pending.length > 0) {
                    // we processed a correctly sequenced inbound op while some are pending
                    // pending might include ops after the current sequence number, so process them now
                    this.processPendingOps(`EnqueueMessages_${telemetryEventSuffix}`);
                }

Reason it's important - Gary is about to prototype a partial solution for the following case:
We can get a gap between initial ops on the wire and next op, due to async processes on server.
This causes client to fetch ops from storage.
But PUSH could close the gap by detecting in in some cases ("write" connection) and re-querying redis and resending missing ops.
They will show up out of order relative to client view. I.e. initial ops: 10..20, join op: 30, ops: 21..29 (maybe).
Not having code above will make client poke at storage indefinitely

Addressed by https://github.com/microsoft/FluidFramework/pull/5687


Return empty location if block is not complete anywhere

wmagent branch version of #7448 (this one needs to be updated later on)

This branch version needs to be updated, don't merge please.

Seangchan, unfortunately this fix diverged from the master one (the master version can still fail on the agent side).
@ticoann can you please review and then I can create the another master PR? Thanks

test this please

Closing this wmagent branch fix. I'll create a new one when needed


Upgrade test matrix to include Django 2.2.

Enable Django 2.2 test environment.
Related to #251 issue.

Thank you


Delete android:label from application section on AndroidManifest.xml

I see <application android:label="@string/app_name"> in your AndroidManifest.xml.  Please delete it.
Because all android application used your sdk have to add tools:replace="android:label" in application section on AndroidManifest.xml.

We were not able to reproduce your issue, as we tested it and in the Merge Manifest view in Android Studio, it automatically replaces it with the actual app name in our Test App.
However its seems that this label is not needed for a Android library, hence we will try to remove it in the next SDK version. Thanks for reporting.

https://github.com/Leanplum/Leanplum-Android-SDK/pull/38

Thanks! A think I reproduced it because we set it android:label in gradle.


Consider replacing curl with something like ureq

Building tch-rs is (as expected) one of the biggest contributors to compilation time. However, I noticed that the execution of the torch-sys buildscript starts surprisingly late in the compilation graph (-> cargo build --timings), seemingly due to the curl crate and its build-script actually taking a good few seconds to build, and moving tch quite late into compilation.
By replacing curl with a smaller dependency such as ureq, I was able to improve clean build time by about 10 seconds. (Through very unscientific testing, of course, but still)
It may be worth considering such a switch. I'd be happy to contribute a PR if you think this is viable!

Closing this now that the change to ureq has been merged.


Tie request response

Description
This logs a request ID when when we log requests and responses so that users can logically tie them together.
I played around with color coding the log messages instead, but there were a few reasons I didn't move forward with it:

I was concerned users would try to attribute meaning to the colors (e.g., does green mean success and orange mean warning?)
We can't just pick random colors. We have to pick distinguishable colors and colors with proper contrast, for example. Doing this programatically is a bit problematic and introduces a bunch of code if we want to get close to doing it right. The alternative is to pick a static list of colors. At that point we have the cost of the list of colors and we have to rotate through them. I stole colors the design organization had already defined within the CloudViz charting libraries, but even then we only have 10 colors that we'd have to rotate through for each request.
Supporting styling of log messages would require us to do some browser detection for browsers that don't support it.
Supporting styling of logs messages would require some changes to our logger that wouldn't be super clean, because we have to work with the fact that only the first argument passed into console.log and friends can be styled (browser rules, not mine). The first argument passed into console.log right now is the SDK instance name, which isn't what we're trying to color.
I'm not sure how accessible it would be for the visually impaired.
At what point do we say, "Let's let the Experience Cloud Debugger exposing this data and handle making things visually nice."? We may actually be past that point already...
Owen agreed with my assessment

Anyway, I ended up going with the request ID approach. I started working on sending the request ID to the gateway, but there are problems with that so right now the ID is only used for this logging. It did require us to add a uuid package which adds about 1 KB to the minified file size. I also removed IE 10 support because Justin said we could and because cryptography APIs don't exist in IE 10, which would cause the UUID package to rely on Math.random() for randomness, which is sketchy.
I should also the UUID package bundles logic for IE 10 (and below?), which does get built into our library and is unnecessary for us. It basically amounts to this amount of extra code:
else {
	    // Math.random()-based (RNG)
	    //
	    // If all else fails, use Math.random().  It's fast, but is of unspecified
	    // quality.
	    var rnds = new Array(16);

	    module.exports = function mathRNG() {
	      for (var i = 0, r; i < 16; i++) {
	        if ((i & 0x03) === 0) r = Math.random() * 0x100000000;
	        rnds[i] = r >>> ((i & 0x03) << 3) & 0xff;
	      }

	      return rnds;
	    };
	  }

I think that's probably an acceptable cost when compared to having to maintain a copy of the uuid library. I've considered giving something like https://www.npmjs.com/package/patch-package a whirl, which may allow us to use a patched/trimmed version without having to maintain a full copy of the npm package. Anyway.....

Related Issue
https://jira.corp.adobe.com/browse/CORE-31390

Motivation and Context
Users wanted to logically tie together request and response log messages.

Screenshots (if appropriate):

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)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] All tests pass and I've made any necessary test changes.
[X] I have run the Sandbox successfully.

Looks like the build is failing. 😢  I'll look at it tomorrow.

It looks like the build failure was due to the sonar-scanner command not being found for some reason. Anyway, I restarted the build today and it passed. I also merged master into this branch because there was a conflict. Merging now.


WIP - Do not merge - Add bug bounty program page

Pending:

[ ] Update copy
[ ] Add links to buttons
[x] Change border color according to severity type:

Low: #F5D73D
Medium: #F5993D
High: #E24436
Critical: #8D4CE6

Checked with @deadpine, who told me that team doesn't want a bug bounty program anymore.


Ring PutOne panics with unaligned 64-bit on Raspberry PI 4 (arm64)

Spotted while running GitLab-CE (for which their KAS component is using this Rueidis client) and it keeps streaming this panic error in logs
2023-11-11_17:52:14.60237 goroutine 67 [running]:
2023-11-11_17:52:14.60243 runtime/internal/atomic.panicUnaligned()
2023-11-11_17:52:14.60247       /usr/local/go/src/runtime/internal/atomic/unaligned.go:8 +0x24
2023-11-11_17:52:14.60250 runtime/internal/atomic.Xadd64(0x4149354, 0x1)
2023-11-11_17:52:14.60361       /usr/local/go/src/runtime/internal/atomic/atomic_arm.s:258 +0x14
2023-11-11_17:52:14.60365 github.com/redis/rueidis.(*ring).PutOne(0x4149320, {0x4012360, 0x3000, 0x8000})
2023-11-11_17:52:14.60372       /root/go/pkg/mod/github.com/redis/rueidis@v1.0.14/ring.go:55 +0x34
2023-11-11_17:52:14.60375 github.com/redis/rueidis.(*pipe).Do(0x3d76820, {0x131c490, 0x3deb560}, {0x4012360, 0x3000, 0x8000})
2023-11-11_17:52:14.60378       /root/go/pkg/mod/github.com/redis/rueidis@v1.0.14/pipe.go:814 +0x2ec
2023-11-11_17:52:14.60387 github.com/redis/rueidis.(*pipe).Receive(0x3d76820, {0x131c490, 0x3deb560}, {0x4012360, 0x3000, 0x8000}, 0x40f0030)
2023-11-11_17:52:14.60390       /root/go/pkg/mod/github.com/redis/rueidis@v1.0.14/pipe.go:687 +0x24c
2023-11-11_17:52:14.60393 github.com/redis/rueidis.(*mux).Receive(0x3d36000, {0x131c490, 0x3deb560}, {0x4012360, 0x3000, 0x8000}, 0x40f0030)
2023-11-11_17:52:14.60396       /root/go/pkg/mod/github.com/redis/rueidis@v1.0.14/mux.go:322 +0xc0
2023-11-11_17:52:14.60399 github.com/redis/rueidis.(*singleClient).Receive(0x3c0f890, {0x131c490, 0x3deb560}, {0x4012360, 0x3000, 0x8000}, 0x40f0030)
2023-11-11_17:52:14.60402       /root/go/pkg/mod/github.com/redis/rueidis@v1.0.14/client.go:107 +0x58
2023-11-11_17:52:14.60405 gitlab.com/gitlab-org/cluster-integration/gitlab-agent/v16/cmd/kas/kasapp.(*serverApi).subscribeToEvents.func1({0x131c490, 0x3deb560})
2023-11-11_17:52:14.60458       /var/cache/omnibus/src/gitlab-kas/cmd/kas/kasapp/server_api.go:103 +0x170
2023-11-11_17:52:14.60471 gitlab.com/gitlab-org/cluster-integration/gitlab-agent/v16/internal/tool/retry.PollWithBackoff({0x131c490, 0x3deb560}, {{0x1313ec8, 0x3ef08c0}, 0x2faf080, 0x1, 0x40ee0c0}, 0x3c2ef50)
2023-11-11_17:52:14.60475       /var/cache/omnibus/src/gitlab-kas/internal/tool/retry/retry.go:80 +0x238
2023-11-11_17:52:14.60478 gitlab.com/gitlab-org/cluster-integration/gitlab-agent/v16/cmd/kas/kasapp.(*serverApi).subscribeToEvents(0x4160b40, {0x131c490, 0x3deb560})
2023-11-11_17:52:14.60481       /var/cache/omnibus/src/gitlab-kas/cmd/kas/kasapp/server_api.go:101 +0x74
2023-11-11_17:52:14.60484 gitlab.com/gitlab-org/cluster-integration/gitlab-agent/v16/internal/module/notifications/server.(*module).Run(0x3fc1048, {0x131c490, 0x3deb560})
2023-11-11_17:52:14.60487       /var/cache/omnibus/src/gitlab-kas/internal/module/notifications/server/module.go:16 +0x30
2023-11-11_17:52:14.60490 gitlab.com/gitlab-org/cluster-integration/gitlab-agent/v16/cmd/kas/kasapp.startModules.func1({0x131c490, 0x3deb560})
2023-11-11_17:52:14.60493       /var/cache/omnibus/src/gitlab-kas/cmd/kas/kasapp/configured_app.go:663 +0x3c
2023-11-11_17:52:14.60497 github.com/ash2k/stager.(*stage).Go.func1()
2023-11-11_17:52:14.60500       /root/go/pkg/mod/github.com/ash2k/stager@v0.3.0/stage.go:25 +0x34
2023-11-11_17:52:14.60503 created by github.com/ash2k/stager.(*stage).Go
2023-11-11_17:52:14.60522       /root/go/pkg/mod/github.com/ash2k/stager@v0.3.0/stage.go:24 +0x7c
2023-11-11_17:52:15.74897 panic: unaligned 64-bit atomic operation

I reproduce the issue when cloning redis/rueidis and running Ring unit test:
go test -v -run TestRing

=== RUN   TestRing
=== RUN   TestRing/PutOne
panic: unaligned 64-bit atomic operation

goroutine 38 [running]:
runtime/internal/atomic.panicUnaligned()
        /usr/local/go/src/runtime/internal/atomic/unaligned.go:8 +0x24
runtime/internal/atomic.Xadd64(0x25244b4, 0x1)
        /usr/local/go/src/runtime/internal/atomic/atomic_arm.s:258 +0x14
github.com/redis/rueidis.(*ring).PutOne(0x2524480, {0x2564588, 0x0, 0x0})
        /home/xavier/go/src/rueidis/ring.go:55 +0x34
created by github.com/redis/rueidis.TestRing.func1 in goroutine 35
        /home/xavier/go/src/rueidis/ring_test.go:24 +0xf8
panic: unaligned 64-bit atomic operation
[...]

My hardware is a Raspberry PI 4, running Raspberry Pi OS (64-bit), I've installed Go using standard instructions and go1.21.4.linux-arm64.tar.gz, however for some reason Go sets GOARCH to "arm" (ie, 32bit) instead of "arm64" - see go env output below:
Distro:
uname -a
Linux raspberrypi 6.1.21-v8+ #1642 SMP PREEMPT Mon Apr  3 17:24:16 BST 2023 aarch64 GNU/Linux

cat /etc/os-release
PRETTY_NAME="Raspbian GNU/Linux 11 (bullseye)"
NAME="Raspbian GNU/Linux"
VERSION_ID="11"
VERSION="11 (bullseye)"
VERSION_CODENAME=bullseye
ID=raspbian
ID_LIKE=debian
HOME_URL="http://www.raspbian.org/"
SUPPORT_URL="http://www.raspbian.org/RaspbianForums"
BUG_REPORT_URL="http://www.raspbian.org/RaspbianBugs"

Go env:
go version
go version go1.21.4 linux/arm64

go env
GO111MODULE=''
GOARCH='arm'
GOBIN=''
GOCACHE='/home/xavier/.cache/go-build'
GOENV='/home/xavier/.config/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='arm64'
GOHOSTOS='linux'
GOINSECURE=''
GOMODCACHE='/home/xavier/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='linux'
GOPATH='/home/xavier/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/usr/local/go'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/usr/local/go/pkg/tool/linux_arm64'
GOVCS=''
GOVERSION='go1.21.4'
GCCGO='gccgo'
GOARM='7'
AR='ar'
CC='gcc'
CXX='g++'
CGO_ENABLED='0'
GOMOD='/home/xavier/go/src/rueidis/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 -marm -Wl,--no-gc-sections -fmessage-length=0 -ffile-prefix-map=/tmp/go-build2790280466=/tmp/go-build -gno-record-gcc-switches'

Googling for it, I found this that may be a clue / starting point: https://github.com/golang/go/issues/23345

I run a couple more of checks, and for info, reverting the alignment change that was introduced in https://github.com/redis/rueidis/commit/d3b3668edeb0bd038cf2d839170d3d0be1373995 seems to solve it:
diff --git a/ring.go b/ring.go
index f3f92b2..b0b6526 100644
--- a/ring.go
+++ b/ring.go
@@ -31,13 +31,12 @@ func newRing(factor int) *ring {
 }

 type ring struct {
-       store []node // store's size must be 2^N to work with the mask
-       _     [5]uint64
        write uint64
        _     [7]uint64
        read1 uint64
        read2 uint64
        mask  uint64
+       store []node // store's size must be 2^N to work with the mask
 }

 type node struct {

go test -v -run TestRing
=== RUN   TestRing
=== RUN   TestRing/PutOne
=== RUN   TestRing/PutMulti
=== RUN   TestRing/NextWriteCmd_&_NextResultCh
=== RUN   TestRing/PutOne_Wakeup_WaitForWrite
=== RUN   TestRing/PutMulti_Wakeup_WaitForWrite
--- PASS: TestRing (0.34s)
    --- PASS: TestRing/PutOne (0.07s)
    --- PASS: TestRing/PutMulti (0.06s)
    --- PASS: TestRing/NextWriteCmd_&_NextResultCh (0.00s)
    --- PASS: TestRing/PutOne_Wakeup_WaitForWrite (0.10s)
    --- PASS: TestRing/PutMulti_Wakeup_WaitForWrite (0.10s)
PASS
ok      github.com/redis/rueidis        0.363s

Hi @xaxa06,
Thank you for finding the bug. While I don't have arm machines, do you find other places that have the same bug?

From what I was able to tell by running go test on the whole project, the atomic.AddUint64 from lru.Flight() are also panicing similarly, ex:
go test -v .
=== RUN   TestCacheStore
=== RUN   TestCacheStore/LRUCacheStore
=== RUN   TestCacheStore/LRUCacheStore/Flight_and_Update
--- FAIL: TestCacheStore (0.00s)
    --- FAIL: TestCacheStore/LRUCacheStore (0.00s)
        --- FAIL: TestCacheStore/LRUCacheStore/Flight_and_Update (0.00s)
panic: unaligned 64-bit atomic operation [recovered]
        panic: unaligned 64-bit atomic operation

goroutine 25 [running]:
testing.tRunner.func1.2({0x422f00, 0x52eba0})
        /usr/local/go/src/testing/testing.go:1545 +0x27c
testing.tRunner.func1()
        /usr/local/go/src/testing/testing.go:1548 +0x42c
panic({0x422f00, 0x52eba0})
        /usr/local/go/src/runtime/panic.go:914 +0x244
runtime/internal/atomic.panicUnaligned()
        /usr/local/go/src/runtime/internal/atomic/unaligned.go:8 +0x24
runtime/internal/atomic.Xadd64(0x1c93234, 0x1)
        /usr/local/go/src/runtime/internal/atomic/atomic_arm.s:258 +0x14
github.com/redis/rueidis.(*lru).Flight(0x1ca6300, {0x46e6c6, 0x3}, {0x46e6c9, 0x3}, 0x5f5e100, {0xc14c5dcc1f58900a, 0xb4bd7b, 0x765ae0})
        /home/xavier/go/src/rueidis/lru.go:124 +0x76c
github.com/redis/rueidis.test.func1(0x1c9a870)
        /home/xavier/go/src/rueidis/cache_test.go:16 +0xa8
testing.tRunner(0x1c9a870, 0x1c8e4e8)
        /usr/local/go/src/testing/testing.go:1595 +0x124
created by testing.(*T).Run in goroutine 24
        /usr/local/go/src/testing/testing.go:1648 +0x3f0
FAIL    github.com/redis/rueidis        0.020s
FAIL

Also seen in these other tests (all with a smiliar call stack up to lru.Flight(..) )

TestLRU/Cache_Hit_&_Expire
TestClientSideCaching

At this stage I got lazy and stopped manually skipping KO testst to figure out the next one, instead I assumed it's unlikely some other path has issue and wasn't tested yet by all these tests.
Also I tried running this ./docker-test.sh, but no luck, looks like some of the images needed by compose do not have arm64 versions:
./dockertest.sh

trap "docker compose down -v" EXIT
docker compose up -d
[+] Running 10/51
[...]
no matching manifest for linux/arm/v8 in the manifest list entries
docker compose down -v

My environment is still identical to the one I gave in my initial message (see https://github.com/redis/rueidis/issues/407#issue-1989078385)
Let me know if you'd like me to run some more tests or provide complementary info.

Hi @xaxa06, I have pushed a fix here https://github.com/redis/rueidis/pull/408. Would you like to test it?

Hi @rueian, nice! I'll try to test it tonight once at home and revert back to you !

I confirm with #408 all tests are now passing, no more panic :)
➜  rueidis git:(fix-atomic-alignment) ✗ go test -v .
[...]
--- PASS: TestParseURL (0.00s)
=== RUN   TestMustParseURL
--- PASS: TestMustParseURL (0.00s)
=== RUN   TestMustParseURLUnix
--- PASS: TestMustParseURLUnix (0.00s)
PASS
ok      github.com/redis/rueidis        74.387s

(note: I skipped the ones for redis_test.go as I still couldn't get that docker compose redis cluster to work on my env for same reason as I mentioned in my earlier comment, but I doubt there would be any uncaught alignment issue hiding there)

it’s OK thank you for your assistance!


Add build tasks

Follow the https://docs.microsoft.com/en-us/learn/modules/create-a-build-pipeline/6-create-the-pipeline

No need


Do not log exceptions when fetching offsets for lag metric

This is leading to a lot of noisy logs. adding control to switch to debug logging in case we are simply fetching offsets for a metric rather than for actual usage

Codecov Report
Attention: Patch coverage is 0% with 10 lines in your changes missing coverage. Please review.

Project coverage is 0.00%. Comparing base (59551e4) to head (fe440df).
Report is 711 commits behind head on master.

Files
Patch %
Lines

...a/manager/realtime/RealtimeSegmentDataManager.java
0.00%
10 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (59551e4) and HEAD (fe440df). Click for more details.
HEAD has 53 uploads less than BASE

Flag
BASE (59551e4)
HEAD (fe440df)

integration
7
1

integration2
3
1

temurin
12
1

java-21
7
1

skip-bytebuffers-true
3
0

skip-bytebuffers-false
7
1

unittests
5
0

unittests1
2
0

java-11
5
0

unittests2
3
0

integration1
2
0

custom-integration1
2
0

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #13528       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436     2482       +46
  Lines        133233   137205     +3972
  Branches      20636    21297      +661
=============================================
- Hits          82274        0    -82274
- Misses        44911   137205    +92294
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <0.00%> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <0.00%> (ø)

java-11
?

java-21
0.00% <0.00%> (-61.63%)
:arrow_down:

skip-bytebuffers-false
0.00% <0.00%> (-61.75%)
:arrow_down:

skip-bytebuffers-true
?

temurin
0.00% <0.00%> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

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.

+1.
Just one comment. Wouldn't be better to just log the message as warn but only once every X seconds?

This one still happens every X seconds where X is generally the fetchTimeout of the consumer. But yeah we can make that more explicit.
Will add your and manish's callouts in future implementation


Fix Fireproofing on domains that authenticate elsewhere

Task/Issue URL: https://app.asana.com/0/488551667048375/1195055028680547/f
Tech Design URL:
CC:
Description:
This PR improves login detection on sites that authenticate via other domains.
We are covering the following cases:

common social login providers
Handling known 2FA steps on common websites
sso via duosecurity or simpleSaml

This PR doens't enables login detection by default.
Steps to test this PR:
Login detection disabled:

Fresh install
Go to any website
Sign in
Ensure we don't ask to fireproof any site

Login detection enabled (social login providers):

Go to settings and enable login detection
Go to any website where you can sign in using your google/github/twitter/facebook/... account (you can take a look to some tested websites here: https://app.asana.com/0/0/1199631139904357/f)
Sign in
Ensure we ask to fireproof the final domain

Login detection enabled (sso):

Go to settings and enable login detection
Go to Asana
Go to their login page (you will need to enable desktop site)
Sign in as usual
Ensure we ask to fireproof Asana domain

Login detection enabled (site form): This test is just to ensure backward compatibility, our JS layer sometimes fails detecting some login forms.

Go to settings and enable login detection
Go to any website where you have an account to sign in
Sign in as usual
Ensure we ask to fireproof that domain

Internal references:
Software Engineering Expectations
Technical Design Template

Thanks for the review.
For the reported cases:

Trello-atlassian: it will be documented
I couldn't reproduce it. I've also tried to debug the code and analyze if there's something that could make it non-deterministic, but I haven't found anything. There are no delays involved on that logic. And checking the urls I see in the logs, everything seems fine and handled correctly.

Thanks for the review.
For the reported cases:

Trello-atlassian: it will be documented
I couldn't reproduce it. I've also tried to debug the code and analyze if there's something that could make it non-deterministic, but I haven't found anything. There are no delays involved on that logic. And checking the urls I see in the logs, everything seems fine and handled correctly.


.toggleClass/.addClass not working well with paging

Been using your grid for a while now and i'd say it saves me a lot of work when it comes to data processing, and is actually much better than other grid extensions out there.
I'm just having this issue with toggleClass and addClass, paging seems to remove the added class to the selected.
Am i missing something? Thanks in advance.
Code below
$(document).on("rowclick", "#thegrid tr", function (e) {
            $(this).addClass('selected').siblings().removeClass('selected');
            //data processing
});

It's not client sided grid, so every page reload the html, thus losing classes.


[unit test] improve unit test coverage for yurthub

What would you like to be added:
improve unit test coverage for yurthub:

current coverage: 45%
https://app.codecov.io/gh/openyurtio/openyurt/compare/911/tree/pkg/yurthub

target coverage: 70%

Why is this needed:
improve unit test coverage for OpenYurt project.
others
/kind feature

Since we're pending at #882 which has great changes on yurthub cache components, could we write unit tests after #882 being merged? Otherwise we have to rewrite these tests later.
Also, we'd better do #857 based on #882, otherwise we have to retest it later.

Since we're pending at #882 which has great changes on yurthub cache components, could we write unit tests after #882 being merged? Otherwise we have to rewrite these tests later.
Also, we'd better do #857 based on #882, otherwise we have to retest it later.

@Congrool ok, i will focus on reviewing #882 and merge this pull request before unit test.

/assign donychen1134


Bug: check "tcp-ping" doesn't get Outage

Tonight I had a (planned) Outage of my "KA"-site on https://thomasmerz.github.io/upptime/. All url-checks for "-KA" has been outaged, but tcp-ping for "-KA" ("Wireguard") was not outaged by upptime's point of view.

Well, I found out, that I would need a udp-check for some of my checks 😢  Therefore I close this issue.


I think you want an na.rm = TRUE

I run into an issue with a dataset where select_rows returns only FALSE and NA.  This leads to a problem in this line, which appears to occur four times in the code.
if (sum(phenotype_cells)>0) {
This code occurs in the nearest neighbor and counts within functions.  I am a little confused how I get only false and NA when I remove phenotypes that have no occurrence in the densities produced.
After I change the code to
if (sum(phenotype_cells, na.rm = TRUE)>0) {
all is well.  Thanks!

Thanks, for the report. I think a better fix for this is for select_rows to return FALSE instead of NA. I made this change in 385790b6. You can try it out with
remotes::install_github('akoyabio/phenoptr@9000')
remotes::install_github('akoyabio/phenoptrReports@9000') # If you are using phenoptrReports, keep them in sync

Thanks Kent!  One clarification, is this change merged into the current master branch?

No, it is on the 9000 branch. See above for installation instructions.


Fix to make custom headers work in test CallFN function

I noticed the EnvAsHeader function was incorrect, and that the only test that really exercises it was also broken in a way that made it pass when it shouldn't.
The fixed test doesn't pass because the attempt to invoke the function fails with: Failed to pull image 'denismakogon/os.environ': pull access denied for denismakogon/os.environ, repository does not exist or may require 'docker login'
@denismakogon, looks like that container isn't publicly readable -- could you make that image publicly accessible somewhere?

@ostrain the images you referred to has gone, so i recommend to make a new one to cover this case.

fn-api-tests are decomissioned, closing.


error with gci 0.8

What version of GCI are you using?

0.8.0

Reproduce Steps

use tidb
https://github.com/pingcap/tidb/pull/38320

make bazel_linter

What did you expect to see?
pass
What did you see instead?
ERROR: /home/jenkins/.tidb/tmp/c81bba9845037d26d5cccd5f3ccb189e/external/com_github_gogo_protobuf/gogoproto/BUILD.bazel:3:11: GoCompilePkg external/com_github_gogo_protobuf/gogoproto/gogoproto.a failed: (Exit 1): builder failed: error executing command bazel-out/k8-opt-exec-2B5CBBC6/bin/external/go_sdk/builder compilepkg -sdk external/go_sdk -installsuffix linux_amd64 -src external/com_github_gogo_protobuf/gogoproto/doc.go -src ... (remaining 29 arguments skipped)

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging

compilepkg: nogo: errors found by nogo during build-time code analysis:

analyzer "gci" failed: 35:1: expected 'STRING', found 'func' (and 10 more errors)

Could you help locate which file lead this error?

When I run the command gci -w hello.go, I got the similar issue: Error: 4:1: expected 'STRING', found '(' (and 2 more errors), with the code in hello.go:
package main

import (
"log"
"code.s.org/a/be_test/app/demo"
"code.s.org/a/be_test/app/demo"
)

func main() {
}

@daixiang0 One more case with v0.8.1, got error: Error: 4:5: expected 'STRING', found '(' (and 4 more errors)
package repo

import(
    "context"
    "code.s.org/a/be_test/app/demo/domain/demomodel"
)

type DemoRepo struct {
}

func (*DemoRepo) Get(ctx context.Context, id int64) (*demomodel.Demo, error) {
    // TODO your repo code here
    return &demomodel.Demo{}, nil

Now GCI assumes the source code is formatted, so you can format it first with help of IDE.

@daixiang0 so would this case be fixed or just by design?

with #117 it is fixed.

It's working correctly for me, just wonder if it could have a new tag :)

New tag comes ;)


Fix TestExportIsolationList

Can you provide a comment explaining how this got broken and why the more permissive parsing is now desirable?

Can you provide a comment explaining how this got broken and why the more permissive parsing is now desirable?

The NumberStyles argument is not important, but I needed to pass in something in order to be able to call the "TryParse" overload that takes a CultureInfo.
The only actual change is the passing in "_cultureInfo" (which is set to "CultureInfo.InvariantCulture").
That is, the implementation of the two-argument float.TryParse looks like this:
    public static bool TryParse(string s, out float result)
    {
      return float.TryParse(s, NumberStyles.Float | NumberStyles.AllowThousands, NumberFormatInfo.CurrentInfo, out result);
    }

So, can I assume this came up because we discovered the RT windows for Agilent were actually RT deltas, making it necessary to 1/2 the window values? And that caused round numbers to gain a decimal point? e.g. 5/2 = 2.5. I guess also there is the possibility of 0.1 for "RT (min)".
Keep explaining. That the field is name "RT Window (min)" makes me a bit nervous, since the code comment I saw called it "RT Delta (min)" and we are now halving it, which seems appropriate for a delta but not a window.

Keep explaining.

Can you take this over? I'm not near a computer anymore. I was just trying to fix this test breaking in French. If this test has bigger problems I can't fix it for a while.


Invalid parameter when trying to load large image

I have hit the limit of image sizes for GDI+ so I was looking for an alternative and came across your library. Unfortunately it seems it does not like large images either. I have a PNG that is 17MB in size that I need to be able to manipulate. I get an "Invalid Parameter" error when trying to load it. Any ideas? Here is the stack trace:
System.ArgumentException: Parameter is not valid.
at System.Drawing.Bitmap..ctor(Int32 width, Int32 height, PixelFormat format)
at ImageProcessor.Common.Extensions.ImageExtensions.Copy(Image source, AnimationProcessMode animationProcessMode, PixelFormat format, Boolean preserveExifData)
at ImageProcessor.ImageFactory.Load(Stream stream)
at GLHOMES.MultiTouch.Converter.Program.Main(String[] args) in C:\Development\Proofs\Multi Touch\Converter\Program.cs:line 39

It looks like you are still running into the limits of System.Drawing. Have you tried ImageSharp: https://github.com/JimBobSquarePants/ImageSharp. This is the .NET Core version of this library that does not use System.Drawing.

I just did. It runs into an OutOfMemoryException for that same file.

Then it looks like you've hit the physical limitation of your machine. Are you running in 32 or 64bit mode? Does the image require transparency?

Probably. Its a png with transparency, and I am running it in a 64 bit console app on Windows 7 with 32GB of memory. I am sure once its unpacked its several gb's of data in memory that .net chokes on.

You might want to give my library a try: https://github.com/dlemstra/Magick.NET. This library has support for reading ridiculous large images.

That was next on my list to try. Thanks!

Agreed. Give that a go.
17MB in compressed form must be massive image dimension-wise. I dunno if I'll ever add file based paging to support files that big in ImageSharp but food for thought.

Yeah, its huge because its for a mapping application and has to be very high resolution. We are trying to use that file to create tiles at lower resolutions for map systems like google and bing. There are commercial products that I can use to do that but they are incredibly expensive. Many posts I found recommend looking into memory mapped files, but that doesn't give me much help in resizing and chopping up the image into tiles.

I would have though svg would be your best bet for mapping. scalable and tiny.

I wish that were the case. Our drawings are so complex from CAD that converting them to any vector format does not perform well once displayed in any technology we have tried. We hit the limit of what it can do. We are now trying rasterized formats instead since there is ever only a set of fixed size tiles loaded into memory at a time. Its generating the tiles at high enough resolution that is the issue.

There's not really much I can offer here I'm afraid. I wasn't aware of performance rendering issues with SVG.
If Magick.NET doesn't work I'm not sure what to suggest.

For others who hit a similar issue:
I had this when running on IISExpress in visual studio which runs the 32bit version by default.
To find the setting for 64 bit version go to:
tools -> options -> Projects and Solutions -> Web Projects   (or similar depending on version of visual studio)

@Myster Yeah that one catches lots of people out. My advice is generally use IIS.


profile update

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[X] My change requires changes to the documentation.
[X] I have updated the documentation accordingly.
[X] All new and existing tests passed.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Already reverted the vscode\settings.json


Execute CRITICAL with LLM Integration, the instruction is not visible.

How to train this framework using a large model, thanks

To Execute CRITICAL with LLM Integration, how to run the 'main,py' file with necessary command.

--use_llm = true, then run main.py, and i get the following error : NameError: name 'llm' is not defined


FileSystemException: Creation failed, path = '.local-chromium' (OS Error: Read-only file system, errno = 30)

/flutter (16193): #0      _Directory.createSync (dart:io/directory_impl.dart:133:7)
E/flutter (16193): #1      _Directory.createSync (dart:io/directory_impl.dart:128:16)
E/flutter (16193): #2      downloadChrome
package:puppeteer/src/downloader.dart:27
E/flutter (16193): #3      _inferExecutablePath
package:puppeteer/src/puppeteer.dart:330
E/flutter (16193): #4      Puppeteer.launch
package:puppeteer/src/puppeteer.dart:140

at the moment it doesn't work for mobile due to the Mobile Manufacturer policy of using the headless browser in mobile... but I'm already looking into another solution by using socket implementation

Then why does the pub page says that it supports both Android and iOS

Then why does the pub page says that it supports both Android and iOS

Please mention it in the Readme section.

Yeah, I will update the ReadMe... the part where it started IOS and Android was automatically done by Dart itself... I don't have control over that...

Then why does the pub page says that it supports both Android and iOS

Please mention it in the Readme section.

Yeah, I will update the ReadMe... the part where it started IOS and Android was automatically done by Dart itself... I don't have control over that...

I have seen some packages that only support a certain platform and only that platform is mentioned. (e.g. android_alarm_manager)

Then why does the pub page says that it supports both Android and iOS

Please mention it in the Readme section.

Yeah, I will update the ReadMe... the part where it started IOS and Android was automatically done by Dart itself... I don't have control over that...

I have seen some packages that only support a certain platform and only that platform is mentioned. (e.g. android_alarm_manager)

Yeah like I said it was automatically.. it is Dart analyzer that does all that... it analyzed the code and determines the compatibility....
What I did suppose to support all platform it was just that it throws the error because of the mobile manufacturer policy...
Dart analyzer only analyzes code...


Cannot GET /index.html

I try to F1 -> Show live server preview and this appears...

Using v.1.21.0-insider

I am getting the same issue. I've reinstalled vscode with version 1.20.1 and reinstalled the preview plugin.

@LizKeay use https://marketplace.visualstudio.com/items?itemName=ritwickdey.LiveServer instead

#7 I think the issue is coming from the fact that pro program is confused between / path separators and \ path separators.

I got this issue when I had two workspaces open.  Switching to just one workspace fixed it.

Hi, I'm having this same exact issue, I only have 1 workspace open. also when I try to use  https://marketplace.visualstudio.com/items?itemName=ritwickdey.LiveServer:
It says it needs a body and html tag, but the html does have a body and html tag. any thoughts or solutions?

Sorry for the late reply.
I will work on this issue.

Thanks - issue resolved for me.

Thanks - issue resolved for me.

how you solved it

In caser it helps anyone else, I originally named my distribution folder as "public_html" and got this error. I renamed it to "public" and updated the config files, etc. restarted VS Code and it worked fine.

rename it to another name and it will work fine

Yes, it works only for files from one workspace (one ancestor folder).

I had this issue and solved it by removing special characters from my file name.  I had a file with #2 in the name, I removed the # from the title and everything worked fine.

If you are using live server on Explorer go to Live share and under section Shared Servers just click on Live Server.

I had that message when another file was opened in that extension. Just close all the folders in the File menu of the VScode and that should solve it.

Thank you @ccomstudio I had the same issue in folder name I have used '&' removing special character solved my problem.

I had just disable all the extensions which were working , and then i enabled it again and it started working as same as before..
I think this might help you.

i were stuck with the same issue but resolved it finally.
here is how:
i were only opening my file which is "index.html" in visual studio code but then i closed the file and opened the folder which contains that file in vs code. after that i used " open with live server by right clicking " and here it worked.

I got this issue when I had two workspaces open. Switching to just one workspace fixed it.

Same as mine, so it means that the live server will work only for 1 workspace..

I had the same problem but here is the solution .
The file you're trying to see through live server might be in a folder which is co-folder of some other folder let say folder name A. And this problem arise when we open the vscode through folder A rather than the exact co-folder in which our file exist. so just close the vscode. open it in that particular co-folder and you're good to go. :)

It seems the issue is that if the path contains JavaScript keywords, it breaks.

Red = broken. Green works.

I had this issue and solved it by removing special characters from my file name. I had a file with #2 in the name, I removed the # from the title and everything worked fine.

thanks!!!

I just ran into this issue and it was the "&" in the folder name. I could still use the option to "Go Live" in the bottom right corner of VSC and navigate to the index.html in that folder name and it worked fine. However, right-clicking to "Open with live server" ended in an Error: Cannot GET /HTML%20%5E&%20CSS/x/index.html

It seems the issue is that if the path contains JavaScript keywords, it breaks.
Red = broken. Green works.
The difference? The broken one contains the JavaScript keywords LET and CONST in the path... That's the only difference I can see. I validated this by renaming the folder to "test" and it worked after that. So the folder name having JS keywords seems to be the issue here.

A very simple change worked for me, just try changing your folder name in which your files are present, sometimes if your folder container has too many symbols and characters with spaces in it, change it to something simple without many characters.
Especially the '&' character.

i have the same problem i still can not solve it

@busybee23 Thanks, that worked for me. The '&' character in the folder name was the issue for me.

I resolved mine by removing the special character "&" from my folder name

@busybee23 Thanks, that worked for me. The '&' character in the folder name was the issue for me.

No worries sir.

you created index.html file which is resides in  (eg:xyz) folder.
next when you was trying to open that file with the help of live server you got the error like 404 can't get
i had also facing the same error but after passed 30 min i got it solution
let's go look a few steps
1)you got an error cause you only putinig filename in path

2)but you created index.html file it's actually resides in Project folder(folder name can be anything )

3)you have to put full path in your browser just like this (/foldername/filename) after that you will render the your index.html page.

Thank you :)

who got a solution to this problem? i'm having the same issue

Basta você renomear o arquivo index.html  , pois pode estar ocorrendo um sobrepujamento de arquivos com nomes iguais, nesse caso (index.html), assim é só salvar o novo nome do arquivo e executar o servidor "LiveServer".

#7 I think the issue is coming from the fact that pro program is confused between / path separators and \ path separators.

Thank you very much bro...
I had used & (ampersand) in one of my repository folders and got this error...
By changing it, my problem was solved! :)
...

I solved this issue by removing special characters like &.
my folder name before issue - drag & drop
my folder name after resolving the issue - drag and drop

The solution !!!!!!!!!!!!!!!! I got for the same Problem.
I was also fed up retrying and finding out what's wrong with my code/ live server ! I also used Ampersand (&) in naming my local repository file....
Avoid This (&) guys.... in naming your repositories.


raise http-types upper bound

I haven't figured out how to build the test or demo projects, but the library seems to build just fine with http-types version 0.11.

@chris-martin Sorry, not sure how I missed this. Merging and releasing now.


Event Navigated, Navigating not fire when custome renderer (HybridWebView)

Event Navigated, Navigating not fire when custome renderer.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.
ID: 9c1bd30c-6e7a-df38-8c20-dff7361e901f
Version Independent ID: 013c6027-db9a-6039-3404-2e4f717133a9
Content: Customizing a WebView - Xamarin
Content Source: docs/xamarin-forms/app-fundamentals/custom-renderer/hybridwebview.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @khoangt
I just tried this and both the Navigated and Navigating events fired just fine. See the images below which contain breakpoints being hit:

Hi @davidbritch
Do you check on Android?

Hi @khoangt
Agreed that Navigated and Navigating don't fire on Android for HybridWebView. Working on a fix now.

Hi @khoangt
I've fixed this issue in both the sample and the doc. The doc changes will go live in the next hour.
In the meantime, the code changes can be seen in the sample commit.

Hi @khoangt
I've fixed this issue in both the sample and the doc. The doc changes will go live in the next hour.
In the meantime, the code changes can be seen in the sample commit.

Thanks @davidbritch
I found another bug relaltive Webview Navigation
On iOS When turn off Internet => Navigated Failure don't fire


The data set is crashed

Hi,
Thank you for this dataset.
I have downloaded the data set in the sub-folder "synthetic". this is the exact link: (https://github.com/microsoft/AEC-Challenge/tree/main/datasets/synthetic).
But I think there is a big problem with this data set because all of them are crashed.
I would like to know do you have another link to download the dataset correctly or could you please help me to solve this?

I have the same problem,Have you got new dataset?

Hi. you should install Git Bash and after that, you can download it.

can you share me a baiduyun netdisk link please? its too slow to down the dataset from here.

The amount of dataset is heavy. you should just download it by Git Bash.


get_pathing_distance is missing from the book leading to crashes

The function is in the source but not documented in the body of the text leading people typing along with crashes
#108
#96
https://github.com/thebracket/rustrogueliketutorial/issues/108#issuecomment-600079261

Looks good, thank you.


[BuildSystem] Fix a race between build() and cancel()

This fixes a race which occurs when delegate is being initialized and
clients call cancel().
This also adds asserts that clients call build() before cancel().
-- rdar://problem/31951869

@sw

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test linux


segmentation fault

I compiled this code to use with PyFeast. I ran make x64 and sudo make install. When I tried to run the code that uses FEAST through PyFeast I got a segmentation fault. I determined the segmentation fault was due to the FEAST code.
I didn't find instructions on proper installation with the make file, could the segmentation fault be due to incorrect installation, or are there any suggestions for another reason I got a segmentation fault?

PyFeast hasn't been updated to use the new API from FEAST v2. It only works with the v1 API. Which version of FEAST did you compile?

I would've compiled the FEAST v2, can I access v1 using the makefile?

I would've compiled v2, can I access v1 through the makefile? Thanks!

What version of MIToolbox is required for v1?

The required MIToolbox is 2.1.2. I'd use FEAST v1.1.4, i.e. the last release before v2.

Did this fix the issue?

Yes, I had to change the name of the MIToolbox-2.1.2 directory to MIToolbox without the version numbers for the FEAST program to recognize it. But after that it was working fine. Thank you for your help.

Ok.


Update BA for already processed payments of the Work Period

Background
Managers can process payments for Work Periods. Money for the payments comes from the Billing Account which is associated with Resource Booking:

Sometimes Managers process payments using the wrong billing account. And they would like to update the billing account for all payments which are done during the current Work Period (week).
Task

Add Actions dropdown menu to each of the row in the table with 1 option: Update BA for payments (or add this option if Actions is already implemented via this task https://github.com/topcoder-platform/micro-frontends-taas-admin-app/issues/82)

The actions dropdown is already implemented in https://github.com/topcoder-platform/taas-app see demo page https://platform.topcoder-dev.com/taas/myteams/18102. Please, port it from there.
Or reuse it if it was already ported during another task https://github.com/topcoder-platform/micro-frontends-taas-admin-app/issues/80 or https://github.com/topcoder-platform/micro-frontends-taas-admin-app/issues/82.

When we click Update BA for payments show a modal:

For modal, use the same design as for adding payment: https://github.com/topcoder-platform/micro-frontends-taas-admin-app/issues/82
Inside Current BA used list the billing accounts which were used for payments at the moment which would be updated

If we click Cancel or X just close the modal.

if we click Update BA for payments show a loading indicator inside the modal OR just disable the fields and all the buttons while sending request to the server.

if the request was successful, show a successful green toastr and close the modal which says, Billing account was successfully updated for all the payments:

in case of error show error red toastr with error message from the server, but keep the modal open so user can try updating payment again

API

API has been implemented to support updating a billing account for multiple payments at once https://github.com/topcoder-platform/taas-apis/issues/433
We have to send a request to update Billing Account for all the payments inside the Work Period by the next request https://topcoder-platform.github.io/taas-apis/#/WorkPeriodPayments/patch_work_period_payments_bulk

@maxceem Below are some of the issues found,

No BA option is getting displayed normally in the dropdown, except the one assigned to the RB through backend.

If we assign some invalid/expired BA to the RB, no error is shown. The payment is going to completed status / BA updated for completed challenges without any error. But if the same BA is assigned at the project level, the payment is failing.
sample Expired BA used: 80000071
is that possible to validate the BA at RB level as the payment is based on the BA at RB level?

@sandhiyakavi

In this list we are show the same list of BA which we show for RB:

So if for RB we are showing only 1 BA, then when updating we also showing only 1 BA.
NOTE: the logic behind showing the list of BA is quite tricky:
- user has to have BAs available to they
- user must be Topcoder Admin, or be a member of the Topcoder Team inside the Project associated with Resource Booking
To verify if we are showing correct BAs, you may open project in Connect and see the list of BAs there, it should be the same in all 3 places: Connect, RB, and Update BA Model.

Theoretically we can try add such check, but I'm not sure if that's needed. Imagine the next situation:

we process payments using 1 BA
after some we realised that we used wrong BA, so we want to set the correct BA, but correct BA at this moment is expired now. I guess we would still would like to update BA in the past payments even though the BA is not more active, just to have correct history.

If you believe we should validate it anyway, I suggest tracking this separately as an enhancement requirement.

Below scenarios are tested  in Dev Env.

Verify Update BA for payments is shown in dropdown Actions only if there is already some payment processed.
Verify whether modal opens for Update BA for payments
Verify whether the contents Work period, ResourceBooking, Job name, Team name, Current BA's used are correctly displayed.
Verify whether all the BA's used for the WP are displayed in Current BA's used.
Verify whether BA are populated from the New Billing Account dropdown
Verify whether BA updated on clicking  'Update BA for payments' button.
Verify whether payment is not processed on clicking cancel or 'x' button.
Verify whether toaster message is displayed for processing payment 'Billing account was successfully updated for all the payments:'
Verify whether BA already updated was displayed if the BA was already displayed for all WP payments.

Verified on Prod Env. Working as expected.
cc: @nkumar-topcoder


Add License

It looks like you wanted to use an MIT License. This PR creates it for you in a format that GitHub recognises. It does not remove the existing one from the README.md file.

Merging now


mlx_lm.generate --ignore-chat-template doesn't accept prompt from a file.

If I run --prompt -<./text.txt, the prompt loads from the file with the chat template.
However, if I run --ignore-chat-template --prompt -<./text-with-chat-template.txt, the prompt is empty.

Thanks for flagging, thats a bug. Fix in #1166


Rust script

Changed the code to make it work with the rust-script ( had to implement serialize and deserialize because I couldn't get it to work with derive). And fixed the corrupt issue hopefully.
I am getting the path from the rust_script_path enviroment variable. could maybe change it to ask for input if you deem that better.

@iRaiko Thanks for updating it, looks great!  I'm curious about the deriving not working; Is it because you can't specify the order on derive(Serialize)?  I'd imagine that deriving deserialization would still work in that case.
I actually think that it would be best to hardcode the path since the TOML files will always be at the same location, and the script will be exclusively used for this repository.
I would also advise against specifying dependencies like serde="*"; I believe the best practice is to write the major and minor version and leave off the debug version, like this: serde="1.0".

@Nezeky The issue with deriving was that it could not find the traits Serialize, Deserialize in the serde crate. I didn't find another workaround for derive except implementing them.


Handle progress bar for live streams

The progress bar can exceed the duration width when it's a live hls stream, need to think of some solutions... Most obvious idea is to limit width to the duration.

<TVPlayer
url={  urlData?.cmd }
title={ch_name}
autoPlay={true}
ref={playerRef}
progressbar={false}
onError={handleError}
onLikePress={()=>{
handleAddToFav(ch_id, likeToggle)}}
customButtons={[
{ action: "previous", align: "center" },
{ action: "playpause", align: "center" },
{ action: "next", align: "center" },
{ action: "mute", align: "left" },
{ action: "like", align: "left", label: "Favorite" },
{
action: "fullscreen",
align: "right",
label: isFullScreen ? " " : "",
onPress: toggleFullScreen,
},
]}
fullscreen={isFullScreen}
/>
On live channels progress bar not working properly can you provide some alternate or how to hide this.
Second  fullscreen section have same icon on both mini screen and full screen .
can you guide me in this.


Remove: ESET.Security version <IP_ADDRESS>

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

URL: https://download.eset.com/com/eset/apps/home/eis/windows/latest/eis_nt64.exe
OK
(Automated message - build 923)

Hi @tobiasdegen,
This package is still available. Why is it being removed?

Hi @stephengillie,
the package under this URL is always the latest version. This leads to filehash mismatches.

[Policy] Needs-Manual-Merge


Terratowns Mock Server

[ ] Download Terratowns mock server into our repo
https://github.com/ExamProCo/terratowns_mock_server

Remove .git so it doesn't get confused with the original.
Installing Ruby with sudo apt install ruby ruby-bundler ruby-dev build-essential
Execute the bundle install
It will recreate the Gemfile.lock


Match screen refresh rate with content for smooth playback

I'm currently using Zapp on my FireTV to see if it could replace my standard tv cable.
Overall it's working pretty well but one of the major issues I have noticed is that the video stutters every few seconds. The reason is that Zapp doesn't adjust the screen refresh rate (60hz most of the time) to the video stream (50fps in the case of Zapp).
On newer FireTV you have two options called "Video resolution" and "Match original framerate". Setting the options to "Auto" and "On" respectively uses to 4K 60hz by default but then apps have the opportunity to manage the screen mode themselves.
Some apps like Amazon Prime Video, Kodi and Plex support this to match the screen refresh rate to that of the content being displayed. So a 24 fps movie will be displayed with the tv set to 24hz and 50 fps content with the tv at 50hz. There is an explanation in this article: https://www.aftvnews.com/explanation-of-the-new-frame-rate-matching-feature-on-the-amazon-fire-tv/
Ideally the refresh rate would be matched dynamically to the content but as far as I know all the content you could access in Zapp should be 50 fps and so setting the screen mode to 50hz for the whole app should be sufficient right now.
I'm not a developer and so I don't know how other apps have implemented this but there seems to be some way to achieve this  on a FireTV at least. In addition it looks like with Android 11 there is now an official frame rate API: https://developer.android.com/guide/topics/media/frame-rate
Adding this feature to Zapp would allow for smooth playback just like with the real tv channels on FireTV and it would run much smoother with a 60< hz Display on Android 11+ (like the Pixel 4) as the device could switch to the appropriate refresh rate to minimize/avoid the frame pacing issues.

Wow, this is a deeply technical issue. I will see what I can do to investigate this issue. But I am afraid I can't fix it without an actual Fire TV.
Has anyone experienced similar issues on Android TV / Chromecast?

There seem to be issues with exo player and Fire TV. Amazon recommends using their port of exo player: https://developer.amazon.com/de/docs/fire-tv/media-players.html (which I will most definitely not do).

Kodi seems to use ffmpeg as video player.

Sorry, I did a poor job explaining this issue. This isn't really a bug, it's more of an enhancement.
There is nothing really "broken" about the way Zapp behaves on FireTV.
This just about judder/stutter duo to a mismatch in frame rate/refresh rate. Pretty much all modern displays run at 60hz and so there will always be some stutters when you watch 50hz content on them because there is no way divide the 50 frames evenly across the 60hz.
If you use Zapp on a Smartphone, you will see the same issue if you pay close attention. Some people are more/less susceptible to this. I always notice it with camera movement and it's more noticeable on the big screen.
It seems like some Apps l like Kodi have found a way to adjust the refresh rate on FireTv (and Nvidia Shield) to smoothly display 50hz content and it would be nice to have this in Zapp as well.

Okay, I understand now.
I don't think I can fix this issue though. Zapp uses Googles ExoPlayer as video player - which is the de facto standard in Android world. This media player does not seem to adjust frame rate/refresh rate accordingly. This is why most Android apps like ZDF, Zapp, etc. show this behavior. Other apps like Kodi use another video player (most likely ffmpeg, very low level) which avoids this issue.
As Fire TV is not supported by ExoPlayer (I can imagine due to some Google vs. Amazon hostility) and I cannot switch to another video player (because of ease of use, support, updates, etc.) I don't think I can work around this issue.
However, ExoPlayer usually has excellent support for new Android features, so the Android 11 frame rate API you mentioned will most likely be supported in the future.
With that said, I'll close this issue as "won't fix". Nevertheless thank you very much for this detailed report!

Thank you for looking into this. I understand that this is not a priority right now.
For reference I have found the following documentation from Amazon: https://developer.amazon.com/docs/fire-tv/4k-apis-for-hdmi-mode-switch.html
It seems like they created their own implementation for Android 5 but are now using the Android 6+ AOSP implementation. So it looks like FireOS doesn't need any special treatment in this regard.
I'm not a developer but do you think ExoPlayer would have to implement this feature or could Zapp set preferredDisplayModeId on launch/start of playback to a 50hz mode independently and then the video player within the app would run at 50hz as well?

@k9janer This looks line an issue that ExoPlayer should be able to handle. But they state, they don't support FireTV (I would suggest for political reasons). You can always try to open an issue on their bug tracker - but I don't think they will accept it.