Sample data generation

Generates sample data to make testing of the app way less tedious:

Realistic first and last names (may repeat)
Realistic salary (higher than minimum wage but not too high, also reasonably round)
Realistic job names
A few realistic department names
Boss and employee with the aforementioned values
Each employee assigned to a boss
Generates more employees than bosses

ID generation was changed as the old one could easily generate duplicate IDs and was needlessly complex.

Conflicts are fixed, code cleaned and nested hierarchy generation implemented.


saver problem

saver.save(sess, os.path.join(model_path, 'model'), global_step=epoch)

File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 1253, in save
{self.saver_def.filename_tensor_name: checkpoint_file})
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 717, in run
run_metadata_ptr)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 915, in _run
feed_dict_string, options, run_metadata)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 965, in _do_run
target_list, options, run_metadata)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 985, in _do_call
raise type(e)(node_def, op, message)
ResourceExhaustedError: ./model/model-49.tempstate5787507096018461664
[[Node: save/save = SaveSlices[T=[DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT,
DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Co
nst_0, save/save/tensor_names, save/save/shapes_and_slices, Wemb, att_W/_2905, att_b/_2907, decode_lstm_W/_2909, decode_lstm_b/_2911, decode_wo
rd_W/_2913, decode_word_b/_2915, hidden_att_W/_2917, image_att_W/_2919, image_encode_W/_2921, init_hidden_W/_2923, init_hidden_b/_2925, init_me
mory_W/_2927, init_memory_b/_2929, lstm_U/_2931, lstm_W/_2933, lstm_b/_2935, pre_att_b/_2937)]]
Caused by op u'save/save', defined at:
File "model_tensorflow.py", line 345, in
train(pretrained_model_path=None)
File "model_tensorflow.py", line 261, in train
saver = tf.train.Saver(max_to_keep=50)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 966, in init
self.build()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 990, in build
restore_sequentially=self._restore_sequentially)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 614, in build
save_tensor = self._AddSaveOps(filename_tensor, saveables)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 294, in _AddSaveOps
save = self.save_op(filename_tensor, saveables)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 240, in save_op
tensor_slices=tensor_slices)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/io_ops.py", line 181, in _save
tensors, name=name)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_io_ops.py", line 438, in _save_slices
data=data, name=name)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/op_def_library.py", line 710, in apply_op
op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2334, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1253, in init
self._traceback = _extract_stack()

Is it possible that you've run out of disk space?

yes, it's out of space

My disk is not out-of-space but I'm also seeing the same error.


Build failing

Hi ,
I'm following this walkthrough in order to get acquainted with AdonisJS but the build fails:
installing dependencies... /npm ERR! Windows_NT 10.0.14393
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program  Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "install"
npm ERR! node v6.5.0
npm ERR! npm  v3.10.3
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-pre-gyp install --fallback-to-build`
npm ERR! Exit status 1`

Any guidance would really help me out.
npm-debug.log.txt

@ScytaleTleilax Simply do npm install --no-optional inside the root of the project.

Oh, of course , silly me. Thanks !

Terribly sorry to reopen
ERR! Tried to download: https://fsevents-binaries.s3-us-west-2.amazonaws.com/v1.0.14/fse-v1.0.14-node-v48-win32-x64.tar.gz node-pre-gyp
ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v48 ABI) (falling back to source compile with node-gyp)
verbose

What is the command you are running to install modules

E:\PROJES\hello-adonis>npm install --no-optional

Trying it on my windows machine. Ideally npm should not try to install optionalDependencies

As reported 16 days ago the --no-optional flag npm issue #8921 still has issues.
I tried with
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Closing and booting my CentOS box since this is a npm issue

As reported 16 days ago the --no-optional flag npm issue #8921 still has issues.
I tried with
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Closing and booting my CentOS box since this is a npm issue


Button's Commit and Rollback are disabled

For production connections, i can choose auto-commit or manual-commit, but button's Commit and Rollback are disabled and i can't control manual-commit.

Since version 4.x commit/rollback become enabled only if you will perform some transaction operation (INSERT/DELETE/UPDATE, DDL, etc).
If you doing such operation and Commit/Rollback are still disabled the it is a bug. Please provide more details.

All right, I did not notice this. But if I make a wrong select and fix the select and try to run again, the following error is displayed:
SQL Error [25P02]: ERROR: current transaction is aborted, commands ignored until end of transaction block
So in the previous version I could use the rollback button and normally execute the select, but now it is not possible. Maybe it's not a bug but it's a drawback.

It is a bug. Or, a kind of miss-function.
This is PostgreSQL specific. Although SELECT query doesn't modify transaction state it is still transactional.
Will be fixed in the hotfix version, thanks for report.
Workaround: execute COMMIT/ROLLBACK query manually (in SQL editor).


Export allvars

allvars gets all the symbols in an expression or formula.
It is a useful function which can meaningfully be extended to a lot of types (like regressionmodels etc). It would be great to export it (I guess it could even be in Base).

+1 But if you export it, maybe add a test for it?
Indeed it could be moved to StatsBase (rather than Base), together with the models infrastructure.

this function no longer exists


Escaping <>

You really should discuss all the ways that <>'s need to be escaped in various situations. For example, in crefs, in normal comment text, in <code> and <c> elements, etc. You examples all conveniently sidestep this constant irritant. The move to DocFX adds additional complexity since it allows markdown in comments, and markdown has its own, often conflicting escaping rules for <>'s.

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

ID: e6f92e76-2785-188b-6046-ec741144fcd0
Version Independent ID: 8708e9f1-a45f-4106-6ba2-bf2e56f76aab
Content: <example> - C# programming guide
Content Source: docs/csharp/programming-guide/xmldoc/example.md
Product: dotnet-csharp
Technology: csharp-fundamentals
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

That's a good point @jo3w4rd
I think this entire section will need to be updated. It was written for different tools than are typically used today.


[OTBN] Feed randomness to the RND CSR/WSR

The CSRNG block provides ports for consumers of random numbers, such as OTBN (or AES). The exact interface isn't defined yet, as is the bandwidth of this interface.
Also add a local LFSR within OTBN if needed.

estimate 16

Depends on https://github.com/lowRISC/opentitan/issues/2693

EDN hookup is here: https://github.com/lowRISC/opentitan/pull/5286, discussions around how EDN bits are accessed via OTBN (including adding an LFSR) are ongoing.

Related issues are #3522 and #4209

RND implementation is here https://github.com/lowRISC/opentitan/pull/5470

Implemented in #5470


Remap ROV:pending-verification upon verification

Coordinate with @hohonuuli first.

Another thought. Instead of putting the verifier in the json, it might be better to just add a verified association to the observation: e.g verified | self | <verifier name>.

We would, of course, still need to change <platform>:pending-verification to <platform>

add a verified association to the observation

The tricky part here is that the verification is tied to the bounding box, of which there may be many for a single observation. I know that in practice, we are almost always doing 1 observation to 1 bounding box association, but that's not always the case (e.g. parts). I guess the question is: what do we want the verification to represent? Just the ID (observation), or also the bounding box coordinates (association)?

we are almost always doing 1 observation to 1 bounding box association, but that's not always the case (e.g. parts).

