Eagle 376 skip false failing hbase related UT code

These faulty UT cases constantly failed because the embeded hbase environment cannot be started up properly, thus they have produced many false failures for long and wasted a lot of building time. Now skip them so that the normal UT cases can expose true issues if there is any.

@anyway1021 I found the following log in Jenkins log
Results :
Failed tests:
testSlideWindow1(org.apache.eagle.alert.siddhi.TestSiddhiSlideWindow)
Tests in error:
testTimeSlideWindow(org.apache.eagle.alert.state.TestSiddhiStateSnapshotAndRestore)

@qingwen220 this PR and jira ticket is not for fixing problems of siddhi, it's just for skipping hbase related UT cases. The failure you pasted is out of the scope of it. I'll try to research and fix them in another thread.
Could you re-consider your judgement based on the above? Thanks.

LGTM for this pr


Alexa announce make one echo show play music randomly

Hi,
Since I use multinotify,  an announcement make one of my alexa play music (the last radio I listened to)
I use it like this :
in the automation :
service: script.multinotify
data:
message: Début nébulisation
alexa_target: media_player.partout (is a group of 3 echos)
alexa_type: announce
In the logs, I have this :
Last multinotify Alexa announcment changé en Début nébulisation.
13:00:07 - Il y a 10 minutes
Media players - Event - Backup volume_level déclenché par l'événement 'state_changed'
13:00:05 - Il y a 10 minutes
Media players - Event - Backup volume_level déclenché par l'automatisation balcon nebulisation déclenché par l'heure pattern
13:00:04 - Il y a 10 minutes
Media players - Event - Backup volume_level déclenché par l'automatisation balcon nebulisation déclenché par l'heure pattern
13:00:01 - Il y a 10 minutes
Media players - Event - Backup volume_level déclenché par l'automatisation balcon nebulisation déclenché par l'heure pattern
13:00:01 - Il y a 10 minutes
Management of Alexa/TTS devices BEFORE the announcment éteint déclenché par l'automatisation balcon nebulisation déclenché par l'heure pattern
13:00:01 - Il y a 10 minutes
Media players - Event - Backup volume_level déclenché par l'événement 'state_changed'
13:00:01 - Il y a 11 minutes
Management of Alexa/TTS devices BEFORE the announcment activé déclenché par l'automatisation balcon nebulisation déclenché par l'heure pattern
13:00:00 - Il y a 11 minutes
Management of Alexa/TTS devices BEFORE the announcment started déclenché par l'automatisation balcon nebulisation déclenché par l'heure pattern
13:00:00 - Il y a 11 minutes - Historique des exécutions
Management of Alexa/TTS devices BEFORE the announcment éteint déclenché par l'automatisation balcon nebulisation déclenché par l'heure pattern
13:00:00 - Il y a 11 minutes
Media players - Event - Backup volume_level déclenché par l'événement 'state_changed'
13:00:00 - Il y a 11 minutes
Management of Alexa/TTS devices BEFORE the announcment activé déclenché par l'automatisation balcon nebulisation déclenché par l'heure pattern
13:00:00 - Il y a 11 minutes
Management of Alexa/TTS devices BEFORE the announcment started déclenché par l'automatisation balcon nebulisation déclenché par l'heure pattern
13:00:00 - Il y a 11 minutes - Historique des exécutions
Management of Alexa/TTS devices BEFORE the announcment éteint déclenché par l'automatisation balcon nebulisation déclenché par l'heure pattern
13:00:00 - Il y a 11 minutes
Media players - Event - Backup volume_level déclenché par l'événement 'state_changed'
13:00:00 - Il y a 11 minutes
Management of Alexa/TTS devices BEFORE the announcment activé déclenché par l'automatisation balcon nebulisation déclenché par l'heure pattern
13:00:00 - Il y a 11 minutes
Management of Alexa/TTS devices BEFORE the announcment started déclenché par l'automatisation balcon nebulisation déclenché par l'heure pattern
13:00:00 - Il y a 11 minutes - Historique des exécutions
Ensure DND initialization for multinotify éteint déclenché par l'automatisation balcon nebulisation déclenché par l'heure pattern
13:00:00 - Il y a 11 minutes
Ensure DND initialization for multinotify activé déclenché par l'automatisation balcon nebulisation déclenché par l'heure pattern
13:00:00 - Il y a 11 minutes
Ensure DND initialization for multinotify started déclenché par l'automatisation balcon nebulisation déclenché par l'heure pattern
13:00:00 - Il y a 11 minutes - Historique des exécutions
Multinotify - Smart, rich and powerful notifications activé déclenché par l'automatisation balcon nebulisation déclenché par l'heure pattern
13:00:00 - Il y a 11 minutes
Multinotify - Smart, rich and powerful notifications started déclenché par l'automatisation balcon nebulisation déclenché par l'heure pattern
13:00:00 - Il y a 11 minutes - Historique des exécutions
The last activity of the echo which has the problem, is tunein radio. But it's not playing while it makes an annoucement, so, it should not restore tunein
EDIT: for what it's worth, the problem happens only on echo show 5 with fire os <IP_ADDRESS> (and there is no update), while others echo show running <IP_ADDRESS> have no problem.
EDIT 2: it seems that this bhavior has to see with the restore music functionality, cause when I do a simple annoucement like this, I have no problem :
service: notify.alexa_media
data:
message: test
target: media_player.evrywhere
data:
type: announce
EDIT 3:
I tried with spotify (I played spotify on it, then stopped it.)
Then I use the card you provide to send a message to a specific echo show, and sent a message to it, and after the message, the echo played spotify.
EDIT 4 and FINAL edit:
Problem is with alexa media integration, even when player is stopped, it keeps in "playing" state. After reloading Alexa media player integration, it works.
So sorry for this
Thanks

Hi Energywave,
thanks for the explanation. Definitively, it's so sad that some alexa media player bugs have a big impact on other integrations, like yours, and for sure, alexa media player is used by so many users, I can't understand why, if the author is not interested anymore, there isn't someone  willing to maintain it.
By the way, i noticed last version is from june 29th, so there is some kind of updates.
I create an automation to reload the integration every night, but it's just to limit the bugs, not even a workaround.
Thanks for your good work on multinotify, I definitively love it. If you plan to make it even better, I have some ideas,. The DND mode is really good with the force option, maybe an "alternative media player" option would be nice with the "force" option.
Anyway, thanks very much, hope that alexa media player bugs will be fixed so that we can enjoy and use your integration as it deserves.

Thank you for your kind words! I didn't understand your idea about DND. Please explain it better so that I can evaluate as I hope to find time to write some improvements that I have in my mind, I evaluate everything.

Sure, let me explain : The DND mode prevent from receiving anything, but we could reroute the TTS or announcements to another device. This way, we could announce on every device during day, and only to a specific device (if param "force" is set to false) during night. So that our wifes can sleep and we, guys, still receive the messages on our desk device :)

Hi, another suggestion : actually, tts is used with an established language (IT). I changed it on the file, but it would be usefull to be able to change the language directly via a variable on the form of an automation. I know it's possible cause when I use TTS google_translate, I can specify a language.  Maybe not usefull for everyone, but in many cases, different languages can be spoken in a house (I speak french, my wife spanish) :)


[BUG] QDial not Styled

Description
The QDial does not have correct styles, leading to too dark notches and too light background styling in the dark theme. The light theme is better, but potentially could be made better.

Update: this only affects Qt6. In Qt5, they're styled quite nicely.

The new example style (on the dark theme) in dial.py is provided here. The slight pixelation isn't noticeable at the scale provided, and isn't an issue with the app. This is because I use QPosF and QRectF with exact positioning and drawArc, drawLine and drawEllipse, so it's to the highest precision as Qt allows me to do.


Is it possible to serialize Desc struts?

Thanks for all the work on this great library! This may be more of a question than an issue..
I have two use-cases:

RigidBodyDesc has a lifetime as mentioned in #205. I have a queue of these descriptions that would be used to create bodies later in the frame. It's a bit painful to enqueue this object because of the lifetimes.
I'd like to store body/collider descriptions in some sort of data file, likely pulled in using serde. I haven't looked at serde yet so perhaps it supports this, but I imagine if the lifetimes weren't there, it would be more straightforward to serialize structs like RigidBodyDesc.

My first thought is that additional helper structs could easily be implemented (even potentially in a separate crate) that are reference-free and explicitly designed with serialization in mind, but I was wondering if I was missing something that might make that step unnecessary.
Another option might be to break apart the RigidBodyDesc.. so for example there would be types (not a serious naming suggestion):

ColliderDesc
RigidBodyDescWithoutColliders
RigidBodyDescWithColliders<'a>

There would need to be some way to combine them like:
fn create_rigid_body_desc<'a>(colliders: &'a Vec<ColliderDesc>, body: &'a RigidBodyDescWithoutColliders) -> RigidBodyWithColliders<'a>

But maybe there is already a good solution that would make something like this unnecessary?

Hi! Actually, the next version of nphysics which will be released at the end of next week or the week after will no longer have any lifetimes in any Desc structures, except for the descriptors of deformable bodies. For example, here is what the RigidBodyDesc will look like: https://github.com/rustsim/nphysics/blob/ccd/src/object/rigid_body.rs#L804-L823
Regarding serialization, keep in mind you will have some difficulty for serializing collider descriptors since there is currently no way of serializing an Arc<Box<Shape<N>>> because it's a trait-object. See that issue. So you will have some custom serialization code to write to handle the collider shapes.

Thanks, sounds great!


CIAB: Fix the environment variable and cron jobs

What does this PR (Pull Request) do?

Fix the environment variable in trafficops-go. SSL keys can not be set without X509-related environment variable.
Fix the traffic_ops_ort path from /usr/bin/traffic_ops_ort to /usr/local/bin/traffic_ops_ort because the python version is upgraded

No procedural change, so no tests or doc or CHANGELOG changes needed.
Which Traffic Control components are affected by this PR?

CDN in a Box

What is the best way to verify this PR?

Start CIAB
The cached demo page can be retrieved.
Enter mid or edge container. There should not any error in /var/log/ort.log

If this is a bug fix, what versions of Traffic Ops are affected?

master
3.0.0
Basically every active branch

The following criteria are ALL met by this PR

[x] This PR includes tests OR I have explained why tests are unnecessary
[x] This PR includes documentation OR I have explained why documentation is unnecessary
[x] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[x] This PR includes any and all required license headers
[x] This PR ensures that database migration sequence is correct OR this PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3556/
Test PASSed.


[Ansible] Update Windows psw_archive_url and dcap_archive_url

Update download urls for the PSW and DCAP Windows installers
Small adjustment for the path of the SGX driver archive

Fixes https://github.com/microsoft/openenclave/issues/1617

bors r+


👾 Implement Performance Statistics Collection.

