Remove hardcoded font

I think it's best to just let pandoc use whatever font it defaults to.
This ensures that if pandoc works on the users machine for generating
PDFs, then it'll work to render our templates.
We can follow this up with changes to make the font configurable,
via #19, or some other means, to that users can specify Helvetica or
whatever else if they so choose.
This came up for me because contract compilation would fail due to not
having Helvetica installed.

merp.. this is the wrong way to fix this, as is. I neglected the style argument.

Superseded by #21


ACM-9792 Backport configure Argo and OCP limit 2.8

https://issues.redhat.com/browse/ACM-9792
Amadeus cannot upgrade anytime soon to ACM 2.10 for this fix.

/test unit-tests-sonarcloud

/test unit-tests-sonarcloud


[BUG] test_window_aggs_for_rows_lead_lag_on_arrays failed

In last night's EGX-Yarn integration test run, test_window_aggs_for_rows_lead_lag_on_arrays failed:
09:52:00  =========================== short test summary info ============================
09:52:00  FAILED integration_tests/src/main/python/window_function_test.py::test_window_aggs_for_rows_lead_lag_on_arrays[partBy:Long-orderBy:Long-orderBy:Long-agg:Array(Double)][IGNORE_ORDER({'local': True})]
09:52:00  FAILED integration_tests/src/main/python/window_function_test.py::test_window_aggs_for_rows_lead_lag_on_arrays[partBy:Long-orderBy:Long-orderBy:Long-agg:Array(Timestamp)][IGNORE_ORDER({'local': True})]
09:52:00  FAILED integration_tests/src/main/python/window_function_test.py::test_window_aggs_for_rows_lead_lag_on_arrays[partBy:Long-orderBy:Long-orderBy:Long-agg:Array(Array(Long))][IGNORE_ORDER({'local': True})]
09:52:00  FAILED integration_tests/src/main/python/window_function_test.py::test_window_aggs_for_rows_lead_lag_on_arrays[partBy:Long-orderBy:Long-orderBy:Long-agg:Array(Array(Array(Double)))][IGNORE_ORDER({'local': True})]
09:52:00  FAILED integration_tests/src/main/python/window_function_test.py::test_window_aggs_for_rows_lead_lag_on_arrays[partBy:Long-orderBy:Long-orderBy:Long-agg:Array(Array(Array(Timestamp)))][IGNORE_ORDER({'local': True})]

A sample error log from one of the failures is below.
I will reproduce it and get the data when it fails.

a
b
c
d

1
null
null
[9,9,9]

1
null
null
[9]

1
null
null
[9,2]

1
null
null
[19,9]

1
null
null
[23]

I just debugged this issue and got the data generated when the test case failed in yarn, and the root cause is like what I described in the previous comment https://github.com/NVIDIA/spark-rapids/issues/2457#issuecomment-845563283
when the rows of a,b,c are equal, then CPU and GPU can't guarantee the row's order, while lead/lag typically depends on the order of row.
The solution is adding column d to the order by,
SELECT LEAD(d, 5) OVER (PARTITION by a ORDER BY b,c,d) FROM Table

but for now, SortOrder has not been supported on arrays, I just filed an issue to follow this  https://github.com/NVIDIA/spark-rapids/issues/2470
So the Workaround is mark the test case as XFAIL.

Submitted a PR to skip the test case https://github.com/NVIDIA/spark-rapids/pull/2471

Still open pending fixing the test case in issue #2473


Make QueuePool.callback non-static, but final

There may be multiple queue pools, a static callback breaks this.

@hylkevds would be great if you can fix the conflicts so that later we can merge it :+1:


Update README.md

Will be nice to add the NomadBSD logo below the title, but there are no logo image files that are large enough to add to the README.

Will be nice to add the NomadBSD logo below the title, but there are no logo image files that are large enough to add to the README.

I've created an artwork repo which contains the logo in higher resolution:
https://github.com/nomadbsd/artwork


Ruby 2.7.1 printing syntax warning

