Exception using notebooks/unimol_mol_repr_demo.ipynb

Hi there,
While using your notebooks/unimol_mol_repr_demo.ipynb, my molecules result in an Exception from smi2coords function.
My molecules are following:
smi_list = [
"CC(C)CC(C(=O)NC(C(C)C)C(=O)O)NC(=O)CNC(=O)C(CCC(=O)N)NC(=O)C(CC(C)C)NC(=O)C(CC(C)C)NC(=O)C(CCCNC(=N)N)NC(=O)C(CCC(=O)N)NC(=O)C(CC(C)C)NC(=O)C(CCCNC(=N)N)NC(=O)C(C)NC(=O)C(CO)NC(=O)C(CC(=O)O)NC(=O)C(CCCNC(=N)N)NC(=O)C(CC(C)C)NC(=O)C(CCCNC(=N)N)NC(=O)C(CO)NC(=O)C(CC(C)C)NC(=O)C(CCC(=O)O)NC(=O)C(CO)NC(=O)C(C(C)O)NC(=O)C(CC1=CC=CC=C1)NC(=O)C(C(C)O)NC(=O)CNC(=O)C(CC(=O)O)NC(=O)C(CO)NC(=O)C(CC2=NC=CN2)N",
"CC(C)CC1C(=O)NC(C(=O)NC(C(=O)NC(CSSCC(C(=O)NC(C(=O)NC(C(=O)N1)CC(=O)N)CO)N)C(=O)NC(C(C)C)C(=O)NC(CC(C)C)C(=O)NCC(=O)NC(CCCCN)C(=O)NC(CC(C)C)C(=O)NC(CO)C(=O)NC(CCC(=O)N)C(=O)NC(CCC(=O)O)C(=O)NC(CC(C)C)C(=O)NC(CC2=CN=CN2)C(=O)NC(CCCCN)C(=O)NC(CC(C)C)C(=O)NC(CCC(=O)N)C(=O)NC(C(C)O)C(=O)NC(CC3=CC=C(C=C3)O)C(=O)N4CCCC4C(=O)NC(CCCNC(=N)N)C(=O)NC(C(C)O)C(=O)NC(CC(=O)N)C(=O)NC(C(C)O)C(=O)NCC(=O)NC(CO)C(=O)NCC(=O)NC(C(C)O)C(=O)N5CCCC5C(=O)N)C(C)O)CO",
"C(C1C(C(C(C(O1)OC2(C(C(C(O2)CO)O)O)COC3(C(C(C(O3)CO)O)O)COC4(C(C(C(O4)CO)O)O)COC5(C(C(C(O5)CO)O)O)COC6(C(C(C(O6)CO)O)O)COC7(C(C(C(O7)CO)O)O)COC8(C(C(C(O8)CO)O)O)COC9(C(C(C(O9)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)COC1(C(C(C(O1)CO)O)O)CO)O)O)O)O",
]

Error Stack is following:
ValueError                                Traceback (most recent call last)
Cell In [4], line 9
      7 batch_size=16
      8 results_path=data_path   # replace to your save path
----> 9 write_lmdb(smi_list, job_name=job_name, seed=seed, outpath=data_path)

Cell In [3], line 45, in write_lmdb(smiles_list, job_name, seed, outpath)
     43 txn_write = env_new.begin(write=True)
     44 for i, smiles in tqdm(enumerate(smiles_list)):
---> 45     inner_output = smi2coords(smiles, seed=seed)
     46     txn_write.put(f"{i}".encode("ascii"), inner_output)
     47 txn_write.commit()

Cell In [3], line 21, in smi2coords(smi, seed)
     19     except:
     20         pass
---> 21     coordinates = mol_tmp.GetConformer().GetPositions()
     22 assert len(atoms) == len(coordinates), "coordinates shape is not align with {}".format(smi)
     23 coordinate_list.append(coordinates.astype(np.float32))

ValueError: Bad Conformer Id

So why this happens? Is this project inable to process such long molecules?

Plus, I've done a scan of your molecule pretrained lmdb dataset, and find that your max length of atoms is 190, which in my senario is ~400 and ~800. Is this the problem?
If true, then how can we use your project to predict long molecules?

This error report is related to RDKit. It looks like RDKit cannot handle such a large molecule; maybe other methods could get the conformation.
Moreover, the "--max-atoms" parameter controls the molecular size. Our default setting is 256, and larger numbers could also be tried.

This error report is related to RDKit. It looks like RDKit cannot handle such a large molecule; maybe other methods could get the conformation. Moreover, the --max-atoms parameter controls the molecular size. Our default setting is 256, and larger numbers could also be tried.

Do you have any clue where to find those methods? I'm new to this field:) Thank you very much!

Do you have any clue where to find those methods? I'm new to this field:) Thank you very much!

Generating conformations of macro-molecules is a challenging task. Perhaps a method for peptides could be helpful. This may require help from pharmacochemical experts.

Do you have any clue where to find those methods? I'm new to this field:) Thank you very much!

Generating conformations of macro-molecules is a challenging task. Perhaps a method for peptides could be helpful. This may require help from pharmacochemical experts.

Thank you for your reply! Since my issue has been solved, I'll close this issue!


Change line wouldn't need extra empty line in editor would be better.

Hi, thanks for such a nice product.
As the title suggests, if not to change the current setting totally, adding a option may be a practial alternative.
P.S. how is the difficulty adding a vertical preview layout ? For long slides, vertical scrolling similar to a long slide-page sized document would be more friendly.

will be fixed in next version


Unable to link osvrRenderManager in debug

I had issue with linking osvrRenerManager in debug mode:

1>------ Build started: Project: osvrRenderManager, Configuration: Debug x64 ------
1>jsoncpp.lib(json_value.obj) : error LNK2038: mismatch detected for '_ITERATOR_DEBUG_LEVEL': value '0' doesn't match value '2' in RenderManagerBase.obj
1>jsoncpp.lib(json_value.obj) : error LNK2038: mismatch detected for 'RuntimeLibrary': value 'MD_DynamicRelease' doesn't match value 'MDd_DynamicDebug' in RenderManagerBase.obj
1>jsoncpp.lib(json_reader.obj) : error LNK2038: mismatch detected for '_ITERATOR_DEBUG_LEVEL': value '0' doesn't match value '2' in RenderManagerBase.obj
1>jsoncpp.lib(json_reader.obj) : error LNK2038: mismatch detected for 'RuntimeLibrary': value 'MD_DynamicRelease' doesn't match value 'MDd_DynamicDebug' in RenderManagerBase.obj
1>jsoncpp.lib(json_writer.obj) : error LNK2038: mismatch detected for '_ITERATOR_DEBUG_LEVEL': value '0' doesn't match value '2' in RenderManagerBase.obj
1>jsoncpp.lib(json_writer.obj) : error LNK2038: mismatch detected for 'RuntimeLibrary': value 'MD_DynamicRelease' doesn't match value 'MDd_DynamicDebug' in RenderManagerBase.obj
1>     Creating library D:/knihovny/OSVR RenderManager/build64-msvc12/bin/Debug/osvrRenderManagerd.lib and object D:/knihovny/OSVR RenderManager/build64-msvc12/bin/Debug/osvrRenderManagerd.exp
1>LINK : warning LNK4098: defaultlib 'MSVCRT' conflicts with use of other libs; use /NODEFAULTLIB:library
1>D:\knihovny\OSVR RenderManager\build64-msvc12\bin\Debug\osvrRenderManagerd.dll : fatal error LNK1319: 6 mismatches detected
========== Build: 0 succeeded, 1 failed, 3 up-to-date, 0 skipped ==========

Solution - wrong version of jsoncpp.lib was being linked. It should be the debug version of the library and not release. When I updated the project configuration in Visual Studio manually, it linked just fine. I recommend adding Jsoncpp_DEBUG_LIBRARY entry in CMAKE and using that library for debug builds.

@rpavlik Are you up for another round of wrestling with jsoncpp?


Missing values report missing Resources

What happened?
Confusing error message for missing variable.
Steps to reproduce
Run pulumi up on the following template:
resources:
  myBucket:
    type: aws:s3:Bucket
    properties:
      bucket: ${other}

name: yaml-example
runtime: yaml
description: A minimal Pulumi YAML program

Expected Behavior
    Error: resource/variable/configuration value "other" not found
      on Pulumi.yaml line 5:
       5:       bucket: ${other}
    resource/variable/configuration value "other" not found

Or something similar.
Actual Behavior
Error: resource "other" not found
  on Pulumi.yaml line 5:
   5:       bucket: ${other}
resource "other" not found

Versions used
3.28.0-alpha.1652812404+65139da6.dirty
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

This is a consequence of https://github.com/pulumi/pulumi-yaml/issues/253. Closing as a duplicate.


fix: footer container breaking in some break points

Notes for Reviewers
This PR fixes #286
In I have fixed some issues like company container was breaking at some points such as 850px and <=400px so i have fixed them .
Here are some Results for 850px
Prev

After

Here are some Results for <=400px
Before

After

Signed commits

[x] Yes, I signed my commits.

Before looks much better than the After

@upsaurav12 please keep the alignment to the left instead of center

@iArchitSharma will do it but rest of it any other suggestion on improvement.
My idea was to utilizr the extra space that was left in right side of the container

@iArchitSharma will do it but rest of it any other suggestion for improvement? My idea was to utilizr the extra space that was left in right side of the container also  company container should be after Solutions

LGTM now, thanks @upsaurav12

Thanks @iArchitSharma First merged PR


Create Face Anti Spoofing dataset

Im very interested in your project. So can you share with me a sample or structure about the dataset. I don't know how to created it. Thank you so much!

You can refer to the papers of FAS datasets，such as OULU-NPU、CASIA_MFSD、Idiap Replay-Attack、MSU-MFSD.


[DNM] [Waiting for decision on RP version support] EPMRPP-80543 || Add parameters method

Related changes from client-javascript.

There is no way to add parameters on test finish now so BE is need to be updated before this changes.

Test will pass after client-javascript merge

@chivekrodis ^


Model Events

It seems that mongo does not have eloquent model events on creating , updating, deleting, etc.
Is this there and im just missing it it?

They should not be disabled.


Listeners on DontDestroyOnLoad objects dont unsubscribe from events when a new scene is loaded

this happens because it first unsubscribes them all and than subscribes again. Probably have to change the logic but cant really see the reason to have DestroyOnLoad = true (default one) on DontDestroyOnLoad objects so this can be great to prevent possible issues

the same obviously happens to static methods that listens to static events within a single monoBehaviour


add physics #74

let's start reviewing this.
We'll need a better way to reconcile determining the "shape" and "quaternion" for different objects.  Let's discuss.
review? @dmarcos
cc @caseyyee

Will this be coming back some day? :D


Adding CONTRIBUTING.md file

Adding CONTRIBUTING.md file

Sure @komalswami , go ahead

hey is the issue still open, i would like to contribute to it.

Hey @jyotika6221, this task has already been completed, so closing it.


LG Performance & Scalability: Speed up loading, fix concurrency and memory problems

Fixes https://github.com/microsoft/botbuilder-dotnet/issues/6060
Summary of the issues found in this investigation

