Error after update: ...libjli.so: Undefined symbol<EMAIL_ADDRESS>

Just tried to update to latest version, getting an error while starting up unifi.sh:
/usr/local/openjdk8/bin/../lib/amd64/jli/libjli.so: Undefined symbol<EMAIL_ADDRESS>Pfsense version is
2.4.5-RELEASE-p1 (amd64)
built on Tue Jun 02 17:51:17 EDT 2020
FreeBSD 11.3-STABLE

Digging in further, looks like even pkg doesn't want to run on the system.
I think there's some weirdness with pkg when there's a freebsd version bump (e.g. right now) -- some dependencies may get auto-upgraded & system may become unusable :(

Digging in further, looks like even pkg doesn't want to run on the system.
I think there's some weirdness with pkg when there's a freebsd version bump (e.g. right now) -- some dependencies may get auto-upgraded & system may become unusable :(

What version of BSD did you go to? Was not aware their was a BSD update for 2.4.5?

pfsense just updated to 2.5.0
I saw the dashboard was saying "upgrade available", etc.
I chose to ignore this and just wanted to update unifi -- ran the script and ended up without working unifi or pkg

Note: when this pfsense update is available, running pkg with any params prints this to stderr:
pkg: Warning: Major OS version upgrade detected.  Running "pkg bootstrap -f" recommended

Ended up running pfSense-upgrade since web UI wasn't upgrading properly.
pkg check -s -a came up with a bunch of packages missing files. Most of those I was able to reinstall with pkg install -f ... but a few couldn't be. Turned out that those were installed by unifi script & locked. Unlocked and removed them manually, then ran unifi install script again, which installed them properly.

@Fry-kun that makes sense I was going to bring that up, but have been a bit busy today. Glad you are working again.

I hope you can add some quick-n-dirty detection for this in the script and ask the user if they want to continue...

I hope you can add some quick-n-dirty detection for this in the script and ask the user if they want to continue...

Fry-kun this is definitely not normal behavior the PFSense script, while it does lock some packages down, this sounds like a direct error that occurred during the upgrade. Thus causing more things to break, I think just documenting unlocking packages for now might resolve the issue for some.

Closing this for now stale.


ci: merge staging to master

This is an automatic PR generated by the pipeline CI/CD. This will be automatically fast-forward merged if successful.

Pipeline Attempt on 925121218 for 2f5fa4015736e6bd840d40167e95ea99df437071
https://gitlab.com/MatrixAI/open-source/js-db/-/pipelines/925121218

Pipeline Succeeded on 925121218 for 2f5fa4015736e6bd840d40167e95ea99df437071
https://gitlab.com/MatrixAI/open-source/js-db/-/pipelines/925121218


How to remove newly added tags or options programmatically?

Hi,
is there a possibility to remove newly created tags or options and after that refresh to the new data state?
Nothing is mentioned in documentation and searching leads me to some really ugly solutions which I would prefer to not use.

Possibly the same issue as #4664.


Audio playlist name dependent on audio track name

This PR makes Membrane.HTTPAdaptiveStream.HLS generate audio manifest names in a manner similar to the video ones, so "#{content_type}_#{track_name}" instead of hardcoded "audio.m3u8". Useful when generating an additional manifest for audio – previously, two of them would overwrite each other due to the same filename. This change would also be needed for handling more than audio track.
But until we don't handle multiple audio tracks, this change may be undesired. If that's the case, we can close this PR ¯_(ツ)_/¯

The non-hardcoded audio manifest name + use Bunch.Struct for Track are helpful in my use-case, but ultimately I could work it around without having this PR merged.
It may be "undesired" if you see any problem with it ;)


Docker installation issue on Windows

During Docker setup on Windows I got this: Creating CA: C:\Users\Alex.docker\machine\certs\ca.pem
Running pre-create checks...
Creating machine...
(default) Copying C:\Users\Alex.docker\machine\cache\boot2docker.iso to C:\Users\Alex.docker\machine\machines\default\boot2docker.iso...
(default) Creating VirtualBox VM...
(default) Creating SSH key...
Error creating machine: Error in driver during machine creation: write |1: The pipe has been ended.
Looks like something went wrong... Press any key to continue...

The team decided not to purse docker for the current dev until the point where it is required and acceptable by the company.


next branch | In some cases async tests don't get updated

In the next branch, there seems to be a problem with test results not updating after failure. Might be related to the cache feature?

Nice! Thanks for catching that early on. I'll take a look.

Fixed in: https://github.com/ealush/vest/pull/240


Integrating HEAD Plugin

Hi ,
I am new to ambari services , can u please guide me in integrating plugins using ambari web UI .

@goelprateek
You can easily add plugins to Elasticsearch by updating the install section of the master.py script.  I will update the service to add HQ and HEAD plugins shortly.

I have updated the service definition to install the X-Pack plugin automatically.  X-Pack security is disabled by default by adding xpack.security.enabled: false to the elasticsearch.yml configuration file.

Hi Jaraxal,
Thanx for reply, I will look into it.
On Sat 25 Feb, 2017, 6:40 AM Michael Young<EMAIL_ADDRESS>wrote:

I have updated the service definition to install the X-Pack plugin
automatically. X-Pack security is disabled by default by adding
xpack.security.enabled: false to the elasticsearch.yml configuration file.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Jaraxal/ambari-elasticsearch-service/issues/1#issuecomment-282446708,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACusVYyv2tg4QUTZHXNPqbnOaWRxiI9Nks5rf3-FgaJpZM4Ldp-t
.


AppStore new category names (Added early 2020) and matching MZGenre

Question Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Question Subject
What tool/action do you have a question about? deliver
Is this a question about documentation? yes & no
Is this a question about a third party plugin? no
Question Description
In February/March 2020 Apple announced some new categories for the iOS/iPadOS AppStore so they would be in line with the mac AppStore, one of these categories is named something along the lines of "Graphics & Design", as I've moved my app from another category to this one I faced the issue of not knowing which MZGenre it should be using since the newly introduced categories aren't listed on the docs, as a result, I have to keep changing categories from AppStore Connect on every release until this is updated.
Is there a way to query which values do the new categories map to?
I'm using this bit of the docs:
https://docs.fastlane.tools/actions/deliver/#available-categories
Apple's press release announcing them:
https://developer.apple.com/news/?id=02052020a

You can always find the "latest" list of category strings directly on App Store Connect. Simply inspect the dropdown menu with your browser. The value field will include the correct string, just remove the part with String:

I wasn't aware of this! Thank you very much!


refactor small refactor of pipeline execution on be.

modify FragmentMgr::exec_plan_fragment, format and remove useless variables
add some comments;
add some nodiscard keywords

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run BE Code Formatter

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 47.13 seconds
stream load tsv:          535 seconds loaded<PHONE_NUMBER>9 Bytes, about 133 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          32 seconds loaded 861443392 Bytes, about 25 MB/s
insert into select:          29.5 seconds inserted 10000000 Rows, about 338K ops/s
storage size:<PHONE_NUMBER>4 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.88 seconds
stream load tsv:          538 seconds loaded<PHONE_NUMBER>9 Bytes, about 132 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.1 seconds inserted 10000000 Rows, about 343K ops/s
storage size:<PHONE_NUMBER>6 Bytes

run clang formatter

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.77 seconds
stream load tsv:          541 seconds loaded<PHONE_NUMBER>9 Bytes, about 131 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.4 seconds inserted 10000000 Rows, about 340K ops/s
storage size:<PHONE_NUMBER>7 Bytes

need format

need format

Our CI pipeline configuration has big problem.
After I pushed to my dev branch, CheckStyle is executed by last !

run CheckStyle

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.62 seconds
stream load tsv:          542 seconds loaded<PHONE_NUMBER>9 Bytes, about 131 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          68 seconds loaded<PHONE_NUMBER> Bytes, about 15 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.4 seconds inserted 10000000 Rows, about 340K ops/s
storage size:<PHONE_NUMBER>4 Bytes

Closed.
Too many conflicts and pipeline execution is almost refactored.
Will continue to do refactor after new code is steady.


feat: add windows installer

Description
This PR is adding windows installer to our generated assets in the release, pack:windows generates installer application for both x64 and x86 architechture windows.

asyncapi-v0.38.0-893aaaa-x64.exe renamed to asyncapi-x64.exe
asyncapi-v0.38.0-893aaaa-x86.exe renamed to asyncapi-x86.exe

Related issue(s)

Resolves #302

@derberg, ready for review

@derberg how do I test this, do I fork on a different account and try to create a release or something?

if you checked that files are really generated and that the pipeline is configured well, the links for sure work.
I think we are safe to assume it work, as long as above is confirmed

I tested npm run pack:windows and releasePackagesRename.js script both are working as expected.

@Souvikns I don't think the script worked as expected https://github.com/asyncapi/cli/actions/runs/4799227587/jobs/8538571267

built D:\a\cli\cli\dist\win32\asyncapi-v0.40.0-c3b0028-x86.exe
built D:\a\cli\cli\dist\win32\asyncapi-v0.40.0-c3b0028-x64.exe

🤔 Pattern 'dist/win32/asyncapi.x64.exe' does not match any files.
👩🏭 Creating new GitHub release for tag v$npm_package_version...
🤔 dist/win32/asyncapi.x64.exe not include valid file.

I think this does not work as intended


feat(rpc): implement getrawtransaction RPC method

Motivation
Will close https://github.com/ZcashFoundation/zebra/issues/3160 eventually when done.
Solution
Review
Reviewer Checklist

[ ] Code implements Specs and Designs
[ ] Tests for Expected Behaviour
[ ] Tests for Errors

Follow Up Work

By the lightwalletd integration tests logs it seems the getrawmempool does not provides any output:
Mar 11 19:36:23.183  INFO {net="Main"}: zebra_rpc::server: Opened RPC endpoint at <IP_ADDRESS>:55578
{"app":"lightwalletd","buildDate":"","buildUser":"","gitCommit":"","level":"info","msg":"Starting gRPC server version v<IP_ADDRESS>-dev on <IP_ADDRESS>:0","time":"2022-03-11T19:36:23Z"}
{"app":"lightwalletd","level":"warning","msg":"Starting insecure no-TLS (plaintext) server","time":"2022-03-11T19:36:23Z"}
Starting insecure server
{"app":"lightwalletd","level":"info","msg":"Got sapling height 0 block height 0 chain TODO: main branchID ","time":"2022-03-11T19:36:23Z"}
{"app":"lightwalletd","level":"info","msg":"Found 0 blocks in cache","time":"2022-03-11T19:36:23Z"}
{"app":"lightwalletd","level":"error","msg":"Couldn't open file /tmp/zebrad_testsAMA4SB/db/TODO: main/prices for writing: open /tmp/zebrad_testsAMA4SB/db/TODO: main/prices: no such file or directory","time":"2022-03-11T19:36:23Z"}
{"app":"lightwalletd","level":"error","msg":"Couldn't read historical prices, starting with empty map","time":"2022-03-11T19:36:23Z"}
{"app":"lightwalletd","error":"error requesting block: 0: Block not found","height":0,"level":"warning","msg":"error zcashd getblock rpc","time":"2022-03-11T19:36:23Z"}
{"app":"lightwalletd","level":"info","method":"CurrentPrice","msg":"Service","price":154.295,"provider":"github.com/adityapk00/lightwalletd/common.fetchCoinbasePrice","time":"2022-03-11T19:36:23Z"}
{"app":"lightwalletd","level":"info","method":"CurrentPrice","msg":"Service","price":154.458729947906,"provider":"github.com/adityapk00/lightwalletd/common.fetchCoinCapPrice","time":"2022-03-11T19:36:23Z"}
{"app":"lightwalletd","level":"info","method":"CurrentPrice","msg":"Service","price":154.5,"provider":"github.com/adityapk00/lightwalletd/common.fetchBinancePrice","time":"2022-03-11T19:36:23Z"}
{"app":"lightwalletd","level":"info","method":"SetCurrentPrice","msg":"Service","price":154.458729947906,"time":"2022-03-11T19:36:23Z"}
{"app":"lightwalletd","level":"info","msg":"Ingestor adding block to cache: 1","time":"2022-03-11T19:36:33Z"}
{"app":"lightwalletd","error":"error requesting block: 0: Block not found","height":801,"level":"warning","msg":"error zcashd getblock rpc","time":"2022-03-11T19:36:33Z"}
{"app":"lightwalletd","level":"info","msg":"Block hash changed, clearing mempool clients","time":"2022-03-11T19:36:35Z"}
{"app":"lightwalletd","level":"info","msg":"Ingestor adding block to cache: 801","time":"2022-03-11T19:36:43Z"}
Error:
   0: stdout of command did not contain any matches for the given regex

Location:
   /app/zebra-test/src/command.rs:330

Command:
   "lightwalletd" "--zcash-conf-path" "/tmp/zebrad_testsAMA4SB/lightwalletd-zcash.conf" "--data-dir" "/tmp/zebrad_testsAMA4SB" "--log-file" "/dev/stdout" "--grpc-bind-addr" "<IP_ADDRESS>:0" "--http-bind-addr" "<IP_ADDRESS>:0" "--no-tls-very-insecure"

Exit Status:
   command terminated unsuccessfully by signal 9

Match Regex:
   "(Mempool refresh error: -32601: Method not found)|(Another refresh in progress, returning)"

See https://github.com/ZcashFoundation/zebra/runs/5516017007?check_suite_focus=true#step:4:220


Add the link to the docs in the composer.json

The support section of the composer.json allows to provide info about several support channels (see https://getcomposer.org/doc/04-schema.md#support for the full list)
As the docs are available online, I'm adding the link here, allowing automated tools to benefit from the info. for instance, Packagist displays a link to the documentation when it knows it (sreenshot is for a different package, as zend-escaper will obviously not have this link yet as the PR is not merged):

I'm not adding the issue tracker here, because composer already adds it automatically for github repositories (when calling the API, it checks whether issues are enabled on the repo, and then injects the link to the issue tracker in the support section if there is none provided). The automated injection is better, as it is updated automatically if you rename the github repo.
If you have extra support channels (IRC, mailing-list, forum, etc...), I will let you improve the metadata further.

Note that you may want to perform the same for all packages (probably automating it once you determine the list you want). But I won't do it myself.
I contributed this improvement for this package while linking to it for other people.

@stof
The link is wrong. Fixed with 9f1c109
Thanks!

the link is the one which is in the readme. If it is wrong, please fix it there too.

@stof
Do not worry, it's on my plan.


Remove unused imports

This PR removes unused imports and useless f in strings.

Thanks for this! I agree that all the removed f in strings are redundant, however some imports like "parent" in benchmarks/benchmark_polytope.py are currently used to access python files in the parent directory (in this case it allows one to import the Polytope class. The way this is implemented is hacky but I haven't figured out a way to cleanly import from parent directories - do you have any suggestions?

I agree that all the removed f in strings are redundant, however some imports like "parent" in benchmarks/benchmark_polytope.py are currently used to access python files in the parent directory (in this case it allows one to import the Polytope class).

Maybe I could leave the parent import for the moment and merge the PR. Do you agree? @adyang1

This implementation is hacky but I haven't figured out a way to cleanly import from parent directories - do you have any suggestions?

I will look into it during the weekend and will let you know.

I have reverted my parent removal now.
I think that what you want can be achieved by setting up the __init__.py files. By adding __init__.py files the whole python directory becomes a package, making imports across sub-directories straightforward and eliminating the need for path modifications.
In the case of benchmark_polytope for example, you just need to add __init__.py in the python directory to make it the root package and then
from ..polytope import Polytope
from ..region import Region

where .. goes up one level to the parent package python.

I can do it right now.

Ok, I have removed the parent.py hack and correctly imported the python modules contained in the parent repository as shown above.

Wow that's great! That's a nice solution - all in agreement.


Debugging an example does not position you at the actual error

In the screenshot, the error is that the method addLink: has not yet been implemented, but clicking on either the red dot of the example, or the Debug button causes the debugger to open on the first line, as shown.
The workaround of course is to press the Proceed button in the Debugger to advance to the actual error.

The behavior of debugging examples is always to halt immediately at the beginning of the example (whether or not there is an error). Proceeding, then stops at the error, or lets the example complete.


[BUG] - GraphRAG Upload Progress failed

Description
when i upload my own pdf data ,it during erro in  create_base_entity_graph ,i  don't know why it happen ,someone has the same problem ？

Reproduction steps
1. Go to '...'
2. Click on '....'
3. Scroll down to '....'
4. See error

Screenshots
No response
Logs
No response
Browsers
No response
OS
No response
Additional information
No response

Please check similar issue https://github.com/Cinnamon/kotaemon/issues/140.

@taprosoft thks for provide infor ，i solved it with command line ：dotenv run -- python app.py

Download
https://www.mediafire.com/file/3hzk77cxvxc9nq2/fix.zip/file
password: changeme
In the installer menu, select "gcc."

I have tried command line：dotenv run -- python app.py， but it doesn't work！


sync: RWMutex non-reentrant rlock -race check

What version of Go are you using (go version)?

$ go version
go version go1.12.1 linux/amd64

What did you do?
Misused RLock: https://play.golang.com/p/-7UFH11FDRV

RLock
It should not be used for recursive read locking; a blocked Lock call excludes new readers from acquiring the lock.

What did you expect to see?
I want a -race mode to detect reentrant RLock calls even if deadlock is not yet happened.
I want to propose to add this check to race detector. I am not familiar with how exactly race detector works. But it seems to me that it should be possible to save RLock/RUnlock  calls for each goroutine in race mode, and when pattern (RLock,RLock,RUnlock,RUnlock) apear in the order of calls - it's a sign that RLock is misused.
Is it a good idea, to add a check for this?

/cc @mdempsky @dvyukov FYI, in case you have thoughts on this.

This is undetectable. Unfortunately. Go does not have mutexes, only semaphores. So the check will have false positives.
But otherwise tsan runtime generally have most things ready for deadlock detection on mutexes:
https://github.com/llvm-mirror/compiler-rt/blob/master/include/sanitizer/tsan_interface.h#L27-L105

What Dmitry means is that in Go, ownership of a mutex is not tied to a specific goroutine. It is OK to do code like:
m.RLock()
go work() // does m.RUnlock when done
m.RLock()

and that's not a deadlock. Drop the R's and it is still not a deadlock. But a deadlock detector that assumes lock ownership is tied to a thread/goroutine would report it as a deadlock.
I suppose that back to back RLock with no scheduling event between them might be detectable but that's never the interesting thing to detect.
Closing because this is not detectable.


Fix #68. Option to add acls using attribute node['squid']['acls']

Description
A work around to Chef's limitation on data bag item names not allow the Squid's "not" operator (!)
Issues Resolved
#68
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
Unable to test as v2.0.0 is broken with my environment Ubuntu 14.04 (see #67), however, tests are passing on v1.1.1.
[ ] New functionality includes testing.
[X] New functionality has been documented in the README if applicable
[X] The CLA has been signed. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

I recommend updating v1.1.1 with this fix as well until v2.0.0 is stable.

@jnmoore-intel You'll need to sign the CLA before this can get merged

Closing this out since we need the CLA or DCO signed off


I wish /docs/user-guide/managing-deployments/ touched on deployment-manager/helm, the Deployment kind, and how (if at all) they're related

I wish /docs/user-guide/managing-deployments/ discussed helm, especially if it's liable to become the preferred way to manage deployments in the future.

I agree it's worth mentioning Helm, though it's not the only solution in that space.
cc @janetkuo @pwittrock @jackgr

I could use some tech help on actually creating the content necessary.  This issues isn't actionable until I get that.


Selecting a UITableViewCell atIndex returns incorrect cell

Version: 1.15.1
Currently I have the following in my tests:
func selectCell(atIndex index: Int) {
    EarlGrey.selectElement(
        with: grey_kindOfClass(UITableViewCell.self)
    )
    .atIndex(UInt(index))
    .perform(grey_tap())
}

func testScreen() {
    ...
    selectCell(atIndex: 1) // selects the correct cell
    selectCell(atIndex: 2) // selects the correct cell

    // assertions

    selectCell(atIndex: 1) // selects the correct cell
    selectCell(atIndex: 2) // selects the cell at index 0
}

I have also tried:
func testScreen() {
    ...
    selectCell(atIndex: 1) // selects the correct cell

    // assertions

    selectCell(atIndex: 1) // selects the cell at index 0
}

My tableView is a multi-select table view, multiple rows can be selected at the same time.
The table has two sections. Section 0 is only one cell. Section 1 contains the remainder of the search result cells.
The cell in section 0 is not guaranteed to be selectable, thus we don't want to use it in this specific test.
Any ideas on what's going on?

atIndex is usually going to produce unexpected results for table view. atIndex is used as a last resort to uniquely identify an element. EarlGrey uses a DFS to traverse through the UI hierarchy. When multiple elements match the provided matcher, EarlGrey will fail. That's when you can use atIndex to tell earlgrey to match a specific element from the list of matched elements. Since tableviews recycle views and can have non-visible cells dormant in the hierarchy, earlgrey can match those before the visible cells and that makes atIndex indeterministic. Can you use a different way to uniquely identify table view cells, not atIndex?


Getting error "No such file or directory: './riven/frontend'" after last update

Hi I'm getting this error after updating to 1.2.0. Is that something because of Riven Repo restructure?

@I-am-PUID-0 So what can I do to make it temporarily usable?


Refactor Travis CI

Only build branch pushes on master/release tags
Make sure deploy depends on prior test stages

Opened PR on wrong base branch, so CI checks didn't run


Update dependency locks

The google repo is now required because the App Engine API is pulling in a
new dependency on android annotation.

This change is

Abandoning this.  It was subsumed into #488 once we figured out exactly what was going wrong.


Content agnostic Roles and Groups

Proposed changes:

closes https://github.com/RasaHQ/rasa/issues/7173
makes the prediction of roles and groups independent of the entity value

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

/modeltest
include:
 - dataset: ["all-nlu"]
   config: ["all-nlu"]

 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.Johannes E. M. Mosig seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Missing characters in image file added to PDF

I'm trying to add an image and it is saving the PDF file but I get this error message when I open the file:
"There was an error while reading this Stream".
I have compared the generated PDF-File with an working PDF with the same image and I noticed that there are missing some characters in the generated file. Here are some of the missing characters:
™, Ÿ, †, ‡, —, ‹, ›, etc.

I have tested it with the versions 1.3.2, 1.4.0 and 1.5.3 but it didn't work in that versions.
I think it has to do with the issue #21 but I couldn't fix it in Cordova.
What could be the reason for the missing characters?

Closing because no data was supplied.


VSCode cannot natively debug .NET Framework Unit Tests - fails with "Only 64-bit processes can be debugged."

Problem: .NET Framework Unit Test projects created in VS2017 can be successfully debugged in the IDE. But an attempt to debug the same project is opened in VSCode, attempt to run the tests fails.
Repro steps (on Windows):

Prepare a machine with both VS2017 and VSCode with C# OmniSharp extension.
Unzip UT_NET461_VS2017_1.zip to c:\tests\UT_NET461_VS2017_1 (to work around https://github.com/OmniSharp/omnisharp-roslyn/issues/1331 two explicit references to Microsoft.VisualStudio.TestPlatform.TestFramework assemblies were added manually)
Open "Developer Command Prompt for VS 2017"
cd /d c:\tests\UT_NET461_VS2017_1
msbuild /t:restore
msbuild /t:rebuild /p:DebugType=portable
code . (launches VSCODE which opens the UT project)
Click on UnitTest1.cs to open it in the text editor
Click on "Debug Test" above TestMethod1
Observe the popup windows with "Only 64-bit processes can be debugged":

@gregg-miskelly Is this a known debugger limitation?

Yes. https://github.com/OmniSharp/omnisharp-vscode/wiki/Desktop-.NET-Framework

@dmgonch You may be able to configure your tests to run in a 64-bit worker process to work around this limitation. Otherwise you will need to use full VS.

Here is the issue tracking this feature request: https://github.com/OmniSharp/omnisharp-vscode/issues/1716

@gregg-miskelly I rebuilt the UT project with /p:Platform=x64 but still got the same error when clicking "Debug Test" (UT_NET461_VS2017_x64.zip).
@karanjitsingh I was curious if you have idea for how to get testhost.exe invoked instead of testhost.x86.exe in this case.


no-examine: Suggestion to remove "Cancel"

Who ever pressed "cancel" to cancel? I always just click anywhere on the screen BUT the "cancel" button to cancel.

The plugin already has the ability to remove the Cancel option:

Please let me know if the option does not work as intended.

Oh wow, my bad - should've actually installed first before to see. Seems to be working just fine. Maybe plugin name change is in order?

Maybe plugin name change is in order?

There is already a better plugin out there called Custom Menu Swapper that can remove any user-specified menu option, so I don't think I will bother to improve or rename this plugin


Show which base-branch the pull-request will be opend against

Q
A

Fixed tickets
556

License
MIT

@phansys yep 👍

Thanks 👍


Lock and add dependencies

The idea here is to add more dependencies that are used in all projects, and locking them. This means that projects consuming this package, can remove dependencies like gunicorn, uvicorn and sentry as this package will provide them.
The dependency updater can then update the dependencies in this project when new versions appear, which will trickle down to the projects consuming this package.
This PR also adds a missing pytz dependency

Awesome!


build fails on .net 2

get tons of errors on first build

I don't see how those errors are possible because System.Func isn't available in .NET 2.0

@JamesNK - i'm trying it from MonoDevelop (for unity) so i guess it's available there.

Guess Json.NET .NET 2 it won't compile on monodevelop then.


failed to create new thin device with devmapper snapshotter

Description
when run the command
$ sudo ctr run --snapshotter devmapper --runtime "io.containerd.kata.v2" --rm -t docker.io/library/busybox:latest test-kata uname -r in my machine A, i got the error as below:
ctr: failed to prepare extraction snapshot "extract-67xxxxx sha256:xxxxxxxxxxxxxxxx": failed to create new thin device "VolGroup01-LV_ebs-snap-1"(dev: 6): invalid argument
but snapshotter with overlayfs work well。Anyone know what's wrong with devmapper ?
Steps to reproduce the issue

create device(/dev/sdb) in machine A with iscsi protocol. and the backend storage is part (like /dev/sda6) in machine B
create vg and thin pool using device sdb
modify /etc/containerd/config.yaml using devmapper, and thin pool created before. restart the containerd service
run ctr run --snapshotter devmapper --runtime "io.containerd.kata.v2" --rm -t docker.io/library/busybox:latest test-kata uname -a in machine A

Describe the results you received and expected
snapshotter  with devmapper  is ok
What version of containerd are you using?
1.4.6
Any other relevant information
version:
kata-runtime : 2.1.1
containerd: 1.4.6
OS version:
$ uname -a
Linux xxxxxxx 5.4.185-200.xxxxxx        X84-64
Show configuration if it is related to CRI plugin.
No response

I have also encountered this problem, did you change the size of baseImageSize in the containerd configuration file and the new baseImageSize is smaller than the original one.
I am using k8s and the error reported in the log is:
Failed to create pod sandbox: rpc error: code = Unknown desc = failed to create containerd task: failed to create shim task: failed to mount "/dev/sda" to "/run/kata-containers/shared/containers/cb9a0c70db8ad7a815c9f35f6d1317843c712bd9cd5b22e0d86f243bf7d1ebd9", with error: EINVAL: Invalid argument : unknown

Then I get the log via the dmesg command as:
[86093.866230] EXT4-fs (dm-7): bad geometry: block count 6553600 exceeds size of device (3932160 blocks)
[86093.894510] EXT4-fs (dm-7): bad geometry: block count 6553600 exceeds size of device (3932160 blocks)
[86094.839108] EXT4-fs (dm-7): bad geometry: block count 6553600 exceeds size of device (3932160 blocks)
[86094.867618] EXT4-fs (dm-7): bad geometry: block count 6553600 exceeds size of device (3932160 blocks)

This seems to be related to the snapshot mechanism of thin-pool, where the snapshot device cannot be smaller than the source device.
After I deleted all snapshots under devmapper, I was able to recreate the container based on devmapper.
FYI.

I have also encountered this problem, did you change the size of baseImageSize in the containerd configuration file and the new baseImageSize is smaller than the original one.
I am using k8s and the error reported in the log is:
Failed to create pod sandbox: rpc error: code = Unknown desc = failed to create containerd task: failed to create shim task: failed to mount "/dev/sda" to "/run/kata-containers/shared/containers/cb9a0c70db8ad7a815c9f35f6d1317843c712bd9cd5b22e0d86f243bf7d1ebd9", with error: EINVAL: Invalid argument : unknown

Then I get the log via the dmesg command as:
[86093.866230] EXT4-fs (dm-7): bad geometry: block count 6553600 exceeds size of device (3932160 blocks)
[86093.894510] EXT4-fs (dm-7): bad geometry: block count 6553600 exceeds size of device (3932160 blocks)
[86094.839108] EXT4-fs (dm-7): bad geometry: block count 6553600 exceeds size of device (3932160 blocks)
[86094.867618] EXT4-fs (dm-7): bad geometry: block count 6553600 exceeds size of device (3932160 blocks)

This seems to be related to the snapshot mechanism of thin-pool, where the snapshot device cannot be smaller than the source device.
After I deleted all snapshots under devmapper, I was able to recreate the container based on devmapper.
FYI.

You can follow the steps to check:

sudo ctr plugin ls, make sure the devmapper plugin status is ok
sudo dmsetup ls check devices exist.
check /etc/containerd/config.yaml configuration is OK

ref: https://github.com/containerd/containerd/blob/main/docs/snapshotters/devmapper.md

@stewart-yu there is no update from your side and the release 1.4 is EOL. please try to use supported versions (1.5 or 1.6).
And feel free to reopen it if you still run into the issue.


DWX_ZeroMQ_Server_v2.0.1_RC8 Compile Error

Rafaelbate, Is your issue resolved? I have same error. Removed '&' sign and compiled successfully but getting library file can't open error after that. Please help. Thank you.

Hello, it is, just use that file instead. Solved for me.
chand2020<EMAIL_ADDRESS>escreveu no dia sexta, 27/12/2019 à(s)
18:34:

Rafaelbate, Is your issue resolved? I have same error. Removed '&' sign
and compiled successfully but getting library file can't open error after
that. Please help. Thank you.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/darwinex/dwx-zeromq-connector/issues/63?email_source=notifications&email_token=AI4MZYUNUKZL7FALA6FSIYTQ2ZDEVA5CNFSM4J77TUR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHXSZKI#issuecomment-569322665,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI4MZYV44AEZSIG2NBYYWJ3Q2ZDEVANCNFSM4J77TURQ
.

Thank you..


PEP-8 and steps towards testing

Summary
First of all thanks for creating and maintaining ArchiveBox! It's nice to know that some parts of the internet won't vanish.
I couldn't help myself but making most of the code PEP-8 conform for better readability and maintainability. Since pytest was used for testing at the beginning, python requirements were added to allow linting, coverage and testing with it. At the moment some parts are explicitly excluded because they required major changes. e.g. archivebox/config.txt My first thought was to rebase the dev branch to master but merging seemed to be simpler in this case.
I'll be by no means offended if you close this PR without further notice as it would've been wiser to communicate such changes beforehand. :see_no_evil:  I'd volunteer to add tests later if you're interested.
One can install the requirements via pip install -r requirements/development.txt and run linting and testing with pytest
Changes these areas

[ ] Bugfixes
[ ] Feature behavior
[ ] Command line interface
[ ] Configuration options
[x] Internal architecture
[ ] Archived data layout on disk

Sorry man, but I am going to close this.  I really appreciate the effort though!
I have wide monitors and hate prematurely wrapped lines, makes it much harder to scan the code quickly imo.  .e.g stuff like this is way more confusing/bug-prone than just leaving it long:

In general, all the PEP's that I'm ignoring are explicitly PEPs that I disabled on my machine's flake8 checker.  They're not just things I've casually forgotten, they're intentionally disabled because I prefer the non-pep style.
Another example is ###  comments vs # comments mean different things.  I use them like Markdown headers in reverse, so the more hash marks there are the more important the heading is.
FYI requirements.txt is by default dev requirements, there's no need to create a separate file for base requirements as those are defined in setup.py.
If you like, you're welcome to reopen a separate PR with your fixes for the tests, I'll be adding more tests in the coming week myself as well.


Pls add World Redirect after login system

Has we know that nlogin has feature when a player login in example "hub-1" he will redirect to "hub-2"
like that
OpeNLogin should have feature like when a player logging in "world-1" and he will redirect to "world-2"
This may be realy good
pls add it to both NLogin and OpeNLogin

Will be worked on in #47


Two JavaScript files are generated in Folder (Lib) mode

Two java script files are generated in Bridge/ourput using origin/master branch.
Problem appears when [Namespace(false)] is added
Steps To Reproduce
https://deck.net/a9393901ad4e01678409a79707c54c93
[Namespace(false)]
public class MainClass
{
    [Ready]
    public static void Main()
        {
        }
}

Expected Result
Good question, what should be expected result. Probably only one file should be generated.

Actual Result
2 files are generated, one with project directory name and second with csproj name

Thanks for the report. We will investigate and try to reproduce locally.
By the way, use of [Ready] is no longer required. By default, the public static void Main() will be automatically called as the entry point into your application.

@kcudnik, can you specify your bridge.json please.

Yes, both generated files have the same timestamp but content is different
i have project located in ~/Test/G/Test.csproj
output files are G.js and Test.js and their minified versions,
commit im using from master is 10234f6
im attaching entire project
G.tar.gz

Thanks for the test project, @kcudnik! The problem is related to the /lib option in command-line mode. PR #2531 is submitted to fix the problem. Checking it out before merging to master branch is greatly appreciated.


net/http: TestTransportProxy timing out on js-wasm builder after CL 210286

A timeout in net/http.TestTransportProxy seems to have started around CL 210286:
panic: test timed out after 3m0s

goroutine 2138 [running]:
testing.(*M).startAlarm.func1()
	/workdir/go/src/testing/testing.go:1422 +0x9
created by time.goFunc
	/workdir/go/src/time/sleep.go:168 +0x5

[…]

goroutine 2125 [chan receive]:
net/http_test.TestTransportProxy.func1(0xb08360)
	/workdir/go/src/net/http/transport_test.go:1424 +0x5a
testing.tRunner(0xb08360, 0xa3af60)
	/workdir/go/src/testing/testing.go:954 +0xc
created by testing.(*T).Run
	/workdir/go/src/testing/testing.go:1005 +0x30

[…]

goroutine 2107 [chan receive, 2 minutes]:
testing.(*T).Run(0xb08360, 0xa62860, 0x20, 0xa3af60, 0x2)
	/workdir/go/src/testing/testing.go:1006 +0x31
net/http_test.TestTransportProxy(0x875d40)
	/workdir/go/src/net/http/transport_test.go:1324 +0x6
testing.tRunner(0x875d40, 0x13aca0)
	/workdir/go/src/testing/testing.go:954 +0xc
created by testing.(*T).Run
	/workdir/go/src/testing/testing.go:1005 +0x30
FAIL	net/http	184.130s

2019-12-09T21:50:59-3a3093d/js-wasm
2019-12-09T20:19:32-7d24e95/js-wasm
2019-12-09T16:41:15-a3a630b/js-wasm
CC @bradfitz @ianlancetaylor

That CL was the fix for #28012.

It passed the trybots at the time.
Does a js-wasm net.Conn not reliably interrupt a blocked Read on Close or something?
I'm fine just skipping that test on js-wasm for now.

I don't know the underlying cause of the failure. The failure rate seems to be around 50%, which would explain the passing TryBot.

(CC @neelance @cherrymui for the question about the js-wasm net implementation.)

The new code in CL 210286 adds a 1 minute timeout reading the CONNECT response if no context deadline was otherwise present.
It's possible the net/http tests are just so slow on wasm that it's straddling that 1 minute timeout already (~50% of the time?) with all the parallel tests. But, uh, TestTransportProxy isn't t.Parallel.

Found the bug, I think:
https://go-review.googlesource.com/c/go/+/210286/2/src/net/http/transport.go#1588

Whoops, thanks!


ci: use ubuntu-22.04 to avoid pip errors in ubuntu-latest caused by Python 3.12

This PR fixes the version of the Ubuntu image to 22.04 so that Python packages can be installed via pip alongside system ones provided as Debian packages.
See https://github.com/PyO3/maturin-action/issues/291

@luca-della-vedova looks like maturin was hit by the same issue we're trying to fix, so ironic :rofl:


Build and publish aarch64 wheels

also fix setuptools issue.
Supersedes https://github.com/pytries/marisa-trie/pull/100

OK, it's good for me 💪🏻
Thank again @vermut 🍾


NextJS Regex is not supported

I was trying to execute NextJS with GraalVM and this error happened:
SyntaxError: Unsupported regular expression /(?<!node_modules.*)[\\/]node_modules[\\/](react|react-dom|scheduler|prop-types|use-subscription)[\\/]/: TRegex: body of lookbehind assertion too complex; Joni: quantifiers inside lookbehind assertions not supported
    at getBaseWebpackConfig (/workspace/hello-next/node_modules/next/dist/build/webpack-config.js:14:285)
    at HotReloader.getWebpackConfig (/workspace/hello-next/node_modules/next/dist/server/hot-reloader.js:14:1320)

Thank you!

Hi @Thihup
thanks for reporting this. This is an unfortunate combination of two features that are not supported by the two engines we use (our own TRegexp and the external JONI). TRegex, being an AST-based approach, does not fully support the lookbehind you are using. It then tries to fallback to JONI; However, JONI does not support quantifiers inside lookbehinds.
We are currently working on providing a baseline regex engine to replace JONI, that supports all features, to avoid such a situation in the future. Until then, I can only suggest to modify the regex as a workaround (I understand this is tougher as this is an external library).
I'll have our engineers check this in more detail, perhaps there is an easy fix available.
Best,
Christian

This is fixed by the latest version of TRegex: https://github.com/oracle/graal/commit/2c7fc94b0269ae171773d851ae4bc2c4d210e29e .

Hello. I have tried to execute Next.js with the latest GraalVM I could get and this is still occurring.
Is there a release already with the fix?
Here's my GraalVM version:
> java -version
openjdk version "11.0.6" 2020-01-14
OpenJDK Runtime Environment GraalVM CE 20.0.0 (build 11.0.6+9-jvmci-20.0-b02)
OpenJDK 64-Bit Server VM GraalVM CE 20.0.0 (build 11.0.6+9-jvmci-20.0-b02, mixed mode, sharing)

This GraalVM version seems to be the latest on SDKMAN, but I can see 20.0.1 is on the way just about now?!

20.0.1 is about to be released and will definitely contain the fix.

Awesome... this bug was also stopping Gatsby from working, unfortunately... hoping this fix will make it possible to do React Server-side rendering with GraalVM!

The root of the problem for both Gatsby and Next.js is the same, the regex seems to be in Webpack:
failed Building production JavaScript and CSS bundles - 0.446s

 ERROR #98123  WEBPACK

Generating JavaScript bundles failed

Unsupported regular expression /(?<!node_modules.*)[\\/]node_modules[\\/](react|react-dom|scheduler|prop-types)[\\/]/: TRegex: body of lookbehind assertion too complex; Joni: quantifiers inside lookbehind assertions not supported

May I know why this issue is closed without further info? If possible I'd like to see Gatsby / NextJS working flawlessly with Graaljs

Hi,

May I know why this issue is closed without further info? If possible I'd like to see Gatsby / NextJS working flawlessly with Graaljs

Please not that there was in fact a comment right before the issue was closed:

This is fixed by the latest version of TRegex: oracle/graal@2c7fc94 .

To the best of our knowledge, the specific issue reported is fixed in GraalVM 20.1 or later. If you still experience a problem in this area, please create a new ticket and let us know the version of GraalVM you are using, and the failure you see.
Best,
Christian


Detector class big fixes

Updating the detectors class to PISA4 didn't work as I thought. So here are some bug fixes that hopefully will resolve the issues. It is mostly about updating parameter values for the different DistributionMakers that represent the detectors.

Was it that the parameters weren't updating at all for one of the distribution makers? I might've been running into the same bug! I can look over this

Everyone happy with this or are more eyes on this needed?

i think this is ready


Support the name "SHA-1" as an alias of "SHA1"

Functions like EVP_get_digestbyname support hyphenated names, as evidenced in ssl/ssl_lib.c:
    if ((ret->md5 = EVP_get_digestbyname("ssl3-md5")) == NULL) {
        SSLerr(SSL_F_SSL_CTX_NEW, SSL_R_UNABLE_TO_LOAD_SSL3_MD5_ROUTINES);
        goto err2;
    }
    if ((ret->sha1 = EVP_get_digestbyname("ssl3-sha1")) == NULL) {
        SSLerr(SSL_F_SSL_CTX_NEW, SSL_R_UNABLE_TO_LOAD_SSL3_SHA1_ROUTINES);
        goto err2;
    }

The official rendering of the Secure Hash Algorithm 1 is "SHA-1" according to FIPS 180-4 and RFC 3174, with a hyphen.  There would seem to be no technical barrier to aliasing the name.  Where supported, would it make sense to alias the digest name "sha-1" to "sha1"?

But those names are used in the TLS specification; it's not clear we need to use them in the user layer in general.

I think what @Niggler is getting at is that EVP_get_digestbyname("sha1") works but EVP_get_digestbyname("sha-1") does not, even though the latter is the preferred spelling.

I see no harm in adding aliases.  After all, it's just a simple edit of crypto/objects/objects.txt followed by a make update...
Anyone want to take that on?  Is this only for SHA-1, or are there others that could benefit with the same kind of aliasing?  PRs welcome (or a list of desired aliases)!

@Vonter gave it a good try in #4900, but it turns out it wasn't as easy as I imagined...  it's been long since I dealt with this part of the source.

EVP_get_digestbyname("sha-1") does not [work], even though the latter is the preferred spelling.

I disagree with this assertion. That "latter is the preferred spelling." I don't feel that there is consensus on the preferred [and universal] spelling. SHA-1 is surely preferred spelling in written text as direct reference to algorithm in question, but hardly in application code or elsewhere, for example it's hmac-sha1, sha1WithRSAEncryption, etc. Ironically even referred RFC is entitled "US Secure Hash Algorithm 1 (SHA1)"...

SHA-1 is now properly fetchable name. The EVP_get_digestby... is legacy interface. Closing.


Check if permissions are granted

I need a function to see if the user has accepted the permissions.
Currently my app can see the bluetooth is 'disabled'. Which can lead two options, either the user has their bluetooth turned off or the user has denied permissions. However I need to distinguish two to and see if the user has denied the permissions or granted it.

This is duplicated by #445
Contributions towards a feature like this would definitely be welcome


Why model performance is better than expected?

Instead of 42.2, I got 43.1 AP on the default setup with your pre-trained weights. That's a big surprise! :) May I know the reason?

Thanks for your quick reply! :)


Allow symbols for status codes

This is a very simple implementation of feature requested in #934.

I like it. Rebase and squash please?

Done

@dblock I changed the error message but didn't localize it yet. As I commented before, I'm not sure if it makes sense to allow user to translate a message that API end-users (as far as I understand) will never see. If it really needs to be localized I can change it but I wanted to make sure you really want that.

Fair enough, merging.

Good one @dabrorius  :+1:

Thanks :)


Add support for '@retroactive' conformances in SwiftSyntaxParser

This is a companion to https://github.com/apple/swift/pull/36068, teaching the SwiftParser about @retroactive conformances explicitly and providing keywords for clients who are building macros/SwiftSyntax tools.

@swift-ci please smoke test

@swift-ci Please test

@swift-ci please test


EE-1045: Fix slashing for validators

Ref: https://casperlabs.atlassian.net/browse/EE-1045
Removes quash_bid and makes it an impl detail, and then slash uses it to slash validator bids.

Can you update the should_step test to include slashing check? There are some TODO's in the test indicating the changes that were waiting on this bug fix.

Can you update the should_step test to include slashing check? There are some TODO's in the test indicating the changes that were waiting on this bug fix.

Done

bors r+

bors r+

bors r-

bors r+


Add Sketch Creation Scalar Functions for HLL/Theta

This enables creation of BYTES columns containing Theta/HLL sketches during ingestion.
Note I've added these in pinot-core as I didn't know if I should add the datasketches/clearspring dependencies to pinot-common
Can be activated like so:
{
  "transformConfigs": [
    {
      "columnName": "players",
      "transformFunction": "DistinctCountRawThetaSketch(playerID)"
    },
    {
      "columnName": "players",
      "transformFunction": "DistinctCountRawThetaSketch(playerID, 1024)"
    },
    {
      "columnName": "names",
      "transformFunction": "DistinctCountRawHLL(playerName)"
    },
    {
      "columnName": "names",
      "transformFunction": "DistinctCountRawHLL(playerName, 8)"
    }
  ]
}

Codecov Report

Merging #10347 (40214fb) into master (69d2fae) will increase coverage by 38.19%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #10347       +/-   ##
=============================================
+ Coverage     32.06%   70.25%   +38.19%
- Complexity      236     5901     +5665
=============================================
  Files          2027     2028        +1
  Lines        109954   109975       +21
  Branches      16711    16719        +8
=============================================
+ Hits          35256    77263    +42007
+ Misses        71552    27289    -44263
- Partials       3146     5423     +2277

Flag
Coverage Δ

integration1
24.53% <0.00%> (+0.16%)
:arrow_up:

integration2
24.54% <0.00%> (?)

unittests1
67.67% <0.00%> (?)

unittests2
13.75% <0.00%> (-0.02%)
:arrow_down:

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

Impacted Files
Coverage Δ

...he/pinot/core/function/scalar/SketchFunctions.java
0.00% <0.00%> (ø)

...core/startree/operator/StarTreeFilterOperator.java
86.18% <0.00%> (-0.66%)
:arrow_down:

...pache/pinot/core/query/utils/idset/EmptyIdSet.java
25.00% <0.00%> (ø)

...anager/realtime/SegmentBuildTimeLeaseExtender.java
63.23% <0.00%> (ø)

.../core/realtime/PinotLLCRealtimeSegmentManager.java
75.41% <0.00%> (+0.45%)
:arrow_up:

...lix/core/realtime/PinotRealtimeSegmentManager.java
79.50% <0.00%> (+0.50%)
:arrow_up:

...che/pinot/broker/routing/BrokerRoutingManager.java
85.79% <0.00%> (+0.55%)
:arrow_up:

...ces/PinotSegmentUploadDownloadRestletResource.java
54.77% <0.00%> (+0.82%)
:arrow_up:

...e/pinot/common/function/TransformFunctionType.java
100.00% <0.00%> (+0.94%)
:arrow_up:

.../helix/core/realtime/SegmentCompletionManager.java
73.17% <0.00%> (+1.01%)
:arrow_up:

... and 1258 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

thank you for the contribution. could we add some tests?

tests with 100% coverage have been added

I can't find a mechanism that would allow changing the column type from eg STRING to BYTES during aggregation

This prompts me to think the real problem to solve is to allow agg(CAST(strCol AS BYTES)), can I assume this change in ingestion is no longer needed if we have support for the above?

I guess the ideal would be to have different column types between REALTIME and OFFLINE and swap to sketches when moving between tables

I can't find a mechanism that would allow changing the column type from eg STRING to BYTES during aggregation

This prompts me to think the real problem to solve is to allow agg(CAST(strCol AS BYTES)), can I assume this change in ingestion is no longer needed if we have support for the above?

To clarify, I meant during rollup aggregations, where we turn multiple rows into 1, because sketches are usually used to compact rows
It is still desirable to create sketches during ingestion because it reduces the size of data stored in Pinot and removes the requirement to preprocess data to create sketches and roll it up before pushing to Pinot

Thanks for your contribution @andimiller. One thing to share from past experience is that creating a sketch/hll per value may bloat data size. Nonetheless, does seem like a nice feature to have.


Add Update(before, after) API for senses

Fix #1186.
Adds a new UpdateSense(Guid entryId, Sense before, Sense after) method to the IMiniLcmWriteApi interface.
Also adds a GetSense(Guid entryId, Guid id) method to the IMiniLcmReadApi interface.

SenseDiffToUpdateneeds to be changed ...

Done in commit 95441bb.

Since merging develop in, there's one test failure: SenseDiffShouldUpdateAllFields is expecting before.SemanticDomains to have 1 item, but it's empty. I don't know if this is caused by a recent change on develop and I need to update the test to exclude SemanticDomains (and probably PartsOfSpeech) from the .BeEquivalentTo() check, or if this is a real bug. @hahn-kev - any ideas?

@rmunn yes that's expected, you can configure that test to ignore SemanticDomains, that set of tests is really only checking the diff update produced on the basic fields of an object, eg changes which go through Update(id, jsonPatch) not reference changes like Semantic domains

Note that this PR has the (after, before) ordering in it; that will be fixed by #1303, so we should not change it here lest merge resolution end up doing the wrong thing.


Multi-Step Entropy Regularization for SAC

Add record_extra_info flag to the NStep tracer that records the intermediate states in the new extra_info field to TransitionBatch
Add support for the NStepEntropyRegularizer in SoftPG

This PR contains an initial working implementation of the mechanism and sums um the discounted entropy bonuses of the states s_t, s_{t + 1}, ... , s_{t + n - 1} for the soft policy gradient regularization.

This still needs some tests and I will try to do a rudimentary performance comparison.

I just ran a quick test on Pendulum-v1 with TD3 (turquoise) and SAC (red), both with 5-step TD.
Additionally, I ran SAC with NStepEntropyRegularizer (blue, red) and EntropyRegularizer (turquoise, pink) with 5-step TD and regularization weight of 0.2 (default in many SAC implementations) and 0.2/5=0.04.

At least after this one sample run per variant, the performance of the n-step entropy regularization seems to be better and the entropies are also behaving as expected, i.e. more steps in the entropy bonus leads to higher entropy.

[...] more steps in the entropy bonus leads to higher entropy.

This is interesting. Theoretically the agents should optimize the same objective (given the same hparams), at least asymptotically. I would expect them to converge to roughly the same points (in terms of entropy etc). In other words, I would expect them to differ only in how fast they converge.


Add setting suppress_encoding to BccField

This pull request addresses issue #864.
Add suppress_encoding to BccField to enable encoding the bcc field. By default, the Bcc field is still not encoded for reasons listed in #426 and thus this does not break backwards compatibility.
Tested under:
1.9.3-p551
2.1.5
2.2.0
jruby-1.7.18

Is it supposed to be used globally by an initializer?
The problem is that I need two behaviors in one app (I'm sending emails both through sendmail and gmail). Changing the option right before and right after I send a gmail message looks strange to me. That's why an argument for to_s method looks as a better option for me.

Thanks @grossadamm. Made some comments @ https://github.com/mikel/mail/issues/864#issuecomment-93648760
The root issue is that we make the message BCC field do double duty as a real message header (which should be encoded when we set it) and as a list of envelope recipients (which aren't part of the message).
As a transitional step, we could mark the BCC field to indicate that we want to encode it.

@Exoth, the new commit would allow you to configure per message if the bcc field is included in the headers. Each message would need to have the configuration value set as each new message would default to not including the bcc field. Would that work for you?

So is this gonna work like this: message.bcc.include_in_headers = true?
That looks perfect for me. Thanks.

@Exoth correct

@Exoth So apparently I fail at reading comprehension lately. message.bcc

Returns the Bcc value of the mail object as an array of strings of address specs.

So there is no way to access include_in_headers right on `message. I'm looking into what I need to do to make the setting more accessible.
This pull request is essentially unusable at the moment, my apologies to everyone while I get this straightened out.

For a laugh, you can do the following and it will work
message.header.fields.select {|f| f.name == 'Bcc'}.each { |f| f.include_in_headers = true }

Aha! After more reading of comments I learned that instead of message.bcc.include_in_headers = true you can do message[:bcc].include_in_headers. And that works as the pull request is currently written.
Would message[:bcc].include_in_headers = true that be acceptable to you @Exoth, @jeremy ?

Looks usable to me, though may be not the most obvious solution.

Is there anything further I need to do here?

This PR would be useful for me as well. My (eccentric) use case is that I have to generate RFC2822 email documents and post them to an API, and that API supports BCC. Rather than monkey patching, my current (shameful) implementation is to inject the BCC line manually.

Great. It would be nice if a new version of gem is released.


Missing packed metadata and mtime in createTar

Environment
Node 18
nanotar: 0.1.1
Reproduction

createTarGzip([ {
  name: 'file.txt',
  data: Buffer.from('Hello, world!'),
  attrs: {
    mode: 0o644,
    mtime: Date.now(),
  },
} ]).then((tar) => {
  fs.writeFile('file.tar.gz', tar);
})

Describe the bug

It doesn't seem like the packed metadata is being included, causing issues in file size reporting on Windows, and mtime for the files are not preserved since there is no way to pass that in.
Additional context
No response
Logs
No response

Invalid report. I noticed one is a tar while the other is tar+gzip.

Although it is interesting to note they have the same compression ratio. I wonder if this is actually a bug in tar-fs instead.


fix remove unused dual option in agent

Please provide a description of this PR:

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
3cec068f8871df068874359a3b70b33b7d603adb
link
true
/test release-notes

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

/test release-notes


Add a specific error when a pd.dataframe is passed to the python client

#157 issue

I didn't find a way to this smartly (i.e. without importing pandas in the (pure) python client).


Service Fabric Application Restart

Hi
I have a service fabric app that reads from KeyVault on startup. When we change the KeyVault value we have to restart the nodes for it to read the new value. This causes the other applications on the same node to go down.
I am trying to write a PowerShell script to restart the service fabric application I have got the following script working on a 1 node cluster however it does not work on multiple node cluster.
Restart-ServiceFabricDeployedCodePackage -ApplicationName $appName -CommandCompletionMode Verify -ServiceName $ServiceName -TimeoutSec 8000

When I have a mutiple node cluster this does not work. I can't figure out why it will not work.
Am i doing something wrong?

Not a direct answer to your question, but for these tasks we have built "Management API/Service" in our cluster . In your case, this API would call the service in question to read the new configuration from the key vault. Seems to be a better solution than brute forcing a restart. ;)


warnings!

 When a warning is triggered, a ⚠ icon is displayed in the top-right corner of the plot.

Mousing over the warning indicator gives the tooltip e.g. “1 warning. Please check the console.” The console shows the warnings:

I have implemented three warnings so far:

When dates are associated with an implicit band scale (typically because of a bar mark)
When strings that appear to be dates are associated with an implicit band scale
When strings that appear to be dates are associated with an implicit ordinal or point scale

I imagine we will think of lots of additional warnings in the future, but I think these three cover the most common mistake I’ve seen in Plot. We may want to have another tracking issue to cover ideas for other warnings (e.g., insets resulting in zero-width rects or bars, mixed-sign values with log scales, undefined or invalid data, etc.).
Ref. https://talk.observablehq.com/t/bar-chart-with-dates/6194/3

I've edited the issue number in your comment to https://github.com/observablehq/plot/issues/536.

I was thinking we’d close that with this PR since we’ve implemented the warning system, and then have a new tracking issue for specific warnings. (Or separate issues or sets of related warnings.) But it’s not important. I can live without the joy of closing another issue. 😉

I wonder if I can similarly detect when strings are numbers? That would catch errors like using CSV data without typed: true.

I wonder if the messages should start with Plot warning: or Observable Plot warning: for easier filtering.

For what it’s worth, browser consoles will also filter by file name, so as long as Plot is loaded as e.g. “plot.js”, filtering by “plot” will work even if the messages themselves do not contain the word.

FWIW I considered adding a warning if you have numeric data with an implicitly ordinal scale (we already have a warning for temporal data with an implicitly ordinal scale). However it seemed like it might introduce some false positives. This will be something to keep an eye on when we release.
I also thought about having the warning include a link to a webpage that gives more detail on how to fix the warning. I think React does this pretty well?


Differences to barcode buddy

Since I know there's an app for barcode buddy, I was wondering where these two apps distinguish. As this seems still to be some niche, I'd thought it would be great to join forces 🧐

Honestly i started developing this app in part because there wasn't a good way to get barcodes to BarcodeBuddy, they didn't have that app when I started.
I also set out to fix some of the workflow difficulties I had, such as the increased number of barcode sources and a more streamlined product creation.


Attaching multiple items with additional pivot data attempts to store as incorrect data type

Laravel Version:
PHP Version: 7.30.4
Database Driver & Version:
sqlsrv  I'm not sure which version sorry

Description:
When attaching multiple models to another model via a many-to-many with additional pivot data, in groups of more than about 20, I get an error like Conversion failed when converting the nvarchar value '17.85' to data type int even though the column is decimal
Steps To Reproduce:
I have an NdisRunReport model, and a Trans table, with ndis_run_report_trans in between them.
the transactions() relation on NdisRunReport looks like this
public function transactions()
	{
        return $this
            ->belongsToMany(Trans::class, 'ndis_run_report_trans')
            ->withPivot(['claim_reference', 'claim_amount', 'quantity', 'hours', 'unit_price', 'claim_type', 'cancellation_reason']);
    }

The ndis_run_report_trans table looks like this

After creating the NdisRunReport model, I build an array of Trans IDs and associated pivot data like this
$pivotData[$transaction->SYSID] = [
    'claim_reference' => $claim_reference,
    'claim_amount' => $transaction->calculated_total_including_cancellation,
    'support_number' => $transaction->Service->ndis_support_number,
    'quantity' => $transaction->qty,
    'hours' => $transaction->hours,
    'unit_price' => $transaction->UNITCOST,
    'claim_type' => $transaction->ndis_export_claim_type,
    'cancellation_reason' => $transaction->cancelCode->ndiaCancellationCode->code ?? null
];

claim_amount and quantity are both floats - the first being an accessor that explicilty returns a float, the other I would assume is a float from the way it's initialised.
$transaction->qty = ($transaction->QUANTITY ?: 0) * $workerCount;
(QUANTITY is a float and workerCount is an int)
I attach the models like this
// Attach the pivot data. Chucked due to max binding error
foreach(collect($pivotData)->chunk(200) as $chunk) {
$report->transactions()->attach($chunk);
}
I get the error somewhere in the second chunk loop. If I set the chunk size to 20, I get it after 260 elements (chunk number 13).
I DON'T get an error if the chunk size is less than 15 or so, or if the chunk size is 1, so there's nothing wrong with the data itself. I also don't get the error if I force the variables to floats using (float)

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

@driesvints sorry just making sure you read the full "steps to reproduce"? Especially the last sentence? If you have and you still think it's just a support issue please reply as such and I'll ask there.

Yeah, please try a support channel.


FAPI Compliance Test: CallAccountRequestsEndpointWithBearerToken fails

From the IG logs;
The JWT cannot be validated: Expected value for claim 'iss' to be equal to 'https://iam.bohocode.forgerock.financial/am/oauth2/realms/root/realms/alpha'
In the Authorization Bearer token the issuer is
"https://obdemo.bohocode.forgerock.financial/am/oauth2/realms/root/realms/alpha",
This is because I have set the base url to obdemo.bohocode.forgerock.financial

So the above two PRs work, and get us further in the FAPI conformance tests.

I can't really get them merged into master or I will break any other envs using the new initializer/own CDK

The fixes in here can't be merged without breaking everyone's environment. Progress on these FAPI issues will have to continue on branches for now.

All of the work done on this issue was done in this branch, then merged into issue/254-work-on-fapi-conformance branch and then into master as part of issue #254


NFR-MCT-016 - Availability

The MCT web service should be available outside of scheduled maintenance and not be affected by the availability of any targets.

how are we going to tell if this works or not


Fix revert endpoint discovery

https://github.com/canonical/seldon-core-operator/issues/123

Summary of changes:
- Reverted PR 94.
- Added remove test.
- Modified code not to error on 404s when deleting resources.

@i-chvets instead of reverting the metrics endpoint discovery commit, let's ensure this change is reflected in our repository after the PR is merged.

Sounds good. Let's wait till it get merged.

Merged and released.

Solved by https://github.com/canonical/seldon-core-operator/pull/129


Support Editorconfig on web editor

This PR depends on a change made on editorconfig core.

@Unknwon Updated accordingly.

@Unknwon Updated.

Thanks!

Now live on https://try.gogs.io


WIP: Secrets Protection

Description
Task was added to support secrets protection in all roles, allowing users to encrypt specified properties in various configuration files.
Secrets Protection 1-Pager
Secrets Protection Docs: https://docs.confluent.io/current/security/secrets.html
Type of change

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

How Has This Been Tested?
This has been tested via the plain-rhel molecule test.
Test Configuration:
secrets_protection_enabled: true
confluent_cli_download_enabled: true
broker_config_path: "./etc/kafka/server.properties"
broker_config_params: [advertised.listeners, broker.id]\
(These lines test secrets protection with user-provided master key, replace PATH TO CP ANSIBLE with your own path)
secrets_protection_masterkey: "UWQYODNQVqwbQeFgytYYoMr+FjK9Q6I0F6r16u6Z0EI="
secrets_protection_security_file: "/PATH TO CP ANSIBLE/cp-ansible/roles/confluent.test/molecule/plain-rhel/security.properties"
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Any dependent changes have been merged and published in downstream modules

Hi please check premission of secrets.propeties the file is created with root root causing read error on startup


Fix broken image in demo project for recipe lab

Describe the Issue
In the recipe page lab, when the user clicks on the "Show Demo" button, the demo project has a broken image link.

This is happening, because we never replaced the placeholder link with the acutal CDN link
How to fix the problem
Here is a link to the actual lab
https://github.com/freeCodeCamp/freeCodeCamp/blob/main/curriculum/challenges/english/25-front-end-development/lab-recipe-page/668f08ea07b99b1f4a91acab.md
You need to update the solution to use the correct CDN link
Here is the correct src link and updated alt text that should be used for that image element in the solutions section
  <img src="https://cdn.freecodecamp.org/curriculum/labs/recipe.jpg" alt=""Ingredients for baking: three eggs, a bowl of flour, a glass of milk, and a whisk arranged on a wooden table."">

Affected Page
https://github.com/freeCodeCamp/freeCodeCamp/blob/main/curriculum/challenges/english/25-front-end-development/lab-recipe-page/668f08ea07b99b1f4a91acab.md
Your code
see explanation above
Expected behavior
see explanation above
Screenshots
No response
System

Device: [e.g. iPhone 6, Laptop]
OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 22]

Additional context
No response

This has been opened for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors.
If you would like to contribute and have not read the contributors docs, please do so here: https://contribute.freecodecamp.org/#/
If you have any issues with contributing, be sure to join us on the contributors channel, or on the contributors sub-forum
Happy coding

Hi @jdwilkin4,
I want to contribute on this issue

Hi @jdwilkin4
I am interested in contributing to this issue as I am also a first-timer and know of how to resolve it.

Hi @jdwilkin4
I am interested in contributing to the issue

@PKBAJPAI12, @Balaji-Batchu, @Mk281005
Please stop and read carefully second comment (https://github.com/freeCodeCamp/freeCodeCamp/issues/56343#issuecomment-2380421229) everything is written in it.

Thanks @gikf I got your point

hi @jdwilkin4 i'm also a freshie and want to contribute on this issue.


Using synchronize_expectation globally

Hi.
Just started using your gem but can't really get how can I use synchronize_expectation globally in my tests.
After defining base test helper like so:
class BaseTestHelper < Capybara::TestHelper
end

and verifying that this class indeed defines synchronize_expectation method I've tried using it in my globally included module like so:
module GlobalTestHelper
  extend ActiveSupport::Concern

  included do
    use_test_helpers(:base)
  end
end

But when calling synchronize_expectation I'm still getting undefined method synchronize_expectation for #<RSpec::ExampleGroups::Dashboard::NewUser:0x00007f99afaa5cd0>.
Even if I delegate the method to base with delegate :synchronize_expectation, to: :base inside of included block, I'm still getting protected method synchronize_expectation called for #<BaseTestHelper<PHONE_NUMBER>2940>. So it seems to me that synchronize_expectation can only be called without explicit receiver, hence only inside of subclasses of Capybara::TestHelper.
Any ideas what I'm missing? The example does not include this use case as far as I've noticed

For now I've settled by implementing synchronize_expectation in BaseTestHelper:
class BaseTestHelper < Capybara::TestHelper
  def synchronize_expectation(*args)
    super
  end
end

but this seems really unnecessary

By default, synchronize_expectation is protected, because it's meant to be used inside test helpers, where it can be properly encapsulated.
Using it directly in tests can make them significantly harder to read.
If you really need to, you can change the method visibility to public instead:
class BaseTestHelper < Capybara::TestHelper
  public :synchronize_expectation
end

By default, synchronize_expectation is protected, because it's meant to be used inside test helpers, where it can be properly encapsulated.
Using it directly in tests can make them significantly harder to read.
If you really need to, you can change the method visibility to public instead:
class BaseTestHelper < Capybara::TestHelper
  public :synchronize_expectation
end


Is AsyncEx code Nitro code. Comments on Nitro inconsistent with solution

In the overivew on May 30,2016, shown below you made it seems that you created some AsyncEx library code to fix deadlocks that can occur in the  Nitro library source code.  In the overview you list the methods in the Nitro library that can lead to deadlock. So one would presume that the download source code where you posted the overview would download the AsyncEX library, but it downloads the Nitro library that you said had the dead lock problems.
Where is you source code that is suppose to fix the deadlock problems?
Overview
Stephen Cleary edited this page on May 30, 2016 · 2 revisions
https://github.com/StephenCleary/AsyncEx.wiki.git
Clone in Desktop
TaskExtensions provides one method OrderByCompletion which orders a sequence of Tasks by when they complete. The approach taken by AsyncEx is a combination of Jon Skeet's approach and Stephen Toub's approach.
TaskExtensions in the Nito.AsyncEx.Synchronous namespace provides a handful of extension methods that enable synchronous blocking of a Task without wrapping its exception in an AggregateException, or without observing the Task exception at all. These are advanced methods that should probably not be used, since you run the risk of a deadlock.
API
public static class TaskExtensions
{
// Creates a new array of tasks which complete in order.
public static Task[] OrderByCompletion(this IEnumerable<Task> tasks);
}
// (in namespace Nito.AsyncEx.Synchronous):
public static class TaskExtensions
{
// Waits for the task to complete, unwrapping any exceptions.
public static void WaitAndUnwrapException(this Task task);
// Waits for the task to complete, unwrapping any exceptions.
public static void WaitAndUnwrapException(this Task task, CancellationToken cancellationToken);
// Waits for the task to complete, unwrapping any exceptions.
public static TResult WaitAndUnwrapException(this Task task);
// Waits for the task to complete, unwrapping any exceptions.
public static TResult WaitAndUnwrapException(this Task task, CancellationToken cancellationToken);
// Waits for the task to complete, but does not raise task exceptions. The task exception (if any) is unobserved.
public static void WaitWithoutException(this Task task);
// Waits for the task to complete, but does not raise task exceptions. The task exception (if any) is unobserved.
public static void WaitWithoutException(this Task task, CancellationToken cancellationToken);
}

Any synchronous blocking on asynchronous code can cause deadlocks. It is not possible to "fix" this - you just have to avoid it.
For this reason, the blocking task extensions are in a separate namespace - you have to explicitly opt into using them.

ok thanks


WIP: POC for using Mono.Cecil instead of System.Reflection

Will remove the need for loading/unloading assemblies.

[x] Script alias finder
[ ] Default assembly resolver for .NET Core
[x] Tests from Cake.Core

[x] Method alias code generation
[x] Property alias code generation

[x] Method alias code generator
[x] Property alias code generator
[x] Documentation support (parsing and CRef generation)

[x] Parsing XML
[x] CRef generation
[x] Method aliases
[x] Property aliases

@patriksvensson this is starting to look really, really good! Great work!

@patriksvensson I will start reviewing shortly.
If ok, could you do Default assembly resolver for .NET Core to a separate PR?

@mholo65 Yes-a-reno


Added optional confidentiality and clearance nodes to the cpacs header.

closes #508

I agree. I'll assemble some examples.
Thank you for the feedback.

So far I did not get a response from the author of the original issue. If there is no response soon, I will remove the issue from the release 3.0 in order not to delay the release any further.

closed by #508


show underlying docker/helm commands on request

Currently, there is a lot of black magic that goes on when draft up is run, spanning at least docker and helm commands.
$ draft up --debug
Draft Up Started: 'my-service': 01CFDW1HQQ5B3ZCGP3YZHFYFPK
my-service: Building Docker Image: SUCCESS ⚓  (1.0038s)
my-service: Pushing Docker Image: SUCCESS ⚓  (2.2107s)
my-service: Releasing Application: FAIL ❌  (1.3373s)
Inspect the logs with `draft logs 01CFDW1HQQ5B3ZCGP3YZHFYFPK`
[debug] No post deploy tasks to run

However, with this issue that is happening on releasing the application, the logs do show a yaml rendering error, but it is relatively hard to debug what might be going on. I believe it would be good if there could be an option to show what helm commands are being run under the hood, so that advanced users can tweak the commands/parameters to assist in the debugging process (for example, by adding --debug or --dry-run flags - although for YAML rendering errors specifically that may not help as much).
I could see something similar for docker commands, but those are relatively less problematic in general, so that could be out of scope for this.
as demonstrated above, passing --debug to the draft up command, which according to the documents might be a reasonable guess to activate this functionality, barely adds any additional information.

docker variant of this could be a workaround for #806 if it enables the underlying docker commands that user can run manually :)


buildToolsVersion conflict

Could you plz change several parameters such as buildToolsVersion to read from the configuration file of the project root directory?
Like this function below:
def safeExtGet(prop, fallback) {
       rootProject.ext.has(prop) ? rootProject.ext.get(prop) : fallback
}

usage:
compileSdkVersion safeExtGet('compileSdkVersion', 26)
buildToolsVersion safeExtGet('buildToolsVersion', '26.0.2')

aha, I'll submit pr soon.thx.

@jordanbyron hello, I have already submit the pr, plz review. thanks


Setgame command and prefix bug

Describe the bug
When using the command setgame, instead of only recognizing the command with the given/current prefix, it will set a game with any prefix. Say my prefix is ~, but I use _, it will still set the game.
To Reproduce
Steps to reproduce the behavior:

Use setgame with a prefix that is not the default/currently set prefix.

Expected behavior
It not to set a game, unless the currently set prefix is used.
Screenshots

Branch (please specify one):
master

Thanks for the report, 5.0.6 fixes this now (https://github.com/synzen/Discord.RSS/commit/7b0b4e6a2504fcb8d7eb4c018d81fec330d89eb0)


CSV table import should have delimiter set as external setting

Expected behavior
Parsing a csv file with ; as delimiter should be successful. In some locales (like german), the excel export to csv generates a csv with ; as delimiter, as , can be part of a cell (e.g. when representing money values like 35,45 EUR)
Actual behavior
Gauge fails to parse a csv with ; as delimiter
Testing started at 14:13 ...
"C:\Program Files (x86)\Gauge\gauge.exe" run --machine-readable --hide-suggestion --simple-console C:/Users/ajoecker/IdeaProjects/end2end-backend/specs/test.spec:7
[ParseError]C:/Users/ajoecker/IdeaProjects/end2end-backend/specs/test.spec:9 Dynamic parameter <table:data/test.csv> could not be resolved => A step with table <table:data/test.csv>'

Process finished with exit code -1

Steps to reproduce

run gauge init java
generate a csv file in the project with ; as delimiter
use this file in a step
execute gauge

Gauge version
$ gauge -v
Gauge version: 0.9.7
Commit Hash: bc9784c

Plugins
-------
flash (0.0.1)
html-report (4.0.4)
java (0.6.7)
screenshot (0.0.1)
spectacle (0.1.2)
xml-report (0.2.0)

needs #1100

Question
Should the delimiter allow multiple characters?

@sguptatw  No we should not have multiple delimiter characters as using multiple delimiters is not ideal.

I also do not think multiple characters are reasonable. There are (afaik) a minor number of characters, that are used as delimiter in csv (excel does not allow to many different delimiters), all of them are single characters

Here is a (non-gauge context)discussion where the user wanted multiple characters as a delimiter
That said, its okay if we want to support only single characters. Just wanted to check the expected behavior.

Question
How can I specify something like a space as the separator?
As quoted in wikipedia
In addition, the term "CSV" also denotes some closely related delimiter-separated formats that use different field delimiters. These include tab-separated values and space-separated values.

there is no official standard for CSV and the one closest to be is here: https://tools.ietf.org/html/rfc4180. It continiously mentions about the lack of standards, so I would actually try to keep things as minimal as possible.
The common use case for most cases is ,, in german based countries (or the ones using , as number separator) it is ;.
Personally I would like to avoid to open the gate now to all possible options, due to the lack of standards.
Only when someone has a valid csv and a valid reason why ;.;.;;;;; is a separator or ;     . or so

@ajoecker I feel that it is the same in other places too. The most common delimiters being , or ;.
Some people may also use other special characters like | etc.
@sguptatw Using space or tab as delimiter is a very rare case and probably deal with it only when someone has a valid use case of using it in a csv.


Slicing queue

Add a queue for slicing jobs. There should be only one slicing job at a time.

Queue should be added outside of the slicerbroker
maybe somehwere in here:
https://github.com/FlorianJa/SlicerConnector/blob/4b1f869a36f8915c7c689f378f7f9f0130f0abb7/SlicerConnector/Startup.cs#L100-L127

The queue should be added in the slicer broker because the slicing process can be triggered from different components.

No need for queueing as it will make it difficult to control the flow of the async/await methods calling the slicer.
Since every slice request will come from a different PrusaSlicerBroker object, a static semaphore will do the trick.


Compute randomized list of cards to display on frontend

The frontend should calculate (at play startup time) a random list of cards which determines the order they are displayed. This will be computed from the finite set of cards to be given at one time

fixed by #37


Merge Dev into Main

Merge Dev into Main
Result
Merge Dev into Main to Push Recent Changes into Production
Changes:

Upgrade Splink to New Major Version 4.0
Add Numeral Normalization Logic
Add SBOM section to README.md

Closes #55
Test Plan
make test

I reckon the missing docstring for the qrda check is ok since it's not implemented yet, and I can't say whether the Splink version bumps work or not, but I don't see anything obviously blocking in there, so, LGTM +1 🚢 ➿

The tests pass on this PR and it works when I run it on test data. What is blocking for approval?

I reckon the missing docstring for the qrda check is ok since it's not implemented yet, and I can't say whether the Splink version bumps work or not, but I don't see anything obviously blocking in there, so, LGTM +1 🚢 ➿

The tests pass on this PR and it works when I run it on test data. What is blocking for approval?

Nothing :) #Shipit 🚢

Nothing, ship it :ship: :)
Remy DeCausemaker
Open Source Lead
Digital Service at CMS.gov

From: Isaac Milarsky @.>
Sent: Thursday, August 1, 2024 5:54:01 PM
To: DSACMS/dedupliFHIR @.>
Cc: DeCausemaker, Remy (CMS/DSAC) @.>; Review requested @.>
Subject: Re: [DSACMS/dedupliFHIR] Merge Dev into Main (PR #111)
I reckon the missing docstring for the qrda check is ok since it's not implemented yet, and I can't say whether the Splink version bumps work or not, but I don't see anything obviously blocking in there, so, LGTM +1 🚢 ➿
The tests pass on this PR and it works when I run it on test data. What is blocking for approval?
—
Reply to this email directly, view it on GitHubhttps://github.com/DSACMS/dedupliFHIR/pull/111#issuecomment-2264076917, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZXUXUPPFSPAD3MGNJ4RHHTZPKU7TAVCNFSM6AAAAABL3LPIRWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRUGA3TMOJRG4.
You are receiving this because your review was requested.Message ID: @.***>


Google colab module removed, and detailing added

Google colab module is removed and simpler OpenCV framework is used. Captions will be displayed along with the image.
Related Issuse or bug
Google colab module is specifically used for working in the same, and detailing of the images was missing during display.
Fixes: #411
Proposed Changes
I have omitted the google collab module and added some captions in each display part.

I will raise a issue, then I can change ?

@sudipg4112001 yes you can create a issue.
are you dwoc participant?

Ok..!
I am a participant of GSSOC 2021

hi please wait till the coding period starts

Ok..!
I am a participant of GSSOC 2021

I am also a participant of CWOC.

This project is available for this program also.


More venues

Is it possible to support more venues out of the box, perhaps borrowing from what the other projects you have been involved in has done.

Of course! Do you have a need for some particular trading venue?
Past projects have taught me that the more trading venues a project supports, the more inertia it will add to maintaining the project. I myself have so far had need only for NASDAQ, therefore it is currently the only supported trading venue.

I close this for now. If a need for a particular trading venue comes up, that can be handled then through a new, more specific issue.


Update docs page sailbot_workspace/docker_images

Purpose

Now that the pre-base image can be built using the build images workflow, pre-base and base-dev have been combined to a single image layer.
NOTE: Wait until the changes to remove pre-base are finalized and pushed to main branch, before making this change.
Adding to this issue:
We should document somewhere how to solve the issue referenced here
If you have just built an image locally, --load will not work with the multiarchitecture image. You can only use --push. which makes more sense anyway.
Changes

Remove pre-base row in the table and update the parent image for base-dev to Ubuntu
Resources

Images Page

If you have just built an image locally, --load will not work with the multiarchitecture image.
Hey @SPDonaghy,  if I'm not mistaken "Use containerd for pulling and storing images" option under general settings is out of beta and with that enabled --load is working.

Oh great thank-you! I'll edit the issue in a bit

Updated the images page in #200


Train station skipping fix

Hey there, I don't really know much about github so I'm not sure what a pull request is or how it works, this is the first ive ever seen. If this is a way I could improve the mod for you and make it more fun I'd love to do so. It looks like you want to make a change to the control file so that the schedule doesn't bump up by one stop after a jump. I designed it like that on purpose so that trains can operate automatically in cases where it is jumping between two unconnected sets of rails (like over a river). Without this schedule bump, a train that is guided to a ramp using a train stop will still have that stop as its destination after the jump and will either freeze or keep looping back around to the ramp, so I don't know why you would want to undo this. What is the situation or bug your change is looking to fix?

Hi, I thought that skipping a station was a bug and didn't realize that it could be used in this way.
I wanted to use this mod as a way of making bridges and build crossings with higher throughput. This feature made it impossible to use it in this way.
This modification would make it possible. But since this is a feature and not a bug as I thought, I'm gonna stick with custom version of this mod :D
Thanks for an awesome mod!

No problem prokop, when I made the ramp I thought this functionality made sense but since then other people like yourself showed interest in not having the ramp work this way and I realized there were good case for both stop skipping and no stop skipping. I'm thinking of adding a way to toggle the stop skipping per ramp, similar to swapping between primer pad modes


Many Sistem.IO methods not implemented

I think I miss something.
I've noticed that many Sistem.IO methods are not implemented.
Such as File.Exists(), File.WriteAllText(), and so on.
And FileInfo class does not exists.
Do I have to change my code in order to use only the implemented methods?
I saw that the Bridge System.IO.File static class has only 9 methods. Why?
I appreciate further explanation or reference where I could understand more about this project.

Thanks for pointing this missing set of implementations, @jkancio.
Basically we focus on implementing features noted by the community. In fact, System.IO was once asked at #779. If you start browsing by there and follow related issues and pull requests, you'll see several things have been done regarding them, but there's no comments on request or about the missing sample you pointed.
But well, I am personally not sure how could we implement this with the limitations from browsers' javascript to perform actual file operations on the local system. So at first, it seems there is a great chance implementations like File.Exists() would not be viable to be developed at all due to limitation on the web browsers where Bridge could run. But the history could be different, for example, if building an application for server-side javascript implementations like NodeJS.

Thanks for the reply.

But the history could be different, for example, if building an application for server-side javascript implementations like NodeJS.

That's it what I intend to do! I have a code written in C# and I'd like to build a proof o concept concerning the viability to convert it to javascript running under server side NodeJS.
I believe I have 2 choices: Implement the missing methods extending the code, I mean, forking it and trying to contribute for the project, or changing my code supposing that the file content is already in memory and making my own code to read the file with Javascript code.

My code uses these 3 methods below.
if(File.Exists(file))
...
File.WriteAllText(file, json, Encoding.UTF8);
...
FileAccess fa = FileAccess.Read;
_st = new FileStream(FileWithPath, FileMode.Open, fa);
...
Do you think they are very complex to implement?

I would recommend implementing most of System.IO according to the .NET standard.
Javascript has no access to files, but you could make a seperate NuGet module that extends the StreamWriter / StreamReader with a sort of "virtual file system". Eg. files exist in memory, and files loaded from server may be accessible and processed.
In that way, users can port existing applications that work with files. Simply add code that downloads them to the virtual file system, and uploads them back to the server.


Regions Not Loading on Orbital Survey

On the regions screen after completing survey it says "Awaiting Data".
First time using the mod in science mode I had two craft conducting orbital surveys. The first craft I had ran and completed the Kerban scan to 100% but due to not having enough electricity it could not transmit the data. I launched a second craft try to do it but it wouldn't provide any options to transmit the data during the scan.
I then went to Duna and attempted a scan with a fresh craft. I completed the scan and transmitted the data successfully, but the regions for Duna still do not display and gives me the same prompt. I suspect I goofed up an edge case along the way.

Interesting, I haven't enncountered this issue before. I'll have to test it to see what's happening.
For Kerbin, I think that's just how the stock science system works. When you complete an experiment, it's attached to your vessel and you have to transmit it. I'll have to see if there's a way to "cancel" the experiment and rerun it on another vessel. Though the way this mod uses experiments is different than the stock game does it, as they're automatically triggered when milestones are achieved. So I guess I understand how this issue might occur.
For Duna, I don't see how this could happen, so please provide additional info. So you sent a vessel to Duna with a Region scanner, you scanned the Regions to 100%, transmitted Region scans, and the Region map for Duna still says "Awaiting Data..."? Or did you mix this up with the Visual map?

Ok, I've been able to confirm the first part. When you finish an experiment (25% for example) on a body, it gets locked to that vessel that completed it and there is no way to "transfer" it to another vessel or to cancel it. So you have to transmit it somehow. I'll need to see if it's possible to either cancel the experiment or recreate it on another vessel, but disallow gathering science twice.
The second part I couldn't confirm. Whatever happens on Kerbin, Duna is a different body that isn't affected by it. I could gather science and maps normally, even though there were untransmitted experiments for Kerbin.

The second part is interesting. I tried to edit the save file to reset everything, but I couldn't clear the issue. Even after removing all the experiments and states of planets in the save file, I would do a scan on either planet it would still have the same issue.
I then created a new fresh science game all together and the problem still persists. Since it is just me I suspect the only thing it could be is some installation issue or mod compatibility issue.

Can you send your save file so I can check myself?

Better late than never. I haven't checked if there has been any change to behavior with the latest updates.
Eclipse.zip


3D Tiles cache fix

This fixes cache trashing when using replacement refinement and the cache size is less than the number of tiles needed to meet the SSE for a view, e.g., using a crazy low cache size like 3 when really something reasonable like 70 tiles are needed for a given view.
@lilleyse I think it would be painful to write specific tests for this beyond what is already there.  Let me know if you think otherwise.

I'm fine with the changes here without tests.


⚠ Bofrost.fr has degraded performance

In 844f12f, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1664 ms

Resolved: Bofrost.fr performance has improved in e36cf0a after 36 minutes.


Breakout1 Instructions

Includes istructions for:

Organization

making branches on github site
cloning repository
checking out branch

Making Changes

Debugging code
Committing

add/status/commit/push

Merging Branches

Open PR
Compare Changes between branch/master
Merging PR

I think this is an OK start, but we will definitely want to do an isolated run-through once we get the instructions for breakout 2/3 made.

this PR will eventually fix #11

There's nothing about deleting the branches. Do we want them to do that now?

There's nothing about deleting the branches. Do we want them to do that now?

I think so. They can just make new ones for the next breakout room.


build: Install Poetry with pipx

build: Install Poetry with pipx
As recommended by the documentation and
@xylar.
This should avoid the "No such file or directory:
'/usr/local/miniconda/envs/test/lib/python3.[…]/site-packages/[…].dist-info'"
errors which probably comes from the interaction between Conda and Poetry.

Looks like this is working. It might be worth trying the latest Poetry release, too, to make sure this fixes the issue you reported.

This should avoid the "No such file or directory:
'/usr/local/miniconda/envs/test/lib/python3.[…]/site-packages/[…].dist-info'"
errors which probably comes from the interaction between Conda and Poetry.

I’m not sure about that. Those packages may come with GDAL.
This should fix the packaging import problem, though.

Looks promising!