Make possibillity to configure what for values will be readed -> speed up runtime

Hi, it is possible to get an option / file to set valus to read? to read all values cost many time ~ 22 secounds for my logger. I have sorted out valus by hand. Now runtime is 9 secounds. But my changes are not update resistant... :-)

With the methods introduced with 75af57f you can use a random selection of registers. I will have to update the deye-read and add some examples

Think it is OK now. See in the examples directory how these methods can be used with arbitrary selection of registers


yetus and golint

Signed-off-by: Petr Fedchenkov<EMAIL_ADDRESS>

Just as an FYI, Yetus uses https://github.com/mgechev/revive to implement golint features.


update storyboard for session timeout

A browser session times out after some time of inactivity. How should the application react to that?

should the user be allowed to renew the session without logging in again?
should the application remain in the state it was in before or show some popup or redirect to a different page?

Issue related to old, deprecated prototype.


Specify pytest collection directory explicitly

I usually have my virtualenvs in the project directory. The default test commands will collect tests from the virtualenv directory. Specifying the tests directory explicitly prevents that.

Yeah, and the coverage tool has problems with this too. I'm fine with the change in the docs, but the one in the toxfile is problematic as one can't specify custom args anymore.

I'll look into it further and see if I can find a solution, or remove the bad changes if I can't.

Closing this because master already has norecursedirs = env* defined in setup.cfg.


PP-7216 update country list

WHAT
Updates country list based on diff against CSV from https://data.gov.uk/dataset/0b412b57-6934-4f28-b495-640cdc7e8f7f/foreign-commonwealth-and-development-office-geographical-names-index (I wrote a little python script to compare the CSV against our current JSON.)
Question should we add the old names as synonyms for the new ones in the autocomplete graph?
HOW
Updated country names should appear in country picker on payment page e.g. North Macedonia.

Question should we add the old names as synonyms for the new ones in the autocomplete graph?
Yes, I think that’s a good idea. People may have become used to typing the old names so it would be surprising if they suddenly stopped working.

@alexbishop1 OK I've added a synonym for Swaziland now (the others contain their new name within the old one so will continue to work if people type the old value)

Do we also want synonyms for Burma and Macedonia? It seems a little delicate to rely on the fact that they’re substrings of the new names.

Do we also want synonyms for Burma and Macedonia?

OK I added these now and forced pushed in 1e13ff4


Use JSON-RPC over HTTP for communication between plugins

While looking into adding request/response logging in #269, i realized that plugin communications were using raw socket I/O rather than HTTP.  This patch uses JSON-RPC over HTTP, which i suspect will be easier for others to implement and test with conventional tools.  Specifically, this uses github.com/gorilla/rpc.  I chose this over net/rpc and net/rpc/jsonrpc to favor traditional HTTP request-oriented behavior.  net/rpc can operate over HTTP, but only as a starting point after which it hijacks connections and no longer uses HTTP requests (similar to websocket behavior).
I've elected to use gopkg.in/tylerb/graceful.v1 for handling graceful shutdown of the HTTP server, which simplified lifecycle management.
In addition, this patch adds HTTP request logging at the debug log level.  I will follow up to refine the log output, as it currently does not log the request body.
The patch comes in 2 commits - one with the code changes, and one with vendor updates in the course of adding dependencies.
Signed-off-by: Bill Farner<EMAIL_ADDRESS>

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "gorilla-rpc-with-logging" git@github.com:wfarner/infrakit.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842353945600
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Current coverage is 70.84% (diff: 84.25%)

Merging #297 into master will increase coverage by 1.39%

@@             master       #297   diff @@
==========================================
  Files            25         25
  Lines          1303       1228    -75
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits            905        870    -35
+ Misses          321        285    -36
+ Partials         77         73     -4

Powered by Codecov. Last update e3de881...078728e


Fix Github Actions of Tauri Plugin with-api template

What kind of change does this PR introduce?

[X] Bugfix
[ ] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[X] No

Checklist

[ ] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[ ] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[X] I have added a convincing reason for adding this feature, if necessary

Other information
check https://discord.com/channels/616186924390023171/1090989894991618171 for more info I guess

Can you add a change file like this one? https://raw.githubusercontent.com/tauri-apps/tauri/dev/.changes/cli-sign-non-interactive.md

Also might as well update actions/checkout to v3 while we're at it.

Also might as well update actions/checkout to v3 while we're at it.

I will do that as well tomorrow for all templates

Can you add a change file like this one? https://raw.githubusercontent.com/tauri-apps/tauri/dev/.changes/cli-sign-non-interactive.md

Never used that type of stuff before.
Btw it said "A change file is added if any packages will require a version bump due to this PR", but there are no packages to be bumped!

The cli will be bumped for this (patch). Here's an example change file for cli changes: https://github.com/tauri-apps/tauri/blob/dev/.changes/cli-info.md

Oke, I will look at it tomorrow and then I will also have my commits signed.

The cli will be bumped for this (patch). Here's an example change file for cli changes: https://github.com/tauri-apps/tauri/blob/dev/.changes/cli-info.md

What files should I mention in it and how?

Let me know if anything else should be changed as well or else just merge it if possible

Maybe I should add a permissions block to the audit workflow(s) so the users don't have to manually change stuff for the audit token to be able to do what it is supposed to do.


Update Words_fr.yml

Update TranslatableWords Words_fr.yml

No problem because I have not finished correct everything and anyway  it isn't urgent


Random Native iOs error : "Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'Could not locate shadow view with tag #8'"

Hi there,
I have regularly a really weird and random error in Xcode when running on the simulator, on iOs only :
I had this bug since RN v0.21 and still have it on RN v0.26.
It's really hard to reproduce, but I think it has to do with the asyncStorage somehow because commenting my code (it seems especially AsyncStorage.multiGet) interacting with the asyncStorage, relaunching the app and uncommenting then reloading the JS package makes the bug go away...
The stack trace in Xcode is :
2016-06-07 10:07:56.514 RNOSDStarter[33977:6342335] Communications error: <OS_xpc_error: <error: 0x106ff7b40> { count = 1, contents =
    "XPCErrorDescription" => <string: 0x106ff7ef0> { length = 22, contents = "Connection interrupted" }
}>
2016-06-07 10:11:39.995 MyProject[33977:6335407] *** Assertion failure in __33-[RCTUIManager setFrame:forView:]_block_invoke(), /Users/adrien/Projects/myProject/node_modules/react-native/React/Modules/RCTUIManager.m:398
2016-06-07 10:11:40.037 MyProject[33977:6335407] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'Could not locate shadow view with tag #8'
*** First throw call stack:
(
    0   CoreFoundation                      0x000000010382ad85 __exceptionPreprocess + 165
    1   libobjc.A.dylib                     0x00000001029f6deb objc_exception_throw + 48
    2   CoreFoundation                      0x000000010382abea +[NSException raise:format:arguments:] + 106
    3   Foundation                          0x0000000102640e1e -[NSAssertionHandler handleFailureInFunction:file:lineNumber:description:] + 169
    4   RNOSDStarter                        0x000000010226b342 __33-[RCTUIManager setFrame:forView:]_block_invoke + 386
    5   libdispatch.dylib                   0x0000000106cb7d9d _dispatch_call_block_and_release + 12
    6   libdispatch.dylib                   0x0000000106cd83eb _dispatch_client_callout + 8
    7   libdispatch.dylib                   0x0000000106cbe82c _dispatch_queue_drain + 2215
    8   libdispatch.dylib                   0x0000000106cbdd4d _dispatch_queue_invoke + 601
    9   libdispatch.dylib                   0x0000000106cc0996 _dispatch_root_queue_drain + 1420
    10  libdispatch.dylib                   0x0000000106cc0405 _dispatch_worker_thread3 + 111
    11  libsystem_pthread.dylib             0x00000001070154de _pthread_wqthread + 1129
    12  libsystem_pthread.dylib             0x0000000107013341 start_wqthread + 13
)
libc++abi.dylib: terminating with uncaught exception of type NSException
(lldb)

Any help or lead on how to debug this would be awesome!
Other info :

Using Mac OSX El Capitan 10.11.5
We are using redux and are interacting with AsyncStorage in a middleware
AsyncStorage is called directly after reloading the package (first thing we do once the app is launched : Get data stored in asyncStorage).

@facebook-github-bot label iOS

@adrienthiery i had the same problem calling AsyncStorage.getItem in a componentDidMount method, and what was causing the problem had to do with setting the state of the component on the callback of AsyncStorage.getItem, i guess the scope was not set at that point in order to call this.setState({myProp: true}); . But just so you know this only happens on "debug" build type on the device, it will NOT happen on Simulator and it will NOT happen on "release" type build because for release assertions are canceled so kinda makes sense.
Here is a snapshot on how my code looked like when it failed, removing this.setState(..) solved it. I wasn't needing it anyway.
https://www.dropbox.com/s/46m8l4v6vzcqk7d/Screenshot 2016-06-10 22.29.58.png?dl=0

Mmmh, okay I will look if I have components directly hooked to the callback.
I indeed saw that it wasn't happening on production builds but it still is sometimes a pain in the butt while developing.
Thanks for the tip, I'll try to fix it and if I do, will update this thread :)

I'm experiencing the same problem. I'm keeping some session data in AsyncStorage, so on logout button press I'm destroying them, then I go to back launcher screen and check for their existence. In almost 100% of the cases it results in NSInternalInconsistencyException.
I'm using RN 0.24.1, will try later on newer RN, too.

After upgrading to RN 0.27.2 I'm still experiencing the issue, with 100% reproduction. I'll try to distill some sample app to showcase this.

I have the same issue, related to setState() as mentioned above.

In my cases, it seems that while fetching from AsyncStorage if view appear and disappear too quickly this error seems occurs. If I do below it works fine for me.
setTimeout(() => {
  store.dispatch(toggleLoading(false));
}, 500);
return;

Event I postpone dispatch using setTimeout, 0 magic this might occurs again.

@alma-socar  Why not using InteractionManager.runAfterInteractions(()> { ... })? why do you need to depend on an arbitrary time in order to avoid it?

@alextkd I just didn't know it. Thanks!

Since it's been a while and we don't have a clear repro, I'm going to close this issue. Please feel free to open a new one, ideally with sample code that makes this bug easy to reproduce.

If anybody still has this problem :
The solution I found was to stop using AsyncStorage altogether, I use RealmDB : It is faster, easier and way more powerful.

I, on the other hand, only had to correctly use AsyncStorage's methods promises.
This results in the above mentioned error:
// 1 - Showing a Dialog component
showModal();
// 2 - Incorrectly handling AsyncStorage and ignoring promise
AsyncStorage.setItem(APP_STORAGE_KEY, JSON.stringify(newState));
// 3 - Hiding the Dialog component
hideModal();