The LG files were loading a lot of things synchronously on constructors, so the 2 classes that did that now have an internal-only and optional LoadAsync method.
Each file was being read multiple times (more than 40 in some cases for our test set). Added a cache since FileResource is used as immutable in our codebase. Could add code to invalidate internal cache on set of any property if needed but no code path requires this now.
The adaptive dialog bot was calling these constructors multiple times in parallel which blew the memory considering that these constructors were reading several hundreds of megs worth of tile content. Plus there were concurrency issues due to them reading matching files on different requests.
The VScode event for LG was causing to keep unnecessary references and over-utilizing cpu cycles while not serving any purpose for production bots (it's a debug only feature)

Problem details and their solutions in this PR.

Efficient LG file loading: LG file content was loaded synchronously in a the TEmplateEngineLanguageGenerator constructor, not allowing for asynchronous execution and blocking the thread while we waited for each file to load. With this fix, we let LG load in the constructor for legacy paths, but the new paths have a LoadAsync method that loads asynchronously. The new path is internal only, meaning that the public surface is not affected. Each LGenerator is loaded only once by using LAzy with the default ExecutionAndPublication thread safety mode.
Concurrency protection and efficiency in AdaptiveDialogBot: LanguageGeneratorManager can take minutes in a sizable bot. If there are multiple parallel requests, LanguageGeneratorManager will get constructed once per request, allowing for servers to do the same work and throw OutOfMemoryException, consuming upwards of 14GB. This fix uses Lazy initialization in AdaptiveDialogBot with ExecutionAndPublication thread safe mode to ensure that only one LanguageGeneratorManager is created at ones
Multiple reads of file contents: For our perf sample, each LG file was being read more than 30 times, despite its content never changing. Also there was no protection for parallel reads, which could lead to unexpected concurrency failures. This fix adds a caching element to FileResource and at the same time uses the Lazy ExecutionAndPublication thread safety mode to ensure that only one thread initializes the value. When there is a change Event, a new Fileresource is created so this does not affect functionality.
LG debugging in VS code disabled due to live site issue critical risk. The event handling for LG debugging was triggering live site issues on customers, so this change entirely disables that feature to be re-enabled if the business requires it and once the design is ready to support significant loads.

Some results for our test dataset:
Before
Single request total load time: 5 min 11 sec
Single request total memory: 4.1GB
Load test (100 users, 20 concurrent) total load time and memory: N/A OutOfMemoryException after 14GB
After
Single request total load time: 1 min 40 sec
Single request total memory: 4.1GB
Load test (100 users, 20 concurrent) total load time and memory: 1min 50 sec, 4.1 GB :)
All unit and integration tests pass + verified with customer repro under load tests.

Pull Request Test Coverage Report for Build 281928

0 of 0   changed or added relevant lines in 0 files are covered.
33 unchanged lines in 9 files lost coverage.
Overall coverage decreased (-0.02%) to 78.66%

Files with Coverage Reduction
New Missed Lines
%

/libraries/AdaptiveExpressions/BuiltinFunctions/GetNextViableTime.cs
1
90.91%

/libraries/AdaptiveExpressions/BuiltinFunctions/GetPreviousViableTime.cs
1
90.91%

/libraries/Microsoft.Bot.Builder.Dialogs.Declarative/Resources/FileResource.cs
1
91.67%

/libraries/Microsoft.Bot.Connector.Streaming/TaskExtensions.cs
1
82.61%

/libraries/Microsoft.Bot.Connector.Streaming/Application/TimerAwaitable.cs
2
85.71%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/AdaptiveDialogBot.cs
4
94.74%

/libraries/Microsoft.Bot.Connector.Streaming/Application/WebSocketClient.cs
4
71.82%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Generators/LanguageGeneratorManager.cs
5
87.8%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Generators/TemplateEngineLanguageGenerator.cs
14
71.43%

Totals

Change from base Build 281321:
-0.02%

Covered Lines:
24862

Relevant Lines:
31607

💛  - Coveralls

/azp run BotBuilder-DotNet-CI-PR-(MacLinux)-yaml


Questions about write_virt_mem(), install_ept_hook() and hide_physical_page()

Is write_virt_mem() able to write to read-only memory in guest usermode processes, specifically protected pages containing import address tables (IAT)? What about read-only memory in the guest kernel space?
If so, how does it achieve this? Does it temporarily disable the CR0.write_protect bit? Or is there some other mechanism in play?
If not, how would I go about implementing a custom hypercall that does this?
Edit:
Some more questions:
I was reading the source of ept.h and noticed that ept_free_page_count was set to 100. Does this mean that only 100 pages of the target address space can be EPT switched on access violation? Is there a set limit for it in the Intel standard somewhere?
What if I wanna allocate a range of pages to let's say manually map a DLL into a guests usermode process, can I then take an arbitrary amount of continuous pages matching the size of my DLL anywhere within the guests process and use EPT switching (install_ept_hook()) on said pages to avoid allocating memory with f.ex. VirtualAlloc?
And lastly what's the difference between hide_physical_page() and install_ept_hook() in practice?

Is write_virt_mem() able to write to read-only memory in guest usermode processes, specifically protected pages containing import address tables (IAT)? What about read-only memory in the guest kernel space?
If so, how does it achieve this? Does it temporarily disable the CR0.write_protect bit? Or is there some other mechanism in play?
If not, how would I go about implementing a custom hypercall that does this?

write_virt_mem() is able to write to read-only memory because in the host page tables, they are set as read-write-executable (https://github.com/jonomango/hv/blob/main/hv/page-tables.cpp#L8-L53). All of physical memory is mapped into the host hypervisor's address space, so there is nothing that we're not able to write to.

I was reading the source of ept.h and noticed that ept_free_page_count was set to 100. Does this mean that only 100 pages of the target address space can be EPT switched on access violation? Is there a set limit for it in the Intel standard somewhere?

ept_free_page_count is just an arbitrary number. It is the amount of pages in the pool of memory that we allocate beforehand. Feel free to increase/decrease it however you want.

What if I wanna allocate a range of pages to let's say manually map a DLL into a guests usermode process, can I then take an arbitrary amount of continuous pages matching the size of my DLL anywhere within the guests process and use EPT switching (install_ept_hook()) on said pages to avoid allocating memory with f.ex. VirtualAlloc?

That won't work. EPT hooking isn't really related to allocating memory at all.

And lastly what's the difference between hide_physical_page() and install_ept_hook() in practice?

They are completely different. Pages that are hidden with hide_physical_page() are not accessible to the guest at all. Reading/writing to those pages will just produce junk. Only the hypervisor has access to those pages. install_ept_hook() on the other hand, is about creating two separate views for the same physical page. One view is used whenever the guest reads/writes memory, and the other view is used when the guest executes that memory. This way, you can modify what gets executed while still appearing as if nothing was modified and circumventing integrity checks.


Added Touchevents and minor fix for handling click and mousedown event conflicts

The library can now be used in touch enabled devices to select colour by dragging pickers using touch. There was a minor glitch in the library which does not invoke 'colorpicker-selected' event while clicking on the picker saturation, hue or alpha. This was also fixed.

Hi @manumqln - I tried your branch and it did not resolve the issue with the 'colorpicker-selected' event on click. The event still only contains the correct data on drag but not on click.


num pad + other languages + arrows

If keybind to numpad number doesn't work properly (maybe just me idk)
When using book you can't use arrow keys as in word etc
Also it doesn't support other languages which i know is not a bug but when there will be support for other languages
(I'm not rude i just really want for this mod to be better and to be more popular)

The numpad numbers should work the same as the normal numbers on your keyboard (they're functionally the same keys, usually), the arrow keys should work in the latest version so you might just need to update it.
I completely rely on contributions for language support, I only speak English and Spanish, so if you're after a specific language and know someone who would be willing to translate it, I can get to work on adding it

I can help with Russian,Ukranian,Polish and German my ds is .dushniy the one with plague doctor

https://github.com/JunePrimavera/Minecraft-Vanilla-Notebook/commit/2bf94fccd78c48174011b00ad2be7c379c14197d


Add feature to disable top bar

VSCode Version: Code 1.16.0 (787b31c0474e6165390b5a5989c9619e3e16f953, 2017-09-06T16:12:42.401Z)
OS Version: Darwin x64 16.6.0
Extensions:

Extension
Author (truncated)
Version

beautify
Hoo
1.1.1

vscode-docker
Pet
0.0.16

vscode-eslint
dba
1.3.1

php-intellisense
fel
1.5.1

phpcs
ika
0.7.0

php-docblocker
nei
1.2.0

vetur
oct
0.9.7

vscode-icons
rob
7.12.0

gitblame
wad
2.2.0

(1 theme extensions excluded)

Disabling tabs gives me nothing, since it still has one huge "Tab" at the top. Disabling tabs with "workbench.editor.showTabs": false only disables the tabs feature, it doesn't actually hide the top bar that's taking all the space. Even zen mode has the tabs.

Steps to Reproduce:

Disable tabs "workbench.editor.showTabs": false and watch the top bar appear

Reproduces without extensions: Yes/No

@bpasero @alexandrudima @chrmarti until such a feature is officially considered/implemented, is there a feature in electron/vscode similar to Firefox's userChrome.css that would let willing users to manually target and tweak/disable UI elements?
In Firefox, I (and many other Firefox users enjoying the feature, there's even a subreddit for it!) routinely use this feature to #whatever-ui-component {display: none !important} the UX parts I don't use. Of course, that means I might kill the only entry point for an editor/extension UI component, and I'm okay with that. Is there something similar in Code?

@bpasero can you point to the bug this one is a duplicate of? I searched for it and couldn't find it.

@ronjouch you can use the label workbench-tabs to narrow down the search results and find it.

Found it, thank you. To other subscribers of this bug, you should look at https://github.com/Microsoft/vscode/issues/33607.


mingw under macOS: build errors

Hello Joshua,
I am using macOS 10.11.6, and mingw-w64 installed via the excellent 'brew' package manager.
My MINGW environment var is set to '/usr/local/Cellar/mingw-w64/5.0.1', which contains a bin/ directory that has links to all of the tools.
Compiling using bgfx.cmake I get the following compile-time errors:
In file included from /Volumes/data/mine/SciFiGameProjects/third_party/bx/src/amalgamated.cpp:10:0:
/Volumes/data/mine/SciFiGameProjects/third_party/bx/src/debug.cpp: In function ‘void bx::debugPrintfData(const void*, uint32_t, const char*, ...)’:
/Volumes/data/mine/SciFiGameProjects/third_party/bx/src/debug.cpp:93:25: error: expected ‘)’ before ‘PRIxPTR’
#define DBG_ADDRESS "%" PRIxPTR
^
/Volumes/data/mine/SciFiGameProjects/third_party/bx/src/debug.cpp:106:26: note: in expansion of macro ‘DBG_ADDRESS’
debugPrintf("\ndata: " DBG_ADDRESS ", size: %d\n", _data, _size);
^~~~~~~~~~~
/Volumes/data/mine/SciFiGameProjects/third_party/bx/src/debug.cpp:93:25: error: expected ‘)’ before ‘PRIxPTR’
#define DBG_ADDRESS "%" PRIxPTR
^
/Volumes/data/mine/SciFiGameProjects/third_party/bx/src/debug.cpp:126:23: note: in expansion of macro ‘DBG_ADDRESS’
debugPrintf("\t" DBG_ADDRESS "\t" HEX_DUMP_FORMAT "\t%s\n", data, hex, ascii);
^~~~~~~~~~~
/Volumes/data/mine/SciFiGameProjects/third_party/bx/src/debug.cpp:93:25: error: expected ‘)’ before ‘PRIxPTR’
#define DBG_ADDRESS "%" PRIxPTR
^
/Volumes/data/mine/SciFiGameProjects/third_party/bx/src/debug.cpp:136:22: note: in expansion of macro ‘DBG_ADDRESS’
debugPrintf("\t" DBG_ADDRESS "\t" HEX_DUMP_FORMAT "\t%s\n", data, hex, ascii);
^~~~~~~~~~~
Note that my first order of business was to try this under bgfx's "native" GENie build system, in case the code base itself had trouble supporting this (somewhat obtuse?) build environment. The GENie build does have a problem (and I've filed an Issue), but GENie makes it to link stage, so it does not have this particular issue.
Notes:

