Figure out passing of env vars into CDK when used as CLI

I want to be able to pass environment variables during deployment, at the same time, I don't want to manage CDK myself as that's unnecessary overhead.
We need a convention on how to pass env vars to Lambda, without exposing all env vars as some of them might be private / unnecessary for app run.

Consider .env file. In case .env file is present during build, it will load on start from it. Basically, next build propagates the file into output and we zip it then.
We should consider that .env file is gonna propagate to the final build and into the code.
Next server's priority when resolving env vars (from top):
process.env
.env.$(NODE_ENV).local
.env.local (Not checked when NODE_ENV is test.)
.env.$(NODE_ENV)
.env

Support implemented by https://github.com/sladg/nextjs-lambda/commit/a85085d6968726dcea70def9ded4eb5f378d824a.
Version ^3.13.0


Graphic for AWS Marketplace announcement

Description
We will be announcing that FlowFuse is now available on AWS Marketplace. https://aws.amazon.com/marketplace/pp/prodview-3ycrknfg67rug?sr=0-1&ref_=beagle&applicationId=AWSMPContessa
It would be nice to get a graphic for this announcement.
Epic/Story
No response
Have you provided an initial effort estimate for this issue?
I have provided an initial effort estimate

PR is now available #1049

Hi @iskerrett wdyt?

Love it :-)


[installers] use mamba instead of conda?

@newville this is relatively low priority for the moment, but I propose to move to mamba as package manager for installing Larch. If we keep using the Miniforge distribution, we could simply switch to mambaforge and then use mamba command instead.
What do you think?
I am proposing this because yesterday I did a Larch installation on Ubuntu 20.04 laptop and it was a pain. For a reason that I do not understand, both binary and installer failed. At the end the only way I managed to install Larch was manually with mamba.
If it is ok for you, I can take care updating the installer scripts and relative installation instructions in the documentation.

@maurov yeah, that is definitely worth trying!


Glossary not available