.rbenv/versions/2.7.1/lib/ruby/gems/2.7.0/gems/http-parser-1.2.1/lib/http-parser/parser.rb:14: warning: `&' interpreted as argument prefix

Just need to change:
        #
        # Returns a new request/response instance variable
        #
        def self.new_instance &block
            ::HttpParser::Instance.new &block
        end

and add parenthesis:
        #
        # Returns a new request/response instance variable
        #
        def self.new_instance &block
            ::HttpParser::Instance.new(&block)
        end

surely
        def self.new_instance(&block)
            ::HttpParser::Instance.new &block
        end

Sure, was just suggesting a minimal change.
https://github.com/cotag/http-parser/pull/5


chore: update sdk dependencies

Signed-off-by: Anish Ramasekar<EMAIL_ADDRESS>
Reason for Change:

Updates deps
Update to go1.16 for e2e tests

Requirements

[x] squashed commits
[ ] included documentation
[x] added unit tests and e2e tests (if applicable).

Issue Fixed:

Does this change contain code from or inspired by another project?

[ ] Yes
[x] No

If "Yes," did you notify that project's maintainers and provide attribution?
Special Notes for Reviewers:

/azp run pr-e2e-azure


Please explain the issue with Minishift's networking

In the README.md I find:
https://github.com/projectodd/openwhisk-openshift/blob/5deb4a2fff52a616713f38b0dc6354b8b2af89dc/README.md#L87-L88
Can you explain with this 'bug' is? I do not remember seeing an issue on our tracker for this...

I believe the ultimate upstream bug is https://github.com/openshift/origin/issues/12111 . That issue is also referenced by https://github.com/minishift/minishift/issues/1269 which describes a similar, albeit not identical situation. Those issues appear to be resolved now, so perhaps this is no longer needed?


[e] cannot open 'ipsw/extracted//Firmware/048-19711-075.dmg.trustcache' Dualboot IOS11 A10

sudo bash dualboot.sh --dualboot 11.4.1 --fixHard --dont-create-part
[*] Command ran: sudo ./dualboot.sh --dualboot 11.4.1 --fixHard --dont-create-part
dualboot | Version mod for ios 13. and 12,11 beta
Written by edwin and some code of palera1n and pyboot:) thanks pelera1n team and seprmvr | thanks for the ramdisk from Nathan | thanks MatthewPierson, Ralph0045, and all people creator of path file boot
[] Waiting for devices
[11:51:48.589][3] usbmuxd v1.1.1 starting up
[11:51:48.589][3] Using libusb 1.0.25
[11:51:48.593][3] Initialization complete
[] Detected recovery mode device
[] Getting device info...
0x8010
d101ap
iPhone9,3
[] To get into DFU mode, you will be guided through 2 steps:
[] Press any key when ready for DFU mode
Get ready (0)
Release side button, but keep holding volume down (9)
Release side button, but keep holding volume down (6)ERROR: Unable to connect to device
Release side button, but keep holding volume down (2)
[] Device entered DFU!
extracting ipsw, hang on please ...
Archive:  ipsw/iPhone_4.7_P3_11.0_11.4.1_15G77_Restore.ipsw
'ipsw/extracted//BuildManifest.plist' -> 'work/BuildManifest.plist'
048-19711-075.dmg
[] Creating ramdisk
[-] Ramdisk is already created so SKIPPING ...
[] Booting ramdisk
usb_timeout: 5
[libusb] Waiting for the USB handle with VID: 0x5AC, PID: 0x1227
CPID: 0x8010
Found the USB handle.
Stage: RESET
ret: true
[libusb] Waiting for the USB handle with VID: 0x5AC, PID: 0x1227
CPID: 0x8010
Found the USB handle.
Stage: SPRAY
ret: true
[libusb] Waiting for the USB handle with VID: 0x5AC, PID: 0x1227
CPID: 0x8010
Found the USB handle.
Stage: SETUP
ret: true
[libusb] Waiting for the USB handle with VID: 0x5AC, PID: 0x1227
CPID: 0x8010
Found the USB handle.
Stage: PATCH
ret: true
[libusb] Waiting for the USB handle with VID: 0x5AC, PID: 0x1227
CPID: 0x8010
Found the USB handle.
Now you can boot untrusted images.
usb_timeout: 5
[libusb] Waiting for the USB handle with VID: 0x5AC, PID: 0x1227
Found the USB handle.
[==================================================] 100.0%
[==================================================] 100.0%
[==================================================] 100.0%
[==================================================] 100.0%
[==================================================] 100.0%
[==================================================] 100.0%
[==================================================] 100.0%
[] Waiting for the ramdisk to finish booting
bind(): Address in use
Error creating socket for listen port 2222: Address in use
libusb: warning [op_get_configuration] device unconfigured
libusb: error [op_get_active_config_descriptor] device unconfigured
[11:53:17.731][3] Could not get old configuration descriptor for device 2-35: LIBUSB_ERROR_NOT_FOUND
[11:53:17.739][3] Connecting to new device on location 0x20023 as ID 1
[11:53:17.784][3] Connected to v2.0 device 1 on location 0x20023 with serial number ramdisk tool Sep 18 2022 20:14:43
[11:53:17.785][2] device_control_input: Got unhandled payload type 5
[11:53:17.785][2] device_control_input: Got unhandled payload type 5
[11:53:18.522][2] device_control_input: Got unhandled payload type 5
8
[] Testing for baseband presence
8
9
10
seputil: Gigalocker file (/mnt7/CE457C94-1FB4-519C-96BA-BFEE51341A2F.gl) exists
seputil: Gigalocker initialization completed
sepi digest (48 bytes): b8eeb54b5a1888e412ad3ebfc38cf822647aae016103512b9bf0d1950624570ee961e702d5b5528be01545f5c899c53d
sepi nonce (20 bytes): bba0e7bf104ab7c6a1b88e227a3755a70b776131
rsep digest (48 bytes): d88e2a136a5e23ecaaa380cd7a4198b775bd5c50d2d0fcb6984b2ac5278ec769ecbe8ed33b419db3d64379074987408a
rsep nonce (20 bytes): bba0e7bf104ab7c6a1b88e227a3755a70b776131
'blobs/iPhone9,3-11.4.1.der' -> 'work/IM4M'
[] Starting step 2
[] Fixing firmwares
AOP FOUND
aopf
StaticTrustCache FOUND
[e] cannot open 'ipsw/extracted//Firmware/048-19711-075.dmg.trustcache'

git pull

git pull

this command must be entered in the dualra1n folder?

I will be able to install IOS11 or 12 through your script?

I was blocked, so I don't have access to information

I was blocked, so I don't have access to information


Fix delete AliasAttribute in UserPool object

I use "add-cognito-user-pool-trigger", but claudia caused error.
% claudia add-cognito-user-pool-trigger --events CustomMessage --user-pool-id ap-northeast-1_XXXXXXXX
{ UnexpectedParameter: Unexpected key 'AliasAttributes' found in params
　：

It is necessary to delete 'AliasAttributes' in describeUserPool's object in add-cognito-user-pool-trigger.js. This PR implemented it.

thanks, this is probably something new they added

this is now on npm as 2.11.1

thanks for your quick response!


[feature] Exam results (semester, overall)

Requirements

Expose overall results (cumulative-gpa)
Expose semester-wise results (sgpa, course metrics like grade, grade point, credits)

Extra info
On the portal, this information is available at Examination > Examination Result from the sidebar. For semester-wise results, there's a dropdown that lets you navigate to any semester for which the result has been declared (the default is the last-declared semester).

Working on this


Updated to fix issue with creating Database and DocumentCollection

Updated to match changes accepted in tutorial documentation :-)

@stevebelton This has been fixed now along with other places as well.


Beta banner is confusing when placed in between menu button and menu items

This issue is from a May 2019 external accessibility audit report.
WCAG Reference: Usability feedback
Issue ID: DAC_4
URL: All pages
Summary
When using a screen reader it is confusing to find the beta banner after the menu button instead of the menu items.

While swiping through the page with VoiceOver for iPhone I located a button which read as ‘Toggle mobile menu’. I selected the button and discovered that content opened further down the page. Although an indication was provided that the content had expanded, I needed to swipe multiple times to locate the new content, with the beta information situated between the menu button and the menu content. This could mean that users do not immediately locate the information and may become confused. Ensuring that either the users focus is taken to the menu, or alternatively that the user is able to locate the menu on the next swipe forward will avoid this potential confusion.”

Screenshot

Had a quick peak at this and if you only move this for the mobile navigation it's not too difficult...


chore: Composer: fix lock file

Description
The composer.json file (as well as the CONTRIBUTING doc) indicate that development should use PHP 7.4+, but the project contains a committed composer.lock file with dependencies which are not compatible with PHP 7.4.
This fixes the composer.lock file to actually contain a PHP 7.4 compatible set of locked dependencies.
Alternatively, it could be considered to remove the lock file altogether or to add a platform-php setting to it.
Motivation and context
Installing on PHP 7.4 doesn't work.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

PR checklist

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING.md document.
[ ] I have added tests to cover my changes.

Alternatively, it could be considered to remove the lock file altogether

I'm giving some thought to this. Originally, composer.json and composer.lock were in the repo only for the sake of testing, but with v2, I decided to start using composer.json as part of development.
I want to make sure that the action runs using the root composer.json and composer.lock files, while also running in a working-directory (you can take a look at the matrix to see the permutations I'm running). Maybe I should remove the lock file from the root and let it run there without the dependency versions, while using dependency-versions within the working-directory?

I'll go ahead and merge this, since it takes care of the PHP version issue, but I'll give more though to removing the lock file from the root altogether.
Thanks!

I want to make sure that the action runs using the root composer.json and composer.lock files, while also running in a working-directory (you can take a look at the matrix to see the permutations I'm running). Maybe I should remove the lock file from the root and let it run there without the dependency versions, while using dependency-versions within the working-directory?

Makes sense to me based on my limited insight into this repo... Though running most tests with a root version of the repo also makes sense...
Would having a separate repo(s) which could be checked out for the integration test runs for testing the functionality using root composer.json and composer.lock files, be an idea ?
actions/checkout allows for checking out another repo using with - repository.


semaphore.sh, go: add semaphore to release-3.1 branch, upgrade release-3.1 to go 1.8.5

Pending approval of https://github.com/coreos/etcd/pull/8805. This adds semaphore to 3.1 branch, like https://github.com/coreos/etcd/commit/2f74456443f97d34ee5a23591fdd4a4ab9405aea did for the 3.2 branch.
This also upgrades the 3.1 branch from go 1.8.3 to 1.8.5, primarly because we already have gcr.io/etcd-development/etcd-test:go1.8.5 in the registry and don't see any point adding 1.8.3 when we can more easily upgrade go by a couple patch versions.

lgtm, after CI greens.
Thanks!

lgtm

Seems like we also need backport https://github.com/coreos/etcd/commit/6825ffe1a4b69167a4a1fcbdac6612e71aff76ac for Go 1.8+.

Same tests failed locally, will dig in.

Thanks @gyuho, I'll cherry pick that as well.

Hmm, nvm. Seems like we already have that in release-3.1.
Looks like it's something else

--- FAIL: TestV2NoRetryNoLeader (0.00s)
expected "no leader", got invalid URL port "123.4686.sock"

Yep, cherry-pick gave me a "empty commit" response-- it's already on 3.1. I'll poke around tomorrow and figure out where .sock came from.

Thanks @gyuho Looks like https://github.com/coreos/etcd/commit/22c52b6d2ea7b477f90e0e2ae299ef89d5b2562d is what we need.

Yeah maybe we can merge this first, and cherry-pick those in a separate PR?

I've split them out as two separate commits in this PR.

Latest failure:
Too many goroutines running after all test(s).
2 instances of:
net.runtime_pollWait(...)
	/usr/local/go/src/runtime/netpoll.go:164 +0x4b
net.(*pollDesc).wait(...)
	/usr/local/go/src/net/fd_poll_runtime.go:75 +0x2b
net.(*pollDesc).waitRead(...)
	/usr/local/go/src/net/fd_poll_runtime.go:80 +0x29
net.(*netFD).Read(...)
	/usr/local/go/src/net/fd_unix.go:250 +0x154
net.(*conn).Read(...)
	/usr/local/go/src/net/net.go:181 +0x56
net/http.(*persistConn).Read(...)
	/usr/local/go/src/net/http/transport.go:1316 +0x15d
bufio.(*Reader).fill(...)
	/usr/local/go/src/bufio/bufio.go:97 +0xd5
bufio.(*Reader).Peek(...)
	/usr/local/go/src/bufio/bufio.go:129 +0x51
net/http.(*persistConn).readLoop(...)
	/usr/local/go/src/net/http/transport.go:1474 +0x185
created by net/http.(*Transport).dialConn
	/usr/local/go/src/net/http/transport.go:1117 +0x88b
2 instances of:
net/http.(*persistConn).writeLoop(...)
	/usr/local/go/src/net/http/transport.go:1704 +0x37f
created by net/http.(*Transport).dialConn
	/usr/local/go/src/net/http/transport.go:1118 +0x8ac

Yeah, release-3.1 branch has been flaky with clientv3 tests.
We can address them when we backport clientv3 fixes.
Shouldn't block mvcc patches.

the semaphoreci is not running. @gyuho do you know why?

@fanminshi It's running?
I just reran a few times https://semaphoreci.com/coreos/etcd/branches/pull-request-8807/builds/5, to see if it passes.

@gyuho you are right. I didn't noticed that you re-run the test.

similar to this failure https://github.com/coreos/etcd/issues/6933. not sure if they are related. @gyuho

@jpbetz thank you.


Adapt docker files for new Rasa CLI and core merge

The dockerfiles need some love on the master branch. They should use the rasa CLI to manage the data and project inside the container.

We should also check whether all the libraries in apt-get install are

actually needed

can be deinstalled after the installation of Rasa (e.g. the dev packages)

done, docks need an update though


Simplify field assembly register declaration

Doesn't work on stable for some reason.

Isn't that just because assembly is nightly-only?


fix build with Win32-2.6.0

My earlier change to rename broke building with the new version of
Win32, which has a Maybe FilePath as the second moveFileEx parameter.
Fixes #35

Thanks

Uploaded as <IP_ADDRESS>


Security: Package.json Reference To Non-NPM Package: "test262"

Security tools dislike references to packages not defined in the public NPM repo because of the risk someone could squat them and intercept installation attempts to inject naughty code.
Is that package still needed? Has it perhaps been renamed? ~If hosted elsewhere can that be defined in the package.json, at least to satisfy audits?~ [EDIT: Now I see the package source is defined, albeit implicitly via GitHub, perhaps that needs to be full URL or must be the public NPM.]

@xtuc much thanks for merging my PR.
Any chance a new version could be tagged so that the change can work its way downstream to WebPack and others?

@paulrrogers  I published<EMAIL_ADDRESS>


Fixed an issue with the apply code snippet.

The original one was missing the quotes and once the quotes were added, it was resulting with:
Error:Plugin with id 'com.bugsnag.android.gradle.BugsnagPlugin' not found.

Thanks for the contribution, @Zlate87. Loading a plugin by either id or class name should work, was there an issue with using the qualified class name without quotes?

My script looked like this:
apply plugin: com.bugsnag.android.gradle.BugsnagPlugin

bugsnag {
    apiKey keys.bugSnagKey
    autoUpload false
}

When I run it, it results in an error:
* What went wrong:
A problem occurred evaluating script.
> Could not find property 'com' on project ':XXXXX'.

Gotcha. I think I’d run into the reverse issue previously (unable to load by id, so the instructions changed to use class reference), and as such it seems that for some versions of gradle, one or the other doesn’t work: https://github.com/bugsnag/bugsnag-android-gradle-plugin/commit/ab85a2026d4ae89224e583f733e20800429452a7
I’ll take a look to see if the id problem is resolved.

If it helps, the project I had the issue on is using gradle v2.14.1.

This seems to work on the latest versions of 2.x and 3.x. Thanks again!


Ogg files not supported by iOS

Whilst android mobiles are reading correctly the ogg files from Commons, iOS mobiles are now not reading correctly the ogg files.

Wikimedia Commons implicitly includes an OGG to MP3 converter. I adjusted the redirection link to point to the output of this audio converter.


Fixed native crash when loading 0 sized files

Fixed native crash due to missing check for 0 size or possible crash due to error in ftell, i.e. ftell might return -1, so lead to incorrect allocation. This will eliminate file check in java code and make image loading less error prone.

Thanks! Merged.


LIST VIEW: New entity on the left side "Funktionsregister"

until this is filled with data make it unclickable


Some confusions about code

Thanks for your paper and sharing code. I have some confusions about code, can you explain it for me.

In function channel_selection()， why the original model is replaced by pruned model? Is it since the loaded pre-trained model without fine-tuning.
In function update_model, why the network is splited repeatly(the network will be split a time if the update_model() be called)？But the segment seems same after each split,
Is it original net just be a baselone. Why origin net is continualy updated(line 503 in channel_pruning.py) if it is just be a baseline ?
Thank you very much.
Expect for yor reply.

To reduce the discrepancy between the original model and pruned model, we replace the original model with the fine-tuned model at the first stage.
After channel selection, the pruned model has changed. Therefore, we need to update the pruned model in the trainer.
Except in the first stage of fine-tuning, we fix the original model during channel pruning.


Temporal Arithmetic Can Panic

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

The temporal kernels make extensive use of chrono kernels, these in turn may panic
Date32Type::to_naive_date(i32::MAX);
TimestampNanosecondType::add_year_months(64, i32::MAX);

Describe the solution you'd like

We should return an error on overflow
Describe alternatives you've considered

Additional context

glad to pick this up

While updating to the  latest chrono https://github.com/apache/arrow-rs/pull/5479/files @tustvold  notes that chrono's APIs are getting less likely to panic, but to preserve API compatibility, now the code in arrow-rs panics explicitly. This seems like progress but we should eventually pass back the values to the caller

I'm new to this codebase, I can see the temporal aritmetic kernels / conversions have been updated and unwraps in place (keeping API compatibility I guess). Is the plan to introduce APIs that return some variant of Error back to caller? Happy to pick this up if anything is left to do on this one. Thanks.

There is no specific plan that I know of
I think introducing new fallable APIs (while keeping the existing non fallible ones) would be a nice improvement
I think the first thing to do would be to propose what the new API would look like


Use clang's __has_feature instead of CMake if possible?

Might be possible to use clang's __has_feature under clang instead of relying on check_cxx_source_compiles(), which is kind of slow.
But might not be worth the trouble.

It is not possible to access the preprocessor from CMake (afaik), so it would need check_cxx_source_compiles() anyway. It could only be used as a better check, but I am too lazy for that.


forge compatibility issue

Hey there, it's not an incompatibility with Forge or something, but rather me setting the maximum dnd5e compatible version as 2.4.1, to make sure no issues arise from Warpgate being a dependency.
I am working (slowly the truth is), on updating MidiSRD for Foundry v12 and latest dnd5e system version.
I'll close this issue for now, until an update is out :)


Does not work on websites such as Salesforce and GitHub

The loading text does not go away, and the search does not execute

Have you tried opening/closing your browser?


Remove DefaultTraceListener

DefaultTraceListener causes lock contention and unnecessary overhead for production environments in most cases. This issue is to track removing it by default for the v3 SDK. The CosmosDiagnostics includes most of the necessary information. Testing should be done to verify that customer can enable it if necessary and to document the process.
https://github.com/Azure/azure-cosmos-dotnet-v3/blob/fca18fbad82d466dc9ba9ef166cda2917e32284e/Microsoft.Azure.Cosmos/src/DocumentClient.cs#L640

DefaultTraceListener is not enabled by the SDK, it is enabled by default to all NETFX applications.
Also it's enabled when running in Debug in VS (https://docs.microsoft.com/en-us/dotnet/api/system.diagnostics.defaulttracelistener?view=netframework-4.8#remarks).
NET 5 might also have this (not sure as I have not used it).
We can advise in our perf guides to disable it, but we cannot forcefully change this behavior as far as I know.


return_passages option returns incorrect passages

Hello,
I have tried to use the return_passages new feature, but it is returning passages that aren't crossed by the maritime path.
Here is an example:
import searoute as sr
import geopandas as gpd
import folium

traj = sr.searoute([52.99, 25.01], [-61.87, 17.15], append_orig_dest=True, restrictions=['northwest', 'chili'], return_passages=True)
print(traj['properties']['traversed_passages'])

gdf = gpd.GeoDataFrame.from_features([traj])
gdf = gdf.set_crs("EPSG:4326")
m = folium.Map(location=[0, 0], zoom_start=2)
folium.GeoJson(gdf).add_to(m)
m

The resulting traversed passages are:
['gibraltar', 'panama', 'sunda', 'malacca', 'ormuz', 'chili', 'suez', 'bosporus', 'babalmandab', 'south_africa']
When they should only be:
['gibraltar', 'suez', 'babalmandab', 'ormuz']
Here is a picture of the trajectory:

Thank you for your help!

Hi @LuciaCam , true, looks like the the new version is not fixing it.
Will commit the new patch soon.

Thanks for the prompt response and fix @genthalili , awesome job!

Hello again!
The fix for returning passages is now correct most of the time, but there are still cases for which it does not return the correct traversed passages:
import searoute as sr
import geopandas as gpd
import folium

traj = sr.searoute([140.02, 35.51], [-97.36, 27.81], append_orig_dest=True, return_passages=True)
print(traj['properties']['traversed_passages'])

gdf = gpd.GeoDataFrame.from_features([traj])
gdf = gdf.set_crs("EPSG:4326")
m = folium.Map(location=[0, 180], zoom_start=3)
folium.GeoJson(gdf).add_to(m)
m

This code chunk should return that panama has been crossed, but it returns an empty list of passages instead. Here is a picture of the trajectory:

I am using searoute 1.4.1, the latest release available from pip.

Hi, this is terrific.  But it should be 'chile' not 'chili'.

I was able to reproduce the issue, as you mentioned the reason is due to the "normalisation" of the lat/lon  when crossing from <-180 to 180.
Should be fixed in the next release.

For the chili, I used the french name for Chile which is Chili,  as I wanted to differentiate the country to the strait, but somehow keep it simple, the correct name should be magellan, as this is called : Strait of Magellan  :)
Most probably should change this to magellan!

Probably - I’m from Texas and chili is, well you know - chili!
'magellan'

On Aug 5, 2024, at 5:07 PM, Gent @.***> wrote:
For the chili, I used the french name for Chile which is Chili, as I wanted to differentiate the country to the strait, but somehow keep it simple, the correct name should be magellan, as this is called : Strait of Magellan  https://en.wikipedia.org/wiki/Strait_of_Magellan:)
Most probably should change this to magellan!
—
Reply to this email directly, view it on GitHub https://github.com/genthalili/searoute-py/issues/35#issuecomment-2270002376, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFIJ2JL4LCXMFLV5FFIYNQ3ZP7ZTTAVCNFSM6AAAAABJGUXR2KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZQGAYDEMZXGY.
You are receiving this because you commented.

https://www.thespruceeats.com/different-spellings-of-chile-2342844

@LuciaCam, could you please check with the new release version 1.4.2 and let me know if this is fixing the bug?

@genthalili The new release has fixed the issue for me! Thanks again 👍


replace query string with entire object from custom values

This is an issue pulled from the original repo.
Instead of iterating over query string values supplied in the custom values, it would be better to just inject the entire object as given, to avoid include params that were omitted in the custom values definition.

I am just trying to understand how do I provide query string value through custom value option. Any help is appreciated

@RajeshAjjarapu have you seen the custom values used for testing? test/process/documents/customValuesTest.json
Try copying this and stripping out what you don't need. Then specify its consumption using the --customValuesFile CLI flag or customValuesFile option field.

Thanks Noah for responding.  I am just trying to understand how do we generate this file or prepare custom values file.  For example,  I am trying to over ride the value in query string with value of my choice so that test case gets updated.
'qs': {"compartmentId":"elit labore","name":"elit reprehenderit minim laboris" ....

oopsie, my bad. Figured and it got updated.  If further questions, I will reach out thanks a lot

Hi Noah,
Any suggestions if we don't need to pass a query parameter itself ?

I'm not sure I understand. You want to exclude a query param? If yes, then that's what this issue is about, excluding query params that aren't explicitly defined in the customValues. It would allow you to supply an empty object for the query params and none would be included in the generated output.

You got it right.  ok. let's say I have 5 query params per spec. I want only two to be included in the request. I tried passing null values, but that query param is not being ignored.  We don't want to pass query params with null values.

Ok. That's what I was saying, this issue is about supporting explicit exclusion of query params. It is not support right now.


Fix installation env file

Removed your personal token from the env file, might wanna look at removing the reference to it.
If you don't know how, you could check out this.

Oh I totally forgot that it had some permissions I don't want people to be able to use. My bad. Will remove reference to it.


Getting QueueDoesNotExistException with local cloud.aws.sqs.endpoint cause cloud.aws.sqs.region is not set

Type: Bug
Component: SQS
Describe the bug
Getting QueueDoesNotExistException when testing locally with LocalStack even after setting both cloud.aws.sqs.endpoint and cloud.aws.sqs.region.
AFAIK, now that LocalStack supports multi region, the region must be provided, but it seems that the AmazonWebserviceClientFactoryBean won't set the region if a custom endpoint is provided: https://github.com/awspring/spring-cloud-aws/blob/218a4d1b66d90f4d6e2c76a9b778d71faec14447/spring-cloud-aws-core/src/main/java/io/awspring/cloud/core/config/AmazonWebserviceClientFactoryBean.java#L113-L128
The workaround is to define a properly configured AmazonSQS bean, but it would be nice if things would also just work for local endpoints.

AWS Java SDK does not allow setting both endpoint and region, so i am not sure really how to make it work. If you have a suggestion you're welcome to draft a PR a a subject to discussion.

Not sure I follow, do you mean the builder will throw an error if you set both? I'm pretty sure we've done that, but maybe it was the AWS SDK v2...
Anyway, the AwsClientBuilder.EndpointConfiguration constructor used above has a signingRegion that is being set to null. Setting that to the configured region should also work (it's what I did when creating a custom AmazonSQS bean as a workaround).

Yes indeed builder will throw in such case. I'll look again into setting signing region


Scandir does not always support int fd

In the Travis tests, some of the MacOS tests fail like this:
ERROR: test_index_access_to_stat_times_returns_int (pyfakefs.tests.fake_os_test.RealScandirFdRelTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/travis/build/jmcgeheeiv/pyfakefs/pyfakefs/tests/fake_os_test.py", line 4784, in setUp
    self.dir_entries = list(self.do_scandir())
  File "/Users/travis/build/jmcgeheeiv/pyfakefs/pyfakefs/tests/fake_os_test.py", line 5007, in do_scandir
    return self.scandir(self.dir_fd)
TypeError: scandir: illegal type for path parameter

This seems to depend on the used scandir version. The tests and the code shall be adapted to emulate the real behavior.

Seems to happen for Python 3.7 only under MacOS and Linux, but not under Windows.


Use gmtime_s on MinGW

gmtime_s is defined on MinGW, but gmtime_r is not.

Thank you for the PR and additional works for travis.


Replace swagger-ui with completely custom verzion

Not sure how the inner-workings are adaptable here, but is there a way to inject or otherwise replace the entire implementation of swagger-ui?
The only reason I ask, is I'd like to take advantage of some modifications done on other swagger-ui forks, such as https://github.com/jensoleg/swagger-ui for making swagger-ui responsive w/ twitter bootstrap.
This might already be possible with a combination of style injection, script injection, and index.html injection, and if so I'm open to enlightenment.

I would recommend disabling the embedded swagger-ui in Swashbuckle by removing the following call:
config.EnableSwaggerUi();

Once this is done, SB will just provide an endpoint for accessing the Swagger JSON document.
Then you can just pull down the latest swagger-ui assets (via bower or directly from repo) and just serve them from your website as static assets. In the index.html, just update the discoveryUrl to be that  provided by Swashbuckle

Okay, I knew I could omit the config.EnableSwaggerUi(); function call, but I didn't quite understand that, having done that, how any other swagger would reference the Swagger schema that Swashbuckle created.
What I've come to understand is, it's created that document accessible via a URL, not a physical file somewhere on the web server (e.g. I was browsing file explorer looking for some type of *.json file). Having figured out that I can reach the Swagger schema by using this url:
http://localhost:61344/swagger/docs/v1

I understand now how I can "de-couple" the Swashbuckle-swagger-ui and a static-swagger-ui.
Thanks for the clarification!


Question: How to use template in visual studio 2019

Hallo everyone, i have some trouble in vs 2019. I just downloaded template and crearte new project from template. Unfortunatally, in vs 2019 seems doenst show tempalte in designer on mainwindow.xaml, only if i execute it.
Code for startwindow got from tutorials and quick start.
<Controls:MetroWindow
    x:Class="ProjectfromTemplate.MainWindow"
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
    xmlns:Controls="http://metro.mahapps.com/winfx/xaml/controls"
    xmlns:iconPacks="http://metro.mahapps.com/winfx/xaml/iconpacks"
    Title="MainWindow"
    Width="525"
    Height="350"
    WindowStartupLocation="CenterScreen">
    <Controls:MetroWindow.RightWindowCommands>
        <Controls:WindowCommands>
            <Button Content="settings" />
            <Button>
                <StackPanel Orientation="Horizontal">
                    <iconPacks:PackIconModern
                        Width="24"
                        Height="24"
                        Kind="FoodCupcake" />
                    <TextBlock
                        Margin="4,0,0,0"
                        VerticalAlignment="Center"
                        Text="deploy cupcakes" />
                </StackPanel>
            </Button>
        </Controls:WindowCommands>
    </Controls:MetroWindow.RightWindowCommands>
    <Grid />
</Controls:MetroWindow>

i use visual studio 2019 and target .net framework 4.7.2.
P.S.: after i created new project from template , i update mahapps.metro libraries to newest 2.0 .
Can someone help me with my "question" or suggest workaround ?

Unfortunatally, in vs 2019 seems doenst show tempalte in designer on mainwindow.xaml, only if i execute it.

@Torchok19081986 Can you post a screenshot? Which MahApps version do you use?

Hallo, @punker76. Looks folowing

I use MahApps.Metro Version. 1.6.5
After i start my project my mainwindow looks folowing:

@Torchok19081986 The designer can't display the caption of a Window. So you will also don't see the window commands or min/max/close buttons. This is a designer limitation.


Correctly implement .source() for errors

This PR overrides the source() method of the Error trait to give the correct error source, and additionally modifies its Display implementation to no longer print sources (as it’s given by .source() instead). This makes programmatic access to the errors easiër.

Merged in https://github.com/rousan/multer-rs/commit/0f5f742dba85766ed09e9374053fa19e5da73f91. Thanks for this!


Getting an error on every login..

Date: 2020-05-28 18:43:14
ID: 1
Error occured in: Global
Count: 1
Message: ...AddOns\stat_weights_classic\stat_weights_classic.lua line 12841:
Usage: GetActionInfo(slot)
Debug:
[C]: ?

Is this something that started happening after an update? Or first time use?

After an update. Should i delete and redownload it? I always update my addons with the Twitch app

Haven't heard of any others experiencing this so this might be a special case, perhaps interactions with other addons. Try reinstalling it but make sure to clear the addon's cache in the middle of it. I think twitch has a way of doing this but if not go into this folder: World of Warcraft_classic_\WTF\Account\ACCOUNT_NAME\SERVER_NAME\CHARACTER_NAME\SavedVariables and delete both stat_weights_classic.lua and .bak files while the game is closed.

Tried it, same error.. =/
I have a feeling, that it is related to my macros somehow..
I'm gonna test later, if i get the error, without macros on my actionbars

Latest update didn't change anything.. =/
Have you been able to replicate the error with the addon Dominos?

Nope, Dominos with default settings works fine on my end.  What Dominos version are you using? I'm thinking that you have configured Dominos in a way that becomes problematic for this addon in some way.
Maybe you can try to back up / save your current Dominos settings and try using the default settings to see if the error goes away. Or maybe you are able to send me your Dominos settings and I can try it out myself.
I can´t see from your error report the nature of the issue but a function call to GetActionInfo(slot) seems to be the issue. Are you able to find out the value of the input variable (slot) is at the time of failure and what it is returning? My lua error logs usually shows a bit more information which might be from another addon but unsure. Or if you have a code editor handy, you can try putting print statements before and after the function call to GetActionInfo(slot) at the line where it fails to visualize input/output variables.

After trying different things, I now found out, that it has in fact something to do with my profile.. I'm guessing it is the scaling of my bars, because when I resinstalled Dominos it didn't give me any errors with the default settings. Now it gets weird though:
When I copied my backup settings back in, the error came back and switching to the default settings didn't help. However, when I reloaded with the default settings, the error didn't show up and stat_weights_classic was working fine. Switching back to my profile, it kept working as intended.
So my assumption is, it has something to do with my profile being first loaded..
I'm attaching my profile into this. Maybe you can take a look into it. Thank you btw for your patience lol
Dominos.zip

I was able to reproduce it using your settings. Strange issue, not sure why it's happening. However, I have pushed a commit as a workaround for this problem. You can try out the most recent master branch and see if it works.

It works! :D
Thank you for your efforts.
One of the best addons for sure.


Extension method for decimal with additional string parameter produces invalid code.

When I define an extension method for decimal with additional string parameter, Bridge.NET compiler tries to create a decimal object out of string.
Steps To Reproduce
https://deck.net/2815a2eef9f46c143d23bc1305fb029d
decimal a = 0;
a.M("text");

public static class Ex
{
    public static void M(this decimal a, string b) { }
}

Expected Result
var a = System.Decimal(0);
Demo.Ex.M(a, "text");

Actual Result
var a = System.Decimal(0);
Demo.Ex.M(a, System.Decimal("text"));

This code leads to System.FormatException: Invalid format.

The pull request with fixes resolving this issue has just been merged upstream! The fix will be available next Bridge version (see attached milestone).


Question: How to ensure my Intel Arc A770 GPU is used?

What is your question?
After reading the documentation, I am still not clear how to get my Intel Arc A770 GPU selected to work over my CPU. Any suggestions would be appreciated. Thanks in advance.

Hi, fabric does not host any LLM on his own, but uses different AI vendors.
E.g. you can use Ollama to host open source models and check Ollama docu how to ensure, that the GPU is used. Actually if you start Ollama you will see in output if the GPU is recognized and used or not.

So I agree and understand that Fabric itself is not the LLM and, however I apologize for my naivete up front but I guess my problem is tying ollama to Fabric and ensuring they work well together? I have gotten ollama to work with my Intel ARC A770 GPU recently on a side project using miniforge etc. But it's not consistent. I know that is not the concern of this project, but I love the project and what possibilities it has for everyone who can successfully run it, preferably on a GPU... HA! So.. LSS if you (or anyone) has any great suggestions or articles to try/read on my issue, I would greatly appreciate it.

use ollama ps to debug what device the model runs on. Models can be offloaded to RAM if they don't fit into VRAM.


[MAIN] update build_wheels action

Update the build_wheels action to use cibuildwheel version 2.13.1

coverage: 86.644% (-0.3%) from 86.977% when pulling 0b728ac61efe212e6809ad5b354abacc1e771c70 on INM-6:maintenance/update_cibuildwheel into 8fdca601b57eab746b694e05efa0663c9435cde4 on NeuralEnsemble:master.


confluent_kafka_acl resource does not allow use as principal "User:*"

Hi,
I am trying to use confluent_kafka_acl resource to grant all the user read operation over a consumer group resource but i am getting an error that says the principal should follow the pattern "User:sa-" or "User:u-" or "User:pool-"
This is the code i am using the create the acl
resource "confluent_kafka_acl" "wildCard-consumer-group-describe" {
  kafka_cluster {
    id = data.confluent_kafka_cluster.cluster.id
  }
  resource_type = "GROUP"
  resource_name = "consumer_group_name"
  pattern_type  = "LITERAL"
  principal     = "User:*"
  host          = "*"
  operation     = "DESCRIBE"
  permission    = "ALLOW"
}

resource "confluent_kafka_acl" "wildCard-consumer-group-allow-read" {
  kafka_cluster {
    id = data.confluent_kafka_cluster.cluster.id
  }
  resource_type = "GROUP"
  resource_name = "consumer_group_name"
  pattern_type  = "LITERAL"
  principal     = "User:*"
  host          = "*"
  operation     = "READ"
  permission    = "ALLOW"
}

This is the error message i got
╷
│ Error: invalid value for principal (the principal must start with 'User:sa-' or 'User:u-' or 'User:pool-'.)
│
│   with confluent_kafka_acl.wildCard-consumer-group-describe,
│   on allow-all.tf line 49, in resource "confluent_kafka_acl" "wildCard-consumer-group-describe":
│   49:   principal     = "User:*"
│
╵
I want to grant all the users permission over that consumer group. I dont want to create a acl for each user.

Thanks for creating this issue @rociomeras!
I'm afraid you might have to create an ACL for each users or you might also try confluent_role_binding resource.

The confluent_role_binding resource also does not allow use as principal "User:*"
The confluent API allow me to create the acl using "User:*" as principal. I can't think in a valid reason for not allow that form the terraform provider. At the end, the terraform provider is a wrapper for the confluent API
Example of the call that work
curl --location --request POST 'https:CLUSTER_ENDPOINT/kafka/v3/clusters/lkc-127zyj/acls'
--header 'Authorization: Basic TOKEN'
--header 'Content-Type: application/json'
--data-raw '{
"resource_type": "TRANSACTIONAL_ID",
"resource_name": "",
"pattern_type": "LITERAL",
"principal": "User:",
"host": "*",
"operation": "ALL",
"permission": "ALLOW"
}'

@rociomeras I double checked with a team and I was told that currently the Admin Client allows such requests so your suggestion looks accurate to me and that's something that should be added to TF Provider.

Hi, I too ran into an issue with this and submitted a PR #158  to address this. I have tested it out against my cluster and it seems to function well. But as I state in the PR I'm a novice with go and have no experience with wiremock so I imagine there is more I need to do to create a higher quality PR.

Thanks for creating a PR! We merged a fix internally and hopefully we'll release it next week.
fyi @joshholl @rociomeras

thank for the PR @joshholl . I will be checking the new release when is ready

I synced with my team and unfortunately we might have to wait until our internal code freeze is lifted (which is going to happen in 2023 😕).

👋 @joshholl @rociomeras
The fix has been released in our latest version of TF Provider.

@linouk23 thank you for seeing this through!


Fix feature flaw of history command

Restrict the usage of data-modifying commands such as 'delete', 'mark' and 'edit' when user is in history view to prevent unintended changes
For other commands such as 'find', 'history' and 'add' are still executable in history view as they do not alter the existing data ('add' is still accessable as it does not depend on the person list)

closes https://github.com/AY2425S1-CS2103T-F14b-3/tp/issues/107

Just wondering why the history command is not restricted since it also uses index? I assume this fix is because the user is unable to see the index in call history view so we're restricting all commands that require index. Not sure why history command is left out of the restricted commands?

I only restrict the data-modifying commands as they might change the existing data. Since history command is only for navigation, so there's no harm if user exceutes it incorrectly, to make the user more freely to explore the history of multiple people.


Added function to return maven command example

Output for maven plugin is now as follows:
[ERROR] Failed to execute goal io.openliberty.tools:liberty-maven-plugin:3.3-SNAPSHOT:dev (default-cli) on project demo-devmode: The server has not started within 3 seconds. Consider increasing the server start timeout if this continues to occur. For example, 'mvn liberty:dev -DserverStartTimeout=90' -> [Help 1]


Allow sending menu as a followup if interaction was already responded…

… to before

This is good fix. Looked at the discord.py docs to see if there was a better way but this does the job. Simple and it works.
Thank you.


--id parameter info wrong

neither service principal name or object id work when reseting the sp credentials. only client id.
Required Parameters
--id
"Service principal name, or object id."
tommydunn% az ad sp credential reset --id 30d7b452-e45c-<object_id>
Resource '30d7b452-e45c-<object_id>' does not exist or one of its queried reference-property objects are not present.

tommydunn% az ad sp credential reset --id b2c-sp
Service principal 'b2c-sp' doesn't exist

tommydunn% az ad sp credential reset --id babc9695-10ff-<client_id>
The output includes credentials that you must protect. Be sure that you do not include these credentials in your code or check the credentials into your source control. For more information, see https://aka.ms/azadsp-cli
{
  "appId": "babc9695-10ff-<client_id>",
  "password": "",
  "tenant": ""
}

tommydunn% az version
{
  "azure-cli": "2.41.0",
  "azure-cli-core": "2.41.0",
  "azure-cli-telemetry": "1.0.8",
  "extensions": {}
}

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

ID: 7bef1f12-de16-93ed-1cb7-3f3d5d99d786
Version Independent ID: 4e9329a6-50f8-ee2a-af63-f750d66ed574
Content: az ad sp credential
Content Source: latest/docs-ref-autogen/ad/sp/credential.yml
Service: active-directory
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

@jiasli for awareness


The SED is  no longer in the child table of a Parent, the Parent still respond Data Request(802.15.4) with ACK(802.15.4). Is this behavior as expected?

Yes, this is expected. IEEE 802.15.4 ACKs are generated at the MAC layer without any input from higher layers.
If a child wants to validate that it is still in its parent's child table, it needs to have a protocol exchange at a higher layer - for example, something that involves exchanging IPv6 datagrams.

As Jonathan said, yes, expected  behavior.
You can check  "Child Supervision" feature in OT as a mechansim related to this.


Fix missing update release body

This PR correct the bug I was getting in #362.
This problem was that draft aren't updated once created.
I've rewrote how the function release handle a draft and a non-draft release to be processed in the same way.
Closes #362

Bumping this PR since I'm facing the same issue. At times the workflow I'm generating will create a new body which I want to overwrite what's contained in a draft release. Currently the draft stays the same.

Hello @softprops, I'm pinging you in hope that you could review this PR :)

@FirelightFlagboy can you fix the merge conflicts? Thanks!

@FirelightFlagboy thanks for helping out.


Revert "Audits"

Reverts TobikoData/sqlmesh#2255

@tobymao Is there a plan to un-revert this improvement? This would be very useful as it comes up quite a bit for us and defining custom audits each time is tedious.

nvm, this PR was closed, not merged... I was confused since I could not find documentation about this to confirm this was now supported...

looks like it's supported https://github.com/TobikoData/sqlmesh/pull/2255


Support Parental Control

Is your feature request related to a problem?
When parental control is enabled on device, YouTubePlayerKit remains stuck in progress.
What solution would you like?
At the minimum it would be great if it can catch the following error and propagate up to the API client:
0x158c23e18 - [pageProxyID=15, webPageID=16, PID=1373] WebPageProxy::didFailProvisionalLoadForFrame: frameID=20, isMainFrame=0, domain=WebKitErrorDomain, code=105, isMainFrame=0, willInternallyHandleFailure=0
An ideal solution would be to make it work with parental control. Enabling youtube.com does not seem to be sufficient...
What alternatives have you considered?
I have not found a workaround
Any additional context?
No response

Hi @kiddosai,
Please check out the develop branch which contains a new commit which added a didFailProvisionalNavigation case to the YouTubePlayer.Error


Auth token not attached from current user

When I try to make requests for most endpoints I will receive a forbidden error.
I've noticed I can only successfully hit public permission based items. It seems the auth token for the signed in user is not being passed along.
The same is true if I set the api keys within Authentication section.
Using Directus v10.5.3 w/ v1.1.1 of the module

Thanks for raising the issue, I haven't seen it myself but will test and fix it if I find the same


Javascript Quiz Game

Describe the project you want to add with tech stack

I want to add a Javascript Quiz Game made with HTML, CSS and Javascript

Expected behavior

A Quiz Game which shows the score after the game ends.

Screenshots (optional)

Additional context (optional )

looking forward to resolve this issue, kindly assign me with the particulars.

looking forward to resolve this issue, kindly assign me with the particulars.

I want to add a Javascript Quiz Game made with HTML, CSS and Javascript.
The UI for game would be user friendly & responsive and there are random problem set allotement to user which he have to solve in a time limit and at the end there would be scorecard for his performance.
Excited  to resolve this issue, kindly assign me with the particulars.

hello there, I am interested in this issue could you plz assign this to me.


8 code review of mikolovs code for word2vec

Word2Vec
Build on Windows10 and training word2vec over http://mattmahoney.net/dc/text8.zip dataset, output is the vectors.bin file which contains vector representation of each word in data set.

https://github.com/AlexisTercero55/AI-Research/issues/8


LightState is missing transitiontime

Per the documentation under section 1.6
http://www.developers.meethue.com/documentation/lights-api
Setting the light state supports transitiontime in the body arguments:

The duration of the transition from the light’s current state to the new state. This is given as a multiple of 100ms and defaults to 4 (400ms). For example, setting transitiontime:10 will make the transition last 1 second.

Now given the current shape of the LightState model, I could see how "transitiontime"  doesn't necessarily fit, especially if a LightState represents the current state of a light; transitiontime is a bit awkward perhaps with its temporal quality.
If LightState is the wrong place, the case could also be made that the signature for
public func updateLightStateForId(_ identifier: String, withLightState lightState: LightState, completionHandler: BridgeSendErrorArrayCompletionHandler)

should then be updated to support transitionTime: Int? = nil similar to createSceneWithName
Do you have a preference?

This would be the change in updateLightStateForId
    var parameters = lightState.toJSON()!

    if let transitionTime = transitionTime{
        parameters["transitiontime"] = transitionTime
    }

See #13


Update permissions bugfix and acl command fix

About this change - What it does
There was an issue when new role is created and adding new permissions. this change fixes it and permissions are saved.
Resolves: #73 #75
Why this way
There was no tenant id set when saving permissions, and hence permissions are being added randomly to a unexisting tenantid. It's a bug, and fixed now.

is there a way to add a test covering this change?

Let's add a test for this case

@jlprat @snuyanzin  Nice. Added tests for roles and permission updates


Cannot use C++ on APPLICATION level initialization

Describe the bug
Cannot use C++ in functions called at APPLICATION initialization level.
Because setup of C++ subsystem is performed after APPLICATION level initialization then C++ usage is limited to main & following threads.
To Reproduce
Steps to reproduce the behavior:

Define static instance of class with non trivial constructor
Use SYS_INIT macro to define function called at APPLICATION level
Try to use C++ object in function defined in point 2
Object is not properly constructed

Expected behavior
C++ sybsystem should be avialable at APPLICATION initialization level.
It would be great if the POST_KERNEL level will be also usable from C++ but maybe I am going too far.
If it is not possible (i.e. because of Zephyr design philosophy) then this limitation should be clearly mentioned in the documentation.
Impact
Annoyance.
Screenshots or console output
Environment (please complete the following information):

OS: Linux
Toolchain: Zephyr SDK
Commit: 6789ecbd5e13cc19e69f25314cf486493df9dde3

Additional context

I agree; I personally would like this to occur even earlier, but there was strong pushback at the suggestion that C++ might be used in drivers.
The documentation says it's supported for applications.  It is ambiguous whether that includes initialization routines run at the application init level, but I think it's worth trying.


Port blaze-core to cats-effect-3

This one is for the brave.  Will require a knowledge of Cats-Effect and Future.

I think that we can put this task into progress.
There is a PR for it: https://github.com/http4s/http4s/pull/3894

I still owe that one a review and will aim to get to it this weekend.

Fixed by #3894.


Update godog dependency

One more thing though... We'll probably have to make analogous changes to Clicumber


Is it necessary to have monthly indexes?

When inspecting Elasticsearch indexes, I notice that API Umbrella is creating an index for every month:

The Elasticsearch documentation describes an index as:

a collection of documents that have somewhat similar characteristics.
Elasticsearch 2.0 Basic Concepts Index

Since the API Umbrella analytics documents do not change significantly in structure, what is the benefit of having monthly indexes?

There's a few reasons for this, but it is common practice when dealing with time-based data in Elasticsearch (Logstash uses daily indexes by default). Here's a bit more detail:
https://www.elastic.co/guide/en/elasticsearch/guide/current/time-based.html
https://www.elastic.co/guide/en/elasticsearch/guide/current/retiring-data.html
Another reason I believe (but can't find a link handy) is that it allows for more efficient querying (by basically partioning the data, if you're only querying 1 month of data, you can explicitly name the indexes involved in that month, rather than relying on those time ranges to be be filtered at query time).
It can also be more beneficial to shift to more granular indexes (like daily), but it depends on how much data you have, how many servers, etc. In a future update, we will probably make it easier to configure different time-frames (so if you'd prefer daily/weekly/monthly indexes, you can do so).
Hope that helps, but let us know if you have any other questions.


Update README.md

I'm so sorry! I forgot writing a preposition 'with' before 'the WASD key' for #3! Well, I still can't use English correctly. And SORRY about this extra PR again.

Thank you!


Permissions for menu items / screens

In React client, it should be possible to configure different available menu structure for users with different roles.
We / users should understand that such permissions are checked only in UI and don't prevent anyone from calling public GraphQL / REST endpoints.
depends on https://github.com/Haulmont/jmix-security/issues/94

QA notes:
menus permissions are responsible for MenuItem and SubMenuItem disaplaying. If user doesn't have permission for current MenuItem or SubMenuItem then they are not be displayed. Mapping with menus permissions and MenuItem/SubMenuItem works by "key" property of MenuItem/SubMenuItem.
screens permissions are responsible for screen content disaplaying (in tab or bradcrumb). If user doesn't have permission for current screen then its content is not be displayed and user will see text "You don't have access to this screen" (depends with locale). Mapping with screens permissions and screens in application works by "screenId".
If screens or menus permissions has item with target equal *, it means that user has permissions for all menu items or screens.
Permission checking for menus and screens could be off. It works by REACT_APP_ENABLE_UI_PERMISSIONS env variable.  If you need to enable checking just set value of REACT_APP_ENABLE_UI_PERMISSIONS to true


Flutter build fails Java 11 required

Hello, colleagues!
Everything was going well. But I had to upgrade gradle to use Firebase, now it is failing build due to this error:

What went wrong:
A problem occurred evaluating project ':app'.

Failed to apply plugin 'com.android.internal.application'.
Android Gradle plugin requires Java 11 to run. You are currently using Java 1.8.
Your current JDK is located in  /Users/runner/hostedtoolcache/Java_Temurin-Hotspot_jdk/8.0.372-7/x64/Contents/Home/jre
You can try some of the following options:
- changing the IDE settings.
- changing the JAVA_HOME environment variable.
- changing org.gradle.java.home in gradle.properties.

How can I use Java 11 with this plugin?
Thank you!

I got it.
Just had to add this before FlutterInstall@0:
      - task: JavaToolInstaller@0
        inputs:
          versionSpec: '11'
          jdkArchitectureOption: 'x64'
          jdkSourceOption: 'PreInstalled'


Improve examples by not using "from amuse.lab import *"

Using "from amuse.lab import *" is easy, but makes it impossible to see where an imported object is defined. It would be better if we are explicit in importing objects, especially in the examples (which can be expected to serve as basis for new scripts).
The imports could still be from amuse.lab, but instead of the line
from amuse.lab import *

we could have e.g.
from amuse.lab import (units, nbody_system, Hermite)

Or, perhaps preferably,
from amuse.units import (units, nbody_system)
from amuse.community.hermite0.interface import Hermite

I'm happy to gradually modify examples if we agree this would be better.

+1

I can see two benefits to the current setup;

When changing the scripts to suit you needs, you do not have to learn / investigate the entire amuse structure to see from where a specific name needs to be imported
The 'from amuse.lab import *' decouples the public view of AMUSE from the internal structure, this we could use to change some of the internal structure without changing the external scripts.

I can also see a benefit of importing the specific module; this forces us to have a  less complex internal structure and make it clean and useful.
I do not want to change  'from amuse.lab import *' to 'from amuse.lab import Hermite' etc, it only increases the number of times a name has to be given. It also removes the ease of extending the script.

I am for clean&useful structure, and de-obfuscation! ;-)  I also agree that
from amuse.community.hermite0.interface import Hermite is better than both versions of amuse.lab import...

When changing the scripts to suit you needs, you do not have to learn / investigate the entire amuse structure to see from where a specific name needs to be imported

Sure, that's a benefit. But I don't think it's a major one, it 's easy enough to figure out where names are imported from. Especially if we use them in examples (as we do in some but not all currently).

The 'from amuse.lab import *' decouples the public view of AMUSE from the internal structure, this we could use to change some of the internal structure without changing the external scripts.

Except we don't change that structure? At least, not without simply deprecating the old structure while still keeping it working... So I don't see this as a problem for using specific imports.

I do not want to change 'from amuse.lab import *' to 'from amuse.lab import Hermite' etc, it only increases the number of times a name has to be given. It also removes the ease of extending the script.

I don't think that's the only thing it changes (it still becomes clearer where objects come from) - but I agree with Inti also, that specific, direct imports are better than using amuse.lab in any way.
I am not saying we should drop support for amuse.lab altogether, but I think we should not use it in our own (example) scripts.

Quote from the upcoming book by @spzwart and @stevemcmillan:

Either way, once Python has started, you can load AMUSE by
from amuse.lab import *
This line is a lazy but very convenient way to import the entire AMUSE package in a single call. We generally do not advise it because it pollutes the Python name space. Rather, we recommend that, wherever possible, individual packages be imported from amuse.lab by name. Still, in many examples below and in later chapters we are unable to resist the simplicity of importing the entire package. Throughout this manuscript we will present source code in this way, and will assume that the appropriate AMUSE libraries have been loaded.

So I don't want to change this in the textbook examples without hearing from them if this was done as a didactic choice (for e.g. reasons Arjen already gave), or because of just being unable to resist the temptation ;-).

This was done for brevity in the printed version of the book, Steven.
I agree, it's bad style, but it saves many pages and allows the reader
to focus on the source rather than the headers.
On 09/28/2018 07:13 AM, Steven Rieder wrote:

Quote from the upcoming book by @spzwart https://github.com/spzwart
and @stevemcmillan https://github.com/stevemcmillan:
Either way, once Python has started, you can load AMUSE by
|from amuse.lab import *|
This line is a lazy but very convenient way to import the entire
AMUSE package in a single call. We generally do not advise it
because it pollutes the Python name space. Rather, we recommend
that, wherever possible, individual packages be imported from
amuse.lab by name. Still, in many examples below and in later
chapters we are unable to resist the simplicity of importing the
entire package. Throughout this manuscript we will present source
code in this way, and will assume that the appropriate AMUSE
libraries have been loaded.

So I don't want to change this in the textbook examples without
hearing from them if this was done as a didactic choice (for e.g.
reasons Arjen already gave), or because of just being unable to resist
the temptation ;-).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/amusecode/amuse/issues/310#issuecomment-425403336,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AETyzIy5fmhj5vqddwTi9BY0ko-VIMmFks5ufgRGgaJpZM4W62FG.

I read this in horror.

that needs an explanation;-)

really?
I also tried to cut and paste (what else can one do) some examples. They all worked, but two strugglings came to a comment:

it's python2 still, but i know you guys are working on it
why oh why do we need those >>> everwhere. I like double or triple click, but i can't paste a few lines to get work done quicker. this is really not needed if you use color to your advantage. Anything to be cut and paste is in color, screen output normal.  I didn't understand why some words were highlighted either.  Here an example of the style I'm hoping for:
https://teuben.github.io/nemo/examples/sh76.html

Hi @teuben,

it's python2 still, but i know you guys are working on it
Indeed. If you wish, you can check out the python3 branch, which is nearly ready for deployment.

why oh why do we need those >>> everwhere. I like double or triple click, but i can't paste a few lines to get work done quicker. this is really not needed if you use color to your advantage. Anything to be cut and paste is in color, screen output normal.  I didn't understand why some words were highlighted either.  Here an example of the style I'm hoping for:
https://teuben.github.io/nemo/examples/sh76.html

I'm sorry but I don't understand this comment, where do you see "those >>>"?
The examples are fully working scripts, so their appearance depends on what you view them in.

@stevemcmillan @spzwart @ipelupessy and myself decided that we will change the examples to use explicit imports, from lab.py.
lab.py in practice will be an environment that has everything needed for at least the textbook examples.

I think the work on this is finished. reopen new issue if necessary

No it's not finished...
The textbook examples need revisiting now that a v2 of the book is being discussed.

better open a fresh issue for this

yea, but do we need the discussion above? in any case maybe a checklist of things to fix?


Feedback

One addition to the sidebar that we would like to make is the link for ‘Questions or Feedback’ as shown on the St. Paul, MN site https://www.adopt-a-drain.org/. It is located on their sidebar, at the very bottom. It provides a direct link to their email address:

I'm big on feedback. It is a great way to gauge your users' needs, but it requires and consumes resources.  It doesn't run by itself and will require the time of many people in your organization to be effective.
We may want to explore best practices before setting this up.

Lets chat more about this in the future!


chore(main): release 0.0.14

:robot: I have created a release beep boop
0.0.14 (2022-11-27)
Bug Fixes

release-please: ARRRG! (7f509c2)
release-please: ARRRG! v2 (ddef925)
release-please: ARRRG! v3 (eeb0c54)
release: Hopefully fixes the deliverables. (45953fd)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/shift/local_exporter/releases/tag/v0.0.14 :sunflower:


[Bug]: Get incorrect partition field name when refresh tables

What happened?
Some tables have customized partition aliases, but in ArcticDataFiles#genericRecord they are extracted based on the default partition aliases, which can lead to throwing NPE errors for not getting the correct fields
Affects Versions
master/0.5.x
What engines are you seeing the problem on?
Core
How to reproduce

create a table with custom partition alias
enable self-optimizing
throw NPE when refresh table

Relevant log output
Refreshing table dls_sjc_iceberg.diag.matsparticipantquality(tableId=1050840) failed.
java.lang.ArrayIndexOutOfBoundsException: null

Anything else
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

Could you provide the complete error log stack trace? Perhaps we should consider the implementation logic of the ArcticDataFiles class comprehensively.

Could you provide the complete error log stack trace? Perhaps we should consider the implementation logic of the ArcticDataFiles class comprehensively.

The full log stacks were dropped since JVM deault optimize stacks if there are too many same exception stacks, refer to https://stackoverflow.com/questions/58696093/when-does-jvm-start-to-omit-stack-traces


Add Files

Add the NPM Package files and
npm author add salvage_dev

Hey, sorry about the delay.
node-caps is going to have a massive rewrite soon. Adding styling and much more!
I will start working on this soon. @simp-node add the original files to the repo and I'll start a code base re-write.
The codebase will be rewritten to es6 and will have TypeScript support.

I will soon, leme re-add the repo and push the files


[Feature request] Upload to selected Imgur account and album

1. Request
It would be nice, would be possible to upload gif or apng to selected Imgur account and album.
2. Argumentation
I organize my images on albums. It brings more order to affairs.
3. Examples in other apps
Imgur API allow to solve this. ShareX supports upload to selected account and album:

Thanks.

I second that.

I'm still polishing it, but it will look like this:

You can select the album right away or pick one before uploading.

@NickeManarin:
Status: :crying_cat_face: Not fixed
1. Environment

ScreenToGif 2.13.1

2. Steps to reproduce
I authorize to my Imgur account in browser.
For ScreenToGif: Options → Upload Services → Get OAuth token
3. Expected behavior
Successful get OAuth token.
4. Actual behavior

Woah there!
The information you sent wasn't quite correct. Please return to the place that sent you to this page and try again.

Thanks.

I forgot to update the Chocolatey package.
In a couple of hours it should be updated.

Status: :heavy_check_mark: Fixed for me in 2.13.3
Thanks!

Type: Feature request
1. Summary
@NickeManarin, could you do the same for .apng files?
2. Behavior
2.1. Expected
.gif:

2.2. .apng
No Upload the file option:

3. Note
Also, it would be nice, if would be possible upload .gif and .apng files to different Imgur albums.
Thanks.

@Kristinita I believe so.


Removes templating on data type.

You did not run update_cmakelists.py

Yep, saw that. However, it's needed on cartographer, not cartographer_ros. Working on it. Strange that Travis didn't catch it...


Update docs for HOST_OS, TEMPFILE and Split

The HOST_ARCH/TARGET_ARCH functions have been defined in two places, generic (Platform.xml) and Windows, these are consolidated into one, leaving a comment in the msvc.xml file to show where that version went to.  HOST_OS/TARGET_OS also got some tweaks - this was in line with these now being set by default for all platforms.
The $TEMPFILE construction var was not defined; added and the whole family had docs updated. (due to the addition, the generated entity definitions were also included in the PR).
A small tweak to Split wording.
References to Python changed to entity references (there are still more of these to change).
lambda as an entity reference is not correct: generated docs show the symbol, but Python does not use the lambda symbol to refer to lambda expressions.
This is a doc-only change.
Signed-off-by: Mats Wichmann<EMAIL_ADDRESS>Contributor Checklist:

[ ] I have created a new test or updated the unit tests to cover the new/changed functionality.
[ ] I have updated CHANGES.txt (and read the README.rst)
[X] I have updated the appropriate documentation

OKay, stepping back a bit, I thought I had the HOST/TARGET things sorted.  Thought again, here's current view:

Currently set by platform: HOST_OS, HOST_ARCH
Not set by platform, defaulted by Environment to None: TARGET_OS, TARGET_ARCH
Used by MSCommon: HOST_ARCH, TARGET_ARCH; if TARGET_ARCH from the env is false, set to HOST_ARCH
Used by other compiler tools: none
Not used by anybody: HOST_OS, TARGET_OS
Immutable: none enforced. Should be HOST_*? (pls confirm)

Okay, stepping back a bit before submitting new version, I thought I had the HOST/TARGET things sorted. Thought again, here's current view:
* Currently set by platform: `HOST_OS`, `HOST_ARCH`

* Not set by platform, but defaulted by `Environment` to `None`: `TARGET_OS`, `TARGET_ARCH`

* Used by `MSCommon`: `HOST_ARCH`, `TARGET_ARCH`; if `TARGET_ARCH` from the env is false, set to `HOST_ARCH`

* Used by other compiler tools: none

* Not used by anybody: `HOST_OS`, `TARGET_OS`

* Immutable: none enforced. Should be `HOST_*`? (pls confirm)

Sounds about right.

I think we can leave the "not used by anyone" bits out of the docs.. ;)

Indeed..  that's part of making sure it's sorted, but not part of the proposal

So... should we set TARGET_OS and TARGET_ARCH if not otherwise set? As in the current Environment.py code:
        self._dict['HOST_OS']      = self._dict.get('HOST_OS',None)
        self._dict['HOST_ARCH']    = self._dict.get('HOST_ARCH',None)

        # Now set defaults for TARGET_{OS|ARCH}
        self._dict['TARGET_OS']      = self._dict.get('TARGET_OS',None)
        self._dict['TARGET_ARCH']    = self._dict.get('TARGET_ARCH',None)

could easily become:
        # the platform should initialize these, just backstop that
        self._dict['HOST_OS'] = self._dict.get('HOST_OS', None)
        self._dict['HOST_ARCH'] = self._dict.get('HOST_ARCH', None)
        self._dict['TARGET_OS'] = self._dict.get('TARGET_OS', self._dict['HOST_OS'])
        self._dict['TARGET_ARCH'] = self._dict.get('TARGET_ARCH', self._dict['HOST_ARCH'])

So... should we set TARGET_OS and TARGET_ARCH if not otherwise set? As in the current Environment.py code:
        self._dict['HOST_OS']      = self._dict.get('HOST_OS',None)
        self._dict['HOST_ARCH']    = self._dict.get('HOST_ARCH',None)

        # Now set defaults for TARGET_{OS|ARCH}
        self._dict['TARGET_OS']      = self._dict.get('TARGET_OS',None)
        self._dict['TARGET_ARCH']    = self._dict.get('TARGET_ARCH',None)

could easily become:
        # the platform should initialize these, just backstop that
        self._dict['HOST_OS'] = self._dict.get('HOST_OS', None)
        self._dict['HOST_ARCH'] = self._dict.get('HOST_ARCH', None)
        self._dict['TARGET_OS'] = self._dict.get('TARGET_OS', self._dict['HOST_OS'])
        self._dict['TARGET_ARCH'] = self._dict.get('TARGET_ARCH', self._dict['HOST_ARCH'])

Nope.
We need to know if there's no explicitly requested TARGET_{OS|ARCH}, which allows SCons to pick whatever is available and applicable to the host system..

It doesn't really affect that, a requested value will still be passed on to set:
env1 = Environment()
print(f"{env1['TARGET_OS']}")

env2 = Environment(TARGET_OS="minix")
print(f"{env2['TARGET_OS']}")

giving:
$ python ../../scripts/scons.py -Q
posix
minix
scons: `.' is up to date.

