Please add advice to enable long paths on Windows 10

The text in: https://github.com/microsoft/vcpkg/blob/master/scripts/cmake/vcpkg_buildpath_length_warning.cmake
Could offer a link to the Microsoft document concerning the issue, which is: https://docs.microsoft.com/en-gb/windows/win32/fileio/naming-a-file#enable-long-paths-in-windows-10-version-1607-and-later
This is on the condition that Windows 10 is the host operating system.
Currently, the console just gets spammed with:
CMake Warning at scripts/cmake/vcpkg_buildpath_length_warning.cmake:4 (message):
  qt5-webengine's buildsystem uses very long paths and may fail on your
  system.

  We recommend moving vcpkg to a short path such as 'C:\src\vcpkg' or using
  the subst command.
Call Stack (most recent call first):
  installed/x64-windows/share/qt5/qt_download_submodule.cmake:13 (vcpkg_buildpath_length_warning)
  ports/qt5-webengine/portfile.cmake:72 (qt_download_submodule)
  scripts/ports.cmake:90 (include)

Thanks!

josm
https://github.com/microsoft/vcpkg/issues/10200

I think we should add such instructions if and only if we have observed that building these ports with that flag enabled in a location wherein the directory structure would result in paths that are too long succeeds with the flag set and fails without.
Note that this probably means build tools and similar would need to be manifested and I'm pretty sure Qt depends on some that are not, but only testing would tell. See https://docs.microsoft.com/en-us/windows/win32/fileio/naming-a-file#enable-long-paths-in-windows-10-version-1607-and-later for additional details. (I believe there were version(s) of Windows 10 that had bugs where either the flag or the manifest were not correct checked but that was considered a bug)


MIQ Fix 18th Aug

Thanks for putting this together. It worked for me last night (Europe time) and stopped on the same slots as the twitter bots. It didn't select the dates or beep (I tried deleting date.elem.click(); to test beep() but no luck there) though - does it do that for you? It also seemed to ignore my dates and stopped on anything available. It's possible I messed something up copying the code over.

Thanks @bennyggg for test and reporting.
I've added one more update to the code to match the new MIQ update and this will make it work as expected (i.e clicking the date, beep, match target dates only)

Hi @monagjr  trying your new code with testDates. I couldn't get it to work. I have no idea to coding, but kindly guide me if I'm doing something wrong. Thanks.
// Set to true to turn on test mode
let testDates = true;
if (await findAvailability(page)) {
const status = "AVAILABLE! Found at: " + new Date().toLocaleString();
console.log(status)
updateElectronStatus('status-count', status);
testDates = ['2021-09-26'];

Thanks @ricoboirn  for testing and reporting.
I've updated the code part related to "test mode", it now works as expected.

Thanks @bennyggg for test and reporting.
I've added one more update to the code to match the new MIQ update, and this will make it works as expected (i.e clicking the date, beep, match target dates only)

Thanks. I've updated to your newest version and it works with testDates

@monagjr  Awesome, working as intended now. I'm on my 4th week now trying my luck securing for a friend. Thanks.

Thanks @ricoboirn @bennyggg for confirmation
I hope this will help more to secure their needed spots

@jvolker @alexDrinkwater @stilljake can you merge this with base code and create new release binaries?

Hi guys, sorry to butt in, thanks much for doing this, this is a lot of help for non-programmers such as myself. If I may ask a favor? would you be so kind to update the MIQ-Booking-Assistance v0.10.3 exe file so that we can redownload and use it as well? Really need to book a slot for my daughter. Thanks a lot

no beep again.. sad..

@jvolker @alexDrinkwater @stilljake can you merge this with base code and create new release binaries?

Sorry, I don't have write access!

@Jefsmango and anyone else who needs ready-made pre-built version of the APP latest version (v0.10.4):
https://github.com/monagjr/miq-nz-booking-tools/releases/latest

https://www.buymeacoffee.com/mojr

Very much appreciated @monagjr , they must have found a way to mess with the script output coz after a few tries, it stops on the date but not selecting it and no more beeping. and even when you click on the date you cant fill it up recaptcha because it doesnt stop refreshing...

@Jefsmango thanks for reporting this, can you please open a discussion topic under "bugs / issues" category here:
https://github.com/monagjr/miq-nz-booking-tools/discussions/new

Thanks, @monagjr. I've tried your changes and was able to get an alert. I've just merged this. New releases will follow in a few minutes.

Welcome back @jvolker :)


Frontend result

Just a quick fix to remove the ID of the question when a user downloads the results in the json. Also cleans up a bit the mock by keeping only two mock elections.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
13 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 63.231%

Files with Coverage Reduction
New Missed Lines
%

services/dkg/pedersen/handler.go
13
84.84%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
3116

Relevant Lines:
4928

💛  - Coveralls


msvcp140.dll and vcruntime140.dll was not found, for Bulk Crap Uninstaller 5.0 Portable

why msvcp140.dll and vcruntime140.dll was not found?
Bulk Crap Uninstaller 5.0 Portable
Windows 10 Home

Helped to install x86: vc_redist.x86.exe from here Visual C++ downloads

Can you test if this an issue when starting win-x64\BCUninstaller.exe directly instead of BCUninstaller.exe in the root directory?

Can you test if this an issue when starting win-x64\BCUninstaller.exe directly instead of BCUninstaller.exe in the root directory?

there is no problem with win-x64\BCUninstaller.exe without x86: vc_redist.x86.exe but why does it happen to BCUninstaller.exe

Can you check if this build works fine without the redist?
BCUninstaller.zip

yes this build works fine without the redist

yes this build works fine without the redist

Thank you for the response!  Unfortunately, the installer for that version won’t even start on my PC.  (I do have all the Microsoft redeistributables installed.)


@ashfurrow => fix nav button placement on rotation

fixes https://github.com/artsy/eigen/issues/267
This was originally written for ios7, which didn't swap frame width and height on rotation for some reason. in iOS 8 they fixed that bug, but our code didn't account for the fix. now it does and it ditches the iOS7 way.

You're fast.


Bump versions to 1.8

Seems that the next release will be 1.8, so let's appropriately publish nightlies that way.

Please keep it 1.7 for now.


Minor bug fixes

Fixes:
https://github.com/Nazariglez/notan/issues/267
https://github.com/Nazariglez/notan/issues/276
https://github.com/Nazariglez/notan/issues/269

hello @devmeat, this is great, thank you so much for the help! Can you please add some lines to the changelog?

Added changelog items

Merged thanks!


Fix projection

Pull Request Test Coverage Report for Build 490

29 of 36 (80.56%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 82.295%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/optimize.js
27
34
79.41%

Totals

Change from base Build 488:
0.01%

Covered Lines:
5891

Relevant Lines:
6973

💛  - Coveralls


Add support for workspacerouting managed by external controller and make routing controller reusable

What does this PR do?
Enables support for alternate controllers for a routingClass in the workspacerouting controller.

Make the sync-to-cluster functionality of the WorkspaceRouting controller reusable. This is done by making the getSolverForRoutingClass function a field in the WorkspaceRoutingController struct, allowing a custom solver to be provided for a given routingClass.
Configure controllers to propagate the restricted-access annotation to all created objects
Update webhooks to only enforce immutability on objects with the restricted-access annotation, to allow for an external controller to update/create resources.

What issues does this PR fix or reference?
Resolves eclipse/che#18520
Is it tested? How?
For our normal samples, nothing should be different.
cat <<EOF | oc apply -f -
kind: DevWorkspace
apiVersion: workspace.devfile.io/v1alpha2
metadata:
  name: theia-external
spec:
  routingClass: external-sample
  started: true
  template:
    projects:
      - name: project
        git:
          remotes:
            origin: "https://github.com/che-samples/web-nodejs-sample.git"
    components:
      - name: theia
        plugin:
          id: eclipse/che-theia/next
      - name: terminal
        plugin:
          id: eclipse/che-machine-exec-plugin/nightly
    commands:
      - id: say hello
        exec:
          component: plugin
          commandLine: echo "Hello from $(pwd)"
          workingDir: ${PROJECTS_ROOT}/project/app
EOF

Workspace deployment does not fail, but hangs with no URL in status field. Logs in devworkspace operator state "waiting on routings to be ready".
Deploy "external" sample operator which is designed to provide routing for routingClass: external-sample
Routing becomes ready, a service is created
Workspace "starts" (but cannot be accessed since sample routing controller implementation is just a test)

/retest
/test v5-devworkspaces-operator-e2e

I've set up a quick sample repo showing how the workspacerouting controller can be reused: https://github.com/amisevsk/devworkspace-controller-external-routing
If a workspace is created with routingclass external-sample, it will define a plain service and allow the workspace start to continue (the workspace is unaccessible, of course).

Updated PR description with current testing info.

One thing that came up from my discussions with @davidfestal is the need to pass down additional information down to the (external) routing class solvers. The best way I could think of is to pass down (prefixed?) annotations from the workspace CR down to workspace routing CRs so that the solver can pick up this additional info when producing the routing objects..
E.g.
kind: DevWorkspace
metadata:
   annotations:
      "routingClass.che/gateway": my-gateway
...

would make the annotation appear on the WorkspaceRouting object as well:
kind: WorkspaceRouting
metadata:
  annotations:
    "routingClass.che/gateway": my-gateway

This would require passing down the whole WorkspaceRouting object down to GetSpecObjects() impls, not just the spec...
Do you think it'd make sense to do that in the scope of this PR or would you rather handle this separately?

The best way I could think of is to pass down (prefixed?) annotations from the workspace CR down to workspace routing CRs so that the solver can pick up this additional info when producing the routing objects..

This may have to be a separate PR; currently passing all annotations is a bit problematic since we add e.g. timing-related annotations, and that could result in looping reconciles as the routing is updated every time we reconcile the devworkspace, and I don't want to hard-code a specific prefix prematurely.
Perhaps we can define an attribute on DevWorkspaces that we handle specifically? Something like routingAdditionalAttributes and add a similar attributes field to WorkspaceRoutings? cc: @davidfestal

This would require passing down the whole WorkspaceRouting object down to GetSpecObjects() impls, not just the spec...

I actually already slipped this change in, on commit a068d39 (the short-form commit message is a little unclear here)

Rebased to clean up the history a bit; also I've updated the sample repo with external routing (https://github.com/amisevsk/devworkspace-controller-external-routing) so it should be in sync with the current setup.

/test v5-devworkspaces-operator-e2e

/retest

/test v5-devworkspaces-operator-e2e

/retest

/test v5-devworkspaces-operator-e2e


Remove batch_kwargs

(Depends on #121)
This appears to be unused now - now pipeline definitions include it, and it's not used in LLMBlock anywhere.
This change might throw up cases where we're passing unknown configuration to LLMBlock. Here's why ...
It's hard to spot, but this:
        def __init__(self, ..., **batch_kwargs):
            ...
            self.batch_params = batch_kwargs.get("batch_kwargs", {})

is equivalent to this:
        def __init__(self, ..., **kwargs):
            ...
            self.batch_params = kwargs.get("batch_kwargs", {})

which is equivalent to this:
        def __init__(self, ..., batch_kwargs={}, **kwargs):
            ...
            self.batch_params = batch_kwargs

except that trailing **kwargs meant we were silently accepting unknown block_config parameters.

The change looks fine, though I'd request putting the extra explanation from the PR description in the commit message, as well

Yeah, there's two commits in this PR, and the first one has that explanation. Easy to miss with the stack of PRs - see commit 18f1513897f6d31f0dd059e398305fa8792843bf


add more minor antag objectives

About the PR
Media

[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

add: Added fun!
remove: Removed fun!

closing as to be replaced by https://github.com/Nyanotrasen/Nyanotrasen/pull/1167


Stack Overflow (Criteria.parse) with JsonPath

According to https://github.com/json-path/JsonPath/issues/973 there is a high severity issue for JsonPath. Are there any plans to deal with this?
JsonPath seems to be a dependency of the DGS client module.

Correct me if I'm wrong, but as far as I understand the issue wouldn't be exposed through DGS. We don't take user input directly that goes through JsonPath. It's only when using the DgsQueryExecutor, for example in tests, where a developer uses the APIs backed by jsonpath.
Once there is a patch for the library we'll update it, but I don't think there's any action to take now.

That's good to hear. I did notice however, that after fully excluding the dependency it does cause NoClassDefFoundError: com/jayway/jsonpath/Configuration, so it used outside of tests too.
java.lang.NoClassDefFoundError: com/jayway/jsonpath/Configuration
	at com.netflix.graphql.dgs.client.GraphQLResponse.<clinit>(GraphQLResponse.kt:131)
	at com.netflix.graphql.dgs.client.WebClientGraphQLClient.handleResponse(WebClientGraphQLClient.kt:146)
	at com.netflix.graphql.dgs.client.WebClientGraphQLClient.access$handleResponse(WebClientGraphQLClient.kt:42)
	at com.netflix.graphql.dgs.client.WebClientGraphQLClient$reactiveExecuteQuery$3.invoke(WebClientGraphQLClient.kt:136)
	at com.netflix.graphql.dgs.client.WebClientGraphQLClient$reactiveExecuteQuery$3.invoke(WebClientGraphQLClient.kt:136)
	at com.netflix.graphql.dgs.client.WebClientGraphQLClient.reactiveExecuteQuery$lambda$3(WebClientGraphQLClient.kt:136)
	...
Caused by: java.lang.ClassNotFoundException: com.jayway.jsonpath.Configuration
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:641)
	at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:188)
	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:525)

