Feat/issue#2046

[x] Make submit trade button the color of the trade side
[x] Make the submit button in the confirmation view the color of the trade side
[x] Update confirmation submit button labels to include side, e.g. "Submit Short Order"
[x] Use the colored trade side label in the trade confirmation view
[x] Update the confirmation view layout

continued in #2062


Pathfinding cleanup

It's been too long
Requires https://github.com/space-wizards/RobustToolbox/pull/2828

[ ] ECSify everything
[ ] Genericise it like atmos so using a navmesh isn't terrible
[ ] Document stuff more
[ ] Structs!
[ ] Parallelism
[ ] More pathfinding args available e.g. for mobs that can break doors
[ ] More tests

I have been distracted with another sidequest by the sergal


Error in Specto when changing movie and series directory to Nas location

Mr Know,
Using the latest version 2<IP_ADDRESS> but still get an error message as soon as I want to change the library directory to a folder on my NAS.
Tried to link as ftp or NFS, both without any luck.
As soon as I change it back to the default folders it resolves the error message.
Copied in the error message below.
22:40:51 T:1456   ERROR: XFILE::CDirectory::GetDirectory - Error getting plugin://plugin.video.specto/?action=toolNavigator
22:40:51 T:1456   ERROR: CGUIMediaWindow::GetDirectory(plugin://plugin.video.specto/?action=toolNavigator) failed
22:40:51 T:4612   ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
- NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
Error Type: <type 'exceptions.WindowsError'>
Error Contents: [Error 123] De syntaxis van de bestandsnaam, mapnaam of volumenaam is onjuist: 'ftp:'
Traceback (most recent call last):
File "C:\Users\Maverick\AppData\Roaming\Kodi\addons\plugin.video.specto\default.py", line 126, in
navigator.navigator().root()
File "C:\Users\Maverick\AppData\Roaming\Kodi\addons\plugin.video.specto\resources\lib\indexers\navigator.py", line 56, in init
os.makedirs(tv_library)
File "C:\Program Files (x86)\Kodi\system\python\Lib\os.py", line 150, in makedirs
makedirs(head, mode)
File "C:\Program Files (x86)\Kodi\system\python\Lib\os.py", line 150, in makedirs
makedirs(head, mode)
File "C:\Program Files (x86)\Kodi\system\python\Lib\os.py", line 150, in makedirs
makedirs(head, mode)
File "C:\Program Files (x86)\Kodi\system\python\Lib\os.py", line 157, in makedirs
mkdir(name, mode)
WindowsError: [Error 123] De syntaxis van de bestandsnaam, mapnaam of volumenaam is onjuist: 'ftp:'
-->End of Python script error report<--
Hope you can help.
Regards
Maverick

I having the same problem, it was working until the most recent update.
File "/storage/emulated/0/Android/data/org.xbmc.kodi/files/.kodi/addons/plugin.video.specto/resources/lib/indexers/navigator.py", line 56, in __init__                                                 os.makedirs(tv_library)                                               File "/home/jenkins/workspace/Android-ARM/tools/depends/xbmc-depends/arm-linux-androideabi-android-17/lib/python2.6/os.py", line 150, in makedirs                                               File "/home/jenkins/workspace/Android-ARM/tools/depends/xbmc-depends/arm-linux-androideabi-android-17/lib/python2.6/os.py", line 150, in makedirs                                               File "/home/jenkins/workspace/Android-ARM/tools/depends/xbmc-depends/arm-linux-androideabi-android-17/lib/python2.6/os.py", line 150, in makedirs                                               File "/home/jenkins/workspace/Android-ARM/tools/depends/xbmc-depends/arm-linux-androideabi-android-17/lib/python2.6/os.py", line 157, in makedirs                                             OSError: [Errno 30] Read-only file system: 'nfs:'                                             -->End of Python script error report<-- 11:32:42 T:1390168368   ERROR: static bool XFILE::CDirectory::GetDirectory(const CURL&, CFileItemList&, const XFILE::CDirectory::CHints&, bool) - Error getting plugin://plugin.video.specto/ 11:32:42 T:1390168368   ERROR: CGUIMediaWindow::GetDirectory(plugin://plugin.video.specto/) failed

I am also seeing this issue.  Set the library locations to a SMB share, and Specto now crashes when opening.  Just installed Specto and was attempting to setup with shared network library, and as soon as I did, it crashes the addon.  It keeps crashing each time I try to open it.
Running on: Amazon FireTV 4K, SPMC Isengard, plugin version 2<IP_ADDRESS>

Try new version 2<IP_ADDRESS>

I am running specto 2<IP_ADDRESS> kodi 16.1 on both win 10 and android 5.1. As soon as I change library folder location to a network share specto crashes and wont launch again until library is moved to a local drive instead of network location.


Silence warnings that as_any and next_batch are never used

Which issue does this PR close?

Closes #6143.
Rationale for this change

What changes are included in this PR?
See discussion in issue.

Are there any user-facing changes?
No

As expected, no warnings showed up according to issue #6143. It looks good to merge this. cc @alamb


Hutto Middle School

@Mace12409 Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes to propagate through our system.


Finish translating API Reference > ReactDOM to Bengali

Some words needed to be kept in English otherwise they were making the sentences hard to understand. Please review the translation and let me know if anything needs to be fixed or updated.

Updated everything as suggested. :)

Welcome @nutboltu. Glad that I could help. :smile:


Vue example

It took me quite a while, but I've finally managed to find some time and provide a sample of a frontend app written 100% in Vue.js.
Long story short, I've been trying to rewrite what you guys have implemented in vanilla HTML + CSS + js and pack it into a single page app (except for a few places where I was a bit too lazy to do things exactly as I should). Some things are mocked out, so don't expect the whole app to work - it's just a client making fake requests to a fake service.
Lessons learned:

Vue is kinda cool
CSS is just black magic
I'm really bad at time management

It's probably not going to too useful, but in case you've come to hate frontend development because of all that manual HTML and CSS management, well, here's how it's done (in a simplified form) these days. Some things are much easier this way, there's still plenty of things to drive people crazy, but, all in all, it's a bit more manageable and, most importantly, you can write unit tests.
Either way - don't worry too much about this PR - it can hang here forever. I just had some spare time during Christmas so I've decided to finally send a PR I've been talking about.
P.S. Happy new Year!

Nice to hear from u!
I will look through it during winter holidays (at least, I hope so)
P.S. Happy New Year to you too!)


Adding Range queries algorithms

They are not considering dsa,cp,algorithm repository :(
all the pr will not be considered in hacktoberfest but your contribution will be for lifetime, this will help many students to practice dsa .
Thank you for your contribution :)
you can contribute to other repo


Hi flesler

I have copied your demo to localhost server. but ajax not working, it stop loading. scroll to work, but ajax not work. why so. help me thanks

Hi, these kind of inquiries you should ask on a forum like Stack Overflow where many more people might reply to you. I keep GH issues for feature/bug reports


AVRO-3056: Bump zstd-jni from 1.4.8-4 to 1.4.8-6 in /lang/java

Make sure you have checked all steps below.
Jira

[x] My PR addresses the following Avro Jira issues and references them in the PR title. For example, "AVRO-1234: My Avro PR"

https://issues.apache.org/jira/browse/AVRO-3056
In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
This is a dependency update and it should pass all the existing unit tests.

Commits

[x] My commits all reference Jira issues in their subject lines. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain Javadoc that explain what it does

cc @iemejia

I'm missed this PR, and already merged the one by Dependabot: https://github.com/apache/avro/pull/1111. However, huge thanks for pointing out the breaking compatibility, and providing a PR.

Thanks @dongjoon-hyun great to see this getting aligned here too. I assigned the ticket to you and let it open so we can track it as part of the 1.10.2 Avro release
CC @RyanSkraba for awareness

No problem. Thanks, @Fokko and @iemejia .


[Not A BUG]: How to set/adjust Image size

i'm using the function editor.setImage(imagePath) for input image.
i tried to crop the image into smaller size before setImage, but not work.
How we cannot  the image size after setImage. or  it, or centering...

https://github.com/user-attachments/assets/7e2da744-6070-451b-9293-3c6791aea981

do you provide further information about this screen code

Hey, there is currently no way to do this.
You can maybe check this out https://www.npmjs.com/package/tiptap-extension-resize-image and try get it to work with the advanced setup!


Fix battery temperature reporting invalid values in extreme temperatures

Description
Moving average logic has been fixed
Requirements

[x] Code style guidelines followed
[ ] Documentation updated

Tests 🧪

[ ] Robot
[ ] Container
[ ] Simulation

Works on real hardware, closing


Move card from a board to another does not work anymore

Hi,
in previous version i was able to move a card from a board to another, which is very handy to organise some task (at least, i found it handy).
There is a way to copy a card and paste it to another board and then delete the original, but in this workaround, attachments are lost.
Am i missing something?
BR,
lezioul

Hmm? Maybe related to https://github.com/wekan/wekan/issues/3377

I am admin, and i am not able to move cards from board to another anymore.
I can test with a "simple" user with another role.


cleanup install dialog

remove unused ui
update dialog text to use the correct application name

Hi @davidkline-ms, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (David Kline (XBOX)). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


replace go objects with dicts in wind streaming example to improve perf

should help resolve #331.
After profiling this app with py-spy, this app running with a 5 tabs open spent 98% of the gunicorn resources updating graphs and most of that time was spent by plotly.py validating them. Getting rid of the go objects reduced the time spent updating the graphs to 33%
Before (using go objects)

After (using dicts)

Issue for app: #[issue number here]
App pull request

[ ] This is a new app
[x] I am improving an existing app (redesigns/code "makeovers")

About

Playground deployment URL (new version):
Current gallery app URL: (delete this line if inapplicable)
Python app repository link: (delete this line if you are working on a Python app)

Workflow

[ ] I have created a branch in the appropriate monorepo, and the
elements necessary for successful deployment are in place.
[ ] If the app is a redesigned and/or restyled version of an
existing gallery app, I've summarized the changes requested in the
appropriate Streambed issue and confirm that they have been applied.
[ ] If the app is on the Dash Gallery portal, I have added a link to
the GitHub repository for the source code in the portal description.
[ ] If the app is a reimplementation of a Python gallery app for the
DashR gallery, the app in this PR mimics, as closely as possible,
the style and functionality of the existing app.=
[ ] I have removed all Google Analytics code from the app's
assets/ folder.

The pre-review review
I have addressed all of the following questions:

