Some short term fixes for job logic

This PR fixes some of the issues with the current job logic that can lead to commits being unfinished when a job finishes. There are still a couple of issues that need to be fixed (mentioned below) and probably some others that have gone unnoticed. These issues should be addressed by @Tryneus  job logic refactor.
A couple of things that I think are still issues:

Stop job should be improved and it does not finish the stats commit:
The way stop job works now is through a goroutine in the master worker being blocked on the output commit for the job finishing and the StopJob function actually finishing this output commit when called. This goroutine will also update the etcd job state to killed. There is probably a better pattern for this that can encapsulate finishing the job in one place instead of having a commit closed in one place (StopJob function) and the job state updated in another (master worker). Also, stop job should be finishing the stats commit, but it is a little unclear where that should be happening in the current model.

Inconsistent order for update job state / commit finish during job finish:
Due to the way stop job is implemented, we cannot finish an output commit before updating job state except for when the job succeeds (where we have to or else we cannot egress). This is a confusing consequence of the stop job implementation and means failures during job finish have to be recovered differently dependent on whether the job succeeded or failed / was killed. To try and mitigate this in the short term, I have tried to stick to a consistent job finish sequence for non-successful and successful jobs with these changes (the exception being job timeout). For a non-successful job finish the sequence is finish stats commit, update job state, finish output commit. For successful job finish, the sequence is finish stats commit, finish output commit, update job state. With the current job logic where the master subscribes to the output branch and checks if each commit is finished then if not checks if the job state is terminal (and skips it if so), we have a situation where we can potentially have an output commit remain unfinished in the non-successful case (the machine crashed / failed between update job state and finish output commit) or a job state to remain in merging in the successful case (the machine crashed / failed between finish output commit and update job state). I added a recovery method for the former issue where we attempt a finish commit if we find an unfinished commit with a terminal job state, but the latter is a bit trickier because we skip based on the commit being finished before checking the job state (also it is harder to reverse engineer the correct job state). I think the former is the more problematic issue because it leaves a commit unfinished, but the latter should be fixed with the refactor.

LGTM

That might make things cleaner with what we are currently working with, but I don't think that is behavior we want to consider successful. I think we should only be making a finish commit call once and the new job refactor should handle wrapping up the job (finishing stats/output commit and updating job state) transactionally in every case so that we should never be in a situation where finish commit is executed more than once on a particular commit.


Support token offset for cached tokens

When we query the token cache for tokens, our mechanism of preemptive token expiration is not honored. Thus we have to check the expiration ourselves and force the refresh.

This is being redesigned. Closing this PR.


Backup and Restore widget settings

We already have the feature: "Feedback" ->"Share events for debugging", which effectively is a backup of settings (and of visible events also...)
but there is no way to restore these settings.
Looks like Backup / Restore may need their own page/section in Settings, or maybe "Feedback" page ("screen") should be renamed...

Implemented in v.3.1.1, see #320


Added spacy component

Added spacy component which works out of the box:

Can be added to any spacy pipeline as "fastcoref" after importing spacy_component
Resolves coreferences automatically and saves the resolved text into doc._.resolved_text
Saves the coreference clusters into doc._.coref_clusters

CorefResult now supports accessing the original text orderings for any down stream tasks, through the text_idx attribute(related issue). As this was also needed for the spacy component implementation.
Added tests for the spacy component

Approved. It would be also nice if you can add a code snippet of how to use the new Spacy integration to the README file.

I missed your comment on adding a snippet to README. I'll make a final commit to update it.


Jtc/fast feedback for concierge authenticators

https://www.pivotaltracker.com/story/show/186391232

There's nothing valuable in this PR, I will close for now.


added numericpattern247 to 254

solved issues #764 to #771

@aryashah2k Also solved issue number #763

1 pattern to go for the beginner tag

@aryashah2k  added numericpattern256. please review it

Please assign those issues before solving them It makes it hard for me to keep track of these things. Take precautions next time.

Please assign those issues before solving them It makes it hard for me to keep track of these things. Take precautions next time.


Feature - security warning on headers and footers

For every page, we require a security warning in the header and footer.
This will be a full-width div.  The background color will be light green, and contain black 14px arial text saying "COMMERCIALLY SENSITIVE"
Note: this appears to be an article covering how to do this: https://www.dita-ot.org/dev/topics/html-customization-header

@IanMayo, I created a small screenshot. Please let me know if I understood it right. I would implement it configurable as described in https://github.com/DeepBlueCLtd/Fi3ldMan/issues/89

Yes,    @frank-zimmermann  - that looks fine.

Need to be changed the footer text?

Now I have the possibility to overwrite existing XSLT templates by creating our own XSLT files in the template folder.

For our own styles, I would create a new CSS file. Changes could of course done in oxygen.css, but for modularity, I would create a new one. Agree?

Should the footer text be changed?

That is a future issue to be worked upon

Now I have the possibility to overwrite existing XSLT templates by creating our own XSLT files in the template folde

This is my favoured sequence for modifying output:

existing DITA-OT params
existing Oxgen WebHelp params
modifying templates in Oxygen publishing template
using XSL transforms.

For now, I'm happy for them to go at the top of the oxygen.css file, please.   Let's  introduce separate  CSS file as a future task - once we know it is justified.


Docker build fails

Hi,
I seem to have some issues with the docker build. Using the provided Dockerfile:
Step 10 : RUN cd /infer &&     eval $(opam config env) &&     make -C infer clang java
 ---> Running in 7d087081c6a2
[WARNING] Running as root is not recommended
make: Entering directory '/opt/infer-linux64-v0.6.0/infer'
make: Leaving directory '/opt/infer-linux64-v0.6.0/infer'
make: *** No rule to make target 'clang'.  Stop.
The command '/bin/sh -c cd /infer &&     eval $(opam config env) &&     make -C infer clang java' returned a non-zero code: 2

The beauty of Docker is that I don't think there's a lot of room for much to go wrong - any assistance appreciated.

It may works if you add './configure &&' command after the command 'eval $(opam config env) &&', but it also fails because of ocaml version.
The error message shown because of ocaml version is,
File "clang/cLocation.ml", line 76, characters 11-12:
Error: Syntax error
Command exited with code 2.
Makefile:187: recipe for target 'build_clang' failed
make[1]: *** [build_clang] Error 10
make[1]: Leaving directory '/opt/infer-linux64-v0.6.0/infer/src'
Makefile:21: recipe for target 'clang' failed
make: Leaving directory '/opt/infer-linux64-v0.6.0/infer'
make: *** [clang] Error 2
The command '/bin/sh -c cd /infer &&     eval $(opam config env) &&     ./configure &&     make -C infer clang java' returned a non-zero code: 2