The following tasks are related to collecting performance statistics from the questionnaire:

[x] Implement addStatistics function to create fetch request.
[x] Implement addPerformanceStatistics endpoint.
[x] Collect data required performance statistics relation.

This PR was completed in #29.


Drop the title-pattern processor in favour of outputting title values instead

I'm seriously considering deprecating and removing the layout:title-pattern processor.  It was originally a feature request from way back (#9), and it's a really good idea because it saves devs a whole lot of trouble when we merge titles.  However, it's become quite the maintenance nightmare! :confused:
I'm not gonna leave developers out in the cold though.  My proposed alternative to this would be to make the titles available as normal variables within the processing context of the page so they can be accessed by any Thymeleaf processor.  eg:
Content.html
<title>My page</title>

Layout.html
<title th:text="|${layout.decoratorTitle} - ${layout.contentTitle}|">My website</title>

Processors will be able to access the titles via this special layout object/map.  I can also use that object as a place to pass processing information to developers who might want any info that comes out of the layout dialect for their own uses, maybe even their own dialects (communication between dialects?  :astonished: )
This gives developers the ability to do whatever they want with the titles, and standard attribute merging rules (content element attributes override layout element attributes during decoration) will keep the 'title-pattern on the content page will be used over the title-pattern on the layout page).

Question.  In this kind of situation:
Content.html
<title th:text="${layout.decoratorTitle}>My page</title>

Layout.html
<title th:text="|${layout.decoratorTitle} - ${layout.contentTitle}|">My website</title>

What constitutes the "decorator title"?  Is it "My website", or is it the result of the th:text processor? (which is actually supposed to be the resulting title!)

No, I don't think I can make this one work because of the situation mentioned in my first comment.  Dropping for now, removing from the 2.0 milestone, might revisit in future.


Allow unified user definitions parser to restart

Similar to previous PR: https://github.com/microsoft/Power-Fx/pull/1153
Restart no longer works with named formulas under the unified parser. This PR fixes that and allows UDFs to continue processing as well if the parser encounters an error.

✅ No public API change.

✅ No public API change.


Typo in src/util/types.ts

Version
5.1.0
Steps to reproduce
It seems to a quick report of an easy-fix bug, so pardon me for not providing reproduction steps.
What is expected?
Successful compilation.
What is actually happening?
Error: node_modules/echarts/types/dist/echarts.d.ts:6391:14 - error TS1005: ']' expected.
6391     [key in ${ComponentMainType}Index]: number;
~
node_modules/echarts/types/dist/echarts.d.ts:6391:38 - error TS1005: ';' expected.
6391     [key in ${ComponentMainType}Index]: number;
~
node_modules/echarts/types/dist/echarts.d.ts:6391:39 - error TS1128: Declaration or statement expected.
6391     [key in ${ComponentMainType}Index]: number;
~
node_modules/echarts/types/dist/echarts.d.ts:6392:1 - error TS1128: Declaration or statement expected.
6392 } & {
~
node_modules/echarts/types/dist/echarts.d.ts:6392:3 - error TS1109: Expression expected.
6392 } & {
~
node_modules/echarts/types/dist/echarts.d.ts:6393:29 - error TS1005: ',' expected.
6393     [key in string]: unknown;
~m

Hi @shijieyao . It's not a typo. It's because we are using typescript template string feature which is introduced in<EMAIL_ADDRESS>It's not expected and will break the project using older typescript versions. We will fix it ASAP. Thanks for the feedback

I see. Thanks for the prompt reply!

Same issue here, wish for resolve

I have the same issue, also with the last update 5.1.1

@Abattista96 5.1.1 should have fixed it. Do you mind providing your reproducible code?

@pissang  Already fixed in my project, thanks.

@Abattista96 5.1.1 should have fixed it. Do you mind providing your reproducible code?

I updated the lib version from 4.9 to 5.1.0, now to 5.1.1.
When i try to compile first time after the update i receive the error from some node_module unfinded, from the second compile the code give me the same error above and fail to compile in all cases. I really don't know what to do


use SDK functions for decoding & encoding uint64

https://discord.com/channels/754286650540032030/754286650540032033/854032882778767370

https://discord.com/channels/754286650540032030/754286650540032033/854032882778767370

https://discord.com/channels/754286650540032030/754286650540032033/854051475733872660


koji_build: allow to set allowed_commiter to maintainers

Description
According to https://packit.dev/docs/configuration/downstream/koji_build#optional-parameters
the parameter allowed_commiters  defaults to an empty list.
When I want to use this job without pull-from-upstream and without using PRs, I had to specify at least one maintainer. And rather all maintainers for this package. But when list of maintainers change, I will have to update the config file.
That is inconvinient.
I would love to have some magical id, that would match all maintainers. So I can do something like:
allowed_committers: ['all_maintainers', 'packit']

Or even specify FAS groups:
allowed_committers: ['@copr-sig', 'packit']

Benefit
No response
Importance
No response
Workaround

[X] There is an existing workaround that can be used until this feature is implemented.

Participation

[ ] I am willing to submit a pull request for this issue. (Packit team is happy to help!)

Thanks @xsuchy for creating this.
I quite like the FAS group approach, but both can work quite well.
Not sure how easy it is to programmatically get the members of a FAS group compared to the list of package maintainers.

I would say both would make sense and should not be hard to implement. For the FAS groups, we can utilise the already used fasjson client (see).

https://src.fedoraproject.org/api/0/#projects-tab
Check /contributors/ and /hasaccess/
And then https://src.fedoraproject.org/api/0/#users-tab
https://src.fedoraproject.org/api/0/#groups-tab

This was brought up again during our workshop as something that could help with the UX.

all_admins sounds good to me. What about all_committers as well? That's another level of contributors with right to push directly to a repository to any branch.

@frenzymadness makes sense to me :+1:


Support for HTMLCollection

Make your constructor support HTMLCollection (returned by document.getElementByXXX() functions).
Currently, it seems to only support array of elements ;)
https://developer.mozilla.org/en-US/docs/Web/API/HTMLCollection

Closing for now - will reopen if more people request it.
Thanks

I'm using const choiceSelects = Array.from(document.getElementsByClassName('choices-select')) to pass an array of elements, although that doesn't seem to work either.


Update the issuer selection code to match topology

Between the initial draft of the JSON schema for stashcp and the final one in topology, there was some drift.  This updates the stashcp side to match the current contents of https://topology.opensciencegrid.org/osdf/namespaces

Merging this to move forward the credential request work.


feat (if-else): add '@ngx-utilities/ngx-if-else-loading'

closes #4
<ng-container *ngxIfElseLoading="items | async; let items; withMessage 'Loading items...'">
  ...content
</ng-container>

Codecov Report

Merging #8 into master will increase coverage by 0.55%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
+ Coverage   84.05%   84.61%   +0.55%
==========================================
  Files           9       10       +1
  Lines          69       78       +9
==========================================
+ Hits           58       66       +8
- Misses         11       12       +1

Impacted Files
Coverage Δ

src/app/app-routing.module.ts
100% <100%> (ø)
:arrow_up:

src/app/app.module.ts
100% <100%> (ø)
:arrow_up:

...else-loading/docs-ngx-if-else-loading.component.ts
83.33% <83.33%> (ø)

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 7349db2...26e51a2. Read the comment docs.


Upgrades ZwaveJS2Mqtt to v5.7.2

Proposed Change:
https://github.com/zwave-js/zwavejs2mqtt/releases/tag/v5.7.2

Hi there @MikeAg03!
Thanks for the PR, but could you please test things before you open a PR?
Because if you did test it, you would have learned that this version breaks the add-on.
This has been fixed upstream and we are awaiting a new release.
So, please... test your stuff!


[Bug] Non Reply Mention is broken

NRM.plugin.js stoped working with the last update of discord it needs to be updated

Discord pushed an update that broke all client mods. While BD is a hotfix, all plugins are now broken and now need to be fixed by the individual plugin developers

forgive me, i had to go to the creator's github, sorry i don't use github


watch() updates

I am using a stream select(db).watch(). When I insert new rows, the stream emits the tables as expected. But when I update(db).write(newData), it does not react. I confirmed that my query affects at least 1 row.
Is it intended? If so, how can I use a stream to watch any of insert, update, delete and others?

I saw the code of moor_flutter, and I found that this should work as expected. But it does not right now.
I am attaching the codes in case it helps.
import 'package:moor_flutter/moor_flutter.dart';
import 'package:singleton_class_annotation/singleton_class_annotation.dart';

part 'app_info_database.g.dart';

class AppInfos extends Table {
  TextColumn get packageName => text()();
  TextColumn get appName => text()();
  BoolColumn get enabled => boolean().withDefault(const Constant(false))();

  @override
  Set<Column> get primaryKey => {packageName};
}

@SingletonClass()
@UseMoor(
  tables: [AppInfos],
)
class AppSettingDatabase extends _$AppSettingDatabase {
  AppSettingDatabase()
      : super(FlutterQueryExecutor.inDatabaseFolder(path: 'pdbox.sqlite'));

  Stream<List<AppInfo>> getAppSettingsStream() => select(appInfos).watch();

  Future updateSetting(AppInfo info) {
    return (update(appInfos)..whereSamePrimaryKey(info)).write(info);
  }

  Future<int> getNumApps() async =>
      (await customSelectQuery('SELECT COUNT(*) as count from appInfos;')
              .getSingle())
          .readInt('count');

  Future insertNewApp(AppInfo info) async {
    bool ret = true;
    try {
      await into(appInfos).insert(info);
    } catch (e) {
      ret = false;
    } finally {}
    return ret;
  }

  Future transactionStart(Function exec) async {
    return transaction(() async {
      if (exec != null) await exec();
    });
  }

  @override
  int get schemaVersion => 1;
}

I get a stream using getAppSettingsStream(), and update a table with updateSetting().

Can you confirm your updateSetting call actually updates the database?
Future<int> updateSetting(AppInfo info) async {
  int rowsAffected = await (update(appInfos)..whereSamePrimaryKey(info)).write(info);
  print('updateSetting affected $rowsAffected rows');
  return rowsAffected;
}

Enabling sql statement logging also helps when debugging: FlutterQueryExecutor.inDatabaseFolder(path: 'pdbox.sqlite', logStatements: true).
Are getAppSettingsStream() and updateSetting() being called on the same AppSettingDatabase instance?

Thanks @Mike278. After your comment, I found the typo which makes the app creating new instance of AppSettingDatabase. This problem is gone now! Appreciate!

It will be great to improve this package to be a singleton style preventing facing the issue like this.


merge package from arlac77/npm-package-template-esm-only

package.json