The problem is that PRIxPTR is not defined (problem disappears when I throw the def in the file).
The macro definition of PRIxPTR should be in inttypes.h, but only the C99 version (not C++).
I have ./toolchain-x86_64/x86_64-w64-mingw32/include/inttypes.h in my environment, the C version that has this definition (confirmed).
I also have a C++ file version: ./toolchain-x86_64/x86_64-w64-mingw32/include/c++/6.3.0/tr1/inttypes.h. This file does not have the definition (not C99 conformant).
The target file (debug.cpp) is a C++ file. I do not know why this should (or shouldn't) work, but it seems to fail under MINGW on macOS.

I also had a problem with linking, which might manifest for MINGW targets on linux & macOS (any cross compile).
To correct the first problem on my end, I did a very hacky thing: copied definitions from mingw headers to cmake -D defines. You probably don't want to do that, but here a patch of my changes:
diff --git a/cmake/bx.cmake b/cmake/bx.cmake
index b5204b6..304908d 100644
--- a/cmake/bx.cmake
+++ b/cmake/bx.cmake
@@ -21,7 +21,7 @@ file( GLOB BX_SOURCES ${BX_DIR}/src/*.cpp )
 add_library( bx STATIC ${BX_SOURCES} )

 # Link against psapi in Visual Studio
-if( MSVC )
+if( MSVC OR MINGW )
        target_link_libraries( bx PUBLIC psapi )
 endif()

@@ -33,6 +33,10 @@ if( MSVC )
        target_include_directories( bx PUBLIC ${BX_DIR}/include/compat/msvc )
        target_compile_definitions( bx PUBLIC "__STDC_FORMAT_MACROS" )
 elseif( MINGW )
+       add_definitions(-DPRIxPTR=\"I64x\")
+       add_definitions(-DPRIx64=\"I64x\")
+       add_definitions(-DPRIi64=\"I64i\")
+       add_definitions(-DPRId64=\"I64d\")
        target_include_directories( bx PUBLIC ${BX_DIR}/include/compat/mingw )
 elseif( APPLE )
        target_include_directories( bx PUBLIC ${BX_DIR}/include/compat/osx )

Hopefully this is instructive on the problem. With this patch (and after linking the project to C & C++ libs statically), everything is honky dory for bgfx, bx, & my project. GENie's problem is presently unresolved, but I can use bgfx.cmake to complete builds in this environment.
I can also use wine to run my project & tests. So thanks to cmake's cross compilation support, I can build & run all of my targets (macOS, iOS, Win32, Android) on macOS!

Can you try copying that target_compile_definitions with __STDC_FORMAT_MACROS in the MSVC block into the MINGW block? Seems that's necessary in c++ environments.
https://sourceforge.net/p/mingw/mingw-org-wsl/ci/master/tree/include/inttypes.h#l44
Looking at bx's genie configuration it defines that under all platforms, along with a few others I should probably copy just in case.
https://github.com/bkaradzic/bx/blob/db5e1d9b8fcf0f76f3b236455420403a3e639357/scripts/toolchain.lua#L570

That does indeed solve the problem. No hack-definitions necessary.

I believe the latest changes have fixed your issue. If not, we can reopen.


[Fallbacks] Support Procs as fallback values

Using procs, it would be possible to delegate setting fallbacks to associated records and be generally more flexible

No, I still like the idea of having procs as default values one day.


Compound fix

update "items-js.js" from postcards demo to work with Spotlight gallery and Boostrap 5

@hgershone01 this PR will fix the issues with the older postcards code to make the "compound" field work in your collection. Keep in mind it is set up for "compound" items to be images only, not a mix of items types. If you need that more complicated functionality let me know!

Evan Williams' email about these fixes: "You just have two little issues right now from pulling in the old code--> I will fork your repository and create a Pull Request with the fixes, but basically in the items-js.js the "compound" field isn't in the items data, and the elements added for the compound items are styled using bootstrap4 and need to be tweaked for the current template using bootstrap 5. I will send you a fix shortly!" (10/25/2022)


Enable CLAsee checker

This enabled the clasee github action which checks the influxdata CLA on PR open.
If the contributor has not yet signed the CLA it fails the PR build.

[x] CHANGELOG.md updated with a link to the PR (not the Issue)
[x] Well-formatted commit messages
[x] Rebased/mergeable
[x] Tests pass
[x] http/swagger.yml updated (if modified Go structs or API)
[x] Documentation updated or issue created (provide link to issue/pr)
[x] Signed CLA (if not already signed)

@goller absolutely. You thinking README / docs folder or wiki?

@e-dard sweet on it

@timhallinflux are you OK with the new license content added to the README?

If it is any help I lifted the second paragraph of this readme change verbatim from our CLA page.

holding off on this because I overlooked access to secrets by forks


New version: PETSc v0.3.0

Registering package: PETSc
Repository: https://github.com/JuliaParallel/PETSc.jl
Created by: @ViralBShah
Version: v0.3.0
Commit: eed0583afd9a268c22bd5957c8f8b724c0e0ff58
Git reference: HEAD
Description: Julia wrappers for the PETSc library

@boriskaus Any ideas how to fix this one?

yes, I've pushed a fix to main


Changing authorization on vertex does not remove it from other workspaces or cache where that user does not have the new authorization.

Create two accounts one with auth 'A' one without.
Create a vertex without auths
Add the new vertex to both accounts workspaces (leaving both windows open)
Change auth on vertex.

Expected:
On user without auth the vertex should disappear and removed from cache.

Possibly fire a property change event.


1020: Fixing Read lid size corruption issue (#525)

Fixing Read lid size corruption issue (#525)
When HOST read/write data to BMC using DMA then BMC checks the length
of the data to be read/write and divide into chunks if data length is
more than DMA max size.
This change will fix the issue in PLDM when it send data transfer
status to the HOST and sending left over length as response which is
wrong and now PLDM sends correct response with length.

Signed-off-by: Kamalkumar Patel<EMAIL_ADDRESS>

jenkins run tests please


Possibility to store on Azure Blob Storage

For one of the project I am working at my company, I need a remote cache server for Azure Blob Storage, I can provisioned blob storage to test the changes.
From the conversation here https://github.com/vercel/turborepo/discussions/381#discussioncomment-1849991, I believe we can make it work for Azure.
Any pointers on how can I get started with it ?

Hey @shubhamdeodia
My apologies for late response to your ticket. The solution uses a library called stow (github.com/graymeta/stow) which already has support for Azure. I would think it might a few small changes to make it work.
The main step is to ensure you could pass the credentials for Azure Blob Storage and enhance this if-statement to coverage support for azure, see: https://github.com/Tapico/tapico-turborepo-remote-cache/blob/main/main.go#L113
Do you know if Azure has a emulator for Azure Blob Storage?
Feel free to reach out to me on Discord (I am on the Vercel/Turborepo and pnpm groups)


CSS settings seem too generic

By including the timeline's css, the generic label style is changed. This holds true for many of the settings in the style.css. I would think these settings would be specific to the timeline component.

I've updated the CSS to address this.


QA forum vote abuse

Not sure who to reach out to about this - I've already submitted feedback on the forum, but have received no response. For some reason, it seems someone has downvoted every answer I've ever posted, sometimes up to 4 times. Is there any way to get this removed? All downvotes happened at the same time.
https://godotengine.org/qa/user/Eric+Ellingson/answers

Sorry that you're experiencing that, it seems to be a coordinated effort from 3 accounts (likely same person) to downvote everything you post.
This is not the place to discuss it though, please reach out privately to webmaster at godotengine · org.

I also fixed the feedback email which previously went only the contributor who set up the Q2A platform, now it will also go to the webmaster email.


[NFC] mark_dependence handling for coroutines and accessors.

Preparation for lifetime dependent coroutines and accessors.
Includes a minor fix to BeginApply that I missed in the previous PR.
Add flexibility to mark_dependence diagnostics. This allows SILGen to emit mark_dependence [unresolved] in more places.
These changes are covered by unit tests on the branch that they were pulled from.

@swift-ci test


Add endtoend crossdock tests

Adding endtoend crossdock tests
Using scratch base docker image to reduce size of the image
Splitting up the travis job into 2 via matrix to improve build times

Coverage remained the same at 83.951% when pulling 9cbc48ffa469f81ddb7c22cb3960d4956068e04a on add_endtoend_crossdock into e5e135ffca8d583507620c1b127230e565b91218 on master.

node and python still dont work in crossdock, gonna have to figure that one out

Coverage remained the same at 83.951% when pulling 89b9b33b06f8093afd95f508535ae26ad70049df on add_endtoend_crossdock into e5e135ffca8d583507620c1b127230e565b91218 on master.


Add meta field to evidence graph edge list

Comparing biolink to solr it looks like we're missing the meta field from our edges.  This contains useful information on the source of the triple - isDefinedBy, and well as some clique merge data, equivalentOriginalNodeTarget, equivalentOriginalNodeSource, and the label for the edge.

fixed with https://github.com/biolink/biolink-api/pull/302


Not executing triggers

when i try to update or save an Action I get the error:
Something went wrong!
Message format incorrect: string value is None for dictionary value @ data['area']
Response error: 400
If i go to YAML mode and change none to Alarmo it can be saved again.
in v1.5.0, v1.5.0b

one addition. Changing area to Alarmo made it possible to save the action but could not edit it any more in the GUI. I edited the config file and changed the area's to numeric value of the Alarmo (only area) area, using a Notification as an example. Now it works fully again.
Creating a new action will fill in the area with correct value, seems only an issue for existing Actions.

one addition. Changing area to Alarmo made it possible to save the action but could not edit it any more in the GUI. I edited the config file and changed the area's to numeric value of the Alarmo (only area) area, using a Notification as an example. Now it works fully again.
Creating a new action will fill in the area with correct value, seems only an issue for existing Actions.

Sorry for this, my latest release contains quite some Easter eggs (due to the drastic changes).
I will look into this

Sorry for this, my latest release contains quite some Easter eggs (due to the drastic changes).
I will look into this

This is taken care of by https://github.com/nielsfaber/alarmo/commit/b6dde71286d1bb9c56499225dc50a322cd7ce623
For next time: please don't try to manually change the area field. Every area has a unique ID (like '11201212'), assigning it with the name of an area may not give the desired behaviour.
I'm planning to update the GUI a bit to hide some fields from YAML that are not meant to be touched.

This is taken care of by https://github.com/nielsfaber/alarmo/commit/b6dde71286d1bb9c56499225dc50a322cd7ce623
For next time: please don't try to manually change the area field. Every area has a unique ID (like '11201212'), assigning it with the name of an area may not give the desired behaviour.
I'm planning to update the GUI a bit to hide some fields from YAML that are not meant to be touched.

Updating to v1.5.1 should fix this problem.
Please close the issue if the update indeed solved your issue.

Updating to v1.5.1 should fix this problem.
Please close the issue if the update indeed solved your issue.

Seems ok but hard to test any more. The issue was in action that where ibn place before the introduction of the area's :-)


feature: support for slash commands

Support for slash commands as described here https://github.com/discord/discord-api-docs/pull/2295
Todo

[x] Add support on the gateway for the new event. (Blocked on missing documentation in the above linked pr)
[ ] Answer unresolved questions. (See TODO: in the patch)
[ ] Make /slash into a example somewhere?
[ ] Write documentation.
[ ] Add builders for CommandOption to make them nicer to work with, possible in misc.
[x] Add support for the endpoints to execute interactions when using it over the gateway.

[x] create

[x] modify

[x] delete

[ ] Check all the new models.
[ ] Format, clean up and sort everything.
[ ] Add new partial member fields to the updates in the cache.
[x] Split the large files up into multiple files.
[ ] Actually make the http example do something and not just be a copy of the other example.
[x] Add message type 20.
[x] Add new field to the ready event.
[ ] Fix tests.
[ ] Test everything.
[ ] Make :paperclip: happy.

One of the prerequisites to this PR, as far I understand, is the routes in PR #643 supporting a different route (webhook ID vs. application ID). So as to avoid ignoring clippy errors about dead code, I've included a patch which includes that support.

Patch
diff --git a/http/src/request/channel/webhook/delete_webhook_message.rs b/http/src/request/channel/webhook/delete_webhook_message.rs
index 743a74e4..9358f277 100644
--- a/http/src/request/channel/webhook/delete_webhook_message.rs
+++ b/http/src/request/channel/webhook/delete_webhook_message.rs
@@ -27,10 +27,8 @@ use twilight_model::id::{MessageId, WebhookId};
 pub struct DeleteWebhookMessage<'a> {
     fut: Option<Pending<'a, ()>>,
     http: &'a Client,
-    message_id: MessageId,
     reason: Option<String>,
-    token: String,
-    webhook_id: WebhookId,
+    route: Route,
 }

 impl<'a> DeleteWebhookMessage<'a> {
@@ -43,19 +41,24 @@ impl<'a> DeleteWebhookMessage<'a> {
         Self {
             fut: None,
             http,
-            message_id,
+            route: Route::DeleteWebhookMessage {
+                message_id: message_id.0,
+                token: token.into(),
+                webhook_id: webhook_id.0,
+            },
             reason: None,
-            token: token.into(),
-            webhook_id,
         }
     }

+    /// Set the ratelimiting route to use in the request.
+    pub(crate) fn route(mut self, route: Route) -> Self {
+        self.route = route;
+
+        self
+    }
+
     fn request(&self) -> Result<Request> {
-        let route = Route::DeleteWebhookMessage {
-            message_id: self.message_id.0,
-            token: self.token.clone(),
-            webhook_id: self.webhook_id.0,
-        };
+        let route = self.route.clone();

         Ok(if let Some(reason) = &self.reason {
             let headers = request::audit_header(&reason)?;
diff --git a/http/src/request/channel/webhook/update_webhook_message.rs b/http/src/request/channel/webhook/update_webhook_message.rs
index 9ccfd652..5cc0f1f7 100644
--- a/http/src/request/channel/webhook/update_webhook_message.rs
+++ b/http/src/request/channel/webhook/update_webhook_message.rs
@@ -120,10 +120,8 @@ pub struct UpdateWebhookMessage<'a> {
     fields: UpdateWebhookMessageFields,
     fut: Option<Pending<'a, ()>>,
     http: &'a Client,
-    message_id: MessageId,
     reason: Option<String>,
-    token: String,
-    webhook_id: WebhookId,
+    route: Route,
 }

 impl<'a> UpdateWebhookMessage<'a> {
@@ -143,10 +141,12 @@ impl<'a> UpdateWebhookMessage<'a> {
             },
             fut: None,
             http,
-            message_id,
             reason: None,
-            token: token.into(),
-            webhook_id,
+            route: Route::UpdateWebhookMessage {
+                message_id: message_id.0,
+                token: token.into(),
+                webhook_id: webhook_id.0,
+            },
         }
     }

@@ -254,13 +254,17 @@ impl<'a> UpdateWebhookMessage<'a> {
         Ok(self)
     }

+
+    /// Set the ratelimiting route to use in the request.
+    pub(crate) fn route(mut self, route: Route) -> Self {
+        self.route = route;
+
+        self
+    }
+
     fn request(&self) -> Result<Request> {
         let body = crate::json_to_vec(&self.fields)?;
-        let route = Route::UpdateWebhookMessage {
-            message_id: self.message_id.0,
-            token: self.token.clone(),
-            webhook_id: self.webhook_id.0,
-        };
+        let route = self.route.clone();

         Ok(if let Some(reason) = &self.reason {
             let headers = request::audit_header(&reason)?;

Please re-open targeting the trunk branch

Please re-open targeting the trunk branch


[storage] edits to blob, queue, and file docstrings

#7785

Can one of the admins verify this patch?


up breaking-02

Adds the latest from master to breaking change

Codecov Report

Merging #265 (bdf0f35) into breaking-02 (7fceb4b) will decrease coverage by 0.02%.
The diff coverage is n/a.

:exclamation: Current head bdf0f35 differs from pull request most recent head dbac4f1. Consider uploading reports for the commit dbac4f1 to get more accurate results

@@               Coverage Diff               @@
##           breaking-02     #265      +/-   ##
===============================================
- Coverage        70.69%   70.67%   -0.03%
===============================================
  Files               15       14       -1
  Lines             2020     1923      -97
===============================================
- Hits              1428     1359      -69
+ Misses             592      564      -28

Impacted Files
Coverage Δ

src/afw.jl
77.05% <ø> (-0.27%)
:arrow_down:

src/types.jl
67.36% <0.00%> (-1.44%)
:arrow_down:

src/oracles.jl
86.59% <0.00%> (-1.14%)
:arrow_down:

src/active_set.jl
84.21% <0.00%> (-0.30%)
:arrow_down:

src/pairwise.jl
65.25% <0.00%> (-0.30%)
:arrow_down:

src/fw_algorithms.jl
77.30% <0.00%> (-0.26%)
:arrow_down:

src/blended_cg.jl
50.13% <0.00%> (-0.26%)
:arrow_down:

src/norm_oracles.jl
95.76% <0.00%> (-0.21%)
:arrow_down:

src/function_gradient.jl
97.05% <0.00%> (-0.17%)
:arrow_down:

... 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 7fceb4b...dbac4f1. Read the comment docs.


On android tapping a graph to get tooltip briefly highlights the graph

Not a big deal

Not getting this now..

fixed in 0c6d0859e5d3b52eb454fc0632fd33b3e48ece9f 👍🏽

Woop!

need to get this merged :)

Thanks :D


Bug in join (and including=)

Hi,
[weppy 0.7.8]
Using the structure of bloggy app:
users = User.all().join('posts').select()
Traceback (most recent call last):
File "", line 1, in
File "../venv/lib/python3.4/site-packages/weppy/dal/helpers.py", line 471, in select
self.db, [], jcolnames[join[1]], compact=False)
KeyError: 'posts'
using including:
users = User.all().select(including='posts')
Traceback (most recent call last):
File "", line 1, in
File "/lib/python3.4/site-packages/weppy/dal/base.py", line 102, in select
self, jdata).select(*fields, **options)
File "/venv/lib/python3.4/site-packages/weppy/dal/helpers.py", line 523, in select
self.db, [], jcolnames[join[1]], compact=False)
KeyError: 'posts
Jose

@josejachuf I'm gonna inspect this asap.

@josejachuf can you give more details about what you have stored inside the db?
Tested with a fresh bloggy app (run setup, so just the admin user and no posts):
Python 3.5.1 (default, Feb 20 2016, 16:05:46)
[GCC 4.2.1 Compatible Apple LLVM 7.0.2 (clang-700.1.81)] on darwin
weppy 0.8-dev shell on app: bloggy
>>> User.all().count()
1
>>> Post.all().count()
0
>>> users = User.all().join('posts').select()
>>> users
<Rows (0)>
>>> users = User.all().select(including='posts')
Traceback (most recent call last):
  File "<console>", line 1, in <module>
  File "/Users/giovanni/Development/Weppy/weppy/weppy/dal/base.py", line 102, in select
    self, jdata).select(*fields, **options)
  File "/Users/giovanni/Development/Weppy/weppy/weppy/dal/helpers.py", line 523, in select
    self.db, [], jcolnames[join[1]], compact=False)
KeyError: 'posts'

As you can see I can reproduce the including error but not the join one.

Hi @gi0baro
I have a user and several post

@josejachuf ok, you're right, I created a post and now I can reproduce both the errors. Will work on these asap.

@gi0baro
print(jcolnames)
{'"posts"': ['"id"', '"title"', '"text"', '"date"', '"user"'], '"users"': ['"id"', '"created_at"', '"updated_at"', '"email"', '"password"', '"registration_key"', '"reset_password_key"', '"registration_id"', '"first_name"', '"last_name"']}
why the double quote? '"posts"' (' and ")

@josejachuf this is published on 0.7.9


python3Packages.bottle: 0.12.25 -> 0.13.0

Description of changes
Changelog: https://bottlepy.org/docs/dev/changelog.html#release-0-13
Changes: https://github.com/bottlepy/bottle/compare/0.12.25...0.13.0
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] 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/)
24.11 Release Notes (or backporting 23.11 and 24.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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Done in https://github.com/NixOS/nixpkgs/commit/0906cdd57fcc49a4723494c968d210d470e55ca3.


CUAV CAN_GPS v1.2 cannode (stm32f412) + UAVCAN bootloader

https://github.com/cuav/hardware/tree/master/CAN_GPS
Waiting on minor NuttX fixes for STM32F412CE.

https://github.com/PX4/NuttX/pull/119

No impact to existing boards, so this is harmless to merge and continue iterating.


Check OU not working

The Check OU exist when adding the user attribute is not working. Due to this it is trying to create the user account at the root level of the domain. Currently it is bypassed and just assuming there are no typos.

Closed by PR #7


WFCORE-4702 Add deployment phases for MicroProfile Fault Tolerance

https://issues.jboss.org/browse/WFCORE-4702

@msve seems MP FT is in move

@jmesnil This has been updated as you requested and rebased so this is ready (I am realizing we probably need to proactively ping here but I assumed there is a notification send by github).

I see there is now an option to ask for re-review, so that is probably what was needed to get proper notifications.

@rhusar thanks


Fix deprecation warnings due to invalid escape sequences.

Fixes #50

Coverage remained the same at 0.0% when pulling 82512803cd095e6f62c5a9e908f84158c5994761 on tirkarthi:fix-warnings into ebbf88a5ac137100732bd7ba23b05cf941d7be83 on note35:master.


[docs] custom default props resolver

The built-in resolver for defaultProps in react-docgen was working so far. With #14761 and especially #15023 we need a custom resolver so that we don't have to duplicate default props resolution (adds noise, potential for bugs etc).

Closed in #15473.


fix: deplacement upload fichier organisme transport - 100

:tada: This PR is included in version 0.11.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


GH-42228: [CI][Java] Suppress transfer progress log in java-jars

Rationale for this change
Transfer progress log is meaningless and noisy in CI log.
What changes are included in this PR?
Use the --no-transfer-progress option that is available Maven 3.6.1: https://maven.apache.org/ref/3.6.1/maven-embedder/cli.html
Other approach is using -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn like other our scripts do.
I chose --no-transfer-progress because it's shorter.
Are these changes tested?
Yes.
Are there any user-facing changes?
No.

GitHub Issue: #42228

@github-actions crossbow submit java-jars

Doesn't seem to work https://github.com/ursacomputing/crossbow/actions/runs/9604601971/job/26491413424#step:6:22

I'm working on java-jars.

@github-actions crossbow submit java-jars

Oh, it seems that we need more --no-transfer-progress.

I'm working on java-jars.

Are you working on this (reduce needless CI logs) too?

I'm working on java-jars.

Are you working on this (reduce needless CI logs) too?

Ah! What I meant was just fixing errors related to java-jars.

https://github.com/apache/arrow/pull/42225

I've merged the other PRs so java-jars should pass now

Same error in both failing CIs. Cannot find protobuf?

Related https://github.com/apache/arrow/issues/42149 ?

Ah! What I meant was just fixing errors related to java-jars. Some errors need to be fixed for a successful java-jars workflow.

Ah! I see!

Log size is reduced:
Before: https://github.com/ursacomputing/crossbow/actions/runs/9594213517/job/26457788792
...
41542 Error:  After correcting the problems, you can resume the build with the command
41543 Error:    mvn <args> -rf :flight-sql-jdbc-driver
41544 Error: Process completed with exit code 1.

After: https://github.com/ursacomputing/crossbow/actions/runs/9606491188/job/26496791524
...
12517 Error:  After correcting the problems, you can resume the build with the command
12518 Error:    mvn <args> -rf :flight-sql-jdbc-driver
12519 Error: Process completed with exit code 1.

I'll merge this.


An user should be able to tell which backend the job ran on if none was passed.

What is the expected feature or enhancement?
If no backend_name is passed, an user should still be able to programmatically find out which system their job ran on.
Acceptance criteria
The above is true.

This should already be true at the api level. If not, we can take it as a defect.
It might be something in the client is not exposing it or it’s not clear how to get it otherwise.

Ok yeah no it was my mistake.  I missed job.backend queries for the device the job is running on.


Unit tests

Added unit tests. Some changes in training file

Merged from unit tests

Merged from branch unit-tests


where can i get the dataset file 'enwiki-20171001-pages-meta-current-withlinks-abstracts'?

I cannot find it in https://hotpotqa.github.io/

https://hotpotqa.github.io/wiki-readme.html


Support for Generic Reference Types

All changes currently live in the custom_links module.
At a glance:

Every intrusive collection has corresponding Link and NodeRef traits, and raw pointer based link and node reference types:

RawLink replaces Link and implements the Link trait.
RawLinkRef replaces NodePtr and implements the NodeRef trait.

Adapter::Link must implement the Link trait of the corresponding collection.
NodeRef associated type is added to Adapter.
Conversion methods added to Adapter:

node_from_pointer
node_into_pointer
node_from_link
node_into_link

Conversion "pathway" for Adapter:
Pointer <-> NodeRef <-> *const Link <-> *const Value

TODO:

[x] SinglyLinkedList
[ ] LinkedList
[ ] RBTree
[ ] XorLinkedList
[ ] Documentation
[ ] intrusive_adapter! support for custom link and node reference type
[ ] move from custom_links module into crate root.

Solves #27

Is it really necessary to expose NodeRef as a configurable type? Can't we just use a raw pointer instead? I would prefer keeping NodeRef as an internal detail.

It's only necessary if a Link implementation cannot use raw pointers, which only happens when trying to minimize the size of Link types.
For example, if all objects are backed by an arena/slab, and the cost of converting NodeRef into *const A::Link is negligible for the application (e.g. shift and add), a u8 or u16 can be used instead, saving 18 to 21 bytes (75% to 87%) per RBTreeLink.
I already planned to move the methods marked #[doc(hidden)] out of NodeRef, would that be satisfactory?

How about something like this:
trait Link {
    type LinkPtr;
    fn next(&self) -> LinkPtr;
    fn set_next(&self, ptr: LinkPtr);
}

The Adapter can take care of converting between LinkPtr and *const Link. The default RawLink just has type LinkPtr = *const Link.
The NodeRef (previously called NodePtr) should stay a private type.

Shouldn't set_next be unsafe? As per your above comments, it should be.
trait Link {
    type LinkPtr;
    fn next(&self) -> Self::LinkPtr;
    unsafe set_next(&self, next: Self::LinkPtr);
}

Also, struct NodePtr was refactored into trait NodeRef and struct RawLinkRef. Did you want me to revert this?

Actually, I'd like to take a few hours to study what the best design for this is. I will get back to you later today.

OK, so here's the design I came up with:
// adapter.rs

pub unsafe trait Adapter {
    /// Collection-specific link type which allows an object to be inserted in
    /// an intrusive collection.
    type LinkOps: LinkOps;

    /// Object type which is inserted in an intrusive collection.
    type Value: ?Sized;

    /// Pointer type which owns an instance of a value.
    type Pointer: IntrusivePointer<Self::Value>;

    /// Gets a reference to an object from a reference to a link in that object.
    unsafe fn get_value(&self, link: <Self::LinkOps as LinkOps>::LinkPtr) -> *const Self::Value;

    /// Gets a reference to the link for the given object.
    unsafe fn get_link(&self, value: *const Self::Value) -> <Self::LinkOps as LinkOps>::LinkPtr;

    fn link_ops(&self) -> &Self::LinkOps;

    fn link_ops_mut(&mut self) -> &mut Self::LinkOps;
}

// Base trait for link operations
pub trait LinkOps {
    type LinkPtr;
    fn is_linked(&self, ptr: Self::LinkPtr) -> bool;
    unsafe fn mark_unlinked(&mut self, ptr: Self::LinkPtr);
}

// singly_linked_list.rs

// Generic LinkOps interface for SinglyLinkedList
pub unsafe trait SinglyLinkedListOps: super::LinkOps {
    fn next(&self, ptr: Self::LinkPtr) -> Self::LinkPtr;
    unsafe fn set_next(&mut self, ptr: Self::LinkPtr, next: Self::LinkPtr);
}

pub struct Link {
    next: Cell<NodePtr>,
}

// Default LinkOps implementation for SinglyLinkedList, which uses Link.
pub struct LinkOps;
impl super::LinkOps for LinkOps {
    type LinkPtr = *const Link;
    fn is_linked(&self, ptr: Self::LinkPtr) -> bool { ... }
    unsafe fn mark_unlinked(&mut self, ptr: Self::LinkPtr) { ... }
}
impl SinglyLinkedListOps for LinkOps {
    fn next(&self, ptr: Self::LinkPtr) -> Self::LinkPtr { ... }
    unsafe fn set_next(&mut self, ptr: Self::LinkPtr, next: Self::LinkPtr) { ... }
}

This design is heavily inspired by Boost.Intrusive's support for custom value traits.
Making LinkOps a separate trait from Adapter has several advantages:

The core of the list/tree manipulation code (e.g. rbtree rebalancing) only has to be generic over LinkOps, not Adapter. This allows signficant compilation time and code size savings since the same LinkOps is often reused for lists of different Value types.
A single Link type can support multiple LinkOps. So for example, a list::Link could support being inserted into a SinglyLinkedList.
List mutation is done with a &mut LinkOps which allows the LinkOps to own the actual Values (e.g. in an arena).

Then is NodePtr necessary?
All of its non-accessor methods manipulate its collection in terms of *const Link via (*self.0). If those methods are refactored into generic free functions over LinkOps, and LinkOps implements the accessors, what should be NodePtr contain?

You are right, NodePtr isn't needed any more. Some of the larger functions (particularly in RBTree) can be replaced with free functions, others can just be inlined directly.
By the way, I've made some minor changes to the design above:

set_next and next use Option<LinkPtr> instead of LinkPtr.
LinkPtr is now Copy.

Why must Adapter::Pointer implement IntrusivePointer, in all cases?
Is it implied that a non-pointer type (e.g. Offset, other !Deref types) must be converted into a type that implements IntrusivePointer before it is inserted?
A trivial use case is allocating an Offset<'arena, T>, and using the default LinkOps to insert it into an intrusive collection that does not outlive the arena.
Yes, Offset<'arena, T> can be converted into &'arena T using a stateful conversion, but what happens when it is removed from the collection?
If the arena supports deallocation (e.g. slab with freelist), &'arena T must be converted back into Offset<'arena, T> or OffsetMut<'arena, T> after every deletion/removal, otherwise there's a memory leak. This becomes a manual process, and is impossible from within the methods drop and clear of the intrusive collection.
None of the intrusive collections actually touch Adapter::Pointer except when converting it into/from *const Adapter::Link or *const Adapter::Value.
In the past you had suggested that Adapter would have an "index <-> pointer conversion".

I guess we could move the from_raw and into_raw of IntrusivePointer into Adapter, which would allow stateful conversions. Pointer is supposed to represent and owned value which is inserted into or taken out of a collection.
So this leaves us with 3 types:

LinkPtr is the pointer type used internally by the collection.
&Value is the reference type returned by get().
Pointer is the owned pointer which is inserted/removed from the collection.

Also LinkOps needs a ptr_eq method because RBTree and LinkedList need pointer equality.
pub trait LinkOps {
    type LinkPtr: Copy;
    //...
    unsafe fn ptr_eq(&self, this: Self::LinkPtr, other: Self::LinkPtr) -> bool;
}

This follows the pattern of Rc and Arc, where the definition of pointer equality is independent of whether or not the pointer implements Deref and Deref::Target implements Eq.

Would it be enough to just have LinkPtr: Copy + Eq? Or can you think of a situation where this wouldn't be enough?

I fixed my above comment. ptr_eq should be preferred to Copy only if we allow types that implement Deref to be LinkPtr, which is why I referenced Rc and Arc.
I believe only a rather fervent "safety purist" would ever attempt to use a pinned smart pointers instead of implementing LinkOps.
Every other case I can think of is covered already:

optimal time: use default LinkOps
optimal space: implement LinkOps<LinkPtr = u8>
intrusive collection in a durable memory segment: LinkOps<LinkPtr = Offset>

I see. I think that if you really wanted to use Rc or Arc then you could just make a wrapper around them which implements Eq using ptr_eq. I don't expect anyone to actually do this in practice though.

I think that from_raw and into_raw should not be moved into the Adapter trait, and instead should be members of a PointerOps<Pointer> trait.
// adapter.rs
pub unsafe trait Adapter {
    /// Collection-specific link type which allows an object to be inserted in
    /// an intrusive collection.
    type LinkOps: LinkOps;

    /// Object type which is inserted in an intrusive collection.
    type Value: ?Sized;

    /// Pointer type which owns an instance of a value.
    type Pointer;

    /// Collection-specific pointer conversions which allow an object to
    /// be inserted in an intrusive collection.
    type PointerOps: PointerOps<Self::Pointer, Value = Self::Value>;

    /// Gets a reference to an object from a reference to a link in that object.
    unsafe fn get_value(&self, link: <Self::LinkOps as LinkOps>::LinkPtr) -> *const Self::Value;

    /// Gets a reference to the link for the given object.
    unsafe fn get_link(&self, value: *const Self::Value) -> <Self::LinkOps as LinkOps>::LinkPtr;

    fn link_ops(&self) -> &Self::LinkOps;

    fn link_ops_mut(&mut self) -> &mut Self::LinkOps;

    fn pointer_ops(&self) -> &Self::PointerOps;
}

// pointer_ops.rs

pub trait PointerOps<Pointer> {
    type Value: ?Sized;

    /// Constructs an owned pointer from a raw pointer.
    ///
    /// # Safety
    /// The raw pointer must have been previously returned by `into_raw`.
    unsafe fn from_raw(&self, value: *const Self::Value) -> Pointer;

    /// Consumes the owned pointer and returns a raw pointer to the owned object.
    fn into_raw(&self, ptr: Pointer) -> *const Self::Value;
}

/// The default implementation that is used when `PointerOps`
/// is not specified in the invocation of `intrusive_adapter!`.
pub struct IntrusivePointerOps;
impl<'a, T: ?Sized> PointerOps<&'a T> for IntrusivePointerOps {
    type Value = T;

    #[inline]
    unsafe fn from_raw(&self, raw: *const Self::Value) -> &'a T { ... }

    #[inline]
    fn into_raw(&self, ptr: &'a T) -> *const Self::Value { ... }
}
impl<T: ?Sized> PointerOps<UnsafeRef<T>> for IntrusivePointerOps {
    type Value = T;

    #[inline]
    unsafe fn from_raw(&self, raw: *const Self::Value) -> UnsafeRef<T> { ... }

    #[inline]
    fn into_raw(&self, ptr: UnsafeRef<T>) -> *const Self::Value { ... }
}
#[cfg(feature = "alloc")]
impl<T: ?Sized> PointerOps<Box<T>> for IntrusivePointerOps {
    type Value = T;

    #[inline]
    unsafe fn from_raw(&self, raw: *const Self::Value) -> Box<T> { ... }

    #[inline]
    fn into_raw(&self, ptr: Box<T>) -> *const Self::Value { ... }
}
#[cfg(feature = "alloc")]
impl<T: ?Sized> PointerOps<Rc<T>> for IntrusivePointerOps {
    type Value = T;

    #[inline]
    unsafe fn from_raw(&self, raw: *const Self::Value) -> Rc<T> { ... }

    #[inline]
    fn into_raw(&self, ptr: Rc<T>) -> *const Self::Value { ... }
}
#[cfg(feature = "alloc")]
impl<T: ?Sized> PointerOps<Arc<T>> for IntrusivePointerOps {
    type Value = T;

    #[inline]
    unsafe fn from_raw(&self, raw: *const Self::Value) -> Arc<T> { ... }

    #[inline]
    fn into_raw(&self, ptr: Arc<T>) -> *const Self::Value { ... }
}

The advantages of not including from_raw and into_raw in Adapter include:

Default implementations of from_raw and into_raw for a set of well-known pointer types. Currently this handled by IntrusivePointer.
intrusive_adapter! matches the same patterns as before, unless you want a custom pointer type.

I generally like the idea of PointerOps, but would make a few small changes:
// adapter.rs
pub unsafe trait Adapter {
    /// Collection-specific link type which allows an object to be inserted in
    /// an intrusive collection.
    type LinkOps: LinkOps;

    /// Collection-specific pointer conversions which allow an object to
    /// be inserted in an intrusive collection.
    type PointerOps: PointerOps;

    /// Gets a reference to an object from a reference to a link in that object.
    unsafe fn get_value(&self, link: <Self::LinkOps as LinkOps>::LinkPtr) -> *const <Self::PointerOps as PointerOps>::Value;

    /// Gets a reference to the link for the given object.
    unsafe fn get_link(&self, value: *const <Self::PointerOps as PointerOps>::Value) -> <Self::LinkOps as LinkOps>::LinkPtr;

    fn link_ops(&self) -> &Self::LinkOps;

    fn link_ops_mut(&mut self) -> &mut Self::LinkOps;

    fn pointer_ops(&self) -> &Self::PointerOps;
}

// pointer_ops.rs

pub unsafe trait PointerOps {
    type Value: ?Sized;
    type Pointer;

    /// Constructs an owned pointer from a raw pointer.
    ///
    /// # Safety
    /// The raw pointer must have been previously returned by `into_raw`.
    unsafe fn from_raw(&self, value: *const Self::Value) -> Self::Pointer;

    /// Consumes the owned pointer and returns a raw pointer to the owned object.
    fn into_raw(&self, ptr: Self::Pointer) -> *const Self::Value;
}

/// The default implementation that is used when `PointerOps`
/// is not specified in the invocation of `intrusive_adapter!`.
pub struct DefaultPointerOps<Pointer>;
unsafe impl<'a, T: ?Sized> PointerOps for DefaultPointerOps<&'a T> {
    type Value = T;
    type Pointer = &'a T;

    #[inline]
    unsafe fn from_raw(&self, raw: *const T) -> &'a T { ... }

    #[inline]
    fn into_raw(&self, ptr: &'a T) -> *const T { ... }
}

Note that the current specification of intrusive_adapter! doesn't have to change. We just need to add Ops as an associated type to Link which the macro will automatically pick up.
intrusive_adapter!(MyAdapter = Box<Test>: Test { link: LinkedListLink })

impl LinkedListLink {
    pub type Ops = LinkOps;
}

Apparently, associated types are not supported in inherent impls (see rust-lang/rust#8995).
A DefaultLinkOps trait would suffice:
// link_ops.rs

pub trait DefaultLinkOps {
    type Ops: LinkOps + Default;
}

// linked_list.rs

impl DefaultLinkOps for Link {
    type Ops = LinkOps;
}

Sounds good. Can you start pushing your changes so that I can start reviewing the implementation?

Thanks a lot for your hard work getting this feature implemented @amari!


Run The Numbers

Run The Numbers
-Countdown.
-Audit.
1 curl https://raw.githubusercontent.com/BashCo/RunTheNumbers/master/runthenumbers_bitcoin-cli.sh | bash
2 gettxoutsetinfo
Option 1 Settings in .sh by BashCo.
#!/bin/bash
IP_ADDRESS="<IP_ADDRESS>"
PORT="8332"
This is the block height we're waiting for
TARGET_BLOCK="650000"
OUTPUT_FILE="the_numbers_${TARGET_BLOCK}.txt"
We'll set this to zero so we can update the console only when there's a new block
LATEST_BLOCK="0"
This will query your node repeatedly until the target block height is reached
while true
do
# This fetches the current block height from your full node.
CURRENT_BLOCK=$(bitcoin-cli getblockcount)

# If the block height matches, call gettxoutsetinfo and print to file
if [ $CURRENT_BLOCK = $TARGET_BLOCK ]; then
    echo "$CURRENT_BLOCK/$TARGET_BLOCK: running the numbers...";
    TXOUTSETINFO=$(bitcoin-cli gettxoutsetinfo)
    echo
    echo
    echo "${TXOUTSETINFO}"
    echo
    echo
    echo "${TXOUTSETINFO}" > "$OUTPUT_FILE"
    echo "congratulations, you have run the numbers. output saved to $OUTPUT_FILE"
    exit

# If block height doesn't match yet, update console when a new block is found
elif [ $CURRENT_BLOCK -lt $TARGET_BLOCK ] && [ $CURRENT_BLOCK -gt $LATEST_BLOCK ]; then
    LATEST_BLOCK=$CURRENT_BLOCK
    REMAINING=$(($TARGET_BLOCK - $CURRENT_BLOCK))
    BLOCKS="blocks"
    if [ $REMAINING = 1 ]; then
        BLOCKS="block"
    fi
    echo "$CURRENT_BLOCK/$TARGET_BLOCK: ${REMAINING} ${BLOCKS} remaining"

# If block height has already passed, go home and do better next time
elif [ $CURRENT_BLOCK -gt $TARGET_BLOCK ]; then
    echo "${TARGET_BLOCK} has already passed. current block is ${CURRENT_BLOCK}";
    exit

fi

# Query block height every n seconds
sleep 5

done

Este código no funciona correctamente_
1 curl https://raw.githubusercontent.com/BashCo/RunTheNumbers/master/runthenumbers_bitcoin-cli.sh | bash
Podrías verificarlo?


get high returns: pancakeswap returns no results

https://momentumbots.palomachain.com/?dex=pancakeswap_new&lag_return=0&daily_volume=0&vol_30=0&market_cap=0
https://momentumbots.palomachain.com/?dex=biswap&lag_return=0&daily_volume=0&vol_30=0&market_cap=0
return no results. All other DEX listed here https://github.com/VolumeFi/paloma/issues/223 do.
Error:
 Traceback (most recent call last):
   File "/root/.cache/pypoetry/virtualenvs/trading-bots-Fx44JTns-py3.10/lib/python3.10/site-packages/bottle.py", line 876, in _handle
     return route.call(**args)
   File "/root/.cache/pypoetry/virtualenvs/trading-bots-Fx44JTns-py3.10/lib/python3.10/site-packages/sentry_sdk/integrations/bottle.py", line 140, in wrapped_callback
     raise exception
   File "/root/.cache/pypoetry/virtualenvs/trading-bots-Fx44JTns-py3.10/lib/python3.10/site-packages/sentry_sdk/integrations/bottle.py", line 130, in wrapped_callback
     res = prepared_callback(*args, **kwargs)
   File "/root/.cache/pypoetry/virtualenvs/trading-bots-Fx44JTns-py3.10/lib/python3.10/site-packages/bottle.py", line 1759, in wrapper
     rv = callback(*a, **ka)
   File "/root/app/momentum_cache_refresh/app.py", line 29, in get_high_returns
     df = momentum_scanner_intraday.get_high_returns(
   File "/root/app/momentum_cache_refresh/momentum_scanner_intraday.py", line 73, in get_high_returns
     vols = gecko.exchanges(dex)
   File "/root/app/momentum_cache_refresh/gecko.py", line 47, in exchanges
     for ticker in get("exchanges", dex)["tickers"]
 KeyError: 'tickers'

as discussed - removed biswap
on 6:27am 5/24 https://momentumbots.palomachain.com/?dex=uniswap_v2&lag_return=6&daily_volume=0&vol_30=1&market_cap=1 also returned {"reason":{}}


Check soundscape jobs in TCN RFCx project

Check soundscape jobs in this project
https://arbimon.org/project/tnc-rfcx-biodiversity-in-parque-nacional-braulio-carrillo-and-la-selva-oet/jobs

Create job LS24 (53079) 2024_2 with the same parameters
[TBD- waiting for params]

LS08 (53080) 2024 and LS25 (53080) 2024 are not completely done. @RatreeOchn Can you put these jobs back in queue?


Update is missing again (28/07/23)

Hallo @Rutger265 ,
thanks for opening the issue. Unfortunately we had some technical problems providing the data. We just uploaded an up to date version from today. If there is still something missing feel free to reopen this issue.
Best regards
Knut Perseke
for the RKI Open Data Team


Add Custom Protocol Support When Notification is clicked

Allows for a custom protocol to be called when the notification is called. This is useful to get around Toast Notifications of now allowing the item to be dismissed. This could be configured to regenerate the Toast Notification or perform some other action such as calling the same protocol that is assigned to the Action button

Sorry. For some reason GitHub's hub CLI tried opened a PR using the Wrong Repo


Document the proxy's behavior w.r.t. localhost/loopback/<IP_ADDRESS>/::1

IIUC, the iptables rules have at least one special exception for loopback interfaces where the proxy won't intercept a connections that involve loopback addresses. I see a test for one case, "connecting to loopback from non-proxy container does not get redirected to proxy." However, in general it isn't clear what the expected behavior w.r.t. loopback addresses is supposed to be:
@pcalcado:

By default, when will the proxy avoid intercepting a request from outside the pod to a loopback address?
By default, when will the proxy avoid intercepting a request from inside the pod to a loopback address?
What features are provided to control whether the proxy will intercept requests between containers in the same pod communicating over a loopback address?

Similarly, it isn't clear how service discovery works w.r.t. these names and addresses:
@seanmonstar:

Does the proxy implement the special-case logic for "localhost" and ".localhost" and ".localhost." names to avoid contacting the Destination service?
Does the proxy implement special-case logic for "<IP_ADDRESS>" and "::1" to avoid contacting the Destination service?

It is important for security and performance reasons that we don't do service discovery for these names and addresses because applications use these names to ensure that they'll never do network traffic outside their node/pod, so it's important to verify that we're not doing so and document that that is the case.

I'm not aware of any special casing in the proxy itself around the handling of localhost or loopback IP addresses.

I think a README.md in https://github.com/runconduit/conduit/tree/master/proxy-init would be a good place to document the iptables behaviors that proxy-init configures.

I'm not aware of any special casing in the proxy itself around the handling of localhost or loopback IP addresses.

I filed #358 and #359 to fix this.

@pcalcado Could you please describe how the proxy-init container is designed to work w.r.t. loopback/localhost. Is it simply that it tries to bypass the proxy for all localhost traffic? And, there's no way to change this, right? I am planning to write the user-facing documentation for this as part of the service discovery doc, and it would be useful to better understand these details of proxy-init.


Got wrong results when parse from File and that includes 3 bytes char with skip_lines

I haven't figured out how to solve that problem, but i found some step to reproduce it:

read data from File.open
skip_lines options are presence
data length is over the chunk size (32 * 1024)
data includes 3 bytes char (such as chinese)
v3.0.1 works correctly, It's happening in v3.0.2 and afters

test case:
def test_three_bytes_chars
  Tempfile.create(['temp', '.csv']) do |tempfile|
    tempfile.close
    path = tempfile.path

    text = "\xE5\x93\x88\xE5\x9B\x89"
    File.open(path, "w") do |csv|
      row = [text].join(',').concat("\n")
      row_count = (32 * 1024) / (StringIO.new(row).length) + 1
      csv << (row * row_count)
    end;
    assert_equal(
      [text],
      CSV.read(File.open(path), headers: true, :skip_lines => /\A#/).to_a.last
    )
  end
end

Results in Ruby 2.5.5:
# Running tests:

[102/756] TestCSVFeatures#test_three_bytes_chars = 0.06 s
  1) Failure:
TestCSVFeatures#test_three_bytes_chars [/Users/jeff/project/csv/test/csv/test_features.rb:341]:
<["哈囉"]> expected but was
<["囉"]>.

[142/756] TestCSVFeatures::DifferentOFS#test_three_bytes_chars = 0.06 s
  2) Failure:
TestCSVFeatures::DifferentOFS#test_three_bytes_chars [/Users/jeff/project/csv/test/csv/test_features.rb:341]:
<["哈囉"]> expected but was
<["囉"]>.

Finished tests in 2.863357s, 264.0258 tests/s, 2669.9430 assertions/s.
756 tests, 7645 assertions, 2 failures, 0 errors, 0 skips

ruby -v: ruby 2.5.5p157 (2019-03-15 revision 67260) [x86_64-darwin18]

Thanks for your report.
I've fixed.


errors from background tasks are not printed

Given:
import responder
# import time <- notice missing import

api = responder.API()

@api.route("/")
def hello(req, resp):

    @api.background.task
    def sleep(s=1):
        time.sleep(s)
        print("slept!")

    sleep()
    resp.content = "processing"

if __name__ == '__main__':
    api.run()

When running sleep NameError is thrown but no output is shown in console.

we need to add logging.

The exception is happening in separate thread that is the reason it is not printed.
@kennethreitz What do you think will be the best approach to handle exception in futures? It is possible to have a custom ThreadPoolExecutor as it is proposed here.

@condemil a custom ThreadPoolExecutor would work for handling exceptions, but the value proposition of needing to take action beyond just logging the error doesn't exist IMO.
Having general logging that the bg tasks could tap in to would be more generically valuable.

@kennethreitz What about passing callback functions to @api.background.task to handle success and error case. I'm implementing this feature, and the following is demo api:
def on_success(result):
     print(result)

def on_error(error):
    print(error)

@api.background.task(on_success, on_error)
def sleep(s=10):
     // Do background tasks here

I feel if you want something more sussinct, you should just use async/await


Tracing: Update Config button is always visible

Kubewarden-UI: head
To reproduce:

install opentelemetry
install jaeger
enable tracing on kubewarden controller

Tracing page does not detect that tracing is enabled, it always shows Update Config button.
Screencast from 2023-10-29 18-27-18.webm

Closing, I realized this was test of PR that's not merged yet - moving info there.


fmt: confusion about Scanf patterns, newline, space

While updating code using fmt.Scanf after CL 16165, I discovered that there were users with patterns ending in \n as a way to assert that Scanf had consumed an entire input.
The examples that follow are given as three-line stanzas printed by the program at the end of the report. The first line is the Scanf call, the second line is its return values, and the third line is the values of x and y after the call.
The case that CL 16165 changed intentionally is:
# Go 1.5
fmt.Sscanf("1,2", "%d,%d\n", &x, &y)
2 <nil>
1 2

which became:
fmt.Sscanf("1,2", "%d,%d\n", &x, &y)
2 input does not match format
1 2

But there exists code using a \n at the end of the pattern to reject matching a partial input token:
# code doesn't want this match
fmt.Sscanf("1,2,3", "%d,%d", &x, &y)
2 <nil>
1 2

# so it uses this instead
fmt.Sscanf("1,2,3", "%d,%d\n", &x, &y)
2 expected space in input to match format
1 2

The \n here is serving to reject inputs that would be accepted without it, because \n matched end-of-string in Go 1.5 and earlier. Unfortunately, in Go 1.6 the \n will reject end-of-string, so code like this needs to be adjusted. How?
It seems that changing \n to space works: space matches end-of-string still.
# space accepts end-of-string
fmt.Sscanf("1,2", "%d,%d ", &x, &y)
2 <nil>
1 2

# space still rejects non-space additional text
fmt.Sscanf("1,2,3", "%d,%d ", &x, &y)
2 expected space in input to match format
1 2

Of course, using space in this way still accepts spaces, which \n did not. So if \n was being used this way:
# Go 1.5
fmt.Sscanf("1 2", "%d %d\n", &x, &y)
2 <nil>
1 2
fmt.Sscanf("1 2 3", "%d %d\n", &x, &y)
2 newline in format does not match input
1 2

now the same code produces:
fmt.Sscanf("1 2", "%d %d\n", &x, &y)
2 input does not match format
1 2
fmt.Sscanf("1 2 3", "%d %d\n", &x, &y)
2 newline in format does not match input
1 2

and changing \n to space in the pattern does not help reject the second input:
fmt.Sscanf("1 2", "%d %d ", &x, &y)
2 <nil>
1 2
fmt.Sscanf("1 2 3", "%d %d ", &x, &y)
2 <nil>
1 2

The reason this seems to come up is that people read a line at a time from some source and then process it with fmt.Scanf with a pattern containing \n, perhaps not realizing that the \n has been stripped off by the line reader, or perhaps just taking advantage of the fact that \n served like the regexp $ in such processing.
I am not sure how people should update their code. I can't find any promise in the docs that a space in the pattern matches end-of-string, and it seems odd to me that it does. But if we don't do that, then there's no way to update some common uses. And even if we do make the promise, it doesn't help reject unmatched text beginning with space, as in the last few examples. (To be fair, \n doesn't help reject unmatched text beginning with \n, but that doesn't come up in line-at-a-time processing, as seems to be common.)

Should we roll back CL 16165 and simply document that a single \n at end of pattern is allowed to match end-of-string.
Or should we document that space at end of pattern matches end-of-string?
Or should we make space at end of pattern not match end-of-string?

If we do (3), then there becomes no way to preserve the semantics of many existing uses, since Scanf gives no indication that it did not process the entire input string.

Program:
package main

import "fmt"

func main() {
	var x, y int

	x = 0
	y = 0
	fmt.Println(`fmt.Sscanf("1,2", "%d,%d ", &x, &y)`)
	fmt.Println(fmt.Sscanf("1,2", "%d,%d ", &x, &y))
	fmt.Println(x, y)

	x = 0
	y = 0
	fmt.Println(`fmt.Sscanf("1,2", "%d,%d\n", &x, &y)`)
	fmt.Println(fmt.Sscanf("1,2", "%d,%d\n", &x, &y))
	fmt.Println(x, y)

	x = 0
	y = 0
	fmt.Println(`fmt.Sscanf("1,2,3", "%d,%d", &x, &y)`)
	fmt.Println(fmt.Sscanf("1,2,3", "%d,%d", &x, &y))
	fmt.Println(x, y)

	x = 0
	y = 0
	fmt.Println(`fmt.Sscanf("1,2,3", "%d,%d\n", &x, &y)`)
	fmt.Println(fmt.Sscanf("1,2,3", "%d,%d\n", &x, &y))
	fmt.Println(x, y)

	x = 0
	y = 0
	fmt.Println(`fmt.Sscanf("1,2,3", "%d,%d ", &x, &y)`)
	fmt.Println(fmt.Sscanf("1,2,3", "%d,%d ", &x, &y))
	fmt.Println(x, y)

	x = 0
	y = 0
	fmt.Println(`fmt.Sscanf("1 2", "%d %d\n", &x, &y)`)
	fmt.Println(fmt.Sscanf("1 2", "%d %d\n", &x, &y))
	fmt.Println(x, y)

	x = 0
	y = 0
	fmt.Println(`fmt.Sscanf("1 2 3", "%d %d\n", &x, &y)`)
	fmt.Println(fmt.Sscanf("1 2 3", "%d %d\n", &x, &y))
	fmt.Println(x, y)

	x = 0
	y = 0
	fmt.Println(`fmt.Sscanf("1 2", "%d %d ", &x, &y)`)
	fmt.Println(fmt.Sscanf("1 2", "%d %d ", &x, &y))
	fmt.Println(x, y)

	x = 0
	y = 0
	fmt.Println(`fmt.Sscanf("1 2 3", "%d %d ", &x, &y)`)
	fmt.Println(fmt.Sscanf("1 2 3", "%d %d ", &x, &y))
	fmt.Println(x, y)
}

"Space in pattern matches end of string" is already documented, but not well enough:
%s and %v on strings scan a space-delimited token

Plus:
func Scanf(format string, a ...interface{}) (n int, err error)
Scanf scans text read from standard input, storing successive
space-separated values into successive arguments as determined by the
format. ...
func Scan(a ...interface{}) (n int, err error)
Scan scans text read from standard input, storing successive space-separated
values into successive arguments. ...
If you think more is needed, I think it can be made explicit in the package comment.

The text you mention all seems to me to be about the way %s and %v handle spaces in input, which is fine. It does not seem to say anything about the meaning of a space in the format pattern. What I do see is:

When scanning with Scanf, Fscanf, and Sscanf, all non-empty runs of space characters (except newline) are equivalent to a single space in both the format and the input. With that proviso, text in the format string must match the input text; scanning stops if it does not, with the return value of the function indicating the number of arguments scanned.

But that would suggest that " " in the format pattern should not match "" in the input. Yet it does.
P.S. The Scanf doc comment seems like it should drop "space-separated": scanning "%d,%d" against "1,2" doesn't have anything to do with spaces.

Well, I don't know how to document the scanning part of this package (or make it work, but that's another problem). To be fair, no one else seems to know how to define it either. C's scanf is well known but not documented to any level of precision. Maybe I should have taken that route and just hinted at what it does.
Suggestions welcome.

I'm happy to figure out docs if we can answer these questions:

Should Sscanf("123", "%d\n") succeed?
Should Sscanf("123", "%d ") succeed?
If the answer to 1 is no but the answer to 2 is yes, why are they different?
If the answer to 1 is no and the answer to 2 is no, how do you write a pattern scanning a single integer that succeeds on input "123" but fails on input "123 456"?

Based entirely on my mental model:

No, because there is a newline in the format and not in the input. (true in 1.6, not in 1.5)
Yes. (true in 1.6 and 1.5)
Because newlines in the format must match the input.
N/A.

Make sense?
I wish fmt.Scanf did not exist. Its overuse and its pain wildly outrun its value.

OK, I'll try to document that.

I sent CL 17723 with documentation of the current behavior, but I think I found a serious enough problem to warrant postponing any changes to Scanf until Go 1.7. Specifically, "X\n Y" does not match "X\n Y" (the same string). The fundamental issue (and a source of much confusion for me until I read the code) is that in the input format, any newline surrounded by zero or more space characters (that is, any section of input format matching / *\n */) corresponds to consuming / *\n/ from the input. Note the trailing space-star in the first regexp but not the second: " \n " in the format is the same as "\n"; both can consume " \n" from the input but not " \n ".
Less serious but still odd, "X %c" does not match "X \n" (reading '\n' into the %c argument) while "X%c" does match "X\n".
I think we should roll back the one newline-related change from the Go 1.6 cycle and revisit the whole big picture for Go 1.7. If we're going to break user code, we might as well do it just once.
I sent CL 17724 for the rollback of Go 1.6's newline change.

SGTM

Didn't happen for Go 1.7. Kicking to 1.8Early.


Riot looks to me like I'm sending the same message twice

Reported by several users:
"Riot iOS suddenly started displaying my messages as if I'd sent the same thing twice. Recipients said it looked like one message to them. I switched rooms and it didn't happen when sending a message to another room."


Can't see cursor when the text has gradient effect

Cursor is invisible when the text has gradient effect
Code to reproduce:
import { css } from '@panda/css'
import { TypeAnimation } from 'react-type-animation';

<TypeAnimation
  className={css({
    textStyle: 'bold.48',
    textGradient: 'linear-gradient(119deg, #0855ED 23.5%, #CD5890 69.1%)'',
  })}
  sequence={[
    'by A',
    1000,
    'by B',
    1000,
  ]}
  speed={50}
  repeat={Infinity}
/>

The cursor is styled based on the css color property. Your gradient styling may not work as expected since textGradient actually applies background-image.
But since any text has to have some default color, the cursor should at least be visible. If it's not visible at all, it's is probably being cut-off by the background-image or something.
You'd have to set cursor={false} and find a custom css solution for this (if there is any, haven't tried this yet).
Maybe something with z-index?
 .custom-cursor-animation::after {
      content: '|';
      position: relative;
      z-index: 5;
      animation: cursor 1.1s infinite step-start;
    }

      @keyframes cursor {
        50% {
          opacity: 0;
        }
      }