That's confusing.. Here's a grep for TARGET_OS in the source code
% pythongrep TARGET_OS
SCons/Environment.py:        self._dict['TARGET_OS']      = self._dict.get('TARGET_OS',None)

There's no default value.
Also when I run your example I get
% python scripts/scons.py -f ../../tmp/sconstruct_target_os.py
scons: Reading SConscript files ...
None
minix
scons: done reading SConscript files.
scons: Building targets ...
scons: `.' is up to date.
scons: done building targets.

When I wrote/modified the TARGET_* code, initially I did initialize to be HOST_, but then I needed to know if the user explicitly requested a TARGET_, and so changed the default to None.

Sigh... irritating non-fail Fail:
 Failed the following test:
	test/ninja\iterative_speedup.py
Error: Process completed with exit code 1.

Also when I run your example I get

% python scripts/scons.py -f ../../tmp/sconstruct_target_os.py
scons: Reading SConscript files ...
None
minix
scons: done reading SConscript files.
scons: Building targets ...
scons: `.' is up to date.
scons: done building targets.

right... that's the current way. defaults to None, if user supplied TARGET_OS it's picked up.  the paste was to show that still happens, even if TARGET_OS fails over to $HOST_OS.

When I wrote/modified the TARGET_* code, initially I did initialize to be HOST__, but then I needed to know if the user explicitly requested a TARGET__, and so changed the default to None.