My 2 cents is that any non-one-to-one relations should be considered a legacy usage at this point. Going forward we will avoid that.
Parts in VARS have been annotated like using a part-of association. So this is would be the canonical usage
nectophore
|- part-of      | Nanomia | nil
`- bounding box | self    | { ... }

I guess the question is: what do we want the verification to represent?

Yeah. That's a discussion to have in the video lab. The simplest (an hence most likely case) is:

If there's no bounding box. A verification means the observation has been verified.
If there is a bounding box. A verification means both the observation and box are verified.


Allow interrupting blocking instructions

Implementing #1910

I tested different combinations of blocking from main thread or from spawn thread, interpreter or aot, with or without hw bound check enabled.
So far the combination that doesn't seem to work is aot +  hw bound check disabled, where I see that the exit code is not the one expected. I'll keep investigating.

I tested different combinations of blocking from main thread or from spawn thread, interpreter or aot, with or without hw bound check enabled. So far the combination that doesn't seem to work is aot + hw bound check disabled, where I see that the exit code is not the one expected. I'll keep investigating. @wenyongh

@eloparco Thanks a lot, the code is much better now, I will look into it soon.

Test this feature with branch dev/interrupt_block_insn, and PR #1946, #1948 were merged.


Vote to boot player

In Warfish you could vote to boot a player, or the creator can boot someone after 14 days of not taking their turn

maybe after X number of days or Y number of nudges, the player's turn is skipped

https://spqr-app.slack.com/archives/C8B1D9AJH/p1725659872156069?thread_ts=1725658649.500429&cid=C8B1D9AJH


Add "external" serde support to certificate chain types

Add serde support for CA Certificates
Derive serde support for CA Chains
Add serde support for SEV Certificates
Derive serde support for SEV Chains
Derive serde support for full certificate chain
Derive serde support for launch::Start

I've concluded my testing with serde_bytes and it does work. I've removed the hand-rolled serialize_seqs.

I've concluded my testing with serde_bytes and it does work. I've removed the hand-rolled serialize_seqs.


Refactor the deep tile matmul config and skip the single-iteration loop generation

Track: https://github.com/intel/graph-compiler/issues/288

The extra memref.copy is caused by write-after-write conflict because the canonicalization pass eliminates the single-iteration loop but preserves the extracted slice and insert slice. So here skip generating them when they are single-iteration loop.
Refactor the config. Infer dimType according to the contractionOpInterface.
introduce padding cost, which minimizes the cost on padding and use divisible block if possible

Overall LGTM, do you think we should add a single simple case to cover the case of skipping single-iteration loop generation?

Overall LGTM, do you think we should add a single simple case to cover the case of skipping single-iteration loop generation?

This test could cover this case. The file check will check two scf.forall before this PR but the single-iteration scf.forall will be skipped now.
https://github.com/intel/graph-compiler/blob/613a333ee5e14731feeca40b54c57adb07ca7edc/test/mlir/test/gc/Transforms/deepTileContractionNamedOp.mlir#L127

Please help to rebase the code base, then we can merge it.

Please help to rebase the code base, then we can merge it.

The code has been rebased now


[BUG] Error from cached load() is passed to the createAsync result as value

Describe the bug
May be related to #374
When the function that is passed to the routes load() function throws an error or rejected promise, the error will be passed to the result primitive as a "successful" value if the page is refreshed and then clicked on a link to the current page (please check the video).
Your Example Website or App
https://codesandbox.io/p/devbox/black-forest-rvp7xj?workspaceId=55b47a62-ca2d-456f-9cef-0a44c292a811&layout=%257B%2522sidebarPanel%2522%253A%2522EXPLORER%2522%252C%2522rootPanelGroup%2522%253A%257B%2522direction%2522%253A%2522horizontal%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522id%2522%253A%2522ROOT_LAYOUT%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522clt62eq4r00053b6kwmsynsmf%2522%252C%2522sizes%2522%253A%255B70%252C30%255D%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522EDITOR%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522id%2522%253A%2522clt62eq4r00023b6kr57gddyu%2522%257D%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522SHELLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522id%2522%253A%2522clt62eq4r00033b6kltnrrsbo%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522DEVTOOLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522id%2522%253A%2522clt62eq4r00043b6kkthi0q6d%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%252C%2522sizes%2522%253A%255B50%252C50%255D%257D%252C%2522tabbedPanels%2522%253A%257B%2522clt62eq4r00023b6kr57gddyu%2522%253A%257B%2522id%2522%253A%2522clt62eq4r00023b6kr57gddyu%2522%252C%2522tabs%2522%253A%255B%255D%257D%252C%2522clt62eq4r00043b6kkthi0q6d%2522%253A%257B%2522id%2522%253A%2522clt62eq4r00043b6kkthi0q6d%2522%252C%2522tabs%2522%253A%255B%257B%2522type%2522%253A%2522UNASSIGNED_PORT%2522%252C%2522port%2522%253A3000%252C%2522id%2522%253A%2522clt63ch5v012g3b6k2nzu1onr%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522path%2522%253A%2522%252Ftest%2522%257D%255D%252C%2522activeTabId%2522%253A%2522clt63ch5v012g3b6k2nzu1onr%2522%257D%252C%2522clt62eq4r00033b6kltnrrsbo%2522%253A%257B%2522id%2522%253A%2522clt62eq4r00033b6kltnrrsbo%2522%252C%2522tabs%2522%253A%255B%257B%2522id%2522%253A%2522clt635se200ee3b6kwkx7pgtz%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522TERMINAL%2522%252C%2522shellId%2522%253A%2522clt635sgp000ydjgrbgyw9chy%2522%257D%255D%252C%2522activeTabId%2522%253A%2522clt635se200ee3b6kwkx7pgtz%2522%257D%257D%252C%2522showDevtools%2522%253Atrue%252C%2522showShells%2522%253Atrue%252C%2522showSidebar%2522%253Atrue%252C%2522sidebarPanelSize%2522%253A15%257D
Steps to Reproduce the Bug or Issue

Open code sandbox
Navigate between test and home page -->all good. Errorboundary is rendered
Go to test page
refresh the window
click on test navlink --> page is rendered with the data primitive containing the error string !?

Expected behavior
The error should always be picked up by the ErrorBoundary and never passed to the primitive returned by the createAsync function.
Screenshots or Videos
https://github.com/solidjs/solid-router/assets/59310516/d855df9c-35a2-4d46-b387-7c64a1134f56
Platform
@solidjs/router:   0.12.4
@solidjs/start:  0.6.0
Additional context
No response

Yeah this one is odd.. codesandbox is not being particularly helpful to me because they have a limit now on drafts unless I pay so I can't play with this much other than confirm the reproduction and that it isn't the same as the other issue. Beyond that this one is very specific to loading on the page... so there probably is an issue related to cache + hydration. I will try to look into this more in a bit.

I added a stackblitz - here the error is on the home page because stackblitz reloads o on the home route when reloading the url.
Added a Github repo so you can dig into it. Thanks a lot and thanks for this amazing framework.
It is such a pleasure to work with SolidJs.

in 1.0 solid-start this seems to still be around:
I see other types like redirects being returned to createAsync causing it to blow up on exceptions as well.  Just throw a redirect from a server function (not wrapped in cache) that is received in createAsync and it blows up on the result type not being a string or something it recognizes.
async function serverApiCallThrow(isIndirect?: boolean): Promise<string> {
    'use server';
    throw redirect(`/${isIndirect ? 'fromIndirect' : 'fromThrow'}/whateva?q=456`);
    return 'ok'
}

...
  const asyncThing = createAsync(()=>serverApiCallThrow())

...
  <Show when={asyncThing()}>

boo!

  </Show>

results in exception:

Error: Unknown error at castError (...server bundle...)

and client shows:


can't create /var/lib/homebridge

Hi There,
I tried to use this iso on a vmware fusion player v. 12.1.0 with a mac os bug sur v. 11.12.1 and I get an error :
'can't create /var/lib/homebridge'.
The vm settings is Other linux 2.4 Kernel 64 bit, Bridged network, 1 cpu and 1GB Ram.
Any help would be appreciate

.

Did you attach a virtual hard disk?

yes 8GB

change disk controller bus type from SCSI to SATA, that worked for me having the same issue.

change disk controller bus type from SCSI to SATA, that worked for me having the same issue.

That did the trick. thank you so much.

Changed the ESXi wanting note in the README to a generic note:

Important: Use a SATA / IDE Controller (SCSI controllers will not work)


Detect when "files were modified by this hook"

Some hooks such as yapf, docformatter, or eslint can either return a non-0 exit code or modify files in place.
yapf -rd . # return non-0 exit code
yapf -ri . # modifies in place
yapf -rid . # 'i' and 'd' are incompatible options.

For each hook, it would be great to use the "modify in-place" option and detect if any files have been modified by this hook (by checking git diff at the end of each hook for instance) in order to either:

say the hook has failed and manually git add the difference
or automatically git add the difference to the current commit (assuming we have a strong trust in our hooks)

This is fairly opinionated, but generally speaking, I would consider that a program modifying the code base during a hook is a bad practice, for the following reasons :

The hook user does not control the changes being performed, more specifically, he can not review them easily since they are added to the staged files
I do think that hooks are just here for enforcement, and should not do the work that should have been done during development, whether it be running a linter wit --fix or have a editor properly configured. Actually, hooks, when used only for changes are terribly efficient for detecting environment misconfigurations.

Moreover, I don't think adding flags to the command are good enables for features, because the hooks are invoked through the git hooks scripts, and this part is automatically written for the user in the init command, thus should be not have to be modified after.
On the other side
I do agree, that fixing easily failing hooks is a big part of the comfort of use, hence I would propose the following modification :

We add a fix option in the hook steps configuration, which would be a command to run to fix any failing hooks.
When a hook step fails, if there is a fix attribute in the step configuration, it is put into evidence for the user.

What about this ?


chore(package): update eslint-config-pretty to version 2.1.1

Fix #6

Codecov Report

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

@@          Coverage Diff          @@
##           master     #7   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           8      8
  Lines          83     83
=====================================
  Hits           83     83

Continue to review full report at Codecov.

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


Correct a typo: lenght -> length

Just a small typo correction because I saw it :D
Second patch solves the issue #14, and seems to work OK on linux too

Thank you very much for this! I love everything but there's one thing I'd like to change before I'll merge: https://github.com/renaudallard/fiche/commit/d771d7bdfd3953a18e40734cd83a207b133a18c5. Statistically, collision likeliness may seem to be pretty big, but I got some empirical data that shows that with current service usage it barely never happens. For last month, users pasted 1806 pieces of text and there's only one paste that has 5 characters name (and since it actually contains nothing but noise, I can point it here, if you're curious: http://termbin.com/jupu6 :) ). I'd like also to keep old set of characters, at least in the main branch ("abcdefghijklmnopqrstuvwxyz0123456789"), it's just easier to remember or retype url on second terminal. I think that maybe the best option would be to leave that choice to end user and add appropriate options to getopt: https://github.com/solusipse/fiche/blob/master/fiche.c#L367. What do you think about this? Would you like me to merge it now and add this functionality, or would you like to write this one also? :)

Yes, it might be a good idea to add the extended charset to either makefile or getopt. Maybe merge everything except d771d7b. I will have a look at providing an option for it.

I have added the -e option for using the extended charset

Thank you!


KeyringError::NoPasswordFound is not used on MacOS

On MacOS, if I try to get a nonexistent password, instead of getting an instance of KeyringError::NoPasswordFound I get Mac Os Keychain Error: The specified item could not be found in the keychain..
The macos.rs file shows no uses of NoPasswordFound.

This is being addressed by #70.  Stay tuned!


Rename orchestrator traits to be verbs

This PR makes the orchestrator traits more idiomatic by making them verbs.
Checklist

[ ] I have updated CHANGELOG.next.toml if I made changes to the smithy-rs codegen or runtime crates
[ ] I have updated CHANGELOG.next.toml if I made changes to the AWS SDK, generated SDK code, or SDK runtime crates

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I didn't rename HttpClient, HttpConnector, RuntimePlugin, AuthScheme, or RetryStrategy since I couldn't think of good names for these, and was worried alternate names for them might make things less clear.


Add sniff for unprepared SQL

It flags any calls to $wpdb->query() and related functions that
incorporate variables, if they aren’t properly escaped using
$wpdb->prepare().
Fixes #20

@JDGrimes Are you serious!? :fireworks: :astonished: :heart:

:sunglasses:


Chanel API to select values

I'm looking for API that can remove any value from the channel based on user select. In other words I need a channel that is FIFO, but allows any value to removed if needed. For example I would like to be able to handle values with a higher priority before regular values.
For this purpose I though of the following design. It loops over all values currently in the channel allowing the user to select the value to receive next, e.g. the one with the highest priority. The user returns some type that indicates what message to remove, using that type the message is removed and returned to the user.
impl<T> Recevier<T> {
    /// Attempt to select a message to remove from the channel.
    pub fn try_select<S>(&mut self, mut selector: S) -> Option<T>
    where
        S: ValueSelector<T>,
    {
        selector.select(Values { recv: self })
            .map(|selection| { /* remove and return selection. */ })
    }
}

pub trait ValueSelector<M> {
    /// Select what value to receive.
    ///
    /// User looks over all values in `Values` and returns a `ValueSelection`, that
    /// message is removed from the channel.
    fn select<'m>(&mut self, values: Values<'m, M>) -> Option<ValueSelection>;
}

/// Iterates over all values currently in the channel.
#[derive(Debug)]
pub struct Values<'r, M> {
    recv: &'r mut Receiver<M>,
}

impl<'r, T> Iterator for Values<'m, T> {
    type Item = (MessageSelection, &'r T);

    fn next(&mut self) -> Option<Self::Item> {
        // Loop over all values in the channel, by reference.
    }
}

/// The type used to indicate what message to select.
#[derive(Copy, Clone, Debug)]
pub struct MessageSelection(/* usize ? */);

@stjepang do you have any feedback whether or not something like this would be consider acceptable for crossbeam's channel?

@Thomasdezeeuw It might be best to try prototyping this in a new crate first. Does the implementation support it? And what happens to other channel send/receive operations while a caller is examining the values in a channel? It seems inherent racy to me.

@BurntSushi I was initially looking for some feedback on whether this would be considered this crate, but I'll work on it in another crate first. I don't think the send operations will be a problem, but I might have forgotten that there can be multiple receivers, that could be a problem indeed.


Support for Zabbix 5.x

5.0 is now out and should be supported!

@Splyth @snehitgajjar Is there anything i can do to help this along? I'm currently using 4.1.2 on Zabbix 5.0.3. (Through
puppet-zabbix)
4.2.0 seems to break this module on any unsupported zabbix version, when previous versions only warned. Making 5.x support even more important now imo. https://github.com/express42/zabbixapi/commit/add21b692b34acaa4ddb568fa13b27a52adf2856#diff-e4c256e6a91f4de6d474565ff8c53dc9R65

Sorry I'm swamped six ways from Sunday.
Honestly what needs to happen is this in my view:

Bump Version
Make Changes
Add/Update Tests

Tests are currently passing and we're trying to keep them that way going forward.
If you have any specific question I'll try to answer. It's going to be a long while before I can get to it.

@aclarkee I'm willing to help as well - this library is too important to just let die like this.

I haven't been able to give this gem the time and attention it deserves and the way things are looking I won't be able to. @aclarkee @svdasein if you'd like I'm more than happy to turn it over to either one of you.

@Splyth I'm willing to get involved. I'm going to confess up front that I have much to learn vis a vis the CI stuff, but I am willing to learn it.  Am I right in thinking that the path forward here is to upgrade the docker test environment to a 5.x and then hammer at the code 'til all the tests pass (and then add new stuff + tests if required)?

@svdasein that's about the long and short of it yeah. You can find my email on my profile and I can work on getting stuff transferred over to you.

I'm also willing to help but I'm in the same camp as @svdasein and not up to speed on all the CI hotness.

@jameskirsop I'm a contributor now - just starting.  Def. could use some help.  Where can I find your email addr?

initial support for Zabbix 5.x (tested with 5.2.2): https://github.com/anapsix/zabbixapi/tree/zabbix-5

@anapsix do you know how to work all the ci stuff in this project? There aren't any dox for it and I'm really kind of out of my element.  If you'd like I can see if I can get you added here.

@svdasein do you mean Rspec, or Travis CI stuff?
Travis recently made it harder for OSS projects to get credits, so it might be worth considering move to Github Actions..
Either way, I'm familiar enough with Travis CI and Github Actions to update CI job with Zabbix 5.x or recreate the entire thing in Gitlab Actions. Though, I'd switch to using Zabbix Docker images, rather than installing Zabbix packages on Ubuntu.

@anapsix ok - I need to get you added as a contributor - you actually know how to work this stuff.  Lemme see if I can figure out who to ping here (I'm pretty new myself).

alrighty..

made a Gitlab Action based CI: https://github.com/anapsix/zabbixapi/actions?query=workflow%3ACI
here's the workflow definition file: https://github.com/anapsix/zabbixapi/blob/zabbix-5/.github/workflows/main.yml

Now that I've got access to the repo, I've pushed zabbix5.2-alpha1 branch here.
Since folks are eager to use zabbixapi gem with latest version of Zabbix, how about we publish a pre-release version 5.2.0-alpha1 to RubyGems based on zabbix5.2-alpha1?
There is still plenty of work to be done, including updating documentation, adding support for previously unsupported API objects (such as problem), etc.. But at very least, folks can start using previously supported API.

Now that I've got access to the repo, I've pushed zabbix5.2-alpha1 branch here.
Since folks are eager to use zabbixapi gem with latest version of Zabbix, how about we publish a pre-release version 5.2.0-alpha1 to RubyGems based on zabbix5.2-alpha1?
There is still plenty of work to be done, including updating documentation, adding support for previously unsupported API objects (such as problem), etc.. But at very least, folks can start using previously supported API.

Nice! We definitely need this gem for puppet-zabbix so pushing it to RubyGems could help us.
@Splyth (sorry for the ping 😄 ) could you possibly add as maintainer @anapsix so they can push to RubyGems?
Thank you!

Nice! We definitely need this gem for puppet-zabbix so pushing it to RubyGems could help us.
@Splyth (sorry for the ping 😄 ) could you possibly add as maintainer @anapsix so they can push to RubyGems?
Thank you!

Alternatively, as @root-expert suggested, we could setup a Github Action to push to Rubygems based on tag pattern v5.2**.
Couple of pre-made actions are available for that:

https://github.com/marketplace/actions/publish-to-rubygems
https://github.com/marketplace/actions/publish-gem

Alternatively, as @root-expert suggested, we could setup a Github Action to push to Rubygems based on tag pattern v5.2**.
Couple of pre-made actions are available for that:

https://github.com/marketplace/actions/publish-to-rubygems
https://github.com/marketplace/actions/publish-gem

UPDATE: I got access to RubyGems for zabbixapi.. will hook up the Github Action job to push based on branch / tag soon

UPDATE: I got access to RubyGems for zabbixapi.. will hook up the Github Action job to push based on branch / tag soon

Nice!

Nice!

I do not have enough permissions on this repo to add secrets for CI job..
.. and exposing a personal API token in public project's repo might not be as good of an idea
I've pushed zabbixapi 5.0.0-alpha1 to Rubygems maually: https://rubygems.org/gems/zabbixapi/versions/5.0.0.pre.alpha1
Install it via
gem install zabbixapi -v 5.0.0-alpha1
# or
gem install zabbixapi -v 5.0.0.pre.alpha1

Or in Gemfile
gem 'zabbixapi', '~> 5.0.0-alpha1
# or
gem 'zabbixapi', '~> 5.0.0.pre.alpha1

I do not have enough permissions on this repo to add secrets for CI job..
.. and exposing a personal API token in public project's repo might not be as good of an idea
I've pushed zabbixapi 5.0.0-alpha1 to Rubygems maually: https://rubygems.org/gems/zabbixapi/versions/5.0.0.pre.alpha1
Install it via
gem install zabbixapi -v 5.0.0-alpha1
# or
gem install zabbixapi -v 5.0.0.pre.alpha1

Or in Gemfile
gem 'zabbixapi', '~> 5.0.0-alpha1
# or
gem 'zabbixapi', '~> 5.0.0.pre.alpha1

It seems that 5.0.0-alpha1 only supports API versions 5.0 and 5.2. Are there breaking changes in later versions, such as 5.4?

You can use ignore_version to get this gem working on 5.4. Works fine for my purposes.
Breaking changes here (Only thing that affected me was the inability to read PSK values)
https://www.zabbix.com/documentation/current/manual/api/changes_5.2_-_5.4

What's left to do before the official 5.x release can be published?


[CIT-339] Use correct settings for azure conenct command.

Proposed changes
Use azure URL settings instead of c8y in tedge connect az.
Types of changes
What types of changes does your code introduce to thin-edge.io?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA (in all commits with git commit -s)
[x] I ran cargo fmt as mentioned in CODING_GUIDELINES
[x] I used cargo clippy as mentioned in CODING_GUIDELINES
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Unfortunately this is not easily testable, but maybe could make it integration tested?

Codecov Report

Merging #208 (721da5c) into main (df0a84c) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #208   +/-   ##
=======================================
  Coverage   61.97%   61.97%
=======================================
  Files          50       50
  Lines        2719     2719
=======================================
  Hits         1685     1685
  Misses       1034     1034

Impacted Files
Coverage Δ

tedge/src/cli/connect/command.rs
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Upgrade to Angular v12?

Hello everyone,
Do you have any plans to upgrade the @agm/core library to Angular v12? I'm using it in my personal project and I can't upgrade right now because it has an incompatible peer dependencies.

The same situation. Install through --force, but the project is not build
`Error: node_modules/@agm/core/lib/services/managers/rectangle-manager.d.ts:17:50 - error TS2503: Cannot find namespace 'google'.
17     setOptions(rectangle: AgmRectangle, options: google.maps.RectangleOptions): Promise;
~~~~~~
Error: node_modules/@agm/core/lib/services/managers/rectangle-manager.d.ts:18:49 - error TS2503: Cannot find namespace 'google'.
18     getBounds(rectangle: AgmRectangle): Promise<google.maps.LatLngBounds>;
~~~~~~
Error: node_modules/@agm/core/lib/utils/mvcarray-utils.d.ts:1:23 - error TS2688: Cannot find type definition file for 'googlemaps'.
1 ///
~~~~~~~~~~
Error: node_modules/@agm/core/lib/utils/mvcarray-utils.d.ts:3:60 - error TS2503: Cannot find namespace 'google'.
3 export declare function createMVCEventObservable(array: google.maps.MVCArray): Observable<MVCEvent>;
~~~~~~
Error: node_modules/@agm/core/lib/utils/mvcarray-utils.d.ts:11:49 - error TS2503: Cannot find namespace 'google'.
11 export declare class MvcArrayMock implements google.maps.MVCArray {
~~~~~~
Error: node_modules/@agm/core/lib/utils/mvcarray-utils.d.ts:24:102 - error TS2503: Cannot find namespace 'google'.
24     addListener(eventName: 'remove_at' | 'insert_at' | 'set_at', handler: (...args: any[]) => void): google.maps.MapsEventListener;
`

@vredka Hi.
I just ran npm install --save @agm/core agm-direction --force successfully on my project.
Some details:
angular 12.0.3
node 14.15.1
npm 7.15.0

@kinggolf you don’t get any errors when running the build command?
I have the same specs, except node v14.17, and it installs successfully with —force but then errors out when I try running the build command.

No.

Sorry, just re-read your messages. Yes I get same error when trying to build.

I wonder is this due to strict mode.

so agm is not working in angular 12?

Instead of waiting and asking, you should start using @angular/google-maps library provided by the Angular team. Support is very good, community is very big and migration from this library to Angular wrapper library is very easy.
https://github.com/angular/components/tree/master/src/google-maps#readme

Hi AGM team,
Is there any development upon the compatibility issue with angular v12 ?
or Is the team planning to release new version of agm which will resolve the compatibility issue?

Also having trouble with this library, it still references invalid types even after installing @types/googlemaps and doesn't seem tested in recent versions of Angular. Will try with @angular/google-maps.

Had the same error on my Angular 12 project but it worked after downgrading @types/googlemaps.
"dependencies": {
    "@agm/core": "^3.0.0-beta.0",
},
"devDependencies": {
    "@types/googlemaps": "^3.39.12",
}

@tapestephan
For me worked exact version:
"devDependencies": {     "@types/googlemaps": "^3.39.12", }


Issue 401: Adding Info Icon to Priority Title in Map Legend

https://github.com/CodeForPhilly/vacant-lots-proj/issues/401
Added the Phosphor Info Icon to the map legend. Had to hardcode it in CSS styling to get the specific icon from Phosphor Icons.

@mskNeo your PR covers the scope of this ticket, but for a more general UI/UX question: @thansidwell should hovering/clicking this icon show a tooltip? Just having an un-interactable 'i' icon feels like it may lead to some confusion, so if a tooltip should be added, I suggest we hold this PR as a draft until the tooltip is ready to be added.

@paulhchoi Yes, we had intended when a user hovers over the icon a tooltip would show up explaining how we determine priority. We still don't have that written yet.

I can try implementing a tooltip, but do we have any specific UI mockups/vision for it? @thansidwell

@mskNeo That would be great if you could. There are a few other places we are using it. We are using the NextUI component library. Here's their tooltip. It's very close to what I designed.  https://nextui.org/docs/components/tooltip
Here's the documentation of what it should look like. https://www.figma.com/file/NAFkgq34abW6uJ0R7PW24T/Prototype---Clean-%26-Green-Philly?type=design&node-id=1435%3A8475&mode=design&t=rwvoZG2F2y3A2hfM-1
@brandonfcohen1 @paulhchoi Do we have a tooltip already?

@mskNeo That would be great if you could. There are a few other places we are using it. We are using the NextUI component library. Here's their tooltip. It's very close to what I designed. https://nextui.org/docs/components/tooltip
Here's the documentation of what it should look like. https://www.figma.com/file/NAFkgq34abW6uJ0R7PW24T/Prototype---Clean-%26-Green-Philly?type=design&node-id=1435%3A8475&mode=design&t=rwvoZG2F2y3A2hfM-1
@brandonfcohen1 @paulhchoi Do we have a tooltip already?

yeah we've been using nextui tooltip

Hey @mskNeo ! When you add the NextUI tooltip, could you please add 2 quick a11y things?

Make the info icon keyboard accessible. So you should be able to TAB to the info icon, focus on it, and the tooltip will appear. Most likely all you need is to add a tabIndex="0" to the icon.
Add an alt text to the icon.

Here is a previous PR where I added these things https://github.com/CodeForPhilly/vacant-lots-proj/pull/350
Let me know if you have any questions!

@bacitracin it seems like the map legend is autogenerated so adding the NextJS tooltip won't work since the legend elements are not in the codebase but rather rendered by the library. I'll try making a custom legend. @thansidwell are there any features of the legend as it is now that are being used? like collapsing it or anything?

@bacitracin it seems like the map legend is autogenerated so adding the NextJS tooltip won't work since the legend elements are not in the codebase but rather rendered by the library. That's why I used a CSS approach to add the info icon. I'll try making a custom legend. @thansidwell are there any features of the legend as it is now that are being used? like collapsing it or anything?

+1 on writing a custom legend

@mskNeo deployment is failing- looks like you need to merge in the latest staging

@brandonfcohen1  I synced my forked repo with staging and merged but it's still failing it seems

@mskNeo I noticed we have two different style tooltips. On the live site and your tooltip. Are you using a different tooltip than what's used on the 'Download' and 'View' buttons? And both don't match the prototype.

We should just be using one toolip component and it should look like the prototype. https://www.figma.com/file/NAFkgq34abW6uJ0R7PW24T/Prototype---Clean-%26-Green-Philly?type=design&node-id=1435%3A8475&mode=design&t=in6whOv1xRCIDRgL-1
Can you fix that on this ticket or should I just write a new one?

@mskNeo deployment is failing- looks like you need to merge in the latest staging

@brandonfcohen1 I synced my forked repo with staging and merged but it's still failing it seems

here is the error- are you sure you have the latest staging? you can run npm run build locally to test the build before deploying

Failed to compile.
./src/components/PropertyMap.tsx:41:23
Type error: Cannot find module './ZoomModal' or its corresponding type declarations.
  39 | import MapboxGeocoder from "@mapbox/mapbox-gl-geocoder";
  40 | import<EMAIL_ADDRESS>> 41 | import ZoomModal from "./ZoomModal";
     |                       ^
  42 | import { Coordinates } from "../app/types";
  43 | import { MapLegendControl } from "./MapLegendControl";
  44 | import { createPortal } from "react-dom";
Error: Command "npm run build" exited with 1

@brandonfcohen1 My forked repository for some reason wasn't syncing properly with the staging branch, thankfully my main branch did so I had to manually check out the missing files. Hopefully that works

legend looks great. Will let @thansidwell approve the UI
Is it intended that clicking the i icon minimizes the legend?

legend looks great. Will let @thansidwell approve the UI
Is it intended that clicking the i icon minimizes the legend?

@brandonfcohen1 clicking the upper bar of the legend in general minimizes, not just the I. So clicking on the empty space to the right of it closes it too. This is the same behavior as the library legend that we had before.

legend looks great. Will let @thansidwell approve the UI
Is it intended that clicking the i icon minimizes the legend?

@brandonfcohen1 clicking the upper bar of the legend in general minimizes, not just the I. So clicking on the empty space to the right of it closes it too. This is the same behavior as the library legend that we had before.

haha nice, had never clicked that before

It looks good to me!
I'll write another ticket to make further styling updates to the tooltip.


The accuracy indicators during the test are very unusual？

Dear Author, I downloaded some data samples of similar type from some website and made a large dataset (more than 100,000 samples) by myself according to the type of dataset in Ojha_CVPR2023 and split the samples into train, test and val in the ratio of 0.8, 0.1 and 0.1.
When I use your method to train and test using the clip model, the results for train and val are good, with acc and ap above 92%, but the results for test are consistently poor, with acc a little over 70% and ap less than 90%.
I don't know what's wrong with it, could you please tell me what could be the reason? Is it because your test _augment has different transforms and data enhancement methods than train_augment and val_augment? I am really confused and would appreciate your answer.

I downloaded some data samples of similar type from some website and made a large dataset (more than 100,000 samples) by myself according to the type of dataset in Ojha_CVPR2023 and split the samples into train, test and val in the ratio of 0.8, 0.1 and 0.1.

The training conducted in our work adheres to the method outlined by Wang et al, comprising 720,000 real and fake images for deepfake detection. Personally, I do not anticipate its effectiveness on smaller datasets since the primary goal of the work focuses on generalization. Training on a large-scale dataset is a significant prerequisite for achieving this.

When I use your method to train and test using the clip model, the results for train and val are good, with acc and ap above 92%, but the results for test are consistently poor, with acc a little over 70% and ap less than 90%.

Accuracy is the metric used in this work, we focused (as well as other papers in this topic) on average precision.

I don't know what's wrong with it, could you please tell me what could be the reason? Is it because your test _augment has different transforms and data enhancement methods than train_augment and val_augment? I am really confused and would appreciate your answer.

Not on the aug used (train, val, and test), we used the same aug from Wang et al, [Gragnaniello et al., and Ojha et al..


feat: support block id for provider get_code() [APE-1203]

What I did
fixes: #1540
How I did it

add missing support
add block id alias

How to verify it

Checklist

[ ] All changes are completed
[ ] New test cases have been added
[ ] Documentation has been updated

Works for now. I was actually hoping to make it an explicit kwarg to use in that method via block_id but that can come at a breaking change

If you want this in 0.7, make sure it is in the 0.7 project (or tag, i forget) on linear and itll for sure get in.


⚠ Processors has degraded performance

In dfc9086, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 64 ms

Resolved: Processors performance has improved in 89d925d.


Implement app_permissions

Checklist
This pull request...

[ ] Fixes a bug
[x] Adds new functionality
[ ] Updates documentation and/or examples
[ ] Adds tests
[ ] Is a breaking change -- existing code written for this library may need to be rewritten to work after these changes.

Description
Today discord introduced a new field to interactions: app_permissions. This field contains the permissions an app has in a guild.
If the app has a bot in the guild, it's the bot's permissions, if not then it will be the permissions of @everyone.
For more info, you can read the chancelog message and the corresponding discord announcement.

Associated discord pull request:
discord/discord-api-docs#5131

lgtm 👍


Correction for raw affiliation Centre de Recherche en Paléontologie – Paris (CR2P)

Correction needed for raw affiliation Centre de Recherche en Paléontologie – Paris (CR2P)
raw_affiliation_name: Centre de Recherche en Paléontologie – Paris (CR2P)
new_rors: ;05ax2x637
previous_rors:
works_examples: W4388470107
contact: 075ef18a252884eb414a37dd673a13f6:b4944524233c2e2d9163d89bb1 @ sorbonne-universite.fr

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


Fuze update requested

CoreOS: 1409.7.0
matchbox: v0.6.1
Can ignition/Fuze (container-linux-config-transpiler) be vendored up to include the following abilities?
https://github.com/coreos/container-linux-config-transpiler/commit/d9eb71c2c77f6269abbc6d15a16fed9b45faddf1#diff-55746646f84e39324ac72516ebd056fd
Including the fix for etcd_endpoints: https://github.com/coreos/container-linux-config-transpiler/commit/873cfedffbf077d3d02330c2b6edb62ca935d92d
I just spent far too long trying to figure out why the following wasn't be respected by the transpiler:
locksmith:
  etcd_endpoints: "http://example.com:2379"

Even though it is documented that it will:
https://coreos.com/os/docs/1409.7.0/configuration.html
Thanks!

Dup #606


feat: zig_code_path quality-of-life improvements

When I first tried zig_code_path, I tried using Path.join in the use opts.
This raised and I settled for a relative path (which I realize is the expected
code path).
Even if this change is not accepted, we can raise in a more informative way
in the new if is_binary(path) conditional.
Also, the put_attribute call makes it so that the code recompiles whenever
the zig source changes.

There's something I didn't find an easy fix for as well: If you pass zig_code_path, Zigler ends up creating separate .{module}.zig files, and all "go to file" links end up taking to that file.
This in turn were, multiple times, the cause for me having to apply a fix again because I applied it to the separate file instead of my source one.


Unclear error message when running plan without parameters

When the controller run subcommand is executed without providing any parameters (-p), the following is returned:
$ blueapi controller run scan
Traceback (most recent call last):
  File "/scratch/30day_tmp/delete15may/bs/blueapi/venv/bin/blueapi", line 8, in <module>
    sys.exit(main())
  File "/scratch/30day_tmp/delete15may/bs/blueapi/venv/lib/python3.10/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/scratch/30day_tmp/delete15may/bs/blueapi/venv/lib/python3.10/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/scratch/30day_tmp/delete15may/bs/blueapi/venv/lib/python3.10/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/scratch/30day_tmp/delete15may/bs/blueapi/venv/lib/python3.10/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/scratch/30day_tmp/delete15may/bs/blueapi/venv/lib/python3.10/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/scratch/30day_tmp/delete15may/bs/blueapi/venv/lib/python3.10/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/scratch/30day_tmp/delete15may/bs/blueapi/src/blueapi/cli/cli.py", line 67, in wrapper
    func(*args, **kwargs)
  File "/scratch/30day_tmp/delete15may/bs/blueapi/venv/lib/python3.10/site-packages/click/decorators.py", line 38, in new_func
    return f(get_current_context().obj, *args, **kwargs)
  File "/scratch/30day_tmp/delete15may/bs/blueapi/src/blueapi/cli/cli.py", line 106, in run_plan
    json=json.loads(parameters),
  File "/dls_sw/apps/python/miniforge/4.10.0-0/envs/python3.10/lib/python3.10/json/__init__.py", line 339, in loads
    raise TypeError(f'the JSON object must be str, bytes or bytearray, '
TypeError: the JSON object must be str, bytes or bytearray, not NoneType

This is not particularly user friendly, if parameters are always required, can this be made into a mandatory argument?

This is now a 500 internal server error from a client perspective, which isn't much better

I believe so, and agreed, and also the CLI should translate HTTP codes into human-readable error messages

debugging the error in parallel server and client side,
here the server

here the client

at the moment the cli raises an error only in two cases:

yet the error text is about a pickle operation, so a different one from the May 2023 original post. I cannot be sure if this is an issue with setup on my end or just natural changing of the repo

The multiprocessing/pickling errors mean it's probably related to #343 if that helps narrow down why it's failing. If you do pass the correct parameters does it run as expected?

reproducing the error documenting along:
name': 'scan',             'parameter_schema': {'additionalProperties': False,                                  'properties': {'axes_to_move': {'additionalProperties': {'_axes_to_move': '<class '                                                                                "'bluesky.protocols.Movable'>"},                                                                  'title': 'Axes '                                                                           'To '                                                                           'Move',                                                                  'type': 'object'},                                                 'detectors': {'items': {'_detectors': '<class '                                                                                "'bluesky.protocols.Readable'>"},                                                               'title': 'Detectors',                                                               'type': 'array'},                                                 'metadata': {'title': 'Metadata',                                                              'type': 'object'},                                                 'spec': {'allOf': [{'type': 'string'}],                                                          'title': 'Spec'}},                                  'required': ['detectors',                                               'axes_to_move',                                               'spec'],                                  'title': 'scan',                                  'type': 'object'}},

are there plans that do not take any parameters?
   parameters = parameters or "{}"     task = RunPlan(name=name, params=json.loads(parameters))
cli.py lines 184-5. A quickfix would be to reject-return here if no parameters are supplied.
A more complex fix would be to check the list of plans to verify the right number of parameters - which would likely occur on the server.

also line 185 referenced above uses Task which is referenced in #253

are there plans that do not take any parameters?

There were when parameters was changed to be optional and it was annoying that they required passing  -p '{}'. It may be that most plans need args now but presumably you can always write ones that don't?

I'll start with doing this
https://github.com/DiamondLightSource/blueapi/issues/206#issuecomment-1969524593

added client-side translation of http status codes, addressing @callumforrester 's concern. now working on the server side

one possible refactor is changing the generic request to a named requests.get/post calls that can then be mocked more granularily. it's about this method in rest.py
 def _request_and_deserialize(         self,         suffix: str,         target_type: Type[T],         data: Optional[Mapping[str, Any]] = None,         method="GET",         raise_if: Callable[[requests.Response], bool] = _is_exception,     ) -> T:         url = self._url(suffix)         # todo consider changing this to a switch statement for easier mocking         # alternatively do single responsiblity principle         # and decoule request from deserializing.         response = requests.request(method, url, json=data)         if raise_if(response):             message = get_status_message(response.status_code)             t = response.text             error_message = f"""Response failed with text: {t},             with error code: {response.status_code}             which corresponds to {message}"""             raise BlueskyRemoteError(error_message)         deserialized = parse_obj_as(target_type, response.json())         return deserialized

hacky solution the resembles the current tests
`mock_requests.return_value = client.get(f"/plan/{plan}")
bunch of testing
mock_requests.return_value = client.post("/tasks")
bunch more testing`

merely trying to run blueapi serve with the config returns an error: NFO:     Application startup complete. ERROR:    [Errno -2] Name or service not known INFO:     Waiting for application shutdown.

still got a false negative erro in the test

`from pydantic import BaseModel, ValidationError, validator
from typing import Optional
class TimeSchema(BaseModel):
time: float
# Optional: Add a custom validator if needed
@validator('time')
def time_must_be_positive(cls, value):
    if value <= 0:
        raise ValueError('Time must be a positive number')
    return value

`
result - runtime validation won't work  just need to write a separate schema for each plan. which I fully support but that's another ticket. all plans and experiments could be standardized...
I mean we could add manual python validation against the 'properties' but that's not using pydantic as it was supposed to work

Hmm, yeah, I think separate ticket for custom per-parameter validation

how to mark this ticket then as pending and dependent on that ticket? I think gitlab has some better support for this kind of thing

example for future reference: `import json
from pydantic import parse_obj_as
from typing import Any, Optional
from pydantic import BaseModel, Field
class MyBaseModel(BaseModel):
name = "base"
class MyPlanModel(MyBaseModel):
name: str = "Plan model"
description: Optional[str] = Field(
description="Docstring of the plan", default=None
)
parameter_schema: Optional[dict[str, Any]] = Field(
description="Schema of the plan's parameters",
alias="schema",
default_factory=dict,
)
plan_model: MyPlanModel = MyPlanModel()
plan_model.parameter_schema = {
"title": "sleep",
"type": "object",
"properties": {"time": {"title": "Time", "type": "number"}},
"required": ["time"],
"additionalProperties": False,
}
print(plan_model.dict())
wrong_params: str = '{"wrong":"aboslutely wrong"}'
p = json.loads(wrong_params) or "{}"
try:
validated = parse_obj_as(type(plan_model.parameter_schema), p)
print(validated)
except ValueError as e:
print(f"Error: {e}")
validated = None
`

From discussion: Validation is already performed server-side and errors are shipped to the client here: https://github.com/DiamondLightSource/blueapi/blob/dba0f257c86f4a3be7d80e9efbb04c463ba52f33/src/blueapi/service/main.py#L152
All the information needed should be in resp here: https://github.com/DiamondLightSource/blueapi/blob/dba0f257c86f4a3be7d80e9efbb04c463ba52f33/src/blueapi/cli/cli.py#L187

todo:

[ ] add verbose error messages to the response


upload/download/move files via SFTP with dartssh

I know that this is not the ideal place to ask this question, but in the repository of the lib "dartssh" there is no way to create issues, so is it possible to upload/download/move files via SFTP with your dartssh lib?
https://github.com/TerminalStudio/dartssh

The dartssh library seems stop maintained and only implement limited functional. The SFTP support is no implemented. You can consider use another way to add sftp support.

Use Openssh executable and FFI to obtain SFTP support
Use bridge to wrap SSH library of different platfrom such as: https://pub.dev/packages/ssh2

Hoplly helpful: )


Adjust the Read & simulation process

Please adjust the read process as discussed. The process has to be capable of handeling different kinds of covariate labels from the inititalization file.
Note that this includes additional adjustments on the simulation process.
We want to get to the point where the labels, specified in the initialization file, are implemented as column labels in the simulated dataframe.

This is done.


[FLINK-4241] [table] Cryptic expression parser exceptions

Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

[x] General

The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message (including the JIRA id)

[x] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[x] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

This PR improves the error message of the ExpressionParser. I tried my best but it is not easy to improve the message as we are using RegexParsers.

+1 to merge


FIX: Clean warnings

This fixes warnings for me on Py3k and Python2 @rgommers.
I do have 18 failures if I compile with MKL (even though numpy has no errors), are those to be expected?

Thanks @Eric89GXL!
No failures are expected with MKL. Can you open a new issue for those?

Sure. I'm going to try updating my Parallel Studio / MKL libraries first (I think I'm using MKL from 2013), which means getting a new license from Intel. Hopefully that doesn't take too long.

This looks good to me. A few very minor cleanups in https://github.com/rgommers/scipy/commit/4861385bd020c0222daa6ad499d688a976af406f, can you take those over?
The one warning that I'm not sure needs investigating or not is that in ellip_harm. @pv what do you think?

@rgommers commit cherry-picked into my PR

Okay, I'll put this in and open a new issue to not lose track of the potential issue with ellip_harm. Thanks @Eric89GXL

Pushed this PR to master in c1fa391


[Packaging][Python] Update wheel dependency files

@github-actions crossbow submit -g wheel

Ping @xhochy for the record (you added tensorflow and keras-processing in https://github.com/apache/arrow/pull/3531).

The failures are related for artifact uploading, because of exceeded API rate limits.

@github-actions crossbow submit -g wheel

@github-actions crossbow submit wheel-win-cp36m


Update Kysely to version 0.17.1

There are a ton of huge improvements and shouldn't be any breaking changes

updated kysely to latest


issue-60

Now it is enabled the creation of subfolders

Thank you so much for your contribution @DarioTheLuca!
Close #60


Missing Azure info in table

There is a table in the "How it works" section that lists, per cloud app, which abilities are supported with App connectors.
However, it's missing Azure with no explanation of whether that is intentional

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

ID: a3cb69cd-d5d4-e058-e20f-d2697cfcd614
Version Independent ID: bb8fbd63-532a-f8f9-6bc6-85eaf09ca171
Content: Connect apps to get visibility and control
Content Source: CloudAppSecurityDocs/enable-instant-visibility-protection-and-governance-actions-for-your-apps.md
Service: cloud-app-security
GitHub Login: @shsagir
Microsoft Alias: shsagir

@thommck Thanks for contacting Microsoft with feedback about our product documentation. Depending on the complexity, it could take from a few business days to a few weeks to look further into your question, issue, or suggestion. We'll contact you if we need further information, and you'll receive an email message each time we "@mention" you in a comment.
Thanks again for submitting your feedback. Your suggestions help improve our documentation.

@thommck Thanks for your feedback. We've submitted a documentation change based on your suggestion, and the updated article should be live by tomorrow. Please let us know if there's anything more we can do for you. To follow up on this issue, leave us a comment with an @mention.


New version: BrokenRecord v0.1.5

Registering package: BrokenRecord
Repository: https://github.com/JuliaTesting/BrokenRecord.jl
Created by: @christopher-dG
Version: v0.1.5
Commit: 85fd70339ede17aec047152de34fbe848a69e696
Reviewed by: @christopher-dG
Reference: https://github.com/JuliaTesting/BrokenRecord.jl/commit/85fd70339ede17aec047152de34fbe848a69e696#commitcomment-44933420
Release notes:

Fixes a bug with an older version of YAML.jl.

block


Remove force-local-policy-eval-at-source flag

The flag is currently enabled by default and force policy evaluation at the source.
Now changes has been done to disable the flag by default,
Making use of existing "connectivity-check-proxy.yaml" tests
to test cilium connectivity test including L7 proxy tests.
Fixes: #15452
Signed-off-by: Venkata Reddy<EMAIL_ADDRESS>Please ensure your pull request adheres to the following guidelines:

[ ] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[ ] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Thanks for contributing!

Fixes: #issue-number
<!-- Enter the release note text here if needed or remove this section! -->

As per https://github.com/cilium/cilium/issues/15452, this is currently blocked on having proper L7 tests running on all major platforms. Marking as draft to pull out from review queues.

@pchaigno could you pls give reference test cases available for all platforms ike (GKE, AKS, EKS).
Also I feel connectivity and L7 proxy test cases already available.
connectivity-check-proxy.txt

I'm talking about the test performed by cilium connectivity test which we then run in all major platforms. See https://github.com/cilium/cilium-cli/ for details.

Should we write new L7 tests and run or  just run the existing L7 tests on all major platforms?
If we have to write new L7 tests and what that tests should do?

#158 added policy testing support, and #194 added ingress policy case. With these we have both ingress and egress L7 proxy connectivity tests.
cilium/cilium-cli#217

I think you meant https://github.com/cilium/cilium-cli/pull/158 and https://github.com/cilium/cilium-cli/pull/194.
That's correct, but cilium/cilium doesn't make use of those new tests yet.


feat: add styled-components package and example

BREAKING CHANGE: styled-components is now a peer dependency

Coverage decreased (-0.004%) to 89.815% when pulling 82509fc3d813e201b3c1b94e94f06c5d0ff1a85a on abutterworth/styled-components into 2e4dc9e5030542fcccd912b334c1f9b14e90f7dd on master.


achievements

What feature do you want to get added on the base engine?
Could we be able to add/edit achievements later update?
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

already possible at source

already possible at source

He means softcoded

already possible at source

He means softcoded

i think it can be updated, idk


[UI] Start with an empty configuration

Starting the editor with an empty JSON ({]) is probably the best option to not overwhelm the user with too many options.

I don't quite understand what you mean by an empty configuration. Currently, I have made it possible for the developer to instantiate the editor with json or leave it empty. there is also a method to change the json data in the editor as well as the schema
@nemesisdesign

An empty configuration is an empty NetJSON, which is just {}.
I think what you have done now is fine.


Add more flexibility to  RedisLock

Those improvement was raised by https://github.com/eclipse/leshan/issues/832 :

~Have WARN log if the lock expire before we try to unlock.~ (#837)
Make an interface about RedisLock to allow several implementation.
Timing of default implementation could be configurable.

This is now integrated in master. #843
(will be available in v1.1.0)


No juice when deleting a selection

+1

Doesn't seem to have quite so fabulous of explosions with CTRL+Backspace, either.

Fixed in Textreme 2.0


Latest NIST-800-53 shall refer to Revision 5 instead of Revision 4

This is wip by now. I recommend merging this no sooner than one month after submission.
Consult #14 to understand implications of merging this.

Nice, thanks!

I hear no opposition to this, @shawndwells.

Yahhhh it's been a few years. We should merge this and risk the breaking change. Not sure how many people use OpenControl (especially who do not define the versions).


[BACKLOG-10378] run dialog flashing on open

@mdamour1976 @bmorrise please review

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f ui/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f ui/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  ui/src/org/pentaho/di/ui/core/dialog/ConfigurationDialog.java


Sanity tests team3

Sanity test that verifies the home page/signin page of the Newsletter Generator app.

Closing PR since this is obsolete.


Migrate the kubernetes dashboard to a plugin.

With optional ingress object and secrets creation.
With autoscaling based on cluster size/nodes.
Upgrade dashboard to kubernetesui/dashboard:v2.0.0-beta4
Templating system improvements: -
Allow use of templating in pki keypairs common names, orgs and dnsNames.
Allow use of templating in plugins values sections.
Fix bugs scanning files for AWS CloudFormation Tags.

Codecov Report

Merging #1754 into master will decrease coverage by 0.04%.
The diff coverage is 14.28%.

@@            Coverage Diff             @@
##           master    #1754      +/-   ##
==========================================
- Coverage   25.16%   25.11%   -0.05%
==========================================
  Files          98       98
  Lines        5023     5032       +9
==========================================
  Hits         1264     1264
- Misses       3617     3626       +9
  Partials      142      142

Impacted Files
Coverage Δ

pkg/api/plugin_config.go
0% <0%> (ø)
:arrow_up:

tmpl/text_to_cfn_expr.go
86.66% <100%> (ø)
:arrow_up:

pkg/model/stack_new.go
34.21% <25%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


code style and missing semicolons in examples.

minor fixes to meet the code style as well as adding missing semicolons in examples.

lol, looks like there's a lot of broken docs surrounding Field methods on FormFields...

@dhensby, I was too quick and wrote the wrong class name on it. Sorry about this. By the way create_tag actually returns a string.  I fixed this.

Related #4321

@spekulatius closed? :(
My PR was merged so this can be amended now?


Make a dropdown list to play a different voice track everytime the button is clicked

DaddyLaugh
Adding a new feature to upgrade the horizon of dad laughs covered on our website.

Hello, please assign me to this issue.

assign me this issue i will make it 100% accurate


[BEAM-8337] publish Flink job server container images

This will make it easier to get started on Beam on Flink on Kubernetes.
cc @udim

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---

---
---

Java

Python

---

---
---

XLang
---
---
---

---
---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

cc @angoenka

Run Python Spark ValidatesRunner

Run Python PreCommit

I triggered Python Spark ValidatesRunner Tests by mistake.
Java failures are a known flake: BEAM-8035


Best score is saved to local storage

Best score is now displayed next to current score and stored into cookies. Fixes issue #6

LGTM!
Do you want to work on something new??

Sure, got any ideas?

Can you create Title screen for the game??? @mantevian

play should start the game class
Settings should have option to change theme and difficulty(which is being worked on other pr)
whenever user presses escape key the title screen is seen and game is paused.


What is exact difference between standard TFT_eSPI library and the one supplied with T-Display-S3

It is hard to replace the TFT_eSPI library again and again.  The one supplied with T-Display-S3 doesn't work with T-Embed.  Standard TFT_eSPI works fine with T-Embed.  I am using both products and its hard to replace the TFT_eSPI library each time i am using a different board.  How can we solve this?

This is messy for all of us. Maybe they should have two libraries with different names

I have submitted a pull request to TFT_eSPI. After waiting for the merge, it can be used along with the main line. You can directly use the TFT library in the T-DISPLAY-S3 repository before it is merged.

I understand the main library has been updated. I didn't see any instructions in the new library use. I will check the lib and assume there is one one more #defunes etc

Nice update and improvements by Bodmer with TFT_ESPI 2.50. Among other things, a new user select profile for LilyGo T Display S3 (170x320) was created with id 206. This works well on Arduino IDE 2.03 on Windows 10 for me. I also selected TFT Inversion ON.


luci-theme-bootstrap: fix overlay display failures

this fixes #5405 - you can verify these fixes by "inspect element" for the overlay elements, and modifying their CSS properties.
Fixed overlay display failures for users on mobile, or users who booted up in safe mode XD. Noticeable when vertical or horizontal real-estate is too small for the displayed overlay.
This failure is most evident when you "edit" an interface, or display your unsaved changes.
When content is wider than the screen view, the table hugs the (widest) content maximally, and the overlay window becomes scrollable.
Disabled -webkit-overflow-scrolling, left, and right, because both FF and Safari disable them as invalid anyway.
Signed-off-by: Paul Dee<EMAIL_ADDRESS>

NAK, this change causes scrolling and overflow issues on iOS.


Typo in the installation instruction

For installing scGPT, I guess you mean
pip install flash-attn==1.0.4
instead of 11.04

Thank you for catching this! :)


Use spec reporters in CI

Use the JUnit XML reporter from jasmine-reporters to produce test reports on CI builds on CircleCI and AppVeyor. This should expose test metadata that we can use to:

More quickly diagnose failures
Produce metadata that we can consume to identify flakes over time

One nice thing about Jasmine reporters vs. Mocha ones: Jasmine reporters are additive by default, so we still get console output and don't have to worry about tests timing out because it's only writing XML.
We will have to add support for respecting the TEST_JUNIT_XML_ROOT environment variable to all of our core packages that use custom runners.