If you find a solution, you will have to apply the custom css class to the component as described here


Bug fix. POSITION_NONE is a valid prop value for the position attribute in toolbarProps

v3 Bug fix. POSITION_NONE is a valid prop value for the position attribute in toolbarProps.

Thanks for your help. I'm going to add you in contributors list.


files in .git/info/exclude ignored whatever "Exclude VCS Ignored path"

Prerequisites

[x]

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Problem about the .git/info/exclude file. Files and repertory specified in it are ignored by all packages, even if the "Exclude VCS Ignored path" option is uncheck.
Steps to Reproduce

Clone a project from a git repo, this project must have a exclude file in .git/info
[I can give you one if needed]
Unvalid the "Exclude VCS Ignored path" option in core

Expected behavior:
Packages such as "fuzzy-finder" or "search in project" should not exclude the files excluded by git specified in .git/info/exclude
Also these files and repertories should not be grayed in the tree-view.
Actual behavior:
All files and repertories specified in .git/info/exclude are ignored by these packages.
Also these files and repertory are grayed in the tree-view.
Reproduces how often:
100%
Versions
Atom    : 1.23.1
Electron: 1.6.15
Chrome  : 56.0.2924.87
Node    : 7.4.0
apm  1.18.11
npm  3.10.10
node 6.9.5 x64
python 2.7.9
git 2.1.4