(From a question on the Atom #electron Slack channel.)
All of the glossary links 404, for example, http://electron.atom.io/docs/glossary#renderer-process. The markdown file exists in electron/electron.

@sebcode fixed it in #689


The regular expression "^[a-zA-Z0-9.-_]+$" should only be applied to fixed-fields under component

Examples map keys defined in components must adhere to a pattern: '^[a-zA-Z0-9._-]+$'. (Source: https://github.com/OAI/OpenAPI-Specification/blob/main/schemas/v3.1/schema.yaml)
However, no such restrictions are defined on the examples map defined inside of a media-type object.
openapi: 3.0.0
info:
  title: Sample API
  description: Optional
  version: 0.1.9

paths:
  /users:
    get:
      summary: Returns a list of users.
      description: Optional extended description in CommonMark or HTML.
      responses:
        '200':
          description: OK
          content:
            application/json:
              schema:
                type: object
              examples:
                My valid example:
                  value:
                    foo: bar
                A_valid_example:
                  value:
                    baz: quz

As you can see on the example above, the name of the example My valid example will be defined as invalid by rule:  spec-components-invalid-map-name.

example-yaml.yaml.zip
Expected behavior
rule: spec-components-invalid-map-name should apply only to the fixed fields under the components, this restriction should not apply to examples mapped inside of a media-type object
 Front conversations

@ZyzychO We added this to resolve issue https://github.com/Redocly/redocly-cli/issues/959. Do we need to change this behavior?
@tatomyr

@IgorKarpiuk I believe we should comply with the specification. If someone wants additional restrictions, they may use a custom rule.

@IgorKarpiuk that example is correct and the behavior should stay the same for components. The sample file provided for #959 though didn't include inline examples defined within paths.
components: { it should check here }
paths: { it should not apply here }

@adamaltman @lornajane  should we validate only top level keys in components? The specification does not tell anything about the nested keys: https://spec.openapis.org/oas/v3.1.0#components-object

Great question! Yes, I think we have to limit this check to the keys inside the fixed fields in components.
It looks like it would technically be valid (although probably unwise) to use a key with spaces in it in JSON Schema, and as you correctly point out, the spec does not actually state that the rules apply beyond the first level keys. Perhaps we can add a rule example for making your keys match sane patterns - I suspect the spaces, even if valid in OpenAPI, would trip up plenty of tools, so users might want to add that for their own sanity.


text over 200 length

Try splitting text over 200 chars long and make multiple audio requests

Or just the first 200 characters
Rather than the text failing.

We've increased this limit to 1000 chars.


Don't bother archiving / unarchiving data.

This change upgrades to PINCache 3.0.1-beta which supports
custom archiving and unarchiving.
This allows us to avoid using NSArchiver/Archiver which adds a
small amount of disk space overhead and between .5 and 10 ms (on an iPhone 4)
for each encode / decode.
To support this, we do have to add versioning for the cache and
destroy it for the old versions. I considered adding a process to migrate
the images, but because that would have to occur on startup and process
all existing images, it could cause issues for very large image cache
databases, so I decided against that.

Also added

(void)removeCachedObjectForKeyFromMemoryCache:(NSString *)key;
to the image caching protocol


Issue with assigned space rule in Yabai

Hi, in my case I have assigned Intellij to space 4 via the config yabai -m rule --add app="IntelliJ IDEA"         space=4.
It works fine but when I move it to a different space let's say to space 5 and any pop window gets opened, then Yabai starts behaving weird.
In my case when I open system preference when it's on space 5, Yabai keeps on toggling between space 4 and 5.
Any idea how I can fix this?
Attaching a video for the same so that it's easier to relate. Let me know if more info is needed from my end.
https://user-images.githubusercontent.com/3875065/126457550-72fe8353-4aa9-4c38-b222-76e4facd10cb.mov

This seems weird. I can't really tell right now why exactly yabai would make that happen.
Just to confirm, have you disabled the following mission-control setting?

In the Mission Control preferences pane in System Preferences, the setting "Automatically rearrange Spaces based on most recent use" should be disabled for commands that rely on the ordering of spaces to work reliably.

Yes I have it disabled.

@koekeishiya Is there anything I can do to debug this from my end? Or share any logs for the same?

So it appears to that this is what's happening:
IntelliJ is launched (for the first time) and yabai moves it to space 4.
Later on at some point you move it to space 5.
When a popup-window is created, yabai moves it to space 4. Because this is a popup, macOS follows the focus (or something, as yabai is not making space 4 focused here). The popup is automatically closed because IntelliJ is no longer the active application/on the active space, and when the popup closes, IntelliJ receives focus again, making the switch to space 5.
Am I understanding the issue correctly?
If so I believe the fix is to adjust your rule such that popup windows (and everything that is not the main editor window) is unaffected by the rule.
If you can try to retrieve the output of running yabai -m query --windows while IntelliJ and some of the popup-windows are visible, while on space 4, the information might give you some tips as how to adjust your rule. you can probably do this by having a terminal session and doing a sleep 2 && yabai -m query --windows. While the sleep is executing, you tab into intelliJ and trigger a popup.

Changes have been made to newest releases of yabai that should have resolved this issue.


Remove user_id from account/events

No longer a thing

Coverage decreased (-0.03%) to 70.897% when pulling d06991a5ba032257018a1f304e3f2ac8d1a7807f on yoshizzle:eventUserID into f35ac2035f01a5b028a58ef6989181fcde4faeef on linode:develop.


Port projectRoot calculation and documentID for the filePath into ProjectAPI

Description:
Port projectRoot calculation and documentID for the filePath into ProjectAPI.
Steps to reproduce:
Affected Versions:
2.0.0-SLP6-SNAPSHOT

Will be solved with https://github.com/ballerina-platform/ballerina-lang/pull/27307

Need to address changes in https://github.com/ballerina-platform/ballerina-lang/issues/27274

Need to address changes in https://github.com/ballerina-platform/ballerina-lang/issues/27274


Support $PYTHONSTARTUP for ptipython

Support the PYTHONSTARTUP environment variable for ptipython.
:beers:

Hi @timofurrer,
Thank you for this! Merged in: https://github.com/jonathanslenders/ptpython/commit/bdd69773f0913bd5a771e66ca957002b94bf473f

Awesome!
Would it be possible to make a release?

Hi @timofurrer,
I just pushed release 0.32. Good luck with try!


content-based publish/subscribe middleware

Is wamp fall into category of topic-based publish-subscribe middleware? how is different from content-based publish/subscribe middleware like padres and IBM Gryphon ?

This is a good question for the WAMP discussion group.  https://groups.google.com/forum/#!forum/wampws

thanks for pointing me to that group, :-D


DevTools location / placement - opening devtools on the bottom instead of side

I have this which works:
 const browser = await puppeteer.launch({
    headless: false,
    devtools: true,
    ignoreDefaultArgs: true,
    args: args.concat([
        '--remote-debugging-port=9223',
        '--load-extension=/some/local/path/to/your/extension'
      ]
    )
  });

however this opens DevTools on the right hand side, instead of the bottom of the browser. See:

Is there a flag we can use to tell Chrome to open DevTools on the bottom of the page?
I looked through the issues and don't see a related question, thanks, sorry if it's a dumb question but was hard to find an answer with Google.

This would be possible with #1693; there's a devtools.currentDockState preference that could be assigned to 'bottom'.

@aslushnikov Would you be so kind to post a code snippet showing how/where to place devtools.currentDockState = 'bottom' ?

Friendly bump. Where does one set devtools.currentDockState = "bottom"?

@chriscalo I will leave it here in case someone is still looking how to launch Puppeteer with DevTools stacked to bottom:
`const puppeteer = require('puppeteer-extra');
const puppeteerPrefs = require('puppeteer-extra-plugin-user-preferences');
(async () => {
puppeteer.use(puppeteerPrefs({
userPrefs: {
devtools: {
preferences: {
currentDockState: '"bottom"'
},
},
},
}));
const browser = await puppeteer.launch({
    headless: false,
    devtools: true
});`


Support for one-to-many mappers

We should have something similar in the push-processor plug-in. Probably at some point we should have just one generic processor that can handle both the Kinesis pull event model and the push notification model.
Please review.

This change is

LGTM.
I just didn't understand a few things in the test_lambda part (the way you're using parametrize) but it seems right.

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Expanded height does not resize on iOS in Safari upon orientation rotation

In iOS 9 (maybe others), if you expand the text and then rotate the phone from portrait to landscape mode the height of the element that .readmore function is applied to does not recalculate. This leaves a large gap between that element and the next one.
This appears to be a mobile Safari specific issue as it does not occur in Chrome on iOS nor on Android.

This is an issue on all devices. If you expand a text, resize the window, so the textbox is wider, the height won't change.

No, that is not true. On all other devices - and in Chrome on iOS - changing the window size causes the expanded text element to resize.

Same problem here in Chrome 53 on Android 6: No recalculation of the height after orientation change.

Seems it will never fixed...


updater.phar times out.

LS,
whenever I want to run updater.phar, I get a timeout:
------------8<-----------------
sudo -u apache php /data/nextcloud/nextcloud/updater/updater.phar -vvv
Nextcloud Updater - version: v20.0.0beta4-11-g68fa0d4
Current version is 21.0.9.
[Exception]
Could not do request to updater server: Operation timed out after 300182 milliseconds with 0 out of 0 bytes receive
d
Exception trace:
() at phar:///data/nextcloud/nextcloud/updater/updater.phar/lib/Updater.php:472
NC\Updater\Updater->getUpdateServerResponse() at phar:///data/nextcloud/nextcloud/updater/updater.phar/lib/Updater.php:119
NC\Updater\Updater->checkForUpdate() at phar:///data/nextcloud/nextcloud/updater/updater.phar/lib/UpdateCommand.php:147
NC\Updater\UpdateCommand->execute() at phar:///data/nextcloud/nextcloud/updater/updater.phar/vendor/symfony/console/Command/Command.php:256
Symfony\Component\Console\Command\Command->run() at phar:///data/nextcloud/nextcloud/updater/updater.phar/vendor/symfony/console/Application.php:820
Symfony\Component\Console\Application->doRunCommand() at phar:///data/nextcloud/nextcloud/updater/updater.phar/vendor/symfony/console/Application.php:187
Symfony\Component\Console\Application->doRun() at phar:///data/nextcloud/nextcloud/updater/updater.phar/vendor/symfony/console/Application.php:118
Symfony\Component\Console\Application->run() at phar:///data/nextcloud/nextcloud/updater/updater.phar/updater.php:10
require() at /data/nextcloud/nextcloud/updater/updater.phar:10
update [--no-backup]
------------8<-----------------
And the option 'verbose' doesn't seem to do anything. And the error is much too sparse and says nothing.

Maybe some load on our side, should be good now

Was an issue on my side: After my ISP changed my IPv6 address, the nextcloud server couldn't download anymore. The network configuration had some errors. After fixing them things work again. Sorry, for the inconvenience.


Feature/add name to date

Adds the ability to pass name property down to individual input elements within the Date Input component.

[x] Documentation
[x] Tests
[x] Ready to be merged

Codecov Report

Merging #367 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #367   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          77     77
  Lines         409    409
  Branches       14     14
=====================================
  Hits          409    409

Impacted Files
Coverage Δ

components/date-input/src/index.js
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


RuntimeError: Exporting/importing meta graphs is not supported when eager execution is enabled. No graph exists when eager execution is enabled.

2022-02-11 11:05:30.792376: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcudart.so.10.1
Traceback (most recent call last):
File "01_GenQuadProposals.py", line 82, in
CNNModel.restoreCNNSess()
File "/home/dms/SupplementaryMaterials/CodeAndData/Code/QuadProposals/CNNQuadDetector.py", line 92, in restoreCNNSess
saver = tf.train.import_meta_graph(self.cfg.cornerdet_sess + '.meta', import_scope="cornerdet")
File "/home/dms/anaconda3/envs/SupplementaryMaterials/lib/python3.8/site-packages/tensorflow/python/training/saver.py", line 1460, in import_meta_graph
return _import_meta_graph_with_return_elements(meta_graph_or_file,
File "/home/dms/anaconda3/envs/SupplementaryMaterials/lib/python3.8/site-packages/tensorflow/python/training/saver.py", line 1472, in _import_meta_graph_with_return_elements
raise RuntimeError("Exporting/importing meta graphs is not supported when "
RuntimeError: Exporting/importing meta graphs is not supported when eager execution is enabled. No graph exists when eager execution is enabled.
System information
tensorboard 2.8.0
tensorboard-data-server 0.6.1
tensorboard-plugin-wit 1.8.1
tensorflow 2.3.1
tensorflow-estimator 2.3.0
tensorflow-hub 0.12.0
tensorflow-io-gcs-filesystem 0.24.0
tensorflowjs 3.13.0
cuda 10.1
cudnn 7.6
python 3.8
has update import tensorflow. as tf  to import tensorflow.compat.v1 as tf

@jackiezhang11 ,
Can you please take a look at this issue with the same error.Also please try to update to latest tf v2.7 and let us know if the issue still persists.Thanks!


Determine and use Consensus Module using On-chain Settings

During genesis block construction, the consensus module can be loaded based on the settings supplied by the sawtooth_config transaction family.  The module is then used to correctly initialize and finalize the block before committing it to the block store.
This does not cover the same behavour in journal, which will be part of a future PR.

Updated with changes needed for Journal, as well as the minor changes to ConsensusFactory requested by @vaporos


Update actions e2e test to cover print_event with multiple events

Closes AAP-5634.

3.9 test fails, not sure why

The process timed out. Not sure why, trying to reproduce locally but so far unable to

can you add -vv here? https://github.com/ansible/ansible-rulebook/blob/main/.github/actions/e2e-tests/action.yml#L9

It's a time race. Let's decrease the event delay


remove unused code with knip

pnpm exec knip --fix
No behavior change, just a dead code removal.
Test plan
CI

Stamp pls!

@sqs should this be added as a CI check so that people don't accidentally commit dead code?

@varungandhi-src Perhaps in the future. I added most of the manual exceptions (where we want to keep some dead code around) to the knip.jsonc file, but it's not totally automatable yet. And there wasn't too much dead code around, so it's not a big problem.


Protect conversation rank/version with an advisory lock

Use an advisory lock to protect the transaction that inject new Messages
Fixes: https://github.com/dust-tt/tasks/issues/129
Based on https://dust.tt/w/0ec9852c2f/assistant/ab3cb5c164
Initial logging to convince ourselves that this is super fast, to be removed

You could release the lock before the transaction is committed though. Not a huge deal but not ideal.

Never-mind, lock must be released on commit for the other transaction to see it (isolation level is read committed).


fix: add -w option to base64 command in circleci activation process

Changes

In order to disable line wrapping, add -w 0 option to the base64 command in the CircleCI activation process

Checklist

[X] Read the contribution guide and accept the
code of conduct
[X] Readme (updated or not needed)
[X] Tests (added, updated or not needed)

I'm not sure why this would be needed, seems to work perfectly well without it.
Could you please provide additional context about the use cases where it does in fact not work?

I'm not sure why this would be needed, seems to work perfectly well without it.
Could you please provide additional context about the use cases where it does in fact not work?

Open a thread on Discord #help channel (https://discord.com/channels/710946343828455455/1054825834512384042) and seems to be the missing option that causes the issue.
But also, could be that I'm using an IL2CPP platform when the game license has on the Mono platform, I will test this again without that option. Thank for your feedback @webbertakken

Sounds good. If this is indeed needed we'd be happy to merge it.

@webbertakken I test again,
failed at the "Prepare the environment" step ❯ cat Unity_v2021.x.ulf | base64 > UNITY_ENCODED_LICENSE (with and without the final newline):
Detected OS: Linux.
base64: invalid input
Failed to find the serial or parse it from the Unity license.
Please try again or open an issue.

Exited with code exit status 1

Pass with ❯ cat Unity_v2021.x.ulf | base64 -w 0 > UNITY_ENCODED_LICENSE (output is a one-line string without any newline):
Detected OS: Linux.
+++ mkdir -p /root/.cache/unity3d
+++ mkdir -p /root/.local/share/unity3d/Unity/
+++ set +x
Writing '$UNITY_LICENSE' to license file /root/.local/share/unity3d/Unity/Unity_lic.ulf

Cool. Feel free to reopen.

Cool. Feel free to reopen.

@webbertakken when we don't use -w 0, binary encoding to multi-lines, and then due to some systems using different base64 binaries, which could be caused by the line separator which defaults to Windows ("\r\n") and Linux ("\n") systems.


[turborepo] Crash in Windows

What version of Turborepo are you using?
1.6.3
What package manager are you using / does the bug impact?
pnpm
What operating system are you using?
Windows
Describe the Bug
It suddenly crashes on one of the repo (still working on other repos):
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x0 pc=0x14bf14c]

goroutine 8 [running]:
github.com/vercel/turbo/cli/internal/daemon/connector.(*Connector).getOrStartDaemon(0xc0015b1b58)
        github.com/vercel/turbo/cli/internal/daemon/connector/connector.go:287 +0xcc
github.com/vercel/turbo/cli/internal/daemon/connector.(*Connector).connectInternal(0x0?, {0x1934458, 0xc00003a0c8})
        github.com/vercel/turbo/cli/internal/daemon/connector/connector.go:219 +0x74
github.com/vercel/turbo/cli/internal/daemon/connector.(*Connector).Connect(0xc0015b1b58, {0x1934458?, 0xc00003a0c8?})
        github.com/vercel/turbo/cli/internal/daemon/connector/connector.go:200 +0x2c
github.com/vercel/turbo/cli/internal/daemon.GetClient({0x1934458, 0xc00003a0c8}, {0xc000035c40, 0x19}, {0x19385b8, 0xc0001a8ab0}, {0x17f7ded, 0x5}, {0x0, 0x0})
        github.com/vercel/turbo/cli/internal/daemon/daemon.go:304 +0x1da
github.com/vercel/turbo/cli/internal/run.(*run).run(0xc000005680, {0x1934458, 0xc00003a0c8}, {0xc0000ab170, 0x1, 0x1})
        github.com/vercel/turbo/cli/internal/run/run.go:211 +0x350
github.com/vercel/turbo/cli/internal/run.GetCmd.func1(0xc000233180, {0xc0000ab170, 0x1, 0x1})
        github.com/vercel/turbo/cli/internal/run/run.go:119 +0x24d
github.com/spf13/cobra.(*Command).execute(0xc000233180, {0xc00008e060, 0x1, 0x2})
        github.com/spf13/cobra@v1.3.0/command.go:856 +0x67c
github.com/spf13/cobra.(*Command).ExecuteC(0xc00021b180)
        github.com/spf13/cobra@v1.3.0/command.go:974 +0x3b4
github.com/spf13/cobra.(*Command).Execute(...)
        github.com/spf13/cobra@v1.3.0/command.go:902
github.com/vercel/turbo/cli/internal/cmd.RunWithArgs.func1()
        github.com/vercel/turbo/cli/internal/cmd/root.go:54 +0x2f
created by github.com/vercel/turbo/cli/internal/cmd.RunWithArgs
        github.com/vercel/turbo/cli/internal/cmd/root.go:53 +0x2b0

Expected Behavior
Not crashing
To Reproduce
This is the repo: https://github.com/justland/just-web
Do not know if you will see the same error. The issue persist for me even if I delete the repo and clone again.
Just do:
pnpm i
pnpm verify # or any other script using turbo

Reproduction Repo
https://github.com/justland/just-web

After restarting Windows, it starts working. Keeping this open to see if you can figure out what to guard based on the backtrace.

This also happened to me, and was also resolved by restarting my Windows PC.
I unsuccessfully tried resolving the issue with the following:

❌ Deleting Turbo's caches
❌ Re-installing Turbo via yarn
❌ Closing & re-opening VS Code
❌ Killing all Node.js and console host instances (via Task Manager)

✅ Only rebooting my PC resolved the issue.
I was unable to reproduce the issue. I tried the following:

❌ Closing & re-opening VS Code while running Turbo
❌ Killing turbo.exe via Task Manager
❌ Killing random Node.js and console host process spawned by Turbo, while it was running
❌ Ctrl-C- terminating Turbo via the command line

My context:

Windows 11 Pro 22H2
VSCode 1.74.0-insider
Node.js v16.18.0
<EMAIL_ADDRESS><EMAIL_ADDRESS>(running via yarn turbo run <etc>)

Avira antivirus detect a malware in turbo.exe and puts it in quarantine.
https://www.virustotal.com/gui/file/30d3c052a097f3984b6eb64985a473b2538f1ea51955db950cf94b2354e02928

Unsure what the problem is, I wonder if upgrading to windows 11 will fix the issue? The error message appeared all of a sudden after running pnpm turbo build

if upgrading to windows 11 will fix the issue?

I am on Windows 11

We also saw this issue after upgrading to latest turborepo version on windows. It does not happen on Mac.

Same here
Windows 10
NodeJS v18.12.1
turbo 1.6.3
pnpm 7.21.0
Restarting windows solved the issue for me.

This was addressed by #2557.


Copy css on drag

Not so much of an issue, but more of a feature, but i would of thought that when an item is being dragged, it would make sense to copy the existing css of the sortable-item and then apply the sortable css afterwards.  This would mean you wouldnt have to copy duplicate css and make things alot cleaner..

:+1:  to that.

+1

+1

+1


ZonedDateTime test vs self failing in non-UTC timezone

When the below code is tested in a non-UTC timezone it fails whereas testing against self should always return true for matchers such as sameDay, sameYear, etc.
final ZonedDateTime date = ZonedDateTime.of(2000,1,1,0,0,0,0,ZoneId.of("UTC"));
assertThat(date, ZonedDateTimeMatchers.sameDay(date).atZone(ZoneIds.UTC);

Found this issue at ZoneId.of("America/New_York").

The error was found under hamcrest-date-2.0.5 taken from maven, but I cannot reproduce  it under 2.0.7-snapshot from this repository.

The code has reworked between 2.0.5 and 2.0.6 to make the logic more
consistent and reusable. I'll add some further tests to confirm that
treating self vs self gives the right answer
On Tue, 24 Sep 2019, 08:38 Alecsvan<EMAIL_ADDRESS>wrote:

The error was found under hamcrest-date-2.0.5 taken from maven, but I
cannot reproduce it under 2.0.7-snapshot from this repository.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/eXparity/hamcrest-date/issues/32?email_source=notifications&email_token=AAPH3VPMILM7WCL7ONJLY3LQLG7XPA5CNFSM4IZ3I4JKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7NMYZQ#issuecomment-534432870,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPH3VNLCM6SYSOXGCLMWTDQLG7XPANCNFSM4IZ3I4JA
.

When 2.0.6 works correctly comparing the same zoned date to itself, there is still a confusion in fail reporting as timezone shifts. E.g. for UTC-5 system timezone:
ZonedDateTime date1 = ZonedDateTime.of(2000,1,1,0,0,0,0,ZoneId.of("UTC"));
ZonedDateTime date2 = ZonedDateTime.of(2001,1,1,0,0,0,0,ZoneId.of("UTC"));
assertThat(date1, sameInstant(date2));
Assert correctly fails but reporting displayes totally incorrect dates:
java.lang.AssertionError:
Expected: the same date as Sun, 31 Dec 2000 07:00:00.000 PM -0500
but: the date is Fri, 31 Dec 1999 07:00:00.000 PM -0500

Hi @Alecsvan, thanks for continuing to look into it and yes the nature of the bug has changed; the comparison is fine but message is using the current timezone.
I'm looking at the application of the atZone change which added atZone to all matchers and was related to issue https://github.com/eXparity/hamcrest-date/issues/20. It may be redundant now for some of the matcher cases with the changes that have made between 2.0.5 -> 2.0.6. These changes removed any code which added missing information such as times or timezones to the actual dates (These fudges were added to try and provide comparable temporal objects).
The original issue https://github.com/eXparity/hamcrest-date/issues/20 which added atZone was failing with the test below because the date was shifting:
@Test
  public void isDay_WhenStartOfDayInGmtPlus1_ThenOffsetShouldBeIgnored() {
    LocalDate firstDayOf2018 = LocalDate.of(2018, Month.JANUARY, 1);
    ZonedDateTime startOfFirstDayOf2018InGmtPlus1 = ZonedDateTime.of(firstDayOf2018, LocalTime.MIDNIGHT, ZoneId.of("GMT+1"));

    Date dateToCompare = Date.from(startOfFirstDayOf2018InGmtPlus1.toInstant());
    assertThat(dateToCompare,
      isDay(firstDayOf2018.getYear(), firstDayOf2018.getMonth(), firstDayOf2018.getDayOfMonth()));
  }

I'll confirm the result of this test is still working and then begin reviewing the error messages

This has now been fixed in master and will be release shortly. There are still issues with java Date but they're wider issues of moving around timezones so i'll raise a separate issue


Figuring out HTML 7

OPS Build status updates of commit 0474702:
:clock10: Preparing: average preparing time is 38 sec(s)

OPS Build status updates of commit 0474702:
:clock10: Incremental building: average incremental building time is 18 sec(s)

OPS Build status updates of commit 0474702:
:warning: Validation status: warnings

File
Status
Preview URL
Details

enterprise-graph/index.yml
:warning:Warning

Details

enterprise-graph/index.yml

[Warning] Author Stephen Flanagan cannot be recognized in GitHub.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Readme: "Create a topic branch" link broken

Under the section Submitting a pull request, the link to "Create a topic branch" goes to:
http://learn.github.com/p/branching.html
This is what the Wayback Machine says it used to look like:
https://web.archive.org/web/20120315041836/http://learn.github.com/p/branching.html
The link in the readme is broken and doesn't resolve to anything useful.  Three possible choices for replacements are:
(A) https://help.github.com/articles/creating-and-deleting-branches-within-your-repository/
(B) https://guides.github.com/introduction/flow/
(C) https://github.github.com/on-demand/intro-to-github/04/
or even things more animated, like http://learngitbranching.js.org/?demo
Which one makes the best replacement may depend on the intended audience.  (B) is the most interesting and shortest, but requires a couple clicks and doesn't give syntax.  (A) is a bit more specific but short, and leads to (A).  (C) is a part of an online class, and might be closest in spirit to the original link content.  It has a video.  But it leads people on to the next lesson in the class and might be confusing for that reason.
I vote (B) but interested in any feedback before submitting a pull request.

Awesome, thanks for opening this @mnorelli. I also cast my vote for (B) -- it feels the closest in spirit to the original link.


Best practice to run FastAPI on Cloud Run with server port as $PORT

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
"""
A sample Hello World server.
"""
import uvicorn

import os

from fastapi import FastAPI, Request
from fastapi.staticfiles import StaticFiles
from fastapi.templating import Jinja2Templates

app = FastAPI()
app.mount("/static", StaticFiles(directory="static"), name="static")
templates = Jinja2Templates(directory="templates")

@app.route('/')
def hello(request: Request):
    """Return a friendly HTTP greeting."""
    message = "It's running!"

    """Get Cloud Run environment variables."""
    service = os.environ.get('K_SERVICE', 'Unknown service')
    revision = os.environ.get('K_REVISION', 'Unknown revision')

    return templates.TemplateResponse('index.html', context={
        "request": request,
        "message": message,
        "Service": service,
        "Revision": revision})

if __name__ == "__main__":
    uvicorn.run("main:app", host="<IP_ADDRESS>", port=int(os.environ.get('PORT', 8000)), log_level="info")

Description
I've been using Google Cloud Code in VSCode and PyCharm and the example they have for Python is a Flask app. I've been trying to rewrite the example app but with FastAPI.
To have it running on Cloud Run you have to start the server with the port binded to an OS env variable called PORT.
My question is:

Should I start uvicorn programmaticaly like in the example or,
Should I have a "normal" FastAPI app main.py file and use the CMD or ENTRYPOINT to bind the $PORT variable?

Note: ENTRYPOINT ["uvicorn", "main:app", "-port", "$PORT"] won't work because the CMD in exec form won't substitute the variable to its value, as documented here. It must be in shell form ENTRYPOINT uvicorn main:app --port $PORT
Operating System
Linux
Operating System Details
No response
FastAPI Version
0.78.0
Python Version
Python 3.10.4
Additional Context
The Cloud Code Flask sample code: https://github.com/GoogleCloudPlatform/cloud-code-samples/tree/v1/python/cloud-run-python-hello-world

Do not run uvicorn alone.
Read this docs from their page.
https://www.uvicorn.org/deployment/
They recommend to run it with gunicorn and to use UvicornWorker:
"Run gunicorn -k uvicorn.workers.UvicornWorker for production"

Do not run uvicorn alone. Read this docs from their page. https://www.uvicorn.org/deployment/
They recommend to run it with gunicorn and to use UvicornWorker: "Run gunicorn -k uvicorn.workers.UvicornWorker for production"

From the FastAPI documentation:

My question is about how should I pass the $PORT variable to the server (uvicorn/gunicorn) not about the server itself to use.

AFAIK you are not able to reference host environment variables in your docker file, so the best way to achieve this is the example you gave. In reality, there isn't really a difference between using the a CLI command vs starting Uvicorn from within a Python file. Using a CLI would also start a Python process, which then would call the exact code as you have in your example (e.g. uvicorn.run()), they are interchangeable.

Thanks for your feedback @JarroVGIT. What made me open this question was the fact that the entire documentation of FastAPI doesn't mention the option to start uvicorn from the python file.

I think (but am not sure!) that is because in the CLI you can have the -reload option that is lacking in the direct way, but for production usecases this wouldn’t make sense anyway :)

You can use reload on uvicorn.run(). The thing is that the application path needs to be a string, and not the application object.
if __name__ == "__main__":
    uvicorn.run("main:app", reload=True)

Ref.: https://www.uvicorn.org/deployment/#running-programmatically

Interesting, didn’t know you could pass a reload parameter, that will come in handy! Although my main pattern is to pass the application object itself, I might need to tweak my workflow a bit :)

another derivative you can do is.
# runserver.py

from uvicorn import Config, Server
from fastapi import FastAPI

app = FastAPI()

if __name__ == "__main__":  # pragma: no cover
    server = Server(
        Config(
            "runserver:app",
            host="<IP_ADDRESS>",
            port=9002,
            reload=True,
        ),
    )

    # do something you want before running the server
    # eg. setting up custom loggers

    server.run()

you may also want to consider using pydantic[dotenv] to fetch the env variables.

Having multiple ways to achieve the same goal can be more confusing than helpful. 😅

@Kludex
You are right,
Though I think this variant is worth mentioning. As he might be looking for a configuration step before running the server.
such as this scenario.
https://github.com/tiangolo/fastapi/issues/1276#issuecomment-615877177
where loguru fails to intercept the first 3 entries in the log.
but yeah this might be too much answer for what has been asked.

I have cloned the Google's example repository for Cloud Code but with FastAPI here: https://github.com/alexsantos/cloud-run-fastapi
It is working both with the Dockerfile and with Cloud Build for Python.

I have cloned the Google's example repository for Cloud Code but with FastAPI here: https://github.com/alexsantos/cloud-run-fastapi It is working both with the Dockerfile and with Cloud Build for Python.

@alexsantos
thank you for sharing this! I'm curious how the Dockerfile + docker-compose.yml would look like if 1 had to link a cloud db? (at the moment, I'm trying to connect w/ AlloyDB, but when I am deploying it to CloudRun I receive a Error: Invalid value for '--port': '$PORT' is not a valid integer error on the cloud run logs)
Any advice would be appreciated!