[ ] Does everything in my code serve some purpose? (I have removed
any dead and/or irrelevant code.)
[ ] Does everything in my code have a clear purpose? (My code is
readable and, where it isn't, it has been commented appropriately.)]
[ ] Am I reinventing the wheel? (I have used appropriate packages to
lessen the volume of code that needs to be maintained.)

Cool. @shammamah , would you be able to merge? I don't have privileges to merge to this branch.


play several sounds in sequence

Hi, just  short one. I'm trying to play 5 different samples with a time interval of 1s. I have a code like this one:
this.sound1 = new Pizzicato.Sound(file1.mp3, () => {
  this.sound1.play();
  setTimeout(() => {
    this.sound2 = new Pizzicato.Sound(file2.mp3, () => {
	  this.sound2.play();
	  setTimeout(() => {
		this.sound3 = new Pizzicato.Sound(file3.mp3, () => {
		  this.sound3.play();
		  setTimeout(() => {
			this.sound4 = new Pizzicato.Sound(file4.mp3, () => {
			  this.sound4.play();
			  setTimeout(() => {
				this.sound5 = new Pizzicato.Sound(file5.mp3, () => {
				  this.sound5.play();
				});
			  }, 1000);
			});
		  }, 1000);
		});
	  }, 1000);
	});
  }, 1000);
});

that's a really complex structure for a simple purpose. Is there any other way to play back a sequence of sounds?

For all those ones interested: as there is no other way, it makes definitely sense to create a rekursive function, as the structure above is obviously a rekursion. Therefore you could code something like this:
    public playSounds(soundpaths: string[]) {
        if (soundpaths.length > 0) {
            const sound = new Pizzicato.Sound(soundpaths[0], () => {
                sound.play();
                if (soundpaths.length === 1) {
                    sound.on('end', () => {
                        console.log("Sequnce of sounds finished!");
                    });
                } else {
                    soundpaths.shift();
                    setTimeout(() => {
                        this.playSounds(soundpaths);
                    }, 1000);
                }
            });
        }
    }

This will play back all sounds in intervals of 1s. You just pass an array with all paths as an argument.


allow bip44 address option when creating wallet

Torba and lbrycrd use a different standard (bip44 vs bip32) so private keys do not produce the same set of addresses. I understand we don't want the bip44 method to be the default, but there should be an option to do so when creating a wallet.

This is potentially a relevant comment from Optech:
"Bitcoin Core #15427 extends the utxoupdatepsbt RPC with a descriptors parameter that takes an output script descriptor and uses it to update a BIP174 Partially-Signed Bitcoin Transaction (PSBT) with information about the scripts (addresses) involved in the transaction. This is in addition to the RPC’s previous behavior of adding information to the PSBT from the node’s mempool and UTXO set. This new feature is especially useful for hardware wallets and other paired wallets as it makes it possible to add HD key-path information to the PSBTs so that wallets asked to sign a PSBT can easily derive the keys needed for signing or verify that a change output does indeed pay back into the wallet."

There are off-the-shelf wallets that support BIP44. Are any of those sufficient to cover this need? If not, is it because of a lack of LBRY support in a particular wallet? Would we do better making a certain 3rd-party wallet work with LBRY?

If we don't want to ever support the same keys on torba and lbrycrd, then we can just close this.
Maybe new torba wallets should use the format to match lbrycrd? Not sure if that makes things difficult to track with the two formats. @eukreign ?

Relevant:
https://github.com/bitcoin/bitcoin/pull/15414
https://github.com/bitcoin/bitcoin/pull/8723

Pasting in from the Optech news letter:

Bitcoin Core #16528 allows the createwallet RPC to create a wallet that uses output script descriptors to derive just the particular scriptPubKeys that the wallet uses to receive payments. This is a major improvement over the way legacy wallets scan for payments by deriving every type of script handled by the wallet for each public key in the wallet—a technique that dates back to the original Bitcoin 0.1 release’s support for receiving to both P2PK and P2PKH scriptPubKeys. Descriptor wallets should be more efficient (because they don’t need to scan for unused script types), easier to upgrade to new types of script (e.g. for taproot), and easier to use with external tools (e.g. multisig wallets or HWI-compatible hardware wallets via partially-signed bitcoin transactions).
By default, descriptor wallets use the popular BIP32 HD wallet paths specified by BIPs 44, 49, and 84 rather than the non-standardized fully-hardened path used in legacy Bitcoin Core HD wallets. A number of wallet RPCs are unavailable with descriptor wallets, either because they don’t make sense with descriptors or because developers are still adapting them to new edge cases. The merge of this PR early in the 0.21 development cycle and the decision to make descriptor wallets a non-default option will give the new features six months to mature before their expected release.


ParseFile Error: Expected '(', found '-'

Describe the bug
I am generating a cookiecutter project in Golang. We have several prompts, one of them being the service name. We are allowing the developer to input values that match this regex r'^[a-z0-9-]*$'. Cookiecutter takes the service name and uses it as a file name. Once the developer goes to generate the project the project fails with this error:
ParseFile error:path/to/file/with-hyphens.go: expected '(', found '-'
I can't find any good information online about this error. Has anyone ran into this issue before? Is this by design?
Expected behavior
I should expect that a file name with hyphens would not fail when parsing.


model: llama3.1

Goal

llama3.1
with Function Calling

This ticket also resolve issue #295

Here is python script that run function calling, GitHub block .py file, so I cannot upload it.
import requests, json

ENDPOINT = "https://litellm.jan.ai/v1/chat/completions" # "http://localhost:3928/v1/chat/completions" #
MODEL = "alan-gift" # "meta-llama3.1-8b-instruct" #

grammar = """
root   ::= object
value  ::= object | array | string | number | ("true" | "false" | "null") ws

object ::=
  "{" ws (
            string ":" ws value
    ("," ws string ":" ws value)*
  )? "}" ws

array  ::=
  "[" ws (
            value
    ("," ws value)*
  )? "]" ws

string ::=
  "\"" (
    [^"\\\x7F\x00-\x1F] |
    "\\" (["\\bfnrt] | "u" [0-9a-fA-F]{4}) # escapes
  )* "\"" ws

number ::= ("-"? ([0-9] | [1-9] [0-9]{0,15})) ("." [0-9]+)? ([eE] [-+]? [0-9] [1-9]{0,15})? ws

# Optional space: by convention, applied in this grammar after literal chars when allowed
ws ::= | " " | "\n" [ \t]{0,20}
"""

system_prompt = """
Environment: ipython
Tools: brave_search, wolfram_alpha
Cutting Knowledge Date: December 2023
Today Date: 20 September 2024

# Tool Instructions
- Always execute python code in messages that you share.
- When looking for real time information use relevant functions if available else fallback to brave_search

You have access to the following CUSTOM functions:

Use the function 'spotify_trending_songs' to: Get top trending songs on Spotify
{
  "name": "spotify_trending_songs",
  "description": "Get top trending songs on Spotify",
  "parameters": {
    "n": {
      "param_type": "int",
      "description": "Number of trending songs to get",
      "required": true
    }
  }
}

Use the function 'get_current_conditions' to: Get the current weather conditions for a specific location
{
    "type": "function",
    "function": {
    "name": "get_current_conditions",
    "description": "Get the current weather conditions for a specific location",
    "parameters": {
        "type": "object",
        "properties": {
        "location": {
            "type": "string",
            "description": "The city and state, e.g., San Francisco, CA"
        },
        "unit": {
            "type": "string",
            "enum": ["Celsius", "Fahrenheit"],
            "description": "The temperature unit to use. Infer this from the user's location."
        }
        },
        "required": ["location", "unit"]
    }
    }
}

If a you choose to call a CUSTOM function ONLY reply in the following format:
<{start_tag}={function_name}>{parameters}{end_tag}
where

start_tag => `<function`
parameters => a JSON dict with the function argument name as key and function argument value as value.
end_tag => `</function>`

Here is an example,
<function=example_function_name>{"example_name": "example_value"}</function>

Reminder:
- Function calls MUST follow the specified format
- Required parameters MUST be specified
- Only call one function at a time
- Put the entire function call reply on one line
- Always add your sources when using search results to answer the user query
- If can not find correct parameters corresponding to function, ask user again to provide.
- No explanation are needed when calling a function.

You are a helpful assistant.
"""
user_prompt = "Who is US president in 2024"
system = {"role":"system","content":system_prompt}
user = {"role":"user","content":user_prompt}

messages = [system,user]
body = {
    "model": MODEL,
    "messages": messages,
    "top_p":0.9,
    "top_k":40,
    "temperature":0.6,
    "stop" : ["</s>","<|eot_id|>"],
    "grammar":grammar,
}

result = requests.post(ENDPOINT, json=body,headers={'content-type': 'application/json'}).json()
print(json.dumps(result,indent=4))
assitant = result["choices"][0]["message"]
users2 = {"role":"user","content":"Maybe CA"}
# ipython = {"role":"ipython",""}
messages = [system,user,assitant,users2]

body = {
    "model": MODEL,
    "messages": messages,
    "top_p":0.9,
    "temperature":0.6,
    "stop" : ["</s>","<|eot_id|>","<|eom_id|>"],
    "grammar":grammar,
}
result = requests.post(ENDPOINT, json=body,headers={'content-type': 'application/json'}).json()
print(json.dumps(result,indent=4))

cc @dan-homebrew @0xSage


useMap always return null for me

Description
any ideas about why useMap is returning null instead of map context?
my map is working ok but i need no use useMap from react
Steps to Reproduce
...
Environment
...
Logs
No response

Do you have a code-snippet for me? useMap returns null if it is used outside the APIProvider component (since it can't lookup map-instances in that case) or when the map with the specified id can't be found (currently, not specifying an id will use the id "default", which is also used by the map if you don't specify the id prop).
See: https://github.com/visgl/react-google-maps/blob/main/src/hooks/use-map.ts

i'm using it inside de APIProvider
`
import { useEffect } from "react"
import { APIProvider, Map, useMap } from<EMAIL_ADDRESS>export default function Mapa() {
const mapa = useMap()
useEffect(() => {
console.log("mapa *", mapa)
}, [mapa])
...
return (

<Map
mapId="rutas"
mapTypeId="terrain"
...
>

)
}
`

the console.log is always returnig null

Here's the problem: useMap isn't actually called inside the APIProvider, the APIProvider is rendered as a child of your Mapa component.
I would recommend to place the APIProvider outside of this component, could be as far up as far up as your App or page component. It doesn't impact the rest of the application in any way.
The alternative would be to place the component using the hook into a child-component like this:
function SomethingWithMapHook() {
  const map = useMap();
  useEffect(() =>{
    if (!map) return;

    // whatever you need to do with the map
  }, [map])

  return <>whatever</>;
}

export default function Mapa() {
  const mapa = useMap()

  useEffect(() => {
    console.log("mapa *", mapa)
  }, [mapa])

  // ...

  return (
    <APIProvider apiKey="...">
      <Map {...mapProps}>
        ...
      </Map>
      <SomethingWithMapHook />
    </APIProvider>
  )
}


chromium: prefer VMS with 'big-parallel'

Motivation for this change
IIRC, hydra's "big-parallel" machines have 32 cores instead of 2. Should help with ~60 hr build times. Unless this causes timeout/OOm issues
may close: #81081
related: #78347
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

ah didn't realize. The chromium derivation is hard to tell what's going on, there's 3 files which facilitate it's existence.

But does ungoogled-chromium inherit the same setting?


Bind runtime slices to queries

When using execDynamic, you can have a runtime-known number of parameters in the query, however the current way of binding the parameters requires the values to be comptime-known.
This PR allows slices of form []T to be passed as the values parameter, binding it to the statement with a runtime loop instead. Not sure how I should approach setting the field_name parameter to bindField, as it is comptime (so I'm just setting it to empty string). Any suggestions?
This was manually tested, but if this idea moves forward on being upstreamed, I plan to add it to the test suite.

I like this feature, thanks for working on it ! Overall LGTM, only one small comment. With some tests I think it will be ok to merge.
As for this question:

Not sure how I should approach setting the field_name parameter to bindField, as it is comptime (so I'm just setting it to empty string). Any suggestions?

IIRC it's only used for debugging in comptime errors, in the case of a dynamic slice it's doesn't make sense so something like "unknown" is fine.

Looks good ! One last thing, could you remove the commit 9ef7561e02a0b4b08b13a6836202c0b13b509948 from the PR and then I can merge


Add another test for ClassLoaderObjectInputStream::resolveProxyClass

Currently, ClassLoaderObjectInputStreamTest::testResolveProxyClass only tests ClassLoaderObjectInputStream::resolveProxyClass with single-element array. This PR adds a new test to check for the case where the Proxy class is assignable from multiple interfaces, and that the Proxy class is only a proxy for the interfaces that it implements.

Coverage increased (+0.04%) to 90.094% when pulling 4d5e45cdb44f61202ce439caa3d417813375f9b7 on testingsavvy:master into 733dc26a281e235d52c965b6053a4e16bd4f85de on apache:master.


b1.3

Build against Scala 2.11 finished with success

Build against Scala 2.10 finished with success

+1


Create LICENSE.md

[Rule key](Rule link)
Description
...
Reasoning
...

@WarbyParker/frontend-developers
One rule per PR

MIT in place of Apache 2.0?

Apache was right


CachedResponse is unpicklable when history is non-null

When history exists for a ClientResponse object, the corresponding CachedResponse cannot be pickled when saving to the backend, since CachedResponse.history is being saved as a generator. I think the intention was to save it as a tuple, but the current syntax used represents a generator expression. Even if it's modified to become a tuple, it's still unpicklable since history will be a tuple of coroutines unless we await the recursive calls. Will try to get a PR out to fix this.

@Jophish Cool, thanks! I'm surprised someone found this so quickly; I've only worked on this for about a week. Redirect history is one of the many things I hadn't tested yet, so thanks for saving me some time there.

Of course! I just migrated a project from requests and requests-cache to aiohttp and was surprised there wasn't a ubiquitous caching library for it, and then stumbled on this -- seemingly perfect timing!

@Jophish Same here, I was surprised this didn't already exist!
I'm spending some time this weekend cleaning up some of the half-finished bits and tasking out the remaining work to do. If you're interested, feel free to pick up any of the open issues, or anything else you can identify that needs to be done. Or, if you'd like to review what I've done so far, a second pair of eyes on this code would be helpful.


Documentation: draw_image is missing the image parameter

I found an error in the documentation, the draw_image function is missing documentation on its second parameter, const Image* image.
https://nappgui.com/en/draw2d/draw.html#f29
draw_image_frame is also missing this parameter as well.

Thanks for the report @stoneface86
I will fix in next documentation review

Fixed
https://nappgui.com/en/draw2d/draw.html#f29
https://nappgui.com/en/draw2d/draw.html#f30


python3Packages.onnx, python3Packages.skl2onnx: fix build

Description of changes
Fix build failure by loosen dependency version requirements.

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

CC: @acairncross for testing python3Packages.onnx

Result of nixpkgs-review pr 197194 run on x86_64-linux 1

10 packages built:

python310Packages.ffcv
python310Packages.onnx
python310Packages.onnxconverter-common
python310Packages.pytorch-pfn-extras
python310Packages.skl2onnx
python39Packages.ffcv
python39Packages.onnx
python39Packages.onnxconverter-common
python39Packages.pytorch-pfn-extras
python39Packages.skl2onnx

Could you check whether it is possible to enable tests for skl2onnx.

I still get Segmentation fault.

Note there is also the pythonRelaxDepsHook for relaxing these constraints.


only trigger deprecation warnings for non-empty attributes

A deprecation that targeted the use of an attribute would always trigger, now we're only checking for deprecations on non-empty attributes.

I'm wondering if it might still be valuable for the caller to be aware of the deprecated attribute, even when it’s empty. Wouldn't it help guide users away from using deprecated fields altogether?

Well, the problem is that there is no way to distinguish between an empty attribute and an unset attribute, so once you unconditionally deprecate an attribute the warning would always trigger.


[Feature] Show multiple network interfaces

Description of the feature
As far as I can tell, it only shows 1 interface currently.
I'd like to view more or at least an option to select which one to show.
As for now it defaults to my 10G NIC which is internal link with another server.
Additional context
No response

Thanks for the request! I will add an option to select your desired interface.

:tada: This issue has been resolved in version 3.4.0
Please check the changelog for more details.


[BUG] Inaccurate code coverage

Hi @cenfun,
When instrumenting playwright tests for an angular app using istanbul via the babel-plugin-istanbul plugin the code coverage shows accurately whilst when using V8 + monocart-reporter some lines that should be covered show as uncovered even though there's an indication those lines have been executed X number of times.
The images below show a part of the code coverage for a file which show the problem.
As per our conversation on Playwright discord channel I've created a repo which allows you to easily reproduce the problem. Please see edumserrano/monocart-code-coverage-demo.

Istanbul code coverage report:

Monocart code coverage report:

It should be fixed, please try new version<EMAIL_ADDRESS>The root cause is that the original location cannot be found from the sourcemap sometimes.
Sourcemap matching is always a problem, especially when some non-native js syntax is used.
For example, the following problem still cannot be solved because there is a wrong match in the sourcemap:

we can find out the mismatch with tool: https://evanw.github.io/source-map-visualization/

It seems that sourcemap related issues should be traced back to compilation tools like babel.
Thanks for such a detailed description of the problem, more suggestions are welcome.

please update the config, the previous sourceFilter is not right

Awesome! All is working 🎉  and you made it even better by not having the webpack:/ at the start so now it starts with just src 👏
I do have a followed question. In the coverage details fly out could we also remove the webpack mention from there? I'd assume it would be related with the fix you did to remove webpack mention from the summary tree ?

no problem.
actually, the webpack:/xxx is the url of the source file, and it will be converted to the path of the source file,
If we don't like url but prefer path, we can use path instead of url next time

I can't be sure if that change makes sense for all cases but I think it does at least for this scenario. I'd like to have it without webpack, I do think using the sourcePath` makes more sense but I'll leave that up for you to decide.
To help others, I'll create a branch on the demo repo that contains the fix and update the README to mention the fix is on the branch:

update to 1.6.29
update the playwright.config.ts

I think you can close this issue whenever you want @cenfun . Thank you for sorting this out and so quickly 🥇

just removed the webpack:/// see commit: https://github.com/cenfun/monocart-reporter/commit/1e9542cb945a72e95af4b3752148944fb38661ba
I prefer to have it without webpack:/// too.
I understand that you spend a lot of time on the code coverage, so do I.
But currently few users use this code coverage feature, suggestions are welcome.
Thanks

This might be a stupid question but the change you mentioned from commit 1e9542c  will be in a new version right? 1.6.30 ?
Just checking cause I've been refreshing https://www.npmjs.com/package/monocart-reporter waiting for the new version 😆

yes, 1.6.30 will be released when few minor issues are fixed. sorry for the later.

1.6.30 have released. please have a try.

I can confirm version 1.6.30 is working as expected. All looking good 👏

@cenfun found some more examples where code coverage from V8+monocart-reporter is incorrect when comparing with coverage using babel+istanbul. Perhaps we should reopen this issue.
I've had to expand the code of the demo repo and created a branch named temp edumserrano/monocart-code-coverage-demo/tree/temp.

You can see a difference in for instance:

src/app/features/find-institution/find-institution.view-model.ts
src/app/features/find-institution/recently-used/recently-used-institutions.service.ts

In both of the above files, the babel+istanbul shows correct code coverage as opposed to the V8+monocart-reporter version.  See images below:

babel+istanbul find-institution.view-model.ts

V8+monocart-reporter find-institution.view-model.ts
The return statement that shows as uncovered should show as covered.

babel+istanbul recently-used-institutions.service.ts

V8+monocart-reporter recently-used-institutions.service.ts
The moveToFront() function and the if/else branch above show as uncovered in this image should show as covered.

Another example that shows incorrect coverage is on an html file. For this I can't compare with babel+istanbul because that setup is only capturing code coverage for TypeScript files. However, the image below from the V8+monocart-reporter setup is for the src/app/features/find-institution/institutions-list/institutions-list.component.html file and it shows as such:

Both of the above red lines should show as covered. And, similarly to the original bug reported on this issue, notice that one of the lines that shows as uncovered as the indication that it has been executed 5x.

Note

there are many tests now in this branch, it make take a few seconds or a minute to complete.
many are screenshot tests which will only work the second time you run then because I haven't pushed the screenshots to the repo. And even if you rerun there are 2 tests that will always fail (that is expected).
this is a temp branch to show the problem and it contains an expanded version of the code in main branch. However I'm not comfortable with leaving this branch available forever so I will be deleting it once you're able to replicate and diagnose the issue.

Let's check if v8 coverage is correct in dist file first

set excludeDistFile: false and run test, the dist file main.js will be in report

find out what the line is in dist file for line 102 in find-institution.view-model.ts (uncovered)

download http://localhost:4999/main.js and http://localhost:4999/main.js.map when server is startup.
open them in the source-map-visualization and find out the line is 11135 in dist file:

check the line 11135 and it is uncovered, so the v8 coverage should be correct.

I don't think we can directly compare Istanbul and V8 coverage like you do.

the runtime code is different even the source code is the same, but the dist file is different
the tests are not executed exactly the same, for example, playwright execute the tests in parallel, test1 may be executed before or after test2
whether the page is reloaded during testing? coverage data may need to be merged. I don't know the details but it looks complicated.

Is it possible that Istanbul is wrong and V8 is correct? maybe need more verification.

confirmed it is a bug. the uncovered lines have not mered with covered lines some times. thanks for the example provided.

It's so hard to find the root cause.
First, upgrage to<EMAIL_ADDRESS>I refactored the coverage related codes.
But still needs some changes to auto fixtures, adding one more option resetOnNavigation: false
await Promise.all([
    page.coverage.startJSCoverage({
        resetOnNavigation: false
    }),
    page.coverage.startCSSCoverage({
        resetOnNavigation: false
    }),
]);

This is because there are multiple pages in one test, the coverage data will be reset if navigation by default
test(`recently used @find-institution @recently-used`, async ({ page }) => {
    const findInstitutionPage = new FindInstitutionPage(page);
    const chooseSignInPage = new ChooseSignInPage(page);

see resetOnNavigation

The issue on the HTML has been fixed with 1.6.34
Before

After


Corrections and Update Needed

Typo under "Prerequisites" > "An IDE — Gogland by Jetbrains"
------- Gogland should be GoLand
Under "Clone the sample application" > "The mgo driver (pronounced as mango)..."
------- mgo will no longer be maintained and is outdated
https://www.compose.com/articles/mongodb-and-go-moving-on-from-mgo/

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

ID: c0704a6e-bdc1-9834-3975-e883360d22f9
Version Independent ID: a5fc5945-b27b-7c7a-2ad3-5f3ff3b63e5a
Content: Azure Cosmos DB: Build a MongoDB API console app with Golang and the Azure portal
Content Source: articles/cosmos-db/create-mongodb-golang.md
Service: cosmos-db
GitHub Login: @slyons
Microsoft Alias: sclyon

@kimlisa thanks for your feedback, I have fixed the typo, changed it to GoLand. WRT to the mgo driver that is used in this doc, we are in the process of revamping most of Cosmos DB docs. We are currenlty working on concepts and how-to guides and will get to Quickstarts and tutorials soon. We will update this doc with the new compatible drivers.
If future, if you see any minor typos, feel free to fix them, we love to take public contributions. You can submit a fix by clicking on the edit button on the article and submitting a PR. We will review and merge it.

#please-close


[OSCI] add author noicebroice

Description
adds author md for noicebroice
Issues Resolved
N/A
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

Can you remove the <> brackets? Thank you!


Future of the project

Since this project seems to be unmaintained and author seems to be no longer responding to the issues, this issue serves as a place to discuss its future.
Is there currently any fork that has up to date styles from GitHub? Is any of the growing list of forks popular enough to serve as a future point of reference?

Hi @rr-
Version 4.6.0 with the remaining style fixes has just been released. More updates on the way. Thanks for opening the issue.

!!! :heart:
Thank you


version 2.8.0 release

Summary
We'll be releasing v2.8.0 after merging these pull requests: https://github.com/slackapi/python-slackclient/pulls?q=is%3Apr+is%3Aopen+milestone%3A2.8.0
This pull request adds v2.8.0 to changelogs. Also, I've fixed a few existing issues in the document.
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

I've run all the integration_tests and they passed.


Add a method to compute a bounding box enclosing a set of points

Objective
Make it easier to create bounding boxes in user code by providing a constructor that computes a box surrounding an arbitrary number of points.
Solution
Add Aabb::enclosing, which accepts iterators, slices, or arrays.

This is implemented on Mesh https://github.com/bevyengine/bevy/blob/fb094eab8793041444294e62df96dd7ae488a980/crates/bevy_render/src/mesh/mesh/mod.rs#L465 but Mesh should probably use this code on Aabb once it has been implemented. I think that should be part of this PR.

I've rewritten the Mesh method and added a unit test. Let me know if there's any other test cases that should be covered.


Wrong link in the documentation

Only documentation on AWS https://docs.aws.amazon.com/eks/latest/userguide/eks-linux-ami-versions.html has a link to the changelog to view the latest images.
However the link points to branch "master" and the new updated branch is "main" (as stated in https://github.com/awslabs/amazon-eks-ami/issues/1656 )
The link should be updated from
https://github.com/awslabs/amazon-eks-ami/blob/master/CHANGELOG.md
to
https://github.com/awslabs/amazon-eks-ami/blob/main/CHANGELOG.md

This has been updated.


Output whats happening on the server to a console

Hey ho it's me again,
let me explain what i thought about this time.
Theres 3 ways to look onto the server and see whats going on:

way is to attach to it
way is to execute docker logs...
way is to "less logs/latest.log"

now i want to display whats happening on the server directly to a webinterface console... but i cant use the docker attach function, as i put in the option "--noconsole"
i cant use docker logs, because there is stuff the console should not output (like the echos in the beginning)
the 3rd way would be possible, but first of all takes too much time, consumpts a lot of network if its big and last but not least is just a crappy solution... :D
Any ideas?

I could create and include a helper tool that interprets the log and emits a summary of it. That would get the best of approach three but minimize the volume of network output.
Otherwise the rcon-cli might be helpful.
What kinds of info do you want to extract from the logs/system?

i would like to have everything that is displayed in the minecraft logs under logs/latest.log basically
so everything the minecraft server echoes when running in a screen for example.
i dont want to see the echoes from the docker (switching to user minecraft, downloading.. etc.)
for example:

This might be worth a look at tmux or screen inside the container. Having that would allow you to attach the container and see the terminal scroll. (Not sure if that would also help with the situations where CPU pegs?) Or is this more about an aggregate copy of the logs for other processing offline?

its more like displaying live ( in real time) what is happening on the server on a webserver.
basically i want a console on the web and then send commands to the server and directly get the result back - same as sitting there and watching the console without docker.

Thinking outside the box (or container :) ), perhaps a more direct/efficient solution is to leave the minecraft server logging to console but start your container with the gelf-log-driver to send the minecraft server output to a central location of your choosing:
https://docs.docker.com/engine/admin/logging/gelf/
Otherwise, docker exec'ing into the container and using rcon-cli might address some other data points. And then finally, there are implementations (in Java for sure) of the Minecraft server protocol where you can do some basic interrogation of the server state. Here's an example of where I do that:
<EMAIL_ADDRESS>

https://hub.docker.com/r/gliderlabs/logspout/ May be an easier to implement solution. I use it quite extensively to route logs to ELK stacks and it's really solid. I do have a couple of other points, but I'll raise those as other Feature Request issues as to not clutter here.

Yes, I forgot about logspout. That one is interesting since I believe you can still docker log concurrently into each container.

Yeah, it basically takes whatever is on stdour/stderr and outputs that to your end target. The docker log docker-compose logs still show as does anything that you would see on an attach to the container.

The logspout approach got me curious about something and by running
ls -l /var/lib/docker/containers/*/*-json.log

I confirmed that the default json file logger does not use a max-size by default and I have some logs that are several hundred MB now.
So I advise anyone that uses logspout to also tune their log-opts

I finally got around to using the gelf driver on my home system and I am quite impressed with the details that come along with each log document:
{
        "_index": "logstash-2017.06.10",
        "_type": "logs",
        "_id": "AVyTu4Gn1ahRCX9Qrxm-",
        "_score": 1,
        "_source": {
          "source_host": "<IP_ADDRESS>",
          "level": 6,
          "created": "2017-06-10T20:39:13.33070792Z",
          "message": """{"type":"log","@timestamp":"2017-06-10T20:39:17Z","tags":["status","plugin:elasticsearch@5.4.0","info"],"pid":12,"state":"yellow","message":"Status changed from uninitialized to yellow - Waiting for Elasticsearch","prevState":"uninitialized","prevMsg":"uninitialized"}""",
          "version": "1.1",
          "command": "/docker-entrypoint.sh kibana",
          "image_name": "kibana",
          "@timestamp": "2017-06-10T20:39:17.446Z",
          "container_name": "nuces_kibana_1",
          "host": "nuc",
          "@version": "1",
          "tag": "cc9be6fce6d3",
          "image_id": "sha256:1dfc1de100f14c4698772801247f940a752cc41160dd2657a38d472a76d7ea50",
          "container_id": "cc9be6fce6d3ed4b2ca32d86acaf50148164eb75ad05680730fd652789c1a53e"
        }
      }

I thought I'd try the new official logstash image from Elastic, so a heads up that not disabling XPACK_MONITORING_ENABLED bit me at first. Here's my compose snippet:
  logstash:
    image: docker.elastic.co/logstash/logstash:5.4.1
    environment:
      XPACK_MONITORING_ENABLED: "false"
    ports:
     - "12201:12201/udp"
    volumes:
      - "${PWD}/logstash-pipeline:/usr/share/logstash/pipeline/:ro"
    restart: always

...looks like I should play with logstash further decoding the "message" as the JSON payload it seems to be (in the case of kibana's output).

FYI here's what I have deployed at home for my "ELK" composition:
https://github.com/itzg/nuc-es/blob/master/docker-compose.yml

5.4.0 comes with X-pack now doesn't it?

Ah, they pre-bundle x-pack in their images, but with a 30 day trial.

Good, we've licensed, but thanks for the confirmation...

@Reviville are you good with me closing this issue? Sounds like streaming the logging via logstash worked for you, right?

@itzg sure go ahead, i ended up with using gelf and logstash but im still trying to remove the unwanted echoes. But again, thank you very much :)


Add resources to manage Zero Trust Connectivity Settings

Current Terraform and Cloudflare provider version
Terraform v1.6.3
on linux_amd64

provider registry.terraform.io/cloudflare/cloudflare v4.18.0

Description
Add resources to manage Zero Trust Connectivity Settingsto manage ICMP proxying and Warp-To-Warp (offramp_warp_enabled) settings
Use cases
This resource will allow us to manage ICMP proxying and Warp-To-Warp using terraform. Not sure if this should be its own resource or be part of an existing resource.
Potential Terraform configuration
resource "zero_trust_connectivity_settings" "organization" {
  icmp_proxy_enabled = true
  offram_warp_enabled = true
}

References
https://developers.cloudflare.com/api/operations/zero-trust-accounts-patch-connectivity-settings

This issue has been closed as we are now tracking this internally with service teams directly. If you would like an update or to be notified when/if the product ships with this change, please reach out to Cloudflare Support or your account team who can watch the internal feature request for you.


Error : Google sign in could not complete it's process correctly

Hi,
I am unable to login using your plugin and i attached my sample project here.
Could you please check this, why i am getting this error.
GoogleNew.zip
Any help @rdelrosario  @Pujolsluis  @RomaRudyak
Thanks in advance

Hi @SatyaRamprasad ,
From first view all configured correctly in the project.
Did you go through manual of configuration from GoogleFirebaseConsoleSetup.md?
On what platform sign-in fails?
Ensure that you provided valid ClientId for you project bundle.

Hi @RomaRudyak
Thanks the reply.
I have configured the project by following these steps for both platforms(Android and IOS)
https://github.com/CrossGeeks/GoogleClientPlugin/blob/master/GoogleClient/docs/GoogleFirebaseConsoleSetup.md
Sign in fails after Successful Authentication(Android) and i have attached my project in the previous comment.
Please look in to it once.

@SatyaRamprasad try next steps for project that you attached:
Android

Add next nuget package Xamarin.GooglePlayServices.Auth
In firebase console add fingerprint of keystore which sign up apk

iOS

Add next nuget package Xamarin.Google.iOS.SighIn
changed Role to Editor for custom schema in Info.plist

@RomaRudyak  i tried for android and i got the same error "Google sign in could not complete it's process correctly"

@SatyaRamprasad  try again but before clean your bin and obj directories with
cd /path/to/project/root

find . -name "bin" -o -name "obj" | xargs rm -rf

Then remove application from device.
And run

@RomaRudyak  still got the same error "Google sign in could not complete it's process correctly"

@SatyaRamprasad please ensure that you added android keystore sha1 fingerprint to firebase console
I attached working solution. Modify it with your keys
GoogleNew.zip

@RomaRudyak Which one is working __MACOSX project or GoogleNew?

@SatyaRamprasad GoogleNew

@RomaRudyak  Thank you. Will check and let you know


Throw an error if no route is found

Coverage remained the same at 100.0% when pulling 9fd8b69f889bc685c0c4ae87829e85a4f9131b6f on frenzzy:not-found-error into 4f69c6286bc2d368314ddaac47928af0cfa47c9c on kriasoft:master.

Coverage remained the same at 100.0% when pulling a29a9b7f170dea71a8fe3f31fc258d1b8a53e86f on frenzzy:not-found-error into 4f69c6286bc2d368314ddaac47928af0cfa47c9c on kriasoft:master.

Coverage remained the same at 100.0% when pulling 6cb1bf034eca66e70a01ea64c750a2c20cae972f on frenzzy:not-found-error into 4f69c6286bc2d368314ddaac47928af0cfa47c9c on kriasoft:master.

Coverage remained the same at 100.0% when pulling fef6746366ea19ca8341fbf42243bb94dbdea54b on frenzzy:not-found-error into 4f69c6286bc2d368314ddaac47928af0cfa47c9c on kriasoft:master.

Coverage remained the same at 100.0% when pulling 82bd07a93d22387c18e4f0061f1de5b6cac3aaa2 on frenzzy:not-found-error into 4f69c6286bc2d368314ddaac47928af0cfa47c9c on kriasoft:master.

Coverage remained the same at 100.0% when pulling<PHONE_NUMBER>d6fd5163b689a34caf88d7271e9e63 on frenzzy:not-found-error into c08ae009da09f3bd7ff88318d0f78b1a4a01853a on kriasoft:master.


New package: NetworkHawkesProcesses v0.1.0

Registering package: NetworkHawkesProcesses
Repository: https://github.com/cswaney/NetworkHawkesProcesses.jl
Created by: @cswaney
Version: v0.1.0
Commit: 9eb5aa607870660f9dae7f4da0cbe26adebe843e
Reviewed by: @cswaney
Reference: https://github.com/cswaney/NetworkHawkesProcesses.jl/commit/9eb5aa607870660f9dae7f4da0cbe26adebe843e#commitcomment-99891178
Description: Network Hawkes processes in Julia.

I will update package and try to auto-merge. Please disregard this PR.


Accept Decimal for Number

Describe the bug
Unable to use decimal with all number based validators, like ExclusiveMaximumAttribute.
To Reproduce
public class SomeEntity
{
    [ExclusiveMaximum(75.00m)]
    public decimal Percentage { get; set; }
}

Expected behavior
I believe the specification in draft-8/2019‑09 says any number

Thanks for the report. I actually use decimal as the default numeric type within the libraries, so I'm curious as to what's going on.
Could you let me know which library and version this is, please?

Thanks for the report. I actually use decimal as the default numeric type within the libraries, so I'm curious as to what's going on.
Could you let me know which library and version this is, please?

I think its just an oversight but have not had too much time to take a deep dive yet. I believe its the fact the attributes only accept a uint and not a decimal, double, or float.
https://github.com/gregsdennis/json-everything/blob/0e941ed08f671cef0689027f44ef7430f28d9a70/JsonSchema.Generation/ExclusiveMaximumAttribute.cs#L22

I think its just an oversight but have not had too much time to take a deep dive yet. I believe its the fact the attributes only accept a uint and not a decimal, double, or float.
https://github.com/gregsdennis/json-everything/blob/0e941ed08f671cef0689027f44ef7430f28d9a70/JsonSchema.Generation/ExclusiveMaximumAttribute.cs#L22

Could you let me know which library and version this is, please?

Okay... so I'm guessing you're looking at the Schema.Net.Generation library.
The reason decimal isn't included is because C# doesn't support decimals as attribute arguments.  The system will still generate the appropriate schema.

Could you let me know which library and version this is, please?

Okay... so I'm guessing you're looking at the Schema.Net.Generation library.
The reason decimal isn't included is because C# doesn't support decimals as attribute arguments.  The system will still generate the appropriate schema.


Fix: LogFactoryTests remove Windows specific values

Fixed some of the tests in LogFactoryTests class which they were using Windows specific temporary directory location i.e. C:\temp. Now the temp directories are created through managed code.

Following the comment from @ArsenShnurkov at #958 "MONO_4_0 support" I removed the Windows specific hardcode values.
It appears though the mono built still breaks.

* Assertion at mini-exceptions.c:856, condition `unwind_options == MONO_UNWIND_NONE' not met

i think that your mono version is not current (because line number is different)
https://github.com/mono/mono/blob/8cfef6d19071ed2215b13b4ded3b916752364c2d/mono/mini/mini-exceptions.c#L843
But it need to be researched even for most recent version of mono

see https://github.com/mono/mono/pull/1649
Should work on mono >= mono-<IP_ADDRESS>

I will have a look. Thank you @ArsenShnurkov

Thanks!
Would be nice if those tests were working with mono.
I dunno which mono version we use at travis. I think 2

travis uses
Mono JIT compiler version 4.0.4 (Stable <IP_ADDRESS>/5ab4c0d Tue Aug 25 23:11:51 UTC 2015)
you can see that in the build log

Mono JIT compiler version 4.0.4 (Stable <IP_ADDRESS>/5ab4c0d Tue Aug 25 23:11:51 UTC 2015)

Haha I'm too late :beer:

By default build service uses latest version of mono from Xamarin Repositories:
https://github.com/travis-ci/travis-ci/issues/4727#issuecomment-136500307
The docs don't say where Xamarin Repositories reside:
http://docs.travis-ci.com/user/languages/csharp/#Choosing-Mono-version-to-test-against

Addition of
mono:
  - weekly

to travis.yml may help

Hey @ArsenShnurkov ,
I think it is preferred to have  NLog  working in Mono 4.0+ and not Mono 4.2+ ? Can we test in code which Mono version we're working with?

PS to test, it's OK to change .appveyor.yml

Is aa nice improvement, so merge :)

I don't understood, what was changed to make it passing the tests. May I ask you to write down (document) the details?

it do not pass mono-4.0.4 yet - it skips those tests in Mono. But that's another issue.

See https://github.com/NLog/NLog/pull/1006, but still not working :(

in 1006 there is no unwind error, there are another texts:
Starting: NLog.UnitTests.dll
   NLog.UnitTests.Targets.FileTargetTests.MaxArchiveFilesWithDate_removesToManyFiles [SKIP]
      TODO not working, needs to be fixed
   NLog.UnitTests.Targets.FileTargetTests.ArchiveAboveSizeWithArchiveNumberingModeDate_maxfiles_o [SKIP]
      this is not supported, because we cannot create multiple archive files with  ArchiveNumberingMode.Date (for one day)
   NLog.UnitTests.Targets.WebServiceTargetTests.WebserviceTest_restapi_httpget [SKIP]
      Not working - ProtocolViolationException - skip for fix later
   NLog.UnitTests.LogReceiverService.LogReceiverServiceTests.RealTestLogReciever_one_way_binary [SKIP]
      unit test should listen to non-http for this
   NLog.UnitTests.LogReceiverService.LogReceiverServiceTests.RealTestLogReciever_two_way_binary [SKIP]
      unit test should listen to non-http for this
   NLog.UnitTests.Contexts.MappedDiagnosticsLogicalContextTests.given_multiple_threads_running_asynchronously_when_setting_and_getting_values_should_return_thread_specific_values [FAIL]
      Assert.Equal() Failure\r\nPosition: First difference is at position 12\r\nExpected: ValueForTask3\r\nActual:   ValueForTask2
      Stack Trace:
         tests\NLog.UnitTests\Contexts\MappedDiagnosticsLogicalContextTests.cs(207,0): at NLog.UnitTests.Contexts.MappedDiagnosticsLogicalContextTests.given_multiple_threads_running_asynchronously_when_setting_and_getting_values_should_return_thread_specific_values()
   NLog.UnitTests.Config.VariableTests.VariablesTest_string_expanding [SKIP]

does this mean that mono version bump helped? Or you bumped mono AND commented out the tests, so we don't know does bump help or not?


Blank Screen After Syncing v1.1 Commits

Install at: https://status.theobearman.com/
Thoughts?

Your site is returning a 500 response (Internal Server Error)
I would check your web server error logs for details on whats causing the issue.

@GenPage Nothing looks abnormal. Is there any way of knowing whether it's something to do with my server it the new changes?

Check your error logs and also the ./storage/logs directory for any output.

@jbrooksuk https://theobearman.com/cachet.txt

@TheoBearman You need to update your config folder too.

And you need to clear the config cache.

DId you forget to run composer install --no-dev or something?

You need to run that everytime you update.

@GrahamCampbell How do I do that?

No I did that.

@GrahamCampbell How do I do that?

In you console, run composer install --no-dev.

You can't have run that command, because you error contain logs that could only exist because you haven't run that command.

I always run "composer install -o --no-dev --no-scripts"

You need to run with script.

Get rid of the no scripts, and it will work.

http://imgur.com/J01qzo7

You need to manually run the windows equilivent of what our scripts do then.

The console doesn't recognise "composer install -o --no-dev" or "composer install --no-dev".

How can I do that? Surely there can be some better support for windows? What changed in v1.1 which is giving me these problems?

Surely there can be some better support for windows?

Nothing planned. I use windows and get along just fine. It's really very simple. Go and delete those files our script deletes, and then manually run the commands our script runs after install.

Delete the stuff in https://github.com/cachethq/Cachet/tree/1.1/bootstrap/cache.

Ok done. Now what scripts am I running?

*Commands

Please read the composer.json.

Trying now...

Delete everything from ./bootstrap/cache.
Run composer install -o --no-dev.

Anything? :)

@jbrooksuk This appeared in my console:

Ah yes, Windows doesn't know about the rm command. Try del /S C:\cachet\bootstrap\cache\* or wherever Cachet is installed.

Don't use composer to run the script.

Run them yourself.

@jbrooksuk I manually deleted the contents of the /cache folder but the composer install -o --no-dev command is not recognised by my console.

@TheoBearman if you don't have Composer installed, how did you get Cachet installed originally?

@jbrooksuk I do have composer installed and it works. For some reason however, composer isn't recognising the command you referred to above.

Did you run composer install -o --no-dev --no-scripts?

@jbrooksuk I did. I usually do and it works. Apparently I need to run it without the --no-scripts bit for v1.1 and that is what is causing the issue. I have attempted to run the scripts individually and that isn't working either.

You need to run WITH scripts for EVERY update, otherwise you've only partially updated, and you might as well just stay on a stable tag.

I've looked into @TheoBearman's server, the .env file is no longer configured.


Amended touch support test as it was returning true on none-touch windows devices

This change is based on the following open issue:
https://github.com/nolimits4web/swiper/issues/3232
I have come across the same issue in my project and have amended the script as per the PR, which has resolved the issues on MS Edge and Chrome on Windows based devices.

This fix is wrong, where did window.DocumentTouch && document instanceof window.DocumentTouch go? There is no navigator.MaxTouchPoints property

#3401
I'd recommend simply reverting commit df41dc94c9d9224fcbeb0166e6a5a9634caa6f1d.
This will check to see if the device has touch-capabilities, which will impact the slider.
maxTouchPoints will test to see if the device supports touch, which is ambiguous since what if the user is using a laptop or desktop with a touch screen.

There is:
https://developer.mozilla.org/en-US/docs/Web/API/Navigator/maxTouchPoints


Clarification on mutable objects

Hi everyone
I am new to automatic differentiation. I decided to use Enzyme since I need to handle mutating objects (even if I am not differentiating with respect to these objects).
I try to write a minimal, self-consistent, example:
using Enzyme

mutable struct my_struct
    p1::Vector{Float64}
    p2::Float64
end

function edit_str!(str::my_struct, x, i)
    str.p2 = str.p2 + 2x^2
    return nothing
end

function foo(x, str)
    edit_str!(str, x, 1)
    return str.p2
end

str = my_struct([2.0, 3.0], 1.0)

foo(1.0, str), autodiff(Reverse, foo, Active, Active(1.0), str)

Here I try to differentiate the function foo in x = 1.0. Note that the function edit_str! inside foo changes the object str. Yet, I do not need to differentiate with respect to str.
I found out that, depending on the mutations on str, the code may or may not work. In the above example it gives the right result 4.0, but if I change it slightly
function edit_str!(str::my_struct, x, i)
    str.p1 = str.p1 .+ 1.0
    str.p2 = str.p2 + 2x^2
    return nothing
end

it does not work anymore and it gives 0.0. Note that str.p1 is not needed anywhere. Perhaps even weirder, this works
function edit_str!(str::my_struct, x, i)
    str.p1 = 2.0 * str.p1
    str.p2 = str.p2 + 2x^2
    return nothing
end

I guess it depends on the kind of mutation on the mutable struct, but I would like to know if there is a robust way of dealing with this.
Thanks a lot.

Yeah essentially if you store active (e.g. differentiable) data into constant memory (such as the struct here), there's some additional complications.
Specifically, since it's stored into constant data, Enzyme can assume that uses of it in that constant data won't impact the derivative -- since its loading/using a constant!
If it acts as a buffer/temporary storage of derivative data, you'll need to mark it as duplicated (see here: https://enzyme.mit.edu/index.fcgi/julia/stable/#Activity-of-temporary-storage). The reason for this is because if you build up some arbitrary data structure, we also need to construct the shadow (aka derivative) data structure. Without the shadow data structure, we have no location as memory to use for the temporary storage of the derivative!
Your first case is actually why I was very careful and said that Enzyme can assume that uses of it won't impact the derivative, rather than Enzyme will assume. The reason in that case, is that the computation was simple enough that an optimization realized it was returning the input variable x and decided to return that directly rather than actually loading from the constant struct. Therefore it is returning an active value, and has a meaningful derivative. If you want guarantee that all uses of a variable will not have their uses change the derivative, we have a construct for that too. In essence it's equivalent to making a custom rule that behaves identity-like, but is marked inactive.


Add RingBuffer and move imgui over to it.

Currently playing with fire by reusing the same buffers for drawing. Need to write a gfx::RingBuffer to reuse that does the right things here, as well as add QueueFence queries and waiting to properly manage the read/write heads.
Can't have QueueFence be an Event, unfortunately, but could add a Wait() method on it as well as a query.
Can support both wait and query easily on vulkan, webgl2/es3, and d3d. Metal needs some custom work via command submit completion handlers, but nothing bad.

Gonna fake it w/ MemoryPool for now.


Re module

Codecov Report

Merging #591 into master will increase coverage by 3.19%.
The diff coverage is 39.39%.

@@            Coverage Diff             @@
##           master     #591      +/-   ##
==========================================
+ Coverage   41.25%   44.45%   +3.19%
==========================================
  Files          75       75
  Lines       17096    17710     +614
  Branches     4538     4964     +426
==========================================
+ Hits         7053     7873     +820
+ Misses       8005     7770     -235
- Partials     2038     2067      +29

Impacted Files
Coverage Δ

vm/src/stdlib/re.rs
37.5% <39.39%> (-10.78%)
:arrow_down:

parser/src/ast.rs
17.44% <0%> (-5.26%)
:arrow_down:

parser/src/lexer.rs
53.8% <0%> (-3.2%)
:arrow_down:

vm/src/obj/objint.rs
47.34% <0%> (+0.17%)
:arrow_up:

vm/src/vm.rs
54.55% <0%> (+0.27%)
:arrow_up:

vm/src/pyobject.rs
63.76% <0%> (+3.34%)
:arrow_up:

vm/src/function.rs
72.5% <0%> (+5.83%)
:arrow_up:

vm/src/frame.rs
62.69% <0%> (+15.52%)
:arrow_up:

vm/src/obj/objstr.rs
53.23% <0%> (+15.68%)
:arrow_up:

... and 3 more

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 6ba25ef...f3791a3. Read the comment docs.


Backwards compability for Python 2 64bit

I discovered a problem when running pyswip on Python 2.7. I get the following error:
pyswip.easy.ArgumentTypeError: Expected an argument of type '("<class 'pyswip.easy.Term'>", "<type 'int'>")' but got '<type 'long'>
This problem is also discussed in https://github.com/yuce/pyswip/issues/40 but the proposed solution is not working, in my case.
I'm running Python 2.7 64bit in combination with SWI-Prolog 64 bit. The problem seems that the integers from the 64bit native library are represented as long in Python 2.7. This is not a problem in Python 3, as int and long are combined in one common datatype.
The workaround for backwards compatibility is adapted from http://python3porting.com/differences.html
I tested the code with 64bit SWI Prolog 8.2.1 on both Python 2.7.16 (64bit) and Python 3.8.3 (64bit).

@tobiasgrubenmann Thanks for your contribution! Could you add your name to CONTRIBUTORS ?
@g-gemignani  Do the changes in this PR have any adverse affect on your end?

Hi @yuce I do not see any issue with this PR. :) Thank you for asking!


Test example

test

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


updating APIs, removing obsolete API usage, fixing build warnings

Motivation

Fixes issue 86198608
Removing obsolete API, upgrading API references, removing additional build warnings

Checklist

[ ] Code follows coding conventions held in this repo
[ ] Automated tests have been added
[ ] Tests are passing
[ ] Docs have been updated (if applicable)
[ ] Temporary settings (e.g. variables used during development and testing) have been reverted to defaults

How to test
If manual testing is required, what are the steps?

Just a heads up, I did a lot of this work already in https://github.com/Kentico/xperience-by-kentico-zapier/pull/34
There is some project structured/config that was missing that I also added.

Just a heads up, I did a lot of this work already in #34
There is some project structured/config that was missing that I also added.

Thank you for pointing that out! From my perspective, your merge request addresses all the issues I was targeting with this one. Therefore, it might make sense to close this MR and proceed with #34 instead. @Rumec


Issues deleting keys while iterating a list of dictionary

Problem Description
While iterating through a list of dictionaries, rarely and very randomly a ClientResponseError gets raised. From Pocketbase logs I can see that the following code somewhat tries to delete the same key twice, resulting in this issue. It does seem a kind of race condition of some sort?
Steps to Reproduce
I have not much to say here, I can paste the code that results in this error but it's very sporadic:

filter: dict = {"filter": f'SomeKey = "SomeValue"', "sort": "-LastSeen"}
query_response = pb_client.collection("Some_collection").get_full_list(query_params=filter)
trimmed_response: list[dict] = []
for element in query_response:
    temp: dict = {}
    temp["id"] = element.id
    temp["LastSeen"] = element.last_seen
    trimmed_response.append(temp)
for element in trimmed_response[1:]:
    pb_client.collection("Some_collection").delete(element["id"])

Expected Behavior
I wouldn't expect this code to randomly try to delete the same key twice.
Current Behavior
Somehow, it does. I think that #18 tries to fix this?
Context

poketbase: 0.9.2

Hi,
can you put in a check directly after building your trimmed_response list and before executing your "delete" loop in order to ensure that your trimmed_response does not include any duplicates in the first place.
for testing purpose you could add something quick-and-dirty like:
duplicates = [
    x
    for x in trimmed_response
    if [e["id"] for e in trimmed_response].count(x["id"]) >= 2
]
if duplicates:
    print("Error: There are the following duplicate IDs: ", duplicates)

The #18 you refer to relates to a fix in the realtime service which is not relevant here as you use the normal record service.
 If someone else is concurrently deleting elements from the pocketbase every once in a while while your code excecutes the behaviour you see would be somehow "expected".
The code being executed within this library to delete a record is rather straightforward and well covered by our library integration tests. In that sense you should double-check if your problem is not  related to your specific context and/or environment somehow.

closed due to inactivity


*: Update Go versions used for Travis tests and test tip

Looks like our env script means we can't build with development versions of Go.


Move stoq-cli.py to a console script

Move content of stoq-cli.py to __init__.py and add console_script to setup.py,
...
        entry_points= {
          'console_scripts': [ 'stoq=stoq:cli' ]
        }
...

Initial working(?) concept is in console_script branch (f7870e6)


SnippetChooserBlock not being indexed with Elasticsearch

I'm posting this here before I submit a pull request in case this has been discussed and rejected.
We use SnippetChooserBlock in tandem with pages to allow us to embed repeatable content in several pages such as a location card, an event preview, things of that nature. Definitely not appropriate use for a block.
However when the page is serialized for Elasticsearch, SnippetChooserBlock is completely ignored as it does not have get_searchable_content implemented. Here is how I made it work
class SearchableSnippetChooserBlock(SnippetChooserBlock):

    def get_searchable_content(self, value):
        if hasattr(value, 'search_fields'):
            fields = value.get_search_fields()
            doc = []
            for field in fields:
                doc.extend([field.get_value(value)])
            return doc
        else:
            return []

Hi @fawx - thanks for the suggestion!
Are you proposing adding SearchableSnippetChooserBlock as a new block type, or making this the standard behaviour for SnippetChooserBlock? If it's the latter, we'd have to be a bit careful, as there's no fixed rule about what a snippet chooser actually 'means'... sometimes, as in your example, it's an 'inclusion'-style relation, where you do want the snippet content to be indexed as part of the page content; at other times, snippets are used for tagging, or linking to external resources, where you don't want the snippet content to be considered part of the page.
As an example of that, suppose we were building a tourism site, where we use a SnippetChooserBlock to tag articles with a particular city, using a City snippet model that looked like this:
class City(models.Model):
    name = models.CharField(max_length=255)
    description = RichTextField()

    search_fields = [
        index.SearchField('name'),
        index.SearchField('description'),
    ]

register_snippet(City)

In this case, you wouldn't want to include the city description as part of the article's indexed content - that would mean that whenever you searched for a word that happened to be included in that description, you'd get lots of irrelevant results.
Not sure what the most intuitive way of distinguishing between these two scenarios would be. Having distinct SearchableSnippetChooserBlock and SnippetChooserBlock classes would be one possibility - or having a new parameter to SnippetChooserBlock: SnippetChooserBlock(City, search_indexed=True). It seems like it would be neater to have all of the search indexing rules centralised in the page's search_fields definition, though. So, as a totally half-baked idea, how about something like:
class Article(Page):
    body = StreamField([
        ('paragraph', RichTextBlock()),
        ('city', SnippetChooserBlock(City)),
    ])

    search_fields = [
        index.SearchField('title'),
        index.SearchStreamField('body', {
            'paragraph': True,  # always index paragraph blocks in full
            'city': ['name'],  # only index the 'name' field of city blocks
        })
    ]

hey @gasman,
thanks for the quick reply! I think another index field class is a better idea. I like the idea of being able to control what's indexed per page.

Revisiting this... I think that descending into the searchable content of a snippet model linked through a SnippetChooserBlock is rarely going to be the right thing to do. The most typical use of snippets would be something like an Author snippet, with a bio field that's shown on each article written by that author - it's better not to treat that bio as part of the indexable content of the article, because that would generate a whole load of duplicate results for search terms that match the bio. More generally - the selling point of snippets is that they can be used multiple times across a site, and normally you wouldn't want one piece of content to be indexed multiple times.
For the cases where you do want it, it would be appropriate to handle that by defining a ChooserBlock subclass with get_searchable_content overridden as desired. That could be done via SearchableSnippetChooserBlock as shown above, but my preferred approach would be to set up a dedicated ChooserViewSet for the model so that it gets a dedicated FooChooserBlock, and use base_block_class to set up a custom get_searchable_content method.


[stencil] tests for file upload interceptor

Description
There is a file-upload interceptor in the common package which helps enable the file upload functionality in a stencil app by just registering the interceptor wherever required. The task is to add tests for this interceptor.
Goals

[ ] test for the file-upload interceptor

Implementational Details
Create a test file named file-upload.interceptor.spec.ts in the tests directory and write the required test cases using jest. Make sure to cover all edge cases.
Product Name
Stencil
Organisation Name
SamagraX
Tech Skills Needed
TypeScript, Jest, SuperTest, Software Testing
Mentors
@techsavvyash
Complexity
Medium
Category
Software Testing

Hey @techsavvyash i am familiar with jest can i work on this?

Please go ahead @Min2who assigning this to you

Hi @techsavvyash sir as I was working on integrating bun.js ,I came across this issue and I think it is solvable shall i try

Hey @vsvishalsharma,
This is being already worked upon, I suggest you focus on the Bun.js ticket itself.

@techsavvyash sure sir

@techsavvyash I understand the problem, and I believe I have the necessary skills. Could I have the opportunity to work on this?

@techsavvyash this is the initial code i have implemented:
`import { ExecutionContext } from '@nestjs/common';
import { Test, TestingModule } from '@nestjs/testing';
import { Observable,of } from 'rxjs';
import { FastifyFileInterceptor } from '../src/interceptors/file-upload.interceptor';
describe('FastifyFileInterceptor', () => {
let interceptor: FastifyFileInterceptor;
beforeEach(async () => {
const module: TestingModule = await Test.createTestingModule({
providers: [FastifyFileInterceptor],
}).compile();
interceptor = module.get<FastifyFileInterceptor>(FastifyFileInterceptor);

});
it('should be defined', () => {
expect(interceptor).toBeDefined();
});
it('should handle file upload', async () => {
const context: ExecutionContext = {
switchToHttp: jest.fn(() => ({
getRequest: jest.fn(() => ({
file: {},
})),
getResponse: jest.fn(),
})),
switchToRpc: jest.fn(),
switchToWs: jest.fn(),
};
const nextHandler = {
  handle: jest.fn(() => of('')),
};

await interceptor.intercept(context, nextHandler);

expect(context.switchToHttp().getRequest).toHaveBeenCalled();
expect(context.switchToHttp().getResponse).toHaveBeenCalled();

expect(nextHandler.handle).toHaveBeenCalled();

});
it('should handle getting an uploaded file', async () => {
const file = { /* mock file object */ };
const context: ExecutionContext = {
switchToHttp: jest.fn(() => ({
getRequest: jest.fn(() => ({
file: file,
})),
getResponse: jest.fn(),
})),
switchToRpc: jest.fn(),
switchToWs: jest.fn(),
};
const nextHandler = {
  handle: jest.fn(() => of('')),
};

await interceptor.intercept(context, nextHandler);

expect(context.switchToHttp().getRequest).toHaveBeenCalled();
expect(context.switchToHttp().getResponse).toHaveBeenCalled();

// Assert that the uploaded file is passed to the next handler
expect(nextHandler.handle).toHaveBeenCalledWith(file);

});
it('should handle errors', async () => {
const context: ExecutionContext = {
switchToHttp: jest.fn(() => ({
getRequest: jest.fn(() => ({
file: {},
})),
getResponse: jest.fn(),
})),
switchToRpc: jest.fn(),
switchToWs: jest.fn(),
};
const errorMessage = 'File upload failed';
const nextHandler = {
  handle: jest.fn(() => throwError(errorMessage)),
};

try {
  await interceptor.intercept(context, nextHandler);
} catch (error) {
  expect(error.message).toBe(errorMessage);
}

expect(context.switchToHttp().getRequest).toHaveBeenCalled();
expect(context.switchToHttp().getResponse).toHaveBeenCalled();
expect(nextHandler.handle).toHaveBeenCalled();

});
});
`

@vsvishalsharma please go ahead and raise a draft PR

Hey @techsavvyash  I also tried my hands on it and I guess I have pretty much done the required task
Can you please assign this to me and can I raise the PR fo it ?

hey @deecodess if you have tests ready, please raise a PR. This is a long running test case and we only assign issues once a draft PR has been raised.


feat: improve error message on invalid site-key

Problem
Currently, when the user passes an invalid siteKey, for example with – (long dash) instead -, SDK will throw not very  verr descriptive error like:
HCaptchaException(hCaptchaError=Invalid data is not accepted by endpoints, message=Invalid data is not accepted by endpoints)
Solution
Throw IllegalArgumentException("hCaptcha site-key must be a valid UUID")
P.S. Are our site-keys always valid UUID https://www.ietf.org/rfc/rfc4122.txt?

Will be done on backend


Fix deleting old releases, if deploy_path is a symlink

Q
A

Bug fix?
Yes

New feature?
No

BC breaks?
No (?)

Deprecations?
No

Fixed tickets

When the {{deploy_path}} is a symlink, find won't find any directories within it. To solve this, the trailing slash is required. This behaviour is explained here: https://chris-lamb.co.uk/posts/find-trailing-slashes-and-symbolic-links-directories

Cool! Thanks!


Create serialize API for server rendering

The const htmlStr = serialize({routes, serializeApplication}) API will convert a routes object into an HTML string. The routes themselves should be omitted from the HTML, and the applications should be converted to an HTML string through a user defined function.

Resolved in https://github.com/single-spa/single-spa-layout/pull/58 via the renderServerResponseBody API


Action Required: Fix Renovate Configuration

There is an error with this repository's Renovate configuration that needs to be fixed. As a precaution, Renovate will stop PRs until it is resolved.
Error type: Cannot find preset's package (local>just-smirnov/.github:renovate-test)

["local>myorgname/.github:renovate-config"]


PEM: constify PEM_write_ routines

There's no reason why the object to be written, or the key string
given by the caller should be non-const.
This makes the IMPLEMENT_PEM_..._const and DECLARE_PEM_..._const
macros superfluous, so we keep them around but mark them deprecated.
In all places where IMPLEMENT_PEM_..._const and DECLARE_PEM_..._const
are used, they are replaced with the corresponding macros without
_const.

This will help #10394, as the PEM hooks can become much cleaner

Merged.
de0799b0fc845869d775520382b4e7f9995732e5 PEM: constify PEM_write_ routines


Suspend/resume hotkeys callbacks

It is useful in modals, prompts etc.
Modes:

Total suspend
Suspend by scope
Suspend by combo

Sorry for the delay here, I'm trying to clean up old tickets. If this is important to you, please open a PR.  Thanks!


Hamming distance support

Please make sure that this is a feature request. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:feature_template
System information

TensorFlow version (you are using): 1.13
Are you willing to contribute it (Yes/No): Yes

Describe the feature and the current behavior/state.
Take binary representation class (predicted label ) number into account and compute loss.
Say, for example, the loss for true label = 0 and predicted label = 1 should be less than the loss for true label = 0 and predicted = 100
Will this change the current API? How?
No. This will add a new function
Who will benefit with this feature?
Helpful for users with the above-mentioned requirement.
If this is of interest, I would like to create a PR.

Can you please provide some more details on the proposed feature and its use case. This would help us to get more understanding on the same. Thanks!

@achandraa Thanks for the review comment. I will update about the same.

@SSaishruthi Please let us know use cases where this can be useful. Thanks!

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Unexpected behavior of built-in append function

Does this issue reproduce with the latest release?
Yes
What did you do?
Run
a := []int{}
a = append(a, 1)
fmt.Println(len(a))
fmt.Println(cap(a))

What did you expect to see?
Both len and cap should be 1
What did you see instead?
len(a) is 1, cap(a) is 2

This is expected. Please read https://blog.golang.org/slices.


Feature/webpack variables

One change we might want to this is have one of the values be true by default if NODE_ENV is not defined


Fixed issue with date library not importing correctly

In Ruby 1.9, DateTime fields are all set to Nil - This is because the Date.parse method is not available unless the 'date' library is imported.
Note that this was rather hard to track down originally because the line
value_to_set = DateTime.parse(value) rescue nil

Was throwing an exception due to the 'parse' method not being found - however this was being silently quashed by 'rescue nil' - this issue could potentially be occurring in other places but might be difficult to find.

Actually this seems to happen to some installs of Ruby 1.8 as well - the standard ubuntu install of Ruby 1.8 seems to break, but a manually compiled version worked.
Am I missing something?

On my local development environment (OS X with Ruby 1.9.2), I get the following results:
clara:~/work/databasedotcom(master)$ irb
ruby-1.9.2-p180 > Date.parse("12/11/2011")
NoMethodError: undefined method `parse' for Date:Class
	from (irb):1
	from /Users/dburkes/.rvm/rubies/ruby-1.9.2-p180/bin/irb:17:in `<main>'

however
clara:~/work/databasedotcom(master)$ bundle console
ruby-1.9.2-p180 > Date.parse("12/11/2011")
 => #<Date: 2011-11-12 (4911755/2,0,2299161)>

What this seems to say is that, while you are correct that Date.parse is not available until date is required, something in the bundle is requiring it.  What the dependencies require might conceivably be different on different platforms.
I'm all for merging this pull request, but I would like to see some tests along with it.  However, it's impossible for me to make the test fail locally because my platform's bundle apparently results in date getting required.
I'm going to push some tests that should fail up to Travis, and see if they fail there.  If they do, I'll then merge the pull request to fix them.

Well, no luck, the tests pass on Travis as well.  In any case, I'm merging the pull request, because requiring it multiple times should be idempotent on the platforms that don't need it.

Pushed a new 1.2.3 version of the gem that includes this pull request

The new test case succeeds for me too without the patch installed - I suspect the test runner is importing 'date' at some point, thus it'll never trigger that issue regardless.
I agree with dburkes however - There should be no issue importing the library again.


Spike: Testing Automation Preparation

[ ] Take Udemy course on testing automation specifically JAVA

[ ] Set up any applicable software needed and get familiar

[x] Shadowing and knowledge transfer for front end and API work

Software to Install:

Visual Studio
Java 8
Maven
ChromeDriver
IntelliJ
ChroPath

Shadowed Maegan Wood to cover API testing, and an intro to UI testing. Will need to follow up once I have everything installed.
Courses:

Learn JavaScript (https://www.youtube.com/watch?v=PkZNo7MFNFg&list=PLWKjhJtqVAblfum5WiQblKPwIbqYXkDoC&index=5)
Rest API Testing (https://www.udemy.com/course/rest-api-automation-testing-rest-assured/)
Selenium WebDriver (https://www.udemy.com/course/selenium-real-time-examplesinterview-questions/)


Correction for raw affiliation MacKay Drive, Room 346, Marshall d'Avray Hall, P.O. Box 4400, Fredericton, NB E3B 5A3. E-mail: [email protected]ca

Correction needed for raw affiliation MacKay Drive, Room 346, Marshall d'Avray Hall, P.O. Box 4400, Fredericton, NB E3B 5A3. E-mail: [email protected]ca
raw_affiliation_name: MacKay Drive, Room 346, Marshall d'Avray Hall, P.O. Box 4400, Fredericton, NB E3B 5A3. E-mail: [email protected]ca
new_rors: 05nkf0n29
previous_rors:
works_examples: W2077235288
contact: 1c4c637bd22fa044060ae41ded0e376e:c9e3a853 @ ourresearch.org

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


Update fs2-core, fs2-io, ... to 3.0.0

Updates

co.fs2:fs2-core
co.fs2:fs2-io
co.fs2:fs2-reactive-streams

from 3.0.0-M9 to 3.0.0.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (3.0.0-M9).
You might want to review and update them manually.
website/src/hugo/content/changelog.md

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "co.fs2" } ]

labels: library-update, semver-pre-release, old-version-remains

#4664


An administrative user can authenticate themselves to access an "admin" area of the service

So that DfE staff in the Financial Incentives Team will be able to access the data relating to claims and process them, we need an authentication mechanism that will grant them access to a secure part of the application.
This story is to do the initial plumbing that will allow a user to sign in using DfE Sign-in to access an "/admin" part of the application. For now, this will simply contain a holding page with a "sign-out" link that ends the session.
The identifier and secret are stored in Heroku. Place them in a .env.local file and run bin/start to start the dev server with https.
Note: Anyone with access to the 'Teacher Payments' service in DfE Sign In will be able to sign in, we currently don't check which organisation is returned. We may want to do this check in the future depending on how access to services is managed.
https://trello.com/c/A7865IBw

I've added the refactor as a separate commit for now to make reviewing easier. If you're happy with it I'll squash it together with the other commit.
The main side effect of the change is that we'll need to request to get our callback URLs changed with DfE Sign In, which is fine I just thought I'd note it here.

I'm gonna hold off merging this PR until the callback URL has changed in DfE Sign In, I've requested it via email (30/5). I'll also add the missing environment vars to Heroku

I thought I would do one last test of this locally and it seems that foreman ignores the .env.local file, preferring .env.
So I've made some changes to reflect that:

added .env to the gitignore and updated readme
added .env.development (for running the app without DfE Sign In) and .env.test (for CI)

The redirect URL has been changed on DfE Sign In and I've added the environment vars to Heroku staging, so this is ready to go


OpenSearch Dashboards - pdf not downloading behind a gateway

I have a gateway, where I am using an internal and external URL (differs from each other). I am assuming that OpenSearch Dashboards are checking if the URL's exist internally as well as externally. In my case the internal URL will exist internally but will fail if accessed externally, the same with the external URL. This is causing that the pdf's cannot be downloaded behind the gateway. Any advice?

The reason the above error occurs is because my internal and external URL's are not the same. OpenSearch Dashboards sits behind a gateway. The application is accessed from the outside using a different URL from the URL on which OpenSearch dashboards are hosted internally (it is routed via the gateway). When downloading the pdf, the URL used is checked if it exists internally (which it does not). The reporting then fails.

Can URL validation for the pdf reporting plugin also be considered as a feature release request?

Is there any feedback on this matter?

https://github.com/opensearch-project/dashboards-reports/issues/345
this might help/ architecturally reporting backend needs to talk to the dashboards server

This issue should be resolved in v2.5 as front-end report generation is used now. https://github.com/opensearch-project/reporting/issues/577
Please reopen issue if you see same issue in v2.5 or greater.


Remove Travis

We're migrating to Github Actions in #3125. Personally, I want to keep Travis alive for a little while, but in not too long -- let's say by the end of Februrary? -- we should remove our repos from travis-ci.com's UI and delete:

.travis.yml
.travis.sh
maybe: util/dockerize.sh -- though this isn't Travis specific, and I'm proud of it, we only use it for Travis

Originally posted by @kousu in https://github.com/neuropoly/spinalcordtoolbox/pull/3125#discussion_r551067762
We should have these transition periods:

Only Travis
Travis + Actions
Travis [non-mandatory] + Actions
Only Actions

That way there won't be too many surprises. @joshuacwnewton figured out that for the intermediate stage, just go to https://github.com/neuropoly/spinalcordtoolbox/settings/branch_protection_rules/12845205 and turn off/on the different checks; hopefully disabling the Travis check still means Travis runs, just that it doesn't block:

Originally posted by @kousu in https://github.com/neuropoly/spinalcordtoolbox/issues/3071#issuecomment-751832655

Note to self: When removing .travis.yml, also remove @kousu's travis.sh and dockerize.sh scripts. Since GitHub Actions has nicer WSL support, I don't think these are needed anymore.
See: https://github.com/neuropoly/spinalcordtoolbox/pull/3144#issuecomment-811250407

Note to self: Also remove the Travis banner from our documentation -- it currently shows "build: Failing" which is a bit ugly. :sweat_smile:


When will Alonzo be activated on the mainnet?

I am a wallet provider, please tell me the time to upgrade, so I can update the node in time

Alonzo will be activated when it is ready. At least a week (probably more) before that a new version of the node will be released that contains support for Alonzo.

Alonzo will be activated when it is ready. At least a week (probably more) before that a new version of the node will be released that contains support for Alonzo.

thanks


cuda build

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin please rerender

@conda-forge-admin please rerender

@conda-forge-admin please restart ci

@conda-forge-admin please restart ci

@conda-forge-admin please rerender

@conda-forge-admin please restart ci


Abi37_0_0.host.exp.exponent.modules.api.components.webview.RNCWebViewManager cannot be cast to abi30_0_0.com.facebook.react.uimanager.ViewGroupManager

**Bug description: Abi37_0_0.host.exp.exponent.modules.api.components.webview.RNCWebViewManager cannot be cast to abi30_0_0.com.facebook.react.uimanager.ViewGroupManager

**To Reproduce: code works in IOS but I get this error when I try android

Expected behavior:

Screenshots/Videos:

Environment:

OS: Android
OS version: latest version
react-native version: 0.68.2
react-native-webview version: 11.18.1

Please try a more recent version of react-native-webview. And, if you still get the error, please share a sample project that reproduces the issue.

I was able to resolve the issue by placing the Webview component inside a
view rather than a SafeAreaView
On Wed, Feb 22, 2023 at 9:27 PM Caleb Clarke @.***>
wrote:

Please try a more recent version of react-native-webview. And, if you
still get the error, please share a sample project that reproduces the
issue.
—
Reply to this email directly, view it on GitHub
https://github.com/react-native-webview/react-native-webview/issues/2871#issuecomment-1441141550,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZYV6S5SZN5SKBIFLUVL773WY3DIBANCNFSM6AAAAAAVEVXWC4
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

Check and Try this
https://github.com/react-native-webview/react-native-webview/issues/1372#issuecomment-1487785231


[explore v2] add control components for TimeFilter, GroupBy, Sql and Filter

https://github.com/airbnb/caravel/pull/1205

Second PR: https://github.com/airbnb/caravel/pull/1224