Thanks for reaching out - do you have Settings > Packages > tree-view > Hide VCS Ignored Files unchecked?

Thanks for reply!
Yes the Settings > Packages > tree-view > Hide VCS Ignored is unchecked.
And I precise that file aren't hide in the tree-view, but only grayed.


Japan Gives an Error

After reading https://github.com/tesseract-ocr/tesseract/issues/1441 I've downloaded the suggested file.  However when attempting to run the following
tesseract my_file.jpg out -l jpn
I'm given the following error
Tesseract Open Source OCR Engine vv4.0.0-beta.1.20180608 with Leptonica
Error during processing.

Please share the input image you gave.
Which traineddata file are you using? Where is it installed/located?

Hello everyone,
I have the same problem

I have the same problems too

"I have same problem" is NOT HELPFUL in solving it.
Please mention:
Version of tesseract
Which Operating system
Which version of jpn.traineddata
What command was used
Sample image which gave error

Closed because of missing input from reporter


downgrade thrift to backend compatible version

What changed?
I downgraded apache/thrfit version to the version compatible on the Cadence backend.

Why?
Cadence backend uses client to run internal operations, so client should be compatible with the backend.

How did you test it?
unit tests/integration tests

Potential risks
Potential failures for customers using up to date thrift version.

hmm in the monorepo the version is set to 0.17.0 and it client works.
Let me try 0.17.0 then, might be it will be fine.


