[ci][R-package] fix cleanup permissions

From RTD builds
Warning: file 'lightgbm/cleanup' did not have execute permissions: corrected

OK, this change breaks every build. So weird, but I have no time to dig deeper. Closing, that original warning doesn't look scary.

@jameslamb Ah, so nice timing for comment! But you was 0.00001sec faster! 😄

Haha no problem. I can open another PR to remove that warning some time this week. I have some time to experiment with it.

@jameslamb Thanks! I think it's suuuper minor 🙂


[Compile]: Packages incompatibility

Operating System
Android
Plugin
package_info_plus
Version
1.4.3
Flutter SDK
3.22.3
Compilation command
flutter run
Dart Dependencies
  line_icons: ^2.0.3
  after_layout: ^1.2.0
  timezone: ^0.9.4
  maps_launcher: ^2.2.1 # REMOVED, USE OTHER INSTEAD
  uuid: ^4.4.2 # IDs for events
  cloudinary_public: ^0.23.1
  showcaseview: ^3.0.0 # help for new users
  provider: ^6.1.2
  cloud_firestore: ^5.2.0
  firebase_messaging: ^15.0.4
  firebase_core: ^3.3.0 # needs Runner/GoogleService-Info.plist
  firebase_auth: ^5.1.3 # needs Runner/GoogleService-Info.plist
  firebase_analytics: ^11.2.1 # needs Runner/GoogleService-Info.plist
  #  firebase_crashlytics: ^2.8.6 # needs Runner/GoogleService-Info.plist and NSAppTransportSecurity in Info.plist
  badges: ^3.1.2
  flutter_translate: ^4.1.0
  flutter_facebook_auth: ^4.3.3
  google_sign_in: ^6.2.1 # needs Info.plist modification
  sign_in_with_apple: ^6.1.1 # needs Xcode modification https://pub.dev/packages/sign_in_with_apple
  shared_preferences: ^2.3.1
  # intl: ^0.19.0 # locales for table_calendar and datetime manipulation
  jiffy: ^6.3.1 # date manipulation
  permission_handler: ^11.3.1 # needs Info.plist modification
  csv: ^6.0.0
  flutter_email_sender: ^6.0.3
  flutter_launcher_icons: ^0.13.1
  image_picker: ^1.1.2 # for camera
  image: ^4.2.0
  url_launcher: ^6.3.0 # needs Info.plist modification
  transparent_image: ^2.0.1
  carousel_slider: ^4.2.1
  flutter_slidable: ^3.1.1
  table_calendar: ^3.1.2
  add_2_calendar: ^3.0.1 # Info.plist modification
  in_app_purchase: ^3.2.0
  flutter_platform_widgets: ^7.0.1
  flutter_native_splash: ^2.4.1
  intl_phone_number_input: ^0.7.4
  country_pickers: ^3.0.1
  currency_picker: ^2.0.21
  #  country_currency_pickers: ^1.0.1
  reorderables: ^0.6.0 # reorder rent fields
  pdf: ^3.11.0
  printing: ^5.13.1 # necessary for the above one
  open_file_safe: ^3.2.3 # necessary for the above one
  #  flutter_statusbarcolor_ns: ^0.4.0 # DO NOT CHANGE
  fimber: ^0.7.0
  contacts_service: ^0.6.3 # get contacts. Info.plist modification
  http: ^0.13.6
  cached_network_image: ^3.3.0
  firebase_performance: ^0.10.0
  fluttercontactpicker: ^4.7.0
  flutter_local_notifications: ^12.0.4
  device_info_plus: ^4.0.0
  share_plus: ^4.5.3
  path_provider: ^2.0.8
  flutter_svg: ^1.1.6
  file_picker: ^4.6.1 # for multiple images selection
  package_info_plus: ^1.4.3

Native dependencies
No response
Compilation Error
Resolving dependencies...
Because no versions of open_file_safe match >3.2.3 <4.0.0 and open_file_safe 3.2.3 depends on ffi ^1.0.0, open_file_safe ^3.2.3 requires ffi ^1.0.0.
And because package_info_plus ^1.4.3 depends on package_info_plus_windows ^2.0.0 which depends on ffi ^2.0.1, open_file_safe ^3.2.3 is incompatible with package_info_plus ^1.4.3.
So, because my_rents depends on both open_file_safe ^3.2.3 and package_info_plus ^1.4.3, version solving failed.

Flutter Doctor
[√] Flutter (Channel stable, 3.22.3, on Microsoft Windows [Version 10.0.22631.3958], locale en-GB)
    • Flutter version 3.22.3 on channel stable at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b0850beeb2 (3 weeks ago), 2024-07-16 21:43:41 -0700
    • Engine revision 235db911ba
    • Dart version 3.4.4
    • DevTools version 2.34.3

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\Users\Dani\AppData\Local\Android\Sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = C:\Users\Dani\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.10+0--11609105)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.9.4)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.9.34714.143
    • Windows 10 SDK version 10.0.22621.0

[√] Android Studio (version 2024.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.10+0--11609105)

[√] VS Code (version 1.92.0)
    • VS Code at C:\Users\Dani\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.94.0

[√] Connected device (4 available)
    • M2101K6G (mobile) • 996046da • android-arm64  • Android 13 (API 33)
    • Windows (desktop) • windows  • windows-x64    • Microsoft Windows [Version 10.0.22631.3958]
    • Chrome (web)      • chrome   • web-javascript • Google Chrome 127.0.6533.89
    • Edge (web)        • edge     • web-javascript • Microsoft Edge 127.0.2651.86

[√] Network resources
    • All expected network resources are available.

• No issues found!

Checklist before submitting a bug

[X] I searched issues in this repository and couldn't find such bug/problem
[X] I Google'd a solution and I couldn't find it
[X] I searched on StackOverflow for a solution and I couldn't find it
[X] I read the README.md file of the plugin
[X] I'm using the latest version of the plugin
[X] All dependencies are up to date with flutter pub upgrade
[X] I did a flutter clean
[X] I tried running the example project

This is a problem with open_file_safe which depends on an older version of ffi. Either report that to the maintainers of that package or see if dependency overrides can fix the issue.


hyprbars: aligning left or right kinda "changes" the order of the buttons