This works :-)
// 1 - Showing a Dialog component
showModal();
// 2 - Correctly handle AsyncStorage's promise
AsyncStorage.setItem(APP_STORAGE_KEY, JSON.stringify(newState))
  .then(() => hideModal())
  .catch(e => {
    hideModal();
    showAlert(e);
  });

For me this was just about removing the application before the next run.

For me this was about presenting and hiding a view too quickly

I'm having this issue.
It has nothing to do with the code. Don't change your code.
My fellas in the company doesn't have any problem with the same xcode, simulator and macos version. Sourcecode is also the very same.
I'd say it's a problem of Apple.

It seems you have replaced  ios Directory with different ios directory from Root of react-native project , so you need to new instance of your project  and configure ios set up .


docs(coagents): add reference documentation for useCoAgent and useCoAgentStateRender

What does this PR do?
This PR adds some CoAgent reference materials, specifically for the useCoAgent and useCoAgentStateRender hooks. It also adds some very minor style changes to improve how things look in darkmode, namely by changing around some colors.
Checklist

[X] I have read the Contribution Guide
[X] If the PR changes or adds functionality, I have updated the relevant documentation

Go ahead to merge from @ataibarkai


Publish charmhub

This PR adds publishing to charmhub actions.
Promote charm action is added as a complimentary action to promote edge channel charm into stable channel.
Note that the default base is sidecar branch, hence Publish to edge action works on only sidecar and track/** branches.

The reason why the default branch is sidecar branch is that we haven't considered the sidecar version of the charm to be ready to replacing the legacy version. I think we should swap rename the sidecar branch to main branch before merging this PR.

Update: this will be merge after Weii's feedback to swap the sidecar branch into main and migrate main into a legacy branch marked as legacy.


vkbasalt: <IP_ADDRESS> -> <IP_ADDRESS>

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/DadSchoorse/vkbasalt/releases.
meta.description for vkbasalt is: A Vulkan post processing layer for Linux
meta.homepage for vkbasalt is: https://github.com/DadSchoorse/vkBasalt
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS> with grep in /nix/store/c327dpra8mal3pwkwwqm48zvxpplxf9w-vkbasalt-<IP_ADDRESS>
found <IP_ADDRESS> in filename of file in /nix/store/c327dpra8mal3pwkwwqm48zvxpplxf9w-vkbasalt-<IP_ADDRESS>

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath
bottles
bottles-unwrapped
vkbasalt
vkbasalt-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/c327dpra8mal3pwkwwqm48zvxpplxf9w-vkbasalt-<IP_ADDRESS> \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A vkbasalt https://github.com/r-ryantm/nixpkgs/archive/b76c4902cac0c7d8f265fef03bc8563773b12e9b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/c327dpra8mal3pwkwwqm48zvxpplxf9w-vkbasalt-<IP_ADDRESS>
ls -la /nix/store/c327dpra8mal3pwkwwqm48zvxpplxf9w-vkbasalt-<IP_ADDRESS>/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

4 packages built:

bottles
bottles-unwrapped
vkbasalt
vkbasalt-cli

Maintainer pings
cc @kira-bruneau for testing.

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

4 packages built:

bottles
bottles-unwrapped
vkbasalt
vkbasalt-cli


Revise circ_diff function

This small pull request makes a small change to the circ_diff function, feel free @aclerc to reject if you prefer as is now, just wanted to offer.  Main points:

uses approach similar to the wrap_180 method in FLORIS
Explicitly accepts type list (implied by the tests) (this change could also be made to type hint of original)
No division (avoid risk of division by 0)
I believe it's simpler and should be faster without the trigonometry but yields all passing tests

Thank you, Paul! The new circ_diff function is a nice improvement. It turns out the previous circ_diff function could be wrong by up to about 1e-12.

Very glad to hear it!


Alternate optimization

Bechmark is provided in the spreadsheet.

It seems I was a bit late to the party, @lstein. =P Hadn't seen you had closed #364
Either way, I've finally finished, taking special care with Mac users as well, as per #431. I still have a little hope to have it tested, even if for another release.
To be clear on the changes: it is supposed to run fast with small images across devices (hard to really test without other's devices). It is also supposed to run faster than previous (Doggettx's) on Mac devices for large images.

To calculate free memory for Mac, you'll need something similar to:
import psutil

if device_type == 'mps':
            mem_free_total = psutil.virtual_memory().available
else:
            stats = torch.cuda.memory_stats(q.device)
            mem_active = stats['active_bytes.all.current']
            mem_reserved = stats['reserved_bytes.all.current']
            mem_free_cuda, _ = torch.cuda.mem_get_info(torch.cuda.current_device())
            mem_free_torch = mem_reserved - mem_active
            mem_free_total = mem_free_cuda + mem_free_torch

As stats = torch.cuda.memory_stats(q.device) is going to fail on M1

Thank you, @Any-Winter-4079, it's updated! If you're feeling generous, could you test main branch with this attention.py and see if the noisy images and the speed issues persist?


Issues I ran into getting started with Cog

I have a few issues I wanted to highlight when getting started with Cog:
1. the Predictor interface
I found it initially confusing how to input an audio file into my Predictor. In retrospect, it makes sense that I just need to take a file as input, but I didn't see anything in the docs to instruct me about this. I ended up figuring it out by looking at the tutorial that does audio tagging.
A similar confusion: what does Predictor.predict return? Can it be, like...anything? (Sorry I know I could probably read this in the docs but I guess I wish it was clearer from the API or default example.)
2. slow workflow
Every time I ran cog predict I would have to wait for cog to start up, install everything, then download the model I needed from a website, and then it would fail with an error. This was a frustratingly slow workflow! I fixed it eventually when @andreasjansson told me that every file in my local folder would be bundled into the docker image. So I downloaded the model and changed the code to load from the local folder, and that sped up cog (although it's still slower to start up than I'd like, but maybe an interface for running predictors locally would fix that)
3. malformatted cog.yaml
I was using the default cog.yaml (I think it was generated by cog.init?) and it had a line that was kinda like this:
system_packages:
# - put system packages here
and I think I tried running cog predict without changing that line, and got an error that said something nondescript:
ⅹ There is a problem in your cog.yaml file.
ⅹ must be a list.

It was annoying but I eventually figured out it was referring to that system_packages key
so two things could be fixed there

error message could look like value for key 'system_packages' in cog.yaml must be a list
the default cog.yaml should probably not be malformed

update from later
I actually ran into a similar issue with a yaml file like this:
  system_packages:
    - "libsndfile1-dev"
    - "ffmpeg"
- "timidity"

and got the following error:
ⅹ Failed to parse config yaml: yaml: line 11: found character that cannot start any token
this one was easier to fix because it had a line number, but also doesn't seem to be a meaningful (or correct?) error message. I was just missing the indentation.
4. returning video from the predictor
It wasn't clear to me how to return a video. I actually tried rendering the video to a file and returning the string 'out.mp4' (the name of the file) but that just propagated to the frontend as a string...
5. uploading clarity
It was unclear when uploading  with cog push actually finished, or what stage it was in. It was printing a lot of weird stuff:
Using default tag: latest
31b608d88350: Pushed
074e37886942: Layer already exists
71a0887c56fc: Layer already exists
d54ed2edbac4: Layer already exists
3058d5dc6d22: Layer already exists
ca2cbf283f0e: Layer already exists
606fc042d492: Layer already exists
eff64713ff03: Layer already exists
5bdaf1f57aee: Layer already exists
19347dc82c13: Layer already exists
aa091f54d0c4: Layer already exists
feacfd9ce92d: Layer already exists
501309e49952: Layer already exists
0c5746584249: Layer already exists
1a1a19626b20: Layer already exists
5b7dc8292d9b: Layer already exists
bbc674332e2e: Layer already exists
da2785b7bb16: Layer already exists

I was worried at one point that it wasn't going to work (because I hadn't updated the version number, I was thinking) and at another point I thought the thing was totally stalled out. But then the command finally finished and I determined that it had actually worked.

This is so helpful, thanks @jxmorris12. 🙌

Just added a couple things to the bottom @bfirsh!


request fail

requests.exceptions.ConnectionError: HTTPSConnectionPool(host='gcm-http.googleapis.com', port=443): Max retries exceeded with url: /gcm/send (Caused by NewConnectionError('<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x10273a050>: Failed to establish a new connection: [Errno 65] No route to host',))
how to solve it.

I can replicate this error when I try to send_topic_message in a request/response cycle (django).
But this goes away when I invoke send_topic_message directly from python shell.

@631068264 The 'no route to host' problem is outside the bounds of the gcm.py program. Check your network configuration on the machine your running the script.
You can use the host, nslookup, dig, ping, tracepath commands to help with the troubleshooting:

http://www.tecmint.com/linux-network-configuration-and-troubleshooting-commands/
http://www.cyberciti.biz/tips/no-route-to-host-error-and-solution.html


compatibility with command-o-matic

issue lies with filling blocks below y-coordinate 0 or above y-coordinate 255. throws error of "cannot place blocks outside of the world", when i am able to place single blocks of any kind above or below that threshold. fill command becomes useless "below" or "above" the original build ceiling or floor.

The issue is in Command-O-Matic rather than in Cubic Chunks, but I submitted a PR to Command-O-Matic that should fix it anyways. (If they don't merge the PR and upload a new build to CurseForge, you should still be able to build the fork I did the PR with for yourself.)

https://github.com/vova7865/command-o-matic/pull/1 closes once PR is accepted

thank you for the quick reply and the hard work! i think this may solve the issue, but my java compiler (javac) gets errors (70 of them) when i try to recompile the java files back into its classes. If you could compile the javas or point me toward a resource that works to compile these minecraft java files, that would be very much appreciated.

Ok, this really has nothing to do here - since the others already pointed out that this was an Issue in command o matic, but I'll just answer for the sake of helping:
Set up a forge MDK environment, put the files of the whole repo of command o matic in the folders of the same names, and use the build system of forge.
Please read some tutorials on forge modding on how to set it up exactly.
Alternatively you could just wait for an update of command o matic since Kandi's changes got merged.

@sazuko07 merged, updated the file on curseforge. thanks @KandiPanda !


Request: How to create an App ID with PowerShell

If attempting to register an app within Azure Portal, the DOD receives this message

However, I've been unable to find information on generating a App Registration using PowerShell.  Can you please add this to PowerShell information (Or point to where it is already documented) and perhaps provide examples. Thank you.

@lordneeko below is a pretty good write up on our public docs about Application objects and Service Principal objects in Azure Active Directory and has documentation on how to do this via PowerShell.
Hope this helps.
https://docs.microsoft.com/en-us/azure/active-directory/develop/app-objects-and-service-principals#service-principal-object


Add documentation and example for dask.array.histogram

After reading http://stackoverflow.com/questions/38086741/how-to-draw-a-histogram-in-dask I realized we are missing good documentation for dask.array.histogram.
http://dask.pydata.org/en/latest/array-api.html#dask.array.histogram:

dask.array.histogram(a, bins=None, range=None, normed=False, weights=None, density=None)

If bins and range are required we should specify so.
The example can be taken from @mrocklin answer or pick a simpler/less random one.

+1


Fix: Include freesurfer mris_topo_fixer command

Info: this needs to be included as recon-all falls back to this binary in some cases.


Create helper methods on bitflag generated structs

Addresses #279
When checking the value of a single flag, rather than typing out:
let s = Status::CURRENT;
s.intersect(Status::CURRENT);
// or
s & Status::CURRENT == STATUS::CURRENT

Add helper methods to simplify the above to:
let s = Status::CURRENT;
s.is_current();

Also, add a Default impl for any bitflag struct which defines a DEFAULT constant which simply returns the DEFAULT constant, allowing for more idiomatic code.  I think it would make sense to do the same for bitflag structs which define a NONE constant with no DEFAULT.
I only checked the top level lib.rs file, if there are more bitflags I'll happily track them down this week.

Nice!


The development server returned response error code: 500

I changes my tailwind version in the package.json but still again this error come how do fix this

Did you run npm install after you changed the version?

Please re-read the error message and then use the search function. There is a pinned issue https://github.com/marklawlor/nativewind/issues/498


browser.debug() times out

In theory you should be able to call brower.debug() to manually inspect the browser state continue when you are ready by pressing enter at the console.
http://webdriver.io/api/utility/debug.html
I get that async tests shouldn't wait forever, but it would be nice if there was a way to disable this automatic timeout when debugging.

Thinking about this issue.. When running tests via WDIO test runner, the jasmine timeout is controlled via by the wdio.conf.js file in
jasmineNodeOpts: {
        // Jasmine default timeout
        defaultTimeoutInterval: timeoutHere
}

I don't know if there's an easy way for this timeout to get ignored when calling browser.debug()... Instead, i'd suggest just changing the default timeout to something like 60000 * 60  when debugging..
As a best practice, you may want to set up a separate debug.wdio.conf.js with custom options like this that you ultimately will not want as your standard wdio configuration.

@patthiel thanks for your comment. @kurtharriger please increase the default timeout as @patthiel suggested


chore(master): release 1.1.56

:robot: I have created a release beep boop
1.1.56 (2023-08-30)
Bug Fixes

deps: update dependency wayofdev/laravel-webhook-client to v1.1.52 (e5da686)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-stripe-webhooks/releases/tag/v1.1.56 :sunflower:


Tabs within TabBarScroller are initialized multiple times

I found an issue with Tabs within a Tabs.TabBarScroller. Here is a simple example to reproduce the issue (this can also be reproduced on the Tabs Scroller example) :
<Tabs.TabBarScroller>
	<Tabs scroller>
		<Tabs.Tab>1</Tabs.Tab>
		<Tabs.Tab>2</Tabs.Tab>
		<Tabs.Tab>3</Tabs.Tab>
	</Tabs>
</Tabs.TabBarScroller>

Details
When Tabs are used within a Tabs.TabBarScroller, this causes MDCTabBar to be constructed multiple times :

Once in the MDCTabBarScroller.initialize method
Once in the Tabs.componentDidMount method

This also causes MDCTab to be constructed multiple times as they are initialized in the MDCTabBar.initialize method.
As MDCTab are created multiple times, the following error occurs when switching tab :
Invalid tab component given as activeTab: Tab not found within this component's tab list
These errors can be seen in the console on the Tabs Scroller example.
Potential fixes
I haven't found a clean fix for now. We might want to condition MDCTabBar initialization on scroller property but we still need to access MDCTabBar instance to make activeTabIndex work :
componentDidMount() {
	if (!this.props.scroller) {
		this.MDComponent = new MDCTabBar(this.control);
	}
	// This will break if `props.activeTabIndex` is set
	setActiveTabIndex(defaultProps, this.props, this.MDComponent);
}

Another idea would be to merge Tabs and Tabs.TabBarScroller components in a single component :
// would only render tab-bar
<Tabs activeTabIndex={1}>
	<Tabs.Tab>1</Tabs.Tab>
	<Tabs.Tab>2</Tabs.Tab>
	<Tabs.Tab>3</Tabs.Tab>
</Tabs>

// would render tab-bar wrapped in a tab-bar-scroller
<Tabs activeTabIndex={1} scroller>
	<Tabs.Tab>1</Tabs.Tab>
	<Tabs.Tab>2</Tabs.Tab>
	<Tabs.Tab>3</Tabs.Tab>
</Tabs>

This allow us to access MDCTabBar instance to handle activeTabIndex changes and might make more sense (scroller property makes little sense if Tabs are not wrapped in a TabBarScroller).
Let me know what you think.

Merging the two rounds reasonable. Do you want to open a PR for this? Else I'll take it but might take a few days


Beyla crashes panic: runtime error: index out of range [1] with length 1

We install Beyla using the Helm chart in a GKE cluster.
One of the Beyla pods in the daemonset continuously crashes with the following exception:
time=2024-10-11T03:47:25.820Z level=INFO msg="new process for already instrumented executable" component=discover.TraceAttacher pid=15718 child=[] exec=/usr/local/bin/envoy
time=2024-10-11T03:47:25.820Z level=INFO msg="new process for already instrumented executable" component=discover.TraceAttacher pid=3912485 child=[] exec=/usr/local/bin/pilot-agent
panic: runtime error: index out of range [1] with length 1

goroutine 257 [running]:
encoding/binary.bigEndian.Uint16(...)
    /usr/local/go/src/encoding/binary/binary.go:146
github.com/grafana/beyla/pkg/internal/ebpf/common.getTopicNameSize({0xc00272eeb8?, 0xc00272eeb8?, 0x410ae5?}, 0x54?, 0x54?, 0x0?)
    /opt/app-root/pkg/internal/ebpf/common/kafka_detect_transform.go:257 +0xfb
github.com/grafana/beyla/pkg/internal/ebpf/common.getTopicName({0xc00272eeb8, 0x80, 0x80}, 0x7b, 0x1, 0xa)
    /opt/app-root/pkg/internal/ebpf/common/kafka_detect_transform.go:212 +0x5f
github.com/grafana/beyla/pkg/internal/ebpf/common.ProcessKafkaRequest({0xc00272eeb8, 0x80, 0x80})
    /opt/app-root/pkg/internal/ebpf/common/kafka_detect_transform.go:93 +0x106
github.com/grafana/beyla/pkg/internal/ebpf/common.ProcessPossibleKafkaEvent(0xc00272ed80, {0xc00272edb8?, 0x80?, 0x4009e40?}, {0xc00272eeb8, 0x80, 0x80})
    /opt/app-root/pkg/internal/ebpf/common/kafka_detect_transform.go:59 +0x52
github.com/grafana/beyla/pkg/internal/ebpf/common.ReadTCPRequestIntoSpan(_, {_, _})
    /opt/app-root/pkg/internal/ebpf/common/tcp_detect_transform.go:68 +0x5e7
github.com/grafana/beyla/pkg/internal/ebpf/common.ReadBPFTraceAsSpan(_, {_, _})
    /opt/app-root/pkg/internal/ebpf/common/common.go:124 +0x330
github.com/grafana/beyla/pkg/internal/ebpf/common.(*ringBufForwarder).processAndForward(0xc000b28120, {{0xc0039c1400, 0x200, 0x200}, 0x208}, 0xc0004791f0)
    /opt/app-root/pkg/internal/ebpf/common/ringbuf.go:173 +0x16b
github.com/grafana/beyla/pkg/internal/ebpf/common.(*ringBufForwarder).readAndForwardInner(0xc000b28120, {0x2796168, 0xc0004c4460}, 0xc0004791f0)
    /opt/app-root/pkg/internal/ebpf/common/ringbuf.go:159 +0x159
github.com/grafana/beyla/pkg/internal/ebpf/common.(*ringBufForwarder).sharedReadAndForward(0xc000b28120, {0x27b07f8, 0xc0000ba730}, {0xc001a6b408, 0x242, 0x24f}, 0xc0004791f0)
    /opt/app-root/pkg/internal/ebpf/common/ringbuf.go:107 +0x23c
github.com/grafana/beyla/pkg/internal/ebpf/nethttp.(*Tracer).Run(0xc000bde000, {0x27b07f8, 0xc0000ba730}, 0xc0004791f0)
    /opt/app-root/pkg/internal/ebpf/nethttp/nethttp.go:251 +0xfa
created by github.com/grafana/beyla/pkg/internal/ebpf.(*ProcessTracer).Run in goroutine 183
    /opt/app-root/pkg/internal/ebpf/tracer_linux.go:36 +0x233

Beyla image version: 1.8.5
Please kindly advise!

Hey, thanks for reporting! Could you provide more details of the setup? Are you using Kafka?

Hey, thanks for reporting! Could you provide more details of the setup? Are you using Kafka?

yes. We have kafka for other purposes, and these Kafka pods doesn't share the same node with the crashing Beyla pod.
If you guys need any further information, I am glad to help!

Hi @tungpun, thanks again for reporting this crash. The fix was merged just now in both our main branch and our 1.8 release. I've made a new release 1.8.6 which contains the fix and it should be available shortly.


Replace NGettext dependency

The FTP protocol supports translation of response texts (while FTP commands and codes never change).
Support for text translations was implemented through ILocalizationFeature and the NGettext dependency and its ICatalog and Catalog types.
The NGettext catalog attempts to load translation files - but by default fails as there are no translation files.
This changeset drops the NGettext dependency in favor of implementing our own interface. The interface surface is reduced by defining only the methods we use (GetString).
This is a breaking change. The impact is likely very low. Few people will have used or implemented custom translations. For those that did, a fixup is simple.

Users who previously implemented and registered an ICatalog for custom text translation will have to instead implement ILocalizationCatalog. This is a trivial type reference replacement - the two methods we use are equal on both interfaces.
Users who made use of NGettext library magic translation file loading (I assume it may identify and load files if placed correctly) will need to implement a simple forwarding catalog type.

References:

Fixes #144
RFC 2640 Internationalization of the File Transfer Protocol
https://github.com/VitaliiTsilnyk/NGettext

Awesome, thank you! 🎉


Malloc fails to allocate last 8K of RAM

I'm trying to do memory intensive operations on my ESP8266 (cryptography) and often run into out of memory issues so I have to be very careful. Profiling showed that cryptography requires ~22K of RAM. At start, I see that xPortGetFreeHeapSize() reports ~32K but sometimes operation still fails. It seems like if next allocation needs to allocate memory so that free heap size will go less than 8K, it fails.
Is there an explanation for this and/or ways to workaround this limitation and get access to last 8K. That would make code MUCH cleaner (and life easier).

Probably fragmentation, which you could confirm by trying to allocate smaller chunks and seeing if that works. There is a block of unused memory that you might be able to use see open_esplibs/libpp/wdec.c.

I guess you're right.


EmailService to send the attachments

Do we have any way out to add the attachment with email send via EmailService?

@justinedelson @davidjgonzalez we wrote an EmailServiceWithAttachment service based on the ACS Commons EmailService. If this is needed, we can submit a PR.

@kaushalmall this is the first I've heard of this requirement, but I'm sure it would be a nice addition. Is a new service necessary or could this be a new method on the existing service?

we could add a new method, we have the code, so most of the work is done already.

I worked with @kaushalmall on this and will submit a PR soon.


[Feature] Better Find

Details

[x] Handle fragmented word
[x] Find according to status
[x] Find according to tags
[x] Find according to name, status and tags
[x] Precise find according to show name
[x] Update UG and DG for find command

Good job!


Simplify the code to parse a color.

I just saw justforfunc №5 and tried to do the homework. I think this is simpler.
But one question: Do you find it clearer to write uint8(n >> 8) or uint8(n >> 8 & 0xff)? The second is more verbose in what it’s doing, but i kinda like the shorter version.
And btw, thanks for #JustForFunc!

Ups, my code was actually wrong, even if it passed the tests. Fixed both :).

hey @googlebot, can you check the CLA?

apparently @googlebot took they day off today
in order to accept your contributions I'd need to you to sign this CLA @zephyr https://cla.developers.google.com/
thanks

I signed it! @campoy

wait, why did you close it? can't merge it now 😢

Sorry – since the change is so small, i squashed the three commits into one and send a new pull reqest #19. I didn’t thought about that this would be more complicated for you. Will not happen again!


Allow directly calling an object field of a function type

Description:
Following snippet result in undefined function 'func' in object 'xxxxx/main:0.1.0:_Bar', even if it's there.
type _Foo abstract object {
    public function(record{}|error? frame) returns record{}|error? func;
    public function process() returns record{}|error?;
};

type _Bar object {
    *_Foo;

    public function __init(function(record{}|error? frame) returns record{}|error? func) {
        self.func = func;
    }

    public function process() returns record{}|error? {
        record{}|error? cFrame = self.func({});
        return cFrame;
    }
};

However, it'll work if we assigning it into a local var and use it like;
    public function process() returns record{}|error? {
        function(record{}|error? frame) returns record{}|error? func = self.func;
        record{}|error? cFrame = func({});
        return cFrame;
    }

Affected Versions:
1.2.1
OS, DB, other environment details and versions:
n/a

This seems to be a generic issue when attempting to call a function which is a field of an object.
import ballerina/io;

class Foo {

    function () bar;

    function init(function () bar) {
        self.bar = bar;
    }

    function callBar() {
        self.bar(); // error: undefined method 'bar' in object 'Foo'
    }
}

public function main() {
    Foo f = new (() => io:println("Hello"));

    f.bar(); // error: undefined method 'bar' in object 'Foo'
}

IINM, this was not allowed previously, and we had to assign the field's value to a function variable and then call that function. But this should now be allowed according to the spec (see https://github.com/ballerina-platform/ballerina-spec/issues/257).
The method-call-expr is now defined as

A method-call-expr either calls an object's method, calls an object's field where the type of the field is a function or calls a function in the lang library.

Probably fixed in https://github.com/ballerina-platform/ballerina-lang/pull/31796. Looping in @rdhananjaya.

This is fixed with #31796


Updated the accessibility document

Current:

Updated:

@benjdlambert , @vinzscam  Please review the changes. Thank you

Can you please approve this. I will raise future PRs with bulk changes. Thank you.

Please ensure that any changes like this are done in bulk. There's an influx of these simple PR's coming in, and whilst they are somewhat nice to have, it takes a considerable amount of review effort to do these one by one.


Signing a bundle now imply resolving image references into immutable references

This aims at having signed bundles truly immutable. That means that all referenced images (invocationImages or images) without a digest are modified into digested references:

if an image is available in the docker engine, and it has already a RepoDigest, we use it
if an image is not available in the docker engine, we pull it and get its RepoDigest
if an image is available in the docker engine, but has no RepoDigest, we consider that it has been built locally and has not yet been pushed. In that case we fail, and tell the user to push it.

Signed-off-by: Simon Ferquel<EMAIL_ADDRESS>

closing in favour of #563.


Implementing parameters and return between blocks

Implementing issue #3 .
The Async is a generic struct with two types: In, Out.

The In is the parameter that it's blocks receive.
The Out is the parameter that it's return

Chain blocks can receive the Out parameters from previous block or ignore it and return a new type if needed.
All previous code should work without any change.
Usage example:
Async.background { () -> Int in
            return 10
        }.main { (i: Int) -> (String) in
            //Do something with i
                return "Test"
        }.background { (s: String) -> (Double) in
            //Do something with s
                return 10
        }.main { (d: Double) -> () in
            //Do something with d
        }

The only problem is that we have to specify the types on the blocks because I didn't find how to constraint a method to not accept Void type as parameter.
Problem example:
Async.background { () -> Int in
            return 10
        }.main { (i) -> (String) in
            //Do something with i
                return "Test"
        }

Error because the compiler can not choose between type Int or Void to parameter i
I'm waiting for your feedback and available to try fix anything that you find wrong.

Great work! Really good to see some progress on this.
I'll try to have a deeper look at it this week and I'll get back to you.

I was trying to fix the compiler inference. If we remove the methods with ()->(R) and ()->() the compiler will start infer the input types for the chain but we will need to explicitly ignore the input when we have a block that don't need the input value. Example:
This will work
Async.background {
            return 10
        }.main { i -> String in
            //Do something with i
                return "Test"
        }

But this don't:
Async.background {
            return 10
        }.main { () -> (String) in
                return "Test"
        }

But we still can do:
Async.background {
            return 10
        }.main { (_) -> String in
                return "Test"
        }

This is truly awesome @BrunoMazzo!
I've been giving it a go, and it seems to be working as expected.
In me testing this does work.
Async.background {
    return 10
}.main { () -> (String) in
    return "Test"
}

That is, even though .background does return something, it's safe to ignore it in main. I don't know if we can enforce that though. A returned value probably should be used.

Ops, sorry I did not pushed the changes to disable implicitly ignore params. However, your example will work but I believe it's a compiler bug. When we do:
Async.background {
    return 10
}.main { () -> (String) in
    return "Test"
}

the compiler is inferring the first block to ()->()

I've added support for this in the Swift 3.0 branch now. https://github.com/duemunk/Async/tree/feature/Swift_3.0
This couldn't have been done without the work in this PR, so big thanks to @BrunoMazzo.
I don't think it has any bad issues, but sometimes might need help by using explicit argument and return types.

Ow, great!!
I'm glad I helped!
Thanks for the feedback and the great work!

@BrunoMazzo Let me know if I missed something. Some minor tweak or some edge case that gives horrible Swift compiler warning!

I'll close this PR. Again, thanks for your great work!


C44391: Example URLs should be escaped

Hello, @MandiOhlinger,
Localization team has reported source content issue that causes localized version to have broken/different format compared to en-us version.
Description of the source issue: Example URLs should be escaped in order to avoid creating a link
Please review and merge the proposed file change to fix to target versions. If you make related fix in another PR  then share your PR number so we can confirm and close this PR.
Many thanks in advance.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/adapters-and-accelerators/accelerator-rosettanet/creating-or-editing-an-agreement.md
:white_check_mark:Succeeded
View

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/adapters-and-accelerators/accelerator-rosettanet/creating-or-editing-an-agreement.md
:white_check_mark:Succeeded
View

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/adapters-and-accelerators/accelerator-rosettanet/creating-or-editing-an-agreement.md
:white_check_mark:Succeeded
View

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


Error #26 (unsupported certificate purpose) in FreeRADIUS

(1) prerequisites:
-OpenSSL 1.1.0g  2 Nov 2017
-FreeRADIUS Version 3.0.15
-Debian OS
(2) error:
-"SSL says error 26 : unsupported certificate purpose" in FreeRADIUS from ../ssl/statem/statem_srvr.c in line 2896 OpenSSL
-the error is thrown, when the certificate of (3) is read - when the error occurs no client certificate is known to the FreeRADIUS (and therefore, i believe, not of any interest here)
(3) details:
3.1. certficate of interest:
-Version: 3 (0x2)
-Serial Number: 4105 (0x1009)
-Signature Algorithm: sha512WithRSAEncryption
-Issuer: C = DE, ST = Sachsen, L = Leipzig, O = DBFZ Deutsches Biomasseforschungszentrum gemeinnuetzige GmbH, OU = IT, CN = DBFZ CA INTERN ROOT, emailAddress =<EMAIL_ADDRESS>-Validity
--Not Before: Jan 16 12:57:19 2018 GMT
--Not After : Jan 11 12:57:19 2038 GMT
-Subject: C = DE, ST = Sachsen, L = Leipzig, O = DBFZ Deutsches Biomasseforschungszentrum gGmbH, OU = IT, CN = CAPF-91d43ef6
-Subject Public Key Info:
--Public Key Algorithm: rsaEncryption
--Public-Key: (2048 bit)
--Modulus: 00:ad:[...]
--Exponent: 65537 (0x10001)
--X509v3 extensions:
---X509v3 Subject Key Identifier: 58:A4:EB:D9:DD:CE:A2:99:72:3B:E1:20:19:1D:40:C1:F9:D5:C2:28
---X509v3 Authority Key Identifier: keyid:E2:E9:20:42:29:83:C4:77:8C:87:AB:FA:4B:A1:A9:C4:CE:00:BD:39
---X509v3 Basic Constraints:
----critical
----CA:TRUE, pathlen:0
---X509v3 Key Usage:
----critical
----Digital Signature, Certificate Sign, CRL Sign
---X509v3 Extended Key Usage:
----critical
----TLS Web Server Authentication
---Signature Algorithm: sha512WithRSAEncryption 08:b4:[...]
3.2. end entities
-the environment includes Cisco IP phones
-the call manager demands in its (unfortunately automatically generated) CSR to be an intermediate CA with all the above keyUsages and extended keyUsages
(4) proposal:
-Cisco's approach to include that extended keyUsage is not chosen nicely, but it is out of our administrativ hands to change this
-RFC 5280 says: "In general, this [extended keyUsage] extension will appear only in end entity certificates."
-this somehow doesn't forbid Cisco to do, what they do
-therfore it should not cause an error within the OpenSSL routine that FreeRADIUS uses

It's not clear to me from the above description what the issue actually is. Please can you expand?

The actual problem is the combination of CA:true and the key usages pressed into one certificate by Cisco.
One can help themself by expanding the enterprise PKI from windows CAs to a intermediate CA which is based on Debian and OpenSSL and allows better csr/request handling than the windows service. You would than execute:
$ openssl ca -config config/openssl.cnf -in csr/telekom/CAPF.csr -out newcerts/_testCAPF/CAPF.crt -extensions v3_intermediate_ca

where openssl.cnf has a section much like the following:
[ v3_intermediate_ca  ]
subjectKeyIdentifier = hash
authorityKeyIdentifier = keyid:always,issuer
basicConstraints = critical,CA:true,pathlen:0
keyUsage = critical,cRLSign,keyCertSign

this throws the problematic extension out and et voilá you wont even have problems with the Cisco system, because the certificate doesn't not need the TLS Web Server Authentication usage.

Not a 1.1.1 specific issue so removing that milestone.

I do not think it is a bug in OpenSSL that it rejects intermediate CA certs with the
---X509v3 Extended Key Usage:
----critical
----TLS Web Server Authentication

Marking as inactive to be closed at the end of 3.4 dev barring further input


[TFLite] Support depthwise convolution multiplier greater than 1

Currently, TFLite FE only support depthwise convolution multiplier equals to 1. However, community users report we need this feature and Tensorflow FE has supported it too recently. This PR does support this.
Related links and discussions:

https://discuss.tvm.ai/t/deployment-of-converted-pb-file-to-tflite-using-relay-frontend-from-tflite/3307/
https://discuss.tvm.ai/t/tf-kernel-layout-hwio-vs-hwoi/3895

cc: @tmoreau89

@yzh119 @tqchen @kevinthesun @icemelon9 could someone help to manage it? Thanks.


remove ability to register own diagnostic source

It should not be possible to register own diagnostic source.
regarding to DiagnosticSource practices:
https://github.com/dotnet/corefx/blob/master/src/System.Diagnostics.DiagnosticSource/src/DiagnosticSourceUsersGuide.md#diagnosticlistener-names

OMG you got a PR in! :)
Seriously, thank you. I definitely want the feedback on the names of the listeners, etc. and need help with how to configure/use so I can put it in the new documentation.

thx! 👍
The use case for DiagnosticSource is the easiest part (except AppInsights, but I've already documented it). All providers I used right now are only AppInsights and Jaeger (with OpenTracing .NET), for Jaeger - it connects to All diagnostic sources (https://github.com/opentracing-contrib/csharp-netcore/blob/f47503f6277b35f0be7ec6db871de0b2285dd2b6/src/OpenTracing.Contrib.NetCore/Internal/DiagnosticManager.cs#L53), same for APM
I'll be glad to help you to document it! :)


Add bazel coverage support

Fixes issue #448. I more or less followed the comments in the issue from @achew22. Then I added code to some other parts that were required. Let me know if I missed something else.
Apart from that, I also added a couple of very simple tests for the test and coverage commands to internal/e2e/simple/simple_test.go.

Sorry for taking so long to fix the PR. I was busy with other stuff :/


Added a config option to specify proxy's listening address/port

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


Database

Describe the chore
Sir, where is the data base file please?
The app create it somewhere or is on the internet?
Would you like to work on the issue?
I cannot find it

Ok, thank you for the explication.


[NSData MIH_dataByBase64DecodingString:] not decoding valid base64 strings

[NSData MIH_dataByBase64DecodingString:] is behaving weird. Below code will return nil while it is a valid base64 encoded string
NSString *derString = @"MIIBVQIBADANBgkqhkiG9w0BAQEFAASCAT8wggE7AgEAAkEAmZruJYeMsKl6IceG\n"
                      "TB/8gZW0m+sdVb8X617zeEHzliIK4A6q9obsizLh1hvAFkqS4ir3H4VVn7pKuxpF\n"
                      "C5oVKQIDAQABAkApWzJVPmqdwfOVFrBM3KvSg+kNtb6/MGUTRQxOS4t21ybGi2Rn\n"
                      "eXU8bucIhUjETTnwgdkRr56/e/C3Mn7BOnq1AiEAyKg/sFruB0be/VW8x4dAuM/V\n"
                      "YlOk6TGRXuE2cm4uAysCIQDD+H3ZU9r8P+3+XRxucfa9p1Lf02zuQ5yBqZaq/yHu\n"
                      "+wIgeUdP70yWT6mjP6Vhi4uRL+LWSy7ZHuUJwzoGCZXUk68CIQCzo1ZhYG/NaSDV\n"
                      "WS7VwxvmD7p1OE6TPmD5rqZRSxwk/wIhAKaDoovCCJuBlVWQlAypZ+PriMqsUHVP\n"
                      "Zg54vniNK3Yv";
NSData *der = [NSData MIH_dataByBase64DecodingString:derString];

while below code decodes it correctly
NSData *der = [[NSData alloc] initWithBase64EncodedString:derString options:NSDataBase64DecodingIgnoreUnknownCharacters];

Drilling down the code, I found out that you have used 0 for NSDataBase64DecodingOptions. And for me to get it working I had to use options:NSDataBase64DecodingIgnoreUnknownCharacters. Did you use 0 beacuse of any particular reason or is it a bug?

I'm sorry, but I can't tell you anymore why I don't used NSDataBase64DecodingIgnoreUnknownCharacters. I'll think about changing it in the next release if I won't find the reason why I used 0.


plugin configuration is broken

With the 0.7.x codestream, the config.schema.json file does not match the expected configuration, resulting in the user unable to configure the plugin using the GUI.
I am working on a fix.

with 0.7.7 I found some way to design the configuration being compatible with config.schema.json.
However, the code had to be redesigned to convert the array into an object. I am not a very good JavaScript developer so I appreciate any help in writing a better (speak: more robust and fault tolerant) code for reading the config.


Warn when downgrading dependency mod

When installing a mod with a dependency that requires an older version of said dependency than what is already installed, we should show a warning and/or option not to downgrade.

Maybe we should even show a warning if two mods use a dependency of a different version.
If we assume everyone adheres to semver, we would only have to show the warning on different major version and otherwise just stick to the newer out of the two versions.


New Parser and Analyzer support KillStmt （Easy）

Required work and output

Use antlr4 to complete parse statements. The generated statement must follow the original data structure, and should be compatible with the old statement.

In the original syntax file  sql_parser.cup, the new syntax needs to be compatible with the syntax of the old version, using the syntax file sql_parser.cup of the old version as the reference standard.

Remove the logic of analytic functions in the statement and move the relevant logic to the new Analyzer framework. The new Analyzer is in the SQL/Analyzer directory.

If the code related to Privilege Check exists in the original analyzer, move the code related to privilege control into PrivilegeChecker.

When submitting the PR, please make sure there is unit test coverage for related changes.

Task

[ ] KillStmt

How to participate?

Pick an issue you are interested in, click the issue link and leave a comment under the issue to sign up.  One person can only choose one issue and the first person who picks the issue will be assigned.

After the issue is assigned, you should complete the task in 14 days. If you are unable to complete it within the specified time, the task will be withdrawn and assigned to others.

Participants are required to produce two pieces of content (forms including but not limited to mind maps, study notes, blogs, and tutorials) in the course of the activity, each of which should be at least 200 words.

Got questions? Join our  Slack channel  and find channel:  #sql-planner.

Before you start

Read  Contributing.md  carefully to understand how to submit a pull request and how to configure BE and FE development environment.

Sign the  Contributor License Agreement  before you make a pull request.

To learn more about StarRocks, please refer to  StarRocks documentation.

Pull request requirements

Create a pull request (PR) after you complete the task.

PR title format: [SR SQL Planner Summer Camp] + PR title

After submitting PR, you can comment under the pinned Issue [SR SQL Planner Summer Camp].

Comment format: "I have finished Issue #" + PR link

please assign this to me, thx

Hello @harveyyue ,Would you mind add my WeChat account (WeChat ID:StarRocks-1) or share your WeChat account with me? I will mail you the gift of StarRocks SQL Planner Summer Camp.

Hello @harveyyue  ，I am cuicui from the StarRocks community.Would you mind add my WeChat account (WeChat ID:StarRocks-1) or share your WeChat account with me? I will mail you the gift of StarRocks SQL Planner Summer Camp.


MemoryError installation at 99%.

Versions

Python: Python 2.7.14 (v2.7.14:84471935ed, Sep 16 2017, 20:19:30) [MSC v.1500 32 bit (Intel)] on win32
OS: Windows 10 Home , Version 1703, OS Build 15063.726
Kivy: 1.10.0
Kivy installation method: https://kivy.org/docs/installation/installation-windows.html#Installation 2. python -m pip install kivy.deps.gstreamer

Description
I tried to install Kivy following the instructions provided on kivy.org. But I encountered the same error 3 times at 99%.
Code and Logs
Collecting kivy.deps.gstreamer
  Downloading kivy.deps.gstreamer-0.1.12-cp27-cp27m-win32.whl (121.0MB)
    99% |############################### | 121.0MB 153kB/s eta 0:00:01Exception:
Traceback (most recent call last):
  File "C:\Python27\lib\site-packages\pip\basecommand.py", line 215, in main
    status = self.run(options, args)
  File "C:\Python27\lib\site-packages\pip\commands\install.py", line 335, in run
    wb.build(autobuilding=True)
  File "C:\Python27\lib\site-packages\pip\wheel.py", line 749, in build
    self.requirement_set.prepare_files(self.finder)
  File "C:\Python27\lib\site-packages\pip\req\req_set.py", line 380, in prepare_files
    ignore_dependencies=self.ignore_dependencies))
  File "C:\Python27\lib\site-packages\pip\req\req_set.py", line 620, in _prepare_file
    session=self.session, hashes=hashes)
  File "C:\Python27\lib\site-packages\pip\download.py", line 821, in unpack_url
    hashes=hashes
  File "C:\Python27\lib\site-packages\pip\download.py", line 659, in unpack_http_url
    hashes)
  File "C:\Python27\lib\site-packages\pip\download.py", line 882, in _download_http_url
    _download_url(resp, link, content_file, hashes)
  File "C:\Python27\lib\site-packages\pip\download.py", line 603, in _download_url
    hashes.check_against_chunks(downloaded_chunks)
  File "C:\Python27\lib\site-packages\pip\utils\hashes.py", line 46, in check_against_chunks
    for chunk in chunks:
  File "C:\Python27\lib\site-packages\pip\download.py", line 571, in written_chunks
    for chunk in chunks:
  File "C:\Python27\lib\site-packages\pip\utils\ui.py", line 139, in iter
    for x in it:
  File "C:\Python27\lib\site-packages\pip\download.py", line 560, in resp_read
    decode_content=False):
  File "C:\Python27\lib\site-packages\pip\_vendor\requests\packages\urllib3\response.py", line 357, in stream
    data = self.read(amt=amt, decode_content=decode_content)
  File "C:\Python27\lib\site-packages\pip\_vendor\requests\packages\urllib3\response.py", line 314, in read
    data = self._fp.read(amt)
  File "C:\Python27\lib\site-packages\pip\_vendor\cachecontrol\filewrapper.py", line 63, in read
    self._close()
  File "C:\Python27\lib\site-packages\pip\_vendor\cachecontrol\filewrapper.py", line 50, in _close
    self.__callback(self.__buf.getvalue())
  File "C:\Python27\lib\site-packages\pip\_vendor\cachecontrol\controller.py", line 275, in cache_response
    self.serializer.dumps(request, response, body=body),
  File "C:\Python27\lib\site-packages\pip\_vendor\cachecontrol\serialize.py", line 87, in dumps
    ).encode("utf8"),
MemoryError

Try with --no-cache-dir. IIRC, Gstreamer is too big for the PIP cache, therefore it results in this error, however the installation works just fine if not put to the cache. If it works for you, feel free to open a pull request with the change in the installation docs :)

Yep it worked. Thank you KeyWeeUsr


make buildKPLConfig public

we're currently using copy/pasta because we need access to this method

created https://github.com/WW-Digital/reactive-kinesis/pull/54 for this
Will wait for @markglh to have the final say since I've been away from the lib for a bit


Added medical machine boards and their respective recipes.

I have added all missing medical machine boards and their respective recipes. Bio press, centrifuge, you name it.

Fails to lint, with:
Unhandled exception. System.ArgumentException: An item with the same key has already been added. Key: ComponentRegistration(Machine: Content.Server.Construction.Components.MachineComponent)
   at System.Collections.Generic.Dictionary`2.TryInsert(TKey key, TValue value, InsertionBehavior behavior)
   at Robust.Shared.Serialization.TypeSerializers.Implementations.ComponentRegistrySerializer.ToTypeIndexedDictionary(SequenceDataNode node, IComponentFactory componentFactory)

Will investigate in a bit.


Week 7 Review Ticket

Crossover Name:
Total Score: /2

[ ] GH Indi Page, Crossover Graded Review Ticket, Study Plan - /1
Comments:

[ ] Quiz5 MCQ results >50% and test corrections - /1
Comments:

Crossover Name: Gennalyn Bongolan
Total Score: /2
GH Indi Page, https://github.com/arushi10/individual/issues/10, https://github.com/arushi10/individual/issues/7 - /1
Comments:
Quiz5 MCQ results >50% and test corrections - /1
Comments:


Understand Hot/Cold regions and disassemble both if required

@JosephTremoulet What's the best doc on hot/cold splitting?
Trying to understand what gets where during normal JIT process.

I went looking for documentation, and found nothing more informative than this reference which indicates that it's only employed in ngen, so likely a moot point for your use case.
Anyway, the idea is that it's better for cache locality to put the "hot" (commonly-executed) code from all methods near each other, and the "cold" (rarely-executed) code off somewhere else that it hopefully never even needs to get paged in when the program is running.  For example, we assume that exceptions are rare, so code in catch handlers is the sort of thing we'd want to exile to the "cold" section, as is code that must lead to throwing an exception, e.g.:
if (x < 5) {
  // code here is cold
  // we'll put it somewhere else
  throw ...;
}
try {
  foo();
} catch (Exception) {
  // this is cold too
  bar();
}
bash();

becomes something like
cmp x, 5
jge <some far-away offset in the cold region>
call foo
<no code here for the catch handler, it's also somewhere far-away>
call bash
...

When we're not doing hot/cold splitting via the separate buffers (which apparently is always the case when jitting), we just move the cold stuff to the end of the method (and each method has one contiguous chunk that gets reported as the "hot" region), as opposed to putting the hot bits from all the methods in one place and the cold bits in another.
If you did encounter a method with both regions populated, it would make sense to disassemble it as
; Function FooBar()
<code from hot region here>
;;; cold code:
<code from cold region here>

@JosephTremoulet Thanks a lot!
I'll use this to track disassembling cold regions (if present), which I don't do yet.


Bring PillLinkComponent usage in line with GovukComponent::BackLinkComponent etc

Review app deployed to https://teaching-vacancies-review-pr-3122.london.cloudapps.digital


build: moved to pyproject.toml/poetry

Fixes #19, and more.
Changes

setup.py/requirements.txt replaced with pyproject.toml/poetry.lock

Dependencies are now managed/locked using poetry

pip install -r requirements.txt is now simply pip install .

Updated the documentation to reflect this fact.
Note that installing with pip install . will not use the exact versions in poetry.lock (but one can achieve the same thing by running poetry export -f requirements.txt; pip install -r requirements.txt as usual). Version constraints specified in pyproject.toml will still be respected.

Added basic GitHub Actions CI. (merged in #24)

It installs the package (with poetry) and then runs: mypy for typechecking, pytest for testing (but there are currently no tests to run).

Loosened version constraints for larger dependencies (which are unlikely to break backwards compatibility without DeprecationWarning)
Fixes ability to install directly from git with pip install git+https://...

TODO (if requested)

[x] Build docs in CI (merged in #24)
[x] Deal with the other requirements*.txt files, include them in pyproject.toml as dev dependencies or extras (that can be enabled with a flag)

I haven't done any extensive testing, but all seems to work as expected.

Hi Erik.
This great - thanks!
Will do some review and testing very soon.
In the meantime, a few other comments:

The installation instructions do need changing to  pip install -e .  from pip install -r requirements.txt anyway, because that is needed for the new CLI command line program. And it's just the correct way to do it anyways. So, this is good that you have done that.

Would this also work with a pip install git+https://github.com/neurotechx/eeg-notebooks ? We would like to have that working so that an explicit git clone isn't necessary. I tried this once and it failed; not sure why.

...and eventually (soon) we would like to put on pypi so it's just pip install

Yes, CI building for the docs is something we want to do also. The sphinx library generation is quite long working from scratch. Don't know if that is an issue.

TBH I'm thinking of removing the other requirements files. The extra doc building libraries are a minimal addition to the other large primary libraries. The no-eeg version could be useful in principle, but currently at least it isn't something we are using extensively. So, don't worry about those for now; they might disappear.

Would this also work with a pip install git+https://github.com/neurotechx/eeg-notebooks? We would like to have that working so that an explicit git clone isn't necessary. I tried this once and it failed; not sure why.

Yes, just tested.

...and eventually (soon) we would like to put on pypi so it's just pip install

Poetry will make that a lot easier for you :)

Yes, CI building for the docs is something we want to do also.

I'll take a look at it then, should be pretty easy.

@JadinTredup Should be ready for review now. The CI can be inspected at https://github.com/ErikBjare/eeg-notebooks/actions (it should work automatically in this repo too, once merged).

I've extracted the CI stuff to a seperate PR #24, I'd appreciate a quick merge so I can rebase this branch on top of it.

PR has been rebased on master following the merge of #24.

lol, the OOM killer ended up killing it:

dmesg:

Not sure what to do about this, I guess ask in https://github.com/python-poetry/poetry/issues/2094 or bring it up with the psychopy devs if they can loosen their dep requirements? (or whatever is taking forever)


#28 Emitter class - Return require inline

Description
Implement @Chemaclass recommendation in the https://github.com/jenshaase/phel-lang/pull/28#discussion_r437644221 comment about return inline the require.

Thanks! Just merged it!


Implementing documents API endpoint

Description

Will implement documents API endpoint along with unit tests.

@happy-san In API call class body parameters for post function is Map<String, String> but documents can have Map<String, dynamic. I will make the required changes in ApiCall class

@HarisaranG I was thinking to change it to Object similar to put function. But I guess if Map<String, dynamic> does it, for now, it's better.

@happy-san For performing import in documents endpoint jsonl string format documents are required. So I need to perform request directly with _requestCore and _requestUri which are private members what should I do now ?

Are you referring to this?

Also, I've added CONTENT_TYPE constant to ApiCall. You can use it like:
apiCall.post( 'endpoint',
  additionalHeaders: {ApiCall.CONTENT_TYPE: 'text/plain'},
);

Are you referring to this?
If yes, then you simply need to update the post function's body parameter to Object and then apiCall.post() would accomplish what this code is accomplishing.

Like the import function takes String as a body parameter. But the post function has Map<String, dynamic>.  So now can I change Map<String, dynamic> to dynamic or change _requestCore and _requestUri to public functions and make the post request using it?

@HarisaranG You can fetch the latest commit from the master repo, I've made the changes required for you to implement the import function.

@happy-san Yeah now I am able to call the post function.
Check this out: Document Import Response
{"success": true}
{"success": true}
{"success": true}

But the response is also a String of jsonl. But the post function return Map<String, dynamic>.

@HarisaranG Could you open a PR with what you've done so far? Then we can brainstorm on this together.

Yeah sure on it !

@HarisaranG Checkout this:
import 'dart:convert';

void main() {
  final jsonl = """
  {"first": 1}
  {"second": 2}
  {"third": 3}""", encoded = json.encode(jsonl), decoded = json.decode(encoded);

  print('$jsonl\n');
  print('$encoded\n');
  print('$decoded\n');
  print(json.decode(jsonl)); // This throws format exception
}

Indeed _handleNodeResponse would freak out if jsonl is returned from the server.

Even json.decode(response.body) would fail passing the jsonl string
Do you have any suggestions on how this could be avoided? I don't have any good solutions atm.

The best way I think to handle this situation is to decouple the node pooling logic and the request/response logic of the ApiCall class. Then there can be multiple implementations with custom request/response but a single node pool.
Something like this:
abstract class BaseApiCall<R> {
  final Configuration _config;

  BaseApiCall(Configuration config) : _config = config;

  Future<R> sendRequest(Future<http.Response> Function(Node) predicate) async {
    http.Response response;
    for (var triesLeft = _config.numRetries;;) {
      final node = NodePool.nextNode();
      try {
        response = await predicate(node).timeout(
          _config.connectionTimeout,
        );
        return handleNodeResponse(node, response);
      } catch (e) {
        if (--triesLeft <= 0) {
          // We've exhausted our tries, rethrow.
          rethrow;
        }

        if (e is RequestException && response.statusCode < 500) {
          // If response is anything but 5xx, rethrow.
          rethrow;
        } else {
          // Retry all other HTTP errors (HTTPStatus > 500) after [retryInterval].
          await Future.delayed(_config.retryInterval);
        }
      }
    }
  }

  Future<R> handleNodeResponse(Node node, http.Response response);
}

Do you have any suggestions on how this could be avoided? I don't have any good solutions atm.

Having statically typed data types is good, but we could change variables to dynamic  types for this case. So that even updating the client on the coming up version would be easier I guess

Which variables are you referring to specifically?

@happy-san
I was looking into api_call.dart primarily we need to change for _handleNodeResponse. Then, it will be a chain reaction to change in all functions.
Or a workaround way would be creating a new public function for this case in ApiCall but this would very difficult for the client updates

Or a workaround way would be creating a new public function for this case in ApiCall but this would very difficult for the client updates

Exactly why I am looking to create an abstraction such that any request/response type can be implemented without any conflict with existing ones. I'll update you regarding this by tomorrow.
P.S. Make sure your PR branch has the latest changes from the master. Currently it's in conflict.

P.S. Make sure your PR branch has the latest changes from the master. Currently, it's in conflict.

Oh really ?. It is in line with the master branch only but I have updated it with some changes for the post function alone in ApiCall,
Future<Map<String, dynamic>> post(
    String endpoint, {
    Map<String, dynamic> queryParams = const {},
    Map<String, dynamic> additionalHeaders = const {},
    Object bodyParameters,
  })

Because queryParams could be a integer also

Oh, I was looking to rebase your PR into master, it can be merged instead. Nothing to worry about, my bad.

Awesome! Waiting for your update ⌛


Not working on Mozilla for  Android

its working almost in all browsers, but i recently got to know that its not working on Mozilla Firefox for  Android. i tested this on Mozilla Firefox 47 for Android.

Hello and thanks for the feedback.
However, I can't fix it. It must be a Firefox Mobile limitation.


No matching package found

I'm having a problem while updating my project. Whenever, I run composer update I get the following error:
[Composer\DependencyResolver\SolverProblemsException]
    Problem 1
      - yiisoft/yii2 dev-master requires bower-asset/jquery.inputmask 3.1.* -
  > no matching package found.
      - yiisoft/yii2 2.0.x-dev requires bower-asset/jquery.inputmask 3.1.* ->
   no matching package found.
      - Installation request for yiisoft/yii2 * -> satisfiable by yiisoft/yii
  2[2.0.x-dev, dev-master].
  Potential causes:
   - A typo in the package name
   - The package is not available in a stable-enough version according to you
  r minimum-stability setting
     see <https://groups.google.com/d/topic/composer-dev/_g3ASeIFlrc/discussi
  on> for more details.
  Read <http://getcomposer.org/doc/articles/troubleshooting.md> for further c
  ommon problems.

composer.json:
{
  "name": "abhi1693/yii2-app-advanced-startup-kit",
  "description": "Yii 2 Practical Advanced Application Template (Startup Kit)",
  "keywords": [
    "yii2",
    "framework",
    "practical",
    "application template",
    "advanced"
  ],
  "type": "project",
  "license": "MIT License",
  "authors": [
    {
      "name": "Abhimanyu Saharan",
      "email"<EMAIL_ADDRESS>      "homepage": "http://abhi1693.github.io/yii2-app-advanced-startup-kit",
      "role": "Developer"
    }
  ],
  "support": {
    "issues": "https://github.com/abhi1693/yii2-app-advanced-startup-kit/issues?state=open",
    "source": "https://github.com/abhi1693/yii2-app-advanced-startup-kit/"
  },
  "minimum-stability": "dev",
  "require": {
    "php": ">=5.4.0",
    "yiisoft/yii2": "*",
    "yiisoft/yii2-bootstrap": "*",
    "yiisoft/yii2-swiftmailer": "*",
    "abhi1693/yii2-user": "*",
    "abhi1693/yii2-config": "1.0.0",
    "abhi1693/yii2-enum": "1.0.0",
    "abhi1693/yii2-installer": "*",
    "abhi1693/yii2-system-info": "*",
    "raoul2000/yii2-bootswatch-asset": "*",
    "kartik-v/yii2-widget-sidenav": "*",
    "bower-asset/admin-lte": "*",
    "knplabs/packagist-api": "1.*@dev"
  },
  "require-dev": {
    "yiisoft/yii2-codeception": "*",
    "yiisoft/yii2-debug": "*",
    "yiisoft/yii2-gii": "*",
    "yiisoft/yii2-faker": "*"
  },
  "config": {
    "process-timeout": 1800
  },
  "extra": {
    "asset-installer-paths": {
      "npm-asset-library": "vendor/npm",
      "bower-asset-library": "vendor/bower"
    }
  }
}

https://github.com/yiisoft/yii2/issues/7402#issuecomment-75782873


vimPlugins.cord-nvim: move to separate folder; vimPlugins.cord-nvim: 0-unstable-2024-09-26 -> 0-unstable-2024-12-17

Things done
Split off https://github.com/NixOS/nixpkgs/pull/366698

Built on platform(s)

[ ] 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

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

[ ] Fits CONTRIBUTING.md.

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

Maybe this could use unstableGitUpdater ?

No, I've been replacing unstableGitUpdater with nix-update-script. I have seen unstableGitUpdater fail too often where nix-update-script works fine.


Support main file of installed npm dependencies

Take for example, angularfire2. It declares a main file in its package.json, angularfire2.js.
If I try to import a symbol without specifying a path to a file, a 404 occurs:
import {FIREBASE_PROVIDERS, defaultFirebase} from 'angularfire2';

GET http://localhost:5555/node_modules/angularfire2.js 404 (Not Found)

I think the fix should be as simple as setting the main field in System.config.packages.angularfire2 to the value from package.json.(main|browser) 'angularfire2.js'. I can open a PR once I figure out where to do that...

Worth noting, I know the easy fix to this issue is to import from angularfire2/angularfire2.

This is an issue for me too, where I can explicitly put a full path but then it fails on a dependency of the lib. My example is angular2-config which fails as

http://localhost:5555/node_modules/angular2-config.js

But if I import from 'angular2-config/dist/index' it fails on

http://localhost:5555/node_modules/nested-property.js

which is it's dependency.

@jeffbcross @beaverusiv I suppose what you are looking at it here.
That's maybe not the right place for this discussion but IMO, following the simple convention {package_name}/{module_name} for your packages (instead of some other fancy ones) make things so much easier (no need of specific config for every packages).

following the simple convention {package_name}/{module_name} for your packages

For the sake of simplicity, let's assume I'm only a consumer of packages, not an author. I'd venture to guess that most modules don't adhere to importing via package/module. We did it with Angular 2 because Angular 2 is big and supports many platforms, and provides several optional submodules. So for most npm packages, I would have to either import from package/index.js or figure out what the main file is and import that, angularfire2/angularfire2.js.

(no need of specific config for every packages).

Declaring a main entry is a convention already supported by node's module resolution via the main field of package.json and System config via the main field of packages config. It wouldn't require package-specific configuration; it would just be a build step to look at the package.json of the package and see if it declares a main file, or if it has an index.js file, and then add it to the system config.

@ludohenin I think that this behavior was implemented in Typescript itself (on version 1.6.3 if my memory is not wrong). So, I think this could be a feature request to SystemJS?

I am wondering if something like system-npm should be used? I've tried integrating myself to no avail, and also looking into issues with SystemJS - of which their official line is "not my problem" when asked about using node-like loading algorithms.

For the first-level dependencies we can generate the configuration with something like that:
function generateConfig(root) {
  return fs.readdirSync(root).map(f => {
    var path = join(root, f, 'package.json');
    if (fs.existsSync(path)) {
      return [f, require(`.${sep}${path}`).main];
    }
  })
  .filter(p => p && p[1] && !/^index\.js|index$/.test(normalize(p[1])))
  .reduce((res, c) => {
    res[c[0]] = { main: c[1] };
    return res
  }, {});
}

This works well in most cases with the flat structure of npm. However, in case we have nested dependencies, therefore name collisions of dependencies with different versions, it won't be trivial to resolve them when required.
It doesn't work with angularfire2 because of its dependency firebase, which exports two bundles:

main - which is the node module
browser - which is the browser bundle

If I change the above implementation to:
if (fs.existsSync(path)) {
  let package = require(`.${sep}${path}`);
  return [f, package.browser || package.name];
}

it fails because other modules are using the browser field in package.json differently.

Would it be better to approach packages with Bower? Would that solve the issue?

@beaverusiv bower, with its flat directory structure, will be a workaround rather than a solution.
The possible case I see is to use the above solution and allow developers to customize the configuration further if required.

@jeffbcross do you have any ideas on this one?

However, in case we have nested dependencies, therefore name collisions of dependencies with different versions, it won't be trivial to resolve them when required.

@mgechev the tooling could just support peerDependencies and leave it up to manual configuration if nested dependencies are required. For cases where manual editing is required, perhaps a couple of commented-out lines could be added to the system config to give hints where a package needs to be configured:
System.config({
  map: {
    /**
     * Package "firebase" needs to be configured.
     * See: https://github.com/systemjs/systemjs/blob/master/docs/config-api.md
     **/
    // firebase: 'node_modules/firebase/',
    'angularfire2': 'node_modules/angulafire2/'
  },
  packages: {
    angularfire2: {
      main: 'angularfire2.js',
      format: 'cjs',
      defaultExtension: 'js'
    }
  }
})