But I agree, looking at what's happening in GraphQLResponse, this is not the same as seen in that issue.
I'll convey the message as I was just forwarding the question from client.

It’s definitely there, but there is no code path to trigger the issue
On Tue, Jan 9, 2024 at 10:37 AM Martin Visser @.***>
wrote:

That's good to hear. I did notice however, that after fully excluding the
dependency it does cause NoClassDefFoundError:
com/jayway/jsonpath/Configuration, so it used outside of tests too.
java.lang.NoClassDefFoundError: com/jayway/jsonpath/Configuration
at com.netflix.graphql.dgs.client.GraphQLResponse.(GraphQLResponse.kt:131)
at com.netflix.graphql.dgs.client.WebClientGraphQLClient.handleResponse(WebClientGraphQLClient.kt:146)
at com.netflix.graphql.dgs.client.WebClientGraphQLClient.access$handleResponse(WebClientGraphQLClient.kt:42)
at com.netflix.graphql.dgs.client.WebClientGraphQLClient$reactiveExecuteQuery$3.invoke(WebClientGraphQLClient.kt:136)
at com.netflix.graphql.dgs.client.WebClientGraphQLClient$reactiveExecuteQuery$3.invoke(WebClientGraphQLClient.kt:136)
at com.netflix.graphql.dgs.client.WebClientGraphQLClient.reactiveExecuteQuery$lambda$3(WebClientGraphQLClient.kt:136)
...
Caused by: java.lang.ClassNotFoundException: com.jayway.jsonpath.Configuration
at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:641)
at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:188)
at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:525)
But I agree, looking at what's happening in GraphQLResponse, this is not
the same as seen in that issue.
I'll convey the message as I was just forwarding the question from client.
—
Reply to this email directly, view it on GitHub
https://github.com/Netflix/dgs-framework/issues/1772#issuecomment-1883583922,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA2XLASDFOTGTKXQ45LN6DYNWE5LAVCNFSM6AAAAABBSZ5TNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBTGU4DGOJSGI
.
You are receiving this because you commented.Message ID:
@.***>

@ntenherkel But it isn't used only in test as my stack trace showed. The dependency is used in production code too. Maybe the path to that issue as stated in JsonPath isn't used in production code, but JsonPath itself is used outside test.

Yes let me rephrase my suggestion. @paulbakker is it possible for the netflix developers to scope this dependency to test? Based on Martin's comment that would also mean some adjustments in the dgs-framework java/kotlin logic.
My feeling is that json-path is not playing an important role in the dgs-framework. The end result having a dependency less (that is poorly maintained) in the jar, is also worth the attempt if you ask me.

I do fully agree with that. If that would be possible, that would be a nice solution.

JsonPath is used in two places:

DgsQueryExecutor - to parse out parts of a query from the result. This is typically used for testing. However it's part of the main API, so it should not be a test only dependency. I'm not too worried about it though, because there is no way for an outside user (someone sending a query) to specify the json path.
The DGS client that you use to call other GraphQL services. That's the stacktrace @martinvisser posted. This could be a problem if you would take user input and pass that on directly as the jsonpath argument.

Like SQL injection, this issue is only a problem if user input is used directly to form the jsonpath argument, which we don't do in the framework itself. We can't make JsonPath a a test-only dependency, but I don't see how the CVE would affect a DGS in general.


Allow changing read and deadlines asynchronously. Timeouts need to implement net.Error. Fixes #356

Looks good once the test are updated!

Updated :+1:


Fix Obvious Issues With Batch and Range

Changes
This PR fixes the fact that batch and range do not reject the commands that should be rejected according to the user guide.
Related Issues

None, but this is a urgent fix.

Codecov Report

Merging #343 (be86a89) into master (40d8297) will decrease coverage by 0.03%.
The diff coverage is 62.50%.

@@             Coverage Diff              @@
##             master     #343      +/-   ##
============================================
- Coverage     72.72%   72.68%   -0.04%
- Complexity     1287     1289       +2
============================================
  Files           160      160
  Lines          3915     3921       +6
  Branches        461      465       +4
============================================
+ Hits           2847     2850       +3
- Misses          996      997       +1
- Partials         72       74       +2

Impacted Files
Coverage Δ

...java/seedu/contax/logic/commands/RangeCommand.java
69.76% <50.00%> (-2.74%)
:arrow_down:

...java/seedu/contax/logic/commands/BatchCommand.java
85.52% <75.00%> (-0.78%)
:arrow_down:

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 40d8297...be86a89. Read the comment docs.


Support setting the content-type on Azure Blob storage

Expected Behavior
When reading a DICOM file from Azure blob storage, the content-type is set to "application/dicom"
Actual Behavior
Content Type is set to "application/octec-stream"

Done


Can't write cells.

Hello!
I am working with js-xlsx in Express and consume this API from Vuejs.
I am trying to create and xlsx file but always get an object in the first cell.
This is my method in Express:

and my code in Vuejs:

My excel file:

Someone have the same problem?
Thanks!

remove the wb.SheetNames.push line -- book_append_sheet does that part for you already.

I tried but not works. This line XLSX.utils.book_append_sheet(wb, ws_data, "People"); is neccesary to add the data.


Use 3 codegen units.

On my desktop PC (4+4 cores i7), this decreased the compile times after
changing client from 19.9 to 9.8 seconds (!).

FPS is unaffected.

Merging now or I'll lose my mind


Add excludes directories

I use the Visual Studio Code(vscode) with local history plugin, which generates a .history folder to hold the old content, so when I run the swag init command, it checks the folder and reports an error.
2018/05/22 09:40:16 Generate swagger docs....
2018/05/22 09:40:16 Generate general API Info
2018/05/22 09:40:16 ParseFile panic:.history\main_20180423195313.go:1:1: expected 'package', found 'EOF'
panic: ParseFile panic:.history\main_20180423195313.go:1:1: expected 'package', found 'EOF'

So I want cli to have a parameter that excludes some directories.

for you #133

Good. Thank you.

@rsinensis it's a workaround. we will add cli parameter to support excluded directories.


Patrick/sdk v0.6.6

Changes

[x] Update rosetta-sdk-go
[x] Update rosetta-cli version
[x] Update CoordinatorHelper
[ ] Add configuration option for ForceRetry

[x] Add to Config.Data
[x] Add to Config.Construction
[ ] Change fetcher initialization to optionally include WithForceRetry

rosetta-sdk-go Changes

Pull Request Test Coverage Report for Build 6623

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

Totals

Change from base Build 6610:
0.0%

Covered Lines:
521

Relevant Lines:
887

💛  - Coveralls


RFE: new variant supporting RHEL for Edge

Creating this issue as a place-holder for discussing a variant specific to the RHEL for Edge use case.
We envision supporting Ignition on RHEL for Edge systems in the future for handling simple configuration tasks like writing files and enabling systemd units.  We don't want to (immediately) support the full complement of Ignition features (i.e. disks, RAID, filesystems, LUKS) so it was suggested a new variant be created in butane to denylist the features we don't want to support.
cc: @runcom

By way of example, the openshift variant also denylists Ignition features.
Is the intent to avoid supporting the partitioning/filesystem features for data partitions as well?  One possible middle ground is to support them for data partitions but not boot partitions.

Is the intent to avoid supporting the partitioning/filesystem features for data partitions as well? One possible middle ground is to support them for data partitions but not boot partitions.

in general anything related to partitioning/layout/luks/boot would be taken out at first - we do that in Image Builder basically

Is there no circumstance in which a user might want to, for example, create filesystems on a second disk during provisioning?

COS-1879

Is the intent to avoid supporting the partitioning/filesystem features for data partitions as well? One possible middle ground is to support them for data partitions but not boot partitions.

in general anything related to partitioning/layout/luks/boot would be taken out at first - we do that in Image Builder basically

@runcom, can you help me understand how Image Builder handles filesystems and partitions? I was under the impression that it can only generate the root filesystem contents. Is it able to configure disk/partition/filesystem layout, or is it that we only want to support the default layout and not other configurations?

@control-d default configuration still - we have use cases for custom partition schemes but we’re not there yet and we didn’t want to expose something as of yet


Check for changes in included files

What I am basically looking for is something like changed('*.tex'). To start easy I wanted to do this with one file. From the description in https://github.com/cereda/arara/wiki/New-feature-highlights-in-4.0

changed(file): the very same idea as the previous method, but with a proper File object instead.

I understand that it should be possible to give a specific file. But while changed('tex') works flawlessly to check for changes of my main file, I tried changed('document.tex'), but this does not trigger arara.
My main.tex:
% arara: pdflatex if changed('document.tex')

