feat: add signet and testnet4 addresses

The generator wasn't working so I added these by hand. The generator code also needs to be updated to correctly handle these new networks.

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

Project coverage is 86.37%. Comparing base (5368eb8) to head (57a4e7c).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #435      +/-   ##
==========================================
+ Coverage   84.06%   86.37%   +2.31%
==========================================
  Files           8        8
  Lines         389      499     +110
  Branches      124      124
==========================================
+ Hits          327      431     +104
- Misses         62       68       +6

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@gartnera ZRC-20 addresses are fetched automatically from https://zetachain-athens.blockpi.network/lcd/v1/public/zeta-chain/fungible/foreign_coins.
Can you, please, move:
  {
    "address": "tb1qy9pqmk2pd9sv63g27jt8r657wy0d9ueeh0nqur",
    "category": "omnichain",
    "chain_id": 18333,
    "chain_name": "btc_signet",
    "type": "tss"
  },
  {
    "address": "tb1qy9pqmk2pd9sv63g27jt8r657wy0d9ueeh0nqur",
    "category": "omnichain",
    "chain_id": 18334,
    "chain_name": "btc_testnet4",
    "type": "tss"
  },

To addresses.testnet.json so they're not overwritten when the script runs again.

@gartnera ZRC-20 addresses are fetched automatically from https://zetachain-athens.blockpi.network/lcd/v1/public/zeta-chain/fungible/foreign_coins.

So I/we can remove all the zrc20 entries?

Can you, please, move:
  {
    "address": "tb1qy9pqmk2pd9sv63g27jt8r657wy0d9ueeh0nqur",
    "category": "omnichain",
    "chain_id": 18333,
    "chain_name": "btc_signet",
    "type": "tss"
  },
  {
    "address": "tb1qy9pqmk2pd9sv63g27jt8r657wy0d9ueeh0nqur",
    "category": "omnichain",
    "chain_id": 18334,
    "chain_name": "btc_testnet4",
    "type": "tss"
  },

To addresses.testnet.json so they're not overwritten when the script runs again.

I already have entries in the addresses.testnet.json file?

@gartnera not exactly.
This script: https://github.com/zeta-chain/protocol-contracts/blob/main/v2/tasks/addresses.ts
And this file (containing addresses we can't fetch automatically): https://github.com/zeta-chain/protocol-contracts/blob/main/v2/tasks/addresses.testnet.json (note it's in the tasks dir) ❗ I propose we add Bitcoin TSS addresses to this file and not the auto-generated file.

Let's do it right this time! 😃
https://www.notion.so/zetachain/Contract-Address-Registry-1517b3fb9049800e9eb4e292ca9867f0?pvs=4

Ok @fadeev I think I know what you're saying. I've updated the PR.
If this still isn't correct maybe you could fix it up?

Removed testnet 4 and signet addresses from the manual JSON file, because they're fetched automatically. Updated the script to take chain names from name rather than chain_name from https://zetachain-athens.blockpi.network/lcd/v1/public/zeta-chain/observer/supportedChains For some reason chain_name is "ampty" for BTC testnets.

CI is failing because RPC endpoints are blocking requests.

CI is failing because RPC endpoints are blocking requests.

This is for v1 it doesn't matter anymore isn't it?

CI is failing because RPC endpoints are blocking requests.

This is for v1 it doesn't matter anymore isn't it? @fadeev

🤦♂ I didn't notice it's only failing for v1. Yes, we can merge, and then disable generated CI for v1.

In fact, shall I disable both generated and test for v1, so we have clean CI before merging?

In fact, shall I disable both generated and test for v1, so we have clean CI before merging?

Yes


Set storeDataManager based on the durable config

I will merge this for the test on indy-perf. thanks all.


feat(skia-draw): add basic text support

Refactor Canvas control mechanism

Create useCanvasControl hook
Create CanvasControlProvider
All canvas state is now collected in the context, accessible and modifiable through the hook

Add support for adding text to the canvas
Add text controls to EDSControlPanel
Update documentation
Upgrade Chronicles to expo 49.0.21
Upgrade peer dependency version on @shopify/react-native-skia to 0.1.225 (needed for text support)


Hash.pick is intrinsically slow on big hashes, maybe it should be reflected in the docs

See https://github.com/rakudo/rakudo/issues/2586.

This is a simple matter of algorithm. Each time you call .roll / .pick without any parameters, it needs to walk half of the keys in the hash (on average).

So we should recommend %h.pick($how-many-elements-you-want).

Maybe it should be reflected in the docs - maybe not. I'm more leaning towards "not".
It's half an implementation detail of runtimes, not even a universal thing for the language, and anyway, I think knowing these things is a whole engineering discipline we can't (nor should) take up on.


fix(Makefile): Prevent makefile from using containers when $USE_CONTAINERS=false

If one sets $USE_CONTAINERS to false, perhaps if one has attempted to use containers beforehand and failed miserably, the makefile will still use containers.
Kaustavs-MacBook-Pro:machine kaustavhaldar$ echo $USE_CONTAINER
true
Kaustavs-MacBook-Pro:machine kaustavhaldar$ export USE_CONTAINER=false
Kaustavs-MacBook-Pro:machine kaustavhaldar$ echo $USE_CONTAINER
false
Kaustavs-MacBook-Pro:machine kaustavhaldar$ make build
Get http:///var/run/docker.sock/v1.20/containers/json?all=1: dial unix /var/run/docker.sock: no such file or directory.
* Are you trying to connect to a TLS-enabled daemon without TLS?
* Is your docker daemon up and running?
Post http:///var/run/docker.sock/v1.20/build?cgroupparent=&cpuperiod=0&cpuquota=0&cpusetcpus=&cpusetmems=&cpushares=0&dockerfile=Dockerfile&memory=0&memswap=0&rm=1&t=docker-machine-build&ulimits=null: dial unix /var/run/docker.sock: no such file or directory.
* Are you trying to connect to a TLS-enabled daemon without TLS?
* Is your docker daemon up and running?
make: *** [build] Error 1

This fixes that.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "kaustavha/fix/Makefile" git@github.com:kaustavha/machine.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

LGTM

LGTM


Rewrite to TypeScript

I will try to do that in the next few days :unicorn:

Closing, since Next.js 8.1.1 is not stable yet.


use 0755 to create new dir

Use Case:
Application is running in production with user A, but when debugging I use user B to login on that machine, but 0744 prevent me from scanning files in the dir which is created by lumberjack. 0755 would be better

so... you can set the mode on the directory.. just create it yourself before starting lumberjack... mkdirall won't change the permissions if it's already there.
I don't really want to create a universally readable directory for logs on the off chance there's anything secret logged in there.  But if you want to do that for your case, you can do that without this PR.

Sorry, this was a brainfart on my part.  I was thinking you were asking to make the directory writable.  755 is correct to allow people to list the contents of the directory.


[Feature Request]  Add Documentation for Missing String Algorithms

🚀 Feature Request
Is your feature request related to a problem? Please describe.
The repository currently lacks documentation for several important string algorithms, which limits the educational resources available for those looking to learn string algorithms.
Describe the solution you'd like
I would like to contribute documentation for the following string algorithms, which are not currently present in the repository:

Boyer-Moore Algorithm
Aho-Corasick Algorithm
Naive String Matching Algorithm
Suffix Tree/Array Algorithms

Each algorithm would include:

A detailed explanation of the algorithm.
Time and space complexity analysis.
Example code (where applicable) in  C++.

Describe alternatives you've considered
N/A
Additional context
This contribution is part of my participation in GirlScript Summer of Code (GSSoC) 2024. Adding these algorithms will expand the repository's resources and benefit learners interested in algorithms.

Would you like to work on this feature?

Yes

Hi, I would like to work on this issue as part of GSSoC - Extended 2024. Could you please assign it to me and add the relevant labels (e.g., 'documentation', 'gssoc-ext')?


brain-score.org submission (user:386) | (public:False)

Plugin-only changes from brain-score.org submission. Generated via Jenkins job create_github_pr, build 260. Plugins added: models: [eBarlow_lmda_03_1_full]

Closing old submission PR. If you would like this submission to be reopened, please feel free to reach out here or on Slack. Thanks!


Setting Content-Type Response Header

Explicitly setting Content-Type to application/json for all successful json responses
Fix for #29

LGTM too, nothing can be done wrong in a line. :+1:

This is in! Kudos @eransevi 🎉

This is in! Kudos to @eransevi 🎉

@thealphadollar This is what I think whenever there is a "one-line" change.
https://twitter.com/bradfitz/status/1054905211372822528
TODO: Assert HTTP response headers in tests 😅


Building WhatsApp Based RAG BOT  (free to use) for UPSC Students

I want to make an free WhatsApp Chatbot for the UPSC Students. Students can easily get the answers from their syllabus, they can also check their knoweldge with the bot, also the bot has the window mechanism where the user can question fromt the chat history etc.
@UppuluriKalyani  @SaiNivedh26  @Neilblaze

@chakka-guna-sekhar-venkata-chennaiah Assigned


Make it work in all browsers via consolte-strings/browser variant.

This MR uses exact same primitives used by NodeJS and Chrome/ium based browsers and transforms these into a %c compatible variant in a 1:1 way, except the API exports a console-like namespace.

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

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

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
72

Relevant Lines:
72

💛  - Coveralls


youtube-dl-gui: init at 0.4

Motivation for this change
Fixes  #69334
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@davidak

@GrahamcOfBorg build youtube-dl-gui

I was able to run the program, add URLs to download, but when i hit start,
it "download latest youtube-dl" (should use packaged) and tries to download the videos. That fails.

I tested it briefly and thought it worked. I did see the downloading youtube-dl but then it went away when I added youtube-dl to propagated build inputs. Maybe mine had just managed to already download it and it needs to be added to the path instead. I'll have an investigate

I've had a look and this finds it's youtube-dl program using it's config file and I can't see any nice way to be able to pass config items in as the config file lives in the home directory. Not sure what is the best way to solve this

Ideally we want a way we could force config options from the command line, as when we could make a wrapper that supplies the correct path to the youtube-dl program

The best solution would be an upstream solution to disable automatic updates at buildtime since we want always our packaged version. I will create an issue there.
I think this state is usable and can be merged. It can later be improved.
Thanks for your work!

I'll make a comment in the code and remove youtube-dl from propagatedBuildInputs as it doesn't find it anyway

was going to make an issue but https://github.com/MrS0m30n3/youtube-dl-gui/issues/380 is worrying.
On a side note did you manage to test the ffmpeg stuff?

MrS0m30n3/youtube-dl-gui#380 is worrying.

oh no. then we might not add an unmaintained package with issue.
I hope there will be an active fork or another GUI.

did you manage to test the ffmpeg stuff?

no. but i did now.
I selected "mp3" as format and added a youtube video. It was downloaded 100% as webm and then status error. I can play the webm, but it has no image, just audio. Same with vorbis and m4a.
When selecting webm, the video has no audio.
mp4 works. it is the default.
maybe the ffmpeg package is missing codecs?