repeaterVal does not work.

$('.repeater').repeaterVal()  does not work, it not exist as a function in jquery.repeater.js file.
TypeError: $(...).repeaterVal is not a function

When I try the $('.repeater-default').repeaterVal(); on http://briandetering.net/repeater I get the same result.

Repeater Val was added relatively recently, (the version on http://briandetering.net/repeater is not up to date with this functionality).
Is it possible you are running an older version of jquery repeater that does not support repeaterVal?

Hi DubFriend.
Thanks for response
Im using :
// jquery.repeater version 0.1.4
// https://github.com/DubFriend/jquery.repeater
// (MIT) 13-08-2014
// Brian Detering<EMAIL_ADDRESS>(http://www.briandetering.net/)
downloaded from: https://plugins.jquery.com/repeater/
where can i find the lastest version ?
Thanks a lot :)

Ahhh...!
Im sorry i newbie here on git :p
i found the 1.1.3 version on code tab.
Thanks for all.

Last time I checked the jquery plugin registry was in read only mode, and I had been unable to publish new versions to the jquery page, so unfortunately the jquery registry is out of date :worried:
Looks like you got things figured out tho :smiley:


disk and path is missing the "" separator for network mounted drives on Win7

Description
Atom path returned by:
atom.workspace.getActiveTextEditor().getPath()
is missing separator \ between disk and directory.
Steps to Reproduce