And the build fail message comes from https://github.com/facebook/infer/blob/6389cb9bd0c47b21c72a2aa3532d0301ddb218fc/infer/src/clang/cLocation.ml#L76.
The syntax does not work with ocaml 4.01, and the ocaml installation command ( https://github.com/facebook/infer/blob/master/docker/Dockerfile#L20 ) installs ocaml 4.01.
Thus we need to change ocaml version using opam, ( opam switch 4.02.3 ).
I am currently trying to fix it and after I am done, I will send a pull request for it.

Thanks! I can confirm, I've managed to get that exact error out of this issue, so it does appear you've got it covered.

@technion I sent a pull request to solve it, so if you need it right now, you can try it. It works for me. https://github.com/facebook/infer/pull/271

excute 'make -C infer java' prompt 'make: *** No rule to make target `java'.  Stop.',I don't konw solution,can you tell me reason?

@jimmyxu0311 Try that again after run ./configure command.

make -C infer java
make: *** No rule to make target `java'.  Stop.
./configure
zsh: no such file or directory: ./configure
help


Running spelled wrong

When a project.json file is open, the Running text in the far lower right corner, next to the smiley face, is spelled incorrectly with 3 n's... "Runnning"
:)

Closing already fixed by https://github.com/Microsoft/vscode/pull/45


Kitchen Sink: Todolist bugs fixed

Please describe the changes this PR makes and why it should be merged:
Two relevant bugs were solved, the first one is the clear input functionality after add a task, and a bug related with list item deletion.
Status

[X] Code changes have been tested against prettier, or there are no code changes
[ ] I know how to update typings and have done so, or typings don't need updating

Semantic versioning classification:

[ ] This PR changes the codebase

[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)
[ ] This PR changes the internal workings with no modifications to the external API (bug fixes, performance improvements)

[x] This PR only includes non-code changes, like changes to documentation, README, etc.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 25.706%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1257

Relevant Lines:
5082

💛  - Coveralls


createServerFn not propagating error

Which project does this relate to?
Router
Describe the bug
export const some_fn = createServerFn("POST", async (_) => {
  throw new Error("always throw");
});

const mutation = useMutation({
        mutationFn: some_fn,
        onError: (error) => {
            console.log(error)
        }
})

<button onClick={() => mutation.mutate()}>click me</button>

The POST request returns a 200 and no error is returned in the network request or logged by the client in any other way.
Your Example Website or App
N/A
Steps to Reproduce the Bug or Issue
See code above
Expected behavior
The onError function is called accordingly.
Screenshots or Videos
No response
Platform

OS: macOS
Browser: Chrome
Version:

    "@tanstack/eslint-plugin-query": "^5.59.7",
    "@tanstack/react-form": "^0.34.0",
    "@tanstack/react-query": "^5.59.15",
    "@tanstack/react-query-devtools": "^5.59.15",
    "@tanstack/react-router": "^1.33.2",
    "@tanstack/react-router-with-query": "^1.70.1",
    "@tanstack/router-devtools": "^1.33.2",
    "@tanstack/router-vite-plugin": "^1.32.17",
    "@tanstack/router-zod-adapter": "^1.70.1",
    "@tanstack/start": "^1.70.0",
    "@tanstack/zod-form-adapter": "^0.34.0",

Additional context
No response

the error not being returned is already tracked here: https://github.com/TanStack/router/issues/2535

the error not being returned is already tracked here: #2535

I believe that is a different issue, though. #2535 concerns error serialization, whereas this one concerns the error never being caught client-side since the status code is 200 (used to be 500 by default before this change). As it stands right now, there is no way to determine if the function failed because there is no way to read Server Function response headers.
One way to catch the error right now is to wrap the server function body into an additional try-catch, which changes the response status code in catch:
import { setResponseStatus } from 'vinxi/http';

function doStuff() {
  throw new Error('It failed');
}

const doStuffFn = createServerFn('POST', async () => {
  try {
    return doStuff();
  } catch (e) {
    setResponseStatus(500);
    throw e;
  }
});

If we put the serialization issue aside, functions that throw an error should not return status code 200 when the user code does not handle the error explicitly.


Added Functor f => MonadRec (Free f) instance.

:+1:

:+1:
If we'd had this, I wouldn't have needed that bindF stuff I think, hah.

#37 also has this (written in a nicer way), should we close this PR?

I thought I'd already merged this actually. I'll do so now anyway as we can release this as a minor bump whereas the new approach will be a breaking change. Thanks!


docs: Document debugging in GoLand and VS Code

How to categorize this PR?
/area dev-productivity
/area documentation
/kind enhancement
What this PR does / why we need it:
The Getting Started Locally documentation has a section that describes how to debug Gardener locally.
Regarding IDE setup it links out to Skaffold's debugging documentation.
Skaffold recommends that the easiest way to debug is to install the Cloud Code IDE extension/plugin, which I see as a push to use GCP and increase usage of Gemini. However, it also did not work for me when trying to debug in GoLand.
Using the inbuilt Go remote debugging capabilities of GoLand or VS Code is far easier. This PR provides minimal guides for setting up debugging in GoLand and VS Code. This could save new Gardener developers some time in the future.
👉 Preview of the debugging section.
Which issue(s) this PR fixes:
n.a.
Special notes for your reviewer:
/cc @LucaBernstein
Release note:
NONE

/retest


Refactor decisionHandler and breakup methods

Right now the logic in decisionHandler is nomothetic. We need to break the methods up. Also decision task timeout logic should live here.

FYI, decision task timeout logic does not need to be merged here, timer queue active processor makes more sense, I will remove the TODO from decision handler


Some blocks are not sawable?

I only noted wool and metal blocks so far, but they at least are not compatible with the table saw. Would be nice to get that fixed, along with others I may not yet have found out! Thanks.

Wool being sawable would greatly increase the number of nodes to create (since there are 16 colors), which would increase the game startup time.

With metals he probably meant moreore metals. While rather belonging added there, not here, they keep coming up for me quite often too. Especially on technic games this becomes quite obvious, when basicly all metals end up being sawable, but Tin and Silver.

@Calinou can we close this? I think we don't want wool to be sawable for now, and the missing ores is something likely better coded in moreores.
Keeping this issue open here suggests that this is still being considered, which I don't think it is?

just a quick note: adding the gloopblocks mod makes wool sawable.


Make History and Memory list items context menu accessible using keyboard

Fixes #311
Description of the changes:

Move History and Memory context menus to ListViewItemContainer so users can open them with keyboards
Modify how Memory manages context menus to use a similar approach than History (a lot simplier)

How changes were validated:

Manually test using keyboard, mouse and touch.

@rudyhuyn this looks good, could you resolve the merge conflicts and then I can merge it in?


Fix profiles analytics category (now "profiles")

Changed "profile" to "profiles" to match the spec: https://openedx.atlassian.net/wiki/display/MA/Profile+Analytics
https://openedx.atlassian.net/browse/MA-1675

@aleffert ?

:+1: cc @mikekatz


added custom art variable

Really simple, added a check for a global variable called g:startify_custom_art that the user can define in their vimrc to change the art work.

Please have a look at :h g:startify_custom_header instead.


Better support for choice-of-type search parameters

Q: What is "choice-of-type"?
A: Choice of type is a somewhat common pattern where FHIR properties are named value[x], where the [x] portion can be one of multiple predefined types.  For example, Observation.value[x] can be valueDecimal, valueQuantity, and many others.
Q: How does "choice-of-type" affect search parameters?
A: Search parameters are defined by SearchParameter resources, which by themselves are somewhat lacking.  For example, the SearchParameter cannot differentiate between "date" and "date/time", SearchParameter does not indicate whether there will be one or many eligible values, etc.  So we use SearchParameter.expresssion, a FHIRPath expresssion, to cross walk back to the schema StructureDefinition and ElementDefinition to find the actual values being searched.
Before, that cross-walking was naive, and did not support the value[x] style choice-of-type.  For those properties, we used a simple text representation of the value.
Now, the cross-walking is more accurate, and should allow for better sorting and filtering on choice-of-type properties.
Most notably, this enables sorting and filtering on Observation.value using the value-quantity search parameter.

Coverage decreased (-0.2%) to 94.99% when pulling 269ed34d029c9e4cabc84b18b2fbf0b3213fc421 on cody-search-param-choice-of-type into c780f079704c8e6b5ddb873350d0aef3c67d393e on main.

Obsoleted by https://github.com/medplum/medplum/pull/674


Upgrade rest-client to v2.0.0.rc1 or newer instead of a git sha

https://github.com/ManageIQ/manageiq/blob/eb2314f4df204465f60f1c5ace2becc1a39cfcc5/gems/pending/Gemfile#L37
https://github.com/rest-client/rest-client/compare/08480eb86aef1e...v2.0.0.rc1
The git sha provided IPv6 support in rest-client, we should get to a released tag.
It looks like we use rest-client to talk to foreman, kubernetes and rhev via the ovirt gem.
@brandondunne @abonas @simon3z  thoughts?  I'm not sure how to test foreman or kubernetes with 2.0.0.rc1.

@blomquisg do you have any idea if we use rest-client elsewhere?

@jrafanie You could add a foreman provider and try to connect and refresh.

Note, rest-client 1.8+ and 2.0.0.rc1 pull in a new dependency,  http-cookie with several indirect dependencies.  Our existing git sha is on the 2.0 branch but before that dependency.  If we upgrade, we'll have to deal with the following added dependencies (as of today):
    http-cookie (1.0.2)
      domain_name (~> 0.5)

    domain_name (0.5.24)
      unf (>= 0.0.5, < 1.0.0)

    unf (0.1.4)
      unf_ext

unf_ext is a C extension needing C++ compiler and libstdc++.
https://github.com/knu/ruby-unf_ext

As mentioned on gitter, unf has a separate -java gem that JRuby will use instead, so it won't be an issue there.


Unable to extract video info

$ youtube-dl http://www.dailymotion.com/video/x34aweq_150902-tif_lifestyle

[dailymotion] x34aweq: Downloading webpage
[dailymotion] x34aweq: Extracting information
[dailymotion] x34aweq: Downloading embed page
ERROR: Unable to extract video info; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

OK, I try to $ youtube-dl -U and it can work. :+1:


gitea remote url disappears - fatal: no path specified

Component
agent
Describe the bug
i am using woodpecker:next docker image and configured for my gitea instance:
# docker-compose.yml
version: '3'
networks:
  put_net:
    driver: bridge
    ipam:
      driver: default
      config:
        -
          subnet: <IP_ADDRESS>/24
          gateway: <IP_ADDRESS>

services:
  woodpecker-server:
    image: woodpeckerci/woodpecker-server:next
    networks:
      put_net:
        ipv4_address: <IP_ADDRESS>
    extra_hosts:
      - "n3s1.sbc-io.xyz:<IP_ADDRESS>"
    ports:
      - 8999:8000
    expose:
      - "9000"
    volumes:
      - woodpecker-server-data:/mnt/m1/wpdata
    environment:
      - WOODPECKER_OPEN=true
      - WOODPECKER_HOST=https://woodpecker.example:3944
      - WOODPECKER_GITEA=true
      - WOODPECKER_GITEA_URL=https://gitea.example:3943/
      - WOODPECKER_GITEA_CLIENT=0c7d543a-b70a-461c-bb06-ef32c912789c
      - WOODPECKER_GITEA_SECRET=gto_secret
      - WOODPECKER_AGENT_SECRET=secretexample
      - WOODPECKER_DATABASE_DRIVER=postgres
      - WOODPECKER_DATABASE_DATASOURCE=postgres://woodpecker_u:passwd@<IP_ADDRESS>:5432/woodpecker_db?sslmode=disable
      - WOODPECKER_AUTHENTICATE_PUBLIC_REPOS=true
      - WOODPECKER_DEFAULT_CLONE_IMAGE=woodpeckerci/plugin-git:next
      - WOODPECKER_ADMIN=WoodpeckerBot

  woodpecker-agent:
    image: woodpeckerci/woodpecker-agent:next
    networks:
      put_net:
        ipv4_address: <IP_ADDRESS>
    extra_hosts:
      - "gitea.example:<IP_ADDRESS>"
    command: agent
    restart: always
    depends_on:
      - woodpecker-server
    links:
      - woodpecker-server
    tmpfs:
      - /tmp
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
    environment:
      - WOODPECKER_SERVER=woodpecker-server:9000
      - WOODPECKER_AGENT_SECRET=secretexample

volumes:
  woodpecker-server-data:

now there is a private repo in the list:

but Pipelines are failing:
+ git init -b main
Initialized empty Git repository in /woodpecker/src/gitea.example/main-mirrors-private/proj/.git/
+ git remote add origin
+ git fetch --no-tags origin +refs/heads/main:
fatal: no path specified; see 'git help pull' for valid url syntax
exit status 128

i found that remote url is gone:
+ git remote add origin *[remote url]* on line 3 above
after execute Repair repository on settings, I can restart this pipeline without issues:
+ git init -b main
Initialized empty Git repository in /woodpecker/src/gitea.example/main-mirrors-private/proj/.git/
+ git remote add origin https://gitea.example:3943/main-mirrors-private/proj.git
+ git fetch --no-tags origin +refs/heads/main:
From https://gitea.example:3943/main-mirrors-private/proj
 * branch            main       -> FETCH_HEAD
 * [new branch]      main       -> origin/main
+ git reset --hard -q 7905a6e4214a2969b5106a19806e6c56442675e9
+ git submodule update --init --recursive

but after I push a new commit to trigger CI, the new pipeline failed:
+ git init -b main
Initialized empty Git repository in /woodpecker/src/gitea.example/main-mirrors-private/proj/.git/
+ git remote add origin
+ git fetch --no-tags origin +refs/heads/main:
fatal: no path specified; see 'git help pull' for valid url syntax
exit status 128

logs:
wp-conf-woodpecker-agent-1   | {"level":"error","error":"io: read/write on closed pipe","time":"2022-09-07T02:47:06Z","message":"copy limited logStream part"}
wp-conf-woodpecker-agent-1   | {"level":"error","error":"rpc error: code = Unknown desc = Proc finished with exitcode 128, ","time":"2022-09-07T02:47:07Z","message":"grpc error: wait(): code: Unknown: rpc error: code = Unknown desc = Proc finished with exitcode 128, "}
wp-conf-woodpecker-agent-1   | {"level":"warn","repo":"main-mirrors-private/repo","build":"21","id":"61","error":"rpc error: code = Unknown desc = Proc finished with exitcode 128, ","time":"2022-09-07T02:47:07Z","message":"cancel signal received"}

System Info
https://woodpecker.example:3944/version
{
  "source":"https://github.com/woodpecker-ci/woodpecker",
  "version":"next-05d660f2"
}

Docker Compose version v2.6.0

Additional context
logs:
wp-conf-woodpecker-agent-1   | {"level":"error","error":"io: read/write on closed pipe","time":"2022-09-07T02:47:06Z","message":"copy limited logStream part"}
wp-conf-woodpecker-agent-1   | {"level":"error","error":"rpc error: code = Unknown desc = Proc finished with exitcode 128, ","time":"2022-09-07T02:47:07Z","message":"grpc error: wait(): code: Unknown: rpc error: code = Unknown desc = Proc finished with exitcode 128, "}
wp-conf-woodpecker-agent-1   | {"level":"warn","repo":"main-mirrors-private/repo","build":"21","id":"61","error":"rpc error: code = Unknown desc = Proc finished with exitcode 128, ","time":"2022-09-07T02:47:07Z","message":"cancel signal received"}

Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A join our Discord Chat Server or the Matrix room.

after reloading, I restarted a failed pipeline and passed:
+ git init -b main
Initialized empty Git repository in /woodpecker/src/gitea.example/main-mirrors-private/proj/.git/
+ git remote add origin https://gitea.example:3943/main-mirrors-private/proj.git
+ git fetch --no-tags origin +refs/heads/main:
From https://gitea.example:3943/main-mirrors-private/proj
 * branch            main       -> FETCH_HEAD
 * [new branch]      main       -> origin/main
+ git reset --hard -q 826e4925c8122356dcf0a3481d98034a83d80db8
+ git submodule update --init --recursive

but CI seems not running after pushing a new commit.

then I used the Repair repository then CI trigger is worked now.
but... failed again:
+ git init -b main
Initialized empty Git repository in /woodpecker/src/gitea.example/main-mirrors-private/proj/.git/
+ git remote add origin
+ git fetch --no-tags origin +refs/heads/main:
fatal: no path specified; see 'git help pull' for valid url syntax
exit status 128

I will try finding something in DB, thanks!

I found these rows on DB:
select * from public.builds where build_id=28;

build_id
build_repo_id
build_number
build_author
build_config_id
build_parent
build_event
build_status
build_error
build_enqueued
build_created
updated
build_started
build_finished
build_deploy
build_commit
build_branch
build_ref
build_refspec
build_remote
build_title
build_message
build_timestamp
build_sender
build_avatar
build_email
build_link
build_signed
build_verified
build_reviewer
build_reviewed
changed_files

28
26
1
sbchild
0
0
push
failure

1662533812
1662533812
1662533814
1662533812
1662533814

0427dfdeddcae3f7ab56f3f0a96a3640b989807e
main
refs/heads/main

更新 '.woodpecker.yml'
1662533812
sbchild
https://gitea.example:3943/avatars/cb6cf0988fec32b918503a55e817c687
<EMAIL_ADDRESS>https://gitea.example:3943/sbchild/ci_test/commit/0427dfdeddcae3f7ab56f3f0a96a3640b989807e
f
t

0
[".woodpecker.yml"]

(1 row)
select * from public.repos where repo_id=26;

repo_id
repo_user_id
repo_owner
repo_name
repo_full_name
repo_avatar
repo_link
repo_clone
repo_branch
repo_scm
repo_timeout
repo_visibility
repo_private
repo_trusted
repo_gated
repo_active
repo_allow_pr
repo_config_path
repo_hash
remote_id
cancel_previous_pipeline_events

26
2
sbchild
ci_test
sbchild/ci_test

https://gitea.example:3943/sbchild/ci_test

60
public
f
f
f
t
f

F43TAE2NQXDLSJ2J3GVVVQA7GHM4JKOJ23D7HKAM5NJHRLP3GBUA====
39
["","","push","pull_request"]

(1 row)
that is a fresh repo with a simple .woodpecker.yml file, but it fails:
+ git init -b main
Initialized empty Git repository in /woodpecker/src/gitea.example/sbchild/ci_test/.git/
+ git remote add origin
+ git fetch --no-tags origin +refs/heads/main:
fatal: no path specified; see 'git help pull' for valid url syntax
exit status 128

should I provide more information? Please let me know, thanks!

@qwerty287
I think I am found out how to resolve this issue
can you review #1170 ?

hi were you able to find any solution I am still stuck and this and it is not getting solved in any way


Ntd1hc/bug/reference version number not set properly

Hi,
This PR introduces the package_context.json which allow this package to work in many contexts.
The function names and documents have also updated accordingly.

To test the stand-alone context, you can checkout this branch and install this package.
To test in RobotFramework AIO context, you can get the build AIO packages from my testing (personal) build pipeline https://github.com/ngoan1608/RobotFramework_AIO/actions/runs/5116933073

Please review and give your feeling about this change.
The PR to update build pipelines will be created soon.
Thank you,
Ngoan

Hi Ngoan,
I also would like to test in AIO context. Where can I find package_context.json?

Hi Ngoan,
great!
Also with package_context.json it works: version control fine, error message file!

Hi Thomas,
Merge conflict has been solved.
Thank you,
Ngoan


DRILL-4759: add a test coverage

@krystaln Can you please review this PR? Thanks.
http://<IP_ADDRESS>:8080/job/Framework-Regression/888/console

LGTM +1

@cchang738  Did we set store.parquet.enable_dictionary_encoding to TRUE in the test, so that way dictionary encoding is enabled ?

@kfaraaz The test does not create data. Test data already in proper form. But thanks for the comment.

@cchang738 thanks for confirming, I asked this because in DRILL-4759 the issue was hit over dictionary encoded parquet data.


what

Describe the bug

To Reproduce

Expected behavior

Screenshots

Applicable Versions:

Netlify CMS version: [e.g. 2.0.4]
Git provider: [e.g. GitHub, BitBucket]
OS: [e.g. Windows 7]
Browser version [e.g. chrome 22, safari 11]

Node.JS version:

CMS configuration

Additional context

Closing, please re-open with the issue template


Fix the build system to allow for C++14 or 17 support

Kinda the requirement for #1 as it isn't fun to do with C++11.

https://github.com/pypa/manylinux
Manylinux host systems:

manylinux1    is based on CentOS 5, which is EOL 2017-05-31
manylinux2010 is based on CentOS 6, which is EOL 2020-10-30
manylinux2014 is based on CentOS 7, which is EOL 2024-06-30

Manylinux pip requirements:

manylinux1    requires pip >= 8.1.0
manylinux2010 requires pip >= 19.0
manylinux2014 requires pip >= 19.3
manylinux_x_y requires pip >= 20.3

Pip Python requirements:

Python 3.6 come with pip 18 and is EOL 2021-12-23

https://endoflife.date/python

3.5 was EOL 2020-09-13
3.6 is EOL 2021-12-23
3.7 is EOL 2023-06-27
3.8 is EOL 2024-10-14
3.9 is EOL 2025-10-25

https://en.wikipedia.org/wiki/CentOS#End-of-support_schedule
Use 'yum list installed | grep gcc' in the manylinux containers:

CentOS 5 comes with GCC 4.8.2 and was EOL 2017-03-31
CentOS 6 comes with GCC 8.3.1 and was EOL 2020-11-30
CentOS 7 comes with GCC 9.3.1 and is  EOL 2024-06-30

C++17 for indexed_bzip2 requires GCC >= 8.3.0
Therefore! Because of Python 3.6 and the insufficient pip version coming with it, manylinux1 theoretically has to be supported until 2021-12-23 even though the manylinux1 container is already EOL for 4 years! But, when it is not available, it will build from source and the oldest CentOS, 7, comes with gcc 9.3.1., which already fully supports C++17, so it should not be an issue. Unlike  indexed_gzip or indexed_zstd, I'm also not using any external libraries eliminating another problem source.


Wagtail Brand Font

The issue of font choice recently came up in https://github.com/wagtail/sphinx_wagtail_theme/issues/120
@allcaps mentioned in a discussion on slack about Wagtail finding a font for use in docs, and elsewhere, that is distinctly Wagtail. Generally speaking I think providing reusable resources of any kind is a net positive for helping strengthen the Wagtail ecosystem.
Currently there is a mix on fonts in use:

Wagtail Admin: Open Sans
Wagtail website and docs: Apercu (this aligns with Torchbox branding)
sphinx_wagtail_theme package has recently switched to Source Sans due to legal reasons, as a stop-gap measure until a more suitable replacement is found.

The issue referenced above means we cannot legally distribute Apercu with the Wagtail docs pip package, nor can the Apercu font files legally be committed on GitHub.
With the new Wagtail admin design refresh on the horizon, it seems like an opportunity to make font usage (and other branding elements such as colors) consistent between the various parts of the Wagtail ecosystem.
I personally have no opinion in the matter, other than consistency. Here are a couple options:

Wagtail community as a whole centers on an open source font.
Torchbox (or other sponsor) purchases a special font license that permits redistribution within the confines of the Wagtail ecosystem. Perhaps Colophon (maker of Apercu) or a smaller boutique font foundry would be willing to make special dispensation (for a fee)? This is a similar arrangement of how Apple licenses Helvetica to be distributed with macOS.
Volunteer or paid consultant makes a custom font for the Wagtail org. Usually such an endeavor starts as a fork of a robust open source font and tweaks the glyphs (e.g. start with Source Sans and modify the glyphs to look wagtail-ish).

Quick update from the page editor design refresh mentioned by Vince:

We most likely won’t retain Open Sans
We’re trying out either sans-serif fonts available on Google Fonts, or a system font (San Francisco on macOS & iOS, Roboto on Android, Segoe UI on Windows)

Last time we spoke @benenright and I were having trouble figuring out how to get our hands on a Segoe UI font file for macOS, so we can test the designs with it – If anyone else knows more about this please let us know.

+1 vote for using system fonts. I wouldn't worry a ton about Segoe; in my experience the metrics are close enough to San Franscisco that it seems unlikely adjustments would be needed. That said, there are plenty of folks around who could help test them out in Windows and relay screenshots or video.

I will also note that @tomdyson also commented in favor of system fonts, either in Slack or in a Core Team meeting.

For the wagtail admin itself, I am also in favor of system fonts. Reason being that the admin is already quite heavy and we can shave off several megabytes of payload. We have already implemented this in coderedcms, as an effort to speed up the admin (because our streamfields are so big).
@thibaudcolas this might be helpful: https://developer.microsoft.com/en-us/fluentui#/resources. BUT, I would caution that testing segoe on macos will likely be fruitless. Windows has a totally different way of handling font rendering, DPI, screen sizes, etc. The mac has very smooth/thick font rendering, and very small UI elements by default. Windows on the other hand has much weaker font rendering, and the UI elements are much larger by default (due to display scaling - most laptops default to 150% scaling). The end result is that things like input controls could end up looking very unnatural in Windows if they have lots of padding and a light weight font (300). Whereas this might look "normal" on a mac.
For an example implementation of native font stack, look to Bootstrap as they have a well-tested implementation: https://getbootstrap.com/docs/5.1/content/reboot/

Thank you both! Re testing, this would be in Figma at this stage. I believe it implements its own font rendering but I don’t know how it compares with the rendering of Windows and macOS.
@vsalvino do you have more info on how Bootstrap picked their font stack? The implementation I’ve referred to in the past has been GitHub’s (https://markdotto.com/2018/02/07/github-system-fonts/):
-apple-system,BlinkMacSystemFont,"Segoe UI",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji"

Their article is a few years old but on there they document issues with system-ui and Linux system fonts.

I agree with following Bootstrap's approach: https://getbootstrap.com/docs/5.1/content/reboot/#native-font-stack. I was also looking at similar references in the docs of Any Design though I don't remember how I got there (maybe from a search engine?)
I was worried about overriding fonts, for example if people use the Dyslexie typeface, so I checked out their Chrome extension and it does a fine job of overriding font choices except if it's trying to override !important styles, so we should be extra sure to avoid using !important.
My other concern with a font stack like this is font weights. Not only because, as @vsalvino noted, fonts show lighter on Windows, but also because not every font is going to have a lot of weights available. So we should make sure that anytime we use font weight to convey semantic information, that does not get lost using a font like Dyslexie (which, as far as I can tell, has only two weights).

I just noticed that uswds also has a system font option, which looks pretty similar to the others.

👍 here are the options floated so far side-by-side for comparison – Bootstrap, GitHub, Ant Design, USWDS
system-ui, -apple-system, "Segoe UI", Roboto, "Helvetica Neue", Arial, "Noto Sans", "Liberation Sans", sans-serif, "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol", "Noto Color Emoji";
-apple-system, BlinkMacSystemFont, "Segoe UI", Helvetica, Arial, sans-serif, "Apple Color Emoji", "Segoe UI Emoji"
-apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, 'Helvetica Neue', Arial, 'Noto Sans', sans-serif, 'Apple Color Emoji', 'Segoe UI Emoji', 'Segoe UI Symbol', 'Noto Color Emoji';
-apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, Helvetica, Arial, sans-serif, "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol"

@nmorduch do you have more info on the weights of the Dyslexie font? This feels like something we might want to explicitly support, even if that just means being aware of it / documenting how we can test it.

We have people on Slack volunteering to help! Message: https://wagtailcms.slack.com/archives/C014L7KJH3N/p1644402583455259.
I would like us to trial the following options:
// Bootstrap
system-ui, -apple-system, "Segoe UI", Roboto, "Helvetica Neue", Arial, "Noto Sans", "Liberation Sans", sans-serif, "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol", "Noto Color Emoji"
// GitHub
-apple-system, BlinkMacSystemFont, "Segoe UI", Helvetica, Arial, sans-serif, "Apple Color Emoji", "Segoe UI Emoji"
// Ant Design
-apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, "Helvetica Neue", Arial, "Noto Sans", sans-serif, "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol", "Noto Color Emoji"
// USWDS
-apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, Helvetica, Arial, sans-serif, "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol"

// GitHub with system-ui after Segoe UI
-apple-system, BlinkMacSystemFont, "Segoe UI", system-ui, Helvetica, Arial, sans-serif, "Apple Color Emoji", "Segoe UI Emoji"
// Ant Design with system-ui after Segoe UI
-apple-system, BlinkMacSystemFont, "Segoe UI", system-ui, Roboto, "Helvetica Neue", Arial, "Noto Sans", sans-serif, "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol", "Noto Color Emoji"
// USWDS with system-ui after Segoe UI
-apple-system, BlinkMacSystemFont, "Segoe UI", system-ui, Roboto, Helvetica, Arial, sans-serif, "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol"

Windows machines that don’t have Segoe UI – apparently that’s Windows XP, Vista, 7, and potentially some versions of Windows Server

Segoe UI became the Windows system font beginning in Vista (and Server 2008). See its Wikipedia entry (first paragraph) and Microsoft documentation.

Has the new font stack also been tested on every supported language in the Wagtail admin? I'm mainly thinking non-Latin alphabets (Arabic, Asian scripts). In those cases some font substitutions will probably be in use, so want to make sure the stack looks OK.

I believe the only script we’ve tested it on that’s not english is Simplified Chinese, since that’s what the system-ui issues were reported on. It’d be nice to know how Wagtail works with other scripts!


Mixed Chart with both Column and Stacked Bar

Is there a way to have mixed charts with

stacked bar
column
line

charts together.

Is there a way to have column and stacked bar chart in the same graph with another line chart?

The way that another graphing library handles this is rather elegant.
For each series, you can pass in a stack id.
       <Bar dataKey="female" stackId="a" fill="#8884d8" />
       <Bar dataKey="male" stackId="a" fill="#82ca9d" />
       <Bar dataKey="uv" fill="#ffc658"/>

https://jsfiddle.net/alidingling/9hjfkp73/

+1 to this feature!

+1 to this feature!

Likewise, I would really love this feature! Since you can specify the type of chart as part of the series, I would expect it to be possible to include some extra options such as stacked that only applies to that series/type combination. :)

Hello guys, any update on this?

Hello guys, any update on this in 2022 :) ?

Hello guys!
Any updates?
I happy to use your library, but want to combine stacked bar chart with any charts...

Hello guys !
Any updates on this ? Your library is so great and would be so perfect with this feature, it would be very useful.

Hello ! I am also looking for this feature !

+1

Hello! This would be an amazing feature that I would love to implement into my current project!

Me too!!


watch task not working in Vagrant shared folder

In my env watch is not working. I have an ubuntu image running inside virtualbox deployed using vagrant on window 7.
The working folder is the shared /vagrant folder (shared from windows to /vagrant in the ubuntu vm).
Using this configuration brings some weird things like having to run npm install with --no-bin-links
In this configuration to make watch available tools/utils/seed/watch.ts file needs to be changed by adding usePooling : true opts
plugins.watch(join(APP_SRC, '**'), {usePolling: true}, (e:any) =>
It took me some time to figure this out... maybe it needs to be added as new config option?

@bogdanbrudiu it is a good idea to create a FAQ page in the wiki and include this information there.
In general, I don't think it is a good idea to use polling by default.


Cast OpenSSL errors to strings.

Resolves #556.
I haven't written any tests for this because of #791. I'll start trying to work on #791 and, if we can get it ironed out quickly, we should block the merge of this behind #791 so that we can actually have a test for this.

+1
Worked for me. Thanks @Lukasa

:custard:

I have changed this line but still have this error.

@qicongchen What error, specifically?

Same here. I changed the line by adding str() around e. But the Error is still there.
File "/usr/local/lib/python2.7/site-packages/requests/packages/urllib3/connectionpool.py", line 318, in _raise_timeout     if 'timed out' in str(err) or 'did not complete (read)' in str(err):  # Python 2.6 TypeError: __str__ returned non-string (type Error)
I also tried deleting the whole check condition:
        if 'timed out' in str(err) or 'did not complete (read)' in str(err):  # Python 2.6             raise ReadTimeoutError(self, url, "Read timed out. (read timeout=%s)" % timeout_value)
Then I got the following error:
  File "/usr/local/lib/python2.7/site-packages/requests/adapters.py", line 447, in send     raise SSLError(e, request=request) requests.exceptions.SSLError
Help please!!!

@Phoenix-Chen can you provide more details?
It's clear you're using requests.

What version of requests are you using?
What file did you edit to add the str()?
How did you install requests?
What version of Python are you using? (python -V)

I got this error on 2017-01-21.  The logs I'm looking at are spotty, but here is what I have:
2017-01-21 12:29:26.216
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/requests/api.py", line 71, in get
    return request('get', url, params=params, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/requests/api.py", line 57, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 475, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 585, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/requests/adapters.py", line 403, in send
    timeout=timeout
  File "/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/connectionpool.py", line 578, in urlopen
    chunked=chunked)
  File "/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/connectionpool.py", line 351, in _make_request
    self._validate_conn(conn)
  File "/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/connectionpool.py", line 814, in _validate_conn
    conn.connect()
  File "/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/connection.py", line 289, in connect
    ssl_version=resolved_ssl_version)
  File "/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/contrib/pyopenssl.py", line 350, in ssl_wrap_socket
    rd, _, _ = select.select([sock], [], [], sock.gettimeout())
error: (4, 'Interrupted system call')

2017-01-21 12:34:15.539
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/requests/api.py", line 71, in get
  File "/usr/local/lib/python2.7/dist-packages/requests/api.py", line 57, in request
  File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 475, in request
  File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 585, in send
  File "/usr/local/lib/python2.7/dist-packages/requests/adapters.py", line 403, in send
  File "/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/connectionpool.py", line 578, in urlopen
  File "/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/connectionpool.py", line 354, in _make_request
  File "/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/connectionpool.py", line 324, in _raise_timeout
TypeError: __str__ returned non-string (type Error)

I am assuming the two errors are related based on the timing.
The line numbers don't line up with the requests/urllib3 versions currently installed, so I don't know the exact release versions that were running at the time.  Looks like somewhere around here: https://github.com/shazow/urllib3/blob/7fa026c37a5326dcaf97cb9c66ecd15df9c1a194/urllib3/connectionpool.py
Ubuntu 16.04, Python 2.7.12

Hrm. =(
So the first one is an issue that got fixed recently, which is that we didn't handle EINTR while doing select operations on a socket.
The second one is still the one we need more details on. If you're seeing regular reproduction of this issue, it'd be nice to add a try...except around the cast to string and to print out the exception itself, ideally with a traceback.

Huzzah for pretending to always implement PEP 475. :)

@Lukasa I'd prefer not to tinker the code directly in dist-packages.  Will a patch at import-time (or instantiation) get the job done?  Something like:
def patch_raise_timeout(pool):
	_orig_raise_timeout = pool._raise_timeout

	def _raise_timeout(err, url, timeout_value):
		try:
			strerror = str(err)
		except TypeError:
			raise err
		else:
			_orig_raise_timeout(err, url, timeout_value)

	pool._raise_timeout = _raise_timeout

This error has come up before, but not regularly.  I'll post next time it shows up in the logs.

That might work.


TypeError: '<' not supported between instances of 'int' and 'UFOFormatVersion'

I’m getting this error from checkoutlinesufo after updating FonTools:
$ checkoutlinesufo -e -q tests/checkoutlinesufo_data/input/ufo2.ufo
Traceback (most recent call last):
  File "python/afdko/checkoutlinesufo.py", line 1109, in <module>
    main()
  File "python/afdko/checkoutlinesufo.py", line 1103, in main
    run(sys.argv[1:])
  File "python/afdko/checkoutlinesufo.py", line 1092, in run
    font_file.save()
  File "python/afdko/checkoutlinesufo.py", line 136, in save
    layerName=PROCD_GLYPHS_LAYER_NAME, defaultLayer=False)
  File "/fonttools/Lib/fontTools/ufoLib/__init__.py", line 1440, in getGlyphSet
    self._formatVersion < UFOFormatVersion.FORMAT_3_0
TypeError: '<' not supported between instances of 'int' and 'UFOFormatVersion'

Hmm, checkoutlinesufo is directly setting _formatVersion, so I guess it is doing something unsupported.

That's likely due to this: https://github.com/fonttools/fonttools/pull/1786. I see what checkout lines has to do here (pretend it's a UFO3 when it's a UFO2). FontTools added a class here for the types, and checkoutlines isn't using that for the version. If you change checkoutlines to pull that in from fonttools, it should be fine.
@anthrotype clearly this is an unintended side effect, likely we should add __eq__ to the UFOFormatVersion class

That checkoutlinesufo code is setting a private method (prefixed with underscore), and in the inline comments it is describing itself as a "hack"
https://github.com/adobe-type-tools/afdko/blob/071efbb4d4626962424db7c53b5ed7bebbe12ee0/python/afdko/checkoutlinesufo.py#L133
I don't think this is a fontTools bug. When accessing a private attribute they are doing at one's risk. It should be easier to just update the code in checkoutlinesufo.

Yes, I don't think it is a FontTools bug. What it is trying to do seems also hackish, opening UFO 2 files then pretending they are version 3 then saving them as version 2!

It's a hack, yes, but I do think that there will be code that has UFO versions as int and is trying to compare version with ufoLib, that's there this is breaking down. An __eq__ for the version class would smooth that over.

(changed mind)
Agreed, @khaledhosny I think that if the hack uses the versions specified by the new class, all will be well.

Not really a FontTools bug.

I know full well why that hack is there, it's just for RF UFO2's layers that was a hack, you can't get around that, so...hack on.

AFDKO will "hack on", but not forever: https://github.com/adobe-type-tools/afdko/issues/1135


Extended querybuilder, type 'Model[]' is missing properties

Hi, i'm trying extend by QueryBuilder, by followed code:
import { Model, QueryBuilder, Page } from 'objection'

class MyQueryBuilder<M extends Model, R = M[]> extends QueryBuilder<M, R> {
  public ArrayQueryBuilderType!: MyQueryBuilder<M, M[]>
  public SingleQueryBuilderType!: MyQueryBuilder<M, M>
  public NumberQueryBuilderType!: MyQueryBuilder<M, number>
  public PageQueryBuilderType!: MyQueryBuilder<M, Page<M>>

  public async findOrCreate(query: any, defaults: any): Promise<this> {
    let result
    const findQuery = await this.findOne({ ...query })
    if (!findQuery) {
      result = await this.insertAndFetch({
        ...defaults,
      }).first()
    } else {
      result = findQuery
    }

    return result
  }
}

// tslint:disable-next-line: max-classes-per-file
export default class Base extends Model {
  public QueryBuilderType!: MyQueryBuilder<this>
  public static QueryBuilder = MyQueryBuilder
}

But, when i'm using that as
const testQuery = await SomeModel.query().findOrCreate({ ... }, { ... })

Got typescript compilation error
Type 'SomeModel[]' is missing the following properties from type 'SomeModel': id, ..., QueryBuilderType, and 33 more.
Is it my fault or library typings bug?

public async findOrCreate(query: any, defaults: any): Promise<this> {

this type is wrong. It returns a promise of a query builder. You need to either return this or a valid promise type like Promise<R>.


Fix formatting of isAnyBlank() and isAnyEmpty().

Coverage remained the same at 95.094% when pulling bbd9f9a23c95d8c9d7a96bc62e24ebc0480cbd3e on Isira-Seneviratne:Fix_formatting_of_isAnyBlank_and_isAnyEmpty into e9225d05b25122e0914f1c6f200aaf8fe0e6a75c on apache:master.


Browse Everything check box should appear with each file.

Having the select all boxes but not an box each item.

I think we should still do this in the browse everything gem.  We are not yet at gem freeze and there are not many changes to the gem so this adds little risk.

I'm wondering if we could add a quick fix for a truncation issue while in the gem.  When I have a really long folder name in Box it just truncates the name.  When I have a really long folder name in Browse everything items get pushed to the side.   I was hoping when we were in adding the check box we could also truncate the display.

@cam156 I'm not seeing that. Which browser are you using? I have a screen shot, but it's not uploading... probably due to the S3 outage.

@awead do you have a really long folder name in you box folder?  It only happens when the name should be truncated.

@cam156 ok, now I'm seeing it. It only affects folders and not files so we ought to be able to apply the same truncating process to the folder names.

Fixed now with https://github.com/projecthydra/browse-everything/pull/151


System access server (ldap) not reachable with WAN IP from Firewall

I'm running an internal ldap server on a machine in my LAN on port 30389 (unfortunately not possible to change to default port 389 due to our internal setup). StartTLS is enforced with the use of a Let's Encrypt certificate.
I configured port-forwarding from WAN:389 to LDAP:30389 for external access via the public domain name and enabled NAT reflection for internal access with public domain.
Because the internal port is different, I can't simply override the ip address in Unbound DNS for local requests.
For firewall authentication and OpenVPN I also want to use the ldap server. But the firewall is not able to reach the server with this setup. Obviously the NAT reflection is not working for traffic originating from the firewalls loopback interface?
Should this already work with the simple port-forwading and NAT reflection or what configurations are necessary?
I tried with a simple NAT Outbound rule from loopback interface targeting the ldap server, which unfortunately didn't work. Most likely because a crucial step or configuration is still missing?
What additional configurations are necessary for the outlined setup?
My setup:
Firewall running opnsense 21.1.7, WAN with public IP

As suggested in this blog post https://www.routerperformance.net/opnsense-bypass-ldaps-errors-via-haproxy, I've installed an HAProxy on the firewall to circumvent the certificate check. Because the unencrypted traffic doesn't leave the firewall I can live with it.


x/build: LUCI linux-riscv64-mengzhuo bot has a flaky network connection

The linux-riscv64-mengzhuo machine has a flaky network connection. Recent builds have timed out trying to obtain things like the tip-of-tree for a repository and trying to download prebuilt toolchains from the CAS service. The consecutive failures of the latter led the machine to be quarantined, draining the pool of linux-riscv64 machines for some time.
I will note that some of the work cmd/coordinator used to do is now done on the bot itself. This is just a consequence of the switch to LUCI's execution model.
@mengzhuo is there anything that can be done either by you or us to improve the stability of the machine? For instance, if we're OK with making builds slower, we can skip the prebuild on this platform and always run make.bash, even for subrepos. Fetching the tip-of-tree is a bit unavoidable for subrepo builds though, because those builds can only be triggered by one source. Perhaps we can target a different service, or add some other exception.

All my riscv-builders are located in my homelab in Shenzhen, China, which is home ADSL network.
Might be I can transfer these riscv development boards/boxes to a server-farm if possible for more stable/better QoS network. (Looking for funding from RISCV companies here.)
This is a ping latency graph of go.googlesoure.com/*.appspot.com in 24 hours, it looks pretty good to me that all packages go around the whole planet (<180ms)

I take a look at some of failures "go.googlesoure.com" resolved to "<IP_ADDRESS>" which my firewall didn't allow this ip.
I had allow "<IP_ADDRESS>/16" now.
https://chromium-swarm.appspot.com/task?id=679f0bed18bf1910

Thanks for looking into that! The other category of issues I've seen is that the cas download step fails (because the attempt to download, and the retries) fail too. I can provide some instructions on how to try to reproduce that if it would be helpful.
Also, it looks like the bot went down recently? https://chromium-swarm.appspot.com/bot?id=linux-riscv64-mengzhuo has status dead and the last task ended with BOT_DIED.


The url of pretrained models is unavailiable

Hi, as I am trying to load from pretrained modules, I find the url where the modules should be downloaded from is unavailiable.
If running generate.py, an error of "socket.gaierror: [Errno -5] No address associated with hostname" will occur.
And if access the url directly, the browser respond with "Can't find the server's IP address".
Grateful if this issue can be solved

Hi, Thank you for your feedback. The URL links are updated, please let us know if you are facing other issues with the pretrained models.
Best,


Retrieve the generated elasticsearch query to include in the response

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[X ] elastic.v6 (for Elasticsearch 6.x)
Please describe the expected behavior
I would like to somehow get the generated query to elasticsearch to include it in my response. Could be a string or a json object.
Please describe the actual behavior
I am using the DSLQuery example: https://github.com/olivere/elastic/wiki/QueryDSL
I pretty much do what the Find method in the example do.
I create a service, build the query and trigger a search: sr, err := search.Do(ctx)
I then, parse the response and construct my custom response to return to my client
How can I intercept the generated elasticsearch query and include it in response as a json.RawMessage or if not possible a string ?
Thanks

To get the generated request body, you can do this:
var query elastic.Query
...
query = ... generate some query ...
...
src, err := query.Source()
if err != nil {
  ...
}
out, err := json.Marshal(src)
if err != nil {
  ...
}
fmt.Printf("Generated query:\n%s\n", out)

If you want to trace the data sent between client and server, you can enable trace logging.
HTH

Hi @olivere
Still trying to do the same thing few months later, sorry.
I did not find a solution to go from my SearchService to the Source of the query
I have this
	search := client.Search().Index(indexName).Type(typeName).Pretty(f.pretty)
	search = f.query(search)
	search = f.aggs(search)
	search = f.postfilter(search)
	search = f.sorting(search)
	search = f.paginate(search)
	search = f.collapse(search)
	sr, err := search.Do(ctx)

How can I extract the Source from the SearchService please?

There currently is no way to get the generated API request sent to Elasticsearch. There are builders for queries etc., and that's what you asked for in the issue. But that's different to getting the whole HTTP request as sent to Elasticsearch.
As a workaround, you can create a SearchSource that can be passed to the SearchService (it is used by SearchService under the covers anyway). That SearchService has a Source method that might help in what you want (excluding query parameters sent in the URL).


Global namespace collision for native node fetch when near-api-js is imported

Prerequisites

[X] I'm using the latest version of near-api-js.
[X] I have tried to start with a fresh project and reproduce the defect with minimal code changes.
[X] I have read the console error messages carefully (if applicable).

Description
Attempting to use the node native fetch API in node >= 18 when the near-api-js is imported directly or by a transitive dependency results in the node global fetch property being overwritten by near-api-js.
These two lines of code here and here explicitly override the global fetch property.
They should not do this.
Reproducible demo
No response
Steps to reproduce

open a project which has near-api-js somewhere in it's dependency tree
open a node console
type fetch
see output is [AsyncFunction: fetch]
type require('near-api-js')
type fetch
see output is [Function: default_1]

From my terminal:
$ node
Welcome to Node.js v18.17.1.
Type ".help" for more information.
> fetch
[AsyncFunction: fetch]
> require('near-api-js')
{
  keyStores: {
    KeyStore: [Getter],
    InMemoryKeyStore: [Getter],
    BrowserLocalStorageKeyStore: [Getter],
    UnencryptedFileSystemKeyStore: [Getter],
    MergeKeyStore: [Getter]
  },
  accountCreator: [Getter],
  providers: [Getter],
  utils: [Getter],
  transactions: [Getter],
  validators: [Getter],
  multisig: [Getter],
  Account: [Getter],
  Connection: [Getter],
  Contract: [Getter],
  InMemorySigner: [Getter],
  Signer: [Getter],
  KeyPair: [Getter],
  Near: [Getter],
  ConnectedWalletAccount: [Getter],
  WalletConnection: [Getter],
  connect: [Getter],
  DEFAULT_FUNCTION_CALL_GAS: [Getter]
}
> fetch
[Function: default_1]

Expected behavior
I would not expect a dependency to overwrite the global fetch property.
Actual behavior
near-api-js overwrites the global fetch property
Your environment

NEAR JavaScript API version used: 2.1.4
Frontend framework (if applicable): N/A
Relevant dependencies (if applicable): near-api-js

Self-service

[x] I'd be willing to fix this bug myself.

I'm having issues getting tests to run. After installing the requisite tools, my pnpm-lock.yaml changes versions from 5.4 to '6.0'. Is there something I'm missing?


Error loading youtube vidoe

Im getting this Error
"Failed to execute 'postMessage' on 'DOMWindow': The target origin provided ('https://www.youtube.com') does not match the recipient window's origin ('https://sarbagyadhaubanjar.github.io').", source: https://s.ytimg.com/yts/jsbin/www-widgetapi-vflPBjLfx/www-widgetapi.js (106)
Does anyone how to solve ?

same here

Could you check if the issue persist on Version 4 ?


Add isCASDown command

Implements/closes #74. I need help writing tests though.

You can add tests by following the examples in other *.test.ts files. You will need to mock axios, and consider both success and failure cases. Keep in mind that our command runner architecture knows what to do with most errors thrown, so don't worry about sending messages to users about network errors.

Hmm, seems that our coverage report tool doesn't like forks. Need to figure a way around that.
I wonder if the changes in #82 make a difference, or worsen the trouble...

I see a light. Will probably make a PR soon-ish to fix the CI such that forks can run tests. Once that's made and merged, this PR should begin working again!

#84 should fix the tests


Launchers Broken

Type: Bug
OS: Ubuntu 22.04
VSCode: code_1.79.2-1686734195_amd64.deb
I have been using VSCode for about 6 months using the startmenu launcher and the context menu by right clicking on a folder
However The terminal
$ code
command didnt work so I added the location of the binary to the $PATH in my .bashrc file
export PATH=$PATH:/usr/share/code
Now the Startmenu launcher produces this error popup when clicked
Launching Visual Studio Code
(Failed)
'Could not find the program cd'
The Context Menu produces a different error popup with similar message
'Could not find the program cd'
Typing
$ code
at the terminal now works but nothing ive tried will make the launchers work
Tried removing the PATH from .bashrc and various tinkering with the launchers. It Broken!
 Changing the PATH probably didn't actually have anything to do with the launchers.

Ive tried uninstalling and reinstalling VSCode  but its had no effect, the extensions are playing up as well. Live Server for example no longer works. Would be good if there was a way to fix this, its really crippled the application and I dont know how to fix it

Hey @roblourens, this issue might need further attention.
@JefMcD, you can help us out by closing this issue if the problem no longer exists, or adding more information.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


EnginePylon guide curves are ignored

When an engine pylon has guide curves, they are not taken into account and the pylon is lofted with straight segment.
In the example attached, if the enginePylon element is moved into  and renamed accordingly, the guide curves are taken into account and the segment is generated correctly.
example_issue#988.zip

sorry, did not see that you closed my other issue, I thought I didn't post it 😅


LONDON | SHIRIN PANAHIAN | feature/wireframe | WEEK1

Learners, PR Template
Self checklist

[x] I have committed my files one by one, on purpose, and for a reason
[x] I have titled my PR with COHORT_NAME | FIRST_NAME LAST_NAME | REPO_NAME | WEEK
[x] I have tested my changes
[x] My changes follow the style guide
[x] My changes meet the requirements of this task

Changelist
Briefly explain your PR.
Questions
Ask any questions you have for your reviewer.

Hi @shirinpanahian , I can see a good work here!
I left some comments, if you have any questions, don't hesitate to ask!
Observation: When submitting a PR, briefly explain your PR, so the reviewer can have a general view of what you've done.


Link to documentation is broken

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports and didn't find any similar reports.

Expected Behavior
Link is working
Current behaviour
Link (https://davish.github.io/obsidian-full-calendar/) provide an Error 404
JavaScript console output
No response
Steps to reproduce
Go to the main page, click on the full documentation link
https://github.com/obsidian-community/obsidian-full-calendar?tab=readme-ov-file
Calendar information
NR
Which Operating Systems are you using?

[X] macOS
[ ] Windows
[ ] Linux
[ ] Android
[ ] iOS

Obsidian Version
NR
Full Calendar Plugin Version
NR
Checks

[ ] I have tried it with all other plugins disabled and the error still occurs.

Possible solution
No response
Comments
No response

same

same here :(

yep :C this one's under obsidian-community -- should that be where it redirects to?

yep :C this one's under obsidian-community -- should that be where it redirects to?
[Edit: That fixes it! Link should be: https://obsidian-community.github.io/obsidian-full-calendar/ . Just need someone with admin access to fix the link in the sidebar/at the bottom of the README as well I think.]

yeah this was really annoying. barely anyone maintains stuff. thanks for the updated link!

Not really a solution but the documentation can be found here as well: http://web.archive.org/web/20231127165354/https://davish.github.io/obsidian-full-calendar/


Integration with PIXIjs

Hi guys,
Did you see possibility on integration with PIXIJS Custom Sprites?
Thanks!

I have not used pixijs, so cannot say.
But I have a proof of concept version of roughjs that renders using webgl.
What exactly are you thinking of?

I'm thinking in integrating it in PIXI Custom components, using graphics(that in underlying implementation uses a canvas context).
Somethink like that, but using rough. Then we will have acess to PIXI Library. It will awesome.
const Rectangle = PixiComponent('Rectangle', {
create: () => new PIXI.Graphics(),
applyProps: (g, _, props) => {
const { fill, x, y, width, height } = props
  g.clear()
  g.beginFill(fill)
  g.drawRect(x, y, width, height)
  g.endFill()
}

})

I'm thinking in integrating it in PIXI Custom components, using graphics(that in underlying implementation uses a canvas context).
Somethink like that, but using rough. Then we will have acess to PIXI Library. It will awesome.
const Rectangle = PixiComponent('Rectangle', {
create: () => new PIXI.Graphics(),
applyProps: (g, _, props) => {
const { fill, x, y, width, height } = props
  g.clear()
  g.beginFill(fill)
  g.drawRect(x, y, width, height)
  g.endFill()
}

})

you can try this

roughGenerator
import rough from 'roughjs/bundled/rough.esm';

import RoughType from 'roughjs';

declare type RoughGenerator = ReturnType<typeof RoughType.generator>;

const generator = rough.generator(undefined);
const roughGeneratorInstance = generator as RoughGenerator;

export const roughGenerator = roughGeneratorInstance;

        const rectangle = roughGenerator.rectangle(0, 0, 100, 100);
        rectangle.sets.forEach(step => {
            if (step.type === 'path') {
                step.ops.forEach(({op, data}) => {

                    const [cp1x, cp1y, cp2x, cp2y, x, y] = data;
                    switch (op) {
                        case "bcurveTo":
                            buttonDraw.bezierCurveTo(cp1x, cp1y, cp2x, cp2y, x, y)
                            break;
                        case "lineTo":
                            buttonDraw.lineTo(cp1x, cp1y)
                            break;
                        case "move":
                            buttonDraw.moveTo(cp1x, cp1y)
                            break;
                        default:
                            break;
                    }
                })
            }
        });

I have not used pixijs, so cannot say. But I have a proof of concept version of roughjs that renders using webgl. What exactly are you thinking of?

Def interested in webgl version


How to set the disabled/ readonly option dynamically to the form properties/attributes [Question]

While rendering the Form, Any input controls created with default behavior should be changed into read-only mode dynamically based on some scenario(s).
scenario 1: If normal user logins, they can only see their email address (email field should not be editable)
scenario 2: If admin logins, they can change the user address (full privileges)
Note: User-based customizations like the normal user login should get the control ready-only/mandatory/hidden and the users like admins should have full privilege on the control(s).
I am using Angular 9
formiojs: "^4.8.0-rc.12"
angular-formio: "^4.6.7"
Any suggestions on how to do it? Any documentation for reference? Will appreciate any help

You can achieve this with Logic.  The trigger will need to reference the logged in user like so.
result = Formio.getUser().roles.indexOf('----- admin id -----') !== -1;

@travist thanks a lot for the response
But my doubt is, I want to make the input field as readOnly/disabled based on my custom condition while rendering the Form dynamically.
For example, If I add readOnly=" true", its making entire from as read-only mode (Ref the attached screenshot below)

Here how I will make the input field "Email" alone as read-only based on my custom condition, while rendering the form?
Any suggestions/documentation/reference will help, Thanks.

You would just attach Logic to that field specifically and not the whole form. The Logic tab can be found by editing that component and then clicking on the Logic tab.

@travist thanks for the response
I will try with the Logic tab when creating a new Form.
Also for dynamic customization, when rendering the form I am adding the conditions like below for some scenarios.
Iterate over the form components and set the flags on any fields.
for (const control of inputFromJSON) {
if (control.key == 'email') {
// read-only
control.disabled = true;
}
if (control.key == 'phone_number') {
// mandatory
control.validate.required = false;
}
if (control.key == 'pin_code') {
// hidden
control.hidden = false;
}
}
this.formjson = { components: inputFromJSON }

like in documentation
we can set value of a component like this
const email = form.getComponent('email');
email.setValue('joe@examle.com');

is there any funtion like "setDisabled" to set the disable property of a component


fix method delete

Coverage remained the same at 22.531% when pulling 81fa471e22bf2b2b6eef339f97a64c2fab8f99be on indriHutabalian:dev into 94b3f33724a5c4384eaf66d6c791781436a2a206 on danliris:dev.

Coverage increased (+0.02%) to 22.548% when pulling a2f5a220453b3d4c4a71e28b44ac45c6466f79e1 on indriHutabalian:dev into 94b3f33724a5c4384eaf66d6c791781436a2a206 on danliris:dev.

Coverage increased (+0.06%) to 22.595% when pulling 6f0a7f0bd0748b0246695e327b124945076ba211 on indriHutabalian:dev into 94b3f33724a5c4384eaf66d6c791781436a2a206 on danliris:dev.

Coverage increased (+0.06%) to 22.595% when pulling 8e5456d62dec09de8233461c35f6e03b6b54b222 on indriHutabalian:dev into 94b3f33724a5c4384eaf66d6c791781436a2a206 on danliris:dev.

Coverage decreased (-0.05%) to 22.484% when pulling 7f9f9b3ffa858ad4a9787f119dbf7653e001b043 on indriHutabalian:dev into 94b3f33724a5c4384eaf66d6c791781436a2a206 on danliris:dev.

Coverage increased (+0.1%) to 22.653% when pulling 7c0fcb30c6b3812cd2da6b6e951ac8b5ae0b540a on indriHutabalian:dev into 94b3f33724a5c4384eaf66d6c791781436a2a206 on danliris:dev.


Uninstall fails:"Object reference not set to an instance of an object"

Visual Studio community 2015, Xamarin VS <IP_ADDRESS>
How to reproduce:

create Xamarin.Android PCL or Forms project
Install NuGet package of sqlite-net-pcl'

Package is installed properly and can be used

Uninstall NuGet package of sqlite-net-pcl

Expected result: Package is uninstalled
Actual result: NuGet console gives following info
PM> Uninstall-Package sqlite-net-pcl
Attempting to gather dependencies information for package 'sqlite-net-pcl.1.3.1' with respect to project 'ClassLibrary2', targeting '.NETPortable,Version=v4.5,Profile=Profile111'
Resolving actions to uninstall package 'sqlite-net-pcl.1.3.1'
Uninstall-Package : Object reference not set to an instance of an object.
At line:1 char:1
+ Uninstall-Package sqlite-net-pcl
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Uninstall-Package], Exception
    + FullyQualifiedErrorId : NuGetCmdletUnhandledException,NuGet.PackageManagement.PowerShellCmdlets.UninstallPackageCommand

PM>

Actually, after installing this package, I'm not able to install any other package either. Result:
Severity	Code	Description	Project	File	Line Error		An error occurred while retrieving package metadata for 'SQLitePCLRaw.bundle_green.1.1.2' from source 'path\visual studio 2015\Projects\ClassLibrary2\packages'.			0

It seems that this problem disappears with Visual Studio 2015 update 3


Drop support for older versions of python

We should drop the support for older python versions and add the new python versions to the test matrix.
The idea is to follow the following as a guide: Recommend Python and NumPy version support.
Therefore, we should drop support for python 3.7 and 3.8, and add python 3.11 to our test matrix.

Hi there!
I have marked this issue as stale because it has not had activity for 45 days.
Consider the following options:

If the issue refers to a large task, break it in smaller issues that can be solved in
less than 45 days;
Label the issue as wontfix or wontfix for now and close it.

Hi there!
I have marked this issue as stale because it has not had activity for 45 days.
Consider the following options:

If the issue refers to a large task, break it in smaller issues that can be solved in
less than 45 days;
Label the issue as wontfix or wontfix for now and close it.


attempt to add scrolling to help pane

no idea if this is in the right place, but this sure worked under chrome inspector:

But the spell palette area is a flexible size, so hard-coding the dimensions here would only work for one particular screen layout...


zeromq 4.2.1 is missing libzmq.so.5 shared library symlink

The latest version of ZeroMQ pushed today (4.2.1) is missing the libzmq.so.5 shared library symlink present in the previous version:
$ ls -lh ~/install/bcbio-vm/anaconda/pkgs/zeromq-4.1.5-0/lib/
total 21M
-rw-r--r-- 2 chapmanb chapmanb  15M Jun 25  2016 libzmq.a
-rwxr-xr-x 1 chapmanb chapmanb  946 Jun 25  2016 libzmq.la
lrwxrwxrwx 1 chapmanb chapmanb   15 Jan 23 11:49 libzmq.so -> libzmq.so.5.0.1
lrwxrwxrwx 1 chapmanb chapmanb   15 Jan 23 11:49 libzmq.so.5 -> libzmq.so.5.0.1
-rwxr-xr-x 2 chapmanb chapmanb 5.4M Jun 25  2016 libzmq.so.5.0.1
drwxrwxr-x 2 chapmanb chapmanb 4.0K Jan 23 11:49 pkgconfig
$ ls -lh anaconda/pkgs/zeromq-4.2.1-0/lib/
total 2.7M
-rw-rw-r-- 1 chapmanb chapmanb 1.8M Jan 23 07:37 libzmq.a
lrwxrwxrwx 1 chapmanb chapmanb   15 Jan 23 11:42 libzmq.so -> libzmq.so.4.2.1
-rwxrwxr-x 1 chapmanb chapmanb 878K Jan 23 07:39 libzmq.so.4.2.1
lrwxrwxrwx 1 chapmanb chapmanb   15 Jan 23 11:42 libzmq.so.5.1.1 -> libzmq.so.4.2.1
drwxrwxr-x 2 chapmanb chapmanb 4.0K Jan 23 11:42 pkgconfig

This causes pyzmq compiled against the previous version to fail with
ImportError: libzmq.so.5: cannot open shared object file: No such file or directory

Thanks so much for looking into this.

Thoughts @SylvainCorlay?

@jakirkham I am looking into it.
@chapmanb is this OS X or Linux?

Thanks so much for investigating, this is on Linux.

I am running into a meeting. Can you check if copying or manually creating the symlink fixes it?

A workaround is to do conda install pyzmq -c conda-forge which rolls back to 4.1.*

Thanks, manually adding a symlink does fix it. I'm maintaining conda packages that depend on this but don't have explicit pins so am trying to figure out the best way to fix this generally. I definitely could pin zeromq to the 4.1.* versions if a proper fix will take a while to put in place. Thanks again.

If am afk for the next 50 min but a PR creating the symlink in build.sh (or copying the .so) would get my immediate attention.

I can merge from my phone.

I did not see it because the latest zmq does not seem to link with the .so.5 file. And installing the package locally did not downgrade zmq unlike an install from conda forge.

Thanks much, I just sent a PR with this fix. I don't have the conda-forge tooling setup locally but hopefully this is simple enough to pass on the first try. I appreciate the help getting this resolved so quickly.

Thanks for the fix! This should also be reported upstream as a bug in the zeromq CMake. It should not result in any different files than the normal build.


MAC

CAN WE INSTALL IT IN MAC

facing too many problems while installing it on mac.

MacOS official support will be removed soon

Hey did anyone get a workaround for Mac?
@six2dez is the resolved for Mac arm based?

Hi, there is no official support for Mac OS sorry


sqlite3-binding.c compiled time

I want to know when the sqlite3-binding.c is compiled? I found no clue and do not the use of the file.

every time when you do go build. if you don't want long time for compilation, go install github.com/mattn/go-sqlite3 should be help.

@mattn  Thanks. I got it.


Fix: No Auth Provider found for name "gcp"

When running kai with a kueconfig that uses gcp auth-provider, it fail with the same error described in kubernetes/client-go#242
This PR follows the solution described there and import auth plugin package of kubernetes/client-go module.

Hi @rogeriobastos, thanks for submitting this! I'll give this a look.


chore(docs/plus): move cdk8s+ docs to cdk8s-plus repo

Summary
This PR moves the CDK8s+ docs to the CDK8s+ repo. Updates the docs/build.sh to copy the files to the versioned docs/plus/$version/ sub dir once they are published.
Lets merge the CDK8s+ docs first so that we can test the docs/build.sh before merging:

https://github.com/cdk8s-team/cdk8s-plus/pull/1224

Issue:

https://github.com/cdk8s-team/cdk8s-plus/issues/493

Child PRs:

https://github.com/cdk8s-team/cdk8s/pull/1006

Merge order
A few of my PRs touch a lot of the same code so i've documented a merge strategy. See:

https://github.com/cdk8s-team/cdk8s-plus/pull/1224

Manually created a backport PR for v22, v23, and v24. We should merge these before merging this branch so that the docs are available to copy.
v24:

https://github.com/cdk8s-team/cdk8s-plus/pull/1250
v23:
https://github.com/cdk8s-team/cdk8s-plus/pull/1249
v22:
https://github.com/cdk8s-team/cdk8s-plus/pull/1248

Tested locally and it looks like it's g2g


Support synchronous testing of transactions

Given a Solidity code that looks like this:
contract TestContract
   address public addr;
   function setter(address a) {
      addr = a;
   }
}

We would get:
TestContract.at(TestContract.deployed_address).setter(val).sendTransaction({from: caller})

and
TestContract.at(TestContract.deployed_address).addr.call()

It would be nice to have a way to be able to synchronously test this by making the test framework determine the tx_blocknumber where the transaction was created then wait a few blocks until such that current_blocknumber > (tx_blocknumber + number) to ensure that the transaction commits to TX storage and be able to make assertions on the results.
There will be situations where the value of addr can be changed by other contracts or EOAs and we would like to be able to elegantly test these different cases.

See this for our rationale:
https://github.com/LeastAuthority/ethereum-analyses/blob/master/GasEcon.md#predictability

Hi @tymat. If I understand what you wrote correctly, this is already built in. Truffle uses Pudding under the hood, which makes ABI functions and sendTransaction() not call their callback until the transaction has been processed. The syntax is different than your example, but the effect is the same. Pudding uses promises to allow you to write the following:
val = "..." # some address
test = TestContract.at(TestContract.deployed_address)
test.setter(val, {from: caller}).then (tx) ->
  # This code block won't be executed until the transaction tx has been processed.
  # Note: The next line returns a promise.
  test.addr.call()
.then (addr) ->
  assert.isEqual(addr, val)

This should produce the effect you're looking for. Let me know if I misunderstood.

The function within Pudding that does exactly what you described is here: https://github.com/ConsenSys/ether-pudding/blob/master/src/pudding.coffee#L100

@tcoulter Thanks.  That seems to work.  Is there a way to set the timeout values?

Ya, that's a feature of mocha. There are three different scopes the timeouts can apply to. Consider this example:
contract "TestContract", (accounts) ->
  # Each test has a timeout of ten seconds.
  @timeout(10000)

  before "do something before", (done) ->
    # This before() hook has a timeout of 15 seconds.
    @timeout(15000)

  it "should test something", (done) ->
    # This test has a timeout of 30 seconds.
    @timeout(30000)

  it "should do something else", (done) ->
    # This test only has a timeout of 10 seconds, based on the timeout above.


Fix 'pod install' error when using react-native >= 0.60

Summary
This change will make this package workin on RN 0.60.0 or above,
Not sure if this is important at the moment, but i have an error when i am tring to install for RN 0.60.4
Detected React Native module pods for RNCReactNativeCheckbox, react-native-cameraroll, react-native-image-editor, react-native-netinfo, and react-native-webview
Analyzing dependencies
Fetching podspec for `RNCReactNativeCheckbox` from `../node_modules/@react-native-community/checkbox/ios`
[!] The `RNCReactNativeCheckbox` pod failed to validate due to 1 error:
    - ERROR | attributes: Missing required attribute `homepage`.
    - WARN  | source: The version should be included in the Git tag.
    - WARN  | description: The description is equal to the summary.

Test Plan

Create a fresh react-native project
Add this package with command yarn or `npm``
run inside your project :cd ios && pod install

What's required for testing (prerequisites)?

A fresh react-native project

What are the steps to reproduce (after prerequisites)?
Use the library in the project
Compatibility

OS
Implemented

iOS
✅

Android
✅

Checklist

[x] I have tested this on a device and a simulator
[x] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[ ] I updated the typed files (TS and Flow)
[ ] I added a sample use of the API in the example project (example/App.js)

Hello, i made the change :)