failing for me with:
builder for '/nix/store/jqn38icyy79rd0l3whwkdav3ddmn4ih2-python2.7-youtube-dl-gui-0.4.drv' failed with exit code 1; last 10 log lines:
  unpacking sources
  unpacking source archive /nix/store/77qdqidp5adszh8p86xgm26wwdmki2kc-source
  source root is source
  setting SOURCE_DATE_EPOCH to timestamp 315619200 of file source/youtube_dl_gui/widgets.py
  patching sources
  configuring
  no configure script, doing nothing
  building
  Executing setuptoolsBuildPhase
  No module named wx
cannot build derivation '/nix/store/c55bf68ads4q1vmk8031fc27d42znq1k-env.drv': 1 dependencies couldn't be built

There is still no popular, maintained fork. We should just close this.

There's https://github.com/axcore/tartube for anyone looking for alterantives.


Added Topic- Cyber Crimes

Create introduction related to cybercrimes and its awareness and list down the topic "Cyber Crimes" under "Table of Content".


test: build and run runtime tests on different custom docker images

What is the problem that this fixes or functionality that this introduces? Does it fix any open issues?
Fix https://github.com/OpenDevin/OpenDevin/issues/3314.
Prerequisite of fixing: https://github.com/OpenDevin/OpenDevin/issues/3415

Give a summary of what the PR does, explaining any non-trivial design decisions

Modify ghcr.yml to build runtime images for multiple base images (e.g., ['nikolaik/python-nodejs:python3.11-nodejs22', 'python:3.11-bookworm', 'node:22-bookworm'])
Run test_runtime.py on all of these images
Add a nodejs-specific test case in test_runtime.py
Move the app image building logic from ghcr.yml into a separate ghcr_app.yml

Other references

I'd concurr with @xingyaoww to collect further changes for a follow up PR

Finally got this working... GH Action is hard 😢


Sandbox: explore how to support direct helper process to renderer communication

Allow direct communication between a helper process and the renderer.

Some raw ideas

what if Electron would provide an API similar to BrowserWindow.loadUrl but UtilityProcess.loadUrl to create such a child process
Electron should delegate this to Chrome and reuse the ability Chrome has to spawn utility processes like GPU and Network
Utility process has node.js integration enabled and direct communication support to a renderer (either via electron IPC or message ports)

Prototype demonstrating communication between Node integration enabled chromium service process and sandbox renderer is complete.
Link: https://streamable.com/tqozwr
Current state of API:
// Main Process
interface ProcessOptions {
 entry: string;
 displayName?: string;
 args?: string[];
}

interface MessageEvent {
  data: any;
}

class UtilityProcess extends NodeJS.EventEmitter {

 constructor(options: ProcessOptions);

 postMessage(channel: string, message: any, transfer?: Electron.MessagePortMain[]): void;

 on(event: 'message', listener: (event: MessageEvent) => void): this;
}

// Service Process
interface MessageEvent {
 data: any;
 ports: MessagePortMain[];
}

interface Process extends NodeJS.EventEmitter {

 on(channel: string, listener: (event: MessageEvent) => void): this;

 send(message: any);
}

Next Steps in order:

[ ] Create custom builds with the new API
[ ] Implement extension host with the above API and iron out edge cases if any
[ ] Send RFC PR to https://github.com/electron/governance/tree/main/wg-api/spec-documents
[ ] Upstream implementation PR to https://github.com/electron/electron

Will this work enable shareable non-copy buffers, rather than one-way ownership transfer?

Exploration is complete and extension host has adopted this behind a setting. Next steps continued in https://github.com/microsoft/vscode/issues/154007


File.Select requesting csvs in Firefox from "text/plain"

I have a Cmd Msg of Select.file ["text/plain"] TextSelected (e.g)., which in Firefox opens a browser window with all files that are not ending .csv grayed out and unselectable.
Whereas in Safari, the .csv files are grayed out in favour of .txt files. In Chrome, both .csv and .txt files can be selected.
See Ellie example, or "irl" example

Can you confirm that this is specific to Elm? What happens when you make an HTML element like:
<input type="file" accept="text/plain"></input>

Just try to get it going in an HTML document and share the results!

Yep, I get the same results as above with the plain html posted above 🤦♂
I guess this means it is not an Elm-specific issue.

I wonder if there is something about it in the issue tracker for the browsers then. Not sure!

Yeah, I'll have a look on bugzilla et al. Thanks for your help!


Use in UIInterfaceOrientation Portrait some problem

po 0x79e6eb30
ProductSuitColletCell: 0x79e6eb30; baseClass = UICollectionViewCell; frame = (0 0; 171 185);
(lldb) po 0x7c97f6e0
ProductSuitColletCell: 0x7c97f6e0; baseClass = UICollectionViewCell; frame = (0 0; 171 185);
you can see the two cell's frame is same

Hello @zhou-honglin thanks for the report... Could you give me more information?


Nisse Engström: Watch repliqueluxemontre.com

Nisse Engström requests the watch of the watch_keyword repliqueluxemontre.com. See the Metasmoke search here and the Stack Exchange search here.

Coverage remained the same at 65.772% when pulling 4cbdb9fb9e65d79f361fdddf9fde80e3079075b5 on auto-blacklist-1512835350.4085498 into 273f2cbb3c76f693bb0b6b72389593cca43ab9e5 on master.

!!/approve


rename admin/expanding-formset.js (using kebab-case file)

small JS file clean up - the file was named expanding_formset.js not expanding-formset.js (all other similar JS files are kebab-case not snake_case).

@gasman agreed. Thanks - I'll add that to this PR


Empty Sequential has different semantics than other cases

As you can see from below, an empty chainer.Sequential() has different semantics: it wraps an argument with a tuple and return it. If at least one built-in links or functions exist, it returns a single Variable object.
I think it's better to forbid calling an empty Sequential.
Code
import numpy
import chainer
from chainer import functions as F

x = numpy.array([1, 2], numpy.float32)
seq = chainer.Sequential()
ret = seq(x)

print(ret)
print(type(ret))
print(type(ret[0]))
print()

seq = chainer.Sequential(F.tanh)
ret = seq(x)
print(type(ret))

Result
(array([1., 2.], dtype=float32),)
<class 'tuple'>
<class 'numpy.ndarray'>

<class 'chainer.variable.Variable'>

I don't think it's "different".  Chainer functions may return Variable, ndarray, or tuple of them.  Example: F.max_pooling_2d(return_indices=True).

@toslunar
That's because F.max_pooling_2d(return_indices=True) itself returns such values. That's a different story, I think.
Let me rephrase:
Sequential can be considered as a sequence of transforms. If there are one or more operations, that holds. If there's no operation, one would expect the input is returned as is. But currently this is not the case in empty Sequential.

What do you think chainer.Sequential()(x0, x1) should return?

@toslunar
I think there's no reasonable result everyone can agree with.
That's why I want to forbid calling an empty Sequential.

Therefore I don't think the semantics is "different".


⚠ Bofrost.es has degraded performance

In 8b12c62, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1861 ms

Resolved: Bofrost.es performance has improved in 95b6eff.


Semver 7 dependency analysis fails

I'd love to use NCC to bundle my Github Actions, but I'm running into an issue with Semver v7, which is a transitive dependency of some packages I use.
Semver 7 uses a pattern to only require() the things that are actually used:
lazyExport('valid', './functions/valid')

https://github.com/npm/node-semver/blob/70d9fb3f1c128227a3cd911867edb07a54260cad/index.js
ncc does not know that files like ./functions/valid should be bundled.
I tried to fix it here: https://github.com/goto-bus-stop/webpack-asset-relocator-loader/commit/f2bae069ee966e6f78f9c6fb8736c13462592295
By rewriting to:
lazyExport('valid', require.resolve('./functions/valid'))

That does put the functions in the bundle, but now the way semver calls require() with a variable with a value that is not known at analysis time seems to be a problem. Instead of using __webpack_require__, it is output as a plain node.js require(), so it ends up doing stuff like require(8) which doesn't work.
At that point I'm stuck. Is there a way to make this pattern work already in ncc, and I'm just missing it?
I assume a more likely path to success is to get those ./functions/valid things output as separate bundles in the dist/ folder. Then the node.js require() with a rewritten path would just work. I'm not sure where to look for that though! The special cases file has access to emitAssetDirectories, but I don't think that bundles them?
I would appreciate any pointers!

Semver 7.1.2 removed this pattern, so it's probably not worth it to spend any time supporting that small range from 7.0 til 7.1.2.


No longer works with the new Reddit UI

Screenshot.

Hi! Thanks for letting me know. I’ll take a look at this soon and see if I can’t get the extension updated.


Couldn't alter Rust env (RUSTFLAGS) for cargo-clippy invoked from rust-analyzer

To make long story short:
On my workstation I have RUSTFLAGS="-C link-arg=-fuse-ld=/usr/local/bin/zld -C target-feature=+sse3,+avx,+avx2,+fma"
It works beautifully until another cargo-check or cargo-clippy runs in my projects… WITHOUT that RUSTFLAGS set. Then, with each and every code change whole codebase gets rebuilt from scratch (which is normal).
I work around this, by disabling rust-analyzer cargo-check feature ("rust-analyzer.checkOnSave.enable": false) and use it from another editor extension (RustEnhanced on ST4) where I can specify env with my RUSTFLAGS for cargo-clippy/cargo-check.
Is it possible to tell rust-analyzer to use my desired env configuration somehow?

If you're using VSCode, you can just add it to rust-analyzer.server.extraEnv. Other editors might have different ways of configuring the environment.

Setting rust-analyzer.server.extraEnv should fix this.


Flaskr tutorial: auth.py raise a type error: tuple indices must be integers not str

In the tutorial of flaskr, an error is raised when running the auth.py. In the development server, the following error is raised:
File "~\flaskr\auth.py", line 90, in login
elif not check_password_hash(user['password'], password):
TypeError: tuple indices must be integers or slices, not str

And then I trace back, in line 84, the following code:
user = db.execute(
        'SELECT * FROM user WHERE username = ?', (username,)
).fetchone().

The function fetchone() should return a tuple. But in line 90, the function try to access the tuple by user['password']. Can tuple be accessed by this calling? I'm not pretty sure. But I can access the tuple by calling user[2] instead of user['password']. And actually the same situation in other python scripts. I'm not sure what is the case.
Environment

Python version: 3.7.0
Flask version: 1.0.2
Werkzeug version: 0.14.1

Be sure to follow the complete tutorial. You didn't set g.db.row_factory = sqlite3.Row. See http://flask.pocoo.org/docs/1.0/tutorial/database/#connect-to-the-database.
def get_db():
    if 'db' not in g:
        g.db = sqlite3.connect(
            current_app.config['DATABASE'],
            detect_types=sqlite3.PARSE_DECLTYPES
        )
        g.db.row_factory = sqlite3.Row

    return g.db

I follow it and I directly copy and paste the code. But there's still the problem.

Hi I discover the origin of the problem, it seems that my editor is having strange behavior. I solve the problem now using another editor. Sorry for inconvenience caused.


GitHub group no longer updates

http://haskellnews.org/grouped and https://www.haskell.org/news both are 11 months old in the GitHub section.  Running stack build && stack exec haskellnews -- haskellnews.conf github locally shows that the GitHub import code still works, as locally I have GitHub entries less than 30 minutes old.  Perhaps the cronjob that updates github no longer occasionally runs.