Export some disk by Samba etc.
Mount the network drive to windows.
Open the file from mounted drive by Atom.
Open Developer tools console and run:
atom.workspace.getActiveTextEditor().getPath()

Expected behavior: [What you expect to happen]
The valid file path should be returned:
"N:\frontend\Templates\TemplatesList.jsx"
Actual behavior: [What actually happens]
The actual path returned is missing \ separator between DiskDrive and Directory:
"N:frontend\Templates\TemplatesList.jsx"
Versions
Windows 7 x64
Atom version: 1.20.0-beta7
Additional Information
Originally the problem was experienced for linter-eslint and reported here:
https://github.com/AtomLinter/linter-eslint/issues/1011
The same is not observed for local drives.
I'm observing some other issues like unable to rename/delete files.

Hmm I did some troubleshooting and it seems to be related to restoring of project folder after Atom restart. Let me note that I wiped whole Atom, downloaded the latest 1.21.0-beta x64(atom-x64-windows.zip). I used vanila install as it was unpacked so no plugins, no customization.

Add new project folder CTRL+SHIFT+A => added  my N:\ network drive.

Open some JSX file

Open Developer Tools CTRL+SHIFT+I

Run atom.workspace.getActiveTextEditor().getPath() = > Got VALID PATH

Restart Atom