chore(package<EMAIL_ADDRESS>chore(scripts): cover@#overwrite c8 --temp-directory build/tmp ava && c8 report -r lcov -o build/coverage --temp-directory build/tmp

.travis.yml

chore(travis): remove node versions 12.1.0
chore(travis): add node versions -10,-11,-12,12.2.0

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Bug] Be coredump in  vectorization  mode

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
1.1
What's Wrong?
WARNING: Logging before InitGoogleLogging() is written to STDERR
I0830 18:25:37.866526 33719 env.cpp:46] Env init successfully.
*** Aborted at<PHONE_NUMBER> (unix time) try "date -d @1662105709" if you are using GNU date ***
*** SIGSEGV address not mapped to object (@0x5622) received by PID 33719 (TID 0x7f61513bb700) from PID 22050; stack trace: ***
0# doris::signal::(anonymous namespace)::FailureSignalHandler(int, siginfo_t*, void*) at /be/src/common/signal_handler.h:420
1# 0x00007F6488BF5400 in /lib64/libc.so.6
2# memcpy at /be/src/glibc-compatibility/memcpy/memcpy_x86_64.cpp:219
3# doris::vectorized::ColumnString::insert_range_from(doris::vectorized::IColumn const&, unsigned long, unsigned long) [clone .part.0] at /be/src/vec/columns/column_string.cpp:83
4# void doris::vectorized::MutableBlock::mergedoris::vectorized::Block&(doris::vectorized::Block&) at /be/src/vec/core/block.h:373
5# doris::vectorized::HashJoinNode::_hash_table_build(doris::RuntimeState*) at /be/src/vec/exec/join/vhash_join_node.cpp:1101
6# doris::vectorized::HashJoinNode::open(doris::RuntimeState*) at /be/src/vec/exec/join/vhash_join_node.cpp:1072
7# doris::PlanFragmentExecutor::open_vectorized_internal() at /be/src/runtime/plan_fragment_executor.cpp:286
8# doris::PlanFragmentExecutor::open() at /be/src/runtime/plan_fragment_executor.cpp:259
9# doris::FragmentExecState::execute() at /be/src/runtime/fragment_mgr.cpp:242
10# doris::FragmentMgr::_exec_actual(std::shared_ptrdoris::FragmentExecState, std::function<void (doris::PlanFragmentExecutor*)>) at /be/src/runtime/fragment_mgr.cpp:483
11# std::_Function_handler<void (), std::_Bind_result<void, void (doris::FragmentMgr::(doris::FragmentMgr, std::shared_ptrdoris::FragmentExecState, std::function<void (doris::PlanFragmentExecutor*)>))(std::shared_ptrdoris::FragmentExecState, std::function<void (doris::PlanFragmentExecutor*)>)> >::_M_invoke(std::_Any_data const&) at /home/disk6/palo/release/baidu/bdg/doris/ldb_toolchain/include/c++/11/bits/std_function.h:291
12# doris::ThreadPool::dispatch_thread() at /be/src/util/threadpool.cpp:578
13# doris::Thread::supervise_thread(void*) at /be/src/util/thread.cpp:407
14# start_thread in /lib64/libpthread.so.0
What You Expected?
run well
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

https://github.com/apache/doris/issues/12410


destroy GEOSBufferParams

fixes #1438

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

11 of 11 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 85.812%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
3006

Relevant Lines:
3503

💛  - Coveralls


Modern Search & Cloud Hybrid Search

How does this works with Modern Search in Office 365. Does it allow to add a new vertical for SP 2016 results to appear as a separate tab in Modern Search results page? Like "All", "Files", "Sites", "People", "News" and "SP2016".

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

ID: d441302d-3491-cfbe-c1a9-1478ece0e49f
Version Independent ID: 45a81d7d-3397-7b18-733a-a70c095f9ca6
Content: Cloud hybrid search service (Cloud SSA) FAQ
Content Source: SharePoint/SharePointServer/hybrid/cloud-hybrid-search-faq.md
Product: sharepoint-server-itpro
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@srinivasamyneni Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@srinivasamyneni Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Open contours

Can this code/algorithm be modified to work with open contours? E.g. a curve rather than a closed shape.

Hi,
In general, no, it cannot be done. However, if your open curve has fixed end-points (i.e., you know where the curve starts and ends) then it might be possible to simulate this behavior using a closed contour that passes through the end-points and updating only the fragment of the contour between the end-points. Whether this will work or not depends on your particular problem.

That's an interesting idea, I may give that a try. Thanks for the advice.


Add Cursor Tail Effector done ! #652

Hello, PR
Add new Feature (Cursor Tail Effector done) - #652
Done -
https://github.com/user-attachments/assets/49f1adad-ef9a-4f3b-80a7-2345b98e7d3b
📄 Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (adds or updates related documentation)

✅ Checklist

[x] I am a participant of GSSoC-ext.
[x] I have followed the contribution guidelines of this project.
[x] I have made this change from my own.
[x] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have added documentation to explain my changes.

Mandatory Tasks

[x] Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

🤝 GSSoC Participation

[x] This PR is submitted under the GSSoC program.
[x] I have taken prior approval for this feature/fix.

@YadavAkhileshh sir🙏🏻 kindly upgrade the level 2 to 3,
sir I add a new feature & sir if you check other projects they also give 3 for this..

Thank You 🙏🏻

@YadavAkhileshh sir kindly upgrade the level 2 to 3 🙏🏻

@YadavAkhileshh sir please upgrade the level 2 to 3 🙏🏻


[stable/owncloud] Update dependencies for owncloud

Signed-off-by: tompizmor<EMAIL_ADDRESS>
What this PR does / why we need it:
Update dependencies for the chart.
Which issue this PR fixes

fixes #

Special notes for your reviewer:
Checklist

[X] DCO signed
[X] Chart Version bumped
[X] Title of the PR starts with chart name (e.g. [stable/chart])

/retest


docs: Fix incorrect function typo in events system.

CZO
Fixes:

Screenshots and screen captures:
Before:

After:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @tnmkr! Kinda odd that we managed to mix up the function names there :)
Glad to see you're reading this document, in any case!


BB YL5ZAHD1 - file:%2F%2F

Description
Fix BB finding YL5ZAHD1
path traversal: file:// is blocked, file:%2F%2F is not.
Your Environment
Todo: Add transformation function t:urlDecodeUni to existing rule 931130.

it's also necessary to add REQUEST_URI to match
curl http://localhost/file:%2f%2fetc/passwd

it turns out that we've a problem with Apache phase:2 and REQUEST_(FILENAME|URI|etc...). I don't want to share too much on public issue.

fix is merged


[CT-2178] Handling column names having / while altering the table

Is this a new bug in dbt-snowflake?

[X] I believe this is a new bug in dbt-snowflake
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
We had few columns with "/" in the column names. During incrementa, run, when dbt runs the alter statement to either add or drop a column it is failing with the below error in the global macro columns.sql
ERROR_MESSAGE
SQL compilation error:
syntax error line 3 at position 12 unexpected '/'.
image
Fix
Modified global macro to add proper quoiting to the column names during alter table statement.
Expected Behavior
We expect that the alter statements to add or drop columns, must run successfully without any failure. Basically if we have added a column in the previous layer(say load layer), and during incremental load in the next layer(raw layer) it should run the alter statement and add the column to the raw table. Please note it failed because it has a character "/" in the column name.
Steps To Reproduce

Add a column to a table in load layer with a name like "KOND/CC"  (with a /)
Do an incremental run so that this column should be added in the next layer which uses this table as source.
Alter statement to add the column is triggered and it fails with error mentioned above
As a fix, we need to include the column within double quotes in the macro "snowflake__alter_relation_add_remove_columns" as we are using Snowflake.
Upon fix it will run the alter statement successfully and add that column to the table.