Dependency updates

Updates for spring boot and several other dependencies. I downgraded rest-assured from 4.3.0 to 4.2.0 to get rid of the groovy dependency. Apparently this is an issue in 4.3.0 (and 4.3.1) that also exists in other Java Version than 14. So let's just wait here.
I didn't manage to get Mockito updated so it's still 3.1.0. I'll create a dependency card for these two points.

 Check Detector dependencies


Re-introduce package-style import into flutter gallery

This has been rolled back in https://github.com/flutter/flutter/pull/8483, but it would be good to have it around to ensure that "package:" style imports of sources in your own project continue to work. Some of our customers seem to use it.

Related: https://github.com/flutter/flutter/issues/8451

Is there a bug tracking the bug that's preventing us from adding this?

/cc @abarth

This is that bug.  We need to add a couple more settings to https://github.com/flutter/flutter/blob/master/examples/flutter_gallery/BUILD.gn

@goderbauer is this issue stale by any chane? Does this still repro?


No explanation for ExpandResource parameter

In some commands, for example Get-AzVirtualNetwork there is no explanation for parameter -ExpandResource. Neither there are examples what this parameter is doing. Sometimes  this missing information it is causing problems on which it is hard to identify the real issue.
It would be helpful to explain also what not using this parameter is doing (i.e. it is returning only Id of referenced objects and all other properties are returned as null).