\documentclass{article}

\begin{document}
\include{document}
\end{document}

and the document.tex:
test

Man, I simply love these ducks! :wink:
A brief explanation of what's happening: changed(String) takes the provided argument, considers it as a file extension and applies to the main file base name. So, having main.tex as your main document, changed('aux') implies checking for modifications in main.aux. When using changed('document.tex'), the method will look for main.document.tex, which is not what you want.
However, do not despair, young grasshopper! There's a second changed(...) method which takes the argument type into consideration: changed(File) will check for modifications directly in that file! Well, how does one use this marvellous contraption?
The secret lies on another helper method in the directive context, toFile(String), which takes a string representing your file and returns a file representation of it! That's why I really need to work on the user manual, there are so many new features and improvements in version 4.0! Your new directive might work now:
% arara: pdflatex if changed(toFile('document.tex'))

Hope it helps! Quack! :wink:

Thanks for your lightning-fast reply! That solves the mystery!

My pleasure, pal! I am glad your experience with version 4.0 is being so positive! All the best! :wink:


Fixed typo

I do think we should give Carsonbot some personality, but this is too much...

Thank you


UnhandledPromiseRejectionWarning

after adding calendar with quasar ext add @quasar/qcalendar
I recieve
(node:2587) UnhandledPromiseRejectionWarning:   TypeError:
[@quasar]/quasar-app-extension-qcalendar/src/index.js:26:7
when doing quasar dev

@KolbeinNL Can you run quasar info and post the results? Thanks.

You really need to give me more information than this.

Sorry, I have to close this. If you feel there is still an issue, feel free to re-open and give more details. Thanks.