GitHub probably just changed their format again or whatever.

Checking both links you provide @herbertjones, it seems to be working again.
Please raise an issue if this comes up again.


Fix shopId in PluginConfigSetCommand

Why is this change necessary?
When the console command
sw:plugin:config:set --shopId=ID
is executed, an error occurs: "The identifier id is missing for a query of Shopware\Models\Shop\Shop"

 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.

Hey @fthunecke,
we cannot merge until you accept the cla.

Hey @shyim,
I have accepted the CLA, unfortunately the status was not adopted.
Even a recheck did not bring success either.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-25683
Please use this issue to track the state of your pull request.

Hey @fthunecke ,
It is already fixed in 5.7 https://github.com/shopware/shopware/blob/5.7/engine/Shopware/Commands/PluginConfigSetCommand.php#L191
Anyway thanks for your Contribution! :blue_heart:


[AMBARI-24917] Implement complex Add Service request using default configs

What changes were proposed in this pull request?
Implement the following for complex "Add Service" request:

create components
create host components
create configs for the new services based on stack defaults
trigger install/start of host components
return request status for monitoring

Custom configs (user-defined and advisor-recommended) will be implemented separately.
How was this patch tested?
Performed initial blueprint deployment.  Tested the new request using API call:
$ curl -X POST -d @- http://c7401.ambari.apache.org:8080/api/v1/clusters/TEST/services
{
  "operation_type": "ADD_SERVICE",
  "stack_name": "HDP",
  "stack_version": "3.0",
  "components": [
    { "component_name": "DATANODE", "fqdn": "c7401.ambari.apache.org" },
    { "component_name": "HDFS_CLIENT", "fqdn": "c7401.ambari.apache.org" },
    { "component_name": "KAFKA_BROKER", "fqdn": "c7402.ambari.apache.org" },
    { "component_name": "METRICS_COLLECTOR", "fqdn": "c7402.ambari.apache.org" },
    { "component_name": "METRICS_MONITOR", "fqdn": "c7401.ambari.apache.org" },
    { "component_name": "METRICS_MONITOR", "fqdn": "c7402.ambari.apache.org" },
    { "component_name": "NAMENODE", "fqdn": "c7401.ambari.apache.org" },
    { "component_name": "SECONDARY_NAMENODE", "fqdn": "c7402.ambari.apache.org" }
  ]
}

HTTP/1.1 202 Accepted
...
{
  "href" : "http://c7401.ambari.apache.org:8080/api/v1/clusters/TEST/requests/12",
  "Requests" : {
    "id" : 12,
    "status" : "Accepted"
  }
}

Monitored the request on Ambari UI.
After successful completion:
$ curl "http://c7401.ambari.apache.org:8080/api/v1/clusters/TEST/requests/12?fields=Requests/request_status,Requests/request_context"
{
  "href" : "http://c7401.ambari.apache.org:8080/api/v1/clusters/TEST/requests/12",
  "Requests" : {
    "cluster_name" : "TEST",
    "id" : 12,
    "request_context" : "Add Services: AMBARI_METRICS, HDFS, KAFKA to cluster TEST",
    "request_status" : "COMPLETED"
  }
}

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4586/
Test PASSed.


TypeNameHandling.Auto not working properly for object type.

Quoted from documentation: "Include the .NET type name when the type of the object being serialized is not the same as its declared type."
class MyClass { object Data { get; set;} }
var list = new List<int> { 1, 2, 3 };
var my = new MyClass { Data = list };
var settings = new JsonSerializerSettings { TypeNameHandling = TypeNameHandling.Auto };
var s = JsonConvert.SerializeObject(my, settings);

Data property is declared as object, but the actual value is List<int>. According to documentation the type information should be included. But the serialized string doesn't have type information for Data. Instead, the Data property is simply serialized as JSON array. This looks okay, but will cause problem during deserialization:
IList<int> value;
var obj = JsonConvert.DeserializeObject<MyClass>(s, settings);
value = (IList<int>)obj.Data; -> throws exception Unable to cast object of type 'Newtonsoft.Json.Linq.JArray' to type 'System.Collections.Generic.List`1[System.Int32]'

Turns out it is due to custom converter.


How to render text layer

I am trying to render the text content of a PDF as is done here. I wasn't able to find an exposed API that would support this, event though #858 makes reference to page.renderTextLayer. Digging through the source I was able to find page.getTextContent and was able to get PDFJS.renderTextLayer by including src/display/text_layer.js in my source. Even then the text layer doesn't render as expected.
Two questions:

Why does the PDF text layer render incorrectly (example | source)?
Why isn't renderTextLayer exposed on either the page or PDFJS objects?

You need some more css. Checkout our main viewer for examples.
But overall add a .textLayer class to your wrapping div and make it relative or absolute and

.textLayer > div {
   position: absolute;
}

Not positive what the above is asking, but it is exposed on PDFJS just not documented. We should probably move it to api.js and add some docs there.

I'd suggest looking into using the viewer components, see https://github.com/mozilla/pdf.js/tree/master/examples/components, which are intended to simplify building a custom viewer.

@brendandahl thanks for your response. I could have sworn that I already tried that, but sure enough it did the trick :facepalm:
Aso for PDFJS.renderTextLayer it doesn't exist unless I include src/display/text_layer.js in my project. I am getting PDFJS from pdfjs-dist/build/pdf.js using the pdfjs-dist npm module.

Aso for PDFJS.renderTextLayer it doesn't exist unless I include src/display/text_layer.js in my project. I am getting PDFJS from pdfjs-dist/build/pdf.js using the pdfjs-dist npm module.

@mzabriskie it shall be included in pdfjs-dist's pdf.js, see https://github.com/mozilla/pdfjs-dist/blob/master/build/pdf.js#L5373

Also find all needed CSS at https://github.com/mozilla/pdfjs-dist/blob/master/web/pdf_viewer.css

You're correct PDFJS.renderTextLayer exists in the current version. I was apparently using an older version. Updating to latest version solves my problem.

Hello，guys，i know how to do that.
1、incllude CSS file in your page

2、create a text layer div with the following code

3、replace js code:  page.render(renderContext);  to the following code
page.render(renderContext).then(() => {
          return page.getTextContent();
        }).then((textContent) => {
          // 创建文本图层div
          const textLayerDiv = document.getElementById('textlayer');
          textLayerDiv.setAttribute('class', 'textLayer');
          // 创建新的TextLayerBuilder实例
          var textLayer =  pdfjsLib.renderTextLayer({
            textContent: textContent,
            container: textLayerDiv,
            viewport: viewport
          });
          textLayer._render();
          //textLayer.setTextContent(textContent);
          //textLayer.render();
        });

https://usefulangle.com/post/90/javascript-pdfjs-enable-text-layer

Any tips on how to run renderTextLayer in node?
return page.getTextContent().then(textContent => {
  const div = document.createElement("div");
  pdfjsLib.renderTextLayer({
    textContent,
    container: div,
    viewport,
    textDivs: [],
  });
});

Raises TypeError: canvas.getContext is not a function
My plan was to extract SVG and media, render server-side then deliver, but mobile browsers still struggle, eg, to zoom. So I'm now trying to super-impose a text layer over an image. I want to extract the text layer exactly as it is rendered as spans in a div, I'm not worried about updating for zoom level, etc.
My full script is below, attempt at text extraction on line 242:
scrape2.js.zip


from tensorflow.python.keras.utils import tf_utils ImportError: cannot import name 'tf_utils'

run main.py,an error has occurred
Using TensorFlow backend.
Traceback (most recent call last):
File "F:/action recognition demo/OPEN-POSE/Online-Realtime-Action-Recognition-based-on-OpenPose/main.py", line 8, in
from Action.recognizer import load_action_premodel, framewise_recognize
File "F:\action recognition demo\OPEN-POSE\Online-Realtime-Action-Recognition-based-on-OpenPose\Action\recognizer.py", line 10, in
from keras.models import load_model
File "C:\Users\Administrator\Anaconda3\lib\site-packages\keras_init_.py", line 3, in
from . import utils
File "C:\Users\Administrator\Anaconda3\lib\site-packages\keras\utils_init_.py", line 6, in
from . import conv_utils
File "C:\Users\Administrator\Anaconda3\lib\site-packages\keras\utils\conv_utils.py", line 9, in
from .. import backend as K
File "C:\Users\Administrator\Anaconda3\lib\site-packages\keras\backend_init_.py", line 1, in
from .load_backend import epsilon
File "C:\Users\Administrator\Anaconda3\lib\site-packages\keras\backend\load_backend.py", line 90, in
from .tensorflow_backend import *
File "C:\Users\Administrator\Anaconda3\lib\site-packages\keras\backend\tensorflow_backend.py", line 13, in
from tensorflow.python.keras.utils import tf_utils
ImportError: cannot import name 'tf_utils'

@FUZIRONG what is your keras version? mine is 2.2.4 and it works fine

@FUZIRONG what is your keras version? mine is 2.2.4 and it works fine

Thanks for the reply, it is caused by the problem of keras version

@FUZIRONG can you tell me which version of keras and which version of tensorflow do you use ?

@FUZIRONG can you tell me which version of keras and which version of tensorflow do you use ?

keras ==2.2.4
tensorflow-gpu==1.14.0

@FUZIRONG can you tell me which version of keras and which version of tensorflow do you use ?

keras ==2.2.4
tensorflow-gpu==1.14.0
it works for me

Can you give me a list of environment packages (including package name and version) for running this program?