Please file this request with the engineering team, who are in charge of cmdlet documentation: https://github.com/azure/azure-powershell/issues


Added folder for sample data and included instructions for the Australian PBS medicines

Added a folder for sample-data.

created folder
created Readme with description of how to use PBS database

Note: I had hoped to include the processed .csv file but realised that it is copyrighted.  Therefore, I included links and instructions on how to use the PBS database file.

I have updated the pull request and moved it to the hospitalrun-server repo.

[x] fixed issue with naming of folder (now sample-data)
[x] added image to Readme.md to better describe column layout of PBS database

Closing this PR in favor of HospitalRun/hospitalrun-server#42


chore: Adding link-unlink icon from remixicon-react

Description
Adding link-unlink icon from remixicon-react
Fixes #25266
Depends on #25398
Type of change

Chore (housekeeping or task changes that don't impact user perception)

How Has This Been Tested?

Manual on storybook

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

tests passed here: https://github.com/appsmithorg/appsmith/pull/25398


Best practices for updating <Control.select> options on field value change

Had a quick question around a fairly simple use case. I've got a form with a "country" dropdown, and then a "state/province" dropdown. When I change the country from "US" to "Canada," I want the state/province dropdown to change from displaying a list of US states to a list of Canadian provinces.
My initial strategy for this was simply to supply the form reducer to the form component and change the options accordingly:
function mapStateToProps(state) {
  return {
    address: state.forms.shippingAddressForm,
  };
}

export class ShippingAddressForm extends Component {
  render() {
    const currentCountry = this.props.address.country;
    const stateProvinceOptions = getStatesOrProvincesForCountry(currentCountry);

    // return form using {stateProvinceOptions}
  }
}

export default connect(mapStateToProps)(ShippingAddressForm);

This sort of works, in that it does update the options in the form. However, it has one major flaw: If the user selects a state, then changes their country, it doesn't unset the previously-selected state.
Unfortunately, I don't immediately see a good pattern for this. I'm sure there's a few ways to do this - one of which would be to use some sort of custom reducer, though I'm not sure what that'd look like, and it seems like a lot of boilerplate - but is there a "best practice" way to handle this use case?

This looks like a very similar solution: https://github.com/davidkpiano/react-redux-form/issues/620#issuecomment-273768987
In short, you would solve this the same way you would solve it using just React + Redux. That is, have an onChange handler that ensures that the second value is related to the first value.
Alternatively, you can make this special "two-in-one" custom component and manage the logic of what is selected in component-specific state. Then, you can just do the custom component pattern of <Control component={CountryStateSelector} /> or something like that.

Oh, I didn't realize you could just pass onChange through to a <Control.select /> (or, in my specific case, a <Field /> that wraps a <select />). Is that explicitly documented somewhere I missed?
This handler seems to work :)
    handleCountryChange(evt) {
        const newCountry = evt.target.value;
        const newStates = getStatesOrProvincesForCountry(newCountry);
        this.props.dispatch(actions.change('forms.shippingAddressForm.stateOrProvince', newStates[0].value));
    }