Relevant log output
Log output with error:
2023-02-16T17:23:47.3708425Z 17:23:47  Database Error in (models/raw/cust.sql
2023-02-16T17:23:47.3714555Z 17:23:47    001003 (42000): SQL compilation error:
2023-02-16T17:23:47.3720917Z 17:23:47    syntax error line 3 at position 12 unexpected '/'.
2023-02-16T17:23:47.3727341Z 17:23:47    syntax error line 3 at position 16 unexpected '/'.

Log output after fix:
2023-02-20T12:39:08.9562443Z 12:39:08  121 of 1023 OK created incremental model qa_raw.cust .......... [SUCCESS 0 in 193.62s]

Environment
- OS: Windows 10
- Python: 3.8.6
- dbt-core: 1.3.1
- dbt-snowflake:1.3.0

Additional Context
No response

This was reported for a few different adapters:

dbt-postgres: https://github.com/dbt-labs/dbt-core/issues/4422
dbt-redshift: https://github.com/dbt-labs/dbt-core/issues/4422#issuecomment-1414127699
dbt-snowflake: https://github.com/dbt-labs/dbt-adapters/issues/63

Acceptance criteria
Implementation detail was originally described here: https://github.com/dbt-labs/dbt-core/pull/7025#issuecomment-1606152211
In short:

Use {{ adapter.quote() }} as described in http://dbt-labs/dbt-core#4423%20(comment)
Add test case(s) to the on_schema_change tests for a column that contains a special character (/, ", etc.) or is a reserved word, e.g. from.
Inherit these test cases within each adapter (dbt-snowflake, etc).

Its been too long I have opened this issue, now I don't even remember where to track back. Now I am not clear on all the comments, so please let me know if this can be published or not ? And what should be my next step towards this? TIA.


Runner issues when starting Microsoft Edge

Microsoft PowerToys version
0.74.1
Installation method
WinGet
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
Betriebssystemname:                            Microsoft Windows 10 Pro
Betriebssystemversion:                         10.0.19045 Nicht zutreffend Build 19045

When Windows and Powertoys start, the first time the runner is given the text "edge" in order to find Microsoft Edge, it correctly shows the application (with icon), as well as an "Aplication Package" without icon. Selecting either of them starts the Edge browser.
But after that first time, when the runner is given the text "edge", it shows both entries "application" and "application package" (both without icon). And the "application" cannot start the Edge browser, displaying the error message "start not possible: C:\Users<user>\AppData\Roaming\Microsoft\Windows\StartMenu\Programs\Microsoft Edge.lnk". Edge can only be started by selecting the entry "application package", but the icon is not shown.
First run:

Second run:

Error shown when selecting the application:

✔ Expected Behavior
Only one entry to be displayed for Edge browser (with correct icon), every time the text edge is given
❌ Actual Behavior
2 different entries for the browser are displayed, one without icon, and the one with icon only works the first time after windows start. After that, it shows an error message
Other Software
Edge Version 118.0.2088.46 (Official build) (64-bit)

Attached the runner log found in AppData\Local\Microsoft\PowerToys\RunnerLogs:
runner-log_2023-10-17.txt

Are you still experiencing this issue? /needinfo


Add env tag in benchmark metrics

What does this PR do?
Add env tag in benchmark metrics
Motivation
To separate debug test metrics and release metrics
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 69.97%. Comparing base (91a98e4) to head (c546b72).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2276      +/-   ##
===========================================
- Coverage    70.06%   69.97%   -0.09%
===========================================
  Files          732      732
  Lines        27227    27226       -1
  Branches      4580     4579       -1
===========================================
- Hits         19076    19050      -26
- Misses        6877     6889      +12
- Partials      1274     1287      +13

see 32 files with indirect coverage changes


Error when running cctyper: sed: cannot rename /sed78a7GM: Permission denied

After running INFO: Running HMMER against Cas profiles , the job will instantly terminate and I receive the following error:
sed: cannot rename <output filename>/sed78a7GM: Permission denied Traceback (most recent call last):   File "/opt/miniconda3/envs/cctyper/bin/cctyper", line 83, in <module>     hmmeri.main_hmm()   File "/opt/miniconda3/envs/cctyper/lib/python3.8/site-packages/cctyper/hmmer.py", line 26, in main_hmm     self.load_hmm()   File "/opt/miniconda3/envs/cctyper/lib/python3.8/site-packages/cctyper/hmmer.py", line 85, in load_hmm     hmm_df = pd.read_csv(self.out+'hmmer.tab', sep='\s+', header=None,   File "/opt/miniconda3/envs/cctyper/lib/python3.8/site-packages/pandas/util/_decorators.py", line 211, in wrapper     return func(*args, **kwargs)   File "/opt/miniconda3/envs/cctyper/lib/python3.8/site-packages/pandas/util/_decorators.py", line 331, in wrapper     return func(*args, **kwargs)   File "/opt/miniconda3/envs/cctyper/lib/python3.8/site-packages/pandas/io/parsers/readers.py", line 950, in read_csv     return _read(filepath_or_buffer, kwds)   File "/opt/miniconda3/envs/cctyper/lib/python3.8/site-packages/pandas/io/parsers/readers.py", line 605, in _read     parser = TextFileReader(filepath_or_buffer, **kwds)   File "/opt/miniconda3/envs/cctyper/lib/python3.8/site-packages/pandas/io/parsers/readers.py", line 1442, in __init__     self._engine = self._make_engine(f, self.engine)   File "/opt/miniconda3/envs/cctyper/lib/python3.8/site-packages/pandas/io/parsers/readers.py", line 1735, in _make_engine     self.handles = get_handle(   File "/opt/miniconda3/envs/cctyper/lib/python3.8/site-packages/pandas/io/common.py", line 856, in get_handle     handle = open( FileNotFoundError: [Errno 2] No such file or directory: '<output filename>/hmmer.tab'
I was just wondering if anyone has come across this error and if so, how they resolved it?
Thanks in advance!

Hi,
Thank you for your response. And thank you for making such a handy tool!
 I was testing the software with a file that had very short sequences. When using a full genome, it worked just fine. Thanks again.


feat(crypto-helper): jwt: implement RS512 signature algorithm

https://github.com/TheBestTvarynka/crypto-helper/issues/3
Check new macros plz, if it's fine, I will add them to other signatures.

also, I have an idea about macros. I'll think about it more and tell you more details later

I am not sure about key_kind_as_str: $key_kind_as_str:expr thing but it the simplest way to provide logging in macros :woozy_face:

@rwwwx
can you also use those (check_asymmetric_key and sign/verify) macros in the Rs256/Rs384


New control: Search input control

I'm always in need of search input control, with either the button next to it or the magnifying glass inside the input.

Now supported in 2.0


MSYS build error.

Steps to reproduce:

Follow quick start tutorial
Error comes up on line: set __MINGW32__=1 && mingw32-make -B

Errors i found:

With default MSYS installation mingw32-make can't be found.
(That problem can simply be fixed, if you would specify the full path to executable. Like: /mingw64/bin/mingw32-make)
After that, when you try to build the executable that error pops out: https://pastebin.com/t8btSMZ0

Personally, i don't know what happens for this error to happen. I didn't found any cc1.exe in msys/mingw installation folder, but i found cc.exe in /mingw64/bin/. Maybe that's just a typo?

Personally I don't use MSYS2 shell to compile. It's just useful to install packages. But then it's easer to use simple cmd.exe or Powershell. You just have to add MinGW to PATH, so it can find C-compiler (cc.exe) and pkg-config. For example:
set PATH=%PATH%;\path\to\msys64\mingw64\bin

and maybe set CXX environment variable to c++ compiler, for example:
set CXX=\path\to\msys64\mingw64\bin\g++.exe

@etoshapovalov hey! Have you tried the things suggested by @kolumb? Did they work for you?


fix: Markdown editor diff issues

Fixes #2398.

please bump up these dependencies version
@github/file-attachment-element to v3.1.0
@github/markdown-toolbar-element v2.1.1

And a small bug when click on Image button for attach image:
(I could not find the problem)
Not Open file attachment windows for select file or image
and write ![](url) and give console error for not find url

And a small bug when click on Image button for attach image:

This isn't a bug, ![](url) is the syntax for inserting an image. File uploads still work if you drag and drop a file onto the textarea :)

Ah yes ,
but when add this ![](url) try to load url and give console error 404 .
I think this problem fix on latest version of @github/file-attachment-element
Thanks

Closing in favour of #2564.


On the issue of fescar server crash transaction recovery

I have read the official documents and recommended blogs of fescar, and I have read some of the source code of fescar myself. I have some questions to discuss:

The current version does not yet support the fescar server cluster? The data synchronization between cluster nodes is not found in the source code. If this is the case, the global transaction file for the crashed node may not be restored normally, and a new global transaction is also enabled.

fescar's global transaction will be stored in memory + asynchronous write file, but the global lock is only stored in memory, restarted in the server node, if there is a new global transaction open during the crash transaction recovery, will there be data inconsistency?

currently it's a single server node， file mode can set rws
when restarted server node，global lock also recovery based on unfinished transaction operation

Thanks for the reply, looking forward to the development of the cluster model

wait for this pr https://github.com/seata/seata/pull/942


reporting-operator: fix Prometheus token configuration

/hold
Using this for the push to quay ;)

@tschuy 3/4 test scenarios passed for me using this PR.  Details of which can be found in the JIRA

/this-is-fine

/this-is-unbearable

/this-is-not-fine

/woof

As mentioned on Jira, I think that was more of a question of surfacing errors correctly. I've updated this PR to reflect that.

/hold cancel

Looks good. Is this ready to merge?

/lgtm
/approve

So many flakes.
/override ci/prow/e2e-aws-metering

@tschuy @chancez yeap, I verified that it failed for the installation with the following error posted in https://jira.coreos.com/browse/MC-469 (which I have marked DONE) now.


Feature/remove rest client

Fixes https://github.com/ahtung/parasut-ruby/issues/19

Coverage remained the same at 100.0% when pulling 0148a9fbb9b6ca6452af9191f99edac4aeb729d7 on feature/remove_rest_client into f1be827e206ed669db01f263bf5fd52e3994e258 on master.


zadd inconsistent in accepting arguments

zadd(name, *args, **kwargs) accepts arguments in two ways:

As *args, in the form of: score, name, …
As **kwargs, in the form of: name=score, …

I pass my parameter in front ways, replacing the name with a variable，but there  is something inconsistent:
>>> import redis
>>> r = redis.StrictRedis()
>>> key = "value"
>>> r.zadd("test",key=3)
1
>>> r.zrange("test",0,-1,withscores=True)
[(b'key', 3.0)]
>>>  r.zadd("test",3,key)
1
>>> r.zrange("test",0,-1,withscores=True)
[(b'key', 3.0), (b'value', 3.0)]

Why  the **kwargs way can not pass real variable?  Why the two ways lead to different results?
And it pass the formal parameter in a function:
>>> def f(r,argkey):
>>>       r.zadd("test2",argkey=2)
>>>
>>> key2 = "value2"
>>> f(r,key2)
>>> r.zrange("test2",0,-1,withscores=True)
>>> [(b'argkey', 2.0)]

@yuyilei I don't see what the issue is.
For the first case;
r.zadd("test",key=3)
The result is:
>>> r.zrange("test",0,-1,withscores=True)
[(b'key', 3.0)]

which is the expected behavior.
For the second case, since you are using *args, you have already set the value of key to value, hence Python will resolve the value:
>>>  r.zadd("test",3,key),
Here key actually represents value because of >>> key = "value"
So:
 >>> r.zrange("test",0,-1,withscores=True)
[(b'key', 3.0), (b'value', 3.0)]

is actually the expected behavior.

@Hello @yuyilei
I agree with @ifiok in that there is no issue here apparently.
Closing this - feel free to reopen if needed.


SongBan module for banning songs

I think this just about works right now.

mergeth?

can we merge this yet?

I guess so XD About time!


removed functions used only in events.py from actions.py

moved functions fetch_initial_state_data, apply_events, do_events_register from actions.py to events.py. added necessary imports to events.py and removed unneccesary funtions from actions.py

Automated message from Dropbox CLA bot
@saisrivathsa, it looks like you've already signed the Dropbox CLA. Thanks!

@saisrivathsa next time you do refactoring, please make sure you rebase onto master first; you've got some merge conflicts.

Thanks for working on this!  It looks like streams_to_dicts_sorted should not have been moved, as it is used elsewhere in actions.py.  So I undid that move, cleaned up the imports in events.py, updated the developer documentation (you can use git grep to find where something is referenced) and broken tests, and merged this as d16730dd6a2d2dc0fdd2a2c394115a86875b7922.
Thanks @saisrivathsa!  I encourage you to read that commit to learn a bit about how to do this kind of refactoring in the future.

Thanks for the advice :D


Intersection of required and provided needs is not sufficient

The test for access in the allows method of the Permission class using the intersection of the required and provided needs in insufficient. The documentation says the needs in a permission "Represents needs, any of which must be present to access a resource". But with the intersection, if you have at least one need in common, the intersection will be not empty and the test succeeds.
I think we should use issubset instead:
def allows(self, identity):
        """Whether the identity can access this permission.

        :param identity: The identity
        """
        if not self.needs.issubset(identity.provides):
            return False

        if self.excludes and self.excludes.intersection(identity.provides):
            return False

        return True

I think you are understanding it wrong. "any of which must be present to access a resource" means that only one must match to gain access. This means they are evaluated as OR, not AND.
This makes the issue obsolete

Yeah, you are right. I have got a little bit confused with the English grammar. Sorry :)

Hi,
I wonder if you'd consider reopening this issue as an enhancement, rather than a bug. Permissions behave as the docs describe, but it seems insufficient to only allow the OR'ing of Needs in Permission requirements.
It would be useful to be able to define complex OR/AND combinations of needs in  Permission's constructor, rather than creating a new Need type for every desired AND of requirements, and then using only ORs at the top level.

Hey @SheepGotoHeaven :) We use Flask-Principal in our project with the following improvment:
import flask.ext.principal

class Permission(flask.ext.principal.Permission):

    def __init__(self, *needs):
        """A set of needs, any of which must be present in an identity to have
        access.
        """
        self.needs = set()

        for need in needs:
            if isinstance(need, str):
                self.needs.add(flask.ext.principal.ActionNeed(need))
            else:
                self.needs.add(need)

        self.excludes = set()

    def __and__(self, permission):
        return AndPermission((self, permission))

    def __or__(self, permission):
        return OrPermission((self, permission))

class AndPermission(Permission):

    def __init__(self, *permissions):
        self.permissions = set(*permissions)

    def __and__(self, permission):
        return Permission(set(self.permissions).add(permission))

    def allows(self, identity):
        for permission in self.permissions:
        	# If any permission fails, the whole or-permission fails
            if not permission.allows(identity):
                return False

        return True

class OrPermission(Permission):

    def __init__(self, *permissions):
        self.permissions = set(*permissions)

    def __or__(self, permission):
        return Permission(set(self.permissions).add(permission))

    def allows(self, identity):
        for permission in self.permissions:
            # If any permission succeeds, the whole or-permission
            # succeeds
            if permission.allows(identity):
                return True

        return False

delete_all_quests_permission         = Permission('delete_all_quests')
delete_quests_without_raw_permission = Permission('delete_quests_without_raw')

# Composite permission, indicating that the user is allowed to delete
# some questionnaires
delete_quests_permission = delete_all_quests_permission | delete_quests_without_raw_permission

Do you think, I should start merge request for this?

@f3anaro I've done something like this already and included the required unittests for this in pull request #21.  It's been over 2 years with no further activity.

@metatoaster You haved looked into your pull request and it seems to be perfect! Nicly done ... but why it is still open and not merged?


chore(tests-integration): add storage, state reader, and config to integration test

Stack:

#1671
#1670
#1669 ⬅
#1668
#1667
#1666
#1665
#1664

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is


Documentation: Additional details needed for custom_attributes

It took me quite some time to ultimately figure out how to set the custom_attributes kwarg in JobAttributes. I think it warrants a brief section in the docs and a clearer docstring.
Basically, as it currently stands, I had assumed I could do custom_attributes={"constraints": "cpu"} but was puzzled when it didn't appear in the Slurm job script. Only through diving through the test suite was I able to figure out that I needed to do custom_attributes={"slurm.constraints": "cpu"} instead.
I am happy to add this, but it may take me a few days to get around to it.

@RedStar-Iron --- let's keep this issue report for psij specific content. Happy to chat about quacc-related things on the dedicated repo. Let's move the conversation to here: https://github.com/Quantum-Accelerators/quacc/discussions/1837.
psij-python is a Python API meant to interface with various job schedulers and, potentially in the (more distant) future, support remote job submission and the pilot job model. It is not meant to read settings from job scripts generated elsewhere.

There is indeed no proper documentation on this issue. We'll get that fixed.


importModuleSpecifierEnding = js doesn't apply for autocomplete within import paths

Bug Report
I have this workspace setting which works great for auto import:
"typescript.preferences.importModuleSpecifierEnding": "js"

But when auto completing the path it still does .ts which requires manually changing:

🔎 Search Terms
importModuleSpecifierEnding
🕗 Version & Regression Information
4.3.0-dev.20210503

Do you have
{
    "compilerOptions": {
        "allowJs": true
    }
}

?

No I don't set that, I can't actually remember why I ended up needing to use .js imports but at some point this not having it was causing exceptions due to some bundling problem. I just tried to remove and it seemed to work, I'm a little hesitant to change it when it working though.
But why would you need allowJS for this to work when I've said I want to use .js imports already with importModuleSpecifierEnding?

Importing a .js file should not be allowed at all if allowJs is not enabled, so files with a .js extension will not show up in the completion list.
That is, if you have allowJs: false (this is the default) and the file
// file: myLib.js
export const myConst = 1;

and type the following in a .ts file,
myConst

You should not be offered a completion at all. If you enable allowJs, this should auto complete to the .js file assuming you have "typescript.preferences.importModuleSpecifierEnding": "js".
What I think is a bug, is that we complete with the .ts ending at all. It looks like the importModuleSpecifierEnding setting is just used to decide whether or not we remove the extension on string literal completion in the import. If allowJs is false, the .js import is not found, but we still offer the completion for the .ts file with the same name, but with the extension (incorrectly) attached. I think in this case we should offer the completion without any extension.

I can't remember the exact reason but using .js fixed a compile/bundling issue for me at some point. I use tsc and then webpack on its output.

Auto imports are working correctly and these path completions are working incorrectly. It’s never correct to write .ts in a module specifier. You are always allowed to write a module specifier for a TS file as a .js path, and you are allowed to write an import for an actual JS source file only if allowJs is enabled. The example shown here is for a TS source file, so the completion should be shown as number.js regardless of the allowJs setting.


Documentation publishing for pre-releases

If you do now
apt-get install powershell

You will get version 6.1.0-preview2.
This version doesn't have corresponding documentation published for Update-Help and is not shipped with the documentation built in. (call me if it's wrong, this is my conclusion from this CI failure https://travis-ci.org/PowerShell/platyPS/builds/387162525#L674)
Should the default installed package on linux actually have the documentation or an option to bring it?

Motivation context: https://github.com/PowerShell/platyPS/pull/358/commits/32ba62d19462a421802b3507da632e2a84284b42


CustomResourceRole missing SSM permissions

When following the deployment tutorial here the creation of the Cloudformation Stack fails due to an exception:

AccessDeniedException: User: arn:aws:sts:::assumed-role/LiveStreamingwithMediaStore-CustomResourceRole-I3LJAWWIUNNA/LiveStreamingwithMediaStore-custom-resources is not authorized to perform: ssm:PutParameter on resource: arn:aws:ssm:us-east-1::parameter/

Selected region was Northern Virginia us-east-1.
To Reproduce
When creating the Stack fill in the source Username and Password as shown in the SS below.
Expected behavior
The stack should be created without failure.
When source username and password were removed from the Stack parameters :arrow_right:  it was created successfully.
Please complete the following information about the solution:

[x] Version: v1.1.0
[x] Region: The stack was created in us-east-1
[ ] Was the solution modified from the version published on this repository? I don't think so.
[ ] If the answer to the previous question was yes, are the changes available on GitHub?
[x] Have you checked your service quotas for the sevices this solution uses?
[x] Were there any errors in the CloudWatch Logs? Yes, here's the full stacktrace:

2020-08-03T12:59:08.336Z	9fcbeed0-d869-4d94-be1c-98a6b2bf2cbe	INFO	ERROR::  AccessDeniedException: User: arn:aws:sts::<iam-id>:assumed-role/LiveStreamingwithMediaStore-CustomResourceRole-I3LJAWWIUNNA/LiveStreamingwithMediaStore-custom-resources is not authorized to perform: ssm:PutParameter on resource: arn:aws:ssm:us-east-1:<iam-id>:parameter/<source-username>
    at Request.extractError (/var/task/node_modules/aws-sdk/lib/protocol/json.js:51:27)
    at Request.callListeners (/var/task/node_modules/aws-sdk/lib/sequential_executor.js:106:20)
    at Request.emit (/var/task/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (/var/task/node_modules/aws-sdk/lib/request.js:683:14)
    at Request.transition (/var/task/node_modules/aws-sdk/lib/request.js:22:10)
    at AcceptorStateMachine.runTo (/var/task/node_modules/aws-sdk/lib/state_machine.js:14:12)
    at /var/task/node_modules/aws-sdk/lib/state_machine.js:26:10
    at Request.<anonymous> (/var/task/node_modules/aws-sdk/lib/request.js:38:9)
    at Request.<anonymous> (/var/task/node_modules/aws-sdk/lib/request.js:685:12)
    at Request.callListeners (/var/task/node_modules/aws-sdk/lib/sequential_executor.js:116:18) {
  code: 'AccessDeniedException',
  time: 2020-08-03T12:59:08.276Z,
  requestId: '9e044931-17cd-4096-8bb8-2285d7698716',
  statusCode: 400,
  retryable: false,
  retryDelay: 79.75711276262896
} AccessDeniedException: User: arn:aws:sts::<iam-id>:assumed-role/LiveStreamingwithMediaStore-CustomResourceRole-I3LJAWWIUNNA/LiveStreamingwithMediaStore-custom-resources is not authorized to perform: ssm:PutParameter on resource: arn:aws:ssm:us-east-1:<iam-id>:parameter/<source-name>
    at Request.extractError (/var/task/node_modules/aws-sdk/lib/protocol/json.js:51:27)
    at Request.callListeners (/var/task/node_modules/aws-sdk/lib/sequential_executor.js:106:20)
    at Request.emit (/var/task/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (/var/task/node_modules/aws-sdk/lib/request.js:683:14)
    at Request.transition (/var/task/node_modules/aws-sdk/lib/request.js:22:10)
    at AcceptorStateMachine.runTo (/var/task/node_modules/aws-sdk/lib/state_machine.js:14:12)
    at /var/task/node_modules/aws-sdk/lib/state_machine.js:26:10
    at Request.<anonymous> (/var/task/node_modules/aws-sdk/lib/request.js:38:9)
    at Request.<anonymous> (/var/task/node_modules/aws-sdk/lib/request.js:685:12)
    at Request.callListeners (/var/task/node_modules/aws-sdk/lib/sequential_executor.js:116:18)

Screenshots

Additional context
Looking at the template policies for CustomResourceRole there's no SSM configuration link to template
I'm not an AWS expert, but I assume adding the policies will fix the issue?
On the other hand, the MediaLiveRole has two identical policies for the SSM here and here Not sure if this deserves a separate issue, but I think it's worth noting it.

@tomnight Just following up to check if this has been issued?

patched in the last commit, v1.1.1
thanks


ng - No such file or directory

- [ X ] bug report

Versions.
Output from: ng --version: No such file or directory
Output from: node --version: v7.10.0
Output from: npm --version: 4.2.0
macOS Sierra 10.12.4
Repro steps.

After running ng new app on a fresh install of Node.js and npm I am getting the error message: -bash: /Users/Liz/.npm-packages/lib/node_modules/@angular/cli/bin/ng: No such file or directory. How can I fix this error?

to me it seems angular cli is not installed ? have you run npm install -g @angular/cli ?

@dave11mj's comment is on point. Check out https://github.com/angular/angular-cli#installation for install instructions.

I ran that install again and I am getting a new error. Do you have any idea what could be causing this?
/Users/Liz/.nvm/versions/node/v4.4.0/lib/node_modules/@angular/cli/models/config/config.js:16
constructor(_configPath, schema, configJson, fallbacks = []) {
^
SyntaxError: Unexpected token =
at exports.runInThisContext (vm.js:53:16)
at Module._compile (module.js:373:25)
at Object.Module._extensions..js (module.js:416:10)
at Module.load (module.js:343:32)
at Function.Module._load (module.js:300:12)
at Module.require (module.js:353:17)
at require (internal/module.js:12:17)
at Object. (/Users/Liz/.nvm/versions/node/v4.4.0/lib/node_modules/@angular/cli/models/config.js:3:18)
at Module._compile (module.js:409:26)
at Object.Module._extensions..js (module.js:416:10)

@erbanach I haven't seen that error before .. but from what I can tell it seems you are using node 4.4.0 and  Angular Cli requires at least node 6.9.0.. Try updating your node to the latest version and that should hopefully clear out that error. ^^

I've seen in some cases NVM global installs getting completely messed up. You report Node 7, but that error log shows that stuff is running from node 4. In this case my advice is to uninstall NVM and all your node/npm versions and then reinstall them.

(Windows machine)
Same issue is happening while installing Angular-CLI with
Node - 8.9.4
NPM  - 5.6.0
ERROR:

No such file or directory, open C:\Users(UserName)\AppData\Roaming\npm\node_modules.staging\rxjs-8af1cb8d_esm2015\Subscription.js.map'

I have so many similar warning/errors in the terminal.
Now doing ng -v gives me "ng" is not recognized as ....

 Have you figured out a solution yet?

alias ng="/usr/local/lib/node_modules/@angular/cli/bin/ng"

Try this

npm remove -g angular-cli
npm cache clean
npm install -g @angular/cli

This is because of NVM in my case it works fine with this
alias ng="/home/usr_name/.nvm/versions/node/v10.16.0/lib/node_modules/@angular/cli/bin/ng


Feature: SQLite-based Message Cache

Problem
Some apps require access to the same messages over and over again. This puts unnecessary strain on the CCN API and increases the latency of apps using Aleph Messages.
Solution
This pull request proposes a solution by introducing a message cache implemented with Peewee and an SQLite database in the Aleph SDK. The message cache effectively stores frequently accessed messages, eliminating the need for repetitive CCN API calls. With the message cache in place, apps utilizing the Aleph SDK will experience enhanced efficiency and responsiveness in accessing messages.

After introducing the AlephClientInterface, I get a mypy error, that I really don't get: https://github.com/aleph-im/aleph-sdk-python/actions/runs/5071082316/jobs/9107016057?pr=31
Even though I am exactly copying the signature of the watch_messages function:
    async def watch_messages(
        self,
        message_type: Optional[MessageType] = None,
        content_types: Optional[Iterable[str]] = None,
        refs: Optional[Iterable[str]] = None,
        addresses: Optional[Iterable[str]] = None,
        tags: Optional[Iterable[str]] = None,
        hashes: Optional[Iterable[str]] = None,
        channels: Optional[Iterable[str]] = None,
        chains: Optional[Iterable[str]] = None,
        start_date: Optional[Union[datetime, float]] = None,
        end_date: Optional[Union[datetime, float]] = None,
    ) -> AsyncIterable[AlephMessage]:

mypy raises:
src/aleph/sdk/cache.py:370:5: error: Return type "AsyncIterable[Any]" of "watch_messages" incompatible with return type "Coroutine[Any, Any, AsyncIterable[Any]]" in supertype "AlephClientInterface"  [override]
src/aleph/sdk/client.py:708:5: error: Return type "AsyncIterable[Any]" of "watch_messages" incompatible with return type "Coroutine[Any, Any, AsyncIterable[Any]]" in supertype "AlephClientInterface"  [override]

Looking into snapshot testing to figure out how to mock API responses.

Thanks @hoh, this helps prioritizing on which points work on next for me.

@hoh
I'd like to point out that this line:
if isinstance(messages, AlephMessage):

is making tests fail in Python versions =< 3.9. The alternative is making comparisons with every single Aleph message type (PostMessage, AggregateMessage, ...) which I think might be a bit ridiculous.
As Python progressed already to 3.11 now, can it be expected of people wanting to use our newest SDK versions to use at least Python 3.10?

@hoh I'd like to point out that this line:
if isinstance(messages, AlephMessage):

is making tests fail in Python versions =< 3.9. The alternative is making comparisons with every single Aleph message type (PostMessage, AggregateMessage, ...) which I think might be a bit ridiculous.

The solution seems to be using isinstance(data, typing.get_args(Constant))
https://stackoverflow.com/a/73874277

As Python progressed already to 3.11 now, can it be expected of people wanting to use our newest SDK versions to use at least Python 3.10?

Not as long as we support Debian 11, which ships with Python 3.9.
The SDK should work on as many environments as possible, and keep doing so for a while. The situation is different for PyAleph since it is only expected to be deployed in containers and already uses Python 3.11.

Closed for now, as it will be broken down into smaller PRs


terraform plan Error: your API TokenID username should contain a !, check your API credentials

Hi there
I am following your tutorial but get the following error when doing terraform plan:
│ Error: your API TokenID username should contain a !, check your API credentials
│
│   with provider["registry.terraform.io/telmate/proxmox"],
│   on main.tf line 10, in provider "proxmox":
│   10: provider "proxmox" {

Any idea what needs to be done?
Thanks
Steve

What does your token id look like @sfoubister?
Should be something along the lines of:
variable "pm_api_token_id" {
    description = "Proxmox API token id"
    default = "username@pam!xxxxxxxxxxxxxx"
}

Where is that block defined? I have generated an API token but where I do I specify it for terrform?

You can export variables via the CLI which is what I would do, or hardcode them into the provider block. There's no explicit documentation showing that though.
https://github.com/Telmate/terraform-provider-proxmox/blob/master/docs/index.md#creating-the-connection-via-username-and-api-token
provider "proxmox" {
  pm_api_url = "https://proxmox-server01.example.com:8006/api2/json"
  pm_api_token_id = "somehardcodedvalue@pam!zxzxzxzxzcxczxczxczxc"
  pm_api_token_secret = "zxxxxzxzxxzxzxzxzxzx"
}

Thanks @pgporada - hard coding the pm_api_token_secret in the provider block worked.


[RFC] Pytest fixture to override settings dynamically

Is your feature request related to a problem? Please describe.
It would be useful to be able to write tests like this:
def test_foo_bar(settings):
    settings.FOO = "bar"

def test_foo_not(settings):
    settings.FOO = "not"

The existing pytest solution relies on use of environments to switch to testing, and thus doesn't support dynamic values.
Describe the solution you'd like
A new utility method on LazySettings that allows temporarily overriding values in tests
class LazySettings:
    ...
    def override():
        ...
        yield
        ...

in app:
@pytest.fixture
def override_settings():
    with settings.override() as wrapped:
        yield wrapped

Describe alternatives you've considered

Tried mocking dynaconf.store doesn't seem to work well.
Tried using_env but I don't really use environments, so there's no "testing" environment here to get default values from.

Additional context

pytest-django supports this feature.

Update: this .. kinda works?
@pytest.fixture(autouse=True)
def settings_reset():
    settings.reload()
    settings.validators.validate()

Not sure why validators.validate() isn't called in reload - I added some default values using validation which seem to get blown away on reload.

We can implement a fixture like that using from_env method.
https://www.dynaconf.com/advanced/#from_env
this gives us a new copy of the settings instance.
The reload should call validators and also keep the default values, probably a bug.


modify logback-classic and logback-core to 1.2.3

What is the purpose of the change
Partial package paths are missing
Brief changelog
Rocketmq 4.7.0 import eclipse ch.qos.logback.classic.joran.JoranConfigurator file does not support
Verifying this change
Compiled by
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage increased (+0.06%) to 50.904% when pulling 4d65d2a16d20cf67aee48319262668deafd86b32 on lovegang:rocketmq-fs into e923a35393a6dfa83677335e96fdb4b063f10d32 on apache:master.

I will close this pull request because there is some confusion in the current commit records. If you have any questions, you can resubmit the pull request. But Plz pay attention to the following points:
Firstly, plz submit related issues to link this PR and explain the purpose of PR according to the format.
Second, you need to change the target branch to develop.
Finally, plz name the pull request in the correct format.


document recent changes and release 0.3

document recent changes done in #50:

[ ] add latex support with latex as in nimib
[ ] document changes

[ ] refactoring and code structure (review)
[ ] templates
[ ] book.json
[ ] new nb/doc object in Book
[ ] blocks context in document.mustache
[ ] refactored Book.book_title into Book.title. Overrides previously existing Book.title which was supposed to be used for the Chapter title
[ ] cfg stuff
[ ] note that changes also fix https://github.com/pietroppeter/nimibook/issues/48 (you should be able to change name of book folder)
[ ] removed book.path_to_root (it is a document/chapter thing)
[ ] remove rootfolder in toc and path field in Toc
[ ] improved init
[ ] remove default parameters and allow to pass parameters to build

[ ] after nimib 0.3 is released, change nimib dep and release this as 0.3

other improvements (next version):

[ ] add nim code highlighting when starting from md file?
[ ] improve clean? (maybe open an issue)

[ ] add a dry-run option
[ ] add keepSrc, keepOut, keepSrcDefaults, keepOutDefaults

[ ] support part title in toc
[ ] support SUMMARY.md

[ ] a default summary.md should be initialized

[ ] provide nbook binary!
[ ] dumped book.json (now a public api) should not contain cfgDir and it should only contain relative dirs
[ ] quit failures for check and build should be only in cli? (useful for test?)

notes:

cfgDir should be set as currentDir of nbook
do I need also a srcDirRel and homeDirRel?
I am removing srcDir from all entry paths
srcDir and homeDir always assumed to be relative to cfgDir and always used as string. (maybe nimib should be like that)
srcDir and homeDir should really be relative strings...
decided indeed that srcDir and homeDir kept as string relative to cfgDir
starting to call use outDir instead of homeDir more and more often... (should I change in nimib?)

also:

[ ] remove the rust file
[ ] add as vendored the html and css stuff
[ ] recent parallel build by beef


update dockernetwork.md and network api

Signed-off-by: Deng Guangxing<EMAIL_ADDRESS>

thanks @x1022as. LGTM.
ping @moxiegirl to pick up these changes.

LGTM


Revert "Bump .NET SDK from 5.0 to 6.0"

This reverts commit 81c37b3949ecbac7beda268e254ac57c0ddd0696 since CI
was picking up test failures when running the move-prover test suite:
test prover unit[simplify]::functional/unused_schema.move ... FAILED
Error: cannot extract version from `/opt/dotnet/tools/boogie`

/priority high

/priority high

/land

/priority high


Add transcript hash feature

Just send for main structure review.
Just test at emu tool currently because of most related unittest case will be effected by #52.
I will sync unittest and re-send request after #52 be fixed if main structure of this is fine.

#22

Dump tool cannot decode securedSPDM if message_f and message_k use transcript hash:
...
88<PHONE_NUMBER>) PCI_DOE(1, 2) RSP->REQ SecuredSPDM(0xffffffff) SPDM(11, 0x67) SPDM_PSK_FINISH_RSP ()
89<PHONE_NUMBER>) PCI_DOE(1, 2) (?)->(?) SecuredSPDM(0xffffffff)
90<PHONE_NUMBER>) PCI_DOE(1, 2) (?)->(?) SecuredSPDM(0xffffffff)
91<PHONE_NUMBER>) PCI_DOE(1, 2) (?)->(?) SecuredSPDM(0xffffffff)
...
I don’t have any inspiration for this question at the moment.

Can we move append_hash_managed_buffer() inside of append_managed_buffer() ?
=================
return append_managed_buffer(&spdm_context->transcript.message_mut_c,
message, message_size);
#ifdef USE_TRANSCRIPT_HASH
return append_hash_managed_buffer(spdm_context,
&spdm_context->transcript.message_mut_c,
message, message_size);
#else
return append_managed_buffer(&spdm_context->transcript.message_mut_c,
message, message_size);
#endif

Can we move append_hash_managed_buffer() inside of append_managed_buffer() ?

Another buffer that dont store hash will use append_managed_buffer(), such like messageA, certchain verify, hmac ...
I think that can be distinguished by buffer size(if the size of these two kinds buffer are always different in future),
or use a flag in buffer type.
This makes me think about that should managed buffers used in calculation of certchain verify, hmac... store hash also?
If sure, only the case of messageA needs to be distinguished.

Optimized issue-solving method


CSS Cafe menu Step 22 accepts answer of nesting a div selector inside the body selector

Describe the Issue
I was assisting a camper in this forum post and they were able to pass step 22 of the cafe menu project with the following answer.
div{/*background-color: burlywood; */}

I don't think this answer should be accepted because it is using selector CSS nesting which doesn't have great browser support in native css right now according to Can I use.
Affected Page
https://www.freecodecamp.org/learn/2022/responsive-web-design/learn-basic-css-by-building-a-cafe-menu/step-22
Your code
code that passes
body{
  div{/*background-color: burlywood; */}
}

Expected behavior
/*background-color: burlywood; */

Screenshots

I'm not sure what we can do here. The step is to comment out the line, which the person did. Seems tough to make a check for other things like this. I feel like we can probably just leave it but we can see if anyone else has any ideas.

The current test is:
code.match(/\/\*\s*background-color:\s*burlywood;?\s*\*\//i)

We could expand that to make sure only body { comes before the commented line and `} follows. That is, if we feel this one aberration is worth the trouble.

I have looked through the forum and it doesn't seem like this is one of the steps that people trip up on that much.
My guess was that the user here was confused by the last sentence because it mentions the word div and that might have been why they wrote it that way.
I think for now we could close this issue.
If for some reason it starts to effect a large group of people, then we could revisit it then. 👍


Merge branch 'dev/front' of https://github.com/Stratio/sparkta into d…

…ev/front
Conflicts:
web/src/languages/en-US.json

LGTM


[id] fix a nit in the feature-state short code

en PR
https://github.com/kubernetes/website/pull/32474/files
/assign @habibrosyad

LGTM


Link to code of conduct not resolved on website

CONTRIBUTING.md is linking to a CODE_OF_CONDUCT.md file, but this file does not exist in the repo yet. Is it in a branch that is yet to be merged?

No, there's no CODE_OF_CONDUCT.md file anywhere. I think this link must have been present in the template that @SaranjeetKaur used for CONTRIBUTING.md. Saranjeet, would you be able to proposed a PR adding a CODE_OF_CONDUCT.md file as well?

@EllaKaye , @alessandrofelder - Yes, the link is in the CONTRIBUTING.md file. Should we go with a contributor covenant code of conduct? Or something else? We might also need an email address for reporting any violations of the code of conduct.
Or for now, should we go with CW24's code of conduct? In which case, I can remove that line from CONTRIBUTING.md?

I'd lean towards the contributor covenant CoC, since it should be familiar and uncontroversial for people working in Open Source. The CW24 CoC has specifics around it being an event, whereas I think of communeRS now as more of an open source project/community than an event. Essentially, the principles in both are similar, so I don't think it makes a huge difference. @SaranjeetKaur, would you be willing to be the contact for it?

Hi @EllaKaye, I don't mind to be a point of contact. But I would want to avoid putting my personal email address there. It might be useful to setup a dedicated email address for this. I also feel that having more than one person monitoring the CoC would be a more sustainable approach.

Agreed. I've set up<EMAIL_ADDRESS>which we can use for the group. I'll share password with you privately when back at my work computer next week. I'm happy to be listed with you as a name for CoC contact, and would be good to have a third person for that too. I'll raise that in the Slack.


A transaction on 0xProtocol.

https://etherscan.io/tx/0x0804e35ffcd0b94d911bc5b7fed563abc8fffc67d0780ec8a689d8a61ffd0aec

Hello there, thanks for participating in our previously concluded "Awesome Transactions Bounty" program. In other news, we have a great update for you! It's the new analyzing tool called "EigenTx" which you can use to analyze and dissect your submission and other transactions you deem necessary or intriguing! Follow the link below to enjoy the experience.
https://tx.eigenphi.io/


fix: proposer with nil fees triggers next block

Issue being fixed or feature implemented

Proposer with nil fees (no transactions in block) added to epoch proposers. It leads to app hash change, which triggers the next block in tenderdash. In the end, we have infinite block generation and high load on the network.
What was done?

Do not add proposer if block fees are nil

How Has This Been Tested?

With tests

Breaking Changes

None
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added or updated relevant unit/integration/functional/e2e tests
[x] I have made corresponding changes to the documentation

For repository code-owners and collaborators only

[x] I have assigned this pull request to a milestone

Not adding the proposer if the fees are nil would lead to a lot of proposers being discarded. This is not the right solution. A malicious MNO with let's say 200 nodes, could close to double their rewards (at the expense of other nodes, with some pretty easy tricks).

There is a much worse problem. We just not gonna payout all the fees according to our current logic https://github.com/dashevo/rs-platform/blob/8cf41a539caf4e63f7616cec4950b864a9b986e1/dash-abci/src/execution/fee_pools/fee_distribution.rs#L203
As we discussed. With same-block execution we won't have this problem so we can just disable this mechanism in tendermint for now.


Feature request: separate registry from tag

Right now, an image "tag" includes its registry location. I would propose separating this out, so that the registry location is not included in the tag. The tag should be independent from where the image is stored. Having the registry location stored as part of the image tag complicates image management commands, for example, pulling from the Docker Hub, and pushing the same image to a different registry.
Currently:
docker pull wordpress
docker tag wordpress https://registry:5000/wordpress
docker push

Proposed:
docker pull wordpress
docker push wordpress --registry https://registry:5000

Cheers,
Trevor Sullivan
Microsoft MVP: Cloud & Data Center Management
https://trevorsullivan.net
https://twitter.com/pcgeek86

@pcgeek86 I assume when you say "tag", you mean "image name". This is known problem with docker. Addressing it is subtlety complex.
There are number of issues discussing this concept. Typically, we discuss this under the topic of decoupling naming and location, which is usually discussed in docker/distribution project.
I'm closing this as a duplicate.

@stevvooe Yep, that's what I meant. Sounds good -- which issue is it a duplicate of?

@pcgeek86 This is a massive arch we are following to address several limitations to get this done. There isn't a single issue that I can point to.
This issue is an example of some of the ongoing work: https://github.com/docker/distribution/pull/1778. We may want to add a road map item, but we are still examining solutions.
If you'd feel more comfortable leaving this issue open, let me know.

@stevvooe It's all good mate, we don't need to keep this issue open. I appreciate the work you're doing internally to resolve this. I'll just wait to see what happens in the coming weeks and months. Appreciate the additional info! 🐋 🐳


Chapter 1 - A Simple Regression Problem

There is only one thing left to do; turn our tensor into a GPU tensor. That is what [to()](https://bit.ly/32Mgxjc) is good for. It sends a tensor to the specified device.

Hi Dan,
I love your book and tutorials! May I kindly ask does to() method copy the data input the device (GPU or CPU) memory directly?
The reason I am asking is that you mentioned before that torch.as_tensor(x_train) will shares the underlying data with the original Numpy array, but when we used torch.as_tensor(x_train).to(device) I found that x_train data won't change.
Do I understand it correctly?
Best,
Jun

Many thanks Daniel! That's really helpful 👍
Best,
Jun


Errors do not appear in the status field for the Dialling Computer

Where the errors should appear, it remains blank

Fixed in release 1.2.0.


Added Angular with TypeScript

Hi, I am Branko from Manning Publications. I thought this video course might be a good resource for your list.
Thanks for checking it out!

added, thank you @ipcenas


[release branch]Update workflows to produce artifact correctly -source (.zip)

Issue Reported: #132 on GitHub
Root Cause Analysis (RCA):
The issue we encountered is related to the use of the softprops/action-gh-release GitHub Action for creating releases on Linux. While uploading release assets, the action defaults to using the repository’s default branch (main) for the assets it creates.
Details:
GitHub Action: softprops/action-gh-release
Problem: The action was utilizing the main branch to create release assets due to its default behavior.
Relevant Input: target_commitish - A string input that specifies the commitish value (branch or commit SHA) from which the Git tag is created. Defaults to the repository’s default branch.
Reference Documentation:
softprops/action-gh-release README
github context documentation
Similar Issue Discussion

LGTM


Check idempotence on dAppAddress

This issue has been marked as closing. If you believe this issue is still relevant or if you have additional information, please feel free to reopen it or create a new issue.


Errors during compilation

When downloading the build-essential, an error occurs with 404 not found.
When apt-get update, an error [13:Permission denied]

These instructions only apply to Linux distributions using apt. It looks like you are running Windows, so you do not have to install anything beforehand.

And what to do with the build-essential because of it as I understood it is not compiled?

No offense, but you understood wrongly, then. build-essential will install a number of utilities that are required for compilation on Linux. Even then, it is very likely that they are already present on the system.
Am I correct in assuming that you are running Windows 10? Which terminal emulator are you using, currently?

Ubuntu 18.09

So you are using the Windows Subsystem for Linux? I've never tried, but I very (very) much doubt that you'll be able to run maniac from in there. Thinking about it, I can pretty much guarantee that you won't.
Use a shell like Babun or Cmder to run build.sh.

Well, i try it

Babun:

My apologies, it appears that Babun cannot properly parse build.sh. I had wrongly assumed that it was able to do that based on previous experience with a different setup.
I don't usually work on my Windows machine, but when I do I use Git Bash which comes with Git, and can be downloaded here. Please make sure you have an up to date version of the repository since it is entirely possible that older versions might be broken on Windows (as I'm not always able to test on there).

Okay

:'(

Can you try to run gcc -v anywhere in your terminal?
This is to confirm that GCC is installed and properly configured on your system. It should look something like this, if successful:

gcc not installed.

GCC should come as part of MinGW. Read the Getting Started guide, and also take note of the Environment Settings section.

In any case, I just published release 0.2.0 which you can download here.

error to parse beatmap (( null )) This is because I have osu! not in the standard path?

That's very likely, yes. If you want support for custom osu! install locations, please create an issue and I'll see if I can implement it in a future version. Else, you could edit the #define DEFAULT_OSU_PATH line in osu.h, but note that the path defined there should be relative to your home directory.

The issue with INT_MAX being undefined should be fixed with 2b41ebe1c3e37c8817824acc836f62225f2b03ff. I was not able to reproduce this issue on my machine, but I am reasonably sure I fixed the crux of the issue.

Are you on an x32 machine? And are you certain that you have the latest version of the repository? Currently, check_chunk (which is defined statically in pattern.c) is on line 20, not 21.

x64 machine, my version of gcc is not similar to yours, probably because of this.

can you compile a maniac with a path to osu! on drive C: ?

You'll want 8.1.0, x84_64 and POSIX threads. Be sure to update your PATH if necessary and make sure that gcc -v yields the correct version information.

If you want a quick fix, just copy your osu! directory to C:\Users\Cherry\AppData\Local\. Alternatively, you could always bypass the map searching by passing the -m option with a valid path to a beatmap. scripts/find.js should be helpful with that.

Okay

failed to parse beatmap (i change path)

I have just updated release 0.2.0 with maniac-win64-debug.exe which does not have any significant code changes but is compiled with some additional debug flags set. If you want to help me track down the issue(s) you are having, please run the build and let me know which messages you get.
This is how it looks like on my machine after having played a beatmap.

okay

Now there is no error, but it clamps the D key

The result is the same.

I threw together a basic test suite that should cover most of maniacs basic functionality. It can be downloaded here and I hope that it can shed some light on what's going wrong on your system.
At this point, I'm really just amazed at how many things can go wrong on a single machine, especially considering that it's running flawlessly for me.

okay

auto closing time to notice only the path found and PID process osu!

error: failed to found beatmap

l, like everything itself is fixed x)

when I turn on more than 4 keys it breaks.

That - works, then - no

The cheat does not currently support anything but 4k Maps, although support
for Xk should be trivial to implement at this point. As always - make an
issue and I'll look into it.
Good to see that you got maniac working after all, by the way, have fun.
On Thu, 23 Aug 2018, 21:55 ssCherry<EMAIL_ADDRESS>wrote:

That - works, then - no
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/LW2904/maniac/issues/14#issuecomment-415550776, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AbPjeH1i8bNX0nYwhs155x4agtVRHL9eks5uTwilgaJpZM4WF8EP
.

oh,no

Could you upload one of the beatmaps which are not working for you somewhere, so I can take a look at them and do some testing on my machines?

All maps do not work.

Since, at this point, this is a different problem with a (likely) very different cause, please create an issue and I'll look into it.

To get back to the original problem, have you been able to compile successfully by now?

no

I am closing this issue since it has ventured far from the original problem. If you are still running into errors, please feel free to create an issue while making sure to provide as many details as possible.


Make API variable examples more accessible

I'm not hip with all the Monad slang, so this documentation is fairly opaque to me: http://docs.folktalejs.org/en/latest/api/data/task/Task.html

are α and β and γ placeholders for something meaningful-yet-ungoogleable, or are they just trendy placeholders akin to foo/bar/baz?
if they have specific meanings, what are they?
what is the meaning of @Task and => and →?  I'm guessing that the => indicates a function and the following (β → γ) indicates that the function takes a β and returns a γ, but I'm not sure.

The text below each method would probably be sufficient if I understood the shorthand above, but it's not enough to get the point across by itself when I don't understand the shorthand.
I don't think the shorthand necessarily needs to be removed or anything, I just need an example showing what the shorthand expands to in actual JavaScript.

Yeah, I need to write the appendix on the type notation used by those signatures.
In general, it's something fairly similar to Haskell's type notation, but:

greek letters are used for type variables (so instead of forall a. forall b. a → b you have α → β).
a → b means something that takes in type a and returns type b.
p => a means something that has type a, as long as it fulfils the predicates dictated in p. @x is a predicate that dictates that the type of this in the type expression must be x. a:b declares a as an alias for type b. t(_) is a partially applied type t.
a | b is a type union, which means it can be either type a or type b.
a :: b means that while something is of type a, evaluating it will cause the effects described in b (it isn't a pure computation).
{ a: b } is an object type with property a having type b.
{ α | b } is a row type, which matches the object type b, and stores any properties not covered by that type in the variable α.


Pass WSA_FLAG_NO_HANDLE_INHERIT flag to the WSASocketA() to avoid handle leaking on the Windows in case of using CreateProcess() on the server

https://docs.microsoft.com/en-us/windows/desktop/api/winsock2/nf-winsock2-wsasocketa

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.
After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

User @frazenshtein isn't covered by a CLA. They will need to complete the form at https://identity.linuxfoundation.org/projects/cncf

Regards,
CLA GitHub bot

Please sign cla.

@frazenshtein   is this PR an attempt to resolve https://github.com/grpc/grpc/issues/17704?
Have you been able to confirm that it fixes the problem?

@frazenshtein is this PR an attempt to resolve #17704?
Have you been able to confirm that it fixes the problem?

I believe this PR will fix this issue. In my case, the problem looked like an error on the client side: grpc on_accept error: the I/o operation was aborted due to the completion of the command flow or the request of the application.

@nicolasnoble  opinions?

@AspirinSJL  I think the general idea is of this PR is alright, can you please review for correctness from C-core perspective?

I think @apolcyn can review this instead of @AspirinSJL.  He has more familiarity with both Windows and with the c-ares code.

Seeing this error in the linux artifact build:
[CXX]     Compiling src/core/lib/gpr/wrap_memcpy.cc
[CXX]     Compiling src/core/lib/gprpp/fork.cc
[CXX]     Compiling src/core/lib/gprpp/thd_posix.cc
[CXX]     Compiling src/core/lib/gprpp/thd_windows.cc
[CXX]     Compiling src/core/lib/profiling/basic_timers.cc
[AR]      Creating /tmp/libs/opt/libaddress_sorting.a
[CXX]     Compiling src/core/lib/profiling/stap_timers.cc
[AR]      Creating /tmp/libs/opt/libz.a
[AR]      Creating /tmp/libs/opt/libares.a
src/core/lib/iomgr/socket_windows.cc: In function 'void grpc_wsa_socket_flags_init()':
src/core/lib/iomgr/socket_windows.cc:193:77: error: passing NULL to non-pointer argument 5 of 'SOCKET WSASocketW(int, int, int, LPWSAPROTOCOL_INFOW, GROUP, DWORD)' [-Werror=conversion-null]
                           grpc_wsa_socket_flags | WSA_FLAG_NO_HANDLE_INHERIT);

is the header that contains the new socket option also available to mingw? That build failure is a mingw issue

I'm also wondering has this been verified to fix the issue. A test would be best but barring that it would be good to know that this has been manually verified

@frazenshtein, are you planning to address review feedback?

@frazenshtein, are you planning to address review feedback?
Yes, I will try to take into account the comments in the nearest future. I have some difficulties with the allocation of free time.

Seeing this error in the linux artifact build:

Yes, I really mixed up NULL and 0. Fixed.

is the header that contains the new socket option also available to mingw? That build failure is a mingw issue

Haven't found this one. Looks strange, because we have added #ifndef to the src/core/lib/iomgr/socket_windows.h Can you provide an error message or a link and keywords to grep?

let's turn this into a function rather than have everyone reference the global directly

Fixed. However, I'm happy to rename it to your liking. I didn't find a good example in the code, so it turned out a bit long.

I'm also wondering has this been verified to fix the issue. A test would be best but barring that it would be good to know that this has been manually verified

I was unable to reproduce the problem from the #17704 issue. If the author provides a minimal example it will be easier.

I triggered an adhoc package build (the nugets) to be able to verify if this patch fixes to do #17704
https://sponge.corp.google.com/target?id=1c772a91-2b04-42ce-8d55-31a2a2990574&target=grpc/core/experimental/grpc_build_artifacts_multiplatform
I'll later post the link to the pre-built nugets.

The adhoc build of packages is here:
https://storage.googleapis.com/grpc-testing-kokoro-prod/test_result_public/prod/grpc/core/master/windows/grpc_build_packages/5341/20190314-085942/github/grpc/artifacts/csharp_nugets_windows_dotnetcli.zip

According to https://github.com/grpc/grpc/issues/17704#issuecomment-481264974 this would fix #17704 if merged.

Do we want to use the default flags for https://github.com/grpc/grpc/blob/934e31d7d1c693348b10154b49fa863b48ed62ce/test/cpp/naming/resolver_component_test.cc#L278 too?

Could you please make clear for me is something expected from my side?

I believe we need need to use grpc_get_default_wsa_socket_flags() here:
https://github.com/grpc/grpc/blob/618a3f561d4a93f263cca23abad086ed8f4d5e86/src/core/lib/iomgr/tcp_server_windows.cc#L258
and here
https://github.com/grpc/grpc/blob/ad4f80569260f03c83fa7b372f50470c119dbb8a/test/cpp/naming/resolver_component_test.cc#L279

@apolcyn  I believe this is good to do, can you give owners approval so we can merge?


PyQt5==5.13.0 not found ?

ERROR: Could not find a version that satisfies the requirement PyQt5==5.13.0 (from -r .\requirements.txt (line 14)) (from versions: 5.12.3, 5.14.0, 5.14.1, 5.14.2, 5.15.0, 5.15.1, 5.15.2)
ERROR: No matching distribution found for PyQt5==5.13.0 (from -r .\requirements.txt (line 14))

I have the same problem with Zorin OS 15.3. Even by re-installing PyQt5 it gives many errors, same with PyQtWebEngine saying
ERROR: Could not find a version that satisfies the requirement PyQtWebEngine
ERROR: No matching distribution found for PyQtWebEngine

I have the same problem with Zorin OS 15.3. Even by re-installing PyQt5 it gives many errors, same with PyQtWebEngine saying
ERROR: Could not find a version that satisfies the requirement PyQtWebEngine
ERROR: No matching distribution found for PyQtWebEngine