I have cloned the Google's example repository for Cloud Code but with FastAPI here: https://github.com/alexsantos/cloud-run-fastapi It is working both with the Dockerfile and with Cloud Build for Python.

@alexsantos thank you for sharing this! I'm curious how the Dockerfile + docker-compose.yml would look like if 1 had to link a cloud db? (at the moment, I'm trying to connect w/ AlloyDB, but when I am deploying it to CloudRun I receive a Error: Invalid value for '--port': '$PORT' is not a valid integer error on the cloud run logs) Any advice would be appreciated!

Hello and sorry for the delay: why are you using a docker-compose with a cloud DB?


Multiple ground polygons results in disappearing on camera change

Sandcastle example: https://cesiumjs.org/Cesium/Build/Apps/Sandcastle/index.html#c=lVbbbts4EP0VIS91UC8tUaIoOW6w3SQFFnCTog5SFPViQUtjW1iZdEnK3jTwv+/o4voSeZPqwZKGc85cOIfWSmhnrXSe3oPWIpPOO+cKTFYsSKJBWPiyt9Z5GksHLw3fCzD2AbSFf2+VXojc9B2rCxjLzfnFWI7lCmlXGaxBI6GE9Zb0obJ13iTV65WSFnlBv+k6DbdJQIJ/fSfzx5qy29jnIlVrDDPFYFtjk8hnkCnojyqFA4its/6k1SrD9f5BnT8zxVyrnEgVmcxyNQECUkxyGGazuc3kDEuool4c+U7VrPFMT7nUdCks7fweM30/w9DG7npdptuOaQre7ceoMpRVks83Vzd/Ptz8fXc7/HqMTmFSzEZztf6gxQLMJ9AjSJRMnV20enuWKn9E27e6WU9DYfuOzwmN3MDnMY2DOIy6zlDJvvOb54WEURbQiEdRSMNN9xmKItANqe/F7BDFqcv8yAu9uAXFXeryAH+PQUEQ4k/EeAso9LgX8MBlcXCIogGLAurxgLagfEo9P8Zs/KNYSBjFkYd1taC8kLkhi3yXHTXDCyPKQ9palcu9MI4iz+dHID8Mgtjnrhe1oSLOQ+y8f9yMIPA451hBvBnLv35uYAozDWDe4yxVG1muTJV2OtXuosW9wNug2miSg5zZORrevj3faq2egkxa9C2dvi1LinJln5osCzPvVH4Es+o6zaOwpYA2u3Eymc2U3JvYK4EHhMmE9MlUq8X1HmlnP0LJ0+uVJDjBVqssfYmjs53Ibt06VlKUBFYLafKi5LnOjD08fG5B6A9CjxKRC93xCAOv61CkwCdkconbHF61npADKwJDRJpuj76yTbNya5r38ppnoIVO5nhgPe3a0N/ryHazN/unTiNB+kyDrF2DUfB/EmTtEqxAJxXIWhVYY04JkLULsAKd1B9r118NOiU/1i6/CnRKfaxdfRXmpPhYu/gq0DPtHSuPviQ9+irt0Xbx0RfVd6A9+uvio+cXr574X5h5enLoE3ON/4SZFKXfa4UeRLyRuh+F+OS67l7eCf7TaUGm+eO92uad7gXpHwZtEksLXb32Ufms22R51j0bGPuYw2Xt9Hu2WCptnULnHUJ6FhbLHD+LTG9SJP+AJYkxJWzQ24IGabZysvTd+OzoG2d85iS5MAZXpkWej7IfMD67HPTQ/wCWK5HiR8fdCnQuHkuXuXc5rI2EkEEPX5+jrFL5RGCQQ2v9ofERZNEa64dSiz8Ka3HDWteNwHKhxaO5/wc