Is that explicitly documented somewhere I missed?

I should probably add that, but you can pass any props that you would normally pass to the component anyway, and they'll be passed to the component.


Remove libnuma, update dependency to ucx>=1.15.0,<1.16

Since UCX 1.15 libnuma is not used anymore and thus it should be removed. Additionally update dependency to ucx>=1.15.0,<1.16.

/merge


Add support for skipLoadingService configuration parameter

Fixes #63

Coverage increased (+0.2%) to 53.196% when pulling cd4df5077c2256ee788b9a2dfecc59240f725e4d on Eurofunk:errorHttpInterceptor-with-skipLoadingService-suport into cce80eb8a5bc2470ca49168f4de9cfda985b1dda on Catalysts:master.

Coverage increased (+0.2%) to 53.196% when pulling cd4df5077c2256ee788b9a2dfecc59240f725e4d on Eurofunk:errorHttpInterceptor-with-skipLoadingService-suport into cce80eb8a5bc2470ca49168f4de9cfda985b1dda on Catalysts:master.

Coverage increased (+0.2%) to 53.196% when pulling 7f890359236d64afb8360e5f77eb01868b777852 on Eurofunk:errorHttpInterceptor-with-skipLoadingService-suport into cce80eb8a5bc2470ca49168f4de9cfda985b1dda on Catalysts:master.


