Fixes equality checking for Map objects.

Currently:
// keys are not checked
const equal = require('./index.js')
const a = new Map([['a', 1]])
const b = new Map([['b', 1]])
equal(a, b) // true
// recursive checking for values doesn't work, either
const x = new Map([['a', [1, 2]]])
const y = new Map([['a', [2, 1]]])
equal(x, y) // true
// empty maps equal those with data b/c the key set is empty and the check falls all the way though the ifs
const z = new Map()
equal(a, z) // true

While Maps don't have good browser support at this time, they are useful in Node, and I've found that I want tape's deepEqual, deepLooseEqual, and friends to work properly with Maps when testing Node applications.
The implementation is a near duplication of the Object handling code, but deduplicating that didn't seem straightforward or all that beneficial to me at the time.
I also made a few small changes to make the style more consistent, and added a .gitignore, but tried to not overreach or get too out of scope. I can revert these changes upon request.

Updated description to include more details of the issue this addresses.
Updated the Travis config so I could get a nice green checkmark.

I'd also like to see this feature. Can anyone review it?


plot_images_and_locs error on SDSS sleep training for LocationEncoder

After #523, plot_images_and_locs requires that the estimated catalog contains the attributed galaxy_bools, which is not present when checking the performance of the LocationEncoder.
This raises two issues. First, plotting with the validation_epoch_end method isn't being tested on CI. Not sure if there's an easy fix for this or not.
Second, plot_images_and_locs() is doing too many things at once and growing overly complex as a result. For the purposes of training plots, it should take just a few lines of code to make each plot (plotting an image with imshow() and overlaying locations with scatter()). I propose we write tailored plotting code for each training method inside validation_epoch_end(), and the current plot_images_and_locs be moved to case_studies/sdss_galaxies.
Stack trace:
Epoch 4: : 100it [00:42,  2.37it/s,Error executing job with overrides: ['mode=train', 'training=sdss_sleep', 'training.save_top_k=1', 'training.version=sdss_sleep']
Traceback (most recent call last):]
  File "./main.py", line 20, in main
    train(cfg)
  File "/home/dereklh/projects/bliss2/bliss/train.py", line 51, in train
    trainer.fit(model, datamodule=dataset)
  File "/home/dereklh/.cache/pypoetry/virtualenvs/bliss-R7H3wSR3-py3.8/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 740, in fit
    self._call_and_handle_interrupt(
  File "/home/dereklh/.cache/pypoetry/virtualenvs/bliss-R7H3wSR3-py3.8/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 685, in _call_and_handle_interrupt
    return trainer_fn(*args, **kwargs)
  File "/home/dereklh/.cache/pypoetry/virtualenvs/bliss-R7H3wSR3-py3.8/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 777, in _fit_impl
    self._run(model, ckpt_path=ckpt_path)
  File "/home/dereklh/.cache/pypoetry/virtualenvs/bliss-R7H3wSR3-py3.8/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1199, in _run
    self._dispatch()
  File "/home/dereklh/.cache/pypoetry/virtualenvs/bliss-R7H3wSR3-py3.8/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1279, in _dispatch
    self.training_type_plugin.start_training(self)
  File "/home/dereklh/.cache/pypoetry/virtualenvs/bliss-R7H3wSR3-py3.8/lib/python3.8/site-packages/pytorch_lightning/plugins/training_type/training_type_plugin.py", line 202, in start_training
    self._results = trainer.run_stage()
  File "/home/dereklh/.cache/pypoetry/virtualenvs/bliss-R7H3wSR3-py3.8/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1289, in run_stage
    return self._run_train()
  File "/home/dereklh/.cache/pypoetry/virtualenvs/bliss-R7H3wSR3-py3.8/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1319, in _run_train
    self.fit_loop.run()
  File "/home/dereklh/.cache/pypoetry/virtualenvs/bliss-R7H3wSR3-py3.8/lib/python3.8/site-packages/pytorch_lightning/loops/base.py", line 145, in run
    self.advance(*args, **kwargs)
  File "/home/dereklh/.cache/pypoetry/virtualenvs/bliss-R7H3wSR3-py3.8/lib/python3.8/site-packages/pytorch_lightning/loops/fit_loop.py", line 234, in advance
    self.epoch_loop.run(data_fetcher)
  File "/home/dereklh/.cache/pypoetry/virtualenvs/bliss-R7H3wSR3-py3.8/lib/python3.8/site-packages/pytorch_lightning/loops/base.py", line 146, in run
    self.on_advance_end()
  File "/home/dereklh/.cache/pypoetry/virtualenvs/bliss-R7H3wSR3-py3.8/lib/python3.8/site-packages/pytorch_lightning/loops/epoch/training_epoch_loop.py", line 242, in on_advance_end
    self._run_validation()
  File "/home/dereklh/.cache/pypoetry/virtualenvs/bliss-R7H3wSR3-py3.8/lib/python3.8/site-packages/pytorch_lightning/loops/epoch/training_epoch_loop.py", line 337, in _run_validation
    self.val_loop.run()
  File "/home/dereklh/.cache/pypoetry/virtualenvs/bliss-R7H3wSR3-py3.8/lib/python3.8/site-packages/pytorch_lightning/loops/base.py", line 151, in run
    output = self.on_run_end()
  File "/home/dereklh/.cache/pypoetry/virtualenvs/bliss-R7H3wSR3-py3.8/lib/python3.8/site-packages/pytorch_lightning/loops/dataloader/evaluation_loop.py", line 131, in on_run_end
    self._evaluation_epoch_end(outputs)
  File "/home/dereklh/.cache/pypoetry/virtualenvs/bliss-R7H3wSR3-py3.8/lib/python3.8/site-packages/pytorch_lightning/loops/dataloader/evaluation_loop.py", line 236, in _evaluation_epoch_end
    model.validation_epoch_end(outputs)
  File "/home/dereklh/projects/bliss2/bliss/sleep.py", line 312, in validation_epoch_end
    self._make_plots(outputs[-1], kind="validation")
  File "/home/dereklh/projects/bliss2/bliss/sleep.py", line 362, in _make_plots
    plot_image_and_locs(fig, axes[i], i, batch["images"], bp, truth, est, labels=labels)
  File "/home/dereklh/projects/bliss2/bliss/reporting.py", line 630, in plot_image_and_locs
    gbools = estimate["galaxy_bools"].float().cpu().numpy().reshape(-1)
  File "/home/dereklh/projects/bliss2/bliss/catalog.py", line 226, in __getitem__
    return super().__getitem__(key)
  File<EMAIL_ADDRESS>line 1010, in __getitem__
    raise KeyError(key)
KeyError: 'galaxy_bools'

Since I'm already working on refactoring the LocationEncoder in #536, I'll replace the plot_images_and_locs() as a proof-of-concept for my idea.

Thanks @dereklhansen - I agree that it's getting complicated. I like your idea to simplify the overall plotting code

This raises two issues. First, plotting with the validation_epoch_end method isn't being tested on CI. Not sure if there's an easy fix for this or not.

I think this should be easy, I can work on testing it after you merge your PR


VimuxRunCommand just echoes (some of) the command into the current buffer

My setup:

wezterm in WSL2
tmux 3.0a
neovim 0.10.0

When I call :call VimuxRunCommand("echo 'foobar'"), I get o 'foobar' printed where my cursor was in the buffer. If I use vim-test with strategy vimux within a Go test, I get the following printed in my buffer:
ear; echo -e 'go test -run '\''TestSomething$'\'' ./dir'; go test -run 'TestSomething$' ./dir

In both cases, no tmux split is created. How can I debug this further and fix it?

Same here with same setup except of Windows Terminal instead of wezterm.


Using params arrays makes the usage easier

MSDN: https://msdn.microsoft.com/en-us/library/w5zay9db.aspx
I would use the params string[] in the given methods to make usage easier.
E.g.
New:
app.UseSwaggerUi(c =>
{
c.SupportedSubmitMethods("get","post","delete");
});
instead of:
var options = new[]{ "post", "put", "delete", "patch" };
app.UseSwaggerUi(c =>
{
c.SupportedSubmitMethods(options);
});

Otherwise I would recommend using IEnumerable as parameter instead of using a strongly typed array.

Looks good, I'm in agreement - could you update the readme too for completeness. Thanks


JSONSchemaType incorrectly parsing generic type as never

What version of Ajv are you using? Does the issue happen if you use the latest version?
8.12.0, yes
Ajv options object
{}

import {JSONSchemaType} from "ajv"

const toSchema = <T extends Record<string, any>>(properties: { [key: string]: any}): JSONSchemaType<T> => {
  return {
    type: 'object',
    properties,
    required: []
  }
}

JSON Schema
NA
Sample data
NA
Your code
TS Playground
import {JSONSchemaType} from "ajv"

const toSchema = <T extends Record<string, any>>(properties: { [key: string]: any}): JSONSchemaType<T> => {
  return {
    type: 'object',
    properties,
    required: []
  }
}

Validation result, data AFTER validation, error messages
Type '{ type: "object"; properties: { [key: string]: any; }; required: never[]; }' is not assignable to type 'UncheckedJSONSchemaType<T, false>'.
  Object literal may only specify known properties, and 'type' does not exist in type 'never'.

What results did you expect?
I expected the type to compile without issue. It looks like the UncheckedJSONSchemaType is not correctly parsing the various if/else statements, so it ends up falling back to never. The error messaging here is pretty bad, but even when explicitly extending the generic from Record ajv fails, which is surprising.
Are you going to resolve the issue?
Maybe

Not sure if “any” is supported - it’s almost never needed, when you do data validation…

What is the correct way to construct generics like the function above? If you remove the explicit record extension it still fails with the same error


chore: test

Images optimized ✨

File
Before
After
Savings

github.png
41.47 KB
27.65 KB
33%

Total:
41.47 KB
27.65 KB
33%


More slots

It would be great to expose more slots for custom styling. For example:

filter button (so you can use different icons etc)
filter input (so you can match the size of my custom button by using my own input component)
pagination buttons

Thanks!

Wrong repo


⚠ Bofrost.ch has degraded performance

In 4b1dd90, Bofrost.ch (https://www.bofrost.ch) experienced degraded performance:

HTTP code: 200
Response time: 5032 ms

Resolved: Bofrost.ch performance has improved in 850e0ec after 6 minutes.


[FEAT] - checksum when deserializing the ledger snapshot file

Internal/External
Internal
Describe the feature you'd like
The deserialization of the ledger snapshot file should fail if what was read from disk differs from what was written to disk (eg disk corruption flipped a bit).
Describe alternatives you've considered
This requested feature is not strictly necessary: some interaction with the rest of the network will (eventually) fail if the node has the wrong ledger state. But it could be arbitrarily later, which would be especially confusing. Also, this would limit one of the ubiquitous "possible corruption" explanations for failures (such as VRFKeyBadProof).
Limitation: the checksum would not ensure that the ledger state was serialized to disk correctly---only that the bitstream written to disk was the same read from disk.

@jorisdral had some suggestions about how some utilities from fs-api might help here?

We could use the file read/write functions from System.FS.CRC. They are used by the other storage components already, e.g.,
https://github.com/IntersectMBO/ouroboros-consensus/blob/706c4b162072a4ccf4d484007130f080f5c1a183/ouroboros-consensus/src/ouroboros-consensus/Ouroboros/Consensus/Storage/ImmutableDB/Impl.hs#L500
Now that we're talking about checksums, though it is orthogonal to this issue, the System.FS.CRC module is consensus specific and should probably be moved to consensus instead.


.exe file icon

When working with C, I need to compile C programs to a .exe file to run them. An icon for .exe files would be very helpful in differentiating different files.


fix(FloatingFocusManager): wait for tree nodes context to be updated

This might be a weird one. I'm not exactly sure if this sort of usage is supported or not, but it seemed like a bug. The test resembles this a little bit, but not too much. I'll add a bigger backstory in a details panel.
Anyway, I found that the getChildren function call in movedToUnrelatedNode does not return any children, because it does not think the child is open yet, even though opening the child is what triggered the focusout in the first place. Seems like it is necessary to wait for this layout effect to have run before actually running the check. I'm not sure if the timeout is the best way to do this, however, I'm always reluctant to add such timeouts. Maybe the getChildren should skip the open check in that case instead? Either way, it worked for this test and did not seem to break any others, so...
WDYT?

My usecase
I have this usecase where:

a dropdown menu item opens a confirmation dialog
when the action is cancelled, focus should be returned to the item
when the action is confirmed, the dropdown is closed and focus returns to the dropdown button

Now, in order to keep the dropdown menu open, so it would not close via focusout events et al, I need to render the modal inside of the dropdown's FloatingNode. I can't render it it in the dropdown menu (that is portalled), as both the dropdown menu and modal have a transition applied to them, and when nesting in the menu, the transitions don't work as expected.
It looks something like this in practice:
export const DropdownTemplate = (args) => {
  const [status, setStatus] = React.useState('Waiting for action press');
  const { pending, proceed, cancel, confirm } = useConfirmation();
  const actionRef = React.useRef(null);
  const handleAction = React.useCallback(async () => {
    setStatus('Waiting for confirmation');
    const confirmed = await confirm();
    if (confirmed) {
      setStatus('Confirmed, doing something');
      const somethingResult = await doSomething();
      if (somethingResult) {
        setStatus('Something is done, waiting for action press');
      } else {
        setStatus('Something is not done, waiting for action press');
      }
      return somethingResult;
    } else {
      setStatus('Cancelled, waiting for action press');
    }
    return confirmed;
  }, [confirm]);
  return (
    <Stack alignItems="center" gap="medium">
      <Dropdown>
        <DropdownButton iconStart="small-more" iconEnd="small-chevron-down" size="small">
          Actions
        </DropdownButton>
        <DropdownMenu>
          <DropdownMenuItem ref={actionRef} onClick={handleAction}>
            Do something
          </DropdownMenuItem>
        </DropdownMenu>
        <AlertModal {...args} trigger={actionRef.current} open={pending} onClose={cancel}>
          <Button onClick={cancel} kind="subtle">
            Cancel
          </Button>
          <Button onClick={proceed} kind="primary">
            Confirm
          </Button>
        </AlertModal>
      </Dropdown>
      <Text>{status}</Text>
    </Stack>
  );
};

I think allowing getChildren to check open or not would be cleaner here as you said. Can't 100% remember when it should be checked though.

I think allowing getChildren to check open or not would be cleaner here as you said. Can't 100% remember when it should be checked though.

I tested this out. If I skip the open check in getChildren, it will return the children. But then the next contains checks fail, because the refs are not updated yet. So those checks also need the tree nodes contexts to be updated? Any other ideas I could try here?

even though opening the child is what triggered the focusout in the first place

Shouldn't relatedTarget checks handle this?

Shouldn't relatedTarget checks handle this?

Not sure what you mean by this? The issue I ran into is that one of the relatedTarget checks (the one using getChildren) fails because the tree nodes context data has not updated yet.

Oh ok 😅  I guess the setTimeout thing works even if not pretty (unless VoiceOver starts glitching for some reason, which is annoyingly possible :\)
I wonder if useInsertionEffect (and directly mutating in render for React 17) works instead?

Maybe a microtask or an animation frame instead of a timeout would be more correct?

I wonder if useInsertionEffect (and directly mutating in render for React 17) works instead?

useInsertionEffect does seem to work, but directly mutating in render for React 17 unfortunately does not.

Maybe a microtask or an animation frame instead of a timeout would be more correct?

queueMicrotask works as well instead of the timeout. I don't really know the specific differences of them, you tell me what should be used :)

Let's go with queueMicrotask - it works in React 17 (ReactDOM.render(...)) too?

I'd say this is fine, gonna merge


showing resources of first category instead of showing "welcome to website section"

@jayk-gupta pls review my PR
I had done the changes for website section.

@jayk-gupta pls review my PR I had done the changes for website section.

Thanks for your contribution Swetha!
It now much better.
You can do it for all the sections.

Sure I'll do it


Select+Copy (shortcut) doesn't work

Maybe I'm missing something, but this would be quite useful

Going to re-purpose this as "select + copy doesn't work" and tag it to make sure it's properly addressed in AF3.
Sorry for the noise 4 years later! :)

You can now select, copy and paste in the new release, and this uses the clipboard so you can copy in and out of asciiflow.


Update the package 'code_transformers' to '>=0.4.2 <0.5.0' version.

'0.4.*' version support 'useSharedSources' for significant speed increase

See also #232, #233

Ok, close dup.


Update manifests/t/Tencent/TencentVideo/11.53.8129.0/Tencent.TencentVideo.locale.en-US.yaml

Fixes an issue where this package has multiple different monikers
Microsoft Reviewers: Open in CodeFlow

[Policy] Reset Labels


Add endpoint to enable proxy

I'm thinking it might be useful to run a server that checks deps and provides endpoint..that way dependants outside the network can use the server as a proxy..I'm almost certainly getting carried way on this one!


Update ELI5 page

/botio build

From: MushinBot (botio-academy.horizen.global)

Received
Command cmd_build from @tuanhvdev received. Current queue size: 0
Live output at: https://botio-academy.horizen.global/13d2318ed5b0b06/output.txt

From: MushinBot (botio-academy.horizen.global)

Success
Full output at https://botio-academy.horizen.global/13d2318ed5b0b06/output.txt
Total script time: 2.46 mins
You can download the generated site at: https://botio-academy.horizen.global/latest.tar.gz
Or view a preview at https://devweb-academy.horizen.global

/botio build

From: MushinBot (botio-academy.horizen.global)

Received
Command cmd_build from @tuanhvdev received. Current queue size: 0
Live output at: https://botio-academy.horizen.global/0bbcf7ca964564a/output.txt

From: MushinBot (botio-academy.horizen.global)

Success
Full output at https://botio-academy.horizen.global/0bbcf7ca964564a/output.txt
Total script time: 2.57 mins
You can download the generated site at: https://botio-academy.horizen.global/latest.tar.gz
Or view a preview at https://devweb-academy.horizen.global

please don't merge yet... need a go from marketing to release video.


[dartdev][flutter_tools] Rerunning commands needs to be a NOP for assets if hooks were cached

After the hooks are run in dartdev/flutter_tools, the assets might be copied/signed/install_name modified etc.
We'd like to ensure that we can cache those steps.
We cannot blindly rely on timestamps, as a hook might return a file with an older timestamp (on Windows downloaded files have their creation date as lastmodified, not the downloaded time). We could fix this by mandating that assets should have monotonically increasing timestamps, but that puts burden on the hook writer.
Maybe there are better solutions (using a hash, ...).
Also flutter run and flutter build --release copy files both to build/native_assets/* which also breaks the NOP-ness.
To investigate

[ ] What does flutter_tools currently do? (Possibly content hashes https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/build_system/build_system.dart#L1174)

Context:

https://github.com/dart-lang/native/pull/1533#issuecomment-2338477561
https://dart-review.googlesource.com/c/sdk/+/381580/5/pkg/dartdev/lib/src/native_assets_bundling.dart

This needs to play well with concurrent invocations as well:

https://github.com/dart-lang/native/issues/1534

Edit:

We probably want to align the hook caching (https://github.com/dart-lang/native/issues/1593) with whatever dartdev and flutter_tools are doing with the results of the hooks (this issue).

Thanks @mkustermann and @blaugold

This is working for flutter_tools, which consumes the BuildOutput.dependencies and does it's own file-content-hashing based tracking of whether those files change.
For Dartdev we have a tracking issue:

[ ] https://github.com/dart-lang/sdk/issues/59668


/deliver or /deliverdeath sometimes fail to send items

Sometimes when sending items to a player's /present inventory by use of the /deliver or /deliverdeath commands the items don't arrive.

Duplicate of #514


Close memory object streams at exit

This avoids the ResourceWarning about unclosed memory object streams, introduced in AnyIO 4.4.0.

Perfect 😊

@all-contributors add @agronholm for bug, code


Use dedicated service account for machine controller pod

The current code is using default service account combined an admin privileged kubeconfig. We need to strip down to appropriate access permission with a dedicated service account.

This is very important, but it seems that it can be added as alpha without major changes to the system, so moving it to beta.

This issue was moved to kubernetes-sigs/cluster-api#92


[WIP] Refer Screen

What does this PR?
Work In Progress on Refer Screen
Issue number
#2182
Screenshots/Video
[WIP]

@noumantahir need further instructions on this to process further.

Added Basic Refer Screen UI with referred users list
Implemented referrals API with converter and referral interface type defined

See the screenshot below. Currently using good-karma username for testing . I have no account with referrals.

@noumantahir please check the video below for latest update.
https://user-images.githubusercontent.com/48380998/154622736-95534bd2-2f0e-4af3-bcd0-6f4c09a2437b.mp4

@aliseyalvi make sure to test the dark mode..

@noumantahir check the update onwards https://github.com/ecency/ecency-mobile/pull/2188/commits/27675be7513d17b4b66abf5a7eda9a972ef5e46d.

added referrals flatlist pagination.
integrated referrals stats endpoint
added flatlist refreshing
fixed autofocus issue when navigating to delegation screen.

@feruzm please review demo video as well so we can remove the test username and replace with real one.
Pending Item: Opening user profile on item tap.
https://user-images.githubusercontent.com/6298342/154961004-9118c666-c072-4ad0-8468-ff23791f1026.mov

wallet branch is now being built in app centre, beta should be available as build is complete

@noumantahir

added navigate to user profile on list item press https://github.com/ecency/ecency-mobile/pull/2188/commits/f23fb3e8fd5c3c25b09e712e60ae43759f9c9e14

@feruzm seems all good from my side, will be great if you could also have a quick look...

@noumantahir looks good, can we change earned text to Earned Points to keep consistency on wording?

@feruzm done


How to get your BLEU score?

Could you please help me replicate the reported scores? If I follow your instructions I don't get the same scores.
What I did:
git clone https://github.com/jungokasai/deep-shallow/
cd deep-shallow

pip install gdown

mkdir -p data

# get model
gdown 'https://drive.google.com/uc?id=1x_G2cjvM1nW5hjAB8-vWxRqtQTlmIaQU'
gdown 'https://drive.google.com/uc?id=1mNufoynJ9-Zy1kJh2TA_lHm2squji0i9'
tar -xvzf trans_ende-dist_12-1_0.2.tar.gz
tar -xvzf wmt16.en-de.deep-shallow.dist.tar.gz

# Reported BLEU Score: 28.3

python generate.py wmt16.en-de.deep-shallow.dist/data-bin/ --path trans_ende-dist_12-1_0.2/checkpoint_best.pt --beam 5 --remove-bpe  --lenpen 1.0 --max-sentences 10
# [...]
# Generate test with beam=5: BLEU4 = 27.37, 58.7/33.2/21.0/13.7 (BP=1.000, ratio=1.002, syslen=63193, reflen=63078)

python generate.py wmt16.en-de.deep-shallow.dist/data-bin/ --path trans_ende-dist_12-1_0.2/checkpoint_top5_average.pt --beam 5 --remove-bpe  --lenpen 1.0 --max-sentences 10
# [...]
# Generate test with beam=5: BLEU4 = 27.66, 59.0/33.5/21.2/14.0 (BP=1.000, ratio=1.004, syslen=63331, reflen=63078)

python generate.py wmt16.en-de.deep-shallow.dist/data-bin/ --path trans_ende-dist_12-1_0.2/checkpoint_last.pt --beam 5 --remove-bpe  --lenpen 1.0 --max-sentences 10
# [...]
# Generate test with beam=5: BLEU4 = 27.67, 59.2/33.6/21.2/13.9 (BP=1.000, ratio=1.000, syslen=63052, reflen=63078)

So it appears that checkpoint_last.pt gets the best score and not checkpoint_best.pt, but it's still below the advertised score.
What am I doing wrong?
Thank you!

Yes, I chose checkpoint_top5_average.pt as explained in the paper. For BLEU calculation, we did compound splitting, following prior work, including Vaswani et al. 2017. It typically yields a 0.5+ BLEU improvement. The compound split function is available here.

Thank you very much for sharing these details and the links, @jungokasai!

Hi @jungokasai,
I noticed on the German side, the Europarl corpus in the data you preprocessed (wmt16.en-de.deep-shallow.dist.tar.gz) is different to what the preprocessed data from the link: https://google.github.io/seq2seq/nmt/ and the original WMT shared task (this link). Actually you can easily see this when open your file and the original training-parallel-europarl-v7.tgz.
Do you know why this happen? Thanks a lot.
Best,

Hi @jungokasai,
I noticed on the German side, the Europarl corpus in the data you preprocessed (wmt16.en-de.deep-shallow.dist.tar.gz) is different to what the preprocessed data from the link: https://google.github.io/seq2seq/nmt/ and the original WMT shared task (this link). Actually you can easily see this when open your file and the original training-parallel-europarl-v7.tgz.
Do you know why this happen? Thanks a lot.
Best,

Sorry if I'm misunderstanding you, but it should be different because that data is the result of knowledge distillation from a transformer large model.

Sorry if I'm misunderstanding you, but it should be different because that data is the result of knowledge distillation from a transformer large model.


Support point in time searches

This PR adds initial support to allow searching over a point in time. It adds a representation of point in time for use in the request and a point in time ID to the response.
During testing, we identified that the current URL resolution for searches against all indices preferred using the _all/_search path. This causes issues with point in time which expects no indices to be specified. I've added logic to ApiUrls which detects when we are searching with _all as the path parameter. It then prefers a match against the root /search URL instead. This required a few test assertions elsewhere to be updated.
Questions

I've opted for some ctors on the PointInTime and PointInTimeDescriptor since an ID is always necessary and this reduces consumer code. I've not seen this applied much in the non-generated code so perhaps this is negative implications? In my opinion it make consumption simpler for the test cases we have at the moment.

By adding this the Fluent syntax is r.Query(...).AllIndices().PointInTime("an-id") and the initializer syntax is:
new SearchRequest<Project>(Nest.Indices.All)
{
	Size = 1,
	Query = new QueryContainer(new MatchAllQuery()),
	PointInTime = new Nest.PointInTime("an-id", "1m")
}

I've used the Time type for KeepAlive on the PointInTimeRequest. This seemed appropriate from other usages, however, the OpenPointInTimeResponse uses a string at the moment (we could change that if it's preferred?)

Would it be nicer if the OpenPointInTimeResponse was deserialised into a PointInTime (without keep alive) so that it could then be passed into the search, rather than creating an instance? One thing to note is this line from the docs

The open point in time request and each subsequent search request can return different id; thus always use the most recently received id for the next search request.

So while convenient for the first request, this might not be super helpful on subsequent searches unless we also converted the pit_id into a PointInTime on the search response also. Some of this we may be able to abstract more nicely behind a helper.

Should we review if ES should allow PIT searches against _all by opening an issue there? The behaviour right now is valid since _all implies to search everything, while a PIT search limits to the scope of the PIT, however, it forces our consumers to use AllIndices.

Future discussion perhaps - Is there any clean way of inferring AllIndices when PointInTime is used, without the consumer having to specify it, or adding support for this?

I've opted for some ctors on the PointInTime and PointInTimeDescriptor since an ID is always necessary and this reduces consumer code. I've not seen this applied much in the non-generated code so perhaps this is negative implications? In my opinion it make consumption simpler for the test cases we have at the moment.

We do this very rarely, largely because we don't do client side validation to enforce required properties. Quite often required properties are contextual. I think in this case it makes a lot of sense.

I've used the Time type for KeepAlive on the PointInTimeRequest. This seemed appropriate from other usages, however, the OpenPointInTimeResponse uses a string at the moment (we could change that if it's preferred?)

Using time on responses has some benefits (e.g ToTimeSpan)

Would it be nicer if the OpenPointInTimeResponse was deserialised into a PointInTime (without keep alive) so that it could then be passed into the search, rather than creating an instance? One thing to note is this line from the docs

The open point in time request and each subsequent search request can return different id; thus always use the most recently received id for the next search request.

So while convenient for the first request, this might not be super helpful on subsequent searches unless we also converted the pit_id into a PointInTime on the search response also. Some of this we may be able to abstract more nicely behind a helper.

Maintaining a 1-1 mapping is key. Long term all of this will be generated. I'd rather focus on the helper.

Should we review if ES should allow PIT searches against _all by opening an issue there? The behaviour right now is valid since _all implies to search everything, while a PIT search limits to the scope of the PIT, however, it forces our consumers to use AllIndices.

This is worthy of a discussion on `elastic/elasticsearch for sure.

Future discussion perhaps - Is there any clean way of inferring AllIndices when PointInTime is used, without the consumer having to specify it, or adding support for this?

I think there is a route to do this through RequestBase.GetUrl

@Mpdreamz:
I've updated this to support overriding the ResolveUrl method, which I've done for search to handle cases where point in time is in use. This means that consumers are no longer required to know that they must use AllIndices when including a pit ID. I've removed the more hacky search URL fixup I had previously.
I thought one thing and wrote another earlier in regards to OpenPointInTimeResponse using Time. The response only includes the ID so that's a non issue. What I meant to query is the auto-generated OpenPointInTimeRequest where KeepAlive is a string by default based on the spec.

What I meant to query is the auto-generated OpenPointInTimeRequest where KeepAlive is a string by default based on the spec.

Can this also be specified in the body? If so you can disable its generation by adding a CodeConfigurationFile for it e.g:
https://github.com/elastic/elasticsearch-net/blob/master/src/ApiGenerator/Configuration/Overrides/Endpoints/ScrollOverrides.cs#L12


fix imul83hAdd

I found a shellcode that some calculates hashes(imul83hAdd algorithm) were wrong.

For example, the first line is a hash of GetProcAddress.
The correct value is "9AB9B854h", However it was "0x1ab9b854" in the script calculation.
I modified the code to get the correct results.

Here is a sample in the wild to test this request.
https://www.virustotal.com/gui/file/3a0165e7b01aabccb6955b7d118c53f3baf3f4abbee22df6872723aedc3d90d9/detection

Thanks so much for the fix!


Potentially incorrect check

Hey, cool project!
While reading through the code I saw a potentially incorrect check
https://github.com/Candy-Labs/CandyContracts/blob/main/contracts/CandyCreatorV1A.sol#L372
I imagine it should be checking for require(_exists(... rather than require(!_exists(...
Cheers

You're right, thank you! I changed an ERC721A revert error to a require with a custom string and so this should be changed ASAP.

Fixed in the latest commit. Thanks for your help. We just updated our contract to v3.0 of ERC721A and would have missed this without you.

Np, glad to help! Great job on the project!


net: lwm2m: add callback for send confirmation

Make send result deprecated as confirmation result shall always be asked. Add user callback to get confirmation result. See #52328
Signed-off-by: romain pelletant<EMAIL_ADDRESS>

Add @broglep-work , @SeppoTakalo

Looks like you need to refactor the existing code to use the new API.
Twister builds are failing because of deprecated code.
See the test results.

add : when network is disconnected, timeout callback shall be called before reseting messages

Looks good please rebase and fix conflict's.

There seem to be conflicts so please rebase on top of main and fix those.
Then fix the shell, like I asked in the first review and drop the one-line rd_client_stop() fix, and I think this is then ready to go in.

@broglep-work @SeppoTakalo @jheiskan81 Requested changes has been done

We still need the lwm2m_send() because it is used inside nRF Connect SDK, so this would break the next upmerge.
Also, I spotted faults in the shell handling.
Here is my proposal
❯ git diff
  diff --git a/include/zephyr/net/lwm2m.h b/include/zephyr/net/lwm2m.h
  index d866fab9bf..63483f4bc2 100644
  --- a/include/zephyr/net/lwm2m.h
  +++ b/include/zephyr/net/lwm2m.h
  @@ -2104,6 +2104,20 @@ enum lwm2m_send_status {
    */
   typedef void (*lwm2m_send_cb_t)(enum lwm2m_send_status status);

  +/**
  + * @brief LwM2M SEND operation to given path list asynchronously with confirmation callback
  + *
  + * @param ctx LwM2M context
  + * @param path_list LwM2M path struct list
  + * @param path_list_size Length of path list. Max size is CONFIG_LWM2M_COMPOSITE_PATH_LIST_SIZE
  + * @param confirmation_request Set to true to use CoAP CON request which is the only allowed.
  + *
  + * @return 0 for success or negative in case of error.
  + *
  + */
  +int lwm2m_send(struct lwm2m_ctx *ctx, const struct lwm2m_obj_path path_list[],
  +                         uint8_t path_list_size, bool confirmation_request);
  +
   /**
    * @brief LwM2M SEND operation to given path list asynchronously with confirmation callback
    *
  diff --git a/subsys/net/lib/lwm2m/lwm2m_message_handling.c b/subsys/net/lib/lwm2m/lwm2m_message_handling.c
  index c27630eb8a..6c4b6f7884 100644
  --- a/subsys/net/lib/lwm2m/lwm2m_message_handling.c
  +++ b/subsys/net/lib/lwm2m/lwm2m_message_handling.c
  @@ -2949,8 +2949,18 @@ static bool init_next_pending_timeseries_data(struct lwm2m_cache_read_info *cach
   }
   #endif

  +int lwm2m_send(struct lwm2m_ctx *ctx, const struct lwm2m_obj_path path_list[],
  +                 uint8_t path_list_size, bool confirmation_request)
  +{
  +       if (!confirmation_request) {
  +               return -EINVAL;
  +       }
  +
  +       return lwm2m_send_cb(ctx, path_list, path_list_size, NULL);
  +}
  +
   int lwm2m_send_cb(struct lwm2m_ctx *ctx, const struct lwm2m_obj_path path_list[],
  -                        uint8_t path_list_size, lwm2m_send_cb_t *reply_cb)
  +                 uint8_t path_list_size, lwm2m_send_cb_t *reply_cb)
   {
   #if defined(CONFIG_LWM2M_SERVER_OBJECT_VERSION_1_1)
          struct lwm2m_message *msg;
  diff --git a/subsys/net/lib/lwm2m/lwm2m_shell.c b/subsys/net/lib/lwm2m/lwm2m_shell.c
  index 7b2090f64d..cbfa154645 100644
  --- a/subsys/net/lib/lwm2m/lwm2m_shell.c
  +++ b/subsys/net/lib/lwm2m/lwm2m_shell.c
  @@ -59,7 +59,6 @@ static int cmd_send(const struct shell *sh, size_t argc, char **argv)
          int ret = 0;
          struct lwm2m_ctx *ctx = lwm2m_rd_client_ctx();
          int path_cnt = argc - 1;
  -       int ignore_cnt = 1; /* Subcmd + arguments preceding the path list */
          struct lwm2m_obj_path lwm2m_path_list[CONFIG_LWM2M_COMPOSITE_PATH_LIST_SIZE];

          if (!ctx) {
  @@ -68,12 +67,6 @@ static int cmd_send(const struct shell *sh, size_t argc, char **argv)
          }

          if (argc < 2) {
  -               shell_error(sh, "no arguments or path(s)\n");
  -               shell_help(sh);
  -               return -EINVAL;
  -       }
  -
  -       if (argc == 0) {
                  shell_error(sh, "no path(s)\n");
                  shell_help(sh);
                  return -EINVAL;
  @@ -83,18 +76,19 @@ static int cmd_send(const struct shell *sh, size_t argc, char **argv)
                  return -E2BIG;
          }

  -       for (int i = ignore_cnt; i < path_cnt; i++) {
  +       for (int i = 0; i < path_cnt; i++) {
                  /* translate path -> path_obj */
  -               ret = lwm2m_string_to_path(argv[i], &lwm2m_path_list[i], '/');
  +               const char *p = argv[1 + i];
  +               ret = lwm2m_string_to_path(p, &lwm2m_path_list[i], '/');
                  if (ret < 0) {
                          return ret;
                  }
          }

  -       ret = lwm2m_send_cb(ctx, lwm2m_path_list, path_cnt, NULL);
  +       ret = lwm2m_send(ctx, lwm2m_path_list, path_cnt, true);

          if (ret < 0) {
  -               shell_error(sh, "can't do send operation, request failed\n");
  +               shell_error(sh, "can't do send operation, request failed (%d)\n", ret);
                  return -ENOEXEC;
          }
          return 0;

Just thinking out loud, but wouldn't a user_ctx void * or similar be useful/almost necessary here?
Considering sth like (pseudo-code-is)
for (int i = 0; i < 10; ++i) {     send(data[i], send_cb); } [...] send_cb(status) {    // now which of my ten messages has been delivered/timed out? }
I suppose you could register ten different cbs in this situation, but that'd be very unwieldy (and impossible once "i" is something dynamic).

@huelsenfruchtzwerg Good point.
From my point of view, lwm2m send method requires SENML_CBOR which is implementing composites reads/write and reduce the needs for several sends.
If I need multiple send, maybe should I only send one message with several objects/data.
Do you think the same way?

@RomainPelletant That's also a fair point and something I also considered.
The Problem is when the code calling the send is "dynamic". Simplest (albeit admittedly somewhat constructed) example I can think of is the shell function cmd_send. Currently this doesn't use the send_status_cb, but assuming it did, how would you differentiate between multiple invocations with different arguments in the cb?
If given the choice, I think I'd rather have a single user defined send_cb for all messages and a void* user_ctx for each message. The user could still use the *user_ctx pointer for individual cbs if he so desires, or he could use it for something else.

@huelsenfruchtzwerg Indeed for the shell it could be a critical problem for multiple send.
Currently, we don't receive any confirmation (from user space) and this pull request at least carry out that need.
I think we shall go further and implement a way to register send callback for multilple sends (as you described) : could we consider opening a new issue or that PR shall implement it ASAP?

Depends on what approach you/we want to take. Just adding an additional void* ctx parameter to the callback (and the msg struct), would be simple enough to do as part of this pr imho, but would increase the msg structs size by one pointer.
The other approach I outlined above (singular send_status_cb with a void *ctx paramer per send invocation instead of a cb per invocation) would be a bigger change.
I'd defer to @SeppoTakalo and @rlubos, I'm indifferent

@RomainPelletant ping


Issue 54

Coverage decreased (-0.1%) to 54.619% when pulling 9ad89590b55d033b80df4c9f12870fdc91728af0 on issue_54 into 102f16f992700a6f1907739db5acee79769d639c on master.


Mix Script Execution Weird Behavior

I am writing a mix script, these are the lines I used:

mix.sass('sass/app.scss', 'bundle1.css');

mix.less('less/app.less', 'bundle2.css');

mix.combine(['css/app.css'], 'bundle3.css');

mix.combine(['bundle2.css', 'bundle1.css', 'bundle3.css'], 'bundle.css');

the problem is every time I run 'npm run dev' ( all bundles not created yet ) it will complain that bundle2.css not exist
assert.js:85
  throw new assert.AssertionError({
  ^
AssertionError: Mix.combine() error: "bundle2.css does not exist.

and then I would comment out the fourth line that combine them all, and run again, it will success.  and then enable the fourth line and execute again, it will success now (because bundle2.css now exist).
my observation is, the fourth line would be executed first, since it produce the error, but that is very weird, isn't this script suppose to be executed sequentially? and I also run into a couple times when the first line of the script is not executed, skipped to second line.
for the record, this is the script I used, just that I removed the paths from them.
regards

seems that mix should not verify combine arguments immediately, but after all compilations finished.
src/index.js

hmmhh, seems that's the problem. anyway while I am still here, is there anyway we can have a something like mix.delete to remove file ( well I want to remove the bundle1.css, bundle2.css and bundle3.css ). we can do this with the gulp one.
regards

I've removed the part that fails if the source files don' exist.


fixed imports so as to work with Python3

This change only works with Python3.
Thus, it might be better to make a branch called python3 and people can clone that branch if they want to use with python3.

Changed PyQt4 to PyQt5
Changed all the imports
Fixed Code2pdf/__init__.py so that it can be imported

@watakandai  I appreciate your contribution. My bad that I have merged another PR https://github.com/tushar-rishav/code2pdf/pull/29 which was more recent with other fixes and overlooked this PR.


prefer removing .view()

@zhuangya @yorkie
.view() is more useless and imperfect, so i prefer removing from paloma, keep kernel clean. We can render view in controller like koa:
app.controller('indexCtrl', async function (ctx, next) {
  await ctx.render(...)
});

Why not directly use return like and using callback in async/await function is a little bit weird.


Fix selectable disabled toggle radio buttons

A disabled radio toggle button is not selectable
Disabled either by
-disabled class on Label container
-input with a disabled attribute

Ah, wrong branch - sorry meant to raise against the master. I'll try again.


How to configure CICD on VSTS for Easyrepro

I have created scripts for Dynamic 365 using Easyrepro and now I want it to trigger the scripts from VSTS on daily basis.

I want to run my UI scripts daily using VSTS. So can any one please suggest me suitable way to do it ijn VSTS

Do you already have an automated CI process using VSTS, which you would like to add these tests to, or are you starting from scratch?
If you are starting from scratch, then this article details how to set up a VM to run tests automatically: https://blogs.msdn.microsoft.com/testingspot/2017/07/26/run-your-independent-automated-tests-using-vnext-build-in-visual-studio-online/
Your release pipeline can be configured to run automatically after each build, or on a schedule, so you should just be able to add a 'Run Visual Studio Tests' task in your release definition, tell it which dll your tests are contained in and which runsettings file to use if appropriate.
Hope this is of some help...

Thanks for the Reply. As of now I dont have Azure environment. So I want to know how to trigger the scripts in our local VCloud from VSTS.

I think that Microsoft are now offering at least one free pipeline, so you may not even need to have an Azure environment to get things going:
https://azure.microsoft.com/en-gb/services/devops/pipelines/
However, if you do want to run the process locally, the steps are going to be something like this:
1.) Install the vsts agent on the machine that you are using as your build / release server
https://docs.microsoft.com/en-us/azure/devops/pipelines/agents/agents?view=vsts
2.) Create a Build definition, triggered on check in of the solution which contains your tests
This should perform a build, and then publish any artefacts needed by the tests (such as the built dll containing the tests, and possibly one or more runsettings files)
3.) Create a release pipeline, which can either be triggered by the build, or can be scheduled to run periodically.
To run the tests you can use the 'Visual Studio Test' task, pointing at the dll which was published by the build.
If you want to run the tests against different environments, or with different browsers, or in some other way you want to parameterise your tests, I'd reccomend using a 'runsettings' file - this allows you to run the same set of tests multiple times, each with a different set of parameters. You can chose the runsettings file when running the tests locally in Visual studio, or you can specify the runsettings file in your release task setup and have multiple tasks, one for each runsetting file that you want to use.
I think that this is probably the best place for you to get started:
https://docs.microsoft.com/en-us/azure/devops/pipelines/get-started/index?view=vsts
It explains build definitions, release pipelines, etc.

Documentation has been added to the wiki for this: https://github.com/Microsoft/EasyRepro/wiki/Azure-DevOps-Test-Execution
I've also written a blog post on it


Incorrect single tag syntax in package reference XML

The C# Tooling support for .proto files section, client project package reference XML is missing single tag closure for Gprc.Tools package reference.
Screenshot as below

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

ID: 7223e694-85ba-4b95-96c8-68ae14753e7c
Version Independent ID: 28ec8399-ba0e-c995-3ac2-b1f28e93331b
Content: gRPC services with C#
Content Source: aspnetcore/grpc/basics.md
Product: aspnet-core
Technology: aspnetcore-grpc
GitHub Login: @JunTaoLuo
Microsoft Alias: johluo

Thanks @gopkumr ... I'll patch that up right now.

Turns out that it's an opening tag ... the full package reference from the file is ...
<PackageReference Include="Grpc.Tools" Version="2.28.1">
  <PrivateAssets>all</PrivateAssets>
  <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
</PackageReference>

However, it is confusing. I'll make that code sample show the full package reference in the topic to avoid confusion.

Yes thats correct. I saw that the source csproj has more code to it but the documentation just leaves out the last few lines.  I agree it is better to include the whole tag pairs.

There are a few 🐞 bug fixes this morning ... this and a bad link on another PR. I'll go ahead merge to live. These updates should hit the live site within 15 minutes. Thanks again for the issue on this.

Thanks for the fix. The documentation looks perfect now. Starters will not face an issue when following the documentation.


Api3TokenLock Integration and Tests according to specifications

I have updated the tests to cover all blocking scenarios (thats what we are going for I presume instead of blacklisting).
Two things we should probably address:

Contract Names
Directory Structure.

Updated the contract names and interfaces and also the tests. If everything is good we can have it merged then work on iterating it later.

LGTM just one last thing before merging:
shouldn't we put the ITokenLockRrpAuthorizerAdmin and ITokenLockRrpAuthorizerAdminExternal interfaces in a folder on the same level as the contracts (rrp/authorizers/admin/interfaces) instead of having them all mixed with authorizers interfaces?

i actually did that initially but moved it back because I noticed the mock contracts were also this way. Ill move them to the new folder and merge it.


counter = 1, but never 2

Starting a new topic here.
I used:
people_counter_esp32.ino
the name of my sensor is tof_people_ard
Changed the value 500 to 5000 for wifi timeout.
The ESP32 is succesfully conected to WiFi and MQTT. I can see the topic realtime changing for:
people_counter/tof_people_ard/distance
I do not see any updates for the counter in topic:
people_counter/tof_people_ard/counter
It did work 1 time hower.

In above image you can see the MQTT realtime going and on some moment I place my hand before the sensor.
the standard config as in the ino is in my compiled upload. And no matter what I try to place the sensor it counts 2... so the home assistant counter is never updated.
Any help appreciated!

// this value has to be true if you don't need to compute the threshold every time the device is turned on
static bool save_calibration_result = false;
Default value is false...try set to true, then will working.


ReplyError: ERR unknown command 'get'

node -v : v9.5.0
"redis": "^2.8.0",
when using  client.get('key') , output
 ReplyError: ERR unknown command 'get'
    at parseError (/node_modules/redis-parser/lib/parser.js:193:12)
    at parseType (/node_modules/redis-parser/lib/parser.js:303:14) command: 'GET', args: [ 'wxTicket' ], code: 'ERR'

also error on select command
ERR unknown command 'select'
    at parseError (/node_modules/redis-parser/lib/parser.js:193:12)
    at parseType (/node_modules/redis-parser/lib/parser.js:303:14) command: 'SELECT', args: [ 3 ], code: 'ERR' }

find it happen was that redis using sentinel model ;  replace with normal model , would working right

Hm, what do you mean with sentinel model?

sentinel mode can not be used like normal redis. I think the approach was wrong.

Closing as question.


departures-with-alert-resource throws "Internal Server Error"

Due to changes within the Metronom API, some functions are broken.
Pending.

Fixed by e1abe0d


Dispose of SeekBarChangeObservable

https://github.com/JakeWharton/RxBinding/blob/24b73f4d2f929cc732d1e7b64e6b900241fb2205/rxbinding/src/main/java/com/jakewharton/rxbinding2/widget/SeekBarChangeObservable.java#L58
I'm just curious, why is SeekBarChangeObservable.Listener#onDispose setting the listener once again?
Instead of: view.setOnSeekBarChangeListener(this);
Shouldn't it be view.setOnSeekBarChangeListener(null);  ?

Yep! Send a PR?
On Fri, Nov 24, 2017, 2:42 PM Ricardo Belchior<EMAIL_ADDRESS>wrote:

https://github.com/JakeWharton/RxBinding/blob/24b73f4d2f929cc732d1e7b64e6b900241fb2205/rxbinding/src/main/java/com/jakewharton/rxbinding2/widget/SeekBarChangeObservable.java#L58
I'm just curious, why is SeekBarChangeObservable.Listener#onDispose
setting the listener once again?
Instead of: view.setOnSeekBarChangeListener(this);
Shouldn't it be view.setOnSeekBarChangeListener(null); ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JakeWharton/RxBinding/issues/419, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEEEYKDAqwSTyOH6PAJQGsmcpxt3-0pks5s5xwsgaJpZM4QqJGv
.

ok


⚠ PrivateHD has degraded performance

In e05ff54, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 4689 ms

Resolved: PrivateHD performance has improved in 24cc13b.


Don't collapse above node when outdenting

Make a node, and give it a child node.
Hit enter, make a new node, and outdent it. the node above will be collapsed. It shouldn't be collapsed—the user didn't give any indication that they wanted to collapse it.
http://recordit.co/PpujGH6Mke

@progrium i think this is a bug but if not then feel free to remove the label


Prevent removal of bounds when combining cubes

There was an issue when calculating precipitation type rates and accumulations by multiplying a total precip rate / accumulation by probability of precipitation type (eg probability of snow).  Since the probability of type was a point time, the bounds on the time coordinate were being incorrectly removed.  This change prevents removal of bounds when combining cubes, and causes expand_bounds to raise an error if it would cause the removal of valid bounds from a coordinate.
Description
Testing:

[x] Ran tests and they passed OK
[x] Added new tests for the new feature(s)

@bayliffe I've had a play with the code and I don't think this is needed - I couldn't actually figure a way of causing that specific issue.  Let's chat offline.


How to disable haskell-indention-mode?

The documentation says that one is supposed to disable haskell-mode's indentation features, but how exactly do I do that? It seems to me like haskell-mode enables those features unconditionally?

In the haskell-mode docs it talks about enabling indentation modes.

I saw the sections in the manual about enabling indention modes ... but I saw nothing about how to enable no indention mode at all, which is what structured-haskell-mode needs, no?

Just don't specify one in your haskell-mode-hook?

@ivan-m, then I'll get the default indention mode.

@peti M-x customize-variable haskell-mode-hook and make sure the hooks are disabled?

@ivan-m, I had no idea this possibility existed.

@peti: and now you do :)