Browser: all modern browsers
Operating System: Macbook Pro (Retina, 15-inch, Mid 2015)
Problem: Multiple ground polygons with some distance between them results in them occasionally disappearing when the zoom level and/or camera angle change. Doesn't seem to happen unless multiple polygons are added and they are not near each other.

Thanks for including the code example @kayleighbray, that made this really easy to reproduce.
@likangning93 this only happens with ground polygons.  Could you take a look into this?

I also handle a similar problem in my atlas application. When I zoom in parts of the map disappear

Browser: all modern browsers and solo en MAC v 10.14.1

Hmm, that looks like it might be a slightly different issue @andresgiraldo117.  Do you see the same thing happen in our hosted Sandcaslte app? https://cesiumjs.org/Cesium/Build/Apps/Sandcastle/

Hi @hpinkos.
i saw your app sandleCastle and dont hapend this issue.
Thanks for you help.

Thanks @andresgiraldo117. Could you put together a code example for us to reproduce this?

https://groups.google.com/forum/#!searchin/cesium-dev/1.33|sort:date/cesium-dev/U64_PJS3Cuc/4SMV9eW6AQAJ


Turning on hot water doesn’t work

Hi there,
My setup:
Vaillant 105/5 AS S2
Vaillant VWL 127/5 IS
Vaillant sensoNET VR 921
When trying to turn on my hot water, HA displays the correct sensor values but if I'm trying to change the current mode by any automation / manually, the action is being executed without any error handling but the new value is not taken into consideration.
Available values:

Auto
Day
Off

It would be great if the value auto could be set for an automated scheduling of the hot water cycle.
When trying to turn off my hot water (value off), I do not experience any issues.

Please create a new ticket and fill out the bug template, I can't help you without logs.


[Feature Request] - .hyperfile file acting as a launch script

I have an environment that has the same setup.

Open hyper as admin
Split vertically
Split vertically
Split previous horizontally
Split next vertically.
Run series of commands on each
New tab
...etc

This has been the optimal environment for my current workflow and I repeat this process sometimes multiple times a day.
Would be really cool to have .hyperfile file type that can be opened by hyper, allowing scripting these start up events.
For example:
startup.hyperfile
// Starts with no tabs so you initialize the first
const tabOne = hyper.createTab()
const tabTwo = hyper.createTab()

const splitOne = tabOne
const splitTwo = tabOne.splitVertically()
const splitThree = tabOne.splitVertically()
const splitFour = splitTwo.splitHorizontally()

splitOne.runCommand('dir')
splitTwo.runCommand('dir')
splitThree.runCommand('dir')
splitFour.runCommand('dir')

tabTwo.runCommand('npm install')

Then from Windows, I'd just double click the .hyper file and it would go ahead and run the sequence.
Thoughts?

I like the idea! I think this is already possible by creating a plugin that registers with .hyperfile extension, and picks that up, runs it with it's own lib converting it to hyper's redux actions, dispatching them to the hyper store and thus opening tabs etc.


Added solution for 268.

Description of Change
Added a new approach of solution to solve the missing number problem using binary search.

References

Checklist

[x] Added detailed description of changes.
[x] Relevant documentation/comments is changed or added.
[x] Added file name matches File name guidelines.
[x] PR title follows semantic commit guidelines.

Notes:
Using the binary search method is another efficient method of solving this problem.

Thanks for the PR 🚀


Added Sandbox Demo For MultiDropdownList

Added Sandbox Demo For MultiDropdownList - https://codesandbox.io/s/m5wy51xv3p

@divyanshu-rawat Please add SelectedFilters in all codesanbox demos.
And keep the same layout as we have in storybook demos( in this example suggestions list covers the result ).

@bietkul
If I will try to have the storybook demos layout then in small screen where to put MultiDropdownList and result?
Ans react one we also have the same layout -

@divyanshu-rawat Right, keep the same for small screen but change it for large screens.
Check out the react example for larger screens https://z2kw0wx6mm.codesandbox.io/.

@bietkul updated the sandbox kindly check it now 👍


Cannot pull git repo on Gitea 1.16.0 (docker)

Gitea Version
1.16.0 docker
Git Version
No response
Operating System
Ubuntu 20.04
How are you running Gitea?
We've updated our docker-compose Gitea instance to 1.16.0, and since then pulling any repo fails with this error:
sh: 1: /usr/local/bin/gitea: not found
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

I've checked inside the gitea server docker image and /usr/local/bin/gitea is present in the container, so I'm not sure why we're getting this error.

Database
PostgreSQL
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Description
No response
Screenshots
No response

Update: I've tried downgrading Gitea to latest 1.15.x docker image and manually ran UPDATE version SET version=189 WHERE id=1; as well on postgres database, but the issue still persists even on the older version.

You should never do UPDATE version SET version=189 WHERE id=1, you will end up in breaking something. https://docs.gitea.io/en-us/upgrade-from-gitea/
You should fix (update) your docker pass-through https://docs.gitea.io/en-us/install-with-docker/#ssh-container-passthrough

Thanks for the docker pass-through fix. It solved the issue.
It might also be a good idea to add this to the BREAKING section of the 1.16 release notes on GitHub, so others can know that pass-through needs to be updated when updating to 1.16.

The relevant PR is https://github.com/go-gitea/gitea/pull/17846
We've already marked it BREAKING and updated the PR comment. Unfortunately that BREAKING notice didn't make it in to CHANGELOG.md but we'll try to remember to mention in the blog post.
We've already updated the online documentation.


docs: add pasteImage to demo and document the event

The demo currently does nothing when we paste images. This inserts the image inline, so this can be demonstrated.
Addressed typos in Demo.html and README
Changed the link to Squire's source-code to the up-to-date one, so no redirect is required.
By running npm i, it made package-lock.json up-to-date with changes from package.json, i.e. version.
Ran npm run build, which changed files despite no changes being made to src in this PR. :thinking:

Related

Closes https://github.com/fastmail/Squire/issues/425
Related to https://github.com/tutao/tutanota/pull/5275

Please consider reviewing/merging the following PR first:

https://github.com/fastmail/Squire/pull/427

Thanks, much appreciated!


Issue with PUT requests

I've been trying to develop an app to interact with your firmware, so people that don't use any home automation system could still use this to control their shades easily.
I've noticed that most endpoints if not all get blocked with the CORS error if I send an http PUT request. Is there any other way I can get around this?

Which endpoints are you using?  The reason I ask is so I can put preflight responses in for your puts.

v2.0.0 has been released.  Your PUTs should now respond to the preflight request.

v2.0.0 has been released. Your PUTs should now respond to the preflight request.

Fixed 👏


Updated Makefile

Summary

Updated Makefile

Motivation
Removes the PHONY at the top and instead rely on assigning it above where it's used. Setting .PHONY multiple times is valid.
Except for some dependencies in make deps, the same Makefile is now used across most of the Wharf repos.

Requesting these to be added:
lint-proto:
	protolint lint api/wharfapi
lint-fix-proto:
	protolint lint -fix api/wharfapi

and adding them to .PHONY together with the other lint related things.

Requesting these to be added:
lint-proto:
	protolint lint api/workerapi
lint-fix-proto:
	protolint lint -fix api/workerapi

and adding them to .PHONY together with the other lint related things.

Let's do that later, as there may be some linting fixes that I don't want to have to deal with yet


chore: Bump holidays to 0.28

SUMMARY

Upgrade holidays to v0.28.
Set holidays package upper bound to '<1.0'
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[x] Has associated issue: related to but may not fix a root cause of #24619
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Thank you!
@dpgaspar @villebro Can you review it?

Hi @rusackas @john-bodley! Merge it?


Fix fatal errors in streaming

Trying to fix several fatal errors, occurring when the connection breaks while reading. These errors were observed in two locations:

buf.read_line() or buf.read_len() returns None when the connection is closed before completing the read. later  on, this output (None) was considered as a string and the program crashed as a side-effect (with AttributionError).
httplib.IncompleteRead was caught in requests, and propagated through buf.read_line() or buf.read_len(). In this case, the retry mechanism didn't catch HTTP errors so it didn't try to reconnect as expected.