license add (CSI-2484)

I think the idea is that the license defaults to false and the user must manually change it to true in a sort of acknowledging the acceptance of licenses used.

A new PR will be opened after getting all the required information (like do we need to add license to community and build a license text/link for this purpose)


fix(@angular-devkit/schematics): Fix issues for file-syetem-engine running in google3

Allow file-syetem-engine to resolve collection that has '@schematics' prefix.
Pass 'name' as part of FileSystemCollectionDesc when creating
collection description.

This is to minify the difference between internal and external CLI and how
they define and invoke schematics through short name.
e.g internally we could still use @schematics/angular or
@schematics/schematics to refer to a schematics
On Fri, Jul 27, 2018 at 17:48 Hans<EMAIL_ADDRESS>wrote:

@hansl commented on this pull request.
In packages/angular_devkit/schematics/tools/file-system-engine-host.ts
https://github.com/angular/angular-cli/pull/11697#discussion_r205926994:

@@ -36,6 +36,17 @@ export class FileSystemEngineHost extends FileSystemEngineHostBase {
return join(this._root, name, 'collection.json');
}

// Allow<EMAIL_ADDRESS>const SCHEMATICS_PREFIX = '@schematics/';

I don't follow the logic in this. If it starts with @schematics/ try
without the prefix? Why not change the default collection instead?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/angular/angular-cli/pull/11697#pullrequestreview-141301391,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdgRrk0OWw9lXfV_1oI0FckaNq12DeA4ks5uK7TjgaJpZM4Vkz5n
.