[ISSUE #1409] Fix the bug when use queryMsgByKey or queryMsgByUniqueKey command in aclEnable = true

What is the purpose of the change
Fix the bug when use queryMsgByKey or queryMsgByUniqueKey command in aclEnable = true
Brief changelog
Filter out the UNIQUE_MSG_QUERY_FLAG attribute when constructing the authentication content to compute the signature
Verifying this change
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

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

Coverage increased (+0.08%) to 50.699% when pulling 43ec2943fe5d23e37b25e287cc60a506d8621901 on zhangjidi2016:develop into<PHONE_NUMBER>646d962f28b837027db379b71d71a7 on apache:develop.

Coverage increased (+0.08%) to 50.699% when pulling 43ec294 on zhangjidi2016:develop into 9374555 on apache:develop.

Coverage increased (+0.08%) to 50.699% when pulling 43ec294 on zhangjidi2016:develop into 9374555 on apache:develop.


Increase verbosity on Heroku (-vv)

This will log debug messages on Heroku as well.

We have a free logging addon that allows to search logs anyway
In case of a misbehavior more information is always appreciated
Usually only the last few hours, maybe days are relevant anyhow

Vibed:
 -v --verbose        Enables diagnostic messages (verbosity level 1).
    --vv, --vverbose Enables debugging output (verbosity level 2).
    --vvv            Enables high frequency debugging output (verbosity level
                     3).
    --vvvv           Enables high frequency trace output (verbosity level 4).

Verbosity level 2 is LogLevel.debug_:
https://github.com/rejectedsoftware/vibe.d/blob/master/core/vibe/core/log.d#L173

Ok, let's give it a shot.


Upgrade keyhunt version

Alberto, please tell me how to upgrade to version 0.2.230507 Satoshi Quest without deleting the existing application. I use version 0.2.211031 Trick or treat ¡Beta! .
Thank you and I want to tell you that Keyhunt is the most powerful, I only reached 10 Petakeys/s/session because I always opened 6 consoles simultaneously, but this is the hardware limitation. It doesn't compare to anything else anyway. I invested in the graphics card because of other applications and in vain.

Timeout

git fetch
git pull

Thank you. I changed the name of the folder to KeyhuntOld and installed a new one.


Add FilterOptions::RETURN_ON_NULL

What does this PR do?
This pull request adds the returnOnNull filter option. This option will break filter chaining if a resulting filtered value is null.
Checklist

[ ] Pull request contains a clear definition of changes
[ ] Tests (either unit, integration, or acceptance) written and passing
[ ] Relevant documentation produced and/or updated

Coverage remained the same at 100.0% when pulling a582745ea557d6b36b3f36f01cf8a64c4f61056c on chadicus:fea/return-on-null into 047bc08ccd438f3a8f4423f43431c5c5b47deceb on traderinteractive:master.

Coverage remained the same at 100.0% when pulling a582745ea557d6b36b3f36f01cf8a64c4f61056c on chadicus:fea/return-on-null into 047bc08ccd438f3a8f4423f43431c5c5b47deceb on traderinteractive:master.

Coverage remained the same at 100.0% when pulling a582745ea557d6b36b3f36f01cf8a64c4f61056c on chadicus:fea/return-on-null into 047bc08ccd438f3a8f4423f43431c5c5b47deceb on traderinteractive:master.


Refactor lobby / state initialisation code

Currently the game state is initialised at game creation. This has a number of consequences:

the number of players must be known exactly before the game is created (no "open" public lobbies")
players can start making moves before all players have joined, however the game will most likely get stuck when an empty slot is reached (or if it is never filled)
the server has no way to know if a player is not present, and cannot take their turn (i.e. to timeout disconnected players)

Ideally the game state would be initialised as a separate step, i.e.

The game is created with min / max player counts
Players connect and claim slots
When the server or a nominated player believes the game is "ready" the game "starts" and the state is initialised

Future enhancements could include onConnect/onDisconnect handlers for players so that the server can apply custom logic for these cases (e.g. timeout players)

Possible implementation:
Too many changes to the lobby API could break a lot of existing games. One way to resolve these issues would be to use the existing state and game definition to handle lobbies. For instance:

When a game is created through the API it could also return a special "host" token,
Games could define a special lobby phase (a basic one could be supplied with boardgame.io)
If a lobby phase is present in the game definition then the state is not initialised when the game is created
in the lobby phase, a number of predefined moves are available such as claimSlot, setName, setReadyState etc.
A custom 'isReady` function could be defined on the phase, which can be checked after each "claimSlot" / "setReadyState" move. When this returns true, the game "starts" and state is initialised in the usual way.

'isReady' could be defined a number of ways, for instance when all players setReadyState(true), or if somebody calls setLobbyReady(host_token) supplying the host token returned during game creation.

I'm happy to start looking at this in a couple of days, but I know there are a number of "lobby" related issues open at the minute (and a design document somewhere?) so I'm not sure how this fits in with this other work.

@will-hart Sorry, I missed this. Will take some time to digest this and get back to you.

As a corollary to/variation on this request, some games can handle players coming and going while the game is in play.
The original description talks about timing out players, presumably to skip them in the play order. This would be the best mode if the player may return, and may be the only mode for some games. But many card games would allow a player to permanently leave by "turning in their cards". This would imply that they aren't coming back and the return of the cards allows the game to preserve resources. A typical result (which can be left to the game to implement, but just for an example) would involve taking all of the players cards and putting them into the global discard pile or shuffling them into the draw pile. For this reason, it might be good to distinguish between a player being unavailable during play and a player that physically left the game.
Additionally, many card games can deal with a player entering after the game has gone a few rounds by simply dealing them a fresh hand. They may be at a disadvantage in terms of game progress, but it is sometimes a preferable solution to waiting for the game to end or abandoning the game and starting a new one.


Updating an index based on changes

I don't know if that has been done yet, but I could not find a mention of it in the documentation. It would be pretty neat to have a function to update an index only for files with changes. One (relatively) easy way to do it would be to hash a file every time it is parsed and store that in the index. Then when running the updater, simply hash every file one by one and if the said file has already been indexed before, compare it's new hash to the hash of the already indexed file to know if anything was changed content-wise.

Very interested in this.

does this kind of get at what you're looking for? https://gpt-index.readthedocs.io/en/latest/how_to/update.html. I'm very open to improving this

From what I understand from the doc, there are already functions to update a specific document, but I would find it more convenient to have a function where I specify a directory, and perform an update-if-changes of the index. The existing functions already allow for performing an update, so it should just be a matter of coupling it with a bit of logic for first checking if changes are present (and an update is needed) to allow for batch updating of only the necessary stuff.
That would be very useful when parsing a large note repository for example. I have a large obsidian vault, and most of the notes don't change between updates, only a subset (which still remains too large to want to keep track of to then update individually).

Ahh I see. That's interesting. Out of curiosity are you using the SimpleDirectoryReader? checking for changes in files is prob easier than checking for changes through arbitrary API's

Haven't had a chance yet, but from the doc I see that is used for opening documents. It should be easy to then run each document's content through a hashing algorithm (any would do, though the faster the better) and store the resulting hash for each document on index (to be used to represent the "state" of the document at indexing). This way no need for any API.

I'd be grateful to have it generalized. I take documents from ElasticSearch index, which is updated irregularly as new stuff is imported. I'd like to run the LlamaIndex update e.g. once a week and update the already existing index in the most convenient way. Hashing the content seems to be a viable option, however, the same document fetched from ES has a different doc-id next time, so it can't be compared this way. We would have to have the ability to set document ID, e.g. by mapping this to a specific ES document field. I believe having a document ID configurable (but still immutable) can potentially help in other scenarios.


Add prometheus resourcefilter

Checklist

[x] Update changelog in CHANGELOG.md.
[ ] Make sure values.yaml and values.schema.json are valid.

This excludes resource from Kyverno webhooks. Is that what you are trying to do?

This excludes resource from Kyverno webhooks. Is that what you are trying to do?

Definitely not, I'm actually trying to do the opposite :/


Available pre-trained models

Hi there,
I found this trained model, which is very useful. I was wondering if other models of this kind are publicly available, eg road, building segmentation from aerial RGB images.
Cheers

:wave: Hey hey, please check this issue and the note

https://github.com/mapbox/robosat/issues/204
https://github.com/mapbox/robosat/blob/cbb1c73328183afd2d6351b7bfa3f430b73103ea/README.md

Thank you and happy weekend! :v:

Many thanks @daniel-j-h ! Appreciated


[Feature Request] Mixed Content

Hello, i'm running Komga behind reverse proxy and when i try to use userscript with komf i can't get it to run as indicated by the error below. Are there  any plans to implement some form of security for komf so we can expose the servise and use it with userscript?
Mixed Content: The page at 'https://komga.mydomain.com/libraries/07MMHWFYGVEQX/series' was loaded over HTTPS, but requested an insecure XMLHttpRequest endpoint 'http://<IP_ADDRESS>:32085/config'. This request has been blocked; the content must be served over HTTPS.
dispatchXhrRequest @ userscript

For now you can only allow mixed content in your browser settings https://superuser.com/a/1672733
As for proper secure way, it is possible to expose komf using your reverse proxy using basic auth.  To access komf you'll need to enter username and password. It should be supported by most reverse proxies
This will require handling basic auth in userscript. In connection tab a new section with komf username and password will need to be added


Not clear how links work

I've been trying to configure links unsuccessfully and I am not sure what I am doing wrong.  No matter what, I do not get any output about links and the links.conf ends up zero bytes.  I've tried specifying links for just the other server, both servers, tried all of the config options I can see not related to TLS and I am not sure what I am missing.

Can you use docker exec inspircd1 /conf/links.sh and check the output?

Sure thing:
$ cat /inspircd/conf/links.sh
#!/bin/sh

########################################
###                                  ###
### DON'T EDIT THIS FILE AFTER BUILD ###
###                                  ###
###    USE ENVIRONMENT VARIABLES     ###
###              INSTEAD             ###
###                                  ###
########################################

# generateLinkBlock <NAME> <IPADDR> <PORT> <SENDPASS> <RECVPASS> <ALLOWMASK> <MORE>
generateLinkBlock() {
if [ "$6" = "" ]; then
    echo "<die reason value=\"Wrong configured link block. Please check your configs! For details see: https://github.com/inspircd/inspircd-docker#linking-servers-and-services\">"
    exit 1
fi
cat <<EOF
<link name="${1}"
      ipaddr="${2}"
      port="${3}"
      sendpass="${4}"
      recvpass="${5}"
      allowmask="${6}"
      ${7}>
EOF
}

# filterEnv SELECTOR SUFFIX
filterEnv() {
    env | grep "INSP_${1}_${2}" | sed -e 's/^[^=]*=//'
}

# Initialize settings
NAME="$(filterEnv "$1" "NAME")"
IPADDR="$(filterEnv "$1" "IPADDR")"
PORT="$(filterEnv "$1" "PORT")"
PASSWORD="$(filterEnv "$1" "PASSWORD")"
SENDPASS="$(filterEnv "$1" "SENDPASS")"
RECVPASS="$(filterEnv "$1" "RECVPASS")"
ALLOWMASK="$(filterEnv "$1" "ALLOWMASK")"
TLS_ON="$(filterEnv "$1" "TLS_ON")"
FINGERPRINT="$(filterEnv "$1" "FINGERPRINT")"
OPTIONS="$(filterEnv "$1" "OPTIONS")"
AUTOCONNECT="$(filterEnv "$1" "AUTOCONNECT")"

# Set sendpass to password if it's not further specified
if [ "$SENDPASS" = "" ] && [ "$PASSWORD" != "" ]; then
    SENDPASS="$PASSWORD"
fi

# Set recvpass to password if it's not further specified
if [ "$RECVPASS" = "" ] && [ "$PASSWORD" != "" ]; then
    RECVPASS="$PASSWORD"
fi

# Check if all needed settings are present
if [ "${SENDPASS}" != "" ] && [ "${RECVPASS}" != "" ] && [ "${NAME}" != "" ] && [ "$IPADDR" != "" ]; then
    # Enable TLS by default
    if [ "$TLS_ON" = "" ]; then
        TLS_ON="yes"
    fi

    # Set default port
    if [ "$PORT" = "" ] && [ "$TLS_ON" = "yes" ]; then
        PORT=7001
    else
        PORT=7000
    fi

    # When not specified, try to be smart and predict a allowmask
    if [ "$ALLOWMASK" = "" ]; then
        ALLOWMASK=$(ip  route show dev eth0 | grep -v default | cut -d" " -f1 | head -1)
    fi

    # Set TLS support by extending the generation config extension
    if [ "${TLS_ON}" = "yes" ]; then
        OPTIONS="$OPTIONS ssl=\"gnutls\""
    fi

    if [ "$FINGERPRINT" != "" ]; then
        FINGERPRINT=$(echo "$FINGERPRINT" | tr '[:upper:]' '[:lower:]' | sed -e 's/://g')
        OPTIONS="$OPTIONS fingerprint=\"$FINGERPRINT\""
    fi

    # generate link block
    generateLinkBlock "$NAME" "$IPADDR" "$PORT" "$SENDPASS" "$RECVPASS" "$ALLOWMASK" "$OPTIONS"

    # Add default value
    if [ "$AUTOCONNECT" = "" ]; then
        AUTOCONNECT="yes"
    fi

    # Setup <autoconnect> block to automatically connect to servers
    if [ "$AUTOCONNECT" = "yes" ]; then
        echo "<autoconnect period=\"30\" server=\"$NAME\">"
    fi
fi

I know the file content :D I want you to execute this file in order to see if the environment variables generate the right config format

Whoosh!  Sorry about that 🤦♂
Running the script outputs nothing:
/inspircd/conf $ /inspircd/conf/links.sh
/inspircd/conf $ echo $?
0

/inspircd/conf $ set -x
/inspircd/conf $ /inspircd/conf/links.sh
+ /inspircd/conf/links.sh
/inspircd/conf $ echo $?
+ echo 0
0

Hehe, just noticed, that I forgot that this script requires a parameter: ./conf/links.sh LINK1 should work.

Ah gotcha, so it runs the script and dynamically uses that in the config instead of writing to the file, assuming I am understanding correctly, right?:
$ /inspircd/conf/links.sh LINK1
<link name="inspircd1.example.com"
      ipaddr="<IP_ADDRESS>"
      port="7000"
      sendpass="password"
      recvpass="password"
      allowmask="<IP_ADDRESS>/24"
      >
<autoconnect period="30" server="inspircd1.example.com">

Well, it's working now after starting from scratch.  I was probably just trying to change settings too quickly and not synchronizing them on both of my servers.  Thanks for the help!

And yes, execute config includes in InspIRCd use the classic Unix way of communicating, they just read std::out and consider it config.


Markdown widget makes many unintended changes

This diff shows what markdown was changed when in actual fact Title was the only thing intended to be changed - https://gitlab.com/dnk8n/research_db_docs/-/commit/46509df301ecb1884cc59dac14098a4fd1727f78
The changes broke the URLs, i.e. see lines 67 - 92

Since it is open-source, it should be possible to fork this repo and deploy to a personal Netlify site. Not all titles are filled in making it difficult to navigate, but in this example the URL is /admin/#/collections/doc/entries/creating-your-site

I created a branch netlify-cms/issues/4497 which will remain in the state above.

Closing as stale


Exception occurred during request processing: ScriptingException TypeError: Cannot call method "indexOf" of null (int_adyen/cartridge/scripts/adyenRedirectVerificationSHA256.ds#162)

https://github.com/Adyen/adyen-salesforce-commerce-cloud/blob/fdbe9f9f88accb4dba6e6e679f04afcdcba54fdc/cartridges/int_adyen/cartridge/scripts/adyenRedirectVerificationSHA256.ds#L162

Hi @kirill-kostiuk ,
Could you make sure you imported the metadata?
See here: https://docs.adyen.com/developers/plug-ins-and-partners/salesforce-commerce-cloud/integrating-a-cartridge/set-up#metadataimport

Hi @maassenbas, yes I've imported all necessary metadata.


Don't bottleneck Wave intrinsics through WaveMask* for spirv.

Fixes #4097.

looks good to me except for that one line.


RN's package.json specifies in-exact version numbers, which routinely breaks builds.

Description
The package.json for this project specifies almost all dependencies with a ^. This routinely breaks older builds.
React Native 0.32.1 was previously using uglify-js 2.8.5. This morning/last night uglify-js published v2.8.8.  Since RN specifies "uglify-js": "^2.6.2" in it's package.json, npm began pulling in the latest v2.8.8.
This new version of uglify-js now conflicts with lodash 4.1.x usage alongside react-native. Our build which worked just fine yesterday was broken by this change. runInContext was no longer defined correctly on a release/minified build from the react-native packager. See npm page here for release history of uglify-js.
Comparison of symbol definition change causing this breakage:
<EMAIL_ADDRESS>
e.runInContext=n

<EMAIL_ADDRESS>
r.runInContext=runInContext

This change meant that our JS file threw an exception on load, and was completely broken. This has to be the dozenth time this has happened to since switching to React Native, and has wasted countless hours.
Solution
Stop using in-exact versions. Specify exactly the version of dependency you need. (Why on earth would you want dependency version changes happening underneath you?)
Additional Information

React Native version: 0.32.1
Platform: iOS/Android
Operating System: OSX

Using exact versions in package.json doesn't solve this thoroughly because of transitive dependencies. Use yarn.lock on the app developer's side if you want to defend against this.

Thanks for catching this @MattFoley. I agree that using yarn.lock would help you avoid this problem on your app, although what happens for new projects? The template for init doesn't include yarn.lock and so all new projects will break. For people still using npm there isn't much they can do either (I guess theres shrinkwrap, but..)
I was able to fix this on someone's project by running yarn add<EMAIL_ADDRESS>-save

Definitely switching to yarn, but that wouldn't fix this issue for new projects. i.e.: If I had started a new project, which didn't yet have a yarn.lock file, it would pull in the wrong versions of RNs dependencies, and this new yarn.lock file would be still be incorrect.
Sent from my iPhone

On Mar 7, 2017, at 1:10 PM, James Ide<EMAIL_ADDRESS>wrote:
Using exact versions in package.json doesn't solve this thoroughly because of transitive dependencies. Use yarn.lock on the app developer's side if you want to defend against this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks Brent, that's exactly what I ended up doing to fix this issue this morning. It just took me a few hours to track this down as the problem.
Sent from my iPhone

On Mar 7, 2017, at 1:16 PM, Brent Vatne<EMAIL_ADDRESS>wrote:
Thanks for catching this @MattFoley. I agree that using yarn.lock would help you avoid this problem on your app, although what happens for new projects? The template for init doesn't include yarn.lock and so all new projects will break. For people still using npm there isn't much they can do either (I guess theres shrinkwrap, but..)
I was able to fix this on someone's project by running yarn add<EMAIL_ADDRESS>-save
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

To be fair, Ide is right, this isn't just a problem with React Native, but more a problem with the JS npm ecosystem as a whole. Having RN specify exact dependencies would at least help a little bit. :)
Sent from my iPhone

On Mar 7, 2017, at 1:16 PM, Brent Vatne<EMAIL_ADDRESS>wrote:
Thanks for catching this @MattFoley. I agree that using yarn.lock would help you avoid this problem on your app, although what happens for new projects? The template for init doesn't include yarn.lock and so all new projects will break. For people still using npm there isn't much they can do either (I guess theres shrinkwrap, but..)
I was able to fix this on someone's project by running yarn add<EMAIL_ADDRESS>-save
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Just re-bundled and can confirm the change in the uglify repo fixed the issue. I was experiencing this issue on RN 0.36.1 today.

Nice effort put in by @brentvatne to help with the debugging to get a patch out. Going to close this issue since we think it's been resolved upstream. If you're running into this problem, clear your node_modules (or yarn upgrade) and try installing them fresh again.

I gather this is why I'm getting Unhandled JS Exception: Can't find variable: runInContext running a release build. However yarn add<EMAIL_ADDRESS>-save didn't seem to work for me. Is this issue unrelated?

That's the same problem. You'll want to verify that 2.8.5 was installed (check the output). I actually changed my package.json manually to 2.8.5.
Sent from my iPhone

On Mar 8, 2017, at 3:02 AM, Kyle Johnson<EMAIL_ADDRESS>wrote:
I gather this is why I'm getting Unhandled JS Exception: Can't find variable: runInContext running a release build. However yarn add<EMAIL_ADDRESS>-save didn't seem to work for me. Is this issue unrelated?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Uglify 2.8.9 (published a few hours ago) the latest works. It was only 2.8.8 that broke RN. Shouldn't need to lock in at 2.8.5 as was previously suggested.

Ah right, I think in my case it was an unfortunate combination of the two issues. Glad this is sorted anyway!

Resolved in https://github.com/mishoo/UglifyJS2/issues/1573

Oh man, the last two days have suucked because of this.

Thank you for figuring this out! @MattFoley

Well, that was an hour of my life I won't get back. Glad I found this issue. This is impacting newer versions as well. I'm on RN 0.41.2 and npm i --save<EMAIL_ADDRESS>fixed the problem.

@brentvatne / @ide I believe I'm running into this same problem again just now, and I'm digging into what the specific issue is.
I'm looking at an old version of my code base that hasn't changed in months, and when rebuilding now, a ListView in that version no longer works properly, and I'm seeing a slew of dependency differences from the npm install. These are all coming from automated build jobs, so user error is doubtful. Will re-ping this thread if I solve it, currently looking at uglifyjs again, which was on v2.7.5 last this was built, and is now on v2.8.12.

That looks like that was the problem again. @brentvatne Thank you for adding that to the package.json!  (v2.7.5 was exactly the right version to be on it looks like!)


I cant open the Terminal

i just try to open the terminal, but in less than one second it is closed, it shows some text that i cant read because of the speed of the closing.

How are you opening the terminal???

Hello SaumitraVerma , what i am doing is:
Terminal menu --> New Terminal

I am using VS Code on Ubuntu 20.04 LTS, 64 bits  and GNOME version 3.36.2

Version: 1.46.1
Commit: cd9ea6488829f560dc949a8b2fb789f3cdc05f5d
Date: 2020-06-17T21:13:08.304Z
Electron: 7.3.1
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Linux x64 5.4.0-40-generic snap

Can you update to 1.47 and try again.

I found the problem, the thing is that I am using ZSH with my Terminal, and I have PyEnv on it.
with the installation of PyEnv, I had to put this lines on .zshrc:
#PyEnv Confirguration
Load pyenv automatically by adding
the following to ~/.bashrc:
export PATH="/home/rafaelortega/.pyenv/bin:$PATH"
eval "$(pyenv init -)"
eval "$(pyenv virtualenv-init -)"
so my terminal detects PyEnv.
If I erase that lines, the terminal in Visual Studio works perfectly.
Do you know hoy to use Terminal from VS Code with PyEnv?

Reinstala


Fix ERB comments in README

This is a pretty silly commit but while working on #36 I was trying to figure out why comments weren't colored appropriately. Comments in erb files have a <%# comment %> syntax rather than # comment as per Ruby. I'm not convinced this is a good change as it adds noise and might take away from the readability of the code sample, but is technically more correct (and it now colors it in grey-ish).
This PR replaces all such occurrences in README.md.

Great that you noticed this.

I'm not convinced this is a good change as it adds noise and might take away from the readability of the code sample

I agree, I have 2 ideas on my mind:

do you think HTML comments would be better maybe <!-- -->?
what about extracting file names from code snippets and adding them to the text above the code snippet?

I've extracted the filenames from the codeblock, thanks for bringing this up @SaladFork 🍺


#916 header global and country shouldnt change to tablet view for screens larger than 1024px

#916 #915
Country and Global headers shouldn't change to tablet/mobile layout until it's <1024px.
Also addresses some glitchyness with the mega menu popup trigger (Issue #915)

@theadamparker , works fine for me, can you review?

Chromatic visual tests don't look entirely accurate, but reviewing in Storybook everything looks correct


Crawl smart contract information from Etherscan

Work already started: https://github.com/bugout-dev/moonstream/pull/93

Ressolved in #93


Simplify Source Code of Website Examples

Note: This is landed. Regressions are tracked in #1873
Background
Regarding the source code of our website examples:

these examples are currently a lot more complex than they would need to be if they weren't part of the website.
In addition the complexity makes it hard for us to update these examples with the latest API simplifications without breaking some existing website integration.
This affects our users who look at these examples as starting points, but do not get to see the DeckGL API at its best/simplest.

Original Functionality
The website augments the examples in the following ways, and any simplification should maintain current functionality:

worker setup for loading
viewport transitions between examples
prop value injection from additional HTML controls
map sharing between examples

Goals
The split of each example into two files (app.js and deckgl-overlay.js) is very artificial from the end user's point of view.

[x] This can be avoided by moving the DeckGLOverlay class into app.js and exporting it from there.
[x] A simple global flag can be used to prevent the examples from instantiating as soon as loaded. This removes one reason for splitting demos into two classes.
[x] The availability of reuseMap prop in react-map-gl means each DeckGLOverlay can manage its own map. This removes the other reason for the split, allowing:
[x] the Root and DeckGLOverlay classes to be unified into a single App class.
[x] Prop injection from the website can be coded in a less intrusive way by using the local demo data as default values in the render method.
[x] export metadata for the demos can be exposed in a more "obvious" way so the user can immediately see what can be cut.

Once these changes are done, the examples should be sufficiently stand-alone that it should be possible to update them to use the latest APIs (like auto resize, auto URL loading etc).
To Do List

[x] Add label and assign to milestone
[x] Coding
[ ] Whats new update
[ ] Test

@Pessimistress Please review ^^

Tracking remaining work in #1873


[Feedback]Doc/Configuration improvement: Make Calico default CNI for K8S, and flannel default for K3S

Describe your scenario
In the deployment option pages, the docs say "Calico on K8S / Flannel on K3S is under public preview, and Flannel on K8S / Calico on K3S is an experimental feature "
In Deployment configuration JSON parameters, the doc says "For K8s cluster, always use calico."
But by default, the auto-generated JSON configuration is using 'Flannel', and for people who test with K8S, they will use Flannel+K8S combination, which is not recommended and may encounter bugs.
Feedback

Fix document diff 1) and 2)
In our script, for K8S, make calico default unless the user explicitly chooses calico, For the K3S, the same.

Updated docs to indicate what is the default and the need to change the default CNI if using K8S.


Arguable improvements to GoodFileDialog

Adds some right-click actions. Uses drives instead of system dirs for the panel on the left. Small improvements in other parts of the codebase.
I'm done with this widget now, it's more than good enough given GodSVG's current quality.

I personally think system dirs should be listes alongside drive names, just like in blender.

The set_dir function fails if the path exists but cannot be opened.

The set_dir function fails if the path exists but cannot be opened.

Any way to detect that?

Yes, DirAccess.open() can return null, which is also exactly the error that happened to me which I sent on the Discord earlier.


Can't startunity

Hi,
chroot had been running great for months until I had the power run out on me and could not restart anymore.
I unsuccessfully tried to update chroot, and the output can be found below.
I don’t mind reinstalling everything, BUT I do have a dir in my home that I really want to backup.
If anybody could help me either get chroot up and running again or back up the dir I would be super appreciative!
Thank you so much!
Terminal outputs:
I unsuccessfully tried to update chroot:
chronos@localhost / $ sudo sh ~/Downloads/crouton -n trusty -u
Downloading latest crouton installer...
######################################################################## 100.0%
/usr/local/chroots/trusty already exists; updating it...
Enter encryption passphrase for trusty:
Preparing chroot environment...
Installing brightness into the chroot...
Installing croutonpowerd into the chroot...
Installing croutonversion into the chroot...
Installing host-dbus into the chroot...
Installing host-x11 into the chroot...
Installing pulseaudio-default.pa into the chroot...
Installing croutoncycle into the chroot...
Installing croutontriggerd into the chroot...
Installing croutonxinitrc-wrapper into the chroot...
Installing setres into the chroot...
Installing xinit into the chroot...
Installing xbindkeysrc.scm into the chroot...
Installing xorg-intel-sna.conf into the chroot...
Installing xserverrc into the chroot...
Installing xserverrc-xorg into the chroot...
Installing xserverrc-local.example into the chroot...
Installing startunity into the host...
Installing crouton-noroot into the chroot...
Installing startunity into the chroot...
Installing gnome-session-wrapper into the chroot...
Installing crouton-unity-autostart into the chroot...
Installing unity-autostart.desktop into the chroot...
Installing unity-profiled into the chroot...
Installing enter-chroot into the host...
Installing delete-chroot into the host...
Installing edit-chroot into the host...
Installing mount-chroot into the host...
Installing unmount-chroot into the host...
Installing crash_reporter_wrapper into the host...
Installing target core...
Preparing environment...
Preparing software sources...
 Done
W: GPG error: http://star-www.st-andrews.ac.uk trusty/ Release: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY 51716619E084DAB9
Ensuring system is up-to-date...
Reading package lists... Done
Building dependency tree
Reading state information... Done
Calculating upgrade... Done
The following packages were automatically installed and are no longer required:
  firefox-locale-en libupstart1 python-pycurl
Use 'apt-get autoremove' to remove them.

Need to get 324 MB of archives.
After this operation, 4643 kB of additional disk space will be used.
WARNING: The following packages cannot be authenticated!
  r-base-core r-cran-boot r-cran-cluster r-cran-foreign r-cran-mass
  r-cran-kernsmooth r-cran-lattice r-cran-nlme r-cran-matrix r-cran-mgcv
  r-cran-survival r-cran-rpart r-cran-class r-cran-nnet r-cran-codetools
  r-base r-recommended r-base-dev r-doc-html r-base-html
E: There are problems and -y was used without --force-yes
Failed to complete chroot setup.
Unmounting /mnt/stateful_partition/crouton/chroots/trusty...

Despite the error, I tried to startunity again:
chronos@localhost / $ sudo startunity
Password:
Enter encryption passphrase for trusty:
Entering /mnt/stateful_partition/crouton/chroots/trusty...
A chroot setup script still exists inside the chroot.
The chroot may not be fully set up.
Would you like to finish the setup? [Y/n/d] Y
Preparing chroot environment...
Installing target core...
Preparing environment...
Preparing software sources...
 Done
W: GPG error: http://star-www.st-andrews.ac.uk trusty/ Release: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY 51716619E084DAB9
Ensuring system is up-to-date...
Reading package lists... Done
Building dependency tree
Reading state information... Done
Calculating upgrade... Done
The following packages were automatically installed and are no longer required:
  firefox-locale-en libupstart1 python-pycurl
Use 'apt-get autoremove' to remove them.

Need to get 324 MB of archives.
After this operation, 4643 kB of additional disk space will be used.
WARNING: The following packages cannot be authenticated!
  r-base-core r-cran-boot r-cran-cluster r-cran-foreign r-cran-mass
  r-cran-kernsmooth r-cran-lattice r-cran-nlme r-cran-matrix r-cran-mgcv
  r-cran-survival r-cran-rpart r-cran-class r-cran-nnet r-cran-codetools
  r-base r-recommended r-base-dev r-doc-html r-base-html
E: There are problems and -y was used without --force-yes
Failed to complete chroot setup.
The chroot setup script may be broken. Your chroot is not fully configured.
Removing the chroot setup script. You may want to update your chroot again.

X.Org X Server 1.15.1
Release Date: 2014-04-13
X Protocol Version 11, Revision 0
Build Operating System: Linux 3.2.0-61-generic x86_64 Ubuntu
Current Operating System: Linux localhost 3.8.11 #1 SMP Mon Apr 27 08:37:50 PDT 2015 x86_64
Kernel command line: cros_secure console= loglevel=7 init=/sbin/init cros_secure oops=panic panic=-1 root=/dev/dm-0 rootwait ro dm_verity.error_behavior=3 dm_verity.max_bios=-1 dm_verity.dev_wait=1 dm="1 vroot none ro 1,0 2506752 verity payload=PARTUUID=4aa30189-29c0-1640-af81-1abb4eb40031/PARTNROFF=1 hashtree=PARTUUID=4aa30189-29c0-1640-af81-1abb4eb40031/PARTNROFF=1 hashstart=2506752 alg=sha1 root_hexdigest=19983e99694f43f58c9b7fe794a36276d6631a9b salt=c1f66bbcb95439a8ee2d0617298166203029c4b74e1da8c2eb976c10a5fbe491" noinitrd vt.global_cursor_default=0 kern_guid=4aa30189-29c0-1640-af81-1abb4eb40031 add_efi_memmap boot=local noresume noswap i915.modeset=1 tpm_tis.force=1 tpm_tis.interrupts=0 nmi_watchdog=panic,lapic iTCO_vendor_support.vendorsupport=3
Build Date: 30 July 2014  12:21:54AM
xorg-server 2:1.15.1-0ubuntu2.1 (For technical support please see http://www.ubuntu.com/support)
Current version of pixman: 0.30.2
        Before reporting problems, check http://wiki.x.org
        to make sure that you have the latest version.
Markers: (--) probed, (**) from config file, (==) default setting,
        (++) from command line, (!!) notice, (II) informational,
        (WW) warning, (EE) error, (NI) not implemented, (??) unknown.
(==) Log file: "/var/log/Xorg.1.log", Time: Mon May  4 12:18:01 2015
(==) Using system config directory "/usr/share/X11/xorg.conf.d"
(EE)
(EE) Backtrace:
(EE) 0: /usr/bin/X (xorg_backtrace+0x48) [0x7f1cf0a4fd28]
(EE) 1: /usr/bin/X (0x7f1cf08a7000+0x1aca19) [0x7f1cf0a53a19]
(EE) 2: /lib/x86_64-linux-gnu/libpthread.so.0 (0x7f1cef9a4000+0x10340) [0x7f1cef9b4340]
(EE) 3: /usr/bin/X (0x7f1cf08a7000+0xb50d6) [0x7f1cf095c0d6]
(EE) 4: /usr/bin/X (xf86BusProbe+0x9) [0x7f1cf092fb19]
(EE) 5: /usr/bin/X (InitOutput+0x74d) [0x7f1cf093e06d]
(EE) 6: /usr/bin/X (0x7f1cf08a7000+0x596bb) [0x7f1cf09006bb]
(EE) 7: /lib/x86_64-linux-gnu/libc.so.6 (__libc_start_main+0xf5) [0x7f1cee3e3ec5]
(EE) 8: /usr/bin/X (0x7f1cf08a7000+0x44dde) [0x7f1cf08ebdde]
(EE)
(EE) Segmentation fault at address 0x0
(EE)
Fatal server error:
(EE) Caught signal 11 (Segmentation fault). Server aborting
(EE)
(EE)
Please consult the The X.Org Foundation support
         at http://wiki.x.org
 for help.
(EE) Please also check the log file at "/var/log/Xorg.1.log" for additional information.
(EE)
(EE) Server terminated with error (1). Closing log file.
/usr/bin/xinit: giving up
/usr/bin/xinit: unable to connect to X server: Connection refused
/usr/bin/xinit: server error
Unmounting /mnt/stateful_partition/crouton/chroots/trusty...
chronos@localhost / $

to backup, enter the chroot:
sudo enter-chroot
navigate to your /home dir
cp -avr /home/myusername/myfoldername ~/Downloads
Where,
-a : Preserve the specified attributes such as directory an file mode, ownership, timestamps, if possible additional attributes: context, links, xattr, all.
-v : Explain what is being done.
-r : Copy directories recursively.
then after you're sure it all copied correctly to Downloads, move it to usb and/or cloud storage as /Downloads may not always remain permanent storage.

That's one way to backup, another way is to use edit-chroot outside the chroot:
sudo edit-chroot -b trusty
That will save a compressed tarball in ~/Downloads by default.
You might want to re-download crouton (https://goo.gl/fd3zc) and then attempt to update your chroot one more time, the latest version of crouton is more forgiving when encountering update errors and that my be just enough to get you going.
I noticed a couple of problems -

W: GPG error: http://star-www.st-andrews.ac.uk trusty/ Release: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY 51716619E084DAB9

WARNING: The following packages cannot be authenticated!
r-base-core r-cran-boot r-cran-cluster r-cran-foreign r-cran-mass
r-cran-kernsmooth r-cran-lattice r-cran-nlme r-cran-matrix r-cran-mgcv
r-cran-survival r-cran-rpart r-cran-class r-cran-nnet r-cran-codetools
r-base r-recommended r-base-dev r-doc-html r-base-html
They could be related but I'm not familiar with them. If you could remove them I think your update would complete successfully.

Will try both solutions. Knowing I can back up my files is already super awesome. Thank you so much for the help!!!

I have the same issue as ericwaraujo.  Did the solution work for you?
These files mentioned are the R programming language packages through a ppa.  Somehow the public key was corrupted in the latest update of chrome OS.
When you say 'remove' these files, can you describe this method more in detail.  These are packages that are part of a ppa source and integral to an application I use.

Hi mathewarndt,
Last night I only had time to do the backup, following tedm suggestion and I have my backup now. Sweet!
Later I realized the packages are from R, but I had no time to remove them. (I have R installed but don't need it anymore, so removing them would work for me).
I also appreciate suggestions on how to remove these packages. Should I do
sudo enter-chroot
and then
apt-get remove
the packages?

By the way, I did try to update chroot as suggested by DennisLfromGA before removing the packages but it did not work.

@ericwaraujo I'd use:
apt-get remove --purge packagename
so you remove the config files as well.

YES! Back in business!
Thank you very much @tedm @DennisLfromGA !!!
What worked:
Enter chroot:
sudo enter-chroot
List all installed packages (the r troublemaking packages will be listed there):
dpkg --get-selections | grep -v deinstall
Remove the problematic packages:
apt-get remove --purge packagename
where packagename will r-base then r-base-core then r-base-html and any other r- packages (see list above). Remove them one by one, after the first few all will be gone. Tab complete works, fyi.
Do another inspection:
dpkg --get-selections | grep -v deinstall
If all r packages are removed, exit chroot:
exit
Get the latest crouton to update chroot:
Follow the link above provided by @DennisLfromGA, or/and follow
Common issues and reporting instructions
I guess if you need R you could still install it again after updating chroot, but that is just my guess, I have not tested this.


add multizone/regional support to gcp

I still need to do some end to end testing of this, but it is ready for review.
Fixes #765

Thanks for the review, I'll get those changes made.
I've manually tested with both regional and zonal disks, but what I haven't done it test that it works when using allowedTopologies  (https://kubernetes.io/docs/concepts/storage/storage-classes/#allowed-topologies) in the StorageClass definition. I'll want to support both I think.

  pvc-8ceb0e39-e88d-11e8-9cd6-42010a8000ac:
    Snapshot ID:        gke-ark-small-sandbox--pvc-5f34055a-618d-4f31-ac38-bd7a82e77270
    Type:               diskTypes/pd-standard
    Availability Zone:  us-central1-a__us-central1-b
    IOPS:               <N/A>

I held off adding allowedTopologies since it is in beta.

@wwitzel3 this is tested and ready for another review, correct?

units are failing due to removing the "\n" from the error msg

LGTM once fixed

Fixed. PTAL

@nrb PTAL / merge if you are happy with it.

@wwitzel3 I guess you should now add a release note to this PR


Don't like the _util.py and effects._fx_util.py modules

They show up as a use and there's no need for them to not just be in the init.py modules.

Resolved in v2.0.0


Is there any way you can make BottomSheet transparent or invisible without it affecting the content?

I used border radius on a Container inside a BottomSheet. The border radius only affects the container and by so doing the bottom sheet still shows. Is there any way to correct this?
I tried opacity but it affects everything.
When I put visible to false it doesn't show the content.
Thank you

Would you share a code sample please?

Sure
On Wed, Dec 21, 2022, 10:33 PM Feodor Fitsner @.***>
wrote:

Would you share a code sample please?
—
Reply to this email directly, view it on GitHub
https://github.com/flet-dev/flet/issues/744#issuecomment-1362184200, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A2HRAW6ML73POHXSISQJHGTWOOAUXANCNFSM6AAAAAATFPNRIQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Sample code here....the bottom sheet still shows after creating a border
radius with the container. Thanks Foeder , more grease to your elbow ❤
On Thu, Dec 22, 2022, 2:57 PM Edwin Setsoafia @.***>
wrote:

Sure
On Wed, Dec 21, 2022, 10:33 PM Feodor Fitsner @.***>
wrote:

Would you share a code sample please?
—
Reply to this email directly, view it on GitHub
https://github.com/flet-dev/flet/issues/744#issuecomment-1362184200,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2HRAW6ML73POHXSISQJHGTWOOAUXANCNFSM6AAAAAATFPNRIQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Added enabled and disabled date properties to date picker.

Hi,
I added a property for enabled and disabled dates to the date picker component which both take an array of date objects.
Enabled dates will only enable all the dates in the array.
Disabled dates will only disable all the dates in the array.
What do you think?
This is useful for a project I work on as it will allow us to use two date pickers for a date range which only have dates enabled that we have data for.
David

Great work! Just refactored a bit the code, apart from that it's very useful, thanks!


Multiple instances of SDLRouter running confuses SDLProxy, not connecting to the router with a HU connection.

In the generic situation this is probably an edge case where two SDL apps check for a running router almost simultaneously and, finding none, each start their routers resulting in two routers running at the same time.  Our situation is different in that we have a companion app that only starts its router once it detects a connection to a known head unit.  The companion app's router will be started regardless of other routers running since only the companion app's router will get a connection from the HU SDLCore.  Unfortunately, the other SDL apps do not always find and connect to this router even though it broadcasts the 'sdl.router.startservice' message for triggering SDLProxy startup.  What I see in the log is that SdlBroadcastReceiver validates the router that broadcast the message but the proxy (or actually the TransportBroker) later tries to check for a transport connected router and always finds the non-connected router first, short-circuiting and returning false.
In order to repro this you need a "companion" app that will start its SdlRouter when it gets a connection to a "known" HU.  This app's SdlRouter needs to use a custom BT UUID which the SDLCore on the "known" HU will connect to to the exclusion of other SdlRouters broadcasting with the standard UUID.

Install the "companion" app and other SDL multiplexer-enabled apps on the HS device
Restart the HS
Start the HU, SDLCore, HMI
Try to run one of the SDL apps

What I see in the logs is that the "companion" app's router gets a connection from SDLCore on the HU and then broadcasts the TransportConstants.START_ROUTER_SERVICE_ACTION message.  Then, other apps' SdlBroadcastReciever check and validate the broadcasting router.  Finally, SDLProxies for those apps check to see of a running router has a transport connection but get back a false response because they checked the wrong running router.  I'm attaching a section of logcat logs which contains messages from two routers (pid 3504, 5368) and one of the apps (pid 3630) which shows what happens.  Short story:
06-21 13:30:26.974  3630  3630 V Sdl Broadcast Receiver: SDL enabled by router service from com.uievolution.companionapp compnent package com.uievolution.companionapp - com.main.companionapp.sdl.SdlRouterService
...
06-21 13:30:27.028  3630  3630 D SdlTransportBroker: Bound to service ComponentInfo{com.uievolution.companionapp/com.main.companionapp.sdl.SdlRouterService}
...
06-21 13:30:27.031  3630  5561 D Sdl Broadcast Receiver: Looking for Service: sdlrouterservice
06-21 13:30:27.034  3630  5561 D Sdl Broadcast Receiver: Found service: com.uievolution.sdl.CTSMediaApp1.livio.SdlRouterService in package com.uievolution.sdl.CTSMediaApp1
06-21 13:30:27.038  3630  5561 W Sdl Broadcast Receiver: Is Connected? Returning false

Note in the first two lines the app is binding to the correct router but in the last three lines it checks the connection state of the incorrect router.
I have thought of a couple ways of dealing with this issue.  One way would be to have each app shut down its own router if the router is not connected to the HU.  Another option would be to propagate the package/service information about the connected router from the broadcast receiver to all parts of the code that need to check the router's connection.  I'll create a pull request for a solution that checks the broadcasting router to see if it is transport connected.

issue.txt

So the issue stems from having a router service that does not obey the current "correct" lifecycle flow. The companion app will always start up and keep up its router service regardless of whether or not another router service is running that has went through the correct lifecycle flow. If the routers obey the correct flow this issue doesn't propagate, correct?

Yes, mostly, I guess.  Though I have seen a couple times where the race-condition case, that I mentioned at the top, occured such that there were two, unconnected, generic UUID routers running at the same time.  I assumed this occured in the edge condition of two SDL apps checking almost simultaneously for a router and, neither finding one, both starting their own.  Also, with the current "correct" flow there doesn't seem to be a "correct" way for different routers to use different UUIDs to advertise their services, which is what we're trying to do.

Hmm I've never seen it with the normal flow, but that's not to say it's impossible to happen.
Yes, that is by design. In 99% of integrations, no app should be trying to change away from the designated SDL UUID. The correct flow is for one of the already installed SDL enabled apps to propagate the router service preventing the user from having to download an extra app.

Any idea when PR https://github.com/smartdevicelink/sdl_android/pull/312 will be reviewed/merged?

@mrapitis any update on the review?

This should be resolved with PR #312 being merged into develop. @jthrun can you confirm?

This issue is fixed.  I'm not seeing the initial connection issue that was happening before.


examples/windowsize: forbid a green button for fullscreening when a window has a size limitation

Run go run ./examples/windowsize -maxwindowsize=640x640
Press F

You will see a small rendering result.

#2546 seems a blocker.

I think we have to forbid the green button in this case.

SetFullscreen(true) should still be allowed. In order to fix this, we need a part of the original fix https://github.com/hajimehoshi/ebiten/commit/9a5ef1b55312dfd0433d957a8b6dd4e07b909849.


Add description for each members

when the user clicks on a particular member on the member's page it should display the description and purpose of that member

Idk about the description but we can have a tooltip when user hovers over role (Mentor, Event Head, etc.) with the purpose/description of that role.

yes that would be cool


Better debug routes

Description
See the title above.
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[ ] Conforms to style guide

Screen capture:

@enix-app Are you able to address @MGatner 's requests in the next couple of days? We're shooting for a release next Monday and would love to include this.

Handled via #2587

Sorry, but thank you.


fix pospone

The following code:
T{ : GT1 123 ; -> }T
T{ : GT4 POSTPONE GT1 ; IMMEDIATE -> }T
T{ : GT5 GT4 ; -> }T
T{ GT5 -> 123 }T

Lead to the core test failure:
Error: in line: 647 T{ : GT5 GT4 ; -> }T  expected [ 291 ] to deeply equal [].
Error: in line: 648 T{ GT5 -> 123 }T  expected [] to deeply equal [ 291 ].

Wow, you have a copy of the standard?

If you have a traditional Forth with just default and immediate words, POSTPONE FOO should be the same as COMPILE FOO if FOO has default compilation semantics, and the same as [COMPILE] FOO if FOO is immediate.

It seems your problem is with POSTPONEing non-immediate words.  If bar is not immediate
: foo   postpone bar ;

should be the same as
: foo   ['] bar postpone literal  ['] compile, compile, ;

Here is hoe COMPILE, described in the standard.

Note: COMPILE, is just the same as , in a traditional threaded Forth.

Yes, that is my issue. I have JS-Threaded Forth ;)

I see your compiler is building JavaScript code at run time.  So COMPILE, should take an xt and generate code to call that xt.
POSTPONE IF should generate code to call IF.
POSTPONE + should generate code to generate code to call +.
Simple!

That is what I am doing. But As you can see in the failing case:
T{ : GT1 123 ; -> }T
T{ : GT4 POSTPONE GT1 ; IMMEDIATE -> }T
T{ : GT5 GT4 ; -> }T
T{ GT5 -> 123 }T

Lead to the core test failure:
Error: in line: 647 T{ : GT5 GT4 ; -> }T  expected [ 291 ] to deeply equal [].
Error: in line: 648 T{ GT5 -> 123 }T  expected [] to deeply equal [ 291 ].

The comp.lang.forth link:
https://groups.google.com/forum/#!topic/comp.lang.forth/BlA-81dmWow

What does your GT5 do?  It seems it doesn't leave 123 on the stack.  Why doesn't it do that?
GT4 is an immediate word.  When GT4 runs it should compile GT1 into the current definition.  So : GT5 GT4 ; should have the same effect as : GT5 GT1 ;.

123 HEX = 291 DEC

Well, I tried http://drom.io/forth, and typed in the test code.
see GT4 clearly shows the generated JavaScript code is wrong.

Looks like POSTPONE has to check if the WORD, it is POSTPONing is immediate or not.
if (immediate) {
    compile the WORD into the current definition.
} else {
   compile the code that will compile the WORD into current definition when being executed.
}

Yes!

Thanks, fixing. 7 more issue to go ;)

from here: https://groups.google.com/d/msg/comp.lang.forth/BlA-81dmWow/j81maLLaAAAJ
GT1 is a word with default compilation semantics (i.e., you get them
from Section <IP_ADDRESS> cited above.  POSTPONE GT1 appends the
compilation semantics of GT1 to GT4, whereas just writing GT1 would
append the execution semantics of GT1 to GT4.  Or, in other words,
POSTPONE GT1 appends the appending of the execution semantics of GT1
to GT4.  When GT4 is performed in the next line, it appends the
execution semantics of GT1 to GT5.  And when GT5 is performed in the
next line, it performs the execution semantics of GT1 and pushes 123
on the stack.

anton

T{ : GT6 345 ; IMMEDIATE -> }T
T{ : GT7 POSTPONE GT6 ; -> }T
T{ GT7 -> 345 }T

GT6 is an immediate word with the following compilation semantic == execution semantic is to push 345 to the stack.
POSTPONE GT6 appends this compilation / execution semantic of GT6 to GT7 by just compiling call to GT6.
GT7 just executes this call.
T{ : NOP : POSTPONE ; ; -> }T
T{ NOP NOP1 NOP NOP2 -> }T
T{ NOP1 -> }T
T{ NOP2 -> }T

; is an immediate word with the following compilation semantic == execution semantic of finishing the colon definition.
POSTPONE ; appends this compilation / execution semantic of ; to NOP by just compiling call to ;.
NOP NOP1 just executes this call.


HARP-9898: Fix TextElement's (labels) TextBufferObject handling.

This changes includes:

Cleanup label's TextBufferObject whenever it's layout changes,
Provide proper copies (not references) of TextRenderStyle and TextLayoutStyle for
TextBufferObject creation.
Cleanup TextBufferObject whenever TextElementState is reset or replaced
(derived) from predecessor.

Signed-off-by: Krystian Kostecki<EMAIL_ADDRESS>Thank you for contributing to harp.gl!
Before requesting a pull request, please remember to check the following documents:

contribution guidelines
coding style

If you are adding new functionality we would highly appreciate if you can describe what is the capability you are adding and even better if you can add some examples. Please also remember to add tests for it.
CI Check
Our bots will check whether your PR can be directly integrated into the mainline. We have some internal integration tests running on the background, our bots will inform you of the next steps and someone from our team will take a look and help if needed!
And please do not forget to sign-off your commit! You can read more about DCO here. But, in short, you just need to use git commit -s or append --signoff when you are committing to the repo.
Happy contributing!

GPU tests please


Thinking of the return of the "utils" module

I liked the idea of simplifying and making OceanLab more direct and specific. But I missed the old utils module, where you could get e.g. argdistnear easily. Why not come up with something like "tools" or "support" and bring back some of the "utils"?

I can take a look later, but either @Ryukamusa or you can also try to do it if have some time.

I'm just not sure yet where and how to add it.
I'm adding two new functions to OceanLab.dyn, which I will push soon to the project


Logical Error in Hierarchical Decoder

The hierarchical decoder is incorrect in terms of mapping an input address to the correct row. For example, for a 64 row decoder, address 1 mapped to row 16 and address 32 mapped to row 2. Unfortunately, my current decoder has widely diverged from the existing implementation so I can't generate a good enough clean patch.
This gist shows my patch for fixing the issue. At the very least, whatever fix for this issue should pass the included tests. This gist shows the full decoder for reference.

Hi,
Yes, this is fixed in our internal copy and will be released soon.
Thanks for letting us know.
Matt
On Wed, Nov 28, 2018, 22:00 Olalekan Afuye<EMAIL_ADDRESS>wrote:

The hierarchical decoder is incorrect in terms of mapping an input address
to the correct row. For example, for a 64 row decoder, address 1 mapped to
row 16 and address 32 mapped to row 2. Unfortunately, my current decoder
has widely diverged from the existing implementation so I can't generate a
good enough clean patch.
This gist
https://gist.github.com/lekez2005/2a1f61a6fd873fd4d14a2c744a90c7a3
shows my patch for fixing the issue. At the very least, whatever fix for
this issue should pass the included tests. This gist
https://gist.github.com/lekez2005/2b40bd5445e0e735a3c80a7f7c94b083
shows the full decoder for reference.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/VLSIDA/OpenRAM/issues/43, or mute the thread
https://github.com/notifications/unsubscribe-auth/AL35Lw_Jpm71tTiSmux35mwjPijg95dpks5uz3fygaJpZM4Y4_21
.

Resolved in upcoming release.


Error during CAS setup.

To setup CAS, the below steps were taken:

run testing/ensure_testbed.sh. This runs to completion, and we see

...
creating static-1 profile
creating dynamic-1 profile
creating dynamic-2-calico profile
creating capi-kubeadm-3node clusterspec
creating capi-eks clusterspec
creating xplane-eks-3node clusterspec
--- All done ---

run testing/test_basecluster_deploy_with_capd.sh, which runs to completion:

Waiting for "set -o pipefail; clusterctl -n capd-1 describe cluster capd-1-capi-quickstart|grep MachineDeployment|grep True > /dev/null" to evaluate to true ...
Waiting for "set -o pipefail; clusterctl -n capd-1 describe cluster capd-1-capi-quickstart|grep MachineDeployment|grep True > /dev/null" to evaluate to true ...
------- waiting for guestbook to become healthy ---------
Waiting for "set -o pipefail; argocd app list|grep capd-1-guestbook-dynamic|grep Healthy > /dev/null" > to evaluate to true ...
--- test ok ---

Create a dynamic bundle from the manifests in bundles/capi-cluster-autoscaler.
arlon bundle create cas-bdl --tags cas,devel,test --desc "CAS Test" --repo-url https://github.com/arlonproj/arlon.git --repo-path bundles/capi-cluster-autoscaler

Create a profile that uses this bundle.
arlon profile create cas-prof --repo-url http://<IP_ADDRESS>:8188/myrepo.git --repo-base-path profiles --bundles cas-bdl,guestbook-dynamic --desc "cas profile" --tags test

Create a clusterspec with CAS enabled.
arlon clusterspec create cas-test-eks --api capi --cloud aws --type eks --kubeversion v1.21.10 --nodecount 2 --nodetype t2.medium --tags devel,test --desc "dev/test" --region us-west-2 --sshkey shaunak-key-2 --casenabled

Deploy a gen1 cluster arlon cluster deploy --repo-url http://<IP_ADDRESS>:8188/myrepo.git --cluster-name cas-gen-2 --profile cas-prof --cluster-spec cas-test-eks.

Run clusterctl to get the kubeconfig of the workload cluster.

~ ➤ clusterctl get kubeconfig cas-gen-1 -n cas-gen-1 > cas-gen-1.yaml
~ ➤ export KUBECONFIG=/home/ubuntu/cas-gen-1.yaml
~ ➤ kubectl get pods -A
NAMESPACE     NAME                                  READY   STATUS              RESTARTS   AGE
default       guestbook-ui-85985d774c-zvv4t         1/1     Running             0          12m
kube-system   aws-node-b8b92                        1/1     Running             0          7m33s
kube-system   aws-node-bk26l                        1/1     Running             0          7m12s
# the cluster autoscaler pod is in ContainerCreating state for 12m
kube-system   cluster-autoscaler-56478974c9-swwkk   0/1     ContainerCreating   0          12m
kube-system   coredns-85d5b4454c-p28cq              1/1     Running             0          16m
kube-system   coredns-85d5b4454c-s8l4k              1/1     Running             0          16m
kube-system   kube-proxy-2cw8m                      1/1     Running             0          7m33s
kube-system   kube-proxy-4qd8f                      1/1     Running             0          7m12s

At this point we get a ComparisonError on running kubectl describe pod -n kube-system cluster-autoscaler-56478974c9-swwkk

A second attempt was made with slight variation where the cluster autoscaler bundle is committed to myrepo and values.yaml
is edited to include:
arlon
  clusterName: cas-gen-1

This time we get a different error:

The cluster autoscaler pod fails to mount the management cluster kubeconfig.

~ ➤ kubectl describe pod -n kube-system cluster-autoscaler-56478974c9-swwkk
Name:             cluster-autoscaler-56478974c9-swwkk
Namespace:        kube-system
Priority:         0
Service Account:  cluster-autoscaler
Node:             ip-10-0-116-231.us-west-2.compute.internal/<IP_ADDRESS>
Start Time:       Tue, 06 Sep 2022 08:43:42 +0000
Labels:           app=cluster-autoscaler
                  pod-template-hash=56478974c9
Annotations:      kubernetes.io/psp: eks.privileged
Status:           Pending
IP:
IPs:              <none>
Controlled By:    ReplicaSet/cluster-autoscaler-56478974c9
Containers:
  cluster-autoscaler:
    Container ID:
    Image:         k8s.gcr.io/autoscaling/cluster-autoscaler:v1.21.1
    Image ID:
    Port:          <none>
    Host Port:     <none>
    Command:
      /cluster-autoscaler
    Args:
      --cloud-provider=clusterapi
      --node-group-auto-discovery=clusterapi:namespace=cas-gen-1
      --cloud-config=/kubeconfigvolume/kubeconfig
    State:          Waiting
      Reason:       ContainerCreating
    Ready:          False
    Restart Count:  0
    Limits:
      cpu:     200m
      memory:  600Mi
    Requests:
      cpu:        100m
      memory:     300Mi
    Environment:  <none>
    Mounts:
      /kubeconfigvolume from kubeconfigvolume (ro)
      /var/run/secrets/kubernetes.io/serviceaccount from kube-api-access-ksp2q (ro)
Conditions:
  Type              Status
  Initialized       True
  Ready             False
  ContainersReady   False
  PodScheduled      True
Volumes:
  kubeconfigvolume:
    Type:        Secret (a volume populated by a Secret)
    SecretName:  cluster-autoscaler-management-kubeconfig
    Optional:    false
  kube-api-access-ksp2q:
    Type:                    Projected (a volume that contains injected data from multiple sources)
    TokenExpirationSeconds:  3607
    ConfigMapName:           kube-root-ca.crt
    ConfigMapOptional:       <nil>
    DownwardAPI:             true
QoS Class:                   Burstable
Node-Selectors:              <none>
Tolerations:                 node-role.kubernetes.io/master:NoSchedule
                             node.kubernetes.io/not-ready:NoExecute op=Exists for 300s
                             node.kubernetes.io/unreachable:NoExecute op=Exists for 300s
Events:
  Type     Reason            Age                    From               Message
  ----     ------            ----                   ----               -------
  Warning  FailedScheduling  8m45s (x6 over 13m)    default-scheduler  no nodes available to schedule pods
  Warning  FailedScheduling  8m16s (x2 over 8m26s)  default-scheduler  0/1 nodes are available: 1 node(s) had taint {node.kubernetes.io/not-ready: }, that the pod didn't tolerate.
  Warning  FailedScheduling  8m5s                   default-scheduler  0/2 nodes are available: 2 node(s) had taint {node.kubernetes.io/not-ready: }, that the pod didn't tolerate.
  Normal   Scheduled         7m55s                  default-scheduler  Successfully assigned kube-system/cluster-autoscaler-56478974c9-swwkk to ip-10-0-116-231.us-west-2.compute.internal
  Warning  FailedMount       103s (x11 over 7m54s)  kubelet            MountVolume.SetUp failed for volume "kubeconfigvolume" : secret "cluster-autoscaler-management-kubeconfig" not found
  Warning  FailedMount       80s (x3 over 5m52s)    kubelet            Unable to attach or mount volumes: unmounted volumes=[kubeconfigvolume], unattached volumes=[kubeconfigvolume kube-api-access-ksp2q]: timed out waiting for the condition

The CallHomeConfig resource entered the Error state. The status shows the specific error:
$ k -n eks-a get -oyaml callhomeconfig cluster-autoscaler
apiVersion: core.arlon.io/v1
kind: CallHomeConfig
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"core.arlon.io/v1","kind":"CallHomeConfig","metadata":{"annotations":{},"labels":{"app.kubernetes.io/instance":"eks-a"},"name":"cluster-autoscaler","namespace":"eks-a"},"spec":{"kubeconfigSecretKeyName":"value","kubeconfigSecretName":"eks-a-kubeconfig","managementClusterUrl":"https://AA61EB28E1D1E8C19BE78AE4F8F7782C.gr7.us-west-2.eks.amazonaws.com","serviceAccountName":"cluster-autoscaler","targetNamespace":"kube-system","targetSecretKeyName":"kubeconfig","targetSecretName":"cluster-autoscaler-management-kubeconfig"}}
  creationTimestamp: "2022-09-08T19:17:00Z"
  generation: 1
  labels:
    app.kubernetes.io/instance: eks-a
  name: cluster-autoscaler
  namespace: eks-a
  resourceVersion: "349420547"
  uid: f73380c1-a2ee-4b89-bc16-7186c1003925
spec:
  kubeconfigSecretKeyName: value
  kubeconfigSecretName: eks-a-kubeconfig
  managementClusterUrl: https://AA61EB28E1D1E8C19BE78AE4F8F7782C.gr7.us-west-2.eks.amazonaws.com
  serviceAccountName: cluster-autoscaler
  targetNamespace: kube-system
  targetSecretKeyName: kubeconfig
  targetSecretName: cluster-autoscaler-management-kubeconfig
status:
  message: 'unexpected error getting service account: serviceaccounts "cluster-autoscaler"
    is forbidden: User "system:serviceaccount:arlon:default" cannot get resource "serviceaccounts"
    in API group "" in the namespace "eks-a"'
  state: error

During reconciliation, the CHC controller logic (inside the arlon controller) failed to read the cluster-autoscaler service
account resource in the cluster namespace. It's missing some RBAC settings. I don't know why this wasn't a problem
before.


build: update to latest lnd version w/ msg router fix

go.sum update is incorrect?
When I ran go mod tidy locally it removed the old sum correctly.


Fix multi-file section content

If you have multiple files, the content/section code no-longer works.
This is because each file in the iteration sets options.content = options.content[option.section]. Therefore on the second call you're essentially settings  options.content = options.content[option.section][option.section], which is therefore undefined.
This PR should fix this bug.

Hello @fewstera thank you for reporting this bug!
Looking at the code I am questioning why the section-evaluation needs to be in the this.files.forEach loop. Would you move the original block
// apply section

if ( options.section ) {

	if ( ! options.content[ options.section ] ) {
		grunt.log.error( "content doesn't have section " + options.section );
	}

	options.content = options.content[ options.section ];
}

up to line 55 (after the normalize content part) and try if that solves the bug as well?

Oh, and more thing for keeping the history clean and to facilitate work for @MathiasPaumgarten when he is going to merge your PR. @fewstera can you please squash your commits to one commit?

@david-zacharias made those changes, and added a test to check multi-files.

Thanks for your contribution! Having tests for that is great. @MathiasPaumgarten will probably merge it soon.

This is awesome. Once again, can't believe that slipped through. It's harder than one thinks to write comprehensive tests.

Your fixes are available on npm under 1.6.1. Thanks I appreciate it. Lots of work happing on grunt-bake lately. I appreciate every contribution.