(Chore) read facetec keys and config settings via api

Description

removed license creds from env & Config
added api client method to get license
calling license just once (license stored in the zoom global state)
updated initialize call to get license (production mode only)

About #2268
Checklist:

[x] PR title matches follow: (Feature|Bug|Chore) Task Name
[x] My code follows the style guidelines of this project
[x] I have followed all the instructions described in the initial task (check Definitions of Done)
[x] I have performed a self-review of my own code
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] I have added reference to a related issue in the repository
[x] I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
[ ] I have added screenshots related to my pull request (for frontend tasks)
[ ] I have pasted a gif showing the feature.
[x] @mentions of the person or team responsible for reviewing proposed changes

we should probably keep the env for dev/local development and testing
yep, i'm applying such changes right now

add retry on license fetch

ok

how are we handling dev key vs prod key?

license key (ZOOM_LICENSE_TEXT) is required for prod only.
the device key identifier (called ZOOM_LICENSE_KEY which isn't 100% right but we won't change env var names) isn't rotated. It's set in the env and won't change never.

@sirpy suggestions were applied


[zh-cn] Resync configure probes page

I open this issue https://github.com/kubernetes/website/issues/34630 for tracking.

/lgtm


Add completeness (by dataset) chart

Motivated by this PR that I repeatedly go back to just for the image that is really useful to understand the available data in the datasets being linked.
Example with linked data
# For linked datasets, and a supplied list of column names
cols = [
    "forename1_std",
    "surname_std",
    "dob",
    "pnc_id",
    "cro_number",
    "booking_no_arr",
    "crn",
    "asn_arr",
    "ptiurn_arr",
    "committal_date_arr",
    "offence_ho_code_arr",
    "postcode_unvalidated_arr"
]
linker.completeness_chart(cols=cols)

Example with demo dedupe-only
Less usefully, it does work for single datasets, where it is effectively an alternative to missingness_chart
# Added to 01_Exploratory_analysis.ipynb (splink_demos)
linker.completeness_chart()

Looks good.
Some suggestions here:
https://github.com/moj-analytical-services/splink/pull/677
And you've forgotten to commit the completeness.json chart spec!

Looks good.
Some suggestions here: #677
And you've forgotten to commit the completeness.json chart spec!

Thanks, that's better than the (uncommitted) check for source dataset column that I used, and of course you have a method for the right columns :+1:

Looks good now.  Please could you remove the commented out code and then feel free to merge


Unable to change stack limit

Maybe I'm doing something wrong, but I'm unable to set the stack limit for miri.
If I call cargo miri test --help it says:
The config flag `miri` is automatically defined for convenience. You can use
it to configure the resource limits

    #![cfg_attr(miri, memory_size = 42)]

available resource limits are `memory_size`, `step_limit`, `stack_limit`

After adding
#![cfg_attr(miri, stack_limit = 100000)]

to my lib.rs I get "stack limit reached error" at the same depth.

@oli-obk did that get lost in the move to rustc?

I think it was.  For quickly grepping rustc's repository it looks like this value is hardcoded to 100:
https://github.com/rust-lang/rust/blob/f47ec2ad5b6887b3d400aee49e2294bd27733d18/src/librustc/session/mod.rs#L1188
I didn't find any place where this field gets changed (https://github.com/rust-lang/rust/search?q=const_eval_stack_frame_limit).
Not sure about the other configurations (memory_size, step_limit).

Hm. yes, they seem to have gotten lost. We should probably just remove that limitation entirely, just like we got rid of the memory limit.

Currently rustc uses this stack_limit = 100 when evaluating const fn.
Do we want to keep this limit for compilation?

For const eval we can just use the recursion limit, which is already configurable by users

The bug part of this is now "fixed" by removing this text from --help.
I am turning this into a ticket to track the enhancement request that the stack limit should be changeable.

I think https://github.com/rust-lang/rust/pull/66726 should address this?
I would just like to note that I hit the stack frame limit exactly, which prompted me to check how difficult it'd be to change this:
>cargo miri test
[...]
error: Miri evaluation error: reached the configured maximum number of stack frames
   --> D:\usr\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib\rustlib\src\rust\src\libcore\mem\mod.rs:245:5
    |
245 |     intrinsics::size_of::<T>()
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^ Miri evaluation error: reached the configured maximum number of stack frames
    |


[Request] Documentation about NaughtyInspector usage

I only saw the info in the readme.md stating some attributes need the NaughtyInspector to work correctly. Maybe I overlooked something?

How do we know what attributes are affected?
Is it sufficient to create a simple "empty" Inspector class (see below) for the classes using the affected attributes? Provide an example.

public class MyInspector : NaughtInspector
{ }

Thanks. I will update the documentation. Basically only the custom attributes won't work. By custom, I mean the ones that aren't using Unity's property drawer system - ReorderableList, Button, ShowNonSerializedField, ShowNativeProperty. All meta attributes won't work as well - EnableIf, DisableIf and so on...

Documented in abea5e9e7cde63551215259850991d5d43c7b48b


Use a access_origins parameters for the security groups