Mobile responsive

When the files are generated they work well on desktop view, however it does not handle mobile responsiveness in regards to text overflow.
Not part of MVP but will need minor styling changes within generation.

fixed on /presentation-portfolio repo, need  to confirm that with deployed site then revisit once complete


Change install article to give more specific information about create-react-app, and to be less opinionated about advocating for its use

[x ] I have read freeCodeCamp's contribution guidelines.
[ x] My pull request has a descriptive title (not a vague title like Update index.md)
[ x] My pull request targets the master branch of freeCodeCamp.
[x ] None of my changes are plagiarized from another source without proper attribution.
[x ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.


Documented ProtobufSerializer

Contains
Added Documentation to ProtobufSerializer - GCI Task
How to test
Verify descriptions
Outstanding before merging
None

Can one of the admins please verify this patch?

@GooeyHub: add to whitelist

Oh - and you for some reason triggered a merge conflict in the ProtobufSerializer.java file - likely you didn't have the very latest version in your workspace when you began your work. Try looking up and applying git rebase to the situation to resolve the conflicts, this is a nice safe little PR to experiment with that anyway :-)

Hooray Jenkins reported success with all tests good!

Uh oh, something went wrong with the build. Need to check on that

I tried to rebase it, but the build is still failing.

Closing PR and creating new one as instructed


TASK [ceph-mgr : set _disabled_ceph_mgr_modules fact] Failed

Bug Report
What happened:
When running the site playbook the following task fails on an existing cluster deployed with an older version of this playbook:
TASK [ceph-mgr : set _disabled_ceph_mgr_modules fact] *******************************************************************************************
fatal: [clotho.mtr.royhousehold.net]: FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: 'list object' has no attribute 'disabled_modules'\n\nThe error appears to have been in '/home/sevenofnine/ceph-ansible/roles/ceph-mgr/tasks/main.yml': line 28, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: set _disabled_ceph_mgr_modules fact\n  ^ here\n"}

I believe this is because this older version of ceph provides an unexpected format of "mgr module ls":
$ sudo ceph mgr module ls
[
    "status"
]

..which does not contain any disabled modules.
Share your group_vars files, inventory:
group_vars:

# Variables here are applicable to ceph-servers
ceph_private_interface: enp8s0
ceph_public_interface: enp3s0
ceph_private_netbase: "192.168.18"
ceph_public_netbase: "192.168.19"
ceph_public_mtu: 1500
ceph_private_mtu: 4096

fsid: 'xxxxxxxxxxxxxxxxxxxxxxxxxxx'
generate_fsid: false

#was: ceph_origin: upstream
ceph_origin: repository
ceph_repository: community
ceph_stable: true
ceph_stable_release: luminous

monitor_interface: "{{ ceph_public_interface }}"
journal_size: 5120 # in MB
public_network: "{{ ceph_public_netbase }}.0/24"
cluster_network: "{{ ceph_private_netbase }}.0/24"
#allow OSDs to discover and use new disks, then dm-crypt them with their journal
osd_auto_discovery: false #this is now broken
devices:
  - /dev/sdb
  - /dev/sdc
  - /dev/sdd
  - /dev/sde
  - /dev/sdf
  - /dev/sdg
osd_scenario: collocated
osd_objectstore: bluestore
dmcrypt: true

cephfs: cephfs # name of the ceph filesystem
cephfs_data: cephfs_data # name of the data pool for a given filesystem
cephfs_metadata: cephfs_metadata # name of the metadata pool for a given filesystem

cephfs_pools:
  - { name: "{{ cephfs_data }}", pgs: "128" }
  - { name: "{{ cephfs_metadata }}", pgs: "128" }

ceph_conf_orverrides:
  global:
    rbd default features: 1

Environment:

OS (e.g. from /etc/os-release): Ubuntu 16.04.3 LTS (Xenial Xerus)
Kernel (e.g. uname -a): Linux 4.4.0-142-generic #168-Ubuntu SMP x86_64 x86_64 x86_64 GNU/Linux
Docker version if applicable (e.g. docker version): N/A
Ansible version (e.g. ansible-playbook --version): 2.6.13
ceph-ansible version (e.g. git head or tag or stable branch): Stable-3.2 commit a43c68df7d3dc5f76432d8768f913e8dbbf64a91
Ceph version (e.g. ceph -v): ceph version 12.2.1 (3e7492b9ada8bdc9a5cd0feafd42fbca27f9c38e) luminous (stable)

is the issue resolved for you guys? What was the fix ?


Localize Wikipedia link/description

This issue involves collaboration with the web team. If a user is viewing the app in a non-English locale (i.e. French), they should see the Wikipedia description and Wikipedia link in their given locale.
A few ways this could potentially happen:

We could try to return different links and content via the API request on TaxonDetails (this might not be possible)
We could expose an additional endpoint for the Wikipedia description via API v2 and make two separate API requests on TaxonDetails (this is similar to what we're doing in Rails on the web)
We could create a separate endpoint for both the description and the Wikipedia link

[ ] If I view https://www.inaturalist.org/taxa/71956-Aloe?locale=fi I see a Wikipedia description, but if I view that taxon in that locale (suomi) in the app, I don't see one
[ ] The "Read More on Wikipedia" link on TaxonDetails always seems to link to en.wikipedia.org


Slow start warnings

The component begins checking for email right away and appears to be causing a slowdown loading other components causing warnings.

This will likely be fixed in the latest PR

Installed the patch and ran a reboot.  No more boatload of components failing to load in 10 seconds errors!  Will see if it spits out an image tomorrow (assuming I get mail).

I'll be testing it on my Pi3 later this evening as well.

So far so good on Pi3 far less "10 second" errors than before.


fixes #1178 - Changed MarathonSchedulerActor to use a Set[PathId] for ap...

...p locks instead of Semaphores
The reason for this change is that previously we blocked on the
semaphores inside of futures. There are several problems with this
approach.

Prone to deadlocks
unnecessarily blocks threads
breaks actor encapsulation by closing over the semaphores
not necessarily fair if multiple forced commands arrive before
the original command has been cancelled

Because this is a very drastic change and caused lots of changes in
the other deployment related classes, this should be thoroughly reviewed
before merging.

Please add an explanation to the commit msg why this change was done. I guess there is some deeper reason than just fixing the bug. What was wrong with the old approach?

In general, I like the change. Main comment: explain in the commit message why we avoid the dead locks with the new approach.

This LGTM, and I've run another battery of tests against it beyond the automated tests in the repo.  Let's land it!

Awesome!


Helm chart 1.0.2 only has charts.yaml updated. Values file still has 1.0.1 image tags

What happened?
Upgraded from 1.0.1 to 1.0.2 using Helm but the tetragon and tetragon-operator images stayed at 1.0.1
Compared the two helm charts and the only difference is in the charts.yaml file. The values.yaml still has 1.0.1 image tags
Tetragon Version
1.0.1 / 1.0.2
Kernel Version
5.15 and 6.5
Kubernetes Version
1.27.9
Bugtool
No response
Relevant log output
No response
Anything else?
No response

Thanks for the report. I believe this should have been fixed now (https://github.com/cilium/tetragon/blob/eaa59471c3e963c3c957e726b320079300496c61/install/kubernetes/tetragon/values.yaml#L226) by https://github.com/cilium/tetragon/pull/1993 but we need to ensure that the next release includes the latest images and the next release the release script will do the right thing: https://github.com/cilium/tetragon/blob/eaa59471c3e963c3c957e726b320079300496c61/contrib/update-helm-chart.sh#L20.


Allow searching in :help screen

Pressing / in the :help menu doesn't work.  It would be nice to support search there.

You can do :help PATTERN (or :help "PATTERN" for multi-word phrases).
Being able to search help with / would be a nice touch.
IIRC, the help table is generated differently than the task table, and I'm guessing it won't be trivial to integrate the search system into help.
It's not something I have any interest in writing, but would consider a PR for it if it was implemented cleanly.

It would be nice for this to be implemented together with #208 so that searching the help screen with / and also filtering with :help PATTERN respect ignorecase and smartcase and also :help \cPATTERN and :help \CPATTERN are possible to override this setting.


Update documentation in Okta Help Center

Hello @AlainODea! Can you please tell me what needs to change in https://support.okta.com/help/Documentation/Knowledge_Article/Integrating-the-Amazon-Web-Services-Command-Line-Interface-Using-Okta so it's up-to-date with this repo?

Sure thing. @smashling is preparing a getting started presentation, so in that process I'll review the Okta Help Center material and suggest updates as well.
Thank you, @mraible :)

Sorry for the delay on this. @smashling can you take a look at the docs and provide some recommendations by Tuesday next week?

In the Part 2:Use the Token section, I would add a note about the new command for users with multiple profiles that they switch between. An example command would be end OKTA_PROFILE=[profile name] withokta aws --profile [profile name] s3 ls. I would also recommend that the user makes a command alias for their various profiles, because of how long the current command is.

@mraible my readme updates in #146 and the corresponding installer scripts should dramatically help ease of adoption for this tool.


[feature] use the public_dns as default name for APIs. fix #181

Coverage remained the same at 66.32% when pulling d258cf3a882db204ef4dace1e278883d88807863 on feat/default-api-name into 4f14025b4fb8d9fd4beb523aefc5720c1daba6f7 on master.


Use components placed outside of django app and use external webpack config

Hello, Mark! Thanks for useful app. I have two questions:

JS components in my app are placed outside of django directory(and then packed with webpack to django_app/staticfiles/js/bundle.js), so it's not possible to find them using django's staticfiles finder.  In addition I don`t want to make this components available via staticfiles finder. How can I configure 'django-react' to work with such layout?
I have webpack.conf.js with lots of options defined. Is there any way to use this config file instead of 'django-webpack' default?

Hi,
You can define a source_url attribute on the the component which will override all the django-webpack integration.
class MyComponent(ReactComponent):
    source_url = os.path.join(STATIC_URL, 'path/to/file.js')

Assuming that you're happy to rely on your pre-existing webpack setup, avoiding django-webpack is probably the easiest way to preserve your config.
Hmm, I think that supporting an external config file is a reasonable use case. Out of curiosity, what's the command you run to bundle the asset(s)? Does webpack walk up the tree to find a conf file, or do you specify it?
Cheers,
Mark

You should now be able to specify a path_to_source attribute, which will enable rendering.
class MyComponent(ReactComponent):
    path_to_source = os.path.join(PROJECT_ROOT, 'path/to/file.js')

MyComponent().render_to_string()

I've edited the previous comment to reflect a name change, source_url is now url_to_source.
You can also override the get_path_to_source and get_url_to_source methods if you want to define things at runtime.

Thanks for soon reply, Mark, will test url_to_source.
As for webpack config file, I use default config file name 'webpack.config.js' as mentioned in official docs.


[shared_preferences] Publish fully federated version

Restores the app-facing package to publishable state, pointing at the
newly-published federated Android and iOS implementation packages.
Part of Part of flutter/flutter#68498
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

submit-queue check is incorrectly showing red; landing.


refactor(Icon): system icons refactor

Summary
Type

Refactor

Summarise concisely:
What is expected?
New system icons with 4 variants (outlined/filled and small/large).
The following changes were made:

Imported the new icons

Added the variants

Updates every component & snapshot using the icons

Currently no test for the component Icon from @ultraviolet/icons

system-icon-fix.zip
Some icons were not exported right : folder, profile, arrow-right-bottom. Here's a .zip with the correct icons.

Delete icon seems smaller than others.


[MI-2159] Test cases for some components

Created test cases for components:

download_button
recordTypePanel
subscriptionTypePanel
eventsPanel

Also check, if any snapshot needs to be updated.


Feat: Allow dev set themes/overwrite default light/dark

Feature: Allow devs to disable the colors by light/dark mode and instead let them decide which colors to use

What is the current behavior? (You can also link to an open issue here) #20

What is the new behavior (if this is a feature change)?

Dev can pass a params of  ITheme which overites the default light dark mode of the package

Other information: nil

@earrietadev please review.

I've tested this and it's working as expected


Make bin dir configurable and default to /usr/bin

The releases for marathon packaged here https://github.com/mesosphere/marathon/releases
default to the marathon executable living at /usr/bin
I've made this a configurable option and set the default value to /usr/bin so any installs of marathon 0.8.0 should work

@tayzlor please check the master branch, this is already configurable with $marathon::bin_path.
Thanks for sending the PR though.


Add support for custom word delimiters

With the default hardcoded delimiter regular expression, the following content would only be counted as 1 word which is not quite correct:
a,sentence,like,this:would;only-be-counted,as;one;word.

Hence, giving users the choice of injecting their own delimiters would let everyone get along.
Enjoy.

Thanks


introducing react-three/xr blog post

Initial concept. Requires feedback.
We should potentially link the examples including their gifs?

I made some edits:

Added your author profile. Feel free to edit as you see fit.
Changed title to "Reintroducing @react-three/xr" since this is a big refresh.
Changed React-Three-Fiber to its properly formatted React Three Fiber.
Changed @react-three/xr to be formatted as a package name.

Some things to consider: Do we want to just refer to the library as React Three XR when written out?
I think a section should be included after the introduction titled "What's new" where you explain more about the rewrite, the name change and introduce yourself as the new maintainer. You can explain the that goal is interop between the React Three ecosystem.


download youtube shows "Unknow command"

windows docker v4.28.0, WSL2
just send the URL
ex: https://youtu.be/XHTrLYShBRQ
and you see the yt_bot log with below error message

2024-03-22 17:14:32 Traceback (most recent call last):
2024-03-22 17:14:32   File "/usr/local/lib/python3.11/site-packages/pyrogram/dispatcher.py", line 268, in handler_worker
2024-03-22 17:14:32     await handler.callback(self.client, *args)
2024-03-22 17:14:32   File "/usr/local/lib/python3.11/site-packages/pyrogram/handlers/message_handler.py", line 151, in resolve_future_or_callback
2024-03-22 17:14:32     await self.original_callback(client, message, *args)
2024-03-22 17:14:32   File "/app/bot/core/callbacks.py", line 38, in on_message
2024-03-22 17:14:32     user = client.allowed_users[get_user_id(message)]
2024-03-22 17:14:32            ~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^
2024-03-22 17:14:32 KeyError:<PHONE_NUMBER>

bot responds: "Unknow command"
im using the zip for the commit: 20c35b0

Hello, I would need to see your configuration with removed personal data and screenshot from Telegram.
Didn't see this issue before. Did you send the link from your user to Bot directly or from some group?

sorry for late reply
win11/docker desktop v4.29.0
re-build again for https://github.com/tropicoo/yt-dlp-bot/commit/32e9b92eeb0d4cf51e10fdc314aea76e31d6b717
in yt_bot log:
2024-04-10 14:32:23 2024-04-10 06:32:23,670 - [ERROR] - [pyrogram.dispatcher:342] -<PHONE_NUMBER>
2024-04-10 14:32:23 Traceback (most recent call last):
2024-04-10 14:32:23   File "/usr/local/lib/python3.12/site-packages/pyrogram/dispatcher.py", line 329, in handler_worker
2024-04-10 14:32:23     await handler.callback(self.client, *args)
2024-04-10 14:32:23   File "/usr/local/lib/python3.12/site-packages/pyrogram/handlers/message_handler.py", line 151, in resolve_future_or_callback
2024-04-10 14:32:23     await self.original_callback(client, message, *args)
2024-04-10 14:32:23   File "/app/bot/core/callbacks.py", line 38, in on_message
2024-04-10 14:32:23     user = client.allowed_users[get_user_id(message)]
2024-04-10 14:32:23            ~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^
2024-04-10 14:32:23 KeyError:<PHONE_NUMBER>
2024-04-10 14:32:26 2024-04-10 06:32:26,602 - [INFO] - [VideoUploadTask:140] - Uploading "Introducing iPhone 15 ｜ WOW ｜ Apple.mp4" [10.1MiB] [cached: False] to chat id "1277514877"
2024-04-10 14:32:32 2024-04-10 06:32:32,962 - [INFO] - [VideoUploadTask:298] - Saving telegram file cache
2024-04-10 14:32:32 2024-04-10 06:32:32,963 - [INFO] - [VideoUploadTask:140] - Uploading "Introducing iPhone 15 ｜ WOW ｜ Apple.mp4" [10.1MiB] [cached: True] to chat id "-1001392905200"
2024-04-10 14:32:33 2024-04-10 06:32:33,753 - [INFO] - [SuccessDownloadHandler:112] - Cleaning up task "4f54ce66-d439-41c9-ad97-ad6073b72fa6": removing download content directory "/tmp/download_tmpfs/downloaded/oxro" with files {'Introducing iPhone 15 ｜ WOW ｜ Apple.jpg': '35.7KiB', 'Introducing iPhone 15 ｜ WOW ｜ Apple.mp4': '10.1MiB'}

Config.yml (pretty sure i use the latest example file to modify such IDs and tokens the issue still there
*removed the personal data
telegram:
  api_id: API_ID
  api_hash: "API_HASH"
  token: "BOT_TOKEN"
  lang_code: !!str "en"
  max_upload_tasks: 2
  url_validation_regexes:
    - "^http(s)?:\\/\\/.+$"
  allowed_users:
    - id: USER_ID
      is_admin: !!bool True
      send_startup_message: !!bool True
      download_media_type: "VIDEO"
      save_to_storage: !!bool False
      use_url_regex_match: !!bool True
      upload:
        upload_video_file: !!bool True
        upload_video_max_file_size:<PHONE_NUMBER>
        forward_to_group: !!bool True
        forward_group_id: GROUP_ID
        silent: !!bool False
        video_caption:
          include_title: !!bool True
          include_filename: !!bool False
          include_link: !!bool True
          include_size: !!bool True
  api:
    upload_video_file: !!bool False
    upload_video_max_file_size:<PHONE_NUMBER>
    upload_to_chat_ids:
      -<PHONE_NUMBER>0
      - -00000000000
    silent: !!bool False
    video_caption:
      include_title: !!bool True
      include_filename: !!bool False
      include_link: !!bool True
      include_size: !!bool True
ytdlp:
  version_check_enabled: !!bool True
  version_check_interval: 86400
  notify_users_on_new_version: !!bool True

the telegram bot message


Additional wiki pages

Capturing some additional wiki pages that we can use to make things easier for our customers.

Information on what is supported in docker, an Azure based VM and on-premises hardware

Used for when a new OS is requested - so they know what type of system to expect and how long it will take to get the new OS onboarded.

Wiki to explain what the various queue designations are and how/why we use them.

This wiki may already exist, but I've gotten several questions about this recently.

Another wiki page to address - #3842

OS Support documentation

Windows
Linux and Docker
macOS, iOS, tvOS

Helix Queue information - https://dev.azure.com/dnceng/internal/_wiki/wikis/DNCEng Services Wiki/1300/Helix-Queues-and-Usage
Will address #3842 within that issue.


SOURCE_BUCKETS has a limit and can't support a long list of buckets

We would like to be able to set SOURCE_BUCKETS=developer-bucket-* so that each developer's bucket is in the whitelist.
This was okay in 3.0.0 but it stopped allowing * in 4.0.0. We tried adding all our dev buckets as a comma separated value string but we went out of string limit.
It would be good to support regex again. Here is a PR for the proof of concept:
https://github.com/awslabs/serverless-image-handler/pull/138
We tested this with a custom built the image and it works with SOURCE_BUCKETS=developer\-bucket\-.*.

We have updated our solution, and I believe your issue has been fixed. If you still see the issue with the latest version (v4.2), please feel free to reopen the issue.
However, as your pull request says, it only accepts the regular expression after launching the solution.
You can refer to the recent changes here

cc @crpahl


Work on render nodes: exposure and bloom

Contains

Continues refactoring work of the renderer, to take advantage of the new node-based architecture.
See commit descriptions below for an overview of the changes

How to test

Run Terasology
Create a new game or load an existing one
Verify nothing has changed in the rendering, especially when activating/deactivating Eye Adaptation and Bloom.

Notes:

While the bloom effect is in now taking advantage of the new architecture... in practice it doesn't seem to work. This seems to be a pre-existing issue however and I won't worry about fixing it now.
In general a number of problems have crept into the renderer while the overhaul is taking place no matter our best intention to keep everything working.
These problems will be solved during planned additional refactoring passes through all the nodes in the renderer. These refactoring passes are anyway needed as work on the later nodes has shown ways to improve the code in earlier nodes.
Furthermore, an overhaul of the visual debug system (showing the content of a number of FBOs) is also planned and will help pinpoint problems that have visual consequences.

Hooray Jenkins reported success with all tests good!

@indianajohn: thank you for the feedback. Indeed having smaller, simpler nodes is one of my aims. It appears switching shaders on modern hardware is not as horribly expensive as I thought it would be and in later PRs I will try to get deeper into the shaders and break those too, as some shaders do a few too many things for my taste.

Hooray Jenkins reported success with all tests good!

Uh oh, something went wrong with the build. Need to check on that

Hooray Jenkins reported success with all tests good!

@Cervator, I think we are good to go with this one...

Hooray Jenkins reported success with all tests good!

Roger that! Merged latest after a quick test :-)
Also took the chance to switch the IntelliJ copyright header year to 2017 real quick (should apply after the next gradlew idea)


Animation is run at every re-render

Hey :). I'm using functional components and useState. Simply changing the value of a textbox (via the function provided by setState) will cause the animation to be re-run. Maybe I'm missing something :).

Can you share your piece of code because if you are using hooks so we have to maintain some rules in hooks..

@Kyliathy @vishalkumarsinghvi
I faced similar situation and I found the cause of the problem.
The old code cause the animation to start again every time render is called:
    render() {
       let animate = { left: ['0px', '250px']}
        return (
            <Tweenful.div
                className="tween-box row ml-2 mb-2 mr-2 mt-2"
                duration={1500}
                easing={elastic(1, 0.25)}
                style={{ position: 'relative' }}
                animate={animate}>
                <video id="video-elem" controls width="320" height="240"></video>
            </Tweenful.div>
        )
    }

What is going on is that the animate object is created every time render is called,
so the animation passed to Tweenful.div is different object, this cause Teenful to start animation again.
One working solution is to put the animate object in component state, like following:
    constructor() {
        super();
        this.state = {
            animate: { left: ['0px', '300px'] }
        }
    }

And in your render function, use the animate object in state, it will be the same reference (object) across different render calls
            <Tweenful.div
                className="tween-box row ml-2 mb-2 mr-2 mt-2"
                duration={1500}
                easing={elastic(1, 0.25)}
                style={{ position: 'relative' }}
                animate={this.state.animate}>
                <video id="video-elem" controls width="320" height="240"></video>
            </Tweenful.div>

Then it just works fine.


Ent param init

This PR fixes how n_samples is being treated in the meyer_wallach method.
If the parameter is None or <1, then the model internal parameters are being used.
This required setting the model internal parameters in each forward call.

[ ] resolve problem of ArrayBox elements: when using inside an optimisation loop, params become an arraybox which cannot be treated as a numpy array


Fixes potential NaN when calling NativeMapView::nativeMoveBy

The code may crash at runtime due to NaN being created when calling NativeMapView::nativeMoveBy
I've just applied the same fix, which is already present in the most recent Mapbox codebase.
https://github.com/mapbox/mapbox-gl-native-android/blob/a82dcd9d8aa21dd9a285f319d33755d9333f917a/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/NativeMapView.java#L269

Thanks @schachi5000 ! As you're likely aware already, this has an open license that complies with ours, but the new mapbox/mapbox-maps-android doesn't, so we can't upstream from that.

The fix is small enough that I do not consider it license violation.


Added scaffolding for a CCP clearing mechanism.

This is a PR for adding additional scaffolding to support central counterparty settlement mechanisms.

This code is being migrated to the contracts-sandbox repo...


[VS Code] Starting collaboration session: 无法连接到服务器。请检查你是否已连接到 Internet。

Error:
Starting collaboration session: 无法连接到服务器。请检查你是否已连接到 Internet。
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
1.0.125

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.33.1

platformName
MacOS

platformVersion
18.5.0

Thanks for reaching out. This is a known issue, but all the bug reports help us understand the frequency of an issue and prioritizing our work. Did a retry fix the issue for you? Closing as a dupe of #1113.


Add iOS 13 support jailbroken by checkra1n

I installed this tweak on my iPhone 7 (iOS 13.1.3) jailbroken with checkra1n, and everything seems to be working except the bar gestures and the lockscreen toggles (they're out of bounds of the screen). I can use the lockscreen gestures but not the rest of the gestures.

I'm working with Ryan to do this.


about a lightweight variant of the ExU-Net

Hello, I saw that ExU-Net-L was mentioned in the article. But it seems that it is not given in the code you gave. Can you tell me what changes have been made based on ExU_Net?

Hello, thank you for your interest in my research.
I apologize for not providing information about the light version of ExU-Net mentioned in the paper.
In ExU-Net's argumet.py, I changed the parameters to the following and left the rest unchanged:
l_channel: [16, 32, 64, 128] -> [16, 32, 40, 64]
l_num_convblocks: [3, 4, 6, 3] -> [3, 3, 3, 3]
I will upload the parameters for the trained light version of ExU-Net as soon as possible.

We have uploaded the weights of the trained ExU-Net-L.
After defining the model and DDP, you can load the model weights based on the following code to get the same results as in the paper.
checkpoint = torch.load(args['path_bestmodel'], map_location=args['device'])
new_dict = {}
for o_key, n_key in zip(list(checkpoint.keys()), list(model.state_dict().keys())):
    new_dict[n_key] = checkpoint[o_key]
model.load_state_dict(new_dict)

Hi, I trained the model according to your guidance, but is there a special testing section? What if I want to test the trained model on other datasets?


How to save keypoints per frame in one json file like "BGpcy4xZKdQ.json"?

Code version (Git Hash) and PyTorch version
Dataset used
I want to create my own dataset.  In your training, do you use 240000 video files for training and 20000 video files for validation?
Expected behavior
Actual behavior
Steps to reproduce the behavior
Other comments

I understand now this. I can do it in your openpose.py. So, fixed.


能否添加自定义AI离线使用，比如说适配chatglm2 等大模型

The custom ai model was supported. Please refer https://github.com/chat2db/Chat2DB#configure-custom-ai to config your ai. You might need to read source code to get more information at this time.


Trim spaces and newlines in text fields

Extra newlines currently get added when a text to HTML conversion happens or when people copy paste text from somewhere else with extra spaces.
Two fixes needed:

Extra spaces in the beginning or end of a text fields need to be trimmed before submission.
Fix text to html code to trim newlines and spaces.

fixed in 0.2.0


[WIP] Support Python 3.12

Add support for Python 3.12.

TODO:

[ ] Unit tests and/or doctests in docstrings
[ ] Tests pass locally
[ ] Docstrings and API docs for any new/modified user-facing classes and functions
[ ] Changes documented in docs/release.rst
[ ] Docs build locally
[ ] GitHub Actions CI passes
[ ] Test coverage to 100% (Codecov passes)

The macOS Python 3.8 build has a weird error. Wonder if we should just drop Python 3.8 based on NEP 29. Raising issue ( https://github.com/zarr-developers/community/issues/65 ) to discuss

Builds work. Nearly all tests pass
However there is one CI error from the tests. It pertains to entrypoints.  Python 3.10+
>           entries.update(eps.select(group="numcodecs.codecs"))
E           ValueError: dictionary update sequence element #0 has length 3; 2 is required

numcodecs/registry.py:16: ValueError

cc @joshmoore @MSanKeys963 @rabernat (in case any of you have time to review)

Going to go ahead and merge to get the release out. Hope that is ok
Happy to discuss further as needed


Show in browser JavaScript error: Uncaught EvalError:

Description
Show in browser is JavaScript error: Uncaught EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "default-src 'self' http: https: data: blob: 'unsafe-inline'".
(https://git.huangdf.xyz/assets/js/index.js?v=1.18.0~rc1 @ 12:47048). Open browser console to see more details.
Screenshots

Gitea Version
1.18.0+rc1
Can you reproduce the bug on the Gitea demo site?
No
Operating System
win11
Browser Version
Microsoft Edge(107.0.1418.56)  Google Chrome( 107.0.5304.107)

You are serving Gitea through an proxy that adds a incompatible CSP header to responses.
CSP compatibility is a goal, but I don't expect it to be fully fixed anytime soon. See
https://github.com/go-gitea/gitea/issues/19851
https://github.com/go-gitea/gitea/issues/305


biblatex-ext-oa fails for retrieving information about open-access-status

Describe the bug
Error message when I try to get the status of a publication.
To Reproduce
\documentclass{scrartcl}

\begin{filecontents*}[overwrite]{testbib.bib}
@Article{wilkinson_fair_2016,
  Author          = {Wilkinson, Mark D. and others},
  Title           = {The {FAIR} Guiding Principles for scientific data
                    management and stewardship},
  JournalTitle    = {Scientific Data},
  Date            = {2016-12},
  DOI             = {10.1038/sdata.2016.18},
  Keywords        = {{FAIR}},
  langid          = {english},
  Number          = {1},
  Pages           = {160018},
  ShortJournal    = {Sci Data},
  URLDate         = {2021-05-31},
  Volume          = {3}
}
\end{filecontents*}

% using quotes esp. for biblatex.
\usepackage{csquotes}
\usepackage[%
backend=biber,
  style=ext-authoryear-ecomp,
  autocite=inline,
  defernumbers=true, % reset numbered bibliography for each chapter
  giveninits=true, % abbreviate the first names
  maxcitenames=1,
  mincitenames=1,
  maxbibnames=99, % show all authors of an bib entry
  dashed=false,
  ]{biblatex}
  % ==== open access publications
  % This allows us to automatically detect if a publication is
  % available open access (does not work always.)
  % In case you need to manually set the open access status
  % use `  options    = {openaccess=auto},`.
  \usepackage[
  doiapi=true,
  enable=true,
  symbolpackage=tikz,
  symbol=oanet]{biblatex-ext-oa}
% vv added
\ExecuteBibliographyOptions{openaccess=doiapi}
<EMAIL_ADDRESS><- this MUST be set for doiapi
% ^^ added

\addbibresource{testbib.bib}

  \begin{document}
  \nocite{*}
  \printbibliography
  \end{document}

Expected behaviour
I get the marker if the publication is available open access.
Output
(/usr/local/texlive/2023/texmf-dist/tex/latex/biblatex-ext/biblatex-ext-oa-doia
pi.sty.../2023/texmf-dist/tex/latex/biblatex-ext/blxextdoiapi.lua:41: module 's
ocket.http' not found:
	no field package.preload['socket.http']
	[kpse lua searcher] file not found: 'socket.http'
stack traceback:
	[C]: in function 'require'
	.../2023/texmf-dist/tex/latex/biblatex-ext/blxextdoiapi.lua:41: in main chunk
	[C]: in function 'require'
	[\directlua]:1: in main chunk.
l.40 ...tlua{blxextdoiapi = require("blxextdoiapi")}

Additional context
Same issue is also reported here: https://tex.stackexchange.com/questions/697757/biblatex-ext-oa-fails-for-retrieving-information-about-open-access-status

For security reasons LuaTeX no longer enables the sockets library by default (see https://tug.org/~mseven/luatex.html#luasocket for details). If you want to use code that uses sockets (like the open access detection), you now need to run LuaTeX explicitly with the option --socket. The MWE should work as expected when compiled with
lualatex --socket mwe
biber mwe
lualatex --socket mwe
lualatex --socket mwe

See also the following change to the docs on DEV https://github.com/moewew/biblatex-ext/commit/bfbd1af7824b42d6e637652342a1868a24c855e1.

Thank you! I have missed that change. For anyone interested in getting this up and running using latexmk, I found out this works:
latexmk -lualatex -lualatex="lualatex --socket" biblatex.tex

Unfortunately this makes it impossible to use this package on Overleaf since (to my knowledge) options cannot be passed to the compiler.
Could a possible workaround be created?

I'm Tom from Overleaf Support and I was made aware of this discussion by a user.
You can indeed pass options to the compiler using a latexmkrc file. (I believe that the right way would be `$lualatex = 'lualatex --socket $O %S', but I haven't tested.)
However, the problem is that we isolate the compiler so you cannot access the web from within it -- the socket operations are doomed to fail. This is not something we would be considering changing as it has potentially serious security implications.
I hope this helps clarify.

I'm Tom from Overleaf Support and I was made aware of this discussion by a user.
You can indeed pass options to the compiler using a latexmkrc file. (I believe that the right way would be `$lualatex = 'lualatex --socket $O %S', but I haven't tested.)
However, the problem is that we isolate the compiler so you cannot access the web from within it -- the socket operations are doomed to fail. This is not something we would be considering changing as it has potentially serious security implications.
I hope this helps clarify.

Thanks for the update Tom.
By the way, I believe that the right way would be
$lualatex = 'lualatex --socket %O %S'

I don't think there is a workaround in general and as Tom confirms there (probably) is not going to be an Overleaf-specific workaround either.
It was seen as security risk to enable sockets by default in LuaLaTeX. That means that in order to get internet access, which we obviously need to obtain the relevant open access data, we need to run LuaLaTeX with the --socket option. Any workaround (like enabling shell escape and asking curl to download stuff for us) would have to be classified as at least as unsafe as using --sockets. I'd therefore expect any workaround to be at least as difficult/impossible as using --socket directly.

BTW: If the open access API requires HTTPS connections (like the DOI.org API did a while back), this whole thing might collapse again, since last time I checked, LuaTeX did not contain the required libraries to issue HTTPS requests (we can only do HTTP).

I'm Tom from Overleaf Support and I was made aware of this discussion by a user.
You can indeed pass options to the compiler using a latexmkrc file. (I believe that the right way would be `$lualatex = 'lualatex --socket $O %S', but I haven't tested.)
However, the problem is that we isolate the compiler so you cannot access the web from within it -- the socket operations are doomed to fail. This is not something we would be considering changing as it has potentially serious security implications.
I hope this helps clarify.

Thanks for the update Tom.
By the way, I believe that the right way would be $lualatex = 'lualatex --socket %O %S'

You're welcome, and thanks for correcting, I also edited to minimize the amount of wrong code flying around (sorry, it was Sunday night after a working weekend...)


[hse] Add extractor

Original PR: 27060
Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[ ] Improvement
[x] New extractor
[ ] New feature

Description of your pull request and other information
Extractor to pull videos from home-shopping site "hse.de". Both product videos (/dpl/p/product) and show recordings (dpl/c/tv-shows) are supported. Extractor extracts and lists all available formats (typically low, mid, high resolution/bitrate). Tests passed on Python3.9.

Before I review the second half, read the CONTRIBUTING.md and make necessary changes

Awesome, so happy to see someone finally taking the time to actually review PR's for ytdl! I've just pushed the changes relating to each of the review points. Thank you @pukkandan !

Thank you @Ashish0804 for your review! Loving the effort you guys are putting into this new project.

Merged in https://github.com/yt-dlp/yt-dlp/commit/ae43a4b9868798097bb1420336294a2a2571be5f
In future, please don't disable "allow maintainers to edit"

@pukkandan Thanks! Noted. Also really loving the edits and underlying code refactoring! Good job.


line wrapping the logs of netopeer2-server

Hi @michalvasko,
netopeer2-server -d -c MSG command is not doing a line wrap of the log messages. So, the data is getting truncated at the end of the screen.
But netopeer2-cli with verb 3 is doing a line wrapping and i could get all the data properly.
Is there any configuration which needs to be changed to get all the logs from netopeer2-server?
Regards,
Sukesh

There is no configuration but it should not be needed, your terminal or whatever application displaying the output should do the wrapping. If it is not, I suggest you try another one.


It seems that Benchmark is currently inaccessible.

It seems that Benchmark is currently inaccessible. The link can not be opened.

It's a huggingface error.


[test] moving intel_inspector to external repo

Fix for UES-720

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1178   +/-   ##
=======================================
  Coverage   91.99%   91.99%
=======================================
  Files          81       81
  Lines       11693    11693
=======================================
  Hits        10757    10757
  Misses        936      936

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 3d5ddb7...34d035b. Read the comment docs.


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[x] Tests pass
[x] Appropriate changes to README are included in PR

Update validations.py script Pyhton.


Update reload.js

May be ignored, files can be now reloaded with aliases.
BUT assumes file name is the same as the command name.

Fixed .


Page.loadEventFired not triggered on redirect

I'm not sure if this is an issue with the dev tools or with this library; but it seems that loadEventFired and other events such as frameNavigated are not triggered when a form is submitted and/or the page is redirected.
I have a page with a login form (https://fabric.io/login); I submit this form, and then the page is redirected to another address. No events are triggered for the "main" frame; I only receive various "frameNavigated" events for iframes etc.

Okay, nevermind.. It turns out that the website in question is using the pushState API, and there really is NO reload happening :)
=> I'm closing the issue.


Add watch to npm

This way I can develop without installing lerna globally.

Thanks for this!  It looks like the command should be lerna run --parallel watch based on this issue comment: https://github.com/lerna/lerna/issues/449#issuecomment-324837965

(and don't worry about the Travis failure, we technically haven't set it up yet pending #49)

@blink1073 Updated.
You can set travis to not build if there is no travis.yml file.

Our scripts are just watch (not build:watch): https://github.com/jupyterlab/jupyter-renderers/blob/09872632fc6786d8ea3539b43269a1545b9ac2be/packages/geojson-extension/package.json#L31

Thanks, toggled the setting.

Thanks. Works now.

Cheers!


typo

Thanks! My typos are a liability. 🙈

no worries. thanks for the awesome library and keep up the good work!


Oathbreaker not progressing past level 6

Describe the bug
I have been trying to play Oathbreaker and as soon as you have to choose a defensive power it will not progress. Tried multiple times but it just will not let me hit next

Fixed in the next update.


Add test for pip installations

This commit adds Dockerfiles and test script to test all variants of MXNet pip packages. I've tested this on a copy of the Jenkins slave host with an updated NVIDIA driver (375.26) with the following command:
tests/jenkins/run_test_pip_installations.sh ${WORKSPACE} where WORKSPACE was set to /home/ec2-user/mxnet, the Jenkins workspace.
Currently the test will fail after testing mxnet with Python 3.6 as the accuracy is lower than expected. We can either put that case at the very end or change the script to not fail until all cases have been tested.
@sandeep-krishnamurthy

nosetest failing possibly because I added arguments to the test_conv.py script. Debugging and will push the changes after.

@szha should we put the pip tests into the pip build repo?

It would be nice to include tests after the build of each variation. The blocker was the Travis build time-limit. We will need to move the release to Jenkins if we want tests like this pr. Currently I manually test all the releases with unit tests.

if we have mac on Jenkins we can move the release to Jenkins

@mli @szha @piiswrong we don't have mac on Jenkins right now and can't prioritize adding that. Can we push this for now so we can run a nightly test?
I can organize this test script under tests/jenkins/pip/ folder if that helps.