These problems were fixed as follows:

buf.read_line() or buf.read_len() fails with httplib.IncompleteRead when the connection breaks in the middle.
The retry mechanism treats httplib.HTTPError similarly to timeouts and disconnections errors, following Twitter's reconnecting advice.

#576
#595
#734
#780

Resolves #576

Hi @edgardojosec,
Did you run your code before applying this patch? did it work as planned?
Anyway, I can think of one change that may solve this issue. I'll push it shortly.
Cheers,
Ben

Now it also fixes another error, when requests.packages.urllib3.exception.ReadTimeout is raised. requests doesn't re-raise it as Timeout, so our retry mechanism didn't catch it.
With the above fixes (excluding the last one), tweepy ran continuously for a whole week without crashing.

Update: after applying this fix, my app runs for the 6 week in a raw (continuously) without any trouble.

These incomplete reads are still a problem, why hasn't this been merged?

This is stale now and should be resolved with https://github.com/tweepy/tweepy/commit/68e19cc6b9b23d72369ca1520093770eb18a5a9f and https://github.com/tweepy/tweepy/commit/37e701e2193dcf617e7235d3b004f790c5ff784e.

This is stale now and should be resolved with https://github.com/tweepy/tweepy/commit/68e19cc6b9b23d72369ca1520093770eb18a5a9f and https://github.com/tweepy/tweepy/commit/37e701e2193dcf617e7235d3b004f790c5ff784e.


proposed math util API enhancement

Currently we have a quite inconsistent naming in math API, making deciphering some operations quite tedious. Take, for example, Matrix4 .tra() & .trn() (I never know which one to use without checking the param signatures in the docs) - and it's not an isolated case. While I'm in favor of short, 3-4 letter names of operations and the chaining rule here, providing a more descriptive (or even JLS-compliant) naming variant would be helpful here IMO. I'm not saying "change the names", because current names are OK in many cases and changing them would obviously break current implementations - I'm rather in favour of, for example
/** @see #trn() */
Matrix4 transpose() {
    return trn();
}

-ish set of methods, complementing what's already there. Also, providing JLS-standard naming of getXXX/setXXX would also reduce the brain overhead for programmers with Java background.
Note we already have many methods named that way, setToXXX, `setAsXXX``` etc - making the naming more consistent with standard one would be a great improvement and would reduce learning curve for libGDX a lot IMO. It would also make possible for a user (or even libGDX team) to create immutable (or other) variants of those classes for further use - for a programmer using a direct field access is bound to a single implementation of object's data access. Allowing the programmer to choose the scheme himself (either direct field for brevity and speed or set/get to allow abstraction and implementation substitution) would be certainly a Good Thing.

Closing this out.


Update iOS app id to reflect the current app.

With this change, we get as far as printing the message
"APNS: Sending apple push notification to devices" to the
log when a recent TestFlight build of the app is due for
a notification, and then don't hit an exception.  But
on the other hand I still don't get an actual notification
on my phone, so there's still some debugging to do.

Automated message from Dropbox CLA bot
@gnprice, it looks like you've already signed the Dropbox CLA. Thanks!


Apps install does not work but show success message in both winutil.exe tool and Powershell.

Bump, Im having the same issue.

you need to install "winget" https://phoenixnap.com/kb/install-winget

Fixed in PR #94


List of the with 1.3 features added

@idvoretskyi Is this the correct place for this?  Is the process documented somewhere?  I thought this was only for creating issues?

@pwittrock yes, it is - we have decided to use this repo for storing this information as well and agreed on the last two weekly community meetings.

SG.  Plz address to with Derek's comment and squash commits.  I'd like to get the actual docs links in there as a follow up.

@pwittrock done.
@derekwaynecarr thank you, updated.


Hide Category property

Are there any way to hide Category property that links to default Episerver categories?
Scaffolding property does not help. Furthermore, you can't override the property.

That should happen out of the box when installing the package. It doesn't for you?

No, as far as you can see on the screenshot below. I can't hide the Category property from Geta based category.

Do you have any other editor descriptors for CategoryList type in your project?


Withdraw the Ubuntu images

Toolbx now offers built-in support for Ubuntu and these images are maintained as part of the Toolbx project.
https://github.com/toolbx-images/images/issues/81

Let's wait until there is at least a release of toolbox with that change and then I'll delete the repo as well.

then I'll delete the repo as well.

You could also leave the Quay.io repo for a while longer.  Some people might have the URLs in their scripts and shell history and what not.

I'm tempted in making it fail hard fast so that users move to the new location as the old one won't be updated anymore.

I'm tempted to make it fail hard fast so that users move to the new location as the old one won't be updated anymore.

Okay, that makes sense, yes.

Let's wait until there is at least a release of toolbox with that change and then I'll delete the repo as well.

We now have a new Toolbx release that offers built-in support for Ubuntu and it's slowly making it's way to various distributions.

LGTM. I'll wait a few weeks to let the new toolbox land in distros and then we can merge this.

Cool.

Unfortunately looks like the deb distros are far behind: https://repology.org/project/go:github-containers-toolbox/versions

Maybe we can add the full URLs as well in the README for compatibility with older toolbox versions? Then we could probably merge this directly.

Done.
I left the stanza with the new --distro usage, and added a new one to show the alternative with --image for compatibility with older Toolbx versions.

Looks good. The images are not cosign-signed but we can improve that in the toolbox repo workflows.

The images are not cosign-signed but we can improve that in the toolbox repo workflows.

It will be good to have the built-in Arch Linux and Ubuntu images signed.


Create Statistic check/dc in constructor and full-load check/dc modifiers

Closes #4414

This can wait until after the next hotfix. The fix for the elite/weak issue is entirely separate from all of this. But it seems to work, so up to you if you wanna throw it in early.


Redirect user to first group

Closes #100
Needs listMy method, so first merge #84

Current coverage is 96.15% (diff: 96.51%)

Merging #102 into master will decrease coverage by 0.03%

@@             master       #102   diff @@
==========================================
  Files            57         61     +4
  Lines           736        807    +71
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            708        776    +68
- Misses           28         31     +3
  Partials          0          0

Powered by Codecov. Last update fc0452a...3b4a8d1


Build GUI buttons & sliders in meshcat?

Hi Robin,
Thanks for this great project! I'm wondering if it's possible (how much efforts it would require) to build some simple graphical interfaces (buttons, sliders) on meshcat?
I'm thinking about having something similar to Qt's signal-slot mechanism. Let's say we launch meshcat through meshcat-python(or .jl), and we want the users to interact with the backend engine (python/Julia) only through the buttons displayed on the meshcat web-browser.
I don't have much experience in three.js but I am curious what would be the engineering effort to achieve this?
Thanks!

That's a great question, and I'm not sure I have the answer yet. One option I've considered is just to say that meshcat itself is only an output, and the way to add controls to it is to compose them as separate elements on the same page. By that I mean that you would render something like:

This is what we chose to do in RigidBodySim.jl: since MeshCat.jl uses WebIO.jl, which already supports creating widgets and combining them together on the same page, we created a simple GUI consisting of a meshcat window and a few buttons by just rendering them together: https://github.com/JuliaRobotics/RigidBodySim.jl/blob/master/src/visualization.jl#L194-L204 In that case, WebIO handles all the messiness of synchronizing the sliders or buttons to the Julia code that they're supposed to control.
That approach worked pretty well, and I generally like the idea of trying to compose different tools rather than adding more features. The downside, however, is that it only works in Julia. I'm sure something similar could be done in Python, but the widgets and layout would probably be a bit different.
So, I think that if your goal is to have a meshcat window with some controls that link to backend code, then using WebIO.jl widgets (like we did in https://github.com/JuliaRobotics/RigidBodySim.jl/blob/master/src/visualization.jl#L194-L204 ) in Julia or something similar in Python would work really well and be pretty easy (in particular, it requires absolutely no knowledge of three.js at all).
On the other hand, if you want widgets that actually live inside the meshcat viewer, that's more of a challenge. What are you actually interested in doing?

The WebIO.jl widgets seem to fit my needs! I think my pipeline will be very similar to the one used in RigidBodySim.
I will try that and post question in RigidBodySim. Thank you so much for your comments and helps!

Can this be also done on Python?


Fix getting wrong tab when opening more than one window

Fix issue:
When opening more than one window, the extension may get wrong tab.

Nice. Thanks!

I've submitted this update to the Chrome Web Store, so hopefully it will be published in the next couple of days.

This is a small but very helpful tool. Thanks for sharing.


Very low performance during pre-training phase on FC100

Hi Yaoyao,
Thank you for this very interesting work and code.
However, I have an issue with the pre-training phase on FC100, the validation performance is very low: 33%, even after several runs to check with various seeds.
Here is the command I run :
python3 main.py --dataset_dir datasets/FC100/  --phase pre_train --gpu 0
And here is the results I get :

Is the performance supposed to be this low ?
I use pytorch 1.8.1 and torchvision 0.9.1
Kind regards,
Tristan.

Thanks for your interest in our work.
I will check the pre-training code for FC100 later.
You may try to use the pre-training code in this project. I have tested it, and I think it would work.
Best,
Yaoyao


Add CDNJS badge in readme

This will add the badge to show its version on CDNJS and also link to its page on CDNJS!

I signed it!

This should not be on CDNJS. It is an application quick-start not a library to include at-will within your other projects.

Actually, it is already on CDNJS https://cdnjs.com/libraries/web-starter-kit. We still can include it through CDNJS.

Whether it is already there or not is hardly relevant. Having a badge on the project encourages its use. Which we shouldn't since that is not the way the repo is meant to be used/distributed.
It is about keeping usage clear to our end-users who are already confused about what tools apply where and in what context. This just muddies things even further and for something that is clearly an incorrect thing to be distributed using something like CDNJS. We're an application starter repo, not a library to be included at-will.
I understand someone may have added it to CDNJS without understanding the consequences it could have on developers. However, that doesn't mean the project should add a badge supporting that context.

Thank you for your explanation. I got it.


Filter Calculated value

https://www.loom.com/share/a1845ce28e7f459b8e817b09dec0a333

I think the new Column Filters should cover this


The plugin qrscan uses a deprecated version of the Android embedding.

when I'm going to use the plugin I get this msg.
To avoid unexpected runtime failures, or future build failures, try to see if this plugin supports the Android V2 embedding. Otherwise, consider removing it since a future release of Flutter will remove these deprecated APIs.
If you are plugin author, take a look at the docs for migrating the plugin to the V2 embedding: https://flutter.dev/go/android-plugin-migration.
exit code 0

Is this plugin abandoned? There is no update to plugin. Did you guys find any other solutions?

I'm getting the same issue, any solution on this, please!

i need solution!
😱😱😱

You can use https://pub.dev/packages/barcode_scan2

place
tools:replace="android:icon"
in the AndroidManifest.xml  tag

Can you clarify more?
On Thu, Jan 6, 2022, 7:12 PM Md Imam Hossain @.***>
wrote:

place
tools:replace="android:icon"
in the AndroidManifest.xml tag
—
Reply to this email directly, view it on GitHub
https://github.com/leyan95/qrcode_scanner/issues/123#issuecomment-1006758814,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKYLL2YXXNOTEQKK2NIYKDUUXEOFANCNFSM5FXBJW5A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

The android atudio IDE suggested me to do that.
When i use drawable/ic_launcher.png
Instead of mipmap/ic_launcher.png
The IDE throw an error.

This will help https://www.youtube.com/watch?v=w-n0jUrUtPI&list=PLQhQEGkwKZUqZC2QAp_u4ZAzqpsCCRvmM&index=10 https://www.youtube.com/watch?v=QpnVENc9dIA&list=PLQhQEGkwKZUqZC2QAp_u4ZAzqpsCCRvmM&index=11


[Feature Request] Add a way to automate the creation of shells

So the idea is simple, have the constructor accept a hopper input of either a golden apple, or a totem of undying, (thinking 4 gapples or 1 totem?) to generate a new body automatically,

I have a more useful application for hoppers in mind - they can be used to pump items into or out of the the stored shell
Also, there's an obvious flaw in this suggestion - for which player should we generate a shell when a totem of undying is pumped into the shell constructor?
I'm not against the idea itself, but I think there should be a better way to implement such thing

Well, if hoppers are already planned for something else, why not make a new Constructor called the "Automated Constructor" which accepts the provided input, and have it so the player has to link the AutoConstructor with themself,


feat(origin-ui-core): certificates inbox page

When clicking View button on inbox item - detailed view of the certificate is not opened.

Check routing bug for Certificates tab, when logged as Issuer.
Here is what I received as default route:

This bug is still there:

Close button is not working for me. Also there is something wrong with energy data.

Select All behavior is broken if user makes a step away from normal flow and remove select from one of the items:

There are 2 certificates, but the submit button says there are 4

Total volume could be easily broken and provide wrong values affecting the total volume:


Fix IndentationError to get our GitHub Action green

Also,

Update to the current pip and add wheel to speed up the process
Make codespell and requirements.txt mandatory

Thank you very much!


DockerV2: Task ignores addPipelineData and always sets two labels on build

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: DockerV2
https://github.com/microsoft/azure-pipelines-tasks/tree/master/Tasks/DockerV2
Environment

Server - Azure Pipelines

Agent - Hosted:

Image: ubuntu-18.04
Pool: Azure Pipelines

Issue Description
[Include task name(s), screenshots and any other relevant details]
By default the Docker Task (build/push/build and push) sets pipeline metadata as labels. This can be deactivated by setting addPipelineData: false in the pipelines.yml definition.
Unfortunately, even after deactivating this explicitly, two labels remain present in the build result. The labels in question are build.sourceversion and teamfoundationcollectionuri. While I don't have a problem with the former, the name and address of the Project Collection is not something I would like to pass around as public data.
Right now there dont seem to be any way to remove these labels other than executing another docker build via the CLI and overwriting/erasing the unwanted labels.
Pipeline metadata is disabled:
- task: Docker@2
displayName: 'build docker image'
inputs:
  command: build
  Dockerfile: $(workdir)/src/web/Dockerfile
  buildContext: $(workdir)/src/web
  addPipelineData: false

The resulting image metadata contains these two labels:
LABEL com.azure.dev.image.build.sourceversion=....
LABEL com.azure.dev.image.system.teamfoundationcollectionuri=https://dev.azure.com/.../

The issue is that the pipeline task ignores the value of addPipelineData.
Offending line (for teamfoundationcollectionuri):
https://github.com/microsoft/azure-pipelines-tasks/blob/714e68f75c1e56ca699f6514700c5824cd231079/Tasks/Common/docker-common-v2/pipelineutils.ts#L23
The task also sets the build.sourceversion label. This is done in pipelineutils.ts line #31
Task logs
Step 16/17 : LABEL com.azure.dev.image.build.sourceversion=e75b414***************0fdef0388ce7550e
 ---> Running in c557bf5ded0a
Removing intermediate container c557bf5ded0a
 ---> 38ee50360568
Step 17/17 : LABEL com.azure.dev.image.system.teamfoundationcollectionuri=https://dev.azure.com/<Project_Collection_Name>/

Expected behavior
I expect that the Task respects my choice and disabling pipeline metadata actually disables pipeline metadata.

@sonayak to take a look

@ammohant to take a look.

Hey all @ds-ms  @ammohant :)
decided to poke a bit to see if this issue didn't slip through your notification filters. Any chance that this makes it past the PR stage in the next month or so? Regards!