2022-10-03 22:00:26.553326: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library cudart64_101.dll
2022-10-03 22:00:33.309597: I tensorflow_io/core/kernels/cpu_check.cc:128] Your CPU supports instructions that this TensorFlow IO binary was not compiled to use: SSE SSE2 SSE3 SSE4.1 SSE4.2 AVX AVX2 FMA
Using TensorFlow backend.
D:\ANPR\anprsys\lib\site-packages\tensorflow_addons\utils\ensure_tf_install.py:67: UserWarning: Tensorflow Addons supports using Python ops for all Tensorflow versions above or equal to 2.8.0 and strictly below 2.11.0 (nightly versions are not supported).
The versions of TensorFlow you are currently using is 2.3.0 and is not supported.
Some things might work, some things might not.
If you were to encounter a bug, do not file an issue.
If you want to make sure you're using a tested and supported configuration, either change the TensorFlow version or the TensorFlow Addons's version.
You can find the compatibility matrix in TensorFlow Addon's readme:
https://github.com/tensorflow/addons
UserWarning,
Traceback (most recent call last):
File "D:\ANPR\anprsys\lib\site-packages\object_detection-0.1-py3.7.egg\object_detection\models\ssd_efficientnet_bifpn_feature_extractor.py", line 35, in
from official.legacy.image_classification.efficientnet import efficientnet_model
File "D:\ANPR\anprsys\lib\site-packages\tf_models_official-2.10.0-py3.7.egg\official\legacy\image_classification\efficientnet\efficientnet_model.py", line 31, in
from official.legacy.image_classification import preprocessing
File "D:\ANPR\anprsys\lib\site-packages\tf_models_official-2.10.0-py3.7.egg\official\legacy\image_classification\preprocessing.py", line 22, in
from official.legacy.image_classification import augment
File "D:\ANPR\anprsys\lib\site-packages\tf_models_official-2.10.0-py3.7.egg\official\legacy\image_classification\augment.py", line 28, in
from keras.layers.preprocessing import image_preprocessing as image_ops
ModuleNotFoundError: No module named 'keras.layers.preprocessing'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "Tensorflow\models\research\object_detection\builders\model_builder_tf2_test.py", line 24, in
from object_detection.builders import model_builder
File "D:\ANPR\anprsys\lib\site-packages\object_detection-0.1-py3.7.egg\object_detection\builders\model_builder.py", line 70, in
from object_detection.models import ssd_efficientnet_bifpn_feature_extractor as ssd_efficientnet_bifpn
File "D:\ANPR\anprsys\lib\site-packages\object_detection-0.1-py3.7.egg\object_detection\models\ssd_efficientnet_bifpn_feature_extractor.py", line 37, in
from official.vision.image_classification.efficientnet import efficientnet_model
File "D:\ANPR\anprsys\lib\site-packages\tf_models_official-2.10.0-py3.7.egg\official\vision_init_.py", line 17, in
from official.vision import configs
File "D:\ANPR\anprsys\lib\site-packages\tf_models_official-2.10.0-py3.7.egg\official\vision\configs_init_.py", line 19, in
from official.vision.configs import common
File "D:\ANPR\anprsys\lib\site-packages\tf_models_official-2.10.0-py3.7.egg\official\vision\configs\common.py", line 22, in
from official.core import config_definitions as cfg
File "D:\ANPR\anprsys\lib\site-packages\tf_models_official-2.10.0-py3.7.egg\official\core_init_.py", line 17, in
from official.core import actions
File "D:\ANPR\anprsys\lib\site-packages\tf_models_official-2.10.0-py3.7.egg\official\core\actions.py", line 25, in
from official.core import base_trainer
File "D:\ANPR\anprsys\lib\site-packages\tf_models_official-2.10.0-py3.7.egg\official\core\base_trainer.py", line 28, in
from official.core import base_task
File "D:\ANPR\anprsys\lib\site-packages\tf_models_official-2.10.0-py3.7.egg\official\core\base_task.py", line 23, in
from official.core import config_definitions
File "D:\ANPR\anprsys\lib\site-packages\tf_models_official-2.10.0-py3.7.egg\official\core\config_definitions.py", line 21, in
from official.modeling.optimization.configs import optimization_config
File "D:\ANPR\anprsys\lib\site-packages\tf_models_official-2.10.0-py3.7.egg\official\modeling\optimization_init_.py", line 23, in
from official.modeling.optimization.optimizer_factory import OptimizerFactory
File "D:\ANPR\anprsys\lib\site-packages\tf_models_official-2.10.0-py3.7.egg\official\modeling\optimization\optimizer_factory.py", line 20, in
import tensorflow_addons.optimizers as tfa_optimizers
File "D:\ANPR\anprsys\lib\site-packages\tensorflow_addons_init_.py", line 22, in
from tensorflow_addons import callbacks
File "D:\ANPR\anprsys\lib\site-packages\tensorflow_addons\callbacks_init_.py", line 17, in
from tensorflow_addons.callbacks.average_model_checkpoint import AverageModelCheckpoint
File "D:\ANPR\anprsys\lib\site-packages\tensorflow_addons\callbacks\average_model_checkpoint.py", line 18, in
from tensorflow_addons.optimizers.average_wrapper import AveragedOptimizerWrapper
File "D:\ANPR\anprsys\lib\site-packages\tensorflow_addons\optimizers_init_.py", line 30, in
from tensorflow_addons.optimizers.discriminative_layer_training import (
File "D:\ANPR\anprsys\lib\site-packages\tensorflow_addons\optimizers\discriminative_layer_training.py", line 25, in
from keras.utils import tf_utils
ImportError: cannot import name 'tf_utils' from 'keras.utils' (D:\ANPR\anprsys\lib\site-packages\keras\utils_init_.py)

I was able to resolve the error above by installing tensorflow addons with the tensorflow addons extension:
pip install tensorflow==2.12 tensorflow-addons[tensorflow]

Note the tensorflow version I am using is version 2.12.0


Don't use setAttribute when updating 'value'

When updating a value, for example for an input field,
updating the attributes value change the default value, so has no effect for later updates.
In the specific case, we should update the 'value' key of the dom object instead of the attribute.

Just close this one, will open a new one, there is a mistake in the commit.

I think not only value, if want to change ...

What do you mean  ?

Maybe fix checked, selected ..., is it better?

Humm, I guess you are right, but we should be carreful with attributes such as checked or selected.
The fact is that
<input id="foo" type="checkbox" checked=false /> <!-- will be checked -->

is different from
document.querySelector('#foo'].checked = false; // not checked

What do you think ?

That's right. And I just give the suggestion. Maybe I should know more about the code.


Purehugo - blacklisted theme

Hi guys,
It seems that during the themes site migration the purehugo theme got blacklisted. I tried cloning the theme and using it against the examplesite repo and everything seems ok, both in 0.19 and 0.20.
I couldn't get the generate theme site script to work unfortunately, so I can't see if there were any errors there. Can you please let me know if there's anything wrong so I can fix the theme and get it removed from the blacklist?
Thanks in advance!

Now it's back to normal. Thanks for the heads up.


HV-1352 Only accessing validated value after a value unwrapper has se…

…t (if present)

@marko-bekhta That's the new PR for HV-1352. WDYT?

Looks good !

Merged to 5.4.


new sample app from microservices-demo: https://github.com/microservices-demo/microservices-demo

What doing this would involve:

look to ecommerce-webapp as a starting point
this is the part of the setup.sh that would likely need changing
most of the actual changes / work would be in configuring the docker-compose.yml

We've got some expertise over in https://github.com/DataDog/bitsboutique on this a bit. We're using it for USM. @ktmq is our expert. Come chat in #demo-environments-backroom if you like,.


Update self.live variable when shutting down

self.live was updated for LegacyGrpcServer but was not for GrpcServer.
This causes issues when shutting down and starting a new server immediately from the same variable.
Perhaps it may be better to add a mechanism upper in the class hierarchy in BaseServer so this variable gets updated for every child class ?

@ansys-akarcher mayeb you could add the script you created to reproduce the bug as a unit test? they are in "/tests" folder


Supportmxking

Keybase GUI Version: 3.1.2-20190312073438+4c383e6a53

Help Me !

?

@supportmxking, do you have a log send ID or more information about your issue?


Do we handle Slack Shortcuts?

is it possible to handle shortcuts from this Provider?

The shortcut is already handled.
I'm going to update the documentation.
The decorator is defined  here https://github.com/bamada/nest-slack-bolt/tree/main/src/decorators


Updated FL0 documentation to use Direct destination template

Proposed changes
Switched to use Direct documentation template instead of Action template.
Merge timing
ASAP once approved

FYI the link we received in the Zendesk ticket contained a link to the Action template. If you're using canned responses, it might be useful to update this based on which destination type the partner is using so they don't accidentally use the wrong template like we did.
Thanks!


Reproduce oscillations caused by a typo in Eq 25 in the Fortran code

Charlotte discovered that correcting a typo in the Fortran code to what was stated in the paper brought the oscillations back. After applying the typo, Fortran and rhytmite reproduce oscillations again. Try to reproduce them also with marlpde.

Crashes with the following:
 34%|███████████▌                      | 10.15536/30.0 [20:15<39:35, 119.71s/it]Traceback (most recent call last):
  File "/Users/emilia/Documents/Documents - UU101746/GitHub/Integrating-diagenetic-equations-using-Python/marlpde/Evolve_scenario.py", line 143, in <module>
    integrate_equations(**all_kwargs)
  File "/Users/emilia/Documents/Documents - UU101746/GitHub/Integrating-diagenetic-equations-using-Python/marlpde/Evolve_scenario.py", line 96, in integrate_equations
    sol, info = eq.solve(state, t_range=End_time, dt=dt, \
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/pde/pdes/base.py", line 591, in solve
    final_state = controller.run(state, dt)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/pde/solvers/controller.py", line 326, in run
    self._run_single(state, dt)
  File "/opt/homebrew/lib/python3.11/site-packages/pde/solvers/controller.py", line 206, in _run_single
    t = stepper(state, t, t_break)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/pde/solvers/base.py", line 603, in wrapped_stepper
    t_last, dt_float, steps, modifications = adaptive_stepper(
                                             ^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/pde/solvers/base.py", line 403, in adjust_dt
    raise RuntimeError(dt_min_err)
RuntimeError: Time step below 1e-10

Might be related to #36 because certain combinations of Phi might overlap with the values corresponding to what is produced in this branch. Maybe worth re-running with other values of b.

Ok I ran out of ideas, let's see if @HannoSpreeuw can figure this out.

Okay, so to clarify this issue: we are trying to have marlpde complete long integrations with incorrect code?
Incorrect as in: not in agreement with the paper?

I reproduced the error.

I tried solver: str   = "adams–bashforth" in parameters.py.
That works with scheme=kwargs["scheme"], and , adaptive=kwargs["adaptive"] removed from sol, info = eq.solve(state, .... in Evolve_scenario.py.
Needs a pretty new version of py-pde though.

The Final_distributions.pdf plot is empty, though, so I need to dig into that.

Is there anything in the Results file?

Yes there is also a hdf5 file, I will look into it.

As a side note - I picked up the result from a parallel run - solver: str   = "crank-nicolson" yielded
raise ConvergenceError("Crank-Nicolson step did not converge.")

at 34% of the integration time (=30 times T*)

The solver: str   = "adams–bashforth" run showed an empty plot because all saved values - except at t=0 - are nans.
So it seems this integration method derails within 1% of T* - data at 3001 times, covering 30 times T* have been saved.
I guess we need an implicit scheme to solve for this code modification, this would require branching off Use_solve_ivp_without_py-pde_wrapper.
To be continued tomorrow.

Alternatively, we could settle for integrating over 10 times T*, with "crank-nicolson", that should just about complete, since this is 33.33% of T*.
Running that now, to see if this yields anything sane.

Yields an interesting plot. After 10 times T* calcite starts to derail while the other four quantities are still physically possible.

It seems you have reproduced the oscillations in #40 so I would like to suggest merging this branch after applying the version of the bottom boundary condition that is in agreement with the rhytmite.

That should be okay after we have agreed on the best i.e. least worrying, ad-hoc bottom boundary condition for aragonite and calcite.


Encapsulated CSS vanishes if a container element has an outro

Pretty sure this is a regression http://jsfiddle.net/rich_harris/1qaL8q37/

New rule: rather than carefully trying to orchestrate the addition and removal of CSS as and when components enter and leave the page, I reckon it makes more sense if CSS is added to the page at the point of Ractive.extend(). At the cost of occasionally having a more complex CSSOM than is necessary, this means...

way less state management (no listening for transition ends, etc)
the CSSOM doesn't keep changing (theoretically better for performance, as it will avoid re-layouts in some situations
easier debugging - you can inspect the encapsulated styles in the page, without them changing under your cursor
easier debugging - non-encapsulated styles (noCssTransform: true) will do whatever damage they're capable of immediately, rather than surprising you when the component in question is rendered for the first time
we can remove some code

Working up a PR now.

rather than carefully trying to orchestrate the addition and removal of CSS as and when components enter and leave the page, I reckon it makes more sense if CSS is added to the page at the point of Ractive.extend()

:+1:


refactor: use parentId instead of remark for tx split checking

@robertu7 I guess can deploy together with next release as the existing check still works


nitiwiki: introduce wikilinks

Nitiwiki now understand the wikilink format using double brackets.
Examples:
[[foo]]: link to the page named foo
[[foo/bar]]: link to the page named bar in foo
[[/foo/bar]]: link to the page located exactly here
It's also possible to add a custom name:
[[foo|My cute custom name <3]]: link to the page named foo
And to use anchors:
[[foo#bar]]: link to the page named foo
nitiwiki will display warnings if a link seems broken.

démo?

Reroll for tests

Is there a specific reason that you choose the [[link|text]] syntax and this automatic search?
What happen when one link [[baz]] from foo.md on the following file system? is this foo/bar1/baz.md or foo/bar2/baz.md or something else?
|-- foo.md
`-- foo
    |-- bar1
    |   `-- baz.md
    `-- bar2
        `-- baz.md

+1

I guess @xymus and @privat said everything that needed to be said, :shipit: then !

@privat: We were discussing the wikilinks syntax, here an overview:
Nitiwiki		[[link|Link Text]]
Wikipedia		[[link|Link Text]]
Office 360 	[[link|Link Text]]
Ikiwiki		[[Link Text|link]]
Github wiki	[[Link Text|link]]
Conluence	[[Link Text|link]]
Jekyll		{{command = args...}}
Tiki wiki		((link|Text)) or [link|Text]
RDoc		{Text}[link]
Doxygen		\ref <link>[“text”]
Sphinx		```:ref:`Text ````
DocBooks	<link  linkend="link">Text</link>
Natural docs	<Objects> ← accepte et traduit les pluriels also <Text at link>

go with ikiwiki & github then

Reroll:

inverted link and title posititon as asked by @privat
nitiwiki now searches children recursively for wikilinks as asked by @privat
Uniformized wikilinks words as asked by @xymus

:shipit:

I never asked to search children recursively. No other wiki that I know does that.. but why not.

You asked exactly for that in your office, aging does not suit you very well :P

@Morriar there was a misunderstanding then, young boy.


Add image template for systemd-debian:sid

Many people need to test systemd stuff on the development version of
Debian before it is released, so it is good to have it available as well.

Thanks @ottok!


1.2.0 changes QnAMaker Dialog subscriptionKey to authKey which is a breaking API change

Hi,
The commit https://github.com/Microsoft/BotBuilder-CognitiveServices/commit/a62a4e2d4f89c59469388c0852cfcf3c29bc4a39 introduced an API change such that any bots generated in Azure today are broken upon deployment, and there's no useful error logged to explain why all requests are 401.
This should've been 2.0.0
A bot created in Azure today will have ^1.1.0 for this project, such that 1.2.0 with the API change will be installed; this is the code the bot will have:
var recognizer = new builder_cognitiveservices.QnAMakerRecognizer({
                knowledgeBaseId: process.env.QnAKnowledgebaseId,
    subscriptionKey: process.env.QnASubscriptionKey});

Yes, but all existing bots that can be reinstalled will have this issue. that's why it needs to be 2.0.0, the way it is now is effectively a surprised forced upgrade / refactor.

There's also no git tags on this repo, making it very very hard to track changes.

Related to what @avoidwork said above not only would git tags be helpful to investigate differences but a changelog would be helpful too - thanks!

Thank you for opening an issue against the Bot Framework Cognitive Services repo. As part of the Bot Framework v4 release, we’ve moved all the Luis and QnA work to platform SDK repo. We will continue to support and offer maintenance updates via this repo.
From now on, https://github.com/Microsoft/botframework-sdk repo will be used as hub, with pointers to all the different SDK languages, tools and samples repos.
As part of this restructuring, we are closing all tickets in this repo.
For defects or feature requests, please create a new issue in the Bot Framework repo found here:
https://github.com/Microsoft/botframework-sdk/issues
For product behavior, how-to, or general understanding questions, please use Stackoverflow.
https://stackoverflow.com/search?q=bot+framework
Thank you.
The Bot Framework Team


TAJO-1437: Resolve findbug warnings on Tajo JDBC Module

Performance: Replace 'new String("UDF")' into 'private static final String'.
Bugfix: execute() saves the result set, now.
Unused variable: warningChain is defined but always set to null.
Standard compliance: raise SQLException on operation with a closed connection

It passed the following command.
mvn clean install -Pparallel-test,hcatalog-0.12.0 -DLOG_LEVEL=INFO -Dmaven.fork.count=2 > TAJO-1437.travis.log.INFO.txt

Result is here. https://app.box.com/s/07oenp32etnrhgwygoq6yz213lnubzpc (72MB)

Rebased.

+1


[feat] Markdown support

close #4

Refer to https://github.com/vuejs/vetur/blob/master/syntaxes/markdown-vue.json


APP-633: Returning HTTP 415 when the payload content-type is not supported.

Payload supports now a MediaType.

Can one of the admins verify this patch?


AsDataSet() and IsFirstRowAsColumnNames not recognized in version 3.1.0

Hey,
Today i updated the ExcelDataReader package to its latest version and i found that it is not working anymore. It doesnt recognize AsDataSet() and IsFirstRowAsColumnNames as properties/methods anymore. My project won't build now.
A namespace such as ExcelDataReader.DataSet doesn't exist either.
When i revert the update to an older verison its working again(<IP_ADDRESS>)
My code atm:
        IExcelDataReader excelReader = fileExtension.ToLower().Equals("xls") ? ExcelReaderFactory.CreateBinaryReader(inputStream) : ExcelReaderFactory.CreateOpenXmlReader(inputStream);
        DataSet result = excelReader.AsDataSet();   <----- Won't work
        excelReader.IsFirstRowAsColumnNames = false;    <----- Won't work
        DataTable table = result.Tables[0];

Please see https://github.com/ExcelDataReader/ExcelDataReader/issues/261#issuecomment-321459098

I just came across this today as well. I found out that AsDataSet() is now part of ExcelDataReader.Dataset. I have not been able to find out what happened to IsFirstRowAsColumnNames

This issue seems to get a lot of traffic from search engines, so I'll replicate the newly added section in the README based on https://github.com/ExcelDataReader/ExcelDataReader/issues/261#issuecomment-321459098
Important note when upgrading from ExcelDataReader 2.x
ExcelDataReader 3 had some breaking changes, and older code may produce error messages similar to:
'IExcelDataReader' does not contain a definition for 'AsDataSet'...
'IExcelDataReader' does not contain a definition for 'IsFirstRowAsColumnNames'...

To fix:

Make sure to rename any Excel namespace references in the code to the new namespace ExcelDataReader

Make sure the project has a reference to the ExcelDataReader.DataSet package to use AsDataSet()

Remove the line of code with IsFirstRowAsColumnNames and change the call to AsDataSet() to something like this:

var result = reader.AsDataSet(new ExcelDataSetConfiguration() {
	ConfigureDataTable = (_) => new ExcelDataTableConfiguration() {
		UseHeaderRow = true
	}
});

Install Nuget ExcelDataReader.DataSet
Add: using System.Data;

This issue seems to get a lot of traffic from search engines, so I'll replicate the newly added section in the README based on #261 (comment)
Important note when upgrading from ExcelDataReader 2.x
ExcelDataReader 3 had some breaking changes, and older code may produce error messages similar to:
'IExcelDataReader' does not contain a definition for 'AsDataSet'...
'IExcelDataReader' does not contain a definition for 'IsFirstRowAsColumnNames'...

To fix:

Make sure to rename any Excel namespace references in the code to the new namespace ExcelDataReader
Make sure the project has a reference to the ExcelDataReader.DataSet package to use AsDataSet()
Remove the line of code with IsFirstRowAsColumnNames and change the call to AsDataSet() to something like this:

var result = reader.AsDataSet(new ExcelDataSetConfiguration() {
	ConfigureDataTable = (_) => new ExcelDataTableConfiguration() {
		UseHeaderRow = true
	}
});

it works for me, thanks so much


Add games

New game considerations:

[x] Diablo II Resurrected,
[x] Diablo III,
[x] Diablo IV,
[x] Path of Exile
[x] Path of Exile 2,
[x] Torchlight: Infinite,
[x] Last Epoch,
[x] PD2 (Diablo II Mod),
[x] Hero Siege,
[ ] Titan Quest 2,
[ ] Undecember,
[ ] MedianXL
[ ] pathofdiablo
[ ] Grim Dawn

Help wanted needed with maintaining the timelines for these games.

pd2 will be rly nice if you add it

(Spellwake)[https://store.steampowered.com/app/921070/Spellwake/] #64

Skipping Spellwake, peak concurrent palyers 48, 6 years ago.


m3u8 stream is loaded but ts file not downloading

I've been using HLS downloader to save streams several times per day over the past month, but a couple days ago it stopped working. I can still load an m3u8 stream (normal behavior, loading from 0% to 100% over the course of a couple minutes). However, once that reaches 100%, the HLS Downloader is no longer proceeding to download the ts file.

Hey! Thanks for opening the issue, if it's possible, please share a link so I would be able to debug it.

https://player.arsenal.com/video/liverpool-h-1st-half-1
https://cf.vod.mp.streamamg.com/vod/_definst_/s3/3000504/20191021/3000/0_6u97ymlv_0_zse41y6l_2.mp4/chunklist.m3u8?Policy=eyJTdGF0ZW1lbnQiOlt7IlJlc291cmNlIjoiaHR0cHM6Ly9jZi52b2QubXAuc3RyZWFtYW1nLmNvbS92b2QvX2RlZmluc3RfL3MzLzMwMDA1MDQvMjAxOTEwMjEvMzAwMC8wXzZ1OTd5bWx2XzBfenNlNDF5NmxfMi5tcDQvY2h1bmtsaXN0Lm0zdTgiLCJDb25kaXRpb24iOnsiRGF0ZUxlc3NUaGFuIjp7IkFXUzpFcG9jaFRpbWUiOjE1OTE1ODQ4ODF9fX1dfQ__&Signature=Ro7ivZaXakR3em9XyrdVs18KYZJO2ljRfyjkmhi9iIWAwz2k~7DTvqGBHIZTvWbS6POxnf5FCTZQWQ~Swy~g~uCLfNNxDYa7L0rUe0XNOXe8MGi~fMDZMsLEd17zw~21YY9ErtS8suWp3-DT-JbSHt2X2h8mOLdZ~D3NcpN~ZwnfTSGEV-0~JI74OgEGrsSIVKIq44BK7cigMBdLnnumyg4JI-yOllIaslEy1quJr8elztqrQZyW6PDvZRAbry0IopfDUBf3Ph6pjHSzCq9q4Tv2Bq2HyOKYi7XcIyL6Dx8kLQhUpZ8ZSO1VaN7Ny8MUktQKnPIXNhNLZBPVOhiQqA__&Key-Pair-Id=APKAJVKZD2WR75FL2ZUQ

It's working now --- very odd! I'll reach out again if it stops working.

Also thank you for building this, I use it frequently. Amazing, and very convenient.

sorry but  loading from 0% to 100% over the course of a couple minutes). However, once that reaches 100%, the HLS Downloader is no longer proceeding to download the ts file

@puemos I have figured out that this only happens for ts files larger than ~1.75 GB

@puemos I've the same problem. Downloading a ~1.92 Gig TS File works. 1.99 Gig did not work.
Can u please fix it?

Work in progress #132


Text field cursor position in RTL goes to the penultimate character - still exists in flutter 3.7.0

When the user writes then taps at the end of a text field in RTL app the cursor position is set to the penultimate character instead of the last one. This issue occurs on both Android (tested on multiple devices from different vendors and emulator) and iOS.
Steps to Reproduce
Execute flutter run on the code sample
Write Arabic or any other RTL language in the text field.
Tap on any place to change the cursor position then tap again at the end of the text field, the cursor position will change to the penultimate character.
Expected results:
The cursor position should be after the last character.
Actual results:
The cursor position moves to the penultimate character.
import 'package:flutter/material.dart';
import 'package:flutter_localizations/flutter_localizations.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      locale: const Locale("ar"),
      localizationsDelegates: const [
        GlobalMaterialLocalizations.delegate,
        GlobalWidgetsLocalizations.delegate,
        GlobalCupertinoLocalizations.delegate,
      ],
      supportedLocales: const [Locale("ar")],
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key, required this.title}) : super(key: key);

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: const Center(
        child: Padding(
          padding: EdgeInsets.all(16),
          child: TextField(),
        ),
      ),
    );
  }
}

here is the link to the original issue

fvm flutter doctor -v
[✓] Flutter (Channel stable, 3.7.0, on macOS 13.2 22D49 darwin-arm64, locale en-IR)
    • Flutter version 3.7.0 on channel stable at /Users/alireza/fvm/versions/3.7.0
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b06b8b2710 (4 weeks ago), 2023-01-23 16:55:55 -0800
    • Engine revision b24591ed32
    • Dart version 2.19.0
    • DevTools version 2.20.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/alireza/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14C18
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] IntelliJ IDEA Community Edition (version 2021.1.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 62.0.2
    • Dart plugin version 211.7798

[✓] VS Code (version 1.75.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.58.0

[!] Proxy Configuration
    • HTTP_PROXY is set
    • NO_PROXY is <IP_ADDRESS>
    ! NO_PROXY does not contain localhost
    • NO_PROXY contains <IP_ADDRESS>
    ! NO_PROXY does not contain ::1

[✓] Connected device (3 available)
    • M2102J20SG (mobile) • d4fc1ae • android-arm64  • Android 12 (API 31)
    • macOS (desktop)     • macos   • darwin-arm64   • macOS 13.2 22D49 darwin-arm64
    • Chrome (web)        • chrome  • web-javascript • Google Chrome 109.0.5414.119

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

@easazade
There's a similar open issue describing your case, https://github.com/flutter/flutter/issues/118566 and per this comment, it seems to be fixed on latest master, so please run your scenario on latest master and see if the issue is fixed or not. Accordingly you can share your findings in the linked issue.
Closing this as a duplicate.


add(kepler) stop button and disable settings when previewing or rendering

add(kepler) stop button for both preview and partial render
add(kepler) disable settings when previewing or rendering

https://user-images.githubusercontent.com/2461547/136872075-bbcf7e69-7893-41df-b45b-d5fdeb983350.mov

@heshan0131 @macrigiuseppe I'm going to merge this in, but one outstanding item remains. I'm not sure how to disable a <Slider/> from kepler.gl. Is that possible?


add OSrPRE filters to dashboard

This PR will fix #342 but still needs work. The filters are added in, but I think something is wrong in the query in the controller because all of the filters are returning 0 results for me. I'm not positive how to use the query language in the preprint.js controller starting at line 217 -- can one of y'all take a look and check me? Thanks!

Upon review, I was able to make the requested tweaks to the controller logic and the filter seems to work fine! Merging.


Added endpoint to calculate monthly and annual mortgage payments

#189  :
Issue Description:

Calculating annual or monthly amortization schedule for a mortgage loan.

Checklist:

[x] Tested the endpoint on sample data.
[x] No conflicts
[x] Have mentioned the endpoint in the ENDPOINTS.md file.

Screenshot of the changes:

--------|----------

Congrats 🎉 @enockzaake, your PR got merged. To be eligible for the swags from ClueLess. Follow the steps mentioned.
Star this repository. ⭐
And don’t forget to share your achievement with your friends.


FIX: Fix occasional error in SQLite3 PDO usage, Add SQLite + MariaDB builds, ensure PDO builds for all, clarify database versions

This PR pulls together a few interrelated changes to the databases we support and the built matrix necessary to cover them:

It fixes a couple of final things blocking SQLite from working nicely.
The build matrix now covers

MySQL (5.6 and 5.7), MariaDB (10.0 and 10.3), PostgreSQL (9.2 and 9.4), and SQLite (3.8 and 3.11)
PDO and non for each database
2 specific database versions for each database (as listed above)

The built matrix is bigger but a couple of the builds are a bit faster. The total build time goes from about 50m to about 1h20. However, the wall-clock time is similar at about 15m - 20m.

The change looks fine to me, but the PDO + SQLite build has timed out. I’ve restarted it in case it’s a one-time blip

The change looks fine to me, but the PDO + SQLite build has timed out. I’ve restarted it in case it’s a one-time blip

It timed out again, i bumped again - if it fails again we may have to investigate some kind of infinite loop??

Same timeout, same location; indicates some kind of bug here. Can you test locally with 7.3 and see what's going on?

I'll see if it relates to the changed approach to 7.3 config I introduced - I'll try reverting that and seeing if it fixes.

OK it looks like this is caused by trying to start a transaction while @runInSeparateProcess is in use.
I'm pretty confident that the underlying issue is that we're doing transactions in PDO wrong, the same issue that we bumped into in #8448, and the "real fix" would be merging that PR.
However, as a quick fix, I'm setting $usesTransactions = false on the two tests that make use of @runInSeparateProcess

Ok I've added MariaDB in the mix since it's been discussed. We now have 9 builds that cover:

PDO and non
SQLite, MariaDB, PostgreSQL, and MySQL
2 versions of each database
PHP 5.6 - 7.3
A CMS test

Seems okay, but I admit it's a lot of builds.

Pretty epic matrix there. Framework would be the most important repository to run a complex set of builds like this, there's a bunch of them but I'm OK with it.

SilverStripe\Security\Tests\MemberAuthenticator\SessionAuthenticationHandlerTest::testAuthenticateRequestStartsSessionWithSessionIdentifier seems grumpy and caused a lot of the build to fail.
Is this related to this change or is it something else?

This bug already exists in PDO + SQLite in the 4 branch, but we’re not currently testing.
The root cause is PHPunit’s runInSeparateProcess combined with transaction use.
I hate PDO, but I’m looking to see if I can avoid the need for both runInSeparateProcess and database access by refactoring the tests

@sminnee  Are you still working on this?

@sminnee  I'm currently going through pull-requests that haven't had much activity for a while to see what is still active and what now makes sense to be closed.
I don't believe Sqlite or MariaDB are supported in any other travis files, though we've recently done a bunch of PR's to all the supported modules to get PDO on MYSQL more consistently there.
Where do you see this one going?  Do you still want to do something further here or would you prefer to now close this one?

We can close this for now


Interning in SQLMetadataSegmentManager may obliterate new segment metadata

DataSegment interning in SQLMetadataSegmentManager.poll() is based on segment ids (if a segment with a matching segment id already exists on the heap, it is used instead of the newly polled segment), that may obliterate some new information about segments, such as different loadSpecs.

@leventov checking my understanding for this fix, so the equals in DataSegment should be either removed or modified to include not just the SegmentId equality, but other fields of DataSegment ?  And the replaceWithExistingSegmentIfPresent in SQLMetadataSegmentManager might return something like
return alreadyExistingSegment != null
           ? (alreadyExistingSegment.equals(segment) ? alreadyExistingSegment : segment)
           : segment;

I'm not sure what would be the best way to handle this in terms of both usability and error-proneness. If you just make equals() more expensive and somebody puts it in HashSet as keys, a lot of unnecessary work may be done in the result. On the other hand, there may be cases when we would still want to have DataSegment as keys in Maps to "pack" more data into a Map without creating extra wrapping objects, that is, having ObjToIntMap<DataSegment> instead of Map<SegmentId, SomeWrapperWithDataSegmentAndIntFields>.
So what I would do is:

Leave DataSegment's equals() and hashCode() as they are now
Add DataSegment.allDataEquals() checking all data.
Add a Structural Search inspection in intelliJ which inhibits using DataSegment in Map keys and Sets, but can still be done using @SuppressWarnings("SSBasedInspection") when somebody really needs that (see the example above).

I thought a segment ID was supposed to be immutable. Why would a new load spec not cause a new segment ID?

If you are migrating your segments from one deep storage to another, you might want to adjust things so the same segment objects have new loadSpecs but retain their IDs. It's valuable because changing the IDs implies changing the version and/or partition numbers, which would cause a lot of churn in the cluster.
Segment metadata also evolves naturally over time as segments are pushed from realtime tasks to deep storage. While they're on realtime tasks, they don't yet have loadSpecs or sizes (maybe other stuff too, I don't remember).

Add a Structural Search inspection in intelliJ which inhibits using DataSegment in Map keys and Sets, but can still be done using @SuppressWarnings("SSBasedInspection") when somebody really needs that (see the example above).

Did you mean to add this structural search inspection as a warning, if I add it, it's manifested as an error, and I'm not sure how to make it a warning while keeping the existing inspections as error. See the last item in this screenshot, how can we set it's severity to warning ?

@surekhasaharan I meant to add it as an error. So every time a developer wants to have Set<DataSegment> in their code, they must put @SuppressWarnings (with justification) to pass CI.


Topic/update terraform providers 22 05

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Considering the providers have been getting updates, it might be good to backport them to 22.05 as those are needed for a few use cases.
@zowoq, please let me know your thoughts on this.


net

Problem encountered on https://dotnet.microsoft.com/en-us/learn/aspnet/blazor-tutorial/install
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Hi @davidmartine! Can you share more information about the problem you're having?


DFI Wallet not start up

After the update from 2.3.5 to 2.4.0, the wallet no longer starts. Stop at the picture and don't go any further. What can I do? Can I completely uninstall the wallet? and then reinstall?

Error_2.4.0.zip

OKAY... you can close the issue!!! I delete the folder ...\AppData\Roaming\defi-wallet and ...\AppData\Roaming\DeFi Blockchain
Than I install Wallet 2.4.0 and now it is working...


Move Lcd out of InternalMemory

The idea of this PR is to propose a new way to manage memory and hardware components.
Before we had everything under InternalMemory but every field was just containing registers.
This is not suitable now that we approach implementing the LCD since it has also its login instead of just a bunch of registers.
Also, having everything under InternalMemory was not "semantically" correct in my opinion, hardware components are external components to the CPU.
If we agree with this type of refactoring, we would also get rid of the IORegister struct which is useless at this point. I added it many months ago to try to handle read/write access through it but then we saw that it was not possible.
To manage read/write access we could do something like this:

If the register is R/W, just make the field pub
If the register is R, keep the field private and write a getter for the field
If the register is W, just make the field pub

I also changed a bit the read/write functions in Bus.
Now we have write_raw and read_raw that map every address to the correct component and we have wrappers like read_word, write_word, read_half_word, etc... that should calculate how many wait cycles the CPU has to be stopped (depending on wait cycles and bus width of the component).

LGTM, left a couple of comments... Also we should consider the trait pub trait IoDevice { Is this still useful? Or should we change it?

Thanks, I think that once we move to this new approach the IoDevice trait will be unused


Activating extension 'yzhang.markdown-all-in-one' failed: Cannot find module './types/bmp'

What is the problem?
The plugin cannot be activated due to the following error:

How can I reproduce it?
The error happens on VSCode start/reload.
Is there any error message in the console?
See screenshot above.

I guess some of the extension files are missing. You can try to uninstall it, close/restart vscode (not reload) and then reinstall this extension.

Reinstalling helped and I guess the error was caused on my end.
I'm syncing all VSCode settings & plugins via Dropbox & had some file conflicts with upper/lower case names lately. So I guess those conflicts caused the missing files.


Add unmanaged binaries to the desktop convenience project

It avoids need to manually copy them over.

:+1:


A New Chart Using Hooks

Re-wrote the chart to use post-install/post-upgrade hook to install the custom resource.   Huge thanks to @ejether for writing the actual hook that I blatantly stole from his branch.
Renamed some variables to match helm best practices.
Removed the need for running the chart twice, and removed that node from the readme
Removed the sub-charts as they were totally unnecessary.
Tested on initial install and on subsequent upgrades
Tested with autohelm
TODO: Test on existing install to make sure it doesn't break them (it should)

I installed using the current master chart, created an rbacdefinition manually, then  installed the new chart with an identical user in the chart config.  Both kubeconfigs generated were identical.  This means that we should be safe to merge this without breaking existing implementations.

TODO:
Move post-install hook to use separate service account.

I like the idea, but don't consider it a blocker.

@robscott Per your concerns I moved the post-hook job to run using it's own service account that only has access to rbacdefinitions.  Tested and working.


Implementation Process - Caret Pipeline Models

Caret allow building complex models with a post-processing step and a classification/regression (a la scikit-learn pipelines). Pipeline elements are already implemented in #2.
Should be easy as scikit-learn pipelines are already supported out of the box.
model  <- train(Species ~ ., data = iris, method = "rpart" , preProcess = c("center", "scale", "pca"))

Added a caret classification pipeline (rpart) with preprocessing (center + scale + pca)
Added a caret regression pipeline (rpart) with preprocessing (center + scale + pca)


Add instructions to run unit tests from a specific file

This adds more documentation for those that want to run specific unit tests in a specific file.

Pull Request Test Coverage Report for Build 640640930

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 34.613%

Totals

Change from base Build 138830812:
0.002%

Covered Lines:
15290

Relevant Lines:
39875

💛  - Coveralls

I'm closing this for now to clean up stale PRs, please reopen with the suggested updates if you still want to do this!


📚 Update the docs

Add new supported models
Update the project labels

Codecov Report

Merging #17 (8e9c820) into master (3742080) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #17   +/-   ##
=======================================
  Coverage   87.09%   87.09%
=======================================
  Files           4        4
  Lines          93       93
  Branches       10       10
=======================================
  Hits           81       81
  Misses          9        9
  Partials        3        3

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 3742080...8e9c820. Read the comment docs.


Try to create category in windows 10 getting exception

Bug report
Title
unable to create a category in windows system.
Issue Description
php version 7.3.1
os windows 10
bagsito master
Steps to reproduce
install bagisto from master
1. login in admin panel
2. try to create a category

Expected result
category should be created.
Actual result

Tell us what happens instead.

getting exception

Hi @john7851
refer this issue for same
https://github.com/bagisto/bagisto/issues/2562

Create db with  utf8_general_ci collation

Feel free to reopen this, if you're still facing issue.
Thanks!


Fixed one code snippet in README.md.

Why make y an INT32?  because imo implicit conversion would promote an int value to float anyway.


TypeError: multinomial() missing 1 required positional arguments: "num_samples"

There is a small problem with the model.py file. Python reported an error when it goes to line 297, where there is a missing argument for the multinomial function.

multinomial() missing 1 required positional arguments: "num_samples"
how to solve the issue

while running the AI Car program

multinomial() missing 1 required positional arguments: "num_samples"
how to solve the issue

you can pass num_samples=1 as parameter, hopefully it will work fine.


Config not shown in help and hard to understand .lsfprc

Describe the bug
The config options mentioned in issue #18 and added in commit 162f45ecf774d04ec57438616aa277c71e1a643d do not appear in the configuration, making it impossible to use if you haven't dived into the issues or commits of this repository, as they are not mentioned in other any place either.
Another issue related to this is the .lsfprc file. The file is truly hard to understand, and it is basically impossible to write by hand. It does not follow any specific convention and repeats the - config:  part twice, apart from the fact that all .*rc files are for config, so there is really no reason to put that part, as far as I understand not looking at the code.
To Reproduce
If you are interested in seeing how the flag does not appear in help, just open help by running lsfp --help or lsfp -h.
For checking the .lsfprc file syntax, run lsfp --config-git=false and lsfp --config-color=false and open the .lsfprc file. You should see the syntax and repetition mentioned above.
Expected behavior
Regarding the help message, simply adding it to help, either as --config-<feature>=<true/false> [explanatory text[ or each one individually.
About the .lsfprc file, I would recommend following some convention, for example JSON, but it may be a bit too overkilling for just two lines at the maximum. Therefore, I just recommend putting it as git=false and same for color, as it is simple enough to understand for everyone and can easily be hand-written.

I'll throw in my (somewhat) fancy config parser from another project that uses the abc=def.

Fixed in 047fb4bd83319ff3af2295ec460c8b02be29ec49 and 988527ffe22077326bced25f4a6ff43032199743.


[10.x] Fix custom themes not reseting on Markdown renderer

This fixes the issue described in #46179. It should reset the theme to 'default' on the Markdown renderer object if no $theme is set on the Mailable object. I've also added a test for this. This PR does not introduce any breaking changes.
There's a small inconsistency between Mailable and Markdown classes, the Mailable class has a default $theme value of null (implicitly the default theme), however Markdown has a default theme value of 'default' (here and here). But fixing this inconsistency might be a breaking change, so not sure if you'd want to fix it.

Seems like it's setting the theme always to default unless it's explicitly defined in the Mailable class thus making the markdown.theme option in config/mail.php configuration file being ignored.

@AEM5299 we'll need to fix that. Can you provide a follow up PR?

Opened PR #46317.

We've decided to revert this PR for now.


Fail to compile although succefully installed

Hello
I just saw a live youtube video on Makie and wante to try it out with
Julia 1.4, MacOS 10.14.6
I could install the package without problem but when I tried to compile the package with using Makie, I got the following:
[ Info: Precompiling Makie [ee78f7c6-11fb-53f2-987a-cfe4a2b5a57a]
WARNING: Method definition convert(Type{#s59} where #s59<:(Observables.Observable{T} where T), Any) in module Observables at /Users/mymac/.julia/packages/Observables/0wrF6/src/Observables.jl:52 overwritten in module AbstractPlotting at /Users/mymac/.julia/packages/AbstractPlotting/B5nOF/src/types.jl:288.
  ** incremental compilation may be fatally broken for this module **

WARNING: Method definition copy(Observables.Observable{T}) where {T} in module Observables at /Users/mymac/.julia/packages/Observables/0wrF6/src/Observables.jl:30 overwritten in module AbstractPlotting at /Users/mymac/.julia/packages/AbstractPlotting/B5nOF/src/interaction/nodes.jl:15.
  ** incremental compilation may be fatally broken for this module **

ERROR: LoadError: LoadError: MethodError: convert(::Type{Observables.Observable}, ::Observables.Observable{Any}) is ambiguous. Candidates:
  convert(::Type{Observables.Observable}, x::Observables.AbstractObservable) in Observables at /Users/mymac/.julia/packages/Observables/0wrF6/src/Observables.jl:41
  convert(::Type{T}, x::T) where T<:Observables.Observable in AbstractPlotting at /Users/mymac/.julia/packages/AbstractPlotting/B5nOF/src/types.jl:289
Possible fix, define
  convert(::Type{Observables.Observable}, ::Observables.Observable)
Stacktrace:
 [1] setindex!(::Dict{Symbol,Observables.Observable}, ::Observables.Observable{Any}, ::Symbol) at ./dict.jl:380
 [2] Dict{Symbol,Observables.Observable}(::Base.Generator{Base.Iterators.Pairs{Symbol,Array{T,1} where T,NTuple{4,Symbol},NamedTuple{(:color, :marker, :linestyle, :side),Tuple{Array{ColorTypes.RGB{Float64},1},Array{Symbol,1},Array{Union{Nothing, Symbol},1},Array{Symbol,1}}}},typeof(AbstractPlotting.node_pairs)}) at ./dict.jl:103
 [3] #Attributes#4 at /Users/mymac/.julia/packages/AbstractPlotting/B5nOF/src/types.jl:292 [inlined]
 [4] top-level scope at /Users/mymac/.julia/packages/AbstractPlotting/B5nOF/src/theming.jl:16
 [5] include(::Module, ::String) at ./Base.jl:377
 [6] include(::String) at /Users/mymac/.julia/packages/AbstractPlotting/B5nOF/src/AbstractPlotting.jl:1
 [7] top-level scope at /Users/mymac/.julia/packages/AbstractPlotting/B5nOF/src/AbstractPlotting.jl:37
 [8] include(::Module, ::String) at ./Base.jl:377
 [9] top-level scope at none:2
 [10] eval at ./boot.jl:331 [inlined]
 [11] eval(::Expr) at ./client.jl:449
 [12] top-level scope at ./none:3
in expression starting at /Users/mymac/.julia/packages/AbstractPlotting/B5nOF/src/theming.jl:16
in expression starting at /Users/mymac/.julia/packages/AbstractPlotting/B5nOF/src/AbstractPlotting.jl:37
ERROR: LoadError: Failed to precompile AbstractPlotting [537997a7-5e4e-5d89-9595-2241ea00577e] to /Users/mymac/.julia/compiled/v1.4/AbstractPlotting/6fydZ_ttyve.ji.
Stacktrace:
 [1] error(::String) at ./error.jl:33
 [2] compilecache(::Base.PkgId, ::String) at ./loading.jl:1272
 [3] _require(::Base.PkgId) at ./loading.jl:1029
 [4] require(::Base.PkgId) at ./loading.jl:927
 [5] require(::Module, ::Symbol) at ./loading.jl:922
 [6] include(::Module, ::String) at ./Base.jl:377
 [7] top-level scope at none:2
 [8] eval at ./boot.jl:331 [inlined]
 [9] eval(::Expr) at ./client.jl:449
 [10] top-level scope at ./none:3
in expression starting at /Users/mymac/.julia/packages/Makie/3CKlT/src/Makie.jl:3
ERROR: Failed to precompile Makie [ee78f7c6-11fb-53f2-987a-cfe4a2b5a57a] to /Users/mymac/.julia/compiled/v1.4/Makie/iZ1Bl_ttyve.ji.
Stacktrace:
 [1] error(::String) at ./error.jl:33
 [2] compilecache(::Base.PkgId, ::String) at ./loading.jl:1272
 [3] _require(::Base.PkgId) at ./loading.jl:1029
 [4] require(::Base.PkgId) at ./loading.jl:927
 [5] require(::Module, ::Symbol) at ./loading.jl:922

Do you know what I could try to fix the problem?
Many thanks in advance!

Any info on how you installed it?
This looks like something went wrong when installing it!

Many thanks for your quick answer @SimonDanisch
Here is the output for the installing step:
Pkg.add("Makie")
  Resolving package versions...
   Updating `~/.julia/environments/v1.4/Project.toml`
 [no changes]
   Updating `~/.julia/environments/v1.4/Manifest.toml`
 [no changes]

what about:
Pkg.add("AbstractPlotting")
Pkg.add("GLMakie")

julia> Pkg.add("AbstractPlotting")
  Resolving package versions...
   Updating `~/.julia/environments/v1.4/Project.toml`
 [no changes]
   Updating `~/.julia/environments/v1.4/Manifest.toml`
 [no changes]

julia> Pkg.add("GLMakie")
  Resolving package versions...
   Updating `~/.julia/environments/v1.4/Project.toml`
 [no changes]
   Updating `~/.julia/environments/v1.4/Manifest.toml`
 [no changes]

What versions of GLMakie, AbstractPlotting and Observables do you have installed?

julia> Pkg.status("GLMakie")
Status `~/.julia/environments/v1.4/Project.toml`
  [e9467ef8] GLMakie v0.0.8

julia> Pkg.status("AbstractPlotting")
Status `~/.julia/environments/v1.4/Project.toml`
  [537997a7] AbstractPlotting v0.9.10

julia> Pkg.status("Makie")
Status `~/.julia/environments/v1.4/Project.toml`
  [ee78f7c6] Makie v0.9.5

These are super old versions, with the newest version of Observables, which likely doesn't match. I'm guessing, that you somehow got downgraded to very old versions, where I still didn't have any version bounds... Try to at least get<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

many thanks for your help @SimonDanisch. That solved the problem indeed!

I have added GLMakie#master still facing similar issues. During CI/CD on gitlab it fails to precompile. Please help.
Link to the pipeline
Astrodynamics is the main package containing GLMakie.

Yes, that's because you don't have OpenGL installed correctly.
Use CairoMakie, WGLMakie or a CI setup like this:
https://github.com/JuliaPlots/GLMakie.jl/blob/master/.github/workflows/ci.yml#L41
https://github.com/JuliaPlots/GLMakie.jl#troubleshooting-opengl

Yes, that's because you don't have OpenGL installed correctly.
Use CairoMakie, WGLMakie or a CI setup like this:
https://github.com/JuliaPlots/GLMakie.jl/blob/master/.github/workflows/ci.yml#L41
https://github.com/JuliaPlots/GLMakie.jl#troubleshooting-opengl

Thanks a lot.
I tried GLMakie CI setup but it didn't resolve the issue. Maybe some mistake from my side only.
Although using WGLMakie I am able to get the same work done.


NOTICK Set valid group policy in CPI upload for smoke tests

Will be looking at this when adding schema validation for group policy on upload so will revisit then. In the meantime, this PR introduces a change to add a proper group policy to the smoke tests https://github.com/corda/corda-runtime-os/pull/1658

Closing.


Data Export: Supported table names don't match that in the Portal

Hi -

Many of the names of supported table do not match what Azure Portal displays. This causes confusion.
If the table is not yet supported, it should be greyed out in the Portal. For example, the AzureActivity table is not supported, but I am able to preview its data. This also causes a great deal of confusion

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

ID: 5daf3365-76b8-b98f-7fc8-51f520e6276b
Version Independent ID: 93a216a6-fd66-cc0a-9346-58955bd0a7ba
Content: Log Analytics workspace data export in Azure Monitor - Azure Monitor
Content Source: articles/azure-monitor/logs/logs-data-export.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @yossi-y
Microsoft Alias: yossiy

Thanks for bringing this to our attention. We will investigate it further and update you shortly.

Thanks for you feedback.
Your suggestion makes sense and would be ideal to simplify configuration.
We considered this implementation initially, but had to exclude it eventually due to priority constraints, and since it's needed temporarily until all tables are supported. We hoped that adding the comment in UI: "Not all tables are supported in data export currently, see a list of supported tables", would give you all the information, although we realize that in suboptimal manner.
Thanks,


Error running SQFlint

Hey, I've been trying to get this working for a while now. Installed it just as the instructions say, but I keep getting an error from it. When I try to save an .sqf file, it says "[Linter] Error Running SQFLint.
This is what I found in the developer tools:
[Linter] Error running SQFlint Error: Traceback (most recent call last):   File "c:\users\niko\appdata\local\programs\python\python37-32\lib\runpy.py", line 193, in _run_module_as_main     "__main__", mod_spec)   File "c:\users\niko\appdata\local\programs\python\python37-32\lib\runpy.py", line 85, in _run_code     exec(code, run_globals)   File "C:\Users\Niko\AppData\Local\Programs\Python\Python37-32\Scripts\sqflint.exe\__main__.py", line 5, in <module> ModuleNotFoundError: No module named 'sqflint'     at ChildProcess.<anonymous> (C:\Users\Niko\.atom\packages\linter-sqf\node_modules\sb-exec\lib\index.js:56)     at emitTwo (events.js:126)     at ChildProcess.emit (events.js:214)     at maybeClose (internal/child_process.js:925)     at Process.ChildProcess._handle.onexit (internal/child_process.js:209)
Any ideas? Would love to get this working.

Could you describe exactly what you did? (e.g. installed sqf via pip or downloaded from github? which commands you used to install sqflint?)
Windows or unix?
Can you try to run sqflint in the powershell?

It seems that the the script sqflint.exe is not finding sqflint. This is odd and hints at an installation problem.


When branch name contains # its throwing 404 from UI on clicking NewPullRequest button.

Fixes: #25830
Before
We use issue reference as branch name and when I pushed my branch, gitea suggested to create new pr, however "#" in the branch name breaks this feature !

After

Clicking on New Pull Request

This issue may be required some other way to fix it.
As per my observation on github.

See the url its only encoded once on clicking on Compare and pull request

I need some feedback on this approach.

Duplicate, hence closing.


fix(label_view): Convert mixed strings to template literals

A few small fixes for some strings and variables.

Closing and moving to /develop


Feature/bb

config-validator changes

Please can you rebase and integrate into the following structure: https://github.com/google/devops-governance/tree/GDC-phase-kickstarter-1/examples/guardrails/bitbucket (We have specifically created a bitbucket folder.
This branch has conflicts that must be resolved
to resolve conflicts before continuing.
Conflicting files
examples/guardrails/project-factory/main.tf
examples/guardrails/project-factory/modules/project_plus/main.tf
examples/guardrails/project-factory/outputs.tf
examples/guardrails/project-factory/variables.tf
examples/guardrails/project-factory/wif.tf
examples/guardrails/skunkworks/main.tf
examples/guardrails/skunkworks/variables.tf

https://github.com/google/devops-governance/pull/42/files , closing as duplicate


Project logo

Just wondering if the jingo project has a logo/icon. There is a public/favicon.ico. Is there a larger version of this?

Yes! Yes! Yes!
This is something definitely and quite urgently missing. I am not a designer myself and I also have not really an idea what kind of "message" to convey with the logo :)
Any idea is greatly appreciated

@claudioc For now, I put a stock icon (https://www.iconfinder.com/icons/173727/wiki_icon#size=128) since I wanted to put your app out there for people to try. As soon as you have an icon, I will be happy to update the app.

I would much more like to be able to put my own logo somewhere to the wiki page and as the favicon (ok that might be possible). I like the decend design very much, but it would be easier and nicer to use the wiki If I could add a logo to the title :)

Added in 1.8.2


Add VDP_compactVRAM() / VDP_setupVRAM() method to allocate maps / sat data depending plan size.

It would be nice to have a method as VDP_compactVRAM() or VDP_setupVRAM() which automatically set the VRAM maps address (PLAN_A, PLAN_B and WINDOW), Sprite Table and HScroll Table addresses depenginto allocate maps / sat data depending the current plan size information.

Done using the VDP_setPlaneSize(..) method.


Add support for ipsidekick.com geocoding service

Thanks for this! The tests are failing, however, primarily because of some missing JSON responses. Could you take a look at that? For recent examples, please see #1223 and #1193.

Ah. My bad for forgetting about the tests.
Question though. The geocoding service doesn't support lat-long (yet. I'm the developer). But hear me out, this is a sample output:
{
  "ip":"<IP_ADDRESS>",
  "country": {
    "code":"US",
    "name":"United States"
  },
  "currency" : {
    "code" : "USD",
    "decimals" : 2,
    "name" : "US dollar"
   },
  "timezone": {
    "gmtOffset":"GMT-7:00",
    "name":"America/Los_Angeles"
  },
  "holiday":"Independence Day"
}

If you'd rather all geocoders that work with the gem produce a lat-long output, I can understand that. Otherwise, I can have create a separate PR to discuss how to modify the test to reflect it not support all the fields in Ah. My bad for forgetting about the tests.
Question though. The geocoding service doesn't support lat-long (yet. I'm the developer). But hear me out, this is a sample output:
{
  "ip":"<IP_ADDRESS>",
  "country": {
    "code":"US",
    "name":"United States"
  },
  "currency" : {
    "code" : "USD",
    "decimals" : 2,
    "name" : "US dollar"
   },
  "timezone": {
    "gmtOffset":"GMT-7:00",
    "name":"America/Los_Angeles"
  },
  "holiday":"Independence Day"
}

If you'd rather all geocoders that work with the gem produce a lat-long output, I can understand that. Otherwise, I can have create a separate PR to discuss how to modify the test to reflect it not support all the fields in https://github.com/alexreisner/geocoder/blob/master/test/unit/result_test.rb#L99

Hmm...interesting question. On one hand, none of the supported IP geolocation services produce really meaningful lat/lon coordinates because, at best, they're returning the location of the ISP, not the user's device (and often they only return the coords at the center of the user's country). However, developers using Geocoder expect to get lat/lon (often for purposes of doing distance calculations) so a lack of coordinates may be confusing (and could be the cause of a "bad experience" with your API). Do you have plans to add lat/lon coordinates in the future?

Do you have plans to add lat/lon coordinates in the future?

No definite plans to. I have some data, but like you say, they are often not accurate enough or become outdated quickly. Product-wise, I'm growing capabilities by attaching additional types of data — now currency, timeZone, holidays, hopefully more sooner.
It returns the country now which I know people have some use for it, but it's your call :)

So, after thinking about this, I don't think it's a good idea to add a service that breaks such a fundamental expectation. If you add lat/lon in the future (even with the low degree of precision offered by other APIs), feel free to re-open this PR.