@jeffbcross we're setting packageConfigPaths here.
This seems to solve the problem of using the main file, to some extent. Do you think we're good here or SystemJS needs further hints?

Closing the issue for now until we renew the discussion if necessary.


creating a new project (try to compile and get this error: LINK : fatal error LNK1104: cannot open file 'dep_zlib.lib

axmol version:

Maybe a PR issue:  Update thirdparty prebuilts to v44 #1076

I create a new projet and its fail ([prebuilt version)

LINK : fatal error LNK1104: cannot open file 'dep_zlib.lib' [D:\______\aismann\__private\axmol\Clipper2\build_64\Clippe
r2.vcxproj]

ther e are some more errors:
OpenSSL::Crypto.lib
OpenSSL::SSL.lib
dep_*

@halx99
I got still the error using latest axmol:
  Generating Code...
LINK : fatal error LNK1104: cannot open file 'dep_zlib.lib' [D:\______\aismann\__private\axmol\Clipper2Tester\build_64\Clipper2Tester.vcxproj
]

plz remove CMakeCache.txt from both engine and your project

It was a new download and newc build of axmol.
And a new Project of course

I will try it on another machine again

ok. Its works. but for me its look the whole axmol stuff is rebuild too:

I tried it again and i got this:

It is VS2022 x64 build

Yes


Tickets/dm 25789

The updates look good. One other thought is to add doc strings for the new photRepeat and calcPhotRepeat functions.

Yep - I was thinking the same re: docstrings. I’m running it on all visits from RC2 to test the persistence of extras. Will add docstrings before I “finalize”.
On Apr 6, 2021, at 7:04 PM, Keith Bechtol<EMAIL_ADDRESS>wrote:
The updates look good. One other thought is to add doc strings for the new photRepeat and calcPhotRepeat functions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/lsst/faro/pull/78#issuecomment-814547068, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFDBY7M7PP2TO47W4H7U7Y3THO4RNANCNFSM42NUOTAA.

Jeff Carlin
DM Science Validation Scientist and Community Scientist
Rubin Observatory/Legacy Survey of Space and Time (LSST)
950 N. Cherry Ave.
Tucson, AZ 85719
<EMAIL_ADDRESS>(520) 318-8533

I believe I have addressed all comments, and have added docstrings. I'm a novice at writing docstrings, so please let me know if anything obvious is amiss.


We could allow connecting to unix sockets on macOS

Required information

Distribution: homebrew package on macOS Catalina
Distribution version: 3.20
The output of "lxc info" or if that fails:

# some ssh command...
~ $ export LXD_SOCKET=$HOME/lxd.socket
~ $ lxc info
This client hasn't been configured to use a remote LXD server yet.
As your platform can't run native Linux instances, you must connect to a remote LXD server.

If you already added a remote server, make it the default with "lxc remote switch NAME".
To easily setup a local LXD server in a virtual machine, consider using: https://multipass.run
1 ~ $ lxc --force-local info
This client hasn't been configured to use a remote LXD server yet.
As your platform can't run native Linux instances, you must connect to a remote LXD server.

If you already added a remote server, make it the default with "lxc remote switch NAME".
To easily setup a local LXD server in a virtual machine, consider using: https://multipass.run

Issue description
SSH can be used to forward unix sockets over network, and connecting via forwarded unix sockets already works when lxc is running on Linux.  Apparently macOS can do the same; do we want to relax this requirement?

Well, even with that, you don't have an actual local LXD.
Does:
lxc remote add blah unix:///path/to/socket
lxc remote switch blah
Work on macOS?

If it does, then I believe our behavior is correct and does support one using a redirected unix socket.

Thanks, that works.  So is there something special with local:?  IMHO that's a bit confusing, we'd better add some docs/hints on this use case.

local is a static remote, same as ubuntu or ubuntu-daily.
That means it is guaranteed to be present on all LXD installations and cannot be altered by the user.
Its address is unix:// which uses the default socket path considering things like the LXD_DIR and LXD_SOCKET environment variables.
So this can never be altered even if the local system doesn't run LXD. What can be altered is the default remote, which is what the LXD client uses as its target server. Out of the box this is set to local but lxc remote switch lets you change this to point to whatever you want.
On systems that don't run LXD locally, you usually want to use that to point to your remote LXD server.
Closing as behavior matches what we expect and adding a unix remote works fine.


feat: optimize github workflow config

Summary

🤖 Generated by Copilot at 945e4fa
This pull request simplifies and optimizes various GitHub workflows in the web-infra-dev/modern.js repository by removing unnecessary or redundant configuration options, using the latest versions of the runners and actions, and fetching only the latest commit when possible. This improves the readability, maintainability, and performance of the workflows, and also deletes the unused check-changeset workflow.
Details

🤖 Generated by Copilot at 945e4fa

Add fetch-depth: 1 option to actions/checkout@v3 steps in various workflows to speed up checkout and reduce disk usage (link, link, link, link, link, link, link, link, link, link, link, link, link)
Simplify workflows by removing unnecessary matrix strategies and hardcoding runs-on and node-version values (link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link)
Add a new step to lint-linux workflow to run check-changeset script, which validates the changeset file for the release process (link)
Update the comment for fetch-depth: 100 option in release-pull-request workflow to clarify its purpose (link)
Delete .github/workflows/check-changeset.yml file, which is no longer needed since the script is integrated in lint-linux workflow (link)

Related Issue

Checklist

[ ] I have added changeset via pnpm run change.
[ ] I have updated the documentation.
[ ] I have added tests to cover my changes.

👏


AcrylicBrush: Update acrylicbrush names

Description

Update acrylic brush names as per #3845
Motivation and Context

Closes #3845
How Has This Been Tested?

Opened a few pages relying on these brushes.
Screenshots (if appropriate):

/azp run


Wrapped scrollback + CMD leads to the commandline being drawn at the top of the window

Canary 1.22.2334.0

just jotting notes as I go

in cmd
ran all the screenbuffer tests, filling the scrollback
as it was running, I scrolled up briefly, then back to the bottom
after it was done, hit up arrow
this is where the commandline got redrawn:

basically I suspect that the mapping from buffer- to VT-viewport-coordinates in cooked read is racy
it's definitely not trivial to reproduce by simply printing text and concurrently scrolling / resizing the window

oh dear, I then hit enter to run the command - it predictably printed starting at the cursor, that makes sense. Problem after that was that another up arrow? brought it back to the weird place from before

oh wait I just remembered that the last this time has happened to me I also ran TAEF tests
HMMMMM probably a bug in my code, but that's sus

THAT IS SUS

Wait I know exactly what happened:

The buffer got resized to 12x12 during one of the tests. This is just /dup #17736


Menu item icon inversion not working anymore

I updated to the latest pebble SDK (Beta10) and also have the latest pebblejs code. Now all item icons are not anymore inverted when selected:

It should look like this:

Thanks for posting! There seems to be an issue with the toolchain causing the inversion in the Basalt Menu Layer not to function, so I implement faux inversion for images with strictly black and white palettes as a work around (see the patch notes for more details).
However, this will only work with images that do not use anti-aliasing. It looks like your power buttons have anti-aliasing, so even the faux inversion won't work for them. You'll want to reduce them to strictly black and white.
If you are willing to develop specifically for Basalt only, completely sidestepping this inversion issue, you would use color and transparency in your icons. Here is an example of how it would look:
![pebble-screenshot_2015-05-11_02-59-20]
(https://cloud.githubusercontent.com/assets/4428900/7562611/9d8efc00-f78a-11e4-84b5-2ed37729d7a3.png)

Thank you the fix worked perfectly! :+1:


Add a few missed instantiation service usages

Missed a few in https://github.com/microsoft/azuredatastudio/pull/17134
Noticed that we never actually used accountForNumberColumn (it was always true) so just removing it

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 3 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 41.479%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/contrib/query/browser/gridPanel.ts
0
1
0.0%

src/sql/workbench/contrib/query/browser/actions.ts
0
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
26892

Relevant Lines:
58801

💛  - Coveralls


Create your portfolio panel and integrate with API

refs #413 (oursky/likedao#108)
Create your portfolio panel and integrate with API created in #437 (oursky/likedao#131)

https://zube.io/files/oursky/fed30c7a0c2f3d8c9b1e6e2a3da94a15-screen-recording-2022-06-16-at-6-07-55-pm.mov


ImprovedChaseCamera 1.5.1 min-max override

provide v 1.5.1 override for 1.0.4 as well as 1.0.3 and .2

Merging with thanks!


Client isn't challenged when authentication is set in map but no Authorization header is sent

I'm facing the situation, that the API client isn't challenged when authorization is required for a service and no Authorization header is given:
my config:
return array(
    'zf-mvc-auth' => array(
        'authentication' => array(
            'map' => array(
                'MyService\\V1' => 'http_digest',
            ),
            'adapters' => array(
                'http_digest' => array(
                    'adapter' => 'ZF\\MvcAuth\\Authentication\\HttpAdapter',
                    'options' => array(
                        'accept_schemes' => array(
                            0 => 'digest',
                        ),
                        'realm' => 'MDM',
                        'digest_domains' => '/my-service',
                        'nonce_timeout' => '3600',
                        'htdigest' => 'data/users.htdigest',
                    ),
                ),
            ),
        ),
        'authorization' => array(
            'deny_by_default' => false,
        ),
    ),
);

Sending the following request, should challange the client, but the server does not include a WWW-Authenticate header in its response:
> GET /my-service/my-action HTTP/1.1
> Host: localhost
> User-Agent: curl/7.43.0
> Accept: application/json
>
< HTTP/1.1 403 Forbidden
< Date: Thu, 24 Sep 2015 15:22:55 GMT
< Server: Apache/2.4.16 (Unix) PHP/5.5.23
< X-Powered-By: PHP/5.5.23
< Vary: Accept-Encoding,User-Agent
< Access-Control-Allow-Headers: x-requested-with, Content-Type, origin, authorization, accept, client-security-token
< Access-Control-Allow-Methods: POST, GET, OPTIONS, DELETE, PUT
< Access-Control-Allow-Origin: *
< Access-Control-Max-Age: 1000
< Content-Length: 119
< Content-Type: application/problem+json

I've identified the following section in ZF\MvcAuth\Authentication\DefaultAuthenticationListener's __invoke function responsible for not challenging the client:
        $type = $this->getTypeFromMap($mvcEvent->getRouteMatch());
        if (false === $type && count($this->adapters) > 1) {
            // Ambiguous situation; no matching type in map, but multiple
            // authentication adapters; return a guest identity.
            $identity = new Identity\GuestIdentity();
            $mvcEvent->setParam('ZF\MvcAuth\Identity', $identity);
            return $identity;
        }

        $type = $type ?: $this->getTypeFromRequest($request);
        if (false === $type) {
            // No authentication type known; trigger any pre-flight actions,
            // and return a guest identity.
            $this->triggerAdapterPreAuth($request, $response);
            $identity = new Identity\GuestIdentity();
            $mvcEvent->setParam('ZF\MvcAuth\Identity', $identity);
            return $identity;
        }

        // Authenticate against first matching adapter
        $identity = $this->authenticate($type, $request, $response, $mvcAuthEvent);

        // If the adapter returns a response instance, return it directly.
        if ($identity instanceof HttpResponse) {
            return $identity;
        }

        // If no identity returned, create a guest identity
        if (! $identity instanceof Identity\IdentityInterface) {
            $identity = new Identity\GuestIdentity();
        }

Changing it like this, the client is at least challenged when no credentials are submitted:
        $type = $this->getTypeFromMap($mvcEvent->getRouteMatch());
        if (false === $type && count($this->adapters) > 1) {
            // Ambiguous situation; no matching type in map, but multiple
            // authentication adapters; return a guest identity.
            $identity = new Identity\GuestIdentity();
            $mvcEvent->setParam('ZF\MvcAuth\Identity', $identity);
            return $identity;
        }

        $typeFromRequest = $this->getTypeFromRequest($request);
        $type = $typeFromRequest ? "{$type}-{$typeFromRequest}" : false;

        if (false === $type) {
            // No authentication type known; trigger any pre-flight actions,
            // and return a guest identity.
            $this->triggerAdapterPreAuth($request, $response);
            $identity = new Identity\GuestIdentity();
            $mvcEvent->setParam('ZF\MvcAuth\Identity', $identity);
            return $identity;
        }

        // Authenticate against first matching adapter
        $identity = $this->authenticate($type, $request, $response, $mvcAuthEvent);

        // If the adapter returns a response instance, return it directly.
        if ($identity instanceof HttpResponse) {
            return $identity;
        }

        // If no identity returned, create a guest identity
        if (! $identity instanceof Identity\IdentityInterface) {
            $identity = new Identity\GuestIdentity();
        }

The solution offered above indeed triggers the Digest challenge, it is not complete though as the response is not an API problem with status 40, but a successful one with data in the body.

This repository has been closed and moved to laminas-api-tools/api-tools-mvc-auth; a new issue has been opened at https://github.com/laminas-api-tools/api-tools-mvc-auth/issues/12.