We are still tracking this issue. Would you (@bmarinov) be willing to raise a PR for the same?
PS: @sonayak has left the team

Hi @ds-ms ,
I can take a look the coming days, I may need a bit of hand holding in case I can't get some parts of the setup to run out of the box. Will get back at you when I have something :)

I'd be happy to help you out in fixing this bug.
You can also use this https://github.com/microsoft/azure-pipelines-tasks/pull/13291 PR for reference.

@bmarinov any updates on the PR?

@bmarinov any updates on the PR?

any update on this issue ?
and would it be fixed in the next task version ?

I am also interested in this issue. Why are three labels (system.teamfoundationcollectionuri, build.sourceversion, release.releaseid) not respecting addPipelineData option?

This behavior is inconsistent with the docker@2 task documentation, which states addPipelineData will disable all metadata labels mentioned in the document. Avoiding this seems to be easy as moving those addLabels inside if condition. May other part of the task depends on these labels?

Is there a workaround for this problem?
These labels are causing some security concerns.


autoFlushInterval options of MyBot is too big.

MyBot endpoints are checked frequently. But the cache clear interval is too big. (3600000ms)

Thanks for your contribution!


How we measure memory needed when armnn processing a model?

How we measure memory needed when armnn processing a model?
According to model' batch number, input tensor or number of layers?

Hi @uikilin100 usually the size of the model itself is the best indicator. The amount of working space needed by tensors is usually a lot less than the data required by all the trained parameters.

Hi @MatteoArm So if I have a model with size 10M, the memory android-nn-driver needed will be 1 * model size, 2 * model size or other proportion of model size?

Hi @uikilin100 It'll be 1 * model size + working set size + overhead.
In the test cases I've run, the model size is the biggest (can be lots of megabytes) and the working set size is the smallest (generally the size of the biggest intermediate tensor in your net).
The working set size is affected by the Memory Manager for the backend you are using. The NEON and OpenCL backends use Compute Library which provides a pretty efficient memory manager. The Reference backend doesn't optimise memory at all so will produce big working sets.
The OpenCL backend has the biggest overhead due to compiled OpenCL kernels. I've not seen this exceed a few MB even for models with hundreds of layers.
At the end of the day, you just have to run your model and see, as model complexity affects the results.

Hi @MatthewARM Thanks for the help
I know the best way to know the memory usage is just run the model.
But it's better when we design model, we know how many memory it need in device.
Thanks for the help.
I will close the issue


APEXMALHAR-2111 Projection Operator config changes

Change params to use List instead of comma-separated field names
Mark Operator as Evolving
Add projection operator exception in pom.xml for japicmp-maven-plugin

@chinmaykolhatkar please review.

Looks fine to me. I'll wait for any other comments for couple days and then merge the changes.


bins/tsan/concurrent_connectivity_test

Note that it's a TSAN failure, under epollex, which is not the same as other issues for the same test described in #11383

Also for poll: https://sponge.corp.google.com/target?id=8ac6ad86-ac1e-40ef-b838-d0f37370fcfb&target=github/grpc/c_linux_tsan_native&searchFor=&show=ALL&sortBy=STATUS