I'm not sure there was a need to distinguish between TARGET_ARCH=x86_64 because scons defaulted it there or because the user asked for it when creating the env.  in either case you'll get that target arch... if the user asked for something it's still passed through, as after this default stanza is run in the env creation, the next thing is the kwargs are handled, and so the one the user set there will be used.
anyway, no change to the code in the current PR, just flogging the topic to death.


PWA-252 - fixes for the filter chips

correct max-height for filter chips wrapper
filter chips "more" button visibility is evaluated more often

The space between the chip and the more button looks wrong

Pull Request Test Coverage Report for Build 1585

2 of 7 (28.57%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 38.11%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pages/FilterAttribute/connector.js
0
1
0.0%

components/ChipLayout/index.jsx
2
6
33.33%

Totals

Change from base Build 1583:
-0.03%

Covered Lines:
1488

Relevant Lines:
3876

💛  - Coveralls


STATE_FAILED with docker-compose sandbox

I prepared two Dell R330 servers, one for TINKERBELL HOST, another for TINKERBELL CLIENT, and two servers eth0 connected to the same switch (Cisco 9200L -48T). Both two servers have Internet access through the switch.
I tried less than 10 times, but TINKERBELL_CLIENT always stuck in the LinuxKid screen, on the TINKERBELL_HOST terminal showed sandbox-workflow   ubuntu-focal   STATE_FAILED
Please I need some help with debugging.
Environment
Cisco 9200L -48T Switch
TINKERBELL HOST OS: Ubuntu 22.04.3 LTS (16G RAM, 2T HDD disk)

UFW status: inactive
NetworkManager status: stop and disable
There no selinux in Ubuntu I think.
os ip: <IP_ADDRESS>/24
os gw: <IP_ADDRESS>
Docker version: 24.0.6, build ed223bc
Docker compose version: v2.21.0
kubectl version: Kustomize Version: v4.5.7

TINKERBELL CLIENT : (8G RAM, 3T HDD disk)

MAC address: 34:17:eb:ee:fd:fb
Client ip: <IP_ADDRESS>/24
Client gw: <IP_ADDRESS>
And can sure the disk only has sda, and no installed os before starting the composer.

I follow the steps in the docker-compose sandbox, https://github.com/tinkerbell/sandbox/blob/main/docs/quickstarts/COMPOSE.md
This is my .evn file below:
# Can be set to your own hook builds
vOSIE=v0.8.0
OSIE_DOWNLOAD_URLS=https://github.com/tinkerbell/hook/releases/download/${vOSIE}/hook_x86_64.tar.gz,https://github.com/tinkerbell/hook/releases/download/${vOSIE}/hook_aarch64.tar.gz

# This is the IP and MAC of the machine to be provisioned
# The IP should normally be in the same network as the IP used for the provisioner
TINKERBELL_CLIENT_IP=<IP_ADDRESS>
TINKERBELL_CLIENT_MAC=34:17:eb:ee:fd:fb

# These are the Gateway and DNS addresses the client should use, required for tink-worker to pull action images
TINKERBELL_CLIENT_GW=<IP_ADDRESS>
TINKERBELL_CLIENT_NAMESERVER_1=<IP_ADDRESS>
TINKERBELL_CLIENT_NAMESERVER_2=<IP_ADDRESS>

# This should be an IP that's on an interface where you will be provisioning machines
TINKERBELL_HOST_IP=<IP_ADDRESS>

# Images used by docker compose natively or in terraform/vagrant, update if necessary
BOOTS_IMAGE=quay.io/tinkerbell/boots:v0.7.0
HEGEL_IMAGE=quay.io/tinkerbell/hegel:v0.8.0
TINK_VERSION=v0.8.0
TINK_SERVER_IMAGE=quay.io/tinkerbell/tink:${TINK_VERSION}
TINK_CONTROLLER_IMAGE=quay.io/tinkerbell/tink-controller:${TINK_VERSION}
TINK_WORKER_IMAGE=quay.io/tinkerbell/tink-worker:${TINK_VERSION}
RUFIO_VERSION=v0.1.0
RUFIO_IMAGE=quay.io/tinkerbell/rufio:${RUFIO_VERSION}
K3S_IMAGE=rancher/k3s:v1.24.4-k3s1

# This is the boot/primary disk device and the device for its first partition
# for the machine to be provisioned (as it would appear with lsblk)
DISK_DEVICE=/dev/sda
DISK_DEVICE_PARTITION_1=/dev/sda1
# Example for a device with an NVME SSD
#DISK_DEVICE=/dev/nvme0n1
#DISK_DEVICE_PARTITION_1=/dev/nvme0n1p1

Current Behaviour

When Starting the provisioner on TINKERBELL HOST, there is no error output.

When continuing step 5, use the command below on TINKERBELL HOST:
KUBECONFIG=./state/kube/kubeconfig.yaml kubectl get -n default workflow sandbox-workflow --watch

Frist time it showed: sandbox-workflow   ubuntu-focal   STATE_PENDING
Then I power on the TINKERBELL CLIENT Server, TINKERBELL CLIENT can normal booting up through ipxe.
And TINKERBELL HOST terminal showed like that:

The TINKERBELL CLIENT screen of  show like that:

It is always stuck on this screen. When I reboot the TINKERBELL CLIENT, It seems to installed the Ubuntu successfully, but can't log in with the username tink and password tink, also I can't ping successfully from TINKERBELL HOST.

Possible Solution

This is my docker composer log in the below URL, can someone figure out where the problem is?
If there are other tools that can be used for debugging,  I would appreciate that.
https://8tako.com/tinker/ubuntu_docker_composer.log

What's your tinkerbell client output with command " cat /proc/cmdline"?

The maintainers don't test a whole lot with the compose setup; typically we use k3d to create a light weight Kubernetes cluster and deploy the Tinkerbell stack using Helm.
Support for compose is falling rapidly by the wayside and will probably be removed soon so I suggest you change over.


Is it reasonable to return ServerFailure when no answer?

I try to let skydns work with normal dns server, so I configure resolve.conf like below:
<IP_ADDRESS> // skydns server
<IP_ADDRESS> // normal dns server

but if I lookup normal domain like google.com, skydns will return no error with empty answer, which block normal dns lookup.
As skydns can't host like normal dns, is it better to return Server Failure when answer is empty? Or return Failure when request is not suffix with skydns domain?

I'm curious why the decision was to make it a forwarder rather than return SERVFAIL - was there more discussion?

This is long dead code - I don't think anybody uses skydns1 anymore - it's our old Raft backed implementation.


Fix go build errors

Two minor changes:

enum declaration should use C++ syntax

swig can't port int64_t to Go's type system. long long used instead.

Can one of the admins verify this patch?

@tensorflow-jenkins test this please

@tensorflow-jenkins test this please

@tensorflow-jenkins test this, pelase

@tensorflow-jenkins test this please

Adding @josh11b to review.
The files had all been moved to tensorflow/c

We really want a 64-bit int type, not whatever long long happens to be
The whole point of this file is to use C not C++ so the enum change is counter productive.

@josh11b fair point, but I was not able to use the Go API bindings until I made these changes. I should probably take a different approach and try to modify the swig helpers rather than the C API itself.

@damienstanton should we keep this PR open for you to try a different approach, or close and wait for a fresh PR?

@danmane since I don't want my branch to get too stale, let's close this and I will open a new PR down the line. Thanks!

OK, look forward to the new one :)