Project folder and files are restored - however the tree view is missing colorization of some files (seems to be like git differences are signed this way, not sure)

Developer Tool - CTRL+SHIFT+I  => atom.workspace.getActiveTextEditor().getPath() => PATH is VALID

Close the file Restored

Open the same file from TreeView

Developer Tool - CTRL+SHIFT+I => atom.workspace.getActiveTextEditor().getPath() => INVALID PATH

Remove Project folder, remove all opened files and start with step 1. All reoccurred same way.

So it seems that TreeView somehow broke the path once it is restored after Atom restart. All files opened after the TreeView restore are getting corrupted path.

This is partly due to Node's decision of treating disk drives without a trailing path separator as referring to the cwd on that disk drive, rather than the root of the disk drive (which is technically correct, but not what is needed in practice 99% of the time).

hmm but why the behavior is different for adding new project folder and auto-restoring session after startup? I would expect that Node is consistent. I mean the CWD provides the same disk/path for both cases.
Can I provide more details or help with troubleshooting somehow? Just let me know what is needed.
Thanks

New observation on latest 1.22.0-beta1.
Atom is unable to recover the Project Folder after start. So I'm adding the project folder manually after any Atom start/restart. The difference is after the project folder is added. Once it is added, all works fine.

I moved to VScode...

I moved to VScode...

I can reproduce on windows 10 with ubuntu network drive.


[Breaking] Change weights generation to follow WeightInitializers.jl

In order to streamline the package and creation of reservoir computing models the creation of the weights of the matrices should follow WeightInitializers.jl

Hi @MartinuzziFrancesco  I guess I can take a look at this, if you can elaborate a bit what exactly needs to be done here.

Hi, of course! So at the moment the construction of the input layers matrices and reservoir matrices uses abstract typing and dispatch over different structs of the different architectures for the matrices and it is a bit cumbersome. I wanted to simplify it a bit and make it follow the standard in WeightInitializers.jl. This way we would also start an easier way to control the randomness in ReservoirComputing.jl. For example SparseRandReservoir would look like this:
using ReservoirComputing, Random
rng = MersenneTwister(42)

# Passing kwargs with explicit rng call
res_cl = rand_sparse(rng; sparsity=0.1, spectral_radius=1.0)
#res_cl is now a callable function that takes in the size of the reservoir
reservoir_weights = res_cl(rng, 10, 10)

# Passing kwargs with default rng call
res_cl = rand_sparse(; sparsity=0.1, spectral_radius=1.0)
#res_cl is now a callable function that takes in the size of the reservoir
reservoir_weights = res_cl(10, 10)

For details of the implementation you can refer to the source in WeightInitializers. I could sketch rand_sparse during this week if that would be of help. This needs to be done for both input layers and reservoirs.
Of course after this we need to modify the ESN constructor but that should be rather straightforward.
This issue is part of a streamlining in order to be more in line with (F)Lux as model building philosophy. This should also make contributions a little bit easier

@Jay-sanjay I have started a PR in #193, you can follow the template to create the remaining functions for the reservoir and input layers. I will need to think about a couple of details (and tests as well)

Thanks @MartinuzziFrancesco  for the template, I will start to work on it with other functions

Addressed in #196 and finalized in #203


Source map is off by one line

I apologize in advance for this horrible issue report. I noticed this a few weeks ago, forgot to report it and since then removed my configuration. At the moment I have no time to create an example.
I just thought I would let you guys know I ran in to this. If you can easily reproduce it: good. If not you can close this issue.

Hey @EECOLOR! We really appreciate you taking the time to report an issue. The collaborators
on this project attempt to help as many people as possible, but we're a limited number of volunteers,
so it's possible this won't be addressed swiftly.
If you need any help, or just have general Babel or JavaScript questions, we have a vibrant Slack
community that typically always has someone willing to help. You can sign-up here
for an invite.

I have the same problem, if you want to give it a shot you can try my repository


Reduce all Dockerfile COPY's to a single layer

This excludes the php images, which are done in another PR

What the value of that?

I don't think layers are as bad as they used to, but less layers in COPY would have some performance boost somewhere

I definitely love the other PR but I'm afraid this one is too much of an optimisation that reduces a lot the clarity of these dockerfiles. I personally vote for keeping them as they are, with two COPY, just for the sake of better user experience.

I'm happy to drop this if that's an issue, but what clarity is lost?

From a user perspective, this is more or less obvious:
COPY ./etc/ /etc
COPY ./usr/ /usr

This one is much less:
COPY ./shared/ /

And these Dockerfiles are also a source of inspiration for the people that are using them, therefore I suggest we keep it very simple.

the latter style is already being used in some images with nginx/apache varients

but I disagree that / is less clear than /folder


Fixed Unpacker exception handling: IZPACK-1252, IZPACK-1253, translated message boxes

This request fixes a couple of issues with the UnpackerBase:
See:

IZPACK-1252 - Crashes if custom listener's afterpack throws InstallerException without root cause exception
IZPACK-1253 - ClassCastException in Unpacker if non-IzPackException is given as root cause to InstallerException from listener

Additionally there have been added some error and warning dialog translations.
At the moment there is available the english, german and czech translation, the other languages have been left english... in case someone can help here.

IzPack project » izpack #817 SUCCESS
This pull request looks good
(what's this?)


ValidatingWebhookConfiguration not created on OpenShift 4.2

Step-by-step process followed:
https://github.com/marcredhat/crcdemos/blob/master/sysdig/admissioncontroller/README.adoc
Expected result: ValidatingWebhookConfiguration created
Actual results:
oc logs image-scan-k8s-webhook-controller-manager-0 --follow
{"level":"info","ts":1571949974.6266456,"logger":"entrypoint","msg":"setting up client for manager"}
{"level":"info","ts":1571949974.6283126,"logger":"entrypoint","msg":"setting up manager"}
{"level":"info","ts":1571949975.0136602,"logger":"entrypoint","msg":"Registering Components."}
{"level":"info","ts":1571949975.0137165,"logger":"entrypoint","msg":"setting up scheme"}
{"level":"info","ts":1571949975.0137308,"logger":"entrypoint","msg":"Setting up controller"}
{"level":"info","ts":1571949975.0137427,"logger":"entrypoint","msg":"setting up webhooks"}
{"level":"info","ts":1571949975.0138788,"logger":"entrypoint","msg":"Starting the Cmd."}
{"level":"info","ts":1571949975.11553,"logger":"kubebuilder.webhook","msg":"installing webhook configuration in cluster"}
However, the pod enters a crash loop and the ValidatingWebhookConfiguration is not created.
Note: Other ValidatingWebhookConfiguration are working fine in the same environment
oc get  ValidatingWebhookConfiguration --all-namespaces
NAME                       CREATED AT
autoscaling.openshift.io   2019-10-16T10:27:28Z
multus.openshift.io        2019-10-16T10:14:17Z
For a lab environment, you can fix this as follows but we need a proper solution:
oc adm policy add-scc-to-user anyuid -z default
securitycontextconstraints.security.openshift.io/anyuid added to: ["system:serviceaccount:image-scan-k8s-webhook-system:default"]

@marcredhat thanks for bring it up, I noticed it too. I will see what I can do here.

The issue has been fix in the latest commit