There are about 5 different TSAN failures exposed by this test - all of them may or may not have the same root cause. I have a fix for one of the failures (https://github.com/grpc/grpc/pull/13294)
Will triage the rest and reassign to folks.

I can no longer repro this issue and this has not happened in a while now. I am closing it.


[SPARK-39896][SQL] UnwrapCastInBinaryComparison should work when the literal of In/InSet downcast failed

Why are the changes needed?

This PR aims to fix the case
sql("create table t1(a decimal(3, 0)) using parquet")
sql("insert into t1 values(100), (10), (1)")
sql("select * from t1 where a in(100000, 1.00)").show

java.lang.RuntimeException: After applying rule org.apache.spark.sql.catalyst.optimizer.UnwrapCastInBinaryComparison in batch Operator Optimization before Inferring Filters, the structural integrity of the plan is broken.
	at org.apache.spark.sql.errors.QueryExecutionErrors$.structuralIntegrityIsBrokenAfterApplyingRuleError(QueryExecutionErrors.scala:1325)

the rule UnwrapCastInBinaryComparison transforms the expression In to Equals

CAST(a as decimal(12,2)) IN (100000.00,1.00)

OR(
   CAST(a as decimal(12,2)) = 100000.00,
   CAST(a as decimal(12,2)) = 1.00
)

using UnwrapCastInBinaryComparison.unwrapCast() to optimize each EqualTo

// Expression1
CAST(a as decimal(12,2)) = 100000.00 => CAST(a as decimal(12,2)) = 100000.00

// Expression2
CAST(a as decimal(12,2)) = 1.00      => a = 1

return the new unwrapped cast expression In

a IN (100000.00, 1.00)

Before this PR:
the method UnwrapCastInBinaryComparison.unwrapCast() returns the original expression when downcasting to a decimal type fails (the Expression1),returns the original expression if the downcast to the decimal type succeeds (the Expression2), the two expressions have different data type which would break the structural integrity
a IN (100000.00, 1.00)
       |           |
    decimal(12, 2) |
               decimal(3, 0)

After this PR:
the PR transform the downcasting failed expression to falseIfNotNull(fromExp)

((isnull(a) AND null) OR a IN (1.00)

Does this PR introduce any user-facing change?

No, only bug fix.
How was this patch tested?

Unit test.

Can one of the admins verify this patch?

cc @cloud-fan @sunchao

looks much clearer now!

thanks, merging to master/3.3!


Cosmos signature verification and fix

Cosmos code hasn't been updated since 2020. I added a test file for CSDK to check whether the signature verification function works correctly and found that the signing did not work at all.

I fixed it by making CSDKAccount.public_key() directly return a base64-encoded string, which is what is being stored inside the signature dictionary.

Added the verifiy_signature() function as in #19.

Added ecdsa and cosmospy to the testing requirements.

This branch is based on #19; therefore it should be merged after #19.

Regarding encoding bytes with base64: https://github.com/cosmos/cosmos-sdk/pull/12910#discussion_r943984849
Cosmos did, or will do, a transition from base64 to hex bytes, similar to Ethereum

Closing for now, as it is not needed and maintaining our own branch of the cosmos-sdk might not seem worth it.


IGNITE-13878 Fixed Allocated RAM metric for IM region on persistent node.

This PR is associated with a resolved ticket. Closing the PR, please reopen it if you still need it.


Stuck at Atari ROMs download

System: Ubuntu 20.04
conda virtual env , python=3.8
Hello, I ran the AutoROM.py and I can't seem to proceed from there,the terminal stays at the following line:
"I agree to not distribute these ROMs and wish to proceed: [Y/n]: y"
It would be helpful to know if running AutoROM to get those roms require access to certain websites because my network is monitored. I can then try to request access from my IT department.
Kindly assist me with this. Thank you.
python3 AutoROM.py
AutoROM.py:228: UserWarning: ale-py package seems to be empty. Perhaps try reinstalling ale-py.
  warnings.warn(
AutoROM will download the Atari 2600 ROMs.
They will be installed to:
	/home/polaris/anaconda3/envs/py38/lib/python3.8/site-packages/multi_agent_ale_py/roms

Existing ROMs will be overwritten.

I own a license to these Atari 2600 ROMs.
I agree to not distribute these ROMs and wish to proceed: [Y/n]: y

Hi @glitchyordis, thanks for reporting this. For now, could you use autorom v0.4.2? It seems your network may be blocking the torrent link, 0.4.2 downloads the ROMs from a different source which may solve your issue.


JAMES-3894 Antora - update xref, fix not found error

Fix not found  error when build html
 docs git:(master) antora antora-playbook-local.yml

[15:20:23.604] ERROR (asciidoctor): target of xref not found: ../architecture/consistency-model.html#_about_multi_data_center_setups
    file: /home/tungtv/workplace/2_JAMES/1james-project/server/apps/distributed-app/docs/modules/ROOT/pages/configure/cassandra.adoc
    source: /home/tungtv/workplace/2_JAMES/1james-project (branch: master <worktree> | start path: server/apps/distributed-app/docs)
[15:20:23.787] ERROR (asciidoctor): target of xref not found: distributed/operate/metrics.adoc
    file: /home/tungtv/workplace/2_JAMES/1james-project/server/apps/distributed-app/docs/modules/ROOT/pages/configure/mailets.adoc
    source: /home/tungtv/workplace/2_JAMES/1james-project (branch: master <worktree> | start path: server/apps/distributed-app/docs)
[15:20:23.816] ERROR (asciidoctor): target of xref not found: architecture.adoc#_blobstore
    file: /home/tungtv/workplace/2_JAMES/1james-project/server/apps/distributed-app/docs/modules/ROOT/pages/configure/mailrepositorystore.adoc
    source: /home/tungtv/workplace/2_JAMES/1james-project (branch: master <worktree> | start path: server/apps/distributed-app/docs)
[15:20:24.293] ERROR (asciidoctor): target of xref not found: confgure/usersrepository.adoc
    file: /home/tungtv/workplace/2_JAMES/1james-project/server/apps/distributed-app/docs/modules/ROOT/pages/operate/webadmin.adoc
    source: /home/tungtv/workplace/2_JAMES/1james-project (branch: master <worktree> | start path: server/apps/distributed-app/docs)
[15:20:24.323] ERROR (asciidoctor): target of xref not found: architecture.adoc#_recipient_rewrite_tables
    file: /home/tungtv/workplace/2_JAMES/1james-project/server/apps/distributed-app/docs/modules/ROOT/pages/operate/webadmin.adoc
    source: /home/tungtv/workplace/2_JAMES/1james-project (branch: master <worktree> | start path: server/apps/distributed-app/docs)
[15:20:24.325] ERROR (asciidoctor): target of xref not found: architecture.adoc#_recipient_rewrite_tables
    file: /home/tungtv/workplace/2_JAMES/1james-project/server/apps/distributed-app/docs/modules/ROOT/pages/operate/webadmin.adoc
    source: /home/tungtv/workplace/2_JAMES/1james-project (branch: master <worktree> | start path: server/apps/distributed-app/docs)
[15:20:24.326] ERROR (asciidoctor): target of xref not found: architecture.adoc#_recipient_rewrite_tables
    file: /home/tungtv/workplace/2_JAMES/1james-project/server/apps/distributed-app/docs/modules/ROOT/pages/operate/webadmin.adoc
    source: /home/tungtv/workplace/2_JAMES/1james-project (branch: master <worktree> | start path: server/apps/distributed-app/docs)
[15:20:24.328] ERROR (asciidoctor): target of xref not found: architecture.adoc#_recipient_rewrite_tables
    file: /home/tungtv/workplace/2_JAMES/1james-project/server/apps/distributed-app/docs/modules/ROOT/pages/operate/webadmin.adoc
    source: /home/tungtv/workplace/2_JAMES/1james-project (branch: master <worktree> | start path: server/apps/distributed-app/docs)
[15:20:24.331] ERROR (asciidoctor): target of xref not found: architecture.adoc#_recipient_rewrite_tables
    file: /home/tungtv/workplace/2_JAMES/1james-project/server/apps/distributed-app/docs/modules/ROOT/pages/operate/webadmin.adoc
    source: /home/tungtv/workplace/2_JAMES/1james-project (branch: master <worktree> | start path: server/apps/distributed-app/docs)
[15:20:24.332] ERROR (asciidoctor): target of xref not found: architecture.adoc#_recipient_rewrite_tables
    file: /home/tungtv/workplace/2_JAMES/1james-project/server/apps/distributed-app/docs/modules/ROOT/pages/operate/webadmin.adoc
    source: /home/tungtv/workplace/2_JAMES/1james-project (branch: master <worktree> | start path: server/apps/distributed-app/docs)
[15:20:24.333] ERROR (asciidoctor): target of xref not found: architecture.adoc#_recipient_rewrite_tables
    file: /home/tungtv/workplace/2_JAMES/1james-project/server/apps/distributed-app/docs/modules/ROOT/pages/operate/webadmin.adoc
    source: /home/tungtv/workplace/2_JAMES/1james-project (branch: master <worktree> | start path: server/apps/distributed-app/docs)
Site generation complete!

Q: Is this really related to the Antora build for the distributed server randomly fails issue?
For the record: I can access https://james.staged.apache.org/james-distributed-app/3.7.3/architecture/index.html now (not include this fix PR), but I could not before...

Q: Is this really related to the Antora build for the distributed server randomly fails issue?

Not sure, this commit just update xref, fix not found error


Add route timetable api

Added a simpler API for querying the routes time tables and some docs - the point is to have more specific APIs with clear use cases, so that by the end users won't need to be familiar with all the siri and gtfs terms.
Manually tested with the production DB - filters and functionality works as expected. Ran the SQL manually as well.
Performance wise - query per one route takes ~1 sec, explain looks fine.

lon and lat should be float

made some minor fixes:

https://github.com/hasadna/open-bus-stride-api/commit/32bcf20fd98dc7bf3c77a4227f257f5b698acd76
https://github.com/hasadna/open-bus-stride-api/commit/7508aa4ac29c9c06ab88668f5cfa63a73a90b880


Linked plugin folder doesn't show up

I'm creating a Kap plugin.
Followed the guidelines but can't seem to get my plugin listed in the app.
Thanks for a great app, thankful for some input
Platform: Darwin MacBook-Pro.local 17.2.0 Darwin Kernel Version 17.2.0: Fri Sep 29 18:27:05 PDT 2017; root:xnu-4570.20.62~3/RELEASE_X86_64 x86_64
Step to reproduce :  Npm linked my local plugin in the plugins folder like described. Added "plugin-name": "latest" to the to the "dependencies" in the package.json there.
Current behaviour: In the plugin tab, no plugins show up. Only link to "Show the plugins folder"
Platform:
Kap Version: Version 2.0.0 (<IP_ADDRESS>1)

The issue was introduced in https://github.com/wulkano/kap/pull/347/commits/c6ba9380310ac460e348cacd6a0edd0fb482ebf1#diff-d5dad5083988c4f93c867122253af8cdR80 @sallar Why was this added?

@sindresorhus no not a mistake actually. Structure of installed plugins and available plugins is different. One has homepage key directly the other has it under “links”. Previously there was two templates for only this reason, I turned them into one and for this issue I added this || thing.

The way you fixed it (by removing the second part) actually dies on my local.

I missed that. Should be fixed for real now.


Document dev setup on mac and windows

It won't work as well as on linux, but it should be possible to make the tests pass.

I'm using WSL on Windows, and would recommend anyone using this on Windows to do so. I followed the Linux directions and once git and python were installed everything worked smoothly. (I'm new to Python, so sorting out py2/py3 and pip and mkvirtualenvwrapper took some time, but I imagine most users will be familiar with Python ecosystem already).

I'm using WSL on Windows, and would recommend anyone using this on Windows to do so. I followed the Linux directions and once git and python were installed everything worked smoothly. (I'm new to Python, so sorting out py2/py3 and pip and mkvirtualenvwrapper took some time, but I imagine most users will be familiar with Python ecosystem already).

There are instructions on how to install with mac and windows here:
https://quantumai.google/cirq/install#installing_on_windows
https://quantumai.google/cirq/install#installing_on_mac_os_x
I am closing this.  If people have specific issues on installing on these platforms, they can open a new bug about the problem(s) they encountered.


How can I change the default MASTER_PORT in Pytorchjob？

What happened?
Hello everyone! I am trying to change the default MASTER_PORT for distributed communication on pytorchjob because my pod (master) is using a host network (hostNetwork: true) and the default port 23456 is not available.
First, I change the command command in the YAML file, like:
spec:
  containers:
    -name: pytorch
        command:
                        - "torchrun"
                         - "--nproc_per_node=1"
                         - "--nnodes=2"
                       - "--node_rank=1"
                     - "--master_addr=<IP_ADDRESS>"
                       - "--master_port=2057"
                    - "/opt/resnet50/src/resnet50.py"
                      - "--podname"
                    - "resnet-2"
                      - "--sleep-duration"
                    - "0"
But this doesn't work, pod still declares to use port 23456.
As far as I know, The pyTorch operator will inject the address and port of the master pod to the environment variables, so I set the env parameter in the YAML file，like:
env:
    -name: MASTER_PORT
      value: "2057"
This way, when I describe the pod, I can indeed see the values I am set, but there will still have injected values：

Unfortunately, at this point, the pod still declares to use the host's default master port 23456.
What did you expect to happen?
How can I change the default MASTER_PORT in Pytorchjob？Do I need to change the Operator code and compile it? Is there an interface to modify port values？Looking forward to everyone's reply, thank you！
Environment
Kubernetes version:
Client Version: v1.30.2
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.30.2
Training Operator version:
kubeflow/training-operator:v1-9e52eb7#

Add pytorchjob-port in pytorchjob yaml, like this:
apiVersion: "kubeflow.org/v1"
kind: "PyTorchJob"
metadata:
  name: "pytorch-dist-basic-sendrecv"
spec:
  pytorchReplicaSpecs:
    Master:
      replicas: 1
      restartPolicy: OnFailure
      template:
        spec:
          containers:
            - name: pytorch
              image: kubeflow/pytorch-dist-sendrecv-test:latest
              ports:
                - containerPort: 2057
                  name: pytorchjob-port
    Worker:
      replicas: 3
      restartPolicy: OnFailure
      template:
        spec:
          containers:
            - name: pytorch
              image: kubeflow/pytorch-dist-sendrecv-test:latest
              ports:
                - containerPort: 2057
                  name: pytorchjob-port

you can refer to the code here:
https://github.com/kubeflow/training-operator/blob/078a16c96eac50a7032150b8f25d963ae01706cf/pkg/apis/kubeflow.org/v1/pytorch_defaults.go#L31-L36

oh, I got it! Cool！
Thank you for your help，have a nice day~


[Fix] Deleted duplicated images

[Fix] Deleted duplicated images

Se eliminan imagenes duplicadas, se crean archivos xml para referenciar a las imagenes cuando es necesario

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@bef1d1b). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #845   +/-   ##
==========================================
  Coverage           ?   31.77%
==========================================
  Files              ?      324
  Lines              ?    17295
  Branches           ?     1552
==========================================
  Hits               ?     5496
  Misses             ?    11136
  Partials           ?      663