Probaly my interpretation of aligning left aligning right (coming form text editors/processors, but I think the order should not change or at least have a starting point/anchor for the alignment to choose the order of the buttons the same or have it the inverse:
Aligned left

Aligned right

...
        hyprbars-button = $btend, 20, 󰖭, hyprctl dispatch killactive
        hyprbars-button = $btmax, 20, 󰖯, hyprctl dispatch fullscreen 1
        hyprbars-button = $btmin, 20, 󰖰, hyprctl dispatch movetoworkspacesilent special
        hyprbars-button = $cosmiccobalt, 20,  , hyprctl dispatch togglefloating
        hyprbars-button = $btmove, 20, , hyprctl dispatch movewindow mon:eDP-1
        hyprbars-button = $btmove, 20, , hyprctl dispatch movewindow mon:HDMI-A-1
        hyprbars-button = rgb(403899), 20, , hyprctl dispatch movetoworkspace special:nebula
        hyprbars-button = rgb(403899), 20, , hyprctl dispatch pin
        hyprbars-button = rgb(403899), 20,  , hyprctl dispatch fullscreenstate 0 3
        hyprbars-button = rgb(403899), 20,  , hyprctl dispatch fullscreen 0
...

no, this is intended behavior. Buttons always "grow" from the edge they are aligned to

Is there a possibility to add a paramenter to toggle the order, so with the paramenter it would only "move" the buttons as group to the left or right not changing the order from the configuration file?

mrs welcome


cfg highlighting is inconsistent

The newest version has cfg highlighting, essentially graying out the code shielded by an inactive feature gate.
This is a great feature, except that I find it inconsistent.
First I'd load a file, and it would color fine.  Then I'll switch to another file, and it won't color at all.  Then I switch back to the original file, and it'll lose the coloring.
Sometimes it'll suddenly come back after a while, sometimes it won't.
Usually when I restart Visual Studio Code, it'll work fine the first time.  Then not afterwards.

I did some more experiments and it seems to work for any document window already open when I start VS Code.  Any window I open afterwards, or closing the original one and reopening, will cause it not to color.

This is a general problem with diagnostics. I'm not sure why it happens though, it might be a VS Code bug.

Found the issue, and opened a PR that should fix it in https://github.com/rust-analyzer/rust-analyzer/pull/6637


Require withRelated on fetch

This is a question. A workaround would be good, but wanted to make sure I didn't miss anything built-in.
I assume this would apply to Model.load() as well.
Let's say I have a hierarchy like Event --> Room --> Table --> Seat, and a User can own in multiple seats (like a future event reservation or something). I query allSeatsForUser with whatever criteria I need, then try to eager fill the seats so they know the table and room.
.fetchAll( {
      withRelated: [ 'table', 'table.room', {
        'table.room.event': function( qb ) {
          qb.where( 'canceled', false );
        }
      } ]
    } )

I check the events canceled flag, not wanting to show any seats for a cancelled event.
My resulting object looks something like this:
[ {
  "id": 1,
  "tableId": 14,
  "userId": 6,
  "table": {
    "id": 14,
    "roomId": 4,
    "name": "Table 1",
    "seatCount": 5,
    "room": {
      "id": 4,
      "eventId": 2,
      "desc": "Green Room",
      "event": {}
    }
  }
} ]

As expected, the event was canceled and not returned in the query. I want the entire seat to not show up, not just an empty event. Any suggestions?
My options for workarounds are:

Use knex prior to fetching to build the innerJoin chain, with an .andOn() when joining the Event table.
Post-process the returned list and remove from the collection if seat.table.room.event.id === undefined. I would use Promise.reduce to re-build the list before Promise resolution.

Not a pressing matter, but wanted to ask.

Is this behavior something you want consistently across the app? If so you can do it by overriding the toJSON method.

Not across the board, just for this particular query. I like that objects are nested on returning, so want to keep Bookshelf for this query instead of just Knex.
Also, FYI, Promise.reduce won't work on the returned collection, as it isn't an array. I went with workaround 2, collection.toJSON(), then .reduce'd the array like normal.
If there was another trick, let me know! (And if one doesn't quickly come to mind, this issue can be closed)

No need to use Promise.reduce as you're not doing anything async. Collections have a reduce method mixed in from underscore.


Multiple Account - Cookies

Hi @wkaisertexas , this is really nice code, I've been using this for last few months.
I'm doing multiple accounts, creating 1 folder for each accounts (upload.py, cookies.txt, mp4 video), everytime I want to switch account (running from different folder), I need to re-download the cookies from get locally to use it in order to make it work, otherwise the tiktok won't login automatically using cookies.
Is there a way to use the cookies file back and forth?
Thank you

You need one cookies.txt file per account. Your cookies file (among other things) contains a session token that identifies you as being logged in (effectively, this library is one big session token spoofing attack).
However, if you log out of one account and log back into another, logging out will invalidate the session tokens from your previous account.
I solved this in two ways:

use one browser (chrome, safari, edge) per account. You can stay signed in and everything is fine
Instead of signing out of accounts, clear your cookies. This will allow you to sign into a new account without signaling to TikTok that you logged out of your old one.

If you have any more questions, let me know!

Instead of signing out of accounts, clear your cookies. This will allow you to sign into a new account without signaling to TikTok that you logged out of your old one.

This is really simple good solution. Problem solved. Now I can easily manage multiple account.
Thank you!

@shandyisme how did you clear cookies?  maybe you can share code snippet please

Let me clarify this issue because I spent some time on finding the solution.
I thought, that the problem is in the library itself and i debug it but it have not helped me.
The problem was in the way I get cookies. I get cookies using chrome extension. So, when i got cookies for one account, I logged out in order to login in another account and get cookies for it.  It was a mistake because when i logged out from the account, it`s cookies became invalid. Instead of logging out, I had to simply clear cookies in chrome settings and then login into another tiktok account to get cookies.
ps.
when i intentionally clicked on logout button, then tiktok backend was notified, that i am logged, thus cookies became corrupted but when i clear cookies in chrome setting, tiktok backend did not get information, that i am logging out, thus cookies have not became invalid.


Add Tracing

Adding Otel Telemetry from subscribed events to the published events
Also updated to use new loggingx package
Always looking for feedback especially for where to and where not to trace.

You'll want to add the tracing config to the chart values as well. https://github.com/infratographer/load-balancer-api/blob/main/chart/load-balancer-api/values.yaml#L84-L102


Add timeouts support to clientTrafficPolicy

Description:
Add requestTimeout and requestHeadersTimeout to clientTrafficPolicy.
According to Envoy documentation, These fields should be configured in the presence of untrusted downstreams.
Additionally, according to the Envoy best practices, configuring the request_timeout is recommended. It is emphasized that this setting "must be disabled for long-lived and streaming requests". hence, instead of enabling it by default, I propose making this field configurable.
These fields weren't previously discussed in this issue because requestHeadersTimeout was not explicitly mentioned in the best-practices document, and requestTimeout wasn't enforced by default to avoid potential disruptions to certain clients (as discussed above).
Links:

request_timeout and request_headers_timeout in Envoy Docs
Envoy best practices

+1 to this,  can we start off with request_timeout w/o adding the more fine grained request_headers_timeout knob ?
sidenote - it maybe too late to get this into v1.0, added this into the backlog for now.

Please assign to me, I will start with request_timeout

+1 to this, can we start off with request_timeout w/o adding the more fine grained request_headers_timeout knob ?

@arkodg - Maybe we can set a secure-by-default value for request_headers_timeout to mitigate slow-header slowloris? e.g. the envoy-documented example of 10s sounds like a reasonable default timeout for reading headers.

this/default for this timeout is worth discussing in the community meeting, @kflynn and I had a similar discussion around route timeout defaults and we decided to not set one, because its hard to gauge user intent. For this specific case (client request timeout) the context is a little different here, we are dealing with untrusted downstreams, so the decision could be different


Question: difference between this lib and browserify/events?

Hi,
Apart from the support for newer methods, it would be useful to know  - from someone who has is familiar with both implementations - more about the difference between these 2 libs and potential reasons to choose one over the other, since both seem to be equally popular on npm.

events is a 1:1 copy of the built-in Node.js module. eventemitter3 has some differences listed in the README and should be a little faster. I think that's all.

events is a 1:1 copy of the built-in Node.js module. eventemitter3 has some differences listed in the README and should be a little faster. I think that's all.

@brunobertolini

@brunobertolini

I'm closing this as answered. Discussion can continue if needed.


SendOneRequestInstedOfTwo

There is no need to get Salt from database as it's already taken in Account request

Why would you name your pull request in pascal case?

Why would you name your pull request in pascal case?

I confused it with branch naming. Will be solved.


Warnings like "Some weights of the model checkpoint at xlm-roberta-large were not used when initializing XLMRobertaModel"

❓ Questions and Help
What is your question?
When I run "comet-score -s test.en-zh.en -t decoder-out -r test.en-zh.zh", I got the following warnings. Is that normal? or am I missing something?
/root/.cache/torch/unbabel_comet/wmt20-comet-da//checkpoints/model.ckpt
Some weights of the model checkpoint at xlm-roberta-large were not used when initializing XLMRobertaModel: ['lm_head.bias', 'roberta.pooler.dense.weight', 'roberta.pooler.dense.bias', 'lm_head.dense.weight', 'lm_head.layer_norm.bias', 'lm_head.decoder.weight', 'lm_head.dense.bias', 'lm_head.layer_norm.weight']
This IS expected if you are initializing XLMRobertaModel from the checkpoint of a model trained on another task or with another architecture (e.g. initializing a BertForSequenceClassification model from a BertForPreTraining model).
This IS NOT expected if you are initializing XLMRobertaModel from the checkpoint of a model that you expect to be exactly identical (initializing a BertForSequenceClassification model from a BertForSequenceClassification model).
Encoder model frozen.
/usr/local/python3/lib/python3.8/site-packages/torch/nn/modules/container.py:435: UserWarning: Setting attributes on ParameterList is not supported.
warnings.warn("Setting attributes on ParameterList is not supported.")
GPU available: True, used: True
What's your environment?

Linux
python 3.8
Version

I'll see if I can fix the warning but you should not worry because after we load the XLMR from hugging face we reload the final weights from our final checkpoints.
To be fair this is a bit inefficient as we are loading XLMR twice, but what counts in the end are the weights from the checkpoint downloaded from the list of available models.

Thanks for the reply!
I want to ask one more question. When evaluating on English->Chinese, should the source/hypothesis/reference file be tokenized or detokenized?

The text should always be detokenized. We run our own tokenization (in most cases XLM-R tokenizer)

Btw as a sanity check I just tried:
from transformers import XLMRobertaModel
model = XLMRobertaModel.from_pretrained("xlm-roberta-large")

And the warnings are exactly the same. This is the expected behaviour

This is because XLM-R in hugging face is an XLMRobertaForMaskedLM model class. I don't care about the prediction head in COMET and I initialize instead a plain XLMRobertaModel (the base architecture without the LM head)

Great! That helps a lot!


Can't select the theme in Atom

When I go to the drop down menu UI and Syntax Theme,  both material-ui and material-syntax don't show up, even though they're both installed.

I can see both options in your screenshot. "Atom Material"


Version printed as 1.7.0 while using release 0.4

triflejs --version

prints out 1.7.0 while the version installed is 0.4.

Hi @paazmaya.
Thats the version of phantom targetted by TrifleJS.
To get the actual TrifleJS version you can use:
C:\> TrifleJS.exe
triflejs> trifle.version
0.5.0

I'm writing TrifleJS to be a drop-in replacement for phantom. But I see your logic in that its a bit odd seeing 1.7.0 in there. Maybe I'll change this sometime in the future.

Thank you for the explanation, @sdesalas

Hi @paazmaya,
I made changes to the codebase yesterday and incorporated this change. If you download the latest binary you'll get the trflejs version (0.5.0) instead.

NP! :)


Checkpoint size keep on increasing with time

Description:
I was doing experiments with the distributed deployment of the stream processor by enabling both the Periodic and Incremental File-based checkpoints. When I was doing the experiments I saw that the checkpoint size keep on increasing with time. ( I used the checkpoint interval as 1 min and no of revisions as 3 )
I suspect that the old checkpoints don't get deleted
Affected Product Version:
Distributed Deployment of the wso2sp-4.3.0
Additional Information:
I used the following SIddhi Application
@App:name("EmailProcessor-checkpointing")
@App:description("Email Processor benchmark for WSO2 Stream Processor 4.x.x")
@source(type = 'kafka', partition.no.list='0', threading.option='single.thread', group.id="group", bootstrap.servers='localhost:9092', topic.list = 'test15',
@map(type = 'json'))
define stream inputEmailsStream (iij_timestamp long, groupID int, fromAddress string, toAddresses string, ccAddresses string, bccAddresses string, subject string, body string, regexstr string);
@Export('outputEmailStream:1.0.0')
define stream outputEmailStream (iij_timestamp long, fromAddress string, toAdds string, ccAdds string, bccAdds string, updatedSubject string, bodyObfuscated string);
@sink(type='log')
define stream filter1(subject string);
@sink(type='log')
define stream AlertStream(iij_timestamp long, fromAddress string);
@name('query 1')
@dist(parallel='9', execGroup='group3')
partition with (groupID of inputEmailsStream)
begin
from inputEmailsStream
select iij_timestamp, groupID, fromAddress, toAddresses,  ccAddresses, bccAddresses, subject, body, regexstr, convert(time:timestampInMilliseconds(),'long') as injected_iijtimestamp, str:length(body) as body_size
insert into #inputEmailsStream1;
from #inputEmailsStream1 [ regex:find(regexstr, fromAddress) == true ] select iij_timestamp, groupID, fromAddress, toAddresses, ccAddresses, bccAddresses, subject, body, injected_iijtimestamp, body_size insert into #filteredEmailStream1;
from #filteredEmailStream1
select iij_timestamp, groupID, fromAddress, emailProcessorBenchmark:filter(toAddresses) as toAdds, emailProcessorBenchmark:filter(ccAddresses) as ccAdds, emailProcessorBenchmark:filter(bccAddresses) as bccAdds, subject, body, injected_iijtimestamp, body_size insert into #filteredEmailStream2;
from #filteredEmailStream2 select iij_timestamp, fromAddress, toAdds, ccAdds, bccAdds, subject, emailProcessorBenchmark:modify(body) as bodyObfuscated1, injected_iijtimestamp, body_size insert into #modifiedEmailStream1;
from #modifiedEmailStream1 select iij_timestamp, fromAddress, toAdds, ccAdds, bccAdds, subject, str:replaceAll(bodyObfuscated1, 'Kenneth Lay', 'Person1') as bodyObfuscated2, injected_iijtimestamp, body_size insert into #modifiedEmailStream2;
from #modifiedEmailStream2 select iij_timestamp, fromAddress, toAdds, ccAdds, bccAdds, subject, str:replaceAll(bodyObfuscated2, 'Jeffrey Skilling', 'Person2') as bodyObfuscated3, injected_iijtimestamp, body_size insert into #modifiedEmailStream3;
from #modifiedEmailStream3 select iij_timestamp, fromAddress, toAdds, ccAdds, bccAdds, subject, str:replaceAll(bodyObfuscated3, 'Andrew Fastow', 'Person3') as bodyObfuscated4, injected_iijtimestamp, body_size insert into #modifiedEmailStream4;
from #modifiedEmailStream4 select iij_timestamp, fromAddress, toAdds, ccAdds, bccAdds, emailProcessorBenchmark:mostFrequentWord(bodyObfuscated4, subject) as updatedSubject, bodyObfuscated4 as bodyObfuscated, injected_iijtimestamp, body_size insert into #modifiedEmailStream5;
from #modifiedEmailStream5#throughput:throughput(injected_iijtimestamp,"throughput", 3 , 9 , "filter", 5)
select iij_timestamp, fromAddress, toAdds, ccAdds, bccAdds, updatedSubject, bodyObfuscated, injected_iijtimestamp, body_size
insert into #modifiedEmailStream;
-- metrics
from #modifiedEmailStream select iij_timestamp, fromAddress, toAdds, ccAdds, bccAdds, updatedSubject, bodyObfuscated, emailProcessorBenchmark:metrics(bodyObfuscated) as metrics, body_size insert into #metricsEmailStream;
from #metricsEmailStream#window.length(1000) select iij_timestamp, fromAddress insert into AlertStream;
end;

Is the groupID of inputEmailsStream a set of static values or are these dynamic?

groupID of inputEmailsStream are fixed values from 1-9 (9 distinct values which is equal to the number of partial siddhi applications)


Verify accumulator behavior in sui transactional tests

Unfortunately, the transactional tests don't use AuthorityStore, so we have to duplicate some logic here, but I think this is well worth it for the increased coverage.

I believe @tnowacki is looking into making the transactional tests work with the in-memory store, FWIW.


[RFC] Gathering/observing data in the background and foreground

This feature will add background observers for an arbitrary number of types of data. Below are my notes and ideas for how to implement this. Your comments and ideas are welcome.
I propose the following:

The app can be initialized with an additional option: observe
E.g:

let options = {
  permissions: {
    read: ["Height", "Weight", "StepCount", "DateOfBirth", "BodyMassIndex"],
    write: ["Weight", "StepCount", "BodyMassIndex"],
  },
  observers: [
    { type: "StepCount" },
    { type: "Weight" }
  ]
};
AppleHealthKit.initHealthKit(options);

Observers (HKObserverQuery) run in the background, saving data to AsyncStorage, perhaps to the RNAppleHealthkit key. To retrieve it later you could run:

const healthKitData = await AsyncStorage.getItem('RNAppleHealthkit');
const steps = healthKitData.StepCount;
// `steps` might contains something like:
// [
//    { date: '1 1 2019', value: 10000}
// ]

To make use of this most effectively, use react-native-background-fetch

BackgroundFetch.configure({
  minimumFetchInterval: 15, // <-- minutes (15 is minimum allowed)
}, () => {

  // Retrieve data
  const healthKitData = await AsyncStorage.getItem('RNAppleHealthkit');

  // Stop if there isn't any
  if (!healthKitData)  {
    BackgroundFetch.finish(BackgroundFetch.FETCH_RESULT_NO_DATA);
  }

  // Get steps
  const steps = healthKitData.StepCount;

  // Upload them
  await fetch('http://placetouploadsteps.com/steps', { method: 'POST', body: JSON.stringify({ steps });

  // No need to waste storage
  AsyncStorage.removeItem('RNAppleHealthkit');

  // Finish
  BackgroundFetch.finish(BackgroundFetch.FETCH_RESULT_NEW_DATA);

}, (error) => {
  console.log("[js] RNBackgroundFetch failed to start");
});

Event triggers still work, if your app is in the foreground (but with a simpler API)
Per https://github.com/terrillo/rn-apple-healthkit/blob/master/docs/initStepCountObserver().md

import { EventEmitter } from 'rn-apple-healthkit';
EventEmitter.addListener(
  'change:StepCount',
  (evt) => {
    const steps = evt.data;
  }
);

I don't have time to find related page in healthKit docs, but I remember that it isn't possible to use react-native-background-fetch to get healthKit data while app in Background. We have to use a built-in HK mechanism for it.

Yes, that’s true of getting the data. That’s why in step 1 it retrieves the data in the background (using healthkit’s HKOvserverQuery, the built-in mechanism) and stores it to AsyncStorage, because (I’m pretty sure) AsyncStorage is available with background fetch.

AFAIK background fetch doesn't run if the device is locked.


Updated device integration checklist with 'pgrep' alternative.

On some Yocto builds, depending on the image config, 'pgrep' is not
configured to be installed by default as part of busybox recipe.
However 'pidof' is available. Documentation has been updated to
give the user the option of either depending upon their setup.
Changelog: None
Signed-off-by: Dell Green<EMAIL_ADDRESS>

yes indeed, my image is based on a minimal image

Agree we should just drop pgrep entirely. Closed and replaced with #493.


DOC: include ctree in the README

Summary by Sourcery
Documentation:

Add documentation for the dvs ctree command in the README, including its options and usage.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

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

Totals

Change from base Build<PHONE_NUMBER>7:
0.0%

Covered Lines:
1178

Relevant Lines:
1283

💛  - Coveralls


scp3 or sftp3 possible?

The shell session works well. The missing piece is file transfer.
Any plan?

The SCP protocol is deprecated and recent versions of scp also use SFTP.

It is totally possible and would be great ! I would first like to do a small review of the technologies we could use for it though.
SFTP is cool, but maybe something like WebDAV is a good candidate as well. Let me know if you have other ideas.
In the meantime we could imagine coming up with a temporary solution that would only need to be implemented by the client. By doing so, we don't have to update the whole SSH3 protocol right now for "scp3" and only update it when we come up with a solid design.

Meanwhile you might be able to use rsync. Here is an example which uses rsync with netcat:
https://superuser.com/questions/1330291/use-rsync-with-nc-as-transport-layer#1517462
It might be possible to modify the example to use SSH3 with cat over the terminal instead of netcat.

Based on @siepkes's example, I pushed on branch rsync_compat a slight CLI modification to be compatible with rsync and how it calls its underlying remote shell:

rsync calls the remote shell protocol with the -l <username> CLI arg to set the username, so I added that arg to ssh3
rsync does not like us specifying the request path in the remote host url like we classically do in ssh3. I therefore added the -url-path <secret-path> CLI arg on the client specifying the url path to query on the remote host.

Here is an example that works with rsync and ssh3:
rsync -a --info=progress2 --rsh 'ssh3 -privkey ~/.ssh3/id_rsa -url-path /ssh3-inl' user@host:~/remote-dir/ ./rsync_output/

This will copy recursively the directory ~/remote-dir on the remote host and place its content in the ./rsync_output local directory.
It works well, but rsync ends with a broken pipe, I don't know why, maybe SSH3 closes the connection too early. We should investigate but right now the files are copied correctly and it seems to work with several files as well.
Let me know your thoughts !

It works well, but rsync ends with a broken pipe

In my case old rsync version failed with rsync error: sibling process terminated abnormally (code 16) which was caused by ssh3 receiving SIGPIPE. I could not track down why it gets it but its possible to workaround by adding signal.Ignore(syscall.SIGPIPE) to the main.

rsync does not like us specifying the request path ... therefore added the -url-path

It also does not support port (see https://stackoverflow.com/questions/4549945/is-it-possible-to-specify-a-different-ssh-port-when-using-rsync) so I've added -url-port flag, see https://github.com/AlexanderYastrebov/ssh3/tree/rsync_compat2


Error while executing tests

When running C ++ tests, an error occurs. After examining the repository, I realized that before starting the tests, the environment is set up, in which the contracts are deployed, which are used in tests and in the main cmkryvesting contract. All this is implemented in the file https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/common.hpp. I found that the error is happening because the other contracts file is missing. Here is a list of files and contracts that are not in the repository.
dappservices.wasm
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/common.hpp#L333
EOSDTNUTOKEN
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/common.hpp#L337
EOSDTSTTOKEN
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/common.hpp#L338
EOSDTKGTOKEN
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/common.hpp#L339
EOSDTORCLIZE
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/common.hpp#L353
EOSDTCNTRACT
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/common.hpp#L354
provableconn.wasm
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/common.hpp#L356
EOSDTLIQDATR
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/common.hpp#L363
EOSDTGOVERNC
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/common.hpp#L364
EOSDTEXCHANG
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/common.hpp#L365
TOKENSWAP_EQ
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/common.hpp#L366
EOSLOTTERYEQ
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/common.hpp#L367
EOS2DTDOTCOM
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/common.hpp#L373

I tried running the tests, but I got errors again. I found in the code that after connecting the contracts, they are still used. For example:
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/common.hpp#L282
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/common.hpp#L162
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/common.hpp#L538-L54
And in the tests themselves of the contract:
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/vesting_contract_test.cpp#L89
https://github.com/CoMakery/comakery-eos-vesting/blob/main/cmkryvesting_tests/vesting_contract_test.cpp#L217
So I think you still need contract files to run tests.


Answer substitution has problems with (=..)/2 and cyclic terms

I must admit I was currious whether some problems
that bugged Scryer Prolog in the past, also bug
Trealla Prolog. This here works now in Scryer Prolog:
/* Scryer Prolog 0.9.4-193 */
?- Y = [x,Y], W =.. Y.
   Y = [x,Y], W = x([x,Y]).

But a simple repaste test tells me the output in
Trealla Prolog is not correct:
/* Trealla Prolog 2.57.16 */
?- Y = [x,Y], W =.. Y.
   Y = [x,Y], W = x([x,[x,[x,W]]]).

?- Y = [x,Y], W =.. Y, W = x([x,[x,[x,W]]]).
   false.

The 2nd query should succeed if the answer substitution
W = x([x,[x,[x,W]]]) were correct.

Maybe pmoura can create some test
cases for that?

Closed because Ozzy Osbourne birthday and nobody is working on this bug.

Now you're making me paranoid.

It's just a printing issue, i'll try and look tomorrow.

$ tpl
?- Y = [x,Y], W =.. Y.
   Y = [x,Y], W = x([x,[x,[x,Y]]]).

I don't know why it's shows down 3 levels but it's equivalent.


How to make depth estimation consistence for a sequence of images?

Thanks for the great works. I just want to get some idea from you.
Here is: I am running a robot move forward to a chair. I get one frame image per every half seconds. then detect the depth estimation by  the dpt nyu model provided  between robot and chair. The depth suppose to become smaller and smaller. but the actual result does not like that.
For the sample images, It works like below. metric as meter

There are three reasons I guess lead something wrong.

even people hard to see the difference between image 1 and 2. how does the computer know.
It is relative metric for every time. so not work well cross a sequence of images.
Image 4 and 5 is barely have any other reference. so it barely works.

Here are the question:

Any suggestion or reference paper I can take a look to work on depth along with a sequence of image?

Thank you very much!

Your sequence is extremely challenging, you explain quite well why things might not work optimally. Here is a very recent paper that integrates multiple images which I really liked and might serve as a good starting point: https://arxiv.org/pdf/2104.14540.pdf

thanks for the suggestion. I will take a look the paper to see what I can get from there. I am currently trying to use some other dataset with lidar distance to see if it works.

Hi, thanks for the prompt answer!
I take the prediction for the run.py code from the updated MiDasv2 repo. Line 123, so this is before you convert to 16bits resolution and save the depth info. Some more examples give me negative output as well. If you are curious as to where this happens in the image, here is an example. In this image this happens in the change from the person's contour and the sky.

Thanks for pointing this out, was able to reproduce. Negative values seem to happen because of the bicubic upsampling. I never noticed this before. So my statement above about the output being non-negative is only true when the upsampling method is guaranteed to not produce negative values.
Yes, you generally want to treat the prediction like you would treat a disparity. See #19 for sample reprojection code.

Great, thanks!
Glad you could reproduce and find the cause of it.

l find using the model_type dpt_hybrid_nyu reduces the flickering effects  ranftlr for create depth maps good code you have created thanks for the code ranftlr


Bug report: Certain profile screenshots have minor issues

A brief summary of the bug
I have noticed some minor issues with certain profile screenshots. For instance, the profile screenshot for "oyepriyansh" is in dark mode while others are in white mode, creating inconsistency. Additionally, some profile screenshots, like that of "gargakshit," appear small in height, which can appear awkward on certain devices with different resolutions. I know that the screenshot height issue has already been resolved with the recent commit (d88749b), but the old screenshots have not been updated.

Thank you for bringing these issues to our attention! Would you mind creating a pull request by deleting the screenshots from the screenshots directory? This will trigger the GitHub workflow to update the screenshots.


Disable shadowing warning of MSVS2015 for external headers

Boost's spirit shadows its own attr_type and attr declarations.
Silence this warning.
@gck-ableton

This also compiles on MSVS2013.


Fix or work around enum Configurable delegate compiler bug

Reference issue: https://youtrack.jetbrains.com/issue/KT-37503

fixed via explicit cast


Do not show customize properties tab for Libraries, fix Library creation

Resolved / Related Issues
Items resolved / related issues by this PR.

Closes #6909
Closes #6973
Closes #7003

Details of Changes
Add details of changes here.

Hidden customize properties tab for Libraries (#6909)
Added ShellLibrary2 to fix a couple of bugs of Vanara implementation (#6973)
Add a default folder (documents) when creating a new library, otherwise it won't be shown on Files sidebar (#6973)
Added missing string resources for Create Library dialog (#6973)
Check if the path is not null before adding an item to favorites (#7003)

Other:

Do not use StorageFile in Fulltrust logger.. keeps throwing "Wrong thread" exceptions and does not log

Validation
How did you test these changes?

[x] Built and ran the app

Hmm, the library section in the sidebar isn't populating for me in this pr. Any idea why?

Full trust process crash I'd say (no sidebar icons either), you get a log file?

Looks like the exception is occurring when initialization the ftp
From debug.log
2021-11-21 14:09:26.9042|WARN|BuildConnection|Could not initialize FTP connection!
	System.IO.FileNotFoundException: The system cannot find the file specified. (Exception from HRESULT: 0x80070002)
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x1c
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x60
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task) + 0x36
   at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task) + 0x16
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult() + 0x9
   at Files.Helpers.AppServiceConnectionHelper.<BuildConnection>d__8.MoveNext() + 0x13d

Are you sure this only happens with my PR? This looks like the issue some people were getting due to some versions of VS acting up.

No, looks like this happens on main as well. However because of this I can't test your pr.
Also no, nothing gets outputted to the ftp log file at all; not even any of the info logs.

No problem, are you on VS 2019? I think what has been written in the docs: https://files.community/docs/contributing/building-from-source was to save your same issue, @yaichenbaum may remember more.

No I was using 2019. However, it doesn't work on 2022 either.

Have you tried the magic trick if deleting the .nugget folder? :)

Sorry @gave92 for the delay, I'm being overbusy those weeks :/
I will try to expend some free time tomorrow for reviewing this.

Have you tried the magic trick of deleting the .nuget folder? :)

Sorry for the delay, the ftp still didn't work after deleting .nuget.


Fix Asset Library URLs not updating without restarting after been changed in editor settings (Fix #46977)

Fix #46977
Issue :
Changes in Asset Library URLs list in editor settings needs a restart to be applied.
Fix Proposal
Building of the OptionButton has been extracted into a new private function.
This function clears the OptionButton and repopulate it from the Editor Settings.
The function is now called in the constructor and when a NOTIFICATION_EDITOR_SETTINGS_CHANGED is received.
So, changes in settings apply immediatly in Asset Library tab.

Thanks!

Cherry-picked for 3.2.4.


hfinder import error

With the new release of astropy v3.0, hfinder does not load...
hfinder
Traceback (most recent call last):
File "/usr/local/python3_modules/bin/hfinder", line 20, in
from hcam_finder.config import (load_config, write_config,
File "/usr/local/lib/python3.5/dist-packages/hcam_finder/init.py", line 7, in
from .hcam_finder import *
File "/usr/local/lib/python3.5/dist-packages/hcam_finder/hcam_finder.py", line 18, in
from astropy.vo.client import conesearch
ImportError: No module named 'astropy.vo'
http://docs.astropy.org/en/stable/vo/conesearch/index.html
says "Cone Search has been moved to Astroquery 0.3.5 and will be removed from Astropy in a future version."
changing hcam_finder.py...
from astropy.vo.client import conesearch
to
import astroquery.vo_conesearch
works.
Cheers
Pablo.

thanks @paulkerry1
Fixed in https://github.com/HiPERCAM/hcam_finder/commit/7545d87b653d275ef5b0032ac00a6636db5d790f


Bump development version to 0.2.1

Codecov Report
Base: 87.71% // Head: 87.71% // No change to project coverage :thumbsup:

Coverage data is based on head (94bd272) compared to base (af8774e).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #97   +/-   ##
=======================================
  Coverage   87.71%   87.71%
=======================================
  Files          81       81
  Lines        2850     2850
  Branches      366      366
=======================================
  Hits         2500     2500
  Misses        270      270
  Partials       80       80

Flag
Coverage Δ

test-macOS-latest-3.10
?

test-macOS-latest-3.8
?

test-ubuntu-latest-3.10
87.64% <100.00%> (ø)

test-ubuntu-latest-3.8
87.64% <100.00%> (ø)

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

Impacted Files
Coverage Δ

src/pyfx/__version__.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Broom for als

HI @javierluraschi and @kevinykuo ,
I've finished broom for als model. I didn't code for ml_als_factorization() because it'll be deprecated.
Here is a code that you can run:
library(sparklyr)
sc <- spark_connect(master = "local")

movies <- data.frame(user   = c(1, 2, 0, 1, 2, 0),
                     item   = c(1, 1, 1, 2, 2, 0),
                     rating = c(3, 1, 2, 4, 5, 4))
movies_tbl <- sdf_copy_to(sc, movies, name = "movies_tbl", overwrite = TRUE)
model <- ml_als(movies_tbl)

tidy(model)
augment(model)
glance(model)

I don't think we want to implement broom methods for any class that doesn't inherit from ml_model for consistency. Could we introduce a formula interface for ml_als.tbl_spark() somehow so we can signal that we want to create an ml_model_als?
Feel free to open a separate PR to fix the bug with find_constructor() btw, thanks for the catch!

Ok, I'll put the fix in find_constructor() in another PR.
I noticed that als just uses ml_fit() instead of ml_construct_model_.
We could create ml_construct_model_recommendation() inside of  ml_als.tbl_spark() and
create a new_ml_model_als constructor.
Something like that:
new_ml_model_als <- function(pipeline_model, formula, dataset,
                                     features_col) {

  new_ml_model_recommendation(
    pipeline_model = pipeline_model,
    formula = formula,
    dataset = dataset,
    features_col = features_col,
    class = "ml_model_als"
  )
}

what do you think?

But how would we know whether to just call ml_fit() or ml_construct_model_recommendation()? For the other algorithms we have something like
  if (is.null(formula)) {
    stage %>%
      ml_fit(x)
  } else {
    ml_construct_model_supervised(
      new_ml_model_linear_regression,
      predictor = stage,
      formula = formula,
      dataset = x,
      features_col = features_col,
      label_col = label_col
    )
  }

Basically, we need to have a sensible way for users to provide a formula to signal they want an ml_model_als back instead of an ml_als_model.

I added the formula interface for ml_als().
I also did some modifications in ml_recommend(), and now it also works for ml_model_als.
Here is an example:
library(sparklyr)
sc <- spark_connect(master = "local")

movies <- data.frame(
  user   = c(1, 2, 0, 1, 2, 0),
  item   = c(1, 1, 1, 2, 2, 0),
  rating = c(3, 1, 2, 4, 5, 4)
)
movies_tbl <- sdf_copy_to(sc, movies)

model <- ml_als(movies_tbl, rating ~ user + item)

tidy(model)
augment(model)
glance(model)
ml_recommend(model, type = "item", 1)

Will follow up with a separate PR


Add new django migration check

ISSUE 113
NOTE: Due to the issue with the virtual environment (Hook is not aware of the poetry env), I have implemented this as a standalone command.

This is not needed, we always run migrate command when we run the backend image so no need to check this in the CI.


Shader issue on Linux

I built TaxiVis on Linux, everything seems to work except the taxis are not rendered on the map. Terminal shows:
grouop plots 0
QGLShader::compile(Vertex): 0:2(12): warning: extension `GL_EXT_gpu_shader4' unsupported in vertex shader
0:16(17): error: `gl_VertexID' undeclared
0:16(17): error: operands to relational operators must be scalar and numeric
0:16(17): error: condition of `?:' must be scalar boolean

QGLShader: could not create shader
Initializing OpenGL...
Finished loading!

That is weird. I pushed a version of code that should remove this warning, although I believe this should not cause the points to disappear.
Are you using the correct time interval for the points?

I now only get:
grouop plots 0
QGLShader: could not create shader

The points still don't show up. I think the configuration is right, I have non-zero data in the line plot.

Could you tell me which data are you using for this so I can try to check what is going on?

I'm using the default data from this repository (sample_merged_1.kdtrip).

That is weird. In my Linux machine I get

I had the same problem as Remi in some of the early MacOS versions. Huy was able to fix that. But I don't remember what the problem was...

Huy,
Do you remember what was the problem?
On Jan 13, 2016 19:49, "Juliana Freire"<EMAIL_ADDRESS>wrote:

I had the same problem as Remi in some of the early MacOS versions. Huy
was able to fix that. But I don't remember what the problem was...
—
Reply to this email directly or view it on GitHub
https://github.com/ViDA-NYU/TaxiVis/issues/1#issuecomment-171514195.

Remi,
Can you try to pull the last commit I made and tell me if it helps?
The trip location layer was using immediate mode rendering, I just
set it to use VBO for now.

No change.
I see the dots for a second before the map loads, as if it was rendered on top of them.
I also just found out that setting LIBGL_ALWAYS_SOFTWARE=1 fixes it (though "could not create shader" still appears in the log).


Add nullable reference type metadata to Grpc.Core/Grpc.Core.Api

Should add nullable reference type metadata to .NET gRPC libraries: https://devblogs.microsoft.com/dotnet/embracing-nullable-reference-types/
This feature requires C# 8.0. What is the allowed version of C# at the moment?

Right now we are using this SDK:
https://github.com/grpc/grpc/blob/e29f3b27b18717c7c3dba27c3bd4008200f01390/templates/tools/dockerfile/csharp_dotnetcli_deps.include#L2
The SDK would have to be updated in multiple places, but in principle it's doable.
The problem is that some internal projects still only support C# 7.2 so we cannot really upgrade at this point.


Added choosing different dirmap mapping if workfile synched locally

Currently this checks if Site Sync is enabled (globally and for project) and if User has set active_site to local (eg. User is synching files locally). If so, it puts into dirmap root(s) from remote_site as a source and local site as a destination.
It completely ignores configuration in Settings, if anyone could produce use case when this would be needed, please go ahead and share.
(I am not super happy about mixing Site Sync logic with host implementation, but couldn't figure out better option.)

Task linked: PYPE-1941 Run different dirmap for locally synched workfiles


Update helm chart's application version to v1.1.0

@adrianchiris  image versions are added to chart in this PR: https://github.com/k8snetworkplumbingwg/helm-charts/pull/12

/lgtm


[Q] Worth changing to ES6 version?

Question is clear: I have a stable working bot. Should I upgrade to ES6 version of parentbot?

Doesn't matter, they both have the same functionality. I mainly made the ES6 one to  teach myself the new syntax and stuff. Some of the old bot uses ES6 (arrow functions, const, etc).


feat: add log handler to be able to use a custom logger

Add custom logger handler to use a custom Logger instead of printing to console.
Hide PusherChannelsPackageLogger from public API.
Fix logger tests not capturing what was being printed to console.
Remove logTest method as it is not needed anymore.

@nicobritos  Thank you very much! I will mention this contribution in CHANGELOG.md


Creating a pool of channels?

Hi,
I am trying to find a way to create a pool of channels on one connection. As I could not find any examples, I tried to come up with my own implementation. Here it is:
type Publisher struct {
	url           string
	pollDuration  time.Duration
	channelsCount int
	conn          *amqp.Connection
	sessions      []session
	errorCh       chan error
	amqpErrorCh   chan *amqp.Error
}

type session struct {
	channel *amqp.Channel
	confirm chan *amqp.Confirmation
}

func NewPublisher(
	url string,
	pollDuration time.Duration,
	channelsCount int,
) *Publisher {
	p := &Publisher{
		url:           url,
		pollDuration:  pollDuration,
		channelsCount: channelsCount,
		errorCh:       make(chan error),
		amqpErrorCh:   make(chan *amqp.Error),
	}

	go func() {
		for {
			select {
			case <-p.amqpErrorCh:
				p.reconnect()
			case <-p.errorCh:
				p.reconnect()
			}
		}
	}()

	go p.connect()

	return p
}

func (p *Publisher) connect() {
	conn, err := amqp.Dial(p.url)

	if err != nil {
		p.errorCh <- err
		return
	}

	conn.NotifyClose(p.amqpErrorCh)

	p.conn = conn
	p.sessions = make([]session, 0)

	for len(p.sessions) < p.channelsCount {
		ch, err := conn.Channel()
		if err != nil {
			p.errorCh <- err
			return
		}

		p.sessions = append(p.sessions, session{ch, make(chan *amqp.Confirmation)})
	}
}

func (p *Publisher) reconnect() {
	time.Sleep(p.pollDuration)
	p.connect()
}

The basic idea here is to create a number of sessions, which holds an *amqp.Channel and and a chan *amqp.Confirmation as I need to use confirm mode in my project.
A go routine is spawned in NewPublisher() in order to listen to the errors, and re-create another connection and a batch of channels.
I really appreciate it if someone can give me some reviews on this implementation. I can contribute to the examples if this is an appropriate approach. Thanks!

I need something similar and this is one way of solving the problem.  I'm not that familiar with AMQP. @streadway should it not be possible to multiplex multiple publish confirms over a single channel ?.

| What are you trying to accomplish with the pool of channels?
Reliable publishes, this is the abstraction I created for this:
type SynchronousPublisher struct {
	mu               sync.Mutex
	name             string
	channel          *amqp.Channel
	confirmationChan chan amqp.Confirmation
	deadline         time.Duration
}

func NewSynchronousPublisher(name string, conn *amqp.Connection) (*SynchronousPublisher, error) {
	var err error
	sp := &SynchronousPublisher{
		name:     name,
		deadline: 100 * time.Millisecond,
	}
	if sp.channel, err = conn.Channel(); err != nil {
		return nil, err
	} else if err = sp.channel.Confirm(false); err != nil {
		return nil, err
	} else {
		sp.confirmationChan = sp.channel.NotifyPublish(make(chan amqp.Confirmation, 1))
	}

	return sp, nil
}

func (sp *SynchronousPublisher) Publish(msg *amqp.Publishing) error {
	sp.mu.Lock()
	if err := sp.channel.Publish("", sp.name, true, false, *msg); err != nil {
		sp.mu.Unlock()
		return err
	} else {
		select {
		case conf := <-sp.confirmationChan:
			sp.mu.Unlock()
			if !conf.Ack {
				return errors.New("negative ack")
			}
			return nil
		case <-time.After(sp.deadline):
			sp.mu.Unlock()
			return errors.New("deadline elapsed")
		}
	}
}

In this case the channel is locked until the publish is confirmed. So a pool would be needed to do publishes in parallel.

The way that publish confirms work on a channel is that a single protocol message can contain a range of delivery to confirm so that the server can batch a sequence in ACKs in one response. The client library turns that range of acks into one confirm per publish.
In this example the timeout would not consume the ack/nack and there wouldn't be one recv on the confirmation channel for every publish and the channel would need to closed and opened again. I'd recommend re-reopening the channel when you reach the deadline.
Creating a pool of these publishers makes sense to handle independently from AMQP. A basic algorithm to limit the number of in-flight publishings would be something like:
type Pool struct {
	publishers chan *SynchronousPublisher
}

func NewPublisherPool(name string, conn *amqp.Connection) (*Pool, error) {
	const inflight = 10

	pool = Pool{
		publishers: make(chan *SynchronousPublisher, inflight),
	}

	for i := 0; i < cap(pool.publishers); i++ {
		sync, err := NewSynchronousPublisher(name, conn)
		if err != nil {
			return nil, err
		}
		pool.publishers <- sync
	}
}

func (p *Pool) Publish(msg *amqp.Publishing) error {
	pub := <-p.publishers
	err := pub.Publish(msg)
	p.publishers <- pool
	return err
}

Try this extension with reconnecting and cluster supported:
https://github.com/sirius1024/go-amqp-reconnect

I created channel/connection pools and have been in production.
https://github.com/houseofcat/turbocookedrabbit

@streadway do you suggest to create a channel and close for every goroutine, a pooling of channel is necessary when we dont want to keep recreating channel (lets say for an api exposed to publish rabbit mq messages) for every api request, this would impact throughput given atleast extra 4-6 tcp packets has to be transmitted to open and close a channel (connection can be kept single)

@epsilon-akshay channels are logical streams that share the same tcp session. They were intended to isolate some error modes, but the n my uses I find it simpler to have 1:1 channel/connection and even separating the publisher connection from the subscriber.
The library is goroutine safe so you should be able to share the connection and channel between goroutines.
Other than that, your usage will depend on your application.

was mostly thinking from a usage perspective, as a user when i write my apis which does some processing and simply publishes to rabbit mq i would not want to create and close channels per request, rather i would love to have a wrapper dialer which would create a set of channels over a single TCP connection that can be reused and maybe autoconnected (going little ahead here 😄). This would solve for a lot of usecases people would want (similar to how http.Client reuses its connections), But i do understand that the initial though going into the library was to have a 1:1 mapping, was just suggesting this kind of interface also to be provided.


Add support for phpredis sentinel

phpredis starting from 5.2.0 (https://pecl.php.net/package/redis/5.2.0) supports RedisSentinel
https://github.com/phpredis/phpredis/blob/develop/sentinel.markdown#readme

@w1nsun Hey! no plan for now.
Thanks for your contribution, this feature request is now tracked.
Feel free to try to implement it.

What is needed apart from passing the right parameters to the Connection class?

2 years passed. any updates?

No interest from my side because I don't use sentinel myself. PR needs to come from somebody who needs it.

Needed it myself now, so good news: It's supported now since 4.4.0 :)


refactor: better logging

closes #104

Codecov Report

Merging #108 (ad27bc0) into master (6b684c1) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #108      +/-   ##
==========================================
- Coverage   93.86%   93.83%   -0.04%
==========================================
  Files          20       20
  Lines         522      519       -3
  Branches       84       84
==========================================
- Hits          490      487       -3
  Misses         32       32

Flag
Coverage Δ

unittests
93.83% <100.00%> (-0.04%)
:arrow_down:

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

Impacted Files
Coverage Δ

src/command/scheduler.ts
100.00% <ø> (ø)

src/command/on-exit.ts
100.00% <100.00%> (ø)

src/command/post-manager/producer.ts
100.00% <100.00%> (ø)

src/command/qq/consumer-handler.ts
100.00% <100.00%> (ø)

src/services/qq-service.ts
100.00% <100.00%> (ø)

src/services/rss/snowball/service.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6b684c1...ad27bc0. Read the comment docs.


Desktop not responding

Problem Description
After installing the tool the desktop stops responding.
Steps to reproduce the behavior:

Install Mitmproxy6
Restart the system (Kali Linux)
Desktop stops responding.

System Information
Paste the output of "mitmproxy6" here.

How did you install mitmproxy?
Did you configure it to start at system boot? If not, are you sure this is related to mitmproxy at all? Can you consistently reproduce this with a new Kali VM?
If you remove mitmproxy (using something like recovery mode), does the problem disappear?

I installed mitmproxy using apt install mitmproxy.
I can consistently  reproduce this issue
I don't know if the problem disappears after I remove mitmproxy

There is a solution for this problem, install an app from the software app into Kali Linux called "GNOME Desktop Extension". This will fix the problems with the desktop.

Hello, Thanks for the reply,
I installed mitmproxy using apt install mitmproxy.
I can consistently reproduce this issue
I don't know if the problem disappears after I remove mitmproxy
There is a solution for this problem, install an app from the software app
into Kali Linux called "GNOME Desktop Extension". This will fix the
problems with the desktop.
On Tue, May 4, 2021 at 11:04 PM Alexander Prinzhorn <
@.***> wrote:

How did you install mitmproxy?
Did you configure it to start at system boot? If not, are you sure
this is related to mitmproxy at all? Can you consistently reproduce this
with a new Kali VM?
If you remove mitmproxy (using something like recovery mode), does
the problem disappear?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mitmproxy/mitmproxy/issues/4582#issuecomment-832118008,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APVHWE3S5SRTSH3YD47AC6LTMAV3JANCNFSM44DCBOFA
.

I already replied to this on Github, I hope this email helps.
On Wed, May 5, 2021 at 10:09 AM Anirudh Dilli @.***>
wrote:

Hello, Thanks for the reply,
I installed mitmproxy using apt install mitmproxy.
I can consistently reproduce this issue
I don't know if the problem disappears after I remove mitmproxy
There is a solution for this problem, install an app from the software app
into Kali Linux called "GNOME Desktop Extension". This will fix the
problems with the desktop.
On Tue, May 4, 2021 at 11:04 PM Alexander Prinzhorn <
@.***> wrote:

How did you install mitmproxy?
Did you configure it to start at system boot? If not, are you sure
this is related to mitmproxy at all? Can you consistently reproduce this
with a new Kali VM?
If you remove mitmproxy (using something like recovery mode), does
the problem disappear?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mitmproxy/mitmproxy/issues/4582#issuecomment-832118008,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APVHWE3S5SRTSH3YD47AC6LTMAV3JANCNFSM44DCBOFA
.

I already replied to this on Github, I hope this email helps.

GitHub sends notification mails, that wasn't me :wink:
If I understand correctly you've solved the hanging by installing  "GNOME Desktop Extension"? I'm still curious what's causing this, but I haven't found time to set up a Kali VM. Did you use the latest 2021.1 version?

No, it is kali 2020.1
On Thu, May 6, 2021 at 9:00 PM Alexander Prinzhorn @.***>
wrote:

I already replied to this on Github, I hope this email helps.
GitHub sends notification mails, that wasn't me 😉
If I understand correctly you've solved the hanging by installing "GNOME
Desktop Extension"? I'm still curious what's causing this, but I haven't
found time to set up a Kali VM. Did you use the latest 2021.1 version?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mitmproxy/mitmproxy/issues/4582#issuecomment-833616664,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APVHWE7VSJ4XN4W6EKEX2HLTMKYZVANCNFSM44DCBOFA
.


Is there a plan for "RocketMQ dledger version"  to support batch msg?

In RocketMQ 4.5.0 dledger version, batch msg is  not supported, may i ask what's the plan and release timepoint?

@lahahana yes, the community is working on this part of work and will be released in the near future.

@lahahana yes, the community is working on this part of work and will be released in the near future.

@lahahana yes, the community is working on this part of work and will be released in the near future.

may I known what's impaction that batch msg will cause if "4.5.0 dledger version" take it in?


Pull request to test os test actions ⚗

tests running correctly, on commits, pull request no longer required for testing


Go to Top Button Update for raylib Site

This update adds the following feature to the raylib website:
-A brand new go to top button that auto hides whenever the top is reached. It works for both mobile and desktop.
-A new file called ui_manager.js. This file will be responsible for all future ui changes, including, but not limited to:
the new mobile navigation menu, and the new navigation menu for the cheatsheet page.

Thanks for the improvement! And thanks for the review to more self-contained small PRs! It's easier to review and keep track of the changes this way!


context argument doesn't work in some methods

`    function diffChars(oldStr:string, newStr:string):IDiffResult[];
function diffWords(oldStr:string, newStr:string):IDiffResult[];

function diffWordsWithSpace(oldStr:string, newStr:string):IDiffResult[];

function diffJson(oldObj: Object, newObj: Object): IDiffResult[];

function diffLines(oldStr:string, newStr:string):IDiffResult[];

function diffCss(oldStr:string, newStr:string):IDiffResult[];

function createPatch(fileName: string, oldStr: string, newStr: string, oldHeader: string, newHeader: string, options?: {context: number}): string;`

some methods seem don't have context argument. However, those method should have according to your API readme.

Looks like this report is about the TypeScript type definitions contained in DefinitelyTyped, which aren't part of jsdiff nor maintained by its maintainers (though perhaps jsdiff should have its own type definitions!). Also looks like this has at least partly fixed by now (perhaps completely - I didn't check).

Separate issue for making type definitions part of jsdiff: https://github.com/kpdecker/jsdiff/issues/303


Allow for a 4 char suffix on the end of hostnames

As we migrate cloud deployments to use a load balancer backed by managed instance groups, we can no longer count on machine names to be predetermined. MIG instances have a "-<4 char>" suffix, and are managed by the instance group, not M-Lab. In order to support this use case, the changes in this PR allow for a suffix to be part of the hostname without failing Parse(). Callers can use the suffix, if found, or not.

This change is

Pull Request Test Coverage Report for Build 1211

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 94.432%

Totals

Change from base Build 1208:
0.002%

Covered Lines:
2154

Relevant Lines:
2281

💛  - Coveralls


Rewrite of lambdas and local definitions, with various bug fixed added in.

Because of the large conflict potential and important bug fixes, this is going in early.


App crash

Future detectFaces(InputImage inputImage, pickedFile) async {
try {
if (pickedFile != null && pickedFile.path.isNotEmpty) {
  final List<Face> faces = await _faceDetector.processImage(inputImage);

  if (faces.length == 1 && faces[0].trackingId != null) {}
}catch(e){}

final List faces = await _faceDetector.processImage(inputImage);
this process image function causing this issue
While uploading some high quality images it crashes the app , i tried try catch also but its keep crashing my app.
Any solutions for this ?????

Your TryCatch block won't catch native exceptions.
Try runZoneGuarded to see what is the problem:
runZonedGuarded<Future<void>>(() async {    if (pickedFile != null && pickedFile.path.isNotEmpty) {      final List<Face> faces = await _faceDetector.processImage(inputImage);      if (faces.length == 1 && faces[0].trackingId != null) {}    } }, (error, stack) => print(error));

I had the same problem with text recognition on iOS. I tried to runZoneGuarded but did not get any information from there. Are there other ways to debug this?

I had the same problem with text recognition on iOS, too

I had the same problem with text recognition on iOS, too
P.S : I have tried on iphone 7, now, changed device to iphone 13 and have updated lates version, everything works okay

What abt Android??


A \ in a link seems to create 2 links in MD instead of one

The problem was that word broke the link into multiple runs inside a hyperlink. This was fixed in #2


Docker in docker rootless is flakey

This image doesnt seem to be well maintained.
The 19 version apparently doesnt work at all and they dont seem to  have any CI to indicate this.
The 20 version afaict uses vfs which is very sub-optimal to say the least

im wondering whether to remove the running in docker section from the docs for now as using it seems to be sub-optimal to point of unusable


The URL of Echo360 has changed.

The URL of Echo360 has changed from org.au to net.au that causes the extension cannot detect the site. I would be appreciate it if you could assist me. Thank you in advance.

I'm facing the same issue:

Hi all,
We've released a fix for this in v0.3.16, please reinstall or wait for the update to come to your browser.
Please be aware that as most of the core developers are not studying any more, fixes and patches to geckodm may be delayed or unimplemented.

@ericjiang97 Hello, thank you for your assistance. The chrome extension can detect the site now but it cannot detect video. I have already click button "LOAD LECTURES ON PAGE" for sure, however, there are still not any videos appeared to download.
If anyone could help me please.
Thank you in advance.

Having the same issue, site can be detected but no videos loaded for
downloading.
On Sun, Jul 4, 2021 at 11:21 PM wanachy @.***> wrote:

@ericjiang97 https://github.com/ericjiang97 Hello, thank you for your
assistance. The chrome extension can detect the site now but it cannot
detect video. I have already click button "LOAD LECTURES ON PAGE" for sure,
however, there are still not any videos appeared to download.
If anyone could help me please.
Thank you in advance.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GeckoDM/GeckoDownloadManager/issues/82#issuecomment-873610856,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUWURUVEUEU7TBKFAOPMEXLTWB4BDANCNFSM47ST3ULA
.

--
Warmest regards,
Isabella Lin 林淑玲
@.* | +65 9639 5478 (Singapore) | +95<PHONE_NUMBER>
(Myanmar)

Hi all,
I'm guessing that there's been an API change. Unfortunately as most of the core devs aren't studying at uni anymore, so we don't have access and able to implement a fix.
However feel free to make a PR and we will be happy to review it.

Hey everyone,
Another update. GeckoDM will not be maintained further, support via issues and emails will ending by September 2021.
See https://github.com/GeckoDM/GeckoDownloadManager/discussions/85 for more info.


Can bypass #[repr(packed)] checking since rust 1.57

Since rust 1.57 it is possible to put proc macro attributes after derive (see https://github.com/rust-lang/rust/issues/81119)
so given proc macro crate like that:

use proc_macro::TokenStream;
use syn::{parse_macro_input, parse_quote, ItemStruct};
use quote::quote;

#[proc_macro_attribute]
pub fn add_repr(_:TokenStream, item: TokenStream) -> TokenStream {
    let mut item = parse_macro_input!(item as ItemStruct);
    item.attrs.push(parse_quote!{#[add_repr_hidden]});
    quote!{#item}.into()
}

#[proc_macro_attribute]
pub fn add_repr_hidden(_:TokenStream, item: TokenStream) -> TokenStream {
    let mut item = parse_macro_input!(item as ItemStruct);
    item.attrs.push(parse_quote!{#[repr(packed)]});
    quote!{#item}.into()
}

and following usage :
#[pin_project]
#[add_repr]
pub struct Test {
    x: i32
}

__PinProjectInternalDerive won't notice that at the end of the expansion there is #[repr(packed)] on struct
Of course, --cap-lints=warn is still required to actually compile it
I'm not sure how probable it is to trigger this problem in real code base..

As well as https://github.com/taiki-e/pin-project-lite/issues/26, it should eventually be fixed by unaligned_references becoming a hard error.

unaligned_references is now a hard error 🎉: https://github.com/rust-lang/rust/pull/102513


Add supporting scripts for running automation in Azure Batch

Add supporting scripts for automation

Build scripts for build both product and test codes.
Application scripts for dispatch and set up tests in Azure Batch.

Update product test codes

Make tests themselves packages.
Remove docstring in test methods so that when listing tests using
nosetests, their method names are exposed correctly.

Codecov Report

Merging #3893 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3893      +/-   ##
==========================================
+ Coverage   70.91%   70.91%   +<.01%
==========================================
  Files         428      429       +1
  Lines       27051    27059       +8
  Branches     4119     4119
==========================================
+ Hits        19182    19190       +8
+ Misses       6618     6614       -4
- Partials     1251     1255       +4

Impacted Files
Coverage Δ

scripts/batch/app/schedule.py
100% <100%> (ø)

...li-cloud/azure/cli/command_modules/cloud/custom.py
14.45% <0%> (ø)
:arrow_up:

...nent/azure/cli/command_modules/component/custom.py
16.23% <0%> (ø)
:arrow_up:

...dback/azure/cli/command_modules/feedback/custom.py
31.25% <0%> (ø)
:arrow_up:

src/azure-cli-core/azure/cli/core/util.py
70.06% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e39f7a9...78f1a7f. Read the comment docs.


Invoking "screen" from Ubuntu bash shell causes scrollbar (and buffer?) to disappear.

Windows Terminal version
1.14.1962.0
Windows build number
10.0.19044.1826
Other Software
Screen version 4.08.00 (GNU) 05-Feb-20
Steps to reproduce
Start a new Ubuntu bash shell in Windows Terminal.
run "ls -la" several times until the content scrolls off the screen.  Note the presence of the scrollbar on the right, and the ability to use the mouse wheel or the scrollbar to scroll back through the content.
Start "screen" via "screen -D -RR".
In the screen session, perform the same "ls -la" steps above.  Note the absence of the scrollbar on the right, and the inability to use the mouse wheel or the scrollbar to scroll back through the content.
Expected Behavior
One should be able to scroll back through content independent of the program being run or the TERM environment variable setting (normal bash: xterm-256color; screen: screen.xterm-256color).
Actual Behavior
One loses the ability to scroll back through content once inside "screen".

Yep, that's just how the alternate screen buffer (\x1b[?1049h) works. You'll see this is pretty consistent in most terminal emulators - for example, gnome-terminal:

Admittedly, Windows Terminal didn't support this correctly at all until very recently.

However, both iTerm2 and VSCode support this feature well~

Just a note on this:
This issue was affecting my work as I mostly work with screens.  After I don't know what release update it suddenly didn't work, I started using Hyper as an alternative, but because it crashed 3 times now in critical times I decided to try and "fix" this issue with the Windows Terminal.
After I rolled back to v1.13.11432.0 the issue was not there, so something is changed around the new releases.

https://stackoverflow.com/a/1125947
This has worked for me. I did exactly what the SO answers suggestst and now I can scroll in screen sessions from Windows Trerminal.


PMComplex miss a signBit method (used in Pharo) Fix

Pharo 6.1 builds are red on another random test: #70
Pharo 7.0 builds are green now for #68

I don't know exactly ... In Pharo 7.0, arcTan: has been redesigned to use signBit instead of sign (in order to detect IEEE-754 negative-zero). PMComplex miss this method and test like PMComplex>>testArcTanDenominator. Maybe you have a better explanation ? :-)

Standards (ANSI Smalltalk?) mandates that 0 sign -> 0, same for +0.0 and -0.0...
So, if we want to be compatible with other Smalltalks (Gemstone in this case) we must not use sign for detecting negativeZero... We collectively opted for signBit after a discussion (I did that in Squeak too).
But that applies to numbers, not complex numbers...
A complex number has a sign for its real part and its imaginary part too, so choosing one is quite arbitrary...
Introducing functions/operations that are not natural, but are rather hacks for making some other part of a software work is dangerous.
One good example is how Matlab defined relational operators for complex (< <= >= >).
There is no natural order, one such that  (a < b and: [c < d]) ==> (a+c < (b+d)).
My experience is that programs instead of failing when we have a complex instead of a real just continue and answer a wrong result...
For what added value? I know of a single usage for sorting complex values: separate stable poles from unstable poles in some algorithms (for example solving a Riccati). For a discrete or continuous system, that's not necessarily the same sort function... (either real part > 0, or module > 1).
It's far better to define the specific sort order where we need it in this case.
That's why I ask for complex sign...

I don't want to add a sign on Complex ... there was already a sign method and there was a failing test :
testArcTanDenominator
	| c1 c2 |
	c1 := 1 i.
	c2 := 0.
	self assert: (c1 arcTan: c1) equals: Float pi / 4.
	self assert: (c2 arcTan: c1) equals: 0.
	self assert: (c2 arcTan: c1 * c1) equals: Float pi

I dunno who wrote this test ?

If I remove signBitmethod, I have this DNU:

Yes, I understand that.
Locally, we had/used a sign, we must now have/use a signBit instead for latest Squeak/Pharo compatibility.
So on one hand, problem solved, issue should be closed.
But on the other hand, while at fixing it, I'm saying that rather than hacking a sign/signBit that will not be usefull outside of arcTan:, and even worse might be surprising in other contexts, and/or hide bugs or delay bug discovery, we might better fix the complex extension of arcTan:
How is 4-quadrant arc-tangent at all defined in Complex?
a + b i arctan: c + d i

Does complex extension fits real function when imaginary parts are null?
(a + 0 i arcTan: c + 0 i)  = (a arcTan: c).

What happens when one or both imaginary part is a negativeZero?
I might be able to answer to above questions since I probably implemented those...
But does the arcTan: implemented in Number works the same as Complex extension when a Complex parameter is provided? Does it use double dispatching for that?
(a arcTan: c + d i)  = (a + 0 i arcTan: c + d i).

Are there tests for all these cases?

I thought PMCompelx>>arcTan: was your code ?

Can you move discussion on a new issue ?

Yes, I once thought this could be discussed in a new issue.
But if Number>>arcTan: has not been patched for being compatible with complex extension of arcTan:, as I think then the correct fix is:

either to remove the test (or change it as shouldRaise: Error) and Complex>>sign
or to fix Number>>arcTan: to make it compatible with Complex>>arcTan:, for example by using double dispatching (but the change must be made in Squeak/Pharo because that's not a PM method)


Undeclared type parameter in _def_tail_length

https://github.com/TuringLang/ParetoSmooth.jl/blob/b9790dc40cfd4ce415ac32da10b040ee5810e414/src/ImportanceSampling.jl#L280

I have literally no idea how this hasn't errored, but it's fixed now.


No tooltips on FloatingActionButton

Would be nice to have the tooltip available on any button component.

#2230


Middle Click Pastes in Linux

Because middle-clicking pastes in Linux, this will cause whatever is in the clipboard to be dumped out whenever I try to scroll. If you have some way to disable that with your extension that would be very helpful. Otherwise, I cannot use your extension without repeatedly running xclip -i /dev/null to clear out the clipboard.

Works for me! I'll see what I can do.

It could be something as simple as setting the mousedown event options to passive: false and adding e.preventDefault() to the startScroll function

Turns out it wasn't quite that simple, but I put a pull request in. I'm a little new to using Github so let me know if I missed an important step.

This should be fixed in v0.6.0


Http2 sidecar 1.1.4

git checkout 1.1.4
git cherry-pick e7a2b2b2d79123f4368c5d809d8bd14488caf130 # Hardcode max-concurrent-streams
git cherry-pick 666f411e05b1df254bbcef3351edfbb3bd3f1df3 # Make prow work with pusher infra
git commit -m "h2sidecar plugin adds alpn h2 on inbound clusters"

@samuelyallop-pusher: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
e545bd17de7e8c6294bae4929e377436d85599e5
link
/test istio-unit-tests

Full PR test history

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@samuelyallop-pusher: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
e545bd17de7e8c6294bae4929e377436d85599e5
link
/test istio-unit-tests

prow/istio-presubmit.sh
e545bd17de7e8c6294bae4929e377436d85599e5
link
/test istio-presubmit

Full PR test history

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

 [3] Istio 1.1: Service mesh proper

/retest

@samuelyallop-pusher: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
e545bd17de7e8c6294bae4929e377436d85599e5
link
/test istio-unit-tests

Full PR test history

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@samuelyallop-pusher: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
6b2e956b355e2a1c7ac805b214c453339df109b3
link
/test istio-unit-tests

Full PR test history

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


nixos/lorri: disable ProtectHome

Description of changes
Lorri is mostly used to interact with directories in $HOME
Fixes https://github.com/nix-community/lorri/issues/98

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

ping lorri maintainers
@grahamc @Profpatsch

wow It seems someone opened  a PR for the exact same problem minutes before I did: #275024
I'll take a moment to ponder upon the likelihood of this coincidence...


Cannot run meteor evaluation

I cannot download the data required for meteor evaluation. The following is the error message.
[W 210731 14:42:32 data:54] <local path> not found -- downloading https://github.com/GEM-benchmark/GEM-metrics/releases/download/data/meteor.tar.gz.
 This may take a few minutes.
[W 210731 14:42:33 meteor:21] Cannot run Meteor -- Skipping: HTTP Error 404: Not Found

<local path> is redacted.

I fixed this by manually copying to <local path> the content in this folder.
This should be the right one, though it'll still be good if we needn't this extra manual step.

@lxuechen Thanks for noticing! Looks like that file got removed from our list of downloadable files by accident. I added it back, now it should auto-download without any problems. Feel free to reopen if it still doesn't work for you.


Scripts for create release. Sign code for open source (Debug)

You can uncomment line with creating dmg in build.sh, need install npm package for it

thanks a lot!


Add support for expanded storage

Closes #54

Sorry about the formatting changes. I have my editor configured for work and didnt realize it was formatting the code till I had already finished with the implementation. I can go through and undo the formatting if you would like.
As for it making the code more complex, I don't see any way around this. I have a few mods I maintain and I have had to add compat in the past and this is how you do it. I could try and use pure FabricAPI hooks to try to simulate the logic, but it would probably be prone to errors since they don't have a specific event for opening and closing containers.
An alternative would be to add the two event's to ExpandedStorage's public API so that Quicksort wouldn't need to mixin ES's internal types, but there would still be the optional dependency on the mod for access to the API. I would be happy to see if they would accept a PR for adding the public API to their mod if you would feel more comfortable with this approach. This would be safer since you would be depending on their public api.
To be honest, I will probably need to maintain a fork for myself since I use this on my personal server which runs Create Mod which runs on 1.20.1 right now, so I am going to need to backport the code. I just figured I would contribute since you mentioned it was something others had asked for in the past. :)

Hey. Thanks a lot for the detailed and thoughtful response. I took another look.
I do think your refactor here to make things more event-based makes sense. And there are indeed some other folks who would benefit from the expandedstorage support (I'm just not sure how many).
My main thing is I just don't want to have to test and maintain the code against a growing list of other mods (especially when I don't even use those mods myself).
But I'd be happy to merge this as an unsupported feature (i.e., unsupported by me); I'd rely on you or other folks to keep it working. That would probably be easier for you than maintaining your own fork.
So, I think I will merge this into for now into a feature stabilization branch for now so I can clean up the whitespace diffs and a few other things, and then eventually merge it into main.
Thanks for the contribution!

Awesome, thanks! Again, sorry about the formatting changes! I am usually the one yelling at people for changing the formatting in code ;)


[TEVA-2687] Send email and destroy inactive jobseeker accounts

Jira ticket URL
https://dfedigital.atlassian.net/browse/TEVA-2687
Screenshots of UI changes:

Ignore the comments. Just re-read the ticket and realised the email is sent after 5 years and the account is destroyed two weeks later. Also realised 5.years.ago - 2.weeks is 5 years and 2 weeks ago.


feature request: ability to sort the files first

Did you check existing requests?

[X] I have searched the existing issues

Describe the feature
the file tree should list file files first and directories last. it allows easier file access.
Provide background
No response
Additional details
No response

I've added an API for configuring the sort order. In your case you will want
require("oil").setup({
  view_options = {
    sort = {
      { "type", "desc" },
      { "name", "asc" },
    }
  }
})


Issue with numberParser when price is .5

Hi I am facing an issue when we pass .5 the number returns 5.00 instead of 0.50. Can we fix that?
var price = .5
console.log(numberParser(price))

Thanks, this is already being tracked in #443


Convert array access with curly braces to array access with square brackets

Today I learned that
<?php
$array = ['key' => 'value'];

var_dump($array{'key'});

is valid syntax.
And while the PHP manual actually documents this

Both square brackets and curly braces can be used interchangeably for accessing array elements

I think that using curly braces for accessing array elements is wrong.
It would be great if php-cs-fixer had a fixer to replace $array{'key'} with $array['key'].

It is great: php-cs-fixer fix --rules=normalize_index_brace and you are already using it.

Sorry for the noise.

no worries @sebastianbergmann , thanks for raising the issue and nice to see it resolved as well :)


cannot decode CBOR data with unknown structure containing map with []uint8 keys

What version of fxamacker/cbor are you using?
v2.3.0
Does this issue reproduce with the latest release?
Yes
What OS and CPU architecture are you using (go env)?
linux/amd64
What did you do?
I've encountered some CBOR in the wild (as part of Cardano testnet block data) that cannot be easily decoded using this library. The data defines a map with a bytestring ([]uint8) key, which Go generally does not allow and this library explicitly disallows it. The data does not have a predefined structure, so I can't easily work around this by mapping it to a map[string]... via a struct.
The data in question can be found at this cbor.me link
What did you expect to see?
A successful decode
What did you see instead?
cbor: invalid map key type: []uint8

I wouldn't exactly call this a bug, but it's also not quite a feature request. I don't have a good suggestion for how to handle this use case.

Thanks for reporting this.

I wouldn't exactly call this a bug, but it's also not quite a feature request.

Yea, me too.  It's not a bug because Go doesn't allow []uint8 as map key.

I don't have a good suggestion for how to handle this use case.

Hmm, I need to think about it some more when time allows so I'll leave this open.  There are some possibilities but they might turn out to be bad ideas.

It seems that an array with a fixed length (vs. a slice) can be used as a map key in Go, so perhaps creating the map as map[[7]uint8]... in cases where the array length is known (and consistent) could be a viable workaround. That seems awfully niche, though.

As a workaround, I was able to define the structure far enough to create an empty UnmarshalCBOR() function for this particular part of the data to just "ignore" it. It's not ideal, but I can live with it for now.
The other part is trying to figure out which set of data structures to use for a given chunk of CBOR to allow this workaround to actually work, which is the focus of #317

How would you feel about a decoder flag to enable optional behavior to encode a byte string map key using either string or a hex encoded string representation (only when the destination is a interface{})? I can take a crack at implementing something like that.

I have some new features planned for a future release (v2.5 or v2.6) that might be useful in resolving some or all of this issue.

How would you feel about a decoder flag to enable optional behavior to encode a byte string map key using either string or a hex encoded string representation (only when the destination is a interface{})? I can take a crack at implementing something like that.

I understand why this approach is being considered.  However, some aspects of it might introduce edge cases that could be problematic for round-trip encoding/decoding. For example, CBOR text string requires valid UTF-8 strings.

I started work on an attempt to address this in a fork last night. I'm using a struct instead of just using a string (but it's still using a string internally so that the struct is still comparable), so it will be easier to handle the round trip edge cases. I hope to have something working this weekend.

What are the features that you have planned for 2.5/2.6? All I really need is the ability to parse CBOR that contains map with bytestring keys without having to predefine the structures. It's not that important to me what form that parsed bytestring takes, as long as the CBOR can be parsed and I can access the original bytestring data.

I created PR #319 with my proposed fix for this issue. However, it cannot currently run the CI workflows due to me being a first-time contributor and it needing approval

I ended up closing the PR due to apparent lack of interest, but I created a wrapper object that can use the ByteString type from #319 when parsing arbitrary CBOR in order to solve my problem. This can be seen in cloudstruct/go-cardano-ledger#16


ADMX policy "Control which extensions are installed silently" Reapplies every sync

ADMX policy "Control which extensions are installed silently" Reapplies every sync which causes the extensions listed in policy to be removed and then reinstalled, this causes aditional popups since many extensions opens a new tab upon installation.
Observed version the seems to be affected.
Public: 103.1264.37 (64bit) version.
Canary: 105.1304.0 (64bit) version.
Edit: add:
After rollback to version 102.0.1245.30 the problem no longer exists, so it seems the problem occured somewhere between these versions

@Temilit Hello - This issue was inherited upstream from Chromium, we are expecting a fix to be included in the next Stable channel release.  For additional information, please see this discussion in the MS Edge Enterprise Tech Community: https://techcommunity.microsoft.com/t5/enterprise/quot-some-extensions-are-not-allowed-quot-after-last-update/m-p/3560964/highlight/true#M4693


Support using multiple ---volume-drivers on swarm

Hi, I am opening this docker issue  here since it is supported in swarm mode but not in swarm classic.

Thanks @suryagaddipati  I am closing this one.

@allencloud why was this closed? Is this currently supported in swarm classic?

Oh, where is my coffee?
I think it is my bad to close this. Sorry for that.

@allencloud could you reopen this issue .

Yes, of source.


chore: remove package lock

There are a few more package-lock.json files:
examples/tracer-web/package-lock.json
getting-started/example/package-lock.json
getting-started/monitored-example/package-lock.json
getting-started/traced-example/package-lock.json


LICENSE question

Hey folks!
I stumbled across this project recently as I was investigating how to get Vortex working in WASM. One of the things I wanted to do was to show how to feed Vortex data into Datafusion so you can run queries directly in the browser. This codebase is really helpful and I wanted to use some sections of it, specifically around the Datafusion and the file_reader.
However, I noticed there's no LICENSE for this code, so I wanted to check how the authors feel about me reusing some of the code for my own open source project. If possible it'd be great to add a LICENSE file to the repo!

Here you go: https://github.com/XiangpengHao/parquet-viewer?tab=readme-ov-file#license

Thank you!


Prints usage instructions

Refer to instructions when I forget how to use the app
(Link to the Github user guide or have a short summary similar to address book level 3)

Done! Also add ESC as shortcut key to close the help window

@fanwgwg Which PR completes this issue?

Still in my chagne-ui branch, it is the help window


Support materialized VIEW in Redshift

Support materialized VIEW in Redshift

Note: materialized views were supported in the past, but it looks like they were deprecated, most likely due to issues round schema binding. Thought I'd share this context in case someone decides to take a stab at it again; it may still be possible to make them work.


Correct Windows build defects noted with recent version updates

Description:
Picks up from https://github.com/envoyproxy/envoy/pull/10433
windows build regression...

Pick up curl-7_69_1
Patch curl to revert change to cmake_minimum_required

latest version of curl added a maximum version of cmake 3.16

Pick up cares-1_16_0
Toggle crosstool build to avoid possibly injecting wrongly assumed
flags (/MD or -MD dynamic linkage override of our /MT static build)
Note that some issues were pre-existing but not previously triggered

Risk Level: Low
Testing: Local on MSVC
Docs Changes: n/a
Release Notes: n/a

Assigning to @junr03 to potentially help sort out the c-ares issues.

From the ASAN failure in https://dev.azure.com/cncf/4684fb3d-0389-4e0b-8251-221942316e06/_apis/build/builds/35151/logs/68
It looks to me that this is a bug in c-ares:
https://github.com/c-ares/c-ares/blob/077a587dccbe2f0d8a1987fbd3525333705c2249/ares_getaddrinfo.c#L548-L553
end_query frees hquery but the code then proceeds to keep using hquery. Which is exactly what the ASAN run is catching.

@mattklein123 this patch makes no changes to affecting quiche or the ActiveQuicListenerTest, as OS/X reports. We should move ahead with this patch to unblock the curl update.

This PR didn't end up updating c-ares from the looks of it @wrowe :
    com_github_c_ares_c_ares = dict(
        sha256 = "de058ad7c128156e2db6dc98b8a359924d6f210a1b99dd36ba15c8f839a83a89",
        strip_prefix = "c-ares-1.16.0",
        urls = ["https://github.com/c-ares/c-ares/releases/download/cares-1_16_0/c-ares-1.16.0.tar.gz"],
    ),

You are correct. We began by updating c-ares in the same lot as curl. However, as noted above, c-ares 1.16 introduced new regressions that must be addressed before it is ready for master.

(From the comments above, it appears c-ares regressions are upstream code defects, not envoy's use of the library.)


Flamethrower breaks mermaid.js

Hello @fireship-io,
I like to use a this great project, but I could not get mermaid diagrams working with it.
I tried to embed flamethrower into Zola blog, and managed to get TOC working, but I could not get mermaid diagrams rendered - I had to hit the refresh button in a browser for mermaid diagram to appear, despite flamethrower documentation saying scripts inside body reload, so I had to revert the change, despite adding flamethrower gave really good user experience overall.
Commit history: https://github.com/reference-architecture-ai/reference-architecture.ai/commit/9f79341244c4b69c0b6731621afc32578d632507
Ways to replicate:
Add example below into home and similar into about, switch between tabs. Observe markdown is not rendered.
<!DOCTYPE html>
<html lang="en">
<head>
  <meta charset="utf-8">
</head>
<body>
  <div class="mermaid">
  graph LR
      A --- B
      B-->C[fa:fa-ban forbidden]
      B-->D(fa:fa-spinner);
  </div>
 <script src="https://cdn.jsdelivr.net/npm/mermaid/dist/mermaid.min.js"></script>
 <script>mermaid.initialize({startOnLoad:true});
</script>
</body>
</html>```

I guess because the body onload event (or whichever mermaid uses) is not fired when navigating between the pages.
you would need tell mermaid, that a navigation happened, use this code instead of the mermaid.initialize:
// this should work on first load and also on navigation:
mermaid.init();

That worked, thank you very much. With your help, I turned my Zola website into super fast.