This pull request is a partial/independant reply to the private VPC issue ( #14 )

@rmessner - Can you elaborate on the purpose of this PR?
If it does not stand alone as a valuable feature, but instead is part of the private VPC work, then could you collapse this PR into #108?
I'd like a single PR to cover adding private VPC support to Flintrock.

@nchammas - This feature alone allow user using Public Subnet to specify more than their own IP who is allowed to access spark pointed in this issue #38 .
But if you prefer a single PR anyway, i can easily merge it.
It's up to you

Ah, I see now. Sorry, I didn't realize at first that this was about #38. (I added a note to that effect to the original PR text so that the issue automatically gets closed when this PR is merged.)
I think this is fine as a separate PR. I will look at it more closely this week.

@rmessner - Now that you can associate specific security groups with a cluster on launch (#112), you can enable access to the cluster on any set of addresses via those security groups.
I prefer that approach since is it more general and since I think it can save us from having to add another option to the interface.
As a result, I'm going to close this PR. But please chime in if you disagree with my position or think I've missed something.


Fix index out of range error.

Many thanks 👍

Coverage decreased (-0.3%) to 51.207% when pulling 204277724bc8036ee78551185e54c2d13309a4d3 on wuestholz:devel into ccb341f188c71ec4ebae2a4f4561d862200ac01f on jayhorn:devel.


Multiple account feature

Hello !
I've install your app on my android tab, i love the concept of this app, Alarm management and web browser in same app, it's really amazing for WAF ! Thank you very much !
I use your app with my Jeedom, I've make it work without difficults and it do the job. I wanna know if it was possible to add some " extra "features. I understand you created this app to integrate with Home Assistant, I think it can stay compatible, just they will not be used by it:
=> Possibility to add an account manager:  With multiple passwords for multiple users. When action is done ( activate / disable alarm ), we can publish to MQTT who's activate / disactivate alarm ( by publish username a new MQTT topic ). This can be used to trigger who did action, and of course increase security because each user have is own password.
Example:
Create user Bob ( pin 1234 ) and Paul ( pin 4321 )
When alarm is disabled with pin 4321, we can publish to MQTT "home/alarm/user Paul" ( with of course home/set DISARM ).
I hope you understand what I mean, tell me if you need more details about my requets.
I wanna tell you again thank you, I love your app :D

Hello and thanks for your suggestion.   I like the feature idea and the way you outlined the MQTT message.   I am glad you have the application working with Jeerdom as it was designed to work with any MQTT client.  I’ve been meaning to check out Jeerdom.   I will add your request to the feature list.

Thank you very much ! =)


Class Goals _Nikhil

Nikhils Goals

[] learn hot wo use issues on GitHub
@githubteacher what other goals should I add here?

https://examples.javacodegeeks.com/wp-content/uploads/2016/03/Screen-Shot-2.jpg
https://www.google.com/url?sa=i&source=images&cd=&ved=2ahUKEwihnozHj5njAhUGZd8KHfjvDxUQjRx6BAgBEAU&url=%2Furl%3Fsa%3Di%26source%3Dimages%26cd%3D%26ved%3D2ahUKEwihnozHj5njAhUGZd8KHfjvDxUQjRx6BAgBEAU%26url%3Dhttps%253A%252F%252Fexamples.javacodegeeks.com%252Fdesktop-java%252Fide%252Feclipse%252Feclipse-tutorial-beginners%252F%26psig%3DAOvVaw2hOvlkGwHnCm-EO8kLQMfG%26ust%3D1562256078406589&psig=AOvVaw2hOvlkGwHnCm-EO8kLQMfG&ust=1562256078406589

Trial comment by Nikhil.
Anybody seeing this issue please take a look


Fix the Alignment of Text in About Section

I changed text-align to justify large displays .
And did text-align center for small devices .

@Ayushmaanagarwal1211  Need O/P Images of different views

For Laptops

For Tabs

For Small devices


feat: advanced analytics for timeseries in echart viz

🏆 Enhancements
added advanced analytics in echart viz
server-side codes at: https://github.com/apache/superset/pull/15279
YTD(year-to-date) + time shift

YoY(year-over-year) / MoM / QoQ ....

redesign

redesign Time shifting on all time-series-like chart(currently only in time-series chart, but it is easy to migrate to others)

rolling calculation on all time-series-like chart(currently only in time-series chart, but it is easy to migrate to others)

How to test it.
A. parpare codes

pull associated PR in superset-ui: https://github.com/apache-superset/superset-ui/pull/1170
pull PR in superset: https://github.com/apache/superset/pull/15279
the directory like this:

$ tree -L 1
.
├── superset-ui
└── superset

B. Build superset-ui
run after commands in terminal
$ cd superset-ui
superset-ui$ yarn clean
superset-ui$ rm -rf ./{packages,plugins}/*/node_modules ./node_modules
superset-ui$ yarn clean-npm-lock

superset-ui$ yarn
superset-ui$ yarn build

C. Build superser-frontend
$ cd superser/superset-frontend
superset-frontend$ npm ci
superset-frontend$ npm link --legacy-peer-deps ../../superset-ui/plugins/plugin-chart-echarts/ ../../superset-ui/packages/superset-ui-chart-controls/ ../../superset-ui/packages/superset-ui-core/

D. Hack core module

change package.json in superset-ui-core

diff --git a/packages/superset-ui-core/package.json b/packages/superset-ui-core/package.json
index 30a78f1e..a67dfd40 100644
--- a/packages/superset-ui-core/package.json
+++ b/packages/superset-ui-core/package.json
@@ -32,8 +32,6 @@
   },
   "dependencies": {
     "@babel/runtime": "^7.1.2",
-    "@emotion/cache": "^11.1.3",
-    "@emotion/react": "^11.1.5",
     "@emotion/styled": "^11.3.0",
     "@types/d3-format": "^1.3.0",
     "@types/d3-interpolate": "^1.3.1",
@@ -65,6 +63,8 @@
     "@types/react": "*",
     "@types/react-loadable": "*",
     "react": "^16.13.1",
-    "react-loadable": "^5.5.0"
+    "react-loadable": "^5.5.0",
+    "@emotion/cache": "^11.1.3",
+    "@emotion/react": "^11.1.5"
   }
 }

build each package of superset-ui

$ cd superset-ui/packages/superset-ui-core/
superset-ui-core$ yarn

$ cd superset-ui/packages/superset-ui-chart-controls/
superset-ui-chart-controls$ yarn

$ cd superset-ui/plugins/plugin-chart-echarts/
plugin-chart-echarts$ yarn

E. run dev server in Superset
$ cd superset/superset-frontend
superset-frontend$ npm run dev-server

@zhaoyongjie
Could you add this to the Mixed Time Series chart as well? That would unblock my team tremendously

@zhaoyongjie
Could you add this to the Mixed Time Series chart as well? That would unblock my team tremendously

Yes, It's addons for all time-series charts. I will migrate this function to all time-series charts.

No biggie! Thanks for the review again.
On Thu, Jul 8, 2021 at 00:42 andrewscarani @.***> wrote:

@.**** commented on this pull request.
In
packages/superset-ui-chart-controls/src/utils/advancedAnalytics/rollingWindow.ts
https://github.com/apache-superset/superset-ui/pull/1170#discussion_r665540837
:

 }

 return [metric.label, metric.label];

}),
);
if (formData.rolling_type === RollingType.Cumsum) {
// rolling must be the first operation(before pivot or other transforms)
post_processing.unshift({

 operation: 'cum',

 options: {

   operator: 'sum',

   columns,

 },

});
}

if ([RollingType.Sum, RollingType.Mean, RollingType.Std].includes(formData.rolling_type)) {

Sorry, I see that now it's returning from the if statement whereas before
it wasn't. Looks good now
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache-superset/superset-ui/pull/1170#discussion_r665540837,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPMKUVAQI4JPFMX4V4MDPTTWR7ZDANCNFSM46VDQ4YA
.

I'd love to see some more documentation comments explaining the how and why of the new functions. That said, this is some awesome work! Happy to approve after feedback is responded to.

I'd love to see some more documentation comments explaining the how and why of the new functions. That said, this is some awesome work! Happy to approve after feedback is responded to.

I have changed it accordingly. Thanks for the reveiw.

Hi @zhaoyongjie, I explicitly asked on this PR if this functionality would be added to the Mixed Time Series chart, but I'm not seeing that in Preset even a month later. Is there any way to easily add that in another PR?


Improve paste selection

Avoid additional erroneous newlines being present when copying the paste content either via mouse selection or Ctrl+a.
No newlines are copied by chromium, firefox still inserts a prefix newline.

Can you add Fixes #72 either in the commit summary or body to close the associated ticket? Change works fine for me, thanks!


[WebUI] Model Merging

This PR brings Model Merging to the WebUI.
Inside the Model Manager, you can now find a new button called Merge Models. Rest of it is self explanatory.

Also thanks for cleaning up the exception handling.

Also thanks for cleaning up the exception handling.

No issues. Just a result of me being lazy. :P


[1.7.10 possible 1.10.2] Scripts are not synced to client if the files have the same name.

Description: If a client and server have a script with the same name, the server does not send the script to the client, even if the server's or client's files have different contents/file sizes. If the recipes are added to a separate file that the client does not have, the recipes will sync.
Steps I did to reproduce the bug: I am playing Project Ozone 2 and the server decided to add recipes for various items. So I added some recipes into the titanCrafting.zs file. I got reports that the items had no recipes and I told them that they had been added and to just relog. Upon relogging I noticed as well that the recipes were no longer there on my client. If the command "/mt reload" is done, the recipes appear, this is true upon server restart.
Forge version: <IP_ADDRESS>4
Modpack: Project Ozone 2 3.1.9
CraftTweaker version: 3.0.11

uh yea, kinda intended, we checking the whole contents of each script would come with a huge performance overhead. I have seen scripts with thousands of lines, it would take a while to check, by using the names, it is a much quicker check.


feat(telemetry-and-inputs): add snapshot input and snapshot artifact upload

Details
This adds the snapshot input available in the accessibility-insights-scan package to the ADO Extension. The snapshot input takes a snapshot of every page scanned and uploads it to a separate artifact folder using the existing artifact name and appending -snapshots.
Here is an example run from my test extension
Here is a direct link to the artifacts for that run
Motivation
Feature work, this will allow users to see what the scanner is seeing when pages are being scanned. This will help in debugging unexpected scan results.
Context
There were a few approaches for the uploads that I considered. When there are a lot of snapshots there will end up being a fairly large stream of upload information in the logs as it uploads them one by one, as you can see in the example run linked above.
There is an alternative that I considered that cuts down on the noise but has other drawbacks.  We could delete all additional files in the key_value_stores working directory (json/diagnostic files output by the scanner), and then upload the entire directory at once. I decided to match on the screenshot instead and upload them each separately in case we ever want to use those files for anything in the future.
It also didn't feel like the noise at the bottom was much of a problem, as the UI defaults to showing the top of the log, which is where the key information will still reside.
Pull request checklist

[n/a] Addresses an existing issue: Fixes #0000
[x] Added relevant unit test for your changes. (yarn test)
[x] Verified code coverage for the changes made. Check coverage report at: <rootDir>/test-results/unit/coverage
[x] Ran prechecking (yarn precheckin)

Codecov Report

Merging #1525 (172aa07) into main (d8131e3) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1525   +/-   ##
=======================================
  Coverage   94.43%   94.44%
=======================================
  Files          38       38
  Lines        1043     1044    +1
  Branches      134      135    +1
=======================================
+ Hits          985      986    +1
  Misses         58       58

Impacted Files
Coverage Δ

packages/shared/src/task-config.ts
75.00% <ø> (ø)

...kages/shared/src/scanner/crawl-argument-handler.ts
100.00% <100.00%> (ø)

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


Documentation feedback

Here are a few high level thoughts on the current documentation:
Looking at the main example on the main page I'm curious if it is realistic:
from dask_jobqueue import PBSCluster

cluster = PBSCluster(processes=6, threads=4, memory="16GB")
cluster.start_workers(10)

from dask.distributed import Client
client = Client(cluster)

Should we include project, queue, resource specs, and other keywords that might both be necessary for realistic use and also recognizable to users of that kind of system?  Similarly I think it would be very useful to include a few real-world examples in the example deployments documentation.  I suspect that this was the original intent of that page (nice idea!).   Perhaps we can socialize this on the pangeo issue tracker and ask people to submit PRs for their clusters?
History
I recommend that we remove the history section from the main page
Description of how it works
My experience trying to explain these projects to users of HPC systems is that most of them are familiar with job scripts.  I wonder if we might include a "How does this work?" section that shows the job script that we generate, and explain that we submit this several times.
Thoughts?

Socialized the examples documentation on https://github.com/pangeo-data/pangeo/issues/218

All your suggestion makes sense, thanks for starting this!

I agree with all the suggestions.
The main example in the main page is realistic to me (I probably updated it according to my use :smiley:  ). We've got a default routing queue on my PBS cluster, project option is not mandatory, and I like automatically computed resource_specs (and once again I'm biased :grin: ).
But I fully agree that this is not the case for everyone! There is a more detailed example that we could put here in the PBSCluster docstrings:
cluster = PBSCluster(queue='regular', project='DaskOnPBS',
                             local_directory=os.getenv('TMPDIR', '/tmp'),
                             threads=4, processes=6, memory='16GB',
                             resource_spec='select=1:ncpus=24:mem=100GB')

interface keyword is important too and should appear.

I provided this on the dask documentation issue but I'll paste it here as well:
cluster = PBSCluster(processes=18, threads=4, memory="6GB",
                     project='P48500028', queue='premium',
                     resource_spec='select=1:ncpus=36:mem=109G',
                     walltime='02:00:00', interface='ib0')

This is what I've been using on NCAR's HPC system Cheyenne.

Looking at the most recent build of the docs, we could probably cleanup the docstrings a bit for the API docs:

I'd like to see us cleanup a few things:

figure out how to drop the __init__ method from the abstract job queue class
fill in docstrings for job_script, scale_down, scale_up, and stop_workers methods

Some of the docstrings probably can go in the distributed.deploy.cluster (all except for job_script).

Actually, everything could be done here and was fairly straightforward: https://github.com/dask/dask-jobqueue/commit/f7b14fe91ea0984f31dcd02301ce71a7477b3984

Closing this, as this has been adressed in #50 by @mrocklin and by @jhamman above.


Vortex Scans domain changed!

Source information
Vortex Scans 1.4.38
Source language
English
Source new URL
https://vortexscans.org/home
Other details
The extension url has changed to the old url https://vortextoon.com/home
Acknowledgements

[X] I have updated all installed extensions.
[X] I have opened WebView and checked that the source URL has not changed yet.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I will fill out all of the requested information in this form.

I don't know how but the extensions url have changed to old url https://vortextoon.com/home. Reopen

Sorry, when I checked there was no issue reported #5861 . I was in the middle typing this. Also I searched for his but there was no results. It seem I have misspelled and serch for it. (vortext instead of vortex)


Add module for retrieving package properties

Provide a common way for retrieving package properties including

Package Directory
Package Version
Package ChangeLog Path
Package ReadMe Path

I really like this @chidozieononiwu
Can you take a look at VerifyPackages in the artifact-metadata-parsing.ps1? I feel like both of these actually do quite similar actions. I think the only difference is that VerifyPackages also checks the status of the SHA, etc and includes the PackageId in there.
I feel like if we're going to be parsing package metadata, we should make use of the same one everywhere. If you feel that it would be a ton of rework, please feel free to file a follow-up issue.

/check-enforcer override


Strange Crash

So, I'm playing 1.12.2 with a lot of mods and recently I added HBM Nuclear Tech and the world started crashing shortly after I enter it. I sent this crash report to them, and they recommended me posting it in here, since it envolves some problem with "new and old states of chunks". I hope someone here can help me with this. ´
crash-2021-03-16_17.28.06-server.txt

Not related to NEID too. Mod's author sould add here something like else return;, so that it wont call world.setBlock if state isn't changed.

Not related to NEID too. Mod's author should add here something like else return;, so that it wont call world.setBlock if state isn't changed.

Any idea of which mod may have caused this crash?

HBM Nuclear Tech port by @Drillgon200 , i guess

Can you produce this only with Nuclear Tech and NEID?

I'll try. But I don't think so.
Also, someone suggested that it's may also be a problem with Mekanis + Galacticraft. He said that, for some reason, the game runs out of IDs

Any news on this?

I tried three cases:

With ONLY Hbm + NEID;
2)With Hbm + NEID + Mekanism + Galacticraft
With Hbm + NEID + Mekanism + Galacticraft + Logistic Pipes

All resulted in nothing. The game ran smooth. I also tried creating a new world, and surprisingly it didn't crash at all. I suspected Logistic Pipes could cause the crash, since it mostly occur when I open the Remote Controler, but now I don't have any idea of what mod can be causing it.

Just in case, I'm sending a print screen of my mods folder:

Can i somehow get your modpack and world?

Yes, I'll try uplouding it and send you the link

Here, try and tell me if the link works:
https://drive.google.com/file/d/1zcfsdBd5e2b-TFZb2th7P5xcr_PvQDPn/view?usp=sharing
Also, feel free to roam the world, if you can.
Note: This world is not actually set in the overwolrd, but the  Lost Cities dimension

Just saw the link was removed. Add me ins Discord: TheAndreo#8440

Mystcraft bug.
Happens, for example, with blocks that have setBlockState in onBlockAdded. (vanilla chest or HBM battery).
World.setBlockState needs old block state to compare with. It asks chunk at that position, and gets MystEmptyChunk, even though there were writes to that chunk through setBlockState. MystEmptyChunk always returns AIR as old blockstate. Chunk.setBlockState then writes changed block state to block storage and calls onBlockAdded, because new state (HBM's battery) and AIR are different. And then it repeats.

Mystcraft issue report


Clarify the semantics of tearing off an inherited abstract method

I have the following code:
abstract class A {
  void test();
}
class B extends A {
  void test() {
    super.test; // code in question
  }
}

Dart analyzer does not produce any static errors, but the CFE is currently issuing an error indicating that the class A does not define a getter named test. I couldn't find any place in the spec that would indicate that this should be an error. In particular, section 16.19.2 "Super Getter Access and Method Closurization" appears to indicate that this should be valid.
Should this code produce an error?
If so, then I'll open an analyzer issue. If not, then I'll open a CFE issue.

Note that CFE only produces error if class B is not abstract and enableSuperMixins is false; it compiles the program successfully otherwise.

/cc @leafpetersen @eernstg

Yes, this should be an error.  See also https://github.com/dart-lang/sdk/issues/33662 .

Filed a spec bug to make sure the text reflects this: https://github.com/dart-lang/sdk/issues/33963 .


Crash in audio library

From Mark Incitti on Slack:
crash in audio - this has happened in random(?) places, 3 times in about 20 runs.
Log file:
2016-09-25 23:58:46,318 : INFO : kivy : Factory: 179 symbols loaded 2016-09-25 23:58:47,843 : INFO : kivy : Image: Providers: img_tex, img_dds, img_gif, img_sdl2 (img_pil, img_ffpyplayer ignored) 2016-09-25 23:58:49,531 : INFO : kivy : VideoGstplayer: Using Gstreamer <IP_ADDRESS> 2016-09-25 23:58:49,532 : INFO : kivy : Video: Provider: gstplayer 2016-09-25 23:58:49,778 : INFO : kivy : Loading MPF-MC controller 2016-09-25 23:58:49,778 : INFO : mpfmc : Mission Pinball Framework Media Controller v0.31.6 2016-09-25 23:58:49,779 : INFO : mpfmc : Machine path: C:\nbx 2016-09-25 23:58:49,779 : INFO : mpfmc : Starting clock at 30.0Hz 2016-09-25 23:58:50,569 : INFO : kivy : Text: Provider: sdl2 2016-09-25 23:58:51,129 : INFO : kivy : OSC: using <thread> for socket 2016-09-25 23:58:51,136 : INFO : kivy : Window: Provider: sdl2 2016-09-25 23:58:52,982 : INFO : kivy : GL: GLEW initialization succeeded 2016-09-25 23:58:52,982 : INFO : kivy : GL: OpenGL version <b'3.3.11672 Compatibility Profile Context'> 2016-09-25 23:58:52,982 : INFO : kivy : GL: OpenGL vendor <b'ATI Technologies Inc.'> 2016-09-25 23:58:52,982 : INFO : kivy : GL: OpenGL renderer <b'ATI Radeon HD 3400 Series'> 2016-09-25 23:58:52,983 : INFO : kivy : GL: OpenGL parsed version: 3, 3 2016-09-25 23:58:52,983 : INFO : kivy : GL: Shading version <b'3.30'> 2016-09-25 23:58:52,983 : INFO : kivy : GL: Texture max size <8192> 2016-09-25 23:58:52,983 : INFO : kivy : GL: Texture max units <16> 2016-09-25 23:58:53,050 : INFO : kivy : Shader: fragment shader: <b'Fragment shader was successfully compiled to run on hardware.'> 2016-09-25 23:58:53,099 : INFO : kivy : Shader: vertex shader: <b'Vertex shader was successfully compiled to run on hardware.'> 2016-09-25 23:58:53,149 : INFO : kivy : Shader: program: <Vertex shader(s) linked, fragment shader(s) linked.> 2016-09-25 23:58:53,152 : INFO : kivy : Window: auto add sdl2 input provider 2016-09-25 23:58:53,152 : INFO : kivy : Window: virtual keyboard not allowed, single mode, not docked 2016-09-25 23:58:54,563 : INFO : AudioInterface : Initialized 0.31.0.dev17 2016-09-25 23:58:54,817 : INFO : kivy : Base: Start application main loop 2016-09-25 23:58:54,972 : INFO : kivy : Shader: program: <Vertex shader(s) linked, fragment shader(s) linked.> 2016-09-25 23:58:54,980 : INFO : kivy : Shader: program: <Vertex shader(s) linked, fragment shader(s) linked.> 2016-09-25 23:58:54,981 : INFO : kivy : GL: NPOT texture support is available 2016-09-25 23:58:54,992 : INFO : kivy : Shader: program: <Vertex shader(s) linked, fragment shader(s) linked.> 2016-09-25 23:58:54,996 : INFO : kivy : Shader: fragment shader: <b'Fragment shader was successfully compiled to run on hardware.'> 2016-09-25 23:58:55,009 : INFO : kivy : Shader: program: <Vertex shader(s) linked, fragment shader(s) linked.> 2016-09-25 23:58:55,124 : INFO : kivy : Shader: program: <Vertex shader(s) linked, fragment shader(s) linked.> 2016-09-25 23:58:55,132 : INFO : kivy : Shader: program: <Vertex shader(s) linked, fragment shader(s) linked.> 2016-09-25 23:58:55,140 : INFO : kivy : Shader: program: <Vertex shader(s) linked, fragment shader(s) linked.> 2016-09-25 23:58:55,143 : INFO : kivy : Shader: fragment shader: <b'Fragment shader was successfully compiled to run on hardware.'> 2016-09-25 23:58:55,153 : INFO : kivy : Shader: program: <Vertex shader(s) linked, fragment shader(s) linked.> 2016-09-25 23:59:16,541 : INFO : MPF-MC BCP Server : Starting up on localhost port 5050 2016-09-25 23:59:16,548 : INFO : MPF-MC BCP Server : Waiting for a connection... 2016-09-25 23:59:16,731 : INFO : MPF-MC BCP Server : Received connection from: <IP_ADDRESS>:53890 2016-09-25 23:59:22,727 : INFO : kivy : GL: Unpack subimage support is available 2016-09-26 00:03:49,057 : WARNING : Track.0.TrackStandard.sfx : Received a notification message for a sound instance (id: 222467520) that is no longer managed in the audio library. Notification will be discarded. 2016-09-26 00:03:49,058 : WARNING : Track.0.TrackStandard.sfx : Received a notification message for a sound instance (id: 222467520) that is no longer managed in the audio library. Notification will be discarded. 2016-09-26 00:13:22,803 : INFO : kivy : Base: Leaving application in progress... 2016-09-26 00:13:22,804 : ERROR : kivy : unorderable types: NoneType() < SoundInstance() Traceback (most recent call last):   File "c:\python34\lib\site-packages\mpfmc\commands\mc.py", line 153, in __init__     thread_stopper=thread_stopper).run()   File "c:\python34\lib\site-packages\kivy\app.py", line 828, in run     runTouchApp()   File "c:\python34\lib\site-packages\kivy\base.py", line 487, in runTouchApp     EventLoop.window.mainloop()   File "c:\python34\lib\site-packages\kivy\core\window\window_sdl2.py", line 619, in mainloop     self._mainloop()   File "c:\python34\lib\site-packages\kivy\core\window\window_sdl2.py", line 362, in _mainloop     EventLoop.idle()   File "c:\python34\lib\site-packages\kivy\base.py", line 327, in idle     Clock.tick()   File "c:\python34\lib\site-packages\kivy\clock.py", line 515, in tick     self._process_events()   File "c:\python34\lib\site-packages\kivy\clock.py", line 647, in _process_events     event.tick(self._last_tick, remove)   File "c:\python34\lib\site-packages\kivy\clock.py", line 406, in tick     ret = callback(self._dt)   File "c:\python34\lib\site-packages\mpfmc\core\audio\__init__.py", line 221, in tick     self.audio_interface.process()   File "Pinball\mpf_mc\mpfmc\core\audio\audio_interface.pyx", line 638, in Pinball.mpf_mc.mpfmc.core.audio.audio_interface.AudioInterface.process (mpfmc/core/audio/audio_interface.c:9819)   File "Pinball\mpf_mc\mpfmc\core\audio\audio_interface.pyx", line 1871, in Pinball.mpf_mc.mpfmc.core.audio.audio_interface.TrackStandard.process (mpfmc/core/audio/audio_interface.c:17799)   File "Pinball\mpf_mc\mpfmc\core\audio\audio_interface.pyx", line 1971, in Pinball.mpf_mc.mpfmc.core.audio.audio_interface.TrackStandard._get_next_sound (mpfmc/core/audio/audio_interface.c:18903)   File "c:\python34\lib\queue.py", line 195, in get_nowait     return self.get(block=False)   File "c:\python34\lib\queue.py", line 177, in get     item = self._get()   File "c:\python34\lib\queue.py", line 233, in _get     return heappop(self.queue) TypeError: unorderable types: NoneType() < SoundInstance() 2016-09-26 00:13:22,861 : INFO : kivy : Stopping child threads... (3 remaining) 2016-09-26 00:13:23,050 : INFO : mpfmc : Stopping... 2016-09-26 00:13:23,070 : INFO : mpfmc : Loop rate 32.37 Hz 2016-09-26 00:13:23,111 : INFO : MPF-MC BCP Server : Stopping BCP sending thread 2016-09-26 00:13:23,370 : INFO : kivy : All child threads stopped.

This crash was caused when more than one sound is stopped that is still in the queue.  The queue entry is left in the queue (so the heap structure does not get corrupted) and the sound instance object in the entry is set to None.  This lead to priority queue sorting problems as there could then be several entries that have the same sort priority and the None value could not be used to break a tie.  A queue entry count/id was added to ensure that every queue entry has a unique identifier to use to break ties when sorting.

Fix included in 0.31.7.


New file does not appear in Leaderf file nor in results of Leaderf rg

vim or neovim?

[ ] vim
[x] neovim

Output of vim --version or nvim --version:
  NVIM v0.4.4
  Build type: Release
  LuaJIT 2.0.5
  Compilation: /usr/bin/clang -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DNDEBUG -DMIN_LOG_LEVEL=3 -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fno-common -fdiagnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -I/tmp/neovim-20200808-21571-gjtb7k/neovim-0.4.4/build/config -I/tmp/neovim-20200808-21571-gjtb7k/neovim-0.4.4/src -I/usr/local/include -I/tmp/neovim-20200808-21571-gjtb7k/neovim-0.4.4/deps-build/include -I/usr/include -I/usr/local/opt/gettext/include -I/tmp/neovim-20200808-21571-gjtb7k/neovim-0.4.4/build/src/nvim/auto -I/tmp/neovim-20200808-21571-gjtb7k/neovim-0.4.4/build/include
  Compiled by brew@HighSierra

  Features: +acl +iconv +tui
  See ":help feature-compile"

     system vimrc file: "$VIM/sysinit.vim"
    fall-back for $VIM: "/usr/local/Cellar/neovim/0.4.4/share/nvim"

  Run :checkhealth for more info

Output of :echo has("python"):  0

Output of :echo has("python3"):  1

Output of :py print(sys.version):  no python provider

Output of :py3 print(sys.version):
 3.9.5 (default, Jun  8 2021, 09:03:00)
[Clang 10.0.0 (clang-1<IP_ADDRESS>)]

Output of :echo g:Lf_Debug_Cmd: rg --no-messages --files --color never --hidden

Output of :echo g:Lf_FilesFromCache:  1

Operating system:

[ ] Linux
[x] Mac OS X
[ ] Windows
[ ] Etc.

Configurations related to LeaderF in vimrc:

let g:Lf_UseVersionControlTool=0

nnoremap <C-p> :Leaderf file<cr>
nnoremap <Leader>f :Leaderf rg -S<cr>

Describe your question, feature request, or bug.
When I create a new file with either :e new_file.txt, or a NerdTree plugin, or with terminal, or any other way - in all the cases file just not appears in Leaderf file files, nor it does in the results of Leaderf rg. Existing issues and their solutions didn't help me to solve this problem.
Steps to reproduce

Open Vim/NeoVim
Open some file with :Leaderf file
Create new file with :e new_file.txt
Save new file with :w
Go to some other file, input :Leaderf file from there
You will not see new_file.txt there

Actual behaviour
No new new_file.txt file inside of :Leaderf file
Expected behaviour
New new_file.txt file inside of :Leaderf file

First, you can use F5 to refresh the result.
Or
let g:Lf_UseCache = 0
let g:Lf_UseMemoryCache = 0

Oh, thank you!


Let's build an Alteryx deployment tool

Get Alteryx workflow from network location
Find a replace robotic passwords by retrieving them from cyberark
Encrypt the workflow
Zip the original and encrypted workflows
Push the code to Git
Call Alteryx api to publish encrypted workflow to the gallery

Done


Configuring the deployment process

Since we working on Censor Tracker in Github, but our CI/CD is Gitlab we need to configure our deployment process somehow. Maybe we can create a mirror of this repository on Gitlab or something.
@HellsYeah @msva What do you think?

Another way is setting up the Github Actions, right?

@HellsYeah The main question is "How to securely deploy the project on our server?". I'm not sure that we can make Github Actions work with our server, but I'm also not sure in the opposite, soo...


Extract versions from Gradle build script

This request moves the plugin and dependency declarations in a new gradle/libs.versions.toml file for better readability.
See the TOML website for more details about its syntax.

Yeaaaah, nice migration, long live the  libs.versions.toml file


Is there textIndex in 0.20.1v?

I can't find textIndex in  your document and I need it for my project
please develop it fast or say me how I can implement it for my project.
db.reviews.createIndex(
   {
     subject: "text",
     comments: "text"
   }
 )

v0.21.0 is supported

v0.21.0 is supported


How to avoid EventEmitter Max Listeners warning?

I use Ipc to exchange data:
     <button id="cmdShowData">Process</button>

    <script>
        (function () {
            const { ipcRenderer } = require("electron");

            document.getElementById("cmdShowData").addEventListener("click", () => {
                ipcRenderer.send("doProcess", 'args');
            });

            ipcRenderer.on('sendData1', (event, msg) => {
                document.getElementById("ResponseOK").innerHTML = msg;
            });
        }());
    </script>

But after 11 page loadings I receive:

(node:29284) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 doProcess listeners added. Use emitter.setMaxListeners() to increase limit

How and where I can increase this limit?
Is it problem or it is warning only?

The code is called again and again when you reload your page. You should therefore not repeat an event registration.
It is best to register your code globally rather than directly on the same page. Or use the unload event and cancel the registrations.
https://developer.mozilla.org/en-US/docs/Web/API/Window/unload_event

Thanx, ok.
Now I do it in Controller/Action method.
But when I click menu the Controller object created again, its constructor etc.
Where is the best place to register it?

It's best to use a single-page app, or see the code from our demo app.
Alternatively, you can also use Blazor.
https://github.com/ElectronNET/electron.net-api-demos


Add a license to the repository

I recently came across your project quantum-computing-basics and I noticed that the repository github.com/quantum-kittens/quantum-computing-basics currently does not have a license specified.
Why Adding a License is Important?
As GitHub documents

Public repositories on GitHub are often used to share open source software. For your repository to truly be open source, you'll need to license it so that others are free to use, change, and distribute the software.

Would be possible to add a valid license to this repository? Probably something really OpenSource such as the Apache 2.0 license would fit really well.
Thanks!

Done. Thanks @cbjuan!


add a submap path

I'm especially pleased with the last test just working, as I only thought of trying it after watching your talk where you did the same thing with a set.
Having submaps created by simulating repeatedly calling get, rather than behaving like select-keys is maybe questionable, but I figured clojure already has a select-keys function.
/cc @nathanmarz

A few comments:

Instead of inserting non-existent keys into the map with the value nil, just leave them out of the map. Using the semantics of select-keys is preferable, as that way the user knows post-submap which keys were in the map originally and which ones had a nil value. Plus it will be more efficient.
Let's just have submap be the defpath and always require it to take in a sequence of keys as input. I understand the desire to use variadic input, but the separation into submap/submap* makes using this along with late-bound parameterization confusing.
Instead of using apply to do the dissoc, let's do it via a reduce on the keys. There's a significant performance difference between the two methods.

I'll also need you to sign a contributor agreement. A contributor agreement puts the project on a sound legal foundation to protect the project and its users. Can you email me at<EMAIL_ADDRESS>from where I should send the CA?

Changes made, CA signed /cc @nathanmarz


MINOR: [Dev] Add Branch Protection

This will add branch protection to main which will prevent force pushing and commits without PR(+review). We recently had an accidental push through a release script to main that would have been prevented by this.
Infra documentation regarding these settings.

Just a small clarification. I think we should not merge this until we have changed the Release process.

Hm yes agreed both that we can not merge this for now as well as that we should think about changes again (also looking at the versioning discussion)

Perhaps unrelated, but do we want to enable required_linear_history: true?

enforcing a linear commit history prevents collaborators from pushing merge commits to the branch. This means that any pull requests merged into the protected branch must use a squash merge or a rebase merge.

I would be +1 on that (again unless our release scripts use merge commits or something?)


Remove need for colon for constructs

This is a big one. It could require to fork nim to make it happen. Ideally, the proposal would be to add a {.ignoreColon.} pragma to nim that can be used for macros and templates to ignore the colon indentation requirement. This could be done for just single argument macros and templates, and it would make omni code to look like this:
struct Phasor
    p : float

def newPhasor() -> Phasor
    return Phasor.new(0.0)

init
    phasor = newPhasor()

perform
    sample
        out1 = phasor.p / samplerate
        phasor.p += 1
        phasor.p = phasor.p mod samplerate

However, this would cause confusion with using standard nim constructs, like if / case statements (among others), which use colon to separate blocks of code:
#...
#from previous comment
#...

perform
    sample
        out1 = phasor.p / samplerate
        phasor.p += 1

        if phasor.p >= samplerate:  #this colon is weird here now that it's not used anywhere else in omni
            phasor.p = 0

Solution 1:
Don't use colon only for def and struct definitions:
struct Phasor
    p : float

def newPhasor() -> Phasor
    return Phasor.new(0.0)

init:
    phasor = newPhasor()

perform:
    sample:
        out1 = phasor.p / samplerate
        phasor.p += 1
        phasor.p = phasor.p mod samplerate

This would also make the def newPhasor() : Phasor syntax possible


Add env variable to enable use of self-signed SSL certs

Some users are testing with self-signed certs
Add an explicit  environment variable,  to bypass the SSL cert validity check, and enable use of self-signed SSL cert

@Sing-Li Is there a way around this for now?

I'm getting a SELF_SIGNED_CERT_IN_CHAIN error. Has anyone figured out a workaround for this?

As a workaround you can set NODE_TLS_REJECT_UNAUTHORIZED = "0" on your docker-compose. For the code, I believe you could add something like ROCKETCHAT_SELFSIGNED_CERT and if set as true then execute process.env.NODE_TLS_REJECT_UNAUTHORIZED = "0"

+1

Related https://forums.rocket.chat/t/error-self-signed-certificate-in-certificate-chain/2495


output of [INST] <><> at the end of every answer

I am using your example and ChatOllama instead of openai chatmodel and I get some UI strange characters at the end of every answer.Following is an image :

I think you can set the stop stop sequence manually to this text this should fix it.
Not sure where this problem is coming from but I think some models have wrong chat schemas in ollama.


Silverstripe Menu doesn't display on Category and Product pages

Silverstripe menu doesn't display on pages using either Category.ss or Products.ss templates! they display just fine on other pages! is there a way to fix this?
This is the code I'm using on my Footer.ss:

      <% loop $Catalogue.Categories %>

$Title

      <% end_loop %>

      <% loop $Menu(1) %>

$MenuTitle

      <% end_loop %>

I really appreciate any help you can provide.

Hi @alialamshahi
Yes, this is an issue that derives from the Category and Product controllers not extending ContentController (so this module can be used independently of the CMS).
This is all going to drastically change in the SS4 version, bit for now I have manually added menu support in commit 0aa5453.
Make sure you use the 3.0 branch. Let me know if it works OK.

Also, just realised this was posted in 2015!? Yet GitHub only sent em a notification a couple  of days ago...
No idea what is going on there...


docs: Correct social authentication URL for development setups.

Since SOCIAL_AUTH_SUBDOMAIN is set to auth in development
environment, this is correct authentication URL.
discussion: https://chat.zulip.org/#narrow/stream/19-documentation/topic/external.20authentical.20methods

Merged, thanks @amanagr!


registry-generator : Fixes for Solidity 0.5.2

Purpose

updated generation code to fix errors and warnings when using Solidity 0.5.2

Does this introduce a breaking change?
[ ] Yes
[X] No

Pull Request Type
What kind of change does this Pull Request introduce?
[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]

Test the code

Start the web-app and press the Generate Code button.
The generated Solidity code should be 0.5.2 compatible

What to Check
Verify that the following are valid

memory is used

Other Information
See https://github.com/mstack/bootcamp-blockchain-smartcontracts/tree/master/Lab 5 (Azure Accelerators) for a working example including solidity code and C# example.

Linked to/from : https://github.com/Azure-Samples/blockchain/pull/151

Workbenchbase has been deprecated from the code generator. Can you please remove references to that base class?

Fixed, please double check

Thanks StefH, Workbench doesn't yet support Solidity 0.5.x but once it is upgraded we can take these changes


SeparationRayShape3D

Description
This issue tracks the progress of implementing the functionality of Godot's SeparationRayShape3D node type. Below you'll find all user-facing properties, methods and signals, including ones inherited from relevant node types. If you notice any missing from this list, please leave a comment down below.
Status

Not supported
Not started
Partial (not functional)
Partial (functional)
Done (as much as it can be) 👈
Done (completely)

Links
Documentation: SeparationRayShape3D
Inherits: Shape3D
Legend

🚧 indicates a partial, incomplete or broken implementation.
🛑 indicates that it's blocked or incompatible with Jolt in some way.

Properties
Uncategorized

[x] Length (length)
[x] Slide on Slope (slide_on_slope)

Properties (Shape3D)
Uncategorized

[ ] 🛑 Custom Solver Bias (custom_solver_bias)

Not supported by Jolt

[x] Margin (margin)

This tracking issue has served its purpose and will now be closed. 🫡
Any remaining work has had new issues created for it.


TP - only show 'manage keys' button on ds forms if protocol>0 or signingAl…

…gorithm is not null but only once the change is persisted

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


Optional and mandatory options

I think it would be nice to have optional and mandatory options
that way no validation is needed to be done by the user

As I've said in several previous issues #44 #25, I don't think that this sort of logic belongs in this library. The main reasoning is that usually options are not just mandatory or optional., but something more complicated. Take for example tar, it has a mutually exclusive set of options, and a bunch of options that only go with other options.
tar: You must specify one of the '-Acdtrux', '--delete' or '--test-label' options

That said, since this has been requested a few times, I think a separate validation library would be appreciated by a few people. I will work on this at some point, but I can't guarantee getting to it straight away.

Thanks!
Some how I missed #44 when searching the issues.
I agree a validation should be on different defined combinations of the arguments
In-order to decide if a set of arguments is valid or not.
I'll be happy to contribute, let me know.

I think a separate validation library is not necessary. If each option could optionally specify required and/or excluded arguments, resolving conflicts is then left to the user to define and the validation simply enforces the supplied rules.

For simple cases it would be very useful to have required options (like boost::program_options::value<>()->required()). At least it allows to drop annoying checks for an every required option.
Of course, for more complicated cases, users can figure out it with own logic.

For simple cases it would be very useful to have required options (like boost::program_options::value<>()->required()). At least it allows to drop annoying checks for an every required option.
Of course, for more complicated cases, users can figure out it with own logic.

I'm not sure if this is still open to discussion or not, because this is quite an old issue, however the only thing that would make this library be perfect to me is the required arguments and if the required arguments were not given it would simply print the --help message.
Considering that this can be an optional parameter it wouldn't force users to use this but instead would allow small more linear programs to keep it's code simpler and avoid checking things manually.

I still have some plans to address this. I wanted to make a separate validator that you can run against the parsed options.

Hi! What's the status on this issue?

I am expecting this feature.

Here's a simple and elegant way to handle "required", "optional" and actually any complex rule: by using template expressions
Here's a small working example that handles constraints of type "option 'foo' must appear exactly once and option 'bar' exactly twice" :
#include <cxxopts.hpp>
#include <cstddef>
#include <iostream>

namespace cxxopts {
namespace rules {

// =============================================================
class Option
{
public:
  Option(std::string option_name) : name(option_name) {}

  inline size_t count(const ParseResult &parsing) const
  {
    return parsing.count(name);
  }

  std::string name;
};
// =============================================================

// =============================================================
class ExactCount
{
public:
  ExactCount(Option option, size_t target_count)
    : option(std::move(option)), target_count(target_count) {};

  inline bool operator () (const ParseResult& parsing) const
  {
    return option.count(parsing) == target_count;
  }

  inline explicit operator std::string () const
  {
    if (target_count == 1)
      return "option '--" + option.name + "' must be provided exactly once";
    else return "option '--" + option.name + "' must be provided exactly " + std::to_string(target_count) + " times";
  }

protected:
  Option option;
  size_t target_count;
};

ExactCount operator == (Option option, size_t target_count)
{
  return ExactCount(std::move(option), target_count);
}
// =============================================================

// =============================================================
template <class SubExpression1, class SubExpression2>
class AND
{
public:
  AND(SubExpression1 expr1, SubExpression2 expr2)
    : expr1(std::move(expr1)), expr2(std::move(expr2)) {}

  inline bool operator () (const ParseResult& parsing) const
  {
    return expr1(parsing) && expr2(parsing);
  }

  inline explicit operator std::string () const
  {
    return "(" + std::string(expr1) + ") and (" + std::string(expr2) + ")";
  }

protected:
  SubExpression1 expr1;
  SubExpression2 expr2;

};

template <class SubExpression1, class SubExpression2>
auto operator && (SubExpression1 expr1, SubExpression2 expr2)
{
  return AND(std::move(expr1), std::move(expr2));
}
// =============================================================

template <class Rule>
void enforce(const Rule& rule, const ParseResult& parsing)
{
  if (not rule(parsing))
    throw std::runtime_error("rule not respected: " + std::string(rule));
}

}
}

int main(int argc, char *argv[])
{
  cxxopts::Options options(
    "hash-id",
    "Advertising-ID hasher");

  options.add_options()
    ("i,input", "input", cxxopts::value<std::string>())
    ("o,output", "output", cxxopts::value<std::string>())
    ("h,help", "Print usage")
    ;

  auto parsing = options.parse(argc, argv);

  auto rule = (cxxopts::rules::Option("input") == 1) and (cxxopts::rules::Option("output") == 1);

  cxxopts::rules::enforce(rule, parsing);

  std::cout << parsing["output"].as<std::string>() << std::endl;

  return 0;
}

After implementing the or and not boolean operation, any rule can be expressed and everyone is happy !


Understanding the scripts for training Guided model

Hello,
I just had a simple doubt, for training the model the command provided is
bash z_train.sh DATA_PATH MODEL_PATH
and for testing the model it is
bash z_test.sh SRC GUIDANCE RESULT_PATH MODEL_DIR MODEL_NAME DATA_BIN
and although the task seems to be guided_summarization in z_train.sh, could you tell me what guidance does the training use, and if I can specify the guidance path?

No wait sorry, I understood that there's a *.bpe.z file requirement which is the guidance.


[Feature Request] 正则表达式工具

Checklist

[X] I searched exising issues and no one else requests similar feature.
[X] I think that 25%+ users are positive towards this feature.

Describe the feature

Additional context
No response

https://github.com/864381832/xJavaFxTool-plugin/tree/master/codeTools/x-RegexTester

#1003

https://github.com/tbee/regexpbuilder


routing.md: fixed lazy matching definition

Fixes #30410

Internal previews

📄 File
🔗 Preview link

aspnetcore/fundamentals/routing.md
Routing in ASP.NET Core

Thanks for the contribution. Be sure to check out your name as a contributor about the 3rd of next month on the What's new page in the Community contributors section at the bottom.
Check out is:issue is:open label:"code help wanted" " and is:open is:issue label:"Good first issue"


fix(designable): subscribe fail when engine mount again(#53)

Before submitting a pull request, please make sure the following is done...

[x] Ensure the pull request title and commit message follow the Commit Specific in English.
[x] Fork the repo and create your branch from main.
[x] If you've added code that should be tested, add tests!
[x] If you've changed APIs, update the documentation.
[x] Ensure the test suite passes (npm test).
[x] Make sure your code lints (npm run lint) - we've done our best to make sure these rules match our internal linting guidelines.

卸载了就应该重新createDesigner，而不应该继续复用原始实例


Create Grantable.sol for access granting

🚀 Description
A smart contract might impose access control to grant high-level access to a group of accounts. Grantable solves the potential limitation of single ownership in Ownable.sol.

[v] 📘 I've reviewed the OpenZeppelin Contributor Guidelines
[ ] ✅ I've added tests where applicable to test my new functionality.
[v] 📖 I've made sure that my contracts are well-documented.
[v] 🎨 I've run the JS/Solidity linters and fixed any issues (npm run lint:all:fix).

Thanks for contributing @yhuag!
Have you seen RBAC? I believe this same functionality is already provided by that contract, correct?

@frangio Oh yes! That's nice to see it be implemented.

Closing then. Thank you!


What happened to the AppImage?

What happened to the AppImage in the recent releases?

As far as I can tell you my application does not work as AppImage because I do not use asar and therefore I cannot write files outside the application.
But I can build you an AppImage version if you want to test it?

Sure, once there is an AppImage I'll be happy to test it.

Here you go: https://github.com/Mokkapps/scrum-daily-standup-picker/releases/download/v1.2.0/scrum-daily-standup-picker-1.2.0-x86_64.AppImage


Migrate the VS project creation from godot to blazium

Primarily Renames the files generated, configuration and texts from godot to blazium.

The documentation needs to be updated too, but that will be a separate PR.

Even if you don't have VS installed you can create the solutions without building the whole project.
scons platform=windows vsproj=yes dev_build=yes
It will create the solution, project and other files.


 Let's merge it and improve it later.
BTW, It's better to prevent huge pull requests...

LGTM. Let's merge it and improve it later.
BTW, It's better to prevent huge pull requests...

Aha, got it.


Issue Downgrading from Crawlee 3.7.2 to 3.4.0 - Persistent Version and TypeScript Errors

Which package is this bug report for? If unsure which one to select, leave blank
@crawlee/http (HttpCrawler)
Issue description
Description
I recently updated Crawlee to version 3.7.2 in my project's package.json and encountered a TypeScript error during the build process. To resolve this, I attempted to downgrade back to version 3.4.0. However, despite modifying package.json and trying various methods to reset the environment, Crawlee remains at version 3.7.2, and the TypeScript error persists.
Steps to Reproduce

Upgrade Crawlee in package.json to 3.7.2 and run npm install.
Run build process; encounter TypeScript error (details below).
Attempt to downgrade to Crawlee 3.4.0 by changing package.json.
Remove package-lock.json and node_modules, then run npm install or npm ci. Crawlee stays at 3.7.2.
Execute additional steps like clearing npm cache (npm cache clean --force), trying intermediate versions (3.5.0, 3.6.0), but issue persists.

Expected Behavior
After changing the version in package.json and clearing the node environment (removing node_modules, package-lock.json, clearing cache), Crawlee should downgrade to the specified version (3.4.0 in this case).
Actual Behavior
Crawlee remains at version 3.7.2 after downgrading attempts. TypeScript error encountered during build:
node_modules/@crawlee/http/internals/http-crawler.d.ts:372:44 - error TS1005: 'assert' expected.
372         request?: import("got-scraping", { with: { "resolution-mode": "import" } }).RequestFunction | undefined;
...
...
...
node_modules/@crawlee/http/internals/http-crawler.d.ts:469:5 - error TS1128: Declaration or statement expected.

469     private _requestAsBrowser;
        ~~~~~~~

node_modules/@crawlee/http/internals/http-crawler.d.ts:470:1 - error TS1128: Declaration or statement expected.

470 }
    ~

Found 140 errors in the same file, starting at: node_modules/@crawlee/http/internals/http-crawler.d.ts:372

The only version that I was able to downgrade are:

3.7.0 - same error like in 3.7.2 case
3.6.1 - other error still unable to build

This is command im using to build:
"build": "tsc --project tsconfig.build.json && tsc-alias --project tsconfig.build.json && echo 'Build complete'"
my package.json version:
"crawlee": "3.4.0", event thought in<EMAIL_ADDRESS>version points to 3.7.2 (in case of 3.7.0, 3.6.1 it points to correct version but there is still error when building the project)
Code sample
There is no code sample for this problem

Package version
3.4.0
Node.js version

18

Operating system
all of them
Apify platform

[ ] Tick me if you encountered this issue on the Apify platform

I have tested this on the next release
3.4.0, 3.4.1, 3.5.0 etc.
Other context
No response

The builds errors should be fixed in master, cc @vladfrangu
We cant do much about old versions, as they are already published, but the issue you are referring to (with downgrades) is already fixed in the latest versions, as we pin the internal dependencies (see #2040) - it was fixed in v3.5.3 if I am not mistaken.

What version of TS are you using? IIRC you need 5.3 now.

I'm using 5.2.2 But I don't understand why npm/yarn works this way, that it install newer packages that I want

Because you use crawlee metapackage, you get the right version of that, but it allows ^ of all the @crawlee/ packages. And that is what is already fixed since v3.5.3.

Try using latest version with TS 5.3, alternatively you can use skipLibCheck to suppress those TS errors.

Oh I get it, so it does not matter that in my project I'm using crawlee: 3.4.0 (without carrot (^)) it uses the carrot sign underneath anyway and bump all versions - I did not expect that

lol, its caret symbol, not carrot :P
yeah exactly, fyi you can get around that with older versions via npm overrides (or yarn resolutions), or by installing the @crawlee/ packages directly instead of using the crawlee metapackage.

Yeah, I meant caret of course haha :D Thanks for quick help! :)
Bumping typescript to 5.3.3 fixed my issue locally and on Jenkins pipeline.


How to run code in incubator-brpc/example/rdma_performance?

Branch: rdma
Question: I am a newbie to BRPC, I have not used BRPC before, I am more interested in his RDMA communication part, so I tried it. After I compiled the code, I couldn’t run the sample code in directory  incubator-brpc/example/rdma_performance , and I couldn’t find a hint on how to modify client.cpp or server.cpp to make them run successfully. The running error message is like this: incubator-brpc/src/brpc /rdma/rdma_helper.cpp:512] Now local connection only uses TCP. Try to set rdma_disable_local_connection to false to allow RDMA local connection

If you use local network connection you should set rdma_disable_local_connection=false.

Hello, the original rdma branch is no longer maintained. We plan to merge rdma feature into master：https://github.com/apache/incubator-brpc/pull/1836
You can try the new rdma implementation. If still have problem, you can create a new issue.
Close this first.


Java: implement LPOS

https://redis.io/docs/latest/commands/lpos/

Looks like this PR has some sort of change to the redis-rs submodule, can you revert that specific change?


Driver and aggregator/executor support for containerization

Why are these changes needed?

provide driver supports for containers
add aggregator/executor wrappers for container init
provide example container config files for job submission
update docker README

What have been changed?

add aggregator/executor wrappers under $FEDSCALE_HOME/examples/containerization
changes in fedscale/core/fllibs.py, fedscale/core/logger/aggragation.py and fedscale/core/logger/execution.py are intended to deal with changes to global args during container init, and I modified how to import args for examples under $FEDSCALE_HOME/examples to propagate this change
add container invoke and management  in driver.py

Related issue number

Checks

[x] I've included any doc changes needed for https://fedscale.readthedocs.io/en/latest/
[x] I've made sure the following tests are passing.
Testing Configurations

Regression (submit jobs in normal mode)

[x] Dry Run (20 training rounds & 1 evaluation round)
[x] Cifar 10 (20 training rounds & 1 evaluation round)
[x] Femnist (20 training rounds & 1 evaluation round)

Containerized

[x] Dry Run (20 training rounds & 1 evaluation round)
[x] Cifar 10 (20 training rounds & 1 evaluation round)
[x] Femnist (20 training rounds & 1 evaluation round)

Thanks a lot! Did you test those three experiments (e.g., FEMNIST) to see whether it works/runs as expected?

Thanks a lot! Did you test those three experiments (e.g., FEMNIST) to see whether it works/runs as expected?

Yes, I ran the three experiments on container and non-container mode for 20 rounds and 1 evaluation, seems good to me based on the logs generated.


Add webhook + IFTTT example

Description:
WIP. Still exploring best fit
This adds a new webhook component as proposed in https://github.com/home-assistant/architecture/issues/80
To test it, I have updated the IFTTT component with a config flow to register a webhook.

If we like architecture, I can add tests.
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

[] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

We should also abort if the host is a local IP address

Tests added, this is ready for review.

CC @dgomes I think that the webhook component is what the push camera could use too.


Fixed Event Reversed Chronological Order

Fixed #13

Codecov Report

Merging #21 (e0cbe20) into main (f12b9a7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #21   +/-   ##
=======================================
  Coverage   33.33%   33.33%
=======================================
  Files           3        3
  Lines          15       15
=======================================
  Hits            5        5
  Misses          9        9
  Partials        1        1

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 f12b9a7...e0cbe20. Read the comment docs.


[Windows:chocolatey] No packages search, Install Shell Commmands option

So I installed atom via chocolatey, it installed to
[blah]\AppData\Local\atom\

which I assume is correct.
Now if I go to settings packages (clicking my way, or ctrl-), there's no option to search for uninstalled packages, only installed packages, As seen here:
Going to the file tab, there's no Install Shell Commmands menu as seen here:
apm is installed, and is found at
[blah]\AppData\Local\atom\app-0.180.0\resources\app\apm\bin

Atom is run as admin.
Info:

Windows 7 x64
Atom 0.180.0
Launched with: C:\Users\Mikkel\AppData\Local\atom\app-0.180.0\atom.exe (launching atom.cmd in local\atom\bin doesn't make a difference)

Now if I go to settings packages (clicking my way, or ctrl-), there's no option to search for uninstalled packages, only installed packages

The Install section on the left side will give you the ability to search for and install packages and themes.

Going to the file tab, there's no Install Shell Commmands menu as seen here:

atom and apm shell commands are automatically installed by the installer so this option isn't needed on Windows.

Ah thankyou, I also found this so yea

Chocolatey has always been a problem with Atom.  You should use the
official installer instead.
On Wed, Feb 18, 2015 at 9:57 AM, dumle29<EMAIL_ADDRESS>wrote:

So I installed atom via chocolatey, it installed to
[blah]\AppData\Local\atom\
which I assume is correct.
Now if I go to settings packages (clicking my way, or ctrl-), there's no
option to search for uninstalled packages, only installed packages, As seen
here: [image: Image of package tab]
https://camo.githubusercontent.com/1543185442ed453705df734d34b93147bd37bf40/687474703a2f2f692e696d6775722e636f6d2f6e456a4356757a2e706e67
Going to the file tab, there's no Install Shell Commmands menu as seen
here: [image: No Install Shell Commmands option]
https://camo.githubusercontent.com/b0716025ec7d10eea3c07fd72b5f4e981d3ac8c6/687474703a2f2f692e696d6775722e636f6d2f314f676a5479512e706e67
apm is installed, and is found at
[blah]\AppData\Local\atom\app-0.180.0\resources\app\apm\bin
Atom is run as admin.
Info:

Windows 7 x64
Atom 0.180.0
Launched with:
C:\Users\Mikkel\AppData\Local\atom\app-0.180.0\atom.exe (launching atom.cmd
in local\atom\bin doesn't make a difference)

—
Reply to this email directly or view it on GitHub
https://github.com/atom/atom/issues/5616.


module 'health' not found

  Error  19:45:01 msg_show.lua_error E5108: Error executing lua ...lazy/clipboard-image.nvim/lua/clipboard-image/health.lua:3: module 'health' not found:
no field package.preload['health']
cache_loader: module health not found
cache_loader_lib: module health not found
no file './health.lua'
no file '/opt/homebrew/share/luajit-2.1.0-beta3/health.lua'
no file '/usr/local/share/lua/5.1/health.lua'
no file '/usr/local/share/lua/5.1/health/init.lua'
no file '/opt/homebrew/share/lua/5.1/health.lua'
no file '/opt/homebrew/share/lua/5.1/health/init.lua'
no file './health.so'
no file '/usr/local/lib/lua/5.1/health.so'
no file '/opt/homebrew/lib/lua/5.1/health.so'
no file '/usr/local/lib/lua/5.1/loadall.so'
stack traceback:
[C]: in function 'require'
...lazy/clipboard-image.nvim/lua/clipboard-image/health.lua:3: in main chunk
[C]: in function 'require'
.../lazy/clipboard-image.nvim/lua/clipboard-image/paste.lua:4: in main chunk
[C]: in function 'require'
[string ":lua"]:1: in main chunk
,
I was still able to use it for a period of time, but now there is such an error, which cannot be solved by reinstallation.

#48

+1

+1

modify the health.lua
local health = vim.health or require "health"

instead of
local health = require "health"

Got same issue using neovim v0.9 and v0.10 and on only on macOS 13.3.1 / M1 chip. Exactly same neovim configs work well on macOS 12.# / Intel chip (not sure it's related, but just FYI).

Here's a temporary fix until the plugin is updated


What's the difference between adding ConcurrencyCheck and not adding?

I refer to this article: https://docs.microsoft.com/zh-cn/ef/core/saving/concurrency
Not added ConcurrencyCheck
You can also access DbUpdateConcurrencyException
So I don't know what it means

@l2999019 DbUpdateConcurrencyException is thrown when EF expected to update a row in the database, but that row was found. This means that the row must have been deleted or updated between the time that EF read the values for that row and the time it attempts to do the update. Concurrency tokens are a way to ensure that any update will be detected by EF, but it will still happen for deletes regardless, and may also happen for other updates in some situations.


action: add Android build test

Android is the major release target of nntrainer.
Build and run test cases for Android.

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2633. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


Add support for auth expiration

Add support for auth expiration. When the auth token expires, we'll log the user out, forcing them to sign in.
persisting this to the local storage so when the token is loaded, we know if it should be refreshed or not.
Also change the existing useCurrentUserInfo to be a callback instead.

in hindsight, the 1 day expiration thing doesn't actually work, so i'll probably remove that "within 1 day" check


Documentations

Coverage decreased (-0.01%) to 88.354% when pulling 6435f0c71509e6592fbe46fc215f98d1f4c9638d on CT15:v1.5 into 0c62a54feecf3038bd255a4c4d632bc84ba247a0 on CS2103AUG2017-T15-B2:master.


Move rescue_from logic into separate method to handle custom exceptions

In Catalog, we have an error that is being raised by a gem automatically, Pundit::NotAuthorizedError, which also auto-generates a not-so-user-friendly error message of "not authorized to <action?> this Class".
We needed a way to hook into the rescue_from process without duplicating the code in common, so I broke out the handler into a separate method and I'm using it to pass control over the error message to the rescue_from we have for specifically handling Pundit::NotAuthorizedErrors.
Example usage: Catalog PR.

@syncrou @lindgrenj6 It looks like travis is failing because it's using an older ruby version that doesn't have the #delete_suffix method for strings. Should we be updating ruby version dependency for this repo or should I use the older .chomp('?')? Seems like 2.4.5 is a bit old, but I don't know if 2.5.x breaks anything else we use in this repo.

@eclarizio @lindgrenj6 My understanding is all the apps are running on 2.5.5 and travis for a number of repos is testing against 2.5 and 2.6.
Seems we should bump the travis rvm version here.

@gmcculloug @eclarizio https://github.com/RedHatInsights/insights-api-common-rails/pull/187
Opened a PR for here and catalog/approval to keep us all in line.

@eclarizio ruby version PR is merged.  Please rebase.

@gmcculloug @lindgrenj6 Rebased/updated. Looks like we're green now! 👍

@syncrou Pushed changes, is this more what you're looking for?


Python with ctypes doesn't find the runtime in the package folder.

And complains about it. See https://github.com/gpufit/GPUfit/issues/75#issuecomment-279726662. We either need:

teach Python to find them there
put them somewhere else where Python can find them
install the runtime separately, if needed
not use ctypes but something like swig (works in specpy which uses swig)

We will statically link to the VC runtime.
See also: https://cmake.org/Wiki/CMake_FAQ#How_can_I_build_my_MSVC_application_with_a_static_runtime.3F

linking runtime statically now

FYI The problem is known but not solved in the case of dynamic dependencies. See for example:
http://stackoverflow.com/questions/10755882/ctypes-wont-load-library
http://stackoverflow.com/questions/7586504/python-accessing-dll-using-ctypes


New flag: --inline-callback obviates load.bzl

When turned on, a user doesn't have to provide a callback when calling maven_dependencies() from their WORKSPACE file.
Instead, maven_dependencies() will call a function defined in the same .bzl file that calls native.maven_jar() or equivalent.
The goal is to simplify setting up a project to work with bazel-deps.

@johnynek
There are existing tests for this, and I don't know Scala well enough (at all, actually) and my IntelliJ doesn't fully work with Scala. In short, if you add a skeleton test for Writer.scala, I'm happy to add tests for this new functionality.
The goal of this PR (and others I'll send) is to simplify setting up a project to work with bazel-deps. Ideally, a user could run bazel-deps on a pom.xml file and be done.

Alternatively, does it make more sense to put this option in the YAML file?

This is great, and I'll merge, but I do agree with you that it should be an option in the yaml file. I think adding a global option there is the right way to go.
As for testing, we don't have a great story currently to test generation. We should ideally generate a workspace and then try to build it, but we haven't set that up yet and I don't want to block you.
So, if you see this PR: https://github.com/johnynek/bazel-deps/pull/66 it has an example.

actually, making it a yaml option is the right call because we already have a maven server in the yaml file which we use to walk the dependencies. If we have this option there, you can generate with the same server, which makes a lot of sense.

Can you elaborate? I thought you already said it should be a yaml option?
How does that relate to the server?
On Sep 27, 2017 19:49, "P. Oscar Boykin"<EMAIL_ADDRESS>wrote:

actually, making it a yaml option is the right call because we already
have a maven server in the yaml file which we use to walk the dependencies.
If we have this option there, you can generate with the same server, which
makes a lot of sense.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/johnynek/bazel-deps/pull/65#issuecomment-332602373,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB5_Yc_G9U6DD3AkzntxNC8Muu1HcG-Sks5smoqdgaJpZM4PiAz0
.

our code looks like:
def maven_load(hash):
    native.maven_jar(
        name = hash["name"],
        artifact = hash["artifact"],
        sha1 = hash["sha1"],
        repository = _nexus,
    )
    native.bind(
        name = hash["bind"],
        actual = hash["actual"],
    )

but actually we just want the one in hash["repository"] I guess...
Sorry, I am a bit foggy on some of this since it has been so long since some of this was written and I have not touched it.
So, take away: carry on. I think my comment was just confusing you. Putting the option in the yaml is still the plan I think we have agreed on.

actually, I don't know why we wouldn't make this the only option. If users want to ignore this function, that's fine. That will be the simplest to operate.
Here is the default, and if you want to customize, knock yourself out.

Do you mean generate it inline always?
I'm in favor ☺
On Sep 29, 2017 20:01, "Oscar Boykin"<EMAIL_ADDRESS>wrote:

actually, I don't know why we wouldn't make this the only option. If users
want to ignore this function, that's fine. That will be the simplest to
operate.
Here is the default, and if you want to customize, knock yourself out.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/johnynek/bazel-deps/pull/65#issuecomment-333196514,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB5_YWx2vUMOJOy5uMF573Dl2RENuo7fks5snTBxgaJpZM4PiAz0
.

yeah. Always generate inline. If you want to wire a custom callback, you can, but you really don't have to.

Updated, PTAL thanks!

you need to regenerate the 3rdparty for this repo. One of the tests is that running that tool produces no diff (to keep us honest about using the latest version of the code).

Ah, looks much better :)
Updated code, reran bazel-deps on itself, and updated docs.


Drop /www folder support

All API handlers should be separated from HTTP, so we have /api folder in application for it and no need old style handlers like get.js, put.js etc.

[x] Step 1: Drop examples
[x] Step 2: Drop support

Done in v2.x


[8.x] Fix deprecation warning in SimpleMessage::formatLine

This PR fixes a deprecation warning that occurs if a null line is appended to a mail using either of these syntaxes:
(new \Illuminate\Notifications\Messages\MailMessage)->line(null);
(new \Illuminate\Notifications\Messages\MailMessage)->with(null);

The SendingMailNotificationsTest does not have a test case passing null lines yet, I would have added one but I am not sure what it should really test for. It might be useful to test if a null parameter is handled gracefully and results into an empty string item in the introLines attribute.
Let me know if that would be a useful addition, otherwise I'll leave it at that.

Looks like the failing test had problems resolving http://www.google.com ;-)


[DO NOT MERGE] Update payment trigger

Refactored the Payment TDTM Trigger.
Only the TDTM class was modified - the major change was to pull the multi-currency logic out of the run for payments function and split it into smaller functions.
This is a precursor to adding new validation on the payments object.
Critical Changes
Changes
Issues Closed
New Metadata
Deleted Metadata

@seanpat09 , @force2b - If one of you can review the refactoring of the TDTM class that would be good.  I have a separate branch that will actually make the change to add the trigger rule using the place holder I've put in here.  Also the EDU team will be using this branch to add some additional validation as well but they won't get started until after the High Ed Summit.  thanks

**lurch: attach W-032340

Tracking W-032340

@patrick-yan-sf - Will this PR make any substantive changes to the behavior of the Payment class, or is this mainly refactoring? Asking to see if we need to make any changes to this doc:
https://powerofus.force.com/s/article/NPSP-Apex-Class-Descriptions

@sam-knox
I think that description does need to be updated but maybe not specifically with this PR.  Maybe with the other one - the one that actually creates a validation rule for Paid and Written Off.
Because the description says it only applies to Multi-Currency Orgs which is no longer true once we put that validation in place.
In terms of this PR (the parent one) - there should be no behavioral change.


Sysbox installer: Add debconf variable to disable changing daemon.json completely

The variable docker_userns_remap_autoconfig controls only one option that is changed in the docker daemon.json.
For automated deployments like puppet does it would be great to have a debconf variable that allows the user to disable any changes to the daemon.json within the debian package scripts because this file is often handled by puppet completely and comes preconfigured with suitable network and other options.

Makes sense, thanks for filing the issue. We are also working on supporting Sysbox on K8s clusters, and I think this will also help installing Sysbox in K8s nodes.

Yes @nudgegoonies, we can definitely add a new debconf variable so that users can explicitly set the desired behavior (manual vs automatic) through the debconf-set-selections interface.
As Cesar mentioned above, our installer will need to growth to cover all the new higher-level runtimes that we are about to support (Cri-o/K8s, Podman, etc), so this option will probably be needed to cover those cases anyways.

This is how we patched the debian package:
diff -Naur CONTROL/config CONTROL/config
--- CONTROL/config	2022-03-23 06:16:52.000000000 +0100
+++ CONTROL/config	2022-03-30 14:35:40.799610746 +0200
@@ -100,6 +100,9 @@
 # Main
 #

+db_get sysbox/configure_docker_daemon
+if [ "$RET" = "true" ]; then
+
 # If a docker-restart is required in this setup, and there are existing docker
 # containers, alert user of the need to stop containers and exit installation
 # process.
@@ -122,4 +125,6 @@
         " installation documentation for details.\n"
 fi

+fi
+

diff -Naur CONTROL/postinst CONTROL/postinst
--- CONTROL/postinst	2022-03-23 06:16:52.000000000 +0100
+++ CONTROL/postinst	2022-03-30 15:15:57.320054067 +0200
@@ -348,11 +348,16 @@
     # in the docker configuration file.
     add_sysbox_user

+    db_get sysbox/configure_docker_daemon
+    if [ "$RET" = "true" ]; then
+
     # Adjust dockerd configuration (if necessary)
     if docker_installed; then
         adjust_docker_config
     fi

+    fi
+
     # Check for kernel-headers.
     check_kernel_headers
 }
diff -Naur CONTROL/postrm CONTROL/postrm
--- CONTROL/postrm	2022-03-23 06:16:52.000000000 +0100
+++ CONTROL/postrm	2022-03-30 14:43:01.755770222 +0200
@@ -90,7 +90,13 @@
 purge)
     # Adjust docker config to eliminate entries added by Sysbox's
     # installation process.
+
+    db_get sysbox/configure_docker_daemon
+    if [ "$RET" = "true" ]; then
+
     adjust_docker_config
+
+    fi
     ;;

 remove | upgrade | failed-upgrade | abort-install | abort-upgrade | disappear) ;;
diff -Naur CONTROL/templates CONTROL/templates
--- CONTROL/templates	1970-01-01 01:00:00.000000000 +0100
+++ CONTROL/templates	2022-03-30 14:31:29.979528268 +0200
@@ -0,0 +1,5 @@
+Template: sysbox/configure_docker_daemon
+Type: boolean
+Default: "true"
+Description: Configure Docker daemon?
+ Configure docker daemon with suitable sysbox config.


Restore broken link in windows-analytics-FAQ-troubleshooting.md

Description:
The Windows Analytics blog page
"You can now check on the status of your computers within hours of running the deployment script" is reported as a Broken Link in issue ticket #3211, because the page has been moved from blogs.technet.microsoft.com to techcommunity.microsoft.com and can be found at

https://techcommunity.microsoft.com/t5/Windows-Analytics-Blog/You-can-now-check-on-the-status-of-your-computers-within-hours/ba-p/187213

Proposed change:
Restore the blog page link by replacing the old link with the new one (shown above).
Closes #3211

@jaimeo - can you please assist to check, approve and merge the content for this PR based on issue #3211?
Thank you.

I don't have merge rights, but I approve this change.

@AndreaBarr  - can you please assist to merge the content for this PR as approved by jaimeo? Please refer to the comment above.
Thank you.

I noticed just moments ago that the PR #3218 was opened less than 2 hours before I opened this PR. Please compare and close my PR here if the other PR contains a more useful version to be merged into the docs.

@illfated Please resolve these conflicts so we can get this reviewed.

Closing this PR because it has already been resolved with PR #3218 being merged into the docs. The link change is the same, but PR #3218 contained a slightly better wording in addition to the link correction.

Sorry, wrong button.


Multiple intervals with missing data when testing zero-downtime-scaling

Describe the bug
In the scaling up gradually section in the zero-downtime-scaling test, the results seem to have multiple large intervals with missing data.
For example:

To Reproduce
Steps to reproduce the behavior:

Follow the gradual scale up instructions in the zero-downtime-scaling test.

Expected behavior
There should not be intervals of missing data.
Your environment

Version of the NGINX Gateway Fabric -
"version":"edge",
"commit":"7a6e198c8b6e1e81058cd9ad0717cffa2cc77bb8",
"date":"2023-12-07T22:27:22Z"

Version of Kubernetes - 1.27

Kubernetes platform (e.g. Mini-kube or GCP) - GKE

Details on how you expose the NGINX Gateway Fabric Pod (e.g. Service of type LoadBalancer or port-forward) - LoadBalancer

Haven't seen this issue for quite some time. Can reopen if it's seen again.


Illegal instruction when using libssl

Bug Report
We've been experiencing a problem with amqproxy which was originally submitted as https://github.com/cloudamqp/amqproxy/issues/174
However I suspect the problem could lie in crystal itself.
Upon negotiating the connection to the upstream server, amqproxy stops with illegal instruction.

We only see this on some machines, AMD EPYC with avx512 capability. Perhaps a red herring.
We only see this when using SSL.
We see this with crystal 1.13.3, 1.13.2 and 1.12.2
We're using alpine based containers, 84codes/crystal:1.13.2-alpine

The backtrace is:
#0  0x00ffffffffffe001 in ?? ()
#1  0x0000800000000000 in ?? ()
#2  0x00007ffff4d26990 in ?? ()
#3  0x00007ffff7866c60 in ?? ()
#4  0x00005555556ed1f2 in new (__arg0=0x7f0000008000 <error: Cannot access memory at address 0x7f0000008000>, __arg1=32767) at /usr/share/crystal/src/slice.cr:70
#5  0x00005555557e9939 in fill_buffer (self=0x7ffff7866c60) at /usr/share/crystal/src/io/buffered.cr:272
#6  0x00005555557e96a3 in read (self=0x7ffff7866c60, slice=...) at /usr/share/crystal/src/io/buffered.cr:93
#7  0x00005555557e6230 in read_fully? (self=0x7ffff7866c60, slice=...) at /usr/share/crystal/src/io.cr:542
#8  0x00005555557e60fb in read_fully (self=0x7ffff7866c60, slice=...) at /usr/share/crystal/src/io.cr:525
#9  0x0000555555878674 in from_io (io=0x7ffff7866c60) at /tmp/lib/amq-protocol/src/amq/protocol/frames.cr:65
#10 0x000055555583b600 in read_loop (self=0x7ffff7864780, socket=0x7ffff7866c60) at /tmp/src/amqproxy/upstream.cr:67
#11 0x000055555583b4f4 in read_loop (self=0x7ffff7864780) at /tmp/src/amqproxy/upstream.cr:63
#12 0x00005555556a90b9 in -> () at /tmp/src/amqproxy/channel_pool.cr:44
#13 0x00005555557295a5 in run (self=0x7ffff785bb40) at /usr/share/crystal/src/fiber.cr:143
#14 0x00005555556a282d in -> (f=0x7ffff785bb40) at /usr/share/crystal/src/fiber.cr:95
#15 0x0000000000000000 in ?? ()

Afaik crystal uses LLVM, whose trap intrinsic will generate a ud2 instruction. That could mean that SIGILL is a red herring for an invalid memory access. Or am I barking up the wrong tree completely?
The default approach is to use libssl as a shared object, but interestingly we have been unable to reproduce if shards build --static is used instead.

If linking a different build of libssl doesn't reproduce the issue, that indicates a problem with the library. For the Crystal bindings it's irrelevant whether libssl is linked statically or dynamically. Codegen is identical in both cases.
To further investigate this hypothesis, I'd suggest to try different builds of OpenSSL. Maybe build it locally to ensure compatibility with the host CPU.

@straight-shoota thanks for the response.
I agree the libs have the potential to be different when static linking is used:

static linking would use the libssl from alpine:latest from when the 84codes/crystal image was built.
dynamic linking would use libssl from alpine:latest from when amqproxy was built.

I'll investigate precisely which versions are being used in each scenario.

If you build and run in the 84codes/crystal image, both static and dynamic versions of the library are the ones available in that image. But static and dynamic versions have some differences and these could be relevant.

Pretty sure they aren't the same, as the dockerfile has multiple sources: https://github.com/cloudamqp/amqproxy/blob/main/Dockerfile#L8
We don't need to discuss how amqproxy structures their build in this project though, out of scope.

This actually provides a quite relevant bit of information.
The app is built on 84codes/crystal:1.13.2-alpine linking against the libssl.so.3 included in that image.
But then the binary is copied to a different image based on alpine:latest. So when you execute it, it uses libssl.so.3 from that image. And that could be a different library version than the one from the build image.
Apart from binary compatibility issues, if the libssl version at load time is different from compile time, the Crystal bindings' expectations about library features is incorrect.
Generally, the loader should error if it detects that the runtime library is incompatible with the linked one. But this might not be 100% accurate. It's generally best to use the same library version in the builder and the runner.
Static linking trivially ensures this, of course.

Agreed. I'll see whether using dynamic linking with libraries copied from the builder image is also stable at runtime.


fix sample bug to run

fix sample bug to run

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


url-path-map videoPathRule never gets added to a rule

I have read, but not attempted the learning activity outlined in the article.
I'm surprised that the url-path-map named videoPathRule is created, but never added to the rule. I'd be surprised if this actually works.
The request:
If this really does work, can you explain how the url-path-map videoPathRule gets associated with the rule rule2?
If this doesn't work can you update the rule definition so it does work? I expected to see 2 instances of  --url-path-map listing the 2 previously defined maps when rule2 was defined.

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

ID: 6adf6adc-e0ff-152e-f714-07120ddf0413
Version Independent ID: 51068118-89b0-e573-fe7c-55c39baab955
Content: Route web traffic based on the URL - Azure CLI
Content Source: articles/application-gateway/tutorial-url-route-cli.md
Service: application-gateway
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@doug-fish
It looks like your issue is resolved since you closed this thread.
If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion. Thanks!!!


Add resource loading support for audio files.

This PR is blocked until I can add interop support for resource loading APIs.


How to migrate to the new location?

Good morning! Since the location change and https://github.com/caskroom/homebrew-cask/pull/21901, I'm now seeing the warning about migrating Casks, as expected. I've searched around a bit but don't understand what I need to do as an end-user. Is there a command for migrating? Is it simply a move of the /opt/homebrew-cask/Caskroom/ directory to /usr/local/Caskroom/? A bit more info would be helpful as I don't want to muck anything up. Thanks!

Yes, the simplest thing would be to mv /opt/homebrew-cask/Caskroom /usr/local. However, any casks installed prior to #13966 were symlinked rather than moved to their destinations, so you'll need to re-install those manually.
If we do wind up phasing out --caskroom, it won't be for a while, so you have plenty of time to find old casks and brew cask install --force them.

Closing the issue as it has been answered, but making it the default to answer the questions that arise.
If new questions come up, we should edit @jawshooah’s comment to make it clearer.

I just re-installed one of my Caskroom apps and noticed that it was still installed to /opt/homebrew-cask/Caskroom? Doesn't seem like homebrew-cask is properly defaulting? I don't have any HOMEBREW_CASK_OPTS options set.

@tehmaspc same for me; it was the eclipse-java app.  Re-installing other apps hasn't had the same problem thus far.

Ah. I tested my sonos app. So there are individual casks that must be expecting /opt/homebrew-cask/Caskroom then.

#21901 changed the behavior to default to the old location if it still exists and print a warning about the new default, to mitigate the issues brought about by the original change. No need to change HOMEBREW_CASK_OPTS if you delete /opt/homebrew-cask/Caskroom.

After receiving the warning, I've moved my Caskroom to /usr/local/Caskroom. Now brew cask list lists the installed casks but brew cask uninstall can't remove them.
➜  Casks git:(master) brew cask list
hazel					textmate
iterm2					the-unarchiver
java					transmission
jettison (!)				vagrant
macports				virtual box
mplayerx				virtualbox-extension-pack
source tree				vlc
stremio
➜  Casks git:(master) brew cask uninstall hazel
Error: hazel is not installed
➜  Casks git:(master) brew cask uninstall mplayerx
Error: mplayerx is not installed

@grootknol you have to brew cask install APP_NAME --force

@grootknol This is a result of a known issue with brew cask list (#14058). You have to brew cask uninstall --force mplayerx

Good to know, I just hit this. Currently have no Casks installed, but will open a PR when BBEdit 11.6 gets released. Assuming that @jawshooah is correct in how this should be handled, is it worth adding a line to the output of brew cask list just to tell people 'here's what you need to do?'

Could I just do for cask in $(brew cask list) do; brew cask install $cask --force; after moving Caskroom to the new directory?

for me I did find /usr/local/Caskroom -name "*.app" -maxdepth 3 -exec mv {} ~/Applications \; after moving cask room and removing all the alias.

@grootknol: Even simpler... brew cask install --force $(brew cask list)

something like this should work mv /opt/homebrew-cask/Caskroom /usr/local && find ~/Applications -type l -exec ls -l {} \; | grep "Caskroom" | sed -e 's/opt\/homebrew-cask/usr\/local/g' | tr -s '[:space:]' | cut -d' ' -f9- | awk 'BEGIN { FS = " -> " } ; { system("ln -sf \"" $2 "\" \"" $1 "\"") } '

I had a ton of Casks that were still symlinked from ~/Applications to /opt/homebrew-cask/Caskroom, so I first moved my Caskroom using the command above mv /opt/homebrew-cask/Caskroom /usr/local then changed all the symlinks to match the new location:
for f in ~/Applications/*.app; do oldloc="$(readlink "$f")"; [[ -e $oldloc ]] || ln -sf "/usr/local${oldloc#/opt/homebrew-cask}" "$f"; done

The solutions presented in this thread should work equally well for other symlinked artifacts, so I'm locking it. Feel free to open a new issue if you have trouble.


Merge main branch into out-of-order

Another attempt for https://github.com/grafana/mimir-prometheus/pull/215

Re-synced with out-of-order branch since compaction PR was just merged


Need clarification for vclmul

Since vclmul has eew = 64, therefore the result would at most be 127 bits.
Does vd stores the whole 127 bits or it only holds the bottom 64 bits?

It returns the least significant EEW=64 bits of the product. I am in the process of adding this and other details to the definition.