Continue to review full report at Codecov.

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


Fix bug of unknown api version

We found some weird issue from prod, that the apiVersion header got from client is "unknown", which caused NUmberFormatException in code. So this code is to solve the problem.

Not sure if we need to bump 1.9.9 version with this fix, but just leave a pr with this here.


Default isolated install

This PR changes the default UAGENT_ISOLATED_INSTALL to OFF to address https://github.com/eProsima/Micro-XRCE-DDS-Agent/issues/166
Integration tests pass here: https://github.com/eProsima/Micro-XRCE-DDS/pull/51#issuecomment-636822136

Build status:

Linux
Windows

Build status:

Linux
Windows


Calling a super method with the same name and a more generic type throws an error

In the sample code below, I've written a method, to, that accepts a Number and delegates to a super-method that accepts a Number or an Object of Numbers.  If the method names are the same, Flow thinks they are incompatible.  If I change the method names (or delegate to a function rather than a super-method), Flow works as expected.
Test
// @flow

type Point2DT = {
  x?:number,
  y?:number
};

export class Tween {
  // This should accept a Number or a Point2DT
  to(location:number|Point2DT) {
    return location;
  }

  // For comparison's sake, this accepts the same type as its sub-method.
  from(location:number) {
    return location;
  }
}

export class Tween1D extends Tween {
  to(location:number) {
    // This has the same name as the super function, and Flow throws an error.
    return super.to(location);
  }

  toAsSuperMethod(location:number) {
    // This has a different name, and Flow accepts it.
    return super.to(location);
  }

  toAsFunction(location:number) {
    // This calls a function with the same name as the method.  Flow is cool
    // with that too.
    return toAsFunction(location);
  }

  from(location:number) {
    // This has the same name as the super function, but the same type also.
    // Flow doesn't mind.
    return super.from(location);
  }
}

export function toAsFunction(location:number|Point2DT) {
  return location;
}

Results
 10:   to(location:number|Point2DT) {
                   ^^^^^^^^^^^^^^^ object type. This type is incompatible with
 21:   to(location:number) {
                   ^^^^^^ number

Possibly related: #1705

I might be wrong here, but it could seem to me that the actual error you are getting is not the call to the super method, but rather your definition of to in Tween1D. You have broken the type contract you have with the Tween class, as now other pieces of code that accepts a Tween object can no longer call to with anything other than a number, and the type system says that it is allowed.

This is working as designed. @kastermester is right too; the call to super is unrelated.
Here's a smaller repro:
/* @flow */

class A {
  m(x: number|string) {}
}

class B extends A {
  m(x: number) {}
}

function f(a: A) {
  a.m("a string");
}

f(new B);

Since B is a subtype of A, it can be passed to f. Note that f passes a string to m—that's not legal for B. This is why there is an error. The definition of B violates subtyping rules w.r.t. A.

Thanks.  When I opened this, I didn't see the bit about generics.  I think that fixed the issue.

The problem is not with generics but rather the contract between super and sub-classes. When overloading a method in a subclass - for each parameter it may only change parameter types such that a call valid on the super class is also valid on the sub class. Likewise the return type declared on the subclass must be a valid return type in the super class method. What this means is that you are able to "open up" the parameter types - and return a more specific type - and nothing more.

Sorry, I wasn't clear.  I understand why my original post errored.  I didn't realize the proper way to accomplish that was with generics.  Should have closed this earlier.
Thanks again.


[DealabsBridge] Follow the website changes

The website Dealabs did changes in the HTML for the Deal title, so I did the necessary changes to get it in a more generic way.

PS : I did not change the index.php file :p

Merged, thanks for the PR!
You are right, index.php fails due to a broken master (already fixed).
I suggest you rebase your branch on master to reduce the number of commits listed in a PR (for the next time :wink:).


Issue #37 + BTCtürk wallet updates

Add Arbitrum and Optimism to Ahbap #37
Added new BTCtürk wallet options for NGOs

Nope, this's a new .js file. It must be an edit of the .ts file.
And please run the project on your computer to see the changes first.

The code supposedly has the content for #26 and #37. However, @tervelix is not a developer and is unable to properly suggest the PR. Feel free to contribute based on his work.

Developments transferred to #72. Thanks @tervelix!


Don't use ToLower() on replay path & parse online score ID as long

On operating systems with case-sensitive path names like Linux, this will break any replay file that has its path with at least one uppercase character.
Newer online scores have IDs exceeding the Int32 range - parsing them will result in an overflow exception.
This patch should fix those issues.

Well I actually used long for online score ID:
https://github.com/nahkd123/osu-replay-viewer/blob/4b4ae53815b96a635f60a5b4a09380e86adb690b/osu-replay-viewer/OsuGameRecorder.cs#L83-L87
The part where you changed from int to long is for local score ID, which is 32-bit integer.

My bad. Fixed it.


RE: Issue #117

Re: Issue #117; working example featuring ndb.StructuredProperty, ndb.LocalStructuredProperty, @EndpointsAliasProperty, and property-attribute validator=.
Also include GAE currently required app.yaml & appengine_config.py entries.
cc/ @dhermes

Per Google's insistence, I have signed something.


GHA: Install and Build also on macOS and test in LTS Node.js versions

see subject :-)

I removed the windows tests for now tikl checked ... somehow build tool has issues on windows


Update algonaut_algod::ext::transaction::Transaction

Mainly add 'dt' and 'apbx' fields.
References:

https://github.com/algorand/go-algorand/blob/75bb6a90320c3eb924c482378663cb8dd451d123/data/transactions/transaction.go#L106
https://github.com/algorand/go-algorand/blob/75bb6a90320c3eb924c482378663cb8dd451d123/data/transactions/signedtxn.go#L32

Hi @etzellux! Thank you for your PR! This crate contains auto-generated code (with a few fixes) starting from the open api specs of the algod api. Would you mind trying to run the code in the code in the README of this crate and check the diff?

Hi @manuelmauro!
I updated the algod.oas3.yml file and followed the instructions.
It produced this:
pub struct GetBlock200Response {
    /// Block header data.
    #[serde(rename = "block")]
    pub block: serde_json::Value,
    /// Optional certificate object. This is only included when the format is set to message pack.
    #[serde(rename = "cert", skip_serializing_if = "Option::is_none")]
    pub cert: Option<serde_json::Value>,
}


Invisible captcha giving timeout on Promise.resolve(observer)

As per your instructions we are doing:
recaptcha.execute()
Promise.resolve(observer)

But observer seems to hang when trying to resolve its promise. We cant seem to get past this.

@gregg-cbs
I'm facing the same issue, did you find a solution?

Yes I did, the docs are very wrong. This should help you.
<script lang="ts">
  import { Recaptcha, recaptcha } from 'svelte-recaptcha-v2';

  let loading = false;
  let init_recaptcha = false;

  // Put this inside an env file.
  const RECAPTCHA_SITE_KEY = 'your recapthca key';

  // to know if captcha is successfully initiated
  const onCaptchaReady = (event: any) => {
    init_recaptcha = true;
  };

  // this happens automatically after form submit
  const onCaptchaSuccess = async (event: any) => {
    loading = true;

    if (event.detail.token) {
      await onSubmit();
    }

    loading = false;
  };

  // if captcha fails to init
  const onCaptchaError = (event: any) => {
    init_recaptcha = false;
    // log a message to yourself to know
  };

  function onCaptchaSubmit() {
    loading = true;
    // over here we are bypassing captcha if it failed to init
    // so at least users can still submit the form
    if (init_recaptcha) {
      recaptcha.execute();
    } else {
      // log to yourself to let you know captcha is failing

      // submit the form in any case
      onSubmit();
    }

    loading = false;
  }

  async function onSubmit() {
    // submit your form
  }
</script>
</script>

<Recaptcha
  class="self-center"
  sitekey={RECAPTCHA_SITE_KEY}
  badge={'top'}
  size="invisible"
  on:success={onCaptchaSuccess}
  on:error={onCaptchaError}
  on:ready={onCaptchaReady}
/>


Broken link on the npmjs.com

Aloha! :-)
You have renamed the repo and currently, links on https://www.npmjs.com/package/typescript-style are not working. It's not a issue, but it's a little note - your npm package had topic position on google queries ))

Thanks for letting me know about this. 🌷
Now this repo contains only guidelines, so maybe removing it from npm is the more reasonable approach. :)


Installation

Hi,
I'm new to Splunk and I have been trying to collect some info about user agents from my web server and by chance, I found your python script which looks very promising. Since there is no instruction about how to install and I'm totally new to all this, I feel lost. Could you please guide me?

Following http://docs.splunk.com/Documentation/Splunk/6.5.2/Search/Customcommandlocation should work
It just needs to go in a apps/whatever/bin/ and then apps/whatever/local/commands.conf

[ua2os]
chunked = true
filename = ua2os.py

Thanks a lot for the help and sorry for the noob questions. It was indeed easy to install. I just had to follow your instructions and ua2os plugin was among the plugins in Splunk.
I've managed to make to visible and now it's just below the search plugin. When I click on it, I get the search bar, just as if I clicked on the search and reporting plugin. I'm not sure how it works. I did a quick query:
host=node1 useragent="*" | stats count by useragent
I actually get all the user agents in that specific host but I still get:
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Firefox/45.0
What am I missing here?

Oh.. it sounds like you created a ua2os app instead of just adding it as a command to an existing app.  The way it's used is by doing a
host=node1 useragent="*" | ua2os | stats count by browser

Ah I understand. OK, I tried to add it to the search & reporting app. I've copied ua2os.py to
/opt/splunk/etc/apps/search/bin
There is no "local" directory inside search & reporting directory but the "commands.conf" exists under the "default" directory. So, I've modified it and added the lines you suggested and restarted splunk.
When I use
host=node1 useragent="*" | ua2os | stats count by browser
It hangs there saying "parsing job....". I also have a warning in the splunk log
"04-04-2017 10:52:45.863 +0200 WARN  InstalledFilesHashChecker - An installed file="/opt/splunk/etc/apps/search/default/commands.conf" did not pass hash-checking due to reason="content mismatch"
Is it because of the warning the search doesn't work?

I've already tried that. The problem is that when I run the query, I get "Parsing job..." and it stays there. Anyways, Thanks a lot for your time and efforts.


[instrumentation-python] context attributes propagation for mistralai

Closed by #420

Closed by https://github.com/Arize-ai/openinference/pull/420


Favicon support ?

Hi,
I'm not sure if this API is still being updated, if it is, can you add a favicon (server-icon) support or explain me how to do it.
It would help me a lot, thank you.

Currently mcstat supports two styles of server list ping -- legacy (pre-1.7) and 1.7 (and greater). To use the 1.7 server list ping, pass true to MinecraftStatus::ping(). The image is presented as a data uri and available at $reply['json']['favicon']. I think most php functions that read from URLs and files can read from data uris.
Here is a short snippet to illustrate how to do it. (View the file hypixel-favicon.png once you run the snippet.):
<?php
$m = new MinecraftStatus('mc.hypixel.net');
$reply = $m->ping(false);
$img = file_get_contents($reply['json']['favicon']);
file_put_contents('hypixel-favicon.png', $img);

I usually try to update mcstat when significant changes are made to the protocol, but Mojang has been good about this, with only one protocol breaking change.
More info:

Server List Ping
data uri on wikipedia
data:// php docs