Error getting card data for 47780, please report this!

Got this while trying to run a scan. The process stops altogether and I am unable to complete it.

Thank you for the report. I'm working on a solution.

Should be resolved with #14


Fix Danger::RequestSources::GitLab#setup_danger_branches

Fixes https://github.com/danger/danger/issues/1193.
///////////////////⚡///////////////////
🚫 AWESOME A PR!

You can just delete all of this and start your PR text anytime -

Hello there, just a quick pre-warning of some of the Danger rules we run on Danger PRs.
The big one is we request that every code change to Danger include a CHANGELOG entry,
this is so that:

People know what changes are between versions
Orta doesn't get all the credit for other people's work

Danger will look for a modification to the CHANGELOG.md when there are changes including
lib/* - if you're fixing unreleased code, or doing a simple typo change, you can
include #trivial in the title or the body of the PR and this is skipped.
We also request that you fill in the body of your PR, a title should be tweet length, but
ideally you can explain the PRs reasoning in the body. We look that it's longer than 5 chars.
Other than that, a lot of the other Danger rules are trickier to trigger so we can address
them as they come up!
❤ THANKS FOR HELPING OUT :D
///////////////////⚡///////////////////

          1 Error

:no_entry_sign:
Tests have failed, see below for more information.

          1 Warning

:warning:
Tests were not updated

Tests:

File
Name

gitlab_spec.rb
Danger::RequestSources::GitLab valid server response setups the danger branches

gitlab_spec.rb
Danger::RequestSources::GitLab valid server response raise error on empty MR

  Generated by :no_entry_sign: Danger

          1 Error

:no_entry_sign:
Tests have failed, see below for more information.

          1 Warning

:warning:
Tests were not updated

Tests:

File
Name

gitlab_spec.rb
Danger::RequestSources::GitLab valid server response raise error on empty MR

gitlab_spec.rb
Danger::RequestSources::GitLab valid server response setups the danger branches

  Generated by :no_entry_sign: Danger

OK, cool - this looks good to me ( I'll merge & deploy this once #1195 is in and I've handled the conflicts)

Merge on green


added ruby code editor extension

Description
This change introduces a new Chrome extension that provides users with quick access to an online Ruby Code Editor. The extension displays a welcome page with a typewriter effect and then navigates users to the Ruby Code Editor at the provided link.
Fixes:  #1147

Type of change

[X ] New feature (non-breaking change which adds functionality)

Checklist:

[X ] I have made this from my own
[ ] I have taken help from some online resourses
[ X] My code follows the style guidelines of this project
[X ] I have performed a self-review of my own code
[ X] I have commented my code, particularly in hard-to-understand areas
[ X] I have made corresponding changes to the documentation
[ X] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

@Sulagna-Dutta-Roy added new extension please check the pr


(GH-2468) Convert the DotNetCoreVerbosity to lowercase

Fixes #2468

@Roemer your changes have been merged, thanks for your contribution 👍


Add public key in a quote for remote attestation

The public key must be added in the report data structure sgx_report_data_t. Not sure what is the "best" encoding to use, probably DER. Must test the encoding/decoding to make sure that when retrieved from the quote & remote attestation verification report it is recoverable, in the PEM format.
See https://holtstrom.com/michael/tools/asn1decoder.php ... perhaps, if that helps to test in the process of working on this.

Try using https://www.openssl.org/docs/man1.1.1/man3/BN_bn2hex.html to store x and y coordinates in report data.

Putting the X and Y coordinates, uncompressed, works.


Testnet Faucet is not working. Different Errors HTTP 500, 524 and now HTTP 400

Describe the bug
Testnet Faucet is not working. Different Errors HTTP 500, 524 and now HTTP 400
Transaction ID
Address
Block#
Time stamp
To Reproduce
Steps to reproduce the behavior:
curl -d "address=STS3W6B4XE23DNA3BZ841FGRBYGEPBF9QBD8W23V" -X POST https://stacks-node-api.testnet.stacks.co/extended/v1/faucets/stx
{
"error": "Error: Response 400: Bad Request fetching http://stacks-node-follower-green.staging-stacks.svc.cluster.local:20443/v2/transactions - {"error":"transaction rejected","reason":"TooMuchChaining","reason_data":{"actual":8391,"expected":8390,"is_origin":true,"message":"Nonce would exceed chaining limit in mempool","principal":"ST2QKZ4FKHAH1NQKYKYAYZPY440FEPK7GZ1R5HBP2"},"txid":"327d6c628214c081b6896a295234e0ae148e8a3a78fe5f8b630e2307c2645ad7"}",
"stack": "Error: Response 400: Bad Request fetching http://stacks-node-follower-green.staging-stacks.svc.cluster.local:20443/v2/transactions - {"error":"transaction rejected","reason":"TooMuchChaining","reason_data":{"actual":8391,"expected":8390,"is_origin":true,"message":"Nonce would exceed chaining limit in mempool","principal":"ST2QKZ4FKHAH1NQKYKYAYZPY440FEPK7GZ1R5HBP2"},"txid":"327d6c628214c081b6896a295234e0ae148e8a3a78fe5f8b630e2307c2645ad7"}\n at StacksCoreRpcClient.fetchText (/app/src/core-rpc/client.ts:132:13)\n at processTicksAndRejections (internal/process/task_queues.js:95:5)\n at StacksCoreRpcClient.fetchJson (/app/src/core-rpc/client.ts:111:26)\n at StacksCoreRpcClient.sendTransaction (/app/src/core-rpc/client.ts:194:20)\n at /app/src/api/routes/faucets.ts:222:25\n at run (/app/node_modules/p-queue/dist/index.js:163:29)",
"errorTag": "25779711-dc27-49c0-a14e-0490d2873d07"
}

It started working today. Closing this issue

Great, thx, must have been bc of the api issues from yesterday. 👍


Format comments

Codecov Report

Merging #711 (7915747) into main (0cff70c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #711   +/-   ##
=======================================
  Coverage   85.83%   85.83%
=======================================
  Files         308      308
  Lines       28719    28719
=======================================
  Hits        24650    24650
  Misses       4069     4069

Impacted Files
Coverage Δ

lib/mir/src/analysis/reference_count/conversion.rs
99.19% <ø> (ø)

lib/mir/src/ir/let_recursive.rs
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


v2 vs v3 differences

Some of these differences may be intentional, but here is a list of ones I have noticed:
Disclaimer
The disclaimer is missing the yellow border on the left (currently it is gray). The unstyled disclaimer also has the same gray border which currently makes it look the same as the regular disclaimer, it should not have a border.
Navbar Dropdowns
The Dropdown trigger doesn't seem to take up the full height, making the click target smaller and causing the dropdown menu itself to not line up
v3:

v2:

Collapsible
Missing the little indicator
v3:

v2:

Colors
v3:

v2:

fixed via b52c389


Unexpected Token when using with react-native-web without Expo

💬 Question
UI Kitten and Eva version

Package
Version

@eva-design/eva
2.1.1

@ui-kitten/components
5.1.1

I am trying to use UI Kittens components in a multiplatform project in which we have ejected from Expo.
My webpack.config.js is:
// web/webpack.config.js

const path = require('path');
const webpack = require('webpack');

const appDirectory = path.resolve(__dirname, '../');

// This is needed for webpack to compile JavaScript.
// Many OSS React Native packages are not compiled to ES5 before being
// published. If you depend on uncompiled packages they may cause webpack build
// errors. To fix this webpack can be configured to compile to the necessary
// `node_module`.
const babelLoaderConfiguration = {
  test: /\.js$/,
  // Add every directory that needs to be compiled by Babel during the build.
  include: [
    path.resolve(appDirectory, 'index.web.js'),
    path.resolve(appDirectory, 'src'),
    path.resolve(appDirectory, 'node_modules/@ui-kitten')
  ],
  use: {
    loader: 'babel-loader',
    options: {
      // cacheDirectory: true,
      // The 'metro-react-native-babel-preset' preset is recommended to match React Native's packager
      presets: ['module:metro-react-native-babel-preset'],
      // Re-write paths to import only the modules needed by the app
      plugins: ['react-native-web'],
    }
  }
};

// This is needed for webpack to import static images in JavaScript files.
const imageLoaderConfiguration = {
  test: /\.(gif|jpe?g|png|svg)$/,
  use: {
    loader: 'url-loader',
    options: {
      name: '[name].[ext]',
      esModule: false,
    }
  }
};

module.exports = {
  entry: [
    // load any web API polyfills
    // path.resolve(appDirectory, 'polyfills-web.js'),
    // your web-specific entry file
    path.resolve(appDirectory, 'index.web.js')
  ],

  // configures where the build ends up
  output: {
    filename: 'bundle.web.js',
    path: path.resolve(appDirectory, 'dist')
  },

  // ...the rest of your config

  module: {
    rules: [
      babelLoaderConfiguration,
      imageLoaderConfiguration,
    ]
  },

  resolve: {
    // This will only alias the exact import "react-native"
    alias: {
      'react-native$': 'react-native-web'
    },
    // If you're working on a multi-platform React Native app, web-specific
    // module implementations should be written in files using the extension
    // `.web.js`.
    extensions: [ '.web.js', '.js' ]
  }
}

And I start serving my project in debug mode with the command: ./node_modules/.bin/webpack serve -d source-map --mode development --config ./web/webpack.config.js  --color --hot.
When I execute the above command I get the following output:
<i> [webpack-dev-server] Project is running at:
<i> [webpack-dev-server] Loopback: http://localhost:8080/
<i> [webpack-dev-server] On Your Network (IPv4): http://<IP_ADDRESS>:8080/
<i> [webpack-dev-server] On Your Network (IPv6): http://[fe80::4795:9a33:59a1:338]:8080/
<i> [webpack-dev-server] Content not from webpack is served from '/home/mario/dev/aladino/public' directory
asset bundle.web.js 1.44 MiB [emitted] (name: main) 1 related asset
runtime modules 27.2 KiB 12 modules
modules by path ./node_modules/ 1.3 MiB 121 modules
modules by path ./*.js 3.56 KiB
  ./index.web.js 429 bytes [built] [code generated]
  ./App.js 3.14 KiB [built] [code generated] [1 error]
./app.json 52 bytes [built] [code generated]

ERROR in ./App.js 29:4
Module parse failed: Unexpected token (29:4)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|   // TODO(mario): define proper theme.
|   return (
>     <ApplicationProvider {...eva} theme={eva.light}>
|       {component}
|     </ApplicationProvider>
 @ ./index.web.js 1:209-225

webpack 5.69.1 compiled with 1 error in 2751 ms

I am no Webpack nor Babel expert but the configuration above looks fine to me. The problem looks like UI Kitten components are not getting parsed by Babel.
Could somebody help with that?
It would be great if the documentation would cover webpack UI Kittens configuration.
Thanks for the great library.

Have the same situation but win expo managed project. Wanted to try out this lib but stuck as soon as I've finished "Getting started" in the docs.


Grid - allow customization of the management column

For example, I'm using the Grid widget with Bootstrap4 and the width of the management column overpowers the other columns in my grid.
Maybe inlineEditing.managementColumn can even accept an object rather than boolean and there could be additional configs to allow further customization, such as fontSize, cssClass, etc.?

You can build the management column as you want and call edit, update, cancel and delete methods from buttons inside that new custom column. Here is an example http://gijgo.com/grid/methods/edit
see also: http://gijgo.com/grid/methods#InlineEditing