@hansl removed the @schematics prefix support, we can add it later when we really need do so.


Issues with mvn install with 10.3.1

after following all setup instructions and cloning this repository, this is the output of mvn install includes a bunch of errors.
I know that some files need to be changed from 10.3.0 to ````10.3.1``` but it seems like there are deeper issues with names changing.
Has anyone validated the build processes with 10.3.1 GES?

What kind of errors? The components have been tested at 10.3.1. Typically errors building in maven happen when maven cannot resolve the paths to the repository and system files and if the version in the pom is not the same as the version of the internal repository.

I believe that the "typical" errors are what I'm seeing, but there is not any guidance that I can see on where to update the version numbers and paths. I am running on windows, so for example, I believe the right path is
<url>file:///C:/Program%20Files/ArcGIS/Server/GeoEvent/sdk/repository</url>
instead of
<url>file:///geoevent-<IP_ADDRESS>71/geoevent-10.3.1/sdk/repository</url>
But should this be changed in every pom.xml file?

Sam,
Do you remember which pom you saw the repo ref to
file:///geoevent-<IP_ADDRESS>71/geoevent-10.3.1/sdk/repository<file:///\geoevent-<IP_ADDRESS>71\geoevent-10.3.1\sdk\repository>?
That is an artifact from using daily builds…
I’ll go through and check the paths…
From: Sam Libby<EMAIL_ADDRESS>Sent: Tuesday, June 02, 2015 12:47 PM
To: Esri/solutions-geoevent-java
Cc: Patrick Hill
Subject: Re: [solutions-geoevent-java] Issues with mvn install with 10.3.1 (#47)
I believe that the "typical" errors are what I'm seeing, but there is not any guidance that I can see on where to update the version numbers and paths. I am running on windows, so for example, I believe the right path is
instead of
<url>file:///geoevent-<IP_ADDRESS>71/geoevent-10.3.1/sdk/repository</url>
But should this be changed in every pom.xml file?
—
Reply to this email directly or view it on GitHubhttps://github.com/Esri/solutions-geoevent-java/issues/47#issuecomment-108075044.

https://github.com/Esri/solutions-geoevent-java/blob/5214186f8c9430085c8170e76e71c3b0ae524833/solutions-geoevent/pom.xml
From: Patrick Hill <notifications<EMAIL_ADDRESS>Reply-To: Esri/solutions-geoevent-java <reply<EMAIL_ADDRESS>Date: Tuesday, June 2, 2015 at 4:36 PM
To: Esri/solutions-geoevent-java <solutions-geoevent-java<EMAIL_ADDRESS>Cc: Sam Libby <slibby<EMAIL_ADDRESS>Subject: Re: [solutions-geoevent-java] Issues with mvn install with 10.3.1 (#47)
Sam,
Do you remember which pom you saw the repo ref to
file:///geoevent-<IP_ADDRESS>71/geoevent-10.3.1/sdk/repository<file:///\geoevent-<IP_ADDRESS>71\geoevent-10.3.1\sdk\repository>?
That is an artifact from using daily builds…
I’ll go through and check the paths…
From: Sam Libby<EMAIL_ADDRESS>Sent: Tuesday, June 02, 2015 12:47 PM
To: Esri/solutions-geoevent-java
Cc: Patrick Hill
Subject: Re: [solutions-geoevent-java] Issues with mvn install with 10.3.1 (#47)
I believe that the "typical" errors are what I'm seeing, but there is not any guidance that I can see on where to update the version numbers and paths. I am running on windows, so for example, I believe the right path is
instead of
<url>file:///geoevent-<IP_ADDRESS>71/geoevent-10.3.1/sdk/repository</url>
But should this be changed in every pom.xml file?
—
Reply to this email directly or view it on GitHubhttps://github.com/Esri/solutions-geoevent-java/issues/47#issuecomment-108075044.
—
Reply to this email directly or view it on GitHubhttps://github.com/Esri/solutions-geoevent-java/issues/47#issuecomment-108090927.

I have modified a few of the paths to repositories in the poms. In some cases they still pointed to daily build locations.

@pHill5136 - Is this ready for @slibby to verify and close?

@pHill5136 I re-cloned the repo and simply did mvn install and here's the output: https://gist.github.com/slibby/078d9599a205b8cf01b8

@slibby I changed back the main pom.xml to 10.3.0 because all of the internal poms were referencing 10.3.0 -- and so wouldn't build. If you have 10.3.1 installed I think you can just change that in your pom (individual and external). If you are just building a few components I would build them separately. I will be updating the poms to 10.3.1 sometime this week.

@slibby One other thing to complicate moving to 10.3.1 - the library declaration has changed slightly
You should use the following for the ges library dependency tag in the pom
<dependency>
			<groupId>com.esri.ges.sdk</groupId>
			<artifactId>ges-lib</artifactId>
			<version>10.3.0</version>
		</dependency>

@slibby Sorry, accidentally sent too soon with the old tag... Dangers of copy paste. This is the tag in the pom. You may want to check the paths in your distribution of geoevent. I typically install from a zipfile on the daily builds. This is how the path is set up in those for GEE 10.3.1 final.
        <dependency>
            <groupId>com.esri.geoevent.sdk</groupId>
            <artifactId>geoevent-sdk</artifactId>
            <version>10.3.1</version>
        </dependency>

Added 10.3.1 compliant poms


Add gluster_volume custom resource

UPDATED
It currently supports the start, stop, and delete actions. It is also useful for checking existence in only_if blocks.
A create action could be added later and the top-level volume methods in libraries/gluster.rb could probably be reimplemented here.

Hold that thought, I'm gonna add some start/stop stuff.

Okay, it's a bit more useful than just delete now!

Hi! Sorry for missing this, I'm going to do some testing and get back to you, apologies for the delay!

@shortdudey123 Could you share to the supermarket or add me as a contributor there so I can share it please?

Thanks! Just a note before you rush to do another release, I'm going to be trying 3.10 very soon. It's already on mirrors.centos.org so there's a one line change to fetch it from there.


In adapter-static,  expand <svlete:head> statically into generated html files.

Describe the problem
In adapter-static, <svlete:head> is not expanded statically into generated html files, need executing the javascript to get final <meta> tags. In use cases where crawler won't run js code (twitter, for example), this behavior will make twitter card failed to show.
Describe the proposed solution
Expand <svlete:head> that can be determined at compile time into generated html files.
Alternatives considered
No response
Importance
nice to have
Additional Information
No response

You need to enable SSR so that the page is rendered server-side during the prerendering process instead of returning an empty shell.